KYLIN-4687 add unify clean sh to excute some clean shells

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Pull Request Test Coverage Report for Build 6226

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.005%) to 28.026%

Files with Coverage Reduction
New Missed Lines
%

tool/src/main/java/org/apache/kylin/tool/query/ProbabilityGenerator.java
1
83.33%

core-job/src/main/java/org/apache/kylin/job/impl/threadpool/DefaultScheduler.java
2
80.23%

core-metrics/src/main/java/org/apache/kylin/metrics/lib/impl/MetricsSystem.java
5
66.67%

Totals

Change from base Build 6212:
-0.005%

Covered Lines:
26223

Relevant Lines:
93568

💛  - Coveralls

This PR is blocked by https://github.com/apache/kylin/pull/1347 , because --deleteAllJobs true  is added.

@hit-lacus remove deleteAllJobs

Codecov Report

Merging #1349 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1349      +/-   ##
============================================
- Coverage     25.52%   25.50%   -0.02%
+ Complexity     6761     6760       -1
============================================
  Files          1506     1506
  Lines         93568    93605      +37
  Branches      13103    13111       +8
============================================
- Hits          23882    23878       -4
- Misses        67331    67364      +33
- Partials       2355     2363       +8

Impacted Files
Coverage Δ
Complexity Δ

.../apache/kylin/tool/query/ProbabilityGenerator.java
75.00% <0.00%> (-5.56%)
11.00% <0.00%> (ø%)

...he/kylin/dict/lookup/cache/RocksDBLookupTable.java
72.97% <0.00%> (-5.41%)
6.00% <0.00%> (-1.00%)

...core/storage/columnar/GeneralColumnDataReader.java
89.47% <0.00%> (-5.27%)
6.00% <0.00%> (-1.00%)

...org/apache/kylin/rest/util/QueryRequestLimits.java
35.71% <0.00%> (-4.77%)
5.00% <0.00%> (-1.00%)

...ream/core/storage/columnar/ColumnarStoreCache.java
52.27% <0.00%> (-3.41%)
8.00% <0.00%> (ø%)

...a/org/apache/kylin/rest/job/StorageCleanupJob.java
61.75% <0.00%> (-2.43%)
34.00% <0.00%> (ø%)

...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java
70.21% <0.00%> (-0.92%)
7.00% <0.00%> (ø%)

...he/kylin/engine/mr/common/MapReduceExecutable.java
12.58% <0.00%> (-0.33%)
6.00% <0.00%> (ø%)

...a/org/apache/kylin/job/common/PatternedLogger.java
0.00% <0.00%> (ø)
0.00% <0.00%> (ø%)

...apache/kylin/query/routing/RealizationChooser.java
0.00% <0.00%> (ø)
0.00% <0.00%> (ø%)

... and 4 more

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 ec726a4...0dade7b. Read the comment docs.

In general, this looks fine to me.
Since it should be a non-breaking enhancement, I will merge it.

@hit-lacus The new file is missing the apache license, which causes the rat check to fail. I create a new pr to fix it.
https://github.com/apache/kylin/pull/1356


[Enemy in Shadows] Missing Herald of Tzeentch

It's not in the Actor compendium. Page 78 of the adventure.

It's called the Guardian Daemon


About the training process

Hello author, when I trained the CTfomer network, I did 200 epoch training, the loss value converged, but the weight I trained myself to test the results are very poor, and the PSNR value of the PSNR value trained with your weight is very good, I want to ask what is the reason, look forward to your reply

训练接近4000epoch才能有这个效果

作者您好，我训练CTfomer网络的时候，做了200个epoch训练，损失值收敛了，但是我自己训练的权重测试结果很差，用你的权重训练的PSNR值的PSNR值非常好，想问问是什么原因，期待你的回复

sorry @liyifeng-xiansheng, I did not see your comments. Thanks @Versua0, Yes, We need to train more epochs to generate better results, like 4000 epochs. We can train fewer epochs with more patches within an image, but we choose to select fewer patches with more epochs. In this way, each set of training samples is more diverse. Also, each epoch usually includes training on different patches.


[Web]: ListTiles inside cards damage the hover experience from TextButtons

Hi,
I am working with clickabele cards in Flutter web.
I put the cards in a TextButton to make them clikcable. Usually I get for free that the mouse changes to a hand while hovering over the card.
This signals the user that he can click here.
This seems to fail if I put a ListTile inside a card. For example:
    Card card = Card(
      child: ListTile(
        leading: cardIcon,
        title: titleWidget,
        subtitle: lastModifiedWidget,
        trailing: expandButton,
      ),
      // child: cardRow,
      color: cardColor,
    );

In this case the hand changes to a mouse only while hovering over the edges of the card. In the middle the mouse course stays itself.
If I put my elements in a Row (uncomment row, comment the ListTile), the hovering effect works on the whole card.

flutter doctor -v
[√] Flutter (Channel beta, 2.0.2, on Microsoft Windows [Version 10.0.19042.804], locale de-DE)
    • Flutter version 2.0.2 at C:\Users\akilian\Flutter SDK\flutter
    • Framework revision 8962f6dc68 (7 days ago), 2021-03-11 13:22:20 -0800
    • Engine revision 5d8bf811b3
    • Dart version 2.12.1
[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at C:\Users\akilian\AppData\Local\Android\sdk
• Platform android-30, build-tools 30.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
X Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/windows#android-setup for more details.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Android Studio (version 4.1.0)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[√] VS Code (version 1.52.1)
• VS Code at C:\Users\akilian\AppData\Local\Programs\Microsoft VS Code
• Flutter extension can be installed from:
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[√] Connected device (2 available)
• Chrome (web) • chrome • web-javascript • Google Chrome 89.0.4389.90
• Edge (web)   • edge   • web-javascript • Microsoft Edge 88.0.705.74
! Doctor found issues in 1 category.

@Wissperwind
Can you provide a complete minimal reproducible code sample along with actual vs expected behavior to properly address this issue ?
Thanks.

Here it is:
import 'package:flutter/material.dart';
import 'package:google_fonts/google_fonts.dart';

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: "Test",
      home: Scaffold(body: TestPage()),
    );
  }
}

class TestPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {

    Card card1 = Card(child: Row(children: [Text("you can click me"), SizedBox(height: 60)]));

    ListTile listTile = ListTile(
      //leading: cardIcon,
      title: Text("you can click me"),
      subtitle: Text("but the mouse keeps the courser shape"),
      //trailing: expandButton,
    );

    Card card2 = Card(child: listTile);

    TextButton clickableCard1 = TextButton(onPressed: () {}, child: card1);
    TextButton clickableCard2 = TextButton(onPressed: () {}, child: card2);

    Column column = Column(children: [clickableCard1, clickableCard2]);

    return Center(child: column);

  }
}

So the expected behavious is, that if the mouse hovers over a card, no matter which, the cpurser changes to a hand. But that is only the case with the first card.

Thanks for the detailed report @Wissperwind.

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.2, on Mac OS X 10.15.4 19E2269 darwin-x64,
    locale en-GB)
    • Flutter version 2.0.2 at /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 8962f6dc68 (7 days ago), 2021-03-11 13:22:20 -0800
    • Engine revision 5d8bf811b3
    • Dart version 2.12.1

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • Mac OS X 10.15.4 19E2269
      darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 89.0.4389.90

! Doctor found issues in 1 category.

This seems to be working as intended. The ListTile widget handles its own mouse cursor. If it's enabled and has an onTap callback, it shows the clickable mouse cursor as expected. /cc @dkwingsmt for confirmation.

Removing the web label as this is a framework-side issue.

This issue is still reproducible in the latest stable and master channels.
Code sample
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      title: "Test",
      home: Scaffold(body: TestPage()),
    );
  }
}

class TestPage extends StatelessWidget {
  const TestPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    Card card1 = Card(
      child: Row(
        children: const [
          Text("you can click me"),
          SizedBox(height: 60),
        ],
      ),
    );

    ListTile listTile = const ListTile(
      title: Text("you can click me"),
      subtitle: Text("but the mouse keeps the courser shape"),
    );

    Card card2 = Card(child: listTile);

    TextButton clickableCard1 = TextButton(onPressed: () {}, child: card1);
    TextButton clickableCard2 = TextButton(onPressed: () {}, child: card2);

    Column column = Column(children: [clickableCard1, clickableCard2]);

    return Center(child: column);
  }
}

Video
https://user-images.githubusercontent.com/50555895/169272243-51918765-be47-4774-86f3-cf28da0c9c78.mov

flutter doctor -v (Stable)
[✓] Flutter (Channel stable, 3.0.0, on macOS 12.3.1 21E258 darwin-arm, locale en-US)
    • Flutter version 3.0.0 at /Users/nabeelparkar/fvm/versions/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ee4e09cce0 (7 days ago), 2022-05-09 16:45:18 -0700
    • Engine revision d1b9a6938a
    • Dart version 2.17.0
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/nabeelparkar/Library/Android/sdk/
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_SDK_ROOT = /Users/nabeelparkar/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Brave Browser.app/Contents/MacOS/Brave Browser

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 67.1.4
    • Dart plugin version 221.5591.52

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.3.1 21E258 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Brave Browser <IP_ADDRESS>

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

• No issues found!

flutter doctor -v (Master)
[✓] Flutter (Channel master, 3.1.0-0.0.pre.799, on macOS 12.3.1 21E258 darwin-arm, locale en-US)
    • Flutter version 3.1.0-0.0.pre.799 at /Users/nabeelparkar/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 348a2b4f2f (2 hours ago), 2022-05-19 00:33:08 -0400
    • Engine revision 1965c92ea4
    • Dart version 2.18.0 (build 2.18.0-130.0.dev)
    • DevTools version 2.13.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
    • Android SDK at /Users/nabeelparkar/Library/Android/sdk/
    • Platform android-32, build-tools 32.1.0-rc1
    • ANDROID_SDK_ROOT = /Users/nabeelparkar/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = /Applications/Brave Browser.app/Contents/MacOS/Brave Browser

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 67.1.4
    • Dart plugin version 221.5591.52

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

[✓] Connected device (4 available)
    • V2050 (mobile)            •<PHONE_NUMBER>000NM • android-arm64  • Android 12 (API 31)
    • sdk gphone arm64 (mobile) • emulator-5554   • android-arm64  • Android 11 (API 30) (emulator)
    • macOS (desktop)           • macos           • darwin-arm64   • macOS 12.3.1 21E258 darwin-arm
    • Chrome (web)              • chrome          • web-javascript • Brave Browser <IP_ADDRESS>

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

• No issues found!


Fix typo where many-tomany should be many-to-many

Describe this PR
Fix minor writing error on relations/troubleshooting-relations doc.
Link: https://www.prisma.io/docs/concepts/components/prisma-schema/relations/troubleshooting-relations

Oooops, thanks @lemuelroberto for catching our mistakes there!!


update: bump Wasmtime 4.0.0, Tokio 1.23.1 and Json5 2.2.3 (node)

