[js] An  Error: Not found: 'dart:js' occurs when creating a plug-in or package using a class with @JS.

Issue for js package. https://pub.dev/packages/js
An error occurs when creating a plug-in or package using a class with @JS.
There is a problem when exporting and using it.
There is no such error on the web, but it is only happening on mobile.
A similar issue was found.
https://github.com/flutter/flutter/issues/70373
I tried to apply a solution to the issue, but it didn't help. Did I make any mistakes?
Error
Invalid depfile: /Users/dooboolab/FlutterProjects/package_sample/example/.dart_tool/flutter_build/b72ff2e0b15f28c357cef8291e39cbdc/kernel_snapshot.d
: Error: Not found: 'dart:js'
../…/lib/js.dart:8
export 'dart:js' show allowInterop, allowInteropCaptureThis;
^

Unhandled exception:
FileSystemException(uri=org-dartlang-untranslatable-uri:dart%3Ajs; message=StandardFileSystem only supports file:* and data:* URIs)
#0      StandardFileSystem.entityForUri (package:front_end/src/api_prototype/standard_file_system.dart:34:7)
#1      asFileUri (package:vm/kernel_front_end.dart:654:37)
#2      writeDepfile (package:vm/kernel_front_end.dart:794:21)
<asynchronous suspension>
#3      FrontendCompiler.compile (package:frontend_server/frontend_server.dart:629:9)
<asynchronous suspension>
#4      starter (package:frontend_server/starter.dart:99:12)
<asynchronous suspension>
#5      main (file:///opt/s/w/ir/x/w/sdk/pkg/frontend_server/bin/frontend_server_starter.dart:13:14)
<asynchronous suspension>
4

FAILURE: Build failed with an exception.

* Where:
Script '/opt/homebrew/Caskroom/flutter/2.8.1/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 1165

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command '/opt/homebrew/Caskroom/flutter/2.8.1/flutter/bin/flutter'' finished with non-zero exit value 1

My code
import 'package:js/js.dart';

@JS("FlutterCrossCertWeb")
class FlutterCrossCertJs {
  external Future<String?> sign(String target);
}

export 'flutter_cross_cert_js.dart';

import 'package:flutter/material.dart';
import 'package:package_sample/flutter_cross_cert_js.dart';

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

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

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(home: MyHomePage());
  }
}

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

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final _crossCert = FlutterCrossCertJs();

  @override
  Widget build(BuildContext context) {
    return Container();
  }
}

Environment
js: ^0.6.4
flutter doctor -v
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 3.4.0-29.0.pre.89, on macOS 12.5.1 21G83 darwin-arm64, locale en-KR)
[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.2)
[✓] IntelliJ IDEA Ultimate Edition (version 2022.2.1)
[✓] VS Code (version 1.71.2)
⣻^C%
❯ flutter doctor -v

[✓] Flutter (Channel master, 3.4.0-29.0.pre.89, on macOS 12.5.1 21G83 darwin-arm64, locale en-KR)
    • Flutter version 3.4.0-29.0.pre.89 on channel master at /opt/homebrew/Caskroom/flutter/2.8.1/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2e0d6f100c (4 days ago), 2022-09-22 05:24:32 -0400
    • Engine revision 7a9f3c75cc
    • Dart version 2.19.0 (build 2.19.0-228.0.dev)
    • DevTools version 2.17.0

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/dooboolab/Library/Android/sdk
    • Platform android-33, build-tools 32.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14A309
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.71.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (3 available)
    • SM N960N (mobile) • 26bd083d61217ece • android-arm64  • Android 10 (API 29)
    • macOS (desktop)   • macos            • darwin-arm64   • macOS 12.5.1 21G83 darwin-arm64
    • Chrome (web)      • chrome           • web-javascript • Google Chrome 105.0.5195.125

[✓] HTTP Host Availability
    • All required HTTP hosts are available

You can't use dart:js on the native target. You can only use dart:js on the Web.

You can't use dart:js on the native target. You can only use dart:js on the Web.

Thank you for your reply.
How do I change to use the web only?
I think it will be easy to understand if you show me the example code.

I changed the code so that it can only be used on the web, but I'm not sure if this is the best way.
If I create a plug-in with the code below, there will be no error even if I create an instance of FlutterCrossscertWeb().
 Is there a better way?
javascript.dart
import 'package:js/js.dart';

@JS("FlutterCrossCertWeb")
class Javascript {
  external Future<String?> sign(String target);
}

js_interface.dart
import 'package:flutter_crosscert/models/sign_result_web.dart';

abstract class JsInterface {
  Future<SignResult?> sign(String target);
}

js_web.dart
// ignore: avoid_web_libraries_in_flutter
import 'dart:js_util';
import 'dart:convert';
import 'package:flutter_crosscert/models/sign_result_web.dart';
import 'package:flutter_crosscert/web/js_interface.dart';
import 'javascript.dart';

class Js implements JsInterface {
  final _js = Javascript();

  @override
  Future<SignResult?> sign(String target) async {
    final res = await promiseToFuture(
      _js.sign(target),
    );

    if (res == null) {
      return null;
    }

    return SignResult.fromJson(jsonDecode(res));
  }
}

js_stub.dart
import 'package:flutter_crosscert/models/sign_result_web.dart';
import 'package:flutter_crosscert/web/js_interface.dart';

class Js implements JsInterface {
  @override
  Future<SignResult?> sign(String target) => throw UnimplementedError();
}

flutter_crosscert_web
import 'package:flutter_crosscert/web/js_interface.dart';
import 'web/js_stub.dart' if (dart.library.js) 'js_web.dart'; // <-For web
import 'models/sign_result_web.dart';

class FlutterCrosscertWeb implements JsInterface {
  FlutterCrosscertWeb() : _js = Js();

  final Js _js;

  @override
  Future<SignResult?> sign(String target) => _js.sign(target);
}

I think you don't really need JsInterface but otherwise conditional imports like the code you shown are the only way to organise this.

I think you don't really need JsInterface but otherwise conditional imports like the code you shown are the only way to organise this.

I don't understand how calls can be made without JsInterface in the FlutterCrossscertWeb class. If you show me the example code, it will be easy to understand.
Thank you for your quick answer!

I don't understand how calls can be made without JsInterface in the FlutterCrossscertWeb class.

You are not declaring any variables of this type. You should just be able to remove this class completely and the code will still compile. No?

I don't understand how calls can be made without JsInterface in the FlutterCrossscertWeb class.

You are not declaring any variables of this type. You should just be able to remove this class completely and the code will still compile. No?

I understood what you said. The compiler works without the Js interface. I think it would be better to leave JsInterface to prevent my mistake, so I'm just going to leave it.
Thank you very much for your reply. It was a great help.


Update CombatEncounters to Foundry VTT 0.7.9

Which class should be updated?
CombatEncounters
For which version should the class be updated?
Foundry VTT 0.7.9

Ill take this one.


Added Stars component

Added a star component that takes in a rating prop and then rounds it to the nearest quarter before rendering it as graph of 5 stars filled according to the passed in rounded rating.
Trello Ticket
Zero Stars

Five Stars

One and a Quarter Stars

(Background color is temporary and will be changed)

 Stars Component


Cannot import v2 of the module

When trying to import v2 I get the following message:
require github.com/carlocayos/go-cod: version "v2.0.0" invalid: should be v0 or v1, not v2

Did a major version update which was causing the issue. I've just added a fix in release v2.1.0.
Try creating a new project and import version 2 v2.
import "github.com/carlocayos/go-cod/v2"

When you download the go mod dependencies, your go.mod file should be v2.1.0.
require github.com/carlocayos/go-cod/v2 v2.1.0

Thanks for the quick fix! 💯
If there are things I could help with, let me know! I would really like to contribute to this project :)


Use correct method as per documentation for restarting Daemon

Calling the restart daemon methods would would fail was because it's sending a PUT request but Forge API documentation states that it accepts only a POST request for that action.

Thanks :)


mysterious devicemapper device appears and causes high IO

I am running Docker 1.9.0 on Debian Jessie, I noticed that a mysterious device appears for a few minutes that causes high IO on the box and then it clears. The high IO impacts the containers.
I cannot trace where the device comes from but you can see from the screenshot below dm-12 util goes to 100%

I tried to do an ls -al /dev/mapper/ to trace down the device and what container it might belong to but it doesnt show up.
Containers: 9
Images: 614
Server Version: 1.9.0
Storage Driver: devicemapper
 Pool Name: docker-252:0-39459481-pool
 Pool Blocksize: 65.54 kB
 Base Device Size: 214.7 GB
 Backing Filesystem: extfs
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 15.46 GB
 Data Space Total: 214.7 GB
 Data Space Available: 199.3 GB
 Metadata Space Used: 37.91 MB
 Metadata Space Total: 4.295 GB
 Metadata Space Available: 4.257 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: false
 Deferred Deletion Enabled: false
 Deferred Deleted Device Count: 0
 Data loop file: /var/lib/docker/devicemapper/devicemapper/data
 Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.90 (2014-09-01)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.18.19
Operating System: Debian GNU/Linux 8 (jessie)
CPUs: 24
Total Memory: 125.8 GiB
Name: appdocker72-sjc1
ID: IYPE:XGV6:P67Q:MQ7V:7NWX:5ROV:6KWT:R2HA:JG5C:6RKQ:K7QP:TECP
WARNING: No swap limit support

Has anyone else experienced this?

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

I couldn't find anything similar to this issue.
BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version: 1.9.0
docker info:
Containers: 9
Images: 614
Server Version: 1.9.0
Storage Driver: devicemapper
Pool Name: docker-252:0-39459481-pool
Pool Blocksize: 65.54 kB
Base Device Size: 214.7 GB
Backing Filesystem: extfs
Data file: /dev/loop0
Metadata file: /dev/loop1
Data Space Used: 15.46 GB
Data Space Total: 214.7 GB
Data Space Available: 199.3 GB
Metadata Space Used: 37.91 MB
Metadata Space Total: 4.295 GB
Metadata Space Available: 4.257 GB
Udev Sync Supported: true
Deferred Removal Enabled: false
Deferred Deletion Enabled: false
Deferred Deleted Device Count: 0
Data loop file: /var/lib/docker/devicemapper/devicemapper/data
Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
Library Version: 1.02.90 (2014-09-01)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.18.19
Operating System: Debian GNU/Linux 8 (jessie)
CPUs: 24
Total Memory: 125.8 GiB
Name: appdocker72-sjc1
ID: IYPE:XGV6:P67Q:MQ7V:7NWX:5ROV:6KWT:R2HA:JG5C:6RKQ:K7QP:TECP
WARNING: No swap limit support
Provide additional environment details (AWS, VirtualBox, physical, etc.): Physical
List the steps to reproduce the issue:

