Filter tools based on course context, process navigation enabled flag (#166)

This PR aims to resolve #166. It is still a draft. Not sure it is an optimal implementation, but it seems to work.

Thanks for the review, @jonespm. If you could take a look at my changes, and respond to my comments, and give another approval, I'd appreciate it.


Migrate SupplyIncreaseProposal

Description

closes: #1290

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[x] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[x] Review Codecov Report in the comment section below once CI passes

In current code, cli would create this proposal, i.e user would not need to worry about the json and just the parameters, any thoughts on creating this for the new path? I might be missing where that is created?

In current code, cli would create this proposal, i.e user would not need to worry about the json and just the parameters, any thoughts on creating this for the new path? I might be missing where that is created?

In our next SDK bump, we'll get the AddGovPropFlagsToCmd and ReadGovPropFlags helpers, which will greatly help facilitate this. I expect a few follow-ups at some point to add new CLI tx commands that generate and send proposals for different purposes (like this).


make display of entered/selected values consistent between literal fields and lookup fields

when user enters a value in a literal field (or when there's a default value), we place the value below the entry field
when user selects a value from a lookup (or enters their own value), or when there's a default value, we leave the value inside the entry field

we should choose one behavior or the other.

I don't love this, but this option could be a compromise.

Make all the fields behave like a literal field.
For fields that only allow 1 value, do not show the text field so user can not enter in another value.

no one else has mentioned this, so i'm going to close it.


python310Packages.samsungtvws: 2.5.0 -> 2.6.0

https://github.com/xchwarze/samsung-tv-ws-api/compare/v2.5.0...v2.6.0
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 230984 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.samsungtv
python310Packages.samsungtvws
python310Packages.samsungtvws.dist
python311Packages.samsungtvws
python311Packages.samsungtvws.dist

Result of nixpkgs-review pr 230984 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.samsungtv
python310Packages.samsungtvws
python311Packages.samsungtvws


Add Instructions/Suggestion To Documentation To Use Code Owners as Permission Work Around

I noticed the current documentation points out the security issue of push-access users effectively getting admin via this application.
One suggestion that might be worth adding to the documentation is setting admins as the code owner of the yaml file, and then selecting the option to require code-owner review for the repository. It allows others to suggest changes via pull-request, but an admin still has to approve it. Not the most dynamic solution (requires update separate from the permissions), but it's something.
If the team things this is worth doing, I'd be happy to put together a pull request (either to add to the main readme or add a separate markdown file like deploy.md to link to from the main one)

👍 pull requests welcome.

Pull request made (#66 )

Thanks!

My pleasure!


Wrong slash used for wsl untitled file display

Open a wsl window
In the terminal, create an untitled file by running code-insiders newFile.txt
This should open a untitled file called newFile.txt
Hover over the new file's tab

bug
The path uses \ even though we are on wsl

Once the file is saved, the path starts using / as expected

I think using forward slash for paths on all systems is reasonable.
I have been writing python with forward slashes on Windows for 20 years.
They work for all win32 APIs.
They work in powershell and WSL.
The only place the don't work is cmd.exe.


Added Super tetris game

PR Description 📜
Fixes # 5002

Mark the task you have completed ✅

[X] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[X] I have performed a self-review of my own code or work.
[ ] I have commented my code, particularly in hard-to-understand areas.
[X] My changes generates no new warnings.
[X] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[X] I have added screenshot for website preview in assets/images
[X] I have added entries for my game in GameZone's README.md
[X] I have added README.md in my Game folder
[X] I have added working video of the game in README.md (optional)
[X] I have specified the respective issue number for which I have requested the new game.
[X] I have added my Game to gamesData.json file and it is loading properly on main website as intended.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @pavitraag ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Please make following changes Add your game entry in main readme. Md file of the gamezone repo Add your game image in asset folder of the repo follow naming convention while naming the image

Ready to merge

@kunjgit can u please merge it

@kunjgit can u please merge this so that i can raise my next issue

@kunjgit please review my pr and merge it

Thank you  @pavitraag , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Add docker support for self-hosting

With the frustrating news of this bot being sunset, I put together docker support for the bot. This will allow users who wish to self-host a less intrusive means of setting up an instance.
General use:
# Bring the bot up
docker-compose up -d
# Rebuild the bot (ex. apply any local changes)
docker-compose build
# Stop the bot
docker-compose stop
# Take down the containers of the bot
docker-compose down
# View logs
docker-compose logs -f bot

Configuration (ex. bot token) with this method is done via .env file, that should not be committed. I have provided an example .env.example file to use as a reference, but it's essentially just the environment variables defined in debian-11-quick-setup.sh.
If you have any other questions, feel free to ask.
And I just want to add, I completely feel your pain with not getting approved for intents. The verification system is so obtuse. It's ridiculous that this bot didn't get approved.

tysm!! gotta add the dep for yt-dlp since im too busy to write a full tiktok client myself, but that should be easy


[request] Add a newline at the end of files in phraseapp pull

I've configured my text editors to add a newline at the end of the file on save.
However,  files organized byphraseapp pull does not have the ending newline, it always makes diffs; thus I'd like for phraseapp to add newline at EOF.
Thanks.

Hi, how often would you say you have to manually edit and save the translation files in your editor? When translations are mainly updated through PhraseApp it would rarely be necessary to edit the file locally

Sorry for my late response here. If you edit the locale only locally you could only push this file but exclude it from the pull config. This way you would never get the added newlines. Would this be a possible solution?

We sometimes pull message files and sometimes push them. Is the workflow not what you phraseapp expected?

We don't require a specific workflow and this workflow should work fine. The only thing we cannot guarantee is that the formatting of the file is entirely kept the same when you edit it manually. For example sorting and this newlines case might change after a push and pull. What you could do for example would to run phraseapp push --wait. This will wait until the upload has been processed. Then you can run phraseapp pull and commit this result. This way your repository contains the same result everyone will get when pulling locales.

I'm also experiencing this issue, where my translation files are in JSON format. Like the op, we mainly create new keys in the code (in our case in English), which we push up to Phraseapp for our translators to work on, but we also allow changes in the Phraseapp dashboard to the same English keys which we pull down to our code, so it needs to work both ways.
The thing is, in another project we use yml format for our translation files, and in that case, the phraseapp cli ensures there is a new line at the end of the file. But with json files it strips it. I think it should at least be consistent. And in my experience, it's good practice to include new lines at the end of all files.

I'm experiencing the same issue with JSON file format. It creates unnecessary diffs for every single file whenever I phrase pull, even if they haven't been edited.
I think this should be a quick fix, right?

@jhwheeler Thanks for bringing it up again. We are currently tracking this issue for our new CLI in https://github.com/phrase/phrase-cli/issues/20
We'll have a look what the implementation would entail and keep you updated when I have some news

@jhwheeler Thanks for bringing it up again. We are currently tracking this issue for our new CLI in https://github.com/phrase/phrase-cli/issues/20
We'll have a look what the implementation would entail and keep you updated when I have some news

Thanks!


Generating options

Would it be possible to generate file only with localization?
Is it possible to change top level struct name from R to something else?
Ideally It would be cool to use it for example like R.exampleText instead of R.string.localizable.exampleText

Thanks for using R.swift!
Currently this is not possible, but these could be features that we implement in R.swift. I'm curious about your use case, could you explain a bit why renaming the struct and only generating localizations would help you?
Knowing the use case for a certain feature/change really helps to decide if it's worth the time and maintenance.

@mac-cain13 Only thing I currently need is to have strongly typed localization.
So basically chaining with R.string.localizable.exampleText is a little long and would be cool to just call R.exampleText.
If it comes to renaming, R can be a little confusing and doesn't follow naming convention.
Those are just questions/suggestions for your library which is super awesome anyway :)

Szwagier zamknal temat, dziekuja

+1
In my case we're running an app extension that needs access to localizable string but including the generated file doesn't work because it includes a lot of unaccessible classes (i.e: view controllers that are in the storyboards, which is not something needed in the app extension)

With version 5.1 you can choose to only run the string generator by adding a flag: --generators string
So the whole call becomes something like:
"$PODS_ROOT/R.swift/rswift" generate "$SRCROOT/R.generated.swift" --generators string


add import/export functions for backup scenarios

Added first version of import/export function
Export: cli export {filename}
Writes Things, Locations, Datastreams, Sensors, ObservedProperties and their relations as json to a file. The current logged in host is used to export the entities from. The Observations, HistoricalLocations and FeaturesOfInterest are ignored for now. Requesting of relations should be done with expand but since the GOST version on our demo server is old and does not yet support $expand so this is done differently for now
Import: cli import {filename} {host}
Imports an exported file by sensorthings-cli to a given SensorThings server, linked id's can differ on the host so this is taken care of by mapping the old reference id's to the new ones given by the server. HistoricalLocations are currently not exported so they will be created by the server after posting a location, so keep in mind that the time of the HistoricalLocation is incorrect.


Remove broken symlink from node_modules on uninstall

Partial fix for #1684.
We could improve this by also removing the symlink on install if the symlink exists. This PR will fix at least some of the cases, however. See issue for more details.

This may be an artificial test situation but:
If I install the extension, verify it works, delete the installed files via
rm -Rf ~/.k8slens/extensions/mirantis--lens-extension-cc

then go to Lens extension page and uninstall the extension, I observe

a notification pops up saying "Error uninstalling..."
that the no-longer-valid link has been removed, as expected.
extension is removed from the list, as expected

However, if instead of clicking "Uninstall" after doing the rm -Rf ... I click "Disable", I observe

Lens disables the extension,
after a few seconds the extension disappears from the list (automatically uninstalled by watcher?)
but the link has not been removed.

This may be an artificial test situation but:
If I install the extension, verify it works, delete the installed files via
rm -Rf ~/.k8slens/extensions/mirantis--lens-extension-cc

then go to Lens extension page and uninstall the extension, I observe

a notification pops up saying "Error uninstalling..."
that the no-longer-valid link has been removed, as expected.
extension is removed from the list, as expected

However, if instead of clicking "Uninstall" after doing the rm -Rf ... I click "Disable", I observe

Lens disables the extension,
after a few seconds the extension disappears from the list (automatically uninstalled by watcher?)
but the link has not been removed.

I believe that is working as intended. I can go through what is happening in more detail tomorrow. I think we can improve various error sitations with other PRs (I have few in mind).


Position and locationAccuracy arent defined in the geolocator package . what is the solution ?

💬 Questions and Help
For questions or help we recommend checking:

The Flutter tag in Stack Overflow

Can you please supply more information? This is not really helpful.

Which version of the geolocator plugin are you using;
Code snippet of how you use the plugin;
Contents of your pubspec.yaml file;
Output from the flutter doctor -v command;
Log of the exact error message.

Some thing you can try:

run flutter clean followed by flutter pub get;
make sure you imported the geolocator in the file where you are using it: import 'package:geolocator/geolocator.dart';
have a close look at the example app for example implementation.

Hi thanks for your attention
here is my flutter doctor - v
[√] Flutter (Channel stable, 2.0.4, on Microsoft Windows [Version 10.0.18363.1621], locale en-US)
• Flutter version 2.0.4 at C:\Users\Parsian\Documents\flutter
• Framework revision b1395592de (2 months ago), 2021-04-01 14:25:01 -0700
• Engine revision 2dce47073a
• Dart version 2.12.2
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
• Android SDK at C:\Users\Parsian\AppData\Local\Android\sdk
• Platform android-30, build-tools 30.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[√] Android Studio (version 4.1.0)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
[√] Connected device (2 available)
• Chrome (web) • chrome • web-javascript • Google Chrome 91.0.4472.77
• Edge (web)   • edge   • web-javascript • Microsoft Edge 91.0.864.48
• No issues found!
the geolocator version I'm using : geolocator: ^7.1.0
I had imported the file
flutter clean doesn't work ... I have used this package in 3 different projects but it has the same error ... this problem occurred when it got updated ...I had no problem with this version =>geolocator: ^7.0.3
here is the code that doesn't work :
Position position = await Geolocator.getCurrentPosition(desiredAccuracy: LocationAccuracy.high);
i have errors for position class and LocationAccuracy (the only thing it says is that position isn't defined ) which doesn't make sense at all ...and I don't have any error for the actual geolocator object..
at last here is how I have depended on this package
dependencies:
flutter:
sdk: flutter
animated_text_kit: ^4.2.1
geolocator: ^7.1.0
I really appreciate your help in advance ...
could you please just import this package into your project and test that line of code above ? i guess the actual problem is within the package.

Hi @delaram-majestic,
I cannot reproduce the problem you mention. I took the following steps:

Create a new Flutter App using: flutter create geo_issue_749;
Open the new App in my IDE (VSCode in this case);
Add the dependency to the geolocator in the pubspec.yaml (copied from your issue);
Added permissions to the ios/Runner/Info.plist and android/app/src/main/AndroidManifest.xml file;
Updated the main.dart to acquire a position instead of increasing the counter.

When I run the App everything works fine, no compile errors and I can get the location on iOS and Android. So I am pretty sure this is something on your system that is not working accordingly. Something you might want to try is to remove all geolocator versions from the Flutter pub cache (on Windows this is usually located in the %LOCALAPPDATA%\Pub\Cache\hosted\pub.dartlang.org\ folder). After removing you will have to run flutter pub get again.
If you want to try here is my code.
main.dart
import 'package:flutter/material.dart';
import 'package:geolocator/geolocator.dart';

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

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  Position? _position;

  void _acquirePosition() async {
    Position position = await Geolocator.getCurrentPosition(
        desiredAccuracy: LocationAccuracy.high);

    setState(() {
      _position = position;
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_position',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _acquirePosition,
        tooltip: 'Increment',
        child: Icon(Icons.add),
      ),
    );
  }
}

pubspec.yaml
name: geo_issue_749
description: A new Flutter project.
publish_to: 'none'
version: 1.0.0+1

environment:
  sdk: ">=2.12.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter
  geolocator: ^7.1.0
  cupertino_icons: ^1.0.2

dev_dependencies:
  flutter_test:
    sdk: flutter

flutter:
  uses-material-design: true

AndroidManifest.xml
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.example.geo_issue_749">

   <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
   <application
        android:label="geo_issue_749"
        android:icon="@mipmap/ic_launcher">
        <activity
            android:name=".MainActivity"
            android:launchMode="singleTop"
            android:theme="@style/LaunchTheme"
            android:configChanges="orientation|keyboardHidden|keyboard|screenSize|smallestScreenSize|locale|layoutDirection|fontScale|screenLayout|density|uiMode"
            android:hardwareAccelerated="true"
            android:windowSoftInputMode="adjustResize">
            <!-- Specifies an Android theme to apply to this Activity as soon as
                 the Android process has started. This theme is visible to the user
                 while the Flutter UI initializes. After that, this theme continues
                 to determine the Window background behind the Flutter UI. -->
            <meta-data
              android:name="io.flutter.embedding.android.NormalTheme"
              android:resource="@style/NormalTheme"
              />
            <!-- Displays an Android View that continues showing the launch screen
                 Drawable until Flutter paints its first frame, then this splash
                 screen fades out. A splash screen is useful to avoid any visual
                 gap between the end of Android's launch screen and the painting of
                 Flutter's first frame. -->
            <meta-data
              android:name="io.flutter.embedding.android.SplashScreenDrawable"
              android:resource="@drawable/launch_background"
              />
            <intent-filter>
                <action android:name="android.intent.action.MAIN"/>
                <category android:name="android.intent.category.LAUNCHER"/>
            </intent-filter>
        </activity>
        <!-- Don't delete the meta-data below.
             This is used by the Flutter tool to generate GeneratedPluginRegistrant.java -->
        <meta-data
            android:name="flutterEmbedding"
            android:value="2" />
    </application>
</manifest>

Info.plist
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
	<key>CFBundleDevelopmentRegion</key>
	<string>$(DEVELOPMENT_LANGUAGE)</string>
	<key>CFBundleExecutable</key>
	<string>$(EXECUTABLE_NAME)</string>
	<key>CFBundleIdentifier</key>
	<string>$(PRODUCT_BUNDLE_IDENTIFIER)</string>
	<key>CFBundleInfoDictionaryVersion</key>
	<string>6.0</string>
	<key>CFBundleName</key>
	<string>geo_issue_749</string>
	<key>CFBundlePackageType</key>
	<string>APPL</string>
	<key>CFBundleShortVersionString</key>
	<string>$(FLUTTER_BUILD_NAME)</string>
	<key>CFBundleSignature</key>
	<string>????</string>
	<key>CFBundleVersion</key>
	<string>$(FLUTTER_BUILD_NUMBER)</string>
	<key>LSRequiresIPhoneOS</key>
	<true/>
	<key>UILaunchStoryboardName</key>
	<string>LaunchScreen</string>
	<key>UIMainStoryboardFile</key>
	<string>Main</string>
	<key>UISupportedInterfaceOrientations</key>
	<array>
		<string>UIInterfaceOrientationPortrait</string>
		<string>UIInterfaceOrientationLandscapeLeft</string>
		<string>UIInterfaceOrientationLandscapeRight</string>
	</array>
	<key>UISupportedInterfaceOrientations~ipad</key>
	<array>
		<string>UIInterfaceOrientationPortrait</string>
		<string>UIInterfaceOrientationPortraitUpsideDown</string>
		<string>UIInterfaceOrientationLandscapeLeft</string>
		<string>UIInterfaceOrientationLandscapeRight</string>
	</array>
	<key>UIViewControllerBasedStatusBarAppearance</key>
	<false/>
	<key>NSLocationWhenInUseUsageDescription</key>
	<string>This app needs access to location when open.</string>
	<key>NSLocationAlwaysUsageDescription</key>
	<string>This app needs access to location when in the background.</string>
</dict>
</plist>

Output of flutter doctor -v
[✓] Flutter (Channel stable, 2.2.1, on macOS 11.4 20F71 darwin-x64,
    locale en-GB)
    • Flutter version 2.2.1 at /Users/maurits/fvm/versions/stable
    • Framework revision 02c026b03c (2 weeks ago), 2021-05-27 12:24:44
      -0700
    • Engine revision 0fdb562ac8
    • Dart version 2.13.1

[✓] Android toolchain - develop for Android devices (Android SDK version
    30.0.3)
    • Android SDK at /Users/maurits/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/maurits/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_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • 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)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.1.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 56.0.5
    • Dart plugin version 211.7233

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

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome
      91.0.4472.101

• No issues found!

I will try it out and then let you know

Hi friend ...sorry for the delay ...i have tried your code and the steps you  have followed but it didnt work ....then i tried to compare my flutter doctor with yours and saw that our versions are different so i upgraded my flutter sdk and now it looks exactly the same but i still get the same unreasonble  error ..its really annoying ...wish i knew where was the problem ...anyway thanks for your attention if anything came to your mind let me know ...

@delaram-majestic have you tried deleting all versions of the geolocator plugin from the %LOCALAPPDATA%\Pub\Cache\hosted\pub.dartlang.org\ folder?
I suggested this in my previous comment, but from your reply it is not clear if you've done this. Another option you could try, from the flutter doctor -v output you supplied I can see you are using Android Studio as IDE, maybe you can try to Invalidate Caches and restart by selecting (in Android Studio): File -> Invalidate Caches / Restart....
I have now tested the application on multiple machines and also the CI/CD configuration builds the app without problems. Also other developers don't experience any issues. I am confident that the issue is somehow related to your configuration or environment. I am going to close this issue now but if you have any follow-up questions please leave another comment and I will try to answer them.

I didnt find the exact location you are talking about ..but i have deleted this file directory => C:\Users\My name \AppData\Local.dartServer.pub-package-details-cache  ..... i have deleted the file packages.json but I didint see any improvement
Im using android studio as the IDE and i have tried that invalidate cach thing but still didnt see any improvement..i will try my best to find the solution and if so , I will share it with others.
thanks for all your effort.


Add support for QNX/Neutrino 7.1

Test cases (ctest2, all succeed):
QNX/Neutrino 7.1 x86_64:  9884
QNX/Neutrino 7.1 aarch64: 9766
Co-authored-by: Tristan Roach<EMAIL_ADDRESS>Co-authored-by: Florian Bartels<EMAIL_ADDRESS>

Thanks! @bors r+

:pushpin: Commit 720151f84493d3a4ca7081ee7b85f3c115a33860 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 720151f84493d3a4ca7081ee7b85f3c115a33860 with merge 318dccc93a46344afabff64001d929fc37432f41...

:eyes: Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.


bug: precommit ends with error TS2786: 'CopyToClipboard' cannot be used as a JSX component.

Is there an existing issue for this?

[X] I have looked through the existing issues

Which method was used to setup Scaffold-ETH 2 ?
git clone
Current Behavior
commit does not work as per eslint check
// .lintstagedrc.js
const buildNextEslintCommand = (filenames) =>
  `yarn next:lint --fix --file ${filenames
    .map((f) => path.relative(path.join("packages", "nextjs"), f))
    .join(" --file ")}`;

the following errors are triggered when trying to commit:
✖ yarn next:check-types:
app/blockexplorer/_components/TransactionHash.tsx(22,10): error TS2786: 'CopyToClipboard' cannot be used as a JSX component.
  Its type 'typeof CopyToClipboard' is not a valid JSX element type.
app/debug/_components/contract/TxReceipt.tsx(21,12): error TS2786: 'CopyToClipboard' cannot be used as a JSX component.
  Its type 'typeof CopyToClipboard' is not a valid JSX element type.
components/scaffold-eth/Address.tsx(119,10): error TS2786: 'CopyToClipboard' cannot be used as a JSX component.
  Its type 'typeof CopyToClipboard' is not a valid JSX element type.
    Type 'typeof CopyToClipboard' is not assignable to type 'new (props: any, deprecatedLegacyContext?: any) => Component<any, any, any>'.
      Construct signature return types 'CopyToClipboard' and 'Component<any, any, any>' are incompatible.
        The types returned by 'render()' are incompatible between these types.
          Type 'import("/home/johnson/used/learning/blockchain/ethGlobal/2024/04/Grouponweb3/eCoupon-ScalingEth24 (copy)/packages/nextjs/node_modules/@types/react-copy-to-clipboard/node_modules/@types/react/index").ReactNode' is not assignable to type 'React.ReactNode'.
            Type 'ReactElement<any, string | JSXElementConstructor<any>>' is not assignable to type 'ReactNode'.
              Property 'children' is missing in type 'ReactElement<any, string | JSXElementConstructor<any>>' but required in type 'ReactPortal'.
components/scaffold-eth/RainbowKitCustomConnectButton/AddressInfoDropdown.tsx(72,16): error TS2786: 'CopyToClipboard' cannot be used as a JSX component.
  Its type 'typeof CopyToClipboard' is not a valid JSX element type.
    Type 'typeof CopyToClipboard' is not assignable to type 'new (props: any, deprecatedLegacyContext?: any) => Component<any, any, any>'.
      Type 'CopyToClipboard' is not assignable to type 'Component<any, any, any>'.
husky - pre-commit hook exited with code 1 (error)

Expected Behavior
no eslint errors are signal and files are committed in repository
Steps To Reproduce
clone the repo scaffold-eth-2
git clone https://github.com/scaffold-eth/scaffold-eth-2.git

make your changes and commit
yarn precommit

Anything else?
No response

Hey @Maltstar thanks for raising it !
Just tried with fresh clone, udpating app/page.tsx and commiting everything seems to work for me
Would really appreciate if you could share a github repo with minimial changes  and this error
You could make changes and run git commit -m "test commit" --no-verify  the --no-verify will allow you bypass error and push to github repo

@Maltstar try to reinstall dependencies from the main branch.
main branch deps was updated yesterdays so if you're installed your packages before and merged main to your branch without installing new deps, you'll get type errors

Both solutions are working the short-term  https://github.com/scaffold-eth/scaffold-eth-2/issues/816#issuecomment-2059414530 and the long-term https://github.com/scaffold-eth/scaffold-eth-2/issues/816#issuecomment-2059446088


Another collection of various suggestions

Dear all!
I quite liked the omniDB, liked it to the point of being the maintainer for the installation code of it on Linux Slackware, and I am trying to use this as replacement of PGAdmin3.
But, yeah, nothing is perfect and i have some suggestion to, maybe, make it a bit better.

Saving the query used on the query tab to a file. This is quite useful. At the moment I have to copy-paste the query back and forth between my text editor and omniDB to do this.

Auto complete is not working on Slackware (but maybe it's the system fault, not the application, but it would be superb if it worked).

Navigation on the table tree by typing the name of the table.
e.g To get to a table named tests I could just type "t" and it would scroll down to tables with names starting with "t".

"Year out of range" exception. Sometimes I get this when there's a date like this on the table "0001-01-01 BC". On PGAdmin3 I get exactly this, but on omniDB I simply get "year out of range" and it doesn't show any of the results.
The only way I can get to see and find which row is the one causing the error is using other application (PGAdmin3).

Well, that's it from me.
And thanks for the attention!

Hi @gabrielgisoldo ,
Thank you very much for your feedback. It is exciting that you are willing to maintain OmniDB on Slackware. I'm not familiar with Slackware, but please let me know how can I help you on this task. For example, I can get some time to try to run OmniDB on Slackware and help you scripting and testing.
Thanks for your suggestions, they are invaluable. I'll try to reply about each of them.

Saving the query used on the query tab to a file. This is quite useful. At the moment I have to copy-paste the query back and forth between my text editor and omniDB to do this.

I included this in our roadmap, expanding for an ability to also open a .sql file inside a Query Tab.

Auto complete is not working on Slackware (but maybe it's the system fault, not the application, but it would be superb if it worked).

Can you please provide more details about this? Does it show some error message? Can you get it to work on a different environment?
I will also try to run it on Slackware, but I can't promise you I'll do this before our next release (next 14th of June).

Navigation on the table tree by typing the name of the table.
e.g To get to a table named tests I could just type "t" and it would scroll down to tables with names starting with "t".

That's a great idea. We have a Filter feature for tree view elements in our roadmap, planned to be implemented soon.

"Year out of range" exception. Sometimes I get this when there's a date like this on the table "0001-01-01 BC". On PGAdmin3 I get exactly this, but on omniDB I simply get "year out of range" and it doesn't show any of the results.
The only way I can get to see and find which row is the one causing the error is using other application (PGAdmin3).

I have already seen this, but what I do to see the value of such fields is to explicitly cast it to text with ::text, for example. I'll try to dig into it and I'll let you know.
Regards,
William Ivanski

Hi @wind39 ,

Can you please provide more details about this? Does it show some error message? Can you get it to work on a different environment?
I will also try to run it on Slackware, but I can't promise you I'll do this before our next release (next 14th of June).

Reading other issues open a found why it wasn't working. The problem was in the documentation on the site.
There, it's written you just need to type the alias and the dot character to open the auto-complete, but on the other issue your answer solved my problem too.

I have already seen this, but what I do to see the value of such fields is to explicitly cast it to text with ::text, for example. I'll try to dig into it and I'll let you know.

Thanks for this tip, I'll be using this from now on too.
Best Regards,
Gabriel

Hi @gabrielgisoldo ,
It is been a while, but I'm here just to inform you that I fixed the issue with the dates:

It will be released in OmniDB 2.15, going out in April this year.
As per your list of requests, the only remaining item is to save a query to a .sql file, which is being handled in issue #480 .
Best regards,
William

Hi @gabrielgisoldo ,
We just released OmniDB 2.15, containing most of your requests in this ticket. The ability to save a query to a .sql file is being handled in #480 .
Can you please download 2.15 and try it? Your valuable feedback will be much appreciated, as always.
Regards,
William

Hi @wind39 ,
Just installed here and everything is working properly.
Thanks for all the help.
Best ragards,
Gabriel

Hi @gabrielgisoldo ,
Thanks for your feedback, closing this now.
Best regards,
William


fix(check-merge-safety): fail job on pull request runs

:pencil: Description

When the check-merge-safety helper is run in a pull request workflow, the job should fail if the PR is not sufficiently up to date with the default branch. Currently only the commit status is set to failure, but the job is able to succeed.

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


Documentation

Some documents are still missing. I wanted to describe the high-level methodology (empty file is created as a placeholder) and some principles behind the classes structure (there is a pattern I have maintained across 4 classes) and how they work. I would like to write it down as well.

Good job overall. The stats and graphs looks nice.
One thing maybe I'm missing: how do we plan to show these results to the client? Let's say I release Dasharo for some platform today and want to present such report. Where do I host/present such report?

I was thinking to add a separate section per board in doc.sdasharo.com in the supported hardware


Public UI: Fix background image on the blog

Summary:
Background is not repeated or didn't have a gradient to the website's background color.
Steps to Reproduce:

Go to supergiant.io
Hit "Blog" on the top menu
Scroll the page down 5 times (4-5 screens )

Expected Results:
Background smoothly turns into the background color of the site.

More Info
Reported by: Taras Hetmanchuk<EMAIL_ADDRESS>Source URL: https://supergiant.io/blog/
BrowserSafari 12.0Screen Size2560 x 1440OSOS X 10.14Viewport Size1742 x 1025Zoom Level100%Pixel Ratio@2x

@hetmanchuk Is this resolved?

https://qboxio.slack.com/archives/CBJDVM0MQ/p1542537050177600

Reported as fixed via slack.


Could not resolve all artifacts for configuration ':react-native-vector-icons:classpath'.

First: download dependecy with yarn : yarn add react-native-vector-icons
Second: linked project with depedency : react-native link react-native-vector-icons
Third: run project with : react-native run-android
and..

What went wrong:
A problem occurred configuring project ':react-native-vector-icons'.
Could not resolve all artifacts for configuration ':react-native-vector-icons:classpath'.
Could not resolve com.android.tools.build:gradle:3.1.4.
Required by:
project :react-native-vector-icons
Could not resolve com.android.tools.build:gradle:3.1.4.
Could not initialize class java.net.IDN
Could not resolve com.android.tools.build:gradle:3.1.4.
Could not initialize class java.net.IDN

i resolve issue with : apply from: "../../node_modules/react-native-vector-icons/fonts.gradle"


Selected text misplaced problem when the system zoom rate is 125%

Attach (recommended) or Link to PDF file here:
BAVP Blockchain-Based Access Verification Protoco.pdf
Configuration:

Web browser and its version: Chrome 95.0.4638.69  and  Edge 95.0.1020.53
Operating system and its version: Win10 10.0.18363.1256
PDF.js version: 2.10.377
Is a browser extension: It does not matter. I have tried to use it in the browser extension and on the website https://mozilla.github.io/pdf.js/web/viewer.html, this problem will appear

Steps to reproduce the problem:

Set the system zoom rate to 125% in Win10 display settings（And there is some other display options in my computer, for reference）

Select a few lines of text arbitrarily

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)
I have two monitors. The selected text is right displayed in 100%-zoom monitor, and misplaced displayed in 125%-zoom monitor.

Duplicate of #12523 (since it looks similar enough, and it's also likely to be a OS/browser problem rather than a PDF.js bug).


📋 [TASK] - Group Constrainer Results by Category

Parent Feature
N/A
Task Description
As an AaC developer, I want to expand the constraint report information, so that the results are grouped by category (error, warning, info) with their counts.
Acceptance Criteria

[ ] Constraint results are properly categorized between error, warning, and info
[ ] Constraint results are presented in groups based on category
[ ] Counts of categorized results are included in report

Unresolved questions
No response

Need to check, but seems that the result categories have been dropped on the return messages in the 0.4.0 work. It's just success or fail messages, nothing about error, warn, or info category.
Dataref constraint failed for value 'alarm': 'alarm' not found in 'actor.name' values '['user', 'alarm_clock', 'timer', 'clock', 'alarms']'
  Source: /workspace/AaC/python/features/alarm_clock/usecase.yaml (68:2:1245:6)


Rework Resource#action to match the 12.3 API.

This means it always coerces to an Array. Also ensures that
Resource#action= goes through the same validation.
Fixes #3604.

ugh, something is wrong with appveyor. I hit that issue as well

:+1:

Agreed

i think ffi 1.9.9 just broke us

https://github.com/chef/chef/pull/3606 until we can figure out why

The test failures in Travis look relevant. Some could be tests I changed that need changing back? But the first one looks alarming.

Should be better now, also fixed/testing the default case for [:nothing].

Though hmm, I think the default in 12.3 would actually be a plain Symbol (:nothing) not an array. How do people feel about that?

Lulz, this is what I get for only running a subset of tests before commit O:-)

:+1: I think this looks right!

Some of the integration test failures look like legit errors but hard to make heads or tails of this test code. @jkeiser?

I am nowhere near a computer at the moment, but all those failures look like they stem from the first one ("the default action is the first declared action"). By declared action including pretty sure it's talking about the new action syntax. So the first action :x do ... end will become the default action unless you say default_action explicitly.

@coderanger
diff --git i/lib/chef/resource.rb w/lib/chef/resource.rb
index 0679406..0240554 100644
--- i/lib/chef/resource.rb
+++ w/lib/chef/resource.rb
@@ -1157,7 +1157,7 @@ class Chef
       action = action.to_sym
       new_action_provider_class.action(action, &recipe_block)
       self.allowed_actions += [ action ]
-      default_action action if default_action == :nothing
+      default_action action if default_action == [:nothing]
     end

     #

diff --git i/spec/integration/recipes/resource_action_spec.rb w/spec/integration/recipes/resource_action_spec.rb
index 4786294..cee7913 100644
--- i/spec/integration/recipes/resource_action_spec.rb
+++ w/spec/integration/recipes/resource_action_spec.rb
@@ -337,7 +337,7 @@ describe "Resource.action" do
           NoActionJackson.action_was = action
         end
       }
