Media error: Format(s) not supported or source(s) not found on the demo page

Using chrome latest version, the demo page on android devices is not loading Hls & mp4 videos with the following error.
"Media error: Format(s) not supported or source(s) not found"
while working fine in desktop mode
i have tested it 5+ android devices.
even using chrome dev tools and selecting another device (from the device toolbar) if you reload the page and tried running any of the sources mentioned above the same error message will appear

@ahmad-rzk mp4 updated, should be fine now

Hey @berryamin thank you,the mp4 source is working fine now.
I was able to fix the Hls source issue on android by manually specifying the "native_hls" renderer before the "html5" renderer, hope that helps.

MPEG-DASH not working on front page here https://www.mediaelementjs.com/

#2742
All ready an issue created for this see my last comment
For HLS the same issue applies as in #2742   Replace
HLS
With:  For example:
https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8

Hey @jaapmarcus
Thanks for your help, i was just wondering does the issue you mentioned occurs on android?
Because the issue (i even tried 4.2.14) is when i run the player from an android device it will give me the mentioned error while the player plays the same .m3u8 file fine on chrome desktop

@ahmad-rzk
What m3u8 source are you using the same in the example?
HLS
This one will not work any more it will return an 404 error not found similar to the error you are providing
Please replace it with
https://bitdash-a.akamaihd.net/content/MI201109210084_1/m3u8s/f08e80da-bf1d-4e3d-8899-f0f6155f6efa.m3u8

Thank you @jaapmarcus
i am using my own hosted .m3u8 source, i think the demo is updated now and the Hls source is not returning an error, with the renderer issue still present on android devices, the fallback to the native_hls is not happening gracefully (i guess or something like that)
Thanks again for your help.

@ahmad-rzk Could you please provide us with a test page. Just checked the home page  (Android Chrome) (mediaelementjs.com) and there its working fine....

Hey @jaapmarcus
Have you tried the Hls source on the demo page?
It still has the same issue
i am using
Google Chrome : Version 80.0.3987.132 (Official Build) (64-bit)
Here is an image from the DevTools
Please note that it is also the case from multiple android devices, even on my local machine where i tried a custom Mediaelement sample
As i mentioned as soon as i exit the responsive mode as soon as i refresh the page the link starts to work normally
Also if you started the link and then opened the DevTools in responsive mode the link will be playing fine until you refresh the page (load the entire page in responsive mood)

I spend today a few hours with testen and checking
When I use chrome on the website I have in Developer mode I have the same error. But currently the version used is 4.2.7
When I try to use the latest version https://codepen.io/jaapmarcus/pen/WNvJMLQ  it seems to work fine.

Hey @jaapmarcus
Yes it is working fine on 4.2.16
Want me to close the issue or wait until the demo page is updated to the latest version ?

demo page has been updated

Finally  :) 👍


Added Clickable Copyright

Hello @codervivek5 ,
Added a clickable copyright section using HTML, CSS & JavaScript
Screenshot -

Video -
Screencast from 26-05-23 01:29:09 AM IST.webm
All set and it is ready to Merge.

Thank you for your contribution.
Kindly close the issue related to this PR so that you can got you full points.


Dynamic VM ID generation

Hello Everyone,
Its not a bug report, I'm just reaching out to get some help to solve a problem of mine.
If you can suggest better forum for this that already helps.
Right now, we are creating Virtual Machines on Proxmox using this provider with Go.
Iterating through a set of input files where different VM details are described, generating VirtualMachineArgs, running NewVirtualMachine on each.
Having different parameters defined by us is  fine for the most part. Every VM has to have name, resource values, list of disk... etc.
Where my problem comes is the ID. If I change the ID of VM, then pulumi recreates the resource. Which is obviously not a desired outcome.
Okay, then do not change the ID you could say. And I do not want to.
But to dynamically generate the ID, I would have to re-calculate it every time we run 'pulumi up'.
The problem is, at the time of 'pulumi up' we cannot differentiate that its a creation of a new resource, or updating an already existing. Or we can, I'm just missing something.
There is this feature ignoreChanges, which works for most fields of the stack json, but not for the VMID, since its part of the resource URN. And not a solution, since I do not want to change the Id at the first place, so no point to ignore it anyways.
How could I only run a piece of code only at creation time, store the result somewhere, and later read the same parameter from that storage. Currently we do not use any sort of database apart from the pulumi stack file stored in S3, and we would like to keep this way unless there is no other option.

Hi miberecz,
I have just been playing around this weekend with this provider and just out of curiosity: why manage VM IDs in the first place? If you dont specify a VM ID, doesn't it get auto-assigned?

Every pulumi resource has an identifier, an URN as it called. Currently we assign the VM ID as URN. It has to be unique obviously. Other option for us would be the VM Name, but then we lose the ability to rename VM-s.
We could figure out a 3rd identifier just for URNs but so far it was just convenient to have them same for both in case we are looking for something.
But maybe you gave me an idea.

do i get it right that you are having a file which contains properties, e.g., name, cpu, memory, and a field id which people are, theoretically, allowed to change? if so, i'd argue that no one should change the field id.
otherwise, you could link, e.g., the file name to the pulumi identifier and let the URN auto-create or would this not work?
maybe you could share an anonymized file and the piece of code which generated the VM for better understanding?

I could be missing something obvious in your system, but can't you add another key in your yaml file so that you have name and vmName that are used to build the resource name and vm name respectfully?


Add banner on sidebar for writing challenge

Context
To try and get people to signup and write we are going to show the temporary writing challenge.
It will also help us get metrics on click-throughs.
The first issue is to add a banner before we can track it:

Are you thinking a reusable component for this or just the PNG/SVG in a div?

@JohnAllenTech - This image inside a container with the text:  and then the link going here: https://www.codu.co/articles/join-our-6-week-writing-challenge-quohtgqb
If that makes sense?
The Figma for the site designs is here: https://www.figma.com/file/yKc7ind3lHUTpqkjTePy1N/codu.co?type=design&node-id=159%3A167&t=KETEwehLrAZbzaI8-1

Sure. I hacked it together by just using the image you provided and displaying that.

Ill do it properly tomorrow if you want a reusable component for future announcements

That's perfect @JohnAllenTech! I think open a PR with this for now and we can worry about migrating it to a reusable component after 🙌


Fix compiler for staging query

Summary

Compiler is breaking for staging query due to no dependency handling in staging query. This PR is a hot fix for staging query.
Why / Goal

Test Plan

[ ] Added Unit Tests
[ ] Covered by existing CI
[x] Integration tested

Checklist

[ ] Documentation update

Reviewers
@rohitgirme @hzding621

@yuli-han can we also cherry-pick the change for the CI: https://github.com/airbnb/chronon/pull/831/files#diff-2ee999fe2c79ef0d107414996430bd309bc68d85dfa221dd1e865d970552bd54


D3.js implementation for table generation inside a markdown widget

Hi,
I am trying to use the d3.js to create a table in a dashboard markdown widget. So far all my tests have failed. Is this implementation possible? If so, is there any example of how I can do this?
Thank you in advance.

I've never used d3 tables in a widget. Could you post some code?

Hi,
Here is the example code I am trying to make it work as a first step:
'use strict';
angular.module('fff.table.widgets', ['adf.provider', 'btford.markdown'])
.value('githubApiUrl', 'https://api.github.com/repos/')
.config(function(dashboardProvider){
dashboardProvider
.widget('reservationTotals', {
title: 'Reservation Totals',
description: 'Reservation Totals widget',
controller: 'reservationTotalsCtrl',
templateUrl: 'scripts/widgets/ff_tables/ff_table.html',
resolve: {
read_data: function(FFFtradingService){
return FFFtradingService.get();
}
},
edit: {
templateUrl: 'scripts/widgets/ff_tables/edit.html',
reload: false
}
});
})
.service('SSStradingService', function($q, $http, githubApiUrl){
return {
get: function(/path/){
var deferred = $q.defer();
$.getJSON('9_Response.json', function(data) {
if (data){
deferred.resolve(data);
}
});
    return deferred.promise;
  }
};

})
.controller('reservationTotalsCtrl', function($scope, config, read_data){
var datapointsjson = {
    "type": "FeatureCollection",
    "features": [{
        "type": "Feature",
        "geometry": {
            "type": "Point",
            "coordinates": [
                136.33, -31.5
            ]
        },
        "properties": {
            "regno": "R123456",
            "taxon": "genus1 species1",
            "sitecode": "",
            "nearestnamedplace": "FREELING ISLAND",
            "preciselocation": "south coast",
            "collection": "Herpetology"
        }
    }, {
        "type": "Feature",
        "geometry": {
            "type": "Point",
            "coordinates": [
                137.07, -36.23
            ]
        },
        "properties": {
            "regno": "R654185",
            "taxon": "genus2 species2",
            "sitecode": "",
            "nearestnamedplace": "Neptune Island",
            "preciselocation": "Middle of island",
            "collection": "Herpetology"
        }
    }, {
        "type": "Feature",
        "geometry": {
            "type": "Point",
            "coordinates": [
                142.0358, -38.7719
            ]
        },
        "properties": {
            "regno": "R6528445",
            "taxon": "genus3 species3",
            "sitecode": "TT02",
            "nearestnamedplace": "Woolongong",
            "preciselocation": "5 km N",
            "collection": "Herpetology"
        }
    }, {
        "type": "Feature",
        "geometry": {
            "type": "Point",
            "coordinates": [
                137.6914, -32.2789
            ]
        },
        "properties": {
            "regno": "R654987",
            "taxon": "genus4 species4",
            "sitecode": "IL0601",
            "nearestnamedplace": "Ballarat",
            "preciselocation": "5.3 KM E",
            "collection": "Herpetology"
        }
    }]
};

//console.log(datapointsjson);

var headers = d3.keys(datapointsjson.features[0].properties);

var thedatatable = d3.select("#propertiestable")
    .append("table")
    .attr("class", "datatable");

var header = thedatatable.selectAll("th")
    .data(headers)
    .enter()
    .append("th")
    .text(function (d) {
        return d;
    });

var tr = thedatatable.selectAll("tr")
    .data(d3.values(datapointsjson.features))
    .enter()
    .append("tr");

var td = tr.selectAll("td")
    .data(function(row) {
        return headers.map(function(d) { console.log(row.properties)
            return {column: d, value: row.properties[d]};
        });
    })
    .enter()
    .append("td")
    .text(function (d) {
        return d.value;
    });

config.content = "

";

});