This is random and haven't found a way to reproduce

Describe the results you received:
High IO due to a unknown device mapper device appearing (Looks to be started by Docker)
Describe the results you expected:
Provide additional info you think is important:

Hi, @dmyerscough . I've get the same problem with yours. Have you solve it?


SA1206 conflicts with IDE0036 for required properties

SA1206 wants required to come before the accessibility keyword. But IDE0036 wants the reverse.
https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/ide0036
Affects version: 1.2.0-beta.435

Duplicate of #3527


Update documentation to suggest running  'ldconfig' after installing C++ libs from RPMs

Is your enhancement request related to a problem? Please describe.
My team was trying to use the Node.js client for Pulsar.  We had installed the RPMs and the libs were present in /usr/lib, but our app still wouldn't start.  We kept getting the error that 'libpulsar.so: cannot open shared object file: No such file or directory'.  I found the solution after searching for Node.js C++ add-ons with the same issue and someone suggested that you need to run 'ldconfig'.  That turns out to have solved our problems.
Describe the solution you'd like
I would like to see that the documentation for installing the C++ libs from RPMs be updated to include a note that you might need to run 'ldconfig'.
Describe alternatives you've considered
There are no alternatives needed.
Additional context
N/A

@gimmeshelter many thanks for your info. Would you like to add docs accordingly? Then I can help review, thanks

Hi @gimmeshelter any feedback on this?

I am not a contributor and I don't really see the need for a pull request to add one line to your documentation.  That being said, I will do this if necessary to save someone else the headache we went through.


make open/closed fingers be part of pybullet_robots.py

closes #747

thanks! for good measure, try out the longrun tests in pybullet_blocks before merging

already did that!


More than 1 datepicker field results in multiple CSS and JS

