evcxr_jupyter support

Hi,
Does plotpy support evcxr_jupyter ? If so, it would be very convinience !  Plotters and plotly.rs support evcxr_jupyter now.

Hi, unfortunately, evcxr is not supported.

Jupyter notebook is very popular in data science and data visualization, it's hard to abandon it for lots of users. Plotpy can get more attractive for people getting used to jupyter if it can support it.
Now I get an idea on how to use plotpy in evcxr_jupyter. Here is the code:
:dep plotpy
use plotpy::{generate3d, Contour, Plot, StrError};
use std::fs;
use std::io;

// Function to show the saved figure in evcxr_jupyter
fn show(figure_path: &str) -> Result<(), io::Error> {
    let figure = fs::read_to_string(figure_path)?;
    println!("EVCXR_BEGIN_CONTENT text/html\n{}\nEVCXR_END_CONTENT", figure);
    Ok(())
}

// Set python path to integrate with evcxr later
let python = "/home/wangjw/programs/miniconda3/envs/py311/bin/python";
// Set the file path and name to be saved
let file = "figure.svg";

let n = 21;
let (x, y, z) = generate3d(-2.0, 2.0, -2.0, 2.0, n, n, |x, y| x * x - y * y);

// configure contour
let mut contour = Contour::new();
contour
    .set_colorbar_label("temperature")
    .set_colormap_name("terrain")
    .set_selected_level(0.0, true);

// draw contour
contour.draw(&x, &y, &z);

// add contour to plot
let mut plot = Plot::new();

// We may have various (ananconda) python environment, but there's not currently any
// integration with evcxr, so we must set python path in order to use the suitable
// python environment
plot.set_python_exe(python);
plot.add(&contour);
plot.save(file)?;
show(file)?;  // impl this method for plot later, then we call just call plot.show()

And here is the running result:

If you think it's feasible, I will try to send you a PR ("try to" because I am a novice of Rust and have few experience on github PR and Rust).

There are alread many Rust visualization crate yet (plotters, plotly.rs, chart-rs, charming and so on), but all of them are lack of maintenance. I don't think it's feasible to construct a independent visualization ecosystem for Rust. Maybe It's a good way to take advantage of some mature visualization tools in other languages, just like rust plotpy and hope you can add more plottng function. I'll help to enhance this crate once I am familiar with Rust and plotpy and have the ability to contribute.

Hi, thank you. This looks fantastic! But you mentioned the keyword here maintenance. Since I'm unfamiliar with evcxr, maintaining such integration would be very difficult. Also, you mentioned that you're getting more familiar with Rust; therefore, it may be time-consuming for you. Nonetheless, if the only required implementation is to call another function like your show function, we may be able to do it.

Evcxr is an evaluation context for Rust under active maintenance by David Lattimore, so don't worry about Evcxr. At present, it appears that the only thing need do is the show function. I'll start to implement it this week.

Hi,
Thanks again for the suggestion and implementation. The function show_in_evcxr is now available in version 1.4.0
Can you please try it to see if it works in your system?

It worked in my windows10 desktop and centos7 server.

Hi, thanks again for the code and suggestions to Readme. I've submitted another PR with the Readme changes.
Now, I'm wondering: would it be better if show_in_evcxr saved the figure already? So, we don't need to call save?
This is how the save_and_show function works. The user could forget to save or use a different filename, so I enforced saving and showing at the same time.
For instance, we could save and show in Jupyter (is this a better name? because we are effectively showing in Jupyter, right? I don't know LOL. I guess evcxr is just a middleman...):
Instead of:
plot.save(...)?
plot.show_in_ivcxr(...)

We would have:
plot.save_and_show_in_jupyter(...)?

I don't have any strong opinions. Naming things is hard.
I assume some of the crates wouldn't have save in the name because they don't need to save a file in order to show it - i.e. they just print out the image data directly to stdout without creating any files.
I assume saving is necessary due to the way plotpy works. If the image were saved to a temporary file that's subsequently deleted, then that seems somewhat like an implementation detail, so maybe wouldn't need to be in the method name.

Thank you both for the feedback.
It makes sense to simply call show(), knowing that a temporary file is created.
Now, on version 1.6.0, we have two functions (one existed already and has been renamed):

show(...) which shows the figure using Python/Matplotlib standard backend
show_in_jupyter(...), which shows the figure in Jupyter via evcxr

Both functions save a temporary file automatically, so the user won't need to call save beforehand.

I see it! I tested it and it worked.
And here is a typo in line 1131 (src/plot.rs), evcxr should be jupyter now (and also in the readme).

readme:

And .save(path)? in the code is not needed now.
// set the python path
let python = "where-is-my/python";

// set the figure path and name to be saved
let path = "my-figure.svg";

// plot and show in a Jupyter notebook
let mut plot = Plot::new();
plot.set_python_exe(python).set_label_x("x").set_label_y("y");
plot.show_in_evcxr(path)?;

The figure will be saved in the path automatically when showing.

Thank you. This has been fixed in version 1.6.1


lk2nd cannot boot U-Boot from thirdstage.ext2 flashed to boot partition

Following the README steps does not work for me on my samsung-a5:
fastboot flash boot build/thirdstage.ext2

lk2nd log
[10] platform_init()
[10] target_init()
[20] MMC card: R311MB (0.0, 02 1999), manufacturer: 15, OEM: 100, capacity:<PHONE_NUMBER>8 bytes
[30] SDHC Running in HS200 mode
[30] Done initialization of the card
[40] lk2nd_init()
[40] Booted @ 0x80008000, r0=0x0, r1=0x0, r2=0x81e00000
[40] Found valid DTB with 2679 bytes total
[50] Command line from previous bootloader: lk2nd androidboot.sec_atd.tty=/dev/ttyHSL0 loglevel=4 sec_log=0x100000@0xafe00008 sec_dbg=0x80000@0xaff00008 androidboot.revision=2 androidboot.warranty_bit=1 androidboot.bootloader=A500FUXXU1CRH1 lpm_levels.sleep_disabled=1 fg_reset=0 androidboot.boot_recovery=0 sec_debug.reset_reason=0x0 androidboot.debug_level=0x4f4c sec_debug.enable=0 sec_debug.enable_user=0 androidboot.cp_debug_level=0x55FF sec_debug.enable_cp_debug=0 vmalloc=340M cordon=1d6857fcf41f3591b612de318be7096f connie=SM-A500FU_OPEN_EUR_1ec4fee05502ae748df4adb3c47ca77c lcd_id=0x822003 androidboot.security_mode=1526595584 androidboot.emmc=true androidboot.serialno=2508c401 androidboot.mode=charger androidboot.baseband=lpm lcd_id=0x822003 mdss_mdp.panel=1:dsi:0:ss_dsi_panel_EA8061V_AMS497EE01_HD:1:none
[120] Failed to find matching lk2nd device node: -1
[130] MDP continuous splash detected: pipe RGB_0, base: 0x8e000000, stride: 2160, src: 720x1280 (0,0), img: 720x1280, out: 720x1280 (0,0), format: 0x2243f (bpp: 3)
[140] Display refresh: cmd mode: 0, auto refresh: 0 (sel: 0x1f20)
[150] Registering wrapper bio devices...
[160] Registering mmc_sdhci bio devices...
[170] Error: Command timeout error
[170] Failure getting OCR response from MMC Card
[170] MMC card failed to respond, try for SD card
[180] Error: Command timeout error
[180] The response for CMD8 does not match the supplied value
[180] Failed to initialize SD card
[190] Failed detecting MMC/SDC @ slot2
[190] SD card MMC is unavailable.
[190] block devices:
[200]  | dev        | label           | size       | Leaf |
[200]  | wrp0p27    | lk2nd           |    512 KiB | Yes  |
[210]  | wrp0p26    | userdata        |  12063 MiB | Yes  |
[210]  | wrp0p25    | hidden          |    180 MiB | Yes  |
[220]  | wrp0p24    | cache           |    200 MiB | Yes  |
[220]  | wrp0p23    | system          |   2400 MiB | Yes  |
[230]  | wrp0p22    | persdata        |      9 MiB | Yes  |
[230]  | wrp0p21    | persist         |      8 MiB | Yes  |
[240]  | wrp0p20    | ssd             |      8 KiB | Yes  |
[240]  | wrp0p19    | fsc             |      3 MiB | Yes  |
[250]  | wrp0p18    | backup          |      6 MiB | Yes  |
[250]  | wrp0p17    | fota            |      5 MiB | Yes  |
[260]  | wrp0p16    | recovery        |     15 MiB | Yes  |
[270]  | wrp0p15    | boot            |     12 MiB | Yes  |
[270]  | wrp0p14    | modemst2        |      3 MiB | Yes  |
[280]  | wrp0p13    | modemst1        |      3 MiB | Yes  |
[280]  | wrp0p12    | efs             |     14 MiB | Yes  |
[290]  | wrp0p11    | param           |     10 MiB | Yes  |
[290]  | wrp0p10    | pad             |     10 MiB | Yes  |
[300]  | wrp0p9     | sec             |     16 KiB | Yes  |
[300]  | wrp0p8     | fsg             |      3 MiB | Yes  |
[310]  | wrp0p7     | qhee            |    512 KiB | Yes  |
[310]  | wrp0p6     | qsee            |    512 KiB | Yes  |
[320]  | wrp0p5     | rpm             |    512 KiB | Yes  |
[320]  | wrp0p4     | aboot           |      2 MiB | Yes  |
[330]  | wrp0p3     | ddr             |     32 KiB | Yes  |
[330]  | wrp0p2     | sbl1            |    512 KiB | Yes  |
[340]  | wrp0p1     | modem           |     57 MiB | Yes  |
[340]  | wrp0p0     | apnhlos         |     15 MiB | Yes  |
[350]  | wrp0       |                 |  15028 MiB |      |
[360] boot: Trying to boot from the file system...
[360] Loading (boot) image (4196352): start
[390] Loading (boot) image (4196352): done
[390] use_signed_kernel=0, is_unlocked=1, is_tampered=0.
[400] Uncpmpressed kernel in use
[400] Kernel image not patched..Unable to locate dt offset
[410] DTB offset is incorrect, kernel image does not have appended DTB
[410] Device info 0x000000ce/00010000/0x00010008/0, pmic 0x2000b/0x0/0x0/0x0
[420] ERROR: Appended Device Tree Blob not found
[420] ERROR: Could not do normal boot.  Total time: 0.002s

I worked around this by wrapping U-Boot in its own boot.img for now:
$ cat build/u-boot-nodtb.bin.gz build/linux/arch/arm64/boot/dts/qcom/msm8916-samsung-a5u-eur.dtb > u-boot.bin
$ ~/src/lk2nd/lk2nd/scripts/mkbootimg \
        --kernel=u-boot.bin \
        --output=u-boot.img \
        --cmdline="lk2nd.pass-simplefb=uboot" \
        --base=0x80000000

Apparently it was my misunderstanding of fastboot usage :S
contrary to what seems obvious, flash sends image as-is, and flash:raw creates a brand new boot.img before flashing it, with the kernel used from the first argument....
caf lk then has a check for partition name to only allow boot.img to be flashed into boot. Seems like I always flashed combined image with stock fastboot so I've never hit this problem :S
Worked around in https://github.com/msm8916-mainline/lk2nd/commit/00b9426cc2ed6155fecceff5a62129dec80c5bbe for now.

I don't think this has fixed the issue yet. I just tried the latest rebase/uboot (rev 00b9426cc) and I'm still having the same problem.
I tried both fastboot flash boot build/thirdstage.ext2 and fastboot flash:raw build/thirdstage.ext2. In both cases I saw essentially the same log output as in the original issue description.


Fix arctic command ReplaceArcticData

Why are the changes needed?

In arctic command ReplaceArcticData, Identifier is of type Option and cannot be obtained in the original way
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #3561 (0b9ad73) into master (ef55e4a) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##             master    #3561   +/-   ##
=========================================
  Coverage     51.68%   51.69%
  Complexity       13       13
=========================================
  Files           482      482
  Lines         26950    26950
  Branches       3761     3761
=========================================
+ Hits          13930    13931    +1
- Misses        11664    11665    +1
+ Partials       1356     1354    -2

Impacted Files
Coverage Δ

.../kyuubi/plugin/spark/authz/PrivilegesBuilder.scala
69.32% <0.00%> (ø)

...rg/apache/kyuubi/ctl/cmd/log/LogBatchCommand.scala
78.00% <0.00%> (-2.00%)
:arrow_down:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
80.74% <0.00%> (-0.63%)
:arrow_down:

...g/apache/kyuubi/operation/BatchJobSubmission.scala
74.64% <0.00%> (ø)

...yuubi/server/metadata/jdbc/JDBCMetadataStore.scala
89.96% <0.00%> (+0.69%)
:arrow_up:

.../org/apache/kyuubi/session/KyuubiSessionImpl.scala
77.65% <0.00%> (+1.06%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Can you provide a test case ?

cc @bowenliang123 Any idea about how to better integrate with other spark catalog extensions ?

As the org.apache.kyuubi.plugin.spark.authz.v2Commands and  IcebergCommands, they are implemented and attempt to adapt a general pattern to intercept logical plan. So far, I think the Arctic could follow the same pattern as IcebergCommands to do it. For further plan, if the pattern is generalized well enough, it's ok to provide a pluggable implementation through SPI mechanism to use custom auth mapping plugin.


TextField with rounded corners - floating label shows extra whitespace

This is a similar issue as #33948.
I need my TextField (or TextFormField) to have rounded corners in order to match my company's design:
(picture taken from Vue.js application)

I successfully achieved it by setting all the borders of the TextField's InputDecoration to:
OutlineInputBorder(
  borderRadius: BorderRadius.circular(32.0),
)

However, the problem I face is that the floating label (which is especially needed for the desired design) is looking strange in my Flutter app:

It has too much whitespace around it. When decreasing the border radius e.g. to 8.0, the floating label displays fine. But I need it to be 32.0 in order to be completely round.

Code to reproduce it
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  final border = OutlineInputBorder(
    borderRadius: BorderRadius.circular(32.0),
  );

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        appBar: AppBar(title: Text("TextField test")),
        body: Container(
          alignment: Alignment.center,
          padding: const EdgeInsets.all(16.0),
          child: TextField(
            obscureText: true,
            decoration: InputDecoration(
              labelText: "Password",
              focusedBorder: border,
              enabledBorder: border,
              errorBorder: border,
              border: border,
              disabledBorder: border,
              focusedErrorBorder: border,
            ),
          ),
        ),
      ),
    );
  }
}

