Lonboard with tiled data

Lonboard is really powerful for rendering large-ish datasets but sometimes people have data that's just too large to send to the frontend. This can be large raster datasets (think Landsat and Sentinel) or large vector datasets (think Overture).
In https://github.com/manzt/anywidget/pull/453 Trevor manz showed how it's possible to have the JS side request data from Python. This is different from all existing data transfer in lonboard, because existing methods push data from Python to JS rather than pulling data from JS.
This also has the potential to greatly simplify authentication because the frontend requests data via the Jupyter comm. If you have raster data on a remote server and you had a separate tile server running on that machine, you'd usually have to manage the firewall to expose the data. With lonboard, we can have an in process tile server as part of lonboard, so there's no authentication aside from setting up Jupyter.
A very bare prototype of the RPC can be found in https://github.com/developmentseed/lonboard/pull/413.
The deck TileLayer has a getTileData callback. We can await the call to Python.
On the raster side, I think the simplest approach would be to add rio-tiler as an optional dependency and use it to generate PNG/JPG tiles. One question is whether we'd need to put rio-tiler in a threadpool so that each request doesn't block the Python kernel.
We could additionally port something like geojson-vt to export GeoArrow tiles. Then we could have a TileLayer that renders GeoArrow layers as sublayers. And then in theory we could handle truly any size of vector data.
cc @batpad @vincentsarago

We should test again with the new AbortSignal support: https://github.com/manzt/anywidget/pull/540

@kylebarron  Do you see a role for PMTiles here?  We've been very happy with PMTiles for leaflet-based maps (e.g. via folium-pmtiles, leafmap, rendered with maplibre) but from what I can tell it's not supported in deck.gl.

deck.gl supports any tiled data source, as long as you provide a callback for how to load each tile. So for a JS application using deck.gl, supporting PMTiles is not too hard.
The harder part is styling the data, especially depending on the trajectory of the data, and specifically whether data goes straight from server to browser or whether it passes through Python.
The part about pydeck that I dislike the most is that it featured a custom, hacky string domain-specific language for how to run JS code to style the input data. As I write in the internals doc, one of the main goals of lonboard is to support data scientists working with data already in Python. The map's styling could then be done in Python rather than in JS.
For comparison, folium-pmtiles, which uses protomaps-leaflet, looks like it implements the Mapbox Style Specification, so it has a way to apply a declarative JSON style to the data fully on the client.
This issue was originally focused on dynamically tiling data from Python to be sent to JS for visualization, but where the data already lived in Python, or at least was being proxied through Python. We could also implement a PMTiles proxy that would load tiles through Python, and give users the option of a callback to generate usual python layers

Awesome, thanks!  Definitely agree with the sentiments here, :100: on supporting data scientists who are already in Python and would like to do styling from python and not JS.  Like you say, this mostly works pretty well in folium-pmtiles because we can provide this information in JSON / dict format, though I could wish that the support for deriving styles from the data attributes was more pythonic than  {"fill-color": ["get", "color"]) .   otoh, that is easy to wrap around, and mapbox / maplibre styling is very flexible and well documented.

I managed to get pydeck working with pmtiles, but it is limited in styling for reasons already mentioned. folium-pmtiles is also embedding maplibre-js (which allows for full application of maplibre styling) but I suppose this would not be in line with lonboard's design at this point.

I think pmtiles rendering via Maplibre is potentially in scope, but as part of the basemap, not something that we'd render via deck.gl. https://github.com/developmentseed/lonboard/issues/494


Add common controller based on presumed tfcli interface

This PR adds a common controller implementation based on a presumed terraform cli interface.
Generated controllers will call terraform.SetupController to setup the controllers. For example:
// Setup adds a controller that reconciles VPC managed resources.
func Setup(mgr ctrl.Manager, l logging.Logger) error {
	return terraform.SetupController(mgr, l, &v1alpha1.Vpc{}, v1alpha1.VpcGroupVersionKind, getProviderConfigFn)
}

We will also need to generate the required function on managed resource structs to satisfy the Terraformed interface.
For example:
package v1alpha1

import (
	"github.com/crossplane-contrib/terrajet/pkg/conversion"
)

func (mg *Vpc) GetTerraformResourceType() string {
	return "aws_vpc"
}

func (mg *Vpc) GetTerraformResourceIdField() string {
	return "id"
}

// GetObservation of this VPC
func (mg *Vpc) GetObservation() ([]byte, error) {
	return conversion.TFParser.Marshal(mg.Status.AtProvider)
}

// SetObservation for this VPC
func (mg *Vpc) SetObservation(data []byte) error {
	return conversion.TFParser.Unmarshal(data, &mg.Status.AtProvider)
}

// GetParameters of this VPC
func (mg *Vpc) GetParameters() ([]byte, error) {
	return conversion.TFParser.Marshal(mg.Spec.ForProvider)
}

// SetParameters for this VPC
func (mg *Vpc) SetParameters(data []byte) error {
	return conversion.TFParser.Unmarshal(data, &mg.Spec.ForProvider)
}

@muvaf @ulucinar I've updated the PR based on our latest discussions today.
It was a bit tricky since it is not possible to test without the actual tfcli implementation but I believe I covered most of the cases that we can think of. The most challenging part was properly persisting annotations (state+external-name) & spec (late-init) and status without losing any data and I would prefer to get back to that with a later PR when things are testable.
That said, I would suggest merging this one if you don't see anything blocking, so we can iterate more easily without dealing with open PRs, rebases and conflicts.


ChangeNotifierProvider used outside of MultiProvider must specify a child

import 'package:firebase_auth/firebase_auth.dart';
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
import 'package:shapeyou/Screen/homeScreen.dart';
import 'package:shapeyou/services/user_services.dart';

class AuthProvider with ChangeNotifier{
  FirebaseAuth _auth = FirebaseAuth.instance;
  String smsOtp;
  String verificationId;
  String error = '';
  UserServices _userServices = UserServices();

  Future<void> verifyPhone(BuildContext context, String number) async {
    final PhoneVerificationCompleted verificationCompleted =
        (PhoneAuthCredential credential) async {
      await _auth.signInWithCredential(credential);
    };

    final PhoneVerificationFailed verificationFailed =
        (FirebaseAuthException e) {
      print(e.code);
    };

    final PhoneCodeSent smsOtpSend = (String verId, int resendToken) async {
      this.verificationId = verId;

      // dialog to enter otp
      smsOtpDialog(context, number);
    };

    try {
      _auth.verifyPhoneNumber(
          phoneNumber: number,
          verificationCompleted: verificationCompleted,
          verificationFailed: verificationFailed,
          codeSent: smsOtpSend,
          codeAutoRetrievalTimeout: (String verId) {
            this.verificationId = verId;
          });
    } catch (e) {
      print(e);
    }
  }

  Future<bool> smsOtpDialog(BuildContext context, String number) {
    return showDialog(
        context: context,
        builder: (BuildContext context) {
          return AlertDialog(
            title: Column(
              children: [
                Text('Verification Code'),
                SizedBox(
                  height: 6,
                ),
                Text(
                  'Enter 6 digit OTP received as SMS',
                  style: TextStyle(color: Colors.grey, fontSize: 12),
                ),
              ],
            ),
            content: Container(
              height: 85,
              child: TextField(
                textAlign: TextAlign.center,
                keyboardType: TextInputType.number,
                maxLength: 6,
                onChanged: (value) {
                  this.smsOtp = value;
                },
              ),
            ),
            actions: [
              FlatButton(
                onPressed: () async {
                  try {
                    PhoneAuthCredential phoneAuthCredential =
                        PhoneAuthProvider.credential(
                            verificationId: verificationId, smsCode: smsOtp);

                    final User user =
                        (await _auth.signInWithCredential(phoneAuthCredential))
                            .user;
                    //create user data in firestore after user sucessfully registered,
                    _createUser(id: user.uid, number: user.phoneNumber);

                    //navigate to homepage after login.
                    if (user != null) {
                      Navigator.of(context).pop();

                      //dont want to come back to welcome screen after logged in
                      Navigator.of(context).pushReplacement(
                        MaterialPageRoute(builder: (context) => HomeScreen()),
                      );
                    } else {
                      print('Login Failed ! ');
                    }
                  } catch (e) {
                    this.error = 'Invalid OTP ';
                    print(e.toString());
                    Navigator.of(context).pop();
                  }
                },
                child: Text('DONE'),
              ),
            ],
          );
        });
  }

  void _createUser({String id, String number}) {
    _userServices.createUserData({
      'id': id,
      'number': number,
    });
  }
}

@ERUSDHARMENDRA
It seems an issue with your code implementation. The error / issue should tell you what is missing from your implementation. You can also ask this question on support channels like StackOverflow to get help.
Also, it seems to be an issue with Firebase which is 3rd party. If you think this is an issue in the plugin, open this issue in dedicated repo: https://github.com/FirebaseExtended/flutterfire/issues
Closing from here, as this isn't an issue with framework. If you disagree, write in comments and I'll reopen it.


Use 404 as a basis for moving from CrossRef to DataCite when adding a reference

This PR revises the logic when adding a reference via DOI. The logic tries CrossRef first, then if the error code was 404, it tries DataCite. If the error was other than 404, it will throw a new error with a message.
Previously, the logic was to use DataCite for all errors.
Fixes #787

Thanks @nsunami - this is much better 😊


Incorrect handling of apostrophe and quotation marks with send_keys

