District names overflowing to the back button - Reopening #304

Reopen #304 as It was closed without fixing.
On mobile, long district names overflow and appear over the back button in expanded view.

Thanks for looking into this and sorry for the huge delay in us getting back to you.
Making this low priority thought.

Closed due to inactivity


add gnu target to sha256 sha512 to dispatch avx2 conditionally at runtime with gnu::target

this only works for ELF targets, not PE targets. Merge


Is there any way to add a script function to a host type?

Is it possible to add a script function to a host type? The below code throws an exception when I try to add function greet to type Person. A host type apparently has no prototype?
using Microsoft.ClearScript.V8;
using System;

namespace ClearScriptDemo
{
    class Person
    {
        public string GetName()
        {
            return "John Doe";
        }
    }

    class Program
    {
        static void Main(string[] args)
        {
            using (var engine = new V8ScriptEngine())
            {
                // expose a host type
                engine.AddHostType("Console", typeof(Console));
                engine.Execute("Console.WriteLine('{0} is an interesting number.', Math.PI)");

                // expose a host object
                engine.AccessContext = typeof(Program);
                engine.AddHostObject("person", new Person());
                engine.Execute("Console.WriteLine(person.GetName())");

                // add script function to host type
                engine.Execute("person.prototype.greet = function() { Console.WriteLine(\"Hello, World!\"); }");

                Console.ReadKey();
            }
        }
    }
}

Exception:
An unhandled exception of type 'Microsoft.ClearScript.ScriptEngineException' occurred in ClearScriptV8-64.dll
Additional information: TypeError: Cannot set property 'greet' of undefined

Hi @tu-lehong,
Neither host objects nor host types have a prototype property (unless the corresponding .NET object or type exposes such a property). However, there are many ways to extend such objects in JavaScript.
One way is to add a link to the object's prototype chain:
function extendHostObject(hostObject) {
    let proto = {};
    proto._proto = proto;
    Object.setPrototypeOf(proto, Object.getPrototypeOf(hostObject));
    Object.setPrototypeOf(hostObject, proto);
}
extendHostObject(person);
person._proto.greet = function() {
    Console.WriteLine('Hello, World!');
}
person.greet();

A more flexible solution is to wrap the object in a Proxy. That way you can add extensions and even override existing host properties.
Good luck!

Thank you!

Well, it won't work for Host types nowadays, will it?

Hi @simontom,

Well, it won't work for Host types nowadays, will it?

The extension technique described above should work for host types as well as objects.
Good luck!

Well, code with an  example is for object but types cannot be modified, can they?
I tried this (or maybe just something really similar to this _proto extension) but it threw an Ex saying that it cannot be modified.
I think this one
"Uncaught TypeError: Immutable prototype object '#' cannot have their prototype set"
Will try again when I have a little time.
Thank you so far

Hi @simontom,

"Uncaught TypeError: Immutable prototype object '#' cannot have their prototype set"

Ouch, you're absolutely correct! Some host resources (types, delegates, and possibly others) are realized on the JavaScript side as immutable prototype exotic objects (see here).
That's a hack. Those objects don't actually require immutable prototypes. It's just a rarely used attribute that ClearScript overloads for a specific unrelated purpose.
We've created Issue #330 to track the fix for this. In the meantime, Proxy might provide an alternative.
Sorry about this inconvenience!

Thanks a lot

Proxy solution, nor does the wrapping solution really work well when roundtripping.
Namely, it's hard (maybe impossible?) for my C# to return something that has picked up the previous MyType.prototype.__defineGetter_ calls that happened in the script. Even if there is, I'd have to do the wrapping on every return value.
Also, if my C# function exported as a global expects a MyType arguments, I don't think I can call it from js with a proxy object and expect it to work?
Are there any solutions to that?
In my specific scenario, js does not instantiate new instances of MyType, just adds extra getters/functions on the ones it receives from functions/globals exported from C#.


Set Server project as the Startup Project no longer works

This was working before the last commit

@ADefWebserver can you try after the PR #92 ?

@anmcgrath - The error went away but now I have a error where you double-click to edit a cell but it starts editing the cell to the cell in the left:

You complete the edit and it shows up in the correct spot...

That's strange, the example site doesn't have that issue. Do you have the issue on the other pages? Simple example or Conditional Formatting for example?

Sorry I just left home I'll check things again tomorrow but if you pull
down my repo you should be able to see it
On Sun, Jun 30, 2024, 4:57 PM anmcgrath @.***> wrote:

That's strange, the example site doesn't have that issue. Do you have the
issue on the other pages? Simple example or Conditional Formatting for
example?
—
Reply to this email directly, view it on GitHub
https://github.com/anmcgrath/BlazorDatasheet/issues/91#issuecomment-2198811104,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOFSB6Q3YVCWW7YIKJFOMLZKCLNJAVCNFSM6AAAAABKEGDD46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJYHAYTCMJQGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

That's strange, the example site doesn't have that issue. Do you have the issue on the other pages? Simple example or Conditional Formatting for example?

I do not see it on your pages 😕

In my examples I'm hiding the first column so I think that is what's causing it

Thanks, that helps.
In your project DataEditor.razor line 488:
sheet.Columns.Hide(0, 0);

should be
sheet.Columns.Hide(0, 1);

since the second parameter is count. I'll fix the sheet to not allow hiding rows/columns when these parameters aren't right.
I realise there's a mix of count and end in various functions so it's a bit confusing sorry.

OK I'll fix my code :) Thanks!

