fatal: unable to access 'https://chromium.googlesource.com/external/gyp.git/'

when I install wren on my Mac like docs:
$ git clone https://github.com/munificent/wren.git
$ cd wren
$ make

I got following errors:
Cloning libuv...
Getting tags...
Checking out libuv v1.10.0...
Downloading gyp...
fatal: unable to access 'https://chromium.googlesource.com/external/gyp.git/': Failed to connect to chromium.googlesource.com port 443: Operation timed out

make[1]: *** [deps/libuv/build/gyp/gyp] Error 128
make: *** [release] Error 2

I tried again and again, but ..., please help me, tks.

Hmm, it's working fine for me. Does visiting https://chromium.googlesource.com/external/gyp with your browser work?

@bjorn
Yes, in browser, the url: https://chromium.googlesource.com/external/gyp works.

Are you on a network which needs a proxy configured, such as a work or school network? It could be that your browser has detected the proxy settings but the command line doesn't know about them.

@iwillspeak
Yes, maybe, I try to download the tgz: https://chromium.googlesource.com/external/gyp.git/+archive/master.tar.gz and place it under the build folder


Update: Runtime SDK to Runtime API

Please see the details in common-samples/issues/2578
Per Rachael

The new developer experience has renamed our product from the ArcGIS Runtime SDK for platform to ArcGIS Runtime API for platform. We should update any references to the former across our sample viewers and repos.

This PR updates all the SDK to API where I think would be proper. Also, 404 URLs are fixed. Some of the changes are adapted from Java's PR: https://github.com/Esri/arcgis-runtime-samples-java/pull/623
Be aware: the name of the project, AppStore's app name and other sdk that is baked into the URL are not updated. Will discuss with Divesh to see if that is appropriate.

Divesh mentioned there are currently no plan to change the name of the product, so let's hold off.


1.0 release

Hey! I have been testing Regle and it looks really good. I would say more stable and with less bugs than Vuelidate. Thank you for fixing all the bugs.
What do you think about releasing version 1.0 stable? Currently it gives the idea that this is not to be used in production apps which is also what README says:

⚠ Project is still in development, do not use it in production API or function names can still change

I would suggest releasing 1.0 and let people start using it for real, report any issues and help the project grow. I think most people see this warning and run away immediately which is a shame because this is shaping up to be a really good library. If we release 1.0 more people will join in testing and reporting bugs as well.
Perhaps the only real missing piece would be to improve the documentation which I want to send in a PR soon.  :)
But honestly, it feels pretty stable already! Awesome job! 🥳

Thanks! 😍
Yep it's definitely planned, but I want to be the 1.0 to be robust. I'm still patching edge cases on my side.
Thanks for contributing to the docs! It's really helpful as it's not easy
I have a little roadmap before releasing the 1.0.
For v1.0

[ ] Unit tests
[ ] E2E tests
[ ] Type tests
[ ] Improve coverage
[ ] Simplify/reduce core code
[ ] Example repo, Reproduction repl

Next iterations

[ ] Valibot support
[ ] Yup support
[ ] TS docs
[ ] withErrorType

Awesome! Maybe a gift for Christmas 🎁

I'm just missing a advanced example repo and this would be good for release :D

I'm just missing a advanced example repo and this would be good for release :D

Awesome! 🥳  I am also currently working on a few more examples to add to the docs, something that I myself would find useful if I was a beginner. Will send in a pull request in the next few days.

So cool! Thank you 🫶🏻🫶🏻

I'm just missing a advanced example repo and this would be good for release :D

I'll tell all my friends about it when 1.0 is released 👼🏻

Yep I hope next week will do! I'm still notifying some edge cases while doing the advanced example, but i'll present it to my company monday morning too


Support TRRosetta NPZ Files

Necessary Changes

Write a TRRosetta_MSADataset class for parsing the MSA from the npz file.
Add a .npz branch to read_contacts.

confirm that the following does work.
mogwai-train <pdb.npz> --structure_file <pdb.npz>

Need to make sure TRRosetta one-hot MSA is converted to an int encoding that uses our vocab. Not sure what the TRRosetta vocab was.


The qzl sql --type option has no help attached.

The sql --type option has no help attached.
Steps to recreate:
qzl sql -t --help
Output:
Parameter '-t' is incorrect.  Enter qzl sql --help for more information.
c:\snjw\code\shared\qzl>qzl sql --help
Usage: qzl sql [options]
Executes an Sql Method.
Options:
-p|--provider      Provider name.
-c|--connection    Connection string or file.
-q|--query         SQL query.
-m|--method        SQL method (reader/scalar/nonquery).
-s|--source        Full path to a SQL query in a file.
-f|--formet        Format of output (XML/SQL/XLSX).
-o|--output        Full path to output file.
-t|--type          Output type.
-d|--dqchar        Character to replace with '"' in query/connection string.
-v|--verbosity     Level of information displayed in console.
-h|--help          Display help.
Note that --formet is mistyped.

Fixed in 1.0.6 by removing --type and folding this into -o and -f


JDK9 artifact/converter naming

Hey,
so we just now merged the "final bits" for the JDK9 Flow bridge/adapter jar as well as the TCK for java.util.concurrent.Flow.
Now, there are some open questions that we need to lock down now, before we cut the releases:

Naming conventions on converters:
Current scheme: toX (toSubscriber, toPublisher, toProcessor) and toFlowX (toFlowSubsriber, toFlowPublisher, toFlowProcessor)
Current rationale: harmonizes with the types: X and Flow.X
Naming of converter class.
Current name: The converters are currently located in a class named org.reactivestreams.ReactiveStreamsFlowBridge
Naming of converter jar
Current artifact name is reactive-streams-flow-bridge
Current rationale: it is intended to bridge implementations in both directions

@reactive-streams/contributors I'd like for us to settle the naming swiftly and with resolve (can we aim for within a week?), so if you have counter proposals, please speak now of forever hold your peace. :)

One possible proposal is:

keep toX, toFlowX
Rename converter class to org.reactivestreams.Adapt
Rename converter jar to reactive-streams-flow-adapter

Maybe like this? :)
<dependency>
  <groupId>org.reactivestreams</groupId>
  <artifactId>adapter-flow</artifactId>
</dependency>

@shenghaiyang All the other artifacts start with reactive-streams so it would seem asymmetric to change that. :)

The toPublisher / toFlowPublisher are idea, it's good to have the name in there and that plays well with the Flow mnemonic, it's also short. The alternatives were with including "ReactiveStreams" or "RS" which is a bit weird.

In Java I'm usually a bit afraid to take "the good names" (Adapt), since conflicting in names is a pain in Java (no import renames like Scala etc). Those methods are likely to be used via a static import anyway I think, so the name does not matter too much. Sure it reads nice and fluently, but that's not really what this lib is about. Would rather use a boring name like ReactiveStreamsAdapters or something like that, also easier to find using autocompletion or searching-for-classname.

Module name sounds good, should align with the above, but that's likely good. -adapters I guess would be my preferred, it's not a single adapter, but each for each type. Other ideas would be -compat but that's perhaps meh, and does not really explain what exacly it is, so -adapters win AFAICS.

Hope this helps :)

@ktoso I agree with your point number 2. I'm fine with pluralizing adapter in the jarname.

👍

@ktoso Would org.reactivestreams.FlowAdapters make sense?

Nice idea, reads great 👍

@reactive-streams/contributors I created #402 as a concrete proposal of name changes. Please provide some counter proposals, or :+1:'s so we can ship the JDK9 TCK and interop pieces as soon as possible. (Would be nice with an RC1 within a week or so)

Closing as changes were merged without additional comments.


Create a Event.[Kind.]Snapshot type that conforms to Codable