Update Wasmtime to 4.0.0 as a core part of this project. Tokio and Json5 has been updated to fix an open CVE. See the attached tasks for more information.
It closes #57, closes #58 and closes #59

I applied your suggestion and udpated all the Rust deps with cargo update @gzurl.


Integrated syzkaller with mac80211_hwsim

Two virtual wireless devices are instantiated during network devices initialization.
Also, two pseudo syscalls are added:

syz_80211_inject_frame(mac_addr, packet, packet_len) - injects an arbitrary packet into the wireless stack. It is injected as if it originated from the device identitied by mac_addr.
syz_80211_join_ibss(ssid, ssid_len, fix_requency, await_up) - sets up and configures an IBSS network. If await_up=1, does not return until the network is established

This code reuses netlink helpers that were already present in the executor/linux_common.h file. This required some minor changes, which do not break the interface of these helpers. Details can be found in commit descriptions.
Also, this pull request contains minor modifications to nl80211 descriptions that focus fuzzer's attention on virtual devices and interfaces introduced here.

Couldn't we move what we do in syz_80211_join_ibss and maybe even hwsim_register_socket in syz_80211_inject_frame into initialize_wifi_devices? That's how we do it for all other devices, e.g. IFF_UP happens when we setup net devices. It would speed up fuzzing and provide devices in some meaningful/useful state right away. Nominally creating devices if they are not ready to actual data exchange is not that useful.
Also it can make bugs more reproducible. Currently all operations on the devices (syz_80211_join_ibss/hwsim_register_socket) alter state of the devices and this sinks to subsequent tests. So if one test does syz_80211_join_ibss and next does syz_80211_inject_frame, it will work. But reproducing crash in syz_80211_inject_frame won't work, because the test is not hermetic.

Are there ways to make these devices interact more with descriptions in sys/linux/socket_netlink_generic_80211.txt? We should make descriptions in socket_netlink_generic_80211.txt use our new device names, ifindex'es and mac addresses. But maybe there are more ways to make them interact.

@dvyukov
I addressed all your comments.
For reasons specified blow, code review fixes come in two commits and are not squashed yet.
all: apply code review fixes will be squashed with all: integrate with mac80211_hwsim,
all: introduce an enhanced version of syz_80211_join_ibss will be either squashed with all: integrate with mac80211_hwsim or deleted.

Couldn't we move what we do in syz_80211_join_ibss and maybe even hwsim_register_socket in syz_80211_inject_frame into initialize_wifi_devices?

I moved IBSS network creation to initialize_wifi_devices and removed the syz_80211_join_ibss pseudo syscall. Interestingly, local syzkaller executions (lasting 2-3 days) demonstrated that the number of detected issues has significantly decreased. Or at least it became noticeably harder to hit them.
I even implemented exact descriptions for NL80211_CMD_SET_INTERFACE and NL80211_CMD_JOIN_IBSS (i.e. not the kind of netlink descriptions used right now - with array of union of attribute types, but as a firm structure). It also didn't seem to help much. It seems that syz_80211_join_ibss really has some positive effect on generating problematic situations.
So I introduced it again, but in a slightly different form

It accepts a network interface parameter (as suggested by @xairy)
It does not attempt to execute for a long time. It either starts a long procedure and exits, starts a short procedure and exists, or starts a short procedure and waits until it's complete.

What do you think about it? Should we have this pseudo call or only rely on initialize_wifi_devices and syzkaller descriptions?
It is added as a separate commit so that it'll be easy to remove it if necessary.

Are there ways to make these devices interact more with descriptions in sys/linux/socket_netlink_generic_80211.txt? We should make descriptions in socket_netlink_generic_80211.txt use our new device names, ifindex'es and mac addresses. But maybe there are more ways to make them interact.

socket_netlink_generic_80211.txt has already been updated to use the same device names, ifindexes, mac addressess, ssid's and bssid's as those that are used to create virtual devices.

Nice! This ifreq change and socket_netlink_generic_80211.txt changes make things better.

Fixed new review comments by @xairy and squashed all commits with CR fixes into the main commit.

PTAL

Excited to observe syzbot reaction!
And thanks for fixing the fault injection/fail, this will benefit fuzzing overall.


Ledger can't sign MsgMultiSend

Response from ledger:
message: 'Ledger device: UNKNOWN_ERROR (0x6984)',
statusCode: 27012,
statusText: 'UNKNOWN_ERROR

trying to sign:
{
    "account_number": "8",
    "chain_id": "testing",
    "fee": {
        "amount": [
            {
                "amount": "0",
                "denom": "uatom"
            }
        ],
        "gas": "200000"
    },
    "memo": "",
    "msgs": [
        {
            "type": "cosmos-sdk/MsgMultiSend",
            "value": {
                "inputs": [
                    {
                        "address": "cosmos1hr9x0sjvel6z3vt9qny8sdd5gnnlgk0p69d6cv",
                        "coins": [
                            {
                                "amount": "101000",
                                "denom": "uatom"
                            }
                        ]
                    }
                ],
                "outputs": [
                    {
                        "address": "cosmos13dr26wdygna3s8fdl5tlc45m2le2ydydgjk8ek",
                        "coins": [
                            {
                                "amount": "1000",
                                "denom": "uatom"
                            }
                        ]
                    },
                    {
                        "address": "cosmos1hr9x0sjvel6z3vt9qny8sdd5gnnlgk0p69d6cv",
                        "coins": [
                            {
                                "amount": "100000",
                                "denom": "uatom"
                            }
                        ]
                    }
                ]
            }
        }
    ],
    "sequence": "0"
}

I can sign
{
    "account_number": "8",
    "chain_id": "testing",
    "fee": {
        "amount": [
            {
                "amount": "0",
                "denom": "uatom"
            }
        ],
        "gas": "200000"
    },
    "memo": "",
    "msgs": [
        {
            "inputs": [
                {
                    "address": "cosmos1hr9x0sjvel6z3vt9qny8sdd5gnnlgk0p69d6cv",
                    "coins": [
                        {
                            "amount": "101000",
                            "denom": "uatom"
                        }
                    ]
                }
            ],
            "outputs": [
                {
                    "address": "cosmos13dr26wdygna3s8fdl5tlc45m2le2ydydgjk8ek",
                    "coins": [
                        {
                            "amount": "1000",
                            "denom": "uatom"
                        }
                    ]
                },
                {
                    "address": "cosmos1hr9x0sjvel6z3vt9qny8sdd5gnnlgk0p69d6cv",
                    "coins": [
                        {
                            "amount": "100000",
                            "denom": "uatom"
                        }
                    ]
                }
            ]
        }
    ],
    "sequence": "0"
}

but the signature fails validation when broadcasting.
I think it's related to https://github.com/cosmos/ledger-cosmos/blob/6c194daa28936e273f9548eabca9e72ba04bb632/app/src/tx_parser.c#L52-L61
https://github.com/cosmos/ledger-cosmos/blob/6c194daa28936e273f9548eabca9e72ba04bb632/app/src/tx_display.c#L422-L426

@jleni Is this repository no longer maintained? I can't create an issue at https://github.com/Zondax/ledger-cosmos.

yes, please. We need to unify repositories

yes, please. We need to unify repositories

I can not create an issue at https://github.com/Zondax/ledger-cosmos. Can you enable the issue tab?

We have included a unit test to confirm that it is possible to sign the example.
However, this message has a different structure and it is currently not formatted in a friendly way. We will discuss this in our next sync call


JPEG files from Chrome/Safari on iPhone, autoconverted from HEIC has rotation issues

I'm uploading files to my server, where Sharp is used for rotating and croping thumbnails. If the upload comes from an iPhone where the following criteria is met, the jpeg gives me rotation trouble.

Photo from camera or album
Photo in portrait mode

Landscape photos works fine. These seems to be represented by real pixels, i.e. w,h = actual pixels.
Portrait photos has w,h as if it was a landscape image.
Orientation = 6, i.e. "image needs to be rotated 90 degrees".
My question:

Should I manually rotate the image before attempting other operations?
Or should Sharp handle this automatically?

I'm asking because I don't want to implement the manual rotation if this is implemented in Sharp later on.
Please see the attached jpeg file and the below metadata from this file.
{
   "format": "jpeg",
   "size": 1786289,
   "width": 4032,
   "height": 3024,
   "space": "srgb",
   "channels": 3,
   "depth": "uchar",
   "density": 72,
   "chromaSubsampling": "4:2:0",
   "isProgressive": false,
   "hasProfile": true,
   "hasAlpha": false,
   "orientation": 6,
  ...
}

sharp will only rotate pixels if you tell it to.
Use rotate() to auto-orient and remove the EXIF Orientation value or use keepExif() to retain the existing EXIF Orientation value.

Thanks for the information. I'm now rotating manually and it works great.
The odd thing about stated pixels and actual pixels might be worth mentioning in the docs. In my case I discovered the problem due to "bad extract area". I tried to crop the file based on the stated pixels, i.e. landscape. In practice the image was portrait and my crop went outside of the image area. It took a while to understand this, because iPhone only seems to create this discrepancy on portrait images, not landscape.
It would also be useful with some more info in the docs about EXIF orientation numbers. It's easy to find, but it would have saved me some hedache because I initially thought that orientation was the same as rotation, i.e. 6 degrees. But 6 means "rotate 90".
I found the info here: https://blog.praveen.science/crazy-image-exif-orientation-bug-or-feature/
And also.
Thank you soo much for the Sharp library. It's really awesome!


Unit tests & continuous integration in Python

Integration with GitHub actions should be fairly easy
Op wo 15 dec. 2021 15:50 schreef StijnKas @.***>:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pegasystems/cdh-datascientist-tools/issues/36, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABEVTKBPPSKZUIU4EDZQDFTURCTKPANCNFSM5KD2UOCA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


ocramius/proxy-manager needed?

Is there a reason why ocramius/proxy-manager is in require-dev?
The tests are running fine without it.

Not required, indeed, but needed for using lazy services.
Can you look up who added it, and why?

The package was added by @xtreamwayz in This commit.
Downloaded a Tarball of that version and searched for anything mentioning "Proxy" and found nothing but the package name in the composer.json and the ExpressiveInstaller config.php
I'm not saying Lazy Services are bad. If this package is only included to encourage using them, i'm all in, but it feels like the skeleton has even more dependencies than the zend-mvc skeleton....

I added this when during development. Most likely because I copied dependencies from somewhere else. As far as I know the proxy-manager is nowhere used and could be used.

but it feels like the skeleton has even more dependencies than the zend-mvc skeleton....

I'm not sure how you get that feeling. It has only 5 dependencies and they all are being used and speak for their self:
    "php": "^5.5 || ^7.0",
    "roave/security-advisories": "dev-master",
    "zendframework/zend-expressive": "^1.0",
    "zendframework/zend-expressive-helpers": "^2.0",
    "zendframework/zend-stdlib": "^2.7 || ^3.0"

If you are referring to all the other dev-requirements, I've added those because they were used in tests or I needed them for code completion. However the code has changed during development so I'm not sure which are obsolete now.

