test: use ts-jest with isolatedModules to improve performance

YJDH-690.
See more:
https://kulshekhar.github.io/ts-jest/docs/27.1/getting-started/options/isolatedModules

Alternative for https://github.com/City-of-Helsinki/yjdh/pull/2876.

API branch is deployed to platta: https://tet-pr2877.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

ADMIN branch is deployed to platta: https://tet-admin-ui-pr2877.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tet-admin-ui-pr2877.dev.hel.ninja :laughing::tada::tada::tada:

YOUTH branch is deployed to platta: https://tet-youth-ui-pr2877.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://tet-youth-ui-pr2877.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://kesaseteli-pr2877.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

EMPLOYER branch is deployed to platta: https://kesaseteli-employer-pr2877.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://kesaseteli-employer-pr2877.dev.hel.ninja :laughing::tada::tada::tada:

HANDLER branch is deployed to platta: https://kesaseteli-handler-pr2877.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://kesaseteli-handler-pr2877.dev.hel.ninja :laughing::tada::tada::tada:

YOUTH branch is deployed to platta: https://kesaseteli-youth-pr2877.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://kesaseteli-youth-pr2877.dev.hel.ninja :laughing::tada::tada::tada:

Closing this because the alternative was merged: https://github.com/City-of-Helsinki/yjdh/pull/2876. It should actually include the insolatedModules as a forced change, if I understood properly, so it kind of includes the changes done in this branch.


Weird title bar on unfocused borderless window

I'm having an issue with this weird title bar thing being visible for a split second on an unfocused borderless window. Here's a video of it on the SimpleImage demo (modified only to make the window borderless):
https://github.com/openfl/lime/assets/47027981/a4cd722e-0607-4068-8f84-e2190e92505f
You can see it show up sometimes for a very short amount of time after I click off the window.
Weirdly enough, when testing with OpenFL, the title bar seems to stay visible for as long as the window is unfocused:
https://github.com/openfl/lime/assets/47027981/5cb084b5-ef13-4a7d-a4a2-226ee6d7ed35
It seems to happen on both Hashlink and Windows C++ targets. I'm using Lime 8.1.1, OpenFL 9.3.2 and Haxe 4.3.4
Is there a way to hide it?

I've narrowed this down to be an issue with SDL2:

However I couldn't get it to happen with SDL3, so I guess this issue waits for #1684 to be merged

This only happens if you change the borderless property after a window has been created with borderless property as true.
Set borderless = "false" in the window tag in your project xml and it will work in hxcpp. For some reason this is broken on hashlink.