Tow problems, the first on is the $.getJSON. This method comes from jQuery and should not be used in an angular application, please use $http to retrieve the data. The next problem is the way you are trying to create the content of the widget, i think you have to create a directive for this.

The following example should work:
table.js
'use strict';

angular.module('adf.widget.table', ['adf.provider'])
  .config(function(dashboardProvider){
    dashboardProvider
      .widget('table', {
        title: 'table Title',
        description: 'Description of table',
        controller: 'tableController',
        templateUrl: '{widgetsPath}/table/src/view.html',
        resolve: {
          tabledata: function(tableService){
            return tableService.get();
          }
        }
      });
  })
  .service('tableService', function($q, $http){
    return {
      get: function(){
        var deferred = $q.defer();
        $http.get('9_Response.json')
          .success(function(data){
            deferred.resolve(data.data);
          })
          .error(function(){
            deferred.reject();
          });
        return deferred.promise;
      }
    };
  })
  .controller('tableController', function($scope, tabledata){
    $scope.data = tabledata;
  })
  .directive('d3Table', function(){
    return {
      restrict: 'EA',
      scope: {
        data: '='
      },
      link: function(scope, element, attrs) {
        var datapointsjson = scope.data;

        var headers = d3.keys(datapointsjson.features[0].properties);

        var thedatatable = d3.select(element[0])
               .append("table")
               .attr("class", "datatable");

        var header = thedatatable.selectAll("th")
             .data(headers)
             .enter()
             .append("th")
             .text(function (d) {
                 return d;
             });

         var tr = thedatatable.selectAll("tr")
             .data(d3.values(datapointsjson.features))
             .enter()
             .append("tr");

         var td = tr.selectAll("td")
             .data(function(row) {
                 return headers.map(function(d) { console.log(row.properties)
                     return {column: d, value: row.properties[d]};
                 });
             })
             .enter()
             .append("td")
             .text(function (d) {
                 return d.value;
             });
      }
    };
  });

view.html

Table data

  <d3-table data="data"></d3-table>