Found them. Have a look in the config sources.

And the proxy manager is installed by default as well: https://github.com/zendframework/zend-expressive-skeleton/blob/master/src/ExpressiveInstaller/config.php#L109

The skeleton itself might not have that many dependencies but the dependencies themselves have several others. But there's nothing we can do about that.

While we're on it: I've discovered that even if you decide not to use filp/whoops as ErrorHandler, it get's added to require-dev by this piece of code. I guess making that require-dev array empty should do the trick.


Microsoft.Diagnostics.NETCore.Client.ServerNotAvailableException: Process xxx not running compatible .NET runtime. when running EventStore as a process

Hi,
I'm not sure this is the best place to ask my question but I don't know where to ask it. Sorry if it's not the  correct place.
I'm new to Nix and devenv. I'm trying to run EventStore in my "devenv".
For this, I had a look at these three links:

Nix EventStore "recipe": https://github.com/NixOS/nixpkgs/blob/master/pkgs/servers/nosql/eventstore/default.nix
Redis devenv declaration: https://github.com/cachix/devenv/blob/main/src/modules/services/redis.nix
EventStore configuration for "dev mode": https://developers.eventstore.com/server/v20.10/installation.html#use-docker-compose

Copying the content of these links, I came up with:
{ pkgs, ... }:

{
  # https://devenv.sh/basics/
  env = {
    GREET = "devenv";
    # See https://developers.eventstore.com/server/v20.10/installation.html#use-docker-compose
    EVENTSTORE_CLUSTER_SIZE = 1;
    EVENTSTORE_RUN_PROJECTIONS = "All";
    EVENTSTORE_START_STANDARD_PROJECTIONS = true;
    EVENTSTORE_EXT_TCP_PORT = 1113;
    EVENTSTORE_HTTP_PORT = 2113;
    EVENTSTORE_INSECURE = true;
    EVENTSTORE_ENABLE_EXTERNAL_TCP = true;
  };

  # A .env file found, but dotenv integration is currently not enabled.
  # To enable it, set `dotenv.enable = true;` in your configuration.
  dotenv.enable = true;

  # https://devenv.sh/packages/
  packages = [
    pkgs.git
    pkgs.eventstore
  ];

  # https://devenv.sh/scripts/
  scripts.hello.exec = "echo hello from $GREET";

  enterShell = ''
    hello
    git --version
  '';

  # https://devenv.sh/languages/
  # languages.nix.enable = true;

  # https://devenv.sh/pre-commit-hooks/
  # pre-commit.hooks.shellcheck.enable = true;

  # https://devenv.sh/processes/
  # processes.ping.exec = "ping example.com";

  # See full reference at https://devenv.sh/reference/options/

  # Inpired from:
  #   - https://github.com/NixOS/nixpkgs/blob/master/pkgs/servers/nosql/eventstore/default.nix
  #   - https://github.com/cachix/devenv/blob/main/src/modules/services/redis.nix
  processes.eventstore = {
    exec = ''
      ${pkgs.eventstore}/bin/EventStore.ClusterNode \
        --db "$DEVENV_STATE/eventstore/data" \
        --index "$DEVENV_STATE/eventstore/index" \
        --log "$DEVENV_STATE/eventstore/log"
    '';
  };

  services.redis = {
    enable = true;
  };
  services.postgres = {
    enable = true;
    package = pkgs.postgresql_15;
    initialDatabases = [{ name = "myDB"; }];
  };
}

EventStore does start and it seems that I can push events into it.
But I'm not sure everything is working fine.
The EventStore web UI doesn't seems to be able list my streams, for example, and in the server logs I have the following error:
...
20:52:07 eventstore.1 | [92383, 6,20:52:07.864,DBG] "streamExistenceFilter" has flushed at 11,513. Diff 796. Took 00:00:00.0236810
20:52:08 eventstore.1 | [92383,31,20:52:08.194,WRN] Error encountered while processing events
20:52:08 eventstore.1 | Microsoft.Diagnostics.NETCore.Client.ServerNotAvailableException: Process 92383 not running compatible .NET runtime.
20:52:08 eventstore.1 |    at Microsoft.Diagnostics.NETCore.Client.PidIpcEndpoint.GetDefaultAddress()
20:52:08 eventstore.1 |    at Microsoft.Diagnostics.NETCore.Client.PidIpcEndpoint.Connect(TimeSpan timeout)
20:52:08 eventstore.1 |    at Microsoft.Diagnostics.NETCore.Client.IpcClient.SendMessageGetContinuation(IpcEndpoint endpoint, IpcMessage message)
20:52:08 eventstore.1 |    at Microsoft.Diagnostics.NETCore.Client.EventPipeSession.Start(IpcEndpoint endpoint, IEnumerable`1 providers, Boolean requestRundown, Int32 circularBufferMB)
20:52:08 eventstore.1 |    at Microsoft.Diagnostics.NETCore.Client.DiagnosticsClient.StartEventPipeSession(IEnumerable`1 providers, Boolean requestRundown, Int32 circularBufferMB)
20:52:08 eventstore.1 |    at EventStore.Core.Services.Monitoring.Utils.EventCountersHelper.<Start>b__8_0() in /private/tmp/nix-build-EventStore-22.10.2.drv-0/source/src/EventStore.Core/Services/Monitoring/Utils/EventCountersHelper.cs:line 48
...

Any idea of what is wrong?
How can I get more information?
Thanks for your help 🙂
Jules

It does start fine for me using the following devenv.nix (which just removed a bunch of things from what you have):
{ pkgs, ... }:

{
  # https://devenv.sh/basics/
  env = {
    # See https://developers.eventstore.com/server/v20.10/installation.html#use-docker-compose
    EVENTSTORE_CLUSTER_SIZE = 1;
    EVENTSTORE_RUN_PROJECTIONS = "All";
    EVENTSTORE_START_STANDARD_PROJECTIONS = true;
    EVENTSTORE_EXT_TCP_PORT = 1113;
    EVENTSTORE_HTTP_PORT = 2113;
    EVENTSTORE_INSECURE = true;
    EVENTSTORE_ENABLE_EXTERNAL_TCP = true;
  };

  # Inpired from:
  #   - https://github.com/NixOS/nixpkgs/blob/master/pkgs/servers/nosql/eventstore/default.nix
  #   - https://github.com/cachix/devenv/blob/main/src/modules/services/redis.nix
  processes.eventstore = {
    exec = ''
      ${pkgs.eventstore}/bin/EventStore.ClusterNode \
        --db "$DEVENV_STATE/eventstore/data" \
        --index "$DEVENV_STATE/eventstore/index" \
        --log "$DEVENV_STATE/eventstore/log"
    '';
  };

  services.redis.enable = true;

  services.postgres = {
    enable = true;
    package = pkgs.postgresql_15;
    initialDatabases = [{ name = "myDB"; }];
  };
}

Maybe you have some specific .net things installed on your machine?

Maybe some additional details:

I'm running on Mac
The error happens after a few seconds.
First, the server seems to be booting fine, blocking for a few seconds on this log line:22:31:17 eventstore.1 | [97094,14,22:31:17.943,INF] "InaugurationManager" in state (Leader, Initial): IGNORING "CheckInaugurationConditions". Not waiting for conditions.

After these few seconds, the error happens.
I'm not sure I have any version of .Net installed, being on macOS 🤔

Well, I decided to go with Docker Compose. It'll be easier for me. Less things to learn

It seems like eventstore is broken on darwin.


epaxos,atlas,tempo: non fault-tolerant reads (NFR)

Follow-up of https://github.com/vitorenesduarte/fantoch/pull/223.

Codecov Report

Merging #226 (c5bab49) into main (d9079f7) will increase coverage by 0.05%.
The diff coverage is 98.63%.

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

@@            Coverage Diff             @@
##             main     #226      +/-   ##
==========================================
+ Coverage   95.06%   95.12%   +0.05%
==========================================
  Files          96       96
  Lines       21041    21282     +241
==========================================
+ Hits        20003    20244     +241
  Misses       1038     1038

Impacted Files
Coverage Δ

...toch_ps/src/protocol/common/graph/deps/keys/mod.rs
99.77% <ø> (ø)

fantoch/src/protocol/mod.rs
60.37% <90.00%> (+2.04%)
:arrow_up:

fantoch/src/protocol/base.rs
98.20% <93.18%> (-0.79%)
:arrow_down:

fantoch/src/config.rs
99.16% <100.00%> (+0.04%)
:arrow_up:

fantoch/src/executor/monitor.rs
53.33% <100.00%> (+7.17%)
:arrow_up:

fantoch/src/kvs.rs
100.00% <100.00%> (ø)

fantoch_ps/src/protocol/atlas.rs
97.47% <100.00%> (+0.03%)
:arrow_up:

fantoch_ps/src/protocol/caesar.rs
97.25% <100.00%> (-0.58%)
:arrow_down:

...h_ps/src/protocol/common/graph/deps/keys/locked.rs
99.18% <100.00%> (ø)

.../src/protocol/common/graph/deps/keys/sequential.rs
95.49% <100.00%> (-0.38%)
:arrow_down:

... and 18 more

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 d9079f7...1280310. Read the comment docs.


Support for PKCS#8 encoded RSA private keys in Windows port

Windows port only support RSA private keys in PKCS#1 format.
On Linux and MacOS, PKCS#8 private keys are supported
See also #259

pkcs 7 should also be supported. Looking into pkcs 8 at the moment

My suggestion would be to look at the first line of the PEM file to determine the format of the PEM: PKCS#1, PKCS#8 or SEC1 and then have different code branches to load the key.
I used this code to decode PKCS#8:        https://github.com/odzhan/rsa/blob/63838b8799143c91b117d0785ef58c9ddbd3e436/rsa.c#L390

+1, particularly for EC keys in PKCS#8 format.
The previously linked Microsoft API has been removed and this is now recommended: https://learn.microsoft.com/en-us/windows/win32/api/wincrypt/nf-wincrypt-pfximportcertstore

I think this is also an issue on mac, was seeing static: failed to import certificate and private key with error 0. error, but that went away after converting the key to pkcs#1.


isEntry hook

Describe the feature
It would be very useful to have a unified hook to check if a module is an entry point. Like this.getModuleInfo(id).isEntry in rollup.
Additional information

[ ] Would you be willing to help implement this feature?

the same to me


Enhancement: null safety and minor issue resolved

I have added null safety and handled any related issues. I also noticed and fixed a minor problem with the top-to-bottom overflow, and I've tested it. You can merge it with the main branch now. Thank you!

merge gaurav-dev into main


Notification icons are not shown.

Hi
When using notification component its icons are not showing.Seems like missing icons.
here is how notification looks like:
https://image.ibb.co/b40BVn/icon_Issue.png
When inspecting html of notification I notice
after <span class = "icon icon-check-filled ng-star-inserted"> </span> , :before is missing.
Am I missing something?
Angular version:  5.2
ngx-ui version:  20 (also tried v19)
Thanks in advance

