Add index on release_summary_no_data.date

(And on release_summary.date when it is a table.)
Users also frequently filter on buyer name, in which case it should be added as a column to release_summary_no_data, and this index should include it as a second column.
@camilamila @yolile A few questions:

When you filter on buyer, do you always also filter on release date?
Do you always filter on buyer name, or do you sometimes filter on buyer ID?
Are there any other columns that you frequently filter on?

With the below query, most of the time (84%) is spent merging the two tables (I expect a CROSS JOIN would be slower). It otherwise spends 15% doing a slow sequence scan to filter by date, buyer, and procurementMethodRationale. However, since that scan is a fairly small part of the total time, it's not really worth adding an index.
I think a parallel hash join is already the fastest join method (there's also merge join), so I think with large datasets, it just takes time to do joins. It's joining 200k releases (filtered from ~8M) to 10M awards, with a result of 500k rows.
The tables involved were fully analyzed, and the plan is not far off from the actual. I ran it a second time with very similar results.
https://explain.dalibo.com/plan
Query:

select
    supplier_id,
    release -> 'buyer' ->> 'name' as entity,
    EXTRACT(YEAR FROM date) as year
FROM release_summary AS r
JOIN award_suppliers_summary AS a
ON r.id = a.id
where release -> 'buyer' ->> 'name' IN ('ATLANTICO - GOBERNACION', 'DISTRITO DE BARRANQUILLA', 'ATLANTICO - AREA METROPOLITANA DE BARRANQUILLA',
'ATLANTICO - BARRANQUILLA - ALCALDIA', 'VALLE DEL CAUCA - CALI - ALCALDIA', 'VALLE DEL CAUCA - GOBERNACION',
'CALDAS - GOBERNACION', 'CALDAS - MANIZALES - ALCALDIA', 'MAGDALENA - GOBERNACION', 'MAGDALENA - SANTA MARTA - ALCALDIA',
'BOLIVAR - GOBERNACION', 'BOLIVAR - CARTAGENA - ALCALDIA', 'SANTANDER - BUCARAMANGA - ALCALDIA', 'SANTANDER - GOBERNACION',
'ANTIOQUIA - GOBERNACION', 'ANTIOQUIA - MEDELLIN - ALCALDIA', 'BOGOTA - ALCALDIA - SECRETARIA GENERAL',
'BOGOTA - SECRETARIA DISTRITAL DE LA MUJER', 'BOGOTA - SECRETARIA DISTRITAL DE MOVILIDAD',
'BOGOTA - SECRETARIA DISTRITAL DE SEGURIDAD, CONVIVENCIA Y JUSTICIA - BOGOTA',
'BOGOTA - SECRETARIA JURIDICA DISTRITAL DE BOGOTA- SJD',
'BOGOTA - SUBRED INTEGRADA DE SERVICIOS DE SALUD CENTRO ORIENTE ESE', 'BOGOTA - SUBRED INTEGRADA DE SERVICIOS DE SALUD SUR OCCIDENTE ESE',
'BOGOTA - SUBRED INTEGRADA SERVICIOS DE SALUD SUR ESE', 'CUNDINAMARCA - GOBERNACION', 'CUNDINAMARCA - UNIDAD ADMINISTRATIVA ESPECIAL DE VIVIENDA SOCIAL',
'CUNDINAMARCA-SECRETARIA DE AGRICULTURA Y DESARROLLO RURAL', 'UNIDAD ADMINISTRATIVA ESPECIAL PARA LA GESTION DEL RIESGO DE DESASTRES DE CUNDINAMARCA',
'CUNDINAMARCA - INSTITUTO DE INFRAESTRUCTURA Y CONCESIONES DE CUNDINAMARCA - ICCU',
'META - GOBERNACION', 'META - VILLAVICENCIO - ALCALDIA')
AND date >='2017-01-01' and date <='2019-12-31'
AND release -> 'tender' ->> 'procurementMethodRationale' NOT IN ('Contratos Interadministrativos (Literal C)', 'Contratación de Empréstitos (Literal B)');

Plan:

[
  {
    "Plan": {
      "Node Type": "Gather",
      "Parallel Aware": false,
      "Startup Cost": 1883764.70,
      "Total Cost": 2293707.89,
      "Plan Rows": 728492,
      "Plan Width": 49,
      "Actual Startup Time": 171222.084,
      "Actual Total Time": 1249144.666,
      "Actual Rows": 558463,
      "Actual Loops": 1,
      "Output": ["a.supplier_id", "(((r.release -> 'buyer'::text) ->> 'name'::text))", "(date_part('year'::text, r.date))"],
      "Workers Planned": 4,
      "Workers Launched": 4,
      "Single Copy": false,
      "Shared Hit Blocks": 197303033,
      "Shared Read Blocks": 5683405,
      "Shared Dirtied Blocks": 0,
      "Shared Written Blocks": 246741,
      "Local Hit Blocks": 0,
      "Local Read Blocks": 0,
      "Local Dirtied Blocks": 0,
      "Local Written Blocks": 0,
      "Temp Read Blocks": 0,
      "Temp Written Blocks": 0,
      "Plans": [
        {
          "Node Type": "Hash Join",
          "Parent Relationship": "Outer",
          "Parallel Aware": true,
          "Join Type": "Inner",
          "Startup Cost": 1882764.70,
          "Total Cost": 2219858.69,
          "Plan Rows": 182123,
          "Plan Width": 49,
          "Actual Startup Time": 171215.573,
          "Actual Total Time": 1249010.699,
          "Actual Rows": 111693,
          "Actual Loops": 5,
          "Output": ["a.supplier_id", "((r.release -> 'buyer'::text) ->> 'name'::text)", "date_part('year'::text, r.date)"],
          "Inner Unique": true,
          "Hash Cond": "(a.id = r.id)",
          "Shared Hit Blocks": 197303033,
          "Shared Read Blocks": 5683405,
          "Shared Dirtied Blocks": 0,
          "Shared Written Blocks": 246741,
          "Local Hit Blocks": 0,
          "Local Read Blocks": 0,
          "Local Dirtied Blocks": 0,
          "Local Written Blocks": 0,
          "Temp Read Blocks": 0,
          "Temp Written Blocks": 0,
          "Workers": [
            {
              "Worker Number": 0,
              "Actual Startup Time": 171213.222,
              "Actual Total Time": 1249050.508,
              "Actual Rows": 112789,
              "Actual Loops": 1,
              "Shared Hit Blocks": 39468642,
              "Shared Read Blocks": 1139686,
              "Shared Dirtied Blocks": 0,
              "Shared Written Blocks": 49280,
              "Local Hit Blocks": 0,
              "Local Read Blocks": 0,
              "Local Dirtied Blocks": 0,
              "Local Written Blocks": 0,
              "Temp Read Blocks": 0,
              "Temp Written Blocks": 0
            },
            {
              "Worker Number": 1,
              "Actual Startup Time": 171214.620,
              "Actual Total Time": 1249054.360,
              "Actual Rows": 111530,
              "Actual Loops": 1,
              "Shared Hit Blocks": 39460191,
              "Shared Read Blocks": 1139310,
              "Shared Dirtied Blocks": 0,
              "Shared Written Blocks": 49430,
              "Local Hit Blocks": 0,
              "Local Read Blocks": 0,
              "Local Dirtied Blocks": 0,
              "Local Written Blocks": 0,
              "Temp Read Blocks": 0,
              "Temp Written Blocks": 0
            },
            {
              "Worker Number": 2,
              "Actual Startup Time": 171214.509,
              "Actual Total Time": 1249053.729,
              "Actual Rows": 110115,
              "Actual Loops": 1,
              "Shared Hit Blocks": 39457997,
              "Shared Read Blocks": 1134332,
              "Shared Dirtied Blocks": 0,
              "Shared Written Blocks": 49165,
              "Local Hit Blocks": 0,
              "Local Read Blocks": 0,
              "Local Dirtied Blocks": 0,
              "Local Written Blocks": 0,
              "Temp Read Blocks": 0,
              "Temp Written Blocks": 0
            },
            {
              "Worker Number": 3,
              "Actual Startup Time": 171214.438,
              "Actual Total Time": 1249057.668,
              "Actual Rows": 109139,
              "Actual Loops": 1,
              "Shared Hit Blocks": 39467902,
              "Shared Read Blocks": 1134925,
              "Shared Dirtied Blocks": 0,
              "Shared Written Blocks": 49609,
              "Local Hit Blocks": 0,
              "Local Read Blocks": 0,
              "Local Dirtied Blocks": 0,
              "Local Written Blocks": 0,
              "Temp Read Blocks": 0,
              "Temp Written Blocks": 0
            }
          ],
          "Plans": [
            {
              "Node Type": "Seq Scan",
              "Parent Relationship": "Outer",
              "Parallel Aware": true,
              "Relation Name": "award_suppliers_summary",
              "Schema": "view_data_collection_1853",
              "Alias": "a",
              "Startup Cost": 0.00,
              "Total Cost": 329071.47,
              "Plan Rows": 2535847,
              "Plan Width": 17,
              "Actual Startup Time": 0.224,
              "Actual Total Time": 5168.857,
              "Actual Rows": 2028889,
              "Actual Loops": 5,
              "Output": ["a.id", "a.award_index", "a.supplier_index", "a.release_type", "a.collection_id", "a.ocid", "a.release_id", "a.data_id", "a.supplier", "a.supplier_id", "a.identifier", "a.unique_identifier_attempt", "a.additionalidentifiers_ids", "a.total_additionalidentifiers", "a.link_to_parties", "a.link_with_role", "a.party_index"],
              "Shared Hit Blocks": 0,
              "Shared Read Blocks": 303713,
              "Shared Dirtied Blocks": 0,
              "Shared Written Blocks": 0,
              "Local Hit Blocks": 0,
              "Local Read Blocks": 0,
              "Local Dirtied Blocks": 0,
              "Local Written Blocks": 0,
              "Temp Read Blocks": 0,
              "Temp Written Blocks": 0,
              "Workers": [
                {
                  "Worker Number": 0,
                  "Actual Startup Time": 0.219,
                  "Actual Total Time": 5149.266,
                  "Actual Rows": 2092246,
                  "Actual Loops": 1,
                  "Shared Hit Blocks": 0,
                  "Shared Read Blocks": 62639,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 0,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0
                },
                {
                  "Worker Number": 1,
                  "Actual Startup Time": 0.206,
                  "Actual Total Time": 5207.916,
                  "Actual Rows": 2039412,
                  "Actual Loops": 1,
                  "Shared Hit Blocks": 0,
                  "Shared Read Blocks": 61053,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 0,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0
                },
                {
                  "Worker Number": 2,
                  "Actual Startup Time": 0.204,
                  "Actual Total Time": 5168.755,
                  "Actual Rows": 2000168,
                  "Actual Loops": 1,
                  "Shared Hit Blocks": 0,
                  "Shared Read Blocks": 59883,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 0,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0
                },
                {
                  "Worker Number": 3,
                  "Actual Startup Time": 0.274,
                  "Actual Total Time": 5129.282,
                  "Actual Rows": 1994149,
                  "Actual Loops": 1,
                  "Shared Hit Blocks": 0,
                  "Shared Read Blocks": 59706,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 0,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0
                }
              ]
            },
            {
              "Node Type": "Hash",
              "Parent Relationship": "Inner",
              "Parallel Aware": true,
              "Startup Cost": 1880815.45,
              "Total Cost": 1880815.45,
              "Plan Rows": 155940,
              "Plan Width": 34,
              "Actual Startup Time": 170678.813,
              "Actual Total Time": 170678.815,
              "Actual Rows": 43156,
              "Actual Loops": 5,
              "Output": ["r.release", "r.date", "r.id"],
              "Hash Buckets": 1048576,
              "Original Hash Buckets": 1048576,
              "Hash Batches": 1,
              "Original Hash Batches": 1,
              "Peak Memory Usage": 23456,
              "Shared Hit Blocks": 28168093,
              "Shared Read Blocks": 5265600,
              "Shared Dirtied Blocks": 0,
              "Shared Written Blocks": 246741,
              "Local Hit Blocks": 0,
              "Local Read Blocks": 0,
              "Local Dirtied Blocks": 0,
              "Local Written Blocks": 0,
              "Temp Read Blocks": 0,
              "Temp Written Blocks": 0,
              "Workers": [
                {
                  "Worker Number": 0,
                  "Actual Startup Time": 170678.638,
                  "Actual Total Time": 170678.640,
                  "Actual Rows": 43121,
                  "Actual Loops": 1,
                  "Shared Hit Blocks": 5634817,
                  "Shared Read Blocks": 1053866,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 49280,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0
                },
                {
                  "Worker Number": 1,
                  "Actual Startup Time": 170680.027,
                  "Actual Total Time": 170680.029,
                  "Actual Rows": 43301,
                  "Actual Loops": 1,
                  "Shared Hit Blocks": 5638064,
                  "Shared Read Blocks": 1054986,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 49430,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0
                },
                {
                  "Worker Number": 2,
                  "Actual Startup Time": 170679.585,
                  "Actual Total Time": 170679.587,
                  "Actual Rows": 43044,
                  "Actual Loops": 1,
                  "Shared Hit Blocks": 5632870,
                  "Shared Read Blocks": 1052293,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 49165,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0
                },
                {
                  "Worker Number": 3,
                  "Actual Startup Time": 170679.715,
                  "Actual Total Time": 170679.716,
                  "Actual Rows": 43433,
                  "Actual Loops": 1,
                  "Shared Hit Blocks": 5642544,
                  "Shared Read Blocks": 1053888,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 49609,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0
                }
              ],
              "Plans": [
                {
                  "Node Type": "Seq Scan",
                  "Parent Relationship": "Outer",
                  "Parallel Aware": true,
                  "Relation Name": "release_summary",
                  "Schema": "view_data_collection_1853",
                  "Alias": "r",
                  "Startup Cost": 0.00,
                  "Total Cost": 1880815.45,
                  "Plan Rows": 155940,
                  "Plan Width": 34,
                  "Actual Startup Time": 2.973,
                  "Actual Total Time": 170605.838,
                  "Actual Rows": 43156,
                  "Actual Loops": 5,
                  "Output": ["r.release", "r.date", "r.id"],
                  "Filter": "((r.date >= '2017-01-01 00:00:00'::timestamp without time zone) AND (r.date <= '2019-12-31 00:00:00'::timestamp without time zone) AND (((r.release -> 'tender'::text) ->> 'procurementMethodRationale'::text) <> ALL ('{\"Contratos Interadministrativos (Literal C)\",\"Contratación de Empréstitos (Literal B)\"}'::text[])) AND (((r.release -> 'buyer'::text) ->> 'name'::text) = ANY ('{\"ATLANTICO - GOBERNACION\",\"DISTRITO DE BARRANQUILLA\",\"ATLANTICO - AREA METROPOLITANA DE BARRANQUILLA\",\"ATLANTICO - BARRANQUILLA - ALCALDIA\",\"VALLE DEL CAUCA - CALI - ALCALDIA\",\"VALLE DEL CAUCA - GOBERNACION\",\"CALDAS - GOBERNACION\",\"CALDAS - MANIZALES - ALCALDIA\",\"MAGDALENA - GOBERNACION\",\"MAGDALENA - SANTA MARTA - ALCALDIA\",\"BOLIVAR - GOBERNACION\",\"BOLIVAR - CARTAGENA - ALCALDIA\",\"SANTANDER - BUCARAMANGA - ALCALDIA\",\"SANTANDER - GOBERNACION\",\"ANTIOQUIA - GOBERNACION\",\"ANTIOQUIA - MEDELLIN - ALCALDIA\",\"BOGOTA - ALCALDIA - SECRETARIA GENERAL\",\"BOGOTA - SECRETARIA DISTRITAL DE LA MUJER\",\"BOGOTA - SECRETARIA DISTRITAL DE MOVILIDAD\",\"BOGOTA - SECRETARIA DISTRITAL DE SEGURIDAD, CONVIVENCIA Y JUSTICIA - BOGOTA\",\"BOGOTA - SECRETARIA JURIDICA DISTRITAL DE BOGOTA- SJD\",\"BOGOTA - SUBRED INTEGRADA DE SERVICIOS DE SALUD CENTRO ORIENTE ESE\",\"BOGOTA - SUBRED INTEGRADA DE SERVICIOS DE SALUD SUR OCCIDENTE ESE\",\"BOGOTA - SUBRED INTEGRADA SERVICIOS DE SALUD SUR ESE\",\"CUNDINAMARCA - GOBERNACION\",\"CUNDINAMARCA - UNIDAD ADMINISTRATIVA ESPECIAL DE VIVIENDA SOCIAL\",\"CUNDINAMARCA-SECRETARIA DE AGRICULTURA Y DESARROLLO RURAL\",\"UNIDAD ADMINISTRATIVA ESPECIAL PARA LA GESTION DEL RIESGO DE DESASTRES DE CUNDINAMARCA\",\"CUNDINAMARCA - INSTITUTO DE INFRAESTRUCTURA Y CONCESIONES DE CUNDINAMARCA - ICCU\",\"META - GOBERNACION\",\"META - VILLAVICENCIO - ALCALDIA\"}'::text[])))",
                  "Rows Removed by Filter": 1693858,
                  "Shared Hit Blocks": 28168093,
                  "Shared Read Blocks": 5265600,
                  "Shared Dirtied Blocks": 0,
                  "Shared Written Blocks": 246741,
                  "Local Hit Blocks": 0,
                  "Local Read Blocks": 0,
                  "Local Dirtied Blocks": 0,
                  "Local Written Blocks": 0,
                  "Temp Read Blocks": 0,
                  "Temp Written Blocks": 0,
                  "Workers": [
                    {
                      "Worker Number": 0,
                      "Actual Startup Time": 0.656,
                      "Actual Total Time": 170605.778,
                      "Actual Rows": 43121,
                      "Actual Loops": 1,
                      "Shared Hit Blocks": 5634817,
                      "Shared Read Blocks": 1053866,
                      "Shared Dirtied Blocks": 0,
                      "Shared Written Blocks": 49280,
                      "Local Hit Blocks": 0,
                      "Local Read Blocks": 0,
                      "Local Dirtied Blocks": 0,
                      "Local Written Blocks": 0,
                      "Temp Read Blocks": 0,
                      "Temp Written Blocks": 0
                    },
                    {
                      "Worker Number": 1,
                      "Actual Startup Time": 4.646,
                      "Actual Total Time": 170604.210,
                      "Actual Rows": 43301,
                      "Actual Loops": 1,
                      "Shared Hit Blocks": 5638064,
                      "Shared Read Blocks": 1054986,
                      "Shared Dirtied Blocks": 0,
                      "Shared Written Blocks": 49430,
                      "Local Hit Blocks": 0,
                      "Local Read Blocks": 0,
                      "Local Dirtied Blocks": 0,
                      "Local Written Blocks": 0,
                      "Temp Read Blocks": 0,
                      "Temp Written Blocks": 0
                    },
                    {
                      "Worker Number": 2,
                      "Actual Startup Time": 3.068,
                      "Actual Total Time": 170605.819,
                      "Actual Rows": 43044,
                      "Actual Loops": 1,
                      "Shared Hit Blocks": 5632870,
                      "Shared Read Blocks": 1052293,
                      "Shared Dirtied Blocks": 0,
                      "Shared Written Blocks": 49165,
                      "Local Hit Blocks": 0,
                      "Local Read Blocks": 0,
                      "Local Dirtied Blocks": 0,
                      "Local Written Blocks": 0,
                      "Temp Read Blocks": 0,
                      "Temp Written Blocks": 0
                    },
                    {
                      "Worker Number": 3,
                      "Actual Startup Time": 4.218,
                      "Actual Total Time": 170605.974,
                      "Actual Rows": 43433,
                      "Actual Loops": 1,
                      "Shared Hit Blocks": 5642544,
                      "Shared Read Blocks": 1053888,
                      "Shared Dirtied Blocks": 0,
                      "Shared Written Blocks": 49609,
                      "Local Hit Blocks": 0,
                      "Local Read Blocks": 0,
                      "Local Dirtied Blocks": 0,
                      "Local Written Blocks": 0,
                      "Temp Read Blocks": 0,
                      "Temp Written Blocks": 0
                    }
                  ]
                }
              ]
            }
          ]
        }
      ]
    },
    "Planning Time": 0.560,
    "Triggers": [
    ],
    "Execution Time": 1249201.431
  }
]

I've added a couple more gather workers (the replica's configuration was to cap gather workers at 4, which makes sense for an application server, but as a data warehouse server, it's better to do half the number of cores).

Just noting that, in PostgreSQL 11 at least, a merge join cannot be parallelized, so hash join will be fastest. A cross join always uses a nested loop join, which also cannot be parallelized.
In other words, the plan is more or less optimal, and I'm not sure that there is much to improve in terms of the server configuration (other than getting a server with more CPUs, so that we can have more workers).
Obviously, the join will be faster if we had fewer than 10M candidates in the award_suppliers_summary table, but I'm not sure what we would do to filter that table.
The remaining option is to avoid a join to begin with. We can perhaps add a calculated field to release_summary that contains the full list of suppliers. (We would need to decide whether to identify the suppliers using supplier_id, unique_identifier_attempt or identifier). @camilamila @yolile What do you think?


DropDown theming issues

in Carbon, the hover background color is not consistent
in Carbon, dropdown retains focus after it is closed
hover effect has no spacing relative to the input timestamp

a few more:

PatternFly shows focus outline on kebab
Carbon's :after rule for its kebab isn't themed
Carbon's background colors aren't correct when the dropdown is open


Incorporated optimizations and improvements

@bruceyang @spencerromo @xiaoyang
I faced some difficulties when debugging the code, so I changed the try except for more specific exceptions.

Improved function signatures with type hints for clarity.
Enhanced variable initialization for efficiency.
Utilized a 'max_steps' variable for loop iteration.
Clarified early termination conditions with a descriptive message.
Introduced separation of concerns for code modularity.
Raised more specific exceptions for error handling.
Provided informative error messages for better debugging.

These changes optimize the code for readability, maintainability, and efficiency.

thanks.


[2.2] Fix for assembly version not matching package version

Revert exclusions added in https://github.com/aspnet/Extensions/pull/1259 to NuGet package verifier which suppresses errors that should not have been ignored.
For future reference, assembly version mismatch errors should be taken seriously and not ignored. If we get assembly versions wrong, we can end up with all sorts of nasty problems with binding redirects, forced downgrades, runtime errors, shared framework mismatches, etc.
Required to unblock @BrennanConroy's work.
Description
Normally, ASP.NET Core packages have the assembly version match the NuGet package version (at least the first three parts, x.y.z). Each patch of should have incremented the assembly versions but didn't. For example, the 2.2.4 release of Microsoft.Extensions.Configuration.KeyPerFile contains an an assembly with version <IP_ADDRESS>, not <IP_ADDRESS>.
Customer impact
This will prevent us from having problems in the future if we have to patch any assemblies for a 2nd time.
Regression?
Yes.
Risk
Low. This restores behavior we had in 2.1 which somehow went missing during source code reorganization. It should have been caught earlier, but devs started suppressing the errors instead of fixing them.

Approved for 2.2.5


The --recursive has the opposite action as expected

The default config of s2i to clone repositories recursively (--recursive) is supposed to be set to true.
Actually it is not the case, even passing explicitly --recursive=true.
But if you pass  --recursive=false, then it is actually recursive !
I believe this option should be named --disable-submodules with a default to false. It would be much more consistent with the behaviour and the code too (as in the code it looks like it is referred to as DisableRecursive)...

i agree we should introduce a --disable-submodules flag w/ a default of false, when set to true we'll either use --recursive or submodule init logic depending whether --ref was also specified, and we'll remove the recursive flag.


⚠ Bofrost.at has degraded performance

In e6cf182, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2115 ms

Resolved: Bofrost.at performance has improved in 18abc08.


Create spss_3.0_odpi_v1.0.1

I think it's just a copy paste error but question 'd' (which ODPi compliant distros did you test with?) isn't answered.

Thanks @sumalaika !
@alanfgates I updated the doc - please review and merge if acceptable.

LGTM. @alanfgates lets merge so we have no outstanding PRs for the release today.


Update rule criteria operators

The definition was missing new regex, iregex and matchwildcard operators. In addition to that, it still contained deprecated matchregex operator.
In the future, we should expose meta / view API in st2 to retrieve available operators so we don't need to hard-code it here (perhaps GET /v1/rules/views/operators or similar).
Caught and reported by Lisa Bekdache on Slack.

We absolutely should not create multiple endpoints exposing different specifics of particular st2 installation. There should be a single endpoint, something like /v1/settings that would tell everything that client might need to know: st2 version, operators, auth backends, things like mask_secret, max_page_size, debug, list of available runners, maybe even their versions, presence of mistral, stream, st2auth. If we are to gather all this information from different endpoints, it might take us dozens of requests to do so and some of it might be critical to the render further decreasing UI responsiveness.


UPDATE UPSERT information after operation

Is there a way to know whether update upsert created the record or updated it?
We have a use case that needs separate operations on create and match - something like Neo4j's Merge - ON CREATE & ON MATCH.
How can we do such a thing in orientdb?

What about Version of the record? Can you use that?

Right that is some information, but then again it would require me to do 2 queries. Wouldn't it defeat the purpose of UPSERT all together?
If I want to set a particular field only on 'modify' or 'create' and not both then I would require 2 queries for this, it would be the same case as selecting the count of existing records and if it is zero then do something for create else update.
Am I right? Any other alternative than this?
On a side-note..while googling about this I came across multiple requests by other users for this same feature. Can we expect any work on this in the near future? It is quite an important distinction between Orientdb and it's competitors that do offer this query support.

@kowalot Thanks for the quick response, I just ran a few tests, and version no. is not very reliant for this purpose. Consider the following scenario:
select from Email where email='test100.hello.com'
Returned 0 records
create vertex Email set email='test100.hello.com'
Returned 1 record @version: 2
delete vertex Email where email='test100.hello.com'
returned 1 row @version: 0 ( successfully deleted )
create vertex Email set email='test100.hello.com'
Returned 1 record @version: 3
Even though the record was newly created, it incremented the version as if it was updated.

As OrientDB user for your purposes I would reconsider
UPDATE SET ..... UPSERT RETURN BEFORE
You won't get previous record content for record creations, based on that you can change logic.
Additionaly to that I would use
UPDATE SET X = IFNULL(X,1) ....
which updates only non-present field (for new record)
Does it solve partially your needs?

It does, that's a great help.
Thanks :)
Do also consider a simpler syntax for a combination of these commands as I guess this is a nice and important feature.


