feat(workflow): Create build frontend workflow

Fixes #5
The workflow file was pretty standard - please let me know if you want to customize any of the options.

@trybick
This is great, I will go ahead and merge this.

@trybick
There was a small issue with the workflow which I went ahead and fixed.
      - run: cd frontend
      - run: npm install
      - run: npm run build --if-present

Each step starts in the workspace directory or root directory so this will first cd into frontend and then for the second command i.e. npm install will run in the root directory and not frontend.
Here is the fix for this
      - run: |
              cd frontend
              npm install
              npm run build --if-present

This Stackoverflow Question explains it very well
https://stackoverflow.com/questions/59529042/difference-between-run-and-multiple-runs-in-github-actions
Also, Docusaurus requires version 10 and above
https://v2.docusaurus.io/docs/installation#requirements
So, I removed that also.
Thanks

@lelouchB Ah, thanks for fixing and explaining it.


Add grep-jump style commands for symbols and references

Adds the following commands piggybacking grep-jump from grep.kak:

lsp-references-next-match: Jump to the next references match
lsp-references-previous-match: Jump to the previous references match
lsp-symbols-next-match: Jump to the next symbols match
lsp-symbols-previous-match: Jump to the previous symbols match

Thank you for your contribution!
Nice idea and implementation =)


[Epic] Canvas Undo/Redo Phase I

Objective
Provide a feature to undo / redo (U/R) actions in the editor (on the canvas) with up to 200 updates.
In the process of building fast, many unforced errors occur which need to be undone. Enabling faster iteration leads to more usage and a better building experience
Related issues
[ ] #3917
[ ] #3916
Success Metrics
List of all metrics you are tracking and the desired goal.

Goal
Metric

Encourage the use of undo/redo
Number of times that people used undo or redo

Let developers to explore and be creative
Number of people that have used undo or redo more than 20 times at once

Assumptions
List any assumptions that you have about your users, technical constraints, or business goals (e.g., Most users will access this feature from tablet).

Developers want to have undo and redo for the property pane as well

Requirements

Requirement
User Story
Importance
Notes

Undo/Redo for the components
As a User, I want to be able to U/R the actions that I make on the canvas
High
For dropping a widget, pasting something, removing a widget (cutting as well), changing size etc.

U/R in small steps in the property pane
As a user I want to be able to undo what I did in property pane and not lose all my work even after I close the property pane
High

Onboarding for the feature
As an existing user that has already tried all the shortcuts for U/R I need to be informed about this new feature
High

Maximum of 200 updates
As a user I want to be able to work care-free when it comes to undo/redo of my work
High

Undo and redo action in a menu behind the logo in the navigation bar
As a user I want to be able to find these actions in the app and not have to search how to do it online
Medium

Version History (Non-linear undo)
As a user I want to skip to a previous version from version history
Medium

Out of Scope
List the things that are out of cope or might be revisited after the first release.

Undo/redo for anything that is not on the canvas

Developer Handoff Document in Figma
Link to the developer Handoff Document:
https://www.figma.com/file/SHl5JRGZ7AtuwIiiZtCe5U/Undo-Redo-Feature-Handoff-Mon-7-Jun-2021?node-id=0%3A1
Questions

Question
Answer
Date Answered

