Test aborted

From @KevinBurton on April 29, 2018 0:41
Steps to reproduce
dotnet test --filter ImportIndexTest
Expected  behavior
Total tests: 1. Passed: 1. Failed: 0. Skipped: 0.
Test Run Successful.

Actual behavior
No test is available in .../bin/Debug/netcoreapp2.0/PocketNurse.dll.
Make sure test project has a nuget reference of package "Microsoft.NET.Test.Sdk" and framework version settings are appropriate and try again.

Test Run Aborted.
Environment data
dotnet --info output:
.NET Command Line Tools (2.1.4)

Product Information:
 Version:            2.1.4
 Commit SHA-1 hash:  5e8add2190

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.13
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/2.1.4/

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.5
  Build    : 17373eb129b3b05aa18ece963f8795d65ef8ea54

The main project
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp2.0</TargetFramework>
    <UserSecretsId>aspnet-PocketNurse-36604171-3424-4347-80E3-30D24138EC43</UserSecretsId>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="EPPlus" Version="4.5.1" />
    <PackageReference Include="Microsoft.AspNetCore.All" Version="2.0.7" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Tools" Version="2.0.2" PrivateAssets="All" />
    <PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="2.0.3" PrivateAssets="All" />
  </ItemGroup>

  <ItemGroup>
    <DotNetCliToolReference Include="Microsoft.EntityFrameworkCore.Tools.DotNet" Version="2.0.2" />
    <DotNetCliToolReference Include="Microsoft.Extensions.SecretManager.Tools" Version="2.0.1" />
    <DotNetCliToolReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Tools" Version="2.0.3" />
  </ItemGroup>

</Project>

The test project
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netcoreapp2.0</TargetFramework>

    <IsPackable>false</IsPackable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="15.7.0" />
    <PackageReference Include="Moq" Version="4.8.2" />
    <PackageReference Include="MSTest.TestAdapter" Version="1.2.1" />
    <PackageReference Include="MSTest.TestFramework" Version="1.2.1" />
  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\PocketNurse\PocketNurse.csproj" />
  </ItemGroup>

</Project>

Why is it aborted?
Copied from original issue: dotnet/cli#9168

I believe --filter does not work that way. If you are trying to filter your tests to a particular class or test name, you need to do something similar to this: dotnet test --filter="FullyQualifiedName~ImportIndexTest"

@kevinburton the way you are providing Filter should work, it's possible such a test does not exists, but the platform should not say run test aborted.
I was unable to repro this on a simple netcore UT projects(no abort), can you please share logs by passing --diag

@mayankbansal018 I have included a copy of the output when I start the test (interesting that I do get a test succeeded along with a test aborted message). I also included the --diag output (There seems to be two logs generated. The output is also placed in a file in the test project.)
Thank you.
output.log
test.log

@KevinBurton I looked at the logs.
a. The first part of the test run which shows you the warning is for the product dll (/Users/rebeccaannburton/Projects/PocketNurse/PocketNurse) which does have the reference to Microsoft.Net.Test.Sdk and no tests.
b. Second part is for the actual test dll (/Users/rebeccaannburton/Projects/PocketNurse/PocketNurseTest) and the given test passes as expected as per the logs.


How to check if User Stream is active before creating a new one

Hello all,
First time posting here (on GitHub in general). I have used Tweetinvi to create a program called "Tweet Remote Control" for astrophotography purposes (to remotely control one's observatory equipment via Twitter). More about it here: http://www.lightvortexastronomy.com/tweet-remote-control.html
When my program starts, it calls a subroutine which has the following code:
var TwitterStream = Stream.CreateUserStream(); TwitterStream.TweetCreatedByMe += (sender, args) =>
This creates a new user stream for tweets created by the user, which works perfectly well. However, since my program runs 24/7 at a remote observatory, it is possible for the Internet connection to drop at any point in time and then return a few minutes later. Therefore, in my program, there's a timer that ticks every two minutes. It calls the following line of code:
Auth.SetUserCredentials(ApplicationAuthenticationKeys.strConsumerKey, ApplicationAuthenticationKeys.strConsumerSecret, strAccessToken, strAccessTokenSecret);
The timer is meant to re-authenticate with Twitter. However, I wonder, is the user stream created earlier still active when the Internet connection drops and returns a few minutes later? If not, I would need to re-create it. I can recall the subroutine that does this, but unfortunately that means creating a new user stream every two minutes. I have observed this seems to create "multiple instances" to my program running code that responds to tweets created by the user, which I certainly want to avoid.
My question is - is there a way to check if the user stream is active prior to deciding if to create a new one? In theory the code would have to go before I define var TwitterStream = Stream.CreateUserStream();, to check, but I can't figure out an efficient way to do it that works.
Of course, if the user stream remains active when the Internet connection returns (given the line of code in my two-minute re-authentication timer), then my question is null. :) Thank you in advance!

Hello there,
Thank you for taking the time to contact me here! Your application looks great, I look forward to use it!
Regarding your code here are few comments that I can provide.

This creates a new user stream for tweets created by the user, which works perfectly well. However, since my program runs 24/7 at a remote observatory, it is possible for the Internet connection to drop at any point in time and then return a few minutes later. Therefore, in my program, there's a timer that ticks every two minutes. It calls the following line of code:

You do not need to do that, you can use the StreamStopped event.
var stream = Stream.CreateUserStream();

stream.StreamStopped += (sender, args) =>
{
    if (args.Exception != null || args.DisconnectMessage != null)
    {
       // A timer can be required here to reduce the number of times this is executed.
       // If the internet has stopped working for a long time, it would reduce the number
       // of requests executed on your network.
        stream.StartStream();
    }
};

The timer is meant to re-authenticate with Twitter. However, I wonder, is the user stream created earlier still active when the Internet connection drops and returns a few minutes later? If not, I would need to re-create it. I can recall the subroutine that does this, but unfortunately that means creating a new user stream every two minutes. I have observed this seems to create "multiple instances" to my program running code that responds to tweets created by the user, which I certainly want to avoid.

If a stream stops because of an internet issue, the connection will be closed and the stream will no longer retrieve any tweet if you have not restarted as explained above.

My question is - is there a way to check if the user stream is active prior to deciding if to create a new one? In theory the code would have to go before I define var TwitterStream = Stream.CreateUserStream();, to check, but I can't figure out an efficient way to do it that works.

Yes there is. You can use the stream.StreamState property:
bool isStreamRunning = stream.StreamState == StreamState.Running;

Of course, if the user stream remains active when the Internet connection returns (given the line of code in my two-minute re-authentication timer), then my question is null. :)

As explained this is not the case, I would strongly suggest that you use the event.

Thank you in advance!

Thank you for your awesome project. I like when people have such good ideas.
Good luck with it!

Thank you so much for your excellent response. It has cleared up my issue. I have now coded in the TwitterStream.StreamStopped += (sender, args) => block of code and am testing it at my own observatory. I have also commented out the code that deals with the two-minute re-authentication timer. If this works out well, it'll be the perfect solution! :)
Again, thank you so much for your work on Tweetinvi. It's absolutely brilliant.

I just wanted to update this to say that it's completely sorted now. I tested the code for two days and it has worked perfectly. Thank you so much! :)


fix: Countdown timer Issue.

Description:
Solves Issue: #22
This pull request introduces several enhancements to the countdown timer while addressing a key issue:
Randomized Countdown Start: When users visit the page, the countdown timer now initiates with a randomized time, adding excitement and unpredictability to their experience.
Page Reload Continuity: One of the primary improvements is ensuring that the countdown timer continues from where it left off when the page is reloaded. This update provides a seamless and consistent user experience, maintaining the timer's state across page refreshes.
"Deals Ended" Message: In addition to this, we've introduced a "Deal Expired" message to enhance user awareness. This message is displayed when the countdown timer reaches zero, making it clear to users when the countdown has concluded.
These changes collectively aim to provide an engaging, user-friendly countdown timer that maintains continuity across page reloads and offers valuable information with the "Deals Ended" message.
Your review and feedback on these changes are welcome. If you have any suggestions or require further adjustments, please don't hesitate to request a review or provide your insights.

Hey @adityaS011 Thanks for merging the pull Request , can you please add necessary labels for Hacktoberfest?


Add option to not cache-bust

Requests have a timestamp added to the end at https://github.com/MoOx/pjax/blob/master/lib/request.js#L15, presumably to get around caches. I don't want to get around caches in this way, since my pages are static. It would be good to have an option to skip adding this query string.

PR welcome :)

How do you get at the options from those lib files?

https://github.com/MoOx/pjax/blob/ee530f4c0a5f4f016f03601865ce36e386a0f7b1/lib/proto/parse-options.js

this.options was the answer I was actually looking for, but that helped too -- thanks.

Closing in favour of pull request #71.


Return fix result to STDOUT

@GrahamCampbell how I can return formatted output for this command: cat test.php | php-cs-fixer fix  --level=psr2 -?

I have a similar question. PHP Beautifier makes it possible to direct the output to a file with a custom extension (e.g. .fixed). This make it easy to compare in PHPStorm to see what fixes to take over and which not.
So, at least make it possible to use STDOUT or otherwise an output destination would be desired.

Could potnetially be looked at after https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1824?

https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1654 will fix the output streaming to STDOUT vs. STDERR, we are close, hold on a bit longer ;)

Both PRs merged, closing

At time of writing, this issue has not been fixed.
However, it should remain closed: not because it has been fixed (to repeat: it has not been), but because it is a duplicate of #454.
Further remarks about this issue should therefore ideally be posted at #454.


compare date validation is not working

if i am using this in my model rules
            'message'=>'{attribute} must be greater than {compareValue} '],```
then it is only comparing with date not with month or year
if start_date is 01-05-2014 and end_date is  01-05-2015,, it is giving error end_date must be greater than start_date
the i changed to ajax validation ,write my own function
```code
   public function validateDate($attribute, $params) {
     st= strtotime( $this->start_date);
     $et=strtotime( $this->end_date);
     if($et<$st){
       $this->addError($attribute, Yii::t('app', 'end date must be greater than start date'));
     }
   }

