Example for dummies?

Hey there, I'm really digging the possibilities of glide, but I have no idea how to get it working.
Would you mind lending a bit of assistance?
My main goal is to be able to use URL queries to retrieve a processed image. This is what I have so far for 'watermark.php':
// for security
use League\Glide\Signatures\SignatureFactory as SignatureFactory;
use League\Glide\Signatures\SignatureException as SignatureException;
use League\Glide\Urls\UrlBuilderFactory as UrlBuilderFactory;

// for cacheing
use League\Flysystem\Adapter\Local;
use League\Flysystem\Filesystem as Filesystem;
use League\Glide\ServerFactory as ServerFactory;

//response
use League\Glide\Responses as Responses;
use League\Flysystem\FilesystemInterface as FSI;

// local variables
$imgPath = $install_path . $rel_path_to_image;
$watermarkPath = $install_path . "assets/img/watermark.png";
$cachePath = $install_path . "cache";

$server = ServerFactory::create([
    'source' =>   new Filesystem(new Local($install_path)),               // Source filesystem
    'source_path_prefix' => "assets/img/logos/",     // Source filesystem path prefix
    'cache' =>       new Filesystem(new Local($install_path)),            // Cache filesystem
    'cache_path_prefix' =>  "cache/league",     // Cache filesystem path prefix
    'group_cache_in_folders' =>  true, // Whether to group cached images in folders
    'watermarks' =>    new Filesystem(new Local($install_path)),          // Watermarks filesystem
    'watermarks_path_prefix' => "assets/watermarks",  // Watermarks filesystem path prefix
    'driver' =>    'imagick'              // Image driver (gd or imagick)
]);

// Set using setter method
$server->setPresets([
    'small' => [
        'w' => 200,
        'h' => 200,
        'fit' => 'crop',
    ],
    'medium' => [
        'w' => 600,
        'h' => 400,
        'fit' => 'crop',
    ]
]);

I've tinkered with this for about two days without luck. Do I have to setup re-direct rules with .htaccess? Do my image links need to include "watermark.php" -- something like "watermark.php?f=image.png&w=500&h=320"?
Anything you could help clarify would be much appreciated.

I ran into this issue this morning. I had to configure Nginx so that all my images threw a 404, and then redirected that 404 to PHP.
So instead of hitting my actual source image /images/something.jpg I used /uploads/something.jpg which 404's in Nginx and then used my PHP to route it to my PHP file, in my case a FilesController.

Hi @davidyell thanks for sharing. I'm actually using Apache and not Nginx. That said, I think I've finally got things working.
I'm using Glide in combination with mikecao/flight, successfully.
For posterity, here is what I'm working with (not to be mistaken for production quality code):
<?PHP  // -*- mode: php; -*-

$script_path = realpath(basename(getenv("SCRIPT_NAME")));
$slash = explode('/', getenv("SCRIPT_NAME"));
$current_filename = $slash[count($slash) - 1];
$install_path = str_replace($current_filename, '', $script_path);// Fixing the host URL.
$host_url = str_replace($current_filename, '', getenv("SCRIPT_NAME"));
$includes = $install_path . 'includes/' . PATH_SEPARATOR . $install_path . 'app/php/';
set_include_path($includes); // tells PHP engine where to look for included files
require_once $install_path . 'app/php/composer/autoload.php';

// non-printing
include_once 'move.php';
include_once 'gettype.php';
include_once 'cache.php';
include_once 'filelist.php';

// for security
use League\Glide\Signatures\SignatureFactory as SignatureFactory;
use League\Glide\Signatures\SignatureException as SignatureException;
use League\Glide\Urls\UrlBuilderFactory as UrlBuilderFactory;

// for cacheing
use League\Flysystem\Adapter\Local as Adapter;
use League\Flysystem\Filesystem as Filesystem;
//use League\Flysystem\FilesystemInterface as Interface;
use League\Glide\ServerFactory as ServerFactory;
use League\Glide\Responses\SymfonyResponseFactory as Response;

// $filesystem = new Filesystem(new Adapter($install_path.'assets/img/logos'));
// var_dump($filesystem->has('aflac.png'));
// die();

// variables

$sourcePath = $install_path . "assets";
$cachePath = $install_path . "cache";
$watermarkPath = $install_path . "assets/img/watermarks";

//$exception = new Exception('Danger, Will Robinson!', 100);

$fs = new Filesystem(new Adapter($sourcePath));
// image processing
$server = ServerFactory::create([
    'source' => $fs,                // Source filesystem
    //'source_path_prefix' =>      // Source filesystem path prefix
    'cache' => new Filesystem(new Adapter($cachePath)),                 // Cache filesystem
    //    'cache_path_prefix' =>       // Cache filesystem path prefix
    'group_cache_in_folders' => true,  // Whether to group cached images in folders
    'watermarks' => new Filesystem(new Adapter($watermarkPath)),            // Watermarks filesystem
    //    'watermarks_path_prefix' =>  // Watermarks filesystem path prefix
    'driver' => 'imagick',               // Image driver (gd or imagick)
    'max_image_size' => 2000*2000,         // Image size limit
    //    'defaults' =>                // Default image manipulations
    //    'presets' =>                 // Preset image manipulations
    //    'base_url' =>                // Base URL of the images
    'response' => new Response()              // Response factory
]);

$server->setDefaults([
    'mark' => 'logo.png',
    'markw' => '30w',
    'markpad' => '5w',
]);

$server->setPresets([
    'small' => [
        'w' => 200,
        'h' => 200,
        'fit' => 'crop',
    ],
    'medium' => [
        'w' => 600,
        'h' => 400,
        'fit' => 'crop',
    ]
]);

$map = array(
    "size" => array(
        "large" => "",
        "medium" => "medium/",
        "small" => "small/"
    ),
    "type" => array(
        "logos" => "img/logos/"
    )
);

Flight::set("flight.views.path",$install_path."views/");

// route views
Flight::route('GET /v/@PHP', function($PHP) {
    global $server;

    $path = Flight::get("flight.views.path").
          $PHP.".php";
    //
    if (file_exists($path)) {
        Flight::render($PHP);
    } else {
        echo "
You done goofed!
";
    }
});

// route for media
Flight::route('GET /i/@type/@item', function($type,$item) {
    global $server;
    $path = $GLOBALS["map"]["type"][$type].
          $item;

    $fileExists = $server->sourceFileExists($path);

    if($fileExists) {
        $server->outputImage($path,$_GET);
    }

});

// must come after all Flight stuff!!!!1
Flight::start();

?>


MySQL's JSON column is serialized in a wrong way