Thank you for your answer.
However still it doesn't work. I used the above code only with a small change for templateURL path (scripts/widgets/tables/view.html as the {widgetsPath}/ is not recgnizable)
I also created the view.html as above and stored according to templateURL path.
In sample.js I added the 'adf.widget.table' (modules declaration)
Modified sample-04.js as follows
...
                 "columns": [
                    {
                      "styleClass": "col-md-12",
                      "widgets": [
                        {
                          "type": "table",
                          "title": "Reservation Totals"
                        }
...

Finally, I can see only the header (Table data) from header of view.html.
What am I doing wrong?
Once again, thank you in advance for your help.

(Sorry, I repeat my last comment as I closed the issue by mistake)
Thank you for your answer.
However still it doesn't work. I used the above code only with a small change for templateURL path (scripts/widgets/tables/view.html as the {widgetsPath}/ is not recgnizable)
I also created the view.html as above and stored according to templateURL path.
In sample.js I added the 'adf.widget.table' (modules declaration)
Modified sample-04.js as follows
...
                 "columns": [
                    {
                      "styleClass": "col-md-12",
                      "widgets": [
                        {
                          "type": "table",
                          "title": "Reservation Totals"
                        }
...

Finally, I can see only the header (Table data) from header of view.html.
What am I doing wrong?
Once again, thank you in advance for your help.

Do you have included the script file in the index.html? Could you post the output of you browser console?

Yes, I did.
Also find attached the screenshot.

It looks like the widget is not able to load your json file:
 Failed to load resource: the server responded with a status of 404 (Not Found)

Please check the network tab of the developer tools in your browser.

I checked and I removed pivot.js I load. Now the Console Output is
TypeError: Cannot read property '0' of undefined
    at link (http://localhost:9001/sample/scripts/widgets/3s_tables/table.js:45:54)
    at http://localhost:9001/sample/components/angular/angular.js:7035:44
    at nodeLinkFn (http://localhost:9001/sample/components/angular/angular.js:6633:13)
    at compositeLinkFn (http://localhost:9001/sample/components/angular/angular.js:6028:13)
    at compositeLinkFn (http://localhost:9001/sample/components/angular/angular.js:6031:13)
    at publicLinkFn (http://localhost:9001/sample/components/angular/angular.js:5923:30)
    at $q.all.then.msg (http://localhost:9001/src/scripts/widget-content.js:100:38)
    at deferred.promise.then.wrappedCallback (http://localhost:9001/sample/components/angular/angular.js:11446:81)
    at http://localhost:9001/sample/components/angular/angular.js:11532:26
    at Scope.$get.Scope.$eval (http://localhost:9001/sample/components/angular/angular.js:12556:28) <d3-table data="data" class="ng-isolate-scope">

In the Network tab I see that json file is loaded.
According to the above console output, It seems there is a problem in line 45:54 of table.js
I removed var headers' and var header` (the variables corresponding to these lines) and I get no errors in the console. However I still get no result, only the widget now appears bigger.

Finally, it works!
Thank you very much for your help!


SSL errors when using powershell 5.1

Current Behavior:
Using Powershell 5.1, Connect-Rubrik failed at the invoke web request even with Unblock-SelfSignedCert
PS D:\rubrik> Unblock-SelfSignedCert
PS D:\rubrik> $r = Invoke-WebRequest -Uri $uri -Method $method -Body (ConvertTo-Json -InputObject $body) -Headers $head
Invoke-WebRequest : The underlying connection was closed: An unexpected error occurred on a send.
At line:1 char:6
+ $r = Invoke-WebRequest -Uri $uri -Method $method -Body (ConvertTo-Jso ...
+      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-WebRequest], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeWebRequestCommand```

**Expected Behavior**:

By adding:
[Net.ServicePointManager]::SecurityProtocol = "tls12, tls11, tls"
The web-request worked.

**Steps to Reproduce**:

Please provide detailed steps for reproducing the issue.

1. Install Powershell 5.1
1. Install Rubrik Module
1. Try using Connect-Rubrik

**Context**:

* **Rubrik PowerShell Module Version**: <IP_ADDRESS>
* **PowerShell Version**: 5.1.14409.1018
* **Operating System**: Windows 7 (x64) - [Version 6.1.7601]

**Failure Logs**

Hi @arnoSCC - Thanks for raising the issue!
Are you able to update to the latest Rubrik module from the PS Gallery?  TLS 1.2 was added in v4.0.0 and should work for you with Connect-Rubrik and PS 5.1...
Let me know if this works for you....
Mike

Hi @mwpreston
I'll try, but since version 4.0.0 of the module is not the latest release here, i didn't used it in the first place.

Fixed in latest version.


Bump brain 0.1.18

Bump brain to v0.1.18
This will allow GET requests to the /eth/v1/keystores endpoint, retrieving the keystores from the web3signer (it actually proxies the request to the signer as the getValidators call to the brain does not follow the same format as the web3signer response)

Also added https://github.com/dappnode/StakingBrain/pull/309

Merge and testing it


CB-8250 invoke the CM setuid script with the migration flag during up…

…grade
As part of Upgrade we need to call the setuid script to configure deterministic
user and group ids. It can happen that the original setuid script is called
without the migration flag so I added an extra guard check to make sure it's
not called during upgrade. If the original installation called the script
alreeady we don't need to migrate anyways. Also in the Upgrade handler
we only updated the CM repo pillar and not the CM settings. However,
the CM settings contain an important upgrade to only call the setuid
script if the CM version is 7.2.1 or above. This was not updated previously
so upgrading from 7.2.0 to 7.2.1 didn't invoke the script.
This commit also fixes another issue when the incorrect Stack CRN was used
as ENV CRN and we didn't properly check for workloads.
See detailed description in the commit message.

I think the swagger check can be ignored. This PR is not touching anything API related.

the swagger failed because Execution failed for task ':autoscale:test'.

This has resolved this issue, too: https://jira.cloudera.com/browse/CB-8253


Custom test command

Thanks for this package. I can't find any option in the README to set a custom test command instead of the default (cy run), for example:
npm run e2e

is there a workaround? need to run e2e as well.

for now there is no workaround, but if someone would like to implement this - you would need to tweak the function runTests here https://github.com/cypress-io/github-action/blob/master/index.js#L300

@bahmutov thanks for responding. Maybe there's another way to hook compiling typescript spec tests?
I'm currently using @briebug/cypress-schematic:cypress to run it locally within the angular json.

I am not sure what hooking compiling typescript spec tests mean in this
issue.
On Wed, Apr 1, 2020 at 5:43 PM Diego Estela<EMAIL_ADDRESS>wrote:

@bahmutov https://github.com/bahmutov thanks for responding. Maybe
there's another way to hook compiling typescript spec tests?
I'm currently using @briebug/cypress-schematic:cypress
https://github.com/briebug/cypress-schematic to run it locally within
the angular json.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cypress-io/github-action/issues/118#issuecomment-607505060,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQ4BJW2WTKELGGDEJUFZK3RKOYR3ANCNFSM4LTGRWWQ
.

--
Dr. Gleb Bahmutov, PhD
Schedule video chat / phone call / meeting with me via
https://calendly.com/bahmutov
<EMAIL_ADDRESS>@bahmutov https://twitter.com/@bahmutov
https://glebbahmutov.com/ https://glebbahmutov.com/blog
https://github.com/bahmutov

Using the default cypress run command won't work with projects that use typescript, that is probably why some people have the need to override that command.

How are typescript projects avoid using “cypress run” Command?
Sent from my iPhone

On Apr 2, 2020, at 14:06, Diego Estela<EMAIL_ADDRESS>wrote:

Using the default cypress run command won't work with projects that use typescript, that is probably why some people have the need to override that command.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Who said anything about avoiding?. The default github action uses the cypress run which won't work on tests that are written in typescript (correct me if I'm wrong). Using schematics like the one I mentioned earlier makes it easier to bundle the commands (tsc, cypress run) into a single one.

Btw, there's an open PR with this feature
https://github.com/cypress-io/github-action/pull/120


Draft: fix(FileDetailsLabel): Kayla wants to change the background

Kayla and Lucia met and decided on a different approach to FileDetailsLabel - updating that here.

Preview: https://virtual-assistant-pr-virtual-assistant-150.surge.sh


Fixing Inflated Lower SR Difficulties

Adjusted the bell curve, reduced impact of term/ratio component, and scaled difficulty down more in order to fix maps brought up to fall under the issue such as the We Need To Calm Down Remix, Holy Macaroni, and Sakazuki for example. Please look over these changes.

Have checked both ranked Nhelv's for them to be nerfed heavily by this change. Will work on balancing before getting some update on this again. Don't accept in the meantime.


Proper equality check

The "=" operator is currently used with its default implementation. This however seems to lead to wrong results sometimes. Instead Z3_is_eq_ast should be used.

Fixed by #6.


Threading through callable to support password rotation

The following has been true since asyncpg v0.5.1 (in 2016):

password: str has been a parameter to connect(...),
Extra **kwargs from Pool(...) have been passed down to connect(...)
create_pool has passed extra **kwargs through to Pool.__init__

Around v0.21.0 (in 2020), password was altered to be a Callable as well, which permits dynamic password lookup on connection attempt. When combined with HashiCorp vault or AWS SSM, this would permit password rotation in environments where redeploying would be undesirable.
Since the password: str kwarg has been around for so long, even if someone is running a particularly old version of asyncpg, this additional parameter is unlikely to break anything, but if you'd like I can add some empty-dict indirection to not even pass the kwarg if it's None.
(For additional context, if necessary, sqlalchemy also supports this)

Do you know if this is also possible for sqlite?

I'm not sure what the right solution is for sqlite DBs - maybe that backend should raise a ValueError if a callable is passed to it?

Ping, at your earliest convenience. I've monkey-patched this functionality into our services for now, so no rush. I'm happy to address any concerns if there are any.

Sorry it has taken a while, I've been thinking about the best solution. I think I'd prefer f443aaf4fcea989284dc386335cdaa8133a28a58 which allows any customer options to be passed through. Whilst this isn't typed, nor as clear as the named arguments, it does support any driver's custom options. What do you think?


[Bug]: On clicking course - redirect to courses page

What bug you encountered?
Inside landing home page, in Nav bar, when we click on Course it should redirect to /courses. You can use useRouter from react to redirect form one to another page.
Show us the trouble with screenshots

Which browsers are affected?
Other
Agreement to Code of Conduct

[X] Code of Conduct

Please assign this to me under GSSOC'23

I would resolve this issue. Plz assign this task to me. I am well versed in frontend techs.

We are closing this issue, as it has been inactive for a long time. If you think you want to work on this issue please create a new ISSUE.


Generated fields not correctly treated as written by Doctrine

Having a generated field in an entity:
    #[Column(type: Types::DATETIME_MUTABLE, insertable: false, updatable: false,
        columnDefinition: 'datetime GENERATED ALWAYS AS (DATE_ADD(Start, INTERVAL Hours * 60 MINUTE))',
        generated: 'ALWAYS')]
    private \DateTimeInterface $end;

Obviously only with a plain getter:
    public function getEnd(): \DateTimeInterface
    {
        return $this->end;
    }

Makes phpstan not a happy camper:
  75     Property WorkEntry::$end is never written, only read.
         💡 See: https://phpstan.org/developing-extensions/always-read-written-properties

Solution is to properly parse the generated property of the attribute, and consider the property always-written when ALWAYS or INSERT, ref https://www.doctrine-project.org/projects/doctrine-orm/en/2.13/reference/annotations-reference.html#column
Feature has actually been in Doctrine ORM since 2.11: https://www.doctrine-project.org/2022/01/11/orm-2.11.html

Needs to be fixed here https://github.com/phpstan/phpstan-doctrine/blob/1.3.x/src/Rules/Doctrine/ORM/PropertiesExtension.php.


Correcting residue numbering problem by getting the index of the resn…

…o in the internal NGL structure.

Codecov Report

Merging #111 into master will decrease coverage by 0.02%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
- Coverage   78.84%   78.81%   -0.03%
==========================================
  Files          42       42
  Lines        1952     1954       +2
  Branches      340      305      -35
==========================================
+ Hits         1539     1540       +1
- Misses        413      414       +1

Impacted Files
Coverage Δ

src/data/ChellPDB.ts
31.57% <14.28%> (+0.32%)
:arrow_up:

Continue to review full report at Codecov.

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


tax update

Docs Build status updates of commit 181c981:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/index.md
:white_check_mark:Succeeded
View

dynamics-gp/installation/installing-on-first-computer.md
:white_check_mark:Succeeded
View

dynamics-gp/payroll/CAN18YETax.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Including a code block inside a heading shows the '#' and code block on separate lines.

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

Action Performed:

Log into an account and open a chat.
Type the following message in the composer and send.
#Some heading in a code block

Expected Result:
There are 3 possible scenarios:
Ignore the 3 backticks and display the text as is styled as a heading like what happens with quotes in a heading.
Show a code block styled as a heading like GitHub does (see attached screenshot)
Display the text all on the same line as regular text i.e. no heading styles applied.
Actual Result:
The '#' and code block are shown on separate lines.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/f5356ef5-d20c-47e6-a5f3-a5df1c93c7f9
https://github.com/Expensify/App/assets/43996225/f1d86582-6520-4945-8bd4-b21ac2d1dbbf
Expensify/Expensify Issue URL:
Issue reported by: @Victor-Nyagudi
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1684486639025769
View all open jobs on GitHub

I'm going to start with adding just the engineer first so it can be decided which option fits best -
Expected Result:
There are 3 possible scenarios:

https://github.com/Expensify/App/issues/14100#issuecomment-1404371634 like what happens with quotes in a heading.
Show a code block styled as a heading like GitHub does (see attached screenshot)
Display the text all on the same line as regular text i.e. no heading styles applied.

IMO this is not a bug and intebded, you can't combine those 2 formatting options.


Adding new classes to existing model trained with 2 classes.

I have built an image classifier with 2 classes, say 'A' and 'B'. I have also saved this model, using model.save().
Now, after a certain time, the requirement arose to add one more class 'C'. Is it possible to load_model() and then add only one class to the previously saved model so that we have the final model with 3 classes ('A','B' and 'C'), without having to retrain the whole model, for classes 'A and 'B' again?
Help me out, I'm stuck here @fchollet or anyone else.

Hi,
I hope that what i'm writing here is correct, after loading your model using load_model you can add a Dense layer that takes in as inputs the outputs of the last layer in your model and outputs the desired number of classes.
Here is an example of a model that outputs 2 classes and after loading the weights you take the output of the last Dense layer as input to a new Dense layer that outputs 3 classes:
from keras.models import Sequential, Model
from keras.layers import Dense
import numpy as np

model = Sequential()
model.add(Dense(32, input_shape=(1,)))
model.add(Dense(2, activation='softmax'))

model.load_weights("D:\\dataset\\fff.hd5")

model.compile(optimizer='rmsprop',
      loss='categorical_crossentropy',
      metrics=['accuracy'])

history = model.fit(np.random.randn(500), np.random.randint(0, 1, (500, 2)))

print(model.predict(np.asarray([4])))

##remove the last Dense layer of our model
model.pop()

base_model_layers = model.output
pred = Dense(3, activation='softmax')(base_model_layers)
model = Model(inputs=model.input, outputs=pred)
model.compile(optimizer='rmsprop',
      loss='categorical_crossentropy',
      metrics=['accuracy'])

model.fit(np.random.randn(500), np.random.randint(0, 2, (500, 3)))
print(model.predict(np.asarray([4])))

Hope it was helpfull
Best Regards

Hi @talhadar
I tried your way, there is no error in execution and the program runs fine, however, the validation is reducing drastically, instead, as per my understanding it should have increased, as it had already pre-trained weights for Classes A and B ( when i trained for it, i got the val_acc: 0.93.) and while training for three classes, i'm getting val_acc of 0.3667
Thanks
Abhijit

Hi @abhijitnathwani, @talhadar
What I think it is happening is that by removing the dense layer you lose all the pretrained weights of the output layer, then you add a new dense layer with an extra class but this layer has new weight, not fitted to your data and of course you would have a lower validation accuracy. l have the same problem and my idea is that instead of deleting the dense layer, just resize it while preserving the pretrained weights. I am not quite sure how to accomplish this though so any advice would be really appreciated

Hi @abhijitnathwani , I'm trying to do same thing as yours. Have you got any solution on this or sample code?

Hi @ivancruzbht
What you suggested is a great idea. Did you find a way to do that? If so, please let me know the steps to train a model in this way.
thanks!

Hi, @abhijitnathwani  @ivancruzbht  @dilipajm @saivineethkumar
how to do this?
Thank you!

Hi @rnirdhar @ivancruzbht @dilipajm
I am using the following codes on Keras 2.2.0, while model is a sequential model. The original weights at the last layer are copied back into the new layer. Hope this can help.
from keras.layers import Dense
import numpy as np

# save the original weights
weights_bak = model.layers[-1].get_weights()
nb_classes = model.layers[-1].output_shape[-1]
model.pop()
model.add(Dense(nb_classes + 1, activation='softmax'))
weights_new = model.layers[-1].get_weights()

# copy the original weights back
weights_new[0][:, :-1] = weights_bak[0]
weights_new[1][:-1] = weights_bak[1]

# use the average weight to init the new class.
weights_new[0][:, -1] = np.mean(weights_bak[0], axis=1)
weights_new[1][-1] = np.mean(weights_bak[1])

model.layers[-1].set_weights(weights_new)

@talhadar @abhijitnathwani @YuriWu Hi all,
I tried all the suggestions described here. My question is how exactly will it work if I make my Dense layer with 3 outputs? Will I still need the training data on which the first model with 2 classes was trained?. If not then, what to give as in terms of data for the other two classes on which the model is already trained, because this new model accepts 3 labels and and corresponding data. Please do comment if I am missing a point here. THANKS!
Rishabh Sahrawat

...
Will I still need the training data on which the first model with 2 classes was trained?
...
Rishabh Sahrawat

From my experience, you still need them. If you fine-tune the 3-output model by data with only 3rd class, the final model will perform poor on the first two classes (often called "catastrophic forgetting").
Using the code I've mentioned, I can learn the (n+1)th classes fewer epoches than from scratch, based on already trained n-class model.
Yuri

@YuriWu thank you for the quick reply. I see, but the problem is I have a huge dataset (700,000 classes) for my problem which can not be loaded onto RAM so using the older dataset for retraining is not possible.
I am basically performing text classification. I have a model trained & saved on 24,000 classes but I want to train it on complete dataset using transfer learning so like 24000, then 24000 and so on.. Unfortunately, I can not find any way with which I can achieve this. I dont' know how companies do this when they have enormous dataset. Please share your ideas if you have any for my problem. I will be very thankful.

@rishabhsahrawat
I don't know how companies do this either. But you can check my recent paper at ICML'19 and the code repo. I tried to solve the partial label space problem locally in multiparty style, and ensemble them together via communications among local models.
Please cite that paper if you find it useful :-)

Hi @rnirdhar @ivancruzbht @dilipajm
I am using the following codes on Keras 2.2.0. The original weights at the last layer are copied back into the new layer. Hope this can help. When model is a sequential model:
from keras.layers import Dense
import numpy as np

# save the original weights
weights_bak = model.layers[-1].get_weights()
nb_classes = model.layers[-1].output_shape[-1]
model.pop()
model.add(Dense(nb_classes + 1, activation='softmax'))
weights_new = model.layers[-1].get_weights()

# copy the original weights back
weights_new[0][:, :-1] = weights_bak[0]
weights_new[1][:-1] = weights_bak[1]

# use the average weight to init the new class.
weights_new[0][:, -1] = np.mean(weights_bak[0], axis=1)
weights_new[1][-1] = np.mean(weights_bak[1])

model.layers[-1].set_weights(weights_new)

When the model is defined by functional API:
from keras.models import Model
import numpy as np

# save the original weights
weights_bak = model.layers[-1].get_weights()
nb_classes = model.layers[-1].output_shape[-1]

model.layers.pop()
new_layer = Dense(nb_classes + 1, activation='softmax')
out = new_layer(model.layers[-1].output)
inp = model.input
model = Model(inp, out)
weights_new = model.layers[-1].get_weights()

# copy the original weights back
weights_new[0][:, :-1] = weights_bak[0]
weights_new[1][:-1] = weights_bak[1]

# use the average weight to init the new class.
weights_new[0][:, -1] = np.mean(weights_bak[0], axis=1)
weights_new[1][-1] = np.mean(weights_bak[1])

model.layers[-1].set_weights(weights_new)

Thanks for the reply. I have one question, I retrained my saved model for a new detected class based on what you provided. So, I could retrain the last layer of my vggface model by adding one new class, however, the model classifies the new class with a lower probability. In other words, the model doesn't learn the new class as good as previous ones. I appreciate your thoughts on this.

hi  @YuriWu thanks for your code for the retrain you use all your dataset or is possibile to use only the train and test of the added class?

Hi @rnirdhar
I was able to achieve this task using darknet framework but didn't find any method to do using keras.

How did you do that using Darknet? Could you share your steps in short, please?
Regards,
dextroza

@YuriWu thank you for the quick reply. I see, but the problem is I have a huge dataset (700,000 classes) for my problem which can not be loaded onto RAM so using the older dataset for retraining is not possible. I am basically performing text classification. I have a model trained & saved on 24,000 classes but I want to train it on complete dataset using transfer learning so like 24000, then 24000 and so on.. Unfortunately, I can not find any way with which I can achieve this. I dont' know how companies do this when they have enormous dataset. Please share your ideas if you have any for my problem. I will be very thankful.

Hi, Could you please tell me if you solved this problem?


mocha-typescript-watch does not support typescript 2.9

Looks like compilation messages changed in typescript 2.9 and watcher doesn't understand them now.
tscl.on("line", line => {
    if (line.indexOf("Compilation complete.") >= 0) {
        console.log(line);
        compilationComplete();
    }
    else if (line.indexOf("File change detected.") >= 0) {
        compilationStarted();
        console.log(line);
    }
    else if (line.indexOf(": error TS") >= 0) {
        console.log(line);
        foundErrors();
    }
});

Can you provide us with a failing example? For example a link to your repo?

@marer nevermind. I have prepared a fix for this.


[Question] Getting error

Hello! I'd like to use toggletasks.nvim which always needs a toggltasks.json with the same pattern to write tasks:
{
    "tasks": [
        {
            "name": "task name",
            "cmd": "the command which has to be executed"
        }
    ]
}

Now I have the following snippet file:
local ls = require "luasnip"
local s = ls.snippet
local sn = ls.snippet_node
local t = ls.text_node
local i = ls.insert_node
local f = ls.function_node
local c = ls.choice_node
local d = ls.dynamic_node
local r = ls.restore_node
local events = require("luasnip.util.events")
local ai = require("luasnip.nodes.absolute_indexer")

local add_task = function(prefix)
    return {
        t({
            prefix .. '{',
            prefix .. '\t"name": "',
        }),
        i(1),
        t({
            '",',
            prefix .. '\t"cmd": "',
        }),
        i(0),
        t({
            '"',
            prefix .. '}'
        })
    }
end

return {
    s("de", {
        t({
            '{',
            '\t"tasks": [',
        }),
        add_task("\t\t"),
        t({
            '\t]',
            '}'
        }),
    }),
    s("t", add_task("")),
}

My t snippet works fine, I'm getting the expected
{
    "name": "",
    "cmd": ""
}

snippet, but if I execute the de (which should mean default) snippet, I'm
getting the following error:
Error executing vim.schedule lua callback: .../pack/packer/start/LuaSnip/lua/luasnip/nodes/snippet.lua:824: attempt to call method 'indent' (a nil value)
stack traceback:
	.../pack/packer/start/LuaSnip/lua/luasnip/nodes/snippet.lua:824: in function 'indent'
	.../pack/packer/start/LuaSnip/lua/luasnip/nodes/snippet.lua:711: in function 'fake_expand'
	.../pack/packer/start/LuaSnip/lua/luasnip/nodes/snippet.lua:775: in function 'get_docstring'
	...e/pack/packer/start/cmp_luasnip/lua/cmp_luasnip/init.lua:35: in function 'get_documentation'
	...e/pack/packer/start/cmp_luasnip/lua/cmp_luasnip/init.lua:119: in function 'resolve'
	.../nvim/site/pack/packer/start/nvim-cmp/lua/cmp/source.lua:363: in function 'resolve'
	...e/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/entry.lua:484: in function 'resolve'
	...re/nvim/site/pack/packer/start/nvim-cmp/lua/cmp/view.lua:234: in function 'fn'
	.../site/pack/packer/start/nvim-cmp/lua/cmp/utils/async.lua:55: in function <.../site/pack/packer/start/nvim-cmp/lua/cmp/utils/async.lua:53>

do you have an idea what I'm doing wrong?

Lol, just checked my mail xD
Ah, the problem is that add_task just returns a table of nodes, which is not allowed at that point (s expects just a flat list/table of nodes).
I'd recommend wrapping add_task's nodes in a snippetNode before returning, downside is that the i(0) will be ignored (i(0) have to be children of the outer snippet itself).
Another solution: merge the three tables (nodes before add_task, nodes from add_task, nodes after add_task) into one list before passing it to s.

Lol, just checked my mail xD

https://www.youtube.com/watch?v=Oxqcoqf8Whs

Ah, the problem is that add_task just returns a table of nodes, which is not allowed at that point (s expects just a flat list/table of nodes).

Ah, ok. Thank you for the information!

Another solution: merge the three tables (nodes before add_task, nodes from add_task, nodes after add_task) into one list before passing it to s.

I think I'll go with this one. Thank you for your (very fast) help! c(^-^)c

https://www.youtube.com/watch?v=Oxqcoqf8Whs

Hah! :D

I think I'll go with this one. Thank you for your (very fast) help! c(^-^)c

You're welcome :D


Question about seemingly missing codes in paper. e.g. AF2

It's very exciting to see the work you did, but I didn't find some codes corresponding to part of the paper.
Specifically, they are:

codes w.r.t. AF2, 4.2.1 in the paper
Figure 7. Comparison of sequence recover, where you implement the distinguishing between core and surface part of the proteins
De novo proteins
antibody designs
I am not sure whether I missed something or not. Could you update these parts of the code or point out the corresponding code section if it's my carelessness?
Thanks a lot.
@zhengzx-nlp

Most of the detailed questions about the paper have been resolved in the email correspondence with
For anyone who may also be curious about annotation and extraction regarding folding regions (core, surface etc.) in Fig 7, here is the script for doing this
from Bio.PDB import PDBParser
from Bio.PDB.DSSP import DSSP
import os
from tqdm import tqdm
import multiprocessing as mp
import pandas as pd
print(mp.cpu_count())

### get structural lables using DSSP
path = 'your pdb path'
result_path = 'your result path'

def DSSP_get_structural_labels(dir,chain):
    protein_f = './pepbdb/'+dir+'/receptor.pdb'
    p = PDBParser()
    structure = p.get_structure(dir, protein_f)[0]
    dssp = DSSP(structure, protein_f)
    # print(dssp.keys())
    a_key = list(dssp.keys())
    all_seq=''
    all_ss=''
    all_sol=''
    for i in range(len(list(dssp.keys()))):
        a_key = list(dssp.keys())[i]
        seq = str(dssp[a_key][1])
        # secondary structure label
        ss = str(dssp[a_key][2])
        # folding core and exposed region label, cutoff is set to 0.1, folding core <=0.1 exposed region >0.1
        sol = str(dssp[a_key][3])
        # print(seq,ss,sol)
        if a_key[0] == chain:
            all_seq += seq
            all_ss += ss
            all_sol += sol+','
    result = ''
    result = (dir +'\t'+ chain +'\t'+ all_seq +'\t'+ all_ss + '\t' + all_sol +'\n')
    return result


Support uploading Android app bundles

Android app bundles were announced today at Google IO.
One of the sessions mentioned that the publisher API will have support for this.
It would be cool if an option was added to upload_to_play_store to support this.

This is a feature request ☝

@eygraber Thanks for the feature request! I started to implement .aab support today but looks like we are held back by https://github.com/google/google-api-ruby-client until .aab support is added there 🙃 We will get this updated as soon as we can 💪

Is there an issue on their repo to watch?

@vanniktech Its up now and should have this out today 💪 I've been out sick so haven't been able to get to it as quickly as I would have liked 😝


ensure region is always set for aws api integration

:nut_and_bolt: Description: What code changed, and why?
this issue was reported in slack
a user found that all controls for the aws profile had results like "region value is not set/missing". they also noticed the env value was being set to gcp-api
this change fixes the env region to be the correct "aws-api" and ensures that the region gets set when calling the awscreds function
:+1: Definition of Done
aws api scans working
:athletic_shoe: How to Build and Test the Change
add a node integration for aws (https://github.com/chef/automate/blob/master/components/compliance-service/scripts/create-pg-data.sh#L144)
run a scan job on it, see the error about missing region
rebuild compliance and nodemanager
run scan again, no region error, just normal results
:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[x] Tests added/updated?
[n/a] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable

i'm not sure about that last commit where i added the test. we've had trouble with those tests before...

TestAWSAPIScanJob: scanjob_aws_api_integration_test.go:148: aws api node found
--
  | TestAWSAPIScanJob: scanjob_aws_api_integration_test.go:149: Beginning test time: 2020-03-24 20:31:53.442815761 +0000 UTC m=+51.887636501
  | TestAWSAPIScanJob: scanjob_aws_api_integration_test.go:150: Found node id:"3635d5af-27ca-45d2-96b7-092344b1d761" name:"aws-account-chef-cd" platform:<name:"aws" release:"train-aws: v0.1.13, aws-sdk-core: v3.85.1" full:"aws train-aws: v0.1.13, aws-sdk-core: v3.85.1" > environment:"aws-api" latest_report:<id:"45f5f783-c385-4926-92ea-aac0a65f1209" end_time:<seconds:1585081921 > status:"failed" controls:<total:4 passed:<total:1 > skipped:<> failed:<total:3 critical:3 > waived:<> > > profiles:<name:"my-aws-profile" version:"0.1.0" id:"7de148c67ef53752695560485a9757e8f88e128e5d8509a26a03c4bd9f6f70b1" status:"failed" full:"InSpec Profile, v0.1.0" > , end time: 2020-03-24 20:32:01 +0000 UTC

https://a2-local-inplace-upgrade-acceptance.cd.chef.co/compliance/reports/nodes?job_id=0c293b90-1f7e-4f9b-ba23-cf054dd2e8c4&end_time=2020-03-24
working api scan ^

but i'm finding that node integrations set up with env creds are still not getting region, and i don't know why, looking into it


Add api documentation as draft version

Description
Add API document as draft version and we will gradually add API contents.
Related Issue : #1814
How Has This Been Tested?

Check url : "http(s)://domain/docs/api-guide.html"

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

run build
deploy to 1


Please sit around a table and talk

https://github.com/AlpacaTravel/react-map-gl-alt
https://github.com/uber/react-map-gl
https://github.com/alex3165/react-mapbox-gl

Both MapboxGL.js and React are 4 years old. There are currently 3 different, active and maintained, libraries out there trying to implement a React friendly API for Mapbox. While each of them is pretty great and hard work, each of them lack features. When coming from a raw MapboxGL.js background, new to react, this is pretty critical and hard to make a reliable choice. I guess coming from a React background and discovering MapboxGL and making the same choice is also hard.
This seems pretty much complicated, as governance, paradygms, workflows and many stuff are different in each of the repos. But it also feels like there is so much relevant and valuable thing to do with all that hard work, than 3 different and incompatible libraries..
The beauty of opensourcing is that everything is possible, isn't it ? So hey, what about sitting around a table and talking ?
Related :

https://github.com/uber/react-map-gl/issues/87
https://github.com/uber/react-map-gl/issues/176
https://github.com/alex3165/react-mapbox-gl/issues/97

Same issue duplicated :

https://github.com/AlpacaTravel/react-map-gl-alt/issues/16
https://github.com/uber/react-map-gl/issues/451

Personally I am glad they are different and it feels like react-mapbox-gl has matured with mapbox and my own app and does everything I wanted it to do, something uber/react-map-gl isn't and doesn't want to from my impression. It's ok to do things differently too, as long as from the github intro page it's clear what it does and doesn't do or want to do.

Closed in favor of more followed discussion inside official mapbox-gl.js repo


Copy output

Copy output

To copy the results of the build to Azure Pipelines or TFS, perform these steps:
Click the Copy Files task. Specify the following arguments:

There is no such task.

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

ID: 62d36368-d41a-a377-2db2-556425c30173
Version Independent ID: 600e6641-27f8-74dd-2f34-c79dc9813887
Content: Build C++ Windows apps - Azure Pipelines
Content Source: docs/pipelines/apps/windows/cpp.md
Product: devops
Technology: devops-cicd-apps
GitHub Login: @juliakm
Microsoft Alias: jukullam

Adding a link to the task.


treewide/nixos: remove with lib; part 8

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@philiptaron i rebased:

nixos/modules/services/networking/nat-iptables.nix
nixos/modules/services/networking/nat-nftables.nix

please look at them carefully. Last time i did something wrong while rebasing.

Based on my experience merging the other PRs, Felix (@Stunkymonkey), I'm going to sit it out for a while. I don't have enough confidence there are zero defects here, and I know with the new script, I'd need to actually run tests against all the modules here -- which I don't have the computing power to do, and I don't think you do either.


i18n(zh-cn): update content.mdx

Description (required)
i18n(zh-cn): update content.mdx
Related issues & labels (optional)

https://github.com/withastro/docs/pull/8640

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
community-resources/content.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

!coauthor


Improve cli tests

It's take too much time to update cli tests through amp versions.
It's mainly due to the fact we don't get enough information when an issue occurs and we have for each to know which regex was used in the issue, understand it, compare with the real output and got how to fix, relaunch all tests to test the fix, and iterate this way until the good result.
Some regex are not so simple and not completely right. Debug them take time, relaunch tests take time.
We take more time fixing/adapting the test itself than the code which should be tested.
To go faster, it should be good to :

display which test is failing
in case of error, display the executed command, the regex and the output which doesn't match.
if possible display exactly where the problem is in the output (for that, perhaps, use plain texts with variables without regex to describe the expected output and regex only internally to compare the output logical bloc by logical block using the plain text description. This intermediate description layer could allow to know on which logical block the issue occurred)
be able to launch only one test or several specific tests to debug them quicker

Addressed by #1159

I think you meant to reference this PR: https://github.com/appcelerator/amp/pull/1160

yes thanks!


[Bug]: Typo in Docker-Compose File causes UI Bug

Description
I made a typo in my Docker Compose file, and now I can't edit the file anymore. Every 4 seconds, Coolify tries to validate the file and calls https://xxxdev/livewire/update, which in turn renders a large overlay that prevents me from editing the file.

You need to click the outside of the overlay and be very fast and edit your compose file

Minimal Reproduction (if possible, example repository)

Create a new Resource - Based on a Docker-Compose File
Edit Compose file and make it invalid (I <3 yml )
pop-up should appear every 4 secs

Exception or Error
No response
Version
236

This will be fixed in the upcoming version.


[vscode] 7/n listeners and .env

[vscode] 7/n listeners and .env
Summary:
When the .env file path is set, vscode will invoke api/set_env on each AIConfig server.
Defines a listener to a newly introduced setting for the env_file_path.
This ensures that users do not need to restart vscode to pickup env changes.
Test Plan:

set env file path
changes picked up
changes picked up on restart
Changes picked up when manually removing vscode setting and re-setting env file path and api key

https://github.com/lastmile-ai/aiconfig/assets/141073967/07ac1d43-28ef-439f-98e7-94a4e73e11c9

Stack created with Sapling. Best reviewed with ReviewStack.

#1398
-> #1390
#1387
#1389

added an await statement when invoking updateServerEnv
modified event handler to execute only if the extension setting for env_file_path is modified


Add reference to youtube playlist in index

Partially addresses #593.
I also took the liberty of changing note -> notice as I felt it was more suitable, but I don't have strong opinions on that.

Great! Merging. Thank you!


Fix asyncio embedding issue

As mentioned in: https://github.com/dan-fritchman/Hdl21/issues/113?notification_referrer_id=NT_kwDOAEf5zbI2MjEwOTk5MDc5OjQ3MTcwMDU#issuecomment-1528319399
https://github.com/Vlsir/Vlsir/blob/e7f5f91a0a4e14469cd33c6a5c3b9b1b321a6058/VlsirTools/vlsirtools/spice/spice.py#L85 should be updated to detect existing loop.
Also can someone add me as a collaborator for easier PR?

You're in

Hm... it seems I still can't push directly in this repo. I am in Hdl21 though.

Hm indeed.
I think you're in (again).

OK this seems bad:
https://github.com/erdewit/nest_asyncio/issues/57
Especially in the context of calling simulations, which fail and generate exceptions all the time.

So it seems given this limitation, one can have either of

(a) Users must know whether they are in an async or synchronous context, and call sim or sim_async accordingly, or
(b) They can get exceptions

Given the two options, I think I'd pretty easily choose (b).
It does bring to mind, what is all this async stuff for. The only actual parallelism we got out of it, is while waiting for external subprocesses to complete. All the python-code runs in a single thread, blocking-ly. There's likely a subprocess-equivalent of asyncio.gather, or a custom version to be written here, that gets rid of this async-stuff. Now async is nice because we don't need to write the state-machine that polls each subprocess before, for example, reading result-files. But we could.

BTW as far as the Jupyter use-case goes - it appears that nowadays you can await directly in the cells:

Granted this would still fail if, for example, you imported a python-module which invokes the synchronous sim function and asyncio.run.

I think asyncio is mainly designed for use cases where the users expects async behavior. e.g., dealing with networks. A library using asyncio should not have an asyncio.run, since it should be the responsibility of the user to call asyncio.run on the top of the user's program.

asyncio should "contagious" sort to say. If a lib uses async functions, other libs that build on top of it should use asyncio too. That's one big con of cooperative multitasking, that everyone has to cooperate :p

Given my experience of using asyncio in various projects, seeing people using it, and reading relevant docs online, people are generally clueless so it's hard to build a library using asyncio and hope the user "cooperate".

Users must know whether they are in an async or synchronous context, and call sim or sim_async accordingly

Always exposing async def interface and ask for explicit usage might be a good idea since it's like say "use only if you know what you are doing".

I think asyncio is mainly designed for use cases where the users expects async behavior. e.g., dealing with networks. A library using asyncio should not have an asyncio.run, since it should be the responsibility of the user to call asyncio.run on the top of the user's program.

asyncio should "contagious" sort to say. If a lib uses async functions, other libs that build on top of it should use asyncio too. That's one big con of cooperative multitasking, that everyone has to cooperate :p

Given my experience of using asyncio in various projects, seeing people using it, and reading relevant docs online, people are generally clueless so it's hard to build a library using asyncio and hope the user "cooperate".

Seems right on all counts.
Async, at least as it stands today, has gotta be an "application level" decision. And this is a library intended for many diverse applications, many of which will be quite clueless about async.
We should probably design it out.

Agreed that it should be removed. Fortunately we don't use it much so changes are quite local.


Native Windows icon set in the Project settings is ignored

Godot version:
3.3 RC6
OS/device including version:
Debian GNU/Linux 10
Issue description:
When exporting the project into a Windows executable, it will not contain the ICO icon specified in the Project settings, only taking into account the path given in the Export dialog itself. The ICNS icon however, seems to work fine either way.
Minimal reproduction project:
windows_icon_bug.zip

Did you configure rcedit in editor settings?

@akien-mga Yes, otherwise neither setting the icon path in the Export dialog would've worked.

This was fixed by #77821.


.github/workflows/editorconfig.yml: switch to pull_request_target

The (re)approvals needed for this action are somewhat inconvenient for the amount of traffic we get in this repo.
If we want to add any further actions I don't think using pull_request_target for all of them is a good idea with the potential issues it has but I think just for this action it's fine (and hopefully only as short term fix until we find a better solution).

I was thinking to get rid of editorconfig as it provides no benefit while requiring work from the contributors.
Spending time on whitespace could have been spent on something else.

I was thinking to get rid of editorconfig as it provides no benefit while requiring work from the contributors.

Spending time on whitespace could have been spent on something else.

Well almost all of the work is already done so I don't think we should throw it away.
I don't think it take much effort from contributors to not introduce tabs and whitespace.

editorconfig-checker -disable-indentation on master reports 89 errors, mostly in perl/python/shell scripts that probably haven't been touched by anyone in ages.

I was thinking to get rid of editorconfig as it provides no benefit while requiring work from the contributors.

Spending time on whitespace could have been spent on something else.

Well almost all of the work is already done so I don't think we should throw it away.
I don't think it takes much effort from contributors to not introduce tabs and whitespace.

I think this lacks empathy to contributors, as what matters is how they feel about it.

I think this lacks empathy to contributors

Editors support .editorconfig via plugins and some will strip whitespace/newlines/etc unless disabled. I don't see the need for much empathy really, it's trivially handled once and forgotten about.
Can we merge this PR to deal with the immediate problem and do the editorconfig good/bad discussion elsewhere?

Sure, I opened https://github.com/NixOS/nixpkgs/issues/121027

@zowoq you didn't address the comment from @grahamc

I added this:
https://github.com/NixOS/nixpkgs/blob/5ba2e4e9c65d32ea3d7def475655227a3835d2dc/.github/workflows/editorconfig.yml#L3

Or did I misunderstand what was asked?

Missed that, sorry!


Fix issue #803 and add test cases

This was introduced by replacing the header parser with htslib lib standard implementation.
The text header was mistakenly truncated.
Closes #803

Thanks Hao!

Hello, this fix doesn't work with the samtools 1.10 package in Debian's current development version:
$ echo "@HD     VN:1.5  SO:coordinate" | samtools view -b
[main_samview] fail to read the header from "-".


Failed to run Python SDK due to No module named 'opt_paillier_c2py'

I followed the instructions from this document. But I still got the following error message:
vscode ➜ /workspaces/primihub/python (develop ✗) $ python primihub/tests/test_disxgb_en.py
Traceback (most recent call last):
  File "/workspaces/primihub/python/primihub/tests/test_disxgb_en.py", line 25, in <module>
    from primihub.examples.disxgb_en import xgb_host_logic, xgb_guest_logic
  File "/workspaces/primihub/python/primihub/examples/disxgb_en.py", line 20, in <module>
    from primihub.primitive.opt_paillier_c2py_warpper import *
  File "/workspaces/primihub/python/primihub/primitive/opt_paillier_c2py_warpper.py", line 1, in <module>
    import opt_paillier_c2py
ModuleNotFoundError: No module named 'opt_paillier_c2py'

Sorry, the documentation you see has timed out.
If you want to execute this test case, complete the following steps:

Compile opt_paillier_c2py, see: Developer-docs
Install Primihub package: see: Python SDK Insell
Run: python primihub/tests/test_disxgb_en.py

We will update the documentation as soon as possible, thk you.


Removal of Consistency Loss

I realised that since the update to include DepthAnything, u removed the consistency loss in the code. Are there any particular reason why ?

This loss is used to ensure the consistency among patches. When we use m1 mode, it would be necessary to add this loss to get a satisfactory result (without fluctions among patches as shown in Fig.3. Note that Fig.3 is from the Fine branch. The result from the Fusion model is much better but still faces a similar problem).
However, compared with the consistency-aware inference strategy, the effectiveness of consistency loss is not so impressive. As shown in Tab.2, we can simply adopt a bit more patches (m1 -> m2 mode) and gain huge improvement in consistecy error.
Another reason is that we need to tune the loss weight carefully for both improvement of standard and consistency metrics. I don't have enough time to do this now.
For reproducing, people can git checkout 2d87adc9 to get more details about consistency loss. It would be good if the community would like to adopt that to the current codebase and search for parameters.


The link of model weights failure?

Thanks for your wonderful jobs~ One question, your link to model weights failure, maybe some mistakes happened?

Can you access OneDrive with provided links?

Yes, I can't access OneDrive links, even I can access google....

You can download the models from Baidu Drive now:

Model
Link
password

HR18-COFW.pth
https://pan.baidu.com/s/1pV_P-fKY_Ec6pLG8KBt4Iw
prpr

HR18-300W.pth
https://pan.baidu.com/s/1Mt-9ZIOrwYb0qN-322kQgg
thwk

HR18-AFLW.pth
https://pan.baidu.com/s/18Cwed7hlw8CLeArlSivOeg
9rtd

HR18-WFLW.pth
https://pan.baidu.com/s/1rw7792j6yHoyve55GRx_uA
nvg3

Wow, thx for your sharing~

and processed annotations files data.zip

Link
Password

https://pan.baidu.com/s/1UIjq2-uQNLriglhRCGVdNw
96j2

You can download the models from Baidu Drive now:
Model	Link	Password
HR18-COFW.pth	https://pan.baidu.com/s/1pV_P-fKY_Ec6pLG8KBt4Iw	prpr
HR18-300W.pth	https://pan.baidu.com/s/1S0kKa8bZikr4cocApPxItg	2mgf
HR18-AFLW.pth	https://pan.baidu.com/s/18Cwed7hlw8CLeArlSivOeg	9rtd
HR18-WFLW.pth	https://pan.baidu.com/s/1rw7792j6yHoyve55GRx_uA	nvg3

Hi, the link of COFW model seems to be 300W model, could you provide the correct link?


Add pkgconfig file

Working with lv-drivers headers is a bit problematic. This is because the headers have not the same path when they are build compared to when they are used from another program. This will make it possible to ask pkg-config --lib to get required libs and pkg-config --cflags to get required cflags.

Thank you! As time passes I more and more convinced that having the drivers in the lvgl repo from v9 is a good idea.  It will solve a lot of versioning and synchronization issues.


Thunks for optimizing rendering

From @staltz on June 24, 2015 16:27
From @staltz on April 8, 2015 20:34
This should directly affect performance in the 'many' example.
Copied from original issue: cyclejs/cycle-core#101
Copied from original issue: cyclejs/cycle-dom#9

From @pH200 on May 20, 2015 6:2
How do you implement thunks when Cycle doesn't have states? applyToDOM takes Observable of vtree which is basically the result of rendering. And custom elements are already handling their own subtree redraws.

From @TylorS on February 24, 2016 22:41
Not sure if this is still relevant?

Stale, old.


Add user/patch test and proper code to update name

There was no test for user patching and with the change
in the user API this was broken.

Coverage remained the same at 100.0% when pulling 5f4106b6fbf893c3dbc64f152536f9fc9a92a542 on feature/user-patch into fffff4fbd0cadecc91b6a2c40ea8de0a4a2c8ed4 on master.

+1 looks good


fix: components relationship update failed

Problem:

Updating resource components relationships can lead to the violation of foreign key constraints. This issue arises during the update of resource components (see https://github.com/seal-io/walrus/blob/main/pkg/apis/resourcerevision/extension.go#L200). In the update process, old resources get deleted. However, if a relationship was created using a resource prior to its deletion, it can result in a query failure.
Solution:

Clear removed resource components before create components relationships.
Related Issue:
#2049

Can we try to get a clean component slice from outside?


Fixed: Unable to deploy and test the kubernetes documentation site locally in windows

ISSUE DETAILS
Windows commands are not mentioned properly in the README.md file to deploy and test the site locally. Even i found out that previously many users also have issue with windows and many contributors are continuously asking the maintainers the same question that how to deploy the site and test in windows locally.
refer #43401
I have changed the approach that i used in the above pr.
WHAT I HAVE CHANGED?
All the commands should be written in command prompt while deploying the site locally.
Added the following commands:
rmdir /s /q i18n  ->  This command will first remove the i18n directory
for /f %A in ('cd') do mklink /J "%A\i18n" "%A\data\i18n"   -> This command will make symbolic link in windows.

Hey @aj11anuj  , i have updated the commands so that it will work on powershell.


errors on augments

The online yangvalidator throws errors on augments.  Taken separately the augment statement validates, but when both augment statements are included in the same file, the validator chokes on them.
I have created four test yang files.  Two of them validate in the online validator and two do not.
The test-ok.yang and test-ok-2.yang files work.
The test-ok file has only one augment (for /nw:networks/nw:network/nw:network-types/tet:te-topology)
The test-ok-2.yang file has only one augment (for /nw:networks/nw:network/nw:node/nt:termination-point/tet:te)
Taken separately they augments work.
The test-1.yang file has both augments  Taken together, the online yanglint spits out:
libyang err : Augment target node "/nw:networks/nw:network/nw:network-types/tet:te-topology" from module "test-1" was not found. (/test-1:{augment='/nw:networks/nw:network/nw:network-types/tet:te-topology'})
libyang err : Augment target node "/nw:networks/nw:network/nw:node/nt:termination-point/tet:te" from module "test-1" was not found. (/test-1:{augment='/nw:networks/nw:network/nw:node/nt:termination-point/tet:te'})
YANGLINT[E]: Processing schema module from /var/yang/tmp/yangvalidator/yangvalidator-v2-workdir-BYiAZxUm/test-1.yang failed.

When I swap the order of the augments, I get different errors – which is strange. (test-2.yang)
libyang err : Augment target node "/nw:networks/nw:network/nw:node/te/te-node-attributes/connectivity-matrices/connectivity-matrix" from module "ietf-te-topology" was not found. (/ietf-te-topology:{augment='/nw:networks/nw:network/nw:node/te/te-node-attributes/connectivity-matrices/connectivity-matrix'})
libyang err : Augment target node "/nw:networks/nw:network/nw:node/te/tunnel-termination-point/local-link-connectivities" from module "ietf-te-topology" was not found. (/ietf-te-topology:{augment='/nw:networks/nw:network/nw:node/te/tunnel-termination-point/local-link-connectivities'})
libyang err : Augment target node "/nw:networks/nw:network/nw:node" from module "ietf-te-topology" was not found. (/ietf-te-topology:{augment='/nw:networks/nw:network/nw:node'})
libyang err : Augment target node "/nw:networks/nw:network/nw:node/te/information-source-entry/connectivity-matrices/connectivity-matrix" from module "ietf-te-topology" was not found. (/ietf-te-topology:{augment='/nw:networks/nw:network/nw:node/te/information-source-entry/connectivity-matrices/connectivity-matrix'})
YANGLINT[E]: Processing schema module from /var/yang/tmp/yangvalidator/yangvalidator-v2-workdir-fXryozgz/test-2.yang failed.

I have also attached the the full module that I'm trying to validate<EMAIL_ADDRESS>BTW... I have yanglint v1 and yanglint v2 running locally in linux and all the yang files I have shared validate (and I have instance data that works too)
augtest.zip
.
thanks

Hello,
first of all, on what version of libyang did you try this and which command did you use?
I tried to validate all the modules you shared with us using console and got the same results as online tool is displaying. I also tried it with different revisions of imported yang modules and also different versions of yanglint (2.0.88, 2.0.7, 2.0.0).
I used the following command:
yanglint -i -p /var/yang/tmp/yangvalidator/yangvalidator-v2-workdir-cIMiaTkm<EMAIL_ADDRESS>
Very interesting/strange thing is also that the errors are different whe you swap the order of the augments as you mentioned.
However, I asume that the error may be in the libyang tool itself.
Regards,
Slavomir Mazur

I use yanglint 2.0.88 on ubuntu and do not see the error, I also have tried
1.10.7.  So, why the online tool doesn't work is a mystery.
I even tried running the online tool uploading all the yang files and got
the same result.
Here is a tar file of all the yang files needed and an expect script and an
instance file.  If there isn't a tar file attached, let me know and I can
try sending a zip or gz file.
When I run this I get the output found in the mw.out file in the tar file.
The fact that the online tool works when the augments are isolated and not
when they are together is odd as well.
If it would help to have a short online meeting for me to demonstrate,
please let me know.
regards,
-scott.
On Wed, Sep 29, 2021 at 8:43 AM SlavomirMazurPantheon <
@.***> wrote:

Hello,
first of all, on what version of libyang did you try this and which
command did you use?
I tried to validate all the modules you shared with us using console and
got the same results as online tool is displaying. I also tried it with
different revisions of imported yang modules and also different versions of
yanglint (2.0.88, 2.0.7, 2.0.0).
I used the following command:
yanglint -i -p /var/yang/tmp/yangvalidator/yangvalidator-v2-workdir-cIMiaTkm @.***
Very interesting/strange thing is also that the errors are different whe
you swap the order of the augments as you mentioned.
However, I asume that the error may be in the libyang tool itself.
Regards,
Slavomir Mazur
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/YangCatalog/bottle-yang-extractor-validator/issues/74#issuecomment-930139318,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB7PR4JM33FBG4U7CAFSMPDUEMCWBANCNFSM5E2ZJ3OA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Since the tar file didn't come through here is a zip file
mw-test.zip

Hello,
I tried to replicate your steps and I got the same result as you. However, there is no way to force the yangvalidator to load files in a particular order currently and we wont going to implement something like that. We still think this should be handled by yanglint itself. Yanglint should also keep eye on the order of files.
I am moving this issue to libyang

yanglint works if the files are ordered, so I will make sure all our
developers/testers know how to do that.  If a "yanglint *.yang" is done on
all the files, I get a validation error that is different from what the
online yangvalidator reports.  But at least we have a way forward.
Since I haven't found a way for the online yangvalidator to validate, I
will tell our developer/testers not to use the online yangvalidator.
If a small example would be helpful, I can create one.
regards,
-scott.
-scott.
On Tue, Oct 5, 2021 at 4:19 AM SlavomirMazurPantheon <
@.***> wrote:

Hello,
I tried to replicate your steps and I got the same result as you. However,
there is no way to force the yangvalidator to load files in a particular
order currently and we wont going to implement something like that. We
still think this should be handled by yanglint itself. Yanglint should also
keep eye on the order of files.
I am moving this issue to libyang https://github.com/CESNET/libyang
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/YangCatalog/bottle-yang-extractor-validator/issues/74#issuecomment-934177594,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB7PR4MWXHDN7LO7A6WO7TTUFKYITANCNFSM5E2ZJ3OA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


FR: Support watchOS platform in xcframework, Carthage distribution

Issue
Downloading frameworks from Firebase distribution (https://firebase.google.com/download/ios) is not included watchOS platform support.
watchOS support is included in SPM and Cocoapods.
Proposal
Add support watchOS in the xcframework.

Thanks for the feature request. Our ZipBuilder tool needs to be updated to generate a watchOS and watchOS simulator slice.

Is there any work being done on this front? I tried to generate them myself, but so far no luck. Currently we are using it through CocoaPods, which is working. But we are trying to migrate everything to SPM / XCFrameworks

@asendra Your question is unclear. This issue is about binary distribution, and your question seems to be about SPM which supports watchOS for all the Firebase products that support it. Please open another issue to describe what isn't working.

I was talking about binary distribution, should not have mentioned SPM, sorry.


Cache - create new tracker for pipe_content stage q2

The tracker should print the pipe q2 stage every time it's valid.
This is the signal:  cache_pipe_lu_q2
This is the pipe struct:  t_pipe_bus

Example
-----------------------------------------------------------------------------------  .........
 Time  ||  OPCODE        || set_way_mru||  mb_hit_cancel|| tag  || Set ||    Data      .........
-----------------------------------------------------------------------------------   .........
   870     RD_LU                  {1,0,0,1}             1                     16     1c      d8e7c65c     .........

You can use XMR - "cross module reference"
From the TB you can access any signal hierarchy using full path:
Example:
Cache_tb.cache.pipe_wrap.pipe.signal_q2
I'm sure you can find examples in the existing code. I'll try and send something


References to VMMS should be VMSS

There are three references to "VMMS" on this topic page. These are references to Virtual Machine Scale Sets, so they should be changed to VMSS.

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

ID: 00343476-cbb4-2f94-a164-e094fece7a6d
Version Independent ID: 8cbbab0d-a0f2-fe64-bb5a-1e280c5332b2
Content: Configure Log Analytics workspace for Azure Monitor for VMs - Azure Monitor
Content Source: articles/azure-monitor/insights/vminsights-configure-workspace.md
Service: azure-monitor
Sub-service: ****
GitHub Login: @bwren
Microsoft Alias: bwren

@robsanf Thanks for pointing this out. The document is updated and changes will go live soon.


Merge newer patches

Thank you for your effords.
Is there any chance that you have a yet unpushed branch that contains newer patches from the official repo?
Have a wonderful day and thank you in advance for the response

@alexanderadam unfortunately, the company Undev was closed in 2015 😢


Fixed #88 -- Enhanced ESLint Configuration and improved Code Quality

Closes #88
Summary
Enhance ESLint rules to align with coding standards and improve code quality.
Changes

[x] Add ESLint rules: Implement coding standards for ESLint.
[x] Configure Tailwind CSS: Include lint rules specific to Tailwind CSS and update .eslintrc.json.
[x] Implement import sorting: Add rules for sorting imports and using type prefixes for importing types.
[ ] Enforce variable naming: Apply camelCase naming conventions for variables (currently unresolved).
[x] Add plugins: Integrate relevant ESLint plugins.
[x] Fix ESLint errors: Address errors across the frontend codebase (commit IDs: 68a0066, 5168fb4, 0a6f8ce).

Type of change

[ ] Bug fix
[X] Feature update
[ ] Breaking change
[ ] Documentation update

Flags

CamelCase Naming: Unresolved camelCase naming conventions across the project; may cause potential issues.
Consider implementing ESLint warnings as a pre-commit hook to enforce code quality.

Demo

No visual changes; focuses on code quality improvements.

How has this been tested?

Ran ESLint with new rules to ensure no new warnings or errors.

Author Checklist

[ ] Code has been commented, particularly in hard-to-understand areas
[ ] Changes generate no new warnings
[ ] Vital changes have been captured in unit and/or integration tests
[ ] New and existing unit tests pass locally with my changes
[ ] Documentation has been extended, if necessary
[x] Merging to main from fork:branchname

Additional context

Addressing camelCase naming conventions may impact existing code.
Consider implementing ESLint warnings as a pre-commit hook to enforce code quality.

This is still giving conflict warnings. I need to check it again.

Is this MR ready for review ? @SureshPradhana

The rule requires camelCase for general naming, but React components use PascalCase, which creates a conflict. For components, PascalCase should be used, while camelCase applies to other names. Additionally, there are some ESLint warnings that need to be fixed before this is ready for review. A custom ESLint plugin is also needed to restrict imports from @/* with a / as there no default's.

@Saurabhsharma884 updated the ESLint rules to avoid using @/*, and the rules for camelCase and PascalCase are conflicting due to React component naming conventions. The rules are either camelCase or PascalCase for variables and functions

@SureshPradhana Can you please rebase/resolve conflicts in this branch ?

@Saurabhsharma884 updated

@Saurabhsharma884 updated

Sorry, missed this. Can you please update again?

@Saurabhsharma884  This might take some time to complete, but I'm working on it.

@Saurabhsharma884 This might take some time to complete, but I'm working on it.

Sure, thanks for that but there has been a major upgrade to eslint v9 which supports flat config only but there has been multiple issues floating on github regarding its compatibility with plugins so we should wait for sometime and let those issues get resolved. we can upgrade to eslint v9 after that.

Changes are picked https://github.com/djangoindia/djangoindia.org/pull/234. Hence, closing this MR. Thanks @SureshPradhana


Fix Rayleigh quotient iteration

Hello, I'm a student interested in working on this package's GSoC project.
I've modified Rayleigh quotient iteration method to be the same as the already implemented Inverse method, except at every iteration it approximates the eigenvalue too with the Rayleigh quotient.
The test is a copy of the Inverse's test as well.

Coverage increased (+1.3%) to 87.706% when pulling 506fad56c2df829d0723c65c6da660e60c893fb8 on lopezm94:master into 4558f41f71fa03fa0a331f58b6aa0e4658fc232d on JuliaLang:master.

I'm bumping this @jiahao @andreasnoack

Coverage increased (+1.4%) to 87.763% when pulling e254bc5460f7f69ee8cb22c4f049971a9a474c37 on lopezm94:master into 4558f41f71fa03fa0a331f58b6aa0e4658fc232d on JuliaLang:master.

[Current coverage][cc-pull] is 87.76%

Merging [#68][cc-pull] into [master][cc-base-branch] will increase coverage by 1.49%

@@             master        #68   diff @@
==========================================
  Files            17         17
  Lines          1092       1095     +3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            942        961    +19
+ Misses          150        134    -16
  Partials          0          0

Powered by Codecov. Last updated by [2b3b9b9...e254bc5][cc-compare]
[cc-base-branch]: https://codecov.io/gh/JuliaLang/IterativeSolvers.jl/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/JuliaLang/IterativeSolvers.jl/compare/2b3b9b9a5cc3324637b9a919b7d67eea27f379f1...e254bc5460f7f69ee8cb22c4f049971a9a474c37?src=pr
[cc-pull]: https://codecov.io/gh/JuliaLang/IterativeSolvers.jl/pull/68?src=pr

Coverage increased (+1.5%) to 87.763% when pulling b816f2f8e686014db9c9b330b8e9d1d0b85dd869 on lopezm94:master into 2b3b9b9a5cc3324637b9a919b7d67eea27f379f1 on JuliaLang:master.


Add Layer Toggle to Stage 6 and various guideline edits

Add layer_toggle to Stage 6
Various guideline text and styling edits

Checked layer toggle on my system, seems to be working properly.


Update Composer dependencies (2021-07-01-00-06)

Visual regression test passed!
View the visual regression test report


pull request for e2e test at 01/16/2017 02:15:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add CRS info in AbstractGeoTable show

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

Project coverage is 85.53%. Comparing base (d35ad83) to head (7f016b9).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #118      +/-   ##
==========================================
+ Coverage   85.48%   85.53%   +0.04%
==========================================
  Files          21       21
  Lines         930      933       +3
==========================================
+ Hits          795      798       +3
  Misses        135      135

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


Error installing v5.6.3 on Windows 7

I seem to have run into an issue installing the v8-profiler (as part of installing node-inspector) on Windows 7. I've confirmed that this is only an issue with v5.6.3 (installation is successful with 5.5.0, 5.6.0 and 5.6.1). Here is the error:
C:\>npm install -g<EMAIL_ADDRESS>
><EMAIL_ADDRESS>preinstall C:\Users\afrasso\AppData\Roaming\npm\node_modules\.staging\v8-profiler-616a83b5
> npm install node-pre-gyp

C:\Users\afrasso\AppData\Roaming\npm
`--<EMAIL_ADDRESS>
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Program Files (x86)\\nodejs\\node.exe" "C:\\Users\\afrasso\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "install" "node-pre-gyp"
npm ERR! node v5.9.1
npm ERR! npm  v3.8.8
npm ERR! path C:\Users\afrasso\AppData\Roaming\npm\node_modules\.staging\v8-profiler-616a83b5
npm ERR! code EBUSY
npm ERR! errno -4082
npm ERR! syscall rmdir

npm ERR! EBUSY: resource busy or locked, rmdir 'C:\Users\afrasso\AppData\Roaming\npm\node_modules\.staging\v8-profiler-616a83b5'
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\afrasso\AppData\Roaming\npm\node_modules\.staging\v8-profiler-616a83b5\npm-debug.log
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Program Files (x86)\\nodejs\\node.exe" "C:\\Users\\afrasso\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "install" "-g"<EMAIL_ADDRESS>npm ERR! node v5.9.1
npm ERR! npm  v3.8.8
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>preinstall: `npm install node-pre-gyp`
npm ERR! Exit status<PHONE_NUMBER>
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script 'npm install node-pre-gyp'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the v8-profiler package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm install node-pre-gyp
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs v8-profiler
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls v8-profiler
npm ERR! There is likely additional logging output above.

I've also attached the debug log:
npm-debug.zip

I'm having the same problem as afrasso
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Dev\\npm\\node_modules\\npm\\bin\\npm-cli.js" "install" "node-pre-gyp"
npm ERR! node v6.1.0
npm ERR! npm  v3.9.0
npm ERR! path C:\Dev\npm\node_modules\.staging\v8-profiler-5db90e09
npm ERR! code EBUSY
npm ERR! errno -4082
npm ERR! syscall rmdir

npm ERR! EBUSY: resource busy or locked, rmdir 'C:\Dev\npm\node_modules\.staging\v8-profiler-5db90e09'
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     C:\Dev\npm\node_modules\.staging\v8-profiler-5db90e09\npm-debug.log

p/s: i keep failing uploading the debug.log, but other than the node and npm version, the content is identical to afrasso's

browsing through the other issue https://github.com/node-inspector/v8-profiler/issues/47
npm install -g node-inspector --msvs_version=2013
this solved the issue

What about 5.6.5?


Coinbase API V2/V3 Updates

This API now uses Coinbase Advanced Trade V2/V3.
If you are using legacy API keys from previous Coinbase versions they must be upgraded to Advanced Trade on the Coinbase site.
These keys must be set before using the Coinbase API (sorry).

@O-Mutt would you like to give this PR a review?

@O-Mutt , @vslee
I've updated the source and applied Matt's suggestions (thanks again, Matt).
All calls were tested for parameters and return values, and I even ran all sockets overnight to test for leaks, etc.  Everything works on my side, so I have to bow out of this side-project. Use this, don't use it, it's up to you, but as far as I can tell, this would be the first library to successfully incorporate the new Coinbase Advance Trade API. (and  it wasn't easy).
I won't be updating the ExchangeSharpTest app, and this no longer compiles. You'll have to do that update, as well as the console app.
Cheers,
Bob

Thank you for the excellent work @BobDeCuir. I will merge these changes now to allow @O-Mutt to test and base his own PRs on if needed. Once @O-Mutt is ready, I will push a new NuGet for wider testing.

I did some minor testing, aside from the fairly lacking logging setup in the class (there are literally no logs) I was able to use it to place successful orders. I'd be happy to push this out wider

Great, I will work on updating the CI and pushing a new release.