with ajax it is working but i cant use ajax because i submitting my form with method get,so on get method ajax validation is not working.

Dates if such format as '01-05-2014' will be compare as a plain strings. So there is not surprise you have unexpected result.
If you wish to perfrom correct comparision, you should consider to apply DateValidator first and convert date values into timestamps before actual comparision.

@klimov-paul but why ajax validation is not working if form is submitting via get method

there is no docs about compare date so plz it will helpful if you give example

why ajax validation is not working if form is submitting via get method

If you have problem with ajax validation - open a separated issue for it, but first search for existing ones.

there is no docs about compare date so plz it will helpful if you give example

Please use Official Yii forum to ask a questions - GitHub is a bug tracking system.

i asked issue lots of time non one reply because there is no docs only who are at master level they can resolve otherwise it is not possible to get your answer or move to laravel
in december 2014 i asked this question i did nt get answer and today i posted this issue again here is link
http://www.yiiframework.com/forum/index.php/topic/63799-pagination-is-not-working-with-arraydataprovider/
now u tell me wat to do if no one is replying,that why every one is asking question here


chore(deps): update common-dev-assets digest to 8631a29

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
c46827e -> 8631a29

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.0.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix wrong daemon options at systemd config

Description
I'd like to fix the wrong daemon options at systemd configuration file of chef-client.
Issues Resolved

#539

Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Nice catch. Thanks


"Model was trained without prototypes" when using a loaded scPoli model

Hi @cdedonno,
Thank you for the great package.
I am following the scPoli tutorial
and running into an issue with the .classify() function, but only if the query model has been saved to disk and then loaded.
The code below runs fine if scpoli_query has just been trained and never stored to disk.
results_dict = scpoli_query.classify(
    adata_query.X,
    adata_query.obs[condition_key]
)

However, when I do
scpoli_query.save(query_model_path)
scpoli_query_loaded = scPoli.load(query_model_path, adata=adata_query)

results_dict = scpoli_query_loaded.classify(
    adata_query.X,
    adata_query.obs[condition_key],
)

Then the I get the AssertionError from the following line:
assert self.prototype_training_ is True, f"Model was trained without prototypes"
This must be because self.prototype_training_ is not saved with the model. Would appreciate your help with this!

Hi @maarten-devries, thanks for the interest and for reporting this. I'll try to fix it asap.

@maarten-devries I think it should be fixed now, can you check?

Yes, that fixed the error. Thank you very much!

Hi @cdedonno,
Towards the end of the tutorial notebook (under Sample Embeddings), there is still one more related issue that arises when using a a model loaded from disk.
scpoli_query_loaded.get_conditional_embeddings() yields ValueError: Length of passed value for obs_names is 17, but this AnnData has shape: (83, 3).
To illustrate what's going wrong:
embeddings = scpoli_query.model.embedding.weight.cpu().detach().numpy()
embeddings.shape[0] is 83
len(scpoli_query.conditions_) is 83
However, scpoli_query.obs_metadata_.shape[0] is 17 (should be 83)
For the reference model: len(scpoli_model.conditions_) is 68 (or 83 - 17)
I see that in load_query_data(), there are some lines that update obs_metadata:
obs_metadata = attr_dict["obs_metadata_"]
new_obs_metadata = adata.obs.groupby(condition_key).first()
obs_metadata = pd.concat([obs_metadata, new_obs_metadata])
...
new_model.obs_metadata_ = obs_metadata

However, this new obs_metadata is not retained when the model is saved to disk.
Really appreciate your help!

Thanks for reporting this. Sorry for the inconvenience, I will try to fix it asap.

Should be fixed now, can you check?

It works now, thank you very much for the quick fixes!
Just a note on a specific edge case I ran into (this may not require a fix, but I just wanted to bring the behavior to your attention) : the current implementation doesn't support the case where adata_ref and adata_query contain cells from the same batch (in my case, donor_id).
In my example, I have 2 donor_ids that occur both in adata_ref and in adata_query.
Therefore, len(scpoli_query.conditions_) is 83, but scpoli_query.obs_metadata_.shape[0] is 85, and as a result, scpoli_query.get_conditional_embeddings() errors out.
Again, this seems like a minor edge case, but good to be aware of. It could be good to add a simple check that there is no overlap in the batch key of adata_query and adata_ref.
But I'll close this issue for now :)

Thanks for reporting the edge case, I will fix that as well!


Reload opens the last Published version, not the last minor version

If you do not Publish (with a click on the green checkmark) and close the browser,
next time Cisar will reload with last Published version and make that the (overwritten) minor version.
Workaround is

close F12 developer tools
checkout the file in the style Library
check the Version history for the last version nyou editted
restore it
Check file in
open F12/Cisar again

Thanks a lot for your bugreport! I think I was able to reproduce the issue. Will fix it asap.

In the latest version this issue doesn't appear any longer. I'll test a bit more and push the update to Chrome Web Store.

Hi, I pushed v1.2 to Web Store! Could you please check if it works for you now?

Out of the office for 2 days.. will check on tuesday.
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-10-25 20:54 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
Hi, I pushed v1.2 to Web Store! Could you please check if it works for you
now?
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-150963559.

I tested it on a client machine, now when cisar opens the spinnig circle
just keeps spinning and nothing happens.. no errors in console.
Just tested it on my home machine.. same thing.. spinning circle.. nothing
else
Uninstalled Cisar, used another Chrome profile... same thing
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-10-25 20:54 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
Hi, I pushed v1.2 to Web Store! Could you please check if it works for you
now?
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-150963559.

This was an extremely silly mistake. Huge thanks for reporting it! I forgot to include 2 js files into the package when publishing it to Chrome WebStore (because their names changed and I create package using a script where names are hardcoded), so the WebStore version actually wasn't working at all :(
Fixed this and published the new version (it might take up to 1 hour for the update to be propagated to Chrome Web Store).