This facilitates progress integrating the testing library with related tools by allowing Events generated by tests to be serialized.
Motivation:
This is the sixth and last in a series of PRs (#67, #68, #69, #73, #79) to make types in the hierarchy of Event conform to Codable, or create Snapshot types for types that can't / shouldn't become Codable.
Modifications:
I created two snapshot types that contain copies of all stored properties of Issue and Event.Kind. I've also added a test that encodes and decodes various Event / EventSnapshots.
Result:
Event and Event.Kind themselves do not conform to Codable but there are equivalent Event.Snapshot and Event.Kind.Snapshot types that are.
Resolves: rdar://117054776

@swift-ci please test

@swift-ci please test

@swift-ci please test


Speed up bandwidth

Hi!
I just installed your VPN with Docker, it works!
I can now use the VPN of my server from a client but the bandwidth is limited to 10Mega.
Here are the tests:
[VPN Enable]

[VPN Disable]

Can you explain to me how can I change that?
Thank you !

Hi!
Absolutely, now, i encounter another problem: https://github.com/kylemanna/docker-openvpn/issues/462
I will be able to retest the bandwidth once this resolves...
Bye!


feat(aws-fargate-sqs): Created new construct

Issue #587 , if available:
Description of changes:
-Created new construct with Fargate connected to SQS
-Added environment variables
-Able to create VPC, service, queue, and accept existing ones
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
fixes #587

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: f30b9ec81f51f7800075f8f19d30b192c6c7a760
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: f30b9ec81f51f7800075f8f19d30b192c6c7a760
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 7bd27ddafbc0053609947a7d781add05657ea741
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 7bd27ddafbc0053609947a7d781add05657ea741
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 645475b67e1cc225ea359c30da7f91338bab1dba
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 645475b67e1cc225ea359c30da7f91338bab1dba
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: d64658a4f2250b40a4487bd04c5d7d3196dc6223
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: d64658a4f2250b40a4487bd04c5d7d3196dc6223
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 6f1c67ff7df5ebeafc635d1c978e0d85fc7d2779
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 6f1c67ff7df5ebeafc635d1c978e0d85fc7d2779
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: f9d5e67814104deb91a400e8720dcf0af9a68e32
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: f9d5e67814104deb91a400e8720dcf0af9a68e32
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: f191ad2655478fa9ef905d5d10c4caf17136c04f
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: f191ad2655478fa9ef905d5d10c4caf17136c04f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: f191ad2655478fa9ef905d5d10c4caf17136c04f
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 74e74c60526e17763edff9d9f5039d85920baff3
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 74e74c60526e17763edff9d9f5039d85920baff3
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 65a451252ad4b38b4f137be2be6025ce01f8a69a
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: b753cd96143923b8c149fa3b9f791c3a689a4cb6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: b753cd96143923b8c149fa3b9f791c3a689a4cb6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Impossible to install a fresh project

Hello,
I launch a fresh installation with this command: composer create-project --prefer-dist nucleon/nucleon app-root
I have this error message:

Declaration of Neutrino\Foundation\Kernelize::registerModules(array $modules, $merge = false) must be compatible with Phalcon\Application\AbstractApplication::registerModules(array $modules, bool $merge = NULL): Phalcon\Application\AbstractApplication in C:\Users\toto\projects\app-root\vendor\nucleon\framework\src\Neutrino\Foundation\Cli\Kernel.php on line 70

Config: windows 10, PHP 7.4, Phalcon 4.
Thanks for your help

Hello,
For the moment, the nucleon is not compatible with phalcon 4.
I am working on nucleon 2.0 which will be compatible with phalcon 4 and PHP 7.4.
Meanwhile, nucleon remains usable with phalcon 3.4 and PHP 7.3.

Close in favor of :
phalcon-nucleon/nucleon/issues/18
and
phalcon-nucleon/framework/issues/68


More functions

Added more assertion functions to the core, which should catch this library up to node's "assert" module - for the most part. This commit also adds other useful assertion functions I love from Chai, like isTrue, isFalse. You can check out the docs for node's "assert" and verify that these functions match that specification if you wish.
This commit also adds tests for these new functions.

Sorry, if you need this you're better off with assert or forking :)


fix: allow empty array as default value for a prop

Allow empty array as default value for a prop.
Checklist

[x] run npm run test and npm run benchmark
[x] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message and code follows Code of conduct

The tests are not failing (?!)
Checking the spec it seems this is a valid schema tho
{
    type: 'object',
    properties: {
      asd: {
        type: 'array',
        items: { type: 'integer' },
        default: []
      }
    }
  }

@Eomm That's weird. It is failing on my machine 🤔

Oh, travis did not run 😕
Anyway, it fails 👍 the case is good!
Would you like to fix it?

Here you go, pushed a fix.

cc @aboutlo


Release

The version 0.1.0 does not allow to create new k8s clusters, can we have a release ?

released


WinForms: improve the designer experience for .NET Framework apps

resolve #4362
After investigation, it was found that FormsPlot inherits FormsPlotBase, and its member Plot will call the method in SkiaSharp, so FormsPlotBase also needs to be adjusted.
Adjusted to detect whether FormsPlotBase throws an exception that libSkiaSharp.dll cannot be found, and display the current version and error message on the control.
like this

I found that I had to be careful whether FormsPlot properties were set in Form.Designer.cs.
If the property calls SkiaSharp, it may also affect the control and cause an error.🤔

Wow @CoderPM2011, great find! This is fantastic! 🚀

Hi @CoderPM2011 - I'm curious, how are you testing this?
I have a SDK project that I set to target net48 and I'm getting this 😅

Oof, it seems to be failing for me the same way when I try to load any control (including a blank one) from the ScottPlot.WinForms assembly 💀

Hi @CoderPM2011 - I'm curious, how are you testing this?

I first have the ScottPlot.WinForms project generate the Nuget package.
Then in the newly created .Net Framework 4.6.2 project, install the locally generated Nuget package for testing.

Hi @CoderPM2011 - I'm curious, how are you testing this?

I first have the ScottPlot.WinForms project generate the Nuget package. Then in the newly created .Net Framework 4.6.2 project, install the locally generated Nuget package for testing.

Ah, interesting! I've only tested this running from source which may explain why I've had difficulty reaching a solution.
Thanks again for your feedback and this PR! I'm excited to give this a try soon 👍

Ah, interesting! I've only tested this running from source which may explain why I've had difficulty reaching a solution.

I'm guessing that the reason for the failure to work from the source code may be related to multiple target frameworks or whether VS itself is running properly. 🤔

In my repository branch test-#4362.
I tested the pattern directly referenced in the source project.
I found that the multi-target framework does affect the toolbox.
Maybe it has something to do with this https://github.com/dotnet/msbuild/issues/7856 .
If I change ScottPlot.WinForms to only target net462 then I get it working.
Sometimes you encounter an exception. After restarting Visual Studio, try again and it will work normally.
My test result

I found that the multi-target framework does affect the toolbox.

 I got it working now and will refine the error screen a bit. This is what I'm thinking of having it show when the DLL fails to load:

... but if this is successful after the next release, I'm tempted to use System.Drawing.Graphics to manually draw a title, grid, and axes, and display it as a Bitmap in a PictureBox and I bet most people will have no idea there was a failure! 🚀

I'll merge this now, but note that it has been modified to only show this message if design time mode is detected. The reason is I don't want that message accidentally showing up on client machines of a third party publishes an app using ScottPlot. Instead, we can explore adding a new PR to display a user-facing message (not aimed at developers necessarily) for when the DLL cannot be located.
This is a semi-common issue actually, unrelated to running inside visual studio, because sometimes people build an app but forget to include all those platform specific subfolders (e.g., I once published a WinForms app and omitted the arm folders but someone with one of those new windows arm tablet PCs couldn't load the app)


C# ARM Support

I see that C# version doesn't allow running using ARM native binary. Is there any reason for that? Is there something that is known to not work on ARM?
I also don't see anywhere in documentation written which architectures are supported. I think it would be best to support arm in general and to have documentation how to cross-compile the C part for arm on Linux.

There is nothing that would prevent C# from being used on ARM in principle

It should be possible to compile C core on ARM (but currently we don't provide any documentation for that).
The native extension loading C# code would need some very minor changes.

The main reason why we haven't done this yet is that we currently don't support Unity or Xamarin, which are the platforms where one would benefit from ARM support (mobile phones). For desktop and server C# (.NET, mono, .NET core) the use for ARM support seems limited - do you have a particular use case in mind?
This issue should probably have been a topic on grpc.io forum, so closing this issue.

My use would be Mono on an embedded ARM Linux server, I don't see much limitations in using ARM on Mono. Also note that Microsoft will soon release desktop Windows ARM version, so even for desktop Windows it will be good to have it.

I really loved if I could use gRPC on an ARM-based embedded Linux! Could you point me to any documentation about how to cross-compile the C core? Thanks.

@tamaskaroly  sorry, we currently don't have any official docs for how to crosscompile C-core.
C# support for ARM is currently not on our roadmap, but in order to make it work, you'd only need to crosscompile libgrpc_csharp_ext.so  for your platform. You can try either with our Makefile (make grpc_csharp_ext) or by using cmake (the C# extension is one of the easier targets to build).
Feel free to share your experience once you're successful.

I managed to compile grpc_csharp_ext using cmake! All I needed to do was

"source" your cross-compile environment (I was using the linaro toolchain)
mkdir build in the grpc repo root
cd build
cmake .. -DCMAKE_TOOLCHAIN_FILE= -Dprotobuf_BUILD_TESTS=Off -DgRPC_INSTALL=Off -DCMAKE_CROSSCOMPILING=1 -DRUN_HAVE_POSIX_REGEX=0
cd ..
make grpc_csharp_ext

@jtattermusch What is the strategy behind encoding the platform in the file name? (eg. grpc_csharp_ext.x86.so) Is it because of x86/x64? I'm asking because these .so files are huge, and as it is now, the NuGet packet includes both the x86 and the x64 version (~10Mb each). I don't think it would be a good idea to add the arm version as well (yet another 10Mb). Instead, the NuGet packet should be able to target multiple architectures. I can look into this if you want. Once done, the native extension loading C# code would also need some minor changes.
On a more generic tone, ARM support is key! With .Net Core now supporting ARM, gRPC becomes a really compelling communication protocol (I don't consider Mono production quality). REST is great, but when it comes to server-side notification or active content push, you need to go beyond REST.

@tamaskaroly  thanks for the input! Very useful to have a proof of concept on ARM.
AFAIR the names of the files (grpc_csharp_ext.x86.so) are based on what native nuget packages require (with the right subdir + filename, the native libraries are copied to the output directory when building automatically on newer platforms. On older platforms we have a msbuild .targets file that does that instead).
On an unrelated note, I think the binaries should be smaller than 10MB (I'd expect ~1MB each version?) - we can take a look at what's going on.
I can see mutiple ways to support ARM:

most basic publish docs how to crosscompile + implmenent a way to define custom path to native extension loader (then you can provide the right path to the crosscompiled extension manually at your app startup) - we even have an issue for this https://github.com/grpc/grpc/issues/4753
provide an additional nuget package with pre-compiled ARM binaries. (Grpc.Core.ArmSupport?) After improving the native extension loader, just adding this depedency would make things work on ARM without any extra code needed on user side.
include all native libraries in Grpc.Core, but as you pointed out, that might make the nuget too big in size and it would also pose a challenge to automated building (we'd need to build on ARM also).

Btw, are you talking about ARM on Windows or under Linux?

@tamaskaroly
Hi Tamas, I to am trying to compile but I keep getting Cmake errors. Did you encounter these errors yourself please?

Also I do not fully understand this ""source" your cross-compile environment"
Thanks in advance :)

@tamaskaroly
Hi Tamaskaroly, I am in the process of trying to build GRPC for ARM. I tried to follow your steps above but after running cmake I keep running into the following error:
make: *** [C:/Users/kgrima/Desktop/GRPC-ARM/ARM2/grpc/objs/opt/src/core/lib/profiling/basic_timers.o]
Did you face any issues yourself?


[full-ci] fix: always select next gateway client

Description
Needed for #10123
@d7oc This is a very important bugfix for kubernetes environments.
Related Issue

Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

I think we could reuse one selector per request. The PR gets a new selector each time we need to contact the gateway, which I'm not sure if it's a good idea. If we want to go this way, a helper method might keep the same complexity.
For example, the RenameFile has several f.gws.Next(), but maybe we only need one for the whole request.

I think we could reuse one selector per request. The PR gets a new selector each time we need to contact the gateway, which I'm not sure if it's a good idea. If we want to go this way, a helper method might keep the same complexity.
For example, the RenameFile has several f.gws.Next(), but maybe we only need one for the whole request.

Discussed that with @butonic
we need to do it every time, because a pod can disappear between two requests.
The „good“ solution will be using a client which can do the next selector by itself.

The „good“ solution will be using a client which can do the next selector by itself.

Canceling this for now because it seems too much work for little gain.
We'd need to implement the whole GatewayAPIClient, which is huge, just to hide the selector code inside our custom client. I think we should enrich our client with something more in order to make it worthy.
I've tried to include a retry policy in case of network errors so the next request would go to a different gateway, but it seems we don't have guarantees that the next request wouldn't go to the same gateway. Right now, the selector policy is random (we could still hit the same gateway), and even with a round robin policy, since the client is shared, concurrent requests could cycle through the clients, so the retried request might still go to the same gateway.

We can just use the go micro client, which is already capable of doing that. And this also applies for http


Float problem on yAxis when all data is decimal

I'm using the latest 2.0.0-dev build (Jan 9th) which contains fix #1567.
In a line chart, when my data is decimal there are floating point problems with the tick labels.
            var ctx = $("#testchart");
            var myLineChart = new Chart(ctx, {
                type: 'line',
                data: {
                    labels: ["1", "2", "3"],
                    datasets: [
                        {
                            label: "My First dataset",
                            data: [0.3, 1.2, 0]
                        },
                        {
                            label: "My Second dataset",
                            data: [0, 0.3, 1.2]
                        }
                    ]
                }
            });
        });

That data renders:

Just an addition.  I tried overriding the callback on the yAxis tick to log the value being passed in for tickValue and it already had the rounding problemon it.  Hope this info helps.

I have the same problem with a single dataset, the problem seems to go away when the canvas is narrow though!
The problem happens in the function buildTicks() when the ticks are calculated as a multiple of spacing, since in theory they shouldn't have more decimal positions than spacing (because the multiplier is integer) I round them as a workaround:
I use function decimalPlaces() from http://stackoverflow.com/questions/10454518/javascript-how-to-retrieve-the-number-of-decimals-of-a-string-number
function decimalPlaces(num) {
  var match = (''+num).match(/(?:\.(\d+))?(?:[eE]([+-]?\d+))?$/);
  if (!match) { return 0; }
  return Math.max(
       0,
       // Number of digits right of decimal point.
       (match[1] ? match[1].length : 0)
       // Adjust for scientific notation.
       - (match[2] ? +match[2] : 0));
}

And round niceMIn, niceMax and the ticks in buildTicks()
			var niceRange = helpers.niceNum(this.max - this.min, false);
			var spacing = helpers.niceNum(niceRange / (maxTicks - 1), true);
			var roundFactor = Math.pow(10, decimalPlaces(spacing));
			var niceMin = Math.round(Math.floor(this.min / spacing) * spacing * roundFactor) / roundFactor; // Round
			var niceMax = Math.round(Math.ceil(this.max / spacing) * spacing * roundFactor) / roundFactor; // Round

			var numSpaces = Math.ceil((niceMax - niceMin) / spacing);

			// Put the values into the ticks array
			this.ticks.push(this.options.ticks.min !== undefined ? this.options.ticks.min : niceMin);
			for (var j = 1; j < numSpaces; ++j) {
				this.ticks.push(Math.round((niceMin + (j * spacing)) * roundFactor) / roundFactor); // Round
			}
			this.ticks.push(this.options.ticks.max !== undefined ? this.options.ticks.max : niceMax);

Hope this helps.

As a workaround I configured my yAxes ticks callback to simply be the following which works for my usage (simple graph displaying rainfall rate and accumulated rainfall over 24 hours).
function (tickValue, index, ticks) {
            return Number(tickValue).toFixed(2);
        }

Fixed in #1984


Bug in Suggester search algorithm

If I search for "trp started", Suggester finds nothing, while it should find three entities having these words in the label (and that can be found by "started" alone). E.g., it does not match "TRP started during", while it should.

@jancimertel Is there a possibility to implement a string similarity algorithm (Levenshtein distance) inside r.filter in the database query without causing any crucial reduction in speed?

@adammertel I restate the need for change here. E.g. if I search for "is" Action type, it does not change it, and so the user will create a new one. The search algorithm is too tolerant, searching too much within the entity. Change it to something which is stricter on the left, i.e. prioritize in Suggester search results (or require) the beginning of the string. This way, when searching for "is", I will find "is" and non "misit", "subrisit" etc.

Uploading search.mkv…
what about this?

Uploading search.mkv…
what about this?

https://user-images.githubusercontent.com/10438157/162617494-6693afec-2cff-476f-8ddc-f3514899c4bf.mp4
what about this?

@jancimertel : very nice, we want this :-) Can you make the same wildcard also in Suggester? I think the search algorithm should be the same in Search and in Suggester.

Another instance of the bug: if searching for "Sentences Caux", I do not find the T entity "Sentences of Bernard of Caux and John of Saint-Pierre".

@jancimertel : very nice, we want this :-) Can you enable the same wildcard also in Suggester? I think the search algorithm should be the same in Search and in Suggester.

@jancimertel : very nice, we want this :-) Can you enable the same wildcard also in Suggester? I think the search algorithm should be the same in Search and in Suggester.