It works fine in hashlink when you create a new borderless window eg.
var winAtt:WindowAttributes = {};
		winAtt.borderless = true;
		winAtt.title = "test";

		Lib.application.createWindow(winAtt);
		```

		I'm trying to figure out why the borderless property fails from the project xml on hashlink. Theres no reason for it really.

@ACrazyTown It works fine if you set borderless ="true" in the project xml window tag for both hl and hxcpp.
We should probably disallow changing the borderless property at runtime.

Actually I think this is fixed in lime 8.2 from github, so you should be able to set the borderless property to true after the window is created.


Update protobuf library dependency.

This now automatically pull in the zlib it needs, so
no need anymore to specify that manually.
Signed-off-by: Henner Zeller<EMAIL_ADDRESS>

Codecov Report

Merging #1340 (43b7660) into master (0b02dd5) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1340   +/-   ##
=======================================
  Coverage   92.91%   92.91%
=======================================
  Files         340      340
  Lines       23686    23685    -1
=======================================
  Hits        22008    22008
+ Misses       1678     1677    -1

Impacted Files
Coverage Δ

verilog/tools/kythe/kythe_proto_output.cc
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


how to print edited text from content editable table

Hi,
Please provide suggestions to get the edited table values from content editable table.
Thanks

This is question for the slack community, not necessarily an issue with the library. Please join us at https://react-chat-signup.herokuapp.com/


Code listing 5.3-using-a-pretrained-convnet, UnboundLocalError: local variable 'batch_outputs' referenced before assignment

I am trying to run  the code list 5.3-using-a-pretrained-convnet
on databricks GPU cluster for image understanding by CNN.
The env:
TensorFlow: 2.2
python 3.7.6
keras: 2.3.0-tf
Unbuntu: 4.4
The code:
import os
import numpy as np
from tensorflow.keras.preprocessing.image import ImageDataGenerator

base_dir = '/Users/Downloads/cats_and_dogs_small'

train_dir = os.path.join(base_dir, 'train')
validation_dir = os.path.join(base_dir, 'validation')
test_dir = os.path.join(base_dir, 'test')

datagen = ImageDataGenerator(rescale=1./255)
batch_size = 20

def extract_features(directory, sample_count):
    features = np.zeros(shape=(sample_count, 4, 4, 512))
    labels = np.zeros(shape=(sample_count))
    generator = datagen.flow_from_directory(
        directory,
        target_size=(150, 150),
        batch_size=batch_size,
        class_mode='binary')
    i = 0
    for inputs_batch, labels_batch in generator:
        features_batch = conv_base.predict(inputs_batch) # error !
        features[i * batch_size : (i + 1) * batch_size] = features_batch
        labels[i * batch_size : (i + 1) * batch_size] = labels_batch
        i += 1
        if i * batch_size >= sample_count:
            # Note that since generators yield data indefinitely in a loop,
            # we must `break` after every image has been seen once.
            break
    return features, labels

train_features, train_labels = extract_features(train_dir, 2000). # error here !
validation_features, validation_labels = extract_features(validation_dir, 1000)
test_features, test_labels = extract_features(test_dir, 1000)

the call stack:
Found 0 images belonging to 0 classes.
UnboundLocalError: local variable 'batch_outputs' referenced before assignment
---------------------------------------------------------------------------
UnboundLocalError                         Traceback (most recent call last)
<command-2528158> in <module>
     33     return features, labels
     34
---> 35 train_features, train_labels = extract_features(train_dir, 2000)
     36 validation_features, validation_labels = extract_features(validation_dir, 1000)
     37 test_features, test_labels = extract_features(test_dir, 1000)

<command-2528158> in extract_features(directory, sample_count)
     23     i = 0
     24     for inputs_batch, labels_batch in generator:
---> 25         features_batch = conv_base.predict(inputs_batch)
     26         features[i * batch_size : (i + 1) * batch_size] = features_batch
     27         labels[i * batch_size : (i + 1) * batch_size] = labels_batch

/databricks/python/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in _method_wrapper(self, *args, **kwargs)
     86       raise ValueError('{} is not supported in multi-worker mode.'.format(
     87           method.__name__))
---> 88     return method(self, *args, **kwargs)
     89
     90   return tf_decorator.make_decorator(

/databricks/python/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py in predict(self, x, batch_size, verbose, steps, callbacks, max_queue_size, workers, use_multiprocessing)
   1283             callbacks.on_predict_batch_end(step, {'outputs': batch_outputs})
   1284       callbacks.on_predict_end()
-> 1285     all_outputs = nest.map_structure_up_to(batch_outputs, concat, outputs)
   1286     return tf_utils.to_numpy_or_python_type(all_outputs)
   1287

UnboundLocalError: local variable 'batch_outputs' referenced before assignment

I have created a pull request regarding the same. The code is running successfully now.


3.1.1-beta Release

Dependencies

#1261
#1283

Checklist

In your sandbox

[x] git checkout main - or whatever branch you're creating the release on
[x] git pull - make sure local repository is up to date
[x] Edit changelog.md to get it ready for release

[x] Replace the existing unreleased section with the new tag and current date
[x] Review git log v3.1.0-beta..HEAD to summarize changes relevant to users since the last release tag (i.e. no need to mention things like linting, tests, or behind-the-scenes code unless it changes the user experience).
[x] markdownlint --config .markdown-lint.yml CHANGELOG.md
[x] git commit

[x] git push - commit and push updated changelog
[x] Ensure all tests pass in GitHub actions
[x] git tag -a v[TAGNAME]

Enter a description of the release when prompted.

[x] git push origin v[TAGNAME]
[x] Follow steps to create a release on GitHub
[x] Edit changelog.md to add back the Unreleased section to changelog.md

[x] Replace the existing unreleased section with the new tag and current date
[x] Review git log v3.1.0-beta..HEAD to summarize changes relevant to users since the last release tag (i.e. no need to mention things like linting, tests, or behind-the-scenes code unless it changes the user experience).
[x] markdownlint --config .markdown-lint.yml CHANGELOG.md
[x] git commit

[x] git push - commit and push updated changelog
[x] If this is a release branch (not main), merge into main

[x] git checkout main
[x] git merge [BRANCH]
[x] git push

[ ] Deploy to tracebase.princeton.edu

See NPLC Admin Docs:TraceBase:New Release for more details

Created tag v3.1.1-beta.


NextJS page: Masthead Search bar - On hover search icon and close icon are hidden

Detailed description
Issue:
On Search bar - On hover search icon and close icon are hidden.
Additional information

Slack post from Ignacio:  I took a quick look and it seems the carbon-component styles are being loaded later, so they override our previous fixes that were done for the upgrade. It may have to do with chunking, or the styles changes may possibly need to have a more specific selector overall. Not sure if it’s NextJS specific or it may affect other sites.

reopened because tooltip fix is not included in the merged PR.


Faulty solution animation of problems using new condition seriescapture

Example:
8/8/8/8/2K5/1pnp4/p7/k7
sti =1
cond seriescapture
Popeye wasm-32Bit v4.89 (512 MB)
1.Kc4*b3*c3*d3-c2 = !

1.Kc4*d3*c3*b3-c2 = !

solution finished. Time = 0.020 s
I think this is due to spaces being present before or after the "*" in the solution.

I think this syntax was just introduced in 4.89. Added the support just now.


"No matching handler for type Int8Array"

Hi there!.. This looks like exactly what I have been looking for.  I am having an issue getting spasm to compile, however.  I've read your note that 1.13.0-beta2 is the necessary version and I've updated to that using the install.sh (I am on linux mint).. when I run dub from the project root I get the following, however:
spasm 0.1.6: building configuration "library"...
source/spasm/sumtype.d(764,13): Error: static assert:  "No matching handler for type Int8Array"
source/spasm/sumtype.d(629,14):        instantiated from here: matchImpl!(const(SumType!(Int8Array, Int16Array, Int32Array, Uint8Array, Uint16Array, Uint32Array, Uint8ClampedArray, Float32Array, Float64Array, DataView)))
source/spasm/sumtype.d(314,15):        instantiated from here: match!(const(SumType!(Int8Array, Int16Array, Int32Array, Uint8Array, Uint16Array, Uint32Array, Uint8ClampedArray, Float32Array, Float64Array, DataView)))
source/spasm/bindings/common.d(4,25):        instantiated from here: SumType!(Int8Array, Int16Array, Int32Array, Uint8Array, Uint16Array, Uint32Array, Uint8ClampedArray, Float32Array, Float64Array, DataView)
/home/wolfie/dlang/ldc-1.13.0-beta2/bin/ldc2 failed with exit code 1.

any advice would be appreciated... thanks

Please accept my apologies.. I just switched to: ldc-1.14.0-beta1 and was able to compile without any issues.  I'm closing the ticket... but I'm sure I'll be back.. this project is great.

Cool. Can I have a look at the code that didn't work with ldc 1.13.0?
The bindings are generated and I haven't tested everything against the browser, so there will be some places that need to be fixed.
Specifically, some web host objects like Sequence, Int8Array, ReadableStream, Iterator and some others (basically everything from https://github.com/skoppe/spasm/blob/master/source/spasm/types.d#L319 down to ArrayPair) are currently opaque objects that don't have any methods on them. Sadly I cannot find any webidl files for them so I implement them whenever I need them.

It was failing to compile spasm itself.  So I cloned your repo and tried to execute dub in the project root and that failed with the above.  Anything that depended on spasm failed the same way (I tried fetch)... but before that I built the wasm.wasm example on the DLang Wiki without any trouble.. obviously it's a lot simpler.... Today is my first day ever looking at DLang but I spent the past three days trying to do fetch in Rust and failed miserably... I'm thinking Moz has really gone off the rails on that one...   My intention is to select a wasm friendly language for a new product I'm developing and so far Dlang + spasm (great name BTW) are in the lead.  I've pretty much tried everything that compiles to wasm..

Have you looked at this?
https://github.com/Microsoft/TSJS-lib-generator/
That's what M$ uses to generate the TS bindings.

Today is my first day ever looking at DLang but I spent the past three days trying to do fetch in Rust and failed miserably...

Even so, there are a lot more people working on Rust+wasm than on this project. So while I am happy that you want to build things with spasm, I do have to manage your expectations a little.

I'm thinking Moz has really gone off the rails on that one... My intention is to select a wasm friendly language for a new product I'm developing and so far Dlang + spasm (great name BTW) are in the lead. I've pretty much tried everything that compiles to wasm..

Spasm can certainly do that, but it does require extra effort on your part beyond the 'it-should-just-work-out-of-the-box'.

Have you looked at this?
https://github.com/Microsoft/TSJS-lib-generator/

It seems they have some extra idl's that I don't have it. Will include them.

@skoppe

Even so, there are a lot more people working on Rust+wasm than on this project. So while I am happy that you want to build things with spasm, I do have to manage your expectations a little.

I appreciate you saying that but I do not expect things to JustWork(tm) where WebAssembly is concerned and I'm also not entirely convinced that "more people" working on software necessarily makes it better though I am hopeful that my efforts will be of value to you and this project eventually.

Spasm can certainly do that, but it does require extra effort on your part beyond the 'it-should-just-work-out-of-the-box'.

Yeah I don't expect anything to work-out-of-the-box especially bleeding edge stuff like this.
I'm sure you can tell by now that I'm willing to put in some effort.  The piece I wanted to solve in Rust that was so daunting was a slightly modified version of your fetch example which I have reproduced in my fork of this project in examples/animated.  It is a spasm / D implementation of the Rust code that I couldn't get to work. (below)
import spasm.bindings;
import spasm.dom;
import spasm.types;

void get_shaders(void delegate(string) callback){
    auto promise = window.fetch(RequestInfo("https://reqres.in/api/users/2"));
    promise.then(r => r.json).then((data){
        callback(data.as!(Json).data.avatar.as!string);
    });
}

/** Export to JS Side */
extern (C) export void _start(){
    get_shaders((data) => console.log(data));
}

The above works as expected although I did have to add a function to the jsExports object and I don't know exactly what it is supposed to do or why it was being called:
let jsExports = {
    _d_allocmemory: () => { console.log("_d_allocmemory")}, /** TODO @ Wolfie -> What's this for? */

The animation project (which will be different than it is now) dumps it's compiled wasm into:
bootstrap-vanilla/webRoot
While I think webpack has a lot of very good use cases I find that during development it makes it difficult for me to understand what's happening... So I implemented this little web server that I wrote a while back in TS for testing and used ES6 import to load your modules rather than webpack to bundle it up into traditional modules...
That aside here's the rust  code that I could not get to work:
fn get_shaders(_callback: &'static Fn(JsValue)) {
    let fetch = & | _val: JsValue | {
        _callback(_val);
    };

    JsValue::get_global("wolf")
        .call_method_with_args("fetch", &[&"shaders/cube.glsl", &wrap(fetch)]);
}

fn init_webgl() {
    shaders(&|_val: JsValue| {
        let console_ = JsValue::get_global("console");
        console_.call_method_with_arg("log", _val);
    });
}

It blew up with some lifetime business that was difficult to understand.  Even when I went to the gitter chat rust forum one of the guys say "yeah that's a tricky one" and I was thinking... I just want this to call back when the fetch is done...  I don't know rust or D but was able to make it work in D without any issues.
Finally please take a look at the commits to my fork, which is up-to-date with your latest changes here:
https://github.com/WolfieWerewolf/spasm
Let me know if that stuff is acceptable to you and I'll submit a PR.  Or if there are things you would like me to change I can do that first, whatever you think?

Yeah I don't expect anything to work-out-of-the-box especially bleeding edge stuff like this.

Great. And I do appreciate you trying this out.

It blew up with some lifetime business that was difficult to understand. Even when I went to the gitter chat rust forum one of the guys say "yeah that's a tricky one" and I was thinking... I just want this to call back when the fetch is done... I don't know rust or D but was able to make it work in D without any issues.

Ai, that Rust code looks awful. Specifically the part where it is calling functions by string. Go has that as well I believe.

Finally please take a look at the commits to my fork, which is up-to-date with your latest changes here:
https://github.com/WolfieWerewolf/spasm
Let me know if that stuff is acceptable to you and I'll submit a PR. Or if there are things you would like me to change I can do that first, whatever you think?

I don't mind having different bootstrappers, in fact it's one of the reasons I called the first one bootstrap-webpack. But to understand whether your changes would be accepted, consider this: the bootstrap sub-package is meant for end-users so that they can easily setup/build the required js glue code. End-users should be able to run a simple dub run spasm:bootstrap-xxx which should copy required js glue code, copy an index[.template].html and whatever required config for that flavor of bootstrapping. Furthermore, after bootstrapping, there should be convenient commands to create a production build for js/index (bundled, minified) and to run a dev-server. Those are the only criteria.
A quick glance over your commits showed your bootstrap-vanilla only does a dev-server, but I might be mistaken.

The above works as expected although I did have to add a function to the jsExports object and I don't know exactly what it is supposed to do or why it was being called:
let jsExports = {
    _d_allocmemory: () => { console.log("_d_allocmemory")}, /** TODO @ Wolfie -> What's this for? */

I don't know. It seems similar to the array_cast thing you got with underrun. You still at ldc 1.14? What command do you use to compile?

From a fresh bash I do this:
source ~/dlang/ldc-1.14.0-beta1/activate
dub build --compiler=ldc2 --build=release --force
subsequently I just execute the dub line...

@skoppe

A quick glance over your commits showed your bootstrap-vanilla only does a dev-server, but I might be mistaken.

You are mostly correct..  It provides a web server but it also loads your existing boilerplate using ES6 imports rather than generating / consuming a UMD module (which is what I believe webpack is producing).  And yes, the intention is to build it out so that it works for end users but I wanted to get your blessing to go ahead and do that before putting a lot of work into it.  I've arrived at Open Source projects in the past where they seemed welcoming at first and after I did a bunch of work and submitted a PR they said "no thank you"... so I wanted to understand your requirements before going all the way with this.
So if we are in agreement does it make sense to move the JS boilerplate stuff out of the bootstrap-webpack to a common location that can be shared by any bootstrapper?

If you think it provides additional value, I'll welcome it. As long as it fulfils the above mentioned criteria.

From a fresh bash I do this:
source ~/dlang/ldc-1.14.0-beta1/activate
dub build --compiler=ldc2 --build=release --force
subsequently I just execute the dub line...

Ok. I'll try with 1.14 as well. Probably update the CI to run both 1.13 and 1.14
BTW, you can loose the --force flag. You don't need it except in exceptional cases, and it slows down compilation a lot.

Thanks mate..


Feature/twitter access token refresh

We can now use puppeteer to automatically go through the oauth authorization flow. However, it only seems to work if we run puppeteer NOT in headless mode (you will see a browser window open up and the information being put in to the form). It also takes ~10 seconds in total.

You also need to store use credentials in the .env.test file, like so:
TEST_USER_TWITTER_CREDENTIALS={"sense_nets": {"user_id": "1742000911797366784", "username": "sense_nets", "password": "PASSWORD-HERE"},"sense_nets_bot": {"user_id": "1773032135814717440", "username": "sense_nets_bot","password": "PASSWORD-HERE"}}

@pepoospina I am unable to run the tests fully since I am having issues with twitter developer account authentication (see discord message). But if all the tests pass I'd say this is ready to merge.
Note that I consolidated some environment variables, so now it expects TEST_USER_TWITTER_ACCOUNTS
which looks like:
TEST_USER_TWITTER_ACCOUNTS=[{"user_id": "1773032135814717440", "username": "sense_nets_bot","password": "PASSWORD"}, {"user_id": "1742000911797366784", "username": "sense_nets", "password": "PASSWORD"}]

I included the user_id in the end so that we can use just this environment variable when created test app users (which need a userId). I no longer use TEST_USERS_BEARER_TOKENS

This is also set up to be able to authenticate any number of twitter account that is specified in the environment variable along with the NUM_TWITTER_USERS constant.

@weswalla, maybe we can change a bit the structure so that the authenticateTwitterUsers is renamed as authenticateUsers and, instead of returning the TwitterUserDetails[], it returns already an array of type AppUserCreate[].
This way we handle entire user profiles and not just twitter credentials as the thing that is computed and stored on a file.
WDYT?

@weswalla, Also, I suggest we keep the file's naming convention to lowercase with dots :)

Sure @pepoospina that all sounds good to me, I wasn't clear on conventions so just went ahead with this approach knowing we would change it with some feedback. Happy to use that file naming convention.

@pepoospina I have integrated your feedback. The only things that remain (which could be handled in another PR) are:

[ ] granting write access to a user before running any tests requiring write access


Improve PowerShell code generation with Here-Strings

PowerShell code response generation could be improved by using here-strings when appropriate. Here's an example: I asked GitHub Copilot Chat to implement code for a REST API call - it generated the following fragment for the Body:
$body = "grant_type=client_credentials&client_id=$appId&client_secret=$appSecret&resource=$resource"
I used the same prompt with Bing Chat, and it provided the following (more readable) code for the Body:
$body = @{
    "resource"=$resourceUrl
    "client_id"=$appId
    "client_secret"=$appSecret
    "grant_type"="client_credentials"
}

Stylistically, the Bing Chat response is more easily read by a human (each part of the string on a separate line) and generating the $resourceUrl variable name vs the $resource variable name generated by GitHub Copilot Chat makes it clearer that the value should be a URL.

Seems like Bing grounds the answer in more modern PowerShell then. We can track this as a test case and see if this is about prompting or requires grounding in language docs.


Add Mnist Demo

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

Could you add localization to this .NET program?

@shishaochen Hi, thanks for review. The current default language of this program is English.


Methods returning new objects

How do we deal with methods that return new objects? For example Vector.c() and (probably) ParallelContext.gid_connect(i, cell).

I think the underlying issue is that we need to increment the returned object's refcount. I think the reason we didn't run into this before was that the refcount for things that already existed was non-zero and so they weren't getting garbage collected on the C-side. Incrementing refcount on return may also fix the underlying issue that led to #68 where MATLAB garbage collection was causing objects to go away when they shouldn't (because the refcount wasn't right).
For an example with Vector.c(), see https://github.com/mcdougallab/neuron-c-api-demos/blob/main/vector-dot-c/vector-dot-c.cpp The main function of interest is: call_object_method_that_returns_object
In particular, commenting out the new_ob_ptr->refcount++; line will lead to the Vector getting garbage collected and not being useable for printf.

Fixed in PR #47


AppImage build using CMake

#1408
@mcallegari I already tested on my side. Please review this PR.

coverage: 28.053%. remained the same when pulling 3af53ca549bf265747ebca31ab2d0487d1356fde on Itay-Lifshitz:cmakebuild into 611817918d0aa66a550c8f94df687edef34b9369 on mcallegari:cmakebuild.

@Itay-Lifshitz thanks but I found this error if I use system Qt libraries:
CMake Error at platforms/linux/cmake_install.cmake:102 (file):
  file INSTALL cannot find "/usr/lib/lib/libQt5Core.so.5": No such file or
  directory.
Call Stack (most recent call first):
  platforms/cmake_install.cmake:47 (include)
  cmake_install.cmake:97 (include)

Can you please have a look?

@Itay-Lifshitz  any news on this?
I will test cmake on macOS but I think the current status is already good.
Thanks

@Itay-Lifshitz any news on this? I will test cmake on macOS but I think the current status is already good, so merging on master could be near. Thanks

Hi, sorry for delay, I have been busy this week.
I will update by this weekend.
And I think current CMake is not ready for MacOS, because I have never tested due to lack of environment.


chore:

- update GHA version and README version
- add dependnecybot in GHA
- fix lint in src/*
- change from zeit/ncc to vercel/ncc
- set version in package reflect to marketplace version
- update target name in workflow

Ref changes in aqa-tests https://github.com/adoptium/aqa-tests/pull/2203/files

LGTM - Could perhaps do with a full usage example in the README.md but that can be added later.

yeah, was thinking we are having an incoming PR for "parallel" so we can add more in that PR

Actually, looks like some of the GH Action tests failed.

I will take a look
the nightly seems have been failed for a while too (but guess that's more problem from the aqa-tests changes)

think the failure is we defined in jdk11 to  target: '_DaaLoadTest_daa1' but we do not have DaaLoadTest_daa1 any more.

2019 still not happy...

2019 still not happy...

seems I do not have options to rerun these 2.
wonder what happened, if functional can pass why system and openj9 build could fail.....

[echo] D:/a/run-aqa/run-aqa/aqa-tests//system/STF does not exist, clone from GitHub
     [exec] error: could not lock config file /home/runneradmin/.gitconfig: No such file or directory      [exec] Result: 255      [echo] Using a git ls-remote command to determine if the jdk source repo has an available 'u' version yet.

I start to think it ever worked on windows before.
did a check all repos in GH using "adoptium/run-aqa" they only running on ubuntu e.g https://github.com/eclipse-openj9/openj9/tree/master/.github/workflows
or our aqa-tests just skip the check https://github.com/adoptium/aqa-tests/blob/master/.github/workflows/directoriesFilesChangePR.yml
as in temurin-build, in order to use run-aqa, there are steps to set up cygwin and install git.
https://github.com/adoptium/temurin-build/blob/master/.github/workflows/build.yml#L260

I can only think the whole story:
in the past we use windows-2016 (when it is still avaiable) which does not set WSL enabled by default.
so the cygwin with git works.
then windows-2016 is not supported in GHA, instead of 2019 or latest/2022 which has WSLv1 enabled by default.
so when ant is building, tasks like "git" is seen as the bash git not git.exe from cygwin.
the error is not seen on in:

temurin-build, because there is no need in GHA to "git clone" functional code
openj9 repo, because it does not run on windows only ubunut-latest
aqa-tests, directoriesFilesChangePR.yml has been skipped way back for months

discussed in the meeting, so the reason we kept to 2016 was because there was certain things not working on 2019+
so i split windows out from non-windows in GHA to have a better way to view the result.
maybe @renfeiw can do a review this PR as well?

Hi @zdtsw I have added @sophia-guo as a reviewer, she will have more knowledge on these files.

Thanks @zdtsw - I have added some comments, but not to be considered a full/final review. There seem to be a lot of reorganizing of code in dist/index.js, remind me, is this file generated or is that conscious (perhaps an overactive) refactoring effort?

the index.js is the one generated by "npm run build" or "npm run all" which action.yaml is using it as the entry.
so the changes in src/* will generate new index.js

Did you run the tests in your local repo? What tests have you run?

@sophia-guo - tagging you as reviewer here
@zdtsw - please change the title of this PR to something more meaningful

Jobs are parallel running on different runners so split windows out from non-windows won't help to have a better view ( always two jobs on windows latest fail) . So maybe just directly update windows 2016 to latest. We can later find a way to fix it? I think  @j-braley hit the same issue and mentioned the solution and I just forgot. I can check the slack message to see what it is.

Jobs are parallel running on different runners so split windows out from non-windows won't help to have a better view ( always two jobs on windows latest fail) . So maybe just directly update windows 2016 to latest. We can later find a way to fix it? I think @j-braley hit the same issue and mentioned the solution and I just forgot. I can check the slack message to see what it is.

yeah, the reason to split it to windows and non-windows is not to fix the issue with windows2019/windows2022/windows-latest but only to have an easy way to see the non-windows is "green" since we are certain things are not working with windows at the moment.

ping @sophia-guo for review too

windows error is not related with this PR. We will open an separate issue to investigate the issue with windows.

Actually, looks like some of the GH Action tests failed.

https://github.com/adoptium/run-aqa/issues/136 will be handled for windows part

@karianna windows failure is not related with PR https://github.com/adoptium/run-aqa/pull/131#issuecomment-1240984227. Will just merge this one and open a separate issue for windows failures.


Cannot get ByteArrays as an array of ints

We have been testing pvws/dbwr with an EPICS waveform PV of type CHAR/UCHAR, e.g.
record (waveform, "test:message")
{
    field (NELM, "100")
    field (FTVL, "CHAR")
}

In Phoebus we can display this PV as a string or as an array of integers, i.e.:

However when we display the same BOB file in dbwr we notice different behaviour and cannot display it as an array of ints:

I took a look at the pvws response and see that no information on the underlying array is sent back - only the converted string is sent.
I forked and created a branch with a potential fix for this, which is basically to also handle ByteBuffer types as integers as well as a string. See https://github.com/ornl-epics/pvws/compare/main...rjwills28:pvws:char_waveform_fix. I can create a PR if this is agreed as a suitable solution?

OK. Does this only require a change to the PVWS, or do you also need to update the 'format' handling in the DBWR?

When the format for the widget is set to decimal in Phoebus, then DBWR will already display the PV correctly as an array of ints. However there is a slight difference in behaviour between Phoebus and DBWR when the format is left as default; Phoebus displays it as an array while DBWR displays it as a string. See example below where I have tested with my proposed fix and note the format: default case.
Phoebus:

DBWR:

If DBWR should follow the behaviour of Phoebus then I think a change is also required in DBWR? I can create an issue on the DBWR repo.

See #21: All byte arrays are sent as strings. On the client end (DBWR) we thus no longer know if the data was really a string, or a char array used to avoid string length limitations.
Fix would be to add a new  "b64byt" data type that sends byte arrays as, well, byte arrays. The client can then by default show them as such, or format as string when the format option is set as such.

Maybe start by handling #21, add "b64flt", "b64srt" and then "b64byt" for this.

By adding b64byt to PVWS and then handling fomat='string' in DBWB, this should be fixed.
Thanks!


TextField Paste option not work web mobile android chrome html render

expected ios safri:

android chrome:

why dont show paste option??
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.3.9, on macOS 12.6 21G115 darwin-x64, locale ko-KR)
[!] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
✗ cmdline-tools component is missing
Run path/to/sdkmanager --install "cmdline-tools;latest"
See https://developer.android.com/studio/command-line for more details.
✗ Android license status unknown.
Run flutter doctor --android-licenses to accept the SDK licenses.
See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.1)
[!] Android Studio
✗ Unable to find bundled Java version.
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
[✓] VS Code (version 1.71.2)
[✓] Connected device (1 available)
[✓] HTTP Host Availability
! Doctor found issues in 2 categories.
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',

      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(child: TextField()),
    );
  }
}

Closing as a duplicate of #98342. Please follow up on it for further updates.


Expand the coverage of keywords and operators

Hi,
Could you please expand the supported tokens for keywords and operators by syncing it with the Atom Syntax for metapost available here?
Thanks in advance

Hi @santhoshtr! Wow, thanks for the link to that repository, that somehow flew entirely under my radar! I will sync the wordlists later today :)

Hi, a gentle ping :-)

Thank you.


os-ddclient 1.20_2 - ddclient updates GoDaddy DNS entry every 300s even if dynamic IP has not changed

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] The title contains the plugin to which this issue belongs

Describe the bug
I just changed my ddclient from using FreeDNS to using GoDaddy. The dynamic ip has been checked properly (via Interface/WAN) and the respective GoDaddy DNS entry has been updated correctly. However, ddclient now seems to "statically/periodically" update the GoDaddy DNS entry every 300 seconds (as configured in ddclient's general settings), even if the dynamic ip has not changed.
ddclient using GoDaddy now shows a different behavior compared to using FreeDNS. Using FreeDNS, the DNS entry was only updated if the dynamic IP had been changed by the provider.
To Reproduce
Steps to reproduce the behavior:

Within ddclient create a new account
Set "Service" to "GoDaddy"
Enter "Username", "Password", "Zone", "Hostname"
Set "IP check method" to "Interface"
Set "Interface to monitor" to "WAN"
Set "Check ip timeout" to "10"
Set "Force SSL" to "enabled"
Enable account.

Expected behavior
Initially, the dynamic IP should be checked, and the "Hostname" should be updated once. After the initial update of the "Hostname" ddclient should continuously check if the dynamic IP has been changed by the provider and only update "Hostname" in case it has been changed.
Relevant log files
Date
Severity
Process
Line
2024-02-07T10:30:24	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T10:25:23	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T10:20:21	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T10:15:19	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T10:10:17	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T10:05:15	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T10:00:13	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T09:55:12	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T09:50:10	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
2024-02-07T09:45:08	Notice	ddclient	SUCCESS: dxxxxxxxxxxt -- Updated successfully to 8xxxxxxxxxx2 (status: 200).
Environment
OPNsense 24.1.1-amd64
FreeBSD 13.2-RELEASE-p9
OpenSSL 3.0.13
AMD GX-412TC SOC (4 cores, 4 threads)

Any comments?

Sure, have you looked for upstream bugs? https://github.com/ddclient/ddclient/issues

To my understanding the "caching/spamming" issue has been fixed with the release of v3.11 in October 2023. The actual version is v3.11.2.
https://github.com/ddclient/ddclient/issues/552
So, if the issue has been officially fixed on the ddclient side, maybe my issue is related to its implementation (plugin) into opnsense?

If the bug is that we start the service script perhaps, but I doubt that we have influence on the internal renew timer or the way it handles these renews.
https://github.com/ddclient/ddclient/commit/f5a1a906d1cd53665490203252c116a7762d1121 was committed two weeks ago. 3.11.2 was released last year? ;)

Well, I have no idea about how the plugin is implemented. But if I have an issue I start with the provider of the plugin. At least according to my logic.

Again, the relevant commit is only 2 weeks old.

Have you read the release notes of your own plugin os-ddclient? The release notes of your plugin tell me, that os-ddclient v1.17 has been updated to ddclient v.3.11.1. The actual version of your os-ddclient is v1.20_2. So?

So, after spending most of your community time with me on trying to prove me wrong you could now finally agree to the fact that the officially fixed version v3.11.2+ (latest release + additional fix) of ddclient is part of your actual plugin os-ddclient v1.20_2. One step forward. Sorry for my sarcasm.
What next? Do you investigate the issue as the author of the plugin? Or do I need to go upstream and address the issue with ddclient?

You could be productive and see that commit I posted is actually in the software you complain about which isn't us. If it isn't apply the patch and tell me if it works.
That would be an ideal way to stop wasting my time by acting like an entitled child. :)

See it as a chance for self-reflection. Have you ever thought about the fact that not everybody trying to contribute here - like me - is a technical expert navigating the depths of Github with ease?  Because I have no idea about what you are suggesting.

See it as a chance for self-reflection.

Yeah, I couldn't agree more.
To be frank I'm unsure what you expect of me and why you feel the need to be abusive towards me (and probably others). I've nothing but helped you and answer your questions.
I'm not here to test upstream changes that might be relevant to you. I'm not here to take a bit of abuse from a random reporter. In fact don't mistake my replies for anything else but honest help, but feel free to still tell me I'm the only problem.
I might be your problem right now, but that's only because you don't know how to solve it. But you are too tough to even ask for help. ;)

Yeah, I couldn't agree more.

Obviously not.

To be frank I'm unsure what you expect of me and why you feel the need to be abusive towards me (and probably others). I've nothing but helped you and answer your questions.

Seriously? Ok, so I also add "abuser" to the list. If it suits you... ;-)

I might be your problem right now, but that's only because you don't know how to solve it yourself. But you are too tough to even ask for help. ;)

If I could have solved the issue by myself, I wouldn't have opened this issue. And the opening of an issue imho implies asking for support or help. Sounds logical? However, supporting implies to noticeably put yourself into the shoes of the one supported, rather than to simply deflect the issue. Otherwise, it is not supporting, it's gatekeeping. And that's how I experienced it.

I think you are simply asking a lot in the worst way possible. We would have already solved this if you followed my hint to the commit which I still will not do for you for at least the reason that I don't have a GoDaddy account to test this. But it is what it is.

I think you are simply asking a lot in the worst way possible.

Then we share at least a comparable experience. As sad as it is.

We would have already solved this if you followed my hint to the commit

I still don't have a clue what to do with the two committed files ddclient.conf.in and ddclient.in.

There is a likely fix for this issue: https://github.com/ddclient/ddclient/pull/670. Unfortunately, I could not test it, as GoDaddy recently updated their account requirements to access parts of their production Domains API and I have lost access.

Since a lot of changes are piling up for 3.11.3 on their end I'd rather wait for an official release package.


Blurry Text in Chrome

Sometimes the text is blurry in Chrome because of the problem with translate3d:
https://bugs.chromium.org/p/chromium/issues/detail?id=573146
https://martinkool.com/post/27618832225/beware-of-half-pixels-in-css

Fixed with #38
Fixed on master (v3.0.1).


MKR1000 and MKR WiFi1010 - EEPROM

Hello,
As indicate in the Readme file, to use the MKR familly, I commented the two lines :
//#define EZTIME_CACHE_EEPROM
// #define EZTIME_CACHE_NVS
What is the impact of not using the epprom ?

My code can use either EEPROM.h or Preferences.h, the MKR comes with FlashStorage.h, which is a different interface to store values in EEPROM. You don't need to use the timezone info cache that is all this is used for. It is not as relevant as it used to be as we longer use timezoneapi.io but our own timezoned info service which does not have a set maximum number of requests.


Add OpenTelemetry Kubernetes tags.

Relevant information
Added K8s tags for OpenTelemetry services as defined here: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/resource/semantic_conventions/k8s.md
Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an
explanation above.

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

@fryckbos is this one still relevant?

Closing. Feel free to reopen if needed


ToRef types Error

Version
3.2.26
Reproduction link
about:blank
Steps to reproduce
function setup<T>(props: { value: T }) {
   const value = toRef(props,'value')
}

What is expected?
typeof value.value should be V
What is actually happening?
typeof value.value is any

may be change defination
- type ToRef<T> = [T] extends [Ref] ? T : Ref<T>;
+ type ToRef<T> = [T] extends [Ref<T>] ? T : Ref<T>;

Do you have an actual reproduction? Because this seems to work:
defineComponent({
  props: {
    value: String
  },
  setup(props) {
    const a = toRef(props, 'value')
    a.value
  }
})

Do you have an actual reproduction? Because this seems to work:
defineComponent({
  props: {
    value: String
  },
  setup(props) {
    const a = toRef(props, 'value')
    a.value
  }
})

You can try this
import { defineComponent, PropType, toRef } from "vue";

export const MyComponent = <T extends string | number>() => {
  return defineComponent({
    props: {
      value: null as unknown as PropType<T>
    },
    setup: (props) => {
      const value = toRef(props, 'value')

      return () => <>{
        value.value
      }</>;
    }
  });
};


BSOD on Windows 7 x64 with Kernel -> ManualMap Injection

Just an FYI, maybe you are not aware... but I tested on numerous Win 7 x64 machines to inject x64 process using Kernel -> ManualMap and every time I got BSOD.
The driver is compiled in release mode, using the latest SDK/WDK etc.
Could I provide you something that would help in resolving this issue ?
Many thanks

BSOD memory dump and blackbone driver binary + pdb symbols will help.

Yes will do, right now I'm finishing something urgent. PS: In Debug mode the driver does not crash... o.O


Perf of Vector.of(T element)

What is the rationale behind Vector.of(T element) method? It performs terribly compared to its varargs cousin Vector.of(T ... element), which builds a trie directly from the elements array.

I see. Currently we have:
public static <T> Vector<T> of(T element) {
    return ofAll(Iterator.of(element));
}

The following solution might perform better (needs to be verified):
public static <T> Vector<T> of(T element) {
    return ofAll(BitMappedTrie.ofAll(new Object[] { element }));
}

Any volunteers for creating a PR?

Pull request here https://github.com/vavr-io/vavr/pull/2659

Another word about the rationale behind Vector.of(T): you are right, of(T) exists on the API level, because it should prevent creating an array, like in the varargs factory method of(T...). In our case of(T) is not really needed, because currently we internally wrap the element into an array. However, I would keep the API, the implementation might change/might be improved over time.


Q App Clean-Up

Now that the HTML for the Q calculator is generated by script, I want to take another look at all the CSS and classes I have sprinkled everywhere. Some is necessary for the JavaScript to work, but I bet not all.
And the HTML is huge, so it might benefit from shorter CSS variable names.

Next, I want to shorten all these CSS class names.


[WIP] Ingest Partition Markers

When trying to get the next scheduler e2e tests working, I immediately realised that I hadn't persisted the partition markers to the DB (doh!).  This PR addresses that.  Summary is:

Partition Markers are now EventMessageEvents so that they can be handled by the SchedulerIngester.
Code In Scheduler Ingester to insert these into the db.
Code in other event handlers to ensure these messages are ignored without triggering warnings etc.

Codecov Report
Base: 46.80% // Head: 46.86% // Increases project coverage by +0.06% :tada:

Coverage data is based on head (b1dcab6) compared to base (92fcbb6).
Patch coverage: 81.48% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2068      +/-   ##
==========================================
+ Coverage   46.80%   46.86%   +0.06%
==========================================
  Files         229      229
  Lines       30186    30234      +48
==========================================
+ Hits        14129    14170      +41
- Misses      15012    15019       +7
  Partials     1045     1045

Flag
Coverage Δ

airflow-operator
89.13% <ø> (ø)

armada-server
45.85% <81.48%> (+0.06%)
:arrow_up:

python-client
93.83% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ernal/lookoutingester/instructions/instructions.go
70.20% <ø> (ø)

...nal/lookoutingesterv2/instructions/instructions.go
75.95% <0.00%> (-0.19%)
:arrow_down:

internal/scheduler/database/query.sql.go
28.44% <0.00%> (-0.27%)
:arrow_down:

internal/scheduleringester/dbops.go
79.61% <66.66%> (-0.79%)
:arrow_down:

internal/scheduleringester/schedulerdb.go
59.71% <75.00%> (+2.46%)
:arrow_up:

...ernal/armada/repository/apimessages/conversions.go
82.09% <100.00%> (ø)

internal/scheduler/publisher.go
84.42% <100.00%> (+2.18%)
:arrow_up:

internal/scheduleringester/instructions.go
69.95% <100.00%> (+1.63%)
:arrow_up:

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.


Make Quick Mode more minimalistic, à la Google Maps

The app would keep scanning all the time.
We could display the info at the bottom of the screen, in a edge to edge card, that you could swipe up to get more info.
You could add some action in the blue button. You could swipe up to go to full product view.
This would make quick mode feel more powerful, less intrusive, almost like AR.

http://stackoverflow.com/questions/34310530/sliding-bottomsheet-like-google-map
https://github.com/umano/AndroidSlidingUpPanel
http://stackoverflow.com/questions/21826892/replicate-google-maps-bottom-panel-swipe-up

@teolemon I will work on this

@teolemon I am working on this I will make a PR soon

@teolemon does the background need to be white or transparent , I think transparent background would be more appealing like below and you said when we swipe up it need to display more info does that mean the whole info we used to display in view pager.

@teolemon can you also tell me what information and in which layout should we display information when user completely swipes up the pannel and I also put link to complete info when user clicks on that dialog_info icon.

I think a white background is preferable, because otherwise, you'd swipe up invisibility :)
I guess swiping up should reveal the full product view

Implemented, only the swipe up is missing. Closing


BBOX Padding feature request

To handle scenarios requiring segmentation of long hair.

hi. sorry i dont quite get the idea. currently the FaceBBoxDetect has a dilation parameter which controls how big the bbox is padded. can this meet your need?


Use common module DscResource.Common

Pull Request (PR) description

NetworkingDsc

Updated to use the common module DscResource.Common.

This Pull Request (PR) fixes the following issues
None.
Task list

[x] Added an entry under the Unreleased section of the change log in the CHANGELOG.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md in resource folder.
[ ] Resource parameter descriptions added/updated in schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[x] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

@PlagueHO this one is ready for review now.


Error from generated cucumber.yml file

I am using rails 5.0.6, ruby 2.2.5p319, cucumber 3.1.0
I set up the cucumber-rails gem with bundle and rails generate cucumber:install.  The following cucumber.yml was generated:
<%
rerun = File.file?('rerun.txt') ? IO.read('rerun.txt') : ""
rerun_opts = rerun.to_s.strip.empty? ? "--format #{ENV['CUCUMBER_FORMAT'] || 'progress'} features" : "--format #{ENV['CUCUMBER_FORMAT'] || 'pretty'} #{rerun}"
std_opts = "--format #{ENV['CUCUMBER_FORMAT'] || 'pretty'} --strict --tags ~@wip"
%>
default: <%= std_opts %> features
wip: --tags @wip:3 --wip features
rerun: <%= rerun_opts %> --format rerun --out rerun.txt --strict --tags ~@wip

After setting up my features and step definitions, when I ran cucumber features, I received the following error.
Failed describing child of #<Cucumber::Core::Ast::Feature "[my feature]" (features/my_feature.feature:1)> - Empty stack (RuntimeError)
I changed the cucumber.yml file to
std_opts = "--format #{ENV['CUCUMBER_FORMAT'] || 'pretty'} --strict"
%>
default: <%= std_opts %> features

and all works nicely.
I'm raising the issue to:

highlight a fix to others who have a similar issue
ask if the initial install can generate a different cucumber.yml

@margOnline I came across this exact issue and wasn't sure what the problem was. However, your suggestion fixed it! Thanks

I ran into this issue on a Rails 4.2 app today after upgrading to cucumber-rails v1.6.0 - it was easily resolved by re-running bundle exec rails g cucumber:install and opting no for files that I'd modified, as suggested in the Readme file.
If this doesn't help, could you please create  a MCVE demonstrating this issue and post it here, so that we can take a look?

Hi @margOnline - Just doing a bit of a tidy and this issue is nearly a year old. Are you still having the issues mentioned after the suggestions from :point_up: ??
If you're no longer experiencing these issues we can close this issue off.

If you're no longer experiencing these issues we can close this issue off.
I have not had the opportunity to install cucumber to an app since I'm working on existing apps.

In an effort to tidy things up I'll close this off.
If people still have similar issues they can re-open a new ticket.


Ignore auxiliary files from \usepackage{cprotect}

cprotect tracks its contents in an auxiliary file, this is safe to ignore.

Thanks!


Analyse extensions and find bad usages of vscode

Just by accident, I observed the Powershell extension was using vscode as a regular dependency instead of as a dev dependency. By moving vscode to a dev dependency, they have reduced their install size from 62.1 MB to 12.9 MB
I suggest we do the following:

Scan for existing extensions in the marketplace and create issues/PRs
Add a lint check in the extension-editing extension that is bundled with VS Code.

@bpasero Sorry, I have no idea

By moving vscode to a dev dependency

This has been done and vsce is now checking whether vscode is dependency or a devDependency, closing.


[UPDATE] Bump LND to v0.17.3

What
Updates LND guide to newly released v0.17.3
Why
Keeping up with latest developments and improvements (see release notes)
How

Updated variable environment and outputs

Scope

[x] significant change to core configuration
[ ] independent bonus guide
[ ] simple bug fix

Test & maintenance
Tested for myself. Working fine ✅

tACK amd64

Why are we hurrying so fast with merges? This doesn't look so critical. It's better to let more people review too, just in case.

Why are we hurrying so fast with merges? This doesn't look so critical. It's better to let more people review too, just in case.

Why is this different for you? https://github.com/raspibolt/raspibolt/pull/1366#pullrequestreview-1687850813
I'm not a fan of the utACK, btw

Why are we hurrying so fast with merges? This doesn't look so critical. It's better to let more people review too, just in case.

Why is this different for you? #1366 (review)
I'm not a fan of the utACK, btw

Configuration / documentation change is more easy to rollback than software update in case something goes wrong. It just gave hint what to do in case of specific error, didn't change core software or even core config.


[QUESTION/FEATURE REQUEST] Include margins in flip and other middleware (offsetHeight/offsetWidth)

We are building a design system based toolkit and as such all our size, spacing, and such foundational values come from design tokens and are either unit-less or in relative units (rem/em).
This means that when using Floating UI we cannot use the offset() middleware as it only accepts pixel values and also cannot reference a CSS design token. We can use CSS directly to properly add spacing/offset in the default initial position of our floating element but once middleware such as flip() take action things get misplaced since the margin values are not taken into account during the calculations for the new position.
It would be great if flip(), and other middleware, had an option to either apply a positional CSS class (user specified array or automatic such as fui-block-start | fui-inline-end | ...) or data attribute (data-fui-position='block-start | inline-end | ...') or even an option flag to include getComputedStyle().margin` values in the positioning calculation.
Would be great to hear what the maintainers think about this kind of enhancement.

