Tidy EditCommand related classes

v1
@Zhiyuan-Amos submitted v1 for review.

[v1 1/3] Update EditCommandParser to use EditPersonDescriptorBuilder
[v1 2/3] Merge EditCommandIntegrationTest with EditCommandTest
[v1 3/3] Tidy up existing EditCommandTest

(:books: Archive)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/445/1/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

v2
@Zhiyuan-Amos submitted v2 for review.

[v2 1/2] Merge EditCommandIntegrationTest with EditCommandTest
[v2 2/2] Tidy up existing EditCommandTest

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v1 and v2)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/445/2/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you want to fetch this PR to.

I have no idea what [2/2] is doing.

I can't tell what obvious single change is being made inside the diff. Commit message doesn't give much clue either.

v3
@Zhiyuan-Amos submitted v3 for review.

[v3 1/2] Merge EditCommandIntegrationTest with EditCommandTest
[v3 2/2] Improve EditCommandTest

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v2 and v3)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/445/3/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

v4
@Zhiyuan-Amos submitted v4 for review.

[v4 1/5] Merge EditCommandIntegrationTest with EditCommandTest
[v4 2/5] EditCommandTest: construct EditCommand directly
[v4 3/5] Replaces this use of PersonBuilder with the more compact version
[v4 4/5] Fix header comments: Remove unnecessary line breaks
[v4 5/5] Replaces all Command command with EditCommand editCommand

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v3 and v4)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/445/4/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

@pyokagan thanks for your review!

Thanks, the commits are much better now.

v5
@Zhiyuan-Amos submitted v5 for review.

[v5 1/5] Merge EditCommandIntegrationTest with EditCommandTest
[v5 2/5] EditCommandTest: construct EditCommand directly
[v5 3/5] Replaces this use of PersonBuilder with the more compact version
[v5 4/5] Fix header comments: Remove unnecessary line breaks
[v5 5/5] Replace all `Command command` with `EditCommand editCommand`

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v4 and v5)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/445/5/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

Did a rebase to master.

@Zhiyuan-Amos Could you also prefix all the commit subjects with EditCommandTest:  as required? Thanks.

One of the commit message has w/ because the word with causes the message to exceed 72 characters limit

v6
@Zhiyuan-Amos submitted v6 for review.

[v6 1/5] EditCommandTest: Merge with EditCommandIntegrationTest
[v6 2/5] EditCommandTest: Construct EditCommand directly
[v6 3/5] EditCommandTest: Update PersonBuilder usage with more compact version
[v6 4/5] EditCommandTest: Remove unnecessary line breaks in header comments
[v6 5/5] EditCommandTest: Replace `Command command` w/ `EditCommand editCommand`

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v5 and v6)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/445/6/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

ah yes. thanks Eugene! :>

@yamgent i have updated the small comment error that Eugene commented. Last call? :>

v7
@Zhiyuan-Amos submitted v7 for review.

[v7 1/5] EditCommandTest: Merge with EditCommandIntegrationTest
[v7 2/5] EditCommandTest: Construct EditCommand directly
[v7 3/5] EditCommandTest: Update PersonBuilder usage with more compact version
[v7 4/5] EditCommandTest: Remove unnecessary line breaks in header comments
[v7 5/5] EditCommandTest: Replace `Command command` w/ `EditCommand editCommand`

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v6 and v7)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/445/7/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.

@Zhiyuan-Amos This needs de-conflicting.

v8
@Zhiyuan-Amos submitted v8 for review.

[v8 1/5] EditCommandTest: Merge with EditCommandIntegrationTest
[v8 2/5] EditCommandTest: Construct EditCommand directly
[v8 3/5] EditCommandTest: Update PersonBuilder usage with more compact version
[v8 4/5] EditCommandTest: Remove unnecessary line breaks in header comments
[v8 5/5] EditCommandTest: Replace `Command command` w/ `EditCommand editCommand`

(:books: Archive) (:chart_with_upwards_trend: Interdiff between v7 and v8)

Checkout this PR version locally
git fetch https://github.com/se-edu/addressbook-level4.git refs/pr/445/8/head:BRANCHNAME

where BRANCHNAME is the name of the local branch you wish to fetch this PR to.


ENS Token Layout and Actions tbc

actions button:

[x] Transfer
[x]  Renew
[ ]  set ETH address
[ ]  set Content: IPFS, Swam
[ ]  Set TEXT-DESCRIPTION
[ ] Set TEXT-URL
[ ] the existing "Open on ENS" "button"

Layout

[x] ENS name
[ ] resolve to address
[ ]  reverse address
[x] DESCRIPTION
[ ] URL
[ ] content
[x] renew price
[x] renew date/expired date

P.S. "Floyd: 但就是批量续费不会弄，觉得很麻烦"

"Floyd: 几十个ENS，一个个RENEW"
有没有人做信息查询工具的？例如，查询2020-8-3重新avaliable的到期ENS，精确到分钟的

Design:

[ ] - Token Design (ENS Name)
[ ] - Transfer
[ ] - Renew
[ ] - Set Reverse ETH Address
[ ] - Set Text Description
[ ] - Set Text URL
[ ] - Subdomains
[ ] - Add Subdomain
[ ] - Renew Price

Main Screens:
https://app.zeplin.io/project/5d088205bff2d15de6a4397b/dashboard?seid=5e9d83dddfa2157a545c7213

Set Resolver:
https://app.zeplin.io/project/5d088205bff2d15de6a4397b/dashboard?seid=5e9d86f3cb2b6e7c55f32fe4

Set Reverse ETH Address:
https://app.zeplin.io/project/5d088205bff2d15de6a4397b/dashboard?seid=5e9d87b30b341979444f6bbf

Renew:
https://app.zeplin.io/project/5d088205bff2d15de6a4397b/dashboard?seid=5e9d890f9264e45331a78f54

Transfer
https://app.zeplin.io/project/5d088205bff2d15de6a4397b/dashboard?seid=5e9d89ea0046c27b9767b63b

Set Text Description
https://app.zeplin.io/project/5d088205bff2d15de6a4397b/dashboard?seid=5e9d8edcb104c078551d9669

Set Text URL
https://app.zeplin.io/project/5d088205bff2d15de6a4397b/dashboard?seid=5e9d8fc5eff10a529100ba0b

@James-Sangalli All set.

@James-Sangalli for ENS, when will we have more information displayed and more actions?

@James-Sangalli for ENS, when will we have more information displayed and more actions?

👆  @James-Sangalli


feat(manifests): Adding implementation of manifest files

Adds an implementation for Manifest Lists, Manifest Entries, and Data Files along with interfaces for wrapping file system IO handling.

CC @coded9 @Fokko @nastra @bitsondatadev @rdblue

@coded9 @Fokko  @nastra @rdblue any chance for a review soon to get this in?

@zeroshade, sorry for the delay! I'll find some time for reviews over here.

Rebased with the merge for the partitioning stuff, so should be all good here

@zeroshade I did a first pass around the manifest schema and code and left a few comments. I still need to review the APIs for IO and the tests

@nastra I've implemented the suggested fixes but I had a few questions that I posed when you get a chance. Thanks again!


Rename MLBlock -> MLPipelineStep

That's it :)

Hi @micahjsmith I agree that the "block" name usage can be a bit confusing within the scope of a pipeline.
However, an MLBlock is thought as an independent block of code that can also be executed on its own, outside of the pipeline scope.
The whole rationale of the project naming is:

An MLBlock is just a wrapper around a third party block of code, which provides it a uniform interface suitable for Machine Learning contexts. This is the core concept of the library.
The MLBlocks library is called like that because its core concept is being able to create precisely that: instances of the MLBlock class.
Eventually, we provide the MLPipeline, which allows to easily combine multiple MLBlock instances in the form of a pipeline. This, of course, ends up being a very important concept in the project, but it still revolves around the MLBlock concept.

So, here is my proposal: Let's keep the MLBlock class name, but rename the block concept inside the pipeline to steps.
This way, the MLPipeline will be composed of multiple steps, which inherently have an execution order, and it just happens that each one of these steps is an instance of an MLBlock class.
Does this sound right?


modal max-width question

I've been trying to alter the size of a modal on the screen using the "max-width" property but was finding that 2xl->6xl were all the same width. Looking modal.php code the "getMaxWidth" function seems to set the max-width to max-w-2xl for everything everything from 2xl-6xl.
Is this correct or should they set the max width to the icoming string (ie. 3xl=> sm:max-w-3xl, 4xl => sm:max-w-4xl, etc.) ?

Yes, it's a bug, I'm refactoring all components to make them smaller and faster.
In this refactoration I found some bugs to fix. In this weekend I will release a new version

Thank you for letting me know, I'll await the update.

@ElrondIT it is now fixed, thanks


Bug ('parallel_for failed' error) in cusp/system/detail/generic/multiply.inl and a quick fix

Hi,
When running cg/preconditioned-cg examples in cusp-cuda9 branch with DIA/CSR/ELL matrix formats on CUDA 9.1, I got the following error:

terminate called after throwing an instance of 'thrust::system::system_error'
what():  parallel_for failed: invalid configuration argument
Aborted (core dumped)

Interestingly, COO and HYB formats worked. I found cusp::multiply function somehow breaks the policy    for other functions such as cusp::blas::axpby and even cusp::print to work (all threw same error).
A quick fix worked for me is to change in cusp/system/detail/generic/multiply.inl the following line

multiply(thrust::detail::derived_cast(exec), A, B, C, initialize, combine, reduce, format1, format2, format3);

into

multiply(exec, A, B, C, initialize, combine, reduce, format1, format2, format3);

Not sure if this fix is robust, but hope this info is helpful for your official fix.
Regards,
Ruxi

Thanks, I updated the launch parameters used to execute functions which should resolve your error.

@sdalton1 I had the same problem as @juhsi when running the cg example, in cusplibrary/performance/solver.  This is on the cuda9 branch.
./cg
Using default matrix (5-pt Laplacian stencil)
Running solver on host...
  Successfully converged after 1520 iterations.
  Solver time 26.2332 seconds (17.2587ms per iteration)
Running solver on device...
terminate called after throwing an instance of 'thrust::system::system_error'
  what():  parallel_for failed: invalid configuration argument
Aborted

Applying @juhsi 's suggestion also fixed the error for me.  I have not dug into the code -- does that seem like a reasonable fix, or is there some other problem going on?
nvcc --version gives
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Fri_Nov__3_21:07:56_CDT_2017
Cuda compilation tools, release 9.1, V9.1.85

