Add functionality to get File by ID

Summary
Starting with Connect 1.3.0, the users can get files that are stored in an Item. Connect Node SDK should enable the users to do that.
In this issue, we will get a file based on its ID.
Tasks to be done

[ ] Implement getFile function in the Items class in src/lib/resources.ts. The API endpoint that this function needs to call is v1/vaults/vaultId/items/itemId/files/fileId./**
 * Get an Item's specific File with a matching ID value.
 *
 * @param {string} vaultId
 * @param {string} itemQuery
 * @param {string} fileId
 * @returns {Promise<ItemFile>}
 * @private
 */
private async getFile(
         vaultId: string,
         itemQuery: string,
         fileId: string,
     ): Promise<ItemFile> {
     // functionality
}

Note: Feel free to place this functionality in a separate class (e.g. Files) if you feel it's more suitable. This is just a suggested approach.
[ ] Implement getFile function in OPConnect class in src/lib/op-connect.ts:/**
 * Get an Item's specific File with a matching ID value.
 *
 * @param {string} vaultId
 * @param {string} itemQuery
 * @param {string} fileId
 * @returns {Promise<ItemFile>}
 */
public async getFile(vaultId: string, itemQuery: string, fileId: string): Promise<ItemFile> {
      return await this.items.getFile(vaultId, itemQuery);
}

[ ] Add a test for the new function in __test__/op-connect.test.ts:test("get file", async () => {
    // actual test here
}

This issue is blocked by #62


jsgrid-grid-body height is always 1px

Can't understand why, but jsgrid-grid-body height is always 1px, even after data loaded from backend:

if I remove this style, grid shown correctly

Please, checkout this issue https://github.com/tabalinas/jsgrid/issues/41. I guess this could be the reason.

Close due to inactivity.

I am facing same for 2 days how can i fixed it.

@mayankvadiya, have you checked this issue https://github.com/tabalinas/jsgrid/issues/41?

Yes, I have checked this issue it was solved, I have changed height:
"auto", in initialization
On Mon, Sep 11, 2017 at 3:31 AM, Artem Tabalin<EMAIL_ADDRESS>wrote:

@mayankvadiya https://github.com/mayankvadiya, have you checked this
issue #41 https://github.com/tabalinas/jsgrid/issues/41?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabalinas/jsgrid/issues/409#issuecomment-328375253,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdG-pj5LT-XY7kO6uCWuCL9MgPa89unoks5shFxVgaJpZM4Jq6uC
.

Hello sir,
I want to make my jsgrid like data-table, Means in my jsgrid i want to put
search textbox(filtering) right top side and display result on keypress
event. Is there any way to do like this ?
On Mon, Sep 11, 2017 at 11:38 AM, mayank kanhasoft <
<EMAIL_ADDRESS>wrote:

Yes, I have checked this issue it was solved, I have changed height:
"auto", in initialization
On Mon, Sep 11, 2017 at 3:31 AM, Artem Tabalin<EMAIL_ADDRESS>wrote:

@mayankvadiya https://github.com/mayankvadiya, have you checked this
issue #41 https://github.com/tabalinas/jsgrid/issues/41?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabalinas/jsgrid/issues/409#issuecomment-328375253,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdG-pj5LT-XY7kO6uCWuCL9MgPa89unoks5shFxVgaJpZM4Jq6uC
.

https://stackoverflow.com/questions/46173816/how-to-bind-textbox-into-tabalinas-jsgrid-and-filter-on-keyup
On Mon, Sep 11, 2017 at 12:38 PM, mayank kanhasoft <
<EMAIL_ADDRESS>wrote:

Hello sir,
I want to make my jsgrid like data-table, Means in my jsgrid i want to put
search textbox(filtering) right top side and display result on keypress
event. Is there any way to do like this ?
On Mon, Sep 11, 2017 at 11:38 AM, mayank kanhasoft <
<EMAIL_ADDRESS>wrote:

Yes, I have checked this issue it was solved, I have changed height:
"auto", in initialization
On Mon, Sep 11, 2017 at 3:31 AM, Artem Tabalin<EMAIL_ADDRESS>wrote:

@mayankvadiya https://github.com/mayankvadiya, have you checked this
issue #41 https://github.com/tabalinas/jsgrid/issues/41?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tabalinas/jsgrid/issues/409#issuecomment-328375253,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AdG-pj5LT-XY7kO6uCWuCL9MgPa89unoks5shFxVgaJpZM4Jq6uC
.

Yes, I have checked this issue it was solved, I have changed height: "auto", in initialization
…
On Mon, Sep 11, 2017 at 3:31 AM, Artem Tabalin @.***> wrote: @mayankvadiya https://github.com/mayankvadiya, have you checked this issue #41 <#41>? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#409 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AdG-pj5LT-XY7kO6uCWuCL9MgPa89unoks5shFxVgaJpZM4Jq6uC .

thanks ！I succeeded as you said


Queue republish_all jobs

Due to the large amount of documents, it is useful to speed up the
republish_all action by sending each document to be republished to a
queue.

It might be worth configuring a lower priority bulk_republishing queue so that normal publishing doesn't get held up by bulk republishes. We configured this in Whitehall here https://github.com/alphagov/whitehall/blob/master/config/sidekiq.yml#L5 and send jobs to that queue here https://github.com/alphagov/whitehall/blob/master/lib/whitehall/publishing_api.rb#L120

@boffbowsh updated commit with config for separate queues.


Steady State Startup

This is only an idea, but not priority
Currently the package only supports a black start condition. That is, when any simulation is run, the network always starts with an initial condition where every capacitor voltage and inductor current is set to zero. Then every source starts ramping up at exactly the same point in time. This is impractical, because it requires instantaneous communication between sources that are spatially distributed. Moreover, the behaviour of the network has a long settling time before entering a steady state, which may hold no interest to the user. As can be seen from the image below, it takes roughly 1 second before the signals reach a steady state.

The solution is to load the LTI system with initial conditions that correspond to steady state conditions. Some of this information is already partially available from the power flow solution. However the power flow solution only contains the steady state values of the cables, not the filters.
What should be done:

Calculate the initial conditions for the filter capacitors and inductors in the time domain (this includes capacitor currents)
Calculate the filter capacitor magnitude and phase
Calculate the source voltage magnitude and phase
Create a Black Start / Steady State flag which then loads the calculated values into the environment (x0 and y0)
Load the controllers with the correct internal conditions

@SeptimusBoshoff, great idea. Is this done already or will be done in future?
If not, please document the status here and close


[BUG] Missing files trying to load PowerPaint

Model
Sanster--PowerPaint-V1-stable-diffusion-inpainting
Describe the bug
I tried running Sanster--PowerPaint-V1-stable-diffusion-inpainting. The first time I ran it, when it downloaded  the model, it returned an error
IncompleteRead(471805400 bytes read, 136210880 more expected))

During handling of the above exception, another exception occurred:

[...]
The second time, I got OSError: Error no file named pytorch_model.bin, tf_model.h5, model.ckpt.index or flax_model.msgpack found in directory C:\Users\inhah\.cache\huggingface\hub\models--Sanster--PowerPaint-V1-stable-diffusion-inpainting\snapshots\4104236baefdb 2a1dd0f32812d8edbbfc4efd164\safety_checker.
I guess the downloads were interrupted, so now it won't work. I'm guessing the downloads being interrupted part was a network error rather than an iopaint bug per se, but I think it should have tried to download the missing files again, if not as soon as the error occurred then at least the next time I ran it.
Here's the full tracebacks from the two or three times I tried running it.
traceback.txt
System Info
Software version used

lama-cleaner: don't know
pytorch: don't know
CUDA: don't know

IncompleteRead(471805400 bytes read, 136210880 more expected))
During handling of the above exception, another exception

I have encountered this error before. At that time, it was because my disk was almost full. Deleting some things solved the problem.

OSError: Error no file

The Huggingface Hub was down last night, I don't know if it has been fixed now.
https://x.com/huggingface/status/1756401140495679871?s=46&t=541K4njVYqJ_ijfXVDR40A

Oh, after checking, I see that I only have about 379 MB free. I guess that's the problem. Thanks.


Remove opensuse-leap-42 offering

Leap 42 is EOL and should not be used/offered anymore; would not expect to see it listed from powershell via wsl --list --online or winget search suse.

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[n/a] Have you validated your manifest locally with winget validate --manifest <path>?
[n/a] Have you tested your manifest locally with winget install --manifest <path>?
[n/a] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


adding creation of selenium data folder to deployment

@OriHoch can you please review this pr?

@OriHoch the permissions reset back to the default after we reboot the instance

@tkalir Thanks, this is indeed a problem that permissions are reset back to default after rebooting.
Merging
Thanks @OriHoch !

hi @OriHoch
the workflow had a problem running after this commit, and even after we removed the the call to bin/apps.py create_selenium_data_folder it still doesn't work.
/home/runner/work/_temp/fe344642-4bc0-466f-b189-3e65f3f5a47a.sh: line 15: bin/apps.py: Permission denied
https://github.com/data-for-change/dfc-iac/actions/runs/12233626737/job/34122649887
do you think it's related to the change?
thanks

@tkalir yes, you removed the execute permission
need to run:
chmod +x bin/apps.py

then commit it again


Bug: If something fails during installing dependencies, it never finishes

Do you want to request a feature or report a bug?
Report a bug.
What is the current behaviour?
If something happens during installing dependencies, an error might show, but it never stops spinning, for example, but not limited to:

No write rights to dest folder
Using force but a file that is overwritten is locked / open
Connection fails

If the current behaviour is a bug, please provide the steps to reproduce.

on windows with UAC open a regular prompt
try to create a new project
rejoice as it fails
cancel
open as administrator
try to create a new project
done

What is the expected behaviour?
When it fails it tells me why, and stops processing, not spinning indefinitely.
Please mention other relevant information.

Node version: v10.7.0
npm version: yarn 1.12.1
Operating system: Windows 10
CLI version: @next

@SleeplessByte, that's an interesting issue - your feature request, too. I personally haven't run into the problem with the standard Windows cmd (or any other shell).
As far as I can see, this block in create.js is where that stage of the create is handled:
if (argv.install) {
    spinner.text = 'Installing dependencies';
    await install(target, isYarn);
}

And the install method in setup.js simply returns a Promise:
exports.install = function(cwd, isYarn) {
    let cmd = isYarn ? 'yarn' : 'npm';
    return spawn(cmd, ['install'], { cwd, stdio });
};

