Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter iqontrol for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.iqontrol
   -
ERRORS:

[ ] :heavy_exclamation_mark: [E157] js-controller 1.4.2 listed as dependency but 4.0.24 is required as minimum, 5.0.19 is recommended. Please update dependency at io-package.json.
[ ] :heavy_exclamation_mark: [E160] "admin" is not allowed in common.dependencies
[ ] :heavy_exclamation_mark: [E184] "common.subscribe" will be removed with js-controller >= 6. Please remove from io-package.json and adapt code if required.

WARNINGS:

[ ] :eyes: [W127] Missing suggested translation into uk of "common.titleLang" in io-package.json.
[ ] :eyes: [W134] Missing suggested translation into uk of "common.desc" in io-package.json.
[ ] :eyes: [W154] Missing suggested translation into uk of some "common.news" in io-package.json.
[ ] :eyes: [W172] "common.localLink" in io-package.json is deprecated. Please define object "common.localLinks": { "_default": "..." }
[ ] :eyes: [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation"
[ ] :eyes: [W184] "common.main" is deprecated and ignored. Please remove from io-package.json. Executable is defined by entry "main" at package.json.
[ ] :eyes: [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
[ ] :eyes: [W184] "common.materializeTab" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
[ ] :eyes: [W184] "common.title" is deprecated and replaced by "common.titleLang". Please remove from io-package.json.
[ ] :eyes: [W505] setTimeout found in "main.js", but no clearTimeout detected

SUGGESTIONS:

[ ] :pushpin: [S522] Please consider migrating to admin 5 UI (jsonConfig).
[ ] :pushpin: [S526] Consider adding plugin "@alcalzone/release-script-plugin-manual-review".

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check or force the creation of a new issue at any time by adding the following comment to this issue:
@iobroker-bot recheck
or
@iobroker-bot recreate
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
Note: This issue replaces issue #320
@mcm1957 for evidence
Last update at Sun, 20 Oct 2024 19:38:38 GMT based on commit f2e5a4686b8ea4b4913410058a5b3c524d53bc7b
ioBroker.repochecker 3.1.3

@iobroker-bot recheck

@ioBroker-Bot recreate


[Feature request] Need to set tilt programmatically.

Platforms
Android, iOS
Description
Hi, Thank you for your awesome plugin. That's amazing animation.
However, I need to know Can I set tilt by not from user interaction? but from code to animate widget.
Thank in advanced.
Why
In case need to animate ifself for interval duration to tilt widget over and over again.
Also, Can we add depth to make it look like it has more dimension?

Thanks for your feature suggestions! 😀
Now that a new version has been released.
It provides the TiltStreamController.
# pubspec.yaml

dependencies:
  flutter_tilt: ^2.4.0-beta.2

Here is an example for you.
TiltStreamController
You can use it through animation or otherwise.

Live Demo: https://amoshuke.github.io/flutter_tilt_book/#/TiltStreamControllerDemo
Document: https://github.com/fluttercandies/flutter_tilt?tab=readme-ov-file#streamcontrollertiltstreammodel-

Code
Click to view code
import 'dart:async';
import 'dart:math' as math;
import 'package:flutter/material.dart';

import 'package:flutter_tilt/flutter_tilt.dart';

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

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

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const Scaffold(body: TiltStreamControllerDemo()),
    );
  }
}

class TiltStreamControllerDemo extends StatefulWidget {
  const TiltStreamControllerDemo({super.key});

  @override
  State<TiltStreamControllerDemo> createState() =>
      _TiltStreamControllerDemoState();
}

class _TiltStreamControllerDemoState extends State<TiltStreamControllerDemo>
    with TickerProviderStateMixin {
  final MaterialStateProperty<Icon?> thumbIcon =
      MaterialStateProperty.resolveWith<Icon?>(
    (Set<MaterialState> states) {
      return states.contains(MaterialState.selected)
          ? const Icon(Icons.link)
          : const Icon(Icons.link_off);
    },
  );
  late StreamController<TiltStreamModel> tiltStreamController;
  bool controllerBind = true;
  AnimationController? animationSelected;

  /// AnimationController around
  late AnimationController aroundAnimationController;

  /// AnimationController leftTop -> rightBottom
  late AnimationController ltrbAnimationController;

  /// AnimationController rightTop -> leftBottom
  late AnimationController rtlbAnimationController;

  @override
  void initState() {
    super.initState();
    tiltStreamController = StreamController.broadcast();

    aroundAnimation();
    ltrbAnimation();
    rtlbAnimation();
    animationSelected = aroundAnimationController;
    animationSelected?.repeat(reverse: false);
  }

  @override
  void dispose() {
    tiltStreamController.close();
    aroundAnimationController.dispose();
    ltrbAnimationController.dispose();
    rtlbAnimationController.dispose();
    super.dispose();
  }

  void stopAllAnimation() {
    aroundAnimationController.stop();
    ltrbAnimationController.stop();
    rtlbAnimationController.stop();
  }

  /// Animation around
  void aroundAnimation() {
    aroundAnimationController = AnimationController(
      lowerBound: 0.0,
      upperBound: 1.0,
      duration: const Duration(milliseconds: 3000),
      vsync: this,
    );
    aroundAnimationController.addListener(() {
      final double radian =
          aroundAnimationController.value * 360 * math.pi / 180;
      const double r = 75;
      // P = (cosθ, sinθ)
      final double x = math.cos(radian) * r + 150;
      final double y = math.sin(radian) * r + 75;
      tiltStreamController.add(TiltStreamModel(position: Offset(x, y)));
    });
  }

  /// Animation leftTop -> rightBottom
  void ltrbAnimation() {
    ltrbAnimationController = AnimationController(
      lowerBound: 0.0,
      upperBound: 1.0,
      duration: const Duration(milliseconds: 3000),
      vsync: this,
    );
    ltrbAnimationController.addListener(() {
      final double x = ltrbAnimationController.value * 300;
      final double y = ltrbAnimationController.value * 150;
      tiltStreamController.add(TiltStreamModel(position: Offset(x, y)));
    });
  }

  /// Animation rightTop -> leftBottom
  void rtlbAnimation() {
    rtlbAnimationController = AnimationController(
      lowerBound: 0.0,
      upperBound: 1.0,
      duration: const Duration(milliseconds: 3000),
      vsync: this,
    );
    rtlbAnimationController.addListener(() {
      final double x = (1 - rtlbAnimationController.value) * 300;
      final double y = (rtlbAnimationController.value) * 150;
      tiltStreamController.add(TiltStreamModel(position: Offset(x, y)));
    });
  }

  @override
  Widget build(BuildContext context) {
    return Column(
      children: [
        /// Tilt here
        Tilt(
          tiltStreamController: controllerBind ? tiltStreamController : null,
          borderRadius: BorderRadius.circular(30),
          tiltConfig: const TiltConfig(enableGestureSensors: false),
          childLayout: const ChildLayout(
            outer: [
              Positioned(
                child: TiltParallax(
                  size: Offset(10, 10),
                  child: Text(
                    'Flutter Tilt 1 ✨',
                    style: TextStyle(
                      fontSize: 14,
                      color: Colors.white,
                    ),
                  ),
                ),
              ),
            ],
          ),
          child: Container(
            width: 300,
            height: 150,
            decoration: const BoxDecoration(
              gradient: LinearGradient(
                begin: Alignment.topRight,
                end: Alignment.bottomLeft,
                colors: [Color(0xFF80d0c7), Color(0xFF13547a)],
              ),
            ),
          ),
        ),

        /// bind
        Padding(
          padding: const EdgeInsets.all(20),
          child: Switch(
            thumbIcon: thumbIcon,
            value: controllerBind,
            onChanged: (value) {
              setState(() {
                controllerBind = value;
              });
            },
          ),
        ),

        /// Tilt here
        Tilt(
          tiltStreamController: controllerBind ? tiltStreamController : null,
          borderRadius: BorderRadius.circular(30),
          tiltConfig: const TiltConfig(enableGestureSensors: false),
          childLayout: const ChildLayout(
            outer: [
              Positioned(
                child: TiltParallax(
                  size: Offset(10, 10),
                  child: Text(
                    'Flutter Tilt 2 ✨',
                    style: TextStyle(
                      fontSize: 14,
                      color: Colors.white,
                    ),
                  ),
                ),
              ),
            ],
          ),
          child: Container(
            width: 300,
            height: 150,
            decoration: const BoxDecoration(
              gradient: LinearGradient(
                begin: Alignment.topCenter,
                end: Alignment.bottomCenter,
                colors: [Color(0xFF8989ba), Color(0xFFa7a6cb)],
              ),
            ),
          ),
        ),

        /// Tools
        /// Animation
        const Text(
          "Animation",
          style: TextStyle(
            fontSize: 16,
            fontWeight: FontWeight.bold,
          ),
        ),
        const SizedBox(height: 12),
        Wrap(
          spacing: 12,
          runSpacing: 12,
          children: [
            FilterChip(
              label: const Text('none'),
              selected: animationSelected == null,
              onSelected: (bool value) {
                setState(() {
                  if (value) {
                    stopAllAnimation();
                    animationSelected = null;
                  }
                });
              },
            ),
            FilterChip(
              label: const Text('around'),
              selected: animationSelected == aroundAnimationController,
              onSelected: (bool value) {
                setState(() {
                  if (value) {
                    animationSelected = aroundAnimationController
                      ..repeat(reverse: false);
                  } else {
                    stopAllAnimation();
                  }
                });
              },
            ),
            FilterChip(
              label: const Text('leftTop -> rightBottom'),
              selected: animationSelected == ltrbAnimationController,
              onSelected: (bool value) {
                setState(() {
                  if (value) {
                    animationSelected = ltrbAnimationController
                      ..repeat(reverse: true);
                  } else {
                    stopAllAnimation();
                  }
                });
              },
            ),
            FilterChip(
              label: const Text('rightTop -> leftBottom'),
              selected: animationSelected == rtlbAnimationController,
              onSelected: (bool value) {
                setState(() {
                  if (value) {
                    animationSelected = rtlbAnimationController
                      ..repeat(reverse: true);
                  } else {
                    stopAllAnimation();
                  }
                });
              },
            ),
          ],
        ),
      ],
    );
  }
}

Video
https://github.com/fluttercandies/flutter_tilt/assets/32262985/72e134ac-d69b-4fc5-8325-f7f8b054b268
About Depth
The current package mainly applies the relevant effects in 2 dimensions and does not consider 3 dimensions for the time being.

For example, transforming from a card to a box.

