Cannot retain previously cached data

[video_player]Locating video progress using the seekTo function cannot retain previously cached data，The video buffer data starts again！
Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
using the seekTo function

Expected results:
Save previous buffered data and progress.
Actual results:
The video buffer data starts again.

Logs

flutter doctor -v
[√] Flutter (Channel master, 2.4.0-5.0.pre.6, on Microsoft Windows [Version 10.0.17763.194], locale zh-CN)
    • Flutter version 2.4.0-5.0.pre.6 at D:\flutter\last-new\flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision 08ef7752d5 (11 days ago), 2021-07-11 23:25:25 -0700
    • Engine revision ed25e8f01e
    • Dart version 2.14.0 (build 2.14.0-301.0.dev)
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at D:\Android\Sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = D:\Android\Sdk
    • Java binary at: D:\AndroidStudio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

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

[√] Android Studio (version 4.2.0)
    • Android Studio at D:\AndroidStudio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[√] Connected device (2 available)
    • SM G9500 (mobile) • 988bdc424259355033 • android-arm64  • Android 9 (API 28)
    • Chrome (web)      • chrome             • web-javascript • Google Chrome 85.0.4183.83

Hi @ModWU
Is this on Android or the web?  As video_player both platforms
Can you please provide a complete reproducible minimal code sample
Thank you

Hi @ModWU
Is this on Android or the web? As video_player both platforms
Can you please provide a complete reproducible minimal code sample
Thank you

Android

Hi @ModWU
Is this on Android or the web? As video_player both platforms
Can you please provide a complete reproducible minimal code sample
Thank you

Android

Hi，Here's my demo.

DEMO
import 'package:flutter/material.dart';
import 'package:video_player/video_player.dart';

void main() => runApp(MaterialApp(home: VideoApp(),));

class VideoApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return DefaultTabController(
      length: 3,
      child: Scaffold(
        key: const ValueKey<String>('home_page'),
        body: _BumbleBeeRemoteVideo(),
      ),
    );
  }
}

class _BumbleBeeRemoteVideo extends StatefulWidget {
  @override
  _BumbleBeeRemoteVideoState createState() => _BumbleBeeRemoteVideoState();
}

class _BumbleBeeRemoteVideoState extends State<_BumbleBeeRemoteVideo> {
  late VideoPlayerController _controller;

  Future<ClosedCaptionFile> _loadCaptions() async {
    final String fileContents = await DefaultAssetBundle.of(context)
        .loadString('assets/bumble_bee_captions.srt');
    return SubRipCaptionFile(fileContents);
  }

  @override
  void initState() {
    super.initState();
    _controller = VideoPlayerController.network(
      'http://vfx.mtime.cn/Video/2019/02/04/mp4/190204084208765161.mp4',
      closedCaptionFile: _loadCaptions(),
      videoPlayerOptions: VideoPlayerOptions(mixWithOthers: true),
    );

    _controller.addListener(() {
      setState(() {});
    });
    _controller.setLooping(true);
    _controller.initialize();
  }

  @override
  void dispose() {
    _controller.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return SingleChildScrollView(
      child: Column(
        children: <Widget>[
          Container(padding: const EdgeInsets.only(top: 20.0)),
          const Text('With remote mp4'),
          Container(
            padding: const EdgeInsets.all(20),
            child: AspectRatio(
              aspectRatio: _controller.value.aspectRatio,
              child: Stack(
                alignment: Alignment.bottomCenter,
                children: <Widget>[
                  VideoPlayer(_controller),
                  ClosedCaption(text: _controller.value.caption.text),
                  _ControlsOverlay(controller: _controller),
                  VideoProgressIndicator(_controller, allowScrubbing: true, colors: const VideoProgressColors(playedColor: Colors.white, bufferedColor: Colors.red),),
                ],
              ),
            ),
          ),
        ],
      ),
    );
  }
}

class _ControlsOverlay extends StatelessWidget {
  const _ControlsOverlay({Key? key, required this.controller})
      : super(key: key);

  static const _examplePlaybackRates = [
    0.25,
    0.5,
    1.0,
    1.5,
    2.0,
    3.0,
    5.0,
    10.0,
  ];

  final VideoPlayerController controller;

  @override
  Widget build(BuildContext context) {
    return Stack(
      children: <Widget>[
        AnimatedSwitcher(
          duration: Duration(milliseconds: 50),
          reverseDuration: Duration(milliseconds: 200),
          child: controller.value.isPlaying
              ? SizedBox.shrink()
              : Container(
                  color: Colors.black26,
                  child: Center(
                    child: Icon(
                      Icons.play_arrow,
                      color: Colors.white,
                      size: 100.0,
                    ),
                  ),
                ),
        ),
        GestureDetector(
          onTap: () {
            controller.value.isPlaying ? controller.pause() : controller.play();
          },
        ),
        Align(
          alignment: Alignment.topRight,
          child: PopupMenuButton<double>(
            initialValue: controller.value.playbackSpeed,
            tooltip: 'Playback speed',
            onSelected: (speed) {
              controller.setPlaybackSpeed(speed);
            },
            itemBuilder: (context) {
              return [
                for (final speed in _examplePlaybackRates)
                  PopupMenuItem(
                    value: speed,
                    child: Text('${speed}x'),
                  )
              ];
            },
            child: Padding(
              padding: const EdgeInsets.symmetric(
                // Using less vertical padding as the text is also longer
                // horizontally, so it feels like it would need more spacing
                // horizontally (matching the aspect ratio of the video).
                vertical: 12,
                horizontal: 16,
              ),
              child: Text('${controller.value.playbackSpeed}x'),
            ),
          ),
        ),
      ],
    );
  }
}

Show
Uploading device-2021-07-26-132301.mp4…
Tips:
In the above video show, each time drag progress is called (called to the seekTo function),
The red buffer starts again!

Hi @ModWU
I see there's an open issue addressing the case you described.
Please follow up on that issue, Closing as duplicate of https://github.com/flutter/flutter/issues/28094
If you disagree, please write in the comments and I will reopen it.
Thank you


Rubocop should ignore the schema in Rails projects

When rubocop -R is run, it takes into account the db/schema.rb file, which as you know is generated automatically when a migration is done so nothing can be done about its offences. Am I right?

:+1:

BTW, now that's Exclude instead of Excludes:
AllCops:
  Exclude:
    - db/schema.rb

Am I facing a bug...? I have it set up exactly like you guys(I've tried with & without quotes and it still complains.

@PrimeTimeTran Can you show me the command line that you executed?
If you specify db/schema.rb with command line, rubocop does not ignore it.
% cat .rubocop.yml| head                                                                                ( ~/ghq/github.com/sideci/sideci )Oo
require: 'rubocop-rspec'

AllCops:
  Exclude:
    - 'vendor/**/*'
    - 'db/schema.rb'
  DisplayCopNames: true
  TargetRubyVersion: 2.4

Layout/AlignArray:

% rubocop | grep 'db/schema.rb'  # nothing displayed

% rubocop db/schema.rb | grep 'db/schema.rb'                                                                                ( ~/ghq/github.com/sideci/sideci )Oo
db/schema.rb:13:1: C: Metrics/BlockLength: Block has too many lines. [496/30]
db/schema.rb:24:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:26:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:48:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:88:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:102:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:215:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:272:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:340:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:353:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:397:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:406:13: C: Style/WordArray: Use %w or %W for an array of words.
db/schema.rb:481:13: C: Style/WordArray: Use %w or %W for an array of words.

See also https://github.com/bbatsov/rubocop/issues/5139#issuecomment-348678866

@pocke it's a git hook =). What can I show you in the hook that could be useful? Sorry, I'm not super familiar with linux so I'm not sure what all this does.
if command -v rubocop > /dev/null; then
  git status --porcelain \
  | grep -E '^A|^M' \
  | grep '.rb' \
  | awk '{print $2}' \
  | xargs rubocop
fi

It looks executing rubocop with changed files.
Can you try updating your script with the following? It adds --force-exclusion option to avoid this problem.
if command -v rubocop > /dev/null; then
  git status --porcelain \
  | grep -E '^A|^M' \
  | grep '.rb' \
  | awk '{print $2}' \
  | xargs rubocop --force-exclusion
fi

@pocke that resolved it. Thanks for the help!


Fetch submodules when computing hash

That's the default behavior of fetchgit, and raw fetchgit is used in output Nix expression, so this previously caused hash conflicts when submodules were present.

I just realized this breaks nix-prefetch for other kind of VCS'... maybe we should make sure every nix-prefetch command have a consistent way to retrieve the hash on Nix side.


[NETBEANS-785] java.lang.IllegalArgumentException

Prevent java.lang.IllegalArgumentException.
https://issues.apache.org/jira/browse/NETBEANS-785

Looks right, merging it.

If there is not any problem, I'll merge this tomorrow.


Return object name in GET

Given this object in the DB:
"posts": [     { "id": 1, "title": "json-server", "author": "typicode" }   ]
Doing a get I receive:
[     { "id": 1, "title": "json-server", "author": "typicode" }   ]
is there any way to receive something like:
"posts": [     { "id": 1, "title": "json-server", "author": "typicode" }   ],"status:{"code":200,"result":"something"}
?
Thank you very much

I really would like to help you but I couldn't understand your use case, do you mean in each request you want to inject the status code? if so, you can yous custom middleware, I can help you in this, otherwise, please explain it more.

I had a working version, but yours is heaps better, especially with the resourceName being dynamic!

I've used @Nilegfx code to make the key singular when requesting single object as well
eg. /posts/1 gets {"post": { "id": 1, ... }}
const pluralize = require('pluralize')
router.render = function (req, res) {
  var path = req.url.replace(/\/$/, '')
  var resourceName = path.split('/')[1] // To get the resource, /people, /posts/
  var last = path.split('/').pop() // To check if the path is resourceName or ID
  var statusCode = res.statusCode

  var json = {}
  if (statusCode < 400) {
    var key = resourceName == last ? resourceName : pluralize.singular(resourceName)
    json[key] = res.locals.data
  }
  json.status = { code: statusCode }

  res.jsonp(json)
}


Switch color icons to imageHtml

Switching from imageUrl to imageHtml for the color icons.
This is necessary to show them in the vs-code-bpmn-io extension, that has a stricter ContentSecurityPolicy for inlined image sources and styles.

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

@smbea : I experimented and found a setting that hopefully solves both issues.

I will follow up with releasing this


[CCI] Added @return to javadoc for MapperExtension class

Description
improved javadoc for this class
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #733 (0b27a6b) into main (a0548ff) will not change coverage.
The diff coverage is n/a.

: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     #733   +/-   ##
=========================================
  Coverage     43.34%   43.34%
  Complexity      311      311
=========================================
  Files            69       69
  Lines          1977     1977
  Branches        138      138
=========================================
  Hits            857      857
  Misses         1104     1104
  Partials         16       16

Impacted Files
Coverage Δ

...n/java/org/opensearch/sdk/api/MapperExtension.java
100.00% <ø> (ø)


Change the name?

Because it sounds like geotiff...!

or alternatively became more famous so they need to change name?


./scripts/docker_build_ml.sh all  fails with “unauthorized: authentication required"

Hi Dusty,
I am trying to build the docker  container on my nano (Jetpack 4.4.1), but the process halts with the message "unauthorized: authentication required"
I cloned the repository:
git clone https://github.com/dusty-nv/jetson-containers.git
cd ed into the repository:
cd jetson-containers/
./scripts/docker_build_ml.sh all
yields the following output:
reading L4T version from /etc/nv_tegra_release
L4T BSP Version:  L4T R32.4.4
l4t-base image:  nvcr.io/nvidian/nvidia-l4t-base:r32.4.4
building PyTorch torch-1.6.0-cp36-cp36m-linux_aarch64.whl, torchvision v0.7.0 (pillow), torchaudio v0.6.0
Building l4t-pytorch:r32.4.4-pth1.6-py3 container...
[sudo] password for XXXX:
Sending build context to Docker daemon  183.3kB
Step 1/22 : ARG BASE_IMAGE=nvcr.io/nvidia/l4t-base:r32.4.3
Step 2/22 : FROM ${BASE_IMAGE}
unauthorized: authentication required
Can you help me out of the ditch? I would eventually like to alter the Dockerfile in order to create my own version, but I already get stuck here.
Best,
Jens

Hi @jvschw , sorry about that - can you try updating your jetson-containers repo?
I just fixed this is commit 215b3c

Hi Dustin,
awesome, thanks for fixing this so quickly.
My attempt to add scikit-image (by adding the line
RUN pip3 install scikit-image --verbose
at line 93 of Dockerfile.ml) unfortunately failed, producing the error output below. I am aware that this does not pertain to the original issue, which I think is resolved, but maybe you can nevertheless help out.
Thanks again,
Jens
error message below:
Caching due to etag
Running setup.py (path:/tmp/pip-build-p5nhoan0/scikit-image/setup.py) egg_info for package scikit-image
Running command python setup.py egg_info
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-build-p5nhoan0/scikit-image/setup.py", line 30, in
LONG_DESCRIPTION = f.read()
File "/usr/lib/python3.6/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 4029: ordinal not in range(128)
Cleaning up...
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-p5nhoan0/scikit-image/
Exception information:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/pip/basecommand.py", line 215, in main
status = self.run(options, args)
File "/usr/lib/python3/dist-packages/pip/commands/install.py", line 353, in run
wb.build(autobuilding=True)
File "/usr/lib/python3/dist-packages/pip/wheel.py", line 749, in build
self.requirement_set.prepare_files(self.finder)
File "/usr/lib/python3/dist-packages/pip/req/req_set.py", line 380, in prepare_files
ignore_dependencies=self.ignore_dependencies))
File "/usr/lib/python3/dist-packages/pip/req/req_set.py", line 634, in _prepare_file
abstract_dist.prep_for_dist()
File "/usr/lib/python3/dist-packages/pip/req/req_set.py", line 129, in prep_for_dist
self.req_to_install.run_egg_info()
File "/usr/lib/python3/dist-packages/pip/req/req_install.py", line 439, in run_egg_info
command_desc='python setup.py egg_info')
File "/usr/lib/python3/dist-packages/pip/utils/init.py", line 725, in call_subprocess
% (command_desc, proc.returncode, cwd))
pip.exceptions.InstallationError: Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-p5nhoan0/scikit-image/
The command '/bin/sh -c pip3 install scikit-image --verbose' returned a non-zero code: 1