I have a form with 6 different date fields. I assign the datepicker to each field, and they all work correctly.
However, where I place {{ form_stylesheet(edit_form) }} in my Twig template results in a  tag to the same CSS file 6 times (i.e. the number of datepicker fields on my form.
The same is true for the JS - it adds a SCRIPT tag for each datepicker field.
Surely these files should only be included once?

any news on this?

I included the js in my main template like this:
{% javascripts
'bundles/scdatetimepicker/js/bootstrap-datetimepicker.min.js'
%}

{% endjavascripts %}
and tried to override the jquery_layout.html.twig, but the bundle is not reading my jquery_layout.html.twig :(


Add reboot lock file to track daemon reload

Question (to @a-detiste, @dbent, maybe others): I think I understand this piece of logic (check for REBOOT_FILE and create it if /var is mounted, to avoid rerunning on-boot jobs when systemd is just reloading), but in case I got it wrong, please elaborate reasons behind this design decision. Thank you.

avoid rerunning on-boot jobs when systemd is just reloading

That's it, by re-using Vixie cron's lock file, it also ensure the on-boot jobs are not run on the moment cron is replaced by systemd-cron.
If /var is not available in early boot, it's better to run the on-boot jobs during the forced daemon-reload.
This was discussed on systemd-devel, everyone complained it was rather ugly, but there wasn't any better solutino found.
Most people are used to run daemons from on-boot job, but this is explicitely forbiden when using Type=oneshot; I guess these people should really think about writing native systemd units or move their stuff to /etc/rc.local. There is now also /etc/tmpfiles.d; so there remain little use cases for on-boot cron job.

Thank you for explanation, you answered my question in greater details than I could anticipate.


Gradients as full TensorBlocks

~This PR is built on top of #252.~
Fixes #96
Start the work for #219

This changes the core data structures to make gradients of a TensorBlock full tensor blocks, simplifying how we handle them and enabling storage of higher order derivatives.
This also required changing the serialization format to store gradients of gradients. There is a script to convert from the old to the new format, but users needs to be aware of it.
All operations currently throw a NotImplementedError if they encounter gradients of gradients, updating them is left to future PR.
TODO before merging:

[x] detect old format when loading files and point the user to the update script
[ ] update keys_to_samples/keys_to_properties to support gradients of gradients. This might be left to a future PR as well
[x] Integrate the new Rust API with rascaline and ensure everything is fine over there

:books: Documentation preview :books:: https://equistore--253.org.readthedocs.build/en/253/

One point we discussed with @frostedoyster: currently, building gradients of gradient needs to happen in this order:
grad_grad = TensorBlock(...)
grad = TensorBlock(...)
grad.add_gradient("positions", grad_grad)

block = TensorBlock(...)
block.add_gradient("positions", grad)

It would be nice to be able to write this code like this:
block = TensorBlock(...)
block.add_gradient("positions", TensorBlock(...))
block.gradient("positions").add_gradient("positions", TensorBlock(...))

The issue is that the return value of block.gradient("positions") is a view of a TensorBlock inside something else, and we currently do not differentiate between view of blocks inside a TensorMap (for which adding new gradient is invalid) and the view of a gradient inside a standalone block (for which adding gradient is valid).
I feel like making this difference should not be too hard, but could be left to a future PR.

I agree that the second example would be also nice to have but could be done in a future PR. I don't see this as blocker.

I started reading the PR but I am a bit confused by the new architecture.

The core of it is that a TensorBlock is now literally
struct TensorBlock {
    values: Array,
    samples: Labels,
    components: Vec<Labels>,
    properties: Labels,
    gradients: HashMap<String, TensorBlock>
}

The previous BasicBlock (source of some confusion) is now gone.
All the other changes are adapting to this, to add gradients to a block, retrieve gradients, and manipulate the corresponding data.

Yes, sorry about the documentation. I'll try to update it today

The current Rust API does not work for rascaline, since we are using the fact that samples and values are disjoint over there to borrow both of them in separate ways. I'll update it here

I think I've handled all the TODO from this PR, so is good to go for me!


WIP: Adding CRD validation

Signed-off-by: Santosh Kaluskar<EMAIL_ADDRESS>This WIP is a part fix to #2993.
Request a review from @pjbgf.

@Santosh1176 please run make generate manifests api-docs to generate the CRD YAMLs, then run make verify, once verify no longer errors out, please commit all changes.


Dask has different prediction results between single GPU and multi GPUs

Hi, I’m running the multi GPUs script from https://github.com/dmlc/xgboost/blob/master/demo/dask/sklearn_gpu_training.py.
I used my customized dataset and replace the X and y in the file, and run it with a single GPU and two GPUs. However, I got different results when print(prediction.compute()).
 Looks like the histories are matched but the predictions are totally different. Could anyone help me with this why the prediction results are totally different. Thanks!

Could you please share your version of xgboost?

Also, could you please split up training and testing into 2 phrases?  The trained model is different when you have different number of workers.  But after having the exact same model, the prediction should be same.

Thanks for the response! I used 1.0.0rc1 version (I will test the latest version too).
I think I find where is the bug but don't know which part in dask causes it. This numerical error in the inference stage is more serious when the data size is pretty large, e.g., m = 10000000. Here is how I modified the code after the training:
regressor.fit(X, y, eval_set=[(X, y)]) # with one GPU or two GPUs
prediction1 = regressor.predict(X)

bst = regressor.get_booster()
prediction2 = bst.predict(xgb.DMatrix(X))

With only one GPU, prediction1 and prediction2 are exactly matched. With two GPUs, they are totally different. I also tested prediction2 is more reasonable, so prediction1 with regressor.predict function is totally wrong.  Not sure what causes this.

could you please split up training and testing into 2 phrases?

Do you mean to save the model param after the training and load it with just one work for inference? The question is the trained models may very different with a single GPU vs multi GPUs. How does it guarantee that they are exactly the same, even if just using one worker for prediction.

Please update to latest xgboost 1.4.2.  The dask interface has improved a lot since the initial introduction.

Do you mean to save the model param after the training and load it with just one work for inference?

You can use as many workers as you like, the result is the same assuming the model is the same and you are using latest xgboost.  The tricky part is, the model returned by training with 1 GPU is different from training with 2 GPUs.  See below.

The question is the trained models may very different with a single GPU vs multi GPUs. How does it guarantee that they are exactly the same, even if just using one worker for prediction?

It's only the problem with training, inference and output from any other feature are guaranteed to be independent to number of workers.  For training, we need to run gk-sketching to get the quantiles from data to build the gradient histogram, sketching results from different workers are merged. This is an approximation algorithm, so different number of workers generates different approximation results.

Thanks! I will update to 1.4.2 and run it again. Will give you update soon.

Hi @cookielee77 any update?

Hi, sorry for the delay on this. We are upgrading the whole python environment. Once we did this, I will test this right away, hopefully get back to you in two weeks.
Thanks again!

I just tested with 1.4.2 version, the previous issue doesn't exist. Thanks for the help!


add on doesnt seem to adhere to backup times

the addon seems to run even outside of set backup times of 1-5

Duplicate #145
Please check other issue and add your logs on the other issue, showing what hours the addon says it will backup in.


Add Cursor Below / Above should not expand folded lines (or at least it should be optional !)

Does this issue occur when all extensions are disabled? Yes

VS Code Version: 1.63.2
OS Version: Windows 10.0.19044 x64

Steps to Reproduce:

Collapse some lines

Use "Add Cursor Below / Above" shortcut (Ctrl+Alt+DownArrow / UpArrow by default) to add multiple cursors below

Expected Behavior :
The cursor should be added after the first collapsed block, right before the next one (line 54 in above example)
Actual Behavior :
The cursor causes the block to expand in order to add the cursor one line below (line 12 in above example)

Feature request for 'Add Cursor Below / Above'..

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


ci: Add CI with GitHub Actions

This PR adds CI using GitHub Actions. The workflow that is added runs the pytest test suite (yay for pytest!) on both push events and pull_request events on Linux and MacOS environments across the Python 3.x runtime spectrum. The CI will also run as a nightly CRON job at 0:01 UTC. The status of the CI on the master branch is reflected publicly by adding a status badge to the README (if viewed now it won't render as there haven't been any runs on FaceBook's project) that will look like the following:

To achieve this PR two changes are made to setup.py (one required and one suggested):

The addition of cytoolz to the tests extra for the test suite to not error as the result of dask
The addition of a complete extra (with some rearrangement) which allows for installing the full environment with

python -m pip install .[complete]

which is both easier for developers and for making sure the full environment works in CI.
* Add 'complete' extra for easier testing setup
* Add cytoolz to 'tests' extra
* Add CI with GitHub Actions
   - Add CI badge to README

Also, awesome to see this open sourced! Congratulations on doing so!

@mickypaganini @jzf2101 This is ready for review. If it is interesting to you please let me know if you have any requested changes.

Woah thanks so much @matthewfeickert! Happy to have this 🙂


Improve display speed and smoothness

Hi there,
Thank you very much for your nice library !! :)
Can you please explain me how i could speed up display of the following pics, my SVG files seem too big, but maybe i do't use your lib correctly : http://www.maxunger.fr/
It's particularly slow on mobile devices (tested on iphone 5)
Thank you very much

Hi,
Thanks for these kind words :)
However, I no longer accept issues without a JSfiddle to check the problem. It's a requirement. If you have a question about how to use Vivus, you better join the conversation on Gitter :)


Accumulators for intra-block book-keeping

Overview
This PR adds a feature that is called "accumulators" in the code and I've also considered calling "scratchpads". An accumulator serves as a temporary place for Tuxedo pieces to do some accounting within a single block. The accumulator will be cleared automatically by the Tuxedo executive at the end of each block, so the stored data will not be available in the next block, or even after the block's execution closes.
Usecases

Tallying up the tips from each transaction so that the block author can claim them at the end of each block.
Making sure that inherents are set exactly once in each block.
No other concrete ones yet, but I suspect there will be.

Fit in Substrate
This feature is similar to https://github.com/paritytech/polkadot-sdk/issues/359 in some ways. However it is implemented in the Tuxedo core which is entirely in the runtime. That issue asks for host functions and performance which this will not provide. This also does not satisfy @burdges's request to have the data in-memory and skip scale encoding.
Drawbacks
This feature is pretty un-UTXO-y in the sense that we are basically providing (a strict structured interface around) storage items to the Tuxedo pieces. I've tried very hard to avoid storage items. I even went to great lengths to handle inherents in a UTXO-native way in #100 and I'm glad I did.
The thing that makes me mostly okay with this is that the storage is cleared at the end of each block, so the data does not persist. I think the term "scratchpad" emphasizes this well as it is a temporary place to write some notes while doing a job and it is not part of the final project.
I have not yet thought of another way to achieve giving tips to block authors, a feature that has existed even since bitcoin. Indeed, even in bitcoin there is a dedicated variable for tallying such tips.
Refactor and Features
@clangenb I remember you saying you like it when the refactor goes in first and the feature that motivated comes in another PR. That has stuck with me. Once I get this ironed out, I plan to separate the core feature from the way I'm using it in the money piece, and put them in separately. Thanks for inspiring more discipline.

As an aside, I'd think "pallets" could work like this:
use spin::Mutex; // https://docs.rs/spin/latest/spin/index.html

struct PalletAcc { ... }

static MUTEX: Mutex<Option<PalletAcc>> = Mutex::new(None);

fn on_initialize() {
    let lock = MUTEX.try_lock().expect("Reentrancy bug!");
    *lock = Some(PalletAcc::new())
}

fn verify_tx() {
    // Setup validation
    let lock = MUTEX.try_lock().expect("Reentrancy bug!");
    match *lock {
        None => {
            // we're not inside execute_block so do full tx validation, abort if fails
        },
        Some(acc) => {
            // we're inside execute_block so use acc: &mut PalletAcc
        }.
    }
    drop(lock);
    // Finish validation, abort if fails
}

fn on_finalize() {
    let lock = MUTEX.try_lock().expect("Reentrancy bug!");
    let acc = lock.take()
    drop(lock);
    // Validate acc itself, abort if fails
}

We do not require the spin lock per se, since the runtime is single threaded, but whatever.

Oh crap, I haven't actually updated to executive to use the new accumulator logic. So there is still that part. But anyway this is coming together now, and some review on the design is very welcome.

@JoshOrndorff I hope this commit is helpful.
The From/Into implementation for ConstraintCheckingSuccess was not feasible, so I added a new method that does exactly that, please use that one instead of the ugly conversion in the proc macro

Thanks for the reviews everyone. I know this PR is pretty far along, but I thought of an alternative design for tallying up fees and giving them to the block author and I need to write it out so we can consider them. My biggest complaint about the accumulators was always that they were not very UTXO native and were more like storage items, and this alternative design solves that problem.
When users send transactions, they must ensure that the input coins exceed the value of the output coins. They also have the ability to protect those output coins with whatever verifiers they want. Typically it might require a signature from the intended recipient. Of course token senders can also specify outputs that are UpForGrabs and can be claimed by anyone. And this is the key to the alternate design.
When you send tokens, you explicitly specify a tip to the block author by putting UpForGrabs. Only tokens that are explicitly included in an output as UpForGrabs may be claimed by the block author. If the input value exceeds the output value, then the difference truly is burned.
Then block authors claim their reward by inserting a transaction at the end of the block that sweeps up all of those UpForGrabs coins that were created during the block. There is no risk that someone else will get to those coins first because the block author can simply drop any transactions that try to claim the UpForGrabs ones.
The implementation would roughly be that when authoring, the block author notes all the UpForGrabs coins in a temporary storage that is cleared before the block ends just like how we do the extrinsics now.
The key benefits are:

All the tip-related state is still in UTXOs, not in the accumulator.
Importing nodes don't need to know anything about the implementation of this feature. They only have to check the normal UTXO rules.

Okay, I'm going to close this one for now.
Block rewards can be done more elegantly by the alternate design described above, and enforcing the right number of inherent calls can be done in the offchain pre-checks.
If we ever decide we need it, we have the design done and can re-open this.


Performance metrics for from_textfile_batched()

Just added the from_textfile_batched() method, which is DaskStream compatible.
Some performance metrics (by varying batch-size as well) I gathered, after pulling in skmatti's cuDF integration:
Throughput w/ Dask & w/ cuDF aggregations, on Quadro P6000: 10MBps, as opposed to the simple from_textfile() which gave ~1MBps.
Throughput w/ Dask & w/ cuDF aggregations, on T4: 12.5MBps.

Does it make a difference how common line terminators are in the input?

Not sure I understand your question. The batches are just based on the delimiter and batch size specified.

I am thinking, more terminators, more individual messages in the non-batched version, so that would affect the throughput.

Yes, that is definitely true. The non-batched version simply emits a single element/message downstream, which lowers throughput (in terms of number of messages/sec). Ideally, when the message size is a little lower, emitting a batch of messages downstream helps.


More builtin code coverage

Codecov Report

Merging #192 into master will increase coverage by 0.42%.
The diff coverage is 94.89%.

@@            Coverage Diff             @@
##           master     #192      +/-   ##
==========================================
+ Coverage   56.52%   56.95%   +0.42%
==========================================
  Files          78       79       +1
  Lines       15777    15787      +10
==========================================
+ Hits         8918     8991      +73
+ Misses       5912     5864      -48
+ Partials      947      932      -15

Impacted Files
Coverage Δ

expr/builtins/builtins.go
56.74% <ø> (-0.74%)
:arrow_down:

value/value.go
38.43% <0%> (-0.66%)
:arrow_down:

expr/builtins/list_map.go
100% <100%> (ø)

Continue to review full report at Codecov.

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


Evaluate how a minimal engo API may look like

I'd like to evaluate which parts of engo make up the core of the engine. In particular, it would be interesting to find out whether we may arrive at a minimal API, from which common systems and components may be implemented as standalone packages, which would use composition, message-passing and prioritization of systems as provided by ecs and engo.
An initial evaluation has identified the six base components and six base systems, namely.

AnimationComponent
AudioComponent
CollisionComponent
MouseComponent
RenderComponent
SpaceComponent

and

AnimationSystem
AudioSystem
CameraSystem
CollisionSystem
MouseSystem
RenderSystem

I think it would be a show of strength if engo could be implemented as a set of decoupled packages using the principles behind ECS.
I am proposing that we evaluate the feasibility of extracting these systems and components out from engo and creating dedicated packages for them. It is too early to tell what effects this may have on the resulting API, and we may very well decide that a single monopackage is the way to go. Still, using this as a thought-experiment it would be interesting to see the dependencies between different parts of the engo engine brought into light by this evaluation.
An obvious place to start is to create a dedicated package for each component and system, but also one for the window handling, as engo may be used headless and a clean separation between the engine and its window handling would result in a much cleaner design.
What are your thoughts on this? Is it worth evaluating?

From commit mewpull/engo@131842414d833847eb2a89b2d992da4b372a3146.

Initial separation of engo into small packages.
This is still very preliminary work. Follow-up commits
will TODOs related to the split, and further decouple
packages from one another.
Updates EngoEngine/engo#245.

The core branch at https://github.com/mewpull/engo/tree/core is tracking preliminary work at evaluating what the minimal API of engo may look like, and whether it is possible to decouple components from one another.

A modified version of Dave Cheney's glyph command was used to visualize the dependencies between the engo packages of the experimental core branch. The modified version of glyph excludes standard library packages from the graphs, to focus on the interesting parts, namely the intricate dependencies between different parts of engo.
Including some preliminary results below.

Dependencies of the now minimal engo package and the recursive dependencies of its imports.

Dependencies of the now minimal engo package highlighted.

Dependencies of the new collision package.

The collision package is a perfect example of a decoupled package, which only use core parts of the engo API to provide a specific functionality (collision detection). Personally, I'd love it if more packages were decoupled in this manner, and only a selective few had the dependency graphs of the engo (and window) packages.
Well, this is still very preliminary work. Just wanted give a teaser which me may use as the basis for discussions.
To produce these dependency graphs, run the following.
[~]$ export GOPATH=/tmp/go
[~]$ mkdir -p $GOPATH/src/engo.io
[~]$ cd $GOPATH/src/engo.io
[/tmp/go/src/engo.io]$ git clone -b core https://github.com/mewpull/engo
[/tmp/go/src/engo.io]$ go get engo.io/engo/...
[/tmp/go/src/engo.io]$ go get github.com/mewspring/junk/glyph
[/tmp/go/src/engo.io]$ $GOPATH/bin/glyph

Then navigate to http://<IP_ADDRESS>:8080/chord/engo.io/engo, http://<IP_ADDRESS>:8080/chord/engo.io/engo/collision, etc. glyph comes with several visualization modes. I've mostly been using "chord" and "tree", but the others may be interesting as well.

Those are some nice graphs. How did you remove the dependencies of the camera / space?
Things like:

CameraSystem depends on SpaceComponent (as do a lot of other systems)
RenderSystem depends on CameraSystem (heavily)

This kinda looks pretty nice.
Not sure if I like what it does to components used by multiple systems (the SpaceComponent and RenderComponent are used quite often), but that's just a matter of taste, I guess.

This kinda looks pretty nice.

I also think it is worth looking into.

Not sure if I like what it does to components used by multiple systems (the SpaceComponent and RenderComponent are used quite often), but that's just a matter of taste, I guess.

After the evaluation finishes and we discuss the matter, there may very well be a set of base or core components which are deemed so essential that it doesn't make sense to decouple them from the general engo engine. However, some things like spritesheets or tmx support seems like obvious candidates, as these are not likely to be used in every game.
Lets keep this issue open as we continue our discussions, and see what the minimal API of engo may look like once we settle for an approach. My main wish is that the Systems and Components that do end up in the core engo engine are well motivated to be there.

To summarize, the current engo package will be split into two packages engo and core. The engo package will implement the minimal engine API upon which useful Systems and Components may be built.
Engo

Window handling (glfw, GopherJS, mobile backends)
Input handling (keyboard, mouse, game controllers)
Message passing/event handling

Core
Every System and Component implementation.

2D Rendering
Audio
Physics
...

For background, see the gitter discussion starting at https://gitter.im/EngoEngine/engo?at=571cb3277469496137b8aeed

FYI I don't want to proceed with this until we have closed #71. Rather scared of the consequences it would have for @otraore :D

I agree @paked :D

Just to keep notes in one place.
From https://gitter.im/EngoEngine/engo?at=571edc676e3ae55e37ea0e63

So, (a future) engo.io/audio implements audio playing support for multiple platforms (desktop, js, mobile). engo provides a tidy API for all the basic parts of the game engine, such as window handling, input and events, message passing, audio (and something else?). Then the core package would implement Systems and Components using the engo API.

I'd love for this to "wait" until PR's https://github.com/EngoEngine/engo/pull/274, https://github.com/EngoEngine/engo/pull/273 and https://github.com/EngoEngine/engo/pull/266 are merged. Those combined change about 2k lines of code throughout the project.
Do we also want to wait for https://github.com/EngoEngine/engo/pull/251 ? @paked @mewmew @otraore ?

Let's do a feature freeze while we make these changes. Basically get as
much merged in as possible, and then split core and engo.
On Sat, 7 May 2016 3:30 am Etienne Bruines<EMAIL_ADDRESS>wrote:

I'd love for this to "wait" until PR's #274
https://github.com/EngoEngine/engo/pull/274, #273
https://github.com/EngoEngine/engo/pull/273 and #266
https://github.com/EngoEngine/engo/pull/266 are merged. Those combined
change about 2k lines of code throughout the project.
Do we also want to wait for #251
https://github.com/EngoEngine/engo/pull/251 ? @paked
https://github.com/paked @mewmew https://github.com/mewmew @otraore
https://github.com/otraore ?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/EngoEngine/engo/issues/245#issuecomment-217507598

I'd love for this to "wait" until PR's #274, #273 and #266 are merged. Those combined change about 2k lines of code throughout the project.

Definitely. The split into engo and core will be affect almost every part of the engine, so having large PRs merged first makes a lot of sense.

Do we also want to wait for #251 ? @paked @mewmew @otraore ?

We could go either way with #251. As the current assets package is tied into basically everything of engo we may as well wait with #251 and do it at the same time as we do the split. At least, I think it may be easier to do both at the same time, but I may be convinced otherwise.

Let's do a feature freeze while we make these changes. Basically get as much merged in as possible, and then split core and engo.

Sounds like a plan.


[Analyzer] Add a way to add inherit properties to element in manifest instead of superclass

Should this be an RFC?

[X] This is not a substantial change

Which package is this a feature request for?
Analyzer (@lit-labs/analyzer)
Description
Components that inherit other Lit webcomponents don't inherit properties (but just set the superclass field). This breaks the compatibility with external tools.
See: https://github.com/break-stuff/cem-tools/issues/69
Alternatives and Workarounds
CEM analyzer works fine

We could possibly add an option for this (contributions accepted!) but tools really should read superclass and mixin properties.


CI/CD and solution patches

Hello @WaelHamze,
Do you have any ideas on how can we (or should we?) integrate solution patches feature in CI/CD pipeline? For now we use monolithic solution and it works well. It is also clear how to build a "hot fix" branch to deliver some urgent fixes to production. But as for now, our hot fix is just another version of the same monolithic solution so it requires several dozens of minutes to install (our async service is on fire).
The main question is how to have complete "main" version on one hand and be able to produce (and store somewhere in repo I guess?) the patches we want to build on the other hand?

We are using similar approach. We have config file in main folder of repository that contains list of CRM solutions. This can be updated in feature branches. Build triggered on merge to master is using this file to export solutions with Xrm.CI.Framework.


ERROR  File "pokecli.py", line 41, in

(PokemonGo-Bot) sh-3.2# ./run.sh
Traceback (most recent call last):
File "pokecli.py", line 41, in
from pokemongo_bot import PokemonGoBot, TreeConfigBuilder
File "/Users/lars/PokemonGo-Bot/pokemongo_bot/init.py", line 309
<<<<<<< HEAD
^
SyntaxError: invalid syntax
Fri Aug 12 18:44:13 CEST 2016 Pokebot  Stopped.
Press any button or wait 20 seconds to continue.
I get this whenever I do ./run.sh
whats wrong?

Merge conflict? Simple way is backup configs, delete all and clone again.

I deleted all, and reinstalled.
now I get this error
`2016-08-12 19:55:56,321 [       cli] [INFO] PokemonGO Bot v1.0
2016-08-12 19:55:56,434 [       cli] [INFO] Configuration initialized
2016-08-12 19:55:56,434 [pokemongo_bot.health_record.bot_event] [INFO] Health check is enabled. For more information:
2016-08-12 19:55:56,434 [pokemongo_bot.health_record.bot_event] [INFO] https://github.com/PokemonGoF/PokemonGo-Bot/tree/dev#analytics
2016-08-12 19:55:56,829 [requests.packages.urllib3.connectionpool] [INFO] Starting new HTTP connection (1): www.google-analytics.com
[19:55:57] Setting start location.Traceback (most recent call last):
File "pokecli.py", line 514, in
main()
File "pokecli.py", line 138, in main
report_summary(bot)
File "pokecli.py", line 147, in report_summary
metrics.capture_stats()
File "/Users/lars/PokemonGo-Bot/pokemongo_bot/metrics.py", line 84, in capture_stats
response_dict = request.call()
File "/Users/lars/PokemonGo-Bot/pokemongo_bot/api_wrapper.py", line 90, in call
if not self.can_call():
File "/Users/lars/PokemonGo-Bot/pokemongo_bot/api_wrapper.py", line 54, in can_call
raise NoPlayerPositionSetException()
pgoapi.exceptions.NoPlayerPositionSetException
2016-08-12 19:55:57,325 [sentry.errors] [ERROR] Sentry responded with an error: 'module' object has no attribute 'HTTPSHandler' (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
File "/Library/Python/2.7/site-packages/raven/transport/threaded.py", line 174, in send_sync
super(ThreadedHTTPTransport, self).send(data, headers)
File "/Library/Python/2.7/site-packages/raven/transport/http.py", line 47, in send
ca_certs=self.ca_certs,
File "/Library/Python/2.7/site-packages/raven/utils/http.py", line 44, in urlopen
class ValidHTTPSHandler(urllib2.HTTPSHandler):
AttributeError: 'module' object has no attribute 'HTTPSHandler'
2016-08-12 19:55:57,326 [sentry.errors.uncaught] [ERROR] [u'NoPlayerPositionSetException', u'  File "pokecli.py", line 514, in ', u'  File "pokecli.py", line 138, in main', u'  File "pokecli.py", line 147, in report_summary', u'  File "pokemongo_bot/metrics.py", line 84, in capture_stats', u'  File "pokemongo_bot/api_wrapper.py", line 90, in call', u'  File "pokemongo_bot/api_wrapper.py", line 54, in can_call']`

try "health_record": false in your config.

#3679


Chore move some mv case from feut to p0

Proposed changes
move some mv case from feut to p0
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run arm


Could not flash nucleo_f746zg

Describe the bug
It's not possible to flash nucleo_f746zg board
To Reproduce
Steps to reproduce the behavior:

mkdir build; cd build
cmake -GNinja -DBOARD=board\nucleo_f746zg
ninja flash
See error

/local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts/target/stm32f7x.cfg:155: Error: invalid command name "stm32f7x.dap"
in procedure 'script'
at file "embedded:startup.tcl", line 60
at file "/local/mcu/zephyr/zephyr-project/boards/arm/nucleo_f746zg/support/openocd.cfg", line 1
at file "/local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts/board/st_nucleo_f7.cfg", line 8
at file "/local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts/target/stm32f7x.cfg", line 155
ERROR: command exited with status 1: /local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/openocd -s /local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts -f /local/mcu/zephyr/zephyr-project/boards/arm/nucleo_f746zg/support/openocd.cfg -c init -c targets -c 'reset halt' -c 'flash write_image erase /local/mcu/zephyr/zephyr-project/samples/drivers/counter_rtc/outdir/nucleo_f746zg/zephyr/zephyr.bin 0x8000000' -c 'reset halt' -c 'verify_image /local/mcu/zephyr/zephyr-project/samples/drivers/counter_rtc/outdir/nucleo_f746zg/zephyr/zephyr.bin 0x8000000' -c 'reset run' -c shutdown

Expected behavior
Should flash.
Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)
Screenshots or console output
/local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts/target/stm32f7x.cfg:155: Error: invalid command name "stm32f7x.dap"
in procedure 'script'
at file "embedded:startup.tcl", line 60
at file "/local/mcu/zephyr/zephyr-project/boards/arm/nucleo_f746zg/support/openocd.cfg", line 1
at file "/local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts/board/st_nucleo_f7.cfg", line 8
at file "/local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts/target/stm32f7x.cfg", line 155
ERROR: command exited with status 1: /local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/bin/openocd -s /local/mcu/zephyr-sdk/sysroots/x86_64-pokysdk-linux/usr/share/openocd/scripts -f /local/mcu/zephyr/zephyr-project/boards/arm/nucleo_f746zg/support/openocd.cfg -c init -c targets -c 'reset halt' -c 'flash write_image erase /local/mcu/zephyr/zephyr-project/samples/drivers/counter_rtc/outdir/nucleo_f746zg/zephyr/zephyr.bin 0x8000000' -c 'reset halt' -c 'verify_image /local/mcu/zephyr/zephyr-project/samples/drivers/counter_rtc/outdir/nucleo_f746zg/zephyr/zephyr.bin 0x8000000' -c 'reset run' -c shutdown