yes, sorry, the suggester/search should work the same way

Another instance of the bug: if searching for "Sentences Caux", I do not find the T entity "Sentences of Bernard of Caux and John of Saint-Pierre".

This is expected behavior when using such database querying. As it has been mentioned somewhere, similar searching should be implemented in search index ie. elasticsearch. I can mimic it to some degree using only db+backend.


Optimization of TypeChecker config function

The function config' in TypeChecker.Cfg can be called frequently in the Normalizer.
This PR optimizes the config' in the following ways:

when debug switches are off which cuts lookups into the Options table.
avoid searching the list s for PureSubtermsWithinComputations  because this information is already steps following to_fsteps.

I see the following results from benchmarking the change (--smt_admit_queries true) on Linux:
                  Old                               New
                  n       total   user    system    n       total   user    system
micro-benchmarks  61      60.09   55.73   3.918     61      57.03   52.85   3.731
ocaml_extract     38      120.6   116.4   4.146     38      117.4   113.1   4.306
ulib              208     399.8   378.6   19.6      208     380.6   359.3   19.68

Thanks Tom, this looks good. By the way, I suspect that the Env.debug calls that are sprinkled everywhere might be big source of inefficiency, specially when there actually are some debug options given. We should probably use some set-like structure for these options instead of the string list.


feat: admin sees saved search banner

The type of this PR is: FEATURE

This PR resolves FX-2964 and FX-2965
Description
For FX-2964
As an admin,
When I visit the artist artworks tab with the "saved search" feature flag enabled
I see a banner with a toggle
Acceptance criteria/approach

Toggle on/off changes banner background and text color
Is sticky with the filter bar, staying at the top of the screen as the user scrolls down and up the screen
Not in scope: conditional display. For this ticket, banner is always shown (when feature flag is on).
Not in scope: any querying or mutations. This ticket is UI-only

For FX-2965
As an admin with the saved search feature flag enabled
When I visit the artworks tab for an artist
I do not see the saved search banner
And when I apply a set of filters
I see the saved search banner
Acceptance criteria

Any set of filters triggers the banner
Not in scope: checking whether the user already has this search saved
Not in scope: any toggle behavior

Demo
https://user-images.githubusercontent.com/3513494/120364028-70417680-c315-11eb-9e2a-1c15d65e5b08.mp4
https://user-images.githubusercontent.com/3513494/120365307-f14d3d80-c316-11eb-9e82-0ee4403ebb85.mp4

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

Fails

:no_entry_sign:

The CHANGELOG is not valid YML:
YAMLSyntaxError: Document is not valid YAML (bad indentation?)
at /home/circleci/project/node_modules/yaml/dist/Document.js:219:28
at Array.forEach ()
at Document.parseContents (/home/circleci/project/node_modules/yaml/dist/Document.js:215:14)
at Document.parse (/home/circleci/project/node_modules/yaml/dist/Document.js:148:10)
at parseDocument (/home/circleci/project/node_modules/yaml/dist/index.js:109:37)
at Object.parse (/home/circleci/project/node_modules/yaml/dist/index.js:120:15)
at dangerfile.ts:127:39
at Generator.next ()
at dangerfile.ts:27:71
at new Promise ()

  Generated by :no_entry_sign: dangerJS against eafbbb964e5f3b06c93cb3f7df0eef6834dd654b

Did a bit of diff merging and the CHANGELOG.yml file changes look as expected now:

#squashongreen


remove admin_enabled

Description
Fortify wants us to not have admin_enabled set to true on the container registry. This PR changes that setting and turns on container_registry_use_managed_identity instead. To test this, we deployed to both a Flexion environment (this PR env) and a CDC one (dev) and confirmed the app deployed successfully and was running
Issue
https://github.com/CDCgov/trusted-intermediary/issues/1209

Would it be beneficial to include why Fortify wants this?

The Fortify rec is actually pretty vague - this is all it says


Added html attribute autocomplete

It is my understanding that the autocomplete attribute has a finite number of potential values:
https://developer.mozilla.org/en-US/docs/Web/HTML/Attributes/autocomplete
To that end, I think we should go farther in our support for it by implementing an enum that encompasses all of them. This will improve the ergonomics for our end users, by saving them from having to consult additional documentation and instead just examine the enum options.

Thank you for your contribution, and for implementing deeper support!


Icons not working on touchbar

According to https://github.com/Microsoft/vscode/issues/35641, icons should appear in the Touch Bar (that issue is about allowing them to be overridden from the command). However, in my testing this doesn't work... I have a command that has icons:
{
	"command": "flutter.hotReload",
	"title": "Hot Reload",
	"category": "Flutter",
	"icon": {
		"dark": "media/commands/hot-reload.svg",
		"light": "media/commands/hot-reload.svg"
	}
},

And it shows up fine on the debug toolbar:

However it just shows up as text in the Touch Bar:

I can't find any solid information on whether this is intended to work or not, only that other issue which shows a screenshot of it working.

@bpasero I was not aware that we have made the touch bar menu contributable.
Might you know why the icons do not apear for Dan? I know that you had to change these icons a while back, do they require some special format?

Only PNGs work!

@bpasero Aha, I'll give that a go! Is that documented anywhere (I did look, but couldn't find much on the Touch Bar).


Fix nil pointer exception

xref: https://github.com/appscode/kubed/issues/304#issuecomment-508024105
Signed-off-by: Tamal Saha<EMAIL_ADDRESS>