remove /en-us/ in links

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

biztalk/core/test-scenarios-for-measuring-mst-of-dta-tracking.md
:white_check_mark:Succeeded
View

biztalk/core/what-is-event-tracking.md
:white_check_mark:Succeeded
View

biztalk/core/what-is-message-tracking.md
:white_check_mark:Succeeded
View

biztalk/technical-guides/monitoring-sql-server-performance.md
:white_check_mark:Succeeded
View

his/core/code-page-mappings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off


added docker reference.

Added initial docker image reference. does not use https and runs on
port 9090.
Signed-off-by: Mike Lloyd<EMAIL_ADDRESS>

Do not merge this yet, it's blocked on the SSL support.

Coverage remained the same at 87.642% when pulling 96fba68ddecb2a565fdbebb45a60b44e394d419e on deployment-manifests into f31987a23e44c5121ef8c8b2f2ea2e8ffa37b068 on master.

Not sure when I'll get to this, closing for now.


Cypress e2e Maintenance - PF6/Test Enhancements/General Maintenance

https://issues.redhat.com/browse/RHOAIENG-17124
Description
Make additional fixes relating to PF6/Newer RHOAI builds (required as builds were down for a number of days).
Including:

Additional URL exclusions linked to https://issues.redhat.com/browse/RHOAIENG-9235
Fixed Workbenches tests to use s2i-minimal-notebook
Added verbose output for clusterStorage test which is currently failing due to an issue
Added steps for storageClasses test to enhance readability
Enhanced clusterSettings test to use correct logic following a discussion with Andrew
Created new logic for clusterAdminSettings to align with a bug fix https://issues.redhat.com/browse/RHOAIENG-7610

How Has This Been Tested?

An oc login should be performed in the cluster before running the test
test-variables.yml should be configured properly
Export the path to the test-variables.yml: $ export CY_TEST_CONFIG=<path_to>/test-variables.yml
Tests executed against 2.17 on a RHOAI Cluster locally and via Jenkins. Note, 3 tests are failing due to known bugs.

Jenkins - RHOI 2.17
cypress-tests/71/

Test Impact

None - this is a test

How to run
After exporting the test-variables.yml we have 2 different ways:
Using the UI
Go to odh-dashboard/frontend/src/tests/cypress and run the command npx cypress open . This will open the Cypress UI where you can run all the tests.
Headless
Go to odh-dashboard/frontend/src/tests/cypress and run the command npx cypress run --spec "cypress/tests/e2e/" --browser chrome
Request review criteria:

Self checklist (all need to be checked):

[x] The developer has manually tested the changes and verified that the changes work
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has added tests or explained why testing cannot be added (unit or cypress tests for related changes)

If you have UI changes:

[x] Included any necessary screenshots or gifs if it was a UI change.
[x] Included tags to the UX team if it was a UI/UX change.

After the PR is posted & before it merges:

[x] The developer has tested their solution on a cluster by using the image produced by the PR to main

@antowaddle Have you looked into running our cypress tests in the github actions split into jobs?
right now we have one job that does the entire test suite
but if we split our unit, lint, cypress, and even cypress areas into their own jobs? Jobs should be able to run in parallel so this would cut down on the one single job running at 45 min to multiple jobs and possibly only taking 10 min?
Just a thought and may be worth investigating while you are here

Test results in latest RHOAI 2.17:

/lgtm


Add archived filter to company search

Issue number: n/a
Description of change
Adds a filter for the archived field to company search.
Checklist

[x] Have any relevant search models been updated?
[x] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[x] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[x] Has the admin site been updated (for new models, fields etc.)?
[x] Has the README been updated (if needed)?

Codecov Report

Merging #992 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #992      +/-   ##
===========================================
+ Coverage    96.78%   96.78%   +<.01%
===========================================
  Files          195      195
  Lines         5501     5502       +1
  Branches       518      518
===========================================
+ Hits          5324     5325       +1
  Misses         116      116
  Partials        61       61

Impacted Files
Coverage Δ

datahub/search/company/views.py
100% <ø> (ø)
:arrow_up:

datahub/search/company/serializers.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Error with batch_jacobian Function: Unexpected Keyword Argument 'mode'

Hi Ziming,
When I run the code in the Physics_3_blackhole notebook, I encounter the following issue:

thanks for reporting this! Just solved, please pull the lastest version.


Update to Zeebe 8.5.0

@megglos FYI, I updated the go version to 1.21 because our client says it wants go 1.21: https://github.com/camunda/zeebe/blob/466c693d1625d0f0b89437eb43c21d546fd52b08/clients/go/go.mod#L3

@megglos FYI, I updated the go version to 1.21 because our client says it wants go 1.21: https://github.com/camunda/zeebe/blob/466c693d1625d0f0b89437eb43c21d546fd52b08/clients/go/go.mod#L3

thanks for raising, I wasn't aware this was bumped with https://github.com/camunda/zeebe/issues/15958 I will actually update the release announcements to raise awareness on this.


[fix][client] Fix deserialized BatchMessageIdImpl acknowledgment failure

Fixes https://github.com/apache/pulsar/issues/19030
Motivation
When a BatchMessageIdImpl is created from a deserialization, the BatchMessageAcker object cannot be shared by all instances in the same batch, which leads to an acknowledgment failure when batch index ACK is disabled (by default).
Modifications
Maintain a map from the (ledger id, entry id) pair to the BatchMessageAcker in ConsumerImpl. If the BatchMessageIdImpl doesn't carry a valid BatchMessageAcker, create and cache a BatchMessageAcker instance and remove it when all messages in the batch are acknowledged.
It requires a change in MessageIdImpl#fromByteArray that a BatchMessageAckerDisabled will be created to indicate there is no shared acker.
To avoid making code more complicated, this patch refactors the existing code that many logics about consumer are moved from the ACK tracker to the consumer. It also removes the AckType parameter when acknowledging a list of messages.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/BewareMyPower/pulsar/pull/15

More explanations to the refactoring parts of this PR:

I don't want to handle the list cumulative acknowledgment. Pulsar never supports this operation. There is no acknowledgeCumulative API that accepts a list of messages.
Process the BatchMessageIdImpl in ConsumerImpl. The previous code processes it mainly in PersistentAcknowledgmentTracker, it appearly does not follow the OOP style and there are a lot of consumer.getXXX() calls. I don't want to write more code based on that, e.g. using consumer.getBatchMessageToAcker() everywhere.

Thanks for your reviews, I'm going to address the comments soon.

@congbobo184 You comments are addressed now, PTAL again.

Codecov Report