Both the cross-spawn-promise package the the cross-spawn that it's using will throw errors as expected.
Logging other stuff can be done by changing the spawn call to:
spawn(cmd, ['install'], { cwd, stdio: 'inherit' });
The end result would be something like this:

I'm not sure that's the intended effect here, though.

We could just remove the automatic install and output Please cd into the directory and install dependencies

But wouldn't that mean removing the --install option, too? Personally, I find that quite convenient. ;D

Still, if it is causing lots of issues, I would consider it.

@MasterRennor
Yeah, it's an edge case that was actually hard to reproduce because it requires a specific state to hit.
spawn(cmd, ['install'], { cwd, stdio: 'inherit' });

This is what I would have expected. stdio: 'ignore' is fine, but then you need to capture the error output and show that in your own way.
@ForsakenHarmony I actually like that it auto installs. The only issue is the lack of communication.

Wanna make a pr for that option?

Which exact option are you referring to 😅 . I'll gladly PR whatevs!

Setting stdio to 'inherit' if you think that's better

I hope you guys don't mind, but I had some spare time on my hands and created a quick PR for that. :)

This is in the latest RC so I think I can close this


[Records] VM debugging support

The title just states VM debugging support, I suppose you meant [Records] VM debugging support ?

Fixed, thanks for the call out @a-siva


Tests templates require git

Tests.Template\unit_template.ps1 and integration_template.ps1 use git to clone DscResource.Tests in order to have Initialize-TestEnvironment function we use to setup environment.
The problem with this is that in production users will likely be getting module from PSGallery, not cloning from git, so they may not have git installed on their production machine. In such cases, running tests will fail.
I'd like to discuss this issue here and come up with preferred solution, some of the options we have:

Make templates self contained, so that they don't rely on code from separate repository, all helper code should be part of the template itself
Ship DscResource.Tests together with module to the PSGallery

Opinions?

I'm in favor of the second option:

Ship DscResource.Tests together with module to the PSGallery

I'm not sure of the PSGallery publishing process the resources currently use, but I'm thinking this would be easy enough to implement so that the latest DSCResource.Tests module is pulled and included when publish occurs?

yup, that should be easy

Part of the discussion on this can be found here: https://github.com/PowerShell/DscResources/pull/128

