Add a basic regression test that operates via checksums

Adds a shell script that tests for regressions in the given materials by exporting and then checksumming them. If even a single pixel changes after the script is rerun, the checksums will be different and you will be alerted.
Right now, the intended use case is manually testing for regressions when doing large invasive changes like mass refactors, and maybe basic sanity checking between version releases.
Also includes fixes to various problems with command line exporting that I encountered along the way.
Adding this into CI may be possible if software rasterization can guarantee consistent results across platforms, but is outside the scope of this PR.

Closing so I can rewrite this, sorry for the noise


Style/NumericPredicate does not honor IgnoredMethods.

Style/NumericPredicate does not appear to be honoring a list of IgnoredMethods.
# .rubocop.yml
Style/NumericPredicate:
  IgnoredMethods: ['zero?']

# test.rb
if foo == 0
  puts "bar"
end

$ rubocop --only Style/NumericPredicate ~/tmp/test.rb
Inspecting 1 file
C
Offenses:
test.rb:1:4: C: Style/NumericPredicate: Use foo.zero? instead of foo == 0.
if foo == 0
   ^^^^^^^^
1 file inspected, 1 offense detected

Nor does it honor IgnoredMethods with EnforcedStyle: comparison.
# .rubocop.yml
Style/NumericPredicate:
  EnforcedStyle: comparison
  IgnoredMethods: ['zero?']

# test.rb
if foo.zero?
  puts 'bar'
end

$ rubocop --only Style/NumericPredicate ~/tmp/test.rb
Inspecting 1 file
C
Offenses:
test.rb:1:4: C: Style/NumericPredicate: Use foo == 0 instead of foo.zero?.
if foo.zero?
   ^^^^^^^^^
1 file inspected, 1 offense detected

RuboCop version
0.78.0 (using Parser <IP_ADDRESS>, running on ruby 2.6.4 x86_64-darwin18)

Doing a little debugging, with EnforcedStyle: comparison the offense happens and then the ignored method is triggered.
This is what node looks like when the offense is triggered. It does not enter the node.each_ancestor block. ignored_method? is not called.
> node
=> s(:send,
  s(:send, nil, :foo), :zero?)

And then after when it does enter the block and ignored_method? is true.
> node
=> s(:send, nil, :foo)

With EnforcedStyle: predicate some extra logic will need to be added to check IgnoredMethods against the replacement so we can add exceptions to allow, for example, foo == 0 to be ok.

Thanks for the feedback. I open the PR #7596.

The following case is expected behavior. Because the method name (==) is not listed in IgnoredMethods.

Style/NumericPredicate does not appear to be honoring a list of IgnoredMethods.
# .rubocop.yml
Style/NumericPredicate:
   IgnoredMethods: ['zero?']

# test.rb
if foo == 0
   puts "bar"
end

$ rubocop --only Style/NumericPredicate ~/tmp/test.rb
Inspecting 1 file
C
Offenses:
test.rb:1:4: C: Style/NumericPredicate: Use foo.zero? instead of foo == 0.
if foo == 0
    ^^^^^^^^
1 file inspected, 1 offense detected

I think the following case is a bug. Because the method name (zero?) is listed in IgnoredMethods.

Nor does it honor IgnoredMethods with EnforcedStyle: comparison.
# .rubocop.yml
Style/NumericPredicate:
   EnforcedStyle: comparison
   IgnoredMethods: ['zero?']

# test.rb
if foo.zero?
   puts 'bar'
end

$ rubocop --only Style/NumericPredicate ~/tmp/test.rb
Inspecting 1 file
C
Offenses:
test.rb:1:4: C: Style/NumericPredicate: Use foo == 0 instead of foo.zero?.
if foo.zero?
    ^^^^^^^^^
1 file inspected, 1 offense detected

The PR #7596 is based on this decision.


(feature) log successful sign-ins

for both Azure and DFE Sign in

@tahb no problem, I was following the instructions on the trello card who specifically mentioned measuring success.
I am also going to move this card back and make the write out to a spreadsheet part of the PR so I can satisfy all requirements as part of this.


zohosprints-plugin permission file

HOSTING-894
Plugin repo : zohosprints
Description

Submitter checklist for changing permissions

Always

[ ] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory at least once

Maintainer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
There are IRC Bot commands for it

Please use the check boxes in the PR template to verify the required items are provided.

Exception in thread "main" java.lang.IllegalStateException: User name not known to Artifactory: zsprints

Please verify this is the username for the Jenkins infrastructure (Jira, Artifactory, etc) and NOT the github username.

Hi @slide  I am not able to change the permission file. Is there any special permission needs to edit the file. Kindly assist

@slide artefact json has been created in permission job. But i am unable to find plugin in artefact repo.
Please assist

Please use the developers mailing list for further assistance.


funcalign: Add RSRM compatibility with scikit-learn >=0.24

Starting with version 0.24, scikit-learn enforces the requirement to set parameters passed to estimators as instance attributes:

Fix base.BaseEstimator.get_params now will raise an AttributeError if a parameter cannot be retrieved as an instance attribute.
https://scikit-learn.org/stable/whats_new/v0.24.html#sklearn-base

RSRM has a gamma parameter, but it does not set a gamma attribute; instead it stores the value in the lam attribute:
https://github.com/brainiak/brainiak/blob/master/brainiak/funcalign/rsrm.py#L111
The resulting error shows up in our tests:
https://travis-ci.org/github/brainiak/brainiak/jobs/759097966#L866
Should we rename lam to gamma?

Yes, renaming it should be fine. Just created a PR


addEvent with only 2 parameters throws error in old Firefox

Hi!
On old versions of Firefox (reproduced with 4.0), the third parameter for addEventListener is required, otherwise an exception is thrown. Would you consider adding the third parameter in the addEvent calls (it will be ignored by attachEvent on IE anyway)?
addEvent(eventPrefix + 'resize', _watch, false);
addEvent(eventPrefix + 'orientationchange', _watch, false);

Sounds fine. Would you mind submitting a PR for the work?


[CL-adc]-Added a method for forcing adc_d_o signal.

Add a method in the chip interface for forcing and releasing adc_d_o signal.
Changed uvm_hdl_force with this method in a test vseq.
Because we do not use uvm_hdl_force in gate level environment and we needed a replacement.

hi
can you tell me please what's the error?
I don't have authorization to open the log file

hi can you tell me please what's the error? I don't have authorization to open the log file

Error-[DTINPCIL] Dynamic type in non-procedural context has 1 failures:

Test default has 1 failures.

default
Line 3150, in log /azp/agent/_work/1/s/scratch/HEAD/chip_earlgrey_asic-sim-vcs/default/build.log
  Error-[DTINPCIL] Dynamic type in non-procedural context
  ../src/lowrisc_dv_chip_env_0.1/chip_if.sv, 703
  "force adc_data_o[0] = channel_val[0];"
    Argument: channel_val[0]
    Automatic variable may not be used in non-procedural context.


Chef Automate not showing all skipped controls in compliance report tab

Describe the bug
Chef Automate not showing all skipped controls in compliance report tab
chef/customer-bugs#350
https://github.com/chef/customer-bugs/issues/431
To Reproduce
Steps to reproduce the behavior:
On few random nodes for profile sec_os_rhel6_cis profile control 6.2.10_Ensure_users_dot_files_are_not_group_or_world_writable is showing under critical controls however it should be showing under skipped controls. No of controls count showing correctly however, this control wrongly appearing under critical controls.
Note: Downloaded report showing correctly as expected.
Expected behavior
Control 6.2.10_Ensure_users_dot_files_are_not_group_or_world_writable status should appear correctly.
Screenshots
Versions (please complete the following information):
Additional context
Aha! Link: https://chef.aha.io/epics/SH-E-274

where can i find this profile?

Another bug related to this issue https://github.com/chef/customer-bugs/issues/665

Update: We have found an area where we can reproduce same issue. Now we are analysing the issue to find the root cause.

@kalroy Has this bug been triaged yet ? We have multiple customers affected by it and ANZ bank is one of them for whom it is becoming critical to have the issue fixed since it has been around for long.

@vkaushik-chef We tried reproducing the issue by creating a "custom profile and adding some dependent profiles inside it" (similar to the ANZ bank profile) but we are not able to reproduce the exact problem with that profile.
Therefore we have requested ANZ bank to provide their profile.

@vkaushik-chef These issues although look similar but are not always the same. These are not obvious bugs but can be because of some corner cases which @atultherajput  is trying to work out with ANZ. If we have multiple customers affected, I would request you to redirect us to the customer bugs/intake issues so that we can have conversations with them to understand the scenario. It can be also an issue with the way customer has created the profile too as in case of https://github.com/chef/customer-bugs/issues/388
Internal conversation not to be passed on to the customer

@vkaushik-chef We tried reproducing the issue by creating a "custom profile and adding some dependent profiles inside it" (similar to the ANZ bank profile) but we are not able to reproduce the exact problem with that profile.
Therefore we have requested ANZ bank to provide their profile.
CC: @kalroy

@atultherajput : The custom profile is actually an intellectual property of the ANZ bank and thus they can't share it. If there is more live sessions needed, please feel free to do so engaging with support team. Customer has informed the same on today's call.


bro，it seems that this cn.cqautotest.sunnybeach.http.glide.GlideApp, can you take a look

GlideApp could not be found in the project

你可以尝试 Build -> Clean Project -> Rebuild Project or Make Project，GlideApp 这个类是自动生成的

你可以尝试 Build -> Clean Project -> Rebuild Project or Make Project，GlideApp 这个类是自动生成的

thank you very muck！


IOS Supply fail to upload to appstore/testflight - Transporter transfer failed

New Issue Checklist

[ ] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
uploading ipa with testflight action fails after 8 min wait:
[11:52:51]: Going to upload updated app to App Store Connect
[11:52:51]: This might take a few minutes. Please don't interrupt the script.
[12:00:17]: [Transporter Error Output]: Could not start delivery: all transports failed diagnostics
[12:00:17]: Transporter transfer failed.
[12:00:17]:
[12:00:17]: Could not start delivery: all transports failed diagnostics
[12:00:18]: [iTMSTransporter]   at com.signiant.interactivetransfer.engine.TransferEngine$8.run(TransferEngine.java:4060)

[12:00:18]: [iTMSTransporter]

[12:00:18]: [iTMSTransporter] [2018-12-20 12:00:17 IST] <Transport Protocol> DEBUG: Received failed transfer notification; aborting

[12:00:18]: [iTMSTransporter] [2018-12-20 12:00:17 IST] <Transport Protocol>  INFO: The Signiant transfer engine's status is DISCONNECTED

