[webview_flutter] Rendering issue on Web in windows 7 and chrome os

What package does this bug report belong to?
webview_flutter
What target platforms are you seeing this bug on?

Yes
Dependency versions
pubspec.lock
# Generated by pub
# See https://dart.dev/tools/pub/glossary#lockfile
packages:
  async:
    dependency: transitive
    description:
      name: async
      sha256: "947bfcf187f74dbc5e146c9eb9c0f10c9f8b30743e341481c1e2ed3ecc18c20c"
      url: "https://pub.dev"
    source: hosted
    version: "2.11.0"
  boolean_selector:
    dependency: transitive
    description:
      name: boolean_selector
      sha256: "6cfb5af12253eaf2b368f07bacc5a80d1301a071c73360d746b7f2e32d762c66"
      url: "https://pub.dev"
    source: hosted
    version: "2.1.1"
  characters:
    dependency: transitive
    description:
      name: characters
      sha256: "04a925763edad70e8443c99234dc3328f442e811f1d8fd1a72f1c8ad0f69a605"
      url: "https://pub.dev"
    source: hosted
    version: "1.3.0"
  clock:
    dependency: transitive
    description:
      name: clock
      sha256: cb6d7f03e1de671e34607e909a7213e31d7752be4fb66a86d29fe1eb14bfb5cf
      url: "https://pub.dev"
    source: hosted
    version: "1.1.1"
  collection:
    dependency: transitive
    description:
      name: collection
      sha256: ee67cb0715911d28db6bf4af1026078bd6f0128b07a5f66fb2ed94ec6783c09a
      url: "https://pub.dev"
    source: hosted
    version: "1.18.0"
  cupertino_icons:
    dependency: "direct main"
    description:
      name: cupertino_icons
      sha256: ba631d1c7f7bef6b729a622b7b752645a2d076dba9976925b8f25725a30e1ee6
      url: "https://pub.dev"
    source: hosted
    version: "1.0.8"
  fake_async:
    dependency: transitive
    description:
      name: fake_async
      sha256: "511392330127add0b769b75a987850d136345d9227c6b94c96a04cf4a391bf78"
      url: "https://pub.dev"
    source: hosted
    version: "1.3.1"
  flutter:
    dependency: "direct main"
    description: flutter
    source: sdk
    version: "0.0.0"
  flutter_lints:
    dependency: "direct dev"
    description:
      name: flutter_lints
      sha256: "3f41d009ba7172d5ff9be5f6e6e6abb4300e263aab8866d2a0842ed2a70f8f0c"
      url: "https://pub.dev"
    source: hosted
    version: "4.0.0"
  flutter_test:
    dependency: "direct dev"
    description: flutter
    source: sdk
    version: "0.0.0"
  flutter_web_plugins:
    dependency: transitive
    description: flutter
    source: sdk
    version: "0.0.0"
  leak_tracker:
    dependency: transitive
    description:
      name: leak_tracker
      sha256: "3f87a60e8c63aecc975dda1ceedbc8f24de75f09e4856ea27daf8958f2f0ce05"
      url: "https://pub.dev"
    source: hosted
    version: "10.0.5"
  leak_tracker_flutter_testing:
    dependency: transitive
    description:
      name: leak_tracker_flutter_testing
      sha256: "932549fb305594d82d7183ecd9fa93463e9914e1b67cacc34bc40906594a1806"
      url: "https://pub.dev"
    source: hosted
    version: "3.0.5"
  leak_tracker_testing:
    dependency: transitive
    description:
      name: leak_tracker_testing
      sha256: "6ba465d5d76e67ddf503e1161d1f4a6bc42306f9d66ca1e8f079a47290fb06d3"
      url: "https://pub.dev"
    source: hosted
    version: "3.0.1"
  lints:
    dependency: transitive
    description:
      name: lints
      sha256: "976c774dd944a42e83e2467f4cc670daef7eed6295b10b36ae8c85bcbf828235"
      url: "https://pub.dev"
    source: hosted
    version: "4.0.0"
  matcher:
    dependency: transitive
    description:
      name: matcher
      sha256: d2323aa2060500f906aa31a895b4030b6da3ebdcc5619d14ce1aada65cd161cb
      url: "https://pub.dev"
    source: hosted
    version: "0.12.16+1"
  material_color_utilities:
    dependency: transitive
    description:
      name: material_color_utilities
      sha256: f7142bb1154231d7ea5f96bc7bde4bda2a0945d2806bb11670e30b850d56bdec
      url: "https://pub.dev"
    source: hosted
    version: "0.11.1"
  meta:
    dependency: transitive
    description:
      name: meta
      sha256: bdb68674043280c3428e9ec998512fb681678676b3c54e773629ffe74419f8c7
      url: "https://pub.dev"
    source: hosted
    version: "1.15.0"
  path:
    dependency: transitive
    description:
      name: path
      sha256: "087ce49c3f0dc39180befefc60fdb4acd8f8620e5682fe2476afd0b3688bb4af"
      url: "https://pub.dev"
    source: hosted
    version: "1.9.0"
  plugin_platform_interface:
    dependency: transitive
    description:
      name: plugin_platform_interface
      sha256: "4820fbfdb9478b1ebae27888254d445073732dae3d6ea81f0b7e06d5dedc3f02"
      url: "https://pub.dev"
    source: hosted
    version: "2.1.8"
  sky_engine:
    dependency: transitive
    description: flutter
    source: sdk
    version: "0.0.99"
  source_span:
    dependency: transitive
    description:
      name: source_span
      sha256: "53e943d4206a5e30df338fd4c6e7a077e02254531b138a15aec3bd143c1a8b3c"
      url: "https://pub.dev"
    source: hosted
    version: "1.10.0"
  stack_trace:
    dependency: transitive
    description:
      name: stack_trace
      sha256: "73713990125a6d93122541237550ee3352a2d84baad52d375a4cad2eb9b7ce0b"
      url: "https://pub.dev"
    source: hosted
    version: "1.11.1"
  stream_channel:
    dependency: transitive
    description:
      name: stream_channel
      sha256: ba2aa5d8cc609d96bbb2899c28934f9e1af5cddbd60a827822ea467161eb54e7
      url: "https://pub.dev"
    source: hosted
    version: "2.1.2"
  string_scanner:
    dependency: transitive
    description:
      name: string_scanner
      sha256: "556692adab6cfa87322a115640c11f13cb77b3f076ddcc5d6ae3c20242bedcde"
      url: "https://pub.dev"
    source: hosted
    version: "1.2.0"
  term_glyph:
    dependency: transitive
    description:
      name: term_glyph
      sha256: a29248a84fbb7c79282b40b8c72a1209db169a2e0542bce341da992fe1bc7e84
      url: "https://pub.dev"
    source: hosted
    version: "1.2.1"
  test_api:
    dependency: transitive
    description:
      name: test_api
      sha256: "5b8a98dafc4d5c4c9c72d8b31ab2b23fc13422348d2997120294d3bac86b4ddb"
      url: "https://pub.dev"
    source: hosted
    version: "0.7.2"
  vector_math:
    dependency: transitive
    description:
      name: vector_math
      sha256: "80b3257d1492ce4d091729e3a67a60407d227c27241d6927be0130c98e741803"
      url: "https://pub.dev"
    source: hosted
    version: "2.1.4"
  vm_service:
    dependency: transitive
    description:
      name: vm_service
      sha256: "5c5f338a667b4c644744b661f309fb8080bb94b18a7e91ef1dbd343bed00ed6d"
      url: "https://pub.dev"
    source: hosted
    version: "14.2.5"
  web:
    dependency: "direct main"
    description:
      name: web
      sha256: cd3543bd5798f6ad290ea73d210f423502e71900302dde696f8bff84bf89a1cb
      url: "https://pub.dev"
    source: hosted
    version: "1.1.0"
  webview_flutter:
    dependency: "direct main"
    description:
      name: webview_flutter
      sha256: "889a0a678e7c793c308c68739996227c9661590605e70b1f6cf6b9a6634f7aec"
      url: "https://pub.dev"
    source: hosted
    version: "4.10.0"
  webview_flutter_android:
    dependency: transitive
    description:
      name: webview_flutter_android
      sha256: "74693a212d990b32e0b7055d27db973a18abf31c53942063948cdfaaef9787ba"
      url: "https://pub.dev"
    source: hosted
    version: "4.0.0"
  webview_flutter_platform_interface:
    dependency: "direct main"
    description:
      name: webview_flutter_platform_interface
      sha256: d937581d6e558908d7ae3dc1989c4f87b786891ab47bb9df7de548a151779d8d
      url: "https://pub.dev"
    source: hosted
    version: "2.10.0"
  webview_flutter_web:
    dependency: "direct main"
    description:
      path: "plugins/webview_flutter_web"
      relative: true
    source: path
    version: "0.2.3+2"
  webview_flutter_wkwebview:
    dependency: transitive
    description:
      name: webview_flutter_wkwebview
      sha256: d4034901d96357beb1b6717ebf7d583c88e40cfc6eb85fe76dd1bf0979a9f251
      url: "https://pub.dev"
    source: hosted
    version: "3.16.0"
sdks:
  dart: ">=3.5.3 <4.0.0"
  flutter: ">=3.24.0"

Steps to reproduce
Steps to reproduce:
Step 1. Visit https://webview-render-issue.web.app/
Step 2: Open console -> More tools -> Network Conditions
Step 3: Select Custom User Agent as below and reload page
Custom User agent: Mozilla/5.0 (X11; CrOS x86_64 14989.107.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Step 4: Interact with the listeview/dropdown/alert box.
Expected results
UI render jank issue should not happen.
Actual results
Users are facing ui render jank issues when webview is present on the screen.
Tried with flutter version 3.24 and 3.27
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:webview_flutter_platform_interface/webview_flutter_platform_interface.dart';
import 'package:webview_flutter_web/webview_flutter_web.dart';

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

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

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
        title: 'Flutter WebView Sample',
        theme: ThemeData(
          primarySwatch: Colors.blue,
        ),
        home: const MyHomePage());
  }
}

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

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

class _MyHomePageState extends State<MyHomePage> {
  final TextEditingController _controller = TextEditingController();
  final WebWebViewController controller = WebWebViewController(
    WebWebViewControllerCreationParams(),
  )..loadRequest(
      LoadRequestParams(
        uri: Uri.parse('https://www.google.com'),
      ),
    );

  String _currentUrl = 'https://www.google.com'; // Default URL

  List<String> scripts =
      List<String>.generate(50, (index) => (index + 1).toString());

  String _selectedValue = 'Option 1';
  final List<String> _dropdownItems = ['Option 1', 'Option 2', 'Option 3'];