Do we enable the users to undo changes in the property pane with tiny steps?
Yes (Slack discussion https://theappsmith.slack.com/archives/C01J5915Z89/p1622640625119600)
June 2nd 2021

Undo Redo Product Note:
Scope Phase I:

[ ] Undo - Redo implementation works in the context of the DSL, which is the Canvas.

Scenarios:

[ ] Undo and Redo will technically walk through DSL updates. So if anything in the DSL updates, it can be undone and redone.
[ ] Undo drop widget should remove widget
[ ] Undo after widget property change should undo the property change
[ ] User updates a widget property, user closes the property pane - undo here undos the property change
[ ] Undo works for a minimum of 10 dsl updates.
[ ] Undo after copy and paste, undos the paste
[ ] Undo after cut undos the cut
[ ] Undo after paste undos the paste

Tasks:

[ ] Platform change to include the DSL in a data structure which allows for tracking and merging changes
[ ] Undo and Redo operation functions - implementation
[ ] Implement and test undo redo - as there can be collateral widget property update operations. These are not always atomic.
We track undo and redo based on actual dsl update actions. Needs triaging
[ ] Undo redo buttons UI - Needs design.
[ ] Unit tests for undo redo methods
[ ] Cypress test for all scenarios above.

@riodeuno will it work for

Size / position changes
Meta property updates

@Nikhil-Nandagopal According to the plans of phase I, this will work for size and position changes, however, it will not effect meta properties.
We can look into meta properties in the second phase of the project.
Undo and redo already works for meta properties in Input widget, Rich Text Editor widget, however, it does not for table widget.

Design overview: https://miro.com/app/board/o9J_l3YFsis=/


delete most branches

i think there are too many branches?
what are the ones apart from master that we should keep?
maybe some old ones could be exported as releases, for documentation purposes?

This is a duplicate of #18
Closing and will address on other issue.


Create delmation.yml

Summary

I would like to add the integration with our IoT portal.
Changes

This is a new integration with our IoT dashboard.
Notes for Reviewers

With this integration, our customers can more easily send the data to our IoT portal.
Checklist

[ x ] Scope: The referenced issue is addressed, there are no unrelated changes.
[ x ] Compatibility: The changes are backwards compatible, they don't break existing deployments.
[ x ] Testing: The changes are tested.
[ x ] Documentation: Relevant documentation is added or updated.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@delmation is this template tested? Can you create some documentation for our docs page? You can file a PR here: https://github.com/TheThingsIndustries/lorawan-stack-docs, and let me know if you need help.

Hi Nejra,
I have tested this template on our 'the things stack enterprise' server installation.
I will also create the documentation about this integration so that it can be posted on your site.
Best regards,
Edward de Graaff
Van: Nejra Selimović @.>
Datum: zondag, 19 februari 2023 om 19:25
Aan: TheThingsNetwork/lorawan-webhook-templates @.>
CC: Edward de Graaff | Delmation Products BV @.>, Mention @.>
Onderwerp: Re: [TheThingsNetwork/lorawan-webhook-templates] Create delmation.yml (PR #77)
@delmationhttps://github.com/delmation is this template tested? Can you create some documentation for our docs page? You can file a PR here: https://github.com/TheThingsIndustries/lorawan-stack-docs, and let me know if you need help.
—
Reply to this email directly, view it on GitHubhttps://github.com/TheThingsNetwork/lorawan-webhook-templates/pull/77#issuecomment-1436059025, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATIBZCQKOI3D5NF27KNC363WYJQRPANCNFSM6AAAAAAVA7RQOE.
You are receiving this because you were mentioned.Message ID: @.***>

Hi Nejra,
I have created the documentation (see attachement)
I just don't understand how I can now add it to github via a PR.
Could you give the steps how I can add/subscribe it now.
Best regards,
Edward de Graaff
Van: Nejra Selimović @.>
Datum: zondag, 19 februari 2023 om 19:25
Aan: TheThingsNetwork/lorawan-webhook-templates @.>
CC: Edward de Graaff | Delmation Products BV @.>, Mention @.>
Onderwerp: Re: [TheThingsNetwork/lorawan-webhook-templates] Create delmation.yml (PR #77)
@delmationhttps://github.com/delmation is this template tested? Can you create some documentation for our docs page? You can file a PR here: https://github.com/TheThingsIndustries/lorawan-stack-docs, and let me know if you need help.
—
Reply to this email directly, view it on GitHubhttps://github.com/TheThingsNetwork/lorawan-webhook-templates/pull/77#issuecomment-1436059025, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATIBZCQKOI3D5NF27KNC363WYJQRPANCNFSM6AAAAAAVA7RQOE.
You are receiving this because you were mentioned.Message ID: @.***>

@delmation not sure why, but I can't find the attachment. Can you please send me directly via email<EMAIL_ADDRESS>and I will create a PR :)

Hi Nejra,
I emailed the files a while ago.
I hope they arrived well.
I would like to know when these will be implemented.
Best regards,
Edward de Graaff
Van: Nejra Selimović @.>
Datum: zondag, 5 maart 2023 om 12:59
Aan: TheThingsNetwork/lorawan-webhook-templates @.>
CC: Edward de Graaff | Delmation Products BV @.>, Mention @.>
Onderwerp: Re: [TheThingsNetwork/lorawan-webhook-templates] Create delmation.yml (PR #77)
@delmationhttps://github.com/delmation not sure why, but I can't find the attachment. Can you please send me directly via email<EMAIL_ADDRESS>and I will create a PR :)
—
Reply to this email directly, view it on GitHubhttps://github.com/TheThingsNetwork/lorawan-webhook-templates/pull/77#issuecomment-1455070324, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ATIBZCSIE3N64VL4HTXNFHDW2R5YXANCNFSM6AAAAAAVA7RQOE.
You are receiving this because you were mentioned.Message ID: @.***>


Posibility to run post-deployment scripts of dependencies

Can it be possible to run the pre- and post deployment scripts of your dependencies too?

This is already possible in a somewhat roundabout way; I do this today in some of my projects.
The pre- and post-deployment scripts allow includes of other files, which can be used to reference the scripts of your dependencies. I would typically couple this with a sqlcmd variable for the dependency path, so it doesn't need to be hardcoded, which could be brittle.

This is already possible in a somewhat roundabout way; I do this today in some of my projects.
The pre- and post-deployment scripts allow includes of other files, which can be used to reference the scripts of your dependencies. I would typically couple this with a sqlcmd variable for the dependency path, so it doesn't need to be hardcoded, which could be brittle.

@jeffrosenberg Interesting. But doesn't that mean that you'd need to have the source code of the referenced project on your local machine as well? That feels kind of heavy since you already have the .dacpac of the referenced project there.
I was actually thinking of making this a feature of MSBuild.Sdk.SqlProj. I totally understand why SqlPackage.exe doesn't do this, because you'd want to deploy each .dacpac separately through its own pipeline. But at development time I can see value in us running the predeploy.sql and postdeploy.sql of referenced projects as well. Ideally this would of course be configurable. I was thinking of throwing something together this weekend ;).

@jeffrosenberg Interesting. But doesn't that mean that you'd need to have the source code of the referenced project on your local machine as well? That feels kind of heavy since you already have the .dacpac of the referenced project there.
I was actually thinking of making this a feature of MSBuild.Sdk.SqlProj. I totally understand why SqlPackage.exe doesn't do this, because you'd want to deploy each .dacpac separately through its own pipeline. But at development time I can see value in us running the predeploy.sql and postdeploy.sql of referenced projects as well. Ideally this would of course be configurable. I was thinking of throwing something together this weekend ;).

@jmezach my use case was for working with SqlPackage, so yeah, that's somewhat different than what you're proposing. I have to admit, I'm not totally sure what this would look like, but interested to see what you come up with! 🙂

@jmezach my use case was for working with SqlPackage, so yeah, that's somewhat different than what you're proposing. I have to admit, I'm not totally sure what this would look like, but interested to see what you come up with! 🙂

I've started some initial work on this in the feature/pre-post-dependencies branch. It is already working fairly well for our test projects, but it needs a bit more work. Most notably we'll need to supply some default SQLCMD variables (for example: DatabaseName) as users will probably expect those to be set.
We might also need to figure out what SQLCMD variables have been defined by the referenced package (which requires them to be present in that package in the first place, which we currently do not do I think) and figure out a way to define their values as well. Not sure if users just expect that to work out of the box though.

I've started some initial work on this in the feature/pre-post-dependencies branch. It is already working fairly well for our test projects, but it needs a bit more work. Most notably we'll need to supply some default SQLCMD variables (for example: DatabaseName) as users will probably expect those to be set.
We might also need to figure out what SQLCMD variables have been defined by the referenced package (which requires them to be present in that package in the first place, which we currently do not do I think) and figure out a way to define their values as well. Not sure if users just expect that to work out of the box though.

The SQLcmd variables issue could be as reason why this is not available with sqlproj?

The SQLcmd variables issue could be as reason why this is not available with sqlproj?

@verhoofstad A pre-release version of MSBuild.Sdk.SqlProj is up on NuGet.org. Could you give that a spin to see if it resolves our issue? You will need to update the SDK version number in the .csproj and also add the new RunScriptsFromReferences property to it.

@verhoofstad A pre-release version of MSBuild.Sdk.SqlProj is up on NuGet.org. Could you give that a spin to see if it resolves our issue? You will need to update the SDK version number in the .csproj and also add the new RunScriptsFromReferences property to it.

@verhoofstad A stable release is now available on NuGet.org.

@verhoofstad A stable release is now available on NuGet.org.


I guess there is something wrong with _get_dataset_inds(self) in mix_loader.py, line 146

When I set MIX_RATIOS = [0.4, 0.4, 0.2] in .yaml, it stop at assert(len(self._dataset_inds) == self._num_gpus*cfg.TRAIN.IMS_PER_BATCH).  And self._dataset_inds = [].

Then I set MIX_RATIOS = [2.0, 2.0, 1.0] in .yaml and comment the assert, since self._dataset_inds = [0, 0, 1, 1, 2]. It seems to work as expect.

So do you have any idea? Or would you please explain the assert?

Hi, how many GPUs do you use? In our experiment, we set the batch_size to 8 (4 GPUs) and the mix_ratio to [0.125, 0.125, 0.5, 0.125, 0.125] to sample samples from each dataset.
your solution can also work.

Hi, how many GPUs do you use? In our experiment, we set the batch_size to 8 (4 GPUs) and the mix_ratio to [0.125, 0.125, 0.5, 0.125, 0.125] to sample samples from each dataset.
your solution can also work.
Two. Now I know why I got the previous mistake. Thanks!


Add Error comment for accessing method of subclass

Currently an error occurs at the line:
department.generateReports();

This happens because department is not of type AccountingDepartment hence can't access the generateReports method. This is however harder to understand because:

The type is declared in the start a few lines above to be Department and later object is assigned from constructor of AccountingDepartment.
There is no comment to point out that the error will occur and neither is it described in the text.

This confused me for a while to understand that the type didn't change and to get the method to work, department should be initialized with the correct type: AccountingDepartment. Even though the error message displayed in the code is very similar, adding the comment makes it known to the reader that this error is expected as this is not discussed in the text.
This PR adds the comment necessary to note that the error is expected and explains why the error occurred.
Hopefully this will cause less confusion for someone else.

Looks good, it's consistent with the rest of the error messagings 👍🏻

Looks good, it's consistent with the rest of the error messagings 👍🏻


npm run build:go:linux crashes

v4.1.0-beta1 + v4.1.0-beta2
npm run build:go:linux does not work, it crashes
We are using an Intel processor.
Crash report:

<EMAIL_ADDRESS>build:go:linux /home/comvay/test.meetupps.com/public_html
make pack-linux

fatal: Not a git repository (or any of the parent directories): .git
GOOS=linux GOARCH=amd64 packr build -ldflags "-X main.gitDescribe=" -o peer-calls
/bin/sh: 1: packr: not found
Makefile:18: recipe for target 'pack-linux' failed
make: *** [pack-linux] Error 127
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>build:go:linux: make pack-linux
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:go:linux script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/comvay/.npm/_logs/2021-04-06T10_12_17_341Z-debug.log

Hey, sorry about that. You'll need to install packr before you can use that script:
go get -u github.com/gobuffalo/packr/packr

You can also download prebuilt binaries here

Version 4.1.0-beta4 no longer depends on packr and uses the embed package from Go stdlib to embed files in to a binary.
I'm going to close this issue now - feel free to reopen if you're still having trouble with the build!


Life expectancy for the D400 series

Hi guys,
It would be much appreciated if anyone can provide the life expectancy of the D400 series. I can not find this in the official spec.
I'm worrying about the life expectancy of the laser related parts in the camera and want to figure out whether D400 series can support a 24x7 application or not.
Thanks!

A publicly released roadmap for the RealSense product range in 2017 predicted a three year life cycle for the 400 Series, from 2017 to end of 2019.  The next generation of RealSense camera, tentatively codenamed D600, was then projected to be introduced in 2020.
A lot of time has passed since 2017, so plans have very likely changed, but I have no reason to think that there will not be another generation of RealSense.  Also, 'end of life' dates have less meaning these days, as cameras receive technical support for years after their projected retirement date and the open-source nature of the RealSense SDK 2.0 means that its community can provide new updates for it long after Intel are no longer producing official releases.
In regard to your question about 24-7: RealSense cameras can run indefinitely as long as they remain within recommended temperature tolerances.  If there are problems with a long-run of an application, they are usually caused by the computer's OS or USB ports.  A good estimation of how long a RealSense application can run before the host computer causes a problem is three days.  Using industrial-grade USB equipment can help to provide stability.

Hi @suiye921
Laser performance may, in fact, degrade over time. This will not render the device completely useless (it will just rely more on passive stereo matching), but this will obviously reduce depth quality. Unfortunately, I don't have the official expected lifetime data.

Thank you for your reply @MartyG-RealSense and sorry for the misleading question I've made.
Yes I'm concerned about the potential lifespan of the components because I'm evaluating the application of an RGB-D camera in a 24-7 production line to monitor the movement of some metal parts, which may be different from the standard scenario such as a video game input device. And as I'm a newcomer in this field, it would be much appreciated if you can provide some advices in this application scenario.

No worries about your question.  I just wanted to make sure we were on the same page.   :)
As I said above, industrial grade USB equipment should help prolong stable running time.  My personal recommendation for such equipment is Newnex, though there is a price premium for that quality of equipment.
https://www.newnex.com/realsense-3d-camera-connectivity.php
You can also use industrial grade PCs for added reliability.
https://en.wikipedia.org/wiki/Industrial_PC
If you are currently in a test phase and do not want to commit to purchasing industrial-grade equipment yet, a workaround for use with ordinary equipment might be to write an application that incorporates a timer in it.  It could count in seconds from zero, and when 86400 (the number of seconds in 24 hours) is reached, a hardware reset instruction could be issued by the script to reset the camera.  This should hopefully preempt any USB problems that might set in during a multi-day run and make 7-day running practical, assuming that a problem does not develop in the computer's OS that requires a reboot.
If the operating must be running 24-7 without interruption, having more than one camera, with each camera attached to its own PC, would provide some redundancy in the event of a computer having an issue that requires a reboot.

It depends a bit on your use-case. For instance, outdoor in daylight the projector is not adding much if at all. Indoor it helps a little, and if you need to work in pitch-black darkness it is required.


JIT resample waveform

I have no problem with torch.device as type when test resample_waveform function. However, when I use resample_waveform in some torch.nn.Module, I got the following error:
RuntimeError:
Unknown type name 'torch.device'

@vincentqb

... I infer that you would like to make sure that Resample can be compiled. If this is so, then you should instead add the relevant test for the Resample transform, see comment below.

Yes, that's right. I will modify code in few days according to your comments and add tests. But I suppose that I will have the same problem that I mentioned above about module (in the same time, transform as function should work).


Auto Column Width

Is there a way to set column width based on cell's content length ?
thank you

+1
Would be very helpful!

+1

+1

+1

+1

+1

+1

#36

there is an array with values if I add the row all well, all values are exported to excel, but I can not build the column ((( who can do??

I've had some luck with the still rough but maybe more accurate approach below in TypeScript:
/** Determined this value based upon experimentation */
export const PIXELS_PER_EXCEL_WIDTH_UNIT = 7.5

export function autoSize(sheet: excel.Worksheet, fromRow: number) {
	const canvas = document.createElement('canvas')
	const ctx = canvas.getContext('2d')
	if (!ctx) {
		return
	}

	const maxColumnLengths: Array<number> = []
	sheet.eachRow((row, rowNum) => {
		if (rowNum < fromRow) {
			return
		}

		row.eachCell((cell, num) => {
			if (typeof cell.value === 'string') {
				if (maxColumnLengths[num] === undefined) {
					maxColumnLengths[num] = 0
				}

				const fontSize = cell.font && cell.font.size ? cell.font.size : 11
				ctx.font = `${fontSize}pt Arial`
				const metrics = ctx.measureText(cell.value)
				const cellWidth = metrics.width

				maxColumnLengths[num] = Math.max(maxColumnLengths[num], cellWidth)
			}
		})
	})

	for (let i = 1; i <= sheet.columnCount; i++) {
		const col = sheet.getColumn(i)
		const width = maxColumnLengths[i]
		if (width) {
			col.width = width / PIXELS_PER_EXCEL_WIDTH_UNIT + 1
		}
	}
}

This worked well with me
// Auto width column
for (let i = 0; i < sheet1.columns.length; i += 1) {
let dataMax = 0;
const column = sheet1.columns[i];
for (let j = 1; j < column.values.length; j += 1) {
const columnLength = column.values[j].length;
if (columnLength > dataMax) {
dataMax = columnLength;
}
}
column.width = dataMax < 10 ? 10 : dataMax;
}

This worked well for me :
// Auto width column
// Auto width column       for (let i = 0; i < sheet1.columns.length; i += 1) {         let dataMax = 0;         const column = sheet1.columns[i];         for (let j = 1; j < column.values.length; j += 1) {           const columnLength = column.values[j].length;           if (columnLength > dataMax) {             dataMax = columnLength;           }         }         column.width = dataMax < 10 ? 10 : dataMax;       }

This worked well for me :
`// Auto width column
const workbook = new Excel.Workbook();
const sheet1 = workbook.addWorksheet('sheet1');
  for (let i = 0; i < sheet1.columns.length; i += 1) {
    let dataMax = 0;
    const column = sheet1.columns[i];
    for (let j = 1; j < column.values.length; j += 1) {
      const columnLength = column.values[j].length;
      if (columnLength > dataMax) {
        dataMax = columnLength;
      }
    }
    column.width = dataMax < 10 ? 10 : dataMax;
  }`

This worked well for me :
`// Auto width column
const workbook = new Excel.Workbook();
const sheet1 = workbook.addWorksheet('sheet1');
  for (let i = 0; i < sheet1.columns.length; i += 1) {
    let dataMax = 0;
    const column = sheet1.columns[i];
    for (let j = 1; j < column.values.length; j += 1) {
      const columnLength = column.values[j].length;
      if (columnLength > dataMax) {
        dataMax = columnLength;
      }
    }
    column.width = dataMax < 10 ? 10 : dataMax;
  }`

Awesome this worked for me too, thanks :+1:

+1

+1

This worked well for me :
`// Auto width column
const workbook = new Excel.Workbook();
const sheet1 = workbook.addWorksheet('sheet1');
  for (let i = 0; i < sheet1.columns.length; i += 1) {
    let dataMax = 0;
    const column = sheet1.columns[i];
    for (let j = 1; j < column.values.length; j += 1) {
      const columnLength = column.values[j].length;
      if (columnLength > dataMax) {
        dataMax = columnLength;
      }
    }
    column.width = dataMax < 10 ? 10 : dataMax;
  }`

Very nice explanation, but when I tried this has a performance issue, my worksheet has lots of rows /columns running into thousands. Any help on optimising my response time.

`      for (let i = 0; i < sheet._rows.length; i++) {
const row = sheet._rows[i];
for (let j = 0; j < row._cells.length; j++) {
const cell = row._cells[j];
const value = cell._value.model.value;
let length = (value || '').toString().length;
if (value instanceof Date) {
length = value.toLocaleString().length;
}
      const col = cell._column;
      if ((col.width || 0) < length) {
        col.width = length + 3;
      }
    }
  }`

This way of approching @ameliarahman solution gave me better performance. hope it helps.
this.worksheet.columns.forEach(function(column){ 	var dataMax = 0; 	column.eachCell({ includeEmpty: true }, function(cell){ 		var columnLength = cell.value.length;	 		if (columnLength > dataMax) { 			dataMax = columnLength; 		 }        })        column.width = dataMax < 10 ? 10 : (dataMax + 10); });
Of course this will only work if you only have strings and cells with data in all cells, otherwise you need to add the correspondent validations, like checking if cell.value is not null or if cell.value is a number, and act accordingly.

A bit different solution and in TypeScript:
private _autosizeColumnCells({columns}: Worksheet) {     let dataMax: number[];     let max: number;     columns.forEach((column: Column) => {         dataMax = [];         column.eachCell({includeEmpty: false}, (cell: Cell) => {             dataMax.push(cell.value.toString().length);         });         max = Math.max(...dataMax);         column.width = max < 10 ? 10 : max;     }); }

    worksheet.columns.forEach(function (column) {       var dataMax = 0;       column.eachCell({ includeEmpty: true }, function (cell) {         dataMax = cell.value?cell.value.toString().length:0;       })       column.width = dataMax < 10 ? 10 : dataMax;     });

worksheet.columns.forEach(function (column) {
  var dataMax = 0;
  column.eachCell({ includeEmpty: true }, function (cell) {
    dataMax = cell.value?cell.value.toString().length:0;
  })
  column.width = dataMax < 10 ? 10 : dataMax;
});

worksheet.columns.forEach(function (column) {
  var dataMax = 0;
  column.eachCell({ includeEmpty: true }, function (cell) {
    dataMax = cell.value?cell.value.toString().length:0;
  })
  column.width = dataMax < 10 ? 10 : dataMax;
});

although the cells can be expanded normally, this calculation method seems to be inaccurate.
do you know the default unit of width? cm or wch?

A little bit diff way to make it reusable for all worksheets.
`class Exporter{
widthConfig(data)
{
data.forEach(function(column){
var dataMax = 0;
column.eachCell({ includeEmpty: true }, function(cell){
var columnLength = cell.value.length;
if (columnLength > dataMax) {
dataMax = columnLength;
}
})
column.width = dataMax <= 10 ? 25 : dataMax;
console.log('column.width: ', column.width)
});
}

export(){
this.widthConfig(worksheet.columns)
}
}
`

Feel free to use my solution
/**
 * Autofit columns by width
 *
 * @param worksheet {ExcelJS.Worksheet}
 * @param minimalWidth
 */
const autoWidth = (worksheet, minimalWidth = 10) => {
    worksheet.columns.forEach((column) => {
        let maxColumnLength = 0;
        column.eachCell({ includeEmpty: true }, (cell) => {
            maxColumnLength = Math.max(
                maxColumnLength,
                minimalWidth,
                cell.value ? cell.value.toString().length : 0
            );
        });
        column.width = maxColumnLength + 2;
    });
};

Inspired by @MeisQuietude solution for autoHeight
const autoHeight = (worksheet) => {
    const lineHeight = 12 // heigh per line is roughly 12
    worksheet.eachRow((row) => {
        let maxLine = 1
        row.eachCell((cell) => {
            maxLine = Math.max(cell.value.split('\n').length, maxLine)
        })
        row.height = lineHeight * maxLine
    })
}

I Use this logic for resizing the height & should also work with merged cells.
public static autoCellHeight(workSheet : Worksheet , rowNumberMin ? : number , rowNumberMax ? : number) {     let columnWidths = workSheet.columns.map((column) => column.width);     if(rowNumberMin && rowNumberMax) {       for(let i = (rowNumberMin || 1); i <= (rowNumberMax || workSheet.lastRow.number); i++) {         let multiplier = 1;         let mergedCells = 0;         let mergedCellWidth = 0;         let mergedContentWidth = 0;         workSheet.getRow(i).eachCell({ includeEmpty: false },(cell, colNumber)=> {           if(cell.isMerged) {             // @ts-ignore             if(cell._mergeCount) {               // @ts-ignore               mergedCells = cell._mergeCount + 1;               mergedContentWidth = cell.value?.toString().length || 0;             }             mergedCellWidth = mergedCellWidth +  columnWidths[colNumber - 1];             mergedCells = mergedCells - 1;             if(mergedCells === 0) {               multiplier = Math.max(Math.ceil(mergedContentWidth / mergedCellWidth) , multiplier);               mergedCells= 0;               mergedCellWidth= 0;               mergedContentWidth = 0;             }           } else {             multiplier = Math.max((cell.value ?               Math.ceil((cell.value.toString().length + 1) / columnWidths[colNumber - 1]) : 1), multiplier);           }         });         workSheet.getRow(i).height = (workSheet.getRow(i).height || 15.75) * multiplier;       }     }   }
if anyone can give me what scaler can be used for different font sizes for arial that would be really helpful

Here is the answer I was looking for, hope it would be helpful for someone.
range.format.autofitColumns();
https://learn.microsoft.com/en-us/javascript/api/excel/excel.rangeformat?view=excel-js-preview#excel-excel-rangeformat-autofitcolumns-member(1)
For height you can see https://learn.microsoft.com/en-us/javascript/api/excel/excel.rangeformat?view=excel-js-preview#excel-excel-rangeformat-autofitrows-member(1)

Here is the answer I was looking for, hope it would be helpful for someone. range.format.autofitColumns(); https://learn.microsoft.com/en-us/javascript/api/excel/excel.rangeformat?view=excel-js-preview#excel-excel-rangeformat-autofitcolumns-member(1)
For height you can see https://learn.microsoft.com/en-us/javascript/api/excel/excel.rangeformat?view=excel-js-preview#excel-excel-rangeformat-autofitrows-member(1)

Unfortunately, that is the Office Add-ins JavaScript API, which will only work inside an instance of Excel. Autofitting columns is an internal API to Excel program itself, and people have been trying to figure out how to gracefully implement it within the Office Open XML spec for years ( what everyone besides Microsoft has to grapple with to create XLSX files ). The best solution to do this (still kindof a hack) has been around for a long time, but It only works when the spreadsheet is opened by excel, and only sometimes... you have to set these attributes on the columns of a worksheet : {BestFit : true, CustomWidth : true} There are caveats to this since this is kindof a backwards way of achieving autofit. See:
how to use "bestfit" attribute to autofit column width - MSDN
also
ECMA-376 - Office Open XML - 1st Edition

When there's a numFmt involved, especially for Date values, the solutions given above can produce very inaccurate results. Below is my solution that uses some of the concepts previously suggested, as well as the SSF library to format cell values with numFmt before getting the length.
import { format } from 'ssf';

// Convert Date object to Microsoft serial date
const dateToSerial = (date: Date): number => {
  const timezoneOffset = date.getTimezoneOffset() / (60 * 24);
  const msDate = date.getTime() / 86400000 + (25569 - timezoneOffset);
  return msDate;
};

// My sheets have extra rows before the tables, so headerIndex allows skipping rows before looking at the cells to consider
const autoFitColumn = (column: ExcelJS.Column, headerIndex: number = 0, numFmt?: string) => {
  let maxLength = 6;
  let index = -1;
  column.eachCell({ includeEmpty: true }, (cell: ExcelJS.Cell) => {
    index++;
    if (index < headerIndex) return;
    let columnLength: number;
    if (numFmt && cell.value != undefined) {
      switch (cell.type) {
        case ExcelJS.ValueType.Date:
          const serialDate = dateToSerial(cell.value as Date);
          const formattedDate = format(numFmt, serialDate);
          columnLength = formattedDate.length;
          break;
        case ExcelJS.ValueType.Number:
          const formattedNumber = format(numFmt, cell.value as Number);
          columnLength = formattedNumber.length;
          break;
        default:
          const formatted = format(numFmt, cell.value);
          columnLength = formatted.length;
          break;
      }
    } else {
      columnLength = cell.text.length;
    }
    maxLength = Math.max(maxLength, columnLength);
  });
  column.width = maxLength + 2;
};

+1


feat(component): Add Badge Component

What is it?

[x] Feature / enhancement
[ ] Bug
[ ] Docs / tests

Description
Add new badge component
Screenshots/Demo

Closed #219


Android Wear

If there was even some basic functionality when it comes to Android Wear it would really make it easier to switch.

Duplicate of #4 , planned for 0.3 release.


Added transactions with type 'Credit'

Thanks for adding the transaction type of "Credit" to this table.


Theme error when using Storybook 5.3+

This is a follow up to issue  #220. I am having the same issue and I do not set a custom theme at all.  I had to add the following to make the error go away:
addParameters({ options: { theme: {} } });

In packages/storybook-readme/src/services/getParameters.js#L17, should be something like
...(parameters.options && parameters.options.theme ? parameters.options.theme : {})

This fixes issue https://github.com/tuchk4/storybook-readme/issues/228 as well

Thanks @bbellmyers
FYI if you're using Typescript you need to define base;
.addParameters({ options: { theme: { base: '' } } });

This is happening to me too. What is exactly the solution? I don't see any concrete example.
I am configuring my theme as it is explained on the docs of storybook. Could you please explain to me what I have to do from the normal up to date configuration?

Ok, if anyone is looking for an up-to-date workaround with complete context, add this to your preview.js under your .storybook folder:
import { addParameters } from '@storybook/react';

addParameters({
  options: {
    theme: {}// this is just a workaround for addon-readme
  },
})

If I use the code proposed by @danielo515, I get "Passed an incorrect argument to a color function, please pass a string representation of a color" instead of the previous error.

@jordymeow Please see my comment above


Docker

Run with Docker image for best peformance

get Docker image building working on laptop ubuntu

done see obiwan/dockerfiles and dockerhub: kaylanb/desi:obiwan_rtd_jupyter and the Docs


Add support for passing and setting file URL

Initial checklist

[x] I read the support docs
[x] I read the contributing guide
[x] I agree to follow the code of conduct
[x] I searched issues and couldn’t find anything (or linked relevant results below)
[x] If applicable, I’ve added docs and tests

Description of changes
This adds support for setting file URLs (WHATWG URL objects that
start with file:) as path, which will convert them to an actual
file path.
This also allows URL objects in the options and on their own as
an argument.
Related to vfile/to-vfile#17
/cc @ChristianMurphy @aduh95

Ping @ChristianMurphy


Add Flake8 Python linter

This would replace the https://github.com/houndci/python repo.

🍔

@gylaz When this deploys, could you add a note do houndci/python stating that it's no longer used. Or, can we delete it?


Question about read overlap parameters and usage

Hi,
I'm trying to understand the difference between the read overlap parameters --force-rs, -S and --force-preovec. Do they all do the same thing?
Also, I've noticed there are different usage instructions in the manpage, the README and output of hifiasm_meta -h. I found this a bit confusing, is it possible to clarify which one is correct, or  more complete?
Thanks!

--force-rs and --force-preovec is the same thing, it downsamples reads to meet the kmer count thresholds (--lowq-10, --lowq-5, --lowq-3).
-S does the above read selection only if the dataset seems to be highly redundant. It was introduced for usage in real datasets without having to guess coverages, but so far all real datasets did better without read selection. I guess it's not useful.
I believe the manpage was correct and readme needs some updates. I will push along with the changes from the other thread.

Great, thanks! So by default all of these options should be off?

They are off by default. And for real data, as long as RAM usage and running time is tolerable, it's better to not enable them (I think).

Thanks for your help with this. We've installed hifiasm_meta on Galaxy Australia here. Please let me know if you think anything is missing from the wrapper.
The wrapper is on the Galaxy Tool Shed so it can be installed on any other Galaxy instance.


inconsistency between PVlib and PVsyst when calculating rotated plane azimuth

When I was comparing the stimulative results of PVlib with PVsyst, I noticed that for PV panels with same configurations, the calculated rotated plane azimuth computed by pvlib.tracking.singleaxis(..)
is different from that of PVsyst.
I did find out where the problem is, and it is related to the coordinator transformation.
Can I create a pull request to submit my fix?

@mayudong yes, that would be great.

I also noticed different results for my POA irradiance results when I used the tutorial(tmy_to_power) and compared the results with PVSyst. Can anyone help me figure out where I am going wrong and fix it?

@tafadzwalgurupira, I believe that @mayudong is discussing a potential issue with the single axis tracker code, whereas you are discussing an issue with some part of the plane of array calculations. So, I'd prefer that you make a new issue with more details to discuss the discrepancy between the tmy_to_power tutorial and your pvsyst results. This helps keep discussions focused and on topic. Thanks.

@mayudong  could you post a summary of your fix?  If there's a code error, it's likely also present in the Matlab toolbox and I'd like to fix it there as well.  Thanks.

Sure. I have summarized the issue and potential fix in the attached file.
Please double check and confirm the issue.
Best
NEX-PVSystemModelComparison-110416-1221-2.pdf

I tried to simply this part of the matlab code and it's possible that I introduced an error. I'll look at it more carefully.

Actually the results calculated using Matlab are different from PVsyst too.
I will create a pull request sometime this week

I looked at the surface azimuth results more carefully and I agree that they are not correct in pvlib python for non-trivial cases. Example 3 of the python single axis tutorial and the matlab version's documentation simulates a tracker with 20 deg surface tilt. I remade the matlab figure to include the surface azimuth...

I'll look at the proposed fix in #145  next.

Closed by #145


[Bug] [Module Name] Bug title Graph is not normal，Appear in large numbers none header chunk

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating system information
Linux
Python version information
3.10
DB-GPT version
main
Related scenes

[ ] Chat Data
[ ] Chat Excel
[ ] Chat DB
[X] Chat Knowledge
[ ] Model Management
[ ] Dashboard
[ ] Plugins

Installation Information

[X] Installation From Source

[ ] Docker Installation

[ ] Docker Compose Installation

[ ] Cluster Installation

[ ] AutoDL Image

[ ] Other

Device information
4gpu ,A6000
Models information
llm:qwen2.5 72B
BGE-M3
What happened
生成的图谱不正常，出现大量none header chunk,检索问答时仅有极少数回答是正确的，大量的报错。

What you expected to happen
不知道
How to reproduce
导入同样的文件
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

You can check your prompt log and check are there some graph nodes recalled by graph retrieive.
Now graph retrieve process is

keyword extract from query
Accurately match graph nodes based on keywords and find node information associated with graph nodes.
Assembling context to form prompt word module


Can I ask you to make a relay game server for a game?

Can I ask you to create a transition game server for a game?
Counter-Strike Online 2
https://github.com/KouKouChan/CSO2-Server - Coded with GOLang
or how to do it?
seriously there are too many people on my server and we're having trouble playing together
We don't want to use vpn
error summary on the server:
The client connects to the server and the user creates a room, the player starts the game. but when other players press start the game progress gets stuck at 10%. players cannot view or access the room owner's game. The reason for this is that the opened rooms have localhost <IP_ADDRESS> installed.
and Can the relay system prevent this?

there is a big community waiting for this if u can help us we will appreciate it

+1

+1


example sql_database toolkit agent notebook throws "KeyError" in create_sql_agent

the example notebook at https://github.com/hwchase17/langchain/blob/master/docs/modules/agents/agent_toolkits/sql_database.ipynb is broken at the following line.
agent_executor = create_sql_agent(
    llm=OpenAI(temperature=0),
    toolkit=toolkit,
    verbose=True
)

It produces a KeyError: 'tools'
This error is related to another errorTypeError("cannot pickle 'module' object"). It appears that during the Pydantic validation process for any of the SQLDataBaseTools (e.g., QuerySQLDataBaseTool), there is an attempt to deep copy the db attribute, which is a SQLDatabase object. The method used is rv = reductor(4). However, that is followed up with a call to _reconstruct(x, memo, *rv). That call will throw an error for creating the new SQLDataBase object because the required engine attribute is missing.
A workaround is to define a __deepcopy__() method in SQLDatabase. If the deep copy is not necessary, the method may simply return self or do a shallow copy. But need to make sure what is the right approach here.

same error

@ndaugreal  Which version of Langchain are you using?

It was the latest version as of the reporting. The issue appears to have been fixed now.

@ndaugreal ok awesome, I also got this issue but got resolved by changing the version.


Error when running ZJU-refine dataset

Traceback (most recent call last):
File "train.py", line 327, in main
training(config)
File "train.py", line 68, in training
scene = Scene(config, gaussians, config.exp_dir)
File "/root/autodl-tmp/3dgs-avatar-release/scene/init.py", line 32, in init
self.train_dataset = load_dataset(cfg.dataset, split='train')
File "/root/autodl-tmp/3dgs-avatar-release/dataset/init.py", line 9, in load_dataset
return dataset_dict[cfg.name](cfg, split)
File "/root/autodl-tmp/3dgs-avatar-release/dataset/zjumocap.py", line 58, in init
with open(os.path.join(self.root_dir, self.subject, 'cam_params.json'), 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: '/autodl-tmp/3dgs-avatar-release/data/zjumocap/CoreView_377/cam_params.json'
I downloaded the ZJU-refine dataset from the Instant-nvr project, named my_377, and it shows missing modules when running

Hi, please follow here to preprocess the dataset first.

Hi, please follow here to preprocess the dataset first.

hi,it's seemed that this script is used for processing zju_mocap not zju_mocap_refine

Hi, please follow here to preprocess the dataset first.

hi,it's seemed that this script is used for processing zju_mocap not zju_mocap_refine

They share the same structure so you can easily adapt the script to change the filepath


Update SecureSoftwareGuidingPrinciples.md

changed "tenants" to "tenets"

Doh! This is duplicates 288.


fix(orchestrator): export the OrchestratorPlugin accordingly

Export the OrchestratorPlugin accordingly to be used in the New Backend System
Update instructions on the README file
Bump @rjsf dependencies to fix a warning

Thanks @caponetto!


Fix renamed and modified files not handled.

Hi,
Thanks for this project!
I've just seen that if you modify and rename a file, git will by default consider it as renamed if less than 50% of it changed.
Here, we want to lint every file whose content has changed, so we have to add the -M100% attribute.
You can see the doc about that here.
Thanks!

@paleite pinging just in case you don't watch the repo (it happened to me a lot! 😅)

@mathieutu Wow, this is awesome! Thanks a lot!
I was wondering how one could solve that issue, and I'm really grateful that you've contributed 🙂 Thanks!

Published: https://www.npmjs.com/package/eslint-plugin-diff/v/1.0.13-5
currently still in next, but will be released as stable soon
Again, thanks for your contribution 🙂


Footer

Thanks for the feedback. can you please

create a separate Pull request for this? the flow that I have seen worked well is create a pull request for each change. it's easier to manage over time as we don't have to deal with stacking too many changes together.

for the popup, I was more thinking about this one: https://codesandbox.io/s/yiwhw

Thanks for the feedback. can you please

create a separate Pull request for this? the flow that I have seen worked well is create a pull request for each change. it's easier to manage over time as we don't have to deal with stacking too many changes together.
for the popup, I was more thinking about this one: https://codesandbox.io/s/yiwhw

yeah... that's what I expected this to do. Not sure why the old commits are included... I believe I need to rebase.
Ah! Okay. I can work on that. Would you like that type of popup for the contributors' credit as well?

Would you like that type of popup for the contributors' credit as well?
Yes I do.  We will have more real estate and control to work with. Thanks!

@xunhuang , I cannot for the life of me figure out how to get rid of that huge stack of commits. I've been struggling with this all afternoon. Every time that I make a PR, they're all there. Do you have any tips?

https://stackoverflow.com/questions/14635672/rebase-a-single-git-commit
On Thu, Apr 30, 2020 at 2:04 PM tumbleshack<EMAIL_ADDRESS>wrote:

@xunhuang https://github.com/xunhuang , I cannot for the life of me
figure out how to get rid of that huge stack of commits. I've been
struggling with this all afternoon. Every time that I make a PR, they're
all there. Do you have any tips?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/pull/94#issuecomment-622110942, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF5NZ2XEVZAC35QEWVLRPHRVJANCNFSM4MWPFWMA
.

I just tried it,
git checkout master
git cherry-pick
this worked. Replace "master" with whatever branch you want to be in.
On Thu, Apr 30, 2020 at 2:07 PM Xun Wilson Huang<EMAIL_ADDRESS>wrote:

https://stackoverflow.com/questions/14635672/rebase-a-single-git-commit
On Thu, Apr 30, 2020 at 2:04 PM tumbleshack<EMAIL_ADDRESS>wrote:

@xunhuang https://github.com/xunhuang , I cannot for the life of me
figure out how to get rid of that huge stack of commits. I've been
struggling with this all afternoon. Every time that I make a PR, they're
all there. Do you have any tips?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/pull/94#issuecomment-622110942,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF5NZ2XEVZAC35QEWVLRPHRVJANCNFSM4MWPFWMA
.

I created a new branch, cherry-picked in the commit I wanted, pushed it to my origin, then tried to open a PR merging my new branch into your master, and it brought the stack of 27 commits along with it

probably your new branch was off your old branch. You should pull from
master, that should have non your changes. Create a branch off that
(master), and then cherrypick your change into that new branch, and create
PR. delete your old branch as quickly as possible as it's already merged.
On Thu, Apr 30, 2020 at 2:52 PM tumbleshack<EMAIL_ADDRESS>wrote:

I created a new branch, cherry-picked in the commit I wanted, pushed it to
my origin, then tried to open a PR merging my new branch into your master,
and it brought the stack of 27 commits along with it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/pull/94#issuecomment-622134852, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF6VWQMBKO2F57OVGPLRPHXJDANCNFSM4MWPFWMA
.

I think the problem is in master. I guess git thinks that all those commits are not yet part of 'master'?
I suggest that you do a git show for your last commit, save it.
Delete everything  and do a git clone again to start fresh.
manually redo your change again (from the saved git diff).
On Thu, Apr 30, 2020 at 3:12 PM tumbleshack<EMAIL_ADDRESS>wrote:

I think the problem is in master. I guess git thinks that all those
commits are not yet part of 'master'?
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/pull/94#issuecomment-622143745, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF5KFL4R5AP73EQIOKDRPHZWHANCNFSM4MWPFWMA
.

That did the trick! Thanks for your help! I did:
git push origin master --force ```

happy to help. Glad you are so persistent. Once a while we all get into
something minor, feel free to ping me.
On Thu, Apr 30, 2020 at 5:08 PM tumbleshack<EMAIL_ADDRESS>wrote:

That did the trick! Thanks for your help! I did:
git reset --hard upstream/master
git push origin master --force
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/xunhuang/covid-19/pull/94#issuecomment-622184119, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFSGF23HG57B3WPDOJYQ6TRPIHIFANCNFSM4MWPFWMA
.


DOC: Num causal

Documentation to reflect the non-dependency of the number of causal sites.

I see... So how would you go about normalising like Martin et al? I think we need to list out the different ways you'd like to normalise here, and show the user how to do it.

I guess this pull request can be closed now. If users are interested in scaling, they can scale the simulated traits by using normalise_genetic_value or normalise_phenotype function, and they are well documented in the tstrait documentation. The simulated phenotypes will not explode after normalized, and the statistical properties of the traits are not distorted by using these normalizing functions (this has been thoroughly examined through validation against external simulators).


Adding role specific override per environments

Currently if we wanted to override things per role in specific environment, it
is not possible (say i need to override a setting on ct role in production). So
adding a role specific override per environments.

I'm a little worried about this feature as a way to make things more different between envs than we want them to be (which can often be a cause of failures).
do you mind saying the specific case?

one case may be say in production, we have internet is on gcp1 where we can run ntp instead of bootstrap.
In fact any role based overrides in specific environment should need this.

In fact this patch was created because of a reason where we had to create a user only on one role.


Configuration Enhancements for MK3S+ with Revo-Hotend

Hello everyone! 🚀
I appreciate the great configuration, but while setting up my MK3S+ with the Revo-Hotend, I encountered and addressed a few issues:

Z-Axis Calibration:

Corrected z-axis calibration by adjusting steps from 210 to 215 in steps.cfg.
The z-axis calibration was not accurate due to a discrepancy in the number of steps, resulting in the axis not reaching its endpoints. This was caused by the prusa.cfg file overwriting the correct configuration in steps.cfg. I've adjusted the steps to ensure the z-axis aligns with the endpoints, mirroring the behavior in the Prusa firmware.

Z-Offset Adjustment:

Increased horizontal_move from 2 to 5 to prevent triggering errors during calibration with a higher z_offset then usual.
I use a slightly thinner steel plate, leading to a z_offset of 1.870. While this offset worked seamlessly with the Prusa firmware, your configuration triggered an error during calibration: "Probe triggered prior to movement klipper." To fix this, I increased the horizontal_move, preventing the PINDA from being triggered excessively.

Revo-Hotend Configuration:

Added a dedicated configuration for the Revo hotend. Prusa now supports the hotend so it might be good to add the configuration.

Looking forward to your feedback!
Cheers,
chrisvgt

If I remember correctly position_max was set to handle Bondtech hot-end correctly.

Wouldn't it be better to create an extra configuration file for this hotend? Otherwise the default hotend will not be calibrated correctly with the old setting.

If I remember correctly position_max was set to handle Bondtech hot-end correctly.

Wouldn't it be better to create an extra configuration file for this hotend? Otherwise the default hotend will not be calibrated correctly with the old setting.

If I remember correctly position_max was set to handle Bondtech hot-end correctly.

Wouldn't it be better to create an extra configuration file for this hotend? Otherwise the default hotend will not be calibrated correctly with the old setting.

Yep it was leftover, removed it.


Initial RPM build

Summary and Scope

Builds a distributable package for SUSE/RedHat.
Issue Type

RFE Pull Request

This build an RPM for SUSE/RedHat that installs the sh/ library into /usr/lib/csm-common-library. The RPM version is determined by the return from git describe --tags, the return will determine if the built RPM is stable or unstable. "Stable" RPMs are created when a semver git-tag is taken (vX.Y.Z), "Unstable" RPMs will be created by any commit that does not have a semver tag.
This also changes the .md (markdown) files into .adoc (asciidoc), this still renders on GitHub and offers some more flexibility for our documentation.
When the library is installed, the following files are available:

Docs and examples (/usr/share/doc/csm-common-library)
redbull-ncn-m001-pit:~ # ll /usr/share/doc/csm-common-library/*
-rwxr-xr-x 1 root root 3108 Nov 12 22:48 /usr/share/doc/csm-common-library/README.adoc

/usr/share/doc/csm-common-library/examples:
total 12
-rwxr-xr-x 1 root root 1690 Nov 12 22:47 cmdhook.sh
-rwxr-xr-x 1 root root 4773 Nov 12 22:47 README.adoc

/usr/share/doc/csm-common-library/sh:
total 4
-rwxr-xr-x 1 root root 1901 Nov 12 22:48 README.adoc

Libs
redbull-ncn-m001-pit:~ # ll -R /usr/lib/csm-common-library/*
/usr/lib/csm-common-library/sh:
total 28
-rwxr-xr-x 1 root root 9108 Nov 12 22:47 cmd.sh
-rwxr-xr-x 1 root root 1927 Nov 12 22:47 internal.sh
-rwxr-xr-x 1 root root 1911 Nov 12 22:47 lib.sh
-rwxr-xr-x 1 root root 4058 Nov 12 22:47 logger.sh
-rwxr-xr-x 1 root root 1936 Nov 12 22:47 yaml.sh

Prerequisites

[ ] I have included documentation in my PR (or it is not required)
[ ] I tested this on internal system (if yes, please include results or a description of the test)
[ ] I tested this on a vshasta system (if yes, please include results or a description of the test)

Idempotency

Risks and Mitigations

@mitchty , when the library loads it prints off the following messages
-bash: SOURCEPREFIX: parameter null or not set
-bash: SOURCEPREFIX: parameter null or not set
-bash: SOURCEPREFIX: parameter null or not set
-bash: SOURCEPREFIX: parameter null or not set

I'm not sure we want to do that, or if we do then I think we need some default value and maybe it should be printed once. Thoughts?

Note that the docs are accidentally all executable, I'll fix that before this goes in.
redbull-ncn-m001-pit:~ # ll /usr/share/doc/csm-common-library/*
-rwxr-xr-x 1 root root 3108 Nov 12 22:48 /usr/share/doc/csm-common-library/README.adoc

/usr/share/doc/csm-common-library/examples:
total 12
-rwxr-xr-x 1 root root 1690 Nov 12 22:47 cmdhook.sh
-rwxr-xr-x 1 root root 4773 Nov 12 22:47 README.adoc

/usr/share/doc/csm-common-library/sh:
total 4
-rwxr-xr-x 1 root root 1901 Nov 12 22:48 README.adoc

@mitchty , when the library loads it prints off the following messages
-bash: SOURCEPREFIX: parameter null or not set
-bash: SOURCEPREFIX: parameter null or not set
-bash: SOURCEPREFIX: parameter null or not set
-bash: SOURCEPREFIX: parameter null or not set

I'm not sure we want to do that, or if we do then I think we need some default value and maybe it should be printed once. Thoughts?

Its intentional, there is no portable bourn/posix shell way to know where a script was imported (BASH_SOURCE doesn't work on ash or dash as an example) to know its own base directory. So each usage site has it used with ? which is why you're seeing that 4x the intent is calling scripts provide the base directory so that lib.sh can import everything else in the right order. Its there to remind you that you're not specifying a required env var.
https://github.com/Cray-HPE/csm-common-library/blob/main/sh/lib.sh#L30-L38
I could wrap all this in an if statement I suppose the readme in the sh dir also lists the rationale.
https://github.com/Cray-HPE/csm-common-library/tree/main/sh#rough-design-plan-not-yet-fully-implemented
The example also shows the intended usage aka calling/sourcing shell sets it before sourcing:
https://github.com/Cray-HPE/csm-common-library/blob/main/sh/lib.sh#L30-L38

Works, ready for a thorough review.
1d05ed4c158b:/workspace # rpm -Uvh https://artifactory.algol60.net/artifactory/csm-rpms/hpe/unstable/sle-15sp4/csm-common-library/x86_64/csm-common-library-0.0.1~7~g2c238a2-1.x86_64.rpm
Retrieving https://artifactory.algol60.net/artifactory/csm-rpms/hpe/unstable/sle-15sp4/csm-common-library/x86_64/csm-common-library-0.0.1~7~g2c238a2-1.x86_64.rpm
Preparing...                          ################################# [100%]
Updating / installing...
   1:csm-common-library-0.0.1~7~g2c238################################# [100%]
1d05ed4c158b:/workspace # python3
Python 3.10.8 (main, Jan 22 2023, 16:48:35) [GCC 7.5.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import csm
>>> exit()

Installed files:
1d05ed4c158b:/workspace # rpm  -qi --filesbypkg --provides csm-common-library
Name        : csm-common-library
Version     : 0.0.1~7~g2c238a2
Release     : 1
Architecture: x86_64
Install Date: Fri Jan 27 00:42:48 2023
Group       : Unspecified
Size        : 78649
License     : MIT
Signature   : (none)
Source RPM  : csm-common-library-0.0.1~7~g2c238a2-1.src.rpm
Build Date  : Fri Jan 27 00:41:47 2023
Build Host  : c50b7e23539c
Relocations : (not relocatable)
Packager    : https://www.suse.com/
Vendor      : Hewlett Packard Enterprise Development LP
URL         : https://github.com/Cray-HPE/csm-common-library.git
Summary     : A library for providing common functions to Cray System Management procedures and operations.
Description :
A library for providing common functions to
Cray System Management procedures and operations.


Batchrunner integration

Consider using mesa's BatchRunner to run over a set of classes in parallel.
Batchrunner accepts a number of processes. Make this configurable in run.py.
The way we construct an output file in SimModel.py needs to be thought about carefully if we add parallelisation. We will need to block when we append output to the dataframe that gets written to a file after the model has finished.
Test whether this parallelisation leads to a performance gain on the Hamilton supercomputer.

Significant speed up on Hamilton was noticed. BatchrunnerMP has been tested on single node runs with 1 task per node and 24 cpus per task (24 cpus is the max for Hamilton's par7.q queue). Results on the small dataset are as follows:
Without batchrunner:
real  8m54.292s
user  8m37.292s
sys 0m5.252s
With batchrunner
real  2m7.336s
user  9m31.102s
sys 0m3.240s
Results on the full dataset to follow.


[stdlib] Implement Never conformance to Codable

This supplies conditional conformance for Never to the Codable protocol.
Pitch thread: https://forums.swift.org/t/pitch-conform-never-to-codable/64056/21

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please test


[BUG] Missing search scores on citations when using Azure Search with ChatCompletions and AzureCognitiveSearchChatExtensionConfiguration

Library name and version
Azure.AI.OpenAI 1.0.0-beta.13
Describe the bug
We're working on a chat bot that uses a GPT model in an Azure OpenAI service with Retrieval Augmented Generation to access internal domain data residing in an Azure Search index. We're using vector search with the ada-002 embedding model.
We use the AzureCognitiveSearchChatExtensionConfiguration object on ChatCompletionsOptions to enable the LLM to access our Azure Search instance.
When we get a chat completion response we are lacking information about search scores for each citation. In this example you've documented that search scores are included when using the REST endpoint.
In our scenario, even when we get Citations in the response, the metadata object is lacking and we can't see any search scores. We're trying to understand what citation is most relevant to improve our UI.
Expected behavior
Citation metadata object populated with search scores, as in the the Microsoft documentation: https://learn.microsoft.com/en-us/azure/ai-services/openai/reference#example-response-3
Actual behavior
No metadata returned with the citations.
Reproduction Steps

Setup an Azure Search index, populate text and vector fields (using ada-002 embeddings)
Setup a Azure OpenAI chat solution using AzureCognitiveSearchChatExtensionConfiguration to search in the Azure Search index.
Trigger a chat completion that get's a citation and see that the metadata is not populated in the response from the Azure OpenAI service (missing search scores for citations).

Environment
.net 8.0
Windows 11

Thank you for your feedback.  Tagging and routing to the team best able to assist.   //cc: @trrwilson

Looks like this was not addressed in the beta.14 version of the package, where the Citations are now a typed object instead of a json blob.
@jsquire  - any indication if the search score will be included in that object or not would be appreciated.

@grankko:  Aside from triaging to the correct owners, I have no insight into the request nor its current state.   You'd want to ask @trrwilson and team.

Thank you for reporting this! The most recent Azure OpenAI service versions introduced structured representation of On Your Data citation information (vs. an opaque JSON document in "content"), but it appears that metadata have been missed in the strongly-typed schematization here:
https://github.com/Azure/azure-rest-api-specs/blob/d4168119adedb41c10321a12cad0d0ba37a77cfe/specification/cognitiveservices/data-plane/AzureOpenAI/inference/preview/2024-02-15-preview/inference.json#L4593
This SDK reflects that potential omission. I'm following up with the service team to check on this; assuming it's missing from the spec, we'll correct that and then propagate the update to all impacted libraries (including this one).

I talked with the On Your Data feature team and was informed that this 'metadata' field was a previously unintentional inclusion in the response payloads -- one that's since been removed on contemporary deployments and thus can't have any representation (structured or otherwise) in API/SDK surfaces.
It's understood that these scores can be important and there's active discussion about more formally (and with clearer detail) exposing the scores in the structure of the citation data. No ETA is yet available for that, but it's scheduled for an internal review next week.

Thank you for looking into this, we really appreciate it.
When we started our initiative we experimented first with creating the search feature as a hand rolled ChatFunction. This gave us more control (like being able to access metadata such as search scores). We've since moved over to the OnYourData-way, which is more of a black box but also nice not having to build and maintain that part ourselves.


Create Python server for octopus, autoconversion, and analysis

Use case
As a developer working on octopus streaming, analysis, and autoconversion, I want a Python server in AWS.
Acceptance Criteria
A python server exists that can be deployed to.
Details
Autoscaling is out of scope, this is just one server node that we can access by IP, likely EC2?
Work with software on this.

Software JIRA ticket for tracking this work: https://aicsjira.corp.alleninstitute.org/browse/SWE-463

We've got the EC2 instance up and running and I just manually deployed octopus to it and was able to get a test smoldyn file to get converted via the example UI!
Server IP address: <IP_ADDRESS>
Name: aics-octopus
AWS account: aics-ac
AWS region: us-east-2
Instance type: r5b.large
Still plenty of work to do (including getting a formal deployment process setup, run more testing, etc), but I think we've made enough progress to close this ticket out.


toJson inconsitency format sometimes when using in

Laravel Version: 5.5.4
PHP Version: PHP 7.1.12-3+ubuntu17.04.1+deb.sury.org+1
Database Driver & Version: mysql  Ver 14.14 Distrib 5.7.20

Description:
Illuminate\Database\Eloquent\Collection and Illuminate\Support\Collection don't give the same responses when using ->toJson() and gives one big object instead an array of objects
Steps To Reproduce:
Cant give all the steps since I dont know when its giving the inconsistency
$players_test1 = $tournament->getRedCardPlayers()->take(2);
        $players_test2 = Player::take(2)->get();
        dd(
            $players_test1->toJson(),
            $players_test2->toJson()
        );

The $players_test1->toJson() that is an instance of  Illuminate\Support\Collection gives
{"0":{"id":91,"name":"Cayla Gorczany","team_id":141,"updated_at":"2018-01-11 10:30:40","red_cards":"1","attendances":1},"4":{"id":95,"name":"Reilly Donnelly","team_id":147,"updated_at":"2018-01-11 10:30:40","red_cards":"1","attendances":0}}

and the $players_test2->toJson() that is an instance of Illuminate\Database\Eloquent\Collection gives
[{"id":1,"name":"Euna Berge","team_id":1,"updated_at":"2018-01-11 10:26:45"},{"id":2,"name":"Monique Yost","team_id":1,"updated_at":"2018-01-11 10:26:45"]

Both are collections of the same model collections
Maybe it is the way that i`m doing my collection, however when using it as an object everything looks normal
    public function getTournamentPlayer()
    {
        $players = collect();
        foreach ($this->teams AS $team) {
            $players = $players->concat($team->players);
        }

        return $players;
    }

Actually it was missing the ->values() method


Update PropelCSVParser.php

Regexp replaced with build in php function str_getcsv(PHP 5.3+). It allow to correct handle empty values in csv file without "N;" as value

As Propel support 5.2.4+ and str_getcsv come only in 5.3+ closing this.

@sh0dow could you have a look if this improvement is something worth to add into Propel2 [1]
[1] https://github.com/propelorm/Propel2


import refresh cli only if click is installed

#22

Thank you! I just merged your changed and published 0.5.1 with your fix: https://pypi.org/project/refreshcss/0.5.1/.


Email Publishing - Error transaction failed - page not found

[<ntfy-a@ntfy.example.com>](mailto:ntfy-a@ntfy.example.com): host mx1.example.com[x.x.x.x] said: 554 5.0.0
    Error: transaction failed, blame it on the weather: error:
    {"code":40401,"http":404,"error":"page not found"} (in reply to end of DATA
    command)

Config can be found here

Wrong base-url, should have http/s://


Improving the LDAP implementation

This PR aims to address flaws encountered in the initial, rudimentary LDAP implementation. The following improvements have been made:

[X] Added PV_LOG_LEVEL environment variable to set backend log level, enabling verbose output for better troubleshooting.
[X] Implemented more verbose logging to help debug LDAP-related issues.
[X] Relaxed the LDAP username regex to allow more flexibility.
[X] Fixed issues with user logins containing special characters within the user's DN by switching to ldapts.
[X] Made the "memberOf" and email attribute names configurable.
[x] Properly handle email-like login credentials in LDAP (instead of completely disabling email login when LDAP is enabled).
[x] sync LDAP users' emails properly (#601).

With this PR, the remaining LDAP-related issues should either be resolved or significantly easier to troubleshoot.

As posted in #582 i can confirm this fixes the issues i was having.

Thanks again for your contribution :)


Add math conversion test case for ast_builder

Goal

Add test cases about "DEGREES", "RADIANS" for AST Builder

Todo

[x]  Add test cases about "DEGREES", "RADIANS" for AST Builder

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

46 of 50 (92.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 98.911%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

test-suite/src/ast_builder/function/math/conversion.rs
46
50
92.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
42700

Relevant Lines:
43170

💛  - Coveralls


Correction for raw affiliation UMR IATE – INRA, Montpellier SupAgro, Bât. 37, 2 place Pierre Viala, 34060 Montpellier Cedex 1, France

Correction needed for raw affiliation UMR IATE – INRA, Montpellier SupAgro, Bât. 37, 2 place Pierre Viala, 34060 Montpellier Cedex 1, France
raw_affiliation_name: UMR IATE – INRA, Montpellier SupAgro, Bât. 37, 2 place Pierre Viala, 34060 Montpellier Cedex 1, France
new_rors: 03rnk6m14;0000n5x09
previous_rors: 03rnk6m14
works_examples: W1967247148
contact: e56c969afb96ccd9ca69bf8453e73417:5c852821dbbe7afb5ca37d474cb3ee69e069 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


WIP: Move to SL official softlayer-go library

The official softlayer-go code for each API data type and service method is pre-generated, using the SoftLayer API metadata endpoint as input, thus ensuring 100% coverage of the API right out of the gate. The libray use a session struct to manage all Softlayer Services. Services-relative data can be queried by calling the Mask(), Filter(), Limit() and Offset() service methods prior to invoking an API method. All non-slice method parameters are passed as pointers. Like method parameters, all non-slice members are declared as pointers in datatypes. A custom error type is returned when API services error occurs, with individual fields that can be parsed separately.
This PR is working items with features, and details are below:

Use official softlayer/softlayer-go API lib.
Recode SoftLayer client and relative UTs with a fake SoftLayer server.
Recode a customize logger to handle softlayer-go logs.
Recode CPI actions, implement snapshot_disk, delete_snapshot, get_disks CPI actions.
Use https strictly to setup VPS client.
Use http registry instead of file registry (user_data.json).
Move network/disk configuration from cpi to bosh-agent to eliminate ssh login.

The new CPI could deploy director by bosh-deployment and deploy other apps. For future applicability, we need to keep on adding new features and test cases to enhance CPI's reliability and applicability.
Hi, @mattcui. Please help to review. The change was so huge that manual review was difficult. Thank you very much.

Hi @maximilien, as you know, this is a very big change, as we changed the understructure and datatype, we have to rewrite almost all of action and service code, as @EdwardStudy stated in the description, there also introduced some outstanding features (like elimination of ssh, using http registry and etc) to make our new CPI be very closed to other CPIs of the community. We are still working on all missing UTs and try to pass BATs. We can't gradually replace the existing CPI, so we didn't submit the PR to master branch, we will merge to master after we make sure it passes all testing and get your reviewed/approved. In the meantime, I hope you can help take a look at the WIP code, maybe client.go is good starting point to review. Please let us know if you have any questions/concerns. Thanks.

I cannot see the file changes. Maybe a better strategy is many PRs... so for instance the ssh changes could be separate from the rest and many PRs would allow easier feedback.
Of course, this is up to you guys since if you pair on this maybe it's OK not to have me review.
Finally, while I realize it's a lot of changes, I'd expect things to not be very different. Lots of text and changes that are simply transformation from one to the next. You could almost write code to do those changes... I suspect this since in the end it's the same cloud we are targetting.

Hi,~ @maximilien @mattcui. We updated UT cases and added some new fixes this week.
The details of the fixes are as follows:

Support using subnetIds to specify SoftLayer's subnets in deployment manifest. So users can specify VM's subnets or vlans right now. To make process clearly, CPI only specify VM's network components by either of the two ways.
Support unordered arrangement of vlanIds/subnetIds fields in deployment manifest. CPI will check which vlan or subnet is to use the public gateway.

And we used coveralls.io to our code test coverage. And I think using this tool is enough to easily inspect our code quality. we are glad to receive your suggestions.
Please help to review. Thank you very much.


Add Summary Status for Facility Lists

Overview
Adds a field to the facility list detail endpoint that is a DISTINCT list of all item statuses in the facility list. This list is inspected in the front-end and is used to show a status message in the pagination row.
Connects #327
Demo
Processing:

Error:

Testing

Check out this branch and server
Go to :6543 and log in
Upload a list

[ ] Ensure the list details page shows a summary status above the list items in the table

manage batch_process the list

[ ] Ensure the list details page shows the correct summary status at each step

How does this look:
Black and bold:

vs
Black only:

vs
Bold only:

Added "Bold only" styling in 6762406.

Cool. Bold only looks great!

Looking at this now

One way of dealing with this would be to re-fetch the list details using the existing API and update the state.

Alright I'll look into this. May be tricky to keep the pagination state in place when refreshing the larger list.

Added ff57c65 which updates the summary status as the user is interacting with the items in the list.
The manner it is implemented in makes the shape of the data from the confirm and reject endpoints different from the other list item details endpoints, but perhaps that is okay for now.
This is ready for another look.

Looks like a reasonable solution. Testing now

Squashed fixups. Merging now. Thanks for all the feedback!


Node is now preinstalled in the base docker image

Unless you need Node v5, NodeJS (v6 right now) is preinstalled in the base image (https://github.com/bitrise-docker/bitrise-base/blob/master/Dockerfile#L81)

@viktorbenei thanks for the patch :)
I'll make sure Cordova Android project compiles fine in bitrise after this patch.

Thanks! If there's any nodejs restriction we should be aware of, just let us know!
We plan to upgrade from Node 6 to 7 in the base image, sound that be an issue?

https://github.com/vgaidarji/cordova-android-sample this project was built correctly with patched docker image. Looks like everything is fine.

🎉 😉


[Feature]: migrate to nx

Package Scope
etc
Description
turbo to nx
Possible Solution
No response
etc.
No response

Why would you move from turbo to nx?

Because nx support configuring project in each package's package.json. It make us feel easy to manage each project's dependencies to other projects
turbo don't support it yet.
Actually, this issue is really minor yet. but I just made this issue to consider it to manage packages easily in long term perspective

Thank you for your kind reply!

https://github.com/vercel/turbo/issues/986
There is no watch mode in turborepo but nx support it.
https://nx.dev/nx-api/nx/documents/watch
https://github.com/TanStack/query/blob/main/package.json#L26C8-L26C8

https://github.com/suspensive/react/issues/551 It's related with this


Support paper 1.21?

Currently, Version 1.21 of Paper has not yet reached a stable version. Once it does, its support will be added to MelodyMine.


Easy disabling of output warnings

In the past, if I remember well, I used to add rules to the package.json file to ignore compile warning types. Now I don't have access to that code any more and Google is not helping me in finding out how it is supposed to be. Tried simply adding:
"eslintConfig": {
    "rules": {
      "jsx-a11y/alt-text": 0
    }
  }

Both to package.json of the project and of the eslint-loader module (hack). In the docs and online I can find how to add eslint to the editor but not to the output and I assume this should be documented, and easily accessible, too.

Please add labels to images. This is important for accessibility.
You can also disable individual lines, as suggested in the terminal. This is helpful if you have some exceptional case where you’re sure you know what you’re doing.
If you categorically don’t agree with the default rules that are enforced, you can run eject, and maintain your own linter configuration.

Thanks for the answer. I actually use a lot create-react-app for prototyping. Anyway, I guess it is not possible to simply override individual eslint rules without ejecting. I probably ejected the projects I mentioned.


Collection view debug dump script empty

This code is showing up empty when used here.
Example

🤔  so what's happening here?


Add support for Rules

Rules are a feature of AWS Service Catalogue 1 but is undocumented for pure CloudFormation. However, it does work in vanilla CloudFormation as well 2.
This adds minimal support for Rules in templates with a Template.add_rule() function.
A future improvement could bring a Rule class to validate the contents of the rule itself.
Originally contributed by @nielslaukens at https://github.com/cloudtools/troposphere/pull/1412

@nielslaukens since this was based on your original PR, can you review and give your +1?

I would add a check for duplicate names, like it's done for .add_resource(). Since we're currently don't have a BaseAWSObject for a Rule, I've just copied the duplicate checking part.
I would also add a check on the maximum number of rules. I've searched for a documented maximum number of rules, but couldn't find one in the documentation. If I find the time, I'll try to figure out the current actual limit and configure that. Or put in a TODO for later...
I've pulled in your changes, and added both of the above in my branch (#1412)

Merged via the updated #1412.


Run tests with warnings treated as errors

To avoid unexpected incompatibilities with Python and dependencies, it is recommended to run tests with warnings treated as errors. Many libraries, and Python itself, issue DeprecationWarning messages when deprecated code is used, so that libraries can be notified early and adapt.
Click 8.0 deprecated some code that Typer used, and the next feature release 8.1 removed the previously deprecated code. Click issued deprecation warnings during this period which explained the deprecation, when it would be removed, and what to use instead. Typer wasn't testing warnings, so it didn't see this and wasn't able to make a proactive compatibility fix.
Note that this isn't specific to code in Click. Python itself, or dependencies in test/docs environments, may also change in ways that can be detected if warnings are treated as errors. It's also a good idea to build docs with that setting for the same reason.

This can be accomplished by adding a pytest.ini file with the following:
[pytest]
filterwarnings =
    error

If there are certain warnings that you want to deliberately keep, you can use with pytest.warns(...) to test them. If there are certain warnings that you deliberately want to ignore, you can add rules for them to filterwarnings, see https://docs.python.org/3/library/warnings.html#warning-filter.

Interesting. I enabled this in my own project just now. It immediately flagged two of my unit tests for not explicitly closing file descriptors - which was true. I added calls to fd.close() and it was happy with that, which actually improved the tests because the functions they are testing are expected to return open file descriptors, and now I'm explicitly closing them which is a test that they were open, if you see what I mean.

It can be nice to have 2 CI runs in parralel. One with warnings and one without, for better feedback about what went wrong for contributors


nvm ls-remote extremely slow on ubuntu 14.04

Since I'm a current n user, I spun up a new Ubuntu 14.04 machine to test out nvm.
But some commands are taking a very, very long time. Two minutes to do an npm ls-remote! Is this normal?
root@ubuntu:~# time nvm ls-remote
        v0.1.14
        ...
        v6.3.0

real	2m8.963s
user	0m0.070s
sys	0m0.083s

Here's my debug info:
root@ubuntu:~# nvm debug
nvm --version: v0.31.2
$SHELL: /bin/bash
$HOME: /root
$NVM_DIR: '$HOME/.nvm'
$PREFIX: ''
$NPM_CONFIG_PREFIX: ''
nvm current: v4.4.7
which node: $NVM_DIR/versions/node/v4.4.7/bin/node
which iojs:
which npm: $NVM_DIR/versions/node/v4.4.7/bin/npm
npm config get prefix: $NVM_DIR/versions/node/v4.4.7
npm root -g: $NVM_DIR/versions/node/v4.4.7/lib/node_modules

Same issue here too.

nvm ls-remote does two curl calls, and then formats the output, including checking which versions you have locally installed.
Can you try nvm ls-remote node and nvm ls-remote iojs? If both are slow, can you try timing nvm_ls_remote and nvm_ls_remote_iojs?
My suspicion is that it's either a network problem, or a "checking the filesystem" problem.

Looks like the iojs command is indeed quite a bit slower.
root@ubuntu:~# time nvm ls-remote node
        v0.1.14
        ...
        v6.2.1
->      v6.2.2
        v6.3.0

real	0m1.605s
user	0m0.100s
sys	0m0.097s

root@ubuntu:~# time nvm ls-remote iojs
    iojs-v1.0.0
    ...
    iojs-v3.3.1

real	0m31.810s
user	0m0.033s
sys	0m0.007s

Also, now that things are primed, the nvm ls-remote is quick:
root@ubuntu:~# time nvm ls-remote
        v0.1.14
        ...
        v6.2.1
->      v6.2.2
        v6.3.0

real	0m2.052s
user	0m0.123s
sys	0m0.097s

I tried logging out and logging back in, and things were still fast. Rebooted the Linode, and things were still fast. This might have something to do with initial machine bootup. Maybe DNS lookup? Some sort of precaching, causing network or disk to be slow?

It could be DNS lookup of iojs.org - network conditions aren't really something I can help with, sadly.
I'm glad it's fast now!

Thanks for spending the time on this. I would appreciate something like a --verbose command that would help me diagnose this myself. Or some sort of 'waiting for request' kind of message that shows up after 10 seconds. Or a timeout passed to curl/wget.
I was afraid that the install had failed, that the process was hanging! Usually applications either respond within about 10 seconds or hang completely. Only after trying a couple times and killing it did I finally relax and wait long enough for it to finish.

That's a fair point - I could certainly add a --verbose command to nvm install/nvm ls-remote/nvm version-remote that would proxy it through to curl/wget. Do you mind filing a new issue for that?


Add securityContext to ARO worker and master operator deployments as required by k8s v1.27.x

Which issue this PR addresses:
Fixes https://issues.redhat.com/browse/ARO-3955
What this PR does / why we need it:
in 4.15 pods will fail to be created unless they correctly satisfy the securitycontext labels on either the namespace or pod(s) themselves.
Test plan for issue:
Create a cluster, update the deployment, check the pod security admission to determine if it's still throwing an error on AdminUpdate

looking for things like pod-security.kubernetes.io/audit-violations in the annotations section of audit logs

Is there any documentation that needs to be updated for this PR?
All clusters will need a round of PUCM if they're >=4.11.

PR looks good.  Logs are not showing any violations on the ARO operator or genevalogging compared to another PR run.
Looks like there may be a violation on an e2e container though as part of our e2e.  I'll dig in and fix in a separate PR.

LGTM, just a few questions for my own understanding really.
One general question: There are a lot of other deployment yamls in the repo that we aren't touching here. Eg https://github.com/Azure/ARO-RP/blob/master/pkg/util/deployer/staticresources/deployment.yaml.
Why can we ignore these other deployments?

I don't think we can.  I originally made the PR on what I saw emitting failures during installation.  Looks like there are others as well which we create.
I'll dig a bit.


linker command failed with exit code 1 (use -v to see invocation)

Hello,
I'm unable to run this plugin with Xcode, I'am getting this error :
116 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Your Environment

Plugin version: "0.4.1"
Platform: iOS
OS version: Any
Device manufacturer and model:  I'm not able to install to a device
React Native version: 0.51

Context
I'm using a detached expo. I'm doing those commands.
$ npm isntall
$ exp detach
$ react-native link react-native-background-geolocation
$ cd ios
$ pod install
All command are success, but when I try to build my project I'm getting this error.
But, If I don't link the react-native-background-geolocation to the project, w
Expected Behavior
It should build successfully.
Actual Behavior
I'm getting this error
116 duplicate symbols for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Possible Fix

Steps to Reproduce
1 .$ npm isntall
2. $ exp detach
3. $ react-native link react-native-background-geolocation
4. $ cd ios
5. $ pod install
6. open xcode and build project.

Can you try install instructions from documentation? react-native link is not supported.

I am getting the same error. I followed the installation instructions and had no problems. But then installed a new library that does use react-native link. Does the documentation include how to undo react-native link?

@wmlutz did you find a solution to this ?

Latest alpha supports react-native link. Please give it a try.

I am running into this issue now with xCode 14


Added copy option as polymorphic option, accepts boolean and function

I couldn't access o inside isCopy as a global variable in my testing, so I pass it as first argument. Then depending on the status of the drag ,the container and the target are not always there, null is passed it one doesn't exists.
It all works fine in my integration, I'll be interested to get some feedback though.

Just place isCopy inside the method where o is scoped so you don't have to pass it around everywhere.

Move isCopy function to where it should be.

Let's discuss this as a whole in #127


Do there exist examples that use IAsyncQuickInfoBroker?

I like to migrate to the new IAsync* QuickInfo api, but I can't find any examples that use this new API. I'm a bit confused how to migrate the following piece of code. Is the IntellisenseController still part of the new API? Where do I call the AsyncQuickInfoBroker?
internal sealed class QuickInfoController : IIntellisenseController
    {
        private readonly IList<ITextBuffer> _subjectBuffers;
        private readonly IAsyncQuickInfoBroker _quickInfoBroker;
        private ITextView _textView;

        internal QuickInfoController(
            ITextView textView,
            IList<ITextBuffer> subjectBuffers,
            IAsyncQuickInfoBroker quickInfoBroker)
        {
            this._textView = textView;
            this._subjectBuffers = subjectBuffers;
            this._quickInfoBroker = quickInfoBroker;

            this._textView.MouseHover += (o, e) => {
                SnapshotPoint? point = GetMousePosition(new SnapshotPoint(this._textView.TextSnapshot, e.Position));
                if (point.HasValue)
                {
                    ITrackingPoint triggerPoint = point.Value.Snapshot.CreateTrackingPoint(point.Value.Position, PointTrackingMode.Positive);
                    if (!this._quickInfoBroker.IsQuickInfoActive(this._textView))
                    {
                        this._quickInfoBroker.TriggerQuickInfoAsync(this._textView, triggerPoint);
                    }
                }
            };
        }

@gundermanc to answer the question, but a sample can be found at https://github.com/Microsoft/VSSDK-Extensibility-Samples/tree/master/AsyncQuickInfo

Turns out the intellisense controller was never needed for quick info because the IDE provides one. Simply implementing the quick info source is enough. You shouldn't need to call the broker at all unless you need to programmatically pop quick info for some reason.
The sample Oleg linked is the definitive sample. Note that you're called on a background thread.

Thank you for the quick reply.
@olegtk, I seemed I hadn't pulled that repo recently, AsyncQuickInfo demo is what I sought.
@gundermanc, I didn't know that I didn't need one! "Note that you're called on a background thread." Is there an simple method to add a UserControl to the QuickInfoItem that is returned by GetQuickInfoItemAsync? It is not obvious (to me) how to do that. Or is it still not advisable to add UserControls (with buttons etc) to a QuickInfoItem?

Adding user controls is still supported, albeit a little bit more roundabout. My recommendation would be to use the built in data-types, but if you need a custom piece of UI, you can 1) define a custom data type and 2) define a IViewElementFactory that defines how it is rendered as UI on the UI thread. AFAIK, Visual FSharp is the only public example of this.
Keep in mind that going this route may prevent your content from working with VS Live Share and other advanced scenarios.

Yes that helped. Thanks for the info!


Update version in npm

What problem does this feature solve?
Currently published proxy module relies on<EMAIL_ADDRESS>which uses some outdated dependencies. In repository package.json already uses ^0.20.0, but after I removed @nuxtjs/axios and installed it back, I still got<EMAIL_ADDRESS>
This feature request is available on Nuxt community (#c42)

Hi. Just published 2.0.0 using<EMAIL_ADDRESS>(just to ensure there is no regressions). Axios and HTTP modules will update dependency accordingly


Fix: VM does not boot if no SSH key is specified

Problem: The authorized_keys file could not be generated if the config.authorized_keys settings was None because it was used in a list comprehension.
Solution: check for None early.

Should we make the authorized keys a List with default [] instead of allowing None ?


Sudden Failures on SaaS deployement

Describe the bug
ERROR: Failed to connect to MSI. Please make sure MSI is configured correctly.
To Reproduce
Steps to reproduce the behavior:
Azure CLI
git clone https://github.com/Azure/Commercial-Marketplace-SaaS-Accelerator.git
./Deploy.ps1 -WebAppNamePrefix app-new -PublisherAdminUsers "[info@contoso.com](mailto:info@contoso.com)" -ResourceGroupForDeployment AppResourceGroup -Location "East US"
See error:

Creating Fulfillment API App Registration
ERROR: Failed to connect to MSI. Please make sure MSI is configured correctly.
Get Token request returned: <Response [400]>
ERROR: argument --id: expected one argument
Expected behavior
FulfilmentAPI App Registration created With a Application ID & App Secret:
This then subsequently fails Key-vault key creation
Environment (please complete the following information):
Using the CloudShell

Resolved by just running az login again
see here : https://github.com/Azure/Commercial-Marketplace-SaaS-Accelerator/issues/451#issue-1621410706


Short-circuit _debugCanPerformMutations if debugDoingLayout is false

It's more or less always ok to perform mutations if we're not doing layout and not painting.
Improves https://github.com/flutter/flutter/issues/76261
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

test-exempt: performance improvement with no semantic change

@goderbauer points out i should remove the paint part, since this never returns false during paint currently anyway and is thus probably not even called during paint.

Conceptually, _debugCanPerformMutations probably should return false during paint as you shouldn't be calling adoptChild/dropChild/markNeedsLayout/setupParentData (the methods guarded by _debugCanPerformMutations) during paint, but it appears that the current implementation of _debugCanPerformMutations doesn't cover the paint case.


Only for trial

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: icygalz
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: myidpt
If they are not already assigned, you can assign the PR to them by writing /assign @myidpt in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@icygalz: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
5280bb28929b03bd8c8fc775a960c31f653e90b8
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Seems like the tests are working fine. Closing this draft!