Put together a quick hack for a component but it would still be nice if Floating UI did this automatically.
// Add the Floating UI data attribute
if (refs.elem.current) {
	refs.elem.current.dataset.fuiPlacement = placement;
}

The CSS below simply converts the default top margin to be negative to shift things appropriately when flip() takes action and moves things from the bottom to the top.
[role='menu'][data-fui-placement^='top'] {
	margin-block-start: calc(token('spacing.2XS' to rem) * -1);
}

Margins aren't supported as they interfere with the lifecycle and can't be computed ahead of time. Only the exported offset() middleware is supported, as it doesn't need to write to the DOM in the middle of the lifecycle. The data attribute you have is one workaround but it's not something that will be built in because of that (cross-platform & perf concerns, extra burden of adding config, etc).
You can convert rem into px to be consumed by offset() using px-to-rem packages available on npm.


Adding large_auto_diff_term.h, for when both number of variables and …

…sizes are only known at runtime.
This is pretty much just the file that Petter gave me to fix a problem I had :-).

Thanks!


Bump senzing from 3.2.0 to 3.3.0

See https://github.com/Senzing/senzingapi-runtime/issues/31

Pull requests:

https://github.com/Senzing/risk-score-calculator/pull/87


Fixed git.get_modified_files

What does this PR do?
Wasn't handling correctly files with spaces inside their name, fixed this
Motivation
Describe how to test/QA your changes
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Included configuration not working