It's possible that this is a phantomjs issue but I'm unsure. I'm happy to move it over to them if so.
If you hook up a keydown handler and log event.which, you discover the following values for given characters inputted to send_keys:
element.native.send_keys(%{'}) # event.which === 0
element.native.send_keys(%{"}) # event.which === 222

In a real browser, both of these keys leave event.which === 222, but in the case of ", event.shiftKey === true.
The end result is that when you send_keys(%{"}), the browser really sees a ' keypress.
I'd love a workaround for this if there is one.
This is somewhat related to #420 (adding modifier support for send_keys), I think.

Closing this as a phantomjs issue  -- if it's still an issue we can reopen


[Not a Bug] Just wanted to share how to register all commands & subcommands as 'singleton'

Hello!
I was not sure where to share this code snippet, I figured it was to small for a pull request, however, i do think that it would be helpful to add to the GREAT library CommandDotNet, or at least have in the wiki.
This command below uses Microsoft.Extensions.DependencyInjection and registers all commands (and subcommands) as singletons.
This will not work if you would like to use interfaces.  If you want to use interfaces, I advise you to check out Scrutor which is a great library for scanning assemblies for dependency injection
Docs for Microsoft.Extensions.DependencyInjection

// Register all commands and sub commands here, then inject the ServiceProvider in the AppRunner as seen Docs for Microsoft.Extensions.DependencyInjectio above

private static void RegisterAllCommands(this IServiceCollection services)
    {
        var commandClasses = from assembly in AppDomain.CurrentDomain.GetAssemblies() from type in assembly.GetTypes() where type.IsDefined(typeof(CommandAttribute)) select type;

        foreach (var commandClass in commandClasses)
        {
            services.AddSingleton(commandClass);

          // This code registers NESTED subcommands. This can be removed if you do not have nested subcommands within your parent command classes
            foreach (var pi in commandClass.GetProperties()
                         .Where(pi => Attribute.IsDefined(pi, typeof(SubcommandAttribute))))
            {
                var subCommandClass = pi.PropertyType;
                services.AddSingleton(subCommandClass);
            }
        }

I hope someone will find this helpful!
Thanks for the very advanced and helpful library CommandDotNet! I hope to contribute some Pull Requests in the future as I build more tools

Edit:
This will only work if for nested subcommands.
If you do not use nested subcommands, remove the code that registers subcommands

Hi @jeffward01,
Thanks for the compliments and for contributing to the community.  Glad you're finding the library helpful.
We have an example documented here at https://commanddotnet.bilal-fazlani.com/otherfeatures/dependency-injection/#registering-all-command-classes.
private static void RegisterSimpleInjector(AppRunner appRunner)
{
    var container = new SimpleInjector.Container();
    foreach(Type type in appRunner.GetCommandClassTypes())
    {
        container.Register(type);
    }
    appRunner.UseSimpleInjector(container);
}

This approach has the benefit of only registering all and only the commands that can be surfaced by the app.  So, it won't matter what assemblies to scan and it won't include commands that are not yet linked as subcommands, etc.
In an effort to improve discoverability of this example, is there something we could have done to make it easier to find?
Cheers,
Drew


config.replace stopped working

@pgte with the new multipart streams PR, config.replace stopped working which I believe is just because it is using the wrong way to send the file. Mind checking?

ref: https://github.com/ipfs/webui/pull/605#issuecomment-345954414


Improve equality and self-check code.

This depends on a build of stellar/stellar-core#614 or later

:+1:


Implement execsnoop.

Hi!
This is a port of BCC's execsnoop in Go. It also adds a new flag to
output the events as json strings.
I'll really appreciate your feedback :wave:
Signed-off-by: David Calavera<EMAIL_ADDRESS>

Thanks. Something still seems to be off. I tested as follows:
In the first shell run execsnoop:
sudo -E go run examples/bcc/execsnoop/execsnoop.go examples/bcc/execsnoop/output.go | grep '/bin/false'

In a second shell exec a few times:
(exec false arg1 arg2 arg3 arg4 arg5 arg6 arg7 arg8 arg9 arg10 arg11 arg12 arg13 arg14 arg15 arg16 arg17 arg18 arg19 arg20 arg21)

The output doesn't look correct:
false            10049  8561     0 arg9 arg4 arg6 arg5 /bin/false arg1 arg2 arg3 arg8 arg14 arg7 arg12 arg11 arg13 arg17 arg15 arg16 arg10 arg18 arg19 ...
false            10099  8561     0 arg3 /bin/false arg1 arg2 arg11 arg4 arg7 arg5 arg6 arg9 arg8 arg15 arg12 arg13 arg14 arg17 arg16 arg10 arg18 arg19 ...
...

that's interesting, I'm not seeing that. I'm going to take a second look

@calavera how many cpus does your test machine have (cat /sys/devices/system/cpu/online). I'm guessing it could be due to random order of events on my box (4 cpus online) but haven't had time yet to look into bcc in detail.

@schu 8 cpus

@schu what can we do to move this forward? I'm not sure what the problem is, I've tried in different environments and I cannot reproduce the problem.

I've updated this to use the new bpf.GetSyscallFnName() call (thx for that btw) and I can reproduce @schu's issue:
$ sudo -E go run examples/bcc/execsnoop/execsnoop.go examples/bcc/execsnoop/output.go| grep false
false            20996  19237    0 /usr/bin/false arg1 arg5 arg2 arg3 arg4 arg7 arg6 arg8 arg9 arg15 arg18 arg16 arg17 arg12 arg10 arg11 arg19 ... arg13 arg14
false            21005  19237    0 /usr/bin/false arg4 arg1 arg2 arg3 arg11 arg5 arg6 arg7 arg8 arg9 arg10 arg15 arg12 arg13 arg14 arg17 arg16 arg18 ... arg19
false            21013  19237    0 arg9 arg4 /usr/bin/false arg1 arg2 arg3 arg14 arg10 arg11 arg12 arg13 arg6 arg5 arg7 arg17 arg15 arg16 arg18 arg19 arg8 ...
false            21021  19237    0 /usr/bin/false arg1 arg2 arg6 arg3 arg4 arg5 arg10 arg7 arg8 arg9 arg14 arg11 arg12 arg13 arg17 arg15 arg16 arg18 arg19 ...

@calavera sorry for the long wait and thanks for the ping! I have been swamped with work and try to look into it in the coming days.

I've modified the program so it sends a timestamp to userspace and printed that and I see this:
2018/09/28 16:05:08<PHONE_NUMBER>9124 /usr/bin/false
2018/09/28 16:05:08<PHONE_NUMBER>3569 arg10
2018/09/28 16:05:08<PHONE_NUMBER>3668 arg1
2018/09/28 16:05:08<PHONE_NUMBER>7671 arg2
2018/09/28 16:05:08<PHONE_NUMBER>1293 arg3
2018/09/28 16:05:08<PHONE_NUMBER>4809 arg4
2018/09/28 16:05:08<PHONE_NUMBER>7994 arg5
2018/09/28 16:05:08<PHONE_NUMBER>1053 arg6
2018/09/28 16:05:08<PHONE_NUMBER>3912 arg7
2018/09/28 16:05:08<PHONE_NUMBER>6941 arg8
2018/09/28 16:05:08<PHONE_NUMBER>0255 arg9
2018/09/28 16:05:08<PHONE_NUMBER>9286 arg15
2018/09/28 16:05:08<PHONE_NUMBER>6829 arg11
2018/09/28 16:05:08<PHONE_NUMBER>0158 arg12
2018/09/28 16:05:08<PHONE_NUMBER>3228 arg13
2018/09/28 16:05:08<PHONE_NUMBER>6222 arg14
2018/09/28 16:05:08<PHONE_NUMBER>4892 arg17
2018/09/28 16:05:08<PHONE_NUMBER>2082 arg16
2018/09/28 16:05:08<PHONE_NUMBER>7827 arg18
2018/09/28 16:05:08<PHONE_NUMBER>0598 arg19
2018/09/28 16:05:08<PHONE_NUMBER>3600 ...

if you sort that you see the right values:
2018/09/28 16:05:08<PHONE_NUMBER>9124 /usr/bin/false
2018/09/28 16:05:08<PHONE_NUMBER>3668 arg1
2018/09/28 16:05:08<PHONE_NUMBER>7671 arg2
2018/09/28 16:05:08<PHONE_NUMBER>1293 arg3
2018/09/28 16:05:08<PHONE_NUMBER>4809 arg4
2018/09/28 16:05:08<PHONE_NUMBER>7994 arg5
2018/09/28 16:05:08<PHONE_NUMBER>1053 arg6
2018/09/28 16:05:08<PHONE_NUMBER>3912 arg7
2018/09/28 16:05:08<PHONE_NUMBER>6941 arg8
2018/09/28 16:05:08<PHONE_NUMBER>0255 arg9
2018/09/28 16:05:08<PHONE_NUMBER>3569 arg10
2018/09/28 16:05:08<PHONE_NUMBER>6829 arg11
2018/09/28 16:05:08<PHONE_NUMBER>0158 arg12
2018/09/28 16:05:08<PHONE_NUMBER>3228 arg13
2018/09/28 16:05:08<PHONE_NUMBER>6222 arg14
2018/09/28 16:05:08<PHONE_NUMBER>9286 arg15
2018/09/28 16:05:08<PHONE_NUMBER>2082 arg16
2018/09/28 16:05:08<PHONE_NUMBER>4892 arg17
2018/09/28 16:05:08<PHONE_NUMBER>7827 arg18
2018/09/28 16:05:08<PHONE_NUMBER>0598 arg19
2018/09/28 16:05:08<PHONE_NUMBER>3600 ...

In the elf module we have a SetTimestampFunc() method to order events and some code to order the events in the Perf map.
I'm not sure/I don't remember how bcc deals with this, maybe @alban remembers?
In any case, an easy solution would be to order the arguments by timestamp before printing them, but maybe there's a way to make the events in the channel appear in order.

Hm, but events come from the same CPU so they should come from the perf map in order...

I think there is a data race condition. I'm looking.

@schu @iaguis I finally figured out the problem, see this change:
https://github.com/iovisor/gobpf/pull/133/commits/8ae4726f6fbba40b70df7cc6cff7f657294bdf78
The order was wrong because each event received spawed a new goroutine to process the data, I don't think that's necessary. By removing that, the events arrive in order, as expected.
I've also updated this PR to get the right syscall name regarding the kernel version by using GetSyscallFnName.

Right, that makes sense.
Thanks for figuring it out! Looks good to me now.

SIGINT does sometimes deadlock execsnoop now. I think that happens when an event arrives between break loop and case <-pm.stop: since callback_to_go is blocking. We could mitigate that with a buffered channel.
I'm not sure if a blocking callback is the right way to go or if we should ask the user to sort by timestamp (bpf_ktime_get_ns()) when order matters, similar to what we do in the elf package.

Thanks for the review!
@schu

SIGINT does sometimes deadlock execsnoop now. I think that happens when an event arrives between break loop and case <-pm.stop: since callback_to_go is blocking. We could mitigate that with a buffered channel.

I've changed the implementation to avoid the deadlock.

I'm not sure if a blocking callback is the right way to go or if we should ask the user to sort by timestamp (bpf_ktime_get_ns()) when order matters, similar to what we do in the elf package.

This new behavior follows BCC implementation, I think this is more correct.

I'm not sure if a blocking callback is the right way to go or if we should ask the user to sort by timestamp (bpf_ktime_get_ns()) when order matters, similar to what we do in the elf package.

This new behavior follows BCC implementation, I think this is more correct.

OK for me.
Could you squash 699d441 and b84869c into 18541a2? Otherwise good to merge I think, thanks!

@schu squashed :raised_hands:


Don't use MD5 with s3v4

We have a function which avoids md5 when in sigv4, but forgot to include s3v4 even though we attach the function to several s3 commands.
cc @jamesls @kyleknap @rayluo

:shipit:

:ship:


[BUG] Nan loss when finetune resmlp on CIFAR10

Describe the bug
Obtain nan loss after 1 epoch when finetune a pretrained  resmlp_12_224.fb_in1k with mixed precision on cifar10
To Reproduce
I use timm Version: 0.8.18.dev0
Steps to reproduce the behavior:

run the command  python train.py  data/ --dataset torch/cifar10 --model resmlp_12_224.fb_in1k --pretrained --amp

Expected behavior
Model can be finetuned normally
Screenshots
Learning rate (0.05) calculated from base learning rate (0.1) and global batch size (128) with linear scaling.
Using native Torch AMP. Training in mixed precision.
Scheduled epochs: 300. LR stepped per epoch.
Train: 0 [   0/390 (  0%)]  Loss: 8.364 (8.36)  Time: 1.735s,   73.77/s  (1.735s,   73.77/s)  LR: 1.000e-05  Data: 0.683 (0.683)
Train: 0 [  50/390 ( 13%)]  Loss: 3.141 (5.03)  Time: 0.684s,  187.02/s  (0.694s,  184.33/s)  LR: 1.000e-05  Data: 0.009 (0.021)
Train: 0 [ 100/390 ( 26%)]  Loss: 2.805 (3.97)  Time: 0.643s,  199.16/s  (0.684s,  187.03/s)  LR: 1.000e-05  Data: 0.008 (0.015)
Train: 0 [ 150/390 ( 39%)]  Loss: 2.700 (3.54)  Time: 0.585s,  218.96/s  (0.663s,  193.18/s)  LR: 1.000e-05  Data: 0.008 (0.013)
Train: 0 [ 200/390 ( 51%)]  Loss: 2.415 (3.29)  Time: 0.582s,  219.86/s  (0.643s,  199.16/s)  LR: 1.000e-05  Data: 0.008 (0.012)
Train: 0 [ 250/390 ( 64%)]  Loss: 2.560 (3.11)  Time: 0.586s,  218.30/s  (0.631s,  202.90/s)  LR: 1.000e-05  Data: 0.008 (0.011)
Train: 0 [ 300/390 ( 77%)]  Loss: 2.296 (2.98)  Time: 0.584s,  219.01/s  (0.623s,  205.42/s)  LR: 1.000e-05  Data: 0.008 (0.010)
Train: 0 [ 350/390 ( 90%)]  Loss: 2.184 (2.88)  Time: 0.583s,  219.72/s  (0.617s,  207.35/s)  LR: 1.000e-05  Data: 0.008 (0.010)
Train: 0 [ 389/390 (100%)]  Loss: 2.217 (2.81)  Time: 0.571s,  224.31/s  (0.614s,  208.57/s)  LR: 1.000e-05  Data: 0.000 (0.010)
Test: [   0/78]  Time: 0.612 (0.612)  Loss:  1.0401 (1.0401)  Acc@1: 73.4375 (73.4375)  Acc@5: 99.2188 (99.2188)
Test: [  50/78]  Time: 0.199 (0.204)  Loss:  1.0240 (1.0031)  Acc@1: 71.8750 (70.3891)  Acc@5: 98.4375 (98.7439)
Test: [  78/78]  Time: 0.103 (0.200)  Loss:  0.8205 (1.0043)  Acc@1: 75.0000 (70.1100)  Acc@5: 100.0000 (98.6800)
Current checkpoints:
('./output/train/20230412-030416-resmlp_12_224_fb_in1k-224/checkpoint-0.pth.tar', 70.11)
Train: 1 [   0/390 (  0%)]  Loss: 2.160 (2.16)  Time: 1.105s,  115.80/s  (1.105s,  115.80/s)  LR: 1.001e-02  Data: 0.528 (0.528)
Train: 1 [  50/390 ( 13%)]  Loss: nan (nan)  Time: 0.578s,  221.63/s  (0.586s,  218.58/s)  LR: 1.001e-02  Data: 0.008 (0.018)
Train: 1 [ 100/390 ( 26%)]  Loss: nan (nan)  Time: 0.575s,  222.61/s  (0.581s,  220.44/s)  LR: 1.001e-02  Data: 0.009 (0.014)
Desktop (please complete the following information):

OS: Ubuntu 18.04
This repository version 0.8.18.dev0
PyTorch version w/ CUDA/cuDNN  torch 1.12.1, cuda 10.2

@nguyenhungquang it's not a bug, bad hparams, ask in forums


Integrate coverage stats for PR

Fixed https://github.com/zetamesh/zetamesh/pull/24

Fixed https://github.com/zetamesh/zetamesh/pull/24


How to save comment use libconf.dump

I found the comment miss after call libconf.dump, how to save comment in .cfg file. Thanks in advance for your feedback！！！

Hi,
python-libconf doesn't support writing comments at the moment.


Add README

[x] Guidelines: How to compile the project, what is the structure and formatting (Prettier e.t.c)
[x] How to fork the project and open PRs
[ ] How to use admin page
[ ] Current issues to be addressed
[x] Add explanation on how commits and PRs should be done

Some of the tasks are only applicable to #26.
Priority issues should be pinned to the tab and not added to the readme.
No further updates.


Update to last common x/text

Govendor vendor multiple version of x/text under /vendor.
This PR update them all to the the last common commit to ease PR #3972 and like #3986

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3994   +/-   ##
=======================================
  Coverage   20.06%   20.06%
=======================================
  Files         153      153
  Lines       30122    30122
=======================================
  Hits         6044     6044
  Misses      23168    23168
  Partials      910      910

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 81b4d38...9daf95f. Read the comment docs.


How to access a component instance from a javascript event?

I created a component whose BuildRenderTree method emits a "div" element. I need that div "onscroll" event to execute some JavaScript and call to the component after it. How can I achieve that?
According to the documentation, to call a method of the component from JavaScript it is necessary to have a reference

Pass the .NET instance to JavaScript by wrapping it in a DotNetObjectRef instance. The .NET instance is passed by reference to JavaScript.

In the sample shown, it passes the reference from the server
    public Task CallHelloHelperSayHello(string name)
    {
        // sayHello is implemented in wwwroot/exampleJsInterop.js
        return _jsRuntime.InvokeAsync<object>(
            "exampleJsFunctions.sayHello",
            new DotNetObjectRef(new HelloHelper(name)));
    }

But in my case, I need the call to the component to be invoked directly from JavaScript (specifically from a function called onscroll event of div)
So I'm wondering whether there is a way to get the dotnet component reference from JavaScript?
Maybe I could pass the reference on the OnAfterRenderAsync method on the component and save it somewhere so when the onscroll event is raised I can use it, but this will need a separate call from server to the client every time the component is rendered. I bet this will be bad design. If I could somehow pass the reference in the BuildRenderTree, that would avoid the extra call.
Do you see my point?

So I'm wondering whether there is a way to get the dotnet component reference from JavaScript?

Yes, that is exactly what the docs are describing when they talk about "wrapping it in a DotNetObjectRef instance".

Maybe I could pass the reference on the OnAfterRenderAsync method on the component and save it somewhere so when the onscroll event is raised I can use it

Yes, that is precisely the correct way to do it! Looks like you're on the right lines :)

If I could somehow pass the reference in the BuildRenderTree, that would avoid the extra call.

That's not applicable. The render tree is a structure that exists to be diffed, so the framework can issue a minimal set of UI update instructions. It's not a general bag of data that gets passed as a whole to JavaScript.
There's nothing wrong with having an extra call from .NET to JS in the first OnAfterRenderAsync invocation of your component. This only has to happen on the first render of each component, so no matter how many times the component re-renders, the JS-side code only has to be called once to receive the reference.

It worked. Thanks for te answer. It really gives confidence on the direction of the development.


Implement Q_ADAPT_PROBS.

This is broken for quantizers > 20.

LGTM.
Tested for four ranges of qindex, and worked.


fix: simplified error handling in build-dashboard.js

Description
Simplified the error handling in the build-dashboard.js file by replacing return Promise.reject(e) to throw e as suggested in PR #3304
Related issue(s)
resolved #3305

Summary by CodeRabbit

Bug Fixes

Improved error handling mechanisms in dashboard build script
Enhanced error propagation for more robust error management

@sambhavgupta0705 @devilkiller-ag Can you please review this.

@devilkiller-ag Sorry Sir I requested for review again
It happened by mistake

@sahitya-chandra  why are we having these changes?

I have just simplified the error handling in build-dashboard.js file as mentioned in #3305 issue

we are using async/await so yes we can use this throw here

/update

@sambhavgupta0705 thanks for the review sir.

@anshgoyalevil can you please take a look at this

This does solve the linked issue, but we are still looking for a better error handling strategy. Can we get https://github.com/asyncapi/website/issues/3358 done instead of just getting this promise thing modified?

This does solve the linked issue, but we are still looking for a better error handling strategy. Can we get #3358 done instead of just getting this promise thing modified?

Ok sir once I understand #3358 issue I will let you know...

@anshgoyalevil Please have a look sir...


Harpy not working in a React Native project (working directly with AppDelegate.m)

There is currently a version of my app in the App Store (1.1.1). Users still on 1.0 or 1.0.1 don't see a Harpy message prompting them to update.
I'm using React Native in my app, but as far as I can tell it shouldn't affect Harpy, because I'm working directly with AppDelegate.m natively for Harpy.
I confirm that I have Harpy in my Podfile.
Here is my AppDelegate.m:
#import "AppDelegate.h"
#import "Harpy.h"
#import "ReactNativeConfig.h"
#import "Amplitude.h"

#import <React/RCTBundleURLProvider.h>
#import <React/RCTRootView.h>
#import "RCTOneSignal.h"

@implementation AppDelegate

@synthesize oneSignal = _oneSignal;

- (BOOL)application:(UIApplication *)application didFinishLaunchingWithOptions:(NSDictionary *)launchOptions
{

  NSURL *jsCodeLocation;

  jsCodeLocation = [[RCTBundleURLProvider sharedSettings]<EMAIL_ADDRESS>fallbackResource:nil];

  // jsCodeLocation = [[NSBundle mainBundle] URLForResource:@"main" withExtension:@"jsbundle"];

  RCTRootView *rootView = [[RCTRootView alloc] initWithBundleURL:jsCodeLocation
                                                      moduleName:@"Skale"
                                               initialProperties:nil
                                                   launchOptions:launchOptions];
  rootView.backgroundColor = [[UIColor alloc] initWithRed:1.0f green:1.0f blue:1.0f alpha:1];

  self.window = [[UIWindow alloc] initWithFrame:[UIScreen mainScreen].bounds];
  UIViewController *rootViewController = [UIViewController new];
  rootViewController.view = rootView;
  self.window.rootViewController = rootViewController;
  [self.window makeKeyAndVisible];

  NSString *oneSignalAppId = [ReactNativeConfig envFor:@"ONESIGNAL_APP_ID"];
  self.oneSignal = [[RCTOneSignal alloc] initWithLaunchOptions:launchOptions appId:oneSignalAppId];

  // Harpy section

  // Set the UIViewController that will present an instance of UIAlertController
  [[Harpy sharedInstance] setPresentingViewController:_window.rootViewController];
  // (Optional) Set the App Name for your app
  [[Harpy sharedInstance] setAppName:@"Skale"];
  // Set the Alert Type for your app
  [[Harpy sharedInstance] setAlertType:HarpyAlertTypeForce];

  // Perform check for new version of your app
  [[Harpy sharedInstance] checkVersion];

  return YES;
}

- (void)applicationWillEnterForeground:(UIApplication *)application {
  /*
   Perform check for new version of your app
   Useful if user returns to you app from background after being sent tot he App Store,
   but doesn't update their app before coming back to your app.

 	 ONLY USE THIS IF YOU ARE USING *HarpyAlertTypeForce*

 	 Also, performs version check on first launch.
   */
  [[Harpy sharedInstance] checkVersion];
}

@end

Any help would be greatly appreciated. Thank you for creating this project -- it's well-documented and easy to understand!

Can you set the debugEnabled flag to true, roll back the version of your app in xcode to 1.0 or 1.0.1, and see what the logs say?
Also, is your app in the US app store? If not, you need to also use setCountryCode

Ah I found out the problem -- it's unrelated to the AppDelegate.m I showed here. Thank you! Closing now.

Glad you figured it out - good luck!


build: general improvements

This patch contains several small changes to the CMake build script:

Set the C++ standard version with a target feature instead of by modifying the global CMAKE_CXX_STANDARD, so that integration as a subproject doesn't create issues.
Drop explicit CMP0091=NEW, as that is already done by setting the cmake_minimum_required to version 3.15.
Permit building the library as a shared one, and also give it a version and a soversion; here I supposed ABI compatibility between releases with the same major.minor version.
Add a Findzstd.cmake module that looks for libzstd using pkg-config if the Config file is not available (like in most Linux distros).
Use values from GNUInstallDirs instead of hardcoded ones.
Support pkg-config by generating a .pc file.

It would be nice to know what your ABI compatibility guarantees are, as my supposition could be wrong.

Lots of good improvements, thank you.

It would be nice to know what your ABI compatibility guarantees are, as my supposition could be wrong.

The ABI for this is stable. Going forward there might be additions but the current interface will stay the same. There will be no backwards compatibility for any new features for obvious reasons.

The ABI for this is stable. Going forward there might be additions but the current interface will stay the same. There will be no backwards compatibility for any new features for obvious reasons.

Ok, so I can assume that 0.1.1 and 0.1.4 are ABI-compatible with each other, but 0.1.1 and 0.2.0 are not?


Add external texture support

Starting a draft to discuss the proper way to integrate support for external texture in Godot 4.x.
This is feature that's forward-ported from Godot 3.x and is key to enable support for texture that can be painted by the target platform (see the original PR for target use-cases).
As mentioned in https://github.com/godotengine/godot/pull/36342#pullrequestreview-369375100, this was punted for the 4.x release due to GLES reimplementation and the need to figure out how this applies to Vulkan.
This draft aims to restart the conversation in order to properly integrate it in Godot 4.x

cc @maunvz

@paddy-exe FYI, this is required to support ARCore in Godot

How does this relate to textures created via TextureStorage::texture_create_external() (in the OpenGL renderer)? That function is being used for OpenXR (on OpenGL) and WebXR.
Or, if this doesn't relate to that (like, if it's a totally different thing) then we should probably rename one of them, because having two unrelated types of textures both called "external" is going to be confusing :-)

How does this relate to textures created via TextureStorage::texture_create_external() (in the OpenGL renderer)? That function is being used for OpenXR (on OpenGL) and WebXR.
Or, if this doesn't relate to that (like, if it's a totally different thing) then we should probably rename one of them, because having two unrelated types of textures both called "external" is going to be confusing :-)

@dsnopek they're indeed different things; see this comment that goes on the difference between the two: https://github.com/godotengine/godot/issues/65778#issuecomment-1249351066
Renaming is made complex by the fact the one I'm referring to is part of the OpenGL spec, so arguably the current Godot one should be renamed to be consistent with external references, but I don't have a strong preferences either way so long as it's clear which one we're referring to.

As mentioned in this StackOverlfow thread, this may be supported in Vulkan using the VK_ANDROID_external_memory_android_hardware_buffer extension.

they're indeed different things; see this comment that goes on the difference between the two: #65778 (comment)
Renaming is made complex by the fact the one I'm referring to is part of the OpenGL spec, so arguably the current Godot one should be renamed to be consistent with external references, but I don't have a strong preferences either way so long as it's clear which one we're referring to.

I think it'd be fine to rename the existing TextureStorage::texture_create_external() to something else - it's not exposed publicly anywhere, and only used in those two places. I don't have any brilliant naming ideas, though. :-)
Maybe:

TextureStorage::texture_create_foreign()
TextureStorage::texture_create_reference() -- the idea being it "references" a texture created elsewhere
TextureStorage::texture_create_remote()
TextureStorage::texture_create_third_party()
TextureStorage::texture_create_independent()

Naming is hard :-)

cc @clayjohn , you should have a look at this too.

Is this android specific or will it work on iOS as well?

Is this android specific or will it work on iOS as well?

It's not Android specific in theory, but it uses OpenGLES and Vulkan extensions which iOS doesn't (officially) support, so there would need to be investigation on whether this is available to iOS through the available compatibility APIs.
cc @BastiaanOlij @clayjohn

Superseded by https://github.com/godotengine/godot/pull/96982


Bugfix/home page typo

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


How to set a maximum MultiPart Form Data size without persisting full file on disk?

I want to check the file size before the file is persisted in the backend. I used a POST Multipart upload.
After checking the issue #1657 the solution was to ByteStreams.limit. But as @pavelkokush mentioned that would mean to persist the file first.
So I was wondering how the file size check could be implemented.

I do not know about special dropwizard features... But file will not be persisted on disk automatically if you work directly on HttpServletRequest
@POST
@Consumes("multipart/form-data")
void upload(@Context HttpServletRequest request);

Then you can use ServletFileUpload
ServletFileUpload upload = new ServletFileUpload();
upload.setSizeMax(XXX);
upload.setFileSizeMax(YYY);
InputStream inputStream = upload.getItemIterator(request).next().openStream();

It will throw exception when you read inputStream if any limits reached.

I would do one of the following using the request input stream:

Maintain a buffer in memory that tracks and track how much data has been read (you may wish to have a pool of reusable buffers for memory/gc reasons) that will throw if the limit is exceeded. I'd implement them using some sort of close/finish mechanic that on close, if no errors found, dump the buffer to disk.
Write to disk and if the limit is exceeded, close the write handle and nuke the file (thus you don't write the entire payload if you don't have to).

Thanks  for your notes. Let me be a concrete. So consider the example  link
@isaki-x
Where would I integrate the in memory buffer?
@pavelkokush
I assume I need to add an additional argument to the uploadFile method to be able to the ServletFileUpload

@LeducH You would need to have it in your resource. A simple implementation would be to have some sort of buffer object, which could be as simple as a byte[] stored in a ThreadLocal. You could also just simply allocate a byte[] on every request as well.
You would need to read the stream yourself (and modify my simple example to match your work):
private final ThreadLocal<byte[]> cache;

public YourConstructor(final int maxFileSize) {
    super();

    Preconditions.checkArgument(maxFileSize > 0);

    this.cache = ThreadLocal.withInitial(() -> new byte[maxFileSize]);
}

@POST
@Path("/post/file")
@Consumes("application/octet-stream")
public Response postTheFile(final InputStream requestStream) throws IOException {
    final byte[] buffer = cache.get();
    int read = requestStream.read();
    int i = 0;
    for (; read != -1 && i < buffer.length; ++i) {
        buffer[i] = (byte) read;
        read = requestStream.read();
    }

    if (read != -1) {
        // Error, I'll just throw, you do what you want
        throw new IOException("Max File Exceeded");
    }

    // This method needs to know how far into the buffer to go.
    this.writeFile(buffer, i);
}

This is based on this: https://stackoverflow.com/questions/10326460/how-to-avoid-outofmemoryerror-when-uploading-a-large-file-using-jersey-client
I don't know if it will work as I've never actually tried this. Let me know if this completely fails and I'll try to come up with something else.

The funny thing is I tried this method that you proposed.  As you might see further down the bad think is sometimes when the files are to large it takes quite long. There was a hint to look into the WriteInterceptor.
When I found something I will post my findings
@POST
  @Consumes(MediaType.MULTIPART_FORM_DATA)
  public Response uploadFile(
      @FormDataParam("file") InputStream uploadedInputStream,
      @FormDataParam("file") FormDataContentDisposition fileDetail)
      throws IOException {
    String fileName = fileDetail.getFileName();
    String uploadedFileLocation = uploadPath + fileName;

    Long size = writeToFile(uploadedInputStream, uploadedFileLocation);
     return Response.ok(size).build();
  }

  // save uploaded file to new location
  private Long writeToFile(InputStream uploadedInputStream, String uploadedFileLocation)
      throws IOException {
    int read;
    final int BUFFER_LENGTH = 1024;
    final byte[] buffer = new byte[BUFFER_LENGTH];
    long size = 0;
    OutputStream out = new FileOutputStream(new File(uploadedFileLocation));
    while ((read = uploadedInputStream.read(buffer)) != -1) {
      out.write(buffer, 0, read);
      size += read;
      if (size > 2 * 1014 * 1024) {
        LOGGER.info("FileMetaData size exceeds 2MB");
        throw new BadRequestException("Server unavaible");
      }
    }
    out.flush();
    out.close();
    return size;
}

I'm closing this issue as the solutions presented seem sufficient. If there is a proposal to better handle these situations in Dropwizard, then I'm game to listen


Introduce a CoapMessage class/interface ?

I don't know if this is a good idea but I face some situation where I need to handle a coap request OR a coap (separated) response in a same way. This is not too surprising as from a CoAP RFC point of view all request and response are conceptually a CoAP Message.
In my case, I need the remote peer address and the transport context of the message.
See :

https://github.com/eclipse-leshan/leshan/blob/170d4095618e1371470ef27c61df5de4885b8610/leshan-tl-javacoap-core/src/main/java/org/eclipse/leshan/transport/javacoap/identity/DefaultCoapIdentityHandler.java#L27-L51
https://github.com/eclipse-leshan/leshan/blob/170d4095618e1371470ef27c61df5de4885b8610/leshan-tl-javacoap-core/src/main/java/org/eclipse/leshan/transport/javacoap/identity/DefaultTlsIdentityHandler.java#L33-L47

Easy one would be to add helper method, would that be enough?

I'm not sure to see how it will looks like ?

Something like:
static InetSocketAddress getPeerAddress(Object coap) {
...

static TransportContext getTransportContext(Object coap) {
...

OK I get it but my question was more about having a kind of Parent class (e.g. CoapMessage) for CoapRequest, CoapResponse, SeparatedResponse. At first sight (maybe I'm wrong) I think this could make java-coap user code more elegant.
Regarding adding this static method, to be honest, I'm not sure will really makes my user code better. :thinking:
Anyway, maybe better to work first on https://github.com/open-coap/java-coap/issues/84 and see if you also need this kind of abstraction in java-coap code too ?


Better configuration dialog

With all the features to be added, the current "Options" menu is not going to fit everything. So we need a better dialog
Need to also work with mobile interface so we don't need to fix that in the future

Also add a better setting infrastructure to separate it from the run time AppState


fix  fix workload group non-existence error

Proposed changes
Issue Number: close #xxx

fix getWorkloadGroupQueryQueue does not get the default_workload_name in the user property, which causes the workload group not to be found;
check whether to open the pipeline engine when getting the workload group query queue.

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


Support non-string formula?

I'm interesting in running regression on the entire set of variables. However it seems that the FormulaRegressionBuilder requires a formula to work. Is there any thought to adding an "all variables" mode, or else provide an array of variable names so that it's a bit easier to automate? I would like to avoid building a string formula programatically if I can.

Hmm yes, the library seems a bit too closely tied to the formula idea.
I've added a data_columns method to FormulaRegressionBuilder on the master branch.
Usage would look like this:
let data = vec![("Y", y), ("X1", x1), ("X2", x2), ("X3", x3)];
let data = RegressionDataBuilder::new().build_from(data).unwrap();
let model = FormulaRegressionBuilder::new()
    .data(&data)
    .data_columns("Y", ["X1", "X2", "X3"])
    .fit()
    .unwrap();

Does that work for your use case?

That's an improvement. But actually I want to support regression with a big unlabelled matrix (ndarray), which doesn't even have vector names. So I would still have to generate names for each vector, but at least that would be easier than building a big string.

Ah, I see.
How about this then?
See doc comment for intended usage.
I'm not really willing to expose nalgebra types in the API, so that's about as low level, as I can reasonably offer.

Yep I think that would be fine. I'm sure there's an ndarray function to unwrap into a row-major array. Thanks!


Separate Hard & Soft mutes

Title, Basically separate the too types into two commands.
(Hard being whisper & chat ignored & Soft just chat)

Will get to this soon when I'm done with PistonQueue for velocity.


[WIP] Add new SVG and HTML APIs

[ ] Deprecate addHTML, fromHTML, addSVG
[ ] Add new cleaner methods: html() and svg()
[ ] Write tests
[ ] Update docs
[ ] Party 🙌

Any progress on this one? Our project could really use this.

Could this help to render a PDF containing MathJax (rendered as SVG) @MrRio?
Here is an updates Jsfiddle showing Math rendered as SVG:
https://jsfiddle.net/c5bz060c/3/
Could jsPDF do something here?

@josephernest Yeah this could absolutely be used to render MathJax

@MrRio I think I tested extensively a few months ago when I posted this comment, but I've always been unsuccessful.
Could you post a small example showing to render a PDF from this jsfiddle using MathJax: https://jsfiddle.net/c5bz060c/3/
It would help the math community very much :) Thanks in advance.

Any idea @MrRio? Could you post a small example showing to render a PDF from this jsfiddle using MathJax: https://jsfiddle.net/c5bz060c/3/ ? I'm still unable to do it successfully. Thanks a lot in advance!

I close this pr.. :P


Test fake-infix adverbs on a construct

Just code provided by the ticket, closes
https://github.com/rakudo/rakudo/issues/1500

Totally fine test, however:

<AlexDaniel> c: 2015.12,HEAD use Test; my Pair $p; is-deeply ($p .= new :key<foo> :value<bar>), :foo<bar>.Pair, ‘foo’
<committable6> AlexDaniel, ¦2015.12: «===SORRY!===␤Unknown QAST node type NQPMu␤ «exit code = 1»» ¦HEAD(e45bb34): «ok 1 - foo␤»

So on rakudo versions with the bug the test doesn't just fail, the whole file doesn't compile. I'm not sure what's the policy on tests like this, I'm OK with it being the way it is.

So on rakudo versions with the bug the test doesn't just fail, the whole file doesn't compile. I'm not sure what's the policy on tests like this

I think if one would break a parser and the file won't compile, instead of saying "Tests are not ok", this failure is just as obvious as nok messages. And it can happen for any file, we have "This thing parses ok" tests along with semantic ones.

Yeah, alright!


Correcting the version reference

Correcting the version number for .NET Framework on the .NET Standards Library file
Summary
Updated the version number for the next version of the .NET Framework, as it has not been officially deemed 4.6.3.
Suggested Reviewers
@richlander

Hi @staceyhaffner, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;


Add support for syntax-highlighted code blocks

Fixes #11.

Branch rebased after the merge of #14.


jedis version up

updated to 2.7.3 from 2.4.2
replace a deplicated method

Hi, can you squash the commits modifying the pom.xml please?
Other than that, looks good to me.

@moli9ma can you rebase this on the latest master please?

@pk11 Hey, could you take another look at this and merge/release when you get the chance? FWIW looks like https://github.com/pk11/sedis/pull/4 was trying to solve the same issue, can probably close that one out


Error: no such option: --keep-outdated

Hello,
I am trying to upload a vanilla serverless python Lambda, just using the following steps:
npm install -g serverless
serverless create --template aws-python3 --name test-lambda
pipenv shell
pipenv install requests
pipenv lock
sls plugin install -n serverless-python-requirements
All this seems to work fine, but when I do a sls deploy --stage dev I am getting the following error:
Error: no such option: --keep-outdated
The debug output is:
  at ServerlessPythonRequirements.pipfileToRequirements (/Users/myuser/projects/guardduty/node_modules/serverless-python-requirements/lib/pipenv.js:34:11) From previous event:     at PluginManager.invoke (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:390:22)     at PluginManager.spawn (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:408:17)     at Deploy.BbPromise.bind.then.then (/usr/local/lib/node_modules/serverless/lib/plugins/deploy/deploy.js:123:50) From previous event:     at Object.before:deploy:deploy [as hook] (/usr/local/lib/node_modules/serverless/lib/plugins/deploy/deploy.js:113:10)     at BbPromise.reduce (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:390:55) From previous event:     at PluginManager.invoke (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:390:22)     at PluginManager.run (/usr/local/lib/node_modules/serverless/lib/classes/PluginManager.js:421:17)     at variables.populateService.then.then (/usr/local/lib/node_modules/serverless/lib/Serverless.js:157:33)     at runCallback (timers.js:672:20)     at tryOnImmediate (timers.js:645:5)     at processImmediate [as _immediateCallback] (timers.js:617:5) From previous event:     at Serverless.run (/usr/local/lib/node_modules/serverless/lib/Serverless.js:144:8)     at serverless.init.then (/usr/local/lib/node_modules/serverless/bin/serverless:43:50
environment:
System Version: macOS 10.14.1 (18B75)
Python 3.6.1
sls verion 1.32.0
I am sure it is something stupid I am doing, but any suggestions would be appreciated!
Thanks! :-)

You probably need to update your pipenv version. pip install -U pipenv

Thanks dschep, just did that, but same error unfortunatley.. I am now on pipenv, version 9.0.1

That's still an old version of pipenv: https://pypi.org/project/pipenv/#history

sorry, should have checked that..
updated with brew, rather than pip and all working fine now...thank you for the swift response! :-)


Example with redux and react router

Hi Razzle team,
Just wanted to first say thanks for doing such an amazing job with this codebase and the ideas surrounding it. It's pretty much exactly what I was looking for without all the bloat.
This isn't really an issue per se but more of an open question. In short I'm wondering if there is an example package with react router & redux. I've searched through all the open and closed requests and the closest thing I have found is https://github.com/jaredpalmer/react-router-nextjs-like-data-fetching .
I was wondering if their might be plans to create an example app that would extend the repo that I've listed to include redux? The closest repo I have found that answers this is https://github.com/faceyspacey/redux-first-router-demo but I am a huge fan of the simplicity of what the razzle codebase does. For my needs the faceyspacey repo seems a bit much.
I've tried pretty extensively extending the 'with-redux' codebase to include react router but seem to not have much luck.
Cheers!
James
Also. Really happy to see the CSS Modules being brought in!

A good example of this would be a splendid idea! Its a pain to set up and documentation is slim.

I second this, a nice example would be great.

well i have integrate redux in already developed razzle app hope this will help
Git hub Repo


[Feature][connector-v2]Support opengauss jdbc connnector using opengauss driver.

Use opengauss driver to implement opengauss jdbc sink and source

Good job. I often encounter authentication issues when using jdbc-pg to connect to opengauss. When the password_encryption_type=1 of the Opengauss database is used to encrypt the password in the sha256 mode, which is compatible with the MD5 user authentication mode of the postgres client， but when password_encryption_type=2 means that the password is encrypted using sha256, which is not compatible with the postgres open source driver and open source client. The postgres open source ecosystem gives an error, and the JDBC GaussDB gives the following error.

Conflict resolved Request merge

This implementation code is almost identical to jdbc-postgresql, so I suggest you just rely on jdbc-postgresql's implementation and override only the implementation of specific methods, such as those involving jdbc-url and the drivers in the pom.

This implementation code is almost identical to jdbc-postgresql, so I suggest you just rely on jdbc-postgresql's implementation and override only the implementation of specific methods, such as those involving jdbc-url and the drivers in the pom.

Make changes to the code as suggested.

@hawk9821 would you mind take a look again?


Provide an explicit means to purge a cached catalog

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2915
Originally created by @aaronsteers on 2021-09-01 19:22:14

Currently cached catalogs are maintained for all taps unless they are non-pip or editable-mode installs.
Proposal
Two options:
Option A - explicit standalone refresh command:
Introduce a CLI means of explicitly refreshing a plugin's cached catalog:

meltano invoke tap-mytap --discover

Option B - explicit inline refresh along with sync:
Per execution of meltano elt and/or meltano invoke, allow the forcing of catalog discovery with an explicit on/off value.

meltano invoke tap-mytap - lets meltano decide
meltano invoke tap-mytap --with_discover=True - forces a refresh before sync
meltano invoke tap-mytap --with_discover=False - forces skipping discovery (tap may fail if none is cached)

And the same for meltano elt:

meltano elt tap-mytap target-something - lets meltano decide
meltano elt --with_discover=True tap-mytap target-something - forces a refresh before sync
meltano elt --with_discover=False tap-mytap target-something - forces skipping discovery (tap may fail if none is cached)

Note:

For the inline use case where sync is also going to be run, I'm biasing towards something like --with_discover because in the singer world --discover by itself means only discover and do not sync.

Related
This also relates to #2907 which proposes a means to always-refresh the catalog. For instance, adding a CLI flag into meltano elt (or meltano.yml) which would ignore cached file and allow a single-step execution with a freshly gotten catalog.

Done by #8580


Detect device connected to see which accounts are available

Overview
Would be really nice if the account container class did some sort of detection (perhaps lazily) of whether a device was connected, and if so, what accounts are connected via that device. Especially useful for something like ape-safe, where it dynamically detects what wallets it can use to sign with
Specification
May need to track some sort of "device ID" (maybe a processed version of the master pubkey) that can be used to detect which devices are connected to which accounts, and then based on the connected devices at the time that a request is made to the account container of which accounts are available we filter that response dynamically.
Dependencies
Include links to any open issues that must be resolved before this feature can be implemented.

could also be useful for ape-trezor or any other hardware/config-dependent plugin (even ape-safe where only certain networks are valid for that account)


Explain Metadata implemention and usage in Pipelines vs Kubeflow core

Update the Metadata overview: There are discussions about merging the metadata UI (see section in the metadata overview) into the Pipelines UI - we need to design docs for UI metadata support. Contact: ajaygopinathan, prodonjs

From further discussions: Users will need metadata functionality for experimentation as well as for pipelines. It's possible that metadata will be implemented as an independent service that can be included in both pipelines and Kubeflow core. The UX team is working on a design for how the metadata service/UI will be exposed to users.

/area docs
/priority p2

/assign @zhenghuiwang

I went through the metadata doc section with KF1.0 and updated it in https://github.com/kubeflow/website/pull/1641. The major change is to update the backend instruction to use gRPC service.
I think this covers how current Kubeflow metadata SDK, UI and backend work. I would leave it to @joeliedtke or @neuromage for Pipeline's view.

I went through the metadata doc section with KF1.0 and updated it in #1641. The major change is to update the backend instruction to use gRPC service.
I think this covers how current Kubeflow metadata SDK, UI and backend work. I would leave it to @joeliedtke or @neuromage for Pipeline's view.

Thanks @zhenghuiwang your Metadata update is great. I've created issue https://github.com/kubeflow/website/issues/1651 to cover the Metadata update for KF1.0. I think your PR can close that issue. Let's restyle this current issue to request a doc explaining the Metadata implemention and usage in Pipelines vs Kubeflow core.
/assign @neuromage
/unassign @zhenghuiwang
/cc @joeliedtke

/reopen
related discussion in https://github.com/kubeflow/metadata/issues/225


chore(package): update prop-types to version 15.5.6

Closes #84
https://greenkeeper.io/

Codecov Report

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

@@           Coverage Diff           @@
##           master      #85   +/-   ##
=======================================
  Coverage   39.16%   39.16%
=======================================
  Files          11       11
  Lines         143      143
  Branches       27       27
=======================================
  Hits           56       56
  Misses         87       87

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 62bd370...5bf45be. Read the comment docs.


Considering generating a nonce for the Flutter Scripts in index.html

We should consider adding a nonce to helmet and the index.html to be able to remove unsafe-inline and unsafe-eval.
How I think it could work:

In the BuildCommand we check the amount of script tags in the build index.html.
Then we generate this amount of nonces and add them to middlewares.dart and afterwards behind every script tag in the index.html

Edit:
TIL Nonce should be generated dynamically.
I'll try the same approach but with a sha256 hash
Edit2:
API will be that we probally can offer hash and nonce scripts, so users will have to add either 'sha256-'/'sha384-'/'sha512' or 'nonce-' to their script tags. The rest will be handled by us while building the container and serving the application
@robiness @passsy your thoughts about this

nonce should work. Set a nonce in the Content-Security-Policy header and set the same nonce to all <script> tags that you serve for that request.
It requires editing the HTML though.
sha256 is the same thing, but instead of editing the HTML, a hash of all inline-js will be set in the header.
I prefer sha256, because you save yourself manipulating HTML, and the responses can be cached.


Added missing languages translations in the dropdown

Complements/Addresses #1094.
Changes proposed in this pull request:

Added language translations for all available languages

cc @HospitalRun/core-maintainers

 and so on.
I am a bit confused how there are two dropdowns for spanish and spanish (colombian). As far as I can tell the languages are pretty much the same and differ in dialect/slang. Would love some clarification.
https://www.quora.com/What-are-the-major-differences-between-Colombian-Spanish-and-Spanish-from-Spain
https://www.lonelyplanet.com/thorntree/forums/americas-mexico/mexico/how-different-is-mexican-spanish-to-colombian-spanish
Also after reviewing my commit files, should I not have included my package-lock.json file (17k lines of code...)? I couldn't find anything about it in the contributing guidelines. Edit: I have now removed it from the commit files.

@tchan I believe you're right, and that the differences are comparable to British vs American English.  Some apps have i18n options which allow for both, a design choice which depends on the app's audience.

Makes sense imo to get rid of it but what do you guys think?

The Spanish and Colombian Spanish are basically the same, there are words that change or expressions but I don't think that is enough to put it as a different language. It the same for example to Argentinian Spanish.


Updates for iOS 13:

Updates for iOS 13:
https://developer.apple.com/videos/play/wwdc2019/707/

Need to reportIncomingCall to Callkit
payload: apps-expiration to 0, need to be updated in Signal.
Need to Call ReportIncoming func in didReciev(Pushkit).
4.App crashed on launchBackgroundSync

Originally posted by @toshanmugaraj in https://github.com/vector-im/riot-ios/issues/3017#issuecomment-597866639

Work in progress in https://github.com/vector-im/riot-ios/issues/2714

@manuroe Can you please update on above issue. I am also facing the same.
"Killing app because it never posted an incoming call to the system after receiving a PushKit VoIP push callback"


colorbox 90% of window size

Hi all,
first of all: excellent js scripting. I'm trying to use it in a wordpress website and it is working smooth. However, the wordpress plugin comes short in sizing the images when zoomed in.
My goal is to have a max width and height of 90% of the window innerwidth and height. I achieve this by setting default values to: "maxWidth: window.innerWidth * 0.9" and "maxHeight: window.innerHeight * 0.9". This works great! There is, however something else I would love to see: when someone resizes there screen, the pictures should also resize and according to the new screen size. This could be done by using:
$(window).resize(function() {
//resize just happened, pixels changed
settings.maxWidth = window.innerWidth * 0.9;
settings.maxHeight = window.innerHeight * 0.9;
});
This sets the settings but the settings are not accpeted or not changed on program level because the code where max size is checked, stil works with the old values.
This made me change the code to: "settings.mw = setSize(window.innerWidth * 0.9, 'x') - loadedWidth - interfaceWidth;". I know this is ugly coding but I couldn't find any better way to solve this problem.
Can this be changed or can you point me in a direction I can work so maybe I can push these changes?
Kind regards
Bob Claerhout

Did you see $.colorbox.resize() ? Sounds like that could solve your problem?


Make CBMC proofs follow release-1.5 directory layout

This PR changes the paths in all of the CBMC proof configuration files so that the proofs can run on the release-1.5 branch. The release-1.5 branch significantly changed the directory structure, meaning that the CBMC proofs could not link against FreeRTOS object files. This PR is in anticipation of merging the release-1.5 branch into master, so that the proofs will continue to run on the master branch.

This isn't going to be finished tonight. I'll hopefully finish tomorrow morning and have it ready to merge by morning Seattle time.

@mmuesly thank you for your comments, I've reduced the scope of this PR as you suggested. This PR now includes only path fixups for the existing proofs, as well as the top-level build files. I'll post unrelated changes as separate PRs.

FWIW, I used the following script to validate that all the updated paths referenced from the Makefile.jsons actually exist in the source tree. The CI should also tell us whether the renaming was successful.
#!/usr/bin/env python3

import os
import os.path
import json
import re

def check(data, fyle_name):
    if isinstance(data, list):
        for item in data:
            check(item, fyle_name)
    elif isinstance(data, dict):
        for item in data.values():
            check(item, fyle_name)
    elif isinstance(data, str):
        if not data.startswith("$(FREERTOS)"):
            return
        data = data[len("$(FREERTOS)") + 1:]
        if data.endswith(".goto"):
            data = "%s.c" % data[:-len(".goto")]
        if not os.path.exists("../../../%s" % data):
            print(fyle_name)
            print("    %s" % data)

for root, _, fyles in os.walk("."):
    for fyle in fyles:
        if os.path.splitext(fyle)[1] != ".json":
            continue
        with open(os.path.join(root, fyle)) as f:
            tmp = f.read().splitlines()
        buf = [l for l in tmp if l and l.strip()[0] != "#"]
        doc = "\n".join(buf)
        data = json.loads(doc)

        check(data, os.path.join(root, fyle))

Today, we completed our new release. And we have merged release-1.5 to master. If it is OK, can you make this PR base off master branch?


:construction: lazy validation

This increases large applications from 10 seconds mutation to 300ms

Need some testing on large applications, will be merged soon


README.md pointer to Fedora packages

The README.md file in https://github.com/XEphem/XEphem
points for Ubuntu, Fedora and CentOS to
https://build.opensuse.org/package/show/home:rjmathar/xephem  .
The Fedora link should point to  https://packages.fedoraproject.org/pkgs/xephem/xephem/
instead, because on that operating system the packages has an
official maintainer.

Thanks for pointing out that there was a better link available! I've updated the README.


Fix logging of windows that are too small for window variation check

If window variation check can not be performed for specific stations because there are not enough observations per window, this should be printed to the user.

As regards issue https://github.com/vergauwenthomas/vlinder_toolkit/issues/68, the propagation of the QC settings works as expected. When you change a setting in the qc_settings.py script, it has an effect on the outliers. Changing a QC setting manually (see figure below) also has the desired effect on the QC outliers

[

Okay I will think about it and we can discuss this later.

Yes that is true but if the user provides a window size that is far too small for most of the stations, the window variation check will nevertheless be applied to this stations for no reason and with no output. So that is then computer power that is unnecessary.


⚠ ClamAV has degraded performance

In 5648f14, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 84 ms

Resolved: ClamAV performance has improved in 19e39ad after 5 minutes.


support nest 7

Is it possible to support nest 7?
using npm I get this error
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@nestjs/common
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/nest-next
npm ERR<EMAIL_ADDRESS>from the root project

I can look into this, it doesn't look like this package uses any of the deprecated items so it could be as simple as bumping the versions of the peer dependencies.

It would be nice if update will be soon.

It would be nice if update will be soon.

@georgeadeishvili PRs are welcome!

@georgeadeishvili PRs are welcome!

Thanks you for the effort @kyle-mccarthy
This is a really sweet library

Thanks you for the effort @kyle-mccarthy
This is a really sweet library


Trusted GPG-key should only accept 160-bit fingerprints

Intro
Our approach to accep GPG IDs in verification can be either:

Optimistic:

E.g. short IDs/long IDs
We know the chance of two keys colliding is higher vs. full IDs
Comfortable, as IDs are concise

Pessimistic:

E.g. fingerprints
Minimizes the chance of collision
Can be more unwieldy compared to a shorter identifier

So far, we allowed trusted-keys or simply <gpg value="..." /> to be a short/long ID.
This approach is not the best, as it can leave space for collision attacks - when it comes to trust, we should be as pessimistic as possible.
When it comes to ignored-keys, we can still use optimistic IDs, as accidentally ignoring more - from the security standpoint - is not a concern.
Changes
This PR changes the appropriate builders to fail, if they meet with an optimistic key in either trusted-keys or gpg elements.
Error messages
The building process became a bit more complicated, so we can emit a sensible error message, like:
FAILURE: Build failed with an exception.

* What went wrong:
Configuration cache state could not be cached: field `classpath` of task `:compileJava` of type `org.gradle.api.tasks.compile.JavaCompile`: error writing value of type 'org.gradle.api.internal.artifacts.configurations.DefaultConfiguration'
> Could not resolve all dependencies for configuration ':compileClasspath'.
   > Dependency verification cannot be performed
      > An error happened meanwhile verifying 'org:foo:1.0':
          - The following trusted GPG IDs are not in 160-bit fingerprint format (see: https://docs.gradle.org/current/userguide/dependency_verification.html#sec:understanding-signature-verification):
              - '14f53f0824875d73'

@bot-gradle test QFL

I've triggered the following builds for you:

QuickFeedbackLinux build

@bot-gradle test QFL

Whelp, forgot that this should be automatic since some time...

Sorry I don't understand what you said.
Currently, the following commands are supported:
@bot-gradle test <BuildTrigger1> <BuildTrigger2> ... <BuildTriggerN>

A trigger is a special build for this PR on TeamCity, common triggers are:

SanityCheck/CompileAll/QuickFeedbackLinux/QuickFeedback/PullRequestFeedback/ReadyForNightly/ReadyForRelease
Shortcuts: SC/CA/QFL/QF/PRF/RFN/RFR
Specific builds:

BD: BuildDistributions/BuildDocs so that you can preview the generated docs/distribution.
PT: PerformanceTest, all performance tests for Ready For Nightly stage.
APT: AllPerformanceTest, all performance tests, including slow performance tests.
AST: AllSmokeTestsPullRequestFeedback
AFT: AllFunctionalTestsPullRequestFeedback
ASB: AllSpecificBuildsPullRequestFeedback
ACC: AllConfigCacheTestsPullRequestFeedback
ACT: AllCrossVersionTestsReadyForNightly
AFTN: AllFunctionalTestsReadyForNightly
ACTR: AllCrossVersionTestsReadyForRelease
AFTR: AllFunctionalTestsReadyForRelease

@bot-gradle test and merge

queues this PR for testing and merges if all tests pass by:

Creating a merge commit from your PR branch HEAD and the target branch
Running a ReadyForNightly build against the merge commit
When it passes, fast-forward the target branch to this merge commit (i.e. merge the PR)

The merge commit is called a pre-tested commit, which means that it fully tests the integration
of your branch HEAD and latest master, instead of only testing your branch HEAD.
@bot-gradle cancel

cancel a running pre-tested commit build or remove it from queue

@bot-gradle clean

clear the conversation history

@bot-gradle help

display this message

To run a command, simply submit a comment.  For detailed instructions see here.

@bot-gradle test RFM please

@bot-gradle test and merge

I've triggered a build for you.


[Documentation Missing] Add JSDoc

There are many functions in background.js which are using chrome.cookies API. Adding a short description before each function will make the code more readable.
For example,
/*
 *
 * @description event listener which notifies whenever cookie's properties are updated
 */
function onCookieChanged(){
  chrome.cookies.onChanged.addListener(function(cookies){
      console.log("cookies are being changed ", cookies.cookie.domain);
      console.log(cookies);
  });
}

Add JSDoc for each function.
Do not make any changes to other files.
Write assumptions if any.

Please refer to JSDoc and chrome.cookies API.

Happy to grab this one.

PR is up


powerpipe: 0.4.4 -> 1.0.0

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@NickCao  please approve. Tested on aarch64-linux
Please backport

Interesting how this went through and got green results.

Once this is good, please backport as well. Thank you


Create Ssup

PLEASE REMOVE THIS TEMPLATE BEFORE SUBMITTING
Before submitting a PR, please, make sure that:

Changes are in a separate branch, not in master.
The branch contains only one commit on top of master (if not, squash them into
one commit).

The commit message contains fixes #XXXX or closes #XXXX to auto-close the
issue(s) that the PR fixes (if any).
Tests for the changes have been added (for bug fixes / features).
Documentation has been added / updated (if applicable).
All new and existing tests passed locally (if not, fix them first and amend
the commit).

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing
guidelines.
PLEASE REMOVE THIS TEMPLATE BEFORE SUBMITTING

Considered as intentional spam. Flagged as invalid. Blocked and reported.

Considered as intentional spam. Flagged as invalid. Blocked and reported.


Implement {Add, Div, Mul, Sub}Assign for Gf256

I implemented the {Add, Div, Mul, Sub}Assign because I believe they make the code cleaner and easier to follow. I didn't write any tests because the implementations are very simply piggy-backing on the code you've already written, but am happy to add some if you'd like.
I want to note that the assignment operators are not identical to the non-assignment counterparts because I didn't find any documentation on this. When using the assignment operators first the right-hand side of the <op>= symbol is calculated and then <op> operates on that and the left-hand side. So one needs to be careful order of operations are respected:
fn main() {
    let mut x = 4;
    x *= 2 + 5;
    println!("{:?}", x);

    let mut x = 4;
    x = x * 2 + 5;
    println!("{:?}", x);
}

28
13

This is relevant in https://github.com/SpinResearch/RustySecrets/compare/SpinResearch:3de1689...nvesely:f3f16dc#diff-6dad4b2a3f944968075fe32bae709406R40

Looks good! Thanks!


element-{web,desktop}: 1.7.33 -> 1.7.34

Motivation for this change
https://github.com/vector-im/element-desktop/releases/tag/v1.7.34
Things done
I'm running this on my desktop currently and it's working.

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 132438 at d1ec2fae run on x86_64-linux 1

1 package built successfully:

element-desktop

3 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:334:7:
    |
334 |       installPhase = attrs.installPhase or ''
    |       ^

warning: build-tools-in-build-inputs
yarn is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:294:7:
    |
294 |       buildInputs = [ yarn nodejs rsync ] ++ extraBuildInputs;
    |       ^

warning: build-tools-in-build-inputs
rsync is a build tool so it likely goes to nativeBuildInputs, not buildInputs.
Near pkgs/development/tools/yarn2nix-moretea/yarn2nix/default.nix:294:7:
    |
294 |       buildInputs = [ yarn nodejs rsync ] ++ extraBuildInputs;
    |       ^


.ZIP download

When downloading the repository as a .Zip File as opposed to git clone, a build directory already exists and cmake . .  does not execute correctly.

zip directory does not contain a build directory. I'm assuming this is user error.


Absolute URLs don't work

The URL in my stylesheet url(/images/socialicons.png) is replaced with the path to the stylesheet and the URL as so: url(/css//images/socialicons.png).

Is it fixed now?

It's fixed, thanks for your help.


Remove presto group when checking config files

Some sudoers files don't allow root to run commands as other groups.
The presto group isn't necessary when grepping for config values,
just the user, so we remove it.
Tested on the Teradata Hadoop Appliance and EMR, as well as docker.

LGTM


Vim9: no proper error for defining legacy dictionary function

Steps to reproduce
Run this shell command:
vim -Nu NONE -S <(tee <<'EOF'
    vim9script
    g:Object = {}
    function g:Object.Method()
    endfunction
EOF
)

E1017 is given:
E1017: Variable already declared: g:Object.Method()

Run this other shell command:
vim -Nu NONE -S <(tee <<'EOF'
    vim9script
    g:Object = {}
    def Func()
        function g:Object.Method()
        endfunction
    enddef
    defcompile
EOF
)

E476 is given:
E476: Invalid command: function g:Object.Method()

Expected behavior
It would be easier to understand the issue if the messages told us that we're trying to use a legacy syntax which is no longer supported.
A proper error message telling us what went wrong would help.  For example:
E1234: cannot declare dict function

Version of Vim
8.2 Included patches: 1-4442
Environment
Operating system: Ubuntu 20.04.4 LTS
Terminal: xterm
Value of $TERM: xterm-256color
Shell: zsh 5.8
Additional context
In the help, we could then add a tag for this new error message at :help vim9-no-dict-function.
Although, the latter tag is not about legacy dictionary function.  It's about Vim9 ones.  Still, it might help if the user intend to turn their legacy function into a Vim9 one.  BTW, we can declare a regular legacy function in a Vim9 script:
vim9script
function Legacy()
endfunction

no error

But not if it's a dictionary one.  Is it working as intended?

Probably fixed by patch 8.2.4460.


Error when trying to destroy video.

This Issue is about (delete as applicable)

A bug in the API

API errors must include example code showing what happens, and why you don't believe this is the expected behavior. Issues posted without code take far longer to get resolved, sometimes not ever.
Here is a slightly modified version of this example http://phaser.io/examples/v2/video/dolby-digital-plus#gv
I'm getting this error when I try to destroy the video.
Uncaught TypeError: Cannot read property 'paused' of null
var game = new Phaser.Game(800, 600, Phaser.CANVAS, 'phaser-example', { preload: preload, create: create });

function preload() {
    game.add.text(100, 100, "Loading videos ...", { font: "65px Arial", fill: "#ff0044" });

    game.load.video('dolby', 'assets/video/MP4_HPL40_30fps_channel_id_51.mp4');
}

var video;

function create() {

    video = game.add.video('dolby');
    video.play(true);
    //  x, y, anchor x, anchor y, scale x, scale y
    video.addToWorld(400, 300, 0.5, 0.5, 0.5, 0.5);
    setTimeout(function(){
        video.destroy();
    },5000);
}

Thanks for submitting this issue. We have fixed this, and the fix has been pushed to the dev branch.


Document all the modifiers in the sass directory

While working on #3261, I noticed that not all of the Modifiers were documented, so I added appropriate comments for them.

Thanks for picking up on this and fixing it @alexwlchan !

Thanks @alexwlchan! And let me know if you'd like another issue suggested for you to look at.


Windows exe

Moved image_dir function to the top

The exe file works now, but its subjected to the action of Windows Defender, which considers it a Trojan virus and deletes it on spot

The exe file works now, but its subjected to the action of Windows Defender, which considers it a Trojan virus and deletes it on spot


Nuget Pack issues with regards to dependencies

Please update the documentation to explain that there will be no dependencies in your package when you use nuget pack on a csproj file.

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

ID: 22767e96-d56f-aa81-f1ca-17e256ed617e
Version Independent ID: d7cdc676-2705-3cb1-bcd6-d1637460d5c7
Content: How to publish NuGet symbol packages using the new symbol package format '.snupkg'
Content Source: docs/create-packages/Symbol-Packages-snupkg.md
Product: nuget
GitHub Login: @cristinamanum
Microsoft Alias: cristinamanu

@LyonBlecher - can you elaborate?
when you pack a csproj (using dotnet.exe of msbuild t:/pack), and if that project has NuGet dependencies they should show up as the dependencies of the package you just created.

Hi Karan,
Not when you use dotnet.exe or msbuild.exe but nuget.exe pack,  I
implemented a powershell script to iterate through my projects pack them,
to include my snupkg.  Everything worked great until I referenced my new
nuget packages from my local feed and none of them had any of their package
dependencies.  I realised, after some googling that this is a known problem
with nuget pack and the new PackageReference implementation.  Please note
it does work apparently with packages.config implementation.  See the
following link for further info.  Just mentioning that nuget pack has some
unmentioned issues or side effects.
https://github.com/NuGet/Home/issues/5979
On Tue, Apr 2, 2019 at 3:32 AM Karan Nandwani<EMAIL_ADDRESS>wrote:

@LyonBlecher https://github.com/LyonBlecher - can you elaborate?
when you pack a csproj (using dotnet.exe of msbuild t:/pack), and if that
project has NuGet dependencies they should show up as the dependencies of
the package you just created.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NuGet/docs.microsoft.com-nuget/issues/1372#issuecomment-478808601,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEWNqbvKi-WaaAoGvvSoQiScyU6sd7I4ks5vcrM2gaJpZM4cVssg
.

--
Kind Regards,
Lyon Blecher

@LyonBlecher This sounds like a project migrated to PackageRef. The current recommendations are to use msbuild -t:pack.
https://docs.microsoft.com/en-us/nuget/reference/migrate-packages-config-to-package-reference#create-a-package-after-migration

I noticed that newly added PackageReferences are missing in the projects References list, but they show up as soon as I had build the project. (VS 2019 16.1.6). The same happens after opening a project that has just been checked out into an empty workspace. It seems that some discovery task needs to complete for the PackageReferences to become visible.

@FrankHeimes That sounds like a PackageRestore operation, which can happen automatically on build.

Closing this as I believe correct usage for this scenario is msbuild -t:pack