weep binary name in ps

This may be an artifact of how netflix packages weep for debian, but:
$ weep serve &
time="2021-10-02T14:09:04-07:00" level=info msg="starting weep on <IP_ADDRESS>:9091"

$ ps axf | grep weep
 708631 pts/0    Sl     0:00      |   \_ /home/[username]/.weep-cache/weep-binaries/current serve

leads to this less than informative output:
$ sudo netstat -tnlp
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:9091          <IP_ADDRESS>:*               LISTEN      708631/current

It would be nice if it set a more descriptive name.

Thanks @PaulMcMillan! This is an internal thing, we'll make a ticket to track it and tag you in it.


.value() returns very small floating point number when it should return 0

It works well on Chrome but I've got the following display on FF 45 mac: -0%, don't hesitate if you need more informations

Do you have a code to reproduce this? How are you setting the text?


The cursor height of the TextField should be consistent, rather than depend on the content typed in.

@GaryQian
On Android:

Type in nothing, the height of cursor depend on white space and is short.
Type in Chinese, the height of cursor become to be longer.

On iOS:

Type in nothing, the height is good.
Type in emoji, the height is still good, but the cursor shifted up.

[✓] Flutter (Channel dev, v1.5.8, on Mac OS X 10.14.4 18E226, locale en-CN)
    • Flutter version 1.5.8 at /Users/yohom/Library/Flutter/flutter
    • Framework revision 0ba67226ee (33 hours ago), 2019-04-24 17:18:28 -0700
    • Engine revision c63d1cf9c9
    • Dart version 2.3.0 (build 2.3.0-dev.0.1 1f1592edce)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/yohom/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/yohom/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2.1, Build version 10E1001
    • ios-deploy 1.9.4
    • CocoaPods version 1.6.0.beta.2

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 34.0.2
    • Dart plugin version 183.5901
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2019.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 34.0.4
    • Dart plugin version 191.6183.88

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

mark

CC @justinmc

This is related to https://github.com/flutter/flutter/issues/24182

Previously, Flutter used a system of manually adjusted caret heights to visually emulate those found on iOS and Android. I have since discovered that the old solution both does not scale and does not properly reflect carets on each OS.
I have begun work replacing this old system with a more modern, scalable, and accurate way of determining caret sizes. The current version has a centered and dynamically adjusting caret size for Android, and no change for iOS. I will be working on a way to reconcile the differing glyph heights, but as it stands, carets should dynamically fit the text it is surrounding, as the following example makes obvious:

The caret should be smaller for the first test and larger for the second test. The changing heights for languages is a result of our layout system remaining faithful to font metrics whereas other systems may choose to ignore some metrics in exchange for consistency across the line.
What I can definitely do now is to center the caret against the text for iOS until a more complete solution is developed.

We should also be able to better align the default no-text caret height with the typical caret height so that any change is unnoticeable. This would involve a slight rework in how we determine caret sizes and what we default to.

any process on this?

As an update, I have not worked on this directly yet, but recently, fixes for the iOS caret being misaligned have landed. This is in the queue of issues I am look it right now though.

@GaryQian #33175 Is the same reason?

https://github.com/flutter/engine/pull/9041 Should allow CSS-style line height where the height property overrides the font's provided metrics and instead scales ascent and descent to add up to a multiple of font size.
When the text is styled with TextStyle(height: <somevalue>), we will be able to guarantee that the height of the line will be consistent for same combinations of fontSize and height. This would prevent carets from being different sizes for text of same size. Whether they are vertically aligned with each other or not is still not guaranteed with the above patch.

Cursor height changes significantly even after single space is typed:

Container(
  color: Colors.grey,
  padding: EdgeInsets.all(16),
  child: Container(
    color: Colors.white,
    child: TextField(),
  ),
)

Android 10, 1080 x 1260 pixels, xxhdpi

Hi, I created a temporary solution for this problem until it gets properly fixed.
Can someone please explain to me why the iOS offset is set to be negative? I made a custom TextField widget and made it positive again and it seems to be working fine.
@GaryQian Thank you for looking into this issue and providing the height: property in your pull regarding this 😌
I'm new and loving Flutter! Everyone stay healthy!

@GaryQian Are there any updates on the @PiN73 issue? A cursor is noticeable smaller when there is no text, an issue is still on the master channel. Would be awesome if someone fixes that

@GaryQian Are there any updates on the @PiN73 issue? A cursor is noticeable smaller when there is no text, an issue is still on the master channel. Would be awesome if someone fixes that

Yeah I’ve been dealing with this recently. You have to set the height of line height I think.
I made a post on it in StackOverflow - look up

flutter doctor -v
[✓] Flutter (Channel dev, 1.19.0-3.0.pre, on Mac OS X 10.15.5 19F101, locale
    en-GB)
    • Flutter version 1.19.0-3.0.pre at /Users/tahatesser/Code/flutter_dev
    • Framework revision 6135091de9 (3 days ago), 2020-06-01 17:17:03 -0700
    • Engine revision e39301f23f
    • Dart version 2.9.0 (build 2.9.0-13.0.dev 6489a0c68d)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

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

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (5 available)
    • SM M305F      • 32003c30dc19668f          • android-arm64  • Android 10
      (API 29)
    • Taha’s iPhone • 00008020-001059882212002E • ios            • iOS 13.5.1
    • macOS         • macOS                     • darwin-x64     • Mac OS X
      10.15.5 19F101
    • Web Server    • web-server                • web-javascript • Flutter Tools
    • Chrome        • chrome                    • web-javascript • Google Chrome
      83.0.4103.97

• No issues found!

Cursor height changes significantly even after single space is typed:

Container(
  color: Colors.grey,
  padding: EdgeInsets.all(16),
  child: Container(
    color: Colors.white,
    child: TextField(),
  ),
)

Android 10, 1080 x 2160 pixels, xxhdpi

I also have this issue. For some reason its height changes when a space or a character is typed in.
Really degrades the UI  :|

same here, cursor shrinks a bit after removing the input, and quickly changes back

same here, cursor shrinks a bit after removing the input, and quickly changes back

It's not a solution but it would help for now.
Thanks to https://stackoverflow.com/a/61151526/10304564 , you can navigate to the text_form.dart by right clicking on the TextForm() widget and clicking on Got to Definition. Once in the text_form.dart, navigate to line 1059 or find the cursorOffset and change it from its original value Offset(......) to cursorOffset = Offset(0, 0);
Hope that helps.

@adeolaex modifications in text_form.dart will be discarded if you upgrade Flutter / move the project to another computer / use CI/CD like codemagick, won't it?
It might be better to copy text_field.dart in the project and modify it then

