Add support for multiple patterns

Changes search pattern to a list.
Also adds default support for R roxygen @section patterns for assumptions and caveats (#9).

Codecov Report

Merging #10 (220f22c) into main (12b6466) will decrease coverage by 0.32%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
- Coverage   50.32%   50.00%   -0.33%
==========================================
  Files           4        4
  Lines         153      154       +1
==========================================
  Hits           77       77
- Misses         76       77       +1

Flag
Coverage Δ

pytests
50.00% <57.14%> (-0.33%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

assumptions/log_items.py
64.91% <57.14%> (-1.16%)
:arrow_down:

Continue to review full report at Codecov.

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


Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
Cannot update SumatraPDF.SumatraPDF
Installer failed with exit code: 1
Steps to reproduce

Installed SumatrPDF.SumatraPDF in version 3.3.3
update the package : winget update SumatraPDF.SumatraPDF

Actual behavior
Console:
Found SumatraPDF [SumatraPDF.SumatraPDF] Version 3.4.6
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Successfully verified installer hash
Starting package install...
Installer failed with exit code: 1

Expected behavior
It should be updated
Environment
Windows Package Manager v1.4.11071
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19045.3086
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.19.11071.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

User Settings: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Screenshots and Logs
Logs (last entries before erorr):
2023-06-25 15:05:27.307 [CLI ] Installer args: -s -install -allusers -upgrade -allusers
2023-06-25 15:05:27.308 [CLI ] Starting: 'C:\Users\[username]\AppData\Local\Temp\WinGet\SumatraPDF.SumatraPDF.3.4.6\SumatraPDF-3.4.6-64-install.exe' with arguments '-s -install -allusers -upgrade -allusers'
2023-06-25 15:05:27.676 [CLI ] ShellExecute installer failed: 1
2023-06-25 15:05:27.677 [CLI ] Terminating context: 0x8a150006 at D:\a\_work\1\s\external\pkg\src\AppInstallerCLICore\Workflows\InstallFlow.cpp:1cb

Does this occur with upgrades outside of WinGet? If version 3.3.3 were downloaded and installed manually, and then upgraded manually, does the error still occur? These will have to be tested, to find if it's an issue with the installer performing the upgrade.

Ok, so I updated manually to 3.4.6 and it worked fine.
Then I uninstalled app and

Installed manually version 3.3.3. Then updated manually to 3.4.6 -> works fine
uninstalled manually and performed a test with winget, so:
winget install SumatraPDF.SumatraPDF --version 3.3.3 -> installed
winget update SumatraPDF.SumatraPDF -> failed
uninstalled with winget remove SumatraPDF.SumatraPDF
winget install SumatraPDF.SumatraPDF -> installed in version 3.4.6

Obviously there is something with winget when performing update from 3.3.3 to 3.4.6


Readme not in line with code

Hi,
we talked on the SBW conference last week, thanks again for answering my questions!
I just wanted to test your workflow and noticed that the readme and the code are not concordant. So when I change the config file as suggested in the readme, I get a bunch of KeyErrors and TypeErrors; for example when I change the "method" section from
method: "prune_gtdb"
to
method:
subsample_refseq: True
subsample_gtdb: True
prune_gtdb: True
as suggested in the readme, I get a TypeError from line 53 of download_genome_data.smk that dictionaries can't be concatenated. Is there any other way to turn on several of the three options or do I have to run separate runs?
Not sure if the readme is outdated or too far ahead here :)
A second issue I had was that on my computer (Mac OS), I could not install diamond from the supplied yaml file but had to add
pip:
before the diamond specification. Not sure if this is only a Mac OS issue or not..
Now that I have adjusted the config file and managed to install diamond, it seems to run just fine.
Cheers
Andreas

Hi,
Nice to meet you up in Umeå and thanks for putting up a great workshop, also thanks for trying out the workflow.
Oh, I must have forgotten to push the latest version of the README, I have updated it now, it was a bit outdated. You will have to run separate runs for the different sampling options in the current version.
Thanks for letting me know about the issue with Diamond on MacOS, I have only tested this on Linux so far. I have added a fix that should solve this.
Let me know if you run into more problems.
Cheers,
Emil

Thank you for the quick response and fix!
I then got stuck at the prokka stage, which turned out to be entirely a Mac issue again, apparently the new hardware (M1 processor) causes some weird issues, and it took me quite a while to get the right setup for java, hmmer, and diamond to work. I had to set up a Rosetta 2 installation of homebrew (described here: https://stackoverflow.com/questions/64882584/how-to-run-the-homebrew-installer-under-rosetta-2-on-m1-macbook) and then install the programs through that on my system because conda refuses to install them no matter what..
I will test it on a Linux system as well to make sure it was only that causing problems. Regarding the Mac issues, it would probably be faster to containerize the workflow than to include workaround for all of that into the readme or the code.
The workflow finally ran through completely now and I will take a look at the output, and try the other methods as soon as I have time.
/Andreas


localizationsDelegates callback returns list of locales that is platform specific

Steps to Reproduce

Install 'Flutter Intl' plugin (I am using Android Studio)
Create a new Flutter project
Use the Flutter Intl plugin menu Tools - Flutter Intl - Initialize for this project
Add callback function for localeListResolutionCallback
Two params are required:
locales - The locales installed on device
supported - The locales supported by app

Expected results:
I wan't the list of locales to return a prioritized list of locales installed on the phone.
Actual results:
On iOS the list of installed locales starts on index 1.
The index 0 ( locales[0] ) is ALWAYS english (en).
The list of supported languages should always be consistent regardless of platform.
[  +20 ms] executing: [/usr/local/Cellar/flutter/stable/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +36 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] 27321ebbad34b0a3fafe99fac037102196d655ff
[        ] executing: [/usr/local/Cellar/flutter/stable/] git describe --match v*.*.* --first-parent --long --tags
[  +13 ms] Exit code 0 from: git describe --match v*.*.* --first-parent --long --tags
[        ] v1.12.13+hotfix.5-0-g27321eb
[   +7 ms] executing: [/usr/local/Cellar/flutter/stable/] git rev-parse --abbrev-ref --symbolic @{u}
[  +14 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/stable
[        ] executing: [/usr/local/Cellar/flutter/stable/] git ls-remote --get-url origin
[  +13 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +57 ms] executing: [/usr/local/Cellar/flutter/stable/] git rev-parse --abbrev-ref HEAD
[  +15 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[   +6 ms] executing: sw_vers -productName
[  +15 ms] Exit code 0 from: sw_vers -productName
[        ] Mac OS X
[        ] executing: sw_vers -productVersion
[  +15 ms] Exit code 0 from: sw_vers -productVersion
[        ] 10.15.2
[        ] executing: sw_vers -buildVersion
[  +15 ms] Exit code 0 from: sw_vers -buildVersion
[        ] 19C57
[  +27 ms] executing: /usr/bin/xcode-select --print-path
[   +8 ms] Exit code 0 from: /usr/bin/xcode-select --print-path
[        ] /Applications/Xcode.app/Contents/Developer
[   +1 ms] executing: /usr/bin/xcodebuild -version
[ +205 ms] Exit code 0 from: /usr/bin/xcodebuild -version
[   +3 ms] Xcode 11.3.1
           Build version 11C504
[  +52 ms] executing: /Users/ovestorholt/Library/Android/sdk/platform-tools/adb devices -l
[   +7 ms] Exit code 0 from: /Users/ovestorholt/Library/Android/sdk/platform-tools/adb devices -l
[        ] List of devices attached
           emulator-5554          device product:sdk_gphone_x86 model:Android_SDK_built_for_x86 device:generic_x86 transport_id:15
[  +18 ms] executing: /usr/local/Cellar/flutter/stable/bin/cache/artifacts/libimobiledevice/idevice_id -h
[  +64 ms] /usr/bin/xcrun simctl list --json devices
[  +94 ms] More than one device connected; please specify a device with the '-d <deviceId>' flag, or use '-d all' to act on all devices.
[   +3 ms] /Users/ovestorholt/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell getprop
[  +31 ms] ro.hardware = ranchu
[   +5 ms] Android SDK built for x86 • emulator-5554                        • android-x86 • Android 10 (API 29) (emulator)
[        ] iPhone 11 Pro Max         • D9C0D762-E002-484B-B177-E5E529313FCE • ios         • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)
[  +10 ms] "flutter run" took 603ms.

#0      throwToolExit (package:flutter_tools/src/base/common.dart:28:3)
#1      RunCommand.validateCommand (package:flutter_tools/src/commands/run.dart:289:7)
<asynchronous suspension>
#2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:590:11)
#3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:515:33)
<asynchronous suspension>
#4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:146:29)
#5      _rootRun (dart:async/zone.dart:1126:13)
#6      _CustomZone.run (dart:async/zone.dart:1023:19)
#7      _runZoned (dart:async/zone.dart:1518:10)
#8      runZoned (dart:async/zone.dart:1465:12)
#9      AppContext.run (package:flutter_tools/src/base/context.dart:145:18)
#10     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:505:20)
#11     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
#12     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:413:21)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:146:29)
#14     _rootRun (dart:async/zone.dart:1126:13)
#15     _CustomZone.run (dart:async/zone.dart:1023:19)
#16     _runZoned (dart:async/zone.dart:1518:10)
#17     runZoned (dart:async/zone.dart:1465:12)
#18     AppContext.run (package:flutter_tools/src/base/context.dart:145:18)
#19     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:364:19)
#20     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#21     new Future.sync (dart:async/future.dart:224:31)
#22     CommandRunner.run (package:args/command_runner.dart:112:14)
#23     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:250:18)
#24     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:63:22)
#25     _rootRun (dart:async/zone.dart:1126:13)
#26     _CustomZone.run (dart:async/zone.dart:1023:19)
#27     _runZoned (dart:async/zone.dart:1518:10)
#28     runZoned (dart:async/zone.dart:1502:12)
#29     run.<anonymous closure> (package:flutter_tools/runner.dart:61:18)
<asynchronous suspension>
#30     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:146:29)
#31     _rootRun (dart:async/zone.dart:1126:13)
#32     _CustomZone.run (dart:async/zone.dart:1023:19)
#33     _runZoned (dart:async/zone.dart:1518:10)
#34     runZoned (dart:async/zone.dart:1465:12)
#35     AppContext.run (package:flutter_tools/src/base/context.dart:145:18)
#36     runInContext (package:flutter_tools/src/context_runner.dart:64:24)
#37     run (package:flutter_tools/runner.dart:50:10)
#38     main (package:flutter_tools/executable.dart:65:9)
#39     main (file:///private/tmp/flutter-20191213-6374-tobl26/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#40     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:305:32)
#41     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

Analyzing mssa-flutter...

   info • Name non-constant identifiers using lowerCamelCase • lib/generated/l10n.dart:41:14 • non_constant_identifier_names
   info • Name non-constant identifiers using lowerCamelCase • lib/generated/l10n.dart:50:10 • non_constant_identifier_names

2 issues found. (ran in 2.5s)

[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.2 19C57, locale nb-NO)
    • Flutter version 1.12.13+hotfix.5 at /usr/local/Cellar/flutter/stable
    • Framework revision 27321ebbad (7 weeks ago), 2019-12-10 18:15:01 -0800
    • Engine revision 2994f7e1e6
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/ovestorholt/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Users/ovestorholt/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/192.6071332/Android Studio 3.6 Preview.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3.1, Build version 11C504
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.6)
    • Android Studio at /Users/ovestorholt/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/192.6071332/Android Studio 3.6 Preview.app/Contents
    • Flutter plugin version 42.2.2
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] Android Studio (version 3.5)
    • Android Studio at /Users/ovestorholt/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.6010548/Android Studio.app/Contents
    • Flutter plugin version 42.1.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] Android Studio
    • Android Studio at /Users/ovestorholt/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-2/193.6137316/Android Studio 4.0 Preview.app/Contents
    • Flutter plugin version 39.0.5
    • Dart plugin version 193.2956.37
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[!] IntelliJ IDEA Ultimate Edition (version 2019.3.2)
    • IntelliJ at /Users/ovestorholt/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

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