Environment (please complete the following information):

OS: Linux
Toolchain Zephyr SDK
Commit SHA: 14084e99608bf6ce31a14c77219ea0047d2a88c9

Additional context
Workaround is to drag and drop .bin file in mounted disk

fixed in latest SDK


change blog post entries to ascending order

Please, how to change the order of posts to ascending (earliest post at top of each web page)

Please use https://discourse.gohugo.io/ for questions/troubleshooting. Also see  Hugo Documentation.


fix postgres idType config not honoured everywhere.

Description
I initially reported on "Internal error: error: type "serial" does not exist" when setting up payload 3 beta 99 with a posgres connection to xata.io in this discussion.
Later I noticed that the payload idType option for postgres was not respected for secondary tables like payload_preferences_rels. This PR patches this.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

I believe this would be a breaking change, you can't really simply migrate a table from numeric ID to UUID (If you already have idType: 'uuid' in the config)

@r1tsuu Thanks for your comment.
Full disclosure, I barely know anything about this project. Trying to set it up the first time made me run into this issue and I opened this PR early (in draft) to be able to get some feedback. The PR is not tested, I haven't added test cases and haven't documented the changes.
Any hints on how I can make this not a breaking change and accepted before v3 gets out of beta?
Simplest solution for my problem would be to abandon trying to fix idType='uuid' and invent a third idType, in the discussion @DanRibbens suggested adding identityColumn. Implementation would be very similar to the changes from this PR but with a separate and new function in order to not cause any breaking changes to uuid as an idType.
Relevant drizzle code that would replace serial: integer("id").primaryKey().generatedAlwaysAsIdentity().
Other option is to write a complex migration script, which I am currently not qualified to do.
What is the stance of the maintainers on this?