Below is the proposal for removing git dependency when running in production:
I'm looking forward to hear your feedback @PlagueHO @tysonjhayes @iainbrighton @mgreenegit @mhendric @nanalakshmanan @TravisEz13 @vors @mbreakey3
GOALS
a.	User should be able to run tests on a production system without relying on git
b.	All common tests should run on AppVeyor builds when a pull request is submitted
c.	Any user who is cloning the repositories for development purposes should be able to run the common tests
d. Do not ship redundant code into packages published on PSGallery
Note
The original solution was to add DscResource.Tests to the packages published to the PS Gallery during the publishing.
This approach required shipping redundant code to multiple resource kit packages on PSGallery (each
Below is the proposal for removing git dependency when running in production:
I'm looking forward to hear your feedback @PlagueHO @tysonjhayes @iainbrighton @mgreenegit @mhendric @nanalakshmanan @TravisEz13 @vors @mbreakey3
GOALS
a.	User should be able to run tests on a production system without relying on git
b.	All common tests should run on AppVeyor builds when a pull request is submitted
c.	Any user who is cloning the repositories for development purposes should be able to run the common tests
d. Do not ship redundant code into packages published on PSGallery
Note
The original solution was to add DscResource.Tests to the packages published to the PS Gallery during the publishing.
This approach required shipping redundant code to multiple resource kit packages on PSGallery (each package contained DscResource.Tests) which as pointed out in #128 is not desirable. As a result we added goal d. and came up with a new proposal below.
PROPOSITION

Publish DscResource.Tests as a package on PSGallery. In fact, we would separate it into two packages:

One containing module with helper code for tests (which all test templates depend on)
Second one containing all the common tests

When running in production (module installed from PSGallery) tests would install package containing test helper module from PSGallery (we would not install common tests, just helper code, cause users typically won’t need those in production – we could revise this decision though if we decide common tests are worth having in production as well)
When running on dev machine (module cloned from github) tests would clone DscResource.Tests from github (common tests + helper code)

ACTION STEPS:

Modify tests which currently clone DscResource.Tests, in a following way:

If module was obtained from github, and DscResource.Tests are not already present, clone DscResource.Tests from github (should we prompt user here as well and skip the prompt in appveyor?)
If module was obtained from github, and DscResource.Tests are already present, pull DscResource.Tests from github to get latest version
If module was obtained from PSGallery and DscResource.Tests are not already present, prompt user whether he wants to download it. If he doesn’t, fail tests with info it’s required.
If module was obtained from PSGallery and DscResource.Tests are already present, proceed to executing tests

Modify test templates in the same way

PROS OF THIS APPROACH:

No redundant code in PSGallery packages (DscResource.Tests isn’t shipped with every DSC module)
Meets our requirements (goals)

CONS OF THIS APPROACH:

Test template logic gets more complex
User interaction is required to run tests in production (this shouldn’t be common scenario though)
We are not running common tests in production, just get helper code (however, we could modify test templates so that we run them if we decide it’s important)

OPEN ISSUES

Shall we add package with test helper code as a dependency for each resource kit module so it gets automatically pulled down from the PS Gallery when installed or should tests install it?

Adding it as dependency would simplify test template (as we wouldn't need to install it from there), but test helper code isn't technically dependency of the module itself (just dependency of tests) and adding modules which aren't really required was already raising some concerns previously: https://github.com/PowerShell/xWebAdministration/pull/104

When running on dev machine, should tests clone helper code + common tests (that would be part of test templates) or should we simply recommend that developers download and run it?

If we will be cloning helper code + common tests, should we do it automatically or shall we prompt user to confirm before doing so?

When running tests in production, shall we install only package with test helper code or also package with common tests? (In fact, running common tests in production was stated as goal c., but I'd like to hear your input whether it's really required).

Note:
There's proposal in pester project to add repo with generic tests: https://github.com/pester/Pester/issues/523
If such tests would indeed be created, we could include them as submodule of DscResource.tests and have published to PSGallery together with our common tests.
We will likely discuss it again once pester guys act on this idea, but just wanted to keep it in mind when agreeing on our approach.
Sorry guys for this wall of text, but there's more issues and scenarios to consider here than it may seem at the first sight. Let's make sure the solution we agree on addresses all of it.
package contained DscResource.Tests) which as pointed out in #128 is not desirable. As a result we added goal d. and came up with a new proposal below.
PROPOSITION

Publish DscResource.Tests as a package on PSGallery. In fact, we would separate it into two packages:

One containing module with helper code for tests (which all test templates depend on)
Second one containing all the common tests

When running in production (module installed from PSGallery) tests would install package containing test helper module from PSGallery (we would not install common tests, just helper code, cause users typically won’t need those in production – we could revise this decision though if we decide common tests are worth having in production as well)
When running on dev machine (module cloned from github) tests would clone DscResource.Tests from github (common tests + helper code)

ACTION STEPS:

Modify tests which currently clone DscResource.Tests, in a following way:

If module was obtained from github, and DscResource.Tests are not already present, clone DscResource.Tests from github (should we prompt user here as well and skip the prompt in appveyor?)
If module was obtained from github, and DscResource.Tests are already present, pull DscResource.Tests from github to get latest version
If module was obtained from PSGallery and DscResource.Tests are not already present, prompt user whether he wants to download it. If he doesn’t, fail tests with info it’s required.
If module was obtained from PSGallery and DscResource.Tests are already present, proceed to executing tests

Modify test templates in the same way

PROS OF THIS APPROACH:

No redundant code in PSGallery packages (DscResource.Tests isn’t shipped with every DSC module)
Meets our requirements (goals)

CONS OF THIS APPROACH:

Test template logic gets more complex
User interaction is required to run tests in production (this shouldn’t be common scenario though)
We are not running common tests in production, just get helper code (however, we could modify test templates so that we run them if we decide it’s important)

OPEN ISSUES

Shall we add package with test helper code as a dependency for each resource kit module so it gets automatically pulled down from the PS Gallery when installed or should tests install it?

Adding it as dependency would simplify test template (as we wouldn't need to install it from there), but test helper code isn't technically dependency of the module itself (just dependency of tests) and adding modules which aren't really required was already raising some concerns previously: https://github.com/PowerShell/xWebAdministration/pull/104

When running on dev machine, should tests clone helper code + common tests (that would be part of test templates) or should we simply recommend that developers download and run it?

If we will be cloning helper code + common tests, should we do it automatically or shall we prompt user to confirm before doing so?

When running tests in production, shall we install only package with test helper code or also package with common tests? (In fact, running common tests in production was stated as goal c., but I'd like to hear your input whether it's really required).

Note:
There's proposal in pester project to add repo with generic tests: https://github.com/pester/Pester/issues/523
If such tests would indeed be created, we could include them as submodule of DscResource.tests and have published to PSGallery together with our common tests.
We will likely discuss it again once pester guys act on this idea, but just wanted to keep it in mind when agreeing on our approach.
Sorry guys for this wall of text, but there's more issues and scenarios to consider here than it may seem at the first sight. Let's make sure the solution we agree on addresses all of it.

@KarolKaczmarek - Great summary- it has helped solidify the issue for me.
Open Issues (my thoughts)

It seems wrong to make "Test Helper Module" a dependency of all resource modules, because as noted the Resource module isn't at all dependent on it. I'm OK with increasing the complexity of the Test.Template code to avoid this.

It seems to me that the method of getting the "Common DSC Tests" may be dependent on what happens with issue pester/Pester#523. So perhaps waiting for progress on that one might be worthwhile.

Putting the "Common DSC Tests" into a module installed from PSGallery would need some thought because to invoke the tests from Pester they need to be in a sub folder of the resource being tested. Could easily work around this by including a cmdlet in the "Common DSC Tests" module to invoke the "Common DSC Tests" on a specified module path, which could do all the work.

I think this also depends on what happens with pester/Pester#523. But my feeling is they would be kept separate. The  "Test Helper Module" should be updated very infrequently, however the "Common DSC Tests" could be updated quite often. But I don't think it will matter too much really.

Note:
I've labelled the two components to help me keep them straight in my mind as well as list the files that I'd expect to end up in each:

Common DSC Tests:

Meta.Tests.ps1

Test Helper Module:

TestRunner.psm1
MetaFixers.psm1
TestHelper.psm1

Is the above what others would expect?

Here is the finalized plan for this issue.
GOALS
a. User should be able to run tests on a production system without relying on git
b. All common tests should run on AppVeyor builds when a pull request is submitted
c. Any user who is cloning the repositories for development purposes should be able to run the common tests
d. Do not ship redundant code into packages published on PSGallery
PROPOSITION
•	Publish DscResource.Tests as two packages on PSGallery.
o	One containing module with helper code for tests (which all test templates depend on)
o	One containing all the common tests (which aren’t necessary for the published modules)
•	When running in production (module installed from PSGallery) tests will install package containing test helper module from PSGallery (helper code package only)
•	When running on dev machine (module cloned from github) tests will clone DscResource.Tests from github (common tests package + helper code package)
ACTION STEPS:

Modify tests which currently clone DscResource.Tests, in the following ways:
•	If module was obtained from github, and DscResource.Tests are not already present, clone DscResource.Tests from github automatically
•	If module was obtained from github, and DscResource.Tests are already present, pull DscResource.Tests from github to get latest version
•	If module was obtained from PSGallery and DscResource.Tests are not already present, prompt user whether he wants to download it. If he doesn’t, fail test with info that it’s required.
•	If module was obtained from PSGallery and DscResource.Tests are already present, proceed to executing tests
•	In order to distinguish whether a module was downloaded from the gallery or from GitHub, the test will use: Get-InstallModule –Name ‘moduleName’ – if Repository == PSGallery, we know it was downloaded from the gallery, if repository is not present we know that the module was cloned from git.
•	2) Modify test templates in the same way

Awesome summary @mbreakey3. Thank you!
For the standard tests (meta.tests.ps1) to run they'll need to be copied into a subfolder within the DSCResource module being tested so that when the user runs Pester, pester finds the meta.tests.ps1 file.
Is the plan that Install-Module would be used to ensure the common tests module on the local machine and then copy the "Meta.tests.ps1" file from that folder and into the DSCResource folder?
I just want to make sure I've got the process clear.

As stated in open issue 3 we were wondering whether there's need to run standard tests in production as well. If that's something we would want to do then we need to make sure those tests are available in the tests folder as well.
Do you see a strong need to run those common tests in production?

I logged the issue around PSModulePath getting changed. https://github.com/PowerShell/DscResource.Tests/issues/55

So we found a few issues with the above solution. Mainly, that when you run Invoke-Pester on your machine for the first time, it won't run the common tests since they're installed/cloned after Pester begins. What we're thinking as a better solution is to write a wrapper function to run Pester that first downloads the common tests and then runs Pester. This way we can also remove this code from the top of all of our tests and just put this one function in our testHelper.ps1 file. Any comments/suggestions on this approach?

@mbreaky3 - that is what we have done. All our internal dsc resource modules and ps modules include an invoke-tests.ps1 file in the \tests\ folder that preforms any init tasks like downloading pssa and pester. Then runs the tests.

@PlagueHO Great! Would it be possible to put this in our testHelper.ps1 file as a function or do you think it's better left in it's own file?

@mbreakey3 - I'm thinking it should be kept as it's own file. Otherwise you'd need some additional code to pull down DSCResources.tests so that you actually had testHelper.ps1
We also usually include a .vscode\tasks.json file as well in each repo - it just kicks off ".\tests\Invoke-pester.ps1" in the repo. But that is just a handy tip - not something that needs to be included.
So my thinking is add Invoke-tests.ps1 into the DSCResource\Test.Templates folder and give instruction to put it into the .\tests folder of any repos.

I agree we want this file in every repo separately, however I'd add comment not to modify it manually. This way if for some reason we want to update that file (e.g. our prerequisites change), we can automatically overwrite them for all repos without updating one by one.
I'm not aware of any good way to disable editing of particular file in git, however we can definitely detect early if such modification happened by signing it and verifying it's signature in one of the common tests.


Embedded files

Question
Good evening, I want more information about the method to embed files or documents in a pdf, more specifically I want to be able to append an existing document as additional or complementary pages to the pdf that I am creating with pdfkit
Reference

I want to use this to include an external pdf inside my code
Code
`
let doc = new PDFDocument({ size: "A4", margin: 50 });
doc.file(config.staticRoute + config.filesRoute + '/attachment.pdf')
    doc.end();
    const routeFile = config.staticRoute + config.filesRoute + '/' + path;
    doc.pipe(fs.createWriteStream('.' + routeFile));`

That generates the following error
     "errno": -4058,     "syscall": "open",     "code": "ENOENT",     "path": "/static/files/attachment.pdf"

This is a node error. File /static/files/attachment.pdf does not exists.
File path must exist in file system (In browser you must pass a Buffer instance or use the virtual file system)

The file actually exists in the directory, I also did another test using a file from aws that is a valid path and that is visible from the browser but the error is the same

Are you using pdfkit in node or in browser?
If in browser, this is not supported. You must download file data and pass a Buffer instance
If you are in node try
const fs = require('fs')
const buf = fs.readFileSync(config.staticRoute + config.filesRoute + '/attachment.pdf)

It should give you the same error

I did not know how to use the implementation you mention, I did the following:
        const buf = fs.readFileSync(config.staticRoute + config.filesRoute + 'attachment.pdf')         doc.file(buf)
But it throws me the following error:
Error [ERR_STREAM_PUSH_AFTER_EOF]: stream.push() after EOF
On the other hand (after implementing this code as it should) as I would do to embed a file that is in aws, I use Heroku to deploy the project and that is why I cannot host files on the local server since they are volatile, as happened in the temporarily file from aws to my server to be able to do the embedding of the pdf?

I would do to embed a file that is in aws, I use Heroku to deploy the project and that is why I cannot host files on the local server since they are volatile, as happened in the temporarily file from aws to my server to be able to do the embedding of the pdf?

I have no idea. You need to learn how to get a Buffer instance from the file content in this cloud environment, and than pass to pdfkit. Something that out of scope of this project


[WIP] Proto for upcoming spt release <IP_ADDRESS>291

We should be good now

I found a way to fix the instant-reveal. The trick is to have a patch-method called before the constructor and forcing examined=true so a passthrough inventory controller is used where all the items are known and examined.

Not working on version <IP_ADDRESS>197

Not working on version <IP_ADDRESS>197

Yes because it is for <IP_ADDRESS>291 which is slightly newer. Have you updated SPT to the latest bleeding edge release?

While fixing the Wishlist-tracking, I spotted that they added an option in settings to display a little hideout icon on interesting items when looting:

And so we have easily access to everything needed to track dynamically all required items to improve the hideout.
I added a new options in loot:

And boom:

Added a spawnhi command as well, to spawn required hideout items:

Не работает на версии <IP_ADDRESS>197

Да, потому что это для <IP_ADDRESS>291, которая немного новее. Вы обновили SPT до последней версии?

I play on the pirate version of the game, maybe you can tell where you can download a version of the newer. Thank you!

Closing this one (but keeping the branch), the upcoming spt-aki 3.10 is now targeting <IP_ADDRESS>678


The README file in this repo has a bad link - [404:NotFound] - and missing image

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[X ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Bug

Which version of SkyWalking, OS and JRE?
N/A
Which company or project?
N/A
What happened?
In the main page for this repo (i.e the readme):

Status code [404:NotFound] - Link: https://camo.githubusercontent.com/20262efc5acf41a0c3e95d73835f73b40009696e81c274a3f6d81a928642c467/687474703a2f2f736b7977616c6b696e672e6170616368652e6f72672f6173736574732f75736572732d32303230303732362e706e67

Requirement or improvement
Both the image and link which it points too both needs fixing.
--
This bad link was found by a tool I recently created as part of an new experimental hobby project: https://github.com/MrCull/GitHub-Repo-ReadMe-Dead-Link-Finder
Re-check this Repo via: http://githubreadmechecker.com/Home/Search?SingleRepoUri=https%3A%2F%2Fgithub.com%2Fapache%2Fskywalking
Check all Repos for this GitHub account: http://githubreadmechecker.com/Home/Search?User=apache
--
I (a human) verified that this link is broken and have manually logged this Issue (i.e. this Issue has not been created by a bot).
If this has been in any way helpful then please consider giving the above Repo a Star.
If you have any feedback on the information provided here, or on the tool itself, then please feel free to share your thoughts and pass on the feedback, or long an “Issue”.

I could access the link. I am not sure what is your issue.

Hmm weird.
Maybe you are logged into https://camo.githubusercontent.com/ or are IP whitelisted (??)
I think this will affect most other people on the web too.
We'll see

Then the link will give this:


Upgrade babel 7

Use custom transform for jest to handle docs test since there is
path change in there
Motivation
#984
(Write your motivation here.)
Have you read the Contributing Guidelines on pull requests?
Yes
(Write your answer here.)
Test Plan

Go to v1 folder
yarn && yarn test

Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

@yangshun I just discover it happen because I had a build in the website directory previously.. Thats why the test seems fine it my computer. Will investigate tomorrow

@endiliey ok! let's do it in a different PR?


CPU Higher than 100% after execute profiler.startProfiling()

CPU Higher than 100% after executing profiler.startProfiling(), it won't back to 0% until profiler.stopProfiling() executed.

@3y3

https://github.com/nodejs/node-v0.x-archive/issues/25295
v0.12.5: 2015.06.22, Version 0.12.5 (Stable)

* openssl: upgrade to 1.0.1o (Addressing multiple CVEs)
* npm: upgrade to 2.11.2
* uv: upgrade to 1.6.1
* V8: avoid deadlock when profiling is active (Dmitri Melikyan)
* install: fix source path for openssl headers (Oguz Bastemur)
* install: make sure opensslconf.h is overwritten (Oguz Bastemur)
* timers: fix timeout when added in timer's callback (Julien Gilli)
* windows: broadcast WM_SETTINGCHANGE after install (Mathias Küsel)


[Doc] Mention that build command only works in tools-distribution

https://github.com/ballerinalang/ballerina/blob/master/docs/tutorials/create-archive.md

now build command works in runtime pack as well, below commit adds packerina jars to runtime pack
https://github.com/ballerina-platform/ballerina-lang/commit/a70e0bc
hence closing the issue.


Adding adaptive dark mode

Automatic Dark Mode has been added, with the setting depending on your base color scheme.

Additionally, I have implemented features to save and retrieve progress, along with an input range for adjusting font size.


Fail to execute the stored procedure query named with the existing function name

OS: Win10
Build Version: 20240112.17
Repro Steps:

Create a PostgreSQL Flexible server.
Expand the PostgreSQL Flexible server -> postgre database -> Create a function "one" for this database.
Create a stored procedure query with name "one" -> Execute this query.
Check whether succeeds to execute the stored procedure query named with the existing function name or not.

Expect:
Succeeds to execute the stored procedure query named with the existing function name.
Actual:
Fail to execute the stored procedure query named with the existing function name.

More Info:

This issue also reproduces for PostgreSQL Single server.
This issue also reproduces when executing the function query named with the existing stored procedure name.

Debugging the code reveals more details:

In PostgreSQL, both functions and procedures are different kinds of routines. If a function with the same name exists, the error message indicates that the new procedure cannot be created because by doing so, it would effectively change the "kind" of the existing function to "procedure". The error is expected.


Publish 09/23/2022 3:30 PM PT

Docs Build status updates of commit 4bb6a2b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/docs-conceptual/app-only-auth-powershell-v2.md
:white_check_mark:Succeeded
View (exchange-ps)

teams/teams-ps/teams/Get-CsTeamsEventsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Grant-CsTeamsEventsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/New-CsTeamsEventsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Remove-CsTeamsEventsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/Set-CsTeamsEventsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

teams/teams-ps/teams/teams.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


make clean fails: "rm: cannot remove"

What happened?
sriov-cni cannot be cleaned properly, running make clean yields a bunch of:
rm: cannot remove<EMAIL_ADDRESS>Permission denied
What did you expect to happen?
make clean should work
What are the minimal steps needed to reproduce the bug?
git clone https://github.com/k8snetworkplumbingwg/sriov-cni.git && cd sriov-cni && make && make clean
Anything else we need to know?
Due to recent move to go.mod

Confirmed.


Closes #2 Bootstrap integration

Description
This PR integrates the bootstrap js & css/scss libraries with webpack.
Issue
Closes #2

ok


No Intellisense or Autocomplete

Describe the bug
No intellisense/errors showing in vscode, everything builds and runs correctly otherwise. When using other packages, I cant see any of its types or methods.
To Reproduce
I have xcode 15 beta 2 and the extension downloaded.
Expected behavior
Intellisense or error feedback from vscode.
Environment

OS: MacOS Ventura 13.4.1
Swift version (output of swift --version)

swift-driver version: 1.82.2 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.10 clang-15<IP_ADDRESS>)
Target: arm64-apple-macosx13.0

Visual Studio Code version: 1.79.2
vscode-swift version: 1.3.0

Additional context

No error in ide

Should see methods like YAMLEncoder, YAMLDecoder, etc.

I get the same issue with 5.9 on macOS. Have you tried 5.8?

How do I switch my toolchain to my 5.8 on vscode?

Try using the Select Xcode toolchain command.

Ok yeah it worked. You have to download the latest version of XCode, not the toolchain. Then in vscode, Cmd+Shift+P and select Swift: Select Xcode Developer Dir and make sure its pointing to latest, not any betas.
Thank you @adam-fowler for the assistance 🙂


Update autorest.csharp to 20220601

API change check
API changes are not detected in this pull request.


Handle urllib3 timeout errors

This is an attempt to fix an issue where a successful HTTP request is made
but while reading the raw low-level response we may get a urllib3 error
raised instead of the corresponding requests error.
The error presents itself when sending/receiving large files to/from S3
with the AWS CLI when on a bad network connection.
This makes sure that our retry handler is capable of handling this case
and adds a test for it.
Traceback sample:
Traceback (most recent call last):
  File ".../site-packages/awscli/customizations/s3/tasks.py", line 347, in __call__
    self._download_part()
  File ".../site-packages/awscli/customizations/s3/tasks.py", line 376, in _download_part
    self._queue_writes(body)
  File ".../site-packages/awscli/customizations/s3/tasks.py", line 407, in _queue_writes
    self._queue_writes_in_chunks(body, iterate_chunk_size)
  File ".../site-packages/awscli/customizations/s3/tasks.py", line 438, in _queue_writes_in_chunks
    current = body.read(iterate_chunk_size)
  File ".../site-packages/botocore/response.py", line 70, in read
    chunk = self._raw_stream.read(amt)
  File ".../site-packages/botocore/vendored/requests/packages/urllib3/response.py", line 210, in read
    raise ReadTimeoutError(self._pool, None, 'Read timed out.')
ReadTimeoutError: HTTPSConnectionPool(host='s3.amazonaws.com', port=443): Read timed out.

It's difficult to reproduce the issue and test the fix. Any ideas you have would be appreciated.
cc @kyleknap @jamesls

Coverage decreased (-0.64%) to 93.87% when pulling 2e2fa6474808de3a21cf21baf0950b1b8bb1f37c on danielgtaylor:urllib3-timeout into 6a63e1884ad12e87479f454affbba5d5863c7f32 on boto:develop.

Have you tested this against the real service?  i.e a read timeout when reading from an s3 body during a GetObject (or even an aws s3 cp based on the traceback).  I'm 99% sure this PR won't do anything, as the retry logic happens within the scope of make_request and we're way past that by the time we're queueing writes to the IO queue in the CLI.

Closing, superseded by https://github.com/aws/aws-cli/pull/1191.  In general, the thing that makes the initial request and pulls from the stream is the thing that needs to own retrying the request on read timeouts.  In this specific scenario, the AWS CLI is where the fix needed to be made.

I am using the boto3 python sdk for executing the GET operation. As I learned, you have to explicitly call read to get the entire object.
I realized that read() does not throws ClientError as mentioned in botocore exception handling. So, I have to catch all exception since read() method urlib3 and other exceptions.
Is there a better way to handle such botocore exceptions.
Snippet of code:
`try:
response = client.get_object(
Bucket=bucket_name, Key=object_name)
response['Body'].read()
except ClientError as c:
logger.error("Encoutered a client error exception")
For catching response['Body'].read() exceptions.
except Exception as e:
logger.error("Encountered a general exception")
`
If I remove Base Exception block, I see ReadTimeoutError.
Traceback
File "/home/ubuntu/ObjectStorage/client-python/ost/client.py", line 124, in <module>     main()   File "/home/ubuntu/ObjectStorage/client-python/ost/client.py", line 99, in main     results = client.get(args.bucket, args.key, args.size)   File "/home/ubuntu/ObjectStorage/client-python/ost/provider/s3.py", line 176, in get     response['Body'].read()   File "/usr/local/lib/python3.5/dist-packages/botocore/response.py", line 74, in read     chunk = self._raw_stream.read(amt)   File "/usr/local/lib/python3.5/dist-packages/botocore/vendored/requests/packages/urllib3/response.py", line 258, in read     raise ReadTimeoutError(self._pool, None, 'Read timed out.') botocore.vendored.requests.packages.urllib3.exceptions.ReadTimeoutError: HTTPSConnectionPool(host='sthree-monitoring.s3.amazonaws.com', port=443): Read timed out


Add Mailer, Email, and MandrillAdapter

Still needs some work, but this is a good foundation to work off of

Added some minor feedback, but overall looked good to me.

@BlakeWilliams Thanks for the feedback! I'll tackle that after lunch :)


add transpose_value_cache

Description

1. Why the change?

2. User API changes

3. Summary of the change

4. How to test?

[ ] N/A
[ ] Unit test: Please manually trigger the PR Validation here by inputting the PR number (e.g., 1234). And paste your action link here once it has been successfully finished.
[ ] Application test
[ ] Document test
[ ] ...

5. New dependencies

[ ] New Python dependencies
- Dependency1
- Dependency2
- ...
[ ] New Java/Scala dependencies and their license
- Dependency1 and license1
- Dependency2 and license2
- ...

Please also update config.yaml

Please confirm these cases : True / False / no setting in config.yaml  if work normally.


TypeError: Cannot read property 'combine' of undefined

Hi, I installed 3.71, I generated an app, attempted to run it and got the error in the title. It seems 3.7.1 cli generates apps that point to winston 2.6.4 package but it needs 3.0.0 to make this error dissappear.

I also ran into this Winston v3 issue.
Yeoman "sometimes" installs as an npm pacakge a version other than the latest. I get around this with npm  cache clean --force.  Its very annoying.

No idea why npm keeps and/or Yeoman acting up about this but I locked the version to 3.0 now.


Add a dynamic LSH table

Excuse me， could you please show me how to add a dynamic LSH table？

I just want to insert or remove some record from the established LSH table

Hi hxwu,
updates are supported in some of the core classes, in particular the DynamicLinearProbingHashTable and the DynamicCompositeHashTable:
https://github.com/ludwigschmidt/fth_lsh/blob/master/lsh_table/src/probing_hash_table.h
https://github.com/ludwigschmidt/fth_lsh/blob/master/lsh_table/src/composite_hash_table.h
What we need next is a dynamic version of the StaticLSHTable in
https://github.com/FALCONN-LIB/FALCONN/blob/master/src/include/falconn/core/lsh_table.h
We should be able to re-use quite a bit of code here. The main difference is that the static LSH table always assumes that the key range is between 0 and data_set_size - 1. This allows us to simply keep a large array for checking whether a certain candidate already appears in the answer for the current query. If the key range is arbitrary, we need a less specialized set data structure here (a hash table would probably be the right thing). Overall it should not be a big performance issue, but it's good to get it right.
The code is organized so that the query objects in the LSHTable are only responsible for producing a list of candidates. The actual distances are then computed in the NearestNeighborQuery class:
https://github.com/ludwigschmidt/fth_lsh/blob/master/lsh_table/src/nn_query.h
Two questions about your application:

Are you planning to use the C++ or Python interface?
How is your dynamic point set stored? Would it make sense to always have the point indices in a small range (say 0 to some_constant * largest_point_set_size)?

Cheers,
Ludwig

Insert and remove record is very important in my application, I need to dynamic update LSH table, I use python interface, Do you have the plan to support it? @ludwigschmidt


Less icon: why the same as the CSS one?

Hi guy.
First of all, thanks for your great package, Atom is really more good looking with it! :)
But I don't really know/understand why the CSS and the Less icons are the same, the Less one being just a little darker. Could you explain it? Or better, can you change the Less icon with the Less logo :)?
Thanks for reading!

This would be why.
Horrible logo, details lost at icon-size, and overall just not recognisable...

OK, I understand now :x
Isn't it possible to get just the l of the Less logo?
Something like:
{l}
instead of:
{less}
?

I already tried that, and it wasn't really recognisable...

Sorry. You can always ask the Less guys to improve their logo. :D


logging: Add log backend lock

When SMP is in use the log backend should not be re-entered concurrently as it results in log output being mangled.
Imagine two cores with log immediate mode enabled where the log message is concurrently being processed by the backend.
Mangled output is highly likely as each backend is almost certainly a single byte stream being written to (uart, shared memory, dma channel, etc).

Feels like the ideal scenario on SMP would be either separate backend instances per core so they can internally deal with this
or
Some sort of mpsc queue setup with messages put in the queue, and a try lock around the processing loop over it.

I don't think this is needed for deferred mode as log is processed in a thread. Maybe #ifdef for immediate mode only?

I don't think this is needed for deferred mode as log is processed in a thread. Maybe #ifdef for immediate mode only?

In deferred mode panic mode induces the same behavior

RFC and DNM because its unclear this is the right spot to do this, at the moment the backends themselves are sort of responsible for dealing with the issue as can be seen in mtrace, adsp, and hda


Merge Latest Changes based on Linting

What does this PR contain?

@nkotzrenko This PR was created to bring the repos in sync

merged


Implement buffer pool to reduce memory use and allocations.

Implement a pool of buffers for eagerReader to reduce allocations and improve performance.  This reduces allocations to roughly (parallelism * targetFragSize), a few additional buffers may be allocated depending on the timing of each response.  Previously htcat allocated buffers equivalent to the size of the request.
To minimize changes to the programs overall structure it was required to provide the eagerReader a pointer to the buffer pool held by defrag in order to free its buffer on close.  HtCat's noParallel "short-circut" path or a non-200 status code from the initial response result in defrag holding a nil pool, therefore eagerReader can accept a nil pool and the pool checks if it is nil on calls to close - otherwise it will panic.

Sorry for the long delay. How much does it help?


fix typo

@javaDeveloperKid Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@pivotal-cla this is an obvious fix

@javaDeveloperKid This Pull Request contains an obvious fix. Signing the Contributor License Agreement is not necessary.


Convert rotate and de casteljau wasm functions to output svg

Refactor the rotate and de_casteljau_points wasm functions to output an svg (in the form of a string) for the interactive bezier-rs demo.

We should also be adding to the rotate svg the point around which we are rotating, but you'd need #792 to do it. Whichever one of you merges first should add this.

If this is ready for me to review, please request me as a reviewer and I can take a look. (I'm currently just going through the open PRs and figuring out which of them are in a state that's waiting on my review, and commenting on the ones that possibly might not be yet.)


fix ip_address error when destroying

1- When an interrupted destroy happens, there is a deadlock condition to resolve "ip_address" value, it happens because either 'google_compute_address.vault_ilb' or 'google_compute_address.vault' was already destroyed then this resource at the position 0 does not exist, locking any terraform execution from this state.
2- minor type fix for storage_bucket_force_destroy variable

Hi @masterleros thanks for the contribution! There was an underlying test issue that was causing your builds to fail. Could you please rebase on master and re-push your changes?


In volreader.vol3_generator(), add missing element of the yielded array.

One yield in vol3reader.vol3_generator() was missing an element in the yielded array.
This patch fixes it.

Cool! Thank you for your contribution!


Update AdvancedSessionsLibrary added server seamless travel

Added Seamless Server travel section to header and exposed function in library header/cpp

Duplicate pull request


net/http: occasional nil pointer dereference in (*Client).send

What version of Go are you using (go version)?
go 1.11 and 1.14
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
set GO111MODULE=
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\USER\AppData\Local\go-build
set GOENV=C:\Users\USER\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GONOPROXY=
set GONOSUMDB=
set GOOS=windows
set GOPATH=C:\go
set GOPRIVATE=
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=c:\go
set GOSUMDB=sum.golang.org
set GOTMPDIR=
set GOTOOLDIR=c:\go\pkg\tool\windows_amd64
set GCCGO=gccgo
set AR=ar
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set GOMOD=
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\USER\AppData\Local\Temp\go-build702297486=/tmp/go-build -gno-record-gcc-switches

What did you do?
Wrote a web crawler using net/http to crawl large networks with lots of different web services of all ages and technologies. The crawler mostly works great, but ocasionally I'm running into nil pointer dereference panics. I cannot reproduce the issue. Looking at the stacktrace it seems to be burried within net/http and independent of my code.
2020-02-29 18:52:11.810 +0100 	Critical 	Scan scope crashed unexpectedly: runtime error: invalid memory address or nil pointer dereference
Stacktrace:
	| goroutine 4239417 [running]:
	| runtime/debug.Stack(0xa1f, 0xc001186d00, 0x1d)
	| 	C:/Go/src/runtime/debug/stack.go:24 +0xa4
	| my-project/log.StacktraceIndented(0xbc8784, 0x1, 0x52, 0x0)
	| 	D:/go/src/my-project/log/logger.go:347 +0x3b
	| my-project/scans/webenum.(*Scanner).Run.func1(0xc000481440, 0xc002e0fdb0)
	| 	D:/go/src/my-project/scans/webenum/webenum.go:175 +0x10f
	| panic(0xb2a700, 0xa88be0)
	| 	C:/Go/src/runtime/panic.go:967 +0x16b
	| net/http.send(0xc000317500, 0xcc32e0, 0xc00107e600, 0xbf8ec712e19c8a1c, 0x67b0b803359d, 0x10b9800, 0xc0005fe868, 0xbf8ec712e19c8a1c, 0x1, 0x0)
	| 	C:/Go/src/net/http/client.go:271 +0x503
	| net/http.(*Client).send(0xc0002dabd0, 0xc000317500, 0xbf8ec712e19c8a1c, 0x67b0b803359d, 0x10b9800, 0xc0005fe868, 0x0, 0x1, 0xc0013f0710)
	| 	C:/Go/src/net/http/client.go:176 +0x101
	| net/http.(*Client).do(0xc0002dabd0, 0xc000317500, 0x0, 0x0, 0x0)
	| 	C:/Go/src/net/http/client.go:699 +0x451
	| net/http.(*Client).Do(...)
	| 	C:/Go/src/net/http/client.go:567
	|......

C:/Go/src/net/http/client.go:271 is where the actual panic araises due to a nil pointer dereference, by accessing .Body on the response of resp, err = rt.RoundTrip(req). Obviously, for some reason,  resp is nil, due to some unhanled error. So, execution continues until it ends up with a panic calling resp.Body. Here is a copy of the source code of client.go lines 252-275 for quick reference:
252:	resp, err = rt.RoundTrip(req)
253:	if err != nil {
254:		stopTimer()
255:		if resp != nil {
256:			log.Printf("RoundTripper returned a response & error; ignoring response")
257:		}
258:		if tlsErr, ok := err.(tls.RecordHeaderError); ok {
259:			// If we get a bad TLS record header, check to see if the
260:			// response looks like HTTP and give a more helpful error.
261:			// See golang.org/issue/11111.
262:			if string(tlsErr.RecordHeader[:]) == "HTTP/" {
263:				err = errors.New("http: server gave HTTP response to HTTPS client")
264:			}
265:		}
266:		return nil, didTimeout, err
267:	}
268:	if !deadline.IsZero() {
269:		resp.Body = &cancelTimerBody{
270:			stop:          stopTimer,
271:			rc:            resp.Body,
272:			reqDidTimeout: didTimeout,
273:		}
274:	}
275:	return resp, nil, nil

Maybe some package expert has an idea, why rt.RoundTrip(req) might return nil without error? Obviously, this edge case should not happen, because resp is required down the road...
What did you expect to see?
I'm expecting anything but panics. Either an error or a normal response.
What did you see instead?
A panic

I searched the last 7 days of my logs for web servers where my crawler ran into this issue. In total 890 web services got crawled. With 8 of them the crawler ran into described issue. I could not reproduce the issue, so I searched for hints/similarities. The similarities I found among all of the them, were:

They communicate via HTTP (not HTTPs)
Microsoft IIS web servers
They respond with "401 unauthorized" and requested NTLM authentication (WWW-Authenticate: NTLM)

(Sorry, can't share the hosts, they are within an internal network.)

As far as I can tell, this can only happen if rt.RoundTrip returns a nil *Response alongside a nil error.
Is your program using a non-default RoundTripper, for example by setting it in the http.Client's Transport field?

Hi,
we are:

configuring our own transport and client:

	return &http.Transport{
		TLSClientConfig:       utils.InsecureTlsConfigFactory(),
		TLSHandshakeTimeout:   timeout,
		ResponseHeaderTimeout: timeout * 2, // TCP connection succeeded, let's give the web app some more time
		DisableKeepAlives:     false,       // Keep-alive is required for NTLM authentication to work!
		Proxy:                 http.ProxyURL(proxy),
	}

	// Create and return fresh client
	return &http.Client{
		Transport: transport,   // HTTP transport to use, may be a reused one with alive connections
		Timeout:   timeout * 2, // TCP connection succeeded, let's give the web app some more time
		// CheckRedirect: -> default is nil, which tells it to stop after 10 consecutive requests
	}

Patching the CheckRedirect function to step through location redirects with our own code:

	// Patch redirect function to not automatically handle location redirects. We will take care of this manually
	// to give us more insight into what's happening, e.g. to count redirects and detect www-authenticate responses.
	client.CheckRedirect = func(req *http.Request, via []*http.Request) error {
		return http.ErrUseLastResponse // Return first response directly. Might contain location redirect.
	}

Wrapping the transport by ntlmssp.Negotiatior (for NTLM authentication). I think they are using the default roundtripper but doing some of their own roundtripping logic first. Now I'm thinking the problem could also be in (or in associateion with) their package. Though it is not a shady unknown one...

	negotiatedTransport := ntlmssp.Negotiator{
		RoundTripper: client.Transport,
	}
	client.Transport = negotiatedTransport

I added exhaustive logging to our code to find out how fare it executes in the flawed situations. I need to wait until i setp into the situation again, but then I should see better what happened, e.g. whether it made use of the ntlmssp negotiator already, etc...

Found this:
https://github.com/Azure/go-ntlmssp/issues/9
and
https://github.com/hashicorp/packer/issues/5865#issuecomment-554531786

Ok it seems to be a bug in go-ntlmssp and there seems to be a magic update: https://github.com/Azure/go-ntlmssp/pull/24
So I will update to that new version and hope the bug disappears. I guess you can close this ticket for now, I'd come back if the issue comes up again. Thank's guys for your time and help!

Azure/go-ntlmssp#24 seems to be fixing another instance of the problem described in #377.
Notably, golang.org/x/tools/go/analysis/passes/shadow/cmd/shadow detects the bug:
go-ntlmssp$ git status
HEAD detached at 4a21cbd
Untracked files:
  (use "git add <file>..." to include in what will be committed)
        go.mod
        go.sum

nothing added to commit but untracked files present (use "git add" to track)

go-ntlmssp$ shadow ./...
/tmp/tmp.pMusf64dqg/go-ntlmssp/negotiator.go:87:9: declaration of "err" shadows declaration at line 30

I suspect that that means other tools like staticcheck would also spot the bug.
So, at least we're aware of this class of bug, but it's a shame that the shadow check isn't in more widespread use.

Nice, I didn't know that tool. But also, we don't have build verification/automation yet (where it would fit into perfectly), as it would be overkill in our current state. Great to have that tool, but we can also assume that there will never be a majority knowing and applying a certain one. Best case, the language doesn't allow or mitigate it.  Otherwise, the first step is to know this kind of bug :) Anyways, not complaining...


Make Responsive Chart using ChartJS (for Mobile Phones)

There is a ChartJS canvas area and I've created the plugin text in the chart area. I want to make a chart as responsive to use mobile and pc. When I deploy my web page on the web browser, and If I do a zoom on my web page, the chart (ChartJS) can not be changed responsively. I want to use ChartJS intended for mobile phones and pc. For rending PC as well, in order to test when I zooming bigger on a web browser; the chart looks as though it is stuck in a tiny space. How can I overcome this problem? My HTML rendering tool is W3.
Sample My Chart:
My CodePen ChartJS

https://www.chartjs.org/docs/latest/general/responsive.html


feat(register): always resolve full file path for tsconfig and add paths if available

Addresses #626

 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.

@Brooooooklyn I'm not sure how these snapshots work. I added the snapshot file on the last commit and it failed as it was not used. Once I deleted it, now it's failing cause it's missing. Any idea how to resolve this? I do not get the error locally.

@Brooooooklyn let me know if this PR needs anything else or if I need to split this up into 2 PR's

for the paths updates
fully resolve tsConfig path


Need a configure file for a heterogeneous plant genome genome size is 600mb.

Actually i want to assemble  heterogeneous plant genome genome size is 600mb. but how to  configure the file for that.  Can you plz provide me a configure file with all best parameters. i go though all the tutorials but i didn't understands how to configure file. i run the with examples data its running fine but with my data I am getting error. i am using pb-assembly  installed with conda install -c bioconda pb-assembly   we have local server 2tb Ram  with 108 core and 4 CPU plz configure the all need computational need also.
Thanking you in advances

Closing due to lack of activity.


Update s3, sdk-core, sqs to 2.16.55

Updates

software.amazon.awssdk:s3
software.amazon.awssdk:sdk-core
software.amazon.awssdk:sqs

from 2.16.26 to 2.16.55.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

labels: library-update, semver-patch

Superseded by #169.


SyntaxError: Unexpected token <

I have added vue-bootstrap-typeahead to my project as follows:
npm i vue-bootstrap-typeahead --save
I have other components I am importing such as vue-bootstrap-datetimepicker like this:
import DatePicker from 'vue-bootstrap-datetimepicker'
However, when I try to import the vue-bootstrap-typeahead like this:
import VueBootstrapTypeahead from 'vue-bootstrap-typeahead'
I get the following error:
SyntaxError: Unexpected token <
I have also tried to use require but face the same error:
var VueBootstrapTypeahead =  require('vue-bootstrap-typeahead')
I haven't even added it to my components or to my template
Please advise why I might be having this issue.
Thanks.

What does your build chain look like? Are you using vue-cli?

I am using this project as my base: https://github.com/SimulatedGREG/electron-vue
This is the main.js where Vue is setup
https://github.com/SimulatedGREG/electron-vue/blob/master/template/src/renderer/main.js
I am trying to import the vue-bootstrap-typeahead on a Component vue
I'm not sure what the issue is as other packages are being imported fine in the same Component vue
I've also tried to import in the main.js to do Vue.use(), however, I get the same unexpected token error as soon as I import

I just setup electron-vue and installed the component and got it working without the syntax error. Perhaps upload your .vue file in a gist (along with the console stack trace) or paste it here so we can see what's going on a bit better?

Thanks for setting up the project and helping!
I've managed to fix the issue.
For reasons i'm not sure of (yet), I had to explicitly import the Component like this:
import VueBootstrapTypeahead from 'vue-bootstrap-typeahead/src/components/VueBootstrapTypeahead'

Still not working. Also I use @HasnatH but still its not working instead it says Module not found

I was getting the same error using nuxt in universal mode.
I solved creating a nuxt plugin with ssr disabled:
// plugins/autocomplete.js
import Vue from 'vue'
import VueBootstrapTypehead from 'vue-bootstrap-typehead'

Vue.component('v-autocomplete', VueBootstrapTypehead)

// nuxt.config.js
{
  plugins: [
    { src: '~/plugins/autocomplete', ssr: false }
  ]
}

Idk why it happens, but apparently this solve the problem.

For me, it happens when page is reloaded. ( I am using nuxt too )
hmm...

@danielbonifacio
Your solution does work, but please mind that there is a typo in your code, it's vue-bootstrap-typeAhead, not vue-bootstrap-typehead.
Otherwise, thank you for solving a major headache ;)

@danielbonifacio Does adding ssr false just mean the component won't show up when the server renders the component?

@tettoffensive It means that the component will be rendered only on the client-side. Nuxt will only have a reference to where it should be on the template, but will not mount or execute it. That's because a server rendering does not have some APIs, like document. They're available on the browser.

I managed to get the vue-typeahead-bootstrap component to render server side as well in a nuxt application by configuring the vue-typeahead-bootstrap component to be transpiled in nuxt.config.js:
{
  build: {
    transpile: [
      ({ isServer }) => 'vue-typeahead-bootstrap'
    ]
  }
}


fix title bar (looks like shit)

https://media.discordapp.net/attachments/1219852978442211338/1236835381454962798/image.png?ex=663a1cf6&is=6638cb76&hm=3c976979d29e659cbc93d8837b0e8ea24d73bbdc0c1aeba5e2c5d244662b192d&

remember when you had the dm icon (I think) set to the msn butterfly flapping? You should totally bring that back too!


Implement Hybrid API's In basic Form

Ketting,magma,mohist,Arc
Atleast so we can get mods version of hybrid so on

Redid the API setup to allow for registering API hooks, just need to implement them now.
And restructured things so that TaterLib only has to be on once side when there's a shared mod/plugin classpath, ie Arclight

Arclight's had some interesting stuff for registering Forge events from plugins, but I skipped that bit for now since TaterLib effectively does the same


Integrate daostar.org and daostar.one websites

Most of the activity & calls-to-action (as well as the working papers themselves) live on daostar.one, making daostar.org feel a bit dead and non-interactive. How can we improve that interaction?

Quoting Josh from Telegram:
Assigned to me

move all content from https://daostar.one (i.e. our Notion) to the Docs (i.e. how to join DAOstar, how to contribute, the team members), remove "Standard"


Added version_tree module to support versioning of arbitrary objects …

…in IPFS

Hey, I was working on the design notes and realized that it would be as much work to write the API up in English as it would be to simply write the code, so I just threw together a quick proof-of-concept of how this would work.  I still need to implement the method to get the chain of ancestors, which may mean some minor refactoring of the interface;  I'm also pretty sleepy and wrote it all on the fly in one pass, so there may be some minor refactoring to make the API simpler or more readable.  But the documentation and type-hinting is pretty thorough, so the general idea should be pretty clear.  The chain-of-ancestors implementation follows pretty trivially from the existing code.
I wrote the API to handle IPFS datatypes (bytes) instead of any specific OpenMined datatypes (Tensors, groups of Tensors), because I figure it'll be easy enough to write a thin wrapper or even just slightly refactor one of these methods to convert arbitrary data to bytes.
I'm planning to write some simple unit tests before opening a pull request into OpenMined, but please lmk what you think!


Getting Uncaught error when I include the px-datetime-range-panel

I am working with the PX Seed App.
I have included-
<link
rel="import"
href="../../bower_components/px-datetime-range-panel/px-datetime-range-panel.html">

in my component and then giving -
<px-datetime-range-panel
 time-format="hh:mm:ss A"
 block-future-dates="true"
 time-zone="UTC">
 </px-datetime-range-panel>

in my local dom. I am getting this error-

some how the module.exports? returns true and the program enters the require('monent') so replaced old one with module.exports=b(a.moment):b(a.moment) to resolve the error. What could be the reason for that, is it related to the gulp task somehow?

Hi, glad you have resolved the issue.
The fix seems to be in a library outside of Predix code.  Consider letting the code owner know and submitting a pull request.  Thanks!


Fix Script error, uniforms for Ai

Helicrew uniforms.
Infantry will have theirs uniform when entering a free vehicle.

Yes. But it is also ok to use the global (gCommon) in the event recievers.
Sent from ProtonMail Mobile
On tis, maj 7, 2019 at 20:15, joseffallman<EMAIL_ADDRESS>wrote:

@joseffallman commented on this pull request.

In Common/Classes/Handlers/CommonSquadHandler.sqx:

@@ -0,0 +1,80 @@
+/*

Name:	CommonSquadHandler

Date:	2019-05-05

Version: 1.0

Author:  Josef

Description:

Common squad handlers related methods

*/
+namespace Intrusion.Common
+{
public class CommonSquadHandler
{

private static fields ["_mUniformConfig" as UniformConfig];

Changed this to a UniformHandler, added as a property in IntrusionCommon.
Added static fields to ClientEventReciever and ServerEventReciever to hold the UniformHandler and make it accessible in those classes. Is this approach following the code guidelines?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Add support for deduplicating headings (#953)

Here's a proof of concept.  Also got rid what seemed like leftover some cruft and some strange work-arounds.

Codecov Report

Merging #1022 into develop will decrease coverage by 0.35%.
The diff coverage is 60%.

@@             Coverage Diff             @@
##           develop    #1022      +/-   ##
===========================================
- Coverage    89.75%   89.39%   -0.36%
===========================================
  Files           12       12
  Lines         2694     2697       +3
  Branches       461      462       +1
===========================================
- Hits          2418     2411       -7
- Misses         187      192       +5
- Partials        89       94       +5

Code needs some formatting and tests to get coverage up.  I may have jumped the gun on removing that type: ignore comment as well (though the PR is merged, so maybe it just needs a requirement update?).

@DylanYoung closing for now, feel free to reopen after making the requested changes.
Thanks!
~Timothy


Purpose of diagnostic logs?

What is the point of these diagnostic logs? Depends on the moment when you have collected diagnostic logs, the file size of the diag. log is around 600KB and 20% of that contains a real active client logs. Rest of the content (over 80%) is something like configuration details.
Using diagnostic logging we could see around 30 minutes back.
And I'm referring to the diagnostic log without any defination (e.g. ...calling, ....cdl, ....cdlWorker).
Can you also be so kind and share description of different diagnostic logging:
A) MSTeams Diagnostics Log <time>.txt
B) MSTeams Diagnostics Log <time>_calling.txt
C) MSTeams Diagnostics Log <time>_cdl.txt
D) MSTeams Diagnostics Log <time>_cdlWorker.txt
E) MSTeams Diagnostics Log <time>_chatListData.txt
F) MSTeams Diagnostics Log <time>_sync.txt