Thanks and hope to hear from you! :wave:

jk -- found out that develop is newer than cuda9, and had no error using develop (using the scons files from cuda9).  Can cuda9 get merged into develop (and can that be merged into master?)?  The branch layout is confusing for first-time users right now, especially with no notes in the readme on which branch to use :)
In any case you should probably close this issue since I can confirm this error is fixed on develop -- thanks!!

Hi All,
I am using Ubuntu 18.04, CUDA 9.2, gcc 7.3.0
I am having issues related to compiling a program that uses CUSP to solve a complex values parse linear system.
The compile errors have to do with /usr/local/cuda/include/cusp/system/detail/generic/multiply.inl and seem related possibly to the problems reported above.
_/usr/local/cuda/include/cusp/system/detail/generic/multiply.inl(56): error: "cusp::system::detail::generic::enable_if" is ambiguous
/usr/local/cuda/include/cusp/system/detail/generic/multiply.inl(74): error: "cusp::system::detail::generic::enable_if" is ambiguous
/usr/local/cuda/include/cusp/system/detail/generic/multiply.inl(80): error: function template "cusp::system::detail::generic::multiply(thrust::execution_policy &, const LinearOperator &, const MatrixOrVector1 &, MatrixOrVector2 &)" has already been defined_
Previous blogs seem to indicate that multiply.inl has been updated on March 14th 2018, but when checking the contents of
https://github.com/cusplibrary/cusplibrary/tree/master/cusp/system/detail/generic
it appears to me that this file is 3 years old under "develop", "cuda9", and 4 years old under "master"
Can anybody kindly advise ? Has multiply.inl actually been updated ? Did anyone run in the same problem?
Best Regards, thanks,
Andrea

I am having the same issue when I use multiply as part of a larger code.
This code
#include <cusp/array1d.h>
#include <cusp/csr_matrix.h>
#include <cusp/monitor.h>
#include <cusp/blas/blas.h>
#include <cusp/linear_operator.h>
#include <cusp/gallery/poisson.h>
// include cusp gauss_seidel header file
#include <cusp/relaxation/gauss_seidel.h>
#include <cusp/relaxation/jacobi.h>
int main()
{
// Construct 5-pt Poisson example
cusp::csr_matrix<int, double, cusp::host_memory> B;
cusp::gallery::poisson7pt(B, 8, 8, 8);
cusp::csr_matrix<int,double, cusp::device_memory> A(B);
// Initialize data
cusp::array1d<double, cusp::device_memory> x(A.num_rows, 0);
cusp::array1d<double, cusp::device_memory> b(A.num_rows, 1);
// Allocate temporaries
cusp::array1d<double, cusp::device_memory> r(A.num_rows);
// Construct gauss_seidel relaxation class
cusp::relaxation::gauss_seidel<double, cusp::device_memory> M(A);
// Compute initial residual
cusp::multiply(A, x, r);
cusp::blas::axpy(b, r, double(-1));
// Construct monitor with stopping criteria of 100 iterations or 1e-4 residual error
cusp::monitor monitor(b, 100, 1e-4, 0, true);
// Iteratively solve system
while (!monitor.finished(r))
{
M(A, b, x);
cusp::multiply(A, x, r);
cusp::blas::axpy(b, r, double(-1));
++monitor;
}
}
compiles and runs fine.


Giving 0 points on empty text submissions is impossible

Describe the bug
Givin 0 points on empty text submissions is impossible
To Reproduce
-Open an empty text submission
-realize that there is nothing you can do
Expected behavior
Either 0 points given automatically or the option to give points in "general feedback"

Nevermind, you can give general feedback and then you can submit


Add first KEKW NFT

[x] I will not ping the Discord about this pull request.

Please provide the following information for your token.
Please include change to the src/lib/.json and in the PR
At minimum each entry should have

Token Address: 5WUab7TCvth43Au5vk6wKjchTzWFeyPEUSJE1MPJtTZE
Token Name: KEKW In Solana Tripping
Token Symbol: KEKN1
Logo: https://www.kekw.io/images/KEKN1_logo.png
Link to the official homepage of token: https://www.kekw.io/

https://github.com/solana-labs/token-list/pull/216


Resolving artifacts may take exponential execution time

In current (0.9.3) implementation, getResolvedArtifactsFromResolvedDependencies doesn't check whether it is already called for each resolvedDependency.
As a result, for projects with complex dependencies, this process may be called multiple times for the same dependency object, and in the worst case the number of times increases exponentially.
This causes the command to take a very long time to run, almost resulting in a hang.
This problem can be fixed by recording the dependencies that have already been processed.
Here is my sample implementation.
https://github.com/monae/gradle-license-plugin/commit/82c683240937225a2d5f8fd729df73bfebfcaea8

I wrote a test for this situation. This test takes about 2 minutes on my machine before the patch is applied, but now takes only a few seconds after the patch is applied. Increasing the depth parameter will cause the execution time to increase more exponentially before patching, but will only increase linearly after patching.
https://github.com/monae/gradle-license-plugin/commit/3fe4e5d429dbc9c00dd4919a733efa06d4707f85
If it's OK, I will make a PR.

yep we're running into the same issue and I locally made a very similar change to make it work. Went from > 20 mins (I eventually canceled it) to ~30s.

@monae Thanks. Yes, send the PR, PRs are always welcome.


UI Does not load.