I am not able to reproduce. Can you try creating a stackblitz or a github repo and reproduce that?
Are you importing loading @swimlane/ngx-ui/release/index.css

@marjan-georgiev
Thanks it works. But the problem is this new theme overwrites my other styles such as buttons and tables. How can I solve this in a clean way?

ngx-ui is a complete UI library and meant to be used as a whole. I assume you only want to use it for some things, but not others? You could possibly do that by importing individual styles from https://github.com/swimlane/ngx-ui/tree/master/release/styles and https://github.com/swimlane/ngx-ui/tree/master/release/assets,
but that can get complicated to set up.
Another way is to have your own style overrides.

I see. @marjan-georgiev .
Yes I am using ngx-ui for only some components. I also use angular material and bootstrap 4. Fromt my point of view, ngx-ui should not add custom styles to elements. It should only only if directive applied to it.

What about trying to only import the icons font? https://github.com/swimlane/ngx-ui/blob/master/release/styles/fonts/index.scss
If you're not using other parts of ngx-ui, see if that might fix your original icons issue.

@marjan-georgiev  . I tried to add @import<EMAIL_ADDRESS>to styles.scss but it now complains

Lessons-app\node_modules\assets\fonts\source-sans\SourceSansPro-SemiboldItalic.ttf doesn't exist
Can I solve this ? Thanks

Try adding this to your webpack config:
      new webpack.LoaderOptionsPlugin({
        options: {
          context: root(),
          sassLoader: {
            includePaths: [
              root('node_modules', '@swimlane/ngx-ui', 'release'),
              root('node_modules', '@swimlane/ngx-ui', 'release', 'styles'),
            ]
          },
          postcss: function() {
            return [ autoprefixer ];
          }
        }
      })

It tells the webpack sass loader where to look for the styles and fonts.

@marjan-georgiev . Thanks so much for your help. Even though I think it is not a good idea to modify webpack, seems it is only choice.

I agree, but since you're importing raw scss files, webpack needs to know how to compile them.

@marjan-georgiev
Without webpack config tried to solve it with custom icon. You have provided api for notification create:
notificationService.create({
    title: 'Patch Failed!',
    icon: 'exclamation-circle',
    rateLimit: false,
    timeout: false
  })

Here I use font awesome icon (exclamation-circle) . But again notification service does not show icon.
Do you have idea how to solve this?
Thanks in advance!

That will not work because it must be a ngx-ui icon.
However, you can choose to have your own custom template. Take a look at Templated Notifications in the demo:
https://swimlane.github.io/ngx-ui/#notification
You can define the whole template of the notification in there.

@marjan-georgiev
yes it is also a good idea. Thanks a lot.


[Bug]Browser download fails

When running example code:
import { plugin } from "playwright-with-fingerprints"

(async () => {
    // Get a fingerprint from the server:
    const fingerprint = await plugin.fetch('', {
        tags: ['Microsoft Windows', 'Chrome'],
    })

    // Apply fingerprint:
    plugin.useFingerprint(fingerprint)

    // Launch the browser instance:
    const browser = await plugin.launch()

    // The rest of the code is the same as for a standard `playwright` library:
    const page = await browser.newPage()
    await page.goto('https://google.com')

    // Print the browser viewport size:
    console.log(
        'Viewport:',
        await page.evaluate(() => ({
            deviceScaleFactor: window.devicePixelRatio,
            width: document.documentElement.clientWidth,
            height: document.documentElement.clientHeight,
        }))
    )

    await browser.close()
})()

Found error:
The browser is downloading - this may take some time.
C:\Users\xxx\Coding\xx\node_modules\bas-remote-node\src\services\socket.js:101
    this._ws.removeAllListeners();
             ^

TypeError: Cannot read properties of undefined (reading 'removeAllListeners')
    at SocketService.close (C:\Users\xxx\Coding\xx\node_modules\bas-remote-node\src\services\socket.js:101:14)
    at BasRemoteClient.close (C:\Users\xxx\Coding\xx\node_modules\bas-remote-node\src\index.js:290:20)
    at C:\Users\xxx\Coding\xx\node_modules\browser-with-fingerprints\src\plugin\connector\index.js:30:20
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Node.js version: 18
ESM + TypeScript Mode

This information is not enough. What plugin version are you using?

Upgraded just now
Here's the package.json
{
  "name": "xx",
  "version": "1.0.0",
  "main": "index.js",
  "license": "MIT",
  "dependencies": {
    "@faker-js/faker": "^7.6.0",
    "axios": "^1.3.4",
    "better-queue": "^3.8.12",
    "free-proxy-finder": "^1.0.1",
    "playwright": "^1.31.2",
    "playwright-with-fingerprints": "^1.1.5",
    "typescript": "^4.9.5"
  },
  "type": "module",
  "scripts": {
    "dev": "ts-node-esm"
  },
  "devDependencies": {
    "@types/better-queue": "^3.8.3",
    "@types/node": "^18.15.0"
  }
}

OK, thanks, i'll take a look.

Please try to use the latest version - 1.1.6. Also, before the next run, remove the data folder from your script's working directory.

Thx, it works.


[Pipelines] Add a skeleton for the generation of KPIs

/test rhoai-light pipelines

/test rhoai-light pipelines

/test rhoai-light kserve

/test rhoai-light pipelines gating

/test rhoai-light pipelines gating

/test rhoai-light pipelines gating

merging this version.


Performance regression since 0.12.11

I have some code that performs a parallel line integral using quadpy.line_segment.integrate_adaptive. The integral is of oscillatory type. Previously, I got reasonable looking result in relatively little time. Since the upgrade to 0.14.11, that integration consumes huge amounts of RAM (50 GB+ for 1k line integrals) and never finishes (running out of patience after 10 min +). Downgrading to 0.13.6 did not help, but to 0.12.11 did.
I assume something in the algorithm must have changed such that the stop condition is not reached anymore. It's well possible that previously, the result was inaccurate (but good enough for my use case).

Without an MWE, I can't do anything about it.

I was hoping that you could point out a change in the implementation of the adaptive integration (e.g. a bug fix), which may have changed the interpretation of the convergence criteria (e.g. the recent introduction of eps_abs and eps_rel) and that the runtime might be an inherent case of my integrand. In that case, it would have been difficult to provide an MVP, since the integrand function is distributed over a larger code-base (a physical model that involves among others a linear interpolation of a sampled spectrum).
However, the MVE that I just provided for #316 seems to trigger the same behaviour. So there it is, just use that MVE for this issue aswell.

The MWE in the other bug is too complex. Please provide a shorter one and also post it here.

It seems that this is fixed in 0.16. The following shortened example would subdivide until memory exhaustion in 0.15:
import numpy as np
import quadpy

# parameters
lo = -2
hi = 2
A0 = np.array(1)[...,None]
A1 = np.array(0)[...,None]
eps = 1e-7

# integrand
def func(x):
    n_times_p, = x.shape
    print(x.shape)
    return A0*np.sign(x) + A1*x

# perform integration
domain = np.array([lo,hi])
val,error = quadpy.c1.integrate_adaptive(
    func,
    domain,
    kronrod_degree=5,
    eps_abs=eps,
    eps_rel=eps,
)

In 0.12 (when adjusting for the API change of #316), the code runs fine too. Given that the true integral is zero and somewhere between 0.12 and 0.15 eps has been split into eps_rel and eps_abs, it likely had to do with eps_abs not being taken into account correctly.

Good to hear! The default concatenation methods for stopping tests were all, meaning for example that both relative and absolute tolerances had to be fulfilled. It's now any, such that one of the two will suffice.


MI - REFERENCE section (HL7 Inbound)

Complete the reference content on [1].
[1] https://github.com/wso2/docs-ei/blob/master/en/micro-integrator/docs/references/inbound_endpoints/listening_inbound_endpoint/hl7-inbound-protocol.md

The original page is removed. Content is separated into 'Concepts'[1] and 'Properties'[2].
[1] https://github.com/wso2/docs-ei/blob/master/en/micro-integrator/docs/concepts/concepts.md
[2] https://github.com/wso2/docs-ei/blob/master/en/micro-integrator/docs/references/synapse-properties/inbound-endpoint-properties.md

Done with the reference section [1].
[2] https://github.com/wso2/docs-ei/blob/master/en/micro-integrator/docs/references/synapse-properties/inbound-endpoint-properties.md


Expose JVM metrics when using Micrometer

Currently, only the custom metrics are exposed when using micrometer. Adding JVM internal metrics can be done in few lines of code and can greatly help when running secor on production.

That sounds a good idea.  Will you be able to make a PR to make the
change?  It would be a great help for the community.
On Fri, Mar 26, 2021 at 2:22 AM DomWos @.***> wrote:

Currently, only the custom metrics are exposed when using micrometer.
Adding JVM internal metrics can be done in few lines of code and can
greatly help when running secor on production.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/1949, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYJP74VPAEYB3VG67EH2T3TFRG5TANCNFSM4Z3AVJVA
.

Sure, will do it over the weekend :)

Thanks
On Fri, Mar 26, 2021 at 2:42 PM DomWos @.***> wrote:

Sure, will do it over the weekend :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/secor/issues/1949#issuecomment-808523938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYJP76BNMQQI3Q2JZTKS3DTFT5TZANCNFSM4Z3AVJVA
.


[feature required] print create args in pouch deamon log

Issue Description
We may need to print the args of /container/create API for debugging.
Describe what happened:
Describe what you expected to happen:
Print create configure in daemon log, you may choose to print it only in debug mode or always.
How to reproduce it (as minimally and precisely as possible):
1.
2.
3.
Anything else we need to know?:
Environment:

pouch version (use pouch version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Promote the priority and could you help to finish this? @ZouRui89

INFO[2018-06-01T04:01:41.725301033Z] Calling POST /v1.24/containers/create?name=foo, client @
INFO[2018-06-01T04:01:41.725692973Z] create containter with args: containerConfig: {false false false false [] map[]  [] [] map[]  docker.io/library/busybox:latest  map[]  false [] false  false  [] map[] false  <nil> false  map[] }, hostConfig: &{false [] [] []  []  [] [] [] false [] []    [] <nil> bridge -500  map[] false false false 0xc420244640 false   [] <nil> map[] map[] map[]    [] {[] [] [] [] 0 []  0 0 0 0 0 0 0   [] [] 0 0  0 0 0xc420876898 0 0 0 0xc420876918 0xc420876928 0 0xc420876931 0 0 []}}, networkingConfig: &{map[]}

INFO[2018-06-01T03:54:09.557808086Z] Calling POST /v1.24/volumes/create, client @
INFO[0093] create volume with args: name: pouch-volume, driver: local, driverOpts: map[opt.size:100g], lables: map[]

INFO[2018-06-01T03:56:51.140429432Z] Calling POST /v1.24/networks/create, client @
INFO[0254] create network with args: name: pouchnet, checkDuplicate: true, driver: bridge, enableIPV6: false, internal: false, ipam: &{[{map[] <IP_ADDRESS>  <IP_ADDRESS>/24}] default map[]}, labels: map[], options: map[]

How about log like this? Any ideas? @allencloud @Letty5411

I think the container creation API need to be more details in the log, like the way network and volume do. @ZouRui89


Update PySAM exportconfig to use new JSON forms etc

Important for generating PySAM code now that we've added hybrid systems. We have intentionally chosen not to do this before the SAM release, but will finish it before the associated PySAM release. @sjanzou please feel free to add more detail to this as needed

Documenting our agreed-upon timeline for this:

Merge hybrids into develop, breaking PySAM for now.
a.	Any urgent PySAM fixes (for dGen, reV, etc) will be built from patch
Build a public Beta, UI only
After the public Beta
a.	Some combination of Brian, Darice, Steve work together to fix the PySAM build (possibly negating the need for second copies of UI forms)
b.	Meanwhile, the rest of the team targets bug fixes, defaults updates, etc (Brian and Darice can possibly start on this before the public Beta as well)
The new public version AND the new PySAM version can then be released simultaneously with fixes from #3, a few weeks after the public Beta.


[Improve] Task#IntoClaimed

Make Task#IntoClaimed() check the current state of the task
and return an error if it isn't a claimable one. Add a unit
test to lock in the behaviour.

Codecov Report

Merging #14 into master will decrease coverage by 0.19%.
The diff coverage is 78.94%.

@@            Coverage Diff            @@
##           master      #14     +/-   ##
=========================================
- Coverage   82.71%   82.52%   -0.2%
=========================================
  Files          18       18
  Lines        1140     1150     +10
=========================================
+ Hits          943      949      +6
- Misses        135      139      +4
  Partials       62       62

Impacted Files
Coverage Δ

internal/domain/task/service.go
84.61% <0%> (-15.39%)
:arrow_down:

internal/domain/task/models.go
88.59% <100%> (+0.63%)
: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 d994ca5...0c027b5. Read the comment docs.


feat: Added Contact manager functions

All the features are functional; however, I couldn't test the synchronization with my phone's calendar because I don't have any devices with that feature enabled.
I leave it here so that anyone with a compatible device can test it.
Todas as funções estão funcionais, porém, eu não pude testar a sincronização com a agenda do meu celular, pois, não tenho nenhum dispositivo que esteja ativo a função.
Deixo a mesma aqui, para quem tenha o dispositivo habilitado, possa-a testar.
Unzip file for test and paste in your browser
wppconnect-wa.zip
await<EMAIL_ADDRESS>'John', { surname: 'Doe', syncAdressBook: true, });
await WPP.contact.remove('[putNumberHere]@c.us');

WPP.contact.save is not a function

tried with this branch
WPP.contact.save is only working for browser, it does not affecting on Mobile device even after refreshing contacts 3 to 4 times

tried with this branch
WPP.contact.save and WPP.contact.remove both are only working for browser, it does not affecting on Mobile device even after refreshing contacts 3 to 4 times

Is the option to sync contacts enabled on your device (iOS or iPhone)? Go to Settings > Privacy > Contacts.


SFTP : Ability to drag and drop files

Describe the feature
Ability to drag and drop files between WebSSH and iOS / macOS

Support drag & drop for SFTP transfers would definitely bump my rating up to 5 stars!
From talz000013 on the App Store


Unable to Resolve MenuCompat SetGroupDividerEnabled

Xamarin.Android Version (eg: 6.0):
I feel stupid, I don't know what this is referring to after looking at my NuGet packages. How do I find this?
Operating System & Version (eg: Mac OSX 10.11):
Mac OS 10.14.2
Support Libraries Version (eg: 23.3.0):
<IP_ADDRESS>
Describe your Issue:
I'm trying to set the group divider in a menu but I'm unable to resolve MenuCompat.setGroupDividerEnabled(Menu menu, boolean enabled) as defined here.
I've tried referencing the v4 support libraries.
Steps to Reproduce (with link to sample solution if possible):
Try to reference the method.
Include any relevant Exception Stack traces, build logs, adb logs:
Not relevant.

Hey there, make sure you install Xamarin.Android.Support.Compat nuget.  I can see the API exists there.

Thank you, I'll try that!


Network Error when creating pull request

Description
Press this button to create pull request and see the error
POST request returns HTML instead of json
Gitea Version
trunk
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
No response
Operating System
No response
How are you running Gitea?
https://try.gitea.io
Database
None

Oops, sorry I missed that
Make backend code respond correct JSON when creating PR #25353


feat: add hmac-sm3 test cases from GM/T 0042-2015 Appendix D.3

Checklist

[x] documentation is added or updated
[x] tests are added or updated

I think this would be better added to evp_test.

which place ?

Look in test/recipes/30-test_evp_data.
Either create a new file with these tests or add them to evpmac_common.txt

Tests are failing for some reason

This PR should not be merged before #18737

Should this be in 3.0?

Okay with 3.0.  Okay without.

Okay with 3.0. Okay without.

Okay with 3.0. Okay without.

with what,
???

Nothing to do here.

This pull request is ready to merge

Kicking the CIs since #18737 is now merged.

Merged, thanks for the contribution.


repeat.for ordering issue with custom element container

I would expect the following to render a list with values 1, 2, 3, 4. Instead, it renders 1, 4, 2, 3. See complete example.
app.html
…
<c-ul>

1

${item}

4

</c-ul>
…

c-ul.html
<template>

<content></content>

</template>

c-ul.js
import {customElement, containerless} from 'aurelia-framework';

@customElement('c-ul')
@containerless()
export class CustomUl {
}

Thanks for the sample code. I'll take a look.

I've reproduced this bug. It's an issue in our shadow dom fallback implementation. We are planning to update that implementation to the shadow dom v1 spec in the near future. That will mean some massive internal implementation changes. As part of that update, we'll address this issue.

Closing this since it's now fixed in the new shadow dom implementation, residing in a branch and preparing for release soon.


still have trailing comma

found the line of "span"property still have trailing comma

Thank you for your contribution!


AOT support

Description
This pull request introduces support for Ahead-Of-Time (AOT) compilation in the library. As the original branch has already had System.Text.Json serialisation implemented, the main effort was to make all the API requests and types involved in communication AOT friendly and to remove everything that wasn't AOT friendly. This introduces another source generator in addition to EnumSerializer.Generator which is standard System.Text.Json source generator.
The library API has been preserved, however, if you for any reason want to serialise or deserialise a request manually in your code (say you get a raw input from your web hook) you'll have to use TelegramBotClientJsonSerializerContext like this:
Update update =
    JsonSerializer.Deserialize(rawTelegramUpdate.Update, TelegramBotClientJsonSerializerContext.Instance.Update);