@tamalsaha is this planned to be part of an upcoming release?

Yes. I looking to cut a release in a 1-2 weeks.

@tamalsaha when is this getting release? Thanks

Left feedback on #363, but can that be a follow-up to a new release that contains this fix?

Is the memory problem fixed by this? If so would like to have a release of this please. Thx

@mbelang I haven't done this yet, but good feedback may be to build the app/image with this fix in place, deploy it and see if it fixes the issue?

I have released v0.11.0 https://github.com/appscode/kubed/releases/tag/v0.11.0 . Please give it a shot.

I'm testing it at the moment. I monitor for memory increase.
Out of curiosity, how much resources you recommend?
I currently set for limits and resources:
cpu:     150m
memory:  128Mi

What you have set should be enough.

So far it restarted twice because of oom killed. So I suspect that the leak is still present. The only feature I use is the config map sync.

Are you seeing any logs like https://github.com/appscode/kubed/issues/304#issuecomment-508024105 ?

The only error log I see from kubed is:
Skipping failed API Groups: unable to retrieve the complete list of server APIs: kubed.appscode.com/v1alpha1: the server is currently unable to handle the request
It restarted 9 times since I updated it to 0.11 and the pattern is consistent. In the picture below, you will see where I updated to 0.11.

Could you increase the memory to say 512MiB ? And then see if the pattern continues.

Sent @tamalsaha.


Update Gradle wrapper JAR and gradlew

via running ./gradlew wrapper --gradle-version 5.0 --distribution-type all

The memory settings were automatically applied when running the ./gradlew wrapper command, which reflects the new defaults: https://docs.gradle.org/5.0/userguide/upgrading_version_4.html#rel5.0:default_memory_settings


Update uber.fx to 1.22.2

What does this PR do?
This PR updates uber.fx to 1.22.2

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Run new-e2e-installer which succeed 5 times in a row.

Closed in favor of https://github.com/DataDog/datadog-agent/pull/28808 (too many conflicts)


remove email info.

GitHub, Twitter, Instagram is enough. Don't include email, we avoid spammers.
better swap gmail to instagram
Safety first.

GitHub, Twitter, Instagram is enough. Don't include email, we avoid spammers.
better swap gmail to instagram
Safety first.

Yes we can do that !! I will add Instagram in place of Gmail


Positively shift denoised data

Currently denoising data results in the data being shifted somewhat into negative values. It would be good if we could offset this by shifting it upwards so that it is all positive. Should be as simple as computing the minimum for the denoised result and subtracting it from the image stack.