-      expect(NoActionJackson.action_was).to eq :nothing
+      expect(NoActionJackson.action_was).to eq [:nothing]
     end
   end
 end

https://github.com/chef/chef/pull/3610

@jaym Mentioned on IRC but for redundancy, we need to make sure to catch the case of:
class MyApp < Chef::Resource
  default_action :one
end

MyApp.new('test', nil).action == [:one]
``

Still a number of failures, FYI (you probably already noticed).  Some calling to_sym on nil values (need a .compact or Array() in there?), some expecting :create and getting [:create].

Yeah, sorting out some weirdness in the why run code.

Should be good to go now.

@coderanger @jkeiser
@tyler-ball has pointed out that this changes the behavior of resource.action to return an array. I'm not sure this is the safest thing to do.

@jaym That is the whole point of this change. In 12.3 and earlier, resource.action always returned an array. This was changed in 12.4 which represents an API break.

I should say, in 12.3 and earlier, setting an action and then looking at the value afterwards always returned an array. With this it should be much closer to "always". To get a non-array now you would either have to touch the ivar manually, or override something in a subclass.

12.3.0
jmundrawala@seamundrawala01 ~/workspace/chef (●)$ bundle exec chef-apply repro.rb                                                                                [2.1.2]
gem install awesome_print  # <-- highly recommended