If you include a configuration with actions for a hook and the local configuration does not contain a configuration for this hook the actions are not executed because the default behavior for a hook configuration is disabled.

As discussed in pull request #50 it should be possible to include actions for a not locally configured hook.
The behavior should be changed in a way that active disabling still works but none existing configuration should not prevent included actions from being executed.

This is fixed with @aboks efforts in pull request #50


Extract dependencies to separate gradle file

As a developer, I want to extract dependencies to a separate gradle file so that the code is cleaner.
Acceptance Criteria

[ ] Dependencies are removed from \wisefy\build.gradle
[ ] Dependencies are added to a new dependencies.gradle

After trying this, I felt it's easier and more readable to keep the dependencies where they are located in the wisefy module.


nixos/teamspeak: ipv6 support

Unlike the options descriptions the service was not listen to any
IPs because the address family was limited to ipv4.
Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Does this really work? Because Teamspeak Server does not support IPv6 at the moment, AFAIK

@Gerschtli good question. I can only confirm that my monitoring system no longer complains because the port is open externally.


Barcode Scan Regex Update

ITS-Pro time category: https://its-pro.ucsd.edu/browse/MA-74
This is complete when:

[ ] Barcodes scanned that do not start with "EXC" is rejected as a failed scanned

@c3bryant Heartbeat check will need to be updated when this is deployed to prod