http plugin set Access-Control-Allow-Credentials header

Is your feature request related to a problem? Please describe.
I'm trying to use the http input plugin to collect logs from a client web app.  I want to do my best to send any remaining logs from the client when the window is closed, so am attempting to use the sendBeacon api to do so:
  let headers = {
    type: 'application/json'
  };
  let blob = new Blob([JSON.stringify(logs)], headers);
  window.navigator.sendBeacon(logUrl, blob)

unfortunately, it sets credentials: "include"  (which is not customizable) on the request, and I get an error on the console:
Access to resource at 'https://app.westus2.stage.stage.hoy.lu/logs/login.app' from origin 'http://localhost:3002' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: The value of the 'Access-Control-Allow-Credentials' header in the response is '' which must be 'true' when the request's credentials mode is 'include'.
Describe the solution you'd like
I believe all that I need is a way to have fluentd set the Access-Control-Allow-Credentials header to 'true' on the response and everything will work as expected.

How about setting up additional headers via headers parameter on built-in out_http plugin?
ref: https://docs.fluentd.org/output/http#headers

And this issue ticket is NOT USER FORUM but for management bugs of Fluentd.
Please send your question into discuss.fluentd.org for the next time? Thanks.

Sorry, I was not aware there was an input and output plugin called http; this feature request is for the input plugin which doe not have any headers parameter documented.
I will edit the original feature request description to be more clear about input vs output.