From: /Users/jmundrawala/workspace/chef/repro.rb @ line 2 self.run_chef_recipe:

    1: require 'pry'
 => 2: binding.pry

2.1.2 (#<Chef::Recipe:0x007f87a5dc5418>):0 > Chef::Resource::File.new("a", nil).action
=> "create"

12.4.0
jmundrawala@seamundrawala01 ~/workspace/chef (●)$ bundle exec chef-apply repro.rb                                                                                [2.1.2]
gem install awesome_print  # <-- highly recommended

From: /Users/jmundrawala/workspace/chef/repro.rb @ line 2 self.run_chef_recipe:

    1: require 'pry'
 => 2: binding.pry

2.1.2 (#<Chef::Recipe:0x007f89b9a1e5d8>):0 > Chef::Resource::File.new("a", nil).action
=> :create

This branch
jmundrawala@seamundrawala01 ~/workspace/chef (coderanger-action●)$ bundle exec chef-apply repro.rb                                                                                [2.1.2]
gem install awesome_print  # <-- highly recommended

From: /Users/jmundrawala/workspace/chef/repro.rb @ line 2 self.run_chef_recipe:

    1: require 'pry'
 => 2: binding.pry

2.1.2 (#<Chef::Recipe:0x007f939f16fa50>):0 > Chef::Resource::File.new("a", nil).action
=> [:create]

Yeah, that's dealing with the default action case. The 12.3 behavior there is being classified as a bug by vote of @thommay as LT. The more important case this fixes is:
r = Chef::Resource::File.new('test', nil)
r.action(:touch)
r.action == ???

In 12.3 that would return [:touch] while in 12.4 it is :touch. This puts it back to the 12.3 behavior.

Acceptable answer. I will let @thommay have the final call. Sucks we are doing this in a patch release, but it really does not matter between 12.4.1 and 12.5, so :+1:

Merged. @jaym please ensure this hits 12.4.1

:+1: Yay! :tada:


Allow cimgui.h to be included in a plain C project

Add struct keywords where required.  The messy part is the ImFont::Glyph inner
struct, which needs to match the implementation in fontAtlas.cpp when building
the library, but the namespace syntax is invalid C, so we have to use a macro to
satisfy both use cases.

I've added a check on Travis to make sure cimgui.h can always compile with a plain C compiler.

Thank you very much! Your help is very appreciated


Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.

Hello,
Thank's for your amazing work.
I install Point Transformer V3 on A6000 GPU and when I run CUDA_LAUNCH_BLOCKING=1 CUDA_VISIBLE_DEVICES=1 sh scripts/train.sh -g 1 -d s3dis -c semseg-pt-v3m1-1-rpe -n semseg-pt-v3m1-1-rpe I got this error
RuntimeError: Trainer: CUDA error: the provided PTX was compiled with an unsupported toolchain. Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
I install point_opts and pointgroup_ops with TORCH_CUDA_ARCH_LIST="7.0 7.5 8.0 8.6+PTX"
Any solutions ?
Thank's in advance.

Hi, no need to set TORCH_CUDA_ARCH_LIST if you build the lib and run the lib on the same GPU arch.

I got the same., with installing without TORCH_CUDA_ARCH_LIST and simply run python setup.py install

Yes, I used it to make docker environments.


Fix typo in Why Qwik docs page

What is it?

[x] Docs / tests

Description
Teeny tiny typo fix:

Frustrat -> Frustrate (users)

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation

Thank you, we fixed a lot of spelling issues in #392.


Handling Duplicate Time Sequences in GDELT Dataset

Hi,
Thank you for sharing your interesting research. While examining GDELT data you uploaded, I noticed numerous duplicate entries in time sequences. Could you clarify how these duplicates are processed within your TPP models? Understanding your approach to handling these repetitions would be incredibly helpful for my research.
Thank you.

Hi,
if i remember correctly, duplicate entries are treated as normal input into the model. for details of the computation, you can have a look at the KnowEvolve paper, we simply follow it to compute the intensities.

Dooray! Failure Notice
    Failure Notice

    Your message sent to

@.***)
has failed to be delivered.
      Please refer to the below for details.

        * Recipient :

@.***)
        * Sent time :

2024-03-20T23:13:45
        * Subject :

Re: [iLampard/lamp] Handling Duplicate Time Sequences in GDELT Dataset (Issue #5)
            * Remote host said :

Your mail was denied from the receiver.
      This message was sent from a notification-only address that cannot accept incoming email.
      For more information, please contact<EMAIL_ADDRESS>

    © Dooray!.


Adds the prefix from the Tailwind config to the dark class

Noticed when I tried to use this with my Tailwind config the dark class wasn't receiving our locally set prefix meaning it wouldn't work. Tried to update the component to include the prefix automatically if the person has already set one in their config.

@renatomoor hopefully this change makes sense!


Removing listitems causes exception

When removing listitems in state list it causes exception:
Uncaught NotFoundError: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.
class ChangeChildrenCount extends Component {
    constructor(props) {
        super(props);

        this.state = {
            list: ['1', '2', '3', '4']
        };

        // Bindings
        this.handleClick = this.handleClick.bind(this);
    }

    handleClick() {
        this.setState({
            list: ['1']
        });
    }

    render() {
        return (

                <button onClick={this.handleClick}>1</button>
                {this.state.list.map(function (x, i) {
                    return
{i}

                })}

        );
    }
}
InfernoDOM.render(<ChangeChildrenCount />, document.body);

Clicking button (changing state to smaller list) causes the exception.
Expected result there should be four elements (0,1,2,3) at the beginning and when clicking button there should be one element (0).

Also opposite is not working. Initial state one item  and then change to four. In initial render there is one, but after changing state nothing happens. no exception either.

This issue is fixed but there is another related. Opening new ticket.


gh-2549: Add csv operation and generator

Related Issue

Resolve #2549

Codecov Report

:exclamation: No coverage uploaded for pull request base (v2-alpha@dea51ea). Click here to learn what that means.
The diff coverage is n/a.

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

@@             Coverage Diff             @@
##             v2-alpha    #2550   +/-   ##
===========================================
  Coverage            ?   64.88%
  Complexity          ?     2432
===========================================
  Files               ?      868
  Lines               ?    27356
  Branches            ?     3171
===========================================
  Hits                ?    17751
  Misses              ?     8228
  Partials            ?     1377

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 dea51ea...53dc7ed. Read the comment docs.

Closed due to #2549


Fixes log4j tag in README.md

README tells to add the pod annotation logs.agent.grafana.com/log-format as log4j to interpret json log4j format, but the river file actually checks for log4j-json string.
https://github.com/grafana/agent-modules/blob/main/modules/kubernetes/README.md?plain=1#L11C56-L11C56
This commit updates the README with the right string

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


Question about MSL dataset

Hello, thanks for sharing your excellent work!
As your article mentioned,  we consider only the three non-trivial ones (A4, C2 and T1) pointed out by [37]. But I found that the labels of MSL are M-6,M-1,M-2,S-2,P-10,T-4,T-5,F-7,M-3,M-4,M-5,P-15,C-1,C-2,T-12,T-13,F-4,F-5,D-14,T-9,P-14,T-8,P-11,D-15,D-16,M-7,F-8. and could not find the A4, C2 and T1.
Could you please tell me which one is used in your experiment?

Thanks. Please refer to src/params.json.


Embed Jitsi as a PlatformView

The most fascinating feature (and also a killer feature comparing to other jitsi flutter repos) could be embedding PlatformView. The reference implementation could be taken from: https://github.com/saibotma/jitsi_meet_wrapper/pull/92 .
Could you please let me know regarding your plans for the implementation (e.g. porting) of this feature (so that I can decide if I can switch to the jitsi-meet-flutter-sdk).

The version for android is here: https://github.com/BindrUK/jitsi_meet/
branch: platfrom-view

We are considering adding Platform View but at the moment is not a priority.

Someone created a PR a while a go implementing platform views for jitsi_meet_wrapper: https://github.com/saibotma/jitsi_meet_wrapper/pull/92
Could be used as guidance, as jitsi-meet-flutter-sdk shares much of its code with jitsi_meet_wrapper.


Bug: Can only increase the priority of energy cards by 100s or 1000s

LaserIO version
1.3.0
Minecraft Version
1.18.2
Forge Version
40.1.52
Modpack & Version
Direwolf20 - 1.18.2 ver 1.6.1
Do you have optifine installed?
Yes
Describe the issue
Can only increase the priority of energy cards by 100s or 1000s
Steps to reproduce
Place energy card in node.
Try to increase priority holding nothing, which should +1
Only can +100.
Same happens on decrease.
Expected behaviour
When editing the priority, it should +/- 1 when you are not holding shift or ctrl
Screenshots
No response
Log files
No response
Additional information
No response

Resolved in v1.3.1 thank you! :)


ngAnimate regression in 1.4.5

I have a JavaScript animate triggered by ng-show that works fine in 1.4.4 and below.  However, in 1.4.5 and 1.4.6 I can observe a strange behavior: maybe about half the time, when the ng-show condition changes via an ng-click handler, I see the ng-animate and ng-hide-animate classes being added onto the element, as well as the data-ng-animate="2" attribute, but the JavaScript animation is not called until something happens to trigger another digest.
Another oddity is that when I'm listening for a raw mousedown event (not through a directive), and invoke the condition-changing code through an $applyAsync, the animation always works correctly.
Any idea what might be going on?  I know that's not much information to go on, but I figured I'd try that first before attempting to make a clean repro since I expect that'll be pretty hard.  Thanks.

First thing first: Have you tried with 1.4.6?

From "However, in 1.4.5 and 1.4.6 [...]" I guess he did :)