Fixed by PR ( https://github.com/nanshe-org/nanshe_workflow/pull/88 ).


Android port possibilities

First, thank for your work as always.
I just want to ask, can there ever be an android port for PixivUtil2?

nope, pixivutil is running on python and has no GUI.
android need java and usually some ui, also why you want to download images in your phone? If you want to browse the site, you can use there's an app for it, or use the mobile site.

I see. Just wanted to keep an offline collection on phone.
Thank you.


CaptchaServlet doesn't send back imageName: "Graph" in start command

If you take a look at
http://demo.visualcaptcha.net/ you can see there is a request to:
http://demo.visualcaptcha.net/start/5?r=i7ba6ivs8gk6s9k9
which returns an imageName":"Tree" - which is needed to fill in the html captcha div client side. Why is the CaptchaServlet not doing this. It seems like visualCaptcha-java is not usable in its current form?
Wim

Can you show me a response that you're getting back from the CaptchaServlet that's not including that value?  All of the same data you show in the demo request you linked is included in the CaptchaFrontEndData object which is serialized as JSON to your front-end.


Fix issue #54

Modified -x option to accept tags as well as ids.

@willthames Any comments?

Apologies @charleswhchan for not responding earlier.
This looks good to me.

Thanks @willthames. No worries.
On a secondary note: Are you planning to cut a release anytime soon? Or do you have other features/enhancements in mind?

@charleswhchan - good point, I'll do that now

2.1.3 has been uploaded to pypi

@willthames: Many thanks! :smile:


New matching case for related devices UI

Issue type:  feature request
There was some discussion on the networktocode slack the other day about how related devices are correlated. I know @jeremystretch mentioned wanting revisit this in a larger fashion but I was wondering if you would be open to adding one more case for matching in the mean time.
This stems from the way I am currently representing switch stacks in netbox. Each member of the stack is named like so: Access-Stack-1F--RE-N1. In this scheme, Access-Stack-1F is the name of the stack and this node's role is RE and it is node 1 in the stack. This is denoted after the delimiter --.
I really like the related devices feature but it obviously cannot match all members of a stack in this case when we factor in multiple roles (RE, LC, etc). Would it be possible to add a case to match anything up to a delimiter of --?

The current approach of looking for devices with similar names was a "good enough" hack for when it was implemented. It should really be replaced with something more robust, but I don't have anything in mind.

Coincidentally I ran across the python built in difflib library in some other work today. It immediately struck me a possible solution to this. Among other things, it can be used to calculate a percentile difference in two string. So for instance, you could say something two device names that are >=90% similar are related. http://stackoverflow.com/questions/17388213/find-the-similarity-percent-between-two-strings

It's one way to go. We'd have to pull in the names from all devices from the database and then do the matching in Python, but that might not be too bad if we limit it to devices assigned to the same site.

difflib has some really neat logic, but after some experimentation it can sometimes return very unexpected results. Probably best to keep things simple and just return a truncated list of devices in the same site with the same role.


Update functions based on Greta pull request

Gre updated the colab, but I wasn't able to see it since the pull request hasn't been merged at this point in time!
(Maybe to avoid having a colab on the repo, we can still implement the missing parts directly in the process.py script)!

addressed in  #2 which integrated #1


Media pipeline server component tests

Summary: Basic Playback Rialto Server component test
Type: Feature
Test Plan: UT, CT, Native Build
Jira: RIALTO-425

https://jenkins.code.rdkcentral.com/job/github-triggered/45215/ started

https://jenkins.code.rdkcentral.com/job/build-component/84022/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/10717/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/16306/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/45219/ started

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/10719/ started

Job for AI/job/AI_BUILDS/job/AI-Trigger-PullRequests started: https://pv-jen015.uk2.scp.bskyb.com/job/AI/job/AI_BUILDS/job/AI-Trigger-PullRequests/30/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/33627/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/33628/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/33629/

Job for AI/job/AI_BUILDS/job/AI-Trigger-PullRequests started: https://pv-jen015.uk2.scp.bskyb.com/job/AI/job/AI_BUILDS/job/AI-Trigger-PullRequests/31/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/github-triggered/10725/ started

https://jenkins.code.rdkcentral.com/job/github-triggered/45226/ started

https://jenkins.code.rdkcentral.com/job/build-component/84046/ started.

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/16328/ started.

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/33662/

Sanity tests for ['RPI4 Hybrid'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/33673/

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/33674/


sentry/kernel-fsync upgraded, breaking akmod builds

I saw these failures...
https://github.com/ublue-os/akmods/actions/runs/8421318593/job/23058006965#step:12:160
I checked the COPR...
https://copr.fedorainfracloud.org/coprs/sentry/kernel-fsync/packages/
Looks like sentry updated fsync-kernel...
from: 6.7.9-204.fsync
to: 6.7.9-207.fsync

Uh... was this a direct to main commit? :grin:
https://github.com/ublue-os/akmods/commit/27bd14986019d9af21a9262bb88e88754bfc0c2d


Unbalanced tuple unpack gets incorrect type

Only x should be an int, but y is being shown as one too. This will technically raise a ValueError ("not enough values to unpack"), but we shouldn't show any completion on y (which would be misleading).
Likely related is this:
(x, *y) = (1,)

Completing on y should tell you that it's a list, but we show nothing.
Found in #334.

Also, this:
(x, y, z) = (1, "")

z is also shown as an int.

Partially fixed by #546:

Probably related to #412.


Incorrect first character on each log line

The first character of each log line is incorrect. It may or may not be a printable character.
===
DockerUI log:
AFeb 18, 2015 11:52:49 PM org.apache.coyote.AbstractProtocol init
5INFO: Initializing ProtocolHandler ["http-bio-8080"]
CFeb 18, 2015 11:52:49 PM org.apache.catalina.startup.Catalina load
)INFO: Initialization processed in 887 ms
PFeb 18, 2015 11:52:49 PM org.apache.catalina.core.StandardService startInternal
INFO: Starting service Catalina
...
Result of "docker logs <container_id>":
Feb 18, 2015 11:52:49 PM org.apache.coyote.AbstractProtocol init
INFO: Initializing ProtocolHandler ["http-bio-8080"]
Feb 18, 2015 11:52:49 PM org.apache.catalina.startup.Catalina load
INFO: Initialization processed in 887 ms
Feb 18, 2015 11:52:49 PM org.apache.catalina.core.StandardService startInternal
INFO: Starting service Catalina
...

Looks like the logs endpoint uses the same 8-byte header that attach does (docs). For DockerUI I think we'd be safe stripping the first 8 bytes of every line before display.


 This PR bumps the version to latest 1.10.3

I guess this was intentional to test that the new script and launcher are still compatible with older sbt versions. @eed3si9n is that correct?

We should rename 'Build and test (6)' to multirepo integration test, but basically it's checking that we can build zinc and lm code against the branch, and that we can build sbt main against zinc and lm HEADs, in part because util will affect them.


Support for when dim(X)>1

Dear senior,
I recently research the paper about gpar and try to run the code which you write.
When dim(X)>1. these codes can be applied to implement?
If there are any examples or documentation how to do this, please send me in that direction (I cannot find any right now).
Best,
GHS

Hey @SGH-188,
Thanks for opening an issue!
The code should already be compatible with multidimensional inputs. For example, the Jura experiment uses two-dimensional inputs. I'm closing the issue for now, but if you encounter any issues, please feel free to reopen it or leave a comment here. :)

Dear wesselb:
Thank you so much for the prompt response! I will study in depth the proposal you sent me.
Best wishes
Yours sincerely
SGH
At 2022-05-03 15:29:23, "Wessel" @.***> wrote:
Hey @SGH-188,
Thanks for opening an issue!
The code should already be compatible with multidimensional inputs. For example, the Jura experiment uses two-dimensional inputs. I'm closing the issue for now, but if you encounter any issues, please feel free to reopen it or leave a comment here. :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Dear wesselb:
Because my data matrix is not positive-definite sometimes, when I use your code of gpar, there is a problem of 'torch.linalg_cholesky: The factorization could not be completed because the input is not positive-definite (the leading minor of order 6 is not positive-definite).' I tried to fix it at the code where the problem occurred, but the predicted results are not satisfactory.
Attachment is the original relevant code where the problem occurred and the code I used to fix it accordingly.
If you need more detailed information, I am ready to send it to you.
I wonder if you have also encountered this problem and how you solved it. Thank you for your guidance and answers.
At 2022-05-03 15:29:23, "Wessel" @.***> wrote:
Hey @SGH-188,
Thanks for opening an issue!
The code should already be compatible with multidimensional inputs. For example, the Jura experiment uses two-dimensional inputs. I'm closing the issue for now, but if you encounter any issues, please feel free to reopen it or leave a comment here. :)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hey @SGH-188!
This typically happens when the covariance matrix of the data is ill conditioned. At the top of your script, could you try to run
import lab as B

B.epsilon = 1e-6

B.epsilon determines the regularisation of the covariance matrices before inversion. 1e-6 is fairly high and should suffice; however, you could try setting it higher if the problem persists.

Dear wesselb:
Thank you for your guidance reply. I initially tried to run the changed code. After changing B.epsilon, the errors as last time disappeared. Next, I will further apply the code you provided. Thank you again for your advice.
Best wishes for you
Yours sincerely
SGH
At 2022-07-28 01:07:32, "Wessel" @.***> wrote:
Hey @SGH-188!
This typically happens when the covariance matrix of the data is ill conditioned. At the top of your script, could you try to run
import lab as B
B.epsilon = 1e-6
B.epsilon determines the regularisation of the covariance matrices before inversion. 1e-6 is fairly high and should suffice; however, you could try setting it higher if the problem persists.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


IIncrementalGenerator vs ISourceGenerator

I maintain a list of Roslyn Source Code Generators at ignatandrei.github.io/RSCG_Examples/v2/docs/List-of-RSCG
However, the source generators have been deprecated in favor of the new incremental generators , that allows better performance
See<EMAIL_ADDRESS>However this means passing from ISourceGenerator to IIncrementalGenerator
Could you do that ?
Thanks
Andrei

Thanks for letting me know!
I'll make plans for that.


Provide an option to collect metrics from all tubes

Hey @davidtannock!
Great job on this exporter, we were testing it out and overall really pleased.
Our only issue is that it currently requires that we provide a list of tubes to collect metrics for which is a bit tricky for us, since we have many different deployments with different tubes being used and in general we want to collect metrics for all of them.
Would you consider an option to collect stats from all tubes?

Hi @dannyk81
Thanks for the feedback.
Yes, that sounds like a useful option. I'll have another branch soon.

Hi @dannyk81
Branch alltubes adds a new option, --beanstalkd.allTubes, for fetching metrics for all tubes. I've updated the readme.
Can you let me know how well that works?
Thanks.

Thanks @davidtannock!
We're going to test it out and update.

Hey @davidtannock, we've deployed this on a couple of nodes and so far seem to work as advertised 👍
Another team needs to validate this, so we may have more feedback later, but overall the allTubes flag seem to work really well.
Thanks again for the prompt response on this.

Thanks @dannyk81 for testing it out.
I'll close the issue for now.


Show init containers

Summary
Show init containers when available for devtron apps as well as helm apps.
Motivation
Init containers should be shown along with other containers.
Proposal
Show init containers(if available) along with other containers at the following places:

App detail > Resource tree: Inside pods

Pod > Logs > Container dropdown

Log Analyser > Container dropdown (Only applicable for helm apps)

NOTE: Init containers will NOT be shown in Pod > Terminal > Container dropdown

To be picked in DT-20


Expose 'languages' so they can be extended outside of the module

I needed support for different scripts (i.e. sr-Latn and sr-Cyrl) in my code. Since those couldn't be added, I (slightly) modified the code so language data can be extended once module is required.
Now I can do the following:
Gta = require('google-translate-api');
Gta.languages['sr-Latn'] = 'Serbian Latin';
Gta.languages['sr-Cyrl'] = 'Serbian Cyrillic';

Coverage decreased (-1.2%) to 89.706% when pulling b16ae1e45fcf2969aba7b7b066d9fddc90190d3c on nwfw:master into 777d7db94f82ec402e7758af1549818c07d55747 on matheuss:master.

Coverage decreased (-1.2%) to 89.706% when pulling b16ae1e45fcf2969aba7b7b066d9fddc90190d3c on nwfw:master into 777d7db94f82ec402e7758af1549818c07d55747 on matheuss:master.

Codecov Report

Merging #31 into master will decrease coverage by 1.2%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #31      +/-   ##
=========================================
- Coverage    90.9%   89.7%   -1.21%
=========================================
  Files           2       2
  Lines          66      68       +2
=========================================
+ Hits           60      61       +1
- Misses          6       7       +1

Impacted Files
Coverage Δ

languages.js
94.11% <100%> (-5.89%)
:arrow_down:

index.js
88.23% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage increased (+0.3%) to 91.176% when pulling 6893e42e963498d3e9bd27ebe6bdcd32a6ed1b26 on nwfw:master into 777d7db94f82ec402e7758af1549818c07d55747 on matheuss:master.


DeadLetterQueueSourceArn missing while receiving message from DLQ

Hey!
I'm currently using your awesome tool to test AWS SQS queues locally.
The goal of my architecture would be to have SQS queues with a DeadLetters Queue.
I'm able to pull messages from the DLQ, However DeadLetterQueueSourceArn is missing from the attributes.
Example with AWS SQS:
{
    "Messages": [
        {
            "MessageId": "************************************",
            "ReceiptHandle": "*****",
            "MD5OfBody": "*****",
            "Body": "example",
            "Attributes": {
                "SenderId": "*********************",
                "ApproximateFirstReceiveTimestamp": "1701176294398",
                "ApproximateReceiveCount": "8",
                "SentTimestamp": "1701176278493",
                "SequenceNumber": "18882245240547823616",
                "MessageDeduplicationId": "************************************",
                "MessageGroupId": "test",
                "DeadLetterQueueSourceArn": "arn:aws:sqs:us-west-2:*****:example.fifo"
            }
        }
    ]
}

Example with ElasticMQ:
{
    "Messages": [
        {
            "MessageId": "************************************",
            "ReceiptHandle": "************************************#************************************",
            "MD5OfBody": "e4b3b7558956e56fcb6b5397bdc6523d",
            "Body": "example",
            "Attributes": {
                "ApproximateFirstReceiveTimestamp": "1701176081533",
                "SentTimestamp": "1701175918154",
                "MessageDeduplicationId": "************************************",
                "SenderId": "*********",
                "ApproximateReceiveCount": "6",
                "MessageGroupId": "test",
                "SequenceNumber": "0"
            }
        }
    ]
}

It would be great if this attribute could be added when the message is moved to the DLQ.

Fixed in v1.5.8
Thanks for reporting the issue!


Fixes plugin configs

What does this PR do?

Fixes bug when parsing plugin configurations to a struct. This is still not the ideal way of doing it, but we need more time to research.

Shouldn't the tests be covering/catching this?

@rafaeljesus Yes it should, if I have time I will do it, otherwise I will need your help to create the cases.

👍

👍


Init add domewatch app

Description
This app displays realtime updates from the US House Floor, including the floor status, the next votes, and a voting scoreboard.
The API data being used is supplied by US House Democratic Leadership for use in the Dome Watch mobile app. The mobile app is frequently used by US House Members, their staff, and the press for real time updates of voting activity. This tidbyt app is meant to be a simple version similar to the mobile app.
The app consists of two displays:

A dome gif along with the current floor activity (debate, morning hour, quorum calls, joint meetings, recess, etc). This screen also shows a marquee of the next expected voting times, when available.
A scoreboard breakdown of a particular vote when live voting occurs. This includes tallies for Democrats, Republicans, Present, No Votes, totals, and the roll call being voted on.

The first user of this new app will be the Office of Congressman Maxwell Alejandro Frost, who will display it proudly in their office.
Copilot

🤖 Generated by Copilot at 55d7501
Summary
🏛🗳📜

This pull request adds a new applet called Dome Watch, which displays the US House floor activity and live vote counts on the Tidbyt device. The applet consists of two files: dome_watch.star, which contains the Starlark code for fetching and rendering the data from the Dome Watch API, and manifest.yaml, which provides the applet metadata and configuration.

Dome Watch applet
Shows US House floor action
Winter of politics

Walkthrough

Create and configure the Dome Watch applet to show US House floor activity and live vote counts (link, link)

Define functions to get and parse the floor data from the Dome Watch API using http and json modules (link)
Define functions to render the text and icons for the floor status, speaker, and vote counts using render and base64 modules (link)
Define functions to handle different scenarios such as voting, adjourned, or not in session using time and math modules (link)
Use the cache module to store and update the floor data periodically (link)
Add the manifest file manifest.yaml to provide the applet metadata and configuration (link)

I have read the CLA Document and I hereby sign the CLA


G4f provider/AI issue

Major issue with the provider/AI. AI assisted report writing is currently DOWN and code is unworkable. Will fix this ASAP --> Top priority!

Fixed issue temporarily by changing provider yet again to gpt3.5turbo (looks like the devs finally fixed it), which does not have too many issues (as of now) and does not require authentication.


[Support] Question from PCBWay on PCB manufacturing

Hi again! Hopefully this is an easy one :)
We've ordered programmer and badge boards from PCBWay. Programmer boards went off without a hitch. Badge boards had a quality assurance question:

Ref attached screenshot, is it correctly designed as no hole is there for the pointed out pad ?

Attached:

After we give them an answer, I'll ask them for suggestion on how to annotate to future clarity.

In the meantime, I forwarded @bhimasta's image from the other thread, so hopefully that will answer question for them:
https://github.com/HumanDynamics/openbadge/issues/115#issue-382709017

Considering we ordered to PCBWay using the same source of design, they didn't ask anything about this issue in my case. Is it weird, isn't it?

Patric - that is correct, no hole there. This large pad is part of the alternative battery holder we used to use in the past. Now the new battery holder is just soldered on top of it.
I kept the old design there as a backup since every change I make to the design triggers an order of another batch of prototypes and tests. It's quite common to have additional "stuff" in the hardware design that isn't used.

