[python] added option to install python-package with mpi version

Refer to #1019, #985.

python version doesn't support protobuf yet

@wxchan I see you've opened PR for it. So, wait it'll be merged.

Will reopen after protobuf revert.

I suppose we can add more options to the setup.py, e.g.
-DOpenCL_LIBRARY=/usr/local/cuda/lib64/libOpenCL.so
-OpenCL_INCLUDE_DIR=/usr/local/cuda/include/
So users will be able to install GPU-version for CUDA even from PyPI with --install-option="--OpenCL_INCLUDE_DIR=/usr/local/cuda/include/" and without the need to compile the library manually.
But unfortunately I can't recollect all possible options for cmake that can be added as options to setup.py. Maybe someone can easily list them here?

Here's a list of what I've found for gpu-version:

-DBoost_ROOT
-DBoost_DIR
-DBoost_INCLUDE_DIR
-DBoost_LIBRARYDIR
-DOpenCL_INCLUDE_DIR
-DOpenCL_LIBRARY

Maybe something should be added or removed?

ping @huanzhang12 for confirmation

@guolinke May mpi-version require any special CMake options?

@StrikerRUS I think it only needs the USE_MPI .

@guolinke OK, then waiting for @huanzhang12's answer about gpu-version.

@StrikerRUS All variables used by CMake for finding Boost and OpenC are listed here:
https://cmake.org/cmake/help/v3.8/module/FindBoost.html
https://cmake.org/cmake/help/v3.8/module/FindOpenCL.html
Usually these variables can be auto detected, and only a few of them need to be set manually in practice. I think you found most variables. But there are subtle differences between Boost_INCLUDE_DIR and Boost_INCLUDEDIR. I guess ideally we probably want to use Boost_INCLUDEDIR, but both of them should work (similarly, Boost_LIBRARYDIR and DBoost_LIBRARY_DIR)

@huanzhang12 Yeah, I've found them from links you've provided and from issues here. Can we set both options if user provides one of them? Or is it better to set more general option which tells CMake search this folder (without underscore)?

@StrikerRUS , Cmake will automatically finding module in path and libpath, I guess user could have  customization on these variables?

