CON-2394_20220228_update-schema

Updated schema.py to include new APIs (specifically, those that allow QA on individual annotations). Also added some trivial code examples for use of those APIs.
Changes:
FLIR/conservator/generated/schema.py

Updated schema

examples/dataset_annotation_qa.py
examples/video_annotation_qa.py

Examples for using the QA APIs on dataset/video annotations

[[JIRA Task][https://jiracommercial.flir.com/browse/CON-2394]]

Looks like black formatter needs run


Paymod refactor

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues,
Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the
changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.

Yeah really solid cleanup here @wraeth-eth. As mentioned on our call, I think we should focus our cleanup efforts around V2 code going forward because there are certainly improvements to be made on that front. But I can't spot any regressions in the UI as a result of this, and it's certainly a lot tidier. Shall merge when tests complete


[UI] /issues page side menu width

Why side menu has width and margin. Can we remove them all to shown as 100% width?
When organization name long, can't see repository names. Can we also add title attribute to shown on hover?

Can you provide a screenshot of what would be the result of the proposed change?

The main problem is i cant see REPOSITORY1 or REPOSITORY2 currently
This is what i think for new style. But someone can propose better looks

For me this change should be OK. Some languages tend to have longer titles (Spanish is my case) and many times the styles do not work very well with them. Can you send a PR for this?

I can send after 6 hours


Seems unmaintained

Hi,
I thought to use this image as a source of binaries in my Dockerfile, but it seems unmaintained. Are there any plans on pushing new versions of this image regularly? The latest version is currently 7 months old.

hey @Zebradil ! Sorry for that impression, our automation stopped working but we fixed it a few months ago. Thanks for the heads up though, just getting around to closing this issue.

Seems broken again? :latest has been created on 2023-01-19T01:30:35.020Z, according to the label org.opencontainers.image.created inside the image.
We, too, want to use the image to copy the binaries from.

As an alternative, we could probably use ghcr.io/carvel-dev/kapp-controller which contains all the binaries, too. Unfortunately, just like this image, it is also not properly tagged. The image from this project only has :latest while ghcr.io/carvel-dev/kapp-controller uses non-descriptive image tags:
$ skopeo list-tags docker://ghcr.io/carvel-dev/kapp-controller
{
    "Repository": "ghcr.io/carvel-dev/kapp-controller",
    "Tags": [
        "rand-1675057777567790718-46346116232-kapp-controller",
        "rand-1675058931368617744-1271351746218-kapp-controller",
        "rand-1675163778348920726-812368450122-kapp-controller",
        "rand-1675193814572979956-253282113186-kapp-controller",
        "rand-1675244127333236169-10210068214167-kapp-controller",
        "rand-1675245774317316042-2464820015738-kapp-controller",
        "kbld-rand-1675401201382102428-62255187123160",
        "kbld-rand-1675402840339791567-23310489178160",
        "kbld-rand-1675407352309901886-7717417023141",
        "kbld-rand-1675439880404786302-1241851372172",
        "kbld-rand-1675452368289632941-17711916793109",
        "rand-1675781866138377603-621713572123-kapp-controller",
        "rand-1675792914553031285-55133160193248-kapp-controller",
        "rand-1678460282626099075-13914813387188-kapp-controller",
        "rand-1680590094810685620-2483821750128-kapp-controller",
        "rand-1680592068259360587-2923224245119-kapp-controller",
        "rand-1681926566974979929-20313523818122-kapp-controller",
        "rand-1683901464977327408-1796785129221-kapp-controller",
        "rand-1686140184663267382-180232123394-kapp-controller",
        "rand-1686726827567580141-452638127244-kapp-controller",
        "rand-1686727096520192185-229118199458-kapp-controller",
        "rand-1686728436715212662-1451085142-kapp-controller",
        "rand-1687286425569287071-23911111561217-kapp-controller",
        "rand-1687287211927885042-24712624661102-kapp-controller",
        "rand-1689069528378621476-8918624717676-kapp-controller",
        "rand-1692209005806725957-24418352143201-kapp-controller",
        "rand-1692289561907427951-191219866131-kapp-controller",
        "rand-1692290847083382201-9613217015964-kapp-controller",
        "rand-1692291184261679450-149351989157-kapp-controller",
        "rand-1692863946420967110-245564812994-kapp-controller"
    ]
}

I wish there was an image that follows the versioning of kapp-controller, or maybe even better: Separate images for each of the carvel tools where the image-tags match the tool versions.


Add Access-Control-Allow-Origin response header response

Could you please add Access-Control-Allow-Origin response header to allow CORS? This will help in StopLight to get the response, else it will throw the error.

Closing since we're using Swagger instead of Stoplight.


Fix typo.

It's a small detail, but I just noticed it... 😄
school → scroll

another great catch, thanks! 🚀


[5.5] Allow to replace asterisks into real parameter values for Exists rule

As far as I know it's not possible to use rule like this:
[
    'activities.*.ticket_id' => [
          Rule::unique('tickets', 'id')->where('project_id', 'activities.*.project_id'),
     ]
]

for data like this:
[
    'activities' => [
        [
            'project_id' => 1,
            'ticket_id' => 2,
        ],
        [
            'project_id' => 5,
            'ticket_id' => 7,
        ],
    ],
];

What you need to do now something like this:
foreach ($this->input('activities') as $key => $activity) {
    $rules['activities.' . $key . '.ticket_id'][] =
        Rule::exists('tickets', 'id')->where('project_id', $activity['project_id']);
}

what is not very convenient.
I believe this could be applied also for other rules (Unique for example) but I believe some extra work should be done for Unique rule because of this PR: https://github.com/laravel/framework/pull/12612
This is of course breaking change, if someone really needs to search for  column with activities.*.project_id value and have same key, the key value will be used instead of simple string.

I dunno, this is a little much for me and the tests are very mocked so hard to tell if anything is working correctly.

@taylorotwell Tests are not very complicated. There are multiple tests in this PHPUnit test method, so in fact the most important line to look at are line like this:
$mock->shouldReceive('getCount')->once()->with('users', 'id', '2', null, null, ['company_id' => 1])->andReturn(true);

that verify whether in extra parameters with have value or the unchanged key like this: tasks.*.not_existing_key.
I could add integration tests that hit database if you want but I think in this case it's not necessary.

Also if you look at any other tests as for example testValidationExists they are using exact same pattern for tests like this:
        $trans = $this->getIlluminateArrayTranslator();
        $v = new Validator($trans, ['email' => 'foo'], ['email' => 'Exists:users']);
        $mock = m::mock('Illuminate\Validation\PresenceVerifierInterface');
        $mock->shouldReceive('setConnection')->once()->with(null);
        $mock->shouldReceive('getCount')->once()->with('users', 'email', 'foo', null, null, [])->andReturn(true);
        $v->setPresenceVerifier($mock);
        $this->assertTrue($v->passes());


Custom ContractJsonConverter is not working

I use alpha from master branch.
I Created JsonNetSerealizer with  overrided   protected override IList<Func<Type, JsonConverter>> ContractConverters => null;
But my ContractJsonConverter is not called.
I did some debugging and found that code in src\Nest\CommonAbstractions\SerializationBehavior\ElasticContractResolver.cs
 protected override JsonContract CreateContract(Type objectType)
{
    JsonContract contract = base.CreateContract(objectType);

    // this will only be called once and then cached

    if (typeof(IDictionary).IsAssignableFrom(objectType) && !typeof(IIsADictionary).IsAssignableFrom(objectType))
        contract.Converter = new VerbatimDictionaryKeysJsonConverter();
    else if (objectType == typeof(ServerError))
        contract.Converter = new ServerErrorJsonConverter();
    else if (objectType == typeof(DateTime) || objectType == typeof(DateTime?))
        contract.Converter = new IsoDateTimeConverter();
    else if (!objectType.FullName.StartsWith("Nest.", StringComparison.OrdinalIgnoreCase)) return contract;

    else if (ApplyExactContractJsonAttribute(objectType, contract)) return contract;
    else if (ApplyContractJsonAttribute(objectType, contract)) return contract;

    if (this._contractConverters.HasAny())
    {
        foreach (var c in this._contractConverters)
        {
            var converter = c(objectType);
            if (converter == null)
                continue;
            contract.Converter = converter;
            break;
        }
    }
    return contract;
}

Since my type name does not start from "Nest." it exits from function in case else if (!objectType.FullName.StartsWith("Nest.", StringComparison.OrdinalIgnoreCase)) return contract; . It does not check _contractConverters and uses the default one.
Can it be the bug?
Thank you a lot in advance!

Hey @zabykrinich, Thanks for reporting :+1:  This does look like a bug


Fix NSInvalidArgumentException in - [ARGeneViewController loadGene], line 108

Version: 2.3.2 (2015.12.01) | net.artsy.artsy
Stacktrace
ARGeneViewController;loadGene;ARGeneViewController.m;108
ARGeneViewController;viewDidLoad;ARGeneViewController.m;60
Reason
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[Artist updateGene:]: unrecognized selector sent to instance 0x138576930'
Link to HockeyApp

https://rink.hockeyapp.net/manage/apps/37029/crash_reasons/72466467

Closing in favour of #599.


Replace License

The "clear-bsd license" appears to be loosing traction, hard to find, becoming obscure.
https://choosealicense.com/licenses/bsd-3-clause-clear/
Replace with the MIT one ?  https://choosealicense.com/licenses/mit/ - appears far more recognisable and still says what we need. Still need to list all the existing, "other" licenses.
Davo

All my stuff, now MIT license


Allow more granular playback speed control via keyboard shortcuts - especially between 1x and 2.5x

First of all, amazing plugin. Works seamlessly and makes video note-taking, both online and offline, a smooth and pleasant experience. Thank you for working on this.
The problem:
AFAIK, at the moment, playback speed can be 0.25x, 0.5x, 0.75x, 1x, 1.25x, 1.5x, 2x, 3x, 5x, 10x, custom. But speeds like 1.6x or 2.1x are not supported when using the keyboard shortcuts.
This can be quite frustrating because different content creators speak at different speeds, or some videos are less or more information dense, thus requiring faster or slower playback. Depending on the video and the creator I am watching I may need a playback speed of 1.7x or 1.4x or even 2.2x, sometimes changing speed even during the same video.
Proposed solution:
Allow the user to have more granular control over the playback speed. Allow increases of 0.1x or even 0.05x, especially in the 0.5x - 3x range.

command increase/decrease speed by 0.1x is included in 3.0.3-beta.1


Changed 'my RG' to 'myRG' in src/rgged.cc, to prevent for follolowing…

[ 97%] Building CXX object src/CMakeFiles/rgged.dir/rgged.cc.o
/packages/rgged/src/rgged.cc: In function ‘int main(int, char**)’:
/packages/rgged/src/rgged.cc:51:15: error: ‘my’ was not declared in this scope
  if(SM!="") { my RG.Sample=SM;}
               ^
/packages/rgged/src/rgged.cc:51:18: error: expected ‘;’ before ‘RG’
  if(SM!="") { my RG.Sample=SM;}
                  ^
make[2]: *** [src/CMakeFiles/rgged.dir/rgged.cc.o] Error 1
make[1]: *** [src/CMakeFiles/rgged.dir/all] Error 2
make: *** [all] Error 2

The above error occurs if this small typo is not fixed.
"my RG" was changed to "myRG" to prevent this error during the compilation step.

Thanks for finding this. I would recommend using samtools addreplacerg in lieu of this code I wrote.

samtools is an industry standard and you can increase the number of threads with<EMAIL_ADDRESS>You cannot do that with rgged and I think rgged saves the output as SAM instead of BAM.
I wrote rgged to learn some C++ and BamTools. However, in my tests, I found BamTools has a lot of overhead and in some cases runs slower than pysam. So why bother writing a C++ program to leverage htslib if you don't get the speed advantage?
Thanks for the interest and helpful comments.


[Request] Add the file uploading feature from henkelmax's Audio Player.

I'd love for my players to be able to upload their own .wav files with a command, instead of having to upload it to the server files.

So something like /songdownload ? Doesnt seem that hard, but currently im working on support for mp3 files, then ill get to that.
Thank you for the suggestion!

I would probably make the command be something like /cd upload instead for better organization.

Yeah I will rework the syntax after the mp3 support update.
I will be adding file uploads as well as a speaker system.

For speakers, could you not just add a feature that lets you play music discs in a jukebox with a hopper?

True, i could do that actually...

Bump, anything happening with this in the near future?

Ive never done anything with file uploading, so this is quite new to me. Im currently trying to find a good way to do this.

It's ok!
The way I've seen it being done before, is to request users to upload the file to filebin or a similar service, and then take the filebin link as a command parameter...

So Ive been reading up on this and it is out of my scope of knowledge right now. Though I know all of the basics of Java and can code plugins like this for Bukkit, I have no idea how the file uploading works. Furthermore, I will be heading back to school tomorrow, and although I hope it doesnt take time off of continuing to support this project, I wont have much time to learn new things. I do take a Java programming class in school, but we have not gotten that far in our course.

Sorry if this dissapoints any of you :(

I can fork and add the file uploading if you want, you just may want to go over it and clean it up after @Navoei. I don't do a lot of Java but ive done it for a different project.

Of coarse! I will greatly appreciate it if you can contribute. I will also credit you in the comments of the code and in the readme as well.

so how is this going? If u don't mind me asking

any progress?

Currently I am working on another project for my Computer Science class which uses google drive to manage playlists
I can use the knowledge i gain from that project and apply it here.
The only problem is that the project, along with my other classes, take up a lot of my time.

I'm sorry for not keeping up to date with this project. It has been a long time but it seems like a developer forked this plugin to add file uploading: https://github.com/NotGeri/CustomDiscs .
If they were to make a pull request I may implement it.

Currently working on this right now.

Version 2.3 is now released:
https://github.com/Navoei/CustomDiscs/releases/tag/Latest


UI: Unify rendering of transport, user names - part 2

Added icon for username
Changed icon for transport to truck
Adapted "staging" page
Renamed CSS-classes because of conflict with classes used on staging page
Added hover info to item type icon (first column in repo view)

Closes #4098

The stage page already used class=“user” and class=“transport” for table columns with JS depending on it. It was easier to change the span style. “-box” isn’t ideal. Open for suggestions

sure, but are they also span? Don't have an access to my system at the moment to check. I'm 100% that it can be solved with proper selectors. Semantically these banner are "transport", "user" and etc. Similarly columns in the table are also semantically "transport", "user" and etc. There is no contradiction.

on the other hand span might not be semantically most clear ... let me check when I'm by a system

maybe it should be not span but label. Though label is for forms, need to check what validation says about it. another option is to use double styling. banner transport and the selector  .banner.transport

It's not about the CSS. it's already limited to the right element (i.e. the spans). The problem is the JS which is using, for example, class="transport" to filter the rows. The JS gets confused if there's another element with this class. So doubling won't help.
https://github.com/abapGit/abapGit/blob/master/src/ui/zabapgit_js_common.w3mi.data.js#L345
https://github.com/abapGit/abapGit/blob/master/src/ui/zabapgit_js_common.w3mi.data.js#L509-L519

Yeah I know. It was me who wrote that JS in the first place :)) But this should not conflict with the tags outside of the table. At least it hadn't for quite a while. Or ?

Is it okay if I merge this PR?


How To Enable Dark Theme

I was buy CoreUI Pro edition, in website said it contains dark theme, I want to use it, how I do it? Is there just change some configuration or something else to use dark theme in react version?

hi @caktoy
Please look at src/scss/style.scss in template source and find the description how to switch to the dark theme in comments of this file.

Oke thanks, I get it.

@xidedix
thanks for the reply. I uncommented
@import "@coreui/coreui-pro/scss/themes/dark/coreui-dark";
but i don't see the themes/dark directory when in
https://ibb.co/fKd5Cq

Hi everyone, I bought tht pro angular UI, I am using the alpha 3.0. I don't see the same namespace so I changed:
@import "~@coreui/coreui-pro/scss/coreui";  TO :  @import "~@coreui/coreui-pro/scss/themes/dark/theme";
But is not working,

document.body.classList.add('dark-theme')


Option to exclude idle threads

Profile currently reports all idle threads sitting in __psynch_cvwait. So the more threads you are using with Julia, the smaller the percentage of the report your actual work will be. For example, this profile was taken on my machine with 6 threads (the default when running in Juno), when i was measuring only single-threaded work:

We learned after asking on the google/pprof repo (they replied very promptly, which is wonderful! ☺) that you can enable relative percentages via -relative_percentages, and then paths you ignore in the Web UI (or through -ignore) will not be counted when computing percentages for CPU usage!
What this means for us:

I think we should probably just always set -relative_percentages when spawning the UI, maybe with an option to not do that. But it seems to me like people would basically always want this?
We could consider an option where we automatically filter out nodes like these __psynch_cvwait when opening PProf :)

