Chain utils

Hmm.. in #279 i was specifically referring to making a version of LeastSquaresEstimator where .optimize returns an Estimator rather than a pipeline. This looks like tools to support that, but can you actually do that last step?

We talked about this offline. There are some namespace issues that make this hard right now, so I'll incorporate the new LeastSquaresEstimator as part of my "Flip the Switch" PR.

LGTM. Thanks @tomerk -  Merging this


Test suite failures

The test suite fails on my machine when trying to build the latest snapshot of OpenGM from master on an up-to-date Debian Testing / Unstable machine:
The following tests FAILED:
	 13 - test-graphicalmodeldecomposer (OTHER_FAULT)
	 46 - test-graphcut (OTHER_FAULT)

For test-graphicalmodeldecomposer:
      Start 13: test-graphicalmodeldecomposer
13/49 Test #13: test-graphicalmodeldecomposer ....***Exception: Other  0.18 sec
Decomposer test...
test-graphicalmodeldecomposer: /home/ghislain/debian/packages/build-area/opengm-2.3.5/src/unittest/test_graphicalmodeldecomposer.cxx:267: void GraphicalModelDecomposerTest<GM, SUBGM>::testGrid() [with GM = opengm::GraphicalModel<float, opengm::Adder>; SUBGM = opengm::GraphicalModel<float, opengm::Adder, opengm::meta::TypeList<opengm::ModelViewFunction<opengm::GraphicalModel<float, opengm::Adder>, marray::Marray<float> >, opengm::meta::TypeList<opengm::ConstantFunction<float>, opengm::meta::ListEnd> > >]: Assertion `decomposition.numberOfSubVariables(i)==16' failed.
    -- 2nd order grid with unary factors...****

For test-graphcut:
      Start 46: test-graphcut
46/49 Test #46: test-graphcut ....................***Exception: Other  9.29 sec
Test Graphcut ...
  * Test Min-Sum with BOOST-Push-Relabel
    - 2nd order grid model with 16 variables and 2 states (functionType = potts with unary ) *value = 5.76712  ,  bound = -inf
*value = 5.76712  ,  bound = -inf
*value = 6.42818  ,  bound = -inf
*value = 3.73098  ,  bound = -inf
*value = 2.72753  ,  bound = -inf
*value = 1.76812  ,  bound = -inf
*value = 13.8766  ,  bound = -inf
*value = 10.5089  ,  bound = -inf
*value = 9.4897  ,  bound = -inf
*value = 4.44939  ,  bound = -inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = potts with unary ) *value = 3.244  ,  bound = -inf
*value = 3.244  ,  bound = -inf
*value = 3.61585  ,  bound = -inf
*value = 2.09868  ,  bound = -inf
*value = 1.53424  ,  bound = -inf
*value = 0.994567  ,  bound = -inf
*value = 7.80558  ,  bound = -inf
*value = 5.91128  ,  bound = -inf
*value = 5.33796  ,  bound = -inf
*value = 2.50278  ,  bound = -inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = potts without unary ) *value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
 OPTIMAL!
    - 2nd order star model with 5 variables and 2 states (functionType = potts with unary ) *value = 6.44692  ,  bound = -inf
*value = 6.44692  ,  bound = -inf
*value = 6.67645  ,  bound = -inf
*value = 5.73992  ,  bound = -inf
*value = 5.3915  ,  bound = -inf
*value = 5.05837  ,  bound = -inf
*value = 9.2627  ,  bound = -inf
*value = 8.09338  ,  bound = -inf
*value = 7.73948  ,  bound = -inf
*value = 5.98937  ,  bound = -inf
 OPTIMAL!
    - 2nd order full model with 5 variables and 2 states (functionType = potts with unary ) *value = 6.44692  ,  bound = -inf
*value = 6.44692  ,  bound = -inf
*value = 6.67645  ,  bound = -inf
*value = 5.73992  ,  bound = -inf
*value = 5.3915  ,  bound = -inf
*value = 5.05837  ,  bound = -inf
*value = 9.2627  ,  bound = -inf
*value = 8.09338  ,  bound = -inf
*value = 7.73948  ,  bound = -inf
*value = 5.98937  ,  bound = -inf
 OPTIMAL!
  * Test Min-Sum with Interegr-BOOST-Push-Relabel
    - 2nd order grid model with 16 variables and 2 states (functionType = potts with unary ) *value = 5.76712  ,  bound = -inf
*value = 5.76712  ,  bound = -inf
*value = 6.42818  ,  bound = -inf
*value = 3.73098  ,  bound = -inf
*value = 2.72753  ,  bound = -inf
*value = 1.76812  ,  bound = -inf
*value = 13.8766  ,  bound = -inf
*value = 10.5089  ,  bound = -inf
*value = 9.4897  ,  bound = -inf
*value = 4.44939  ,  bound = -inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = potts with unary ) *value = 3.244  ,  bound = -inf
*value = 3.244  ,  bound = -inf
*value = 3.61585  ,  bound = -inf
*value = 2.09868  ,  bound = -inf
*value = 1.53424  ,  bound = -inf
*value = 0.994567  ,  bound = -inf
*value = 7.80558  ,  bound = -inf
*value = 5.91128  ,  bound = -inf
*value = 5.33796  ,  bound = -inf
*value = 2.50278  ,  bound = -inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = potts without unary ) *value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
 OPTIMAL!
    - 2nd order star model with 5 variables and 2 states (functionType = potts with unary ) *value = 6.44692  ,  bound = -inf
*value = 6.44692  ,  bound = -inf
*value = 6.67645  ,  bound = -inf
*value = 5.73992  ,  bound = -inf
*value = 5.3915  ,  bound = -inf
*value = 5.05837  ,  bound = -inf
*value = 9.2627  ,  bound = -inf
*value = 8.09338  ,  bound = -inf
*value = 7.73948  ,  bound = -inf
*value = 5.98937  ,  bound = -inf
 OPTIMAL!
    - 2nd order full model with 5 variables and 2 states (functionType = potts with unary ) *value = 6.44692  ,  bound = -inf
*value = 6.44692  ,  bound = -inf
*value = 6.67645  ,  bound = -inf
*value = 5.73992  ,  bound = -inf
*value = 5.3915  ,  bound = -inf
*value = 5.05837  ,  bound = -inf
*value = 9.2627  ,  bound = -inf
*value = 8.09338  ,  bound = -inf
*value = 7.73948  ,  bound = -inf
*value = 5.98937  ,  bound = -inf
 OPTIMAL!
  * Test Min-Sum with BOOST-Edmonds-Karp
    - 2nd order grid model with 16 variables and 2 states (functionType = potts with unary ) *value = 5.76712  ,  bound = -inf
*value = 5.76712  ,  bound = -inf
*value = 6.42818  ,  bound = -inf
*value = 3.73098  ,  bound = -inf
*value = 2.72753  ,  bound = -inf
*value = 1.76812  ,  bound = -inf
*value = 13.8766  ,  bound = -inf
*value = 10.5089  ,  bound = -inf
*value = 9.4897  ,  bound = -inf
*value = 4.44939  ,  bound = -inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = potts with unary ) *value = 3.244  ,  bound = -inf
*value = 3.244  ,  bound = -inf
*value = 3.61585  ,  bound = -inf
*value = 2.09868  ,  bound = -inf
*value = 1.53424  ,  bound = -inf
*value = 0.994567  ,  bound = -inf
*value = 7.80558  ,  bound = -inf
*value = 5.91128  ,  bound = -inf
*value = 5.33796  ,  bound = -inf
*value = 2.50278  ,  bound = -inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = potts without unary ) *value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
 OPTIMAL!
    - 2nd order star model with 5 variables and 2 states (functionType = potts with unary ) *value = 6.44692  ,  bound = -inf
*value = 6.44692  ,  bound = -inf
*value = 6.67645  ,  bound = -inf
*value = 5.73992  ,  bound = -inf
*value = 5.3915  ,  bound = -inf
*value = 5.05837  ,  bound = -inf
*value = 9.2627  ,  bound = -inf
*value = 8.09338  ,  bound = -inf
*value = 7.73948  ,  bound = -inf
*value = 5.98937  ,  bound = -inf
 OPTIMAL!
    - 2nd order full model with 5 variables and 2 states (functionType = potts with unary ) *value = 6.44692  ,  bound = -inf
*value = 6.44692  ,  bound = -inf
*value = 6.67645  ,  bound = -inf
*value = 5.73992  ,  bound = -inf
*value = 5.3915  ,  bound = -inf
*value = 5.05837  ,  bound = -inf
*value = 9.2627  ,  bound = -inf
*value = 8.09338  ,  bound = -inf
*value = 7.73948  ,  bound = -inf
*value = 5.98937  ,  bound = -inf
 OPTIMAL!
  * Test Min-Sum with BOOST-Kolmogorov
    - 2nd order grid model with 16 variables and 2 states (functionType = potts with unary ) *value = 5.76712  ,  bound = -inf
*value = 5.76712  ,  bound = -inf
*value = 6.42818  ,  bound = -inf
*value = 3.73098  ,  bound = -inf
*value = 2.72753  ,  bound = -inf
*value = 1.76812  ,  bound = -inf
*value = 13.8766  ,  bound = -inf
*value = 10.5089  ,  bound = -inf
*value = 9.4897  ,  bound = -inf
*value = 4.44939  ,  bound = -inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = potts with unary ) *value = 3.244  ,  bound = -inf
*value = 3.244  ,  bound = -inf
*value = 3.61585  ,  bound = -inf
*value = 2.09868  ,  bound = -inf
*value = 1.53424  ,  bound = -inf
*value = 0.994567  ,  bound = -inf
*value = 7.80558  ,  bound = -inf
*value = 5.91128  ,  bound = -inf
*value = 5.33796  ,  bound = -inf
*value = 2.50278  ,  bound = -inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = potts without unary ) *value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
*value = 0  ,  bound = -inf
 OPTIMAL!
    - 2nd order star model with 5 variables and 2 states (functionType = potts with unary ) *value = 6.44692  ,  bound = -inf
*value = 6.44692  ,  bound = -inf
*value = 6.67645  ,  bound = -inf
*value = 5.73992  ,  bound = -inf
*value = 5.3915  ,  bound = -inf
*value = 5.05837  ,  bound = -inf
*value = 9.2627  ,  bound = -inf
*value = 8.09338  ,  bound = -inf
*value = 7.73948  ,  bound = -inf
*value = 5.98937  ,  bound = -inf
 OPTIMAL!
    - 2nd order full model with 5 variables and 2 states (functionType = potts with unary ) *value = 6.44692  ,  bound = -inf
*value = 6.44692  ,  bound = -inf
*value = 6.67645  ,  bound = -inf
*value = 5.73992  ,  bound = -inf
*value = 5.3915  ,  bound = -inf
*value = 5.05837  ,  bound = -inf
*value = 9.2627  ,  bound = -inf
*value = 8.09338  ,  bound = -inf
*value = 7.73948  ,  bound = -inf
*value = 5.98937  ,  bound = -inf
 OPTIMAL!
  * Test Max-Sum with BOOST-Push-Relabel
    - 2nd order grid model with 16 variables and 2 states (functionType = negative potts with unary ) *value = 15.0048  ,  bound = inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = negative potts with unary ) *value = 8.44018  ,  bound = inf
*value = 8.44018  ,  bound = inf
*value = 7.11042  ,  bound = inf
 OPTIMAL!
    - 2nd order grid model with 9 variables and 2 states (functionType = negative potts without unary ) *value = 0  ,  bound = inf
*value = 0  ,  bound = inf
*value = 0  ,  bound = inf
 OPTIMAL!
    - 2nd order star model with 5 variables and 2 states (functionType = negative potts with unary ) *value = 9.65443  ,  bound = inf
*value = 9.65443  ,  bound = inf
*value = 8.8336  ,  bound = inf
 OPTIMAL!
test-graphcut: /home/ghislain/debian/packages/build-area/opengm-2.3.5/include/opengm/inference/auxiliary/minstcutboost.hxx:93: void opengm::MinSTCutBoost<NType, VType, mfalg>::addEdge(opengm::MinSTCutBoost<NType, VType, mfalg>::node_type, opengm::MinSTCutBoost<NType, VType, mfalg>::node_type, opengm::MinSTCutBoost<NType, VType, mfalg>::ValueType) [with NType = long unsigned int; VType = float; opengm::BoostMaxFlowAlgorithm mfalg = (opengm::BoostMaxFlowAlgorithm)0u; opengm::MinSTCutBoost<NType, VType, mfalg>::node_type = long unsigned int; opengm::MinSTCutBoost<NType, VType, mfalg>::ValueType = float]: Assertion `cost >= 0' failed.
    - 2nd order full model with 5 variables and 2 states (functionType = negative potts with unary ) *

