Dependency management using Python Poetry for fabric cli

What this PR does

dependencies for the fabric cli are now managed via poetry.
fabric can be called directly as a CLI from the activated python virtualenv without the need to call it as python fabric.py.
updates Fabric CLI install instructions.
partially solves #27.

Why so big?

Poetry.lock maintains the hashes of all actually resolved dependencies. It's generated when one installs dependencies with poetry add foo automatically.

What's in it for this project

Deterministic dependencies pull and reproducibility: Since dependencies are resolved, locked and committed to source control, collaborators of this project can work on the same exact Python environment regardless of which OS they use and when they pulled dependencies.
Isolated packages: Poetry helps avoid version conflicts in Python files and projects by isolating packages from the OS.
Single configuration file: Poetry uses a pyproject.toml file to manage all dependencies, replacing multiple files like setup.py, requirements.txt, setup.cfg, and Pipfile.
Accessible project metadata: Project metadata is easy to access and understand from the pyproject.toml file.

What will the install actually look like

What's next:
Do this for the Flask server app if accepted.

I think we might have addressed a number of these already, and we're getting conflicts on this one now. Can you look again and resubmit?


defineModel ignores TS types

Vue version
3.3.8
Link to minimal reproduction
https://play.vuejs.org/#eNqFUstOwzAQ/JXFlxapNEK9VWklQJUAiYcAwcWXNN2mKY5t+VGKQv6dtUNDQTxO9s7semc9W7MTrYcbj2zMUpubUjuw6LwGkcliwpmznE25LCutjIMaDC6hgaVRFfSorNdRZ6rSLc7ZMAlReJYzLrnMlbQOKlvAJDzQ752jEAqelBGLg94hl2nS9qZOFDistMgcUgSQro6ndR2LmyZNKIpobLc5qtQCBckkmjNIiEuTrpwNSD71XpbFcG2VpBnrUMxZTtWlQHOjXUnaOBtDZAKXkbSXy4g543Gww/MV5s8/4Gu7DRhntwYtmg3N3HEuMwW6lp7dX+OW7h1J2r2g7D/IO7RK+KCxTTv1ckGy9/Ki2otoQSmLBzvbOpR2N1QQGjKbmM8ZORI+7rfRP+WOhqNYx2VDv7hz858l+bA5WPKYCY/k9gKXpcSrAKXWGZIIbyB9NUcTL0LQMVdKYCan/a+b8G0PSqm9A/eqMXSMv7W/AF3XKGV/C5p3e9v5Lg==
Steps to reproduce
Create a Component  with a defineModel that got multiple types
<script setup lang="ts">
const modelValue = defineModel<string | number | null | boolean>()
</script>
<template>
  <input type="text" v-model="modelValue">
</template>

Use the component.
<script setup lang="ts">
import { ref } from 'vue'
import Comp from "./Comp.vue"

const msg = ref('Hello World!')
</script>

<template>
  <h1>{{ msg }}</h1>
  <Comp v-model="msg" />
</template>

Observe warning:

What is expected?
Expected is that the component accepts the string without any errors.
What is actually happening?
The component throws a warning that it got passed a String instead a Boolean
System Info
No response
Any additional comments?
Im now defineModel is an experimental feature , but i guess the user would expect that it works as the usual prop type definition that throws no warning.

I think this is related to special handling for boolean props that we also have in other scenarios.
When we define props with a type interface, we generate the prop runtime definition without a runtime type (because the component will provide TS types to devs):
const modelValue = defineModel<string | number | null>()

// results in runtime prop def:
  props: {
    "modelValue": {},
  },

But when the prop is of type Boolean, we need to know that during runtime for casting props values properly.
const modelValue = defineModel<boolean>()

// results in runtime prop def:
  props: {
    "modelValue": { type: Boolean },
  },

Problem is that we get the same result for a union type containing boolean like in the repro:
const modelValue = defineModel<string | number | null | boolean>()

// results in runtime prop def:
  props: {
    "modelValue": { type: Boolean },
  },

I briefly reviewed the source code and it looks like it was intentionally done. In the product environment, supports Boolean and Function when options are passed in.
https://github.com/vuejs/core/blob/fc7902982a4bc9f0c9cdb0c9c222bc43a9d5c458/packages/compiler-sfc/src/script/defineModel.ts#L123-L128

@RicardoErii
This logic is incorrect. No filtering is required if there is a Boolean or Function.
https://github.com/vuejs/core/blob/fc7902982a4bc9f0c9cdb0c9c222bc43a9d5c458/packages/compiler-sfc/src/script/defineProps.ts#L262-L276

Any update on this issue ?
I've created a reproduction link (library mode) : https://stackblitz.com/edit/vitejs-vite-zz3d8x?file=dist%2Fmy-lib.js&view=editor and I've noticed another weird case.
When typing using TypeScript AND setting type option, the property is duplicated :
<script setup lang="ts">
const a = defineModel<boolean>('a');
const b = defineModel<string>('b');
const c = defineModel<string | boolean>('c');
const d = defineModel<string | boolean>('d', { type: [String, Boolean] });
const e = defineModel('e', {type: [String, Boolean]});
</script>

import { defineComponent as t, useModel as o } from 'vue';
const n = /* @__PURE__ */ t({
  __name: 'SampleComponent',
  props: {
    a: { type: Boolean },
    aModifiers: {},
    b: {},
    bModifiers: {},
    c: { type: Boolean },
    cModifiers: {},
    d: { type: Boolean, type: [String, Boolean] }, // <-- type property is duplicated with different values
    dModifiers: {},
    e: { type: [String, Boolean] },
    eModifiers: {},
  },
  emits: ['update:a', 'update:b', 'update:c', 'update:d', 'update:e'],
  setup(e) {
    return o(e, 'a'), o(e, 'b'), o(e, 'c'), o(e, 'd'), o(e, 'e'), () => {};
  },
});
export { n as SampleComponent };
//# sourceMappingURL=my-lib.js.map

A possible workaround could be to specify type property, but TypeScript complains about it in my editor :


Not scrobbling on SomaFM

Wen I play any station I get the message:
"Play some music to get started
Is it already playing?"
and the lastFM red icon, not the "?" gray one.
Pls help, thanks a bunch!

Hmm #874  might fix this for you. Are you able to run unpacked extensions? It worked for me. Would be good to have someone else verify this.

@inverse it works fine with your fix.

I never tried unpacked, only from the Web store. Thanks for helping me, I  will try the unpacked

@inverse it worked, thx allot!

Fixed by #874


Failed to use Attributes to add Constraints

Dear authors,
I am trying to use seq2pat to find patterns in about 30,000 clickstream sequences. The maximum length of the sequences is 400, while the average length is about 300. Each action is represented by an integer. At the same time, I assigned a score for each action in the sequences. The score is a number between 0 and 1, e.g., [0.9997021565900573, 0.9467735896573791, 0.8021896491513977].
In the script, I try to constrain the average scores of the patterns to be identified.
Here is my script:
score = Attribute(list_of_scores)
seq2pat = Seq2Pat(sequences)
constranted_seq = seq2pat.add_constraint(score.average() <= 0.3)
patterns = seq2pat.get_patterns(min_frequency=5)
The scripts works and returns a list of patterns. However, the issue is that when I changed constranted_seq from "score.average() <= 0.3" to "score.average() >= 0.7", I got the exactly same patterns. The constraints seems not working for my data.
Do you have any suggestions on how to solve this issue? Thanks!

2 quick comments;

You are calling get_patterns() on seq2pat, not constrained_s2p. May be that's a typo here but worth checking the actual code
Is floating points allowed for attributes? Need to double check but I am not sure

Thanks for the quick reply!

For the script, I was following this example: https://github.com/fidelity/seq2pat/blob/master/notebooks/sequential_pattern_mining.ipynb

**# Constraint to specify the average price of patterns
avg_constraint = seq2pat.add_constraint(3 <= price.average() <= 4)
Find patterns that occur at least twice within average price range
results = seq2pat.get_patterns(min_frequency=2)
print(results)**

Let me try to use integers for the attributes instead.


An openlab CLI

I think having a lightweight CLI is an important building block for release openlab v0

Agreed - just moving the spec from the README so I can push the code I've been working on:
Spec (moved from initial README)
A basic CLI that encompasses the following functions:

lab login - prompts the deposition of an estuary API key and public wallet address
lab file push dir/file - returns hash of uploaded file after calling estuary
lab file pull dir/file - returns hash of downloaded file after calling estuary
lab file ls - lists uploaded files and files that have been sent to the wallet address as NFTs
lab service ls - lists available services and jobs that have been triggered
lab service push dir/file.json 0x...42 - submits a job and triggers the signing of a transaction with metamask
lab service pull service_name - downloads a json template for the service metadata json

awesome!

service = application
openlab service list
service name, service id, provider name, provider id, price

openlab app name --example - JSON template
openlab app name  - specs for the app with links to documentation
openlab job list - job id, application id, timestamp etc.

openlab job submit job.json
openlab job status

openlab token mint jobid, metadataCID, logCID, fileCID
the jobID is an IPFS CID that is generated after uploading an input job description and automatic addition of UUID
open token mint is basically linking to a directoryCID - 4 files (as described above)

openlab token transfer
openlab transaction claim
openlab transaction pay
more functions to rate provider etc.

openlab provider blame :D

Closing for posterity


Write mcsolve as a solver

mcsolve as a Solver

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