Thanks Oren!


PHP 7 PHP-PROTOBUF INSTALLATION

Hello, I'm trying to install php-protobuf for php 7.0. I have followed this guide to install both protobuf and cruve: link. I have installed cruve, but couldn't install protobuf. Instead I get this when I try to check which modules are loaded (php -m):
^[[6~PHP Warning:  PHP Startup: Unable to load dynamic library '/usr/lib/php/20151012/protobuf.so' - /usr/lib/php/20151012/protobuf.so: cannot open shared object file: No such file or directory in Unknown on line 0
Can anyone help me?

Closing Issue because I didn't search enough to find that it was already answered. To all those who are having same problem and couldn't find it, you need to follow:
git clone https://github.com/serggp/php-protobuf cd php-protobuf git checkout php7 phpize ./configure sudo make sudo make install
Then php -i | grep .ini to to find your php.ini file. It is stored in Loaded Configuration File => /etc/php/7.0/cli/php.ini It might be different that yours or not.
nano /etc/php/7.0/cli/php.ini then CTRL + - then 1916 to get to the last part of the .ini file. It might have few more lines. Navigate to the end and then add extension=protobuf.so and then save it. Use php -m to list all the installed extensions and try to search for protobuf. Cheers!


add stop/start NBDSlaveSync command to tlog

This command replaces the waitNBDSlaveSync command, as we never want to stop all slave sync at once, instead we want to stop 1 or multiple targeted slave servers, which the NBDServer is now (temporarily) writing and reading directly from, while waiting on a follow-up action from the 0-Orchestrator. Later we also want to be able to (re)start the syncing to 1 or multiple target slave servers.
This will change the TlogClientMessage capnp sturct into:
# message to send from client to server
struct TlogClientMessage {
	union {
		block @0 :TlogBlock;        # block message
		forceFlushAtSeq @1 :UInt64; # force flush at seq message
		stopNBDSlaveSync @2 :List<UInt64>;  # Stop NBD Slave Sync message
		startNBDSlaveSync @3 :List<UInt64>;  # Start NBD Slave Sync message
	}
}

among all the other required code changes.

This issue is cancelled, as the new landscape (using the new tlog code in the seperate tlog repo), will make our requirements for this a lot different.


Cannot find the script that running recurrent layer benchmarks on Intel hardware.

For recurrent layer benchmarks, I have seen that you have updated its' results for Intel hardware. But I cannot find the scripts to run this workload on Intel hardware. Could you please help to update it.
Thanks.

The results/DeepBench_IA_KNL7250.xlsx sheet does not yet include the performance numbers for the recurrent layers (both vanilla RNNs and LSTMs). MKL-DNN will be adding support for recurrent layers shortly - we will update the numbers following that.


optimize code and user experience

Signed-off-by: zhang chen<EMAIL_ADDRESS>

Codecov Report

Merging #596 (81b1821) into training (346726b) will decrease coverage by 0.16%.
The diff coverage is 85.54%.

@@             Coverage Diff              @@
##           training     #596      +/-   ##
============================================
- Coverage     59.96%   59.79%   -0.17%
============================================
  Files           152      151       -1
  Lines          6107     5997     -110
  Branches        883      888       +5
============================================
- Hits           3662     3586      -76
+ Misses         2209     2171      -38
- Partials        236      240       +4

Impacted Files
Coverage Δ

towhee/trainer/callback.py
66.66% <ø> (-0.58%)
:arrow_down:

towhee/trainer/optimization/optimization.py
32.30% <ø> (-2.03%)
:arrow_down:

towhee/trainer/modelcard.py
51.06% <33.33%> (-1.02%)
:arrow_down:

towhee/trainer/trainer.py
66.28% <79.12%> (-0.67%)
:arrow_down:

towhee/trainer/training_config.py
88.63% <95.77%> (-1.48%)
:arrow_down:

towhee/utils/log.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

/lgtm

/approve


Send digitally signed emails with email certificates

Is it possible to send digitally signed email using email certificates. For example Comodo: https://www.comodo.com/home/email-security/free-email-certificate.php
In Mozilla Thunderbird I can import certificates to digitally sign my emails as well as use that certificates to encrypt my messages. I know about OpenKeyChain support in K9 but I'm only interested in sending digitally signed emails using Comodo certificates just the way I can easily do it on Thunderbird?
Is it possible?

See #1003


[YAML] Refactor: Rename MEXCOrder -> MexcOrder, ERC721Factory

Why: follow naming convention

Fixed in commit https://github.com/oceanprotocol/pdr-backend/commit/9f33dc21b97bd13885aecb6b6a29409d63f8c576

The first commit caused a bug.
This commit fixes the bug: https://github.com/oceanprotocol/pdr-backend/commit/8fe215d0da1206ee2c1d8e302bd39d45acc441ed


⚠ Bofrost.at has degraded performance

In c9806fd, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1345 ms

Resolved: Bofrost.at performance has improved in bf3cc52 after 27 minutes.


[Question] How to add a proxy to backend api?

Hi @Kikobeats, how can I add  the proxy to my backend API? I have tested with that in web.config, without no results:
devServer: {
proxy: {
'/api': 'http://localhost:8080',
changeOrigin: true,
hot: true,
inline: true
}
},
Thanks.

Ok, I founded it in bin folder and server.js ;)


Use app's HTTP settings, handle feature settings race condition, CI/build updates, lazy server discovery

References

supersedes #881
fixes #880
progress on #851

Code changes

/

[x] resolve and deduplicate yarn.lock: get "one true version" of typescript, prettier, webpack
[x] add sourcemaps (we can take these out, but... would be really nice for downstream debugging)

.github

[x] bump more CI action versions to squash warnings

@jupyter-lsp/jupyterlab-lsp

[x] initialize LanguageServerManager with app.serviceManager.settingsis being used
[x] use these settings for doing API requests and WebSockets
[x] handle a race condition for very fast settings loading (no http request) for plugins with a PromiseDelegate
[x] update @jupyterlab/* dependencies to match python-level dependency

@jupyter-lsp/lsp-ws-connection

[x] use webpack 5 for lsp-ws-connection
[x] normalize void for handling of promises
[x] bumps version

jupyter_lsp

[x] use importlib(_|.)metadata for entrypoints
[x] bumps version
[x] defers language server spec discovery until the event loop is ready

User-facing changes

hopefully few visible changes

Backwards-incompatible changes

python

downstreams which interact with jupyter_lsp.LanguageServerManager directly may need to await more things

Chores

[x] linted
[x] tested
[x] documented
[x] changelog entry

Ha, green!
I can roll back the server lazy loading changes if desired.

Cool, glad to have this one in.
I'll have another look at working the CI assets into the downstream on jupyterlite-lsp...


Make the @yamada-ui/radio component headless

script: make-component-headless
package: @yamada-ui/radio
Description
Currently, the @yamada-ui/radio component has minimal styles set within the component's source code.
We want to make this style transferable to the theme and make the component headless, enabling more flexible UI customization.
Problem Statement/Justification
By making the @yamada-ui/radio component headless, we can achieve more flexible styling tailored to the project.
Proposed Solution or API
The styles within the component are mostly defined by css: CSSUIObject and set to a prop named __css. We will transfer these styles to the packages/theme/src/components.
The code below is just an example and not the actual code.
packages/components/input/src/input-group.tsx
const css: CSSUIObject = {
- width: "100%",
- display: "flex",
- position: "relative",
  ...styles.container,
}

packages/theme/src/components/input.ts
baseStyle: {
  container: {
+   width: "100%",
+   display: "flex",
+   position: "relative",
  },
  field: {
    width: "100%",
    minWidth: 0,
    outline: 0,
    position: "relative",
    appearance: "none",
    transitionProperty: "common",
    transitionDuration: "normal",
    _disabled: {
      opacity: 0.4,
      cursor: "not-allowed",
    },
  },
  element: {
    color: ["blackAlpha.600", "whiteAlpha.700"],
  },
},

[!TIP]
In cases where the styles within the component are dynamically changed, there may be instances where it is not possible to transfer the styles to the theme.
In such cases, it needs to be discussed in this issue. Please feel free to send comments to this issue.

Additional Information
The @yamada-ui/radio component exists in packages/components/radio/src. You need to check all the components in this folder and if they contain styles internally, you need to modify them.
This change could have a significant impact on projects using Yamada UI, so we will work on it within the scope of v2.0.
Please create the branch for working on this issue from the remote branch v2.0, and direct your pull requests to the v2.0 branch as well.
Please select major for the changeset and create a pull request.

Hi @108yen , can I work on this one?

Fixed #1936


Doc links to specific props not scrolling down the page

A link that references a specific prop on a component page should scroll down to that section when you open it, but instead the link opens, goes to the specific prop section and then takes you back to the top of the page.
Example: https://v2.grommet.io/list#onOrder

This issue is occurring again


Gatweay Port Hangs

Describe the bug
The docker image runs well, the terminal logs no errors as far as I have seen (I will attach a log just in case). But when I do a "curl -X GET "localhost:$port", the command hangs. I am sure I am using the right port as when I mistake the live for the paper port (or vice versa), the command returns an error.
Note: I am using a custom image, but it is based on the latest stable with the only difference being that I installed curl for healthcheck purposes. I may be missing something really simple, perhaps something firewall related? I haven't touched firewall settings. But I just can't figure it out. Any help would be appreciated.
docker-compose.yaml:

.env:

(username and password are filled in when running, the rest are as is)
I build the project with 'docker compose up'
Expected
The second package is a python app that calls the TWS API. I need the ib-gateway to make the api available so the healthcheck passes and the app connects to the API.
Container logs
logs.txt## Versions
Please complete the following information:

OS: macOS 14
Image Tag: stable

Additional context
Add any other context about the problem here.
What have you tried and failed.
I have tried stable/latest branches
I have tried accessing the port from outside of the container, same result.
I have tried accessing the port from inside of the container, same result.
I have tried using a paper account instead of a live one (ports adjusted), same result.
I have tried using <IP_ADDRESS>:4001:4003 instead of 4001:4003.

Hi,
not shure what's your expectation, why would "curl -X GET "localhost:$port/v1/api/one/user" work, this is not a RESTfull API.
I just did a quick test on my running TWS instance and ports are working.

if you need a healthcheck(which just is telling you that the port is open, but not that is actually working) you can use that
this is how it would look like on ib-gateway with the port down (ibgateway is not running)

something like this to have it working as a docker healthcheck
HEALTHCHECK --interval=30m --timeout=15s --start-period=10s \
  CMD timeout 1 bash -c '</dev/tcp/<IP_ADDRESS>/4001 && echo "TWS API up" || echo "TWS API down"' || echo "Connection timeout"

take into account that any disconnection to IBKR server farm will not be reported by this health check. I use ib_async and disconnectedEvent to keep TWS api connectivity.
hope this helps

Hm, yeah thanks for the healthcheck, I could see it being useful. And thanks for the answer, I figured it out. I wasn't managing my connection IDs correctly.


Improve project marketing

When releasing 0.2, we should announce the project existence in certain medium, such as Reddit and Rust forums.
The README should gather bullet points of the goals of this crate.

[x] announce project release
[x] write about project goals in README

https://www.reddit.com/r/rust/comments/8nze3p/announcing_a2_a_batteriesincluded_apple_push/


lr rate

The default lr rate is 5e-4 rather than 1e-3. So which lr rate should I use to reproduce your results?

Lr is linearly scaled based on batch size. Reference lr rate is 0.0005 with batch size of 512.


Handle 'X' for a ColumnDataType.

Hi! Thank you so much for your work on this crate - it's been incredibly useful for me.
I have some fits files that store data using the BIT type in a BINTABLE extension.  With a one-line change, I was able to unpack the contents into a Vec<u32> just fine.  I don't know the ramifications of this change, so please feel free to consult me on this.
An example file is shared below:
https://drive.google.com/file/d/1JTYEQD2f42HZMb7D3wskSx6Wkto3HsVO/view?usp=sharing
Thanks again for your time and effort!

Hi, thanks for your kind words and contribution.
Before I merge this, I'd like to check both reading and writing of this data type. I'd also like some tests to ensure that this does the right thing. I'm happy to add this functionality (based on this starting point), but if you'd like to add some test cases or a more complete implementation, I'd welcome that!

Thanks. And I'm glad you're cautious about adding new functionality.  I'll see if I can add some tests over the next few days.

Hmmm this looks trickier than I had first hoped.  I think if I were to contribute to your library properly, I would need your guidance to make sure things fit in.
Reading the document that comes along with the cfitsio source code, two interesting parts pop out:
One feature of the CFITSIO routines is that they can operate on a `X'
(bit) column in a binary table as though it were a `B' (byte) column.
For example a `11X' data type column can be interpreted the same as a
`2B' column (i.e., 2 unsigned 8-bit bytes).  In some instances, it can
be more efficient to read and write whole bytes at a time, rather than
reading or writing each individual bit.

and
\begin{description}
\item[4 ] Write bit values into a binary byte ('B') or bit ('X') table column (in
    the CDU).  Larray is an array of characters corresponding to the
    sequence of bits to be written.  If an element of larray is true
    (not equal to zero) then the corresponding bit in the FITS table is
    set to 1, otherwise the bit is set to 0.  The 'X' column in a FITS
    table is always padded out to a multiple of 8 bits where the bit
    array starts with the most significant bit of the byte and works
    down towards the 1's bit.  For example, a '4X' array, with the
    first bit = 1 and the remaining 3 bits = 0 is equivalent to the 8-bit
    unsigned byte decimal value of 128  ('1000 0000B').  In the case of
    'X' columns, CFITSIO can write to all 8 bits of each byte whether
    they are formally valid or not.  Thus if the column is defined as
    '4X', and one calls ffpclx with firstbit=1 and nbits=8, then all
    8 bits will be written into the first byte (as opposed to writing
    the first 4 bits into the first row and then the next 4 bits into
    the next row), even though the last 4 bits of each byte are formally
    not defined and should all be set = 0.  It should also be noted that
    it is more efficient to write 'X' columns an entire byte at a time,
    instead of bit by bit.  Any of the CFITSIO routines that write to
    columns (e.g. fits\_write\_col\_byt) may be used for this purpose.
    These routines will interpret 'X' columns as though they were 'B'
    columns (e.g.,  '1X' through '8X' is equivalent
   to '1B', and '9X' through '16X' is equivalent to '2B').  \label{ffpclx}
\end{description}

I personally am not at all interested in writing out individual bits, and probably not writing anything using the 'X' data type, but maybe you would want to see that done rather than something half-baked.  Reading in 'X' data type columns as bytes suits me fine, and I can debate with myself later whether I want a collection of bools or ints.  But, I don't know how the number leading the data type would be correctly handled in rust-fitsio.  With that information, maybe I can make more progress.  It may also be worth introducing a new enum member to ColumnDataType.

Thanks for this in-depth investigation! I agree it is more nuanced than that. I have a branch that is currently handling this situation with an additional member to ColumnDataType. As you say, reading your example file (with a 32X column type) into a Vec<u32> works but the bits are treated as a single u32. Take a look at my branch and how it compares with your understanding of the situation [compare].
I got as far as an implementation, but I was struggling to test it. I feel a proper implementation would allow the user to read e.g. a 16X column into Vec<Vec<bool>>, but we possibly want a more intuitive byte array type.
The (possibly suitable) alternative is to:

add the X data type (including updating ColumnDataType)
add some documentation around this data type, stating that it can  be read into u8, u16, u32 or u64 i.e. disallow non-aligned number of sub-columns, but beyond that must be handled by the user (e.g. they can easily unpack a u32 into individual bits using bitmasking/shifting)
possibly think about supporting NX where N < 64

Thanks for coming with me on this journey, I'm keen to get input from users of this crate.

Well, thank you for putting in additional effort. I'll see what I can do over the next few days.

So I've been playing with vanilla C cfitsio, and maybe I'm not understanding it correctly, but the results I'm getting are confusing.
Below is some code I'm using to test the mwaf file I provided earlier:
    /* unsigned char *buffer = calloc(sizeof(unsigned char), naxis2); */
    /* uint8_t *buffer = calloc(sizeof(uint8_t), naxis2); */
    /* uint16_t *buffer = calloc(sizeof(uint16_t), naxis2); */
    uint32_t *buffer = calloc(sizeof(uint32_t), naxis2);

    if (fits_read_col(fptr, TBIT, 1, 1, 1, naxis2, NULL, buffer, NULL, &status)) {
        fits_get_errstatus(status, errorMessage);
        printf("%s\n", errorMessage);
        return EXIT_FAILURE;
    }

    for (int i = 0; i < 200; i++) {
        printf("%4d: ", 4*i);
        for (int j = 0; j < 4; j++) {
            printf("%5d", buffer[4*i + j]);
        }
        printf("\n");
    }

The hexadecimal contents of the first few bytes is: 0000 0000 c000 8003 c000 8003 c000 8003
unsigned char and uint8_t work the same, and unpack the bits into bytes:
   0:     0    0    0    0
   4:     0    0    0    0
   8:     0    0    0    0
  12:     0    0    0    0
  16:     0    0    0    0
  20:     0    0    0    0
  24:     0    0    0    0
  28:     0    0    0    0
  32:     1    1    0    0
  36:     0    0    0    0
  40:     0    0    0    0
  44:     0    0    0    0
  48:     1    0    0    0
  52:     0    0    0    0
  56:     0    0    0    0
  60:     0    0    1    1

However, using larger integers gives confusing results.  uint32_t:
   0:     0    0    0    0
   4:     0    0    0    0
   8:   257    0    0    0
  12:     1    0    016842752
  16:   257    0    0    0
  20:     1    0    016842752
  24:   257    0    0    0
  28:     1    0    016842752
  32:   257    0    0    0
  36:     1    0    016842752
  40:   257    0    0    0
  44:     1    0    016842752
  48:   257    0    0    0
  52:     1    0    016842752
  56:   257    0    0    0
  60:     1    0    016842752

So it appears that using larger integers in C doesn't work correctly when using the TBIT type.
Again with uint32_t, but using TINT as an argument to fits_read_col:
   0:     0    0    0    0
   4:   192    0  128    3
   8:   192    0  128    3
  12:   192    0  128    3
  16:   192    0  128    3
  20:   192    0  128    3
  24:   192    0  128    3
  28:   192    0  128    3
  32:   192    0  128    3
  36:   192    0  128    3
  40:   192    0  128  131
  44:   192    0  128    3
  48:   192    0  128    3
  52:   192    0  128    3
  56:   192    0  128    3
  60:   192    0  128    3

... which is more like what we would expect, but apparently not as densely packed as it should be.
So I guess matching cfitsio's behaviour is a little absurd.  The C code that I'm in the process of re-writing currently unpacks the bits into unsigned chars and carries on its merry way, and that seems excessive.
I guess the best course of action would be to make the behaviour of dealing with the X data type clear in the documentation (or even just a couple of examples in the main docs).  I personally would hate to inflate the data by 8x just get access to it (and would be happy to handle bit shifts etc. myself), so I think unpacking the bytes of the fits table into regular bytes of some description makes sense.  As for writing out bits... maybe it just makes sense to provide bytes, and those get written as bytes with an X data type?
I haven't done anything in rust-fitsio except for writing a trivial test.   I'm wondering if you've had any thoughts on what I said above.  I'm also wondering if you didn't see my post, as I accidentally closed the PR with some force-push witchery :slightly_smiling_face:

Hi @cjordan, sorry I did see your post above. I apologise for not getting back to you sooner, this seems like a solid investigation, thank you! I will try to find some time to absorb what you've suggested. I agree that we don't want to read too much just to bit shift and throw it away - it would be nice to do the right thing. At least we have the ability to only implement the ReadsCol trait for the correct data type so we can specify that the function can only return e.g. u8 or even something like bit-vec. I will try to find some time.
Is this issue blocking you in any way? If it is, you can use the fitsio-sys crate with the (unsafe) fptr from fitsio and treat the code as if you were using cfitsio. Unfortunately there is no documentation (as it's autogenerated), but check out the fitsio source code for usage, and the cfitsio docs of course.

Thanks @mindriot101, and I apologise for making you react.
I'm not exactly in a rush to see this work finished, but I would like to see it tied off within say 4 weeks.  Obviously if that doesn't suit you, I can have a crack in my own branch and develop against that -- no worries.
And I appreciate that you're (probably) very busy.  The world isn't exactly helping right now, either...

Hi again. I'm coming up against this again, and now I'm in two minds on how to proceed.  As a quick path forward, I've simply converted the Vec<u32> into Vec<u8>.  I have no idea if there's some combination of parameters that would mean that the values could exceed a u8 (i.e. 255), but this solution works, and is already consistent with cfitsio.  In addition, because the ReadsCol trait doesn't work for types smaller than u32, we don't get that seemingly crazy behaviour I found before.  So, with your current branch, we could just leave this as-is, even just for now.  I would really appreciate it if you could merge the branch into master and make a new release, though.
My second mind is that, yeah, it would be really nice to only use u8 if it's really all that's needed.  But, that will probably cause some pain in augmenting the ReadsCol trait when adding error checks on everything else that uses the trait.  And, of course, the behaviour is a departure from what cfitsio does.
I guess if you wanted maximum efficiency, the discussion we've had here should go into the documentation, as well as a good example.  I could have a crack at getting that going, as we will soon be using your library as key dependency to churn petabytes in our observatory.  But let me know what you think - this isn't exactly low-hanging fruit, and maybe you're happy with what you've already done.

I'm happy to merge in what you need for now. I apologise again for the delay. Did you mean this branch in particular? You've force pushed and removed the changes, so there is nothing to merge :(
As it sounds like you will use this crate far more than I do (I am unfortunately no longer working doing astronomy), I am happy to give you commit rights to this repo, if you wouldn't mind helping me with maintenance and new features? I appreciate the work you have put in, and the persistance you've shown.
After I give you commit rights, feel free to make any changes you need to support your workflow. Once you have something workable, then maybe we can discuss what we would consider the proper implementation.

No worries on the delay.  I was meaning your branch "fix/bit-type" - I'll merge it in now.  Anyway, yeah, I would be happy to help maintain the repo.  I'll try to raise any new features or issues as PRs first, but feel free to say that you don't have time to look or check.
Sorry to hear about you no longer working in astronomy - it is a reality many of us face.  In a sense, the software I'm currently writing acts as a CV for the next chapter in my own path...

Thanks for your help, I've published the crate to crates.io so you can pin the version. Thank you for using and supporting this project!


Storybook : Product Details Component

Description
Storybook : Product Details Components Solition
Fixes # (issue)
Storybook : Product Details Component
How to test?
tested by npm run dev and npm run storybook
Checklist

[X] I have performed a self-review of my own code
[X] I have followed the name conventions for CSS Classnames and filenames, Components names and filenames, Style filenames, if you are in doubt check the the project README.MD and here https://github.com/HackYourFuture-CPH/curriculum/blob/master/review/review-checklist.md
[X] I have commented my code, particularly in hard-to-understand areas, if you code was simple enough mark the box anyway
[X] I have made corresponding changes to the documentation, if you code was simple enough mark the box anyway
[X] This PR is ready to be merged and not breaking any other functionality

hi Semih, I reviewed this PR. it is working as it should be.

Thank you @varsha84


Option for ordering by new arrivals

Look at albums publish date and order results. Though should be noted that only cached albums can be returned of course so albums would need to be retrieved. Possibly could be done by seeding & searching for most sold but when recaching searches for new arrivals.

Possibly add support for sorting by popularity. Use the order an album gets when it was retrieved by a tag as a measure of popularity (as albums are returned in order of popularity for that tag).


change state cache to channel

This PR is a following update for crane agent with these changes:

changed state collector store from a shared map to channel
removed 10 second interval in analyzer, it is triggered by state channel instead
some refactors like rename and method extraction.

/LGTM


[config] which-key descriptions for prefixes

SPC w m did not have a prefix.
Neither did SPC h d and SPC h r
I used a variable to define the help key to avoid hard-coding to h. I don't know if it's the smartest choice.
(I also don't really know why the help-map is not mapped with map! but I left it like that)

@hlissner I rebased onto your latest changes. I still think there ought to be a which-key description to the window-map maximization prefix

I still think there ought to be a which-key description to the window-map maximization prefix

Unfortunately, that will not work. Which-key does not support labels on non-major-mode keymaps (like evil-window-map). This change will have the unintended effect of labeling all m keys as "Maximize".
The only way around this is to do as we did with SPC h {d,r} and use absolute key sequences, e.g. C-w m, <leader> w m and <alt-leader> w m.

ah I see, thanks for the info. I'll see about that

I added a working config for maximize in +evil-bindings.el and changed the regexp for the help sub-prefixes, because C-h doesn't seem to correspond to <help>, or at least it doesn't work when I press C-h

I hate that this is necessary (no fault of your own; I which which-key was more versatile), but I think this is good enough. Thanks again for your help, and for putting up with me!


FAQ Section to be updated

change content as per the website information referring github

Hey, am interested in taking on this issue if available.


Lifetime options

I haven't tested this, so I don't know if it'll work or not.
This change replaces "expiresecs" with "maxexpiresecs" and should allow a "life=" parameter which sets the lifetime in seconds.

Did you try it and confirm that it is working?

I haven't set it up yet, so unfortunately no.

I'll try it later.

Don't change indentation and don't mix indentation.
Reformat with Tabs, like the original source.

Now invalid due to conflicts and neglect


Coverage chart in Variant detail clips at half exon

I'm not seeing this anymore, and it is also mentioned in a different issue.


ClassNotFoundException: org.postgresql.Driver、mysql。。。

Search before asking

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

What happened
执行工作流任务报ClassNotFoundException: org.postgresql.Driver
What you expected to happen
如何修复
How to reproduce
运行apache/dolphinscheduler:2.0.0-alpha镜像
Anything else
No response
Version
2.0.0-alpha
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Please put the driver package under the path lib/plugin/task/sql
This problem has been fixed in version 2.0.1

ok, thanks!


Is it possible to skip fulltext catalogs?

For a while I used backup from the azure UI to create a bacpac, but since the addition of fulltext search I cant restore those on our dev machines (with localdb). Can this script do a partial backup leaving out the fulltext features?

No, there is no option for that.

@Jogai a few minutes ago I ran into the same problem (trying to import an azure bacpac backup locally). I found this issue while looking for a solution.
You can find my solution here:
https://dba.stackexchange.com/a/260714/33001
And here are the steps (copy/paste) just in case:

Rename your backup to zip
extract model.xml and Origin.xml
open model.xml with a text editor and look for "fulltext"
Delete all fulltext elements in the xml
set fulltext to false in the header
Calculate the SHA256 hash of the edited xml (I used hashtab for that)
Fix the hash/checksum in origin.xml
Add the updated files back to the zip
Rename back to bacpac and import
"Worked on my machine" - I hope it helps


fix: 🧑💻 add tmxToast to doubles unpairing failure

When unpairing alternate doubles pairings, and a pair has already been to a draw, it results in nothing being displayed to the user.
Following on from patterns seen in the codebase, when using callbacks to the mutationRequest and encountering an error, it's up to you to display a toast.
Hope this is the right way to do it!
This was a real issue I saw Conrad doing and it seeming very broken.

James,
Thanks for this!
I approved and merged yesterday but have not yet deployed a new build. May
get around to that Thursday
On Tue, Aug 27, 2024 at 7:57 AM James Hartt @.***>
wrote:

@jameshartt https://github.com/jameshartt requested your review on: #459
https://github.com/CourtHive/TMX/pull/459 fix: 🧑💻 add tmxToast to
doubles unpairing failure.
—
Reply to this email directly, view it on GitHub
https://github.com/CourtHive/TMX/pull/459#event-14027381988, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AIWYICZIILBPRBBYAWB6OLDZTRSQ7AVCNFSM6AAAAABNF63EHGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGAZDOMZYGE4TQOA
.
You are receiving this because your review was requested.Message ID:
@.***>


Fix the CI error

https://github.com/casbin-lua/kong-authz/runs/7718648324?check_suite_focus=true

Description
kong #7317
This issue mentioned this issue, the reasons for summarizing：
In Postgres version 13, the default value is scram-sha-256.
But Kong does not supported scram-sha-256.
Two solutions

The current better solution is to use postgres:9.6.
You need to modify your password_encryption setting in postgresql.conf file.You need to motify it from scram-sha-256 to md5.And restart your Postgres service.

In CI, I think we can use postgres:9.6.

I think lowering the postgres version in the CI is a better alternative.

Agreed


TaskList

The file '.editorconfig' is irrelevant here
If the user exit the session - the list is deleted, please fix
If the user sends an empty string, the list will be deleted, please fix
You need to format the document before creating a pull request, search google how to do this and fix :)
Add some description in the pr (pull request)

Add a session for editing a tasks list, add a delete and view the list


Add License management to Hunter

Enable this for Boost.
[PACKAGE]_LICENSE holds location of installed license.
Only enabled for url_sha1_boost scheme. Can be enabled for others later.
Partially solves #162 for Boost

Yes. I am making these changes.
I do feel however, that all schemes should support this. This is an important part of redistributing a package, but not important if a package is only consumed internally somewhere.

I do feel however, that all schemes should support this

Sure. It's just a matter of adding:
+    COMMAND # Copy license files
+    "@CMAKE_COMMAND@"
+    "-Dsrcdir=@HUNTER_PACKAGE_SOURCE_DIR@"
+    "-Ddstfile=@HUNTER_PACKAGE_LICENSE_FILE@"
+    -P
+    "<EMAIL_ADDRESS>

Added commands to all non-autotools schemes to add copy license step.
It was strange to try to do this in the autotools dependent schemes, because this step would basically need to add arguments to the hunter_autotools_project call passing in srcdir and dstfile.
Leaving that as a separate enhancement?

Leaving that as a separate enhancement?

Yes

@ruslo Has this passed tests?

@akalsi87 Yep, looks good. Just for your information I'm not notified about new commits in pull request so I haven't seen your last updates

Minor cleanup: https://github.com/ruslo/hunter/commit/d1576715578001f33d3a22fa48497dcfd436fc97

@akalsi87 New wiki package added: https://github.com/ruslo/hunter/wiki/Using-license-file-of-package. Usage section of home page updated: https://github.com/ruslo/hunter/wiki#usage. Feel free to review/add something. Thanks!

Release: https://github.com/ruslo/hunter/releases/tag/v0.12.16

Thanks @ruslo !