OK, thanks for the detailed description. I reopened this issue.

refs:

https://w3c.github.io/beacon/
https://fetch.spec.whatwg.org/#http-access-control-allow-credentials
https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS#requests_with_credentials
https://github.com/fluent/fluentd/blob/5bc4c6bd76ca88ba79ab24f64dacf48de793a454/lib/fluent/plugin/in_http.rb#L492-L494
https://github.com/fluent/fluentd/blob/5bc4c6bd76ca88ba79ab24f64dacf48de793a454/lib/fluent/plugin/in_http.rb#L577-L579

One PoC implementation:
diff --git a/lib/fluent/plugin/in_http.rb b/lib/fluent/plugin/in_http.rb
index 2af6835f..e7c81215 100644
--- a/lib/fluent/plugin/in_http.rb
+++ b/lib/fluent/plugin/in_http.rb
@@ -82,6 +82,8 @@ module Fluent::Plugin
     config_param :dump_error_log, :bool, default: true
     desc 'Add QUERY_ prefix query params to record'
     config_param :add_query_params, :bool, default: false
+    desc 'Additional headers to the HTTP response'
+    config_param :headers, :hash, default: nil

     config_section :parse do
       config_set_default :@type, 'in_http'
@@ -279,7 +281,7 @@ module Fluent::Plugin
     private

     def on_server_connect(conn)
-      handler = Handler.new(conn, @km, method(:on_request), @body_size_limit, @format_name, log, @cors_allow_origins, @add_query_params)
+      handler = Handler.new(conn, @km, method(:on_request), @body_size_limit, @format_name, log, @cors_allow_origins, @add_query_params, @headers)

       conn.on(:data) do |data|
         handler.on_read(data)
@@ -356,7 +358,7 @@ module Fluent::Plugin
     class Handler
       attr_reader :content_type

-      def initialize(io, km, callback, body_size_limit, format_name, log, cors_allow_origins, add_query_params)
+      def initialize(io, km, callback, body_size_limit, format_name, log, cors_allow_origins, add_query_params, headers)
         @io = io
         @km = km
         @callback = callback
@@ -367,6 +369,7 @@ module Fluent::Plugin
         @cors_allow_origins = cors_allow_origins
         @idle = 0
         @add_query_params = add_query_params
+        @headers = headers
         @km.add(self)

         @remote_port, @remote_addr = io.remote_port, io.remote_addr
@@ -605,6 +608,7 @@ module Fluent::Plugin
       end

       def send_response(code, header, body)
+        header = @headers.merge(header)
         header['Content-Length'] ||= body.bytesize
         header['Content-Type'] ||= 'text/plain'.freeze

diff --git a/test/plugin/test_in_http.rb b/test/plugin/test_in_http.rb
index 3711218e..b82523d2 100644
--- a/test/plugin/test_in_http.rb
+++ b/test/plugin/test_in_http.rb
@@ -944,6 +944,23 @@ class HttpInputTest < Test::Unit::TestCase
     assert_equal events, d.events
   end

+  def test_headers
+    header = "Access-Control-Allow-Credentials"
+    d = create_driver(config + "headers {\"#{header}\": true}")
+    headers = {"#{header}" => true}
+    assert_equal headers, d.instance.headers
+
+    time = event_time("2011-01-02 13:14:15 UTC")
+    record = ["tag1", time, {"a"=>1}]
+    d.run do
+      res = post("/", {"json"=>record.to_json, "time"=>time.to_s})
+      res.each_name do |name|
+        next if name != header.downcase
+        assert_equal res.fetch(name), d.instance.headers[header].to_s
+      end
+    end
+  end
+
   $test_in_http_connection_object_ids = []
   $test_in_http_content_types = []
   $test_in_http_content_types_flag = false

Always adding Access-Control-Allow-Credentials is inappropriate.
Please check the specification I listed above.


Adds icons for various design tools

Icon Request

Icon name: design tools
Use case: on UI design interfaces