The TelegramBotClientJsonSerializerContext contains all the generated code needed for serialisation and deserialisation. Another thing worth mentioning is this TelegramBotClientJsonSerializerContext.Instance static member which should be used instead of default implementation. The standard implementation of JsonSerializerContext generator generates TelegramBotClientJsonSerializerContext.Default but it lacks some important JsonSerializerOptions like JsonNamingPolicy.SnakeCaseLower etc. The Instance instance has them. You can also change  default JsonSerializerOptions  in Instance like this:
TelegramBotClientJsonSerializerContext.JsonSerializerOptions = new JsonSerializerOptions { // your options }

However, if you just use the library API methods for sending or polling requests you won't need this and there are no changes needed in your code as well.
Changes
There are some major internal changes in the library though. First comes from the necessity of having two source generators and the fact that the one depends on the output of the other. Two of them don't properly work in the same project so the library has been split in two: Telegram.Bot and Telegram.Bot.Base. The last one includes the code generated by EnumSerializer.Generatorand the first the code generated by System.Text.Json source generator. So, there will be most likely two Nuget packages as well.
Other modifications include removing JsonSerializerOptionsProvider and some convenient attributes like  CustomJsonDerivedTypeAttribute, CustomJsonPolymorphicAttribute,  PolymorphicJsonConverter,PolymorphicJsonConverterFactory as they weren't AOT friendly.
Unit Tests Changes
Unit tests had to be changed as well. To test the same serialisation context that is used in actual API calls it was necessary to test serialisation with the actual (not container-like) types.

Another thing I forgot to mention is that AOT warnings only start appearing in .NET 7. However, the target frameworks have been left unchanged for compatibility reasons. Therefore, to check for any new AOT compatibility warnings, it is necessary to temporarily switch the target framework to .NET 7 or higher.

Honestly, I'm not sure I will use this PR. It's based on old branch, old approach to STJ. Your PR is hard to read because of lots of change in all files plus an extra Telegram.Bot.Base folder that duplicate all files (not sure of the reason to keep those)
Anyway, I'm pretty sure we can achieve the same without adding TelegramBotClientJsonSerializerContext.Instance.* arg to every constructor. My experience of doing it in WTelegramClient seemed much simpler.
Also I can probably simplify things with my code generation and maybe source generator.
So I will probably investigate this issue myself and find a good simpler way to achieve AOT/Trimming support from current codebase.

Yeah, this PR is pretty old and I probably would have done it differently myself now.


⚠ SolarOscp has degraded performance

In 4a346eb, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 507 ms

Resolved: SolarOscp performance has improved in 85bba12 after 11 minutes.


Gilles node syntax 2019 08 scheduling

Done for the changelog.
scheduleWorkflow and scheduleTask are already in ProcessorInterface.
You can now merge the pull request to your branch.

@jalric despite having merged this PR - I have reopened it because something is wrong:
In 201908-draft, each client has its own credentials, so you should not use credentials from client.js, credentials should come directly from the Client instance. Actually Alfred should probably be initialized with a client instance (new Alfred(this)).
Please update this branch


A (possible) fix for the ghost Edge stuff

A fix for the ghost edge icons (Altough one icon may still stay...)
reg delete "HKLM\zSOFTWARE\WOW6432Node\Microsoft\Windows\CurrentVersion\Uninstall\Microsoft Edge" /f >nul 2>&1
reg delete "HKLM\zSOFTWARE\WOW6432Node\Microsoft\Windows\CurrentVersion\Uninstall\Microsoft Edge Update" /f >nul 2>&1
reg delete "HKLM\zSOFTWARE\WOW6432Node\Microsoft\Windows\CurrentVersion\Uninstall\Microsoft EdgeWebView" /f >nul 2>&1

Also I didn't really know what 2>&1 is so I added it.

Also I didn't really know what 2>&1 is so I added it.

Side Note: why isn't it seeing oscdimg.exe?

Those registry keys just save uninstall info, not icons on a taskbar.
2>&1 redirects stderr (error messages) to stdout (standard output). In context of this line 1>NUL so 2>NUL - both errors and standard output are supressed.
Download oscdimg.exe and put it to the same folder as bat file, or to any location under your PATH variable.

The registry keys do get rid of remnants of Edge in the Apps tab in settings.

Also I did put oscdimg in the folder and it still didn't detect it.


Custom discovery and code endpoints, optional fields in discovery document

Is your feature request related to a problem? Please describe.
I am creating an application using data from a resource server that requires an OAuth 2 token obtained from a specific authorization server. Let's assume that the address of this server is "http://example.org". Then the authorization endpoint will be "/v2/oauth/authorize", the token endpoint "/v2/oauth/token", and the discovery endpoint "/.well-known/oauth-authorization-server".
In addition, their discovery document does not have some fields declared in OidcDiscoveryDoc as required and contains fields that are missing in OidcDiscoveryDoc, so even if the endpoint is changed, it will be impossible to load it.
At the moment, I can only set the endpoint of the token in the config, the other two are hardcoded in the library and cannot be changed. This prevents me from using the given cool library in my application
Describe the solution you'd like
Move the discovery and authorization endpoints to the AuthConfig with default values equal to the current hardcoded values. Make fields in the discovery document optional
Describe alternatives you've considered
Move the discovery and authorization endpoints to the AuthConfig with default values equal to the current hardcoded values. Give the opportunity to provide custom discovery document class

Ooh, interesting issue you're facing with your authorization server.
Not sure if this totally prevents you from using the library. I think your question / request is the same as #1164 which also requests a way to partially override the results from loading the discovery document.
I personally think these extra moving parts are perhaps not worth the complexity to all other users of this library, as there are already workarounds available:

Use an HttpInterceptor to tweak the discovery document data before it goes into this library;
Don't use loading the discovery document via this library, and instead load what you can get yourself, add the other fields, and configure the library fully yourself (and omit all the loadDisco...() methods from the library alltogether)

But in the end the call to find these workarounds lacking and consider it a feature request after all I'll leave up to the maintainer.
Hope this helps!

Thank you for your reply! And yes, this is an interesting auth server, something between OAuth2 and OpenID Connect. But unfortunately, the resource server I need only accepts tokens from that server.

I personally think these extra moving parts are perhaps not worth the complexity to all other users of this library

I don’t think so, because the changes that I proposed (move the address of the discovery endpoint to the config, making it optional and specifying the default value equal to the current hardcoded one) will not affect current users in any way (this item will be absent in their config and will still be use the default value, that is, the current one), but it will make it possible to redefine this address for people like me, saving us an extra work. Perhaps I can arrange it myself in the form of a pull request, if no one is against it and I find the time
By the way, thanks for mentioning this workarounds. Indeed, I can use it, did not think about it and also was confused by some naming (token endpoint is called tokenEndpoint, authorization endpoint is called loginUrl) so I thought that I can't correctly specify the endpoints for the library

Sorry, unintentional click

Cheers, glad my comment helped you out!
Regarding:

will not affect current users in any way

I agree that it can be easily made in a backwards compatible fashion.
The concern I have is that it is yet another moving part and yet more configuration options. The cost lies in complexity to understand and use the library, as well as more code inherently leading to more (possible) bugs.
Your use case is certainly valid, I appreciate you (and others) have to work with a wide range of IDSes out there. I just don't think "more specific config options" is healthy, or at least my order of preference for options:

Generic interception options that allow not only this use case, but also others. Requires more work, but is worth it since this lib is so highly used.
Workarounds for non-spec-compliant IDSes such as http interceptors or not loading the discovery doc at all (the workarounds I mention above)
Specific config options to deal with this, even though I prefer above 2 options, I feel your suggestion is still better than option 4.
Don't do anything and leave people that need this without any option other than using a different library.

But again that's my 2 cents as a library user / community moderator. I'll leave the call up to the library's maintainer.

Okay, let's wait for the opinion of the maintainers. After all, this is not a critical issue.
By the way, could you please suggest a custom token async storage workaround? I use Ionic and Capacitor Storage, but OAuthStorage  does not allow me to make the getItem method async, and the Capacitor Storage returns a promise. So returning value is always null, because getItem is synchronous and Storage returns value after method's completion
Here is screenshotted implementation:

setItem and removeItem is fine, getItem always return null


Missing @types/unist in package.json dependencies

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Affected packages and versions
mdast-util-directive 2.1.1
Link to runnable example
No response
Steps to reproduce
When using Yarn with strict PnP mode, complex-types.d.ts cannot import unist because @types/unist is not explicitly listed in the package.json dependencies field. This causes the interfaces TextDirective, LeafDirective and ContainerDirective to be incomplete because they cannot resolve the parent interface Parent from unist.
Expected behavior
@types/unist is explicitly listed in the package.json dependencies field.
Actual behavior
@types/unist is not explicitly listed in the package.json dependencies field.
Runtime
Node v16
Package manager
Other (please specify in steps to reproduce)
OS
Linux
Build and bundle tools
Webpack

👍 PR welcome!


Power Platform SVG Icons not Importing to Lucid

Importing these Power Platform icons, https://learn.microsoft.com/en-us/power-platform/guidance/icons, to Lucid chart does not work.
However, the Azure icons here import to Lucid correctly, https://learn.microsoft.com/en-us/azure/architecture/icons/.

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

ID: 2e1e3e9c-4177-7991-9e9a-861baf21cbd2
Version Independent ID: 51970c35-5399-e9b2-ac1b-250032f2840d
Content: Microsoft Power Platform icons - Power Platform
Content Source: power-platform/guidance/icons.md
Service: power-platform
Sub-service: guidance
GitHub Login: @sericks007
Microsoft Alias: sericks

Thanks for your comment, @DanielTillmannTR. This issue has been assigned to the doc author for review.


openstack-ardana: enable dac-3cp input model

Add support for the dac-3cp input model, which combines
a 3 controller cluster with a deployer-as-controller
scenario. This can be used to test controller reboot
scenarios (3 controllers are needed to ensure galera
cluster quorum).

Jenkins job here: https://ci.suse.de/job/openstack-ardana/1205
Failures are related to post-rpm checks:
Modified RPM files:
05:48:01.983 .M.......    /var/log/monasca
05:48:01.983 .M...U...    /etc/monasca/agent
05:48:01.983 .M...U...    /etc/monasca/agent/conf.d
05:48:01.983 .M....G..    /var/log/cassandra
05:48:01.983 SM5..U.T.  c /etc/storm/storm.yaml
05:48:01.983 .M.......    /var/log/spark
Introduced rogue files:
06:02:52.332 file /usr/lib64/python2.7/site-packages/deltarpm.pyc is not owned by any package
06:02:52.332 file /usr/lib64/storm/extlib/mysql-connector-java.jar is not owned by any package
06:02:52.332 file /usr/lib/tmpfiles.d/monasca-transform.conf is not owned by any package
06:02:52.332 file /usr/lib/monasca/agent/custom_checks.d/nova_heartbeat_check.py is not owned by any package
06:02:52.332 file /usr/lib/monasca/agent/custom_checks.d/cinderlm_check.py is not owned by any package
06:02:52.332 file /usr/lib/systemd/system/octavia-common-rundir.service is not owned by any package
06:02:52.332 file /usr/lib/systemd/system/octavia-housekeeping.service is not owned by any package
06:02:52.332 file /usr/lib/systemd/system/octavia-api.service is not owned by any package
06:02:52.332 file /usr/lib/systemd/system/octavia-worker.service is not owned by any package
06:02:52.332 file /usr/lib/systemd/system/octavia-health-manager.service is not owned by any package
06:02:52.332 file /usr/share/octavia is not owned by any package
06:02:52.332 file /usr/share/spark/lib/mysql-connector-java.jar is not owned by any package

Tempest smoke passed (including heat test cases).


Running mix test fails without an existing database

Ecto 2.0.0-beta.1
Postgrex 0.11.1

After migrating to 2.0.0-beta.1 all CI builds started to fail during mix test, with the error that the database didn't exist. Running mix test a second time or creating the db beforehand always works. It is easy enough to work around, but is confusing and provides a poor first time experience. The explicit sequencing of create/migrate/start sandbox mode is common and is what Phoenix generates by default.
This can be reproduced in any existing app by dropping the test db and running mix test.
I'd love to fix this, but I'm not sure where the race condition is coming in.

Can you paste precisely the error here and what it is in your test helper file? Because this has been supposedly fixed.

Here is the end of test_helper.exs:
ExUnit.start

Mix.Task.run "ecto.create", ~w(-r Px.Repo --quiet)
Mix.Task.run "ecto.migrate", ~w(-r Px.Repo --quiet)
Ecto.Adapters.SQL.Sandbox.mode(Px.Repo, :manual)

Here's the stack trace (with individual identical errors for each pid removed) :
14:29:42.594 [error] Postgrex.Protocol (#PID<0.978.0>) failed to connect: ** (Postgrex.Error) FATAL (invalid_catalog_name): database "px_test" does not exist
** (DBConnection.Error) connection not available
    (db_connection) lib/db_connection.ex:817: DBConnection.checkout/2
    (db_connection) lib/db_connection.ex:717: DBConnection.run/3
    (db_connection) lib/db_connection.ex:957: DBConnection.run_meter/3
    (db_connection) lib/db_connection.ex:421: DBConnection.query/4
    (ecto) lib/ecto/adapters/sql.ex:178: Ecto.Adapters.SQL.query!/5
    (ecto) lib/ecto/adapters/postgres.ex:56: anonymous fn/4 in Ecto.Adapters.Postgres.execute_ddl/3
    (elixir) lib/enum.ex:1473: Enum."-reduce/3-lists^foldl/2-0-"/3
    (ecto) lib/ecto/adapters/postgres.ex:56: Ecto.Adapters.Postgres.execute_ddl/3

14:29:42.676 [error] GenServer #PID<0.1007.0> terminating
** (DBConnection.Error) connection not available
    (db_connection) lib/db_connection.ex:817: DBConnection.checkout/2
    (db_connection) lib/db_connection.ex:717: DBConnection.run/3
    (db_connection) lib/db_connection.ex:957: DBConnection.run_meter/3
    (db_connection) lib/db_connection.ex:421: DBConnection.query/4
    (ecto) lib/ecto/adapters/sql.ex:178: Ecto.Adapters.SQL.query!/5
    (ecto) lib/ecto/adapters/postgres.ex:56: anonymous fn/4 in Ecto.Adapters.Postgres.execute_ddl/3
    (elixir) lib/enum.ex:1473: Enum."-reduce/3-lists^foldl/2-0-"/3
    (ecto) lib/ecto/adapters/postgres.ex:56: Ecto.Adapters.Postgres.execute_ddl/3

Is it an umbrella project by any chance? do you also have a repo that reproduces the error?

Nope, it isn't an umbrella project. I only have a private repo, so I'll create a new one to recreate it.

@josevalim https://github.com/sorentwo/mix-test-failure demonstrates the failure. After cloning and installing deps simply run mix test, it fails with the same message as above. Running it a second time will work.
Aside from updating the generated phoenix-ecto code for Ecto 2.0 all I've done is generate a model with a migration.

Yes, the fix is to remove those from test_helper and do it on your aliases in your mix.exs:
"test": ["ecto.create --quiet", "ecto.migrate", "test"]

Otherwise we will start the app without a database and it will fail. :) Thank you for the report, we will tackle this when we migrate Phoenix!

Indeed it is, thanks!

what about umbrella apps ?
is correct to setup the aliases into the umbrella mix.exs?
seems is the only way to make it work, if I setup aliases into each app it won't work,
it bails out with error "(Mix) No repository available (project MyProject.Mixfile has no :app configured). Please pass a repo with the -r option."

I'm having trouble using this in an umbrella app. I have app A which defines schemas and has migrations and I have app B that depends on app A (in_umbrella: true). If I add the alias to mix.exs for both of those apps and then run the tests separately, everything works. However, when I run mix test in the umbrella app, it fails.
Randomized with seed 54646
==> AppA
....................................................

Finished in 0.5 seconds
52 tests, 0 failures

Randomized with seed 188921
** (DBConnection.OwnershipError) cannot find ownership process for #PID<0.46.0>.

When using ownership, you must manage connections in one
of the three ways:

  * By explicitly checking out a connection
  * By explicitly allowing a spawned process
  * By running the pool in shared mode

The first two options require every new process to explicitly
check a connection out or be allowed by calling checkout or
allow respectively.

The third option requires a {:shared, pid} mode to be set.
If using shared mode in tests, make sure your tests are not
async.

If you are reading this error, it means you have not done one
of the steps above or that the owner process has crashed.

See Ecto.Adapters.SQL.Sandbox docs for more information.
    (db_connection) lib/db_connection.ex:722: DBConnection.checkout/2
    (db_connection) lib/db_connection.ex:623: DBConnection.run/3
    (db_connection) lib/db_connection.ex:845: DBConnection.run_meter/3
    (db_connection) lib/db_connection.ex:464: DBConnection.prepare_execute/4
    (ecto) lib/ecto/adapters/postgres/connection.ex:82: Ecto.Adapters.Postgres.Connection.execute/4
    (ecto) lib/ecto/adapters/sql.ex:228: Ecto.Adapters.SQL.sql_call/6
    (ecto) lib/ecto/adapters/sql.ex:178: Ecto.Adapters.SQL.query!/5
    (ecto) lib/ecto/adapters/postgres.ex:69: Ecto.Adapters.Postgres.execute_ddl/3
    (ecto) lib/ecto/migrator.ex:43: Ecto.Migrator.migrated_versions/2
    (ecto) lib/ecto/migrator.ex:142: Ecto.Migrator.run/4
    (ecto) lib/mix/tasks/ecto.migrate.ex:74: anonymous fn/4 in Mix.Tasks.Ecto.Migrate.run/2
    (elixir) lib/enum.ex:651: Enum."-each/2-lists^foreach/1-0-"/2
    (elixir) lib/enum.ex:651: Enum.each/2
    (mix) lib/mix/task.ex:296: Mix.Task.run_task/3
    (mix) lib/mix/task.ex:328: Mix.Task.run_alias/3
    (mix) lib/mix/task.ex:261: Mix.Task.run/2
    (mix) lib/mix/project.ex:211: Mix.Project.in_project/4
    (elixir) lib/file.ex:1145: File.cd!/2
    (mix) lib/mix/task.ex:387: anonymous fn/4 in Mix.Task.recur/1
    (elixir) lib/enum.ex:1623: Enum."-reduce/3-lists^foldl/2-0-"/3

The problem appears to be in the ecto.migrate section of app B's alias. If I remove that, it works again, however that means that app B won't automatically setup the DB.

@hunterboerner can you provide a simple app that reproduces the error and open a new issue?

Here ya go: https://github.com/hunterboerner/ecto_sandbox_bug. I've noticed that the pid in this line never seems to change:
cannot find ownership process for #PID<0.46.0>.


Fix crash when passing empty String to UIAccessibilityCustomAction

Hi @kurabi,
Creating a failable initializer for SwipeAccessibilityCustomAction will avoid an NSInternalConsistency exception that occurs when passing an empty String to UIAccessibilityCustomAction.
Thank you for the great library 🙇

Awesome. Thanks for the PR.


Does not compile because of lacking GCDAsyncUpdSocket

When the source files of ios-ntp are added to a project, trying to compile gives an error from NetAssociation.h because of importing a non-existent <CocoaAsyncSocket/GCDAsyncUpdScoket.h>
From what I can gather, this is not a Cocoa system library, but apparently some random third-party library from somewhere. This doesn't seem to be included in the ios-ntp git project, nor can I find a single mention of it anywhere in its documentation, apart from an obscure "the files from CocoaAsyncSocket have been replaced with the most recent versions" (which quite clearly is untrue, as no such files are found anywhere in the ios-ntp git project.)

Mea culpa .. iOS-ntp has not had much love for quite a long time.  I respond to issues but have not actually tried to compile the code recently.  I have been asked to not delete the repo, so it continues in a somewhat unsupported state .. such is the nature of open source, I'm sorry.
CocoaAsyncSocket is easy to find; it's the first hit if you Google for that term.  The two necessary files used to be included in this project but were removed when CocoaAsyncSocket went through a period of changes, but iOS-ntp's dependency on CocoaAsyncSocket is clearly present in the Podspec, and is referenced in the most recent change note: "removed asyncsocket files from network-lib".
I have added the CocoaAsyncSocket URL to the README file, and will review the rest of it for inconsistencies that might be confusing, but it's always good to remember that free software is often worth all the money you spend on it.  I wrote this this library a long time ago, when iPhones got their time from the phone company and it could off by minutes (!), in order to get an accurate time for my app, and others have used it to check that the system clock time hasn't been fiddled with to avoid registration.  In the intervening years iOS has adopted the reference NTP implementation making system time accurate, and eliminating my need for it.

Given that it only requires GCDAsyncUpdSocket.h and GCDAsyncUpdSocket.m, and nothing else from that project, it would be easy enough to just add those two files into the project...

It used to be like that but freezing them in this project meant they went out of date and gave people trouble.  The CocoaPods mechanism refers to the CocoaAsyncSocket repo, so "pod install" gets the latest version.  Just to be sure, I pulled a fresh copy of the iOS-ntp repo yesterday, did a pod install, and built and run the sample app.  There were a few Xcode complaints but, otherwise, success.


Fixing broken symbols.

o Adding back missing symbols: Earth and Si3402-B-GM(QFN-20_5x5mm) in
ESP32-PoE_Rev_C.lib

thanks, something happened during our conversion of the library


[Privacy] Obfuscate or filter out node info that includes any precise location

Today, nodes are reported on the map with precise location. Newcomers who are not aware or familiar with how location is reported (particularly on older firmware) will be leaking their precise location.
Two options to resolve:

Ideal Scenario - location is auto obfuscated on the way in. Then we can still populate the map but mitigate the risk.
Less Ideal Scenario - packets with precise location are dropped.

PR opened here: https://github.com/liamcottle/meshtastic-map/pull/74

I don't think this is necessary. I like having this map as an option with the location history, and also, at least for the US, nobody is going to upload to this map unless they know about it, as the public meshtastic.org server is not connected.
Also, newer versions of the firmware are opt-in for MQTT. If that flag is added, this won't be necessary.
If it must be done, then I would like the limit to be 16 or 18 bits of precision (About 100m)

In the PR its behind a flag now, and defaulted to false.

Also added configurable precision.


Allow terraform workspace to be variable.

I'm curious as to why the TF_WORKSPACE_NAME is a hardcoded value when the rest of the terraform values when using tfc are variables. It would be nice to control the workspace name in my organization.
Thoughts?

Hey there,
Thank you for the feedback. I've created a backlog item for the team to explore allowing the workspace name to be defined as a variable.


Remove tekton as a provenance format

This PR removes tekton as a provenance format since it was deprecated in an earlier release.

Changes

Removes the directory tekton in formats/.
Updates all unit tests to point to in-toto instead of tekton.
Updated default format to in-toto.

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Remove tekton from supported provenance formats

/kind feature

/assign @lcarva

/test pull-tekton-chains-integration-tests

/lgtm cancel

docs/config.md also needs to be updated.

Done!


SD card slot

It is not an issue but the storage on that thing is not a big so bigger storage will be awesome or maybe something like run scripts saved on your PC
but the still awesome project and great job

would be cool to have a List of Scripts available, that would be stored on the MicroSD.
Could Also work with a Rolling Pot Switch, with some lights, so you can have 3 or 4 Scripts, that are assigned to the those lights, work right when it is plugged in


[Language-related-refactoring] TestClassBuilderHelper refactoring

Describe the bug
More details in here and here, here, here, here

fixed in PR


Strange math bug in "Graphic 3D-Cube.Amos"

Describe the bug:
Strange math bug in "Morphing2.amos" and "Graphic 3D-Cube.Amos"
(I think bug is similar, so reported as one bug.)

"Morphing.amos" / "Morphing2.amos" found a bug in "Screen Mode" command.

Expected cause, if many arrays with index are used in the same mathematical formula, I guess the index get confused. might be store value at wrong place, maybe? This needs investigation.

Issue with arrays inside expressions with in parenthesis, think its fundamental issue with how arrays are handled in Amos kittens, two things come to mind, its pretty complicated, does it need to be? why can’t indexes be handled in the same why as parenthesis is implemented. Should not parenthesis remember last variable? I can imagine problems with nested arrays.

Looks like some problem is solved.
At least cube is now drawn, but is not centered like it should.

Wrote a few tests, should be working now.


Add Python 3.9, update others

Note Python 3.9 EOL is an estimate. Official EOL is currently "approximately October 2025" and will be updated closer to the time:

After that, it is expected that security updates (source only) will be released until 5 years after the release of 3.9 final, so until approximately October 2025.

https://www.python.org/dev/peps/pep-0596/

Thank you for the first PR and the support of the project. I have verified your changes on the website and everything is correct.


Dependence of derivation failed to build

Log:
❯ nix run github:thericecold/kaizen
error: builder for '/nix/store/qljqkzn7pnnlr6c90in45ci5ilya5jal-kaizen.drv' failed with exit code 1;
       last 10 log lines:
       >       4 │ import AppLauncher from './AppLauncher'
       >         ╵        ~~~~~~~~~~~
       >
       > ✘ [ERROR] No matching export in "lib/utils.ts" for import "getPlayer"
       >
       >     desktop/fullscreen/dashboard/home/player/index.ts:7:9:
       >       7 │ import { getPlayer } from 'lib/utils'
       >         ╵          ~~~~~~~~~
       >
       > 4 errors
       For full logs, run 'nix log /nix/store/qljqkzn7pnnlr6c90in45ci5ilya5jal-kaizen.drv'.
error: 1 dependencies of derivation '/nix/store/sid4hlqf4q4ygzxz7nfbsb32428avvz0-kaizen.drv' failed to build

Fixed, thanks for opening an issue.


Deprecated env() vs get()

Q
A

Issue Type
Question

Deployer Version
4

Local Machine OS
OS X 10.11.6

Remote Machine OS
Debian

Description
I was just updating to version 4.0 from version 3.3 and noticed that the env() function has been deprecated. How can I now check which stage server (defined using setup()) is being used inside tasks?
In version 3.3 I was able to check like this:
task('mytask', function(){

	if (env('server.host') == 'myawesome.server.com') {
		// Do something
	} else {
		// Do something else
	}

});

Now this doesn't work anymore and get('server.host') just returns an error:
[RuntimeException]
Configuration parameter server.host does not exists

This is a bug, and will be fixed soon. Now you can use get('server')['host'].
Duplicate #873


S3 access denied

Environment:

aws-parallelcluster-2.1.1]
alinux
SGE
t3.any
t3.any

