Contraints on FilterChips deleteIcon too tight.

Steps to reproduce

Create a FilterChip and set an icon for deleteIcon
Increase the size

Expected results
It should be possible to support any size icon on FilterChip.
This is more noticable when using a smaller labelStyle, but still happens even with the default.
Dialing in some custom padding numbers seems to make it work.
I've left a nice repoduction in the linked dartpad
Actual results
The icon is painted off center.
I dont't know if this is rude, so please forgive me.
@TahaTesser I'm going to ping you being that you would probably know the most about this.
Thanks!
Code sample
Code sample
https://dartpad.dev/?id=4aa1ad145d994fda54c9e87556f38bd9

Screenshots or Video

Screenshots / Video demonstration

Logs
Logs
No notable logs are printed when rendering this widget

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.19.5, on Microsoft Windows [Version 10.0.22621.3296], locale en-US)
    • Flutter version 3.19.5 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 300451adae (10 days ago), 2024-03-27 21:54:07 -0500
    • Engine revision e76c956498
    • Dart version 3.3.3
    • DevTools version 2.31.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at C:\Users\dicke\AppData\Local\Android\sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.8.6)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.8.34525.116
    • Windows 10 SDK version 10.0.22621.0

[√] Android Studio (version 2023.1)
    • 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 17.0.7+0-b2043.56-10550314)

[√] VS Code, 64-bit edition (version 1.86.2)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.86.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    ! NO_PROXY is not set

[√] Connected device (4 available)
    • SM G970U (mobile) • R58M230DQHV • android-arm64  • Android 12 (API 31)
    • Windows (desktop) • windows     • windows-x64    • Microsoft Windows [Version 10.0.22621.3296]
    • Chrome (web)      • chrome      • web-javascript • Google Chrome 123.0.6312.106
    • Edge (web)        • edge        • web-javascript • Microsoft Edge 123.0.2420.81

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Thank you for the report. I can see the issue on Flutter master channel as well.  Adding a demo with enabling Devtools's overlay guidelines:

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.19.5, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.19.5 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 300451adae (8 days ago), 2024-03-27 21:54:07 -0500
    • Engine revision e76c956498
    • Dart version 3.3.3
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.15.2

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

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 123.0.6312.106

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.22.0-5.0.pre.55, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.22.0-5.0.pre.55 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ddf79b755d (5 hours ago), 2024-04-07 18:34:24 -0400
    • Engine revision c10e950c66
    • Dart version 3.5.0 (build 3.5.0-18.0.dev)
    • DevTools version 2.34.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.15.2

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

[✓] Android Studio (version 2023.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.9+0-17.0.9b1087.7-11185874)

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

[✓] Connected device (4 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D          • android-arm64  • Android 11 (API 30)
    • iPad (mobile)    • 00008103-000A1464346A201E • ios            • iOS 17.2 21C62
    • macOS (desktop)  • macos                     • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome                    • web-javascript • Google Chrome 123.0.6312.107

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Providing the sample from dartpad link here as backup.
Code sample

expand to view the code sample
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        body: Center(
          child: Column(
              crossAxisAlignment: CrossAxisAlignment.end,
              children: [
                Row(mainAxisSize: MainAxisSize.min, children: [
                  Text("Default, Slightly Off Center"),
                  SizedBox(width: 8),
                  FilterChip(
                    label: Text("Test Text"),
                    onSelected: (_) {},
                    onDeleted: () {},
                    deleteIcon: Icon(Icons.expand_more),
                  )
                ]),
                Row(mainAxisSize: MainAxisSize.min, children: [
                  Text("Label Medium, Very Off Center"),
                  SizedBox(width: 8),
                  FilterChip(
                    labelStyle: Theme.of(context).textTheme.labelMedium,
                    label: Text("Test Text"),
                    onSelected: (_) {},
                    onDeleted: () {},
                    deleteIcon: Icon(Icons.expand_more),
                  )
                ]),
                Row(mainAxisSize: MainAxisSize.min, children: [
                  Text("Custom Padding ---------- Fixed"),
                  SizedBox(width: 8),
                  FilterChip(
                    labelStyle: Theme.of(context).textTheme.labelMedium,
                    label: Text("Test Text"),
                    onSelected: (_) {},
                    onDeleted: () {},
                    padding: EdgeInsets.only(left: 16),
                    labelPadding: EdgeInsets.only(left: 0, right: 0),
                    deleteIcon: Icon(Icons.expand_more),
                  )
                ]),
                Row(mainAxisSize: MainAxisSize.min, children: [
                  Text("Label Medium, Very Off Center"),
                  SizedBox(width: 8),
                  FilterChip(
                    labelStyle: Theme.of(context).textTheme.labelMedium,
                    label: Text("Test Text With More Text"),
                    onSelected: (_) {},
                    onDeleted: () {},
                    deleteIcon: Icon(Icons.expand_more),
                  )
                ]),
                Row(mainAxisSize: MainAxisSize.min, children: [
                  Text("Custom Padding ---------- Fixed"),
                  SizedBox(width: 8),
                  FilterChip(
                    labelStyle: Theme.of(context).textTheme.labelMedium,
                    label: Text("Test Text With More Text"),
                    onSelected: (_) {},
                    onDeleted: () {},
                    padding: EdgeInsets.only(left: 16),
                    labelPadding: EdgeInsets.only(left: 0, right: 0),
                    deleteIcon: Icon(Icons.expand_more),
                  )
                ]),
              ],
              mainAxisSize: MainAxisSize.min),
        ),
      ),
    );
  }
}

@dickermoshe
I'll take a look, thanks!


Optimize TextView implementation for iOS, Android

Label used to support displaying images as part of the label. With dedicated widgets for ImageView and TextView the implementation can be simplified and made more efficient.
https://github.com/eclipsesource/tabris-js/issues/269

We already have native TextView implementations on both platforms that support only the required features. Label is not used by Tabris.js anymore. See efc678489f0d4ac1399b0585b80e9a57c9ca229a.


Messages in queue not cleaned

There are a number of messages in the queued_messages table which have the locked_by field filled in with a reference presumably to a no-longer-existing process. The messages date back some months which means the original message has been removed from the server.
This means that there are several queue pages in the UI which throw exceptions because they're trying to load messages which don't exist.
Is the issue that queued_messages are never cleaned up by the automatic process or that the automatic clean up process removes messages despite them being present in the queue?

@willpower232 we can try that by using smtp relays......but there is problem with authentication in relays
Is there any solution for that????

