It doesn't start 😢

Hey there, I've been watching those git-graph posts for a while now and happy for you to publish this thing to the neovim community ❤...
But this is not working (sadly):
https://github.com/user-attachments/assets/fb1e91f3-304f-42a9-9dd2-3b056178f624
I also waited for around a minute, which didn't help

Dont worry.There are known and expected bugs / performance limitations that I will be addressing asap :)
Thanks for bringing this up to my attention!

Most likely found the cause.
Will aim for a patch within 24 hours. 🤞

This should now be fixed, please verify if you can :)


[vm/ffi] Optimize Pointer operations

Hot loops with structs usually contain loading a struct in the hot loop. For example
void doStoreInt32(Pointer<VeryLargeStruct> pointer, int length) {
  for (int i = 0; i < length; i++) {
    pointer.elementAt(i).load<VeryLargeStruct>().c = 1;
    // pointer[i].c = 1; // With extension method syntax.

1. Loading a struct from a Pointer
The way that we implement this runtime entry is:
// Find <class name>.#fromPointer constructor and call it.

https://github.com/dart-lang/sdk/tree/master/runtime/lib/ffi.cc
The constructor is generated in kernel
// Add a constructor which 'load' can use.
// C.#fromPointer(Pointer<Void> address) : super.fromPointer(address);

https://github.com/dart-lang/sdk/tree/master/pkg/vm/lib/transformations/ffi_definitions.dart
It would be simple to generate IL per call site to call the right constructor, but our recognized methods generate method bodies, not specialized per call site. We could use a hashtable from class to method to make a generic method body, but we would need to populate that hashtable at some point.
2. Looking up the size of a struct
Same kind of problem: we find the right #sizeOf field, and read its value in the runtime entry.
This would also be trivial per (non-generic) call site.
@sjindel-google @mkustermann Any suggestion on how to implement this optimization?

We should just require that the struct type being loaded is statically known, as in C.

As discussed with @mkustermann offline, we can optimize the load and sizeOf of statically known type arguments by rewiring in the frontend.
We need to keep supporting generic invocations, at least for sizeOf, if we want support package:ffi exposing allocate with the current signature (https://github.com/dart-lang/sdk/issues/38520):
malloc = // lookedUpSymbol

Pointer<T> allocate<T>(int count){
  final int size = sizeOf<T>() * count;
  final int addr = malloc(size);
  return Pointer.fromAddress(addr);
}

Pointer.ref is dominating benchmarks with structs: https://github.com/mannprerak2/exp-ffi-dart/tree/master/json_parser_ffi#observatory-cpu-profile
Thanks for the benchmark @mannprerak2!


use firebase as database

I'm submitting a ...
[ ] bug report
[ x] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
A feature, anyone know how to connect this to firebase with working CRUD?
can you please provide tutorial, about it?

What is the current behavior?

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).

What is the expected behavior?

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 2.0.0-rc.X
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

Firebase provides a complete API for their DB, all you need is some JavaScript on the client side to do CRUD operations on your DB. Look for Firebase for web here.

I've used angularfire2 with ng2-admin - works really well.
Pretty straight forward implementation as well - checkout Angular Fire here

Thanks @neelavar!

I've problem on implementation of versions ... what is the true configuration? pleaseeee

@garycsebastian  this is the latest version of angular fire
Check this out
https://github.com/angular/angularfire2/blob/master/docs/version-5-upgrade.md


LRAClient::set * CoordinatorURI is implementation specific

This is really implementation detail and not required by MP/LRA itself
https://github.com/eclipse/microprofile-lra/blob/99d6e7494ae8db3169452f37799186d66d9f5e5c/api/src/main/java/org/eclipse/microprofile/lra/client/LRAClient.java#L94
/cc @mmusgrov @xstefank

I just mean that it seems like configuration option, sorry I was a bit prescriptive I will edit

I will create a PR for this issue removing both setCoordinatorURI and setRecoveryCoordinatorURI.
@tomjenkinson is it ok like this?

@xstefank perfectly - thanks!

the setCoordinatorURI is used by TCK to set the right endpoint it expects the coordinator will be available at. We need to discuss this as this may have impact of how TCK is configured.

Does it actually use that though? I thought we had discussed and it was not required at the TCK level? Which of the methods subsequent to that call rely on that method?
IOW can't we remove the reference to where it is used?

@tomjenkinson https://github.com/eclipse/microprofile-lra/blob/master/tck/src/main/java/org/eclipse/microprofile/lra/tck/TckTests.java#L126
The question is if we can expect in the TCK that injected LRA client is already configured to be called. IMO it makes sense.


FAMD bugs in transform()

Hi,
I am not able to transform FAMD on "new" data.
Using version 0.13.0.
Please see the following code:
import pandas as pd
from prince import MCA, PCA, FAMD
df = pd.DataFrame({'var1':['c', 'a', 'b','c'], 'var2':['x','y','y','z'],'var2': [0.,10.,30.4,0.]})

famd = FAMD(n_components=2)
famd.fit(df[:3])
print(famd.transform(df[0:3]))
print(famd.transform(df[0:2]))
print(famd.transform(df[3:]))

I get the output:
component         0         1
0         -1.303760 -0.658334
1         -0.335621  0.981047
2          1.639381 -0.322713
component   0   1
0         NaN NaN
1         NaN NaN
component   0   1
3         NaN NaN

So FAMD is able to transform the training data itself, but NOT a selection of rows from it, or a new row.
Thanks!

Yep that isn't supported yet. I'm aware of the issue.


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.webui

[ ] [E145] No news found for actual version 0.0.13
[ ] [E154] common.dependencies must contain{"js-controller": ">=2.0.0"} or {"js-controller": ">=3.0.0"}
[ ] [E500] node_modules found in repo. Please delete it
[ ] [E605] No actual year found in copyright. Please add "Copyright (c) 2022<EMAIL_ADDRESS>at the end of README.md
[ ] [E701] No actual year found in LICENSE. Please add "Copyright (c) 2022<EMAIL_ADDRESS>at the start of LICENSE
[ ] [E812] file test/ found in repository, but not found in .npmignore
[ ] [E813] file src/ found in repository, but not found in .npmignore
[ ] Please check your pull request to latest: https://github.com/ioBroker/ioBroker.repositories/pull/1552

I have also found warnings that may be fixed if possible.

[ ] [W400] Cannot find "webui" in latest repository

Thanks,
your automatic adapter checker.

Thanks, that all bugs have been fixed.


NotFoundError: Failed to execute 'insertBefore' on 'Node': The node before which the new node is to be inserted is not a child of this node.

Please include some context and the contents of the console in your browser's developer tools.
JavaScript Error
Error: Failed to execute 'insertBefore' on 'Node': The node before which the new node is to be inserted is not a child of this node.
    at http://localhost/chatSystem/public/register:198:251111
    at $p (http://localhost/chatSystem/public/register:198:251129)
    at $p (http://localhost/chatSystem/public/register:198:251221)
    at Gp (http://localhost/chatSystem/public/register:198:250542)
    at gh (http://localhost/chatSystem/public/register:198:267493)
    at HTMLUnknownElement.y (http://localhost/chatSystem/public/register:198:103635)
    at Object.Hn (http://localhost/chatSystem/public/register:198:103960)
    at Kn (http://localhost/chatSystem/public/register:198:104921)
    at mh (http://localhost/chatSystem/public/register:198:265330)
    at t.unstable_runWithPriority (http://localhost/chatSystem/public/register:198:38308)

Reproduction Steps
Please tell us what you were doing when this error occurred, so we can more easily debug it and find a solution.

…

User Agent
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

Bonjour ,j'étais entrain de travailler sur un projet  laravel de discussions mais arrivée à ce niveau ça met  d'erreur que l'ignion


super-slicer: <IP_ADDRESS> -> <IP_ADDRESS>

We are finally up a version!

Motivation for this change
SuperSlicer version bump and minor derivation fixes.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

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

[X] Fits CONTRIBUTING.md.

Submodules should be now enabled — preset files were extracted into them

Result of nixpkgs-review pr 126671 run on x86_64-linux 1

1 package built:

super-slicer

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/545

:+1: gave it a spin, looks good, no exceptions or crashes so far. The test print result looks good, too.
In case "Background processing" is enabled, it will always switch to gcode preview, but there is an option in the GUI preferences tab to disable that behavior https://github.com/supermerill/SuperSlicer/issues/1147 "Switch from 3D view to Preview when sliced"

Ooof, probably would go ahead and bump it to 5 now it got released)

Nope, it segfaults right after the boot :D

Okay, that had some polluted context, it works too.

Result of nixpkgs-review pr 126671 run on x86_64-linux 1

1 package built:

super-slicer

Everything seems to work perfectly, clickity-click through UI doesn't cause any errors, all smooth 🎉  Have no printer to test though 🤷🏻


Fixed : issue #165

Related Issue

Closes: #165
Description of Changes

Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[ ] I have included comments in areas that may be difficult to understand.
[ ] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Please provide any necessary screenshots to illustrate the changes made.

@anmode in npx prettier --write after --write I have to give the file name right ???

npx prettier --write .    remember i used . so it will cover whole code

npx prettier --write

@anmode After running the command . Am I supposed to get this result ?

npx prettier --write .  remember i used . so it will cover whole code

@anmode After running this command should I add all the changes or changes I made ??

npx prettier --write .  remember i used . so it will cover whole code

@anmode After running this command should I add all the changes or changes I made only  ??

npx prettier --write .  remember i used . so it will cover whole code

@anmode After running this command should I add all the changes or changes I made ??


Implement a way to capture ays actions

The operator needs to be able to easily fetch information about what the system is doing.
When sealhealing is triggered we need to make sure the operator is able to get information about such actions.

AYS exposes a way to list the jobs of the last 2days. We need a way to expose this as an API

Need to take along AYS error conditions as well


[Accessibility]A11y_VSCode_VSCStartMenuFolder_Screenreader: NVDA reads repetitive content on the screen.

Environment Details:
VSCode Version : 1.24.1
OS Version : Win10
Additional Details:
NVDA 2018.2,MAS1.3.1
Repro Steps:
1.On Visual Studio Code Select Destination Location screen, click on "Next" button.
2.Check using insert+b key whether screenreader reads repetitive content or not.
Actual:
Screenreader reads "Select Start Menu Folder Where should Setup place the program's shortcuts?" content twice .
Expected:
Screenreader should not read the same content twice.
Recommendations:
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
User Impact:
The user will get confused when it reads the same content twice.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={8492c4eb-c179-40ae-8777-cd044ed725a2}
Does this issue occur when all extensions are disabled?: Yes
Attachment for Reference:

@AccessibilityTestingTeam-TCS This needs to go into https://github.com/jrsoftware/issrc


[MINOR][CORE][TEST] Remove unnecessary sort in UnsafeInMemorySorterSuite

What changes were proposed in this pull request?
We don't require specific ordering of the input data, the sort action is not necessary and misleading.
How was this patch tested?
Existing test suite.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3926/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/36/
Test PASSed.

Merged build finished. Test PASSed.

Test build #91691 has started for PR 21536 at commit 2ea2181.

Test build #91691 has finished for PR 21536 at commit 2ea2181.

This patch fails Java style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Hm .. ? retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3928/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/38/
Test PASSed.

Test build #91695 has started for PR 21536 at commit 2ea2181.

Test build #91695 has finished for PR 21536 at commit 2ea2181.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3946/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/57/
Test PASSed.

Test build #91721 has started for PR 21536 at commit 2ea2181.

Test build #91721 has finished for PR 21536 at commit 2ea2181.

This patch fails Java style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3966/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/76/
Test PASSed.

Test build #91746 has started for PR 21536 at commit 2ea2181.

Test build #91746 has finished for PR 21536 at commit 2ea2181.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #91753 has started for PR 21536 at commit 2ea2181.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/82/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/3972/
Test PASSed.

Test build #91753 has finished for PR 21536 at commit 2ea2181.

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

Merged build finished. Test PASSed.

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

@HyukjinKwon can we merge this?

Sure.

Thanks! @HyukjinKwon


Is it possible to get the score returned

I want to store this in the database to see the score someone received while registering. If I try and "re-assess" the token after validation it gives me an error "DUPE".  Is there a way around this or to append it to the request after validation?

Hi
Yeah I think tokens are meant to be single use.
For your use case (I don't know if it will work) but you might want to try binding your own \Oneduo\RecaptchaEnterprise\Services\RecaptchaService
By binding a new implementation that extends the base class, and overrides the assess() method
https://github.com/oneduo/laravel-recaptcha-enterprise/blob/7708513ab9ab2c82c0f2931c7e946cdf14cdfb2e/src/Services/RecaptchaService.php#L85-L86
and perhaps dispatch an event (to be able to listen to it sync/async)
event(new RecaptchaAssessmentScoreEvent($this->score));

and then from within a listener save it to a database


Write the config out to file.

The current implementation of the service writes out the job payload to workspace file instead of the raw config.  This is due to the fact that the AsyncPlatform automatically records the raw job config passed through the queue.  This file is not immediately usable by the plugins directly as it is an envelope type meant simply to pass data through the queue.  The raw config itself should also be written to file.

The written config should be the raw request (see ReservedFiles.InputRequest).
Since generics will be erased when (de)serializing, this will be a minor headache.


Fix typo in "timed ot"

Signed-off-by: Dan Garfield<EMAIL_ADDRESS>

thank you for he fix, Dan.
can you please bump the version as well? just search for the version string (from the Makefile first line, without the v), everywhere in the project, and bump it by one. it should replace 4 occurrences in 3 files - docs/releaes/release_notes.md, manifests/runtime.yaml and Makefile itself.

Done. LGTM.


CLI option to specify Storybook configuration directory

This PR is a reaction to #29
I have added new CLI parameter
-c ( --config-dir )
which allows you to specify the path to Storybook directory with config files,
So we won't be fixed to .storybook directory
Docs updated as well.
Enjoy.

Okay. This is awesome.
Released as v1.4.0.


SQL rollbacks

WIP, currently just added a test case to show that failing queries don't cause the implicit transaction to rollback, which we rely on for D1 batch-processing. Also sketched out what using JS transactions to capture the same thing might look like, but that throws with Error: transaction() not yet implemented for SQLite-backed storage for now.
I'm not sure what the best API is here. Normal DO transactions implicitly call blockConcurrencyWhile and, as a result, reset the DO if you don't catch the errors, which feels like massive overkill as SQL statements can fail all the time.
Then again, most of the time you can use a DO without thinking about implicit transactions, but implicit rollbacks might be too surprising:
sql.exec("CREATE TABLE uniques (id INTEGER PRIMARY KEY, value TEXT UNIQUE);")
sql.exec("INSERT INTO uniques (value) VALUES ('one'),('two'),('three');")
sql.exec("INSERT INTO uniques (value) VALUES ('three'),('four'),('five');")

With no await in between, this will throw on line 3, but should it roll back the changes from the first two lines?
sql.exec("CREATE TABLE uniques (id INTEGER PRIMARY KEY, value TEXT UNIQUE);")
await scheduler.wait(1)
sql.exec("INSERT INTO uniques (value) VALUES ('one'),('two'),('three');")
await scheduler.wait(1)
sql.exec("INSERT INTO uniques (value) VALUES ('three'),('four'),('five');")

By adding await scheduler.wait(1), a user can effectively commit after each statement, so they can ensure the DB is left with a uniques table and three rows. So maybe implicit rollbacks aren't too bad?
But then, it becomes super awkward to catch errors. You might think you're being defensive by wrapping statements in a try/catch:
sql.exec("CREATE TABLE uniques (id INTEGER PRIMARY KEY, value TEXT UNIQUE);")
try {
  sql.exec("INSERT INTO uniques (value) VALUES ('one'),('two'),('three');")
} catch (e) {}
try {
  sql.exec("INSERT INTO uniques (value) VALUES ('three'),('four'),('five');")
} catch (e) {}

But depending on how we implement implicit rollbacks, the failure of one statement could have already rolled back the txn.
I think potentially we should introduce a lighter-weight SQL-specific transaction syntax:
sql.exec("CREATE TABLE uniques (id INTEGER PRIMARY KEY, value TEXT UNIQUE);")
// Note: not async, and doesn't get a txn object passed in. Exceptions are rollbacks.
const result = sql.transaction(() => {
  sql.exec("INSERT INTO uniques (value) VALUES ('one'),('two'),('three');")
  sql.exec("INSERT INTO uniques (value) VALUES ('three'),('four'),('five');")
});
// result: {success: boolean, changeCount, error: string}

This could be implemented using SAVEPOINTs, so these could be nested within the outer, implicit transaction.

Just following on from this, it occurs to me that the reason we disallowed transactions was, first, in case a user left one open, and secondly, because the implicit DO transaction was already present. But in both cases, it's fine for a user to use a SAVEPOINT:

The COMMIT command commits all outstanding transactions and leaves the transaction stack empty.
The ROLLBACK command without a TO clause rolls backs all transactions and leaves the transaction stack empty.
From https://www.sqlite.org/lang_savepoint.html

This should mean that the DO's outer (implicit) transaction basically overrides whatever the user is doing with their own savepoints. So maybe we could allow that?
The only downside I can see is if you do this:
sql.exec("SAVEPOINT a");
try {
  sql.exec("create table ...")
  await scheduler.wait(1)
  sql.exec("insert ...")
  sql.exec("RELEASE x"); // throws
} catch (e) {
  sql.exec("ROLLBACK TO x"); // also throws
}

This is because the await scheduler.wait(1) has caused the outer transaction to commit, so the savepoint a has been consumed. So maybe we do want a sql.savepoint API, that doesn't accept an async callback, and continue to disallow the use of the SAVEPOINT statement itself.

Can this be closed now that we've implemented the explicit transaction API?


React 18 support?

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>
Examples 🌈
No response
Motivation 🔦
No response

@mui/styles is a legacy styling solution and will not support React 18 (see https://mui.com/system/styles/basics/)


Update Packal

The current version available at packal.org appears to be 1.0.
http://www.packal.org/workflow/network-location
I realize this isn't a code change exactly :)

Packal's operator walked away a few years ago, and it's an unmaintained zombie now. I don't upload workflows to it any more because it doesn't work most of the time.

I came to that conclusion a little while after submitting this issue. It’s a shame. The Alfred community would really benefit from a package manager.
Sent from my iPhone

On Dec 7, 2019, at 10:11 AM, Dean Jackson<EMAIL_ADDRESS>wrote:

Packal's operator walked away a few years ago, and it's an unmaintained zombie now. I don't upload workflows to it any more because it doesn't work most of the time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

The Alfred community would really benefit from a package manager.

Well, a decent workflow index/search engine. Many, if not all, workflows can update themselves. The main problem is discovery.

Well thanks for getting back to me :)


[bug] Smart start fixes

[x] Rename smart start in Inclusion by QR Code
[ ] Replacing a node by scanning a QR code should never add the QR code to the provisioning list. You'll end up with two nodes, one of which is dead and the other has a new ID. Instead just do an S2 inclusion even if the QR code is SmartStart
[x] Can't replace a failed node that has a DSK. It will tell me that the node is already added, although I'm just trying to replace it
[x] Add dialog that shows when a node is successfully added

See Slack, issue 2 isn't solved :)


Update Developer Guide and User Guide

Changes

Added Target User Profile and Value Proposition to DeveloperGuide.md
Removed references to AB3 in UserGuide.md

Codecov Report
Base: 72.15% // Head: 72.15% // No change to project coverage :thumbsup:

Coverage data is based on head (756a264) compared to base (103c409).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #24   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Improve ECS performance

A known limitation with the ECS implementation is the reliance on an associative array of dynamically allocated components. A better strategy is a greedily resizing array per component to ensure have linear access to entity data.
We don't currently support the notion of archetypes but if this becomes part of the plan then we can in theory chunk entity data across archetypes. For now lets look at large per-component datasets with a nicer memory layout.

I've read hundreds of debates on SoA vs AoS and tbh its super convoluted.
If you use POD structs and align+pack them correctly, there is no real difference.
Thus an array of vec4f position components has no inherit performance issues.
If we wisely chose our components and ensure each is correctly packed to allow fitting
in cache lines, we don't need super ugly code with merge tables and whatnot. At the end
of the day we're focused on making 2D games right now, with thousands of entities, not
millions.
If we ever need to revisit this, we can. For now, switch from dynamically allocated structs
to a single dynamically allocated, greedily regrowing array of structs. This alone will
significantly improve performance.
To minimize the cost of large empty sets we can chunk the data into 16kb regions, making
it very mmap friendly.

i.e. 100,000 entities and 20 components (sparsely populated) have one component, it makes zero sense to have storage for 100,000 of that component.

OK been considering this one for a good long while now. The obvious solution is to use dynamically expanding (greedily realloc) arrays for component storage. Furthermore, we don't want to use sparse arrays as its a large waste of memory.
Instead we'll have each component array sorted by an entityID, instead of being keyed by one. Each component will have the same sort indexes, thus reducing memory waste and improve cache coherency
It may well make sense to teach the ECS the notion of signatures in terms of queries, i.e. multiple components. Additionally we'll want to group archetypes to ensure an entity position is valid across multiple components, reducing lookup times considerably, and allowing a natural chunking behaviour.

True story, just looked at our current ECS and it sucks. Working on huge improvements to it :P

Added our initial GreedyArray implementations in tree which will simplify this somewhat.

This has now been landed into master.

Signature matching for component-entity queries
Archetype to group similar entities (must all possess same component types only)
GreedyArray of 16KiB storage chunks merged into pages, pooled by a PoolAllocator
Chunks are self managed and insert their own padding at compile time
EntityIdentifier struct as private primary key
Reclamation systems (free lists)
Clear() features
Threadsafe within context of processor usage
RO/RW compiler templated queries
Staging archetype for deferred component manipulation (thread-safe)
Really, really fast.
Chunked and joined behaviour for queries


Add IDEs and containers ignore

Reasons for making this change:
Ignore idea and containerization tool generated files and folders.

These are already covered in more details by existing templates:

Global/JetBrains.gitignore
Global/VisualStudioCode.gitignore
Global/Vagrant.gitignore

It is recommended to tailor your repository .gitignore to suit the tools you are using, using these reference templates, as duplicating rules everywhere becomes a maintenance burden over time.


Add Manage Cloud Keys and Manage Node Templates buttons

Version: Master 2.2 (2/13/19)
What kind of request is this (question/bug/enhancement/feature request):
Enhancement Request
Steps to reproduce (least amount of steps as possible):

Please add a "Manage Cloud Keys" button to the Node Templates page
Please also add a "Manage Node Templates" button to the Cloud Keys page.

Result:
This is similar to the Multi-Cluster Apps page where we have a button as a shortcut to bring us to a related page:

Version: Master 2.2 (2/20/19)
Confirmed enhancement. There are now shortcuts to each page. These work without any issues.

Minor Cloud Keys header alignment issue discovered and tracked here: #18287


GPU gemms true fp16 (#17466)

Temporal solution for fp16 accumulation in Bert gemms

Resolve alpha/beta type issue

add documentation for env variable MXNET_FC_TRUE_FP16

Improve description of env variable

Add unitest checking environment variable

keep pseudo-fp16 if architecture does not support Float16Compute

Fix cpplint

Description
This PR allows true FP16 in CUBLAS gemms if environment variable MXNET_FC_TRUE_FP16 is set to true
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator):
tests/python/gpu/test_gluon_gpu:test_gemms_true_fp16
[x] Code is well-documented:
[x] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] General half_precision Gemm: allows true fp16 if MXNET_FC_TRUE_FP16 is set
[x] Transformer: allows true fp16 if MXNET_FC_TRUE_FP16 is set

Comments

Adding this to 1.7.0 roadmap https://github.com/apache/incubator-mxnet/issues/16864.

@mxnet-bot run ci [windows-gpu]

@mxnet-bot run ci [all]


Uart IRAM optimization crashing

Environment

Development Kit:      [ESP32-Wrover-Kit]
IDF version: v4.2

Problem Description
If the CONFIG_UART_ISR_IN_IRAM SdkConfig option is enabled, the interrupt driver may crash because some resources may not be allocated in internal ram depending on another configuration (CONFIG_SPIRAM_MALLOC_ALWAYSINTERNAL).
If CONFIG_SPIRAM_MALLOC_ALWAYSINTERNAL is very small, like 4 Bytes, some components in the uart_driver_install function (components/driver/uart.c) will not be allocated in the internal ram.
Following allocation have to be fixed to work properly in all cases
p_uart_obj[uart_num]->xQueueUart = xQueueCreate(queue_size, sizeof(uart_event_t));
p_uart_obj[uart_num]->rx_ring_buf = xRingbufferCreate(rx_buffer_size, RINGBUF_TYPE_BYTEBUF);
p_uart_obj[uart_num]->tx_ring_buf = xRingbufferCreate(tx_buffer_size, RINGBUF_TYPE_NOSPLIT);

Those 3 allocations have to be replaced by xQueueCreateStatic and xRingbufferCreateStatic when the CONFIG_UART_ISR_IN_IRAM option is enabled
for reference , i2c driver has the driver installation in a proper way
i2c_driver_install function (components/driver/i2c.c) use xQueueCreateStatic on internal memory case ESP_INTR_FLAG_IRAM was enabled!
as shown below from function i2c_driver_install ()
            if ( !(intr_alloc_flags & ESP_INTR_FLAG_IRAM) ) {
                p_i2c->cmd_evt_queue = xQueueCreate(I2C_EVT_QUEUE_LEN, sizeof(i2c_cmd_evt_t));
            } else {
                p_i2c->evt_queue_storage = (uint8_t *)heap_caps_calloc(I2C_EVT_QUEUE_LEN, sizeof(i2c_cmd_evt_t), MALLOC_CAP_INTERNAL | MALLOC_CAP_8BIT);
                if ( p_i2c->evt_queue_storage == NULL ) {
                    ESP_LOGE(I2C_TAG, I2C_DRIVER_MALLOC_ERR_STR);
                    goto err;
                }

uart_driver_delete has t be also modified to free the statically generated queue and ringbuffer resources that shall be used case  ESP_INTR_FLAG_IRAM was enabled!
Code to reproduce this issue

Enable CONFIG_UART_ISR_IN_IRAM in the SdkConfig
Set CONFIG_SPIRAM_MALLOC_ALWAYSINTERNAL to 4
Use any uart example, Crash will occur

Thanks for reporting.

Any new informations about this bug?

@Alvin1Zhang  Issue exists in IDF 4.2.2 as well ..
Lost packets on the uart occur when the CONFIG_UART_ISR_IN_IRAM is unticked as well!
any updates on that issue ?

@formus14 Thanks for reporting and sorry for the very slow turnaround. Thanks for being patient while waiting for the fix, we had a fix under internal reviewing and will be back ported to release/4.2. The issue will be closed automatically once the fix is available on GitHub. Thanks.


Global Key using on multiple widgets issue.. please fix it

Multiple widgets used the same GlobalKey.
The key [GlobalObjectKey int#00001] was used by multiple widgets. The parents of those widgets were:

Issue solved


function symbol returns empty

I'm trying to understand how this module works.
This is my ContractCallContext:
  {
        reference: 'testContract',
        contractAddress: '0xCc7bb2D219A0FC08033E130629C2B854b7bA9195',
        abi: [        {
            name: "symbol",
            type: "function",
            inputs: [{name: "", type: 'string'}],
            outputs: [
                {
                    name: "",
                    type: "string"
                }
            ],
            stateMutability: "view",

        },],
        calls: [{ reference: 'symbolCall', methodName: 'symbol', methodParameters: [42] }]
    }

returns this:
callsReturnContext: [
    {
      returnValues: [],
      decoded: false,
      reference: 'symbolCall',
      methodName: 'symbol',
      methodParameters: [Array],
      success: false
    }
  ]

What am I doing wrong?
Could you also show me an example of something more complex like erc20 token approval and swap?
thanks for this module.

this has success: false so its a invalid mapping you sure you want to pass 42 into method parameters for symbol? closing as this is not an issue with package


Release

I would like to push out a new 2.2 release for dask/dask and dask/distributed.  Is there anything that we should be aware of before releasing?  How are fsspec @martindurant and parquet @rjzamora ?  Anything we should be concerned about with the Pandas 0.25 release @TomAugspurger or Numpy 1.17 @jakirkham ?

There is a parquet bug that should be fixed by PR#5157

I am looking through the remote data services doc now. I don't imagine it needs too much. Documentation updates at fsspec and s3fs/gcsfs would be good too, but not urgent.

Pandas stuff should be good.

E            +  where [<SpawnProcess(SpawnPoolWorker-403, started daemon)>, <SpawnProcess(SpawnPoolWorker-401, started daemon)>, <SpawnProce...aemon)>, <SpawnProcess(SpawnPoolWorker-399, started daemon)>, <SpawnProcess(SpawnPoolWorker-404, started daemon)>, ...] = <function active_children at 0x7fc16b899400>()
E            +    where <function active_children at 0x7fc16b899400> = <multiprocessing.context.ForkServerContext object at 0x7fc16b8c5c50>.active_children
../../../miniconda/envs/test-environment/lib/python3.7/site-packages/distributed/utils_test.py:1473: AssertionError

Would it be possible to include #5150 ? I believe there's just the one unit test to wrap up.

https://github.com/dask/dask/pull/5170

There may be some distributed-related upstream dev failures that should be looked at by someone (can't volunteer today, sorry): https://travis-ci.org/dask/dask/jobs/564703528#L1444

@TomAugspurger how prevalent are these?  Have they occurred in more than one PR?  If not, what is that PR?
Dask.distributed got a lot more strict about resources persisting between tests.  It may be that we don't want to be this strict on the Dask CI side.  We can disable these if necessary.  It's a bit odd that they should be popping up though.  I haven't yet been able to reproduce locally, but I'm working on it.

Would it be possible to include #5150 ? I believe there's just the one unit test to wrap up.

@mmccarty just checking, this is a nice-to-have, and not something that should block the release, yes?  Mostly my question is "is there anything that would make a release of the current state a worse situation than our most recent release"

I only checked the latest cron job, and didn't look closely at the error
message. It may just be a test thing.
I believe if you include "test-upstream" in the commit message, you'll
trigger an upstream build on the PR.
On Mon, Jul 29, 2019 at 11:24 AM Matthew Rocklin<EMAIL_ADDRESS>wrote:

There may be some distributed-related upstream dev failures that should be
looked at by someone (can't volunteer today, sorry):
https://travis-ci.org/dask/dask/jobs/564703528#L1444
@TomAugspurger https://github.com/TomAugspurger how prevalent are
these? Have they occurred in more than one PR? If not, what is that PR?
Dask.distributed got a lot more strict about resources persisting between
tests. It may be that we don't want to be this strict on the Dask CI side.
We can disable these if necessary. It's a bit odd that they should be
popping up though. I haven't yet been able to reproduce locally, but I'm
working on it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/5168?email_source=notifications&email_token=AAKAOIRZKFHKCTG6WJ2VCGTQB4KUVA5CNFSM4IHTLD52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3BHWTY#issuecomment-516062031,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKAOIVL7VP7GP523LFTUVDQB4KUVANCNFSM4IHTLD5Q
.

@mmccarty just checking, this is a nice-to-have, and not something that should block the release, yes? Mostly my question is "is there anything that would make a release of the current state a worse situation than our most recent release"

@mrocklin Absolutely a nice-to-have!  Please don't hold up the release for it.  This is just blocking some work on my end and it would be great to get it wrapped up for the next release.
Unfortunately, my team is not yet in a position to jump in and help.  Soon!

You've done a lot of work recently on the setup/teardown of scheduler/workers. I'm assuming this was not backwards compatible? In this case both dask-yarn and dask-gateway will need to be updated or a new release will break them.

Most of the backwards incompatible changes were infrequently used.  I'm checking things on dask-mpi and dask-jobqueue now (see https://github.com/dask/dask-mpi/pull/38 and https://github.com/dask/dask-jobqueue/pull/303) and plan to do dask-kubernetes in a bit.  If you're not using Adaptive and not passing args into Server.start(...) then my guess is that you'll be ok.  We should check though.

Anything we should be concerned about with the Pandas 0.25 release @TomAugspurger or Numpy 1.17 @jakirkham ?

Let's see. ( https://github.com/dask/dask/pull/5179 ) 🙂

I plan to release dask-jobqueue, dask-mpi, and dask-kubernetes as part of the dask 2.2 release process.
@jhamman @guillaumeeb @lesteve @andersy005 @kmpaul @jacobtomlinson please let me know if I should hold off on any of those.

I've also just raised issues on all of those projects about which version number to use.  One of the options is to use 2.2, the next dask version.  The idea here is that maybe we should all use roughly the same versioning.  There are both costs and benefits to this:
Benefits

We quickly bump everyone past 1.0, which is where they should probably be today
It's easy for users to see if components are being upgraded consisteently

Costs

Projects lose the ability to signal large internal changes using version numbers in the future
If a project is inactive then its version number may slip and appear to be old, even if it's fine and really just doesn't need any work

Maybe we do the version update as projects align with SpecCluster? (if that's the approach that we want to take)

If a project is inactive then its version number may slip and appear to be old, even if it's fine and really just doesn't need any work

I feel that dask-jobqueue pace of development is a lot slower than dask development at the moment (probably a good thing, given that dask-jobqueue works okish enough as far I am concerned).
I does not make complete sense for me to keep versions in sync if not much has happened in dask-jobqueue while a lot has happened in dask/distributed. Do you expect breaking changes as much as we have seen recently? I think part of it may be dask-jobqueue's fault by the way but I need to clarify my thoughts and open an issue in dask-jobqueue about this.

I think that dask-jobqueue is doing great. The fact that it hasn't needed changes is a sign that the design is good.
I do anticipate doing a rewrite on top of SpecCluster sometime soon (if you all are ok with it).  But other than that I don't anticipate that the project will need many large changes.

I've pushed dask, distributed, dask-jobqueue, and dask-kubernetes to PyPI.  I'm waiting for dask and distributed to arrive on conda-forge for dask-mpi (which will have a strict dependence) and for CI in a PR for dask-labextension.  @jcrist can I ask you to handle dask-yarn?
I don't think that any other dask subproject is likely to be affected by things.  If I'm wrong on this please speak up.

There's a new dask-yarn release out on PyPI and conda-forge. Is it safe to close this issue?

I think so. Closing.

Thanks all for your work on the release!


Video pixelated

Hello.
Camera: A4 TECH USB2.0 PC Camera PK-130MJ
For MediaStreamConstraints:
{
  "video": true,
  "audio": true
}

No Problem
For MediaStreamConstraints:
{
  "video": {
    "width":      { "exact": 1280 },
    "height":     { "exact": 800 }
  },
  "audio": true
}

After 1 minute

This is very much a result of packet loss.  Is everything running on the same system?

This is very much a result of packet loss. Is everything running on the same system?
Hello.

This problem takes place within the localhost.
My experiments show that the problem is in cases where the client is chrome vp8 on linux. firefox with codec vp8 no problem.
FIXME videodecoder gstvideodecoder.c:933:gst_video_decoder_drain_out: Sub-class should implement drain()
On linux (Fedora 28, AltLinux 7.0) chrome video is not restored at all, nothing is visible, all pixelated.
On android chrome video pixelated, but restored after change camera location.
On windows chrome no pixelated about 10 minutes, then the pixelated occurs and restored after black video(close the camera lens).
In the case of chrome with codec vp9 there is no pixelated, but the connection is broken very often in the log I see "vp9dec0 Failed to decode frame".
There is also video pixelated in the case of IOS Safari h264.
Fedora 28, gstreamer 1.14.1 - 1.14.3

And does not work chrome h264
For vaapi:
WARN  codecparsers_h264 gsth264parser.c:2086:gst_h264_parser_parse_slice_hdr: couldn't find associated picture parameter set with id: 1
ERROR vaapidecode gstvaapidecode.c:1178:gst_vaapidecode_parse_frame: parse error -1
For libav:
WARN  libav gstavviddec.c:1751:gst_ffmpegviddec_frame:<avdec_h264-0> avdec_h264: decoding error (len: -1094995529, have_data: 0)
ERROR libav :0:: non-existing PPS 7 referenced
ERROR libav :0:: decode_slice_header error
ERROR libav :0:: no frame!

Hello.
One more thing.
I also conducted experiments with the transfer of video files via webrtc.
I took 720p video clip (~ 2000 kbit/s), split the video and audio into different files.
Replace videotestsrc on filesrc ! decodebin.
Established connection through the Internet(~15000 kbit/s) gstreamer <-> chrome.
gstreamer received pixelated video from camera.
chrome received video from file normally reproduced in good quality. Periodically, the video freezes(slows down), but continues to play.
Further I did the same for gstreamer <-> gstreamer. Both videos are pixelated.
For localhost gstreamer <-> gstreamer video periodic pixelated and restored.
I also print stats of webrtcbin get-stats. When there are packet-lost then video not pixelated.
 According to my observations pixelation starts because of the slowdown on the CPU.
I was able to reproduce the problem on CPU Intel Core i7-7500U Kaby Lake.
Pixelated only when videodecodebin used vp8dec.
On CPU Intel Kaby Lake i7-7500U videodecodebin using vaapi decoder, the video just freezes in both directions.
 Or something is missing this demo?

Hello.
I checked what was discussed in issue #61 .
Sending this event causes chrome to be sent to KeyFrame and the video is restored. This can be seen on the graphichs in chrome://webtrc-internals.
I send this event on a timer once a second. This is not enough. Video is very pixelated, key frames need to be sent by event when the decoder cannot decode frame.
I have 3 questions left:

How to determine what you need to pass the event GstForceKeyUnit?
Why within 20 seconds there are no packet losses, and then 50 packets are lost per second on localhost?
How to make the video not pixelating, and stop until KeyFrame comes? This is what browsers do.

After about 5800 events GstForceKeyUnit chrome no longer receive PLI and video full pixelated. At the same time, the gstreamer continues to send event GstForceKeyUnit, but chrome does not receive anything.

Sounds like your network setup is dropping data.


Add mdc key for storing the job id to distinguish log entries generat…

…ed in gobblin cluster.

@abti please review.

Looks like this overlaps with #967 and currently doesn't handle the thread pool in the task runner.


[question] context.portfolio.positions is empty.

Dear Catalyst Maintainers,
I think this is more a question instead of an issue. If it's really a problem I think it should have been opened long ago.
I created a very simple test strategy to try out the framework, as below:
`from catalyst import run_algorithm
from catalyst.api import order, record, symbol
def initialize(context):
context.asset = symbol('bch_usd')
def handle_data(context, data):
... ...
record(btc=data.current(context.asset, 'price'))
if name == 'main':
run_algorithm(
capital_base=101,
initialize=initialize,
handle_data=handle_data,
exchange_name='bitfinex',
live=True,
simulate_orders=False,
algo_namespace='my_algo_name',
quote_currency='usd'
)`
Given I already have some position on BCH/USDT, I was expecting in handle_data the context.portfolio.positions should contains that position, however I always end up with an empty dict for that variable.
The position was initiated by myself manually, I'm suspecting if that is the reason it's not tracked by catalyst.
Could you please confirm ? or kindly suggest what I might have done wrong.
I can get the position directly with bitfinex API v2 as:
In [38]: pos = ex2.private_post_auth_r_positions()
In [39]: pos
Out[39]:
[['tBCHUSD',
'ACTIVE',
-30.50597163,
543.92676566,
-0.01776972,
0,
225.31178805,
1.35787134,
624.46843668,
-3.2016]]
Thanks.

according to https://enigma.co/catalyst/status/#Bitfinex, only bch_usd (no t) is supported on Bitfinex

@ripley , this question has been raised several times already (#480, #185).
By design, the algorithm should not have the ability to impact the orders/positions that were not performed by it.
When starting a new strategy, its parameters - like quote currency and capital base - define the strategy boundaries, meaning that any existing position you might have, should not be disrupted by your algorithm decisions.
Closing this, feel free to reopen if you feel otherwise.


page

[Enter feedback here]

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

ID: 91153855-ada2-3420-eb1e-f0f4fa0d46c7
Version Independent ID: 55d0d4e4-5232-882f-c600-fb9c7d409608
Content: Page Data Type - Business Central
Content Source: dev-itpro/developer/methods-auto/page/page-data-type.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Hello,
There is no feedback entered for this issue. We'll close this for now, but if you do have some feedback, you're welcome to provide it and we'll reopen the issue.
Thanks!


Fix all warnings output by Clang++

Im assuming a bunch of the code being commented out is cause its not in use anymore? should prob look into deleting it instead if its no longer needed

Im assuming a bunch of the code being commented out is cause its not in use anymore? should prob look into deleting it instead if its no longer needed

This is what my process was when commenting vs deleting unused blocks:
If it looked completely useless I just deleted it.  If it looked like it was either used before or looked like it was important I left it in a comment for someone to look at just in case.


Vendor ksonnet-lib/ksonnet-gen | Log warning on k8s v1.8.x

Fixes #30
@abiogenesis-now, @bryanl - Please review! The change is relatively straightforward. I'm vendoring the changes to ksonnet-lib and  displaying a warning to users who attempt to use v1.8.x
P.S. You might notice some other missing dependencies -- this is pulled in from govendor list +missing
You can test this using ./ks init foo3 --api-spec=version:v1.8.0. Check out what gets created in the environment's metadata folder~

Hold on, going to test, but this change looks relatively straightforward


Update README with working example

LGTM

Might be worth mentioning other runtime implementations while we're at it:
https://github.com/NVIDIA/enroot/blob/master/conf/hooks/99-nvidia.sh
https://github.com/lxc/lxc/blob/master/hooks/nvidia

For some reason never saw this PR, sorry about that...


Some tests seems to run multiple times

From https://github.com/lab-cosmo/scikit-cosmo/runs/6552874752?check_suite_focus=true
GLOB sdist-make: /home/runner/work/scikit-cosmo/scikit-cosmo/setup.py
tests create: /home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests
tests installdeps: coverage[toml], parameterized
tests inst: /home/runner/work/scikit-cosmo/scikit-cosmo/.tox/.tmp/package/1/skcosmo-0.1.1.zip
tests installed: coverage==6.4,joblib==1.1.0,numpy==1.22.4,parameterized==0.8.1,scikit-learn==1.1.1,scipy==1.8.1,skcosmo @ file:///home/runner/work/scikit-cosmo/scikit-cosmo/.tox/.tmp/package/1/skcosmo-0.1.1.zip,threadpoolctl==3.1.0,tomli==2.0.1
tests run-test-pre: PYTHONHASHSEED='920532320'
tests run-test: commands[0] | coverage run -m unittest discover -p '*.py'
....................................................../home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests/lib/python3.9/site-packages/sklearn/linear_model/_ridge.py:251: UserWarning: Singular matrix in solving dual problem. Using least-squares solution instead.
  warnings.warn(
.............................................................................../home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests/lib/python3.9/site-packages/skcosmo/utils/_orthogonalizers.py:57: UserWarning: Column vector contains only zeros.
  warnings.warn("Column vector contains only zeros.")
.........../home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests/lib/python3.9/site-packages/skcosmo/_selection.py:214: UserWarning: Score threshold of 0.05 reached.Terminating search at 7 / 10.
  warnings.warn(
/home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests/lib/python3.9/site-packages/skcosmo/_selection.py:214: UserWarning: Score threshold of 0.4 reached.Terminating search at 6 / 10.
  warnings.warn(
............................/home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests/lib/python3.9/site-packages/skcosmo/_selection.py:214: UserWarning: Score threshold of 0.05 reached.Terminating search at 7 / 10.
  warnings.warn(
/home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests/lib/python3.9/site-packages/skcosmo/_selection.py:214: UserWarning: Score threshold of 0.4 reached.Terminating search at 6 / 10.
  warnings.warn(
........./home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests/lib/python3.9/site-packages/skcosmo/_selection.py:214: UserWarning: Score threshold of 0.05 reached.Terminating search at 7 / 10.
  warnings.warn(
/home/runner/work/scikit-cosmo/scikit-cosmo/.tox/tests/lib/python3.9/site-packages/skcosmo/_selection.py:214: UserWarning: Score threshold of 0.4 reached.Terminating search at 6 / 10.
  warnings.warn(
.
----------------------------------------------------------------------
Ran 182 tests in 69.540s

OK

The UserWarning: Score threshold of 0.4 reached.Terminating search at 6 / 10. should only appear once, but is there three times.
Changing https://github.com/lab-cosmo/scikit-cosmo/blob/f433e28a8e5ff13f2dd1aa9a9f2dd2fc3f606218/pyproject.toml#L19 to coverage run -m unittest discover -p "test_sample_simple_fps.py" makes the test run only once, so there might be some strange interaction between unittest, discover and coverage.

From the latest CI run with pytest:

tests/test_sample_simple_fps.py::TestFPS::test_threshold
tests/test_voronoi_fps.py::TestFPS::test_threshold
tests/test_voronoi_fps.py::TestVoronoiFPS::test_threshold
  /home/runner/work/scikit-matter/scikit-matter/src/skmatter/_selection.py:211: UserWarning: Score threshold of 0.4 reached.Terminating search at 6 / 10.
    warnings.warn(

So it seems like the same warning was sent by multiple tests, throwing me off initially.


Unsafe memory handling when decoding frames

Memory handling for decoded AVFrames can either be the responsibility of the caller or of the codec, depending on whether AVCodecContext.refcounted_frames is set to 1 or to 0.
Explained here (parameter picture):
http://ffmpeg.org/doxygen/3.2/group__lavc__decoding.html#ga3ac51525b7ad8bca4ced9f3446e96532
If refcounted_frames is set to 0, the decoder owns the memory of the frames and will reuse this memory when decoding further frames. Therefore frames are only valid until the next call to the decoder and this can cause problems for a rust programmer that expects the obtained ffmpeg::util::frame::video::Video struct to remain valid until it gets destroyed.
In my case I was extensively moving the obtained frames around (even through various threads) and it took me a long time to realize that the origin of my problems (frame duplication/reordering, contrast/color separation) was the ffmpeg decoder overwriting the frames that I had already decoded and which were in various stages of further processing. Cloning the frames immediately after decoding solved the problem for me.
So far I'm not sure how to solve this problem "cleanly". Maybe the decoder could just return something like a borrowed reference to a frame and therefore force the rust programmer to either release the reference or to clone the frame before calling the decoder again.

The issue is that it's decided at runtime whether you can keep the frame around or not.
I guess it's a matter of deciding whether to require the user to always clone the frame to pass it around, or clone it internally when refcounted_frames is 0.
What do you think is the best option?

I was thinking about something like this: https://is.gd/BxBCSC
Try to comment out the curly braces in the main function to see how I propose to use the borrow checker to force the programmer to drop all references to the current frame before calling the decoder again to get the next frame.
This way, we should get optimal performance in all situations.

Short, immediate access to the Frame data structure that is finished before decoding the next frame:
Can be done through the borrowed reference obtained by get_next_frame() to the internal Frame struct encapsulating the AVFrame and requires no copying.
Before decoding the next frame the compiler forces the programmer to drop any borrowed handles to this internal data structure, so no data corruption can take place.

Long-term access to Frame data structure beyond the decoding of the next frame:
Can be done with the get_next_frame().to_cloned() method combination. With copy-cloning the frame memory, the decoder memory management gets decoupled from rust memory management and allows further processing of the frames independent of the decoder.

A final, maybe over-the-top optimization could take place in the to_cloned()-method. In case refcounted_frame is 1 the internal Frame could just be moved to the caller instead of copy-cloning the entire frame data. That would be faster. But the downside is, that you can move out the frame data only one time. So this method needs to return a Option<Frame> with Some() for the first call and None the following calls.

There are the std::borrow::Cow and std::borrow::ToOwned traits that potentially could model this idea. However, so far I haven't been able to figure out what happens to objects on the stack when they get encapsulated with Cow, do they get copied to the Heap? Ideally I would like to avoid any unnecessary copy operation.


Incorrect response message after changing the status of Payment Method.

Framework Version: 0.1.3
On changing the status of Payment method its displaying that shipping method is created successfully.

Framework Version: 0.1.4
Fixed


Update recordId description in unlinkFromConversation method

Update recordId property description to correctly reflect the unlink action

Closing this PR. Have addressed this issue in a fresh PR and merged the changes. Here's the updated content:
unlinkFromConversation
Thanks


Make a cli

Product and Version [VS/VSCode]:N/A
OS Version [macOS/Windows]:macOS/Windows/Linux
Live Share Extension Version: N/A
Target Platform or Language [e.g. Node.js]:N/A
Situation: I have a very specific case. I work with dotnet core. I'm not admin on the computer. Because of a bug i need to work on other machine (a vm). I use Live share to work on this machine without RDP.
I would like to be able to launch a share session without lauching the UI (in SSH). And without VS.

@GeraudFabien Hey! We’re tracking this request via #74, so if you could upvote that item, that would be great. In the meantime, I’ll close this issue as a duplicate, that way we can consolidate the conversation to #74. Please re-open this if you don’t think #74 accommodates your use case. Thanks!


Implement Demo Mode

Would like to have a Demo button in order to see the app working without having to initially paste data

Fixed this in https://github.com/jasonlutterloh/metrics-peloton/releases/tag/v1.2


Unmatched ( in regex

I'm getting this:
Unmatched ( in regex; marked by <-- HERE in m/^( <-- HERE $/ at /opt/homebrew/bin/bibcop line 115, <$fh> line 1.

@rultor release, tag is 0.0.22

@rultor release, tag is 0.0.22

@yegor256 OK, I will release it now. Please check the progress here.

@rultor release, tag is 0.0.22

@yegor256 Done! FYI, the full log is here (took me 5min).


RemoteSelectField: add onChange prop

closes inveniosoftware/invenio-app-rdm#763

After discussion, this is not needed anymore.


Opt-depend on default

Let other games than just default use the circular-saw/stairsplus api
TODO:

[ ] don't load the nodes.lua file if the default mod isn't available
[ ] don't load the redefinitions.lua file if the default mod isn't available
[ ] don't load the crafting.lua file if the default mod isn't available (will also disable the recipe for moreblocks:circular_saw, only obtainable via /giveme moreblocks:circular_saw then)
[ ] don't prepend default.gui_* fields to the circular formspec if they are not defined (https://github.com/minetest-mods/moreblocks/blob/1f1959ba54e503043aa1f90e2639beb1353a9c5a/circular_saw.lua#L368)
[ ] add nil-fallback for default.node_sound_* calls (no sounds)

@Calinou opinions/objections?

@Calinou opinions/objections?

Sounds good to me, feel free to open a pull request for this :slightly_smiling_face:


perf: don't manipulate an empty value

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

We should not iterate and make O(n) operations on an empty string, just to receive the same empty string back.
📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have added tests (if possible).
[ ] I have updated the documentation accordingly.

Well, the failing tests don't really have anything to do with the PR... I haven't changed anything in the component island URL's.

@danielroe is this something I accidentally did with one of the changes?

Always glad to help!


Extend device selectors to refer to devices by name

So we can ask "turn on my bedroom lights"
This is a feature asked by @balloob.
This is only part 1. Part 2 is the dialog agent to actually select the device. Part 3 is Genie to train those sentences. Part 4 is every Thingpedia device that would like this functionality should add relevant templates and change their confirmation string.

Pull Request Test Coverage Report for Build 715

105 of 128 (82.03%)  changed or added relevant lines in 10 files are covered.
42 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 82.108%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/ast/slots.js
9
10
90.0%

lib/describe.js
16
18
88.89%

lib/prettyprint.js
10
12
83.33%

lib/typecheck.js
23
27
85.19%

lib/ast/class_def.js
9
16
56.25%

lib/nn-syntax/tonn_converter.js
9
16
56.25%

Files with Coverage Reduction
New Missed Lines
%

lib/nn-syntax/parser.js
42
68.86%

Totals

Change from base Build 713:
-0.07%

Covered Lines:
7164

Relevant Lines:
8487

💛  - Coveralls


[BUG] Can't test a component that uses useCustomMutation

Describe the bug
Components that are using refine hooks are impossible to test.
One of the examples is a component making a custom request to server.

Here is my component code.
import { FileUpload, UploadOutlined } from '@mui/icons-material';
import { useApiUrl, useCustomMutation } from '@pankod/refine-core';
import {
  Button,
  Card,
  CardActions,
  CardContent,
  CardMedia,
  LoadingButton,
  Modal,
  SxProps,
  Typography,
} from '@pankod/refine-mui';
import { FC, useState } from 'react';
import { ThumbnailUploadResponse } from 'types';

const cardStyle = {
  position: 'absolute',
  top: '50%',
  left: '50%',
  transform: 'translate(-50%, -50%)',
  maxWidth: 400,
};

type ThumbnailUploaderProps = {
  handleThumbnailUploadSuccess: (link: string) => any;
  sx?: SxProps;
};
const ThumbnailUploader: FC<ThumbnailUploaderProps> = ({
  handleThumbnailUploadSuccess,
  sx,
}) => {
  const [modalOpen, setModalOpen] = useState(false);
  const [fileValue, setFileValue] = useState<File | null>(null);
  const [fileUrl, setFileUrl] = useState('');
  const { mutate, isLoading } = useCustomMutation<ThumbnailUploadResponse>();
  const API_URL = useApiUrl();

  const handleOpen = () => setModalOpen(true);
  const handleClose = () => setModalOpen(false);

  const handleInputChange: React.ChangeEventHandler<HTMLInputElement> = (
    event
  ) => {
    if (!event.target.files?.length) return;
    const file = event.target.files[0];
    setFileValue(file);
    setFileUrl(URL.createObjectURL(file));
    handleOpen();
    event.target.value = '';
  };

  const handleCancelUpload = () => {
    URL.revokeObjectURL(fileUrl);
    setFileUrl('');
    handleClose();
  };
  const handleSubmitUpload = () => {
    if (!fileValue) {
      return;
    }
    const formData = new FormData();
    formData.append('file', fileValue);
    mutate(
      {
        url: `${API_URL}/thumbnail`,
        values: formData,
        method: 'post',
        successNotification: {
          message: 'Succesfully uploaded thumbnail.',
          type: 'success',
        },
        errorNotification: {
          message: 'Could not save thumbnail. Please try again later.',
          type: 'error',
        },
      },
      {
        onSuccess: (data) => {
          handleCancelUpload();
          handleThumbnailUploadSuccess(data.data.url);
        },
      }
    );
  };
  return (
    <>
      <Button
        variant="outlined"
        color="primary"
        aria-label="upload picture"
        component="label"
        size="large"
        sx={sx}
      >
        <input
          hidden
          accept="image/*"
          type="file"
          onChange={handleInputChange}
          data-testid="image-input"
          id="image-input"
        />
        <FileUpload />
      </Button>
      <Modal
        open={modalOpen}
        onClose={handleClose}
        aria-labelledby="modal-modal-title"
        aria-describedby="modal-modal-description"
      >
        <Card sx={cardStyle}>
          <CardMedia
            component="img"
            image={fileUrl}
            sx={{
              maxHeight: '400px',
              objectFit: 'contain',
            }}
          />
          <CardContent>
            <Typography id="modal-modal-title" variant="h6" component="h2">
              Are you sure you want to upload this image as a thumbnail?
            </Typography>
          </CardContent>
          <CardActions sx={{ justifyContent: 'end' }}>
            <Button
              variant="outlined"
              color="error"
              disabled={isLoading}
              onClick={handleCancelUpload}
            >
              Cancel
            </Button>
            <LoadingButton
              variant="contained"
              startIcon={<UploadOutlined />}
              loading={isLoading}
              loadingPosition="start"
              onClick={handleSubmitUpload}
            >
              Upload
            </LoadingButton>
          </CardActions>
        </Card>
      </Modal>
    </>
  );
};

export default ThumbnailUploader;

Here is my test code.

/**
 * @jest-environment jsdom
 */
import { fireEvent, render, waitFor } from '@testing-library/react';
import axios from 'axios';
import ThumbnailUploader from 'components/ThumbnailUploader';
import { QueryClient, QueryClientProvider } from 'react-query';
import { ThumbnailUploadResponse } from 'types';

const queryClient = new QueryClient();

jest.mock('axios');
const mockedAxios = axios as jest.Mocked<typeof axios>;

it('', async () => {
  const url = 'mockedUrlValue';
  const resp: ThumbnailUploadResponse = { url };
  mockedAxios.post.mockResolvedValue(resp);

  const file = new File(['(⌐□_□)'], 'chucknorris.png', {
    type: 'image/png',
  });

  const handleThumbnailUploadSuccess = jest.fn((link: string) => link);
  const { queryByLabelText, getByLabelText, getByTestId } = render(
    <QueryClientProvider client={queryClient} contextSharing={true}>
      <ThumbnailUploader
        handleThumbnailUploadSuccess={handleThumbnailUploadSuccess}
      />
    </QueryClientProvider>
  );

  const uploadInput = getByTestId('image-input');
  await waitFor(() =>
    fireEvent.change(uploadInput, {
      target: { files: [file] },
    })
  );

  let uploadInputFromDom = document.getElementById(
    'image-input'
  ) as HTMLInputElement | null;
  // file should be saved to state and cleared from DOM
  expect(uploadInputFromDom?.files?.length).toBe(0);
});

As you can see I even tried wrapping it onto QueryProvider(as suggested here) but it didn't work saying:
No QueryClient set, use QueryClientProvider to set one
Screenshot below.
Steps To Reproduce
Just try to use custom hook in separate component and try to run tests for it.
Expected behavior
Should be easy to test components(Or at least documentation should have more info about how to actually unit test components).
Screenshot

Desktop

Mobile

Additional Context
No response

Hey @MetaMmodern, instead of just wrapping the component to QueryClientProvider, you probably need to prepare a test wrapper with <Refine /> component. Let me share you our UI package tests and wrappers that might give you an idea about that,
@pankod/refine-antd's test setup here: https://github.com/refinedev/refine/blob/next/packages/antd/test/index.tsx#L28
@pankod/refine-mantine's here: https://github.com/refinedev/refine/blob/next/packages/mantine/test/index.tsx
ErrorComponent's test here as an example of component testing https://github.com/refinedev/refine/blob/next/packages/antd/src/components/pages/error/index.spec.tsx#L21-L27
Edit component's test here for testing with routing https://github.com/refinedev/refine/blob/next/packages/antd/src/components/crud/edit/index.spec.tsx#L10-L55
useTable hook test example here: https://github.com/refinedev/refine/blob/next/packages/antd/src/hooks/table/useTable/useTable.spec.ts#L27-L29
You can check out our codebase for more details.
Hope this helps!

@aliemir I'm using mui so I guess this one will help me: https://github.com/refinedev/refine/blob/next/packages/mui/test/index.tsx
Please don't close this issue, I'll see how this works and get back to you(hopefully today)

Ok, I see, I need to create my own wrapper. It would be nice though if you could expose those wrappers, they might bbe useful for most cases.

@aliemir awesome, works perfectly) I have a side question maybe you may know. As you saw above I've mocked axios post call and currently I'm trying to check what was that mock called with(with what arguments) and I see that it's being called with undefined. Do you maybe know what may be the reason for that?
I've updated the mock block so it's now like this:
  const url = 'mockedUrlValue';
  const resp: { data: ThumbnailUploadResponse } = { data: { url } };
  mockedAxios.post.mockResolvedValue(resp);

So in tests I see it's being called after mutate being called, but value is undefined:

@aliemir what if I get this error when trying to provide a custom daatProvider:
There is no "default" data provider. Please pass dataProviderName.

Hey @MetaMmodern sorry missed your comments but I guess the case resolved, right? 😅

Ok, I see, I need to create my own wrapper. It would be nice though if you could expose those wrappers, they might bbe useful for most cases.

Not all of the props are required and we thought exporting those mocks might lead to misunderstandings since they're just dummy placeholders and we need to mock properly when we need them in tests 😅

Hey @MetaMmodern sorry missed your comments but I guess the case resolved, right? 😅

Yes, case resolved, thanks a lot) you can close the ticket if you want

Ok, I see, I need to create my own wrapper. It would be nice though if you could expose those wrappers, they might bbe useful for most cases.

Not all of the props are required and we thought exporting those mocks might lead to misunderstandings since they're just dummy placeholders and we need to mock properly when we need them in tests 😅

Well, according to code most of props of TestWrapper are optional. And that's pretty comfy for user tests, because in my case I just copypasted your testwrapper and used it as is. Only on test I provided my own dataProvider, I didn't have ro pass all the other providers since they were mocked by TestWrapper if props not passed.

everything seems to be solved about this issue :)

@aliemir what if I get this error when trying to provide a custom daatProvider: There is no "default" data provider. Please pass dataProviderName.
UPDATE: I found the reason. I had to add all methods(or some of required ones) to dataProvider so it counts as a normal dataProvider. Again, case solved) hehe

Hi. May i know your where did you get RefineTestWrapper ?


Bug: kpt fn source includes non-KRM resources

Expected behavior
kpt fn source "${SRC_DIR}" to exclude non-KRM resources (like .prettierrc.json files)
Actual behavior
kpt fn source "${SRC_DIR}" includes non-KRM resources (like .prettierrc.json files), which can cause subsequent kpt functions to error because they expect all resources to be KRM resources.
Workaround
Filter for KRM resources manually:
kpt fn source "${SRC_DIR}" \
| kpt cfg grep "kind=" \
| kpt fn run --image "${IMAGE}" -- "key=value" \
| kpt fn sink "${DEST_DIR}"

Information
Kpt Version: 0.37.1

This issue needs to update the cmd/config, will do it after #1698 merged.

@droot @frankfarzan Should we filter out non-KRM resources in fn render and fn eval?

Here is my mental model: kpt should process files with yaml, yml, json extensions by default. If user wants to exclude certain files from processing,  they can add such files to.krmignore. So in this case, adding .prettierrc.json to .krmignore should work unless there is a bug. @Shell32-Natsu can you pl. verify that ?
There's been a discussion of tracking these ignore files in the Kptfile instead extra files such .kptignore (I do think that can make discovering it easier).
/cc @frankfarzan @mortent

@droot The .krmignore works fine.

Different discussions here:

Mechanism for kpt to ignore files and directories:

For next branch, we need to finalize the mechanism for ignoring files and directories. https://github.com/GoogleContainerTools/kpt/issues/1664
On master branch, we should document the current behavior of using .krmignore.

kpt should enforce a precondition that all items in ResourceList (.items and .functionConfig fields) should have the minimum KRM type meta. It's debatable which of GVKNN needs to be required. I think GVK is a must. @droot This ties into enforcing preconditions for hydration.

@karlkfi So for this particular issue, the solution is adding the non-KRM resources you don't want to include to .krmginore file. We may improve the ignore list functionality in the kpt v1.

#1733


feat(DEV-12231): custom Tabs for Receivables 🎲

Added support for configuring custom tabs in Receivables Tables using MUI Theming.
Maintained backward compatibility with default tabs: Invoices, Quotes, and Credit Notes.
Updated the <ReceivablesTable/> component to use numeric tab indices instead of ReceivablesTableTabEnum.
Changed default tab indices:

Invoices: 0
Quotes: 1
Credit Notes: 2

@hkosova, could I ask you to validate the status machine?

could I ask you to validate the status machine?

@radist2s LGTM


A11y_Azure Tools for VS Code [BM]_Create Function App in Azure_Screen Reader:NVDA is not reading the heading for the pop up when we press enter on "Create Function App in azure" menu item.

Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
Actual Result:
NVDA/Narrator is not reading about check box information when we navigate to list items present inside "Filter" combo box.
NOTE:
No issue with Narrator
Expected Result:
NVDA/Narrator should read about check box information when we navigate to list items present inside "Filter" combo box. It should read the state checked/un checked
GitHubTags:
#Azure Tools for VS Code; #A11ySev2; #A11yTCS; #BM-Azure Tools for VS Code-Win-Feb2021; #DesktopApp; #Screen Reader; #WCAG1.3.1; #NVDA; #Win10; #Element:Listitem;
OS Details:
Windows 10 Version 20H2 (OS Build 19042.685)
Environment Details:
Application Name: Azure Tools for VS Code
Version: 1.53.1
Commit: 5d424b828ada08e1eb9f95d6cb41120234ef57c7
Date: 2021-02-08T23:29:42.785Z
Electron: 11.2.1
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
Azure Function Version: 1.2.1
Repro Steps:
Open Visual studio code
Turn on NVDA/Narrator
Tab till extension button and press enter on it.
Search for "Azure Functions" and select it.
Tab till "Install" button and press enter on it.
Tab till "Azure" button from left side pane and press enter on it.
Default screen will get open.
Tab to Tree view and use down arrow to navigate to tree items.
Press enter on Context menu and navigate to "Create Function App in Azure" list item
Press enter on it and verify NVDA is reading heading of the pop up or not.
User Impact:
Screen Reader users will not know the functionality of controls if screen reader is not giving complete information
Attachment for Reference:

2691_NVDA is not reading heading for pop up.zip

This should be filed directly on VS Code as I can repro with the sample quick input extension: https://github.com/microsoft/vscode-extension-samples/tree/main/quickinput-sample


EOSS  Cycle 6 - Jupyter Accessibility proposal

I am opening this issue as @isabela-pf, @gabalafou, and I are working on a CZI EOSS Cycle 6 proposal to continue and expand on Jupyter accessibility work carried out during through the grant submitted in EOSS Cycle 4 (ref: https://github.com/jupyter/governance/issues/100)
You can read the submitted and accepted letter of interest in pdf format[^1]
So, as in the past, I am requesting approval for this work and will post the entire proposal as it becomes available.
[^1]: I have been on leave and just got back, but I will be adding this to the jupyter/accessibility repository as we have done in the past

Looks like a great project!


SS-6180, SS-7851

Selection input:
https://shapediver.atlassian.net/browse/SS-7851
https://shapediver.atlassian.net/browse/SS-7800
Gumball input:
https://shapediver.atlassian.net/browse/SS-6180

@MajorMeerkatThe3rd I noticed that the line endings of the files you added are all LF, while the line endings of other files are CRLF. We need to set a git attribute to make sure we don't get such a mix (I will do that once the PR is merged). For now please convert everything to CRLF and push.


Only check flags for config samples

I moved and renamed a couple random other, unrelated things that I noticed, but the main things to look at are the fetch_flag_files() function and the Checker class
Resolve https://github.com/epigen/looper/issues/177

I think my review somehow got posted on the wrong commit, but anyway, a couple of first thoughts:

do you really think we need a function class here instead of just a function for checking?
what do you think of @rcorces' suggestion to have a -a option to preserve current behavior, is that unnecessary?
what is the new partial flag?
I think we should move the flag list to a global setting somewhere

Cool thanks for the review. I'll try to match up replies to each item:

The Checker class is just to match what's already in place with the other looper programs now (Runner, Cleaner, etc.). I think the framework for it is here: https://github.com/epigen/looper/commit/15298d6a76dddae566d26fb836301bc04b09eba5
Ah yeah forgot about the -a; I think it's a good idea and will add it in. I actually first did that way, then converted, so it won't be too tricky to have both.
"partial" is there for use with the checkpoint concept in pypiper
Totally agree, the flags should be at higher scope. I was going to do that along with pep migration but it could be done now also, within looper. I bubbled them up in pypiper so wouldn't hurt to do the same here.

Ok sounds good. On the flags -- would it be possible to make this function totally open so it could handle any possible flags? What if other software uses different flags, etc?

Yes

OK, capability for just Project samples and all folders within output dir is in now. I went with -A and --all-folders for the short/long options to trigger for check. There's also the ability to just check for specific flag(s) now with -F/ --flags


Upgrade to rustls 0.21

Currently webrtc-rs uses the WebPKI crate, which has a RUSTSEC advisory open. Because webpki is not actively maintained, it's unlikely to get fixed.
I investigated a bit, and currently webrtc-rs needs it because it's using rustls 0.19, which interoperates with the webpki crate.
However, in rustls 0.21, the webpki crate is removed and replaced by rustls-webpki, which is actively maintained. So if we upgrade to Rustls 0.21 we will avoid the Rustsec warning.
I tried to open a PR myself but I'm not familiar enough with the WebRTC internals and got stuck, sorry!

I will take a look once I got time.
But any PR to fix it is also welcome.

I understand. I'm not familiar enough with WebRTC's security model, but if I find myself with enough time to figure it out, I'll open a PR.


Add DoesDocumentExists check for stress tests

Add explicit check DoesDocumentExists for stress tests, to check if the document exists.
Previously, we just assume that the document exists when a testId is supplied to nodeStressTest.ts; This change ensures that we explicitly check if a document corresponding to the testId passed does indeed exist.
Implemented for odsp driver, raises "not implemented" for other drivers.
Modify type validation checks to take into account the forward compatibility that will be broken for certain classes and functions because of this change (addition of the doesDocumentExists method to ITestDriver).

why is this necessary? please open an issue that describe the scenario/need for this change

@anthony-murphy
Sure, I'll encode this behavior into the function itself rather than create a separate method for it - so that it does not regress other stress tests.
Prior to this change, we just assume that the corresponding document exists when a testId is supplied to nodeStressTest.ts; And when such a document does not actually exist, the stress test would simply error out. Rather, what we want to do now is to check if the corresponding document exists, and in the case that it does not exist, we go ahead and create a document (which is the default behavior when no testId is passed to nodeStressTest.ts) rather than error out and stop the test.

Hey Tony, I can see that all the jobs in the pipeline are succeeding except for the "Check for extraneous modified files"
https://dev.azure.com/fluidframework/public/_build/results?buildId=78640&view=logs&j=3dc8fd7e-4368-5a92-293e-d53cefc8c4b3&t=035f10b1-6721-511e-bd48-d7e028467fc2
Could you please help me understand what this is about? (I'm not able to see which bash script does this check for extraneous modified files, and which ones it is referring to)

Oh yes, certainly. I did do a full local build - and it was successful. The type validation files were generated as well.

you can't just delete files.

I guess you're referring to commit c90e5f1 [remove type validation files that will be auto generated on build]. I had checked in a couple of auto generated type validation files in the commit just before that. I'm just removing those files in commit c90e5f1 as I suppose that the auto generated type validation files shouldn't be checked in. (please correct me if my understanding is incorrect/if those files should also be checked in)

Oh yes, certainly. I did do a full local build - and it was successful. The type validation files were generated as well.

you can't just delete files.

I guess you're referring to commit c90e5f1 [remove type validation files that will be auto generated on build]. I had checked in a couple of auto generated type validation files in the commit just before that. I'm just removing those files in commit c90e5f1 as I suppose that the auto generated type validation files shouldn't be checked in. (please correct me if my understanding is incorrect/if those files should also be checked in)

they should be checked in. you are actively deleting files that exist in the remote git repository

Seems like something went wrong here with a merge/rebase, 48 commits and 380 files with changes is much more than the PR is about. @Harivallabha-Rangarajan maybe it is a good idea to rebase this branch off the tip of main so we see only the intended changes in the PR? Or start a new branch from the tip of main and copy the changes to that one, if the rebase is too tricky.

@alexvy86 You're right, yeah it's become quite messy after the rebase. I'll close this PR and recreate the changes on top of main again, the changes themselves are small, and it would be cleaner to do so :")


[Feature] Support in-built Grafana for Ray Cluster

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Ray in-built dashboard is pretty helpful for most telemetry use case. Internally, we also like to leave an eye on control plane metrics like gcs, gc object etc.  These metrics are already exposed via prometheus and what we need to do is to build a ready to use dashboard for cluster admins to monitor the cluster.
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

/cc @scarlet25151

I think we have put grafana config in the repos and we may close the story?

PR https://github.com/ray-project/kuberay/pull/230 add the support and we can close this one.


Add RBF FAQ

[ ] Needs polishing
[ ] add to navigation

Looks good @btcdrak :+1:

@greenaddress did you have any novel use case that isn't listed here?

@dcousens not really, we plan to support the main wallet use cases such as bump fee to begin with and then more advanced features to update unconfirmed transaction with new input and output when sending transactions in quick succession, etc

Not sure now this got merged, I must have mad a mistake on the command line.

Reopened in #60


Call method unselectFeatures() Exception

Each time featureLayer.unselectFeatures(selectFrature) is called, an error is reported, and both featureLayer and selectFrature are not null.
in sdk 100.3
Thank you very much for taking the time to pay attention to this issue.

Hi,
Sorry for the late response.  If you are still experiencing issues with your app code, then you should post your question in the following forum https://github.com/Esri/arcgis-runtime-samples-android/issues/598
This repository is for issues related to the sample code only.


Learn gem auto-updater?

If we want to output a message encouraging the student to update:

Check if learn-co shows up in gem outdated
Grab current version from RubyGems API and then gem list -i learn-co -v <version>

If we want to automatically update the student's gem:

gem install learn-co

Could make any of these options a cron job w/ whenever, or we could throw a hook into something like learn-open.

@gj is this still something that you'd be interested in having implemented? I'm a flatiron grad and would like to take a look at it if so!

Hey @kreopelle, I'll bring this up at work and get back to you this week. Thanks for the enthusiasm!

@gj - thank you! any word? also if there are other things that would be more beneficial, would be interested in giving back any way I can

Really appreciate the enthusiasm @kreopelle ! I asked about this last week but haven't received a definitive answer. If you want to spike out an implementation, I'm happy to review your code, but I can't promise if/when it would make it into the distributed gem.

Hey @kreopelle, I have a small project you could work on if you're still interested! Let me know 🙂


Fixed one typo

I ended up fixing this is a different push, it was easier than merging this PR... but thanks!


getrawchangeaddres: status code: 401, response: ""

Going through your readme file but instead of using btcatomicswap with dcratomicswap, I am using btcatomicswap with ltcatomicswap.
When I get to the command:
dcratomicswap --testnet redeem
Or in my case
ltcatomicswap --testnet redeem
I get the following error:
getrawchangeaddres: status code: 401, response: ""
Any idea why?

I'm not a maintainer of this repo so I cannot give you a definitive answer, but I can tell you that a 401 HTTP status typically means "Unauthorized". I guess this probably means you need to unlock your wallet.

Usage: ltcatomicswap [flags] cmd [cmd args]

Commands:
  initiate <participant address> <amount>
  participate <initiator address> <amount> <secret hash>
  redeem <contract> <contract transaction> <secret>
  refund <contract> <contract transaction>
  extractsecret <redemption transaction> <secret hash>
  auditcontract <contract> <contract transaction>

Flags:
  -rpcpass string
        password for wallet RPC authentication
  -rpcuser string
        username for wallet RPC authentication
  -s string
        host[:port] of Litecoin Core wallet RPC server (default "localhost")
  -testnet
        use testnet network

Similarly to btcatomicswap, for ltcatomicswap, you need to specify the -rpcuser and -rpcpass flags.


Contribution process

Hello!
In our country we have specific format of key store and signature mechanism (little endian vs big endian).
My idea is to contribute to BC in order to support our formats.
Please, can you point me a link to contribution guide.

It's at the bottom of the master README... a more helpful suggestion would  to implement one thing and then send a pull request. It's very difficult to review submissions with lots of files, it's better to send us a few things at a time. Thanks.

Regarded as answered.


Support for function calling

Hi, I am wondering if you could add Function calling support that was added in gpt-3.5-turbo-0613. I've been able to use raw mode to implement my own using this package but it would be nice to have it like token mode, but for the arguments field in function_call.
https://platform.openai.com/docs/guides/gpt/function-calling

@ctjlewis curious if this is on the roadmap? :)


Ajax example for update tree

Any example using update_tree endpoint with jquery?
So I created an endpoint like this:
MvcRouter::admin_ajax_connect(array('controller' => 'admin_questions', 'action' => 'save_ajx'));

and its save_ajx method, now I don't know how the jquery request should look like, I have this:
jQuery.post('<?php echo mvc_admin_url(array('controller' => 'admin_questions', 'action' => 'save_ajx')); ?>', function() {
    console.log('success')
});

but it's not doing what I think it should, any help ? :)

Sounds like you found your answer? If so could you post in case anyone else comes looking here? :)


centre all mind maps no matter how much you zoomed in/out previously

Currently it keeps the zoom level when exploring different maps and it is really unintuitive.

Im gonna take a look at this, sounds like the the zoom just needs to be reset after every new request


Leftover 'use-bank' config setting.

https://github.com/darbyjack/Guilds-Plugin/blob/master/src/main/resources/config.yml#L548
Not part of any other role position, so I assume this wasn't removed with the others.

Good catch!


rotate function typo

silly mistake, but the Euler-Rodrigues matrix is incomplete.
The variables are assigned like:
aa, ab, ac,ad = powers[0:4]
BB, bb, bc, bd = powers[4:8]
CC, cb, cc, cd = powers[8:12]
DD, db, dc, dd = powers[12:16]
Since ba = ab, ca = ac and da = ad; there are no issues in the calculations stemming from this, but it's still a bit silly. ;)
In the same category of silly:
Rotate 90 degrees over the X axis
meshes[0].rotate([0.0, 0.5, 0.0], math.radians(90)) <- This should be [1., 0., 0.] right? Else you rotate about the y axis.
(and the one right after as well)

Oops... silly me. Guess I was a bit tired when writing that :P


Add a ip-based rate limiter

Currently there is no rate limiter on the jsonhero.io cloud, but we should probably add one to prevent abuse

Currently JSONHero is meant to be deployed to Cloudflare on Cloudflare workers, so we have access the Cloudflare KV and Cloudflare DurableObjects, one of which might make sense for this rate-limiter? I'm guessing DurableObjects would be a better fit but I haven't looked into it.

Also I'm thinking the initial rate limit should be at most 10 per minute per IP, but would be good to control this with an env var

/bounty $400

/attempt #158

@ericallam  taking a look at DurableObjects

@daveads i think hold off for now. It actually probably doesn't make sense to include the rate limiter in code as self-hosters wouldn't really need it and we can instead use Cloudflare firewall to rate limit JSONHero.io. I'm going to close this issue for now, hoping you didn't spend too much time on this.

@ericallam I don't think this even needs to be hard coded with Cloudflare workers. You can create a rate limiter based on the IP by going onto the dashboard and setting the rule on your firewall. Here is a link:
https://www.cloudflare.com/rate-limiting/


How to deal with slice of pointers

package main

import (
	"fmt"

	"github.com/koss-null/funcfrog/pkg/pipe"
)

type Student struct {
	Number uint32
	Name   string
}

func main() {
	items := []*Student{
		{Number: 10000, Name: "Tom"},
		{Number: 10001, Name: "Jerry"},
	}

	item := pipe.Slice(items).
		Filter(pipe.Pred(func(item *Student) bool {
			return item.Number > 10000
		})).
		First()

	fmt.Println(item, *item, **item)
}

Hi! It looks like you need a double pointer on predicate reciever here. Filter expects the type in the predicate to be a pointer on your slice type wich is *Student itself.
Also you may use FuncP initialization and return *Student[i] from there.

item := pipe.
	Fn(func(i int) *Student {
		return items[i]
	}).
	Gen(len(items)).
	Filter(pipe.Pred(func(s *Student) bool {
		return s.Number > 10000
	})).
	First()

Is that right? But it feels a little complicated.

Not quite. This is the wright way https://play.golang.com/p/649_akFwmug
Also you don't need to explicitly convert to pipe.Pred, you just send the function as is.


bug: no back button when push page from popover (beta11)

Short description of the problem:
When using NavController from a popover.dismiss to push a page, the page is shown without a back button on navbar.
When it is pushed without dismiss, it works ok.
Hardware back button works as expected.
What behavior are you expecting?
To see a back button on navbar
Steps to reproduce:

ionic serve on https://github.com/hagai26/ionic-conference-app
go to About Page using the side menu
click popover -> push signup page no dismiss (there is back button)
click popover -> push signup page (no back button - BUG)

  // bug
  pushSignupPage() {
    this.viewCtrl.dismiss().then(() => {
      this.navCtrl.push(SignupPage);
    });
  }
  // no bug
  pushSignupPageNoDismiss() {
    this.navCtrl.push(SignupPage);
  }

Which Ionic Version?
Your system information:
Cordova CLI: 6.3.0
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0-beta.36
Ionic App Lib Version: 2.0.0-beta.19
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v6.3.1
Xcode version: Xcode 7.3.1 Build version 7D1014

Hello! Thanks for opening an issue with us! Im pretty sure that the issue here is that your are calling push() from the popovers navController (because all overlay components now have their own nav in beta.11) instead of the "main" navController. Would you mind trying this?
 import { App } from 'ionic-angular';

constuctor(public app: App) {};

pushSignupPage() {
    this.viewCtrl.dismiss().then(() => {
      this.app.getRootNav().push(SignupPage);
    });
  }

Thanks for using Ionic!

Great, I didn't notice that.
It works great! 10x

This work great with ionic back button
but how to join it with device Back Button


Moving palace 6 and new kasuto

Special locations because they are hidden on the map.

436F: A5 73	LDA $73		; Y position on map (square unit)
4371: C9 64	CMP #$64 (100)	; Y position of Hidden Palace call spot
4373: D0 12	BNE $4387
4375: A5 74	LDA $74		; X position on map (square unit)
4377: C9 2D	CMP #$2D (45)	; X position of Hidden Palace call spot
4379: D0 0C	BNE $4387


go get fails

Should go get work? If so, this is what I am seeing.
../../go/src/github.com/jlhawn/dockramp/build/util/util.go:78: undefined: v2.ValidateRepositoryName

If not, can we add a Makefile similar to docker?

Also, README should be updated explaining how to get golem. :)

Adding a Makefile is reasonable. It should build fine with godep.
Also note #1, building for mac is currently not supported since the binary builds itself into a docker image  with the current approach. I will likely just document this until figuring out what the best solution for developing Golem on the mac is, using a docker image will likely be the preferred usage.  Thanks for trying it out.

I'm pulling from a linux system. :)
I'll submit a starter Makefile later on unless someone beats me to it.