Screen corrupted upon program start

Hello,
Trying this project on STM32F429i-disco board.
Project compiles without errors on STm32CubeIDE, but displays a corrupted main screen. Trying to touch different areas results in nothing. I think the process is hung somewhere.
The "standard" ST demo project loads and runs correctly, so I guess my hardware is OK.
Any ideas?
-Gideon

Please remove line 47 from main.c
https://github.com/lvgl/lv_port_stm32f429_disco/blob/master/src/main.c#L47

Not sure how that ended up there - I've removed the extra line in https://github.com/lvgl/lv_port_stm32f429_disco/commit/4ee7d4bf603fa7c7141281b0eb4a17ef54f34118. I'm not sure if two demos  would cause graphical artifacts, but the second line definitely doesn't belong.


Unquoting doesn't work inside functors

This seems to involve names inside of functors.
Require Import Template.Loader.
Require Import String.
Require Import Template.Ast.

Definition x : nat := 3.
Quote Definition _t := (x = x).
Run TemplateProgram (tmBind (tmUnquote _t) (fun x => tmPrint x)). (* works! *)

Module Type M.
  Parameter x : nat.
End M.

Module V (x : M).
  Quote Definition _t := (x.x = x.x).

  Run TemplateProgram (tmBind (tmUnquote _t) (fun x => tmPrint x)).
  (* Toplevel input, characters 0-65: *)
  (* > Run TemplateProgram (tmBind (tmUnquote _t) (fun x => tmPrint x)). *)
  (* > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ *)
  (* Error: Constant not found : Top.x.x *)

I spent some time digging into this. The issue seems to be with the Top. If you remove the Top. from the beginning of the string then everything goes through. Of course, we aren't the ones adding the Top., that is being done by Coq, so it seems like the functions that we are using for quoting and denoting are not inverses.

Should be solved by #418. Reopen if not


Craft 2: Calling console/yiic fails when using environment variables for db.php

Description
This is a probably a bit of unique case, but for Craft 2 calling console/yiic under Nitro doesn't work, as it cannot connect to the database. I am using environment variables in the config/db.php but I don't believe the environment variables are available to the console request, as all the db parameters are blank. If I hard code this in the db.php it works:
'console' => [
   'server' => 'mysql-5.6-3307.database.nitro',
   'database' => 'database_name',
   'user' => 'nitro',
  'password' => 'nitro'
]

The environment variables do work for web requests however.
Is there anything that can be done, other than hard coding?

@jamesmacwhite maybe its worth looking at the code where Craft calls dotenv and adding it to the console file that loads PHP? Its been a really long time since I looked at Craft 2 but I will try to dig up an installation and ask someone on the team if they have the same issue.

@jasonmccallister Ah, having a minor brain failure. Yes, I can just include the vendor and DotEnv load in my console/yiic before it's called. I assume previously the env vars have been available without this, but because of the Docker container being split, the env vars aren't set in the PHP container.
Don't worry, I think I have a fix, thanks!


Plex code cleanup