Describe the bug
I activate the server I get this:
[03:57:35] INFO     [03:57:35] - INFO - Logger set up with log level: 20(info)                             logger.py:28
INFO     [03:57:35] - INFO - Log file: logs\langflow.log                                        logger.py:30
←[32mINFO←[0m:     Started server process [←[36m34988←[0m]
←[32mINFO←[0m:     Waiting for application startup.
←[32mINFO←[0m:     Application startup complete.
←[32mINFO←[0m:     Uvicorn running on ←[1mhttp://<IP_ADDRESS>:7860←[0m (Press CTRL+C to quit)
←[32mINFO←[0m:     <IP_ADDRESS>:57223 - "←[1mGET / HTTP/1.1←[0m" ←[32m200 OK←[0m
←[32mINFO←[0m:     <IP_ADDRESS>:57223 - "←[1mGET /assets/index-e39661aa.js HTTP/1.1←[0m" ←[32m200 OK←[0m
←[32mINFO←[0m:     <IP_ADDRESS>:57224 - "←[1mGET /assets/index-d5403acc.css HTTP/1.1←[0m" ←[32m200 OK←[0m
←[32mINFO←[0m:     <IP_ADDRESS>:57224 - "←[1mGET /favicon.ico HTTP/1.1←[0m" ←[32m200 OK←[0m
When I load the localhost URL (<IP_ADDRESS>:7860) into my browser I get a blank screen.  I am running in a miniconda Virtual Environment
Browser and Version

Browser : Firefox 113.0.0 64Bit, also testing with Edge 113.0.1774.50 (Official build) (64-bit)

To Reproduce
Steps to reproduce the behaviour:
1 conda create langflow
2 conda activate langflow
3 pip install langflow
4 python -m langflow
4 Type http://<IP_ADDRESS>:7860 to URL bar
See error (Blank Screen)

I had the same issue and to get the interface working I created a new named tab - possibly related to the root cause.

Could you walk me through how to do that please?

Facing the same issue , any resolutions?

This issue is a duplicate. I'm closing this one in favor of #353. Please try deleting cookies and cache and reporting in that issue.


Internal Server Issue

I already resolve the window & navigator(#55) issue
Project is successfully build with no error "node dist/server.js" is also run successfully with no error
but when I'm going to access in browser then I'm getting  " 500 Server error occurred"
Error: Minified React error #149; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=149&args[]=carouselWrapper for the full message or use the non-minified dev environment for full errors and additional helpful warnings.

Well are you using multiple react version? is carouselWrapper a plugin or npm package?

I ran into the same error. Refs cannot be defined by strings.
Use this notation:
<Element ref={(el) => {this.el = el;}} />

On Dec 15, 2017, at 8:24 AM, Tirth Bodawala<EMAIL_ADDRESS>wrote:
Well are you using multiple react version? is carouselWrapper a plugin or npm package?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@RizwanZaheer Does the solution provided by @dav-is solves your aissue?

Closing due to no-reply.

@tirthbodawala No, I'm using single verison of react.
carouselWrapper is a simple class defined in this project for carousel
In Dev mode It's work fine but this issue came in production build

@RizwanZaheer  Have you upgraded to latest version of react-pwa boilerplate? its using pawjs.


Check targetgroup health status during asg replacement

Targetgroup needs to be healthy in order for ALBs to route traffic to
them.  This patch requires nemesys to check for target group health
alongside elb and asg health status; all before moving to terminating
the old asg.
This patch should prevent an issue when replacing ASGs with only
targetgroups. The load balancer returns no unhealthy instances,
therefore nemesys proceeds to terminate the old ASG even though the new
ASG can't take new traffic yet.
The way we check is still leaky.  The passing condition is that there
are no unhealthy instances in either the target group or the load
balancer.  But 0 unhealthy instances and no healhty instances are not
synonymous.  In the future, we should probably switch to the
number of healthy instances across elbs and tgs needs to match the
desired capacity.  An additional health check would be to actually hit
the uri endpoint of the load balancer to make sure traffic is routable
before finally terminating the old asg.

thinking about this a little more after reading your pr description again, i think it does make sense to look for these instances to explicitly be healthy, rather than checking that they are not unhealthy. so, once all the instances are healthy that are in the asg, it is time to start spinning down the old ones. lmk what you think, it doesnt seem like that would be a huge change


[Question] Can I use DolphinScheduler to view the logs of Flink or Spark cluster tasks?

Describe the question
I use DolphinScheduler to submit Spark and Flink tasks, found that the following task view log has task running output:

Spark on local
Spark on yarn(client)

The following task view logs only have JobID (or ApplicationID or no running output):

Flink on local
Spark on yarn (cluster)

So I can only manually open the page of Flink or Yarn to view the logs. I hope to view the output of these tasks more conveniently.

我使用 DolphinScheduler 提交 Spark 和 Flink 任务，发现以下任务查看日志有任务运行输出：

Spark on local
Spark on yarn(client)

以下任务查看日志仅有 JobID （或ApplicationID或无运行输出），

Flink on local
Spark on yarn（cluster）

所以我只能手动打开 Flink 或 Yarn 的页面查看日志，我希望能更方便地查看这些任务的运行输出。
Versions

DolphinScheduler: 1.3.2
Spark: 2.3.2
Flink: 1.11.2

Requirement or improvement
Does DolphinScheduler (v1.3.2 or dev) support log viewing of these tasks? If not, is there a plan for implementation?
If it is to be implemented, the implementation strategy is to configure the front-end jump (such as #851, #853), or grab the log and save it in the DolphinScheduler system?

DolphinScheduler（v1.3.2或dev）是否支持这些任务的日志查看?  如果不支持，是否有实现计划？
如果要实现，实现策略是配置前端跳转（如#851，#853），还是抓取日志存到 DolphinScheduler 系统里？

There is currently no better way to solve this problem

Have dolphin solved this problem yet?

调度spark任务的时候怎么在任务实例日志中查看到更详细的日志，show的代码部分不显示

Is there any guy who‘d like to implement the feature of showing yarn task log? A simple solution is to add a task link that jumps to yarn, and it feels like there is no need to capture yarn logs to DS.


HDDS-11845. Extract k8s definitions for HttpFS and Recon from getting-started example

What changes were proposed in this pull request?
HDDS-11806 added HttpFS and Recon in k8s/examples/getting-started.  The goal of this task is to extract the new components to k8s/definitions/ozone.
Changes to hadoop-ozone/dist/src/main/k8s/examples/getting-started in d5c96085877f49a7719b292d00b124d0e0d0d14e show that re-generated example are the same (with trivial differences in ordering and quoting).
Reorder config properties by name.
Also add a helper script to regenerate examples more easily, without having to manually install Flekszible.
https://issues.apache.org/jira/browse/HDDS-11845
How was this patch tested?
CI:
https://github.com/adoroszlai/ozone/actions/runs/12135757461

Thanks @dombizita for the review.


Align repo structure with team structure

I already had #14 open for a longer time that did some of this. I also just forced through #18 which does some of what is done here.
Please rebase.


add CustomAuthDomain parameter to template.yaml

Description of changes:
Cognito allows you to configure a User Pool with a custom domain (which is usually the {auth subdomain}.{your domain} of the SPA). This change gives the option for consumers of the application to specify their own domain, and have the Lambda@Edge functions use that auth domain (for redirects, etc.) instead of the default Cognito provided domain.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Please note this does add the custom auth domain config to the User Pool for the consumer. A valid custom domain config needs a valid AWS managed certificate that covers the custom auth domain. The config can be done in the AWS console, CLI, or with a CloudFormation template.

First off, thank you for your work!
I have been reviewing this PR, and I like how clean it is.
Going through it, here's a couple of review remarks:

I can imagine users also expecting the parameter "CustomAuthDomain" to entail that the user pool would be created with that domain name. The description clears this up, but "ExistingAuthDomainName" (or so) would I think be a better name for the parameter.
The Custom::UserPoolDomain resource should become conditional now right?
It would be great if we could also have an example on the use of the new "CustomAuthDomain" in the folder "example-serverless-app-reuse"

But right now, I'm doubting if we should actually abandon this approach ... (sorry) ... and go for an alternate approach only: make it possible to use an existing user pool (and domain and client) (there is an enhancement request for this already: #19 ). That would solve for even more usage scenario's:

Have the User Pool be in another region (often requested)
Ability to customize the user pool (e.g. attributes, MFA, advanced security, domain).

Also, that would keep the serverless application more simple. You added a parameter today, but it is conceivable that there will be more PR's for other similar customizations of the user pool, domain and/or client. It will become many parameters, and then it would surely be better to just let users setup their own user pool just the way they want it, and use that.
So TBH I am leaning towards implementing #19 instead of going forward with this PR.
I would appreciate hearing your thoughts on this.

Sorry for the delay.
I agree with all of your first set of points. And no worries if you want to abandon this approach. My opinion on #19 is the difficulty it would be to ensure the consumer configures the user pool correctly for use in the edge Lambdas. I believe in giving the consumer a very specific set of parameters that can be customized, and reserving the rest to guarantee a correct configuration. So I am in favor of future PR's for similar customizations as they are needed.

the difficulty it would be to ensure the consumer configures the user pool correctly for use in the edge Lambdas

Can you elaborate on that? What difficulties do you perceive? The User Pool client with the callback URL's and such?
It may be an option for #19 to create the client in our code anyway.

Closing in favor of #75
Also: CloudFormation now supports natively the creation of User Pool Domain and User Pool Client with OAuth config.


Question: Why is UInt32 transferred as LongValue?

I did some tests with SparkplugNet and came across some ambiguities.
A UInt32 is interpreted by the server as a 0 because the server implementation evaluates IntValue of the protobuf message.
The mapping between SparkplugB data types and Protobuf data types is somewhat confusing. In this case, however, it is relatively clear to me: unsinged Int32 should become unsigned Int32, as described in the specification:
https://github.com/eclipse-sparkplug/sparkplug/blob/e45b61002a2a0cacb4f11fa4be88670435ccfcb0/specification/src/main/asciidoc/chapters/Sparkplug_6_Payloads.adoc?plain=1#L899-L902
But your implementation writes the value into LongValue:
https://github.com/SeppPenner/SparkplugNet/blob/758b561206807f8c46ee11e6594054ce3ca75ff8/src/SparkplugNet/VersionB/PayloadConverter.cs#L221-L225
Now my questions:

Does the server need to be able to handle it if there is a UInt32 value in the LongValue field?
Is there a reason for this?


Add Docker support for bdutil

Corporate CLA applies, merged as https://github.com/GoogleCloudPlatform/bdutil/commit/21bc14fa65da11c033b35d18de88c6556c8da9f7
Thanks!


docker-alpine redis how to get into bash

CONTAINER ID        IMAGE                     COMMAND             CREATED             STATUS              PORTS               NAMES
714ecf80a71b        smebberson/alpine-redis   "/init"             39 minutes ago      Up 39 minutes       6379/tcp            redis
$ docker exec -it redis bash
exec: "bash": executable file not found in $PATH

These images don't contain Bash. They're based on Alpine Linux, but can you use docker exec -it with-contenv sh. This will give you a command line in the container with environment variables loaded.
I hope that helps. Please let me know so I can close out this issue if this has resolved your issue.

Yes. You can close it. I use ash for now from alpine linux.
I have few questions regarding the redis-server and redis-cli,
I want one Generic Redis Server Container and then 'N' number of Redis-CLI Container(with only the redis-cli tools)
I wanted only the redis-cli container to get created when i run the below docker-compose file
docker-compose.yml file
Original compose-file
rcli:
image: redis:latest
links:
- redis
command: redis-cli -h redis
Converting to alpine linux:
**_rcli
image: smebberson/alpine-redis
links:

redis
command: redis-cli -h redis
_**
with the Docker Network options with the link below can we achieve what am intended?
https://github.com/sickp/docker-alpine-redis

I wanted to call the below compose command from the webapp, the idea is to share the redis-server container with mulitple users only via redis-cli container exposing the redis-cli in web
docker-compose run rcli

Let me look into this and get back to you.

any luck? regarding the cli and server redis container with cli container exposed to web and allowing the cli and redis server talking to internal private n/w

you can install bash by adding to your Dockerfile:
RUN apk add --update bash && rm -rf /var/cache/apk/*

If you don't want to add to the size by installing bash, you can just run /bin/sh

Usually, an Alpine Linux image doesn't contain bash, Instead you can use /bin/ash, /bin/sh, ash or only sh.
docker run -it --rm alpine /bin/ash
docker run -it --rm alpine /bin/sh
docker run -it --rm alpine ash
docker run -it --rm alpine sh
--rm Automatically remove the container when it exits.
I hope this information helps you.

Usually, an Alpine Linux image doesn't contain bash, Instead you can use /bin/ash, /bin/sh, ash or only sh.
docker run -it --rm alpine /bin/ash
docker run -it --rm alpine /bin/sh
docker run -it --rm alpine ash
docker run -it --rm alpine sh
I hope this information helps you.

apk --update bash
if you need bash for other things also, like running scripts, this command could be useful.

docker exec -it some-docker_app_1 sh

docker exec -it CONTAINER_ID sh

@jansanchez is there any difference in any of these shells?


0 and 1 inputs in database. Why?

I had changed the array variable allowedRatings in the program and it shows the result.
Is that result obtained is correct?

Hi,
￼
As stated in the README, the ratings should currently follow the following format:
"""
a matrix containing of 0:s (not rated) and 1:s (liked the item) and follow this format.
"""
No checks are currently done to ensure that the data in the rating matrix only consists of 0:s and 1:s, meaning you will get a result but a result that is meaningless (incorrect)

For non-binary rating values, how should I proceed with the code?
Any suggestions from your side. Like changing similarities functions, or changes in Comatrix.
Suggestions from you will be appericiated.

This should definitely be doable, I just have not made all the necessary research myself on the mathematics of how to expand the current implementation to support such data input. If you want to use the same approach as in this library, I would suggest reading about Jaccard index. I think understanding this part might be more difficult than implementing the code, at least it was in my experience when I wrote this library. As it was a long time ago I wrote this library, I don't remember much about Jaccard similarities and unfortunately, I currently don't have time to expand the library myself.
Good luck!


How is join implemented in ClickHouse ?

How is Join implemented in ClickHouse ?
There is a Join.cpp file under interpreters directory, but it looks like all the join operator implementation details is just insert data into blocks. Where is the major algorithmic procedure for join there ?
Best Regards,
Hao

https://github.com/yandex/ClickHouse/blob/e1271ae1f2a9bfb635ef83b23e48e9791195cce3/dbms/src/Interpreters/Join.h#L65-L120

Right side of JOIN is collected in hash table in memory.
Then while reading from left side of JOIN we'll do lookups into hash table and find rows to be joined.

Normally in RDBMS optimizers take care of the JOIN order (Left, Right). In the case of ClickHouse, tables with high selectivity, i.e. fewer rows at the right side of the JOIN should be preferred and performance will be significantly better.


tabs indentation

There's an issue with the cursor width when the file has tabs for indentation, where the cursor is as wide as the tab stop, no problems when using spaces for indentation. Is this a known issue or someone can corroborate?
Thank you.

Same for me - if you have tabs and jump one word to right - cursor position is completely broken and editing happens in wrong places.


Documentation review - Indoor Hotspot -

Bold text should be reformulated/edited in the following files:
https://github.com/NebraLtd/Helium-Guides/blob/main/docs/indoor-hotspot/quick-start.md#preparing-your-nebra-indoor-hotspot

Step 3: If you are using a wired Ethernet connection, connect an Ethernet cable between a router or switch and the Ethernet jack on the Hotspot.
Step 5: Finally connect the DC Jack from the power supply into the power receptacle on the Hotspot.

You should then see the lower LED on the back of the unit turn on, the hotspot will now take up to 5 minutes to configure for it’s first boot. --> turns on
https://github.com/NebraLtd/Helium-Guides/blob/main/docs/indoor-hotspot/quick-start.md#configuring-your-nebra-indoor-hotspot
Configuring Your Nebra Indoor Hotspot
To configure your Hotspot you will require the Helium Network application installed on a Mobile Phone and for you to have gone through the account setup process to continue.
Step 2: Next click Set up Hotspot , from here you will want to select Nebra Indoor Hotspot. --> ** Delete space before the comma :P**
Step 8: Finally you can confirm the location of your hotspot. Click continue and you should be presented with a map to then place where your hotspot is on the app.
https://github.com/NebraLtd/Helium-Guides/blob/main/docs/indoor-hotspot/troubleshooting.md#no-lights-are-turning-on
No lights are turning on:
You should see the lower light on the back of the unit turn on as soon as power is applied. If this is the case, ensure the power supply is firmly plugged into the wall outlet, the power jack is plugged into the unit and that the wall outlet is switched on.  --> turned on
Only the lower light will turn on to begin with until the unit is running. --> At the beginning, only the lower light....
https://github.com/NebraLtd/Helium-Guides/blob/main/docs/indoor-hotspot/troubleshooting.md#my-hotspot-is-not-showing-in-the-helium-app
My hotspot is not showing in the Helium App:
It can sometimes take 1 minute from power on for the Hotspot to show in the App, simply re-scanning a few times can then find the hotspot.
https://github.com/NebraLtd/Helium-Guides/blob/main/docs/indoor-hotspot/troubleshooting.md#updating-your-nebra-hotspot
Updating your Nebra Hotspot
If after completing initial troubleshooting steps above you're still having issues, or you are advised to you may find your issues can be resolved by ensuring it is running the latest up to date software.
https://github.com/NebraLtd/Helium-Guides/blob/main/docs/indoor-hotspot/indoor-faq.md#what-antenna-is-included-with-the-hotspot
We are currently confirming the final model of antenna included with the indoor hotspot.  --> Not confirmed yet?
https://github.com/NebraLtd/Helium-Guides/blob/main/docs/indoor-hotspot/indoor-faq.md#can-i-use-poe-with-the-indoor-hotspot
If you wish to use PoE with the Indoor Hotspot you will require an active PoE splitter that outputs 12V to the unit. Alternatively you can use passive PoE adapters as long as the power provided to the unit is between 9 and 16 Volts and can provide 15W of power.     --> Injector then Splitter?

Thanks, Only need to make the text bold that mentioned paragraphs right?

The bold text, except the titles, should be reformulated in English.

@fouad-semaan Could you help me with the following?

Step 3: If you are using a wired Ethernet connection, connect an Ethernet cable between a router or switch and the Ethernet jack on the Hotspot.

What's incorrect with the above? To me this reads fine.

You should then see the lower LED on the back of the unit turn on, the hotspot will now take up to 5 minutes to configure for it’s first boot. --> turns on

"Turns On" doesn't sound correct to me, verbally the led on the back of the unit would turn on. Turns on would be past tense?

No lights are turning on:
You should see the lower light on the back of the unit turn on as soon as power is applied. If this is the case, ensure the power supply is firmly plugged into the wall outlet, the power jack is plugged into the unit and that the wall outlet is switched on. --> turned on

As above, turned on would be past tense.

Only the lower light will turn on to begin with until the unit is running. --> At the beginning, only the lower light....

Reworded

Updating your Nebra Hotspot
If after completing initial troubleshooting steps above you're still having issues, or you are advised to you may find your issues can be resolved by ensuring it is running the latest up to date software.

I've re-worded this as then realised that if we're guiding them on what to do they wouldn't be at this page. Reads slightly better now.

We are currently confirming the final model of antenna included with the indoor hotspot. --> Not confirmed yet?

Added in details.

If you wish to use PoE with the Indoor Hotspot you will require an active PoE splitter that outputs 12V to the unit. Alternatively you can use passive PoE adapters as long as the power provided to the unit is between 9 and 16 Volts and can provide 15W of power. --> Injector then Splitter?

I think for this just splitter is fine, as it's indicating the hardware required for the hotspot more on a "I know enough about POE but not sure what splitter I need" route rather than a "How do a DIY POE" with it.

connect an Ethernet cable between a router or switch and the Ethernet jack on the Hotspot.
Ahhh now I understand it, switch here is a word and not a verb. So maybe we can add a switch. Newbies as me will understand better.
For the rest, that's ok!
Thanks.


Problem building CARLA 0.9.8

When building CARLA  0.98 during initial setup, the "make launch" command fails with the following error. Any help is greatly appreciated.
Following is the error log.
/home/grasp/carla/Unreal/CarlaUE4/Plugins/Carla/CarlaDependencies/include/boost/geometry/index/detail/rtree/pack_create.hpp:83:49: warning: declaration shadows a local variable [-Wshadow]
typename coordinate_type::type median
^
/home/grasp/carla/Unreal/CarlaUE4/Plugins/Carla/CarlaDependencies/include/boost/geometry/index/detail/rtree/pack_create.hpp:73:45: note: previous declaration is here
static inline void apply(EIt first, EIt median, EIt last, Box const& box, Box & left, Box & right, std::size_t dim_index)
^
In file included from /home/grasp/carla/Unreal/CarlaUE4/Plugins/Carla/Intermediate/Build/Linux/B4D820EA/UE4Editor/Development/Carla/Module.Carla.cpp:12:
/home/grasp/carla/Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Game/CarlaEngine.cpp:63:57: error: no matching member function for call to 'AddRaw'
OnPreTickHandle = FWorldDelegates::OnWorldTickStart.AddRaw(
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
/home/grasp/UnrealEngine_4.22/Engine/Source/Runtime/Core/Public/Delegates/DelegateSignatureImpl.inl:765:25: note: candidate function [with UserClass = FCarlaEngine, VarTypes = <>] not viable: no known conversion from 'void (FCarlaEngine::)(UWorld , ELevelTick, float)' to 'typename TMemFunPtrType<false, FCarlaEngine, void (ELevelTick, float)>::Type' (aka 'void (FCarlaEngine::)(ELevelTick, float)') for 2nd argument
inline FDelegateHandle AddRaw(UserClass InUserObject, typename TMemFunPtrType<false, UserClass, void (ParamTypes..., VarTypes...)>::Type InFunc, VarTypes... Vars)
^
/home/grasp/UnrealEngine_4.22/Engine/Source/Runtime/Core/Public/Delegates/DelegateSignatureImpl.inl:770:25: note: candidate function [with UserClass = FCarlaEngine, VarTypes = <>] not viable: no known conversion from 'void (FCarlaEngine::)(UWorld , ELevelTick, float)' to 'typename TMemFunPtrType<true, FCarlaEngine, void (ELevelTick, float)>::Type' (aka 'void (FCarlaEngine::)(ELevelTick, float) const') for 2nd argument
inline FDelegateHandle AddRaw(UserClass InUserObject, typename TMemFunPtrType<true, UserClass, void (ParamTypes..., VarTypes...)>::Type InFunc, VarTypes... Vars)
^
2 warnings and 1 error generated.
Makefile:215: recipe for target 'CarlaUE4Editor' failed
make[1]: *** [CarlaUE4Editor] Error 5
make[1]: Leaving directory '/home/grasp/carla/Unreal/CarlaUE4'
Util/BuildTools/Linux.mk:7: recipe for target 'launch' failed
make: *** [launch] Error 2

I also have this problem, have you figured out how to solve it?

the same here  :/

Hi,
Are you using 0.9.8 or the version in master? If anyone of you is using master, please upgrade UE to 4.24. Otherwise, we are looking into this issue.

I was using UE 4.22. However I fixed the wrror by changing the function signature from  FCarlaEngine::OnPreTick(UWorld*, ELevelTick TickType, float DeltaSeconds) to  FCarlaEngine::OnPreTick(ELevelTick TickType, float DeltaSeconds) in 'carla/Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Game/CarlaEngine.cpp' and 'carla/Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Game/CarlaEngine.h'
Looks like they had a unnamed variable in their function signature, possibly to allow for future addition of feautres. But the Unreal Engine was using DECLARE_MULTICAST_DELEGATE_TwoParams(FOnWorldTickStart, ELevelTick, float), which takes only two params, to define FOnWorldTickStart, in '/UnrealEngine_4.22/Engine/Source/Runtime/Engine/Classes/Engine/World.h'

Hi, the problem is that with Unreal 4.24 the signature of those functions have changed. We have upgraded to version 4.24, and now we are working on the documentation to say that clearly.
Sorry for the inconvenient.

Hi,
Thank you for getting back to us. Can I continue using 4.22 ? or would you recommend upgrading to 4.24? If it’s just this minor change then I think it should be fine?

From our point of view we are upgrading because seems that UE4.24 is more stable using the Vulkan driver with high demanding memory. But we are not using exclusive features of 4.24 right now. That means that our code has only minor changes to adjust the new version.
But the thing is that with the new Unreal 4.24 we need to upgrade to a new version of the compiler (LLVM 8.0) so going back and forward will not be straight forward.
My suggestion is that as soon as we publish the new documentation with the new instructions to install and compile UE4.24, you should do that.
Regards

Chang to fix Unreal4.24 fix my problem, thanks for help :)

Chang to UnrealEngine4.24 fix my problem, thanks for help :)


[utils] Fix build with Parcel v2

[X] I have followed (at least) the PR section of the contributing guide.

I ran into an error while building a webapp with Material UI 5.0.0-alpha.25 using Parcel v2 which relies on Terser to minify JS.
Here's what the error message looks like:
➜ yarn parcel build src/index.jsx --no-cache
🚨 Build failed.
@parcel/optimizer-terser: Illegal newline after 'throw'
/home/user/Development/project/.yarn/$$virtual/@material-ui-utils-virtual-bc8a155bc8/0/cache/@material-ui-utils-npm-5.0.0-alpha.25-4f8f056857-49c676c074.zip/node_modules/@material-ui/utils/esm/capitalize.js:9:10
   8 |       /* FIXME (minify-errors-in-prod): Unminified error message in production build! */
>  9 |       new Error(`Material-UI: \`capitalize(string)\` expects a string argument.`)
>    |          ^ Illegal newline after 'throw'
  10 |     );
  11 |   }
It's likely that Terser doesn't support this syntax yet.

Looking further into it I figured there must've been a throw statement right before the error, but the auto-generated comment gets inserted between them, which according to this on MDN automatically terminates the throw statement since no newline is allowed.
There might be something here for Terser to look into as well, but I figured it was right to come to Material UI with the issue since it seems like a syntax error rather than something for Terser to fix.

Thanks for the report.
Babel already wraps the error in () so that you don't have a newline between throw and new. Mind that we don't generate
throw
/* FIXME (minify-errors-in-prod): Unminified error message in production build! */
new Error(`Material-UI: \`capitalize(string)\` expects a string argument.`)

but
throw (
  /* FIXME (minify-errors-in-prod): Unminified error message in production build! */
  new Error(`Material-UI: \`capitalize(string)\` expects a string argument.`)
);

If I put our generated code into https://try.terser.org/ then I don't get an error but throw new Error("Material-UI: capitalize(string) expects a string argument.");
Could you file this as a parcel bug as well because I'm pretty sure we're doing nothing wrong here.

@eps1lon Interesting, now that you mention it I notice the ); at the end of the Parcel error. This looks like it should be valid code indeed.
I've submitted an issue to Parcel here: parcel-bundler/parcel#5931

Thanks for clarifying!
Going to close this since I don't want to push unnecessary changes. Older versions of Material-UI would still be affected.


clickhouse-client interactive mode save to file

Hi,
Can we get an option to save results from interactive mode to a file?
e.g. "SELECT * FROM temp_table FORMAT CSV TOFILE '~/results.csv'"

Probably
SELECT * FROM temp_table FORMAT CSV INTO OUTFILE '~/results.csv'
(for those who familiar with MySQL)
It will be client only option; client will send query to server without this modifier, receive result and write to outfile. If server will receive query with INTO OUTFILE, that was sent directly, it will understand syntax, but throw an exception. (Also possible to allow server to write data to local filesystem, but need special respect to some security considerations.)

Client only option will be great! Thanks

Maybe we need to place INTO OUTFILE before FORMAT.
SELECT * FROM temp_table INTO OUTFILE '~/results.csv' FORMAT CSV
Because maybe we later would like to add support for something like FROM FILE for INSERT query, and in INSERT query, everything after FORMAT section is parsed as data for insertion by separate parser, so in INSERTs, FORMAT must be last.

I don't think INTO OUTFILE before FORMAT is necessary as INTO is already a keyword in INSERT so you can't use it twice, moreover I can't think of a scenario where you insert data to a database and also output that to a file.

I mean different thing.
If we have an option to run clickhouse-client interactively, and then save query result to file,
than, for symmetry,
we also would like to have an option to run clickhouse-client interactively, and then use a file to read as source of data for INSERT query.

okay then the INTO OUTFILE before FORMAT will work as well.

in stable now


Fix exposing of .env variables in Create React App

Create React Apps have the nifty feature of getting environment variables either from the shell or .env files.  These are then resolved and baked into the JS bundle by webpack.
When using debug, the entirety of your .env file gets exposed in the bundle, e.g.:
    function u() {
      var e;
      try {
        e = t.storage.debug
      } catch (e) {
      }
      return !e && "undefined" !== typeof r && "env" in r && (e = Object({
        NODE_ENV: "production",
        PUBLIC_URL: "",
        REACT_APP_FOO: "1"
      }).DEBUG), e
    }

This PR fixes that situation by moving the env access to a separate file that isn't accessed when in browser mode.
Also should resolve https://github.com/visionmedia/debug/issues/467#issuecomment-324054986.

Coverage increased (+1.2%) to 75.0% when pulling 3d42235dac35b1fb4f3b77c00589128759bd1202 on moos:fix-cra-env into 22f993216dcdcee07eb0601ea71a917e4925a30a on visionmedia:master.

Coverage increased (+1.2%) to 75.0% when pulling 3d42235dac35b1fb4f3b77c00589128759bd1202 on moos:fix-cra-env into 22f993216dcdcee07eb0601ea71a917e4925a30a on visionmedia:master.

Coverage increased (+1.2%) to 75.0% when pulling 3d42235dac35b1fb4f3b77c00589128759bd1202 on moos:fix-cra-env into 22f993216dcdcee07eb0601ea71a917e4925a30a on visionmedia:master.

I'm :-1: on this. It's adding very specific checks where this is really a shortcoming in electron. Is this even still an issue? I know this is kind of an old PR.

Yup - definitely still an issue.  It's not an electron issue -- the code referenced above ends up in CRA bundle that's consumed by the browser, thereby exposing the entirety of CRA app's various .env files to the web site visitor -- clearly, an unintended (and hard-to-detect, unless you analyze the bundle -- I bet most app devs don't do that!) side-effect of .env + debug.
I'd be happy to submit an update if there is interest.

Hi, sorry it took so long to get back. Could you rebase please? I'm okay with adding this check.

holy 🐟  -- let's hope I don't have to do that again!

Ping!  Would hate to see this become stale again.


Yetus-61 Add support for gerrit

I haven't tested with different branches and how ROBOT works. If someone has a project they can hook up to this and get me some feedback would be awesome.
Need to look into how different branches work too.

Wildly out of date and no further activity on it. Closing this PR.


Add hidden bar

Project URL

https://github.com/dwarvesf/hidden
Category

Menubar
Description

Add Hidden Bar app.
Why it should be included to Awesome macOS open source applications  (optional)
Hidden Bar is a great open source alternative to Bartender.
Checklist

[x] Edit applications.json instead of README.md.
[x] Only one project/change is in this pull request
[x] Screenshots(s) added if any
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 12 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

4643
Error
https://klaussinani.tech/ao/media/list-navigation.gif  SSL_connect returned=1 errno=0 state=error: certificate verify failed (self signed certificate)

4644
Error
https://klaussinani.tech/ao/media/header.png  SSL_connect returned=1 errno=0 state=error: certificate verify failed (self signed certificate)

4645
Error
https://klaussinani.tech/ao/media/sepia-theme.png  SSL_connect returned=1 errno=0 state=error: certificate verify failed (self signed certificate)

4646
Error
https://klaussinani.tech/ao/media/black-theme.png  SSL_connect returned=1 errno=0 state=error: certificate verify failed (self signed certificate)

4647
Error
https://klaussinani.tech/ao/media/dark-theme.png  SSL_connect returned=1 errno=0 state=error: certificate verify failed (self signed certificate)

4648
Error
https://klaussinani.tech/ao/media/compact-mode.png  SSL_connect returned=1 errno=0 state=error: certificate verify failed (self signed certificate)

4649
Error
https://klaussinani.tech/ao/media/custom-shortcut-keys.gif  SSL_connect returned=1 errno=0 state=error: certificate verify failed (self signed certificate)

4650
Error
https://klaussinani.tech/ao/media/scalable-interface.gif  SSL_connect returned=1 errno=0 state=error: certificate verify failed (self signed certificate)

5568
404
https://raw.githubusercontent.com/newmarcel/KeepingYouAwake/master/Extras/Screenshot.jpg

6271
404
https://bitbucket.org/kosovan/theunarchiver/src

7583
Dupe
https://github.com/spotter-application/spotter

7616
404
https://khrykin.github.io/strategr/resources/Strategr.png

  Generated by :no_entry_sign: Danger

@jadchaar  Thank you! 👍


Google-search-cleanup: "People also search for" is not blocked on mobile versions

What filter does this issue target?
https://letsblock.it/filters/google-search-cleanup
What changes should be made?
Checked with: desktop chromium + uBlock origin, android chromium + Adguard, android firefox + uBlock origin, android kiwi browser + uBlock origin.
Link:
https://www.google.com/search?q=remove+people+also+search+for&ie=UTF-8&hl=en

Did you write a uBlock filter solving this?
No
Additional notes
No response

Can't reproduce. Can you give a better screenshot of the html code of that google page?

Can't reproduce. Can you give a better screenshot of the html code of that google page?

I only get "People also ask" which is properly blocked by the template.

Hello @opzch, thanks for opening an issue!
Currently, the only templates that have tested mobile support are youtube and nebula. It's very probable that google search returns a different markup that would need a whole different set of rules. For now, mobile support is best effort for most templates.
You'll notice the out rules currently start with #rso (which means an element of id="rso" in order to not match non-search pages on the same domain. We'll need a check the DOM higher than your screenshot, to confirm the #rso element is still on mobile.
I'll be out of town for two weeks and won't have the time to look into this, but happy to review and merge a PR if another contributor manages to update the rule.

Hey @opzch,
Thanks to @JohnyP36, the update has just been deployed! Your browser will get the update in less than 24h, but you can manually refresh your list right now to grab it!


support text builder

Provides initial support to allow processing nbsphinx nodes using a Sphinx text translator.

Thanks! Looking forward to further improvements!


Array SmartForm error messages don't have correct labels for nested items

Because of schema flattening, getLabel cannot find the correct name of nested labels in the error messages resulting in errors that look like this Field “memberInvites.0.firstName” is required..

Should
Error.properties = {
id: "errors.required"
path: "memberInvites.0.firstName"
}

have a path of firstName because of the schema flattening?

Hmm… maybe getLabel should only keep the last segment of any path then?

So keep the error path how it is, but getLabel should use a regular expression to get the last segment of the path?

You don't even need a regexp, you can just use split I think. But yes, I think that would be the easiest fix if I'm understanding the issue correctly.


ExternalItem with with a list property

how should I create an ExternalItem with a list property?
I defined the Schema property as type=PropertyType.StringCollection:
Property_(
name="collaborators",
type=PropertyType.StringCollection,
is_searchable=True,
is_retrievable=True,
),
how should this property be defined in the ExternalItem object?
        request_body = ExternalItem(
            id = repo["name"],
            acl=[
                Acl(
                    type=AclType.Everyone,
                    value=repo["name"],
                    access_type=AccessType.Grant
                )
            ],
            properties=Properties(
                additional_data={
                    "collaborators": ['aaa','bbb']
                }
            )

I think I need to use odata_type - but I don't see any documentation which data types are valid.
Could you please instruct which odata_type to use and to which object it should be added?

Looking at https://learn.microsoft.com/en-us/graph/api/externalconnectors-externalconnection-put-items?view=graph-rest-1.0&tabs=http, collections do require specifying the odata.type. I think this should get you going:
properties = Properties(
    additional_data = {
<EMAIL_ADDRESS>"Collection(String)",
        "collaborators": ['aaa','bbb']
    }
)

that works. thanks a lot!

just want to understand - what is the odata_type property used for? I was mixed up and didn't find any documentation on it

It's part of the OData specification - it's a way to specify the type of an object.
https://docs.oasis-open.org/odata/odata-json-format/v4.01/odata-json-format-v4.01.html#_Toc38457738


Feedback for Samsung page

It seems that almost all of these menu's have changed. I'm also having issues with apps sleeping again. So sorry. I went back to the page since it's been such a help last time.
I tried the Android 11 settings today. My current version of Android is 13.

I'm also on 13. More discussion here: https://github.com/urbandroid-team/dont-kill-my-app/issues/689
Did you trigger those 3 options and you're still having the problem?


feat(raiko-lib): unify protocol instance for on chain verification

Looks good!
sgx_verifier_address removed so potentially a breaking change @smtmfft.

it's not in API, could be little impact I think.


gitignore .env and replace with .env.sample

Although our .env file doesn't currently have sensitive values, it's best practice not to check in .env files.
I added .env to our gitignore, and replaced it with a sample file that future devs can copy values from to create their own .env file locally. I also added instructions to the README for copying values from .env.sample to .env locally.
The main reasons for not checking .env files into git are to avoid publishing sensitive information, and to avoid coupling the application with any specific environment.
sources for .env best practice:
https://stackoverflow.com/questions/42014558/why-dotenv-files-should-not-be-checked-in
https://www.freecodecamp.org/news/heres-how-you-can-actually-use-node-environment-variables-8fdf98f53a0a/#:~:text=env files allow you to,in there on different lines.&text=To read these values%2C there,the dotenv package from npm.

Going to merge this to get dev env working


center nav menu

thank you for your work.how i can center the nav menu i mean the logo stay in the left and the menu in the center?

To do that you have to modify the file partials/header.hbs and delete from line 33 to line 45.
After that you have to paste the following code after line 30:
{{#if @site.logo}}
  <a<EMAIL_ADDRESS>class="m-logo">
    <img<EMAIL_ADDRESS><EMAIL_ADDRESS>  </a>
{{else}}
  <a<EMAIL_ADDRESS>class="m-site-name in-desktop-menu">
<EMAIL_ADDRESS>  </a>
{{/if}}

Finally you have to head to the Code injection page and paste the following code into the Site Header block:
<style>
.m-menu .m-logo,
.m-menu .m-site-name.in-desktop-menu {
  display: none;
}

@media only screen and (min-width:48rem) {
  .m-menu .m-logo,
  .m-menu .m-site-name.in-desktop-menu {
    display: inline-block;
  }

  .m-menu .m-site-name.in-desktop-menu {
    font-size: 1.125rem;
  }
}
</style>

Great.
Thank you


[$500] Request money - Distance request money is not deleted

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.4-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to https://staging.new.expensify.com/
Tap fab
Tap request money---Distance
Enter start and finish waypoints
Tap next
Select a Workspace
Tap request
Tap distance request created to open distance request detail page
Tap receipt
Tap 3 dots on top right
Tap delete receipt

Expected Result:
When user taps delete receipt, the receipt must be deleted
Actual Result:
In distance request, user taps delete receipt but the receipt reappears again and not deleted
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/32adf2d8-9629-4ea0-af5c-afa1602beb82
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01968e0702cd968607
Upwork Job ID:<PHONE_NUMBER>547236352
Last Price Increase: 2023-11-27

This is a dupe of https://github.com/Expensify/App/issues/29684


updating animated examples

This should resolve #16.  Using static paths for the images pointing at the github repo makes contributions slightly harder but I wasn't sure if there was a reason. Local refs would certainly make things easier to validate in GitHub but might complicate documentation elsewhere?

Thanks Colin!


Fix cy.click() timeout error

supposed to fix #1108
we have found that timeout increasing can't solve this problem


JSON parsing differences between iOS and Android

JSON parsing from a web API is handled differently on iOS and Android. Apparently the problem occurs when an API returns JSON code but no "application/json" content type declaration. See short example demonstrating the problem (and workaround) at https://www.jasonbase.com/things/Wxzm.json

I tried this a couple of hours ago when it wasn't working and confirmed that the app was failing. Then I headed out for a bit, then came back and sat down to work on the problem.
By then, for some reason the problem was gone. It's now loading just fine and the app is loading fine.
The only difference I am seeing is how the coindesk api was returning the malformed JSON earlier, but not anymore.
I tried to replicate this by saving the JSON as a text file https://s3-us-west-2.amazonaws.com/fm.ethan.jason/price.txt but even this seems to be working.
My only guess is that the coindesk api was messing up something and returning some corrupt data that looked like JSON but actually wasn't.
Can you still replicate this?

Yes, I can still replicate the original error.
I created two Jason files to showcase the issue in more detail: one works on iOS but fails on Android (it displays {{JSON.parse($jason).bpi.USD.rate}}) (see https://www.jasonbase.com/things/rzD2.json); the other works on Android but fails on iOS (it displays a blank screen) (see https://www.jasonbase.com/things/53op.json).
However, the issue appears unrelated to the CoinDesk API. It seems that Android doesn't process the JSON.parse() method at all. I created another Jason file using a different API that works on iOS but fails on Android (it displays {{JSON.parse($jason)[0].title}} (see https://www.jasonbase.com/things/ERYn.json).


NTR sent to ENVO: material used in building a house

See:
https://github.com/EnvironmentOntology/envo/issues/229

Request terms added


Rhn has been deprecated for SubscriptionManager

All the rest of the warnings have been fixed save this one.
Today is deprecation warning fixing day.

@miq-bot add_label technical debt


Remove completely unused Symfony dependency

The Symfony composer depency is created in commit https://github.com/mailersend/mailersend-php/commit/e3d0be351f7877729fd113d15f72c5f4b50bf786
If you look at the complete source and to the changes in this commit there is no code reference to Symfony at all. So I removed the dependency.
I runned al tests, they work.

@PeterOcansey why did you add Symfony as a dependency?

This is required for upstream compatibility.

Which Symfony dependency is MailerSend using?
Upstream to what?


Enhancements to LLM Instance Gateway: Scheduling Logic, and Documentation Updates

This pull request includes updates to the documentation and significant enhancements to the request scheduling and handling logic in the pkg/ext-proc package. The most important changes involve refining the handling of response headers, improving the scheduling algorithms for better load balancing and resource utilization, and updating the README file to include new scheduling flowchart.
Documentation Updates:

pkg/README.md: Updated the deployment steps to include new configurations for LLM Service and LLMServerPool, and restructured the steps for deploying the gateway and ext-proc. Added a new section on the scheduling package and included a flowchart for the scheduling algorithm. [1] [2] [3]

Response Handling Enhancements:

pkg/ext-proc/handlers/response.go: Modified the HandleResponseHeaders method to include additional headers when a target pod is specified, and added a fallback response for cases where no target pod is provided. [1] [2]

Scheduling Algorithm Improvements:

pkg/ext-proc/scheduling/filter.go: Added new filter functions lowQueuingFilterFunc, loRAAffinityPredicate, and minLoRAPredicate to enhance the scheduling logic for better load balancing and resource utilization. [1] [2]
pkg/ext-proc/scheduling/scheduler.go: Introduced new queueing thresholds and updated the scheduling filters to incorporate the new filter functions. Added lowLatencyFilterModified to dynamically adjust scheduling based on queueing thresholds. [1] [2] [3]

It'll be easier to split the algo change and the user guide change (I think they are not coupled). And we can get the user guide update checked in much faster.
Did you run some benchmarks to see how the updated algo improve things?

This PR only has the filter related changes. The manifest updates are in a separate PR: https://github.com/kubernetes-sigs/llm-instance-gateway/pull/81

Yes, we have new benchmarks based on the updated scheduling logic shared internally

Yes, we have new benchmarks based on the updated scheduling logic shared internally

Can we share something external? This is an OSS repo, and anyone in the future that wants to follow the story of this PR should be able to have all the context

Yes, we have new benchmarks based on the updated scheduling logic shared internally

Can we share something external? This is an OSS repo, and anyone in the future that wants to follow the story of this PR should be able to have all the context

Yes, I absolutely agree. We should definitely have an external version of our benchmarking docs accompanying this PR, and the OSS repo in general. Lets discuss more.

Yes, we have new benchmarks based on the updated scheduling logic shared internally

Can we share something external? This is an OSS repo, and anyone in the future that wants to follow the story of this PR should be able to have all the context

Yes, I absolutely agree. We should definitely have an external version of our benchmarking docs accompanying this PR, and the OSS repo in general. Lets discuss more.

@kfswain @ahg-g created this issue to create external benchmarking doc: https://github.com/kubernetes-sigs/llm-instance-gateway/issues/88

/label tide/merge-method-squash

/lgtm
Thanks for the deep dives and improving the algo! Unfortunately the decision tree is getting more complex and hopefully we can find ways to simply it in the future.
One potential is to at least simplify the non-LoRA mode. We can add a flag to indicate whether LoRA is enabled or not. Another reason to do this is that vLLM doesn't expose LoRA metrics when LoRA is not enabled, which resulted in noisy error logs.

/approve
/lgtm


Ability to make the normalization of images optional

When working with Prismic repos that have large media libraries, booting can sometimes take upwards of ten minutes or more, depending on your internet connection. This is due to the process of normalizing images during boot, in which case images are downloaded to the cache for use with gatsby-image-sharp and anything else that lets your process images.
Because this process can take a very long time, and there's no feedback when this is happening, this sometimes makes it seem like it gets stuck on the source and transform nodes part of booting the project. While this is happening, in one of the projects I'm working on, it downloads what seems like GBs of image data.
In cases where no local image processing is taking place, this option allows you to turn off normalization and local access to images. This is especially useful in cases where we can rely on prismic's image processing to link remotely to images.
In testing, instead waiting minutes to finish booting the project, a fresh boot from a cleared cache will take ~10s to finish the same task.

What do you think of using a function rather than a boolean? Then you could set which images to download and which to skip. Right now it looks to disable images altogether.
Something like…
{
  shouldNormalizeImage: ({ node, key, value }) => {
    switch (node.type) {
      case 'PrismicPage': return true
      default: return false
    }
  }
}

With the node object available, you could fine-tune which images are downloaded. And of course it could always return false.
{
  shouldNormalizeImage: () => false,
}

Alternatively, @daydream05 suggests using gatsby-source-contentful's method for handling images (https://github.com/angeloashmore/gatsby-source-prismic/issues/68).
I believe it uses Contentful's image processing via URL params to create the images on the fly. Not 100% if it downloads it locally or if Contentful's URLs end up in your build.

Looking closer, I don't think Prismic has dynamic image sizes set by the URL. It looks to be limited to the dimensions set on the custom type. So not really a viable solution.

@angeloashmore making it a function is a fantastic idea 👍 Also, yeah I'm pretty envious of Contentful's dynamic image service, kinda like Imgix.
PR coming up!

@angeloashmore Tested this update with different content types and it seems to work as expected. The only difference is that node type is the raw Prismic type (i.e. page not PrismicPage). Not sure if that's a big deal, but works for me 👍

Looks good to me!

Published as v2.2.0!


plot_cross_section color map error

The following command errors out on color map issue.
-bash-4.2$ ./plot_cross_section.py -b 34.0,-118.0 -u 35.0,-117.0 -h 100 -v 10 -d vs -c cvms -a d -s 0 -e 500
Using parameters:
lat1 = 34.0
data_type = vs
lat2 = 35.0
cvm_selected = cvms
ending_depth = 500
horizontal_spacing = 100
color = d
lon1 = -118.0
vertical_spacing = 10
lon2 = -117.0
starting_depth = 0
Retrieving data. Please wait...
Traceback (most recent call last):
File "./plot_cross_section.py", line 204, in
d.plot(data_type)
File "/home/scec-00/maechlin/t1/utilities/pycvm/pycvm/cross_section.py", line 201, in plot
img = plt.imshow(datapoints, cmap=colormap, norm=norm)
UnboundLocalError: local variable 'colormap' referenced before assignment

This was resolved with a modification to pycvm/cross_section.py which initialized the colormap before use.


Fix severe vulnuerability - malicious package flatmap-sream removed

As I was filling out another issue (#10) outlining a version upgrade from Jade to Pug, I realized that this project's dependencies had not been updated in some time.
There is a well known vulnerability in the package flatmap-stream, which was a subdependency of nodemon up until version 1.18.7. You can review the conversation regarding updating it on GitHub here.
This pull request simply upgrades the nodemon package to the most recent version (1.18.7), which no longer uses the malicious package. All previous functionality remains intact.
Note that I'm also happy to submit an additional pr with my upgrade from #10, and any other needed packages.

Closed due to committing package.lock unnecessarily. Will submit separate PR just updating package.json.


CMake Build Fails @ FlashMcu (Linux Only?)

When attempting to build line 18 in the flash_mcu.cmake throws the following error:
PS C:\repo\mvp-hover-demo\hover-test\build> cmake .. -DMCU=IMXRT1062_T41
-- Selecting Windows SDK version 10.0.19041.0 to target Windows 10.0.19044.
-- The C compiler identification is MSVC 19.30.30706.0
-- The CXX compiler identification is MSVC 19.30.30706.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.30.30705/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.30.30705/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
Configuring IMXRT1062 T4.1 build.
Configuring IMXRT1062 T4.1 build.
CMake Error at cmake/flash_mcu.cmake:18 (file):
file failed to open for reading (No such file or directory):
/proc/version

Call Stack (most recent call first):
CMakeLists.txt:58 (FlashMcu)
-- Configuring incomplete, errors occurred!

My assumption was this would be built on Linux or in WSL. That line giving the error is checking to see whether it's a Linux or WSL environment to use the correct version of the Teensy CLI Uploader. I would guess that you can remove lines 18 - 21 and 26 - 31 of the flash_mcu.cmake to only use the Windows version of the uploader. I haven't tried building in a Windows environment, outside of using WSL, so I would appreciate any suggestions that you might have.


Be able to use test runner in SDK repo

(Capturing an issue that came up in today's analyzer X-team sync meeting)
It would be nice if VSCode and IntelliJ could run tests in the SDK repo using the test runner (rather than just running the test file as an ordinary Dart program).  This would allow SDK developers to take advantage of the UI affordances in VSCode and IntelliJ for examining test failures, as well as take advantage of the test runner's parallelism on multi-core machines.

If I may add something. All my analyzer tests with IntelliJ are with dart.exe run test, as hinted in #48780.


Hover circle misplaced on a radio button when zooming to 1000% level

Affected versions

81.0a1 (2020-08-18)

Affected platforms

Windows 10 64bit
Ubuntu 18.04 64bit
macOS 10.15

Steps to reproduce
pdfjs.renderInteractiveForms - true

Launch Firefox
Access a PDF form with fillable content (eg: https://pdf.ninja/interactiveform_enabled.pdf)
Scroll down to the radio buttons
Zoom to level 1000%
Hover over the radio button

Expected result

Hover circle is within the borders of the radio button

Actual result

Hover circle is misplaced a bit when zoomed in

Additional notes

S4 as suggested severity, only noticeable when zooming to 1000%.
Here is a .gif showing the issue:

Note that this also applies to checkboxes as the PR above suggests.

I am interested in fixing this.
The problem is the rounding here.
When the canvas has more rounded pixels than the canvas style or vice versa, misalignment occurs.
Why is it necessary for the canvas and canvas style to be multiples of the approximation of
fractions and not just an integer?
Rounding to just an integer fixes the problem.

The problem is the rounding here.

Unfortunately I believe that that's most likely a red herring, since the thing that becomes slightly wrong when CSS-only zooming is being used is probably the annotationLayer-positioning and not necessarily the canvas itself (since it works just fine if "regular" zooming is force-enabled, even at very large values).

Why is it necessary for the canvas and canvas style to be multiples of the approximation of fractions and not just an integer?

Please refer to PR #6552; basically it's necessary to avoid issues/glitches on HiDPI screens. Hence removing that would likely re-introduce other issues, and again it's probably not the actual underlying issue here unfortunately.

How do you force-enable regular zooming?

How do you force-enable regular zooming?

You'd have to increase the maxCanvasPixels option, however that's probably not entirely relevant here (since this issue just concerns the CSS-only zooming mode).
Thinking about this some more, I suppose that another possible cause of the bug could be that the PDFPageView.cssTransform-method in particular isn't placing the canvas entirely correct (e.g. not handling fractional widths/heights correctly). However, in order to fix this issue I think that the very first step is to figure out exactly why the canvas/annotationLayer positions differ specifically when CSS-only zooming is being used.

When you open the file at 600% zoom, before applying any zoom, don't you see a misalignment?


Empty object type assignable to type with index signature

TypeScript Version:  2.9.0-dev.20180330
Search Terms:  Empty object type, index signature
Code
let a: {} = '123';
let b: { [key: string]: number } = { value: 1 };

b = a;

Expected behavior: Type Error on 'b = a'. It is possible to assign almost anything to value of Empty object type. So it should not be possible to assign value of Empty object type to value of more specific type.
Actual behavior: Compiles without errors
Playground Link: http://www.typescriptlang.org/play/index.html#src=let a%3A {} %3D '123'%3B
let b%3A { [key%3A string]%3A number } %3D { value%3A 1 }%3B

b %3D a%3B

this is working as intended/designed. On the first line {} is the top type, so anything is assignable to it. On the second line, we have made some affordances to allow for a type with only an index signature to behave like any/ map.

Thank you for explanation. Can you clarify what do you mean by 'any / map' type?


Revert "Tag useUtilityProcess for exp"

Reverts microsoft/vscode#158811

According to @lramos15 this is not causing harm.


Race in integration tests

Looks like #1611 introduced a new data race in the integration tests:
    --- FAIL: TestDeleteServiceInstance/deprovision_instance_after_in_progress_provision (10.90s)
    	framework.go:76: Starting server on port: 4184
    	controller_test.go:479: controller start
    	controller_test.go:491: informers start
    	framework.go:72: Shutting down server on port: 4184
    	testing.go:699: race detected during execution of test
	testing.go:699: race detected during execution of test

Looking into this now.

#1626 commented out the test.

There is an issue with the way the controllerTest framework is written. When controllerTest.Run is called, it runs the setup func from the testcase, or otherwise previously defined. This func is run AFTER the controllers have been setup and are running. I am not sure if this is what is being hit, but I've hit it while trying to modify the cases and controllers.

From what I can see it's mostly due to prependActions being called after the controllers have been started. We can't modify what the controller is to return after we start it. Right now we're very aggressively tied to the internals of the reconcilliation loops as well as the watchiness of the controller interaction with the various clients.


custom model attributes in selects

by executing the query before plucking model attributes can be used as 'name' property
Fixes #
Select::make('category.parent_id')
                ->title('Categoria padre')
                ->fromQuery(Category::byCurrentShop()->orderBy('path'), 'indented_name') // indented_name is magic attribute of Category (getIndentedNameAttribute),

Proposed Changes

Execute the query and perform the pluck on models

Hi @G3z . Thanks.
Can you add a test to be sure that it will not be broken in the future?

I did this, but how can I test that 'name' can be a runtime attribute ?
public function testAttributesCanBeUsed()
    {

        $select = Select::make('choice')
            ->empty('empty')
            ->fromQuery(Role::orderBy('id'), 'name');

        $view = self::minifyRenderField($select);

        foreach ($this->roles as $role) {
            $option = $this->stringOption($role->name, $role->id);
            $this->assertStringContainsString($option, $view);
        }

        $option = $this->stringOption('empty');
        $this->assertStringContainsString($option, $view);
    }

also, maybe a function can be cool
$select = Select::make('choice')
            ->empty('empty')
            ->fromQuery(Role::orderBy('id'), function($row){
                  return [$row->key => $row->title];
            });

I think we can do the same thing as checking the calculated field of the Relation field :
$select = Select::make('choice')
            ->empty('empty')
            ->fromQuery(EmptyUserModel::orderBy('id'), 'full');


LG 32un880-B

It sadly does not work for the LG Ergo 32un880-B.
It would be gorgeous if someone could fix it 🔢

It will not work if you updated to mac os 11.3,
it was working fine before , I have the same screen

Ok Thanks I'll update it in a few minutes and give a feedback.
If it's still not working it may is because I have a M1 Macbook.
The error inside the app is just because of the monitor so I thing the update could fix it

Yes, it's due to M1 not being supported yet, see #323.


channels/server: Add APIs for DVCs to get notified of channel id

This is basically https://github.com/FreeRDP/FreeRDP/commit/3337810520d7062fe8d477b12fa27d65544ac2cc, but for the other dynamic channel.
I tested the changes for the DISP and RDPGFX channel with g-r-d. For the other channels, I don't have the hook up ready yet, so they are only compile-tested.

@freerdp-bot test


Feature Request: Ripley door prying

Description
the Ripley APLU has no way of getting through firelocks or unpowered doors.
Why?
This is really, really annoying. TG allows hydraulic clamps to pry unpowered doors/firelocks, so logically we should have this too.
How?
As far as I know, this is possible via adding the crowbar prying component to the hydraulic clamp, but mech code is bad so this might not work.

just give it to the mech itself how is a big metal powered mech not able to pry a door when a human can

When has anyone ever used a ripley to move crates in a spaced area? I understand the point of "if human can robot can" but it removes the one restriction for ripleys, wouldn't that make them more overpowered than they already are?

When has anyone ever used a ripley to move crates in a spaced area? I understand the point of "if human can robot can" but it removes the one restriction for ripleys, wouldn't that make them more overpowered than they already are?

No not really? It's really expensive to craft Ripleys as it is, and this would just be a QoL thing not a buff.

When has anyone ever used a ripley to move crates in a spaced area?

me

wouldn't that make them more overpowered than they already are?

ripleys are not overpowered what are you smoking


Integration tests are no longer executed

It looks like the integration tests are no longer executed.

Last change on .github was removing my name  from the Codeowners file idk whether this is related

No, your change is not related at all.
The integration tests were super flaky and could not run unattended (always something fails).
In #475, I've enhanced a bit to run each test case individually and allow rerunning of the failed ones only (before, all tests ran in the same github action build, and if one failed, we needed to rerun everything and we needed a couple of runs to have all of them green at the same time).
I've also limited them to the PRs only since now we have a PR-based release so they will always run with new features and new releases.
We have a plan to review the integration tests to find the best way to have them stable and with a short feedback loop.

I'm closing this issue since it's intended behavior.


ER Diagramm NPE

After exporting the diagram and then opening it. On version Версия <IP_ADDRESS>203060510 DBeaver Community - taked error.
I can nor fix this problem. Please, help me.
eclipse.buildId=unknown
java.version=11.0.12
java.vendor=Eclipse Foundation
BootLoader constants: OS=win32, ARCH=x86_64, WS=win32, NL=ru
Command-line arguments:  -os win32 -ws win32 -arch x86_64
org.jkiss.dbeaver.model
Ошибка
Thu Apr 07 10:36:06 MSK 2022
NullPointerException
java.lang.NullPointerException
at org.jkiss.dbeaver.erd.ui.editor.ERDEditorPart.getDiagram(ERDEditorPart.java:400)
at org.jkiss.dbeaver.erd.ui.editor.ERDEditorEmbedded.loadFromDatabase(ERDEditorEmbedded.java:212)
at org.jkiss.dbeaver.erd.ui.editor.ERDEditorEmbedded$1.evaluate(ERDEditorEmbedded.java:171)
at org.jkiss.dbeaver.erd.ui.editor.ERDEditorEmbedded$1.evaluate(ERDEditorEmbedded.java:1)
at org.jkiss.dbeaver.ui.LoadingJob.run(LoadingJob.java:88)
at org.jkiss.dbeaver.ui.LoadingJob.run(LoadingJob.java:72)
at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)

@Bouncer77 , unfortunately, cannot reproduce it. Could you provide a video with the issue and more clear steps to reproduce it?

Connected to https://github.com/dbeaver/dbeaver/issues/15766

Closed as duplicate of https://github.com/dbeaver/dbeaver/issues/15766


Switch to go mod

Isolating go mod switch from k8s bump.

/assign @soltysh
Splitting my k8s bump into two - this just covers the go mod switch.
/cc @deads2k

/test e2e-aws-builds
/test e2e-aws


Added a tutorial folder along with a tutorial in tools/materials/

This pull request adds a tutorial folder that initially contains a detailed solution to a classification problem.

the tutorials shouldn't be .ipynb but .py because notebooks aren't visible directly off of GitHub.

I will change it to .py


Add documentation on cluster issues

There are a limited set of issues that can be presented in the UI, for example:

Servers that should be removed from the cluster
Metadata conflicts
Redundancy issues
Zombie servers

...and a few more. It would be useful to describe these different cases, and the consequences of resolving them (and would help simplify some error messages if we had a public resource for users to refer to).

While we don't have an explicit "cluster issues" document, some of these things are covered in other places in the docs—and I'm not sure how much of what's here now still applies given that this is two and a half years old. If this is still relevant, please re-open a modern version of this ticket.


interval stuck on 1D

doesn't seem the interval prop is persisting when passed in, all charts load as 1D

Hey, thanks for reporting!
The reason this happened is because interval and range can not both be defined. I had default values set for both. Fixed this in the new release (v1.0.4). When a range is added as a prop, the chart will load with the given range instead of the default interval.


Added optional parameter to the set method for a custom update event

This PR (delete as applicable)

Updates the Documentation
Adds a new feature

Describe the changes below:
This PR adds an optional parameter flag to the set method which, on true, will emit a custom event for that key.

I think it would be simpler to emit this event, without an extra boolean parameter.

I agree.
This PR was in response to a conversation Rich and I had via Slack.  The
optional parameter was suggested by Rich.  He had valid reasoning for it at
the time, but for the life of me I can't remember.  The discussion was
deleted by Slack's message limit.
On Mon, Jun 4, 2018 at 12:53 AM Chris Andrew<EMAIL_ADDRESS>wrote:

I think it would be simpler to emit this event, without an extra boolean
parameter.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/photonstorm/phaser/pull/3585#issuecomment-394233765,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJeqIuK2OBC_p3xkcYU7knP7rtVjhy38ks5t5L0-gaJpZM4TZE67
.

--
My regards,
Travis Oneal

Sorry for sitting on this one for so long and then not merging it :) but I've done some serious overhauls to the Data Manager tonight, which provides this feature via another route, and a lot more besides. So, thank you for the idea at least :)


SocketStream read error - Socket SO_ERROR

This issue is for a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] feature request
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

I have same issue when app is running in background.  IoTHubClient_Destroy(iotHubClientHandle) is not cleaning everything after execution.

Minimal steps to reproduce:

The flow is working fine when app is in foreground (C2D & D2C) Messages, FileUpload to Blob and updating DeviceTwin. When we move app to background it works perfectly until after 15 - 20 min. Then the "tlsio_appleios_dowork" initiate exception on CFReadStreamHasBytesAvailable. We are properly opening and closing connection by waking up the app over push notification to do the work. Interesting is that app works perfectly and then after around 20 min, crash happens.

Expected/desired behavior

Connection should be created stays alive, and the iOS device should be able to send another c2d and d2c message. After issuing "IoTHubClient_Destroy(iotHubClientHandle)" SDK should clean all the connections and memory, so the next time connection can be established without errors.

OS and Version?

macOS 10.14.6
iPhone X, iOS13
XCode 11, Swift 4.2

Any log messages given by the failure

Crashed: Thread
0  CoreFoundation        ----     0x1a9e803b8   ----        CFReadStreamHasBytesAvailable + 48
1  AzureIoTUtility       ----         0x104f409e4    ----        tlsio_appleios_dowork + 524
2  AzureIoTuMqtt        ----        0x104f7b210      ----      mqtt_client_dowork + 40
3  AzureIoTHubClient     ----   0x104e7d9f4     ----        IoTHubTransport_MQTT_Common_DoWork + 2472
4  AzureIoTHubClient     ----   0x104e651d8    ----        IoTHubClient_LL_DoWork + 412
5  AzureIoTHubClient     ----   0x104e61b74     ----       ScheduleWork_Thread + 60
6  AzureIoTUtility      ----         0x104f3fcf8      ----        ThreadWrapper + 16
7  libsystem_pthread.dylib  ----      0x1a9c091ec ----    _pthread_start + 124
8  libsystem_pthread.dylib   ----     0x1a9c0caec ----    thread_start + 8

Debug error

"DigitalTwin send REPORTED: CONFIRMATION_OK"
"DigitalTwin send REPORTED: CONFIRMATION_OK"
2019-10-03 11:05:26.179856+0200 WaaS[5692:581567] SocketStream read error [0x280304b40]: 1 57
Info: Communications error while reading
2019-10-03 11:05:26.192765+0200 WaaS[5692:581456] [] nw_socket_handle_socket_event [C34.1:1] Socket SO_ERROR [9: Bad file descriptor]
2019-10-03 11:05:26.195259+0200 WaaS[5692:581456] [] nw_socket_get_input_frames [C34.1:1] recvmsg(fd 18, 1024 bytes) [57: Socket is not connected]

Some time it throws this error:

2019-08-28 11:50:45.040906+0200 Generic[2511:721237] [] nw_socket_handle_socket_event [C11.1:1] Socket SO_ERROR [9: Bad file descriptor]
"Expire timer fired"
2019-08-28 11:50:45.041235+0200 Generic[2511:721237] [] nw_socket_get_input_frames [C11.1:1] recvmsg(fd 17, 1024 bytes) [57: Socket is not connected]
2019-08-28 11:50:45.045218+0200 Generic[2511:721243] SocketStream read error [0x280ff9d40]: 1 57
Info: Communications error while reading

Mention any other details that might be useful

I noticed that when doing the destroy the client handle and clean all the connection it still something stay in memory so the crash is produce when trying to read. The code I'm using to clean all the connections before app is Terminated/Suspended by iOS is:

    public func removeIoTHubClientHandle() {
        if self.iotHubClientHandle != nil {
            debugPrint("REMOVE IoTHub Client Handle: \(String(describing: iotHubClientHandle))")
            IoTHubClient_Destroy(iotHubClientHandle)
            iotHubClientHandle = nil
            iotConnectionStatus = false
            return
        }
    }

@SashaMilic Could you please provide more information about the document link that you used before seeing this error? This will help us direct the issue to the right team.

@SashaMilic It would be great if you can provide us the doc link of the steps you followed before seeing this error. This will help us to direct this issue to the right team.

@SashaMilic We will now proceed to close this issue. Please feel free to raise an issue against the document you used from the feedback section of the page.