Yes upgrading flutter and moving the project will most likely discard the modifications( don't know if a CI/CD would do that), but you can copy the text_field.dart into another file. The only drawback is you have to copy other dart files from the flutter sdk because they are used by text_form.dart.

CI/CD won't just know about your changes in framework code and will simply use its own version

When copying text_form.dart, there is no need to copy other files. You can just import them in text_dorm.dart as usually

Can you confirm all the imports work when copying it ? I tried it some days ago and it failed to recognise the URL target

@adeolaex it's because of relative imports. When you write e.g.
import 'theme.dart';

it tries to find file in the same directory.
Remove all relative imports and add
import 'package:flutter/material.dart';

instead.
Then you can use it like this:
import 'my_text_field.dart' as my_text_field;
...
my_text_field.TextField()

code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      home: Scaffold(
        appBar: AppBar(
          title: Text('Material App Bar'),
        ),
        body: Center(
          child: Container(
            color: Colors.grey,
            padding: EdgeInsets.all(16),
            child: Container(
              color: Colors.white,
              child: TextField(),
            ),
          ),
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.22.5 at /Users/tahatesser/Code/flutter_stable
    • Framework revision<PHONE_NUMBER> (6 weeks ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (1 available)
    • Taha’s iPad (mobile) • 00008020-000255113EE8402E • ios • iOS 14.3

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 1.26.0-2.0.pre.402, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.26.0-2.0.pre.402 at /Users/tahatesser/Code/flutter_master
    • Framework revision 2a188eeca3 (35 hours ago), 2021-01-17 19:27:00 -0800
    • Engine revision 609036f2bf
    • Dart version 2.12.0 (build 2.12.0-236.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • 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 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • Taha’s iPad (mobile) • 00008020-000255113EE8402E • ios            • iOS 14.3
    • macOS (desktop)      • macos                     • darwin-x64     • macOS 11.1 20C69 darwin-x64
    • Chrome (web)         • chrome                    • web-javascript • Google Chrome 87.0.4280.141

• No issues found!

code sample
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      home: Scaffold(
        appBar: AppBar(
          title: Text('Material App Bar'),
        ),
        body: Center(
          child: Container(
            color: Colors.grey,
            padding: EdgeInsets.all(16),
            child: Container(
              color: Colors.white,
              child: TextField(),
            ),
          ),
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.22.5 at /Users/tahatesser/Code/flutter_stable
    • Framework revision<PHONE_NUMBER> (6 weeks ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (1 available)
    • Taha’s iPad (mobile) • 00008020-000255113EE8402E • ios • iOS 14.3

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 1.26.0-2.0.pre.402, on macOS 11.1 20C69 darwin-x64, locale en-GB)
    • Flutter version 1.26.0-2.0.pre.402 at /Users/tahatesser/Code/flutter_master
    • Framework revision 2a188eeca3 (35 hours ago), 2021-01-17 19:27:00 -0800
    • Engine revision 609036f2bf
    • Dart version 2.12.0 (build 2.12.0-236.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • 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 1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • Taha’s iPad (mobile) • 00008020-000255113EE8402E • ios            • iOS 14.3
    • macOS (desktop)      • macos                     • darwin-x64     • macOS 11.1 20C69 darwin-x64
    • Chrome (web)         • chrome                    • web-javascript • Google Chrome 87.0.4280.141

• No issues found!

I have the same issue

Same issue, will this ever get fixed?

I am facing this type of cursor glitch.
Code:
                Container(
                  width: 140,
                  child: TextField(
                    controller: _pincodeController,
                    decoration: AppTheme.registrationInputDecoration(
                      labelText: "Pincode",
                      prefixIcon: Icon(
                        FontAwesomeIcons.mapMarkerAlt,
                        color: Colors.black,
                      ),
                    ),
                    style: AppText.registrationFieldStyle(),
                    onSubmitted: (str) => _searchService(),
                  ),
                ),

This widget is inside of a column.

@aadarsh-patel The bug you reported is a separate bug from this issue. Can you file a new issue and cc me on it + link to it here? Thanks!

The final solution to this issue may be a more configurable caret rendering system. It seems that depending on the use case, different caret behavior is desired. There may not be a one-solution fits all caret.
Perhaps something like the following may work better for most use cases:
TextField(
  ...
  caretStyle: CaretStyle.dynamic/fixed/strut/textStyle(
    heightAdjustment: ...,
    (various other params as necessary)
  ),
  ...
)

Meanwhile, we currently have already added the cursorHeight parameter to text fields. If you are experiencing the single space changing the caret height, try setting the cursorHeight to lock the caret height to a specific value.

I believe the original issue of caret alignment and height is resolved via new parameters, and more fine tuned control would likely require a breaking overhaul of the caret config system. Breaking changes/deprecating the current config is a large API commitment and thus I will close this issue for now. Let's continue to discuss the tangent bugs mentioned above in separate issues.


Remove unused method from ObservationTabsHelper

The prefs_tabset has since a few months been moved to the account preferences view.
This just deletes the method that used to define the tabset, back when AccountController was part of ObserverController.

Coverage: 92.514% (+0.008%) from 92.507% when pulling 30fc715747681238330637fc46e6dd9a896342d7 on nimmo-remove-unused-prefs-tabset into a49100a54808bd73f70e4bb3aa545af3e128fc69 on master.


📃: Archery Game

🔴 Title :Archery Game
🔴 Tech stack : html, css,js
🔴 Objective : our fav game :}
🔴 Summary :
📸 Screenshots

✅ Details to Include When Taking the Issue:
Name :ae-hertz
Participant Role (Specify the Open Source Proagram name, e.g., GSSOC, Hacktoberfest, etc.):

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎
Assign me this issue

This issue is not duplicate as their is no updates form existing its been morethan a month. So please dont mark it as duplicate.


Location RealTime not work

Description
[Description of the bug, When Issue Happens]
Environment

Wppconnect-server version(s): 2.0.0
OS: Windows 10
Node version: 19.9
WhatsApp version: ?
MultiDevice (BETA): yes
config.json:
repository default!

Steps to Reproduce

[First Step]
Connect in a session and ask a user to send a location in REAL TIME!

[Second Step]
Accompanying the console it registers the onLiveLocation which should send the location information in real time! This information ends up going only from the first location through "onMenssage" and in some cases comes 1 new webhook with "location" type! However, the location is not updated at any other time!

Log Output
Console Log

WebHook 1

need to handle a sequence of coordinates one after the other? similar to using a "for wait" loop


Create unprivileged OSD containers

Hi all.
I'm going to create a containerized Ceph cluster with unprivileged OSD containers. but all the scenarios defined in ceph-ansible use privileged containers. Is there a solution to get around this issue?
Thanks in advance.
English is not my mother tongue; please excuse any errors on my part.

Hello, unfortunately, there are currently no ways to run unprivileged  OSD containers. Things might change in the future with Bluestore though.
Please close this issue if I answered your question.


Typo fix: "estiamte"

Hello :) I found a typo while looking through the book. Figure 10.6 had the word "estimate" misspelled as "estiamte".

By the way, when do you expect the book to be published? I'd like to get a physical copy. :)

By the way, when do you expect the book to be published? I'd like to get a physical copy. :)


fix(Header): fix home url

** Please provide as much detail as possible, but feel free to remove irrelevant sections **
Change to correct home url
update surge version because npm suggested.
BREAKING CHANGE:
{{ Description of breaking API change }}
RFC URL:
{{ URL of associated RFC for API change }}
MIGRATION GUIDE:
Before:
{{ Old code }}

After:
{{ New code }}

EXAMPLE USAGE:
{{ Code }}

http://seek-asia-style-guide--c464210f7df8d5a5d6dd2213bbbc2537ae5e7687.surge.sh/

When you say correct home url - are the home urls in the localisations correct?

@mengtzu  yeah. checked the localization files.


Decrypt file in React Native

Hello,
I am looking for a way to decrypt files encrypted on server with this module on mobile app using React Native. Do you know how to do that?
This package does not work with React Native so I need to find another package for the mobile part.
Thanks for your help

Thanks for your message!
Unfortunately, there is no alternative to the @sounisi5011/encrypted-archive package. The data generated by this package is in a format specific to this package. Since I designed it, there is no other implementation.
However, the data format uses unsigned varint and Protocol Buffers, so you should be able to decrypt the file without using this package. All you need is:

A decipher function that supports AES-GCM (256-bit) or ChaCha20-Poly1305
You can use the Node.js built-in crypto module for this.

An Argon2 implementation
This package uses argon2-browser, but argon2 is also available.

It's interesting that @sounisi5011/encrypted-archive doesn't work in React Native. I'm not familiar with React Native, but I'll try to fix it as best I can.
If possible, could you please provide the following information?

Version of React Native
Target platform for your app - iOS, Android, etc.
The operating system used to build - In other words, the operating system of the computer you are using to develop.
Circumstances in which the problem occurs - When you build, launch an app, try to decrypt, etc.
Problem details - Error messages are generated, apps are force-quit, decryption always fails, etc.

Thank you for your answer @sounisi5011.
React Native does not have the same modules as nodejs, that's why compatibility is not there by default. Here are some details about my current setup :

react native: v0.66
nodejs: v16
os: macos 12.01

The problem happens when I try to decrypt a file, just as I import the decrypt function (the app crashes)

Interim report: Currently, I'm using React Native, which I built at Expo, to conduct my research.

https://github.com/sounisi5011/react-native-test--using-sounisi5011-encrypted-archive
https://github.com/sounisi5011/react-native-test--using-sounisi5011-encrypted-archive/pull/2#issuecomment-991852583

Since the @sounisi5011/encrypted-archive package was designed for Node.js, it depends on many built-in modules that React Native does not support. These can be solved by editing metro.config.js. However, a more serious problem was found. Support is going to take a long time.

Thank you for your time @sounisi5011. On my side, I try to implement a decrypt without this package as you described before.

I am still aware of this issue. However, I can't fix it for now because I can't build Native Modules for React Native.
At the very least, @sounisi5011/encrypted-archive requires an Argon2 library that supports binary data input. However, the React Native-compatible Argon2 library that currently exists in npm only accepts string input. For this reason, I need to choose one of the following:

create my own React Native-compatible Argon2 library
send a pull request to the existing Argon2 library
wait until React Native supports WebAssembly

At first, I tried 1. However, I could not build React Native on my macOS using XCode and CocoaPods. I get an unknown error, and moreover, the error is not reproducible. I could not even choose 2 because I could not build it in my local environment.
With react-native-wasm, I can solve this problem without waiting for 3. But methods like react-native-wasm use WebView. I think the way a mere encryption/decryption library calls WebView has a fatal negative impact on the performance of an app using @sounisi5011/encrypted-archive.
Writing again, I am still aware of this issue. I have not forgotten about it.
But I am leaving it now.
I will keep this Issue open until it is resolved, but I don't know when I will be able to resolve it (possibly I will have to get a new Mac to use for development only).


Refactor and update heatmap

Extract slice and formData according to [SIP-5]
Fix small issues and syntax inside heatmap.
Move legend to the right
Make axis labels align to the ticks

Codecov Report

Merging #5704 into master will decrease coverage by 0.14%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #5704      +/-   ##
=========================================
- Coverage   63.45%   63.3%   -0.15%
=========================================
  Files         361     361
  Lines       22972   22978       +6
  Branches     2557    2558       +1
=========================================
- Hits        14576   14546      -30
- Misses       8381    8417      +36
  Partials       15      15

Impacted Files
Coverage Δ

superset/assets/src/visualizations/heatmap.js
0% <0%> (ø)
:arrow_up:

superset/db_engine_specs.py
52.14% <0%> (-2.01%)
:arrow_down:

superset/utils.py
88.22% <0%> (-0.91%)
:arrow_down:

superset/connectors/sqla/models.py
77.86% <0%> (-0.77%)
:arrow_down:

superset/models/core.py
84.6% <0%> (-0.5%)
:arrow_down:

superset/views/core.py
73.64% <0%> (-0.3%)
:arrow_down:

Continue to review full report at Codecov.

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


Plug hole that allowed concurrent replication streams

fix for concurrent replication streams
update the 'first seqnum' in replication code-path (this was missing)
added more logs around open/close extent

Coverage increased (+0.002%) to 66.63% when pulling f7189ba803112598f94b19b90f9464d77f6b6bba on storehost-replication into 65a1bb1404b63ce07025a2a3f6d34187fddce030 on master.

Can you add unit test?

There is a unit-test that checks to see that when two OpenReplicateStream requests come in, only one will succeed and the other will fail. Unfortunately, this bug shows up when a third OpenReplicateStream request comes in!

Coverage increased (+0.3%) to 66.895% when pulling d1b5c4df9a1dc9dd53b78d598e061fdff37f03fe on storehost-replication into 65a1bb1404b63ce07025a2a3f6d34187fddce030 on master.

Coverage increased (+0.1%) to 66.763% when pulling 2ef7258288103c4b5040d1433fe29f9b099f5768 on storehost-replication into 65a1bb1404b63ce07025a2a3f6d34187fddce030 on master.

@thuningxu I have updated the existing test to try another (third) replicate-extent request, which should fail; verified that it fails without fix and passes with.


Update MultiQC version

A recent MultiQC update broke the pipeline. I've switched back to a past version and it's running fine, but it would be nice to stay current.

Still need to update the pipeline to successfully use the new version.

Given that we're not using any of the fancier features of MultiQC and are essentially using it as a FastQC data aggregator, and might be switching to Falco anyway, I'm no longer sure it makes sense to use MultiQC here at all. I expect we'll revisit this after we've made the changes discussed in #74 and #78.


FOSDEM 2019 Python devroom

- title: FOSDEM 2019
  dir: fosdem-2019
  language: eng
  related_urls:
  - label: Event schedule
    url: https://fosdem.org/2019/schedule/track/python/
  videos: # Some of this page (python*)
    https://mirror.librelabucm.org/fosdem/2019/UD2.120/

No minimal download coded for this in https://github.com/Daniel-at-github/pyvideo_scrape


Bugfix: meta_train_dataset etc. are attributes of benchmark

Since meta_train_dataset etc. are attributes of benachmark, accessing meta_train_dataset results in a error.

Nice catch, thank you!


[DO NOT MERGE] ci: moving universe and docs integration tests to the dagger CI plan

This is a follow-up of PR #2284
It adds integration tests for:

documentation
universe packages

Closes #1522

The timeout you are getting in the universe tests should be fixed by this: https://github.com/dagger/dagger/pull/2495

Comparing the previous runs of integrations tests, looks like we took a 3x hit on performance for universe tests... I'll dig a bit deeper before we can merge this.

Just re-ran the universe tests and they only took 6m, which is pretty good. I highly suspect what happened on the previous run was that we had too many tests running in parallel between this PR and others, which causes GHA cache to throttle us and take longer to download/upload cache. Issue for that here: https://github.com/dagger/dagger/issues/2365
The tracing I added to the integration tests let us 100% confirm that's what is happening, but I've held off on adding them to the universe tests just because there are several open PRs it would conflict with. So I can't be 100% sure that's what happened but feel fairly confident given the symptoms.
Either way, I think this should be good to merge.


This seems good, but please note that in React Fiber, this probably won't be the case, as you'll be able to use a promise for a component as if it were a component.

A promise for a component or a promise for an element?

@taion a promise for a component, is my understanding.

I'm not sure what you mean. A promise for a component class? These aren't that, and there are no component instances involved here anyway.
These would be component classes that try to render a promise for elements, which wouldn't be the same as the parent of one of these components trying to render a promise for elements.

Merged, thanks!


Tree Sort in Python

🚀 Feature
Sorting a list of numbers using binary search tree in Python
Have you read the Contributing Guidelines on Pull Requests?
Yes
Pitch
Tree Sort is implemented using Binary Search tree which is an important data structure. A binary search tree is constructed with the numbers to be sorted and inorder traversal is performed to get sorted list

/assign


Broken badge lmages in README.md

There is some issue with the images right now, they seem to be broken. @the-dagger Please check if this is the issue with you as well.

@mananwason This seems to be fixed now.
Can you confirm?

Closing this, as badges show up fine.
Tried from multiple browsers.


[PR][CLOSED] [PR-Comment] Rephrase incorrect sentences

 Issue by myungjoo-ham
Friday Jun 15, 2018 at 00:42 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/pull/30

The previous sentence, "Note that you don't must merge your own PR firsthand",
was quite itching. The sentence is rephrased.
Note that this does not change any logic / behaviors.
Signed-off-by: MyungJoo Ham<EMAIL_ADDRESS>
myungjoo-ham included the following code: https://github.sec.samsung.net/STAR/TAOS-CI/pull/30/commits

:octocat: cibot: Thank you for posting issue #47. The person in charge will reply soon.


bug

An important bug was reported by another user, I tested it and am also able to reproduce the issue, but the bot mistakenly closed the original #226501 and marked it as invalid, without a response.
I request that #226501 be reopened and then close this issue itself.

looks like it was closed by the bot because it was created by a deleted user - reopening and re-assigning


Update Go dependencies

Description
Upgrade go-ethereum and correct Web3 gateway reference.
Supersede https://github.com/oasisprotocol/sapphire-paratime/pull/201
TODO

[x] Test with go-ethereum tutorial locally

Task linked: CU-8692vytr5 Bump go-ethereum dependency

Task linked: CU-8692vytr5 Bump go-ethereum dependency

Task linked: CU-8692vytr5 Bump go-ethereum dependency

Seems like you got hit by a mystery bug in one of the CI contracts tests (unrelated to this PR):

  1) Signing
       Secp256k1 (Oasis):

      AssertionError: expected true to equal false
      + expected - actual

      -true
      +false

      at testSignThenVerify (test/signing.ts:48:7)
      at processTicksAndRejections (node:internal/process/task_queues:95:5)
      at async Context.<anonymous> (test/signing.ts:165:9)


Password/token lost on Android Studio at each start

The plugin is really great and I'm kinda used to the new workflow, unfortunatelly I have a small issue with Android Studio which reset the password each time the IDE is closed (the field is blank).
I tried with a password and a token but to the same result unfortunatelly.
I'm using Windows 10 if it's revelant.
I used two different Android Studio (4.0 and 4.1) both has issue and the last one is based on Intellij 2020.1.

The plugin doesn't store passwords and tokens by itself, delegating to the native IDE secure store feature instead. It might be that you have selected "Do not save, forget password after restart" in Appearance & Behavior | System Settings | Passwords in settings.
Could you please check if this is indeed the case?

Ah yes I thought about that but forgot to check how the passwords were saved in the IDE.
It should works now, thanks !


Publish v6 on npm

Hi there!
I see that version 6.0.0 is available in package.json on master. But not on npm. Why ? Is there any reason ? Do you need help ?

It is available from NPM: npm i<EMAIL_ADDRESS>It is just not marked as a latest version. I'm not sure about the details, but looks like it had some issues and oughta be released once those are resolved.

Last I heard, about two months ago, Protractor 6 wasn't ready for prime-time. They talked about it in Gitter.
The specific issues that prevented me from updating our automation to Protractor 6:

Can't run in sharded mode due to bug in Protractor
Screenshots are broken (?)
Can't run without seleniumAddress unless directConnect: true
Documentation out of date
Typings out of date

Here's Craig Nishina's comment on it (via gitter):

Yes unfortunately Protractor 6 is not ready for prime time. I did a release and found some issues. We will create a beta version with fixes. This will be 6.0.1-beta.1.


Update delivery dates

Grant Application Checklist

[ ] The application template has been copied, renamed ( project_name.md) and updated.
[ ] A BTC or Ethereum (DAI) address for the payment of the milestones is provided inside the application.
[ ] I have read and acknowledged the Terms and Conditions.
[ ] The software delivered for this grant will be released under an open-source license specified in the application.
[ ] The total funding amount of the project is below USD $30k for initial grant applications and $100k for follow-up grants.
[ ] The initial PR contains only one commit (squash if needed before submitting your PR).
[ ] The grant will only be announced once the first milestone has been accepted.

For reference see https://github.com/w3f/Open-Grants-Program/pull/351#issuecomment-877489859


Ignores yml indentation

VSCode Version: 1.38.1 b37e54c98e1a74ba89e03073e5a3761284e3ffb0 x64
OS Version: Linux Mint 5.0.0-27-generic #28~18.04.1-Ubuntu SMP Thu Aug 22 03:00:32 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Steps to Reproduce:

open / create yml file
set settings to

"editor.tabSize": 2,
"editor.insertSpaces": true,
"editor.detectIndentation": false,

and tested with
"[yml]": {
  "editor.insertSpaces": true,
  "editor.tabSize": 2,
  "editor.detectIndentation": false
}

(global setting is "tabSize: 4", but even setting it to 2 dosnt work)
Does this issue occur when all extensions are disabled?: Yes
Worked in code version 1.37.1

nvm it was a prettier issue


Why 32? Why 8? For every number, please add a [...]

https://github.com/Tox-Docs/Text/blob/master/src_text/DHT.txt#L54
Comment on
Toxcore stores the 32 nodes

found at column 1.
Why 32? Why 8? For every number, please add a reason for it, and a reasonable range for other implementations. Describe the effect a change to those numbers has on the distributed system and on the node itself.

I picked these numbers because they felt like the best numbers and since they worked they stayed the same.
Not sure how to explain that in the docs.

@irungentoo "Unless otherwise specified, each number was chosen because it was likely to be the best, and haven't been changed for legacy compatibility."

https://github.com/Tox-Docs/Text/commit/b74d74781d8108e3422a51cbc32ca02dda2a49b1


ReferenceError: Can't find variable: require

I have everything up and running just fine using Karma/Mocha/Chai by itself, but when I try to add chai-as-promised (along with karma-chai-as-promised) I keep running into this wall:
PhantomJS 2.1.1 (Windows 8 0.0.0) ERROR
  ReferenceError: Can't find variable: require
  at node_modules/chai-as-promised/lib/chai-as-promised.js:2

the line that it seems to be failing on in chai-as-promised.js is
var checkError = require("check-error");
I've tried adding Browserify and Karma-Browserify, but I still get the same issue. Is there some other pre-requisite I need?
It does this with Chrome as well:
Chrome 56.0.2924 (Windows 8.1 0.0.0) ERROR
  Uncaught ReferenceError: require is not defined
  at node_modules/chai-as-promised/lib/chai-as-promised.js:2

The same issues for me, and I can't find any workaround for this

And according to the documentation:
Chai as Promised is only compatible with modern browsers (IE ≥9, Safari ≥6, no PhantomJS).
So my issues was that I've tried to run it under PhantomJS.
I'm still trying to find solution with some browserify plugin for chai-as-promise module

This doesn't appear to be a phantom issue as it does the exact same thing with Chrome

Yep, this is not a PhantoJS issue, it is because chai-as-promised implemented as ES6 module, which doesn't have require method.
So I fixed the issue by load module directly in karma entry point file:
import chai from 'chai'
import chaiAsPromised from 'chai-as-promised'

chai.use(chaiAsPromised)

But expect was not loaded globally (and I don't know how to fix it)
And in tests where I need chai-as-promised assertions I'm using:
import { expect } from 'chai'

So I'm not using karma-chai-as-promised. When I tried it, I felt into the same issues.
So I hope my workaround will help you.

It looks like the require line is not in version 5.x, so you can use that one for now, but I am still curious how to get it to work with version 6.x and any future versions

Over any browserify transpilers

From the readme:

To use Chai as Promised in environments that don't support Node.js-like CommonJS modules, you'll need to use a bundling tool like browserify. See also the note below about browser compatibility.

I use karma + webpack + karma-chai-as-promised and I'm getting this error


Define a standardized format for map issues

Currently to-fix has hardcoded logic to read from various sources of CSV error streams. This does not scale well, and thought a better approach might be to define a standard spec for QA reporting tools to generate issues. Benefits:

Map QA APIs that can talk to each other
Allow any micro tasking QA tool to interpret issues from any source
Help expand the ecosystem of QA tools for more varied approaches and ultimately a better map

Went through some of the popular QA tools to understand the existing specs:

maproulette challenge
osmose API
keepright
to-fix project

It seems like there are three useful ways to generate issues from OSM data:

Postgres -> CSV
Overpass -> Geojson
osmlint -> Geojson

CSV is a nice natural format for a listing of issues.
Map QA issue spec
Draft https://gist.github.com/planemad/9e2a010ce5fcdefb38192dd63e925968
Not sure if this all makes sense, but this has been chewing my head for a while and had to put it out. Now I can 😴
cc @mvexel @mueschel @jocelynj @Rub21 @aaronlidman @batpad @ajithranka @maning

I think to-fix should use GeoJSON. GeoJSON should be the standard.
We decided on CSV for a couple of reasons which no longer hold up:

we were mostly using keepright data
we were not primarily using spatial data but tabular data with references to osm ids
to import millions of items into Postgres as fast as possible with COPY

Now:

we almost entirely provide our own data from osmlint or other sources
the data we're generating is almost always line-delimited GeoJSON
we have better tooling around line-delimited GeoJSON like tippecanoe and other CLI tools

these tools will only become more numerous

we should write a tool to stream data into Postgres quickly, relying on pg-copy-streams

If we define a spec it should be as simple as the simplestyle-spec (inheriting as many existing properties as possible) with a few extra properties specific to map errors.
This change would be fundamental to how the backend and frontend works but I think this is the best way of doing it and timing couldn't be better. We have upcoming projects that need to use to-fix and will be outputting ldgeojson.

I think we can literally use the simplestyle-spec to start this.

each feature is on it's own line
each feature is valid geojson
features are styled and UI slightly modified based on simplestyle-spec properties found

@planemad - What do you think? Who in Bangalore is in a good spot to run this out with @Rub21 and I?

Beyond the GeoJSON specification, I think it will also be helpful if we can standardise the way we represent

osm data (like feature type, id, tags, user details etc.), and
relations.

Both these have been pain points when using generic GeoJSONs in Autochethan. Different tools we have output these in varying formats. As a standard, the format returned by osmtogeojson might be a good place to start.
I'm completely in for having the styling information as part of the GeoJSON. This would simply much of fiddly bits on the frontend. I have a few concerns about using the simplestyle-spec though:

The spec doesn't map one-to-one with the GL style properties. If we are going to display these features on a GL map, there will be some fiddling required to map these to the correct GL paint properties.
The spec is limited and doesn't include some useful things like labels (though we could repurpose title for that). Labels have been very useful when representing multiple features (like from, via and to as part of a turn restriction).

Would it be better if we use properties from the GL style spec directly instead of simplestyle?

@aaronlidman @ajithranka @planemad
I agree to standardize the import files to GeoJSON. I guess that every task will be created by each geojson file. Nice!
Couple of things:

Standardize the properties maybe isn't really necessary, maybe we can do it with just a couple of features (id of the objects which intervene in an error) to recognize and then do a selection of this object JOSM or iD,  in case of geometry it is already established  for Point, LineString and Polygon.

Handle the best way to serve the API for to-fix-plugin and to-fix-web. Some  features would be nice to serve for  to-fix-web but not necessarily for to-fix-plugin because it could turn slow the plugin.

We need to figure out how to do more faster the transfer between to-fix-backend and clients (to-fix-plugin and to-fix-web), make a request to to-fix API + download data in JOSM or iD could spend a time, so I think is necessary to simplify the Geojson and reduce the features to display the error.

For example:
osmlint: unconnected-highways validator detects for an error two roads which are unconnected and a point where to display the exact point to resolve. The point contains the id of the objects that intervene in the error, so if we are going to transfer a bunch of data (two roads + a point) it could take time, so we need to filter the geojson with an exact errors to display.
osmlint: crossing-highways validator detects two roads and a point where the roads are in crossing, in this case also the API should serve just the point where the road are crossing.
So, for some issues, is not necessary to serve all context of the roads, only the point where to display the error, all context of roads we will get them in JOSM or iD when we start the edition. But I think it depends on what Geojson we will load in to-fix, maybe we can build a small tool to filter the exact features we want, but if the community wants to upload a geojson with high context it should work too.
After talking with @ajith autochetan tool gets the Geojson file in once and load it, in the case of to-fix I think we should continue taking feature by feature as we are doing now  to take control  of  fixed and not an error issues.

MapRoulette uses plain GeoJSON, it would be great if the formats would be interchangable.

Here a JSON version of the CSV schemas proposed by @planemad
https://gist.github.com/rodowi/0bd87524ee8c6c5594d7bffd1e5ea29a

Basically the metadata lives in a separate JSON.
Following the suggestions of @aaronlidman the project issues are listed as line-delimited GeoJSON features in a separate file. Issues can be grouped by project_id.
I'm skipping the styling properties because I still don't understand the requirements on that front.

cc: @Rub21
Any comments?

Switched to GeoJSONs for tasks.


Knife SSH fails with signed ssh key

Description
Using Knife SSH with a signed ssh identify file fails.  We have a setup similar to what is outlined here: https://www.digitalocean.com/community/tutorials/how-to-create-an-ssh-ca-to-validate-hosts-and-clients-with-ubuntu.  We allow access to VMs through signing of SSH keys by a CA that is included on all boxes.  This functionality works fine for normal commands such as ssh @.  When attempting to use knife ssh and pass in the signed identify file it fails reverting back to user/pass.
We believe this to be an original limitation on the net-ssh gem which knife ssh relies on.  This has since been fixed in net-ssh gem.  What are the chances we can get this functionality upstream into knife so that we can use commands such as knife bootstrap or knife ssh with a signed ssh key?  I believe there needs to be some changes in knife ssh to allow for this functionality but not 100% sure what the issue is.
This was implemented in net-ssh in this commit: https://github.com/net-ssh/net-ssh/pull/485
Chef Version
'''
╰─ chef --version
Chef Development Kit Version: 1.2.20
chef-client version: 12.18.31
delivery version: master (0b746cafed65a9ea1a79de3cc546e7922de9187c)
berks version: 5.5.0
kitchen version: 1.15.0
'''
Platform Version
OSX 10.11.6
Replication Case
You may need to follow a guide similar to what is outlined below.
https://www.digitalocean.com/community/tutorials/how-to-create-an-ssh-ca-to-validate-hosts-and-clients-with-ubuntu
You should then be able to ssh to a given box using standard ssh with that signed key.  Using knife ssh will fail.
Client Output
I can provide this output but I had to muck around with a few things to even get the updated net-ssh gem working with knife so I am not sure how valuable that will be here.
Stacktrace
Likewise, I can provide a stacktrace but I was manually injecting the updated net-ssh gem into the chef gem space and forcing knife to use it.  This was not fruitful as I believe there are other compatibility issues with net-ssh 4.x and the current knife ssh code.
Apologies for some of the vagueness here and I will gladly provide more information as needed.

Can you link to the commit/PR where net-ssh added this? Would make it easier to track which version of things we need.

I updated the description to include the PR where this was fixed.  Thank you for the prompt response.

There hasn't been a release of net-ssh since then, so there is nothing for us to do at this time. It doesn't look like this requires any code changes on our side so this will be picked up during the normal dependency upgrades in the ChefDK release after net-ssh releases the change. ChefDK releases are usually in the 4th week of each month.

@coderanger I agree with that assessment that there hasn't been an official release which includes this but I tried to validate it ahead of time by building the gem locally myself and forcing knife to use it.
There was a breaking change on this line: https://github.com/net-ssh/net-ssh/blob/master/lib/net/ssh.rb#L263  as it used to default to true.  That is no longer the case and I initially received an argument error.  After tweaking that and getting knife to play nicely with this gem I still ran into an issue where it simply failed on using the key and reverted back to user/pass.
I am not convinced this will just work when the latest version of chefdk pulls it in but I understand if there is not much action to take until that point.

Unfortunately updating things piecemeal inside ChefDK is tricky because of how we package things. If you install the beta gem and then edit the top of your knife executable and update the gem 'net-ssh'... line it should probably work but you would be in somewhat uncharted territory since we haven't tested the beta of net-ssh :)

@coderanger That is essentially what I did but still ran into issues.  I did verify it was using the new version of the gem as in the -vv output of knife ssh I can see it using the version I specified.   I will prove out this functionality works in net-ssh gem next as maybe their implementation is flawed.  Then maybe look to follow back up for the knife side of things down the road when this makes it into an official release.
I realize I am in uncharted waters here! I greatly appreciate the info and quick responses.

@coderanger We actually were able to get this working in our hacked together knife.  We ended up having to drop the -i and rely on ssh-agent to provide the keys.  When passing in a single key with -i set, it tripped the keys_only option which didn't allow the ssh agent to grab the other cert identity and pass that in.  At least thats my suspicion.
Sounds like we just have to wait for an official release of net-ssh and then get that into knife. Appreciate the quick responses yesterday.  I believe this can likely be closed unless you want to keep it open until this gem version makes it into knife.

yeah, this will just happen as part of the release process of net-ssh and chef/chef-dk and leaving it open doesn't help us track it at all


Feature test: for the step And the user gets...

Since the tests now support both sentences ("the user gets..." and "I get..."), should one of the tests still retain "the user gets..." language to make sure that is still tested?
Originally posted by @niharikasingh in https://github.com/plocket/docassemble-cucumber/pull/288#discussion_r656422994

Sorry it's been on so long. This is low hanging fruit I'm hoping we can give to someone new if they come along.


Update cats-effect version to 3.3.0

Steward is a bit late for now, so here is CE 3.3.0 (though it doesn't migrate changes of TestContext https://github.com/typelevel/fs2/runs/4344942116?check_suite_focus=true).

Trying to do my best :)

Btw, where is steward? It just opened https://github.com/http4s/http4s/pull/5621 but nothing for CE3.


There needs to be a way to execute Rule CG0664

This PR adds 2 new operations for validating column order against library metadata: get_column_order_from_dataset and get_column_order_from_library.
Rule example:
{
   "core_id":"MockRule",
   "standards":[
      {
         "Name":"SDTMIG",
         "Version":"3.3"
      }
   ],
   "classes":{
      "Include":[
         "Events"
      ]
   },
   "domains":{
      "Include":[
         "AE"
      ]
   },
   "sensitivity": "Dataset",
   "operations":[
      {
         "operator":"get_column_order_from_dataset",
         "domain":"AE",
         "id":"$column_order_from_dataset"
      },
      {
         "operator":"get_column_order_from_library",
         "domain":"AE",
         "id":"$column_order_from_library"
      }
   ],
   "conditions": {
      "any":[
         {
            "name":"get_dataset",
            "operator":"not_equal_to",
            "value":{
               "target":"$column_order_from_dataset",
               "comparator":"$column_order_from_library"
            }
         }
      ]
   },
   "actions":[
      {
         "name":"generate_dataset_error_objects",
         "params":{
            "message":"Order of variables is invalid"
         }
      }
   ],
   "output_variables":[
      "$column_order_from_dataset",
      "$column_order_from_library"
   ]
}

The idea is that get_column_order_from_dataset operation returns a list of dataset columns in accordance to their order and get_column_order_from_library gets the similar list of columns not from the dataset but from the library metadata. Finally, not_equal_to operator is used to check the equality of these lists.

Based on some information I got from the standards developers. Checking order is not as simple as getting it from the domain. Here is the logic we agreed upon that should work in all cases:
Given a domain and that domains class. Check the parent MODEL for ordering using the logic as follows:

If the model has the domain, pull the expected order from the model domain
If the model does not have the domain, pull the expected order from the model class. If the class is a general observations class (Findings, Events, Interventions) Add the Identifier Variables from the general observations class to the beginning of the list of variables and the Timing Variables from the general observations class to the end of the list of variables.
Verify that the order of variables matches the order specified in the resulting list.
Note: Some variables are permissible so they could be excluded and the variable order still be correct.

Ex:
// Validating variable order for a domain in a general observation class
// Suppose this is the variable order specified in the model class
{
  --TESTCD: 1
  --STRESC: 2
}
// And this is the variable order specified in the general observations class
{
  IDVAR: 1
  TIMINGVAR: 2
}

// Then the correct order of variables to validate against would be:
{
  IDVAR: 1
  --TESTCD: 2
  --STRESC: 3
 TIMINGVAR: 4
}

Now assuming that TESTCD is permissible the following variable order is still valid
[IDVAR, LBSTRESC, TIMINGVAR]
This is because the ordinals associated with those variables are in order: [1, 3, 4]

Based on some information I got from the standards developers. Checking order is not as simple as getting it from the domain. Here is the logic we agreed upon that should work in all cases: Given a domain and that domains class. Check the parent MODEL for ordering using the logic as follows:

If the model has the domain, pull the expected order from the model domain
If the model does not have the domain, pull the expected order from the model class. If the class is a general observations class (Findings, Events, Interventions) Add the Identifier Variables from the general observations class to the beginning of the list of variables and the Timing Variables from the general observations class to the end of the list of variables.
Verify that the order of variables matches the order specified in the resulting list.
Note: Some variables are permissible so they could be excluded and the variable order still be correct.

Ex:
// Validating variable order for a domain in a general observation class
// Suppose this is the variable order specified in the model class
{
  --TESTCD: 1
  --STRESC: 2
}
// And this is the variable order specified in the general observations class
{
  IDVAR: 1
  TIMINGVAR: 2
}

// Then the correct order of variables to validate against would be:
{
  IDVAR: 1
  --TESTCD: 2
  --STRESC: 3
 TIMINGVAR: 4
}

Now assuming that TESTCD is permissible the following variable order is still valid
[IDVAR, LBSTRESC, TIMINGVAR]
This is because the ordinals associated with those variables are in order: [1, 3, 4]

Thanks for the clarification, I would only like to understand where can I take the parent MODEL? As I understand, it is not the standard details that I am using right now.

@alexfurmenkov we have not been caching the model metadata because people would be using implementations instead of models. However, every implementation guide (SDTMIG 3.4, SDTMIG 3.3, etc...) implements a model. The model it implements can be found in the model link in the response when requesting the implementation guide from the library:
{
  "implementedBy": [],
  "model": {
    "href": "/mdr/sdtm/2-0",
    "title": "Study Data Tabulation Model Version 2.0",
    "type": "Foundational Model"
  },
  "priorVersion": {
    "href": "/mdr/sdtmig/3-3",
    "title": "Study Data Tabulation Model Implementation Guide: Human Clinical Trials",
    "type": "Implementation Guide"
  },
  "self": {
    "href": "/mdr/sdtmig/3-4",
    "title": "Study Data Tabulation Model Implementation Guide: Human Clinical Trials",
    "type": "Implementation Guide"
  }
}

We likely will have to add some data to our cache, and pull data from the models. I would suggest updating the cache feeder in a separate issue, and having that issue block this PR.

@alexfurmenkov we have not been caching the model metadata because people would be using implementations instead of models. However, every implementation guide (SDTMIG 3.4, SDTMIG 3.3, etc...) implements a model. The model it implements can be found in the model link in the response when requesting the implementation guide from the library:
{
  "implementedBy": [],
  "model": {
    "href": "/mdr/sdtm/2-0",
    "title": "Study Data Tabulation Model Version 2.0",
    "type": "Foundational Model"
  },
  "priorVersion": {
    "href": "/mdr/sdtmig/3-3",
    "title": "Study Data Tabulation Model Implementation Guide: Human Clinical Trials",
    "type": "Implementation Guide"
  },
  "self": {
    "href": "/mdr/sdtmig/3-4",
    "title": "Study Data Tabulation Model Implementation Guide: Human Clinical Trials",
    "type": "Implementation Guide"
  }
}

We likely will have to add some data to our cache, and pull data from the models. I would suggest updating the cache feeder in a separate issue, and having that issue block this PR.

Okay, thanks. I think that we should implement this in terms of this PR and Issue since it is about supporting this rule. I will work on it.

@nhaydel we need to request the model for each standard, right?

@nhaydel we need to request the model for each standard, right?

yes. The products endpoint also provides a list of models if you want to just preemptively load them all

@nhaydel we need to request the model for each standard, right?

yes. The products endpoint also provides a list of models if you want to just preemptively load them all

@nhaydel updated the logic to cache model metadata and use it in the operation. What do you think, should we update existing pkl files with model metadata?


how to deal with text recognition with images whose width are more than 280

thanks for your sharing, great work.
I am using your code with CRAFT to deal with a chinese and english text recognition task. I set the image width as 280. And the width of training data of chinese is all 280, but the training data of english is like yours as MJ.
However, the width of the output chinese text image of CRAFT is usually more than 280, and the english text image of Craft is less than 280. So the recognition accuracy for the real chinese document is very low. So can I solve this problem?


Move Windows binary to bintray

Since sourceforge is horribly unreliable.
also fixup REQUIRE

Thanks! I think the changes are uncontroversial. But do you know if the failure on AppVeyor related to 0.4 or windows?

Probably 0.4, similarly to what's happening on Travis. We could adjust the AppVeyor setup to build only on stable, or both stable and nightly.

What about just putting them on S3?

I only have indirect access to S3. The prefix url ends up caching these to S3 since they're added to a whitelist at https://github.com/staticfloat/cache.julialang.org

@tkelman It would be good to have AppVeyor test for stable. I don't have access to a Windows machine to test, although I doubt that your changes cause any problems...

Okay, I adjusted appveyor.yml to the latest recommended version which will test on both nightly and 0.3. I'm not sure why, but it looks like this repository was enabled in appveyor by both @tshort and @scidom, and the status links are occasionally overwriting one another. I can probably disable one of the webhooks, if you don't mind. Also it would be good to contact appveyor's support address and ask for your accounts to be upgraded to the newer free-and-fast plan, since these builds are looking pretty slow and taking a while to queue.

Thanks again. I guess I can merge this now? It would be great if you could look into the webhooks. Who should ask the support for the upgrade?

I don't think this makes the situation any worse, Travis and Appveyor are all verifying that things are still working on 0.3 and still broken on 0.4. I disabled the webhook that I think was going to @tshort's AppVeyor account since I'm pretty sure it's redundant. If @scidom could send an email to<EMAIL_ADDRESS>asking to have his account moved to the faster VM's (ref http://www.appveyor.com/blog/2015/06/23/new-oss-build-environment-and-xamarin-support) then future builds should run faster and spend less time queued.

Thanks @tkelman for figuring that out...


fix tuple empty error when module is disabled

what

data source used in local value returns an empty tuple

why

Terraform errors out with empty tuple error, caused by data source is disabled correctly, but not taken care of in locals value where it is being used.

references

Issue link

Merge this code please, it really help


Mute says it succeeds, then fails

Really loving this client, it's just what I've been looking for.  One problem I'm having though, whenever I try to mute a user, it says they are muted, and then immediately says it's failed.  And indeed they are not actually muted.

[@EricAsberry]: mute @somenoisyuser
@somenoisyuser is muted.
Something is wrong, can not mute now :(

I've tried restarting, and also insured the config file is accessible by the user.  Not sure what else to check.  Thanks!

This appears to have been caused by a typo in the code. I created a fix, which I'll submit as a pull request shortly.

Thanks! This is fixed in v1.3.2.


No opencollective found

Been using Tailwind CSS for a while now and wanted to make a very small donation ($5) but could not find where to do it. I'm thankful for your work on this, and would like to gift something back, probably more people would like to do that too.

Me too! Though I think GitHub Sponsors would be more appropriate/immediate. Is sponsorship available for an organization?
Anyway, I’d be happy to give $5/mo or similar. 👋


Error on Video Encoding

I've just been doing a screen recording (< 1minute in length) and had this error when stopping the recording.
To clear this error I had to go to the folder that it says, and then delete all of the contents and force quit the application & reboot.
I'm on 10.10.3 btw & attached is a screenshot.

Hmm. That'd be related to using 10.10.x. Video recording has to save every frame to a temporary folder, so that can be pretty intensive on older machines too.
I've been looking at ways of encoding frames directly into the video file but that doesn't seem to be possible as of now. Try using quicktime to record or update to 10.11.

An odd one, having used it for a few more days (on the same setting) I am yet to experience anything similar.
Will report back if it raises it head!


Repeater with BelongsToMany relationship doesn't work as expected

Package
filament/forms
Package Version
v3.0.53
Laravel Version
v10.24.0
Livewire Version
v3.0.5
PHP Version
8.2
Problem description
Following the documentation  for the Repeater type doesn't work as expected.

Not all related fields are shown. In the linked repo example, we have a user with 2 roles but only one is shown
Any edits to the roles usually lead to some sort of error, depending on the action (eg trying to remove an existing role might give a different error than trying to add a new one)

I have tried changing the name of the repeater and other suggested solutions from here (discord link) but nothing seemed to work.
Expected behavior
Using the Repeater I should be able to manage BelongsToMany relationships
Steps to reproduce
Using the linked repo

Build the app
Run the migrations and serve
Navigate to the homepage
You should have at least one user with 2 roles but only one will appear
Try editing the roles

Note that creating new relationships seems to work for the 1st time. After that, we get back to the original issue where not all records appear in the form.
Reproduction repository
https://github.com/sprtk-ches/laravel-filament
Relevant log output
No response

@sprtk-ches, hi, friend! I had the same problem. While researching the Filament code, I discovered that it uses a model key to display the elements in the repeater. Try adding something like this to your pivot model:
protected $primaryKey = 'custom_id';
public function getCustomIdAttribute(): string {
    return $this->model1_id. '_' . $this->model2_id;
}

This code will fix the problem with displaying one entry, but it will not be possible to change it. I'm dealing with the change right now.

@MarkKhromov Thanks for the workaround 👍 ! I'll leave the issue open and let the filament team decide how they want to handle this.

Hi, this is explained in the docs. https://filamentphp.com/docs/3.x/forms/fields/repeater#integrating-with-a-belongstomany-eloquent-relationship
You must use the HasMany pivot model as the relationship for the repeater, as each item represents a pivot record and not a related record

Hi, this is explained in the docs. https://filamentphp.com/docs/3.x/forms/fields/repeater#integrating-with-a-belongstomany-eloquent-relationship
You must use the HasMany pivot model as the relationship for the repeater, as each item represents a pivot record and not a related record

@danharrin Yes, that's exactly what we do. But this doesn't work at all as expected. This will only work correctly with one entry. I won't be able to add a new entry if the identifier is not specified. Then it is necessary to describe this in the documentation. My code uses the Pivot table, as in the example. It has two columns work_id, employee_id and nothing else. And it doesn't work for me. I studied the repeater code and noticed that it simply cannot work correctly if there is no identifier for the record, no matter whether it is a pivot or a regular model. I suggest considering the possibility of working with a primary key formed from several columns. For example, if the model is set to $primaryKey = ['column1', 'column2'];
Just if you provide for this in the repeater, then the scenario with pivots will work correctly, and not as it does now. How do you look at this? I can help, try to implement this mechanism, but I need your help.

We need a primary key, otherwise we cant easily keep track of which pivot entries have been created or deleted.

We need a primary key, otherwise we cant easily keep track of which pivot entries have been created or deleted.

That's what I'm talking about too, but for a pivot table, the primary key can be made up of two columns.

You can just use an easy auto increment, it doesnt need to be composite

You can just use an easy auto increment, it doesnt need to be composite

Yes, I understand you. In order not to use auto-increment, I simply made my identifier in the pivot as uuid and everything works fine for me. But still, it wasn't entirely obvious that it should be used that way. It seems to me that the documentation should be supplemented to indicate that the table must have an identifier with a primary key.

You can just use an easy auto increment, it doesnt need to be composite

@danharrin Another perspective from my side:
I typically only create data structures that are absolutely necessary. Hence, most pivot tables do not have an "id" column as the primary key is combined of both referenced tables. This is fine in vanilla Laravel. However, when using Filament, this becomes an issue as belongsToMany is built using hasMany structures. Not providing a single primary key column in the pivot table causes strange bugs in Filament forms that, in my case, took significant time to debug and pinpoint the issue to the "missing" id column. If Filament requires such a column, it would be great if this was stated in the docs.

Great, I agree it should be in the docs. There is an "Edit" button at the bottom of each docs page when you are ready to make the contribution 😄

Please add this in docs

i just use formatStateUsing
TextEntry::make( 'members' ) ->formatStateUsing( function ( $state ) { return $state->name; } ) ->placeholder( 'No Members' ) ->badge(),


Suggestion - add a rule to detect type mismatches

for example:
@arrayProp(Field, 10) scopes: CircuitString[];

should fail the linter on type mismatch

Thanks for bring raising this issue @omrikiei . We will investigate further.


Shotgun Broken

Currently the shotgun does the animation for reloading a single shell and pauses. The count does not increase either and the shotgun remains stuck in this manner.
I am unsure if this is due to the base, or how the weapon is programmed, but this makes shotguns unusable.
Hopefully this can be fixed soon! Love your work.

Strange, must have broken due to a recent sbox update...
Glad you love the base, more updates coming soon!

Problem solved! I will be making a new release candidate shortly.


Automatic message when no location in the WA chat

When there is no location collected in the WhatsApp chat, could we get an automatic message in Kapta saying "There is not location in  the chat". It's better than having an empty map.

@acholyn I've started a branch for this 54/handle-no-locations based off of your reactify branch. So far I've made process-text return no features if there are no locations. I guess one option would be to test for that in parseFile and display a dialog or something if there's nothing to display?


bug: Cannot open the diffview

Did you check docs and existing issues?

[X] I have read all the lazy.nvim docs
[X] I have searched the existing issues of lazy.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.0-dev-289+gfa1baa9a4
Operating system/version
Ubuntu 22.04.2 LTS
Describe the bug
After Lazy sync, I selected one updated plugin commit and pressed d for opening diffview, then I failed.

And this can be reproduced on my MacOS
Steps To Reproduce

Lazy sync
Select one commit
Press d

Expected Behavior
Just open diffview correctly like before.
Repro
No response

Works fine for me. Does diffview work without launching it from lazy?

Works fine for me. Does diffview work without launching it from lazy?

Yes, it does.


Adding Z3 language

What programming language should we add?
Z3
What is the official website for the language?
Z3 is relatively new and was started as an open-source language by Microsoft Research. Therefore, I believe this is similar to an "official website":  https://github.com/Z3Prover/z3
Is this a language that comes in many variants? If so, which variant should we support?
Z3 can be integrated with multiple higher-level languages. We will support Python integration.
Does the language have an official logo?
No
Does the language have an unofficial logo? If so

please explain the ways in which people use the unofficial logo
please include a link to an example
please list the attribution rights of that logo
A logo is utilized on the github page linked above as the "official website" and can be found here: https://www.google.com/search?q=z3+programming&tbm=isch&ved=2ahUKEwiA2cvRl-XuAhUPdqwKHayQAjIQ2-cCegQIABAA&oq=z3+programming&gs_lcp=CgNpbWcQAzIECCMQJzIECCMQJzIECAAQGDIECAAQGFC20QVYttIFYL_UBWgAcAB4AIABW4gBogGSAQEymAEAoAEBqgELZ3dzLXdpei1pbWfAAQE&sclient=img&ei=5eQmYMDSLo_ssQWsoYqQAw&safe=active#imgrc=6mlmkJfK_-vl2M. I could not find attribution rights for this image.

Is there a testing framework available for the language?
There is a testing framework for Z3 when it is integrated with Python. It can use the SMT-LIB2 standard.
Is this language listed as 'supported' by Prism.js?
No
If it is not supported by Prism.js, what is the closest supported language it maps to?
Python
Who will be leading the effort to launch the track?
I am part of a group of 5 that plans to add 10-15 exercises to this track over the next 2-3 months.

@Newby555 Great. Thanks! :) We'll bootstrap a new track for you on Tuesday.
 You can still work on building the track but we might need to script some PRs for you to rearrange directory structures etc during the period, and you will find that the docs are somewhat in transition too. But please ask @ErikSchierboom any questions (opening issues is great, or on Slack too). If you're not on Slack yet, please email<EMAIL_ADDRESS>and he'll get you set up :)

Your repo is ready for you here: https://github.com/exercism/z3/issues/1
✨ Thanks for taking the time to spin up a Z3 track!

I've been looking at some of the language tracks that haven't gotten a lot of attention over the past few years, and this one is a bit of an odd one out.
Despite it being called a language, it's more often referred to as a "theorem prover". As such it really doesn't fit into the Exercism mold.
There was a flurry of activity in June and July 2021, and there are 14 exercises (including Hello World). All the other commits are from bots or Erik doing work to get tracks into shape for v3.
I get a 404 on exercism.org for the track, so it seems like it hasn't made it onto the site yet: https://exercism.org/tracks/z3
I wish I had done better due diligence before we created the repository for this, and I feel bad for coming back 18 months later to say that I don't think it makes a good candidate, but from what I can tell it is not a good fit for the platform since the intended scope that it applies to is so narrowly focused.
I would like to revert the decision to add the track, and archive the track repository.
@iHiD @ErikSchierboom what do you think?

I agree with that fully.


Out of memory exception when letting it run for too long

When letting the program run for too long it might get an out of memory exception due to some stored resource (I assume from closing and reopening the browser everytime.
Might be good to switch to refreshing instead of closing and re-opening for a default cycle. Closing can be done when an exception occured.

Might be a problem with writing the last_card.txt, because it is opened 2 times but closed only once


notify

Just wondering when the blynk notify will be coming or is is not on the road map?

Hello,
unlike the previous platform, notifications are sent by the server here, there is no way to send them directly from the devices.
As you see here, to send a notification you have to use events and automations, you can use the "Log Event" node to send an event and then you have to connect an automation to this event to fire a notification, it's all more complicated than the legacy blynk.
Regards


AudioFrame behaviour is different for V1/V2

The example given behaves differently on V1 and V2
import audio
import math
def repeated_frame(frame, count):
    for i in range(count):
        yield frame
frame = audio.AudioFrame()

while 1:
    for i in range(len(frame)):
        frame[i] = int(math.sin(math.pi*i/16)*124+128.5)
    audio.play(repeated_frame(frame, 100),wait=True)

And the one in https://microbit-micropython.readthedocs.io/en/v2-docs/audio.html# seems to have another behaviour where the square wave is combined with the sawtooth one rather than being distinct from it.
From user:

I tried the previous program which can write 32 points of a beep that is repeated.
I saw that in your document, these 32 points were sent over a period of 4ms.
So with the following program we should hear a sound of 250Hz. One period on 32 points of the frame.
If instead of dividing by 16, we divide by 8, we should with a sound at 500Hz.
Using a smartphone application that does the FFT, I can't see the spectral line at 250Hz or 500Hz in my spectrum? A priori the sound intensity of your speaker is far too low. Obviously we can only generate multiple sounds of 250Hz. Obviously it will be difficult to go beyond 1000Hz a priori, because the sinusoid on 32 points is then too distorted.
I made a very simple little program with the V1 and with an external speaker grove. This program allows me to generate sound at the exact frequency. This program works perfectly

@martinwork can you take a look if you are able to this week.

@microbit-mark Will do!

The code that plays the AudioFrame in V2 is completely different to the V1 code. I don't know enough about the CODAL audio pipeline right now to be able to tell if the V2 problem is in CODAL or micropython.
The data supplied to uBit.audio.mixer by the AudioSource class is the AudioFrame expanded BUFFER_EXPANSION times by linear interpolation, but the sample rate is multiplied by  BUFFER_EXPANSION / 4.
I've experimented in C++ with supplying similar data to MemorySource (which is similar to AudioSource) and that doesn't sound right either.
AudioSource
https://github.com/microbit-foundation/micropython-microbit-v2/blob/a76e1413bcd66f128a31d98756fc3d1f336d1580/src/codal_app/microbithal_audio.cpp#L30
audio_init
https://github.com/microbit-foundation/micropython-microbit-v2/blob/a76e1413bcd66f128a31d98756fc3d1f336d1580/src/codal_port/modaudio.c#L113
linear interpolation
https://github.com/microbit-foundation/micropython-microbit-v2/blob/a76e1413bcd66f128a31d98756fc3d1f336d1580/src/codal_port/modaudio.c#L80

@dpgeorge I think this is related to the reproducing case you are building for Joe?
This would be great to fix because there are a number of fun tutorials around for how to play wav from a micro:bit with MicroPython and they don't, at present, work well on V2.

@dpgeorge I think this is related to the reproducing case you are building for Joe?
This would be great to fix because there are a number of fun tutorials around for how to play wav from a micro:bit with MicroPython and they don't, at present, work well on V2.

I have been trying to reproduce the "buffering" problem in pure C++ but I can not do it.  So at this stage I'd suggest working around any buffering issues by just making it work correctly from the MicroPython side.

is there a CODAL ticket present already, or were we pending the C++ reproducer?

AFAIK there is no CODAL ticket for this, mainly because I cannot pin point it enough to describe it in technical detail.  This was first seen with the initial issues with speech when moving it over to use the mixer, and fixed only by adding some extra buffering to the speech output.  I'm still not sure why it doesn't work without the buffering, but the plan is to do a similar buffering for audio.play.

Okay, thanks for the clarification Damien.
We are expecting a CODAL tag at the end of today and probably do a MicroPython release tomorrow. Will this work fit for the beta.5 or do we need to move it to beta.6?

Will this work fit for the beta.5 or do we need to move it to beta.6?

I will try!

Audio output should now be improved, see commit<PHONE_NUMBER>0cd821272d985200c35e1435eb2cd9
I'm not sure if it's exactly the same as v1, it will need to be tested on a variety of examples.

CODAL v0.2.25 was updated in 76054c2e9445d8eea285132c2374a18ddc690514 and that improved the sampling rate accuracy of audio.play.

Great, thanks Damien!
@microbit-mark would you be able to test this issue again with the hex from https://github.com/microbit-foundation/micropython-microbit-v2/actions/runs/655946723 ?

The example in https://microbit-micropython.readthedocs.io/en/v2-docs/audio.html# still sounds different on V1 and V2. On V2 There is an audible click between samples that is less evident on V1.
I will ask the person on the support ticket to also test.

@microbit-mark If the PWM output is well behaved and the samples start and end at the midpoint you shouldn't get any clicks/pops at start/end?

This repo was updated to use CODAL v0.2.35 in 76c1b3153970e967401dee6a2566fbfec36d8fe2, which changed the audio sampling time from 50ms to 5ms, to make V2 behave more like V1.

@martinwork as with #67 could you please test with the version of MicroPython in https://python.microbit.org/v/beta and close this if the issue is now resolved?

@jaustin I loaded the original python sample into beta and WebUSB downloaded it.
V1 makes a tone with lots of crackles, and clicks between.
V2 makes a very quiet tone with a machine noise and stops after a few seconds with an error: usually "line 5 ValueError: generator already executing"; occasionally "line 11 ValueError: generator already executing"
Is it already known that switching to the beta editor tab resets micro:bit?

I can confirm that with the latest MicroPython (2b57ddd5344ebeeee736b513459bf23af5df1675, which has been updated to CODAL v0.2.40), this script still throws the ValueError: generator already executing errors.

The ValueError: generator already executing problem should be fixed by 3b5652898eb3d1e9850e9e99db854a0e49043df3.

As difference in V1 and V2 was shipped in v2.0.0, and we have tight deadline for v2.1.0, we'll push this one to v2.2.0.

Some of the differences in the original code were due to the CPU differences between V1 and V2, as V2 can process the sine wave quicker and constantly starting and stopping audio.play in an infinite loop.
This example shows the output sound difference between V1 and V2 when outputting a sine wave on pin0, which needs headphones.

Okay, the test above on a V2 was with MicroPython v2.1.2 from the Python Editor.
Using the latest build from the recording and playback branch and the sine wave on the V2 sounds much much better.
Maybe sounded a bit more triangular than the V1, as it was louder as well, but close enough.
I think this is as close as we are gonna get, so we can close this as completed.


BANO tiles never appear

For example : https://www.cartes.xyz/t/0b0e90-Bano

I see, the remote certificate is considered bad by the browser. I'll use the http one.

It's released and available online.


constantly warning about unverified HTTPS request

Tried with verify_ssl: false but still get this warning on every scrape
 /usr/lib/python3/dist-packages/urllib3/connectionpool.py:1015: InsecureRequestWarning: Unverified HTTPS request is being made to host 'localhost'. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings

This warning is originating from python urllib3. Not something which can be fixed in prometheus-pve-exporter.

This would fix it though:
import urllib3
urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning)

@znerol I can't find where verify_ssl setting would have an effect:
https://github.com/prometheus-pve/prometheus-pve-exporter/search?q=verify_ssl

verify_ssl is passed directly to proxmoxer/proxmoxer.

@nvollmar: Did you see proxmoxer/proxmoxer#128 ? Does upgrading proxmoxer to 2.0.1 fix the problem you are having?

Thanks for the hint, I'll look into that.

Tested proxmoxer 2.0.1, still the same issue. Downgraded to 1.3.1 (latest version before 2.0.0) which solved it.

This is a super crude way to fix this, but until proxmoxer updates their utilization of the requests library, it seems like this is the only way to do it without editing the library code:
I changed the service to set the environment variable: PYTHONWARNINGS=ignore
The service declaration of my systemd service now looks like:
$ systemctl cat prometheus-pve-exporter.service
# /etc/systemd/system/prometheus-pve-exporter.service
[Unit]
Description=Prometheus Proxmox VE Exporter
Documentation=https://github.com/prometheus-pve/prometheus-pve-exporter

[Service]
Environment=PYTHONWARNINGS=ignore
Restart=always
User=pve-exporter
ExecStart=/opt/prometheus-pve-exporter/bin/pve_exporter /etc/prometheus/pve.yml

[Install]
WantedBy=multi-user.target

After that, reload the systemd daemon: systemctl daemon-reload and restart the application: systemctl restart prometheus-pve-exporter.service
Note, that this will also squash other legitimate python warnings, but most people probably aren't looking at the service output anyway.

Just to complement the last answer, you can also do:
PYTHONWARNINGS=ignore:Unverified HTTPS request

https://docs.python.org/3/library/warnings.html


chore(deps): update module go to 1.20

This PR contains the following updates:

Package
Type
Update
Change

go (source)
golang
minor
1.19 -> 1.20

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


get / edit WAF package

This PR adds entrypoints to get and edit WAF rule packages as per https://api.cloudflare.com/#waf-rule-packages-firewall-package-details and https://api.cloudflare.com/#waf-rule-packages-edit-firewall-package
It will contribute to implementation of #321
Types of changes
What sort of change does your code introduce/modify?

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

Checklist:

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

@patryk @jacobbednarz would appreciate you taking a look ... I'm creating a few PRs to update the terraform provider

It's great, thanks for the contribution @sbfaulkner. I'm sorry about the delay, I must've miss the notification.


onSuccess is only triggered once?

Bug report
Description / Observed Behavior
Our application uses onSuccess to trigger certain desired behavior. However, in testing, it appears that onSuccess is only triggered if that specific useSWR call is responsible for the success - if the data was already loaded from another useSWR call, it's not triggered.
Expected Behavior
onSuccess is triggered whenever data is successfully available, whether that successful request came from this instance or another.
Repro Steps / Code Example
https://codesandbox.io/s/swr-on-success-uuigol?file=/src/App.tsx
Additional Context
SWR version: 2.1.1

Thanks for reporting and reproduction !
This is currently a expected behavior since SWR will deduplicate the requests so only one request will be executed and trigger the onSuccess callback once.
You could use useEffect as alternative.
const { data } = useSWR('api', fetcher)
React.useEffect(() => {
    if (data)
      //do something
    }
}, [data])

useEffect

Is it guaranteed that the data object reference will be the same between renders so that hook doesn't get executed until there's a new request (for example when using immutable swr hooks)?

Thanks for reporting and reproduction !
This is currently a expected behavior since SWR will deduplicate the requests so only one request will be executed and trigger the onSuccess callback once. #1580
You could use useEffect as alternative.
const { data } = useSWR('api', fetcher)
React.useEffect(() => {
    if (data)
      //do something
    }
}, [data])

Is it guaranteed that the data object reference will be the same between renders so that hook doesn't get executed until there's a new request (for example when using immutable swr hooks)?


feat #41: Added example of using the docker image to run a migration …

…against a sql server, using docker compose for repeatability.
This replaces #70 due to the volume of changes I was trying to rebase....

@erikbra Are you waiting on anything else from me with this mate?

Hi, sorry, just contention in the pipeline here. Got hung up on implementing Sqlite support. Thanks for your example work!


Invalid requirement citation in error for 005-1-invalid-au?

In running AU 005-1-invalid-au, we receive an error citing requirement <IP_ADDRESS>-1 (d).  In reviewing the lrs, we accepted two statements, an Initialized (cmi5 defined) and an Experienced (cmi5 allowed).  Both were sent by the AU with a statement id.  I'm sure I am missing something, but it is hard to find without knowing what requirement fails.
Initialized:
{"id":"0cacc365-ed8d-4735-8000-80c8e9273529","timestamp":"2022-04-19T16:06:05.352Z","actor":{"objectType":"Agent","name":"Boss, Lisa J.","account":{"homePage":"https://www2.vta.com/Demo/LearnerConnection/","name":"27701-4"}},"verb":{"id":"http://adlnet.gov/expapi/verbs/initialized","display":{"en":"initialized"}},"object":{"id":"https://www2.vta.com/Demo/LearnerConnection/AU/13168"},"context":{"contextActivities":{"grouping":[{"objectType":"Activity","id":"https://w3id.org/xapi/cmi5/catapult/lts/au/005-1-invalid-au"}],"other":[{"id":"http://id.tincanapi.com/activity/software/@rusticisoftware/cmi5/3.0.0","definition":{"name":{"und":"@rusticisoftware/cmi5 (3.0.0)"},"description":{"en":"A JavaScript library implementing the cmi5 specification for AUs during runtime."},"type":"http://id.tincanapi.com/activitytype/source"}}],"category":[{"id":"https://w3id.org/xapi/cmi5/context/categories/cmi5"}]},"extensions":{"https://w3id.org/xapi/cmi5/context/extensions/sessionid":"2120"},"registration":"a98aef38-eda7-4985-ad97-780729746029"}}
Experienced:
{"id":"644558eb-67a8-4f33-8d7f-667cfedde421","timestamp":"2022-04-19T16:06:07.360Z","actor":{"objectType":"Agent","name":"Boss, Lisa J.","account":{"homePage":"https://www2.vta.com/Demo/LearnerConnection/","name":"27701-4"}},"verb":{"id":"http://adlnet.gov/expapi/verbs/experienced"},"object":{"id":"https://www2.vta.com/Demo/LearnerConnection/AU/13168"},"context":{"contextActivities":{"grouping":[{"objectType":"Activity","id":"https://w3id.org/xapi/cmi5/catapult/lts/au/005-1-invalid-au"}],"other":[{"id":"http://id.tincanapi.com/activity/software/@rusticisoftware/cmi5/3.0.0","definition":{"name":{"und":"@rusticisoftware/cmi5 (3.0.0)"},"description":{"en":"A JavaScript library implementing the cmi5 specification for AUs during runtime."},"type":"http://id.tincanapi.com/activitytype/source"}}]},"extensions":{"https://w3id.org/xapi/cmi5/context/extensions/sessionid":"2120"},"registration":"a98aef38-eda7-4985-ad97-780729746029"}}

The statements were not sent without an ID.  The statements I posted in the issue were pulled from the browser where the AU was running, not from the LRS.


turn off drives to vp learning

Codecov Report
Base: 37.74% // Head: 37.74% // No change to project coverage :thumbsup:

Coverage data is based on head (f4f55b4) compared to base (c759bc0).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #98   +/-   ##
=======================================
  Coverage   37.74%   37.74%
=======================================
  Files          23       23
  Lines        5060     5060
=======================================
  Hits         1910     1910
  Misses       3003     3003
  Partials      147      147

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.