@anmcgrath - Yes, that fixed it (https://github.com/ADefWebserver/BlazorDatasheet/commit/91f31848019a9f5b5a798fc4152e9b07448acc5e). We're good now 👍🏽


[Style] About section is not aligned

What's the style idea?
We need to improve the contents appear more attractive by proper alignment
Kindly assign me this issue under #gssoc'24.

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSoC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

I would like to take on the task of resolving this alignment issue. With my experience, I am confident that I can rectify this issue promptly and ensure that the "About" section is properly aligned.
Could you please assign this task to me?

#72 Work on total about section make is good looking and responsive in all views


Removes apache http libraries and replace with azure core httpclient

This is still in progress.
General Notes:
The old Apache Client came with an exception called ParseException that was being used to indicate parsing failures. The version of said exception from the V4 client library, extended RuntimeException. However the version in the newer V5 Apache library is a checked exception. To replace it I created a KustoParseException that extends RuntimeException and as such is unchecked. This should work as a drop in replacement. You can see this new Exception class in use in the ClientRequestProperties class.
Rather than using the Netty HttpClient implementation from core, I used the HttpClient interface itself. This way, if someone wants to replace the Netty client they can do so easily by adding a different classname to their GlobalConfig or excluding the Netty client jar from their dependencies and replacing it with their own jar. For instance, I have a personal interest in utilizing the azure-core-http-vertx client once it comes out of Beta. This client better supports native image building with GraalVM. The Netty client implementation is the default implementation included with azure core.
I deleted the HttpResponseWrapper. Since the Apache client is removed, this class should no longer be needed.
BREAKING CHANGES:
In the ClientRequestProperties class the proxy option used HttpHost from the Apache libraries directly. This is exposed directly to the users of the library, so in order to remove the Apache dependencies I needed to remove HttpHost. I replaced this with the ProxyOptions from Azure core but this would be a breaking change for users of the library that specify a custom proxy. They would need to rework a few lines of code, no more than 3 or 4 max.
In Tests:
The UtilitiesTest in the data module made use of BasicHttpResponse from the Apache libraries. Because the HTTPResponse class from Azure core is an abstract class, creating new instances of it manually is not simple. This is probably by design. But in the event that we want to create a fake HTTPResponse, for instance in tests, we need a way to do so. There is a utility class in Tests now that will allow for the creation of fake HttpResponses. I created a simple implementation for it including only what had been used previously, and a builder to make it easier to instantiate new instances.
Still To Do:
Right now there are a lot of fixme's and todos in the code. I am not familiar with Flux and Mono from reactor. I work on a lot of Quarkus projects so I tend to use the Mutiny API. I need to familiarize myself with these so that I can add the async options.
There's a lot of block() in use because of my lack of familiarity with the reactor syntax.  If anyone with more experience in reactor would like to help, it would be much appreciated.
The removal of the entity stuff was difficult as expected. In order to handle it, I kept the Apache core. There was a lot of use of Entity and a lot of use of the URIBuilder from Apache core. I simplified this to some extent and I'd like to open a new PR in the future to further isolate these specific dependencies.
Lastly, the HttpClient from azure-core is not closeable. I think the idea was that clients aren't open/closed, connections are, and as such HttpResponse, the portion that comes back from the remote end, is what holds open and implements Closeable, instead of the client itself. Because of this change in logic from the CloseableHttpClient from Apache, I may need to delete some close() methods and remove Closeable from the Client interface. That's still to come. There is one specific client that makes use of its close method to close a different resource (ResourceManager class). For this reason I may return Closeable to the base client interfaces and the close methods will not do anything, although that seems like a bit of a hack to support a single client type.

Hey
Sounds like a very well thought process really like the usage of HttpClient - looks like a great fit.

Reactor is pretty much the same as mutiny (not familiar with the latter but it looks pretty similar approach ) but i could help with overcoming any problems you got trying to use it - you intend that we overcome the Todos in this PR ? As my intentions were that the Async part will be handled in the next PR and now only have simple sync impl - please tell me which part would you like me to take in this PR i would do it first thing this week or the next.
ProxyOptions is a very good implementation that we want anyway.
Regarding closeable - IngestClient has to be closeable, and only the STreamingIngestClient will have it as a noOp

@ohadbitt
You're right about the closeable.
I like your plan of a separate PR to work on the async implementation. That's a lot more changes.
There's also one or two other design things that I think might make sense while we're performing surgery around the HTTP Clients. I think it would be nice to get rid of HttpPostUtils and move the post methods onto an abstract class that all of the KustoClients should extend. That way the enclosed HTTP client can be a protected property of the parent abstract class, as well as the post methods. That would mean less passing around of the HTTP client, meaning shorter method signatures. I saw there was also a comment about making the base URL string a class property that can be set one time upon KustoClient creation and then reused. I liked that idea as well.

@The-Funk Sounds good - maybe better @AsafMah will look as well at all the planning

I'm not sure about KustoParseException extending RuntimeException, don't we want our exceptions to be checked?

I too am against RuntimeExceptions - @The-Funk , sounds like you should go ahead with the other changes, when ready hit the "ready for review"

Speaking of unchecked exceptions, I had extra throws for that exception that I didn't need, I just pushed that.
In this case, I think KustoParseException made sense because the original ParseException that needed replaced was unchecked, so to get the same behavior without changing the method signatures, I replaced it with a new unchecked exception.
If you would like I could use the new Apache HTTP core's ParseException which is a checked exception in V5.
Setting ready for review now.

@ohadbitt  For the const values, I created a specific class called HttpStatus in the http package and put const values in it.
The reason is, the other status codes constants come from the Apache library, and if we want to remove even the core library eventually, we'll need to migrate away from the Apache codes.
Those same status Apache status codes are provided as a transitive dependency of another dependency in the project (nimbus) but it probably wouldn't be a good idea to rely on them being there since we wouldn't be including the codes directly anymore if we get rid of Apache HttpCore.

Went through it all. Main issue is testing this properly, we need some long query, big data, multi threaded stress tests to really know we are ok here as we never had such a change. Will you take this as well ?

I have a development cluster that I can run some long running queries against. I can also try some test ingestions as well. I usually ingest data as JSON though. The part that worries me is ingestions involving file sources and streams.
For scale I can use JMeter and compare/contrast my service RAM/CPU with the current version of the library.
I don't know if I can give you really good in-depth insights though. That might be beyond me.

@AsafMah @ohadbitt
I left a comment above about my most recent push in response to Asaf.
I can revert it if you do not like the changes.
I was getting very confused where everything was happening and wanted to break things down into phases.
To that end, there's a LOT of changes above, but I think they're very useful changes in terms of code readability.
I know this PR was originally only supposed to be a replacement of the HTTPClient but there was a lot of fragmentation in where headers were being set, where the request URI was being edited, etc, and I think now that problem is mostly solved.
There's still some scoping issues to hide things from the users of the library but that would involve moving files and I wanted to try to avoid that.

Updated the changelog in the initial PR description.

I've marked this for review. For the record there are 2 failing tests. I've tracked the issue down to the ConnectionStringBuilder. Something in there doesn't like the colon : in certain URIs and replaces them with %3A.
I have yet to find the exact cause.

@ohadbitt @AsafMah I was able to get this to finally build and all tests to pass! Including the ones that check weird URIs!
The issue was the Apache URIBuilder in their new library.
I don't know if the behavior in the HTTPClient 4 URI builder is correct or the behavior in Apache HTTP5 URI builder is correct but using the older URIBuilder all tests pass and all results match what we expect.
Again, I have never seen a URI like the one above, and I am positive I don't know my protocols and standards to the degree the Apache Devs do.

@AsafMah @ohadbitt @yihezkel is there still interest in this?
I've been aligning this with new PRs as they come in but don't want to keep working on it if there's no chance it will be merged. I haven't heard from y'all in a little while.

Hi @The-Funk
Sorry for taking a lot of time , hurt my foot badly , couldnt walk - then was working on testing in the back.
To our matter - This looks all looks well but is failing for me now - ill try now and see why
com.microsoft.azure.kusto.data.exceptions.DataServiceException: IOException when trying to retrieve cluster metadata:Illegal character ((CTRL-CHAR, code 31)): only regular white space (\r, \n, \t) is allowed between tokens
at [Source: (String)"\u001F�\u0008\u0000\u0000\u0000\u0000\u0000\u0004\u0000}R]O�0\u0014}G�?Ty�i�8\u001F�-�Lc�ijA{��5�H��v�\u0018��N\u0009Pi�%J��{�����l��\u007F\u0007\u000B�29�=��P�1�J_j�\u001B�}�'\u001B�{w>�w��n\u0015��\u0019���mr�\u0011�]�\u0015�\u001E�\u0005\u0011Вv\u000E���༹�\u0014h���U�'�UU.x�\u0016\u0002\u0090qD\u001B\��Ъ��dP6o\u0004\u001F\u0016�@\u0004
�﬚�\u001Co��\u0018�\u000F!k�;�a\u0005�\u000C�Þy��\u0010�J��G��yt���
_�u�'���\u001D��X��l7�\u001Euf.\u001B.@\u0016\u0019�E\u0016�1L\u0011)\u0000�R�u\u0006$�˒&ѽ�ѣD�o�a��Eg\u00061e�\u001E�T\u007F"���:�V@�m\u001C��%�!B1$d!�\u000Cr����CD��\u00084B���\u0014��}

���\u0011��}��\u001F\u000C\u001E�\u0013O\u007F��'�E��\u0010<�5c\u0008s�@�d\u0015**�\u0008`Ij��OD��_�oٴBXp.
�\u0012\u001E���9��#\u0002G�v����(��(/3�pEj�pU#X����\u00027�܇vz��\u000F����G\u0003\u0000\u0000"; line: 1, column: 2]
at com.microsoft.azure.kusto.data.auth.CloudInfo.lambda$retrieveCloudInfoForCluster$0(CloudInfo.java:105) ~[kusto-ingest-6.0.0-jar-with-de

@ohadbitt   Sorry to hear about your foot, I hope you are doing better now. Those uXXXX numbers are unicode. I am wondering if this is a charset related issue. Looking at the changes I made in the CloudInfo file, there's no Accept-Charset header and there's no charset defined in the Accept header either. I am wondering if you just changed line 124 to:
request.setHeader(HttpHeaderName.ACCEPT, "application/json;charset=UTF-8");

I'm not sure if that would fix it. Being external I don't have the secret key to run the E2ETest so it always fails for me and I'm kind of blind to issues there. If I could help more, I would.
I'm seeing in the build failures that the missing applicationId is what is causing the E2ETest to fail, and I assume that comes from this CloudInfo endpoint.

will push a fix

I see some different behavior here , we sometimes get "handshake timed out after 10000ms"
seems like its a different timeout  configuration then the timeout we use - maybe wise to configure value for that as well if the user provided client timeout
https://github.com/reactor/reactor-netty/issues/1617

I see some different behavior here , we sometimes get "handshake timed out after 10000ms" seems like its a different timeout configuration then the timeout we use - maybe wise to configure value for that as well if the user provided client timeout . It seems the cause is just too many connections at a given time - solution is configuring cmd arg reactor.netty.ioWorkerCount or by making a fixed connection pool. Anyway user can configure this from outside or limit his concurrent requests. reactor/reactor-netty#1617

Hey @ohadbitt is this an observation or something you'd like to see action taken to address? If you would like, I can add more options to the HttpClientProperties and HttpClientPropertiesBuilder to account for more of the options available in azure-core's HttpClientOptions.
Alternatively, or perhaps longer term in a separate PR, do you think it would be worthwhile to do away with the HttpClientProperties and HttpClientPropertiesBuilder in this project and instead expose the ability to the users to directly provide an instance of HTTPClientOptions from azure-core? That would be a breaking change for anyone providing custom HTTP options currently, but it would mean that you wouldn't have to update the HttpClientProperties class in the future. As it is, HttpClientProperties will just be acting as a wrapper around the azure-core HttpClientOptions, so doing away with it would just remove the middleman.
By the way, related to HTTP Tracing properties. I noticed HttpClientOptions in azure-core provides for that as well. They have various implementations there including open-telemetry implementations. It may very well be that the HttpTracing class I added and the work done to add telemetry here could be replaced by the azure-core implementations eventually, to further simplify things in this project. That's just an aside though, work for a different PR on a different day.

Hey @The-Funk - regarding your question - i think the properties are good as they are - the user can already give the client he wants to use - if he want to really customize it .
Regarding my perf tests - im actually uploading the jmeter file here
https://github.com/Azure/azure-kusto-java/pull/354
I ran it in Azure Load Test resource with 2 instances , these are 5 tests that are running simultaneously
It seems to be showing a little worse performance - showing here version 5.0.4 vs funk pr  :)
I'm still thinking what to do next - but this doesn't necessarily mean we don't want to make the change as we can't really say if
Kusto is not the one making the jitter.

percentile_ms_20
percentile_ms_80
percentile_ms_90
percentile_ms_95
count_
avg_ms
t
testRun
Type

3,308
3,862
4,094
4,365
797
3821
funk
3
streaming query successfully

3,059
3,596
3,776
3,909
791
3332
504
3
streaming query successfully

1,834
2,226
2,356
2,573
158
2051
funk
3
streaming ingest successfully

1,705
2,019
2,133
2,222
160
1,878
504
3
streaming ingest successfully

3,262
4,038
4,243
4,578
794
3862
funk
3
simple query successfully

2,892
3,689
3,879
4,084
791
3412
504
3
simple query successfully

721
1,059
1,161
1,733
158
1,077
funk
3
queuing upload ingest successfully

797
991
1,092
1,149
160
1,056
504
3
queuing upload ingest successfully

546
851
1,018
1,183
397
760
funk
3
queue and update table '16' times succesfully

518
711
752
829
400
668
504
3
queue and update table '16' times succesfully

@ohadbitt I think the slightly worse performance is to be expected. I think the default azure-core client, Netty, is slightly slower than AHC because under the hood it uses NIO for async and there's some not-insignificant extra work there, per request, to support that async io.
Whereas the AHC just spawns a thread per connection, up to a certain number of threads, which is a much simpler design, albeit it uses significantly more heap space. I think Netty doesn't start to shine until you ramp up the number of concurrent requests being made or the size of the data in the requests. I saw this on medium about that a while back, and the sources were pretty good.

@The-Funk
After some more changes and tests runs that also included using OKHttp client instead of netty (its basically user configureable)- results seems to be a little better and we agreed that we are ok with the now slight worse perf
@AsafMah still wants to review this but after you (or us ?? )  resolve the merge from master we can go through with this change :)

@ohadbitt @AsafMah I've updated the branch, let me know if there's anything in particular you'd like to see after review.

@AsafMah I think I addressed most points. All tests that I can run appear to be passing. I have not loaded in OkHttp but if you would like me to remove Netty and add OkHttp now, I can.

FYI @AsafMah @ohadbitt I fully expect that with this big of a change there will come challenges and potentially regressions. I am willing to help if issues arise.

@The-Funk
What's next from here?
Are you going to work on the async api?

Thank you @The-Funk for your contribution,
As Asaf mentioned- you did want to impl the async api as well, if its not relevant anymore do tell :)

@The-Funk
What's next from here?
Are you going to work on the async api?

Indeed!


Returns palette_colors from kanagwa.colors.setup()

Thanks for the beautiful scheme.
Recently I tried to override and set some highlight groups based on this theme colors palette, and found that I couldn't
get proper code completion from the lua LSP (which sometimes caused misspelling as I has to write them manually),
apparently the cause was since the kanagawa.colors module returns table from vim.tbl_extend, not the actual
palette table, my workaround for this is to assign the final result to palette_colors.

Previously, the completion came from what I wrote for the c variable, although it is unrelated

After, completion came from the actual palette_colors

diff --git a/lua/kanagawa/colors.lua b/lua/kanagawa/colors.lua
index baf2fde..9facef9 100644
--- a/lua/kanagawa/colors.lua
+++ b/lua/kanagawa/colors.lua
@@ -67,7 +67,8 @@ function M.setup(config)
     local colors = vim.tbl_extend("force", palette_colors, config.colors)
     local theme = require("kanagawa.themes")[config.theme](colors)
     theme = vim.tbl_extend("force", theme, config.colors)
-    return vim.tbl_extend("force", theme, colors)
+    palette_colors = vim.tbl_extend("force", theme, colors)
+    return palette_colors
 end

 return M

Sure, this has nothing to do with the color scheme, but it's a good DX.

the problem with this is that it may change the original palette_colors. Also it may be misleading, as the output from kanagawa.colors.setup() is the merged table of palette_colors and theme, but only the former show a nice LSP completion. I am not sure how many would notice, but modifying the original colors invalidates the whole config design

Yes, and I think this is due to lua's limitations as an untyped language


[Makefile] change default path to value from sgx tutorial  SGX_SDK ?= /opt/sgxsdk

