Fixed Mac address for external_service_interface

I have attempted to add a macaddress variable to kubeinit_libvirt_external_service_interface and then use it in the ifcfg-eth1.j2 template like this: MACADDR:<MAC-address> but it did not work. I have not tried HWADDR:<MAC-address> instead. What do you think would be the best solution? And will you accept a PR for it?
Thank you!

Hi @Bilal-io PRs are always welcomed, mostly because PRs that solve users issues are always good examples of how to improve the code. In the specific case of the external interface mac +1 from my side, to do this you should add these steps:
1- An additional dict variable like:
mac: 52:54:00:10:57:05 (note that 54:54:00: is always required in libvirt)
In https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/roles/kubeinit_libvirt/defaults/main.yml#L148
2- Add the mac variable to the eth1 interface in all the supported distros like:
--network bridge={{ kubeinit_libvirt_external_service_interface.attached }},mac={{ kubeinit_libvirt_external_service_interface.mac }},model=virtio \
In:
https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/roles/kubeinit_libvirt/tasks/10_rke_deploy_nodes.yml#L131
https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/roles/kubeinit_libvirt/tasks/10_k8s_deploy_nodes.yml#L133
https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/roles/kubeinit_libvirt/tasks/10_okd_deploy_service_nodes.yml#L128
3- The last step should be updating the interface template with an extra parameter like:
MACADDR={{ kubeinit_libvirt_external_service_interface.mac }}
In: https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/roles/kubeinit_libvirt/templates/ifcfg-eth1.j2#L18
With these steps you should be able to add a fixed mac address to this external interface.
Once you have this, a nice improvement can be to extract the IP's information from the libvirt role and put it in each env inventory, if you notice the variables here:
https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/roles/kubeinit_libvirt/defaults/main.yml#L144-L147
attached: "{{ kubeinit_inventory_network_bridge_external }}"
dev: eth1
ip: <IP_ADDRESS>
gateway: <IP_ADDRESS>
netmask: <IP_ADDRESS>
Should be something like:
attached: "{{ kubeinit_inventory_network_bridge_external }}"
dev: "{{ kubeinit_inventory_network_bridge_external_dev }}"
ip: "{{ kubeinit_inventory_network_bridge_external_ip }}"
gateway: "{{ kubeinit_inventory_network_bridge_external_gateway }}"
netmask: "{{ kubeinit_inventory_network_bridge_external_netmask }}"
mac: "{{ kubeinit_inventory_network_bridge_external_mac }}"
And they should be defined in:
https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/hosts/cdk/inventory#L16
https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/hosts/k8s/inventory#L16
https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/hosts/okd/inventory#L16
https://github.com/Kubeinit/kubeinit/blob/master/kubeinit/hosts/rke/inventory#L16
Like:
kubeinit_inventory_network_bridge_external_dev=eth1
kubeinit_inventory_network_bridge_external_ip=<IP_ADDRESS>
kubeinit_inventory_network_bridge_external_gateway=<IP_ADDRESS>
kubeinit_inventory_network_bridge_external_netmask=<IP_ADDRESS>
kubeinit_inventory_network_bridge_external_mac=52:54:00:10:57:05
The reasoning for this refactor is to have all the network details specified in the same place (the inventory for each distro).
I hope this is enough info :) and thanks for raising this issue.

You pretty much did all the work in that comment, so thank you!


Project refactor and changes to offline map

Project reorganization to match the current template for flutter plugins.
Added mechanism to read the tiles from the local storage when in offline map.

Please check if the reorganization is something that workout for you guys.

looks good to me - thanks for the PR and for your patience!


The Open Graph Parser should parse arrays of images with properties.

I updated the Open Graph Parser so that it can parse arrays of images with properties and present them as arrays of image objects with properties. There are other Open Graph meta tags which can also contain arrays of objects with properties, and this update does not address those, but provides a template which works differently than the general code which was featured before. This update is neither fully general nor fully specific but presents another way to do it. The new code still passes the old tests and the new tests, but I certainly image many other test cases which will still fail.

I'm going to resubmit with the suggested linter updates.


GROUP BY () not supported

Hello. First of all, thanks for this amazing project!
I've ran into an issue that explicit GROUP BY () is not supported.
For example, when we write:
SELECT SUM (value) from sales,
it is supposed to be the same as:
SELECT SUM (value) from sales group by (),
but the latter is not parseable by the parser. It produces:
ParserError("Expected an expression:, found: )").
I suppose that the issue is that it expects a row expression there, but () is just a special syntax for grouping by nothing.
You technically can specify GROUP BY (), value, which would just group by value, but if only () is present it treats all rows like a single group.
How would you suggest fixing this?

I would recommend finding a sql reference for the syntax (aka which database invented / used this syntax) to understand what () means in this context and then update the parser accordingly

() can be used in GROUPING SETS, ROLLUP, etc. It is used to include "single group" into the list of sets, for example:
SELECT count(*), o_orderkey, o_custkey
FROM orders
GROUP BY GROUPING SETS ((), (o_orderkey, o_custkey));

Will return something like:
 count | o_orderkey | o_custkey
-------+------------+-----------
     4 |            |
     1 |          1 |         1
     1 |          4 |         4
     1 |          3 |         3
     1 |          2 |         2

More useful here than in the example I provided in issue description actually :)

Thanks for the update @michael-2956  -- since this is an actual feature gap I'll plan to leave this issue open in case anyone else runs into the limitation and might benefit from your research.

Hi @alamb, I would like to take this task, and I want to hear your thoughts before starting the implementation.
We can implement this feature in three ways:

Add a dedicated group by type like GroupByExpr::All, maybe call it GroupByExpr::Nothing
Regard () as a special case for GroupByExpr::Expressions and an value like Value::Nothing, then return GroupByExpr::Expressions(vec![Value::Nothing]) to represent the group by nothing clause

I think the better solution is to use GroupByExpr::Expressions(vec![]) to represent the group by nothing condition, but it's now used to represent the GROUP BY clause doesn't exist and we should not break this pattern. So I prefer using solution 1 which adds a new enum value GroupByExpr::Nothing for this purpose. What do you think?

Sounds good! Representing it as an new variant sounds reasonable to me! (i.e. option 1. with GroupBy::Nothing)

@iffyio Thank you!


Update documentation re: pip install --no-use-wheel --download

The following command described in the installation instructions:
pip install --no-use-wheel --download ./ephemerol/vendor -r requirements.txt
did not work for me on my MacBook. I got the following deprecation messages:
DEPRECATION: --no-use-wheel is deprecated and will be removed in the future.  Please use --no-binary :all: instead.
DEPRECATION: pip install --download has been deprecated and will be removed in the future. Pip now has a download command that should be used instead.
Per the deprecation messages, I tried the following command and it did work:
pip download --no-binary ./ephemerol/vendor -r requirements.txt
I propose that the installation instructions be updated.

This change has been added to the Readme. Thank you for providing the suggestion!


Update guide of demo server

Pull Request Test Coverage Report for Build 7538

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 64.609%

Files with Coverage Reduction
New Missed Lines
%

src/common/scheduler/scheduler.go
1
81.29%

Totals

Change from base Build 7537:
0.01%

Covered Lines:
11083

Relevant Lines:
17154

💛  - Coveralls

@hainingzhang Updated.


It should not warn if a parent has an aria role and a child component has an aria-label

I ran across this issue and I would NOT expect a warning to be logged in this case.
it('does not warn when a child component is empty label and has an aria-label', () => {
  var Foo = React.createClass({
    render: function() {
      return (

      );
    }
  });

  doNotExpectWarning(assertions.render.NO_LABEL.msg, () => {
    React.render(
<Foo/>
, fixture);
  });
});

It appears the assertion code is only checking for text contents in the case of spans and is not taking into consideration aria-* attributes or role attributes.  I attempted role="presentation" and role="none" when rendering Foo but I still  received a warning.
Also, I believe there may be an off by one error in the assertions.assertLabel code as the following will pass in the previous test:
  doNotExpectWarning(assertions.render.NO_LABEL.msg, () => {
    React.render(
test<Foo/>
, fixture);
  });

+1
I'm running into this issue only when the first child has no text content.  For example:

Blah...

However, switching the child order doesn't trigger an aria-label warning:

Blah...

+1
I'm experiencing a variant of this problem when trying to make a dropdown menu. Each link starts with a contentless span with aria-hidden="true" (think FontAwesome icon), followed by the text of that link. react-a11y thinks all these links don't have content.

I'll need to do some checking but I'd be curious if after updating to v1.0.0 this is still an issue?

I'm going to close this as I'm not sure this is still an issue post v1.0.0.  If someone is still having this issue after updating please feel free to re-open.


Scenario sample framework