The following build options were used:
cmake .. -DCMAKE_INSTALL_PREFIX=/usr -DCMAKE_VERBOSE_MAKEFILE=ON -DCMAKE_BUILD_TYPE=None -DCMAKE_INSTALL_SYSCONFDIR=/etc -DCMAKE_INSTALL_LOCALSTATEDIR=/var -DBUILD_EXAMPLES=OFF -DBUILD_TUTORIALS=OFF -DBUILD_BUILD_COMMANDLINE=ON -DWITH_BOOST=ON -DWITH_HDF5=ON -DWITH_OPENMP=ON -DWITH_VIGRA=ON -DBUILD_CONVERTER=ON -DBUILD_PYTHON_WRAPPER=ON -DBUILD_TESTING=ON -DBUILD_DOCS=OFF -DBUILD_PYTHON_DOCS=OFF

The problem with boost-graph is known. The check in the assert for c>=0.
This should be either c>=0.000001 or one should not use floating-types ;-)
The error shows up depending of the implementation of your random number generator.
The problem with "graphicalmodeldecomposer" seemed to be new. I will have a look.
Joerg

The problem with boost-graph is known. The check in the assert for c>=0.

Glad you know where it comes from.

This should be either c>=0.000001 or one should not use floating-types ;-)
The error shows up depending of the implementation of your random number generator.

I am confused now. What's the proper fix for this. Do you have a patch?

The problem with "graphicalmodeldecomposer" seemed to be new. I will have a look.

Thanks.
FYI, I cannot submit a package without a working test suite. That's why I am insisting on this issue.

Give me 5 more minutes its nearly fixed ;-)

Damn, you're fast ;-)

Should be fixed.
The decomposer, was a bug in the test. Furthermore, U found a serious problem in our test env, I have to fix. Thanks for that.
For the GraphCut I deactivate the asserts in the boost graph part - that's the best we can do at the moment. Any way boost graph is not really fast and should be the first choice.

Great. I'll re-run the packaging process and report back whether this is fixed on my end.

Works for me after #387 is applied.

Fixed with merge of #387.


AWS lambda setup error in .NET Core 3.1

Hello I am trying to Configure AWS lambda in .NET core 3.1 project but when I am installing Amazon.Lambda.Tools from Nuget I am getting error that  Amazon.Lambda.Tools 4.0.1 doesn't support .netcoreapp 3.1. Please help me fixing this issue. I can't downgrade my project to .net 2.1

Hi @Ryan881,
Good morning.
Could you please try executing the following command from .NET CLI to install the latest version of the tool:
dotnet tool install --global Amazon.Lambda.Tools --version 4.1.0

Also, I don't see version 4.0.1 on https://www.nuget.org/packages/Amazon.Lambda.Tools/.
Thanks,
Ashish


How to modify the same row repeatedly when using NewStreamWriter

When using NewStreamWriter,  I change the value of cell A1 and manipulate the next row of data, how can I come back to modify the value of cell C1?

Thanks for your issue. As the documentation of the StreamWriter function says,  you need to ensure that the order of row numbers is ascending. The stream write function is used to generate a new worksheet with a mound of data, and it doesn't support modifying rows which already been written. Please prepare all cells in a row before writing them into the worksheet.

I've closed this issue, if you have any questions, please let me know to reopen this anytime.


New format for flow_types.csv

Changed flow_types.csv to a new format to incorporate sources for each data entry. Added a bunch of new materials with densities and heating values.

There's one more thing I noticed: some of the entries you looked up for energycontent are in fact energy content per volume, not per mass.
For instance, let's compare the two energy carriers nh3 and ch4 in the updated file flow_types.csv:
nh3,default_unit,t,
nh3,energycontent_LHV,18.90 MJ/kg,H2tools2023
nh3,energycontent_HHV,22.50 MJ/kg,H2tools2023
nh3,density_norm,0.71606  kg/m³,Lemmon2023
nh3,density_std,0.77155  kg/m³,Lemmon2023
ch4,name,Methane,
ch4,default_unit,MWh;LHV,
ch4,energycontent_LHV,962 Btu/ft³,GREET2008
ch4,energycontent_HHV,1068 Btu/ft³,GREET2008
ch4,density_norm,0.66816 kg/m³,Lemmon2023
ch4,density_std,0.71746 kg/m³,Lemmon2023

For nh3, it will work just fine: the unit is MJ/kg, i.e. [energy]/[mass]. The ch4, it will not work out-of-the-box. The unit is provided is Btu/ft³, i.e. [energy]/[volume].
You could compute the correct variable by dividing it by the density that you also have further below before passing the parameters on to the rest of POSTED. But you will then face the problem that you may not know whether the density is defined w.r.t. STP or NTP.
In other words, which is the correct LHV of methane?
>>> from posted.units.units import ureg
>>> (ureg('962 Btu/ft³') / ureg('0.66816 kg/m³')).to('MJ/kg')
<Quantity(53.6445026, 'megajoule / kilogram')>
>>> (ureg('962 Btu/ft³') / ureg('0.71746 kg/m³')).to('MJ/kg')
<Quantity(49.9583404, 'megajoule / kilogram')>

Options to resolve this:

Simply find a source that reports the data relative to mass.
Let the code handle it in the background. This will require you to add a new attribute (e.g. energydensity with dimensionality [energy]/[volume]). You will then need to add both variants to make it unambiguous, i.e. energydensity_LHV_std.
Convert that number into the expected dimensions outside of the code and simply add the number with the existing attribute energycontent. You could then add a comment field explaining this.

Any of the options seems fine to me. The last one may be the one that requires the least effort.

There's one more thing I noticed: some of the entries you looked up for energycontent are in fact energy content per volume, not per mass.
For instance, let's compare the two energy carriers nh3 and ch4 in the updated file flow_types.csv:
nh3,default_unit,t,
nh3,energycontent_LHV,18.90 MJ/kg,H2tools2023
nh3,energycontent_HHV,22.50 MJ/kg,H2tools2023
nh3,density_norm,0.71606  kg/m³,Lemmon2023
nh3,density_std,0.77155  kg/m³,Lemmon2023
ch4,name,Methane,
ch4,default_unit,MWh;LHV,
ch4,energycontent_LHV,962 Btu/ft³,GREET2008
ch4,energycontent_HHV,1068 Btu/ft³,GREET2008
ch4,density_norm,0.66816 kg/m³,Lemmon2023
ch4,density_std,0.71746 kg/m³,Lemmon2023

For nh3, it will work just fine: the unit is MJ/kg, i.e. [energy]/[mass]. The ch4, it will not work out-of-the-box. The unit is provided is Btu/ft³, i.e. [energy]/[volume].
You could compute the correct variable by dividing it by the density that you also have further below before passing the parameters on to the rest of POSTED. But you will then face the problem that you may not know whether the density is defined w.r.t. STP or NTP.
In other words, which is the correct LHV of methane?
>>> from posted.units.units import ureg
>>> (ureg('962 Btu/ft³') / ureg('0.66816 kg/m³')).to('MJ/kg')
<Quantity(53.6445026, 'megajoule / kilogram')>
>>> (ureg('962 Btu/ft³') / ureg('0.71746 kg/m³')).to('MJ/kg')
<Quantity(49.9583404, 'megajoule / kilogram')>

Options to resolve this:
* Simply find a source that reports the data relative to mass.

* Let the code handle it in the background. This will require you to add a new attribute (e.g. `energydensity` with dimensionality `[energy]/[volume]`). You will then need to add both variants to make it unambiguous, i.e. `energydensity_LHV_std`.

* Convert that number into the expected dimensions outside of the code and simply add the number with the existing attribute `energycontent`. You could then add a comment field explaining this.

Any of the options seems fine to me. The last one may be the one that requires the least effort.

The source for all these values (GREET) also gives a density for all materials. For the gaseous fuels it is even specified, that the Energycontent/volume is given at NTP. I therefore propose to just juse these densities to convert the LHV and HHV to MJ/kg and use the converted values for our flow_types.csv, i.e. your proposed solution 3.


Way for Collectives to apply to c6 host besides Github star verification

We need a workflow for Collectives who want to apply to c6 but need to be independently verified because they are new projects by proven maintainers or have their codebase elsewhere. We're getting a fair number of applications to c-corp host because people are trying to route around this, as well as direct requests. I think there should be some sort of "other" option where they can put in verifying evidence.

@piamancini told me to tell user they can select 'other' and then select Open Source Collective, if they can't go through the stars verification. However, I just tried to do that myself and Open Source Collective does not come up as an option, even if you put 'open source' as a tag.

I finally figured out how they can do it. They have to select other, then bypass Open Collective Host and click "view all hosts", where they can actually select Open Source Collective. Very convoluted.


Resource page: edit

Platform admin - manage resources:
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=3449%3A99436
Individual resource:
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=3449%3A99706
This is essentially the resource page, but with the platform admin toolbar and with extra controls (edit, delete, remove from collection).

Platform admin - manage resources:
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=3449%3A99436
Individual resource:
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=3449%3A99706
This is essentially the resource page, but with the platform admin toolbar and with extra controls (edit, delete, remove from collection).
Add a resource to a collection: https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=3462%3A99923
https://www.figma.com/file/AXufBnFAVHvGdsxnQMVhNF/The-Accessibility-Exchange-Wireframes?node-id=3462%3A100130


Bump build/release workflow to JDK 17

See https://github.com/quarkiverse/quarkiverse-devops/issues/218

Closing in favor of https://github.com/quarkiverse/quarkus-microprofile/pull/137.


[FP]: org.apache.ftpserver:ftpserver-core:jar is matched with apache mina

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:apache:mina:1.2.0:::::::*  (Confidence:Low)
CVE
CVE-2021-41973
ODC Integration
{"label"=>"Maven Plugin"}
ODC Version
10.0.4
Description
After https://github.com/jeremylong/DependencyCheck/issues/7026#issuecomment-2408540665 was resolved, the package ftplet-api is now matched with apache mina. Interestingly, ftplet-core has a dependency to apache mina, but in another (patched) version: 2.1.6 which means I get the following false positive:
ftplet-api-1.2.0.jar<EMAIL_ADDRESS>cpe:2.3:a:apache:mina:1.2.0:::::::*) : CVE-2021-41973