Thanks.

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

ID: 97d5e6d3-b7e0-d79f-0dea-ed423186a157
Version Independent ID: ec6e68aa-552f-ab45-3c8e-92eff526b173
Content: Use log files in troubleshooting Microsoft Teams - Microsoft Teams
Content Source: Teams/log-files.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@Petri-X Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hello @Petri-X How are you? Hope you are doing good. 👍

What is the point of these diagnostic logs? Depends on the moment when you have collected diagnostic logs, the file size of the diag. log is around 600KB and 20% of that contains a real active client logs. Rest of the content (over 80%) is something like configuration details.
Using diagnostic logging we could see around 30 minutes back.
And I'm referring to the diagnostic log without any defination (e.g. ...calling, ....cdl, ....cdlWorker).

It seems you refer here to logs.txt file, that is the Desktop log and as the article says:

Desktop logs, also known as bootstrapper logs, contains log data that occurs between the desktop client and the browser. Like media logs, these logs are only needed if requested by Microsoft.

(In my case the log file was much bigger.)
and

Can you also be so kind and share description of different diagnostic logging:

Those are the Debug logs and as the article currently says

These are the most common logs and are required for all Microsoft support cases. Debug logs are produced by the Windows and Mac desktop clients, as well as browser based clients. The logs are text based and are read from the bottom up. They can be read using any text based editor and new logs are created when logging into the client.

