Error: up: reading migrations: migration "Create TSM metadata buckets": migration specification not found

I use quay.io/influxdb/influxdb:2.0.0-beta in the previous. Now I upgrade to use quay.io/influxdb/influxdb:2.0.0-rc, but it show me this logs:
2020-10-20T22:45:26.182765Z	info	Welcome to InfluxDB	{"log_id": "0Pz2rl9W000", "version": "2.0.0-rc.1", "commit": "5304b77577", "build_date": "2020-10-14T21:54:15Z"}
,2020-10-20T22:45:26.193817Z	info	Resources opened	{"log_id": "0Pz2rl9W000", "service": "bolt", "path": "/root/.influxdbv2/influxd.bolt"}
,2020-10-20T22:45:26.196551Z	info	Migration "Create TSM metadata buckets" started (up)	{"log_id": "0Pz2rl9W000", "service": "migrations"}
,2020-10-20T22:45:26.204594Z	info	Migration "Create TSM metadata buckets" completed (up)	{"log_id": "0Pz2rl9W000", "service": "migrations"}
,2020-10-20T22:45:26.215412Z	info	Checking InfluxDB metadata for prior version.	{"log_id": "0Pz2rl9W000", "bolt_path": "/root/.influxdbv2/influxd.bolt"}
,2020-10-20T22:45:26.215485Z	error	Missing metadata for bucket.	{"log_id": "0Pz2rl9W000", "bucket": "finance_data", "bucket_id": "e84ad912dbdee4c5"}
,2020-10-20T22:45:26.215540Z	error	Incompatible InfluxDB 2.0 metadata found. File must be moved before influxd will start.	{"log_id": "0Pz2rl9W000", "path": "/root/.influxdbv2/influxd.bolt"}
,2020-10-20T22:45:26.216630Z	error	Found directory that is incompatible with this version of InfluxDB.	{"log_id": "0Pz2rl9W000", "path": "/root/.influxdbv2/engine/_series"}
,2020-10-20T22:45:26.217729Z	error	Found directory that is incompatible with this version of InfluxDB.	{"log_id": "0Pz2rl9W000", "path": "/root/.influxdbv2/engine/index"}
,2020-10-20T22:45:26.217798Z	error	Incompatible InfluxDB 2.0 version found. Move all files outside of engine_path before influxd will start.	{"log_id": "0Pz2rl9W000", "engine_path": "/root/.influxdbv2/engine"}
,

I'm going back to the old version for data migration. While I run (restart) the quay.io/influxdb/influxdb:2.0.0-beta, it show me these logs as below, and I can't launch it again, how can I retrieve my old data?
ts=2020-10-20T23:06:13.687807Z lvl=info msg="Welcome to InfluxDB" log_id=0Pz42uDl000 version=2.0.0-beta.16 commit=50964d732c build_date=2020-08-07T20:18:07Z,
ts=2020-10-20T23:06:13.696520Z lvl=info msg="Resources opened" log_id=0Pz42uDl000 service=bolt path=/root/.influxdbv2/influxd.bolt,
Error: up: reading migrations: migration "Create TSM metadata buckets": migration specification not found,
ts=2020-10-20T23:06:13.698391Z lvl=error msg="Failed to apply migrations" log_id=0Pz42uDl000 error="up: reading migrations: migration \"Create TSM metadata buckets\": migration specification not found",
Usage:,
  influxd [flags],
  influxd [command],
,
Available Commands:,
  generate    Generate time series data sets using TOML schema,
  help        Help about any command,
  inspect     Commands for inspecting on-disk database data,
  restore     Restore data and metadata from a backup,
  run         Start the influxd server (default),
  version     Print the influxd server version,
,
Flags:,
      --assets-path string               override default assets by serving from a specific directory (developer mode),
      --bolt-path string                 path to boltdb database (default "/root/.influxdbv2/influxd.bolt"),
      --e2e-testing                      add /debug/flush endpoint to clear stores; used for end-to-end tests,
      --engine-path string               path to persistent engine files (default "/root/.influxdbv2/engine"),
      --feature-flags stringToString     feature flag overrides (default []),
  -h, --help                             help for influxd,
      --http-bind-address string         bind address for the REST HTTP API (default ":9999"),
      --log-level string                 supported log levels are debug, info, and error (default "info"),
      --no-tasks                         disables the task scheduler,
      --page-fault-rate int              the number of page faults allowed per second in the storage engine,
      --query-concurrency int            the number of queries that are allowed to execute concurrently (default 10),
      --query-initial-memory-bytes int   the initial number of bytes allocated for a query when it is started. If this is unset, then query-memory-bytes will be used,
      --query-max-memory-bytes int       the maximum amount of memory used for queries. If this is unset, then this number is query-concurrency * query-memory-bytes,
      --query-memory-bytes int           maximum number of bytes a query is allowed to use at any given time. This must be greater or equal to query-initial-memory-bytes (default<PHONE_NUMBER>854775807),
      --query-queue-size int             the number of queries that are allowed to be awaiting execution before new queries are rejected (default 10),
      --reporting-disabled               disable sending telemetry data to https://telemetry.influxdata.com every 8 hours,
      --secret-store string              data store for secrets (bolt or vault) (default "bolt"),
      --session-length int               ttl in minutes for newly created sessions (default 60),
      --session-renew-disabled           disables automatically extending session ttl on request,
      --store string                     backing store for REST resources (bolt or memory) (default "bolt"),
      --tls-cert string                  TLS certificate for HTTPs,
      --tls-key string                   TLS key for HTTPs,
      --tls-min-version string           Minimum accepted TLS version (default "1.2"),
      --tls-strict-ciphers               Restrict accept ciphers to: ECDHE_RSA_WITH_AES_256_GCM_SHA384, ECDHE_RSA_WITH_AES_256_CBC_SHA, RSA_WITH_AES_256_GCM_SHA384, RSA_WITH_AES_256_CBC_SHA,
      --tracing-type string              supported tracing types are log, jaeger,
      --vault-addr string                address of the Vault server expressed as a URL and port, for example: https://<IP_ADDRESS>:8200/.,
      --vault-cacert string              path to a PEM-encoded CA certificate file on the local disk. This file is used to verify the Vault server's SSL certificate. This environment variable takes precedence over VAULT_CAPATH.,
      --vault-capath string              path to a directory of PEM-encoded CA certificate files on the local disk. These certificates are used to verify the Vault server's SSL certificate.,
      --vault-client-cert string         path to a PEM-encoded client certificate on the local disk. This file is used for TLS communication with the Vault server.,
      --vault-client-key string          path to an unencrypted, PEM-encoded private key on disk which corresponds to the matching client certificate.,
      --vault-client-timeout duration    timeout variable. The default value is 60s.,
      --vault-max-retries int            maximum number of retries when a 5xx error code is encountered. The default is 2, for three total attempts. Set this to 0 or less to disable retrying.,
      --vault-skip-verify                do not verify Vault's presented certificate before communicating with it. Setting this variable is not recommended and voids Vault's security model.,
      --vault-tls-server-name string     name to use as the SNI host when connecting via TLS.,
      --vault-token string               vault authentication token,
,
Use "influxd [command] --help" for more information about a command.,
,

Solved by this: https://github.com/influxdata/influxdb/issues/19673.
https://docs.influxdata.com/influxdb/v2.0/reference/rc1-upgrade-guide/#5-start-old-influxdb-beta-instance:

If you run into a problem about a missing migration, manually edit your bolt file to remove it. Use BoltBrowser to open and edit your old influxd.bolt file and manually remove the migration.

Caution: Backup your old influxd.bolt file before doing this, as manually editing this file could cause you to lose all your data.

Highlight the record under the migrationsv1 path and press D.


arktos-network-controller introduces arg to make names of resources it creates not fully hardcoded

What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR is to address one point of issue #1168

arktos network controller may need to have extra setting to make the resources (like sa,secret) of system tenant (from multiple TPs) unique across whole cluster

This change is necessary for scale-out env, as some resources (like sa and configmap) used by DNS pods may clash if they are from different TPs, which would make DNS pod fail.
Which issue(s) this PR fixes:
part of issue 1168
Special notes for your reviewer:
unit test code also has minimum change
Does this PR introduce a user-facing change?:
yes
in scale-out env, core DNS names and coreDNS sa/configmap names could have TP-specific suffixes.

/lgtm

/approve


Indicate existence of --traceback option

I had to spelunk the code to find this.

Coverage remained the same at 37.542% when pulling d0eb6aab11cbdbdc0bc831efeaad9b2ba7890b0d on gtd:patch-1 into dbc8f74a3f20ab66cfa782baa668aab17ba9ac51 on django-extensions:master.

Thanks !


bugfix: filling maps from config files

switch to different pointer package: github.com/AlekSi/pointer
measure coverage with debugging on

Codecov Report

Merging #12 (023b4fc) into main (a3ae8f4) will increase coverage by 2.21%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   66.50%   68.71%   +2.21%
==========================================
  Files          11       11
  Lines        1624     1640      +16
==========================================
+ Hits         1080     1127      +47
+ Misses        429      390      -39
- Partials      115      123       +8

Impacted Files
Coverage Δ

flags_command.go
62.07% <ø> (ø)

debug.go
88.23% <100.00%> (ø)

fill.go
72.49% <100.00%> (+8.82%)
:arrow_up:

fillers.go
94.38% <100.00%> (ø)

source.go
45.66% <100.00%> (+4.72%)
:arrow_up:

release.go

flags_import.go
70.05% <0.00%> (+2.13%)
: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 a3ae8f4...023b4fc. Read the comment docs.


[Polygon Drawing] UI linked to ViewModel and Fragments.

Contributes #358

In this PR UI is linked to ViewModel and fragments.

@gino-m @shobhitagarwal1612  PTAL?

@HemanParbhakar Left a few more comments. Please let me know if anything need clarification.

@HemanParbhakar Gentle ping.

@shobhitagarwal1612 the PolygonDrawingViewModel is not updating the visibility of the buttons thats get updated using LiveData.

In the meantime, could you please cleanup the code as mentioned in the code review?

Hi @HemanParbhakar, how do I trigger the polygon drawing feature? It doesn't seem like it's called anywhere yet?

Hello @gino-m for polygon drawing its same as we do for point. Where we select point there we can select polygon.

Hello @shobhitagarwal1612 cleaned code removed the unused functions.