version 1.2.1
The file is created and saved in the style library
Now any edits in the F12 Cisar does not have any effect.
and revert to your original template when pressing F5
The publish (green checkbox) says it published the file
but the style library version is still a minor version.
(clicking the green checkmark multiple times does NOT create any more minor
versions)
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-10-31 17:22 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
This was an extremely silly mistake. Huge thanks for reporting it! I
forgot to include 2 js files into the package when publishing it to Chrome
WebStore (because their names changed and I create package using a script
where names are hardcoded), so the WebStore version actually wasn't working
at all :(
Fixed this and published the new version (it might take up to 1 hour for
the update to be propagated to Chrome Web Store).
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-152746756.

Heck, one more essential file seems to be missing from the WebStore package... I'm updating it one more time now. I'll check the WebStore version when it's updated and inform you.
Good thing is that local version works very well for me, so basically it's just technical problems because of some files missing from the package. Sorry and thanks again for your bug reports. This is invaluable! :+1:

1.2.2
It had a problem with the file that 1.2.1 created:

File Not Found

I deleted it using the x in Cisar. It was deleted from the Style Library,
but Not from the JSlink setting; so appears in the cisar panel again on F5
Only way to get rid of it is to go into the webpart properties and delete
the link manually.
Which makes for a feature request: Edit the JSlink for (all webparts) on
the current page, without using that dreadfull SP UI.
Second try:
I deleted all JSlinks manually, cleaned style library
Created a new file with cisar, it exists in the style library, the JSlink
on the webpart is correct:
But cisar says:

File Not Found

Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-10-31 20:53 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
Heck, one more essential file seems to be missing from the WebStore
package... I'm updating it one more time now. I'll check the WebStore
version when it's updated and inform you.
Good thing is that local version works very well for me, so basically it's
just technical problems because of some files missing from the package.
Sorry and thanks again for your bug reports. This is invaluable! [image:
:+1:]
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-152766637.

Hi,
I'm testing it right now with 1.2.2 and for me it works fine.
Maybe I'm doing something not same as you're.
So what I did:

navigate to a page which contains a List View (also tried with List Form)
F12, go to Cisar tab
create new file OR make changes to existing file (tried both, and also tried creating file in non-default folder)
go to Console, wait a little bit (4-5 seconds) until appears "Cisar: file saved successfully"
close devtools
F5
open devtools, go to Cisar, click to the file, it's contents loaded - everything OK

I also tried this both on root site collection and non-root site collection.
Maybe I'm doing something not as you are?

Also I just checked versioning settings, on root site collection Style Library has "No versioning", and on non-root site collection Style Library has "Create major and minor (draft) versions" - so these two modes at least work for me.

This is the result I get at step 7:
[image: Inline afbeelding 1]
The complete line 9 is:
window.location.replace("https:\u002f\u002f365csi.sharepoint.com
http://u002f365csi.sharepoint.com\u002fsites\u002fVM\u002fPages\u002fPageNotFoundError.aspx?requestUrl=https:\u002f\u002f365csi.sharepoint.com
http://u002f365csi.sharepoint.com\u002fsites\u002fVM\u002fsites\u002fvm\u002fstyle\u002520library\u002fcsr3.js")
see.., there is double /sites/VM in there
and then this Cisar content is overwritten to the file in the style
library..
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-10-31 22:21 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
Also I just checked versioning settings, on root site collection Style
Library has "No versioning", and on non-root site collection Style Library
has "Create major and minor (draft) versions" - so these two modes at least
work for me.
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-152774166.

Unfortunately inline images from email are not shown on github. Anyway, I understand the issue but cannot reproduce it, so I'm trying to figure out what can be different in our setups.
As soon as I can reproduce it, I'll be able to fix it.
Can you check what is the path you're using when adding the file? (the "Change path..." link)
It must be relative to site collection, even if it starts with "/". So for your case it should be "/Style Library/", but not "/sites/vm/Style Library/"

Problem is with (re)opening the file: http://i.imgur.com/HBH5Won.jpg
cisar tries to read from the wrong location (the JSlink on the webpart is
correct)
and then immediatly saves this HTML to the correct location:
http://i.imgur.com/cWaCEZZ.jpg
thus overwriting the orginal file
I did one quick test on a production environment; this did not happen there.
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-11-01 9:26 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
Unfortunately inline images from email are not shown on github. Anyway, I
understand the issue but cannot reproduce it, so I'm trying to figure out
what can be different in our setups.
As soon as I can reproduce it, I'll be able to fix it.
Can you check what is the path you're using when adding the file? (the
"Change path..." link)
It must be relative to site collection, even if it starts with "/". So for
your case it should be "/Style Library/", but not "/sites/vm/Style
Library/"
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-152805831.

I was not able to reproduce, but I think I know what is the problem.
I analysed the code for getting file out from SharePoint, it reads like this:
            url = url.replace(_spPageContextInfo.siteServerRelativeUrl, '');
            if (url[0] != '/')
                url = '/' + url;
            var r = new Sys.Net.WebRequest();
            r.set_url(_spPageContextInfo.siteAbsoluteUrl + url + "?" + Date.now());

the url parameter that gets there is something like "/sites/vm/style%20library/test.js"
and then it tries to remove "/sites/vm" part from there, and then adds site collection absolute url to the beginning of the string (which is something like "http://mytenant.sharepoint.com/sites/vm").
in this case the problem most likely happens because "sites/VM" and "sites/vm" are of different case, and thus first replace doesn't work.
I will change the code so that this problem will happen no longer and push the changes a bit later today, create a site collection with uppercase letters, and inform you after I check that this new version works.
Huge thanks for reporting this bug!

There is also a casing thingie (not an issue because IIS is not case
sensitive) when you create files with capitals;
they are correctly saved with capitals, but displayed in Cisar with
lowercase.
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-11-01 13:24 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
I was not able to reproduce, but I think I know what is the problem.
I analysed the code for getting file out from SharePoint, it reads like
this:
        url = url.replace(_spPageContextInfo.siteServerRelativeUrl, '');
        if (url[0] != '/')
            url = '/' + url;
        var r = new Sys.Net.WebRequest();
        r.set_url(_spPageContextInfo.siteAbsoluteUrl + url + "?" + Date.now());

the url parameter that gets there is something like
"/sites/vm/style%20library/test.js"
and then it tries to remove "/sites/vm" part from there, and then adds
site collection absolute url to the beginning of the string (which is
something like "http://mytenant.sharepoint.com/sites/vm").
in this case the problem most likely happens because "sites/VM" and
"sites/vm" are of different case, and thus first replace doesn't work.
I will change the code so that this problem will happen no longer and push
the changes a bit later today, create a site collection with uppercase
letters, and inform you after I check that this new version works.
Huge thanks for reporting this bug!
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-152822502.

and the JSlink on the WP has the correct (upper)case as well..
so where do you get that lowercased url from??
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-11-01 14:19 GMT+01:00 Danny Engelman<EMAIL_ADDRESS>
There is also a casing thingie (not an issue because IIS is not case
sensitive) when you create files with capitals;
they are correctly saved with capitals, but displayed in Cisar with
lowercase.
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-11-01 13:24 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
I was not able to reproduce, but I think I know what is the problem.
I analysed the code for getting file out from SharePoint, it reads like
this:
        url = url.replace(_spPageContextInfo.siteServerRelativeUrl, '');
        if (url[0] != '/')
            url = '/' + url;
        var r = new Sys.Net.WebRequest();
        r.set_url(_spPageContextInfo.siteAbsoluteUrl + url + "?" + Date.now());

the url parameter that gets there is something like
"/sites/vm/style%20library/test.js"
and then it tries to remove "/sites/vm" part from there, and then adds
site collection absolute url to the beginning of the string (which is
something like "http://mytenant.sharepoint.com/sites/vm").
in this case the problem most likely happens because "sites/VM" and
"sites/vm" are of different case, and thus first replace doesn't work.
I will change the code so that this problem will happen no longer and
push the changes a bit later today, create a site collection with uppercase
letters, and inform you after I check that this new version works.
Huge thanks for reporting this bug!
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-152822502
.

The lowercased one comes from Chrome extension API I think.
I checked and 1.2.3 does work for me with site collections that have uppercase letters in their url.
Does it work for you now?

Yes, seems to work now.
But.. it now has problems saving the "other files on page"
http://i.imgur.com/YYZySQN.jpg
Can't recall if I ever tried that before.. those 3 files I edit with
another editor.
I still do not understand between saving and publishing you do.. (as it
does not 'Publish' to a major version)
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-11-01 18:55 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
The lowercased one comes from Chrome extension API I think.
I checked and 1.2.3 does work for me with site collections that have
uppercase letters in their url.
Does it work for you now?
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-152848826.

It's very good news it finally works :)
URLs are unfortunately extremely hard to make right. Sometimes they have "%20" and sometimes normal spaces, sometimes they have ~sitecollection token, sometimes they come in lowercased. And when I use different APIs, they also expect URLs in different formats :( Chrome extension limitations make it particularly troublesome to manage, because working with underlying page heavily depends on evals and thus making utility functions and using them everywhere is not that easy.
But hopefully with time I'll be able to make this URLs stuff easier to manage - I have some ideas already.
I investigated publish stuff and you were completely right, it didn't work. There was a lot of changes to saving functionality and publishing was broken during one of those it seems. Thanks for pointing that out! I fixed it (based on local testing) and the fix will appear in 1.2.4 as soon as it propagates to the Web Store.
I'll also definitely check "Other files" saving as soon as possible.
Once again thanks for your bug reports, and keep it up! :+1: I'm extremely eager to make Cisar stable, but it has been plagued with bugs for some time and despite my latest efforts even recent versions had some critical bugs in them, as it turned out.

I still think Cisar is the best thing since sliced bread.
But then again.. I am also impressed by LoRaWan technology recently :-)
Danny Engelman http://www.engelman.nl/
<EMAIL_ADDRESS>http://www.LinkedIN.com/in/dannyengelman
http://www.linkedin.com/in/dannyengelman
► 365CSI.nl http://www.CSI365.nl - Microsoft Office 365 Ervaren en Leren
bikeswap.org http://www.bikeswap.org - borrow a motorcycle, loan a
motorcyle
"Love the moment and the energy of that moment will spread beyond all
boundaries"
2015-11-01 21:47 GMT+01:00 Andrei Markeev<EMAIL_ADDRESS>
It's very good news it finally works :)
URLs are unfortunately extremely hard to make right. Sometimes they have
"%20" and sometimes normal spaces, sometimes they have ~sitecollection
token, sometimes they come in lowercased. And when I use different APIs,
they also expect URLs in different formats :( Chrome extension limitations
make it particularly troublesome to manage, because working with underlying
page heavily depends on evals and thus making utility functions and using
them everywhere is not that easy.
But hopefully with time I'll be able to make this URLs stuff easier to
manage - I have some ideas already.
I investigated publish stuff and you were completely right, it didn't
work. There was a lot of changes to saving functionality and publishing was
broken during one of those it seems. Thanks for pointing that out! I fixed
it (based on local testing) and the fix will appear in 1.2.4 as soon as it
propagates to the Web Store.
I'll also definitely check "Other files" saving as soon as possible.
Once again thanks for your bug reports, and keep it up! [image: :+1:] I'm
extremely eager to make Cisar stable, but it has been plagued with bugs for
some time and despite my latest efforts even recent versions had some
critical bugs in them, as it turned out.
—
Reply to this email directly or view it on GitHub
https://github.com/andrei-markeev/cisar/issues/6#issuecomment-152862635.

So it is so that the original issue doesn't happen anymore? Should we close this issue?

Seems okay now, haven't tested any edge cases.


Suboptimal compilation with template literals

When a template literal contains a dynamic variable (eg: function argument), the rest of the template isn't optimized as much as it could be. In particular, ${..} is kept around constants, where it could be removed.
Code:
# x.js
window['f'] = function(n) {
  alert(`Hey, ${"Maxime"}. You have ${n} cookies. 10+2=${10+2}`);
}

// This works:
{
  let n = 10;
   alert(`Hey, ${"Maxime"}. You have ${n} cookies. 10+2=${10+2}`);
}

Output:
$ google-closure-compiler --js x.js -O ADVANCED
window.f=function(a){alert(`Hey, ${"Maxime"}. You have ${a} cookies. 10+2=${12}`)};alert("Hey, Maxime. You have 10 cookies. 10+2=12");

Expected output:
window.f=function(a){alert(`Hey, Maxime. You have ${a} cookies. 10+2=12`)};alert("Hey, Maxime. You have 10 cookies. 10+2=12");

Diff:
- window.f=function(a){alert(`Hey, Maxime. You have ${a} cookies. 10+2=12`)};alert("Hey, Maxime. You have 10 cookies. 10+2=12");
+ window.f=function(a){alert(`Hey, ${"Maxime"}. You have ${a} cookies. 10+2=${12}`)};alert("Hey, Maxime. You have 10 cookies. 10+2=12");

This does reproduce with the latest version:
$ google-closure-compiler --version
Closure Compiler (http://github.com/google/closure-compiler)
Version: v20221102
$ google-closure-compiler --js x.js -O ADVANCED
window.f=function(a){alert(`Hey, ${"Maxime"}. You have ${a} cookies. 10+2=${12}`)};alert("Hey, Maxime. You have 10 cookies. 10+2=12");

I happened to also run this under v20210808, where it does not reproduce:
$ google-closure-compiler --js x.js -O ADVANCED
window.f=function(a){alert("Hey, Maxime. You have "+a+" cookies. 10+2=12")};alert("Hey, Maxime. You have 10 cookies. 10+2=12");

Seems that this is a duplicate of https://github.com/google/closure-compiler/issues/3938.

Looks like it 👍  closing this issue, I'll watch the other one
thanks


Terminal Support within Neovim

Hi,
Great work on this colorscheme! I was wondering if you were looking to add support for the terminal buffer within neovim? I imagine the comments in this thread could be of help? https://github.com/chriskempson/base16-vim/issues/69
Thanks,

Was wanting to do this for a while!
Thanks for reminding me! 😀


Implement search page

Description
Implement the search page
Additional context
Before submitting the PR, please make sure you do the following:

[X] Add your change under the Unreleased heading in the package's CHANGELOG.md
[X] Read the Contributing Guidelines
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
[X] (Shopifolk only) Open a PR in the Shopify Dev Docs with updates to the Hydrogen documentation, if needed

this one looks great, I think an autocomplete component based on algolia or any other search mechanism would be great

Closing since we've moved the template and are waiting on design. Feel free to re-open when ready ❤


Its possible conflicts happen ?

it is possible to occur conflicts with $conn->resourceID?
I realized that it uses something like a count starting from the moment the server starts ..

Not likely as the resourceID comes from the stream socket id.


How Can I initialize pool using PDA account?

Hey, How Can I initialize pool using PDA account?
PDA account is a System account, but creator is required Signer account.

Hey, How Can I initialize pool using PDA account?
PDA account is a System account, but creator is required Signer account.

You can refer to this link: https://solana.stackexchange.com/questions/6528/can-we-use-pda-as-signer-and-payer-to-create-an-account-in-any-solana-program

@RainRaydium
Regarding the reference link you provided, one answer said it is not possible, while another answer said it's possible.
From my perspective, utilizing the PDA as the payer seems viable.
Could you please confirm this?


improve doc strings

should resemble: https://ewatercycle.readthedocs.io/en/latest/autoapi/ewatercycle/base/forcing/index.html
with examples etc in code

[ ] check code for
[ ] add example of use per class
[ ] explain thought process of implementation
[ ] improve formating

Better description of model origins and that its most like HBV-3 from https://urn.kb.se/resolve?urn=urn:nbn:se:smhi:diva-5738,
with HBV-96 being a more modern version: https://doi.org/10.1016/S0022-1694(97)00041-3 but containing 12 parameters insetead of the 9 used here


API returns deleted projects

After deleting a project through the API, it still shows up in the list of projects returned by the API - only with its name set to nil. This seemingly undesirable behavior should be fixed on the API side. We work around it by removing any result with an empty name.
This does not seem like desirable behavior for the API.  Once a project has been deleted, the API should not include it in the list or projects.

We see this behavior expressed several ways:
The list of projects returned by GET /api/1/projects includes projects that have been deleted.
The deleted projects have their name field set to null.  The deleted projects have a somewhat different set of fields than non-deleted projects.  The fields are mostly blank; those fields that are populated may be full of bogus data.
A call to GET /api/1/project/{projectId} will return 200 OK status and the same bogus project data structure.
Desired behavior

List projects should not return deleted projects.
Get project for a deleted project ID should return a HTTP status 404 Not Found.

API will list access tokens for deleted projects:
# Project delete suceeds
$ http DELETE https://api.rollbar.com/api/1/project/412646/access_tokens X-Rollbar-Access-Token:redacted

{
  "err": 0
}

# Current expected behavior for a deleted project
$ http GET https://api.rollbar.com/api/1/project/412646 X-Rollbar-Access-Token:redacted  | grep name
    "name": null

# We can still list project access tokens for the deleted project.
$ http GET https://api.rollbar.com/api/1/project/412646/access_tokens X-Rollbar-Access-Token:redacted

{
  "err": 0,
  "result": [
    {
      "project_id": 412646,
      "access_token": "2b4f4180c8814c519577889faf80c9db",
      "name": "post_server_item",
      "status": "enabled",
      "rate_limit_window_size": null,
      "rate_limit_window_count": null,
      "cur_rate_limit_window_start": null,
      "cur_rate_limit_window_count": null,
      "date_created":<PHONE_NUMBER>,
      "date_modified":<PHONE_NUMBER>,
      "scopes": [
        "post_server_item"
      ]
    },
    {
      "project_id": 412646,
      "access_token": "38eb9ceaca24408088d7f5ea0059dc90",
      "name": "post_client_item",
      "status": "enabled",
      "rate_limit_window_size": null,
      "rate_limit_window_count": null,
      "cur_rate_limit_window_start": null,
      "cur_rate_limit_window_count": null,
      "date_created":<PHONE_NUMBER>,
      "date_modified":<PHONE_NUMBER>,
      "scopes": [
        "post_client_item"
      ]
    },
    {
      "project_id": 412646,
      "access_token": "b320e4c7ca5d4171876ad95e3c05c4bd",
      "name": "write",
      "status": "enabled",
      "rate_limit_window_size": null,
      "rate_limit_window_count": null,
      "cur_rate_limit_window_start": null,
      "cur_rate_limit_window_count": null,
      "date_created":<PHONE_NUMBER>,
      "date_modified":<PHONE_NUMBER>,
      "scopes": [
        "write"
      ]
    },
    {
      "project_id": 412646,
      "access_token": "fb15423a455548b4afca410effdf968b",
      "name": "read",
      "status": "enabled",
      "rate_limit_window_size": null,
      "rate_limit_window_count": null,
      "cur_rate_limit_window_start": null,
      "cur_rate_limit_window_count": null,
      "date_created":<PHONE_NUMBER>,
      "date_modified":<PHONE_NUMBER>,
      "scopes": [
        "read"
      ]
    }
  ]
}

After investigating this I think this is intentional as I see there is a way to restore a deleted project. If someone accidentally deletes a project, there should be a way to restore that project. To restore a project one needs a project id. Maybe you can check if "status" is "enabled" for active projects otherwise they are deleted or can be ignored. @coryvirok, correct me if I'm wrong.

If the behavior is intentional, there is probably a better way to implement it.  Consider these results, showing 1 deleted project and 1 active project:
{
  "err": 0,
  "result": [
    {
      "id": 411692,
      "account_id": 317418,
      "settings_data": {
        "integrations": {
          "jira": {},
          "clubhouse": {},
          "bitbucket": {},
          "github": {},
          "trello": {},
          "slack": {},
          "datadog": {},
          "pagerduty": {},
          "gitlab": {},
          "webhook": {},
          "victorops": {},
          "ciscospark": {},
          "asana": {},
          "pivotal": {},
          "campfire": {},
          "azuredevops": {},
          "sprintly": {},
          "hipchat": {},
          "lightstep": {},
          "email": {},
          "flowdock": {}
        },
        "grouping": {
          "auto_upgrade": true,
          "recent_versions": [
            "5.0.0"
          ]
        }
      },
      "date_created":<PHONE_NUMBER>,
      "date_modified":<PHONE_NUMBER>,
      "name": null
    },
    {
      "id": 411704,
      "account_id": 317418,
      "status": "enabled",
      "settings_data": {
        "grouping": {
          "auto_upgrade": true,
          "recent_versions": [
            "5.0.0"
          ]
        }
      },
      "date_created":<PHONE_NUMBER>,
      "date_modified":<PHONE_NUMBER>,
      "name": "foo"
    }
  ]
}

The active project returns a field "status": "enabled".  So we would expect the deleted project to return a field "status": "disabled".  But it does not.  It does not return any status field at all.
Instead the deleted project's name is set to null.  That makes the data not very useful for restoring a project, since we don't know which project it was.

Another thing to consider...  The most recent time I ran the acceptance tests, listing all projects returned 756 items, of which only 4 were active valid projects.  So the API is pumping out ton of useless data just to get the small amount of useful data.  See raw_projects and cleaned_projects in the debug log below.
{"level":"debug","url":"https://api.rollbar.com/api/1/projects","raw_projects":756,"cleaned_projects":4,"time":"2020-10-15T18:59:20+07:00","caller":"/home/jmcvetta/projects/terraform-provider-rollbar/client/project.go:109","message":"Successfully listed projects"}

The acceptance tests run on each push to Github.  So the total number of deleted projects is going to keep climbing and climbing.  Each deleted case that's sent over the wire increases database load, CPU load, and network usage.  And the data really has no utility at all, because a) the API doesn't currently have an "undelete" endpoint; and b) the project names are stripped out, so they're just anonymous ID numbers.

Looks like this is actually a bug in our API since it shouldn't return deleted projects. Ideally, we can support returning soft deleted projects by passing in a "deleted=true" or something similar.
@mrunalk can you file a story internally to fix that in the API and maybe a second to implement the option to return deleted projects?

Any movement on this?
As of right now, listing projects for my account returns 2985 projects.  100% of which are deleted.
$ http GET https://api.rollbar.com/api/1/projects X-Rollbar-Access-Token:redacted | jq ' .result | length'
2985

@jmcvetta, plan to take this up next on my list after #12 is fixed.

plan to take this up next on my list after #12 is fixed

Awesome!

I am no longer seeing deleted projects returned in the projects list.  @mrunalk are we good to close this ticket?

Yes. @jmcvetta we can close this ticket. I've released the fix for this.


Appstudio update test-component-pac-dvhw

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-dvhw-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
12 seconds
clone-repository

✅ Succeeded
21 seconds
build-container

✅ Succeeded
16 seconds
inspect-image

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
38 seconds
clamav-scan

✅ Succeeded
6 seconds
optional-label-check

✅ Succeeded
6 seconds
label-check

✅ Succeeded
8 seconds
show-summary


[BD] Support django major to 1.11

Description
Support django major to 1.11.
Part of .https://openedx.atlassian.net/browse/BOM-1360
Reviewers

[x] @morenol
[x]  Is this ready for edX's review?
[ ] @jmbowman

Post-review

[ ] Rebase and squash commits

Thanks for the pull request, @andrey-canon! I've created OSPR-4256 to keep track of it in JIRA. JIRA is a place for product owners to prioritize feature reviews by the engineering development teams.
Feel free to add as much of the following information to the ticket:

supporting documentation
edx-code email threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will still be done via the GitHub pull request interface. As a reminder, our process documentation is here.

LGTM

I've merged Travis support to master with https://github.com/edx/django-lang-pref-middleware/pull/5 -- the build is currently broken, but with your changes on this branch + a few linter issues, it should go green for everything. (I did a local merge and test.) I don't think the rebase/merge conflicts should be too bad.

@jmbowman  This is ready for another review :D

In the release notes, it should be added that the support to python 2.7 was dropped


ScrollViewer: Hold mouse wheel to pan is not supported

Which version of the app?
WinUI 3 Gallery
Description
In other scrolling containers like websites, users can hold down the scroll wheel of their mouse to pan continuously horizontal or vertical. This interaction is not supported for WinUI 3 ScrollViewers.
Screenshots
No response
Windows version
No response
Additional context
No response

Closing, was meant to file on actual WinUI platform.


Dexie throws QuotaExceededError even when I have enough space on my disk

Hi,
I am checking for error.name = AbortError and error.inner.name = QuotaExceededError.. This is being triggered even when I have a lot of disk space remaining. Is there some additional check that I am missing for identifying low disk space ?

If you're in chrome or firefox, you can check available storage using StorageManager api.

Also have a look at https://developer.mozilla.org/en-US/docs/Web/API/IndexedDB_API/Browser_storage_limits_and_eviction_criteria#Storage_limits

Hi @dfahlander ,
Thanks for the reply. I just went through the StorageManager APIs. Wouldn't it be expensive to check usage and quota frequently for catching QuotaExceeded.
Also isn't QuotaExceededError imply that writing to indexedDB has failed because of lack of disk space. Or it is a possible error code in other scenarios as well

@dfahlander the MDN documentation doesn't seem to be correct for Chrome.
// The amount of the device's storage the browser attempts to
--
  | // keep free. If there is less than this amount of storage free
  | // on the device, Chrome will grant 0 quota to origins.
  | //
  | // Prior to M66, this was 10% of total storage instead of a fixed value on
  | // all devices. Now the minimum of a fixed value (2GB) and 10% is used to
  | // limit the reserve on devices with plenty of storage, but scale down for
  | // devices with extremely limited storage.
  | // *   1TB storage -- min(100GB,2GB) = 2GB
  | // * 500GB storage -- min(50GB,2GB) = 2GB
  | // *  64GB storage -- min(6GB,2GB) = 2GB
  | // *  16GB storage -- min(1.6GB,2GB) = 1.6GB
  | // *   8GB storage -- min(800MB,2GB) = 800MB

// The amount of the device's storage the browser attempts to
--
  | // keep free at all costs. Data will be aggressively evicted.
  | //
  | // Prior to M66, this was 1% of total storage instead of a fixed value on
  | // all devices. Now the minimum of a fixed value (1GB) and 1% is used to
  | // limit the reserve on devices with plenty of storage, but scale down for
  | // devices with extremely limited storage.
  | // *   1TB storage -- min(10GB,1GB) = 1GB
  | // * 500GB storage -- min(5GB,1GB) = 1GB
  | // *  64GB storage -- min(640MB,1GB) = 640MB
  | // *  16GB storage -- min(160MB,1GB) = 160MB
  | // *   8GB storage -- min(80MB,1GB) = 80MB

Ref: https://chromium.googlesource.com/chromium/src.git/+/refs/tags/76.0.3775.5/storage/browser/quota/quota_settings.cc

Thanks @holmberd for this finding! Would you think it would be good to have a notice about this on the Storage Manager page? Would you (if so) have a chance to edit that page to acknowledge this current behavior in Chrome? Might be useful for people that want to understand the API.
Thanks,
David

@dfahlander No worries. I'm doing some research around it since I have an app running in production that needs to handle these kind of errors gracefully. Looking at potentially switching from my own IndexedDB wrapper to Dexie.
It would probably be a good idea to mention it on the Storage Manager page. I've added a note about Chrome quota limits to the MDN page you referenced above, and we could probably add a note and link to that page from the Storage Manager page.

Hi @dfahlander  or anyone who spent time in reading me.
I have a hybrid app, android and web view. I have a weird error and I don't know why. I mean, I insert like 40,000 entries using the bulkAdd function and the app support this. then after a while I delete the db using the delete function, and I insert again the same entries, and the app works well, but if I try again for many time, in the fourth try the app show this error QuotaExceededError. the browser doesn't support Storage Manager API. I think if I delete the entire data, and then insert the same data. the error shouldn't exist.
Please If someone could help me, I don't know if Dexie save a history or something that produce this error.
Thank you.

Dexie does not store history unless you are using add-ons like dexie-observable. Even when using that add-on, deleting the whole database will delete history as well.
Anyone else that has experience with this error after deleted and recreated database?

We hit issues with this recently, trying to warn our users before they ran out of local space.
https://bugs.chromium.org/p/chromium/issues/detail?id=960305 is a Chromium change currently being rolled out, so not affecting everyone yet. It prevents visibility of how much space is available to avoid some kind of side-channel attack where an external resource's size can be inferred. Looking on chrome://quota-internals shows plenty of space even when usage is close to (or in some cases we have even seen greater than!) the quota.
Does Dexie use storage.estimate() and throw if there is not enough space? We have started seeing erroneous QuotaExceededErrors from at least one user. If not Dexie, I guess I will raise an issue with Chromium.

Dexie never checks with storage manager in advance to predict if an operation would fail. If an operation fails with QuotaExceededError, it comes directly from a transaction error from IDB.

Very useful to know - and also relevant to the Chromium issue, does Dexie throw QuotaExceededErrors itself, or would that always be the browser?

What happens in Chrome on QuotaExceeded is rather special and strange. This was nailed down by a dexie user in #776 - that neither "success" or "error" events of the IDBRequest are being called. Instead, IDB will trigger the "abort" event on the transaction. What Dexie does then, is to provide the error from the IDBTransaction of the abort event into the "inner" property of Dexie's AbortError. As a result, it is the promise returned from db.transaction() that fails with the AbortError containing the QuotaExceeded error from IDB, while the promise from the specific operation never settle.


How to use the bidi algorithm with TinyUTF8?

TinyUTF8 is awesome, however, I am using it in a project where I encounter bidirectional text (Arabic + English) frequently, which is not represented correctly by TinyUTF8. Is there any way to use the bidi algorithm with TinyUTF8's string class?

Hi Sourav,
thank you for your Message!
As you probably already guessed, tiny-utf8 only does the detection of codepoint boundaries within UTF-8 encoded data. When you iterate your string, it is iterated from the first codepoint to the last (in the data), not the "leftmost" to the "rightmost". The bidi algorithm strictly speaking is a higher abstraction level than the one of codepoints (what tiny utf8 does) but at the same time requires a codepoint abstraction level. Therefore, tiny-utf8 is a viable foundation to implement the bidi algorithm on but the bidi algorithm is not in scope of it.
I hope, this answeres your question at least in the sense to what extent tiny-utf8 can meet your needs for this use case. If you have further questions, I'm happy to help you.
All the Best,
Jakob

Thanks a lot for the response. I have managed to solve my problem.
I was working on a unicode compliant text rendering engine for that I was using to render text in OpenGL ES. The target platform was Android where I had access to the ICU C API only (no UnicodeString :( ) owing to https://github.com/android/ndk/issues/548.
What I ended up doing was using TinyUTF8 as a container for my string while still using ICU for the bidi conversions. It worked out really well!
Maybe I'll do a pure bidi implementation soon using TinyUTF8 in the near future. Thanks a lot for your awesome project!

I'm very glad to hear you got it done! I would be very interested in how you got it done. Sounds like a great solution!
If you feel like it, I'd be happy to understand how you got it done. Thanks anyway for your encouraging message :)
Makes me happy to hear my project helped you.
Cheers!

I just closed it for now, as it is not an open "issue" :)


Allow user to select the project name in the Import flow

@gastaldi is this necessary? We auto-define the project name from the git imported repo

@ia3andy sometimes the git imported repo name is already used in another project in the same cluster and the user doesn't have permission to deploy to it.
In another words, it is correct to auto-define the project name based on the git imported repo name but it would be awesome if we could change that

Also since the Import flow is in Beta, we can bump this to Train M

Use case: generate new app in creator.  Go to import.  Because you cannot change the name on import, there are name clashes and failure.  Prioritizing to current Summit train.


Cli: run oxlint with --rules will also lint file unexpectedly

Run the following command to reproduce:
npx oxlint --rules

Duplicate of https://github.com/oxc-project/oxc/issues/1751


Copy/Paste/Move multiple files

Copy/paste multiple files dialog must have checkbox "Replace all" to replace all existing files instead of selecting "Replace" button on each one. Replacing a lot of files without this becomes very long.
Copy/paste multiple files with drag drop must have an option to move files, not just copy (drag with +shift or +ctrl)
Replace files dialog must show source and target files comparision on file size and create/modified dates

The second point is standard in win32, I really miss it.


Minimal versions of some deps

Old versions of attrs and jsonschema prevent eodatasets3 from even being imported.
Pip doesn't always check versions, but it's getting better and it's good to document these requirements.

Coverage remained the same at 85.638% when pulling b24f5be8fa514d78a5762aaf71ac41a50afad605 on minimum-deps into 72617c7ecfbdf407e0c182b7a5f67cff1ea662f9 on eodatasets3.


IFC displaced  unaligned geometries

Describe the bug 📝
while i am loading IFC file i got noticed displaced or unaligned geometries and i have checked it also using tutorial viewer and i got same result but i checked my ifc file with web-ifc  Viewer and it looks fine and every geometry is in its place
so i have checked it with streamer and IFCfragment loader and tutorial viewer and for all gave me displaced geometries

Reproduction ▶
No response
Steps to reproduce 🔢
No response
System Info 💻
"dependencies": {
    "bim-fragment": "^1.4.0",
    "camera-controls": "^2.7.4",
    "openbim-components": "^1.4.21",
    "stats": "^1.0.0",
    "three": "0.160.1",
    "web-ifc": "^0.0.53"
  }

Used Package Manager 📦
npm
Error Trace/Logs 📃
No response
Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a repository issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[X] Check that this is a concrete bug. For Q&A join our Community.
[X] The provided reproduction is a minimal reproducible example of the bug.

Hey @MohamedAbdelkrim-coder we would need to have the IFC to solve it. Can you share it privately with<EMAIL_ADDRESS>Thanks!

File privately shared here.

Hey @MohamedAbdelkrim-coder!
This has been already solved in web-ifc and pushed to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Please, check out!

I will close the issue now. Feel free to leave any messages here if you have further concerns.
Thanks!


Fix dashboard typos for CPU and DRAM metrics on Total Pod Energy Consumption graph

Fixes typo in Total pod CPU energy consumption metric query which resulted in missing CPU Total Pod Energy Consumption
Fixes typo in Total pod DRAM energy consumption metric query which resulted in missing DRAM Total Pod Energy Consumption on graph.

Both issues can be seen on dashboard posted in #118 PR. Now all of the metrics can be observed, as noted on screenshot below.

@Feelas thank you! There are a couple of fixes today, would you please try the latest image?


gestures/gesture_board.py -- docstring and screenshot

I'm not sure what is happening with those 3 consecutive spaces, we discussed this so much, you even wrote a style hook for it. The screenshot seems kinda off around the borders.
But other than that, +1 for it, if you fix the mentioned problems, this should be mergeable.

I run it manually; git gets enough confusion issues as it was.  Fixed it.  Generally, don't worry about the spaces.  A week ago there were hundreds of spacing issues in dozens of files.   Now there's only one in uix/slider.py.  If I miss a space, it will cleaned up the next batch of spacing updates.

Please run make hook, it is best to avoid committing errors, especially when we have a nice git hook for detecting them.

No.  Sorry.  It would break the github tools.
On Fri, Feb 13, 2015 at 11:42 PM, dessant<EMAIL_ADDRESS>wrote:

Please run make hook, it is best to avoid committing errors, especially
when we have a nice git hook for detecting them.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/pull/3027#issuecomment-74365855.

Closing this, ping us if you change your mind and commit the mentioned fixes on your branch, so we can reopen the pr.

OK, I'm pretty clear this has nothing to do with the code.  I have over
five pull requests sitting in limbo over spacing or semantics.   I cannot
do more work on Kivy with all the changes in limbo.
The pull requests submitted clearly improve the quality of Kivy, and you
seem to dervie some odd pleasure over trying to get your input into every
small piece.   I actually love to code, and find that part the enjoyable
portion of any contribution.  Quality improves as I touch the code, not
from constantly asking for permission for changes to be adopted.
What is it you hope to get out of your contributions to Kivy?
On Sat, Feb 14, 2015 at 11:07 AM, dessant<EMAIL_ADDRESS>wrote:

Closed #3027 https://github.com/kivy/kivy/pull/3027.
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/pull/3027#event-235879308.

All I was asking the whole time (over and over) is to respect our contribution guide: http://kivy.org/docs/contribute.html#code-contributions.
The rest of your post shows a lack of respect towards those who spend time reviewing contributions, so i don't see a point in arguing over this topic.


Docker documentation path independent

The current README for docker installation is dependent on the path where the repository is located. This small commit attempts to make explicit the use of the path where the repository is located.

If we have time, we should document this in [docs] so that it is rendered on readthedocs


Improved fetch of aircraft db

Codecov Report
Attention: Patch coverage is 23.37662% with 59 lines in your changes are missing coverage. Please review.

Project coverage is 69.98%. Comparing base (24ba17b) to head (682e893).
Report is 2 commits behind head on main.

Files
Patch %
Lines

src/aircraft.rs
11.86%
52 Missing :warning:

src/csv.rs
64.70%
6 Missing :warning:

src/fs_s3.rs
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #53      +/-   ##
==========================================
- Coverage   72.54%   69.98%   -2.56%
==========================================
  Files          16       16
  Lines         987      993       +6
==========================================
- Hits          716      695      -21
- Misses        271      298      +27

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


[FIX] Fix issues with Linux

This should resolve some issues with linux, as seen #69.

I was facing the similar issue when running :CodeSnap where the clipboard would remain empty. This PR fixed the issue on testing from my end
0S: Arch Linux (x86_64)


The resourceTemplate from [#588] would be better named metadataTemplate as that is all it applies to

deprecate the resourceTemplate and replace with metadataTemplate, we can remove in the next release as it won't have existed for long. Naming is hard :-)

am working on adding patch support to the resourceTemplate, which would expand the usage to match the name.

obsolete with https://github.com/artemiscloud/activemq-artemis-operator/issues/758


Capstone review

Congrats Sir, you have done a good project so far. According to the capstone requirement, it's a two pages project, not one page. I suggest taking into account this remark.

Thank you for your review 😃
I have made the required changes ♻


workspaces only on primary display

Hi, first off I am loving berry. It appears to be exactly what I have wanted in a wm and nothing more (which is great!)
Previously my daily driver was budgie-destkop, which uses a wm based off of gnomes wm, mutter. I am trying to replicate an optional mutter behavior, workspaces-only-on-primary-display.
My setup is simple:

Laptop display (0, primary)
2nd Display (1)

To describe the wanted behavior as simply as possible: Open firefox in workspace 0 on primary, 0 display. Open mpv in workspace, 0 on 2nd, 1 display. When cycling through workspaces on primary display mpv should always remain visible on 2nd display.
Is this workflow possible, or something with a similar result? Using the command / option save_monitor i j I have not been able to replicate it yet.
Any tips would be greatly appreciated.  Thanks for your time.
EDIT:
Put another way - the end goal would be to make a window sticky across workspaces.

This behavior is currently possible, but I believe that you're using the save_monitor command slightly incorrectly to achieve the desired result.
Workspaces cannot span multiple monitors. So, given your setup, you'd want to do something like the following:
berryc save_monitor 0 1 # this saves workspace 0 to monitor 1 (your second display)

Now, switch to workspace 0 and open mpv. Now, when you cycle through workspaces 1+, they should always appear on your laptop display (primary), and shouldn't cycle through the contents on your other display. If they do, then there is some sort of bug :(


ARROW-16556: [Go] Add Layout method to DataTypes

Not clear exactly how this is going to be used, so a few nitpicks but looks OK

@emkornfield This allows someone to grab the expected physical layout from a datatype such as how many buffers a type should have and so on. I use this in my implementation of the compute stuff for Go (I have an implementation of a basic framework + working Cast kernel) for determining how many buffers to pre-allocate based on the expected output type.


Components should listen on both ipv6 and ipv4

Describe the bug
Most/all of the components listen on <IP_ADDRESS>, which causes problems on IPv6 clusters, where often times Services are SingleStack and IPv6 only.
This causes errors in the probe components, preventing other components from starting:
Thu Aug 29 15:09:23 UTC 2024
Waiting for REST API endpoint to become available
nc: openebs-openebs-api-rest ([fdd2:14fe:fb0:1000::64bc]:8081): Operation not permitted

This can be fixed by changing switches like --grpc-server-addr and --http from <IP_ADDRESS>:port to [::]:port. Though I'll grant it's not fully clear to me the implications of this if the cluster is IPv4 only.
To Reproduce
Try to install OpenEBS with Helm on a cluster with the primary Service CIDR being ipv6.
Expected behavior
Components should listen for connections from both IP families.
OS info (please complete the following information):

Distro: Talos
6.6.33
Mayastor 4.1

Seems reasonable to listen on "[::]:port".
Would you be willing to raise a PR for this?


Fix readme for actions

Under the new Actions section, this sentence isn't quite right:

This will clone the contents of user/another-repo on top of the existing repo.

It should read as something like this:

This will clone the contents of user/another-repo into the current directory. If duplicate files exist, the one in the current directory will overwrite the one in any cloned-down repos. This allows you to easily customize the final state of your scaffold when combining multiple repos.

Last sentence could also be cut if the intent of this behavior is clear enough.

Docs for actions should be clarified in https://github.com/Rich-Harris/degit/pull/65


[Docs] Maintain whitespace in propType documentation

When looking at PropType information for certain props the formatting puts everything on one line which makes it quite hard to read. This PR ensures that the whitespace extracted from react-docgen is preserved.
For example the documentation for Transforms goes from:

to:

Test plan:
Check PropType documentation is being rendered correctly on other components.
N.B. more style tweaks could be made here to make things more readable (e.g. reducing line height, monospace font) but I'll refrain from making them here in case there are explicit designs for the docs.

Good work, thanks for the PR!

@facebook-github-bot shipit

Thanks!


Wrong texture url

I've noticed when checking the demo that the jeep never appears on the model viewer. This is caused by the querying of the jeep's jpg texture file that is returning a 404 error.
The url currently querying is: models//.\jeep1.jpg
It is defined on the model's json: jeep1.ms3d.json line 41930.
I think it needs to be updated in order to point to the correct jpg path.

I've downloaded the project myself and saw this working so I'm closing it


validation and check in cron scaler for same start and end case

Signed-off-by: Ritikaa96<EMAIL_ADDRESS>Provide a description of what has been changed
In corn scaler, the target doesn't scale down when the start and end schedule are given the same input.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added
[ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[ ] A PR is opened to update the documentation on (repo) (if applicable)
[ ] Changelog has been updated

Fixes #2031

@zroubalik @ahmelsayed PTAL

I'd add a note to the docs here (https://github.com/kedacore/keda-docs/blob/master/content/docs/2.5/scalers/cron.md) about this error case. @tomkerkhove what do you think?

Agreed! Let's add it to all versions where CRON is available.

@Ritikaa96 looking good, could you please add a note to the Changelog (Improvements section) and add note to the docs as @ahmelsayed mentioned. Thanks!

Sure @zroubalik. Working on it.

@zroubalik @ahmelsayed @tomkerkhove
Added the note in docs with https://github.com/kedacore/keda-docs/pull/515
updated the changelog as well. PTAL

Thanks @Ritikaa96


installation fail

I have tried to download xsar packages and requirements directly. using conda install -c conda-forge xsar. Python version 3.8.  I have also tried to download from requirements with:  pip install -r https://raw.githubusercontent.com/umr-lops/xsar/develop/requirements.txt
got the error related GDAL. When I tried to force installation with sudo apt-get install libgdal-dev, it still doesnt recognise the module.
What else can I can try please?

Hi,
you can try 2 things:

make sure your conda is up to date
conda update conda
install rasterio and geopandas before anything else:
conda install rasterio geopandas
conda install xsar
(tested successfuly few minutes ago on Ubuntu 22.04.1 LTS).

conda install xsar
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: done
...
Preparing transaction: done
Verifying transaction: done
Executing transaction: done

Which OS are you using?
The auto build are OK on all systems accept for version python 3.11 numba not yet supported)

I close this issue, do not hesitate to open again if needed.

I am using poetry and was seeing this error after poetry add xsar:
      In file included from /home/jthetzel/.pyenv/versions/3.10.13/include/python3.10/Python.h:130,
                       from extensions/gdal_wrap.cpp:180:
      /home/jthetzel/.pyenv/versions/3.10.13/include/python3.10/ceval.h:122:37: note: declared here
        122 | Py_DEPRECATED(3.9) PyAPI_FUNC(void) PyEval_InitThreads(void);
            |                                     ^~~~~~~~~~~~~~~~~~
      error: command '/usr/bin/gcc' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for GDAL
  Running setup.py clean for GDAL
Failed to build GDAL
ERROR: Could not build wheels for GDAL, which is required to install pyproject.toml-based projects

Following a hint from https://stackoverflow.com/questions/38630474/error-while-installing-gdal#comment70408136_38630941 , I tried:
poetry add GDAL==$(gdal-config --version | awk -F'[.]' '{print $1"."$2}')

Followed by:
poetry add xsar

And it works! 🎉


Daemon expiry in configs

Add a new field daemon_expiry to configs. It can be "never", or a date/time in the same format as we use for genesis timestamps.
If it's not "never", on startup, log the fact that the daemon will expire. The daemon shuts down with an exit 0 at the appointed hour, logging the expiry.
Closes #3398.

Yes, I tested with a valid timestamp to confirm that the daemon does shut down.


Task - Manager - API endpoint for search

Should allow for multiple filters (we might expand in the future).
For now profiles matching any classification and all skills should be returned.
Needs to handle pagination


Udacity DivOps Course - packer build error

I'm taking the Udacity course to learn more in the field of DevOps (https://github.com/udacity/devops-intro-project), but if I use the following line in the Windows command prompt:
packer build -only=virtualbox-iso application-server.json
I get the following error message:
Error: Failed to prepare build: "virtualbox-iso"

1 error occurred:
        * Deprecated configuration key: 'iso_checksum_type'. Please call `packer fix`
against your template to update your template to be compatible with the current
version of Packer. Visit https://www.packer.io/docs/commands/fix/ for more
detail.

==> Wait completed after 0 seconds

==> Builds finished but no artifacts were created.

.. Does anyone know how i can fix this?

Hi @Rania199935 , you should be able to run the command "packer fix" to fix the issue for this template. Also, have you tried sending the Udacity Devops project maintainers a message with this issue? Perhaps they might be better suited to handling this issue. I don't think this project, which is a Vagrant/Packer project for spinning up a security lab, is the right one for this topic.
Apologies!

@Rania199935 application-server.json isn't from this project. I think this issue might be misplaced.


Log X-Request-Id for failed tests

https://czi-sci.slack.com/archives/C023Q1APASK/p1665081289822889?thread_ts=1664995115.318699&cid=C023Q1APASK
I think in FE we’d be able to monitor the traffic and print out all X-Request-Id along with their API URLs

Done as part of: https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/single-cell-data-portal/3188


Add Netsparker.

Added Netsparker profile.

Hey @miynat, can you add a region to the readme please?

Done

:+1:


[bug] S3 list_object missing the NextMarkder is truncated

Reporting Bugs

put 100 object into one bucket
list_objects with batch_size as 10
list response will have the IsTruncated as True, but the NextMarker field is missed.

Closed with #2483


Image Caption Evaluation

Thanks for releasing this great project.
But, when I using the evaluation example script of COCO Captioning
python run.py with data_root=/data2/dsets/dataset num_gpus=8 num_nodes=1 task_finetune_caption_mle_coco per_gpu_batchsize=32 load_path=fiber_coco_caption.ckpt
I get the following error:
Traceback (most recent calls WITHOUT Sacred internals):
  File "run.py", line 25, in main
    model = FIBERTransformerSS(_config)
  File "/homeu/project/OVOD/FIBER/coarse_grained/fiber/modules/fiber_module.py", line 146, in __init__
    state_dict = swin_adapt_position_encoding(
  File "/home/project/OVOD/FIBER/coarse_grained/fiber/modules/swin_helpers.py", line 32, in swin_adapt_position_encoding
    pos_embed = pos_embed.transpose(0, 1).view(-1, before, before)
RuntimeError: shape '[-1, 23, 23]' is invalid for input of size 4900

What is the problem that fiber_coco_caption.ckpt model does not match the input size?
Hope to get your reply！

Thanks! Could you try changing 'task_finetune_caption_mle_coco' to 'task_finetune_caption_cider_coco' and see if it can fix the issue?

The task_finetune_caption_cider_coco also does not work since I don't have the coco-train-words.p  file

FileNotFoundError: [Errno 2] No such file or directory: 'coco-train-words.p'

When I trying the task_finetune_caption_gold_coco , the following error arised

TypeError: __init__() got an unexpected keyword argument 'prepare_data_per_node'

Do you have any other suggestions?

coco-train-words.p can be found here https://drive.google.com/file/d/1N_O7tkBjJRCueQj7MHMqrazDTWr4OCDE/view?usp=sharing
TypeError: init() got an unexpected keyword argument 'prepare_data_per_node' seems like a pytorch-lightning version issue. here's the package version information https://github.com/microsoft/FIBER/blob/main/coarse_grained/requirements.txt.
if these don't work, maybe you can just try adding resolution_before=576 to the command line for task_finetune_caption_mle_coco

coco-train-words.p can be found here https://drive.google.com/file/d/1N_O7tkBjJRCueQj7MHMqrazDTWr4OCDE/view?usp=sharing
TypeError: init() got an unexpected keyword argument 'prepare_data_per_node' seems like a pytorch-lightning version issue. here's the package version information https://github.com/microsoft/FIBER/blob/main/coarse_grained/requirements.txt.
if these don't work, maybe you can just try adding resolution_before=576 to the command line for task_finetune_caption_mle_coco

Thanks for your advice! It works for me.


Incorrect or not defined C++ version for Common project

After the last configuration commit, it seems the C++ version of Common project it's the wrong one (or it's not defined).

It seems the C++98/03 it's used for the project in cause.

What is that file? I could not find it

What is that file? I could not find it

What file? The __cplusplus it's a macro which it's defined automatically by the C++ language. The 199711L represent a not configured version of C++ and the default one. (More info: https://learn.microsoft.com/en-us/cpp/build/reference/zc-cplusplus?view=msvc-170)

I'm reopening this. It seems setting the cppdialect "C++20" in BuildCommon.lua set the dialect at C++14:

Removing the BuildCommon.lua from premake5.lua will set the correct C++ version when using files from common project into others.
Don't get it why it's not working.

Interesting, how did you come up with that solution?
Yes I believe it is actually fit for common project to be StaticLib

I just saw the difference between the 3 projects premake configurations. I tested with it.
I feel like changing kind to StaticLib may be a temporary solution until we come with something better. It's actually not a shared library and the solution it seems superficial.. :(

Did we want to turn Common into a static library? If you're getting an error about __cplusplus when editing in the Common project, it is because the Common project is set to nothing, no language or anything. That's because those files will be included by the Server or Client so it is just the IDE saying "this isn't defined" when it will be in the Server and Client projects. Since I'm pretty sure this shouldn't be giving any actual compiler errors, there isn't a problem other than the intelisense being annoying.
We can fix it by making Common a static library and setting the C++ version to C++20 yes, or we could probably just keep the kind to None and add the premake line to use C++20? Need to test.
Either solution sounds good, but if we make Common a static library, we need to change the client and server to link it.

So like we just chatted about, intellisense is being annoying and it would be worth turning Common into a static library and making sure server and client link it.

This is the behavior i observed:

If no cppdialect it's set for common project, it will assume C++98/03.
If the option cppdialect "C++20" exists under common project configurations, C++14 it's assumed.
Making it SharedLib will work as expected.

Also, some linking errors occurs when trying to use files under common to Client or Server project, solved here by changing their build configuration files: 82f9c43b200c0f8db1fa30533848f7f806304a16.

So like we just chatted about, intellisense is being annoying and it would be worth turning Common into a static library and making sure server and client link it.

It's not Intellisense, building fails too.


Classic autoloader support

Classic autoload reloading seems to be broken in the dummy app (and apps that use the classic autoloader). You can reproduce the error with the following:

cd test/dummy
CLASSIC_AUTOLOADER=1 bin/rails c
reload!

Reloading...
~/src/github.com/Shopify/maintenance_tasks/app/models/maintenance_tasks/task.rb:62:in `csv_collection': uninitialized constant CsvCollectionBuilder (NameError)

Adding a require_dependency to app/models/maintenance_tasks/task.rb fixes the problem, but our support for non-Zeitwerk applications is unclear. Should we support them, or not? If not, I suggest we remove the ENV var to boot in classic mode. Also, keep in mind Rails 7 will not include the classic autoloader as Zeitwerk has been the default for quite awhile now.

our support for non-Zeitwerk applications is unclear

We currently ensure that we're compatible with the classic autoloader by eager loading the app and requiring all task files in the engine:
https://github.com/Shopify/maintenance_tasks/blob/05f03c4467422934120963ce4afcbd19086c5fc0/lib/maintenance_tasks/engine.rb#L11-L13
https://github.com/Shopify/maintenance_tasks/blob/05f03c4467422934120963ce4afcbd19086c5fc0/lib/maintenance_tasks/engine.rb#L21-L26
So I'd say that we have tried to support it up until this point 😄  That said, I'm not sure it's worth doing. As you said, Zeitwerk is the default in Rails 6 apps, and classic autoloading won't be supported soon, so I think it's fair game to enforce Zeitwerk. It also lets us get rid of the above code. I'd be OK to disable classic mode and call it out in the docs.

I think it was just a matter of qualifying CsvCollectionBuilder (=> MaintenanceTasks::CsvCollectionBuilder).
I couldn't write a test to cause the exception though (using Rails.application.reloader.reload! or even including the Rails::ConsoleMethods and calling reload! wasn't working from a rake task for some reason).
I'm conflicted because this particular means of testing autoloading has been failing forever basically (since v1.1.1), but in general autoloading from the UI works, and you can run your task. I mean it's pretty bad that we broke reload!, but you could run your tasks with the classic autoloader, now the UI is entirely broken.
On the other hand, migrating to Zeitwerk doesn't seem too hard on users, and we can avoid having to support it…
I don't know I just feel bad dropping support abruptly. I guess we'll see if we get some bug reports whenever this is released.

On the other hand, migrating to Zeitwerk doesn't seem too hard on users, and we can avoid having to support it…

My feeling was that we'd been supporting classic autoloading in a half-broken manner up until recently with no complaints, and support for non-Zeitwerk apps is going to be dropped soon with Rails 7, so apps should be focused on migrating anyways.
I agree that dropping it with no notice is probably bad OSS etiquette though 😅
So maybe we should revert, qualify the constant to fix reloading, and soft deprecate for now?


Apply a role="region" to datagrid filter pop-up container along with a meaningful label.

Describe the accessibility issue
We got a report from AccessLevel where they note that role="region" should be applied to datagrid filter pop-up container along with a meaningful label.
WCAG Guideline
[Issue - WCAG20]
The datagrid filter pop-ups do not have an appropriate role and label to denote the boundaries of the pop-up.
[Impact]
Screen reader users will have difficulty identifying where the pop-up begins and ends, as well as the purpose of the pop-up.
[Recommendation]
Apply a role="region" to the expanded pop-up container along with a meaningful label provided using the aria-label or aria-labelledby attribute.
How to reproduce
https://clarity.design/documentation/datagrid/custom-filtering

Versions
App

Angular: [e.g. 6]
Node: [e.g. 8.10.0]
Clarity: [e.g. 0.12.5]

Device:

Type: [e.g. MacBook]
OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]
Screen Reader: [e.g. VoiceOver, NVDA]

For screen readers, we primarily support the following combinations, and other combinations will be considered on a case by case basis.

NVDA + Firefox (Windows)
VoiceOver + Safari (Apple)

Additional notes
Add any other notes about the problem here.

@scroniser is this valid or is it captured already in our list?

@gnomeontherun, this is already on our list.


Review the course website before sharing with students

@akshath03, @zofiatycz, Rosanne, and Domas
I have a final draft of the course website up here:
https://moorepants.github.io/me41055/
Can you all carefully read the three pages and reply here with any feedback on the clarity and completeness of the information? You can also read the panel feedback in #5 and see if my version of the course address things there (or introduces likely new issues). I also would like feedback on the academic integrity part, in particular. I thought about this and wonder your takes on how we will handle direct copying of homework solutions.
Please check this over in the next couple of days.

About academic integrity, rules are quite clear. However, it is not clear to me what to do with the example solution posted from the previous week, can it be used to solve problem student didn't cope with? The second remark is that last year using the help from other students was possible if it was clearly written who and to what extent helped with the code.
The content of the course is clear, as is the schedule of work.
There is the word repetition in the Vocareum part ("edit" is doubled)

In the assignments and grades section, it is not clear whether the students are supposed to submit individually if they are working in a pair or a single submission for the pair. I would suggest to submit individually which eases the process of grading and also for plagiarism checks.

However, it is not clear to me what to do with the example solution posted from the previous week, can it be used to solve problem student didn't cope with?

If we share a solution (ours or one of the student submissions), those are free to use in any way they want after it is shared. The subsequent homeworks should not rely on the solutions to specific problems from the prior week (maybe that is what you are implying?)

The second remark is that last year using the help from other students was possible if it was clearly written who and to what extent helped with the code.

My current plan is to require a "contribution statement" for students that submit the homework as a pair. That statement should explain who did what in that homework. For individual submissions, no statement is required. For all submissions (individual or pair) the code and solution has to be unique with respect to other student submissions as well as any existing materials students find (notebooks online, solutions online, past homeworks).
I can change this plan to make all students write a custom "contribution statement". This would require them to document any help they received in completing the assignment. Maybe that's simpler.

thought about this and wonder your takes on how we will handle direct copying of homework solutions.

Ideally we figure out how to run automated plagiarism detection on the submissions. If we can figure that out, I may have you all scan through a random subset of submissions manually to detect plagiarism.

I would suggest to submit individually which eases the process of grading and also for plagiarism checks.

I could have every student submit the notebook. If they are working in a pair, then those notebooks could be identical copies of each other. Given @zofiatycz's suggestion, I could have each individual student write their custom contribution statement whether submitted as a pair or individual, i.e. we get unique contribution statements from every student, but solutions can be identical with their submission partner.

Some feedback from a student from last year:
General notes:

I’m personally not familiar with Vocareum (I think) if that is the case for other students you
might have to explain what that is during the course

You will probably include this elsewhere, but it would be nice to have an overview of the
deadlines for the course. This means when to hand in which homework and when the
deadline for reviewing this homework is

Specific notes:
@ Time and Location, in the last paragraph; “you must self sign up” seems like a bit of odd phrasing.
(not entirely sure if you wanted feedback on stuff like this too).
@Assignment & grades, in the first paragraph; I would add “homework grade or IF you are taking a
resit exam”

One more: make sure the front page points them to the schedule page, as it seems people aren't seeing the other pages on the website.

The draft notification is now removed and the brightspace page is active.


guard-livereload issue due to symlinked dirs

When using guard-livereload, I noticed performance issues on OSX (100% CPU usage, livereload working unreliably). After upgrading to latest guard and guard-livereload gems, I noticed warnings related to duplicated directory errors. This was probably the underlying cause of performance issues in the first place.
I added a note to the README.md, including the workaround by explicitly specifying directories to watch.

@jesenko Thanks!  :beers:


Is there a way to make tendons collide with one another?

Hi,
I'm a student and I'm trying to make a stretchy rope by combining a series of geom spheres with short thick tendons in between.
(I'm doing this because I've tried using rope with main+stretch joints, but I found it kind of unstable and I wasn't happy with the result.)
The stretchy-ness of the rope is working great, but the trouble is that the rope does not get wound as a rope is supposed to.
As shown in the figure below, the pitch of the coil changes with tension, because the contact is working only among the geom spheres and not among the tendons.

So, I want to make my tendons collide with one another just like the geoms do.
I know it's a weird thing to ask, but to me it kind of seems like there is some sort of contact between tendons in the Tendon-driven leg example. Or maybe it's just transparent geoms.
I would be much obliged with any piece of advice.

This was asked and answered 2 days ago. The answer is unfortunately no, tendons cannot collide other than with wrapping geoms (which is not what you want, in your case).
As a temporary solution you can try to use capsules rather than spheres?
I can add that improving soft bodies generally in MuJoCo, and rope-like 1D soft bodies in particular, is something we consider high priority and are actively working on. When these are released we'll also ping you here in this issue.
Cheers

Hi traplotz, we will soon release a new rope/cable which is more stable than the current one. We will let you know once this is completed, it would be great if you could test it. The initial release won't be stretchable, since that would require updating the geometries for collisions, which is not supported yet. But it would be easy to add a stretchable joint as currently done for the composite rope (I could give you code for doing this).

@youvaltassa
I tried using capsules and it kind of does the job. The rope touches itself much better than it used to with spheres, and the pitch stays constant with different tensions.
The rope has become quite unstable, though. When I give the rope some tension and wait long enough, the capsules start to vibrate more and more wildly, and suddenly the runtime ends and starts over.
I'm not sure why but it seems like the global kinetic energy of the rope is transferred to each capsule and causes some sort of resonance.
I am testing with different site pos and tendon damping, etc, but the only remedy that seems to work is shorter timestep.

@quagla
It would be an honor to participate, but I'm not sure if I am qualified for the testing.
I am just a beginner and I dont' even know how to use the API yet.
If some inept beginner-level comments would be fine, please let me test the code.
Else, thanks for asking though.

Hi I'm glad to inform you that the new cable is available. Please have a look at https://mujoco.readthedocs.io/en/latest/changelog.html where there are 3 examples. I am really curious to know if you find this cable more stable than the rope.
Note that this cable is unstretchable, but you can add stretching either in the code (which sounds like you would rather avoid) or directly in the XML. If you load and save one of the 3 examples, you will see how the Composite is expanded in each single element that composes the rope. You could add one slider joint per element in order to obtain your stretch.


compiler crash on scala-logging 3.7.0 and Scala 2.11.11

Hi there! I am getting a compiler crash on scala-loggin 3.7.0 and Scala 2.11.11. I've resolved by going back to version 3.5.0 for now. I created a repository that shows the problem here: https://github.com/sullivan-/scala-logging.bug
But it's pretty small example:
import com.typesafe.scalalogging.LazyLogging

 trait Foo extends LazyLogging {
   logger.warn("foo")
 }

 class Bar extends Foo

Class Bar now crashes the compiler with:
[info] Compiling 1 Scala source to /Users/jsmscs/ws/scratch/scala-logging-bug/target/scala-2.11/classes...
exception when typing @volatile lazy private[this] val logger: com.typesafe.scalalogging.Logger = _/class scala.ref
lect.internal.Trees$ValDef
values cannot be volatile in file /Users/jsmscs/ws/scratch/scala-logging-bug/Bar.scala
scala.reflect.internal.Types$TypeError: values cannot be volatile
...

I can provide a complete trace of the compiler crash if you need it.

Fixed in 3.7.1.