Add Builder Perks Section to Founder Hub (near.org/founders)

Add Builder Perks Section to Founder Hub
Label
Type: Feature Request
Request
Add a new "Builder Perks" section to the Founder Hub on the NEAR Horizon Accelerator landing page.
Background
To further enrich the Founder Hub with valuable resources, we are introducing a dedicated "Builder Perks" section that highlights the curated benefits for HZN Founders.
File to Edit
The changes will be made to the founders.tsx file in the repository.

File Location: founders.tsx

Changes Required
Founder Hub Section

Action: Implement a new box within the Founder Hub titled "Builder Perks".

Content: We've curated a growing collection of curated resources and over $500K in discounts for HZN Founders. Interested in having your product listed as a perk? Send us a message at<EMAIL_ADDRESS>

Visuals and Interactivity:

The new box should feature a design consistent with the existing brand design of the page.

It should include a clickable element that directs users to the detailed Builder Perks page.

Implementation Details

Placement: Ensure the "Builder Perks" box is easily noticeable within the Founder Hub to optimize engagement.
Link out to Notion page: Builder Perks

Additional Context

This feature is part of a strategic enhancement to make the Founder Hub a central pillar of value for early-stage builders.

Attachments

@calebjacob This is fantastic! Exactly what we were looking for, thank you!