flutter doctor
   Doctor summary (to see all details, run flutter doctor -v):
  [✓] Flutter (Channel stable, 2.0.6, on macOS 11.3.1 20E241 darwin-x64, locale en-GB)
  [✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
  [✓] Xcode - develop for iOS and macOS
  [✗] Chrome - develop for the web (Cannot find Chrome executable at /Applications/Google
      Chrome.app/Contents/MacOS/Google Chrome)
      ! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
  [✓] Android Studio (version 4.2)
  [✓] VS Code (version 1.56.0)
  [✓] Connected device (1 available)

  ! Doctor found issues in 1 category.

Thanks for the detailed report and code sample. I see the same behavior.

flutter doctor -v

[✓] Flutter (Channel master, 2.3.0-2.0.pre.139, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-GB)
    • Flutter version 2.3.0-2.0.pre.139 at
      /Users/dhs/documents/fluttersdk/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e621ebfb34 (3 hours ago), 2021-05-11 23:29:03 -0400
    • Engine revision 8e332453b8
    • Dart version 2.14.0 (build
      2.14.0-edge.b99466d472e369990eb22de332cb5b2967b3ea00)

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see
      https://guides.cocoapods.org/using/getting-started.html#installation for
      instructions.

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

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

[✓] Connected device (3 available)
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 90.0.4430.212

! Doctor found issues in 1 category.

Cc: @justinmc

This has come up before... I see https://github.com/flutter/flutter/issues/28862#issuecomment-469783029 but I can't remember where else.

Like I mentionned in my duplicate issue, if you have a suffixIcon (even if it's an empty one), the label displays fine despite 32 borderRadius. Really strange issue.

With https://github.com/flutter/flutter/pull/90157, you'll be able to set the alignment of the floating label.  This could help adjust the position to a place that doesn't overlap with the curvature of the border.
Currently the PR only allows centering, but we could make it allow arbitrary positions.  Would that help solve the problem?
Here's how this rounded border looks like with a centered label:

With #90157, you'll be able to set the alignment of the floating label. This could help adjust the position to a place that doesn't overlap with the curvature of the border.
Currently the PR only allows centering, but we could make it allow arbitrary positions. Would that help solve the problem?
Here's how this rounded border looks like with a centered label:

No, that would not.

That's fair, I'll advise https://github.com/flutter/flutter/pull/90157 not to allow arbitrary positions.
I'll chime in a bit later about the background gap in the border.

@WieFel @mat100payette lemme know if this hack helps.

code
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    final border = OutlineInputBorder(
      borderRadius: BorderRadius.circular(32.0),
    );
    return MaterialApp(
      themeMode: ThemeMode.dark,
      home: Scaffold(
        appBar: AppBar(title: Text("TextField test")),
        body: Container(
          alignment: Alignment.center,
          padding: const EdgeInsets.all(16.0),
          child: TextField(
            obscureText: true,
            decoration: InputDecoration(
              contentPadding: EdgeInsets.fromLTRB(28, 20, 12, 12),
              labelText: "Password",
              focusedBorder: border,
              enabledBorder: border,
              errorBorder: border,
              border: border,
              disabledBorder: border,
              focusedErrorBorder: border,
            ),
          ),
        ),
      ),
    );
  }
}

@werainkhatri Although it does keep the label's background constant in terms of size, if forces the textfield's content to be padded too, plus the label isn't fully to the left of the field because of that same padding. So that's not really helpful sadly.

@mat100payette @WieFel could you confirm this is what you expect? Thank you for your feedback.

@mat100payette @WieFel could you confirm this is what you expect? Thank you for your feedback.

@werainkhatri Yes, given that the label background you show in that image is invariant wherever it's located.

This may be a bug in the border gap calculation.  I'm seeing if I can come up with a fix here: https://github.com/flutter/flutter/pull/92940

This may be a bug in the border gap calculation. I'm seeing if I can come up with a fix here: #92940

Yes there is. I have found it and fixed it for ltr. Am currently working on rtl too.

Good stuff guys. Just a quick question, will @justinmc 's PR gibe the same result as the last (correct) image @werainkhatri posted ? Because in the PR, the suggested "correct result" isn't exactly that, and it is in fact Not correct. Just making sure here.

Good stuff guys. Just a quick question, will @justinmc 's PR gibe the same result as the last (correct) image @werainkhatri posted ? Because in the PR, the suggested "correct result" isn't exactly that, and it is in fact Not correct. Just making sure here.

Yes it will. I have reviewed the PR and it has the identical changes I made in order to produce the above image.

@HansMuller I've unassigned you.  I didn't realize you were assigned when I started working on this, but looks like it's an old assignment so I assume we're good...

Reopening due to revert.  This will go in again in https://github.com/flutter/flutter/pull/93933.


resolved errors related to unicode, iteritems and AS400Reflector obje…

resolved errors related to unicode, iteritems and AS400Reflector object has no attribute get_unique_constraints

@sidnaik03 Please rebase your branch to latest level and resolve the conflicts. Most of the changes may be done already. Thanks.

Code change suggested by this pull request has been already merged by Abhinav as per this comment. Hence, closing the pull request. Thanks.


⚠ ESPN has degraded performance

In 3d21daf, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 656 ms

Resolved: ESPN performance has improved in 0694e39.


feat: prompt user to install Japa and its AdonisJS preset