This creates a space for larger, scenario-driven samples to be shared. It includes one such sample that analyzes web traffic records to determine the user's origin.

Docs Build status updates of commit b06f444:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

_dependentPackages/CommonPlugins/tools/JoinTOC.ps1
:x:Error

Details

.openpublishing.publish.config.json
:x:Error

Details

:x:Error

Details

_dependentPackages/CommonPlugins/tools/JoinTOC.ps1

Line 82: [Error] Error happened when executing plug-in script _dependentPackages/CommonPlugins/tools/JoinTOC.ps1 at step docset_prebuild. Please check your .openpublishing.publish.config.json?on the configuration of the plug-ins or ask for help?in Docs Support teams channel. Exit code: -1073741502
Line 217: [Error] Error happened when executing plug-in script _dependentPackages/CommonPlugins/tools/JoinTOC.ps1 at step docset_prebuild. Please check your .openpublishing.publish.config.json?on the configuration of the plug-ins or ask for help?in Docs Support teams channel. Please also include the build report and the following error code for our team to troubleshoot: 1

.openpublishing.publish.config.json

Line 80: [Error] Failed to execute customized script at step docset_prebuild in docset office-scripts-docs. Please check your .openpublishing.publish.config.json?on the configuration of the plug-ins for docset office-scripts-docs step docset_prebuild or ask for help?in Docs Support teams channel. Please also include the build report and the following error code for our team to troubleshoot: 1

Line 138: [Error] W:\xbmu-s\.optemp\packages\opbuild.scripts.1.84.4\tools\opbuild\docset.prebuild.ps1 failed to execute

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.

Docs Build status updates of commit 03bf7dc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit c8342a8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit 6a8c789:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit f09c353:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit c950bfc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit a239bb1:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/resources/scenarios/analyze-web-downloads.md
:warning:Warning

Details

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md

Line 31: [Warning] Invalid file link:(~/resources/scenarios/analyze-web-downloads.xlsx).

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.

Docs Build status updates of commit 2fc52fb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit c91e0b6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit 9e1f061:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit 816b996:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Thanks @kbrandl! Updated in response to your feedback. Good catch on the script overwriting data. I've corrected the script and the "after" screenshot.

Received some offline feedback from @sumurthy. I'll address it and update the PR soon.

Docs Build status updates of commit e45dcad:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit 1d7cd2a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/resources/scenarios/grade-calculator.md
:warning:Warning

Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/images/scenario-grade-calculator-after.png
:white_check_mark:Succeeded

docs/images/scenario-grade-calculator-before.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/grade-calculator.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

docs/resources/scenarios/grade-calculator.md

Line 122: [Warning] Invalid file link:(~/images/scenario-grade=calculator-after.png).

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.

Docs Build status updates of commit e0376cf:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/images/scenario-grade-calculator-after.png
:white_check_mark:Succeeded

docs/images/scenario-grade-calculator-before.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/grade-calculator.md
:white_check_mark:Succeeded

docs/resources/scenarios/grade-calculator.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit 71d772f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/images/scenario-grade-calculator-after.png
:white_check_mark:Succeeded

docs/images/scenario-grade-calculator-before.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/grade-calculator.md
:white_check_mark:Succeeded

docs/resources/scenarios/grade-calculator.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

Docs Build status updates of commit 0391d99:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/docfx.json
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-after.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-before.png
:white_check_mark:Succeeded

docs/images/scenario-analyze-web-downloads-table.png
:white_check_mark:Succeeded

docs/images/scenario-grade-calculator-after.png
:white_check_mark:Succeeded

docs/images/scenario-grade-calculator-before.png
:white_check_mark:Succeeded

docs/resources/excel-samples.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.md
:white_check_mark:Succeeded

docs/resources/scenarios/analyze-web-downloads.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/grade-calculator.md
:white_check_mark:Succeeded

docs/resources/scenarios/grade-calculator.xlsx
:white_check_mark:Succeeded

docs/resources/scenarios/sample-scenario-overview.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

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.

@jedunn signed off in person. Merging!


grpc on windows generates warning C4244

What version of gRPC and what language are you using?
1.30.2
What operating system (Linux, Windows,...) and version?
Windows Server 2012R2
What runtime / compiler are you using (e.g. python version or version of gcc)
VS2017
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.

define a proto file similar to this:
syntax = "proto3";

option java_multiple_files = true;
option java_package = "com.intel.cloudfreed.ipservice.grpcproto";
option java_outer_classname = "IpServiceProto";
package MyServices;
message Test {
uint32 field1 = 1;
}
Another reproduction can be made on the supplied examples project cpp\helloworld building greeter_async_client.cc.
In the proto file grpc\examples\protos\helloworld.proto change the following message:
// The request message containing the user's name.
message HelloRequest {
string name = 1;
}
to:
// The request message containing the user's name.
message HelloRequest {
string name = 1;
int32 id = 2;
}

built the code in VS2017(building for 64 bit)

What did you expect to see?
passed build, no warnings/errors
What did you see instead?
compiler issued warning C4244
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?
The warning is caused due to assigning to the 32 bit (field1_) a value form an internal method which returns 64bits.
field1_ = ::PROTOBUF_NAMESPACE_ID::internal::ReadVarint64(&ptr);
the line is part of
const char* RoiData::_InternalParse(const char* ptr, ::PROTOBUF_NAMESPACE_ID::internal::ParseContext* ctx)
the method is part of the generated [proto file].pd.cc (from my proto file)
The problem does not exist in Linux

Hey Nico - would you be the right person for this issue?

This looks like the same issue as protocolbuffers/protobuf#7103, which has a fix in a PR right now.

We're not trying to remove warnings in all platforms & compilers but PRs to address this specific warning is welcome.


Unification of RuntimeManager{Request, Response} across Nitro, IceCap, and Linux

This is ready for review.


Auto-connect Bluetooth Speakers?

LG TVs have an issue with automatically connecting to Bluetooth speakers.

If I first turn on the TV and then the speakers, it identifies the speakers and prompts me to connect.
If I first turn on the speakers, wait a second, and then the TV, it connects them automatically.

In either case, the speakers go off automatically when the TV goes off.
Problem: Since the speakers have no smart functionality, I can't automate them to turn on and connect to the TV.
If I turn only the TV on, and connect to the recent speakers in the sound settings, this turns them on and connects them. So the TV has the ability to turn on and connect to the speakers, but I see no way of doing this automatically.
I thought I might be able to achieve this with Homebridge. I'm using your plugin, I've set the soundOutput button to bt_soundbar and it does set Sound Out to Bluetooth when the TV comes on. However, it doesn't connect to the speakers. Any idea?

Hi,
as far as I know it is it possible to trigger a connection using the public webos api. You can only change the sound output device.
What might work for you is that after you switch the sound output, you run a remote button sequence which triggers the connection. You would need to first check out what exact sequence is required for that and then try it out.

Okay, good to know. And thanks for the tip. I'll try that. 🙂


Does Redis-om support fuzzy search

Yes, you fuzzy matching can be done by surrounding the word you are interested in with %, that will do a match up to a levenshtein distance of the number of % on either side of the search term:
Member.find(m.Member.bio % "%parey%") # levenshtein of 1
Member.find(m.Member.bio % "%%parey%%") # levenshtein of 2
Member.find(m.Member.bio % "%%%parey%%%") # levenshtein of 3

this will work up to a levenshtein of 3
See Fuzzy Searching in our docs for more details.


space-in-parens exception allows for no space before closing paren

The current space-in-parens rule is:
'space-in-parens': ['error', 'always', {exceptions: ['{}', '[]']}],