[12:00:18]: [iTMSTransporter] [2018-12-20 12:00:17 IST] <Transport Protocol>  INFO: The Signiant transfer engine's status is DISCONNECTED

[12:00:18]: [iTMSTransporter] [2018-12-20 12:00:17 IST] <main> ERROR: Could not start delivery: all transports failed diagnostics

[12:00:18]: [iTMSTransporter]

[12:00:18]: [iTMSTransporter]

[12:00:18]: [iTMSTransporter]

[12:00:18]: [iTMSTransporter] Package Summary:

[12:00:18]: [iTMSTransporter]

[12:00:18]: [iTMSTransporter] 1 package(s) were not uploaded because they had problems:

[12:00:18]: [iTMSTransporter]   /var/folders/n8/lxck8kbx0lg4kfbcf27qkbg40000gn/T/d20181220-36585-19oebu8/1327728378.itmsp - Error Messages:

[12:00:18]: [iTMSTransporter]           Could not start delivery: all transports failed diagnostics

[12:00:18]: [iTMSTransporter] [2018-12-20 12:00:17 IST] <main> DBG-X: Returning 1

[12:00:18]: iTunes Transporter output above ^
[12:00:18]: Could not start delivery: all transports failed diagnostics
Return status of iTunes Transporter was 1: Could not start delivery: all transports failed diagnostics
The call to the iTMSTransporter completed with a non-zero exit status: 1. This indicates a failure.

Complete output when running fastlane, including the stack trace and command used

[INSERT OUTPUT HERE]

Environment

🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.14

Ruby
2.2.4

Bundler?
false

Git
git version 2.17.2 (Apple Git-113)

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.14 (18A391)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
false

Is homebrew
true

Is installed via Fabric.app
false

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

Xcode Version
10.1

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE
en_US.UTF-8
✅

fastlane files:
`./Fastfile`
# Customise this file, documentation can be found here:
# https://github.com/fastlane/fastlane/tree/master/fastlane/docs
# All available actions: https://docs.fastlane.tools/actions
# 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 "2.59.0"

default_platform :ios

platform :ios do
  lane :beta do |options|

  # build your iOS
  gym(
    scheme: "Farmer",
  )

  if options[:tag]
    add_git_tag(
      tag: options[:tag]
    )
    end
    if options[:dropbox_path]
      puts "ipa has been saved to your Dropbox path: "+options[:dropbox_path]+"/QA/ipas/"+options[:tag]
        copy_artifacts(
          target_path: options[:dropbox_path]+"/QA/apks/"+options[:tag],
          artifacts: ["*.ipa"]
        )
    end

  if options[:push]
       push_to_git_remote(
          local_branch: git_branch
       )
    end

  if options[:upload]
    # upload to testFlight
    testflight(distribute_external: false, skip_submission: true, skip_waiting_for_build_processing: true)
    # if set  skip_waiting_for_build_processing: true // cannot distribute_external.
    #  To show the documentation in your terminal, run:
    # $ fastlane action testflight

    # upload bugsnag sourcemap
    sh "../../bugsnag-mapsource-uploader-ios.sh"
  end
  end
end

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

team_id "[team_id]" # Developer Portal Team ID
itc_team_name "[team_name]"

# 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

fastlane
2.101.1
🚫 Update available

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
2.3.2

atomos
0.1.3

CFPropertyList
2.3.6

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.6

xcodeproj
1.5.9

rouge
2.0.7

xcpretty
0.2.8

terminal-notifier
1.8.0

unicode-display_width
1.1.3

terminal-table
1.7.3

plist
3.4.0

public_suffix
2.0.5

addressable
2.5.2

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.6.4

tty-cursor
0.5.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

excon
0.60.0

faraday
0.13.1

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.1

gh_inspector
1.1.2

json
1.8.1

mini_magick
4.5.1

multi_json
1.12.2

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.2.1

bundler
1.14.6

faraday_middleware
<IP_ADDRESS>

naturally
2.2.0

simctl
1.6.5

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.1

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

logging
2.2.2

jwt
1.5.6

memoist
0.16.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
2.8.3

google-api-client
0.23.4

emoji_regex
0.1.1

libxml-ruby
3.0.0

generated on: 2018-12-20

when uploading with xcode it takes a long time but success

Hi! This could mean lots of things, but as it happened to me a while ago, please check that the signing profiles are still active in the Developer Portal, as they can be valid in the local machine but unable to be verified by Apple in the upload process.

@minuscorp hi, thanx
i can upload ipa manually and sign and export but fail with fastlate only

Have you checked that fastlane is using the same conde signing identities that you use to archive manually the ipa?

You should update your fastlane to a recent version - your are using 2.101.1 which is outdated.
supply is our tool "for updating Android apps and their metadata on the Google Play Store" - so this can't be right. Are you using deliver?
If you are, have you tried this: https://docs.fastlane.tools/actions/deliver/#firewall-issues?

My mistake
I ment testflight action as you can see in fastfile attached above
Will updating fastlane realy make the difference?

The same what I mentioned in 3 applies to Testflight a well: https://docs.fastlane.tools/actions/testflight/#firewall-issues

Will updating fastlane realy make the difference?

Possibly. We release new versions of the software when something breaks. Investing time to debug and understand an issue is a waste of time, if it could possibly be resolved by just using the newest version of fastlane. Please update and report back if the problem persists.


[WIP] :sparkles: supporting cloudevents for work agent

Summary
Related issue(s)
Fixes #

/assign @qiujian16

/assign @clyang82

How to test the MQTT protocol here?

How to test the MQTT protocol here?

I'm not sure, the work client with MQTT is already tested in the api repo, but for work agent, now we don't have a server supporting this, currently, we may not be able to start the e2e test in the ocm, after the maestro server is ready, we may use it to do the e2e test in the ocm?

Do we also need to add cloudevents support to the singleton agent ? https://github.com/morvencao/OCM/tree/main/pkg/singleton/spoke

the singleton agent calls the work agent code, so this support will be also in the singleton agent

TODO, instead of using flags. It will be better to have a config file for mqtt setup. We should only have one flag to specify driver for the work agent

TODO, instead of using flags. It will be better to have a config file for mqtt setup. We should only have one flag to specify driver for the work agent

good suggestion, add the TODO to the code firstly :)

/hold

/unhold


[CERT-TEST-FAILURE]- TC-ACFREMON-1.1 - Test failed

Feature Area
Other
Test Case
TC-ACFREMON-1.1
Reproduction steps

Bring up the DUT with all-cluster-app.
Pair the DUT using chip-tool through "onnetwork" transport.
Run the command ./chip-tool activatedcarbonfiltermonitoring read feature-map 1 1

Expected Outcome:

[1690181499.378839][8062:8064] CHIP:DMG: }
[1690181499.379113][8062:8064] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0071 Attribute 0x0000_FFFC DataVersion:<PHONE_NUMBER>
[1690181499.379203][8062:8064] CHIP:TOO:   FeatureMap: 7
[1690181499.379409][8062:8064] CHIP:EM: <<< [E:11917i S:34360 M:35635505 (Ack:251547236)] (S) Msg TX to 1:0000000000000001 [5C90] --- Type 0000:10 (SecureChannel:StandaloneAck)

Actual Outcome:

[1690181499.378839][8062:8064] CHIP:DMG: }
[1690181499.379113][8062:8064] CHIP:TOO: Endpoint: 1 Cluster: 0x0000_0071 Attribute 0x0000_FFFC DataVersion:<PHONE_NUMBER>
[1690181499.379203][8062:8064] CHIP:TOO:   FeatureMap: 3
[1690181499.379409][8062:8064] CHIP:EM: <<< [E:11917i S:34360 M:35635505 (Ack:251547236)] (S) Msg TX to 1:0000000000000001 [5C90] --- Type 0000:10 (SecureChannel:StandaloneAck)

Bug prevalence
Whenever I try to read FeatureMap.
GitHub hash of the SDK that was being used
c1ec2d777456924dcaa59b53351b00d73caf378f
Platform
raspi
Anything else?
As per the Replacement Product List feature implementation, the FeatureMap attribute value should be 7. But currently getting FeatureMap value as 3 in SDK.

@cliffamzn Could you take a look regarding feature map flags here?


fix: change date column to timestamp in time_series_v2

Part of https://github.com/SigNoz/engineering-pod/issues/484
This change is made for the use of prometheus reader in the query service. This timestamp is used load the delta row from the table since the last time.

cc @mindhash


Update date_picker_theme.dart

Please merge as soon as possible!!

Please merge as soon as possible!!

Now the version after 1.20 is not available. The temporary solution is to modify the source code and change "DiagnosticableMixin" to "DiagnosticableTreeMixin" so that both the new version and the old version are available. I don’t know if this modification is correct, I hope to update the version as soon as possible (some people say it can be changed to "Diagnosticable", but this only solves the problem of the new version of the SDK, the old version of the SDK is not compatible)


[Bug]: build fail not able to run the project in macos 1 warning generated. /Users/trentiumsolution/.pub-cache/hosted/pub.dev/geolocator_apple-2.2.7/macos/Classes/Handlers/GeolocationHandler.m:63:22: warning: unused variable 'locationManager' [-Wunused-variable]   CLLocationManager *locationManager = [self getLocationManager];                      ^ 1 warning generated. 1 error generated. ** BUILD FAILED **

Please check the following before submitting a new issue.

[ ] I have searched the existing issues.
[X] I have carefully read the documentation and verified I have added the required platform specific configuration.

Please select affected platform(s)

[ ] Android
[ ] iOS
[ ] Linux
[X] macOS
[ ] Web
[ ] Windows

Steps to reproduce
1 warning generated.
/Users/trentiumsolution/.pub-cache/hosted/pub.dev/geolocator_apple-2.2.7/macos/Classes/Handlers/GeolocationHandler.m:63:22: warning: unused variable 'locationManager' [-Wunused-variable]
CLLocationManager *locationManager = [self getLocationManager];
^
1 warning generated.
1 error generated.
** BUILD FAILED **
Expected results
getting this error
Launching lib/main.dart on macOS in debug mode...
Running pod install...
Building macOS application...
/Users/trentiumsolution/.pub-cache/hosted/pub.dev/geolocator_apple-2.2.7/macos/Classes/Utils/LocationMapper.m:36:43: warning: 'sourceInformation' is only available on macOS 12.0 or newer [-Wunguarded-availability-new]
[locationMap setObject:@(location.sourceInformation.isSimulatedBySoftware) forKey: @"is_mocked"];
^~~~~~~~~~~~~~~~~
In module 'CoreLocation' imported from /Users/trentiumsolution/.pub-cache/hosted/pub.dev/geolocator_apple-2.2.7/macos/Classes/Utils/LocationMapper.h:9:
@property(readonly, nonatomic, nullable) CLLocationSourceInformation *sourceInformation API_AVAILABLE(ios(15.0), watchos(8.0), tvos(15.0), macos(12.0));
^
[locationMap setObject:@(location.sourceInformation.isSimulatedBySoftware) forKey: @"is_mocked"];
^~~~~~~~~~~~~~~~~
1 warning generated.
/Users/trentiumsolution/.pub-cache/hosted/pub.dev/geolocator_apple-2.2.7/macos/Classes/Handlers/GeolocationHandler.m:63:22: warning: unused variable 'locationManager' [-Wunused-variable]
CLLocationManager *locationManager = [self getLocationManager];
^
1 warning generated.
1 error generated.
** BUILD FAILED **
Exception: Build process failed
Actual results
not able to run the macos build
Code sample
Code sample
[Paste your code here]