Refactored Plex code.
Fixed almost all glitches on Plex clients. (May increse the startup time of JMMServer the first time)
Added experimental UPnP to the FileServer. Experimental UPnP requires changes in the plex plugin, but, may fix (Remote Clients may be unable to play video, if the video do not requires transcoding. PlexPlugin Issue #9) if the router supports Hairpin NAT.
Fixed subtitle/audiostreams languages name in plex (somehow visual studio dosn't like kana strings).
Fixed Plex Plugin does not display info text about the episode correctly. PlexPlugin Issue #8
May fix Plex for XBMC has thumbnails broken. PlexPlugin #2. (Require testing).
Fixed unicode characters in hasher.dll pinvoke

I think the new files are missing in the commit. At least all the files in the plex folder and the UPnP.cs in the FileServer folder is nowhere to be found.

strange, only changes got commited... Fixed in the new pull...
On Thu, Feb 12, 2015 at 8:46 PM, miracle152004<EMAIL_ADDRESS>wrote:

I think the new files are missing in the commit. At least all the files in
the plex folder and the UPnP.cs in the FileServer folder is nowhere to be
found.
—
Reply to this email directly or view it on GitHub
https://github.com/japanesemediamanager/jmmserver/pull/128#issuecomment-74170205
.


1px top misalignment with "segment" and "card".

Here is the reproduction of the problem: https://plnkr.co/edit/SKSo5rEGLYMetnC5vSF7?p=preview
Here a picture:

Is the card too high or the segment too low?

+1


fix: wrong node path for android interstitial id

This help the saved ID display correctly when reload the plugin

Merged! Thank you for help to improve the repository!


Close response body correctly and optimize base58 decode

Problem

Request body was not being closed properly for non-200 responses.
Unneeded map usage and allocations in base58 package.
client.Ping does not close tcp connections (no fix) -- should be addressed. Will cause too many open files and possible memory leak.

Solution

Move defer response.Body.Close() before status code. Better tests should be added, but this fixes a memory leak.
Couple of changes here:

convert defaultAlphabet to a hash table -- safe to be read concurrently, no need to copy map for each key.
use only slice of bytes instead of bytes.Buffer for return payload

No change. A solution should be planned

Before:
go test -bench=. -benchmem -cpu 1,2,4
BenchmarkBase58           100000             20614 ns/op            6128 B/op         86 allocs/op
BenchmarkBase58-2         100000             19076 ns/op            6128 B/op         86 allocs/op
BenchmarkBase58-4         100000             19707 ns/op            6127 B/op         86 allocs/op

After:
go test -bench=. -benchmem -cpu 1,2,4
BenchmarkBase58           200000              7300 ns/op            2784 B/op         70 allocs/op
BenchmarkBase58-2         200000              6725 ns/op            2784 B/op         70 allocs/op
BenchmarkBase58-4         200000              6834 ns/op            2784 B/op         70 allocs/op

Remember

Bump VERSION
Update CHANGELOG.md
Update README.md
Write tests

Thanks for the PR @eramus! Checking it out now, sorry about the delay.

@eramus I added a hotfix for the .Ping() close issue with #9. Thanks for spotting it!


#525 Background color for different sections

Related Issue

#525

Proposed Changes

Changed the background colors for 2 sections in the styles
Changed the background colors for 2 sections for light mode as well

Additional Info

The screenshot does not show the UI properly, Recommend to run on local computer.
These colors for the section looked suitable. If required to change pl let me know.

Checklist

[x] ✅ My code follows the code style of this project.
[x] 📝 My change requires a change to the documentation.
[x] 🎀 I have updated the documentation accordingly.
[x] 🌟 ed the repo

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Summary
Before the background color as just one as did not look as much it should have. Changed the color to suit the theme of the website and did the same for the light version as well. Looks much better when the website is opened in local system than the screenshot that I have submitted. I recommend to run on local computer once if possible, to check out the UI.

Output Screenshots

Screenshot #1
Screenshot #2

Dark Mode
Light Mode

please fetch latest code


[JTC] set_hold_position() doesn't actually stop the robot

Describe the bug
Upon aborting a goal or some other unrecoverable error, trajectory execution is stopped in the realtime loop by calling set_hold_position(), which sets an empty trajectory. This doesn't actually stop the robot. Instead, the empty trajectory does not have any valid point, so nothing at all is commanded to the hardware interface. This means that the last-commanded hardware state remains in place, which has no guarantee of actually stopping the robot.
To Reproduce

Start up JTC with e.g. velocity or acceleration control.
Start any trajectory.
Cancel the trajectory goal mid-execution so the trajectory is aborted
Observe that the robot continues to execute the last-commanded velocity or acceleration or effort.

This issue isn't visible with position-only control.
Expected behavior
We should handle this like a transition to the deactivate lifecycle state. Namely: set_hold_position() should echo the current position back to the robot, and velocity/acceleration should be commanded to 0.
I'll submit a PR to this effect.
Screenshots
N/A
But I can provide video of our physical robot (a modified Kinova JACO2) if necessary.
Environment (please complete the following information)

OS: Ubuntu 22.04 Jammy
ROS Version Humble
With current package ros-humble-joint-trajectory-controller v2.20.0-1jammy.20230522.072811

Additional Context
Note the existing TODO with regards to effort interfaces, it is unclear how to stop an effort-only interface in a robot-agnostic way.

Duplicate of #514? At least the same behavior causing both issues IMHO.


Can make a hospital without a name: requires validation

A new hospital can be created without specifying a name, as shown. Notice that this newly created hospital does can not be accessed/viewed/edited either, since the url was part of the name:

Fix is to add validation to name field. Should probably add validation to some other fields as well.

Fixed by adding the following to hospital/forms.py.


Neo-tree Rename doesn't work

As of current, Netman has no concept of "move" or "rename" and thus there is no safe way to implement this. That is a complete oversight on my part in designing the API and I am working through a respec of it right now to account for moving files. Until then, sorry.

So move (and to a lesser extent copy) are going to be tricky. Moving/copying within a single host is easy, I can just have the provider do that (if the provider allows for that).
But one of the nice things about a UI is that you can interact with multiple providers/hosts at the same time and why not extend Netman to interop with that very idea?
Considerations when implementing this

How do we "copy" a directory structure? In a perfect world, every provider has some implementation of tar but thats highly unlikely. Do we require an "archive" function? What about providers that don't care about that? How do we facilitate them?
How do we manage permissions? This will go in hand with allowing users to modify metadata on a remote resource but that hasn't been implemented yet. Do we just transfer the permissions across?
What do we do if the URI is a STREAM?  Should we only support moves of type EXPLORE or FILE? How do we notify the user that the requested move is not supported?

Etc. This may be a longer lived issue as I flesh these problems out.

Internal support (within the same host in the same provider) has now been added via commit f79f4c8 ](https://github.com/miversen33/netman.nvim/commit/f79f4c89f136ded97319c5c4c7a6226e8f0734a1) but this commit does not cover pretty much any of the above listed issues. It does however provide a semi stable experience when browsing within a host which is the first goal

Current headspace for this

A new required function move as used in docker and ssh
A new optional pair of functions archive_get and archive_put.

If the archive functions are not provided, netman api will assume that the provider can only handle moves within itself and prevent external moves to or from that provider. If the archive functions are available, netman api will allow inter provider moving.
This means that if ssh and docker implement the archive functions, you will be able to move files/directories between the 2.
To accomplish this, netman api will perform an archive_get on the first provider (at whatever URI is provided) and then an archive_put on the second provider (at whatever URI is provided). Netman api will allow the receiving provider to handle the archive in whatever way it needs. This should solve most of the above issues called out.
I will stub out the API changes tomorrow and see what it looks like in a POC on the v1.1 branch

So I have worked through what this will look like. I believe the best way to accomplish this feature is to have a provider implement an optional attribute (archive), with this attribute containing the following 3 functions

put
get
`schemes

The goal would be that the api will reach out to the target provider (the provider that the URIs will be moved to) in order to get the list of available compression schemes that the target can handle.
It will then group all the URIs to move by their provider. The api will then reach out to each groups provider, give that provider the list of URIs to archive and the available archive schemes that can be used. After each URI group has been processed, the api will reach out to the target provider and give it each archive (individually) as well as the scheme used for each archive. The target will then be responsible for uploading the archive to the target's host, and then extracting and placing the data where it needs to go.
In practice, this has been mocked out in this commit where I worked through how this would function in the API and docker. Subsequently it was implemented in the ssh provider
Note, there are no tests for this yet as I am still working through how the implementation will look. Tests are required for this kind of feature before I am willing to call it "ready for use" however.  Unfortunately, that means I had to completely rewrite the shell module as there was an async bug (see #88 for details), and then rewrite the docker  module as it was getting to be too much spaghetti code to maintain.
So the docker module is now shiny and new and has a nifty abstraction layer for interacting with a container (maybe using the unix socket in the future?)
I have to get ssh updated to utilize the new APIs I am providing as well, and then I have a whole :hankey: load of documentation I need to write :upside_down_face:
Progress is being made though :)

keep up the good work :D

Completed with #91 and merged into v1.1 :)


Support for unix socket?

It would be create to be able to do:
RestClient.get "unix:///tmp/named_socket.sock", { params: {some: "param"} }

And have RestClient use a named socket the same way it uses a url.
Is this something that is possible? - If not, could this please be added?

Any update on this issue? The fork of @kamerblauwlicht is also working for me.
There is also a small problem with cookies: https://github.com/sparklemotion/http-cookie/issues/7

I too would really like to see this merged and released. Is there anything (other than maintainer availability) preventing this from happening?

+1

+1

+1


[fix] gws schedule some issues

Coverage increased (+8.8%) to 92.857% when pulling 9c452682e39698139eed984191ef38f65c598eb8 on fix-gws-schedules into dc177f212100bcdc1ff700f2e097e4aa1d690a70 on master.


Chained expressions in top-level let clause

let x = 100 in x + 1;

let randCarthesian() =
    let maxX = random(0, 640) in
    let maxY = random(0, 480) in
    from x in 0 .. maxX
    from y in 0 .. maxY
    select (x, y)

An alternative to consider would be SQL's with clause:
let randCarthesian() =
    with maxX as random(0, 640)
    with maxY as random(0, 480)
    from x in 0 .. maxX
    from y in 0 .. maxY
    select (x, y)

will be fixed using the let ident = expr in expr syntax - it's closer to f# and the existing syntax.


Create security-file-token-provider to issue Vault tokens (6 of 6)

Implements an executable that reads a configuration file to
create per-service security tokens for EdgeX services.
Besides the executable, several modules are involved:

fileioperformer - an abstraction layer for file I/O used
to facilicitate unit testing

authtokenreader - a component that reads a Vault token
out of a JSON file; can be a service token or root token

vaultclient - a standalone component to talk to Vault
enhanced with methods for creating policies and tokens

tokenconfig - a component to process the config file
used by fileprovider to create service tokens

fileprovider - contains main business logic

security-file-token-provider - executable wrapper

Resolves #1677
This is part 6 of 6 merges for this feature.
Depends on

https://github.com/edgexfoundry/edgex-go/pull/1788
https://github.com/edgexfoundry/edgex-go/pull/1789
https://github.com/edgexfoundry/edgex-go/pull/1790
https://github.com/edgexfoundry/edgex-go/pull/1791
https://github.com/edgexfoundry/edgex-go/pull/1792

Signed-off-by: Bryon Nevis<EMAIL_ADDRESS>

cc @eno-intel

@michaelestrin Integerated common bootstrap package.
Implemented workaround for https://github.com/edgexfoundry/edgex-go/pull/1828 not being merged yet (need to send SIGTERM to self in order to quit)

This PR should be enhanced with chown/chgrp capabilities before being resubmitted for PR.
See https://github.com/edgexfoundry/blackbox-testing/issues/302


The Keeper does not kick the ball away or chip

If you put the ball in the defence are with the keeper running default strategy the keeper approaches the ball but cannot kick it away

Someone completely broke the getBall skill compared to it's old behavior.

So ballhandle posControl is sending 0 velocities.


Support UNLs with future effective dates: (UNL v2)

High Level Overview of Change

Creates a version 2 of the UNL file format allowing publishers to
pre-publish the next UNL while the current one is still valid.
Version 1 of the UNL file format is still valid and backward
compatible.
Also causes rippled to lock down if it has no valid UNLs, similar to
being amendment blocked, except reversible.
Resolves #3548
Resolves #3470

Context of Change
Internal Spec
Type of Change

[x] New feature (non-breaking change which adds functionality)

Test Plan
Set up an internal site similar to https://vl.ripple.com. Publish a version 2 UNL file on that site, as described in the spec. (Replace blob and signature with a blobs_v2 array containing one or more blob/signature pairs.) Include one or more VLs in the array containing future effective dates (using Ripple-epoch time value). Verify that nodes switch to the new VL at the appropriate time.
Future Tasks
Some relatively minor deviations and additions were made from the spec in the development process. I'll update the spec ASAP.

This change is

Rebased on to 1.7.0-b3

The node crashes for me when I provide validators under the validators config section.

@ximinez: please rebase this to the latest develop, resolving all conflicts and force-pushing here. Thanks.

I forgot to mention this yesterday, but that last push included both rebasing onto 1.7.0-b8 and the latest round of feedback from @seelabs.

Merged via 4b9d3ca7de7c5758fbd6ae774c74196fa6b4268c

While effective date is documented there is no notes on XRPL.org that you need to specify version as 2.

While effective date is documented there is no notes on XRPL.org that you need to specify version as 2.

Strictly speaking, I think a version 1 blob with an effective date would work fine. It would not be best practice because the current VL blob wouldn't be included with it.


How can I force sync using a command?

Hello,
I would like to know if there is a command telling your software to sync with Google Drive. Also, can I use this on a launcher? I am using XFCE 4.12 as my desktop environment.
Thanks

I'm sorry, but this is not a sync tool. It just lets you mount your Google Drive as a local filesystem, but you have to be online as there is no offline mode. Consider using https://github.com/odeke-em/drive or https://github.com/ncw/rclone.

Thank you for your fast response! I would like to add that it actually syncs the changes I made to a file, I just had to open the Google Drive folder.


Could not import Flash Attention enabled models: CUDA is not available - Could not import Mistral model: CUDA is not available

System Info
text-generation-inference: 1.3
cuda: 12.0
2x RTX 3090
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
model=meta-llama/Llama-2-7b-chat-hf
volume=$PWD/data # share a volume with the Docker container to avoid downloading weights every run
token=
docker run --gpus all --shm-size 1g -e HUGGING_FACE_HUB_TOKEN=$token -p 8080:80 -v $volume:/data ghcr.io/huggingface/text-generation-inference:1.3 --model-id $model
Expected behavior
text-generation-inference dont use GPUs

log:
2023-12-22T09:07:47.076618Z  INFO download: text_generation_launcher: Starting download process.
2023-12-22T09:07:55.371986Z  INFO text_generation_launcher: Files are already present on the host. Skipping download.
2023-12-22T09:07:56.187633Z  INFO download: text_generation_launcher: Successfully downloaded weights.
2023-12-22T09:07:56.187917Z  INFO shard-manager: text_generation_launcher: Starting shard rank=0
2023-12-22T09:08:03.760350Z  WARN text_generation_launcher: We're not using custom kernels.
2023-12-22T09:08:03.864304Z  WARN text_generation_launcher: Could not import Flash Attention enabled models: CUDA is not available
2023-12-22T09:08:03.868436Z  WARN text_generation_launcher: Could not import Mistral model: CUDA is not available
2023-12-22T09:08:03.870054Z  WARN text_generation_launcher: Could not import Mixtral model: CUDA is not available
2023-12-22T09:08:06.204069Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:08:16.220041Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:08:26.234653Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:08:36.251014Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:08:46.270059Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:08:56.287644Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:09:06.298710Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:09:16.316446Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:09:26.331553Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:09:36.351414Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:09:46.367035Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:09:56.382787Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:10:06.401441Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:10:16.429193Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:10:26.460837Z  INFO shard-manager: text_generation_launcher: Waiting for shard to be ready... rank=0
2023-12-22T09:10:28.095820Z  INFO text_generation_launcher: Server started at unix:///tmp/text-generation-server-0
2023-12-22T09:10:28.168326Z  INFO shard-manager: text_generation_launcher: Shard ready in<PHONE_NUMBER>67s rank=0
2023-12-22T09:10:28.232271Z  INFO text_generation_launcher: Starting Webserver
2023-12-22T09:10:33.017809Z  WARN text_generation_router: router/src/main.rs:349: --revision is not set
2023-12-22T09:10:33.017856Z  WARN text_generation_router: router/src/main.rs:350: We strongly advise to set it to a known supported commit.
2023-12-22T09:10:33.388356Z  INFO text_generation_router: router/src/main.rs:371: Serving revision d184db6c72949683168f8c374f0c9ccd7bf9a5bd of model anhnh2002/seallm-7b-chat-summary
2023-12-22T09:10:33.396069Z  INFO text_generation_router: router/src/main.rs:213: Warming up model
2023-12-22T09:12:20.918654Z  WARN text_generation_router: router/src/main.rs:224: Model does not support automatic max batch total tokens
2023-12-22T09:12:20.918718Z  INFO text_generation_router: router/src/main.rs:246: Setting max batch total tokens to 16000
2023-12-22T09:12:20.918749Z  INFO text_generation_router: router/src/main.rs:247: Connected
2023-12-22T09:12:20.918784Z  WARN text_generation_router: router/src/main.rs:252: Invalid hostname, defaulting to <IP_ADDRESS>

+1. I am also seeing same logs and GPU is not being used.

+1. I am also seeing same logs and GPU is not being used.
i ve solved this issue by replace "1.3" to "latest"
`model=meta-llama/Llama-2-7b-chat-hf
volume=$PWD/data # share a volume with the Docker container to avoid downloading weights every run
token=

docker run --gpus all --shm-size 1g -e HUGGING_FACE_HUB_TOKEN=$token -p 8080:80 -v $volume:/data ghcr.io/huggingface/text-generation-inference:1.3 --model-id $modeltomodel=meta-llama/Llama-2-7b-chat-hf
volume=$PWD/data # share a volume with the Docker container to avoid downloading weights every run
token=
docker run --gpus all --shm-size 1g -e HUGGING_FACE_HUB_TOKEN=$token -p 8080:80 -v $volume:/data ghcr.io/huggingface/text-generation-inference:latest --model-id $model`

+1. I am also seeing same logs and GPU is not being used.

if you want to use multiple gpus, add --num-shard <n_of_gpus> following docker run command


Window Resize Issue

Hi,
I think I love this little plugin, however, I'm having the same issues with resizing the browser window that I've seen a few posts about. The columns don't adapt as the screen is pulled towards a narrow, mobile-like view. Here is my site in staging: http://www.hsstudio.ca/staging/team/
Hoping you can help me figure out why I can't get it to respond... I'll buy you a beer!

I think the problem is with

<link href="http://fonts.googleapis.com/css?family=Lato:400,100,100italic,300,300italic,400italic,700,700italic" rel="stylesheet" type="text/css">

Salvattore scans for all the css and media queries and adds listener if they have data-colmns::beofe on the element.
When scanning happens for this file, it gets undefined for length of css rules and failed to execute hence no listener is added.
Looks like you are using 1.0.5 version and this problem is resolved in latest version. Can you try with latest version and see if the problem gone?
Cheers..


Broken Link

In the opening sentence: This sample shows how to send an email using SendEmailRequest message. You can download the sample from here.
The download the sample from here link needs to be updated (it's missing the d in SendEmail, currently reads C%23/SenEmail)

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

ID: 538f3a1a-423a-8b79-3eea-69edb9f5c65f
Version Independent ID: 0dcb5000-3722-8894-7135-36300f8bce06
Content: Sample: Send an email (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/org-service/samples/send-email.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @JimDaly
Microsoft Alias: jdaly

Thanks for your comment, @gjbundy. This issue has been assigned to the doc author for review.

@gjbundy
Thank you for reporting this. I've fixed it.
Change should be applied around 5PM PST.


Visualize feature maps from the contrastive encoder

Added a script to perform PCA along the channel dimension to visualize multi-scale feature maps in the contrastive model's encoder stages.

Not urgent to merge at this moment.

@ziw-liu, feature visualization vs. time didn't serve the purpose in early iterations. Did you also try it after we started training time-aware models? The temporal smoothness in the output layer does not enforce the temporal smoothness in intermediate layers. This is interesting.


Add queue insertion information to statistics.

We have seen on occasion in the wild instances where the gen_bunny erlang message queue for the indexing queue gets backed up, and requests fail due to timeouts. This can be a bit of a pain to for users to debug.
This adds statistics gathering to

log the insertion time
Add the queue depth to the _status command

@chef/chef-server-maintainers I'd love review if you have a chance.


Example does not work

There are several deprecation's.
audioPlayer.play() api has changed.

@hapasa can you say which are deprecated so I can work on a fix? can you tell me more information, on what you are trying to do? Like flutter version, etc..

It should all work now on the latest version, thanks


Fix type for Android st_nlink

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@JohnTitor This changes st_nlink from i32 to u32. Is this an acceptable breaking change?

Actually these are already unsigned, so this isn't a breaking change.
@bors r+

:pushpin: Commit 6055dcb8ab59903ed72df036e37eece3b23e6ce1 has been approved by Amanieu

:hourglass: Testing commit 6055dcb8ab59903ed72df036e37eece3b23e6ce1 with merge 56fb9a97f471a2c748ed4f8b3d7abefe74931800...

:broken_heart: Test failed - checks-actions

@bors retry

:hourglass: Testing commit 6055dcb8ab59903ed72df036e37eece3b23e6ce1 with merge 2261e1a63bf1d7a2e457bea756693305aad820fd...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 2261e1a63bf1d7a2e457bea756693305aad820fd to master...


Use fully qualified name for CSRs

From kubernetes 1.19 onwards, csr no longer refers to certificates.k8s.io/v1beta1, but certificates.k8s.io/v1. This is not compatible with using kubectl 1.13, which still leans towards the beta.
This PR makes kubectl explicitly set certificatesigningrequests.v1beta1.certificates.k8s.io rather than just csr.
Additionally, it also changes the e2e tests to no longer "cheat" on the kubectl version being used, and always go for the one defined in the makefile. This was causing the tests to pass while it wasn't the case in the real world.
Thanks to @baderbuddy for their suggestion on #31.

Fixes #31
Closes #32

Tested this locally on 1.19:
nri-bundle-nri-metadata-injection-766ff7cbfc-n2vgq   1/1     Running     0          12d
nri-bundle-nri-metadata-injection-job2-krm7m         0/1     Completed   0          3m1s

$ k exec -ti nginx env| grep NEW
NEW_RELIC_METADATA_KUBERNETES_CONTAINER_NAME=nginx
NEW_RELIC_METADATA_KUBERNETES_CONTAINER_IMAGE_NAME=nginx
NEW_RELIC_METADATA_KUBERNETES_CLUSTER_NAME=test
NEW_RELIC_METADATA_KUBERNETES_NODE_NAME=minikube
NEW_RELIC_METADATA_KUBERNETES_NAMESPACE_NAME=default
NEW_RELIC_METADATA_KUBERNETES_POD_NAME=nginx

Everything seems to be working as expected

Tested this locally on 1.19:
nri-bundle-nri-metadata-injection-766ff7cbfc-n2vgq   1/1     Running     0          12d
nri-bundle-nri-metadata-injection-job2-krm7m         0/1     Completed   0          3m1s

$ k exec -ti nginx env| grep NEW
NEW_RELIC_METADATA_KUBERNETES_CONTAINER_NAME=nginx
NEW_RELIC_METADATA_KUBERNETES_CONTAINER_IMAGE_NAME=nginx
NEW_RELIC_METADATA_KUBERNETES_CLUSTER_NAME=test
NEW_RELIC_METADATA_KUBERNETES_NODE_NAME=minikube
NEW_RELIC_METADATA_KUBERNETES_NAMESPACE_NAME=default
NEW_RELIC_METADATA_KUBERNETES_POD_NAME=nginx

Everything seems to be working as expected


How to manage multi-thread in native extension

Hello every body,
I try to create a native extension that allow me to create VFS (Virtual File System) and i have some problem.
Actually i use dokan because I have a windows.
When dokan call his dokan operations I don't know on which thread i am and the vm was not happy.
But I have found Dart_EnterIsolate(your_isolate) but now I have an other error which is :
vm\dart_api_impl.cc:1382: error: Multiple mutators within one isolate is not supported
If you can take a look at my extension and give me some tips on how to do this kind of things.
The code is here : https://github.com/Rollercops/Dart-Dokany
Thanks a lot

I think its fair to say this won't be addressed. Sorry about that.


Getting this message : Not sure what needs to be done ?

An attempt was made to load a type with the fully qualified name of 'SyncAgents', but no type could be located with that name.'
I have a very complex class with embedded and cascading classes
class SyncAgents
string value1
string value2
class class1
class class2[]
class class3[]
string value  3
etc......
class2 is a embedded class in class1, and class3 is a embedded class in class 2.
I suspect I do not have this setup correctly for the complex class.

Hi @scarryjerry thanks for writing in... this sounds like the namespace is not being emitted correctly or that it doesn't exist in the document.  Are you creating the XML with ExtendedXmlSerializer or does it already exist and written by another xml serializer?

The xml file already exists On Feb 14, 2020 2:59 PM, Mike-EEE<EMAIL_ADDRESS>wrote:Hi @scarryjerry thanks for writing in... this sounds like the namespace is not being emitted correctly or that it doesn't exist in the document.  Are you creating the XML with ExtendedXmlSerializer or does it already exist and written by another xml serializer?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

In that case you will have to do a little bit of work.  ExtendedXmlSerializer works best if it generates the document, but should be able to handle legacy documents.  Try using the EnableImplicitTyping call:
new ConfigurationContainer().EnableImplicitTyping(typeof(SyncAgents), typeof(Type1), typeof(Type1), etc... ).Create()

And see if that helps you out.  Let me know of any further issues you run into and I will do my best to assist you with them.

Closing this now as there has not been any activity with this question.  Please feel free to respond with any additional questions or comments with this issue, however, and I will do my best to assist you. 👍


feat: add wallet key derivation method option

Adds key derivation option as documented in the indy-sdk: https://github.com/hyperledger/indy-sdk/tree/master/wrappers/nodejs#createwallet--config-credentials----void
Also tested to work in React Native

Codecov Report

Merging #650 (531c1b5) into main (810d7bb) will increase coverage by 0.00%.
The diff coverage is 90.00%.

@@           Coverage Diff           @@
##             main     #650   +/-   ##
=======================================
  Coverage   85.64%   85.65%
=======================================
  Files         337      337
  Lines        7267     7271    +4
  Branches     1138     1138
=======================================
+ Hits         6224     6228    +4
  Misses       1042     1042
  Partials        1        1

Impacted Files
Coverage Δ

packages/core/src/wallet/IndyWallet.ts
69.86% <83.33%> (ø)

packages/core/src/types.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Upgrade to electron 29

Clear and concise description of the problem
should support latest version
Suggested solution
update the code
Alternative
No response
Additional context
https://releases.electronjs.org/
Validations

[X] Follow the Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that requests the same feature to avoid creating a duplicate.

always support latest version


Missing functions in "reporters.py"

While trying to run, I am receiving an error from "population.py" when trying to import functions
from reporters.py  plot_complexity, report_output.
These functions do not exist within reporters.py.
File "XXXX\DeepHyperNEAT\population.py", line 11, in
from reporters import report_fitness, report_species, plot_fitness, plot_complexity, report_output
ImportError: cannot import name 'plot_complexity' from 'reporters' (XXXX\DeepHyperNEAT\reporters.py)

Will add these functionalities. Thanks for the patience, I haven't paid much attention to this since I published it but am now using it again for new research. Will update these functions and close the issue when done!


TypeError: assert is not a function

I get this when i try to use the most basic form of assertion :
test('something simple', (assert) => {
  assert.plan(2)

  assert(something)
  assert(somethingElse)
});

While it is documented in Chai, and in node.js core assert module itself that the usage of assert(test [, message]) is equivalent to assert.equal(true, test[, message]) (and is less verbose)

Hey @thetutlage , what do you mean by "japa puts a wrapper on top of assert module, which improves the top level function".
can you explain more?
also @zipang , i'd be happy if you can explain what's wrong?


Add support for MFA with IAM credentials

What changed?
A new assumer is added with support for MFA
Why?
MFA does not work with IAM credentials

Hey @mfzl Thanks for this PR, i'm wondering why the original AWS IAM assumer did not work for you, it does have some logic to prompt for MFA.
I see there is a difference between how it operates when the profile has a parent.
In your situation, are you using a base profile and a child profile or using a profile which has the MFASerial config on it?
Maybe we can fix the issue causing the original IAM assumer not to work.
I'm inclined to think that this also needs to set the aro.SerialNumber when there is no parent profile.
			// set the token provider up
			aro.TokenProvider = MfaTokenProvider
			aro.Duration = configOpts.Duration

			// If the mfa_serial is defined on the root profile, we need to set it in this config so that the aws SDK knows to prompt for MFA token
			if len(c.Parents) > 0 {
				if c.Parents[0].AWSConfig.MFASerial != "" {
					aro.SerialNumber = aws.String(c.Parents[0].AWSConfig.MFASerial)

				}
			}

Hey @JoshuaWilkes, thanks for looking into this.
This can be handled in the AWS IAM assumer as well. My profile is basically
[profile xyz]
credential_process    = granted credential-process --profile xyz
mfa_serial            = arn:aws:iam::xxx
region                = us-east-1

You guessed right about the current assumer, it's making the assumption that user assumes a role in the child profile so it's not triggered when using the credentials directly with MFA using the root profile (no parent)
This can be fixed in the current AWS IAM assumer as well. I felt it's safer to leave the existing code as is without introducing token caching and behavior change but it can be made to work.
In the case of updating the current assumer this logic will go into
	if c.HasSecureStorageIAMCredentials {
		secureIAMCredentialStorage := securestorage.NewSecureIAMCredentialStorage()
		crendentials, err := secureIAMCredentialStorage.GetCredentials(c.Name)
		if err != nil {
		    return credentials, err
		}
                if c.AWSConfig.MFASerial == "" {
		    return credentials, nil
	        }

		// Handle MFA code
	}


CI Details link opens New Tab

When a user clicks on the "details" link to view a CI build instead of taking you away from GitHub, it just opens the CI build information in a new browser tab.

I like this idea. We should also add rel="noopener" if we're using target="_blank".

I don't think we should start changing where links open. GitHub is pretty nice about that.

I guess I do not understand what is wrong with doing this. I have a pretty big dev team, and they have always been annoyed by this as well. It takes you away from GitHub which from my experience people do not like.

I'm constantly annoyed by this. 👍

As mentioned before, the current behavior is a bit frustrating, although the same behavior is seen in other places throughout the website. For example, the link to your website also opens in the current tab. I guess, in the end, it boils down to the user's preference.
Maybe add a global option that opens external URLs — like CI build details, personal website, repository URL, etcetera — in a new tab?

My annoyance also stems from the fact the repositories I use most often have more than 1 system adding commit statuses. This makes GitHub open a tooltip when you click on a commit status icon listing all of the commit statuses with labels from each system.
However, when you are used to that behaviour you click directly on the icon to open the tooltip, but when there is only one commit status you go directly to the CI in the same tab. So it's not just preference of same-tab vs new-tab, it's also the habit of a tooltip which GitHub is abusing by being inconsistent.
Opening a tooltip every time doesn't seem like a good solution to me, though, as most projects would have only one commit status. So addressing the same-tab behaviour for me is great.

@bfred-it I generally agree, but in this case, I can't imagine anyone wants it to open in the same tab. You usually go to the CI to check something and then back to the PR again.


More of a question than a request

I am working on a project that requires that an input expression (in the form that you support) is converted into the python equivalent  code to be put into a file with other code. Is it possible for the Parser output to be exported in a code-like manner?

I'd like to talk about a possible job involving modifying py-expression. Could you contact me at<EMAIL_ADDRESS>


[MINOR] Fix a typo in FsHistoryProvider loginfo

What changes were proposed in this pull request?
a typo in logging. (just added : )

Can one of the admins verify this patch?

Can one of the admins verify this patch?

It's okay but mind double checking if there are some more typos around here? I am sure there are more and let's fix it while we're here.

Also, let's keep the PR template (https://github.com/apache/spark/blob/master/.github/PULL_REQUEST_TEMPLATE).

Jenkins test this please

Test build #122620 has started for PR 28505 at commit 679a1bc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27282/
Test PASSed.

Test build #122620 has finished for PR 28505 at commit 679a1bc.

This patch fails build dependency tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122620/
Test FAILed.

Test build #5004 has started for PR 28505 at commit 679a1bc.

Test build #5004 has finished for PR 28505 at commit 679a1bc.

This patch fails build dependency tests.
This patch does not merge cleanly.
This patch adds no public classes.

Jenkins test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/27388/
Test PASSed.

Test build #122729 has started for PR 28505 at commit 679a1bc.

Test build #122729 has finished for PR 28505 at commit 679a1bc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/122729/
Test PASSed.

OK, merged to master


Change permissions of uid and group

Change permissions of uid and group     OWLS-83349

@ddsharpe I made changes do you mind reviewing again?

@ddsharpe could you review?