gives you code like
if ( ! m[12]) {

instead of the expected
if ( ! m[12] ) {

I'm not sure if this is an eslint bug, or a ruleset bug. What do you think?

Similarly:
this.progressBar.addEventListener( 'touchstart', _.bind( this.seek, this ), { passive: true }) ;

gets changed to:
this.progressBar.addEventListener( 'touchstart', _.bind( this.seek, this ), { passive: true });

I checked the rule in eslint docs. It says,
// Examples of correct code for this rule with the "always", { "exceptions": ["{}", "[]"] }] option:
/*eslint space-in-parens: ["error", "always", { "exceptions": ["{}", "[]"] }]*/

bar({bar:'baz'});
baz( 1, [1,2]); // <-- Notice the space here
foo({bar: 'baz'}, [1, 2]);

I guess the rule simply means, there should be space inside parens, until it is followed or preceded  by any of these combinations {}, [].
IMO for the ^4.18 version of ESLint, what we need is,
"space-in-parens": ["error", "always", { "exceptions": ["empty"] }]

To disallow space for empty parens.
Thoughts @ntwb

Still happening with eslint 5.0.1.


Fixed the hangs in Ion_Force subroutine.

In some situations, Ion_Force subroutine hangs because the dzudr variable requests huge space to the stack memory.
From now on, would you please check each array size on the stack to avoid memory allocation error.

@yhirokawa san, thank you for your contribution.
Presently, one of the regression test of diana-gpu01 raise the error code,
due to the temporal problem in script to check the calculation results.
(This is not a critical problem of code, I guess it is possible to merge
with ignoring the error from diana-gpu)
Since I'm preparing the fix of the GPU testcases, please wait for a moment.
Sorry for inconvenience.

I confirm your PR works well, and it is possible to merge.
Thank you for your contribution!


⚠ ClamAV has degraded performance

In cc08ab5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 771b276 after 6 minutes.


[REQUEST] Sugarcane recycle into biomass

Description:

Sugarcane should be recyclable into biomass

Reason:

it is plant

done


Peer to Peer code review.

Hello @MayPyone,
I want to commend your exceptional effort thus far. Your code demonstrates a remarkable level of proficiency and craftsmanship. 💯
It showcases a flawless application of HTML and adheres to the best practices of CSS development. 🥇
I wish you the very best for your ongoing project. Keep up the outstanding work, and happy coding! 👍
Thank You,
Faiz

Thank You so much for your feedback @MayPyone . 😄


Wrong plugin directory location for .deb

graylag-server plugin directory is located at /usr/share/graylog-server/plugin but graylog-plugin-slack .deb package installed the .jar file at /usr/share/graylog2-server/plugin/graylog-plugin-slack-1.1.5.jar. Therefore, slack alarm callback doesn't show up in grayling web interface.

Thank you for the report!


[DUBBO-3137]: step4 - move more constants from RemotingConstants into dubbo-remoting-api

What is the purpose of the change
issue #3137: step4 - move more constants from RemotingConstants into dubbo-remoting-api
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #4077 into master will increase coverage by <.01%.
The diff coverage is 75.89%.

@@             Coverage Diff             @@
##             master   #4077      +/-   ##
===========================================
+ Coverage     62.99%     63%   +<.01%
  Complexity      544     544
===========================================
  Files           756     756
  Lines         32578   32578
  Branches       5163    5163
===========================================
+ Hits          20523   20525       +2
+ Misses         9694    9693       -1
+ Partials       2361    2360       -1

Impacted Files
Coverage Δ
Complexity Δ

...che/dubbo/registry/dubbo/DubboRegistryFactory.java
89.74% <ø> (ø)
6 <0> (ø)
:arrow_down:

...src/main/java/org/apache/dubbo/remoting/Codec.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...pache/dubbo/monitor/dubbo/DubboMonitorFactory.java
100% <ø> (ø)
6 <0> (ø)
:arrow_down:

...rc/main/java/org/apache/dubbo/remoting/Codec2.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...he/dubbo/remoting/transport/netty/NettyClient.java
66.1% <ø> (ø)
11 <0> (ø)
:arrow_down:

...java/org/apache/dubbo/remoting/etcd/Constants.java
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

...g/apache/dubbo/config/AbstractReferenceConfig.java
94% <ø> (ø)
0 <0> (ø)
:arrow_down:

...e/dubbo/registry/integration/RegistryProtocol.java
82.69% <ø> (ø)
0 <0> (ø)
:arrow_down:

...ache/dubbo/remoting/transport/mina/MinaServer.java
63.33% <ø> (ø)
6 <0> (ø)
:arrow_down:

.../apache/dubbo/remoting/p2p/support/ServerPeer.java
21.27% <0%> (ø)
0 <0> (ø)
:arrow_down:

... and 58 more

Continue to review full report at Codecov.

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


Column provided for geojson renderer should not be null

When providing columns as option in layers null should be skipped.
See https://github.com/CartoDB/cartodb/issues/7520

cc @dgaubert


Newbie

I'm not sure what im doing wrong, but i get permission denied when doing step 1 to boot user config. Then on step 3, im not sure how to run read.py. I ran a similar program using talaexe's spotify rfid player. But when i boot up my volumio pi, the volumio player shows up, not the normal PI Linux. Any tips/help?

@bbendolph

What do you mean by boot user config? Step 1 is to add the line dtparam-spi-on to your /boot/userconfig.txt file. You can use the nano text editor to do it even if the file has not existed yet.

nano /boot/userconfig.txt

For step 3, you first need to be in the rfid-volumio working directory and then run a utility program to read the details of your rfid cards with python3 utils/read.py. Make sure you have installed all the packages required in step 2. Keep in mind that this step just allows you to read the details of your empty rfid cards so that you can insert the IDs of those cards in the CSV file that you'll be creating in step 5.

# Assuming you're in the root directory
cd rfid-volumio
python3 utils/read.py

Volumio is itself another Linux-based operating system, so that's why you'd see Volumio's UI when you boot your system up.
I checked out talaexe/Spotify-RFID-Record-Player, and I assumed they were running their application in the standard Raspberry Pi OS.
You can run talaexe's application in Volumio OS too, as it just runs headlessly in the background.

I guess my problem is I don't really understand the difference between Volumios UI and the "normal Linux" PI UI, and how to run things like the read.py in the bakground or even get them to the Volumio Pi. I ran the spotify one on a separate SD card as a test.

May I know your setup currently?

Current setup is a pi with the 7 in touchscreen display, and an MFRC522 reader. THe pi OS SD card in the system now is base Volumio flashed. The other SD card not in the pi is the standard pi linux with tala's programs on it

Do you have access to the command line of Volumio?

I believe so, using SSH(?) and also on initial boot up
On Fri, Jul 21, 2023 at 3:25 AM Francis Yeong @.***>
wrote:

Do you have access to the command line of Volumio?
—
Reply to this email directly, view it on GitHub
https://github.com/fywk/rfid-volumio/issues/2#issuecomment-1645109822,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BARIC2NORVSJ6VM2RI6OLTDXRIVGXANCNFSM6AAAAAA2QQKRBQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great. Then what seems to be the problem?
You mentioned:

...how to run things like the read.py in the background.

You only need to run read.py when you want to read the id of your cards. Once the id is known, you can exit the script.
Here's the flow of how :

Scan your rfid card with read.py (only run this to get to know the id of your card)
Copy the id of the card from the output
Use the id that you've just acquired and paste it inside the playback.csv` file
Associate the id with a valid playback URI

So let's say you have a card with an ID of<PHONE_NUMBER>00, and you want this card to play BBC Radio 2 every time you scan it. The following configuration in playback.csv would do just that.
ID,Service,URI,Name,Type
000000000000,webradio,http://stream.live.vc.bbcmedia.co.uk/bbc_radio_two,BBC Radio Two,Web Radio

I'm closing this for now. Feel free to ping me if you have any questions :)


Remove fillna call in ArbitraryImputer (Issue 195)

ArbitraryImputer was calling fillna both in it's own transform method, and in the BaseImputer method.
To increase tubular's efficiency and maintainability, have removed fillna call in ArbitraryImputer and moved the logic for "casting imputer value as same dtype as original column" below the BaseImputer.transform call.
Note that the above change caused the test_impute_values_set in test_ArbitraryImputer.py to fail since the initial mock setup for BaseImputer.transform was returning a simple integer:1234. However, this causes 'TypeError: 'int' object is not superscriptable' for the following code line in transform method of ArbitraryImputer: X_transformed[c] = X_transformed[c].astype(dtype), since an integer cannot be treated as a collection. Therefore have updated test_impute_values_set to mock BaseImputer.transform to return a dataframe rather than an int

This looks good to me RJ, looks like there are just a couple of ruff things failing in CI - could you fix those and then will approve? :)

Done! Have a look at the changelog and let me know if it's fine :)


Python/pandas apply

This adds a new recipe that ....

Closing since this was for demonstration of workflow.


Duane Bester published some videos, providing much needed explanation on how to write apps with GPUI.

Thank you!


No validation when adding ValidateNumber scalar

Describe the bug
Constraint validations are not triggered when adding scalar ValidateNumber in GraphQL schema. Removing it make the API working with validations, but GraphQL tools such as GraphQL codegen are not working because it doesn't recognize ValidateNumber scalar.
To Reproduce
Steps to reproduce the behavior:
scalar ValidateNumber

directive @constraint(
    # String constraints
    minLength: Int
    maxLength: Int
    startsWith: String
    endsWith: String
    notContains: String
    pattern: String
    format: String

    # Number constraints
    min: Int
    max: Int
    exclusiveMin: Int
    exclusiveMax: Int
    multipleOf: Int
) on INPUT_FIELD_DEFINITION

Expected behavior
Graphql constraint directive should be called with scalar ValidateNumber defined in the schema.
Additional context
After debugging into the library, it seems that @constraint directive is actually detected, and the wrapType method is called. But when making a GraphQL query or mutation with wrong inputs, the custom ValidateNumber parseLitteral or parseValue method is not called.

It seems https://github.com/confuser/graphql-constraint-directive/issues/2 solved this problem. Using a custom scalar wrapping the type directly is not working anymore. @confuser solved it by using schemaTransform and providing his own typeDefs.

Thanks for the ping @JeremiFerre, the original package resolves this issue, and is fully compatible with apollo v2 so not sure the reason for this fork but if there's anything missing happy to discuss/open to PRs.

Yeah I've switched to the original package and everything works great. Thank you for your quick answer !


Delegate list empty when trying to view stakes

Clicking on Vote for Delegate to see staked coins is taking me to an empty web page. Unable to see or manage stakes

it was due to analytics data issue. it should be fine now.


⚠ Bofrost.be has degraded performance

In 6c0eee6, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1202 ms

Resolved: Bofrost.be performance has improved in 010947e after 6 minutes.


RLHF in MARL

I want to konw if there are any researches focusing on leveraging RLHF in MARL problems.

As of 2023.4.14, there is no such solid or insightful research work for RLHF in MARL.

Thanks a lot!


Offset to Sync Video and Audio?

I record my videos with external audio, so sometimes, the files have different start times. It would be fantastic to be able to give a number of frames as the offset between the video and audio track, so I don't have to export a synced file first.
´auto-editor file_name --video_offset 5 -exr`
Since this is using FFmpeg, I was looking at the usage of the --itsoffset flag. I don't know if this would be viable. Here's a good link I found about it: https://superuser.com/questions/982342/in-ffmpeg-how-to-delay-only-the-audio-of-a-mp4-video-without-converting-the-au
I don't expect the auto-editor to automatically find the offset. That would be some magic I don't know how to do. But supplying a constant offset rather This example would cause the video to be offset by a constant 5 frames against the audio syncing both. I have looked through the help and the documents and couldn't see it. Apologies if it exists already.

Also, thank you for this amazing tool.

yeah I don't see the benefit of this being embedded in auto-editor.


Adding License for dll

Hello Sir,
Could you please include license in GitHub for Control.Draggable.dll
Thanks.

Hi @Harini2019 , I've added MIT license file to repo. Does it satisfy your needs?

Thanks much !!
I believe that should be fine, I will definitely post if there is a need.
Thanks again for quick response.
On Thu, Feb 7, 2019 at 7:07 PM Tymur Mudzhyri<EMAIL_ADDRESS>wrote:

Hi @Harini2019 https://github.com/Harini2019 , I've added MIT license
file to repo. Does it satisfy your needs?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/intrueder/Control.Draggable/issues/1#issuecomment-461421627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AtNyOkbOIGd_Y6eoDr9d6zOg7kLGz52nks5vLCwVgaJpZM4amuVS
.


[lexical-playground][TableCellResizer] Feature: Center the drag zone over the edge of the cell

Description
Shift the selection rectangles to the center of the edge, instead of being outside the cell. This makes for a more intuitive user experience as the user will try to select the edge, not the outside of the table cell.
Test plan
Before
(briefly visualized the areas so that the change can be seen)

After

Insert relevant screenshots/recordings/automated-tests

@ebads67 is this the final version we settled on?

@ebads67 is this the final version we settled on?

Yes, this is just centering the cell resizer drag zone.


Why are we getting so many connection failures in mock on SM?

When we run tests in SM on desktop we are seeing a huge number of connection failures which really shouldn't be happening. We also see that the test 48. test repeat write 2 caused hangs in at least two of the 4 failure logs that Artem had captured. But even when we pass 48 we see a lot of failures. We need to compare the log output from SM to the log output from v8 and see if we are seeing the same errors there. But even if we are we shouldn't have this many connection errors, what is going on?

Same about test 13.

The output from tests 46, 47 and 48 are full of errors on both SM and V8
[33mDEBUG createPeerListener: failedConnection

DEBUG createPeerListener: Recreating listener

DEBUG createPeerListener: failed incoming connection because of mux promise failure - Error: Connection could not be established

DEBUG createPeerListener: createPeerListener creating new server

DEBUG createPeerListener: listening 58004

DEBUG createPeerListener: pleaseConnect= false

DEBUG createPeerListener: first connection

The tests hang on test 195 on SM
2016-09-01T13:24:28.538Z - info: Running on ios test: 195. Test TransientState

# 195. Test TransientState

# 195. Test TransientState

# 195. Test TransientState

ok 816 should throw

ok 807 should throw

ok 808 should throw

ok 817 should throw

ok 809 should be equal

ok 818 should be equal

ok 810 should be equal

ok 811 should be equal

ok 819 should be equal

ok 812 should be equal

ok 820 should be equal

ok 821 should be equal

ok 813 (unnamed assert)

ok 814 (unnamed assert)

ok 822 (unnamed assert)

ok 823 (unnamed assert)

# teardown

# teardown

undefined
2016-09-01T13:24:32.617Z - info: Socket to device 8abbf58d-0437-4ba7-9579-845b445922f7 disconnected: transport close

2016-09-01T13:26:27.954Z - debug: Too many emit retries to device: 8abbf58d-0437-4ba7-9579-845b445922f7

but on V8 all is fine.

Obviously the hang on Test TransientState is very bad. But the other errors you are seeing aren't necessarily bad. For example, there are race conditions where we are tearing down a test and before we have it completely torn down we might be trying to connect to remote peers who are already tearing down so we will get connection errors. That isn't bad and the test itself shouldn't fail as a result.  So just getting a mux promise failure or an ECONNECTIONREFUSED is not enough to say that we have a failure. We have to see where this error occurs. That having been said we at least know that wifi based mock isn't clearing its state properly so that would cause lots of fun problems.

I believe this is now fixed when we fixed the retry logic.


Using ./ in a relative path corrupts the bundle zip

When using apigeecli apis create bundle -n helloworld -f ./myfolder/apiproxy, the proxy gets created, but is undeployable. This is due to differences in the bundle zip.
I wrote a program to output the entries in the bundle zip.
This is what a bundle looks like that was created with the-f proxies/hellowworld/apiproxy option.

contents of healthy_proxy.zip

apiproxy/
apiproxy/policies/
apiproxy/policies/add-cors.xml
apiproxy/policies/check-quota.xml
apiproxy/proxies/
apiproxy/proxies/default.xml
apiproxy/targets/
apiproxy/targets/default.xml
apiproxy/helloworld.xml

...and this is what a bundle looks like that was created with-f ./proxies/helloworld/apiproxy option.

contents of unhealthy_proxy.zip

./
./proxies/
./proxies/helloworld/
./proxies/helloworld/apiproxy/
apiproxy/
apiproxy/policies/
apiproxy/policies/add-cors.xml
apiproxy/policies/check-quota.xml
apiproxy/proxies/
apiproxy/proxies/default.xml
apiproxy/targets/
apiproxy/targets/default.xml
apiproxy/helloworld.xml

On Mac:
Using -f ./apiproxy worked for me and I could deploy, the downloaded zip looked right too.
Using -f ./folder/apiproxy worked too and I could deploy in the UI, the downloaded zip has:
unzip test-path-with-dot_rev3_2023_09_19.zip
Archive:  test-path-with-dot_rev3_2023_09_19.zip
   creating: folder/
   creating: folder/apiproxy/
   creating: apiproxy/
   creating: apiproxy/policies/
  inflating: apiproxy/policies/AM-Response.xml
  inflating: apiproxy/policies/AM-XFF.xml
  inflating: apiproxy/policies/PY-OSVariables.xml
   creating: apiproxy/proxies/
  inflating: apiproxy/proxies/default.xml
   creating: apiproxy/resources/
   creating: apiproxy/resources/py/
  inflating: apiproxy/resources/py/PY-OSVariables.py
  inflating: apiproxy/test-path-with-dot.xml

Interesting. I'm also on a mac and it was totally broken for me when using the ./ prefix.

@gislikonrad / @kurtkanaskie - can you share the apigeecli version you are running?

> apigeecli -v
apigeecli version 1.123 date: 2023-09-07T15:53:40Z [commit: 863de4d]

I installed yesterday

Same for me regarding Mac and cli

I do notice that the exported bundle has an extraneous folder in it, but it still deploys.
apis create bundle --name=test-path-with-dot --proxy-folder=./folder/apiproxy
apis fetch --name=test-path-with-dot
unzip test-path-with-dot_3.zip

tree
.
├── apiproxy
│   ├── policies
│   │   ├── AM-Response.xml
│   │   ├── AM-XFF.xml
│   │   └── PY-OSVariables.xml
│   ├── proxies
│   │   └── default.xml
│   ├── resources
│   │   └── py
│   │       └── PY-OSVariables.py
│   └── test-path-with-dot.xml
├── folder
│   └── apiproxy
└── test-path-with-dot_3.zip

@kurtkanaskie What if you point to a subfolder that is deeper that two levels? Example, './proxies/folder/apiproxy'

There have been others with a similar problem (#276). Unfortunately we have not been able to reproduce it.
I was able to create and deploy a proxy with: apigeecli apis create bundle -n test1 -f ./proxies/test1/apiproxy -t $token, followed by apigeecli apis deploy -n test1 -e dev -v 1 --wait -t $token

I think I have identified the problem. My comment here was incorrect. That proxy did not deploy successfully.
I narrowed the problem to this: 1) If there are two or more subfolders 2) And the relative path begins with a folder that is used in the proxy bundle (ex: proxies, policies, sharedflows etc), then we run into a problem. I think this fix addresses the problem.
@kurtkanaskie - I can show you how to reproduce the problem and then we can see if the fix addresses all variations.

This has been successfully fixed. Thank you.


update Google maps API key for 2021.alaoweb.org

The map isn't being used anywhere right now, is it? Since we're virtual again, we can clear out the key for this year's conference, or I think when Derek set up the Google Maps API, it was for the alaoweb.org domain, so there probably isn't a need to update it in the first place, is there?

Yup - I think you're right; might be needed  next year, but that's next year's problem.


[Question] Can go-ping be used as an ICMP listener only?

First off, this is an awesome library!
The title pretty much says it all. I'd like to set up an ICMP listener (and logger) on Windows.
Can go-ping be used for this purpose?

It cannot be used for this, but if you want to write a listener yourself you might want to look at the code here: https://github.com/sparrc/go-ping/blob/master/ping.go#L397


Accessing has many on user

Discussed in https://github.com/duncanmcclean/runway/discussions/195

Originally posted by Keoghan January 18, 2023
Hi,
Thanks for this package, it's a super addition to the world of Statamic.
I'm having trouble accessing a HasMany relationship on a user record.
The relations are all in the DB, and I have the relationship set up in the \App\Models\User. I suspect the issue is that the instance of the User in the control panel is not an \App\Models\User but something else?
Any thoughts on how to work with it?
Thanks, Keoghan

Not sure where to put this additional information:
The user is coming through as Statamic\Auth\Eloquent\User. As standard this doesn't include any relationships from the usual App\Models\User.
I can begin to pass the relationships through using computed - in this case jobs.
User::computed('jobs', function ($user, $value) {
     return $user->model()->jobs;
});

However, when the related models appear in the table, the first column is a json representation of the whole model, and the remaining columns are empty.

It might be worth checking if you can do updates?

I am having some issues with updates - will attempt to summarise tomorrow.

Yeah, I'll take a look when I have some free time.
My suspicion is that you won't be able to manage "real" HasMany relationships without making the User a resource on Runway.

Hi Duncan,
Thanks - I have been playing with adjusting settings etc and I can get the jobs table to behave nicely on its own. When it is drawn in through the mechanism above, the data is not processing the same way (so belongs to relations on jobs show as integers, dates are not formatted etc) - and updates fail for the moment.
I will create an alternative model to test through with it as a Resource on Runway - doing so with User didn't change the outcome so far.

Thanks Duncan. I see what you are saying but I am slightly confused by it.
Loosely here we have:

User, a Statamic Eloquent model.
Department, a Runway Model, ID, name
Job, a Runway model with belongsTo relations to User and Department.

The user has a hasMany relation with Jobs.
In this situation, all the data is stored in the Runway tables - so I don't see why we would need to be trying to save id's only. We can access full eloquent models for each job from the user using the computed jobs property, for example.
The table could be populated with a separate call Ajax outside of the main script - using user_id as a filter variable. As the straight Jobs Runway listing works well.
Perhaps I have missed something? I did try setting up a Runway model for users but since I am trying to extend the norma l user edit form, it is still passed the Statamic Auth User model.

You won't be able to use the Has Many fieldtype with native Eloquent relationships when using the normal users edit form, even though you're using Eloquent for your users.
You will need to setup a separate Runway resource for users if you want to use the native Eloquent relationships, that's just the way the fieldtype works.
If you need to manage the Eloquent relationships using the normal users edit form, I'd suggest looking into creating your own fieldtype to handle it.

Ok, thanks for your assistance 👍

Not sure where to ask this, but I've ran into the same issue where a relationship on the user form would not function correctly. Since it also causes some computed fields to not work correctly, I decided to create a normal runway resource for the users in order to fix that.
While the relationship and computed field now function perfectly, it has caused a new issue.
The roles & groups fields no longer function correctly, it seems Runway does not support the user_roles & user_groups field types?
I don´t see it listed in the list of supported fieldtypes: https://runway.duncanmcclean.com/blueprints#content-supported-fieldtypes
Would it be possible to add support for these fields?

This issue is over a year old. To avoid past participants being notified, are you able to open a new issue instead? Feel free to reference this one.
Runway should support any fieldtype, you just need to make sure the column type is right (eg. if it has "max items" = 1, it should be a string, otherwise, it should be a json column). If any fieldtypes are missing from the docs, feel free to submit a PR to update the docs.


Not able to list polices on management group

Describe the bug
Not able to list polices on management group.
az policy assignment list \
--disable-scope-strict-match \
--scope "/providers/Microsoft.Management/managementGroups/my-mg-group"
--query [].name

To Reproduce
Steps to reproduce the behavior.

Create a policy

 az policy definition create \
   --management-group my-mg-group
   --name 'sumit-enforce-nsg-on-subnett' \
   --display-name 'Sumit- NSG X on every subnet' \
   --description 'This policy enforces a specific NSG on every subnet' \
   --rules <azurepolicy.rules.json> \
   --params <azurepolicy.parameters.json> \
   --mode All

Assign the policy to management group.

az policy assignment create \
  --name "Force nsg in subnet" \
  --scope "/providers/Microsoft.Management/managementGroups/my-mg-group" \
  --policy "/providers/Microsoft.Management/managementgroups/my-mg-group/providers/Microsoft.Authorization/policyDefinitions/sumit-enforce-nsg-on-subnett" \
  -p '{"nsgId": { "value":"sumit-test-vm-nsg"}}'

below command should work, but not working for me.
az policy assignment list \
--disable-scope-strict-match \
--scope "/providers/Microsoft.Management/managementGroups/my-mg-group"
--query [].name

Expected behavior
I should be able to list the assigned polices in management group
Environment summary
Install : pip
CLI version: azure-cli                         2.0.60 *
OS version: macOS Mojave 10.14.4
Shell type: bash
Additional context
Add any other context about the problem here.

Hi Sumit (@murarisumit) ,
Thanks a lot for your bug report! Could you please send us what error you are getting when you try to list the assigned policies?
Thanks,
Arun Chandrasekhar
Principal Program Manager
Azure Management Experience

Sorry for late reply, but when I just went into scope without --disable-scope-strict-match flag. I am getting empty list.
When trying with --disable-scope-strict-match flag, I am getting policies from different scope, but still not the policy that I'm able to view in azure UI portal and created by using az policy assignment create command.

add to S164.

We are already upgrading this issue, you can work around it for a while in this way:
az rest --method get --uri "https://management.azure.com/providers/Microsoft.Management/managementGroups/{managementGroups}/providers/Microsoft.Authorization/policyAssignments?api-version=2019-09-01&$filter=atScope()"

@pilor  Please pay attention to this issue. After upgrading the service and SDK, please let us know. Thanks

ack

@pilor any update for this issue?

There is a problem with the azure-mgmt-resource python package generation in recent versions which is preventing me from generating a package with the changes I need.  @zikalino is looking into it.

@pilor  nice to know, please let us know when the PR is ready.  @zhoxing-ms  to follow up.

@zhoxing-ms issue is fixed in the linked PR

@zhoxing-ms please check the PR to fix the issue.

@murarisumit Hello, this issue has been fixed in the latest version of azure-cli (2.2.0), please download the latest version of the package to try.
If there are any questions, please feel free to give us feedback~


rotation stalls process indefinitely

A small rotation "stress test" surfaces a defect in rotation. Configure a logger with small size that guarantees a rotation after every logger.info(). Then run a series of logger.info() paced at 10ms, and the process will stall. Enabling some console logging after each logger.info() avoids the stall, allowing the series to complete and produce the expected log files.
While the process is in the stall, an ls command shows an interesting error that seems to indicate a rotation file is in the process of being created but isn't quite there yet.
Same result with node 8.9.4 or 6.11.2; CentOS6 or Windows 2016. Repro module attached.
rotatebug.zip

Hah test bug on my part! Sorry.


Can't Reselect NPM or Yarn after first project creation

Version
3.0.0-beta.10
Reproduction link
https://codesandbox.io/s/vue
Steps to reproduce

install the vue-cli globally.
run vue create project-name
follow the steps.
It then asks you to select NPM or Yarn.
install project
create another project and the option for NPM or Yarn does not show again

What is expected?
Option to select the npm or yarn again when creating a project
What is actually happening?
just using the one you initially set, without an option to reset the initial option you selected

After this when creating a new vue project it will always use the package manager you originally set and not give you and option to select npm or yarn again. There is no way to reset this either. Uninstalling the package globally closing/opening the terminal does nothing either.
OS
macOS 10.13.4

Same results ^
OS: Ubuntu 18.04

Further findings
You can manually set the package manager. by doing
vue create test --packageManager=npm
But we still need a way to reset the default. As if I go
vue create test it will try and use yarn (which I initially selected)

Figured it out >_<.
there is a file in your home dir called .vuerc you can set the package manager there.
npm or yarn for e.g
{
  "useTaobaoRegistry": false,
  "packageManager": "npm"
}

@jackiboy do you know what's the location of that file on Windows?
Does anyone know?

<Your home folder>/.vuerc. It's the same path on all OSes.

On Windows it's not know as Home folder, but User folder.
After some troubles you can find it by typing on cmd vue config. The command will show the path to the .vuerc file (most likely in C:\Users\user_name\.vuerc) along with its content.

For the location, I just searched the file name .vuerc in C folder.

I thought the .vuerc file wasn't there either, but then I realized it was just hidden.

I thought the .vuerc file wasn't there either, but then I realized it was just hidden.

for my case i don't have it even inside my hidden files i check it since longtime ago ,even right now i don't have it
>ls -a ~
....                 .vmodules
.nix-defexpr         .vmware
.nix-profile         .vscode
.node_repl_history   .wget-hsts
.npm                 .wine
.npmrc               ....

I thought the .vuerc file wasn't there either, but then I realized it was just hidden.

for my case i don't have it even inside my hidden files i check it since longtime ago ,even right now i don't have it
>ls -a ~
....                 .vmodules
.nix-defexpr         .vmware
.nix-profile         .vscode
.node_repl_history   .wget-hsts
.npm                 .wine
.npmrc               ....

Well if it helps any, I'm on Windows and I used the File Explorer. It didn't seem to be there at first, but then when I started writing the path for it, it just appeared as you can see in the picture I attached.

Well if it helps any, I'm on Windows and I used the File Explorer. It didn't seem to be there at first, but then when I started writing the path for it, it just appeared as you can see in the picture I attached.

As Windows i think you have it however for linux (deepin-20.2 & ubuntu-20.10) i don't have it ,instead of it i've .pnpmrc & .yarnrc. @jpuljko i think there's an issue related to the Linux part on how to treat it.

Changing the default package manager.
On windows open your C drive it can be see or whatever in my case it is C. Locate Users folder open the current user folder. You will find a file called .vuerc edit the file and change where it says packageManager to whatever package manager you are using. It can be yarn or npm.  This is how the .vuerc  looks like:
{
  "useTaobaoRegistry": false,
  "packageManager": "yarn",
  "latestVersion": "4.5.13",
  "lastChecked":<PHONE_NUMBER>045,
  "presets": {
    "vue-ts": {
      "useConfigFiles": true,
      "plugins": {
        "@vue/cli-plugin-typescript": {
          "classComponent": false
        },
        "@vue/cli-plugin-router": {
          "historyMode": false
        },
        "@vue/cli-plugin-vuex": {},
        "@vue/cli-plugin-eslint": {
          "config": "prettier",
          "lintOn": [
            "save"
          ]
        }
      },
      "vueVersion": "2"
    }
  }
}

Selecting the package manager during project creation.
Note You can select the package manager you want to use during project creation when using the vue-cli by running the command vue create <app name> with the --packageManager flag for example:
vue create <app_name> --packageManager=yarn

You can see your Vue config with
vue config
 {
  "useTaobaoRegistry": false,
  "packageManager": "pnpm"
}

You can edit your Vue config object with :
vue config -s <KEY> <VALUE>
So, for setup npm as default Package manager for Vue :
vue config -s packageManager npm
More Vue config:
vue config -h


acquisition entries not appearing

This may have to due with the sheer number of them (51)
https://neurosift.app/?p=/nwb&url=https://api.dandiarchive.org/api/assets/4f821f30-01c0-4a8a-a7fc-9144bc0019a5/download/&dandisetId=000020&dandisetVersion=0.210913.1639

I am also seeing Error: Data for /specifications/core/2.2.4/namespace is not a string in the dev console.
This is a pretty old NWB file, so it may not have all the latest best practices and features

Yeah, there was a problem parsing the specifications.  So it should be working now.


Scrolling automatically

Describe the bug
I'm creating a platform with the terminal, and I'd include the component at the bottom of the website when we enter the first time the platform scrolls to the position of the terminal. I tried to fixed with the overflow-anchor: none but didn't work.
To Reproduce
Steps to reproduce the behavior:

Create a website with different sections that needs to scroll to the position of you're react-terminal-ui
Load the website
The website is going to scroll automatically to the terminal.

Expected behavior
I don't want this to automatically scroll to the terminal. How can I configure it to avoid this?

Hey @tomasmalio . Right now the terminal is setup so the terminal cursor scrolls into view (so the user always sees the cursor). We may be able to change it so the scroll into view is localized to the terminal container. Feel free to submit a PR for the change.

@tomasmalio This is available in v1.3.0.

HI @jonmbake I'm checking if everything works because right now in my app doesn't work

@tomasmalio What browser are you using? I'm not seeing the behavior you described when testing in Firefox 125.0.3 on MacOs. Created a 1000px height div and a terminal below (off screen). When I load the page, it does not scroll to the terminal:

Also, I should have more closely reviewed the code change... The scroll into view only happens on key press so the code change you made is likely unnecessary. I suspect maybe the browser you are testing in auto-scrolls to focused input on page load; the issue is likely with the autoFocus={ onInput != null } bit.

Hi @jonmbake, I'm testing in MacOS with Chrome 124.0.6367.119.
I tried to do it with the autoFocus, but it also goes to the terminal. I'm trying to understand why.

@jonmbake I have tried multiple ways, but I have not succeeded. Keep scrolling.
I have developed a platform with react js, and when it initializes, it directs me to where the terminal is.

@tomasmalio Is your platform/website publicly accessible? Can you email me the URL? I would like to take a look at the behavior. I'm not able to reproduce it locally.
Also, did you try disabling autofocus from the terminal code? Did it have an effect?


[Block] - Media

Use the video block that was created to combine to create a new block type - Media
Support:
Embed block
Video block
Vimeo
Make extensible to more types if needed

Closed per #51


Dang 1423/phase-out old components Button, Link, Typography (breaking changes)

JIRA
DANG-1423 phase-out components: Button, Link, Typography
Description
Breaking changes
Removes Typography component. Instead of Typography wrapper components, we now use Tailwind Component Classes
Renames components:

NewButton -> Button, exported as LobButton
NewLink -> Link, exported as LobLink

Removes old Button & Link components.

broken preview


eyeddx

Differential diagnosis of eye complaints using rule of 2s.

Also, thanks so much for contributing @pgop88! Looking forward to more in the future! 😄

@pgop88 Thanks for fixing the items brought up by @jvoros. We still need you to also fix the things I mentioned before this can be merged.
Thanks 😄

@pgop88 thank you so much for working through this! We set up this new system, and we're trying to figure out how it will work with the community as a whole. So thank you for being a 'guinea pig'.
I've added some screencasts to the Wiki tutorials.
Check out this one on Building a Card: https://github.com/aliemteam/aliemcards/wiki/How-to-Build-an-ALiEM-Card
And then I've updated the Adding a Card tutorial with video help as well: https://github.com/aliemteam/aliemcards/wiki/Tutorial:-Adding-a-Card

@jvoros The videos you created are amazingly helpful and demystified the entire process! Had no idea it was really this simple to create a card. Can't believe I was struggling so much.
Thank you again!!
By the way
@dsifford @jvoros just created a new card/folder and incorporated the changes you mentioned for this card!

Thanks @pgop88. Closing this Pull Request in favor of #170


Latex Processing Unit wastes containers of latex

Industrial Foregoing Version:  1.11.5
TeslaCoreLib Version:  1.12.2-1.0.15
Crashlog (If Any):
Using Forestry, get a stack of latex cans, put them in the latex processing unit, and it eats the entire stack very quickly and does not dump 64 buckets worth into the machine.  It looks like each tick an item is eaten regardless of whether it's dumped into the machine or not.
Also happens with Buckets of latex, so not Forestry specific.

Can't replicate in a clean enviroment with Forestry.


Lightweight Structure testing

Hi Everyone.
I just added new option called lightweightStructure. Could you let me know if you have any issues / suggestions would be great!
Demo:
http://www.owlgraphic.com/owlcarousel2/demos/lightweight.html

Closing this.  Someone please reopen if this is a feature that people would like to see.


Basic Map Pack Support

This PR includes basic map pack support. When a game change is detected, the skin will refresh, allowing for the proper JavaScript to be loaded, allowing the skin to load the correct map.
Additionally, game-specific settings are now supported.

Moving this PR to the 3-3-0 branch instead.


Using ngImgCrop with ng-file-upload

I'm having some issues when trying to use ngImgCrop in conjunction with ng-file-upload. Basically, I can get both working, but not together.
Does anyone have any experience using these two directives together and how did you do it?

If it helps, here is a partial overview of how these are both implemented.
There is an event listener for the the input field (#fileInput) that calls handleFileSelect.
	<input type="file" id="fileInput" />

	angular.element(document.querySelector('#fileInput')).on('change',handleFileSelect);

And ng-file-upload is implemented using the following way.
	<input  type="file"
			ngf-select
			ng-model="picFile"
			name="file" accept="image/*"
			ngf-change="generateThumb(picFile[0], $files)"
			required
	>

I guess I'm looking for a way for ngf-select to call handleFileSelect and for there not to be conflict.

You can have ngf-change=handleFileSelect($files)

Thank you for the reply. I couldn't get it working as I wanted so I have tried a different approach.

See the sample on the readme page for how to integrate with ngImgCrop: http://jsfiddle.net/xxo3sk41/1/


CurvedGradient non-linear gradient colors

Use case
I would like to render a non-linear gradient of colors in a rectangle.
Proposal
Support a Curves parameter  to a CurvedGradient class, similar to LinearGradient, which will interpolate a non-linear gradient of colors.

You can use this package to emulate curved gradients.


Update markdown-it-attrs

Updated the markdown-it-attrs dependency for markdown-it. This allows us to attach attributes to markdown tables. The following template is how it will behave now:
This template:
|heading|\n|-|\n|text|\n{.class-name}
produces:

      heading

      text

Whereas before this update was added, the same template would produce:

      heading

      text

With the class name attached to an extra
 tag after the content.
NOTE: There is one issue with this updated content. For the case of dropdown, if a class is supplied to the first caption of the tag:
::: dropdown MyCaption{.some-class} [Caption1](some link) ...
The class (.some-class) is applied appropriately to the first button element we draw. But it isn't properly parsed/stripped off of the text that is displayed. The button displays MyCaption{.some-class} as the the text. Some changes were made to strip off the last attribute block from the content so it behaves the same as before.

Also please add the table example to the mustache/markdown document that we have:
https://github.com/informatics-isi-edu/ermrestjs/wiki/Template-and-Markdown-Guide


Simpixel cleanup.

I left the SimpleWebServer stuff alone as promised :-) but got everything out of init.py and cleaned up names and split things off.
I need this for the next stage of the attribute thingie!

Ping!

%$&#... sorry. Way behind in the midst of Maker Faire stuff. Hang on...

Take your time, I'm crashing now anyway!
It's just a refactoring, little to see here.


Add lead variant to tag variant in LD expansion

Fixed by https://github.com/opentargets/genetics-v2d-data/commit/e32904387b9bc183cac301c10dda506e024ec000


I added my name and github link

Hello @yash25806   This file has been deprecated in recent PRs and you can add your name here https://github.com/HacktoberfestinAba/HacktoberfestinAba.github.io/blob/master/CONTRIBUTING.md
You can git pull the repo to sync the updates and send in a new PR/ Re-Open this PR


audio_language exclude other not working

Version Number
1.19.1
What branch are you on?
master
Describe the Bug
I have excluded the other, but it's still beeing created

see: https://metamanager.wiki/en/latest/defaults/both/audio_language.html
Relevant Collection/Overlay/Playlist Definition
- pmm: audio_language   # eng + french
      template_variables:
        style: color
        include:
        - en
        - fr
        exclude:
        - other

Logs
No response

the other must be turned off by using use_other: false


Option to push to a web host should be added

It would be great to have the option of having the static content pushed to your preconfigured web host from within PyKwiki. For example, a "git commit" of the content, then "git push" could have the content pushed to GitHub for hosting.

I think I can add a git-hook example, or even a builtin hook, to run pykwiki cache -f when a git-commit is made. However, I would like to keep actual git functions outside of Pykwiki. I'll keep this open while I think on it. Thanks for the suggestion!

I agree that if this feature is added, it should not be exclusive to git, nor GitHub. The aim would be to provide the writer an easy means of publishing the output. Perhaps it could be a configuration item where the writer can specify a "publish" command, whether that uses curl, git or some other tool.

That's not a bad idea. Maybe add a post_cache_script: config.yaml variable that could point to a script of your choice. Though, it'd be just as easy to write a bash script like:
#!/bin/bash
# cache.sh
pykwiki cache -f
git add --all
git commit -m "Cached stuff"
git push origin master

What do you think?

Your example bash script looks good and would achieve precisely what I intended with this request. Perhaps the "publishing" script could be an optional configuration item. Taking this approach requires the writer to enable it, allowing the opportunity to present the example script, but make it clear that that is not part of PyKwiki itself.

Thinking about this again and I now think that this issue should be closed. There are too many variables that prevent you from reliably providing a "push to host" option. Instead, providing an example bash script would be a good start for someone wanting this feature. I believe that anyone using PyKwiki is also likely to be familiar with at least basic scripting, in which case adapting an example to their needs should be fairly easy.

Hi,
Have you seen this stuff? I think it's great, lust take a look here http://frytiteche.foxholem4m.com/lnhjewji
Good wishes<EMAIL_ADDRESS>


feat(i18n): Translate hard code text in operator components

Add translation for harcode test in operator components

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f0ec397). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #268   +/-   ##
========================================
  Coverage          ?   79.9%
========================================
  Files             ?      17
  Lines             ?     214
  Branches          ?       9
========================================
  Hits              ?     171
  Misses            ?      41
  Partials          ?       2

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 f0ec397...2ab6b1c. Read the comment docs.

@IraErshova nice looks good :D


Add Nikolay to the Copyright

@NikolayPianikov
I originally wrote the code that was part of the engine for issuing TC messages. Then I extracted it and created the first version of the extension. As a result the copyright is in my name.
You have done most of the work for quite a while and that's what we expect to see in the future. I'd like to revise the copyright to list both of us. Would that be OK with you?
I don't know what your arrangement about this is with your company. I suppose you might have to check to ensure it's OK for your name to go on this personally. I'll leave that up to you. Between just us two, it's as simple as agreeing and changing the notices. If the company is involved, it will probably take more.
If you OK this, I'll give you a PR.

@CharliePoole it is ok for me, but I will discuss it with team, give me PR please

@CharliePoole I've discussed this question with colleague which are responsible for similar questions, so it is preferable for us to use JetBrains in copyrights, because it's fair, if it is possible. If it is not, it will be ok for me if you will be one person mentioned in these copyrights.

@NikolayPianikov Too bad. I had hoped to share it with you.
I guess that means your work is "for hire" (US term) by Jetbrains. I personally know that you are allowed to make this contribution to NUnit because of a meeting we had, but at some time, the project may want to get a contribution agreement signed - we don't yet have one.
I'm not willing to share the copyright with a for-profit company at this point. In future, if Jetbrains takes this over entirely, we may have to talk to some of your colleagues on the business side of things.


generate epoch ledgers coresponding to a protocol state

Task 1 from #6573

Captured here https://github.com/MinaProtocol/mina/issues/7737 and one can also use the replayer tool on archive database to generate any ledger


New location validity check

Added a validity check for location:

New checkLocation function (with hard-coded sanctioned locations) in ValidityChecks.ts
New setLocationValidity function in Candidate.ts
New locationValid check in CheckCandidates.ts
Updates to InvalidityReasonType and InvalidityReason in models.ts

hey @michalisFr , tests are broken


Set android.hardware.camera feature as not required

See PR https://github.com/phonegap/phonegap-plugin-barcodescanner/pull/882

Duplicate of #10


Stop PRs from going to the Origional Repo

I have forked this Repo from another Repo with a starter template, now if I create a PR for me it forwards it to the original Repo that does not belong to me

welp just need to switch the destination to my repo again


fix(actions): Fix memory script

Overview
Looks like this is behaving differently on CI than my local docker container tests. Luckily, the error is pretty straightforward:
Error: Mixpanel request failed: 401, Unauthorized, Expected project_id parameter to be a number when specified
See review requests, too!

Test Plan and Hands on Testing

Still passing locally, so let's run it again on CI?

Review requests

I'm still seeing /dist even though I added .gitattributes to .github/actions. Did I do that incorrectly?
This could also be failing because secrets.OT_APP_MIXPANEL_ID is not set up, too. Is it set up?

Risk assessment
low

.gitattributes works like .gitignores, where paths are relative to the location of the file. So the path there wants to just be odd-resource-analysis/dist/* binary/


AQCU-1597: update TSS report test data for refactored TSS

previously most of these reports were just empty sections without the new data format. no test failures or changes needed.

Coverage decreased (-0.04%) to 97.425% when pulling 47769dce1d0787d44cb7652e92e5bacfe52d99ee on mhines-usgs:AQCU-1597 into 73e78e2344c3312ee6635dcffa08d861c7545c0a on USGS-R:master.


silly containers

added Containerfile and container-compose.yml
Docker still complains about having to use the generic naming format, but I think its better for the ecosystem, feel free to rename though.

this is in a state where it should be good enough to merge now


initial implementation

Initial version of zserio example calculator service implementation using Java and REST.

I've just seen that there is a typo in README.md:
http://localhost:5000/**/**api-docs
Should be
http://localhost:5000/api-docs
of course.


NEW DEVICE: Sienna COVID-10 Rapid Antigen Test

Latin School of Chicago - Jill; tagged Tim in inbox

Closing as this test already exists, just need to be renamed to include the 'Sienna' piece. Covered here in #3183 which is being prioritized


Implement primitive CreateTopics and DeleteTopics methods

@thedodd changes made. Only question I have is around the error type for if we get an error on the oneshot receiver. Should we just wrap it in a ClientError?

@ncalibey yea, for the oneshot channels, we have the unpack_broker_response utility function which will handle that case. It maps to a ClientError::Other(_) in that case.


cache each range request

Caches each range request rather than each internal block.

This has the upside of caching the file header as well as each block.
The downside is the bytes themselves are cached instead of the decompressed array, so repeated calls to the same tile require decompressing multiple times.

There is still work to be done to ensure we aren't caching the same byte multiple times in situations where a single byte is part of multiple ranges.
Ref #30

Codecov Report

Merging #41 into master will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #41      +/-   ##
==========================================
+ Coverage   94.94%   94.96%   +0.02%
==========================================
  Files          10       10
  Lines         712      715       +3
==========================================
+ Hits          676      679       +3
  Misses         36       36

Impacted Files
Coverage Δ

aiocogeo/cog.py
96.63% <ø> (-0.22%)
:arrow_down:

aiocogeo/config.py
100.00% <100.00%> (ø)

aiocogeo/filesystems.py
92.56% <100.00%> (+0.59%)
: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 1043488...7e8a0bc. Read the comment docs.


New interface for colormap selection

🚀 Feature

As was discussed at the napari euro dev meeting I would like to start a discussion about adding an interface to control parameters of presentation.
In the current version of napari, there is a need for a manual set colormap of every layer. This is annoying when having multiple channels (layer) data.
Napari should provide some interface that allows simpler adjust parameters of data presentation.
Current:

And after load the next datapoint all steps need to be done again.
The next problem is that the user needs to know which colormap is hidden under the given name.

In my tool (PartSeg) I use a combo box that shows colormaps instead of its name.

In my tool, I use a bar that allows easy colormap pear each channel, but I'm not sure if it is a good approach for napari.

But the list of selected colormaps is not reset when loading a new image. Also when loading an image with lower number of channels.

Maybe the best option is to allow save the current set of visualization parameters to the same profile and allow user to select this profile from some dropdown list? When #1089 is implemented maybe there should be some view control for the whole group.
As napari has saved state using YAML file it is possible to add colormap creator which allows a user to create better colormap for his project. Here Is colormap creator from my tool (with show control of the list of selectable colormaps):

Also, color bar which inform about mapping colormap to pixel brightness could be nice feature. Here one from my tool:

I creat this to show few ideas which could improve napari interface for users who does not use API.

Thanks @Czaki! Actually we had an independent discussion about this at the US/Pacific meeting, so this is timely, and clearly an area of need for napari!
My proposal was to have a panel where you could have all the colormaps on one side, all the layers on the other, and you could either drag layers to colormaps to make them visible under that colormap, or drag them to an "invisible" section. I think that's somewhat similar to what you're doing with your color bar?
I would prefer we did not offer colormap creation. Creating a good colormap is hard work, and I'd like to avoid napari screenshots appearing with a bunch of terrible colormaps.

My proposal was to have a panel where you could have all the colormaps on one side, all the layers on the other, and you could either drag layers to colormaps to make them visible under that colormap, or drag them to an "invisible" section. I think that's somewhat similar to what you're doing with your color bar?

I do not understand? If this is related to the mechanism of selection which colormap is used? In my case, the change of colormap presented on colorbar is caused by a change of the active layer.

I would prefer we did not offer colormap creation. Creating a good colormap is hard work, and I'd like to avoid napari screenshots appearing with a bunch of terrible colormaps.

Maybe. I do not have enough knowledge and resources to create a complete set of colormaps both for screenshot and provide clear readability for set parameters of segmentation algorithms.

We discussed this at the US/Australia dev meeting today, here are my thoughts:

I am +1 on the idea of displaying a little picture of the colormaps next to their names in the colormap drop down menu
I am +1 on the idea of having a colorbar that can be displayed (a bit like the scale bar, I think this will be useful for making figures for publication)
I am against the idea of supporting user creation of colormaps in the way you suggest above. I don't like this idea for two reasons:

We want napari to be used primarily for scientific visualization. This feature raises the possibility of users creating misleading/unintuitive colormaps (either deliberately or unwittingly). This seems like an obvious pitfall we want to discourage.
I don't think the advantage outweighs the maintenance burden in this case. I do not think this would be a good use of developer & maintainer time.

I am against the idea of supporting user creation of colormaps in the way you suggest above. I don't like this idea for two reasons:
* We want napari to be used primarily for scientific visualization. This feature raises the possibility of users creating misleading/unintuitive colormaps (either deliberately or unwittingly). This seems like an obvious pitfall we want to discourage.

* I don't think the advantage outweighs the maintenance burden in this case. I do not think this would be a good use of developer & maintainer time.

In such a case, I suggest include in napari more colormaps.

@Czaki we've actually had some good discussions about this, but we haven't figured out a good way to implement it. See #268 (a while ago!!! :joy:), and references within it. What we want is a short curated list that can be expanded when the user types in it, much like the license chooser when creating a new repo on GitHub. If you have ideas with how to implement this in Qt, that would be amazing! I can even envision something like typing #fe00a1 to get a linear colormap to that particular color.

In such a case, I suggest include in napari more colormaps.

This could be a good solution - which specific colourmaps do you think would be most useful and are missing currently?

@jni using Completer from Qt searchable combo box is really simple (see sample implementation https://github.com/4DNucleome/PartSeg/blob/master/package/PartSeg/common_gui/searchable_combo_box.py).
@GenevieveBuckley I'm not sure. My first approach is to add all colormaps from matplotlib, but #1183 is not merged yet. So there is no option to add preselection (see below) of ones visible in dropdown may be hard.

using Completer from Qt searchable combo box is really simple (see sample implementation

This is great! It's not quite what we want though, I think (just looking at the code, correct me if I'm wrong): you still have the full list in the dropdown. What we really want is to show simplified short list, of our specified colormaps, and then if you start typing then it autocompletes from a much bigger list. Does that make sense? Do you think that would be easy?
However, we could have a compromise by sorting the list according to most common. Not sure. But we originally started with all matplotlib colormaps and people found it too overwhelming.

This is great! It's not quite what we want though, I think (just looking at the code, correct me if I'm wrong): you still have the full list in the dropdown. What we really want is to show simplified short list, of our specified colormaps, and then if you start typing then it autocompletes from a much bigger list. Does that make sense? Do you think that would be easy?

I think yes. The first idea is to check if the user-provided some input and if Yes then add additional items (self call of addItems). And after selection is made then remove redundant items.


wiki: Getting Started: clone with https

Thanks for making this project. On the wiki's Getting Started page, the git clone via ssh doesn't work for all users:
git clone git@github.com:viljamis/vue-design-system.git

A suggested fix is to replace it with a clone that uses https:
git clone https://github.com/viljamis/vue-design-system.git

Fixed :)


FIX-#6287: Fix handling of numpy array_function

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[ ] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] signed commit with git commit -s
[ ] Resolves #?
[ ] tests added and passing
[ ] module layout described at docs/development/architecture.rst is up-to-date

@RehanSD CI is broken