Borrowed parameters cannot be used in autoclosure

Description
A function's borrowed parameter cannot be used in a precondition's autoclosure. The error claims that the parameter "cannot be captured by an escaping closure since it is a borrowed parameter". It's not clear why an autoclosure should be considered escaping.
Reproduction
public func test(_ i: borrowing Int) -> Bool {
    precondition(i > 0)
    return true
}

Expected behavior
This should compile.
However,
<source>:1:20: error: 'i' cannot be captured by an escaping closure since it is a borrowed parameter
1 | public func test(_ i: borrowing Int) -> Bool {
  |                    `- error: 'i' cannot be captured by an escaping closure since it is a borrowed parameter
2 |     precondition(i > 0)
  |                  `- note: closure capturing 'i' here
3 |     return true
4 | }

Environment
Swift version 6.1-dev (LLVM 95f3fb07f8f5294, Swift 8ae66ec1a522af4)
Target: x86_64-unknown-linux-gnu
Additional information
No response

@glessard
I reproduced the snippet, looks like this is fixed since the commit you shared?
My environment:
Target: macOS (Apple Silicon)
Commit: 1af1af7d4509b68a0a3acf2cdce27b3eef9a0789

This also reproduces for me using swift-DEVELOPMENT-SNAPSHOT-2024-10-27-a, by having the snippet above be the only code in a new package vending a library – i.e. just created with swift package init.

I am not sure I understand, but to reproduce I created a .swift file and as arguments passed .../Debug.swift -typecheck with swift-frontend as the executable, and the code runs fine without any errors.

I am not sure I understand, but to reproduce I created a .swift file and as arguments passed .../Debug.swift -typecheck with swift-frontend as the executable, and the code runs fine without any errors.

this error occurs after typechecking. if you invoke the compiler frontend with -emit-sil (or no specific pipeline stage arguments) you will presumably see the same error.


Matches that are rejected in the linking process are nevertheless added to the results

e.g when downloading Tudela matches like Yaqut:dig03683 - BJ5 are added even though we clicked "different"

In this page:

The two links that allow the download of the dataset in linked--places format and tsv format create a file that contains all matches and not only the approved ones.

The download link is generated by this code in ds_summary.html:_

LP-TSV (augmented) {% fontawesome_icon 'download'%}

This is eventually handled by a celery task in datasets/task.py named make_downloadd

Fixed by changing the code that creates the list of matches to exclude those place_link records that contain a relation "different"


The transform task of text-generation without using GPU resources.

I am running ghcr.io/postgresml/postgresml:2.7.12 image using docker run, with the parameter --gpus all. When I execute the following SQL:
SELECT pgml.transform(   task   => '{     "task": "text-generation",     "model": "Qwen/Qwen2.5-0.5B"   }'::JSONB,   inputs => ARRAY[         'hello'     ] );
The SQL is running very slowly (Approximately 100 seconds), and there is no observed VRAM usage when using the nvidia-smi command. When I use the Qwen/Qwen2.5-Coder-7B model, it causes out-of-memory (OOM) issues. My machine has 4 cores, 15GB of RAM, and uses an NVIDIA T4 GPU with 16GB of VRAM.
How can I utilize GPU resources to run tasks involving large models like Qwen/Qwen2.5-0.5B?

The Qwen/Qwen2.5-0.5B model does not support setting the device parameter in AutoModelForCausalLM.from_pretrained(), and trying to set it will result in an error: TypeError: Qwen2ForCausalLM.init() got an unexpected keyword argument 'device'.
Solution: When calling pgml.transform in SQL, add the parameter "device_map": "auto".


VideoFrame.reformat ignores source colorspace by default

Overview
Unless otherwise told, it seems like VideoFrame.to_rgb (and conversely to_image) assumes the source colorspace to be 601, regardless of what it actually is.
If given a source that, for example, is using a 709 colorspace, the resulting RGB output is incorrect.
Expected behavior
to_rgb should use the colorspace specified by the source.
Actual behavior
If src_colorspace is not supplied, 601 is assumed.
Investigation
On a source using a 709 colorspace, specifying src_colorspace='ITU709' in to_rgb/to_image causes the correct output to be generated. Unfortunately, it seems like PyAV doesn't offer a way to determine what the source colorspace is, so this strategy isn't feasible unless the colorspace can be determined via other means.
Reproduction
Use the keyframe saving example with a source using a BT.709 colorspace.
Versions

PyAV: 8.1.0
libav*: 4.4

I don't really know Cython, but I fumbled a change that seems to work. Until code is updated here, this might serve as a guide for those encountering the same issue:
https://github.com/PyAV-Org/PyAV/compare/main...animetosho:fix_src_color

@mikeboers this looks like a good catch. I'm not familiar with the colorspace conversion code, why do we even offer an src_colorspace argument?

I suppose a possibility is when you don't trust the detected colorspace to be accurate. libavcodec does support the idea of an 'unknown' colorspace, so being able to explicitly override it could have its uses.

There are times when color tags are not present in the container, the video stream or the packet / frame. So the consumer of the content is left to sort of guess as to what the best color space is.
I believe AVFoundation (Apples analog to libAV) uses a heuristic that if there is no colorspace tagged, and the frame is below 480p, assume 601 colorimetry. If its above, assume 709. Best practice is of course to tag your content.
I've not tested that but think its correct.


Fix Card Width

Issue
fixes #697
What is the change?
Changed with of Join the Event card on small screen
Is Development Tested?

[ ] Yes
[x] No
because there is just UI change

Before

After
Small Screen

it says have you tested locally, you should be selecting yes as option.


Notification for PV filling up in customer namespace

Custom message for KubePersistentVolumeFillingUp  error in a namespace created by customer. Reference: https://redhat.pagerduty.com/incidents/PIV0IBR

Based on conversation

openshift-* customer namespaces are not allowed. this situation was unique and was remedied with alert silence
we have logging PV filling warning and error notifications
we generally don't expect to encounter this use case


Error adding a payment to a Project

When adding a Payment to a project I get the following error.

@Ovifer13 how can I test this if I get this error?

@sofiaromorales you can still access projects from clients


[Task]: Write a Test Suite for Revamped Twilio Connector

Connector Name
ballerinax/twilio (Twilio)
Related Epic Issue
Issue https://github.com/ballerina-platform/ballerina-standard-library/issues/4950
Task Description
A total of 1,219 endpoint operations spanning 42 submodules have been identified and proposed for the new Twilio connector and are implemented.
Identified submodules :

accounts (v1, v2)
flex (v1, v2)
media (v1)
pricing (v1, v2)
taskrouter (v1)
autopilot (v1)
messaging (v1, v2)
trunking (v1)
bulkexports (v1)
frontline (v1)
microvisor (v1)
proxy (v1)
trusthub (v1)
chat (v1, v2, v3)
insights (v1)
monitor (v1)
routes (v2)
verify (v2)
intelligence (v2)
notify (v1)
serverless (v1)
video (v1)
ip_messaging (v1, v2)
numbers (v1, v2)
studio (v1, v2)
voice (v1)
content (v1)
wireless (v1)
conversations (v1)
lookups (v1, v2)
oauth (v1)
supersim (v1)
events (v1)
preview
sync (v1)

Each submodule will be generated using Ballerina OpenAPI Tool using this specifications.
For each submodules tests should be written manually.

as it's done


BLE Host - Persist bonding material to sys/config

This pull request adds a new package: ble/host/store/config.  This package holds security material and CCCDs in RAM, and optionally saves them to sys/config.  Because this package is a superset of store/ram, it deprecates that one.
This PR incorporates changes from other PRs:

277 (#277)
279 (#279)

Therefore, this PR should only be merged after those.

I could review this. Do you mind resolving the conflicts first, given that the other PRs have been merged? That would make the patch easier to read :)

Thanks, Marko.  I didn't realize so many conflicts had sprouted.  They should be resolved now.


[E2E_TEST] Move run_initializer_tests into runTests()

In hack/make-rules/test-cmd-util.sh e2e function run_initializer_test was not being record by function record_command, it would be better if we can records its output/error messages in junit format

I closed #55295 due to inactivity if someone else wants to try taking this on

/remove-lifecycle stale

/sig cli
/kind feature
/priority P2

/remove-lifecycle stale

/reopen
/remove-lifecycle rotten


Need a JAR file for this plugin

[ ] 🐞 Bug
[x]  🚀 Feature
[ ] 🛠 Task
[ ] 📄 Documentation

Description
I want the JAR file for this plugin so that I can mock the behaviour using homeway Jenkins SPOCK testing framework

You should be able to download it from the build server by clicking on the build badge on the start page o this repo

@hoegertn  thanks for the direction , later I checked the pom.xml & got the groupId as well -> de.taimos


RGW: Add support for usage log configuration

Adds support for configuring the RadosGW usage log described at
http://docs.ceph.com/docs/jewel/man/8/radosgw/#usage-logging.

please rebase

done ^

Rolled into #1226 instead


[bug] command pouch inspect does not output an array

Ⅰ. Issue Description
command pouch inspect does not output an array,  output a map struct instread. which make stale orchastration system(like swarm) cannot correctlty unmarshal the response of api /containers/xxxx/json.
Ⅱ. Describe what happened
Ⅲ. Describe what you expected to happen
Ⅳ. How to reproduce it (as minimally and precisely as possible)

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

pouch version (use pouch version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

Thanks for the feedback. Actually you are right. It is an incompatibility with docker cli. However it is a client side incompatibility rather than API side.
Will fix it soon. @yyb196

@allencloud you are right, it's only on the client side.

@faycheng Could you help finish this feature? inspect related feature is implemented by you. It could not be better if you help to make inspect output an array.


Implements TIMESTAMPDIFF Function for MYSQL

This PR implements the TIMESTAMPDIFF function for mysql:
https://dev.mysql.com/doc/refman/8.0/en/date-and-time-functions.html#function_timestampdiff
Allowing alasql to better support mysql specific syntax and queries.
Functionality is mostly identical to the DATEDIFF function, but the syntax is slightly different:
SELECT TIMESTAMPDIFF(MONTH,'2003-02-01','2003-05-01');

MONTH, DAY and YEAR are not string params passed to the function in mysql, unlike DATEDIFF.
the parser (alasqlparser.jison) has been updated to account for this case.

Not sure why the test is failing. Code looks like it would not affect this.
Did you do a npm test before pushing your last change? did it went ok?

@mathiasrw Yes, it looks like those tests are failing due to a mismatch between the code in
alasqlparser.jison and the compiled jison in alasqlparser.js
Is the correct way to generate that file still:
npm run jison

?

Yes - first do a npm run jison and then a npm test

@mathiasrw so it looks like, without any changes, running npm run jison and then npm test has these same test failures.

alasqlparser.js is autogenerated so any changes will be overwritten.
I will have to look into the issue.


Debian repository is out-of-date

OS:
Ubuntu 16.04
Issue Descriptions:
Try to get latest version from repo. That version is 4.0.5 instead of latest.
Steps to Reproduce:
Add repository, apt-get update, apt-get install google-play-music-desktop-player

Woops, uploaded and never clicked publish...........


Teams and Roles username in Database limits management of Users to be salted.

Describe the bug
To salt the users in the database requires the ability to substitute the username.
The foreign key constraints

fk_userinfo_roles
fk_userinfo_username

between these tables

userinfo
userinfo_roles
userinfo_teams

prevent the ability to update the username with an UPDATE statement.
To provide this ability the tables need to be restructured to this SQL data definition language (ddl)
userinfo  >---< userinfo_roles
username
id              user_id
                id

userinfo  >---< userinfo_teams
username
id              user_id
                id

To Reproduce
INSERT INTO team VALUES ( nextval('team_id_seq'),'perf-team');
INSERT INTO userinfo_teams VALUES ('jwhiting', 1, 'perf-team');
UPDATE userinfo SET username='e11d25c51f1c6ba08acfdf2fabd47951778e8db74bea2e5a6134399445973523' where username='jwhiting';
ERROR:  update or delete on table "userinfo" violates foreign key constraint "fk_userinfo_username" on table "userinfo_teams"
DETAIL:  Key (username)=(jwhiting) is still referenced from table "userinfo_teams".

Version
0.12.1

PostgreSQL
What is the version of PostgreSQL ?
16

These changes are not necessary.


Fix cookie not deleted in stateless session

Fixes #26
This enables destroying the session when using stateless sessions, by making sure to delete the cookie in session.destroy()

Good catch! Thanks! Released as v2.1.1.


Ability to set rider responsibility per shift

The rider responsibility is currently set by the rider. For Freewheelers EVS, this represents the shift type that they ride.  Most riders have more than one shift type and it needs to be set for each shift.  It doesn’t work to set this per rider only.  It needs to be set per rider per shift. It would be logical for the coord to be able to do this at the start of a shift.
This information does need to be available for reporting. For example, how many jobs were on the East patch.

Hi Paul. Would it be enough for the coord to be able to set the responsibility on the user's profile before the shift starts?

As long as it’s intuitive that would work.
Thanks
Paul
From: Theo @.>
Sent: 31 March 2022 13:32
To: platelet-app/platelet @.>
Cc: PaulFreewheelersEVS @.>; Author @.>
Subject: Re: [platelet-app/platelet] Ability to set rider responsibility per shift (Issue #48)
Hi Paul. Would it be enough for the coord to be able to set the responsibility on the user's profile before the shift starts?
—
Reply to this email directly, view it on GitHub https://github.com/platelet-app/platelet/issues/48#issuecomment-1084519531 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AYPZ2XCXPP5ANTUXSJ3FKYLVCWLMFANCNFSM5SFCW6HQ .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AYPZ2XAJGRGFC7EL3YXEZH3VCWLMFA5CNFSM5SFCW6H2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOICSHI2Y.gif Message ID: @.*** @.***> >

The rider responsibility can now be set on the user's profile page.


How to read excel template file and replace a text finded

How can I implement this flow when got workbook read from excel file.
Next, find cell with text "" and modify it to become TextA, update to workbook
Write new excel file .

if i read an excel file with format(eg: color, border), the outfile displays a plain with data without styles

@abelmagana88920 did you ever get an answer for this? Im experiencing the same thing

no, I haven't receive any response from them. I'm currently using xlsx-populate which can inherit the styles of the template but has a limited features. It cannot style programatically but can populate data.

@abelmagana88920 @TheJegan if you could share a sample template, it would help in terms of understanding what features would need to be read and written from the files.
There are a few libraries that specialize in the particular use case of reading a file and performing a find/replace.  Full disclosure: I have not used either module in a serious way and cannot vouch for their applicability and fitness:

xlsx-populate gives more direct cell-level access to the workbook.
xlsx-template lets you make a template and populate it with data

They do the obvious thing: unzip the xlsx file, search cells until it finds the placeholder token, manually replace the placeholder string with the substitution, then save the xlsx file.  This approach generally preserves the rest of the original file, but is restricted to XLSX templates.
js-xlsx reads the entire file and produces a JS representation of the data.  Partially due to the general approach and partially due to the full spectrum of formats supported, preserving the original file features is limited to features that are currently parsed and features that are currently written.
The approach in https://github.com/SheetJS/js-xlsx/issues/121#issuecomment-56834484 is the generally recommended way of manipulating the JS object.   As more features are implemented, the output workbook will slowly converge to what you expect.  In the meanwhile, if features aren't represented, we recommend using another library like the aforementioned examples.
ping @dtjohnson for posterity sake can you provide some sample code using xlsx-populate for the case of a single find/replace?
ping @optilude for posterity sake can you provide some sample code using xlsx-template for the case of a single find/replace?

@reviewher, you are spot on about how xlsx-populate works. The intention is not to support the full XLSX spec nor a variety or formats. The idea is to modify the existing workbook to set values without impacting the styling or anything else in the workbook.
There is no support in xlsx-populate for manipulating styles at this point. It is on the roadmap though if I can ever get time to work on it. There is also no support for searching for a particular value. It should be possible but it is tricky with shared strings and formulas.
People use xlsx-populate to manipulate cell values by cell address or name:
var Workbook = require('xlsx-populate');
var workbook = Workbook.fromFileSync("./workbook.xlsx");
var sheet = workbook.getSheet("Sheet1");

sheet.getCell("A1").setValue("hi");
workbook.getNamedCell("foo").setValue("ho");

workbook.toFileSync("./out.xlsx");

@abelmagana88920 xlsx-populate might be what I need then. I don't only need to manipulate cell value while preserving the style. Thanks will look into that.

@dtjohnson thanks for chiming in!    Following the example from https://github.com/SheetJS/js-xlsx/issues/121#issuecomment-56834484 , I'd like to write:
var Workbook = require('xlsx-populate');
var workbook = Workbook.fromFileSync("./workbook.xlsx");
var sheet = workbook.getSheet("Sheet1");

worksheet_find_and_replace(sheet, "OLD", "NEW"); // this will change all of the text cells with value "OLD"

workbook.toFileSync("./out.xlsx");

The features og xlsx-populate can inherit the the style of excel template but It cannot  set the styles (borders, color) programatically.
The features of js-xlsx can do the styles programatically but I don't know if this package  can used an excel template to preserved the style.
It is very nice if the both features combine.
@TheJegan  your welcome. I'm glad that I share it with you.
@dtjohnson xlsx-populate is good. Tnx for this package
@reviewher Tnx for helping for giving us a reference

@dtjohnson I know this isnt the thread for this, but how do u read a cell value using xlsx-populate.
sheet.getCell("A1") just returns a giant object

I responded to the ticket for this here:
https://github.com/dtjohnson/xlsx-populate/issues/15
But I wanted to share the same info here. Currently xlsx-populate does not support reading the value of a cell. It was designed for population. However, I've started work on the next version of the API which would allow for getting values (as well as a number of other features like styling and search/replace). It'll be a bit before I get it released though :(.

I just released v1.0.0 of xlsx-populate. It now supports reading cell values, styles, etc. It also supports find/replace.
@reviewher, @abelmagana88920, here is an example of find and replace:
const XlsxPopulate = require('xlsx-populate');
XlsxPopulate.fromFileAsync("./workbook.xlsx")
    .then(workbook => {
        // Find and replace in the entire workbook
        workbook.find("OLD", "NEW");

        // Find and replace only in a particular sheet
        workbook.sheet("Sheet1").find("OLD", "NEW");

        // Find and replace only in a particular cell
        workbook.sheet(0).cell("A5").find("OLD", "NEW");

        // Write to file
        return workbook.toFileAsync("./out.xlsx");
    });

The find method also supports RegExp patterns and function replacements just like String.prototype.replace.

@dtjohnson, Thanks for updating your package xlsx-populate. I will test it. I'm sure it helps a lot. Thanks again

@dtjohnson,Hii can you look into this method.I have to call this method from another page and i have to return the contents of execute array from this method
exports.RetriveValue=  function( filepath){
    const XlsxPopulate = require('xlsx-populate');
	var workbook = XlsxPopulate.fromFileAsync(filepath);
	//var sheet = workbook.getSheet("Sheet1");
           const rowValue =[];
    	   var execute =[];
    	   var xlRange = workbook.sheet("Sheet1").usedRange();
    	   var rowCount =  workbook.sheet("Sheet1").usedRange(). _numRows;
    	   var colCount =  workbook.sheet("Sheet1").usedRange(). _numColumns;

    	 for (var i = 2; i <= rowCount; i++)
            {
    		 rowValue.length= 0;

                for (var j = 1; j <= colCount; j++)
                {
                try
                    {
                      rowValue.push(workbook.sheet("Sheet1").cell(i, j).value().toString());
                    }
                    catch ( ex)
                    {
                        break;
                    }
                }
                if (rowValue[4] == "Yes")
                {
                	var testexec = rowValue[2].toString();
					  execute.push(testexec);

                }

             }
    	 console.log( execute);
    	return execute;

}

This isn't the right repo for questions about xlsx-populate, but the issue is that fromFileAsync returns a promise.

I implement xlsx template engine: https://github.com/optimistex/xlsx-template-ex
It bases on exceljs. Try it. May be it will be useful for you!?

@nvcken It was discussed in a previous issue: #82
Here is a sample that should find all cells with text "OLD" and replace it with "NEW" (I left some comments that should hopefully show how you do certain things like loop through all of the cells):
var oldtext = "OLD";
var newtext = "NEW";

var XLSX = require('xlsx'); // require the module

/* read the file */
var workbook = XLSX.readFile('original.xlsx'); // parse the file
var sheet = workbook.Sheets[workbook.SheetNames[0]]; // get the first worksheet

/* loop through every cell manually */
var range = XLSX.utils.decode_range(sheet['!ref']); // get the range
for(var R = range.s.r; R <= range.e.r; ++R) {
  for(var C = range.s.c; C <= range.e.c; ++C) {
    /* find the cell object */
    var cellref = XLSX.utils.encode_cell({c:C, r:R}); // construct A1 reference for cell
    if(!sheet[cellref]) continue; // if cell doesn't exist, move on
    var cell = sheet[cellref];

    /* if the cell is a text cell with the old string, change it */
    if(!(cell.t == 's' || cell.t == 'str')) continue; // skip if cell is not text
    if(cell.v === oldtext) cell.v = newtext; // change the cell value
  }
}
XLSX.writeFile(workbook, 'newfile.xlsx');

It's working but excel style not available in downloaded file. Any one have a solution of it?


Add protections against SQL injection

The Kusto client takes raw Strings as queries, which is a very dangerous practice.
If part of the query contains a user-provided String (e.g. in a where clause), a malicious user could easily modify the query to run arbitrary code against the cluster.
For example consider something like String query = "Users | where UserId = " + userIdParam"; If userIdParam turns out to be the string 123 ; Users | take 100, we'd end up running the query `"Users | where UserId = 123; Users | take 100" which would return both the records for user 123 as well as 100 other records that might belong to other users. This can be used as a way of gaining access to forbidden information as well as performing denial of service attacks by running computationally expensive queries.
Every database driver I've ever seen (both relational and NoSQL) has protections against SQL injection in the form of either query builders or prepared statements, but I don't see anything of that sort in Kusto.

@yihezkel, please see that we expose the parametrized queries abilities in Java

Having a protection against SQL injection is a potential blocker and mandatory for our adoption of ADX.
Is there an expected timeline for such a feature and any workaround until this will be available?

jPrest The obvious workaround is employing very strict input validation although I completely agree that this is something that should be provided by the sdk.

@maltalex
Fully agreed, this is a feasible workaround.
But it is very error prone and thus a constant security risk, especially if the code base is evolving and features get added.

@jPrest thanks for the ping on this. We don't have a specific timeline, but it is the 2nd highest priority feature request in this SDK, so I intend to implement it soon.

@yihezkel is there any update on this topic?

Hi @taulant01,
I looked into this further, and see that parameterized queries are already supported in the Java SDK. Please see the following test I wrote:
   @Test     void testParameterizedQuery() throws DataServiceException, DataClientException {         ClientRequestProperties crp = new ClientRequestProperties();         crp.setParameter("nameParam", "myName");         crp.setParameter("metricParam", "8");         String query = "declare query_parameters(nameParam:string, metricParam:string); TestTable | where Name == nameParam and Metric == metricParam";         KustoOperationResult resultObj = queryClient.execute("TestDatabase", query, crp);         KustoResultSetTable mainTableResult = resultObj.getPrimaryResults();         mainTableResult.next();         int results = mainTableResult.getInt(2);         assertEquals(8, results);     }
I will try to commit a modified version of this test so others will see an example of how to use it.

I'm adding a significantly more robust interface for setting parameterized query values. Please let me know if you have any issues with it.


fix: preview inline error in non-buffer windows

Added a condition to check whether the bcache is nil or not in get_hunks of lua/gitsigns/actions.lua.
Fix #979

Okay, we can fix that "where it is being passed nil"


Parent page scroll when scrolling inside long md-select dropdown

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Parent page with scrollbar does not scroll when scrolling inside md-select dropdown
What is the current behavior?
The md-select dropdown has fixed position in the page. When scrolling a long md-select dropdown (with scrollbar) in parent page (which is also with scrollbar), the parent page scrolls together with the select dropdown.
What are the steps to reproduce?
https://plnkr.co/edit/pm4iFZk5iimvQbRWpjpt?p=preview
Which versions of Angular, Material, OS, browsers are affected?
OS: Windows 8.1, 10
Browsers: Chrome 55.0.2883.87 m (64-bit), IE 11, Firefox 52.0a2 (32-bit)
Angular: 2.2.0
Material: 2.0.0-beta.1

Should be resolved once we're able to block scrolling https://github.com/angular/material2/pull/1971

Related to #4093

Will this be synced with AngularJS/Material? I'm noticing the same issue there.


wanted: [hyperrealm/libconfig]

Module location
https://github.com/hyperrealm/libconfig
Link to bzlmod issue in the module's repository
No response
Any other context to provide?
Used by:

https://github.com/kubevirt/kubevirt
https://github.com/JFreegman/toxic

Fund our work

[ ] Sponsor our community's open source work by donating a feature bounty

Solved in https://github.com/bazelbuild/bazel-central-registry/pull/2456


Learning to use clr-loader

I am learning to use clr-loader because my environment does not allow non-pure python libraries.
If I were using pythonnet I would something like:
>>> import clr
>>> clr.AddReference('System.Data')
<System.Reflection.RuntimeAssembly object at 0x00000229C0EC3288>
>>> from System.Data import DataTable
>>> DataTable()
<System.Data.DataTable object at 0x00000229C0EC3288>

What would be the equivalent in clr-loader?
FYI, my goal is to call the Server constructor of this.

clr_loader is extremely limited with regard to what you can call, pretty much exactly what is given in the example: A function with the signature static int ...(IntPtr arg, int size)
The .NET Core (or 5, 6, ...) hosting api is intentionally very limited. If you need more, please use Python.NET.


Password reset email uses bare mercycorps.org link, omits hostname

Currently the "sites" setting on all our servers (QA, dev2, production) is set to "mercycorps.org" which redirects to www. We need to set this explicitly to [hostname].mercycorps.org if only for the sake of our password reset email:

Update: per conversation with @sanjuroj I am going to extend the auth view that sends this email so it can get a proper hostname using request.build_absolute_uri() or a global setting

QA Passed:

Dev 2: Both gmail and password page links were accessible as expected

QA: Both gmail and password page links were accessible as expected


Adjust assignments to new autoupdate

Adjust assignment list and detail to be compatible to the new
autoupdate system

Number of candidates badge shows 0

was apparently fixed by your commit

reload doesn't work
candidates are not shown

Fixed thouse. Polls seem to be a little more complicated to resolve.
Please note the FIXME, not entirely sure if I am using the candidates / users the intended way.

Number of candidates badge shows 0

was apparently fixed by your commit

I cannot confirm this. If I reload the list, both badges show 0. You should check, if the counting relies on just the id-array (like candidate_ids.length) or actually counting view-models (something like candidates.length). In the latter case, you need to either change the logic, or request the candidates, too (without users should be sufficient).
A note about the fieldset you changed: It is not worthy to include id-fields there. They will not be resolved and most time, they are useless without resolving them. And for things like the list, where you might need just the id-array, you can include it in additionalFields. THis is a list of fields to request next to the fields in the fieldset.

See comments above. The detail page works fine (except for polls, this is another thing for later).

no comments to resolve except

I cannot confirm this

Which is pretty odd. Perhaps due to the client?

no comments to resolve except

The fixme is ok, so it can be removed.
Regarding the candidate-amount-badge: Let's talk about this after the meetings later.

Number of candidates badge shows 0
ok. I confused the speaker badge with the amount of candidates.

could be done now.
I added the candidate_ids (array of numbers) to the list, since it's the most reliable and direct way to determine the amount of candidates on the list view.


Options not being injected properly

I think there is a problem with the way the tree options are being injected, I specifically had a problem with the injectClasses options, it seems that the attributes are not always on the $scope when ensureDefault() is being called  in the controller.
As per http://stackoverflow.com/questions/15151116/angularjs-directives-attributes-access-from-the-controller
The values of the attributes / scope will not be available until the compile phase, so these lines
https://github.com/wix/angular-tree-control/blob/master/angular-tree-control.js#L58 will need to be changed to use a $watch? Or be moved to the later compile function.
Interestingly this problem only occurs with a brand new incognito tab, if you refresh the tab once the page has loaded it all seems to work fine. I guess this is some race condition to do with loading resources for the first time.. not sure.

I'm seeing the same issue in IE.

Occurs in Chrome Version 40.0.2214.115 m also in Firefox 35.0.1 running angularjs 1.2.28 and jQuery 1.11.2


Allow Multiple Accounts

Right now the app works by accepting a single access token. I should design a UI to support multiple accounts and allow for adding new accounts/tokens, as well as account/token management.

I’m optimistically closing this since v1.1 added support for teams!


mouse: adding synonyms for starting left drag.

I would regularly say drag start which would cause an asterisk to be inserted.

This makes sense from a consistency perspective, thanks. I'll leave this open for others to chime in.


eksctl: 0.189.0 -> 0.190.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/weaveworks/eksctl/releases.
meta.description for eksctl is: CLI for Amazon EKS
meta.homepage for eksctl is: https://github.com/weaveworks/eksctl
meta.changelog for eksctl is: https://github.com/eksctl-io/eksctl/releases/tag/v0.190.0
Updates performed

Golang update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.190.0 with grep in /nix/store/qsjbh6vd69d2r4baqhxbzjfk8pzdydmy-eksctl-0.190.0
found 0.190.0 in filename of file in /nix/store/qsjbh6vd69d2r4baqhxbzjfk8pzdydmy-eksctl-0.190.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

eksctl
nuv

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qsjbh6vd69d2r4baqhxbzjfk8pzdydmy-eksctl-0.190.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A eksctl https://github.com/r-ryantm/nixpkgs/archive/123b06425c10b367ced57fa67994b23e8efb523c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/123b06425c10b367ced57fa67994b23e8efb523c#eksctl

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/qsjbh6vd69d2r4baqhxbzjfk8pzdydmy-eksctl-0.190.0
ls -la /nix/store/qsjbh6vd69d2r4baqhxbzjfk8pzdydmy-eksctl-0.190.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

eksctl
nuv

Maintainer pings
cc @xrelkd @Chili-Man for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@NixOS/nixpkgs-merge-bot merge


KAFKA-2720: expire group metadata when all offsets have expired

cc @onurkaraman @guozhangwang
This patch moves group metadata removal to the periodic offset expiration thread so that group metadata (including the generation) is preserved until all offsets for the group have expired. Since the last member may leave the group before offsets have expired, this patch changes the behavior to increment the generation and essentially write an otherwise empty group metadata object to the log. The main idea is to avoid resetting the generation to 0 until it is safe to do so. This also helps users who currently have no easy way to tell if the group simply doesn't exist or it is dead with possibly some offset state left behind.
Couple notes on the current patch:

I've added a new Empty state to the coordinator's group state machine. This may be unnecessary, since it seems Stable could work as well, but I preferred slightly to keep the logic for this case separate. It also gets around the need to have a direct transition from PreparingRebalance to Stable, which should not be possible normally. Instead we allow PreparingRebalance to transition to Empty when there are no members left in the group.
I've done a little bit of additional cleanup. The most notable change is the removal of the offset expiration lock in GroupMetadataManager, which seemed unnecessary with a small change to the offset loading logic. Rather than loading offsets directly into the cache, I first stage them in a local collection as is done with the group metadata. This should prevent the expiration thread from seeing stale values, but let me know if I've missed something.

Just browsed through the code change, regarding 2) I am still not clear how it prevent concurrent access on the offsetCache. Let's discuss offline.

Specifically, could this issue be likely to triggered even without log compaction stopping? (second incident of the following blog post):
https://engineering.linkedin.com/blog/2016/05/kafkaesque-days-at-linkedin--part-1

Summary of brief offline discussion: I think we agree that concurrent loading and expiration is safe with this change, but we are trying to find a way to also solve the problem of expiration with a current offset commit which is unprotected with and without this patch.

hey @hachikuji I'll try to get to this patch after I finish up testing the topic indexing idea you mentioned in the "[DISCUSS] scalability limits in the coordinator" thread.

Also ping @jjkoshy for reviews as well since you fixed the problem of race condition between offset expiration and added the lock; we are trying to safely remove the lock, and at the same time fix the race condition between offset expiration and offset commit as well.

@onurkaraman @guozhangwang Ready for another look. The main change I made is moving the offset cache of each group into GroupMetadata. For groups which only use offset commits, the group will stay in the Empty state until all offsets for the group expire. The change helps to unify expiration logic and addresses the race condition between offset commits and offset expiration, which could previously result in an inconsistent cache. Now offset update/expiration is protected with the group lock.

Thanks @hachikuji , will review.
The Jenkins failure looks familiar to me. I think we already have a JIRA for it?

@guozhangwang Yeah, looks like an instance of KAFKA-3155.

The updated state machine diagram looks reasonable to me, and I will let @onurkaraman to take another check on it. One general comment about naming of removeGroup and removeGroupForPartition, and onGroupLoaded, onGroupUnloaded, could we differentiate them better by name the ones, for example: deleteGroup, evictGroupsFromCacheForPartition, and onGroupLoadedToCache onGroupEvictedFromCache. And also add some comments on when they could be triggered?

@guozhangwang @ijuma @Ishiihara @onurkaraman: I'm removing the WIP tag on this patch. I've cleaned up a few small problems and added additional test cases. Take a look when you have a chance.

@hachikuji All my comments are from in the previous pass and I do not have further ones, would be OK to go with it if others have made their own passes.

FYI: There are some system test failures that I'm looking into.

Found the problem. Here's a nearly clean run: http://testing.confluent.io/confluent-kafka-branch-builder-system-test-results/?prefix=2016-06-15--001.1465997374--hachikuji--KAFKA-2720--9ffe4d0/. The two failures appear to be unrelated.

LGTM. Merged to trunk.

Thanks @ijuma. I'll prepare a follow-up.


Timeline for v8

Hi, is there a timeline for v8 ?

Hello @nhaberl,
You should be expecting v8 between mid. to the end of November max.
Cheers, have a nice day! :)

Big hug!


Add more animation behaviors

Description of Change
Added more animations behaviors:

BounceInAnimation and BounceOutAnimation
ColorAnimation
HeartAnimation
RelRotateAnimation, RotateXAnimation and RotateYAnimation
RelScaleAnimation
TranslateAnimation
TurnstileInAnimation and TurnstileOutAnimation

API Changes
Changed FlipHorizontalAnimation and FlipVerticalAnimation classes to include FlipDirection property.
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[x] Has samples (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[x] Changes adhere to coding standard

[ ] Updated documentation

@jsuarezruiz Can you take a look at fixing the merge conflicts?

Thanks! However, we are no longer adding new features to Xamarin Community Toolkit, focusing on the .NET MAUI Community Toolkit.
Please open a New Feature Discussion to implement this feature in the .NET MAUI Community Toolkit.
I've posted more information about the Future Of Xamarin Community Toolkit here: https://devblogs.microsoft.com/xamarin/the-future-of-xamarin-community-toolkit/?WT.mc_id=mobile-0000-bramin


Support for user-provided tsserver.js path

Would be useful for projects where tsserver doesn't live in node_modules

@WesleyT4N  @siduck it's implemented https://github.com/pmizio/typescript-tools.nvim/pull/86 here can you test this branch?

@WesleyT4N @siduck it's implemented #86 here can you test this branch?

it works


[Feature] Per-app settings

OS
macOS 版本13.2.1（版号22D68）
LinearMouse
v0.7.7
Is your feature request related to a problem?
In adobe's software, scrolling becomes very fast
Describe the solution you'd like
hope to add the function of excluding software list.
Describe alternatives you've considered
No response
Additional context
No response

Hey, @burning!
Thanks for the feedback.
Currently, you can manually edit the configuration to add an app-specific scheme using app matching.

Per-app settings GUI has been implemented in v0.8.0-beta.1.


Tutorial on how to contribute

Better explain how to contribute and deal with Hugo

fixed with PR #116 and PR #117


Database migrations to put local DB into parity with production

This PR contains a series of Knex database migrations that should put local development/test databases into parity with the production database (in Heroku).
:warning: Some of these migrations are making arguably negative changes, such as removing NOT NULL constraints and indexes. 🙅🏻 Rest assured, this is intentional to get our databases into parity but I will follow-up with another PR of migrations to re-apply these positive precautions across the board. 👌🏻

Follow-up PR: https://github.com/ProgramEquity/amplify-back-end/pull/147

:mega: Importantly, this also adds a heroku-postbuild npm script to automatically execute the latest database migrations in production upon deployment (via internal hooks in the heroku/nodejs buildpack that our Heroku App uses).
As part of this update, I have also manually inserted knex_migrations records 3-9 into the production database in order to avoid attempting to run migrations that have effectively already happened (outside of Knex migrations):

@BeeXiong @teakopp @nawazkhan Still would like someone to give this a review but I can YOLO-admin-merge it if I must. 😅 🙏🏻

@andyfeller @wallace thoughts here?

Made the call to merge this without a review after waiting for a long time. 🤷🏻♂


CS lime-145, 5.2 incorrect repayment distribution

Description
The lender can withdraw repaid funds by calling withdrawRepayment. This calculates their share of the repaid interest and collateral and pays it to the calling lender based on the pool token share the lender has. But after withdrawing the repayments a malicious lender can simply transfer the pool tokens to another account and call withdrawRepayment again to receive another share of the repayments. This can be done until no funds to repay are left over.
Integrations Checklist

[ ]  Have any function signatures changed? If yes, outline below.
[ ]  Have any features changed or been added? If yes, outline below.
[ ]  Have any events changed or been added? If yes, outline below.
[ ]  Has all documentation been updated?

Changelog

Fixing repayment distribution while withdrawing repayments.

Event Signature Changes
Function Signature Changes
Features
Events

Codecov Report

Merging #288 (0a09fa8) into v1-alpha (f3e84c8) will increase coverage by 0.06%.
The diff coverage is 76.92%.

@@             Coverage Diff              @@
##           v1-alpha     #288      +/-   ##
============================================
+ Coverage     78.49%   78.56%   +0.06%
============================================
  Files            16       16
  Lines          1679     1689      +10
  Branches        309      311       +2
============================================
+ Hits           1318     1327       +9
- Misses          361      362       +1

Impacted Files
Coverage Δ

contracts/CreditLine/CreditLine.sol
84.01% <ø> (ø)

contracts/Pool/Extension.sol
84.05% <ø> (ø)

contracts/Pool/PoolFactory.sol
68.96% <ø> (ø)

contracts/SavingsAccount/SavingsAccountUtil.sol
86.04% <ø> (ø)

contracts/yield/NoYield.sol
78.57% <ø> (ø)

contracts/Pool/Pool.sol
94.33% <76.92%> (-0.15%)
:arrow_down:

Continue to review full report at Codecov.

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


Custom filename support

Pass in filename

Thanks for the pull request, but I will not merge it because it makes passing a file name mandatory, which is not something I want.
Making it optional would be fine thought.

As I wrote, I'd like to keep it flexible, so that the file name can also be derived. I'll release a new version where you can also specify to file name on an optional basis very soon. Thanks again for the input.


Support OAuth authentication for the Bigcommerce API.

This code is pulled in the mechatama/bigcommerce-api-ruby repo on the
oauth branch.

Updates to the README
OAuth support
Bump minor version
Remove unnecessary code

Coverage decreased (-1.73%) to 62.08% when pulling 30bea909b4dc6e2f345b2155e1a77a49cf2e1e84 on pedelman:add_oauth_support into f457030835845adce72cdf0d5eb2cfcd3ef274a1 on bigcommerce:master.

Coverage decreased (-1.73%) to 62.08% when pulling 30bea909b4dc6e2f345b2155e1a77a49cf2e1e84 on pedelman:add_oauth_support into f457030835845adce72cdf0d5eb2cfcd3ef274a1 on bigcommerce:master.

Coverage decreased (-1.74%) to 62.07% when pulling cfd6b108d17db4f9185fa5fbc41d4af2f37e303d on pedelman:add_oauth_support into f457030835845adce72cdf0d5eb2cfcd3ef274a1 on bigcommerce:master.

Overall, looks good, thanks for this. Please fix the ci-reporter stuff.

Changes Unknown when pulling 3defdfb80b6553b52e4eb92ad9154178f20e0ae4 on pedelman:add_oauth_support into ** on bigcommerce:master**.

Changes Unknown when pulling 3defdfb80b6553b52e4eb92ad9154178f20e0ae4 on pedelman:add_oauth_support into ** on bigcommerce:master**.

Changes Unknown when pulling 3defdfb80b6553b52e4eb92ad9154178f20e0ae4 on pedelman:add_oauth_support into ** on bigcommerce:master**.

Changes Unknown when pulling aaf3820029c3c1d747a77b2444d166e1e7eab86a on pedelman:add_oauth_support into ** on bigcommerce:master**.

Changes Unknown when pulling a28ecc821c971892f50ff15b03ad737603e3c1c4 on pedelman:add_oauth_support into ** on bigcommerce:master**.

Changes Unknown when pulling a6105d228284b4665df659b20ac12f92f13da41e on pedelman:add_oauth_support into ** on bigcommerce:master**.

Awesome, thank you!


Add memory arbitration strategy

Adds a process-wide MemoryManagerStrategy instance and an abstract
base class MemoryConsumer. The strategy keeps a list of consumers.
the consumers each have a top level MemoryUsageTracker that gives
their current usage and limit. The consumers have a method for that
returns an estimate of the memory they can free when requested. The
recover() method attempts to free a requested amount of memory. The
actually freed memory is reflected in the tracker.
Adds a TaskMemoryStrategy that arbitrates memory asks between
Tasks. The strategy is invoked via the growCallback of the Task top
level MemoryUsageTracker. TaskMemoryStrategy::recover first checks if
the process-wide top tracker has capacity and if so, increases the
limit of the requester by the requested amount. If not enough capacity
is available, this does a dirty read of all running Task's recoverable
memory and sortrs the potential memory donors largest first. This then
pauses the candidates one after the other and forces enough of them to
release memory to meet the demand. The paused tasks are then resumed.
The requested part of the memory ask is used to increase the limit of
the requester and anything left over is added as available in the top
tracker.  Adds a test with a custom operator that keeps growing a
reservation and can be made to decrese the reservation by contending
users.

Thank you for the review, addressed the comments. Answer to questions below:

int64_t recoverableMemory() const;

s/recoverableMemory/revocableMemory/ is more accurate in this context? Thanks!
A: Considering the meaning of the words, revoke' means to take away a right or privilege. It is on or off. Recover means among other things to re-acquire something lost, e.g. the part of the stolen money was recovered.' So, I'd say recover is better since it implies a scale rather than all or none.

topTracker->update(-tracker().maxTotalBytes());

Can we let the tracker dtor to release the bytes and propagate to its parent?
A: the top tracker is not the parent of the task tracker.
If the top tracker were the parent, memory recovered in the grow callback would be posted there. This is not wanted because then somebody else could get it from there. Instead, the grow callback gathers memory from tasks and does ont put it back into anything except at the end where it divides it betwen between the requested and top tracker.
A:

int64_t minSize =

Why we set minSize to kInitialSize if tracker.maxTotalBytes() < kMinTransfer? Thanks!
A: A task in this model starts with 0 reservation. Its first action is to call grow callback to get itself something. The initial request is rounded up for the purpose. The thread that makes the Task cannot go via the grow callback to the arbiter because only threads of the Task itself can do that. So getting the initial reservation via the growth route is most consistent and least special cases. The first Driver gets the initial reservation. The next might queue up to do the same but when it is its turn it will see the reservation is already increased.
pool_ should have already been assigned tracker. Can we just set grow callback to it?
A: We want a tracker that is a child of any other tracker. There is no such guarantee for the pre-existing one, if any.

 MemoryConsumer* consumer,

Can we use task id here as key to consumer?
A:The pointer to the Task is guaranteed unique whereas the id is not necessarily so. The pointer is the most compact

for (auto& driver : drivers_) {

NYC: in which case a driver slot could be null? Thanks!
A: During destruction or after the Driver is finished it could be nullptr, so need to check.

int64_t total = tracker().maxTotalBytes() - tracker().getCurrentUserBytes();

Shall we init total to available reservation bytes here?
A: The Task tracker is the sum of the reservations of child trackers. So the current user bytes is the sum that we want, it does not have a reservation of its own, rather the subsidiary reservations are added up and counted as child allocations. Reservation at the child means a guarantee to  be able to allocate so much, so to the parent it counts as already allocated.

     if (recoverable >= 0 /*kMinTransfer*/) {

if (recoverable > 0)? If it is zero, then we can revoke any memory from this task?
A: If there is less than min transfer to get from a Task, then there is no point trying to get anything.

 delta = roundDown(delta, kMinTransfer);

nit: how about kMinReclaimBytes/?
A: The constant means the smallest amount by which it makes sense to shrink a Task. The practical size is the full spill of one operator. This is, supposing 1GB tasks, around 50MB, supposing 20 threads per task.  The minimum should be small enough,~10MB.

Meng, Masha
I expect to still rearrange the files a bit, like extracting and renaming the arbiter class.
Thanks
Orri

available reser

Thanks for the explanation!

@oerling The PR description talks about arbitrating among Tasks, but doesn't explain who the same works for a single Task with multiple operators. Sapphire-on-Velox will use only one task, but run it multi-threaded. I assume we'll have multiple threads running same kind of operator. Would you describe how things work in that use cases? Can we have add a test that exercises this code path?

Thank you for the review, addressed the comments, details follow where applicable.

if (!numDrivers) {

When would this happen?
A: If the Task were shutting down. We could get references from the collection in the arbiter object.

// Per driver recovery target is 16MB + target / number of drivers.

Why 16MB ? What does "per driver recovery target" means? Different drivers may be running different pipelines, difference operators, hence, may have different memory usage.
A: There is a certain minimum below which recovering memory does not make sense. In practice The recovery consists of spilling the whole operator. The question is, how many operators have to be wholly spilled to make the space? Ch. comment.
+void Task::recover(int64_t size) {
Would it make sense for this method to return number of recovered bytes?
A: The actionable quantity is the level in the Task's tracker. For clarity we should refer to the level in the tracker as opposed to reading it and subtracting from it.
Shouldn't we use 'bytes' instead of 'minSize' here?
A: We use the rounded-up quantity minSize.
Do we allow the application to decide which operators should spill?
A: No. You could say that the preference would be order by first, group by next and join last. Join is a bit different because it must spill on all drivers in a coordinated way. This is possible though since at the time the external induced spill happens the task is stopped. If the task itself decides to spill then the deciding driver must stop the other ones. We'll get back to this  with hash join spill.
Generally the application could have a way to express preferences on spill choice but we need use cases to  see how this would go.


[BUG] Group message fails with Invalid condition found in packet header

Reproduction steps
I am trying to send a command to the OnOff cluster to a group of end devices. Below are my environment and reproduction steps:
Environment:
Matter Controller: chip-tool running on Linux
End devices: nRF52840 Dev Kits running modified Template from nRF Connect SDK v2.1.0
Reproduction steps:
Commission 2 end devices
sudo ./chip-tool pairing ble-thread 1 hex:0e080000... 20202021 3840
sudo ./chip-tool pairing ble-thread 2 hex:0e080000... 20202021 3841
Key set write to endpoint 0 of both end devices
sudo ./chip-tool groupkeymanagement key-set-write '{"groupKeySetID": "0x01a1", "groupKeySecurityPolicy": 0, "epochKey0":"a0a1a2a3a4a5a6a7a8a9aaabacadaeaf", "epochStartTime0": 1110000, "epochKey1":"b0b1b2b3b4b5b6b7b8b9babbbcbdbebf", "epochStartTime1": 1110001, "epochKey2":"c0c1c2c3c4c5c6c7c8c9cacbcccdcecf", "epochStartTime2": 1110002 }' 1 0
sudo ./chip-tool groupkeymanagement key-set-write '{"groupKeySetID": "0x01a1", "groupKeySecurityPolicy": 0, "epochKey0":"a0a1a2a3a4a5a6a7a8a9aaabacadaeaf", "epochStartTime0": 1110000, "epochKey1":"b0b1b2b3b4b5b6b7b8b9babbbcbdbebf", "epochStartTime1": 1110001, "epochKey2":"c0c1c2c3c4c5c6c7c8c9cacbcccdcecf", "epochStartTime2": 1110002 }' 2 0
Map key to group
sudo ./chip-tool groupkeymanagement write group-key-map '[{"fabricIndex": 0, "groupId": "0x0101", "groupKeySetID": "0x01a1"}]' 1 0
sudo ./chip-tool groupkeymanagement write group-key-map '[{"fabricIndex": 0, "groupId": "0x0101", "groupKeySetID": "0x01a1"}]' 2 0
Add endpoints to group
sudo ./chip-tool groups add-group  0x0101 "ARGB" 1 1
sudo ./chip-tool groups add-group  0x0101 "ARGB" 2 1
Send on command to group
sudo ./chip-tool onoff on 0xffffffffffff0101 1
This command returns an error on the end device, also attached output from chip-tool:
end device output:
E: 7401806 [IN]Invalid condition found in packet header
This points me to https://github.com/project-chip/connectedhomeip/blob/c279578c5bc37f117335aa96cec6c5552f070cc0/src/transport/SessionManager.cpp line 763
// Check if Message Header is valid first
    if (!(packetHeader.IsValidMCSPMsg() || packetHeader.IsValidGroupMsg()))
    {
        ChipLogError(Inet, "Invalid condition found in packet header");
        return;
    }

chip-tool
output.txt
Any suggestions are appreciated
Bug prevalence
Every attempt
GitHub hash of the SDK that was being used
16bcd30c33241b4b1c8993cfbc92e65f4de9708d
Platform
nrf
Platform Version(s)
v2.1.0
Anything else?
No response

16bcd30c33241b4b1c8993cfbc92e65f4de9708d

Which branch is that hash from?  I just tried the above steps on tip and they seem to work fine....

@bzbarsky-apple I am using Nordic Semiconductor connectedhomeip repo, which makes me question if this is the right place to post this error.
https://github.com/nrfconnect/sdk-connectedhomeip
v2.1-branch

@Jakebluhm I would start with reporting the error to them, then...

Thanks, @bzbarsky-apple
For anyone with similar issues of not being able to send multicast commands, I've created an issue for this error here -> https://devzone.nordicsemi.com/f/nordic-q-a/95698/matter---how-can-i-control-the-group-created-in-nrf-connect-sdk-matter-light-switch-and-light-bulb-samples-from-the-chip-tool-rather-than-the-light-switch

Hello @Jakebluhm I think our tech support replied.
The nRF Connect SDK 2.1.0 includes experimental and old version of Matter SDK (pre Matter 1.0) - so I'm not sure if group communication worked at this moment of time. I suggested to use nRF Connect SDK 2.2.0 which includes production support for Matter.
Btw. I have tested that with below commands group communication works well.
My setup:
SDK: nRF Connect SDK 2.2.0
HW: nRF52840
OTBR: RaspPi and nrfconnect/otbr:1813352
Sample: samples/matter/light_bulb build without any extra parameters
Chip-tool: From nRF Connect SDK 2.2.0 - https://github.com/nrfconnect/sdk-connectedhomeip/releases/tag/v2.2.0
# Commission device to the Thread network
sudo ./chip-tool-debug pairing ble-thread 1 hex:0e080000000000010000000300000f35060004001fffe0020811111111222222220708fd7b8210c9e19eb1051000112233445566778899aabbccddeeff030e4f70656e54687265616444656d6f010212340410445f2b5ca6f2a93a55ce570a70efeecb0c0402a0f7f8 20202021 3840
# Check that unicast onoff works well (observe LED2)
sudo ./chip-tool-debug onoff on 1 1
sudo ./chip-tool-debug onoff off 1 1
# Configure test group
sudo ./chip-tool-debug groupkeymanagement key-set-write '{"groupKeySetID": "0x01a1", "groupKeySecurityPolicy": 0, "epochKey0":"a0a1a2a3a4a5a6a7a8a9aaabacadaeaf", "epochStartTime0": 1110000, "epochKey1":"b0b1b2b3b4b5b6b7b8b9babbbcbdbebf", "epochStartTime1": 1110001, "epochKey2":"c0c1c2c3c4c5c6c7c8c9cacbcccdcecf", "epochStartTime2": 1110002 }' 1 0
sudo ./chip-tool-debug groupkeymanagement write group-key-map '[{"fabricIndex": 0, "groupId": "0x0101", "groupKeySetID": "0x01a1"}]' 1 0
sudo ./chip-tool-debug groups add-group 0x0101 "ARGB" 1 1
# Configure ACL (257 = 0x0101)
sudo ./chip-tool-debug accesscontrol write acl '[{"fabricIndex": 1, "privilege": 5, "authMode": 2, "subjects": null, "targets": null },{"fabricIndex": 1, "privilege": 3, "authMode": 3, "subjects": [257], "targets": null }]' 1 0
# Test that Group communication works as expected
sudo ./chip-tool-debug onoff on 0xffffffffffff0101 1
sudo ./chip-tool-debug onoff off 0xffffffffffff0101 1


Error in logits_processors.py - OutputNumbersTokens().call() for some models

Hello: When running the tiiuae/falcon-7b model, I get no issue using the package as intented. But some models, such as tiiuae/falcon-rw-1b will get an error in OutputNumbersTokens().call() like below:
The expanded size of the tensor (50304) must match the existing size (50257) at non-singleton dimension 1.  Target sizes: [1, 50304].  Tensor sizes: [50257]
I've been trying to debug this on my own but have not figured out why sometimes self.allowed_mask and scores sometimes have mismatching shapes (depending on model) that will cause the above error when trying to run:
self.allowed_mask.expand_as(scores)

I thought this was a bug in transformers or in the specific model, but I tried modifying the example notebook in lm-format-enforcer ( https://github.com/noamgat/lm-format-enforcer/blob/main/samples/colab_llama2_enforcer.ipynb ) which achieves a similar goal, to use tiiuae/falcon-rw-1b and it worked.
So I'm not sure

Hi, I encountered the same problem where there was a size mismatch when running other models. Do you know how to solve it?


os-dnscrypt-proxy: Missing ODoH aliases because of old base version

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] When the request is meant for an existing plugin, I've added its name to the title.

I would like to use the dnscrypt server alias "odoh-cloudflare" in "Services: DNSCrypt-Proxy: Configuration: Server List" but this is not possible because the base version seems to be 2.1.0 while ODoH servers were introduced in 2.1.2, to quote the dnscrypt-proxy changelog:

Version 2.1.2

dnscrypt-proxy -list now includes ODoH (Oblivious DoH) servers.

Is it possible that you update the dnscrypt-proxy to the latest version 2.1.5?
os-dnscrypt-proxy version: 1.15_1

Oh, I just looked at the plugin changelog which only mentioned "2.1". So it was probably not working because I didn't enter a ODoH relay server.
I just wanted to suggest to add "Base Version" in "System: Firmware: Plugins". But then I noticed it already is mentioned in "Packages".
It would be nice if there was some sort of link to the underlying package in "Plugins" that would be more convenient for non-developers. ☺

Basically all versioning of packages is the same. The most convenient think would be to add a note to the upstream version in the plugin changelog. If you want to propose a change to there feel free (changing 2.1 to 2.1.5 which seems correct).

I think I found the culprit and posted it here yesterday with my private account: https://github.com/opnsense/plugins/pull/3810
But I noticed another cosmetic bug in "DNSCrypt-Proxy: Log", the texts are not aligned to the column headings.

That's probably because it's directly parsed from dnscrypt-proxy and not translated to the OPNsense format.


Question: Is this repository still maintained?

Is this repository still maintained and new features will be added soon as the contributors decided to do so? Because this component helps me a lot on my project and I would like to know if I can contribute to it after I finish my project.

Hi @RENCENDANT,
There's no feature development planned on this project, but it is maintained. Feature proposals and pull requests will be reviewed, but there's no guarantee of a timely response.
Best regards,
Stefan


Space out errors and warnings for readability

I am working on  #308, and I'm noticing that when a lot of errors are generated, they can be very difficult to visually differentiate.

This PR adds (somewhat primitive) visual boundaries between the errors/warnings:

I don't know if this is the best way of doing this, and I'm open to feedback and/or using an alternate separator. But I do think this improves the UX a bit.

I guess --> error is the separator already, but visually, it all blends together to me. I'm open to being overridden here, I suppose.

I'm not sure what exactly changed, but I just reset the lockfile and rebuilt, and it got updated again. Maybe the toml changed on master but the lockfile wasn't committed?

Ah makes sense. Can you make a separate PR to update the lockfile in isolation?


Power Toys Run: command line arguments

Summary of the new feature/enhancement
Allow command line arguments for Power Toys Run commands
Proposed technical implementation details (optional)
notepad <arg1> "<arg .. 2>" <arg3>
Power Toys Run to find program (notepad in this example) from PATH. Then pass arguments to the matching command.

Closing this as a duplicate of #3405


Add a module for running Kroxylicious as a standalone application with logging

Is your feature request related to a problem? Please describe.
When I run kroxylicious/target/kroxylicious-0.3.0.jar after building with dist profile I see SLF4J: No SLF4J providers were found. This is fair that the kroxylicious module is unopinionated about logging implementations since we could use that code for embedding Kroxylicious and use a supplied slf4j backend. But we sometimes want to run Kroxylicious from our terminals and having logs would be useful, you can build up a command yourself to do it but it's inconvenient.
Describe the solution you'd like
We could add a kroxylicious-app module and make it responsible for being our opinionated way to run Kroxylicious, it would be responsible for picocli code and supplying a logging implementation.

TODO split into a couple of issues

Implemented by #499


Unable to add MSZ-AP50VGD

I am currently running home assistant in Docker on a Windows machine. After adding echonet Lite to my install, I am unable to add the heatpump. When adding the static IP of the heatpump, I get an error "Failed to connect."
The logs also show:
2022-12-09 10:03:11.602 DEBUG (MainThread) [custom_components.echonetlite.config_flow] Beginning ECHONET node discovery
2022-12-09 10:03:51.904 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONET Node Discovery Failed!

Is there any other form of logging I can add to get more details? From looking at the code base, it simply isn't getting a response on the discover endpoint.
I have confirmed I can ping the device from the container.

Added additional logging found in another closed case. My results are
homeassistant  | s6-rc: info: service s6rc-oneshot-runner successfully started
homeassistant  | s6-rc: info: service fix-attrs: starting
homeassistant  | s6-rc: info: service fix-attrs successfully started
homeassistant  | s6-rc: info: service legacy-cont-init: starting
homeassistant  | s6-rc: info: service legacy-cont-init successfully started
homeassistant  | s6-rc: info: service legacy-services: starting
homeassistant  | services-up: info: copying legacy longrun home-assistant (no readiness notification)
homeassistant  | s6-rc: info: service legacy-services successfully started
homeassistant  | 2022-12-10 00:47:05.379 WARNING (SyncWorker_2) [homeassistant.loader] We found a custom integration echonetlite which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
homeassistant  | 2022-12-10 00:47:06.085 WARNING (Recorder) [homeassistant.components.recorder.util] The system could not validate that the sqlite3 database at //config/home-assistant_v2.db was shutdown cleanly
homeassistant  | 2022-12-10 00:47:06.532 WARNING (Recorder) [homeassistant.components.recorder.util] Ended unfinished session (id=7 from 2022-12-09 23:52:34.728505)
homeassistant  | 2022-12-10 00:48:36.829 DEBUG (MainThread) [custom_components.echonetlite.config_flow] IP address is <IP_ADDRESS>
homeassistant  | 2022-12-10 00:48:36.830 DEBUG (MainThread) [custom_components.echonetlite.config_flow] Beginning ECHONET node discovery
homeassistant  | 2022-12-10 00:48:36.847 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONETLite Message Received from <IP_ADDRESS> - Raw data is b'\x10\x81\x00\x01\x0e\xf0\x01\x05\xff\x01r\x03\x8a\x03\x00\x00\x06\x83\x11\xfe\x00\x00\x06\x01\x05pa\xbe\xff\xfe\x83KJ\x0e\xf0\x01\xd6\x04\x01\x010\x01'
homeassistant  | 2022-12-10 00:48:36.847 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONETLite Message Received from <IP_ADDRESS> - Processed data is {'EHD1': 16, 'EHD2': 129, 'TID': 1, 'SEOJGC': 14, 'SEOJCC': 240, 'SEOJCI': 1, 'DEOJGC': 5, 'DEOJCC': 255, 'DEOJCI': 1, 'ESV': 114, 'OPC': [{'EPC': 138, 'PDC': 3, 'EDT': b'\x00\x00\x06'}, {'EPC': 131, 'PDC': 17, 'EDT': b'\xfe\x00\x00\x06\x01\x05pa\xbe\xff\xfe\x83KJ\x0e\xf0\x01'}, {'EPC': 214, 'PDC': 4, 'EDT': b'\x01\x010\x01'}]}
homeassistant  | 2022-12-10 00:48:36.847 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONETLite Message Received from <IP_ADDRESS> - tid_data is {}
homeassistant  | 2022-12-10 00:48:36.847 DEBUG (MainThread) [custom_components.echonetlite.config_flow] Unknown ECHONETLite node has been identified - <IP_ADDRESS>
homeassistant  | 2022-12-10 00:49:17.127 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONET Node Discovery Failed!
homeassistant  | 2022-12-10 00:58:44.702 DEBUG (MainThread) [custom_components.echonetlite.config_flow] IP address is [Auto Discovery].k,um 😊😊😊+++++000+0+33++++++++++++++++++++++++++++++++++++++++++++++++++                     hhhhh                      6nnnn5hy4ttttttttttttttq``````````````.........................................0
homeassistant  | 2022-12-10 00:58:44.703 DEBUG (MainThread) [custom_components.echonetlite.config_flow] Beginning ECHONET node discovery
homeassistant  | 2022-12-10 00:58:44.705 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
homeassistant  | Traceback (most recent call last):
homeassistant  |   File "/usr/local/lib/python3.10/site-packages/pychonet/lib/udpserver.py", line 113, in _send_periodically
homeassistant  |     bytes_sent = await self._sock_send(data, addr)
homeassistant  |   File "/usr/local/lib/python3.10/site-packages/pychonet/lib/udpserver.py", line 92, in _sock_send
homeassistant  |     bytes_sent = self._sock.sendto(data, addr)
homeassistant  | TypeError: encoding of hostname failed
homeassistant  | 2022-12-10 00:59:24.948 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONET Node Discovery Failed!
homeassistant  | 2022-12-10 06:21:48.157 WARNING (MainThread) [homeassistant.components.http.ban] Login attempt or request with invalid authentication from <IP_ADDRESS> (<IP_ADDRESS>). Requested URL: '/auth/login_flow/d5bdc93575acbc9dfdc06e75d3655b01'. (Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36)
homeassistant  | 2022-12-10 06:22:13.526 DEBUG (MainThread) [custom_components.echonetlite.config_flow] IP address is <IP_ADDRESS>
homeassistant  | 2022-12-10 06:22:13.529 DEBUG (MainThread) [custom_components.echonetlite.config_flow] Beginning ECHONET node discovery
homeassistant  | 2022-12-10 06:22:13.552 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONETLite Message Received from <IP_ADDRESS> - Raw data is b'\x10\x81\x00\x01\x0e\xf0\x01\x05\xff\x01r\x03\x8a\x03\x00\x00\x06\x83\x11\xfe\x00\x00\x06\x01\x05pa\xbe\xff\xfe\x83KJ\x0e\xf0\x01\xd6\x04\x01\x010\x01'
homeassistant  | 2022-12-10 06:22:13.552 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONETLite Message Received from <IP_ADDRESS> - Processed data is {'EHD1': 16, 'EHD2': 129, 'TID': 1, 'SEOJGC': 14, 'SEOJCC': 240, 'SEOJCI': 1, 'DEOJGC': 5, 'DEOJCC': 255, 'DEOJCI': 1, 'ESV': 114, 'OPC': [{'EPC': 138, 'PDC': 3, 'EDT': b'\x00\x00\x06'}, {'EPC': 131, 'PDC': 17, 'EDT': b'\xfe\x00\x00\x06\x01\x05pa\xbe\xff\xfe\x83KJ\x0e\xf0\x01'}, {'EPC': 214, 'PDC': 4, 'EDT': b'\x01\x010\x01'}]}
homeassistant  | 2022-12-10 06:22:13.552 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONETLite Message Received from <IP_ADDRESS> - tid_data is {}
homeassistant  | 2022-12-10 06:22:13.552 DEBUG (MainThread) [custom_components.echonetlite.config_flow] Unknown ECHONETLite node has been identified - <IP_ADDRESS>
homeassistant  | 2022-12-10 06:22:53.790 DEBUG (MainThread) [custom_components.echonetlite.config_flow] ECHONET Node Discovery Failed!

It looks like <IP_ADDRESS> was detected and sent a request, but the reply comes from <IP_ADDRESS>. Check IP address mapping for UDP communication between host/Docker container.

@wroof echonet lite uses port number 3610 for UDP communication, so try port forwarding port 3610 in Docker settings. Ask the Docker community how to do that.

Thanks @nao-pon , sorry for the late reply, I have been on holiday the last few weeks and not at my computer.
I guess I should have posted my docker-compose file. I already have port 3610 mapped for UDP and TCP. Not sure how to stop the remapping of IP addresses though. I will ask around on some Docker forums unless you know how?
services:
  homeassistant:
    container_name: homeassistant
    image: "ghcr.io/home-assistant/home-assistant:stable"
    volumes:
      - E:\Docker Config\Home Assistant\Config:/config
      - /etc/localtime:/etc/localtime:ro
    restart: unless-stopped
    privileged: true
    ports:
      - 8123:8123
      - 3610:3610/udp
      - 3610:3610/tcp

Docker seems to have a method to fix the IP address of the container to the same network segment as the host, so I think that method is also effective.

I really don't want to be "that guy" but running Home Assistant in a Windows Docker environment is not really an officially recommended method. The suggested methods for a Windows environment are running HAOS in a VM (eg Hyper-V) or running Home Assistant Core directly in a WSL Python environment. https://www.home-assistant.io/installation/windows
Putting my network engineering hat on here and there is some sort of weird NAT thing going on where the HVAC address is being natted to the Docker enviroment 172.20.x.x range.
This is really a corner case here and traditionally I have closed off these sorts of issues because the expectation is that the Home Assistant environment at the very least resembles an officially recommended. Yes Docker container deployments of Core are supported and yes Windows environments are supported, but not really the two in combination...


MatTree examples not applying material styling

What is the expected behavior?
I'm trying to recreate the examples found here https://material.angular.io/components/tree/overview
What is the current behavior?
The examples all behave strange: The nested tree example shows only top level chevrons, not even node names and appears unstyled. The flat architecture does list all nodes but still looks unstyled.

What are the steps to reproduce?
The issue is not reproducible in StackBlitz.

Initialize new Angular project: ng new mat-test
Angular Routing: Y
Stylesheets: SCSS
ng add @angular/material
Indigo/Pink theme
HammerJS: Yes
Copy paste example code into a new component (TS & HTML code)

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 7
Is there anything else we should know?
I first got a yellow notification in the console about a missing stylesheet, but the ng add command did import the indigo theme stylesheet into angular.json's styles array.

The example uses <button mat-icon-button> and it looks like you may not have imported the MatButtonModule which would bring in the correct styles.
Try adding MatButtonModule to your module:
...
import {MatButtonModule} from '@angular/material';

@NgModule({
  ...
  imports: [
    ...
    MatButtonModule,
  ],
})
export class AppModule { }

If this doesn't work, feel free to open the issue again


ThreeJS version update to r115?

When I get some time I'll take a look at updating three.js.

Yeah, this is a necessary feature. I am currently working with this repo but there were some issues with gltf loader. When I upgraded to v115 everything worked perfectly. Unfortunately, build process is failing because of the three js version upgrade. Thank you beforehand

When I get some time I'll take a look at updating three.js.

I need your help bro. Please update to r115 asap brooo😭

Sorry, it took forever for me to find the time to get to this. I just updated all the packages to latest with the newest release and I added the ability to load via GLTF. Let me know if you see any issues after using that release.


Shifted installation of helm, for local-volume/helm/test/run.sh

It is no longer installed during execution of integration test automatically, as root.
It is installed during travisci test.sh.

FYI: I'm contacting linux foundation about CLA issue.

/lgtm

Triggering CLA again.

Comment for CLA

/assign @wongma7


Time stamping entries

Is there a preferred way to capture the datetime of an entry? Noticed there's a {@timestamp} placeholder in the documentation but couldn't figure out how to use that for capturing comment data.
I'm looking to get .yml datafiles something like this to iterate on and spit out in my Jekyll layouts:
name: Your Name
date: 2015-06-26T04:39:00-04:00   #comment timestamp
email:   #MD5 hash
url:
message: "Your message."

Thanks!

The {@timestamp} placeholder can only be used to specify the filename and the path of the data file. At the moment, there is no way to inject a timestamp into the actual entry (which is what you're after, right?).
If so, it's definitely something I can implement. How do you envision this working from a user perspective? As in, how would you tell Staticman that it needs to inject the current date in a field called date? And things like the format to use, etc.
Any thoughts will be appreciated. :)

Yes, injecting a timestamp of some sort into each entry is what I'm after. If that can happen at the time of it's creation that would be great.
As a workaround I was trying to find a way to assign a date to a hidden field
<input type="hidden" name="field[date]" value="{{ site.time }}">

But obviously that went nowhere as I couldn't really find a way to grab the current time dynamically. site.time is only as good as your last deployment...
So if there was a way to trigger Staticman to include a timestamp or date value in the datafile at creation that would be beneficial.
Noticed in your personal site repo that you were capturing the date in your comment datafiles. That's what led me down this road. That and comment's typically include that sort of metadata.

Ah, you're right! That was being used in the previous iteration of Staticman and hasn't been implemented in the rebuilt app.
I'll add this tomorrow. Thanks.

I've added support for this using the concept of generated fields. At the moment, the only type of generated field is date, but we could introduce other dynamic fields in the future (like geolocation, language, etc.)
I need to add this to the docs, but in the meantime here's an example of how you can use it to add a timestamp to your posts:

This is fantastic. Seems to be working well for timestamp.
One thing I noticed... and I don't know if this is an issue or not.
Using format: timestamp generates something like date: '2016-08-11T12:27:02.882Z' which when filtered through Liquid {{ date | date_to_xmlschema }} outputs as 2016-08-11T13:27:02+01:00 as expected.
But if switched to format: iso8601 it looks like the year is completely wrong when filtered.
date:<PHONE_NUMBER>344 filtered with {{ date | date_to_xmlschema }} outputs as 48581-07-27T11:32:24+01:00
Any ideas there? I was able to filtered other Unix timestamps OK with date_to_xmlschema, so not sure what the deal is.
Example: date:<PHONE_NUMBER> filters to 2015-05-11T17:18:00+01:00.

That's because timestamp is returning the Unix timestamp in milliseconds. I've added a timestamp-seconds which returns seconds. Give that one a try.
I wonder if I should make seconds the default and create a timestamp-milliseconds.

Tested timestamp-seconds and that works. I don't use Unix timestamps enough to have a preference either way. The fact that the format can be set to either works for me.
Maybe lean on the side of defaulting to seconds since that's how some of the Jekyll/Liquid filters are looking for it. Not sure what other static-site gens prefer.

Added to docs: https://staticman.net/docs/configuration#generatedFields


dotnet_style_prefer* or dotnet_prefer in Expression-level configs

In doc there are two different identifiers. Which is the good one?
In the table:
dotnet_prefer_inferred_tuple_names
dotnet_prefer_inferred_anonymous_type_member_names
In the config example
dotnet_style_prefer_inferred_tuple_names = true:suggestion
dotnet_style_prefer_inferred_anonymous_type_member_names = true:suggestion

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

ID: 410f3156-99cc-2a5c-2696-9d354c4f34c7
Version Independent ID: fe3c3dfe-251f-2585-fe18-7fae3bcc76d0
Content
Content Source
Service: unspecified

@pherbel good catch, thanks. @kuhlenh can you clarify which is correct? I think that dotnet_prefer is correct based on this example file:
Expression-level preferences
dotnet_style_object_initializer = true:silent
dotnet_style_collection_initializer = true:silent
dotnet_style_explicit_tuple_names = true:suggestion
dotnet_style_null_propagation = true:suggestion
dotnet_style_coalesce_expression = true:suggestion
dotnet_style_prefer_is_null_check_over_reference_equality_method = true:none
dotnet_style_require_accessibility_modifiers = always:suggestion
dotnet_prefer_inferred_tuple_names = true:suggestion
dotnet_prefer_inferred_anonymous_type_member_names = true:suggestion

According to https://github.com/madskristensen/EditorConfigLanguage/issues/40#issuecomment-372037059 it should be dotnet_style_*

https://github.com/dotnet/roslyn/blob/master/src/Workspaces/Core/Portable/CodeStyle/CodeStyleOptions.cs#L153-L167 also confirms that it's dotnet_style_*

The following two wrong names still show up incorrectly in the list at the top.
dotnet_prefer_inferred_tuple_names
dotnet_prefer_inferred_anonymous_type_member_names
They are specified differently in the correct dotnet_style_prefer namespace later in the article.

@gewarren @fenxu Could you please reopen this issue? As written by @grantborthwick there are still wrong names left in the article.

@cremor @grantborthwick Thank you both; I'll get it updated.

@gewarren I don't want to be annoying, but there are still wrong names left. To be more specific, it's in the Example EditorConfig file section

@cremor Thanks for catching that! It may have snuck in after I did a find/replace. This should be fixed later today.

@gewarren Looks like the update didn't make it: dotnet_prefer_inferred_tuple_names - without style ;) - is still listed in the example EditorConfig.

@fowl2 Thanks for letting me know. Feel free to fix it, or I can.

@fowl2 Thanks for letting me know. Feel free to fix it, or I can.

@gewarren  Touche, PR #1674 open ;)

Looks like these are all fixed now.


[ENG-3640] Fix issue where file tag modification was forbidden on registrations

Purpose
With the new Files Page redesign we decided to change the old static file tags behavior to allow Registration files to have updatable tags.
Changes

removed Registration Read-only permissions class for file detail
Re-did logic for permissions so tags are the only update
changed check-out behavior, so it's no longer required, just to simplify

QA Notes
Please make verification statements inspired by your code and what your code touches.

Verify
Verify

What are the areas of risk?
Any concerns/considerations/questions that development raised?
Documentation

Side Effects

Ticket
https://openscience.atlassian.net/browse/ENG-3640

Merged

@Johnetordoff Where was this merged to?

I thought this got merged with the Quickfiles migration, but forgot to un-close it, sorry!

Looks like there's a merge conflict.


CDAT Migration Phase 2: Refactor arm_diags set

Description

Closes #667

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

If applicable:

[ ] New and existing unit tests pass with my changes (locally and CI/CD build)
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have noted that this is a breaking change for a major release (fix or feature that would cause existing functionality to not work as expected)

This branch needs to be rebased on the latest cdat-migration-fy24 branch.

This branch needs to be rebased on the latest cdat-migration-fy24 branch.

I was not aware this branch had a force-push. I plan to start from fresh with a new PR.

This branch needs to be rebased on the latest cdat-migration-fy24 branch.

I was not aware this branch had a force-push. I plan to start from fresh with a new PR.

I re-based cdat-migration-fy24 on top of main with the latest integration test fix from #835. It is a bit of a hassle to do this because it requires the dev branches stemming from cdat-migration-fy24 to also be re-based. We only have a few sets to deal with this if it happens though.

Closed in favor of #842


Frontmatter images conflicts with Hugo definition

Describe the bug
Markdown pages fail to build with images: set in the front matter.
To Reproduce
Steps to reproduce the behavior:

Add front matter to markdown with

images:
  - src: /test.jpg

Fails to build

Expected behavior
Expect to build
Screenshots
N/A
Desktop (please complete the following information):

OS: [e.g. iOS] Any
Browser [e.g. chrome, safari] Any
Version [e.g. 22] Any

Additional context
https://github.com/gurusabarish/hugo-profile/blob/4687d533368288e9f71d358dca30099f29535531/layouts/_default/gallery.html#L22
This uses a .src on the images array, whereas the Hugo definition for front matter images only includes images as []string which makes it conflicting. If Hugo does not error, the theme errors and vice versa.
Fix would be to change it to {{ . }}.

On second look, this was modified in behavior with https://github.com/gurusabarish/hugo-profile/commit/0c564693b306ae4fa629ef68f72a2cefa840f0fa

But this is not released as a tag. Will keep this issue open.

@adyanth new tag released.


Update jsoniter-scala-core to 2.23.4

About this PR
📦 Updates com.github.plokhotnyuk.jsoniter-scala:jsoniter-scala-core from 2.17.9 to 2.23.4
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.plokhotnyuk.jsoniter-scala", artifactId = "jsoniter-scala-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.github.plokhotnyuk.jsoniter-scala", artifactId = "jsoniter-scala-core" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #252.


Remove the eager check in the browse middleware

Hello,
it is just the removing of the eager check in the browse middleware as stated in https://github.com/mholt/caddy/issues/1085
Caddy will start and throw a 404-error until the directory will be created.
Hopes that helps you ;)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Add a test?

Hey,
i'm not really sure how to write this test. I tried
{"root " + nonExistantDirPath + "\n browse", []string{nonExistantDirPath, "/"}, false},
in the https://github.com/mholt/caddy/blob/master/caddyhttp/browse/setup_test.go file.
But the test never fails. I'm looking into it. :) But I'm a Golang newbie. So it will need some time.
Sorry for the inconvenience

Maybe you could use httpserver.GetConfig to get the config and then manipulate the site root by config.Root. If it is appointed to a non exist directory, you will get the expected failure.
For the reason your method doesn't trigger the failure is because test config's site root is the default one which is .

Excellent. Thank you!


feature request: auto-trim chatcontext to respect model context limit.

great app btw. really nice simple chat app for the terminal.
one problem I have is even with 32k context, when I have the bot set to generate multi paragraph responses, then I'm hitting the context really quickly within 10 minutes of chat.
what I'd like is  somehow to cut out the middle of that chat history, so only the earliest and then most recent messages are added to the context history for inference. then I can chat with the same character without forgetting or losing track of its place.
Thanks!~

Hey there,
Thanks for the praise :)
For the time being oterm does not actually pass the previous messages to the chat completion. What we do instead is store the "context" Ollama returns, which is essentially an embedding of the entire conversation.
I am planning to change that now that the official client supports passing a list of previous messages (it did not when I started oterm). When that happens you will be able to select the number of messages you want to keep passing back and forth.


[EPIC]: Testground's Infrastructure as Code

Background
ATM, testground infrastructure is written in bash scripts. This solves the initial task of using eks to execute test runs.
Nevertheless, this is not enough to reliably use testground for:

Infra module upgrades
Metrics collection / aggregation
Logs collection / aggregation
Metrics / Logs storage
CI/CD for big network tests

Hence, we need to redesign the infrastructure to a layers, which will let us be more flexible on setups
Layer 1 - Basic Infrastructure in EKS(AWS)
Goal: Setup basics for the layers above

Control Plane + node groups + security groups + OIDC
EBS, EFS, S3, Autoscaling
ArgoCD

TerraForm is how we are going to implement all 3 points
Every item in points above are treated as TerraForm modules

[ ] https://github.com/celestiaorg/test-infra/issues/148
[ ] https://github.com/celestiaorg/test-infra/issues/149

Layer 2 - Testground Applications(Mandatory)
Goal: make it easy aka helm chart

Master helm chart for testground, which contains:

helm charts for each of components that we have rn in functions.sh
ArgoCD will take care of deployments

Required Components (caveats → we need to finalise which ones are DS/STS/RS):

Redis
Multus
Weave
Flannel
Testground-Sidecar
Testground-Daemon
Sync-Service

[ ] https://github.com/celestiaorg/test-infra/issues/150
[ ] #151

Layer 3 - Add-ons(dashboards, metrics, observability logs)
Goal: full visibility for the dev teams to analyse test runs
Components -> We need to finalise which ones below are DS/STS/RS:

Prometheus
InfluxDB
Promtail
Grafana
Metrics Scraper (we need do develop it in-house)
Log Metrics Analysers

cc: @sysrex @jrmanes

Why do we need to develop the Metrics Scraper in-house? Can't we use Prometheus for scraping?

Cosmos support: https://github.com/celestiaorg/test-infra/issues/179

Grooming 12/05/2023:
Putting it to the board. It will be closed by https://github.com/celestiaorg/devops/issues/123

Closing as done https://github.com/celestiaorg/devops/issues/123


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 168 ms

Resolved: ClamAV performance has improved in 344aeb1.


thank you so much for creating this software

thank you so much for creating this software it solved all the issues with my games

Thanks for the feedback!


Dns name resolve from vpn

I use cli installed from brew and use openvpn for my work environment, but I cannot connect to cluster:
nats: error: setup failed: dial tcp: lookup nats.dev.svc.cluster.local on <IP_ADDRESS>:53: no such host
all other tools work great through vpn

I don’t think I do anything special dns wise. Just rely on the go standard library.
But I suspect what’s happening is NATS is learning names of your cluster via its topology discovery and you do not configure advertise names that are correctly resolvable.

this name is resolving by ping command:

~ ❯ ping nats.dev.svc.cluster.local
PING nats.dev.svc.cluster.local (<IP_ADDRESS>): 56 data bytes
64 bytes from <IP_ADDRESS>: icmp_seq=0 ttl=61 time=15.877 ms
64 bytes from <IP_ADDRESS>: icmp_seq=1 ttl=61 time=20.659 ms
^C

moreover this dns successfully resolve when I use DataGrip for example for going to db in this cluster

Hard to say. For that I just call out to nats.go.
@kozlovic or maybe @wallyqs might have seen something like this before?

see this https://github.com/golang/go/issues/12524 and in my config:

no I have not, I gave you a screen of my config, it is alone

Sure, you wont see from that one file that you also have other files in /etc/resolver/*, you linked the issue which states a problem that specifically relates to those files, so seems like a reasonable question to ask.
Not sure, I guess you can do a test build locally without disabling CGO and seeing if it resolves the issue for you.

hi @z0mb1ek, checking just in case where you using auth credentials and/or a load balancer when you ran into this?

I'm also hitting this issue trying to connect to out  nats cluster deployed in k8s over VPN (Cloudflare WARP).
We do have accounts and mTLS configured. If I use pure nats.go it works fine but through the CLI it fails:
package main

import (
	"log"

	"github.com/nats-io/nats.go"
)

const NATS_URL = "nats://cluster.namespace.svc.cluster.local:4222"

func main() {
	opts := []nats.Option{
		nats.RootCAs("ca.crt"),
		nats.ClientCert("tls.crt", "tls.key"),
	}

	conn, err := nats.Connect(NATS_URL, opts...)
	if err != nil {
		log.Fatal(err)
	}

	js, err := conn.JetStream()
	if err != nil {
		log.Fatal(err)
	}

	for stream := range js.StreamNames() {
		log.Println(stream)
	}
}

// Outputs
2022/06/13 13:00:06 MY_STREAM_NAME

However this does not work with natscli (IP obfuscated):
nats --tlscert=./tls.crt --tlskey=./tls.key --tlsca=./ca.crt -s nats://cluster.namespace.svc.cluster.local:4222 stream ls
nats: error: setup failed: dial tcp: lookup cluster.namespace.svc.cluster.local on X.X.X.X:53: no such host

In my /etc/resolver/cluster.local (IP address obfuscated):
nameserver X.X.X.X

I think in the case of /etc/resolver/cluster.local files you could probably compile the CLI yourself with CGO enabled and that would resolve

ahhh ok @ripienaar I installed via brew so will give a go at compiling locally 👍

Will close this, let us know if you still have issues.


[Activation] Shutdown sequence and bugfixes

shutdown sequence fix

Is this ready for review?

Almost.
Will need around 30 minutes more. Still didn't test it properly and didn't check everything we've discussed on a call


Add corect timestamp to logging in non production environments

Closes https://github.com/denali-js/denali/issues/370

I can haz test plz? 😸


Fix KDEPlot hue tests

See https://github.com/ResidentMario/geoplot/pull/272#issuecomment-1053926367 for context.

Any chance you can release a 0.5.1 with these fixed?

Do you have something downstream that's relying on these tests passing?

For packaging, I always run tests to ensure things are working. Otherwise, the update doesn't happen. I guess no-one is asking for it, so there's no pressing need just yet.

Thanks, that is working everywhere now.


Not able to provision a service

Hi there,
After minibroker being deployed, I was trying to create a mariadb service. Two pods were created and pending:
NAME                                                               READY   STATUS    RESTARTS   AGE
mariadb-4144419a028d4d1620c1-master-0   0/1         Pending   0                68m
mariadb-4144419a028d4d1620c1-slave-0      0/1         Pending   0                68m

And the pod description show error:
Events:
  Type     Reason            Age                 From               Message
  ----     ------            ----                ----               -------
  Warning  FailedScheduling  26s (x46 over 65m)  default-scheduler  0/3 nodes are available: 3 pod has unbound immediate PersistentVolumeClaims.

I check the PV, there was no PV created.
Any idea how to solve this problem?
Thank you!

Hi @kaleo211 - maybe an issue with your storage class?

@kaleo211 - could you confirm your storage class?

@kaleo211 - could you confirm your storage class?


Add ability to keep gamerules setting on world regen.

Fixes #2516. By default regening a world will keep the gamerule. If they don't want this behaviour, they can run with this new flag: /mv regen <world> --reset-gamerules.
I also did a bit of code cleanup in the regen method while at it.

Test build:
Multiverse-Core-4.2.3-SNAPSHOT.zip

Should be good!

Rebased to make it work with new queue command system #2585