Proposed changes
As proposed here : https://github.com/adonisjs/core/discussions/3709
     _       _             _         _ ____
    / \   __| | ___  _ __ (_)___    | / ___|
   / _ \ / _` |/ _ \| '_ \| / __|_  | \___ \
  / ___ \ (_| | (_) | | | | \__ \ |_| |___) |
 /_/   \_\__,_|\___/|_| |_|_|___/\___/|____/

CUSTOMIZE PROJECT
....
> Setup Japa ( testing framework ) and its preset for AdonisJS? (y/N) · true

Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[ ] Bugfix (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 not work as expected)

Should we have yes as a default instead of no?

Totally agree ! Done

Btw, I would also be in favor to have a "yes" by default on the installation of prettier and eslint, but it's another subject

Do we really want to do this?
What is the alternative to Japa right now when it comes to testing AdonisJS apps?
If you use Jest, you cannot go too far, because there are no helpers to login users during HTTP requests, no helpers to easily migrate/rollback database.

It should be possible to use TestUtils from another test/runner, no?
For example migrations, from what I see it seems possible ( tell me if I'm wrong )
For Auth on the other hand it seems more complicated because it depends on Japa. But the user should be able to create his own helpers to facilitate his tests or it is simply impossible?

For Auth on the other hand it seems more complicated because it depends on Japa. But the user should be able to create his own helpers to facilitate his tests or it is simply impossible?

It's possible. But no one has done it so far, so I am not sure what are optimizing for when making the Japa test runner optional.
If someone comes with a good Jest implementation, then yes we can make it optional. But right now, we are doing it just for the sake of modularity.

Now someone might ask, well what's the issue in being modular?
The issue is decision fatigue. Why are we making others select/ignore packages which does not have a good alternative anyways. Infact, I was thinking to pre-install lucid and auth for api and web boilerplates

Here is a small proposal. I think discuss it in long term too (if required)

The web and api boilerplate to always come with lucid and auth
We can drop the tests runner from the slim boilerplate and keep it for those who wants to configure every bit by hand

Again, the reason others use AdonisJS because it comes with everything you need to build an end to end application. And minimalists anyways drop off and use express or something similar

Okay, I understand and I even agree.
I have no argument for this PR except that it is indeed more "modular", but as you say, there is no serious alternative, so it is not really relevant. I jumped on the PR without thinking about it when I saw Romain's ping 😅
I also agree to integrate lucid and auth by default. But I think that, more widely, it is worth discussing my proposal here: https://github.com/adonisjs-community/create-adonis-ts-app/issues/43
IMO, the best solution would probably be to propose the installation of these packages during create-adonis-ts-app. In my case, most of my AdonisJS applications use mailer/lucid/auth/Redis/drive.
I like this full-featured side of AdonisJS, and it would be great to have all the tools I need in one command line.
Today, as it is, I'll have to do npm install && node ace configure on all the packages I'll need, and it's a bit boring.

I like this full-featured side of AdonisJS, and it would be great to have all the tools I need in one command line.

Yup, I agree. We can have a default set of packages and then additional one's can be configured with CLI flags.
Let's see if @RomainLanz has something else in mind that we cannot see.
My take is to make the initial boilerplates even more opinionated vs making them modular.

@thetutlage I would say you have somewhat of a point for now, but we have already been using our own custom runners (using just japa/core)
If the runner can at least be dropped from the slim version as you mentioned before, then that would be good enough for me.
Personally, I was drawn to AdonisJS since it isn't as opinionated as Laravel and with much less boilerplate.

Sorry for the late reply.
I like the idea to be able to customize the application the way we want during the scaffolding process.
Some people may not want to have the testing framework at all.
IMO, removing it from default installation is the way to go. We stay opinionated and modular, we don't force people to use our module, we give them the opportunity to use whatever they want.

I like the idea to be able to customize the application the way we want during the scaffolding process. Some people may not want to have the testing framework at all.

This will hamper the experience for the once that needs a fully featured scaffold by default (that is the majority of users). Just making them answer 7-8 prompts.
I will keep the web and api boilerplates highly opinionated and keep the slim one as slim and minimalists can go with the slim one

We could keep the three default boilerplates and add a "custom" next to them? When the user selects "custom" instead of "web/api/slim" then at that point we do prompts the user what they want to install?

This will hamper the experience for the once that needs a fully featured scaffold by default (that is the majority of users). Just making them answer 7-8 prompts.

I am not sure about that, but if it is a concern, we could add a --customize that asks before installing dependencies; if this option is not provided, then we use the "default"  for each boilerplate. What do you think?

We could keep the three default boilerplates and add a "custom" next to them? When the user selects "custom" instead of "web/api/slim" then at that point we do prompts the user what they want to install?
Trust me, if we do a poll today, then 90% will be in the favor of just install lucid, auth and japa by default. Yes, 10% needs more control. But there is no way we can cater 100% of people with our choices.
Infact, we can drop slim and add custom and let the user cherry pick every small bit

I am not sure about that, but if it is a concern, we could add a --customize that asks before installing dependencies; if this option is not provided, then we use the "default" for each boilerplate. What do you think?

Not a bad idea

But I agree; when people select a boilerplate, it looks like everything should be set up, and no question should be asked.
Now the question would be: what happened with prettier& eslint or with Encore?

Now the question would be: what happened with prettier& eslint or with Encore?

Yeah, these are tricky one's and I think I have some reasoning behind them (atleast I have told this to myself :) )
Prettier and EsLint both are not part of the framework, so if I don't use the settings suggested by the framework, then it is very easy for me to install them afterwards with my own/team settings. Also, here the chances of using our defaults are low. If, I am part of a team, I will prefer to use the team settings.
Regarding Encore. Yes, let's install it by default. One less question to answer

So, to recap:

Boilerplate web:

Lucid
Repl
Session
View
Auth
Japa
Bouncer
Shield (?)
Mailer (?)

Boilerplate API :

Lucid
Repl
Session
Auth
Japa
Bouncer
Shield (?)
Mailer (?)

"Boilerplate" Custom comes only with core/repl and we prompts the user for installing every AdonisJS packages, so :

Lucid
Repl
Session
Auth
Japa
Bouncer
Shield
Mailer
Ally
Drive
i18n
Encore
Redis

The three boilerplate ask the user if he wants to install eslint /prettier
That are going to be a lot of prompts for the custom version, but personally, I'm totally okay with that, and that will be my default choice for all my future applications because I will not have to do x npm install x && node ace configure x. And I can also avoid installing packages that will be useless in my case. For example, I could very well make an API without Lucid
Now I'm not sure I understand the interest in the --customize flag proposed by Romain. If I select the web boilerplate with this flag, finally it's like using the "custom" boilerplate? So what would be the point? The questions will have a default answer of "yes" in case the package is integrated into the boilerplate in question?

Now I'm not sure I understand the interest in the --customize flag proposed by Romain. If I select the web boilerplate with this flag, finally it's like using the "custom" boilerplate? So what would be the point? The questions will have a default answer of "yes" in case the package is integrated into the boilerplate in question?

I am not sure it would be useful with the tailored boilerplate, at least, not at the beginning with the amount of packages we have.
The goal of that flag would be to customize the selected boilerplate, but we can forget about it for the moment.

@Julien-R44 Yup, I am onboard with the choices

Instead of having a y/n choice, what do you think of having a select with some options pre-selected?

Instead of having a y/n choice, what do you think of having a select with some options pre-selected?

For the slim boilerplate?

Instead of having a y/n choice, what do you think of having a select with some options pre-selected?

For the slim boilerplate?

Yes, for the custom/tailored/slim

Yup. That does make sense

Ok cool guys, thanks !
I'll add this on my todo list and try to propose something in the next weeks


exec: "absolute\path\to\hub.exe.exe": file does not exist

Command attempted:
git help <hub-command>
What happened:
I set git as an alias for hub with Set-Alias -Name git -Value "absolute\path\to\hub.exe". And it works. But only the above command caused this problem.

Reload $PROFILE and run git help <hub-command>

Windows 8.1 x64
git version 2.22.0.windows.1
hub version 2.13.0

This works for me:
Set-Alias -Name git -Value "C:\Users\Mislav\AppData\Local\GitHubCLI\bin\hub.exe"

git help fork

Can you paste the exact git help ... command you've ran and the error message? You can scrub sensitive directory names from the path if needed

This is what I put in my $PROFILE:
Set-Alias -Name git -Value "$env:Bin\github\bin\hub.exe"

Thank you! You've helped me find the bug. 🙇


Dependabot automerge doesn't wait for tests to pass

I think we need to enable "Require Status Checks Before Merging" in the repo settings.
The alternative is to use the Wait For Status action.
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-protected-branches/about-protected-branches#require-status-checks-before-merging
https://stackoverflow.com/questions/64116781/how-do-i-automerge-dependabot-updates-config-version-2

Pinging @christian-bromann as I don't have access to this repo setting.

@goosewobbler I gave you all repository access, let me know if you can make all necessary changes.

Thanks, resolved.


fix: artificial change to trigger version update

Previous master build failed because I had once accidentally published v1.0.0 on NPM and cannot remove. So need to trigger another build to update to next version.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   94.80%   94.80%
=======================================
  Files           3        3
  Lines          77       77
  Branches       22       22
=======================================
  Hits           73       73
  Misses          4        4

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 25099ee...08dbe75. Read the comment docs.


Fix a typo in the zsh completion

I accidentally fired this miniature footgun earlier. Serves me right for being too hurried to double-check the manpage:-)

Thanks!


Added instagram icon as one of the event social link

Partially fixes the issue #1345
Changes:

Instagram link was not shown as one of the event links. Added it to the web app.

Screenshots for the change:

There is an Instagram link at the top-right corner.
Test Server
https://secure-meadow-20680.herokuapp.com

@aayusharora The tests are failing because the sample used for testing currently is FOSSASIA 16 which didn't contain the Instagram link. No, the control never reaches the added lines of code. After the PR #1342, related to the generation of different events, this added part of the code will be covered and the code coverage will increase :)


Crash when select string include '(' char.

I am searching a "script" node in facebook html source, the node is like
<.s.c.r.i.p.t.>require("TimeSlice").guard(function() ... ///< The dots in "script" is for showing this line normally in issue page.
So I use this selector to find this node
CSelection c = doc.find("script:contains(require("TimeSlice"))");
But, the app crashed with error "terminate called after throwing an instance of 'std::string'", GDB says it crash in doc.find function.
If I use CSelection c = doc.find("script:contains(require)"), it works well. But these nodes are not what I want. So, I think gumbo-query's "contains" filter does not support '(' in it.

No, gumbo-query does not properly parse string sequences like this. Once the parser encounters a "special" character like "(" or even """, it will discard whatever previous operation it was tasked with doing (such as parsing a string) and change its task based on the newly discovered "special" character.
I'm not criticizing @lazytiger when I criticize stuff like this, just to be clear. He said he blindly ported cascadia over without doing too much testing. The problem is that cascadia has lots of bugs, lots of them. This is one such example, improper string parsing.


Replace loop with cl-loop

Fixes #1.
Tested on Emacs 27.1. Autloads at least back to the Emacs-24.4 that's required.

Looks sensible. I'm using this PR and it works fine.

Sorry for having missed this.  I fixed this myself.


Examine, how to properly cancel 1 task within anyio.TaskGroup

May be related to:
https://github.com/agronholm/anyio/issues/374
https://github.com/agronholm/anyio/pull/496

In websocket endpoint, we should pass task_id from outer scope (on_release), so that user can re-define it and have possibility to call task_group.start instead of task_group.start_soon


Add header background

At the moment, the header background is a colour. Have an option to add an image that's easy to swap to something else and works on multiple screen sizes.

Taking this issue

@wilkesalex Could you explain this issue in a bit more detail? I am not able to get add an image and the swap part.


GARCH model implementation

Rachel, could you please also add the GARCH model to the environment as a function and run the t-tests on the resulting time series?
Follows example implementation:
https://colab.research.google.com/drive/1NUP5kmQzNOQjj1GVG6TtiIN4fmi_qNyH#scrollTo=tJY1NUYhiiuU

What are the start and end values for a series for <component,failure>?

The start is could be the minimum value of the original series, e.g., Min_accuracy_increase(<C1,F2>)
You might also need the initial variance, which could be the variance of the original series.
Do we need to define the end? I think we only need to define the coefficients for the Garch model.

No I do not understand the variables of the function - how is this influenced by <component,failure>?

In this code here https://colab.research.google.com/drive/1NUP5kmQzNOQjj1GVG6TtiIN4fmi_qNyH#scrollTo=tJY1NUYhiiuU
I believe that we only need to change the value of t[0]. All the rest can stay the same. t[0] can be set be the minimum value of accuracy_increase of <component,failure> pair.

Okay I will set t[0].
But I would use the mean value of the transformed and shifted pair ?!?

Yes. Since the series produced by the Garch model fluctuates around a mean, it would make sense to use the mean of the transformed_shifted pair.

Running the ttest, which ordering to take?
If I use the ordering from before working on the data, I get different results as if I do this based on the ordering retrieved from the new data.


Where can I disclose a security bug?

On the Polyfill.io site it mentions the following at https://polyfill.io/v3/report-a-bug#security-bugs.

Polyfill.io has a bounty program for the safe disclosure of security bugs. With that in mind, please do not file public issues; go through the process outlined on that page.

But it's not actually clear where the mentioned page is.
A direct link to the HackerOne setup would be helpful.

any news ?

@revolunet Thank you for the reminder, we haven't yet opened up our bug bounty program to the public yet, if you have a security question or issue to report, please email<EMAIL_ADDRESS>I will update the website to say this as well :-)

@JakeChampion The page still doesn't include either a link to a separate page or the email address to use.
(no I don't have the need for it at this time, just thought I should mention it when I stumbled upon this issue)


GLIBC 2.38 on Ubuntu 22 and less

I built tdlib binaries with build instructions for Ubuntu 22 on Ubuntu 22 machine. It works fine for Ubuntu 24.
But when I build for instruction Ubuntu 22 with two options (g++/clang) any result binary after run java program print that lib requires GLIBC 2.38 (glibc 2.38 not found).
Is it expected? Ubuntu 22 contains only GLIBC 2.35. Should I locally build GLIBC with excepted version?

Binaries aren't portable. You have to build them on Ubuntu 22 to guarantee that they work there. They can't require glibc 2.38 if build using the build instructions on the system that has only glibc 2.35.

I tried again. I build new libtdjni.so with instruction for Ubuntu 22 with g++. I build it on Ubuntu 22.04 jammie machine. And call new .so in System.load(.so file) in java application print error:
java.lang.UnsatisfiedLinkError: /home/administrator/tgclinet/sample-linux/libtdjni.so: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.38' not found (required by /home/administrator/tgclinet/sample-linux/libtdjni.so)
        at java.base/jdk.internal.loader.NativeLibraries.load(Native Method)
        at java.base/jdk.internal.loader.NativeLibraries$NativeLibraryImpl.open(NativeLibraries.java:388)

For me it's a bit curious because i waited result as you said by default. But it's not


There is an undocumented option that can be applied to FTP

The option "rejectUnauthorized" for connections using the node-ftp plugin is not mentioned in the documentation.
This option would normally be inside a "secureOptions" object (which doesn't work). I was looking into that bit of code to try and implement a workaround and found exactly the functionality I needed built in!
If you let me know the best place to mention this in the docs I would be to make the changes required :)

@carlwhittick
Sorry that I answer so late!
Feel free to fork the repository, make the required changes there and send it back as pull request.
Btw.: Can you post this "workaround" here or some code that demonstrates the issue?


JsonPath matching does not work anymore

Expected Behavior
jsonPath $..[?(@.Name.match('Doe'))] the result should only show the 2 doe entries
[
  {
    "Id": "c6cb074b-e4eb-8534-d659-be2d2c1a29a0",
    "Name": "John Doe 123",
    "Serial":<PHONE_NUMBER>,
    "DeviceEnergyType": 1,
    "MeterSubType": 3,
    "FamilyType": 1001,
    "ActivePower": 0.0,
    "ActivePowerUnit": "kW",
    "CounterReading": 1545.62,
    "CounterReadingUnit": "kWh",
    "CounterReadingT1": 1545.62,
    "CounterReadingImport": 1545.62,
    "ValueDate": "2021-11-22T15:51:33.8083885Z"
  },
  {
    "Id": "a9260872-64d1-22af-2a78-9451021ca4dc",
    "Name": "Jane Doe 123",
    "Serial": 22610343,
    "DeviceEnergyType": 1,
    "MeterSubType": 3,
    "FamilyType": 1001,
    "ActivePower": 0.0,
    "ActivePowerUnit": "kW",
    "CounterReading": 88.905,
    "CounterReadingUnit": "kWh",
    "CounterReadingT1": 88.905,
    "CounterReadingImport": 88.905,
    "ValueDate": "2021-11-22T15:51:29.9646345Z"
  },
  {
    "Id": "9dfa944e-adbd-0ad5-b391-11ffb4eb80e8",
    "Name": "Karl User 123",
    "Serial": 21783681,
    "DeviceEnergyType": 1,
    "MeterSubType": 3,
    "FamilyType": 1001,
    "ActivePower": 0.0,
    "ActivePowerUnit": "kW",
    "CounterReading": 0.0,
    "CounterReadingUnit": "kWh",
    "CounterReadingT1": 0.0,
    "CounterReadingImport": 0.0,
    "ValueDate": "2021-11-22T15:51:30.5740389Z"
  }
]

Actual Behavior
the preview shows []instead
Reproduction Steps

Access api that returns unnamed array of objects
try to filter the results with match

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2021.6.0
What operating system are you using?
Windows
Operating System Version
21H1 (Build 19043.1288)
Installation method
chocolatey
Last Known Working Insomnia version
about 2 month ago

I also have this bug.
JsonPath does not return expected results when a filter of this type is set<EMAIL_ADDRESS>Other types of jsonpath queries seems to work.
Version: Insomnia 2021.7.2
Release date: 12/9/2021
OS: Darwin x64 19.6.0
Electron: 11.2.3
Node: 12.18.3
V8: <IP_ADDRESS>-electron.0
Architecture: x64
node-libcurl: libcurl/7.73.0 OpenSSL/1.1.1k zlib/1.2.11 brotli/1.0.9 zstd/1.4.9 libidn2/2.1.1 libssh2/1.9.0 nghttp2/1.42.0

sorry about this!  I wish there was more we can do, but we're sorta beholden to the state of JSONPath, I'm sad to say.
please see https://github.com/Kong/insomnia/issues/4213#issuecomment-967353033
TLDR, use ` to escape the @