Is this project dead?

Is this project dead? It looks like multiple of its packages are missing from npm.

Sorry about that, I didn't realise anyone was actually using this!
I am in the process of modernising it with all the stuff I have learned over the last 4 years! There are a few forks that kept things going but I don't expect it will be long before I publish a new version.


feat(compiler): Add is_introspection helper method

Add is_introspection helper method to to OperationDefinition.
Closes #396

Example usage:
let query_input = r#"
query Op {
  __type(name: "User") {
    name
  }
}
"#;

let mut compiler = ApolloCompiler::new();
let query_id = compiler.add_executable(query_input, "query.graphql");
let db = compiler.db;
let introspect_query_op: Arc<OperationDefinition> = db
            .find_operation_by_name(query_id, String::from("TypeIntrospect"))
            .expect("TypeIntrospect operation does not exist");

let is_introspection = introspect_query_op.is_introspection(&db);

@jregistr: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

For traversing the inline fragments and fragment spreads, I decided to use a "dfs" approach. I do have a bfs impl I tested. For that one though, I needed the VecDeque to be of owned SelectionSet and hence needed to clone/to_owned wherever I pulled a selection_set. Do let me know if that'd be preferred and I can clean up that implementation and update the PR.

@jregistr: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

Signed!

Could not find a usable config.yml, you may have revoked the CircleCI OAuth ap

I'm guessing this is from the recent breach at CircleCI?
Is there plan to switch to a new CI?

Awesome!!


Multi node training

Hi @joonson,
Could you please give some hints to make it work for a multi-node multi gpu distributed training?

It will require many changes to the code, including but not limited to:
https://github.com/clovaai/voxceleb_trainer/blob/343af8bc9b325a05bcf28772431b83f5b8817f5a/trainSpeakerNet.py#L120-L131
We cannot provide support for this at this stage.


Implement and test ValidatorDataSource, refactor a few things

Build ValidatorDataSource class that calls validator endpoints through HttpService #16
Let me know if this is looking good so far and what to improve.
One thing I wasn't sure about: should the ValidatorDataSource use the HttpClient instead? Then I'd make the HttpClient's methods generic to allow getting the desired data class immediately. Something like suspend inline fun <reified T> doGet(path: String): T = networkClient.client.get<T>(path).

@Niekon01 If I'm not mistaken, this wasn't paid yet. I'd appreciate it if you could take a look :)
2c9a538d3952a780ba65ee5a672e9ae4b415f542ab265e9517687b25ae4897aa

Hey @petersommerhoff there is no PR reward on the issue that this PR closes. This is why I did not pay any amount.

@IanArb: Same here, would it be possible to attach a fitting number of coins?


Added try block to catch Chrome private keys generated in pkcs8 format.

When Chrome generates a private key, it is generated in the pkcs8 header format, i.e.
-----BEGIN PUBLIC KEY-----
as opposed to the pkcs1 format:
-----BEGIN RSA PUBLIC KEY-----
this throws InvalidAsn1Error: encoding too long when generating a public key from a Chrome generated private key.

Any taker on this?

My take is we should generate keys as Chrome and openssl used to do, which means pkcs8-private-pem. It might explain why people had to regenerate their private keys.
Although it would be a BC break so a bump to another major version, IMO.

Published as v3.0.0 :-)


Android driver - GetSettings: 'NotYetImplementedError: Method has not yet been implemented'