Debug logs show the following data flows:
Login
Connection requests to middle tier services
Call/conversation

Does that helps answering your question?
Thank you

No, I'm referring to the diagnostic logs:
By pressing Ctrl+Alt+Shift+1 I get this:

Funny thing is of course that in download folder I have six files:

So one file which I was downloaded, was actually divided into six different files. But name of those logs are still "MSTeams Diagnostic Logs", don't you agree?
But still, instead of the names: debug/diagnostic, the content and description is still highly appreciated.

@Petri-X
Yes, they are named Debug Logs in this article but the filename names them Diagnostics Log. We are talking of the same thing, and the description in the article that I shared here still applies.
If you would like an expanded explanation, other from what the filename states (chatListData, sync..) of each of them, We would need to ask @LolaJacobsen if she can get it.
Thank you

@get-itips
Yes we speak about the same files. Unfortunately Teams seems to be full of confuses. Would be nice to see that these could be consistent. :)
It would be awesome if you or your team could share a little more details how to use those logs in a best way and what we are expected to be found from the files. I have played with some of the PowerShell CMDLets to get data out from there. But wouldn't that be easier if you could share the tools and commands for all?
About the length of the log, do you agree that you do not see more than 60 mins back? I'm asking this, because if you do have any connections available, it would be nice if they (product group?) could get this extended longer time, like 48 hours back. And I mean the file which have no extra extension.