(The complete set designed is shown on the 100% view, I've chosen not to include all of them, since some are bit too random).
There's a naming conflict with the current scale icon, not sure how this should be resolved, that one probably should have been called scales.

Woow these icons look really cool 🚀!
@ericfennis Thanks! What do you think about the naming conflict?


Feature request: USE_FOLDERS and FOLDER

Brief Issue Summary
I have not seen any other issue requesting this feature.
Please support the folder feature of CMake. Including CTest alone will create a lot of targets, which are automatically grouped in a folder when USE_FOLDERS is enabled. This feature is already supported by VS:

The value of the FOLDER property is accessible in the File API via the root level folder field. I'm not sure about server mode.

I remember there has been an issue regarding this for a while (https://github.com/microsoft/vscode-cmake-tools/issues/491) though I don't know when this will be planned.

You are right. I tried searching for a similar issue, but somehow I didn't find that.


Support for gui themes

Please apply theme colors from guibg and guifg

It would be nice if this software could read the ginit.vim file and automatically apply the settings from it at startup.
Using gsettings is not a good idea unless there is an entry specifically for nvim-pygtk3 (which, so far, I haven't found).


New Generated feathersjs app failed when start

I have tried switch between nodejs versions from version 12 upwards and still have this error.
I'm new to feathers js, i dont understand the error and dont know where to fix it. Please any help
info: Feathers application started on http://localhost:3030
internal/crypto/keys.js:299
throw new ERR_INVALID_ARG_TYPE(
^
TypeError [ERR_INVALID_ARG_TYPE]: The "key" argument must be one of type Buffer, TypedArray, DataView, string, or KeyObject. Received type object
at prepareSecretKey (internal/crypto/keys.js:299:13)
at new Hmac (internal/crypto/hash.js:101:9)
at Object.createHmac (crypto.js:139:10)
at createHMAC (C:**\Desktop\DreamFox\node_modules\pg\lib\sasl.js:133:17)
at Hi (C:**\Desktop\DreamFox\node_modules\pg\lib\sasl.js:137:13)
at Object.continueSession (C:**\Desktop\DreamFox\node_modules\pg\lib\sasl.js:32:24)
at Client._handleAuthSASLContinue (C:**\Desktop\DreamFox\node_modules\pg\lib\client.js:257:10)
at Connection.emit (events.js:210:5)
at C:**\Desktop\DreamFox\node_modules\pg\lib\connection.js:115:12
at Parser.parse (C:**\Desktop\DreamFox\node_modules\pg-protocol\dist\parser.js:40:17) {
code: 'ERR_INVALID_ARG_TYPE'
}
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: node server/
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:*******\npm-cache_logs\2021-02-04T16_32_52_210Z-debug.log

Try to update and reinstall your node modules.

i just deleted node_modules folder and npm install the whole package, still the error persist.

This is probably related to https://github.com/brianc/node-postgres/issues/2371 and most likely a configuration problem with your database. Make sure the database has been created and you set the correct password in the connection string.

Thanks for the help! this been my first time getting my hands on feathersjs,  i missed passing user:password to the connection string. Finally the app is up and running now.


update-design-file

General Checks

[x] the branch is up to date with main/master
[x] the code works when pulled and run locally
[ ] All CI checks pass
[ ] all conflicts are resolved (if any)
[x] PR has a descriptive title
[x] PR has appropriate labels and milestones for easy identification
[x] PR it is assigned to the owner
[x] reviewers are assigned
[x] the PR contributes only one focused change
[ ] It is in the appropriate column in the project board (if necessary)
[x] has short and clear description
[x] is linked to an issue (if it is related)
[x] feedback is addressed (if any and if it is appropriate feedback.)

Markdown

[x] the markdown source is formatted
[x] spelling and grammar is correct in all text
[x] The markdown looks correct when you preview the file
[x] all links and images work

HTML

[ ] pages besides /index.html are stored in /pages
[ ] the code is well-formatted
[ ] the HTML is valid
[ ] there are no inline styles (example: style='color: red;')
[ ] there are no <style> tags with CSS, all styles are hrefs
[ ] there is no inline JavaScript (example: onclick='doSomething()')
[ ] there are no <script> tags with JS, all JS is in an separate file
[ ] ids are used for JavaScript only, not for CSS
[ ] semantic tags are used
[ ] spelling and grammar is correct in all site content

CSS

[ ] the code is well-formatted
[ ] passes all the linting checks

Deare @Neha, maybe it's better to write a title "All about trees''? Because we've  mentioned everywhere that our web site's title - All about trees.
also, have we decided to add "Search"? if yes, Search line should be in the separate "box"?
You see, there should be individual box for each type of information. for ex., one block for the title, one block for navigation, one block for introduction (ex., some info for nature-lovers), one block for main info (how to grown a tree, why it's important ect.), one block     of addition info (ex., links to other sites), one block  for extra links (you've done it).


Processes with no ports

Issue from redmine
http://smithers.aquaq.co.uk/issues/2488

This update means that if a process's port is not found in process.csv, then the process will no longer start.

Ok I don't think this change is correct. We don't want processes to start up without a host and port. If the user supplies proctype+procname but no port, then it will attempt to read the port from the process.csv file. The change we need here is that if the user supplies proctype+procname and no port can be found in process.csv (based on proctype+procname+host) then the process should fail to start.

Updated description as we changed the attempt on fixing the issue.

Can you document some test cases on here with test evidence?
So

if proctype+procname specified, it needs to use it's host to work out which port it should be on from process .csv and set it to that port
if as (1) but can't find the port using proctype+procname+host then it fails to start
If proctype+procname+port specified then it just starts using those
if port specified, it looks up using hostname to find it's proctype+procname. If it finds them, it sets them. If it doesn't, it fails.

I've just tested (1) by renaming the specified host in process.csv and the process starts anyway, will need to address that.

All the tests pass except for 2 when the hostname is empty. It does work in the case where hostname is not empty but does not match host.

@jonnypress is that acceptable behaviour for (2)?

@ThomasSmyth I think that's ok (sorry only just going through this). So if the hostname is empty in the config file the process just starts up? I don't think that's necessarily bad


Docs: incomplete or wrong example

In the docs section: https://keras.io/getting-started/functional-api-guide/#multi-input-and-multi-output-models
The example is either incomplete or wrong. The following complete example that generates data fails. Examples, should be complete with random data.

import numpy as np
from tensorflow import keras
from tensorflow.keras.layers import Input, Embedding, LSTM, Dense
from tensorflow.keras.models import Model

# Headline input: meant to receive sequences of 100 integers, between 1 and 10000.
# Note that we can name any layer by passing it a "name" argument.
main_input = Input(shape=(100,), dtype='float32', name='main_input')

# This embedding layer will encode the input sequence
# into a sequence of dense 512-dimensional vectors.
x = Embedding(output_dim=512, input_dim=10000, input_length=100)(main_input)

# A LSTM will transform the vector sequence into a single vector,
# containing information about the entire sequence
lstm_out = LSTM(32)(x)
auxiliary_output = Dense(1, activation='sigmoid', name='aux_output')(lstm_out)

auxiliary_input = Input(shape=(5,), name='aux_input')
x = keras.layers.concatenate([lstm_out, auxiliary_input])

# We stack a deep densely-connected network on top
x = Dense(64, activation='relu')(x)
x = Dense(64, activation='relu')(x)
x = Dense(64, activation='relu')(x)

# And finally we add the main logistic regression layer
main_output = Dense(1, activation='sigmoid', name='main_output')(x)

model = Model(inputs=[main_input, auxiliary_input], outputs=[main_output, auxiliary_output])

X = np.random.randn(12, 100)
Z = np.random.randn(1, 5, 1)

model.predict({'main_input': X, 'aux_input': Z})

ValueError: Error when checking input: expected aux_input to have 2 dimensions, but got array with shape (1, 5, 1)
I can update the docs if someone knows the correction.

I think that there should be model.fit before model.predict.
Also, change Z to Z=np.random.randn(12,5) since the shape of auxiliary_input is (5,), that means a vector of dimension 5 (not a matrix 5x1)

This code "runs" without out the fit:
import numpy as np
from tensorflow import keras
from tensorflow.keras.layers import Input, Embedding, LSTM, Dense
from tensorflow.keras.models import Model

# Headline input: meant to receive sequences of 100 integers, between 1 and 10000.
# Note that we can name any layer by passing it a "name" argument.
main_input = Input(shape=(100,), dtype='float32', name='main_input')

# This embedding layer will encode the input sequence
# into a sequence of dense 512-dimensional vectors.
x = Embedding(output_dim=512, input_dim=10000, input_length=100)(main_input)

# A LSTM will transform the vector sequence into a single vector,
# containing information about the entire sequence
lstm_out = LSTM(32)(x)
auxiliary_output = Dense(1, activation='sigmoid', name='aux_output')(lstm_out)

auxiliary_input = Input(shape=(5,), name='aux_input')
x = keras.layers.concatenate([lstm_out, auxiliary_input])

# We stack a deep densely-connected network on top
x = Dense(64, activation='relu')(x)
x = Dense(64, activation='relu')(x)
x = Dense(64, activation='relu')(x)

# And finally we add the main logistic regression layer
main_output = Dense(1, activation='sigmoid', name='main_output')(x)

model = Model(inputs=[main_input, auxiliary_input], outputs=[main_output, auxiliary_output])
model.compile(optimizer=keras.optimizers.Adam(), loss=keras.losses.MeanSquaredError())

X = np.round(np.abs(np.random.rand(12, 100) * 100))
Z = np.random.randn(12, 5)
Y0 = np.random.randn(12, 1)
Y1 = np.random.randn(12, 1)

# model.fit([X, Z], [Y0, Y1])
pred = model.predict([X, Z])

Closing this issue since the associated PR has been merged. Thanks!


upgrade pydftracer package

Hi @zhenghh04 and @hariharan-devarajan,
This PR will update pydftracer package from 1.0.2 to 1.0.6.
Note that pydftracer package 1.0.6 still does not include the new API of dlp.iter(self, func, name, iter_name) that is needed for torch data loader but it is not a big of a deal for now.
Thanks!

@rayandrew Could you fix the github action issue?


feat: add log function

使用npmlog第三方库封装log函数

手动点错关闭了上一个pr,重新提交下

Hello, Thank you for your contribution，because the cli package is being refactored, can you submit your code in the refactor/cli branch?


net/http: http.Transport: support HSTS (Strict-Transport-Security)

Hello,
It doesn't seem that net/http's http.Transport supports HSTS (HTTP Strict-Transport-Security, http://tools.ietf.org/html/rfc6797). Is it something we could add?
Initially, I'm thinking just the http->https upgrade bit and not certificate pins. Also, just dynamic mode and not static configuration like Chromium has.
That is: if we receive an HSTS header we remember it in the current http.Transport and upgrade any future http request to https, but terminating the program means we forget about it.

This doesn't seem relevant to the net/http package because we don't receive raw user strings like foo.com in an Omnibar from Chrome or Firefox. We work in terms of complete URLs, so it's up to the calling application to do HSTS if they want.
Plus, any HSTS state should usually persist longer than the lifetime of a given Go process, so it necessarily involves writing state somewhere, which is outside the scope of the net/http package.
Tentatively closing this bug, unless I'm convinced otherwise.
/cc @agl for opinions, if any.


Add Data Structure Implementations

Add projects under <language>/Data-structures/ for example a linked list implementation of a queue in C++ will go under the folder C++/Data-Structures/Queue/

hii, can you please assign to me this issue?
Thank you

@SaloniK17 No need for assignment you can start working, just mention this issue in your PR.

@abhpd Can I add "Implementation of a Queue using 2 stacks" in C++? Can you assign me this issue?

please review the PR #174  @abhpd

@abhpd please review my pull request #179

@SaloniK17 No need for assignment you can start working, just mention this issue in your PR.

hi @abhpd I want to implement a doubly linked list Can you please assign me this issue

Hi! I would love to work on this issue? can you please assign it to me?

@tanvideshmukh29 No need for assignment you can start working, just mention this issue in your PR

Hii, I'm a beginner in this can you tell me what to do exactly here.
I'll be very grateful to you

@abhi26ak Welcome to Open Souce! Please read this: https://github.com/abhpd/hacktoberfest2021/blob/main/README.md

I can implement stack using array and linked list. Please assign this task to me.

Please Assign this to me ,I  would like to work on DP in C++

Sir,I have added C++ code for implemetation of stacks using two queues in Data structures pls accept my PR

I have added implementation of Segment Tree for sum in C++ under Data structure. Please review and merge PR #647

please merge PR #573
Thank you

Please merge my pr #906

Hey, can I add String algorithms in java?

Sir I want to add Stack Implementation using vector

Please assign this issue to me. Thank you!

Sir @abhpd , please review my PR #1312
This PR contains the following implementation of Stack functionalities in C.
isEmpty - To check whether the stack is empty.
isFull - To check whether the stack is full.
peek - To get the topmost element of the stack.
push - To push element in the stack.
pop - To remove the topmost element in the stack.
display - To print all the stack elements.

@abhpd Sir please accept my pull request #1451 . It has the implementation for BFS

@abhpd Sir I have also added implementation for DFS

@abhpd Please review this PR - https://github.com/abhpd/hacktoberfest2021/pull/1550


[MetaSchedule] Support Tuning w/ No Cost Model & Fix Integration Test

Previously even for replay trace & replay func strategy, an xgboost cost model will be created and retrained after each iteration of building and running. This PR introduces a none cost model option to avoid creating cost model. Also fixed the integration test for correctness verification by using the correct tvm device.

CC @masahi @junrushao


Debug log should be as similar as possible to stdout log

I believe the debug log should be as similar as possible to the stdout log:

debugFile should allow relative paths (currently requires absolute path)

debugFile: https://github.com/aspnet/IISIntegration/blob/release/2.2/src/AspNetCoreModuleV2/CommonLib/debugutil.cpp#L138
stdout: https://github.com/aspnet/IISIntegration/blob/release/2.2/src/AspNetCoreModuleV2/RequestHandlerLib/FileOutputManager.cpp#L85

debugFile should have a default value like .\logs\debug (similar to .\logs\stdout)
debugFile should be a prefix, and ANCM should append a timestamp and process ID.

Ideally, the following config would be sufficient to enable both stdout and debug logs:
<aspNetCore stdoutLogEnabled="true">
  <handlerSettings>
    <handlerSetting name="debugLevel" value="4" />
  </handlerSettings>
</aspNetCore>

Currently, the following is required:
<aspNetCore stdoutLogEnabled="true">
  <handlerSettings>
    <handlerSetting name="debugLevel" value="4" />
    <handlerSetting name="debugFile" value="C:\path\to\app\logs\debug.txt" />
  </handlerSettings>
</aspNetCore>

Default stdout file is ".\aspnetcore-stdout according to schema.
Otherwise, I agree, we need to cleanup debug logging a bit.

I agree that's the default in schema, but WebSDK sets default to %home%\LogFiles for App Service and .\logs\stdout otherwise.  I would argue the values set by WebSDK are the effective defaults for most customers.
https://github.com/aspnet/IISIntegration/blob/e1f1705d11d97560c574dbe68a0b560441470f58/src/AspNetCoreModuleV1/AspNetCore/aspnetcore_schema.xml#L22
https://github.com/aspnet/IISIntegration/blob/e1f1705d11d97560c574dbe68a0b560441470f58/src/AspNetCoreModuleV2/AspNetCore/aspnetcore_schema_v2.xml#L24
https://github.com/aspnet/websdk/blob/0c47545ea675938160725e30dab7f0efdc58d4cb/src/Publish/Microsoft.NET.Sdk.Publish.Tasks/WebConfigTransform.cs#L118

Most of these things are done except debugFile should be a prefix, and ANCM should append a timestamp and process ID..
Do you feel strongly about it or can I close the issue?

Is debugFile intended for customers to use?  If so, is it something they would only enable temporarily, meaning it's OK to use the file name without appending timestamp/ID?  Or is it something they might want to keep enabled long-term like stdout?

It's something to be enabled temporarily to get more information about ANCM failures.

Do you think it's better if the filename does not have the time/id appended, or do you just not think it's worth the effort?

It's better for what we use it, for example when piping log for tests you don't need to figure out timestamps and generate file name you just get logs written where you expect them.

OK, the current behavior seems fine then, we can revisit based on customer feedback.


Fixed localized strings if app language and system language are different

Resolves #4407

This feels like you're trying to fix a lot of problems in one pull request. Can you please pick one problem at a time and create a PR to fix that? Please also add a description to describe the problem you're solving.

The commits resolve the following small issues:

Wrong title of incoming server settings (see issue description 5.5)
Quotation of answers or forwarded messages (see issue description 5.6 and 5.7)
Wrong title of unified inbox on startup
Wrong title of unified inbox after K9 language has changed
Wrong title on new account setup
Removing dead code

I can split my commits in six pull requests, each referencing issue 4407. Would you prefer that?

Yes, it's easier to deal with one issue at a time. You can group the ones that can be fixed using Activity.setTitle() into one PR.
For the rest please create one pull request per problem. Also include instructions on how to reproduce the issue (including the Android version you used for testing).

I opened pull requests https://github.com/k9mail/k-9/pull/5318, https://github.com/k9mail/k-9/pull/5319, https://github.com/k9mail/k-9/pull/5320 and https://github.com/k9mail/k-9/pull/5321.
After executing gradle clean I could not reproduce my finding 3 (wrong title of unified inbox on startup) so please ignore it. All issues where tested with Android 10 (API version 29).


libusb failure in seek_thermal

Hello,
at first, I wanna thank you for this lib. I use it with a seek_thermal_pro and it works perfectly. Unfortunately, I have an issue when I connect a seek_thermal camera. When I start the see_test example it returns:
Error: libusb init failed: LIBUSB_ERROR_ACCESS
Error: open failed
failed to open seek cam
Cheers,
Phil

Did you set the permissions correctly for the device?

I just realized. I havent. Thank you very much it is working fine now.


DOC-3054: Add docs on AWS DMS

Fixes DOC-3054

This change is

@otan Thanks for the review! I'm having some difficulty getting my DMS setup working. Would you be at all opposed if I used the screenshots you supplied in that document?

@otan Ignore last statement. I got full access now. Generating screenshots.

thanks for update on screenshot, lgtm

@otan and @ajstorm once you approve i think we can merge this


Can't get Python models running on 1.3.0b0

Describe the bug
I am trying to run Python models using the dbt-dtabricks adapter but get some errors.
Steps To Reproduce

Create some models in your dbt project.
With only SQL models, they run correctly.
When adding a Python model, the run fails with the error:

Unhandled error while executing model.my_project.my_model
Python model doesn't support SQL Warehouses

My profiles.yml is configured with the http_path config.

If I add a cluster in addition to http_path, I still get the error.
If I keep the cluster and remove http_path I get the error:

dbt.exceptions.DbtProfileError: Runtime Error
`http_path` must set when using the dbsql method to connect to Databricks

Expected behavior
The Python model runs correctly
System information
The output of dbt --version:
Core:
  - installed: 1.3.0-b2
  - latest:    1.2.1    - Ahead of latest version!

Plugins:
  - databricks: 1.3.0b0 - Ahead of latest version!
  - snowflake:  1.3.0b2 - Ahead of latest version!
  - spark:      1.3.0b2 - Ahead of latest version!

Additional context
Is there specific config that needs to filled-in in profiles.yml to work with Python models?

Right, so far Python model only works on the multi-purpose clusters.
The discussion to provide a way to have a separate config to run on the multi-purpose clusters even when the main connection is to SQL Warehouse. dbt-labs/dbt-spark#444.

@ueshin  Just to clarify — for folks who want to use Python models with dbt-databricks today — they need to specify an additional cluster_id field in their profiles.yml. Does that sound right?
I left a new comment in https://github.com/dbt-labs/dbt-spark/issues/444#issuecomment-1253344541. I think you could use the pseudo-code there to allow users to specify cluster_id / cluster as a model-level configuration, and pull that into the submit_python_job, in case they have not added it as configuration in profiles.yml, or specified it within their http_path to be extracted.
Also a thought about reconciling the teensy naming difference between cluster + cluster_id.

The cluster_id is extracted from the provided http_path if it's for an all-purpose cluster, so the users don't need to specify the additional config.
The cluster_id will not be exposed to users.
I'm hoping users could specify the http_path in the model config instead of cluster or cluster_id to be consistent with the config names of profile, and use it to call APIs:
def model(...):
    dbt.config(http_path='...')
    ....


WIP: Kernel developers tutorial for DAL

Are there any prerequisites? Obviously it needs Rust; does it need any other things, like Clang and LLVM like the existing Smart Rollup tutorial?

Any prerequisite for building kernels apply. In addition, the Octez binaries compatible with the current Weeklynet are required. I tried to hint that in a “Note” block, with a link the weeklynet page, but it’s probably too light.
We probably need a tutorial specifically for interacting with Weeklynet on docs.tezos.com actually.

I needed to run rustup target add wasm32-unknown-unknown before building -- this is mentioned but make it a prereq or an explicit step; I missed it the first time.

Can do indeed. We should add a proper prerequisite section as you suggested.

How do I install the proto-alpha feature mentioned after the Cargo.toml file? Is it enough to have it in the features field in Cargo.toml?

Yes indeed.

Weeklynet faucet doesn't seem to work with temple browser extension; I get a message that says "Parameters invalid;" is that a known issue with Weeklynet? I can connect by scanning the qr code in the mobile app and the faucet tried to send test, but it says "An error occurred" and the console shows a 500 with no further info.

That’s not surprising: Weeklynet is the very last version of Tezos, not yet deployed on Mainnet. Temple cannot keep track. We come back to the benefit of having a tutorial for Weeklynet.

Instructions about interacting with weeklynet link to commands for octez-node; do I need to work with octez-node, or is there something similar I should be doing with octez-client?

You don’t need to start an Octez node directly. You just have to use the octez-client from the correct commit.

Tutorials are much easier to follow if each thing that the user needs to do is a numbered step, with the actual thing that the user is doing in the first sentence of that numbered step. Otherwise I don't know if statements like "The proto-alpha feature is necessary..." are telling me to do something or not.

Any input for improving the tutorial is welcome. Readers familiar with Rust are likely to understand this sentence in particular I think, but maybe it’s not good enough to assume familiarity with Rust.

I got as far as the originate smart rollup command. I have my private key aliased in octez but I'm not sure if it has any tez, because of the issues with the faucet. When I run the command I get this:

This error is most certainly coming from the fact you are not using the correct version of Octez-client.

Out of curiosity, would you have the bandwidth to contribute directly to the tutorial to improve its structure, fluidity, etc.?

Yes, I can find some time for that this week. I can also look into writing a page on using weeklynet.

Can I close this now that https://github.com/trilitech/tezos-developer-docs/pull/264 is good to go?

Can I close this now that #264 is good to go?

Yes, go ahead and close if you are happy with #264 .


Update instructions for generating Trello secrets

Whilst taking a look at this (very helpful) gem, I spotted that the
instructions for generating your Trello secrets were out-dated. This
minor Pull Request updates the README.md to match the new Trello
workflow for secret generation.

You're most welcome, @qcam, Happy New Year to you, too!


[🔶 Change request ]:  Activations should be case insensitive

#44 Activations should be case insensitive

Making the activations all lowercase to be case insensitive.
activation.js, name of activation made lowercase before inserting into activations dictionary.
addHiddenLayer.js, act - name of activation made lowercase before checking the activation name from dictionary.
outputActivation.js, act - name of activation made lowercase before checking the activation name
setFunc.js, act - activation name made lowercase
stringTofunc.js, act - activation name made lowercase
actfuncs.js, made each of the activation names to be all lowercase from definition of activations
test/unit/classes/dann.js - update tests to have all activations functions be lowercase
test/unit/core/functions/actfuncs.js - update tests to have all activations functions be lowercase
test/unit/core/functions/add.js - include test that users can add a new layer that will have activations function name be changed to lowercase and a minor typo from a test.

Thank you and1can!
I am merging your PR.


fix titleFlex param in title

Similar to this pull request: https://github.com/ivirtex/cupertino_onboarding/pull/23, but without the version change. I encountered the same issue with the package and reached the same conclusion.

Thank you!


Please verify option 118 accuracy

Based on the DHCP subnet selections options page, 118 was removed from the documentation since it is not available in 2016 (https://github.com/MicrosoftDocs/windowsserverdocs/issues/1012).  Can you verify the statement on this page "DHCP now supports options 118 and 82 (sub-option 5). You can use these options to allow DHCP proxy clients and relay agents to request an IP address for a specific subnet, and from a specific IP address range and scope." is accurate, or adjust it to remove 118 accordingly?  Don't know if it just came in update and is now available, or if it is referencing the now-defunct information from the subnet selection options page

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

ID: 4e6bf5e2-7bc5-bbd5-54e2-772fbcc3cd8e
Version Independent ID: 181cd9db-73b5-510c-f259-6fa28134e566
Content: What's New in DHCP
Content Source: WindowsServerDocs/networking/technologies/dhcp/What-s-New-in-DHCP.md
Product: windows-server-threshold
GitHub Login: @shortpatti
Microsoft Alias: pashort

#please-close


Team Responsive

I have created the teams page responsive
closes #322

https://github.com/mdazfar2/HelpOps-Hub/assets/118350936/65f39146-53e6-427f-88f8-62b25e88647f

https://github.com/mdazfar2/HelpOps-Hub/assets/118350936/4502b3bb-b8cb-43fe-9343-3a02aee2cfee


Coul deploy cluster on vmvare vcenter

Could not create vSphere cluster but it seems something is broken on there
Expected Behavior
I am building configuration and start cluster creation

Screenshots or additional information and context

Environment Details

Build version (v0.11.4):
Deployment (Managed):
Infrastructure Provider (vSphere):
Operating System (Ubuntu 20.04):

I have try more simpler password and it resolve problem the are some bug in deployment with complex password (with exclamation mark !)


Test

Part of the  project
Main issue:
Doc section:
Project:

Manual Test Steps
#Automated Tests

Ah, looks like NewFeature does auto-assign Shawn to everything...


Unable to route to websites

I have multiple routes in my fabio routing tables with many services and a UI application. I can see fabio is forwarding the request correctly in the case of services but in the case of UI application the request is getting forwarded but all the resources of the web site is not getting downloaded.
Sample routing table

/myservice http://xxxx.xxx.com strip=/myservice
/uiapp http://yyy.yyy.com strip=/uiapp
In the case of uiapp I can see the index.html of the web application is hit bu the resources refered in the file are not getting downloaded.
For example

All the href links are broken.
Any help on this issues is highly appreciated.

Presumably the resource paths don't start with /uiapp. If that's a case you have to find a way to make "uiapp" do that or change the routing rules. Since you are using different domains it seems unnecessary to specify the path prefixes, for instance.

The reason I have the prefix /uiapp is that I have different services and web apps in by routing table and I need to differentiate by apps. Hence in this case I added the prefix /uiapp and added a strip for the same prefix. Please let me know if you are seeing any issue with this approach.

fabio doesn't rewrite the response bodies for you, so you have to get "uiapp" to produce correct URLs. Your options are:

use a unique domain and stop rewriting the path
make the app produce absolute URLs with the /uiapp prefix
make the app produce relative URLs
add some other proxy that's capable of rewriting URLs in response bodies

Whether options 2 and 3 are possible depends on the app you are proxying; it has to let you configure the base URL.
I always recommend not messing with the request path (i.e. option 1), precisely because of issues like this.

Thank you for your quick responses.
Currently I do have relative URLs in the main.html file like
"link href="/static/main.88dc7d7ac8d00b6f86c1.css" rel="stylesheet"
But this file doesn't get downloaded.
Option 1: use a unique domain and stop rewriting the path
Does that mean I should be using the redirect option of Fabio?
Option 2: make the app produce absolute URLs with the /uiapp prefix
I am using a open source web app which I'm trying to host it internal to my network. Hence I was trying to avoid any changes to it, also the paths seems to be relative too.

Option 1: use a unique domain and stop rewriting the path

Does that mean I should be using the redirect option of Fabio?

No. It means to change your route as follows:
-/uiapp http://yyy.yyy.com strip=/uiapp
+/ http://uiapp.yyy.yyy.com

Which domain you choose is irrelevant, of course, as long as it doesn't conflict with any other route.


texstudio: 4.6.3 add support to darwin

Description of changes
texstudio is a cross-platform software for editing LaTeX projects. It's dependencies and build dependencies are already supported on darwin projects. Only by changing the supported platforms, texstudio compiles and runs successfully.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

@ajs124 @CFouche can you have a look? Thank you.

@2xsaiko Thank you for your suggestion. I followed the pattern in another nix file. I agree that the cp and the variable are unnecessary


How to use ???

could someone write a redme about how to run yhis project... it would be very helpfull

@Rasantis did you find any solution to run this project?


Implement Color Switching

Using CSS variables, declare the color scheme of the website and implement a javascript function that inverts those colors from dark to light. These CSS variables should be used for all colors in the portfolio

CSS color variables are added.  Also added new functions to switch between colors.
CSS animations will need to be added in order to have the colors switch nicely - this can be handled later.


added some tourist info and a picture

fixes bug 123- no info whatsoever about Cordoba Argentina, added info and a picture

Thanks Werner, would be good though to have the name of the city in the title as it wasn't obvious :-)


Potentially missing data

This looks really interesting …but is the actual data missing? I can only see the README.md and LICENSE files in the repo, not any .csv file :(

Ah, thanks a lot for pointing this out. I think this might be a repo I started, forgot about and then put the files somewhere else - as in this blog post, which links to the csvs
http://www.statsmapsnpix.com/2021/10/food-for-thought.html

Perfect, thank you!


mbed/libraries/USBDevice/USBSerial/CircBuffer.h should get its size as a template parameter

template<typename T>
CircBuffer
seems to only be used in the USBSerial library where its size is set at compile time (in the USBSerial class constructor). If it were rewritten to take its size as a compile time parameter
template<typename T, int Size>
CircBuffer
the code would be much more efficient in terms of flash size, execution speed and ram footprint.

I would not mind rewriting the code if someone is willing to test it. I do not have the ability to test it right now.

fix has been merged


EPIC- Sonataflow accessing different clusters from same workflow definition through configuration

Description
It would be nice to have the possibility of configuring access to different clusters from the same workflow definition, without the need of deploying the workflow in different K8S namespaces.
This issue includes:

[x] https://github.com/apache/incubator-kie-kogito-runtimes/issues/3573
[x] https://github.com/apache/incubator-kie-kogito-runtimes/issues/3590

@ricardozanini Im closing this one since all associated runtimes issues has been completed and I do not think anything else needs to be done at operator side.


[WIP] Supports sysmon events.

Codecov Report
Patch coverage: 2.13% and project coverage change: -9.61% :warning:

Comparison is base (4e098cc) 68.50% compared to head (f52d440) 58.89%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #488      +/-   ##
==========================================
- Coverage   68.50%   58.89%   -9.61%
==========================================
  Files          22       23       +1
  Lines        4166     4854     +688
==========================================
+ Hits         2854     2859       +5
- Misses       1312     1995     +683

Files Changed
Coverage Δ

src/graphql.rs
74.59% <ø> (ø)

src/graphql/sysmon.rs
0.00% <0.00%> (ø)

src/ingest/implement.rs
52.48% <0.00%> (-45.46%)
:arrow_down:

src/storage.rs
62.83% <0.00%> (-10.40%)
:arrow_down:

src/ingest.rs
60.32% <7.07%> (-28.00%)
:arrow_down:

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

@BLYKIM
rebase 해주세요.


The input file seems to be corrupt

com.documents4j.throwables.ConversionInputException: The input file seems to be corrupt 	at com.documents4j.util.Reaction$ConversionInputExceptionBuilder.make(Reaction.java:159) 	at com.documents4j.util.Reaction$ExceptionalReaction.apply(Reaction.java:75) 	at com.documents4j.conversion.ExternalConverterScriptResult.resolve(ExternalConverterScriptResult.java:70) 	at com.documents4j.conversion.ProcessFutureWrapper.evaluateExitValue(ProcessFutureWrapper.java:48) 	at com.documents4j.conversion.ProcessFutureWrapper.get(ProcessFutureWrapper.java:36) 	at com.documents4j.conversion.ProcessFutureWrapper.get(ProcessFutureWrapper.java:11) 	at com.documents4j.job.AbstractFutureWrappingPriorityFuture.run(AbstractFutureWrappingPriorityFuture.java:78) 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) 	at java.lang.Thread.run(Thread.java:745)
This situation occurs when convert to pdf quickly, not always.  I have created a folder C:\Windows\SysWOW64\config\systemprofile\Desktop ，but not work.
this is my code
`    private volatile IConverter iConverter;
@PostConstruct
void init() {
    iConverter = LocalConverter.builder()
            .baseFolder(new File(doc4jtemp))
            .processTimeout(5L, TimeUnit.MINUTES)
            .workerPool(15,30,10,TimeUnit.MINUTES)
            .build();
}
public void doc2pdf(String docPath, String pdfPath) {
    File source = new File(docPath);
    File target = new File(pdfPath);
    iConverter.convert(source).as(DocumentType.MS_WORD)
            .to(target).as(DocumentType.PDF)
            .execute();
}

`

Seems to me like you are not closing a stream to the file that writes the input or maybe you are holding a lock on the output. Might that be the case?

see

I am facing the same error.
Could you please tell me how have you resolved this?


A User Guide page on creating a Cube from scratch

📚 Documentation

Creating a Cube from scratch is valuable in many contexts, but many of our users are unaware that this is even an option (believing instead that they can only be created via loading). We should advertise that this is possible, and how to do it.

Reference : In the peloton meeting this morning it occurred to me that this issue was worth another mention, in the context of the #4447 : for example. how can you create an abstract target grid for regridding to


Enviro weather MQTT with Mosuito on Hassio

I configured mosquito on Hassio and set up enviro weather to send readings to it (one readfing every 15 mins and upload after 10 readings). But I noticed on the graph are reported only the value  from the last reading of each group of json files. Do I have to configure somenthing more on mosquito? Thanks in advance?
I attached the graph. It is strange that the value changes only after every upload. I download the json files from the enviro board and I saw that the values are different for each reading.

I could be wrong here but it used to be that the data uploaded even tho it may have 10 sets of readings stored in it only has one timestamp (unless this has been fixed) so there is no way to split them up to get the individual readings out and the only reading that makes sense to ha is the last one with a timestamp. I could be wrong on this but from memory that is how it used to work,. Maybe somone else with more knowledge of the code can let you know if I am wrong.

I think the way Home Assistant works, this is not possible?. See:

https://github.com/home-assistant/home-assistant.io/issues/33742

@CBDesignS: Do you mean something like this: https://community.home-assistant.io/t/sensor-transmits-multiple-time-points-how-to-ingest-into-ha/527887?
Actually, when I tried the same thing as @davidbenucci, the config file on Enviro was:
# how often to wake up and take a reading (in minutes)
reading_frequency = 2

# how often to upload data (number of cached readings)
upload_frequency = 5

I see this in MQTT Explorer (https://mqtt-explorer.com/) in the history section—multiple messages, but each of them has its own timestamp.
@CBDesignS  not sure what you are referring to with:
it may have 10 sets of readings stored in it only has one timestamp (unless this has been fixed)
Example from the "history" section:
Last message:
07/15/2024 9:46:17 AM
{
  "readings": {
    "pressure": 963.92,
    "pm1": 10,
    "pm2_5": 24,
    "noise": 0.046,
    "humidity": 55.7,
    "temperature": 25.69,
    "pm10": 24
  },
  "nickname": "enviro-urban",
  "model": "urban",
  "uid": "e66164084356202d",
  "timestamp": "2024-07-15T07:46:09Z"
}

07/15/2024 9:46:16 AM (-0.51 seconds)

{
  "readings": {
    "pressure": 963.93,
    "pm1": 10,
    "pm2_5": 16,
    "noise": 0.05,
    "humidity": 55.89,
    "temperature": 25.66,
    "pm10": 30
  },
  "nickname": "enviro-urban",
  "model": "urban",
  "uid": "e66164084356202d",
  "timestamp": "2024-07-15T07:44:09Z"
}

07/15/2024 9:46:16 AM (-0.51 seconds)

...
"timestamp": "2024-07-15T07:42:09Z"
...

07/15/2024 9:46:16 AM (-0.51 seconds)

...
"timestamp": "2024-07-15T07:40:09Z"
...

07/15/2024 9:46:15 AM (-0.3 seconds)

...
"timestamp": "2024-07-15T07:38:09Z"
...

07/15/2024 9:36:14 AM (-10 minutes)

...
"timestamp": "2024-07-15T07:36:09Z"
...

I am not the person to help here as I actually gave up on the enviro weather board after bench testing for a month or two and it was put back in the box it arrived in. I already had an old but functioning FineOffset WHx80 setup that I was looking to replace with the Enviro Weather but the problems and lack of support at the time I bought the enviro made me decided to spend the time getting the old reliable setup working and I even added to the system with a lightening strike detector and Lux meter. They are stand alone 433mhz radio units that just send data to a 433mhz reciever that talks directly to HA and HA defined sensors and then HA uses the data to give me the graphs and readings I need, No software, hardware, firmware or support problems with them. I really wanted to use the Enviro Weather as it was newer and more compact.
Good luck getting your board and ha setup.
this is what my weather station looks like in HA.


Idva6 548 update status

Short description outlining key changes/additions
An endpoint to update the status of an association.
JIRA Ticket Number
IDVA6-548
Type of change

[ ] Bug fix
[x] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[x] I have added unit tests for new code that I have added
[x] I have added/updated functional tests where appropriate
[x] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: No security warnings found

CI: No security warnings found


add timeout arg to ProjectRequest.init

Short description: add timeout arg to ProjectRequest init
More details:
What this PR does / why we need it:
Due to timeouts in the last teardown, i had to increase the timeout.
This was not possible, because ProjectRequest did not have timeout to populate
Which issue(s) this PR fixes:

occasional timeouts
inability to intiailize ProjectRequest object with a custom timeout

Special notes for reviewer:
Bug:

/approve

/release-note-none


Wrong hierarchy for child nodes

Hi, this is useful for organizing my meeting notes, which are always just collections of points and views! Thanks a lot! But during usage, I found that the child nodes are all in the same hierarchy as the node where the wiki link is, as shown below.
Original file

Backlinks

Thanks again for developing this plugin!

It seems the hierarchy for the bullet lists is correct.

Implemented/fixed in latest version.


Do not use google-authenticator when ssh from LAN ?

Hi,
I don't really know if that is the right place to ask, so appologies if it is not, and any hint as to where to get an answer is welcome.
That said, I would like to setup my ssh to accept :

(public key) or (user's password), when the request comes from LAN,
(public key)  or (user's password + google-auth) when the request comes from Internet.

I can disable PasswordAuthentication in sshd_config and at the end add the following to still allow  user login from LAN.
Match Address <IP_ADDRESS>/8,<IP_ADDRESS>/12,<IP_ADDRESS>/16
PasswordAuthentication yes

However, I fail to get a working setup to the usecase described above.
Any idea ? suggestion ?
I run ubuntu 20.04 if that is of any use.

https://github.com/google/google-authenticator-libpam/issues/188#issuecomment-712951620
You could do something like here but for IPnets instead by using “pam_accees.so”

Yeah this is a PAM question, not a GA question.


Updating stepper.py to allow for 4 pin motors.

Awkward argument order in order to preserve backward compatibility.
Requires changes to the firmware, as well.

Thank you! please send me another PR to add yourself to the AUTHORS file!

@hegartybr added you to CONTRIBUTORS, PR is no more needed :smile_cat: thank you for your contribution!


Merge release into master

Manual action triggered by: https://builds.gradle.org/buildConfiguration/Gradle_Release_Promotion_Nightly/60821046?hideProblemsFromDependencies=false&hideTestsFromDependencies=false

@bot-gradle test and merge please

Your PR is queued. See the queue page for details.

I've triggered a build for you.


Fixed error accessing to event info

It seems that the file name is not accessible under data property since this property is undefined
The file name is accesible directly within the event. Since GCFunctions are in beta some API change may have been been the cause of the faulty behavior
In this article from the docs you can check that name property is directly under the event instead of under the "data" property
Also added a console.log for the sake of debugging

Hi @LuisMayo Thanks for these PRs. I'll have a look at those this weekend and will merge them.


Bigquery and Google Cloud Storage operators

Added a BigQuery operator to transfer data to Google Cloud Storage. Also added a hook for GCS, and an operator to download files from GCS to local file system.

:+1:


common: #23 removed useless if statements for nullptr checks

There where a lot of if statements to check if a new operation would return null
Because "new" can never return null, at least in the official standard, we remove those.
I did no put any effort to in correcting anything but removing those if's
The cleanings I removed would never have been called anyway, so no break should be happening.
I think it is necessary to replace a lot of those pointers with smart pointers and do the cleaning in RAII fashion,
but that should be part of another PR

Tested and didn't find any issues. Thanks!


Not finding Islands (only picking Patreon islands)

I noticed too that it's only finding Patreon islands, this is probably due to the website change I think its defaulting

Hi! The website changed, so we will need one additional selection click with the geckodriver.
I'll have a look ASAP, meanwhile are you able to find the code changes and submit a PR?
Otherwise I'll get it done by tomorrow evening, not a big problem

Fixed as per #4


changed inspector label for hingejoint3d/motor/targetVelocity to radi…

…ans/s

The targetVelocity property for HingeJoint3D motors is properly specified as radians/second, as specified in godotengine/godot#80505.

Thank you for contributing
This is already done in #80523, also the unit should be degrees per second, as described on the issue, as the unit is clearly not radians (-200, i.e. almost 30 turns per second, is clearly not radians, AFAIK)

Closing in favor of https://github.com/godotengine/godot/pull/80523. Thanks for your contribution nevertheless!


Cannot Properly Center Title on Android

The default for title alignment on Android is left but in our app we center it.
To center the title, I specified a style in headerTitleStyle that has alignSelf: 'center'.  That moved the title to the middle, but it was offset by 40px, as shown here:

I found the cause, which is in
this line from Header.js.  On Android, this essentially cancels out the offset from TITLE_OFFSET.  Since there is not 'right' component but there is a 'left' back button, the style on the title's container is { left: 40, right: 0 }, which causes the problem.  It would also be nice to have access to magic constants like TITLE_OFFSET as well.  Maybe there could be a config setting for those as well.
It took a lot of work to get the title to align to the center.  I suggest adding a property like headerTitleAlign.  That could allow someone like me to override the default behavior.  Would you be open to something like that?
By the way, I have fixed the problem in our app by subclassing Header.js and adding:
  _renderTitle(props: SceneProps, options: *): ?React.Element<*> {
    const view = super._renderTitle(props, options)
    // see https://github.com/react-community/react-navigation/blob/30ca3c1f80617abc28fdd71f0b37d4f62b3e388a/src/views/Header.js#L162
    // by setting the right to 0 and not the left, the title is off center
    // style is an array and we check for left/right being set and if so, match
    let left, right
    for (const style of view.props.style) {
      if (style.left !== undefined) {
        left = style.left
      }
      if (style.right !== undefined) {
        right = style.right
      }
    }
    if (left !== undefined || right !== undefined) {
      if (left === undefined) {
        left = right
      } else if (right === undefined) {
        right = left
      } else {
        left = right = Math.max(left, right)
      }
      view.props.style.push({ right, left })
    }

    return view
  }

This logic works for centering only.

@StevePotter Adding an empty view as 'headerRight' works too.

@hecvasro   empty view means headerRight=null or headerRight= ?

@ShenghuGong I meant <View/>.

I added a border to the "headerTitle" component and we can see a diff on android and ios: on android it does not take 100% of the screen  when you add an element on the right. The center part is moved (which make sense in a way).
On iOS, the headerTitle always take 100%, even when you add a thing on the left or on the right.
Looks like a unification of the behavior is desired.

Pinging OP @StevePotter since this issue is quite old and related to an old version of the lib. If no answer I'll close in 7 days.
Moreover the comments provide ways to do this.

The responsible code is still in master https://github.com/react-community/react-navigation/blob/d93acdd918a306fb911442eb0a9d40558869e339/src/views/Header/Header.js#L185
I don't really understand why there is such a different behaviour for those 2 platforms. It's really weird to (by default) does not center title in the header bar on android, especially when you use the same visual pattern for this 2 OS in your app.
@Freddy03h I saw that you edited this recently to bring the iOS part below this code. Can you take a look to this issue and give us your thoughts? Thanks in advance!

I don't really understand why there is such a different behaviour for those 2 platforms.

Because the two platforms (android, ios) have different guidelines?

If you take the initial screenshot associated with this issue, I am really sure it's not the desired behavior on any platform.
I don't have to read android guidelines to see that the non centred text is not acceptable for the end user and won't be advised by any guidelines. Just look at the clock + title not aligned...

Well, then. Feel free to submit a PR to fix this issue, we'll review asap.

It's different between Android and iOS because title isn't centered by default on Android.
Consequences of this behavior are :

on iOS left and right must have the same values to keep the title centered (before my PR, left and right were always at TITLE_OFFSET, now left and right are set to 0 if there is no left and right component)
on Android the title component take all the available space, left and right are independents and based of the associate left/right component.

For now, if you want a centered title component on Android and have a component on left or right, you must set an empty view on the opposite side.
If you want to unify this behavior between iOS and Android because you want to centered the title on both. You'll break the default Android behavior :

for a left to right language with a title and a right component : the title will not be at the left of the header.
for a right to left language with a title and a left component : the title will not be at the right of the header.

+1 Please fix guys 🥇

+1 We're looking to do the exact opposite: left align titles on iOS to match Android. Seems it is not possible to override the styles on Animated.View in src/views/header/header.js line 222 which would otherwise provide the flexibility we're looking for.
If there is a way to pass styles through to this component from NavigationOptions, we haven't found it yet. If you know of one let us know.

Latest 1.2.1 does not allow for flex styles in header, therefore it does not allow for center header for android. Is there going to be a fix for this, or any work arounds?
Thanks in advance!

hi, I've assigned myself to this and will look into it when I have some time. if anyone else wants to help it would be very welcome. we should probably use a completely different header implementation on android.

I think it's can be cool to have an option on it instead of doing strict Platform difference in the component. centeredTitle and 'leftAlignTitle' for example, or iosStyle and androidStyle.
So if I want a left aligned title on iOS I can, and if I want a centered title on Android I can too.

To centre title text on Android, I was able to get away with this for now:
headerTitleStyle: {
  flex: 1,
  textAlign: "center"
}

Unfortunately,  { flex: 1, textAlign: 'center' } doesn't work properly when you have either headerRight or headerLeft. Adding headerRight with empty <View /> doesn't solve the problem when index === 0 (root screen) and headerLeft not displayed.

static navigationOptions = {
title: 'Canlı Yardım',
    headerStyle: {
      backgroundColor: '#fff',
    },
    headerTintColor: '#000',
    headerTitleStyle: {
      fontWeight: '200',
      color: '#2c38ff',
      textAlign: 'center',
      flex: 1,
    },
}

static navigationOptions = {
 title: 'Canlı Yardım',
    headerStyle: {
      backgroundColor: '#fff',
    },
    headerTintColor: '#000',
    headerTitleStyle: {
      fontWeight: '200',
      color: '#2c38ff',
      textAlign: 'center',
      flex: 1,
    },
    headerRight: <View />, // add headerRight for fix
}

I ran into the same issue on Android (not iOS) on the latest version. Adding marginLeft: 'auto' and marginRight: 'auto' worked for me.
navigationOptions: {
      title: 'BrightID',
      headerTintColor: '#fff',
      headerTitleStyle: {
        fontFamily: 'EurostileRegular',
        fontWeight: '200',
        fontSize: 24,
        marginLeft: 'auto',
        marginRight: 'auto',
      },
      headerBackground: (
        <LinearGradient
          colors={['#F52828', '#F76B1C']}
          style={{ flex: 1, width: '100%' }}
        />
      ),
    },

Here is my ugly-ugly hack. Tested on react-navigation v2.2.3.
import { createStackNavigator } from 'react-navigation';

import HeaderStyleInterpolator from 'react-navigation/src/views/Header/HeaderStyleInterpolator';
import Header from 'react-navigation/src/views/Header/Header';

/*
 * Copied from react-navigation/src/views/Header/Header.js
 */
const TITLE_OFFSET = Platform.OS === 'ios' ? 70 : 56;

createStackNavigator(
    {/* routes */},
    {
        navigationOptions: (...args) => ({
            header: props => {
                return <Header
                    {...props}
                    titleInterpolator={(props) => {
                        return {
                            ...HeaderStyleInterpolator.forCenter(props),
                            left: TITLE_OFFSET,
                            right: TITLE_OFFSET,
                        };
                    }}
                />
            },
            headerTitleStyle: {
                flex: 1,
                textAlign: 'center',
            },
        })
    }
)

working on https://github.com/react-navigation/react-navigation/pull/4588 which will make this easier

Cant do this..please help with it

https://stackoverflow.com/questions/52866912/cannot-center-align-header-title-in-react-native-android
Please solve this issue

@PrinceSharzeel - please read the comment directly above yours :P
headerLayoutPreset: 'center'
done


Detect when mediaconvert encounters errors specific to the input file

Closing, we already have these error codes humanized in task-runner


Possibility to skip certain checks in report generation

Is your request related to a problem? Please describe.
I periodically scan all Exchange servers with healthChecker.ps1. The scan result are processed with BuildHtmlServersReport. These reports always (correctly) report a 'Vulnerability Detected' in a big red square, because of download domains not being implemented. Unfortunately we can't use download domains yet, as this doesn't work in a ADFS fronted setup. (MS is aware and a fix is in the making)
Describe The Request
It would be helpful if I could skip or acknowledge this during generation of the HTML report, so that the 'Vulnerability Detected' warning would be useful again and would again indicate we need to take immediate action. I can imagine this could be useful for other checks as well that trigger the detection.

Would need to investigate this to even determine how to do this and how much time it would even take to implement something like this.

We discussed this quite a bit (at least one other request like this has been made too). While we can appreciate the fact that due to specific config some customers are unable to follow some of our recommendations right away, the bottom line is that the Health Checker is a recommendation engine and it is a reflection of what Engineering and Support want to highlight to our customers as things that will bring the environment to be configured as securely as possible. Because of this, we decided that we would not provide switches to "skip checks" or "filter out" things from reports but want to call them out every time HC is run because they are still our recommendations. The argument that some of the recommendations can be hard to implement could be true for many of them (for period of time) but the Health Checker goal is to call out any that have not been implemented so far, when it comes to security matters.


datapath: Pass proxy port in to-proxy traces

Include proxy port in the to-proxy trace messages.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

/test

The CEP in question (default/k8s2-client) is not part of this test, it must be a remnant from a prior test. It would be misleading to track this as a flake on the Istio test.

This happens during endpoint regeneration, after which the endpoint is deleted. Is it possible that this error log from the endpoint synchronizer should not be considered an error at all? K8s has deleted the pod, but Cilium agent has not yet processed the deletion as it is in the middle of regeneration?

:thinking: I think this makes sense. This controller is running because the endpoint was previously provisioned. Presumably by the time this controller first runs, there is guaranteed to be a k8s pod available so there's no way for this error to be returned. When the pod is deleted, if regeneration is under way then we probably haven't yet cancelled the ctx that is passed into this controller, so we don't yet know that the resource is gone. That said, if we attempt to update it and get a 'not found' error (specifically this one) then the only case I can think where this would occur.. is if the pod is being deleted. So yeah we shouldn't need to care in that case since this is probably the one last spurious error that this controller would see before it gets shut down.
Either we can move the 'Context' cancellation earlier for the ctx passed into this controller, or we can just detect the 'not found' error type and ignore that specific one without logging. :+1:

/test


Key error in power states

Dec  7 23:41:17 sfci-smk6su INFO sf-cleaner[16899] Updating power states
Dec  7 23:41:17 sfci-smk6su sh[2913]: Traceback (most recent call last):
Dec  7 23:41:17 sfci-smk6su sh[2913]:   File "/usr/local/bin/sf-daemon", line 8, in <module>
Dec  7 23:41:17 sfci-smk6su sh[2913]:     sys.exit(main())
Dec  7 23:41:17 sfci-smk6su sh[2913]:   File "/usr/local/lib/python3.6/dist-packages/shakenfist/daemons/main.py", line 182, in main
Dec  7 23:41:17 sfci-smk6su sh[2913]:     _audit_daemons()
Dec  7 23:41:17 sfci-smk6su sh[2913]:   File "/usr/local/lib/python3.6/dist-packages/shakenfist/daemons/main.py", line 162, in _audit_daemons
Dec  7 23:41:17 sfci-smk6su sh[2913]:     _start_daemon(d)
Dec  7 23:41:17 sfci-smk6su sh[2913]:   File "/usr/local/lib/python3.6/dist-packages/shakenfist/daemons/main.py", line 103, in _start_daemon
Dec  7 23:41:17 sfci-smk6su sh[2913]:     DAEMON_IMPLEMENTATIONS[d].Monitor(d).run()
Dec  7 23:41:17 sfci-smk6su sh[2913]:   File "/usr/local/lib/python3.6/dist-packages/shakenfist/daemons/cleaner.py", line 160, in run
Dec  7 23:41:17 sfci-smk6su sh[2913]:     self._update_power_states()
Dec  7 23:41:17 sfci-smk6su sh[2913]:   File "/usr/local/lib/python3.6/dist-packages/shakenfist/daemons/cleaner.py", line 125, in _update_power_states
Dec  7 23:41:17 sfci-smk6su sh[2913]:     elif dbpowerstate['power_state'] != 'off':
Dec  7 23:41:17 sfci-smk6su sh[2913]: KeyError: 'power_state'

This might be a missing initial value like state was? I need to work out what happened before the refactor.