What steps will reproduce the problem?
Install MariaDB 10.3.17, and create a json column, then save a PaymentIntent (https://github.com/stripe/stripe-php/blob/master/lib/PaymentIntent.php  ) object into it.
What is the expected result?
{
    "id": "pi_1Hn54bCadkDflulLUsGqg40B",
    "amount": 1000,
    "object": "payment_intent",
    ....
}

What do you get instead?
Stripe\PaymentIntent JSON: {
    "id": "pi_1HoBQoCadkDflulLbveqgcPh",
    "object": "payment_intent",
    "amount": 3000,
    ....
}

Additional info
On our local environment, using MySQL Server version: 8.0.19, the behavior is correct, we get the right value saved in the database column.
What is more interesting, is that if we try to save an actual array, i.e by explicitly calling PaymentIntent::toArray then Yii errors out with:
{name: "PHP Notice", message: "Array to string conversion", code: 8, type: "yii\base\ErrorException",…}
code: 8
file: "/home/dev/httpdocs/web/vendor/yiisoft/yii2/db/ColumnSchema.php"
line: 157
message: "Array to string conversion"
name: "PHP Notice"
stack-trace: [,…]
0: "#0 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/ColumnSchema.php(157): yii\base\ErrorHandler->handleError()"
1: "#1 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/mysql/ColumnSchema.php(49): yii\db\ColumnSchema->typecast()"
2: "#2 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/QueryBuilder.php(380): yii\db\mysql\ColumnSchema->dbTypecast()"
3: "#3 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/mysql/QueryBuilder.php(258): yii\db\QueryBuilder->prepareInsertValues()"
4: "#4 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/QueryBuilder.php(350): yii\db\mysql\QueryBuilder->prepareInsertValues()"
5: "#5 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/Command.php(478): yii\db\QueryBuilder->insert()"
6: "#6 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/Schema.php(432): yii\db\Command->insert()"
7: "#7 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/ActiveRecord.php(600): yii\db\Schema->insert()"
8: "#8 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/ActiveRecord.php(566): yii\db\ActiveRecord->insertInternal()"
9: "#9 /home/dev/httpdocs/web/vendor/yiisoft/yii2/db/BaseActiveRecord.php(678): yii\db\ActiveRecord->insert()"
10: "#10 /home/dev/httpdocs/web/app/api/modules/v1/modules/event/controllers/OrdersController.php(366): yii\db\BaseActiveRecord->save()"
11: "#11 [internal function]: api\modules\v1\modules\event\controllers\OrdersController->actionCharge()"
12: "#12 /home/dev/httpdocs/web/vendor/yiisoft/yii2/base/InlineAction.php(57): call_user_func_array()"
13: "#13 /home/dev/httpdocs/web/vendor/yiisoft/yii2/base/Controller.php(180): yii\base\InlineAction->runWithParams()"
14: "#14 /home/dev/httpdocs/web/vendor/yiisoft/yii2/base/Module.php(528): yii\base\Controller->runAction()"
15: "#15 /home/dev/httpdocs/web/vendor/yiisoft/yii2/web/Application.php(104): yii\base\Module->runAction()"
16: "#16 /home/dev/httpdocs/web/vendor/yiisoft/yii2/base/Application.php(392): yii\web\Application->handleRequest()"
17: "#17 /home/dev/httpdocs/web/app/init.php(188): yii\base\Application->run()"
18: "#18 /home/dev/httpdocs/web/public_html/api/index.php(15): require('/home/dev/httpd...')"
19: "#19 {main}"
type: "yii\base\ErrorException"

The documentation at https://www.yiiframework.com/doc/guide/2.0/en/db-active-record is not really clear on how exactly we should save json data into a json column, shouldn't it accept an array/object/etc and then just json_encode it when it saves it? Because if that's the case, it isn't working correctly.
I also tried passing a json encoded string, but this resulted into escaped json saved into the database, which at this point i don't know if it's caused by Yii or by the database engine.

Q
A

Yii version
<IP_ADDRESS>

PHP version
7.4

Operating system
CentOS 7

Please hold with this, i think i know what is going on.

Okay, so it's the database engine...
create table testing_json ( the_content JSON );

show create table testing_json
# result
CREATE TABLE `testing_json` (
 `the_content` longtext CHARACTER SET utf8mb4 COLLATE utf8mb4_bin DEFAULT NULL
) ENGINE=InnoDB DEFAULT CHARSET=latin1

So MariaDB silently fails to create the JSON column and creates a longtext instead.
but wait, there is more. If you read the docs at https://mariadb.com/kb/en/json-data-type/ they say:
JSON is an alias for LONGTEXT introduced for compatibility reasons with MySQL's JSON data type. MariaDB implements this as a LONGTEXT rather, as the JSON data type contradicts the SQL standard, and MariaDB's benchmarks indicate that performance is at least equivalent.

So while they implement all the functionality for json, their data type is actually a longtext.
This is why Yii cannot detect the right the column type, after all, for Yii this is a longtext column which only accepts text. This is also the reason why when I pass an array to the property it errors out with the array to string conversion error.
What are our options here?
Could we wrap our values into \yii\db\JsonExpression maybe? Or what options do we have, except changing the database engine from mariadb to mysql?

Additionally, please beware that the casting done by yii\db\mysql\JsonExpressionBuilder::build will fail for mariadb as the JSON data type does not exists.

https://www.yiiframework.com/extension/sam-it/yii2-mariadb?

@samdark - Thanks, i didn't know about that link.
Since I don't actually need to search the json right now, but just store it, I will use the longtext option for now, so from my point of view, you can close this issue, but I think I raised a good problem that should be covered by the framework and documented.

I think we can explicitly link to yii2-mariadb in DB drivers list.

@twisted1919 for now for such problem I used such solution:
class JsonModel extends ActiveRecord
{
    public function beforeSave($insert)
    {
        $this->json_field = json_encode($this->json_field);
        return parent::beforeSave($insert);
    }

    public function afterFind()
    {
        $this->json_field = json_decode($this->json_field);
        parent::afterFind();
    }
}

what a great post discussing one man's issues on his job. Thank God nobody wasted their time


digital envelope routines:EVP_DecryptFinal_ex:bad decrypt:evp_enc.c:531

There is an incompatibility between the openssl Version OpenSSL 1.1.0f and OpenSSL 1.0.2m-dev.
I have encrypted some files on my laptop with openssl on Ubuntu 16.04 and debian 8 and migrated now to Debian 9.  After the migration to debian 9 with openssl 1.1.0f i got the error in the title and was no more be able to decrypt my files.  I must install a older version of openssl to decrypt my files.
You can test ist with the following Script:
#!/bin/bash
echo "Test openssl with different Versions"
dsnin="/tmp/testfile.txt"
echo "Testfile for openssl" >$dsnin
cat $dsnin
openssl version
openssl aes-256-cbc -a -salt -k 12345678 -in $dsnin -out $dsnin.enc
echo "File after decrypt"
/home/xx/opt/bin/openssl version
/home/xx/opt/bin/openssl aes-256-cbc -d -a -k 12345678 -in $dsnin.enc -out $dsnin
cat $dsnin
The output when i running this script is:
./testopenssl.sh
Test openssl with different Versions
Testfile for openssl
OpenSSL 1.1.0f  25 May 2017
File after decrypt
OpenSSL 1.0.2m-dev  xx XXX xxxx
bad decrypt
140633576617624:error:06065064:digital envelope routines:EVP_DecryptFinal_ex:bad decrypt:evp_enc.c:531:
Caution: This script is testing the opposit way as my error is, because i get the error from old to new version. There should be no difference in the Data between Versions, because the user could not decrypt all files and encrypt it again, every time a new update arrives.

The default digest changed from MD5 to SHA256.  So add "-digest md5" to 1.1.0 if you need compatibility with older versions or data.

sorry, I mean use the "-md" flag.


Invalid syntax hightlighting

The ; character should close the syntax of the string
Actual

ah, i think this used to be the syntax in an older version, nice find @pleshevskiy


Briankang tpuv5e gemma2b training

Notebook using TPUv5e to fine-tune Gemma. Based on the distributed Gemma tuning example on our Gemma docs: https://ai.google.dev/gemma/docs/distributed_tuning
Requirements to enable this to successfully run during automated testing: Vertex custom model training quota for TPUv5e,
Kaggle tokens and keys supplied for the notebook to execute, and disk size of 200GB+

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[x] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[x] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

If you are opening a PR for Community Notebooks under the notebooks/community folder:

[ ] This notebook has been added to the CODEOWNERS file under the Community Notebooks section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

If you are opening a PR for Community Content under the community-content folder:

[ ] Make sure your main Content Directory Name is descriptive, informative, and includes some of the key products and attributes of your content, so that it is differentiable from other content
[ ] The main content directory has been added to the CODEOWNERS file under the Community Content section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

The test is failing on Kaggle access. The integration test currently doesn't have the capability to set the Kaggle key automatically. Opened enhancement b/330199110 to track.


[FEAT]: Auto update profile option

Is your feature request related to a problem?

[ ] The feature should solve a problem.

Describe the Problem your feature should fix.
If you are in a profile and you modify the configuration of the plugins it is not saved automatically, if I change the profile I lose the current configuration.
Describe the feature you would like to have.
A checkbox in the profile selection so that that profile is automatically updated when modifying something within obsidian.
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

Thanks for your request I know about this problem. But at the moment I don't know how to fixes this problem. I have no event found that gets triggered when settings are changed. I hope I can fix this Issue in the future. I would appreciate if anyone has a idea how to fix.
For now make sure you press the "Save profile" button before switching profile.

What if before switching to the new profile the current one is saved automatically (according to your auto-save configuration)?

Is it possible to detect when you close the settings window?
It could also be a good time to save.
I was telling you about the time period because if you implement the ability to use it in several vaults, perhaps this would avoid problems.

The best would be to get the event when Obsidian saves the settings. Closing the settings window would also be a good option. I need to read the documentation and code a little more closely to find out if there is a way to detect that settings have been changed or the window has been closed. This doesn't seem to be a basic functionality of the Obsidian plugin interface.
As mentioned, saving would be best if triggered by an event. However, when loading profiles, it might be useful to have a period of time to check for changes in profiles so that the changed settings are reflected when two instances are running in parallel with the same profiles.

I thought about it again and this is what I came up with:

When profiles should be saved as discussed above.

The best would be to get the event when Obsidian saves the settings. Closing the settings window would also be a good option. I need to read the documentation and code a little more closely to find out if there is a way to detect that settings have been changed or the window has been closed. This doesn't seem to be a basic functionality of the Obsidian plugin interface.

[x] Need to find a event that triggers when settings changed or when settings closed

When profiles should be loaded

However, when loading profiles, it might be useful to have a period of time to check for changes in profiles so that the changed settings are reflected when two instances are running in parallel with the same profiles.

The problem I have with this approach is that Obsidian needs to be reloaded for changes in the settings to take effect and when this keeps happening it's very annoying.

[ ] Option 1: Use a interval based reload and first comparing the files and if nothing has changed, nothing is loaded.
[ ] Option 2: Include paths to vaults where there are active and update the files in the save process. Problem opened vaults don't know that they need to reload.

If any other ideas I wish to hear them.

I've been thinking about it and I think a way to simplify the loading could be something like the workspaces plus plugin https://github.com/nothingislost/obsidian-workspaces-plus. In case a profile has changed in another vault, it may appear red and when pressed it updates.
Maybe you have to take into account that there may be conflicts with the auto save if you have 2 open vaults... I don't know, I think the workspace system is simple and you will avoid problems.
I give you an example of conflict:

Open vault 1
Open vault 2
You modify some configuration in 1.
You change to the vault 2 window and without being updated you change something in the configuration.
What profile is the good one now? ..

Also a good option

@javiavid
I have added a new pre-release version. This contains parts of the features discussed here. You can test it with the BRAT plugin.
How to BRAT:

"Add Beta plugin with frozen version"

https://github.com/4Source/settings-profiles-obsidian-plugin
0.5.0-beta.0

Thanks
I can't get it to update from one vault to another automatically, how does it work?

It also seems that it does not save the plugin configuration well. I attached a test.
https://github.com/4Source/settings-profiles-obsidian-plugin/assets/7627504/59e4e939-20b3-4f37-80ab-499a836eab9b

The auto save and reload is currently not included. But i added a status bar Icon that currently displays the profile is up-to-date you can click on that to reload. The save needs to be done manually in settings. But now you there is at least a way you can see the profile is up-to-date. Sorry, I should have said that more clearly.

No problem, it's great to have it in the status bar, I think it's very convenient, although I think it doesn't work well or maybe I did something wrong. I attached a video.
https://github.com/4Source/settings-profiles-obsidian-plugin/assets/7627504/722a3263-2c66-4bbc-806e-f011777c738c

Interesting I will take a look at it. Can you tell me the name of the plugin you are using

File Color

Added new Beta
Change BRAT to:

https://github.com/4Source/settings-profiles-obsidian-plugin
0.5.0-beta.1

Actually you should test this version directly, I have the autoSync feature implemented. Loading doesn't happen by itself, but I think it's better that way.
Change BRAT to:

https://github.com/4Source/settings-profiles-obsidian-plugin
0.5.0-beta.2

No problems found on your side?

No, at least for now, it is true that I have not spent much time testing with work, but for now I have it in some test vaults in case something strange happens.
Maybe it would be good to launch it in the community and have more people try it.

I think this will happen during the day I need to do small changes


Editing Bots Status

Question again, does Discordeno support Bot Statuses? All I can find related to that is the presenceUpdate property in Events so how would that be done?
I will also make a list of bugs I come across and once enough i'll make a bug report with them all, just to ease of the clutter of bug reports I make for each bug
Originally posted by @NTMNathan in https://github.com/Skillz4Killz/Discordeno-bot-template/issues/10#issuecomment-629577180

@NTMNathan This has been added and deployed. Example: https://github.com/Skillz4Killz/Discordeno/blob/b2866d89169706e6899dca14d38d8d9393a1e3f4/mod.ts#L17


Added 1.12 implementation and fixed Testing Module

Thank you for your pull request, I really appreciate the work! I'll review and merge the code tomorrow

Im closing this, because I'm no longer working on it.


Bump framework and governance version

$Subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/1585755818

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/1585755818
Status: success


Add optional epoch to IConnect message

Related to issue: https://github.com/microsoft/FluidFramework/pull/4899
Add epoch in IConnect message, so that is the client has it and wants to match on server, then it can pass it.

Should we call this "fileEpoch/storageEpoch/summaryEpoch"? To make them more explicit?

@GaryWilber I think odsp already checks epoch field? Will it require a change on odsp server?

Should we call this "fileEpoch/storageEpoch/summaryEpoch"? To make them more explicit?

@GaryWilber I think odsp already checks epoch field? Will it require a change on odsp server?

Should we call this "fileEpoch/storageEpoch/summaryEpoch"? To make them more explicit?

@GaryWilber I think odsp already checks epoch field? Will it require a change on odsp server?

We currently check epoch. Changing it would require server changes.

Should we call this "fileEpoch/storageEpoch/summaryEpoch"? To make them more explicit?

@GaryWilber I think odsp already checks epoch field? Will it require a change on odsp server?

We currently check epoch. Changing it would require server changes.


Restore OneDrive session with existing access token

Is there any way to restore session in OneDrive if I have an access token which was received earlier (e.g. I authorized in OneDrive on another device and got access token and then I need restore session with the same token)?

When requesting scopes by adding the wl.offline_access scope the authentication response will include a refresh token which can be used to acquire a new access token without a interactive login flow.
If you use the MSAAuthenticator, this will automatically store/manage/use this refresh token.  To see a full application example that uses the offline access scope of scenario see, the OneDrive Explorer for Android, and look here for the  MSAAuthenticator configuration.

Closing out this issue, please reopen this is still a problem.


[WIP] Add action update test that expects failure when missing file is specified

Add action update test that expects failure when missing file is specified

@dubeejw - Ready for review.

IPG 269

Fixes #565

IPG 'action update with missing file` test case results look good
     [exec]     Starting test Wsk Action CLI should reject action update when specified file is missing at 2016-07-07 20:28:16.514
     [exec]
     [exec] bluemix.cli.BluemixWskBasicTests > Wsk Action CLI should reject action update when specified file is missing STANDARD_OUT
     [exec]     deleting updateMissingFile
     [exec]
     [exec] bluemix.cli.BluemixWskBasicTests > Wsk Action CLI should reject action update when specified file is missing PASSED

I'm not sure I understand this commit - It seems to reverse a previous commit: https://github.com/mdeuser/openwhisk/commit/5295f25514e455c1eaa492bce519b33f8b1a554d#diff-1a947144b344358c7bf829db2b035bdc?

The proposed Go error message when a nonexistent file name is provided is errMsg := fmt.Sprintf("File '%s' is not a valid file or it does not exist: %s", artifact, err).  The os.Stat() is pretty much an existence test.  The existing action create test case is looking for 'not a valid file' in the "missing file" error message, so the Go CLI included that bit of text in that error message.  Here's the test case snippet.
    it should "reject create with missing file" in {
        wsk.action.create("missingFile", Some("notfound"),
            expectedExitCode = MISUSE_EXIT).
            stderr should include("not a valid file")

Merged as https://github.com/openwhisk/openwhisk/commit/f31c0920a8c4535e4804212a1b0787d4a445097e.


Disabling RDP from the internet

Is it possible to block the cluster's RDP ports from being access from the internet? Maybe limiting them to specific IP's. Are the ports necessary or can I disable them entirely? I have not been able to do so from the portal after I create the cluster and LB. I used one of the ARM templates and the instructions from this guide. Thanks!

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

ID: 98d553a1-59ed-c80c-5a72-e8237b042490
Version Independent ID: 4e7e025d-ca39-3099-57b7-449ec2914da7
Content: Remote connect to an Azure Service Fabric cluster node
Content Source: articles/service-fabric/service-fabric-cluster-remote-connect-to-azure-cluster-node.md
Service: service-fabric
GitHub Login: @aljo-microsoft
Microsoft Alias: aljo

@kaiyoshino you could use a Network Security group on the subnet of your Vnet your cluster is using to block traffic. Simply add a list of allowed IPs or a range of IPs and the rest will be blocked.

Okay. I will give this a shot. Thanks.

Just checking if you figured out a way to disable this RDP.


Nose test error

When I run nosetests cvxpy for cvxpy on Python 3, I get 26 errors and one failure.
Some of the error messages complain about SCS not being installed and  an undefined symbol: GOMP_parallel. I was wondering how I can fix it.
======================================================================
ERROR: test_sdp_problem (cvxpy.tests.test_semidefinite_vars.TestSemidefiniteVariable)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/conda/lib/python3.5/site-packages/cvxpy/tests/test_semidefinite_vars.py", line 46, in test_sdp_problem
    result = p.solve()
  File "/opt/conda/lib/python3.5/site-packages/cvxpy/problems/problem.py", line 201, in solve
    return self._solve(*args, **kwargs)
  File "/opt/conda/lib/python3.5/site-packages/cvxpy/problems/problem.py", line 323, in _solve
    kwargs)
  File "/opt/conda/lib/python3.5/site-packages/cvxpy/problems/solvers/scs_intf.py", line 96, in solve
    import scs
  File "/opt/conda/lib/python3.5/site-packages/scs-1.2.6-py3.5-linux-x86_64.egg/scs.py", line 4, in <module>
    import _scs_indirect
ImportError: /opt/conda/lib/python3.5/site-packages/scs-1.2.6-py3.5-linux-x86_64.egg/_scs_indirect.cpython-35m-x86_64-linux-gnu.so:
 undefined symbol: GOMP_parallel
======================================================================
FAIL: Tests that chaining constraints raises an error.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/conda/lib/python3.5/site-packages/cvxpy/tests/test_constraints.py", line 242, in test_chained_constraints
    self.assertEqual(str(cm.exception), error_str)
AssertionError: "'LeqConstraint' object has no attribute '__nonzero__'" != 'Cannot evaluate the truth value of a con[44 chars]= 0.'
- 'LeqConstraint' object has no attribute '__nonzero__'
+ Cannot evaluate the truth value of a constraint or chain constraints, e.g., 1 >= x >= 0.

Somebody else asked about this here: https://groups.google.com/forum/#!topic/cvxpy/cIYiUABZzVA
I suspect you have an old version of cvxpy. How did you install it?

I used:
conda install mkl
conda install -c cvxgrp cvxpy
conda install nose

You can try it here:
https://hub.docker.com/r/bsamadi/cvxpy-notebook/

It seems that something is wrong with the numpy installation and it has to be reinstalled. I found the solution here:
https://github.com/ContinuumIO/anaconda-issues/issues/720
I added these commands after cvxpy and nose installation commands:
# Reinstall numpy
RUN conda remove nomkl
RUN conda install mkl
RUN conda install -f  numpy
RUN conda remove -n python2 nomkl
RUN conda install -n python2 mkl
RUN conda install -n python2 -f  numpy
Now, I can run all the 1730 tests in Python 2 successfully. In Python 3, I get only one failure:
/opt/conda/lib/python3.5/site-packages/dill/dill.py:149: ResourceWarning: unclosed file <_io.FileIO name='/dev/null' mode='rb' closefd=True>
FileType = type(open(os.devnull, 'rb', buffering=0))
/opt/conda/lib/python3.5/site-packages/dill/dill.py:150: ResourceWarning: unclosed file <_io.TextIOWrapper name='/dev/null' mode='r' encoding='UTF-8'>
TextWrapperType = type(open(os.devnull, 'r', buffering=-1))
/opt/conda/lib/python3.5/site-packages/dill/dill.py:151: ResourceWarning: unclosed file <_io.BufferedRandom name='/dev/null'>
BufferedRandomType = type(open(os.devnull, 'r+b', buffering=-1))
/opt/conda/lib/python3.5/site-packages/dill/dill.py:152: ResourceWarning: unclosed file <_io.BufferedReader name='/dev/null'>
BufferedReaderType = type(open(os.devnull, 'rb', buffering=-1))
/opt/conda/lib/python3.5/site-packages/dill/dill.py:153: ResourceWarning: unclosed file <_io.BufferedWriter name='/dev/null'>
BufferedWriterType = type(open(os.devnull, 'wb', buffering=-1))
FAIL: Tests that chaining constraints raises an error.
Traceback (most recent call last):
File "/opt/conda/lib/python3.5/site-packages/cvxpy/tests/test_constraints.py", line 242, in test_chained_constraints
self.assertEqual(str(cm.exception), error_str)
AssertionError: "'LeqConstraint' object has no attribute 'nonzero'" != 'Cannot evaluate the truth value of a con[44 chars]= 0.'

'LeqConstraint' object has no attribute 'nonzero'

Cannot evaluate the truth value of a constraint or chain constraints, e.g., 1 >= x >= 0.

What is the output of "cvxpy.version"?

0.4.5
This is a fresh install using pip.

The current version is 0.4.6.
On Thu, Oct 27, 2016 at 1:33 PM Behzad Samadi<EMAIL_ADDRESS>wrote:

0.4.5
This is a fresh install using pip.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cvxgrp/cvxpy/issues/305#issuecomment-256760685, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfyIJ4rhiJMaernIpgojizdRFeIOnF6ks5q4QqQgaJpZM4Kgj0O
.

Will conda get the 0.4.6 version soon?

Oops, sorry. I thought it was on conda because I accidentally uploaded it to my personal channel instead of cvxgrp. The cvxgrp channel now has 0.4.6. Sorry about that.

Thank you. I still get 0.4.5 with conda. I should probably wait a couple of days.

Try conda install -c cvxgrp cvxpy=0.4.6
You can see it's uploaded here: https://anaconda.org/cvxgrp/cvxpy

I get this error:
PackageNotFoundError: Package not found: '' Package missing in current linux-64 channels:

cvxpy 0.4.6*

I forgot to upload linux versions. The test error was truly bizarre, due to a difference between pip install and conda build. Everything should work now if you install cvxpy 0.4.8.

Thank you for looking at this. With conda install -c cvxgrp cvxpy, conda on Linux installs the 0.4.8 version! and on Windows it installs the 0.4.2 version.
I installed the 0.4.6 version with conda install -c cvxgrp cvxpy=0.4.6
The single failure in Python 3.5 is still the same. On Python 2.7, the tests run successfully.
FAIL: Tests that chaining constraints raises an error.
Traceback (most recent call last):
File "/opt/conda/lib/python3.5/site-packages/cvxpy/tests/test_constraints.py", line 242, in test_chained_constraints
self.assertEqual(str(cm.exception), error_str)
AssertionError: "'LeqConstraint' object has no attribute 'nonzero'" != 'Cannot evaluate the truth value of a con[44 chars]= 0.'

'LeqConstraint' object has no attribute 'nonzero'

Cannot evaluate the truth value of a constraint or chain constraints, e.g., 1 >= x >= 0.

Yes, you need to update to cvxpy 0.4.8 for the test error to go away. The
conda installation doesn't work on windows.
On Sat, Oct 29, 2016 at 7:04 PM Behzad Samadi<EMAIL_ADDRESS>wrote:

Thank you for looking at this. With conda install -c cvxgrp cvxpy, conda
on Linux installs the 0.4.8 version! and on Windows it installs the 0.4.2
version.
I installed the 0.4.6 version with conda install -c cvxgrp cvxpy=0.4.6
The single failure in Python 3.5 is still the same. On Python 2.7, the
tests run successfully.
FAIL: Tests that chaining constraints raises an error.
Traceback (most recent call last):
File
"/opt/conda/lib/python3.5/site-packages/cvxpy/tests/test_constraints.py",
line 242, in test_chained_constraints
self.assertEqual(str(cm.exception), error_str)
AssertionError: "'LeqConstraint' object has no attribute 'nonzero'" !=
'Cannot evaluate the truth value of a con[44 chars]= 0.'

'LeqConstraint' object has no attribute 'nonzero'

Cannot evaluate the truth value of a constraint or chain
constraints, e.g., 1 >= x >= 0.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cvxgrp/cvxpy/issues/305#issuecomment-257127376, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfyIHaxoAvwlqEadHel5MRJ7i9LK2kLks5q4_smgaJpZM4Kgj0O
.


sync/waitgroup: why v == int32(delta) is required when judging if the waitgroup is misused?

What version of Go are you using (go version)?

$ go version
go version go1.14.2 darwin/amd64

Question?
Why v == int32(delta) is required here? Is w != 0 && delta > 0 insufficient to judge if a waitgroup is misused for Add called concurrently with Wait?

Because delta is declared in the function declaration as int. There are no few implicit conversion rules in Go, so int and int32 are not the same type, even if the program is compiled on a 32 bit platform.
Unlike many projects on GitHub, the Go project does not use its bug tracker for general discussion or asking questions. We only use our bug tracker for tracking bugs and tracking proposals going through the Proposal Process.
Please see https://golang.org/wiki/Questions for good places to ask questions.


Remove spanning deletions if the deletion was removed

@ronlevine commented on Tue Jul 19 2016
Port https://github.com/broadinstitute/gsa-unstable/pull/1417.

@ronlevine commented on Wed Jul 20 2016
Need to port GenotypeGVCFs before testing the proposed changes in GenotypingEngine.java.

@ronlevine commented on Thu Sep 29 2016
@droazen Should this be a gatk-protected ticket instead since it depends on GenotypeGVCFs?

Issue moved to broadinstitute/gatk #2193 via ZenHub


Minor: 2nd argument to stache system renderer is assumed to be a scope object, but it might not be.

https://github.com/bitovi/canjs/commit/1d7be0a30e4893f660fab3ab071dc7411c18cd93#commitcomment-12607871
See what https://github.com/bitovi/canjs/blob/master/view/stache/html_section.js#L68 does.

This is fixed in minor.


Typo in ssl/s3_srvr.c

Commit 69f682374868b introduced a typo. My pull request fixes.

Pull request was applied. Closing issue.


Huggingface mlx-community/Phi-3-mini-128k-instruct-4bit model cannot be loaded

I am interested in running the mlx-community/Phi-3-mini-128k-instruct-4bit model with swift, but it cannot be loaded. Here is the output I am seeing:
➜  mlx-swift-examples git:(main) ./mlx-run llm-tool eval --model mlx-community/Phi-3-mini-128k-instruct-4bit -m 100000 -p "<|system|>
You are a helpful assistant.<|end|>
<|user|>
How to explain Internet for a medieval knight?<|end|>
<|assistant|> "
Error: typeMismatch(Swift.String, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "rope_scaling", intValue: nil), _JSONKey(stringValue: "long_factor", intValue: nil)], debugDescription: "Expected to decode String but found an array instead.", underlyingError: nil)