As FTPServer is an independently managed subproject under the umbrella of the  Apache MINA project it would indeed get a CPE separate from the MINA product assigned should a CVE get reported for it in the future, but the suppression suggested by the bot should not be used as it is very likely that NVD would opt for product mina-ftpserver (after the github repo) and the CPE suppression quoted would suppress it due to comparing using case-insensitive prefix matching.
<suppress base="true">
   <notes><![CDATA[
   FP per issue #7043
   ]]></notes>
   <packageUrl<EMAIL_ADDRESS>   <cpe regex="true">^cpe:/a:apache:mina:.*</cpe>
</suppress>

Should do the right suppression IIRC. I'll doublecheck that for proper working and will add it manually to the hosted suppressions (and update this comment should I discover that the suppression rule should be further adapted to be correct).


Error when running playground

I pulled the latest from the repo and did the following:

npm install
gulp build:run:playground

I consistently get this error in the console:
TypeError: a is not a constructor
...ainer=$("#container"),this.hostControls=new a($("#options")),this.hostControls.s...
or this error
Uncaught TypeError: HostControls is not a function
I am building this on a Mac.  When I build this on Windows it works fine.

It was fixed in this commit https://github.com/Microsoft/PowerBI-visuals/commit/15726a02d40c116da610dd92790f970a65b9272a
Please pull the latest sources or just rename file HostControls.ts to hostControls.ts
and run again:

gulp build:run:playground

Thanks, I appreciate it. I figured it out last night.

Great!


Sample program running on server:
package main

import (
    "fmt"
    "net/http"
)

type proxy struct {}

func (s *proxy) ServeHTTP(w http.ResponseWriter, req *http.Request) {
    fmt.Println("Requested URL:", req.URL.String())
    w.Write([]byte("OK"))
}

func main() {
    p := &proxy{}
    err := http.ListenAndServeTLS(":443", "r.fullchain.pem", "r.privkey.pem", p)
    if err != nil {
        panic("ListenAndServe")
    }
}