CC: @cscherrer

https://github.com/google/pprof/issues/516#issuecomment-616162459

I am in favour for doing relative percentages!

:) Cool! I just added a flag and verified that it works! :) Will push up now


Port certificate-transparency-go's Trillian CTFE to Trillian Tessera

#88
This PR has two commits to ease review:

Moves over all the files that we need to
Has all the edits. You can find a chain with bite-sized edits over here. Commits are not self contained, so it might not make it easier to follow. Here are all the changes the happen in this commit:

"s-../testdata/-./testdata" since the testdata directory is lower in the tree than it was before
Remove all the read APIs: no need with buckety style Tessera!
Remove all the mirror/frozen-sth stuff: no need with buckety style Tessera!
Remove multi backend configuration: this doesn't really make sense anymore with Tessera's a model with single server
Remove chain storage service: we'll have to tweak it for static-ct-api, let's remove it for now, and see what we can re-use later.
Delete prefix overrides: As per https://c2sp.org/static-ct-api, the origin MUST be the submission prefix
Rename prefix to origin: As per https://c2sp.org/static-ct-api, the origin MUST be the submission prefix
Delete log id: the log id used to be the unique log identifier, used both for configuration and in monitoring metrics and logs. We can now replace this with a human friendly log origin, which also needs to be unique.
Add pieces of config to connect to configure Tessera's storage system: there are multiple ways of doing this, I've experimented with this a bit and there are very likely options I haven't thought of yet. I propose that we move with the current approach, and change things later.
Define storage.go to abstract Tessera's APIs and the others we'll need for CT (deduplication, chain storage)
Move some parsing function over: a few copies from the sunlight repo and c-t-go to parse data / objects
Replace Trillian backend grpc things with Tessera's APIs