Any chance you could put together a plunkr for me?

@matsko, I can try tomorrow, but since this looks like a race condition it might be tricky to get it to repro cleanly.  I was hoping that the details I gave and knowing which version it broke in might give you an idea...  If you have things you'd like me to try, I'm happy to poke around the Angular source to gather more data.
Another option could be to set up a broken instance of my app, with the Angular libs unminified, so you could reproduce it in situ.  Would that be helpful at all?

My apologies, the problem was most likely my fault.  My animation code uses $rootScope.$$postDigest to schedule the actual animation, and apparently in Angular 1.4.5 the animation isn't necessarily called from within a digest any more.  So unless the Angular docs happen to specify that JS animation calls happen within a digest it's a case of me depending on an implementation artifact.  I've worked around the issue.


Spec: MediaGallery

What would you like to be added:
The MediaGallery class is a static class designed for interaction with the device's media gallery on iOS and Android platforms. It provides functionality to check user permissions for accessing the gallery and saving media files to the gallery.
Why is this needed:
User requirement to efficiently save files into storage.
Public API
Uno.Toolkit.UI.MediaFileType enum
Represents a media file type.
Members

Image - Image media file type.
Video - Video media file type

Uno.Toolkit.UI.MediaGallery class
Allows interaction with the device's media gallery.
This class is available only on Android and iOS targets, where media gallery requires a specific API.
CheckAccessAsync method
/// <summary>
/// Checks the user's permission to access the device's gallery.
/// Will trigger the permission request if not already granted.
/// </summary>
/// <returns>A Task that represents a boolean indicating whether the user has access.</returns>
public static async Task<bool> CheckAccessAsync()