Checking with PM TJ Singh.

@CarolynRowe Hi Carolyn, any possible updates available?

Since this issue was opened, we have made several revisions to this file to clarify the usage. I have another work item open to clarify what we mean by debug files because what we output is different. Those changes should be done and go live over the next couple of weeks. Accordingly, I'm closing this issue. Thank you for the feedback and helping us improve our content.


Update expiry notice (EXPOSUREAPP-13111)

According to Figma the expiry notice was outdated, it is not really related to reissuance.
while expiring soon should have still the old text see Figma
Ticket

https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-13111
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-13113

According to Anna-Marias comment in https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-13111, this correction should be implemented. Further changes to be handled separately.


recursive re-listing of non-existent directory when file not found

Delete the home dir for a user, login to my home then repeatedly, recursively generates file not found and attempts to get to file.
Instead we need to substitute a file not found page (not message) when clicking on a subdir instead of resetting to a location, this can show in the main browser center pane and allow alternative navigation selections, perhaps showing a breadcrumb to the path they were trying to go to.

We don't worry about bad breadcrumbs at this point. Will worry about them in 1.1.


feat: read the TraceId from Header and propagate it

The goal of this PR is to add the capability to use interpret the W3C headers as specified here: https://www.w3.org/TR/trace-context/ and at the moment we dump them in the log, that according to the application.yml is sending them to the console.
Please note I might have added more than needed in the application.yml in particular regarding the health endpoints, if that is the case I will amend it.