1 warning generated.
/Users/trentiumsolution/.pub-cache/hosted/pub.dev/geolocator_apple-2.2.7/macos/Classes/Handlers/GeolocationHandler.m:63:22: warning: unused variable 'locationManager' [-Wunused-variable]
CLLocationManager *locationManager = [self getLocationManager];
^
1 warning generated.
1 error generated.
** BUILD FAILED **

Screenshots or video

Screenshots or video demonstration
[Upload media here]

1 warning generated.
/Users/trentiumsolution/.pub-cache/hosted/pub.dev/geolocator_apple-2.2.7/macos/Classes/Handlers/GeolocationHandler.m:63:22: warning: unused variable 'locationManager' [-Wunused-variable]
  CLLocationManager *locationManager = [self getLocationManager];
                     ^
1 warning generated.
1 error generated.
** BUILD FAILED **
Version
geolocator: ^10.0.1
Flutter Doctor output
Doctor output
[Paste your output here]

[✓] Flutter (Channel stable, 3.13.5, on macOS 12.1 21C52 darwin-x64, locale
en-GB)
• Flutter version 3.13.5 on channel stable at
/Users/trentiumsolution/development/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 12fccda598 (3 days ago), 2023-09-19 13:56:11 -0700
• Engine revision bd986c5ed2
• Dart version 3.1.2
• DevTools version 2.25.0
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
• Android SDK at /Users/trentiumsolution/Library/Android/sdk
• Platform android-33, build-tools 33.0.2
• Java binary at: /Applications/Android
Studio.app/Contents/jbr/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build
17.0.6+0-17.0.6b802.4-9586694)
• All Android licenses accepted.
[!] Xcode - develop for iOS and macOS (Xcode 13.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 13F100
✗ Flutter requires Xcode 14 or higher.
Download the latest version or update via the Mac App Store.
• CocoaPods version 1.12.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2022.2)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build
17.0.6+0-17.0.6b802.4-9586694)
[✓] VS Code (version 1.77.3)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension can be installed from:
🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-x64     • macOS 12.1 21C52 darwin-x64
• Chrome (web)    • chrome • web-javascript • Google Chrome 116.0.5845.187
[✓] Network resources
• All expected network resources are available.

Hi @sanket2798, thank you for filing this issue!
Fix should be coming in shortly.

The issue has been resolved. You can pull in the latest version of the apple dependency by running dart pub upgrade. Your pubspec.lock file should contain geolocator_apple...version: "2.3.2".


Helmet changes after CDN host fix

…ad - should fix any cidev environment issues
JIRA link
Change description
Work checklist

[ ] Tests added where applicable
[ ] UI changes meet accessibility criteria

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

Closing as new branch was made for these changes: https://github.com/companieshouse/overseas-entities-web/pull/1666


Markers on map show locations names instead of resource title

Comment from user group.
Use case:
Point-details – If the user clicks on one of the indicators then a popup displays a summary of the metadata (equal to list-view) and a link to the preview-page of the entry.
The markers show resources and not locations, which makes it more reasonable to show resource title and perhaps some more metadata. Link should possibly go to the resource and not to a new search.

The markers intentionally represent places and not resources since sometimes there is more than one resource connected to a place (e.g. http://ariadne-portal.dcu.gr/search?q=spatial.location.lon:"0.889007" AND spatial.location.lat:"51.13811").
We could however try to cover both cases and directly link to the resource if there is only one.

Another possible solution would be to implement clustering of markers representing single resource. Several markers at the same position on the map could be visualized as a stack of markers that can be extended by clicking on it (similar to Google Earth).


[WIP] Spotify artist URI to social pages, Wiki summary

Since seeing this feature on the roadmap, I thought I'd get the initial architecture in place to make sure this might be the best/most effective string of API calls to make.
Right now it takes two calls to reach social links, four to reach a Wiki summary:

MusicBrainz url lookup:

Takes Spotify artist URL
Returns MusicBrains ID

MusicBrainz artist social links:

Takes MusicBrainz ID
Returns social links (ie. Wikidata, YouTube, SoundCloud, Facebook, Discogs, official pages, etc.)

Wikidata to Wikipedia page:

Takes Wikidata ID
Returns Wikipedia pages by language

Fetch summary:

Takes Wikipedia page
Returns summary of artist page

I feel like it should be possible to reach the MusicBrainz social links with one call, but their API docs are a bit of a mess. The total call time with reqwests is around 3-4s.

How are you getting the musicbrainz ID from a Spotify URL? Is there some API to do that that I've missed?

The MusicBrainz ID is retrieved by looking up which artist has the Spotify URL in their metadata on MusicBrainz.

Excellent, thank you!

That sounds great @jacksongoode!

A proof of concept using ureq can be checked out here. First time working with Rust so it might take me a while to integrate :)

Heya, spotify does this and I am probably able to use the same API request format as I did with getting the lyrics for the songs as it's the same background api provider.

Closed with #537


updating encoding.h, added test for mconfigptr

Description
Add read-only CSR MCONFIGPTR test (1 read/1 write).
Related Issues
Requires sail update for signatures to pass: https://github.com/riscv/sail-riscv/pull/293
Ratified/Unratified Extensions

[X] Ratified
[ ] Unratified

List Extensions
MCONFIGPTR exists in the priv spec 1.12 and used by the discovery task group.
Reference Model Used

[x] SAIL
[X] Spike
[ ] Other - < SPECIFY HERE >

Mandatory Checklist:

[ X] All tests are compliant with the test-format spec present in this repo ?
[ X] Ran the new tests on RISCOF with SAIL/Spike as reference model successfully ?
requires https://github.com/riscv/sail-riscv/pull/293 and reference this issue to workaround SAIL compile issue: https://github.com/riscv/sail-riscv/issues/292
[ ] Ran the new tests on RISCOF in coverage mode
[ ] Link to Google-Drive folder containing the new coverage reports (See this for more info): < SPECIFY HERE >
[ ] Link to PR in RISCV-ISAC from which the reports were generated : < SPECIFY HERE >
[ ] Changelog entry created with a minor patch

Optional Checklist:

[ ] RISCV-V CTG PR link if tests were generated using it : < SPECIFY HERE >
[ ] Were the tests hand-written/modified ?
[ ] Have you run these on any hard DUT model ? Please specify name and provide link if possible in the description
[ ] If you have modified arch_test.h Please provide a detailed description of the changes in the Description section above.

closing.  Not sure how to have more than one fork at a time.  since these are technical debt, no rush.  will open a new pull for smcsrind.


resolveAPIReviewComments

Added  @distributed_trace for read_feed_ranges
Changed force_refresh parameter from optional[bool] to bool
Removed Async from read_feed_ranges in aio package

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-cosmos

/azp run python - cosmos - tests

/azp run python - cosmos - tests

/azp run python - cosmos - tests

/check-enforcer override

Pipeline cancelled/failed not due to change in this PR


⚠ Fortnite has degraded performance

In d0a3edc, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1535 ms

Resolved: Fortnite performance has improved in 6103972.


Allow animation delays