242 of 267 (90.64%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 71.918%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qutip/solver/mcsolve.py
167
179
93.3%

qutip/solver/multitraj.py
66
79
83.54%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
13330

Relevant Lines:
18535

💛  - Coveralls


Official Docker image

Hey guys!
Quick question: is there an official Docker image for local-web-server?
If don't, what do you think about create one and make it available for the community? Maybe I could try help with this.
:grin:

Howdy :grin:!
I've created a docker image and I would like to know if you guys have any suggestions for it. The image contains: Alpine Linux, Node, Npm and local-web-server provided by npm install command. It work pretty well in our solution.
Link for the image: https://hub.docker.com/repository/docker/lucasravy/local-web-server-alpine
What do you guys think about create a section in README file with instructions of how run the server using Docker? I could do that PR, if the maintainers think that is a good idea.
Please, let me know.
Thanks!

Hi, this particular project covers only the Javascript codebase - whether ws is deployed on Linux, Mac or Windows via Docker, VirtualBox or Vagrant is a usage decision beyond the scope of this project..
However, your image sounds useful! Feel create to create a "Deployment" section at the bottom of the Wiki and add a link to a separate Wiki page (which you'll also need to create) documenting your image.
The new section would look something like this:
Deployment

Link to my new page describing my Docker image


The back button in the signup page doesn't work.

Reproduction steps
Go to the signup page .Then click the arrow button.
Expected behavior
The button should have taken the user to the main page of the website.
Actual Behavior
The button doesn't take the user to the main page due to wrong reference of the site.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Please assign me this issue.

@AbhinavBattu Go for it.

Will you assign it or i directly file a PR

Can you please tell ,why did this issue wasn't merged.Was there any problem with it.


Where is the new webmail

I don't see nothing for months, where is the new webmail?

??? Any updates?

I think this is what Roundcube Next should have been: https://open-paas.org/


[1.3.0-rc.0] Sourcemaps seem borken when using new build optimizer

First of big thanks to @filipesilva for adding the --build-optimizer! It saved about 200kb (non gzip) in overall size on a blank project with material and about 500kb on a production one, good job 👍
(as a note: up until now I could not see any real size difference when enabling the vendor chunk again, always <1kb)
Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.
    _                      _                 ____ _     ___
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/
@angular/cli: 1.3.0-rc.0
node: 8.1.1
os: win32 x64
@angular/animations: 4.3.1
@angular/cdk: 2.0.0-beta.8
@angular/common: 4.3.1
@angular/compiler: 4.3.1
@angular/core: 4.3.1
@angular/forms: 4.3.1
@angular/http: 4.3.1
@angular/material: 2.0.0-beta.8
@angular/platform-browser: 4.3.1
@angular/platform-browser-dynamic: 4.3.1
@angular/router: 4.3.1
@angular/cli: 1.3.0-rc.0
@angular/compiler-cli: 4.3.1
@angular/language-service: 4.3.1

on Windows 10
Repro steps.
Install latest<EMAIL_ADDRESS>generate a new project and
run ng build --prod --bo  --oh=none --vendor-chunk=true. (same goes when vendor chunk is disabled)
change to dist and run source-map-explorer vendor.bundle.js (same for webpack-bundle-analyzer)
or use https://github.com/bboehm86/mat-chunk-test
run npm run build:bo:vc
The log given by the failure.
No failure but just no usable informations for vendor files (there should be angular, angular material and rxjs inside)

Desired functionality.
It would be nice if still could still have usabe sourcemap infos to take a look into our bundle and see whats inside :-)

Thanks a bunch @bboehm86, this is super useful info - especially with easy repro and numbers. This is exactly the stuff I want to get fixed for 1.3.0. I'll get on it.
Regarding size differences, they were there but the stats didn't show it. We fixed the stats in https://github.com/angular/angular-cli/pull/6989, which went in rc.0 as well.

Thanks a lot for the quick response, you guys are rly doing a great job!
As for the size differences.. I guess you are refering to the vendor chunk disbaled/enabled option. I checked the actual size of the files for that (main.bundle.js and vendor.bundle.js) since I was used to do that anyways and didn't look at the console output. (but I was aware of the fix of the stats output) :-)
Since you like numbers I took a look again. So for the example repo it would be:

vc disabled: 708.961 bytes [with gzip: 153.871 bytes] (main.bundle.js)
vc enabled: 709.010 bytes [with gzip: 153.919 bytes] (main.bundle.js & vendor.bundle.js)

Not that big a difference (for this example at least)

I digged into the sourcemaps and we're definitely breaking them somewhere, and also the default was broken (prod build should have it turned off by default).
It's cool to see that the VC didn't make much of a difference in your setup. In another setup (angular.io actually) it made a huge difference. About 40% of the total savings were gone by having the VS. But I hope that webpack improvements in scope hoisting might bring that down. It's something we'll keep monitoring.

@filipesilva Good point you've got there it slipped my mind that they are disabled by default...
And you are quite right about the vendor chunk size at aio. I tried it with the version you commited with the new CLI and I'm getting the same differences in size you've mentioned. Not sure why though...
If you are going to investigate this, it might help you that I found some Material assets in the vendor chunk that shouldn't be there (as they aren't present in the version without the vendor chunk).
eg. the MdDatepicker gets removed while building since it's not used. But when using the vc the error messages, the default month names, etc. are still present in the vendor chunk file (while getting remove when disabling the vc)...