Closes #333

Closing this with the reasoning of  #333


Update index.md

Added additional installation step to make it clear that Vite needs to be installed separately.
Description
Related issues

Thanks!


PHP 7 Support?

I couldn't find cdhb in the PHP7 extension catalog. Is a port to PHP7 planned?

Hi @filex,
Did you solve this issue? Did you find a port?

Hi @maorcatmyheritage,
we have written our own PHP7 extension to fill that gap. We plan to open source the code very soon in https://github.com/sevenval/.
I'll post it here, when it's ready!

Wow! Great news. Thanks.
Can you send me the source in the meanwhile? We are in a process of migrating to PHP 7 and we're stuck because of this missing extension.

Our PHP7 compatible replacement for CHDB is now available in https://github.com/sevenval/SHMT.
To make it quickly available, for the moment we have just dumped our code. We have automated test, but they don't work standalone yet. This will all follow soon.
However, it does work well for us in production on CentOS 7, Ubuntu 1404 and SLES12. We didn't test/use it on Windows or other non-Linux platforms.
Feedback in our issue tracker is highly welcome!

Thanks. We'll have a chance to test it next week (Sunday probably). We'll give a feedback as soon as possible.
Thank you very much for sharing this with the community :)

@filex We managed to compile the source and already did an initial test. So far it looks really good.
We'll open issues in your SHMT project if we find any.
Thanks again!


Dg/10322 rn0.61

Hi. I have upgraded most of dependencies and have worked this library with react-native 0.61.4
this PR includes also refactoring commits from https://github.com/jarden-digital/react-native-pincode/pull/106 and some fixes
I think that after merge this you will need to bumb major or at least minor version of library

why this is not merged yet?


Run-android does not support launching Activity with different applicationId from package name defined in productFlavor

Migrated from React Native:
https://github.com/facebook/react-native/issues/22022

Thanks for importing this issue. Have you tried using --appId flag available for react-native run-android?
If yes, and it doesn't provide expected results, then it's something we should definitely fix.


add hint_constantly nbextension

addresses #563, enabling a code-completion dropdown all the time.