If this (big?!) PR is too hard to review, I could send self contained chunks corresponding to the list above. I wish I had done this in the first (or previous to first!) place... but I had to do it first to know exactly how to break it down.
There's probably more things I could have put in this PR, like copying more things over to avoid puling in more dependencies for instance. I've made a list of follow up AIs in #88.

Codecov Report
Attention: Patch coverage is 32.82353% with 571 lines in your changes missing coverage. Please review.

Project coverage is 27.41%. Comparing base (46ec9c2) to head (dd10900).
Report is 25 commits behind head on main.

Files
Patch %
Lines

personalities/ct-static-api/handlers.go
17.92%
174 Missing :warning:

personalities/ct-static-api/ct_server/main.go
0.00%
155 Missing :warning:

personalities/ct-static-api/configpb/config.pb.go
23.19%
147 Missing and 2 partials :warning:

personalities/ct-static-api/config.go
73.23%
16 Missing and 3 partials :warning:

ctonly/ct.go
0.00%
18 Missing :warning:

personalities/ct-static-api/instance.go
72.13%
16 Missing and 1 partial :warning:

personalities/ct-static-api/requestlog.go
0.00%
17 Missing :warning:

personalities/ct-static-api/cert_checker.go
90.00%
4 Missing and 4 partials :warning:

personalities/ct-static-api/serialize.go
78.57%
3 Missing and 3 partials :warning:

personalities/ct-static-api/storage.go
0.00%
6 Missing :warning:

... and 1 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
- Coverage   35.80%   27.41%   -8.39%
==========================================
  Files          16       31      +15
  Lines        1363     2790    +1427
==========================================
+ Hits          488      765     +277
- Misses        801     1937    +1136
- Partials       74       88      +14

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

Just before I dive in... :)
The stuff under personalities/ct_static_api is, I think, intended to move to another repo (e.g. ct-go?) in the near term, and it's just here in this PR as a means to iterate quickly given that Tessera APIs etc. are still a little in flux. Is that right?

Just before I dive in... :)
The stuff under personalities/ct_static_api is, I think, intended to move to another repo (e.g. ct-go?) in the near term, and it's just here in this PR as a means to iterate quickly given that Tessera APIs etc. are still a little in flux. Is that right?

Yes, see #88, I added a list of tasks there, including one to move it to its own repo.


Spotfiy ??

can you add spotfiy ?

No Spotify only supports playback for users & there is no way to scrap it
What you can do is using the Spotify API to get the track meta data like name & search that on yt

https://fredboat.com/docs/donate

The freadboat doc actually says The ability to add Spotify playlists with tracks found on YouTube and Soundcloud
So they query track data from spotify API and then search those data on Youtube or Soundcloud, which most bots do nowdays.
But lavalink could add such a feature to the search api

But lavalink could add such a feature to the search api

Meh.
I'm more inclined to add a plugin system. In which case you could just yank FredBoat's custom source provider into a plugin.

In theory you could make a spotify account and just use something like puppeteer, selenium or something like that to just scrape it.

In theory you could make a spotify account and just use something like puppeteer, selenium or something like that to just scrape it.

In theory you could make a spotify account and just use something like puppeteer, selenium or something like that to just scrape it.

you would get ratelimited quite quickly

Spotify integration is quite hacky.
In theory there are a few routes you can take
If your bot is only for your use, and you know it won't be used more than once at the same time, you could disguise it as a spotify enabled speaker (it'll act like google home) and with the help of a few libraries i was able to control my spotify client using a program. You could pipe that output to your discord bot in one way or another.
A friend of mine once wrote a lil local server in golang that could somehow host spotify songs on a local webpage that lavaplayer could then happily accept. I cannot tell you how to do this and i cannot recommend doing it as it technically violates TOS if you're using it on a large scale (sharing spotify accounts is technically not allowed)
I think you would have an easier time reading track metadata and looking it up on other services that lavaplayer does support.

I don't think there'll ever be true Spotify integration due to the obstacles you'd need to overcome to be able play directly. That, and you would probably need a lot of premium accounts.
I will, however, be closing this with the release of the plugin API. Used in conjunction with Topi's Source Manager Plugin, you can achieve pretty effective Spotify support.


Force commit message to have issue number.

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

2.0.0
GitHub release

Your semantic-release bot :package::rocket:


Bug on evaluation during training

@tianrun-chen In func eval_psnr(),pred_list = torch.cat(pred_list, 1)     gt_list = torch.cat(gt_list, 1) should be pred_list = torch.cat(pred_list, 0)     gt_list = torch.cat(gt_list, 0)

I meet the same question