  @override
  void initState() {
    super.initState();
    _controller.text = _currentUrl;
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Flutter WebView Sample'),
      ),
      body: Column(
        children: [
          const Text('Steps to reproduce:'),
          const Text(
              'Step 1: Open console -> More tools -> Network Conditions '),
          const Text(
              'Step 2: Select Custom User Agent as below and reload page'),
          const SelectableText(
            'Mozilla/5.0 (X11; CrOS x86_64 14989.107.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36',
          ),
          Expanded(
            child: Row(
              children: [
                // First Column: TextField and WebView
                Expanded(
                  flex: 2,
                  child: Column(
                    children: [
                      Padding(
                        padding: const EdgeInsets.all(8.0),
                        child: TextField(
                          controller: _controller,
                          decoration: InputDecoration(
                            labelText: 'Enter URL',
                            suffixIcon: IconButton(
                              icon: const Icon(Icons.arrow_forward),
                              onPressed: () {
                                String url = _controller.text;
                                setState(() {
                                  _currentUrl = url;
                                });
                                controller.loadRequest(
                                  LoadRequestParams(
                                    uri: Uri.parse(url),
                                  ),
                                );
                              },
                            ),
                          ),
                          onSubmitted: (value) {
                            setState(() {
                              _currentUrl = value;
                            });
                            controller.loadRequest(
                              LoadRequestParams(
                                uri: Uri.parse(value),
                              ),
                            );
                          },
                        ),
                      ),
                      Expanded(
                        child: Material(
                          color: Colors.white,
                          elevation: 4,
                          child: PlatformWebViewWidget(
                            PlatformWebViewWidgetCreationParams(
                                controller: controller),
                          ).build(context),
                        ),
                      ),
                    ],
                  ),
                ),
                // Second Column: ListView of scripts
                Expanded(
                  flex: 1,
                  child: Column(
                    children: [
                      Expanded(
                        child: ListView.builder(
                          itemCount: scripts.length,
                          itemBuilder: (context, index) {
                            return ListTile(
                              title: Text(scripts[index]),
                              onTap: () {},
                            );
                          },
                        ),
                      ),
                      Padding(
                        padding: const EdgeInsets.all(8.0),
                        child: DropdownButton<String>(
                          value: _selectedValue,
                          onChanged: (String? newValue) {
                            setState(() {
                              _selectedValue = newValue!;
                            });
                          },
                          items: _dropdownItems
                              .map<DropdownMenuItem<String>>((String value) {
                            return DropdownMenuItem<String>(
                              value: value,
                              child: Text(value),
                            );
                          }).toList(),
                        ),
                      ),
                      ElevatedButton(
                        onPressed: () {
                          _showAlertDialog(context);
                        },
                        child: const Text('Show Alert Dialog'),
                      ),
                    ],
                  ),
                ),
              ],
            ),
          ),
        ],
      ),
    );
  }

  void _showAlertDialog(BuildContext context) {
    showDialog(
      context: context,
      builder: (BuildContext context) {
        return AlertDialog(
          title: const Text('Alert'),
          content: const Text('This is an alert dialog box.'),
          actions: <Widget>[
            TextButton(
              child: const Text('OK'),
              onPressed: () {
                Navigator.of(context).pop();
              },
            ),
          ],
        );
      },
    );
  }
}

Screenshots or Videos

Screenshots / Video demonstration
https://github.com/user-attachments/assets/069dccd8-34b8-49b8-aa8d-928003c5ccd3

Logs
Logs
------

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.3, on macOS 15.2 24C101 darwin-arm64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] VS Code (version 1.96.1)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

++ @stuartmorgan @loic-sharma

Hi @yashharkhani, which version of 3.27, did you test? I tested 3.27.1 and aside the dialog scrim being darker than it usually is, I don't see any rendering issues. Please also note that the platforms you're using are no longer supported by flutter. See
https://docs.flutter.dev/reference/supported-platforms
https://github.com/user-attachments/assets/8d953141-380a-4750-8ac2-2b9fe33c9c29


Help! This plugin makes me way too excited!

Since I found out about this plugin, I'm constantly in awe.
Not only does it federate my blog posts into the fediverse and makes my blog appear as an actor, it also displays comments that are made from mastodon and co. below my articles! Who would do such bold things??
I'm not sure if I can handle so much awesomeness... Please let me know, if this is intended behaviour.

Can confirm.

Can reproduce w. WordPress 5.3 and SemPress theme.

I installed it.. i don't known how to use it.. :(

@albjeremias enable it, create a post and voila, you can find your post on mastodon if you search for the post url there. You can also follow your blog from any fediverse platform.

Me too! I can't stop thinking of all the possibilities this plugin can afford. Much love!

I wonder what the plugin cannot do - ie what part of Activitypub is one really missing out on? I am kinda new to both the fediverse and the plugin. For example, could I follow/reply to others and access my account from mastodon-compatible apps? Like, no need to have a frontend for it, but have this as backend and apps as frontend?
My main reason for using this plugin was to have a nice fediverse handle<EMAIL_ADDRESS>instead of some<EMAIL_ADDRESS>but of course, I am surprised how much more it does.

Since the recent updates out of Automattic this issue has reemerged for me. My psychiatrist had previously been helping me custom patch my setup, but now it's fully broken. Plz address in core.

Next release is going to have this issue again judging from the recent commits.

+1 big issue for me


Cannot handle attribute's handles keyword

Given a code like:
#| Description
has A $.a handles<something>;

Its markdown output is:
class Attribute+{<anon|2>}.new(handles => "something")
------------------------------------------------------

Description

Thanks for reporting this!
Pod::To::Text and Pod::To::HTML have the same bug. It may not be possible to fix with the current Rakudo Pod implementation, but I'll see. I'm open to suggestions if anyone knows another class that does it correctly or has a fix handy.


Script injection broken by 1362f49

Here's how to reproduce this:

On one shell, run:
./replay.py --port=4080 --ssl_port=4443 --no-dns_forwarding --record foo.wpr

On a second shell, run:
timeout 10 $MY_FAVORITE/chrome --ignore-certificate-errors \
  --host-resolver-rules="MAP *:80 localhost:4080, MAP *:443 localhost:4443, EXCLUDE localhost" \
  'https://www.google.de/search?q=v8'

Ctrl-C the replay.py command on the first shell.
At this point, you can verify that some CHUNK_BOUNDARY are (probably) not being properly handled by running:
./httparchive.py cat foo.wpr | grep BOUNDARY | head -2

Prepare a simple script to be injected.  Run:
cat > inj.js <<EOF
(function() {
  window.addEventListener("load", function() {
    alert("Running the injected script!");
  });
})();
EOF

On the first shell, run:
./replay.py --port=4080 --ssl_port=4443 --no-dns_forwarding --inject_scripts=deterministic.js,inj.js --use_closest_match foo.wpr

On the second shell, run:
$MY_FAVORITE/chrome --ignore-certificate-errors \
  --host-resolver-rules="MAP *:80 localhost:4080, MAP *:443 localhost:4443, EXCLUDE localhost" \
  'https://www.google.de/search?q=v8'

At step 6, you should see the alert message "Running the injected script!", but you will not.
If you repeat step 6 after you have checkout out commit d23e5a8, it will work as expected.  The commit that revealed the problem (I believe, not the one that caused it) is 1362f49.

Fix uploaded here: https://codereview.appspot.com/297440043/

Thanks nickie for the fix!


FileStorage perhaps has incorrect typing metadata

Describe how to replicate the bug.
from flask import request
# snip some app-specific logic...
fileitem = request.files["file"]
blob = fileitem.read()

and then:
mypy test.py
test.py:3: error: "FileStorage" has no attribute "read"

Environment:

Python version: 3.9.5
Flask version: 2.0.1

I'm also experiencing this. Just to be clear @davidism are you saying werkzeug has its metadata correctly set, and that this is a bug/limitation in mypy itself?

Looks like I didn't define the __getattr__ method in the datastructures.pyi file, so the type system isn't picking it up (it doesn't look at the source if there's a pyi file). Happy to consider a PR that fixes the pyi file.


@mixin Builder instead of @mixin Eloquent?

I'm on Laravel 9 but still getting @mixin Eloquent instead of @mixin Builder from ide-helper after "artisan ide-helper:models", which doesn't work for static methods like ::where() on my models. Any idea how tell ide-helper to do that? Or another way to achieve that? Thanks! :)

Just waiting here with you for a fix...

any news on this?