@AmosHuKe That's awesome. Thanks for your effort. 😄👍


CORE-6082 named query integration into flows

https://github.com/corda/corda-api/pull/527

I objected on the matching Corda-api pull request to the partial renaming to American spellings, so that same comment applies here to all the changes that follow from the rename.

@dickon I've removed the ized change from the API and runtime PRs. Will deal with it separately as its a more general thing.


Error message - not taking the user back out the page

From our accessibility testing it has been reported that we need to improve error messages on the page where the error occurs and not take the user back out the page. Also the error message should have improved the colour contrast which relates to #1457 and we should add format restrictions to fields - e.g. telephone numbers.

Going to park this as I am not sure about where this is required.

This is a major change and would require rethinking our error  process. At least a weeks worth of work. Might make sense to handle when we move to rails 5. @xsrust can add more details here.


Automated cherry pick of #8487: fix: avoid delete public ip when sync eip list

Cherry pick of #8487 on release/3.5.
#8487: fix: avoid delete public ip when sync eip list

/lgtm
/approve


openlane(gf180): update config file

disable ir drop report
update padding variable names
set min global routing layer to Metal2 to avoid pin access issues at drt
update PLACE_SITE for GF018hv5v_mcu_sc7

Pulled and merged.


[BUG]

Please fix rotations
if i use silent alternative on future and 3arth surround it desync / cant place
lol, weird

where are your checkboxes, logs, screenshots etc?? pls follow the issue template

its kinda unfixable without breaking future in the process. is this with the compatibility module on?

Please fix rotations
if i use silent alternative on future and 3arth surround it desync / cant place
lol, weird

user issue nigger


Enable object variables and ForEach menu item

For instance:
{
  "variables": [
    {
      "name": "DTAP",
      "value": [
        {
          "key": "D",
          "url": "https://develop.local"
        },
        {
          "key": "T",
          "url": "https://test.local"
        },
        {
          "key": "A",
          "url": "https://acceptance.local"
        },
        {
          "key": "p",
          "url": "https://production.local"
        }
      ]
    }
  ],

  "repository-actions": {
    "actions": [
      {
        "type": "foreach@1",
        "loop": "{var.DTAP}",
        "variable": "environment",
        "actions": [
          {
            "type": "browser@1",
            "name": "{var.environment.key}",
            "url": "{var.environment.url}"
          }
        ]
      }
    ]
  }
}

or quivalent in yml:
variables:
- name: DTAP
  value:
  - key: D
    url: https://develop.local
  - key: T
    url: https://test.local
  - key: A
    url: https://acceptance.local
  - key: p
    url: https://production.local

repository-actions:
  actions:
  - type: foreach@1
    loop: '{var.DTAP}'
    variable: environment
    actions:
    - type: browser@1
      name: '{var.environment.key}'
      url: '{var.environment.url}'

Implementes #34  and #35

Inline variables doesn't work


Whisper.cpp and Faster Whisper models cannot be downloaded

Hello.
I can't download the model from Preferences, after clicking download, buzz will flash back directly without error message.
Directly import the file and select
Whisper.cpp
or
Faster Whisper
After clicking rom, almost all models will flash back directly.
Only when you choose
Faster Whisper
inside the (medium) model in order to download successfully.
Also, if they have a new version of the model, can I also update the model from buzz?
Thanks!

Another question, can buzz be accelerated by GPU?

Are you on Windows 10? Also what's the output of your logs at C:\Users\<username>\AppData\Local\Buzz\Buzz\Logs?

This is the log
logs.txt

I am windows 10 22H2


Languages in tabs of change form from PageAdmin's add_view come from request rather than Page site ID

In update_language_tab_context, called from add_view, the obj is always None, and does not exist for languages to be determined by a site_id. Instead the tabs are populated from the languages of the site the request is from:
language = get_language_from_request(request, obj)
languages = self._get_site_languages(obj)

This is causing form validation errors without displayed messages when trying to create a page for a site that does not share any languages with the site originating the requests.

Hello @nolsto,
Thank you very much for your contribution to the django CMS project. The django CMS project has recently joined forces under the banner of the newly founded django CMS Association to revive the development of django CMS. For this reason we have analyzed all pull requests and issues in detail and decided which ones fit the product vision we want to go with django CMS.
Due to the age of this issue we are unable to safely accept the contribution. If you feel this issue is still required please inform a member of our technical committee<EMAIL_ADDRESS>Nevertheless we thank you for your participation and would like to emphasize that we do not want to discourage you in your contribution to the django CMS project. On the contrary, we encourage everyone who is interested in the success of django CMS to join us!
If you want to learn more about the work of the technical committee, our product vision and how you can become a member of the dCA, you can find all information here:

The Technical Committee of the django CMS Association
Technical Roadmap and Product Vision for django CMS
How to become a member of the django CMS Association

We are looking forward to every companion who would like to shape the future of django CMS together with us!
If you have any questions, feel free to reach out to Nicolai, our Community Manager<EMAIL_ADDRESS>Sign up for the Association now!


spec: add seccomp isolator for Linux

It would be nice to expose seccomp isolators to allow apps/pods running on Linux to restrict the permitted set of system calls.
http://man7.org/linux/man-pages/man2/seccomp.2.html
https://en.wikipedia.org/wiki/Seccomp

Should we make use of the syscall groups on those issues?

https://github.com/systemd/systemd/issues/3053
https://github.com/systemd/systemd/pull/3157

xref https://github.com/kubernetes/kubernetes/pull/24602
/cc @lucab @jfrazelle

I drafted up an initial proposal for seccomp support in #620, feel free to comment there. It already allows for custom groups, under scoped namespaces.

Moved to proposal stage, spec and code now at #621.


Option for email notifications to use common sender email address

Feature Description
Gitea currently sends notification emails with the From header set to the email address of the user who triggered the notification - e.g. the user who created an issue. It would be useful if there was an option to disable this so that the email notifications would instead be sent from the same email address that's used for general emails - the one in ENVELOPE_FROM / FROM.
Currently, I can't use the email account that I should be using because it is only allowed to send email as one generic address (like<EMAIL_ADDRESS>and is not allowed to send email as arbitrary users.
Screenshots
No response

"currently" is used incorrectly in the first paragraph.
Also the title, maybe:
"Support sending notification emails as the person who made the change"

"currently" is used incorrectly in the first paragraph.
Also the title, maybe: "Support sending notification emails as the person who made the change"

This issue was created quite some time ago - specifically the 6th of May, 2022!
I believe that my description is (or, at least, was) accurate - it reflects what was seen in the Gitea instance I was using at the time I created the issue. I imagine that version is probably considerably out-of-date by now. I perhaps should have specified the version that was being used at the time and/or began "Currently our Gitea instance" (etc.).
I have admittedly not tested the notification email behaviour in a newer version of Gitea since creating this issue (priorities have changed, Gitea is being used less etc.).
To (hopefully) clarify, what you appear to be suggesting in your title is - or, at least, was - the behaviour at the time of creating this issue. The desired functionality was the opposite - i.e., to be able to not have the email sent as the person who made the change and instead send as the common sender email address that is used for non-notification emails (regardless of who made a change or otherwise triggered a notification).
I'd be interested to know if the ability to send notification emails as the common sender email address does exist in a version of Gitea that was released since I originally created this issue though, thanks!

Currently the notification emails are sent from a unique email address. (In my team we want the opposite)

The original description of the issue is confusing, the two paragraphs are contradicting each other.


Explain the difference between L1 and L2

There is a lot of confusion in the community about what should be called an L2. The intention of l2beat.com is to provide transparent and verifiable info about the state of "L2" for Ethereum L1 chain. Other blockchains (e.g. Solana, Avalanche, Fantom, BSC, Polychain, etc...) are L1s. The main page of l2beat.com should explain the approach and the difference clearly

Closing as this is reflected in the FAQ


ask users about renewals notifications

fixes issue #25
Have a look in particular at /service-patterns/concessionary-travel/example-service/renewal-notification, and the page which links to that which is /service-patterns/concessionary-travel/example-service/photo/success

Hey @kateiw on this page:
/service-patterns/concessionary-travel/example-service/renewal-notification
I think if the user selects the "Letter (send to billing address)" option we should show the billing address in a dropdown?
I think it might need some fancy javascript but as I'm pairing with Ralph this week I think he should be able to help.


FAB missed icon and became transparent

This bug shows periodically:

Hi, can you post your code and the com.android.support:design version?

"com.android.support:appcompat-v7:25.3.1"
<FloatingActionButton
     rippleEffect={true}
     icon={'add-shopping-cart'}
     iconProvider={Icon}
     autoAnchor
     iconColor={Constants.color.WHITE}
     onPress={() => {}}
     backgroundColor={Constants.color.PRIMARY}/>

if you're using BottomSheetHeader, you might missing the backgroundColorExpanded prop, am i right? if you're not using it, what you mean with periodically? when exactly the FAB turns transparent?

I don't use BottomSheetHeader here. Sometimes FAB turns transparent just after I open application, and it become normal after I reopen app.

I am unaware of any issue like this to be honest, so if you find any pattern that you are sure that will always occurs this issue, or any race condition or provide a sample that i could easily reproduce, i would really appreciate to take a look at it.
Thanks.


SQL Server Endpoint string in Resources list is unusable.

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

When a SQL Server reports something along the lines of tcp://localhost:1234 it is accurately indicating information about the sever, but it is not indicating valuable information to the developer because connection to the server requires the IP address and a comma, like <IP_ADDRESS>,1234 which they would use in SSMS, or ADS, or VSCode, or Visual Studio SQL Server Object Explorer. Should they instead use the provided string, to novice developers this will appear correct, but will fail on them every single time.
When this is a web site or web API, the text becomes a link which can be clicked. How amazing would it be if this became a link here too - but for SQL server launching any of the tools or a simple web interface to interrogate the schema. Or even set this value programmatically when I create a SqlServer in AppHost. Sorry. I realize I just leaned into a second or even third issue. Please disregard this paragraph.
Solution

Show the IP address.
Replace the colon with a comma.

Goal
Make it so developers have it easier.
Expected Behavior
No response
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version info
No response
Anything else?
No response

AFAIK this is the data that we get from DCP for the container, and don't know whether its SQL or some other component. It will probably need coordination between:

App model to know the specifics about how an endpoint should be presented to the user for the specific service
DCP to supply that information to the dashboard via the gRPC interface
Dashboard to display it.

@mitchdenny @karolz-ms @drewnoakes  for their respective parts of the problem.