@TengliEd Hello, it seems like I've encountered some issues during the evaluation. Could you please tell me what this error message is? Has the following error occurred?
aceback (most recent call last):
File "train.py", line 279, in
main(config, save_path, args=args)
File "train.py", line 214, in main
result1, result2, result3, result4, metric1, metric2, metric3, metric4 = eval_psnr(val_loader, model,
File "train.py", line 89, in eval_psnr
pred_list = torch.cat(pred_list, 1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 2 but got size 1 for tensor number 92 in the list.
Traceback (most recent call last):
File "train.py", line 279, in
main(config, save_path, args=args)
File "train.py", line 214, in main
result1, result2, result3, result4, metric1, metric2, metric3, metric4 = eval_psnr(val_loader, model,
File "train.py", line 89, in eval_psnr
pred_list = torch.cat(pred_list, 1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 2 but got size 1 for tensor number 92 in the list.
Traceback (most recent call last):
File "train.py", line 279, in
main(config, save_path, args=args)
File "train.py", line 214, in main
result1, result2, result3, result4, metric1, metric2, metric3, metric4 = eval_psnr(val_loader, model,
File "train.py", line 89, in eval_psnr
pred_list = torch.cat(pred_list, 1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 2 but got size 1 for tensor number 92 in the list.
Traceback (most recent call last):
File "train.py", line 279, in
main(config, save_path, args=args)
File "train.py", line 214, in main
result1, result2, result3, result4, metric1, metric2, metric3, metric4 = eval_psnr(val_loader, model,
File "train.py", line 89, in eval_psnr
pred_list = torch.cat(pred_list, 1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 2 but got size 1 for tensor number 92 in the list.
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 21242) of binary: /usr/local/miniconda3/bin/python
Traceback (most recent call last):
File "/usr/local/miniconda3/bin/torchrun", line 8, in
sys.exit(main())
File "/usr/local/miniconda3/lib/python3.8/site-packages/torch/distributed/elastic/multiprocessing/errors/init.py", line 346, in wrapper
return f(*args, **kwargs)
File "/usr/local/miniconda3/lib/python3.8/site-packages/torch/distributed/run.py", line 762, in main
run(args)
File "/usr/local/miniconda3/lib/python3.8/site-packages/torch/distributed/run.py", line 753, in run
elastic_launch(
File "/usr/local/miniconda3/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 132, in call
return launch_agent(self._config, self._entrypoint, list(args))
File "/usr/local/miniconda3/lib/python3.8/site-packages/torch/distributed/launcher/api.py", line 246, in launch_agent
raise ChildFailedError(
traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
[2]:
time      : 2023-11-12_14:41:17
host      : I1637d69a3200801799
rank      : 2 (local_rank: 2)
exitcode  : 1 (pid: 21244)
error_file: <N/A>
traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
[3]:
time      : 2023-11-12_14:41:17
host      : I1637d69a3200801799
[2]:
time      : 2023-11-12_14:41:17
host      : I1637d69a3200801799
rank      : 2 (local_rank: 2)
exitcode  : 1 (pid: 21244)
error_file: <N/A>
traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
[3]:
time      : 2023-11-12_14:41:17
host      : I1637d69a3200801799
rank      : 3 (local_rank: 3)
exitcode  : 1 (pid: 21245)
error_file: <N/A>
traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
Root Cause (first observed failure):
[0]:
time      : 2023-11-12_14:41:17
host      : I1637d69a3200801799
rank      : 0 (local_rank: 0)
exitcode  : 1 (pid: 21242)
error_file: <N/A>
traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html

I believe it should be torch.cat(pred_list, 0) instead of torch.cat(pred_list, 1)

According to https://github.com/tianrun-chen/SAM-Adapter-PyTorch/blob/main/utils.py#L122
def calc_cod(y_pred, y_true):
    batchsize = y_true.shape[0]

    metric_FM = sod_metric.Fmeasure()
    metric_WFM = sod_metric.WeightedFmeasure()
    metric_SM = sod_metric.Smeasure()
    metric_EM = sod_metric.Emeasure()
    metric_MAE = sod_metric.MAE()
    with torch.no_grad():
        assert y_pred.shape == y_true.shape

        for i in range(batchsize):
            true, pred = \
                y_true[i, 0].cpu().data.numpy() * 255, y_pred[i, 0].cpu().data.numpy() * 255

The code iterate over batch_size, torch.cat(pred_list, 1) would only consider 1 batch samples. But the code means evaluate on all the validation set (len(val set)).


Option to choose calendars from server is missing when editing account

The screen which lists the calendars available to your account on the server to choose which ones to sync and allows you to rename them is missing when going into the Sync Settings.  It used to be the first option right at the top of the settings, and the option is completely gone.  I'm not sure how long it's been gone, it's been a long time since I had to use it.  It looks like new calendars automatically show up, and I can choose whether or not to sync them from the Calendar app, but the Calendar app doesn't give me a way to rename them or change their color (it just uses the names and colors from the server)
Android 9, Samsung Galaxy S8
CalDAV Sync 0.4.32 build 748

FWIW, I can change colors and names when first setting up the account, but that means if any of them need to be changed at any point later, I currently have to delete and re-add the account to get to it.


ffmpeg._run.Error: ffmpeg error

seeing this error when attempting to generate subtitles:
ffmpeg version 6.0
macOS Ventura 13.4.1 (22F82)
admin@MacBook-Air ~ % auto_subtitle /Users/admin/Movies/slovenia.mp4 -o subtitled/ /Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/whisper/timing.py:57: NumbaDeprecationWarning: The 'nopython' keyword argument was not supplied to the 'numba.jit' decorator. The implicit default value for this argument is currently False, but it will be changed to True in Numba 0.59.0. See https://numba.readthedocs.io/en/stable/reference/deprecation.html#deprecation-of-object-mode-fall-back-behaviour-when-using-jit for details.   @numba.jit Extracting audio from slovenia... Traceback (most recent call last):   File "/Library/Frameworks/Python.framework/Versions/3.11/bin/auto_subtitle", line 8, in <module>     sys.exit(main())              ^^^^^^   File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/auto_subtitle/cli.py", line 49, in main     audios = get_audio(args.pop("video"))              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/auto_subtitle/cli.py", line 84, in get_audio     ).run(quiet=True, overwrite_output=True)       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/site-packages/ffmpeg/_run.py", line 325, in run     raise Error('ffmpeg', out, err) ffmpeg._run.Error: ffmpeg error (see stderr output for detail)

i fix it. Just don't use "--output_dir" or "-o". At least in my case in windows. And avoid use ffmpeg, just get the srt with --srt_only
Example:
auto_subtitle C:\Users\fedeg\videos\Captures\phineasFerb\test.mp4 --srt_only True

still getting this error

https://github.com/Irvingouj/auto-subtitle
I created a fork, this works for windows. the python path management is such a pain

I changed code for ffmpeg to show error and it was "file not found". You may have double extension of file, your "video.mp4" is in fact "video.mp4.mp4". After i renamed my file to "vid1" everything worked!


F5 Accordion nested in another accordion not functional

As it says in the title, when someone tries to put an accordion inside of another accordion, the functionality is totally broken. If you click on a child element of the parent accordion section, it will close that entire parent section while still considering it 'active'. Thus, to open it again, you have to click twice.
The depreciation of F4 'Sections' left quite a big gap in necessary functionality here. Is it possible that the placement of data-slug could fix this?

Not seeing that here - http://codepen.io/rafibomb/pen/JYdrjN?editors=110 Can you add your example so we can see what's happening?

Closing this due to no response. If you are still having an issue, please reopen.


AttributeError: Cannot find field 'logits' in the given Instances!

When I train the T1 task, this error is reported after the training is over, how can I solve it?

Hello YH, have you solved this problem?


Update grype wrapper methods and tests to work with the new grype db schema.

Updates methods in the grype wrapper to use the new grype v3 schema. Also updates corresponding tests.

Need to ensure schema definitions here are inline with grype definitions https://github.com/anchore/grype-db/tree/main/pkg/db/model. Noticed a foreign key association that's not in the original model

Link to grype-db models is an older version. Refer to db schema v3 models here https://github.com/anchore/grype-db/tree/v3/pkg/db/model

Per a slack conversation with @nightfurys , I'm going to merge this pr once the CI passes, since it unblocks several things. And in the follow-up PR I will plan to revisit how the grype wrapper joins those the two relevant tables from grype db in sql alchemy.


Added the trans tag with the required i18nKey

⚠ IMPORTANT: Please do not create a Pull Request without creating an Issue first.
All changes need to be discussed before proceeding. Failure to do so may result in the pull request being rejected.
Before submitting a pull request, please be sure to review:

Our Code of Conduct: https://github.com/COVID-19-electronic-health-system/.github/blob/master/CODE_OF_CONDUCT.md
Our Contributing Guidelines: https://github.com/COVID-19-electronic-health-system/.github/blob/master/CONTRIBUTING.md
Our Support documentation: https://github.com/COVID-19-electronic-health-system/.github/blob/master/SUPPORT.md

Please include the issue number the pull request fixes by replacing YOUR-ISSUE-HERE in the text below.
Fixes #726
Summary

Details

Test Plan (required)

Final Checklist

[ ] For CoronaTracker, did you bump the version in package.json?

Update the second decimal for a major change
Update the third decimal for a minor change
Numbers can go past 9, e.g. 1.0.9 => 1.0.10
For more info, read about Semantic Versioning

[ ] Did you add any new tests as necessary?
[ ] Is your PR rebased off the most current master?
[ ] Have you squashed all commits? (can be done at merge)
[ ] Did you use yarn not npm? (important!)
[ ] Did you use Material-UI wherever possible?
[ ] Did you run yarn lint on the code?
[ ] Did you run all of your most recent changes locally to make sure everything is working?

@thedeveloperhere I didn't review the code yet. But couple of things:

Please follow the PR template and provide the asked info.
If you did the above you would run the test locally and travis wouldn't fail 😁  Please run the test locally and follow the instructions, you might need to press u or r I dont recall, to make new snapshot for the test. Or you can delete the snaps manually "Don't recommend"

Thanks and good luck


RFC7967 Implementation

RFC7967, a new coap option called No-Response would be useful to tell the server explicitly to suppress the responses for any particular request from the server.

Thank you for suggestion. Please if you have time - could you contribute and make merge request ?

ok, do you have any guideline for raising a PR?

Thx for contributing. PR: https://github.com/go-ocf/go-coap/pull/37


clone  not working

clone this repo not working

I missed adding .git in the desired line (Getting Started, Installation, Step 1).
To clone this repo use this web url  https://github.com/ladunjexa/Threejs_3D_Portfolio.git.
In any repository, press the green code button and there you can learn about how-to clone (via HTTPS, SSH or GitHub CLI).


B#45953 - Image docker container no longer buildable (part 2)

This is an update from the original flyimage right?

Most of it, also had to update some of our code to be compatible with the new code. We needed at least a newer version of the upstream because the Dockerfile was referring to a image that no longer existed.

Becomes obsolete with https://github.com/SkyDreams/flyimg/pull/7 and switching to master-v2 branch


Skype update module for GE tests does not seem to update the test results on Skype

I am suspecting requests package compatibility issue with Python 3.11 could be causing this issue.
I tried to check if we are installing the latest request package, the pip installation step does seem to include requests package, I am assuming one of the packages we are installing is installing request as a dependant package.
But this message Successfully sent the Skype message - <b>Newsletter Automation - data validations outcome: </b> after the Skype update script completion could suggest it could be some other issue too.

This issue is not reproducible


Add !inSnippetMode context on Tab for acceptSelectedSuggestion shortcut

There is a conflict between jump-to-next-placeholder and accept-suggestion shortcut.
Right now, pressing the tab key while editing a snipper will stop snippet mode and accept the first suggestion.
This can be easily fixed by adding additional context for this key:

I think the editor should have this by default.
Related issues:
https://github.com/microsoft/vscode/issues/50776

We are not planning on change the default here. We can gone back and forth once and the user feedback was pretty obvious that the way it is right now is the preferred way. Tho, all of this is configurable and can be adjusted to your likings

The user feedback was pretty obvious that the way it is right now is the preferred way.

Can you please provide any supporting issues on this? I didn't found any. And I don't see the reason why it should be closed.

There is the editor.suggest.snippetsPreventQuickSuggestions setting for this

The user feedback was pretty obvious that the way it is right now is the preferred way.

Can you please provide any supporting issues on this? I didn't found any. And I don't see the reason why it should be closed.
This suggestion is not about changing the tab to the different key. Is about an additional context for it, that other editors have by default.
It does not interfere with #52999 or #51722.


Minimum cases/deaths threshold

Am considering revisiting a few decisions, triggered by reading #47, and a few emails from folks wondering why their countries aren't included in the deaths graph. At the moment this is mostly a note to self, however let me know if anyone has feedback on this or wants to take a stab at implementing this.
Initially I'd set minCasesInCountry to 50 as the global confirmed cases data was noisier below this value (the variable refers to the threshold for including a country on the graph). However I think we could have different thresholds for minCasesInCountry and minDeathsInCountry. Just looking at it by eye, it seems like setting minDeathsInCountry to 10 creates a clean looking graph.
Also when looking at US states, it seems like both can be set to 10 just fine as the case data has less variability for low values (< 50).
Finally whenever minCasesInCountry is 50, the minimum on the visual settings should probably be set accordingly, to emphasize this threshold.

I think what I'm seeing right now is related to this.   A few states dropped out of trends entirely and I'm assuming it's because the new reported deaths  in the last week drops below the 10 threshold.   I'd still like to see the original trajectory of the virus in that country or state.   I can see not including a  country or state where the total reported deaths never rose above a minimum threshold (as happened early on) but now virtually everywhere has had it.    The value of the app is comparing one state or country to another.   Having them simply disappear when they "succeed" in clearing the virus is odd.


Candlestick chart loads indefinitely

As seen in the screenshot below, the candlestick chart on the trade screen loads indefinitely.

Possibly related (at least locally) log:
flutter: | 2023-07-25 14:41:30.211 | LogLevel.ERROR | Failed to initialise: Error: DB uninitialised

Okay, when moving the candlestick initialisation to before the try/catch block and moving the await walletChangeNotifier.refreshWalletInfo(); call into it, the app does not crash and the candlestick chart loads!

Note that this just fixes the candlestick sync and not the underlying issue, which appears to be the DB uninitialized error.

Bisecting this.

https://github.com/get10101/10101/commit/1e337dccb1b566caf8119c4806c67134de1aca7d works
https://github.com/get10101/10101/commit/2e737dd8e618e036329af36b97cd78df90f877bf works
https://github.com/get10101/10101/commit/6b05827e5ed6726165fba3b34fd7139de3de63e0 works
https://github.com/get10101/10101/commit/8d33367b248eb8ec47b728da91e30fb090986950 works
https://github.com/get10101/10101/commit/97fc70a3f39f98fa7a072f0d5d659c669c72919a works
https://github.com/get10101/10101/commit/5647f8780783b3427308d6b890632a74e77f343e does not work


Text shows over animated widget in the presence of a network image when in Stack

Steps to Reproduce
When there is a network image (and upon further testing, an Asset Image as well) in a stack, it causes all of the text widgets in the stack to appear above an animated widget when they should appear below due to Stack order. However, when the network image is removed, the Stack behaves as expected.
This seems to only be an issue with chrome and MacOS... (and dartpad) This functioned correctly on iOS and Android. I tested this with the latest master channel code, 1.16.2-pre.41
I created this dartpad to show what I'm seeing. https://dartpad.dev/4bd49a19a9db745f003375e6b0e7689e
Expected results:
Due to the placement in the stack, the kitty pic should cover the 'text below kitty' and the Topdraw() widget should cover both 'text below and text above kitty' while the 'text above drawer' is always visible. However, when the animated widget is run by pressing the burger, the "text above kitty" is visible when the cat picture is uncommented... however, when this picture is commented out, the text behaves correctly.
Actual results:
When the network image is active, the text that should be below the animated widget is visible.
code
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(),
      home: Scaffold(body: MyPage()),
    );
  }
}

class MyPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Stack(
      children: [
        Positioned(top: 25, child: Column(children: [Text("text below kitty")])),
        Image.network('http://placekitten.com/200/300'), // <-- COMMENT OUT THIS LINE
        Positioned(top: 50, child: Column(children: [Text("text above kitty, but below drawer")])),
        TopDrawer(),
        Positioned(top: 75, child: Column(children: [Text("text above drawer")])),
      ],
    );
  }
}

class TopDrawer extends StatefulWidget {
  TopDrawer({Key key}) : super(key: key);

  @override
  _TopDrawerState createState() => _TopDrawerState();
}

class _TopDrawerState extends State<TopDrawer>
    with SingleTickerProviderStateMixin {
  AnimationController controller;
  Animation<double> animation;
  bool _drawerState;

  @override
  void initState() {
    controller =
        AnimationController(duration: Duration(milliseconds: 200), vsync: this)
          ..addListener(() => setState(() {}));

    animation = Tween(begin: 0.0, end: 200.0).animate(controller);
    _drawerState = false;
    super.initState();
  }

  @override
  void dispose() {
    controller.dispose();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return Stack(
      children: [
        Positioned(
          top: 0,
          left: -225,
          child: Transform.translate(
            offset: Offset((animation.value * (225 / 200)), 0.0),
            child: Container(
              height: 300,
              width: 225,
              color: Colors.pink,
            ),
          ),
        ),
        Positioned(
          top: 0,
          child: Transform.translate(
            offset: Offset(animation.value, 0.0),
            child: GestureDetector(
              onTap: () {
                setState(
                  () {
                    _drawerState = !_drawerState;
                  },
                );
                _drawerState ? controller.forward() : controller.reverse();
              },
              child: Container(
                height: 25,
                width: 25,
                color: Colors.pink,
                child: Transform.rotate(
                  angle: (animation.value / 200) * 3.14 / 4,
                  child: Icon(Icons.dehaze),
                ),
              ),
            ),
          ),
        ),
      ],
    );
  }
}

[✓] Flutter (Channel master, v1.16.2-pre.41, on Mac OS X 10.15.3 19D76, locale
    en-US)
    • Flutter version 1.16.2-pre.41 at /Users/williamterrill/git/flutter/flutter
    • Framework revision 8c0b43bda4 (2 days ago), 2020-03-20 19:01:01 -0400
    • Engine revision fdf4f7883f
    • Dart version 2.8.0 (build 2.8.0-dev.16.0 0296286c03)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/williamterrill/Library/Android/sdk/
    • Android NDK location not configured (optional; useful for native profiling
      support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/williamterrill/Library/Android/sdk/
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 11.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3.1, Build version 11C504
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin
        code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install:
        sudo gem install cocoapods

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 41.1.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build
      1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.43.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.8.1

[✓] Connected device (3 available)
    • macOS      • macOS      • darwin-x64     • Mac OS X 10.15.3 19D76
    • Chrome     • chrome     • web-javascript • Google Chrome 80.0.3987.149
    • Web Server • web-server • web-javascript • Flutter Tools

! Doctor found issues in 1 category.

This also happens for an asset image

@wterrill , we reproduced problem. Thank you for filing issue. As we work on a fix, please use the following to workaround the issue for now:
replace TopDrawer() with. RepaintBoundary(child: TopDrawer()),

The RepaintBoundary workaround worked well.  Thank you very much for looking into it.


chore: Inject k8s pod name and uid

Injected as env vars and picked up as part of the otel resource for resource association.

Tested with the following (thx @just-mitch!) and got the new attrs on the logs exported to Loki:
NAMESPACE=smoke FRESH_INSTALL=true VALUES_FILE="3-validators-with-metrics.yaml" LOG_LEVEL=debug yarn-project/end-to-end/scripts/network_test.sh ./src/spartan/smoke.test.ts


Add Code Snippets for Swift

Code snippets are templates that make it easier to enter repeating code patterns, such as loops or conditional-statements.
https://code.visualstudio.com/docs/editor/userdefinedsnippets

Would need a list of what we wanted. Maybe check what other languages are doing

These are included in the built-in Swift extension: https://github.com/microsoft/vscode/tree/main/extensions/swift/snippets


Format rust code with new style

🤔 This is a ...

[ ] New feature
[ ] Bug fix
[ ] Performance optimization
[ ] Enhancement feature
[ ] Refactoring
[x] Code style optimization
[ ] Test Case
[ ] Branch merge
[ ] Workflow
[ ] Other (about what?)

💡 Background and solution
Hello, everyone. I am about to start working on this project, and found something can be done to improve the development experience. One of them is the code style. Some lines in the current project exceed 85 characters, which will cause inconvenient to vim or neovim users, because they usually split the IDE into two windows, each one holds 85 characters at most.
So this PR aims to change the code style by adding rustfmt.toml and re-formating the code with command cargo fmt.
☑ Self-Check before Merge
Self-checked by executing cargo build and cargo run and the basic functions remain unchanged.
⚠ Please check all items below before requesting a reviewing. ⚠

[x] Doc is updated/provided or not needed
[x] Changelog is provided or not needed

nice

I'm a bit confused. Does vim not just display the lines as whatever the screen width is, why it's limited to 85*2 when splitted ?

I'm a bit confused. Does vim not just display the lines as whatever the screen width is, why it's limited to 85*2 when splitted ?

It will display like this, which is ugly:

But if we re-format it, it will like this:

And this PR is just suggesting, if the maintainer think it's bad, we can close it. Thanks. :)

Thanks for explaining this.
I'm not against it at all I also use vim(for python mostly, and i was with 79 line limit for PEP8 for many years, before switching to vscode).
I'm more thinking if it's possible to make vim show wider lines as sow nowadays lots of people have wider screen sizes so we can fully utilize them.

Thanks for explaining this.
I'm not against it at all I also use vim(for python mostly, and i was with 79 line limit for PEP8 for many years, before switching to vscode).
I'm more thinking if it's possible to make vim show wider lines as sow nowadays lots of people have wider screen sizes so we can fully utilize them.

I  got you. It seems like a difficult topic because different people have different conditions. In my case, I'm using 24-inch Dell monitor and the font-size in my terminal was set to 16, then I got the upper problem.
By the way, I found that there's another open source project written in Rust with 20k+ stars, iced, which is configured with max_width = 20 in rustfmt.toml. So I think the number between 80 to 90 maybe a suitable choice.

Consider add
tab_spaces = 4
reorder_imports = true
reorder_impl_items = true
use_field_init_shorthand = true

to rustfmt.toml
https://rust-lang.github.io/rustfmt/?version=v1.6.0&search=tab_spaces#tab_spaces
https://rust-lang.github.io/rustfmt/?version=v1.6.0&search=reorder_imports#reorder_imports
https://rust-lang.github.io/rustfmt/?version=v1.6.0&search=reorder_impl_items#reorder_impl_items
https://rust-lang.github.io/rustfmt/?version=master&search=use_field_init_shorthand#use_field_init_shorthand
if We use nightly rustfmt, there will be more features can be enabled
unstable_features = true

version = "Two"

tab_spaces = 2

group_imports = "StdExternalCrate"
imports_granularity = "Crate"
reorder_imports = true

wrap_comments = true
normalize_comments = true

reorder_impl_items = true
condense_wildcard_suffixes = true
enum_discrim_align_threshold = 20
use_field_init_shorthand = true

format_strings = true
format_code_in_doc_comments = true
format_macro_matchers = true

Yeah we are using nightly now

Consider add
tab_spaces = 4
reorder_imports = true
reorder_impl_items = true
use_field_init_shorthand = true

to rustfmt.toml
https://rust-lang.github.io/rustfmt/?version=v1.6.0&search=tab_spaces#tab_spaces
https://rust-lang.github.io/rustfmt/?version=v1.6.0&search=reorder_imports#reorder_imports
https://rust-lang.github.io/rustfmt/?version=v1.6.0&search=reorder_impl_items#reorder_impl_items
https://rust-lang.github.io/rustfmt/?version=master&search=use_field_init_shorthand#use_field_init_shorthand
IF We use nightly rustfmt (cargo +nightly fmt), there will be more features can be enabled

As reference

unstable_features = true

version = "Two"

tab_spaces = 4

group_imports = "StdExternalCrate"
imports_granularity = "Crate"
reorder_imports = true

wrap_comments = true
normalize_comments = true

reorder_impl_items = true
condense_wildcard_suffixes = true
enum_discrim_align_threshold = 20
use_field_init_shorthand = true

format_strings = true
format_code_in_doc_comments = true
format_macro_matchers = true

Done with new commit.


fix(test-command): actually exit the process

no issue

without this, the test command does not actually exit when all the tests are finished

Tests exit for me. I'd like to avoid forcing a process exit - it doesn't give the rest of the app a chance to clean up after itself properly. Let's figure out what is keeping the process alive instead.


ibmcloud cli autocompletion is not working on Mac OS

First of all documented path does not exist
source /usr/local/ibmcloud/autocomplete/zsh_autocomplete into ~/.zshrc.
I found autocompletion script under ../ibmcloud/bx folder, but seems like they are not functional for ibmcloud cli (not bluemix)
Any clues?

The doc is not synchronized. The auto-completion script path will be changed to /usr/local/ibmcloud/autocomplete from v0.9.0(coming in  a couple of days)
@weieigao, pls look at this issue.

@boyang9527 ok

@fg78nc if your cli version is 0.8.0, run the following command:
source /usr/local/ibmcloud/bx/zsh_autocomplete

And see if auto completion works.


Add time comparison for KNN_KDTREE

This PR is for https://github.com/shogun-toolbox/shogun/issues/3513 and I also create a gist as preview https://gist.github.com/MikeLing/6c08b1d5eaebc385c2a63fa1314b13b1

I found I push a submodule folder at the same time mistery. How could I remove that from this pr?

@karlnapf please give me some feedback , thanks a lot :)

BTW please read our develop readme on amending commits, and use google to see how to update existing PRs (i.e. remove your debian folder thingi)

here is the pr for shogun data https://github.com/shogun-toolbox/shogun-data/pull/133

Can you update the gist as well so I can see the rendered version?

@karlnapf oh, sorry, I forget to update the gist. Here is the gist https://gist.github.com/MikeLing/2f842d828276c60310b4d6a93e5d406b

You can update gists btw, so can always just update the old link rather than putting up a new one

@karlnapf I had update the gist in https://gist.github.com/MikeLing/c57eef21a17c4371e2dcb3f4ce7505eb and the pr for data set is in here. https://github.com/shogun-toolbox/shogun-data/pull/134

Let's get this one going, should be merged soon and the changes are minimal ... :)

The latest gist is in https://gist.github.com/MikeLing/c57eef21a17c4371e2dcb3f4ce7505eb. But I forget to keep the output of netbook on gist. However, I haven't change any code in this PR(except reduce the number of data set), so I guess the output should keep the same :)

Still some open issues, see above

Here is the latest gist https://gist.github.com/MikeLing/c57eef21a17c4371e2dcb3f4ce7505eb. :)

Still some open issues, see above

I had fix the https://github.com/shogun-toolbox/shogun/pull/3620#discussion_r102998106 in https://github.com/shogun-toolbox/shogun/pull/3620/files#diff-d1bb343558154b73b46a88e9680e63f4R364 and add more items in the legend :)

There is still some cells that talk about "with and without covertree" which is not relevant anymore.


29 update package structure

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 84.375%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
27

Relevant Lines:
32

💛  - Coveralls


Create 2022-10-27-cassandra-monitoring-part2.mdx

Cassandra monitoring part 2

Should the title not be different for both posts?

Should the title not be different for both posts?

Yes - updated.