Description
I'm trying to access 'Settings' of the Android driver in my .NET code. I'm using the latest version (4.3.1) of Appium.WebDriver.
The purpose of access settings (if it matters) is that I need to update the driver to 'compose' (with reference to https://github.com/appium/appium-espresso-driver#jetpack-compose-support).
var currentDriver = driver.Settings["driver"]; results in:

System.NotImplementedException : Method has not yet been implemented
at OpenQA.Selenium.Remote.RemoteWebDriver.UnpackAndThrowOnError (OpenQA.Selenium.Remote.Response errorResponse) [0x000dc] in <19027cc8501444099501b87290061165>:0
at OpenQA.Selenium.Remote.RemoteWebDriver.Execute (System.String driverCommandToExecute, System.Collections.Generic.Dictionary2[TKey,TValue] parameters) [0x0003d] in <19027cc8501444099501b87290061165>:0    at OpenQA.Selenium.Appium.AppiumDriver1[W].Execute (System.String driverCommandToExecute, System.Collections.Generic.Dictionary2[TKey,TValue] parameters) [0x00000] in <7a9e53e0f0754367b36cb9234afdb018>:0    at OpenQA.Selenium.Appium.AppiumDriver1[W].OpenQA.Selenium.Appium.Interfaces.IExecuteMethod.Execute (System.String driverCommand) [0x00000] in <7a9e53e0f0754367b36cb9234afdb018>:0
at OpenQA.Selenium.Appium.Android.AndroidCommandExecutionHelper.GetSettings (OpenQA.Selenium.Appium.Interfaces.IExecuteMethod executeMethod) [0x00000] in <7a9e53e0f0754367b36cb9234afdb018>:0
at OpenQA.Selenium.Appium.Android.AndroidDriver`1[W].get_Settings () [0x00000] in <7a9e53e0f0754367b36cb9234afdb018>:0

Logs from the server:

2022-04-21 11:29:41:181 [HTTP] --> GET /wd/hub/session/8cfb9c8e-b45c-4cbb-97b1-c47ded47bd71/appium/settings
2022-04-21 11:29:41:181 [HTTP] {}
2022-04-21 11:29:41:189 [W3C (8cfb9c8e)] Calling AppiumDriver.getSettings() with args: ["8cfb9c8e-b45c-4cbb-97b1-c47ded47bd71"]
2022-04-21 11:29:41:248 [W3C (8cfb9c8e)] Encountered internal error running command: NotYetImplementedError: Method has not yet been implemented
2022-04-21 11:29:41:248 [W3C (8cfb9c8e)]     at EspressoDriver.getSettings (/Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/node_modules/appium-espresso-driver/lib/commands/general.js:288:9)
2022-04-21 11:29:41:248 [W3C (8cfb9c8e)]     at commandExecutor (/Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/node_modules/appium-base-driver/lib/basedriver/driver.js:334:9)
2022-04-21 11:29:41:248 [W3C (8cfb9c8e)]     at /Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/node_modules/async-lock/lib/index.js:129:12
2022-04-21 11:29:41:248 [W3C (8cfb9c8e)]     at AsyncLock._promiseTry (/Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/node_modules/async-lock/lib/index.js:253:31)
2022-04-21 11:29:41:248 [W3C (8cfb9c8e)]     at exec (/Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/node_modules/async-lock/lib/index.js:128:9)
2022-04-21 11:29:41:283 [W3C (8cfb9c8e)]     at AsyncLock.acquire (/Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/node_modules/async-lock/lib/index.js:144:3)
2022-04-21 11:29:41:283 [W3C (8cfb9c8e)]     at EspressoDriver.executeCommand (/Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/node_modules/appium-base-driver/lib/basedriver/driver.js:347:39)
2022-04-21 11:29:41:283 [W3C (8cfb9c8e)]     at AppiumDriver.executeCommand (/Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/lib/appium.js:547:36)
2022-04-21 11:29:41:284 [W3C (8cfb9c8e)]     at processTicksAndRejections (internal/process/task_queues.js:93:5)
2022-04-21 11:29:41:284 [W3C (8cfb9c8e)]     at asyncHandler (/Users/user/.nvm/versions/node/v14.16.1/lib/node_modules/appium/node_modules/appium-base-driver/lib/protocol/protocol.js:297:21)
2022-04-21 11:29:41:285 [HTTP] <-- GET /wd/hub/session/8cfb9c8e-b45c-4cbb-97b1-c47ded47bd71/appium/settings 405 104 ms - 1525

Are you sure you use Appium2 along with the most recent driver version?

I am using appium 1.22.3 installed via npm install -g appium. When running npm -g list | grep appium-espresso-driver I see version 1.45.3. So how can I upgrade the espresso driver to 1.50.1 for example, or even 2.2.1 which seems to be the latest.

It is only available with Appium2. Check https://appiumpro.com/editions/122-installing-appium-20-and-the-driver-and-plugins-cli for more details.

@mykola-mokhnach It seems the Settings API is only available to the iOS driver as part of the implementation with this pr: https://github.com/appium/appium-dotnet-driver/pull/466
v5.0.0-alpha2

It is only available with Appium2. Check https://appiumpro.com/editions/122-installing-appium-20-and-the-driver-and-plugins-cli for more details.

Thanks, I've changed to Appium2.


[PE-D][Tester G] Allowed strange behaviour for edit command

Before

After

When multiple n/ prefixes are supplied to the edit command, the command takes the last n/ arguments and continues to perform the edit. In the case above, it's possible for a user to key in the command wrongly and accidentally enter the same prefix twice when he actually wants to edit 2 separate fields (in this case, person name and project name). This would lead to the above scenario where the user can accidentally edit the name. Maybe some kind of warning could be given or an error could be shown to the user when this is attempted.

Labels: type.FunctionalityBug severity.Medium
original: nqt230/ped#5

Duplicate of #180


Storage unit labels - GB or GiB?

I see Windows displays storage values with the label GB when they are actually GiB. I don't know whether SQL Server displays GiB values labeled as GB. Are the labels in this doc page correct or should they be GiB?

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

ID: 453dc1e2-0c9c-fe95-c6fc-90be886432ad
Version Independent ID: cd472bc0-26b9-a4a6-4089-31b79be787cb
Content: Editions and supported features - SQL Server 2017
Content Source: docs/sql-server/editions-and-components-of-sql-server-2017.md
Product: sql
Technology: release-landing
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@julienshepherd -- thank you for your question.
@MikeRayMSFT -- please look into this issue.

The labels in that article should be GiB. #please-close


Guide clarification about the database

The readme.md tells
"Before running the examples you have to migrate the database. Make sure you have set all the credentials to the database and then you can run the command:"
I kindly suggest to explicitly indicate that the admin must create a database e.g. "htmxdemo" as preliminary operation

Thank you for your suggestion! The readme has been updated!


Screen size issue between Chrome and Headless Chrome

Two pictures to illustrate:
With Chrome:

With Headless Chrome:

Same with Firefox. That's on Debian if it makes a difference.
I wonder if there is an issue when the browser is reporting its viewport dimensions to the app.

I would try to see if you get the same with puppeteer in node. If so I would report that to chromiumbugs.

We are seeing a similar issue when generating a PDF server-side:

& in the browser/client-side

@curtiswmcmillin I think your case is different. Can you generate an HTML I can use to test it?

Thank you for the quick reply :-).  Here's the code:

I'll get some HTML for reference, too.

@kblok if it helps, we first noticed the issue on 11/13 in the evening about 6 pm EST

@curtiswmcmillin did you open that HTML locally? It doesn't look good.

The html & css hasn't changed in quite a while, but has been working fine.

@kblok I've tried using an older version of Chromium, too.  Based on the C# code above, do you have suggestions on a different approach?

@curtiswmcmillin what puppeteer-sharp version are you using?

<PackageReference Include="PuppeteerSharp" Version="12.0.0" />

.NET 7

@curtiswmcmillin the HTML you shared won't help :(

That's why I included the CSS.  You can reference the CSS here, too: https://npvgenerator.azurewebsites.net

I can adjust the HTML
<link rel="stylesheet" href="https://npvgenerator.azurewebsites.net/css/style.css">
<link rel="stylesheet" href="https://npvgenerator.azurewebsites.net/css/style.madison-specs.css">

Does that help? Sorry for the confusion.

If you can give me a piece of code, in text, that I can work in isolation, that would be more productive. A zip that I can download and it works the same way as in your machine.

html.txt

It seems it looks ok on my side

You could send me an email with your browserless URL.

Should have it, if not lmk

I've tried a number of different options in this code block but no luck:
    ```

using var browserFetcher = new BrowserFetcher();
await browserFetcher.DownloadAsync();
await using var browser = await Puppeteer.ConnectAsync(
new ConnectOptions { BrowserWSEndpoint = $"{_appSettingsDocumentService.Value.BrowserWSEndpoint}{_appSettingsDocumentService.Value.BrowserlessToken}" });
    await using var page = await browser.NewPageAsync();
    await page.SetContentAsync(html);

    await page.WaitForNetworkIdleAsync(options: new WaitForNetworkIdleOptions
    {
        Timeout = _appSettingsDocumentService.Value.NetworkOptionsTimeout,
        IdleTime = _appSettingsDocumentService.Value.NetworkOptionsIdleTime
    });

    var pdfOptions = new PdfOptions
    {
        Scale = 1,
        PreferCSSPageSize = true,
        PrintBackground = true
    };

    var marginOptions = new MarginOptions
    {
        Top = "20px",
        Bottom = "20px",
        Left = "25px",
        Right = "25px"
    };
    pdfOptions.MarginOptions = marginOptions;


Request - Freeing slaves as a daily for characters

It would be nice to have a extra task for freeing slaves , nothing major but its cool to add if you got time.

Was added in an update yesterday.


Ability to disable default implementations of HTTP and WebSocket clients

Which branch your pull-request should merge into?

2.1: BugFixes and Improvements for Current LTS branch

Describe your changes
Allow the HttpClient and WebSocket implementations to be excluded from the build.  This may be because they are either not required, or the game project has its own implementations, so this would prevent build time conflicts in code.
Allow LLHTTP to be disabled if not required to avoid build time conflicts with game-side libraries.
Issue ticket number and link
This may help with #1642
Checklist before requesting a review
For each PR

[ ] Add Copyright if it missed:
- "Copyright (c) 2019-present Axmol Engine contributors (see AUTHORS.md)."

[x] I have performed a self-review of my code.
Optional:

[ ] I have checked readme and add important infos to this PR.
[ ] I have added/adapted some tests too.

For core/new feature PR

[ ] I have checked readme and add important infos to this PR.
[ ] I have added thorough tests.

Just an additional note.  As a work-around for the issue raised in #612, I'm currently using the changes in this PR to allow an alternative http library to be used in the a game project, otherwise there are too many compilation errors due to type conflicts.

Suggestion:

add option description to CMakeOptions.md
rename AX_USE_XXX to AX_ENABLE_XXX for core features

Suggestion:
1. add option description to `CMakeOptions.md`

2. rename `AX_USE_XXX` to `AX_ENABLE_XXX` for core features

Will do.

Should AX_WITH_LLHTTP be changed as well, or leave it as is?

Should AX_WITH_LLHTTP be changed as well, or leave it as is?

3rdparty just leave as-is


Add missing uncommited changes

n/a

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 64.33%. Comparing base (a20c8e9) to head (6f56e69).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3913   +/-   ##
=======================================
  Coverage   64.33%   64.33%
=======================================
  Files         381      381
  Lines       20577    20577
=======================================
  Hits        13238    13238
  Misses       6050     6050
  Partials     1289     1289

Flag
Coverage Δ

unittests
64.33% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Add repository description, short introduction and CI status overview

I took the liberty of further writing the README. From my point of view, this is Ready for review.
I've added an explanation about the URDF structure with all the config files, as this has been generating confusion in the past.

@destogl could you please give a quick approval if this state is fine with you?


System Health Monitoring

https://github.com/homarr-labs/homarr/issues/655

Hold up, sorry I only saw this now, this widget might be part of https://github.com/homarr-labs/homarr/pull/848
@homarr-labs/maintainers What do you think?

OMV integration is ready.
@homarr-labs/maintainers What do you think?

feat: Health monitoring widget #1144


Where to find Examples and compile error

Hi! Nice Job. However, I met some problems when I use it.
1.How do I find the HDK examples(toolkit/samples/SOP/SOP_WindingNumber)?

I included these files and use the class.When I compile,  I meet error of "undefined reference to " memeber function of UT_SolidAngle, which is caused by method of member function written in .cpp file.

This GitHub account is a 3D software company, and the software name is Houdini. HDK examples (toolkit/samples/SOP/SOP-WindingNumber) means that you need to install Houdini and find it in the software directory.
I am a user of Houdini and a fan of HDK. I can successfully compile and run this code based on HDK Examples. If you need any help, you can contact me.


Implement upsteam/downstream CI job pipeline bases on the setup/run test suite/results present job.

1, it could run tests in parallel.
2, it should be upsteam/downstream mode.

already done.


Add pixi self-update

Supersede #582

You can test it locally in your dev env with:
$ pixi run cargo run -- self-update --force --version 0.8.0
$ ./target/debug/pixi --version
pixi 0.8.0

See the current API:
$ pixi self-update
✔ Pixi will be updated from '0.1.0' to '0.12.0'
✔ Pixi archive downloaded.
✔ Pixi archive uncompressed.
✔ Pixi has been updated to version 0.12.0.

$ pixi self-update # wrong pixi location
✘ pixi is not installed in the default location:
- Default pixi location: /Users/hadim/.pixi/bin/pixi
- Pixi location detected: /Users/hadim/Code/libs/pixi/pixi/target/debug/pixi

It can happen when pixi has been installed via a dedicated package manager (such as Homebrew on macOS).
You can always use `pixi self-update --force` to force the update.

$ pixi self-update --version 0.8.0
✔ Pixi will be updated from '0.1.0' to '0.8.0'
✔ Pixi archive downloaded.
✔ Pixi archive uncompressed.
✔ Pixi has been updated to version 0.8.0.

$ pixi self-update
✔ pixi is already up-to-date (version 0.12.0)

$ pixi self-update --version 0.8.99
✘ The version you specified is not available: 0.8.99

I am happy with the logic. I'll let Bas and Ruben chime in with their thoughts!

@baszalmstra ready for another round!
I removed all the unwrap and replaced most of them by except (when the error is quite unlikely to happen).
I am also now using miette for "expected" errors.
I also replaced all the println to eprintln.

I also replaced all the println to eprintln.

I am actually unsure whether those should be printed to stdout or stderr looking at other cli commands.

Thanks @baszalmstra - should be good now.

@hadim maybe you can follow up with a few docs. Great feature! :)