@dimitropoulos might be switching to jsmespath be a solution


Feat. req. Use CUDA / OpenCL / Vulkan

Is your feature request related to a problem? Please describe.
Nope
Describe the solution you'd like
Use CUDA / OPENCL or Vulkan to accelerate encoding/decoding

would accelerating encoding/decoding be of any real benefit?
as I can say for non-handcrafted decodes, it's not worth the added effort and may likely end up slowing the process down.

would accelerating encoding/decoding be of any real benefit?

Intel HW jpeg decoder is used in some application and is to be supported in Chrome. Same from NVjpeg library.

would accelerating encoding/decoding be of any real benefit?

Intel HW jpeg decoder is used in some applications and is to be supported in Chrome. Same from nvJPEG library.

The Intel HW* Jpeg decoder is an example decoder utilizing OpenCL for Huffman and inverse DCT decoding.
The Question is, is it worth the effort to implement support within the reference library. For hardware accelerated applications this 100% would be faster as you can implement it as a one short texture shader. As the issue with the Intel HW* Jpeg decoder is the PCIe bus speed. If we implement this into the reference library. we would 1 need to implement a GPU device and OpenCL Instance manager, 2 transfer the entire image data to and from the GPU for decoding, and 3 heavily modify the library to support OpenCL.
which simply said the transferring of the image to and from GPU memory. could cause slowdowns due to bus saturation, especially if that image is just sent straight back to the GPU to be rendered via HW accelerated renderer.
in short, its not a bad idea, its just not exactly something to implement in reference software. let people who create their own implementation take advantage of their environments.

BTW, I totally missed this! Wow. https://docs.nvidia.com/cuda/nvjpeg2000/userguide.html

BTW,

The A100 includes a 5-core hardware JPEG decode engine. nvJPEG takes advantage of the hardware backend for batched processing of JPEG images.