Tried it: it works and is very nice. Well done @jcb91 !
It would be super nice if we could detect the opening parenthesis ( and show the function hint (signature) as it shows with shift-tab...
[have a small issue that the extension does not show up in my /nbextension/ tab; installed manually instead - but it could be my own config which is a bit messed up]

It would be super nice if we could detect the opening parenthesis ( and show the function hint (signature) as it shows with shift-tab...

this is https://github.com/jupyter/notebook/issues/1296 -> basically the hints are sorted "suboptimal", the parameter are already in there (look for "para=").

I just tried it and works great! So helpful.
One more small suggestion is to make the match filtering a fuzzy match instead of exact character sequence, so you don't have to type underscores etc.
(FYI -- I had to install manually as well as the addition did not show up in my /nbextension/ tab)

Thanks @jfbercher :smiley:
So, it seems that something strange is happening (at least for two of you!) with it not appearing on the config page, but I'm at a loss so far to try to diagnose what that might be - it shows up fine for me. Are there any odd (server) log messages when loading /nbextensions? Things like failing to load the yaml file?
The shift-tab bit might be possible until core fixes the sorting, will investigate.
@rkpatel33, that fuzzy matching is, I think, something that'd have to be provided not by the individual kernels, which ultimately provide the suggestions lists to the notebook, so that's something I can't really fix here, although it might be something you could suggest to jupyter core to add a fuzzy-mode completion request to the existing regular one, so that kernels can implement it.

@jcb91 No specific message in logs. For my part, replacing "Jupyter Notebook" by "IPython Notebook" in the Type keyword of the yaml file solves the nbextension tab issue. Does it means something for you?

Ah, yes, @jfbercher, that presumably means that the config extension is ignoring the yaml file as its Type key doesn't match the expected. #544 altered the config extension to allow Jupyter Notebook Extension in addition to IPython Notebook Extension, but it's possible that if you haven't restarted your jupyter server after installing the newer repo version, that you'll still be running with the older one, and hence having the config ignored...

you'll still be running with the older one

Indeed @jcb91 . Had not refresh recently.

Added a few updates, including:

rename (apologies for those for whom this causes complications, I just liked the new name :stuck_out_tongue_closed_eyes:
a help-menu item to toggle the extra hinting off (in case it makes UI slow on laggy connections etc)
an attempt to integrate shift-tab on opening parentheses. This isn't ideal, as it seems to supply the signature of the most recently referenced function, even if the current parenthesis is actually not a function call (e.g. it might be just a bracket, or part of a tuple).
ignore more special keys - I found starting completion on pressing 'delete' was particularly strange

Initial test. Works for me and is really nice!
Thanks for implementing Shift-tab! Even if it is not ideal, it is a nice and useful functionality.


nvmesensor: Disable io_uring

io_uring_setup(2) was returning ENOMEM in unexpected circumstances. Avoid use of io_uring for nvmesensor, which does its own threading anyway.

jenkins run tests please

Patrick should have submitted a change upstream to fix this CI fail that we can pull in:
Exception: dep.get_variable() with positional argument requires meson_Version: '>=0.58'
Traceback (most recent call last):


Don't set runtime library directory on windows

Fixes #19. It's not supported on windows. Fixing this allows pygrib to build on Windows.

Thanks @dopplershift !


Add ecosystem page like  Docusaurus's Showcase page

See issue #227 for detail.

 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.


Chinese graphic theme / colors

Let's give the page some Chinese-ish theme. Should have red stuff, small gif and serious stuff. Here is some original duang inspiration : http://www.bilibili.com/video/av2023391/ . Loves the info overload but we have to keep it readable

This is weird enough to raise interest : http://images.google.fr/imgres?imgurl=https%3A%2F%2Fupload.wikimedia.org%2Fwikipedia%2Fcommons%2F5%2F57%2FElectronics_factory_in_Shenzhen.jpg&imgrefurl=https%3A%2F%2Fcommons.wikimedia.org%2Fwiki%2FFile%3AElectronics_factory_in_Shenzhen.jpg&h=1138&w=2074&tbnid=L9hh_bgO1AojIM%3A&docid=tolRfiFFnOlbWM&ei=rPGxVp2KC8PuUNb8r9AL&tbm=isch&iact=rc&uact=3&dur=681&page=1&start=0&ndsp=17&ved=0ahUKEwidkIavy9vKAhVDNxQKHVb-C7oQrQMIKjAD

Logo-wise, sth like this wil be awesome

Should have red stuff, small gif and serious stuff. Here is some original duang inspiration : http://www.bilibili.com/video/av2023391/ . Loves the info overload but we have to keep it readable

Hahaha all these are horribly agressive... perfect !

Absolutely!
On Wednesday, February 3, 2016, Clément Renaud<EMAIL_ADDRESS>wrote:

Hahaha all these are horribly agressive... perfect !
—
Reply to this email directly or view it on GitHub
https://github.com/duangish/duangish.github.io/issues/8#issuecomment-179196821
.


GPL license about "pkg/diskimage/guestfs/guestfs.go"

There is a GPL license in code "pkg/diskimage/guestfs/guestfs.go" about formating disk image, which add because of PR #18 and mentions LGPL license. does it effect whole apache 2.0 license?  will GPL license infection all program to GPL?

We are linking to non modified file with this licence so afaik it's ok.

Actually according to https://www.apache.org/legal/resolved.html#category-x there could be an issue with it. I'll try to escalate the issue to our lawyers.

OK, get it


feat: introduce a new 'get_bulk()' API

This new API allows fetching severals items at once (i.e. in bulk) efficiently. A res table is created, and populated with all items already in L1 and L2. If L2 misses are experienced, get_bulk() will spawn a number threads (configurable) to run L3 callbacks concurrently, and efficiently.
This API aims at hiding away the complexity of scheduling concurrent lookups when using only cache:get(). All lookup operations still fully integrate with other concurrent lookups currently being executed in the workers (e.g. L1/L2 hits/misses storage, L3 mutex, etc...).
Here is an example usage:
local res, err = cache:get_bulk({
  -- key     opts          L3 callback                    callback argument
    "key_a", { ttl = 60 }, function() return "hello" end, nil,
    "key_b", nil,          function() return "world" end, nil,
    "key_c", nil,          function() return "bye" end,   nil,
    n = 3 -- number of operations
})
if err then
     ngx.log(ngx.ERR, "could not execute bulk lookup: ", err)
     return
end

for i, data, err, hit_lvl in mlcache.each_bulk_res(res) do
    if not err then
        ngx.say("lookup ", i, ": ", data)
    end
end

It comes with two sugar functions that allow for easier interaction with regards to the argument and result tables:

mlcache.new_bulk()
mlcache.each_bulk_res()

Usage of these sugar functions is optional, and users can manipulate these tables by hand if they are familiar with their expected layouts.
Feedback and code reviews welcome!

@jdesgats @javierguerragiraldez Would love a review from you both if you are still using mlcache internally. Cheers!


Normalize headers in auth

https://github.com/micro/go-platform/blob/master/auth/platform.go#L155
When using auth-srv and the HandlerWrapper and ClientWrapper, the metadata map has the key Authorization, but the lookup is for authorization.
These should be normalized so you don't get lookup errors.

Here's a fix https://github.com/micro/go-platform/commit/1e631bb6815c2dbabe589649b8d89b5a460c41f5


Cannot read property 'EXTENSION_SEARCHSYNC_CONFIG_PATH' of undefined

Hi,
I get error below after update directus to 9.1.2
⚠  Couldn't register hook "directus-extension-searchsync"
⚠  Cannot read property 'EXTENSION_SEARCHSYNC_CONFIG_PATH' of undefined

It should be fixed with latest version. @usamecavga could you validate please.

Hi,
There are some errors belongs to node version;
Now using node v16.10.0 (npm v7.24.0) and Now using node v14.16.0 (npm v6.14.11)
⚠  Couldn't register hook "directus-extension-searchsync"
⚠  Cannot find module 'cosmiconfig'

Now using node v12.20.2 (npm v7.24.1)
⚠  Couldn't register hook "directus-extension-searchsync"
⚠  Unexpected token '.'

Hi,
There are some errors belongs to node version;
Now using node v16.10.0 (npm v7.24.0) and Now using node v14.16.0 (npm v6.14.11)
⚠ Couldn't register hook "directus-extension-searchsync"
⚠ Cannot find module 'cosmiconfig'
Now using node v12.20.2 (npm v7.24.1)
⚠ Couldn't register hook "directus-extension-searchsync"
⚠ Unexpected token '.'

Oh, yes in Node 12 there was issue indeed, it should be fixed with latest 1.0.2. I have tested it with Node 12, 14 and 16
About the first error, ensure you install it with npm install dimitrov-adrian/directus-extension-searchsync

Hi,
There are some errors too.
Now using node v12.20.2 (npm v7.24.1) and Now using node v14.16.0 (npm v6.14.11) and Now using node v16.10.0 (npm v7.24.0)
14:33:12 ⚠  Couldn't register hook "directus-extension-searchsync"
14:33:12 ⚠  Cannot find module 'axios'

How you install it? axios should be installed already by directus, and the extension is defining it as peer dependency.

Hi,
yarn add dimitrov-adrian/directus-extension-searchsync
yarn add v1.22.10

[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[4/4] 🔨  Building fresh packages...

success Saved lockfile.
warning No license field
success Saved 25 new dependencies.
info Direct dependencies
└─<EMAIL_ADDRESS>info All dependencies
├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>├─<EMAIL_ADDRESS>└─<EMAIL_ADDRESS>✨  Done in 5.16s.

I do believe you are out of (may be parent) of your directus project directory, and it should be inside.
One more thin (I just saw I made an mistake in the readme) the config files should be one of next

package.json "searchsync":{...}
.searchsyncrc
.searchsyncrc.json
.searchsyncrc.yaml
.searchsyncrc.yml
.searchsyncrc.js
.searchsyncrc.cjs
searchsync.config.js
searchsync.config.cjs

Hi,
I'm in directus directory. And .searchsyncrc.json is near package.json.
When I add axios package with "yarn add axios" it works.
{
  "license": "MIT",
  "dependencies": {
    "axios": "^0.24.0",
    "directus-extension-searchsync": "dimitrov-adrian/directus-extension-searchsync",
    "pg": "^8.7.1"
  }
}

Not sure why this actually happen, but might be better then to include the axios in the actual dependencies.
Thanks for all the feedback!


Resolve synchronizations problems

Resolve synchronization problems caused by double-checked locking

This issue has been resolved after jdk1.5，add volatile to solve this problem private volatile static VideoViewManager sInstance;


Return the first consecutive sub block of ops for frs driver instead of 0 ops on violation

Description
Item: https://dev.azure.com/fluidframework/internal/_workitems/edit/4668
When we fetch ops for frs through delta storage or through snapshot, it could happen that the ops are not contiguous. In that case, return the first set of contiguous ops so that we can make at least some progress.

I'd add to your description that it's an issue service needs to address, but this change is a temp experiment to see if it mitigates issues for users while we are waiting for proper fix.

Q: any other places to use validateMessages()? I believe we were hitting (in the past) asserts somewhere in GetOps flow (in shared code), and code there was doing pretty much same validation (but using asserts).
Maybe worth glancing at code around 0x26d, 0x26e, 0x26f. I'm not sure what changed and why we are not hitting them anymore (maybe FRS fixed issues here? Current set of problems seems to be only with ops that are part of snapshot).
If so, maybe it's fine to leave it as is (at least for now).

Q: any other places to use validateMessages()? I believe we were hitting (in the past) asserts somewhere in GetOps flow (in shared code), and code there was doing pretty much same validation (but using asserts).
Maybe worth glancing at code around 0x26d, 0x26e, 0x26f. I'm not sure what changed and why we are not hitting them anymore (maybe FRS fixed issues here? Current set of problems seems to be only with ops that are part of snapshot). If so, maybe it's fine to leave it as is (at least for now).

As mentioned in the issue item, here are the findings. We seem to hit "OpViolation" in both snapshot ops and in storage ops.
let opsViolation = union office_fluid_ffautomation*| where Event_Time > ago(30d)| where Data_hostName == "@fluid-internal/test-service-load"| where Data_branch startswith "refs/heads/main"| where Data_driverEndpointName contains "frs"| where Data_eventName has "OpsFetchViolation"| summarize count() by Data_reason;//| project Data_eventName, Data_reason, Data_from, Data_start, Data_last, Data_length;//| summarize count() by bin(Event_Time, 1d);opsViolationTable0

Data_reason
count_

snapshotOps
15,547

storage
9,984

I am following up on this with @MaggieLiMS and they will be fixing this at their end.

However, these asserts are not present anymore for last 30 days. However, we will catch them here and not cause those asserts any more it seems. Also we have this item opened here: https://dev.azure.com/fluidframework/internal/_workitems/edit/4667
I will be looking into those asserts as part of above item.


Imports not compiled

Seems like pundle handles imports nicely, but doesn't output down to require. Is this possible?

Ah this makes sense. I have modules: false in preset-env. Need to split out the main app to have modules: true, and packages to have modules: false.


Implementing (not deriving) foreign traits for generated types?

I would like to implement, i.e. TryInto for some of our types generated by prost. Is there a way to include code, probably in the  "rust_prost_library" bazel target, that includes a .rs file as a dependency, in which I can place trait implementations?
It seems like I could embed source code in my Bazel file as a string, and pass it into the type_attribute:
https://docs.rs/prost-build/0.5.0/prost_build/struct.Config.html#method.type_attribute
but that's probably not going to work with rust_analyzer, or cache well since it's outside the build graph.
I also see extern_path:
https://docs.rs/prost-build/0.5.0/prost_build/struct.Config.html#method.extern_path
but that takes a proto file too, so it doesn't seem like the right thing.

First of all, I don't have experience with bazel, so I can't help you with that.
If you follow the example from prost-build docs, you have a lib.rs which includes the generated files. You could add your impls there. For example:
use prost::Name;
use alloc::string::String;

include!(concat!(env!("OUT_DIR"), "/type_names.rs"));

impl Into<String> for Foo {
    fn into(self) -> String {
        Foo::full_name()
    }
}

Thanks for the response! In hindsight this is indeed more of a build issue.


Install IcedTea-Web pre-reqs on new Windows build machine

Building IcedTea-Web on Windows requires the following packages:

[x] cygwin
[x] preinstalled jdk
[ ] tagsoup
[ ] mslinks
[ ] wixgen
[x] wix
[x] rust
[x] cargo
[ ] rhino

Would it be possible to get the required packages installed on a Windows machine?  I have captured the install instructions below for packages I believe are not already installed or in the Ansible playbooks.  I will create a PR to add them to the Ansible playbooks.
For the required jar files below the common folder that IcedTea-Web looks in is /usr/share/java/. If possible it would be great if we could put them all in that folder as it would make it easy to check to see if the requirements are there.
Since the Windows machines can compile OpenJDK I believe cygwin and a JDK are available.
tagsoup is a jar called tagsoup.jar which can be downloaded here. IcedTea-Web configure expect this file to be located in /usr/share/java/tagsoup.jar or /usr/local/share/java/tagsoup.jar so if it could be placed in either of those locations it would be ideal. If it can not put in either of those locations the location can be specified via an option to configure.
mslinks is a jar called mslinks.jar which can be downloaded here. IcedTea-Web configure expect this file to be located in /usr/share/java/mslinks.jar or /usr/local/share/java/mslinks.jar so if it could be placed in either of those locations it would be ideal. If it can not put in either of those locations the location can be specified via an option to configure.
wixgen is a jar called wixgen.jar which can be download here. IcedTea-Web configure expects this file to be located in /usr/share/java/wixgen.jar or $WIX_TOOLSET_DIR/wixgen.jar or $WIX_TOOLSET_DIR/../wixgen.jar. I do not see any way to override where it looks.
Rust can be installed via the instructions in the Ansible playbook. I believe cargo is installed with Rust.
rhino is a jar called js.jar which can be download here. IcedTea-Web configure expects this file to be located in /usr/share/java/js.jar or /usr/share/rhino-1.6/lib/js.jar. I do not see any way to override where it looks. You can unzip the file above with this command to get the jar file:
unzip -j -d /usr/share/java rhino1_6R1.zip rhino1_6R1/js.jar

@sxa555 I'm tempted to say we should spin up specific hosts just for Icedtea-web and have it in its own playbook...

We've got plenty of windows 2016 so could build it on that?

I have not personally tried to build it on Windows 2016. @judovana have you tried?

Yeah with there being so many additional things I'd prefer to do it in a standalone machine at least for now especially given we don't have a tried and tested playbook for it - @gdams can you allocate on for this please? I was expecting this work item to just be a rust installation :-)

The other 4 things are just simple jar files I believe. Any update? I could progress by just downloading them as part of the build until the machine can be configured.

@charliegracie Have you added the other jar files to the playbook yet? I'd be reluctant to approve anything that downloaded things to a location outside the build workspace (which from your comment sounds like it is required).

@charliegracie in order to unblock you I have installed those prereqs on one of the build machines and added an icedteabuild tag on the machine that you can use in your job to ensure it gets scheduled on there: https://ci.adoptopenjdk.net/computer/build-softlayer-win2012r2-x64-1/

Thanks @sxa555. Did those installs include rust? If so can you do a which rustc on the machine and tell me the path? Thanks

It's installed as per the playbook, so from the cygwin shell it's /cygdrive/c/rust/bin/rustc

I've been trying to compile ITW 1.8.x on windows using GH Actions. Currently hitting the following failure:
Configure IcedTea-Web
configure:7136: error: possibly undefined macro: AC_CACHE_CHECK
      If this token and others are legitimate, please use m4_pattern_allow.
      See the Autoconf documentation.
configure:8621: error: possibly undefined macro: AC_DEFINE
autoreconf-2.69: autoconf failed with exit status: 1
configure: error: cannot find install-sh, install.sh, or shtool in "." "./.." "./../.."

Any idea what the issue might be?
Link to PR https://github.com/AdoptOpenJDK/IcedTea-Web/pull/761/checks

Closing this. We have now ported all ITW builds over to use GH Actions: https://github.com/AdoptOpenJDK/IcedTea-Web/wiki/IcedTea-Web-1.8.x-Release-Guide

@gdams As per your last comment, if this issue is resolved could you please close it. Thanks 👍🏻


feat: sidebar

rough implementation of sidebar

Not bad, but i think it looks kinda funky on mobile. Ill try to tweak it


fix: Update the favicon logo to the latest version

Things added/changed:

Update the favicon logo to the latest version.

Here you can see the preview of the website (somehow I found it 👀 ): http://tech-phantoms-tech_phantoms_website-preview-pr-305.surge.sh/


是否考虑增加k8s的部署方式

后期是否考虑增加nebula-graph-studio在k8s中的部署方式，如在operator里增加

Dear @dylanyht,
This is actually quite an interesting point.
@MegaByte875 @nianiaJR @veezhang @kqzh what do you think?
Maybe not necessarily in nebulaCluster CRD but only a helm chart considering it's basically stateless?


[DOC] tutorial/workshop for pydata Seattle 2023

We will need to prepare a tutorial/workshop for pydata Seattle 2023 - 90min incl Q/A, modus is live presentation (but can be prepared from notebooks or dev IDE)
To collect thoughts.
My thoughts:

the "obvious" content focuses on a step-by-step guide to create a package using skbase, perhaps an "easy" use case and an "advanced" one.
we probably want 15min intro on the general interface, i.e., a 101 of the sklearn/sktime-like interface that skbase follows, before we get into implementation details.
the patterns are pretty clean imo, so perhaps we can also show sktime (assuming the refactor is released by then), and maybe another package

draft content sketch
Part 1 – the sklearn-like interface – 20-30min
Exposition – what are the key features? 1 jupyter notebook
Using sktime as an example?
“objects”
constructor
get_params/set_params basic
tags
configs
mention repr, pretty-printing
composition – simple
show get_params/set_params for composition
composition – heterogeneous, pipelines
show get_params/set_params again
“estimators”
Fitting, is_fitted
Get_fitted_params
Show atomic, composition simple, composition pipeline
Lookup
all_objects aka sktime all_estimators
all_tags
Testing
Get_test_params
Create_test_instance
Create_test_instances_and_names
Check_estimator
Part 2 – creating your own sklearn-like with skbase
Showcase simple mock package – 5min
Quick walkthrough on usage, parallel to part 1
Show codebase, check through the below – 5min
Step-by-step instructions – 20min
Import of BaseObject as parent
Special methods in child package
Tags in child package
Configs in child package
Testing (superficial)
2a search/retrieval – 5min
All_objects interface
2b estimators – 5min
Fit method
Get_fitted_params
2c heterogeneous estimators – 10min
Heterogeneous mixins
Example composite
2d testing – 5min
Importing BaseObject, BaseEstimator tests
Extending tests
3 wrapup, summary, invite to contribute – 5min

done!
here: https://github.com/sktime/sktime-tutorial-pydata-seattle-2023


[fix] Create documentation.yml

Fixes Issue
Closes #930

Changes proposed

I created a DOCS issue template
Screenshots

Note to reviewers

Feedback would be greatly appreciated.

@rupali-codes, we are going to need your force push here


Update Vendor: Vimeo

I have no knowledge of this companies pricing, but it looks like the pricing might be incorrect.
The notes says $10k annually, but the pricing section is based on it being $10k/month.
The $20/month is correct for the "base level" pricing.
At the very least, I think it should be updated to be $833.33/month to correct the whacky percentages.
Feel free to close, just wanted to point it out.

Thank you so much for point that out, just fixed it! 🙏


About the experiments in few shot learning scenario

Thank you for your impressive job!
I am working on few shot learning scenario. After reading your paper and code, I have some questions:

Could you please upload the few shot data split?
The few shot trainning set up is different from CoverTree? It seems that CoverTree do both the pretranning and down-stream task training in few shot scenario? but your work runs pretraining using the whole  ModelNet40 dataset, then train the down-stream task in few show scenario?

Hi @priceee , thank you for your question.
I didn't have the data splits, as the data stored on the Cambridge's server should be erased now..
Yes we pretrain the models on the whole dataset.


Version 6.5.3, Antivirus claiming Unauthorized File Encryption?

We run Trend Micro Apex One and recently it's now flagging DriverAutomationTool.exe as a Virus/Malware due to Unauthorized File Encryption.
I ran it for several Dell models yesterday with no issue. I barely ran it today and it's quarantined the exe. I am using the 7zip compression option as I'm going through and re-doing all of the packages.
Should this be deemed as a false alarm with exceptions made?
Thank you

Submitted file to Trend for analysis, their response...

These samples are verified and here is the result:
    * File (Normal)
            DriverAutomationTool.exe / DriverAutomationTool.exe (0e9b9952b6bc14834e37d586399c6a91e4f87e2e)

The file will be added in our certified safe software databases and may take 12-24 hours to reflect in the systems.
Please take note that for this exemption to take effect, the product must maintain a stable internet connection.

This may help someone else that encounters it.
Thanks


Not creating new records while cloning association has_many: through:

Hello,
I observed one thing while cloning nested associations. Whenever am trying to clone a has_many through  association,deep_clone instead creating new association objects, it just assigning association records of original record to cloned record.
ex:- if Model has association has_many: :something, through: :another
here let's say Something.count  == > 10
if i do clone = Model.find(1).deep_clone include: [:something]
clone.save!
after cloning also Something.count == 10
Is it a intented functionality or bug? Thanks
FYI:
ruby - 2.3.3p222
Rails - 4.2.4
deep_clonable - 2.2.2

Hi,
I think it is, this is related to #22. I think you should clone via the through association instead:
Model.find(1).deep_clone include: { another: :something }

#27 is also related.

@moiristo Tried both before raising a issue here. Those solutions seems to not working with the versions i provided. Please take a look into it. Thanks.


Select text + Shift/Ctrl + Scroll behaviour

VSCode Version: 1.5.3
OS Version: Windows 10 Version 1607 OS Build: 14393.187

Steps to Reproduce:

Select some text in a file.
Shift/Ctrl + scroll behaviour should be same as in Visual Studio.

To keep the number of issues in our inbox at a manageable level, we're closing issues that have been on the backlog for a long time but haven't gained traction: we look at the number of votes the issue has received and the number of duplicate issues filed. Thank you for your time and understanding.
If you disagree and feel that this issue is crucial: we're happy to listen and to reconsider.


SSL configuration doesn't support certificate chains

When SSL is enabled for the server and the certificate buffer in the SSL configuration contains a chain of certificates, the server is only configured with the last certificate in the chain, rather than the whole chain. This is because the server calls SslContext::set_certificate(), which only sets one certificate even if it receives a whole chain. This leads to failures when connecting to a Rustls client, since Rustls expects the server to present a chain of certificates starting from the root CA. For example, I tested with these certs. The Rustls client used ca.cert, while the tiny-http server used end.fullchain and end.key, and I got a TLS error on the client.

Thanks for the bug report @YuhanLiin, coincidentally we noticed this mistake while working on Rustls support in https://github.com/tiny-http/tiny-http/commit/3e2ca0908365fb333751eb640d4cfa1cae05db04, I'll try and include a fix for the OpenSSL mode in that release when it's ready.
If you need this patched urgently though I would accept a PR which I can tidy up later.

Fixed and released in 0.11.0, please let me know if you have any more issues!


Bug: Unavailable buttons don't use grey icons

What feature or improvement do you think would benefit Files?
Now in version 3.5, the usability of a button is not directly demonstrated by its appearance; only when the mouse passes over the button is there explicit feedback about whether it is usable or not.
In Windows File Explorer, as well as in most web browsers, the usability of buttons can be seen directly from their color - grey or black; this feature was also present in previous versions of the program. However, after updating to version 3.5, this feature is no longer available.
Requirements

If one function is unavailable, the button of it should be grey; instead of still be black like now.
If one function is available, the button can be black.
In any case, the availability of program buttons should not be displayed only when the mouse passes over the button icon, but should be presented in a clear manner at all times.

Files Version
<IP_ADDRESS>
Windows Version
10.0.26120.961
Comments
These buttons should look like buttons in the same position in:

File Explorer:
Microsoft Edge:
Firefox:
Even Apple OSs:

Instead of what they are now.

We noticed this a few hours ago and will change to hide unavailable buttons.
Thanks!

This is fixed in main and will be resolved in the next update.


How can I add Allure or any other HTML report using Playwright Java?

I am using Playwright with Java and TestNG and I want to generate HTML report.. Can you share how can I implement that?

Solved - Done using Extent Report

@Mosorathia Even i am looking to implement the same, can you please share your implementation. Thanks

@Mosorathia Even i am looking to implement the same, can you please share your implementation. Thanks

@kishorejawalkar You can check this out.. Here the example is of selenium but it works in similar way for Playwright and TestNG


BUG: Instructables pages don't clip user visible content

Version (please complete the following information):

OS: macOS 15.1.1 or iPadOS 18.1
Browser: Safari
Web Clipper version:0.9.8
Obsidian version: 1.7.7

Describe the bug

Visit https://www.instructables.com/A-Simple-Drill-Press-Tray/
Clip the page

The visible page content, what a user would obviously expect to have clipped, isn't selected. Instead, some obscure privacy notice content embedded on the page but not normally (or ever?) visible is clipped and none of the visible page content is clipped. This only happens on some pages, but not all; I haven't identified a pattern one way or another.
Expected behavior
The expected user visible content should be selected for clipping.
URLs where the bug occurs

https://www.instructables.com/Fire-Roasted-Salsa/
https://www.instructables.com/A-Simple-Drill-Press-Tray/

An example of a correct page on this site:

https://www.instructables.com/Micro-Planck-3D-Printable-Low-profile-Ortholinear-/

Your template file
Default template is in use.

For comparison, MarkDownload clips the expected page content and none of the weird privacy text.

ugh, this bug also appears to be intermittent. After numerous incorrect clips of the drill press tray article with this extension, I just did yet another one, and it came up correctly.
I added an example of the "wrong" clip text to this gist: https://gist.github.com/jwhitley/da55ab9cef28547083e6d9103e7fb119

This can be solved using a template and the selector {{selector:article|markdown}}. Please make sure to review Troubleshooting instructions.

@kepano, thanks! If I understand correctly, that selector is effectively a workaround for Readability’s behavior. Is that correct? Mostly, I’m trying to understand if I should look for and/or file an upstream issue.


fix(#316): better support for typeIn and ngModel

closes #316

v11.10.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.


Display ec2 instance name when tag name exist

Q
A

🐛 Bug fix?
no

🚀 New feature?
yes

⚠ Deprecations?
no

❌ BC Break
no

🔗 Related issues
#419

❓ Documentation
no

Description

We should display the instance name for aws_ec2_instance and other resource that use tags for a name.

@sundowndev Target branch should be 0.8


getting below error bucket-antivirus-function

START RequestId: 44a5bfce-0772-11e9-a343-cd0a75d4f9ef Version: $LATEST
Script starting at 2018/12/24 11:51:37 UTC
'Records': KeyError
Traceback (most recent call last):
File "/var/task/scan.py", line 140, in lambda_handler
s3_object = event_object(event)
File "/var/task/scan.py", line 29, in event_object
bucket = event['Records'][0]['s3']['bucket']['name']
KeyError: 'Records'
END RequestId: 44a5bfce-0772-11e9-a343-cd0a75d4f9ef
REPORT RequestId: 44a5bfce-0772-11e9-a343-cd0a75d4f9ef	Duration: 1.95 ms	Billed Duration: 100 ms 	Memory Size: 1152 MB	Max Memory Used: 61 MB

I'm assuming this is while testing the function in the AWS console? If so, can you share the event data you are passing to the function?

I added AV_DEFINITION_S3_BUCKET value bucket name for both the functions

Now getting below error
{
"stackTrace": [
[
"/var/task/scan.py",
145,
"lambda_handler",
"scan_result = clamav.scan_file(file_path)"
],
[
"/var/task/clamav.py",
136,
"scan_file",
"raise Exception(msg)"
]
],
"errorType": "Exception",
"errorMessage": "Unexpected exit code from clamscan: 127.\n"
}

Can you try again with master? Make sure to update your IAM permissions. I'm going to close this because I think it should be fixed but I'll be happy to reopen and investigate.


Strongly type geomEach function to GeoJSONObject

Closes #29

tests passed locally and in ci, so I'll see if I can at least refactor this to be more maintainable/readable

Question for discussion: How critical should we consider the comment about speed an efficiency in the geomEach turf.js implementation? I'm not sure if turf.js has any benchmarks out there for measuring performance that we could run against our dart implementation to ensure we maintain performance.
The only thing I think I'd change is to basically have a few more if branches deeper in my refactored implementation, rather than passing around several unnecessary arguments. I'll push a new commit that does this for thoughts and review.

https://github.com/Turfjs/turf/blob/d72985ce1a577b42340fed5fc70efe8e4bc8b062/packages/turf-meta/index.js#L523 this remark?
Maybe we could create a similar benchmark suite: https://github.com/Turfjs/turf/blob/master/packages/turf-meta/bench.js
I found this package for example:
https://pub.dev/packages/benchmark

I rebased this on top of #25 and squashed the commits down to one

Closes #29

Did you mean another issue here? I don't see an issue that this closes, but it relates to #27 at least

@lukas-h @tobrun does this refactor work for you guys? Anything you'd like to do differently?

@lukas-h @tobrun does this refactor work for you guys? Anything you'd like to do differently?

Looks much cleaner now! I tried to make it even a little smaller here: #33

@lukas-h @tobrun does this refactor work for you guys? Anything you'd like to do differently?

Looks much cleaner now! 😄 I tried to make it even a little smaller here: #33

thanks! approved and merged in here


Removing pwncollege from inactive list

pwncollege is back to streaming now that school in session

Don't trust the automation! Just come watch the stream :p


Does unitas_setup_tool download latest build

I forgot I haven't tested what happens if the local dl is detected as "downloaded" and if it would redownload if cloud has new build

It does


scan: report generation takes a rather long time

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
When I run scan, the final step to generate test reports take a very long time to complete.  My project is complicated to compile, but (unfortunately) there are not a ton of tests.  To give an example, compilation takes ~20m on CircleCI, the 174 tests run in 13s, and then generating a junit report takes another 8m.  I turned off HTML report generation, as it was taking even longer to run.
Is there anything I can do to speed this up?  Can I (should I?) turn it off entirely?  My builds are slow enough as it is - I'd love to shave off those extra minutes.  TBH, I'm not even sure if I've set it up properly for Circle to see the results.
Complete output when running fastlane, including the stack trace and command used
Just the end of the scan command, happy to provide more context if it would be useful:
08:35:31: ▸      Executed 174 tests, with 0 failures (0 unexpected) in 13.240 (13.369) seconds
[08:35:35]: ▸ Test Succeeded
+--------------------+-------+
|        Test Results        |
+--------------------+-------+
| Number of tests    | 174   |
| Number of failures | 0     |
+--------------------+-------+
[08:43:05]: Successfully generated report at '/Users/distiller/project-ios/fastlane/test_output/report.junit'
Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
Redacted the Fastfile - literally just running scan

✅ fastlane environment ✅
Stack

Key
Value

OS
10.11.6

Ruby
2.2.5

Bundler?
true

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
8.0

Git
git version 2.8.4 (Apple Git-73)

Installation Source
/Users/andyi/.rvm/gems/ruby-2.2.5/bin/fastlane

Host
Mac OS X 10.11.6 (15G1004)

Ruby Lib Dir
/Users/andyi/.rvm/rubies/ruby-2.2.5/lib

OpenSSL Version
OpenSSL 1.0.2e 3 Dec 2015

fastlane gems

Gem
Version
Update-Status

credentials_manager
0.16.2
✅ Up-To-Date

fastlane_core
0.53.0
✅ Up-To-Date

spaceship
0.36.2
✅ Up-To-Date

cert
1.4.3
✅ Up-To-Date

deliver
1.14.4
✅ Up-To-Date

frameit
3.0.0
✅ Up-To-Date

gym
1.11.3
✅ Up-To-Date

sigh
1.11.2
✅ Up-To-Date

match
0.9.0
✅ Up-To-Date

pem
1.3.2
✅ Up-To-Date

pilot
1.11.1
✅ Up-To-Date

produce
1.2.1
✅ Up-To-Date

scan
0.13.1
✅ Up-To-Date

screengrab
0.5.5
✅ Up-To-Date

snapshot
1.16.2
✅ Up-To-Date

supply
0.7.1
✅ Up-To-Date

fastlane
1.106.1
✅ Up-To-Date

Loaded fastlane plugins:

Plugin
Version
Update-Status

fastlane-plugin-versioning
0.2.4
✅ Up-To-Date

Loaded gems

Gem
Version

ZenTest
4.11.1

RubyInline
3.12.4

i18n
0.7.0

json
1.8.3

minitest
5.9.1

thread_safe
0.3.5

tzinfo
1.2.2

activesupport
<IP_ADDRESS>

addressable
2.4.0

babosa
1.0.2

bundler
1.12.5

colored
1.2

highline
1.7.8

commander
4.4.0

security
0.1.3

credentials_manager
0.16.2

excon
0.54.0

gh_inspector
1.0.2

multi_json
1.12.1

plist
3.1.0

rubyzip
1.1.7

terminal-table
1.4.5

fastlane_core
0.53.0

multipart-post
2.0.0

faraday
0.9.2

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20160826

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

faraday_middleware
0.10.0

fastimage
1.6.8

multi_xml
0.5.5

claide
1.0.1

fuzzy_match
2.0.4

nap
1.1.0

cocoapods-core
1.1.1

cocoapods-deintegrate
1.0.1

cocoapods-downloader
1.1.2

cocoapods-plugins
1.0.0

cocoapods-search
1.0.0

cocoapods-stats
1.0.0

netrc
0.7.8

cocoapods-trunk
1.1.1

cocoapods-try
1.1.0

escape
0.0.4

fourflusher
2.0.1

molinillo
0.5.1

xcodeproj
1.3.3

cocoapods
1.1.1

dotenv
2.1.1

osx_keychain
1.0.1

cocoapods-keys
1.7.0

mini_magick
4.5.1

rouge
1.11.1

xcpretty
0.2.4

net-ssh
3.2.0

net-sftp
2.1.2

krausefx-shenzhen
0.14.10

slack-notifier
1.5.1

xcpretty-travis-formatter
0.0.4

jwt
1.5.6

little-plugger
1.1.4

logging
2.1.0

memoist
0.15.0

os
0.9.6

signet
0.7.3

googleauth
0.5.1

httpclient
<IP_ADDRESS>

hurley
0.2

mime-types-data
3.2016.0521

mime-types
3.1

uber
0.0.15

representable
2.3.0

retriable
2.1.0

google-api-client
0.9.19

terminal-notifier
1.7.1

word_wrap
1.0.0

xcode-install
2.0.7

fastlane-plugin-versioning
0.2.4

generated on: 2016-10-25

@AndyIsaacson  Thanks for reaching out. Can you add the --verbose flag to your run of this to see if we get any more details on why the report is taking a long time to generate? Have you tried working with the --output_style option? Thanks!

Hi @TKBurner !  Unfortunately, nothing extra comes when I add --verbose (assuming I'm doing it right bundle exec fastlane scan --verbose).  I haven't tried the output_style option, but TBH, I've been trying to figure out where the docs for these options are, and short of scouring source, haven't found a good reference.  Is there a spot where those options are documented?

@AndyIsaacson Thanks for  following up. There isn't a good resource for the documentation except by running scan help. Can you try erasing your simulators? I just want to make sure we optimize your experience. Thanks!

same issue:
[12:33:02]: ▸ Test Succeeded
+--------------------+-------+
|        Test Results        |
+--------------------+-------+
| Number of tests    | 1112  |
| Number of failures | 0     |
+--------------------+-------+

[12:33:47]: Successfully generated report at '.../fastlane/test_output/report.html'
[12:34:10]: Successfully generated report at '.../fastlane/test_output/report.junit'

#######################################################################
# scan 0.14.1 is available.
# It is recommended to use the latest version.
# Update using 'bundle update scan'.
#######################################################################

0.14.1 Hot Fix
* Fix crash when running scan on macOS projects (#6864)

0.14.0 Single scheme Xcode projects, better failure handling & more
* Properly handle report generation failures (#6788)
* Handle single scheme Xcode projects (#6637)
* Fix importing the Scanfile twice when Scanfile is in project's directory (#6598)
* Show Xcode path in summary table (#6470)
* Update internal dependencies

Update using 'sudo gem update scan'

+------+-------------------------------------+-------------+
|                     fastlane summary                     |
+------+-------------------------------------+-------------+
| Step | Action                              | Time (in s) |
+------+-------------------------------------+-------------+
| 1    | Verifying required fastlane version | 0           |
| 2    | default_platform                    | 0           |
| 3    | export LC_ALL=en_US.UTF-8           | 0           |
| 4    | scan                                | 284         |
+------+-------------------------------------+-------------+

[12:34:11]: fastlane.tools finished successfully 🎉

Would be cool to reduce the time from [12:33:02]: ▸ Test Succeeded  to [12:34:11]: fastlane.tools finished successfully 🎉 since that's about 25% of the test running time.  this is with
       scheme: "...",
       device: "iPhone 7",
       clean: true,
       skip_build: true,
       slack_only_on_failure: true,
       slack_channel: "...",

and running with bundle exec if that changes anything.

Thanks for following up! If anyone from the community is interested in looking into this, we'd be interested in a PR. Thanks!

This issue is affecting us too. Noting a lack of console output around the freeze time, I dug into this locally.
The root of the issue is that scan doesn't generate the report with the piped input, but instead waits until xcodebuild has finished. It reads the entire log file back in through a separate cat $file | xcpretty process to generate the report. Here's the most relevant source: scan/lib/scan/report_collector.rb#L48-L52
Ultimately, it might be a limitation of xcpretty if it isn't optimized to skip through all the build output to parse the test output for report generation, and we're trying to use it on a finished build log.
One workaround might involve putting the report generation options on the first (terminal output) xcpretty command, which is https://github.com/fastlane/fastlane/blob/aa34de7/scan/lib/scan/test_command_generator.rb#L91. The biggest downside here is we can't generate multiple reports in this way, unless we make some changes to the xcpretty executable. I've experimented with this locally and it seems to be promising but I have yet to test the impact on the json-compilation-database report type. Also, no clue if the xcpretty devs would accept this change, though their code suggests that they would want to support it.

Update: my bad! just realized, they already do support that, i just didn't understand how OptionParser worked in that case. https://github.com/supermarin/xcpretty/issues/135 still, i think we can make improvements using this info.

Nice @keatongreve !  I wanted to add a data point, as I've been digging into my builds a bit more.  The size of the raw output of xcodebuild for my project is ~250 MB.  Didn't expect that, but certainly adds some color to why things move slowly while processing.

Can someone please reopen this issue? I am still facing it.

Hey @RishabhTayal, https://github.com/fastlane/fastlane/pull/7365 is still open and being reviewed. I think it will be merged soon but I'm sure the fastlane team is busy. In the meantime, feel free to switch to my fork and let me know if it helps. That feedback would help.

Thanks. Hoping to get it merged soon.

@keatongreve I probably will need to use it because scan report takes about 20 min in my project, after we split code base to another 5 submodules. So now I'm facing this issue. Thanks for your input here.


undefined is not an object (evaluating '_reactNavigation.ThemeColors.light')

Current behaviour
After upgraded react-navigation from v3 to v4, it just explodes.
Expected behaviour
Just navigate to screens

Code sample
import { createBottomTabNavigator } from "react-navigation";
// Screens
import HomeScreen from "../../scenes/home/Home";
import TaskDetailsScreen from "../../scenes/task-details";

const defaultNavigationOptions = {
  tabBarVisible: false,
  header: null
};

export const Navigator = createBottomTabNavigator(
  {
    home: HomeScreen,
    taskDetails: TaskDetailsScreen
  },
  {
    defaultNavigationOptions
  }
);

Screenshots (if applicable)

What have you tried
Tried to remove every cache, node_modules, package-lock.json etc but the problem still exists.
Your Environment

software
version

ios or android
iOS 12.3

react-native
0.60.5

react-navigation-tabs
2.4.1

react-navigation
4.0.2

node
v10.15.3

npm or yarn
npm 6.10.3

experiencing the same

I have been experiencing this since Yesterday.

Hi @satya164,
I've tried all of these. Cleaned the simulator's cache etc nothing fixed this problem. Could not update the v3 to v4.

Can you provide a runnable demo?

Sure, let me create one

@satya164 I have made the example repo.
https://github.com/WrathChaos/react-navigation-tabs-error-example
FYI, I've tried the newest react-navigation v4 with a scratch project and it works well however updating from v3 to v4. It simply does not work and gives the error.

@satya164 did you have a chance to look at? I really want to migrate v4

experiencing the same, anyone find a solution?

I just upgraded from 3 to 4 and having the exact same issue.  Anyone make progress here?

@satya164 did you have a chance to look at? I really want to migrate v4

Can I ask why you closed this issue?   DId you resolve?

Reopened it. I just re-wrote the whole navigation system with v4 again :(
Could not migrate v3 to v4 also v5 is on the way :O Whole migration pain is on coming

Thanks WrathChaos.  I followed this post https://github.com/react-navigation/tabs/issues/156 and resolved my issue.
Basically, added experimentalImportSupport: true, in metro.config.js.
Then reset.
react-native start --reset-cache
watchman watch-del-all && rm -rf $TMPDIR/react-native-packager-cache-* && rm -rf $TMPDIR/metro-bundler-cache-* && rm -rf node_modules/ && rm -rf /tmp/haste-map-react-native-package && npm cache clean --force && npm install
And then everything worked...
(Although sounds like you rewrote everything)

I'm having this issue as well, the above solutions haven't worked for me.

I experience the above issue but only in my tests.
    TypeError: Cannot read property 'light' of undefined

      at Object.<anonymous> (node_modules/react-navigation-tabs/lib/commonjs/views/BottomTabBar.tsx:450:19)
      at Object.<anonymous> (node_modules/react-navigation-tabs/lib/commonjs/navigators/createBottomTabNavigator.tsx:16:1)

I managed to resolve it by introducing a mock for ThemeColors.
export const ThemeColors = {
  light: { header: null },
  dark: { header: null },
};

I then subsequently hit the following issue
    TypeError: Cannot read property 'propTypes' of undefined

      at createAnimatedComponent (node_modules/react-native/Libraries/Animated/src/createAnimatedComponent.js:178:29)
      at Object.AnimatedImplementation.createAnimatedComponent (node_modules/react-native/jest/setup.js:143:19)
      at Object.BaseButton (node_modules/react-navigation-stack/lib/commonjs/views/BorderlessButton.tsx:5:61)
      at Object.<anonymous> (node_modules/react-navigation-stack/lib/commonjs/views/TouchableItem.tsx:19:1)

I haven't been able to find a way to reproduce it in a clean repository yet, so I have included below a simplified version of the code that is causing the issue.
CustomTabNavigator-test.js
import React from 'react';
import ShallowRenderer from 'react-test-renderer/shallow';

import { CustomTabNavigator } from '../../custom-navigators/CustomTabNavigator';

describe(CustomTabNavigator, () => {
  it('Broken Test', () => {
    const shallowRenderer = new ShallowRenderer();
    const tree = shallowRenderer.render(
      <CustomTabNavigator fetchProviders={jest.fn()} />
    );
    expect(tree).toMatchSnapshot();
  });
});

CustomTabNavigator.js
import PropTypes from 'prop-types';
import React from 'react';
import { createAppContainer } from 'react-navigation';
import { createMaterialTopTabNavigator } from 'react-navigation-tabs';

import TabA from './TabA';
import TabB from './TabB';
import requireAuth from '../guards/requireAuth';
import LoadingView from '../LoadingView';
import ErrorView from '../ui/ErrorView';

class CustomTabNavigator extends React.PureComponent {
  render() {
    const {
      error,
      hasTabA,
      hasTabB,
    } = this.props;

    if (error) {
      return <ErrorView details={error}/>;
    }

    const screens = {
    };

    if (hasTabA) {
      screens.tabA = {
        screen: requireAuth(TabA),
        navigationOptions: {
          title: 'TabA',
        },
      };
    }

    if (hasTabB) {
      screens.tabB = {
        screen: requireAuth(TabB),
        navigationOptions: {
          title: 'TabB',
        },
      };
    }

    const TabNavigator = createAppContainer(createMaterialTopTabNavigator(screens));

    return <TabNavigator detached={true} />;
  }
}

export { CustomTabNavigator };

package.json
...
    "react-native": "0.61.1",
...
    "react-navigation": "^3.13.0",
    "react-navigation-stack": "^1.9.3",
    "react-navigation-tabs": "^2.5.5",
...
  "jest": {
    "preset": "react-native",
    "moduleNameMapper": {
      "^image![a-zA-Z0-9$_-]+$": "GlobalImageStub",
      "^[@./a-zA-Z0-9$_-]+\\.(png|gif)$": "<rootDir>/node_modules/react-native/Libraries/Image/RelativeImageStub"
    },
    "modulePathIgnorePatterns": [
      "^[@./a-zA-Z0-9$_-]+\\.(png|gif)$"
    ],
    "setupFiles": [
      "<rootDir>/test-setup.js",
      "./node_modules/react-native-gesture-handler/jestSetup.js"
    ],
    "setupFilesAfterEnv": [
      "<rootDir>/enzyme-test-setup.js",
      "./node_modules/jest-enzyme/lib/index.js"
    ],
    "transformIgnorePatterns": [
      "node_modules/(?!(jest-)?react-native|react-navigation|smk|@react-native-community)"
    ],
    "transform": {
      "^.+\\.js$": "<rootDir>/preprocessor.js"
    }
  }

@nar213 please upgade react-navigation to 4.x

@satya164 Apologies typo, copied that from working branch, that error occurs in 4.0.10

delete your lockfile and node_modules and reinstall, you probably have an old version of some package like @react-navigation/core or @react-navigation/native somehow

delete your lockfile and node_modules and reinstall, you probably have an old version of some package like @react-navigation/core or @react-navigation/native somehow

Sorry but this does not work if you're trying to migrate from v3 to v4.

Cheers for the advice, followed those steps but this does not fix the issue. References to @react-navigation/core and @react-navigation/native in the lock file are 3.5.1 and 3.6.2 respectively, so don't think there's an issue of referencing an old version of dependencies 🤔

just upgraded react-native to 0.59.10, did all steps like yarn cache clean, rm -rf node_modules, yarn install but still getting the same error. Using react navigation 3.0.0

I have the same problem, I tried all the solutions, nothing works. 🤔

still issue, already latest library

TypeError: Cannot read property 'light' of undefined.
same problem here

Did anyone find any solution for this?

if you have this issue then you have some version of react-navigation-tabs that is incompatible with the react-navigation version that you are using. make sure you are using the latest version for whatever major version you're using and that your lockfiles aren't preventing that. feel free to share a reproducible example if you think you have found a situation where this is incorrect

if you have this issue then you have some version of react-navigation-tabs that is incompatible with the react-navigation version that you are using. make sure you are using the latest version for whatever major version you're using and that your lockfiles aren't preventing that. feel free to share a reproducible example if you think you have found a situation where this is incorrect

Actually I've already shared the reproducible example at the above posts.
Here you go:
https://github.com/WrathChaos/react-navigation-tabs-error-example

@WrathChaos - thanks, looking

@WrathChaos - wow there is a lot going on here... in the future please make a minimal example as per https://stackoverflow.com/help/mcve

@WrathChaos - the project does not build and run successfully. please make a minimal example with no extraneous code or dependencies and make sure it runs, see https://stackoverflow.com/help/mcve for more information on how to do that.

@WrathChaos - wow there is a lot going on here... in the future please make a minimal example as per https://stackoverflow.com/help/mcve

Yes, sorry for the huge example. It just the exact example from the migration from v3 to v4. v4 itself just works perfectly, however, migration from v3 is seemed impossible.

sure. just let me know when you can create an example that reproduces the issue with the minimum amount of required dependencies and i can look into it. until then i maintain that the issue is a mismatch of versions and you need to ensure you're using the latest minor version of whatever major version of react-navigation libraries that you're using

@brentvatne Actually I tried to fix the mismatch version but it simply does not fix for the migration. Actually I have no idea why React Navigation v4 is working but it does not work from migration.

i'll leave this open and if someone can share a minimal reproducible example i'd be very happy to look at it and provide a clear solution for everyone

once I added and used the react-navigation-tabs the issue appeared. Maybe try removing that. For example, I was importing the createMaterialTopTabNavigator from react-navigation-tabs, and the issue came about. I decide to import it from react-navigation, and everything works.

once I added and used the react-navigation-tabs the issue appeared. Maybe try removing that. For example, I was importing the createMaterialTopTabNavigator from react-navigation-tabs, and the issue came about. I decide to import it from react-navigation, and everything works.

I was having the exact same problem.  This fixed it.  What are the differences between importing it from react-navigation vs react-navigation-tabs.  My only guess would be that react-navigation-tabs isn't typescript friendly as the error was a TypeError.

If you are using Expo, running expo start -c fixed this for me.

I fixed this issue by specifying the appropriate version of react-navigation-tabs (see rel notes https://github.com/react-navigation/react-navigation/releases/tag/v4.0.0)
"react-navigation-tabs": "^1.2.0",

Newer versions of react-native-tabs cause an error.

have you tried deleting node_modules and yarn.lock/package-lock.json and running yarn/npm install again?
@brentvatne do you have any idea why this is happening? they seem to have latest versions of both react navigation and tabs which should always ensure that latest core and native are installed without having to delete node_modules

this works for me.

have you tried deleting node_modules and yarn.lock/package-lock.json and running yarn/npm install again?
@brentvatne do you have any idea why this is happening? they seem to have latest versions of both react navigation and tabs which should always ensure that latest core and native are installed without having to delete node_modules

this works for me.

Worked for me too.

Thanks WrathChaos. I followed this post #156 and resolved my issue.
Basically, added experimentalImportSupport: true, in metro.config.js.
Then reset.
react-native start --reset-cache
watchman watch-del-all && rm -rf $TMPDIR/react-native-packager-cache-* && rm -rf $TMPDIR/metro-bundler-cache-* && rm -rf node_modules/ && rm -rf /tmp/haste-map-react-native-package && npm cache clean --force && npm install
And then everything worked...
(Although sounds like you rewrote everything)

This worked for me. I even din't have to reset or delete anything,  just set the
experimentalImportBundleSupport:  true //here// metro-config/src/defaults/index.js
stop dev' server and restart. Should Work

any solution for this @brentvatne ?
I use the new react-navigation in an expo previous version (in the package.json have more details)
I got a light project to show sort of the problem
https://github.com/j-avila/fakestagram.git

@j-avila in your project you're using react-navigation v4 and v5 libs in the same place.
remove these:
    "@react-navigation/core": "^5.0.0-alpha.24",
    "@react-navigation/stack": "^5.0.0-alpha.37",

+1

Just ran into this for a very different reason. Everything was working in both android and ios simulators, and then I did this:
brew update
xcode-select --install
brew doctor
sudo mkdir -p /usr/local/Frameworks /usr/local/sbin
sudo chown -R $(whoami) /usr/local/Frameworks /usr/local/sbin
brew link python
brew cleanup
brew install watchman
And then I get this same error. Clearly not an issue with node_modules or module versions.


_fzf_complete_ssh() and known_hosts file

Category

[ ] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[x] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[x] Linux
[x] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

What's the purpose of known_hosts in _fzf_complete_ssh()?

By the way, telnet version does not use known_hosts and just outputs hosts.

I'm not sure what you mean. I like that it shows hostnames that I have ever connected to.

I don't have hostnames in known_hosts.  I get |1|...=|....= useless entries from it into completion candidates.


add explicit test for fewer classes in test vs train

better to have an explicit test for the case addressed by https://github.com/NVIDIA/spark-rapids-ml/pull/738

build


[ANDROID] isValidFile method always returns false

On android, the isValidFile method on android returns and object
{   "duration": -1,   "fileType": "unknown",   "isValid": false }
This is what the path looks like: file:///storage/emulated/0/Android/data/[package_name]/cache/temp/1728725249271.mp4
On iOS everything works fine.
I am using the library react-native-image-crop-picker to pick the video file.

I guess the file is already deleted?
can you inspect it using Android studio?

@maitrungduc1410 thanks for your reply. You are correct. I was cleaning the temporal storage on component unmount.  It works fine now thanks.


Cherry picks for v0.0.6

This cherry-picks PR from main to the release/1.0 branch, in preparation for a 0.0.6 release (ahead of 1.0 later).

#81 Fix typos: missing subscript parameters (by @ejmarchant)
#82 Fix documentation for types conforming to ExpressibleByArrayLiteral or ExpressibleByDictionaryLiteral (by @ejmarchant)
#86 Stop depending on swift-collections-benchmark (by @lorentey)

@swift-ci test

@swift-ci test


request

I'm not a coder but testing 2.0 RC ...
Not quite certain, but i used the equivalent branch in the addon git?
git diff:
--- a/util/add_addon_repo
+++ b/util/add_addon_repo
@@ -18,7 +18,7 @@ fi
mkdir -p extend/addon/$2
mkdir addon > /dev/null 2>&1
-git clone $1 extend/addon/$2
+git clone -b 2.0RC $1 extend/addon/$2
if [ $? -ne 0 ]; then
exit $?

git clone copies the entire repository - all branches.
After you've added the repository,
cd extend/addon/xyz
git checkout 2.0RC
replacing xyz with your local repository name.


implement card component [ issue#54 ]

The card component is created along with the docs.

@cogor I have done the changes 😊 Please check them and let me know if more changes need to be done

@cogor I have done the changes 😊 Please check them and let me know if more changes need to be done

Thanks, I'll check them as soon as possible

@cogor Thank You for your feedback, I have done the changes and if more changes are needed let me know 😃

Everything is ok, thank you for your work and time 🙏


Fix Functions unit tests for Catalyst

As @ncooke3 mentioned, this must have been a beta issue as the failure doesn't reproduce in shipping versions of Xcode 12 or 13.


Remove version check

The current version check logic has a number of drawbacks. I.e.

It creates (otherwise unnecessary) directories and config files
It runs http requests to check latest pypi version
It produces unnecessary warnings (e.g. if the client is compatible you shouldn't need to upgrade)

All of this happens on import of the redbrick module and so there is no way to opt out of it. This unnecessarily slows down any production code which imports redbrick, inadvertently or otherwise, even if the client isn't being used. (and has side effect causing e.g. https://github.com/redbrick-ai/redbrick-sdk/issues/148)
This behavior might make sense for ex-production use cases of the redbrick SDK but I think it is otherwise a bit unorthodox.
Suggestion would be, either remove the version check, or delay it to when a client is created and allow opt-out.

Hi @andychisholm
We've released a new version v2.13.4 addressing this issue.
Now importing redbrick won't automatically do the version checking.
Could you give it a try and let us know if it works.

Thanks @pritamrungta , looking good 🚀 .


Issue#55 Add principal to Pac4jUser and add user to RoutingContext on…

… authorize
The user has to be added to the RoutingContext, I'm not sure if I chose the best spot to do this.
Also returning null as vertx user principal is not too nice. I gave it a try with a very simple conversion to JsonObject, works fine for me.

Thanks, I'll need to add an automated test or two based on your reproducer in the demo, but I think the problem you've identified is pretty much the one I suspected. I think there may be two places that need the fix (one for direct client login, one for indirect login so will think quite carefully about the fix).

Hi Hendrik,
I've been unable to work on this over the weekend, so not progressed issue #55. What I can offer is to merge your PR in now (assuming it doesn't cause any existing tests to fail) but to leave the issue open so that I can revisit it with the additional testing/validation I want to ensure. What this means is that your fix will go in temporarily, but when I close issue #55 you might wish to re-test to ensure that I don't break any of your client code. If you'd prefer I can hold off altogether but it seems to me you might prefer to be able to use an up to date snapshot containing your PR as a temporary measure.

Hi Jez,
I go into winter holidays end of next week and actually would appreciate to switch back to pac4j:master before that. But I don't want to cause you any additional work, while nothing is happening on that branch I'm fine with my local fix. Thanks!

That’s cool, I’ll try and get merged in this evening, though issue will remain open so I can do a more extensive review and fix later (I think there is more work to be done for completeness).

On 7 Dec 2015, at 08:48, HendrikLanghammer<EMAIL_ADDRESS>wrote:
Hi Jez,
I go into winter holidays end of next week and actually would appreciate to switch back to pac4j:master before that. But I don't want to cause you any additional work, while nothing is happening on that branch I'm fine with my local fix. Thanks!
—
Reply to this email directly or view it on GitHub https://github.com/pac4j/vertx-pac4j/pull/56#issuecomment-162450255.

This is merged, but issue will remain open pending review against the other considerations i recorded on it


Be precise about B1plusmap, B1minusmap and B1map.

Hey,
I notice that there are some inconsistencies in the spec right now. So in general, people seem to use B1map and B1plusmap interchangeably. But for the MPM examples, we use B1minusmap. What should we do here? Replace everything with B1plusmap?
-Gilles

So the options would be:

B1map and B1minusmap
B1map and Sensmap
B1plusmap and B1minusmap
(B1map + B1plusmap) and B1minusmap

First off, would 4 be possible? I've almost never seen anyone who uses B1map to refer to B1minusmap. Can we somehow accommodate that interchangeability in the spec? I know this is not ideal, but don't know which one would be more expensive. Changing everything or having two labels pointing to the same entity?

I think B1map is not in the spec yet. Only in some of our other examples.
So I really prefer option 3.

Let's discuss this in the meeting later.
On Wed, 15 May 2019, 16:32 Agah<EMAIL_ADDRESS>wrote:

So the options would be:

B1map and B1minusmap
B1map and Sensmap
B1plusmap and B1minusmap
(B1map + B1plusmap) and B1minusmap

First off, would 4 be possible? I've almost never seen anyone who uses
B1map to refer to B1minusmap. Can we somehow accommodate that
interchangeability in the spec? I know this is not ideal, but don't know
which one would be more expensive. Changing everything or having two labels
pointing to the same entity?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bids-standard/bep001/issues/51?email_source=notifications&email_token=AAIPRTW3XXVM6VWQ7HRJEYTPVQNIDA5CNFSM4HND3NK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVO3KJI#issuecomment-492680485,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIPRTQALNWPEEQHR7WNTNTPVQNIDANCNFSM4HND3NKQ
.

Need to go back to method paper to find out.

Let's discuss this in the meeting later.

Cool!

+1 on option 3 as well then.

From the original paper:

The MPM comprised three multi-echo 3D FLASH (fast low angle shot) scans, one RF transmit field map and one static magnetic (B0) field map scan (Weiskopf et al., 2011).

The h-MRI toolbox allows generation of a B1minus map (N4ITK kind of bias correction) if inputs for B1map fitting (3D segmented EPI) are not provided by user. At 3T, B1+ and B1- are fairly comparable, but not quite sure how processing advances in that case.

OK, but a standard MPM protocol then apparently includes a B1 plus map
right?
On Wed, 15 May 2019, 16:56 Agah<EMAIL_ADDRESS>wrote:

From the original paper:
The MPM comprised three multi-echo 3D FLASH (fast low angle shot) scans,
one RF transmit field map and one static magnetic (B0) field map scan
(Weiskopf et al., 2011).
The h-MRI toolbox allows generation of a B1minus map (N4ITK kind of bias
correction) if inputs for B1map fitting (3D segmented EPI) are not provided
by user. At 3T, B1+ and B1- are fairly comparable, but not quite sure how
processing advances in that case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bids-standard/bep001/issues/51?email_source=notifications&email_token=AAIPRTXBX3O7WYDS7YUADRDPVQQBTA5CNFSM4HND3NK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVO5ZWI#issuecomment-492690649,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIPRTUDV7AC5ATVC5UUIITPVQQBTANCNFSM4HND3NKQ
.

@christophephillips - can you answer this question? What is expected for the MPM protocol?

As far as I understand MPM/hMRI uses both transmit and receive fields (although if I’m not wrong Weiskopf 2013 only used B1+, B1- was added after and is now part of hMRI in Tabelow 2019). The MPM protocol I use definitely include both. I’d love to hear confirmation from actual MPM experts like @ChristophePhillips  though!


Roy

Adds supporting styles to stylesheet and reformats indents(I'm using Prettier autoformat on save and it attacked the file.). Added a span with classname for roy.

Thanks Roy, awesome stuff!


does this work on flutter web

does this package work in flutter web??

No it wont work
Hello Please use my new plugin which supports youtube for inline
ext_video_player
Thanks


Hotfix/#400 #417 project creation

create missing experimental design object necessary to view project in projectbrowser

 Test and address remaining exceptions in the create project process

I left some comments. Other than that I tried to test it but got some errors:
Failed to execute goal org.codehaus.gmavenplus:gmavenplus-plugin:1.12.1:compile (default) on project offer-manager-app: Error occurred while calling a method on a Groovy class from classpath.

You need Java SDK 8 not 11 :)

I left some comments. Other than that I tried to test it but got some errors:
Failed to execute goal org.codehaus.gmavenplus:gmavenplus-plugin:1.12.1:compile (default) on project offer-manager-app: Error occurred while calling a method on a Groovy class from classpath.

You need Java SDK 8 not 11 :)

Why was it required to change it?

I left some comments. Other than that I tried to test it but got some errors:
Failed to execute goal org.codehaus.gmavenplus:gmavenplus-plugin:1.12.1:compile (default) on project offer-manager-app: Error occurred while calling a method on a Groovy class from classpath.

You need Java SDK 8 not 11 :)

Why was it required to change it?

1st, we run on JRE 8 on production (this is why the central Github actions for package and deploy use Java JDK 8)
2nd, @wow-such-code makes use of a javax.xml.bind feature, that is not included natively in Java 11 anymore.