The resource service exposes a set of Urls with the following properties:
https://github.com/dotnet/aspire/blob/df935f4ee2e76d088235ea0fe91a0b61405cb3fe/src/Aspire.Hosting/Dashboard/proto/resource_service.proto#L113-L121
The protocol, as written, assumes that the full_url value is a valid RFC 1738 URL.
We may be able to provide a "display value" for these URLs, where some component close to DCP or the app model produces that optional display value when a different, non-URL form is required.

DCP does not really "know" it is dealing with a SQL Server. It only understands fairly low-level objects: containers, executables, IP addresses, ports, and the like.
What does know is the SQL Server Aspire hosting package. It could collaborate with the dashboard to turn the raw (IP address:port) information into something more useful. For example, we could introduce a notion of "endpoint formatter annotation" to the Aspire model, with various formatters for different types of resources. For SQL Server, a SQL Server endpoint formatter would be able  to turn raw SQL Server endpoint into a database connection string, or URL to administrative interface. And the dashboard protocol could be enhanced to decorate vanilla endpoints with these other ways of display endpoint information (produced by endpoint formatters) and expose them to users. Waving my hands a lot here, but hopefully pointing in the right direction.

@drewnoakes this is actually an example of a context-menu extensibility scenario where someone might want to copy the SQLMS connection string from the row in the dashboard. I wish there was a good defacto web-based replacement for SQLMS that we could just host in a container like we do with PgAdmin etc

Waiting on https://github.com/dotnet/aspire/pull/5380 so we can show secrets in properties


error when using pykml with python 3

just tried using pykml in a  python 3 project and got this error when importing it
File "/usr/local/lib/python3.5/dist-packages/pykml/factory.py", line 216
    print write_python_script_for_kml_document(doc)
                                            ^ SyntaxError: invalid syntax

the ^ that points to the syntax error is probably wrong cause i had to copy this from an escaped string, and some indentation got lost
i couldnt find anything about that issue, so lets hope im not just being stupid (pycharm also complains about that part in the code)

i just noticed this is a fork... my bad

wait... is this the official one or not?

wait... is this the official one or not?