Description: Asynchronously checks if the application has permission to access the media gallery. If permission is not already granted, it will prompt the user for permission.
Returns: A Task<bool> that completes with true if access is granted, and false otherwise.

SaveAsync (stream)
/// <summary>
/// Saves a media file to the device's gallery.
/// </summary>
/// <param name="type">The type of the media file.</param>
/// <param name="stream">A stream representing the media file.</param>
/// <param name="targetFileName">The name to save the file as.</param>
/// <returns>A Task that represents the progress of the save operation.</returns>
public static async Task SaveAsync(MediaFileType type, Stream stream, string targetFileName)

Description: Asynchronously saves a media file to the device's gallery using a stream.
Parameters:

MediaFileType type: The type of the media file (image or video).
Stream stream: The media file data in a stream.
string targetFileName: The desired file name for the saved media.

Returns: A Task that completes when the save operation is finished.

SaveAsync (byte array)
/// <summary>
/// Saves a media file to the device's gallery.
/// </summary>
/// <param name="type">The type of the media file.</param>
/// <param name="data">A byte array representing the media file.</param>
/// <param name="targetFileName">The name to save the file as.</param>
/// <returns>A Task that represents the progress of the save operation.</returns>
public static async Task SaveAsync(MediaFileType type, byte[] data, string targetFileName)

Description: Asynchronously saves a media file to the device's gallery provided a byte array.
Parameters:

MediaFileType type: The type of the media file (image or video).
byte[] data: The media file data in a byte array.
string targetFileName: The desired file name for the saved media.

Returns: A Task that completes when the save operation is finished.

Usage
// Check for gallery access
bool hasAccess = await MediaGallery.CheckAccessAsync();

if (hasAccess)
{
	// Save a video to the gallery using stream
	using Stream videoStream = ...; // Video stream
	await MediaGallery.SaveAsync(MediaFileType.Video, videoStream, "MyVideo.mp4");
}