It would be great if we could specify an animation delay in the options of the animate function (similar to the transition-delay property from CSS). This way we can avoid using setTimeout just for achieving this.
    this.view.animate('something', val, {
               duration: 250,
               delay: 100 //<----
       }

I think a composite transition is the best solution to this too, so I'm going to forward it to #987


ghapi.page.paged does not respect rate limits

Calling it almost always results in HTTP403ForbiddenError: HTTP Error 403: Forbidden
for result in ghapi.page.paged(gh_api.search.code, q=f"filename:component.yaml repo:Ark-kun/pipeline_components"):
    print(len(result["items"]))

opt/conda/lib/python3.7/site-packages/ghapi/core.py in __call__(self, path, verb, headers, route, query, data)
    107             for k,v in route.items(): route[k] = quote(str(route[k]))
    108         res,self.recv_hdrs = urlsend(path, verb, headers=headers or None, debug=self.debug, return_headers=True,
--> 109                                      route=route or None, query=query or None, data=data or None)
    110         if 'X-RateLimit-Remaining' in self.recv_hdrs:
    111             newlim = self.recv_hdrs['X-RateLimit-Remaining']

/opt/conda/lib/python3.7/site-packages/fastcore/net.py in urlsend(url, verb, headers, route, query, data, json_data, return_json, return_headers, debug)
    210         return urlread(req, decode=False, return_json=False, return_headers=return_headers)
    211
--> 212     return urlread(req, return_json=return_json, return_headers=return_headers)
    213
    214 # Cell

/opt/conda/lib/python3.7/site-packages/fastcore/net.py in urlread(url, data, headers, decode, return_json, return_headers, timeout, **kwargs)
    111         with urlopen(url, data=data, headers=headers, timeout=timeout, **kwargs) as u: res,hdrs = u.read(),u.headers
    112     except HTTPError as e:
--> 113         if 400 <= e.code < 500: raise ExceptionsHTTP[e.code](e.url, e.hdrs, e.fp) from None
    114         else: raise
    115

HTTP403ForbiddenError: HTTP Error 403: Forbidden

I also am getting lots of secondary rate limits. It would be great if at least I could access the Retry-After header that github sets. Otherwise I just need to guess.


Test case fix: 200 => 300

FIx: no mock aws-sdk
Fix: change lamba function name
Fix: miss input params
Fix: deploy test
Fix: Minified handlers not build
https://app.asana.com/0/1198213092647463/1200109614369250

To 650 after update.


The file size limit is 5 MB is incorrect.

Hi,
After multiple times of testing, we can only upload the customized voice message file of MFA size between 1B - 1MB.  If the size file larger than 1 MB, error message: "file must be between 1 and 1048576 bytes".
The information on this page " The file size limit is 5 MB" is incorrect.
Regards,
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 84b26750-880c-8f14-1628-436293de915e
Version Independent ID: 518b727f-79e8-b5c2-bcb3-ff4de3444c6f
Content: Configure Azure Multi-Factor Authentication - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-mfasettings.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@elening2018 Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

#please-close resolved in private PR


Update the peer dependency on @cycle/core

I'm not sure if 3 -> 5 had breaking changes for this module, or I would have sent a PR.

Would you be willing to try changing the peer dependency to rx and updating the code to import Rx from 'rx'?

Sure. I'm a bit apprehensive, however, because I don't seem to be able to test:
$ git clone git@github.com:jessaustin/cycle-notification-driver.git
  ...
$ cd cycle-notification-driver
$ npm install
  ...
$ npm test
><EMAIL_ADDRESS>test /home/jess/dev/cycle-notification-driver
> mocha --compilers js:babel/register test

  0 passing (4ms)

I'm sure there's something wrong with my environment (probably a missing global module), but I'm not a mocha expert and I don't see a way forward here. I can skip testing, but I'm not sure how helpful that would be for you.

This should be fixed in the 2.0.0 release as addressed in #8.


Create pull request (PR) to devops repository to change revproxy configuration

Value Statement
As a platform admin
I want to receive delivery statuses from Granicus
So that I know the status of the notifications the VA Notify platform is sending via Granicus
Acceptance Criteria
GIVEN a URL dev-api.va.gov/vanotify/*
WHEN a request is made
THEN the request is handled by dev VANotify service
Acceptance Criteria
GIVEN a URL staging-api.va.gov/vanotify/*
WHEN a request is made
THEN the request is handled by staging VANotify service
Acceptance Criteria
GIVEN a URL api.va.gov/vanotify/*
WHEN a request is made
THEN the request is handled by production VANotify service
Checklist

[ ]Pull request with rev proxy confirmation changes

Assumptions

This PR will have to be approved by VSP operations team.

Additional Info/Resources

That configuration will allow us to route traffic from api.va.go/vanotify (as an example) to our own load balancer and service, which will in turn allow us to configure webhooks with Granicus.

Out of Scope

Open Questions

[ ] Will we need to make any changes to our load balancer?

Blocked while waiting for security review

Dev & Staging:
https://github.com/department-of-veterans-affairs/devops/pull/8465
Production PR:
https://github.com/department-of-veterans-affairs/devops/pull/8643


Offer a --set-exit-if-changed option

Like dart format so that users can add it to their continuous integration and be alerted if the layer changed but the diagrams were not updated.

You can make GitHub re-generating the diagrams after every push, by copying content of run-layerlens.yaml to .github/workflows.
Will it solve your problem or you want to guarantee that the diagrams are up-to-date during code reviews?
What package are you thinking to apply it to?

Oh, that solves my use case perfectly, thanks!
(I didn't know that you could modify content in a github workflow)


add property-based tests for audio

This PR integrates fast-check property-based testing into the existing test suite to ensure that the spy calls on the fetcher function match the expected API calls. The tests now cover multiple scenarios and edge cases, dynamically generating inputs to validate the robustness of the API interaction.
Key changes include:

Mocking the fetcher function using vitest to intercept and assert its calls
Utilizing fast-check to create a wide range of inputs, including URLs and options
Implementing an asyncProperty test to asynchronously verify the behavior of the API calling function under various generated conditions
Clearing the mock and setting up a resolved value before each property test to isolate the test cases

By employing property-based testing, we aim to catch more bugs by checking the correctness of the code against a broader set of possible inputs. This should lead to a more reliable and maintainable codebase.

I cannot check why the deployment failed... EVen though I have an account, Vercel prevents me from logging in...

@sharabash @osamasayed could you please help on this ?

@ahmedriad1 any issue on this ? anything I could do to unblock it ?

No response, so I'm closing this abandoned PR.


Adds dataDir to ucr bridge cni configuration

High-level description
Currently, host-local ipam cni plugin uses a persistent location
on the disk as its data dir where it keeps the information about
allocated IP addresses. This leads to orphan IP addresses if
the agent reboots. This patch moves the data dir to a tmpfs location
so that all the allocated IP addresses are recycled upon agent reboot.
Corresponding DC/OS tickets (obligatory)
These DC/OS JIRA ticket(s) must be updated (ideally closed) in the moment this PR lands:

DCOS_OSS-3750 CNI module should store configuration on a tmpfs dir

Checklist for all PRs

[ ] Added a comprehensible changelog entry to CHANGES.md or explain why this is not a user-facing change:
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain here:
[ ] Read the DC/OS contributing guidelines
[ ] Followed relevant code rules Rules for Packages and Systemd

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

PLEASE FILL IN THE TEMPLATE ABOVE / DO NOT REMOVE ANY SECTIONS ABOVE THIS LINE
Instructions and review process
What is the review process and when will my changes land?
All PRs require 2 approvals using GitHub's pull request reviews.
Reviewers should be:

Developers who understand the code being modified.
Developers responsible for code that interacts with or depends on the code being modified.

It is best to proactively ask for 2 reviews by @mentioning the candidate reviewers in the PR comments area. The responsibility is on the developer submitting the PR to follow-up with reviewers and make sure a PR is reviewed in a timely manner. Once a PR has 2 ship-it's, no red reviews, and all tests are green it will be included in the next train.

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

@urbanserj @jieyu  request for review

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee


Error: 400-InvalidParameter, Compartment {ocid1.compartment.oc1....} does not exist or is not part of the policy compartment subtree

Hi,
I am trying to deploy the OELZ version  https://github.com/oracle-quickstart/oci-landing-zones/releases/tag/v2.1.2
 with Resouce Manager.
It failed during the "module.prod_environment.module.workload.module.workload_osms_dg_policy.oci_identity_policy.policy," creation.
Below is the plan and error log.
PLAN:
module.prod_environment.module.workload.module.workload_osms_dg_policy.oci_identity_policy.policy will be created

resource "oci_identity_policy" "policy" {

ETag           = (known after apply)
compartment_id = "ocid1.compartment.oc1..aaaaaaaawmbt6l4wgvf6zdwchpfds3pni2egwt5cujfxshgy4ulax2rb4dcq"
defined_tags   = (known after apply)
description    = "Workload OCI Landing Zone OS Management Service Dynamic Group Policy"
freeform_tags  = (known after apply)
id             = (known after apply)
inactive_state = (known after apply)
lastUpdateETag = (known after apply)
name           = "OCI-P-ELZ-Workload1-OSMS-DG-Policy"
policyHash     = (known after apply)
state          = (known after apply)
statements     = [

"Allow dynamic-group OCI-P-ELZ-Workload1-DG to read instance-family in compartment ocid1.compartment.oc1..aaaaaaaawmbt6l4wgvf6zdwchpfds3pni2egwt5cujfxshgy4ulax2rb4dcq",
"Allow dynamic-group OCI-P-ELZ-Workload1-DG to use osms-managed-instances in compartment ocid1.compartment.oc1..aaaaaaaawmbt6l4wgvf6zdwchpfds3pni2egwt5cujfxshgy4ulax2rb4dcq",
]

time_created   = (known after apply)
version_date   = (known after apply)
}

Error log:
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Error: 400-InvalidParameter, Compartment {ocid1.compartment.oc1..aaaaaaaaacxbinb6bzfo36g4i55radlsepp2pb53aekcx6x3cqoj72udt65q} does not exist or is not part of the policy compartment subtree
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Suggestion: Please update the parameter(s) in the Terraform config as per error message Compartment {ocid1.compartment.oc1..aaaaaaaaacxbinb6bzfo36g4i55radlsepp2pb53aekcx6x3cqoj72udt65q} does not exist or is not part of the policy compartment subtree
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Documentation: https://registry.terraform.io/providers/oracle/oci/latest/docs/resources/identity_policy
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] API Reference: https://docs.oracle.com/iaas/api/#/en/identity/20160918/Policy/CreatePolicy
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Request Target: POST https://identity.eu-milan-1.oci.oraclecloud.com/20160918/policies
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Provider version: 5.1.0, released on 2023-06-13. This provider is 13 Update(s) behind to current.
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Service: Identity Policy
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Operation Name: CreatePolicy
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] OPC request ID: 534cea82b65ffa33a5559b4898870d0b/FFDE662416A3FDCFE7F6BC31689BCF76/F0F0C07537FCD17CDFE25B91C0522329
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]   with module.nonprod_environment.module.workload.module.workload_osms_dg_policy.oci_identity_policy.policy,
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]   on ../../modules/policies/main.tf line 9, in resource "oci_identity_policy" "policy":
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]    9: resource "oci_identity_policy" "policy" {
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Error: 400-InvalidParameter, Compartment {ocid1.compartment.oc1..aaaaaaaawmbt6l4wgvf6zdwchpfds3pni2egwt5cujfxshgy4ulax2rb4dcq} does not exist or is not part of the policy compartment subtree
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Suggestion: Please update the parameter(s) in the Terraform config as per error message Compartment {ocid1.compartment.oc1..aaaaaaaawmbt6l4wgvf6zdwchpfds3pni2egwt5cujfxshgy4ulax2rb4dcq} does not exist or is not part of the policy compartment subtree
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Documentation: https://registry.terraform.io/providers/oracle/oci/latest/docs/resources/identity_policy
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] API Reference: https://docs.oracle.com/iaas/api/#/en/identity/20160918/Policy/CreatePolicy
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Request Target: POST https://identity.eu-milan-1.oci.oraclecloud.com/20160918/policies
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Provider version: 5.1.0, released on 2023-06-13. This provider is 13 Update(s) behind to current.
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Service: Identity Policy
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] Operation Name: CreatePolicy
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO] OPC request ID: bd820545a1472ab2b697394e0091ab98/C2CFD28C6FCD5C5381D4E0FE17109682/F3A8FC75899AB9D4307E7A9AB7300706
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]   with module.prod_environment.module.workload.module.workload_osms_dg_policy.oci_identity_policy.policy,
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]   on ../../modules/policies/main.tf line 9, in resource "oci_identity_policy" "policy":
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]    9: resource "oci_identity_policy" "policy" {
2023/09/15 09:25:30[TERRAFORM_CONSOLE] [INFO]
2023/09/15 09:25:31[TERRAFORM_CONSOLE] [INFO]

The statements of the policy are wrong. The correct ones are:
statements = [
"Allow dynamic-group OCI-P-ELZ-Workload1-DG to read instance-family in compartment in ocid1.compartment.oc1..aaaaaaaawmbt6l4wgvf6zdwchpfds3pni2egwt5cujfxshgy4ulax2rb4dcq",
"Allow dynamic-group OCI-P-ELZ-Workload1-DG to use osms-managed-instances in compartment in ocid1.compartment.oc1..aaaaaaaawmbt6l4wgvf6zdwchpfds3pni2egwt5cujfxshgy4ulax2rb4dcq",
]
The correct syntax is explained here: https://docs.oracle.com/en-us/iaas/Content/Identity/Concepts/policysyntax.htm#:~:text=To specify a compartment by OCID
Needs to be changed the policy statement in the file "security.tf" of the "elz-workload", specifically the "local.osms_dg_policy_workload.statements"

Fixed the issue on latest release v2.2.0.


EiriniX crashed after applying a change to values.yaml

Describe the bug
Logs from eirini-loggregator-bridge-eirini-loggregator-bridge:
{"level":"info","ts":1587421772.3172147,"caller":"config/getter.go:53","msg":"Using in-cluster kube config"}
{"level":"info","ts":1587421772.3172426,"caller":"config/checker.go:36","msg":"Checking kube config"}
Received non-pod object in watcher channel
Watcher channel closed

To Reproduce

Deploy with Eirini.
Apply a change to values.yaml, in my case, I changed the number of nodes on CATS:
From:

properties:
  acceptance-tests:
    acceptance-tests:
      acceptance_tests:
        ginkgo:
          nodes: 6

To:
properties:
  acceptance-tests:
    acceptance-tests:
      acceptance_tests:
        ginkgo:
          nodes: 1

After the KubeCF installation gets updated, Eirini crashes.
Expected behavior
Eirini should not crash.
Environment
Tip of master.

does this happen with the new native Eirini helm chart @f0rmiga ?

FWIW, I've been seeing this in my eirinix helm chart work; it does not happen on the first deployment in a cluster (deleting the kubecf namespaces doesn't fix it).  Probably means eirinix is creating something and then falling over if it already exists.
PodWatcher.Handle isn't checking for e.Type == watch.ERROR; adding that and dumping the event says something about too old resource version, IIRC.  Currently suspecting this but I'm not very confident about it.

FWIW, I've been seeing this in my eirinix helm chart work; it does not happen on the first deployment in a cluster (deleting the kubecf namespaces doesn't fix it). Probably means eirinix is creating something and then falling over if it already exists.
PodWatcher.Handle isn't checking for e.Type == watch.ERROR; adding that and dumping the event says something about too old resource version, IIRC. Currently suspecting this but I'm not very confident about it.

I think you are right, probably we need to handle the starting event as described here: https://github.com/kubernetes/client-go/blob/master/tools/watch/until.go#L175

EiriniX changes: https://github.com/SUSE/eirinix/pull/38 (ready for review), loggregator-bridge: https://github.com/SUSE/eirini-loggregator-bridge/pull/17 (Draft, need to test on a KubeCF deployment)

Waiting for https://github.com/cloudfoundry-incubator/kubecf/pull/800 to confirm that the issue is fixed by the above changes


Update machine-learning-apps-anomaly-detection.md

Remove word "Microsoft"

Hi @gronster, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


[Dispatch] Allow case insensitive image type names while getting version from versionSet and rampup manager (Jira: BDP-4551).

[Dispatch] Allow case insensitive image type names while getting version from versionSet and rampup manager (Jira: BDP-4551). This would help getting image version for an image type in case insensitive manner in KubernetesContainerizedImpl.java as well.

Pull Request Test Coverage Report for Build 5762

13 of 17 (76.47%)  changed or added relevant lines in 4 files are covered.
21 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 43.051%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

azkaban-common/src/main/java/azkaban/imagemgmt/utils/CaseInsensitiveKeyDeserializers.java
5
7
71.43%

azkaban-common/src/main/java/azkaban/imagemgmt/version/VersionSet.java
6
8
75.0%

Files with Coverage Reduction
New Missed Lines
%

azkaban-exec-server/src/main/java/azkaban/execapp/event/JobCallbackRequestMaker.java
21
56.25%

Totals

Change from base Build 5759:
-0.03%

Covered Lines:
15640

Relevant Lines:
36329

💛  - Coveralls

Pull Request Test Coverage Report for Build 5762

13 of 17 (76.47%)  changed or added relevant lines in 4 files are covered.
21 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 43.051%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

azkaban-common/src/main/java/azkaban/imagemgmt/utils/CaseInsensitiveKeyDeserializers.java
5
7
71.43%

azkaban-common/src/main/java/azkaban/imagemgmt/version/VersionSet.java
6
8
75.0%

Files with Coverage Reduction
New Missed Lines
%

azkaban-exec-server/src/main/java/azkaban/execapp/event/JobCallbackRequestMaker.java
21
56.25%

Totals

Change from base Build 5759:
-0.03%

Covered Lines:
15640

Relevant Lines:
36329

💛  - Coveralls


Why are you using dropout for validation steps?

https://github.com/IBM/molformer/blob/1ae8d842695d9b412fb937385ef93f05a359162e/finetune/finetune_pubchem_light.py#L251C1-L252C1
AFAIK, dropout shouldn't be use to get validation stats on your model, and you're using 0.1 dropout for this. Could you elaborate? I may be missing something.

Hi. I thought I could write a brief answer in case you are still looking for it.
Dropouts are not used in .eval() mode and pytorch-lightning runs validation steps in eval() mode under the hood. so its equivalent to not having them.
Hope it helps! Cheers.

@subhalingamd Wow I had no idea, thanks for clarifying. Makes total sense.
Cheers


Logging should not go to Transcript by default

A user should be able to turn on and off RSR logging. Right now, they don't get a choice and it goes to Transcript by default.

Logging is available on a per-connection basis. Additionally, it does not default to going to the transcript.


[DDO-3170] auth using access/id tokens instead of sa keys.

Jira Ticket: https://broadworkbench.atlassian.net/browse/DDO-3170

Summary of changes:

What

Updates integration test code to alternatively take short lived tokens instead of service account keys for:

lyle-user
terra-sa (firecloud-alpha)

Prioritizes the above tokens and will avoid pulling sa keys if they exist, but will fallback if tokens are not provided. This will protect backwards compatbility with current dev processes and CI/CD
the GHA REQUIRES tokens and does not accept sa-keys so there should be no way for the GHA to attempt to pull SA keys
Does some renaming: including the acknowledgement that terra-sa is used exclusively to get a user access token. NOTE: This new logic does diverge the userEmail source of truth.

Why

This presents a fairly large security risk as terra-ui is a public repo so there is a risk of leaking these service account keys without knowledge.
Standardizes CI/CD access to using WIF instead of having applications
The GHA-ification of this allows for arbitrary testing against arbitrary environments instead of being locked into the current alpha/staging testing cycle

Testing strategy

Example GHA runs can be found here:

https://github.com/broadinstitute/monkey/pull/22

With these changes, what steps does one need to do to run the integration tests locally? The wiki page will need to be updated with this information-- https://github.com/DataBiosphere/terra-ui/wiki/Integration-tests


Equivalent of 'state' oauth parameter in Stripe Apps

We are investigating how to migrate our Extension into an (published) App.
Today our install flow works like this:
-User is logged in our site
-User clicks 'Connect Stripe account'
-Our site redirects to the Stripe Connect OAuth url, passing a dynamic state parameter (as documented here)
-Once OAuth is complete, we inspect the state parameter to retrieve information that is required to complete the installation process on our end.
How would this work with the new authentication mechanism of Stripe Apps?
How do we trigger the installation of the app from our site and pass a state parameter?
I am aware of the post install action to link to an external url but that won't help since we need dynamic state on each app install and this seems to only work to redirect to a static url provided in the stripe-app.json manifest.
This is related to https://github.com/stripe/stripe-apps/issues/210

Hi there,
any update on this?

Hi @clement911,
At the moment we don't support state as a part of our app installation flow with post install actions.
What dynamic state do you need to provide for your migrating Connect Extension?

I was checking back on this and realized you had left a comment I had missed.
We need some dynamic state to record the user id and project id (these are concepts in our own app) that the Stripe account should be linked to. It is possible for differents users and projects to connect the same Stripe account, so this is critical.
In practice, a simple dynamic string such as the oauth state param is enough, and we can encode in there whatever information we need (in our case the user id and project id).
This is critical for us.
For example, a user might belong to several teams and might want to connect a stripe account to a particular team.
So yeah, without dynamic state, we don't have any work around to migrate our connect extension into a stripe app.
Can you please to support dynamic state in Stripe apps?

@clement911 Thank you! This is great feedback.
We are working on making this better and enabling parity with OAuth for Connect Extensions. I can't give you a specific timeline right now, but we will get back to you on this. Stay tuned.

@auchenberg-stripe  that's music to our ears 🙂
We are looking forward to this so that we can move forward with our migration from a Connect extension to Stripe app.

Hi @auchenberg-stripe , are you able to share a bit more on progress on this issue at this point?

@clement911 Not at the moment, but know that we are aware that you are blocked, and it's top of mind for us.

Thanks @auchenberg-stripe for the quick update. We are looking forward to hearing more

@auchenberg-stripe we are still blocked by this. Do you have a bit more to share or is this still in the works?

@clement911 I can't provide you with specific timeline for when we will address this, but it's very top of mind for us. Would you be open to jumping on a 30min call to share what your needs are? What's a good email to reach you at?

Sure, happy to have a call. Feel free to reach out at clement at betterreports.com

Was there any resolution to this? I have same concerns as @clement911. The upgrade path from Stripe Connect OAuth to Stripe Apps isn't so clear. Thanks all!

@roncito FYI I had a chat with the nice people at Stripe and they confirmed to me that they are working on allowing apps to trigger installation via a oauth url where we can specify a state parameter.  So, I believe this will be addressed in the near future.

Hi @roncito
Yes, to second what @clement911 wrote: We are working on this, and we'd love your feedback. What's the easiest way to get in touch with you?

We cam make this work with install links.
Closing.


64x32 1/32 only works properly when PANEL_RES_Y set to 64

Hi. I have a problem, in my current configuration i have a 64x32 panel. i got it working properly, but it only shows fully when in the sketch i use PANEL_RES_Y to 64, if i set if i change it to 32 it only draws 16 pixels. Also the display shifted 1 pixel in row number 17 and below. Heres the attached photos
PANEL_RES_Y = 32

PANEL_RES_Y = 64

Update : Changing clockphase to false does fix the layer shifting issue, but PANEL_RES_Y Still need to be declared at 64 in order to use the panel fully

I have always had this confusion and don't know how to solve it


performReadonlyTransformation() for SearchableDropdownField and SearchableMultiDropdownField doesn't respect lazy loading

A follow-up to https://github.com/silverstripe/silverstripe-framework/issues/11272
If you call performReadonlyTransformation() on a SearchableDropdownField or SearchableMultiDropdownField, it will convert the source DataList into an array, which requires pulling data for all records in the list from the database.
All that's really needed is data for the record(s) which are currently selected for the field, since it's read-only it doesn't matter what other options could have been selected, because they weren't selected.
We may need to create a new readonly formfield for these - at the very least we'll need to implement some specific logic in the performReadonlyTransformation() method to instantiate the readonly field correctly without pulling unnecessary data.
Acceptance criteria

Calling performReadonlyTransformation() only loads data from the DB that is required to display the currently selected record(s) in the readonly form.

I'm working on it. Feel free to assign the issue to me.
Me first Idea was to overwrite performReadonlyTransformation and just change source to the selected items if the control is readonly.
Unfortunately that wont work. The value can still be set on a readonly Formfield. For example in form scaffolding the field is created and performReadonlyTransformation is run before the value is set.
So the acceptance criteria is probably missing changes to the value after performReadonlyTransformation.

PR for this is at https://github.com/silverstripe/silverstripe-framework/pull/11297
I'm not to happy with it, but I guess it works and is in line with the rest of the controls implementation.

I was looking for that issue, I think it is what's bugging the UploadField.
I have this Error: Injector.get(): Component SearchableDropdownField does not exist (when calling vendor.js)
It happens when I try to select an image from the media folder, is that related?

@digitweaks That sounds like a separate (but tangentially related) issue. Feel free to open a new issue, and explain what problems that's causing. Note that if it's just a console error but everything is otherwise working as expected, it is very unlikely to be prioritised.


Update mismatches in the view when helices change

Similar to #237 but applied to mismatches instead of modifications. Mismatches are drawn correctly when helices are removed, but not when "Display only selected helices" is checked. (They seem to default to the position they would be if all helices are displayed).

@dave-doty
https://github.com/UC-Davis-molecular-computing/scadnano/blob/a2b6e5f5e81c2a30b52ceaf0f26efb108165a5ef/lib/src/view/design_main_mismatches.dart#L34-L35
I don't understand what this FIXME is for, is there a reason why you didn't use the props.dna_design?
I'm planning to replace that line with:
 var helix = props.dna_design.helices[substrand.helix];

Hmm, I know I recently added dna_design as a prop to fix that FIXME. But I must have forgotten to actually change the line of code. Yes, please change it to reference props.dna_design (and remove the parameter dna_design from the method).


Admin Search Feature

allow team to use search features in the admin. currently this feature is causing an error.

Added to BRP version 1.0.5


I think, this fixed all the crossed quad walls

Have you played with debug example? It is really useful for debugging. Can you reproduce this twist?

This code was really only for the quads not being able to build into a polygon2D, but I'll try to fix that too.

Ah sorry, read to fast and thought you were talking about the whole polygon


Support dnsPolicy and dnsConfigs for operator and webhook in Helm chart.

Hello. I'm trying to use the spark-on-k8s-operator using a different dnsConfig  and dnsPolicy  due to some restrictions and limitations of k8s cluster that I'm using.
Something like this in values.yaml of helm chart:
dnsConfig:
          nameservers:
          - XXX.XX.X.XXX
dnsPolicy: ClusterFirstWithHostNet

This is necessary because, in my case, if I don't set dnsPolicy: ClusterFirstWithHostNet for the pods in my cluster, it's not solving the dns kubernetes.default.svc used to initialize the webhook. The image below shows the error if I don't set dnsPolicy: ClusterFirstWithHostNet (it happens for both init-webhook and clean-up-webhook)

I need to use the webhook activated in order to some customizations in SparkApplication take effect (like also, configure dnsConfig and dnsPolicy) for driver and executors, otherwise, it will not take effect.
Thanks!

Hi @leticiasss. In my case, the k8s cluster network was fixed and it started to work ( the webhook ). I think one possible workaround for it is to generate the yaml mafinests files from the helm chart, edit the manifests network configs and deploy it manually.

Hi @messerzen. Thank you for updated. Initially the installation unsuccessful, but did it again, then worked as expected.


module 'tensorflow._api.v1.config.experimental' has no attribute 'set_lms_enabled'

I am new to tensorflow LMS, and struggling with it from the start.
I tried to enable tensorflow LMS with "tf.config.experimental.set_lms_enabled(True)" as instructed in READEM on this github, but the it produces the following error.
I am using tensorflow 1.15 from WML-CE 1.6.2 with python 3.7.5.
import tensorflow as tf
tf.version.VERSION

'1.15.0'

What did I do wrong ?
AttributeError                            Traceback (most recent call last)
<ipython-input-2-92d4b0125d7e> in <module>
     26 from tensorflow_large_model_support import LMS
     27
---> 28 tf.config.experimental.set_lms_enabled(True)
     29
     30 gpu_devices = ['/device:GPU:0', '/device:GPU:1', '/device:GPU:2', '/device:GPU:3']

~/anaconda3/envs/wmlce_env/lib/python3.7/site-packages/tensorflow_core/python/util/module_wrapper.py in __getattr__(self, name)
    191   def __getattr__(self, name):
    192     try:
--> 193       attr = getattr(self._tfmw_wrapped_module, name)
    194     except AttributeError:
    195       if not self._tfmw_public_apis:

AttributeError: module 'tensorflow._api.v1.config.experimental' has no attribute 'set_lms_enabled'

Thank you for reaching out. The most current implementation of large model support is in TensorFlow 2.1.0 in WML CE's early access channel. For more information about the early access channel and how to use it you can see this post.
WML CE 1.6.2 does contain an implementation of large model support, named TFLMSv2. It's install and usage instructions are documented here. However, since you are new to TensorFlow I would suggest starting with the latest TensorFlow release (2.1.0) and the LMS implementation for it. The new LMS for TensorFlow 2 is much easier to use than the LMS for TensorFlow 1.x.

that you can find it for TensorFlow 2.1.0 in WML CE's early access channel could be mentioned in the readme of the LMS git repo.

TF 2.1 has since been promoted to the main WML CE channel and LMS is included, so the README is now accurate:
https://github.com/IBM/tensorflow-large-model-support#installing-tensorflow-large-model-support

Is it #29 ?


Create Player class for yupiko

Create a Player class to update the player board and check if the player got yupiko(Bingo) or not.

hi!! can I be assigned to this issue please?

Hi @cmcgee0308 of course!


Fix incorrect types in client os.getenv calls

When I set LANGFUSE_FLUSH_AT=50, I saw this error in logs:
File "/usr/local/lib/python3.11/site-packages/langfuse/task_manager.py", line 99, in _next
while len(items) < self._flush_at:
^^^^^^^^^^^^^^^^^^^^^^^^^^^
 TypeError: '<' not supported between instances of 'int' and 'str'

This is because env vars are all strings:
$ FOO=50 python -c 'import os; print(type(os.getenv("FOO")))'
<class 'str'>

This fixes the code to instead get with a default value, and then cast to the expected type.

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

@rohan-mehta thanks a lot for the contribution and fixing this. We released this in: https://github.com/langfuse/langfuse-python/releases/tag/v2.33.0


Custom reporting enum

You should be able to create a custom Priority enum and create n number of values to use. I have included a ReadMe explaining some of the work that would need to be done in order to use a Custom enum.

Seems that adding a priority customization requires a lot of non-backward compatible API changes.
Wouldn't it be much simpler to let the priority untouched and adding an addtional customizable categorization for warnings?


Class Sensio\Bundle\DistributionBundle\Composer\ScriptHandler is not autoloadable, can not call post-update-cmd script

try to composer update
Writing lock file
Generating autoload files
Class Incenteev\ParameterHandler\ScriptHandler is not autoloadable, can not call post-update-cmd script
Class Sensio\Bundle\DistributionBundle\Composer\ScriptHandler is not autoloadable, can not call post-update-cmd script
Class Sensio\Bundle\DistributionBundle\Composer\ScriptHandler is not autoloadable, can not call post-update-cmd script
Class Sensio\Bundle\DistributionBundle\Composer\ScriptHandler is not autoloadable, can not call post-update-cmd script
Class Sensio\Bundle\DistributionBundle\Composer\ScriptHandler is not autoloadable, can not call post-update-cmd script
Class Sensio\Bundle\DistributionBundle\Composer\ScriptHandler is not autoloadable, can not call post-update-cmd script
Class Sensio\Bundle\DistributionBundle\Composer\ScriptHandler is not autoloadable, can not call post-update-cmd script

Do you still have sensio/distribution-bundle in your requirements?

Found the bug - a misconfiguration in my composer.json.

@heyalbert its very frustrating when we search for same issue and ppl close ticket just saying I fix the error without actually say what it was so other ppl can profite of the fix.

Damnit! WHAT misconfiguration? :D

Got the same issue.

What worked for me was to completely blow away the vendors folder rm -rf ./vendor then run a composer update.

Got the same issue. Try to add SensioDistributionBundle if not already added by executing:
composer require sensio/distribution-bundle

Sorry for also posting in this thread. But it is a top result on Google when searching for this error.
###To solve this:
Make sure you require both sensio/distribution-bundle and incenteev/composer-parameter-handler. Then make sure your composer.json looks similar to Symfony standard's. Look at the autoload, extra and the script section.

got same issue during upgrade symfony3.4 to  v4.4 what should i do ?


Make vitest --coverage cover all files by defaults to  or configurable directly in CLI

Clear and concise description of the problem
In my experience, more often than not, it's desirable to have code coverage for all files, not only the tested ones.
Considering that I tend to keep my config files at the bare minimum (package.json and tsconfig.json only) I usually have the following npm test script:
"test": "standard && tsc && vitest --coverage"

This lints, type-checks, tests and cover my source.
The only problem with this is the code coverage not reaching all files.
Having to create a vitest.config.js just to declare test.coverage.all = true feels a bit awkward and an unnecessary step.
Suggested solution
The simplest (and unlikely to cause any trouble) solution is to add a --coverage-all or similar flag to CLI.
Another solution is to make vitest --coverage to cover all files by default.
Although I can't think of a reason for this to be a breaking change (in the sense that things would stop working) I understand that this would change the behavior of the tooling in an unexpectable way for some users.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Doesn't --coverage.all just work?

Doesn't --coverage.enabled --coverage.all just work?

Wow! Thanks @sheremet-va.
Indeed, it works!
Question is, is this documented somewhere? I couldn't find it.


txt file upload

First of all, thank you very much for the package. I want to upload a txt file to icloud. I want to specify the name of this file myself. I didn't understand how to do it. Can you help me?

Just specify the file name in the 'destinationRelativePath', such as 'xyz.txt'.

thank you very much ✨ I have one last question. It uploaded successfully, but I can't see it in icloud drive. Is this normal?

Maybe this will help you: https://github.com/deansyd/icloud_storage/issues/14#issuecomment-1090811902

@gizemgizg were you able to check the files created on iCloud ? I am not able to find the files even after successful upload

I managed to keep data as an object in cloudkit, not as a file. This plugin creates a record type called StorageItem. You can see the indexes in the StorageItem from the CloudKit Dashboard and since iCloud is hidden, you cannot see the data you saved, but you can see the value you saved with the key. I made some changes and used it in my project  @SunilKividor

But this is not regarding cloudkit right ? This is about iCloud and app containers

yes about the cloud kit

Oh, sorry. I just realized. I didn't use this package, I had the same problem so I decided to use cloud_kit. I recommend it. @SunilKividor


il2cpp: couldn't resolve export il2cpp_get_corlib

Device:
Emulator LDPlayer Android 7.1 64-bit
Command
frida -H <IP_ADDRESS>  -l dist/agent.js --realm=emulated
so, i can get libil2cpp.so address, but i cant dump game
Il2Cpp.dump() dont working
i didnt tested other functions
log:
il2cpp: couldn't resolve export il2cpp_get_corlib
    at r (il2cpp/api.ts:501)
    at get getCorlib (il2cpp/api.ts:272)
    at call (native)
    at <anonymous> (utils/lazy.ts:10)
    at initialize (il2cpp/module.ts:17)

script:
import "frida-il2cpp-bridge";

console.log("start!");
Il2Cpp.perform(() => {
    console.log("true");
});

maybe, i fixed this by script:
import "frida-il2cpp-bridge";

globalThis.IL2CPP_MODULE_NAME = "libil2cpp.so";
Reflect.defineProperty(Il2Cpp, "module", {value: Process.findModuleByAddress(DebugSymbol.fromName("il2cpp_init").address)});

declare global {
    let IL2CPP_EXPORTS: Record<string, () => NativePointer>;
}

(globalThis as any).IL2CPP_EXPORTS = {
}

for (const module_ of Process.enumerateModules()) {
    for (const export_ of module_.enumerateExports()) {
        if (export_.name.includes("il2cpp")) {
            send(`${module_.name}!${export_.name} @ ${export_.address}`);
            IL2CPP_EXPORTS[export_.name] = () => Il2Cpp.module.base.add(new NativePointer(export_.address));
        }
    }
}

but i have another problem in #452


Update main.go

Exporting your read function so it can be used when using your code as a library

Thanks for this patch @mathew-bowersox . I'm assuming are importing this in which case this patch will work fine for you. However as it stands this would break main() and the tests in main_test.go. If you could update this patch so that go test works. and main() still works. I'll merge the patch.
Additionally to do this correctly main should be moved to cmd/jfect and package code should move to . pkg with tests. then cmd/jfect can import the package. and anything you need to import can just use the github.com/mroset/jflect/pkg.

@mathew-bowersox thanks for the changes, just a couple of things to fix and then almost done.


peg i18n gem to 1.13

There is an issue with 1.14 that breaks tests, and possibly functionality, too. Within 3 hours a number of people have referenced this issue so we are not alone
This is fixed by
https://github.com/ruby-i18n/i18n/pull/662
Goal is to revert this once 1.15 goes live

Backported to petrosian in commit ceb7df0584db6c409f365fff0595ab7ae7f04bd0.
commit ceb7df0584db6c409f365fff0595ab7ae7f04bd0
Author: Joe Rafaniello<EMAIL_ADDRESS>Date:   Fri Jun 2 12:39:33 2023 -0400

    Merge pull request #22545 from kbrock/peg_i18n

    peg i18n gem to 1.13

    (cherry picked from commit 6e1123215424d2197080836e6538b5f755d03496)


Official SVD from espressif fails to be parsed by svd2nim, but works correctly in svd2rust

Non sure if this is a problem of svd2nim, or is just espressif stretching the official xml schema outside the original use.
Here an open issue in the official repository talking about these mis-alignment: https://github.com/espressif/svd/issues/27

Seems there are a few things at play.
First, I fixed the crash due to unknown CPU when trying to look up CPU interrupts: https://github.com/EmbeddedNim/svd2nim/commit/511e00c6f977b2f93ddc4132b9d0b5d636fa071e
But it's still failing because the esp32.svd file uses derivedFrom in an enumeratedValues item, which is valid per the SVD spec (https://arm-software.github.io/CMSIS_5/SVD/html/elem_registers.html#elem_enumeratedValues) but not currently implemented by svd2nim (its usage is pretty rare), and doing so will take a bit of work.

Good news, you can try the esp32-compat branch, I implemented derivedFrom for enums and it doesn't crash on the esp32.svd file anymore :)
Still not 100% working though, because I also noticed that the SVD contains dimLists/dimArrays (basically repeats) of field elements, which is also not currently implemented. At least svd2nim doesn't crash, it warns about the situation and there's probably stuff missing in the generated code. Warnings like this:
WARNING: Register field INT_RAW.CH%s_TX_END_INT_RAW of peripheral RMT contains dimGroup, not implemented.

You'll also notice that svd2nim generates a lot of warnings like this:
WARNING: Property "access" for register AES.IDLE is undefined. Defaulting to read-write.

To get rid of those, you can simply add a default <access> tag at the <device> level (https://arm-software.github.io/CMSIS_5/SVD/html/elem_device.html). STM32 SVDs seem to have the same issue. The SVD spec requires that access is specified somewhere but they don't bother adding a default for some reason.

I added support for field dimLists. Codegen looks good to me with these changes.
Thanks for reporting and please open another issue if you find other problems with the esp32 SVDs.

Has anyone noticed the esp32s3.svd has a "%s" in some of the name fields?

@Joebeazelman yeah that's a feature of the SVD spec called dimElements: https://arm-software.github.io/CMSIS_5/SVD/html/elem_special.html#dimElementGroup_gr
It's used to generate a repeating array of contiguous items (registers), without defining all of them manually, for example PORT1, PORT2, PORT3, or whatever.
It's commonly used and should work well with svd2nim.

Why did they base SVD on the XML standard only to violate it. The point of XML is to provide a standard representation of data. Once it relies on external metadata, it's no longer XML.
It's just senseless over-engineering. For what purpose, to shave a few bytes at the cost of breaking existing parsers?


Included metadataPolicy: Fetch in Hashicorp Vault

Fixes #1877
Checklist

[x] I have read the contribution guidelines
[x] All commits are signed with git commit --signoff
[x] My changes have reasonable test coverage
[x] All tests pass with make test
[x] I ensured my PR is ready for review with make reviewable

Nice and Simple 😄 from a first look LGTM. I'll just run up some manual tests, but I think it looks pretty solid 💪

/approve


Stuck at the final step in the pipeline

Really great guide - thanks for sharing.
I seem to be stuck at the last step running kubectl get nodes
error: group map[extensions:0xc8204062a0 federation:0xc82034d7a0 batch:0xc8204060e0 certificates.k8s.io:0xc820406150 authorization.k8s.io:0xc82034dea0 autoscaling:0xc82034df10 componentconfig:0xc8204061c0 policy:0xc820406310 rbac.authorization.k8s.io:0xc820406380 :0xc82034dc70 apps:0xc82034dce0 authentication.k8s.io:0xc82034dd50 storage.k8s.io:0xc8204063f0] is already registered
I had this on both aws and gce.
Any suggestions appreciated.
Cheers!
script returned exit code 1
Versions used
  - kubernetes:0.12
  - workflow-aggregator:2.5
  - credentials-binding:1.12
  - git:3.5.1
  - pipeline-github-lib:1.0
  - ghprb:1.39.0
  - blueocean:1.1.4

'jenkinsci/jnlp-slave:2.62
docker:1.12.6
golang:1.8.3
lachlanevenson/k8s-helm:v2.5.0', command
lachlanevenson/k8s-kubectl:v1.7.6

Tweaked some versions and all is well.


Add "v25.0.x" option for "Docker Install URL" dropdown

Is your feature request related to a problem? Please describe.
As part of https://github.com/rancher/rancher/issues/44459 we're adding support for Docker 25.0.x.
Describe the solution you'd like
UI dropdowns that allow picking "Docker Install URL" (e.g. on Node Template details page - not sure where else it may be referenced) should additional have "v25.0 (Recommended)" choice that populates https://releases.rancher.com/install-docker/25.0.sh when selected.
Describe alternatives you've considered
Note "Docker Install URL" is editable and doesn't limit users to only the choices available, so they can specify https://releases.rancher.com/install-docker/25.0.sh manually as well before this enhancement is done.
Consequently existing option "v24.0.x (Recommended)" should be changed to "v25.0.x".

@mitulshah-suse is this needed for 2.8.next1? We'll target 2.9.0 first but let us know if this is tied to additional backported work.

@gaktive yes, its needed for 2.8-next1.

/backport v2.8.next1

For the records: the value is hardcoded in Ember.

Found on RKE1 provisioning.
File: lib/shared/addon/components/form-engine-opts/component.js
Example from a past change: https://github.com/rancher/ui/pull/5045


Fix for issue #146: Performance Counter Instances Never Removed

The fix turned out to be quite simple (thanks @jonorossi for the tips!), the InstanceLifetime of the created PerformanceCounter had to be set to Process. This causes the created performance counter instances to be removed when the process closes.
I also updated performance-counters.md to reflect this.

Nice one

Looks good, fixes #146.


optional argument --scale issue

This used to scale the plots so that there would only be lysines. Here's what the plot looks like now when I try to use this command:
new-host-3:proscattertest katelynmshipper$ python proScatter.py -s UvrD.fasta 2is1symxwalk/ 2is1monomer/ EC/
Summary for UvrD-UvrD
2is1symxwalk:  3
2is1monomer:  3
EC:  1
Intersection:  1

Yes, looks like a bug. I'll look into it.


Alert users when upgrading provisioned k3s cluster from k8s <1.25 to >=1.25 if PSP enabled

Is your feature request related to a problem? Please describe.
If the provisioned cluster has PodSecurityPolicy (PSP) admission controller enabled, it needs to be removed before proceeding with the upgrade.
Additionally, PSPs will be removed automatically as k8s 1.25+ no longer supports PSPs.
Describe the solution you'd like
On the "Edit Cluster" screen for a provisioned k8s cluster, when k8s version is changed from <1.25 to >=1.25, present the user with the confirmation message along the lines of Kubernetes has removed support for Pod Security Policies (PSPs) starting with version 1.25. If your cluster has PodSecurityPolicy admission controller enabled via "kube-apiserver-arg.enable-admission-plugins" in Cluster YAML, it has to be MANUALLY removed before proceeding with the upgrade. Additionally, any PSPs that may be present in the cluster will no longer be available/enforced. Proceed? with the options "Yes" and "No".
Describe alternatives you've considered
Ideally, we want to only show this warning if PSP admission controller is indeed enabled. If time permits only shows the following warning if Cluster YAML's kube-apiserver-arg.enable-admission-plugins contains PodSecurityPolicy value, e.g. kube-apiserver-arg: enable-admission-plugins=NodeRestriction,PodSecurityPolicy,NamespaceLifecycle,ServiceAccount: Kubernetes has removed support for Pod Security Policies (PSPs) starting with version 1.25. Your cluster has PodSecurityPolicy admission controller enabled via "kube-apiserver-arg.enable-admission-plugins" in Cluster YAML, it has to be MANUALLY removed before proceeding with the upgrade. Additionally, any PSPs that may be present in the cluster will no longer be available/enforced. Proceed? with the options "Yes" and "No".
Additional context
N/A

As the logic already contains a modal in this process, we'll rather update the existing logic and text.

Please note that PSP must be enabled in order to display the dialog.
As reference to how retrieve the information, please refer to the comment: https://github.com/rancher/dashboard/issues/7553#issuecomment-1430202568

@snasovich
Where is supposed to be found the value kube-apiserver-arg.enable-admission-plugins?
How can I create a K3S cluster with PSP enabled?
Why is the check different from https://github.com/rancher/dashboard/issues/7553#issuecomment-1430202568?

@jiaqiluo Do you have eventually further information about kube-apiserver-arg.enable-admission-plugins?

Identified some clashes with modal styles in another issue https://github.com/rancher/dashboard/issues/8207

Hi @cnotv , let me try to answer your questions, and please let me know if it makes to you.
The way to enable PSP for the K3S cluster in rancher UI is to set the extra args for the kube-apiserver (see the screenshot below), and as a result, the field .spec.rkeConfig.machineGlobalConfig.kube-apiserver-arg will reflect the values ( see screenshot below).
For UI to check if PSP is enabled, UI needs to read the value of the above-mentioned field from the cluster's YAML file and check if the word PodSecurityPolicy exists.

For the sake of copy-pasting, this is the line to be added: enable-admission-plugins=NodeRestriction,PodSecurityPolicy

This test plan covers this issue as the fixes implemented are similar. As such, once that issue has been validated, I will leave a comment here and close this issue.

This has been validated as part of the test results here. Closing issue.


SCM tracks folder even after removing it from workspace (multi root)

VSCode Version: Code 1.18.1 (929bacba01ef658b873545e26034d1a8067445e9, 2017-11-16T18:34:22.110Z)
OS Version: Linux x64 4.10.0-40-generic
Extensions:

Extension
Author (truncated)
Version

vscode-eslint
dba
1.4.3

tslint
eg2
1.0.24

MagicPython
mag
1.0.12

python
ms-
0.8.0

cpptools
ms-
0.14.3

debugger-for-chrome
msj
3.5.0

python-extended-snippets
tus
0.0.1

Steps to Reproduce:

Open multiple folders with git initialised
Remove a folder from workspace
We can still stage, commit to the folder

Reproduces without extensions: Yes/No

You have to remove the repository itself: right click on the repository and close it.


Broken terminal after killing jest watcher with SIGTERM

🐛 Bug Report
After running watcher programmatically and killing it with SIGTERM (which seems like an appropriate way of killing jest in such scenario) my terminal is broken with - its lflags are are different than normal. stty gives such result before running the script:
speed 9600 baud;
lflags: echoe echoke echoctl pendin
iflags: iutf8
oflags: -oxtabs
cflags: cs8 -parenb

and such result after running it:
lflags: -icanon -isig -iexten -echo echoe echoke echoctl
iflags: -icrnl -ixon iutf8 -brkint
oflags: -oxtabs
cflags: cs8 -parenb

Notice how i.e. -echo gets added to lflags.
To Reproduce
Running such script causes this:
const spawnCommand = require("spawn-command")

const test = spawnCommand("yarn watch", {
  stdio: "inherit"
})

setTimeout(() => {
  test.kill("SIGTERM")
}, 3000)

I've managed to dig deeper and I've discovered that this is "caused" by using execa package to run different scripts (i.e. git one) which in turn uses signal-exit package internally. The latter one adds SIGTERM listener to the current process (jest) so it can call a callback before process exitting which execa listens to so it can clean up after itself (its spawned commands etc). This is all good and expected - also the listener in question gets removed appropriately, but somehow this breaks default SIGTERM handler and original terminal settings are not being restored.
After analyzing this problem - I've managed to come up with a simplified "fake jest" (those exact steps are executed by jest):
process.stdin.setRawMode(true)

const handler = () => {}

process.on('SIGTERM', handler)
process.off('SIGTERM', handler)

setInterval(() => {
    console.log('jest fake')
}, 2000)

The key culprits are:

using raw mode
adding & removing SIGTERM handler

Expected behavior
Functional, "untouched", terminal.
Link to repl or repo
https://github.com/Andarist/jest-watcher-sigterm-repro
Just install dependencies and run yarn dev or yarn simplified
Run npx envinfo --preset jest
  System:
    OS: macOS Sierra 10.12.6
    CPU: (8) x64 Intel(R) Core(TM) i7-6920HQ CPU @ 2.90GHz
  Binaries:
    Node: 11.8.0 - ~/.nvm/versions/node/v11.8.0/bin/node
    Yarn: 1.13.0 - ~/.nvm/versions/node/v11.8.0/bin/yarn
    npm: 6.5.0 - ~/.nvm/versions/node/v11.8.0/bin/npm
  npmPackages:
    jest: ^24.8.0 => 24.8.0

cc @thymikee

@rogeliog are you able to help here?

After thinking about it a little bit more this kinda seems like a node problem, but maybe something can be done in jest to overcome this. And even if it's node problem there is always a chance that we'd like to kill jest between adding custom SIGTERM handler and removing it, so when that happens and we receive SIGTERM we should be able to clean up. I haven't found a way to do it, but I'm not too familiar with node's APIs around this space.
BTW. "needs repro" is not applicable anymore as I've presented the repro case 😉

Yes, I can help with this

Since the issue was reproduced and downsized to a minimal repro - I think this is a problem, even if not a high priority one. It might very well not be a problem with jest itself - but it's not obvious how to overcome it and I lack the expertise to figure this out on my own.

Bump for stale bot


NPM Support

Support using NPM instead of bun / yarn (fixes https://github.com/rails/cssbundling-rails/issues/138). This mirrors the fixes applied in: https://github.com/rails/jsbundling-rails/pull/180

Included via https://github.com/rails/cssbundling-rails/pull/145


Add options for ratio and tileType

Goal
Adds cli options for 'pixel ratio' and 'tiletype' to allow changing the type of tiles generated from 'jpg', 'png', or 'webp'. It defaults to a ratio of 1 and tiletype of 'jpg', which were the old hardcoded defaults.
Screenshots
with ratio 1, tiletype webp
`node . --style self --stylelocation styles\http_pmtiles_local_spites_fonts.json --stylesources styles\data --bounds "-79,37,-78,38" -Z

with ratio 2, tiletype webp
node . --style self --stylelocation styles\http_pmtiles_local_spites_fonts.json --stylesources styles\data --bounds "-79,37,-78,38" -Z 11 -f wdb_11_r2_webp -r 2 -t webp

with ratio 1, tiletype png
node . --style self --stylelocation styles\http_pmtiles_local_spites_fonts.json --stylesources styles\data --bounds "-79,37,-78,38" -Z 11 -f wdb_11_r1_png -r 1 -t png

with ratio 2, tiletype png

with ratio 1, tiletype jpg
node . --style self --stylelocation styles\http_pmtiles_local_spites_fonts.json --stylesources styles\data --bounds "-79,37,-78,38" -Z 11 -f wdb_11_r1_jpg -r 1 -t jpg

with ratio 2, tiletype jpg

it seems size wise, webp could actually be the smallest sized images. i wonder what the compatibility of webp is these days

This is example of the effect of using ratio when viewing the files in qgis. you can see the ratio 2 tiles are much sharper
ratio: 1

ratio: 2

Thanks for the quick review @rudokemper. Can you make an NPM release with this new feature?

Sure thing, done.

Just an FY since I didn't note it.
You can have a ratio higher than 2, it just increased your tile image size the more you raise it. Tileserver GL accepts a max ratio of 3 by default, but can go all the way up to a ratio of 9 depending on your options.

Interesting, I wonder what kind of use case could benefit from such a high ratio.
Speaking of which, I'd be super curious to learn more about how ya'll are using the tool @acalcutt @wipfli!

I don't really have a use yet, but I have wanted to make a rendered mbtiles in the past for some atv trail ideas I have had. sometimes on trails the cell signal is low, so using web based maps isn't ideal. I'd love eventually to make a maplibre-native android app with local tiles that can be used offline.
In the past I have tried to make tiles using Qtiles, like
https://techidiots.net/notes/mapbox-gl-tileserver-gl/create-a-esri-satellite-mbtiles-base-map-with-qgis-and-qtiles
which i remember was a huge pain
On the maplibre-native side, I know I haave seen request for something like this in
https://github.com/maplibre/maplibre-native/discussions/2070
Really, mostly I was bored over the weekend and saw something I could contribute here. hope to see more progress on this. Now I am trying to figure out how to speed it up at higher zoom levels. I was trying a few suggestions from google code AI which were good starts, but so far nothing successful.

Super cool, thanks for sharing! Our use case is Indigenous users in an offline context who need maps for data collection or visualization, but I figured the tool could be helpful for a lot of different use cases. So it's affirming to read about other interests, and contribute something back to the broader Maplibre community that everyone can use.
(On an somewhat related note, over at https://github.com/terrastories/terrastories we're super grateful for the recent love you and others have given Tileserver-GL, and in particular support for arm architecture has been great for deploying the tool on low energy devices!)

Interesting, I wonder what kind of use case could benefit from such a high ratio.
Speaking of which, I'd be super curious to learn more about how ya'll are using the tool @acalcutt @wipfli!

I'd imagine at minimum it helps make overzooming look sharper. So if you made a low max zoom tile (like say zoom 8) and you were zoomed into zoom 14, a ratio 1 tile would probably be very blury, but a ratio 5 tile may still look decent, since it is basically the size it should be for that zoom level.

Glad my work on maplibre-native node and tileserver-gl is helpful to others :-) I have probably spent way to much time on getting that working :-)