No. I put it here from a conversion from (IIRC) mercurial project on code.google.com back when it was alive, and fixed a few bugs. I haven't touched it for some years, so py3k support is deeply unlikely unless someone volunteers :-)
Your error appears to be with py3 requiring print(...)as a function rather than theprint ...` it has there. I daresay there will be other places that need fixing too though

yeah i guessed that its the missing ()
well, i just have to write an xml string to a kml file. will go with formatting a string then. yay deadlines


Removed empty captcha directory and file

It is a mistake? Or it is on purpose?

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

I have accidentally merged it from 2.0


Array of objects support

Done in https://github.com/monke-systems/monke-turbo-config/commit/1b89bc9bc9cbdc82558653b832f4d372ac7c6fac


HHH-16035 add tests demoing bugs in HQL Duration support

Some of these issues probably arose with the change to use SQL interval types to store Duration, but some of them might have pre-existed that, since we just did not have very good tests here.

@beikov I'm trying to kill PostgreSQLIntervalSecondJdbcType. Please speak up if you really want to save its life.

Yeah, please don't remove it. I want to allow people to use that DDL type.

But it's untested, unused, and is causing bugs.
How about I remove it, and when you've fixed the bugs, you can put it back in?

Oh also: I'm not sure, but I think you need to reimplement it without the use of reflection, because I bet that reflecting on JDBC driver classes is going to cause problems in Quarkus. Is that right @Sanne?

Oh also: I'm not sure, but I think you need to reimplement it without the use of reflection, because I bet that reflecting on JDBC driver classes is going to cause problems in Quarkus. Is that right @Sanne?

Well, perhaps it simply gets disabled in Quarkus. (Its use is protected by a call to PostgreSQLPGObjectJdbcType.isUsable().) But that's all the more reason to dislike this thing.

@beikov alternatively, I could change the default value of hibernate.type.preferred_duration_jdbc_type to NUMERIC.
This would fix the bug I'm most worried about on both h2 and Postgres. The cost would be that Duration would not be stored as interval second on h2. But that seems completely fine to me. People usually use h2 for testing, and so it's not terribly useful that we store Durations as numeric on every other database, but then as interval second on h2. Make it less useful as a platform for tests.
(I'm fine with either solution.)

Right we should generally avoid reflection, however if there's compelling need for it it can be handled, just needs some extra steps. The most important being to keep track of it, as otherwise the missing metadata causes really puzzling hidden problems: at least being aware that it might be related to a new use of reflection helps a lot.
The Hibernate ORM module hibernate-graalvm can be used to define such metadata.
Bear in mind also: any use of reflection which is happening in a static initializer, or generally happens before the SessionFactory is started (e.g. during the constructor of the Dialect) is not a problem and should work w/o any further concern.

But it's untested, unused, and is causing bugs.
How about I remove it, and when you've fixed the bugs, and written tests for it, you can put it back in?

I like that approach - if a new features has no tests or documentation it's not ready.

if there's compelling need for it it can be handled, just needs some extra steps

Yeah, I don't see this one as compelling at all.

any use of reflection which is happening in a static initializer, or generally happens before the SessionFactory is started (e.g. during the constructor of the Dialect) is not a problem and should work w/o any further concern.

Here the reflection lookups happen at startup, but the use of reflection happens at runtime, so I think it counts as a concern.
Actually, now that you mention this, it looks to me like what would happen is that:

the reflection lookups would succeed at build-time, and so the Postgres Dialect would decide to use all this reflective stuff, but then
it would fail to work at runtime, where reflection is unavailable.

But I have not actually tested this to see if it's true.

@beikov alternatively, I could change the default value of hibernate.type.preferred_duration_jdbc_type to NUMERIC.

Here's that alternative: https://github.com/hibernate/hibernate-orm/pull/5951

it would fail to work at runtime, where reflection is unavailable.

Which reflection at runtime are you referring to? PostgreSQLIntervalSecondJdbcType sets up all the reflective parts in a static block, so this should be fine also for Quarkus.
Also see

Bear in mind also: any use of reflection which is happening in a static initializer, or generally happens before the SessionFactory is started (e.g. during the constructor of the Dialect) is not a problem and should work w/o any further concern.

There is some documentation that talks about this:

https://docs.jboss.org/hibernate/orm/6.2/userguide/html_single/Hibernate_User_Guide.html#_duration
https://docs.jboss.org/hibernate/orm/6.2/userguide/html_single/Hibernate_User_Guide.html#hql-duration-literals
https://docs.jboss.org/hibernate/orm/6.2/userguide/html_single/Hibernate_User_Guide.html#basic-legacy-provided

and also the migration guide: https://github.com/hibernate/hibernate-orm/blob/6.0/migration-guide.adoc#instant-mapping-changes
If two of the new features don't work together well in some scenarios, then how about we fix that instead of removing one feature? I'm on PTO now for a week, but this sounds like a bug that we can fix. No reason to remove features that were introduced in 6.0 now, literally hours before doing the 6.2 final release.

sets up all the reflective parts in a static block, so this should be fine also for Quarkus.

It sets them up in a static, but it calls them at runtime. What are the rules here: am I allowed to call Method objects at runtime if I've retrieved them at build time. I'm not sure of the answer to that. I thought I was not allowed to do that but I may simply be wrong and also I may just be way out of date. @Sanne?

There is some documentation that talks about this:

Not sure what your point is. The only thing those docs say are:

By default, Hibernate will map Duration to the INTERVAL_SECOND SQL type and fallback to NUMERIC if necessary.

In fact, the only database where Hibernate maps Duration to the SQL type INTERVAL SECOND is h2, so this is almost verging on misleading.

If two of the new features don't work together well in some scenarios, then how about we fix that instead of removing one feature?

Fine, if you prefer, I can go with the proposed "alternative" solution instead of removing this class. Like I said: I'm fine either way.

It sets them up in a static, but it calls them at runtime. What are the rules here: am I allowed to call Method objects at runtime if I've retrieved them at build time. I'm not sure of the answer to that. I thought I was not allowed to do that but I may simply be wrong and also I may just be way out of date. @Sanne?

FTR I asked Sanne via backchannel and he's not sure either. He thinks it's more likely to work via VarHandle but we would have to test it.

Closing in favor of #5951.


Add bypass parser

This branch adds in a complete parser that parses any statement as only BypassParts. There are some changes from the initial skeleton of the parser that existed in main already. There is a new struct, ParsedExprBuilder, that factors out the slice of parsed parts from the Parser itself. This means the checkpoint does not have to make copies of this slice when save() is called.
The behaviour of the add function is retained from the experiment. When add(p) is called the BypassPart preceding p is also added to the slice of parts. The ParsedExprBuilder keeps track of the relevant markers to make this possible.

Admittedly the rational behind these design changes should have been more clearly explained in this PR. The benefits are not apparent with only the Bypass parser present.
The experiment did not include ParsedExprBuilder. That is the main change introduced in this PR. The ParsedExprBuilder separates the aggregation of parts from the checkpoint functionality, primarily to aid clarity.
In the full parser we plan to have a separate, complete, parsing functions for input and output parts. This functionality will mirror the experiment. These functions attempt to parse an InputPart/OutputPart. If they fail they restore the checkpoint, thus consuming no input. If they succeed, then, with the current DSL specification, there is no proceeding input that could cause that parse to be rolled back. Therefore, there is no reason for the parts to be stored in the checkpoint.
The ParsedExprBuilder is introduced to store this information about the parts desperately from the Parser. It is then not included in the checkpoint. We can continue to use the checkpoint as it has been used in the experiment. For the reasons in the paragraph above, it does not affect its functionality.
The ParsedExprBuilder also stores the information about the start and end of the previous chunk of text to bypass. And the checkpoint is reduced to only its essential parts.
This approach has been fully tested in a complete implementation here: https://github.com/Aflynn50/sqlair/tree/lazy-prepare .


TypeError: unhashable type: 'dict' when loading the RasaNLUInterpreter

Rasa NLU version: 0.12.3
Operating system : Ubuntu 16.04
Content of model configuration file:
config_spacy_duckling.yml
language: "en"

pipeline:
# this is using the spacy sklearn pipeline, adding duckling
# all components will use their default values
- name: "nlp_spacy"
- name: "tokenizer_spacy"
- name: "intent_featurizer_spacy"
- name: "ner_crf"
- name: "ner_synonyms"
- name: "intent_classifier_sklearn"
- name: "ner_duckling"

Issue:
Anytime I try to launch train_online.py I got a TypeError: unhashable type: 'dict' when loading the RasaNLUInterpreter with RasaNLUInterpreter('./models/nlu/default/moodnlu'). I don't know if it is related with the pickle files. Here is the content of moodnlu/folder:
(MyFlaskAppEnv) mike@mike-thinks:~/Programming/Rasa/myflaskapp/models/nlu/default/moodnlu$ ls
crf_model.pkl  intent_classifier_sklearn.pkl  metadata.json  training_data.json

Here is the error:
(MyFlaskAppEnv) mike@mike-thinks:~/Programming/Rasa/myflaskapp$ python3 train_online.py
Traceback (most recent call last):
  File "train_online.py", line 37, in <module>
    nlu_interpreter = RasaNLUInterpreter('./models/nlu/default/moodnlu')
  File "/usr/local/lib/python3.5/dist-packages/rasa_core/interpreter.py", line 219, in __init__
    self._load_interpreter()
  File "/usr/local/lib/python3.5/dist-packages/rasa_core/interpreter.py", line 238, in _load_interpreter
    os.environ))
  File "/usr/local/lib/python3.5/dist-packages/rasa_nlu/model.py", line 230, in load
    skip_valdation)
  File "/usr/local/lib/python3.5/dist-packages/rasa_nlu/model.py", line 253, in create
    components.validate_requirements(model_metadata.pipeline)
  File "/usr/local/lib/python3.5/dist-packages/rasa_nlu/components.py", line 56, in validate_requirements
    component_class = registry.get_component_class(component_name)
  File "/usr/local/lib/python3.5/dist-packages/rasa_nlu/registry.py", line 125, in get_component_class
    if component_name not in registered_components:
TypeError: unhashable type: 'dict'

I had this issue before but I don't remember how I dealt with it.

Could it be that that's an old model? Have you tried retraining it?

True ! I didn't know it had an expiry date


Tasks with empty input files are marked as NO-SOURCE instead of UP-TO-DATE

This PR is related to https://github.com/gradle/gradle-private/issues/641

@breskeby Late review: LGTM. The only thing I'd request is that you add this to the 3.4 release notes.

Hey,
It’s already mentioned in the release notes at https://github.com/gradle/gradle/blob/b4564fdb0057ded104a4736a7152f6370973577b/subprojects/docs/src/docs/release/notes.md#improved-feedback-when-skipping-tasks-with-no-source-input https://github.com/gradle/gradle/blob/b4564fdb0057ded104a4736a7152f6370973577b/subprojects/docs/src/docs/release/notes.md#improved-feedback-when-skipping-tasks-with-no-source-input

On 12 Jan 2017, at 21:11, Eric Wendelin<EMAIL_ADDRESS>wrote:
@breskeby https://github.com/breskeby Late review: LGTM. The only thing I'd request is that you add this to the 3.4 release notes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/gradle/gradle/pull/1045#issuecomment-272284619, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEt9GkEZkL2gbBoWPnBbqXiXN6Z8HXJks5rRpb3gaJpZM4LSAfY.

Brilliant. Thank you!
On Thu, Jan 12, 2017 at 2:16 PM Rene Groeschke<EMAIL_ADDRESS>wrote:

Hey,
It’s already mentioned in the release notes at
https://github.com/gradle/gradle/blob/b4564fdb0057ded104a4736a7152f6370973577b/subprojects/docs/src/docs/release/notes.md#improved-feedback-when-skipping-tasks-with-no-source-input
<
https://github.com/gradle/gradle/blob/b4564fdb0057ded104a4736a7152f6370973577b/subprojects/docs/src/docs/release/notes.md#improved-feedback-when-skipping-tasks-with-no-source-input

On 12 Jan 2017, at 21:11, Eric Wendelin<EMAIL_ADDRESS>wrote:
@breskeby https://github.com/breskeby Late review: LGTM. The only
thing I'd request is that you add this to the 3.4 release notes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/gradle/gradle/pull/1045#issuecomment-272284619>, or
mute the thread <
https://github.com/notifications/unsubscribe-auth/AAEt9GkEZkL2gbBoWPnBbqXiXN6Z8HXJks5rRpb3gaJpZM4LSAfY
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gradle/gradle/pull/1045#issuecomment-272285957, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADJToru5AeVMbWiv3ZQbVVEsOHuxm2Fks5rRpg2gaJpZM4LSAfY
.


Add my work to your page!

Do you think that my work is worthy to be mentioned here? :smile_cat:

Most definitely! Can you submit a PR to add it to README.md in the bottom in the Examples section? Otherwise I will try to get to it some time this week.

Thanks for the PR.


XEMM: rate oracle confirmation not shown during start of the strategy

Describe the bug
After creating a new cross exchange strategy, once started, it is expected to show the current rate conversion if use_oracle_conversion_rate is set to true. However, the client doesn't seem to display the rate, although the strategy still works as expected. Issues randomly happen.
Current behavior:

Expected behavior:

Steps To Reproduce
Here are the steps to reproduce the issue (see attachments in the section below):

Create a cross-exchange market making strategy using Binance as maker exchange and Crypto_com as taker (can use other exchanges)
Enable use_oracle_conversion_rate
Start the strategy
Verify if the rate conversation is shown
...

Screenshots
Release version
master and dev (1.3.0)
Attachments
WARNING: Do NOT publish any exchange API keys or your wallet's private key. Whoever has access to them may steal your assets!
Optional: your discord username:

Reproduced on dev-1.8.0
Bot not showing rate oracle for the same pairs on taker and maker exchanges
Also we have this issue using different pairs https://github.com/hummingbot/hummingbot/issues/5068

Closing ticket, XEMM has been redesigned since the deployment of Pydantic to Hummingbot client. A new approach using conversion_rate_mode has been added to improve UX


Whitelist hidden files from globbing

Renamed a controller to have a . in front of the file and caused the following error when running lux serve.

This can pose a problem when using hidden files for custom linting .eslintrc, ignoring files .gitignore .eslintignore or even keeping an empty directory on git  .gitkeep

You shouldn't rename controllers, models, or serializer's because they are resolved by file name to be dynamically loaded into your application.
This will not be an issue with .eslintrc or any other dot files as they are not in the controller/model/serializer directory.

I disagree.  Like automatic backups or logs that you as a framework have no control over.
This can potentially affect your app because you are trying to include all files regardless of extension or hidden status.

I think the real issue here is that the loader does not filter out non .js extensions. That doesn't mean that we should allow for models to be prefixed with .. Part of using a framework is conforming to it's opinions such as file naming conventions.
Can you create a new issue for filtering out any file that doesn't have .js extension in the loader package?

I think I've been misunderstood. I don't want you to let people access hidden files. It's the other way around. Ignore all files that have a . in front of the file name regardless of extension. And also ignore any other file that doesn't have a .js extension.

That makes 100% sense. Can you write that in a new issue? 😃


Add Encode() and Raw() overrides to IHtmlHelper<TModel>

#2392
dynamic does not work correctly when inherited from a base interface

/cc @Eilon this is the fix we discussed

:shipit:

8b5931d


az postgres flexible-server execute cannot read UTF-8 with BOM SQL files

Related command
az postgres flexible-server execute
Describe the bug
Using the dotnet-ef tool https://docs.microsoft.com/en-us/ef/core/cli/dotnet#dotnet-ef-migrations-script it generates a UTF-8 BOM SQL file. When send to az postgres flexible-server execute via -f it fails with following error:
2022-07-18T12:19:53.7465981Z ERROR: syntax error at or near "CREATE"
2022-07-18T12:19:53.7466937Z LINE 1: CREATE TABLE IF NOT EXISTS "__EFMigrationsHistory" (
2022-07-18T12:19:53.7467668Z         ^
2022-07-18T12:19:53.7467916Z

To Reproduce

Create a SQL file with UTF-8 with BOM encoding
Send it via az postgres flexible-server execute

Expected behavior
It should be possible to read UTF-8 with BOM encoded SQL files.
Environment summary
az --version dump
azure-cli                         2.38.0

core                              2.38.0
telemetry                          1.0.6

Extensions:
azure-devops                      0.25.0
rdbms-connect                      1.0.2

Dependencies:
msal                            1.18.0b1
azure-mgmt-resource             21.1.0b1

Additional context
The flexible server is completely as default. Maybe it is a postgres setting, or the rdbms-connect tool which is required to run the execute command that can't handle UTF-8 with BOM.

@julian-code Engg. team working on finding the root cause of it. I will update you as soon as I hear back from them.

@julian-code The bug was fixed in rdbms-connect 1.0.3, can you please check?

@julian-code Please let me know if you are seeing any issues.


galera::status must be conditionally included

Now galera::status in just included unconditionally in galera::validation so setting status_check to false doesn't prevent installing clustercheck script and xinetd service in case validation = true. I suggest to use for validation either status user/password if they are given and status_check enabled or to use root account as failback

@efoft Sorry for not being clear on this. We should keep all three class parameters as OPTIONALs, this includes $user and $password. Could you update your submission again? Thanks :)

definately, no problem! :)
@fraenki, Would you mind if I set them all to undef instead of $galera::status_* since they are optional? because if let's say we set status_check to false, galera::status won't be included and effectively this 3 params becomes undef anyway...

@efoft Merged, thanks!


Create SDK for developers to access data via modules

Create SDK for developers to access data via modules

Dup of this: https://github.com/projectjellyfish/api/issues/524


Add option to set minimum height for scroll tracking

Some pages are smaller or equal to the size of the viewport, and scroll tracking is not as helpful. We should make it possible to disable tracking in those contexts.

Done with #6


CRASH Commit ce3da11 on branch release/0.0.1-beta.2

  ☠  Coda Daemon crashed. The Coda Protocol developers would like to know why!

  Please:
    Open an issue:
      https://github.com/CodaProtocol/coda/issues/new

    Briefly describe what you were doing, and include the last 20 lines from .coda-config/coda.log.
    And then paste the following:

    (monitor.ml.Error (Unix.Unix_error "Bad address" read "")
  ("Raised at file \"src/result.ml\" (inlined), line 158, characters 17-26"
    "Called from file \"src/raw_fd.ml\", line 289, characters 4-60"
    "Called from file \"src/raw_fd.ml\", line 282, characters 10-26"
    "Re-raised at file \"src/reader0.ml\", line 344, characters 30-39"
    "Called from file \"src/job_queue.ml\" (inlined), line 131, characters 2-5"
    "Called from file \"src/job_queue.ml\", line 170, characters 6-47"
    "Caught by monitor coda"))

{"timestamp":"2019-08-08 22:50:20.790108Z","level":"Info","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 206, characters 49-56"},"message":"Coda daemon is booting up; built with commit $commit on branch $branch${"timestamp":"2019-08-08 22:50:20.793635Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 268, characters 63-70"},"message":"Error reading coda.version: $error","metadata":{"error":"(monitor.ml.${"timestamp":"2019-08-08 22:50:20.793947Z","level":"Debug","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 271, characters 63-70"},"message":"Failed to read coda.version, cleaning up the config directory $config${"timestamp":"2019-08-08 22:50:20.796177Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 301, characters 65-72"},"message":"Error reading daemon.json: $error","metadata":{"error":"(Unix.Unix_er${"timestamp":"2019-08-08 22:50:20.796247Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 323, characters 65-72"},"message":"Key '$key' not found in the config file, using default","metadata":{"${"timestamp":"2019-08-08 22:50:20.796276Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 323, characters 65-72"},"message":"Key '$key' not found in the config file, using default","metadata":{"${"timestamp":"2019-08-08 22:50:20.796302Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 323, characters 65-72"},"message":"Key '$key' not found in the config file, using default","metadata":{"${"timestamp":"2019-08-08 22:50:20.796323Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 323, characters 65-72"},"message":"Key '$key' not found in the config file, using default","metadata":{"${"timestamp":"2019-08-08 22:50:20.796338Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 323, characters 65-72"},"message":"Key '$key' not found in the config file, using default","metadata":{"${"timestamp":"2019-08-08 22:50:20.796353Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 323, characters 65-72"},"message":"Key '$key' not found in the config file, using default","metadata":{"${"timestamp":"2019-08-08 22:50:21.534223Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 496, characters 54-61"},"message":"Creating database of type $typ in directory $directory","metadata":{"${"timestamp":"2019-08-08 22:50:34.977769Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 510, characters 63-70"},"message":"Creating database of type $typ in directory $directory","metadata":{"${"timestamp":"2019-08-08 22:50:34.978242Z","level":"Debug","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 477, characters 63-70"},"message":"Long async cycle: $span milliseconds","metadata":{"pid":1762,"span":"${"timestamp":"2019-08-08 22:50:35.023455Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 533, characters 64-71"},"message":"Creating database of type $typ in directory $directory","metadata":{"${"timestamp":"2019-08-08 22:50:35.077782Z","level":"Trace","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 541, characters 62-69"},"message":"Creating database of type $typ in directory $directory","metadata":{"${"timestamp":"2019-08-08 22:50:41.817370Z","level":"Debug","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 477, characters 63-70"},"message":"Long async cycle: $span milliseconds","metadata":{"pid":1762,"span":"${"timestamp":"2019-08-08 22:50:41.836333Z","level":"Debug","source":{"module":"Kademlia__Membership","location":"File \"lib/kademlia/membership.ml\", line 191, characters 56-63"},"message":"Kademlia command-line arguments: $argv","metad${"timestamp":"2019-08-08 22:50:41.895387Z","level":"Info","source":{"module":"Transition_router","location":"File \"lib/transition_router/transition_router.ml\", line 98, characters 53-60"},"message":"Starting Bootstrap Controller phase${"timestamp":"2019-08-08 22:50:41.897544Z","level":"Debug","source":{"module":"Network_pool__Transaction_pool","location":"File \"lib/network_pool/transaction_pool.ml\", line 224, characters 69-76"},"message":"no frontier","metadata":{"${"timestamp":"2019-08-08 22:50:41.897696Z","level":"Debug","source":{"module":"Network_pool__Transaction_pool","location":"File \"lib/network_pool/transaction_pool.ml\", line 224, characters 69-76"},"message":"no frontier","metadata":{"${"timestamp":"2019-08-08 22:50:41.900422Z","level":"Info","source":{"module":"Coda_lib","location":"File \"lib/coda_lib/coda_lib.ml\", line 150, characters 63-70"},"message":"Coda daemon is now connecting","metadata":{"pid":1762}}
{"timestamp":"2019-08-08 22:50:41.900891Z","level":"Info","source":{"module":"Proposer","location":"File \"lib/proposer/proposer.ml\", line 224, characters 57-64"},"message":"Pausing block production while bootstrapping","metadata":{"pi${"timestamp":"2019-08-08 22:50:41.902432Z","level":"Trace","source":{"module":"Web_pipe","location":"File \"app/cli/src/web_pipe.ml\", line 122, characters 58-65"},"message":"Not running a web client pipe","metadata":{"pid":1762}}
{"timestamp":"2019-08-08 22:50:41.902865Z","level":"Info","source":{"module":"Coda","location":"File \"app/cli/src/coda.ml\", line 600, characters 56-63"},"message":"Daemon ready. Clients can now connect","metadata":{"pid":1762}}
{"timestamp":"2019-08-08 22:50:42.896385Z","level":"Debug","source":{"module":"Coda_run","location":"File \"app/cli/src/coda_run.ml\", line 70, characters 52-59"},"message":"Successfully wrote the visualization of the registered masks a${"timestamp":"2019-08-08 22:50:42.916931Z","level":"Debug","source":{"module":"Coda_run","location":"File \"app/cli/src/coda_run.ml\", line 83, characters 58-65"},"message":"Could not visualize transition frontier since daemon is curren$

This would need more info, and the time to collect that info has long passed. Closing.


error: the item TryFrom is imported redundantly

Search for duplicate issues

[X] I already searched, and this issue is not a duplicate.

Issue scope
Linux (E.g. Arch Linux, Debian, Alpine, etc)
Describe the bug
   Compiling static-web-server v2.27.0 (/home/yonas/.cargo/git/checkouts/static-web-server-d0678737845d2249/2286e45)
error: the item `TryFrom` is imported redundantly
  --> src/cors.rs:16:33
   |
16 | use std::{collections::HashSet, convert::TryFrom};
   |                                 ^^^^^^^^^^^^^^^^
  --> /rustc/0ecbd0605770f45c9151715e66ba2b3cae367fcb/library/std/src/prelude/mod.rs:129:13
   |
   = note: the item `TryFrom` is already defined here
   |
note: the lint level is defined here
  --> src/lib.rs:99:9
   |
99 | #![deny(warnings)]
   |         ^^^^^^^^
   = note: `#[deny(unused_imports)]` implied by `#[deny(warnings)]`