Hi @lfdesousa! Thank you for this PR.
There was recently in IPF a move from Brave to Micrometer: https://github.com/oehf/ipf/commit/70d4cd739c5233ce8aa8683c8d111cdce5fa24ff
Does it impact this implementation? Should we do the same?

@lfdesousa will update the PR

I did an update on this PR, I also updated something related with ITI67, in particular RouteBuilder, there was an exception when the Mag was starting, though I will need that to be reviewed.

Is it ready for review, or have you planned other changes?
In the route builder, the custom indentation helps to see the underlying structure of the route. It was particularly helpful here because of all the branching happening in this route.

Let me just check the indentation and then it will be ready for review.

I updated the identitation in the Route Builder, it should be better now, other than that I do not have other stuff to add for now.

I quickly implemented the manual forwarding of the header to other outgoing messages, to be tested during the Projectathon: https://github.com/i4mi/MobileAccessGateway/commit/0ce2029f38c4f33aa1040d5ad44853d718827573
There is one feature missing in IPF, we can't set HTTP headers in FHIR responses yet: https://github.com/oehf/ipf/issues/458. Otherwise, it should be working.

Rebased and merged in #186


Freeing a Joint/Joint2D throws "Invalid ID"

Something I noticed, is when a Joint has rigid bodies as nodes a, and b, when freeing the joint, the physics server (PhysicsServerSW) throws "Invalid ID", I don't know what this will cause, but my other scenes seem to be unstable, joints randomly don't connect, and the simulation isn't consistent.
Bugsquad edit: Initial issue was fixed, but still doesn't work with Joint2D-s.

I think this remains true for Joint2D.

I can confirm what kakoeimon said. This should be reopened or a new issue created for 2D joints.

Reopening since it doesn't work for Joint2D-s (feel free to refer to df5fa62ab955437f05bfd7acdf6ed61d6d9c827e for pointers).