(Manish's comment is unrelated and resolved by https://github.com/atech/postal/issues/48#issuecomment-373334092)

can this help with #717 ? and can I run this via rails console bin/rails c?

If you have deleted the messages from the main table and are now unable to open the queued messages page, then this will help you.
The process of catching the exception to delete a disassociated message might be able to be run from the console but I am not sure how.

In case anyone else needs the fix, the workaround looks like this
Add the following at the end of the queue method here https://github.com/atech/postal/blob/master
       countbroken = 0
       for message in @messages
               begin
                       test = message.message
               rescue
                       message.destroy
                       countbroken += 1
               end
       end

       if countbroken > 0
               @messages = @server.queued_messages.order(:id => :desc).page(params[:page])
       end

Hello @willpower232
I trust you are doing great.
What is the file location to apply this solution?
Thanks

Terribly sorry, not sure what happened to the link.
I added that code between these two lines https://github.com/atech/postal/blob/master/app/controllers/servers_controller.rb#L70-L71

Alright.
Thank you!

can confirm that the "queued messages" counter is increasing over month. locked at and locked by was set, thats why those messages were never cleaned up.
my fix for this is running regulary:
update queued_messages set locked_at=null, locked_by=null WHERE locked_at is not NULL AND locked_at <= NOW() - INTERVAL 6 WEEK
Somewhat dirty, but still solving the issue of stuck queued messages bringing the counter back to 0

@willpower232 are you still running this patch in Postal V2? Is it still needed?

I'm not aware of this issue being fixed so am still running this patch.

@willpower232 Is this going to be fixed for V2? Is not feasible this approach with docker. In fact any fix that require us to bind files from containers and modify them is not ok.

Hello, any updates?
still error in v2

@willpower232
How the def will be at the end?
71  def queue
 72 #    @messages = @server.queued_messages.order(:id => :desc).page(params[:page])
 73 #    @messages_with_message = @messages.include_message
 74     countbroken = 0
 75        for message in @messages
 76                begin
 77                        test = message.message
 78                rescue
 79                        message.destroy
 80                        countbroken += 1
 81                end
 82        end
 83
 84        if countbroken > 0
 85                @messages = @server.queued_messages.order(:id => :desc).page(params[:page])
 86        end
 87
 88   end

I added the code after @message = and before @messages_with_message = as those lines need to remain
def queue
	@messages = @server.queued_messages.order(:id => :desc).page(params[:page])

	countbroken = 0
	for message in @messages
		begin
			test = message.message
		rescue
			message.destroy
			countbroken += 1
		end
	end
	if countbroken > 0
		@messages = @server.queued_messages.order(:id => :desc).page(params[:page])
	end

	@messages_with_message = @messages.include_message
end

See #2869


Check if texts in the website can be changed through Transifex

UPDATE: We can update texts through Transifex but it is a paid feature

Where are the files in the repository? And what is the process for using the Transifex CLI to push the updates to the central language up to the project on Transifex?

I spoke to Transifex. We are already on the open source tier - which gets us the basic plan for free. There is no way to get premium features without significant cost.
I see @danbjoseph made the text change yesterday - I guess this is the way forward, just needs documenting (which you might already be on?).

Ok Transifex have followed up and said they'd be willing to give us the Premium features for free if we add their logo to the website as a partner.
I think this would have to be agreed by the governance team but it would be good to understand technically if this is needed. Perhaps editing directly on Github is good enough with sufficient documentation?
cc. @danbjoseph @Katbaloo @kopitek8

i've started a convo with the rest of the governance team in slack and will post the outcome of our discussion here.

editing on GitHub is fine


Implement Lever's special requirement for tracking pairs of matching cards returned

This is the text on the card:
You may return any number of cards from your hand. For every two cards of matching value returned, draw a card of value one higher.
The "return any number of cards" part of this card means that you can return multiple of the same age to draw up to the next age.  For example, if you have 4 2's in hand, you can return all 4 of them to draw 2 3's.  So tracking how many of each age returned is important.
So figuring out how to store both ages and counts is the challenge here.  This card is kind of like currency, but the number of each age returned is important.

A 'multiselect' feature might be useful here (like we discussed with Tools briefly).

this can be closed I think.


Request timeout for subtitler.py

I tested on short and long voices where the latter just returns to request timeout
Edit: Actually it's not about the duration, it now occurs often somehow both short and long.

Maybe try increasing the request timeout? Also play around with RECORD_TIMEOUT and PHRASE_TIMEOUT in the .env file if you can

Maybe try increasing the REQUEST_TIMEOUT? Also play around with RECORD_TIMEOUT and PHRASE_TIMEOUT in the .env file if you can

here I have the same problem and I try to fix by set REQUEST_TIMEOUT=50 but its still  Request timeout


[wip] Add pre-upgrade task for moving credentials file

This reverts commit 7ef9f4dfdd7d64876aacc48a982313dbea8a06f5.

ci check this

@Atoms  can you re-review this?


Sass deprecation warning

Describe the bug
Sass is throwing a depreciation warning during build time:
Deprecation Warning: $weight: Passing a number without unit % (0.8) is deprecated.

To preserve current behavior: $weight * 1%

More info: https://sass-lang.com/d/function-units

   ╷
31 │ $select-color-dropdown-border-top: mix($input-border-color, $input-bg, 0.8) !default;
   │                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   ╵
    node_modules/tom-select/src/scss/tom-select.bootstrap5.scss 31:36

To Reproduce
Do an import tom-select.bootstrap5.scss and build your project with Sass 1.56.0. The depreciation message above will be displayed.
Expected behavior
Don't receive any deprecation messages.
Additional context
It's happening with the latest release of this package: v2.2.2.
<!--
Thanks for taking the time to submit your bug report,
and don't forget to star our repo!

same deprecation is present with BS4
@import '~tom-select/src/scss/tom-select.bootstrap4.scss';


Prevent 3-to-4 project converter from adding extra line for move_and_slide

Hi, I've come up with solution to issue #66054.
The "velocity" line adds because of move_and_slide returning velocity in older version. I'm checking now if the line begins with variable declaration or something else. If yes it's adding the velocity line.
I made the same change in move_and_slide_with_snap part, where was the same problem.

I have failed static checks. Is it because I didn't use clang-format?

Yeah, I added now test and changed solution a bit so the converter will keep the comments in the line with move_and_slide function

After doing the edit I requested, could you squash the commits? See PR workflow for instructions.

Please also amend the commit message so that it says what it does in a readable way. The PR title for example is good, and the commit message should be similar, e.g.:
Prevent 3-to-4 project converter from adding extra line for `move_and_slide`

Fixes #66054.

Thanks! And congrats for your first merged Godot contribution :tada:


Add support for default delay response

Motivations
Let's say an user wants to add a fixed delay to all of his responses, so his mock better resembles actual network conditions. On the current version of Rodo, the only way to achieve that is using the withDelay method explicitly on each response.
Proposed solution
Add support for Rodo server to receive an optional parameter to define a default delay for all responses.
Implementation

Aside from port and hostname, when instantiating Rodo server you can now pass a third argument containing an object with supported extra options.
The only supported option for now is defaultResponseDelay, which has a default value of 0ms and can be used to define a default delay for all responses.
Even if a default was defined, user can still use withDelay method to specify another delay.
I didn't pass the options received by the server directly to the Builder, as eventually there may be options intended just for the server. The same applies for the Builder when passing options to the Response object.
Added tests and updated Readme.

published in 2.2.0


this package don't support the latest version of gradle-7.3.3

yes, not supported. can anyone update it?

if you mean to update gradle so yes.
but package is up to provider of it
في الجمعة، 28 يناير 2022 في 12:58 م تمت كتابة ما يلي بواسطة SimonCC <
@.***>:

yes, not supported. can anyone update it?
—
Reply to this email directly, view it on GitHub
https://github.com/mtellect/CameraDeepAR/issues/37#issuecomment-1024058362,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUVKOXAUK2YVCC555MAQ5FTUYJSGDANCNFSM5MZSSSJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


feat(pricing): update pricing page

PR Checklist
Please review the guidelines for contributing to this repository.

[X] I am requesting to pull a topic/feature/bugfix branch (right side). In other words, not master.
[X] I have run npm test against my changes and tests pass.
[X] I have added or edited necessary documentation, or no docs changes are needed.

Description
Comprehensive revision of pricing page.
Caveats:

Not sure what's going on with the fourth column getting bumped over, it happens regardless of screen size
Needs a styling pass
New content is open to revision

I'm working off this branch for the content, but will open another PR with the new design.


data race detected

We (cockroachdb) are running test builds with the race-detector enabled. These have caught a data race multiple times in lightstep.
Please see https://github.com/cockroachdb/cockroach/issues/8793 for details (and full trace log)

Looks like you guys have gone down the road of supporting GRPC to avoid the data races in thrift. According to @RaduBerinde using grpc results in the following log message:
LightStep instrumentation error (rpc error: code = 4 desc = context deadline exceeded). Set the Verbose option to enable more logging.

Is this something expected, or are we simply doing it wrong? The change was made in https://github.com/cockroachdb/cockroach/commit/0abeb5a48dcde340ca14bd09789857792b58455c.
cc @bensigelman @tschottdorf

Just to follow-up, the answer to the above question is here: https://github.com/cockroachdb/cockroach/pull/10121#issuecomment-255507645 and the issue was fixed in #54

Closing as this is resolved.


Customize different types of carousels

It would be nice to make some modification to the carousel we already have on the Chico UI.
We would like to have the availability to customize it, to be able to add our own properties, example: Width, Height, Margin, Padding...etc.
Nowadays if we change properties, the limit per page does not work correctly.
The idea is to create and customize different types of carousels like the attached example.

@rociocalvo, as per Chico v2.0.6 you have the ability to disable autoHeight and autoMargin which along with limitPerPage will allow you to customize the Carousel to a certain extent.
Options to allow further customizations will not be added since are not aligned with the project's current objectives.
Thank you very much for the suggestion. 😀


Add docs clarification to resolve #3046

There must be at least one subchart or template file, therefore the charts and templates directories are not strictly optional.

CLA signed

Thanks!
On Sat, 28 Oct 2017 at 19:43, Taylor Thomas<EMAIL_ADDRESS>wrote:

Merged #3047 https://github.com/kubernetes/helm/pull/3047.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/helm/pull/3047#event-1315271682, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABiUYg-muBOetiKePVbrOYzToYq6yR5Xks5sw3XYgaJpZM4QAiAJ
.


Added exception data attribute

Description

Adds a data attribute to the custom exception classes
Adds a few new exception classes

This will be very helpful when rescuing from errors, because now you can attach and see additional data related to the error. For instance, the response object returned from a bad REST call. You can do whatever you want with it, not just have to deal with the message string.
Issues Resolved
(none)
Check List

[x] New functionality includes testing.

[x] All tests pass ($ rake test).

[x] New functionality has been documented in the README if applicable.

[ ] New functionality has been thoroughly documented in the examples (please include helpful comments).

[x] Changes are documented in the CHANGELOG.

@tmiotto , changes made


Add update command to upkie_tool

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 90.172%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
2679

Relevant Lines:
2971

💛  - Coveralls


[Automation] JS/Language Specific Static Checks and Lint

Proposed changes
Language Specific (Go) static should adhere to known standards. Look into:

TODO

Context

Why is this change important to you? How would you use it? How can it benefit other users?

Possible Implementation

Not obligatory, but suggest an idea for implementing addition or change

Other information

Anything else we should know? (e.g. detailed explanation, related issues, links for us to have context, eg. stack overflow, codepen, etc)

Can you provide more context here? The SDK is predominently static typed and linted already.

I thought they were already done to begin with, but these were just issues to track. If they are done, I can close.


Generate "protocol" log with version info, etc. for report validation

tweasel version
version and hash of subject
etc.

This is pretty much what I'm doing in https://github.com/tweaselORG/cyanoacrylate/issues/39. And since cyanoacrylate write the metadata into the HAR file, we'll get the functionality here "for free" after we upgrade.
See https://github.com/tweaselORG/cli/issues/34 for displaying the information.


Minor script updates and quality of life improvements

Summary

This PR makes the following changes:
- Renames the wdio scripts for consistency
The sub-scripts can now be called as wdio:default instead of wdio-default. To prevent bloat in this PR, the wdio scripts at the package level will be updated in a follow up PR.
- Adds a no-coverage jest script
To help speed up testing and prevent boated output when focusing on specific tests. This was also added to the CICD to optimize the build process as the coverage report is meant to be human-readable and has no impact on the output.
- Generates the package-lock.json file
To help optimize the npm install process both locally and in the CICD.
- Updates the Dockerfile to npm 9
It was using the default npm v6.
- Adds a new no-warnings sub-script for scss
This script was added to suppress warnings as it was inflating output and making it difficult to spot errors.
Before:

After:

Testing

This change was tested using:

[ ] WDIO
[ ] Jest
[ ] Visual testing (please attach a screenshot or recording)
[x] Other (please describe below)
[ ] No tests are needed

This was tested by successfully running the scripts.

Jest output

 Stylelint output

Reviews
In addition to engineering reviews, this PR needs:

[ ] UX review
[ ] Accessibility review
[ ] Functional review
[X] Tech review

The Dockerfile changes will need to be tested by running a devcontainer and testing the npm version.
Additional Details

Thank you for contributing to Terra.
@cerner/terra

Generates the package-lock.json file
To help optimize the npm install process both locally and in the CICD.

Package.lock would need to be re-generated and committed back to repo every time the component that is dependent on is updated. I think this will be a additional step for existing projects and might require additional efforts to maintenance.

Generates the package-lock.json file
To help optimize the npm install process both locally and in the CICD.

Package.lock would need to be re-generated and committed back to repo every time the component that is dependent on is updated. I think this will be a additional step for existing projects and might require additional efforts to maintenance.

What maintenance concerns do you have? This will automatically be added/updated whenever we do npm install (which ideally should happen at least once per story) and we just need to commit an potential changes. If there are conflicts, then we would simply need to delete the file and run npm install again.
In return, we'll get:

Faster install times both locally and on the CICD.
More consistent dev environment across all engineers.

Generates the package-lock.json file
To help optimize the npm install process both locally and in the CICD.

Package.lock would need to be re-generated and committed back to repo every time the component that is dependent on is updated. I think this will be a additional step for existing projects and might require additional efforts to maintenance.

What maintenance concerns do you have? This will automatically be added/updated whenever we do npm install (which ideally should happen at least once per story) and we just need to commit an potential changes. If there are conflicts, then we would simply need to delete the file and run npm install again.
In return, we'll get:

Faster install times both locally and on the CICD.
More consistent dev environment across all engineers.

We had scenario where we had made fix on terra-arrange as part of impact testing we performed testing of terra-menu and other components of terra-framework by having tar.gz file of terra-menu. since we had changed the version to refer to tar.gz  file version package.lock file was re-generated on local but this fix was not available on terra-ui even after a week of terra-arrange release. Cause for fix not being available on terra-ui is package.lock file of terra-framework which was still referring to an older version of terra-arrange. So we were required open-up a PR update package.lock manually to pull in the latest version. ( so this is one of maintenance concern as it is new additional step which will be required on each of those components which has dependent components on different repo ) .
Does this scripts auto updates the package.lock on timely interval to ensure latest versions are being pulled into project.

@supreethmr

We had scenario where we had ...

Are you sure that the package-lock was the cause? Because that does not sound correct. The package-lock is for the terra-framework repo and Terra UI has no visibility to that. Nor should Terra UI be affected by the terra-framework repo changes as they are two independent repos. I can delete the terra-framework repo and this will have no impact on Terra UI.
What Terra UI does is pull in the terra-framework-docs package and we have disabled package-lock for each individual package. Even if it wasn't, package-lock won't be be published to the npm registery (https://docs.npmjs.com/cli/v9/configuring-npm/package-lock-json).
It feels like there was a different cause because Terra UI only has visibility to the terra-framework-docs package and should not be affected by the repo at all.

@supreethmr

We had scenario where we had ...

Are you sure that the package-lock was the cause? Because that does not sound correct. The package-lock is for the terra-framework repo and Terra UI has no visibility to that. Nor should Terra UI be affected by the terra-framework repo changes as they are two independent repos. I can delete the terra-framework repo and this will have no impact on Terra UI.
What Terra UI does is pull in the terra-framework-docs package and we have disabled package-lock for each individual package. Even if it wasn't, package-lock won't be be published to the npm registery (https://docs.npmjs.com/cli/v9/configuring-npm/package-lock-json).
It feels like there was a different cause because Terra UI only has visibility to the terra-framework-docs package and should not be affected by the repo at all.

I mean terra-framework site : https://engineering.cerner.com/terra-framework/components/terra-abstract-modal/abstract-modal/abstract-modal
And also I would like to know time difference in build time before and after inclusion of package.lock file in repo.

And also I would like to know time difference in build time before and after inclusion of package.lock file in repo.
@supreethmr

This branch (with package-lock)

Main (without package-lock)


fix: Oracle validations of tables with hyphen in name

Before this PR, SQL Alchemy has conflicting behaviour around Oracle table names.

Any table name is uppercased before querying the data dictionary unless it contains special characters. Really the presence of special characters is irrelevant for this decision. We either uppercase or don't uppercase, special characters should not be a factor.
Any table name is double quoted during query generation if it contains special characters but it is not upper cased. The double quotes force Oracle to look for a lower case table name and fail. If we are forcing an upper case name for data dictionary (item 1 in this list) then we may as well force upper case in item 2 also.

This PR changes both code paths above so we always upper case the name in item 1 and be sure to upper case it if adding double quotes in item 2.

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Replace current kernel and xam to 17559 in older dash flash images to allow new game to work on old dashes

Is your feature request related to a problem or missing feature? Please describe.
I believe it could be possible to replace xbkrnl.exe and xam.xex of older dashboards with a 17559 hacked kernel version and allow it to boot newer games while using older dashboards and launching games via xex menu at least if not regularly naturally replacing the kernel should also replace the hypervisor.. so with updated kernel + hypervisor + xam it should work right? can you confirm this I haven't tried it yet myself but I have a triple nand with external nand wires to test
Describe the solution you'd like
.
Additional context
No response

This is not the place for this. It is unlikely that this would work.


refactor: break down the design page(the first step)

Description
In this PR
This is the first step to break down the design page. This PR just split the design page to four parts, but the states in every component is not optimal.

Create this PR to avoid too many conflicts
Next step
Optimize every component and states

Task Item
refs #5559

Screenshots

Thanks @GeoffCoxMSFT  @srinaath @hatpick
Do you think we can merge this PR first if all checks pass? Then I can do optimize for every single component in successive PR's

Thanks @GeoffCoxMSFT  @srinaath @hatpick
Do you think we can merge this PR first if all checks pass? Then I can do optimize for every single component in successive PR's

I think it makes sense to get this PR in early. The more we delay more workstreams are going to refactor their code furthur later.

I think it makes sense to get this PR in early. The more we delay more workstreams are going to refactor their code furthur later.

Coverage increased (+0.09%) to 54.555% when pulling d5087266630e833d2aab2faaa5bcf5ae0b9ffd91 on lei9444:breakdesign into 5af3f50ef41e3ef471771deb141437b3c3ae6e92 on microsoft:main.

Coverage increased (+0.09%) to 54.555% when pulling d5087266630e833d2aab2faaa5bcf5ae0b9ffd91 on lei9444:breakdesign into 5af3f50ef41e3ef471771deb141437b3c3ae6e92 on microsoft:main.


Format all files; Fix Pylint errors for app_api_v1.py

Format all files using:
https://marketplace.visualstudio.com/items?itemName=jbockle.jbockle-format-files
Pylint fixes for app_api_v1.py using:
https://pypi.org/project/pylint/
We should gradually fix all Pylint errors and then add Pylint to Github Actions to catch any new lint errors.


Update ReadDataFromKinesis URN to registered URN

This PR fixes issue #23693 as discussed with @mosche. There is a bug where the ReadDataFromKinesis class references the wrong URN, preventing it to start the expansion service.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[X] Choose reviewer(s) and mention them in a comment (R: @username).
[X] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @mosche


Uncaught runtime errors: Hook useDoc is called outside the .

Uncaught runtime errors:
ERROR
Hook useDoc is called outside the .
ReactContextError
at useDoc (webpack-internal:///./node_modules/@docusaurus/preset-classic/node_modules/@docusaurus/theme-common/lib/contexts/doc.js:29:112)
at DocItemMetadata (webpack-internal:///./node_modules/@docusaurus/preset-classic/node_modules/@docusaurus/theme-classic/lib/theme/DocItem/Metadata/index.js:14:139)
at renderWithHooks (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:16306:18)
at mountIndeterminateComponent (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:20070:13)
at beginWork (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:21583:16)
at HTMLUnknownElement.callCallback (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4165:14)
at Object.invokeGuardedCallbackDev (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4214:16)
at invokeGuardedCallback (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:4278:31)
at beginWork$1 (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:27447:7)
at performUnitOfWork (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:26553:12)
My package.json dependencies are:
"dependencies": {
"@docusaurus/core": "^2.4.3",
"@docusaurus/plugin-content-docs": "^2.4.3",
"@docusaurus/preset-classic": "^2.4.3",
"@mdx-js/react": "^1.6.22",
"clsx": "^1.2.1",
"docusaurus-plugin-openapi-docs": "^2.0.0-beta.4",
"docusaurus-plugin-sass": "^0.2.5",
"docusaurus-theme-openapi-docs": "^2.0.0-beta.4",
"prism-react-renderer": "^1.3.5",
"react": "^18.0.0",
"react-dom": "^18.0.0",
"react-lottie": "^1.2.3"
},

I'm having the same issue

Also having this issue
  "dependencies": {
    "@docusaurus/core": "2.4.3",
    "@docusaurus/preset-classic": "2.4.3",
    "@mdx-js/react": "^1.6.22",
    "clsx": "^1.2.1",
    "docusaurus-plugin-openapi-docs": "^v1.7.2",
    "docusaurus-theme-openapi-docs": "^v1.7.2",
    "prism-react-renderer": "^1.3.5",
    "react": "^17.0.2",
    "react-dom": "^17.0.2"
  },

Hi everyone, Docusaurus introduced some breaking changes to our plugin in 2.4.1. For now, please stick to 2.4.0.

Hi @sserrata - this is still happening with yesterdays latest release v3.0.1 and a fresh install of docusaurus latest version.
Are they any plans to fix it?

I had the same issue. I downloaded the latest of both packages docusaurus-theme-openapi-docs & docusaurus-plugin-openapi-docs.
"docusaurus-plugin-openapi-docs": "^3.0.1",
"docusaurus-theme-openapi-docs": "^2.2.1",

For some reason the latest for docusaurus-theme-openapi-docs is 2.2.1. I upgraded that to 3.0.1 and it worked
"docusaurus-plugin-openapi-docs": "^3.0.1",
"docusaurus-theme-openapi-docs": "^3.0.1",

@Leonzon95 doing that fixed it for me too. Not sure why but running:
yarn upgrade docusaurus-theme-openapi-docs@latest
installs v2.2.1 like you said. Not sure if its important but I also cleaned (removed) all openAPI docs before upgrading, and then regenerated them afterward.
`yarn clean-api-docs all
change docusaurus-theme-openapi-docs version in package.json
yarn install
yarn gen-api-docs all
yarn start`

It was a situation where v2.2.1 was released after 3.0.1, which resulted in v2.2.1 getting tagged as "latest". I manually tagged 3.0.1 as latest for docusaurus-plugin-openapi-docs but forgot to do the same for the theme. Should be fixed now.

I also met the same problem, but now, I changed the node version to 20 and pnpm version to 9.6.0.
Then, it works


bug: component box with a child component box doesn't render

Description

A component returning a box with a child component returning another box doesn't render

Version

pre v.0.0.1

Steps to Reproduce

Create two components:

from reactpy import component, run, html
from reactpy_material import box

@component  # @componet is removed: OK
def hello_box():
    return box(
    # return html.div(   # OK if s/box/html.div/
        html.h1(
            {"key": "title1"},
            "Hello World"
        ),
        key="hello box"
    )

@component
def hello_box_inside_box():
    return box(
        hello_box(),
        key="external box"
    )

run(hello_box_inside_box)
# run(hello_box) # running child component - OK

Run the code and inspect in the browser and nothing is rendered, resulting in a blank page.
Inspect the web page and find the error: "Uncaught DOMException: Failed to execute 'createElement' on 'Document': The tag name provided ('') is not a valid name."

same problem


Add support for rocky linux in bootstrap

I tried running this inside rocky linux and it complained that it was not supported. Rocky in the EL family so I added the name there.

@soumyakoduri Hello, this is a small change to support rocky linux

@soumyakoduri will this ever be merged?

@soumyakoduri will this ever be merged?

I was waiting then for additional review. I guess this can be merged.

cherry-picked to ceph-master - https://github.com/ceph/s3-tests/commits/ceph-master


enable restart button

:+1:

Can we make this a POST button please? If not now then let's at least get a ticket for it. If one hits the back button the app will restart again.

Ah yes, that was the 1 other thing that was still pending for @olpie101 to do (which is probably why the button wasn't there). We did it for the Delete button #21
I'll ticket it

:cake: awesome, thank you.


Transitive runtime dependencies are part of compile configuration

If I declare a compile dependency (e.g. to io.reactivex:rxnetty:0.4.19) I see that gradle also puts all transitive runtime dependencies into the compile configuration (in this case for example io.reactivex:rxjava:1.0.17).
Expected Behavior

Example:
dependencies {compile 'io.reactivex:rxnetty:0.4.19'}
My Java Code using the class rx.exceptions.Exceptions should not compile.
The compile should fail because the dependency io.reactivex:rxnetty:0.4.19 provides only runtime dependencies and rx.exceptions.Exceptions is part of io.reactivex:rxjava.

Current Behavior

The transitive runtime dependencies are added to the compile configuration. They should not be added to the compile configuration.

Context

It leads to unwanted dependendencies and has a reasonable impact for the migration to gradle in our company regarding tool acceptance and management of our dependencies.

Steps to Reproduce (for bugs)

I will upload a sample project here.

Your Environment

There is no specific environment needed. A pure call of gradle dependencies in the sample project shows the tree including the transitive runtime dependencies in the compile configuration.
gradle_example.zip

Build scan URL:

We are actively working on this, so you can expect a fix in Gradle 3.4.

This is fixed for project dependencies in 3.4 if you use the new java-library plugin. We still need to fix it for external dependencies (e.g. separating compile and runtime scope for Maven dependencies).

@oehme Is this still scheduled for 4.0?

I don't think we'll be able to make it. 4.1 seems more realistic.

@oehme cool, moved

Is this going to be scheduled soon? :)

This is now implemented (see #3360). Please feel free to give the latest nightly a try.
It will be in 4.5, but we will only make it the default behavior in 5.0 as it might break existing builds.
If you try the nightly, you can switch the behavior on by adding this to your settings.gradle:
gradle.experimentalFeatures.enable()
(4.5 will include this opt-in switch as well but the syntax might change for the release)

Closing the loop here, Gradle 4.6 added
enableFeaturePreview('IMPROVED_POM_SUPPORT') in settings.gradle
for this. Gradle 5.0 will make it the default.

@oehme you mentioned in feb 2017

We still need to fix it for external dependencies (e.g. separating compile and runtime scope for Maven dependencies).

Is this fixed by now and if so, what do I need to do that maven <scope>runtime</scope> dependencies are not treated as compile?

This is for external dependencies, see the title of the issue.

Well then, it still does not work with 4.5.1
I got a maven project which has a dependency defined as <scope>runtime</scope>, I have put gradle.experimentalFeatures.enable() in my gradle.properties. I am depending on the maven project and can still access code from the dependency defined in scope runtime.

Man... my fault, I should have put it in settings.gradle => works. Sorry for the noise

@trevjonez Sorry, I don't see the problem. The compile classpath contains your api dependency and the runtime classpath contains your implementation dependency, just as expected.

@oehme I didn't do a good job of explaining it. I am expecting debugCompileClasspath to resolve as:
\--- com.trevjonez:and-lib:0.1.0
     \--- com.trevjonez:and-lib_release:0.1.0
          \--- io.reactivex.rxjava2:rxjava:2.2.0
               \--- org.reactivestreams:reactive-streams:1.0.2

but it is resolving as:
\--- com.trevjonez:and-lib:0.1.0
     \--- com.trevjonez:and-lib_release:0.1.0
          +--- io.reactivex.rxjava2:rxjava:2.2.0
          |    \--- org.reactivestreams:reactive-streams:1.0.2
          \--- com.squareup.moshi:moshi:1.6.0
               \--- com.squareup.okio:okio:1.14.0

all of the implementation dependencies are being exposed on the compile classpath as well as the runtime classpath.

Are you sure you linked the right test case? I only see asserts on files being there.

oops, I linked to master and had removed the asserts last night. I updated the original link with this commit specific url

@oehme
here is a build scan for the library publishing build:
https://scans.gradle.com/s/3x4vcfaxtgt5m/
here is a build scan for the consuming application build: https://scans.gradle.com/s/mfmp2gtq2duwk/dependencies?toggled=W1swXSxbMCw3XSxbMCw3LFswXV0sWzAsNyxbMCwxXV0sWzAsNyxbMCwxLDRdXSxbMCw3LFswLDEsMl1dLFswLDhdLFswLDgsWzBdXSxbMCw4LFswLDFdXSxbMCw4LFswLDEsNF1dLFswLDgsWzAsMSwyXV1d

so I went that route so that I could still support consumers depending on the library without specifying the default variant artifact directly. I opened this issue explaining the issue I was seeing when publishing the same artifact under both the root component and release variant. https://github.com/gradle/gradle/issues/6389

Have you checked whether the POM for that default variant actually has properly separated <compile>/<runtime> dependencies? Just making sure this is not a problem on the publishing side.

I did yea. Here is a gist with the relevant POM files and the .module so you can see what is placed there in order for me to generate a usable POM given the issues from 6389.
https://gist.github.com/trevjonez/62b3d7907c13ca0728139d85ddf9cfbb

Oh, you simply forgot the IMPROVED_POM_SUPPORT feature preview, that's what the api/impl separation is bound to until Gradle 5.0

Can you try consuming that lib from a plain Java project? Just making sure this issue is not specific to the Android plugin.

looks like a java library is also pulling in all the runtime transitives as compile dependencies
https://scans.gradle.com/s/dmsu7idjx2c2k/dependencies?toggled=W1swXSxbMCwxXSxbMCwxLFswXV0sWzAsMSxbMCwxXV0sWzAsMSxbMCwxLDRdXSxbMCwxLFswLDEsMl1dXQ
scan from java-lib project after modifications in this commit

It's exactly as you suspected @oehme: when the packaging is not a "known Jar packaging" then we don't create any variants for the module, which means that the compile/runtime separation is lost.
The non-jar packaging case is not one I considered when I put together this complicated sheet of dependency interactions. (Apologies, I can't seem to make that sheet public). We really need to think about the behaviour we want to make the default in 5.0.

Thanks Daz, I've opened a follow-up issue for 5.0

I can confirm this is fixed from Gradle 5.0 for transitive aar (android) runtime dependency.


无法使用了，错位代码403

#################### Log for 控制论(或关于在动物和机器中控制和通信的科学) (科学元典丛书) ####################
Running identify query with parameters:
{'title': '控制论(或关于在动物和机器中控制和通信的科学) (科学元典丛书)', 'authors': ['维纳'], 'identifiers': {'asin': 'B008H0V4QE'}, 'timeout': 30}
Using plugins: New Douban Books (1, 0, 7)
The log from individual plugins is below
****************************** New Douban Books (1, 0, 7) ******************************
Found 0 results
Downloading from New Douban Books took 0.6469862461090088
Plugin New Douban Books failed
Traceback (most recent call last):
File "calibre\ebooks\metadata\sources\identify.py", line 47, in run
File "calibre_plugins.dummy1.init", line 293, in identify
File "calibre_plugins.dummy1.init", line 61, in search_books
File "calibre_plugins.dummy1.init", line 45, in load_book_urls
File "urllib\request.py", line 216, in urlopen
File "urllib\request.py", line 525, in open
File "urllib\request.py", line 634, in http_response
File "urllib\request.py", line 557, in error
File "urllib\request.py", line 496, in _call_chain
File "urllib\request.py", line 749, in http_error_302
File "urllib\request.py", line 525, in open
File "urllib\request.py", line 634, in http_response
File "urllib\request.py", line 563, in error
File "urllib\request.py", line 496, in _call_chain
File "urllib\request.py", line 643, in http_error_default
urllib.error.HTTPError: HTTP Error 403: Forbidden

The identify phase took 0.83 seconds
The longest time (0.646986) was taken by: New Douban Books
Merging results from different sources
We have 0 merged results, merging took: 0.00 seconds
Failed to download metadata for 控制论(或关于在动物和机器中控制和通信的科学) (科学元典丛书)
****************************** New Douban Books Covers ******************************
Request extra headers: [('User-agent', 'Mozilla/5.0 (Windows NT 10.0; WOW64; Trident/7.0; rv:11.0) like Gecko')]
Failed to download valid cover
Took 0.6448569297790527 seconds
No cached cover found, running identify
Failed to download cover from New Douban Books
Traceback (most recent call last):
File "calibre\ebooks\metadata\sources\covers.py", line 46, in run
File "calibre_plugins.dummy1.init", line 227, in download_cover
File "calibre_plugins.dummy1.init", line 293, in identify
File "calibre_plugins.dummy1.init", line 61, in search_books
File "calibre_plugins.dummy1.init", line 45, in load_book_urls
File "urllib\request.py", line 216, in urlopen
File "urllib\request.py", line 525, in open
File "urllib\request.py", line 634, in http_response
File "urllib\request.py", line 557, in error
File "urllib\request.py", line 496, in _call_chain
File "urllib\request.py", line 749, in http_error_302
File "urllib\request.py", line 525, in open
File "urllib\request.py", line 634, in http_response
File "urllib\request.py", line 563, in error
File "urllib\request.py", line 496, in _call_chain
File "urllib\request.py", line 643, in http_error_default
urllib.error.HTTPError: HTTP Error 403: Forbidden

我也遇到了一样的问题

403是没有权限，一般都是访问多了被豆瓣拦了

已修复这个问题，获取书籍列表的地址被屏蔽了，改成直接用网页获取列表


Delete unused run-examples script

CI uses ./scripts/run-example.sh (singular)

ping @NicholasLYang @gsoltis since this has been open for a while now


Artefact with Rasters

When using GeoMakie with Rasters.jl, I have artifacts on the final figure.
There is a line continuing the value from -180 to 180. It happens with contourf function but also, and more surprising, with surface function.

Here is the code I'm using:
        val = ds[:,:,1]

        fig = Figure()

        ax = GeoAxis(fig[1,1]; aspect = DataAspect(), title=title, dest = "+proj=robin", coastlines = true)
        p = surface!(ax, val; colormap= :darktest, shading = false, colorrange=(-40,40),  interpolate=false)
        cb = Colorbar(fig[1, 2], p, label=string(product.unit))
        cb.alignmode = Mixed(right = 0)

        ax = GeoAxis(fig[2,1]; aspect = DataAspect(), title=title, dest = "+proj=robin", coastlines = true)
        p = contourf!(ax, val; colormap= :darktest, shading = false, colorrange=(-40,40), interpolate=false)
        cb = Colorbar(fig[2, 2], p, label=string(product.unit))
        cb.alignmode = Mixed(right = 0)

        resize_to_layout!(fig)
        save(filepath, fig)

I'm not sure it's the same issue than this ticket : https://github.com/MakieOrg/GeoMakie.jl/issues/119

Looks like something similar.  I'm not sure if this is still an issue with the current Float64 handling?


SCCP-2132: Reduce target staking ratio to 250%

Closing this as no longer relevant @dbeal-eth , let me know otherwise,


combine_javascript with multiple aliased domains causes JS exception

This is on the latest production version of ModPagespeed.

Oddly when I navigate to www.bimomotor.co.uk, there is a javascript error:

ReferenceError: mod_pagespeed_7QK8uOvs$9 is not defined
[Break On This Error]

No idea why this happens - as www.bimomotor.com and www.bimomotor.es both work
(and use exactly the same javascript!).

This only seems to have happened since 1st March.

I have tried clearing the modpagespeed cache completely, but to no effect.

I have had to use
ModPagespeedDisableFilters outline_javascript,rewrite_javascript

The setup in pagespeed.conf causing this issue was:

    # Turn on mod_pagespeed. To completely disable mod_pagespeed, you
    # can set this to "off".
    ModPagespeed on
    ModPagespeedDisallow            */jquery.tinymce.js
    ModPagespeedDisallow            */tiny_mce.js

    # We want VHosts to inherit global configuration.
    # If this is not included, they'll be independent (except for inherently
    # global options), at least for backwards compatibility.
    ModPagespeedInheritVHostConfig on

    # Direct Apache to send all HTML output to the mod_pagespeed
    # output handler.
    AddOutputFilterByType MOD_PAGESPEED_OUTPUT_FILTER text/html

    # Override the mod_pagespeed 'rewrite level'. The default level
    # "CoreFilters" uses a set of rewrite filters that are generally
    # safe for most web pages. Most sites should not need to change
    # this value and can instead fine-tune the configuration using the
    # ModPagespeedDisableFilters and ModPagespeedEnableFilters
    # directives, below. Valid values for ModPagespeedRewriteLevel are
    # PassThrough, CoreFilters and TestingCoreFilters.
    #
    ModPagespeedRewriteLevel CoreFilters

    ModPagespeedEnableFilters collapse_whitespace,outline_css,outline_javascript,rewrite_javascript
    ModPagespeedEnableFilters combine_css,rewrite_css,elide_attributes,rewrite_style_attributes
    ModPagespeedEnableFilters make_google_analytics_async,combine_javascript
    ModPagespeedEnableFilters move_css_above_scripts,remove_comments,sprite_images
    ModPagespeedEnableFilters inline_images,convert_jpeg_to_webp,resize_mobile_images
    ModPagespeedEnableFilters convert_jpeg_to_progressive,fallback_rewrite_css_urls
    ModPagespeedEnableFilters move_css_above_scripts,trim_urls
    ModPagespeedEnableFilters extend_cache
    # ModPagespeedEnableFilters insert_image_dimensions,inline_preview_images,defer_javascript

    ModPagespeedDomain www.bimomotor.*
    ModPagespeedDomain bimomotor.*

    ModPagespeedMapOriginDomain localhost *.bimomotor.*
    ModPagespeedMapOriginDomain localhost bimomotor.*
    ModPagespeedMapOriginDomain http://localhost https://www.bimomotor.*

    ModPagespeedLoadFromFileMatch "^https?://(?:www.)?bimomotor.*/themes/mails/" "/var/www/html/themes/mails/"
    ModPagespeedLoadFromFileMatch "^https?://(?:www.)?bimomotor.*/uploaded/img/" "/var/www/html/uploaded/img/"
    ModPagespeedLoadFromFileMatch "^https?://(?:www.)?bimomotor.*/themes/enuukGrey/css/" "/var/www/html/themes/enuukGrey/css/"
    ModPagespeedLoadFromFileMatch "^https?://(?:www.)?bimomotor.*/themes/enuukGrey/img/" "/var/www/html/themes/enuukGrey/img/"
    ModPagespeedLoadFromFileMatch "^https?://(?:www.)?bimomotor.*/themes/enuukGrey/js/" "/var/www/html/themes/default/js/"
    ModPagespeedLoadFromFileMatch "^https?://(?:www.)?bimomotor.*/themes/default/js/" "/var/www/html/themes/default/js/"

    ModPagespeedLoadFromFileMatch "^https?://<IP_ADDRESS>/.*themes/mails/" "/var/www/html/themes/mails/"
    ModPagespeedLoadFromFileMatch "^https?://<IP_ADDRESS>/.*uploaded/img/" "/var/www/html/uploaded/img/"
    ModPagespeedLoadFromFileMatch "^https?://<IP_ADDRESS>/.*themes/enuukGrey/css/" "/var/www/html/themes/enuukGrey/css/"
    ModPagespeedLoadFromFileMatch "^https?://<IP_ADDRESS>/.*themes/enuukGrey/img/" "/var/www/html/themes/enuukGrey/img/"
    ModPagespeedLoadFromFileMatch "^https?://<IP_ADDRESS>/.*themes/enuukGrey/js/" "/var/www/html/themes/default/js/"
    ModPagespeedLoadFromFileMatch "^https?://<IP_ADDRESS>/.*themes/default/js/" "/var/www/html/themes/default/js/"

    ModPagespeedLoadFromFileMatch "^http://(?:www.)?bimomotor.*/phpbb/styles/" "/var/www/html/phpbb/styles"
    ModPagespeedLoadFromFile "http://<IP_ADDRESS>/phpbb/styles/" "/var/www/html/phpbb/styles"

    ModPagespeedFetchWithGzip on
    SetOutputFilter DEFLATE

Original issue reported on code.google.com by rwap.services on 4 Mar 2013 at 11:12

I'm facing the same issue. It's been 3 and I assume it's not fixed, right?
My setup is the following A.com/b [dir /b proxy_pass to] -> B.com/ [this is where pagespeed is installed]
B.com authorizes both domains via pagespeed Domain directive. And here the result:
URL: https://a.com/b/somepage/

 <-- this id are not mentioned in above script
 <-- this id are not mentioned in above script

Same URL at second domain: https://b.com/somepage/

 <-- this id IS mentioned in above script
 <-- this id IS mentioned in above script

What is funny here:

2 generated scripts have different names (...XQoUShKijI.js,  ...Rxyn2edOtN.js) but they are identical
so, js placed at A.com has IDs from B.com

its worth to mention that sub_filter is running when A.com alias is used. Sub_filter modifies links b.com -> a.com/b

I can say for sure that the nginx sub_filter is the cause (not guilty though). Once disabled all ids of combine filter works as expected.

Maybe it’s a filter ordering issue. Which one runs last, the sub filter or the pagespeed filter? Tweaking the optimized output may break stuff

@oschaaf afaik there is no way to influence the order of output filters in nginx however I'm sure that sub_filter runs before pagespeed. What sub_filter does is simply changes href links from clk to a clk
Is there any hashing of html source involved in pagespeed? Or does it query for html page in background as it does for static resources?


How to run test on Windows Chrome in such a way that it directly allows download without showing Keep/Discard option in headless mode ?

OpenTest version number: 1.2.4
Running a test on windows chrome includes downloading an an .exe file. Chrome blocks downloaded .exe file by showing Keep/Discard. After  modifying chrome policy chrome allowed downloads & test passed.
But when we run the test in headless mode(adding --headless to actor.yaml) download is stuck.
Steps to reproduce:

Add '--headless' to parameter 'chromeDriverExeArgs' in actor.yaml
Run a test that downloads any .exe files(e.g. Whatsapp.exe for Desktop)

I think actor.yaml need some more argument like '--safebrowsing-disable-download-protection', '--safebrowsing-disable-download-protection'. But I tried all none seems to be working !

There's a lot of contradictory/outdated/confusing information out there about setting Chrome options and command-line switches. This really is not an OpenTest issue but one that is related to setting up Chrome for automation, so you might want to try looking for help on Stack Overflow and other similar forums. Based on the information I was able to find, here are some things you can try adding in your "actor.yaml" file (I also included the syntax for adding the safebrowsing-disable-download-protection just to make sure you are passing it the right way):
selenium:
  chromeExperimentalOptions:
    prefs:
      safebrowsing.enabled: false
      download.safebrowsing.enabled: false
      download.default_directory: C:\Downloads
      download.prompt_for_download: false
      download.directory_upgrade: true
  chromeDriverExeArgs: [--headless, --safebrowsing-disable-download-protection]

Please replace C:\Downloads with a path that exists on your machine.


Consignments Flow Feedback

Took a look at the submission flow – great work! I've got some feedback, feel free to break these into separate tickets.

Category field can't be selected, pressing "return" while in the previous field dismisses the keyboard.
Metadata view's done button is obscured by keyboard until dismissed.
Width/height keyboards should be number-keyboards by default.
The inches/centimetres switch changes colour based on the unit, which seems weird. cm looks "on" and in looks "off" – intentional?
Artist name field should autocapitalize each word.
Partially filled-in metadata appears empty from overview view (the plus sign is still there instead of a checkmark).
Text cursor on provenance view is not vertically centred with placeholder text.
Text-selection caret is very difficult to invoke on the provenance view, making cut/copy/paste difficult.
Pressing "Next" didn't do anything for me (on the most recent Emission beta)

closing in favor of a few specific issues left from the list, thanks for QA @ashfurrow


Add linter rules from serverless. Minor refactor

This PR includes the changes required to use the same linter rules than serverless/serverless. It also includes the required fixes to comply with the linter and a minor refactor for some common functions and follow the same style than serverless/serverless plugins.
It also removes the duplicated file located at remove/remove/kubelessRemove.js

LGTM except esLint I have no idea what it's used for

I am landing this PR for sending another one based on this. Let me know if you find something else and I will open an new one.


When I edit a listing, click save and then click view I get...

What version of the client are you on?
Does the listing this happens to have variants?
I assume you are waiting until the save completes before clicking view? (it should say "saved" in the bottom the bar and then you are free to nav away without warning)

client 2.0.14, server 0.9.3  I'm on OSX, I downloaded the binary about a week ago.
example.mov.zip
Is there a preferred way to post vids to github? It didn't seem to like me uploading a .mov file.

This has been fixed but hasn't made it into a release yet. Should be in the next one which should be soon. Please let us know if you are still seeing the issue.

Is there a way of getting pinged for each release that comes out? So I can ensure I'm always testing on the latest one?

If you're using the bundled app and you're on Windows or OSX, there's an auto-updater.
If you're running form source, then you just need to pull latest (in which case you'd have updates even before they are released - but you might also get bugs that may otherwise be caught before they make it into a release).


[Fix #7849] Add ExplicitOperatorPrecedence cop

Fixes #7849
This cop checks if binary operators of different precedents are used without explicit parenthesis.
# bad
a && b || c
a * b + c
a ** b * c / d % e + f - g << h >> i & j | k ^ l

# good
(a && b) || c
(a * b) + c
(((((a**b) * c / d % e) + f - g) << h >> i) & j) | k ^ l

@rahul404 I see the build is failing, so I assume this is still WIP. Let me know when it's ready for review.

@bbatsov, The build fails due to a test case I am not able to figure out.

Hi @bbatsov, I have made a PR for this feature. There is one issue I can fix in it.
The test case to fix a ** b * c / d % e + f - g << h >> i & j | k ^ l fails. Somehow the cop exercises the correction method only once.
https://github.com/rubocop-hq/rubocop/issues/7849#issuecomment-619625542

I'm closing this PR due to no recent activity. Feel free to re-open it if you ever come back to it.


Delegate precondition, fix SNARK failure, tests

The fix in #13043 for SNARK failures seen when new accounts have a delegate precondition wasn't quite right. In-SNARK, the account has an empty public key, so setting the delegate to that key is incorrect. The fix is to set the delegate to the account update's public key.
Remove the redundant code that conditionally sets the base_delegate based on whether the account is new, and the token is the default. The fix has already set the account delegate using the same criteria.
Add two tests using the delegate account precondition, one with the default token (the zkApp succeeds), another with a custom token (the zkApp gets the expected failure). Neither should trigger a SNARK failure.
Rename party_preconditions to zkapp_preconditions (while account_update_preconditions is more accurate, it's too long).
Closes #13040. Closes #13051.

!ci-build-me

!approved-for-mainnet


Porting "Out"

[Enter feedback here]
Some may have the expectation that Microsoft is able to move ported numbers out to other providers. We believe the correct answer is that a "port in" request would need to be filed with the other provider - that no provider pushes out numbers to other providers.
It would be helpful if this article made some mention of the process of porting numbers already ported to Microsoft.

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

ID: ca4cd82c-d4e0-fd46-a9d6-19c46e217b7a
Version Independent ID: d4a536ae-1f0d-fab5-4617-a414b829374a
Content: Manually submit a port order - Microsoft Teams
Content Source: Teams/phone-number-calling-plans/manually-submit-port-order.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Hello, @scapero how are you? that's answered here
https://learn.microsoft.com/en-us/MicrosoftTeams/phone-number-calling-plans/port-order-overview#can-i-port-out-my-numbers-from-teams-to-a-different-phone-service-provider-or-carrier


[Custom DC] Various setup improvements

Cluster

Migrate the cluster management from shell script to Terraform module. This allows Terraform to also delete the GKE cluster, 1 step closer for E2E test
Allow zonal cluster instead of enforcing regional clusters, which should cut the resource by 2/3. Location specifies

Helm chart

Allow gcr repo project to be specified, this is a follow up for @shifucun's support for building images in custom dc project
Move helm out of Terraform management. In the past few weeks, issue 1 is that it was easy to get into a circular situation where the state needs to be read at the beginning (reading what resource a cluster has requires kube config), but the kube config is yet to be fetched because it happens in the part after reading the states. May be cleaner to separate GCP resource level and k8s resource levels.
This does not affect GCP level resource lifecycle, as even if pods and services are in a cluster, the cluster can still be deleted.
Taking the k8s resources out of the Terraform lifecycle may make things easier when integrating with Argo, as Argo changes will not touch Terraform at all.

I used this to deploy to RFF instance, but I'm not confident enough without E2E testing so the version is not bumped yet.

@juliawu Also adding you as the reviewer, hope you can familiarize yourself with some of the changes happening in custom DC!


sql protocols 2.0

Hey @tanner0101, you just merged a pull request, have a coin!
You now have 751 coins.


Extend member_action? by checking for instance_name id

Currently only :id and :id_param are checked for member_action?. I think it makes sense to extend it by instance_name id.
Given the following url
/authors/1/books/1
the rails route might look like
/authors/:id/books/:book_id or /authors/:author_id/books/:book_id
In these cases the member_action? will return false for either one or both controllers, since it won't find the :id param. This results in not loading the instance for actions like show/edit when using authorize_resource. The solution here is to explicity set the id_param option. But from my point of view, this id naming scheme is quite common and should be considered when checking the id param.
This PR adds an id check for instance_name + _id, which would fullfil most of these cases.

I was also on the search for this. For me, this was not enough, I needed to also patch the id_param method of the CanCan::ControllerResourceLoader.
My quick and dirty monkeypatch looks the following way:
module PatchCanCanControllerResource
  def member_action?
    new_actions.include?(@params[:action].to_sym) || @options[:singleton] ||
      ((@params[:id] || @params["#{instance_name}_id"] || @params[@options[:id_param]]) &&
        !collection_actions.include?(@params[:action].to_sym))
  end
end

CanCan::ControllerResource.prepend PatchCanCanControllerResource

module PatchCanCanControllerResourceLoader
  def id_param
    return @params[id_param_key].to_s if @params[id_param_key].present?
    return @params["#{instance_name}_id"].to_s if @params["#{instance_name}_id"].present?
  end
end

CanCan::ControllerResourceLoader.prepend PatchCanCanControllerResourceLoader


#[pyo3(get)] much slower than python attribute access

I have the following setup:
// lib.rs
use pyo3::prelude::*;

#[pyclass]
struct RustClass {
    #[pyo3(get)]
    value: i32
}

#[pymethods]
impl RustClass {
    #[new]
    fn new() -> Self {
        RustClass{value: 0}
    }
}

/// A Python module implemented in Rust.
#[pymodule]
fn _lowlevel(_py: Python, m: &PyModule) -> PyResult<()> {
    m.add_class::<RustClass>()?;
    Ok(())
}

and
import timeit

from pyo3_bench._lowlevel import RustClass

class PyClass:
    def __init__(self):
        self.value = 0

def bench():
    py = PyClass()
    rust = RustClass()
    print("Python:")
    print(timeit.timeit(lambda: py.value, number=1000000))
    print("Rust:")
    print(timeit.timeit(lambda: rust.value, number=1000000))

When I run bench I find that the rust attribute access is much slower than the python direct access.
>>> bench()
Python:
0.06078508700011298
Rust:
0.6096129899960943
>>> bench()
Python:
0.05267535999882966
Rust:
0.6028135990200099

Its unclear whether this is intended or not.  Is there a good way for me to replicate the performance of python here?
If its useful:

macOS
Python 3.12
pyo3 0.21.2

When using pyo3(get) on a primitive, it will need to be turned into a Python object at each attribute access -- that is to say, there's a heap allocation for a new PyLong object each time.
A strictly apples-to-apples comparison would be a Py<SomeType> attribute.

Hmm I appreciate the suggestion but can't seem to get that to behave any differently.
I now have
#[pyclass]
struct RustClass {
    #[pyo3(get)]
    value: i32,

    #[pyo3(get)]
    pyvalue: PyObject,
}

and get pretty much the same result:
>>> bench()
Python:
0.057306197995785624
Rust value:
0.6657147670048289
Rust pyvalue:
0.6800711160176434

Perhaps I'm not understanding your suggestion correctly?

I attempted to reproduce this locally, and I find that the difference is much smaller than you're finding. Did you compile with --release?
$ python test.py
Python:
0.02138412299996162
Rust:
0.04930254800001421

PyO3 0.22 is expected to be faster still, I get numbers more like this:
$ python test.py
Python:
0.021781480999948144
Rust:
0.040761033000080715

Hmm I appreciate the suggestion but can't seem to get that to behave any differently.

I find the same as you; I think this is because constructing a Python zero from a Rust zero is also extremely cheap. But I still find the slowdown to be only 2x rather than 12x.
I'm going to do a quick bit of profiling and see if I can understand and close the gap...

Ok, so I did some experimentation and if I teach Python to read directly from a Py<PyAny> value using an ffi::PyMemberDef definition, then I get (unsurprisingly) performance which matches the Python lookup:
$ python test.py
Python:
0.02066981300004045
Rust:
0.03565813099976367
Rust (Py<PyAny>):
0.02015118600002097

I did the same trick for the Rust i32, but it's still slower than Python because it needs to create the int object. That's not a huge surprise and a natural cost of crossing the boundary.
I also think that this provides us a nice solution to the lack of Py::clone without the py-clone feature, so I'm going to see if I can turn this into a real implementation immediately. Between perf optimization and a usability improvement, seems like a clear win to get into 0.22 quickly.
I think we cannot apply the same optimization for #[pyo3(set)] because we have the borrow-checking to guard against write aliasing.

Ah - you're totally right - I wasn't passing --release.
>>> bench()
Python:
0.06525319800130092
Python property:
0.08756525401258841
Python fn:
0.08051640499616042
Rust value:
0.1070495129970368
Rust pyvalue:
0.11770369400619529

I'm much closer now.
Can you show me how you're using ffi::PyMemberDef - that's exactly what I'm looking for I think.
Appreciate all the help!

So PyMemberDef only works for a very specific set of types and only for read-only fields due to Rust's guarantees around mutable aliasing.
We don't really have a point in the #[pyclass] machinery where you can hook one in. Given the safety risks, I'd rather not make one either. Instead I'm working on a PR which would automatically choose it as the implementation when possible. I'm going to push this today.

I think with #4254 released in 0.22 this is essentially solved as much as it can be. There's maybe a short follow-up to add more types beyond Py<T>, e.g. i32, but that's a small task and I doubt will net much performance improvement.


Improve schema for dues history

Reticketed from https://github.com/gratipay/gratipay.com/commit/918cf40a8727680377103970d80ea55c8fbb582b#commitcomment-13966584

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27724464-improve-schema-for-dues-history?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

I put this on Finish Teams Basics per https://github.com/gratipay/gratipay.com/pull/3873#issuecomment-165448321.

Aaaaaaaand I'm taking it off again, because https://github.com/gratipay/gratipay.com/issues/3992#issuecomment-207852330 and https://github.com/gratipay/gratipay.com/issues/3705#issuecomment-207852318.


Open all DataChannels before accepting remote

A race existed with Negotiated DataChannels before this. Before the remote
could sends us a DataChannel message before the datachannel was
negotiated locally. We would then discard the message and print an
error.
This PR moves the processing of remote datachannel messages after we have
registered all local ones.

@edaniels is working on this instead.


updates for pipecat 0.0.49

Short summary of changes (please include issue #)

Manually implemented in jpt/clerk


Bitfinex example produces HTTP 500 error

I have the following code (virtually identical to the code in the bitfinex example)
import org.knowm.xchange.Exchange;
import org.knowm.xchange.ExchangeFactory;
import org.knowm.xchange.ExchangeSpecification;
import org.knowm.xchange.bitfinex.v1.BitfinexExchange;

public class MWE {
    public static void main(String[] args) {
        Exchange bfx = ExchangeFactory.INSTANCE.createExchange(BitfinexExchange.class.getName());
        ExchangeSpecification bfxSpec = bfx.getDefaultExchangeSpecification();

        bfxSpec.setApiKey(...);
        bfxSpec.setSecretKey(...);
        bfx.applySpecification(bfxSpec);
    }
}

Which produces the following error:
Exception in thread "main" org.knowm.xchange.exceptions.ExchangeException: si.mazi.rescu.HttpStatusIOException: HTTP status code was not OK: 500
	at org.knowm.xchange.BaseExchange.applySpecification(BaseExchange.java:117)
	at nl.frijt.adapter.orderservice.MWE.main(MWE.java:13)
Caused by: si.mazi.rescu.HttpStatusIOException: HTTP status code was not OK: 500
	at si.mazi.rescu.ResponseReader.read(ResponseReader.java:104)
	at si.mazi.rescu.RestInvocationHandler.mapInvocationResult(RestInvocationHandler.java:175)
	at si.mazi.rescu.RestInvocationHandler.receiveAndMap(RestInvocationHandler.java:163)
	at si.mazi.rescu.RestInvocationHandler.invoke(RestInvocationHandler.java:119)
	at com.sun.proxy.$Proxy15.accountFees(Unknown Source)
	at org.knowm.xchange.bitfinex.v1.service.BitfinexAccountServiceRaw.getAccountFees(BitfinexAccountServiceRaw.java:153)
	at org.knowm.xchange.bitfinex.v1.BitfinexExchange.remoteInit(BitfinexExchange.java:66)
	at org.knowm.xchange.BaseExchange.applySpecification(BaseExchange.java:115)
	... 1 more

I have checked the API key and secret I'm using with the bitfinex nodejs library and they are correct.
Is there something obvious I'm missing, or has Bitfinex stopped working for this library?

500 error codes suggests a problem at the exchange it self
please try to capture the HTTP request / response using a tool like Wireshark or https://github.com/knowm/XChange/wiki/New-Implementation-Best-Practices#step-3
and send it here and to Bitfinexes support

So after enabling debug messages, we came to the following before the exception:
2018-09-06 08:40:32,808 INFO  [main] [org.knowm.xchange.bitfinex.v1.BitfinexExchange] Calling Remote Init...  2018-09-06 08:40:32,816 DEBUG [main] [si.mazi.rescu.HttpTemplate] Executing GET request at https://api.bitfinex.com/v1/symbols  2018-09-06 08:40:33,029 DEBUG [main] [si.mazi.rescu.HttpTemplate] Request http status = 200  2018-09-06 08:40:33,051 DEBUG [main] [si.mazi.rescu.HttpTemplate] Executing GET request at https://api.bitfinex.com/v1/symbols_details  2018-09-06 08:40:38,201 DEBUG [main] [si.mazi.rescu.HttpTemplate] Request http status = 200  2018-09-06 08:40:38,270 DEBUG [main] [si.mazi.rescu.HttpTemplate] Executing POST request at https://api.bitfinex.com/v1/account_fees  2018-09-06 08:40:38,369 DEBUG [main] [si.mazi.rescu.HttpTemplate] Request http status = 500  2018-09-06 08:40:38,371 ERROR [main] [Starter] Exception caught: si.mazi.rescu.HttpStatusIOException: HTTP status code was not OK: 500 Stacktrace: org.knowm.xchange.exceptions.ExchangeException: si.mazi.rescu.HttpStatusIOException: HTTP status code was not OK: 500
This indicates that Bitfinex has problems with the https://api.bitfinex.com/v1/account_fees endpoint, which I cannot find in the authenticated endpoints of Bitfinex: https://docs.bitfinex.com/docs/rest-auth
Is it possible that they removed the endpoint? But it is weird that they would send 500 as response back for a not existing endpoint...

you need to use the trace level to see the responces content in the logs
https://github.com/mmazi/rescu#logging
did you report this to the exchange?

I have exactly the same issue and same error response when fetching account fees. Any suggestions?

I have exactly the same issue and same error response when fetching account fees. Any suggestions?
After enabling TRACE level on 'rescu' I got the following log stream:
22:33:07.633 [main] DEBUG si.mazi.rescu.HttpTemplate - Executing POST request at https://api.bitfinex.com/v1/account_fees 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Request body = {"request":"/v1/account_fees","nonce":"729166347"} 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Request headers = {X-BFX-PAYLOAD=eyJyZXF1ZXN0IjoiL3YxL2FjY291bnRfZmVlcyIsIm5vbmNlIjoiNzI5MTY2MzQ3In0=, Accept=application/json, X-BFX-APIKEY=M+HfJj, X-BFX-SIGNATURE=d70fc7f7d36afbe50801a7d0690dfab759dfb51367601e410c6519ade0e3ed4a918e9e1e30d888318f52f9a6b9d9784b, Content-Type=application/json} 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Header request property: key='Accept-Charset', value='UTF-8' 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Header request property: key='X-BFX-PAYLOAD', value='eyJyZXF1ZXN0IjoiL3YxL2FjY291bnRfZmVlcyIsIm5vbmNlIjoiNzI5MTY2MzQ3In0=' 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Header request property: key='Accept', value='application/json' 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Header request property: key='X-BFX-SIGNATURE', value='d70fc7f7d36afbe50801a7d0690dfab759dfb51367601e410c6519ade0e3ed4a918e9e1e30d888318f52f9a6b9d9784b' 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Header request property: key='User-Agent', value='ResCU JDK/6 AppleWebKit/535.7 Chrome/16.0.912.36 Safari/535.7' 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Header request property: key='X-BFX-APIKEY', value='++HfJj' 22:33:07.634 [main] TRACE si.mazi.rescu.HttpTemplate - Header request property: key='Content-Type', value='application/json' 22:33:07.732 [main] DEBUG si.mazi.rescu.HttpTemplate - Request http status = 500 22:33:07.732 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='Strict-Transport-Security', value='[max-age=31536000]' 22:33:07.732 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='CF-RAY', value='[468419985dae6f24-FCO]' 22:33:07.732 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='Server', value='[cloudflare]' 22:33:07.732 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='X-Request-Id', value='[8940d8e6-e55e-491e-bb60-5e86faf9699f]' 22:33:07.732 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='X-Runtime', value='[0.007401]' 22:33:07.733 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='Connection', value='[keep-alive]' 22:33:07.733 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='Set-Cookie', value='[__cfduid=d90d830a2c0abad81ab27e215bf2b06761539289987; expires=Fri, 11-Oct-19 20:33:07 GMT; path=/; domain=.bitfinex.com; HttpOnly]' 22:33:07.733 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='Content-Length', value='[48]' 22:33:07.733 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='Date', value='[Thu, 11 Oct 2018 20:33:07 GMT]' 22:33:07.733 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='Expect-CT', value='[max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"]' 22:33:07.733 [main] TRACE si.mazi.rescu.HttpTemplate - Header response property: key='Content-Type', value='[application/json; charset=utf-8]' 22:33:07.733 [main] TRACE si.mazi.rescu.HttpTemplate - Http call returned 500; response body: {"status":"500","error":"Internal Server Error"}

I have exactly the same issue and same error response when fetching account fees. Any suggestions?
After enabling TRACE level on 'rescu' I got the same log stream:

23:22:36.171 [main] DEBUG si.mazi.rescu.HttpTemplate - Executing POST request at https://api.bitfinex.com/v1/account_fees
23:22:36.172 [main] TRACE si.mazi.rescu.HttpTemplate - Request body = {"request":"/v1/account_fees","nonce":"729178219"}
...
[main] TRACE si.mazi.rescu.HttpTemplate - Http call returned 500; response body: {"status":"500","error":"Internal Server Error"}

5xx errors indicate that there is a problem with the exchange, not the client
did any one send the request / response to Bitfinexes support?

I did. I am waiting for their response.
Customer support anyway suggested me to do not throw too fast requests to their exchange. I suspect that the call to

https://api.bitfinex.com/v1/symbols_details

followed immediately by

https://api.bitfinex.com/v1/account_fees

is causing troubles...