error: could not compile `static-web-server` (lib) due to 1 previous error

How to reproduce it

git clone ...
cargo build --release

Expected behavior
Should compile without error.
Complementary information
No response
Build target
x86_64-unknown-linux-gnu
Environment and specs

[v2.27.0] static-web-server:
[Ubuntu 23] OS:
[x86_64] Arch:

Additional context
No response

What Rust version are you using?
Because interestingly I built SWS successfully in Debian 12 with Rust 1.75/1.76.
What if you comment just remove the std::convert::TryFrom from src/cors.rs:16:33, does it work?

What Rust version are you using?

rustc 1.78.0-nightly (0ecbd0605 2024-02-25)

What if you just remove the std::convert::TryFrom from src/cors.rs:16:33, does it work?

Yes.

I see, then just feel free to submit a patch for nightly.

Done. https://github.com/static-web-server/static-web-server/pull/318


Adding

Hey I've been having problems trying to add third party analytic scripts to the boilerplate.  When adding to helpers/Html.js, it just spazes out.  Anyone here have any ideas? thanks!

Found solution
https://github.com/erikras/react-redux-universal-hot-example/issues/686


Slow state registration

Hey there, I've upgraded to<EMAIL_ADDRESS>Really love the new changes! Only one thing I'm having an issue with. I have a pretty big site that registers quite a few states in the config phase. I've noticed that with the new router my state registration is really really slow. It's actually taking 3.5s on my computer just to register states. It's showing nothing while this happens since it can't even route until the states are registered.
You can see the code for my states/routes here: https://github.com/gamejolt/gamejolt/tree/master/src/app/views They're in the -state.js/ts files. Nothing too fancy, I think. Just URL-based states.
I profiled the code in Chrome to get a CPU profile of the start of the page. I noticed there were two methods that were taking most of the time. I was able to get it down to ~50ms from 3.5s by just commenting them out. You can see the two changes here: https://github.com/gamejolt/gamejolt/blob/cb8b65838f4f544f042c9c3df8385f5f29c8845f/src/lib/ui-router.js Just search for @MODIFY.
I'm not very comfortable modifying the code, so I'm not totally sure what I'm changing here. The change here: https://github.com/gamejolt/gamejolt/blob/cb8b65838f4f544f042c9c3df8385f5f29c8845f/src/lib/ui-router.js#L4729 seems to be taking the majority of the time. I logged out the matches and it was always undefined so I kept that commented out. Sadly the change at https://github.com/gamejolt/gamejolt/blob/cb8b65838f4f544f042c9c3df8385f5f29c8845f/src/lib/ui-router.js#L3737 broke my states so I had to revert it. The second change was still taking about 1-1.5s of the total time of the slow registration. It seems to be sorting the states so it can match on them properly? Is there a way to do this after the main states are registered for the app in the config phase instead of doing it every time a state is registered?
I'll be working on trying to figure out how to hack the perf for the second change I made, so the site may be changed once you get to this, but you can try to grab a profile here: https://gamejolt.com
Willing to help out in any way I can since it really affects the performance of the initial load of the site. Thanks!
Edit: To add a little more info, there is currently 120 states registered on start up.

Just pushed out a new change that seems to sort the routes correctly but delays it until the "match" against all states. This is still a bit slow (10-50ms) each time it needs to route to a new page. But it seems better to eat the cost each route change vs such a large load time before showing anything.
Here's the relevant commit: https://github.com/gamejolt/gamejolt/commit/535fb9fd26725a2bb44c62f94c2db92abdd10790
I'm sure I broke something else while doing this, but I have no clue. =P

+1, this needs to be fixed before a stable release. State registration shouldn't be this slow, 200-300 states isn't that big of a deal for a complex app.
After investigating the code I've found a workaround without actually changing anything in the angular ui source code.
You need to replace the sort function on the router with an empty function before starting to register states. After every state has been registered, you need to add the correct sort function back and execute it.
    let slowSort;
    app.config(['$urlRouterProvider', function ($urlRouterProvider) {
        slowSort = $urlRouterProvider._urlRouter.sort;
        $urlRouterProvider._urlRouter.sort = function(){};
    }]);

    app.config(['$stateProvider', function ($stateProvider) {
        //state definitions go in here
        //$stateProvider.state(.....)
    }]);

    app.config(['$urlRouterProvider', function ($urlRouterProvider) {
        $urlRouterProvider._urlRouter.sort = slowSort;
        $urlRouterProvider._urlRouter.sort();
    }]);