I confirm it doesn't work for PinJoint2D.
Here's a small demo scene to illustrate the problem.
PinJoint2D_Issue.zip
You can grab the box, node_a and node_b are set then when you drop it, it sets them to empty paths.
You can try and free them too (it's commented).
It basically throws the error and the physics for the box just stop working altogether.

Confirmed on 2.1.4 beta 3.
Some notes:

The exclusion on the joint is not cleared on release.
As workaround, remove the exclusion before removing the nodes from the joint.

pinjoint.set_exclude_nodes_from_collision(false)

The other errors on the console may not be errors, the joint should ignore if there is a missing element, maybe complain only if is invalid, but not null.

#10769 to the rescue!

Let's recapitulate.
#10769 is intended to fix the issues here for 2.1, and as far as I've been able to test, it does.
#10770 was the same for 3.0 master, which is already merged.
So, in 2.1 with #10769 merged or in master, is anyone still able to reproduce any of these issues?

@RandomShaper Tested your PR on 2.1 and I don't see any error with the sample project.

Thank you, @eon-s! I've removed the Needs testing tag from #10769.
@akien-mga, I'd say that PR is ready to get merged and this issue to be closed. Alas, the 3.0 variant didn't include the fixes annotation so manual closing is needed.

Fixed by #10770.


Hello, if I am using a dataset with a size of 512, can I still use your pre trained model?

请问一下，有没有复现SNUNet在levir数据集上实验呢，为什么我在levir上f1分数才69呢

Yes, I think you should be able to do that. But it might require good amount of GPU memory. I will try to pre-process the dataset to 256x256 images and then train on it and modify the inference code to get the final result on 512x512.
Another approach would be to down sample images by a factor of 2 and train on downsampled images. And during the testing, you can still give the downsampled image, but you can bilinear upsample the final probability map, then apply threshold to get the binary change mask. That should work fine as well.
I would try both these methods and check which one works best.

请问一下，你有没有复现SNUNet在levir上的效果呢，看文章中复现的f1分数有88，而我复现的只有70呢，望告知，谢谢！
------------------ 原始邮件 ------------------
发件人:                                                                                "wgcban/ChangeFormer"                                                                                @.>;
发送时间: 2023年11月22日(星期三) 晚上11:29
@.>;
<EMAIL_ADDRESS>主题: Re: [wgcban/ChangeFormer] Hello, if I am using a dataset with a size of 512, can I still use your pre trained model? (Issue #81)
Yes, I think you should be able to do that. But it might require good amount of GPU memory. I will try to pre-process the dataset to 256x256 images and then train on it and modify the inference code to get the final result on 512x512.
Another approach would be to down sample images by a factor of 2 and train on downsampled images. And during the testing, you can still give the downsampled image, but you can bilinear upsample the final probability map, then apply threshold to get the binary change mask. That should work fine as well.
I would try both these methods and check which one works best.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Hey, can you pls translate that to English? Sorry about that.


Training error: type object 'Trainer' has no attribute 'add_argparse_args'

When running the command bash test/integration_test.sh or the command python keypoint_detector/train/train.py I get the error message AttributeError: type object 'Trainer' has no attribute 'add_argparse_args'.
Error message:
Traceback (most recent call last):   File "/home/jarnboga/keypoint-detection/keypoint_detection/train/train.py", line 88, in <module>     parser = Trainer.add_argparse_args(parser) AttributeError: type object 'Trainer' has no attribute 'add_argparse_args'
I encountered this issue which reported the same error.

Tested with pytorch-lightning==1.9.0 instead of pytorch-lightning==2.0.0 and the error disappeared.

@jarnebogaert,  pytorch 2.0 and PL 2.0 have some breaking changes, most prominently the CLI for PL as you have encountered.
I'll lock the version of PL to 1.9 untill I have found some time to rethink the CLI of the keypoint detector.
Thanks for reporting (@lucasvandijck  also mentioned it already).


test: use kubectl helper for cilium cleanup in upgrade tests

Use kubectl helper for Cilium cleanup in upgrade tests instead of cleaning up manually. This deduplicates the logic and keeps it in a single place as a kubectl helper function.

test-me-please

retest-net-next

retest-runtime

retest-4.19

Marking this for backport for 1.9, as I have a PR that relies on this one to fix a CI flake which is currently quarantined on 1.9: https://github.com/cilium/cilium/pull/14186


Incorrect "result is not defined" error

Describe your problem and - if possible - how to reproduce it
When attempting to complete the lesson "Regular Expressions - Using the Test Method", I get the error "result is not defined" when submitting the following code:
let myString = "Hello, World!";
let myRegex = /Hello/;
myRegex.test(myString);
When i use this code in the Chrome console it evaluates to true.
Add a Link to the page with the problem
https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/regular-expressions/using-the-test-method
Tell us about your browser and operating system

Browser Name: Chrome
Browser Version: 66.0.3359.181 (Official Build) (64-bit)
Operating System: macOS High Sierra v10.13

If possible, add a screenshot here

@theChepito for the tests to pass the variable result has to have the expected value, you've removed it from the code completely

got it.  once i included let result= it passed.


Curation after sorting by group.

How can manual curation be performed after sorting by group using run_sorter_by_property?

Was this with a probe group?


added next/previous weekday functionality

I have added two functions which return e.g. next Monday, second next Monday, previous Friday etc.

I agree with @kmonsoor his understanding is exactly how I took it.

Well, Arrow is already using "weekday" to mean "day of the week", so the proposed naming here is just consistent with that. But perhaps find_weekday would be more appropriate, and it would allow a single function to supplant the proposed two. The weeks argument could then be either positive or negative. Also, the weekday argument should be an integer, not a non-localized string.


Bug: Trying to Load Disabled Rules Before Formatting Tags Causes Error

When format tags: [#aa, #bb]:

There is an error in the yaml:
unexpected end or the stream within a
flow collection (3:1)

Thanks for reporting this @computbiol , by chance do you have the rule to remove hashtags in place?
If you do, it may be that we do not currently support the above format for fixing invalid Yaml tags.

Here is the rule I am referring to: https://github.com/platers/obsidian-linter/blob/master/docs/rules.md#format-tags-in-yaml

Looks like the issue does still exist when I have the rule on. I will see what I can do about this.

Seems to be an issue with it running in the right order. I need to take a closer look at this which will likely be tomorrow. It looks like something is not being handled correctly as my attempts to console log values is not showing up, but that may be operator error.
I did notice that the rule for fixing Yaml values showed up before removing the hashtags which will be fixed in the next release, but that does not seem to be the cause of this issue.

The problem is the loading of disabled rules before removing the hashtags. It will likely need to be rewritten to handle the new order of events.

I think I have a fix, but I need to make sure it is a good option before proceeding with it. This issue should be fixed for next release.

Thanks for reporting this @computbiol . By chance do you have the rule to remove hashtags in place?
If you do, it may be that we do not currently support the above format for fixing invalid Yaml tags.

Thank you for your re[ly. It was working until today, but today there was a sudden error. The rule? This is the effect I want:
Example: Format tags in array

Before:

---
tags: [#one #two #three]
---
After:

---
tags: [one two three]
---

That makes sense. It is likely that you updated to one of the more recent versions of the plugin recently. The rule is working fine, but some logic was getting run before the rule got applied, so it hit the error you are seeing. This should be fixed on master in the next couple of days. It will likely be released next week since there have not been a lot of changes this week, and I don't have a lot of time to work on it this weekend.

That makes sense. It is likely that you updated to one of the more recent versions of the plugin recently. The rule is working fine, but some logic was getting run before the rule got applied, so it hit the error you are seeing. This should be fixed on master in the next couple of days. It will likely be released next week since there have not been a lot of changes this week, and I don't have a lot of time to work on it this weekend.

Thank you very much for your kind reply.

The changes have been merged into master. Please let us know if the next release or master does not fix your issue.
Sorry for the inconvenience that this bug may have caused.
If you would like to manually install master, you can follow the steps mentioned here: https://github.com/platers/obsidian-linter#manual-installation
Otherwise you may need to wait for the next release or use a plugin to pull master for you for the plugin (I think this may exist, but I may be misremembering).


Multi-select is not correct

VSCode Version: v1.7
OS Version: win7

Steps to Reproduce:

Open vscode

Input the contents:
Account
acb

select this character: "c", and click Ctrl + F, then you will see the issue. It should have 3 cursors, but it only have 2 cursors.

When do you fix this issue?

Duplicate of #6661


connect ha-bridge to homebridge

I wonder if its possible to connect ha-bridge to homebridge.
I would like to control homebridge devices with ha-bridge without manually re-adding them into ha-bridge.
so I would be great if there is a solution to directly connect this two programs.

This is a topic that has been touched upon. I have been looking into how this could work as the ha-bridge would have to talk to homebridge using the Apple HAP protocol. So, is there something that homebridge talks to that ha-bridge cannot?

the main reason I would like to connect this two is quite simple,
its way easier for me to connect my devices with homebridge - for example I have wlan leds, with homebridge I install the plugin type the ip into the config file and thats it. I don't know who to do that in ha-bridge.
second example, Homematic. on homebridge I install the plugin type the ip address into the config and thats it. every Homematic device is now working on homebridge.
in ha-bridge I have to manual add every function through http requests - as fas I know, maybe there is a easier, more automatic way.
i also want to use the homebridge function for apples HomeKit system - something ha-bridge doesn't offer (as far as I know) but I also want to control some devices through my Logitech harmony (something homebridge can't do).
for example I would like to enable or disable my Homematic light switch with my harmony but also want to have full control with apples HomeKit system.
i also don't want to have duplicated entries, the same in homebridge as in ha-bridge. the perfect way would be that theses two program could talk to each other for maximum connectivity.
I hope i made my point, ansonsten kann ich das gerne in deutsch ausführen ;-)

Those are all good reasons. I have been looking into this and it may come about.

This is also mentioned in #366

With the implementation of Mozilla IOT, Which has homebridge connectivity, I will suggest that is the path to implement this.
See https://iot.mozilla.org/


Validate domain file for repeat intents, entities, slots

sql for rasa-x does not like this. we should validate on training and also before upload to rasa x.

currently in rasa, you can have templates with no - text properly if they don't have any other properties, as they just get turned into text ones in the processing. But this throws an issue when loading the domain into rasa x: https://forum.rasa.com/t/error-while-running-rasa-x-command/10173
I think the better way to handle this is to just deprecate the ability to provide templates without the text attribute, instead of also working to handle this in rasa x

Why isn't rasa x just using the rasa way of loading the domain, normalizing it on the way?

@tmbo i have no idea, you'd have to ask whoever implemented that in rasa x

I guess that might be @ricwo

@tabergma this can be closed right?

closed via https://github.com/RasaHQ/rasa/pull/3601


github.com/gorilla has been archived

The Gorilla Web Toolkit has been archived. I think it'll be a good thing to look into how this will affect the various components with dependencies to it.

go mod why github.com/gorilla/mux
# github.com/gorilla/mux
github.com/dapr/components-contrib/middleware/http/routeralias
github.com/gorilla/mux

It is only used in the HTTP middleware router.
The Go SDK also uses it -- and the Go SDK itself is used in various Certification tests.


README.md collapsed example fix & bower install

Collapsed doesn't work without ui-layout-loaded attribute.
Also updated bower install instructions based on https://github.com/angular-ui/ui-layout/issues/183

Based on the comments in the uiLayoutLoaded directive we have to get ui-layout-loaded to broadcast  changes for it to work:
// Currently necessary for programmatic toggling to work with "initially" collapsed containers,
// because prog. toggling depends on the logic of prevButton and nextButton (which should be probably refactored out)
//
// This is how it currently works:
// 1. uiLayoutContainer in prelink phase resets @collapsed to false, because layout has to be calculated
//    with all containers uncollapsed to get the correct dimensions
// 2. layout with ui-layout-loaded attributes broadcasts "ui.layout.loaded"
// 3. user changes values of @collapsed which triggers 'click' on either of the buttons
// 3. the other button is hidden and container size set to 0


Quotes section hovering

Description
The quotes section when gets hovered on doesn't animate perfectly ,like the top and bottom protion of the box it is enclosed in is not displayed when it gets hovered
Screenshots

Any additional information?
No response
What browser are you seeing the problem on?
Chrome
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I have read the discussion tab thoroughly and got the project idea
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

@harshitcompcode Also remove the outline as it doesn't look good and add shadow on each card


Fixed broken links - 2024.10

What this PR does / why we need it:
This PR fixes broken links in the documentation.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

@gardener/chaos-engineering-maintainers Could I ask that you review this PR? Thank you!


Compatibility with 1.20.4 and respective versions of dependencies

Updated function for onMouseScrolled event implementation which accepts a third, unused parameter (amountX, for compatibility with newer versions of Architectury API). This is to prevent a crash when scrolling with the mouse wheel which was present when trying to use 2.0.8-beta.5 on MC 1.20.4.
Updated all build dependencies in gradle.properties to their latest 1.20.4 compatible versions.
Updated project gradlew to 8.7.

Jar files for this patch can be found here.


Improvement: allow key to be defined as concatenation of columns

Problem: Currently, the program only allows for a key to be defined using the first column. This does not allow for other types of comparisons that are useful.
Test case1: The below tables would not be able to be compared in the current program because it violates the unique key (apple is included 2x). If the program could concatenate columns 1 and 2, I could run comparisons on the excel.
User story: As an end user, I want to be able to compare spreadsheets by concatenating column values to make a unique key.

Col1
Col2
Col3

Apple
1
True

Apple
2
False

Cranberry
1
False

Comparison_long and Original_long are the excels associated with this issue. I still need to write unit tests to demonstrate this behavior with the test excels.