same issue :(

Hi, This is a workaround, but try adding @mixin \Eloquent to the actual model class
e.g.:
/**
 * @mixin \Eloquent
 */
class Dummy extends Model
{
    // ...snip
}


TIM-239 feat(home): events and function

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8  👈
#7
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @xdjeremy and the rest of your teammates on  Graphite

Merge activity

Jun 6, 2:37 AM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


feat(#major); aave v3; add aave v3 ethereum

Test: https://okgraph.xyz/?q=messari%2Faave-v3-ethereum

@ishraq8 could you make a QA on this next week?

@dmelotik Forget next week, looked through the subgraph. Everything looks great and makes sense. It is hard to assess the stable APR rate but it seems to be reasonable. Also front end shows WETH & ETH, but both are separate pools
Only issue, all markets showing up as Not active

QA issues resolved just need code review.


Add team assignment

This adds a new script command, teams, with sub-commands suggest and send for suggesting and sending team allocations respectively.
This is what the email looks like:

@jaredkhan could you review?

Maybe add a quick note in the README while we're here too


Error: Internal torch error: Legacy model format is not supported on mobile while loading efficientnet-* weights

Command cargo run --example pretrained-models -- efficientnet-b1.ot img.png
Error:
Error: Internal torch error: Legacy model format is not supported on mobile.
Exception raised from deserialize at ../torch/csrc/jit/serialization/import.cpp:276 (most recent call first):
...

Similar issue:

#469

Using:
tch-rs ver commit c0bb7e7325d943dc46e964a5ece7f19505ec7c69
sha256sum:
281d75bb97dab428c003f7ba69a60ca8a04f40e50d40287b7001338ca5dafa8c  efficientnet-b0.ot
1ac0a803455f81aedef8a631b802e4f55a3e629b45eeb5ba48ea7fab94938fff  efficientnet-b1.ot
daf3e07e1e2facbe681229a94a4906bcefd314a4d60739abbfbc7d840613e186  efficientnet-b2.ot
30909974ca657527fd352d5478b63404e15b893f4f5dfc73481bef3600f29f5b  efficientnet-b3.ot
5b1b51e0213139b5c9fd34ce9e5c30a403b802106e005e5a823c5086d76d49d6  efficientnet-b4.ot

This should be fixed now, I've also used this opportunity to switch these models to use the safetensors format which should be stable over time (whereas the .ot format was changing which created the issue), the url for the model weights can be found here and the Python script to generate the safetensors weight files here.
Feel free to re-open if you run into further issues.


UCVM version in install script

20 0.307 All Flag: True
#20 0.307 dynamic Flag: True
#20 0.307
#20 0.307 UCVM 21.7.0 Installation
#20 0.307 Copyright (C) 2021 SCEC. All rights reserved.
#20 0.307 Using local setup.list and system.list ....
#20 0.307 Now check system specific conditions.
Also, the copyright is by USC rather than SCEC

Updated in ucvm_setup.py


Add feature in program menu for configuring with iwpriv.

sudo t2u-driver should show:

Enable/Disable adapter
Configuration:

This configuration should show sequentialy all the parameters included in RT2870STA.dat allowing the user to configure completely the adapter through iwpriv commands.

Merged in v.0.2


⚠ Bofrost.at has degraded performance

In 3269bc6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 868 ms

Resolved: Bofrost.at performance has improved in a0fba24.


Creating DiCE Explanation Instance

I am attempting to use DiCE on a classification problem and form a DiCE Explanation Instance as such:
# DiCE explanation instance
exp = dice_ml.Dice(d,m)

However, after spending time on this I am running into difficulty in defining m for this purpose.
I have created a neural network configuration and the model trained successfully.
>>> from tensorflow.keras.models import Sequential
>>> from tensorflow.python.keras.layers import Dense
>>> from tensorflow.python.keras.wrappers.scikit_learn import KerasRegressor

>>> sess = tf.InteractiveSession()

>>> # Fitting a dense neural network model
>>> ann_model = Sequential()
>>> ann_model.add(Dense(20, input_shape=(x1.shape[1],), activation=tf.nn.relu))
>>> ann_model.add(Dense(1, activation=tf.nn.sigmoid))
>>> ann_model.compile(loss='binary_crossentropy', metrics=['accuracy'])
>>> ann_model.fit(x1, y, validation_split=0.20, epochs=100)

Epoch 1/100
320/320 [==============================] - 0s 226us/sample - loss: 1.3851 - acc: 0.6750 - val_loss: 1.1140 - val_acc: 0.6375
Epoch 2/100
320/320 [==============================] - 0s 74us/sample - loss: 0.9420 - acc: 0.6969 - val_loss: 0.9910 - val_acc: 0.6375
...

The model is of the type:
tensorflow.python.keras.engine.sequential.Sequential

However, when I attempt to generate the DiCE model:
# Generate the DiCE model for explanation
m = model.Model(model=ann_model)

I get the error:
NameError                                 Traceback (most recent call last)

<ipython-input-15-ec54e1758be4> in <module>()
      1 # Generate the DiCE model for explanation
----> 2 m = model.Model(model=ann_model)

NameError: name 'model' is not defined

For reference, I am using TensorFlow 1.15 and Python 3.6.9. Any help or advice greatly appreciated.

Hi, can you try m = dice_ml.Model(model=ann_model)? - replace model with dice_ml.

Hi, can you try m = dice_ml.Model(model=ann_model)? - replace model with dice_ml.

That worked perfectly, thank you for the quick reply! For reference, I now get a model of type:
<dice_ml.model_interfaces.keras_tensorflow_model.KerasTensorFlowModel at 0x7f4409307cf8>


Route::resource overrides already defined Route::resource for given Model

Laravel Version: 5.3
PHP Version: 7.1

Description:
I'm in the middle of separating routes in already existing system - so far the resource groups were separated by route files (all following routes are defined within routes/admin.php, so they don't clash with frontend routes), though we're moving towards additional separation via URL as well. To illustrate - route for URL: /admin/journeys/123/show/ exists as:
Route::group(['prefix' => 'admin'], function(){
Route::resource('journeys', '\App\Http\Controllers\Admin\JourneyController');
});

which generates defined routes, amongst them journeys.show. Additionally, for time being, I'm adding:
Route::group(['as' => 'admin.', 'prefix' => 'admin'], function(){
Route::resource('journeys', '\App\Http\Controllers\Admin\JourneyController');
});

which generates defined routes, amongst them admin.journeys.show. The problem is that it overrides already existing resource routes - route list shows that my journeys.show doesn't exist anymore. The desired behaviour is to have them exist alongside.
Steps To Reproduce:

Create resource route for given model
Create another resource route for given model within 'as' => 'prefix.' group

I can't reproduce your issue on a fresh install of Laravel 5.3.
I created a Car model and two resources controller:

\App\Http\Controller\CarsController
\App\Http\Controller\Admin\CarsController

I've then created the routes like so:
<?php

Route::resource('cars', '\App\Http\Controllers\CarsController');

Route::group(['prefix' => 'admin', 'as' => 'admin.'], function() {
    Route::resource('cars', '\App\Http\Controllers\Admin\CarsController');
});

Routes list:

That's because you're creating two routes that point to two different controllers that handle the same Model. As in OP, I'm generating routes as so:
Route::group(['prefix' => 'admin'], function(){
	Route::resource('journeys', '\App\Http\Controllers\Admin\JourneyController');
});

Route::group(['as' => 'admin.', 'prefix' => 'admin'], function(){
	Route::resource('journeys', '\App\Http\Controllers\Admin\JourneyController');
});

Route list:
[img]http://i.imgur.com/hcYI4E2.png[/img]
I can't go through all existing controllers and template files and change route('journeys.show') to route('admin.journeys.show') (it will happen at some point though), and hope there's a functionality that enables me to do so.

You're using the same prefix two times, which generates two times the same URI.
You can't have the same URI pointing to two different things (even though in your case it seems to be the same thing: the same method on the same controller).
If I'm following you, you just have duplicate routes.
Did I miss something in your initial explanation ?

Let me try to explain one more time - there's a named route journeys.show that is generated by:
Route::group(['prefix' => 'admin'], function(){
	Route::resource('journeys', '\App\Http\Controllers\Admin\JourneyController');
});

used in unknown number of places in both templates and controllers; it resolves to admin/journeys/123/.
I want to add admin.journeys.show and use it per route('admin.journeys.show', $whatever) and resolve to admin/journeys/123/. However, if I'll add it to existing route list, per:
Route::group(['prefix' => 'admin'], function(){
	Route::resource('journeys', '\App\Http\Controllers\Admin\JourneyController');
});

Route::group(['as' => 'admin.', 'prefix' => 'admin'], function(){
	Route::resource('journeys', '\App\Http\Controllers\Admin\JourneyController');
});

the second declaration will overshadow the first one. I want to be able to use both journeys.show and admin.journeys.show to point to the same controller, and resolve to the same URL.

So you basically need some kind of way to alias a route ?

Yes indeed :)

Laravel 5.3 is not supported, please test on 5.5 and report if your issue still exists.
Thanks


How can I obtain the results for the AHA 17-segment polar map?

Hello,
I am currently working on a very meaningful project and need some assistance. Specifically, I am looking to map 3D LV image onto a 2D polar map. Despite trying various methods, I have not achieved the desired results. Could you please share the code related to the AHA 17-segment polar map?
Thank you!

Hi, check this “https://carimas.fi/carimas-research/”


Notify of issuance data encoding costs / allow dust level tweaks

Currently when a user enters a long description, the data encoding switches to multisig, which requires additional BTC in dust outputs to encode the asset description data.
Need to add a notification / confirmation window to let user know if the additional data encoding costs

also should add option to freewallet to allow users to specify their own dust level (default to 1000 for multisig) in system settings.... so users can tweak their dust limits as needed

update should be in next release


Checklist could be different than 2 entries

https://github.com/nyxnor/onionservice/blob/d429606e546b65321ce8b7191cd23702f8871dbb/onionservice-tui#L70
Issue
It seems (I use whiptail instead of dialog) that the "2" representing two entries is hardcoded in this line of code. However, someone may have only one or several entries.
Describe the solution you'd like
For the above reasons, I propose to replace "2" with "$i". To be sure that the list is not getting too long and is fitting to a little LCD display, the entire code could look like that:
if [ $i -gt 11 ]; then i=11; fi
CHOICE_SERVICE="$(dialog --clear --backtitle "${BACKTITLE}" --title "${TITLE}" --"${DIALOG_TYPE}" "${MENU}" \
"$((i+8))" 80 ${i} ${SERVICE_LIST} 2>&1 >/dev/tty)"

Additional remarks
I didn't submit a pull request yet because I don't use dialog, and I'm not entirely sure it reacts like whiptail.

On dialog, one or more services works fine
1 service:

3 services:

LCD display
Yes, max will be pinned, so this change would be accepted, if possible to consider also the AUTH_* and any other relevant option to pin/fix lines regarding clients names (why 11?).
    [ "${i}" -gt 11 ] && i=11
    # shellcheck disable=SC2086
    CHOICE_SERVICE="$(dialog --clear --backtitle "${BACKTITLE}" --title "${TITLE}" --"${DIALOG_TYPE}" "${MENU}" \
    "$((i+8))" 80 ${i} ${SERVICE_LIST} 2>&1 >/dev/tty)"

11 because this is the maximum number I can squeeze into my PiTFT Plus 480x320 3.5".


When the password is changed, it is always displayed "User's token has expired"

In GitLab by @luotian-github on Aug 22, 2017, 11:47

In GitLab by @liyanq528 on Nov 8, 2019, 07:35
fixed by push request #206

In GitLab by @sokril on Nov 8, 2019, 08:08
closed

In GitLab by @sokril on Nov 8, 2019, 08:08
assigned to @liyanq528

In GitLab by @liyanq528 on Nov 11, 2019, 07:42
mentioned in merge request !90

In GitLab by @liyanq528 on Nov 11, 2019, 08:05
mentioned in merge request !196

In GitLab by @liyanq528 on Nov 13, 2019, 03:52
mentioned in merge request !86

In GitLab by @liyanq528 on Nov 13, 2019, 06:33
mentioned in merge request !72


Repo names are not case-sensitive

Gitea version (or commit ref): 1.3.2
Git version: 2.7.4
Operating system:
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
When new repos are created and a home folder is generated, the naming convention is to name the folder using all lower-case letters.
This is not reflected in the UI.
As an example I created a repo called "fetchDNS".
The created folder on the server was "fetchdns"; this results in an error that the directory doesn't exist.
...
Screenshots

On the server:
:~/repos/methodbox$ ls
total 12K
drwxr-xr-x 3 git git 4.0K May 22 00:50 .
drwxrwxr-x 3 git git 4.0K May 22 00:49 ..
drwxr-xr-x 7 git git 4.0K May 22 00:50 fetchdns.git

@mibcat it seems to me as if you don't use the correct url. Why /volume1/gitea/? Usually the urls look like ssh://git@nas2:<user/org>/<repo>.git

gitea is running on a Synology Diskstation DS116.
/volume1/gitea is the root directory for all my repositories.
I have to put this path to all URL otherwise git access via ssh fails.
Maybe a misconfiguration ??? Any help to get rid of this pre-path is appreciated.
But as I can see this is independent from the capitalize problem.
A push to the correct URL is working as you can see in screen shot above.

@mibcat a response on another issue that @daviian gave a great response to may help you:

To clarify the SSH url: the stuff after the colon is for internal use in gitea. Gitea itself provides the mapping to the actual repository location.
So what you do when using the original path is bypassing gitea, which shouldn’t be possible at all!
One reason this happens is that you set your SSH key in the authorized_keys file yourself with e.g. ssh-copy-id. However that’s wrong and you should definitely let gitea handle this file.
Furthermore everytime a user adds an SSH key via gitea web-UI the authorized_keys file gets overwritten.
Another thing is that usually the user from the SSH url is git or whatever is configured in the app.ini and not the actual gitea-user itself.
The first thing you should try now is to go into the Admin settings on the gitea UI and execute:
Update the ‘.ssh/authorized_keys’ file with Gitea SSH keys. (Not needed for the built-in SSH server.)
Resynchronize pre-receive, update and post-receive hooks of all repositories.

After that you HAVE to set your SSH keys through gitea web-UI.
Hopefully that solves your issue!

It seems the SSH issue and this issue are kind of related:
If you don’t have a key setup in Gitea UI you can’t use their URL because you can’t connect via their server daemon using SSH.
You can use the default Git path of git://hostname:/literal/path/to/repo if you have a key previously configured in ~/.ssh/authorized_keys.
So this begs the question: does Gitea store keys independently, and if so, why?
It seems the solution would be to at least give the user the option of using the default authorized_keys list. Then have the daemon listen for a request to the shortened URL and know that it’s association is with that given directory.
I fail to see why the shortened “Gitea URL” requires authorization independent of existing keys.
git-daemon can be enabled with the default Git install in Linux and allow this type of URL:
git://hostname/gitparentdir/repodir
You configure an absolute path(s) for the daemon to check (which would be prior to the Git parent in my example like /home/username/gitparent) and the URL is simply the host slash relative repo path.
...and it’s case-sensitive by default and works with existing authorized keys.
It seems like you’ve created your own daemon that doesn’t operate quite the same.

@methodbox I can give you a very concise answer to the question why gitea is keeping track of the keys itself.
If you manage your authroized_keys by yourself, e.g. append your public key, use ssh-copy-id or whatever doesn't prevent the person from actually login into the host via ssh.
Gitea prevents that by only allowing the user to use ssh in combination with git.
So if you've a system with many users (e.g. github) it would be a huge security risk letting everyone connect to your server and play around with folders/files and so on.
Furthermore each user would have access to everyones repositories, as for ssh and the server it is the same user, no matter who is trying to clone.
Gitea instead manages the permissions for each user by differ between ssh keys and giving them accurate permissions.

@techknowlogick: thanks for the tip.
I think the problem was that I used the same ssh key for gitea which was previously uploaded via ssh-copy-id for the git user on the machine. gitea added the same key after the already existing key in authorized_keys.
==> If an access to a git repo via ssh take place the first key was taken (not handled by gitea) and therefore the correct full path must be given (including the right capital letters).
The solution for me was to create a new ssh key and upload it to gitea (and removing the git user key)
Then the URL can be something of ssh://git@nas2:<user/org>/<repo>.git
Btw: this solves also the bare repro issue (#3458) ...
But a small drawback remains: using two different keys (for gitea and for git user) leads into problems which depends on the sequence the keys are loaded by ssh-add:

if the gitea key was loaded first, I'm not able to login:

michel@ideapad ~/tmp/Crc16 $ ssh git@nas2 PTY allocation request failed on channel 0 Hi there, You've successfully authenticated, but Gitea does not provide shell access. If this is unexpected, please log in with password and setup Gitea under another user. Connection to nas2 closed.
(maybe for the reason daviian mentioned right above this post)

if the git user key was loaded first, git is not working due to the wrong URL path and gitea does not get the request

OK -I see.
Thanks !

@daviian That makes perfect sense except that it’s incorrect.
With the default Git daemon you can allow Git via SSH without allowing SSH login.
You can also control SSH in a number of other ways (as you can see Gitea is able to do ;) ) so I’m not sure that’s a truly concise answer but it does explain that Gitea’s approach is to do this by default rather than require additional configuration, which I appreciate.
Thanks!

Closing this thread now


PR: Cache correlation coefficient

Currently, the correlation coefficient are calculated for ALL stations every time the target station is changed.
This PR improve this by caching the correlation coefficients in memory and on the disk, so that they are calculated only once.

Hello @jnsebgosselin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file cdprep/gapfill_data/gapfill_weather_algorithm.py:

Line 984:56: E226 missing whitespace around arithmetic operator


Preserve case when parsing domain names

When implementing DNS servers, one might need to return
domain names in exactly the same case as was in the question.
For example, dig eXamPle.cOm. includes
;; ANSWER SECTION:
eXamPle.cOm.		20885	IN	A	<IP_ADDRESS>

Notice the answer section says eXamPle.cOm and not
example.com. This is currently not possible when using this
crate as it automatically converts to lowercase.
Fixes https://github.com/LinkTed/dns-message-parser/issues/12.

@LinkTed I still believe using unicase in the PartialEq implementations instead of to_lowercase is the right way to go for better IDN support (it's faster too) but I'm not sure if you would welcome pulling in an extra dependency. If you are open to that idea I will be happy to submit another pull request.

Hi @LinkTed,

I wil look into the unicase crate and notify you if we should use this crate instead of to_lowercase function.

:+1:

As far as I see, the last commit of unicase was a 12th November 2019. Is the crate currently maintained?

I'm not sure what its current status is but that is the go to crate for this kind of thing in the Rust community as far as I know. It also has a stable version and has no dependencies so it may be that it is mature.

Looks good.


Adding pending changes alerts

📝 Description

Adding pending changes alert to the VM details page
Adding pending changes alert to the CPU & Memory / Hardware devices modal (Boot order modal is missing, will be added in following PR)

Add a brief description

🎥 Demo
https://user-images.githubusercontent.com/67270715/165142608-f22f37c6-9583-40a0-b7e2-cb1837a3f0ff.mp4
Signed-off-by: Aviv Turgeman<EMAIL_ADDRESS>

/retest

/retest

/hold
waiting for https://github.com/kubevirt-ui/kubevirt-plugin/pull/325 to be merged and will add the pending changes alert to the boot order modal as well

/hold cancel

/lgtm


how to load pretrained chainer npz parameter file but modify some layer?

not caffe pretrained file, but chainer pretrained npz file.
I have pretrained a VGG network in chainer loadable npz file format, but add a new FC layer to last layer , and I modify last layer output class_number.
I have already modify the layer name in order to use chainer loadable file to other unaltered layer.
But I failed.
Traceback (most recent call last):
  File "chainercv/trainer/train.py", line 194, in <module>
    main()
  File "chainercv/trainer/train.py", line 85, in main
    mean_file=args.mean)  # 可改为/home/machen/face_expr/result/snapshot_model.npz
  File "/home/machen/face_expr/chainercv/links/model/faster_rcnn/faster_rcnn_vgg.py", line 131, in __init__
    chainer.serializers.load_npz(pretrained_model, self)
  File "/usr/local/anaconda3/lib/python3.6/site-packages/chainer-3.0.0a1-py3.6.egg/chainer/serializers/npz.py", line 140, in load_npz
    d.load(obj)
  File "/usr/local/anaconda3/lib/python3.6/site-packages/chainer-3.0.0a1-py3.6.egg/chainer/serializer.py", line 82, in load
    obj.serialize(self)
  File "/usr/local/anaconda3/lib/python3.6/site-packages/chainer-3.0.0a1-py3.6.egg/chainer/link.py", line 794, in serialize
    d[name].serialize(serializer[name])
  File "/usr/local/anaconda3/lib/python3.6/site-packages/chainer-3.0.0a1-py3.6.egg/chainer/link.py", line 794, in serialize
    d[name].serialize(serializer[name])
  File "/usr/local/anaconda3/lib/python3.6/site-packages/chainer-3.0.0a1-py3.6.egg/chainer/link.py", line 550, in serialize
    data = serializer(name, param.data)
  File "/usr/local/anaconda3/lib/python3.6/site-packages/chainer-3.0.0a1-py3.6.egg/chainer/serializers/npz.py", line 106, in __call__
    dataset = self.npz[key]
  File "/usr/local/anaconda3/lib/python3.6/site-packages/numpy/lib/npyio.py", line 237, in __getitem__
    raise KeyError("%s is not a file in the archive" % key)
KeyError: 'head/score_mod/W is not a file in the archive'

Thank you for posting a question!
However, GitHub issues are for bug reports / feature requests.
Please consider asking in StackOverflow (with chainer tag).


Update

How to categorize this PR?

/area landscaper
/kind enhancement
/priority blocker
What this PR does / why we need it:
Upgrades the controller-runtime dependency to v0.8.1.
Also updates the k8s dependency in apis/go.mod
Special notes for your reviewer:
Release note:

Update controller-runtime dependency

The problem is probably that with github.com/gardener/landscaper/apis v0.0.0 in the main go.mod file, the wrong version of the apis is vendored. Not sure how to resolve this, since I would need to vendor the version that I'm currently working on ...

The problem is probably that with github.com/gardener/landscaper/apis v0.0.0 in the main go.mod file, the wrong version of the apis is vendored. Not sure how to resolve this, since I would need to vendor the version that I'm currently working on ...

looks good when the test is fixed.
Can you directly do a refactoring for that todo now that the new clinet.Object is available: https://github.com/gardener/landscaper/blob/master/test/utils/envtest/types.go#L20

done

looks good when the test is fixed.
Can you directly do a refactoring for that todo now that the new clinet.Object is available: https://github.com/gardener/landscaper/blob/master/test/utils/envtest/types.go#L20

done


Add graphql Setup with processes resource

Codecov Report

Merging #245 (7a1d100) into main (6674bbd) will decrease coverage by 1.12%.
The diff coverage is 53.57%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #245      +/-   ##
==========================================
- Coverage   77.62%   76.51%   -1.12%
==========================================
  Files         112      120       +8
  Lines        6937     7273     +336
  Branches     1172     1242      +70
==========================================
+ Hits         5385     5565     +180
- Misses       1381     1533     +152
- Partials      171      175       +4

Impacted Files
Coverage Δ

orchestrator/graphql/resolvers/process.py
28.81% <28.81%> (ø)

orchestrator/utils/helpers.py
37.83% <37.83%> (ø)

...ator/graphql/extensions/ErrorCollectorExtension.py
50.00% <50.00%> (ø)

...raphql/extensions/deprecation_checker_extension.py
58.73% <58.73%> (ø)

orchestrator/graphql/__init__.py
73.33% <73.33%> (ø)

orchestrator/graphql/types.py
76.92% <76.92%> (ø)

orchestrator/app.py
81.48% <100.00%> (+0.46%)
:arrow_up:

orchestrator/config/assignee.py
100.00% <100.00%> (ø)

orchestrator/domain/lifecycle.py
77.14% <100.00%> (+1.38%)
:arrow_up:

orchestrator/graphql/pagination.py
100.00% <100.00%> (ø)

... and 3 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


(IE 11 & edge) Scrollbar on video when overflow auto/scroll appears