So apparently A100 for servers can even decode jpeg literally with HW accel.

According to https://github.com/elFarto/nvidia-vaapi-driver NVDEC expose the JPEG ASIC

I just checked ffmpeg.exe -i 02-14817ce690189825ddc36fe847d7f436.png -distance 0 -effort 9 jxq2.jxl
And it is so slow (the default is -effort 7)!! CUDA is really needed.
Interestingly enough cjxl.exe 02-14817ce690189825ddc36fe847d7f436.png image.jxl --modular --quality 100 --speed 9 -E 3 is even slower but compresses slightly better.

would accelerating encoding/decoding be of any real benefit?
as I can say for non-handcrafted decodes, it's not worth the added effort and may likely end up slowing the process down.

Two years have passed and it seems that this functionality has not been considered interesting, when in fact it is very interesting. JXL requires a lot of computation in compression and when it is done massively, the CPU practically "burns". Does nobody really think that it is important to be able to use the GPU to encode in JXL?


⚠ PrivateHD has degraded performance

In 1dc76a1, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1853 ms

Resolved: PrivateHD performance has improved in 1188c57.


[EPIC] Event store so that we can measure statistical quality of factory and frontend

As an admin, I want to record most activities (sign up, sign in, push etc) so that I'm know how frequently users are using platform and how often they achieve expected results
As a Publisher, I want to see what has happened recently with my data (and in future what my team has done) so that I know things are working, what's changed etc
As a Viewer I want to see what a Publisher has been up to so that I know whether they are active, what's new and what's hot (so that ...)
Acceptance Criteria

[ ] We have easily measurable statistics

Tasks

[ ] Report events to ES: WHO did WHAT WHEN #52

[ ] do analysis how to gather needed info
[ ] do analysis how to put this on elastic search

[ ] Record push completion (always successful)
[ ] Display push completion in dashboard (?)
[ ] Record push fail
[ ] Record push start
[ ] Record ...
[ ] Write blog post about new dashboard

Analysis
In order to do statistical analyses we need to record most possible activities on the platform. Good place for storing info would be the elastic search
What are the events?

[Flow] Push completed [assembler]

Success / Failure

Dataset created (special case of push - where push is a new dataset) [spec store?]
[Flow] Push started (time elapsed to success can be ca) [assembler]
Download (clicks, GET on data file) [Google Analytics]
User sign up (happens through out system)
(?) User sign in to frontend (?) (happens through our system)
(Page view) [Google Analytics]

Search of term [Google Analytics]

Design of the ES index
{
  "timestamp": ...
  // type of event
  "event_entity": "flow|account|..."
  "event_action": "create/finished/deleted/...
  // event filters
  "owner": ...
  "dataset": ... // may not always be there ...
  // results
  "outcome": "ok/error", or "status": "good/bad/ugly",
  "messsage": ... // mostly blank
  "findability": ...
  "payload": {
    // anything you want add - customizing the event info per object
    flow-id (aka pipeline-id)
  }
}

{
  "timestamp":
  "owner":
  "payload": ...
}

Desired Queries
For user dashboard:
SELECT * FROM events where event.owner = userid SORT DESC timestamp;
SELECT * FROM events where ... AND
For user public profile
SELECT * FROM events where event.owner = profilid AND findability == 'public' AND type != 'login'
Internally
SELECT count(*) FROM events where type = "dataset/push" AND outcome == "failed"
Collection

should a collection of data happen daily, or on every event?

Depends on where we are getting data from but preferably on every event (push style). For some external data e.g. downloads we may need to pull on a regular basis.

How to automate this?
how to gather as they are split across different services/API?

Issue moved to datahq/pm #43 via ZenHub


pkg/k8s: set the ciliumNodeStore as nil when connecting to KVStore

When connecting to the KVStore the CiliumNodeStore will stop receiving any updates from kube-apiserver since we stop the kubernetes informer.
This will cause any "Gets" done into this store cache to potentially return CiliumNodes out-of-sync. Thus we will be setting the CiliumNodeStore as nil to avoid reading from it and fetch the state directly from kube-apiserver.
Fixes: 76cc93eb7975 ("k8s: optimize API calls made to kube-apiserver")
Avoid k8s CiliumNode initialization problems when Cilium connects to the KVStore

/test

/test-runtime

There's also similar potential issues with CEP/CES stores. We should probably avoid the whole "GetWatcher(...)" pattern.


Roll Flutter Engine from af1fd1724986 to 0aabfa642430 (4 revisions)

