Server Rendering example not working, using v1.0.0-beta2

Hi,
I'm trying to use the server side example documented here: https://github.com/rackt/react-router/blob/master/doc/00 Guides/Server Rendering.md
but I'm getting an error.
TypeError: undefined is not a function
    at /Users/erik/test/flux-examples/react-router/server.js:27:12
    at Layer.handle [as handle_request] (/Users/erik/test/flux-examples/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/Users/erik/test/flux-examples/node_modules/express/lib/router/index.js:312:13)
    at /Users/erik/test/flux-examples/node_modules/express/lib/router/index.js:280:7
    at Function.process_params (/Users/erik/test/flux-examples/node_modules/express/lib/router/index.js:330:12)
    at next (/Users/erik/test/flux-examples/node_modules/express/lib/router/index.js:271:10)
    at favicon (/Users/erik/test/flux-examples/node_modules/serve-favicon/index.js:72:7)
    at Layer.handle [as handle_request] (/Users/erik/test/flux-examples/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/Users/erik/test/flux-examples/node_modules/express/lib/router/index.js:312:13)
    at /Users/erik/test/flux-examples/node_modules/express/lib/router/index.js:280:7

and my line: 27 is
    Router.run(app.getComponent(), req.path, function (Handler, state) {

the complete implementation is:
require('babel/register');
var express = require('express');
var favicon = require('serve-favicon');
var serialize = require('serialize-javascript');
var navigateAction = require('./actions/navigate');
var debug = require('debug')('Example');
var React = require('react');
var app = require('./app');
var HtmlComponent = React.createFactory(require('./components/Html.jsx'));
var FluxibleComponent = require('fluxible/addons/FluxibleComponent');
var Router = require('react-router');

var server = express();
server.use(favicon(__dirname + '/../favicon.ico'));
server.use('/public', express.static(__dirname + '/build'));

server.use(function (req, res, next) {
    var context = app.createContext();

    debug('Executing navigate action');
    Router.run(app.getComponent(), req.path, function (Handler, state) {
        context.executeAction(navigateAction, state, function () {
            debug('Exposing context state');
            var exposed = 'window.App=' + serialize(app.dehydrate(context)) + ';';

            debug('Rendering Application component into html');
            var Component = React.createFactory(Handler);
            var html = React.renderToStaticMarkup(HtmlComponent({
                state: exposed,
                markup: React.renderToString(
                    React.createElement(
                        FluxibleComponent,
                        { context: context.getComponentContext() },
                        Component()
                    )
                )
            }));

            debug('Sending markup');
            res.send(html);
        });
    });
});

var port = process.env.PORT || 3000;

And basically it means that  run() method of Router object undefined.
I'm using ver 1.0.0-beta2
Best regards

try it with
var location = new Location(req.path, req.query);
Router.run(app.getComponent(), location, (error, initialState, transition) => {

and instead of Component()
<Router {...initialState}/>

thanks @heebtob,
but the initial problem is that
var Router = require('react-router');

is returning  https://github.com/rackt/react-router/blob/master/modules/index.js
and it is a Router object https://github.com/rackt/react-router/blob/master/modules/Router.js
that have not the method run()
Regards.

https://github.com/rackt/react-router/blob/master/modules/Router.js
line 65
run(routes, location, transitionHooks, callback) {

I think if you are using require instead of ES6 import, you have to ask for the default.
try
var Router = require('react-router').default;
https://github.com/rackt/react-router/issues/1370

thanks @tmbtech

thanks @tmbtech - this should be mentioned in the docs somewhere..


Updated 'randomcolor' command

Removed randomcolor dependency as it is no longer used
Added preview for generated color in 'randomcolor' command

Forgot about adding new api to the docs..

Forgot about adding new api to the docs..

don't worry bout it


Broken link for filing issues on datagovmy docs

Issue:

The link to leave a GitHub issue is broken.

Can be found on the datagovmy Response Code docs here.

The link to open a new issue is broken.

Can also be found on the datagovmy FAQ docs here.

hi @britneywwc ,
Thanks for alerting us regarding the broken links!
It was using an old implementation that we no longer use and we have amended the link accordingly so it should now be working.


Management of accounts with non default security policies appears broken

This includes ownership requests, approvals/additions, manage packages page, admin UI for policy subscription management, etc (to be found out).
@chenriksson @dtivel
Exception is: Subscription 'ControlRequiredSignerPolicy' not found.
Stack trace (kind of):
{"assembly":"NuGetGallery, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null","method":"NuGetGallery.Security.SecurityPolicyService.IsSubscribed","level":0,"line":225,"fileName":"C:\vsts-agent\_work\36\s\src\Gallery\submodules\Gallery\src\NuGetGallery\Security\SecurityPolicyService.cs"}
{"assembly":"NuGetGallery, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null","method":"NuGetGallery.ListPackageItemRequiredSignerViewModel..ctor","level":1,"line":70,"fileName":"C:\vsts-agent\_work\36\s\src\Gallery\submodules\Gallery\src\NuGetGallery\ViewModels\ListPackageItemRequiredSignerViewModel.cs"}
{"assembly":"NuGetGallery, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null","method":"NuGetGallery.UsersController+<>c__DisplayClass27_0.b__2","level":2,"line":445,"fileName":"C:\vsts-agent\_work\36\s\src\Gallery\submodules\Gallery\src\NuGetGallery\Controllers\UsersController.cs"}
{"assembly":"System.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089","method":"System.Linq.Enumerable+WhereSelectListIterator2.MoveNext","level":3,"line":0}  {"assembly":"System.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089","method":"System.Linq.Buffer1..ctor","level":4,"line":0}
{"assembly":"System.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089","method":"System.Linq.OrderedEnumerable1+<GetEnumerator>d__1.MoveNext","level":5,"line":0}  {"assembly":"mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089","method":"System.Collections.Generic.List1..ctor","level":6,"line":0}
{"assembly":"System.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089","method":"System.Linq.Enumerable.ToList","level":7,"line":0}
{"assembly":"NuGetGallery, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null","method":"NuGetGallery.UsersController.Packages","level":8,"line":443,"fileName":"C:\vsts-agent\_work\36\s\src\Gallery\submodules\Gallery\src\NuGetGallery\Controllers\UsersController.cs"}
{"assembly":"Anonymously Hosted DynamicMethods Assembly, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null","method":"lambda_method","level":9,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.ReflectedActionDescriptor.Execute","level":10,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.ControllerActionInvoker.InvokeActionMethod","level":11,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.Async.AsyncControllerActionInvoker.b__39","level":12,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.Async.AsyncResultWrapper+WrappedAsyncResult`2.CallEndDelegate","level":13,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.Async.AsyncControllerActionInvoker.EndInvokeActionMethod","level":14,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.Async.AsyncControllerActionInvoker+AsyncInvocationWithFilters.b__3d","level":15,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.Async.AsyncControllerActionInvoker+AsyncInvocationWithFilters+<>c__DisplayClass46.b__3f","level":16,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.Async.AsyncControllerActionInvoker.EndInvokeActionMethodWithFilters","level":17,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.Async.AsyncControllerActionInvoker+<>c__DisplayClass21+<>c__DisplayClass2b.b__1c","level":18,"line":0}
{"assembly":"System.Web.Mvc, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35","method":"System.Web.Mvc.Async.AsyncControllerActionInvoker+<>c__DisplayClass21.b__1e","level":19,"line":0}
An example event can be found in gallery AI logs with operation id 9Rt9zyxGs4I=.
This was discovered by @karann-msft during the following scenario:

Organization recieved ownership request email.
Link was clicked to accept ownership.
500 exception page was encountered, and no action appears to have taken place.
Navigating to manage packages page also return 500.

The issue is due to this change, the OrganizationSubscriptions seems to be missing the _controlRequiredSignerPolicy policy.


wording

Just some suggestions on the English-language copy.

still feel pretty strong about we versus you, closing out till its updated


Figure out {build_commit} for github actions

original motivation: #67 , implementation for all the rest besides github actions: #68 . github actions seems to not provide the {build_commit} sha :-/, @jwodder filed a support issue, more information etc: https://github.com/con/tinuous/pull/68#discussion_r643293302

no feedback from github on the support ticket?

@yarikoptic Just a form reply saying that the ticket was routed to the Product team.


Added a MQTT pub/sub example of using AWS IoT

This example demonstrates ESP8266 is capable of working with the AWS IoT service through TLS1.2 based MQTT connection.
Readme file included.
Some memory usage stats of the example:

Very exciting! Can't wait to test this.
Do you think we could wrap some of this functionality into a component/library? I know people have tried to port the actual AWS IoT SDK, but this looks like it might be an easier approach as the AWS SDK itself makes a lot of assumptions about the system you're using.

We could definitely wrap portion of the code into a library, however the MQTT task and SSL connection are not abstracted enough to be consumed as a library. Given this example, I think it's just matter of time before someone eventually ports the AWS IoT device SDK to this project, which may make more sense to be consumed as a library.
I guess a lot of applications probably just want basic pub/sub function like this sample does so they can save more memory for other uses. I could potentially port the AWS IoT device SDK over if it turns out more people want to use it that way.

I added two more commits, specifically around handling over-sized messages received - one is to fix the issue reported in https://github.com/SuperHouse/esp-open-rtos/issues/152 (after the fix, connection will still get reset for sanity purpose but the system won't crash) and the other one is to reset SSL upon read/write errors, such as over-sized record received.

This example is perfect for our use case. Adding basic support for IoT shadows would be nice but as stated, most people just need pub/sub.
Only suggestion is maybe add some comments around the NewMQTTClient on what the values are how you would change them for larger messages.
NewMQTTClient(&client, &network, 5000, mqtt_buf, 100, mqtt_readbuf, 100);

This is awesome.. I was able to test it out today, and its all working.
Now, I think I will need the thing-shadows capability.. In my project I will need to check if my device is ON/OFF from a  website. Looks like this feature is still missing? Is there something I can do myself? Any recommendations?
(PS: But very glad I got this far. I almost gave up 2months ago. Then I had to go on a vacation. By the time I am back its all implemented. Super excited.)

Thanks @rongsaws and @toashwin. I will merge this one, sorry it took so long.

@toashwin, for simple shadow interactions from the device, you could publish the below shadow update message to topic $aws/things/thingName/shadow/update:
{
    "state": {
        "reported": {
            "switch": "ON",
        }
    }
}

More details can be found here. Please let me know whether this works for you. Thanks.

@rongsaws,
Cool. Thank you. That seem to be working when tested with MQTT-Client from the AWS-IOT website. I will test with ESP8266 tonight.
I got few questions:
This is what I did:
I created a thing. It had following values:
REST API Endpoint: https://.iot.us-east-1.amazonaws.com/things/mybutton/shadow
MQTT topic: "$aws/things/mybutton/shadow/update"
Question 1: I wonder why $aws in the topic name? Is it like a variable? In the c-code, do I just replace the value in ( #define MQTT_PUB_TOPIC "esp8266/status" ) with "$aws/things/mybutton/shadow/update" along with $sign?
Question 2: For the client endpoint, would it be:
const char *client_endpoint ="https://.iot.us-east-1.amazonaws.com/things/mybutton/shadow ";
OR
const char *client_endpoint ="https://.iot.us-east-1.amazonaws.com/ ";
Question 3: Not related to ESP8266. From the AWS IOT user guide, I see that AWS IOT Button has ARN:
arn:aws:iot:us-east-1:123456789012:topic/iotbutton/G030JF055364XVRB
For me, ARN has, "things" or "policy" in the path. but I dont see "topic" as in the guide.
And how can I create a device ID into the path? I can do something like:
iotbutton_G030JF055364XVRB
but not
iotbutton/G030JF055364XVRB
Thanks a lot.

@toashwin, I answered your questions below. If you have any other AWS IoT specific questions, I encourage you to post them on the AWS IoT forum here, so we can stay focused on ESP8266 here. Thanks.

Question 1: I wonder why $aws in the topic name? Is it like a variable? In the c-code, do I just replace the value in ( #define MQTT_PUB_TOPIC "esp8266/status" ) with "$aws/things/mybutton/shadow/update" along with $sign?

Topics beginning with '$' are called reserved topics, and no, it's not a variable. You can subscribe to these reserved shadow topics just as you do with other regular topics. See here for more details.

Question 2: For the client endpoint, would it be:
const char *client_endpoint ="https://.iot.us-east-1.amazonaws.com/things/mybutton/shadow ";
OR
const char *client_endpoint ="https://.iot.us-east-1.amazonaws.com/ ";

The client_endpoint variable defined in client_config.c should just be the host name, without https scheme or path.

Question 3: Not related to ESP8266. From the AWS IOT user guide, I see that AWS IOT Button has ARN:
arn:aws:iot:us-east-1:123456789012:topic/iotbutton/G030JF055364XVRB

Different resource types have different ARNs, such as topic ARN, client ID ARN, and thing ARN, see here  for more details. Don't confuse ARN with REST API endpoint path as they are different things. When you access shadow using REST APIs, you use the provided API endpoint, and in this case, no MQTT topics are involved; and when you use MQTT to access the shadows, you publish updates to the reserved shadow topics (as the aws_iot example does) to archive the same purpose as you do with REST APIs.

@rongsaws  - Thanks for your suggestions.. Made very good progress.
How can I retrieve the latest stored data? I want to access that in the device, at start up.
From the code, I see you are only doing subscribe/publish. Do we need something like a "get"?
Thanks
Ashwin

@rongsaws  Thank you so much! This was the last thing missing in my project. I was almost about to move away from ESP8266.
I got the whole project working now..
My device is connecting to AWS IOT over MQTT.  I got it hooked up to Alexa as well. And even thats working perfectly.
Next I am going  to approach Alexa Fund for some financial help to take this project to the next step.  So I can do field testing, get some real customer feedback!  Hopefully that goes through. :(
Thank you.
PS: Please answer my previous question. I have a workaround for that. But still good to know.

@toashwin,

How can I retrieve the latest stored data? I want to access that in the device, at start up.
From the code, I see you are only doing subscribe/publish. Do we need something like a "get"?

Yes, you can make a Get request by sending an empty message to topic $aws/things/ThingName/shadow/update, the shadow content will be published to topic $aws/things/ThingName/shadow/get/accepted.

Hi
Do you have any plans to support AWS IOT for ESP8266 on Arduino SDK?
Thanks
Ashwin

I'm struggling to get AWS IOT working for my application.
I copied the AWT_IOT files to my project rather than build it directly in examples/aws_iot
I get compile erros such as:
`conflicting types for 'sdk_wifi_station_get_connect_status'
/home/chiz/share/esp-open-rtos/include/espressif/esp_sta.h:73:9: note: previous declaration of 'sdk_wifi_station_get_connect_status' was here
uint8_t sdk_wifi_station_get_connect_status(void);
`
Apparently sdk_wifi_station_get_connect_status(void) is already defined in MQTTClient.h as follows:
`
void sdk_wifi_station_get_connect_status(mqtt_client_t*, mqtt_network_t*, unsigned int, unsigned char*, size_t, unsigned char*, size_t);
'
How does this get resolved?

@only1chi there's no such definition in MQTTClient.h? https://github.com/SuperHouse/esp-open-rtos/blob/master/extras/paho_mqtt_c/MQTTClient.h

Hello @toashwin

I got the whole project working now..
My device is connecting to AWS IOT over MQTT. I got it hooked up to Alexa as well. And even thats working perfectly.

Glad to read about your progress. I also trying to subsribe/publish to AWS IOT thing topics, however I am having some issues. I have posted about this in a separate topic as well (#329), below is an excert from that:
I have gotten far enough to be able to publish messages to the shadow. However, there seems to be an issue when ESP is subscribed to the 'thing/shadow/update/delta' topic. The error happens when AWS detects a difference between 'desired' and 'reported' states of the thing shadow and it sends a message to the '/delta' topic. The error message is:
'Connection dropped, request restart'
I am sure that the message being sent to the '/delta' topic reaches ESP because this error happens only when there is a difference between 'desired' and 'reported' states.
I have been successful in subscribing and receiving messages from 'non-thing' topics, but got stuck with receiving the messages from any of the 'thing shadow topics'.
Could it be the size of the payload? The messages received by ESP from 'non-thing' topics were just 'on' and 'off'.


feat(python): Add help on pickle errors

Fixes: ENG-1432

can we make the nice error appear on top?


Keyset mode computation should be based on the whole query

Currently the checks for the keyset mode only rely on the order by expressions which is not really safe but good enough for now. Users have to take care when using keyset pagination to not use a keyset on a wrong query. We should try to make a more complete check that involves the whole query instead of just the order by expressions.

We leave it as it is now since this makes it possible for the user to provide a custom keyset.


Initial AIX support in System.Native/PAL

The two ugliest hacks are:

AIX 6.1, the lowest version we target, has FD_CLOEXEC without O_CLOEXEC. I believe the fake definition should work though, as since AIX doesn't have pipe2, it has to apply FD_CLOEXEC afterwards with fcntl.

I ripped out the error for no epoll/kqueue and put nopped out versions of the async I/O stuff. I don't believe this is wired up yet, but I'll eventually turn this into a poll based fallback. (AIX has pollsets, but PASE only has select and poll.)

I haven't implemented the bit dependent on ifaddrs. There is an alternative, but it's a disgusting amount of code, as you can compare in mono/utils/networking-posix.c.

For now, despite the nopping, it's enough to get the runtime to build as it did before on AIX, and get the where we were before.

I'm not familiar with the process here, so if I did land this in upstream CoreFX; would the changes be cherry picked back into this one?

yes, we sync with corefx on release branches but could cherry-pick a change if it's needed faster


Support Redis TLS config from Env

Why
Loading config from environment does not (nicely) support enabling TLS on Redis due to the fact that ioredis expects an object.
It turns out that setting MEDPLUM_REDIS_TLS to a string (any string - including '{}') does work, but it violates the type signature that ioredis expects and if the implementation changes that could break.
Yes, we could use SSM / SecretsManager and not need this - but the secretops platform we use doesn't support case sensitive keys (fail, I know...), so Medplum's config loader won't quite work for us as intended. That itself is kind of funny, because we are actually using SSM to load the config into ENV. Nonetheless...
What

Improves the typing of MedplumRedisConfig by adding a tls field typed as Record<string, unknown>. This isn't quite as strict as possible, but ioredis doesn't export the ConnectionOptions type that tls is expected to be. Nonetheless, this is at least better than the current state where tls is hidden from the type completely and Record<string, unknown> is at least compatible with ConnectionOptions.
Adds an isObjectConfig helper which returns true for the tls key
Use JSON.parse on the value when isObjectConfig is true
Adds an expectation to the existing env loading test in config.test.ts

Thanks @jdjkelly 🙏


Fix OTEL Collector creation

Error from server (BadRequest): error when creating "otel-collector-instance.yaml": OpenTelemetryCollector in version "v1alpha1" cannot be handled as a OpenTelemetryCollector: strict decoding error: unknown field "spec.observability.enableMetrics"

Thanks!


Cats Spawning Survival

I managed to get cats to spawn in survival with tweaks to the insideBlockCode array.
Each string needs 'game:' prepended to it when trying to spawn inside vanilla blocks.
Meows for days!

Yeah I noticed an error message when loading the world that the block codes are not valid.
Guess thats what I get for simply copy pasting from the fox spawning ^^


[bc breaking] move most previously-global configs to Float8LinearConfig

Stack from ghstack (oldest at bottom):

#325
-> #324

Summary:
Adds a Float8LinearConfig to unify the user facing per-linear
configuration, and moves most of the previously global config options
there.
In future PRs (to keep PRs small), we will move emulation, scaling
and gemm configurations to also live here.
Test Plan:
./test/test_everything.sh

Reviewers:
Subscribers:
Tasks:
Tags:
Differential Revision: D60176981

@vkuzo has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


Calling method from Native Not Working. Getting error - MissingPluginException No implementation found on channel

I'm making project on IOT so I'm trying to call start BLE scan functionality from Native android from onStart Method at that time I'm getting error of MissingPluginException No implementation found on channel. So how can i use Native methods. If it is possible in foreground and background share me demo or example.

It may be difficult to scan ble devices in the background using this plugin. See this repository. Contains logic to scan for ble devices in the background.

I have the same issue.
I was also using flutter_reactive_ble and flutter_foreground task, and it was working well.
But since I updated the repo, it is working in debug but not in release anymore.
It throws the same error : MissingPlugging exception.
@akospwc is it still working in release with the last version of the library for you ?

move issue #149


Add classification functions

Closes #36.

Looks like aarch64 segfaulted during the travis build (not the test). No clue how that happened...

That's... odd. This requires rebasing anyways, can you do that and see if it repros?

Looks like we do indeed have a particularly gribbly bug!

I read somewhere that LLVM segfaults when an assert fails.  When I run this in docker (I don't have access to an actual aarch64 machine) it succeeds, so I thought maybe we're running out of memory and a malloc assert is failing.

I'm unclear why codegen-units=1 is in the build flags on some targets but not all targets, and I'm worried about that not being picked up by downstream, when this crate is built into core/std, or is inlined into a user's program.
If we need to reduce codegen-units for this to build without crashing LLVM, and a user of the crate leads the codegen-units at the default quantity of 256 of 16, will their build explode when they use this crate?

I read in an issue elsewhere that codegen-units=1 could fix aarch64 issues, but it didn't here.  I just didn't remove it yet.  I'm still not completely certain what is causing the issue.  I can't replicate it locally with cross, so I think it's particular to the build agent in some way?

I think I've figured out the issue, but I'm not sure what's really happening.  The problem is the simd_ne intrinsic which is only used in is_nan, is_normal, and is_subnormal.  If I replace it with a naive implementation everything compiles fine on aarch64.  I think this is probably a bug in LLVM?

Following up from discussion: these two examples offer repro of our bug, but only when targeting aarch64, but seemingly only when generating tests, and only in release mode.
#[test]
fn is_normal() {
    assert!(core_simd::SimdF32::<64>::splat(0.).is_normal().to_array()[0]);
}

#[test]
fn bug() {
    use core_simd::{SimdF32, SimdU32};
    let v = SimdF32::<64>::splat(0.);
    let m = v.lanes_eq(SimdF32::splat(0.0)) & v.lanes_eq(SimdF32::splat(0.0));
    assert!(m.to_array()[0]);
}

I reduced the maximum lane count from 64 to 32, since the error only occurred with 64-length vectors.
This prevents AVX-512 vectors of u8, but this is really only temporary until we can get an LLVM fix (and get that fixed version into rustc)

There's an LLVM bug filed (though I'm not 100% sure it's the same bug).  I still need to file the rust and stdsimd bugs to track it.

Filed #90 and rust-lang/rust#84020.


Tensorflow lite conversion problem

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (or github SHA if from source):

Provide the text output from tflite_convert
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime and are not recognized by TensorFlow. If you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: FULLY_CONNECTED, RESHAPE, TRANSPOSE. Here is a list of operators for which you will need custom implementations: EmptyTensorList, TensorListFromTensor, TensorListReserve, TensorListStack, While.

Graph code:
self.inputs = tf.placeholder(tf.float32, [None, None, num_features], name='inputs')
    # # Here we use sparse_placeholder that will generate a
    # # SparseTensor required by ctc_loss op.
    # # https://www.tensorflow.org/api_docs/python/tf/sparse/SparseTensor
    # # https://www.tensorflow.org/api_docs/python/tf/nn/ctc_loss
    self.targets = tf.sparse_placeholder(tf.int32, name='targets')

    # # 1d array of size [batch_size]
    self.seq_len = tf.placeholder(tf.int32, [None], name='seq_len')
    self.lstm_cell = tf.lite.experimental.nn.TFLiteLSTMCell(num_hidden)
    self.outputs, _ =tf.lite.experimental.nn.dynamic_rnn(self.lstm_cell,self.inputs,dtype='float32')

    self.shape = tf.shape(self.inputs)
    self.batch_s, self.max_time_steps = self.shape[0], self.shape[1]

    # Reshaping to apply the same weights over the timesteps
    self.outputs = tf.reshape(self.outputs, [-1, num_hidden])

    # Truncated normal with mean 0 and stdev=0.1
    # Tip: Try another initialization
    # see https://www.tensorflow.org/versions/r0.9/api_docs/python/contrib.layers.html#initializers
    self.W = tf.Variable(tf.truncated_normal([num_hidden,
                                         num_classes],
                                        stddev=0.1))
    # Zero initialization
    # Tip: Is tf.zeros_initializer the same?
    self.b = tf.Variable(tf.constant(0., shape=[num_classes]))

    # Doing the affine projection
    self.logits = tf.matmul(self.outputs, self.W) + self.b

    # Reshaping back to the original shape
    self.logits = tf.reshape(self.logits, [self.batch_s, -1, num_classes])

    # Time major
    self.logits = tf.transpose(self.logits, (1, 0, 2))

    self.logits = tf.identity(self.logits,name="output")

    self.loss = tf.nn.ctc_loss(self.targets, self.logits, self.seq_len)
    self.cost = tf.reduce_mean(self.loss)

    with tf.variable_scope("gs"):
        self.global_step = tf.Variable(0, name='global_step', trainable=False)

    # optimizer = tf.train.AdamOptimizer().minimize(cost)
    # optimizer = tf.train.MomentumOptimizer(learning_rate=0.01, momentum=0.9).minimize(cost)
    self.optimizer = tf.train.AdamOptimizer(learning_rate=5e-4)
    self.gvs = self.optimizer.compute_gradients(self.cost)
    self.clipped = []
    for grad, var in self.gvs:
        grad = tf.clip_by_value(grad, -1., 1.)
        self.clipped.append((grad, var))
        self.train_op = self.optimizer.apply_gradients(self.clipped, global_step=self.global_step)

Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Hi,
The conversion fails because it has a dynamic rnn in your model, and the ops used by the rnn are 'TensorList' ops and while op, which is not supported by TOCO converter.
We are currently working on a new converter that could handle all these use cases. Please stay tuned!

But I am using tf.lite.experimental.dynamic_rnn which is supported as TFLite Op as given in the documentation. Can you please check this link https://github.com/tensorflow/tensorflow/tree/r1.14/tensorflow/lite/experimental/examples/lstm/g3doc
According to this doc we can convert dynamic_rnn to tensorflow lite if we use experimental dynamic_rnn op instead of tf.nn.dynamic_rnn.
Also please check this tutorial given by tensorflow team https://github.com/tensorflow/tensorflow/blob/r1.15/tensorflow/lite/experimental/examples/lstm/TensorFlowLite_LSTM_Keras_Tutorial.ipynb. They succesfully converted to TFLite model

Which version of tensorflow are you using?
could you try repeating the steps here:
https://github.com/tensorflow/tensorflow/tree/r1.14/tensorflow/lite/experimental/examples/lstm/g3doc#0-turn-on-control_flow_v2
Could you also share the converter command you use?

Tensorflow Version - 1.14.
For saving the model I used simple_save() function.
For freezing the graph I used :
python -m tensorflow.python.tools.freeze_graph --input_saved_model=saved_model.pb --output_node_names="output"
This is the code I used to converted the frozen graph into tflite model.
graph_def_file = pbfile_path
input_arrays = ["inputs"]
output_arrays = ["output"]
    converter = tf.lite.TFLiteConverter.from_frozen_graph(
      graph_def_file, input_arrays, output_arrays,input_shapes = {"inputs":[1,100,13]})
    converter.allow_custom_ops=False
    converter.target_ops = [tf.lite.OpsSet.TFLITE_BUILTINS,tf.lite.OpsSet.SELECT_TF_OPS]
    #converter.optimizations = [tf.lite.Optimize.DEFAULT]
    #converter.post_training_quantize=True
    #converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
    tflite_model = converter.convert()

    open("output.tflite", "wb").write(tflite_model)

Thanks for the info.
I'm wondering if there are certain corner cases caused the conversion to fail. Renjie, could you help also take a look?

Hi, you don't need to manually freeze the graph, you can just
use converter = tf.lite.TFLiteConverter.from_session(sess, input_arrays, output_arrays,input_shapes = {"inputs":[1,100,13]})
like here
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/experimental/examples/lstm/g3doc#3-lets-define-the-export-to-tensorflow-lite-model-function
also please note tf.lite.experimental.nn.dynamic_rnn only accepts time_major=True (and it's the default value).

Yeah its working I can directly convert to tensorflow lite model from session.
But what is the problem with above process I did?

Great to know it's working. :)
If you converted from frozen graph, you need to use tf.lite.experimental.convert_op_hints_to_stubs
like here:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/python/op_hint.py#L36-L40
using 'tf.lite.TFLiteConverter.from_session' is much simpler. :)

Ok. Thanks for the info.

Hey @renjie-liu  tf.lite.TFLiteConverter.from_session doesn't have any parameter as input_shapes. Can you please check it again.

@renjie-liu   #33429 Can you please reopen this issue

are you trying to use a different shape? from_session takes in tensors, which have shapes implicitly

Yes I am using different shapes. Because at training stage I am having shape of [None,None,13] which is not accepted by tflite converter as it is having shape of None in 2nd dimension.

I see.
I think you have two options:

freeze the graph & call tf.lite.experimental.convert_op_hints_to_stubs then pass to tfliteconverter

refactor your model building structure, yes you have [None,None,13] for training, but export a fixed-shape graph for inference

How to use tf.lite.experimental.convert_op_hints_to_stubs with tf.lite.TFLiteConverter. Can you give me a sample code.
I will be trying the 2nd option mean while

Second option worked for me. Its fine. It will be better if you share some details regarding how to use 1st option.

please check here: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/python/op_hint.py#L38-L41
but ignore the toco_convert (should be tfliteconverter)

Ok Thanks.


After joining challenge - route to team selection since you have to do that after joining a challenge

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Describe alternatives you've considered
Additional context

Done.  Routed to teams after joining challenge.  After joining team, routed to dashboard


AppExecutionAlias over SSH

I've been trying to run a console app with an AppExecutionAlias in PowerShell via SSH. It does not seem to recognize the aliases. E.g this would be perfect to create a unit-test app to run over Jenkins.
I ssh into my jenkins agent and start powershell. Installing the app works perfectly fine but when running it with the alias it fails with CommandNotFoundException. Anyone else have this experience?

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

ID: a18c877b-42e4-a463-8092-72520d8c8915
Version Independent ID: b35cd713-4d41-f4c6-b559-84f8da3d14cf
Content: Create a Universal Windows Platform console app - UWP app developer
Content Source: windows-apps-src/launch-resume/console-uwp.md
Product: windows
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Hi @aronyu79 -- thanks for your comment. It looks like you're hoping for some dev community help? This channel is really for feedback specific to our docs...have you tried the Windows UserVoice? https://wpdev.uservoice.com/. If you do have doc feedback, please let us know.


test

Codecov Report

Merging #5 (26be70e) into main (80abf58) will decrease coverage by 6.90%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
- Coverage   97.22%   90.32%   -6.90%
==========================================
  Files           2        2
  Lines          36       31       -5
==========================================
- Hits           35       28       -7
- Misses          1        3       +2

Files Changed
Coverage Δ

api/calculator/test_calculator.py
100.00% <ø> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Illegal calls to romdisk_register()

Several examples (and other things under apps/) make illegal calls to romdisk_register().  This both violates the portable  POSIX OS interface and makes these applications un-usable in PROTECTED and KERNEL build modes.
Non-compliant examples include:

examples/bastest, examples/elf, examples/module,
examples/nxflat, examples/posix_spawn, examples/romfs,
examples/sotest, examples/thttpd, examples/unionfs

These examples are simple demos and, hence, you could argue that it is not so bad that they violate the interface for the purpose of demonstration (although they do set a bad example because of  this).

Let's use https://github.com/apache/incubator-nuttx-apps/issues/246 track the problem and close this issue.


Issues while doing aws migration from hyper-v

Even though I am giving correct management console password getting this error

is there any way to copy paste commands to hyper-v guest vms (for example i want to create mysql db in our one of the gues VM database , but copy pste is not working from hyper-v host to guest in all 3 vms)

Please help.

install gitbash on host machine and the buse scp command the syntax look like this
scp <sourcefile> <destination>
for example;
scp  test.txt destinationuser@destinationvmip:/home/user
if this is helpful to you please close the issue.


read data from serial

Hi,
how I can to read with arduino interface, data from serial input (RX) and put it on display ?
for example, a device send to Nextion  numbers from 1 to 9 like   123456789, and I need to see it on diplay.
please a sample code..

and if you want to send data from nextion to arduino use something like:
printh 65 01 01 00 ff ff ff   <---- this will triget event handler function for page 1 component 1
get vFeedIgnWork.val <---- this you can read with nxRecvRetNumber function


tag v0.01 does not exist

Taken From Readme
[dependencies.tauri-plugin-store]
git = "https://github.com/tauri-apps/tauri-plugin-store"
tag = "v0.1.0"
#branch = "main"

My Cargo.toml:

[package]
name = "app"
version = "0.1.0"
description = "A Tauri App"
authors = ["you"]
license = ""
repository = ""
default-run = "app"
edition = "2018"
build = "src/build.rs"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[build-dependencies]
tauri-build = { version = "1.0.0-beta.4" }

[dependencies]
serde_json = "1.0"
serde = { version = "1.0", features = ["derive"] }
tauri = { version = "1.0.0-beta.8", features = ["api-all", "system-tray"] }

[features]
default = [ "custom-protocol" ]
custom-protocol = [ "tauri/custom-protocol" ]

[dependencies.tauri-plugin-store]
git = "https://github.com/tauri-apps/tauri-plugin-store"
tag = "v0.1.0"
#branch = "main"

Results in the error:

error: failed to get `tauri-plugin-store` as a dependency of package `app v0.1.0 (D:\Programming\Github Restoration Project\Dynamic-Wallpaper\src-tauri)`

Caused by:
  failed to load source for dependency `tauri-plugin-store`

Caused by:
  Unable to update https://github.com/tauri-apps/tauri-plugin-store?tag=v0.1.0

Caused by:
  failed to find tag `v0.1.0`

Caused by:
  reference 'refs/remotes/origin/tags/v0.1.0' not found; class=Reference (4); code=NotFound (-3)

This repo has no tags or releases and the latest commit causes version conflicts with the latest tauri release version

Basically the same i said on discord, posted if someone else has the same problem.
Yeah that's because we're just copy-pasting the readme 😅
You can just remove the tag line to make it work. Speaking of "making it work", this plugin requires you to use the git version of tauri, but we're in the process of preparing the first 1.0 rc out so there are massive (ongoing) changes.
These changes currently prevent the js/npm cli from working until the next release (among other things not working perfectly), so imo the way with the least friction is to wait just a tiny bit longer for the rc.


CIS Check  Ensure 'AUTO_CLOSE' is set to 'OFF' on contained databases - CIS check to only check contained databases

Ensure 'AUTO_CLOSE' is set to 'OFF' on contained databases - CIS check to only check contained databases

PR #721 takes care of this.


Refactor Base.get_or_create to use EXISTS function

Instead of counting, leveraging the SQL EXISTS function could improve performance.
http://techspot.zzzeek.org/2008/09/09/selecting-booleans/

Implemented this in commit 5f32ca6. This improved performance by about 30%.


Please add the assignment operator to hip_bfloat16

Please add the assignment operator and bool conversion to hip_bfloat16. It would be usefull in rocPRIM and hipCUB.

We will change the rocPRIM and we will handle the bfloat16 without the extra assigment operator.


kh_get question

Hello,
I have a question about how kh_get is implemented in khash. https://github.com/attractivechaos/klib/blob/master/khash.h#L237 (Line 237)
Before the loop runs, the last index is set to the starting value of i.
In the worst case scenario, the loop would have to go through all the elements.
But as the code is currently written, I believe the loop will make 2*h->n_buckets -1 iterations.
If the assignment of last is moved inside the loop, the iterations should become h->n_buckets.
So my question is whether I'm wrong in this assumption?
Also, if my math is correct, using if(step == h->n_buckets) should also work (which would also remove the need for the last variable. Or maybe it should be if(step > h->n_buckets).

See the comment at line 51, in particular the quote from this post:

if the hash table size is a power of two and the probe function is p(K, i) = (i*i + i)/2, then every slot in the table will be visited by the probe function.

I have not mathematically proved this claim, but it is easy to empirically test it with a program:
#include <stdio.h>
#include <stdlib.h>
int main(int argc, char *argv[])
{
  int i, n = 1<<10;
  if (argc > 1) n = 1<<atoi(argv[1]);
  for (i = 0; i < n; ++i)
    printf("%ld\n", ((long)i*(i+1)/2)&(n-1));
  return 0;
}

You will see the output has no duplicates.
Khash is using power-of-2 number of buckets. This means that the loop at line 237 is guaranteed to traverse each bucket once in the worse case. It takes at most h->n_buckets iterations. Yes if(step == h->n_buckets) should work. Testing last is a remnant of older versions.

Oh, wait... You are right and I was wrong. It takes more than h->n_buckets in the worst case. The better way is if(step == h->n_buckets).

The same thing is used in kh_put.
I've got an other question, though.
This time, taking into account the whole code, not just kh_get:
The minimum number of buckets is 4 (or 0, but that case is handled). Looking at the way the resizing logic is written, will there ever come a situation, when all the buckets have to be iterated through?
The code is written in such a way, that there is always at least one empty or deleted bucket. So shouldn't the clause at the while loop be enough to handle all cases, without the need for the extra if and break?

I was thinking that, too. If the program is correctly implemented elsewhere, we should not need that if(i==last) line. On my test, with or without this line makes little difference in performance.
Thanks a lot for this.

Great.
I have two more questions (and I should probably create a new issue, but they seem a bit related). These should be the last questions I bother you with.
About https://github.com/attractivechaos/klib/blob/master/khash.h#L328 (Line 328).
kh_put uses the same logic, but removing the if (i == last) past shouldn't be possible with a minimum size of 4. If the minimum size is changed to 8, then by my understanding it should be okay to remove that line. And if it is removed, then if (x == h->n_buckets) should go as well. Is this correct?
Second question:
The site variable is used for the last index with a deleted bucket. Would it not be better for future lookups, if new elements were inserted not at the last deleted bucket index in the chain, but at the first. If I'm correct, this shouldn't break the hash table as a whole and it should speed-up future lookups.
To accomplish this, I think of changing this line:
if (__ac_isdel(h->flags, i)) site = i;
to
if (__ac_isdel(h->flags, i) && site == h->n_buckets) site = i;
or
if (site == h->n_buckets && __ac_isdel(h->flags, i)) site = i;
I believe the first the be faster.
Am I wrong on this one? Will using the first deleted bucket index make the hash table unusable?

Great.
I have two more questions (and I should probably create a new issue, but they seem a bit related). These should be the last questions I bother you with.
About https://github.com/attractivechaos/klib/blob/master/khash.h#L328 (Line 328).
kh_put uses the same logic, but removing the if (i == last) past shouldn't be possible with a minimum size of 4. If the minimum size is changed to 8, then by my understanding it should be okay to remove that line. And if it is removed, then if (x == h->n_buckets) should go as well. Is this correct?
Second question:
The site variable is used for the last index with a deleted bucket. Would it not be better for future lookups, if new elements were inserted not at the last deleted bucket index in the chain, but at the first. If I'm correct, this shouldn't break the hash table as a whole and it should speed-up future lookups.
To accomplish this, I think of changing this line:
if (__ac_isdel(h->flags, i)) site = i;
to
if (__ac_isdel(h->flags, i) && site == h->n_buckets) site = i;
or
if (site == h->n_buckets && __ac_isdel(h->flags, i)) site = i;
I believe the first to be faster.
Am I wrong on this one? Will using the first deleted bucket index make the hash table unusable?

Wouldn't it seem better if we directly use:

while(!__ac_iseither(h->flags, i)) {
if(__hash_equal(h->keys[i], key)
{ *ret = 0; return i; }
i = (i + (++step)) & mask;
}


Object Spread Operator Plugin does not work on Version 7+

Hi,
I just upgrade to version 7, and my ember-cli-build.js has this
babel: {
      plugins: ['transform-object-rest-spread']
},

and I have the plugin installed on package.json
But instead it show this error when I build the app.
Build Error (StubGenerator)

Error parsing code while looking for "npm:" imports in file: /home/w3o-paska/workspace/sbiz-payment-devtest/client/tmp/stub_generator-input_base_path-1OrjpvkS.tmp/client/pods/general-payment/checkout/route.js
SyntaxError: Unexpected token (25:33)
    at Parser.pp$4.raise (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:2757:13)
    at Parser.pp.unexpected (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:647:8)
    at Parser.pp$3.parseIdent (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:2712:10)
    at Parser.pp$3.parsePropertyName (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:2496:105)
    at Parser.pp$3.parseProperty (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:2426:8)
    at Parser.pp$3.parseObj (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:2380:23)
    at Parser.pp$3.parseExprAtom (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:2179:17)
    at Parser.pp$3.parseExprSubscripts (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:2047:19)
    at Parser.pp$3.parseMaybeUnary (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:2024:17)
    at Parser.pp$3.parseExprOps (/home/w3o-paska/workspace/sbiz-payment-devtest/client/node_modules/acorn/dist/acorn.js:1966:19)

Stack Trace and Error Report: /tmp/error.dump.3607a46632af708cfc87f493a638308b.log

I revert to version 6.6.0, and it is working normally.
Is this a new bug on version 7 ?
Thank you !

You need to update the Babel plugin you are using for rest spread to the one provided by Babel. Specifically it should be within the @babel scope...

Also, that error seems to be coming from ember-browserify which is having a hard time processing the file which has rest/spread (it seems)...

@rwjblue
Thank you for reply, it works after I remove ember-browserify dependency.
Since it is already deprecated and replaced by ember-auto-import, then it is ok to remove this.
Thank you !


Filtering checks

@codequest-eu/developers PTAL
Related to: https://github.com/codequest-eu/codeguard/issues/11

@kruszczynski PTAL


Ensure code is formatted properly

Black should be setup. We can re-use the configuration from MONAI if convenient.

Closing as flake8 checks are now enabled in CI


API link called only once.

Your Environment

Plugin version: 0.2.0-alpha.5
Platform: iOS and Android
OS version:
Device manufacturer and model:
React Native version: 0.40
Plugin configuration options:
Link to your project:

Context
I was trying to use it for testing and I just follow basic example in the documentation it shows the geolocation started in the debug log and also send the info to api (I add a url) but only once after that it never call the Api or never show anything in debug am I missing something ?
Expected Behavior

Calling the API link after x milliseconds based on interval
Actual Behavior

only call the API link once on start.

Notice: this issue has been closed because it has been inactive for 276 days. You may reopen this issue if it has been closed in error.


Convert the last (hopefully) Cert API to util::Status and update callers.

I think this is the last change before cleanup.

StatusOrBoolToCertStatus() and CertStatusToStatusOrBool() are now unused, can you ding 'em? (they're causing the compiler to barf because -Werror)

Ah. I've got a separate cleanup commit. Looks like I'll have to bring part of it into this then.

I think I fixed all your comments, though I probably should have held off until you'd finished. PTAL.

Couple more var name change suggestions, but LGTM otherwise.

Latest renamings done and pushed, both those should now be fixed.


How disable tooltip by other button

i have a list with tooltips on  i want to disable and enable these tooltips by an other button.

On button click you can call
$('.element').tooltipster('enable');
and
$('.element').tooltipster('disable');


DI Refactor

Refactored the DI container usage according to the recommended PHP-DI usage.

Manually merged.


Bug: Multiline Quotes are difficult

https://user-images.githubusercontent.com/2949816/186534695-dd903009-bbea-449e-bd41-de7040219e31.mov

pressing enter in a quote-block, returns to normal paragraph

unfortunately also when pressing enter in the middle of the text of a quote

it appears impossible to have a multi-line (returns) in a quote. the quote line on the left gets disconnected.

It would be nice to be able to create quotes with multiple lines. If we want quotes to close on enter press, we should at least wait for two enters. We should also connect quote blocks that are siblings into one.
cc @zurfyx

Can’t you use shift+enter?

Yes, that works. I would not have discovered it.
I wonder why bulleted, numbered and check lists and code blocks allow to simply press enter (without shift). What is different about quotes, that they need to behave differently?

Shift+Enter is quite a well known key command to add soft line breaks. Lists are different because you're actually inside a list block, instead of the root. So the context of the list is to add a new item to that list. This is how other text editors work too.
Code blocks are handled with a special case here to emulate code editing, but in most editors they also work like how quote/heading nodes work too.

Closing this task as I feel like this is fairly standard behaviour, replicable on Slack and Discord.


Config failed for Replica dataset

Hi @ChanganVR , when I tried to run the Soundspace 2.0 tutorial jupyter notebook on the Replica dataset, I keep get the errors reported that
[Warning]:[Scene] SemanticScene.h(328)::checkFileExists : ::loadSemanticSceneDescriptor:
Filedata/scene_datasets/office_4/mesh.scn does not exist.  Aborting load.

Though the Replica room has a specified semantic decriptor as the info_semantic.json as shwon below
replica_stage.stage_config.json:
{
  "render_asset": "../mesh.ply",
  "semantic_descriptor_filename": "info_semantic.json",
  "semantic_asset": "mesh_semantic.ply",
  "nav_asset": "mesh_semantic.navmesh"
}

The problem is that habitat-sim seems to keep searching for mesh.scn as a semanic descriptor.
Has anyone encountered this before? What should I do to enable SpundSpace2.0 on this Replica dataset?

I don't think we've got material rendering work for Replica due to some reasons most likely this. Turning off material rendering should make it work on Replica.


Update XSSFSheet.cs

Fix for https://github.com/nissl-lab/npoi/issues/450

Please write a unit test to do code coverage and make sure the logic is correct

Sorry but I dont really have the time to get that deep into your build infrastructure and test layout.

hmm..... I'm not giving you pressure but since NPOI has too many users, it may be a critical bug if you change anything. Unit test is a way to make this kind of bug disappear.


[Idea] Leverage TMP environment variable to control placement of DynamicWrapperX

The current use of the Activation Context Manifest actctx objects, allows you to control the DLL location and search order.
By default this is in the AppData\Temp location.
However you can override this by setting TMP at runtime.
new ActiveXObject('WScript.Shell').Environment('Process')('TMP') = 'c:\\Tools\\';

As an example, here is a poc to drop the DLL to C:\Tools
https://gist.github.com/caseysmithrc/55ea32aae02f5aa1498982663ccba1e7
You could extend this to be configurable I suspect.
Anyway, just an idea.
Cheers,

We definitely should fix that.
At the same time we're also not using ManifestText, which we should instead of dropping the file.

I think you could also, consider using this for x64 systems.
https://github.com/ctxis/DynamicWrapperEx
x64 Registration-Free In-Process COM Automation Server.
I can open another issue if you want, this is an open source version of DynamicWrapperX and probably a better choice.  Might require a bit of refactoring to check architecture of the target, and some of the API calls.
Just an idea.
Cheers.

We were looking to replace DynWrapperX with our own TashLib, although hit a couple snags (i.e. #86). I'm sure an afternoon windbg session would uncover the issues, I just personally haven't had the willpower to dig through the COM swamp apparantly.
The main idea with starting TashLib was to stop burning other people's stuff. Unclear what ctxis's intent is in regards to that, and who even knows what GPL means to us. It's true that the closed source DynWrapperX isn't ideal but I did at least meagerly RE it when we put it in.
I think the main issue with the ctxis version is it's x64 only (which I mean we could always use as a jumping off point to make a x86 version). We're still popping a lot of x86 with dynwrapx on pentests. Our mimikatz/dynwrapx shim for example was coded to explicitly launch in x86 mode on all machines and still be able to dump a x64 lsass.
Thanks for making us aware of the ctxis project, slipped under my radar at least and very cool to see someone publish. I'm still inclined to (one day) get TashLib working correctly. A working project being available is a great reference.

That makes sense.
Completely understand, there is a law of diminishing returns on COM expertise lol.
Just wanted to point it out. I like the ctxis is open source at a minimum.
There is always these DLLs from 1998 ;-)
http://www.borncity.com/web/WSHBazaar1/WSHDynaCall.htm
Keep up the great work. Cheers.


[Bug]: SHAFT_ Reporting _ no screenshot added to allure report in case of "multiple elements found matching this locator" exception

Describe the bug.
no screenshot added to allure report in case of "multiple elements found matching this locator" exception
Link to SHAFT_Engine Console logs
https://gist.github.com/zanoon2020/2bd483d32cf2a65702ca33d94d3d87dd
Environment

SHAFT_Engine version that exhibits the issue (Note: We support only the latest release):7.2.20230411
Last SHAFT_Engine version that did not exhibit the issue (if applicable):
Desktop OS: [e.g. iOS]windows 10
Desktop Browser (if applicable): [e.g. chrome, safari] all browsers

To Reproduce
Steps to reproduce the behavior:

in VisaulValidations.properties >  set screenshotParams_whenToTakeAScreenshot=ValidationPointsOnly
run any test that will fail due to multiple element found exception

for exmaple
 public void test_ClickUsingJavaScript() {
        driver.browser().navigateToURL("https://www.saucedemo.com");
            driver.element().type( By.xpath("//input"), "standard_user")
}

Actual behavior
no screenshot attached at failed step in allure report
failed step duplicated at allure report
Expected behavior
test will fail and screnshot from faluire will be attached to allure report as all other exceptions and step displayed one time not twice as appear in screenshot
Screenshots
If applicable, add screenshots, gifs, or video recordings to help explain the issue you're facing.

.

@MohabMohie , screenshot added but new issue appeared
the green mark and step log not appear

Mmm... Would you share the code snippet to reproduce please?
It's working fine on the code you provided me with above... But I'm sure there are some uncovered cases.

public void test_ClickUsingJavaScript() {
driver.browser().navigateToURL("https://www.saucedemo.com");
driver.element().type( By.xpath("//input"), "standard_user")
}

That's exactly the same code I'm using...
😄 Now I feel lost...
Would you kindly delete your properties directory and try again on the branch that's linked to this issue?

duplication error fixed but no screenshot
new log
https://gist.github.com/zanoon2020/927df25868163f97c74983274952ec41

@MohabMohie i will say weird , it works with type action and not working with click action

okay... I refactored the entire element actions failure logic, and tested it with type and click, for multiple elements found, and for no such element exceptions.
would you take another look now please?
I added a couple of test classes... please feel free to add more test methods as you see fit to cover the cases you can identify.

Dear @MohabMohie , sorry for late replay , kindly advice on how to test , i updated my fork and run my test but for some resaon i got same result . what i'm missing here

Please delete any local branches you have, and retest on the main branch.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Ahmed Zanoon @.>
Sent: Friday, April 14, 2023 5:00:08 PM
To: ShaftHQ/SHAFT_ENGINE @.>
Cc: Mohab Mohie @.>; Mention @.>
Subject: Re: [ShaftHQ/SHAFT_ENGINE] [Bug]: SHAFT_ Reporting _ no screenshot added to allure report in case of "multiple elements found matching this locator" exception (Issue #1052)
Dear @MohabMohiehttps://github.com/MohabMohie , sorry for late replay , kindly advice on how to test , i updated my fork and run my test but for some resaon i got same result . what i'm missing here
[image]https://user-images.githubusercontent.com/65794900/232080443-424bd62e-a31f-4b57-a4f0-f54525aeb391.png
—
Reply to this email directly, view it on GitHubhttps://github.com/ShaftHQ/SHAFT_ENGINE/issues/1052#issuecomment-1508715893, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AESP62TNFUOELRGHYLNRFLLXBFQ7RANCNFSM6AAAAAAW4ETXCM.
You are receiving this because you were mentioned.Message ID: @.***>


⚠ GitLab Server has degraded performance

In cf083fc, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11468 ms

Resolved: GitLab Server performance has improved in 2b9517f.


[fix] using ${var} in strings is deprecated

Using ${var} in strings is deprecated with php 8.2

Test failed. Need to fix.
https://github.com/catfan/Medoo/actions/runs/3846360436/jobs/6555788430

Test failed. Need to fix.
https://github.com/catfan/Medoo/actions/runs/3846360436/jobs/6555788430

I saw, I'm try to fix

Hello,
Tests passed. Could you please merge this PR :)?
Thank you,
Enzo


Update encrypted travis data

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[ ] #patch# - backwards-compatible bug fix
[x] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

This is an aggregation of the changes from all the numerous PRs that have been merged while version-bumping and publishing has been broken.
CHANGELOG

Respect .remarkignore file for lint-markdown
Changed maximum-line-length for Markdown linter to 120
Document ability to use .eslintignore file with Javascript linter
Fixed link in documentation
Fixed #67
Fixed #65

:+1:

:+1:


text.split is not a function

System Info
transformer.js version: 2.14.0
Framework: React (18.2.0)
Browser: Chrome (120.0.6099.218)
Node.js version: 20.2.0
Environment/Platform

[X] Website/web-app
[ ] Browser extension
[ ] Server-side (e.g., Node.js, Deno, Bun)
[ ] Desktop app (e.g., Electron)
[ ] Other (e.g., VSCode extension)

Description
I am attempting to use the crossencoder-distilcamembert-mmarcoFR model as a re-ranker in React, following the provided model card and inference code. The original Python code using the transformers library functions as expected, but when translating it to React using transformer.js, I encounter a TypeError related to the PreTrainedTokenizer constructor.
Model Information

Original model: crossencoder-distilcamembert-mmarcoFR
ONNX model: crossencoder-distilcamembert-mmarcoFR_ONNX

Python code from the model card
from transformers import AutoTokenizer, AutoModelForSequenceClassification
import torch

model = AutoModelForSequenceClassification.from_pretrained('antoinelouis/crossencoder-distilcamembert-mmarcoFR')
tokenizer = AutoTokenizer.from_pretrained('antoinelouis/crossencoder-distilcamembert-mmarcoFR')

pairs = [('Query', 'Paragraph1'), ('Query', 'Paragraph2') , ('Query', 'Paragraph3')]
features = tokenizer(pairs, padding=True, truncation=True, return_tensors='pt')

model.eval()
with torch.no_grad():
    scores = model(**features).logits
print(scores)

Reproduction

Use the provided React code to load the model and tokenizer.
Attempt to tokenize a nested array as shown in the code snippet.
Observe the encountered error.

Code Snippet
useEffect(() => {
    const classifySequence = async () => {
        const modelName = "Oblix/crossencoder-camembert-base-mmarcoFR_ONNX";
        const model = await AutoModelForSequenceClassification.from_pretrained(modelName);
        const tokenizer = await AutoTokenizer.from_pretrained(modelName);
        const pairs = [
            ["Quelle est la capitale de la France?", "La capitale de la France est Paris"]
        ];
        try {
            const input = await tokenizer(pairs, {
                padding: true,
                truncation: true
            });
            const output = await model(input);
            console.log(output);
        } catch (error) {
            console.error("Error:", error);
        }
    };

    classifySequence();
}, []);

Error

Hi there 👋 Due to the differences in how JavaScript and Python handle optional positional and keyword arguments, we modified the API slightly to account for this. See here for example usage:
import { AutoTokenizer, AutoModelForSequenceClassification } from '@xenova/transformers';

const model = await AutoModelForSequenceClassification.from_pretrained('Xenova/ms-marco-TinyBERT-L-2-v2');
const tokenizer = await AutoTokenizer.from_pretrained('Xenova/ms-marco-TinyBERT-L-2-v2');

const features = tokenizer(
    ['How many people live in Berlin?', 'How many people live in Berlin?'],
    {
        text_pair: [
            'Berlin has a population of 3,520,031 registered inhabitants in an area of 891.82 square kilometers.',
            'New York City is famous for the Metropolitan Museum of Art.',
        ],
        padding: true,
        truncation: true,
    }
)

const scores = await model(features)
console.log(scores);
// quantized:   [ 7.210887908935547, -11.559350967407227 ]
// unquantized: [ 7.235750675201416, -11.562294006347656 ]

This is what is was looking for, although it doesn't seem to work for this specific model crossencoder-distilcamembert-mmarcoFR_ONNX (it does work for ms-marco-TinyBERT-L-2-v2 and all its MiniLM variants).
I will be using TinyBERT since it seems to be performing really well for my use case!
Just a quick question, is the example you gave me (or a similar one) available in the documentation? If not I think that would be a great addition since almost every cross-encoders need to tokenize text pairs.
Thank you for this great library by the way :heart:

@Maxzurek I guess I had the same issue than you when using https://huggingface.co/Oblix/crossencoder-distilcamembert-mmarcoFR_ONNX (logits property is returned undefined).
But just curious if you know more about why : what could explain this model cannot be used? Is it something related to the initial model https://huggingface.co/antoinelouis/crossencoder-distilcamembert-mmarcoFR when built ? Or is Transformers.js is not yet taking into account all cases for models?
By the way @xenova , thanks for porting ML stuff to the JS/TS environment, it's very valuable!

@sneko It might be because the model is not yet supported by Transformers.js, but @xenova might be able to answer better. I've had good results so far with xenova/ms-marco-TinyBERT-L-2-v2, although I don't think the model is multilingual (I use it mostly for English and French).


Assertion `Ty && "Invalid GetElementPtrInst indices for type!"' failed

Kernel: https://gist.github.com/PlasmaPower/9c8abea0a6bdb31e76cb5b3b74a08661
This one takes a while to crash, so I might have a harder time reducing it. I'll look into making a release build with assertions enabled.

I've reduced it down to 50 lines: https://gist.github.com/70976b869f30c3036de3d652426101c8
Could probably be reduced further, but I'm a bit short on time atm.


[Feature]: Introduce auto scroll in property pane when editing any property

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Introduce Auto scroll in the property pane when a user is editing any property so that we do not have to manually scroll down for additional fields.
https://www.loom.com/share/e8d2e87300e54d9e80c634fc6f2e4320
Why should this be worked on?
Right now when we try to edit any property (that is situated near the bottom right corner of the screen) in the property pane, we have to manually scroll down in order to customise it. Introducing auto scroll will eliminate the need for us to do so and the related fields will be automatically visible.

I think this is solved as part of context switching so closing it

I think this does not fall under the scope of context switching.
@rahulramesha Can you tell us whether this is taken care of in the context-switching project?


Native Linux and Windows

Add support to native Linux and windows

It's support native Linux and windows from v1.0.1

Can't work under pure dart (dart native)

This package should be able to work in pure Dart with little modification.


[nginx-ingress-controller] Nginx status page and Prometheus metrics

I'm having issues locating the nginx-status page as described here

The documentation says to look at the example/rc-default. However, I cannot find this example. I would like the status page to be accessible on a specific url: nginx.example.com via the ingress-controller. Would that be possible?
Further, I would like my Prometheus setup to be able to use the metrics as well. I see that there is an exporter that can export metrics from the nginx_status. However, since I'm using 0.9.0, and this comes with Prometheus metrics, which works great by the way, but missing more detailed metrics such as HTTP_500_errors etc. Is this on the drawing board for an upcoming release?
I know the repo is in a transformation stage at the moment. If anybody can give me some pointers, that would be great. Thank you.

@kaspernissen right now the prometheus exporter uses the standard nginx status page. There is an issue to use the vts information. Please check https://github.com/kubernetes/ingress/issues/72


Remove unused file

pytorch_pfn_extras/dataloaders/_utils is not exposed to users, and is not used from anywhere internally.

/test


Just Ported Everforest to Gnome Terminal, You are Free to Use!

https://github.com/em3n/Everforest-GnomeTerminal (Hopefully Owner will Add Me to Wiki)

You should be able to edit the wiki page though.
I've added your repository to the wiki page, thanks for your contributing ;)


New DDI fields @ CDC

New DDI Fields are to be added @ CDC as part of the EOSC project with ARIA and OpenAIRE. This is a preparation of how will these be represented.
CDC will use the same Access Rights vocabulary as OpenAIRE. The name of the vocabulary is “info:eu-repo-Access-Terms vocabulary”. Only the codes “restrictedAccess” and “openAccess” will be used.
/codeBook/stdyDscr/dataAccs/useStmt/restrctn -> "general free text"
/codeBook/stdyDscr/dataAccs/useStmt/conditions -> OpenAIRE Vocabulary
/codeBook/stdyDscr/dataAccs/useStmt/specPerm/@URI -> application form URL
/codeBook/stdyDscr/dataAccs/useStmt/specPerm  -> more information
examples
<conditions elementVersion=”info:eu-repo-Access-Terms vocabulary”>restrictedAccess</conditions>
<specPerm URI=”https://www.link-to-form.com”>some text</specPerm>
there is already a test set from SoDaNet Dev Datacatalogue
https://dev-datacatalogue.sodanet.gr/oai?verb=ListIdentifiers&set=SODADEV-CESSDA-ARIA&metadataPrefix=oai_ddi

@kpapag Please provide IDs of some studies that contain magic links

I cannot find either in https://datacatalogue-dev.cessda.eu (which harvested from the SoDaNet DEV endpoint last night).
Have you made them available via OAI-PMH?

I've added the new endpoint under the SoDaNet-ARIA code with a friendly name of "SoDaNet - ARIA DEV ENDPOINT". See https://github.com/cessda/cessda.cdc.aggregator.deploy/commit/b7215d1308cee5f2d3b6e5a4e3280bfd75e4d629.

@matthew-morris-cessda The UI updates need to be in place and tested by Monday 11 September at the very latest. Is that achievable?

@kpapag  I've noticed that the URIs are invalid. The errors reported are as follows

doi:10.17903/FK2/HP56HH - java.net.URISyntaxException: Illegal character in query at index 145: https://beta.structuralbiology.eu/submit-proposal/new?platform_reference=https://dev-datacatalogue.sodanet.gr/oai:ARIA-TEST&acid=165&machine=a:1:{s:9:"aria-test";a:1:{i:0;s:64:"https://dev-datacatalogue.sodanet.gr/oai:doi:10.17903/FK2/HP56HH
doi:10.17903/FK2/TOJ7AH - java.net.URISyntaxException: Illegal character in query at index 145: https://beta.structuralbiology.eu/submit-proposal/new?platform_reference=https://dev-datacatalogue.sodanet.gr/oai:ARIA-TEST&acid=165&machine=a:1:{s:9:"aria-test";a:1:{i:0;s:64:"https://dev-datacatalogue.sodanet.gr/oai:doi:10.17903/FK2/TOJ7AH

This is caused by the presence of { and } characters, which are disallowed in valid URIs (see https://www.ietf.org/rfc/rfc2396.html#section-2.4.3). The URI format should either escape these characters or use different ones.

Yes, these characters will be fine if escaped

Very quickly

An example of how this will look when fully implemented.

The URIs still have issues as they contain the " character, which isn't allowed.


fix(version): update fabric8-runtime-console to 0.8.35

There were the following issues with your Pull Request

Commit: 5bf60422b6f844e8b555162d87220fb515a6ef6b
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

There were the following issues with your Pull Request

Commit: 5bf60422b6f844e8b555162d87220fb515a6ef6b
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com


Python: modules

Description :
What is modules in python?
Note :

Changes should be made inside the Python/ directory & Python branch.
Issue only for GWOC'21 contributors.
Issue will be assigned on a first come first serve basis, 1 Issue  ==  1 PR.
In the PR, keep this issue's title as your PR's title.

Must Follow : Contributing Guidelines & Code of Conduct before start Contributing.

I would like to work on this please documentation
Peddi sai lohith
Python batch 4

I would like to work on this!!

I would like to work on this issue via documentation. Thank you!!

Hi I'd like to contribute through documentation on this topic
Harmandeep - python - batch 4

Interested in working on this issue via documentation

I would like to contribute for this. Please assign me this issue.

I would be thankful if you  assign me Documentation for this

I would like to work on this issue.
Name: Basudev Tyagi
Batch: Python Batch-3
Type: Documentation

I would like to work on this issue via documentation.

I would like to work on this issue.
Name: Saurav Kumar
Batch: Python Batch-5
Type: Documentation

@adityabisoi
Sir, I would like to contribute to this issue.
Name: Arushi Rautela
Type: Video
Batch: 6

@adityabisoi I would like to work on documentation for this issue
Name : Yash Kataria
Batch : 3

I would like to contribute to this issue in the form of documentation
kindly please assign me this issue.
Thanks
Ishu khare
batch-1 (python)

@Randrita Is the mentor for this issue. Please reach out to her in case of queries.

Hello
My name is Rohit Goyal
I would like to work on the documentation part
Pls assign me this issue
i am from Python Batch-6

@rohit-goyal already assigned for docs! choose another part!

@adityabisoi , Can I contribute on this issue through video.
python batch - 5
Thanks

/assign

@adityabisoi  is it open for video or doc plz let me know


Bring parse-LI8100.R coverage to 100%

Codecov Report

Merging #165 into master will increase coverage by 0.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #165      +/-   ##
==========================================
+ Coverage   95.41%   95.68%   +0.27%
==========================================
  Files           8        8
  Lines         719      719
==========================================
+ Hits          686      688       +2
+ Misses         33       31       -2

Impacted Files
Coverage Δ

R/parse-LI8100.R
100.00% <0.00%> (+2.15%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #165 into master will increase coverage by 0.27%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #165      +/-   ##
==========================================
+ Coverage   95.41%   95.68%   +0.27%
==========================================
  Files           8        8
  Lines         719      719
==========================================
+ Hits          686      688       +2
+ Misses         33       31       -2

Impacted Files
Coverage Δ

R/parse-LI8100.R
100% <100%> (+2.15%)
:arrow_up:

Continue to review full report at Codecov.

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


[Bug]: Dynamically added shadow DOM elements cannot be found

Minimal, reproducible example
import puppeteer from "puppeteer";
import fs from "fs";

const pageContent = fs.readFileSync("test.html").toString();
(async () => {
    const browser = await puppeteer.launch({ headless: false });
    const page = await browser.newPage();
    await page.setContent(pageContent);
    let shadowDiv = await page.waitForSelector("div[id=container]", {
        timeout: 5000,
        visible: true,
    });
    const shadowH1 = await shadowDiv.$eval(">>> h1", (h1) => h1.innerText);
    console.log(shadowH1);
    shadowDiv = await page.waitForSelector("div[id=container] >>> h2", {
        timeout: 10000,
        visible: true,
    });
    const shadowH2 = await shadowDiv.$eval(">>> h2", (h2) => h2.innerText);

    console.log(shadowH2);
    await browser.close();
})();

test.html:

DOM element

Background
I have narrowed down the regression point to version 22.12.1, more precisely commit (56d1d3f).
Before changes to polling methods, this scenario used to work as expected.
Expectation
After element is added with setTimeout function, puppeteer shall be able to find it using waitForSelector method.
Reality
After shadow element is dynamically added, it is not possible to retreive it.
Puppeteer configuration file (if used)
No response
Puppeteer version
23.5.0
Node version
20.17
Package manager
yarn
Package manager version
3.5.1
Operating system
Linux

This still does not work on 23.5.1
a bit more ergonomic replication:
import puppeteer from "puppeteer";
import fs from "fs";

const pageContent = `

DOM element

`;

(async () => {
    const browser = await puppeteer.launch({ headless: false });
    const page = await browser.newPage();
    await page.setContent(pageContent);
    page.setDefaultTimeout(2000);

    await Promise.all([
        page
            .waitForSelector(">>> h2")
            .catch(() => console.log("This fails because the element does not exist yet")),
        new Promise((resolve) => {
            setTimeout(() => {
                page.waitForSelector(">>> h2").then(() => {
                    console.log("This works because the element already exists");
                    resolve();
                });
            }, 1000);
        }),
    ]);

    await browser.close();
})();

The support for that is tracked here https://github.com/puppeteer/puppeteer/issues/13163 Currently, only the visible|hidden flags would change the default mode of polling (relying on the mutation observer).


Why the PathBase?

I'm probably missing something, but I don't understand the reasoning of having PathBase<T>, since there is only one derived class (Path)?

Fair question. The idea, originally, was that you could inherit from Path (or as it turns out PathBase) and make versions that would for example implement different type systems, without needing to re-implement everything. But here's the thing: fluent APIs don't play well with inheritance, because methods that return an instance of the type need to return the derived type and not the base. So it needs to be generic so the return type can be the type parameter. That never really materialized however, so it doesn't really have a purpose as tings stand today. In fact, it took about ten minutes to make a version without the base class: https://github.com/bleroy/FluentPath/tree/no-base-class Functionally, and in terms of public API surface, pretty much nothing changes.


Duplicate completion suggestions in .ts files

Both of two files are in the same vue project, but as you can see from the screenshot, the .vue file has normal suggestions, but the .ts file has duplicate suggestions.
Moreover, in the .ts file of non-vue projects, there are also duplicate suggestions, also caused by coc-volar, even if I set "volar.enable" to false, it can't be avoided, and the only way to get back to normal is to uninstall coc-volar completely.
I can reproduce it in https://github.com/jekip/naive-ui-admin with an empty coc-settings.json.
If you need any additional information, I'll provide it in the reply below.

Hi. Thank you for reporting the issue. I am experiencing the same symptoms in my environment. I will investigate to see if I can make adjustments to avoid affecting the tsserver.
It is likely that this issue is related to the tsplugin.

https://github.com/yaegassy/coc-volar/blob/8f0d27b83baf3cf6720ca57865ce3bbada677f6e/package.json#L322-L333

@hexh250786313 The adjustments are complete, and the new version has been published to npm.

@yaegassy It's working fine now. I tested it in both vue and non-vue projects. Awesome thanks for such a quick fix!


[feature] Navbar Dynamic Highlighting

Added dynamic highlighting to navbar
Previously:

Links on navbar (About, Work, Contact) stay dark unless clicked on
Clicked link stayed active no matter where user scrolls on page

Now:

Navbar link highlighting is based on current visible section
Clicking a link will scroll though navbar links until page arrives

https://github.com/ladunjexa/Threejs-3D-Portfolio/assets/25694146/1e146401-2a10-45e1-973b-6f64eb93b433

Hi CameronBeneteau,
Thank you for taking the time to add features & fix bugs in my repository.
I have reviewed the changes and everything looks great.


HTTP/2 support in the client

It would be wonderful if you could support HTTP/2 in the client at least.
This would greatly improve the performance when running the server behind a reverse proxy, to help mix wstunnel traffic with other HTTPS traffic.

Hello,
What do you mean by supporting HTTP2 ? Websocket is a different protocol than http2, and has less overhead than it.
If you are referring to the first request (the upgrade one) which in made with http1.1 that's normal, this is the way ws protocol is doing handshake.

I mean this: https://www.rfc-editor.org/rfc/rfc8441

Thanks for pointing out the RFC, I understand better the request :)
Regarding its support, I won’t implement it because websocket over http2 is using the stream notion from it.
Sadly, while using http2 stream would speed up the connection process for new tunnels, it will create a head of line blocking issue.
All stream of a http2 connection share the same underlying TCP stream, which mean that if a packet is dropped for this http2 connection, all streams/tunnels are going to stale.
As wstunnel, especially with socks5, can multiplex a lot of parallel stream, it is damageable for user experience.
If speed-up of new connections are an issue, you can use the -c option. You will get fast new tunnels, and without the issue of head of line blocking, as there is 1 tunnel per tcp connection.
 -c, --connection-min-idle <INT>
          Client will maintain a pool of open connection to the server, in order to speed up the connection process.
          This option set the maximum number of connection that will be kept open.
          This is useful if you plan to create/destroy a lot of tunnel (i.e: with socks5 to navigate with a browser)
          It will avoid the latency of doing tcp + tls handshake with the server [default: 0]

Fair enough.
I suppose the more future-proof course of action is implementing WebTransport for HTTP/3, but that's a different matter.
I'll close the ticket, then.


Website: Testnet page

This PR implements the testnet page.
Currently WIP

This PR has moved here: https://github.com/CodaProtocol/coda/pull/6139


value overflow when profiling table with large numeric IDs

I'm new to PipeRider and have really enjoyed it so far. The ease-of-use is pretty remarkable.
I've profiled a number of databases over the last few days and ran into an issue, in a handful of databases, where numeric ID columns cause an overflow error that prevents the profiler from processing a table. E.g.

[ 9/33] <redacted_table_name>                           0/31 0:00:15
Error: Profiler Exception: ProgrammingError('(snowflake.connector.errors.ProgrammingError) 100058 (22000): Value overflow in a SUM aggregate
[SQL: WITH anon_2 AS
(SELECT <redacted_table_name>.<redacted_numeric_id> AS c, <redacted_table_name>.<redacted_numeric_id>  AS orig
FROM <redacted_table_name> ),
anon_1 AS
(SELECT anon_2.c AS c, CASE WHEN (anon_2.c = %(c_1)s) THEN %(param_1)s END AS zero, CASE WHEN (anon_2.c < %(c_2)s) THEN %(param_2)s END AS negative, anon_2.orig AS orig
FROM anon_2)
SELECT count(*) AS _total, count(anon_1.orig) AS _non_nulls, count(anon_1.c) AS _valids, count(anon_1.zero) AS _zeros, count(anon_1.negative) AS _negatives, count(DISTINCT anon_1.c) AS _distinct, sum(CAST(anon_1.c AS FLOAT)) AS _sum, avg(anon_1.c) AS _avg, min(anon_1.c) AS _min, max(anon_1.c) AS _max, stddev(anon_1.c) AS _stddev
FROM anon_1]
[parameters: {'c_1': 0, 'param_1': 1, 'c_2': 0, 'param_2': 1}]
(Background on this error at: https://sqlalche.me/e/14/f405)')

The first time this happened I thought it was an obscure edge-case. I removed the problematic table from config.yml and moved on. In my environment, large numeric ID columns are fairly common, and the boxplot of the numeric ID values isn't meaningful. The result of this behavior is that we skip profiling some of our most frequently used data assets.
A quick & dirty workaround would be to copy the entire table, drop the problematic column, document its existence, and profile the copy. That would result in the generated HTML reports becoming disorganized. It's also expensive and slow.
Perhaps config.yml could be modified to accept an optional "exclude_column" property for any table in the includes list. Or maybe PipeRider could optionally warn on error, instead of fail. Or perhaps there's a way to write the profiling SQL statements to capture the data in a way that doesn't cause the overflow.
If a table has 100 columns, and it was possible to profile 99 without any errors, I'm sure there are other people like me who'd prefer to have 99 profiled columns and a warning rather than an error and no report.

@popcornylu This should be closed with the above merge?

Closed by #557