I already added the cli doc at docs/cli.md in that PR. Do you mean something else?

perfect, nope. sorry. didn't look closely enough 🤦 (getting late over here haha)


refactor(batcher): rename ProposalContentId to ProposalExecutionFin

This change is

#1083  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @matan-starkware and the rest of your teammates on  Graphite

Merge activity

Sep 29, 9:40 AM EDT: @matan-starkware started a stack merge that includes this pull request via Graphite.


Build/optimize express build and deployment

In further preparation for the react 16 upgrade, we can safely modernize the React and Express builds.

Move React Build to Facebook's react-scripts to optimize output, tests, HMR, and simplify build complexity while adding build features like import images and css.

Stop compiling express application and run the server as native node.

There is not longer a need to use Webpack, Gulp and babel directly on the ReactJS side. With the release of the React Scripts we can use the default build, run and test with zero config.
Moreover, with node 8.9.x+ we can stop compiling the express server and just run node server/server.js directly, no transpilation needed.
This will change current deployment and development models slightly.
Development Process (no more need to use watch script)

npm run server (Handles the Express Hot Reloading and runs the express server)
npm start (Handles the React HMR and express proxy)
I added a watch script for any existing users that use watch as part of their process.

Deployment Process:
npm run build

copy modules and server directory to the server
run "node server/server.js" on the remote server

@lordbender Thank you very much for the PRs.
This PR would accompany #696 in the major version upgrade.

@apanicker-nflx should I change the target branch? The major version would be a good time to put in the React 16+, React Router and Redux upgrades too.

@lordbender Please keep the PR pointed to dev branch for now. We will be merging the 2.0 branch to dev once it is stable, and we can merge the UI changes into dev then.  This would keep a clean separation between the UI and backend changes. Thanks.

Closing this too, please see #696 for comment.


setting up a superuser with em_auth_username

Environment

OS: FreeBSD
Erlang/OTP: 18
EMQ: 22

Description
I try to setup a superuser with only em_auth_username. With sql based system and all others it is working.
Is there a way?

@wiiikiii EMQ R2.2 supports Authentication/ACL chain. You can load more than one authentication plugin at the same time.
Documentation: http://emqtt.io/docs/v2/guide.html#authentication


backend/session/libseat: register log handler

Route libseat errors through wlroots logging infrastructure.
cc @kennylevinsen

We need to address the build failures before we can merge this.

Build was broken due to the new logging API not yet being released. I have now created a 0.2.0 release of seatd/libseat containing the feature and updated the Arch package accordingly.

Oh, thanks. No big deal, we'll wait. I've updated the Meson build file to require 0.2.0.

Pick up https://github.com/freebsd/freebsd-ports/commit/ec167dd776b8c132b4eeeba847ecdf4117868a28 or wait ~3 days for pkg.freebsd.org to catch up.


Add changefeed fail with IMPORT INTO limitation to IMPORT INTO docs

Add that the changefeed and import into limitation to the import into limitations and general limitations page

Note this is already listed in the Considerations section on the IMPORT INTO page and so I'm not sure that this requires further doc'ing in limitations without a tracking issue. There is ongoing docs work to clarify known limitation vs. consideration/guideline.


[FLINK-23571][table-planner] Set internal query-start options when translate ExecNodeGraph to Transformation

What is the purpose of the change

This pull request set the internal query-start configuration options when translate translate ExecNodeGraph to Transformation

Brief change log

Set the internal query-start configuration options when translate translate ExecNodeGraph to Transformation

Verifying this change
This change added tests and can be verified as follows:

Add unit tests in  InternalConfigOptionsTest

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

cc @wuchong


logic error in dielectric::scatter

cosine = m_refIdx * -Dot( In.m_direction , Rec.normal ) / In.m_direction.Length( );
multiplying refraction index to cosine got rid of a black pixel artifact I was seeing on my render.
P.S: notice the minus sign next to the Dot, this is for the else part where Dot product is <=0.

Before:
https://drive.google.com/open?id=1TSHYCntsAyYkX1RdQr5tlUQvnYgG6c0F
After:
https://drive.google.com/open?id=16RgI4JC34g7ldxKd3S1_ucGbyjYjLQDh


S3: speedup transfers

Hello,
I use Rusoto for download & upload files to S3 buckets.
I have tried to compared performances, and it's far from aws cli.
Do you have any idea/recommendation to speedup transfers ?
Is it possible to open multi-connections to send/receive data simultaneously ?
Thank you,
Marc-Antoine

Have you tried compiling --release? You may see a 10x improvement. See #1794

In some sense the aws is a client application.
rustoto is a client.
The aws cli has options that drive individual requests and spawns to parallelize based on configuration.
This is possible with rusoto in your application.  Rusoto itself is not an application.


how to use it?

I have tried everything but I can't get it to work, please help.

There's something I need to know before I explain.
Do you know the usage of node.js and npm?

I know the basics

the truth is that I don't know how to use npm or node.js, if it wouldn't be too much trouble, I would like some reference images to know what exactly I should do.

Install node.js at here.
Download all the source files of this repository.
Open cli at where the sources are.
Run command npm install.
See readme.

thank you very much for your help, I just needed to install npm

Can I close thie issue?


Icons: Fantastical → Calendar, HEY → Mail, Neovide → Vim

HEY.app being that from https://www.hey.com/apps/

Neovide.app being https://github.com/neovide/neovide (Intel from releases or M1 fromcargo bundle --release)

Includes by deconflicted merge (and will close) #195

Thanks for your work!


Make libTXT test and benchmark runs identical on Mac and Linux.

Currently, those harnesses can only be run on Linux because of the OpenType shaper on Linux vs CoreText on Mac. We can use OpenType on Mac and CoreText on iOS for size instead and make the host builds match.

Obsolete after SkParagraph migration now underway. Closing.


Use consistent capitalization in Activity feeds

Assigned ownership (instead of Ownership)
Started following (instead of Following)
Can you also please check if for other types of changes we are using the right capitalization?
cc @harshach

We should also change the order here - Removed tier Tier1. Added tier Tier2.


data-line plugin not working

I have been using prism successfully for some time. Recently when I wanted to use line highlight plugin, adding data-line attribute to the pre element, it did not work. I made sure that the data-hightlight plugin is selected while downloading prism. Will be thankful if somebody helps to fix the issue. One may refer https://www.w3resource.com/python/python-bytes.php for this where the issue reported lies.

Well, adding line-height: 1em; to the  tag, this isssue is fixed.


[BUG] Geocoding control breaks OpenLayers map with useGeographic()

Description
When initializing an OpenLayers based map in conjunction with useGeographic() any result clicked from the search bar renders the map black and hence useless.
Steps to reproduce
To reproduce the issue, simply create a sample OpenLayers based map together with useGeographic() and instantiate the Geocoding control, type something into the search bar, click on the first result and watch the map black out.
Expected behavior
The Geocoding control should support OpenLayers maps with useGeographic() as well

Thanks for the report!

Hey Martin,
I just had the opportunity to test and must say that the issue still persists. I referenced the latest release 1.3.1 and it still happens.
Here's some sample html code to reproduce the issue:

  <meta name="viewport" content="initial-scale=1,maximum-scale=1,user-scalable=no" />
  <title>MapTiler Geocoding control</title>

  <link href="https://cdn.maptiler.com/maptiler-geocoding-control/v1.3.1/style.css" rel="stylesheet">

```

Hope this helps.

With 1.3.2 it should work now. Thanks for your patience :-).
Fixed in https://github.com/maptiler/maptiler-geocoding-control/pull/40