So there's no WinUI/WinRT-equivalent API for this? even at lower level (Storage/...)?
If there's really nothing, shouldnt this be in a Toolkit (non-UI) package vs a Toolkit.UI has it's more of a WinRT api and not a XAML/View component?

I'll let @MartinZikmund expand on it but I believe it's in .UI because of the need for launching the Android/iOS native UI. The non-UI Toolkit lib is just a netstandard2.0 library

So there's no WinUI/WinRT-equivalent API for this? even at lower level (Storage/...)?

It would be StorageFile, but its API is too large for what we need to support. We could end up adding support for it in the future, if that becomes necessary.

Dont we already have StorageFile support? Can we just add what is needed?
Maybe easier to discuss.

@francoistanguay unfortunately not easily possible - for example on iOS one can save the file, but not retrieve it back (unless another permission is added). The API is just so specific, that mapping it onto StorageFile is impossible. If we could get it to work, it would be very specific and most operations would just result in NotSupportedException. In this case having separate API feels more user friendly


Missing definitions

How are we supposed to use these if there isn't a definition of what each constant means?

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

ID: 92ed0679-e142-2ae6-d5ac-2f0d60e17970
Version Independent ID: eca3ef39-2a6e-d14c-f3c5-a703ee3b1017
Content: _WDF_DEVICE_PNP_STATE - Windows driver
Content Source: wdk-ddi-src/content/wdfdevice/ne-wdfdevice-_wdf_device_pnp_state.md
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Thanks for the feedback @BrianCatlin and apologies for the delay.  The state machine APIs are just there for internal MS use.  I've added a note to that effect to https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/content/wdfdevice/nf-wdfdevice-wdfdevicegetdevicepnpstate and related pages.


[FEAT]: Try this optimization

Description
There is excessive usage of split-path and join. Maybe export once to env then use them.
Additional Information
No response

Not worth


Added "SkillBridge - AI-Driven Competitive P2P Learning and Career Pathway Platform"

Aim: To develop an AI-Driven Competitive P2P Learning and Career Pathway Platform
whose objectives are:

To empower students in excelling at competitive exams while bridging the gap between their career goals and required skills by utilizing AI-driven personalized learning, mock tests, and peer-to-peer collaboration.
To create AI-powered mock tests using fine-tuned Gemini API for personalized exam preparation.
To foster a dynamic learning environment where peers can engage in collaborative learning through a feature-rich platform enabling video calls, screen sharing, file sharing, session recording, chatting, whiteboarding, etc.
To provide real-time assistance and guidance powered by fine-tuned LLMs for users through query analysis of text or image data.
To develop an AI model to predict the likelihood of admission and career success based on user skills and data analysis.
To utilize Gemini API to analyze skill gaps and recommend tailored courses.
To create a secure and robust user interactive platform with engaging learning content and pathways.

Screenshots:

Resolves #863

@iamrahulmahato Please consider the features and improve the level if possible. I've added the snips of main features like video calling for p2p connect, personal user dashboard to access courses, roadmaps, quiz to test their knowledge after taking courses and skill gap analysis.


Make sure can get endpoint list with empty keyword.

When keyword is empty, don't use matchQuery as a condition. FYI @peng-yongsheng
fix #1718

Coverage decreased (-0.0004%) to 14.367% when pulling 972801b214361c2c005cacbbce8857c87089b35f on fix-endpoint-query into 1bfc68c8abf37692f00ac2ceab6a8b0defb43e91 on master.


Feedback button not working for Android Remote Desktop app help page

Pressing the Feedback button says that User voice no longer works.

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

ID: 5e9e26ca-6ca8-4f7f-b033-2398576369af
Version Independent ID: b5b50e01-baf9-c2fe-7a1e-6f04a276e1fc
Content: Get started with the Android client
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/remote-desktop-android.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Also when sending feedback to the developer email address listed in the playstore description<EMAIL_ADDRESS>the email comes back with the following content:

From: Microsoft Remote Desktop for Android<EMAIL_ADDRESS>We appreciate your feedback, but you have posted it in a channel that we don't monitor.
Please submit your feedback via UserVoice https://aka.ms/rdandfbk to ensure we see it.

And that again leads to the closed instance:

Me too. No place to feedback.

#assign:xelu86

Hey henmo, nfedera, and lesca,
Thanks again for contacting Microsoft. We value your feedback.
All instances of providing feedback to the Android Remote Desktop App has been removed from this document. Feedback can still be provided by visiting one of the following:

https://aka.ms/avdandroidclientfeedback (for this app specifically)
https://feedbackportal.microsoft.com/
Feedback Hub app (if you're running a Windows 10/11 OS)

The last section also includes a way to provide feedback:

We will now be closing this thread. Thank you and have a most pleasant day.
#please-close


[SMALLFIX] Moved comments to Javadoc in 'TieredStoreIntegrationTest'

Moved the existing comments about a test to Javadocs.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7130/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7144/
Test PASSed.

LGTM. Thanks!


Run tests for some 32-bit platform in CI

A while ago there was a 32-bit-specific bug that broke some tests, to prevent this in the future we should add a step to run 32-bit tests in CI, probably just for Linux.

Implemented in 830c5f7


Fixed typo in README.md

@tisnik Your image is available in the registry: docker pull quay.io/openshiftio/rhel-fabric8-analytics-f8a-api-gateway:SNAPSHOT-PR-36


examples/hello_rust: Add feature to blink the LED

Summary

The existing Rust Demo App hello_rust only prints a message, it might be simplistic for a NuttX App. This PR extends hello_rust to call NuttX POSIX APIs like open() and ioctl(), so that it blinks an LED.
If the LED Driver is not available, the app terminates gracefully.
The updated hello_rust app is also used for testing the GPIO and LED Drivers for Ox64 BL808 SBC in Google Summer of Code.
The code is based on examples/leds in C.

Modified file examples/hello_rust:

hello_rust_main.rs: Call LED and GPIO drivers to turn the LED on and off

Impact

With this PR, NuttX for Ox64 will be able to test and verify the operation of LED through GPIO pins.
No impact on existing code, since we are extending the functionality of a Rust Demo App.
Build Instructions for QEMU RISC-V 32-bit
Build Instructions for QEMU RISC-V 64-bit
Build Instructions for Ox64

Testing
We tested the updated hello_rust app on Ox64 Emulator. The app blinks the Simulated LED correctly:

Build Instructions for Ox64
Ox64 Demo Video
Ox64 Test Log
Test Log for Ox64 Emulator

rushabhvg@rushabhvg-laptop:~/Desktop/nuttx$ ./temu nuttx.cfg
TinyEMU Emulator for Ox64 BL808 RISC-V SBC

NuttShell (NSH)
nsh> nx_start: CPU0: Beginning Idle Loop

nsh> uname -a
nxposix_spawn_exec: ERROR: exec failed: 2
NuttX 0.0.0 b8c1ce45e6 Jul  8 2024 13:30:17 risc-v ox64

nsh> hello_rust
nxtask_activate: hello_rust pid=4,TCB=0x5040b290
nxsig_tcbdispatch: TCB=0x5040b290 pid=4 signo=0 code=0 value=0 masked=NO
Hello, Rust!!
Opening /dev/userleds
Set LED 1 to 1
{"nuttxemu":{"gpio29":1}}
Sleeping...
Set LED 1 to 0
{"nuttxemu":{"gpio29":0}}
nxsig_tcbdispatch: TCB=0x5040af80 pid=3 signo=17 code=5 value=0 masked=NO
nxtask_exit: hello_rust pid=4,TCB=0x5040b290
nxsig_tcbdispatch: TCB=0x5040af80 pid=3 signo=17 code=5 value=0 masked=NO

Hmmm the NuttX Build doesn't seem happy with Rust Files. Lemme find out why...
https://github.com/apache/nuttx-apps/actions/runs/9835985035/job/27150755232?pr=2437
Run echo "::add-matcher::nuttx/.github/nxstyle.json"
Defaulting to user installation because normal site-packages is not writeable
b167569c0 Merge 17de01ddbb37445b2d4b3f0394bc2a91b7a57b88 into 6b9d51793f7373e3c48d4590a368ab8b8f42e61e
17de01ddb examples/hello_rust: Add feature to blink the LED
../nuttx/tools/checkpatch.sh -m -g 6b9d51793f7373e3c48d4590a368ab8b8f42e61e..HEAD
Diff in /home/runner/work/nuttx-apps/nuttx-apps/apps/examples/hello_rust/hello_rust_main.rs at line 22:
  * Attributes
  ****************************************************************************/
- #![no_main]
- #![no_std]
+#![no_main]
+#![no_std]
 /****************************************************************************
  * Uses
Diff in /home/runner/work/nuttx-apps/nuttx-apps/apps/examples/hello_rust/hello_rust_main.rs at line 73:
 #[no_mangle]
 pub extern "C" fn hello_rust_main(_argc: i32, _argv: *const *const u8) -> i32 {
-
     unsafe {
-
         /* "Hello, Rust!!" using printf() from libc */
         printf(b"Hello, Rust!!\n\0" as *const u8);
Diff in /home/runner/work/nuttx-apps/nuttx-apps/apps/examples/hello_rust/hello_rust_main.rs at line [11](https://github.com/apache/nuttx-apps/actions/runs/9835985035/job/27150755232?pr=2437#step:4:12)5:
     0
 }

@rushabhvg Sorry rustfmt is super particular with our whitespace and blank lines. Could you do the following:

Overwrite Rust Blinky with this version that agrees with rustfmt: https://github.com/lupyuen2/wip-nuttx-apps/blob/gpio7/examples/hello_rust/hello_rust_main.rs

Run rustfmt. It should not show anything:
$ rustfmt --edition 2021 --check hello_rust_main.rs
[should not show anything]
$

There should be a blank line at the end of the file. Otherwise rustfmt will fail:
$ rustfmt --edition 2021 --check hello_rust_main.rs
     0
 }