Note that the mlx-community/Phi-3-mini-4k-instruct-4bit model can be loaded. The 128k model includes a non-null rope_scaling field in the config.json, and the 4k model doesn't have a rope_scaling factor. The model 128k load is failing because it can't process the rope scaling

Interestingly we have support for that model in Python but it looks like it is just ignores the rope scaling.
Somehow it still gives pretty good results for short generations 🤔 but presumably it would not work too well beyond 4k

It looks like the ropeScaling is a dictionary of heterogenous types.  We already have:
    var ropeScaling: [String: StringOrNumber]? = nil

I could change this to StringOrNumberOrArray (something like that anyway!)
The code uses the ropeScaling["type"] to figure out what to do and if it isn't a type it recognizes it will just use a ropeScaling of 1.
Specifically it needs to handle:
        "long_factor": [
            1.0700000524520874,
            1.1200000047683716,

I am not sure what is intended with the array -- I guess they are supposed to correspond to the hidden layer counts?  There are also two sets of values here.
Anyway, we could at least enable it to load if we changed this.

I am not sure what is intended with the array -- I guess they are supposed to correspond to the hidden layer counts? There are also two sets of values here.

Those are values per dimension. So you scale each dimension with a unique value. Python implementation is here.
There are two of them because one gets used for short sequences and the other gets used for positions after a certain step in longer sequences.

I think I've fixed this in https://github.com/ml-explore/mlx-swift-examples/pull/155.

This should be working as of #107


change deprecated governator-annotations to replacement (governator-api)

NetflixOSS » eureka » eureka-pull-requests #584 SUCCESS
This pull request looks good


Web components 'apiOrigin' and 'iframeOrigin' props

Components take optional props for apiOrigin and iframeOrigin which will override the production values (currently in config.ts).
Storybook should be updated so that in development an environment picker is available and overrides apiOrigin and iframeOrigin. Document this as optional and used for internal testing

Now with .env files this may be not necessary anymore


Update known_presence_issues to clear CI break

Adding to OpenTelemetry.Exporter.AzureMonitor package path known_presence_issues to fix CI failure

There is another PR https://github.com/Azure/azure-sdk-for-net/pull/14413 to fix this issue.

Closing this since https://github.com/Azure/azure-sdk-for-net/pull/14413/files has been merged


[Feature] Add colors to specific words/phrases in chart title

Bug []
Feature Request [x]
Expected Behavior

When defining the text in the title object, a user should have the option to add specific colours to words/phrases; like how a span is used in HTML. For example:
This is my chart!

Current Behavior

The user only has the option to change the colour of the whole chart title through fontColor.
Context

I'd like this feature as it's a nice preventative attribute to add to a visualisation - it distinguishes important places to look in the corresponding chart.

Tagged as an enhancement. @simonbrunel not sure if we want to go down this route. I almost thing this might be better as an HTML title otherwise we're opening up a never-ending amount of text rendering work

Hi. If you want to customize the title visually, you can implement the title with HTML and CSS. Due to Chart.js using canvas, we can't implement every requested feature cause this will bloat the code.


[DOC] Get Involved webpage

Describe the issue linked to the documentation
The "get involved" webpage still needs a bit of tidying. It currently contains a guide on contribution, the code of conduct and the governance page. The last two are separate issues if they are to be revamped IMO (i.e. #2), but the other contents are up for discussion.
https://aeon-toolkit--598.org.readthedocs.build/en/598/get_involved.html
Suggest a potential alternative/fix
No response

This page was removed, with the contribution guide replacing it as a top level page.


[BUG] SAS tokens cannot be generated when using the "UseDevelopmentStorage=true" connection string

duplicate of #23404


deduce rule for bitwise builtin

Doing a PR just for the deduction rule of the builtin.
Rationale:
Currently, our implementation has a TODO for deduceMemoryCell and the way its implemented would inform the 'shape' of the builtin
see: https://github.com/keep-starknet-strange/cairo-zig/blob/32dc2bfcfe7a6d6d6f9c342cb3085d91266a2db0/src/vm/core.zig#L206-L215
An example implementation from the o.g. python:
https://github.com/starkware-libs/cairo-lang/blob/master/src/starkware/cairo/lang/vm/virtual_machine_base.py#L452-L468
I think it would be good to create a separate issue for lining up deduceMemoryCell and  adding the calls that computeOperands need to be adjusted to make (see: https://github.com/starkware-libs/cairo-lang/blob/master/src/starkware/cairo/lang/vm/vm_core.py#L294-L297).
This separate issue would also define an interface for how builtins register themselves into the vm (see: https://github.com/starkware-libs/cairo-lang/blob//src/starkware/cairo/lang/builtins/bitwise/bitwise_builtin_runner.py#L23)
I'd be happy to create the issue, just let me know!

Doing a PR just for the deduction rule of the builtin.
Rationale: Currently, our implementation has a TODO for deduceMemoryCell and the way its implemented would inform the 'shape' of the builtin
see:
https://github.com/keep-starknet-strange/cairo-zig/blob/32dc2bfcfe7a6d6d6f9c342cb3085d91266a2db0/src/vm/core.zig#L206-L215
An example implementation from the o.g. python: https://github.com/starkware-libs/cairo-lang/blob/master/src/starkware/cairo/lang/vm/virtual_machine_base.py#L452-L468
I think it would be good to create a separate issue for lining up deduceMemoryCell and adding the calls that computeOperands need to be adjusted to make (see: https://github.com/starkware-libs/cairo-lang/blob/master/src/starkware/cairo/lang/vm/vm_core.py#L294-L297).
This separate issue would also define an interface for how builtins register themselves into the vm (see: https://github.com/starkware-libs/cairo-lang/blob//src/starkware/cairo/lang/builtins/bitwise/bitwise_builtin_runner.py#L23)
I'd be happy to create the issue, just let me know!

Yeah I think you can create a separate issue for this. cc @lana-shanghai

Yeah I think you can create a separate issue for this. cc @lana-shanghai

There is already an issue for deducing a memory cell, we can reword it accordingly. Plus we need an issue for builtin runner.


Display and/or specify Ensembl Release and Reference Genome

I was wondering if it will be possible to display and/or specify the Ensembl release and reference genome used.

@sandertan where? In cBioPortal or in the output from the API?

@inodb Hi Ino, I think it would be useful to query the API for both hg19 and hg38. I understand this would be a lot of work to implement, but we sometimes hear that users would like hg38 for their cBioPortal data. Also, Kelsey is working on a PR to cBioPortal to support multiple reference genomes for mutations https://github.com/cBioPortal/cbioportal/pull/3703 .
Regarding the output, I can imagine it would be useful to display the used Ensembl Release and the reference genome. This would explain the context of the returned result.

Hi Sander,
Yeah we were thinking to run a hg38 instance as well at some point. Similar
to how ensembl has multiple instances. It would prolly just be passing some
parameter to genome-nexus-importer and setting some config for
genome-nexus. Then hg19 or hg38 would be in the URL.
On Fri, Apr 6, 2018, 4:32 AM Sander Tan<EMAIL_ADDRESS>wrote:

@inodb https://github.com/inodb Hi Ino, I think it would be useful to
query the API for both hg19 and hg38. I understand this would be a lot of
work to implement, but we sometimes hear that users would like hg38 for
their cBioPortal data. Also, Kelsey is working on a PR to cBioPortal to
support multiple reference genomes for mutations
cBioPortal/cbioportal#3703
https://github.com/cBioPortal/cbioportal/pull/3703 .
Regarding the output, I can imagine it would be useful to display the used
Ensembl Release and the reference genome. This would explain the context of
the returned result.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/genome-nexus/genome-nexus/issues/156#issuecomment-379186098,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABRa9Ie67mbVT40DzR95IwtVPbA1JQduks5tlygzgaJpZM4S-bPq
.


Addition of Wark C of E School

Addition of a school (Wark C of E School): https://www.warkprimaryschool.co.uk/

@p0ep0e First stage of verification passed, official domain is warkprimaryschool.co.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://www.warkprimaryschool.co.uk/contact.html

Thanks for the update, let me know if you need anything further from the school

@p0ep0e Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Remove IDE project files from repo.

IML files for Android Studio are generated files and should not be checked into source. They are always derived from the build.gradle files which represent the source of truth in the project. Their format is subject to variation as different versions of Studio will generate a different XML structure in the files.
This commit removes the checked-in files and adds the type to .gitignore.

See #371

Do you have a source saying that this files are not useful?

@g-ortuno Certainly, see below…
Comments from an Android Studio tools team developer indicating they are generated content:
https://code.google.com/p/android/issues/detail?id=77983#c7
https://code.google.com/p/android/issues/detail?id=156708#c6
StackOverflow answer form another tools team developer indicating this + the entire .idea/ directory should not be tracked in a Studio project:
http://stackoverflow.com/a/27570950/246461
…If I can find the Google I/O video where they made the same statement on stage, I'll post it here as well.

LGTM.

@cco3 beat me to the lgtm


How to read the data from the Axiom device using python?

I need to read the data that the axiom touchpad is throwing but only a part of the data I am receiving with the help of pywinusb library. I have connected the device via usb.

I need to read the data that the axiom touchpad is throwing but only a part of the data I am receiving with the help of pywinusb library. I have connected the device via usb.

Hi,
Are you able to provide more details?
We use the python package hid to handle the low level USB comms. If you are on Windows, it is a little more involved to setup. Refer to the USB Interface section of the README.md.
If you are running on Linux, you may have to be a super user to access the USB interface.
Best Regards, Mark

I want to read the data from the touch pad device in Windows OS and there are two types of data.

Touchnetix Press Data
Touchnetix Control Data
[https://github.com/rene-aguirre/pywinusb/blob/master/examples/raw_data.py]
I have used the above mentioned script and able to read the press data but whenever I have tried to read the control data it is not throwing me anything. After some research it seems I need to send some data to the device to establish a connection over usb and then maybe I can get the data.
So, any help from your side is very much appreciated.

As you have found, the USB bridge has multiple interfaces:

TBP Control
TBP Press
(Digitizer - if enabled)

After a reset, the bridge will start sending data to the Press interface without the host needing to send any requests/commands. The data coming from this interface contains XYZ information for up to 5 targets.
To get the bridge to send data to the Control interface, the host needs to send a command to put the bridge into what we call Proxy mode. In this mode, the bridge will autonomously read all reports from aXiom (from u34) and send these to the host.
To put the bridge into this mode, this function can be called.
As a simple example, it can be used as follows:
from axiom_tc import axiom
from axiom_tc.USB_Comms import USB_Comms

usb = USB_Comms()
ax = axiom(usb)
usb.set_proxy_mode()

@aakash-chakraborty1995 is your issue resolved?

Thanks a lot. The issue got resolved.


Commenting out code working incorrectly /* {{ $aa.foo }} */

When i try to comment out this code
{{ $aa.foo }}

it makes it like this
/* {{ $aa.foo }} */

Correct format should be
{{/* $aa.foo  */}}

I have the same problem, any idea how to fix this or is there an alternative?


Permission Nodes?

does this have a perm node or just based off op status?

This plugin is not ready for production use yet, no support will be provided yet

ahh okay, sorry, was trying it out, really neat btw!!

@FienixEC it is based off op status

@TheRoyalBlock the plugin is in beta rn
@FienixEC I will be adding perms soon

Thanks @ZeroDevOfficial  Good Work!!

Wait... @ZeroDevOfficial u own PlexusStudio?

Yes I am the owner @FienixEC

i went to ur main page and noticed ur from seattle, lol me too! Thats awesome!

nice

@FienixEC hey
Commit please

I am going to lock this and it will be a todo for now


Restore full env range for gdt 1.1.0

Prematurely dropped some of the PHP versions on test matrix in https://github.com/phase2/grunt-drupal-tasks/pull/345, that's a change for v2, not for v1.1.

Give or take some things not compatible with D8 anymore :)


ASP.NET Core Entity Framework Migration in Separate App

@anthonychu @julielerman We talked about this approach to migrations at the MVP Summit. Below is a link to a minimum demo this is quick to clone and run.
https://github.com/bigfont/StackOverflow/tree/master/AspNetCoreEntityFrameworkMigrations

thanks. I was curious about your use of the ApplyMigrations method in the startup of your aspnet core app.


Tightly pinned dependencies

Hi,
Looking at the setup.py file, it looks like the following are all required runtime dependencies, all of which need to be pinned very precisely:
requirements = [    "Cython==0.29.23",    "cvxpy==1.1.12",    "fbprophet==0.5",    "holidays==0.9.10",  # 0.10.2,    "ipykernel==4.8.2",    "ipython==7.1.1",    "ipywidgets==7.2.1",    "jupyter==1.0.0",    "jupyter-client==6.1.5",    "jupyter-console==6.",  # used version 6 to avoid conflict with ipython version    "jupyter-core==4.7.1",    "matplotlib==3.4.1",    "nbformat==5.1.3",    "notebook==5.4.1",    "numpy==1.20.2",    "osqp==0.6.1",    "overrides==2.8.0",    "pandas==1.1.3",    "patsy==0.5.1",    "Pillow==8.0.1",    "plotly==3.10.0",    "pystan==<IP_ADDRESS>",    "pyzmq==22.0.3",    "scipy==1.5.4",    "seaborn==0.9.0",    "six==1.15.0",    "scikit-learn==0.24.1",    "Sphinx==3.2.1",    "sphinx-gallery==0.6.1",    "sphinx-rtd-theme==0.4.2",    "statsmodels==0.12.2",    "testfixtures==6.14.2",    "tornado==5.1.1",    "tqdm==4.52.0"]

My question is - why pin them so tightly, and are all of them really necessary? E.g. do I really need sphinx-gallery? Such tight pins make it very difficult to integrate into any existing project. Why not just require a lower bound for many/most of these?

Sorry, looks like I accidentally opened this twice, dupe #11


Only log client facing http routes

We use our own http server for server level functions, as well as client facing functions.  With the addition of continuous queries, the http log is very noisy.  This change now only logs end user generated traffic.

+1 on green build.


Recreate assay metadata

The assay metadata file was folded into common. This file should be added back and imported by common.

+1

@david4096 @kozbo @ejacox Any comments/way forward on this?


Increased default timeout to 120secs

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #41   +/-   ##
=========================================
  Coverage     98.91%   98.91%
  Complexity      394      394
=========================================
  Files            51       51
  Lines          1015     1015
=========================================
  Hits           1004     1004
  Misses           11       11

Impacted Files
Coverage Δ
Complexity Δ

src/Config.php
100% <ø> (ø)
15 <0> (ø)
:arrow_down:

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 7e722f9...b74b50c. Read the comment docs.


Enable limiting stack traces to failure section

The report gets pretty verbose when both stack traces and the failure section are enabled since the stack traces end up being reported twice. Personally, I think it would be easier to read the report if the stack traces were only shown in the failure section, but there's currently no combination of options that would enable this.

+1

To limit stack traces to failure section, you could use the option:
{
  displayStacktrace: 'summary',
  ...
}

Will be available in v2.2.0 and v1.2.0, soon to be released!


Exception translation

Would it be possible to implement translation for a future version?

Not sure that is really the job of this library. You could catch them in your app and translate them there.

I suppose it's not possible to extend \Upload\Exception for each validation so each exception can have it's on class name? You could add an array to Codeguy\Upload where validation exceptions are stored, or extract a part of isValid() so programmer's can wrap it in their own try/catchblock.


Cross-signing & SSSS bootstrap

This is a placeholder issue for tracking.
We will let riot-web iterate on UX before attacking it.

Closed in favour of #3292, #3293, #3296, #3299.
The UX has changed. Cross-signing and 4S are now fully unrelated.


please add me as collaborator to the exercise repositories

added, please check your mail for the invite and accept it. Thanks!


move noteName method from main to navbar.

What is this PR for?
Notebook list doesn't show when you reload(F5) the zeppelin web page not in home.
What type of PR is it?
Bug Fix
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-863
How should this be tested?
Outline the steps to test the PR here.
Screenshots (if appropriate)
before

after

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

It works well. Thanks for fixing this 👍

Good catch, LGTM

LGTM. good fix

LGTM

Tested and LGTM.
Merge it into master branch!


[TOA-311] Quota API returns low-priority node count

Adds the low-pri node count to the return object of the quota api.

Just fyi, I think this is included in https://github.com/DataBiosphere/terra-workspace-manager/pull/1006, which I plan to merge soon.


fixed uint to ulong from LLVMOrcTargetAddress

see: https://github.com/llvm-mirror/llvm/blob/master/include/llvm-c/OrcBindings.h#L34

Thank you.


Offscreen canvas transfer to dedicated worker

Does somebody know the most efficient way of doing this?
I cant Ser/De canvas to transfer it to worker.
var htmlCanvas = document.getElementById("canvas");
var offscreen = htmlCanvas.transferControlToOffscreen();

var worker = new Worker("offscreencanvas.js");
worker.postMessage({canvas: offscreen}, [offscreen]);

Closing this. See #1823 for details


Progress bar doesn't disappear when a progress-command fails

Type get-help eggs
Observe progress bar doesn't go away when an error is returned.

Also shows up under:

This is now showing up even on successful completions.


5.0.1  build in Xcode 12.4 error: failed to build module 'AudioKit' from its module interface; it may have been damaged or it may have triggered a bug in the Swift compiler when it was produced.

Confirm you should be adding the bug here
If your issue is implementing AudioKit in your own code, post your problem to StackOverflow with the #AudioKit hashtag.
If you can make the issue happen on AudioKit 5's Cookbook app then you can create an issue directly on that repository as well.
GitHub Issues should only be used for confirmed problems with AudioKit and
well-defined feature requests.
Sponsors will get more attention: https://github.com/sponsors/aure
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
See / hear error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Details (please complete the following information):

Type: [e.g. iPhone6, iPad, Mac]
OS: [e.g. iOS13.1]
AudioKit Version [e.g. 5.0]

Additional context
Add any other context about the problem here.

AudioKit 5.1 is out and most of this issue is just the boilerplate text.


Documenting the form in tabpages makes blank

Hello,
I have used dockpanel in the tabpages of tabcontol. But when i document the form in tabpages it becomes blank. Docking Right left works proper.

Could not reproduce this with HEAD on master branch.

WindowsFormsApplication1.zip
Attached a sample project that I used, which could not show the issue you described.

Ok.. Thanks Lex.. My Document Style was causing the issue


support for providing paths within git for Dockerfile, Dockstore.cwl, etc

+1 this has been requested by UCSC and would allow direct support of tools organized like https://github.com/BD2KGenomics/cgl-docker-lib or https://github.com/common-workflow-language/workflows

:+1:

Also, you shouldn't require the CWL be named Dockstore.cwl.  Since we're allowing a user to specify a path you should allow anything (although enforcing it ends in .cwl seems reasonable).

I think this is done now.  @victoroicr?

Finished in 0.2.1.


Add flexibility on to snake case

Support both foo_bar_1 and foo_bar1.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (c242fb7) to head (d561ced).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##                main      #222   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity       612       612
===========================================
  Files             55        55
  Lines           1891      1898    +7
===========================================
+ Hits            1891      1898    +7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


About the method of upsampling

Thanks for your significiant work. I would like to ask the method of upsampling the depth hypotheses. Is it the nearest interpolation or the bilinear interpolation? I think both of them would influence the accuarcy, especially in edge regions.
Look forward for your reply!

Hi, among the different scales, we directly upsample the depth map of the previous scale and then recompute which bin each depth is in. When upsampling the depth maps, we use bilinear interpolation.

Thanks a lot for your kind reply.


Plugin Verifier can't find Gateway module

I'm working on a plugin for Gateway and I have a Gatway module declared as a dependency. buildPlugin and runIde tasks are fine, but runPluginVerifier  complains about missing dependencies:
Plugin com.coder.gateway:1.0.0 against GW-221.5921.14: 1 missing mandatory dependency. 1 usage of internal API Missing dependencies:     com.jetbrains.gateway: Plugin com.jetbrains.gateway is not available in Marketplace https://plugins.jetbrains.com

@fioan89 Could you please add me to the project?

@hsz seems like I don't have permissions to add new people on the project. Is it ok if I just post the content of the plugin.xml?

`

com.coder.gateway
Coder Gateway
Coder
<!-- Product and plugin compatibility requirements -->
<!-- https://plugins.jetbrains.com/docs/intellij/plugin-compatibility.html -->
<depends>com.intellij.modules.platform</depends>
<depends>com.jetbrains.gateway</depends>
<extensions defaultExtensionNs="com.intellij">
    <applicationService serviceImplementation="com.coder.gateway.sdk.CoderRestClientService"></applicationService>
    <applicationService serviceImplementation="com.coder.gateway.services.CoderRecentWorkspaceConnectionsService"></applicationService>
</extensions>
<extensions defaultExtensionNs="com.jetbrains">
    <gatewayConnector implementation="com.coder.gateway.CoderGatewayMainView"/>
    <gatewayConnectionProvider implementation="com.coder.gateway.CoderGatewayConnectionProvider"/>
</extensions>

`
IntelliJ Platform Artifacts Repositories
-> https://plugins.jetbrains.com/docs/intellij/intellij-artifacts.html
pluginGroup=com.coder.gateway
pluginName=coder-gateway
SemVer format -> https://semver.org
pluginVersion=1.0.0
See https://plugins.jetbrains.com/docs/intellij/build-number-ranges.html
for insight into build numbers and IntelliJ Platform versions.
pluginSinceBuild=213
pluginUntilBuild=221.*
IntelliJ Platform Properties -> https://github.com/JetBrains/gradle-intellij-plugin#intellij-platform-properties
Gateway available build versions https://www.jetbrains.com/intellij-repository/snapshots and https://www.jetbrains.com/intellij-repository/releases
platformType=GW
platformVersion=221.5787.23-CUSTOM-SNAPSHOT
instrumentationCompiler=221.5787.23
platformDownloadSources=true
Plugin Dependencies -> https://plugins.jetbrains.com/docs/intellij/plugin-dependencies.html
Example: platformPlugins = com.intellij.java, com.jetbrains.php:203.4449.22
platformPlugins=
Java language level used to compile sources and to generate the files for - Java 11 is required since 2020.3
javaVersion=11
Gradle Releases -> https://github.com/gradle/gradle/releases
gradleVersion=7.4
Opt-out flag for bundling Kotlin standard library.
See https://plugins.jetbrains.com/docs/intellij/kotlin.html#kotlin-standard-library for details.
suppress inspection "UnusedProperty"
kotlin.stdlib.default.dependency=false
`
This is what I have in the plugin.xml and gradle.properties

Looks like this is an issue with Plugin Verifier itself – the com.jetbrains.gateway plugin is not supposed to be resolved in JetBrains Marketplace as it is a module available in the GW release. Please ignore it for now.

Thx... I believe a similar issue is raised by Gateway itself when installing the plugin from the disk, though the install is just fine. But I guess this needs to be raised with Gateway.

please watch https://youtrack.jetbrains.com/issue/MP-4677/Plugin-Verifier-cant-find-Gateway-module-on-Marketplace-bogus-error it's a problem in Plugin Verifier itself


Thanks!

This is obviously not an issue, but a thank you!
I have been working on a project to attach a Behringer X-Touch mini controller to an Pico W, have 10 OLED displays for channel and master and forward MIDI over wifi.
I feel with your work on this (especially USB Host MIDI) I may be much closer to success on my project!

@burtonrodman Thank you for your kind words. I have already software that runs on 2 pico boards that implements the display for each channel and the 7-segment LEDs (VPot mode and timecode) for Mackie control protocol. It uses 9 OLED displays. I got distracted from working on it but I can make my in progress repo public if you think it would be helpful to you. I developed it to work with the Korg nanoKONTROL Studio in Mackie Control mode. It needs 2 Pico boards because driving 9 OLEDs consumes both PIOs, so none are left for the USB host.

that would be awesome to see your work!  i also have a nanoKONTROL, but switched to x-touch mini for the encoders.
it's unfortunate that midi or even proprietary protocol, they don't allow you to query the current position of a fader/knob.  if you know of any workaround (beside jumping or latching on movement) please let me know!
thanks again!

@burtonrodman OK, I will try to get around to exposing something to the world this week.
Sadly, for the nanoKONTROL Studio there is no get current state of the faders and knobs method that I know of.. I use fader pickup (it is one of the processors in this project); fader movements are not sent until the fader moves past the last position
the DAW sent to the controller.

@rppicomidi i know you're using PIO for your USB host port.  i've been reading your PR to tinyusb and if I understand correctly, you've implemented the appropriate functionality for the rp2040 driver.  Am I reading that correctly?  Does your USB MIDI host work on the built-in USB controller (not PIO)?  have you tested this scenario at all?
thanks again for your awesome work!

@burtonrodman Please in future open a new issue if you have a new question so I am able to see it. I don't check my email that often and this issue is marked closed so it doesn't show up normally in my issues list.
MIDI Host can work using the built-in RP2040 USB hardware. You may wish to try this pull request. This project for example uses it.
However, a processor requires both an input and an output, and the RP2040 has only one USB port. That is why I used PIO for the host interface in this project.

@burtonrodman So sorry this took so long, but I when I looked at my pico-mc-display-bridge code I realized it was not in any shape to publish. Then I tried it on my hardware and found my hardware broke and needed to be rebuilt. Then I wanted to add some features. Then I found some more bugs. Then I realized from the photo you posted, your hardware seems to be a bit different from mine. Enough excuses for the delay. Hopefully you will find this useful.


Make Thread::to_userdata return Option<NonNull<u8>>

Thread::check_udata returns NonNull<u8>, and it would probably make sense to return something similar from Thread::to_userdata.

One argument against this, I think, is that Thread::test_udata already does this, and those two functions are related.
Meanwhile, Thread::to_userdata has the Thread::push_light_userdata, which also uses *mut c_void.


Add support for offline website previewing

This is a Feature Request

What would you like to be added

I am on the MacOS with an M1 chip, trying to set up the website locally, but after following the process, it is showing the blank screen. I am following the docs to set up the project locally. I am attaching the SS for the same,

Is there something that I am doing wrong?
Why is this needed
Comments

/retitle Add support for offline website previewing
based on https://kubernetes.slack.com/archives/C09R23FHP/p1701797990349859?thread_ts=1701716259.355739&cid=C09R23FHP

/language en
/sig docs

/remove-language en

Hey @sftim, I am not able to see any preview of the website, as a result it will be really difficult to contribute, is there someone who has encountered this issue earlier and would like to assist me on this?

I'm sorry I'm late on the conversation, Have you tried 'make serve' @neutr0n420?

@neutr0n420 if someone adds this feature, you'll be able to contribute.
In the meantime, you could also try a different connection to the internet.

/triage accepted
/priority backlog
(it's valuable, but other things are even more valuable IMO)

Yep I have tried that @gauravpadam , it is giving me another error something like Error: error building site: failed to acquire a build lock: open /Users/neutron│ /coding/open_source/website/.hugo_build.lock: too many open files              │ make: *** [serve] Error 1

Help with this is welcome.

@sftim Looks like help-wanted label is missing here

/assign

Looks like help-wanted label is missing here

Ah, no: not every issue qualifies for that particular label, even for issues where we'd welcome help.

Should be fixed in https://github.com/kubernetes/website/pull/45375

Is this issue fixed in #45375 ? If yes, then we can close this issue


Run match appstore but nothing happen

When run match appstore he start with a message: Cloning remote git repo... but after this, nothing more happened.
Ruby version
ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-darwin14]
When i force the stop process, i received this message:
 ^C[16:02:43]: 🔒  Successfully encrypted certificates repo
I repeat this process several times, but I got always the same result

Same thing here.

@nmcdonaldd
My problem was here:
with problem: i commented this line code
#type "development" # The default type, can be: appstore, adhoc or development
without problem: i uncommented this line code
type "development" # The default type, can be: appstore, adhoc or development

Just tried that and it doesn't work for me still unfortunately. Thanks for your input, however!

HI @nmcdonaldd Can you share your matchfile? Thanks!


Add key-value argument for dynamic feed config

Hi there,
For my personal usage I have been adding a feature to the feeds config.
I am subscribing to a website that has different "profiles" (a bit like mastodon or instagram).
The website is the same but the content is different for each profile and I would like to subscribe to different profiles as separate feeds.
I could duplicate the feed config in the feeds.toml but I have close to 100 subscriptions so that would become cumbersome.
That is why I implemented this feature which enables the passing of an arbitrary key-value parameter that will be inserted into the config.
Example:
rsspls --parameter username=john
with the config:
[rsspls]

[[feed]]
title = "%<username> - Feed"
filename = "feed_%<username>.rss"

[feed.config]
url = "https://www.example.com/profile/%<username>/"
...

will simply replace the "%" with "john" and then generate the feed as before.
I already opened the PR so thats its easier to discuss, but feel free to reject it if you don't think its applicable.

Hmm I appreciate the solid use case but I not sure introducing templating is the way to go. It kinda feels like an external tool that generated the config would be a better option. It would also have the benefit of fetching multiple feeds in parallel.
I've recently been playing with Nickel and it would be pretty neat for doing this. Of course you could do similar with a Python script or something. E.g.
if you had accounts.json (or accounts.toml) with the accounts you wanted follow like:
{
  "accounts": [
    "one",
    "two",
    "three"
  ]
}

You could generate rsspls.toml using Nickel code like this:
let accounts = import "accounts.json" in

let account_feed = fun account =>
  {
    title = "Posts from %{account}",
    filename = "%{account}.rss",
    config = {
      url = "https://example.com/user/%{account}",
      item = "article",
      heading = "h3",
      link = "h3 a",
      summary = ".post-body",
      date = "time"
    }
  }
in

{
  rsspls = {
    output = "/tmp"
  },
  feed = std.array.map account_feed accounts
}

The config would be generated with nickel export -f toml rsspls.ncl, which from the above two files looks like this:
[[feed]]
filename = "one.rss"
title = "Posts from one"

[feed.config]
date = "time"
heading = "h3"
item = "article"
link = "h3 a"
summary = ".post-body"
url = "https://example.com/user/one"

[[feed]]
filename = "two.rss"
title = "Posts from two"

[feed.config]
date = "time"
heading = "h3"
item = "article"
link = "h3 a"
summary = ".post-body"
url = "https://example.com/user/two"

[[feed]]
filename = "three.rss"
title = "Posts from three"

[feed.config]
date = "time"
heading = "h3"
item = "article"
link = "h3 a"
summary = ".post-body"
url = "https://example.com/user/three"

[rsspls]
output = "/tmp"

Sorry for the late response!
Yes I actually agree, this is not how I've been using rsspls until now but I think you're right that using an external tool will be better for rsspls as it will stay focused.
I will close the PR.
One other problem I might run into, is that when having a lot of "profile page" feeds being scraped from the same website, fetching all of those pages at once in parallel might end up in a rate limit or IP ban from the website.
Would you be interested in a PR to add the option to fetch a single feed? (something like --single-feed feed_name)
I can open a separate issue for this if you prefer.

One other problem I might run into, is that when having a lot of "profile page" feeds being scraped from the same website, fetching all of those pages at once in parallel might end up in a rate limit or IP ban from the website.

Yeah I can't seem to see a low per-domain connection limit like browsers have in reqwest/hyper so this could be a problem. An option to fetch one feed is one possibility. Another would be a rate limit of sorts like --wait in wget:
       -w seconds
       --wait=seconds
           Wait the specified number of seconds between the retrievals.
           Use of this option is recommended, as it lightens the server
           load by making the requests less frequent.

I actually just realized that when dynamically generating the config, I could also just write the single feed I want to fetch with rsspls in the config, making the addition of --single-feed superfluous.
With this I am actually all settled witht the current state.
I see that --wait could be useful in some cases, but I don't need it right now.
Just for context, I am looking into these specific ways of using rsspls as I am actually then serving the rss files locally to my FreshRss instance that I use to keep track of my read items and other feed.
As Freshrss has features to spread out the feed refresh calls, I want to let it trigger rsspls on single feeds.

I/you can close the issue if you feel contempt too.

With this I am actually all settled witht the current state.

Cool. I will close this off then.


Obfuscator not working on Ubuntu 20.04 focal

I downloaded the provided binaries that corresponds to my Linux distribution. But I don't see to get  verible-verilog-obfuscate working.
when I do ./verible-verilog-obfuscate without any arguments, the binary just keeps hanging there. The same happens when I do ./verible-verilog-obfuscate myfile.v out.v .
I tried multiple latest available releases of verible.
Do you have any idea what I am missing here ?
Thank you

Right usage is :
./verible-verilog-obfuscate <myfile.v> out.v


bug(madsim): Kafka mock produces EOF

Describe the bug
Kafka is not meant to produce an EOF even when the data is exhausted
To Reproduce
As seen here: https://buildkite.com/risingwavelabs/pull-request/builds/19271#0186d84a-07af-4cab-9c5b-28b8428ec29a
Expected behavior
No response
Additional context
No response

https://github.com/risingwavelabs/risingwave/pull/8493#issuecomment-1469590643


Crashes when using Save Selected and Changing Page

Steps to reproduce

Search something
Select many Images
Save the images using Save Selected from context menu
Go to next page
Crash

Context
There is nothing in the main.log.
settings.zip
System information

OS: Windows 10 20H2
Grabber version: Nightly

Additional Request
Can you also make it so that the images are deselected as they get downloaded or when all get downloaded.

Additional Request
Can you also make it so that the images are deselected as they get downloaded or when all get downloaded.


Additional split on SAMLResponse split

name: Extra split to remove additional SAML Response Data
about: PR to address special case for some providers
title: ''
labels: ''
assignees: ''

Describe the bug
Some providers add additional information in the SAML Response and this appears to be randomly ordered, causing intermittent issues with validation.
Example:
SAMLResponse=dHJpYnV0ZVN0YXRlbWVudD48L3NhbWw6QXNzZXJ0aW9uPjwvc2FtbHA6.........UmVzcG9uc2U+&RelayState=https://console.aws.amazon.com
In this case RelayState is being included and AWS to base64decode it, which is not succesful, and causes
InvalidIdentityToken: Invalid base64 SAMLResponse (Service: AWSOpenIdDiscoveryService; Status Code: 400; Error Code: AuthSamlInvalidSamlResponseException;
To Reproduce
Steps to reproduce the behavior:

Use the tool such as aws-cli-auth saml -v -d 3600 -p https://foo.example.com --role arn:aws:iam::xxx:role/xxx --principal arn:aws:iam::xxx:saml-provider/xxx -s --cfg-section nonprod
Log in using the correct credentials
Authentication completes successfully
See error

Expected behavior
User able to login with correct credentials.
Screenshots
n/a
Desktop (please complete the following information):

OS: Windows / Linux
Browser: Chrome
Version: all

Additional context
n/a

ah sorry @stephanpieterse - totally missed this

I can merge this in now - though will need to bypass the require signed commits policy, since I have not come back to this in a while I realise it's probably not on your active watchlist.


[WIP] Ignored instrumentation files

This is an initial attempt at including a configuration option to filter files being instrumented by Istanbul.
I've added the configuration at the top level as the coverage configuration is based on a suite which it not available from within Teaspoon::Instrumentation.
This is currently allowing me to define a list of strings which are then used as regular expressions to check against the assets pathname.
Initial discussion was here: #315.

Looks like a good start.. can we put this in the coverage specific configuration? Seems like it's pretty much directly tied to that.

I appreciate the effort, and will merge it when it's moved into that area of the configuration. =)

I was unsure about where it would go in the coverage specific configurations.
If it goes within the Coverage class then its linked to a suite, which is unknown from within Instrumentation. Where do you think it should go? If it put it in Coverage and then look up the configuration assuming default as the suite name?

haha, fair enough. I hadn't thought about that. pfff... thinking on that, but thanks for making the point.

@mikepack how do you feel about this conversation? Do you think it's really bad if we set the current_suite at a Teaspoon level from within the controller so the instrumentation can determine what files to instrument or not? We're hobbled by how sprockets hooks work.

@jejacks0n I'm very unfamiliar with this part of the codebase. We should stay away from a current_suite, IMO. What prevents us from calling Teaspoon.coverage_configs? Is it that we need to know what coverage config is being used (eg default or CI)?

Because you can specify a coverage config at runtime, or by configuring one within your suite.

I'm thinking setting the current_suite is really the only way to accomplish it.. so, I dislike requirejs, have I said that already? haha. ;-P

@jejacks0n thats fine with me. I'll make changes based on the way the matcher works and the other feedback.

Ok, so I appreciate your effort on this, and sorry I haven't merged it yet.
There seems to be a solution that'll work better than this, and you have my apology for leading you down a less optimal path. I'm happy to do this work myself now, but I'll explain it anyway in case you want to give it a shot -- it's not far from this implementation really.
Ok, so instead of putting it at a top level configuraiton, it should be a coverage configuration option.. essentially just moving no_config from suite to coverage. The reason for this, is that the coverage configuration is known globally, and so after some thought it seems like it should technically belong within the coverage configuration anyway. This change does involve more changes than what's in this PR to make the suite understand these concepts through the coverage configuration.

Thats fine, and it does make sense to have this option on the coverage configuration.
This is where I had originally put the option, but I ran into the issue of being unaware of which suite was currently being run when Teaspoon:: Instrumentation.add? was executed. What is your idea for this?

It's not associated to the suite. It's set at the global use_coverage configuration level. That configuration can be specified at run time as well.

Jeremy Jackson

On Mar 5, 2015, at 2:22 AM, Dave Stevens<EMAIL_ADDRESS>wrote:
Thats fine, and it does make sense to have this option on the coverage configuration.
This is where I had originally put the option, but I ran into the issue of being unaware of which suite was currently being run when Teaspoon:: Instrumentation.add? was executed. What is your idea for this?
—
Reply to this email directly or view it on GitHub.

@davestevens thanks to this PR, we've had some discussions internally and have plans to rework the coverage configuration for our 1.0 release (#308). We're thinking of removing all coverage options at the suite level, ie. moving the no_coverage to the coverage block.
Going to close this PR. Happy to reopen if you want to take a stab at getting this into the coverage block. Otherwise, we'll address it in the not-so-distant future.


Remove deprecated IOError

Since python version 3.3 IOError has been deprecated and merged (with various different errors) into OSError.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (bc480fb) 93.47% compared to head (89189aa) 93.43%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3074      +/-   ##
===========================================
- Coverage    93.47%   93.43%   -0.05%
===========================================
  Files           66       66
  Lines        14038    14038
===========================================
- Hits         13122    13116       -6
- Misses         916      922       +6

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks for the PR, @jxdv. This is a vendored package we don't intend to modify at this time.


[firebase_auth] Confirm password reset

Description
This PR gives the ability to confirm password reset in order to control the entire "forgot password" flow from within an app. All of the pieces were there for the platforms, so this PR is just hooking up the plumbing to be able to access from dart.
Related Issues
#1739
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] If the pull request affects only one plugin, the PR title starts with the name of the plugin in brackets (e.g. [cloud_firestore])
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

closing to favor splitting into two separate PRs. one for the platform_interface side, and then when that lands, one for the firebase_auth side.


Updated pull request template to be compatible with other repositories

Description
Updated pull request template to be compatible with other repositories
Fixes #100
Type of change

New feature (non-breaking change which adds functionality)

Testing steps
N/A

Codecov Report

Merging #101 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #101   +/-   ##
=======================================
  Coverage   40.42%   40.42%
=======================================
  Files           7        7
  Lines         141      141
=======================================
  Hits           57       57
  Misses         83       83
  Partials        1        1

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 2fc61f3...10b80c8. Read the comment docs.


feat(ci): add e2e test framework

add e2e test framework

Codecov Report

Merging #72 (81bbec6) into master (8fdac12) will increase coverage by 0.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #72      +/-   ##
==========================================
+ Coverage   80.84%   81.23%   +0.38%
==========================================
  Files          11       11
  Lines         710      746      +36
==========================================
+ Hits          574      606      +32
- Misses        100      102       +2
- Partials       36       38       +2

Impacted Files
Coverage Δ

internal/http/request.go
93.49% <0.00%> (-0.79%)
:arrow_down:

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 8fdac12...81bbec6. Read the comment docs.


Handling of np.nan in test asserts

What?
Dandan raised Q of whether == checks are safe for np.NaN, which occurs in our tests e.g., investigate whether we need to improve handling here
Why?
Ensure test behaviour is as expected

had a little bit of a play with the linked example:
>>> import numpy as np
>>> np.nan == np.nan
False

np.nan != np.nan.  In fact a recommended test for if something is np.nan is to  check x != x
>>> {"a": [np.nan, np.nan], "b": [None, np.nan], "c": [np.nan, None]} == {"a": [np.nan, np.nan], "b": [None, np.nan], "c": [np.nan, None]}
True
>>> {"a": [np.nan, np.nan], "b": [None, np.nan], "c": [np.nan, None]} == {"a": [np.nan, np.nan], "b": [None, np.nan], "c": [None, None]}
False
>>> {"a": [np.nan, np.nan], "b": [None, np.nan], "c": np.nan} == {"a": [np.nan, np.nan], "b": [None, np.nan], "c": np.nan}
True`

comparing dictionaries containing np.nan as values, or as elements in list values does appear to work as expected.
Would appear that this is indeed safe used as it is in the example.

Is the question whether __eq__ is safe for dicts that contain np.nan? Dave shows that it's ok (phew).
But the general question of testing for np.nan - we should use np.isnan rather than an equality match?

I guess that using {}.eq without any documentation promising that behaviour risks a change being implemented which could break tests.  It does not however seem likely to me that this behaviour would be removed, cant imagine it isn't intentional!
Would maybe be more concerned if we had tests which might start to pass when they shouldn't (i.e. silent errors) but the change would cause some of our tests to start failing and would be fairly obvious from the traceback that the dict comparison isn't working.   Personally I feel comfortable leaving this alone now and closing the issue.

I feel comfortable closing this too.


CTS Blog

Create Blog to introduce and provide an overview of CTS.

I think Graham is working this - can you also confirm if targeted at 1.3?

It's going out roughly the same time as 1.2 - but is not ties to the release branch obviously

Blogs are at
overview: https://www.odpi.org/blog/egeria/2019/12/04/what-is-egerias-conformance-test-suite-for-metadata-repositories
detail: https://www.odpi.org/blog/tech-deep-dive/2019/12/04/the-odpi-egeria-open-metadata-conformance-suite-repository-workbench


Injecting parameters to dagster ops/job

Hello,
I have a run like this:
meltano run tap-rest-api-msdk target-jsonl
and a meltano.yaml
plugins:
  extractors:
  - name: tap-rest-api-msdk
    variant: widen
    pip_url: tap-rest-api-msdk
    config:
      api_url: ...
      streams:
       - name: ...
         ...
         params:
           date_after: 01/01/2021
           ...
         primary_keys:
         - id

Is it possible to override parameters and inject them with _defs definitions at any level ?
Reading your code it is not feasible (due to command.split()). And meltano does not allow to inject variable via CLI. Only override meltano.yaml or with environment variable injection which we cannot do since dagster is running.
How would you override on the fly the date_after withtout touching the file ?

Hello, I have a run like this: meltano run tap-rest-api-msdk target-jsonl
and a meltano.yaml
plugins:
  extractors:
  - name: tap-rest-api-msdk
    variant: widen
    pip_url: tap-rest-api-msdk
    config:
      api_url: ...
      streams:
       - name: ...
         ...
         params:
           date_after: 01/01/2021
           ...
         primary_keys:
         - id

Is it possible to override parameters and inject them with _defs definitions at any level ?
Reading your code it is not feasible (due to command.split()). And meltano does not allow to inject variable via CLI. Only override meltano.yaml or with environment variable injection which we cannot do since dagster is running.
How would you override on the fly the date_after withtout touching the file ?

@slamer59 One way we could achieve this is by altering the ìns` of the Meltano run op. https://github.com/quantile-development/dagster-meltano/blob/a6e597e21313ad0f721a6b5d286e73f27ce29946/dagster_meltano/ops.py#L39
If we create some sort of custom typing for Meltano config, this could be injected on the fly by the previous op.

I need to think deeper since I never used ins.
Another way, is to pass an argument from meltano_run_op
def meltano_run_op(command: str, args: dict) ->

and pass it to
  log_results = meltano_resource.meltano_invoker.run_and_log(
        "run",
        MetadataLogProcessor,
        command.split(), **args
    )

etc... to subprocess.Popen
By the way, we dont you build on top of dagster-shell for curiosity.

I need to think deeper since I never used ins. Another way, is to pass an argument from meltano_run_op
def meltano_run_op(command: str, args: dict) ->

and pass it to
  log_results = meltano_resource.meltano_invoker.run_and_log(
        "run",
        MetadataLogProcessor,
        command.split(), **args
    )

etc... to subprocess.Popen

Ah you don't need to inject values at Dagster runtime. Yes then a solution like this would work. It would relate to #18. As you can set config using environment variables.

By the way, we dont you build on top of dagster-shell for curiosity.

I wanted to process the process stdout and stderr in real time to gather Singer.io statistics. But looking at it again, this might be possible with dagster-shell as well.


How can find duplicate contacts? there is no method.

@pethanihiten, the newer version does not have this method but you can find this code in older versions,
Ref: https://github.com/satishbabariya/SwiftyContacts/blob/2.0.0/SwiftyContacts/Classes/SwiftyContacts.swift#L463

ok thanks.

we check this method and apply but return all contact not only duplicate?


Implement Builder Header component

Implement the Builder Header component according to spec: https://www.lightningdesignsystem.com/components/builder-header/

Here's what I'm thinking for the interface.  What do you think?
BuilderHeader:

Name
Type
Description

assistiveText.icon
string
Assistive text for the title icon

className
string
Additional class names

children
Element[]
Structure of the header. BuilderHeaderNav (optional) or BuilderHeaderToolbar (optional)

events.onClickBack
function
Callback for when "Back" is clicked

events.onClickHelp
function
Callback for when "Help" is clicked

pageType
string
Title of the builder

BuilderHeaderNav:

Name
Type
Description

children
Element[]
Dropdown or Button elements

BuilderHeaderToolbar:

Name
Type
Description

assistiveText.actions
string
Assistive text for the actions section of the toolbar

children
ButtonGroup[]
Button groups in the toolbar

onRenderActions
functions
Renders the actions section of the toolbar

Example:
Here's an example using the example given in the spec.
<BuilderHeader
	assistiveText={{ icon: 'Builder' }}
	pageType="Page Type"
	title="App Name"
>
	<BuilderHeaderNav>
		<Button
			iconCategory="utility"
			iconName="settings"
			label="Link"
		/>
		<Dropdown
			iconCategory="utility"
			iconCategory="page"
			label="Dropdown"
		/>
	</BuilderHeaderNav>
	<BuilderHeaderToolbar>
		<ButtonGroup>
			<Button
				assistiveText={{ icon: 'Undo' }}
				iconCategory="utility"
				iconName="undo"
			/>
			<Button
				assistiveText={{ icon: 'Redo' }}
				iconCategory="utility"
				iconName="redo"
			/>
		</ButtonGroup>
		<ButtonGroup>
			<Button
				assistiveText={{ icon: 'Cut' }}
				iconCategory="utility"
				iconName="cut"
			/>
			<Button
				assistiveText={{ icon: 'Copy' }}
				iconCategory="utility"
				iconName="copy"
			/>
			<Button
				assistiveText={{ icon: 'Paste' }}
				iconCategory="utility"
				iconName="paste"
			/>
		</ButtonGroup>
	</BuilderHeaderToolbar>
</BuilderHeader>

Approved!
I don't think we want to force classnames like slds-button_icon-border on consumers. This should probably be a new variant or something like that.

After working on implementing this, here are some changes that I thought needed to be made to the component API.  The biggest change is to use a new link component instead of <Button>.
After reading the "Button vs. Link" explanation, I thought that this makes sense since the spec indicated that the nav items should be links.
BuilderHeader:

Name
Type
Description

assistiveText.backIcon
string
[NEW] Assistive text for the back icon

assistiveText.helpIcon
string
[NEW] Assistive text for the help icon

assistiveText.icon
string
Assistive text for the title icon

children
Element[]
Structure of the header. BuilderHeaderNav (optional) and BuilderHeaderToolbar (optional)

className
string
Additional class names

events.onClickBack
function
Callback for when "Back" is clicked

events.onClickHelp
function
Callback for when "Help" is clicked

labels.back
string
Label for the "Back" action

labels.help
string
Label for the "Help" action

labels.pageType
string
Label for the page type

labels.title
string
Title of the builder

style
object
[NEW] Custom styles to be passed to the component

BuilderHeaderNav:

Name
Type
Description

children
Element[]
Dropdown or Button BuilderHeaderNavLink elements

[NEW] BuilderHeaderNavLink:

Name
Type
Description

assistiveText.icon
string
Assitive text for the icon next to the link text

iconCategory
'action','custom','doctype','standard','utility'
Name of the icon category

iconName
string
Name of the icon

iconPath
string
Path to the icon

label
string
Text for the link

onClick
function
Triggered when the link is clicked

BuilderHeaderToolbar:

Name
Type
Description

assistiveText.actions
string
[NEW] Assistive text for the actions section

children
ButtonGroup[]
Button groups in the toolbar

onRenderActions
function
Renders the actions section of the toolbar


default code example key > value not byte array

TypeError: key: expected bytes or bytearray, but got 'str'

I found the same. MacOS, python 3.7. Py37's hmac raises TypeError if the key isn't bytes or bytesarray. Simply encoding the key didn't seem to work, though I'm not sure, as I don't have a reference "good" call for this httpsig api to compare.

Thanks for reporting, this is now fixed.

Thank you!


Add CLI support for importing an EKS cluster

In the context of importing EKS clusters (and potentially clusters from other providers), we want our CLI (kubectl-gs) to support the importing of a cluster.
### Tasks
- [ ] Spec
- [ ] Implementation
- [ ] Documentation

@marians are you okay with closing this? or is this still needed?

No priority currently, closing