Bug description and how to reproduce:
Following https://aws.amazon.com/blogs/opensource/aws-parallelcluster/ but included aws_access_key_id and aws_secret_access_key in config under [aws] section (understand it's not best practice). in [cluster default] added s3_read_write_access_resource = arn:aws:s3:::mybucket/*, cluster_type = spot and spot_price = X. Cluster spins up fine and trivial tests of SGE is fine but no access to S3 (permission denied).
E.g. on the head node:
$ aws s3 ls
An error occurred (AccessDenied) when calling the ListBuckets operation: Access Denied
if i copy ~/.aws/credentials and ~/.aws/config to head node, then access to S3 is fine from head node and compute nodes.
i realize i can workaround w/ a post-install script, but....
i also understand i can add a custom AMI (based on cfncluster AMI), but i'm looking for a minimal change.
not clear why aws_access_key_id and aws_secret_access_key are not being propagated to parallelcluster.

@gjparker9
The issue is the arn:aws:s3:::mybucket/* resource you specified gives you read/write access only to mybucket. You could do:
$ aws s3 ls s3://mybucket

However listing all buckets, like you did with aws s3 ls won't work.
The aws_access_key_id and aws_secret_access_key aren't copied onto the instance, a separate IAM role is created and the credentials are available on the instance magically (via Instance Profile )
I know this arn schema is confusing, so I wrote https://aws-parallelcluster.readthedocs.io/en/latest/s3_resources.html to better explain it.

thanks for the response @sean-smith
i was just saving typing in my comment.
$ aws s3 ls s3://mybucket
still returns
An error occurred (AccessDenied) when calling the ListObjectsV2 operation: Access Denied on head/compute nodes.
i'll dig around in the generated IAM role.... hmmm.  given that if i copy over ~/.aws to the head node, S3 access is ok, seems something fell over during this IAM creation.
so, i am correct, this should be working out-of-the-box?

Did you mean to specify s3_read_write_resource  instead of s3_read_write_access_resource ?

i did specify 's3_read_write_resource'. should have copied and pasted directly from config file.

the generated IAM role looks file (attached a policy called 'S3ReadWrite' which appears to have the correct info).

We should really add some logic to validate parameters that do nothing, like:
Did you mean to specify X parameter? It has no effect.

yes, warnings on unsupported parameters would be helpful...
still no joy, though.

oh I see, re-reading your previous comments, you specified s3_read_write_resource in your config. Can you paste your config?
What does the generated IAM role look like? You can see it by clicking on Master Server > IAM Role in the EC2 Console.

here's config (X is redacted) (so S3 bucket is actually com.parker9.wp)
[aws]
aws_region_name = us-west-1
aws_access_key_id = X
aws_secret_access_key = X
[cluster default]
vpc_settings = public
key_name = X
compute_instance_type = t3.nano
cluster_type = spot
spot_price = 0.003
s3_read_write_resource = arn:aws:s3:::com.parker9.wp/*
[vpc public]
master_subnet_id = subnet-20b7f647
vpc_id = vpc-a2d061c5
ssh_from = X/32
[global]
update_check = true
sanity_check = true
cluster_template = default
[aliases]
ssh = ssh {CFN_USER}@{MASTER_IP} {ARGS}

IAM role on master automatically generated w/ name: parallelcluster-t3-nano-RootRole-1N78K7571BIOB
has two policies attached: parallelcluster and S3ReadWrite.
parallelcluster (account number redacted w/ X)
{
"Statement": [
{
"Action": [
"ec2:DescribeVolumes",
"ec2:AttachVolume",
"ec2:DescribeInstanceAttribute",
"ec2:DescribeInstanceStatus",
"ec2:DescribeInstances"
],
"Resource": [
""
],
"Effect": "Allow",
"Sid": "EC2"
},
{
"Action": [
"dynamodb:ListTables"
],
"Resource": [
""
],
"Effect": "Allow",
"Sid": "DynamoDBList"
},
{
"Action": [
"sqs:SendMessage",
"sqs:ReceiveMessage",
"sqs:ChangeMessageVisibility",
"sqs:DeleteMessage",
"sqs:GetQueueUrl"
],
"Resource": [
"arn:aws:sqs:us-west-1:X:parallelcluster-t3-nano-SQS-CJPJIDW6Z837"
],
"Effect": "Allow",
"Sid": "SQSQueue"
},
{
"Action": [
"autoscaling:DescribeAutoScalingGroups",
"autoscaling:TerminateInstanceInAutoScalingGroup",
"autoscaling:SetDesiredCapacity",
"autoscaling:UpdateAutoScalingGroup",
"autoscaling:DescribeTags"
],
"Resource": [
""
],
"Effect": "Allow",
"Sid": "Autoscaling"
},
{
"Action": [
"cloudformation:DescribeStacks"
],
"Resource": [
"arn:aws:cloudformation:us-west-1:X:stack/parallelcluster-t3-nano/"
],
"Effect": "Allow",
"Sid": "Cloudformation"
},
{
"Action": [
"dynamodb:PutItem",
"dynamodb:Query",
"dynamodb:GetItem",
"dynamodb:DeleteItem",
"dynamodb:DescribeTable"
],
"Resource": [
"arn:aws:dynamodb:us-west-1:X:table/parallelcluster-t3-nano-DynamoDBTable-1TXX12M94IHKS"
],
"Effect": "Allow",
"Sid": "DynamoDBTable"
},
{
"Action": [
"s3:GetObject"
],
"Resource": [
"arn:aws:s3:::us-west-1-aws-parallelcluster/"
],
"Effect": "Allow",
"Sid": "S3GetObj"
},
{
"Action": [
"s3:PutObject"
],
"Resource": [
"arn:aws:s3:::NONE/batch/"
],
"Effect": "Allow",
"Sid": "S3PutObj"
},
{
"Action": [
"sqs:ListQueues"
],
"Resource": [
""
],
"Effect": "Allow",
"Sid": "SQSList"
},
{
"Action": [
"iam:PassRole"
],
"Resource": [
"arn:aws:iam::X:role/parallelcluster-"
],
"Effect": "Allow",
"Sid": "BatchJobPassRole"
}
]
}
and S3ReadWrite plocity:
{
"Version": "2012-10-17",
"Statement": [
{
"Action": [
"s3:"
],
"Resource": [
"arn:aws:s3:::com.parker9.wp/"
],
"Effect": "Allow",
"Sid": "S3ReadWrite"
}
]
}

copy and paste didn't work, in S3ReadWrite policy the Resource value is
"arn:aws:s3:::com.parker9.wp/*"
similarly Action value is
"s3:*"
seems it's having problems pasting "*"...

ok, the fix was simple. in config, under [cluster default] needed this line:
s3_read_write_resource = arn:aws:s3:::com.parker9.wp*
note, no slash after the bucket name. i can get/put/list and delete objects in the bucket on both head and compute nodes now. woo woo!
you can close this issue.


mount destination [ not absolute: unknown.

Hi,
when i create the container, the docker gives the following error:
docker: Error response from daemon: failed to create shim: OCI runtime create failed: invalid mount {Destination:[ Type:bind Source:/docker/app/volumes/0848ec46c970b30b2b209ea2dda8cb57f5bff3c0264f39b28b64aca80b54207a/_data Options:[rbind]}: mount destination [ not absolute: unknown.
This looks like https://github.com/moby/moby/issues/42480
But when i tried to build the image with modified DockerFile
-VOLUME [ '/opt/data' ]
+VOLUME [ "/opt/data" ]
the build failed...
Here is my docker version:
docker version
Client:
Version:           20.10.7
API version:       1.41
Go version:        go1.13.8
Git commit:        20.10.7-0ubuntu1~18.04.1
Built:             Wed Aug  4 22:43:25 2021
OS/Arch:           linux/amd64
Context:           default
Experimental:      true
Server:
Engine:
Version:          20.10.7
API version:      1.41 (minimum version 1.12)
Go version:       go1.13.8
Git commit:       20.10.7-0ubuntu1~18.04.1
Built:            Wed Aug  4 19:22:59 2021
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.5.2-0ubuntu1~18.04.2
GitCommit:
runc:
Version:          1.0.0~rc95-0ubuntu1~18.04.2
GitCommit:
docker-init:
Version:          0.19.0
GitCommit:

1. replace the content of ubunt/sources.list
deb http://mirrors.aliyun.com/ubuntu/ focal main restricted universe multiverse
deb-src http://mirrors.aliyun.com/ubuntu/ focal main restricted universe multiverse
deb http://mirrors.aliyun.com/ubuntu/ focal-security main restricted universe multiverse
deb-src http://mirrors.aliyun.com/ubuntu/ focal-security main restricted universe multiverse
deb http://mirrors.aliyun.com/ubuntu/ focal-updates main restricted universe multiverse
deb-src http://mirrors.aliyun.com/ubuntu/ focal-updates main restricted universe multiverse
deb http://mirrors.aliyun.com/ubuntu/ focal-proposed main restricted universe multiverse
deb-src http://mirrors.aliyun.com/ubuntu/ focal-proposed main restricted universe multiverse
deb http://mirrors.aliyun.com/ubuntu/ focal-backports main restricted universe multiverse
deb-src http://mirrors.aliyun.com/ubuntu/ focal-backports main restricted universe multiverse
2.modify the DockerFile
insert after LABEL maintainer "zhonger<EMAIL_ADDRESS>COPY ./ubuntu/sources.list /etc/apt/sources.list
RUN apt-get update
change
VOLUME [ "/opt/data" ]
3.build it
docker build . -t shuosc/dokuwiki

I am so sorry for not answering you in time.
https://github.com/shuosc/docker-dokuwiki/issues/3#issue-967938305
This is because we recommended using docker-compose to create or destroy the container.
https://github.com/shuosc/docker-dokuwiki/blob/39702a92d6dfc41c813e8aa794fbbe4c5c541752/docker-compose.yml#L11
The  volume /opt/data has been attached with local directory ./.data in docker-compose.yml file.
If you don‘t want use docker-compose, you can just create the container with this command:
docker run -ti -d -v local_file_path:/opt/data -p 80:80 shuosc/dokuwiki:latest

https://github.com/shuosc/docker-dokuwiki/issues/3#issuecomment-1060444885
If you want to change the sources file to Aliyun Mirror, you can just fork this repository and make changes as you like.
For me, it's not essential to change this because it does not affect normal usage.
Thank you for your use and kindful feedback!