+
[rustfmt needs a blank line]

Commit the changes. Squash the commits into one single commit. Thank you so much! :-)

@rushabhvg should we enable rust on some board's defconfig?

should we enable rust on some board's defconfig?

@xiaoxiang781216 Thanks for the suggestion! We might add to QEMU RISC-V later, first we need to add the LED Driver to QEMU. Also we have a few manual build steps needed for Rust on QEMU, we will fix them in the coming weeks.

should we keep rust_hello independent from the hardware? but create a new example(e.g. blink_rust)?

should we keep rust_hello independent from the hardware? but create a new example(e.g. blink_rust)?

@xiaoxiang781216 Good point! Rushabh suggested a separate app, but I thought we should extend hello_rust because:
(1) By convention, there seems to be only 1 sample app per language? hello_nim, hello_zig, hello_d, ...
(2) If we look at hello_nim: It's actually doing quite a lot. Maybe hello_rust should also do something substantial?
(3) Our Rust Blinky App will terminate gracefully if the LED Driver is missing anyway
If we have no concerns about breaking convention (1), then sure we can split into a blink_rust app. Lemme know what you think. Thanks!

should we keep rust_hello independent from the hardware? but create a new example(e.g. blink_rust)?

@xiaoxiang781216 Good point! Rushabh suggested a separate app, but I thought we should extend hello_rust because: (1) By convention, there seems to be only 1 sample app per language? hello_nim, hello_zig, hello_d, ... (2) If we look at hello_nim: It's actually doing quite a lot. Maybe hello_rust should also do something substantial? (3) Our Rust Blinky App will terminate gracefully if the LED Driver is missing anyway
If we have no concerns about breaking convention (1), then sure we can split into a blink_rust app. Lemme know what you think. Thanks!

It' fine to put into hello_rust If the demo doesn't depend on any hardware feature and  can run with a basic porting. But, the change in this patch depends on userleds which mayn't exist on many board.

Yep sure no worries, we'll repackage the app as blink_rust. Thanks!


Prometheus Metrics

Added a Gauge for watching the prefill backlog size. Note port 9090 is conventional for prometheus metrics.

Looks good. Can we do a end-2-end run to ensure there no impact on perf?

Zijun, PTAL .

Could you share the Prometheus Metrics results?

@Bslabe123 yep, it would be good if we could verify the metric is added in the container logs, thanks!

Could you share the Prometheus Metrics results?

@Bslabe123 yep, it would be good if we could verify the metric is added in the container logs, thanks!

Able to validate via also deploying a prometheus server to the cluster to scrape the newly emitted metrics, will follow up with an ai-on-gke PR with validation instructions/demo

LGTM! Would also like to see the E2E validation result of this metric.

Deployed on GKE with this maxtext setup plus a prometheus server, these are metrics after  1000 concurrent curls to the maxtext http endpoint via seq 100 | xargs -P 100 -n 1 curl --request POST --header "Content-type: application/json" -s localhost:8000/generate --data '{     "prompt": "What are the top 5 programming languages",     "max_tokens": 200 }'

Looks good. Can we do a end-2-end run to ensure there no impact on perf?

Zijun, PTAL .

Timed the above 100 requests with  bashs time and got the following
real	0m26.914s
user	0m0.940s
sys	0m1.313s

Can do more extensive benchmarking if these numbers seem questionable

lgtm, thanks @Bslabe123 !


Can not build the image locally

What version of Go are you using (go version)?

go version go1.16.2 darwin/arm64

Does this issue reproduce with the latest release?
reproducible from master
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/hoaphan/Library/Caches/go-build"
GOENV="/Users/hoaphan/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/hoaphan/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/hoaphan/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.16.2"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/hoaphan/dev/code/argo-cd/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/lz/h996692x0gj09d3_hvhnn9180000gn/T/go-build3722296872=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
make from master
What did you expect to see?
Image is built successfully.
What did you see instead?
Broken at line
COPY --from=builder /usr/local/bin/packr /usr/local/bin/packr

Because there's no /usr/local/bin/packr in builder
Checking the builder itself, there seems to be several things in there that doesn't match the expectation.
If we spin up FROM golang:1.16.2 as builder
and peform the steps ininstall-packr-linux.sh, there won't be:
$GOPATH/src/github.com/gobuffalo/packr or  $GOPATH/bin/packr
and there won't be a $BIN/packr for downstream.
All I can see in GOPATH after go get are:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./pkg/mod/cache/download/sumdb/sum.golang.org/lookup/github.com/gobuffalo/packr
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./pkg/mod/cache/download/github.com/gobuffalo/packr
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Both bin and src are empty.

sorry, wrong repo


Update travis script to handle new installation procedure of osqp

This fixes travis script by handling the new installation procedure of osqp+qldld.

This closes #10


Azata Spells not working + songs.

Azata spells that recover spell slots anda abilities lvl 6 and 7 do nothing at all. Also some songs are not working at all, one that taunts specific character. Is it possible to fix those :)?

Also Icy Prison is bugged. Chance to get out of it is 15+CL but it is 19 in my case..PRobably DC instead of that check ?

I want to look into the abilities, the song I am unsure of so no promises.


add host removal from dev into main branch, version release bump

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <outdir>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

need to revise CHANGELOG to list release version number

Last update I promise. Added hostile and sra-human-scrubber outputs to the table in docs/output.md.


Support 'Show Submission History'

As the title says, it's an extension of the issue 59.
If this logic is complicated, it's better to add a show latest submission button.

+1
The following entries make sense to me:

Show Recent Submissions
Create New Submission

Can we fetch our previous submissions? Or the latest one?


Various changes/suggestions

WIP - DO NOT MERGE
Changes:

Add yarn, editorconfig, and fix a couple things in server.js

@jvoros
Some thoughts... (updated as thoughts come up)
1. Directory structure is non-optimal

bits at best isn't descriptive and at worst reminds me of the term "naughty bits". Probably should rename that and break apart stuff within to a more sensible structure.
www is an apache server convention. It is not a convention to any javascript project. I'd completely rename that to something that makes sense.
Compiled code should not be in version control (webpack bundles, etc). Bad practice to do so.

I like the cleanup.
I'm fine changing the www directory name. I come from PHP. So the apache way made sense. A little more descriptive than src, especially because this project has both the code to build the database and run the website, being clear about which items were made for the web seemed good.
I'll get webpack-dev-server set up. I had it in the other project. Used nodemon in a smaller test project, never did the necessary configuration here.

I'm fine changing the www directory name. I come from PHP. So the apache way made sense. A little more descriptive than src

Disagree. A JS developer with no PHP experience will have no idea what www means. Not a big deal; just a style thing. I only suggest src because I also think it would be wise to have a dist build directory (that is .gitignoreed).

Hmmm. The gulp build script uses the schema file too. Maybe it should be in the top level. Also considering switching to Mongorito.js which doesn't require a schema.
I'm fine changing the www folder to a different name. Just explaining my thought process. How about app? I should put server and client side code into their own files too.

It's your baby. Consider everything I say here to just be a suggestion. Ultimately, your decision for what you'd like to do.
What's wrong with the standard mongodb node package?

I like the suggestions. I want to build a robust project, not just the hacky side projects I've done in the past.
MongoDB uses the schema, which seems weird in a mongo environment, which touts a schema-less db as a feature.

I've used the standard mongodb driver in the past and I don't recall ever having to build schemas. Is this just a convenience feature?

That way it'll pave the way for your unit tests as well.

That's a good idea. I'm going to use app instead of src. Something about src bugs me ;)

that is nice too because I am going to move the component styles to a module-style format with a separate style file for each component, imported with the React component. That structure will make it easy to build

Have you seen styled-components or styled-jsx? Either of those would be cool to play around with.


Create Demo/Test App for OpenAPI Java Client

Create a demo/test application which uses the OpenAPI Java client.

@Jeeppler
I would like to ask you to explain a little bit this task.
Do you need a JavaFX or Swing application or something else?
Are there any documentation about this task I have to read?
Thanks

@zigfridus sure, here are more details about the issue/task.
The OpenAPI Java client is a library, which is generated from the OpenAPI file using OpenAPI Generator.
The OpenAPI file is generated from REST Docs tests using a REST Docs OpenAPI plugin.
The pipeline looks like this:
Spring Boot Rest Doc Tests ----> Generate OpenAPI file ----> OpenAPI file (JSON) ----> OpenAPI Generator ----> Java client library

To generate the Java client library the following commands are necessary:
// generates OpenAPI file into sechub-doc/build/api-spec/openapi3.json
./gradlew ensureLocalhostCertificate
./gradlew generateOpenapi

// build Java client library the jar can be found in sechub-api-java/gen/build/libs/sechub-api-java-0.0.0.jar
./gradlew :sechub-api-java:buildAPIJava

My suggestion would be to write a small CLI tool which allows one to interact with the server. For example, a CLI tool which allows one to check whether the server is alive or not and list all users. The objective is not to make a full Java CLI client (we already have a CLI client written in Go), instead the objective is to get some understanding of whether the generated library works at all. A small CLI client which implements an anonymous and a privileged command suffices.
To understand how to run the SecHub server follow the Quickstart Guide:  https://mercedes-benz.github.io/sechub/latest/sechub-quickstart-guide.html.
The REST API is well documented and can be found here: https://mercedes-benz.github.io/sechub/latest/sechub-restapi.html.
Feel free to ask any questions which you still have.

I assigned you the task. I hope this is okay for you.

@Jeeppler
I try to generate openapi3.json using
./gradlew generateOpenapi
but get a lot the same testing errors:

java.lang.IllegalStateException: Could not initialize plugin: interface org.mockito.plugins.MockMaker
Could not initialize inline Byte Buddy mock maker.

Java               : 11
JVM vendor name    : Debian
JVM vendor version : 11.0.16+8-post-Debian-1deb11u1
JVM name           : OpenJDK 64-Bit Server VM
JVM version        : 11.0.16+8-post-Debian-1deb11u1
JVM info           : mixed mode, sharing
OS name            : Linux
OS version         : <IP_ADDRESS>-microsoft-standard-WSL2
Caused by: java.lang.IllegalStateException: Could not self-attach to current VM using external process

It seems to me the problem is in my virtual machine. I'm using Debian on the Windows 10 host. I would like to ask what environment do you use for the Sechub building?

We are using Debian/Ubuntu to build SecHub.
As far as I can see you are using OS version : <IP_ADDRESS>-microsoft-standard-WSL2, meaning you are not actually using Debian with a Linux Kernel, but with the Windows Subsystem for Linux version 2.
My suggestion would be to try building it in a container. A simple Debian 11 container running an infinite while loop and then shell into the container to execute the quickstart guide commands. Or you can use a virtual machine.
From my own experience working with WSL2, it can cause unexpected behaviors. It is different from a real Linux environment.

@Jeeppler
Thank you for your last advice. The use Debian in VirtualBox took away the last problem.

@Jeeppler
Maybe I found a little bug.
When I build Java client library with

./gradlew :sechub-api-java:buildAPIJava

I see this error

/mnt/sechub/sechub-api-java/gen/src/main/java/com/mercedesbenz/sechub/api/java/model/ProjectWhiteList.java:41: error: class ProjectWhitelist is public, should be declared in a file named ProjectWhitelist.java
public class ProjectWhitelist implements Serializable {

It seems to me that there is a misspelling in the generated class name. It should be
public class ProjectWhiteList implements Serializable {
but not
public class ProjectWhitelist implements Serializable {

@zigfridus I was unable to reproduce the issue.
Your own forked version of SecHub is 193 commits behind: "This branch is 193 commits behind mercedes-benz:develop."
Please update your fork of SecHub: https://github.com/zigfridus/sechub. Your fork should only be a few commits back.
Have a look at these articles on how to do it:

https://stackoverflow.com/questions/7244321/how-do-i-update-or-sync-a-forked-repository-on-github
https://medium.com/@sahoosunilkumar/how-to-update-a-fork-in-git-95a7daadc14e

After you updated the fork, try to build the client again.

@zigfridus let me know, whether that fixed the issue.

@Jeeppler
Unfortunately updating my fork didn't help.

git remote -v
origin  https://github.com/mercedes-benz/sechub.git (fetch)
origin  https://github.com/mercedes-benz/sechub.git (push)
upstream        https://github.com/zigfridus/sechub (fetch)
upstream        https://github.com/zigfridus/sechub (push)

I see the last closed request in my sechub:

git log
commit 8d830e999ca3a11d8bab07cbb878507d3578949e (HEAD -> develop, upstream/develop)
Merge: c822c305 9d20fa8e
Author: Sven<EMAIL_ADDRESS>Date:   Wed Aug 17 16:47:19 2022 +0200

I found out that after changing the values ProjectWhitelist to the ProjectWhiteList in the previously generated openapi3.json, I can build the client without errors.
"$ref": "#/components/schemas/ProjectWhitelist"
"ProjectWhiteList": {
    "title": "ProjectWhitelist",

@zigfridus good job.
A small hint. The SecHub server is now containerized. Have a look into the sechub-solution folder and it has some setup scripts for projects/pds-solutions as well. You could use it for the Demo/Test of the OpenAPI Java client.

@Jeeppler thank you for the hint.

@Jeeppler
I have a question.
I can use in the CLI client, I working on, the AnonymousApi library and find out if a server is alive.
AnonymousApi anonymousApiInstance = new AnonymousApi(defaultClient);
anonymousApiInstance.anonymousCheckAliveGet();

But I can't find in the documentation of OpenApi Java Client, how to provide the admin_id (int-test_superadmin) and admin_password (int-test_superadmin-pwd) for the AdminApi library.
Can you help me with it?

After some exploring I guess that credentials should be provided via ApiClient. I was trying use basic authentication.
HttpBasicAuth basicAuthentication = new HttpBasicAuth();
basicAuthentication.setUsername("int-test_superadmin");
basicAuthentication.setPassword("int-test_superadmin-pwd");

Map<String, Authentication> authMap = new HashMap<>();
authMap.put("basic", basicAuthentication);

ApiClient apiClient = new ApiClient(authMap);
apiClient.setBasePath("https://localhost:8443");

AdminApi adminApiInstance = new AdminApi(apiClient);
try {
           System.out.println(adminApiInstance.adminListsAllUsers());
} catch (ApiException e) {
            System.err.println("Exception when calling AdminApi#adminListsAllUsers");
            System.err.println("Status code: " + e.getCode());
            System.err.println("Reason: " + e.getResponseBody());
            System.err.println("Response headers: " + e.getResponseHeaders());
            e.printStackTrace();
}

But this attempt failed. I've got the same error:

Exception when calling AdminApi#adminListsAllUsers
Status code: 401
Reason: {"status":401,"error":"Unauthorized","message":"Unauthorized","details":[],"timeStamp":"Wed Aug 24 17:53:00 EEST 2022"}

I get stuck in this and still need your advice @Jeeppler.
Thanks

@zigfridus sorry, for my late reply. The S3 upload problem is keeping me busy.
Your code is correct. However, the problem is, there is no information in the open API file that we are using basic authentication for adminListsAllUsers or any other Admin API. Meaning, you found a bug.
The security information should be in the OpenAPI file definition. The security information in the OpenAPI file should look similar to: https://swagger.io/docs/specification/authentication/basic-authentication/.
SecHub uses Spring REST Docs for the API documentation. To generate the OpenAPI file the com.epages.restdocs-api-spec plugin is used.
There are two main tasks:

Open an issue describing, that the basic authentication information ismissing.
Figure out how to add the basic authentication information to OpenAPI.

Let me know if you want to do any of the tasks or if I shall do them.

@zigfridus I created an issue for the missing authentication information in OpenAPI: #1581.

@Jeeppler
Ok, I'll try to resolve this authentication bug.

@zigfridus I am already on it.

@Jeeppler
Is better for me to wait until you resolve the issue #1581 or start to work on another issue?

@zigfridus you can continue with this issue. I opened a pull request for the missing OpenAPI headers: https://github.com/mercedes-benz/sechub/pull/1593.

@Jeeppler
I fetched your branch #1593 and build the new OpenApi Java Client. The admin authentication works right now.
Thanks

@zigfridus thanks for testing it. Glad to hear it works.

@Jeeppler
I would like to ask where should I put my small CLI? In which Sechub module?
Also I would like to know how to provide the constants (server path, admin username and password) for the CLI.
Should it be a config file, environment variables, program arguments or something else?

We use a domain-driven design as our architecture style for SecHub. Oversimplified: what belongs together is in the same folder. In your case, your demo app should go into the sechub-api-java folder.
In fact, there is already a class called SecHubClient.java in sechub-api-java/src/main/java/com/mercedesbenz/sechub/api/java. This class is intended to wrap around the generated AdminApi and the AnonymousApi. Providing an easy to use interface for users (other developers). The SecHubClient knows something about the SecHub report model. The SecHubClient.java is intended as a library, not a CLI tool.
To answer your questions, your code should go into sechub-api-java/src/main/java/com/mercedesbenz/sechub/api/java/demo/cli. The CLI should accept parameters passed as arguments and environment variable.
You can have a look at:

sechub-analyzer-cli - a small CLI tool which analyzes if there is a NOSECHUB or END-NOSECHUB comment in code. It uses Apache Commons CLI.
sechub-wrapper-owasp-zap - a wrapper around the API of OWASP ZAP. It uses jcommander to parse the command line parameters.

Both are small CLI tools, which you can look at on how they are build using Gradle and how the dependency management works. As they are both standalone CLI applications.
Looking into the future. SecHub needs a web UI. For the web UI SecHub needs a SecHubClient-Java library. The  demo app is important to showcase how to use the SecHubClient-Java library, especially for people wanting to integrate with SecHub from Java.

@Jeeppler
I created the small CLI tool. I hope this is what you need. I used some sources from the sechub-wrapper-owasp-zap.  I liked the way they are created.
Please review my PR.


⚠ Bofrost.be has degraded performance

In 9f603be, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1564 ms

Resolved: Bofrost.be performance has improved in 4c82f56.


Document attributeOverride tool.

Attribute override text file format is not documented in a publically available document.  Please create a readme under Hostboot and/or open-power/docs.

I have added a README file to the top level of open-power/hostboot. Someone please review to ensure it has the information we want.

Second draft is up

Third draft

We should leave issue open until code shows up.  Please put 'Resolves open-power/hostboot#11' in your commit message and github will automatically close this issue once the commit has been merged and mirrored to github.

What code? isnt this issue just for the README?

  This is what I meant.  I guess if it is just documentation there is no code but there is still a commit.

I see the confusion.  You modified hostboot directly in github, which is not allowed.  You need to make a commit in gerrit for review.


Add macro guards to semaphore/mutex tests

Add macros guards to helper functions in semaphore and mutex tests.
These allow the testsuite to be built without an implementation of thread
flags by disabling certain tests, which is useful for partial
implementations of CMSIS-RTOSv2.
The macro guards are needed because ARMClang requires osThreadFlagsWait() and osThreadFlagsSet() to be defined, even in functions that are not used.

Thanks for providing this!
Eventually all helper threads shall be moved closer to test case implementation and renamed to be consistent with similar test cases (where helper threads share the same name as test case).