Hello @gino-m for polygon drawing its same as we do for point. Where we select point there we can select polygon.

In the current UI we don't have a dialog to  select "Point", only to select which layer to add a point to. What is the UI you're referring to where we can now select "Polygon"?

Hello @gino-m for polygon drawing its same as we do for point. Where we select point there we can select polygon.

In the current UI we don't have a dialog to select "Point", only to select which layer to add a point to. What is the UI you're referring to where we can now select "Polygon"?
After selecting the layer this dialog appears.

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Wesolowski proofs

This should not be too difficult.  That said, we might want to gate it by a Cargo feature, since it needs parallelism (such as provided by Rayon) to perform well.

wesolowski is implemented, but n-wesolowski isn’t.


[BREAKING] OverlayTutorialWidgetHint is rebuilt too frequently

Great library, it makes highlighting stuff very easy. Thank you a lot.
However, there is a big issue I noticed. The builder for OverlayTutorialWidgetHint is called too frequently. What I mean is I think it is called every frame.
To prove this just add a print statement to the build method.
For example here, add print("IAMREBUILT") to line 156  and you will see that the widget gets rebuilt very frequently. This also has the following problem. I have a skipbutton in my highlights and when pressed I turns every highlight off by switching OverlayTutorialScope to a SizedBox.shrink() but OverlayTutorialWidgetHint throws an exception saying:

`
I/flutter ( 8700): ══╡ EXCEPTION CAUGHT BY SCHEDULER LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter ( 8700): The following assertion was thrown during a scheduler callback:
I/flutter ( 8700): setState() called after dispose(): _OverlayTutorialScopeState#eff5b(lifecycle state: defunct, not
I/flutter ( 8700): mounted)
I/flutter ( 8700): This error happens if you call setState() on a State object for a widget that no longer appears in
I/flutter ( 8700): the widget tree (e.g., whose parent widget no longer includes the widget in its build). This error
I/flutter ( 8700): can occur when code calls setState() from a timer or an animation callback.
I/flutter ( 8700): The preferred solution is to cancel the timer or stop listening to the animation in the dispose()
I/flutter ( 8700): callback. Another solution is to check the "mounted" property of this object before calling
I/flutter ( 8700): setState() to ensure the object is still in the tree.
I/flutter ( 8700): This error might indicate a memory leak if setState() is being called because another object is
I/flutter ( 8700): retaining a reference to this State object after it has been removed from the tree. To avoid memory
I/flutter ( 8700): leaks, consider breaking the reference to this object during dispose().
I/flutter ( 8700): #2      _OverlayTutorialScopeState.updateChildren
package:overlay_tutorial/…/widgets/overlay_tutorial_scope.dart:43
I/flutter ( 8700): #3      _OverlayTutorialHoleState.deactivate.
package:overlay_tutorial/…/widgets/overlay_tutorial_hole.dart:70
`

Is there a way to fix this? I am really enjoying the library and I want to use it but this bug is a breaking bug for me.

This is fixed in 2.0.4.
Reopen this issue if the issue persists.
Cheers!