Merging #19031 (f409267) into master (492a9c3) will increase coverage by 23.21%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #19031       +/-   ##
=============================================
+ Coverage     47.46%   70.67%   +23.21%
+ Complexity    10727      436    -10291
=============================================
  Files           711       26      -685
  Lines         69456     2247    -67209
  Branches       7452      245     -7207
=============================================
- Hits          32964     1588    -31376
+ Misses        32810      486    -32324
+ Partials       3682      173     -3509

Flag
Coverage Δ

unittests
70.67% <ø> (+23.21%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../apache/pulsar/client/impl/BatchMessageIdImpl.java

...va/org/apache/pulsar/client/impl/ConsumerBase.java

...va/org/apache/pulsar/client/impl/ConsumerImpl.java

...a/org/apache/pulsar/client/impl/MessageIdImpl.java

...he/pulsar/client/impl/MultiTopicsConsumerImpl.java

...pl/NonPersistentAcknowledgmentGroupingTracker.java

...impl/PersistentAcknowledgmentsGroupingTracker.java

...lsar/client/impl/conf/ClientConfigurationData.java

...apache/pulsar/common/naming/BundleSplitOption.java

...ker/stats/prometheus/AggregatedNamespaceStats.java

... and 675 more

Your comments are addressed now, PTAL @315157973
BTW, since I restored the API compatibility so I added the release/x.y.z labels.

As discussed on the mailing list https://lists.apache.org/thread/w4jzk27qhtosgsz7l9bmhf1t7o9mxjhp, there is no plan to release 2.9.6, so I am going to remove the release/2.9.6 label


Browser History based routing, improved unsaved change checking.

Description
This PR introduces deep linking / browser navigation support to Orchest. A huge quality of life improvement that will make it easier to work with Orchest like you would with other web applications.
In addition, since the unsaved changes checking is relatively tightly coupled with page navigation, we introduce improvement unsaved changes checking to make sure you don't accidentally lose your hard earned work.
Checklist

[x] The documentation reflects the changes.
[x] I have manually tested the application to make sure the changes don’t cause any downstream issues.
[x] In case I changed one of the services’ models.py I have performed the appropriate database migrations.

This PR is further discussed (and committed to) in: https://github.com/orchest/orchest/pull/151


Op-Ed and Letter to the editor

Contributions needed for an Op-Ed and letter to the editor
@vukosim and @MikeMcMalace and another Professor in Public Health Philosophy are busy with putting together an important Op-Ed and a letter to the editor based on how data is shared with the public.
We want inputs
We would like inputs from different contributors to assist with this contribution. We are asking for inputs in the following ways:
Please send a reply on your thoughts or ideas
Please provide a high level overview of your inputs.
The deadline for this is next Wednesday the 12th of January
We will create a shared Google Doc for all those interested so that we can work on this together. Authorship on the Letter to the editor with be shared, and your affiliations will be added if you wish to contribute.
At a high level, here is the argument we are working on
The start of any disease outbreak requires an epidemiological birds-eye-view which includes three specific data points/variables: Time, Place, and number. Initially these are good enough to make decisions, and calculate inferences based on this information. However, as time continues (and the outbreak in not contained), the level of sophistication and detailed required to make informed decisions increases over time for the decision makers and members of the public. We recommend monitoring the correct data and using the correct visualizations and indicators to respond effectively.
Key Points
The specific data sources that will be needed for the public to do the appropriate monitoring of the outbreak and elements within the healthcare system that needs to be shared, including hospitalization, will be explained so that the public can evaluate the impact of a disease outbreak/ public health concern.  These articles will outline the importance of data for public health measures in terms of disease outbreak and how information should be shared so that meaningful insights can be derived from both stakeholders managing the outbreak and members of the public.
Everyone that is working in this repo is valuable, and we would love inputs from you all
@krokkie @lrossouw @shaze @lizelgreyling @elolelo @heerden @dmackie @richardyoung00 @codeChap and everyone else who contributed toward this Repo are welcome to submit a comment. A reminder again the deadline is next Wednesday the 12th of January

it's not clear what outcomes you want based on the above?

@lrossouw, thank you for the question.
The outcomes that we want:

We are proposing a new way of sharing data and information to the public, in addition to what is already being shared
We want people who are not data or tech savvy to understand what any disease outbreak means by looking at simplified visualisations  different from the current infographics shared with people
Initial epidemiological responses focus on efforts led by three primary groups of variables, all related to Date, Time, and number. However, as outbreaks move to Epidemics etc. the level of sophistication required to make decisions (whether you are a public healthcare expert, or member of the Public) needs to be more nuanced and contextually relevant
We would like to know if would you like to be part of the engagement so that we can setup a Google doc and invite you to contribute on there and give your inputs

This is very interesting, and sounds like something I would certainly like to assist with if I can. I am currently travelling though, and won't be able to contribute anything meaningful until 12 January. However, if any inputs during the morning of 12 January could still be worthwhile to you, please count me in.

@lizelgreyling Most certainly

Once we have all the contributors, we will create a separate document so that we can collaborative work on this within the week. We will invite everyone end of day on the 12th to the document

Great, thank you for everyone's input. Please email me on<EMAIL_ADDRESS>if you want to be invited to the Google Doc to contribute to the output

Good morning,
We are going to invite the people to the Google Doc for comments on the Op-Ed, after the argument is developed, we will move onto the letter to the editor.
Thank you for all the responses.

@HerkulaasCombrink I suggest you put the link to the google doc here and ask people to ask for permission to access it.

https://docs.google.com/document/d/1EcF2mNqXL3RRhNBaC1s8O7AmRW10ImHwSD_d7kDPyfw/edit

Thank you everyone for your contributions thus far.
This is a reminder to contribute toward the peace.
Please contribute on the Google doc if you have not done so - so that you can be included in the Op-Ed. The Op-Ed is being drafted, so please use the link specified above for your contributions. If you did not contribute, you cannot be an author for the Op-Ed. We want to ensure that your valuable inputs are part of the process.

Sorry for the delay in responding. I have been on leave and back today. It will try to respond today

Thanks for the good start. Where are we aiming this for.
Having read it I think there are two issues: (1) the need for the underlying raw data to be released in a computer readable version and publicly accessible way; and (2) the need for public health authorities (and others) to be clear about how to present data in a way that is meaningful.
These two points are of course related but I think that the paper needs to separate out these more -- at the moment it weaves between the two which is a little confusing. I would suggest we separate them out: (a) the need for raw data to be available -- the problems if not available and examples of what can be done with this (b) successes and failures of the communicating the data and examples of how things could be done differently and what the impact of choices made are (e.g., showing results per capita or the gross amount -- gives a very different perspective of what the burdens or problems are)

@shaze we are writing the Op-Ed in the Conversation, the proposal for this is accepted and you will be added to the online editing platform once I have all the contributors handles.
I agree with your separation of these concepts in an academic paper, which is why we are submitting a letter to the editor. The part that was edited is just for the Op-Ed - hence the writing style etc. being a bit more emotive. Thank you for your comments prof. We can look online for the edits. If need be, we can also setup a brief virtual meeting with the group tomorrow just to make sure that we all agree and are comfortable with the argument.

Dear everyone, nearly there with the Op-Ed. The academic article will follow this one, with a similar logic but with more detail.
Thank you for your contributions thus far. We are nearly finished with the Op-Ed. The updated article is in the Doc, and you all would have been invited to make edits online. There are three comments we still need to address. Please go through these as at 12pm on the 24th, I will submit this to the Conversation.

In case you haven't seen it
https://twitter.com/EricTopol/status/1496135944193425414/photo/1

Sorry -- this is a better link: https://www.nature.com/articles/d41586-022-00424-9


Improper GetAsyncKeyState usage

Contrary to superficial appearances, GetKeyState and GetAsyncKeyState actually have significantly different behavior, aside from skipping the input buffer queue.
GetKeyState: docs

If the high-order bit is 1, the key is down; otherwise, it is up.
If the low-order bit is 1, the key is toggled.

Allows detection of lock-key toggle state (e.g. "is CapsLock on?")
Allows detection of currently-pressed state (e.g. "is CapsLock being pressed?")

GetAsyncKeyState: docs

If the most significant bit is set, the key is down
If the least significant bit is set, the key was pressed after the previous call to GetAsyncKeyState. However, you should not rely on this last behavior

Does not detect toggle state (try it! Note, this is different from the way Robot works on OSX)
Allows detection of "recent" keypresses (almost useless since this can get reset by other processes, and is different from the way Robot works on OSX)
Allows detection of currently-pressed state (e.g. "is CapsLock being pressed?")

I think Robot should probably be using GetKeyState rather than GetAsyncKeyState.  The reasoning for using GetAsyncKeyState is that it allows you to check the current realtime keyboard state regardless of queued input to your process, however it is your process so you could just as well drain your input before calling GetKeyState to get the same result, with the added benefit that you can obtain lock-key toggle information, and can also obtain input-buffer key information if you want.
One still-missing feature would be the ability to distinguish between CapsLock-on-and-down, CapsLock-on-and-up, and CapsLock-off-and-down. Unfortunately, sorting this out would probably require a new method signature other than bool Keyboard::GetState (Key keycode), and it's fairly rare to need to know if CapsLock (or ScrollLock or NumLock) is actually being pressed at the moment.

I wrote:

One still-missing feature would be the ability to distinguish between CapsLock-on-and-down, CapsLock-on-and-up, and CapsLock-off-and-down. Unfortunately, sorting this out would probably require a new method signature other than bool Keyboard::GetState (Key keycode), and it's fairly rare to need to know if CapsLock (or ScrollLock or NumLock) is actually being pressed at the moment.

But upon further testing, it seems that the GetKeyState function only returns lock-state for lock type virtual keys, so this conflation is actually not really present anyway -- if you are testing VK_CAPITAL, you get lock-state only, not lock-state | down-state.  And if you are testing some other non-lock key, you only get down-state.


Standardize the Navbar

Navbars on different sites(sub-domains of RDS) are different as of now.
We need to make it look similar and familiar to the user. So we are going to redesign it.
Design of New Navbar:
https://www.figma.com/file/bpFHG9hXM13GSLmTkoNsrF/Navbar?node-id=0%3A1
For CSS, refer to the link below
https://github.com/Real-Dev-Squad/website-www/blob/develop/css/navbar.css
Few Details

font : Roboto
weight: 700
active color : #49A82E
hover color : #49A82E
font size : 16 px (try making it in rem)
active underline height : 3px (color : #49A82E)
RDS Logo Size - 50x50

Pay Attention
=> Underline should be under the text as shown in the design and not inside the container in which the text persists.
=> Underline should be only under the active tab.
=> Clicking on the RDS logo we should be redirected to https://www.realdevsquad.com/
=> The full navbar should be changed to the toggle menu when the screen width is less than 970px.
=> User should be directed to my.realdevsquad.com on clicking User Greet
There might be slight changes for different domains and you are free to ask in the comments for that

HI @rohan09-raj , I'll pick this task.

HI @rohan09-raj , I'll pick this task.

As you already have task to work on unassigning from you.

I'll take it up

Please assign this to me

@vvaibhavdesai Please continue from Akanksha's draft PR and fix the issues on it


learning rate?

Did you adopt the lr scheduler in the experiments ? or it's ok for 1e-4 all the time?

If I remember correctly the results were slightly better when using a lr scheduler.


add multiple menus over two level display on left nav-bar?

Feature Request
I'm using the awasome theme, but when config mutilple menus over 2 levels on layout docs, it cannot display on left nav-bar normally, so only config level below 2 levels;
Is it possible to support display mutilple menus on left nav-bar?
example:
L1/1.md
L1/_index.md
L1/L2/_index.md
L1/L2/2.md
L1/L2/L3/_index.md
L1/L2/L3/3.md

these files like this. I want to show left nav-bar like this directory tree, but It DO NOT!
by the way,  when I manage lots of markdown files, It is tedious to add [[menu.X]] in every markdown files, Is it possible to support generate menu by directory and subdirectory name?

The menu system is implemented by Hugo - you can open a request with Hugo team to make it easier to define menus: https://github.com/gohugoio/hugo/issues .


Option to automatically loop through paginated playlists with length more than 100 in spotipy.spotify.playlist() function

Is your feature request related to a problem? Please describe.
With the spotipy.spotify.playlist() function the items or tracks returned by default is limited to 100 and the results are paginated. And there is no way to read further details.
Describe the solution you'd like
Provide a parameter which would take care of looping through the paginated results and return the full playlist items.
Describe alternatives you've considered
An alternate solution that I ended up using is to get the total track number and loop over the results using spotipy.spotify,playlist_items with offsets at steps of 50 (max value allowed by spotify) to get the complete playlist items. Please see the code example below
sp = spotipy.Spotify(access_token)
results = sp.playlist(playlist_id)

total_track_count = results['tracks']['total']
max_limit = 50
offset = 0
remaining_track_count = total_track_count

tracks = []

while remaining_track_count > 0:
    results = sp.playlist_items(playlist_id,limit=max_limit,offset=offset)
    tracks.extend(results['items'])
    offset = offset + max_limit
    remaining_track_count = remaining_track_count - max_limit

Won't do for the reason given here https://github.com/plamere/spotipy/pull/751#issuecomment-991626458
However thanks both for raising this!


XDP programs are accounted to xdp-dispatcher

Thanks for bpftop! I gave it a try and saw that XDP programs - which are managed by something called xdp-dispatcher - are not accounted indiviually, but rather to the xdp-dispatcher.
Is there a way to distinguish BPF statistics between individual XDP programs, or is this something that would have to be fixed in XDP?

@hhoffstaette, could you paste the output of sudo bpftool prog? I want to see how that tool aggregates your apps.

Here you go:
bpftool prog
2: tracing  name dump_bpf_map  tag 3d73ab123e737406  gpl
	loaded_at 2024-02-27T17:26:40+0100  uid 0
	xlated 280B  jited 166B  memlock 4096B  map_ids 2
	btf_id 86
3: tracing  name dump_bpf_prog  tag a555684b684cb7c0  gpl
	loaded_at 2024-02-27T17:26:40+0100  uid 0
	xlated 520B  jited 647B  memlock 4096B  map_ids 2
	btf_id 86
13: xdp  name xdp_dispatcher  tag 90f686eb86991928  gpl run_time_ns 18735 run_cnt 32
	loaded_at 2024-02-27T17:26:42+0100  uid 0
	xlated 672B  jited 516B  memlock 4096B  map_ids 7
	btf_id 97
22: ext  name homeplug_drop  tag fbd415544de357c1  gpl
	loaded_at 2024-02-27T17:26:42+0100  uid 0
	xlated 136B  jited 91B  memlock 4096B
	btf_id 100

Sorry for the naming confusion. The CLI tool is alled xdp-loader, whereas the XDP manager is called xdp-dispatcher, which is an XDP program itself, see here

I think the problem here is that XDP pipelines are "not really" individual programs, but I'm not familiar enough with the internals and eBPF statistics mechanism, hence the question.

Ok, this type of eBPF pattern is new to me. To the Kernel, this will look like just loading one XDP app. At first glance, it seems like the libxdp library loads each individual XDP program as an ext type, and then the dispatcher possibly calls to each one using this feature.
This is where the stats we use are incremented by the Kernel when a BPF program exits. It appears to me that it's not doing that for these ext prog types. https://github.com/torvalds/linux/blob/master/kernel/bpf/trampoline.c#L874
I'm very curious about this, I'll dig into the library a bit more to understand it.

Thanks! If this turns out to be impossible at the moment we might need to open an issue in xdp-tools and see what Toke thinks.

The dispatcher created by libxdp (used by xdp-loader) declares a few global dummy functions to act as the "slots" where XDP programs can be hooked. When a new XDP program needs to be attached, libxdp replaces one of the dummy functions with the code for the XDP program being loaded using the freplace mechanism.
So when using libxdp and the dispatcher, there's a single XDP program (the dispatcher) and each additional user program will be an EXT type eBPF program attached to the dispatcher.
A bit more context on this:
https://github.com/xdp-project/xdp-tools/blob/master/lib/libxdp/protocol.org
https://ebpf-docs.dylanreimerink.nl/linux/program-type/BPF_PROG_TYPE_EXT/

@mscastanho, thanks for the additional context. It appears to me that these EXT type programs do not go through the bpf/trampoline.c path I linked above, and thus, don't get individual performance statistics. If that is the case, then a Kernel patch would be required to enable stats for them. I'll verify that's the limiting factor here and investigate the feasibility of this.

@hhoffstaette could you attach an example libxdp app that I can use to verify a few things?

I started a thread in the BPF mailing list to discuss this limiation https://lore.kernel.org/bpf/20240407052135.n3vwjrhw22kjehrh@ubuntu/T/#u


Add no-string-jsx rule

PR checklist

[ ] Addresses an existing issue: #0000
[x] New feature, bugfix, or enhancement

[x] Includes tests

[x] Documentation update

What changes did you make?
Adds no-string-jsx rule for projects that use translation functions to enforce use of those functions

Moved to https://github.com/palantir/tslint-react/pull/65
Any ideas about the failure there?


Revise household propagation for hospitalised people

Hospitalised people should not propagate infections across their household as much as non-hospitalised people.

This is a funny one, because I just checked and this bug actually doesn't exit! ;)


refactor suggestion: add g`/g' clues together with their children

Minor nit, but given that MiniClue.gen_clues.marks() generates clues for keymaps prefixed by g' or g`, I think it would make more sense to move the clue for g' and for g` inside MiniClue.gen_clues.marks(), so that the prefix descriptions are included as well by only using MiniClue.gen_clues.marks() :)

Thanks for suggestion, but I don't think this is a good idea. For at least these reasons:

It would introduce duplication of clues for  "suggested" setup with all gen_clues entries.
It would be visible only if g is set as trigger, which it doesn't have to be for marks.
It is not much burden to add for people who both use g as trigger and gen_clues.marks() but not gen_clues.g() as clues.

Closing as not planned.

It would introduce duplication of clues for "suggested" setup with all gen_clues entries.

Well my suggestion was moving the clues, not copying them.
The other 2 points are fair though.


Downgrade itsdangerous<2.1.0 in app/requirements.txt (fix generate model graph failure in CI)

Flask is importing itsdangerous.json which has been removed in 2.1.0. The
documentation said to use the standard library instead.
Error when generating model graph:
Traceback (most recent call last):
  File "/home/runner/work/houston/houston/./scripts/docs/generate_models_graph.py", line 34, in <module>
    for model in get_models():
  File "/home/runner/work/houston/houston/./scripts/docs/generate_models_graph.py", line 13, in get_models
    config = get_preliminary_config()
  File "/home/runner/work/houston/houston/config/__init__.py", line 96, in get_preliminary_config
    config_cls = import_class(CONFIG_CLS_MAPPER[ctx][env])
  File "/home/runner/work/houston/houston/config/utils.py", line 18, in import_class
    module = importlib.import_module(path)
  File "/opt/hostedtoolcache/Python/3.9.10/x64/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/home/runner/work/houston/houston/config/codex.py", line 5, in <module>
    from .base import (
  File "/home/runner/work/houston/houston/config/base.py", line 8, in <module>
    import flask
  File "/opt/hostedtoolcache/Python/3.9.10/x64/lib/python3.9/site-packages/flask/__init__.py", line 19, in <module>
    from . import json
  File "/opt/hostedtoolcache/Python/3.9.10/x64/lib/python3.9/site-packages/flask/json/__init__.py", line 15, in <module>
    from itsdangerous import json as _json
ImportError: cannot import name 'json' from 'itsdangerous' (/opt/hostedtoolcache/Python/3.9.10/x64/lib/python3.9/site-packages/itsdangerous/__init__.py)

/rebase

/rebase


Use es6 class instead of mixins for React.

Is there any plans to update the react mixin to use a es6 class (It seems like mixins are being phased out)?

I see no reason.

The reason being that people using the es6 classes can't take advantage of the mixin(easily). If it was written as a composable es6 component it would still work with "create class" components.

In my opinion such a little while, and those who use es6 know how to get around this limitation.

Would you consider a PR? (Obviously it would break the existing react api)

If your solution will not break backward compatibility for those who do not use ES6, then yes.

There is es7 decorators which babel and typescript already support it.
It can easily use same as react mixin.

Do you have example/PR of es7 decorator instead of mixin?
There are actually few good reasons<EMAIL_ADDRESS>

The big advantage is decorators is an ES feature. It's can work with ES6 classes, which react mixin force you to not use ES6 classes.
I found something may be useful.
http://asaf.github.io/blog/2015/06/23/extending-behavior-of-react-components-by-es6-decorators/
This may be the simplest react decorator.
https://github.com/felixgirault/pure-render-decorator
There is a module that turns a mixin into a decorator. But I haven't used yet.
https://github.com/timbur/react-mixin-decorator

any news about this issue? i cannot seem to figure out how to work with this mixin as HOC.
specially Sortable not having access, to internal components ref's and such.
+1 for decorators.


Add some examples of using the example templates

Add some examples of how a couple of the templates could be used.

I have a few minor comments on this:

How do intrinsic types and kinds fit in? (Possibly even parametrised types)
I see that interfaces are consistently defined, like:
interface filter    module procedure filter_ end interface
Is that a necessary part of the template specification?
In the interfaces (map_) the type T and U are not imported. Is that deliberate or an oversight?
Line 62 has typos: "ys%indeces" in stead of "ys%indices"
Is the template parameter (template sparse_array_tmpl(T) for instance) automatically public? Or does that not matter?
In swap_m, assignment of the type(T) variables is used, How to require that (intrinsic) operations or assignment (intrinsic or otherwise) are defined? What syntax to use?

These are detailed remarks and perhaps they should be put elsewhere but reading the code examples I began to wonder about these details.


DeployAppDefinition fails due to recent change

It seems that the contents of the  element in the .btdfproj are not being processed. In release 5.0.13 the references are no longer added to application being deployed. I had no problems in 5.0.11.  It's seems it has to do with some changes that were applied on the DeployAppDefinition target in the BizTalkDeploymentFramework.targets file.

I replaced the the DeployAppDefinition target found in the 5.0.13 release with the one from the 5.0.11 release and everyting works fine now.

release 5.0.13

<Exec Command="BTSTask.exe AddApp -ApplicationName:$(BizTalkAppName) -Description:"$(ProjectName)"" />

release 5.0.11

This work item was migrated from CodePlex
CodePlex work item ID: '6231'
Vote count: '1'

[tfabraham@3/24/2010]
Please review the V5.0 Release Notes.  The file structure changed as of 5.0.12 and you have not modified your .BTDFPROJ file to match the new format.

[bugfix@3/24/2010]
Hmmm I guess I must have overlooked it. Works like a charm!

[UnknownUser@3/24/2010]

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]

Issue closed by tfabraham with comment
Working as designed


Safari 9 undefined is not an object (evaluating 'window.top.document.referrer')

On Safari 9.1.3 we get error
undefined is not an object (evaluating 'window.top.document.referrer')
Seems like it's related to https://github.com/BranchMetrics/web-branch-deep-linking/commit/5c10ccf5f3f8e18b520099a5e8e4e79df52b19e9#diff-7ab92c1898404b32a293ae8501a733f9R1262

Due to the age and inactivity of this issue, I am closing this now. If you are encountering this problem still, please reach out to our support team (support.branch.io) or open a new github issue and we would be happy to help you out!


Multi machine setup - issues

Hello,
I've set up a five machine environment (Portal, hosted Server, Data Store, Image Server, GeoEvent Server). I used three configuration files:

BaseDeployment-ThreeMachine-10.6-ssl
GISServer-GeoEvent
GISServer-GeneralPurpose

During and after the installation, I see two issues:

Configurations 2. (GeoEvent Server) and 3. (Image Server) display incomplete server URLs.

The various servers are federated in portal with different ports (443 and 6443 respectively), although in all three configuration files I declared port 443 as "Federation - PortalPort".

Any idea or explanation for these issues?
Thank you very much, best regards,
Jürgen

Pull request #95 should solve this issue. You can fork the PR and try it out.

Closing the issue. Please open again if you have any further issues.


chore: consolidate several renovate-bot changes

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Can we put "Closes #X" for each of the bot PRs this replaces in the PR description?

Can we put "Closes #X" for each of the bot PRs this replaces in the PR description?

Sure


error https://nuxters.nuxt.com/nuxter/[username] not found

Impossible to show the route Atinux nuxter route https://nuxters.nuxt.com/nuxter/Atinux.
I think you need to modify the API call https://github.com/nuxt/nuxters/blob/main/pages/nuxter/[username].vue#L9 with the https://api.nuxt.com/contributors route

It's actually in progress in https://github.com/nuxt/nuxters/tree/feat/single-page

https://nuxters.nuxt.com/maximepvrt 👀


Changing the timer intervals should apply immediately

When changing a timer interval, it should apply immediately.
For example, when the Data timer is set to 5 minutes, when the application starts, it sets a timer for every 5 minutes.
When I change the timer to every 1 minute, it should:

immediately run the data tick
delete and recreate (or update) the timer interval to the new 1 minute interval

Fixed in 0.0.2


Assertions within async callbacks timeout instead of failing

So it seems like any assertions that are made within meteor-specifc callbacks (e.g. Meteor.setTimeout or Meteor.call) will not fail the test immediately, and will wait for mocha to timeout.
@dangilkerson has a solution in https://github.com/mad-eye/meteor-mocha-web/issues/70. The solution is illustrated in the last example below, but it would be nice to not have to do this.
Seems like failed assertions should behave in the same way as if the assertion passes, and cause the test to fail immediately. It also means that unless you use the try/catch solution, the test failure is reported as 'timeout' instead of the actual assertion.
Here's a few examples to demonstrate:
      it 'will pass immediately', (done) ->
        Meteor.setTimeout ->
          assert.equal 1, 1
          done()
        , 1

      it 'will fail after 2000ms', (done) ->
        Meteor.setTimeout ->
          assert.equal 1, 2
          done()
        , 1

      it 'will fail immediately', (done) ->
        Meteor.setTimeout ->
          try
            assert.equal 1, 2
            done()
          catch err
            done err
        , 1

Could this please be marked somewhere in the README?
It's an incredibly unintuitive behavior and coming up with the workaround isn't that obvious either.
Especially for people that are just getting started...

I tried using the try/catch in my Meteor app to catch the asynchronous assertions and it prevented velocity from working. I had to remove the try catch and velocity would then reload and run the tests again.
The solution that worked for me was wrapping the chai.assert statement in a setTimeout. Does this solution work for others using Meteor mocha? Any downsides to this hack/fix?


Windows Phone 8 never really discards NFC listener properly and re-assigning doesn't seem to work properly

If I have a page..
My HTML files
Index.html
createAction.html
I browse to createAction.html, do addNdefListener, write some tags then do a window.location = index.html then re-visit createAction.html the second time the addNdefListener fails..
Initialized the NfcPlugin is only output..
I think if there is a way I can check before I try to do addNdefListener if it's already been executed that would help..  Can you replicate this issue on WP by trying to add addNdefListener twice?
It's very probable my code is to blame here, it works fine on Android tho..

This works on windows platform. Tested with Lumia 640 running Windows Phone 8.1 Update 2.
Not planning on fixing Windows Phone 8 / SilverLight version.
@JohnMcLear if your still seeing a problem re-open with an updated example.


no episodes found for intercepted, deconstructed

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/GetStream/Winds/issues

Description
Winds is unable to find episodes for The Intercept's Intercepted and Deconstructed podcasts.
Steps to Reproduce

Search for "Intercepted" or "Deconstructed"
Select the associated "Podcast"

Expected behavior:
I expect the podcast episodes to appear in the app.
Actual behavior:
A screen appears that says:
We haven't found any episodes for this podcast feed yet :(

It might be because the podcast feed doesn't have any episodes, or because it just got added and we're still parsing them. Come check back in a few minutes.

If you're pretty sure there's supposed to be some episodes here, and they aren't showing up, please file a GitHub Issue.

Reproduces how often:
100%
Versions
Version 2.0.195 (2.0.195)
Additional Information
I'm happy to help debug or help fix the problem.  Let me know what i can do to help!

This issue has been addressed (see attached image). Thanks for bringing up the issue!


[CommonCodeClean] Remove getElemsPerThreadForOperands

Remove getElemsPerThreadForOperands

This PR waits the upstream changes to be merged. https://github.com/triton-lang/triton/pull/5328

LGTM assuming upstream PR get accepted.

Let's wait for the upstream PR to be accepted before merging


Flow does not detect type error in argument list at function call

Here is the code sample:
// @flow
someLib.module("xyz", function(exports) {
exports.app = new someLib.ClassDefinition(function() {
return {
'$extends': demo.Application,
total: function(a:number, b:number) {
return a+b*3;
}
'someFunc': function() {
this.total(3,"Hi");
}
};
});
});
var obj=someLib.module("xyz").app;
obj.prototype.total("Hello", 12);
When running this code with flow, the type error in the function call (when we pass a string to the function: total instead of a number) is not detected at all. Whereas if there is a problem inside the function itself (with the return type ie.,), Flow throws an error. However, if i look at the logs, i see an error like below:
Error: Sys_error("Invalid argument")
Raised by primitive operation at file "pervasives.ml", line 381, characters 12-32
Called from file "src\server\serverProt.ml", line 102, characters 10-23
Called from file "src\server\server.ml", line 639, characters 14-51
Called from file "src\server\serverFunctors.ml", line 134, characters 6-43
I suppose this error happens because oCaml is trying to convert the string to a number. But my question is how to make Flow detect the error at the line where the erroneous function is being called and display it  ?

Related, FYI: Jetbrains Webstorm -- [Flow] WS gets error from flow.exe

I have another example of this error:
declare var plugin: {
   set_log_level: Function
};
type logLevel = 1 | 2 | 3|  4

export function setLogLevel(level: logLevel): void {
   plugin.set_log_level(level);
}

but, i solved it like that:
type logLevel = 1 | 2 | 3|  4

declare var plugin: {
  set_log_level: (level: logLevel) => void
};

export function setLogLevel(level: logLevel): void {
   plugin.set_log_level(level);
}

Do not get any errors.


[Translate ru] Accessibility ->how-to-review

Check out a review request at GitLocalize

@agektmr Reviewed

:tada: This has been pushed live to https://developers.google.com/web/


Create Staging Environment for Firebase

I need to create a copy of the prod Firebase instance to use as an environment for other devs to work on the app. I did not want to expose the prod server just yet as I do not know the extent to which it will cost me traffic-wise until it goes live. I'm also trying to avoid folks with ill intentions taking the app and trying to run up the request limits/traffic artificially - not sure how this whole Firebase thing works yet. :)

Hello! Comp Sci major whos a big fan with not enough time to be able to assist. But a suggestion to this still open issue (if it's still prevelant).
Have you tried exporting the current firebase db and putting it in a sub folder at the repo's highest directory (be sure to omit all your account info if thats included in the export). Then just ammend the README to have the location where individual firebase account credentials can be entered in the api calling classes.  Firebase has a free trial plan thats more then enough for a single dev to test on their own. Contributers can just get the free version and manipulate their own db as to not interfere with public operations.
Best of luck!


Cannot type # with Neo keyboard layout

VSCode Version: 1.5.2
OS Version: OSX 10.11.6

Steps to Reproduce:

Install and activate Neo
Try to type a # in any file (option+ü with Neo)

Expected Result
The character # gets inserted at the cursor position.
Actual Result
No character gets inserted at the cursor position.

Might that key combination be used by one of the keybindings?

I looked through VSCode's menu items, but did not find any item that had that key binding. I know that keybindings of option/alt+something are common on Windows and Linux, but not so much on OSX, since option is used as keyboard modifier to type characters.
I did a quick scan through the system-wide keyboard shortcuts, but did not find anything that maps to option+ü.

Ah, there is also the keybinding JSON of VSCode. I scanned through that too, but did not find anything.

Do other combinations of option+umlaut work?
When you open the keybindings JSON, there is a 'Define Keybindings' button in the lower right of the writeable editor. Could you press that and then type option+ü in the opened text field and let me know what that tells you?

Other combinations with umlauts work, for example, option+ö yields $ as expected.
Define Keybindings shows alt+z when I press it. This explains what happens, since this is the keybinding for Toggle Word Wrap. Further tests reveal that the actual problem seems to be with umlauts: Every key combination with umlauts shows up as if I pressed the keys in QWERTY layout. All other key combinations show up correctly according to Neo layout. So I can get alt+z by either actually pressing option+z or option+ü.
To further investigate, I switched to standard german QWERTZ layout, and there, the umlauts also generate key shortcuts as if the keys which reside there in standard QWERTY are pressed. So, the issue seems to be that for umlauts, VS Code for some reason interprets the keycode as QWERTZ input.

@flyx Can you please try the following:

open Preferences > Keybindings
click on Define Keybinding
press the keybinding that isn't working for you (option+ü with Neo)
look at how we receive it from Chromium (the thing shown in the textbox)
it might be something like alt+3 or whatever
there might be a default keybinding we ship with that captures it.
search for that keybinding in the default keybindings
remove the default keybinding we ship with by writing the rule { "key": "alt+3" } in your keybindings.json (use whatever we got instead of alt+3 -- I'm just guessing here)

@alexandrudima I answered most of these questions already in my previous comment. Define Keybinding recognizes option+ü as alt+z, which is an existing keybinding. As I said, the underlying problem is that shortcuts containing umlauts are misinterpreted as shortcuts containing the key at the umlaut's position in QWERTY layout. This is easily reproducible with the standard german QWERTZ layout.
I will try editing the keybindings later today and I think that it will solve my symptom, but not the problem itself. I suggest that:

VS Code does not predefine any keyboard shortcuts containing only the option modifier and some printable-character-key for OSX, because those are used for writing characters.
This issue gets propagated to Electron, Chromium or wherever it belongs, since from what you wrote, this is likely where the keyboard shortcut gets misinterpreted.

I will report the result of editing the keybindings later.

Adding { "key": "alt+z" } to my keybindings.json resolves my issue.

@flyx Starting with today's insiders version (i.e. after PR #22894 aka #17521) we have changed how keybinding dispatching works on OSX from using e.keyCode to using e.code (i.e. scan codes). Can you please download the insider's version and check if we work now correctly (i.e. labels reflect what you press).

It works correctly with the insider's version.

Thank you for following up on this


HELICS 2.0 Java interface and test cases

Issue for completing the HELICS 2.0 Java interface and test cases

merged to 2.0 in #481


Separate section for "errors and warning" output

See screenshots below; I use VCS for Golang development and after saving a file, the output goes into the "Errors and Warnings" section. However this is ONLY visible in the status bar on the bottom of the editor. I would like the output to be visible in a separate section of the screen. Is this possible now? Or is there some kind of extension which does this?
I know I can use COMMAND + SHIFT + M to open it, but I would like to have it visible directly after saving my file.
[edit]
Kind of related: https://github.com/Microsoft/vscode/issues/1135

@alexandrudima did you not start this?

The Error List is in limbo.
It can be enabled with a hack I use every now and then:

Reveal Debug Console (Ctrl+J on Windows)
Open developer tools
Go to Resources -> Local Storage
Find the key containing the folder + workbench.panelpart.activepanelid.
Edit from workbench.panel.repl to workbench.panel.errorList.
Reload window.

Works until the Debug Console is shown again.

@alexandrudima cool, I think this is a good start to demo for UX to get an idea, adding the label.

Closing, we have problems panel for a while now.


provide ability to customise shape properties

Currently it is difficult and time consuming to customise individual shape properties within a pptx. We do have a 'shape_properties' parameter which takes a dict defining properties however it needs to be more flexible. I plan to expose all shape settings currently available to a user to customise their pptx as they wish.

Resolved by #354.


My account info was missing from my commits

The commits I made in the last few days were the first time I contributed on Github. Unfortunately, my Git config wasn't correctly setup and so my account was not actually connected to those commits. The commits all had no email address and the name of "Andrew Lord", instead of my email and name of "andrewlord1990".
I ran the script from this page:
https://help.github.com/articles/changing-author-info/
This has correctly linked my commits to my Github account. However, it works by rewriting the git history.
I understand if you don't want to allow this, however, I would really like it if my commits could be linked to my account. Would there be anyway for this to be made possible?
An alternative would be if you reverted all of the pull requests I made and I redo all of the commits and make the pull requests again? I would really appreciate it.

Yeah this is a bit much to write over everything at the moment

Yeah that is okay, I understand.
I will revert all the changes I made, then re-commit them and create a pull request for that. That means the commits will have my details correctly connected and there is no dangerous changes to the repo.


pipeline: Manually delete problematic file for Azure Windows

ref: https://github.com/AdoptOpenJDK/openjdk-build/pull/1892/ , https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1419 , https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1396
Changes to allow the problematic ..the_ file to be deleted for both Softlayer and Azure windows machines before cleanWS is used. I was unsure if there was a specific variable that could've been used to make this more general ( i.e.  context.sh(script: "rm -rf {buildconfig.WORKSPACE}/build/src/build/*/jdk/gensrc , if anyone could enlighten me?
ping @M-Davies @sxa @karianna
Signed-off-by: William Parker<EMAIL_ADDRESS>

@Willsparker I think ${WORKSPACE} can be used - you'll need to test the substitution though.

run tests

@M-Davies How would I go about testing this PR?

@Willsparker I setup a WIP pipeline for you to play around with https://ci.adoptopenjdk.net/view/work in progress/job/will-jdk11u-windows-x64-hotspot/

Perfect, thanks! :) Unfortunately, I don't think ${WORKSPACE} will work:
Failed to clean groovy.lang.MissingPropertyException: No such property: WORKSPACE for class: Build

https://ci.adoptopenjdk.net/view/work in progress/job/will-jdk11u-windows-x64-hotspot/6/
It appears keeping the two rm -rf commands separately works anyway. I'll change the PR back to using 2 separate scripts, and I'll open an issue to improve it in the future- by someone whose better at groovy than I.

Do we have any idea where this file is coming from?

It's being generated from a build, but I'm not sure why or how, or why it's 0 length. https://github.com/AdoptOpenJDK/openjdk-jdk11u/search?q=_the..&unscoped_q=_the..

I wonder if we could refactor the buildSCripts function and split out the clean workspace into its own function

I wonder if we could refactor the buildSCripts function and split out the clean workspace into its own function

That might be a nice addition to the jenkins helper seeing as we use cleanws() so often in multiple repos


Laravel Analyzer

This pull request includes changes and recommendations for crafting your application "The Laravel Way". Feel free to commit any additional changes to the shift-9152 branch.
Before merging, you should:

Checkout the shift-9152 branch
Review all pull request comments for additional changes
Thoroughly test your application

If you want help with your upgrade, check out the Shift Human Services or join the Shifty Coders Slack room to get answers to all your Laravel questions.

⚠ The following controllers contain actions outside of the 7 resource actions (index, create, store, show, edit, update, destroy). For more details, review the docs or watch Cruddy by Design to see if you can rework these into resource controllers.

[ ] app/Http/Controllers/Admin/ThreadController.php
[ ] app/Http/Controllers/AdminController.php
[ ] app/Http/Controllers/Auth/LoginController.php
[ ] app/Http/Controllers/Auth/RegisterController.php
[ ] app/Http/Controllers/Frontend/PageController.php
[ ] app/Http/Controllers/Frontend/ProfileController.php
[ ] app/Http/Controllers/Frontend/ThreadController.php
[ ] app/Http/Controllers/FrontendController.php

⚠ The following classes do not extend the standard Laravel Model and Controller class. This may add complexity which makes your application harder to upgrade. Often a trait can be used instead of inheritance. You should review the following classes:

[ ] app/Http/Controllers/Admin/DashboardController.php
[ ] app/Http/Controllers/Admin/ThreadController.php
[ ] app/Http/Controllers/Admin/UserController.php
[ ] app/Http/Controllers/Frontend/HomeController.php
[ ] app/Http/Controllers/Frontend/PageController.php
[ ] app/Http/Controllers/Frontend/ProfileController.php
[ ] app/Http/Controllers/Frontend/ThreadController.php
[ ] app/Models/Post.php

⚠ The following files reference Laravel facades through the global namespace. For example, you're referencing \DB or importing use DB. Instead import Illuminate\Support\Facades\DB and reference DB.
While global references are allowed through aliases, you should import the facade explicitly. This can improve clarity not only for developers, but static analysis used by your IDE.

[ ] app/Http/Controllers/Controller.php
[ ] app/Http/Controllers/Frontend/ThreadController.php
[ ] app/Http/Controllers/Frontend/ThreadController.php

⚠ Shift found inline validation in the following controllers. Unless you require this level of control, you should use a Form Request to encapsulate this validation logic and keep your controller clean.

[ ] app/Http/Controllers/Auth/RegisterController.php
[ ] app/Http/Controllers/Frontend/ThreadController.php

⚠ Shift found opportunities to use the built-in Blade directives like @auth, @csrf in the following views:

[ ] resources/views/auth/passwords/email.blade.php
[ ] resources/views/auth/passwords/reset.blade.php
[ ] resources/views/frontend/partials/header.blade.php
[ ] resources/views/frontend/partials/login-form.blade.php
[ ] resources/views/frontend/partials/register-form.blade.php
[ ] resources/views/frontend/partials/thread-meta.blade.php
[ ] resources/views/layouts/admin.blade.php

⚠ Shift detected the following HTTP components accessing the authenticated user through Auth::user(). Within these components, you can access the authenticated user through the request object as well. Doing so limits your dependencies by leveraging objects already available.

[ ] app/Http/Controllers/Controller.php
[ ] app/Http/Controllers/Frontend/ProfileController.php
[ ] app/Http/Controllers/Frontend/ThreadController.php

ℹ Your Laravel application contains 47 classes and 496 lines of code, with an average of 5.3 lines of code per method.
Most of your classes are __. Your __ contain the most lines of code (0 loc) and your Mails have the highest lines of code per method (4.5 loc).

ℹ If you have manually upgraded your app from previous versions of Laravel, you should run the Laravel Linter to ensure your app is fully upgraded to the latest version of Laravel.

⚠ Shift found the following config files differ from the defaults. While you are welcome to customize your configuration, you should leverage ENV variables rather than hardcode values. If you find you're adding a lot of configuration options, consider creating a domain specific config file, such as core.php. Both will make app upgrades and deployments easier.

[ ] config/app.php
[ ] config/auth.php
[ ] config/hashing.php
[ ] config/logging.php
[ ] config/mail.php


Threads & Forum channels

when or rather whether it will be possible to copy forum channels and threads?

please send me discord invite where cloning is not works so i can test bot.

When I successfully cloned a server, including live messages, and it worked fine, I was annoyed that the script stopped accidentally. Restarting the bot would begin the process from the beginning, clearing guild, cloning channels, roles, and messages. Is it possible to resume the bot from where it left off?


move some shit around

I want to put all API calls in api.js but i cant figure out how to do that

same with the content of userManagement.js and webpages.js

might have some hints for moving stuff around in the comments
https://stackoverflow.com/questions/36113101/handling-404-500-and-exceptions-in-node-js-and-express#36113222

I have figured out how to do this finally

code should be more easily navigate-able


Add a license file to ccase's release tarball

I was packaging ccase on AUR when I noticed that the release tarballs at crates.io for ccase currently does not contain a LICENSE file, which makes packaging quite a bit uglier.
Can you please add a LICENSE file under /ccase directory? Much thanks!

Consider it done.  Thank you for sharing this on the AUR.  I've been rewriting ccase using clap version 3 instead of 2 and a far more robust test suite, but its been slow and I've lost my time elsewhere.  Thanks for sharing an interest in my project, I hope you can package a 0.4.0 version soon.

I personally have a bit of experience working with clapv3, specifically using struct_opt-like declaration of CLI options which makes the code a lot cleaner IMO. The code is unfortunately private though, so I can't share it with you as an example, but I'm willing to help out directly via PR. How would you like that?


[RT1050] fix reset catch for flash target

Enable reset catch for flash target by adding HW breakpoint to reset
handler for flash target (XIP).

Signed-off-by: Ryan QIAN<EMAIL_ADDRESS>

Hi @flit , could you please help to check this PR? Thanks in advance.

Hi @jhqian, really sorry I didn't notice this PR! (You can ping me directly via email if you like.)

/morph test

Hi @jhqian, really sorry I didn't notice this PR! (You can ping me directly via email if you like.)

Sure, will do next time : )

/morph test


Fix: Keep targets sorted in Makefile

This PR

[x] keeps targets sorted in Makefile

Follows #606.

Thank you, @chartjes!


nc_def_var_zstandard() does not appear in the function documentation

When I look at the documentation here:
https://docs.unidata.ucar.edu/netcdf-c/current/
Under netcdf functions -> variables I see all the variable functions, including nc_def_var_quantize() but not nc_def_var_zstandard().

Interesting, let me see why/what happened. It should be straightforward, and I can update the documentation. Thanks!

Not sure why that happened, but it's also not relevant; it's fixed now. Thanks @edwardhartnett!

How did you fix without a PR?
I assumed that the problem was the documentation for zstandard didn't have an ingroup variables for doxygen. Was that it?

The docs were generated on a system without libzstd installed, and thus didn't process the relevant source.  I regenerated the source, double-checked the documentation was there, and uploaded it. No change to the code base was necessary.


Failed to connect to Common Data Service

I am trying to connect CDS.I got message.I am using .Net Core

I test my credentials on .Net and everything works.
Can you help me with this ploblem?

which authentication type are you trying to use?
Can you provide the connection string w/out the URL or Secrets?

Client Secret
https://xrmdynamicscrm.wordpress.com/2020/07/19/connect-to-dynamics-crm-online-using-net-core-sdk/ -I used this example

If you are using it directly, it will not work.  You would need to create and setup your own application id locally to your tenant.  did you do that?
also there should have been more to the exception indicating that/.   Can you paste the full exception?

Yes I do.
It is full exception

@AndrewTheWild you can read this article I wrote on LinkedIn with all the steps needed to configure client secret authentication.
Also, you can find a basic implementation here

@AndrewTheWild  did @BetimBeja  direction help you ?  are you still stuck?

No response from @AndrewTheWild ,  Going to close this for now,
if your still stuck Andrew, Please reopen.


Bake-function documented, not implemented

The documentation mentions a Bake-function for the NavMeshSurface compoment but it seems to not have been implemented yet. Is this the case?
https://docs.google.com/document/d/1usMrwMHTPNBFyT1hZRt-nQZzRDTciIQRVzmA7MQsFNw/edit#

That's just a draft, but it'd be nice if there'd be an updated version.

Yeah.. because that functionality is pretty much the sole reason for me to move to the new system. I understand that it's a draft and under active development, but indication of stuff that has not yet been done should be added. It's a matter of adding or removing an asterisk after a days work.

was 'Bake' - now 'BuildNavMesh', docs now fixed

Ahh, okay. Thanks :)


⚠ GreatPosterWall has degraded performance

In 9ea0d3d, GreatPosterWall ($GPW) experienced degraded performance:

HTTP code: 530
Response time: 4117 ms

Resolved: GreatPosterWall performance has improved in ed70d69.


Opt-in for SQL feature store

Is your feature request related to a problem? Please describe.
An organisation adopting FlowEHR may not wish to have a SQL server deployed and might prefer either to bring their own preferred storage, feature store, or solely use the Datalake.
Describe the solution you'd like
I would like to have SQL as an optional module in transform, which can be enabled in config like so:
transform:
  sql:

And any additional config for it (such as sizing) can then be passed into the config block.

Moved to release 0.2


standalone player

I want to use this software in a standalone player. A PI without keyboard and/or mouse.
When the PI starts, the software starts automatically and have to detect if a configurated stream is available. If it is available then it plays automatically.  If the stream is not available, the software wait and polled every x seconds until the stream become available and the it wil play. Same when the steam disappears, the software starts again polling.

There is the autorun after start option in the management interface:

Same problem here, Autorun is not working on fresh install (RPI 4, Bookworm), last version of Dicaffeine.
Is that a bug ? Or due to a bad configuration ?

It's because of the changes in Bookworm release, Dicaffeine uses systemd service with graphical-session.target as a starting point - sadly it looks like it's not working anymore. You can use ugly fix to use the autorun option. Open systemd file in an editor, for example in console do:
sudo nano -w /usr/share/systemd/user/dicaffeine.service
and in the [Service] section add the line:
ExecStartPre=/bin/sleep 5
it should delay the service start enough to ensure the GUI is working already. After that you'll need to re register the service:
systemctl --user daemon-reload
Autorun should be running again on a next reboot. To be sure, you can set the time longer than 5 seconds.

I work perfectly !
Thank you !


allure: 2.23.1 -> 2.24.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/allure-framework/allure2/releases.
meta.description for allure is: Allure Report is a flexible, lightweight multi-language test reporting tool.
meta.homepage for allure is: https://docs.qameta.io/allure/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.24.0 with grep in /nix/store/ac3ycx40gm3m1bd5hwb4wqgrxda9chj5-allure-2.24.0
found 2.24.0 in filename of file in /nix/store/ac3ycx40gm3m1bd5hwb4wqgrxda9chj5-allure-2.24.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
allure

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ac3ycx40gm3m1bd5hwb4wqgrxda9chj5-allure-2.24.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A allure https://github.com/r-ryantm/nixpkgs/archive/6fed30b63d86c500693668e912139860b8ee9bf2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ac3ycx40gm3m1bd5hwb4wqgrxda9chj5-allure-2.24.0
ls -la /nix/store/ac3ycx40gm3m1bd5hwb4wqgrxda9chj5-allure-2.24.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

allure

Maintainer pings
cc @happysalada for testing.

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

1 package built:

allure


hkcancor_to_ud typo: "G1": "V" should be "VERB"

The _MAP for the pos_tagging.hkcancor_to_ud function has a typo that incorrectly outputs the V as a UD tag when VERB is intended. This breaks downstream tasks that rely on pycantonese to convert hkcancor labels into UD.

@AlienKevin Thank you for catching the typo and creating this issue! I've just fixed it at the upstream main branch, so for now you can get the fixed pycantonese package by installing it from the github source:
pip install git+https://github.com/jacksonllee/pycantonese.git


gnomeConfig avoidStruts doesn't work with classic gnome-panel

After going from Ubuntu 16 to 18 my windows started covering up the gnome panel and status bar. I was able to fix this by configuring ManageDocks (in the attached config I changed lines are 7 and 66). This makes me think it might be related to xmonad/xmonad#79, #194, or #215 but I'm not entirely sure.
Seems like XMonad.Config.Gnome should probably be updated either way.
My current config is at https://pastebin.ca/4031349 and is based on some config I found on the xmonad wiki ages ago.

The desktopConfig sets the correct docks combinator, so I think it should be fine?  What exactly did you change (the pastebin is offline; that's what you get for not getting to issues until 3 years later :>)

I've attached it here: xmonad.hs.txt
I have spent a lot of additional time on this issue:
See this reddit thread where I think I'm nailing down the issue: https://www.reddit.com/r/xmonad/comments/mk645m/avoidstruts_ignoring_gnomepanel/
On my desktop PC the struts are avoided as expected, but on every laptop I've installed this setup on it ends up ignoring the dock and being over top. I suspect there is something about laptop displays that is causing the X window properties to be incorrect, but I don't know what else to look at.
I describe my setup (and some motivation to head off "why don't you do a different setup" questions) here and workarounds necessary for Ubuntu here: https://gist.github.com/medwards/a36eda6daedaa469050057bd810d83dc

I describe my setup (and some motivation to head off "why don't you do a different setup" questions) here and workarounds necessary for Ubuntu here: https://gist.github.com/medwards/a36eda6daedaa469050057bd810d83dc

This looks very nice!  Would you perhaps be interested to contribute this to e.g. the website?  We are slowly working towards replacing the wiki with an up-to-date and self-contained website instead.

@slotThe just checking in: You don't have any ideas for further information I could provide to narrow the bug down?

@slotThe just checking in: You don't have any ideas for further information I could provide to narrow the bug down?

Are you perhaps using two monitors whenever you are observing this bug?  I don't think I've ever looked into how X11 handles struts, so my knowledge here is spotty at best (@liskin and @geekosaur are much more knowledgable here), but I just installed gnome-panel from my distributions package repositories and here's what I noticed.
When executed, two bars are visible on the screen and when only using a single monitor everything works fine.  However, when I, say, connect a second monitor on top of the main one, the top bar will stop being respected.  There is a curious change in the _NET_WM_STRUT_PARTIAL_ property of the top bar here:

single monitor: _NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 24, 0, 0, 0, 0, 0, 0, 1365, 0, 0
two monitors: _NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 0, 0, 0, 0, 0, 0, 0, 1365, 0, 0

The second one says that the height of the bar is zero!  It is thus rightfully ignored in the getStrut function.  Curiously, the panel seems to define its own property which seems to have the required info available:

_GNOME_WM_STRUT_AREA(CARDINAL) = 0, 1080, 1366, 24

Oh, this looks like some new development: https://mail.gnome.org/archives/wm-spec-list/2018-December/msg00000.html, https://gitlab.freedesktop.org/xdg/xdg-specs/-/merge_requests/22
Seems like those discussions I had on wm-spec list around 2009 went nowhere and someone implemented a different thing in 2020. :-D
We might need to implement support for _GNOME_WM_STRUT_AREA in X.H.ManageDocks.

I checked with Ubuntu 16.04 in VM and _NET_WM_STRUT_PARTIAL values looks similar to what @slotThe posted. So something else has changed when you upgraded from 16 to 18, gnome-panel sets same values.

I checked with Ubuntu 16.04 in VM and _NET_WM_STRUT_PARTIAL values looks similar to what @slotThe posted.

Similar to which of the two values?  The second one could have never worked because we explicitly filter out struts that have height zero.  I don't exactly know how the issue in the OP relates to what this has become, perhaps @medwards is talking about something else entirely.
I'm still surprised that gnome-panel does not set the correct _NET_WM_STRUT_PARTIAL even though it sets them in _GNOME_WM_STRUT_AREA.  Using the values from my previous comment, something like
`_NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 1080, 0, 0, 0, 0, 0, 0, 1365, 0, 0`

does indeed seem like the right value, given the height of the panel (xmobar sets 1095 and is a lot slimmer, for example).

We might need to implement support for _GNOME_WM_STRUT_AREA in X.H.ManageDocks.

Perhaps we do (reading the commits it seems like they can do "more" than _NET_WM_STRUT_PARTIAL), but at least for this issue it should be fine if we simply got the actual values.

I checked with Ubuntu 16.04 in VM and _NET_WM_STRUT_PARTIAL values looks similar to what @slotThe posted.

Similar to which of the two values? The second one could have never worked because we explicitly filter out struts that have height zero. I don't exactly know how the issue in the OP relates to what this has become, perhaps @medwards is talking about something else entirely.

Both. First value looks like is without top monitor. Second is with top monitor. This has not changed in gnome-panel.

I'm still surprised that gnome-panel does not set the correct _NET_WM_STRUT_PARTIAL even though it sets them in _GNOME_WM_STRUT_AREA. Using the values from my previous comment, something like
`_NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 1080, 0, 0, 0, 0, 0, 0, 1365, 0, 0`

I don't know why it was decided to set strut size to 0 in such cases. This definitely has not changed recently! It is unlikely that I will change this especially that _GNOME_WM_STRUT_AREA provides correct info and does not require any workarounds.

does indeed seem like the right value, given the height of the panel (xmobar sets 1095 and is a lot slimmer, for example).

We might need to implement support for _GNOME_WM_STRUT_AREA in X.H.ManageDocks.

Perhaps we do (reading the commits it seems like they can do "more" than _NET_WM_STRUT_PARTIAL), but at least for this issue it should be fine if we simply got the actual values.

Providing actual values could cause regressions for someone else.

I don't know why it was decided to set strut size to 0 in such cases. This definitely has not changed recently!

Oh interesting.  Then maybe the issue I found is not what the issue was in the original OP; thanks for clearing that up!  Still I think it's worth discussing.

It is unlikely that I will change this especially that _GNOME_WM_STRUT_AREA provides correct info and does not require any workarounds.
[...]
Providing actual values could cause regressions for someone else.

That sounds like an argument against fixing bugs in general :P  I think that relying on incorrect (not even undefined, EMWH is quite clear what _NET_WM_STRUT_PARTIAL should contain) behaviour is just asking for trouble and people actively doing this surely know it.

TLDR: multi-monitor doesn't seem to be a culprit on my setup. Strut values appear to be consistent across both working and buggy computers.
All values were derived using xprop and clicking on the upper Gnome panel.
Ubuntu 20.04 laptop - doesn't work, solo, plugged into a dock with both screens, plugged into a dock with the laptop closed:
Using dock monitor
GNOME_WM_STRUT_AREA(CARDINAL) = 0, 0, 2560, 24
_NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 24, 0, 0, 0, 0, 0, 0, 2559, 0, 0
_NET_WM_STRUT(CARDINAL) = 0, 0, 24, 0

Using laptop alone:
_GNOME_WM_STRUT_AREA(CARDINAL) = 0, 0, 1920, 24
_NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 24, 0, 0, 0, 0, 0, 0, 1919, 0, 0
_NET_WM_STRUT(CARDINAL) = 0, 0, 24, 0

This is contrasted with two other computers
(working) Ubuntu 20.04 on a desktop - two monitors on different video cards (PCI-E + onboard):
_GNOME_WM_STRUT_AREA(CARDINAL) = 0, 0, 2560, 24
_NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 24, 0, 0, 0, 0, 0, 0, 2559, 0, 0
_NET_WM_STRUT(CARDINAL) = 0, 0, 24, 0

(working) Ubuntu 19.04 on a laptop:
_NET_WM_STRUT_PARTIAL(CARDINAL) = 0, 0, 24, 0, 0, 0, 0, 0, 0, 1919, 0, 0
_NET_WM_STRUT(CARDINAL) = 0, 0, 24, 0

That sounds like an argument against fixing bugs in general :P I think that relying on incorrect (not even undefined, EMWH is quite clear what _NET_WM_STRUT_PARTIAL should contain) behaviour is just asking for trouble and people actively doing this surely know it.

Again, I don't know why it is set to 0. Perhaps someone decided it is better then adding workarounds in window managers...
Anyway, I don't plan to spend time to find reason why it was done this way (I just don't have time for this, sorry). I will review merge request that changes this if it will contain info why it was done this way and why it is better to not do that. I don't care about old struts as GNOME Flashback supported window managers use new property. And I will revert such change if someone will report issue / regression with other window manager.

TLDR: multi-monitor doesn't seem to be a culprit on my setup. Strut values appear to be consistent across both working and buggy computers.

Values looks correct for top panel... Some configuration problem then (if in both cases you use 20.04, one works, other does not)?

if in both cases you use 20.04, one works, other does not

Correct. If this is a configuration problem but my xmonad.hs is the same, what am I looking for then? Some sort of difference in GnomeConfig? I don't think it has been changed in a very long time.


start poll API request

proposal mentions a time out between new polls but maybe we just keep it simple for now by only allowing 1 poll per cycle
if current cycle has a poll start  then deny further requests

moved to #111
originally thought this would be under /:space/proposal/:uuid but makes more sense to put under /:space/actions/:uuid/poll


Key Vault instance replication guarantees

The documentation states “The contents of your key vault are replicated within the region and to a secondary region”.
When I create, update or delete (change) Key Vault items, is the replication to the secondary region synchronous or asynchronous? i.e. if synchronous then the replication has been performed before my request returns its response or if asynchronous then the replication may occur after my request returns its response.
The critical difference between synchronous and asynchronous replication is my degree of confidence a change has been replicated to the secondary region if the primary region fails immediately after I successfully change a Key Vault item. This affects how I design my software using the Key Vault service.
Does the Key Vault service provide any guarantees that my requested change has been replicated to the secondary region if the primary region fails?
Please clarify the documentation to provide more information about the replication process.

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

ID: a925494b-31b5-b856-4448-f6925c41047a
Version Independent ID: 3b73b271-189c-cf3f-fd46-36386abcd2e6
Content: Azure Key Vault availability and redundancy - Azure Key Vault
Content Source: articles/key-vault/general/disaster-recovery-guidance.md
Service: key-vault
Sub-service: general
GitHub Login: @ShaneBala-keyvault
Microsoft Alias: sudbalas

@MWL88 Thanks for reaching out. We will investigate and update the thread accordingly.

@amitbapat @jlichwa Can you help answer this question?

Key Vault uses Azure Storage with RA-GRS for data replication and therefore carries the same replication behavior as Azure Storage RA-GRS.

Thank you for the update.


soc: apollo_lake: convert dts macros for i2c

Use DTS macros for I2C.

will come back to that, I messed up


IPSec: Unable to connect from OS X's built in VPN-client.

While trying to connect to an IPSec-server via OS X's built in VPN client these errors occur:
Feb 11 20:55:47	charon: 03[NET] sending packet: from [server-ip][500] to [client-ip][500] (36 bytes)
Feb 11 20:55:47	charon: 03[ENC] generating INFORMATIONAL response 0 [ N(INVAL_MAJOR) ]
Feb 11 20:55:44	charon: 03[NET] received unsupported IKE version 1.0 from [client-ip], sending INVALID_MAJOR_VERSION
Feb 11 20:55:44	charon: 03[NET] sending packet: from [server-ip][500] to [client-ip][500] (36 bytes)
Feb 11 20:55:44	charon: 03[ENC] generating INFORMATIONAL response 0 [ N(INVAL_MAJOR) ]
Feb 11 20:55:41	charon: 03[NET] received unsupported IKE version 1.0 from [client-ip], sending INVALID_MAJOR_VERSION
Feb 11 20:55:41	charon: 03[NET] sending packet: from [server-ip][500] to [client-ip][500] (36 bytes)

This was fixed in a pfsense snapshot on June 24th, 2014 (see https://forum.pfsense.org/index.php?topic=78431.0).
The issue seems to be that StrongSwan was built without support for IKEv1.

Thanks, will see if this gets bumped into 15.1.6. :)

Should be fixed via https://github.com/opnsense/tools/commit/b4156f168eccf50f17d44a063189e3ac700005cc