https://github.com/flutter/engine/compare/af1fd1724986...0aabfa642430
2024-03-12<EMAIL_ADDRESS>Reland "Move emscripten out of the buildroot into the flutter repo" (flutter/engine#51353)
2024-03-12<EMAIL_ADDRESS>Roll Skia from c123cce23f20 to 6e1f7333b2c7 (1 revision) (flutter/engine#51356)
2024-03-12<EMAIL_ADDRESS>Roll Skia from 32f1cb50734a to c123cce23f20 (9 revisions) (flutter/engine#51351)
2024-03-12<EMAIL_ADDRESS>Migrate perfetto to flutter/third_party (flutter/engine#51272)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Sim2Real in Go1

I am trying to do sim2real on Go1 based on your code.
In your code, controller_dt = 0.01 # in sec, but I am curious whether controller_dt was set to 0.01 in the actual robot.

Thanks for your interests.
The controller is running at 200 Hz while the mpc solver is running at 40 Hz on real robot.

Thanks to you, I was able to successfully run sim2real on unitree go1. thank you.

Congrats! Would you like to contribute to this repo regarding the sim2real transfer?


Include @-named dependencies with --onlyDirectDependencies

Fixes https://github.com/mwittig/npm-license-crawler/issues/20
Output is now:
node ../npm-license-crawler/bin/npm-license-crawler --onlyDirectDependencies
...
├─<EMAIL_ADDRESS>│  ├─ licenses: MIT
│  ├─ repository: https://github.com/godaddy/terminus
│  ├─ licenseUrl: https://github.com/godaddy/terminus/raw/master/LICENSE
│  └─ parents: fake
└─<EMAIL_ADDRESS>   ├─ licenses: MIT
   ├─ repository: https://github.com/lodash/lodash
   ├─ licenseUrl: https://github.com/lodash/lodash/raw/master/LICENSE
   └─ parents: fake

Number of entries found: 2

for package.json:
{
  "name": "demo",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "@godaddy/terminus": "^2.2.0",
    "lodash": "^4.17.10"
  }
}

@mwittig could you PTAL ?

Thanks for your contribution and sorry for the delay. I'll review asap


Per Domain Bayes Learning

When I try and learn a mail using domain, the web panel doesn't respond.

When clicking Validate nothing happens. If I select global database, it will work.

Some further investigation finds a 500 error on the request
POST /quarantine/learning_recipient/ => generated 16141 bytes in 51 msecs (HTTP/1.1 500) 4 headers in 146 bytes (1 switches on core 0)

Can you activate the debug mode please? (set DEBUG to True inside settings.py and reload uwsgi)
You should get a complete error.
By the way, this error is not related to modoboa but to modoboa-amavis. I'll move it to the right repository.

This issue was moved to modoboa/modoboa-amavis#38


Ubuntu VM not saving files from previous session

Describe the issue
Hi Everybody. I am using UTM on my 2020 m1 Macbook air to virtualize Ubuntu Desktop (v20.04.3 amd64). Every time I start the ubuntu VM, I need to go through the entire installation process and all my files from the previous session are lost. After the boot process, I am met with the install GUI. When I leave the VM, I shutdown from Ubuntu (as recommended by pop-up when pressing the on/off button in UTM's GUI). What is the procedure to safely terminate the VM such that all files from that session are saved? Please let me know if I am unclear or if I can give any additional info. Thank you!
** VM Config **

Architecture: x86_64
Machine: Standard PC (Q35 + ICH9, 2009) (alias of pc-q35-6.2) (q35)
Memory: 4.0 GB
Size: 744 KB
CD/DVD: ubuntu-20.04.3-desktop-amd64.iso

Configuration

UTM Version: 3.0.4 (46)
OS Version: Big Sur 11.5.2
Apple Silicon

Hi @adespoton ! I believe that I am booting from the install image everytime but I am not sure (I am very new to VMs). When I start up the VM, I am greeted with this screen

Everytime I get this screen, I choose the first option.

After you've installed onto a blank disk image, you need to eject the DVD image so you don't boot into the installer anymore.  At that point, the VM will boot up from the "internal" disk instead.
Just like with a real computer; if the DVD is still in the drive, that's what you're booting from.  Its only purpose is to install the OS onto your main drive image.

Okay I think I understand.
So from UTM, I boot the VM with the ubuntu-20.04.3-desktop-amd64.iso as the DVD/CD image. Then, after ubuntu has completely installed onto the VM, I shutdown and exit the VM. Then, I will use UTM to clear the DVD/CD slot of the image and then start the VM again.
Is that correct?

Correct.

On Jan 27, 2022, at 12:54 PM, zbeucler2018 @.***> wrote:

Okay I think I understand.
So from UTM, I boot the VM with the ubuntu-20.04.3-desktop-amd64.iso as the DVD/CD image. Then, after ubuntu has completely installed onto the VM, I shutdown and exit the VM. Then, I will use UTM to clear the DVD/CD slot of the image and then start the VM again.
Is that correct?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.

I think this issue can be closed now.

HOW to eject the DVD image


[ MB-16381, MB-16029 ] Fixing bug around SIT Status

Summary

[x] Using Optional Chaining to not have the app break;
[x] Removing the required sitStatus;

This was tested locally in a Slack huddle to see the fix working. cc: @mlm55 @mmayo-truss

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-16381, MB-16029

  Generated by :no_entry_sign: dangerJS against 6cd4fd2d6d668ba17778c7481784655b9de8e971


reduce duplicative check test setup

Issues
Part 1 of the cleanup

Somehow all of this got added to main from a different way.

It appears to have come in through the https://github.com/OpsLevel/opslevel-go/pull/442 as a dependent commit


Sit cannot pull site_vars from init(self)

https://github.com/kjaymiller/render_engine/blob/520f0becaa51a79d384c92fc8b18185712a1f3b1/tests/test_site.py#L18-L37
Variables that are expected for proper function should be passable in the class or through __init__.

after some discussion this isn't going to be something that we recommend doing.


SelfHosted: increase timeout to wait pod running

lgtm

Awesome, thanks!

@etcd-bot retest this please
Tracking issue separately.


For-loop in ConfigManager.build_config_calculated_fields() does nothing

Not really a bug, but I noticed that the following function has a for-loop which, aside from printing out some logs, does nothing that has any effect. Would be curious if this is just something that was accidentally left this way, or if there is some explanation for it.
https://github.com/GoogleCloudPlatform/professional-services-data-validator/blob/54403e3014ff5dbcc50ca540eb786dfc33759f0b/data_validation/config_manager.py#L703-L748

Thanks for pointing this out - this is intended for validations over all columns i.e -sum '*' so we can log when columns are skipped (for non-numeric types) or missing from the target table. It's useful to log so the user can know which columns aren't included in the validation.


The minified code is not working for me...

...but the unminified works fine. If I minify the code with CodeKit it also doesn't work.
And: "JSHint found 125 issues in the file", I have no idea if it has any significance to this issue.
Lovely highlighter though. Thanks!

Hey @hilja, I know it's been awhile, but do you have an example of this?
The minified code works just fine for me, so maybe this got fixed?

@Golmote I think this one can be closed

Unable to reproduce the bug. I close the issue.


Update kubebuilder shasums due to upstream update

The kubebuilder assets location has been updated upstream: https://github.com/kubernetes/k8s.io/issues/2647

/approve


edit profile not loading

Hi @MuhammadNaeem42 ; Could you pull the latest version and check again please? This should no longer be an issue. Thanks

Closing as I was unable to reproduce it. Thanks


Draggable items break if canDrop() causes them to shift

Describe the bug
When dragging an item, I want my drop zones to enlarge. The item that I'm trying to drag can be pushed out from under the cursor because a valid drop zone above it enlarges. If this would happen, the draggable item just fails to be dragged.
In the debugger, I can see that there are two calls to my collect function. During the second call, monitor.getItem() returns null.
Reproduction
Live Reproduction
Steps to reproduce the behavior:

Drag the first "Drag Me" item. See the drop zone enlarge.
Attempt to drag the second "Drag Me" item.

Expected behavior
The drop zone should enlarge when the second item is dragged.
Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome (this does not repro in Edge or Firefox!)
Version: 77.0.3865.120 (Official Build) (64-bit)

@cynicaloptimist What is the work around?

Not really a workaround; the workaround is to not change the size of drop zones when dragging.

Same issue as #1476
The workaround is to delay the size modification from ~50-100ms. It can be done with CSS animation or setTimeout in js/ts

It doesn't seem limited to just size. I changed my app to just change color rather than size and I still have the same issue.


Is there an easy way to process multiple files?

I am trying to rewrite the makefile of the old project to cargo-make，Is there an easy way to something like that?
APPS := $(wildcard $(APP_DIR)/*.rs)
ELFS := $(patsubst $(APP_DIR)/%.rs, $(TARGET_DIR)/%, $(APPS))
BINS := $(patsubst $(APP_DIR)/%.rs, $(TARGET_DIR)/%.bin, $(APPS))

binary: elf
	$(foreach elf, $(ELFS), $(OBJCOPY) $(elf) --strip-all -O binary $(patsubst $(TARGET_DIR)/%, $(TARGET_DIR)/%.bin, $(elf));)

I only thought of using duckscipt, but I don’t know how to get the multiple files

This is my attempt:
[tasks.binary]
command = "rust-objcopy"
args = ["--binary-architecture=riscv64",
    "--strip-all", "-O", "binary", "${FILE_ELF}", "${FILE_BIN}"]
dependencies = ["elf"]

[tasks.build]
script_runner = "@duckscript"
script = '''
handle = glob_array ${APP_DIR}/*.rs
target_dir = get_env TARGET_DIR
for path in ${handle}
    app_name = basename ${path}
    app_name = substring ${app_name} -3
    file_elf = concat ${target_dir} ${app_name}
    set_env FILE_ELF ${file_elf}
    file_bin = concat ${file_elf} .bin
    set_env FILE_BIN ${file_bin}
    cm_run_task binary
end
'''

that actually looks spot on

but if I use env value like concat ${TARGET_DIR} ${app_name} will get an error：
 Source: Unknown Line: 8 - Found an end for/in command but not currently running part of a for/in invocation flow.

not sure i can help without looking at the make file.
i wrote my own based on yours:
[env]
APP_DIR = "./dir1"

[config]
skip_core_tasks = true

[tasks.binary]
command = "echo"
args = ["--binary-architecture=riscv64",
    "--strip-all", "-O", "binary", "${FILE_ELF}", "${FILE_BIN}"]

[tasks.build]
script_runner = "@duckscript"
script = '''
handle = glob_array ${APP_DIR}/*.rs
target_dir = get_env TARGET_DIR
for path in ${handle}
    app_name = basename ${path}
    app_name = substring ${app_name} -3
    file_elf = concat ${target_dir} ${app_name}
    set_env FILE_ELF ${file_elf}
    file_bin = concat ${file_elf} .bin
    set_env FILE_BIN ${file_bin}
    cm_run_task binary
end
'''

and ran it via cargo make build and it works well and prints out the rs files i have at that dir (i changed the rust objcopy to echo)

Yes, it is correct. But if I remove target_dir = get_env TARGET_DIR and call the concat method like    file_elf = concat ${TARGET_DIR} ${app_name}, an error will occur. TARGET_DIR is set in [env],

@umaYnit is this still an issue? trying to figure out if we have something actionable here to do

I got an error from this file:
[config]
skip_core_tasks = true

[env]
PREFIX = "hello"

[tasks.build]
script_runner = "@duckscript"
script = '''
values = range 1 10
# get_env PREFIX
for path in ${values}
    file_elf = concat ${PREFIX} ${path}
    echo ${file_elf}
end
'''

If delete the commented line # get_env PREFIX, an error will occured.
 Error while running duckscript: Source: Unknown Line: 6 - Found an end for/in command but not currently running part of a for/in invocation flow.

i'll check it thanks

@umaYnit ok you seem to have found a big bug in the duckscript flow control commands.
its similar to sagiegurari/duckscript#110 but the fix i did there was for functions and only partial for flow control and seems partial wasn't good enough. so its amazing you found it.
you can now reinstall cargo make (doesn't matter what version) and you should have the fix.

@umaYnit closing this one as a fix is already available and released. if you see more issues or if its not fixed just open/reopen. thanks


W tensorflow/stream_executor/cuda/cuda_dnn.cc:2223]

The source code I try to compile has been changed, and now it requires tf-nightly build(1.4 version). For that reason, I created new conda environment which includes 1.4 version of tensorflow.
However, I still use the same set-up except the version of tensorflow. I've read they anticipate releasing 1.4 version of tensorflow wih cuDNN 7 instead of cuDNN 6. Is this the reason that I get these warnings? Should I upgrade the cuDNN version from 6 to 7.
I'm also getting "RuntimeWarning: compiletime version 3.5 of module 'tensorflow.python.framework.fast_tensor_util' does not match runtime version 3.6" although I loaded the 3.6 version of tf-nightly build.
This is what I get:
2017-10-11 15:42:05.897046: W tensorflow/stream_executor/cuda/cuda_dnn.cc:2223]
train 0 4000 0.764138 1.0965 0.604099
2017-10-11 15:42:40.117190: W tensorflow/stream_executor/cuda/cuda_dnn.cc:2223]
2017-10-11 15:42:40.118152: W tensorflow/stream_executor/cuda/cuda_dnn.cc:2223]
2017-10-11 15:42:40.118270: W tensorflow/stream_executor/cuda/cuda_dnn.cc:2223]
train 0 4100 0.778095 1.09562 0.603462
2017-10-11 15:43:14.819378: W tensorflow/stream_executor/cuda/cuda_dnn.cc:2223]
2017-10-11 15:43:14.820271: W tensorflow/stream_executor/cuda/cuda_dnn.cc:2223]
2017-10-11 15:43:14.820375: W tensorflow/stream_executor/cuda/cuda_dnn.cc:2223]
train 0 4200 1.55563 1.09606 0.603783

System information

OS Platform and Distribution : Linux Ubuntu 16.04 LTS
TensorFlow installed from (source or binary): tf_nightly_gpu-1.4.0.dev20171010-cp36-cp36m-manylinux1_x86_64.whl
TensorFlow version : 1.4.0-dev20171010
Python version: 3.6.1 Anaconda 64bit
CUDA/cuDNN version:  cuda-ga2_8.0_amd64 / cudnn-8.0-linux-x64-v6.0
GPU model and memory: Nvidia GeForce GT 710  1GB memory
Exact command to reproduce: python train_xxx --train_batch_size 16 --val_batch_size 16

The fast_tensor_util warning is because we only build for 3.5, then copy the pip package to a 3.6 name assuming they will be compatible (I think). This was true, but we're using a bit of Cython-compiled code now. Probably Cython is a bit overcautious with the version checking here, but we should get rid of the warning anyway.
@av8ramit would it be difficult to start building nightlies (and releases?) separately for Python 3.5 and 3.6? I could look into ways to suppress the warning, but it's probably there for a reason.

Yes, we unfortunately build for Python 3.5 and then copy to 3.6 for Linux. I'll look into separate builds.

And to be clear I have no idea what's up with the CUDA warnings.

@yzhwang It looks like you added these warnings, with the code looking something like this:
      auto allocated = scratch_allocator->AllocateBytes(stream, size_in_bytes);
      if (allocated.ok()) {
        scratch = allocated.ValueOrDie();
      } else {
        LOG(WARNING) << allocated.status().error_message();
      }

The code looks reasonable.  Do you have any idea why we're getting these nonsense status messages?

This means that the previous cudnn function call: https://github.com/tensorflow/tensorflow/blob/r1.4/tensorflow/stream_executor/cuda/cuda_dnn.cc#L2195 returns a size_in_bytes that caused AllocateBytes() to fail. It's interesting that the error_message() is empty though.
The solution for this is: since we are not officially supporting cudnn7 (tests and integration in progress), I recommend you to switch back to cudnn6 for now.

The problem has been solved when I switched back to cudnn6. It was mainly about version issue as @yzhwang stated. I really appreciated it.
Thanks for dropping a line.

@Narthyard Thanks for the update. Note that TensorFlow now officially support cudnn7+CUDA9. Let me know if you think this bug still exists when switching to cudnn 7.

@yzhwang The bug is still there. I am using Tensorflow 1.5 with CUDA 9.0 with cudnn 7

Still there with 1.10.1 with CUDA 9.2 and cudnn 7.2

Still there with 1.14 with XLA on.


quick fix

Codecov Report

Merging #602 into master will decrease coverage by 0.82%.
The diff coverage is 69.86%.

@@            Coverage Diff             @@
##           master     #602      +/-   ##
==========================================
- Coverage   78.04%   77.22%   -0.83%
==========================================
  Files         120      122       +2
  Lines        3913     3986      +73
  Branches       90       97       +7
==========================================
+ Hits         3054     3078      +24
- Misses        859      908      +49

Impacted Files
Coverage Δ

...com/ing/baker/baas/interaction/HealthService.scala
0.00% <0.00%> (ø)

...in/scala/com/ing/baker/baas/interaction/Main.scala
0.00% <0.00%> (ø)

...ker/baas/interaction/RemoteInteractionLoader.scala
0.00% <0.00%> (ø)

...cala/com/ing/baker/baas/state/KafkaEventSink.scala
18.18% <ø> (ø)

...src/main/scala/com/ing/baker/baas/state/Main.scala
0.00% <0.00%> (ø)

...ry/clustercontroller/BakeryControllerService.scala
0.00% <0.00%> (ø)

.../scala/com/ing/bakery/clustercontroller/Main.scala
0.00% <0.00%> (ø)

...y/clustercontroller/MutualAuthKeystoreConfig.scala
0.00% <0.00%> (ø)

...lustercontroller/controllers/BakerController.scala
86.32% <71.42%> (-2.36%)
:arrow_down:

...ing/baker/runtime/serialization/JsonDecoders.scala
88.46% <77.77%> (-7.00%)
:arrow_down:

... and 18 more

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 94ff1cd...3341b27. Read the comment docs.


deleted a 404 url

I see it was made by https://maemoon.me but I don't see where it was moved to if it lives somewhere else now

good shout
i did find https://maemoon.me/experiments/logstats which is pretty cool


Plugin [id: 'org.sonarqube', version: '2.7'] was not found

ERROR: Plugin [id: 'org.sonarqube', version: '2.7'] was not found in any of the following sources:

- Gradle Core Plugins (plugin is not in 'org.gradle' namespace)
- Plugin Repositories (could not resolve plugin artifact 'org.sonarqube:org.sonarqube.gradle.plugin:2.7')
  Searched in the following repositories:
    Gradle Central Plugin Repository

@teolemon news?

let's close


Counting number of scene.play and wait calls

I'm refactoring the progressbar and t-values generation. See #183
I'm facing an issue ; I would like to be able to count the number of calls of scene.play or scene.wait in construct() defintion, because I need to know when is the last animation.
As far as I know there is no way to do it in manim, as right now manim just takes scene.play and scene.wait one after the other (so it does not need to get the total number of scene.play and scene.wait calls.
So, to do that, I think the only solution is to disassemble construct function to get the number CALL_FUNCTION instances with scene.play and scene.wait. Something like this https://stackoverflow.com/a/16042229/12009882
But my question is, is it good? Like, is it ok to have something like this? Because this method is slightly too much hacky in my mind (although it would work and is not very complicated).
Maybe this is totally fine and I'm just nitpicking, please tell me :D
EDIT : Looking for CALL_FUNCTION in function's instructions (disassembled) won't work as the name of the function does not appear. Instead, we can look for LOAD_METHOD with argval 'wait' or 'play'.
Example :
Instruction(opname='LOAD_METHOD', opcode=160, arg=3, argval='wait', argrepr='wait', offset=22, starts_line=None, is_jump_target=False)

Instead of disassembling a function, just keep an instance member that counts the number of times these have been called.
class Scene:
    def play(self, ...):
        ...
        self._play_calls += 1

I know, but I need to get the number of play and wait calls before they are called (i.e before scene.construct() is called.
(Btw the variable you proposed already exists and is scene.num_plays).
What I need to do is to get if the play/wait call is last one of the scene. I thought to do something like (scene.num_plays == scene.total_num_plays= to check if this is the last call. Then I would use disassembling to get scene.total_num_plays.
On second thought, I don't think that using disassembling is not orthodox. I think I will use that, if no one objects :D.

I think it's fair game, though the inspect module in general tends to be a bit slow. Would be interesting to see if there are any speed changes after your PR.

That's a problem.
But there is another that I didn't think about: when play and wait are called from outside the construct() (e.g through a nonlocal function), just disassembling construct won't work as it is not recursive by default.
I remember facing this issue in #166 when I had to implement serialization of functions, and I had to wrote something recursive which was very slow and not viable.
So I don't think its possible to count number of play and wait. I will find a workaround.
I will close this in a few days in no one has an idea.

One workaround (that would require a major rewrite of some parts of manim) is to implement scene rendering in a lazy fashion allowing for dry runs. For example, you can run all of deconstruct without actually rendering anything, just changing the coordinates/colors/etc of your abstract mobjects. After that dry run, you can easily get the count of play/wait, as well as many other Things About the Scene that may be interesting on their own accord. After that you can actually do the rendering. BTW, such a system might also help the scene caching stuff.

This is a brilliant idea, but IMO this is too much work for what we need. Unless there are other projects of features that would need   this kind of thing, I think this is worthless.

BTW, such a system might also help the scene caching stuff.

And could you clarify this point ? I don't see why this could improve caching.

If we had such an abstract system detached from the rendering system, we could run the old Scene and the new Scene without rendering them to determine whether they are the same. If they are, then don't render the new one.

But I agree that would require a major rewrite of basically everything..

If we had such an abstract system detached from the rendering system, we could run the old Scene and the new Scene without rendering them to determine whether they are the same. If they are, then don't render the new one.

I see. This is pretty much what it does in scene caching.

But I agree that would require a major rewrite of basically everything..

In fact this could be easily done with a decorator placed for wait and play, that would control the handling of the play or wait call, so we could call two times construct(), first time while doing a dry run so the decorator just counts, and second time it would go normally.
But this would cause trouble when there are heavy functions used in construct().
Or we could do like save the arguments that were passed to play and wait , to execute play and wait later with these arguments. We then won't need to do construct() twice. The question is, how to do it? How to save arguments? Maybe with pickle, something like that.

Ugh let's not pickle things unless absolutely necessary :)

Ugh let's not pickle things unless absolutely necessary :)

Then I don't see how we could do without calling scene.construct() (one as a dry_run). IMO this is a bad idea when it needs to call other heavy functions.

What's the problem with using inspect.getsource(SceneClass) and counting the occurrence of self.play and self.wait?

@kilacoda I first tought to do this only for construct(),  but as obvious as it may seem I didn't think to do it with the whole SceneClass lol, I'm stupid. So thank you very much for bringing up this idea.
The issue was/is that if these calls are outside the construct, then we won't be able to detect them (except by doing a weird recursive thing that would take long). We should not have this issue if we take the source (or the disassembly) of the whole class as all the wait/play calls are supposedly within the latter.

@kilacoda On second thought it won't work when there is a loop. Like, manim will be trick into finding only one self.play when there is multiple invocation because of the loop.

So I think this is impossible (without rewriting the whole codebase)
Closing this.


Add "ignore_session" option to configuration

ignore_session option allows to authenticate a user with strategies even if there is a user in the session:
warden.authenticate(scope: :user, ignore_session: true)

I have a controller where I want to authenticate a user only by one specific strategy.
So I write something like this:
warden.authenticate(:my_strategy, scope: :user, store: false)

It works fine until a user with a cookie sends a request to this controller. In this case, my custom strategy will be ignored and the user will be found in cookies.
I know I can achieve what I want by changing the scope. But it doesn't give a 100% guarantee. If somehow the cookie will be set for user scope - the problem will repeat.
ignore_session option gives a 100% guarantee that my strategy will be used.
It can be helpful for other people I believe. Like it can be used for API controllers where you need to authenticate a user only for some kind of token.
Currently, a user with a cookie can send requests to such controller with different tokens and these tokens will be ignored.
What do you think?

Hey @jsmestad, any thoughts on this?

@shatalov-boris I like the idea. I need to read through it more before we push it into master, but I think it looks good at first glance

Just wanted to second the value here. We're working to separate some overlapping auth mechanisms (both sessions and tokens being used by different legacy apps on the same domain, unfortunately), and having more control over how sessions are handled (without introducing scopes) would be a big help. Thanks!


bug(MatTabGroup): Pagination for tabs isn't properly re-initialized when underlying tab contents gets changed

Reproduction
Use StackBlitz to reproduce your issue:

Components: https://stackblitz.com/edit/angular-xucgvn

Steps to reproduce:

Navigate to https://stackblitz.com/edit/angular-xucgvn
Select the "Sixth" or "Seventh" tab
Put the tick next to "Switch between pages" which causes the contents within the MatTabGroup component to change.

Expected Behavior
What behavior were you expecting to see?
I do expect that pagination-related chevrons won't appear in case if tab contents dynamically get reduced from greater to lesser like in the provided case.
Actual Behavior
What behavior did you actually see?

I see that the tab pagination block is rendered with pagination controls
Environment

Angular: "^11.0.0"
CDK/Material: "^11.0.0"
Browser(s): Google Chrome Version 90.0.4417.0 (Official Build) canary (x86_64), Firefox 86.0b9
Operating System (e.g. Windows, macOS, Ubuntu): OS X 11.2

Duplicate of #13539
(I believe it's the same underlying issue)


Websocket implementation

following the esp_websocket_client found here^1, this commit
wraps the mentioned C library in Rust to expose a safe WebSocket
interface.
The exposed API copies the previously implemented MQTT API. On creating
a EspWebSocketClientConfig and passing it to a EspWebSocketClient,
the user can call send to send data. Manual sending of fragmented data,
Ping/Pong, or Connect/Disconnect are not supported due to the underlying
C library not supporting this, either.
Receiving is done in a new thread with the returned
EspWebSocketConnection and its blocking next function call, which
wakes on receiving one of the events defined in WebSocketEventType.
The EspWebSocketClient takes a time::Duration timeout since the
underlying C lib requires this but the defined interface in the
embedded_svc create does not provide it as a parameter.
On dropping the EspWebSocketClient the C functions
esp_websocket_client_close and esp_websocket_client_destroy are
being called, cleanly closing the connection to the WebSocket server.
This requires esp-rs/esp-idf-sys/pull/88.

Can you look at the build output. It fails not just with clippy, but also with real compilation errors?

Can you look at the build output. It fails not just with clippy, but also with real compilation errors?

Sorry, my mistake. I need to first merge the changes to esp-idf-sys.

@ivmarkov build failed due to ws being specified as an experimental feature in embedded-svc.
More errors follow after fixing this, which don't happen on the fork I develop on locally, but which can be reproduced when cloning cleanly. I'll investigate.

Ease change the PR so that the experimental feature is temporarily enabled.

You need to rebase your fork on top of latest esp-idf-svc master to resolve the conflicts.

@ivmarkov There are some warnings left in src/eth.rs which I suspect will prevent the Build | Clippy step in the CI from passing due to warnings being treated as errors.
Should I fix these, too?

No it's OK like that. The eth related warnings seem to be for other architecture (xtens). Please keep in mind that my review will be delayed as I would be traveling tmr and over the weekend


jaxb-api 2.3.0 declares java.activation transitive dependency in module-info, but not in pom.xml

The jaxb-api 2.3.0 library has a dependency on the java.activation module, which has been removed from JDK 11. The dependency is declared in module-info.java but not in pom.xml:
http://search.maven.org/#artifactdetails|javax.xml.bind|jaxb-api|2.3.0|jar
It should be declared there as well, so clients of the jaxb-api don't have to discover this dependency at runtime due to NoClassDefFoundError

Fixed in 2.3.1


Install librdkafka

What does this PR do?

Install librdkafka
Motivation

Relates to this PR
Additional Notes

Related PRs:

https://github.com/DataDog/datadog-agent/pull/15470
https://github.com/DataDog/datadog-agent-buildimages/pull/359
https://github.com/DataDog/homebrew-datadog-agent-macos-build/pull/2

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

Closed as this will be needed at runtime


spotbugs: variable/return value check and modify

spotbugs: variable/return value check and modify

close this and submit to branch 'dev'


Note can't be imported

Describe the bug
When starting a bot with the framework it just thrwos the error
Traceback (most recent call last):
  File "/home/ente/roboduck/rdbot.py", line 10, in <module>
    from mi import Note
ImportError: cannot import name 'Note' from 'mi' (/home/ente/.local/lib/python3.9/site-packages/mi/__init__.py)

To Reproduce
Steps to reproduce the behavior:

Update to Mi.py 3.9.9
Start a bot that worked on Mi.py 3.2.1
See error

Expected behavior
Bot starts
Behaviour
Error right at the start up...

As I already answered in the chat, the import has been changed. So, from mi.framework import Note should solve the problem.

If that doesn't work, try from mi.framework.models import Note. If that doesn't work either, please report again.

It's currently a problem that can be solved with from mi.framework.models import Note, but I'm considering going back to from mi import Note. The reason for this is that import becomes longer.

To be honest, I'm not sure if I should change this back or keep it the way it is. So, if you have any opinions, please open a discussion.

Looks good, so I'll close it.


Plugin hangs

Obsidian version: 1.7.4-1.7.7
I'm getting considerable hang time ~20seconds with this plugin enabled. It doesn't occur in a sandbox vault, maybe it has to do with large vaults? Have you checked out https://docs.obsidian.md/Plugins/Guides/Optimizing+plugin+load+time?

I've also noticed the loading time taking too long in large vaults. I'll try to optimize it in the next version along with some new features. It can take one week or two, though.

Hi! I've fixed the startup performance issue in 0.10.1. Please try out! But if you open Manage Page with hundreds of tasks listed, it may still slow down Obsidian (although not significantly). I'm working on it.

Yes! It's much faster than before. Still noticeable, though.
I wonder, would you consider also adding an option to limit the directories it parses? That would surely improve performance. I have 10+ top level directories and only 2 of the smaller ones really need to be parsed

Hi! I've fixed the startup performance issue in 0.10.1. Please try out! But if you open Manage Page with hundreds of tasks listed, it may still slow down Obsidian (although not significantly). I'm working on it.

Yes! It's much faster than before. Still noticeable, though.
I wonder, would you consider also adding an option to limit the directories it parses? That would surely improve performance. I have 10+ top level directories and only 2 of the smaller ones really need to be parsed

It's absolute possible. I'll see if I have some spare time to do it!


Tutorials need to mention prerequisites

For instance, PIL requires additional development packages (sudo aptitude build-dep python-imaging) to be present at build time to be useful.

Actually, the command I mentioned isn't enough.

you need python-dev too. We have a list here from getting Sergio set up. Someone on will have to figure out the same on a mac from a fresh install.

Ubuntu 11.10's multiarch support makes PIL not find the libraries it needs. This should fix it on 64-bit machines:
sudo aptitude build-dep python-imaging
sudo ln -s /usr/lib/x86_64-linux-gnu/libz.so /usr/lib
sudo ln -s /usr/lib/x86_64-linux-gnu/libjpeg.so /usr/lib
sudo ln -s /usr/lib/x86_64-linux-gnu/libfreetype.so /usr/lib
pip install -U PIL

For 32-bit installations, replace x86_64 with i386.


Add Licensing guide for Open Source

As per Alex.

Once accepted, I will update all our Open Source repositories to match.


Game Doesn't Detect Multiple Map Packs

Game doesn't look for or display multiple map packs.

The framework is mostly in place, however I need to add the code to read the lines of all of the map pack files for the titles, display that, page it if necessary, etc.

Game now detects multiple map packs, now it just has to choose the one that loads.

Added in its entirety.


WIP: Enable configurations

Fix #19, #23
This is working, but still open for options/QP feedback.

@stefanpenner do you have a moment to review? This would really help profiling our app with https://github.com/ember-best-practices/ember-perf-timeline/issues/23

released as v2.1.0 🎉


Orchestrator Recognizer: Rename misleading parameter names

Note: This checkin associated with :

C# Orchestrator recognizer update (dotnet pr)
A samples update that uses Orchestrator recognizer (TODO: add PR#)
Composer updates (TODO: add PR#)

Description
From the first preview, we've received feedback that our parameter names are confusing.
Specific Changes
This performs the following parameter renames:
modelPath => modelFolder
snapshotPath => snapshotFile
Testing
Tests have been updated to reflect new parameter names.

Pull Request Test Coverage Report for Build 529180206

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.003%) to 84.678%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build 529161273:
-0.003%

Covered Lines:
18198

Relevant Lines:
20503

💛  - Coveralls


Android: Banner ads cutout in phone safe area

Run example on android phone, show banner. if banner position in phone safe area, then part of banner cut out

@kornyushin I can't repo this bug on my phones. But I tried a fix, could you pls check if this branch https://github.com/AGulev/DefVideoAds/archive/refs/heads/issue-24.zip has the same issue (or at least different in a some way)

Now I can reproduce it on Huawei P20 Pro, Android 10.
Oki, let me try a couple of tricks to fix that

https://github.com/googleads/googleads-mobile-unity/issues/902


Cannot support Chinese in the Interactive Window?

I use R3.6.1
When I excute R code in VScode by using R extension, the Chinese characters are just ignored and they disappear.

I tried sending code with Chinese characters to both radian and R under Ubuntu, everything looks normal here.

I guess it is a Windows-specific issue?

May be it is an issuse for Windows, I use Windows 10.
Are there any suggestions for that problem ?
I have tried to change my file encoding into utf-8 and gbk, but neither works

@Thinking2018 I’ve duplicated this on Windows 10. I’ve tried pasting Chinese characters in Rterm started outside of VSCode and that doesn’t work either.
As a workaround, I suggest radian. I was able to send Chinese characters to that on Windows 10 with no problems. (It’s also much nicer to use than the normal R console!)

RTerm.exe doesn't support unicode chars at all, there is no way to type and display non-ascii chars. On the other hand, radian supports unicode chars (with a catch): it only works perfectly if your computer locale is the same as that of the text.
If your computer doesn't have a matching locale (for example, your computer locale is English but the text is in Chinese), then radian might fail to display the text. You are still able to paste the text, just not able to display it probably.

(PS: if you are using a relatively old version of radian, the text may be displayed correctly, but it was based on a hack that breaks a lot of other things so I have reverted it in recent versions.)
Ultimately, the issue is caused by upstream R because R doesn't print unicode chars directly unless RGui is used. I have filed an issue https://bugs.r-project.org/bugzilla/show_bug.cgi?id=17677
and radian will be updated if we hear anything from R maintainers.

Forgot the mention that the above only applies to Windows. radian fully supports unicode chars in Linux and macOS.

@randy3k Thank you for that info! Hopefully Rterm will be updated to support Unicode at some point.
I’m going to close this as it’s an R problem rather than a vscode-R problem, and there’s a good workaround (radian).

你好，同样遇到vscode里输入中文的问题，有没有好的解决方案呢？谢谢

您好，
谜之bug，目前无解。可用radian替代，但它不是R。
谢谢
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2019年12月29日(星期天) 晚上6:15
收件人<EMAIL_ADDRESS>抄送: "彩虹后的阳光"<761134060<EMAIL_ADDRESS>主题: Re: [Ikuyadeu/vscode-R] Cannot support Chinese in the Interactive Window? (#152)
你好，同样遇到vscode里输入中文的问题，有没有好的解决方案呢？谢谢
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

radian不能成功部署。

vscode 能不能直接使用Rgui呢