Related to https://github.com/angular-ui/ui-router/issues/3274

Closing as duplicate of #3274
This is fixed in master and will be released in rc.2
The analysis is correct, the perf regression is due to sorting each time a state is registered.  Sorting is an expensive task.  In master sorting is deferred till Just In Time (wait until the sorted data is used for the first time).  I also reduced the cost of sorting in general.  rc.2 will be the same perf as ui-router 0.2.x.


Clippy

What does this PR try to resolve?
A few more small anti-patterns from our code base. Each in their own commit, found by turning on a clippy warning and manually reviewing the changes. Most of the changes are from switching to let-else.
How should we test and review this PR?
Internal refactoring and tests still pass.
Additional information
I know we don't like "fix clippy" PR's. Sorry.
On the other hand having reviewed for these lints, I wouldn't mind turning them to warn. 🤷

Anyway, thanks!
@bors r+

:pushpin: Commit afe5333d86632a887090b8a220885004f464cb95 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit afe5333d86632a887090b8a220885004f464cb95 with merge 39162c7ff382b5ceee527a0e55d6349264d099cc...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 39162c7ff382b5ceee527a0e55d6349264d099cc to master...

:eyes: Test was successful, but fast-forwarding failed: 422 Update is not a fast forward

@bors retry

:hourglass: Testing commit afe5333d86632a887090b8a220885004f464cb95 with merge 80932be2e6c492718efa7991ecc4117fe1f9eb7f...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 80932be2e6c492718efa7991ecc4117fe1f9eb7f to master...


Added operation: "XPath expression"

This adds XPath expressions under the 'Extractors' category.
It evaluates an XPath expression and lists the results.

The xpath evaluation is done with the jQuery plugin jquery.xpath.js
Fixes #13

Hey,
I also added a css-query operation (select some nodes from an HTML document with an css query).
If you want these in seperate pull requests or something I can split them up.
But I think they kinda fit together.
Greets
~ Mike

Hi @Mikescher,
This looks good. Just a few changes to make before merging.

Take a look at the Design Principals on the wiki. If possible, we'd rather not use a library that relies on jQuery. It looks like the jquery.xpath.js library is based on xpath.js - could this be used instead?
Add an @author field to the JSdoc comment for each of your added operations so that you have the credit for them.
Sign our Contributor Licence Agreement so that we can accept your code.

Thanks very much for your contributions, I look forward to merging them.

Take a look at the Design Principals on the wiki. If possible, we'd rather not use a library that relies on jQuery. It looks like the jquery.xpath.js library is based on xpath.js - could this be used instead?

Ok, i changed it so it uses xpath.js directly.

Add an @author field to the JSdoc comment for each of your added operations so that you have the credit for them.

ok, done

Sign our Contributor Licence Agreement so that we can accept your code.

ok, signed and send


Waveforms of no sound at all appear at their max size instead of their min size

My code:
@override
  void initState() {
    super.initState();
    _recorderController = RecorderController();
    _recorderController.record();
  }

...

AudioWaveforms(
    size: const Size(230, 50),
    recorderController: _recorderController,
    waveStyle: const WaveStyle(
      waveColor: Colors.pink,
      spacing: 8.0,
      extendWaveform: true,
      showMiddleLine: false,
   ),
)

For some reason the wave forms of absolute silence appear at max size.
Here is a photo - took the screenshot when there was no sound at all

It is important to mention that when there is sound the waveforms seems to work as expected.

From your this #44 issue I see you are using an emulator. Can you please test it on a real device because android emulators behave weirdly in different scenarios?

Here the same thing happens, I tested it on a real device.

I found the possible cause, in RecorderWavePainter the waveData[i] is coming a negative value, using this condition the wave worked as it should: (waveData[i] < 0 ? 0 : waveData[i]).

Adding this ..normalizationFactor = 40, works here for me. The default for android is 60.

That seems to improve this, thank you so much

Acutely after a lot of testing setting the ..normalizationFactor = 40 is still not good enough. for some reason the sound waves does not react correctly to my voice. further more when I play the recording using AudioFileWaveforms the sound waves look completely different than the sound waves while recording. I think it is a bug.


Can't add new tourists to payment

Should be able to add new people to a tour.
Process: add name and shirt size
offer to add to payer's address?
Show person information card in tourist block so it's easy to see which Mike or Dave it is.


(SERVER-325) Add support for max-requests

This commit adds support for flushing an individual JRuby
instance after it has exceeded the number of requests
specified by the max-requests-per-instance setting.
There are a couple of additional tests I would like to add to this, but other than that I think it should be ready to go.  I can add those in a subsequent PR if this one gets reviewed and is deemed mergeable, and that might help us minimize merge conflicts.

This is rebased, and I have addressed @nwolfe and @KevinCorcoran 's initial stylistic suggestions.  I've also added another test based on a suggestion that @camlow325 in an earlier version of this PR.  I have one more integration test that I'm going to attempt to write but other than that I think this is finished, pending additional feedback.

Added a few more commits to improve the integration test coverage (specifically, testing an instance flush via max-requests-per-instance while a pool flush is already in progress).
I'm done with this PR, AFAIK, at this point.

Increase the max heap in the project.clj until travis stops failing ( :no_good: )

Are there any downsides to this option?  Seems reasonable to me.  Although ... I wonder why you're not seeing the same behavior locally?  I wouldn't expect memory usage to differ that much between your laptop and travis.

the max heap is set in the project file to 256MB

This is wrong, it's actually MaxPermSize.
Interestingly, the OutOfMemoryError in https://travis-ci.org/puppetlabs/puppet-server/jobs/58275376 does not seem to be about perm gen, so increasing that value in project.clj wouldn't have helped.
But, yeah, we could still try setting Xmx.

I tried setting -Xmx super low in project.clj and confirmed that I could reproduce the OutOfMemoryError during lein test, so, it seems like setting that value to something reasonable for our test suite could be a path to nirvana here.

I've reviewed all of the production code.  It looks mostly good, aside from my confusion around next-instance-id.  I'll take a look at the test code tomorrow.

Increase the max heap in the project.clj until travis stops failing ( :no_good: )
Are there any downsides to this option? Seems reasonable to me. Although ... I wonder why you're not seeing the same behavior locally? I wouldn't expect memory usage to differ that much between your laptop and travis.

Downsides:

I don't really know what a sane value would be there, nor how high Travis will let us go, or even what Travis's default is
This test, on my laptop, increases the total run time of the test suite from 3 minutes to 5 minutes.  That seems like it might be pushing it.  We could always consider extracting the JRuby service to a separate git project at some point, but still...

I'm basically :+1: on this PR, pending what we end up deciding to do about the double-flushing test.  I also still want to do some work around SERVER-251 to assess whether or not turning this feature on might lead to a Bad File Descriptor if a container is terminated while a file handle is open in another container.

Summarizing what I believe to be the outstanding work on this PR:

Look into mocking the JRubies for the one giant insane test
Rename :request-count to :borrow-count
Change with-jruby-puppet to borrow/return in j_p_service_test
Attempt to resolve WP violation
constances typo
Look into having the trigger-flush test helper fn return something other than a boolean

@KevinCorcoran @camlow325 did I miss anything?
Also, @camlow325, were you thinking that we need to gate this PR on your BFD exploration?  Or that if the BFD exploration turns up something that we'll just need to do some additional work to solve that?

looks right to me

The summary all sounds right to me.

Also, @camlow325, were you thinking that we need to gate this PR on your BFD exploration? Or that if the BFD exploration turns up something that we'll just need to do some additional work to solve that?

Probably better to just let the PR go in and handle the outcome of the BFD exploration later.  If it turns out this change would make the implementation more susceptible to BFDs and any residual work needed to overcome it wouldn't be doable in time for whenever our next release needs to land, we'll have to talk about how we want to deal with that - e.g., maybe even leaving the code in place but turning off the max-requests cap by default.  I understand that discussion would be very premature to have at this point.  Without significant performance penalties - which may make the whole feature untenable - I'd have a hard time envisioning that the discovery of BFD susceptibility would lead to any major architectural changes to what is currently in the PR.
So, yeah, I'm okay letting the PR come in once the items in your summary have been addressed.

Rebased and added some commits.  This includes:

Mocking JRubies for the one giant test; this shaves almost 2 minutes off of the test suite run on my laptop.
Change request-count to borrow-count
Fixed WP violation
Fixed constances typo

There were two things in the list that I didn't end up doing.  Here's why, and I'm open to additional discussion on both:

Did not change with-jruby-puppet to borrow/return in the jruby_puppet_service test, because the test in question is actually intended to test the with-jruby-puppet macro.  I did add a clarifying comment to try to make this more clear.
I didn't change the trigger-flush test fn, because I didn't see anything that it could return that would actually be meaningful in the tests.  I started going down the road of returning the HTTP status code and having each test verify it, but it just seemed weird and out-of-place to me, so I rolled it back.  I can go ahead and do it anyway if you think it's important, or we can refactor as needed as the test namespace evolves in the future.

:+1: Left one more nitpick but we can sure :ship: this without it.  @camlow325 I will let you have one more look at this, feel free to merge.

Interesting on the mock JRubies for the big test - I'd been thinking that creating a ScriptingContainer and doing a runScriptlet on it would still be reasonably expensive - though certainly not nearly as expensive as initializing all of Puppet within that container, of course.  So if we're still including "real" ScriptingContainers and running some Ruby code through these, doesn't seem like we lost much by going with "mocks" anyway.  Seems good to me.

:+1: :shipit:


Profile updates are not updated realtime in the chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Start a chat in two windows
Update the Profile Picture or Display Name for one of the users
You'll see until the page is reloaded, the updates are not visible to the other participant.

Expected Result:
It should update the image and the display name for the user.
Actual Result:
https://user-images.githubusercontent.com/3069065/127178983-d2e72a87-38b6-46f6-87c8-5a5b7e9f61d4.mp4
Check the video that the participant's info is not updated until the user refreshes the chat.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:
Where is this issue occurring?