@chivee Do you mean environmental variables?
Sure, user can do it, but since we are offering to user do it through CMake options in docs (like #1037), I think it's good point to create the possibility do the same things with Python-package in one line and without using precompiled library.
E.g. pip install lightgbm --install-option=--gpu --install-option="--OpenCL_INCLUDE_DIR=/usr/local/cuda/include/" --install-option="--Boost_LIBRARYDIR=/path/to/boost/" instead of modifying PATH.

@StrikerRUS ,Got it, I'm suggesting to set general option (if capable) for Cmake to search rather than set *INCLUDEDIR and *LIBRARYDIR, and it should be easy for user to understand

@chivee Thanks for the explanation. Unfortunately, I'm not very familiar with CMake. Will your described method work? And how to pass both paths (OpenCL and Boost) at once in such case?

ping @chivee

@StrikerRUS ， sorry for the later response.
for this case. we can simply alter the setup.py following as :
https://github.com/Microsoft/LightGBM/blob/f42e6c36b47a3b6366244df2f2771eba8b2d32a2/python-package/setup.py#L88
to append the "--OpenCL_INCLUDE_DIR and --Boost_LIBRARYDIR.

@chivee Yeah, that's what I wanted to implement. But your suggestion was about one single option:

I'm suggesting to set general option (if capable) for Cmake to search rather than set *INCLUDEDIR and *LIBRARYDIR, and it should be easy for user to understand

@StrikerRUS , Yes, it can be but more complicated
you can alter the option PATHS refer to this
and the documentation:

Search paths specified by the PATHS option. These are typically hard-coded guesses.

@StrikerRUS it seems osx test is broken

@chivee OK, I think this PR is ready.
@guolinke They've broken something again: https://twitter.com/travisci/status/935677242230956032

@huanzhang12 Have you any concerns about the GPU part?

ping @huanzhang12

@StrikerRUS The check-docs seems fail due to a warning.

@guolinke I've created PR #1118.

@huanzhang12 Thank you very much for your attentive review! I'll try to deal with confusing errors. But unfortunately we can't print out raw error from CMake or any other program due to this issue https://github.com/Microsoft/LightGBM/issues/874 (to be more precise, this's the reason why we use "silent" call).

@StrikerRUS yes you are right... we can probably save the error message to a file like cmake.stderr and ask users to check that file if cmake fails?

@huanzhang12 Yeah, good point! I thought about it. But is it good decision to leave any "rubbish" after installation?

@StrikerRUS I think it is okay as long as we produce an extra file containing error messages only when the build fails. If there is no failure, no extra files will be produced.

@huanzhang12 Then we can write into a file any output, but delete this file only in case of everything is OK.

@StrikerRUS is this ready to merge?

@guolinke Not yet. I need to rework the exception handling mechanism but have no time to do it these days.

Fixed #1121.

@huanzhang12 Please review new changes.
I'm not sure about the place where to save the installation log file. It seems that current directory isn't a good place, because in case of installation via pip this directory is automatically removed at Cleaning up stage. I nay other suggestions better than $HOME directory?

ping @huanzhang12

@StrikerRUS I think $HOME directory should work fine, as long as we always clean it up after installation.
Another possible option is to use a temporary directory returned by tempfile.gettempdir() (which will be translated to /tmp on Linux and something like c:\temp on Windows)

@huanzhang12 Thanks for your re-review!
On Windows gettempdir() returns C:\\Users\\username\\AppData\\Local\\Temp which is hidden folder with default settings of Explorer, so I think $HOME directory is OK, because user has an easy access to it in case he need to see installation log.

@guolinke Please look through this PR and merge it if have no complaints.


[Fix-10795][plugin] sqoop task query columns bug

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

fix: #10795

Codecov Report

Merging #10776 (f7a5c92) into dev (35a10d0) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #10776      +/-   ##
============================================
- Coverage     40.64%   40.63%   -0.01%
+ Complexity     4818     4817       -1
============================================
  Files           902      903       +1
  Lines         36195    36198       +3
  Branches       3994     3989       -5
============================================
- Hits          14710    14708       -2
- Misses        20022    20027       +5
  Partials       1463     1463

Impacted Files
Coverage Δ

...inscheduler/plugin/task/sqoop/SqoopColumnType.java
0.00% <0.00%> (ø)

.../sqoop/generator/sources/MySQLSourceGenerator.java
0.00% <0.00%> (ø)

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.60% <0.00%> (-0.30%)
:arrow_down:

...eduler/api/service/impl/DataSourceServiceImpl.java
29.49% <0.00%> (ø)

...r/server/master/runner/MasterSchedulerService.java
0.00% <0.00%> (ø)

.../dolphinscheduler/plugin/task/datax/DataxTask.java
0.00% <0.00%> (ø)

...cheduler/plugin/task/api/ShellCommandExecutor.java
0.00% <0.00%> (ø)

...er/api/service/impl/TaskDefinitionServiceImpl.java
24.01% <0.00%> (+0.05%)
:arrow_up:

...hinscheduler/plugin/alert/wechat/WeChatSender.java
29.78% <0.00%> (+0.20%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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

Hi @CallMeKingsley97 , please associate the issue.

@SbloodyS  hi, there is no issue for this bug, Do I need to submit an issue?

@SbloodyS hi, there is no issue for this bug, Do I need to submit an issue?

Yes. To target specific issue from PR, we recommend use keyword fix: #issue_id or close: #issue_id or closes: #issue_id in your PR describe(not title, just desc). It would not only connect issue to PR but also close issue automatically when PR is be closed.

hello, I associated the issue.

hello, I associated the issue.

I'll review this pr later. Thanks.

@CallMeKingsley97 should we also need to make some change of our frontend code?

Yep, when user change the Column Type from Some Columns to All Columns , frontend should not transmit the data of Some Columns to the backend.   more infomation you can see the issue https://github.com/apache/dolphinscheduler/issues/10795

please check the fail CI, I found that the checkstyle fail. @CallMeKingsley97

please check the fail CI, I found that the checkstyle fail. @CallMeKingsley97

Is that ok? 😂

@CallMeKingsley97 I found no modification of the front end part, so this is a full pr, can this issue be fixed?

@CallMeKingsley97 I found no modification of the front end part, so this is a full pr, can this issue be fixed?

ok

@CallMeKingsley97 May I ask whether there are any updates for this PR?

@CallMeKingsley97 May I ask whether there are any updates for this PR?
Completed！

LGTM

Can this PR be merged now？🤭

Please run mvn spotless:apply to fix format error. @CallMeKingsley97

Please run mvn spotless:apply to fix format error. @CallMeKingsley97

Done


[network] Add integration tests for gossip-based discovery

Motivation
This PR concern the implementation of integration test for the communication between peers based on the discovery direct send protocol. This will help to avoid bugs which cannot be prevented from the unit tests.
Closes: #1771
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

Unfortunately I'm not familiar enough with the area to go beyond what support I have given. Perhaps one of the other folks I have tagged can help out.

:umbrella: The latest upstream changes (presumably e8c6d063731026f6f5158c304570360c3c513b4a) made this pull request unmergeable. Please resolve the merge conflicts.

@nikakis : We've made some changes to the network module which require you to rebase your changes --  you can look at other protocol tests for pointers. I'll be happy to help you land this PR. Thanks for taking this up.

Great, I will do this asap. Apologies for my late response.

@bothra90 If am not wrong the integration test introduced with this PR has already covered from this commit 58dce3b065a1e6bbf06da47a3f8d009f1382ddca, probably this PR can close.

@nikakis : The commit you mentioned tests only the initialization part of the discovery module where it dials all the seed peers. It does not test the actual gossip protocol like your commit does. So it will still be valuable to land this.

@bothra90 ok great, now i understand, i will do it asap. Thanks !!

Feel free to reopen if you'd like to continue.


Remove content-encoding header

Otherwise Chrome thinks content is gzipped, when it's not :/

*Do this only for gzip


LeastSquares - very slow when more than 6 features

Hi,  when using LeastSquares things are fine up to 5 or 6 features, but as soon as I add a 7th or 8th everything slows to a crawl.  Is there anything I can do to remedy this?  Is anyone else seeing the same issue?

I will check this with Blackfire and let you know. Maybe there is some space for optimization.

@exantedata Can you provide more details about your data? I have just tested a set of 17k samples. with 15 features. Here are the results:
Wall Time     1.33s
Memory       59.3MB

Dataset: http://archive.ics.uci.edu/ml/datasets/Bike+Sharing+Dataset
And graph from blackfire, as you can see Matrix::multiply is most time consuming function but for now it is hard to optimize.

One of the options would be to use threads, but this will require some extensions, because php has nothing built-in.

Thanks for looking into this!
So our data isn't all that complex.  All features are decimal values.  And the sample size is usually 260 points.  We do rolling regressions, so its a large dataset with a small 260period window.  I should have mentioned my speed complaint was about doing about 500 of these calculations....
5 features is very quick (5 seconds), 6 is acceptably quick (30seconds), 7+ is not worth waiting for the results (over 3 minutes).
Yeah I was looking through everything late Friday and found that multiply function is the biggest culprit however I too couldn't really think of a way to optimize.
Thanks again for looking into, appreciate the attention!

@akondas Can you share your code for the test?
I've wondered and tested LeastSquares with the dataset you provided. Strangely, it just took 0,82 sec to complete the training operation.
Here is my code, please let me know in case I am doing something wrong:
use Phpml\Regression\LeastSquares;
use Phpml\Dataset\CsvDataset;

$dataset = new CsvDataset('hour.csv', 16, true);
$regression = new LeastSquares();
$start = microtime(true);
// removing non-numeric and/or irrelevant columns
// which reduces number of columns to 12
$dataset->removeColumns([0, 1, 14,15]);
$regression->train($dataset->getSamples(), $dataset->getTargets());
echo (microtime(true)- $start) . ' seconds';

P.S: I reduced number of columns from 16 to 12. But FYI, the speed is more or less the same even with 16 columns.

By the way, #255 may be helpful for this issue.

Here is my test with this dataset: http://archive.ics.uci.edu/ml/datasets/Bike+Sharing+Dataset
use Phpml\Dataset\CsvDataset;
use Phpml\Dataset\Demo\GlassDataset;
use Phpml\Dataset\Demo\WineDataset;
use Phpml\Regression\LeastSquares;

require 'vendor/autoload.php';

$dataset = new CsvDataset(__DIR__.'/hour.csv', 16);
$dataset->removeColumns([0,1]);

$reg = new LeastSquares();
$reg->train($dataset->getSamples(), $dataset->getTargets());

For speed and memory usage: blackfire run (https://blackfire.io)
So it is very strange to me that 7 feature kill performance. @exantedata could you give as some data example (you can send it in private message if can't be made public).

Hi Akondas and Exante,
First, sorry about my bad english...
Currently I was both learning and developing some applications using python, keras, tensorflow and numpy. To better develop my learning, I decided to understand the operation of these libraries and implement them in PHP (for a challenge of its own).
The need for a fast library of array calculations for PHP is a key piece for the development of intelligence frameworks, and probably the only barrier separating us from intelligent PHP websites.
Considering this, I have developed a library for matrix multiplication that uses a C backend and basically uses the same function names and Numpy parameters. The results were reasonably good.
While developing an AI framework, I was very happy to find a project like yours, and for this reason I am separating the array library from the AI framework. In this way, PHP-ML can also implement the PHPSci Matrix. I will also use my efforts, if necessary, to create a PHP-ML fork with implementation using my library.
At the moment I am doing the separation and compilation of the library, this week I already return with the implementation. Below is an example of the performance of element-wise add of 2 square matrices of shape [iter,iter]:
Iterations 1000 to 5000. 1000 step:
PHP IjK:
Iter 1000 [1000,1000]: 0.10760498046875
Iter 2000 [2000,2000]: 0.4336831569671630859375
Iter 3000 [3000,3000]: 1.07429599761962890625
Iter 4000 [4000,4000]: 1.85231685638427734375
Iter 5000 [5000,5000]: 3.6177790164947509765625
Total: 7.085649967193603515625
PHPSci-Matrix with EXT
Iter 1000 [1000,1000]: 0.020347118377685546875
Iter 2000 [2000,2000]: 0.113109111785888671875
Iter 3000 [3000,3000]: 0.316626071929931640625
Iter 4000 [4000,4000]: 0.51078891754150390625
Iter 5000 [5000,5000]: 1.60066699981689453125
Total: 2.5615179538726806640625
The library will also work without the extension (using pure PHP).
Soon we get in touch.

4X better does not seem like much, but this is only a first step. Initially, I had to convert PHP arrays, which are basically Z_VAL internally, into linear vectors of doubles in C, and to return the result, I need to rebuild the vector of doubles in a new PHP array (fors and more fors).
After the first version, I'll try to find more efficient ways like multi-threading and even OpenCV.

Code used in example above:
`use PHPSci\Matrix\Matrix;
$min = 1000;
$max = 6000;
$step = 1000;
function simple_add($x, $y)
{
$n_array = [];
foreach($x as $xk => $xv) {
foreach($xv as $xy => $v) {
$n_array[$xk][$xy] = $v + $y[$xk][$xy];
}
}
return $n_array;
}
$total = 0;
for($i=$min;$i<$max;$i+=$step){
$x = Matrix::zeros([$i,$i]);
$y = Matrix::ones([$i,$i]);
$x = $x->toArray();
$y = $y->toArray();
$start = microtime(true);
$x = simple_add($x, $y);
$total += (microtime(true) - $start);
echo "Iter $i [$i,$i]: ".(microtime(true) - $start)."\n";
}
echo "Total: ".$total."\n\n";
$total = 0;
for($i=$min;$i<$max;$i+=$step){
$x = Matrix::zeros([$i,$i]);
$y = Matrix::ones([$i,$i]);
$x = $x->toArray();
$y = $y->toArray();
$start = microtime(true);
PHPSciMatrix::add($x, $y);
$total += (microtime(true) - $start);
echo "Iter $i [$i,$i]: ".(microtime(true) - $start)."\n";
}
echo "Total: ".$total."\n\n";
`

@akondas Attached is a sample CSV, this would be for one run of the regression.  You can see the data is very minimal.  The next round shifts the rows and adds one new one for another regression and so forth.  Do you think maybe it has to do with the decimal precision?
samplestargets.txt

@henrique-borba You library looks interesting. I can not wait to show you something more (read: share sources).

@exantedata You data on my machine took 12.7ms :warning: so something here is very strange
I noticed one interesting issue for this, maybe it's the fault. Encoding of you file is UTF-16 - PHP does not even see these figures as numbers:

Hi kondas, I'm currently traveling to brazil (16 hours). As soon as I get there I will publish the code so we can start work on it. Thanks again.

@akondas Hmm... I dunno, that could be because I had to save it as a .txt from Excel to post here in github... everything works as expected when I do 5-6 features.

@exantedata maybe you have xdebug enable? I don't have any idea why you loose performance. I can even double your example samples (16 features) and even then everything go faster. If you have time I recommend to install Blackfire and check it with blackfire run: https://blackfire.io/docs/up-and-running/installation

@akondas Cool cool, again thanks for the testing/attention.  I'll dig back into my side and see if I can get some info with blackfire etc.  Always possible my side of the code needs to be optimized not yours :-)  Thanks again!

Also one question related with performance want to know.
Do we think about SplFixedArray ?
http://php.net/manual/en/splfixedarray.fromarray.php
I want to try it if could improve performance.
But Estimator interface force input with php native array.
https://github.com/php-ai/php-ml/blob/master/src/Estimator.php

Hi HillLiu,
In my personal tests, SplFixedArrays worked 2 or 3 times faster when used in matrix creation (full of zeros), but perfomance in math operations wasn't that fast. It uses less memory but you can't change matrix dimension without creating another one. I'm working hard at a C extension for matrices, I had to delay the release cause I found better ways to compute stuff, in the next few days I will come with a prototype.

Hi ,
This topic is getting more interesting because of the possibility of including a fast matrix/array library. Last year, I was trying to develop something for PHP-ML that required intense matrix operations. Back then, I need to compare that feature of PHP-ML with that of scikit-learn. The results were almost the same but the performance - there was a huge difference, scikit-learn was blazingly fast, I mean fast!
According to the benchmarks, PHP, as a language, is not slower than Python, especially after PHP 7. But the array performance is too bad if you need to handle a lot of math operations. Since then, I am more than interested in libraries/extensions to speed-up PHP's native array performance and truly excited to see what @henrique-borba offered.
In my opinion, if @henrique-borba has a working version, we must immediately test it to see if it fits to the needs without any major conflict. Then, as a separate repository, it can be continously developed & updated. Existing PHP-ML users should not be forced to use it but it will be nice to have it as an option to speed-up the computational performance.

I have great news!
I was able to store the C array internally in the memory of the PHP class, without it being visible to the user (like a weird malloc). The PHP part of the component (which will be made available via composer) is using magic methods and extending an ArrayIterator to simulate a PHP array, however the whole operation will be directly done in C array.
This is a big step! Now only the print operation of the array (visualization) will require that it be transformed into a PHP array.
The downside is simple: print_r won't print your array, only echo. You can use print_r after using the toArray() method.
Some outputs:
<?php
$autoloader = require_once dirname(__DIR__) . '/vendor/autoload.php';

use PHPSci\PHPSci;
use PHPSci\Backend\Exceptions\PHPSciException;

try {
    $new = new PHPSci([[1,2,3],[4,5,6],[1,2,3],[4,5,6]]);
    print_r($new);
    echo "\n\n".$new;
    echo "\n\nCARRAY SIZE (KILOBYTES): ".$new->sizeOf("kb"). " KB";
    echo "\n\n";
} catch(PHPSciException $e) {
    echo $e->getMessage();
}

PHPSci\PHPSci Object
(
    [c_array:protected] => CPHPSci Object
        (
            [dim] => 2
            [c_array_size] => 96
            [m] => 4
            [n] => 3
        )

)

[
   [ 1 2 3 ]
   [ 4 5 6 ]
   [ 1 2 3 ]
   [ 4 5 6 ]
]

CARRAY SIZE (KILOBYTES): 0.09375 KB

You can see the C class (CPHPSci Object) and some basic information stored with print_r, there is no PHP array at all. Using toArray() and you can get the PHP version of it:
<?php
$autoloader = require_once dirname(__DIR__) . '/vendor/autoload.php';

use PHPSci\PHPSci;
use PHPSci\Backend\Exceptions\PHPSciException;

try {
    $new = new PHPSci([[1,2,3],[4,5,6]]);
    print_r($new->toArray());
    echo "\n\nCARRAY SIZE (KILOBYTES): ".$new->sizeOf("kb"). " KB";
    echo "\n\n";
} catch(PHPSciException $e) {
    echo $e->getMessage();
}

Array
(
    [0] => Array
        (
            [0] => 1
            [1] => 2
            [2] => 3
        )

    [1] => Array
        (
            [0] => 4
            [1] => 5
            [2] => 6
        )

)

CARRAY SIZE (KILOBYTES): 0.046875 KB

Today I must start the matmul tests! See you

@MustafaKarabulut Thank you for your kind attention, I already made a PHP-ML fork to start some tests and today I should already have some results with basic operations (matmul, add, etc).
After creating the basic documentation I can already publish something minimally functional.

Got the first results with the new implementation
MATRICES WITH SHAPE [ITER, ITER], STEP 100 AND RAND NUMBERS
MATMUL PHP IJK
========================================
Matrix (100,100): 0.28207802772522 secs
Matrix (200,200): 2.2361738681793 secs
Matrix (300,300): 7.6536920070648 secs
Matrix (400,400): 17.980815172195 secs
Matrix (500,500): 35.14064002037 secs

MATMUL PHPSCI
========================================
Iter (100,100): 0.0017571449279785 secs
Iter (200,200): 0.0089671611785889 secs
Iter (300,300): 0.027651071548462 secs
Iter (400,400): 0.070709228515625 secs
Iter (500,500): 0.13997387886047 secs

DIFFERENCE PER ITER
========================================
Iter (100,100): PHPSci is **~160x** faster them PHP
Iter (200,200): PHPSci is **~249x** faster them PHP
Iter (300,300): PHPSci is **~276x** faster them PHP
Iter (400,400): PHPSci is **~254x** faster them PHP
Iter (500,500): PHPSci is **~251x** faster them PHP

Good results, much better them the older 3x faster version.
Now I should test it within PHP-ML

@henrique-borba I would love to try this with our data, see if it helps.  Let me know when you create a fork of php-ml using this.  Thanks for the efforts!

@akondas I have good (though embarrassing) news!
It turns out that my composer was only updating this package to 0.4.  So today I removed the package and reinstalled into 0.6 and its fast as lightning now, no issues what so ever.   I'm embarrassed that I didn't check that before starting this thread.

Good to know :-1:
I made worse mistakes so no problem. The main thing is that everything works correctly now.
:rocket:

@henrique-borba looks very promising. Maybe we can create separate issue for that?

@akondas Done: https://github.com/php-ai/php-ml/issues/268


ipam: Avoid empty CIDR in ENI mode

In some cases, there might be race condition between Instance Manager (running in operator) and IPAM (running in agent) modules, which could lead to IP details are not populated properly, and hence cause the below fatal error. This commit is to make sure that we don't add empty CIDR into the allocation result. Just a note that once the CRD is update, the existing resync process will kick off and perform the needful.
time="2024-10-22T22:48:31Z" level=fatal msg="failed to start: daemon creation failed: failed to coalesce CIDRs: invalid CIDR address: " subsys=daemon

Relates: https://github.com/cilium/cilium/issues/32855

/test


Questions: Have 3 levels of authentication

Hi,
Following our twitter discussion: https://twitter.com/gakuzzzz/status/729984863244189697
I am not sure what to do... For the moment, I have 3 different levels in authorize function:
type User = models.UserLoggedData //a container that also contains a authorityUser which is the database object
type Authority = models.Role //simple role trait really close to your default one

def authorize(user: User, authority: Authority)(implicit ctx: ExecutionContext): Future[Boolean] = Future.successful {
    (user.authorityUser.role, authority) match {
      case ("admin", _)       => true
      case ("moderator", Role.moderator)   => true
      case ("simpleuser", Role.simpleuser)  => true
      case _                  => false
    }
  }

I can use this by having a StackAction with AuthorityKey -> admin (or moderator, or simpleuser). But I would like to have one action which can be used by 2 roles (moderator AND simpleuser).
Is there a simple way to do this ? (I think I found a workaround, but I guess you know a normal/easy way to do this)
Thanks,
Yoann

We found the solution, way too simple...
We finally got 2 "similar" ways of doing it:

Use an additional role for both "moderator" and "simpleuser"
models.Role:

case object admin extends Role
case object moderator extends Role
case object moderatoruser extends Role
case object simpleuser extends Role
...

def authorize(user: User, authority: Authority)(implicit ctx: ExecutionContext): Future[Boolean] = Future.successful {
    (user.authorityUser.role, authority) match {
      case ("admin", _)       => true
      case ("moderator", Role.moderator)   => true
      case ("moderator", Role.moderatoruser)   => true
      case ("simpleuser", Role.moderatoruser)   => true
      case ("simpleuser", Role.simpleuser)  => true
      case _                  => false
    }
  }

That works great if you have actions that need to be used by both simpleuser and moderator, but not all pages.
Then we realized that all actions of a simpleuser could be used by a moderator so we went back a bit and decided to use:
2.
models.Role:
case object admin extends Role
case object moderator extends Role
case object simpleuser extends Role
...

def authorize(user: User, authority: Authority)(implicit ctx: ExecutionContext): Future[Boolean] = Future.successful {
    (user.authorityUser.role, authority) match {
      case ("admin", _)       => true
      case ("moderator", Role.moderator)   => true
      case ("moderator", Role.simpleuser)   => true
      case ("simpleuser", Role.simpleuser)  => true
      case _                  => false
    }
  }

Anyway, it took us a while to realize how easy it was... Anyway, thanks for the help and for the play2auth module.


__abort_with_payload / _dyld_start

I am trying to develop an iOS (and Android) app.
It works perfectly on Android, but I constantly get the same error on iOS. I have even started a brand new react-native project from scratch in an effort to narrow it down to the lowest common denominator, and this issue happens each time I try to run it after adding and linking the react-native-ble-plx package.
Here is a screenshot of the error

I've tried clearing caches and rebooting
I've tried rolling back to previous versions
I've tried embedding the 3rd party package to the embedded binaries, but I'm unsure what to add (I don't see a .framework file)

Any help would be greatly appreciated, I've already invested about 40 hours trying to figure this out, and I'm stumped.
I'm new to iOS development, so I'm not sure what information is would help, but here are the versions to several things:

OS: maxOS High Sierra Version 10.13.6
Xcode: Version 9.4.1 (9F2000)
Swift: Version 4.1.2 (swiftlang-902.0.54 clang-9<IP_ADDRESS>)
react: Version 16.4.1
react-native: Version 0.56.0
react-native-ble-plx: Version 0.9.1
npm: Version: 6.2.0
Yarn: Version 1.7.0
Carthage: Version 0.30.1

In case anybody else was as stumped as I was, maybe this will help.
I submitted this to another site, and was graciously helped by another user there.  He created this repo, which works perfectly.
I haven't taken the time to figure out what I did wrong just yet (time crunch), but hopefully this will help others.

It's great that it finally works for you! Closing this issue


feat: updated design system package to 1.0.13

Description
This version includes the fix for local dev button tab issue
Fixes #15913
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Tested in local

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=627b618

This PR has been tested and is working as expected.

/ok-to-test sha=255acd8


Unsnarl Scrutinizer issues

Coverage remained the same at 99.364% when pulling c432de70c705894952e97ad723dfbd05d73d71eb on CyberiaResurrection:UnsnarlScrutinizerIssues into a886aed7e968f856aa81b72f8a9a6d119d1f604c on Algo-Web:master.

Coverage remained the same at 99.364% when pulling c432de70c705894952e97ad723dfbd05d73d71eb on CyberiaResurrection:UnsnarlScrutinizerIssues into a886aed7e968f856aa81b72f8a9a6d119d1f604c on Algo-Web:master.


Add Unix socket support to PostgreSQLPersistentStore

Google's serverless Cloud Run framework is a great option for deploying Aqueduct applications, but it requires Unix sockets when connecting to Cloud SQL.
Luckily, as of version 2.2.0, Postgres supports Unix sockets.
So to close the loop on all of this, it would be great if Aqueduct's PostgreSQLPersistentStore constructors exposed this new isUnixSocket behavior.

I would need this as well, hopefully this gets implemented soon.


Appstudio update test-component-pac-itfh

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-itfh-on-pull-request has failed.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
17 seconds
clone-repository

🏃 Running
---
build-container

Pipelines as Code CI/test-component-pac-itfh-on-pull-request has failed.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
17 seconds
clone-repository

🏃 Running
---
build-container

Pipelines as Code CI/test-component-pac-itfh-on-pull-request has failed.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
17 seconds
clone-repository

🏃 Running
---
build-container


[RR2, BUG?] On one of the windows machine, stderr messages showing inconsistent

Take a look, that the messages separated randomly.
OS Build: 10.0.18363.1256

After discussion with @SerafimArts , we think, that the buffer instantiated in the PHP script which flushes data to the stderr, should be set to the 10240 bytes.

After discussion with @SerafimArts , we think, that the buffer instantiated in the PHP script which flushes data to the stderr, should be set to the 10240 bytes.

Just in case, I will give an example of the current code, which is the output buffer to the stderr (to avoid getting extra data into the output buffer, which breaks the protocol):
    /**
     * @return $this
     */
    private function bootStdoutHandlers(): self
    {
        // symfony/var-dumper interceptor
        if (\class_exists(CliDumper::class)) {
            CliDumper::$defaultOutput = 'php://stderr';
        }

        // Intercept all output messages
        \ob_start(fn(string $chunk) => $this->write($chunk));

        // Intercept all exceptions
        \set_exception_handler(fn(\Throwable $e) => $this->writeException($e));

        // Intercept all errors
        \set_error_handler(function (int $code, string $message, string $file, int $line) {
            $this->writeException(
                new \ErrorException($message, $code, $code, $file, $line)
            );
        });

        return $this;
    }

    /**
     * @param string $message
     */
    private function write(string $message): void
    {
        \file_put_contents('php://stderr', $message);
    }

    /**
     * @param \Throwable $e
     */
    private function writeException(\Throwable $e): void
    {
        $this->write((string)$e);
    }

Possible cause of the problem is the output buffer (OB) redirection function ob_start without specifying the size of chunks (second argument of the function): https://www.php.net/manual/en/function.ob-start.php

Just in case, I will give an example of the current code, which is the output buffer to the stderr (to avoid getting extra data into the output buffer, which breaks the protocol):
    /**
     * @return $this
     */
    private function bootStdoutHandlers(): self
    {
        // symfony/var-dumper interceptor
        if (\class_exists(CliDumper::class)) {
            CliDumper::$defaultOutput = 'php://stderr';
        }

        // Intercept all output messages
        \ob_start(fn(string $chunk) => $this->write($chunk));

        // Intercept all exceptions
        \set_exception_handler(fn(\Throwable $e) => $this->writeException($e));

        // Intercept all errors
        \set_error_handler(function (int $code, string $message, string $file, int $line) {
            $this->writeException(
                new \ErrorException($message, $code, $code, $file, $line)
            );
        });

        return $this;
    }

    /**
     * @param string $message
     */
    private function write(string $message): void
    {
        \file_put_contents('php://stderr', $message);
    }

    /**
     * @param \Throwable $e
     */
    private function writeException(\Throwable $e): void
    {
        $this->write((string)$e);
    }

Possible cause of the problem is the output buffer (OB) redirection function ob_start without specifying the size of chunks (second argument of the function): https://www.php.net/manual/en/function.ob-start.php

Sample code to reproduce.
<?php

use Spiral\RoadRunner;
use Nyholm\Psr7\Factory;

ini_set('display_errors', 'stderr');
include "vendor/autoload.php";

$worker = new RoadRunner\Http\PSR7Worker(
    RoadRunner\Worker::create(),
    new Factory\Psr17Factory(),
    new Factory\Psr17Factory(),
    new Factory\Psr17Factory()
);

while ($req = $worker->waitRequest()) {
    try {
        $rsp = new \Nyholm\Psr7\Response();
        $rsp->getBody()->write("hello world");
        error_log("hello");
        $worker->respond($rsp);
    } catch (\Throwable $e) {
        $worker->getWorker()->error((string)$e);
    }
}```

Sample code to reproduce.
<?php

use Spiral\RoadRunner;
use Nyholm\Psr7\Factory;

ini_set('display_errors', 'stderr');
include "vendor/autoload.php";

$worker = new RoadRunner\Http\PSR7Worker(
    RoadRunner\Worker::create(),
    new Factory\Psr17Factory(),
    new Factory\Psr17Factory(),
    new Factory\Psr17Factory()
);

while ($req = $worker->waitRequest()) {
    try {
        $rsp = new \Nyholm\Psr7\Response();
        $rsp->getBody()->write("hello world");
        error_log("hello");
        $worker->respond($rsp);
    } catch (\Throwable $e) {
        $worker->getWorker()->error((string)$e);
    }
}```

@wolfy-j Please, post here the link to the commit fixing that issue

The actual bug is located in the error_log function. It can not be used for reliable logging purposes due to internal implementation. It's recommended to write directly to stderr, for example:
file_put_contents('php://stderr', 'message');


feat(cache,gateway,model,standby)!: support guild scheduled events gateway events

This PR adds support for Guild Scheduled Event related gateway events.
EventTypeFlags has been updated to use u128.
Closes: #1557

This is a breaking change of our own API, as event type flags are our implementation and not mapped to Discord's API in the model crate. As a result of this, this needs to be targeted against next. It may be worth looking into related types to see if we need to bump their size to plan ahead for situations like this.

Yes I've missed that. Waiting for https://github.com/twilight-rs/twilight/pull/1577 and then I will retarget this PR to next.


556 css pointer events

Linking
Problem
There are situations in which we don't want a model or UI element to receive pointer-events. irrelevant elements intercepting these events can be frustrating.
Solution
implement support for pointer-events css property, and check it's value in the PhysicsSystem

Required to Merge

[x] PASS - all necessary actions must pass (excluding the auto-skipped ones)
[x] TEST IN HEADSET - main dev-testing-example and any of the other examples still work as expected
[ ] VIDEO - if this pr changes something visually - post a video here of it in headset-MR and/or on desktop (depending on what it affects) for the reviewer to reference.
[x] TITLE - make sure the pr's title is updated appropriately as it will be used to name the commit on merge
[x] BREAKING CHANGE

DOCUMENTATION: This includes any changes to html tags and their components

make a pr in the documentation repo that updates the manual docs to match the breaking change
link the pr of the documentation repo here: #pr
that pr must be approved by @lobau

SAMPLES/INDEX.HTML: This includes any changes (html tags or otherwise) that must be done to our landing page submodule as an effect of this pr's updates

make a pr in the mrjs landing page repo that updates the landing page to match the breaking change
link the pr of the landing page repo here: #pr
that pr must be approved by @hanbollar

btw @michaelthatsit just as heads up for breaking changes - make sure to link whatever documentation pr or landing page pr to this pr in the comment body so it doesnt get lost

btw @michaelthatsit just as heads up for breaking changes - just make sure to link whatever documentation pr or landing page pr to this pr in the comment body so it doesnt get lost

Oh I thought it was a checklist item not a flag that it has one 😅

btw @michaelthatsit just as heads up for breaking changes - just make sure to link whatever documentation pr or landing page pr to this pr in the comment body so it doesnt get lost

Oh I thought it was a checklist item not a flag that it has one 😅

no worries! it's kinda both specifically for breaking change, lmk if there's a better way to word that part of the PR template to reduce some of that confusion - since others might miss that as well


docs(samples): add try/catch to delete operations in testing

Fixes b:202972969

@irataxy, will these close #17 and #18?


docs(readme): update readme

Codecov Report
Patch and project coverage have no change.

Comparison is base (0402605) 79.40% compared to head (0b274d6) 79.40%.

:exclamation: Current head 0b274d6 differs from pull request most recent head 1648f69. Consider uploading reports for the commit 1648f69 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #655   +/-   ##
=======================================
  Coverage   79.40%   79.40%
=======================================
  Files          31       31
  Lines         369      369
  Branches       60       60
=======================================
  Hits          293      293
  Misses         70       70
  Partials        6        6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Issue templates now use Issue forms instead of markdown

Markdown templates often made it difficult to capture the whole context, by expecting the contributors to read the markup, uncomment lines and delete relevant context.
Issue forms beta was released for public repositories last month, as mentioned by GitHub on this blog. We would like to migrate the issue templates to YAML-based forms which are easy to fill up and maintain, helping us capture relevant context.
https://user-images.githubusercontent.com/67006255/157601493-30b1757d-1acf-4ab3-b732-a8973a1b4139.png

#128


Upcoming releases

Can someone share please the release program/lifecycle?
Thanks!

What would you like to know precisely? We don't have a strict release program for huggingface_hub. Usually you can expect a minor release every 4 to 6 weeks depending on how active has been the development. But we don't refrain ourselves to release more often if new cool features are added quickly. Latest version is 0.18.0.
About lifecycle, we always try to ensure backward compatibility when making changes to an existing API. If a breaking change will be introduced, we usually deprecate the feature 3 releases before breaking it. Deprecation are mentioned in the release notes and a warning message is triggered each time a deprecation method is used. For some legacy features that have been at the core of the library in the past (for example Repository or InferenceAPI classes) but are now abandoned in favor of more robust/feature-complete classes, we set a deprecation cycle until the next major release 1.0.0, which is not planned yet.
Hope this will help you. Please let me know if you want any further details.

Can you share the exact next release date?

@oriyafe most likely tomorrow

thanks!


chore: add dependency to Rasa integration testing (#51)

Description
Add a dependency to Rasa integration testing so that it can be executed as a local module in the project. This will be necessary for the pipeline.

Related issues

#51

Checklist

[ ] PR title follows Conventional Commits guidelines
[ ] All relevant PR sections are populated, irrelevant ones are removed

We should add a task in the Makefile to run the tests and a small section in the README.


fix: remove yarn.lock

This project currently use package-lock.json, remove yarn.lock for avoiding confuse.

yeap

@tiendn thanks, nice catch👍


Failed to install ffmpeg. Failed to download resource "x265".

Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[x] ran brew update and can still reproduce the problem?
[x] ran brew doctor, fixed all issues and can still reproduce the problem?
[x] ran brew config and brew doctor and included their output with your issue?

To help us debug your issue, please complete these sections:
What you were trying to do (and why)
I was trying to run "brew install ffmpeg"
What happened (include command output)
The download link failed.

Command output

==> Downloading https://mirrors.edge.kernel.org/pub/software/scm/git/git-htmldocs-2.28.0.tar.
curl: (60) SSL certificate problem: Invalid certificate chain
More details here: http://curl.haxx.se/docs/sslcerts.html
curl performs SSL certificate verification by default, using a "bundle"
of Certificate Authority (CA) public keys (CA certs). If the default
bundle file isn't adequate, you can specify an alternate file
using the --cacert option.
If this HTTPS server uses a certificate signed by a CA represented in
the bundle, the certificate verification probably failed due to a
problem with the certificate (it might be expired, or the name might
not match the domain name in the URL).
If you'd like to turn off curl's verification of the certificate, use
the -k (or --insecure) option.
Error: Failed to download resource "git--html"
Download failed: https://mirrors.edge.kernel.org/pub/software/scm/git/git-htmldocs-2.28.0.tar.xz
==> Downloading https://bitbucket.org/multicoreware/x265/downloads/x265_3.4.tar.gz
curl: (22) The requested URL returned error: 403 Forbidden
Error: Failed to download resource "x265"
Download failed: https://bitbucket.org/multicoreware/x265/downloads/x265_3.4.tar.gz

What you expected to happen
I expected brew to complete the installation
Step-by-step reproduction instructions (by running brew commands)
brew install ffmpeg
Output of brew config and brew doctor commands

HOMEBREW_VERSION: 2.4.12
ORIGIN: https://github.com/Homebrew/brew
HEAD: 19ef93288573e565cec82679d0cd834a4ad080ab
Last commit: 2 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: ccdb268d1deaa67f89050b252eff78a2489edcfc
Core tap last commit: 10 hours ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 4
CPU: quad-core 64-bit sandybridge
Homebrew Ruby: 2.6.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3_2/bin/ruby
Clang: 8.0 build 800
Git: 2.10.1 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.43.0 => /usr/bin/curl
Java: 1.8.0_265, 1.6.0_65-b14-468
macOS: 10.11.6-x86_64
CLT: <IP_ADDRESS>.1.1480973914
Xcode: N/A

Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: "config" scripts exist outside your system or Homebrew directories.
`./configure` scripts often look for *-config scripts to determine if
software packages are installed, and which additional flags to use when
compiling and linking.

Having additional scripts in your path can confuse software installed via
Homebrew if the config script overrides a system or Homebrew-provided
script of the same name. We found the following "config" scripts:
  /Library/Frameworks/Python.framework/Versions/3.8/bin/python3-config
  /Library/Frameworks/Python.framework/Versions/3.8/bin/python3.8-config

Warning: You have unlinked kegs in your Cellar.
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
<EMAIL_ADDRESS>
Warning: You are using macOS 10.11.
We (and Apple) do not provide support for this old version.
You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Discourse, Twitter or IRC. You are responsible for resolving any issues you
experience while you are running this old version.

Warning: An outdated version (2.10.1) of Git was detected in your PATH.
Git 2.14.3 or newer is required for Homebrew.
Please upgrade:
  brew install git

Warning: You are using macOS 10.11.
We (and Apple) do not provide support for this old version.
You will encounter build failures with some formulae.
Please create pull requests instead of asking for help on Homebrew's GitHub,
Discourse, Twitter or IRC. You are responsible for resolving any issues you
experience while you are running this old version.


Issue with Bootstrap accordions and mmenu

First things first - THANK YOU for this awesome plugin, you rock!
I'm using mmenu on a new webapp that uses the standard jquery accordion on some pages. After loading up mmenu (which works great!) the accordions stopped working. Further investigation it seems that the .collapse class isn't being removed from the accordion UL on click of the corresponding A with the data-toggle.  I remove mmenu and the accordion works as normal. Any ideas?

Sorry. This issue is not with mmenu it's with the unlaying theme.


Add custom alias to username

#91

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: lprintf:x: Huan (李卓桓)Huan (李卓桓) seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


squid:S2208 - Wildcard imports should not be used

This pull request is focused on resolving occurrences of Sonar rule squid:S2208 - Wildcard imports should not be used
You can find more information about the issue here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S2208
Please let me know if you have any questions.
M-Ezzat

Anoher rule I don't agree with


Boolean type is not subtype of boolean literals

TypeScript Version:  master

Search Terms:
Code
type B<T extends true | false> = T;
type N<T extends 0 | 1> = T;
type S<T extends ''> = T;
true as B<boolean>;
0 as N<number>;
'' as S<string>;

Expected behavior:
$ node built/local/tsc.js --noEmit index.ts
index.ts(4,11): error TS2344: Type 'boolean' does not satisfy the constraint 'true | false'.
index.ts(5,8): error TS2344: Type 'number' does not satisfy the constraint '0 | 1'.
index.ts(6,9): error TS2344: Type 'string' does not satisfy the constraint '""'.

Actual behavior:
$ node built/local/tsc.js --noEmit index.ts
index.ts(5,8): error TS2344: Type 'number' does not satisfy the constraint '0 | 1'.
index.ts(6,9): error TS2344: Type 'string' does not satisfy the constraint '""'.

Playground Link:
Related Issues:

boolean is just true | false. it is diffrent from number or string that are infinite sets.. not sure i see the issue here.

type B<T extends true> = T; will be error correctly.


New Feature: compute the fingerprint of a Plonk verification key for recursive verification

Description
This PR added a new feature to compute the fingerprint (MiMc hash) of a Plonk verification key. The fingerprint could be used to identify a unique verification key or circuit during recursive verification.
Fixes # (issue)
None
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How has this been tested?

[x] Added codes to std/recursion/plonk/verifier_test.go to ensure all curves are supported;
[x] Added file std/recursion/plonk/vkey_fp_test.go to ensure the correctness of the added functionality.

How has this been benchmarked?
We adopted the snark-friendly hash MiMc to minimize the constraints count. However no benchmark suites were added.
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I did not modify files generated from templates
[x] golangci-lint does not output errors locally
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

This PR depends on https://github.com/Consensys/gnark/pull/1040/

Converted to draft. Just came to awareness that MiMc might output different data under different underlying fields. Reviewing impacts.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: weijiguo:x: keepkeep seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


ML package

Hello,
I'm trying to use this tool on a small dataset.
When running 'progressive_docking.py', this line (19):
from ML.DDModel import DDModel

raises this error:

ModuleNotFoundError: No module named 'ML'

What is 'ML'? Is this something that needs to be installed?
Thank you,
Yair Frank

Seems someone forget to upload the private "ML" module? It does not sound to be publicly available anywhere

I think so, too

You're right, somehow the ML module was not included in our upload, we will fix this very soon

Thank you

Let us known if you still have problems


Fix: this prevents nginx to emergency shutdown if optional services are not started

Description:
This PR prevents nginx to emergency shutdown (host "xxx" not found in upstream) if an optional service like the mrv-sender, topic-viewer or mongodb-admin is not started (i.e., it's excluded from the docker-compose file);
Related issue(s):
Fixes # N/A
Notes for reviewer:
Checklist

[x] Documented (Code comments, README, etc.)
[x] Tested (unit, integration, etc.)

@prernaadev01 can you ensure this is included in the august release? Thanks.

@Neurone Sure. Will make sure this is reviewed asap. Thanks

[like]  Prernaa Agarwal reacted to your message:

From: simvalery @.>
Sent: Wednesday, August 28, 2024 2:01:12 PM
To: hashgraph/guardian @.>
Cc: Prernaa Agarwal @.>; Mention @.>
Subject: Re: [hashgraph/guardian] Fix: this prevents nginx to emergency shutdown if optional services are not started (PR #3752)
Merged #3752https://github.com/hashgraph/guardian/pull/3752 into develop.
—
Reply to this email directly, view it on GitHubhttps://github.com/hashgraph/guardian/pull/3752#event-14047233880, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS463CNBY6UOZEVMOTPJPX3ZTXJ2RAVCNFSM6AAAAABI2ZVAO6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGA2DOMRTGM4DQMA.
You are receiving this because you were mentioned.Message ID: @.***>


wombatfuse

think about implementing a fuse module at some point which will spit out the uncompressed raw dd as a loop device.
can use affuse/ewfuse as a guide.

almost got the size working, the ls is working, and the reading of content from the raw file isn't quite there yet..

once i get the reading the compressed file content, then fuse is working and i can then work on getting the actual uncompressed content read...
to do this i would need to keep track of the current uncompressed offset, discard buffers that aren't within the offset i care about...
then once i get the offset i care about, write that to the buffer containing the offset and size to read, and then keep reading the next buffers and adding until i get the necessary size and then return the buffer and end the uncompression and the discarded buffers.

wombat fuse is working.


ENH: Added an option to invert the volume mask before cropping the tu…

…bes.

Operations on masks should be done before passing them to a function.  ImageMath can invert a mask, for example.  Let's not duplicate that functionality in all of our apps that use a mask.

Okay. Closing the pull request.


How to mark/reject resources using unsupported features

Problem statement:
At the moment there is no standardized way to check a HTTPRoute uses an unsupported feature. At Kong, we have a validating webhook that rejects HTTPRoutes using unsupported features. GEP-2162 introduced the supportedFeatures field to highlight the features supported by the implementation, and in the future work section such topic is mentioned. A standardized and shared way of accepting or rejecting resources using unsupported features is desirable, and this issue is about sensing the community's feelings.
Proposed solution
I think a viable (and desirable) solution would be to move this logic away from implementations' webhooks and instead rely on the resources' status. We could introduce a new reason for gateways and routes Accepted condition so that no resource using an unsupported feature is accepted and proper feedback is given to the user.

Could https://github.com/kubernetes-sigs/gateway-api/blob/a919f8b2b21ffb3aecae0a0a03b6e7ed0d17e95e/apis/v1/shared_types.go#L344 work?
We use it in our implementation for HTTPRoute fields that are not supported (ex. an unsupported filter). Although, we also use it for invalid values too

Could
https://github.com/kubernetes-sigs/gateway-api/blob/a919f8b2b21ffb3aecae0a0a03b6e7ed0d17e95e/apis/v1/shared_types.go#L344
work?
We use it in our implementation for HTTPRoute fields that are not supported (ex. an unsupported filter). Although, we also use it for invalid values too

They are intended for a different use, when a field value does not belong to the enum. Example:
https://github.com/kubernetes-sigs/gateway-api/blob/6b8c23701bc03040a59771f17d73300fcea742a9/apis/v1/httproute_types.go#L318-L338

/kind support


⚠ Samsung has degraded performance

In d5fd69e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1997 ms

Resolved: Samsung performance has improved in 6da5654.


Display name validation is misleading

There are a few issues with the display name validation:

Display name must be prefixed with opentelemetry/ according to the regex https://github.com/googleinterns/cloud-operations-api-mock/blob/e8bcc34f971a612c9e598ce4c66c192448c7dac6/internal/validation/mock_metric_validation.go#L48 Why do we check for this? This is not documented in go/ot-gcp-requirements

When display name doesn't match the regex mentioned above, mock server returns the error "displayName has max length of 128 bytes" (see https://github.com/googleinterns/cloud-operations-api-mock/blob/7216ea71824fc5232bfac8f328a11ed31724c0db/internal/validation/statuses.go#L40-L41), which is super misleading: we don't even check for the string length, and should have specified in the error that we expected it to be prefixed with opentelemetry/ or something more user friendly.

I noticed the same


Update the PACs and resolve breaking changes, simplify rmt driver

Updates all PACs to their latest (unpublished) versions

I'm holding off on publishing new PACs until we're ready for new HAL releases

Resolve all breaking changes from the PAC updates
Re-organize and simplify the rmt driver

Was able to remove a confusing parameter from the macros!
Sorry for the noise at the beginning of the driver, I like things to be in order :)

I haven't done much testing here yet so we will need to verify that nothing has been broken, but most of the changes are pretty straight-forward I think.

finally, the s2 definitions fit on my monitor 😂


Add warning for bad relativePath api version

Fixes #1242

Turns out 2020-06-01 is valid for template specs, and there's no way for me to detect when it's for a template spec.  Earlier api versions get an error for relativePath.


wallet_v4r2

wallet_v4r2
Originally posted by @tiaraton in https://github.com/ton-blockchain/wallet-contract/pull/11#issuecomment-2345579385

wallet_v4r2
Originally posted by @tiaraton in https://github.com/ton-blockchain/wallet-contract/pull/11#issuecomment-2345579385

Hey guys! I’ve claimed my tokens but i cant find it in any of my wallets. How should i know which wallet it is?

When i pasted the address on tonveiwer it said that it is a wallet_v42 type the adress is : UQAzxxHMlf171r45uZvj9E6W-JquSDz46Q74OIpZYneD9Sil


Update the R3 Corda Enterprise images to the right versions & naming schemes

Description
As a developer, I want to update the R3 Corda Enterprise images to the right versions & naming schemes, so that you can easily deploy multiple versions of the CENM and Nodes.
Currently, the CENM Notary docker image uses the 1.2 tag as it's baseline, as seen in the code snippet below:
https://github.com/hyperledger-labs/blockchain-automation-framework/blob/f48d1a3070f082feb1fd918df1130d11dd853e53/platforms/r3-corda-ent/images/notary.dockerfile#L1
This tag has to be updated to the 4.4-zulu-openjdk8u242 as per this reference link.
Accordingly, update deployment file creation logic to include this version in the  main.yaml for notary & notary-initial-registration.
Additionally, update the /platforms/r3-corda-ent/configuration/roles/helm_component/vars/main.yaml to include logic to use multiple versions.
Acceptance Criteria

Update internal BAF repository with the updated tag for the notary image
Multiple versions of images can be supported by changing the versions in the network.yaml
Test the BAF deployment by deploying CENM + nodes and verifying the nodes are registered by using the /network-map-user/node-infos API endpoint

To support multiple versions of images, the org.type = node also needs a version tag, like the org.type: cenm has. I'll raise a PR to baf-configuration for this.

Verified the network deployment by using the network-map-user/node-infos endpoint:


Add requestUri support

I try to send parameters like this:
@GET("/users")   Future<UsersResponse> getUsers(       {       @Query("id") List<String>ids);
I wait request like this /v2/users?id=5&id=7,
But got this: /v2/users?id=%5b%5d5&id=%5b%5d7
Description of dio https://developpaper.com/flutter-http-library-dio-2-1-was-officially-released/ says that in such case we must use uri.
How to fix this?

I am working on other urgent issues right now. I will spend effort on this when I get time.

I this feature realized, or just closed?

It's closed automatically by the robot.
It needs a lot of refactoring work to implement this feature.  I have to do changes almost for every single method in generator package.
It may affect the stability of this package. Considering adding this functionality in a future release.
Will let you know when I finished this.

Thanks for quick answer. I'll be wait.

I'm facing with this problem too

@hautvfami temporary solution - make request manually

There's another solution I've found, which is to make use of @Queries() Map<String, dynamic> params.
And then create the queries map like this:
final params = {
    'skills[]': ['1', '2'],
};


Create DPAS Analysis infrastructure

Create a DPASAnalysis class that analyzes a function (e.g. kernel) and determines whether tt.dot operations can be lowered to DPAS instructions on ATS/PVC target.  In this PR we replace uses of supportDPAS with a public member function provided by the new class. A future PR will use the same infrastructure to change the subgroup size in the module if the analysis detects that a kernel contains tt.dot operations that would be optimizable (could be lowered to DPAS instructions) if the subgroup size was the one supported by the DPAS instruction for the target architecture (e.g. 16 on PVC).

Looks to me we don't need a map at all. We can just:

Check precondition (attribute+threads per warp)

Walk and check each dot operation supports dpas:

If an operation does not support it, set a member to False, exit
If an operation potentially does not support it, set some member to MayBe

True we do not necessarily need the mat but I want to keep it to have the flexibility to identify which tt.dot operations can be lowered to DPAS instructions. We might want to use that flexibility later.


Load policies on windows without need for an Active dictionary

copied from https://github.com/Eloston/ungoogled-chromium/issues/1335
Is your feature request related to a problem? Please describe.
On firefox, I use policies to force myself to have certain extensions installed and centain websites blocked. This does not work with ungoogled chromium as it only loads policies if the user has joined an active dictionary.
Describe the solution you'd like
Patch the call to 'isOS' function exported by 'SHLWAPI.dll', so that chromium always believes that it is in an active dictionary.
Optionally add a setting to do that.
This solution is based on a stackoverflow post
Describe alternatives you've considered
Changing chromium to load json policies as it does on windows from its intallation directionary. This seems to be a more extensive change.

Patching isOS seems overkill. The relevant functions seems to be around here: https://github.com/chromium/chromium/blob/fd8a8914ca0183f0add65ae55f04e287543c7d4a/components/policy/core/common/policy_loader_win.cc#L177-L183


Allow operator to set prefix used in HTML includes

This will improve usability with POST and when the content is served via proxy on another URL.
The operator should be able to set the prefix via the configuration, where default is "." - meaning relative paths are used (same as now).

The prefix can now be set using configuration property resources_url_prefix.

@jakubklimek We should use this on demo instance, to allow experiment using third party proxy gates.

@skodapetr done


Initial Commit

Initial Commit, moved server/client code from the gentleman cli

@Gabs00 could you create a new PR into the branch develop? Thanks!


Systems in tests

Uses System instead of ase.Atoms in the tests. Closes #167.

📚 Documentation preview 📚: https://metatrain--238.org.readthedocs.build/en/238/

Replaced ase.io.read with our reader


Explain Snapshots graph on Replication dashboard

https://www.cockroachlabs.com/docs/dev/admin-ui-replication-dashboard.html#other-graphs
If/when we want to document the Snapshots graph on the Replication dashboard, here's a good summary from @dt:

Usually the nodes in a raft group stay synchronized by following along the log message by message — e.g. 1) a=3 2) b=4 3) a=1 4) c=5, … — , however if a node is far enough behind (e.g. if it fell behind or is a new node getting up to speed), rather than send the individual messages that changed the range, we can send it a snapshot of range, e.g. at time 4) a=1, b=4, c=5 and then it can start following along from there instead. (edited)
generated is when a node makes one of these, applied is when a node got one and used it. preemptive means we anticipated a node was going to want such a snapshot (e.g. about to be added to a raft group) and sent it ahead of time. (edited)

We may want to change the labels because "normal" snapshots should in fact be rare. The vast majority of snapshots should be preemptive. Internally we refer to non-preemptive snapshots as "raft-initiated". Seeing a lot of "normal" snapshots actually indicates a problem: some replicas are struggling to keep up with write traffic.


Fix antipattern in asdl initialization.

Codecov Report
Base: 27.84% // Head: 27.82% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (0257a11) compared to base (e2dd30e).
Patch coverage: 50.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           stable     #523      +/-   ##
==========================================
- Coverage   27.84%   27.82%   -0.03%
==========================================
  Files          64       67       +3
  Lines        5756     5747       -9
  Branches      317      317
==========================================
- Hits         1603     1599       -4
+ Misses       4082     4077       -5
  Partials       71       71

Impacted Files
Coverage Δ

library/asdl/asdl.hpp
100.00% <ø> (ø)

library/asdl/parse_interactive_istream.cpp
0.00% <0.00%> (ø)

library/asdl/parse_interactive_string_view.cpp
0.00% <0.00%> (ø)

library/virtual_machine/process_context.cpp
14.85% <50.00%> (+0.14%)
:arrow_up:

library/asdl/parse_module_istream.cpp
100.00% <100.00%> (ø)

library/asdl/parse_module_string_view.cpp
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Not working with fileClass, only with Map to Tag

When using fileClass:: , the 'Table view | Fileclass fields | Fileclass settings' is available, but the icon next to each file doesn't appear so the 'Fields of ...' dialog box doesn't show.

Kind of same problem for me : fileClasses are created, but I cannot access them outside the "mapWithTag:true" activated. And the "Table view | FileClass Fields | FileClass" settings are not available neither. As a result, I am not able (1) to select a class for any new file (Add class to XX / No result found - whereas several classes are available); (2) to access any of the settings for a class.


test: prepare proposal block data size

Closes https://github.com/celestiaorg/celestia-app/issues/3573
Note: we could add an explicit conditional at the end of prepare proposal like:
	if getSize(txs) > req.BlockDataSize {
		panic("the size of the proposal block data exceeds the block data size")
	}

but I'm not sure that actually gets us much. Instead of panic'ing in celestia-core (like it does currently), it'll panic in celestia-app. Since this is a change to prepare proposal which has already been audited, I'm hesitant to add the conditional unless reviewers feel strongly that it is helpful.

Is this some form of defensive programming in the instance that we modify the algorithm such that there is a possibility that bytes are added and that therefore this test would catch it?

Is this some form of defensive programming in the instance that we modify the algorithm such that there is a possibility that bytes are added and that therefore this test would catch it?

Unfortunately I don't think this test would actually catch it because it doesn't fuzz inputs into prepare proposal. If we wanted to be defensive then we would have to add the explicit check in prepare proposal (see PR description).
This PR is motivated by added a test case to for the audit finding. It communicates the "intention" of prepare proposal but doesn't actually enforce any new constraints.

Yeah, I'm a little confused by it's existence. Are you saying it's there just to communicate to people that we don't expect PrepareProposal to somehow increase the bytes returned?

Yea exactly

Discussed with @cmwaters offline and we decided this informational audit finding should just be acknowledged and the unit test doesn't add much value.


Add Configuration of VM storage path on VMware

CC can configure VM storage of Virtualbox. We should also add configuration VM storage path of VMware.

Fixed in Master

Windows:

Brought a peer up using vmware on the other vm storage.
fixed in prod


Internal server error during fetch language which does not exist

$ exercism fetch A
2015/10/07 17:54:23 an internal server error was received.
Please file a bug report with the contents of 'exercism debug' at: https://github.com/exercism/x-api/issues

We can definitely do better with that error! Thanks.

Deploying a fix now.

Thanks

Seems that error still exists. But in other manner:
 ✘ alex@bender > ~/ > exercism f c
2015/10/08 14:57:35 an internal server error was received.
Please file a bug report with the contents of 'exercism debug' at: https://github.com/exercism/x-api/issues
 ✘ alex@bender > ~/ > exercism f clang
2015/10/08 14:57:37 unable to fetch problems (HTTP: 400) - no language track 'clang'
 ✘ alex@bender > ~/ >

btw, how can I fetch problems for C?

C isn't an active track yet, but it's possible to fetch individual exercises for tracks that haven't been launched yet.
Thanks for uncovering these bugs, btw! These are things that we didn't have tests for.

Hm. The issue in the C track is non-utf-8 characters in one of the problems.

OK, I've narrowed the problem down and we will fix it in the C repo https://github.com/exercism/xc/issues/16
I'm going to close this again, but if you see any other weird errors, please holler!

I will.

This should be fixed, btw. It might now be possible to get results with exercism fetch c.

Thanks, it is works.


No module named 'faq'

Hi!
I followed your instructions, but when starting the server I get ModuleNotFoundError: No module named 'faq'
I did install it:
pip freeze | fgrep faq
django-easy-faq==1.1

Any idea?

This was an issue for an older version of this app. I will have to look into this for version 1.1.
Thanks for reporting

yes

It does, thanks!


Use the time types instead of a u64

Use the Instant and Duration types defined in #110
This isn't nearly done. I simply use total_millis where we should probably continue to pass the time type along. I'd like to get feedback and make sure I'm going the right direction before making all of the changes.

@whitequark would you like all the changes to the TCP timestamps etc to be in this PR too? If so, I'm assuming they should be in a separate commit

@dlrobertson Sure, separate commit seems fine.

Updated all the things!
Added commits that update the socket modules to use time etc.

A few things before this should be merged.

From<::std::time::Instant> isn't implemented. Adding this now
from_system_time should be From<::std::time::SystemTime> now that it isn't a result type.
Question: Should Instant::now() be a "wrapper" for ::std::time::Instant::now() or what is currently done with the UNIX_EPOCH?

Updated and I didn't change Instant::now. I think sticking with the current implementation makes more sense.

@m-labs-homu delegate+
@dlrobertson r=me once the comments above are fixed

@m-labs-homu r=whitequark

:pushpin: Commit cbedd74 has been approved by whitequark

:hourglass: Testing commit cbedd743db9bef88e4e0dd8e15bf042fa38c0ae4 with merge c418b60b5db93753999ae51d33ec4dc1d1631c69...

:sunny: Test successful - status-travis
Approved by: whitequark
Pushing c418b60b5db93753999ae51d33ec4dc1d1631c69 to master...

Thanks! Great work.


Group Issues/Milestones

Feature Request

What is your feature?

Support the ability to create and assign issues/milestones at a group level.
Issues example with GitLab:

Milestones example with GitLab:

How would implementing this feature improve your experience?

allows for creating and organizing issues relating to multiple projects at a group level, reducing the need to create a dedicated project repository or utilize a third party solution such as Trello

Do you plan on submitting a pull request?

[ ] Yes, I intend on submitting a pull request
[X] No, I will not submit a pull request

I think this is related #1872


Google cloud platform

Continuing the work from #33.
i think the general process we want is:

[x] Merge this into a branch in this repo
[x] Make sure that the branch still works as expected against Oracle
[x] Merge the Ansible PR into a branch in that repo
[x] Make sure that this repo still works against that branch for Oracle
[ ] Test the GCP Terraform workflow
[ ] Test the GCP Terraform+Ansible workflow
[ ] Add some automated tests
[ ] Merge all

This branch now works fine against Oracle. The difference needed is that you now need to add oracle-cloud-infrastructure to any terraform commands you run, e.g.:
terraform init oracle-cloud-infrastructure
terraform plan oracle-cloud-infrastructure
terraform apply oracle-cloud-infrastructure

and the terraform.tfvars example file now needs to be copied from oracle-cloud-infrastructure/terraform.tfvars.example
The next step is to merge the Ansible changes into a branch and make sure that these continue to work on Oracle.

Given that we now have to specify for each terraform command whether we are working with Oracle or Google, e.g.:
$ mv oracle-cloud-infrastructure/terraform.tfvars.example terraform.tfvars
$ terraform apply oracle-cloud-infrastructure

or
$ mv google-cloud-platform/terraform.tfvars.example terraform.tfvars
$ terraform apply google-cloud-platform

do we instead want to rename these directories to shorter names oracle and google or oci and gcp so that it becomes:
$ mv oracle/terraform.tfvars.example terraform.tfvars
$ terraform apply oracle


Flare-VM Install Fails

I've been using Flare vm for a while now, but when I did a fresh Windows 7 install via my VM, and used the latest Flare-VM repo, I'm now getting this error, which I have never got before. I don't even know where to start here, so any help would be appreciated.

After some trial and error, updating PowerShell to 5.1 and adding this line of code to the two .ps1 files allowed everything to run smoothly.


Pass Authorization header to RR backend

Using a PR as the way to discuss this, given that it's pretty simple and I needed it anyway.
An alternate approach could be to have a configurable whitelist of headers that are passed-on to the backend, but I'm not convinced we need to do that unless we have a need for it.

Remote read already supports bearer tokens from the config file, see https://prometheus.io/docs/prometheus/latest/configuration/configuration/#remote_read

Yes I know, but this is for a scenario (SaaS-like) where different clients are making queries, each with their own token.

Prometheus explicitly doesn't offer authorisation related features, nor should the same query give different results for different people. A more typical way to approach this is via a label that you ensure is present on all selectors.

A more typical way to approach this is via a label that you ensure is present on all selectors.

Do you have any examples of anyone doing that? It seems like a pretty bad user experience to have to maintain tokens in your PromQL queries (and questionable security-wise).

Prometheus explicitly doesn't offer authorisation related features

Well, it  does offer some authorisation-related features, such as the one you pointed out in your first comment above. My understanding is that Prometheus doesn't want to authenticate or participate in auth processes (which makes sense), but this PR is  just about relaying a header.

Do you have any examples of anyone doing that?

I believe Cortex does this. The label is never visible to the user, it's injected into their PromQL for them via whatever API server they're using once they're authenticated. These sort of setups often use Prometheus components as a library, rather than as a binary.

Well, it does offer some authorisation-related features, such as the one you pointed out in your first comment above.

That's client authentication, we don't offer any server auth.

this PR is just about relaying a header.

It's still an authorisation feature, and also a layering violation (e.g. it doesn't work with the local tsdb), and a new way users could complicate their setup that would make supporting them more difficult. Any proposal involves passing opaque data from PromQL down to remote read is going to have the same issues. We don't allow collectors to get information from the original HTTP /metrics request for similar reasons.

Thanks for the explanations, makes sense.


Adding some UI manifest attributes as parameters to the CreateManifest class.

This should allow easier customization of the launcher UI because you
can tell FXLauncher to put the UI customizations in the manifest for you.
The parameters that I added—which already existed in Manifest—are

updateText (--update-text)
updateLabelStyle (--update-label-style)
progressBarStyle (--progress-bar-style)
wrapperStyle (--wrapper-style)

I realized after I updated my project that frknikiz in PR #144 had already included the updateText parameter. Sorry to move your code, but now it's in the same order as it is in the fxlauncher.FXManifest class. Hopefully, we're all good.

Thanks @edvin, really appreciate it.


Different Type Errors between webpack compile and vue-tsc lint

Hi There,
I am trying to setup a vue project with vue-3 and typescript. Development and Build is done with the following chain:

vue-loader
ts-loader (transpileOnly)
babel-loader

For linting during webpack dev / build, I use https://github.com/TypeStrong/fork-ts-checker-webpack-plugin.
The config is:
new ForkTsCheckerWebpackPlugin({
      // don't block webpack's emit to wait for type checker in development mode, errors only visible inside CLI
      async: !isProduction,
      typescript: {
        diagnosticOptions: {
          semantic: true,
          syntactic: true,
        },
        extensions: {
          vue: {
            enabled: true,
            compiler: '@vue/compiler-sfc'
          }
        },
        // Set the tsconfig.json path
        configFile: './tsconfig.json',
      },
      eslint: {
        files: './src/**/*.{ts,js}' // required - same as command `eslint ./src/**/*.{ts,tsx,js,jsx} --ext .ts,.tsx,.js,.jsx`
      },
    }),

This setup works so far. No I am trying to setup lint-staged and es-lint to work also for my vue sfc files. I added your plugin to my setup.
My Problem is now, that I get different type errors when running my webpack build (no errors) and a bunch of errors, when running vue-tsc --noEmit.
My TsConfig:
{
  "compilerOptions": {
    "target": "es2019",
    "module": "esnext",
    "strict": true,
    "jsx": "preserve",
    "importHelpers": true,
    "moduleResolution": "node",
    "allowJs": true,
    "skipLibCheck": true,
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "resolveJsonModule": true,
    "sourceMap": true,
    "baseUrl": ".",
    "paths": {
      "@/*": [
        "src/*"
      ]
    },
    "lib": ["es2015", "es2016", "es2017", "es2018", "dom"]
  },
  "include": [
    "src/**/*.ts",
    "src/**/*.tsx",
    "src/**/*.vue",
    "tests/**/*.ts",
    "tests/**/*.tsx"
  ],
  "exclude": [
    "node_modules"
  ]
}

I assume, that this is caused by the different tools using different typescript options?
Do you have any idea / input? Is vue-tsc automatically using my tsconfig.json from the project root?

It seem is a similar problem to #77? Can you try use --project flag to a definite tsconfig?


Better error messages when trying to use PRO features in FREE

Currently what the dev sees when trying to use a PRO feature inside FREE is one of

"None of the views in the given array exist."
"Cannot find 'image' field view in any of the regular locations."

These messages are NOT clear enough. They should say something like "This is a PRO field. Use a FREE field instead, or purchase and use backpack/pro".
We've avoided doing this... because it would bloat up backpack/crud with the information of "what's inside backpack/pro" but... seeing the issues and imagining the frustration... I don't think we can avoid that... I think it'll be better to bloat up the codebase a bit... but provide a good Developer Experience.
Originally posted by @tabacitu in https://github.com/Laravel-Backpack/PageManager/issues/123#issuecomment-1037224537

Partially fixed in https://github.com/Laravel-Backpack/CRUD/pull/4315 for the most common scenario - but to close this entirely, we should do a roundup of what's happening when one tries to use ANY PRO feature, not just fields.

fields
chart widget
filters
List - export buttons
List - details row
List - PRO columns
PRO operations (eg Clone)

And assess if it's enough, or we should do better.

Let's close this in favor of #4315