Then I using it as proxy with CURL:
curl -vvv --proxy-http2 -x https://example.server:443 http:///remote.host/path
* Host example.server:443 was resolved.
* IPv6: (none)
* IPv4: <IP_ADDRESS>
*   Trying <IP_ADDRESS>:443...
* Connected to example.server (<IP_ADDRESS>) port 443
* ALPN: curl offers h2,http/1.1
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
*  CAfile: /etc/ssl/certs/ca-certificates.crt
*  CApath: /etc/ssl/certs
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
* TLSv1.3 (IN), TLS handshake, Certificate (11):
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
* TLSv1.3 (IN), TLS handshake, Finished (20):
* TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.3 (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / TLS_CHACHA20_POLY1305_SHA256 / X25519 / RSASSA-PSS
* ALPN: server accepted h2
* Proxy certificate:
*  subject: CN=example.server
*  start date: May 16 03:00:12 2024 GMT
*  expire date: Aug 14 03:00:11 2024 GMT
*  subjectAltName: host "example.server" matched cert's "example.server"
*  issuer: C=US; O=Let's Encrypt; CN=R3
*  SSL certificate verify ok.
*   Certificate level 0: Public key type RSA (2048/112 Bits/secBits), signed using sha256WithRSAEncryption
*   Certificate level 1: Public key type RSA (2048/112 Bits/secBits), signed using sha256WithRSAEncryption
*   Certificate level 2: Public key type RSA (4096/152 Bits/secBits), signed using sha256WithRSAEncryption
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* [HTTP/2] [1] OPENED stream for http://remote.host/path
* [HTTP/2] [1] [:method: GET]
* [HTTP/2] [1] [:scheme: http]
* [HTTP/2] [1] [:authority: remote.host]
* [HTTP/2] [1] [:path: /path]
* [HTTP/2] [1] [user-agent: curl/8.5.0]
* [HTTP/2] [1] [accept: */*]
> GET http://remote.host/path HTTP/2
> Host: remote.host
> User-Agent: curl/8.5.0
> Accept: */*
> Proxy-Connection: Keep-Alive
>
< HTTP/2 200
< content-type: text/plain; charset=utf-8
< content-length: 2
< date: Wed, 10 Jul 2024 04:26:39 GMT
<
* Connection #0 to host example.server left intact

What did you see happen?
Example program prints:
Requested URL: /path

What did you expect to see?
Example program expected to print:
Requested URL: http://remote.host/path

Related Issues and Documentation

net/http: h2 does not support other net.Conn than *tls.Conn  #55076
net/http: ConfigureTransport does not work for Transport with custom DialTLS #41236
net/http: Get request hangs until timeout, but responds immediately over curl #61823 (closed)
x/net/http2: support http2 proxy connections #26479
net/http/httptest: NewTLSServer doesn't support HTTP/2 requests #34939 (closed)
x/net/http2: `http.Server.Serve` doesn't serve http2 traffic #14374 (closed)
No way to retrieve the proxy host name #49007 (closed)
net.http: http client with dailContext send a https request different from client without dailContext #61136 (closed)
x/net/http2: HTTP/2 slower than HTTP/1 #36440 (closed)
crypto/tls: cannot enforce "h2" with ALPN #59734

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

looking at the curl logs:
* [HTTP/2] [1] OPENED stream for http://remote.host/path
* [HTTP/2] [1] [:method: GET]
* [HTTP/2] [1] [:scheme: http]
* [HTTP/2] [1] [:authority: remote.host]
* [HTTP/2] [1] [:path: /path]
* [HTTP/2] [1] [user-agent: curl/8.5.0]
* [HTTP/2] [1] [accept: */*]

and http2 debug logs
2024/07/10 15:48:25 http2: decoded hpack field header field ":method" = "GET"
2024/07/10 15:48:25 http2: decoded hpack field header field ":scheme" = "http"
2024/07/10 15:48:25 http2: decoded hpack field header field ":authority" = "remote.host"
2024/07/10 15:48:25 http2: decoded hpack field header field ":path" = "/path"
2024/07/10 15:48:25 http2: decoded hpack field header field "user-agent" = "curl/8.8.0"
2024/07/10 15:48:25 http2: decoded hpack field header field "accept" = "*/*"

I believe this is working as intended.

@seankhliao , how can I extract these values in request handler?


Hillsborough ACS variables correlation analysis

Merged! @snnehete - can you add a description to this PR + a few labels. Looks like you have a good description at the bottom of the notebook.


Completing French translation with "Goal"

Translated "Goal" to "Objectif" in French

Thanks for your pull request.
I added some new predefined clues for the next version. Would you mind adding those french translations as well and change the merge request to the #17 branch.

was added in #17


legend-mouseover-inactive states are being reset after chart update

Bug report
Codepen
https://codepen.io/ovipo/pen/QWNLYvd
(based on realtime demo from apexcharts website)
Explanation

What is the behavior you expect?

When I hover the series name in the legend the other lines should fade out and stay like this until the mouse is moved.

What is happening instead?

The other series reset to default view after chart update

What error message are you getting?

No error message

The same happens when I click on one item in the legend. The corresponding line first disappears, but comes back on update.

I am having this issue as well - most importantly, when i hide some series, they come back the next time updateSeries is called


linking image picker causes error when build

Hello,
I created a project
react-native init test
After this run this command
npm install react-native-image-picker@latest --save
react-native link

Here is the error...

C:\Users\ozer\Desktop\rn\test33\node_modules\react-native-image-picker\android\src\main\java\com\imagepicker\ImagePickerPackage.java:34: error: method does not override or implement a method from a supertype
@Override
^
:react-native-image-picker:compileReleaseJavaWithJavac FAILED
1 error
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':react-native-image-picker:compileReleaseJavaWithJavac'.

Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Also guys, I get this error all packages just not to this..

react-native-cli: 2.0.1
react-native: 0.47.1

I do everything, tried all solution ways. But still same errors... I can't link projects, always gives error.

react-native 0.47 deprecated createJSModules with https://github.com/facebook/react-native/issues/15232
many PR is waiting to be merged to solve this #661 #657 #653 #649
fastest way to fix this issue is to go to the __PROJECT_ROOT__/node_modules/react-native-image-picker/android/src/main/java/com/imagepicker/ImagePickerPackage.java
and remove line 34 - 37
@Override
public List<Class<? extends JavaScriptModule>> createJSModules() {
    return Collections.emptyList();
}

Fixed in 0.26.4


brings back removed truss file

:rocket: What

Adds back removed file

:computer: How

:microscope: Testing

yup - searched the org for it and found it @squidarth  - left a comment


Disable usage of bare instructions in AOT

The compressed hello world application regressed. One of the reasons is less efficient compression of the AOT output (instructions and isolate snapshot)
This change temporarily disables usage of bare instructions.
Issue https://github.com/flutter/flutter/issues/27250

With test updates: https://github.com/flutter/flutter/pull/27410

It seems like this was addressed elsewhere, closing for now.


Editor 6 OS X — Tab Key

Please allow the option to insert tab instead of spaces. This is especially useful when including code samples and editing CSS Files.

+1

In raw mode or document mode, currently:

In raw mode: we include 2 spaces when pressing Tab (I can change it to detect indentation)
In document mode: we detect indentation when pressing Tab in a code block (we default to 4 spaces).

I'll add a configuration for default indentation when pressing tab in a code block.

Fixed in 6.0.3


Copying text into BPA keeps formatting

Describe the bug
Copying text from the Advanced Scripting window into the Rule Expression Editor of the BPA keeps the formatting from the Advanced Scripting window (font size, font, and color).
To Reproduce
Steps to reproduce the behavior:

Write code in the Advanced Scripting window
Copy some of the code
Open the BPA manager
Edit a rule
Ctrl-V to paste the code in the Rule Expression Editor

Expected behavior
It should reformat the text to be the standard format of the text in the Rule Expression Editor
Application specifics

Tabular Editor version: 2.13.0

Fixed in Tabular Editor 2.13.2


New input parsing code to allow more flexibility

As per @joaomcteixeira's suggestion, here's a tentative proposal of a new input parser to allow for any options and validation of values on those options. We should benchmark this against a similar version using argparse but I'm almost sure this will still be faster.
Partially addresses #9
Pinging @joaomcteixeira @mtrellet @Adrimel and @amjjbonvin for comments.

Just a detail, in the usage example you are using "resi" as keyword but the script needs "resid".

If it does what argparse does and faster then fine with me!
I guess the argparse method is slightly more elegant and easy to tune but quick benchmarks I've performed yesterday did show that it is slower than a pure parsing of the arguments (0.6s difference over 100 scripts execution - only testing the parsing and exiting the script right after). I can still test further argparse if you're interested in the comparison.

Addressed [1]. Will merge! We will take care of [2] later.


NumDiff abstraction and unit-test - [merged]

In GitLab by @cmastalli on Dec 7, 2018, 20:57
Merges topic/unittest -> devel
In this PR I have included the unit-test for checking analytical differentiation of spring-mass and forward-dynamics systems. For that, I have to work in the NumDiff abstraction for whatever system. It also allows the user to start developing a system without the needed of adding the system derivatives. All this is possible because of the new API for dynamics and cost functions. In few works, the state and control vectors aren't included in the dynamics data.
This PR tackles the task #21 and partially the task #36.

In GitLab by @cmastalli on Dec 7, 2018, 20:57
mentioned in commit e582e6da1b7a963c68934868d55f457d8a144bc2

In GitLab by @cmastalli on Dec 7, 2018, 20:57
merged

In GitLab by @cmastalli on Dec 11, 2018, 18:13
changed title from {-Topic/unit-}test to {+NumDiff abstraction and unit-+}test

In GitLab by @cmastalli on Dec 11, 2018, 18:13
changed the description


Bug: Call to undefined method Composer\InstalledVersions::getAllRawData() in Autoloader.php

PHP Fatal error:  Uncaught Error: Call to undefined method Composer\InstalledVersions::getAllRawData() in .../vendor/codeigniter4/framework/system/Autoloader/Autoloader.php:376

Ref #7101
See https://forum.codeigniter.com/showthread.php?tid=86123
How to Reproduce:
$ composer self-update 2.0.13
$ composer create-project codeigniter4/appstarter ci4
$ cd ci4
$ php spark serve

This is not related in v4.3.0.
getAllRawData() was introduced in Composer 2.0.14.
https://getcomposer.org/changelog/2.0.14
https://github.com/composer/composer/pull/9816/commits/f5e6cc89cd8d45c99cff77ee80d0f324ada45686

Disaster in every sense.

This is related to v4.3.0.
InstalledVersions::getAllRawData() has been used in https://github.com/codeigniter4/CodeIgniter4/commit/8c99deddd4fd53e89b6f4a574b16cd5a0433be18#diff-907319ab25c4e05e576a1754147d0b980076b79b4b48f92d99314e7366c0f816R373


Nextjs refactor: Remove the zoom controls from the mini map on the mobile location details page

We don't need these controls in this view, as they are obscured by the 'View Street View' button.

Confirmed fixed in https://github.com/streetlives/yourpeer.nyc-nextjs/pull/18


Improve gce bootstrapping in various ways.

This PR:

Add GCE metadata containerd-env. We can pass environment variables to cluster/configure.sh via containerd-env now.
Parse kube-env. We can get kubernetes environment variables from kube-env.
Consolidate test/configure.sh and cluster/gce/configure.sh.
Add preload support.
Add kubernetes environment variable NETWORK_POLICY_PROVIDER support.

Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

/test pull-cri-containerd-node-e2e

/test pull-cri-containerd-node-e2e

/test pull-cri-containerd-node-build

/test pull-cri-containerd-build

@yujuhong Addressed comments.

/lgtm

Just squash the "Address comments" commit, reapply the lgtm label.


Wallet allows creation of account names less than 5 characters

Problem
Wallet documentation
https://docs.nearprotocol.com/docs/local-setup/create-account
and wallet app
https://wallet.nearprotocol.com
both indicate a minimum account length of 5 characters.  I was able to create accounts with length 2, 3, and 4 characters.
Expected Behavior
Account names less than 5 characters should be rejected.
Steps to reproduce

Go to https://wallet.nearprotocol.com
Create an account with 2, 3, or 4 characters.

Thanks @dantrevino! Moving this issue to the docs repo, as we currently do allow shorter names in the wallet intentionally (above 2 characters).
We're deploying some changes soon (#407) that will disallow the creation of top-level domain names (e.g. dan) and instead create a subdomain, which will be based on the network (dan.testnet for TestNet, dan.near for MainNet).

Issue moved to nearprotocol/docs #281 via ZenHub


FINERACT-1178 : Added a SQL schema connection parameter field

FINERACT-1178
Description
Added a way to add connection parameters to the JDBC string.
Added a field schema_connection_parameters in the tenant_server_connections table in fineract_tenants database, and added a migration file to implement those modifications in existing databases.
Not sure if this is the right way to implement this feature, again as mentionned in the Jira issue the main goal is to allow the usage of Cloud SQL JDBC connection strings with the GCP socket factory. If you have any suggestions please do not hesitate to mention them.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[x] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[x] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[x] Create/update unit or integration tests for verifying the changes made.

[x] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[x] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[x] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

Thank you for the reviewing, I've fixed the merge conflict.


O>require review of file repository changes

We have too many people with too little experience with git to let everyone commit without review.
For example, the blame view of CONTRIBUTING.md got corrupted in the process of a a recent tagalog translation 5e795fb.

[X] change the repository settings to require review of pull requests.
[ ] update CONTRIBUTING.md to document the requirement

@dckc I don't know how to manage this Github Rep. I leave it up to you to make all the necessary changes/restrictions.
I've made you admin of rchain/members

Strongly agree with this. Pull request is the recommended way instead of pushing commit directly.

Git and GitHub knowledge should be a requirement. Pull Requests is a nice wall between people who know what they are doing and people who don't.

Hey @dckc . I have pushed a PR adding the git workflow chapter in CONTRIBUTION.md.
PTAL.

@dckc There's a lot of stuff here and here

@ICA3DaR5 Hey, i push a PR which mentioned above. I add a workflow example in the contributing file just like the most opensource project. Do you think which way is better:

Add a simple workflow example
Add a tutorial link

We should choose one so that newbie can learn how to use git.

Since CONTRIBUTING.md is the authoritative record on what the bounty contract is, its history needs to remain in-tact. So that we can once again see who wrote which parts of CONTRIBUTING.md and when, I reverted the master branch to Feb 7, before it was clobbered in some translation work:
To github.com:rchain/Members.git

8f687b3...a517753 master -> master (forced update)

What was previously the master branch is now a needs-cleanup branch.

@dckc where is going to be the place for translations now? (after the clean up)

I made a suggestion about directory organization in #351.
But this issue is not about changing file locations; it's about cleaning up the git version control history.


Update README.md

mongojs doesnt have ObjectId method, but the Database object has it, as we can see in: https://github.com/mafintosh/mongojs/blob/master/lib/database.js#L12

mongojs should expose this as on the module itself. /cc @sorribas

@mafintosh Yeah, that is right. I just published a fix for it.

This PR is thus, no longer relevant.

Great, Awesome library guys! thanks!


Support null as a child element for ListItem

Last few years React supports null as a valid child (it just ignores this element). Used mostly for conditional statements. This pull request makes ListItem support nulls

Codecov Report

Merging #584 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #584   +/-   ##
=======================================
  Coverage   94.49%   94.49%
=======================================
  Files          61       61
  Lines        2560     2560
  Branches      371      371
=======================================
  Hits         2419     2419
  Misses         50       50
  Partials       91       91

Impacted Files
Coverage Δ

packages/list/ListItem.tsx
93.61% <0%> (ø)
: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 1d08700...c1312c1. Read the comment docs.


OP-16992 : Added parameter for PipelineAndPolicy summarycount (#229)

https://devopsmx.atlassian.net/browse/OP-16992

This PR  triggered this build in this execution, please wait for it  and child pipelines to succeed before merging:  https://isd.prod.opsmx.net/deck/#/applications/gcp-gitprtrigger/executions/details/01G9PY1SAM2QA38NNBAAK0X76B

This PR  triggered this build in this execution, please merge to v4.0 branch only if this succeeds:  https://isd.prod.opsmx.net/deck/#/applications/isd40/executions/details/01G9PY2BFDNC6EGDKF375TQ4PJ

RunTestJar RESULT: Tests run: 229, Failures: 14, Errors: 0, Skipped: 29, Time elapsed: 427.466 sec <<< FAILURE! Reviewers , please check that the changes in this PR did not cause the failures in  http://jenkins.opsmx.net:8181/jenkins/job/Dev-API-Automation-Test-v4.0/ws/emailable-report-194.html


Invalid options are silently ignored

This is more of an experience report than a bug report, since it highlights multiple problems while trying to update my config to the latest version.
Updated to v1.0.0-beta.125 and decided to replace redoc-cli build with openapi build-docs .
The old invocation used a number of CLI options:
redoc-cli build openapispec.json -t template.hbs\
- --title "API Documentation"\
- --options.requiredPropsFirst=true\
- --options.hideDownloadButton=true\
- --options.theme.spacing.sectionVertical='8'\
- --options.theme.breakpoints.small='50rem'\
- --options.theme.breakpoints.medium='85rem'\
- --options.theme.breakpoints.large='105rem'\
- --options.theme.typography.fontSize='16px'\
- --options.theme.typography.lineHeight='1.75'\
- --options.theme.typography.fontFamily='Overpass,Arial,OpenSans,sans-serif'\
- --options.theme.typography.headings.fontFamily='Moderat,Overpass,Arial,OpenSans,sans-serif'\
- --options.theme.typography.headings.fontWeight='700'\
- --options.theme.typography.code.fontFamily='Moderat Mono,Courier New,monospace'\
- --options.theme.menu.width='300px'\
- --options.theme.menu.backgroundColor='#fafafa'\
- --options.theme.menu.groupItems.textTransform='none'\
- --options.theme.menu.level1Items.textTransform='none'\
- --options.theme.rightPanel.backgroundColor='#263238'\
- --options.theme.rightPanel.width='40%'

Replaced just the command name and got an error about --options no longer existing. Opened the command docs. Can't figure out what should be in --templateOptions (no examples given) and what should be in --theme.openapi.
Tried:
openapi build-docs openapispec.json -t template.hbs\
 --title "API Documentation"\
 --templateOptions.requiredPropsFirst=true\
 --templateOptions.hideDownloadButton=true\
 --theme.openapi.spacing.sectionVertical='8'\
 --theme.openapi.breakpoints.small='50rem'\
 --theme.openapi.breakpoints.medium='85rem'\
 --theme.openapi.breakpoints.large='105rem'\
 --theme.openapi.typography.fontSize='16px'\
 --theme.openapi.typography.lineHeight='1.75'\
 --theme.openapi.typography.fontFamily='Overpass,Arial,OpenSans,sans-serif'\
 --theme.openapi.typography.headings.fontFamily='Moderat,Overpass,Arial,OpenSans,sans-serif'\
 --theme.openapi.typography.headings.fontWeight='700'\
 --theme.openapi.typography.code.fontFamily='Moderat Mono,Courier New,monospace'\
 --theme.openapi.menu.width='300px'\
 --theme.openapi.menu.backgroundColor='#fafafa'\
 --theme.openapi.menu.groupItems.textTransform='none'\
 --theme.openapi.menu.level1Items.textTransform='none'\
 --theme.openapi.rightPanel.backgroundColor='#263238'\
 --theme.openapi.rightPanel.width='40%'

Looking at the output, it seems that a number of my settings were ignored. Tried to falsify them by introducing a misspelling in each one, and got no error.  I would expect any unknown option to give me an error.
Realized this might not be the way redocly-cli wants to be used nowadays, and I had a .redocly.yaml file already, so decided to move my options there. Referencing https://redocly.com/docs/api-reference-docs/configuration/functionality/ I added the following section:
features:
  openapi:
    hideDownloadButton: true
    requiredPropsFirst: true
    theme:
      breakpoints:
        small: "50rem"
        medium: "85rem"
        large: "105rem"
      rightPanel:
        backgroundColor: "#263238"
        width: "40%"
      sidebar:
        backgroundColor: "#fafafa"
        width: "300px"
        groupItems:
          textTransform: "none"
        level1Items:
          textTransform: "none"
      spacing:
        sectionVertical: 8
      typography:
        fontFamily: "Overpass,Arial,OpenSans,sans-serif"
        fontSize: "16px"
        lineHeight: "1.75em"
        code:
          fontFamily: "Moderat Mono,Courier New,monospace"
        headings:
          fontFamily: "Moderat,Overpass,Arial,OpenSans,sans-serif"

only for the theme options to be ignored again. I tried different structures (features.openapi.theme => theme.openapi as used here) but no change and no error.
So, I am suggesting two things:

For the CLI to start throwing errors when given unknown command options or configuration objects.
A docs page extended to demonstrate specifying theme options in the .redocly.yaml

This is really valuable, thank you! We are working on migration guides for redoc-cli users and your example command gives lots of context that will help get this right. I'd like to leave this issue open for now, and update when the first guides are published.

Update (so you don't have to wait for the docs to be finalised and published):

Use redocly instead of redoc-cli (the old tool) or openapi (an old alias for the new tool).
Replace the prefix of the commandline options, so instead of starting with --options., you instead use --theme.openapi. ... so you end up with options like   --theme.openapi.theme.typography.fontSize='16px'.

It would be useful to know if these changes produce the output you expected.
I'll find an example of doing the same with the config file as well, I know this was also requested and I agree it would be good to have.

For the config file, change features at the top level to theme, the rest I think looks pretty good, but let us know how you get on?

I think this is pretty much covered by the migration guide https://redocly.com/docs/cli/guides/migrate-from-redoc-cli/ so I'll close it. More questions on this topic are very welcome though - open a new issue if you need something.


I found a clerical error in the code

class ResNet(nn.Layer):
shared = ['norm_type']
def __init__(self,
             depth=50,
             ch_in=64,
             variant='b',
             lr_mult_list=[1.0, 1.0, 1.0, 1.0],
             groups=1,
             base_width=64,
             norm_type='bn',
             norm_decay=0,
             freeze_norm=True,
             freeze_at=0,
             return_idx=[0, 1, 2, 3],
             dcn_v2_stages=[-1],
             num_stages=4,
             ### std_senet=False,
             freeze_stem_only=False):
    """
    Residual Network, see https://arxiv.org/abs/1512.03385

    Args:
        depth (int): ResNet depth, should be 18, 34, 50, 101, 152.
        ch_in (int): output channel of first stage, default 64
        variant (str): ResNet variant, supports 'a', 'b', 'c', 'd' currently
        lr_mult_list (list): learning rate ratio of different resnet stages(2,3,4,5),
                             lower learning rate ratio is need for pretrained model
                             got using distillation(default as [1.0, 1.0, 1.0, 1.0]).
        groups (int): group convolution cardinality
        base_width (int): base width of each group convolution
        norm_type (str): normalization type, 'bn', 'sync_bn' or 'affine_channel'
        norm_decay (float): weight decay for normalization layer weights
        freeze_norm (bool): freeze normalization layers
        freeze_at (int): freeze the backbone at which stage
        return_idx (list): index of the stages whose feature maps are returned
        dcn_v2_stages (list): index of stages who select deformable conv v2
        num_stages (int): total num of stages
        std_senet (bool): whether use senet, ### default True
    """

std_senet is False,but in marginal notes it's default True

Thanks.  Fix it  https://github.com/lyuwenyu/RT-DETR/commit/77ed89e096e900a2489f2134854908cdae17eaea


update Starforged Asset data to 122020

ive got them in Dataforged, they just need to be formatted properly

resolved by commit b0a200ab5f9033f0ae32c088c2eca8809fc172c2


Delete_Kth_Node_From_End.cpp

this is delete kth node from linked list
Assigned Issue Number ?
#1260
Language Opted ?
C++
Number of files added or updated ?
1
Name of these files ?
Data_Structures => Linked_List => Singly_Linked_List => Delete_Kth_Node_From_End => Delete_Kth_Node_From_End.cpp
Checklist

[x] I've read the contribution guidelines.
[x] I've referred the correct issue number.
[x] I've fill up this entire template correctly.

@ahampriyanshu  please check my pull request

@ahampriyanshu  please check my pull request


Support for gemini function calling?

What are your thoughts about adding support for gemini function calling?
https://ai.google.dev/docs/function_calling
Shouldnt be too complicated.

Based on this document, it appears gemini function calling requires the function definitions to be made using the classes/types defined within the google-generativeai package, which is internally different from the format OpenAI's requires for function calling (a JSON encoded dictionary).
https://json-schema.org/understanding-json-schema
I'd have to either create a completely separate schema generation system just for Gemini (which is out of the question as it will make custom converters harder to implement), or create a bulky translator module that turns the JSON Schema compatible Dictionary OpenAI package uses into something that the google-generativeai package can give to Gemini.
Invoking the functions would require it's own set of changes.


added gsm_plus

GSM-Plus Math benchmark
Paper: https://arxiv.org/abs/2402.19255
Strengths:

More updated and more capable version of gsm8k

Thanks!
Are you able to run a sanity check, maybe with models like LLaMA-2-7B and see if the eval results are similar to the paper?

Thanks!
Are you able to run a sanity check, maybe with models like LLaMA-2-7B and see if the eval results are similar to the paper?

same as the evals on paper.

Thanks!
Are you able to run a sanity check, maybe with models like LLaMA-2-7B and see if the eval results are similar to the paper?

running llama-2-7b, same as evals on paper

UPDATE:

Reverted to original GSM-Plus dataset for attribution
Added GSM-Plus_mini subtask


[FEAT] Replace this module with Magneto Vault

I don't see discussion board on this repo so had to open a issue.
Do we have a plan to sunset this module or back port it to support magento vault?
It is beneficial to manage all saved CC in the same way for merchants if they have multi CC payment gateways, and it would also allow enabling instant purchase with ParadoxLabs modules

Hi Ryan, thanks for the feedback!
We're not going to sunset TokenBase in favor of Magento Vault -- TokenBase's functionality goes quite a ways beyond Vault (including card add/edit interfaces). It would be a major rewrite, and with TokenBase being quite stable in its current form, that's not something we're eager to invest in.
That said, it is possible that we could more thoroughly support the Vault interfaces and features. We already implement the Vault card interface via TokenBase. Whether that happens is a question of how much more is required and whether that makes business sense (unless it gets sponsored or contributed by others).


ENH: add a "busy" device

We had a request for a way to set a sleep in the RE that would give a
progress bar.
You can get an interuptable sleep via bps.sleep, but that provides no
feedback.  Rather than trying to invent another way to do progress bars, re-use
the progress bar machinery for hardware motion.
This still needs tests, but @bruceravel is using this at his beamline so I wanted to get a PR for ASAP.

Different than support for the EPICS busy record from synApps.  https://github.com/BCDA-APS/apstools/blob/main/apstools/synApps/busy.py

Correct, it was inspired by the busy record, but re-using the name is a probably a terrible idea.  I think this is a very good idea and I can see it being used both as a way to provide a delay on its own and as a way to put a "time box" around some other activity (because it is a "settable" it can be set in parallel with either other sets or a trigger!).  \
What should we call this instead?

SleepWithProgressBar
TimeMover
TimeTraveler
Delay
SleepMachine
??

The name is ok. Just leaving a trail to thee other one.
On Tue, Feb 8, 2022, 10:14 PM Thomas A Caswell @.***>
wrote:

Correct, it was inspired by the busy record, but re-using the name is a
probably a terrible idea. I think this is a very good idea and I can see it
being used both as a way to provide a delay on its own and as a way to put
a "time box" around some other activity (because it is a "settable" it can
be set in parallel with either other sets or a trigger!). \
What should we call this instead?

SleepWithProgressBar
TimeMover
TimeTraveler
Delay
SleepMachine
??

—
Reply to this email directly, view it on GitHub
https://github.com/bluesky/ophyd/pull/1028#issuecomment-1033328150, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AARMUMGQ5YITTYJLJBPAFTDU2HS3XANCNFSM5N4KQCXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

I am not sure when I will have time to work on this again, I would be very 💯 👍🏻 if someone was motivated and wanted to take this PR over!


Notifications double firing

Some notifications in due-dates branch double fire.

This might actually just be the fact that I've been having multiple notifications fire at once during testing.


Make ExprList initializable by result builder

Since the Array: ExpressibleAsExprList conformance is unlikely to be added due to type system limitations, here is a small patch to make expression lists a bit more convenient to use by making them builder-initializable (currently only affects SequenceExpr and YieldList).
Using this we can shorten
SequenceExpr(elements: ExprList {
  ...
})

to
SequenceExpr {
  ...
}

@swift-ci please test


about:killAllOnCompletedTaskNumber has been obsoleted

Hi,
I am trying to submit a job by the webportal.
The json file is cifar10.json from "Quick start: how to write and submit a CIFAR-10 job".
However, I got a error: "killAllOnCompletedTaskNumber has been obsoleted, please use minFailedTaskCount and "minSucceededTaskCount instead."
I wonder how can fix the problem?
Thank you!

killAllOnCompletedTaskNumber has been removed from the current master branch's [Quick Start section] (https://github.com/Microsoft/pai/tree/master/examples#quickstart), feel free to update yours and give us more feedback.
Ping @YanjieGao for a possible doc issue.

If you deploy the latest bits from master branch, killAllOnCompletedTaskNumber has been obsoleted. If you use our stable release 0.6.y, etc. It has not yet. We suggest you deployment our stable release for the moment.

current branch doc and example doesn't show killAllOnCompletedTaskNumber to user, i guess @boozyguo  write killAllOnCompletedTaskNumber  directly in job json file ?

@YanjieGao , thank you for help. I use the json file from  "Quick start: how to write and submit a CIFAR-10 job".
the  json file content is below:
{
"jobName": "tensorflow-cifar10",
"image": "openpai/pai.example.tensorflow",
"dataDir": "/tmp/data",
"outputDir": "/tmp/output",
"taskRoles": [
{
"name": "cifar_train",
"taskNumber": 1,
"cpuNumber": 8,
"memoryMB": 32768,
"gpuNumber": 1,
"command": "git clone https://github.com/tensorflow/models && cd models/research/slim && python download_and_convert_data.py --dataset_name=cifar10 --dataset_dir=$PAI_DATA_DIR && python train_image_classifier.py --batch_size=64 --model_name=inception_v3 --dataset_name=cifar10 --dataset_split_name=train --dataset_dir=$PAI_DATA_DIR --train_dir=$PAI_OUTPUT_DIR"
}
]
}
There is no killAllOnCompletedTaskNumber  in json file.
However, the "submit job" webportal  has the killAllOnCompletedTaskNumber  filed.

@boozyguo  you could refer fan's answer. use the stable release of pai

Hi @hao1939 , dev-box docker is use master branch for deployment, this may lead to customer use unstable version problem. Is it better each release use the specific release version of PAI?
![image](https://user-images.githubusercontent.com/5576848/44566584-ed248680-a7a0-11e8-9efa-f01a0d655a14.png

@boozyguo please go to https://github.com/Microsoft/pai/releases to checkout a latest release.

thank you. @fanyangCS @YanjieGao
So,  I should download PAI v0.6.1 in dev-box, then  prepare quick-start.yaml and run paictl.py?

Hi @boozyguo ,
You should match the image tags and code branch.
In case you are using currently latest release v0.6.1：

In dev-box, make sure checkout the code with tag v0.6.1
In config file services-configuration.yaml, set the docker-tag: v0.6.1

If you want to update the config, recommend fellow steps:

In dev-box, checkout the tag v0.6.1
generate the config (make sure generate in another directory, it won't override the previously config)
In config file services-configuration.yaml, set the docker-tag: v0.6.1
stop old installation, using paictl.py service stop ...
start with new config, using paictl.py service start ...

That's all.

Hi @boozyguo ,
You should using python paictl.py cluster generate-configuration ....
There are break changes after v0.6.1.
So please refer to the doc under the branch pai-0.6.y: https://github.com/Microsoft/pai/blob/pai-0.6.y/pai-management/doc/single-box-deployment.md

thank you @hao1939
I have try :
https://github.com/Microsoft/pai/blob/pai-0.6.y/pai-management/doc/single-box-deployment.md
https://github.com/Microsoft/pai/blob/pai-0.6.y/pai-management/doc/cluster-bootup.md
however, I got another error, deploy failed.
.........
2018-08-24 13:15:35,871 [ERROR] - k8sPaiLibrary.maintainlib.common : There will be a delay after installing, please wait.
Segmentation fault
2018-08-24 13:15:35,874 [ERROR] - k8sPaiLibrary.maintainlib.common : There will be a delay after installing, please wait.
Segmentation fault
2018-08-24 13:15:35,878 [ERROR] - k8sPaiLibrary.maintainlib.common : There will be a delay after installing, please wait.
Segmentation fault
2018-08-24 13:15:35,880 [ERROR] - k8sPaiLibrary.maintainlib.common : There will be a delay after installing, please wait.
Segmentation fault
2018-08-24 13:15:35,884 [ERROR] - k8sPaiLibrary.maintainlib.common : There will be a delay after installing, please wait.
Segmentation fault
..............

Hi @boozyguo ,
The log is some kind of misleading, actually it's waiting for the pod to be ready.
Could you check the k8s dashboard in the same time?
It's on http://your_master_ip:9090.

Hello, @hao1939
I have try other version of PAI, but there always are some network problem(can not pull images from gcr.io or docker.io).
So, I choose the latest version. At "submit job" webpage, I found a label named "Properties". When click it, there are some items including "killAllOnCompletedTaskNumber".  After uncheck the "killAllOnCompletedTaskNumber" item, the job can submitted succed.
I consider there is a bug between "webportal" and "api-server" or another parts of PAI. They have a different version.
Thank you for help.

Hi @boozyguo ,
You are right, we fixed the network problem on the release v0.6.1, because in some region gcr.io is not reachable.
'api-server' and 'webportal' do have different version.
And you can check the images tag on k8s dashboard.
In you case, it should be docker.io/openpai/rest-server:latest.

Hi @boozyguo ,
I will close it for now, if you have more questions, feel free to reopen it.

hello, @hao1939  thank you for your help. I got the method to deal with the problem.


Cannot access the CF API endpoint

Hello, all
I am trying to deploy CF to AWS using the bosh-lite.
Normally the api endpoint is api.bosh-lite.com which points to a local IP address like 10.244
If I need to access it via xip.io domain name, I need to create a stub file to override the domain to PUBLIC_IP_OF_LITE_VM.xip.io and generate the cf.yml
from generated yml I can see the domain name are changed to the xip.io related.
But after deploying the CF, I still cannot access the API endpoint, can anyone help on it?
when launching the vagrant VM, the related parameters are:
export BOSH_AWS_ACCESS_KEY_ID=
export BOSH_AWS_SECRET_ACCESS_KEY=
export BOSH_LITE_SECURITY_GROUP=sg-d63f72b2
export BOSH_LITE_PRIVATE_KEY=/home/ubuntu/.ssh/id_rsa
export BOSH_LITE_KEYPAIR=bosh
export BOSH_LITE_NAME=vagrant
I also assigned a elastic ip to the bosh lite VM.
If I try to "telnet liteVMIP 443", it says " Connection refused".
The 443 port is opened already in the security group.

Can you confirm that you have:

the BOSH-Lite VM you created with vagrant has an elastic IP actually associated to it
it belongs to a security group that allows incoming TCP connections to port 443 from any source IP
telnet BOSH_LITE_ELASTIC_IP 443 fails

Next, are you deploying to a VPC? Or plain EC2?  If you're deploying to a VPC, then you need to make sure that your BOSH-Lite VM is in a publicly routable subnet of your VPC, namely one with an Internet Gateway as the default route in its associated route table.

Thanks for reply.
Yes it's deploying to VPC,and I create the VPC based on http://bosh.io/docs/init-aws.html
And here are my network related settings, and I think it's already there regarding your question?
Is there anything wrong here?
VPC ID:   vpc-7cedb719 | bosh
Network ACL: acl-ed664388
State: available
Tenancy: Default
VPC CIDR: <IP_ADDRESS>/16
DNS resolution: yes
DHCP options set: dopt-9bc6dff9
DNS hostnames: yes
Route table: rtb-1c2e0079
Subnet ID: subnet-ddf5bbaa | public
Availability Zone: us-west-2a
CIDR: <IP_ADDRESS>/24
Route table: rtb-1f2e007a
State:available
Network ACL:acl-ed664388
VPC: vpc-7cedb719 (<IP_ADDRESS>/16) | bosh
Default subnet: no
Available IPs: 249
Auto-assign Public IP: yes
Route Table ID: rtb-1f2e007a
Main: no
Explicitly Associated With: 1 Subnet
VPC: vpc-7cedb719 (<IP_ADDRESS>/16) | bosh
Destination   Target    Status  Propagated
<IP_ADDRESS>/16   local     Active  No
<IP_ADDRESS>/0  igw-432fb726	Active  No
ID: igw-432fb726
Attached VPC ID: vpc-7cedb719 (<IP_ADDRESS>/16) | bosh
State: attached
Attachment state: available

Attach instance info and security group too.
Instance:
Instance ID : i-0bab71d2
Public DNS : ec2-52-25-158-120.us-west-2.compute.amazonaws.com
Instance state : running
Public IP : <IP_ADDRESS>
Instance type : m3.xlarge
Elastic IP  : <IP_ADDRESS>
Private DNS :ip-10-0-0-245.us-west-2.compute.internal
Availability zone:  us-west-2a
Private IPs :<IP_ADDRESS>
Security groups : bosh-lite. view rules
Secondary private IPs
Scheduled events:  No scheduled events
VPC ID : vpc-7cedb719
AMI ID : boshlite-9000.69.0 (ami-de5f49bf)
Subnet ID : subnet-ddf5bbaa
Platform -
Security Group:
HTTP     			TCP 	 80             <IP_ADDRESS>/0
All TCP  			TCP 	0 - 65535       sg-d63f72b2 (bosh-lite)
All traffic			All 	All  			sg-d63f72b2 (bosh-lite)
SSH 				TCP 	22				<IP_ADDRESS>/0
Custom TCP Rule 	TCP		25555			<IP_ADDRESS>/0
All UDP 			UDP		0 - 65535		sg-d63f72b2 (bosh-lite)
Custom TCP Rule 	TCP		4443			<IP_ADDRESS>/0
HTTPS				TCP		443				<IP_ADDRESS>/0

And just to confirm, those are the inbound traffic rules for the security group, right?

yes, they are at the inbound traffic rules.

This excerpt from here may be relevant:

As part of Vagrant provisioning bosh-lite is setting IP tables rules to direct future traffic received on the instance to another IP (the HAProxy). These rules are cleared on restart. In case of restart they can be created by running vagrant provision.

I can telnet to 25555 and 22, just not 443 and 80.  443 and 80 connections are meant to be handled by the HA Proxy process running within a container inside the BOSH-Lite VM, so the fact that this fails to telnet suggests either that process is down or the iptables rule forwarding traffic on those ports to the container need to be re-established.  I am able to see that the HA Proxy process is running fine, so perhaps this is the issue.
BTW, you may wish to change your director credentials.  See https://bosh.io/docs/director-users.html

Thanks, I just had a try, after running the vagrant provision, I can telnet the 443 port.
As I changed the domain name to bosh-lite.com yesterday, I will have another try for the xip.io later. (should work as 443 can be telnet)
For director credentials, I will change them if I get the deployment well.
Thank a lot for asking the questions, will update it again later.

Tried it, it works.
And the problem should be that:
when I ran vagrant up --provider=aws  the vagrant is hang there and it didn't ssh to the bosh-lite VM to set the iptables rule forwarding.
I generated the id_ras public/private keys, and I replaced the bosh private key to id_rsa file.
But I cannot ssh to bosh-lite VM using ssh ubuntu@bosh-lite
If I change the id_ras to new name like id_ras.bosh and update the environment for it and then I can ssh to bosh-lite VM.
I am new on it, thanks for help, I appreciate that.


Q: How to retrieve callback fragment parameters from frontend after login?

This is offtopic. I was just curious about people who are using supabase for their PHP projects, how do you authenticate your users when the async login didn't return anything useful? Do you parse the url fragment returned on the callback url and make a hidden form on the page to initiate a POST request to server so that PHP able to get the values?
I'm implementing Magic Link auth on my Laravel project, and I was brain f*ed thinking how to pass those values to the backend because the values are placed in fragments. So I came across this but found out it is also using the API endpoints only and nothing special, all the auth login function return void. So it should be similar results to my current implementation using supabase JS.
Sorry for my noob question... I'm not an expert, I hope to get more information about this, since there's not much information about this on the internet. Thanks!

It seems I didn't understand this library properly. I was too quick to ask a question. Sorry! Closing this now...


Make frontend prototype

https://docs.google.com/drawings/d/1P-h5IiIxSucx8YNn2PX7_d0BMDHF-npX8OJX-ZlBv8c/edit?usp=sharing

@chemeshuggah we need call for discuss this picture

Ok. I will be available today around 19-20 o'clock or earlier if you will be in town :)


Microsoft.Azure.Mobile.Server.Authentication nuget package constraint to JWT

I found a problem trying to implement authentication on .NET server.
Basically, the Microsoft.Azure.Mobile.Server.Authentication nuget package has a dependency to System.IdentityModel.Tokens.Jwt (>= <IP_ADDRESS>111437).
But in the current version of System.IdentityModel.Tokens.Jwt (5.0.0) has incompatibility between classes on AppServiceLoginHandler.CreateToken method.
For now I downgraded to the <IP_ADDRESS>8261200 version of JWT, but, or the nuget package constraint need to be changed or the next version of Microsoft.Azure.Mobile.Server.Authentication need to have compatibility with JWT 5.0.0.
PS1: I'm in on the alpha channel, but the constraint is the same on the stable channel.

The System.IdentityModel.Tokens.Jwt has this issue all over the place.  It is only valid for .NET Core.    What the system is saying is "use the latest version of ...Jwt that is compatible with our runtime", but the runtime match was left out of the Jwt packaging.
You've done the right fix.


Keep GitHub Actions up to date with GitHub's Dependabot

Fixes the warnings at the bottom right of
https://github.com/JarvusInnovations/background-action/actions/runs/8310867573

https://docs.github.com/en/code-security/dependabot/working-with-dependabot/keeping-your-actions-up-to-date-with-dependabot
https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#package-ecosystem

Also see:

#184

@Multimo, @marcinlesek  Your reviews, please

This looks good to me @cclauss, but I don't have permission to give approves here. 😉 You need to call @ryon and @themightychris as they are people in JarvusInnovations organisation.

@jmealo Your review, please.

@cclauss: Looks good to me. Thanks for the PR. If the tests run I'll merge.

Generated:

#190


bukubrow: 2.4.0 -> 5.0.0

Motivation for this change
Seems like this hasn't been touched in a while? Unfortunately it tries to be smart and generates the json on-demand and guesses where it should go and I'm not even going to begin to try using that in the build so I've just included the files themselves. Could use toJSON instead if anyone wants it to be fancy I guess?
Not entirely sure about the chromium path, chrome docs say to use /etc/chromium but I've only tested with firefox.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build bukubrow
@arcnmx how could I test this, please?

Something like...
nix run '((import <nixpkgs> {config.firefox.enableBukubrow = true;}).firefox) -c firefox'
and installing the extension from AMO should work

Resolved conflict, and also inlined the json because it seemed slightly neater.
cc @Infinisil

Nice!


mrk_success.js replaces the notification html with a hardcoded english sentence

The javascript code to toggle the read status of a notification contains a hardcoded english sentence:
...action = 'read';
...action = 'unread';
notification.html('Mark as '+ action);

As the inner html comes from template, I think the best approach to fix this is to also retrieve the translated text from the template. In this approach, the user can fully customize the way it wants his messages, without receiving an unwanted text fragment.

Thanks for all the work :+1:


Update README.md

What is included in this PR?
updated readme.md to provide some more alternative options to set up and run the web UI locally, and addressed the issue that ODK hamster service is now outdated/deprecated.


The extension manifest is invalid

Hi,
I am trying to install the extension for Visual Studio 2017 Professional Edition. But it says that "The extension manifest is invalid". Is there any way I can install this extension for VS 2017 Professional Edition. It worked fine with VS 2015 Professional Edition.
Thanks Prasun

I tried to reproduce this issue with VS2017 Pro 15.8.4 and TGit 4.9.40. Uninstallation and installation completed successfully.

Thanks for testing @GreenKn1ght !
@prasunbanik : Maybe try to redownload the vsix file?

Hi Samir,
I freshly downloaded the vsix installer file and tried to install the same.
While doing so, I got this error
[image: image.png]
I also tried to install it from the VS Extension menu, then also I get an
error but this error is different. I have also attached the log file that
is generated during the installation for your reference.
[image: image.png]
--
Thanks and Regards
Prasun Banik
On Thu, Sep 20, 2018 at 3:01 PM Samir Boulema<EMAIL_ADDRESS>wrote:

Thanks for testing @GreenKn1ght https://github.com/GreenKn1ght !
@prasunbanik https://github.com/prasunbanik : Maybe try to redownload
the vsix file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sboulema/TGit/issues/47#issuecomment-423111871, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AorR79BZuh5VJchCRPVsV-2GOZNuAIXBks5uc2BcgaJpZM4WxddC
.

@prasunbarik images and log did not make it to the GitHub issue page. Can you please attach them?

Hi Samir,
Sorry for the late reply. Please find the images attached in the same order.

Hey,
Especially the last screenshot indicates that it may be caused by a corrupted visual studio installation :(
Maybe try the steps in this thread:
https://social.msdn.microsoft.com/Forums/vstudio/en-US/6e88f10a-2890-405b-a3a0-cbc7a1031a8f/visual-studio-2017-extensions-and-update-quotcannot-find-setup-engine-instancequot?forum=visualstudiogeneral


update: openshift manifests

update: openshift manifests

@sallyom CI requires the commit subject line less than 50 characters

/hold this PR - turns out we need cgroupsV2 when deploying on VMs, so I'll add that back - the kernel-devel cluster-prereq is no longer required.

@williamcaban @rootfs finally found the correct incantation for commit msgs :smile_cat:  this is ready for review, thank you

@williamcaban wdyt?

thank you @dagrayvid for confirmation!


docs: update database adpater installation commands to use beta version

While following the "Adding to an existing app" instructions for the beta docs, I noticed that the pnpm installation commands for the database adapters were missing the @beta tag, which will result in errors in the project.

Thank you! Great catch!


GEODE-5637: move SingleHopClientExecutorSubmitTaskWithExceptionTest t…

…o integrationTest

Increase Awaitility timeout due to flaky failure
Cleanup test and use AssertJ

Please review: @dhemery @balesh2

Test is still flaky but I'm going to merge this since it should be moved from test to integrationTest src set.


Links to generated documentation

Description
Links to generated documentation
Type of change

Documentation update

Testing steps
N/A

Codecov Report

Merging #168 (862d13e) into master (cf65393) will decrease coverage by 3.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #168      +/-   ##
==========================================
- Coverage   92.52%   89.35%   -3.17%
==========================================
  Files          17       19       +2
  Lines         602      667      +65
==========================================
+ Hits          557      596      +39
- Misses         44       59      +15
- Partials        1       12      +11

Impacted Files
Coverage Δ

collections/slices.go
100.00% <ø> (ø)

env/env.go
100.00% <ø> (ø)

http/http.go
100.00% <ø> (ø)

http/metrics_middleware.go
85.00% <ø> (-2.50%)
:arrow_down:

http/openapi.go
41.66% <ø> (-4.68%)
:arrow_down:

http/router_utils.go
100.00% <ø> (ø)

metrics/metrics.go
0.00% <ø> (ø)

responses/responses.go
100.00% <ø> (ø)

tests/helpers/errors.go
100.00% <ø> (ø)

tests/helpers/frisby.go
100.00% <ø> (ø)

... and 25 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 cf65393...862d13e. Read the comment docs.


go log lock in High concurrency

In my project ,too many goroutine made my server no response.When I use pprof to locate the problem,found above.Does it means logger's occure dead lock? Or it was a normal phenomenon in high concurrency?

As you can see, loggers use a lock, so only one goroutine can lock at a time. If you have many cores and cpu-intensive work, you may consider working around the limitation, such as using multiple loggers or having certain critical functions not log at all.
But this isn't a bug in the log package nor in Go, as far as I can see. For questions about Go, see https://golang.org/wiki/Questions.


Add to .gitignore option in Source Control tab

The Source Control tab already detects when git is used.
Can we have "Add to .gitignore" on the list for non tracked files?

I can work on this, if possible.

@BugraC wait for @joaomoreno to give a go ahead first 😃

#23797

@BugraC #23797 is marked as help wanted so go for it :smiley:


Design tokens

I understand it is possible to change the theme in the CDK for example the colours like this:
const colours = {
button: {
color: "#FFED00",
hoverColor: "#000000"
}
}
…
<ThemeContext.Provider value={colours}>

</ThemeContext.Provider>
In the https://developer.dxc.com/design/principles it is mentioned: “brand white-labeling through the configuration of our design tokens for each and every component.”.  Is the above example what you refer to as a “design token”, or is a design token something else?
If it is something else, is it possible to link to/create the documentation for it.

This has been resolved in conversation.

This has been resolved in conversation.


Questions about building the pico-pedal

Hi,
I really liked your conversion of the pi-pedal into the pico-pedal.
I have already ordered printed PCB's and parts to make one for myself, but I would really appreciate it if you could share some pictures of your build to help me figure out what goes where.
I am writing this issue because I knew of no other ways to contact you. If you would be okay with sharing some pictures.
kind regards,
Lex

I am also curious as to which specific I2C display you use, as that is the only part I have not yet ordered.

Ah I see. Unfortunately the LCD is the one thing I did not wire yet. Just because the PCB I have has a bug (it is the very first version) and I did not want to waste components. If you look at the code there is also no support for the display yet.
Also when converting the PCB for the RPi Pico I did not try to keep compatibility with the pi-pedal enclosure. The plan was to create a new enclosure to 3D print but I did.not get that far yet. If you want to use the original enclosure I suppose you would need to adjust some little things, nothing major I reckon.


ci: fetch submodules on checkout action

Still failing; the submodule was checked out successfully.
Maybe move to a simple docker test container, leaving nix for local dev.

superseded by #115


Removed button margin for iOS and adjusted padding of buttons stackla…

This is a fix for issue #118
On iOS devices, the Read and Share buttons text was getting cut off due to the negative margin being applied to them. Used the OnPlatform markup extension to apply this margin only on android.
Also adjusted the padding of the StackLayout holding these buttons to align them with the Caption.

Awesome! Thanks!


Backend Spike

Simple User Auth with database
Metadata endpoint (with auth)
File Endpoint (with auth)

Can we use the Firebase authentication engine that they provide?
I think could be a suitable solution
What else can be used?


Move away from gutil, use Vinyl & PluginError

Based on https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5, an attempt to move away from gulp-util in this library.

Coverage decreased (-56.3%) to 43.75% when pulling 6235b49255b34a03625d6f692942dc5dcb6ae39b on stefandoorn:remove-gutil into 86bd6b3340327f6eb19b0e25d7a01faa2dd9a99f on UltCombo:master.

Coverage decreased (-56.3%) to 43.75% when pulling 6235b49255b34a03625d6f692942dc5dcb6ae39b on stefandoorn:remove-gutil into 86bd6b3340327f6eb19b0e25d7a01faa2dd9a99f on UltCombo:master.

Coverage decreased (-56.3%) to 43.75% when pulling 6235b49255b34a03625d6f692942dc5dcb6ae39b on stefandoorn:remove-gutil into 86bd6b3340327f6eb19b0e25d7a01faa2dd9a99f on UltCombo:master.

Coverage decreased (-56.3%) to 43.75% when pulling 6235b49255b34a03625d6f692942dc5dcb6ae39b on stefandoorn:remove-gutil into 86bd6b3340327f6eb19b0e25d7a01faa2dd9a99f on UltCombo:master.

Coverage decreased (-56.3%) to 43.75% when pulling 6235b49255b34a03625d6f692942dc5dcb6ae39b on stefandoorn:remove-gutil into 86bd6b3340327f6eb19b0e25d7a01faa2dd9a99f on UltCombo:master.

@UltCombo Could you have a look at this PR?

Given that this hasn't been updated since 2016, it looks like @UltCombo has abandoned this plugin.

I'm afraid so, @Sora2455

I have created a forked package with gulp-util removed as it seems this repo is no longer supported:
https://www.npmjs.com/package/@elamping/gulp-buster


Telegram WebApp Clone

Description
Description of the project here:
I want to add Telegram WebApp Clone which will be made using HTML,CSS,JS
Code of Conduct

[X] I follow Contributing Guidelines of this project.

/assign


Replace native_file with ASIO files

Since ASIO will be used internally for networking & async, use of ASIO files is preferred.

native_file is kept but API is refactored


feat(coverage): add --json compatibility to forge coverage --report summary

Component
Forge
Describe the feature you would like
Coverage Snapshot
i.e. gas-snapshot but for coverage.
Snapshot
This script records line and function coverage in the format:
lineCoverage result 66.7 %
functionCoverage result 50.0 %

Coverage Script
#!/usr/bin/env bash

set -euo pipefail

mkdir -p "$PWD"/.coverage/

coverageSnapshotPhase()
{
    mkdir -p "$PWD"/.coverage/
    genhtml lcov.info -o  --branch-coverage "$PWD"/.coverage > log
    # genhtml lcov.info $lcovExtraTraceFiles -o $out/.coverage > log
    lineCoverage="$(sed 's/.*lines\.*: \([0-9\.]\+\)%.*/\1/; t ; d' log)"
    functionCoverage="$(sed 's/.*functions\.*: \([0-9\.]\+\)%.*/\1/; t ; d' log)"
        if [ -z "$lineCoverage" -o -z "$functionCoverage" ]; then
            echo "⛔ Failed to get coverage statistics"
            exit 1
        fi
    echo "lineCoverage result $lineCoverage %" >> "$PWD"/.coverage/snapshot
    echo "functionCoverage result $functionCoverage %" >> "$PWD"/.coverage/snapshot
    touch "$PWD"/.coverage/CI_TIMESTAMP
    date +"%Y%m%d%H%M%S" >> "$PWD"/.coverage/CI_TIMESTAMP
}

coverageSnapshotPhase

echo "Snapshot recorded"

FOUND_WARNINGS=$(grep "\d+ result" -Po .coverage/snapshot | sed 's/ result//g')
if (( $(echo "$FOUND_WARNINGS <= 50" | bc -l) )) ; then
  COLOR=red
elif (( $(echo "$FOUND_WARNINGS > 60" | bc -l) )); then
  COLOR=green
else
  COLOR=orange
fi

echo "$FOUND_WARNINGS"

rm log
sleep 1
exit 0

This does not follow the gas snapshot format which tbh I dislike as it is difficult to regex against.
Additional context
Note: I flatten the contract to run coverage reporting against, ergo this script works for that context.

I think this can best be captured by adding compatibility with the --json flag so it serializes the summary table
| File                 | % Lines       | % Statements  | % Branches    | % Funcs       |
|----------------------|---------------|---------------|---------------|---------------|
| script/Counter.s.sol | 0.00% (0/3)   | 0.00% (0/3)   | 100.00% (0/0) | 0.00% (0/2)   |
| src/Counter.sol      | 100.00% (2/2) | 100.00% (2/2) | 100.00% (0/0) | 100.00% (2/2) |
| Total                | 40.00% (2/5)  | 40.00% (2/5)  | 100.00% (0/0) | 50.00% (2/4)  |

into a JSON structure
Right now passing the --json flag is incompatible

Completed in https://github.com/foundry-rs/foundry/issues/9111


Add one dimensional variants of Rectangle::resized

Using Rectangle::resized to only change either the width or height of a rectangle is a fairly common operation in my code. But the resulting code is very noisy and this could be improved by adding two new methods:
// Current code:
let bounding_box = target.bounding_box(); // Note that a temporary variable is necessary
let right = bounding_box.resized(Size::new(1, bounding_box.size.height), AnchorPoint::TopRight);

// With the new method:
let right = target.bounding_box().resized_x(1, AnchorPoint::TopRight);

Remaining questions:

How should the methods be named?
resized_x/resized_y, resized_horizontal/resized_vertical, change_width/change_height, ...

Should we reuse AnchorPoint or create two new enums?
resized_x(width, AnchorPoint::TopLeft), resized_x(width, AnchorPoint::CenterLeft), and resized_x(width, AnchorPoint::BottomLeft) would, for example, produce the same output.
The new types could look like this and provide some methods to convert from/to AnchorPoint:
pub enum AnchorPointX { Left, Center, Right }
pub enum AnchorPointY { Top, Center, Bottom }

impl AnchorPoint {
    pub fn from_xy(x: AnchorPointX, y: AnchorPointY) -> Self { ... }
    pub fn x(self) -> AnchorPointX { ... }
    pub fn y(self) -> AnchorPointY { ... }
}

How should the methods be named?

How about the 4th, secret option: resized_width/resized_height? I feel like that fits with Size which also uses the "width" and "height" wording.

Should we reuse AnchorPoint or create two new enums?

Separate X/Y types would encode the intent in the type system which is nice. Reusing AnchorPoint for individual axes would make "half" of the given variant meaningless, e.g. resizing on X with AnchorPoint::TopRight, the Top component is ignored.
As we're going for a bit of API sugar here too, we could call the two new enums AnchorX and AnchorY to reduce chars typed slightly.

How about the 4th, secret option: resized_width/resized_height? I feel like that fits with Size which also uses the "width" and "height" wording.

SGTM.

Separate X/Y types would encode the intent in the type system which is nice. Reusing AnchorPoint for individual axes would make "half" of the given variant meaningless, e.g. resizing on X with AnchorPoint::TopRight, the Top component is ignored.
As we're going for a bit of API sugar here too, we could call the two new enums AnchorX and AnchorY to reduce chars typed slightly.

Also SGTM. AnchorX and AnchorY is a good idea, not only because it's shorter but perhaps also more accurate, because the entire edge could be considered to be the anchor and not only a single point.


License

@prollin @KonajuGames
I noticed PVRTexLibNET doesn't define license terms.
@tomspilman
Doesn't the source code comes from Sickheadgames?
https://github.com/SickheadGames/ManagedPVRTC

We intended our lib to be non-viral open source... we just never added one.  So anything not GPL pretty much.

@tomspilman @nkast Super late reply but is this still an issue? What license would make sense?

It's an issue with redistributing MGCB & pipeline.

Added MIT license; let me know if something else is needed


AIO setup was never called because of incorrect ifdef check

ifdef was checking for an incorrect definition, so aio setup code was
never called. Using USE_PIN_MUX instead of HAVE_PIN_MUX
Signed-off-by: Otavio Pontes<EMAIL_ADDRESS>

+1


No error when variable used has not been defined.

I am not sure how to explain the best.
so here is a code example
def some_function():
    print(some_variable)

some_function()

There should be a red wavy line under the variable name in the print statement because there is no function level or module level deceleration of the variable.
I know if i hover over the variable there is no "tool tip" stating the variable data type. maybe this is the mechanism to use to add the red wavy line.

Thank you for the feedback.
We had this functionality enabled for a brief period of time during 15.8 development but ended up turning it off because it was giving too many false positives.

really...
The tooltip is spot on. if i do not have a variable declared the tooltip is blank. maybe there is a way to leverage VS to see what the tooltip information is and if it is blank then to say the variable is empty.


Add base_path config to spaces

Cc @Pierrci @julien-c
Thinking about it maybe we should rename the param to initial_url before making it widely available?
(cc @christophe-rannou for visibility, although this config is purely on the hub-side)

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

no idea 🤷
i think the current name is ok, it's gonna be the most frequent use case no?

ok then it's ready to be documented / merged


Clicking "Create issue on the dartlang package" opens error page

Trying to file issue #307 using the convenient link I was greeted by this error page:
414 Request-URI Too Large

Thanks for the report! Atom's notification package sends that info to the github repo as an http GET. The size of the stack traces for dart2js compiled code blow throw the browser limit for url lengths.
Reported here: https://github.com/atom/notifications/issues/89.


Associate the custom domain with your Front Door Endpoint

Hello,
At step 3
Please confirm that we must create a CNAME record for a custom domain if the DNS provider is not Azure DNS as the following:

If the record value as an example is "zzz.azurefd.net"
If the domain/record name is for an example "mydomain.com"
Then the CNMAE must be: from "mydomain.com" to "zzz.azurefd.net"

Correct?
You have explained in the doc only if Azure DNS exists, but not if a custom DNS provider is used :)

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

ID: 7290db2b-f5b0-58ab-d840-21dbc0a971c0
Version Independent ID: ebcbe474-7105-8d24-3c48-06a08a309d53
Content: How to add a custom domain - Azure Front Door
Content Source: articles/frontdoor/standard-premium/how-to-add-custom-domain.md
Service: frontdoor
GitHub Login: @duongau
Microsoft Alias: duau

Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.


 Thank you

@DaniellMesquita Do you validate your change, do you plan to migrate your spaces on ENS? Thank you

Closing this PR for now, feel free to open it again once you've done the changes.

Closing this PR for now, feel free to open it again once you've done the changes.


Polish diactric marks

I cannot use Polish diactric marks.

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-09T00:26:45.394Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

code-runner
formulahendry
0.6.14

terminal
formulahendry
0.0.7

mediawiki
jakeboone02
2.1.0

dokuwiki
kirozen
1.4.0

cpptools
ms-vscode
0.10.2

csharp
ms-vscode
1.7.0

bash-debug
rogalmic
0.0.7

Steps to Reproduce:

Please update to the latest version, if you're still having an issue please create another issue with more information and a screenshot.

No change. I can use only "Alt Gr + EASLZX = ĘĄŚŁŻŹ" and I can't use "Alt Gr + O = Ó" this entering to powershell, "Alt Gr + N = Ń" this return "Info: Code language is not supported or defined" and "Alt Gr + C = Ć" this return nothing.

If you're talking about within the terminal you're likely seeing https://github.com/Microsoft/vscode/issues/35901

I need only text editor with dark theme, terminal for me is optional.


BUG: Fix memory corruption in linalg._decomp_update C extension

Fixes #5338
Tested with scipy-1.0 on Windows, Python 3.7.0a3 64-bit, Visual Studio 2017.5.1, Intel Fortran 18, MKL 2018.1

I've verified on a Windows 10 machine and easy to revert if unforeseen problems arise (just in case).
Thank you @cgohlke

Can this be backported to 1.0.x?

Unfortunately, I think that window has passed. My apologies, I should have added the milestone during the merge.

Sorry @cgohlke, you asked some hours after I'd tagged v1.0.1. Can still backport to 1.0.x if it helps - but there's unlikely to be a v1.0.2

Never mind. It's not urgent.


Removed .bind() from lambda

wallet.js did not compile with .bind after the lambda function, and after talking in #dev in Gitter we came to the conclusion to just remove the bind as it's not necessary

Thanks! I removed all the other bind() calls too; there's really no need for it in our codebase now that we're fully onto ES6.


Render new trace server connection status bar item

Add a contextual (secondary, right-side) VS Code status bar item, that renders the current trace server status. Have the extension context pass this status service to each active web-view provider, for them to update the rendered status upon every related message from the view.
Have TraceExplorerOpenedTracesViewProvider pass its status service to the TraceViewerPanel upon using it, in turn. Without this, there are instances where the latest server status isn't shown as expected. Keep this undefined in trace-tree.ts where that is not required.
This makes the server status listener-based. Add this connection listener through the TspClientProvider constructor, thus consistently use the latter across. This means also using it in the TraceViewerContainer constructor, instead of TspClient's directly prior.
Properly stringify (thus parse) the boolean status value, as passing it straight would break the JSON message otherwise.
Explicitly defer showing the item to only once Trace Viewer gets selected for the first time. Like open trace-explorer views which remain shown if unselecting Trace Viewer, keep showing the status item from there on. The VS Code API, anyway, doesn't hold an event that could selectively hide anything upon solely exiting Trace Viewer.
Prefer a standard VS Code status bar item to the side widget approach of theia-trace-extension. This should be more standard for such a connection status display, from a VS Code UI (and UX) perspective. This also allows not having to implement a custom widget that differs from the Trace Viewer panels. -The latter being meant to render real views.
Base this simple design of the status bar item on [1] below. Now, use the warning background for a found server, for such a positive status to be highlighted, thus noticeable. Use the usual red background for a trace server that is not found, highlighting that as an error. No other color can or should be used anyway for status bar items, per [1]. The latter reference also includes [2]'s, used to come up with this change.
[1]https://code.visualstudio.com/api/ux-guidelines/status-bar
[2]https://github.com/microsoft/vscode-extension-samples/blob/main/statusbar-sample/src/extension.ts
Use a short (thus usable) enough statusBarItem.text, along with a tooltip to complement the trace server status narrative.
Overall, with this approach, defer the introduction of the inversify module, used in theia-trace-extension. This would be to inject instances such as objects involved in this change and likely other ones; some being potential singletons.
Slightly refactor a few surrounding blank lines while editing some of these files.
Fixes #33.
Signed-off-by: Marco Miller<EMAIL_ADDRESS>

I going to review this today.

This PR is ready for review again; thanks.


Scaffolding MVC w/views assumes the presence of _ValidationScriptsPartial.cshtml

When scaffolding a new MVC controller with views using Entity Framework, the Create and Edit views reference _ValidationScriptsPartial.cshtml, which is only included in the Individual Auth template, causing this error in other templates:
InvalidOperationException: The partial view '_ValidationScriptsPartial' was not found. The following locations were searched:
 /Views/Movies/_ValidationScriptsPartial.cshtml
 /Views/Shared/_ValidationScriptsPartial.cshtml

Everything seems to work after copying the file from another project.

TFS Bug#233433

This appears to be controlled by the "Reference script libraries" checkbox on the Add Controller dialog.  It's interesting that it won't generate the file if it's missing.

For anyone that lands here in the mean time...  I scaffolded a blank asp.net core project and found that the file should look like:
Add a file _ValidationScriptsPartial.cshtml to Views/Shared folder with the following:
<environment names="Development">
    <script src="~/lib/jquery-validation/dist/jquery.validate.js"></script>
    <script src="~/lib/jquery-validation-unobtrusive/jquery.validate.unobtrusive.js"></script>
</environment>
<environment names="Staging,Production">
    <script src="https://ajax.aspnetcdn.com/ajax/jquery.validate/1.15.0/jquery.validate.min.js"
            asp-fallback-src="~/lib/jquery-validation/dist/jquery.validate.min.js"
            asp-fallback-test="window.jQuery && window.jQuery.validator">
    </script>
    <script src="https://ajax.aspnetcdn.com/ajax/jquery.validation.unobtrusive/3.2.6/jquery.validate.unobtrusive.min.js"
            asp-fallback-src="~/lib/jquery-validation-unobtrusive/jquery.validate.unobtrusive.min.js"
            asp-fallback-test="window.jQuery && window.jQuery.validator && window.jQuery.validator.unobtrusive">
    </script>
</environment>