[x] Web
[x] iOS
[x] Android
[x] Desktop App
[x] Mobile Web

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @CortneyOfstad (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Triggered auto assignment to @marcaaron (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

I'm not sure this is a priority for us right now. Making these names change requires that the recipient of the message be able to identify that a recipient's name has changed from whatever we have locally for them and then personal details will need to be refetched for that use.
We can maybe keep talking about how this should work at a high level, but it feels pretty low priority + we should discuss whether to pick one of these solutions over the other:

When the personal details change alert all users who need them that they have changed
When we handle a comment try to detect a change in personal details and refetch the details

I'm not too sure which one would be correct but solution A must be worked on internally and solution B feels like a hack.

Gonna close this since I don't think we're going to get to it anytime soon and if we do it might be in a design doc.


Alternative header layout

What
Adds a new layout alternative to the layout header.
https://trello.com/c/vMiHP2yB/194-build-header-layout-for-search-on-left
Why
This is needed for pages in whitehall such as https://www.gov.uk/government/how-government-works
Notes: The layout change meant that the skeleton of the html needed to change. To avoid repeating blocks they are now in partials. I tried to do this with content blocks but it broke the component's examples page as each render of the content block appended itself to the previous ones before displaying.
This is being pushed back into a non-master branch so that individual chunks of work can be reviewed and monitored on Trello rather than pushing to one huge PR.
Screenshot

is it known that the nav menu is not visible on mobile?

is it known that the nav menu is not visible on mobile?

No, it wasn't... it was working previously I must have broken something. Looking into it.

is it known that the nav menu is not visible on mobile?

Fixed now, I'd accidentally optimised away the div that set the relative position.

A few initial visual discrepancies (I haven't dived into the code yet).

The layout at mobile is different anyway - the search bar is full width deliberately rather than half width to some arbitrary point where it gets hidden on live. Likewise the menu items don't try to cram into 2 columns. I thought it made more sense for the menu button to appear on the left than on the right if we're not putting the search over there. If you think it should be on the right, why?
I'll look into the separated icon, though.

I'm not sure which side it should be on, I was just pointing out the difference.
Another slightly odd behaviour - if you expand the menu on mobile then switch to desktop the whole of the menu is pushed down slightly. Obviously this is a niche use case and doesn't represent a barrier to anything, it's just a little odd.

@kr8n3r @andysellick @vanitabarrett @injms
I believe I've addressed the multiple comments and concerns on the initial version of this branch. Would be good to get feedback on it as it stands now. Updates are broken down into separate commits.

Is this placement of the menu link here correct? Could the spacing below the Menu link match the spacing above, as it looks quite close to the blue border?

Not sure if this is a problem which existed before, but it looks like a search bar on it's own needs some bottom spacing on smaller devices so it doesn't touch the blue bar

There are a couple of false positives with the automated accessibility tests caused by the same components being repeated on the example page - it's not a problem with the component, as this is (hopefully) something that wouldn't happen in the real world use.
We can turn off the checks by adding duplicate-id-aria and form-field-multiple-labels to the accessibility_excluded_rules in the layout header's YAML file.
An alternative fix for the form-field-multilple-labels problem is to make the id of the search component unique by appending #{SecureRandom.hex(4)} to the id - which is what's done on the accordion component.

Is this placement of the menu link here correct? Could the spacing below the Menu link match the spacing above, as it looks quite close to the blue border?

Fixed the spacing issues and the stray new lines introduce/overlooked previously.

There are a couple of false positives with the automated accessibility tests caused by the same components being repeated on the example page

I've ignored those checks. The accordion example is different - you could potentially have multiple accordions on a single page, but you shouldn't have multiple headers at any time, so it needs the randomising approach whereas we don't here.


Order on table columns

Which class is this unit test associated with?
I'm creating dynamic tables based on generic class, and using custom attributes to show, change text, width, etc, everything is ok and works good, but for now the order is the same order of the properties on the class, so I would like to have a custom order, I searched on the component and found the property Order on columns, but when I'm using this is not working.
My code is something like this, everything works, except order.
private Task OnColumnCreating(List columns)
{
foreach (var item in columns)
{
int width = GetWidth(item);
if (width > 0) item.Width = width;
  item.Text = GetText(item);
  int order = GetOrder(item);
  if (order > 0) item.Order = order;
 item.Visible = GetVisibility(item);;
}
return Task.CompletedTask;

}
am I doing wrong? theres other way to do? or this is not working?
thank you very much

@gmondin hi. could you provide a mini-sample project? I will repro this and PR into your repo if resolve it.

Hi @ArgoZhang the code is something like this, its a simple table, the only thing is that is generated by T = class dynamically in other parts of the project. The table is created normally and works, I only try to add the order (I put an example, because in my code I take from Property Attribute, but doesnt show in the order that I'm putting on)

private Task OnColumnCreating(List columns)
{
int order = columns.count();
foreach (var item in columns)
{
item.Order = order;
order --;
}
return Task.CompletedTask;
}

@gmondin sorry, I can't provide any help if there is no mini-repro project.


Cloud Firestore API: firestore.googleapis.com

Just wanted to note that this service has two server-side streaming (aka streaming read) RPCs:

google.firestore.v1.Firestore.BatchGetDocuments
google.firestore.v1.Firestore.RunQuery

Found using:
cat  ~/googleapis/api-index-v1.json | jq '.apis | map(select(.services != null)) | map(.services | map(select(.methods != null)) | map(.methods | map(select(.mode == "SERVER_STREAMING") | [.mode, .fullName]  ))) | flatten ' | grep -v SERVER_STREAMING  | grep -v beta

It also has bidirectional streaming RPCs:

google.firestore.v1.Firestore.Listen
google.firestore.v1.Firestore.Write

Found using:
cat  ~/googleapis/api-index-v1.json | jq '.apis | map(select(.services != null)) | map(.services | map(select(.methods != null)) | map(.methods | map(select(.mode == "BIDIRECTIONAL_STREAMING") | [.mode, .fullName]  ))) | flatten ' | grep -v BIDIRECTIONAL_STREAMING  | grep -v beta

We do not have enough time to work on this service. Closing.


DOC: fix typos in doc

The typo changes involved in tutorials and documentation are worth making. However, those involved with release notes (which seem to be most of the changes) are not. This is because the typos involved in the latter are from git commit messages, and we're not going to go making changes to those.

@andyfaff Done. Please check.

Thank you @kianmeng

:tada: :tada: :tada: :tada: :tada:


Ubuntu 1404 build failure

Hi there,
I am facing a build issue on my own ubuntu 1404 desktop,
there is nothing special when compiling the dependencies, but when I turn to $OPENWEBRTC/build.sh -Cbr linux to make the final lib, the error as follows blocked me:
... ...

I found out that there was a commit recently which related to gst_plugin for replacing erdtls by dtls :+:

GST_PLUGIN_STATIC_REGISTER(erdtls);

GST_PLUGIN_STATIC_REGISTER(dtls);

I am wondering if any other libraries or configurations needed for this change, or shall we temporaryly roll back to erdtls one?
Your professional suggestions will be highly appreciated.
Regards,
Weixing

You will need to rebuild (which will update) the gst-plugins-bad dependency. This should fix the build error.

Hi Nirbheek,
Thanks for your reply, I downloaded the source code last night, and updated source just now, but unforturnately this issue still occurred when I build the
are you using the following options for your environment?
./build-gst-plugins-bad.sh -Cbr linux
and what configuration should be used for eliminate the undefined reference error for 'gst_plugin_dtls_register' ? do I need to install more libs or settings for environmental variables ?
I didn't change anything in the script of build-gst-plugins-bad.sh:
${home}/$BUILD_DIR/configure
--prefix=${installdir}
--host=${target_triple}
--disable-eglgles
--disable-asfmux
--disable-freeverb
--disable-decklink
--disable-mpegtsmux
--disable-mpegpsmux
--disable-yadif
--disable-fatal-warnings
--enable-static
--disable-shared
--enable-static-plugins
--disable-introspection
--enable-gobject-cast-checks=no ${extra_configure_flags}
|| exit 1
Regards,
Weixing

Sorry, I misunderstood how the build scripts work (I don't actually use it myself; I use Cerbero to build OpenWebRTC). build-gst-plugins-bad.sh needs to be updated to the latest version of upstream git.
You will likely have better luck building everything with Cerbero, to be honest. See https://github.com/EricssonResearch/openwebrtc/issues/222 :)

As @nirbheek mentioned, we're fully focused on moving to the cerbero build system to get everything building there and even to make a first official release with downloadable binaries so that people who are most interested in just trying out OpenWebRTC can get going as quickly as possible.

Hi Superdump,
Thank you for introducing me the cerbero, I am switching to work on it now.
But I got an issue when building glib component: saying that my pkg-config is too old, but I have double confirmed pkg-config is installed and upgraded to 0.28 revision.
I checked the glib dir on ~/cerbero/sources/linux_x86_64/glib-2.42.2,
the configure script shown that the version is required as _pkg_min_version=0.16, I think my 0.28 is good enough, right ?
Any idea about it ?
checking for zlib.h... yes
checking for LIBFFI... no
configure: error: in `/home/sun/cerbero/sources/linux_x86_64/glib-2.42.2':
configure: error: The pkg-config script could not be found or is too old.  Make sure it
is in your PATH or set the PKG_CONFIG environment variable to the full
path to pkg-config.
Alternatively, you may set the environment variables LIBFFI_CFLAGS
and LIBFFI_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.
To get pkg-config, see http://pkg-config.freedesktop.org/.
See `config.log' for more details
Running command 'sh autogen.sh --prefix /home/sun/cerbero/dist/linux_x86_64 --libdir /home/sun/cerbero/dist/linux_x86_64/lib  --enable-static --disable-selinux --disable-maintainer-mode  --disable-silent-rules  --enable-introspection '

Hello team,
I am glad to moving forward on my build with cerbero branch, previous issue is due to no bootstrap compiled .
But in the step 35/36 of ./cerbero-uninstalled package openwebrtc, there is a patch:113 could not be proceeded.
Does anyone meet this issue before ?
[(35/36) openwebrtc -> extract ]
HEAD is now at 977a564 Update ROADMAP.md
fatal: A branch named 'cerbero_build' already exists.
Already on 'cerbero_build'
HEAD is now at 977a564 Update ROADMAP.md
Cloning into '.'...
done.
Applying: Owr: Use videoconvert instead of ercolorspace on Android
Applying: Fix build on Android and iOS when not using build.sh
/home/sun/cerbero/sources/linux_x86_64/openwebrtc-0.1/.git/rebase-apply/patch:113: trailing whitespace.
dnl The GNU C++ STL is needed for JNI. This pkg-config file is
error: patch failed: configure.ac:115
error: configure.ac: patch does not apply
Patch failed at 0001 Fix build on Android and iOS when not using build.sh
The copy of the patch that failed is found in:
/home/sun/cerbero/sources/linux_x86_64/openwebrtc-0.1/.git/rebase-apply/patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Running command 'git am --ignore-whitespace /home/sun/openwebrtc/cerbero/cerbero/recipes/openwebrtc/0001-Owr-Use-videoconvert-instead-of-ercolorspace-on-Andr.patch'
Running command 'git am --ignore-whitespace /home/sun/openwebrtc/cerbero/cerbero/recipes/openwebrtc/0002-Fix-build-on-Android-and-iOS-when-not-using-build.sh.patch'
Recipe 'openwebrtc' failed at the build step 'extract' ....

@weixingsun That problem should be fixed now, thanks! Could you try again? (Also, in future, please open new issues for unrelated/new problems!)

https://github.com/EricssonResearch/openwebrtc/wiki/Building-OpenWebRTC


Failed create issue. Post https://gitlab.com/api/v4/projects/issues: http: ContentLength=77 with Body length 0

lab issue -i "Title" -m "Message"
Failed create issue. Post https://gitlab.com/api/v4/projects/issues: http: ContentLength=77 with Body length 0

Thanks for contribution.
Fix complete. 7612d4a0bae9589400181158e5940ac7bb6c7abf


Display field stretches horizontally after entering 11th digit

Description
Display field stretches horizontally after entering 11th digit
Steps

Enter 11 or more digits

Expected

Not sure... error? Scientific notation? emoji?

Actual

Display field stretches to the right.

Gif

Found another manifestation of this.
If you enter 3.14 times 5 and hit =  a few times, you can eventually get a long enough number that doesn't yield NaN yet, but that causes the display field to stretch off to the right even further.


Update zig-wayland and zig-xkbcommon

I'm not sure all the generated protocol code is necessary for tinywl. But the compositor runs correctly.

Please update the build.zig in the root of the repository as well so that zig build test passes.

Should be good now.


Unsaved Change Warning on UpdateUserProfile - #1060

Uses vanilla JS and should be replaced with navigation guards if we switch to vue router. Solves #1060

we should deploy to stage and test before merging, have you done that before?


Fast blockwise parallel sarray iterator

This PR adds a new sarray iterator that runs through data block by
block for fast sequential access.

Will be pulled in through a massive merge of closed-source development into SFrame.


GitHub Actions Migration

This cron-based repos can become GitHub Actions, pending access to their schedules on
Heroku.


Update Reaction Comments description

Adds a notice that encourages limiting the scope of the installation and updates the configuration guide.

View rendered _apps/reaction.md

@JasonEtco, sorry, I'll submit them in one PR in the future. :yum:


ERROR: Could not find a version that satisfies the requirement tmpdump (from versions: none) ERROR: No matching distribution found for tmpdump

ERROR: Could not find a version that satisfies the requirement tcpdump (from versions: none)
ERROR: No matching distribution found for tcpdump
anyone meet with this problem?  with MacOS Catalina and python version 3.8.5

Try without the tcpdump package and report the errors.

Hi, just had the same issue here (MacOS Mojave 10.14.6), solved by removing tcpdump as suggested (but since I had already installed scapy (brew install scapy) and libpcap (brew install libpcap) in the past, I am not sure if other steps are needed for iot-inspector-client to work on a stock machine.
My 2 cents: using virtualenv+virtualenvwrapper helps me keep my system clean. If you also have these packages on your Mac :
$ mkproject -p python3 iot-inspector
$ git clone https://github.com/noise-lab/iot-inspector-client.git
$ cd iot-inspector-client/src
# edit requirements.txt to comment (#) tcpdump
$ pip install -r requirements.txt
$ sudo python start_inspector.py


add Go to PATH

Resolves the issue when lndconnect -j  is not available if Go is not freshly installed.
LNDconnect displays this under the QR, causing confusion if the command is not available:

makes sense - thx :)


update compatibility requirement to 1.6+ (required for winrm/ssh communicators)

The communicators which abstract the ssh/winrm communication were introduced in Vagrant 1.6.
Since we rely on that for winrm support I just updated the compatibiliy requirement (validation in plugin, documentation in README)

This should help for #40 in terms of better error reporting


DL4J: failed test ValidateMKLDNN

Detected test failure in master for ValidateMKLDNN (Either validateBatchNorm or validateConvSubsampling are failing depending on environment):
java.lang.AssertionError: MAX, mb=3, cm=Same, kernel=[2, 2], stride=[1, 1] - Gradients are not equal: 0_W - highest relative error = 1.0 > max relative error = 1.0E-5
Output below is from Window 10 machine, libnd4j was built with -a avx2 -h mkldnn.
https://gist.github.com/alexanderst/8fd93e83b72143ea2b9020c522657574

For the batch norm failures, very likely the same issue as this: https://github.com/eclipse/deeplearning4j/issues/8335


Add option to hide in the viewer parts that have not changed

I'm trying to use JSONDiffKit to visualize diff between two, relatively big JSON payloads.
Usually the diff is only about 5 properties out of the whole structure.
The current problem I face is out of 480 lines the viewer show only 7 actually have change but I still see the whole structure.
Of course the solution is not as simple as just hiding any line that haven't been changed because you can have an inner object have a single property change but you still want to see the parent properties to understand the context of the change.
Do you think it is possible to add the ability to somehow decide which lines to hide to make the diff more concise and easy to understand?

I think it's ok and it'll be useful in my own business project.
I'm going to add an optional prop hideUnchangedLines for the viewer component, which value will be:
/**
 * all fields are optional,
 * default is `false` which means it won't hide unchanged lines.
 */
type HideUnchangedLines = false | {
  /**
   * if there are continuous unchanged lines exceeding the limit, they should be hidden.
   * default is `6`
   */
  threshold?: number;
  /**
   * if there are `x` continuous unchanged lines to be hidden, we can keep displaying some lines.
   * default is `3`
   */
  margin?: number;
  /**
   * the custom renderer of the "expand" line
   * default renderer will produce the following buttons in this line:
   * [⬆ Show 20 lines] [↕ Show all unchanged lines] [⬇ Show 20 lines]
   */
  expandLineRenderer?: ({
    /**
     * if this is `true`, you can show a "⬆ Show xx lines" button
     */
    hasLinesBefore: boolean;
    /**
     * if this is `true`, you can show a "⬇ Show xx lines" button
     */
    hasLinesAfter: boolean;
    /**
     * call this function to expand `lines` lines before,
     * if there are not enough lines before, it will expand all lines before.
     */
    onExpandBefore: (lines: number) => void;
    /**
     * call this function to expand `lines` lines after,
     * if there are not enough lines after, it will expand all lines after.
     */
    onExpandAfter: (lines: number) => void;
    /**
     * call this function to expand all lines in this continuous part.
     */
    onExpandAll: () => void;
  }) => void;
}

For example, if I call the viewer component like this:
<Viewer
  {...otherViewerProps}
  hideUnchangedLines={{ threshold: 6, margin: 3 }}
/>

And the data (the unchanged parts) is like this:
1
2
3
4
5
6
7
8
9
10

Which will produce the output like this:
1
2
3
[⬆ Show 20 lines] [↕ Show all unchanged lines] [⬇ Show 20 lines]
8
9
10

Here are 2 screenshots for preview, the feature itself is still under development:

Hi, you can pass the hideUnchangedLines={true} prop to the <Viewer> component to support hiding the unchanged lines. I think the default config is enough in most situations, or you can render your own "expand" line using the given API (please refer to type definitions).


A fatal error occurred: Unable to verify flash chip connection (No serial data received.).

Board was working fine until it started to give me this error:
Sketch uses 886997 bytes (45%) of program storage space. Maximum is 1966080 bytes.
Global variables use 52960 bytes (16%) of dynamic memory, leaving 274720 bytes for local variables. Maximum is 327680 bytes.
esptool.py v4.5.1
Serial port /dev/cu.usbmodem223401
Connecting...
Chip is ESP32-S3 (revision v0.1)
Features: WiFi, BLE
Crystal is 40MHz
MAC: 34:85:18:71:7e:78
Uploading stub...
Running stub...
Stub running...

A fatal error occurred: Unable to verify flash chip connection (No serial data received.).
Failed uploading: uploading error: exit status 2

Other times the error is:
 fatal error occurred: No serial data received.
Failed uploading: uploading error: exit status 2

How can I reset it? I have trying different cables, sketches, settings, everything but no success.

https://github.com/Xinyuan-LilyGO/T-Display-S3#quick-start

I have tried it so many times without success (hold the Boot button, connect the USB cable, press RESET, release Boot, release Reset). I have noticed that the board time changes from ESP32S3 Dev module to u-blox NORA-W10.

No, insert USB, press and hold BOOT, do not release the boot button, then press RST. Finally, release the boot

I am probably doing something wrong:

I open Arduino IDE with my sketch
I select board ESP32S3 Dev Module
No serial port is select yet because the board is not connected
I connect the board via the USB cable
I press and hold the BOOT button
While still pressing the BOOT button, I press RST
I release the BOOT button
I release the RST - when should I do that?
Board shows in Arduino IDE as u-blox NORA-W10 and auto-captures a serial port (cu.usbmodem223401)
I run the code and get the same error message again :(

connect the board via the USB - cable
press and hold the BOOT button
While still pressing the BOOT button,  press RST
3.release the RST
4.release the BOOT button

You are the best! Thank you!

No, insert USB, press and hold BOOT, do not release the boot button, then press RST. Finally, release the boot

For you that arrived here via search engines:
are you stuck in this loop?
Connecting......................................

A fatal error occurred: Failed to connect to ESP32-S3: No serial data received.
For troubleshooting steps visit: https://docs.espressif.com/projects/esptool/en/latest/troubleshooting.html
Failed uploading: uploading error: exit status 2

The quoted message is the solution! This finally worked for me, thanks

Thank you, thank you! The Boot - reset - release worked for me too!

Awesome!  It worked for me too.
Its strange that this problem just presented itself and has occurred with multiple esp32-s3 xiao units,  Even sending the basic 'blink' program had the issue.  Smells like a windows update problem.  The problem started after an update,  Hmmm.

update.....

Awesome! It worked for me too. Its strange that this problem just presented itself and has occurred with multiple esp32-s3 xiao units, Even sending the basic 'blink' program hxperd the issue. Smells like a windows update problem. The problem started after an update, Hmmm.

Update....After some more testing I discovered the error occurred with both esp32s3 and esp32c3 boards.  I checked for windows updates and there was one available.  The update included a BIOS update too, but that is probably just a Dell thing.  Either way....after installing the update the problem has completely disappeared.