Your findings corroborate what I've been seeing as well. I think it's wholly related to scope hoisting and the current status of angular material tree shaking (https://github.com/angular/material2/issues/4137).
So by itself material is not tree-shaked properly; but with scope hoisting + uglify + build-optimizer, parts of it are. Scope hoisting is very important here as it helps uglify drop more stuff. When you have a vendor chunk, there should be at least 2 scope hoisting contexts (one for main, or for vendor). Having them separated reduces gains by not allowing uglify to drop as much.
This seems to be more important when the code is stuff to which build optimizer adds annotations (downleved TS code and angular specific code). So when you have a lot of angular libs in the same chunk, more is saved.
If you have a lot of stuff that the build optimizer doesn't touch, the difference isn't as dramatic.

I see, great explanation thanks al lot 👍

Hi @filipesilva, I'm not sure if this is related to this error, but now source map explorer is working again, but I get strange results in the reported sizes.
For instance my "main.bundle.js" file is 683kb (not gziped) but the reporting in the source map explorer is only 297kb.
Is it possible that we are still loosing some info in the source map?
( this was done with rc.5)

Will look at it, reopening.

@ianchi with 1.3.0 final I could see a difference, but not directly related to build optimizer. I built the application three times, and compared the disk size of main.bundle.js with the size reported by source-map-explorer:

--aot --build-optimizer: 3.57 MB disk-size, 3.26 MB source-map-explorer
--prod: 1.1 MB disk-size, 407 kB source-map-explorer
--prod --build-optimizer: 540 kB disk-size, 288 kB source-map-explorer

I also tried webpack-bundle-analyzer and it seems to report much closer values for stat/parsed sizes.
So there's always a difference, I'm not sure why. The difference without --prod is rather tame and could possibly be webpack helper functions.
But the difference in prod is around half the size. It seems related to either uglification or webpack concatenated modules (that's what changes in production).
@TheLarkInn do you have any insight on this?


Error building color scheme with vedo.color.buildPalette

Vedo version: 2020.4.2
vtk version: 9.0.1
Hello,
I'm attempting to build a colour palette between two hue values using HSV colour system. When I attempt to use the function vedo.colors.buildPalette, it returns: TypeError: no overloads of RGBToHSV() take 1 argument.
Upon investigating, it seems that in colors.py, hsv2rgb is indeed only passing in one parameter to HSVtoRGB, when vtk.math.HSVtoRGB doesn't only take one parameter and return the RGB, but instead edits a reference passed through a second parameter.
I changed hsv2rgb to:
def hsv2rgb(hsv):
    """Convert HSV to RGB color."""
    ma = vtk.vtkMath()
    rgb = [0,0,0]
    ma.HSVToRGB(hsv, rgb)
    return rgb

And that seemed to fix as a workaround.
Thanks,
Daniel

Hi Daniel
thanks a lot for reporting the issue, it will be fixed in the next release.
Marco


Not implemented chalk_solve::clauses::push_auto_trait_impls causing RA to fail

rust-analyzer version: unreleased (e5f252a)
The funny thing about this bug is that it's triggered only in some strange line-dependent context (after line 219 everything from semantic highlighting to completion fails):

thread '<unnamed>' panicked at 'not implemented', /home/korvin/.cargo/registry/src/github.com-1ecc6299db9ec823/chalk-solve-0.30.0/src/clauses.rs:60:40
stack backtrace:
   0: std::panicking::begin_panic
   1: chalk_solve::clauses::push_auto_trait_impls
   2: chalk_solve::clauses::builder::ClauseBuilder<I>::push_binders
   3: chalk_solve::clauses::program_clauses_for_goal
   4: chalk_recursive::recursive::Solver<I>::solve_new_subgoal
   5: <chalk_recursive::recursive::Solver<I> as chalk_recursive::solve::SolveDatabase<I>>::solve_goal
   6: chalk_recursive::fulfill::Fulfill<I,Solver,Infer>::prove
   7: chalk_recursive::fulfill::Fulfill<I,Solver,Infer>::solve
   8: chalk_recursive::recursive::Solver<I>::solve_new_subgoal
   9: <chalk_recursive::recursive::Solver<I> as chalk_recursive::solve::SolveDatabase<I>>::solve_goal
  10: chalk_recursive::fulfill::Fulfill<I,Solver,Infer>::prove
  11: chalk_recursive::fulfill::Fulfill<I,Solver,Infer>::solve
  12: chalk_recursive::recursive::Solver<I>::solve_new_subgoal
  13: <chalk_recursive::recursive::Solver<I> as chalk_recursive::solve::SolveDatabase<I>>::solve_goal
  14: chalk_recursive::fulfill::Fulfill<I,Solver,Infer>::prove
  15: chalk_recursive::fulfill::Fulfill<I,Solver,Infer>::solve
  16: chalk_recursive::recursive::Solver<I>::solve_new_subgoal
  17: <chalk_recursive::recursive::Solver<I> as chalk_recursive::solve::SolveDatabase<I>>::solve_goal
  18: <chalk_recursive::recursive::RecursiveSolver<I> as chalk_solve::solve::Solver<I>>::solve_limited
  19: hir_ty::traits::trait_solve_query
  20: salsa::runtime::Runtime::execute_query_implementation
  21: salsa::derived::slot::Slot<Q,MP>::read_upgrade
  22: salsa::derived::slot::Slot<Q,MP>::read
  23: <salsa::derived::DerivedStorage<Q,MP> as salsa::plumbing::QueryStorageOps<Q>>::try_fetch
  24: <DB as hir_ty::db::HirDatabase>::trait_solve::__shim
  25: <DB as hir_ty::db::HirDatabase>::trait_solve
  26: hir_ty::infer::InferenceContext::resolve_ty_as_possible
  27: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_inner
  28: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_coerce
  29: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_inner
  30: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_coerce
  31: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_inner
  32: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_coerce
  33: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::check_call_arguments
  34: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_method_call
  35: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_inner
  36: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_coerce
  37: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_inner
  38: hir_ty::infer::expr::<impl hir_ty::infer::InferenceContext>::infer_expr_coerce
  39: hir_ty::infer::infer_query
  40: salsa::runtime::Runtime::execute_query_implementation
  41: salsa::derived::slot::Slot<Q,MP>::read_upgrade
  42: salsa::derived::slot::Slot<Q,MP>::read
  43: <salsa::derived::DerivedStorage<Q,MP> as salsa::plumbing::QueryStorageOps<Q>>::try_fetch
  44: <DB as hir_ty::db::HirDatabase>::infer_query::__shim
  45: hir_ty::db::infer_wait
  46: hir::source_analyzer::SourceAnalyzer::new_for_body
  47: hir::semantics::SemanticsImpl::analyze2
  48: hir::semantics::SemanticsImpl::descend_into_macros
  49: ide::hover::hover
  50: std::panicking::try
  51: ide::Analysis::hover
  52: rust_analyzer::handlers::handle_hover
  53: <F as threadpool::FnBox>::call_box
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Duplicate of https://github.com/rust-analyzer/rust-analyzer/issues/6134, fixed in the nightly version.


Let's add MeasureSizeBox

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

class MeasureSizeBox extends SingleChildRenderObjectWidget {
  final ValueChanged<Size> onSizeCallback;

  const MeasureSizeBox({
    required super.child,
    required this.onSizeCallback,
    super.key,
  });

  @override
  RenderProxyBox createRenderObject(BuildContext context) {
    return _RenderMeasureSizeBox(onSizeCallback: onSizeCallback);
  }
}

class _RenderMeasureSizeBox extends RenderProxyBox {
  final ValueChanged<Size> onSizeCallback;

  _RenderMeasureSizeBox({required this.onSizeCallback});

  @override
  void layout(Constraints constraints, {bool parentUsesSize = false}) {
    super.layout(constraints, parentUsesSize: parentUsesSize);
    if (size.isEmpty) return;
    onSizeCallback(Size.copy(size));
  }

  @override
  void setupParentData(covariant RenderObject child) {
    child.parentData = parentData;
  }

  @override
  void adoptChild(covariant RenderObject child) {
    child.parentData = ParentData();
    super.adoptChild(child);
  }

  @override
  void dropChild(covariant RenderObject child) {
    if (child.parentData is FlexParentData) {
      child.parentData = ParentData();
    }

    super.dropChild(child);
    // super.dropChild(child);
  }
}

I checked with ParentdData It Worked well and with RenderObject. I am not sure about optimization. But can you see ?

OKAY IT HAS A BUG

import 'package:flutter/rendering.dart';

class MeasureSizeBox extends SingleChildRenderObjectWidget {
  final ValueChanged<Size> onSizeCallback;

  const MeasureSizeBox({
    required super.child,
    required this.onSizeCallback,
    super.key,
  });

  @override
  RenderProxyBox createRenderObject(BuildContext context) {
    return _RenderMeasureSizeBox(onSizeCallback: onSizeCallback);
  }
}

class _RenderMeasureSizeBox extends RenderProxyBox {
  final ValueChanged<Size> onSizeCallback;

  _RenderMeasureSizeBox({required this.onSizeCallback});

  @override
  void layout(Constraints constraints, {bool parentUsesSize = false}) {
    super.layout(constraints, parentUsesSize: parentUsesSize);
    if (size.isEmpty) return;
    onSizeCallback(Size.copy(size));
  }

  @override
  void setupParentData(covariant RenderObject child) {
    child.parentData = parentData;
  }

  @override
  void adoptChild(covariant RenderObject child) {
    child.parentData = ParentData();
    super.adoptChild(child);
  }

  @override
  void dropChild(covariant RenderObject child) {
    if (child.parentData is FlexParentData) {
      (child.parentData as FlexParentData).flex = null;
      child.parentData = ParentData();
    }

    super.dropChild(child);
    // super.dropChild(child);
  }
}

FIXED

Hi @Hellomik2002, what's the use case for this Widget?

Labeling for further insight from the team

you want create animation that depends on sizes of other widgets.

I don't see how you could do that with the MeasureSizeBox widget. At the time you know the size you can't really modify the widget tree anymore because the build phase is over. So how would you implement that animation?

you want create animation that depends on sizes of other widgets.

I don't see how you could do that with the MeasureSizeBox widget. At the time you know the size you can't really modify the widget tree anymore because the build phase is over. So how would you implement that animation?

You're right. I can not modify size in build phase. I can only schedule it and do it after. I am thinking about creating linking or another thing but Indeed I don't have idea how bind it to one widget's size to another widget's size. I can measure size only now and should think how make this idea compatible with phases.


Retrieve and use frontmatter slug for generating links

Updated the code to get the current file as the source for the getFirstLinkpathDest function
Retrieved the frontmatter and slug of the linked note
Used the slug from the frontmatter, if available, to generate the link URL
Adjusted linkText for proper display when using slugs, e.g., [[Hello World#hello-world]]

This PR makes the following use-cases work:
Check out [[Hello World#hello-world|hello world]].

[[Another Post]]

Correctly generating links to the slug hello-world and another-post and using hello world and Another Post as text. In the first one, we're explicit about setting the slug because the linked note might not have a slug in its frontmatter, while in the second we can rely on the note having a slug in its frontmatter.
This enables easy publishing for vaults that don't use zettelkasten, and instead use regular human readable names for their notes.

This is great, thank you @RobertoGongora!


Command for generating modlist.html

would be nice if a list of all mods used in a pack could be generated, either when exporting so that a modlist.md or modlist.html exists in the output .zip, or as a standalone command.

Oh yes, this would be nice.

@froehlichA You could look into this: https://github.com/ModdingX/ModListCreator its that but in java someone only have to recode it in nim


Hi, this is a very good idea, is there any open source program?

Hi, this is a very good idea, is there any open source program?

the code will be releases soon. it might be Nov.-------- 原始邮件 --------发件人： Watcher-shen @.>日期： 2024年11月7日周四 11:22收件人： CQU-ADHRI-Lab/DI-MaskDINO @.>抄送： Subscribed @.***>主    题： [CQU-ADHRI-Lab/DI-MaskDINO] Hi, this is a very good idea, is there any open source program? (Issue #1)
Hi, this is a very good idea, is there any open source program?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Dear Authors,
Thank you for writing a fascinating paper—a humble request to release the code kindly.
thank you


Add a "version" key to tsconfig.json

Same as the TypeScript version. This is required to increase the lifetime of tsconfig.json files. Few years down:

if someone wants to open a typescript project they should have an easy way to see why the tsconfig.json has compiler options that no longer make sense. And can ask smart questions.
tools can deny touching a file for a version they do not support.

Sample
{
    "version" : "1.4.1",
    "compilerOptions": {
        "target": "es5",
        "module": "commonjs",
        "declaration": false,
        "noImplicitAny": false,
        "removeComments": true,
        "noLib": false
    }
}

One thing to clarify, since it is a JSON file new options are actually ignored by older compilers, so is this intended for documentation purposes or for blocking compilation, i.e. is it expected that tools (compiler, editors, linters etc..) would error if they have a version that is lower than the version in the file?

new options are actually ignored by older compilers

Not necessarily always the case e.g. old options with new values : 'target':'es7' so the older compiler would barf.
Even without this it would still be awesome for documentation purposes. Just suggesting if some tooling (like VS) creates these tsconfig.json files, it should add the version field.

+1

+10 - I think this will be super useful.

One remark, I would make the version only the first 2 digits, e.g. '1.4'. There should be no breaking changes between 1.4.0 and 1.4.1 (or 1.4.8)...

Recommend consistent three numbers. Semver analysis should be left up to the clients. If they don't care about the patch number it's their decision.

:+1:

We'd like a proposal of what this does in terms of behavior, i.e. what would TypeScript do with this field? Or would it just ignore it?

We'd like a proposal of what this does in terms of behavior, i.e. what would TypeScript do with this field? Or would it just ignore it?

I choose to ignore it. But I've found that it has confused people. So in hindsight just using it for docs was a bad choice on my part 🌹 failed experiment.

Better if we say that tsconfig.json "must" support comments. And we leave it up to the creator what they feel like commenting.

We'd like a proposal of what this does in terms of behavior, i.e. what would TypeScript do with this field? Or would it just ignore it?

The case I can see this being useful is for the selection of a version-appropriate services library by editors for a given project. Which is a good use case, IMO. (Flagging the editor to automatically use the nightly services library so it won't choke on abstract and is, for example! It's be really nice if I could include the version in my tsconfig.json (or package.json... since I already declare a TS dependency version there? Not true in VS, I suppose...) and have VSCode fetch it on its own!)
But no, I don't think TS itself needs to do anything with it. TS is whatever version it is when it reads the file - it's not version managing itself, I don't think.

+1 for a "version" key.
TS could directly state whether the given tsconfig.json is supported or not (e.g., version more recent than the compiler which would be clear and wouldn't allow for side effects/surprises).
TS could also embed versioned schema files (whatever the state of those is in the json universe) so as to allow editors to provide more contextual information (auto completion and the like).
Having them part of the TS installation would be nice so as to avoid the XML schema hell where all tools embed copies of the schemas etc.
As a side note, knowing that JSON does not support comments (that is sooo wrong Mr Crockford! :p), TS could also ignore all unknown keys, maybe provide hints if a keyname resembles a known one -- or let editors handle that, or at least accept unknown keys matching some rule (e.g., start with _ or whatever).
This would allow us to put inline documentation in the file (e.g., "_target": "Use ES6 because our build pipeline then uses Babel to emit ES5 code").

As a side note, knowing that JSON does not support comments (that is sooo wrong Mr Crockford! :p)

JSON was not originally intended to be a format for humans to read and write, although that's what it's become haha. JSON was just meant to be a way for machines to talk to each other. There are other formats, like yaml and toml that are better for config files.

It does not seem that there is a need for this other than documentation. for that, adding a version property to the tsconfig file is definitely fine. the compiler will not enforce any checks though.


SharingServices.exe is missing

Hello,
in your repository last year ago, the folder  "External/Holotoolkit/Sharing/Server" contained a cool file called "SharingService.exe". You could connect several Hololenses with that server and you could use them for Multiplayer.
Was that "SharingService.exe" moved to another folder? I can't find it anymore :(

When the Toolkit was called "HoloTookit" in 2017, the SharingServices.exe was there. Was ist removed some months ago?

If you're looking for it in MRTK v2, we're still working on our sharing implementation (#2509, #2721, #2722, #3664, #4016).
If you're looking for it in HTK, it can still be found in the release branch: https://github.com/Microsoft/MixedRealityToolkit-Unity/tree/htk_release/External/HoloToolkit/Sharing/Server


context.done called twice error message confusing for async / promise returning functions

Today, error message is: Error: 'done' has already been called. Please check your script for extraneous calls to 'done'.
This is confusing when the first 'done' has been called due to the function being async or the function returning a promise. If we can tell which scenario it is, our error message should be specific to this case. (note: it should be easy to tell if the 'done' call was internal or not).

sorry to bump this issue but there are a lot of examples in the docs that have an async handler + a call to context.done(), ie: https://docs.microsoft.com/it-it/azure/azure-functions/functions-bindings-warmup?tabs=javascript#trigger---example (just to quote one)
is this an accepted pattern in v2 ?

@gunzip - thanks for pointing that out! this definitely isn't a recommended pattern. We recommend either use async only or callbacks only (using context.done()). I'll try to change those


Finished radixsort and tests

Coverage remained the same at 100.0% when pulling e18be60e6ec4c7e77a01dcf1515a0465a4dd528c on radixsort into ea74ead3a20aa1d73838f880a1d6fc6c72ee7283 on master.


Refactoring Dexie

One thing that may make it difficult for contributors to Dexie is the relatively monolithic code base. Having tried to do some amount of work on Dexie in the past, it can be difficult to identify the scope and responsibilities of individual parts when everything is technically in scope.
Can we have a concerted effort to refactor and move classes out of the main Dexie function so it is more clear what components are more tightly coupled?

Agree. The code suffers from originally being written with heavily use of closures. I think it's time to refactor it to use classes.

this would be very nice 👍

Here's what I've got down so far. This can be done over several PRs to ensure progress and reduce risk:

[ ] replace utility functions with modern equivalents - reduces boilerplate imports in each class

[ ] getProto: Object.getPrototypeOf
[ ] keys: Object.keys
[ ] isArray: Array.isArray
[ ] hasOwn: Object.hasOwnProperty
[ ] defineProperty: Object.defineProperty
[ ] asap: polyfill setImmediate instead and use that (if it's not already done)
[ ] shallowClone: Object.assign({}, obj)

[ ] Consolidate indexedDB utilities/constants into separate file
From utils:

getByKeyPath
setByKeyPath
delByKeyPath
minKey
maxKey

from Dexie:

cmp
min
max
ascending
descending
READONLY
READWRITE
safariMultiStoreFix
maxString
minKey
maxKey
adjustToExistingIndexNames
hasGetAll: also move assignment from adjustToExistingIndexNames
getNativeGetDatabaseNamesFn

[ ] Move functions used in a single context or move to more descriptive namespace
from Dexie:

tempTransaction: move to Table, only used there
preventDefault: move to Events, then use elsewhere
simpleCompare: move to WhereClause, only used there
simpleCompareReverse: move to WhereClause, only used there
parseIndexSyntax: add as static method on IndexSpec, then use elsewhere
readGlobalSchema: move most of function to static method on TableSchema
getSchemaDiff: add as static method on TableSchema

from utils:

getArrayOf: move to WhereClause, only used there
trycatcher: move to Collection, only used there

With those done, the following dependencies will still exist:

Transaction

db: may be made available via constructor (interface change)
idbdb: add as property of db and mark as private API
allTables: property of db
dbOpenError: add as property of db and mark as private API

Table

db: may be made available via constructor (interface change)
allTables: property of db

Once those interface changes are done then all classes may be moved
from Dexie and then refactored to use modern constructs:

[ ] Classes moved from Dexie body into separate files

[ ] Collection
[ ] Table
[ ] Transaction
[ ] WhereClause
[ ] IndexSpec
[ ] TableSchema

[ ] For each class, move 'private methods' out of props callback into top-level

[ ] For each class, function -> class

[ ] function-level code moved to constructor

[ ] public methods moved from return value to props callback to class body

Would it make sense to start writing things in typescript during the refactoring? Typescript is also mentioned in the vision for dexie issue (https://github.com/dfahlander/Dexie.js/issues/427). Maybe some other things from there are relevant for the refactoring.
We should also consider browser support when using modern constructs. Do we offer multiple bundles for modern and older browsers? Who adds polyfills? The user or dexie?
We would also need to consider breaking changes. It is probably safer to release v3 with so many changes. Even if not everything is done in one go, just replacing functions (example shallowClone vs. assign) might break things.

Would it make sense to start writing things in typescript during the refactoring?

I consider this issue resolved when the classes that comprise Dexie are in separate files and all tests pass. I think this is a pre-requisite for several changes like those mentioned in #427, but we can let those be separate initiatives.

Do we offer multiple bundles for modern and older browsers? Who adds polyfills? The user or dexie?

The current distribution method (single package for all users) should be maintained. Polyfill was the wrong word, I meant that the compiler should provide an implementation that gracefully handles the case when a native version doesn't exist.

It is probably safer to release v3 with so many changes. Even if not everything is done in one go, just replacing functions (example shallowClone vs. assign) might break things.

The majority of changes are private. The only publicly-visible items should be the interface changes noted for Transaction and Table, but those can be made in a backwards-compatible way.

I don't think dexie should handle the case where for example Object.assign is missing (this is what I meant with polyfills, I'm not talking about syntax like class). Let the user decide if the browsers he uses need a polyfill. This also makes sure that polyfills are just imported once.

Refactoring out classes
We can start with refactoring out classes like Table, Collection, WhereClause etc into typescript classes. If not used to typescript, just start out with plain ES6. We already use tsc to transpile the JS code, so it's just to create a file table.ts and start refactoring the code in.
What needs to be taken care of are:

Some closures used as private props today, need to be put as real props (maybe typescript-private and prefixed with underscore)
Class constructors should be instance-local and put on db (db.Table, db.Collection, etc), as it keeps it possible for an addon to extend prototype only for db instances that use that addon. This can be accomplished by either copying the prototype, or extending it for each db instance.

ES5 compliance
I still think libraries are expected to work on ES5, even though the major browsers are on ES2016 already.  This would force us to either stay away from Object.assign() or polyfill it in the transpilation process.
Minifaction size
Generally, I've been aliasing Object.getPrototypeOf(), Array.isArray etc for a reason - it makes the minified code much smaller. I could understand the readability issue though, but I'd rather see getProto() renamed to getPrototypeOf() - something that should be more readable for lots of people.

Should add that we should stay away from async/await or yield in the source code - also because we don't want the transpiled output to be too large. This will stay true as long as we need to transpile to ES5, which will be true as long as libraries are expected to be delivered in ES5. Time will come when that is not true anymore, but it's not yet.

I think we would be able to use importHelpers to polyfill missing ES5 parts including Object.assign()
etc. Described in this blog. I suppose installing tslib and marking it as an external library with in rollup config would do it for us. Needs to be verified. Would be good because then we can feel free to use Object.assign() as well as spread operators on objects.

@chrahunt What's the status of your PR? Do you want to complete your PR before anyone of use starts the refactoring?

Object spread does not strictly require Object.assign. In fact object spread and Object.assign are not equivalent. Object.assign calls setters whereas object spread does not.
I don't think adding polyfills is a good idea. It is better to just mention the needed polyfills as requirements and let the user add those. React, Angular and probably all other major libraries do the same. If we add the polyfills there is a big chance that the user will also add his/her own polyfills and there would be a bunch of duplicated code there. Also all modern browsers support ES2015 so adding a polyfill would only add more data to download for the browsers that don't need it.

Did some testing with the latest tsc. It seems tsc will transpile
const clone = {...orig};

...to use __assign(), and inline a definition of __assign var __assign = this.__assign || Object.assign || function (){...} in top of the ES5 output, as Object.assign() isn't part of the ES5 standard so it can't rely on it. It is possible to configure tsc to not do this with the option noEmitHelpers, but this will require our users to include tslib for dexie to work. I still lean to use importHelpers instead and let rollup do the treeshaking to only include the helpers that are being used, like __assign etc. The code will be small if we try avoiding all fancy stuff like async/await. Object and array spread could be very handy though and improve readability at some places.
Though if we want to use Object.assign() explicitely, tsc would leave it as is so we would require user to include polyfills. We could workaround that by using __assign() in place of Object.assign():
import { __assign } from 'tslib';

Rollup will fix it so that __assign is only defined once in dist/dexie.js.

I've started a limited refactoring work #622

Closing this as the discussion here is already implemented/refactored into typescript classes, interfaces and functions.


Create the equivalent of boring model for issues template for bugs

🚀 Feature
We should have the equivalent of boringmodel  in PL for issues template for bugs
Motivation
Just make our life easier to report and fix :)
Pitch
I reported an issue with this https://colab.research.google.com/drive/1uamw6SNaOr_4ch24JNxAj2yfgLUKfJqO?usp=sharing I think it just makes it easier overall to understand the issue.
I'd suggest changing the logo, adding a better way to create the different kind of data files.

Thanks Marc!
We'll get there, we do have a simple BoringTransformerModel and DataModule for testing: https://github.com/PyTorchLightning/lightning-transformers/blob/master/tests/core/boring_model.py exposing this to the user would be great.
I think we can have non-hydra boring model template script and a hydra CLI command to run to replicate issues.


Updatevelocity rewrite

renamed UpdateVelocity to SendTritium, added Toggles to Cruise and Regen buttons

The current switch de-bouncing code runs every iteration of the FSM (100 ms) which is about an order of magnitude longer than the expected settling time of our buttons and switches. So it probably wouldn't really do much. I'm not convinced we really need de-bouncing since we check the switches on a time interval; but if we do add it, the de-bouncing should happen more quickly (maybe with a timer callback). @babusid do you have a preferred approach here?

@Cam0Cow What if we sped up the FSM, and had a periodic timer callback running solely to handle the motor CAN messages? The FSM would push motor messages to a 1-entry buffer that overwrites itself as messages come in, and the callback just grabs the present message at the given speed.

All that's left is finishing up test cases and then this should be good to merge with beta_test_0

This is in beta_test_0. Closing.

Closes #259


Fixing the WORKER_COUNT

This code was erroring out on the WORKER_COUNT line with the following error:
Traceback (most recent call last):
  File "/opt/cloud_custodian/bin/c7n-org", line 9, in <module>
    load_entry_point('c7n-org==0.0.1', 'console_scripts', 'c7n-org')()
  File "/opt/cloud_custodian/lib/python2.7/site-packages/pkg_resources.py", line 378, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/opt/cloud_custodian/lib/python2.7/site-packages/pkg_resources.py", line 2566, in load_entry_point
    return ep.load()
  File "/opt/cloud_custodian/lib/python2.7/site-packages/pkg_resources.py", line 2260, in load
    entry = __import__(self.module_name, globals(),globals(), ['__name__'])
  File "/opt/cloud-custodian/tools/c7n_org/c7n_org/cli.py", line 49, in <module>
    WORKER_COUNT = os.environ.get('C7N_ORG_PARALLEL', multiprocessing.cpu_count * 4)
TypeError: unsupported operand type(s) for *: 'function' and 'int'

This was due to the cpu_count not having () at the end of it to call the function

doh. sorry, thanks

Thanks for the quick response Kapil!


Safari cannot install TweetDeck Enhancer

I have reported this to you on Twitter, but I assume that it has gotten lost in all your mentions. Safari throws me this error when I try installing:

Steps to reproduce on Safari 8.0.3:

Download Safari extension on https://ryandolan123.com/TweetDeckEnhancer
Click the downloaded extension in Finder.
Receive this pop-up and choose "Install":

You will receive this error.

Since I am not a Chrome user, I hope that you can fix this soon! Enjoy the rest of your day/winter break (if there is any left :P)! ^.^

Why are you using the one linked ? Try downloading the one in this repo (zipped)

@Apph3x I have tried using the one on the repo, but I figure the one on his site is considered the "official build" (even though there might not even be a difference). I've even tried loading the .safariextension folder into the Safari's Extension Builder (Safari -> Develop -> Show Extension Builder or if Develop isn't there, then Safari -> Preferences -> Advanced and select the Show Develop menu in menu bar option) and building the extension there/installing it there.

There is a difference the repo one is more recent, do u have a Safari dev cert installed, you need it to build them

@Apph3x Nope, I used my magical fairy powers to build it. Of course I have a Safari certificate, how else would I have built it? Did you ever get TweetDeck Enhancer to install in Safari?

Yeah actually I did
-----Original Message-----
From: "Ninty Apple"<EMAIL_ADDRESS>Sent: 10/01/2015 17:25
To: "ryandolan123/TweetDeck-Enhancer"<EMAIL_ADDRESS>Cc: "Jamie"<EMAIL_ADDRESS>Subject: Re: [TweetDeck-Enhancer] Safari cannot install TweetDeck Enhancer (#12)
@Apph3x Nope, I used my magical fairy powers to build it. Of course I have a Safari certificate, how else would I have built it? Did you ever get TweetDeck Enhancer to install in Safari?
—
Reply to this email directly or view it on GitHub.=

@Apph3x Forgive me for being skeptical, but may I see a screenshot?

Yeah, sure, although this is the old, old 4.0 build
-----Original Message-----
From: "Ninty Apple"<EMAIL_ADDRESS>Sent: 12/01/2015 03:07
To: "ryandolan123/TweetDeck-Enhancer"<EMAIL_ADDRESS>Cc: "Jamie"<EMAIL_ADDRESS>Subject: Re: [TweetDeck-Enhancer] Safari cannot install TweetDeck Enhancer (#12)
@Apph3x Forgive me for being skeptical, but may I see a screenshot?
—
Reply to this email directly or view it on GitHub.=

@Apph3x @nintyapple I have done the exact same thing as you and I have also been unable to install the extension on Safari 8.0.3. I have also tried the one from the website and signing the one here with a certificate but that has produced the exact same error.

yep @tbraun1551

@Apph3x Is there a fix for it

IDK @tbraun1551

Anybody tried this on 8.0.4 yet?


UI: Add support for hiding columns.

Sometimes you only want to view some columns. It would be nice to have an option to hide/show certain columns.
Maintain this option when switching between tables

There are some options we have here:

In custom queries show only returned columns
Option to hide columns, but clashes with option 1

Or both?
When using a custom query ignore the hide column option.


fix(deps): update dependency puppeteer to v4

This PR contains the following updates:

Package
Type
Update
Change

puppeteer
dependencies
major
3.3.0 -> 4.0.1

Release Notes

puppeteer/puppeteer
v4.0.1
Compare Source
Bug fixes

The link the troubleshooting docs is now fixed and links to the main branch, not the removed master branch. Fixes #6082.

Raw notes
16972ca - pull in master => main renaming
b537a67 - chore: mark v4.0.1
v4.0.0
Compare Source
Breaking change: Puppeteer no longer uses Node’s EventEmitter library
As part of our work to make Puppeteer agnostic of its environment we are removing the dependency on Node’s EventEmitter in favour of an event emitter that is not tied to Node. Under the hood we use Mitt, but we extend Mitt with additional functionality to match most of the methods that Node’s EventEmitter provides. The following methods have been removed from Puppeteer classes that extend EventEmitter:

eventNames()
getMaxListeners()
listeners(eventName)
prependListener
prependOnceListener
setMaxListeners(n)
rawListeners(eventName)

Additionally all the static methods on the EventEmitter class are not supported:

listenerCount(emitter, eventName)
defaultMaxListeners
errorMonitor

Node’s EventEmitter emitted a newListener event when a listener was added and a removeListener event when one was removed. These are not supported and will not be emitted.
New features and improvements

Send/receive debug logs are now split out into separate channels making the logs much clearer (#6017)
You can now call isJavaScriptEnabled() on a Puppeteer page to find out if JS is enabled on the page (#5993)

Bug fixes

Puppeteer is now much better at killing lingering browser processes, especially when you exit a test run with Ctrl-C (#6011)

Behind the scenes

We've started work on a new documentation system that uses TSDoc to generate documentation from our source code and have been porting documentation into code accordingly.

Raw notes
5493494 - chore: mark version 4.0.0
03ab1c1 - fix: improve Ctrl + C support (#6011)
b659969 - chore: migrate away from Node's EventEmitter (#5979)
6e060ce - fix(connection): separate send/receive debug logging (#6017)
c701ea1 - chore: fix npm test command (#6010)
398c16d - chore: fix docs linter (#6009)
4205ff7 - docs(api): remove subheader for non-namespace
3d56a9e - chore: add test configuration options for running tests against multiple products (#5964)
5c91dfb - chore: move index.js into src (#6007)
8a099a0 - docs: replace @return with @returns (#6006)
23f18d8 - docs(new): start documenting the Page class (#6001)
c1d7be3 - docs(api): add copy-pasting text example to Mouse class (#6000)
91eb745 - chore: ensure new-docs are up to date (#5994)
b86ff21 - feat(api): add Page.isJavaScriptEnabled + mark properties as private (#5993)
354f942 - feat: improve error reporting on aarch64 (#5167)
9c656d4 - chore: remove prefers-color-scheme: no-preference (#5990)
83c29d9 - chore: clean lib when running tsc (#5982)
ca8b0d6 - feat(new-docs): migrate Dialog documentation to TSDoc (#5981)
0b3d52a - feat(new-docs): add TSDoc comments to Accessibility (#5971)
086c089 - chore: commit base new-docs (#5973)
0032420 - chore: Introduce API Extractor and start generating documentation (#5967)
ef63c64 - test: check installation with Firefox (#5965)
c2d32b1 - chore: rename image assets
a84e07c - chore: increase readability of Puppeteer diagram (#5947)
58d1eab - chore: mark version v3.3.0-post (#5960)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v5.1.4
GitHub release

Your semantic-release bot :package::rocket:


Updates to breaking-changes.md

Specifically:

New note for https://github.com/aspnet/EntityFrameworkCore/issues/13573
First attempt at a summary--feedback appreciated. Not sure if a table is right here?

@ajcvickers  I'm fixing all the broken #in-page-bookmarks in #1586
Next time you have a page like this, I can edit the page and add the bookmarks.
cc @tdykstra

@ajcvickers I would like to see how the table looks like in the review page, but apparently it is not being rendered.
It is possible that something is broken in the build (I sent email asking for help), but the build is also complaining about numerous invalid links in this particular PR. For example, #The-Relational:TypeMapping-annotation-is-now-just-TypeMapping should be #the-relationaltypemapping-annotation-is-now-just-typemapping, so perhaps that is blocking the review page form being published.

Ah, I missed your message @Rick-Anderson. Let's see how the preview looks like in your PR.

Reviews have been re-enabled as of this morning so I was finally able to look at what the change looks like. The formatting of the table looks pretty neat and I like having the information about the impact alongside the issue.
On the other hand, it is hard for me to appreciate the value this adds when I compare it with the clunckiness it introduces in to both the UI (the whole list is now repeated three times) and to the process of updating the documentation page.
Perhaps we should figure out a way to add the impact alongside (or below) individual tittles.

@ajcvickers @divega  the  Internal Review URL  page is working.

@divega My thoughts after looking at it:

There is too much here not to have a summary where one can scan a list to find what's here
The summary created to the right is useless--it's just a squashed up mess

If we could get it not to render, then great, but I doubt that is possible
I can't think of any way to make it readable without making the contents so terse that they are useful
So mentally, I'm just abandoning it as an unfortunate U.I. implementation

I can't currently think of a different way of putting the summary at the top, but I'm open to ideas.

Bottom line for me:

There needs to be an effective summary
The panel at the right is useless for this (in this case--it does work somewhat in other docs, although I think it looks pretty bad most of the time.)
I can't currently think of a better way to do the summary.

The summary created to the right is useless--it's just a squashed up mess

I can get rid of that

@Rick-Anderson Great!

@ajcvickers approve  #1597 - see Internal review URL which removes everything but summary. Lef me know if I can S&M into your branch.

@Rick-Anderson That looks pretty good to me.
@divega Thoughts?

@ajcvickers, I agree that now it looks pretty good. Thanks @Rick-Anderson, nice hack! Now that we overcame the main issue, I will finish the review.


Make enumeratee classes private and add new one

Codecov Report

Merging #191 into master will increase coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #191      +/-   ##
==========================================
+ Coverage   97.06%   97.13%   +0.07%
==========================================
  Files          28       28
  Lines         853      839      -14
  Branches       46       54       +8
==========================================
- Hits          828      815      -13
+ Misses         25       24       -1

Impacted Files
Coverage Δ

core/src/main/scala/io/iteratee/Enumeratee.scala
98.74% <100%> (+0.47%)
:arrow_up:

Continue to review full report at Codecov.

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


New editor DND insert cursor shows up in notebook cell editors

Drag a notebook cell over another one
The cursor shows up like a drop location inside the editor, but it shouldn't do that here

I assume this is related to the new feature to drag file paths into editors? Do you know about this @mjbvz?

Good find! I've gone ahead and disable dropping into the in all editors except our main text editors
In the future though I would like to support dropping into notebook cells too. Will investigate how that can be supported as we continue working on this api


When using pie chart this warning arise "Each child in a list should have a unique "key" prop".

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.

Here is the diff that solved my problem:
diff --git a/node_modules/react-native-gifted-charts/src/PieChart/main.tsx b/node_modules/react-native-gifted-charts/src/PieChart/main.tsx
index d524aa1..fe8a975 100644
--- a/node_modules/react-native-gifted-charts/src/PieChart/main.tsx
+++ b/node_modules/react-native-gifted-charts/src/PieChart/main.tsx
@@ -371,7 +371,7 @@ export const PieChartMain = (props: propTypes) => {
             // console.log('ax', ax);
             // console.log('ay', ay);
             return (
-              <>
+              <React.Fragment key={index}>
                 {/* <Line x1={mx} x2={cx} y1={my} y2={cy} stroke="black" /> */}
                 {showTextBackground && (
                   <Circle
@@ -441,7 +441,7 @@ export const PieChartMain = (props: propTypes) => {
                   }}>
                   {item.text || (showValuesAsLabels ? item.value + '' : '')}
                 </SvgText>
-              </>
+              </React.Fragment>
             );
           })}
       </Svg>

This issue body was partially generated by patch-package.

Thanks @Huzaifa1911 for pointing out the issue and also suggesting a working solution. I have made the suggested change. It is available version 1.3.0 onwards.


DBZ-4911 Use Jedis' clientSetname when establishing Redis connections

@spicy-sauce This is good feature! Could you please just convert the hard-coded strings into constants and ideally centralize them in a single class? Thanks.


Filter and group features by prefix in the Azure Portal

Like Azure App Configuration Configuration Manager, have a filter on the portal to filter for prefixes; application ID prefixes are the documentation guidance for configuration and feature multitenancy.
In addition, it would be really nice to group by feature and configuration name prefix.

Hi, @mack0196
If my understanding is correct, the Portal has supported the scenario you've mentioned.

Could you double check about this?


[Bug]  REST API endpoint broken - /v1/completions

🐛 Bug
With mlc_chat.rest running the example program sample_openai.py,  server crashes when /v1/completions endpoint is invoked by the sample program.
To Reproduce
Steps to reproduce the behavior:

set up mlc_chat.rest to handle request, I used llama-2 7b f16 model
run the sample_openai.py program (require installing the openai package)
server crashes in the 3rd part of this program when the /v1/completions  endpoint returns

File "/Users/sam/miniconda3/lib/python3.11/site-packages/starlette/routing.py", line 276, in handle
    await self.app(scope, receive, send)
  File "/Users/sam/miniconda3/lib/python3.11/site-packages/starlette/routing.py", line 66, in app
    response = await func(request)
               ^^^^^^^^^^^^^^^^^^^
  File "/Users/sam/miniconda3/lib/python3.11/site-packages/fastapi/routing.py", line 273, in app
    raw_response = await run_endpoint_function(
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/sam/miniconda3/lib/python3.11/site-packages/fastapi/routing.py", line 190, in run_endpoint_function
    return await dependant.call(**values)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/sam/miniconda3/lib/python3.11/site-packages/mlc_chat/rest.py", line 156, in request_completion
    CompletionResponseChoice(
  File "/Users/sam/miniconda3/lib/python3.11/site-packages/pydantic/main.py", line 150, in __init__
    __pydantic_self__.__pydantic_validator__.validate_python(data, self_instance=__pydantic_self__)
pydantic_core._pydantic_core.ValidationError: 1 validation error for CompletionResponseChoice
finish_reason
  Field required [type=missing, input_value={'index': 0, 'text': "I'm...my best to assist you."}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.0.3/v/missing

Expected behavior
Example program not to crash, but to print a completion in this 3rd part similar to the first two part.
Environment

Platform  metal
Operating system  Mac OS Ventura 13.4.1
Device   M2 Mac Mini
How you installed MLC-LLM (conda, source):  miniconda3
How you installed TVM-Unity (pip, source): NA
Python version (e.g. 3.10):  3.11.4
GPU driver version (if applicable):
CUDA/cuDNN version (if applicable):
TVM Unity Hash Tag (python -c "import tvm; print('\n'.join(f'{k}: {v}' for k, v in tvm.support.libinfo().items()))", applicable if you compile models):
Any other relevant information:

Additional context
Seems to be in the return value not conforming to class CompletionResponseChoice

Same as https://github.com/mlc-ai/mlc-llm/issues/581. We are putting up a PR to add support for pydantic v2.

Awesome thanks.  Will test and confirm with nightly.

Verified fixed in Nightly
However as pointed out in #592       The examples/rest/langchain.py    now fails.
It required installation of pydantic less than 2  (which is actually in the comments of the example now)  -
pip install pydantic==1.10.9


Javascrpt generated icon has width of tag text

This:
var materialicon = document.createElement('i');
    materialicon.classList.add('material-icons');
var icon_check = materialicon.cloneNode(true);
    icon_check.innerHTML = 'check';
var button_delete = document.createElement('a');
    button_delete.classList.add('btn');
    button_delete.classList.add('waves-effect');
    button_delete.classList.add('waves-light');
    button_delete.append(icon_delete);

Results in:

  icon_delete

Which is correct, but the button has the width of the text "icon_delete".

I'm not sure what you are trying to achieve. Can you make a codepen?

Written HTML works, programmatic via JS, which has the same endresult, doesn't. It then takes the width of text content "icon_delete"


Suggestion: Add a .text method for plain text tooltips

Sometimes you just want to display plain text in your tooltips, and doing so requires that you first escape your text before you use it for .html(). If you forget, you can introduce XSS security issues. It would be nice if there was a .text() method you could use instead, so you don't have to worry about escaping text yourself. This would be similar to .text() in JQuery.

Here is some activity. Don't automatically close issues with a robot.

Here is more activity.

Here is more activity.

The library is no longer developed... https://github.com/Caged/d3-tip/issues/239#issuecomment-443053926.

Maybe someone can fork it and work on a fork.

Here is more activity. ;)

More activity.

Don't stale this!


Mismatch between formal-spec and executable-spec UTXOW rule

The Byron-era ledger formal spec (link) requires the UTXOW rule (p. 12, Fig. 10, rule 6) to check that the transaction size is at most the maximum allowed size. The executable spec implementation does not (link) check this.

Closing as duplicate of #971 - sorry!


api: http 500 error on list-all-tx

Feb 24 20:14:42 raspberrypi python[2296698]: Task exception was never retrieved
Feb 24 20:14:42 raspberrypi python[2296698]: future: <Task finished name='Task-4051' coro=<warmup_new_connections() done, defined at /home/blitzapi/blitz_api/app/main.py:278> exception=AttributeError("'HTTPException' object has no attribute 'dict'")>
Feb 24 20:14:42 raspberrypi python[2296698]: Traceback (most recent call last):
Feb 24 20:14:42 raspberrypi python[2296698]:   File "/home/blitzapi/blitz_api/app/main.py", line 303, in warmup_new_connections
Feb 24 20:14:42 raspberrypi python[2296698]:     _send_sse_event(id, SSE.SYSTEM_INFO, res[0].dict()),
Feb 24 20:14:42 raspberrypi python[2296698]:                                          ^^^^^^^^^^^
Feb 24 20:14:42 raspberrypi python[2296698]: AttributeError: 'HTTPException' object has no attribute 'dict'
Feb 24 20:14:44 raspberrypi python[2296698]: 2024-02-24 20:14:44 | ❌ | service.py:210 | Initializing Bitcoin Core (loading, verifying blocks or starting network threads etc)

This is also still a problem with the v1.11.0rc3

@fusion44 is there already an issue on the API repo addressing this or schould I open a different one?

This seems the corresponding issue on the API repo: fusion44/blitz_api#247

They are two different problems. This issue should be fixed in the dev branch.

This looks good now. Closing for release.


Adding mongoid as a supported ORM.

ORM to create correct simple_form initializer. Extract
the ORM-specific code into modules that the initializer
will use.
Extraction code inspired by Rafael Franca's initial PR.
Tested for Rails 3.0.9 and Mongoid 2.0

It looks like there's still an issue using mongoid and f.association method implying a belongs_to association.
Not passing a collection:
undefined method `options' for #<Mongoid::Relations::Metadata:0xb33b3cc>

Passing a collection:
undefined method `category_ids' for #<Product:0xb36dc14>

I'm concordant with Github Issues being not a place to ask for, but any kind of help (hacks or workaround will do) on this would be highly appreciated.

hey @gmile, passing the collection it should work. I worked with a project that we used SimpleForm and Mongoid without any problems. But if I remember you need to pass the name of field like this:
f.input :category_id, :collection => Category.all

Maybe in the SimpleForm 2.1 we can work to integrate with MongoId.
Thanks to use SimpleForm and for ask here.

@rafaelfranca it helped! Thank you so much, Rafael!

@gmile glad to know that it worked :smile:


feat: detects usage of global variable 'global'

if global is used in modules then 'var global = this' is injected in the head of paeckchen

Coverage decreased (-0.9%) to 99.111% when pulling 0492071fd10354a7296bc6f3966f9283c68c5b24 on detect-global-var into d627fef592cfd5d79fb4271d65f4726b4bd41718 on master.

Coverage remained the same at 100.0% when pulling 6f7fbb28947f469f779f0db515396b8c9e9895b6 on detect-global-var into d627fef592cfd5d79fb4271d65f4726b4bd41718 on master.


Mixtral MoE support?

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[x] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[x] I carefully followed the README.md.
[x] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[x] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Feature Description
Support for Mixtral MoE

https://github.com/SJTU-IPADS/PowerInfer/issues/13

@mathieu-heyarctic https://github.com/SJTU-IPADS/PowerInfer/issues/13 is for Mistral, not Mixtral

This would be OP


Add @rspack/core to the peer dependencies of html-webpack-plugin and react-refresh-webpack-plugin

What problem does this feature solve?
Can we add @rspack/core to the peer dependencies of html-webpack-plugin and react-refresh-webpack-plugin (and more popular webpack plugins), then make webpack as an optional dependency?

https://github.com/jantimon/html-webpack-plugin/blob/main/package.json#L58-L60
https://github.com/pmmmwh/react-refresh-webpack-plugin/blob/main/package.json#L114-L123

The benifits:

Users do not need to install webpack as a dependency.
No peer dependency missing warning.

What does the proposed API of configuration look like?
{
  "peerDependencies": {
    "@rspack/core": ">=0.3.0",
    "webpack": ">=4.43.0 <6.0.0",
  },
  "peerDependenciesMeta": {
    "webpack": {
      "optional": true
    },
   "@rspack/core": {
      "optional": true
    }
  },
}

yes, I already did this to https://github.com/webpack/webpack-dev-server/pull/4778

Update: using forked html-rspack-plugin instead of html-webpack-plugin
https://github.com/rspack-contrib/html-rspack-plugin


BrnColumnDefComponent broken in Angular 17.2

Please provide the environment you discovered this bug in.
Use the table component in any angular 17.2 application.
Which area/package is the issue in?
table
Description
In Angular 17.2 the Input Signal Type changed to a Generic with only one input.  The current usage of the BrrnColumnDef Component will break the Application during an 17.2 Update.
Please provide the exception or error you saw
X [ERROR] TS2314: Generic type 'InputSignal<ReadT>' requires 1 type argument(s). [plugin angular-compiler]

    node_modules/.pnpm/@spartan-ng+ui-table-brain@0.0.1-alpha.336_@angular+cdk@17.2.1_@angular+core@17.2.2_@spartan-ng+ui-core@0.0.1-alpha.336/node_modules/@spartan-ng/ui-table-brain/lib/brn-column-def.component.d.ts:10:20:
      10 │ ...adonly class: import("@angular/core").InputSignal<string, string>;

Other information
17.1:
/**
 * `InputSignal` is represents a special `Signal` for a directive/component input.
 *
 * An input signal is similar to a non-writable signal except that it also
 * carries additional type-information for transforms, and that Angular internally
 * updates the signal whenever a new value is bound.
 *
 * @developerPreview
 */
export declare interface InputSignal<ReadT, WriteT = ReadT> extends Signal<ReadT> {
    [SIGNAL]: InputSignalNode<ReadT, WriteT>;
    [ɵINPUT_SIGNAL_BRAND_READ_TYPE]: ReadT;
    [ɵINPUT_SIGNAL_BRAND_WRITE_TYPE]: WriteT;
}

17.2:
/**
 * `InputSignal` represents a special `Signal` for a directive/component input.
 *
 * An input signal is similar to a non-writable signal except that it also
 * carries additional type-information for transforms, and that Angular internally
 * updates the signal whenever a new value is bound.
 *
 * @see {@link InputOptionsWithTransform} for inputs with transforms.
 *
 * @developerPreview
 */
export declare interface InputSignal<ReadT> extends InputSignalWithTransform<ReadT, ReadT> {
}

I would be willing to submit a PR to fix this issue

[X] Yes
[ ] No

Should be automatically be fixed if this projects is updated to any 17.2 version.

We have the same problem with select component on angular 17.1.3:
Error: node_modules/@spartan-ng/ui-select-brain/lib/brn-select.component.d.ts:17:10 - error TS2314: Generic type 'InputSignal<ReadT>' requires 1 type argument(s).

dir: import("@angular/core").InputSignal<BrnReadDirection, BrnReadDirection>;

I just released a new version. It's now on 17.3 of Angular. Please let me know if this fixes the issue

@badsgahhl @toliklevy should be good in the newest release


[BUG] App Configuration eTag should not be type String

Describe the bug
eTag should be it's own type. See DotNet SDK https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/appconfiguration/Azure.Data.AppConfiguration/src/ConfigurationSetting.cs in the DotNet SDK this is part of Azure core https://github.com/Azure/azure-sdk-for-net/blob/965869fd80da2487ad08389a8cc6172f6d63d68d/sdk/core/Azure.Core/src/ETag.cs.

Fix this to stay language consistency but having breaking change is not a worth action to take.
Wait to fix it until we ship the next Major version.

This should stay active


Update to use branch-url-update-action action

branch-url-update-action

Reopening to address issue arising from pushing to a protected main branch


feat(tasks-processor): Add recurring task to clean up old recurring task runs

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[ ] I have added information to docs/ if required, so people know about the feature!
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Add a recurring task to delete recurring task runs older than RECURRING_TASK_RUN_RETENTION_DAYS
How did you test this code?

By running task processor manually to verify, old task runs were deleted
Adds unit tests

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (2a9cd7c) 95.95% compared to head (6d1338d) 96.11%.
Report is 18 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3151      +/-   ##
==========================================
+ Coverage   95.95%   96.11%   +0.15%
==========================================
  Files        1053     1058       +5
  Lines       31588    31914     +326
==========================================
+ Hits        30311    30673     +362
+ Misses       1277     1241      -36

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add TextAnnotation

Description
Foxglove Studio currently supports 2D text annotations via an unofficial extension to visualization_msgs/ImageMarker. Bringing support to foxglove schemas would allow us to drop this ROS message.

Related https://github.com/orgs/foxglove/discussions/611


Android debug only recieves notifications after unstalling app

On Android I only receive notifications after I uninstalled the app and start a new debug session. If I stop the debug session and start a new one, notifications are not received anymore. To receive notifications again I have to uninstall the app and start a new debug session.

Fixed by setting resetToken to false in DEBUG as well:
#if DEBUG
            //FirebasePushNotificationManager.Initialize(this, true);
            FirebasePushNotificationManager.Initialize(this, false);
#else


CSHARP-2724: POC - Specification for handling native UUID types

Evergreen patch build:
https://evergreen.mongodb.com/version/5dd15cb661837d1a3376c44b

If you've been waiting to review this until the prose tests were written they are now written.

New Evergreen patch build:
https://evergreen.mongodb.com/version/5dfc3d5a3627e0526c6bcd88

Thanks @DmitryLukyanov for the good suggestions and for prototyping them in a commit.
I have adopted your concepts with some renaming and refactoring:

I want to keep GuidRepresentationMode as a simple enum, so I did not make it a class

I also didn't add a ThrowIfInvalidMode helper method because I don't want to add such a method to the public API

I renamed ClassDataValuesAttribute to ClassValuesAttribute ("ClassData" is for test cases)

I added an IValueGenerator interface to simplify type checking in ClassValuesAttribute class. While it duplicates the existing IValueGeneratorAttribute interface, I didn't want to use an interface with the word "Attribute" in it where there was no attribute involved.

I replaced your GuidRepresentationModeTestCases class (which returns test cases) with GuidModeValues (which returns values and is used with the existing ParameterAttributeData and the new ClassValuesAttribute classes).

I renamed ProtectGuidRepresentationAttribute to ResetGuidModeAfterTestAttribute

I shortened the name TemporaryGuidRepresentationMode to GuidMode. "Temporary" only really made sense when Set returned an IDisposable.

New Evergreen patch build:
https://evergreen.mongodb.com/version/5e155b4ed6d80a64ad124bd3

Ready for re-review.

Squashed, rebased and merged to master.


Module 2 exercises [Bakwena Keabetswe]

Please "drag & drop" exercise 1 pdf file below.
exercise 1.pdf

Please "drag & drop" exercise 2 pdf file below.
Exercise 2 - M2.pdf

Please "drag & drop" exercise 3 pdf file below.
exercise 3.pdf

Please "drag & drop" exercise 4 pdf file below.

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

If you haven't done so, please go back to Module 2 and make sure to take the post-training survey. Participating in surveys is required for receiving the "GALUP Certificate" by the University of Florida.

Thanks for your submission Bakwena. You have done well and showed you grasped the use of the tools. Keep pushing!


Added ADR explaining the implementation of Realistic Job Preview Link Tracking

Context
We want to outline how we will implement Candidate tracking when following the Realistic Job Preview links added to email content.
Changes proposed in this pull request
Introduction of a new ADR for review
Guidance to review
N/A
Link to Trello card
https://trello.com/c/csDRylWV
Things to check

[x] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[x] API release notes have been updated if necessary
[x] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[x] Required environment variables have been updated added to the Azure KeyVault

 Implement Realistic Job Preview (RJP) links

 Add link and content to https://platform.teachersuccess.co.uk/p/XK4rV0xN16 appending the query param user_id=#{candidate.pseudonymised_id}


43 UI fixes

Fixes #43

[x] I have added the tests to cover my changes.
[x] I have updated the documentation, VERSION, and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@valentinkostadinov please sign on behalf of your employer


About the performance of PCPA

Hi，
After I retrained PCPA with the settings in the paper, the performance was not as good as the data in the paper. While you were answering your other question, I noticed that the performance on JAAD would drop due to some issues, but it was still higher than my results. I use the windows system to train and do not use docker. I want to know if this is the main reason and whether using docker will affect the experimental results.
Thanks for your answer.

Closing due to inactivity


Fabric support

Would be cool if you guys made a server-side Fabric mod version of this plugins

Adding support for a new platform poses many challenges, including but not limited to the initial development and active maintenance of that platform. I personally consider Fabric as a server platform still too niche and thus deem active support not worth the effort.
A solution for you would be to use BungeeCord in front of your Fabric server and running our plugin on the BungeeCord.
Should the demand for Fabric rise, I'd be happy to look further into supporting it natively.


Remove Unused pt5 Code

None of this code is used, so it should probably be removed to avoid confusion.

After checking these codes, running the pT5 building don't give error when removing these functions. PR merged.


Receive input from non-GUI/WPF以外からの入力受付

Support non-GUI input to swap avatar etc.
Should consider before implement:

Support hotkey or not
Messaging style
Whether GUI-equivalent message set is enough or not
Risk the GUI indication becomes incorrect

ref: https://twitter.com/xisumavoid/status/1321008802884427776

Oh! Thank you for considering this idea!
I would use this process to change my avatar on my streaming. I have several different outfits for my character and change it manually. If this is even possible, I will have to see if Twitch can talk to the program and switch the avatar for me when the chat uses the right command.
I don't have any programming background, so I don't think I can help with the creation of any code. But i am very interested in this feature.

Plan:

GUI process (WPF) supports non-GUI input by option.
For the first version, supports following commands only

Load model, with local file path (without license check)
Setting file load, with file path

This support leads model license check skip, so:

Before enabling this option, GUI shows warning about model license management
Does not support VRoid SDK based model load automation

Plan:

GUI process (WPF) supports non-GUI input by option.
For the first version, supports following commands only

Load model, with local file path (without license check)
Setting file load, with file path

This support leads model license check skip, so:

Before enabling this option, GUI shows warning about model license management
Does not support VRoid SDK based model load automation

Implemented in #446 and above PR in WPF, https://github.com/malaybaku/VMagicMirrorConfig/pull/143 .
Available in v1.6.2.


generated _RLexer doesn't build

Hi Holger,
I'm interested in contributing to this plugin. I love R and IntelliJ.
However, I checked out Master and have hit some issues:

/misc/JFlex/JFlex.jar was missing
I downloaded 1.6.1 to move on
Ant task create_flex4r then didn't work
I commented out <arg value="--charat"/> to move on
Now, the new _RLexer.java doesn't build

Can you advise on how to get up and running with the plugin?
-Andy

Hey Andy, thanks for interest in r4intellij.
I do it as follows.

Install jflex plugin in Intellij
Restart Intellij. Now there's a new menu entry to rerun the lexer generation

Somehow the  import import com.r4intellij.psi.RTypes;
is missing in the generated com.r4intellij.lang.lexer._RLexer, which I add manually.

The jflex plugin somehow picks up the lexer-build config from ./build.xml.
I'm aware that having a proper build-script for creating the lexer would be better. However, since I hardly ever had to change the lexer code, it worked well so far using the clicky-clicky method described above.
-H

Believe it or not, the secret is that comment at the top of _RLexer.java in the background of your screenshot. I found a copy of JFlex.jar from the Intelli-arc project on com.google.com. That version of jflex accepts the option -charat which was need to get everything working. Link to jar
I've now starting playing around with rebuilding the grammar. Which .bnf file are you currently using?
-Andy

You're right, I was not using the JFlex plugin but simply ant. I thought that JFlex.jar would be part of the repo, but the used .gitignore prevented it. I've bundled it with f90360cf641a527c9ecfd8fbfc71bb701cf9aea7.
I'm using gramgen/RGrammar.bnf. To build it I'm using the IJ plugin 'Grammar-Kit'. It adds a menu entry named Generate Parser Code

if the bnf file has the focus. Running it will recreate all the code under gen


Swapping tile levels

After a deep dive into the workings of the tile layer loading. I would like to toggle visibility of layers when zooming in and out.
I need to set in componentDidUpdate the prevProps.tileLevel to visible = false and see if this does the job.

Loading is smoother without this.  I have fixed tile loading in the custom-shaders branch, tiles without images yet are now hidden.  I was not using uniforms properly.  In order to test logic on them you need to evaluate the value of a uniform.

Going to cherry pick the optimisations from custom-shaders

Needs testing, not sure with the current implementation if this works in the best way. Maybe this is only specific when using custom shader that don't have transparency!

Closing for now.


[Feature request] Provide search pattern as command line parameter

Some editors allow to specify search pattern as a command line parameter. When editor opens the file it positions cursor at the found pattern and allows to search for next occurrences.
It would be great if micro would support command line arguments such as --find <regexp> and --find-literal <string>

Would a Lua command be a more powerful and generic pattern? If you are calling from an environment rich enough to generate a search string, then generating a string of Lua commands should not be difficult. A dofile command would run a whole file of Lua script. G

@Gavin-Holt Could you please be more specific? What are you suggesting?

It can be done using plugin:
local micro = import("micro")
local buffer = import("micro/buffer")
local os = import("os")

local onstart = true

function onBufPaneOpen(bp)
	if onstart == false then
		return
	end

	local str = os.Getenv("FINDSTR")
	if str ~= "" then
		bp:Search(str, false, true)
	end
	onstart = false
end

And you run FINDSTR=string micro file.txt.
However found string is not fully visible since the text scrolled left after bp:Search. @zyedidia, any recommendations how to scroll to the beginning of the line and keep a selection?

I have thought about this approach.
I don't like the idea of fiddling with environment variables.
I would prefer command line approach. I need it for the use cases when micro is launched not by a shell but by some program that executes it directly.
Is there any way to get access to the command line parameters from within the plugin?

You can wrap env var into script (i.e. FINDSTR=$1 micro $2) and run as ./script string file.txt. Would it work for you?

I need this feature mostly for the advanced Find in files function that I use:
# Advanced Find in Files

fif() {
    HIDDEN=--hidden
    if [[ -n "$2" ]]
    then
        GLOB="--glob=\"$2\""
    else
        unset GLOB
    fi
    LIST_FILES="ugrep --dereference-recursive --files-with-matches --ignore-case --exclude-from=~/.gitignore --ignore-files=.gitignore ${HIDDEN} ${GLOB}"
    PREVIEW="ugrep --ignore-case --pretty --color=always --context=10 --line-number {q} {}"
    # VIEW_CMD="sandy -r -f {q} {}"
    # EDIT_CMD="sandy -f {q} {}"
    VIEW_CMD="FINDSTR={q} micro --ignorecase true --readonly true {}"
    EDIT_CMD="FINDSTR={q} micro --ignorecase true {}"
    local file
    file="$(
           FZF_DEFAULT_COMMAND="${LIST_FILES} \"${@}\"" \
           fzf --sort \
               --preview="[[ ! -z {} ]] && ${PREVIEW}" \
               --disabled \
               --query "$1" \
               --bind "change:reload(${LIST_FILES} {q}),f3:execute(${VIEW_CMD}),f4:execute(${EDIT_CMD})" \
               --preview-window="66%:wrap"
    )" && cd $(dirname "${file}")
}

This function allows me to find (amazingly fast) a string in the directory tree and preview found strings in the grep manner: the lines containing the string plus some context.
When I press F3 the currently selected file will be opened in an editor (readonly mode) and when I press F4 the file will be opened in the editor (normal mode).
While playing with micro I discovered weird behavior:

micro ignores --readonly true when it is launched by fzf allowing to edit the loaded buffer.
micro doesn't prompt to save the changed buffer before exit when it is launched by fzf.

I think this has something to do with redirected stdin and stdout.
Does anybody know how could I workaround this problem?
Should I open new Bug/Feature request?

You can wrap env var into script (i.e. FINDSTR=$1 micro $2) and run as ./script string file.txt. Would it work for you?

This still would require the launching process to run a shell instead of running micro, directly, right. It is similar to
sh -c "FINDSTR=$1 micro $2"

Running a shell is always more slow comparing with direct execution of a process.

Did you try -readonly (with one dash)?

It works fine from the command line. I suspect this happens because micro detects stdin and stdout redirection and thinks that it is editing a stream. When you edit a stream readonly and asking to save modified buffer doesn't make sense.

Hi,
I was suggesting using the micro command line with an -e option (as outlined in PIL[1]). This would offer an interface to launch micro with any lua "one liner". Adding a -script option to micro would allow a full script of instructions. The init.lua would have to be executed beforehand to import Go goodness and define functions. e.g
micro -e "Current.Buf:HandleCommand([[find 'your search term']] )" TargetFile
micro -e "MyCustomFindFunction('your search term')" TargetFile
micro -script MyScript.lua TargetFile
I guess this may be difficult without exposing the lua loadstring[2] command - see https://github.com/zyedidia/micro/issues/1992
Kind Regards Gavin Holt
PS I love the ability to use different quoting characters as required in Lua[3] - rather than having to escaping stuff!
[1] http://www.lua.org/pil/1.4.html
[2] http://www.lua.org/pil/8.html
[2] https://www.tutorialspoint.com/lua/lua_strings.htm

@Gavin-Holt
Wow! I have never known such command line parameters existed!
I couldn't even find them in the source files. How does it work?

@serge-v

Did you try -readonly (with one dash)?

The problem is here: https://github.com/zyedidia/micro/blob/master/cmd/micro/micro.go#L160
I think buffer type should be changed only if no files were provided in the command line. If I comment these 3 lines out then everything works as expected.

yes. In subshell call "$(...)" the terminal does not exists:
if [[ $1 == "-check" ]] ; then
	if [ -t 1 ] ; then echo terminal; else echo "not a terminal"; fi
	exit
fi

echo === check direct

./checkterm -check

echo === check subshell

res=$( ./checkterm -check )
echo ${res}

@serge-v
That's correct. But irrespective of this, I don't think that buffer type should be changed just because stdout is not a terminal.

Hi,
This is "feature request" issue. My suggested -e and -script options for micro do not exist. I was generalising the idea of passing instructions at launch. Sorry
G


F-Droid release and reproducible builds

Current Implementation

App is likely only planned to be released in Google Play Store.
Suggested Enhancement

It would be great to see this Android app on F-Droid!
F-Droid is an Android app store specifically for free/libre open-source apps. It would be great if your app could be released there, as it is the number one for getting FLOSS Android apps for many people.
F-Droid also builds all apps from source (optionally even reproducible), so downloads from there can be trusted.
The app developer FAQ or the quick start guide may help you to get started.
BTW a release on F-Droid could also bring some (more) popularity (in case that is intended), as it will show up in the app (new apps are featured there).
Expected Benefits

The benefits are trust (that the code shown here is the code you deliver), security (as untrusted modification of the code is nearly impossible then and you make an independent analysis possible) an increased user base/alternative installation options combined with an increased robustness by not having a single point of failure (Google Play Store) for app delivery.
And also, as said, popularity/marketing if it is visible in the main F-Droid store.
See also https://github.com/corona-warn-app/cwa-app-android/issues/1483 for the same issue for the German Corona-Warn-App that explains more advantages especially of reproducible builds.

The Swiss app will most likely be published on F-Droid, with reproducible builds (see https://github.com/admin-ch/CovidCertificate-App-Android/issues/40)
I started looking into getting this one there as well but I'm a bit too tired to do things properly I think, will try again tomorrow probably :)
Let me put the link of the MR where I'll be working on it here:
https://gitlab.com/fdroid/fdroiddata/-/merge_requests/9163

It seems this app has a proprietary dependency on com.google.android.gms/play-services-oss-licenses-license
This should be removed before inclusion into F-Droid I would say.

See https://github.com/mozilla-mobile/fenix/pull/13767/files for some inspiration :wink:

@joerg-oppermann-tsi can you clarify if reproducible builds are in scope for this app, as it's only a template?

In F-Droid a fork of https://f-droid.org/de/packages/de.rki.covpass.app/ is available: https://gitlab.com/covpass-android-foss/covpass-android/
As there are currently no plans by the European Commission known to me to release on alternative app stores I will close this issue - to be reopened if there is a change.


High Availability

I'm currently exploring options on which ingress to use for a new AKS cluster
I've deployed a cluster with the AGIC add-on, and noticed that there was only 1 pod, so it seems to me that there's a single point of failure here (unless I'm missing something). When deploying an nginx ingress, I can configure as many replicas as I need
My question is, what happens if the AGIC pod has an issue? Bad things I guess as there's no replicas
Can anyone shed any light on this?

@thepaulmacca you raise a good point; allowing AGIC to have multiple replicas is something that is on our backlog of improvements to AGIC. We're currently prioritizing hi-pri asks on AGIC like adding URL/header rewrite support but we're currently planning to tackle this in CY22.

That's good to know, thanks. I really like using the add-ons, but the lack of replicas is putting me off the ACIC right now to be honest, and will most likely go with an nginx ingress for the moment

When is this feature planned to be implemented?

Is there an update on this?


serve model other than downloaded via llama download ...

Hi,
I have successfully done the cycle of
llama download --source huggingface --model-id  Llama3.1-8B-Instruct --hf-token <HF_TOKEN>
llama stack build
llama stack configure testerx
llama stack run testerx

now my next assignment is to
- get model via tune download meta-llama/Meta-Llama-3.1-8B-Instruct
- LORA finetune it  using
  tune run lora_finetune_single_device --config llama3_1/8B_lora_single_device`
- and then serve it using
  llama stack build
  llama stack configure testerx
  llama stack run testerx

my question is how can i change the checkpoint path to my custom downloaded model

I think thats the holy grail of opensource LLMs
One should be able to download -> finetune -> serve via REST Endpoint (api)
and it should be done in single tool chain.

If you use our meta-reference inference provider you can now do that using this: https://github.com/meta-llama/llama-stack/blob/main/llama_stack/providers/impls/meta_reference/inference/config.py#L34 although we don't quite support or advertise this super well (because you could technically put any random checkpoint in there with params.json not being correct or weights being wrong, etc.)


Short error message workaround should be listed in readme

Documentation should note that starting a node is needed when using buidler when the following error is printed on run:
  1) Uncaught error outside test suite
with no additional information.

@cgewecke if there's no way to make the plugin work with the in-process buidler EVM, maybe something like this could be added to the plugin?
      if (options.enabled && network.name !== 'buidlerevm') {

It's a hack, but it's probably better than failing without further information.

This should finally be resolved with 0.1.4 (just released). Very sorry about the delays here.
There's no longer an independent node requirement. The reporter works out of the box with regular BuidlerEVM, requires no special config, is much faster, and all the other BuidlerEVM features should work as well.
Closing here, please just lmk if you run into any problems / bugs when upgrading...