I tried with 2.0.4 but sadly the issue persists,
Here is a full example, use this widget somewhere and you will see the console spam I_AM_REBUILT. due to line 100.
class TutorialOverlay extends StatefulWidget {
  const TutorialOverlay({Key key}) : super(key: key);

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

class _TutorialOverlayState extends State<TutorialOverlay>
    with SingleTickerProviderStateMixin {
  AnimationController _animationController;
  Animation<Offset> _tweenAnimation;

  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  void initState() {
    super.initState();
    _animationController = AnimationController(
      duration: const Duration(milliseconds: 300),
      vsync: this,
    );
  }

  @override
  Widget build(BuildContext context) {
    final textTheme = Theme.of(context).textTheme;
    final tutorialColor = Colors.yellow;
    return OverlayTutorialScope(
      enabled: true,
      overlayColor: Colors.blueAccent.withOpacity(.6),
      child: SafeArea(
        child: Scaffold(
          appBar: AppBar(),
          body: Center(
            child: Column(
              mainAxisAlignment: MainAxisAlignment.center,
              children: <Widget>[
                const SizedBox(height: 64),
                Text(
                  'You have pushed the button this many times:',
                ),
                OverlayTutorialHole(
                  enabled: true,
                  overlayTutorialEntry: OverlayTutorialRectEntry(
                    padding: const EdgeInsets.symmetric(horizontal: 8.0),
                    radius: const Radius.circular(16.0),
                    overlayTutorialHints: <OverlayTutorialWidgetHint>[
                      OverlayTutorialWidgetHint(
                        position: (rect) => Offset(0, rect.bottom),
                        builder: (context, rect, rRect) {
                          return SizedBox(
                            width: MediaQuery.of(context).size.width,
                            child: Center(
                              child: Padding(
                                padding: const EdgeInsets.all(8.0),
                                child: Text(
                                  'Current Counter will be displayed here',
                                  style: textTheme.bodyText2
                                      .copyWith(color: tutorialColor),
                                ),
                              ),
                            ),
                          );
                        },
                      ),
                    ],
                  ),
                  child: Text(
                    '$_counter',
                    style: Theme.of(context).textTheme.headline4,
                  ),
                ),
              ],
            ),
          ),
          floatingActionButton: OverlayTutorialHole(
            enabled: true,
            overlayTutorialEntry: OverlayTutorialRectEntry(
              padding: const EdgeInsets.all(8.0),
              radius: const Radius.circular(16.0),
              overlayTutorialHints: <OverlayTutorialWidgetHint>[
                OverlayTutorialWidgetHint(
                  builder: (context, rect, rRect) {
                    print("I_AM_REBUILT");
                    return Positioned(
                      top: rRect.top - 24.0,
                      left: rRect.left,
                      child: Text(
                        'Try this out',
                        style:
                            textTheme.bodyText2.copyWith(color: tutorialColor),
                      ),
                    );
                  },
                ),
                OverlayTutorialWidgetHint(
                  position: (rect) => Offset(0, rect.center.dy),
                  builder: (context, rect, rRect) {
                    return SizedBox(
                      width: rRect.left,
                      child: Padding(
                        padding: const EdgeInsets.only(right: 8.0),
                        child: Row(
                          mainAxisAlignment: MainAxisAlignment.end,
                          children: [
                            Flexible(
                              child: Text(
                                'Click here to add counter',
                                style: textTheme.bodyText2
                                    .copyWith(color: tutorialColor),
                              ),
                            ),
                          ],
                        ),
                      ),
                    );
                  },
                ),
              ],
            ),
            child: FloatingActionButton(
              onPressed: _incrementCounter,
              tooltip: 'Increment',
              child: Icon(Icons.add),
            ),
          ), // This trailing comma makes auto-formatting nicer for build methods.
        ),
      ),
    );
  }
}

Hi @ilkercankaya, I am sorry to hear that this issue persists. However, please bear with this, currently, there is no way we can get around with this as far as I know. This is a Flutter issue where we are unable to listen to the position changes of a widget. Hence, I need to rebuild every time to ensure that the hole can be cropped correctly.
Is there any sample code that you can show me which makes this issue a deal-breaker for you? If it is unforgivable, I can try to escalate this issue to Flutter team to seek for help.

Hi @TabooSun, Thank you for the details. What I am trying to do is basically I have a queue of tutorials and I pop from these tutorials and display the corresponding tutorials hint in a OverlayTutorialWidgetHint. I do not pop tutorials inside the builder method but with gesture detectors such as swipe, onTap etc. However, the repeated builder calls sometimes makes my code throw because it somehow rebuilds even If I turn off the hint by switching it to a SizedBox.shrink()  and even if i safe guard it with code like ìf (queue.isNotEmpty)` I get a strange bug where, When I navigate back to the screen where the tutorial overlay lies from another screen the builder draws the same hint several times rather than one. I would want to share my code but its too complicated and due to NDA's I cant give you the exact code.
I do not know if my description was enough for you, I wanted to share.

Hi @ilkercankaya, I have been trying to picture your description. However, it turns out quite hard because there are many possibilities. Without any minimal reproducible code, I am unable to correctly pinpoint the issue.
Is it possible that you somehow provide your code? I am pretty sure that you can rewrite some of the logic in a sample project, like the queue you mentioned.


Kata translation in inconsistent state

I've created a JS version of this kata.
The publishing ended with "Failed Tests" however there had been no failing tests.
The Javascript version is not visible in Kata Editor. When I insert the translation code directly to the Kata editor and run it, all tests pass but the result show that the tests failed.
When I try to publish a fork of the Kata it ends with the same issue.

I did a quick check by forking it. It didn't show any passed tests, which is considered as failure on CW.
What is assert.deepEqual thing by the way? I don't remember having used such an assertion before.

I don't know what's causing the original issue, but here's some information about Node 10 (I'll add to docs later).

What is assert.deepEqual thing by the way?

For versions before Node 10, that's from assertion package shipped with Node. We include it by default.
From Node 10, we don't do that and we recommend using assertion methods from Chai.
Codewars assertions under Test are still usable (just wraps Chai), but not recommended for new tests.

shows only first failed assertion per it

This is because we use Mocha instead of cw-2 starting from Node 10. This is more standard behavior. it is required and that represents a test case. Codewars' custom frameworks are the only ones that relies on assertions for the output.
I'd recommend writing tests like the following:

const fixed = [
  ["abracadabra", [['a', 5], ['b', 2], ['r', 2], ['c', 1], ['d', 1]]],
  ["Code Wars",  [['C', 1], ['o', 1], ['d', 1], ['e', 1], [' ', 1], ['W', 1], ['a', 1], ['r', 1], ['s', 1]]],
];

describe("Fixed Tests", function(){
  for (const [text, expected] of fixed) {
    it('text: ' + JSON.stringify(text), function() {
      assert.deepEqual(orderedCount(text), expected);
    });
  }
});

doesn't support expected/actual tabs

Those tabs are from cw-2 and we no longer use it. We can consider adding it as a new package if the community really need it. I don't know how difficult that can be yet.
If the deep equality output is truncated, you can adjust chai.config.truncateThreshold.

For random tests with long input, using test case name is not ideal. You can log the input in that case inside each it.
    for (let index = 0; index < 100; index++) {
      rs = randomString();
      it(`Random test ${index + 1}`, function(){
         console.log(`<LOG::-Text>${JSON.stringify(rs)}`);
         assert.deepEqual(orderedCount(rs), solution(rs));
      });
    }

By prefixing each with <LOG::-Text>, you get the output in labeled container.

The - prefix makes it collapsed by default.
We should document these better.

You can also use fast-check from Node 10.

@Bubbler-4 thanks for fixing the tests. I approved your fork so the issue is de facto resolved.
@kazk thanks for the feedback and the info about Node 10. I don't know what was the issue with original translations. However, it's weird that the issue only appeared on/after publishing.

It's still not working as expected :/

I'll try editing in the kata editor.

Now it's working, but only Node v10.x is enabled. I'm not sure why it doesn't have Node v8.x

I'll close this for now, but please let me know if this happens again.

@kazk thanks


Open outline with cursor at current element

Currently, when running :Lspsaga outline the outline opens with the cursor at the beginning of the outline, but it would be nice if the cursor will be placed at the current element.
Example:
# 1
## 1.1
## 1.2
<cursor_here>
# 2

and run :Lspsaga outline, the cursor will be at 1.2

Are there any parameters that should be applied in the setup? Because it seems to not work as expected
Screencast from 2023-07-25 16-14-51.webm

update and retry

same as in the video from this message

you must in title line like ## 1.2  . server just parse title head.  marksman does not parse those text .

oh, I see. It works, thanks!

Thanks! Now it works perfectly


Add Elmstatic and sort list of generators

Summary

This commit adds Elmstatic (the static site generator for Elm sites) and sorts the list of generators. It also links to the specific pull requests section of the docs for filing a pull request, since it's at the bottom of the linked page.
Test plan

Only a doc change.
A picture of a cute animal (not mandatory but encouraged)

Sorting doesn't really work since this is a two dimensional list, but happy to add Elmstatic 👍


sync to live

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/windows-iotcore-docs, working directory: W:\mfdj-s, target branch: live, target branch commit: 9e2621a9711b70170d2b3f17c6f39f5d1cf47aa0, source branch: master, source branch commit: .
Learn more: https://help.github.com/articles/resolving-a-merge-conflict-using-the-command-line/

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.


How do I control the arrow length?

a1=>start: test
a2=>parallel: test
a3=>operation: test
a4=>parallel: test
a5=>operation: test
a6=>parallel: test
a7=>operation: test
a8=>operation: test
b1=>operation: testtesttesttesttesttesttesttesttesttesttest
testtesttesttest
b2=>operation: testtesttesttesttesttesttest
test
b3=>end: test
c1=>condition: test
d1=>operation: test

a1->a2(path1,bottom)->a3->a4(path1,bottom)->a5->a6(path1)->a7->a8(right)->b3
a2(path2,right)->b1
a4(path2,right,)->b2
a6(path2,right)->c1
c1(no)->a7
c1(yes,right)->d1

The arrows in the flowchart are always too long to the left or right, probably because the text is too long, but I want to customize the length of the arrows so that the text box on the right of the diagram can be aligned vertically. What should I do? I can't find the corresponding grammar.

How do you align?

you can try with these options: https://github.com/adrai/flowchart.js/blob/master/example/index.html#L33


Pin to apispec 1.0b1

Builds keep breaking as the apispec API is in flux. Pin requirements to first beta and have a link to the latest version of the apispec docs.

Codecov Report

Merging #8 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #8   +/-   ##
=======================================
  Coverage   97.05%   97.05%
=======================================
  Files           3        3
  Lines          34       34
=======================================
  Hits           33       33
  Misses          1        1

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 996232e...8835667. Read the comment docs.


Fixing bug where an empty commit hash is produced when no files have been staged

@SirWindfield how about this change as a fix for #34?
Works on my machine. 🙄

My apologies, I completely forgot to rebase after your merge. I'll fix tomorrow.


Close #1393 Move {readxl} to "Depends"

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the devel branch until you have checked off each task.

[x] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[x] Code is formatted according to the tidyverse style guide
[x] Updated relevant unit tests or have written new unit tests - See Unit Test Guide
[x] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[x] Update to all relevant roxygen headers and examples
[x] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[x] Address any updates needed for vignettes and/or templates
[x] Update NEWS.md if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[x] Build admiral site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the "Reference" page.
[x] Address or fix all lintr warnings and errors - lintr::lint_package()
[x] Run R CMD check locally and address all errors and warnings - devtools::check()
[x] Link the issue in the Development Section on the right hand side.
[x] Address all merge conflicts and resolve appropriately
[x] Pat yourself on the back for a job well done! Much love to your accomplishment!

@cicdguy The R CMD check workflows fail because {readxl} is not available. I moved it from "Imports" to "Depends". Are packages listed in "Depends" not installed in the workflow?

@cicdguy The R CMD check workflows fail because {readxl} is not available. I moved it from "Imports" to "Depends". Are packages listed in "Depends" not installed in the workflow?

"Depends", "Imports", "LinkingTo" and "Suggests" should all be installed. I'll look into this.

@thomas-neitmann I messed around with this and I don't think this is acting how we want it to.  I put readxl back in depends and updated admiral-packages.R with @importFrom readxl read_excel and everything is working again :(


internal/graphicsdriver: use uint32 indices when possible

Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] Xbox
[ ] Web Browsers

What feature would you like to be added?
On modern machines, uint32 is available for indices instead of uint16. Let's use uint32 whenever possible. This is a pure optimization without breaking compatibility.
Why is this needed?
Uint32 would decrease draw calls by more batchings.

/CC @Zyko0

This requires an extension: https://developer.mozilla.org/en-US/docs/Web/API/OES_element_index_uint

As this seems a risky change, let me reset the milestone.

Now the internal uses []uint32. @SolarLune Do you think we need to have a []uint32 version of Draw*Triangles?

Now the internal uses []uint32. @SolarLune Do you think we need to have a []uint32 version of Draw*Triangles?

I would probably just make the transition as quickly and suddenly as possible and just make the DrawTriangles functions take []uint32 instead immediately, rather than have multiple versions of functions and deprecating two of them.
Deprecation is fine, but I feel like it can be confusing when the deprecated approach still exists right alongside the intended replacement for an extended period of time. It's also not as important in a language that takes advantage of modern version control systems (like Go), since you can simply target a version of a framework that has the functions you require at any time - if you upgrade and suddenly your code doesn't work, you can just roll back to a previous version of Ebitengine that had yet to replace deprecated functions.
If you are still in favor of deprecation and the existing DrawTriangles functions aren't hurting performance, then I guess I'd rather you hold off on adding []uint32 versions until some "breaking change version", like v3.0, I guess, without having to deprecate anything.

If you are still in favor of deprecation and the existing DrawTriangles functions aren't hurting performance, then I guess I'd rather you hold off on adding []uint32 versions until some "breaking change version", like v3.0, I guess, without having to deprecate anything.

Yeah, I am in favor of deprecation over adding a breaking change. As []uint32 version is not so strongly on demand, I'll defer this change to v3 (#2829). I think v3 will be released in a few years. Thanks,

if you upgrade and suddenly your code doesn't work, you can just roll back to a previous version of Ebitengine that had yet to replace deprecated functions.

Things would not be so simple. If a thrid party library specifies a new Ebitengine version for its features, a user might not be able to downgread Ebitengine.


Change license to a permissive one

MIT or Apache might be good options to make it permissive.

You're right. We've discussed it internally and we're gonna change the license to MIT. Thank you for the suggestion.


Edge options

You should be able to choose different edge-of-the-world options

Current (what name for this?)
Void
Wrap

Press = for wrapping mode. Press - to go back to normal mode.

merged into #11


String quote example config string values

This will help to avoid the most common misconfiguration mistake: special characters in some of the configuration values.

I wish we didn't have to do this, but YAML is just too easy to mess up.


[dashboard] color-code broken dashboard jobs per card

Each card corresponds to a build job. As of https://github.com/flutter/dashboard_box/commit/4794e9c94282942a1eb7e24d1de46973c0298507 each job can succeed and fail independently of others. We can now color-code breakages on a per-card basis.

Fixed.


[WIP] Cnn classification sols

Worked solutions for the MNIST CNN classification exercise.

Clearly much faster on GPU.
Can you check over the spaces that were left for answers and add anything I've missed? Not sure if there were any other transformations that were useful to include, or what to set the parameters in jitter to.

I had a go with smaller CNNs and it was still quite slow. On my hope PC, the time per epoch using the CPU was only a couple of minutes of so using mobile net, and not much different with smaller models.
I will look over the spaces shortly.

Haven't yet checked on intel, but I found that installing the nightly build allows access to the TPU capabilities using a similar approach to cuda:
import torch
if torch.backends.mps.is_available():
    mps_device = torch.device("mps")
    x = torch.ones(1, device=mps_device)
    print (x)
else:
    print ("MPS device not found.")

https://pytorch.org/blog/introducing-accelerated-pytorch-training-on-mac/
https://developer.apple.com/metal/pytorch/
Might need to add this to the docs (will open issue) and to the code (will open PR).

Just running this on my frame.work with an i5
About 3.5 min per epoch (plus a lot of fan) so seems reasonable.
My guess is that no-one has bothered implementing pytorch properly for apple silicon cpu architecture on the assumption that people will use the tpu backend.
Shocked it's as slow as it is though!
Anyway, happy to sign off on the timing issues as resolved.
@jdenholm did you have any comments on the addition of the mps.is_available()?
Requires torch >= 2.0 I think which I have opened a PR for to the main branch.


store channel memberships associated with primary nickname

This could be a NICKSERV SET option. If it's set, it's recorded anytime you JOIN or PART a channel. When you create a new client associated with the account and primary nickname, you get autojoined.

bogdomania asked about the auth/privs aspect of this. These autojoins should have the same semantics as bare JOIN commands that a client would issue on connect. So if the channel is +k or +I, the join won't succeed unless you have persistent channel privileges or are on the invite list or whatever.

just curious, is this part of the oragono.io/bnc suite of functionality? not sure I understand exactly what this is referring to is all~

More or less, yes. Right now, if you reattach with oragono.io/bnc, you get "autojoined" to the channels your client is already in (we just play you the appropriate JOIN lines). This functionality is similar, but applies to clients who are reconnecting from scratch, not just reattaching.
A similar feature would be remembering your last-used nickname (in the wild, we got annoying split-brain situations where someone would attach two clients, /NICK on one of them, then when reconnecting, reattach would fail because the clients would be trying to get two different nicks).

ahh very interesting. so basically just store a list of 'current channels' on the account and when someone logs in via sasl, try to join all those channels again? definitely make it conditional because it's a fair bit of magic, but sounds cool

Closed in #792


If an incoming RTP packet exceeds receiveMTU we stop handling a Track

Just hit this because I was testing locally with ffmpeg. I am not sure where the error handling is failing.

Our demuxer in internal is failing. Here we construct with receiveMTU https://github.com/pion/webrtc/blob/master/icetransport.go#L155. We would hit the error here. This was very hard to diagnose because we have no error handling or logging.
We could raise the MTU here but I am not sure of a proper value?
I would like to get proper error handling for the demuxer first. We should handle long buffers properly.

This is also only an issue for me on loopback. My MTU everywhere else is <= 1500. I only noticed this because a user was doing ffmpeg -> WebRTC -> WebRTC on the same host.


[ISSUE #998] Overriding Session.java hashCode function

Overriding hashCode function #issue-998

Fixes #998
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Describe the modifications you've done.
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Its throwing error on whitespace indentation after ? on line 288-290 in https://github.com/himansh295/incubator-eventmesh/edit/himansh295/issue-998/eventmesh-runtime/src/main/java/org/apache/eventmesh/runtime/core/protocol/tcp/client/session/Session.java
Why is it happening?

@xwm1992 Can you please tell where to write the test cases to cover the above lines

@xwm1992 Can you please tell me in which class I have to write the test cases for covering the hashCode function?

despite the code test coverage, everything is ok.


Update UI

This makes a few adjustments to the extension UI, mainly:

Move logo to the left
Move advanced settings to a simple icon on the right
Add "loading" status icon to the left side of the settings icon (while it's usually instant, it sets a helpful pattern for when it isn't)
Add "error" status icon when a session isn't found
Move success status message to a simple icon to the left side of the settings icon
Improve UX for initial status (easier to read/find link)

It also includes some other fixes, like:

Don't show the summarize section until there's a valid session
Add rel="noopener noreferrer" to all target="_blank" links

This paves the way to add a new UI/UX for summarizing

Files
kagi_chrome_0.3.zip
kagi_firefox_0.3.zip
Screenshots

Slow loading demo (I had to "fake" a slow loading so it could be visible)

slow loading demo.webm

Love the attention to detail!


Improve the editor role based on first feedback round

Checklist

[x] I tested all changes and all related features with different users (student, tutor, editor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.

Description

Editor can now change the weights of tests. Additionally the grading page button is shown in the overview page of an exercise.
Steps for Testing

Go to the overview of a programming exercise
Click on "Grading"
Change the grading (weight of tests and penalty for SCA)

I assume the refactoring in the ProgrammingSubmissionIntegrationTest was due to code style issues?
Yes, these are just issues detected by IntelliJ - only the wrong order of imports and the compliance to our 180 character limit for each line.


Discern better packaging for Debian and macOS

The current status quo is releasing through pacman and makepkg. Installation of the PKGBUILDs on macOS is done using dkp-pacman, but devkitPro has made it clear that they don’t care to support the operating system with the patches needed for safe behaviour, so we need to settle on an alternative solution.
A few general couses of action:

Move all packaging efforts onto GitHub as ‘releases’,
Support macOS through Homebrew via GitHub,
Support macOS with a Homebrew tap hosted on Bahariya or Siwa,
Support Debian by building a .deb file, or
Support Debian with a PPA hosted on Bahariya or Siwa.

pacman isn't a devkitPro project, so I really don't understand why you're still complaining about our downstream of it.

I’m partial to the hosting avenue, as while it will be more work initially, it will be a more consistent release process and will be more robust for users to get updates (like it is now on Arch). I don’t foresee caring much about other distros or unices, but of course, that could be changed with some evangelising.


Excludes not working with multiple directories

Build does not exclude the directories mentioned in openedge-project.json
  { "type": "source" , "path": "srcNG", "excludes": "srcNG/Updprocs/ARCHIVE/**,srcNG/Updprocs/R200/**", "build": "build"},

Paths in excludes attribute is relative to the source path, so no need to repeat srcNG

Thank you, i will check this later.

Assuming this works now.

yes it does!
Groeten,
Maurice

Van: Gilles Querret @.>
Verzonden: vrijdag 26 april 2024 19:00
Aan: vscode-abl/vscode-abl @.>
CC: Maurice Stassen | WICS @.>; Author @.>
Onderwerp: Re: [vscode-abl/vscode-abl] Excludes not working with multiple directories (Issue #137)
Assuming this works now.
—
Reply to this email directly, view it on GitHubhttps://github.com/vscode-abl/vscode-abl/issues/137#issuecomment-2079757963, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BHG3SEAQ2R3KC5CLFSZZNYDY7KB2FAVCNFSM6AAAAABFW6LTUWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZZG42TOOJWGM.
You are receiving this because you authored the thread.Message ID: @.***>


chore: Add LICENSE

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Advisory, request, response, highligthed response not showing

hello. thanks for this project... i am using burp suite 2020.4 .. it is last version.. i am testing this extension with testphp.vulnweb.com.. the program finds some vulnerabilities, but request, response information is not visible.

ok.. it is working...I can confirm that the program is working properly.


Improve Clicking Affordances

Explanation About What Code Achieves:

This PR improves the communication from the app to user that something is clickable in the following ways:

Suggestion Chips:

Change the cursor from the text selection to default.

Editor Collapsed Sections:

Change the cursor from the text selection to default.
Hide scrollbars that can be accidentally clicked.
Change the background-color on :hover state.

The choice of cursors was driven by Microsoft's, Apple's, and the W3C's Interface Guidelines.
Steps To Test:

Before PR
After PR

nice, thank you!


poetry self add "poetry-dynamic-versioning[plugin]" -> Too many arguments.

m1 CPU
MacOS  Ventura 13.0.1
Python 3.9.16, managed with pyenv
Poetry version 1.1.11

old? but I just installed with with the suggested curl method

updated to poetry 1.3. works.


release: pack requirements for next version

Next QSFS version will contains:

0-db v2.0.0-rc3
0-db-fs v0.1.7
0-stor v0.3.0-rc.1
Configurations

Theses build needs to be staticly compiled since every project attached now support that (previously, we had to ship some shared libraries).
This needs to build on latest V release.

update the versioning @maxux

Updated next QSFS version will contains:

0-db v2.0.0-rc5 (static)
0-db-fs v0.1.7
0-stor v0.3.0-rc.3


Object.preventExtensions/isExtensible documentation

Note this function modifies the object but does not return unit.

Consolidated into https://github.com/rescript-association/rescript-core/pull/117


qpdf 8.0.1: PDF with no content stream triggers "page operation called on non-Page object"

The /Contents key of PDF page is optional and can be omitted to create a blank page. PDF spec 1.7, section <IP_ADDRESS> has this in a table.
For page objects that omit /Contents, qpdf --check says "page operation called on non-Page object" and treats this an error.
Test file attached.
no_contents.pdf

Thanks. Yes, this is a mistake. All the assertion functions in QPDFObjectHandle get me in trouble. I should finish removing the rest of the assertions. I'll have to update the code to make sure a null content stream is treated appropriately. Should be an easy fix. Thanks for reporting.

As far as I can tell, 100% of the places in the code that call assertPageObject work fine even in cases where it would have thrown an exception. I'm removing all calls to assertPageObject(). I had to make a one-line change to make piping contents work on pages with no contents.


feat: Support configure frontmatter classifier scope layout

Summary
fix: #23
What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[x] Docs
[ ] Build-related changes
[ ] Other, please describe:

If changing the UI of default theme, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

[ ] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge
[ ] IE

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[x] Related documents have been updated
[ ] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

Nice.
Just some suggestions.

Done.


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1743 ms

Resolved: Bofrost.at performance has improved in 9411757.


Log for app_change should contain app names

If I change an app, the log looks like this:

But this does not give a hint about the app that a station ran before and which one it is supposed to run now.
I propose to change the message to "App changed from 'App1' to 'App2'".

Fixed in 542e1ccf52e13943a1e1be3973e8e601d9ddf946


Update README.md

flows summarize

magic

Hello, I am a code review bot on flows.network.
It could take a few minutes for me to analyze this PR. Relax, grab a cup of coffee and check back later. Thanks!


Include flattened array test case for allocation of empty array of size N

As a minor enhancement, it would be good to include a test case for flattening an empty array of size N assert.strictEqual(_.flatten([new Array(10)]).length, 10, 'can flatten an empty array of size N');
to make debugging a bit easier when
the test case for assert.strictEqual(_.flatten([new Array(1000000), _.range(56000), 5, 1, 3]).length, 1056003, 'can handle massive arrays') fails

Sounds fine — feel free to send a PR!


NaN values in climate files found

@jarad reported today in our team meeting finding NaN values in the climate files I provided him.  Alas, some do exist

$ find . -type f -exec grep -il nan {} \;
./102x036/102.25x036.25.cli
./102x036/102.75x036.75.cli
./102x036/102.25x036.00.cli
./102x036/102.75x036.25.cli
./102x036/102.00x036.00.cli
./102x036/102.00x036.25.cli
./102x036/102.00x036.50.cli
./102x036/102.75x036.50.cli
./102x036/102.75x036.00.cli
./102x036/102.25x036.50.cli
./100x036/100.25x036.75.cli
...and some more....

ufff, I now see that even some infinity values exist :(  Presently, all found examples limited to April 2019 dates with non-precip data fields.

Reprocessed:

[ ] 24 April 2019
[ ] 17 April 2019

A grep over all files returns clean now.  This must have been some transient error in code that no longer exists today :/

Turns out present day code could still generate these, #96 squashed this bug hopefully.


feat: add npcheck review on Ref arch for Q3 2022

Signed-off-by: Michael Dawson<EMAIL_ADDRESS>

@lholmquist would be great if you could to a review of this PR as well.


fix(overlay): trigger getting shared conversations if feature presented

Description:
When feature shared conversation is not presented by default we are not getting it in list of conversations even if we pass it through overlay options
Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[ ] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


Beacon light client updates tracker

Specs

https://github.com/ethereum/consensus-specs/pull/2916
https://github.com/ethereum/consensus-specs/pull/2932

Lodestar

[x] https://github.com/ChainSafe/lodestar/pull/4478

https://github.com/ChainSafe/lodestar/issues/4705

https://github.com/sigp/lighthouse/issues/3651

light client summit

https://github.com/a16z/helios/issues/59


Field().setField(cells) gives different coordinates as Field(cells)

When creating a new f2c.Field using the setField() method, the coordinates are not correct:
import fields2cover as f2c

ring = f2c.LinearRing()
ring.addPoint(95, 62)
ring.addPoint(15, 2)
ring.addPoint(0, 95)
ring.addPoint(94, 75)
ring.addPoint(95, 62)

cells = f2c.Cells(f2c.Cell(ring))
field = f2c.Field(cells)

f2c.Visualizer.figure()
f2c.Visualizer.plot(field)
f2c.Visualizer.save("field1.png")

field = f2c.Field()
field.setField(cells)

f2c.Visualizer.figure()
f2c.Visualizer.plot(field)
f2c.Visualizer.save("field2.png")

Field 1:

Field 2:

For field 1, all coordinates are negative. If I look at the code, the difference is (I think):
https://github.com/Fields2Cover/Fields2Cover/blob/1d417de070d53cee163829000000414d3c60887c/src/fields2cover/types/Field.cpp#L14-L17
This part is not done when using the .setField() function. I would say that the second field is correct. What is the reason for subtracting all coordinates from the reference point?

Hi @Rick-v-E,

This part is not done when using the .setField() function. I would say that the second field is correct. What is the reason for subtracting all coordinates from the reference point?

To avoid overflow errors when working with UTM coordinate systems, and to debug/visualize easier using smaller units. To receive the correct coordinates, use Field::getCellsAbsPosition()


Simulate a page swipe with different fade out fade in transitions

I would like to simulate a page swipe so that one page fades out left and the following page fading in right, and vise-versa is this possible?

Yes, could you not use .fade-in-left and then .fade-out-right ?


Container App with secret parameter | ERROR: ambiguous option: --secret could match --secrets, --secret-volume-mount

I am facing the issue while deploying the new container app using the command az containerapp create. This command was running perfectly good April 24, 2023 and it started creating problems on May 10, 2023.
Error Message:
ERROR: ambiguous option: --secret could match --secrets, --secret-volume-mount
Command:
az containerapp create  --resource-group <rg-name>  --environment <env-name>  --name <container-name>  --image <docker-image>  --cpu 2.0 --memory 4.0Gi  --registry-server <docker-registry>  --registry-username <docker-username>  --registry-password <docker-password>  --ingress 'external'  --env-vars <envs>  --secret azure-client-id=$AZURE_CLIENT_ID azure-client-secret=$AZURE_CLIENT_SECRET azure-tenant-id=$AZURE_TENANT_ID  --target-port <port-number>  --exposed-port <exposed-port-number>  --max-replicas 3  --min-replicas 6  --scale-rule-name <scale-rule-name>  --scale-rule-type http  --scale-rule-http-concurrency 1
Any help would be appreciated. Thank you.

Hi @khushkore. From your command I believe you are trying to define secrets for your container app. You are just missing a letter 's' in your flag. The correct flag for that is --secrets (documentation: https://learn.microsoft.com/en-us/cli/azure/containerapp?view=azure-cli-latest#az-containerapp-create).
The reason this command worked before is because there were not multiple flags that began with --secret. But we recently added the new flag --secret-volume-mount. Therefore, now you have to be exact with the spelling of the flags or else you will run into the above error.
please let me know if this does not solve your issue.

@smzeng Thank you for your reply. It looks good and resolved my issue. Appreciated!


Always Quiting

hello AxelBjork, first, thanks for your wonderful work.
Im trying to run the bot and got this error.
G:\Games\bot\Rush-Royale-Bot-main>jupyter notebook
[I 13:32:49.893 NotebookApp] Serving notebooks from local directory: G:\Games\bot\Rush-Royale-Bot-main
[I 13:32:49.893 NotebookApp] Jupyter Notebook 6.4.12 is running at:
[I 13:32:49.893 NotebookApp] http://localhost:8888/?token=7f75f856010b71022d2e4187762900667af8a93bb14f2897
[I 13:32:49.894 NotebookApp]  or http://<IP_ADDRESS>:8888/?token=7f75f856010b71022d2e4187762900667af8a93bb14f2897
[I 13:32:49.894 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).
[C 13:32:49.943 NotebookApp]
To access the notebook, open this file in a browser:
    file:///C:/Users/xxx/AppData/jupyter/runtime/nbserver-24488-open.html
Or copy and paste one of these URLs:
    http://localhost:8888/?token=7f75f856010b71022d2e4187762900667af8a93bb14f2897
 or http://<IP_ADDRESS>:8888/?token=7f75f856010b71022d2e4187762900667af8a93bb14f2897

[I 13:33:07.158 NotebookApp] Kernel started: 9da57ad9-e136-4964-afe4-260e60edbf9d, name: python3
[IPKernelApp] ERROR | No such comm target registered: jupyter.widget.control
[IPKernelApp] WARNING | No such comm: 4aa2e676-ae2a-42b2-b5ae-f67b94b00ed2
List of devices attached
emulator-5554   device
error: no such device '\r:5555'
error: no such device '':5555'
scrcpy 1.24 https://github.com/Genymobile/scrcpy
G:\Games\bot\sc\scrcpy-server: 1 file pushed, 0 skipped. 44.2 MB/s (41159 bytes in 0.001s)
[server] INFO: Device: OnePlus ONEPLUS A3010 (Android 7.1.1)
INFO: Renderer: direct3d
INFO: Initial texture: 1080x1920
/system/bin/sh: G:Gamesbotscadb: not found
Events injected: 1
Network stats: elapsed time=6ms (0ms mobile, 0ms wifi, 6ms not connected)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 34.8 MB/s (1490995 bytes in 0.041s)
[I 13:34:01.697 NotebookApp] Saving file at /RR_bot.ipynb
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 31.2 MB/s (1492959 bytes in 0.046s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 43.4 MB/s (785747 bytes in 0.017s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 61.2 MB/s (1497881 bytes in 0.023s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 63.8 MB/s (1497183 bytes in 0.022s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 47.2 MB/s (1495990 bytes in 0.030s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 53.4 MB/s (782818 bytes in 0.014s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 58.9 MB/s (1490902 bytes in 0.024s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 60.0 MB/s (748883 bytes in 0.012s)
[I 13:35:07.114 NotebookApp] Saving file at /RR_bot.ipynb
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 49.8 MB/s (1484904 bytes in 0.028s)
[I 13:35:16.184 NotebookApp] Kernel interrupted: 9da57ad9-e136-4964-afe4-260e60edbf9d
[I 13:37:07.187 NotebookApp] Saving file at /RR_bot.ipynb

Thanks!
The device name being given to Scrcpy is wrong, it should be handled automatically so I'm not sure why that is.
You could always just manually put device = 'emulator-5554' as it will always be that for you. You can also try seeing if just opening Scrcpy normally with the .exe works.
The port scanning code has been changed lately, so check if you have the latest version, this file,
https://github.com/AxelBjork/Rush-Royale-Bot/blob/2e4d59d1db553fe52bcf9c0f553f468811a5a81f/Src/port_scan.py
Otherwise you can try restarting ADB, either by killing the ADB in task manager or "ADB kill-server" in command line.

Hello,
I delete the previous comment because I usually run the notebook from visual code instead of normal Jupyter notebook so not super familiar with the log you posted.
[I 08:12:06.242 NotebookApp] Kernel started: 59e466dc-444d-43b8-b833-84ceb2e7f550, name: python3
[IPKernelApp] ERROR | No such comm target registered: jupyter.widget.control
[IPKernelApp] WARNING | No such comm: f320d224-23df-40a9-9328-4a77f9aee5bc
* daemon not running; starting now at tcp:5037
* daemon started successfully
List of devices attached
emulator-5554   device

error: no such device '\r:5555'
error: no such device '':5555'
scrcpy 1.24 <https://github.com/Genymobile/scrcpy>
C:\Programs\Scrcpy\scrcpy-server: 1 file pushed, 0 skipped. 90.0 MB/s (41159 bytes in 0.000s)
[server] INFO: Device: OnePlus ONEPLUS A3010 (Android 7.1.1)
INFO: Renderer: direct3d
INFO: Initial texture: 896x1600
/system/bin/sh: C:ProgramsScrcpyadb: not found
Events injected: 1
## Network stats: elapsed time=12ms (0ms mobile, 0ms wifi, 12ms not connected)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 64.5 MB/s (1499915 bytes in 0.022s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 63.3 MB/s (1456610 bytes in 0.022s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 78.1 MB/s (1523853 bytes in 0.019s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 67.3 MB/s (1522665 bytes in 0.022s)
/sdcard/bot_feed.png: 1 file pulled, 0 skipped. 74.5 MB/s (1496154 bytes in 0.019s)

This is the log I get when I launch it the same way. Basically looks like the exact same thing, so I don't think any of that is a problem. It seems to be running and pulling images from the device.
Do you get an error in the cell when you run it?

Closed since bot should be run with GUI version now.


[WIP]Add ANNIndex base and milvus db

Signed-off-by: shiyu22<EMAIL_ADDRESS>

Codecov Report

Merging #1159 (28e0ebc) into main (f50fccc) will decrease coverage by 0.18%.
The diff coverage is 38.83%.

@@            Coverage Diff             @@
##             main    #1159      +/-   ##
==========================================
- Coverage   67.86%   67.68%   -0.19%
==========================================
  Files         226      229       +3
  Lines        9984    10084     +100
  Branches     1644     1661      +17
==========================================
+ Hits         6776     6825      +49
- Misses       2783     2831      +48
- Partials      425      428       +3

Impacted Files
Coverage Δ

towhee/utils/milvus_utils.py
0.00% <0.00%> (ø)

towhee/ann_index/milvus.py
27.39% <27.39%> (ø)

towhee/ann_index/base.py
61.90% <61.90%> (ø)

towhee/ann_index/__init__.py
83.33% <83.33%> (ø)

towhee/__init__.py
65.71% <100.00%> (ø)

towhee/dataframe/dataframe.py
73.10% <0.00%> (-1.05%)
:arrow_down:

towhee/engine/operator_runner/runner_base.py
92.16% <0.00%> (+0.60%)
:arrow_up:

towhee/engine/operator_context.py
84.78% <0.00%> (+2.17%)
:arrow_up:

towhee/engine/graph_context.py
90.76% <0.00%> (+4.61%)
:arrow_up:

towhee/engine/task_scheduler.py
71.60% <0.00%> (+11.11%)
: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 f50fccc...28e0ebc. Read the comment docs.

/lgtm
/approve

#1149


ur_modern_driver repo unaccessable

The ur_modern_driver source repo is not available from https://github.com/TheDash/ur_modern_driver.git
As indicated in #11474 by @TheDash
I found a repo with the same name at: https://github.com/ThomasTimm/ur_modern_driver @ThomasTimm is this the same?

I don't know if it is exactly the same, but my repo is the original and what is recommended to use by ros-Industrial until we get it merged.

Thanks I've updated the url in #14565


Develop merge

Add recent changes from Develop

Let's get this merged.  I was just working with Griffin on final touches for the UI merge of develop->master/main.  Planning to do this merge tomorrow.


OpenAI accepts URIs as connection strings

Fixes #2652
Feature is already documented
Microsoft Reviewers: Open in CodeFlow

Unrelated test failure
  Failed Aspire.Dashboard.Tests.TelemetryRepositoryTests.LogTests.Subscription_MultipleUpdates_MinExecuteIntervalApplied [499 ms]
  Error Message:
   Elapsed time 00:00:00.4949117 should be greater than min execute interval 00:00:00.5000000 on read.
Expected: True
Actual:   False
  Stack Trace:
     at Aspire.Dashboard.Tests.TelemetryRepositoryTests.LogTests.Subscription_MultipleUpdates_MinExecuteIntervalApplied() in /_/tests/Aspire.Dashboard.Tests/TelemetryRepositoryTests/LogTests.cs:line 709
--- End of stack trace from previous location ---
  Passed Aspire.Dashboard.Tests.TelemetryRepositoryTests.LogTests.AddLogs_Error_UnviewedCount_WithReadSubscriptionOneApp [< 1 ms]
  Passed Aspire.Dashboard.Tests.TelemetryRepositoryTests.LogTests.AddLogs_MultipleOutOfOrder [1 ms]
  Passed Aspire.Dashboard.Tests.TelemetryRepositoryTests.LogTests.GetLogs_UnknownApplication [< 1 ms]

Related: https://github.com/dotnet/aspire/pull/2703

/azp run


feat: vercel ci

Current dependencies on/for this PR:

main

PR #16   👈

This stack of pull requests is managed by Graphite.


Adding SHA256 information for 0.8.1 release.

Fairly straight forward update of the hash strings.

@ghorejsi Thanks!

Codecov Report

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

@@            Coverage Diff            @@
##             master     #431   +/-   ##
=========================================
  Coverage          ?   61.62%
=========================================
  Files             ?        7
  Lines             ?      357
  Branches          ?        0
=========================================
  Hits              ?      220
  Misses            ?      137
  Partials          ?        0

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 f01a12d...84e8906. Read the comment docs.


Add unit tests

It would be good to add unit tests for the functions in the knime extension script.
Tests to add:

CSVValidator

[ ] Failed validation cases
[ ] No or invalid metadata entrance

CSVNormalizer

[ ] No or invalid metadata entrance
[ ] More than one row in the input
[ ] Input URL is not valid or not linked to a CSV file
[ ] Input CSV file not found in the metadata file

Tests to add:

CSVValidator

[ ] failed validation cases
[ ] no metadata entrance

CSVNormalizer

[ ] More than one row in the input
[ ] Input URL is not valid or not linked to a CSV file
[ ] Input CSV file not found in the metadata file

Check https://docs.python.org/3/library/unittest.html#unittest.TestCase.assertRaises for testing if a function raises an exception with a set of args.


add some more docs for ui_user and ui_slave

Should address https://github.com/ros-infrastructure/buildfarm_deployment/issues/112.

+1 for the content.
Style nitpick: each sentences should start on new line (https://github.com/ros2/ros2/wiki/Developer-Guide#markdown).

I wasn't sure I could do that within a bullet, so didn't try the first time.


Allow for external CNI config

Atm we are using a template, which gets populated with some vars from CLI args.
Lets also implement the usage of an external cni config file. So users can gain more control of the mgmt network options.

closing due to low intereset in containerd runtime at this point


update Contributing for Qiskit 0.9

Needs link to Ignis repo.
Install Terra from source.  Step 4. Pip install . Means install qiskit-terra to the current working directory, so it’s not really installing. Should be python setup.py install.

Umm I'm not sure what you think is wrong with step 4 on the terra install, but pip install . means install the package located in the current working directory. Which at that point in the instructions is the terra repo root. It is essentially the same as calling python setup.py install see my comment in It doesn't mean install to the current working directory.

See my comment here for more details: https://github.com/Qiskit/qiskit/pull/207#discussion_r279474595

Ok yes, this is my mistake in reading the pip docs.  However, I still think it should be replaced with setup.py install as that is the standard way to install Python packages, not via pip.

That was definitely the case pre-python 3.4 (which we don't support anyway) but as things moved forward in the python community (especially after pip is bundled in cpython now) the recommend way to install local packages from source is pip: https://packaging.python.org/tutorials/installing-packages/#installing-from-a-local-src-tree

Fair enough.  Let's stick with pip.


MyAccount/Password change: a few minor fixes

Hi @bradleybeddoes ,
I have received a support request to clarify the wording of the password requirement for 16+ character password (only "MUST not contain username" applies) and while looking into it, I found a few more issues in the code:

AccountController was returning incorrect message code - so users were seeing the message code (controllers.aaf.vhr.account.completedetailschange.password.invalid) instead of the actual message (Your password did not meet the requirements.)
The internal log message logged when the password contained the username was incorrectly saying "The submitted plaintext passwords for $subject don't match, rejecting." (copied from another check just above).

I have already merged this into my master branch and I'm running it on virtualhome.dev.tuakiri.ac.nz (and likely to be pushed soon to Staging and PROD).
Would you be happy to merge this into your code as well?
Cheers,
Vlad

@vladimir-mencl-eresearch happy to merge this.
Please note that I will be archiving this repository on ~11/11/2019. At that time the software will be  EOL at the AAF and decommissioned from our environment. Your own fork would become authoritative for future Tuakiri needs.
cc @ausaccessfed/developers re: EOL.


bug: Does not produce valid SARIF

Describe the bug
The SARIF format files produced are not valid according to the SARIF Validator at https://www.sarif.info/Validation
To Reproduce
Steps to reproduce the behavior:

Install the windows version of tfsec v1.28.1
From the folder containing your terraform, run tfsec-windows-amd64.exe --out tfsec.sarif --format sarif
Load the output file, tfsec.sarif into https://www.sarif.info/Validation
See the errors:

SARIF2005: runs[0].tool.driver: The tool 'defsec' does not provide any of the version-related properties 'version', 'semanticVersion', 'dottedQuadFileVersion'. Providing version information enables the log file consumer to determine whether the file was produced by an up to date version, and to avoid accidentally comparing log files produced by different tool versions.
SARIF1002: runs[0].results[0].locations[0].physicalLocation.artifactLocation.uri: The string 'D:\dev\tftest\mod\tfsec\test.tf' is not a valid URI reference. URIs must conform to RFC 3986.
SARIF1002: runs[0].results[1].locations[0].physicalLocation.artifactLocation.uri: The string 'D:\dev\tftest\mod\tfsec\test.tf' is not a valid URI reference. URIs must conform to RFC 3986.

Expected behavior
The SARIF produced passes all validation rules
Output of your tfsec command with --debug flag
34:58.697695600 system.info                      APP       tfsec-windows-amd64.exe
34:58.698735100 system.info                      VERSION   v1.28.1
34:58.698735100 system.info                      OS        windows
34:58.698735100 system.info                      ARCH      amd64
34:58.698735100 system.info                      KERNEL
34:58.698735100 system.info                      TERM
34:58.698735100 system.info                      SHELL
34:58.698735100 system.info                      GOVERSION go1.18.7
34:58.698735100 system.info                      GOROOT    /opt/hostedtoolcache/go/1.18.7/x64
34:58.698735100 system.info                      CGO       false
34:58.698735100 system.info                      CPUCOUNT  20
34:58.698735100 system.info                      MAXPROCS  20
34:58.698735100 system.info                      WORKDIR   D:\dev\tftest\mod\tfsec
34:58.699251000 system.info                      UID       -1
34:58.699251000 system.info                      EUID      -1
34:58.699251000 system.info                      DOCKER    false
34:58.699251000 system.info                      CI        false
34:58.699251000 system.info                      HOSTNAME  LON-D009
34:58.699251000 system.info                      TEMP      C:\Users\tbutler\AppData\Local\Temp
34:58.699251000 system.info                      PATHSEP   \
34:58.699251000 system.info                      CMD       D:\dev\tftest\mod\bin\tfsec-windows-amd64.exe --out tfsec.sarif --format sarif --debug
34:58.699251000 cmd                              Command args=[]string{}
34:58.699251000 cmd                              Determined path dir=D:\dev\tftest\mod\tfsec
34:58.699766500 cmd                              Determined path root=D:
34:58.699766500 cmd                              Determined path rel=dev\tftest\mod\tfsec
34:58.699766500 terraform.scanner                Scanning [&{D: D:}] at 'dev\tftest\mod\tfsec'...
34:58.702328200 terraform.scanner.rego           Loaded 3 embedded libraries.
34:58.725964800 terraform.scanner.rego           Loaded 119 embedded policies.
34:58.852981700 terraform.scanner                Scanning root module 'dev\tftest\mod\tfsec'...
34:58.852981700 terraform.parser.<root>          Setting project/module root to 'dev\tftest\mod\tfsec'
34:58.852981700 terraform.parser.<root>          Parsing FS from 'dev/tftest/mod/tfsec'
34:58.853982100 terraform.parser.<root>          Parsing 'dev\tftest\mod\tfsec\test.tf'...
34:58.854983200 terraform.parser.<root>          Added file dev\tftest\mod\tfsec\test.tf.
34:58.854983200 terraform.parser.<root>          Evaluating module...
34:58.854983200 terraform.parser.<root>          Read 2 block(s) and 0 ignore(s) for module 'root' (1 file[s])...
34:58.854983200 terraform.parser.<root>          Added 0 variables from tfvars.
34:58.854983200 terraform.parser.<root>          Error loading module metadata: open dev\tftest\mod\tfsec\.terraform\modules\modules.json: invalid argument.
34:58.854983200 terraform.parser.<root>          Working directory for module evaluation is 'D:\dev\tftest\mod\tfsec'
34:58.854983200 terraform.parser.<root>.evaluator Filesystem key is '13f2fce6c4d994f625afc87015910f25ea99570b5399855f2391043ccb849239'
34:58.854983200 terraform.parser.<root>.evaluator Starting module evaluation...
34:58.854983200 terraform.parser.<root>.evaluator Starting submodule evaluation...
34:58.854983200 terraform.parser.<root>.evaluator Finished processing 0 submodule(s).
34:58.854983200 terraform.parser.<root>.evaluator Starting post-submodule evaluation...
34:58.854983200 terraform.parser.<root>.evaluator Module evaluation complete.
34:58.854983200 terraform.parser.<root>          Finished parsing module 'root'.
34:58.854983200 terraform.executor               Adapting modules...
34:58.855981200 terraform.executor               Adapted 1 module(s) into defsec state data.
34:58.855981200 terraform.executor               Using max routines of 19
34:58.855981200 terraform.executor               Applying state modifier functions...
34:58.855981200 terraform.executor               Initialised 392 rule(s).
34:58.855981200 terraform.executor               Created pool with 19 worker(s) to apply rules.
34:58.855981200 terraform.scanner.rego           Scanning 1 inputs...
34:58.857981200 terraform.executor               Finished applying rules.
34:58.857981200 terraform.executor               Applying ignores...
34:58.857981200 cmd                              Exit code based on results: 1

System Info

tfsec version: v1.28.1
terraform version: 1.3.7
OS: Windows 10 21H2

Example Code
terraform {
  required_version = ">= 0.15.4"

  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "~>3.37.0"
    }
  }
}

 resource "azurerm_storage_account" "bad_example" {
   name                      = "storageaccountname"
   resource_group_name       = azurerm_resource_group.example.name
   location                  = azurerm_resource_group.example.location
   account_tier              = "Standard"
   account_replication_type  = "GRS"
   enable_https_traffic_only = false
 }

Additional context
I need to produce the reports in SARIF format to import into SonarQube.  It fails with:
WARN: Failed to import an issue raised by tool defsec, error: Illegal character in opaque part at index 2: D:\dev\tftest\mod\tfsec\test.tf
SARIF2005 issue may not necessarily be a problem because version info is optional so I suspect the only "problem" as such is the SARIF1002 issue - the locations are not valid URIs.  Instead of "D:\\dev\\tftest\\mod\\tfsec\\test.tf the value should presumably be something like file:///D:/dev/tftest/mod/tfsec/test.tf?
Sample output attached from my run.
tfsec.sarif.txt

This is still an issue - can you remove the stale label please?

Hi - tflint had exactly the same problem and they have fixed it so perhaps you can leverage their work here: https://github.com/terraform-linters/tflint/pull/1686 ?

Our efforts have moved to Trivy. Please file a discussion in the Trivy repo if the issue still persists.


[BUG] RuntimeError:          An attempt has been made to start a new process before the         current process has finished its bootstrapping phase.          This probably means that you are not using fork to start your         child processes and you have forgotten to use the proper idiom         in the main module:              if name == 'main':                 freeze_support()                 ...          The "freeze_support()" line can be omitted if the program         is not going to be frozen to produce an executable. /Users/yanghualei/miniconda3/envs/market-analy-39/lib/python3.9/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 69 leaked semaphore objects to clean up at shutdown   warnings.warn('resource_tracker: There appear to be %d '

Expected Behavior
chart = Chart() 出现error
Current Behaviour
chart = Chart() 出现error
Reproducible Example
chart = Chart() 出现error

Environment
- OS:
- Library:

RuntimeError:          An attempt has been made to start a new process before the         current process has finished its bootstrapping phase.          This probably means that you are not using fork to start your         child processes and you have forgotten to use the proper idiom         in the main module:              if name == 'main':                 freeze_support()                 ...          The "freeze_support()" line can be omitted if the program         is not going to be frozen to produce an executable. /Users/yanghualei/miniconda3/envs/market-analy-39/lib/python3.9/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 69 leaked semaphore objects to clean up at shutdown   warnings.warn('resource_tracker: There appear to be %d '

Just for clarity for others who may experience the same issue:
You must encapsulate the Chart object within an if __name__ == '__main__' block.
Louis


Is there a way to get the face rectangle?

I'm wondering is there an easy way to get a face rectangle similarly to other face detectors?

This is done first hand during the face detection. See getBoundingBox to get an idea.
If you want to just have bounding boxes around a face, jsfeat might already solve your problem.


Order-independent fake signatures

Make fake signatures produce the same results independent of order of the signers (assumes signatures are unique).

Codecov Report
Attention: Patch coverage is 80.00000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 71.58%. Comparing base (db09264) to head (1a1a304).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #265      +/-   ##
==========================================
- Coverage   71.63%   71.58%   -0.05%
==========================================
  Files          31       31
  Lines        2337     2344       +7
==========================================
+ Hits         1674     1678       +4
- Misses        533      534       +1
- Partials      130      132       +2

Files
Coverage Δ

sim/signing/fake.go
86.41% <80.00%> (-2.77%)
:arrow_down:

The BLS scheme we use is order dependant, which is why the fake signature scheme replicates it.

The BLS scheme we use is order dependant, which is why the fake signature scheme replicates it.

I see. And we expect to sort by power descending, ID ascending (same as the power table)?

Whatever is easier. We settled on power descending because we store signers as bitfields of indexes in the power table.

Yeah, fair enough.


change yaml tutorial link to learnxinyminutes

Why:

mainly because codeproject's yaml tutorial page is littered with <br /> in examples and it's imho confusing especially for beginners (who are target audience)
https://learnxinyminutes.com/docs/yaml/ is imho easier to read
codeproject uses https://en.wikipedia.org/wiki/Code_Project_Open_License which needs author's consent to use it

What's being changed:
link to external page
Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

@michalbcz Thanks so much for opening a PR! I'll get this triaged for review 💖

On Mar 25, 2021 2:37 PM, Sarah Edwards @.***> wrote:
Merged #4695 into main.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.


pin pybind11 down to patch version at build time

pybind used for code generation must match exactly the version used at build time to avoid issues like:
ImportError: generic_type: type "Conductivity" referenced unknown base type "dolfin::Expression"

It's unclear how true this is in general, but we at least know that when dolfin is built with 2.2.3, code generation fails with 2.2.4.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Fix application of facade to batch events

Currently, for events invoked through a batch integration, we return the nested event object, which basically removes the facade. This change fixes that code so the facade is properly sent to batch integration methods.

This counts as a breaking change, I'm a bit unfamiliar w/ version changes. Can I just increment the version to 6.2.0 in a commit similar to this: https://github.com/segmentio/integration/commit/72fe2c675a92a1482c81706b37ba532b7538d057
And that should allow me to bump the version of the dependency in other projects?

This counts as a breaking change, I'm a bit unfamiliar w/ version changes. Can I just increment the version to 6.2.0 in a commit similar to this: 72fe2c6
And that should allow me to bump the version of the dependency in other projects?

Yep go ahead and bump version in package.json to 7.0.0 since this is a breaking change.
Update HISTORY.md as well.
merge the PR
And then npm publish.


updating if a timedelta is subtracted

if a timedelta is subtracted from MTime an error is thrown, this fixes that error.

[x] Need to return MTime if the subtracted object is a timedelta
[x] Need to return total seconds if the subtracted object is another MTime
[ ] Update tests

Codecov Report
Patch coverage: 80.00% and project coverage change: +0.04 :tada:

Comparison is base (7f9b110) 80.32% compared to head (b7756f0) 80.37%.

: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
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #131      +/-   ##
==========================================
+ Coverage   80.32%   80.37%   +0.04%
==========================================
  Files         221      221
  Lines       15033    15056      +23
==========================================
+ Hits        12076    12102      +26
+ Misses       2957     2954       -3

Impacted Files
Coverage Δ

mt_metadata/utils/mttime.py
81.64% <70.58%> (+2.99%)
:arrow_up:

tests/test_mttime.py
98.34% <100.00%> (+0.07%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Added logic to if a float or int is input and testing whether units are seconds or nanoseconds.


Independent routing for multiple pages

Is there some solution for routing multiple pages at the same time? I'm talking about something similar to this question.

Thanks for your question!
We want to make sure that the GitHub issue tracker remains the best place to track bug reports and feature requests that affect the development of React Router.
Questions like yours deserve a purpose-built Q&A forum. Could you post this question to Stack Overflow with the tag #react-router? https://stackoverflow.com/questions/ask?tags=react-router
We also have an active and helpful React Router community on Reactiflux, which is a great place to get fast help with React Router and with the rest of the React ecosystem. You can join at https://discord.gg/0ZcbPKXt5bYaNQ46.


Solver codegen generates incorrect csc matrix for linsys_solver_L

On the master branch the workspace.c file generated seems to set the linsys_solver_L nzmax variable to a random value at each codegen. My guess would be uninitialized memory  usage but I haven't tracked down where it's coming from yet.
clc; clear all; close all;

% Load P, q, A, l, u.
P = zeros(500, 500);
A = zeros(500, 500);
q = zeros(1, 500);
l = zeros(500, 1);
u = zeros(500, 1);

problem = osqp;
settings = problem.default_settings();
settings.verbose = 1;
problem.setup(P, q, A, l, u, settings);

% Not using LONG means that the compiler should warn you that you're overflowing an integer
% if you get a very big random value
problem.codegen('osqp_solver', 'force_rewrite', true, 'FLOAT', true, 'LONG', false);

problem.solve()

When I do this I get nzmax initialised to -1, but this may be a difference of versions.   Could you please post the version number that you are using?
Note that since your problem data is entirely zeros, you should expect that the LDL factorisation of the KKT matrix to result in an empty L matrix.   The D matrix will have terms determined by the values of sigma and rho.     I'm not sure if nzmax = -1 for L is correct either in this case btw, so that will require some follow up.

I'm using fa7362e9fbd2140e89a2979d7a1bf91c3c07b4f8 (current master) and if I revert to tag v0.6.0 it still occurs.
I chose zeros just to make it easy to create.
// The line that is generated in workspace.c
csc linsys_solver_L = {30681249209647152, 740, 740, linsys_solver_L_p, linsys_solver_L_i, linsys_solver_L_x, -1};

@goulart-paul the only -1 I get is for the nz field of the csc struct. The nzmax field (the first one) does appear to be uninitialized in the solver. It looks like it should be set inside LDL_factor (qdldl_interface.c:71) when the actual matrix is allocated, but it isn't. It looks like the best thing to do is to just set the nzmax field to be the number of actual non-zeros in the L matrix. I can put a PR together for this if you agree.

You are correct and I agree.   I wonder if that value is ever actually used anywhere in our code, since we don't ever reallocate arrays within a csc formatted matrix.    I'm not sure what it's for.

I think this is fixed by Ian's PR.   Please let us know if it continues to be an issue.

Looks like it's fixed! I'll leave it up to you whether you want to close this now or wait for the submodule to be updated in this repository


Husky pre-commit should only affect staged files

Husky pre-commit should only affect staged files. Currently, runs on all changed files even if not yet being committed.

Fixed in https://github.com/OultimoCoder/cloudflare-planetscale-hono-boilerplate/releases/tag/v1.2.1


Lost all of Plugins [Android]  After  running "Flutter clean"

Hello everyone, I got a problem that it's very annoying to me. So, when I run "Flutter Clean" I lost all of the plugins in Android which path is  "android/app/src/main/java/io/flutter/plugins/GeneratedPluginRegistrant.java" and when I run "pub get", it's never got all of these anymore. I try flutter pub get and then flutter pub run many times and then I got the problem like this all the time.
this is one of my plugins. and the problem happened to all of the plugins in my projects as follows.
error: package io.flutter.plugins.googlemaps does not exist
flutterEngine.getPlugins().add(new io.flutter.plugins.googlemaps.GoogleMapsPlugin());
error: package io.flutter.plugins.webviewflutter does not exist
flutterEngine.getPlugins().add(new io.flutter.plugins.webviewflutter.WebViewFlutterPlugin());

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.2, on Mac OS X 10.15.7 19H1217 darwin-x64, locale en-TH)
    • Flutter version 2.2.2 at /Users/armotesomboonkaew/Documents/flutter
    • Framework revision d79295af24 (2 weeks ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/armotesomboonkaew/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] Connected device (2 available)
    • SM G925F (mobile) • 1215fca2b83e0704 • android-arm64  • Android 7.0 (API 24)
    • Chrome (web)      • chrome           • web-javascript • Google Chrome 91.0.4472.114

Thank you in advance. I wanna cry for this. Please somebody help me.

HI @izzabellerina
Please provide the details from above
Thank you then

@TahaTesser  @iapicca  Thank you for looking after my problem, So its  "pub get" works fine, and then I got a new android folder after that I add the same config as the previous android like permission, etc., finally. I got the same problem that is Missing Plugins. I try to run pub get and the result was the same-Missing plugin. as follow
Unhandled Exception: MissingPluginException(No implementation found for method isLocationServiceEnabled on channel flutter.baseflow.com/geolocator)
Thank you in advance. And so sorry for late reply

@izzabellerina
I recommend to close the issue as the original reported error is solved,
also you may want to open your new issue with geolocator in the the dedicated repo

@iapicca The MissingPluginException It's not just Geolocator, it's all of my plugins. Seem like the following errors
" error: package io.flutter.plugins.googlemaps does not exist
flutterEngine.getPlugins().add(new io.flutter.plugins.googlemaps.GoogleMapsPlugin()); "

hi @izzabellerina
Can you please provide complete following the steps?
flutter create mypp
cd myaop
add your packages
flutter clean
flutter pub get -v (complete logs for this please )
Thank you

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Convert async --> non_blocking for Python 3.7

Fixes #8.  async is a reserved word in Python 3.7 so convert all instances of async into instances of non_blocking.  Also document that the global variable c is created by Jupyter.

Hm, I was actually thinking of renaming it to async_ and later adding run_async....run_non_blocking sounds a bit hard to remember

Sure.  pytorch and cuda went with non_blocking https://github.com/pytorch/pytorch/pull/4999/files but either way works.  The other way to go would be to flip the boolean and go with sync so that you could go with run() (defaults to asynchronous execution) and run_sync() (defaults to synchronous execution).

I slept on it, and couldn't see a huge difference between approaches, thanks for the PR


Filters are in milliseconds but indicates seconds

Request
Took longer than  seconds  seconds, executed more than  n
queries, and spent longer than seconds  seconds executing queries.

Hello @avelis @SebCorbin ,
I'm new to OSS and would love to contribute to this.
From what I can gather I need to change the text seconds to milliseconds in profile.html and requests.html. Does that work?

@ka28kumar yes I believe that is all that needs to change to meet the acceptance criteria of this ticket.


Admin SPARQL User Query Table

Create a workflow in which the admin can view all users in fueski in a table, edit, and push back to fueski database.  A potential command-line interface will be integrated based on this idea as well.

Some Admin Dashboard Changes:

[ ] UI Progress Bar Fix
[ ] Small UI Updates


Implement DATE_TRUNC by day

hi @goccy, I've implemented the DATE_TRUNC by day case.
Thanks

Thank you for the great PR !! LGTM


Fix data type for pr_files field in PR description prompts

PR Type
bug_fix

Description

Fixed the data type of the pr_files field in pr_description_prompts.toml from [List[FileDescription]] to List[FileDescription]

Changes walkthrough
Relevant filesBug fix

pr_description_prompts.tomlCorrected data type for pr_files field in PR description prompt settings

pr_agent/settings/pr_description_prompts.toml
Fixed the data type of the pr_files field from [List[FileDescription]] to List[FileDescription]

+1/-1

👑 🌖 I've updated the PR description to match the latest commit (https://github.com/Codium-ai/pr-agent/commit/174b4b76eb993a47409e5a8c237c139cf82a9124).

PR Analysis 🌗

⏰  Estimated effort to review: 2, as the changes are straightforward and localized.
🥇  Score: 95, because the PR addresses a specific bug fix and implements a clear data type correction.
🧪  No relevant tests
🔒  No security concerns identified
💡  Key issues to review
Data Type CorrectionThe data type for pr_files field was corrected from [List[FileDescription]] to List[FileDescription].
Code CleanupUnnecessary brackets removed from the pr_files field declaration.

LGTM


[Documentation] Local Dev Server Broken

The dev server for documentation does not work on a Windows 7 machine. I have not test on Linux/Mac.
I have the latest from master, and I have followed the directions in the docs readme and the dev server doesn't work correctly. I believe the issue stems from 2 configurations in the webpack-dev-server.config.js file.

Currently the devServer config is set to this:

devServer: {
    contentBase: '',
    devtool: 'eval',
    hot: true,
    inline: true,
    port: 3000
}

I believe the contentBase needs to be updated to 'src/www', so it looks like this:
devServer: {
    contentBase: 'src/www',
    devtool: 'eval',
    hot: true,
    inline: true,
    port: 3000
}

The TransferWebpackPlugin does not work with webpack-dev-server, see current open issue here: https://github.com/molforp/transfer-webpack-plugin/issues/6

To clarify, the npm run build does work correctly (...even though that command is not documented in the readme...), but that doesn't really help for improving documentation, as I would have to rebuild every time a change is made. It would be much preferable if the hot loading dev server would work correctly on Docs.
Docs are probably one of the biggest sore spots for this project, and if we can get this dev server working correctly and quickly, people can start actually contributing to making them better.

@NogsMPLS I couldn't agree more. We definitely need to put more effort on the Windows side.
I think I have seen somebody else also mention the code fragment that you wrote above. I can confirm that changing contentBase to src/www works in MacOS.
Do you wanna submit a PR with the fix? :)

Sure, I think I might have an idea how to make it all work smoothly for both dev and prd...Give me a few hours.

:+1: @NogsMPLS

Submitted https://github.com/callemall/material-ui/pull/1774 to hopefully close this


change to using MacOS 11 (Big Sur)

We use a M1 box to build sac2c, and so we require >= XCode 12.5.
DO NOT MERGE: macos-11 support in GitHub is still not generally available, but I've sent in a request to have access to this. Only after this is approved can we merge the change.
For info from Github, see https://github.com/actions/virtual-environments/blob/main/docs/macos-11-onboarding.md.

An alternative would be to downgrade the sac2c macos package, but this could lead to other problems... hmm not sure.

Access to the macos-11 seems to be granted!


fix: pagination: ensure jumper applies current page

SUMMARY:
Fixes the page to use the page set by the event handler
JIRA TASK (Eightfold Employees Only):
ENG-13961
CHANGE TYPE:

[X] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[X] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:
Use the jumper and ensure the page updates

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@4cde1d5). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #182   +/-   ##
=======================================
  Coverage        ?   76.53%
=======================================
  Files           ?       71
  Lines           ?     1756
  Branches        ?      683
=======================================
  Hits            ?     1344
  Misses          ?      401
  Partials        ?       11

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 4cde1d5...3538e3b. Read the comment docs.


Remove usages of getrawtransaction

This PR removes all usages of getrawtransaction in the backend in a really straight-forward way, and creates only 2 small issues:

In case of double spends, the state of the in-memory mempool will be incorrect for maximum 21s as the double-spent transaction won't be removed until node's mempool is re-cloned.
This PR disables unconfirmed-transactions-chain, but old clients (i.e. clients prior merge of #13193) will still try to access it. This will cause the old clients to have no details about the confirmation ETA for transactions such as coinjoins (like before @adamPetho's work), and these clients will re-try the request 3 times in a row, leading of 3 log warning asking to update. This behavior can be changed if we want by either:

Returning a Ok(dummy value) instead of BadRequest
Keep providing a correct response by requesting the data to a third party such as mempool.space (i.e. implement #13193 on server side), for which only problem would be rate limitation

I merged this but #13193 should be merged and released before it is deployed


Add support for CCM FIR packets.

This adds support for CCM FIR packets, as defined in RFC 5104 Section 3.5.1.

Amazing! Fantastic work @jech
Would you be interested in being added to the Pion org? You will be able to merge your own PRs after 1 review, and you can review other peoples code :)
No real obligation, but I am always trying to drag people better then me into getting involved :)