(Relative to issue #210 and its fix #211)
On IE Edge (and IE 11 if you fix #210 with request #211), if you insert a video in a parent which is set overflow: auto/scroll, an horizontal scrollbar appears.
(try to set overflow: auto on  on https://plyr.io/ )
The reason is that the .plyr__tooltip (in the .plyr__controls) go over the edges of the video.
No left or right position are fixed, and the scale -50% doesn't fix the problem; the last-right tooltip (particularly this one) go over the edges.
* First easy fix: set an overflow hidden on the entier video div.plyr;

second (more difficult but probably better) fix: not using scale to position tooltips, but adding right/left position.

Using left/right positioning wouldn't be ideal as it would mean the tips were either left or right aligned or you'd have to fix the width and use negative margins. Using translate (it's not scale) means its always center aligned based on the content. The overflow is not ideal I know and I've come up against this in full screen mode also. A possible solution might be to do custom styling for the outer tooltips on the play/pause button and fullscreen toggle to be left and right aligned respectively. I'll take a look at this one.


How about taking place in battle automatically?

Ok. I have joined the battle, but I also need to take place in it manually.

It's hard to do but i don't give up,

Are you still trying to do it because i need it so much.

Are you still trying to do it because i need it so much.
yes but i told already it's hard for me (i hate javascript)


Error while signing data with privateKey

Version: 1.0.103
Platform: win32 10.0.18362
Plugins: altair, relaxed-theme, scrollbar, theme-gruvbox, theme-hype, theme-windows10
On new version I receive error during the connection with private key: Error while signing data with privateKey: error:06000066:public key routines:OPENSSL_internal:DECODE_ERROR
Same key worked fine with old version and no problems when using TermiNus

+1

+1

+1
Platform: MacOS 10.15.3


nmt-rs Multihash compatibility

subtask of #133
Custom multihash + nmt-rs compatibility https://github.com/celestiaorg/celestia-node/blob/34775bbf11e102f4a2457e09acdfc83c288ded73/share/ipld/nmt.go#L61-L68

Turns out we have most of the required blocks already in types/src/byzantine.rs and we should be able to reuse them when needed.

we only had to read them correctly so far, but if creation will be needed on our side, we don't have it yet.
It's most likely about creating an adapter which will use nmt_rs::NamespacedSha256Hasher for the multihash hasher like
https://github.com/multiformats/rust-multihash/blob/master/derive/src/lib.rs#L27


add: added an endpoint to refresh the token

updated the eslint configuration to avoid some false positive linting
errors
improved styling in other files
refactored some JWT settings into the config file to avoid code
duplication

Closes #141
Test:

run the frontend application
log in from the browser
add an axios call to the /api/auth/refreshToken endpoint
check if the token has been refreshed and the response contains the user data

If you want to check the service separately - I have user something like this:
import refreshToken from "./services/refreshTokenService.js";
import config from "./config/config.example.js";
import pkg from "jsonwebtoken";
const { sign } = pkg;

const token = sign(
  {username: "test_username1", accountType: 1, accountStatus: 1},
  config.secretKey,
  {
    algorithm: config.tokenAlgorithm,
    expiresIn: config.tokenExpiration,
    issuer: "WikiOnFire"
  },
);

const res = refreshToken(token);
res.then((obj) => { console.dir(obj); });

Spotlight is updated. I am working on possibly passing cookies via spotlight, so fingers crossed we can test using spotlight. We'll see.

Also please rebase with main because i need implemented logging in to test it

Can you remind me what was wrong with this one?

Can you remind me what was wrong with this one?

I get 'invalidToken: token argument must be a string' error everytime

Can you check after the conflict resolution as well?


Error when running AdamOptimizer

Hi,
Whenever I try to optimize a model using Adam, I get the following error:
FailedPreconditionError: Attempting to use uninitialized value beta1_power
This happens when I run my own models, as well as when I run GPflow tests and notebooks. All other optimizers work perfectly fine. Any ideas on what is causing this?
Thanks!

After further invesitgation, I found that this issue is caused by incompatibility with Tensorflow version 1.6. I am going to keep this issue open for now so that the devs aware of this problem.
Thanks

Hello @aboustati! We are well aware of this issue, will fix it soon.Thanks!

See #677

Solved #714


grafanaPlugins: use API to generate plugins.nix

Description of changes
This makes sure that all the plugins are available by using a jq script that generates plugins.nix from the API.
@nukaduka @lukegb what do y'all think?
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

My remaining concern here is that we lose information about licenses; https://grafana.com/legal/plugins/?plcmt=footer claims that it needs to be one of the listed AGPL-compatible licenses, but I don't think it's particularly easy for us to determine which one from the Grafana API. I'm also not sure whether this also applies to Grafana's own plugins published in the catalog...

@lukegb correct, this is the only downside of this approach. All of grafana's own plugins are licensed apache2, which is why I set the license to apache2 in grafana-plugin.nix.

@lukegb do you reckon there's a way forward with this change? Or do we need to scrape the license information somehow if we want to include the plugins?
We could also default the plugins to unfree and provide overrides

We could also default the plugins to unfree and provide overrides

Since free packages are cached by hydra, and in nixos we haven't reached a consensus even regarding caching unfree-redistributable (https://github.com/NixOS/nixpkgs/issues/83884), it makes sense to avoid any problems and consider plugin to be unfree until proven free


Some unclarity and potential imprecision in the PDQ profile description

Section Number
Section 5.3
Issue
Consider the next sentence:

XCPD is similar in function to PDQ but is used with the Federated Discovery and Retrieve model of Document Sharing.

Here I have two comments about:

I do not think this is a correct statement. PDQ allows for a small subset of functionality provided by XCPD
TO my knowledge, the XCPD profile does not limit its applicability to the Federated Discovery and Retrieve model of Document Sharing. In fact, XCPD does not have to be used in the context of document sharing.

Proposed Change
Consider improving or substantiating the statement.
Priority:
Medium

I agree with the technical truth of the comments, but I am not clear on how correct it. Given that the whitepaper is focused on HIE, it is already a context constraint. This is not a whitepaper about each profile standing alone, it is about their role in and HIE.
@ivan-zapreev-work do you have a recommendation given my comment?

Well, may be it is just worth stating that the considered profiles are not discussed in their full power but just in the narrower scope and thus the provided descriptions may be incomplete?

I think I did say that.

Ok, then it is resolved 👍


2.0 Deploy

Here are some notes for deployment:

Change travis-ci links in README.
javadocs
Update poms
Deploy to maven.linqs.org

Completed a few days ago.


Please push to production for fix https://github.com/sap-tutorials/Tutorials/issues/18225

Thank you, but there is no need to create a pull request for making changes to the rules.vr file. The tutorials on production are using the files from QA.


Not picking up logs

Keen to use this plugin as it would make my life better.
I've installed it but it is having no noticeable impact on how my logs are displayed. This may just be me not knowing how to use this properly so apologies if so!

Installed v1.5
Accessing logs in Cloud Watch, picking a Log Group, then opening a Log Stream to view
URL is https://ap-southeast-2.console.aws.amazon.com/cloudwatch/home?region=ap-southeast-2#logsV2:log-groups/log-group/*
The extension icon is coloured/bright in the toolbar showing it is active (it is grey on other pages)
The extension drop down menu for "This can read and change site data" is selected as "On ap-southeast-2.console.aws.amazon.com"
I'm using "tslog" to generate log (from a Node/express server, via Elastic Beanstalk) which creates log like this:

Sep 10 02:20:21 ip-172-31-8-114 web: #033[90m2022-09-10 02:20:21.610  #033[39m#033[1m#033[92mDEBUG#033[39m#033[22m #033[90m[src/service/character.service.ts:17 undefined.]#033[39m Found 1 characters for ConceptVersion with ID 1

Log shows as above, no change to show colour highlights.

I'm also having this issue! Any clue what's going on @Falydoor

@anchpop @zonski I just released a version (1.6) and tslog should be supported now!

Unffortunatelly this version is not working yet.


 \_,__/ /\____\
              \/____/\/__/\/_/ \/___/  \/____/
               Controller ABstraction Library
                                  v. 0.9.7-dev

[2018-10-27] cabl v. 0.9.7-dev - development version built on a local machine
-- Checking for LibUSB...
-- Found: libusb-1.0:
--  - Includes: /usr/include/libusb-1.0
--  - Libraries: /usr/lib/arm-linux-gnueabihf/libusb-1.0.so
-- Checking for HIDAPI...
-- Found: HIDAPI
--  - Includes: /usr/include/hidapi
--  - Libraries: /usr/lib/arm-linux-gnueabihf/libhidapi-libusb.so
-- Checking for RtMidi...
-- Found: RtMidi
--  - Includes: RTMIDI_INCLUDE_DIRS-NOTFOUND
--  - Libraries: /usr/lib/arm-linux-gnueabihf/librtmidi.so
-- Boost version: 1.62.0
-- Found the following Boost libraries:
--   python
-- Could NOT find Doxygen (missing:  DOXYGEN_EXECUTABLE)
CMake Warning at cmake/colors.cmake:29 (_message):
  Doxygen has not been found, the documentation will not be
  created.
Call Stack (most recent call first):
  CMakeLists.txt:184 (message)

BUILD_STATIC_LIBS is ON
-- Checking out external repository: lodepng
-- Configuring done
-- Generating done
-- Build files have been written to: /home/pi/cabl/build/lodepng/build
[ 12%] Performing update step for 'lodepng'
Current branch master is up to date.
[ 25%] Performing configure step for 'lodepng'

[ 37%] Performing build step for 'lodepng'

[ 50%] No install step for 'lodepng'
[ 62%] Completed 'lodepng'
[100%] Built target lodepng
-- lodepng path: /home/pi/cabl/build/lodepng/src/lodepng
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
/home/pi/cabl/RTMIDI_INCLUDE_DIRS
   used as include directory in directory /home/pi/cabl
   used as include directory in directory /home/pi/cabl

-- Configuring incomplete, errors occurred!
See also "/home/pi/cabl/build/CMakeFiles/CMakeOutput.log".

check where rtmidi was installed.
If include files are in /usr/include/rtmidi, I had the same problem on ubuntu and I fixed it in commit https://github.com/shaduzlabs/cabl/commit/f9564bb36f17e76c557fa198bb11607e34f6ad8e

@quasart- weird.. i tried the develop branch..will recheck later

maybe it was installed in a different place on your distribution then

I had the same issue which led me here. Updating to the develop branch solved my problem.
$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=18.04
DISTRIB_CODENAME=bionic
DISTRIB_DESCRIPTION="Ubuntu 18.04.1 LTS"
$ find /usr -name rtmidi
/usr/share/doc/librtmidi-dev
/usr/share/doc/librtmidi4
/usr/share/doc/libportmidi0
/usr/include/rtmidi
/usr/include/rtmidi/rtmidi_c.h
/usr/lib/x86_64-linux-gnu/libportmidi.so.0
/usr/lib/x86_64-linux-gnu/libportmidi.so.0.0.0
/usr/lib/x86_64-linux-gnu/librtmidi.so.4
/usr/lib/x86_64-linux-gnu/pkgconfig/rtmidi.pc
/usr/lib/x86_64-linux-gnu/librtmidi.so.4.0.0
/usr/lib/x86_64-linux-gnu/librtmidi.so


Cover unit test for use case GetCurrentCartUseCase.kt

Currently there is no testing for use case. It would be nice if we can set up one and cover them

I did some in app/src/test/java/com/hieuwu/groceriesstore/domain/usecases/impl/GetProfileUseCaseImplTest.kt, you can start writing test following the convention.

Can I take this up as well?

Done


chore(log) remove duplicate CP plugin version output in log

Summary
Duplicated CP plugin version information is being appended to a log message when the CP and DP major versions differ. The CP plugin version is already part of msg.
18:44:01 [warn] 2997#0: *26 [lua] control_plane.lua:247: check_version_compatibility(): [clustering] data plane acl plugin version 3.0.1 is different to control plane plugin version <IP_ADDRESS>.1 [id: e5ecbe97-b04a-464f-8667-ba82edea4953, host: d9d9a0877271, ip: <IP_ADDRESS>, version: <IP_ADDRESS>-dev-enterprise-edition], client: <IP_ADDRESS>, server: kong_cluster_listener, request: "GET /v1/outlet?node_id=e5ecbe97-b04a-464f-8667-ba82edea4953&node_hostname=d9d9a0877271&node_version=<IP_ADDRESS>-dev-enterprise-edition HTTP/1.1", host: "<IP_ADDRESS>:9005"
18:44:01 [warn] 2997#0: *26 [lua] control_plane.lua:247: check_version_compatibility(): [clustering] data plane acme plugin version 0.2.14 is different to control plane plugin version <IP_ADDRESS>.14 [id: e5ecbe97-b04a-464f-8667-ba82edea4953, host: d9d9a0877271, ip: <IP_ADDRESS>, version: <IP_ADDRESS>-dev-enterprise-edition], client: <IP_ADDRESS>, server: kong_cluster_listener, request: "GET /v1/outlet?node_id=e5ecbe97-b04a-464f-8667-ba82edea4953&node_hostname=d9d9a0877271&node_version=<IP_ADDRESS>-dev-enterprise-edition HTTP/1.1", host: "<IP_ADDRESS>:9005"

Note: The log message above is just for demonstration to show the extra 3.013.01 and <IP_ADDRESS>.14 in the log messages.
Full changelog

chore(log) remove extra CP plugin version output in log

Skipping including this in the changelog, since while it is technically a fix, it is extremely minor.


Improve error messages when operation conversion fails

After upgrading to 0.0.6, I got NPEs with my existing OpenAPI definitions. Evidently there is a response in my code somewhere missing its schema, but it's difficult to track down where. To aid in debugging, I added the operationId to error messages. I also added a more clear error message when a schema is missing instead of letting it produce an NPE.

Wrap errors during operation processing in an ex-info. The message contains the operation name and the ex-data contains the source operation.
Throw ex-infos with a "missing schema" message instead of NPEs when a necessary schema is null.

This turns errors that previously looked like:
java.lang.NullPointerException: Cannot invoke "io.swagger.v3.oas.models.media.Schema.getTypes()" because "schema" is null
into this:
clojure.lang.ExceptionInfo: Exception processing operation "TestOp": Missing schema

Looks good, thanks a lot!


Upgrade google-cloud-sdk

What this PR does / why we need it:

Upgrade google-cloud-sdk to latest
Use x86_64 version of google-cloud-sdk

Which issue(s) this PR fixes:
We upgraded the alpine to 3.20 in this PR
Alpine 3.20 uses the newer Python, so this issue occurred.
https://issuetracker.google.com/issues/303280713?pli=1
The issue is already fixed in the latest google-cloud-sdk, so I upgrade it.
Fixes #
Does this PR introduce a user-facing change?: No

How are users affected by this change:
Is this breaking change:
How to migrate (if breaking change):

I tested with a command below
docker buildx build --platform linux/amd64 --file cmd/pipecd/Dockerfile . -t pipecd-debug


Write multi-conformer PDBS with RDKit

[X] Fix #568
[X] Add tests
[x] Update docstrings/documentation, if applicable
[ ] Update changelog

This should fix RDKit and allow it to write multi-conformer PDB files, also added a test for this.

This has made me realise I don't think we test all of the possible file output formats which I could add test for here if required?

Which formats are you referring to? We are in the process of adding some in #569

Ahh yes I mean backend toolkit formats more like PDB and SDF I know SDF is more heavily tested now that the property PR is merged.

This has made me realise I don't think we test all of the possible file output formats which I could add test for here if required?

I'm starting to appreciate how "output formats" are complicated. That is, writing to a particular output format isn't just "making a valid file within that format's specification", but rather "deciding on exactly which characteristics of a molecule we guarantee to preserve in a round trip to that format".
So, this doesn't mean "don't add tests for new formats" -- after all, we do "support" them in the API, and testing is better than no testing. I mean something more like "don't be surprised if this task is a rabbit hole, and we end up with a big table of what we do and don't preserve through trips to different formats"
So, this PR shouldn't add tests for every output format, because it will take a long time.

Oh, and please update the release notes before you merge!!

Good point I had a feeling it may be a bigger job than I expected Ill get this finished up today.


create a tutorial for mac

Would be really nice to have a walkthrough on how to set this up on a pair of macs

Second this request!

Does this actually work on Mac? Or is there just no details on how to get it up and going on Macs?

It will work on a Mac

Hi
that is good news indeed. Would you mind sharing a guide ? As a special request I would actually like to keep a standard PMS on my MAC but use the idle capacity from the mac for remote transcoding as well if that would work (having a Raspberry Pi as Master)


Support all float dtypes in F.normalize

This PR follows #4582 to support all float dtypes in F.normalize.

Jenkins, test this please.

Jenkins CI test (for commit e2209952f1838ed73da64027872b8cbe0ee4cd64, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

You mean, computing L2 norm without using numpy.linalg.norm to keep float16 at least for old numpy environment? (Or just casting back to FP16 in such case is enough?)

I'd like to compare with numpy.linalg.norm in the tests.  Therefore I prefer casting back to float16.

Okay, I got.

Jenkins, test this please.

Jenkins CI test (for commit 63722afdaa8413ac1529f2705659b1dcb587effb, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.


Feature request: support eip-4844

Background
Holesky has released the Cankun fork, and supported the eip-4844, it added new transaction type(3), so zeth will panic when decode the new transaction from holesky
https://github.com/risc0/zeth/blob/6867af4ebf012476248c57258422dfd0d379fe5b/primitives/src/ethers.rs#L221
FYI
EIP: https://eips.ethereum.org/EIPS/eip-4844
ethers-rs status
Related issues: https://github.com/gakonst/ethers-rs/issues/1742, https://github.com/gakonst/ethers-rs/issues/2719
Suggestion
By the way, reth has already supported the new EIP https://github.com/paradigmxyz/reth/blob/34216727d2994761ae6c756e899404f3b5eb312a/crates/primitives/src/transaction/eip4844.rs#L192, maybe zeth can move from ethers to reth
TODOs

[ ] Wait and update the ethers-core to support 4844(or use reth instead of ethers
[ ] Add new definition of BlobTx
[ ] Convert between ethers-core and zeth's transaction definition

Yes, to support EIP-4844 we'd need support for blob transactions.
We tried to use the reth types some time ago, but it wasn't feasible because their code base wasn't modular enough and included stuff that wasn't zkVM compatible. This was exactly the reason we had to start the primitives' crate and reimplement certain types. This may have changed, but I was waiting for transactions to be added to alloy` to be able to use them.
Also, I don't think just adding the new transaction type is enough for Cancun support, there are also changes in execution and header for example. So it is a slightly larger change that is not a top priority for us right now.

Also, I don't think just adding the new transaction type is enough for Cancun support, there are also changes in execution and header for example. So it is a slightly larger change that is not a top priority for us right now.

Exactly

primitives' crate and reimplement certain types. This may have changed, but I was waiting for transactions to be added to alloy`

Alloy has already supported the 4844,
https://github.com/alloy-rs/alloy/blob/42213c0d78c18141836f7129eda92ba2c8c59525/crates/consensus/src/transaction/eip4844.rs#L74
and rpc-provider, so is it a timing to migrate from ethers to alloy?

Strong rec to move towards Alloy and we'll prioritize anything missing, Alloy is good to go on 4844.

Strong rec to move towards Alloy and we'll prioritize anything missing, Alloy is good to go on 4844 and has everything you need except some client side log queries.

Alloy requires the MSRV 1.75<EMAIL_ADDRESS>but the latest version of risc0's toolchain is only 1.70.

Risc0's rustc has already supported the 1.75 by upgradecargo risczero install --version v2024-02-08.1


[FEATURE REQ] Remove the Key Vault SDK content from the Azure Spring Apps KV doc

Is your feature request related to a problem? Please describe.
We should improve this doc by removing the KV secrets SDK part, https://learn.microsoft.com/en-us/azure/spring-apps/tutorial-managed-identities-key-vault?source=recommendations&tabs=system-assigned-managed-identity.
For an Azure Spring Apps application that wants to integrate with Azure Key Vault Secrets, the better choice should always be using the Spring Cloud Azure Key Vault starter, either for secret management or for auto-wiring secret clients.
The SDK part in this doc is very confusing to users.
Describe the solution you'd like
Remove the SDK part from this doc.
Describe alternatives you've considered
N/A
Additional context

@chenrujun:
And even if the doc want to demonstrate the usage of kv secret client, it should use the auto-configured one.

Information Checklist

[x] Description Added
[x] Expected solution specified

Close it via https://github.com/MicrosoftDocs/azure-docs-pr/pull/215032.


Possibility to include the source node in the node/dataset options list

Related ticket: https://github.com/CartoDB/cartodb/issues/12057
In the closest analysis we need to provide the source node as an option in the target list, that's all. By default, we will not provide it (in any other analysis), but it could be added by the includeSourceNode parameter.
And about how to test it, we should check we provide that option in that analysis and also check if in the rest of them we don't provide it in the target editor.

Accepted!


zpaq gui numkey issue

Hi there.
First of all i need to say that i simply love the new GUI, even if is still not complete! What a beautiful idea!
During my playing with that, i have found a little issue that i want to report.
Under the filter/search/replace/extract dialog, this two keys from the numpad doesn't work: / and *****
And this other two keys have a strange effects: + and -
Actually the + and - keys are binded to another function but what if i need to insert these symbols as part of a filename? Like "zpaq+.txt" or "zpaq-advanced.txt"
Thank you!

As mentioned it is only a very first alpha version, more for the next step (PAKKA-like)
AKA: for every file a different sort
Example
file1.cpp
v001 2023-02-03 3.000 byes
v002 2023-02-05 3.500 bytes
v003 2023-02-06 4.000 bytes
pippo.cpp
v001 2023...
v002 ....
work in progress...


command-API clarification question

what is the difference between correlation-id and request-id?

correlationId*
string($uuid)example: 82eb2e26-0f24-48aa-ae4c-de9dac3fb9bcIdentifies the batch to which this response belongs. If used via REST, this value should be the same as that of the ''X-Correlation-ID'' HTTP header.

requestId
string($uuid)example: e6e8a2f4-eb14-4649-9e2b-175247911369Uniquely identifies the request within the larger batch resulting in this response.


how to install it to hl1 mmod?

/scripts/install-to-HalfLifeMMod.bat is available since https://github.com/hzqst/MetaHookSv/releases/tag/v20241012g


POC for unique test/hook/suite identifiers

@segrey please see this one.  It will allow a reporter to call runner.linkPartialObjects(true) (see example in docstring) to replace duplicated deserialized objects with references.  The objects themselves are still shallow, but if there's a particular property you need that is missing, we can likely add it.
I didn't test this too hard, but please let me know if this would help.  I'm not sure about the missing root prop issue...
Ref: #4403

fixed an issue in rollup config causing weird problems
changed how we define global.Mocha in browser-entry.js (tangential)
removed object.assign polyfill (tangential)
renamed misnamed buffered-runner.spec.js to parallel-buffered-runner.spec.js

Coverage increased (+0.05%) to 93.846% when pulling de5bdfcf665b4242fd64ab140e9f0714b6611545 on boneskull/issue/4403-object-ids into 4e2b47df31fbc0865cbfa4477226e8609d2ee56e on master.

@boneskull Great! I've just tried it with the example from #4403, and now test objects are the same across different events. This fixes "started: false instead of started: true" sub-problem from #4403 description.

@segrey So this looks good as-is?  I'll wrap it up then.

@segrey if you can share it, I'd love to see the implementation you've built with this

@mochajs/core please review, otherwise will merge in next couple days

@boneskull Please see mocha-intellij.zip. Main reporter is mocha-intellij/lib/mochaIntellijReporter.js. There is some code duplication in main reporter and worker reporter, but it's just a proof-of-concept :)
There are some minor questions left in https://github.com/mochajs/mocha/issues/4403#issuecomment-684091528. Would be great if you could take a look.

@segrey This has landed... will be released in next version which will be v8.2.0


Allow implementations to replace serialization

We have an idea for Search that would use discriminated types for input/output models. For deserializing the type, we'd like to "override" the deserialization to properly instantiate and populate the right type instead of merely wrapping it later.
There are a couple of ways to workaround this, so not high-pri, but would be optimal eventually.
One idea would be to check first if we already implemented IUtf8JsonSerializable in a class attributed with CodeGenSchemaAttribute. Assuming that's not hard to check, that's probably the easiest to support but maybe not the most robust.
Another way is might be to call into a partial method that we could optionally implement to read properties (without advancing the reader) and return which types (or instances - probably just easier for that code to create them so we don't need factories) to use.

I would like to add user may like to access Serialization/Deserialization methods for simpler object construction. Currently marking it internal prevents that.

For models we generate from swagger, or for their own models like used with Cosmos DB, Search documents, etc.? Could you clarify the scenario of why a user (did you mean consumer of our SDKs?) should be able to modify it? The ask here was for SDK devs to be able to change de/serialization for when the deafult isn't quite what we want?

I think the original request here is done.


Consider restoring runtime/rgb.txt

or at least referencing https://github.com/ctrlcctrlv/rgb.txt
see https://github.com/ofavre/vimcat/issues/13

Ehm, why?

So vimcat will build again w/o my hacks.

I don't know about vimcat, but what is rgb.txt used for?

@k-takata : See :help rgb.txt which also says why this file is no longer used.
Best regards,
Tony.

@tonymec
I'd like to know the reason why vimcat requires rgb.txt.

P.S. By reading :h rgb.txt and continuing at :h v:colornames I conclude from a paragraph at the end of the latter that the desired result can be achieved by a Vim script named one (or more) of
~/.vim/colors/lists/default.vim
$HOME/colors/lists/default.vim
$VIM/vimfiles/colors/lists/default.vim
$VIM/vimfiles/after/colors/lists/default.vim
$HOME/vimfiles/after/colors/lists/default.vim
~/.vim/after/colors/lists/default.vim

some of which depend on the OS gvim (or Console Vim with either +termguicolors or some plugin such as CSApprox or similar) is running on. See :help 'runtimepath to and including :help after-directory for an explanation of which one(s) of these directories to use in which circumstances.

Oops: in my previous post, the second pathfilename ought to be $HOME/vimfiles/colors/lists/default.vim not $HOME/colors/ etc.
Best regards,
Tony.

@k-takata : I don't know what vimcat is, or is about. Maybe after reading the help pages I mentioned @ctrlcctrlv can tell us that (as I believe) rgb.txt is not needed after all, or why it is.

Hi, I'm the author of vimcat.
I read vim's code as to why this file was deleted and end up using v:colornames instead of processing runtime/rgb.txt at package build time. So vimcat no longer needs this file either.
Thanks @ctrlcctrlv, but you can close this issue.

Closing then.

Thanks for the patch @ofavre :innocent:


Socket driver

This PR addresses/fixes comments on https://github.com/ros-drivers/nmea_navsat_driver/pull/16

Retargeted to master.

Also please add the new node to the list of installed scripts in the CMakeLists.txt file.

Reviews processed. Please note that this PR is building on an older PR (not by me) that had some unprocessed review comments, so not all the code in this PR is mine.

Done. There is some archeology in 92d156c concerning that regression.

Could the UDP forwarder be added as well, as a test tool?


Extracted actions into middleware

In a future PR I need to specify the home director for DAT operations. This PR also moves globals of actions (dats) from a global variable into a class/instance system. This also turns the LOAD_ACTION - (which wasn't a real action) in ./app/index.js into an regular operation.
It also theoretically allows multiple react windows interacting with the same middleware through a very basic listener system.
It furthermore adds the success/error concept to all middleware operations. My thinking is that there should be a {ACTION}-action before {ACTION}_SUCCESS or {ACTION}_ERROR if an error/success happened during the operation. This would allow to show errors if errors occur and progress displays in the time between an action starts and an action finishes.

@martinheidegger i haven't checked the code yet but tried to use the app with this pr, and i've run into few issues..

i'm unable to delete the dat.
behaviour of pause/resume dat is non-deterministic in the sense that it won't pause the download of dat at very moment and sometimes lag if i click the pause/resume button in relatively small time span.

also do you think it would be good idea for us to refactor actions etc once we have deployed the react on master? we don't have any tests for actions in place for now so i guess we could do that once we deploy this and then start on with the refactoring?

As mentioned in gitter. This PR is necessary to re-add CLI-options that exist on master and make sure that the integration tests work again. This PR is just a precursor to follow-up work. That being said: It should be possible to delete dats and pause/play behavior should be deterministic.
The PR was mostly nothing but a move operation of the actions to the middleware. As far as I understood I didn't change any logic. This means that the bugs could be refactoring bugs or existed before. In any case: thanks for testing it, let me investigate!

I fixed the download bug in #5da2ff8 and found another issue in #02444df. But I couldn't reproduce your temporal issues with the dats. My suggestion would be to merge this as-is and try to figure out how to make the pause/resume behavior more stable.

@aks- Found the issue in pause/resume state, should be addressed in https://github.com/dat-land/dat-desktop/pull/529/commits/eb08fb1b60af665491a549e836260b65519e7398

@martinheidegger deleting the dat is failing on this one

@martinheidegger should we close this in favour of https://github.com/dat-land/dat-desktop/pull/530?

Following our discussion in gitter I merged this. To your question: I tend to build on existing PR with new PR if the first ones don't get merged quickly. The expectation on my side is that the first PR (this) gets merged before the discussion on the second PR (530) begins.


To show files saveds from table

Expected behavior
Using a foreign key, show in the dropzone which files are saved in my customized table
Actual behavior
I yet added members at collection at page load process, but the files are not displayed
Steps to reproduce the issue
creat a pre rendering process, with a loop on my table, and to each interaction adding member at collection
APEX version (4.2.6 / 5.0.3)
20.1
Used web server / version and platform (ORDS 3.0.3 / Tomcat 7 / Apache 2.4 / Linux x64)
Used web browser / version and platform (Chrome 48 Mac / Firefox 44 Windows)
Chrome version 85.0.4183.121 for Windows

That is my pre rendering code, but nothing happens at my dropzone
`DECLARE
ID NUMBER;

BEGIN
FOR A IN (SELECT ID_UPLOAD_ARQUIVOS, ARQUIVO, FILENAME, MIMETYPE, DATA_REG FROM UPLOAD_ARQUIVOS) LOOP
APEX_DEBUG.MESSAGE('ENTROU AQUI');

    ID := apex_collection.add_member(p_collection_name => 'DROPZONE_UPLOAD',
                                                    p_c001            => A.FILENAME, -- filename
                                                    p_c002            => A.MIMETYPE, -- mime_type
                                                    p_d001            => A.DATA_REG, -- date created
                                                    p_n001            => A.ID_UPLOAD_ARQUIVOS, -- random file id
                                                    p_blob001         => A.ARQUIVO); -- BLOB file content

APEX_DEBUG.MESSAGE('ID INSERIDO: ' || ID);

END LOOP;

END;`


Versioned transaction

How can I create a versioned (v0) transaction like in js :

const messageV0 = new TransactionMessage({
        payerKey: SIGNER_WALLET.publicKey,
        recentBlockhash: latestBlockhash.blockhash,
        instructions: txInstructions
    }).compileToV0Message();
    console.log("   ✅ - Compiled Transaction Message");
    const transaction = new VersionedTransaction(messageV0);

Have u already finded the decision?

I have the same question

you can pass a lookup table as an option when creating the txn
tx, err := solana.NewTransaction(
	        instructions,
		block.Blockhash,
		solana.TransactionPayer(user.PublicKey()),
		solana.TransactionAddressTables(transactionAddressTables),
	)

i need to do without lookup table, any luck


auth - security: Demo users are included in "default" installation

If  an installation is started without any flags as described in the git manual, a lot of demo users are created with critical roles (such as auditor)
We could make the installation without demo users new default
@tpurschke What do you think

We could tell first time users at the initial page, that unsecure demo users are present on the system

fixed in v5.3.2


Added new day/night toggle button

This commit introduces a new day/night toggle button to the Website. The toggle button allows users to switch between a day and night theme for a more personalized browsing experience. solved Issue -: #403

ok i can change the code and then sync the repo as in my side its working

Please test the switch properly before pushing and review the deploy preview.
When the dark mode is switched to light mode and then switched back the switch changes but not the theme.

Ok but i think the java script is not working accordingly as i am not that much good in java script as i need help of anyone so you can merge my pull request then anyone can raise the isuues to solve this issue so please consider it

@khadar1020 since you worked on fixing this, can you work on this PR?

ok i will work on this issue @AshutoshDash1999

Can you comment on the issue? #403


About Loss Function

Thanks for your work, it inspires me a lot. But I have one question about the loss function in Section3.4, in order to filter crucial samples, you applied Online Hard Example Mining to L_las, I wonder how this is implemented in this project?

Thank you for your recognition. In this project, the OHEM code is located in glass.py from L408 to L418. The argument --p is a parameter that controls the selection of hard samples. In our method, the default value is p=0.5, which means only those samples with an error degree (fake_dist) greater than or equal to d_hard are considered hard samples. In this case, d_hard is the median of fake_dist. Subsequently, these hard samples are used for the calculation of focal loss.


[Bug] Excel列名计算器在超出界限时空指针

Checklist

[X] I am using LTS version of Java.
[X] I am following the docs and I am sure I am using the correct configuration.

Package version
1.0.8
which version of java are you using?

[ ] 8
[ ] 11
[X] 17

Which operating system are you using?

[ ] macOS
[X] Windows
[ ] Linux

Describe the bug
输入A 偏移量 -1
结果null
Additional context
No response

Hello @unknowIfGuestInDream. This issue is marked as invalid and closed. Please make sure you are reporting an issue and following the issue template.


fix: offset segment arena instance counter for older vm versions

This change is

Python side: https://reviewable.io/reviews/starkware-industries/starkware/35539

crates/blockifier/src/execution/deprecated_entry_point_execution.rs line 248 at r1 (raw file):
Previously, noaov1 (Noa Oved) wrote…
Please add it only for the vm_resources_without_inner_calls (those are the vm resources).
Same for cairo1.

Should I do it before substract &previous_resources?
If the vm_resources are only in vm_resources_without_inner_calls shouldn't the value be the same whether I multiply it here or there?


Shall I use the same client with multiple queries?

Which is better? this example:
def last_locations(conn, params) do
  user_ids= Poison.decode!(params["user_ids"])

  locations= user_ids
  |> Enum.map(&Task.async(fn -> last_location(&1) end))
  |> Enum.map(&Task.await/1)

  json(conn, locations)
end

defp last_location(user_id) do
    client = CQEx.Client.new!
    query= "SELECT * from myapp.latests WHERE user_id = #{user_id};"
    for [user_id: _, lat: lat, long: long, unix_time: unix_time] <- Q.call!(client, query)
    |> Enum.to_list,
    do: [lat, long, unix_time]
end

or this example: ? (where I am using one client for all queries)
def last_locations(conn, params) do
  user_ids= Poison.decode!(params["user_ids"])
  client = CQEx.Client.new!
  locations= user_ids
  |> Enum.map(&Task.async(fn -> last_location(&1, client) end))
  |> Enum.map(&Task.await/1)

  json(conn, locations)
end

defp last_location(user_id, client) do

    query= "SELECT * from myapp.latests WHERE user_id = #{user_id};"
    for [user_id: _, lat: lat, long: long, unix_time: unix_time] <- Q.call!(client, query)
    |> Enum.to_list,
    do: [lat, long, unix_time]
end

The first one, each query should be done using a different client reference. Getting a client reference is really fast, doesn't introduces bottlenecks, and ensures proper load-balancing.

Thanks for advice

But what happen after getting the result of query? how would the client get disposed? or ex: terminating process?


Command Injection vul fix: Replace execSync with execFileSync

📊 Metadata *
react-dev-utils includes some utilities used by Create React App.
The function getProcessForPort in react-dev-utils is vulnerable to command injection.
Bounty URL: https://www.huntr.dev/bounties/1-npm-react-dev-utils/
⚙ Description *
Used child_process.execFileSync() instead of child_process.execSync().
💻 Technical Description *
The use of the child_process function execSync() is highly discouraged if you accept user input and don't sanitize/escape them. This PR replaces it with execFileSync() which mitigates any possible Command Injections as it accepts input as arrays.
🐛 Proof of Concept (PoC) *
Create a .js file with the content below and run it, then the file pzhou@shu can be illegally created.
// poc.js
var getProcessForPort = require('react-dev-utils/getProcessForPort');
getProcessForPort('11;$(touch pzhou@shu)');
🔥 Proof of Fix (PoF) *
use "return execFileSync('lsof', ['-i:'+port, '-P', '-t', '-sTCP:LISTEN'], execOptions)" to replace "return execSync('lsof -i:' + port + ' -P -t -sTCP:LISTEN', execOptions)"
👍 User Acceptance Testing (UAT)
var getProcessForPort = require('react-dev-utils/getProcessForPort');
getProcessForPort(3000) // works correctly
🔗 Relates to...
https://github.com/418sec/huntr/pull/1962

👋 Hello, @gaearon - @zpbrent has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above.
Ultimately, you get to decide if the fix is 👍 or 👎. If you are happy with the fix, please write a new comment (@huntr-helper - LGTM) and we will open a PR to your repository with the fix. All remaining PRs for this vulnerability will be automatically closed.
If you have any questions or need support, come and join us on our community Discord!
@gaearon & @zpbrent - thank you for your efforts in securing the world’s open source code! 🎉

🔨 Want more security researchers protecting your repository?
Stick our badge on your README.md, and let security researchers know that they can win bounties protecting your repositories. Sounds cool, huh? 😎
Copy this small code snippet and insert it into your README.md:
[![huntr](https://cdn.huntr.dev/huntr_security_badge.svg)](https://huntr.dev)
👇 👇 👇

@huntr-helper - LGTM


Advertise objects more smartly

While we receive the inventory of a newly connected node, we advertise all the incoming items.

they most probably get advertised to the node we just received them from
if we have a lot of catching up to do and are connected to more than one node, advertising everything we receive is probably pointless

Solving issue #9 might bring forward some hints as of how to approach this issue.


 Her fleece was white as snow"
from nltk.tokenize import word_tokenize, sent_tokenize
sents=sent_tokenize(text)
print(sents)

You should get an error like so:

Jupyter shows this same error like so:

I believe jupyter is doing a different ansi conversion that is stylable. We should do this and we might be able to then translate these automatically to the theme colors.

You can see it their css:


Share to facebook when click not open app

Closing this as an empty issue. Please use the template and fill out requested details so that we can address it properly.


Storybook 8 compatibility

Hey @geometricpanda! I’m reaching out from the core SB team :)
Storybook 8 (our next major release) is coming soon, and it introduces some breaking changes that your addon might need to account for.
We’ve listed all the key changes in a new Storybook 8 migration guide, and we’d encourage you to check it out and see if your addon is impacted.
We also have an addons channel in our Discord community, which we’d like to use to better communicate with addon authors in the future. If you’re not part of the server, consider joining here and message me (@joevaugh4n) so that I can give you the addon creator role! If you’re already in the server, please just ping me and I’ll respond to you there.
Thanks for being a part of our addon community!
Joe

Just upgraded to version 8, and I'm seeing this error flagged by storybook-addon-badges:
❯ npx storybook@latest doctor
🩺 The doctor is checking the health of your Storybook..
╭ Incompatible packages found ───────────────────────────────────────────────────────────────────────────────────────────────────────╮
│                                                                                │
│   The following packages are incompatible with Storybook 8.0.0 as they depend on different major versions of Storybook packages:   │                                                   │
│   -<EMAIL_ADDRESS>                                                                                │                                                                              │                                                                                │
│                                                                                │
│                                                                                │
│   Please consider updating your packages or contacting the maintainers for compatibility details.                                  │
│   For more on Storybook 8 compatibility, see the linked GitHub issue:                                                              │
│   https://github.com/storybookjs/storybook/issues/26031                                                                            │
│                                                                                │
╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯

I'm able to successfully display Badges, But I see this error when starting up 8.04
15% building 0/2 entries 62/161 dependencies 4/42 modulesWARN Failed to parse /Users/carl.welch/Documents/yubico/yubienterprise-services/services/console/.storybook/preview.ts with react-docgen. Please use the below error message and the content of the file which causes the error to report the issue to the maintainers of react-docgen. https://github.com/reactjs/react-docgen
SyntaxError: ./.storybook/preview.ts: Unexpected token, expected "}" (37:18)

  35 |     },
  36 |     badgesConfig: <BadgesConfig>{
> 37 |       [BADGE.BETA]: {
     |                   ^
  38 |         styles: {
  39 |           backgroundColor: '#FFF',
  40 |           borderColor: '#018786',

Do we know when a new release will be created to include this fix?

I pulled down the latest code and noticed that this should be resolved with whats on main, looks like we're just waiting for a successful release to happen.

Any news on the release of the new version? 😔
It looks like it broke on the yarn publishing process.

@geometricpanda, could you release the new version by any chance?

Yeah I'll give it a try when I get to my laptop. I fear it's gonna end in debugging however

We would love to have the 8.x compatibility for a current project we have set up.

🆙

Also looking forward to v8 compatibility :) @geometricpanda any progress?

Happy to test out if needed, excited for the release!

I'm pretty sure it was just a release issues that happened. As i mentioned in this issue https://github.com/geometricpanda/storybook-addon-badges/issues/25

Would also appreciate v8 compatibility 🤩

Just upgraded to the latest Storybook and getting this warning!
WARN The following packages are incompatible with Storybook 8.2.9 as they depend on different major versions of Storybook packages:
WARN -<EMAIL_ADDRESS>
@geometricpanda, Any updates on this?? 😅😬

So I'm waiting for Storybook to come back to with proposed changes as they have suggestions for how this should work.

So i've fixed the dodgy release pipeline, hopefully that will have solved the compatibility thanks to the work @evg4b did.
I'm going to close this for now, please feel free to comment if you're experiencing issues


ament_package: 0.13.1-1 in 'galactic/distribution.yaml' [bloom]

Increasing version of package(s) in repository ament_package to 0.13.1-1:

upstream repository: https://github.com/ament/ament_package.git
release repository: https://github.com/ros2-gbp/ament_package-release.git
distro file: galactic/distribution.yaml
bloom version: 0.10.7
previous version for package: 0.12.0-2

ament_package
* Add support for appending to environment variables (#130 <https://github.com/ament/ament_package/issues/130>) (#137 <https://github.com/ament/ament_package/issues/137>)
* Update maintainers to Audrow Nash (#135 <https://github.com/ament/ament_package/issues/135>) (#136 <https://github.com/ament/ament_package/issues/136>)
* Revert "Generate Setuptools Dict Helper Method (#126 <https://github.com/ament/ament_package/issues/126>)" (#131 <https://github.com/ament/ament_package/issues/131>)
* Contributors: Scott K Logan, Audrow Nash

Still holding per https://github.com/ros/rosdistro/pull/31837#issuecomment-1017784886


opensans-ttf: change source to salsa, rename to open-sans

Motivation for this change
We really shouldn't be fetching this from a web archive.
~This also includes the condensed variant.~
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Tested locally otherwise. A quick diff between some TTF from Debian and the current archives' content from https://www.opensans.com/ shows it's the same files.
But I'd rather fetch from the Salsa tag. More stable over time I guess.

Oh just noticed that we'll be losing the hebrew fonts by switching.

Though they should probably be added as a separate package open-sans-hebrew.

Yes it's another font made by some else so it would be another derivation for nixpkgs (IMHO)

Just double checking the fetching:
@GrahamcOfBorg build open-sans