Add the "minmax" kernel

What does this implement/fix? Explain your changes
This PR provides a memory-efficient implementation for the minmax kernel from my personal BO repo.
It uses a formulation from Ioffe (2010) that allows torch.cdist to be used instead of directly computing mins/maxes.
What testing did you do to verify the changes in this PR?
None: unfortunately I could not get the environment to build properly on my machine so I could not run the tests. Maybe you can run them if you have a pre-built environment which works? I think the environment file in the main repository is out of date...
Pull Request Checklist

[ ] Added a note about the modification or contribution to the ./CHANGELOG.md file (if applicable)
[ ] Added appropriate unit test functions in the ./gauche/tests/* directories (if applicable)
[ ] Modify documentation in the corresponding Jupyter Notebook under ./notebooks/ (if applicable)
[ ] Ran python -m py.test tests/ and make sure that all unit tests pass (for small modifications, it might be sufficient to only run the specific test file, e.g., python -m py.test tests/kernels/test_graph_kernels.py)
[ ] Checked for style issues by running black . and isort .

Thanks very much for adding this Austin! Will run the tests on my setup!

Apologies, I just revamped the fingerprint kernel classes in line with recent changes in GPyTorch. Merged over your MinMax implementation in a separate PR. Will add a note to the file that you authored the kernel in a bit!

@Ryan-Rhys  can you link me to the PR where this was merged? Happy to close this PR if the functionality has been added.


Generics

Is it possible for Swashbuckle to generate documentation for action methods with generics.  I am using version 2.4.0 on AspNetCore 2.0.7
Action Method
    [HttpPost("/policy/{id}/document/cancel")]
    [Produces("application/json")]
    [SwaggerResponse((int)HttpStatusCode.OK, typeof(Document), "The cancellation document was generated.")]
    [SwaggerResponse(422, null, "The request was invalid.")]
    public override  Cancel<Cancellation>([FromBody] Cancellation productModel)
    {
        return this.View("Views.Policy.Cancellation", productModel);
    }

And the API documentation produces
<member name="M:BarbonCoreFoundationMicroservice.Innovate.Controllers.V1.InnovatePolicyDocumentController.Cancel``1(``0)">
            <summary>
            Cancel the specified policy/service.
            </summary>
            <returns>The cancel.</returns>
            <param name="productModel">Product model.</param>
            <typeparam name="Cancellation">The type of model to expect</typeparam>
        </member>

What do I need to do to get the action method showing up in Swagger?

Sorry, closed by mistake.

Solved.


Dynamic size output

Hi, I noticed in one of the examples, the dynamic size input/output was fixed. Does so.add_output currently support dynamic size dimensions?

read the output and read protobuf code, figured out myself.


Dart Dev: 3.0.0-30.dev - 3.0.0-51.dev

Hi @worawit ,
Dart in it's dev version from 3.0.0-30.dev to 3.0.0-51.dev had used field_names() instead of GetFieldNames(thread).
I modified blutter/src/DartTypes which works, but I can't think of a way to make it such it works as a detection if GetFieldNames isn't available it should try using field_names, unless we add an env var through python and check for dart version and apply conditionals, but that'll change structure of blutter a little so I haven't worked on that.
If it is possible without those please try to implement it. ❤
Thanks

I think it is not worth to make it work for these versions. Very few application development use these dev versions. Even someone use it, I believe they will upgrade their flutter in a new application version.

Got it, thanks


List command is slow for many entries

The list command takes a noticeably long time to list everyone's results beyond 7 persons with data, probably something is inefficient

same with deleting many entries

mentioned in planned enhancements


Add note about SQLite3 connection

Should this be an OTEL issue?

Yes. I'm going to create it there as well.


--pipeline simple is now a required parameter to successfully complete quantized model training on M1/2/3 Macbook with v0.19 & v0.19.1

With v0.19, I started receiving this error when running lab -v -v model train:
DEBUG 2024-10-03 09:05:14,819 instructlab.model.train:502: train_file=/Users/waughus.ibm.com/.local/share/instructlab/datasets/train_merlinite-7b-lab-Q4_K_M_2024-10-01T14_00_43.jsonl
DEBUG 2024-10-03 09:05:14,819 instructlab.model.train:503: test_file=/Users/waughus.ibm.com/.local/share/instructlab/datasets/test_merlinite-7b-lab-Q4_K_M_2024-10-01T14_00_43.jsonl
Usage: ilab model train [OPTIONS]
Try 'ilab model train --help' for help.

Error: Data path must be to a valid .jsonl file. Value given: /Users/waughus.ibm.com/.local/share/instructlab/datasets

If I try to specify the data path using ilab -v -v model train --data-path /Users/waughus.ibm.com/.local/share/instructlab/datasets/train_merlinite-7b-lab-Q4_K_M_2024-10-01T14_00_43.jsonl, the command runs and seems to complete but never initiates the training.
I was able to successfully run with the following command ilab -v -v model train --pipeline simple with v0.19.1
With v0.17.x, the default worked on my MacBook and the --pipeline simple parameter was not required.
Per BJ Hargrave, the default seems to be --pipeline accelerated which gives the above error.

opened against the wrong repo
created https://github.com/instructlab/instructlab/issues/2380


Add node/analysis reference to widgets

@urbanphes commented on Thu Sep 29 2016
Now in widgets we don't have any reference to know which is the layer or step.

And when we add a widget on the dropdown selected this kind of parameters don't appears when add the widget in a right bar.

The idea is to have this parameters on the widgets sidebar.

@arianaescobar commented on Thu Sep 29 2016
Reference to the node (Builder view)

Reference to an analysis (Builder view)

No reference (Embed)

@saleiva commented on Thu Sep 29 2016
Close, promote, and assign to @piensaenpixel :)

Prioritizing this, having lots of feedback. We should get this for the first bathc.

CARTO.js and DI.js don't know about layer colors cc @alonsogarciapablo. But we could do it from now on and add the info, what do you think @alonsogarciapablo @donflopez?

Yes... widgets could have a source_color attr or something along those lines...

Widgets or layers? I would not pass the color to the widgets due to it is not related to them.

Yes, you're right @xavijam. It makes more sense to associate colors with the analysis node... The only problem is that we don't have analysis nodes in DI.js/CARTO.js....

Nodes don't know anything about color either, we should "propagate" that to the layer-model, not to any other thing, IMHO.

@xavijam when we'll have the color thing ready please assign it to me :)

Deployed thanks to @matallo


Estate complexity summary

(sqrt(connections) / app) + ( sqrt(servers) / app ) + ( #capabilites / app )

Closed as part of #12


Indicated size is invalid. when running parallel instances of dart pub run

This tracker is for issues related to:

Dart VM

description
I have a CI task that is using Dart to do code generation.  After I parallelized execution of the instances of "pub run" I noticed a flake where sometimes the execution would fail to load the kernel binary.  In this instances pub run is not caching the build because I'm using it to execute local code, so there appears to be a race condition in reading artifacts from the disk.
error
Can't load Kernel binary: Invalid kernel binary: Indicated size is invalid.

example
https://cirrus-ci.com/task/6651964250390528?logs=local_tests#L408
steps to reproduce
Here is the roughly the code I introduced and it is executing 6 instances at once:
Future<int> _runPigeon(...) async {
  final Process generate =
    await Process.start('dart', <String>['pub', 'run', 'pigeon', '--input', input]);
  return await generate.exitCode;
}

Future<int> _generateDart(Map<String, String> jobs) async {
  final List<Future<int>> results = <Future<int>>[];
  for (final MapEntry<String, String> job in jobs.entries) {
    results.add(
        _runPigeon(input: job.key, dartOut: job.value, streamOutput: false));
  }
  final List<int> resultsList = await Future.wait(results);
  return resultsList.firstWhere((int x) => x != 0, orElse: () => 0);
}

version
Dart version 2.17.0 (build 2.17.0-27.0.dev)

This could also be a bug in that version of Dart and I just happened to see it after I parallelized the tasks.  I can't prove it's related.

Looks like a pub issue, probably a duplicate of https://github.com/dart-lang/pub/issues/3165.
/cc @sigurdm

Yeah - hopefully this should be fixed by: https://github.com/dart-lang/pub/pull/3285 that has just landed
(Please reopen if this is not true).
A workaround would be to do a single. non-parallel run first to ensure the compilation is stable, and then run the rest in parallel...


Why requestTimeout property got deprecated?

Version: 2.9.2
I need to configure request timeout because we have some long operations.
I found a property UiConfiguration.requestTimeout, but it is deprecated since 2.8.0. So what is the reason and what is the alternative?

Its deprecated because its no longer used by swagger-ui

@dilipkrish So it's not possible to configure this timeout at all??

https://github.com/swagger-api/swagger-ui/issues/3659

@dilipkrish Sorry, it's still not clear for me. The issue you referenced is still open, so, for now, it's impossible to configure request timeout (especially if I don't have swagger-ui's index.html explicitly), isn't it?

I don't say it is an issue of springfox. But okay I'm a developer who decided to use springfox/swagger-ui combination to create clickable Rest API UI, right? I have a method which can be executed for a long period of time. So does it mean that I don't have any chance to increase this timeout? If so it is very disappointing.

It can be adjusted, but unless the swagger-ui exposes a way to do so, our library cannot actually set it no matter what we try.

I don't understand how swagger users live with this restriction.


No ability to flip parallax background

There's no ability to flip parallax background. Maybe add an option?

just flip the nodes children of ParallaxLayer.. you can scale them by -1 at
some axis
On Mon, Jun 13, 2016 at 5:16 PM, zerornull<EMAIL_ADDRESS>wrote:

There's no ability to flip parallax background. Maybe add an option?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5198, or mute the thread
https://github.com/notifications/unsubscribe/AF-Z2zVLL2SFGvcmG3flaHtlPkoj4KRiks5qLbq6gaJpZM4I0sbk
.

@reduz It's not centered so it needs to get parallax background's size and add it as an offset.

Have you tried to set scaling negative on ParallaxLayer say -1, 1. It should flip it and preserve any offsets.

@trollworkout Since it's not centered, it won't be on screen. And it would require to get background's size, which I can't get. Simply scaling it doesn't work, it's not centered.

@reduz It doesn't work, tried that. They're in wrong position.

@trollworkout No, wouldn't work, since parallax layers aren't centered too.

@reduz Also what about adding to canvas layer not only ignoring camera zoom option, but ignoring it's rotation and scale, which could be unchecked? Or simpy making parallax background not rotating with camera?

Its probably fixed(and I dont see other complains about same problem), the issue is also bad formed and does not contain subheaders, version or screenshots. Closed by bugsquad.


fix: disconnect all wc dApps sessions on wcv2 connect

Description

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #5859   👈

This stack of pull requests is managed by Graphite.


Show character limit count on input fields / text areas (maybe?)

There's a need for having the character count of input fields in some of the inputs in use in stories. Currently this is not provided by Catalyst.
Dependent ticket: HSCDB-344

Input fields shall receive the option to display their maximum length of characters within their components' UI.
This should be done by having a common dynamic element, displaying X/Y (X characters out of Y total).
The character count is attached to the label of the input and is available for normal labels, as well as external ones.
Please see the design specifications for implementation details.
https://www.figma.com/file/Uflyvg0Ei23C4iIlC4c97X/(Catalyst)-Web?node-id=2854%3A7651&t=S75KDkBTwZmvYDms-11

Hello, was implementing the change on a HSCDB ticket and noticed a difference in behavior that might be a bug

In our use case both fields are required and the char-count moves next to the label, instead of keeping the right side position. For the comparison i made one of the fields non required and that's the result. Same happens on the Catalyst documentation.
Is this a bug or intended behavior?
@mrkskmn @fynnfeldpausch @emmoldovan FYI

I think it is a bug. I am on it

#158


Swipe refresh problem while scrolling

Hi I'm using swipe refresh for my app but the problem is when I want to scroll my listview up The refresh circle comes down. what can I do?

Use this in your activity.
import android.support.v4.view.ViewCompat;

@Override
public boolean canSwipeRefreshChildScrollUp() {
    if (mListView != null) {
        return ViewCompat.canScrollVertically(mListView, -1);
    }
    return super.canSwipeRefreshChildScrollUp();
}


snipe-it: 6.2.2 -> 6.2.3

Description of changes
https://github.com/snipe/snipe-it/releases/tag/v6.2.3

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Needs a refactor.


Using js_namespace = [] causes a crash in the CLI

Describe the Bug
#[wasm_bindgen]
extern "C" {
    #[wasm_bindgen(js_namespace = [])]
    fn my_function();
}

The proc macro allows this, but the CLI crashes with:
    status: exit code: 101
    stderr:
        thread 'main' panicked at crates\cli-support\src\wit\mod.rs:1087:34:
        range start index 1 out of range for slice of length 0
        note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Steps to Reproduce
Compile the code and run the CLI.
Expected Behavior
I'm not sure, whether the proc macro should just not allow this, or whether this should be interpreted as "no namespace".
Actual Behavior
Crash.
Additional Context
We currently internally represent the namespace of an imported item as Option<Vec<String>>. The CLI apparently assumes that the inner Vec is non-empty.

I'm not sure, whether the proc macro should just not allow this, or whether this should be interpreted as "no namespace".

I think we should allow this but emit a warning. The easier way is definitely to disallow it.
The CLI should obviously not crash either way.


Fix netstim_inhpoisson.mod for CoreNEURON direct mode

Context
Some changes needed for making bbcore_read work with direct mode of CoreNEURON were missing from this file. Same for a fix for clearing the random123 object.
This is the netstim_inhpoisson.mod file which gets used by the neurodamus-models and not the one in common.
Scope
This is file is now the same as the Neurodamus closed source file.
Testing
Should be tested by the blueconfigs CI.
Review

[x] PR description is complete
[ ] Coding style (imports, function length, New functions, classes or files) are good
[ ] Unit/Scientific test added
[ ] Updated Readme, in-code, developer documentation

Can someone remind me why we keep this file in neurodamus/core instead of common ?

I was hoping that someone could answer this as well. @ferdonline @pramodk ?

@iomaganaris Everything ready with this PR?

@iomaganaris Everything ready with this PR?

Yes, ready to go :rocket:


QR scanning Specter export (multisig wallet)

Specter wallet generates a pdf with export functionality and derivation paths.  This pdf also includes a QR code which contains json data for multisig.
Using sparrow QR code scanning to import a multisig of specter wallet generated error "not in json format"

I've tried to reproduce this using Specter 1.2.2, and I haven't been able to.
I'm able to create a multisig 2of2 wallet on Specter and successfully import it to Sparrow using the Specter PDF export (both non-SLIP and SLIP versions) by scanning the QR code using File > Import Wallet > Specter Desktop > Scan in Sparrow.
If you use the "Export To Wallet Software" feature in the Settings > Export Wallet area of the Specter wallet, does the JSON file import to Sparrow? The contents of the JSON file should match the contents of the QR code in the PDF, if you use a QR code reader.

Closing this off, but feel free to add more information.


output ramp?

This is more of a question.
How do I change output value of a plugin over time? I tried to use the following syntax but it seems like not working.
cverb.set('output', [[0.0], [1.0, 2, 1], [0.0, 3, 2]]);   // cverb is a WX.ConVerb

This goes into GenericParam.prototype.set() but does not work since this.$callback is not defined.
cverb.param.output.set(1.0, 2, 1) also works but it only change abruptly.
Cheers!

Thanks for the report.
I believe this.$callback is defined alright, but somehow the events are not scheduled correctly. Needs further investigation.

cverb.param.output.set(1.0, 2, 1) also works but it only change abruptly.

This works as intended. param.set() is basically a simple wrapper of AudioParam automation methods. set(1, 2, 1)' is identical to AudioParam.linearRampToValueAtTime(1, 2)'. If you don't have a previous time/value point, you can't generate a linear ramp.
I have some plans to redesign WAAX's parameter control. Feel free to throw your ideas!


The keyboard may pop twice when TextField's autofocus is true on some phone models.

In Vivo and Huawei p40 pro mobile phone, we found that the keyboard will pop twice when TextField's autofocus is true.
demo
https://user-images.githubusercontent.com/4476512/147572011-62e11ab8-cc51-4c99-8e24-20c15331c725.mp4
code
flutter_test_keyboard.zip
import 'package:flutter/material.dart';
void main() {
  runApp(MyApp());
}
class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);
  final String title;

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

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

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

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: TextField(
        autofocus: true,
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: Icon(Icons.add),
      ),
    );
  }
}

flutter doctor -v

Hi @wangying3426, Thanks for filing the issue. I see that you are using a very old version of flutter and Unfortunately we do not provide support to older versions I would advise you to please upgrade to latest stable 2.8.1 and verify the issue. Please write back with the updated output of flutter doctor -v (as a text)  if you still encounter the issue after the upgrade.
Thank you.

Yes，stable 2.0 has fixed this problem, but don‘t know which commit fix it. Could you tell me which commit fixed the problem? Because some online business will not be upgraded to the new version over a period of time, if possible, we can cherry-pick it firstly.

Thanks for the update @wangying3426, I am sorry Unfortunately, I wouldn't be able to pinpoint the exact commit which fixed it. Probably you will have to figure out this by digging into the source code and comparing it.
Alternatively, if you can't upgrade to 2.0 then try to find flutter versions below 2.0.0  which fixes it and see if you can upgrade to that version.

Closing as fixed in the new release.


TF32 - we should enable this - or is it already done?

https://huggingface.co/docs/transformers/perf_train_gpu_one#tf32
https://developer.nvidia.com/blog/accelerating-ai-training-with-tf32-tensor-cores/
import torch
torch.backends.cuda.matmul.allow_tf32 = True
torch.backends.cudnn.allow_tf32 = True

or
TrainingArguments(tf32=True, **default_args)

This appears to be a no-loss way to speed up training by a bit.  Is it already enabled?  I couldn't tell easily - though since this is 2 years old, it may be.

"TF32 mode is the default option for AI training with 32-bit variables on Ampere GPU architecture"


Html special characters not escaped in unittest output

Not escaping html special characters can lead to failed unittests being shown as expected: but was:. When you use a html debugger you see that the message actually says expected: <foo[]> but was: <foo[bar]>. However, <foo[]> is parsed as an html tag and not shown.
I have only tested this with java.
The following test-case reproduces the example used above.
import org.junit.Test;
import org.junit.Assert;

public class UnitTestHtmlEscapeTest {
  @Test
  public void testHtmlEscape() {
    Assert.assertEquals("foo", "foobar");
  }

I made a temporary kata that shows this bug.
http://www.codewars.com/kata/55b1da58e96f8279800000bf/train/java

Could you edit your issue description and add backticks around your code? E.g. `expected:…but was…`. GitHub strips unknown/invalid HTML tags.

This does not only affect unit testing. I wrote a C++ Kumite with templates. Sadly when publishing it everything between < and > is hidden. Also includes are hidden, so
#include
just becomes
#include
etc.

See #844 for more details. @danamlund you can go to http://preview.codewars.com/kata/55b1da58e96f8279800000bf/train/java to test. Note that it still doesn't work until you take the HTML out of the message, then the auto-detection will detect no HTML and render correctly.

Fix verified.


NadekoBot Closes With Error After Opening

I've changed nothing with NadekoBot as it worked fine until a couple days ago. This is the error I receive when trying to start NadekoBot. After sustaining for a few seconds, the program closes out. I don't know much code, so I've know clue what this means.
-1 18:50:32 ShardsCoordinator | Starting NadekoBot v2.24.5
Unhandled Exception: StackExchange.Redis.RedisConnectionException: It was not possible to connect to the redis server(s); to create a disconnected multiplexer, disable AbortOnConnectFail. SocketFailure on PING
at StackExchange.Redis.ConnectionMultiplexer.ConnectImpl(Func`1 multiplexerFactory, TextWriter log) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\ConnectionMultiplexer.cs:line 876
at NadekoBot.Core.Services.ShardsCoordinator..ctor()
at NadekoBot.Program.Main(String[] args)
at NadekoBot.Program.(String[] args)

Did you install Redis?

Fixed the issue. Thank you.

Glad that did the trick! 😄


WIP - Use grid item wrappers for species data

Use grid item wrappers for species data
Do not allow selection of projects within the species projects list view
Fix species name in rejected messaging
Add additional fields for species in internal view

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

#2639  👈
#2628 : 2 other dependent PRs (#2629 , #2632 )
main

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

Join @nickgraz and the rest of your teammates on  Graphite


[MODEL] XLNet conversion scripts

Description
Add XLNet conversion scripts
Checklist
Essentials

[X] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented

Changes

[X] Pretrained XLNet

Comments
This PR resolves #787

Job PR-866/1 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/1/index.html

Job PR-866/2 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/2/index.html

Job PR-866/4 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/4/index.html

Job PR-866/5 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/5/index.html

Is this ready?

This only contains the model and no fine-tuning scripts. For finetuning, designing a joint API encompasses the existing bert scripts and works for XLNet and moving that to the main package would be a way forward. That can be done in a separate PR though I guess.

@leezu pinging for an update

@leezu @eric-haibin-lin gentle ping

@eric-haibin-lin an example is included now https://github.com/dmlc/gluon-nlp/pull/866/files#diff-820020a4c66a085eb27014cc377a8658

Job PR-866/7 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/7/index.html

Job PR-866/8 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/8/index.html

Job PR-866/9 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/9/index.html

Job PR-866/10 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/10/index.html

Job PR-866/11 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/11/index.html

Job PR-866/12 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/12/index.html

Job PR-866/13 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/13/index.html

Job PR-866/15 is complete.
Docs are uploaded to http://gluon-nlp-staging.s3-accelerate.dualstack.amazonaws.com/PR-866/15/index.html

CI passes now


Support for loading TC-Gen CTC output

Add support for loading TC-Gen CTC output
Acceptance Testing
List input data types and sources.
Describe tests required for new functionality.
Time Estimate
Estimate the amount of work required here.
Issues should represent approximately 1 to 3 days of work.
Sub-Issues
Consider breaking the new feature down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[ ] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[ ] Select component(s)
[ ] Select priority
[ ] Select requestor(s)

Projects and Milestone

[ ] Review projects and select relevant Repository and Organization ones
[ ] Select milestone

Define Related Issue(s)
Consider the impact to the other METplus components.

[ ] METplus, MET, METdb, METviewer, METexpress, METcalcpy, METplotpy

New Feature Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above.
[ ] Fork this repository or create a branch of develop.
Branch name:  feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into develop.
Pull request: feature <Issue Number> <Description>
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

This file looks like the standard CTC file except for 'NA' in fcst and obs lead time.
In the database these columns have a type "INT UNSIGNED" and I can't insert "-9999" or "NA".
Suggestion: replace NA to 0 in this case

MET file with TC-Gen data is added to the test data set


Self-service reset password User flow do not include Custom Attribute claim in Token

[Enter feedback here]
I've created a Signin User Flow and I've enabled Password Reset following your recommandation.
I've also added a custom attribute in the token by adding an API connector on the Signin User Flow as explained here.
It works fine when I test Signin User Flow and enter Email address and password: token contains my custom attribute. But if I click on Forgot your password link and go through Password Reset pages, token contains basic claims included isForgotPassword, but not my custom attribute. My API connector seems not being called.
What's wrong? Can you help me?

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

ID: ee017677-e3c0-fdc7-7dc0-994152d6acf7
Version Independent ID: 271089de-eb61-9ad9-d24e-e98197b2ecc4
Content: Set up a password reset flow - Azure AD B2C
Content Source: articles/active-directory-b2c/add-password-reset-policy.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

@ducalai Apologies for the delay on this one. We use this repository for improvements in the documentation. We checked this internally and it would require a deeper troubleshooting to understand the reason . If you already have a support plan , you can engage Microsoft support on this . If you do not have a support plan please email us at azcommunity [at] microsoft [dot] com with a subject line "ATTN:shashi" referencing this thread along with your azure subscription ID and we will help you with alternate support options.
Thank you .


Improve the progress bar during upload

This does 3 things:

switch from hooking into stream.read into stream.push to draw the progress bar exactly when the data was actually pushed into the buffer. This makes the progress bar itself less accurate (since a push call doesn't imply the data was read) but more reliable (because we don't always know when data was read)
Since it's less accurate, it'll always say 100% even when the HTTP requests are still in-flight. Till all the files are done uploading, we'll render an ora spinner that reads Completing upload…
We always render the progress bar, even if all the files have made it through so the progress is a little more obvious

https://asciinema.org/a/v179wwRTUpUTwe42brTw393Kb


Update README.md

Hello,
I'd like to add this paper to under 'Linguistics':
Nguyen, Li (2018) “Borrowing or code-switching? Traces of community norms in Vietnamese-English speech.” Australian Journal of Linguistics 38(4), pp.443-466. doi:10.1080/07268602.2018.1510727
Also the below paper isn't a survey paper. It's a Position paper, so could we please have a section for that?
Nguyen, Li, Yuan, Z., and Seed, G. (2022) “ Building Educational Technologies for Code-Switching: Current Practices, Difficulties and Future Directions.” Languages 7(220). doi:10.3390/languages7030220
Thanks!


Data-oriented design

stack/dictionary don't exist as structs and instead are just namespaces with functions to call on registers
Stack.push(top: Register, stackSize: usize)
maybe registers can stay as-is or maybe theres a more general way to think about them?

tried it out and the design just gravitated back to what it was originally just with more usage of comptime to set offsets for registers 👍 wahoo


There is a need for a comprehensive example for modifying WAF Policies

This example is very far from reality, and does very little to help with a real-life WAF rule/policy customization.
There is a need for prescriptive guidance to modify an existing WAF by disabling some managed rules, and adding some custom rules, possibly with some exceptions.
When you set out to do that, you land on WAF Policies. And the documentation shows managing the WAF policies on the portal GUI only.
There are some CLI documentation about modifying WAF policies, but examples are "generated" and not really usable. I am yet to find any usable PoSh example or documentation for managing WAF Policies programmatically.
I managed to put together a PoSh script to disable a given list of managed rules WITHOUT adding custom rules on the WAF. This does not require a custom policy.
I managed to put together another script to create a WAF policy that uses OWAP 3.2 as managed rules and has multiple custom rules.
But I am still struggling to disable a selected set of managed rules inside the policy programmatically.
I also opened a support case, the MSFT engineers selflessly tried their best to help, but so far, not much yield.
This makes me wonder if anybody customizes WAF Policies for real... Maybe it is not the best thing to use, or there are only a few brave souls that ever done this.
Pleading for some help.

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

ID: 5a31c2f7-5c4c-60a8-0630-e5f046ba69f7
Version Independent ID: 56009458-2829-0211-4538-6407b4c4c102
Content: Azure PowerShell Script Sample - Create WAF custom  rules
Content Source: articles/application-gateway/scripts/waf-custom-rules-powershell.md
Service: application-gateway
GitHub Login: @greg-lindsay
Microsoft Alias: greglin

Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

@durayakar I initially had the same concerns but ultimately landed on using an ARM template with my policy as the only resource defined. Any time I need to push changes to my many policies, I simply update my template (validated on a test/stg env) and then push with New-AzResourceGroupDeployment -TemplateFile templates/resourceGroup.wafPolicy.json -ResourceGroupName example -Confirm.
This allows me to codify the policy and maintain a record of exclusions and disabled CRS rules.

@durayakar I initially had the same concerns but ultimately landed on using an ARM template with my policy as the only resource defined. Any time I need to push changes to my many policies, I simply update my template (validated on a test/stg env) and then push with New-AzResourceGroupDeployment -TemplateFile templates/resourceGroup.wafPolicy.json -ResourceGroupName example -Confirm.
This allows me to codify the policy and maintain a record of exclusions and disabled CRS rules.

Sorry, not looking for ARM templates but scripting examples that can be used to update multiple WAF policies.
ARM template approach does not work in my case.
Apparently, this was a "feature request".
A new feature is now added to the portal GUI.
But scripting examples and guidance (PowerShell, CLI or Bicep) are still non-existent...

Thanks for the information here. We currently have a large backlog of issues, so I am working through them slowly. I will address this issue shortly.

This is being tracked as internal content issue: 91192
#please-close


samples in the oncoplot

Hi,
As you are getting  total # samples from maf file, it maybe incorrect in cases where maf is only created on targeted panel or it includes actionable variants only, for such cases would it be possible for you to either get the #number of samples from clinical annotation file or from an user provided argument and calculate all the % (gene level and cohort level top header) using this number as denominator?
Thanks,
Rajesh

HI,
These are rare cases, and dont know how often one would expect. But, I am happy to include and argument to provide a custom cohort size. I will let you know once its done.

Thanks @PoisonAlien, will wait for the update.

Hi Rajesh,
Sorry for the delay. I have added cohortSize argument to oncoplot which should do the trick. Please let me know if this helps.

Thank you!!


app crashes when I open it

Hi seht,
I installed the new version 1.1.0-beta1 of your app "Volume Notification" today. Unfortunately, the app crashes when I open it. However, the previous version 1.0.7-alpha3 works correctly. The version before last, 1.06.3-alpha2, also worked. If you need more information I can send it to you.
My hardware:
Oneplus 8 (IN2013)
Android version 13
What can I do to ensure that the latest version runs on my hardware?
Regards
Michael

Please try reinstalling. Thank you!


[Bug report] Photo manager image provider import doesn't work

Version
3.0.0
Platforms
dart
Device Model
Pixel
flutter info
[√] Flutter (Channel stable, 3.16.0, on Microsoft Windows [Version 10.0.22621.2715], locale pl-PL)
    • Flutter version 3.16.0 on channel stable at C:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (9 days ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2

How to reproduce?
Hey, anyone knows what to do when I added this image provider to use AssetEntityImageProvider and tried to add import for that as it is said in docs about migrating (https://github.com/fluttercandies/flutter_photo_manager/blob/main/MIGRATION_GUIDE.md#assetentityimage-and-assetentityimageprovider) and nothing is recognized? Is it a typo? Should it be any other import? I have no clue
Logs
No response
Example code (optional)
No response
Contact
No response

Try restart you editor.


Update framework type information for SDK 10.0.18362.1

I have published the code I used to extract this information here: https://github.com/chausner/XbfGenExtractor

Hello @misenhower
Will it be possible to merge this PR?
I'd like to contribute data from Win11 SDK as well as some changes regarding Win11 XBF decoding.

@ge0rdi I think unfortunately, this repo is no longer being maintained by @misenhower. If you want, feel free to contribute your changes to my fork where I have just merged these changes: https://github.com/chausner/XbfAnalyzer

@chausner
Cool, thank you.
I will have to put it together. As I was not working on this stuff for long time.
Though I'm definitely interested in contributing.


fix: Modify futures_open_interest func to correct api url

related issue: #883 raise BinanceAPIException when call futures_open_interest

Thanks @Jeonghun-Ban


flask run is not compatible with werkzeug middlewares

Expected Behavior
When using DispatcherMiddleware to combine multiple flask apps, I expect to be able to run the app with the following command:
FLASK_APP=main:app flask run

from flask import Flask
from werkzeug.middleware.dispatcher import DispatcherMiddleware

api = Flask("api")
admin = Flask("admin")
app = DispatcherMiddleware(api, {"/admin": admin})

Actual Behavior
The app object fails the following typecheck:
https://github.com/pallets/flask/blob/d4b688bd035b9704b3168b28fed39c8fcfe3b997/src/flask/cli.py#L198
<IP_ADDRESS> - - [09/Jun/2019 15:16:30] "GET /admin/ HTTP/1.1" 500 -
Traceback (most recent call last):
  File "/Users/nshebanov/.pyenv/versions/3.7.2/lib/python3.7/site-packages/flask/_compat.py", line 36, in reraise
    raise value
  File "/Users/nshebanov/.pyenv/versions/3.7.2/lib/python3.7/site-packages/flask/cli.py", line 199, in find_app_by_string
    module=module.__name__, app_name=app_name
flask.cli.NoAppException: A valid Flask application was not obtained from "main:app".

Environment

Python version: 3.7.2
Flask version: 1.0.3
Werkzeug version: 0.15.4

Suggestion
Drop the check when FLASK_APP is specified and it is not a factory, or change the check to allow for middlewares (it's hard to imagine such a check though).
Checking for type Flask makes sense within find_best_app and at the moment of call_factory result evaluation within find_app_by_string.

I implemented the patch according to the suggestion, and it seems to work with the existing tests, however, I didn't add any new – there is a multiapp example already that isn't covered with tests, so would be great to understand why it's left out, and whether it can/should be extended for this case.

The WSGI callable for a Flask application is app.wsgi_app. Try this:
app = Flask("api")
admin = Flask("admin")
app.wsgi_app = DispatcherMiddleware(app.wsgi_app, {"/admin": admin})

See the docs.

I could do that too but it feels... a hack? It obviously doesn't do any harm, but still.
Also, the current behavior makes the [application dispatching example] from docs technically broken.

@killthekitten the solution I pointed you at is the recommended way to do this. The wsgi_app attribute has been in Flask for ages and it was created explicitly to prevent the Flask app instance from being buried under a potentially long chain of middlewares.
The solution that you are proposing breaks the CLI, by the way. If your api application has any custom CLI commands, those are gone once you add the dispatcher middleware. Do it in the recommended way with wsgi_app and the commands remain accessible.

the example is correct
flask.run uses Flask applications, the examples explicitly use wsgi server dispatching
the documented method to apply a wsgi  middleware to the wsgi details of an application is to replace the attribute as documented (its a necessary evil as wsgi middleware itself is a "decorator/proxy style pattern")

👍 thanks for the details guys. I guess I'll have to live with it.


'docker login host-IP' cannot success?

Linux docker-container-B 3.10.0-327.28.3.el7.x86_64 #1 SMP Thu Aug 18 19:05:49 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Docker version 1.11.2, build b9f10c9
docker-compose version 1.8.0, build f3628c7
no error logs.
I installed the harbor-0.3.5, and then I login the web.
I cannot login docker when I use the host IP
docker login <IP_ADDRESS> Username: admin Password:  Error response from daemon: Get https://<IP_ADDRESS>/v1/users/: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused
I can login docker when I used <IP_ADDRESS>, but I cannot push docker image successfully.
docker login <IP_ADDRESS> Username (admin): admin Password:  Login Succeeded
docker push <IP_ADDRESS>/public/harbor-log:0.3.5 The push refers to a repository [<IP_ADDRESS>/public/harbor-log] Get https://<IP_ADDRESS>/v1/_ping: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused
I don't know why?

If you are reporting a problem, please make sure the following information are provided:
1)Version of docker engine and docker-compose
2)Config files of harbor, you can get them by packaging "Deploy/config" directory
3)Log files, you can get them by package the /var/log/harbor/

Oh! Sorry, I uploaded files yesterday, but now I can not find them too?
1)Version of docker engine and docker-compose
Linux docker-container-B 3.10.0-327.28.3.el7.x86_64 #1 SMP Thu Aug 18 19:05:49 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Docker version 1.11.2, build b9f10c9
docker-compose version 1.8.0, build f3628c7
2)Config files of harbor, you can get them by packaging "Deploy/config" directory
config.zip
3)Log files, you can get them by package the /var/log/harbor/
harbor-0.3.5.zip

@tuxknight
I cannot login docker when I use the host IP
docker login <IP_ADDRESS>
Username: admin
Password:
Error response from daemon: Get https://<IP_ADDRESS>/v1/users/: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused
I can login docker when I used <IP_ADDRESS>, but I cannot push docker image successfully.
docker login <IP_ADDRESS>
Username (admin): admin
Password: Login Succeeded
docker push <IP_ADDRESS>/public/harbor-log:0.3.5
The push refers to a repository [<IP_ADDRESS>/public/harbor-log] Get https://<IP_ADDRESS>/v1/_ping: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused

I used wrong password to login and found that:
Why <IP_ADDRESS> used http, but <IP_ADDRESS> used https?
I`m not change any config but "hostname = <IP_ADDRESS>".
docker login <IP_ADDRESS>
Username (admin): admin
Password:
Error response from daemon: Get http://<IP_ADDRESS>/v2/: unauthorized: authentication required
docker login <IP_ADDRESS>
Username: admin
Password:
Error response from daemon: Get https://<IP_ADDRESS>/v1/users/: dial tcp <IP_ADDRESS>:443: getsockopt: connection refused

@wangfeiping You should check arguments of docker daemon.  Insecure registry communication over http only enabled with option --insecure-registry

@tuxknight Thank you!
I don't know why, but I found that:
ps -ef | grep dockerd
root       6250      1  1 18:42 ?        00:00:01 dockerd -H tcp://<IP_ADDRESS>:2376 -H unix:///var/run/docker.sock --storage-driver devicemapper --tlsverify --tlscacert /etc/docker/ca.pem --tlscert /etc/docker/server.pem --tlskey /etc/docker/server-key.pem --label provider=generic
dockerd is not execute according to the config (/usr/lib/systemd/system/docker.service):
ExecStart=/usr/bin/dockerd --insecure-registry <IP_ADDRESS>
Fortunately, I run docker with "/usr/bin/dockerd --insecure-registry <IP_ADDRESS>", and all is ok!
I still don't know why, may be my mistake or just a docker's issue.

https://wiki.archlinux.org/index.php/Systemd_
systemd
The available unit files can be seen in /usr/lib/systemd/system/ and /etc/systemd/system/ (the latter takes precedence).

I find if you only use docker-registry,you can use  "--insecure-registry <IP_ADDRESS>'' to solve this question,and you can pull or push images;but if you use harbor, using "--insecure-registry <IP_ADDRESS>' can't sovle this question ,and have a "connection refused" error

解决方法见: https://docs.docker.com/registry/insecure/#deploy-a-plain-http-registry

Edit the daemon.json file, whose default location is /etc/docker/daemon.json on Linux or C:\ProgramData\docker\config\daemon.json on Windows Server. If you use Docker for Mac or Docker for Windows, click the Docker icon, choose Preferences, and choose +Daemon.
If the daemon.json file does not exist, create it. Assuming there are no other settings in the file, it should have the following contents:

{
  "insecure-registries":["<IP_ADDRESS>/0"]
}

I try it but it can't  solve this problem ,I run my docker with daemon,so I add 'insecure-registries' with my docker.srvice like this;
vim /etc/systemd/system/docker.service
[Unit]
Description=Docker Application Container Engine
Documentation=http://docs.docker.io

[Service]
Environment="PATH=/usr/local/bin:/bin:/sbin:/usr/bin:/usr/sbin"
ExecStart=/usr/bin/dockerd --log-level=error \
--graph=/app/docker/data \
--registry-mirror=https://docker.mirrors.ustc.edu.cn \
--insecure-registry=<IP_ADDRESS>:5000 \
--insecure-registry=https://docker.mirrors.ustc.edu.cn
ExecReload=/bin/kill -s HUP $MAINPID
Restart=on-failure
RestartSec=5
LimitNOFILE=infinity
LimitNPROC=infinity
LimitCORE=infinity
Delegate=yes
KillMode=process

[Install]
WantedBy=multi-user.target

and then execute it with commend below
systemctl daemon-reload
systemctl restart docker

I think I have rum command like  'insecure-registries',It runs well with docker-registry,but wrong with harbor


Update service-security-sensitivity-label-change-enforcement.md

Simply having EDIT should not technically allow to edit or remove the label. Can this be reviewed please.

@paulinbar
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @nachiketkmaat, did you take into account that the case where just EDIT is sufficient is in the section on relaxations to accommodate  automatic labeling scenarios, such as inheritance from data source and downstream inheritance? Here's the key sentence concerning this:
These automated scenarios can result in situations where no user has been set as the RMS label issuer for a label on an item. This means that there is no user who is guaranteed to be able to change or remove the label. In such cases, the usage rights requirements for changing or removing the label are relaxed - a user needs just one of the following usage rights to be able to change or remove the label:
OWNER
EXPORT
EDIT
I believe that the documentation is correct as written, but will double-check with the product team in any case. Thanks for raising your concern. I'll update you here as soon as I hear back from them.

Hi @nachiketkmaat, I checked with the product team and indeed the documentation is correct. In ordinary cases, EDIT alone is not sufficient for changing or removing the label, but because of situations that can arise as a result of auto-labeling (for example, inherit from data source and downstream inheritance) there is a relaxation of enforcement in those cases whereby EDIT alone is sufficient.
Thanks again for reaching out. I'm closing this PR now but feel free to comment if you have further questions.


[Question] Specifying the hostname on the web interface

Hello,
Is it possible with Verdaccio 4.3.4 to a custom hostname in order for the web interface to display the hostname instead of "localhost" ?

Thank you !

Partial answer : the web interface automatically retrieves the content of the URL to display it in the help modal.
But, is it possible to fix a specific hostname in the config file of the web interface ?

hi @flavienbwk , it is not possible, it uses the one defined as host. See technical details here,  https://github.com/verdaccio/verdaccio/blob/master/src/api/web/index.ts#L79
We do that due most of the users use proxies as Nginx or HAProxy. Otherwise would be missleading.

Thank you for your quick answer !


YSort incorrectly sorts nodes above a y-position of 100000

Godot version
3.5.stable
System information
Windows 10
Issue description
I discovered that the YSort node stops properly sorting nodes with small separation values, as soon as the y position of the nodes gets above 100k.
Nodes that only differ 1 pixel get sorted properly until the nodes reach position 100001 and 100002, at which they stop being sorted at all by the Ysort. From this point on, only nodes with a separation of at least 2 pixels get sorted.
This discrepancy gets bigger for every 100k pixels that you move. Meaning that at positions of 500k+, only nodes that are separated at least 5 pixels get sorted.
This is the scene tree:

The Sprite named Front is higher in the scene tree, meaning it will always be drawn behind the Back Sprite. I modulated the Back sprite to be red. As is the case in the editor view:

To make sure we never fully see the Back Sprite, I offset the Front Sprite by 1 pixel. The Ysort should therefore always drawn the Front sprite OVER the red Back Sprite. And that works, until we get to the 100k line. After that the Ysorting stops functioning as expected

Steps to reproduce

Open the Reproduction project
Press Play
The sprites will initially order properly, but when they cross over the 100k line, they stop sorting completely. You'll notice this because the Back Sprite, which is completely red and normally hidden, due to the normal Front sprite having a higher position, suddenly becoming drawn in front.

You can test this with different separation and yposition values. You'll find that for every 100k pixels (both positive and negative), the separation needs to increase by 1 for the Ysort to continue to function properly.
enable tween to smoothly move the sprite over a range, or disable it to just see what the sorting is at your input position.
Minimal reproduction project
ysorttest.zip

This looks suspiciously like it's caused by floating point precision problems, if you are using single precision builds you get the floating point precision of that, which has only 6 significant digits of precision

Forgive me if I'm misunderstanding, but if it has 6 digits of precision, shouldn't it work until 999999, rather than 100001?
I considered floating point issues early on, since that could definitely cause both values to be approximately the same, if there is only 1 pixel difference. However, this also happens if there are 5 pixels of difference and the position is above 500k.
500000.0002 and<PHONE_NUMBER> might lose their decimal precision, but in that case the Ysort should still handle 500004 > 500000, right? Or does the uncertainty become bigger, the larger the number becomes?

The uncertainty becomes bigger, because we're dealing with binary numbers, so it's really 24 digits of binary precision
It seems to potentially be related to other comparison aspects, or cumulative details, unsure, will investigate

It does feel rather odd to say that 500001 and 5000005 are approximately equal. In most scenarios I can think of, they'd be as equal as 501 and 505 are. The tolerance seems a lot larger than it (possibly) needs to be for large numbers.
But I'm most likely overlooking cases in which that tolerance is important, so I'll leave the decision up to those that have a better understanding of the whole picture

Changing the behaviour in is_equal_approx would be breaking and major, but some change might be required here as it's a pretty significant mess here, but at least it should be documented

Since it would be breaking, this wouldn't happen until Godot 5.x which is... a couple years in the future, so it should be documented for now

I mean the sorting part could be reworked without touching equal approx, but unsure about some of the details there, but it should at the very least be documented both in 3.x and 4.x

It should probably also be documented in other nodes/basic functionality that uses is_equal_approx and can handle large numbers, because it might give unexpected results there too.

It's documented, at least in 4.x:

Here, "approximately equal" means that a and b are within a small internal epsilon of each other, which scales with the magnitude of the numbers.

It might be worth mentioning in the documentation for 3.x and 4.x that the small internal epsilon is 1/100000. That will help users to understand the limitations slightly better. In my opinion it would even be worth adding a NOTE showing that numbers over 100000 might give unexpected results, since they are considered approximately equal if the difference between them is 1, and that this is a limitation of the function.
The documentation is fine if you already understand the logic, but if you run into an issue only reading a small internal epsilon and scales with the magnitude of the numbers wouldn't be enough to understand that you have ran into a limitation, at least to me.

I remember when it was discussed to make is_equal_approx() smarter (i.e. automatically determine epsilon based on the value of the two numbers to compare), but this wasn't done for some reason.

Confirmed this happens in Godot 4 (YSort behaviour is now part of Node2D, though you need to set "y_sort_enabled" to true).
Attached is the project converted to run on Godot 4: ysorttest_4.zip

Actually @Calinou , could you link me to the discussion about improving is_equal_approx()? I've been doing some research and I've got some thought I want to discuss.


Fix GLB export with characters that don't fit into single UTF-16 code point

Fixes #10672. Uses TextEncoder when available to encode text correctly when converting from JSON to binary. If not available, simply places "_" characters when the true character doesn't fit into a single code point (ie the value is greater than 0xFFFF). See this MDN article for more details: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/charCodeAt

@sebavan the array needs to have a buffer and byteOffset property but these can only be set when the array is constructed

ohhh Is textEncode.encode() not returning a uint8array containing this ?


94 cart

Closing this PR since I need to change some stuff now that the menu is up and running. Will be a pretty substantial change since I need to update all the views that use a frappe because now I want to use menu items


Dereferencing rules

Hello everybody,
This issue is specifically about AsyncApi 2.0.0, but applies to v2.1.0 respectively. I'm working on a dereferencing tool a bumped to another issue regarding JSON Schema dereference. We've already stipulated in https://github.com/asyncapi/spec/issues/552 that whenever the $ref field is intercepted in Schema Object, this Schema Object becomes a Reference Object and not a Schema Object with $ref keyword. This effectively means that dereference is purely guided using rules around Reference Object.
Never the less, the File Structure section contains following sentence:

The A2S representation of the API is made of a single file. However, parts of the definitions can be split into separate files, at the discretion of the user. This is applicable for $ref fields in the specification as follows from the JSON Schema definitions.

The interesting part is the second (last) sentence. It's not very clear what this sentence actually tries to tell us.
There can be two possible interpretations (that I can see):

$ref fields follow rules around JSON Schema dereference

As stipulated in https://github.com/asyncapi/spec/issues/552 this in not true.

$ref fields supports only referencing JSON Schemas

Again this in not true. $ref field can only be present (and have semantics) in Reference Object and Reference Object can reference other specification objects (Security Scheme, Parameter, Message, etc..).
IMHO this sentence mandates a major change in wording to clarify it's true meaning.
We can take inspiration from OpenApi 3.1:
An AsyncAPI document MAY be made up of a single document or be divided into multiple, connected parts at the discretion of the author. In the latter case, Reference Objects are used.

@char0n Hi! I think that this sentence points to logic how it works in the JSON Schema (...This is applicable for $ref fields...), I mean, referencing between files, but doesn't tell that it's only used for Schema Objects, but also for another part of spec - something like an example in correlation to another "tool", which is JSON Schema.

@char0n Do you have any suggestions to improve the description of dereferencing rules?

Hi @magicmatatjahu,

Hi! I think that this sentence points to logic how it works in the JSON Schema (...This is applicable for $ref fields...)

How it works in JSON Schema is completely different how Reference Object works in AsyncApi spec. That's my whole point. This is why this sentence is confusing for the reader and opens door for different interpretations.

I mean, referencing between files

In AsyncApi, referencing is defined exclusively by Reference Object and not by JSON Schema rules. Even JSON Schema uses Reference Object to process references and transclution.
I've already provided my suggestion in my first comment:
I'd replace this sentence:

The A2S representation of the API is made of a single file. However, parts of the definitions can be split into separate files, at the discretion of the user. This is applicable for $ref fields in the specification as follows from the JSON Schema definitions.

with the following one:

An AsyncAPI document MAY be made up of a single document or be divided into multiple, connected parts at the discretion of the author. In the latter case, Reference Objects is used.

If agreed I can issue a PR.

I've issued a PR with the clarification: https://github.com/asyncapi/spec/pull/602

Attached PR has been merged, closing this.


fix(app): Update ios deeplink handling

Codecov Report
Patch and project coverage have no change.

Comparison is base (5ee15ba) 89.78% compared to head (35a440c) 89.78%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #429   +/-   ##
=======================================
  Coverage   89.78%   89.78%
=======================================
  Files          79       79
  Lines        2947     2947
=======================================
  Hits         2646     2646
  Misses        199      199
  Partials      102      102

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


Minor issues in plotters output

I noticed that the plotters integration has some minor rendering issues when compared to the "reference" plotters backend. Given the two minimal reproduction examples:

Example using cushy/kludgine backend
// Note that this requires to enable the corresponding plotters feature,
// i.e., default-feature = false will not work.
use cushy::widget::MakeWidget;
use cushy::widgets::Canvas;
use cushy::Run;
use plotters::prelude::*;

pub fn basic_plot<A>(
    root: &DrawingArea<A, plotters::coord::Shift>,
) -> Result<(), Box<dyn std::error::Error>>
where
    A: DrawingBackend,
    A::ErrorType: 'static,
{
    root.fill(&WHITE)?;
    let mut chart = ChartBuilder::on(&root)
        .caption("y=x^2", ("sans-serif", 50).into_font())
        .margin(5)
        .x_label_area_size(30)
        .y_label_area_size(30)
        .build_cartesian_2d(-2f32..2f32, -0.1f32..1f32)?;

    chart.configure_mesh().draw()?;

    chart
        .draw_series(LineSeries::new(
            (-100..=100).map(|x| x as f32 / 50.0).map(|x| (x, x * x)),
            &RED,
        ))?
        .label("y = x^2")
        .legend(|(x, y)| PathElement::new(vec![(x, y), (x + 20, y)], &RED));

    chart
        .configure_series_labels()
        .background_style(&WHITE.mix(0.8))
        .border_style(&BLACK)
        .draw()?;

    Ok(())
}

fn plotters() -> impl MakeWidget {
    Canvas::new({
        move |context| {
            basic_plot(&context.gfx.as_plot_area()).unwrap();
        }
    })
}

fn main() -> cushy::Result<()> {
    plotters().run()
}

Example using plain plotters backend
use plotters::prelude::*;

fn main() -> Result<(), Box<dyn std::error::Error>> {
    let root = BitMapBackend::new("basic_plot.png", (640, 480)).into_drawing_area();

    root.fill(&WHITE)?;
    let mut chart = ChartBuilder::on(&root)
        .caption("y=x^2", ("sans-serif", 50).into_font())
        .margin(5)
        .x_label_area_size(30)
        .y_label_area_size(30)
        .build_cartesian_2d(-2f32..2f32, -0.1f32..1f32)?;

    chart.configure_mesh().draw()?;

    chart
        .draw_series(LineSeries::new(
            (-100..=100).map(|x| x as f32 / 50.0).map(|x| (x, x * x)),
            &RED,
        ))?
        .label("y = x^2")
        .legend(|(x, y)| PathElement::new(vec![(x, y), (x + 20, y)], &RED));

    chart
        .configure_series_labels()
        .background_style(&WHITE.mix(0.8))
        .border_style(&BLACK)
        .draw()?;

    root.present()?;

    Ok(())
}

The cushy/kludgine based output is:

The plotters based output is:

The main issue is that text labels are wrongly aligned. Minor issues are that the line anti-aliasing doesn't look as good, and that the grid get aliased instead of being pixel perfect (but likely that is expected if the size of the drawing area doesn't allow for integer multiples of the grid size).
Since I'm not entirely sure if this a cushy or kludgine issue (or even further upstream?), I decided to report it on cushy. Do you generally prefer such issues being raised further upstream if possible?
Note that the lack of clipping is an upsteam issue in plotters https://github.com/plotters-rs/plotters/issues/622

Thank you for reporting this! I knew I had under-tested some of the plotters code, but some of that label drawing is pretty bad too.
The fuzziness is almost certainly due to me trying to automatically scale the plotters output based on DPI scaling. It seems like the more correct approach is going to be to let it be pixel-perfect rendering, and require that the plotters-integrator worry about dpi scaling themselves.
As for where to report the issue, I'm happy with most issues being reported on this repository -- it's where most users might actually encounter a bug. I can always create a separate issue and link them if I think someone else might tackle it, but right now I'm usually the person to fix something in Kludgine!

Things are looking better! I can't seem to figure out how the labels are sized -- I put a lot of logging code in, and I notice plotters measures the chart caption, but not the labels. My picture shows two series because I was trying to see if that would cause it to measure the text... but it didn't.
The issues were not what I originally suspected. It was that to draw a pixel-perfect 1px wide line, the coordinates need to be offset because lyon generates a rectangle centered on the path coordinates. To ensure the 1px wide line isn't split across two lines with Kludgine's subpixel rendering support, the coordinates need to be offset by 1/2 a pixel, or more generalized, half of the stroke width.
Beyond that, the text drawing code didn't support anchoring or alignment or rotation. This plot doesn't rotate text, but in theory everything is handled correctly now -- don't hesitate to let me know if you run into other issues! If you test the main branch, be sure to use cargo update to get Kludgine's changes.

Actually I'm reopening this because I'd like to have you weigh in on what's happening with the labels, if you know more about how plotters works.

Thanks for the fix!

Actually I'm reopening this because I'd like to have you weigh in on what's happening with the labels, if you know more about how plotters works.

In my opinion, this looks pretty much what I would expect now, so closing the issue should be fine. In case anything else comes up I'll let you know. I also don't have much expertise with plotters, mainly just trying things out so far.


Cannot save output video file

Strangely I cannot get my local mediapipe save output video file. I am using the exact the same command on another laptop and it works. On my main computer I can see my GPU process the whole video, however nothing is saved.
I have checked the file permissions, changed the input and output video path to several different locations and also updated the local repository to the lastest. Still, I cannot save the result file.
GLOG_logtostderr=1 bazel-bin/mediapipe/examples/desktop/multi_hand_tracking/multi_hand_tracking_gpu   --calculator_graph_config_file=mediapipe/graphs/hand_tracking/multi_hand_tracking_mobile.pbtxt \
> --input_video_path=/home/tony/mediapipe/output.avi \ --output_video_path=/home/tony/mediapipe/output4.avi
I20200803 21:01:47.609367 13926 demo_run_graph_main_gpu.cc:51] Get calculator graph config contents: # MediaPipe graph that performs multi-hand tracking with TensorFlow Lite on GPU.
# Used in the examples in
# mediapipe/examples/android/src/java/com/mediapipe/apps/multihandtrackinggpu.

# Images coming into and out of the graph.
input_stream: "input_video"
output_stream: "output_video"

# Throttles the images flowing downstream for flow control. It passes through
# the very first incoming image unaltered, and waits for downstream nodes
# (calculators and subgraphs) in the graph to finish their tasks before it
# passes through another image. All images that come in while waiting are
# dropped, limiting the number of in-flight images in most part of the graph to
# 1. This prevents the downstream nodes from queuing up incoming images and data
# excessively, which leads to increased latency and memory usage, unwanted in
# real-time mobile applications. It also eliminates unnecessarily computation,
# e.g., the output produced by a node may get dropped downstream if the
# subsequent nodes are still busy processing previous inputs.
node {
  calculator: "FlowLimiterCalculator"
  input_stream: "input_video"
  input_stream: "FINISHED:multi_hand_rects"
  input_stream_info: {
    tag_index: "FINISHED"
    back_edge: true
  }
  output_stream: "throttled_input_video"
}

# Determines if an input vector of NormalizedRect has a size greater than or
# equal to the provided min_size.
node {
  calculator: "NormalizedRectVectorHasMinSizeCalculator"
  input_stream: "ITERABLE:prev_multi_hand_rects_from_landmarks"
  output_stream: "prev_has_enough_hands"
  node_options: {
    [type.googleapis.com/mediapipe.CollectionHasMinSizeCalculatorOptions] {
      # This value can be changed to support tracking arbitrary number of hands.
      # Please also remember to modify max_vec_size in
      # ClipVectorSizeCalculatorOptions in
      # mediapipe/graphs/hand_tracking/subgraphs/multi_hand_detection_gpu.pbtxt
      min_size: 2
    }
  }
}

# Drops the incoming image if the previous frame had at least N hands.
# Otherwise, passes the incoming image through to trigger a new round of hand
# detection in MultiHandDetectionSubgraph.
node {
  calculator: "GateCalculator"
  input_stream: "throttled_input_video"
  input_stream: "DISALLOW:prev_has_enough_hands"
  output_stream: "multi_hand_detection_input_video"
  node_options: {
    [type.googleapis.com/mediapipe.GateCalculatorOptions] {
      empty_packets_as_allow: true
    }
  }
}

# Subgraph that detections hands (see multi_hand_detection_gpu.pbtxt).
node {
  calculator: "MultiHandDetectionSubgraph"
  input_stream: "multi_hand_detection_input_video"
  output_stream: "DETECTIONS:multi_palm_detections"
  output_stream: "NORM_RECTS:multi_palm_rects"
}

# Subgraph that localizes hand landmarks for multiple hands (see
# multi_hand_landmark.pbtxt).
node {
  calculator: "MultiHandLandmarkSubgraph"
  input_stream: "IMAGE:throttled_input_video"
  input_stream: "NORM_RECTS:multi_hand_rects"
  output_stream: "LANDMARKS:multi_hand_landmarks"
  output_stream: "NORM_RECTS:multi_hand_rects_from_landmarks"
}

# Caches a hand rectangle fed back from MultiHandLandmarkSubgraph, and upon the
# arrival of the next input image sends out the cached rectangle with the
# timestamp replaced by that of the input image, essentially generating a packet
# that carries the previous hand rectangle. Note that upon the arrival of the
# very first input image, an empty packet is sent out to jump start the
# feedback loop.
node {
  calculator: "PreviousLoopbackCalculator"
  input_stream: "MAIN:throttled_input_video"
  input_stream: "LOOP:multi_hand_rects_from_landmarks"
  input_stream_info: {
    tag_index: "LOOP"
    back_edge: true
  }
  output_stream: "PREV_LOOP:prev_multi_hand_rects_from_landmarks"
}

# Performs association between NormalizedRect vector elements from previous
# frame and those from the current frame if MultiHandDetectionSubgraph runs.
# This calculator ensures that the output multi_hand_rects vector doesn't
# contain overlapping regions based on the specified min_similarity_threshold.
node {
  calculator: "AssociationNormRectCalculator"
  input_stream: "prev_multi_hand_rects_from_landmarks"
  input_stream: "multi_palm_rects"
  output_stream: "multi_hand_rects"
  node_options: {
    [type.googleapis.com/mediapipe.AssociationCalculatorOptions] {
      min_similarity_threshold: 0.5
    }
  }
}

# Subgraph that renders annotations and overlays them on top of the input
# images (see multi_hand_renderer_gpu.pbtxt).
node {
  calculator: "MultiHandRendererSubgraph"
  input_stream: "IMAGE:throttled_input_video"
  input_stream: "DETECTIONS:multi_palm_detections"
  input_stream: "LANDMARKS:multi_hand_landmarks"
  input_stream: "NORM_RECTS:0:multi_palm_rects"
  input_stream: "NORM_RECTS:1:multi_hand_rects"
  output_stream: "IMAGE:output_video"
}
I20200803 21:01:47.609887 13926 demo_run_graph_main_gpu.cc:57] Initialize the calculator graph.
I20200803 21:01:47.611706 13926 demo_run_graph_main_gpu.cc:61] Initialize the GPU.
I20200803 21:01:47.616763 13926 gl_context_egl.cc:158] Successfully initialized EGL. Major : 1 Minor: 5
I20200803 21:01:47.658483 13939 gl_context.cc:324] GL version: 3.2 (OpenGL ES 3.2 NVIDIA 440.100)
I20200803 21:01:47.658557 13926 demo_run_graph_main_gpu.cc:67] Initialize the camera or load the video.
I20200803 21:01:47.668368 13926 demo_run_graph_main_gpu.cc:88] Start running the calculator graph.
I20200803 21:01:47.671754 13926 demo_run_graph_main_gpu.cc:93] Start grabbing and processing frames.
INFO: Created TensorFlow Lite delegate for GPU.
I20200803 21:01:52.169677 13926 demo_run_graph_main_gpu.cc:175] Shutting down.
I20200803 21:01:52.173394 13926 demo_run_graph_main_gpu.cc:189] Success!
Segmentation fault (core dumped)

You have a segfault.

I put strace for program, not sure if that is any help

Somehow I tried it again today, and somehow it just worked. Maybe I cannot insert the \ character before the output_video_path


Shebang needs changing

Otherwise /usr/bin/python is used instead of conda python

I thought catkin and/or conda takes care of this but for some reason it's not being replaced.

Packages that are affected:
./lib/python3.6/site-packages/tf2_ros/__init__.py:#! /usr/bin/python
./lib/python3.6/site-packages/diagnostic_analysis/exporter.py:#!/usr/bin/python
./lib/python3.6/site-packages/diagnostic_analysis/sparse.py:#!/usr/bin/python
./lib/python3.6/site-packages/tf2_py/__init__.py:#! /usr/bin/python
./lib/python3.6/site-packages/docutils/utils/code_analyzer.py:#!/usr/bin/python
./lib/python3.6/site-packages/docutils/utils/smartquotes.py:#!/usr/bin/python
./lib/python3.6/site-packages/rqt_shell/xterm_widget.py:#!/usr/bin/python
./lib/diagnostic_analysis/export_csv.py:#!/usr/bin/python
./lib/diagnostic_analysis/sparse_csv.py:#!/usr/bin/python
./lib/tf2_kdl/test.py:#!/usr/bin/python

And actionlib, too. Any clue why conda doesn't change the shebang?


updates license date

If anything I'd hyphenate (2014-2016), but in any case I'm not so sure that this is necessary in the first place. My understanding for OSS license is valid from the first year listed in the license to current day. Please correct me if I'm wrong.

Yeah I agree.  @sdaitzman can you update the language to indicate a date range please?

Fixed!


Product Spec: Thumbnails

Product Spec: Thumbnails
A video thumbnail is a still image that acts as the preview image for the video.
Why this is important
Thumbnails are a natural extension of VOD because they allow developers to create a video gallery (e.g. Youtube) like experience for their customers. Without thumbnails, it’s very difficult for developers to build gallery experience.
Building thumbnails materially improves the VOD product and enables more use cases of the existing VOD MVP.
Requirements:

Thumbnail images need to allow the user to effectively preview video (i.e. it needs to contain an image of the video content vs a blank screen)
Thumbnail images need to be connected to the video that they are representing in the dashboard and API
Thumbnail images need to be able to retrieved in the API
Thumbnail images need to be easily embedded in a frontend
Thumbnails need to be available in a variety of sizes (or size needs to be able to be customized) to adapt to different experiences

Nice to have (but not required):

Allowing the thumbnail to be customized
Moving thumbnails or several frames so that it’s easier to preview the video effectively

MVP

Thumbnails are available in the API
Thumbnail size is easy to customize
It’s possible to build a sample front end application with a gallery

Future Iterations

Dashboard integration
Allow the customization of thumbnails (i.e. changing the contents or selecting a frame)
Moving thumbnails that allow several frames to be displayed

@victorges @gioelecerati Please take a look ^^ leave comments here: https://www.notion.so/livepeer/Product-Spec-Thumbnails-96182db91ba74e3bb28a3787b928b7b9


Qi4j + Spatial Queries

Hi guys,
this pull contains a spatial implementation based on ES.
Lets talk..
Cheers,
jj

Hey @ramtej !
That's a pretty big one :)
Will review & test as soon as I get free cycles.
In the meantime, could you review the complete diff yourself to remove any whitespace-only-like changes ? (https://github.com/Qi4j/qi4j-sdk/pull/22/files)


Floorceilround

Add hackneyed BCMath support for floor, ceil, round operations, in preparation for the upcoming RFC

Adds ->floor(), ->ceil(), ->round() methods to Qed
Adds polyfills for bcfloor, bcceil, bcround

CAUTION: These polyfills are truly, embarassingly terrible. Little more than placeholders. DO NOT RELY ON THEM. (Please help fix them.)

Adds some basic preliminary tests

Addresses Issue #1

Updated the PR to work with PHP_ROUND_HALF_* modes properly, and added many more test cases that currently pass.
This is still not comprehensive and should not be relied upon.


Add i18n support to Feedback

@dtdesign Okay, then this PR is ready. I also added a small smoke test for the i18n support.
If you are happy with this and press the green button then I'll update the version in WoltLab/WCF#3449 and adjust the PasswordStrength class over there to make use of localized feedback.


Add odeint max step size optional argument

It'd be nice if odeint had an optional argument analogous to scipy solve_ivp's max_step argument, which controls the maximum step size. I'll copy a code example below, but I'm running into an issue with odeint in which it steps over a feature in the ODE rhs after a long-ish period for which rhs=0.
One minor issue is that max_step unfortunately has a naming collision with odeint's optional arg mxstep, which is an integer controlling the total number of iterations odeint can take before exiting, so this name may not be usable without changing mxstep which may not be desirable.
Aside from choosing the argument name, I think implementing this is nearly trivial: just need to add the argument, and I believe change line 199 to take the minimum of the max step size and the computed optimal step. The example I post below could be rolled into a unittest validating this behaviour (perhaps with comparison directly with scipy).
Thoughts? Given how simple it is I'm happy to implement this myself, but would definitely defer to the core JAX developers for what argument name to use (or for any insight into why this might be an issue with other parts of odeint/JAX control flow).
Motivating example
import jax
jax.config.update("jax_enable_x64", True)
jax.config.update('jax_platform_name', 'cpu')

import jax.numpy as jnp
from jax.experimental.ode import odeint

dt = jnp.array(1.)
k = 5
def deriv(t):
    t = jnp.array(t)
    return jnp.piecewise(
        t,
        [t <= 2 * dt, (t >= k * dt) & (t <= (k + 2) * dt)],
        [lambda s: jnp.array(1.), lambda s: jnp.array(-1.), lambda s: jnp.array(0.)]
    )

def rhs(y, t):
    return deriv(t)

odeint(
    func=rhs,
    y0=jnp.array(0.),
    t=jnp.array([0., k*dt, 2*k*dt]),
    atol=1e-8, rtol=1e-8
)

Using odeint, the above code is essentially integrating a function which is 1. over the interval [0., 2*dt], -1. over the interval [k*dt, (k+1)*dt], and zero elsewhere. If integrated over an interval containing both non-zero intervals, the correct result should be 0.. However , the above code outputs:
DeviceArray([0.        , 1.99999565, 1.99999565], dtype=float64)

and changing the tolerance doesn't change this (it just yields better or worse approximations of [0., 2., 2.])
The analogous code in scipy using solve_ivp with method='RK45' yields the same results, but with solve_ivp this can be corrected by setting max_step=dt. Shortening the intermediate 0. interval by either reducing dt or k both solve the problem as well, so I'm pretty confident it's an issue of step size overshoot.
Slightly more context
The above is a minimal example, but the similarities to the situation in which I'm using odeint and the above are:

I don't know ahead of time if a period of 0. will be present or for how long.
I do however have something analogous to dt - basically a natural time scale over which to restrict the step so that features aren't missed, but that also isn't so short that it will totally mess with the efficiency of the solver.

@patrick-kidger we are actually in the process of adding diffrax as a dependency to the project in which this came up :) .
In any case it would still be a nice feature to have in odeint


Crash when updating message if a list is present

Describe the bug
If you have a dialog with list: item added, and then update the message:, Dialog crashes.
To Reproduce
dialog &
echo 'listitem: add, title: hi' >> /var/tmp/dialog.log
echo 'list: show' >> /var/tmp/dialog.log
echo 'message: A test message' >> /var/tmp/dialog.log

This will cause /usr/local/bin/dialog: line 58: 75605 Trace/BPT trap: 5       "$dialogbin" "$@" to be printed to console and Dialog to crash.
Expected behavior
Does not crash :)
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

macOS version: 15.0 B3
swiftDialog version: <IP_ADDRESS>68

@BigMacAdmin  -- Good callout, I'll edit the report, but we're not seeing the behavior on 14.5 or other previous OS's.
Only seeing on 15.0 B3 so far.

I ran @BigMacAdmin's script and it crashed for me with the following errors.
2024-07-17 13:35:07.213 Dialog[32705:1229453] +[IMKClient subclass]: chose IMKClient_Legacy 2024-07-17 13:35:07.213 Dialog[32705:1229453] +[IMKInputSession subclass]: chose IMKInputSession_Legacy [/usr/local/bin/dialog: line 58](coderunner:///usr/local/bin/dialog#58:0!): 32705 Trace/BPT trap: 5       "$dialogbin" "$@"
Swift Dialog <IP_ADDRESS>68
macOS 15.0 Beta (24A5289h)
Executing Script in CodeRunner Version 4.3 (67365)

Fixed in https://github.com/swiftDialog/swiftDialog/releases/tag/v2.5.1


Draftkings settings.py should require teams from 2 games for most sports

Unless I'm mistaken, one of the key lineup rules for classic DK is not being implemented in settings. For all classic lineups in team sports (NFL, NBA, MLB, NHL, LOL, etc - only excluding TEN, NASCAR, Golf) there must be players from at least 2 games. This is different from Fanduel, where there must be players from 3 teams in most sports. Meaning you could have players from only 2 teams in your lineup as long as those 2 teams aren't playing each other.
A (bad) workaround that I've been using is setting min_teams = 3 but that becomes particularly bad when you have to use stacks of 4 and 3 in a lineup of size 7 for LOL to have any shot of winning the tournament.

Fixed DK settings in version 3.3.0


Migrate Teleport to docker

TODO

[ ] Backup existing data from Teleport
[ ] Remove existing teleport installation
[ ] Setup docker-compose file with Teleport
[ ] Import backup data
[ ] Make sure docker-compose file works with Renovatebot

When running Teleport in docker the underlying node, provisioner, still needs to run some Teleport software to be able to SSH into the provisioner node. Therefore there's no real benefit / migrating teleport to docker just makes things more complicated.


Update ru-RU.json

🇷🇺 Changes have been made to the Russian version (the text is corrected)
I tried to shorten some words

macOS Big Sur
Version 11.6 (20G165)
__
Symbol Wallet Version 1.0.6 (1.0.6)
@AnthonyLaw

@klimgeran could you please resolve conflicts?

@klimgeran could you please resolve conflicts?

I closed this request as there is an updated extensive pull request here
https://github.com/symbol/desktop-wallet/pull/1791


feat: change env setup

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

kj


Webpack 2.x cannot extract the emmet package (while Webpack 1.x is fine with the same package)

Do you want to request a feature or report a bug?
I believe this is a bug.

What is the current behavior?
Webpack 2.x refuses to extract a package while Webpack 1.x is completely fine extracting the same package.
Webpack 2.5.1 gave me the following error:
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted
If the current behavior is a bug, please provide the steps to reproduce.
A file with require('emmet') will be processed normally with Webpack 1.x but it will give the error above with Webpack 2.x

What is the expected behavior?
Webpack 2.x should extract the emmet library fine just like Webpack 1.x does. There is nothing wrong with how the require function is used in the emmet package.
If this is a feature request, what is motivation or use case for changing the behavior?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
I have tested this with Node 7.9, Webpack 1.15.0, and Webpack 2.5.1

Check the line referenced by the error.

This is what Emmet is doing for each file:
if (typeof module === 'object' && typeof define !== 'function') {
  var define = function (factory) {
    module.exports = factory(require, exports, module);
  };
}

define(function(require, exports, module) {
  // ...
  return {
    // the exported object
  };
});

This is still an issue on Webpack 3.0


Expose the underlying TLS session

I think the method should be called crypto_session(), but otherwise I'm okay with this.

@zeroqn would this also satisfy your use case?

Yes. Thanks.


cuda runtime error (209) : no kernel image is available for execution on the device

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug

RuntimeError                              Traceback (most recent call last)
 in ()
----> 1 train_detector(model, datasets, cfg, distributed=distributed, validate=validate, timestamp=timestamp, meta=meta)
17 frames
/content/gdrive/My Drive/Developer/Repositories/mmdetection/mmdet/ops/sigmoid_focal_loss/sigmoid_focal_loss.py in forward(ctx, input, target, gamma, alpha)
17
18         loss = sigmoid_focal_loss_ext.forward(input, target, num_classes,
---> 19                                               gamma, alpha)
20         return loss
21

RuntimeError: cuda runtime error (209) : no kernel image is available for execution on the device at mmdet/ops/sigmoid_focal_loss/src/cuda/sigmoid_focal_loss_cuda.cu:128

Reproduction
train_detector(model, datasets, cfg, distributed=distributed, validate=validate, timestamp=timestamp, meta=meta)

Did you make any modifications on the code or config? Did you understand what you have modified?
Just added a new dataset in the config.

What dataset did you use?
I am using a custom dataset, similar to PASCAL VOC format.

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment infomation and paste it here.

sys.platform: linux
Python: 3.6.9 (default, Nov  7 2019, 10:44:02) [GCC 8.3.0]
CUDA available: True
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 10.1, V10.1.243
GPU 0: Tesla K80
GCC: gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
PyTorch: 1.5.0+cu101
PyTorch compiling details: PyTorch built with:

GCC 7.3
C++ Version: 201402
Intel(R) Math Kernel Library Version 2019.0.5 Product Build 20190808 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v0.21.1 (Git Hash 7d2fd500bc78936d1d648ca713b901012f470dbc)
OpenMP 201511 (a.k.a. OpenMP 4.5)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 10.1
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_37,code=compute_37
CuDNN 7.6.3
Magma 2.5.2
Build settings: BLAS=MKL, BUILD_TYPE=Release, CXX_FLAGS= -Wno-deprecated -fvisibility-inlines-hidden -fopenmp -DNDEBUG -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DUSE_INTERNAL_THREADPOOL_IMPL -O2 -fPIC -Wno-narrowing -Wall -Wextra -Werror=return-type -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wno-sign-compare -Wno-unused-parameter -Wno-unused-variable -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Wno-stringop-overflow, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, USE_CUDA=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_STATIC_DISPATCH=OFF,

TorchVision: 0.6.0+cu101
OpenCV: 4.2.0
MMCV: 0.5.0
MMDetection: 2.0.0+5a0e55a
MMDetection Compiler: GCC 7.5
MMDetection CUDA Compiler: 10.1

You may add addition that may be helpful for locating the problem:
I am using Google Colab with the default environment

Error traceback
I am wondering if the error arises from the following. When I look at the /usr/local/ directory, I find three folders named: (a) cuda, (b) cuda-10.0, (c) cuda-10.1
Looking into each of them, there exist a /bin/nvcc directory in each.
Could these multiple versions of cuda be the source of error?
If so, how do I fix it?
Bug fix

https://github.com/pytorch/pytorch/issues/31285

I think the issue is probably with focal loss, as the error also suggests:

RuntimeError: cuda runtime error (209) : no kernel image is available for execution on the device at mmdet/ops/sigmoid_focal_loss/src/cuda/sigmoid_focal_loss_cuda.cu:128

Because when I change the model from retinanet to ssd, it works good

It's because your GPU is too old and not supported by the latest pytorch. You may refer to the issue above.

@hellock By saying "GPU is too old", do you mean the cuda version running on the gpu? But I was able to run detectron2 with the same gpu ( pytorch 1.5, cuda 10.1).

@madiltalay Downgrade pytorch to 1.4 and it will work !!

This might sound completely useless but try removing build and running setup.py.
It worked for me.


gutter option for table.hline()

Description
In good old booktabs manner, I would love to see a gutter (or the like) option for table.hline(), so table.hline() can introduce an additional vertical space around the line.
I'm currently achieving something similar with a rather hacky workaround:
#let hline(columns) = {
  return (
    table.cell(colspan: columns, inset: 1.5pt)[],
    table.hline(),
    table.cell(colspan: columns, inset: 1.5pt)[],
  )
}

so I can replace table.hline() by ..hline(4) (if the table has 4 columns) to add vertical space around the line.
Use Case
The booktabs LaTeX package has some spacing around horizontal lines (\midrule) within the table by default. A gutter option for table.hline() would allow template authors to style tables similar to booktabs in their templates (by setting a default value for the gutter parameter). :slightly_smiling_face:

How would table.hline’s gutter interact with the table’s gutter setting?

@bluebear94 Good point! Intuitively, I would assume they would add up. At least for the booktabs-style usage of table.hline(), that would be quite convenient. Alternatively, maybe the larger gutter value of both should be used? WDYT?

I agree that adding the values would make the most sense.


Add hover state to table and list view

(more details to come re: hover state for list and table view)
Note: but it needs to be restricted to only non-touch devices.
https://www.prowebdesign.ro/how-to-deal-with-hover-on-touch-screen-devices/

closing this one since it's redundant with #2370 and #2371


Small fix of zk rows for chunking

This is part of the investigation of the chunking bug in Pickles and o1js (RFCs here and here).
This branch is based on this branch by @Trivo25.
Explain your changes:

Discussions taking place in the o1js RFC PR suggest that the number of zero knowledge rows was not being correctly propagated to o1js. I started inspecting where the zk rows parameter was coming from and found a few places were it was not being correctly used.
in kimchi_bindings/wasm/src/plonk_verifier_index.rs there were two places where a hardcoded 3 was still being used to initialize w and the permutation polynomial. Instead, the updated zk_rows should be used:

res.set(zk_w(domain, index.zk_rows)).unwrap();
res.set(permutation_vanishing_polynomial(domain, index.zk_rows)).unwrap();

in src/lib/pickles/compile.ml and src/lib/pickles/step_branch_data.ml, the number of rows was not computed correctly.

I modified the calculation to become  ((2 * (permuts + 1) * num_chunks) - 2 + permuts) / permuts instead.
According to the chunking RFC and Kimchi,

   zk_rows = (16 * (domain_size / max_poly_size) + 5) / 7

Substituted hardcoded occurrences of num_chunks and zk_rows with a constant default value for ease of debug (merged from this PR)

Explain how you tested your changes:

Before these changes were performed, the chunk4.ml used to fail with the error  (Pickles.verify dlog_check).

The fix in the verifier index solves that error.
Nonetheless, the o1js tests still complain that there are not enough random rows to achieve zero-knowledge (expected: 4, got: 3). This suggests some miscalculation in the bindings that I still haven't found.

The second fix does not have an impact in all scenarios, but only in somewhat specific situations where the old formula and the new one return different values. An example is when num_chunks = 11, but this si not a power of 2. Another one is when num_chunks = 4 (basically anytime (16*c+6)/7 gives an integer number). There, the old computation gives 10 rows whereas this one gives 9. Having more rows is not a problem for zero knowledge-ness itself (it would be the other way round though). But this mismatch between the Kimchi side and the Pickles side can be a source of problems. Thus, it is better if we use the same formula on both sides. This, prevents us from getting another (Pickles.verify dlog_check) error.

TODO:

Find further places in the codebase where a hardcoded 3 might be used to instantiate variables instead of a proper variable default_zk_rows or similar.
Look into o1js-bindings and learn about the dependencies between libraries across our stack.

Partially addresses https://github.com/o1-labs/o1js/issues/1806
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

Can you redirect this PR against compatible? I think these changes shouldn't change our existing circuits, so these can go in the next soft-fork upgrade (and can thus also be included in an upcoming o1js release).

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me


Fix - TRSV2-615 - ROI timeout error

Codecov Report
Patch coverage: 51.07% and project coverage change: +0.07 :tada:

Comparison is base (841f0fb) 42.83% compared to head (c499390) 42.91%.

:exclamation: Current head c499390 differs from pull request most recent head 438602b. Consider uploading reports for the commit 438602b to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #336      +/-   ##
===========================================
+ Coverage    42.83%   42.91%   +0.07%
===========================================
  Files           97       97
  Lines         5862     5842      -20
  Branches       682      677       -5
===========================================
- Hits          2511     2507       -4
+ Misses        3351     3335      -16

Impacted Files
Coverage Δ

trade_remedies_public/cases/utils.py
11.39% <0.00%> (ø)

trade_remedies_public/config/constants.py
100.00% <ø> (ø)

trade_remedies_public/config/gunicorn.py
0.00% <0.00%> (ø)

trade_remedies_public/config/settings/bdd.py
0.00% <ø> (ø)

trade_remedies_public/config/settings/hardening.py
0.00% <0.00%> (ø)

trade_remedies_public/config/settings/local.py
0.00% <ø> (ø)

trade_remedies_public/config/settings/prod.py
0.00% <0.00%> (ø)

trade_remedies_public/config/settings/staging.py
0.00% <0.00%> (ø)

trade_remedies_public/config/settings/uat.py
0.00% <0.00%> (ø)

trade_remedies_public/config/wsgi.py
0.00% <0.00%> (ø)

... and 59 more

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