[✓] Connected device (2 available)
    • Android SDK built for x86 • emulator-5554                        • android-x86 • Android 10 (API 29) (emulator)
    • iPhone 11 Pro Max         • D9C0D762-E002-484B-B177-E5E529313FCE • ios         • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)

! Doctor found issues in 1 category.

Looks like this is happening due to flutter/shell/platform/darwin/ios/framework/Source/FlutterEngine.mm:763
Apparently, we are prepending NSLocale.currentLocale to the beginning as explained by the comment:

// Force prepend the [NSLocale currentLocale] to the front of the list
// to ensure we are including the full default locale. preferredLocales
// is not guaranteed to include anything beyond the languageCode.

We should take a look and see if this is still the correct course of action.

Hi @ovestoerholt
Are you still facing this issue? If yes, can you please provide your flutter doctor -v, your flutter run --verbose and a minimal complete reproducible code sample?
Thank you

Hei @pedromassango
Have done some testing and localization seems to work without me needing to skip the index 0 from the system locale callback when trying to find a supported language.
So yeah! I think we're good.
Thanks! 👍


column type json is returning unknown | null type[BUG]:

What version of drizzle-orm are you using?
0.31.2
What version of drizzle-kit are you using?
0.22.8
Describe the Bug
import { pgTable, json } from "drizzle-orm/pg-core"

export const testTable = pgTable("testTable",{
	id: text("id").primaryKey().notNull(),
	testobject:json('testobject').array()
})

Expected behavior
The return type of column type json shouldn't be of type unknown
Environment & setup
No response

You need to give the type of your Json column like this:
export const testTable = pgTable("testTable",{
  id: text("id").primaryKey().notNull(),
  testobject:json('testobject').$type<YourType>().array()
})

You need to give the type of your Json column like this:
export const testTable = pgTable("testTable",{
  id: text("id").primaryKey().notNull(),
  testobject: json('testobject').$type<YourType>().array()
})

@noook Hey, would you know the reason as well by chance, why the return type of column type json is unknown?

If you mean by default, json is not necessarily an object, it could be any json serializable value, so a string, a number, an array, an object, or null.
So the safest to assume is that your column is of an unknown type. This type exists so that you must assert your variable type before using it, as opposed to any that allows you to do any operation on it without asserting the type.

Thanks @noook, I understood that but one thing I want to know is do we need to JSON.stringify() the data everytime while inserting?? with json type I'm talking about not jsonb because I have tried many ways but still its throwing the error TypeError: Value is not JSON serializable

@naeem-qv I don't believe you need to stringify, as a matter of fact I think this would defeat the purpose of saving data as json because you would insert a string instead.
Your error just seems to come from your data structure. I don't have enough information regarding your data, but if you have functions, date objects (not sure about this one as they have a .toString()  method), circular references, classes or any other non-serializable variable it would indeed throw this error.
What I suggest is to write a serializer for when you need to insert data. The serializer's role would be to pick and extract the properties you need so you're sure those are JSON serializable.

thanks @noook, you are right, json should have a type defined


Add support for array/number types and parser type in YAML

Add support for Array, Angle and Numeric types in YAML serialiser
Allow setting of YAML parser type, so faster C library can be used for parsing

Having some trouble with Python 3.5, which I believe is to do with dictionary ordering. Will have to set up Python 3.5 environment to debug properly.

Codecov Report

Merging #194 into master will increase coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #194      +/-   ##
==========================================
+ Coverage   91.92%   92.10%   +0.18%
==========================================
  Files         112      112
  Lines        3840     3877      +37
  Branches      556      557       +1
==========================================
+ Hits         3530     3571      +41
+ Misses        227      224       -3
+ Partials       83       82       -1

Impacted Files
Coverage Δ

stonesoup/reader/yaml.py
93.18% <100.00%> (ø)

stonesoup/serialise.py
99.25% <100.00%> (+2.34%)
:arrow_up:

stonesoup/types/angle.py
94.00% <0.00%> (+2.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Feature/acces single heads

Add accessing single-head activations in source_forward_pass and target_forward_pass methods of Patchscope
Warning This currently only works for models with the GPT2Attention implementation (https://github.com/huggingface/transformers/blob/v4.39.2/src/transformers/models/gpt2/modeling_gpt2.py#L123)
We probably need one implementation per attention architecture

Modify activation_patching_ioi.py to create a plot by layer & head

I think this PR is logically sound.
But do you want to merge https://github.com/obvslib/obvs/pull/41 first to fix CI before merging in this PR?

You know what, let's merge this. I'll rebase the other one.


ITCH.IO Filter Submission via Platform Error

Hallo,
ITCH.IO Filter Submission via Platform Error,
where: OUTPUT : submitted application does NOT appear
using: keyphrase: "reconnect"

observed: such OUTPUT for all of the following available platforms:
Play in browser
Windows
macOS
Linux
Android

Thank you. Health!

Hallo,
Thank you for the notification.
After verification, executed ACTION based on ITCH.IO references;

OUTPUT: submitted application now appears on the following available platforms,
using: keyphrase: "reconnect"
Windows
macOS
Linux
Android

At present, only the following platform does NOT display the application,
using: keyphrase: "reconnect"
Play in browser
--
Despite this, the following is displayed on the ITCH.IO project web page:
https://masarapmabuhay.itch.io/just-reconnect

PLAY ON WEB BROWSER
http://store.usbong.ph/gameoff

Thank you. Health!
References:

https://itch.io/updates/classify-your-projects-uploads; last accessed: 20221202

https://itch.io/t/211178/how-to-select-platform-for-game; last accessed: 20221202


Add tests for FileSystem storage

First tests should target the file system storage backend.

Should be solved by #20 ....


Add version to tk8

Add version command to tk8.
For example,
$ tk8 version
0.0.1

I am working on this issue.

Awesome @niki-1905 :)


Fix destroy on empty state

PR o'clock
Description
This PR addresses issue #568 by replacing hard-coded 0 index for accessing conditional resources with a call to join("", ...) and the splat syntax representing the resource.
aws_security_group.cluster[0].id => join("", aws_security_group.cluster.*.id)

I've tested the change with the basic example and the cluster was created without issues.
I've then destroyed it twice to test if (1) destroying the cluster worked and (2) destroying on an empty state worked as well. My tests were positive.
Checklist

[x] terraform fmt and terraform validate both work from the root and examples/* directories
[x] CI tests are passing
[x] I've added my change to CHANGELOG.md and highlighted any breaking changes
[x] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

You haven't updated workers_launch_template.tf but is there no more elegant way of fixing this? Perhaps @barryib know something

You haven't updated workers_launch_template.tf but is there no more elegant way of fixing this? Perhaps @barryib know something

I didn't have the need to change that file in order to fix the use case that was broken for me. But if you tell me what updates are you missing, I'll be happy to make them as part of this PR.

@barryib I've pushed the two changes you requested, and in addition I've applied the same principle to the workers.tf since there the resources are also conditional on the same variable as the local.

Hello .. I just hit the error below, and since it seems related, just want to be sure it will also be fixed in this PR
Error: Invalid index

  on .terraform/modules/cus01_cheapo_euwest.eks.eks/terraform-aws-modules-terraform-aws-eks-1be1a02/local.tf line 14, in locals:
  14:   worker_security_group_id  = var.worker_create_security_group ? aws_security_group.workers[0].id : var.worker_security_group_id
    |----------------
    | aws_security_group.workers is empty tuple

The given key does not identify an element in this collection value.

@max-rocket-internet can we merge this PR ? Another comments ?

Question: should I address the issue of @kim0 in this PR as well? It's not a lot of work for me so I'm happy to do that if needed.

I think this will solve @kim0 's problem. From his comment, you can see that the error is coming from the local.worker_security_group_id declaration in the local.tf file. And you already use the splat sintax there.

LGTM. Please fix the conflict so I can merge.

Conflict fixed. Thanks!

@miguelaferreira Thank you for your contribution.


HDDS-6420. Ratis and Standalone ReplicationConfig should return a static instance

What changes were proposed in this pull request?
There is generally a RatisReplicationConfig object stored in each ContainerInfo in SCM, and in each Pipeline too. New instances of the Ratis and Standalone objects are scattered throughout the code and used extensively in the tests.
Each object is basically identical - it only contains the factor, or ONE or THREE. We can save some memory and object churn if we just return a static value each time someone requests a Standalone/RatisReplicationConfig object.
Unfortunately this PR is quite large. I have split it into 3 commits:

Makes the changes in the non-test classes - the changes here are fairly small.

Replaces new RatisReplicationConfig() across the tests and make the constructor private

Replaces new StandalineReplicationConfig() across the tests makes the constructor private

The diff is large due to the number of places the replacement happens, but there is no logic change - its just a global replace and then some tidy up for checkstyle line lengths.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6420
How was this patch tested?
Existing tests - not real functionality changed.

Thanks @sodonnel for the patch.


Fix typos "should" and "calendar" in index.html , still remaining.

I would like to contribute

Hi , can you please assign me this task .


Appointment changes

What does this pull request do?
Moves Appointment into its own table.
Rename action_plan_appointment to session_delivery_appointment
What is the intent behind these changes?
Make Appointment generic so that it can be used for multiple appointment types.

Still need to look at fixing the sql report, but raising pr for early sight.

please can you include the jira ticket number in the PR title, so it links up with jira ?


fix(huggingface): fix not working default tool

The default tool in hugging-face - text-downloader is not working so I replaced it with custom run to avoid the error in tests.

coverage: 89.274% (-0.1%) from 89.378%
when pulling 1e9acdeef72c43e98a27363c07feec819a2a150c on 0-fix-hugginface-extensions
into 33ac9224bfe613da91fa46479a86a214cd09e0b1 on main.


Add commands to add or remove entries

To add changelog entries that are correctly formatted, there should also be a simple way using CLI commands.
This should check e.g. if there is an open pull request for the checked out branch, that the branch is not main or whatever the base branch is, and ideally already prefill the PR title as the changelog entry.

Update: For now there's no reason to add the remove subcommand for entries so it will be skipped.


UserWarning: Error while starting the viewer client.

Hi, I'm new to Pinocchio, and I followed the tutorial here. I already got so much help from @nim65s , and I wanna use gepetto-gui to visualize the robot.

What I ran:
usr_robot = load('ur5_gripper'). it didn't report error.

tiago_robot = load('tiago', display=True). This report:
In [5]: tiago_robot = load('tiago', display=True)
/opt/openrobots/lib/python3.6/site-packages/pinocchio/robot_wrapper.py:272: UserWarning: Error while starting the viewer client.
Check whether gepetto-viewer is properly started
  self.viz.initViewer(*args, **kwargs)

I followed here to install the gepetto, what I ran is:
sudo apt update && sudo apt install robotpkg-py36-qt5-gepetto-viewer-corba
And it's successful.
Reading package lists... Done
Building dependency tree
Reading state information... Done
robotpkg-py36-qt5-gepetto-viewer-corba is already the newest version (5.6.0).
The following packages were automatically installed and are no longer required:
  libomniorb4-2 libomniorb4-dev libomnithread4 libomnithread4-dev libpythonqt-qt5-python2-3 libpythonqt-qt5-python2-dev libpythonqt-qtall-qt5-python2-3
  libpythonqt-qtall-qt5-python2-dev omniidl omniidl-python omniorb-idl omniorb-nameserver python-omniorb python-omniorb-omg

Have I missed something here?

And when I ran gepetto-gui, the gui can be launched but I cannot choose robot from file, which is no robot I can choose.
I already set the export ROS_PACKAGE_PATH=$ROS_PACKAGE_PATH:/opt/openrobots/share

Thanks in advance.

You should run in another terminal gepetto-gui.
Other issues in Pinocchio already mentioned this issue.
Next time, please look at the history first. It will be quicker and simpler for everybody.

You should run in another terminal gepetto-gui.
Other issues in Pinocchio already mentioned this issue.
Next time, please look at the history first. It will be quicker and simpler for everybody.
Thank you so much and sorry for this.

@jcarpent Hi, sorry to ask this again. I can run the gepetto-gui when I load robot from another terminal. But the second thing I mentioned above:
when I ran gepetto-gui, the gui can be launched but I cannot choose robot from file as this video do, which is no robot I can choose.
I already set the export ROS_PACKAGE_PATH=$ROS_PACKAGE_PATH:/opt/openrobots/share
Have I missed something?

@hjw-1014 : this is a question on gepetto-viewer., and has nothing to do with pinocchio Please ask it on the right tracker.

@hjw-1014 : this is a question on gepetto-viewer., and has nothing to do with pinocchio Please ask it on the right tracker.

Understood, thank you.

@hjw-1014 If you troubles with gepetto-gui, just switch to the Meshcat viewer solution. It is simpler to install and use.

@jcarpent Thank you very much, I will search Meshchat viewer.


Make server psk identity comparison constant time

Resolved issues:
Addresses follow-up from https://github.com/awslabs/s2n/issues/2390
Description of changes:
Avoid leaking information about PSK identities known to the server. This is really more "better safe than sorry" than mandatory for security.
Testing:
How is this change tested (unit tests, fuzz tests, etc.)? Existing tests pass. It does not seem worth a sidetrail test.
Is this a refactor change? Yes. Tests exist already, they were not updated as part of this change, but they continue to pass.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2437 (4372e65) into main (28d64f6) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2437      +/-   ##
==========================================
- Coverage   81.92%   81.90%   -0.03%
==========================================
  Files         271      271
  Lines       18663    18661       -2
==========================================
- Hits        15290    15284       -6
- Misses       3373     3377       +4


Fixed issue #343 - Don't throw an exception when the data doesn't exist !

Fixed issue #343 - Don't throw an exception when the data doesn't exist
!!

Please help me review and merge the code, thank you.
@mariofusco @MalcolmOdd @tkobayas

This is a breaking change that will modify the behavior of other software depending on the fact that the exception is thrown when trying to access a missing property. Also this is not Javascript where every messed up expression is evaluated at any cost regardless of how wrong it is, always leading to surprising and inconsistent results. If I write an expression trying to use a not existing property I believe that the evaluation of that expression should fail fast and loudly.
For all these reasons I'm personally against this change, but I'm curious to know what others think about it and open to discuss.

I do not agree with this change.
The current behaviour is as intended, this is revealed by the existence of the '?' construct which explicitly  overrides it.
Also the proposed change breaks when executing the expression a second time since the accessor gets optimized (tested with testOrConditionIfPropertyNotExist()).

This is a breaking change that will modify the behavior of other software depending on the fact that the exception is thrown when trying to access a missing property. Also this is not Javascript where every messed up expression is evaluated at any cost regardless of how wrong it is, always leading to surprising and inconsistent results. If I write an expression trying to use a not existing property I believe that the evaluation of that expression should fail fast and loudly.
For all these reasons I'm personally against this change, but I'm curious to know what others think about it and open to discuss.

The non-existence of data does not mean that an error is reported, it should mean that the rules are met. In actual use, our data comes from various aspects. It is impossible to predict that the data must include the data in the rule expression. However, if an error is reported, the judgment of the data that meets the conditions will be terminated. This is not good. accomplish. for example:  It should have output true, but the result is exception.
public void testOrConditionIfPropertyNotExist() {
String expression = "name == 'joy' || age > 10";
Map<String, Object> paramMap = new HashMap<>();
paramMap.put("age", 15);
CompiledExpression compiledExpression = new ExpressionCompiler(expression).compile();
assertEquals(true, MVEL.executeExpression(compiledExpression, paramMap));
assertEquals(true, MVEL.eval(expression, paramMap));
}

This is a breaking change that will modify the behavior of other software depending on the fact that the exception is thrown when trying to access a missing property. Also this is not Javascript where every messed up expression is evaluated at any cost regardless of how wrong it is, always leading to surprising and inconsistent results. If I write an expression trying to use a not existing property I believe that the evaluation of that expression should fail fast and loudly.
For all these reasons I'm personally against this change, but I'm curious to know what others think about it and open to discuss.
@mariofusco
Javascript is support this.

mvel is an expression judgment rule, not a language, and various situations should be taken into consideration. As far as I know, the following rule engines support the situation we are discussing.
https://github.com/mithunsatheesh/node-rules
https://github.com/aws/event-ruler
Why can others support it, but mvel doesn't? In real environments, the data used for judgment is often unpredictable and unstructured.
@mariofusco @MalcolmOdd

Hi @vampireslove @mariofusco,
Just hit this issue on version 2.4.15.Final when an entry in the map does not exist: -
expression - "val='R'; if (additionalValues.setVisible) {val='D'};val");
[Error: could not access: setVisible; in class: java.util.HashMap]
[Near : {... val='R'; if (additionalValues.setVisible) { ....}]
Wondering if these changes are going to make it a new release and if so would it resolve my expression? So with the fix I would expect the if statement to be bypassed.
Thanks for the time, much appreciated.


Sims being shown too frequently

@tsboyajian and @Sleekery both report that they're getting a lot of sims.
This is randomised here after each subject is loaded and should only be showing sims 12.5% of the time.

@zooniverse/developers Does anyone have a minute to look at this? My hunch is that the summary screen is getting confused and labelling things as sims when they're not, but I'm swamped at the moment and haven't had a chance to investigate yet.


Update bip 158 test vectors

The bip 158 test vectors are now in json format (using the original file) and there is a new tested scenario. Additionally, the vectors use the new serialization which consists of N as a varint. This means
an empty filter is serialized as "00" instead of "" (empty).
NOTE: The NBitcoin code was already using this serialization format, that's why no changes in the code were required.
See this BIP PR for more info: https://github.com/bitcoin/bips/commit/fe1d3f84ba260bb39a49861564c6bbf1f371f801#diff-36d11b70d5a30a00085e9bc4699b35b1
By last, there were a missing changes after the dotnet update to 2.1 that are fixed here too.

Btw, the CI builds fail because neither travis nor appveyor support netcoreapp2.1 yet. We had the same problem with Wasabi Wallet today (See: https://github.com/zkSNACKs/WalletWasabi/pull/262#issuecomment-393416267)

ah shit, we should dockerize the tests


Why the difference between two versions？

Hello, I would like to ask if there is any special purpose for the different architectures in TensorFlow and PyTorch versions as shown in PMNet_torch_figure.png?

Hi @ThaliaHuang, We intended our model to support both TensorFlow and PyTorch. However, since we used a pre-trained audio embedder distributed as a TFLite model, using it directly in PyTorch was challenging. Therefore, in the PyTorch version, we modified the implementation so that the output of the pre-trained embedder would be saved separately and utilized during training. If you aim for complete reproducibility of our paper, we recommend using the TensorFlow version.


Tamil characters are broken

Hi
I found an issue with road-ways which is not displaying tamil text correctly.
https://www.openstreetmap.org/edit?editor=id#map=21/13.07963/80.28161
Road should display:முத்துசுவாமி சாலை
but it is splitted and showing like below.
ம+ு+த்த+ு+ச+ு+வ+ா+மி ச+ா+ல+ை
Please refer the unicode characters for tamil here.
http://www.fileformat.info/info/unicode/block/tamil/list.htm
in chrome console, i see error as below:
//Refused to load the font 'https://cdn.joinhoney.com/fonts/icons/honey-icons.woff?vbeftd' because it violates the following Content Security Policy directive: "font-src 'none'".//
Thanks
Mahir

Follow up for:
https://github.com/openstreetmap/openstreetmap-website/issues/2004
You didn’t mention the error message last time, so maybe this is really a csp issue.

My guess is that the CSP issue is unrelated - that's a browser extension trying to load some other icons.
I just tried it out on my machine (Chrome 68.0.3440.106 on macOS 10.13.6) and it does indeed look wrong.

The issue seems to be only with characters drawn along the textpaths, because the road name looks correct in the sidebar:

cc @mapmeld who knows much more about this than I do..

I just checked and Hindi has the same issue, only in roads, and I don't see it in Firefox (so, it's a Chrome/Webkit SVG textPath issue). Arabic is also doing something new with word order. I don't get the font-loading error - only thing which fails to download is piwik js.

My guess is that Chrome recently changed textPath in an unexpected and uncool way.
One idea which I saw just now, and hadn't considered before, is using  to embed a horizontal HTML span into the SVG on Chome. This wouldn't follow the road, but it would be legible. I think it's worth considering.

any fix for chrome. Firefox have no issue

any fix for chrome? Firefox have no issue

AFAIK nobody is working on this issue currently.  From @mapmeld's comment it sounds like a font shaping bug in Chrome's SVG text renderer, not something that iD is doing wrong.

Hi, I would like to explore this issue

I just checked and Hindi has the same issue
Please could you let me know what you did exactly, did you set your language in open street map or in chrome etc?

@TAQ2 It's a problem in Chrome (and related browsers, like Brave). It's not based on setting your language... it's more like if a road or other line label has complex text. A "grapheme cluster" like हि  will be split up ("ह ि") in the label

Ok then I don't understand how to recreate the error. When I click the OP's link and then click the road, all the writing is in English for me. Hence I thought the language should be changed

Please could you explain step by step how to recreate the scenario. Thanks :)

Thanks @TAQ2 for looking into this..
You can reproduce the error by using Chrome and creating a road and pasting the Tamil characters into the name of that road.
In this screenshot, I copied them from the nearby road "முத்துசுவாமி சாலை" which has both English and Tamil names.  (It's ok to draw a road like this for testing, but please don't save the map this way!).

Ok after many tests there seems to be no hack that will fix this. My simplified example of the error here for reference https://jsfiddle.net/xt1o6kg9/
There are a few related tickets on the chromium bug tracker that I have subscribed to for updates, namely:
https://bugs.chromium.org/p/chromium/issues/detail?id=927214
https://bugs.chromium.org/p/chromium/issues/detail?id=593570
https://bugs.chromium.org/p/chromium/issues/detail?id=374526
My recommendation for this ticket would be to add the label waitingfor and remove the help wanted label as there is nothing that can be done at this time

My recommendation for this ticket would be to add the label waitingfor and remove the help wanted label as there is nothing that can be done at this time

Sounds good - thanks for digging into the issue!

I have some good news - after a long long wait, the SVG textPath bug is fixed and it's in early release Chrome Canary / Chromium 96.0. This ends the need for OSM / iD to find another render option.
I'll make a PR soon to turn off our RTL patches on Chromium >= 96.0

This bug was fixed in today's stable version of Chrome / Chromium browser, and should be closed.


Allow going to same symbol

st=>start
end=>end
op=>operation: Operation
cond=>condition: Condition

st->cond
cond(yes)->op->op
cond(no)->cond

before:

after:

v1.6.3


Create news.js

Coverage remained the same at 67.866% when pulling 2b2714e1eac3b1ae693fa39af29a25b2d450ff7f on avi-hue:patch-1 into 9042e38719e38e1262151b2a835e10180f438b88 on microsoft:master.

Hi @avi-hue ,
Sorry, but we are not going to approve this PR. When opening a PR, please try to reference an existing issue that the PR resolves or addresses. Or include a description that describes what the PR accomplishes.


unable to add child

Describe the bug
when attempting to add a child from a card's view, it will not allow it and states
Cannot read property 'rank' of undefined
Environment:

chrome
Azure Devops

Screenshots
If applicable, add

screenshots

error in the devconsole (f12)

To Reproduce
Steps to reproduce the behavior:

Go to any card
Click on Add Child
Add a child title and press enter
observe error code at bottom of component

Expected behavior
a child task should be added to the card
Additional context
this worked when I first installed this add-in. it's quit working recently.

that did it! thanks.


Use of OpenSSL v3

This afternoon OpenSSL version 3.0.7 is released. A security fix that was tagged as CRITICAL is included. Is OpenSSL v3 currently used in Gardenlinux and do updates of clusters using Gardenlinux need to be scheduled in the short term?

https://mta.openssl.org/pipermail/openssl-announce/2022-October/000238.html

Garden Linux 576.x, which is our mainstream release is not affected. 934.0 which is currently in draft is affected and there will be a patch.

This afternoon OpenSSL version 3.0.7 is released. A security fix that was tagged as CRITICAL is included. Is OpenSSL v3 currently used in Gardenlinux and do updates of clusters using Gardenlinux need to be scheduled in the short term?
* https://mta.openssl.org/pipermail/openssl-announce/2022-October/000238.html

This should be patched ASAP of course,  but note that they downgraded the severity fromCRITICAL to HIGH since the above pre-announcement, see https://www.openssl.org/news/vulnerabilities.html#CVE-2022-3602

With the Garden Linux release 934.1 the openssl package has been updated to 3.0.7 (see also: https://github.com/gardenlinux/gardenlinux/releases/tag/934.1).


How do I uninstall ?

Hey! First of all, thanks for making this but I have to remove this due to some reasons. Is there anything I could do/ does an uninstall script exist to help me remove it?

Oh, I should definitely add this to the README, simply do the following to remove it:
rm -f ~/.local/share/nautilus-python/extensions/VSCodeExtension.py
rm -f ~/.local/share/nautilus-python/extensions/code-nautilus.py


Provide CLI and remote API options to force remove networks and endpoints

It is not possible to force remove an endpoint or a network through the cli and remote API.
If a network depends on many endpoints, we need to remove the endpoints first before removing the network. This would be convenient to provide a way to force remove a network, which means that the dependency is ignored or that all dependent endpoints are also deleted by this operation (the same way we are able to force remove containers and images with -f).

@abronan network and endpoint objects are global in nature and hence forcing a removal on a node can be communicated effectively across the cluster and perform the cleanup. But, unfortunately, the container (sandbox) that is attached to the endpoint is a locally scoped object and hence attach/detach event is done on the local node where the container exist. Also, a distributed forced removal requires strong lock support, which we don't make use of today in libnetwork.
Also, a -f on a container or image has a localized effect, whereas a forced removal of network will have a cluster-wide effect that will impact connectivity for many containers and even worse,we cannot guarantee bringing the containers back to the same networking state as before.
If there is a strong requirement for this feature request, then it has to go through a thorough design process.

Thanks for the clarification :+1: , yes the distributed nature of the operation: ie. spreading network configuration and information between machines makes the operation a lot more complex to implement. I guess that from a UI perspective I expected a -f to be present. As of now I'm creating a lot of endpoints and networks for testing and removing them is quite tedious.
For the containers being in an inconsistent state, I guess that -f implies that the containers attached to an endpoint that is force removed must be removed/re-started and re-attached to a new endpoint (for now, we can probably find better ways to handle that later on).
We can leave this on hold, the feature can wait but I feel like I will not be the only one expecting a -f flag to be present :smile:

@abronan Yes. We have a similar request from @shykes on atomic service swap, where it will replace the 2 step process that we have dettach from an existing service and attach to another service. The idea is to perform this operation from any of the host in the cluster. These distributed configuration enhancements can all be considered under the same design effort.


refactor(booksAPI): refactor getCommonsCatalog

Current dependencies on/for this PR:

master

PR #128

PR #129

PR #130

PR #131   👈

This comment was auto-generated by Graphite.


Polish around default binding IllegalStateException

If no binders were found the exception should reflect that instead of indicating that multiple binders were found.
Use case:

spring-cloud-stream-test-support is on the classpath
An @Configuration class contains @EnableBinding that comes into play in the test
Either auto configuration is disabled altogether or via a test slice, like in my case with @DataMongoTest

In this case the fix is that you need to manually @Import(TestSupportBinderAutoConfiguration.class). But the IllegalStateException that you get is confusing:
...
...
Caused by: java.lang.IllegalStateException: A default binder has been requested, but there is more than one binder available for 'org.springframework.integration.channel.DirectChannel' : , and no default binder has been set.
	at org.springframework.cloud.stream.binder.DefaultBinderFactory.getBinder(DefaultBinderFactory.java:127)
	at org.springframework.cloud.stream.binding.BindingService.getBinder(BindingService.java:308)
	at org.springframework.cloud.stream.binding.BindingService.bindProducer(BindingService.java:208)

Dunno what's up here with Travis, it's got a checkstyle issue with ContentTypeTckTests which is not within the purview of the changes I made.
I briefly merged this into master to see if there was anything required there, but it looked like it merged cleanly even with the differences in DefaultBinderFactory. And it still only hits this code when there is not exactly 1 binder found and it goes back to doGetBinding() legacy behavior.
Let me know if there's anything you all need me to change to get this merged in!

@phillipuniverse can you please re-issue this PR on the master instead of 2.0.x branch and then I can back port it, otherwise its a bit more complicated with merge conflicts.

@olegz no problem, superseded by #1561.


fix: render emoji

[x] I am sure that the code is update-to-date with the dev branch.

close #1190

perfect


Avoid build protobuf

setup.py does not need anymore the commands to build protobuf, since
they are generated inside the repo

Thanks.


I am still having issues getting the syntax right and so I was unable…

Comfort: 2
Clarity: 4

Wanted to get it in before I ran out of time
I was unable to test my spec because my syntax was incorrect. This made it impossible to check my answers in time because I was spending all my time trying to get the right syntax


Adding ability to remove self from a group chat

Hello,
I can't seem to find a way with this api to remove the user I use from a group chat.
I've tried api.removeUserFromGroup but it doesn't seem to work.

Could you show us your code? Did you noticed any error messages?

Sorry, but can't show the code right now.
And no, there aren't any errors it works fine whwn removing other users from a group but doesn't work with the user I'm using.

Yes. First I got the user ID from the function then wrote it to a file so I don't use the function every time.

I am also trying this API from master branch and got the following error when calling removeUserFromGroup with the current user ID (from api.getCurrentUserID) and threadID (from api.getThreadList).
{ __ar: 1,
  error: 1545003,
  errorSummary: 'Invalid Action',
  errorDescription: 'You cannot perform that action.',
  payload: null,
  bootloadable: {},
  ixData: {},
  bxData: {},
  gkxData: {},
  qexData: {},
  lid: '<redacted>'
}


Fixed Clock4 to catch up with missed frames when paused

Pausing the clocks when hovering over other clocks caused Clock 04 (the rainbow clock) to skip frames when it starts up again.
This commit fixes that by making sure that the clock catches up on the missed frames, so it can smoothly continue in its rainbow glory.
Also, I'm not sure if I did anything wrong so that it couldn't be merged automatically - if so, sorry! (And please tell me what it was so I can not do it again).

Looks good. I'll fix the merge conflict!


Delete manifests/j/JanDeDobbeleer/OhMyPosh/7.62.1 directory

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Example generator not complaining about contradictions

Describe the bug
When using allOf for describing a message, and the schemas don't allow for a legal message, I still get an example. Only that example wouldn't validate.
How to Reproduce
In a visual studio code instance, with the official asyncApi preview extension installed (marketplace link), open this yaml file:
asyncapi: '2.6.0'
info:
  title: Contradictory demo
  version: 1.0.0
defaultContentType: application/json
servers:
  ondevice:
    url: localhost
    protocol: mqtt
    bindings:
      mqtt:
        bindingVersion: 0.1.0
        cleanSession: false
channels:
  /message:
    publish:
      operationId: messageOperation
      message:
        payload:
          allOf:
            - type: object
              additionalProperties: false
              required:
                - prop1
              properties:
                prop1:
                  type: string
            - type: object
              additionalProperties: false
              required:
                - prop2
              properties:
                prop2:
                  type: string

The preview will contain an example containing both properties:

Expected behavior
I would have expected

either an error
or a message in the example section, detailing that it couldn't create a valid example.

Hello @jonaslagoni !
I'm sorry to mention you directly, but when I created this bug there was apparently a slack error and I fear this bug has been lost in the process.
So I just opened another bug with message traffic and you are the unlucky one. :-(
Maybe you could help me figuring out whether something went wrong during the creation of this bug? My goal is just that someone starts sorting it in or whatever is necessary to get the process going.

@vhe1 this example JSON is generated the AsyncAPI React component if I am not mistaken.
Regardless, it's impossible (or near it) to detect JSON Schema semantic errors as you have here, the examples are just generated on a "you (as the creator of the schema) know what your doing" approach, i.e. in a best-effort type of approach.
This is something a validator would probably want to give errors on, but as I mentioned, I don't think is possible to do on arbitrary syntactically correct JSON Schema inputs.
So I don't think this constitutes as a bug. Cause the example is generated as intended IMO.
You could try raise the issue on the AsyncAPI slack channel or in the other repo, but yea, sorry, cant help you any further then that 🙂

no followup, closing


Draft: Req serving isolation e2e enxebre

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/retest-required

/retest

/test e2e-kubevirt-aws-ovn

/retest

/test all

/test all

/test all

/test all

/test all

/test e2e-aws
/test e2e-kubevirt-aws-ovn

/jira refresh

/test all

/test e2e-kubevirt-aws-ovn

would like to see it pass a couple more times
/test e2e-aws

/approve
/lgtm


Where is the API documentation?

docs/index.html refers to http://downloads.jasig.org/cas-clients/php/1.3.6/docs/api/ which is a 404, and I don't see any mention of API documentation at the official phpCAS client page, nor anywhere inside this repo.  BTW, where is docs/index.html hosted, anyway?  It's not what's displayed on the CASC wiki page; if it's not actually used anywhere, wouldn't a markdown or RST version be better, so GitHub would render it?
  Perhaps the API docs on jasig.org were recently turned off?
Apologies if I'm missing something obvious.

You are absolutely right. Most of our stuff broke when the old jasig servers were retired. I have made some interim changes and the API docs can be found under:
https://apereo.github.io/phpCAS/


left in right documentation is incorrect?

Elixir version
1.14.4
Database and Version
MySQL 8.0.30
Ecto Versions
3.10.1
Database Adapter and Versions (postgrex, myxql, etc)
myxql 0.6.3
Current behavior
Current Docs
from c in Comment, where: c.post_id in subquery(
  from(p in Post, where: p.created_at > ^since)
)

...is the current documented example. I was curious to see how this "inferred" that the in was deducing that the id column of Post was the comparison.
Emulating that in my DB with similar schema in my db I tried:
from c in Sct.Internal.Tenant, where: c.org_id in subquery(
   from(x in Sct.Network.Collective)
)

which returns the query
 #Ecto.Query<from t0 in Sct.Internal.Tenant,
 where: t0.org_id in subquery(#Ecto.Query<from c0 in Sct.Network.Collective>)>

However fetching the data results in ...
from c in Sct.Internal.Tenant, where: c.org_id in subquery(
   from(x in Sct.Network.Collective)
) |> Sct.Repo.all

** (Ecto.Query.CompileError) Sct.Repo.all(c.org_id in subquery(from(x in Sct.Network.Collective))) is not a valid query expression.
From that I deduced that the example was missing proper parentheses (even though it did interpret the Query without them. So I added parentheses>>>
from(c in Sct.Internal.Tenant, where: c.org_id in subquery(from(x in Sct.Network.Collective))) |> Sct.Repo.all

... which led me to the answer I was investigating
** (Ecto.QueryError) subquery must return a single field in order to be used on the right-side of `in` in query:

from t0 in Sct.Internal.Tenant,
  where: t0.org_id in subquery(
  #Ecto.Query<from c0 in Sct.Network.Collective, select: %{
  id: c0.id,
  name: c0.name,
  active: c0.active,
  source: c0.source,
  community_id: c0.community_id,
  eusage: c0.eusage,
  expose: c0.expose,
  keyword: c0.keyword,
  notes: c0.notes,
  permalink: c0.permalink,
  type: c0.type,
  tenant_id: c0.tenant_id,
  created_at: c0.created_at,
  updated_at: c0.updated_at
}>
),
  select: t0

    (elixir 1.14.4) lib/enum.ex:2468: Enum."-reduce/3-lists^foldl/2-0-"/3
    (elixir 1.14.4) lib/enum.ex:2468: Enum."-reduce/3-lists^foldl/2-0-"/3
    (ecto 3.10.1) lib/ecto/repo/queryable.ex:211: Ecto.Repo.Queryable.execute/4
    (ecto 3.10.1) lib/ecto/repo/queryable.ex:19: Ecto.Repo.Queryable.all/3

Expected behavior
So to summarize:
The query as written is most likely missing 2 elements:

The initial from should have an outer parentheses around the whole example.
The subquery MUST return a column for comparison using select (which is significant to this example)

Corrected query (IMHO):
from(c in Comment, where: c.post_id in subquery(
  from(p in Post, where: p.created_at > ^since, select: p.id)
)
)

Hello, @tjchambers, thanks for the report!
The example is indeed wrong about not selecting a field. I opened a PR fixing it: https://github.com/elixir-ecto/ecto/pull/4177
Regarding the parenthesis, the issue that you have faced is related to the pipeline operator associating to part of the query expression. You could, for example, assign it to a variable and use Repo.all/1 on it. So I'm not sure we should fix it. I think we have similar code in many other examples.


[Bug]: preview deployments ignore specified ports in domains section for docker compose build pack

Description
Specified ports for domains section were ignored for preview deployment builds.
I've checked /data/coolify/applications/[id] and for main build it has the right labels:
cat docker-compose.yml:
- 'caddy_0.encode=zstd gzip'
- 'caddy_0.handle_path.0_reverse_proxy={{upstreams 3000}}'
- 'caddy_0.handle_path=/*'
- caddy_0.header=-Server
- 'caddy_0.try_files={path} /index.html /index.php'
- 'caddy_0=https://[domain].ua'
- caddy_ingress_network=[id]
- coolify.managed=true
- coolify.version=4.0.0-beta.277
- coolify.applicationId=3
- coolify.type=application
- coolify.name=front-[id]-155945191632
- coolify.pullRequestId=0

but not for preview deployments:
cat docker-compose-pr-1.yml:
- 'caddy_0.encode=zstd gzip'
- 'caddy_0.handle_path.0_reverse_proxy={{upstreams}}'
- 'caddy_0.handle_path=/*'
- caddy_0.header=-Server
- 'caddy_0.try_files={path} /index.html /index.php'
- 'caddy_0=https://1.[domain].ua'
- caddy_ingress_network=[id]
- coolify.managed=true
- coolify.version=4.0.0-beta.277
- coolify.applicationId=3
- coolify.type=application
- coolify.name=front-[id]-pr-1
- coolify.pullRequestId=1

Minimal Reproduction (if possible, example repository)

Exception or Error
HTTP ERROR 503
Version
4.0.0-beta.277

Could you please provide me with a sample repository or compose file where you see the issue? This will help us investigate and resolve the issue. If this issue has been resolved by now, no worries, I will close this.


Fixed chunkmemset should be called on larger chunks assert

When compressing fireworks.jpeg using deflate level 1 and mem level 1, during inflate the following assert would be hit:
chunkmemset should be called on larger chunks

This PR fixes that by providing a path for copying lengths that are less than sizeof(uint64_t) when using INFFAST_CHUNKSIZE. Also see #284.

It would be good to get some reviewers my usage of the parameters passed into these functions. I'm pretty sure they are right.

Yep, I'd like someone who knows the inffast chunkcopy code better than me to review this.

Did you happen to benchmark this to see whether it made any difference?

I didn't do any benchmarks, but I saw the code I was using wasn't working in a certain instance so I decided to close the PR. The assert still exists in the code, because len < sizeof(uint64_t). In the first change lines I made, that would be okay because of the comment above (although an assert is still thrown).


Set AutoSize for columns to multiple sheets

Hi!
I'm getting problems to set AutoSizeColumns when there multiple sheets.
I think this is a bug.
I do not find any solution for this.
Anybody had this problem?

Duplicated  of #26

For me does not seem to be duplicated.
I have many spreadsheets in the same file (xlsx) and autosize doesn't not work at all.

@pedrofernandesfilho Maybe custom column for each line is the workaround.
                for (int k = 0; k <= dt.Columns.Count; k++)
                {
                    sheet1.AutoSizeColumn(i);
                }

                for (int columnNum = 0; columnNum <= dt.Columns.Count; columnNum++)
                {
                    int columnWidth = sheet1.GetColumnWidth(columnNum) / 256;
                    for (int rowNum = 0; rowNum <= sheet1.LastRowNum; rowNum++)
                    {
                        IRow currentRow;
                        if (sheet1.GetRow(rowNum) == null)
                        {
                            currentRow = sheet1.CreateRow(rowNum);
                        }
                        else
                        {
                            currentRow = sheet1.GetRow(rowNum);
                        }

                        if (currentRow.GetCell(columnNum) != null)
                        {
                            ICell currentCell = currentRow.GetCell(columnNum);
                            int length = Encoding.GetEncoding("gb18030").GetBytes(currentCell.ToString()).Length;
                            if (columnWidth < length)
                            {
                                columnWidth = length;
                            }
                        }
                    }
                    sheet1.SetColumnWidth(columnNum, columnWidth * 256);
                }


[Feat] Production Widgets

Is your feature request related to a problem? Please describe.
Only the following widgets should enter production:
accident_count_by_severity
most_severe_accidents_table
most_severe_accidents
head_on_collisions_comparison_percentage
As more and more widgets will be ready, we'll add them one by one to production.
@galraij @yuvalbl FYI

@9Syncopation would you like to take this one?

@9Syncopation would you like to take this one?

just to make sure I get it right:
add const productionCards with the list mentioned above and at .env.production verify DEMO_CARDS and OPER_CARDS are false?

No, I meant just update  operationalCards.
see https://github.com/hasadna/anyway-newsflash-infographics/pull/581


refactor(ast): remove JsImportNamespaceClause

Summary
While I was working on #428, I noticed that a namespace import, such as import * as X from "", can be represented in two ways. One way uses JsImportNamesapceClause and the other use JsNamespaceImportSpecifier embedded inside a JsNamedImportClause with a missing JsDefaultImportSpecifier.
This PR removes JsImportNamespaceClause and ensure that we have a unique way of representing a namespace import.
I think we should refactor the import AST nodes further. There is room for better factorization.
Test Plan
Updated snapshots.

I am closing this PR because I think my alternative proposal is better

I am closing this PR because I think my https://github.com/biomejs/biome/discussions/436 is better


Fixed case sensitive check of farm admins resource

@ykuijs Can I get you to review this one dude since it was your resource to begin with? The issue is that I was seeing SharePoint return "domain\user" where as my config had "Domain\User" (casing was the only difference) and it would fail a test, despite the user being in the group already. Just switched the Contains() method out for the PowerShell contains operator which isn't case sensitive.

Looks perfect!


Object Spread should not invoke setter.

Current stage-3 spec requires object spread should define new properties on rest object, not set value on rest object.
Spec:
http://sebmarkbage.github.io/ecmascript-rest-spread/#AbstractOperations-CopyDataProperties
More readable example:
https://github.com/sebmarkbage/ecmascript-rest-spread/blob/master/Issues.md#setters
TypeScript Version:  nightly
Code
let record = { x: 1 };
let obj = { set x(a) { throw new Error(); }, ...record }; // not error
obj.x; // 1

Expected behavior:
No runtime error.
Actual behavior:
Runtime error.
Note: Babel does not define properties by default either.

Is this a curious issue of spec compliance, or a real-world thing?

Purely compliance issue, I haven't seen any real world problem. Not even in Babel's issue list.

It's not just a question of spec compliance. First of all, Babel implements this behavior so people migrating from Babel to TypeScript may introduce bugs into their codebases. Secondly, this is important for some security-enhanced environments as it's possible to monkey-patch methods like Object.assign but not syntax like object spread. See the meeting notes of TC39 regarding this decision: https://tc39.github.io/tc39-notes/2014-09_sept-25.html#58-object-rest-destructuring-and-spread-properties

@RyanCavanaugh based on some of your more recent comments I think you are not actively interested in issues like this. Perhaps this one can be closed?


the validation set should be used instead of the test set in train_vq.py

It seems that in train_vq.py the validation set should be used instead of the test set to save the best fid model?

 It's probably a problem left over from some RVQ ablation experiments. We have fixed it. Thanks.


[otbn/sw] Add missing OTBN ECDSA sign R value to test verification

The ECDSA signature verification step requires the R value as input.
In the OTBN test code, a signing is done followed by a verification.
Because the correct R value already exists in OTBN memory,
we get a successful signature verification. However when clearing
OTBN memory between the signing and verification step, the
verification step fails. Adding the R value to verification step
solves problem. Fix for #5626
Signed-off-by: Thomas Rosted Jensen<EMAIL_ADDRESS>

@felixmiller can you have a quick look from an API perspective?

Thank you for the updates, I just merged your changes.
Congratulations @TRJensen to your first ever pull request, and thanks for making OpenTitan your project of choice for such a memorable event :fireworks: :tada:

Thank you @imphil 🎉🎊☺👍


Add default amount of 0 to transactions (excluding type 0)

The abstract transaction that is extended should always have a default amount of 0 and require type 0 to have an amount greater than 0.

Addressed in #80

Resolved in #85
This issue can now be closed, @faustbrian


Adding flag to ignore missing stats messages

@hlinnaka please take a look

Pushed to merge_8_3_22, thanks!


E03: Unable to open swap file for "[No Name]", recovery impossible

nvim-qt, win10
Get those error when :source $MYVIMRC, how can I fix this?

This is caused by the swap file settings in vimrc, rather than this plugin.


Arnej/refactor document type manager configurer

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@baldersheim please review

There was test failure that seemed unrelated. I triggered another build. Will merge once green.


git head failed to build target win32

because variable HOST_DC not set (from user point of view, not CI)

2.067.1: needn't this.
2.068: fails to build on local install but ok with this patch.
git head: ditto.

appologies if this variable is set in the D organisation infrastructure but for a user it seems to be required (typically dmd is found with PATH)

Unfortunately this causes HOST_DC to override the environment setting.  Even if dmd is found with PATH, the dmd.exe in the build dir will take priority (if it has been build).


gen: add two configuration parameters for container IPC.

High-level description
What features does this change enable? What bugs does this change fix?
Add two configuration parameters for container IPC.
Corresponding DC/OS tickets (required)

DCOS-56619 Add IPC related configuration parameters in DC/OS

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
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@qianzhangxa  Is it possible to write a test for these changes?

@qianzhangxa  Also, do we need any CHANGELOG update for these flags?

@vinodkone @Gilbert88 I have updated the CHANGES.md and also added a test, thanks!

@mesosphere-mergebot label Ship It


[CURL] Remove unnecessary renaming.

CMake updated the FindCURL module and the renaming is no longer required.
From FindCURL.cmake in CMake:
  find_library(CURL_LIBRARY_RELEASE NAMES
      curl
    # Windows MSVC prebuilts:
      curllib
      libcurl_imp
      curllib_static
    # Windows older "Win32 - MSVC" prebuilts (libcurl.lib, e.g. libcurl-7.15.5-win32-msvc.zip):
      libcurl
      HINTS ${PC_CURL_LIBRARY_DIRS}
  )
  mark_as_advanced(CURL_LIBRARY_RELEASE)

  find_library(CURL_LIBRARY_DEBUG NAMES
    # Windows MSVC CMake builds in debug configuration on vcpkg:
      libcurl-d_imp
      libcurl-d
      HINTS ${PC_CURL_LIBRARY_DIRS}
  )
  mark_as_advanced(CURL_LIBRARY_DEBU

regressions? probably a few ports which come with there own FindCURL module which can simply be removed.

Hi @Neumann-A, thanks for your PR.


Consider publishing on Open VSX

Open VSX is an Eclipse open-source project and alternative to the Visual Studio Marketplace. It's used by default by other VSCode-based IDEs and editors such as VSCodium or Theia IDE.

Sure...I will publish it soon..Thanks for letting me know😃

Open VSX is an Eclipse open-source project and alternative to the Visual Studio Marketplace. It's used by default by other VSCode-based IDEs and editors such as VSCodium or Theia IDE.

I published the extension in Open VSX..here is the link
https://open-vsx.org/extension/dreamcatcher45/podmanager


Mod icon generator

Generate mod icon based on the mod name, in Comic Sans (of course).
The Deno CLI needs --unstable as it uses native lib skia_canvas, as the wasm canvas has a limitation that needed to be able to resize the text properly.

A screenshot of the generated icon in the PR desc would also help visualise it

Resolved the comments 👍

Found a canvas library that works without FFI.
Unfortunately, it is an NPM package, emit doesn't seem to support bundling it currently.
Blergh.

Found a canvas library that works without FFI. Unfortunately, it is an NPM package, emit doesn't seem to support bundling it currently. Blergh.

Oh awesome, have you tried updating deno (I know they have been working on improving NPM support) seems the easyist thing to start with? The error its spitting out is a little odd const ret = new Error(getStringFromWasm0(arg0, arg1)); as I dont see any references to wasm in its github repo.
If that doesnt work I can possibly take a look at it, one possible hack would be to build a js bundle with it using npm from the main website build, and then import that into the CLI in the same way we import the template stuff.

Seems to be working with esbuild instead of emit.


Constitution should be set/configurable for new contracts

Expected Behavior
Consitution set for StableTokenEUR, ExchangeEUR, new methods, etc.
Current Behavior
Default of simple majority taken for all new contracts/endpoints

related https://github.com/celo-org/celo-monorepo/issues/6411
https://github.com/celo-org/celo-monorepo/issues/6412

@Tjudice good candidate


Unable to start Jupyter on Mac (Silicon) Monterey

OS: Mac Silicon (M1) Monterey
Problem

Unable to run a cell as Jupyter fails to start

Logs
File "/opt/homebrew/lib/python3.9/site-packages/traitlets/config/loader.py", line 933, in __call__
namespace._flags.append(self.flag)
AttributeError: 'Namespace' object has no attribute '_flags'

We believe this could resolved by updating traitlets to the latest version as follows python -m pip install 'traitlets>=5.1.1.
More information on this can be found here https://discourse.jupyter.org/t/attributeerror-namespace-object-has-no-attribute-flags-when-running-jupyter-kernelspec-list-json/11690

Solution:

Update traitlets to the latest version as follows python -m pip install 'traitlets>=5.1.1
Please ensure you install traitlets into the Python enviornment used to start Jupyter
To determine the Python environment used to start Jupyter, please do the following

Select the command 'Jupyter: Select Interpreter to start Jupyter Server`
In the quickpick the currently selected interrpeter will be displayed in the intput box as a placeholder.
Please install traitlets into this environment

Please test using this test plan https://github.com/microsoft/vscode-jupyter/issues/8357

/verified with testplan


Release 1.0.24 breaks diesel

First of all, thank you for maintaining this crate! Unfortunately, since release 1.0.24 diesel fails to compile:
   Compiling diesel_derives v2.0.1
error[E0277]: the trait bound `FieldAttr: ToTokens` is not satisfied
   --> /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/diesel_derives-2.0.1/src/attrs.rs:143:6
    |
143 | impl Spanned for FieldAttr {
    |      ^^^^^^^ the trait `ToTokens` is not implemented for `FieldAttr`
    |
    = help: the following other types implement trait `ToTokens`:
              &'a T
              &'a mut T
              Abi
              Abstract
              AndAnd
              AndEq
              AngleBracketedGenericArguments
              Arm
            and 313 others
    = note: required because of the requirements on the impl of `quote::spanned::private::Sealed` for `FieldAttr`
note: required by a bound in `quote::spanned::Spanned`
   --> /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/quote-1.0.25/src/spanned.rs:6:20
    |
6   | pub trait Spanned: private::Sealed {
    |                    ^^^^^^^^^^^^^^^ required by this bound in `quote::spanned::Spanned`

error[E0277]: the trait bound `StructAttr: ToTokens` is not satisfied
   --> /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/diesel_derives-2.0.1/src/attrs.rs:242:6
    |
242 | impl Spanned for StructAttr {
    |      ^^^^^^^ the trait `ToTokens` is not implemented for `StructAttr`
    |
    = help: the following other types implement trait `ToTokens`:
              &'a T
              &'a mut T
              Abi
              Abstract
              AndAnd
              AndEq
              AngleBracketedGenericArguments
              Arm
            and 313 others
    = note: required because of the requirements on the impl of `quote::spanned::private::Sealed` for `StructAttr`
note: required by a bound in `quote::spanned::Spanned`
   --> /home/runner/.cargo/registry/src/github.com-1ecc6299db9ec823/quote-1.0.25/src/spanned.rs:6:20
    |
6   | pub trait Spanned: private::Sealed {
    |                    ^^^^^^^^^^^^^^^ required by this bound in `quote::spanned::Spanned`

For more information about this error, try `rustc --explain E0277`.
error: could not compile `diesel_derives` due to 2 previous errors

I believe making a trait that was publicly implementable requiring a private seal is a breaking change and should not happen in a patch release.

Sorry didn't see the other issue


Errors when attempting to build Test Project

Following the instructions and attempting to use the files found in test_poject to build the project. But when I get to the last step to make within the build folder while in a terminal in VS Code I receive a large number of errors:
/home/dev/build # cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=1 ..
PICO_SDK_PATH is /usr/share/pico_sdk
PICO platform is rp2040.
Build type is Release
PICO target board is pico.
Using board configuration from /usr/share/pico_sdk/src/boards/include/boards/pico.h
TinyUSB available at /usr/share/pico_sdk/lib/tinyusb/src/portable/raspberrypi/rp2040; enabling build support for USB.
BTstack available at /usr/share/pico_sdk/lib/btstack
cyw43-driver available at /usr/share/pico_sdk/lib/cyw43-driver
Pico W Bluetooth build support available.
lwIP available at /usr/share/pico_sdk/lib/lwip
mbedtls available at /usr/share/pico_sdk/lib/mbedtls
-- Configuring done
-- Generating done
-- Build files have been written to: /home/dev/build
/home/dev/build # make
[  1%] Creating directories for 'ELF2UF2Build'
[  2%] No download step for 'ELF2UF2Build'
[  3%] No update step for 'ELF2UF2Build'
[  4%] No patch step for 'ELF2UF2Build'
[  5%] Performing configure step for 'ELF2UF2Build'
-- The C compiler identification is GNU 12.2.1
-- The CXX compiler identification is GNU 12.2.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: /home/dev/build/elf2uf2
[  6%] Performing build step for 'ELF2UF2Build'
[ 50%] Building CXX object CMakeFiles/elf2uf2.dir/main.cpp.o
[100%] Linking CXX executable elf2uf2
[100%] Built target elf2uf2
[  7%] No install step for 'ELF2UF2Build'
[  8%] Completed 'ELF2UF2Build'
[  8%] Built target ELF2UF2Build
Scanning dependencies of target bs2_default
[  9%] Building ASM object pico-sdk/src/rp2_common/boot_stage2/CMakeFiles/bs2_default.dir/compile_time_choice.S.o
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S: Assembler messages:
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:89: Error: unknown pseudo-op: `.syntax'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:89: Error: unknown pseudo-op: `.cpu'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:89: Error: unknown pseudo-op: `.thumb'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:98: Error: unknown pseudo-op: `.thumb_func'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:99: Error: invalid char '{' beginning operand 1 `{lr}'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:105: Error: no such instruction: `ldr r3,=0x40020000'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:106: Warning: `r0' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:106: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:107: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:108: Error: no such instruction: `ldr r0,[r3,'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:109: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:109: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:110: Error: no such instruction: `bics r0,r1'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:111: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:112: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:113: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:114: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:116: Error: no such instruction: `ldr r3,=0x18000000'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:119: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:119: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:120: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:123: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:123: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:124: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:132: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:132: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:133: Warning: `r2' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:133: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:134: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:145: Error: no such instruction: `ldr r1,=((7<<16)|(0x0<<8))'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:146: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:149: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:149: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:150: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:153: Warning: `r0' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:153: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:154: Error: no such instruction: `bl read_flash_sreg'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:155: Warning: `r2' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:155: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:156: Error: too many memory references for `cmp'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:157: Error: no such instruction: `beq skip_sreg_programming'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:160: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:160: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:161: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:164: Error: no such instruction: `bl wait_ssi_ready'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:165: Error: no such instruction: `ldr r1,[r3,'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:168: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:168: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:169: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:170: Warning: `r0' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:170: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:171: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:172: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:174: Error: no such instruction: `bl wait_ssi_ready'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:175: Error: no such instruction: `ldr r1,[r3,'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:176: Error: no such instruction: `ldr r1,[r3,'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:177: Error: no such instruction: `ldr r1,[r3,'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:181: Warning: `r0' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:181: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:182: Error: no such instruction: `bl read_flash_sreg'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:183: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:183: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:184: Error: no such instruction: `tst r0,r1'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:185: Error: no such instruction: `bne 1b'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:190: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:190: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:191: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:208: Error: no such instruction: `ldr r1,=((0x2<<21)|(31<<16)|(0x3<<8))'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:209: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:211: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:211: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:212: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:222: Error: no such instruction: `ldr r1,=((8<<2)|(4<<11)|(0x2<<8)|(0x1<<0))'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:223: Error: no such instruction: `ldr r0,=(0x18000000+0x000000f4)'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:224: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:226: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:226: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:227: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:229: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:229: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:230: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:231: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:231: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:232: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:235: Error: no such instruction: `bl wait_ssi_ready'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:241: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:241: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:242: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:259: Error: no such instruction: `ldr r1,=((0xa0<<24)|(8<<2)|(4<<11)|(0x0<<8)|(0x2<<0))'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:260: Error: no such instruction: `ldr r0,=(0x18000000+0x000000f4)'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:261: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:263: Warning: `r1' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:263: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:264: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:16: Error: invalid char '{' beginning operand 1 `{r0}'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:17: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:18: Error: no such instruction: `beq vector_into_flash'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:19: Error: no such instruction: `bx r0'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:21: Error: no such instruction: `ldr r0,=(0x10000000+0x100)'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:22: Error: no such instruction: `ldr r1,=(0xe0000000+0x0000ed08)'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:23: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:24: Error: no such instruction: `ldmia r0,{r0,r1}'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:25: Error: no such instruction: `msr msp,r0'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/exit_from_boot2.S:26: Error: no such instruction: `bx r1'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:11: Error: invalid char '{' beginning operand 1 `{r0'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:16: Error: no such instruction: `ldr r1,[r3,'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:17: Warning: `r0' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:17: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:18: Error: no such instruction: `tst r1,r0'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:19: Error: no such instruction: `beq 1b'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:20: Warning: `r0' is not valid here (expected `(%rsi)')
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:20: Error: expecting operand after ','; got nothing
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:21: Error: no such instruction: `tst r1,r0'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:22: Error: no such instruction: `bne 1b'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/wait_ssi_ready.S:24: Error: invalid char '{' beginning operand 1 `{r0'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/read_flash_sreg.S:16: Error: unknown pseudo-op: `.thumb_func'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/read_flash_sreg.S:18: Error: invalid char '{' beginning operand 1 `{r1'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/read_flash_sreg.S:19: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/read_flash_sreg.S:21: Error: too many memory references for `str'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/read_flash_sreg.S:23: Error: no such instruction: `bl wait_ssi_ready'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/read_flash_sreg.S:25: Error: no such instruction: `ldr r0,[r3,'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/read_flash_sreg.S:26: Error: no such instruction: `ldr r0,[r3,'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/asminclude/boot2_helpers/read_flash_sreg.S:28: Error: invalid char '{' beginning operand 1 `{r1'
/usr/share/pico_sdk/src/rp2_common/boot_stage2/boot2_w25q080.S:280: Error: unknown pseudo-op: `.ltorg'
make[2]: *** [pico-sdk/src/rp2_common/boot_stage2/CMakeFiles/bs2_default.dir/build.make:75: pico-sdk/src/rp2_common/boot_stage2/CMakeFiles/bs2_default.dir/compile_time_choice.S.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1561: pico-sdk/src/rp2_common/boot_stage2/CMakeFiles/bs2_default.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

Did you build the container image yourself, or did you use the image from DockerHub?
Please provide a version of the SDK you use.

I used the commands from the README:
docker run -d -it --name pico-sdk --mount type=bind,source=${PWD},target=/home/dev lukstep/raspberry-pi-pico-sdk:latest

docker exec -it pico-sdk /bin/sh

So it should have been using the latest image from DockerHub.
I'm about to reattempt it though.

Well, started from scratch again with a new container and it seemed to have worked this time. I'm not clear on what could have gone wrong the first time. I had just copied the files from the test project. I did have some other files initially in my working directory but I don't think they should have caused a problem (Example micropython code, etc). I've tried a few times to see if I could recreate the issue but things seem to work now. Appreciate the work you've done!

-- Check for working C compiler: /usr/bin/cc - skipped
...
-- Check for working CXX compiler: /usr/bin/c++ - skipped

These lines look like the build system has detected inappropriate (non-cross) compilers for building the Pico project.
ARM compiler should be used for building the Pico binary as mentioned in the log below.
-- Check for working C compiler: /usr/bin/arm-none-eabi-gcc - skipped
...
-- Check for working CXX compiler: /usr/bin/arm-none-eabi-g++ - skipped


Digdagignore disables digdag run

I had been waiting for the .digdagignore functionality for a while, so that I could have DAGs in my repository that wouldn't be pushed to the production server (in my case, test DAGs that I only want to run locally and on my CI/CD pipeline).
However, I am now testing the .digdagignore file, and even thought it does in fact prevent the DAGs from being pished, it also makes it so that I cannot run the DAGs locally (error: Workflow <...> does not exist in current directory. You may need to change directory first, or set --project option. (resource not found))
Is there any way to go around this issue? Otherwise, I will have to implement some hack to delete the test .dig files dynamically before pushing to the production server!
Thanks

NOTE: Though it is intended, the message might be confusing and can be improved.
% digdag run test.dig
2021-10-25 22:17:10 +0900: Digdag v0.10.3
error: Workflow 'test' does not exist in current directory. You may need to change directory first, or set --project option. (resource not found)


Handle the change for UnaryPositive now extending RuntimeReplaceable [databricks]

This PR adds a UnaryPositiveOverrides object that properly overrides UnaryPositive expression which now extends RuntimeReplaceable.
fixes #11136

Build

build

build

build


codechef clone

🛠 591
Closes # 591

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

hey, @gurjeetsinghvirdee @Rayman-Sodhi review my PR

hey, @gurjeetsinghvirdee @Rayman-Sodhi can you check now I have removed package-lock.json file.