Graceful shutdown

Problem
I'd like to shutdown my application gracefully and do some database work before I exit the process myself.
However, Prisma exits the process on a SIGINT event on its own. Even though I got a work-around by monkey-patching "process.exit", the Prisma query engine still exits with the SIGINT event and I'm unable to execute queries therefore.
The Node.js process already received a SIGINT signal, therefore the Prisma query engine exited and your request can't be processed.
Suggested solution

Make prisma stop listening to process events by setting an option
Stop the engines manually by executing a stop method (?) just after the application is ready to exit

Alternatives
here's the very hacky work-around that I use (though I'd appreciate something else)
const realProcessExit = process.exit;
process.exit = () => {
  // Monkey-patch process.exit because Prisma wants to exit before
  // we can shut down gracefully
};

import { PrismaClient } from "@prisma/client";

["EXIT", "SIGINT", "SIGUSR1", "SIGUSR2", "uncaughtException", "SIGTERM"].forEach((eventType) => {
  const previousListeners = process.listeners(eventType);
  process.removeAllListeners(eventType);

  process.once(eventType, async (param) => {
    await shutdown();

    // Make prisma stop the engines after the application is ready to exit
    previousListeners.forEach((l) => l(param));

    if (eventType !== "EXIT") {
      realProcessExit();
    }
  });
});

I discovered the same problem today

Any news about this issue, I have the same problem

Thanks for reporting @StefanNemeth!
Your use-case makes a lot of sense and we should definitely support it.
Unfortunately there doesn't seem to be a mechanism for running the exit hooks in order.
If such a thing would exist, we could run the disconnect after your cleanup would have run.
I also don't see a way right now to remove our exit even listeners, as for "ordinary" users, this provides safety to make sure, that the query engine and database connection is stopped properly, because we can't rely on Node.js there.
Therefore, your suggestion about disabling the listeners makes sense to me.
Suggestion for configuration:
new PrismaClient({
  disconnectOnExit: false
})

Would this solve your issue @StefanNemeth ?
Does this make sense to add @matthewmueller @Jolg42 ?

Cleans up spawned processes when the parent process dies.

@timsuchanek it looks like execa deals with this same problem.
I did a quick search through their codebase and found the signal-exit module. I didn't spend enough time tracing the code, but I think this is what they use to cleanup a zombie child.
Using the reproduction you provided me.
index.js
const { spawn } = require('child_process')
const path = require('path')
const child = spawn('node', [path.join(__dirname, 'run.js')], {
  stdio: 'inherit',
})
setTimeout(() => {
  process.exit()
}, 100)

run.js
console.log('hao', process.pid)

setTimeout(() => {
  console.log('hi')
}, 100000000)

It appears that adding signal-exit to index.js will work:
const { spawn } = require('child_process')
const onExit = require('signal-exit')
const path = require('path')
const child = spawn('node', [path.join(__dirname, 'run.js')], {
  stdio: 'inherit',
})

onExit(function (code, signal) {
  console.log('killing child!')
  child.kill()
})

onExit(function (code, signal) {
  console.log('killing child2!')
})

setTimeout(() => {
  console.log('done')
}, 5000)

Two neat things about signal-exit from my testing:

You don't need to call process.exit() yourself. This may fix the problem described in this issue already.
We can recommend using this module to hook into the exit process. It calls call onExit callbacks.

One limitation is these callbacks must run within the same tick, so you couldn't have a setTimeout inside them. I think this should work for most cases though.

The suggested solution with disconnectOnExit: false however has the disadvantage, that child processes might get leaked.
Would a "before exit" hook like this work for you?
const prisma = new PrismaClient()

prisma.on('beforeExit', async () => {
  await prisma.user.deleteMany()
})

The suggested solution with disconnectOnExit: false however has the disadvantage, that child processes might get leaked.
Would a "before exit" hook like this work for you?
const prisma = new PrismaClient()

prisma.on('beforeExit', async () => {
  await prisma.user.deleteMany()
})

This would definitely work. Nonetheless, I'd personally prefer the disconnectOnExit approach because I don't think that it should be the responsibility of a dependency (Prisma) to execute the shutdown logic of the entire application.

Thanks for the feedback @StefanNemeth!
The reason Prisma is cleaning up its child processes, is this example:
index.js
const { spawn } = require('child_process')
const path = require('path')
const child = spawn('node', [path.join(__dirname, 'run.js')], {
  stdio: 'inherit',
})
setTimeout(() => {
  process.exit()
}, 100)

run.js
console.log('hao', process.pid)
setTimeout(() => {
  console.log('hi')
}, 100000000)

In this case, we'll end up with a dangling child process.
So unfortunately Node.js does not make sure, that the child process is killed properly.
That's why Prisma Client is cleaning this up, for now, to not introduce a foot gun for Prisma users.

The opt-out and callback on exit seems to make sense though. I just discovered that the reason my NestJs app isn't performing it's onApplicationShutdown cleanups are that Prisma is killing the process before it gets there.
I know this is slatted for current development, so wanted to get my thoughts in. Thanks!

Thanks for all the helpful comments!
We just fixed it and introduced the prisma.$on('beforeExit', () => ...) hook.
While we're still killing the child processes, after the beforeExit hook has been executed, we're not killing the current process anymore, if there are other listeners on the signals, like SIGINT.
The reason is, that if you provide a SIGINT listener yourself, you are responsible for exiting the process yourself anyway already.
This way you have more control. It should fix the issue you just described @dfee and also the one mentioned in https://github.com/prisma/prisma/issues/3773

I apologize for reacting to a closed issue, but I have to say I find the solution backward. It basically says we don't trust you for closing the client properly and you will end up with a leak. Let us disable the behavior and give us an example of how to do it properly.
The solution I have now is to stop Prisma from exiting by letting it wait on a promise in the beforeExit listener, and call the resolve of said promise once NestJS reaches the onApplicationShutdown hook of the Prisma client.

Hey @microwavekonijn, could you clarify the problem you're having?
To clarify the current behavior as I understand it, when the process receives a SIGINT:

beforeExit hooks are called (if defined)
Child processes (query engine) also receive the SIGINT and presumably exit
The node process will exit if there's no SIGINT handler, if there is a SIGINT handler, that handler will be called.

Does this behavior match your understanding? Do you have a problem with this behavior?

I use NextJS to manage the life cycle of my application. I want to start Prisma in the onModuleInit hook and close it in the OnApplicationShutdown hook. Before that shutdown hook is called other 'onApplicationShutdownandonModuleDestorycan be called, if I want to execute any queries there I can't as I have to add them to thebeforeExithook from Prisma. NestJS is the one that figures things out like what order to execute the hooks, but with thebeforeExit` hook this is not possible(or at least not easily done).
The simplest solution for me would be to disable the SIGINT listener from Prisma and when the application closes when it is Prisma's turn I tell it to $disconnect and wait for it to clean up. It is nice to have a helping hand to avoid issues like leaks, but it can also greatly complicate things.

Would listening for SIGINT only if there's a beforeExit handler work for you? Then if there's no beforeExit defined, it doesn't listen for SIGINT?

Would listening for SIGINT only if there's a beforeExit handler work for you? Then if there's no beforeExit defined, it doesn't listen for SIGINT?

Hi Matthew! Would this even work? How does the query engine know when to exit then?

All I want is to call $disconnect myself even in the case of a SIGINT, that way I have completely predictable behavior for my app.
I looked at the code and found that I can just returned a never resolving promise in the callback for beforeExit, blocking the SIGINT behavior.


Allow types.json to be packaged

Npm docs call install hooks an anti-pattern; one of the reasons is that they prevent a package from being bundled and subsequently installed on another system:
$ cat package.json
{
  "author": {
    "name": "Simon Ratner"
  },
  "dependencies": {
      "mime": "^1.3.0"
  },
  "bundleDependencies": [
      "mime"
  ],
  "name": "mime-test",
  "version": "0.0.1",
  "scripts": {
      "test": "node 'require(\"mime\")'"
  }
}

$ npm pack
mime-test-0.0.1.tgz

$ npm install --prefix tmp --production --no-rebuild-bundle mime-test-0.0.1.tgz
npm WARN package.json<EMAIL_ADDRESS>No description
npm WARN package.json<EMAIL_ADDRESS>No repository field.
<EMAIL_ADDRESS>tmp/node_modules/mime-test

# Note that the install hook is not run, due to no-rebuild flag

$ (cd tmp/node_modules/mime-test && npm test)

><EMAIL_ADDRESS>test /Users/simon.ratner/Code/node-mime-test/tmp/node_modules/mime-test
> node ./test.js

module.js:340
    throw err;
          ^
Error: Cannot find module './types.json'

This change uses a prepublish hook to generate types.json, and adds an empty .npmignore to ensure that types.json is packaged in the bundle (otherwise, npm uses .gitignore which excludes this file).

Yeah, I was sort of wondering about this. Thanks for the fix!   Published as v1.3.4


Add download link for OVA that is present on the security page - on main document

(https://learn.microsoft.com/en-us/azure/migrate/how-to-set-up-appliance-vmware#download-the-ova-template)
Please could you add the OVA download link to this front page?
Currently it can be found on the older security pages: https://learn.microsoft.com/en-us/azure/migrate/tutorial-discover-vmware#verify-security
Cheers

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

ID: 3627fc8d-b1f2-0d8e-958b-a6e3a95c939b
Version Independent ID: 47425ae8-9f63-046b-c1f0-daf64cb6734d
Content: Set up an Azure Migrate appliance for server assessment in a VMware environment - Azure Migrate
Content Source: articles/migrate/how-to-set-up-appliance-vmware.md
Service: azure-migrate
GitHub Login: @Vikram1988
Microsoft Alias: vibansa

@danielmarker
Thanks for your feedback! We will investigate and update as appropriate.

Requesting @molishv to look into this ask and take the action needed

Hi @danielmarker,
Thank you for your suggestion. This update is live at https://learn.microsoft.com/en-us/azure/migrate/how-to-set-up-appliance-vmware#verify-security.
Hence, closing this issue.

Changes added. #please-close


Is this dead?

Hi there,
As Rust std/core collections still don't support fallible allocations, this library is still quite important for the language ecosystem as a whole. With the rapid rampage of Rust use on embedded systems, it is even more critical to have collections with fallible allocation supported.
@vcombey It would be nice to have a status of the project, as well as if you need any help maintaining it.
Thanks in advance; this is an awesome library you have here!

Hi,
I've just released version 0.5.1, incorporating the recent pull requests.
Apologies for leaving some PRs unattended; they slipped my mind. If this happens again, please feel free to ping me.
Although I don't have much time for new development, I'm available for reviewing PRs and handling releases.
Though I don't feel entirely competent reviewing the most obscure PRs. If anyone is interested in helping out with those, I can add him as a maintainer

Hey @vcombey, I'll get in contact with you through email; maybe we can work something out to better structure the project and alleviate the burden of reviewing PRs.


Azure CLI tasks are far too verbose by default distracting from the actual task results

Type: Feature
Enter Task Name: AzureCLI
Environment

Server - Azure Pipelines

Agent - Private, both Windows-latest and Ubuntu-latest

If using Hosted agent, provide agent queue name:

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
By default, Azure CLI tasks produce a lot of verbose text including the subscription full details and other information that has nothing to do with what was actually written in the script for the task.
There does not seem to be a way to suppress that, so its very hard to follow what the task's actual output was.
Please consider moving it to verbose/debug text or allowing some flag to "quiet" the output.

It would be great if the log viewer would allow filtering by log level. Probably be a good idea if it would even filter out "info" or "verbose" by default too.

I agree with @ransagy. By default the task is way too chatty. Microsoft is normally in the habit of making things default to "only what you need to know", with options to increase verbosity. For the AzureCLI task I would recommend that the following steps in the task be suppressed under normal conditions:

az --version
az cloud set -n AzureCloud
az login --service-principal -u *** "--password=***" --tenant ********-****-****-****-************ --allow-no-subscriptions
az account set --subscription ********-****-****-****-************
az account clear
This would leave only the output of the command(s) specified in the task inputs to be displayed, by default. Then, add an input for the task that would allow for showing the output of the other steps that are built in to the task.

I've come across this issue too and attempted to change the code to facilitate hiding extra output: https://github.com/microsoft/azure-pipelines-tasks/pull/16701

Well, it still is a very annoying issue.

Anyone can review and approve the PR? Thanks.

If anybody can help, my PR #16701 seems to be stuck waiting endlessly for some CI tasks that I have no idea how to trigger, or if it even should be my action to trigger them. Any advice welcome.

Any updates on this issue? We could really use this in our IaC environment.

@vuylstekebram I have PR open,  #16701 , but can't seem to be able to complete it. There are build tasks marked as required, but I have no idea how to initiate/complete these tasks.

@v-ibshaik @bishal-pdMSFT Can anyone review or at least trigger the CI for that PR? Or assign to someone who can?

Commenting for updates - this would be a great fix

+1

+1

Mirroring a comment left on the PR mentioned - https://github.com/microsoft/azure-pipelines-tasks/pull/16701#issuecomment-2042407084 - you can control SOME of the output with env vars on the task and some clever use of the flags on each actual script command.
As said, more of a workaround and not a full one, but something.
Perhaps a start would be to remove or mute the az --version call, or at least make it adhere to the flag of the diagnostic/debug mode being enabled in the pipeline.

Great ! Thank you @ransagy


please link to "how to fido" in "Tutorials" section

https://github.com/fido-alliance/how-to-fido/blob/master/HowToFIDO.md
thx, =JeffH

Weird that I forgot to do that.
Fixed now. Thx for reporting @equalsJeffH


Code to reproduce NYU RGBD results / input pipeline

Hi,
thanks for your repo.
It would be nice if you could provide the code / the input pipeline which you used to run the NYU RGB-D experiments as well (similar to #4 ). To me it is not entirely clear, how you added the different modalities.
Best,

@HobbitLong As I don't know if there is any timeline on this issue and just to be sure to understand everything correctly to replicate the NYU RGB-D experiments the following steps would be required to replicate them wouldn't they?

Initialize the different models per modality

feat_l, feat_ab, feat_depth = model(inputs)

Create different contrast objects for the different pairs (e.g. in the core view scheme contrast_l_ab, contrast_l_depth)
2.1. This would enable something like

out_l, out_ab = contrast_l_and_ab(feat_l, feat_ab, index)
out_l_2, out_depth = contrast_l_and_depth(feat_l, feat_depth, index)

For each of the modalities one need to create a criterion object (twice for the core modality "L"). Which would lead to something like

l_loss_from_l_and_depth = criterion_l2(out_l_2)
depth_loss_from_l_and_depth = criterion_depth(out_l_2)

And then add the results from above to the general loss term.

Is this the correct way to go forward?
How exactly did you do the patched based training on the "L" modality? Could you provide some hyper-parameters for that?
Thanks for the otherwise very nice repo.

@meyerjo ,
To your question, we used "patch-based contrastive objective" for this task, please refer to section 3.5.2.
That being said, for each of the modality, we extract global feature as well as local feature. Then we contrast between (1)global feature from modality A to local feature from modality B and (2)global feature from modality B to local feature from modality A. The way we build this global-local contrastive loss is exactly the same way s DIM paper. The code for this loss can be found here.
The way of extracting global local feature can be found here.
And then we sum up those pair-wise loss as described in our paper.

@HobbitLong thanks for this additional info.
Just to confirm, the use of a patch-wise method here is only motivated by a small dataset?
I am also trying to find an explanation of the global/local feature concept in your CMC paper - is this a deviation from the method presented there?
Many thanks!

@jnyjxn ,
Yes, NYU dataset only has less than 2k images.
This is different from the ImageNet experiment, but has been described in the supplementary of the paper.


Fix pagination

Fix pagination
Refs https://github.com/remoteorigin/git-issues-downloader/issues/21

@jeremysuriel Excellent, could you please update the tests so we don't have failing CI?

All right, here we go 🎉


Matplot lib example - can't get StaticMeshComponent

Following your matplotlib example, after installing matplot lib to the embedded python in UE4.22, I can't get access to the static mesh component in the graph actor. Have followed as per the example but it's not working. Has the syntax changed on accessing the static mesh component from an actor?

Error in the console is:
unreal_engine.UObject' object has no attribute 'StaticMeshComponent
...
self.uobject.get_owner().StaticMeshComponent.OverrideMaterials[0].set_material_texture_parameter('Graph', self.texture)
AttributeError: 'unreal_engine.UObject' object has no attribute 'StaticMeshComponent'

In the example, the component was named 'StaticMeshComponent' but from the screenshot above it looks like you named it something else. This should be totally fine, you just need to change your code to use the same name.


Added check for getterParams is not undefined, to enable documented "…

…return null" feature
Added check for getterParams is not undefined, to enable documented "return null" feature as described in https://vuex.feathersjs.com/composition-api.html#options (before returning null in the computed threw an error because paginate couldn't be found on a null object)

Good catch.  Thank you!


Add func-go to repository config

Changes

:gift: Add func-go to repository config

Per our earlier conversation on #knative-functions channel. In case we want to enable syncing various common bits, inluding GH actions, owners/community files etc.
The team is already created per: https://github.com/knative/community/blob/main/peribolos/knative-extensions-OWNERS_ALIASES#L116-L121. To leverage the alias in owners.
We can use exclude lists to drop unnecessary updates as needed for the func-go.
/hold
Adding hold for evaluating if it's needed in the repository.
/cc @matzew @lkingland @matejvasek

/unhold


[1093] Add button back to site on all error pages

Context
https://trello.com/c/H0t2FXqL/1093-sign-out-link-is-missing-on-all-static-and-error-pages
Changes proposed in this pull request
Adds a button to all error pages, allowing the user to return back to the trainee list. If they're not signed in, then it will redirect back to sign in.

Guidance to review

Visit the error pages /404, /422, 500
Check that they all have a button returning you to the correct page

 Sign Out link is missing on all static and error pages


Suggestion: tell about how Database#each is promisified

The SQLite module documentation refers to the SQLite3 module documentation, which documents Database#each. After the definition of the latter, it's hard to guess how it is promisified in the SQLite module. It's rather mandatory to separately document this.

Agreed and as always PRs are welcomed

Yeah, I got fed up with figure out/get this to work myself and just switched back to SQLite3

Credit Space from the Discord.js discord server. As he put it: "promises can only resolve once, you have to pass a callback for the 'each' part, the promise resolves after all rows have been run iirc" because of this, it's kinda impossible to circumvent callbacks without rewriting sqlite3 itself.
I would have to imagine the promise doesn't resolve until the loop has finished calling the supplied callback for each of the rows found by the query.
db.each(SELECT * FROM table, function(err, row) {       if (err) console.log(err);       console.log(row);     }).then(function() {       console.log("finished callbacks on each row");     });

v4 now has this documented in the readme, thanks


Disk Cache Provider

Thanks for the contribution. I think the idea of persisting a cache to disk could be useful. I am not, however, a fan of this implementation. It looks like the in-memory cache provider, but modified to read from/write to disk on every operation. If that's the intended effect, it seems like a cleaner approach would be implementing dump and restore operations for the existing cache providers, and having custom application logic to read from/write to disk (or S3, or wherever).
In any case, the library lets you supply any cache provider you want, so this change doesn't have to land in the library for you to use it.


EEPROM: Fixes and enhancements

@OpusK and others,
This is another option to #59
This combined all of my EEPROM fixes from the other branch(s), which include:
a) Fix the PAGE1 issue which would cause us to write some stuff in bad page which caused us to have to recvoer the firmware to work.
b) More or less rid us of the VirtAddVarTab table, which was eating about 1K of user memory.
c) Limited number of variables we could store to 255. This was because the Pages that we write to are 1K and with 4 bytes per element we only had 256 minus the first one which had a page status value
d) Put check in to not allow us to write to random address (>= NB_OF_VAR).
e) Now since number of entries is < 256 and the EEPROM library only stores byte values I changed the code to save in two bye elements, which reduced how many times we had to copy pages back and forth.
f) If we do an EEPROM.write of some value and the EEPROM is already set to that value, don't write new entry for it.
Update lib_f103.a to these updates.
Note: I have another set of branches with this, but merged it all here to hopefully avoid the issue of needing to handle conflict of the lib_f103.a file

@KurtE,
Thanks for your contribution!
I think it would be better to first fix the bug with the 4-byte method.
Then how about discussing and improving on the EEPROM drvier's algorithm in a separate thread?

Either way.  The other pr has that as far as I know...

I made a replacement PR for #59 which is #62  which I merged into the changes to an currently up to date develop branch, I rebuilt the lib_f103.a file with these changes and create the PR, so there are no conflicts...
I did merge in one additional fix I had in the 2 byte fix, which was to the code that verified that a page was fully erased.

@OpusK  I  put in a new PR #64 which has these changes merged into develop after the merge, so should not have conflicts.  Also on the PR comments I put in my Arduino sketch in the off chance you want to play with it.
Probably can/should close this PR

@KurtE ,
Thanks for your contribution!


Rebase onto 0.36.0

I rebased, but it turns out there are no conflicts. So lots of things would have worked fine. You might prefer to merge from origin/master yourself.

Tests pass.


Gizmos circle_2d does not appear for the first few times it is requested after bevy startup

Bevy version
v0.14.1
What you did
Called gizmos.circle_2d and auto-captured screenshots for every frame for debugging a timing issue.
What went wrong
The circle_2d command (and potentially others, I only tried the one) did not draw for the first few frames I requested it. It looks like there is some sort of async initialization delay. If you warm start the gizmo by drawing an invisible circle_2d in a zero-alpha color immediately upon running, before requesting a circle in earnest, this problem seems to go away.
Without gizmo warm start:

With gizmo warm start:

Could you try with synchronous_pipeline_compilation enabled?


match time

There must be something wrong with the match time in 2015. Average is about half of the other years, only 54 minutes.

Duplicate of #178


test: Add testing to verify consistent generation for a no change update

… same config

Fixes #N/A
Description

As part of the conversion webhooks, it is possible to apply the same resource multiple times and have the resource generation increase each time. Adding testing to make sure the resource generation does not increases for applying the same resource multiple times.

How was this change tested?

/karpenter snapshot

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 79.261%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
91.67%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.01%

Covered Lines:
5981

Relevant Lines:
7546

💛  - Coveralls


SHM feature On NuttX

Hi,
I am confused about the share memory feature On Nuttx.

Can I publish a topic from program A and receive it both from program B and program C?
Should I create sessions/streams/entities/publishers/subscribers for each topic that I want to publish/subscribe to in a single program?

Thanks!

Can I publish a topic from program A and receive it both from program B and program C?

Yes if those programs share the same memory and address space. SHM in XRCE is implemented as a naive approach that does not take into account process memory protection.

Should I create sessions/streams/entities/publishers/subscribers for each topic that I want to publish/subscribe to in a single program?

One session can have multiple streams, and those streams and the session can handle multiple pubs, subs, and topics.

@pablogs9 Thanks for your reply!

One session can have multiple streams, and those streams and the session can handle multiple pubs, subs, and topics.

Are there any tutorials or examples about that?

You have all the available examples here: https://github.com/eProsima/Micro-XRCE-DDS-Client/tree/master/examples


Autoassess Stratosphere recipe and metrics documentation

Work on Autoassess recipes documentation:

stratosphere (this PR - Stratosphere will go in the Paper Part 2)
landsurface surfrad (later, in a different PR)
landsurface snow (later, in a different PR)

age of air works if the run is > 15 years; how correct, I dunno...I'll remove the reference but keep the code in the code base so we can review it next time
Yesh, I'll remove the MERRA pointers
:beer:

cheers a ton @alistairsellar :beer: :beer:


one way data flow problem?

this works
props: ['initialCounter'],
data: function () {
  return {
    counter: this.initialCounter
  }
}

this does not work
props: ['initialCounter'],
data:  () => ({
    counter: this.initialCounter //this.initialCounter is undefined
})

is this expected? I'm not sure about this

closing to open other the right way

@diego-lipinski-de-castro Please don't, you're falling into the fat arrow scope caveat. Search for it or ask in the Discord server, it's a well known JS caveat


Error installing from manager

ComfyUI-Elegant-Resource-Monitor install failed: Bad Request

I'm not able to reproduce this in a new comfyui installation.
If anyone else runs into this issue, please retry the installation -

delete the resource monitor custom_node folder,
click 'update all' in ComfyUI Manager,
and reinstall the plugin
close ComfyUi and relaunch

Ok, thank you.


The arrow indicates indent？

Problem Description
After i make the Indent Line visible
My tab is "...",but the code that i copy is "->"，if i want to make some change,it must be "...",and consle will show "IndentationError: unindent does not match any outer indentation level".
So I want to know if spyder have any function can change "->" to "..."
Thank you!
What steps reproduce the problem?
maybe this code can have "->"
class Button():
def init(self, screen, text, x, y, color, enable):
self.screen = screen
self.width = BUTTON_WIDTH
self.height = BUTTON_HEIGHT
self.button_color = color
self.text_color = (255, 255, 255)
self.enable = enable
self.font = pygame.font.SysFont(None, BUTTON_HEIGHT*2//3)
	self.rect = pygame.Rect(0, 0, self.width, self.height)
	self.rect.topleft = (x, y)
	self.text = text
	self.init_msg()

Paste Traceback/Error Below (if applicable)

Versions

Spyder version: 3.2.6
Python version: 3.6
Operating System name/version: win10

Hi @33RoboTcan, from your screenshots, it seems that your are mixing Tabs (shown as →) with Spaces, which is not allowed in any Python file (even outside Spyder). If you want to use Tabs instead of spaces to indent your code, you can set the option Tabulations in Preferences  → Editor → Source Code → Indentation characters, otherwise, you'll need to convert your tabs into spaces.


Migration to Qiskit Primitives

Summary
This PR aim to resolve issue #83.
Details and comments
File: test_braket_backend.py
Function test_vqe():

used from qiskit.algorithms.minimum_eigensolvers import VQE, VQEResult
substituted QuantumInstance with BackendEstimator
used estimator as argument for VQE

@IceKhan13 I have noticed something that might be unrelated to this issue. There is a problem in test_adaper.py to be specific function test_standard_gate_decomp(). I have made no changes to this function so far, and this error must have been present before this PR started. I will try to resolve the issue as part of this PR
For more info, you can see here

@IceKhan13 I have noticed something that might be unrelated to this issue. There is a problem in test_adaper.py to be specific function test_standard_gate_decomp(). I have made no changes to this function so far, and this error must have been present before this PR started. I have been working with Python 3.11
For more info, you can see here

@IceKhan13 I have noticed something that might be unrelated to this issue. There is a problem in test_adaper.py to be specific function test_standard_gate_decomp(). I have made no changes to this function so far, and this error must have been present before this PR started. I have been working with Python 3.11
======================================================================
ERROR: test_standard_gate_decomp (test_adapter.TestAdapter.test_standard_gate_decomp)
Tests adapter decomposition of all standard gates to forms that can be translated
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/robotastray/Desktop/github/qiskit-braket-provider/tests/providers/test_adapter.py", line 202, in test_standard_gate_decomp
    qiskit_circuit = qiskit_circuit.assign_parameters(parameter_bindings)
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/robotastray/Desktop/github/qiskit-braket-provider/venv/lib/python3.11/site-packages/qiskit/circuit/quantumcircuit.py", line 2762, in assign_parameters
    raise CircuitError(
qiskit.circuit.exceptions.CircuitError: 'Cannot bind parameters (0) not present in the circuit.'

----------------------------------------------------------------------

For more info, you can see here

The error with test_adapter.py is due to the following standard_gates:
XXMinusYYGate(Parameter("ϴ")),
 XXPlusYYGate(Parameter("ϴ")),

Note this is unrelated to this PR, it something I stumbled upon.
If one comments these gates out of the code there are no issues. I have commented  them out for now. Let me know how you wish to proceed regarding this.

Hey @robotAstray ! Thank for this PR!
hmm, something is wrong with this gate specifically. You are right test failing in main branch too https://github.com/qiskit-community/qiskit-braket-provider/actions/runs/5768152751/job/15638813727 and only for python 3.10, but not for 3.7 🤔
Give me couple of days, I will try to figure out what is going on, so we can remove commented gates from this PR.
As a heads-up PR looks great! :)

@robotAstray fixed failing tests in https://github.com/qiskit-community/qiskit-braket-provider/pull/118. Make sure to merge changes from main, uncomment tests and we can move with PR review!

Hello @IceKhan13,
Thank you for your message and help. I have completed the steps now.


Make Session a newtype

This is breaking change so remember to major bump this future me.
Closes #34

Excuse the CI, tests are passing locally


fix selection tray collider

Fix an issue with the selection tray prefab, seems to have been damaged somehow.

@mikeskydev , we have an (automatic) PR (#505 ) to update the github action we use for building, which is failing on a prefab error:
Start importing Assets/Prefabs/UIComponents/SelectionTray.prefab using Guid(9b674e23f212f5a40b5d981d3ba5f4e2) Importer(-1,00000000000000000000000000000000) Problem detected while importing the Prefab file: 'Assets/Prefabs/UIComponents/SelectionTray.prefab'.
The file might be corrupt or have missing nested Prefabs. See details below.
Errors:
	Restored Transform child parent pointer from NULL
 -> (artifact id: '16e0ea62ac9a5abac71aa778cef9da71') in 0.009217 seconds

Is that fixed by this PR?
I think it might only show up there because the Library/ cache is not valid and so it has to re-import everything. Maybe.

Yes! I did a fresh clone yesterday and noticed this same error, so it should fix it

Great, thanks! Merge away?


not able to initiate webcam for capturing image *

""looking for complete javascript code to access multiple camera drivers"""".
I am trying to use multiple cameras on web application . List of camera drivers should come in the drop down. I should be able to choose required camera driver from the drop down. Based on my selection, i will capture the photo. Whenver i change the drop down option, the corresponding camera driver should open."
CURRENTLY USING RTCMulticonnection for accessing multiple devices *

Here is a demo:

https://www.webrtc-experiment.com/DetectRTC/select-cameras.html

Demo's source code:

https://github.com/muaz-khan/DetectRTC/blob/master/simple-demos/select-cameras.html


Error at the final step

Hello,
I recently installed MicrobeMod (the pytest revealed no problems). I'm following the tutorial step by step with my data and I get a problem at what I think is the last step of the pipeline:
MicrobeMod call_methylation -b KHSMHJ_1_pod5.mapped.bam -r KHSMHJ_1_KG925.fna -t 30
24-12-11 14:43:18 INFO     Running modkit: modkit pileup -t 30 KHSMHJ_1_pod5.mapped.bam KHSMHJ_1_KG925_low.bed -r KHSMHJ_1_KG925.fna --only-tabs --filter-threshold 0.66
24-12-11 14:45:00 INFO     Reading Modkit table...
24-12-11 14:45:03 INFO     Retrieving sequences
24-12-11 14:45:08 INFO     Total number of sites with any possible calls (including low quality): 4380727 bases
24-12-11 14:45:08 INFO     Median coverage (stranded): 388.0x
24-12-11 14:45:08 INFO     Total number of sites with greater than min coverage (10x): 4380724 bases
24-12-11 14:45:09 INFO     Potential sites (>33%) for methylation type m: 72 bases
24-12-11 14:45:09 INFO     High quality sites (>66%) for methylation type m: 4 bases
24-12-11 14:45:09 INFO     Very high quality sites (>90%) for methylation type m: 0 bases
24-12-11 14:45:09 INFO     Potential sites (>33%) for methylation type 21839: 1578 bases
24-12-11 14:45:09 INFO     High quality sites (>66%) for methylation type 21839: 101 bases
24-12-11 14:45:09 INFO     Very high quality sites (>90%) for methylation type 21839: 0 bases
24-12-11 14:45:09 INFO     Potential sites (>33%) for methylation type a: 25943 bases
24-12-11 14:45:09 INFO     High quality sites (>66%) for methylation type a: 20944 bases
24-12-11 14:45:09 INFO     Very high quality sites (>90%) for methylation type a: 14245 bases
24-12-11 14:45:09 INFO     Running for Methylation type: 6mA
24-12-11 14:45:09 INFO     Modkit table size: 1892995
24-12-11 14:45:09 INFO     14245 sites for STREME with cutoff 0.9 and min coverage 10
24-12-11 14:45:10 INFO     Creating background control distribution: KHSMHJ_1_KG925_a_control.fasta
24-12-11 14:45:10 INFO     Running STREME: streme . --n KHSMHJ_1_KG925_a_control.fasta -p KHSMHJ_1_KG925_a_pos.fasta -o KHSMHJ_1_KG925_a_pos_streme
Traceback (most recent call last):
File "/zfs-hdd/Antony/miniconda3/envs/MicrobeMod_env/bin/MicrobeMod", line 220, in
microbemod.main(
File "/zfs-hdd/Antony/miniconda3/envs/MicrobeMod_env/lib/python3.9/site-packages/MicrobeMod/microbemod.py", line 557, in main
modkit_table_tmp = assign_motifs(
File "/zfs-hdd/Antony/miniconda3/envs/MicrobeMod_env/lib/python3.9/site-packages/MicrobeMod/microbemod.py", line 295, in assign_motifs
evalue = float(motif.get("test_evalue"))
TypeError: float() argument must be a string or a number, not 'NoneType'
Do you have any ideas for solving the problem? Thank you and have a nice day!

I think this is a streme version issue - what version are you using? Test it with streme --version.
If you're using an older version of streme than 5.4.1 or 5.5, try upgrading to those - if you're using a newer version, let me know the version and I'll see if I can update to work with it.

Hello,
I upgraded the streme version and the problem is solved. Thank you for this excellent tool and the quick response.


Some Feature Ideas and bugs for this client

Hello. I've been using and testing this client for about a day, and I really like it. It's a good example of how a program should be made. However, despite this, there are some problems, and I also have some feature ideas.
First of all, it appears to crash on almost a schedule. It crashes to home menu about every five minutes, so I have to re-start the program.
Now, some ideas. I would like it to automatically display some history, rather than having to press Y. It'd be nice to have a selector GUI on the bottom screen that allows you to tap, or at least see, the channels. Besides that, great project!

The random crashing doesn't seem to be affecting me, and plus it happens randomly, so this crash is going to be a tough one to fix. About the feature request, before a GUI like that can be made, a few things will need to be implemented to get that to work. Things like drawing and loading web image formats for it look the way I want it to.

Yes, I had thought of the loading server images. Also, I thought of some new key functions:
Dpad Up/Down = Scroll
A = send message
B = delete last sent message/select message to delete
X = edit last sent message/select message to edit
Y + Dpad Up/Down = Fast Scroll
L/R = Switch Server
Dpad Left/Right = Switch Channel
Select = Toggle Debug Mode (show bytes sent, etc., what is currently shown on lower screen.

I like most of what you are subjecting, but I would change
Switch Channel = Dpad Up/Down
Switch Server  = Dpad Left/Right
Select         = Touchscreen Tap     # messages will also be on the bottom screen
Scroll         = Touchscreen Swipe   # like on a smartphone
Delete or Edit = Swipe Left or Right or Long Press on Touchscreen

My plans are a bit harder to implement then what you are asking tho, I'll consider your as the first version.


"Off" icon (line from upper left to lower right)

I would like to overlay other icons, which don't have an "off" state, with this "off" icon, so I can visualize any icon as "not on".
My usecase: the priority_high icon
Alternative: an "no_priority" icon which is a strikethrough exclamation point.

I helped myself and created an SVG, which I can use as an Angular Material Icon, to strikethrough other icons.
If anyone is interested, here's the SVG (you have to define a CSS variable with the current background color, to support different themes):
<svg width="24" height="24">
  <g>
    <rect fill="var(--my-background-color)" x="2" y="-1.5" width="23" height="2" transform="translate(3.5, 2) rotate(45)" />
    <rect x="2" y="0" width="23" height="1.75" transform="translate(3.5, 2) rotate(45)" />
  </g>
</svg>


feat: add javascript to emmet_ls

Adds  javascript support to emmet_ls as it was missing and emmet_ls implemented support for it (as seen in https://github.com/aca/emmet-ls#supported-filetypes)

You could manually select filetypes during setup for the language server, like so:
require('lspconfig').emmet_ls.setup({ filetypes = { "css", "eruby", "html", "javascriptreact", "less", "sass", "scss", "svelte", "pug", "typescriptreact", "vue" }, })  I'd be fine with reversing it though, since I realize most people who would want to use emmet would be working with .jsx most likely.

Oh, thanks, I didn't think of that. But I believe many people find this annoying. What even is the use case for this in JS?

@pedrobl85 Could you please do this?


RequestTimeTooSkewedException on backup

I get this when I try to run a backup.
[Aws\S3\Exception\RequestTimeTooSkewedException]
The difference between the request time and the current time is too large.
From start to finish, it takes about 10secs to make the commands.
The gzip'd sql files are only 8KB.
My VM is running a different timezone to that of my project, if that makes any difference.
	vagrant@homestead:~/www/ag-aus$ php artisan db:backup
	These arguments haven't been filled yet: database, destination, destinationPath, compression
	The following questions will fill these in for you.

	Available database connections: sqlite, mysql, pgsql, sqlsrv
	From which database connection you want to dump?mysql

	Available storage services: local, s3, rackspace, dropbox, ftp, sftp
	To which storage service you want to save?s3

	How do you want to name the backup? mysql-backups/ag-aus-local.sql

	Available compression types: null, gzip
	Which compression type you want to use?gzip

	Just to be sure...
	Do you want to create a backup of mysql, store it on s3 at mysql-backups/ag-aus-local.sql and compress it to gzip?

	Are these correct? [Y/n] y
	Dumping database and uploading...

	  [Aws\S3\Exception\RequestTimeTooSkewedException]
	  The difference between the request time and the current time is too large.

	db:backup [--database[="..."]] [--destination[="..."]] [--destinationPath[="..."]] [--compression[="..."]]

From my logfile:
[2015-02-02 14:00:18] local.ERROR: Aws\S3\Exception\RequestTimeTooSkewedException: AWS Error Code: RequestTimeTooSkewed, Status Code: 403, AWS Request ID: ABE3226CE02932DC, AWS Error Type: client, AWS Error Message: The difference between the request time and the current time is too large., User-Agent: aws-sdk-php2/2.7.17 Guzzle/3.9.2 curl/7.35.0 PHP/5.5.15RC1 [] []

Ok, so I added a script to my Homestead provisioning to set my timezone correctly and setup local and production timezones in my project and it works fine.
However, the production server will be located in the US, but running my local timezone NZ and the application timezone will be AUS, so this will still be an issue in production.
It seems that the command to backup to S3 takes the time from the application and the time from the environment, somehow. Could the the backup-manager be modified to only use one timezone for its transactions, rather than both?

Hmm. I don't use homestead but I've never run the backup-manager without being inside a Vagrant VM. My intuition is that this is a configuration problem.


Deleting Generated Files

When the user quits the program there should be a dialog prompt asking if they want to save the files that have been downloaded during the session.  These file paths should be stored in a list in Global_Settings.py.  So every time a .fasta, feature_table, .gff, is downloaded or a .cspr file is generated in NewGenome or in Cotargeting, the file path should be added to the Global_Settings.py list.  The dialog prompt should give them the option of selecting and deselecting files that the want to save and files that they want to delete.

With my latest push this enhancement is completed.
Once the program closes the program, a little window opens up that lists all of the files in the CSPR_DB folder. User selects whichever ones they want(multiple or none) and hits submit. Files selected are deleted. If no files are selected, the program just closes without doing anything.
Currently this code is called from 4 classes: mainWindow, annotationsWindow, results, and multitargetting.


Update Remapper class with new _dict_to_correct method

Description:
I have added a new method _dict_to_correct to the Remapper class in utils.ipynb`. This method creates a dictionary for manual corrections of fuzzy matches.
Changes made:

Added new method _dict_to_correct to the Remapper class
The method returns a dictionary with:

Keys: source_name from each Match object in self.lut
Values: Empty strings, ready for manual input of corrections

Removing this for now.


Propeller doesn't unfeather on startup, stays in the yellow band

Current Version
Default/Mod
Issue Description
During the engine startup sequence, the propeller doesn't come out of the feathering range (yellow band) even after the NG > 52% and ITT is within the green limits.
Post startup, on flight idle, it stays at ~500 RPM in the yellow band which is unrealistic.
Expected Behaviour
Given normal operating parameters, second set of ignition should start at NG > 52%, starter EICAS message should disappear and the propeller should start unfeathering, its RPM should increase and stabilize above the yellow band.
References
https://www.youtube.com/watch?v=25xlVkuEYdA
https://www.youtube.com/watch?v=V-z4AkMWXU4 (from 4:40)

I know that, and I already posted that on the forums:

That's something I spent a hole night trying to fix and couldn't.

Noted!


Investigate and compare against JDK7 sorting algorithms. [HPPC-33]

JDK7 introduces new sorting algorithms: dual pivot quicksort and "tim's merge sort". Evaluate on synthetic data and see what the difference is.
http://gdtoolbox.com/DualPivotQuicksort.pdf
–
I created a benchmark against sun-jdk1.7.0_ea_b104. The benchmark tried to compare "new" vs. "old" implementation of Collections.sort() first; on nearly-sorted strings, representing integers from 0..1000000:
        benchmark    ms linear runtime
       LegacySort  39.5 ===
          NewSort  25.3 ==

this turns out really well compared to IndirectSort:
        benchmark    ms linear runtime
IndirectMergeSort  63.7 ======
IndirectQuickSort 313.3 ==============================
       LegacySort  39.5 ===
          NewSort  25.3 ==

but there is a catch: that was nearly sorted data. The same benchmark, but executed on 1.6_20 shows hotspot improvements have been made as well, not only algorithmic improvements (I assume LegacySort didn't change between 1.6 and 1.7; haven't verified this):
        benchmark    ms linear runtime
IndirectMergeSort  63.5 =====
IndirectQuickSort 317.8 ==============================
       LegacySort  53.3 =====
          NewSort  53.2 =====

Very similar results from jrockit:
        benchmark    ms linear runtime
IndirectMergeSort  69.6 =====
IndirectQuickSort 396.9 ==============================
       LegacySort  60.5 ====
          NewSort  60.2 ====

vm: c:\java\jrmc-4.0.0-1.6.0\bin\java.exe

The second benchmark changed the scenario: I randomized the input so it wasn't nearly sorted and I changed Collections.sort() to Arrays.sort() calls to run on Integer[] arrays with a custom comparator, simulating IndirectSort's functionality. This yields, from 1.7:
        benchmark   ms linear runtime
IndirectMergeSort  785 ====================
IndirectQuickSort  918 =======================
       LegacySort 1156 =============================
          NewSort 1163 ==============================

Ok, much more sensible.
Conclusions: if data is nearly sorted and can be reordered, stick to Collections.sort(). If data is randomized and you don't want to reorder the input (for whatever reason), use IndirectSort.

Issue: HPPC-33 (migrated from JIRA), created by Dawid Weiss (@dweiss), resolved Mar 14 2011

Comment by Dawid Weiss (@dweiss) (migrated from JIRA)
Investigation completed. I don't see a reason to change IndirectSort for now.


intensive care

can one safely assume that on a specific day:
num_intensive_care = ncumul_ICU + ncumul_ICU_intub + ncumul_ICF + ncumul_vent
is this correct?

Hi @cybermax, ncumul_ICU should be the column containing the total number of patients in intensive care (with or without ventilation / incubation). This column is already available in the merged swiss file. The same stands for the ncumul_vent, which should contain the number of patients requiring ventilation. In a next step, we want to include the column for the intubated cases in all files as well (this column is available for selected Cantons only at the moment). We will make some changes shortly, so it might be better to wait before using the columns beyond the total ICU column.
We still need some feedback from the Cantons that report these numbers and what the definitions are, see : #215
As soon as we are ready to make the needed changes we will comunicate this actively.
https://github.com/openZH/covid_19/blob/master/COVID19_Fallzahlen_CH_total.csv

A significant share of patients in ICUs appear to be intubated (or 'künstlich beatmet'):
Currently in AG 9/10, TI 51/61 , GE 48/52
So i ask myself how relevant it is to even include 'ncumul_vent' and 'ncumul_intub'  besides the 'ncumul_ICU' column in the merged Swiss file. If we can't get precise definitions of what is meant by 'künstlich beatmet' or 'intubato' in the different Cantons, we can't grant comparability and cannot judge if we should count cases as 'ncumul_vent' or 'ncumul_intub', or if the reported number contains both cases.
One option would be to could exclude those detail-columns from the merged Swiss Dataset alltogether and just leave them in the Cantonal-files, while highlighting in the Readme, that only the columns in the Swiss file fulfill the criteria of good comparability.
I'm still trying to get the definitions from the competent authorities... but i guess it will not be that easy.
However, i'm really not an expert in this domain, so i'd be very interested to hear your opionion on this @SarahNadeau .

To be fair, I am absolutely not an expert in this domain either! For now, I really like the idea of having the cantonal data as descriptive as possible and at the national level aggregating only the fields we are confident are directly comparable. This assumes that at least data from the same canton is comparable day-to-day and allows us to present everything the cantons make available.
More importantly for the long term, is it feasible to ask for changes in the data that is reported? E.g. can we ask the Krisenstäbe/ hospitals to adhere to particular definitions of ventilated and intubated patients? This question is going in the direction of whether the OGD team are just data scrapers or also involved in data generation.

On 27 Mar 2020, at 15:19, Thomas Lo Russo<EMAIL_ADDRESS>wrote:
A significant share of patients in ICUs appear to be intubated (or 'künstlich beatmet'):
Currently in AG 9/10, TI 51/61 , GE 48/52
So i ask myself how relevant it is to even include 'ncumul_vent' and 'ncumul_intub' besides the 'ncumul_ICU' column in the merged Swiss file. If we can't get precise definitions of what is meant by 'künstlich beatmet' or 'intubato' in the different Cantons, we can't grant comparability and cannot judge if we should count cases as 'ncumul_vent' or 'ncumul_intub', or if the reported number contains both cases.
One option would be to could exclude those detail-columns from the merged Swiss Dataset alltogether and just leave them in the Cantonal-files, while highlighting in the Readme, that only the columns in the Swiss file fulfill the criteria of good comparability.
I'm still trying to get the definitions from the competent authorities... but i guess it will not be that easy.
However, i'm really not an expert in this domain, so i'd be very interested to hear your opionion on this @SarahNadeau https://github.com/SarahNadeau .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openZH/covid_19/issues/260#issuecomment-605025966, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHH5AME3Y3D6I6B5FNZRARDRJSYW3ANCNFSM4LU46ICQ.

Definitions i've received so far:
TI just reports intubated (as intubato)
ZH reports intubated and ventilated together (künstlich beatmet)
... further will follow.

AFAIK. ncumul_ICU_intub (or ninst_ICU) and ncumul_vent are already included in ncumul_ICU.

Thx for tracking and contributing to this discussion on definitions @baryluk besides all the scraping / technical ones. Indeed, the total number of patients in ICU (ncumul_ICU) already gives a very good idea of how many patients need ventilation / intubation, as most of the ICU-cases do. However, we will try to clarify how to handle the additional information on how many ICU patients need which kind of ventilation in a consistent way within the next days (still waiting for some answers...).

After extensive research and having received feedback by several Cantons we have confirmation that the numbers they report focus on invasive ventilation. The figures seem thus to be consistent.
Shortly, we will adapt to this. The intub columns in the files for GE and TI will thus become obsolete, the respective values will be stored in the ncumul_vent-column. If selected Cantons will start reporting figures on non-invasive ventilation we can then still add an additional column in the Cantonal files. However, this seems not to be likely to happen at the moment.
@baryluk @metaodi @SarahNadeau

@tlorusso That make sense. Thanks for a work on this.
This is really good to converge on common definitions. Do we have some place to track if reported ventilation numbers are inclusive of mechanical ventilation and non-invasive respiration aids for each canton?

Practice has been to ask whenever a Canton publishes data on this. So far those who report these numbers seem to count invasive-respiration aids. At this point we just mention in the definition in the Readme that the data is based on invasive ventilation. But if further Cantons start publishing we might turn to them as well to ensure that the data is comparable, if not we might have to create an overview and declare differences. @baryluk

i'll close this issue for now - we can reopen it if differences in counting show up.


x/build: cmd/cl does not build

golang.org/x/build/cmd/cl does not build, as its import "golang.org/x/build/godash" does not exist.
remove it?
cc @bradfitz @andybons @quentinmit

Dupe of #21216
Someone is working on fixing it.


[C++/WinRT] Make readonly properties readonly

Today, wil::single_threaded_property<T> exposes operator<Q>()(Q&&), which means that it can be used for {get; set;}; properties. This change removes that operator, so only wil::single_threaded_rw_property<T> can be use for settable properties.
This is a breaking change.
What changed?

Removed operator()(Q&&) overload.
Added comment explaining this.

How tested?

[ ] Tests pass
[ ] Calls to MyProperty(42) no longer compile unless _rw_ properties are used.

Fixes #353


Replace a hacky (and now broken) client-side test to allow connecting to external servers.

This assumption was broken with 5470f0e613bad359c32d50d3da3f04eec47c9a82. It resulted in an IllegalStateException from trying to grab the Server below whenever someone tries to connect to an external server. This PR fixes that.

Thanks!


Update app btns & QR code

Changes

added white icons in light mode
extracted qr and playstore btns in a component
used component in deposit dialog and in profile manager

Related Issues
#3648
Tested Scenarios

in light and dark mode do the following:
- clicked on playstore and app store btns, see that they redirect to correct threefold connect play store and app store pages respectively
- open up threefold connect app, go to wallet, send page, scan qr code see that it adds correct twin id

Checklist

[x] Screenshots/Video attached (needed for UI changes)

not sure if we should include the twin page qr code as well,


Resetting cross-endorsements or dossing between switches feels clumsy

The keybinds for cross-endorsement (Shift+E by default) and for dossing nations from an activity feed (Shift+D by default) take the user through the nations they need to endorse or dossier, and the user knows when they've endorsed or dossed all the nations they need to when the keybind no longer responds to input. (Yeah...)
This feels clumsy because, among other reasons, there isn't a clear way to reset this between switching. To get an updated list of nations from the same activity feed, the fastest method feels like going back through previously visited pages (and hoping you go back the right number of pages) or just bookmarking the activity feed.
It should be possible to save the URL of the activity feed used when generating the list of nations to endorse or dossier, and send the user back to that when they're done crossing/dossing or when they press a different keybind.

The use-case here is for chasing, right? I'm just pondering what the "least surprising UX" is.
I intuitively find your suggestion the most sane: the final keypress simply takes the user back to the initiating page with a popup saying "All nations endorsed/dossiered" (which could be triggered by URL parameter, i.e. "gauntlet-message=all_endorsed" added to the query string).
I can imagine situations where resetting the cross state on any other keybind leads to surprises. For example, suppose you begin crossing, then remember "hey I might have updated", press U, see it's all swell, then want to continue where you left off. Clearing the state could be restricted to detecting a nation switch as well as when pressing Shift+E/D on an initiating page (i.e. activity feed), and in other cases it will continue where it left off.

Yeah, aside from chasing, I can't think of other use-cases right now. I
wasn't sold on the UX aspect either, though I do like the idea of a pop-up
(or perhaps just a notification so dismissing it doesn't require a
keypress). Detecting when the logged-in nation has actually changed is a
good point as well and could probably also be used to exclude the currently
logged-in nation from the list of nations to cross-endorse.
On Thu, Jan 19, 2023 at 11:41 PM Roavin @.***> wrote:

The use-case here is for chasing, right? I'm just pondering what the
"least surprising UX" is.
I intuitively find your suggestion the most sane: the final keypress
simply takes the user back to the initiating page with a popup saying "All
nations endorsed/dossiered" (which could be triggered by URL parameter,
i.e. "gauntlet-message=all_endorsed" added to the query string).
I can imagine situations where resetting the cross state on any other
keybind leads to surprises. For example, suppose you begin crossing, then
remember "hey I might have updated", press U, see it's all swell, then want
to continue where you left off. Clearing the state could be restricted to
detecting a nation switch as well as when pressing Shift+E/D on an
initiating page (i.e. activity feed), and in other cases it will continue
where it left off.
—
Reply to this email directly, view it on GitHub
https://github.com/libcord-tech/gauntlet/issues/11#issuecomment-1398020533,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXDPMQPJOPNC3ECDH5WBVE3WTI6TJANCNFSM6AAAAAAUBE6VZA
.
You are receiving this because you were assigned.Message ID:
@.***>


Sensor for current fan speed

Please add a sensor that displays the Fan Speed in % of full speed.

This will be added in the next version.
Currently, I have a version that uses services to control the fan. The next version will use a config flow and will setup a fan domain to control the fan using the UI


Get a Provisioning Profile by UUID

🗣 Context
When fetching specific profiles.
💬 Narrative
As a user
I want to be able to fetch a provisioning profile by uuid
So that I can download a single profile
📝 Notes
If I know the UUID of a provisioning profile I think it would be useful to be able to download just that single profile.
🎨 Design
asc profiles read [--api-issuer <uuid>] [--api-key-id <keyid>] [--csv] [--json] [--table] [--yaml] [--download-path <path>] <uuid>

✅ Acceptance Criteria
GIVEN there is a provisioning profile with the specified uuid
WHEN I request that profile
THEN The profile information is displayed
GIVEN there is a provisioning profile with the specified uuid
WHEN I request that profile and I specify a valid --download-path
THEN The profile is saved to the specified <path>

This was not actually possible with just the filtering options provided by the API.  So instead @DechengMa implemented filtering by resource ID in #216.  That will have to do for now.


Turning off AOT in angular.json for production does not work at runtime

I'm submitting a...
[X] Bug report

Current behavior
main.331a9edcffa8ca2717f0.js:1 Uncaught Error: Unexpected value 'e' imported by the module 'function(){}'. Please add a @NgModule annotation.
    at ka (main.331a9edcffa8ca2717f0.js:1)
    at main.331a9edcffa8ca2717f0.js:1
    at Array.forEach (<anonymous>)
    at e.getNgModuleMetadata (main.331a9edcffa8ca2717f0.js:1)
    at e._loadModules (main.331a9edcffa8ca2717f0.js:1)
    at e._compileModuleAndComponents (main.331a9edcffa8ca2717f0.js:1)
    at e.compileModuleAsync (main.331a9edcffa8ca2717f0.js:1)
    at e.compileModuleAsync (main.331a9edcffa8ca2717f0.js:1)
    at e.bootstrapModule (main.331a9edcffa8ca2717f0.js:1)
    at Object.zUnb (main.331a9edcffa8ca2717f0.js:1)

Expected behavior
It should work.
Minimal reproduction of the problem with instructions
ng new MyProject
cd MyProject
EDIT line "aot": true, replace by "aot": false
ng serve --configuration production

Navigate to http://localhost:4200/
What is the motivation / use case for changing the behavior?

Environment

Angular version: 6.0.3
Windows 10.

https://github.com/angular/angular/issues/24450


Particles2D: Explosivness = 0 and Emit Timeout <= Lifetime - never emit

Operating system or device - Godot version:
Windows 10
Godot 2.1.3 - stable (current steam version)
Issue description:
When i set Particles2D.Explosivness to 0 (for some reason, any other value works) and Emit_Timeout to <= Lifetime, emitter will not produce any particles when you call set_emitting(true)  (with exception of calling directly from _ready())
Steps to reproduce:

add Particles2D node

set Lifetime to x (lets say 3)
set Explosiveness to 0
set Emit_Timeout to anything <= x (let say 3 so Emit_Timeout = Lifetime or 2.9 so Emit_Timeout < Lifetime)

add and enable (lets say, it works as expected from _ready()...) _unhandled_key_input(key_event)
call get_node("Particles2D").set_emitting(true) from script function _unhandled_key_input(key_event) (added in prev step) - again, NOT from _ready() - that works.

Btw - when i was creating minimal example project, i noticed that if you set Emit_timeout to any float >= Lifetime (in my example, lets say 3.1) it looks like it start AFTER Emit_Timeout (3.1 sec) and i have strong feeling that it is related.
Link to minimal example project:
quick and dirty example: buggy_emit.zip - press space to emit.
Workaround:
Set Explosivness to 0.1 (its almost unnoticeable)

Yeah, I'm getting this too.

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the current stable version, and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about its current status here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.


Create metrics from graph

Implement common fragmentation metrics from (reduced) graph.
See Fahrig (2003) for some metrics.
Requires: graph data input from graph algorithm

Implement common fragmentation metrics from (reduced) graph.
See Fahrig (2003) for some metrics.
Requires: graph data input from graph algorithm

This issue has been replaced the more up-to-date issue #35.


Run trained model on CPU

I try to run eval.py on my laptop that doesn't have cuda and I got the following error:
Traceback (most recent call last):
  File "eval.py", line 62, in <module>
    pred = model(x)
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 547, in __call__
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/parallel/data_parallel.py", line 146, in forward
    "them on device: {}".format(self.src_device_obj, t.device))
RuntimeError: module must have its parameters and buffers on device cuda:0 (device_ids[0]) but found one of them on device: cpu

I also modified the load state code to
checkpoint = torch.load(checkpoint, map_location=lambda storage, loc: storage)
but it doesn't fix all issues.

Fixed, feel free to activate this if it repro.


A question about α in the loss formula of AOST

Hello!
In your new model AOST, you use "α" in the loss formula. Is every α_l equal to each other? As the paper says:" To balance the gradient contribution of sub-networks, we have to increase the weight of deeper subnetworks.", I think it means when L=3, α_1 < α_2 < α_3. But the default setting is α=2, and I am very confused about which α_l is set to 2?
Thank you!

In α^l, α is a base and l is an exponent. When setting α=2, we have α^1 = 2, α^2=4, α^3=8......

Thanks for your reply, I mistakenly thought l represented the subscript.


[Feature Request]Use Gzip for compress latent file size

from reddit by joycaption auther:
https://old.reddit.com/r/StableDiffusion/comments/1gdkpqp/the_gory_details_of_finetuning_sdxl_for_40m/
Each image in the dataset is about 1MB, which means the dataset as a whole is nearly 7TB, making it infeasible for me to do training in the cloud where I can utilize larger machines. But once gzipped, the latents are only about 100KB each, 10% the size, dropping it to 725GB for the whole dataset. Much more manageable. (Note: I tried zstandard to see if it could compress further, but it resulted in worse compression ratios even at higher settings. Need to investigate.)

The latent is already saved in .npz, i.e. zip format. However, in my experience zipping latents has little effect. Therefore, I am a little skeptical as to how much the size will be reduced even if gzip is used.
Currently, the sd-scripts cache is saved in Numpy's .npz format with float32 or float16. I am considering changing this to .safetensors format and saving it as float16/bfloat16/float8_e4m3fn.

Having said that, the compression ratio of NPZ is actually 100%, and I checked that the npy format decompressed to exactly the same size as NPZ.Using just zip or gzip compression can improve the compression ratio by reducing it to 25%, i.e. 1.6mb to about 400kb.The extreme compression of 7z on the other hand can be reduced to 20%, i.e. 300kb/1.6mb


change the __nuxt id

What problem does this feature solve?
I have learned that the team has the idea to delete the layout. I hope that I can modify the id name of the element __nuxt or add an option to set the id name.

This feature request is available on Nuxt community (#c10187)

Hi @songsChen
It's already possible, see https://nuxtjs.org/api/configuration-global-name


Convert enum selectors to CamelCase

In the project we currently have code like:
	| antialiasOption |
	antialiasOption := AeCairoAntialias CAIRO_ANTIALIAS_FAST.

	cairoContext
		primSetTolerance: 0.2;
		primSetAntialias: antialiasOption.

	fontOptions := AeCairoFontOptions new
		antialias: antialiasOption;
		hintMetrics: AeCairoHintMetrics CAIRO_HINT_METRICS_ON;
		hintStyle: AeCairoHintStyle CAIRO_HINT_STYLE_SLIGHT;
		subpixelOrder: AeCairoSubpixelOrder CAIRO_SUBPIXEL_ORDER_DEFAULT;
		yourself

Where the underscore capitalized class-side methods are defines literally copy-pasted from the C header.
Probably we should convert to AeCairoAntialias CAIRO_ANTIALIAS_FAST -> AeCairoAntialias fast, or at least AeCairoAntialias CAIRO_ANTIALIAS_FAST -> AeCairoAntialias cairoAntialiasFast, but that's quite redundant.

Only missing are:

[ ] AeCairoFontType
[ ] AeCairoFreetypeSynthesize
[ ] AeCairoStatus
[ ] AeFTFaceFlag
[ ] AeHbScript


Google sheets url is not whitelisted while trying to embed the url

When i try to add the embed web component and try to embed an publicly accessible google sheets url but says it is not whitelisted.

If I recall correctly, google docs can't be embedded as they prevent it. I'll have to check again.

Google Sheets and Google Docs can be embedded without problem in editing mode.
It also includes a specific function to publish the document by generating the iframe code:

+1


feat(ec2-image): include disabled and deprecated images

Overview
This improves the EC2Image resource by introducing 3 new settings, by default these settings are false because they would change the behavior of the EC2Image discovery.
In Version 4, these settings will be enabled by default.
New Settings

IncludeDisabled
IncludeDeprecated
DisableDeregistrationProtection

About Disabling Deregistration Protection
There are 2 types of protection, one with a 24 hour cool-down, and 1 without a cool-down. If the image is protected by the option without a cooldown, the image will be removed immediately given that the DisableDeregistrationProtection setting is true.
If the image is protected with the 24 hour cool-down and DisableDeregistrationProtection is true, then aws-nuke will disable the deregistration protection and then filter the resource out due to it not being eligible for removal. You will then have to re-run aws-nuke after 24 hours to remove the remaining images. Using CLI argument --include EC2Image will target the specific resource type.

:tada: This PR is included in version 3.0.0-beta.62 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Updated modern lifecycle

Learn Build status updates of commit ee1748f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/terms/lifecycle.md
:white_check_mark:Succeeded
View

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel


Missing parameter

Please provide the description for the NAV service tier parameter XmlMetadataCacheSize

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

ID: e77b5afc-fe2d-56f9-2418-07d17da7a3d8
Version Independent ID: a1a9791c-0112-6579-8eef-906ef110a82b
Content: Dynamics NAV Server Configuration - Dynamics NAV
Content Source: dynamics-nav/Configuring-Microsoft-Dynamics-NAV-Server.md
Service: unspecified
GitHub Login: @jswymer

Thanks for the feedback,
This setting is primarily for internal use now. It is in the CustomSetting.config file temporarily for backwards compatibility with older versions of Dynamics NAV, which cached metadata primarily based on XML. This has changed for newer versions, which use NCLMetadata objects that are type based.  There are no benefits of changing this setting from the default. We will eventually remove this setting from the file.
I have updated the content to indicate that this is for internal use only. This change will be available the next time we refresh the content.
Thanks again


Improve test file downloader

This PR improves the download_file function to

download to a temporary file first, before renaming to the final destination; if you e.g. KeyboardInterrupt a download, you won't end up with a corrupt half-downloaded file
add logging, so it's a bit clearer why tests are taking a while

Would it be better to use python's temporary file context manager? (The with syntax thing)

I highly recommend getting the ruff vscode extension and enabling formatting on save

I highly recommend getting the ruff vscode extension and enabling formatting on save

I'm used to running Ruff via pre-commit, but yeah, good call, configured the Ruff PyCharm plugin to fix-on-save 😁


package.json for npm installations

done

Regards,
Jan CW
*
Kroeze
* T
his
is a version number, "CW"
is more recent than
"H"
.

On Tue, May 31, 2016 at 2:55 PM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project. Before we can look at your
pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlemaps/js-rich-marker/pull/10#issuecomment-222679628,
or mute the thread
https://github.com/notifications/unsubscribe/ACPIv6GZvNdX98_MPXLfUORRGguGAzSxks5qHC-2gaJpZM4Iqf-t
.

Can you please merge it?


Build a geofencing app

Authentication failure in step 10 in simulator.
Please specify the credentials that need to be entered by giving example

Hi @palakagarwal15
You need to login with your SAP Cloud Platform credentials (same you used in Step 1)

after entering the credentials. I am being authenticated and thereafter an "http status 404 - not found" error is received in the red block.

Where do you get the 404 error from? The authentication, or the retrieval of the OData service?
Can you check whether your HANA MDC is still running, and you can access your OData service?

HANA MDC is still running but for OData Service, my xml was not created(I dont know why), so i asked my friend to create it for me from his id. Thereafter i used that xml file in my project.. Will that affect?? 404 error is after logging the page and then a blue dialog is shown "You are being authenticated" and thereafter "404 error is found"

Thank you for your feedback. We haven't heard from you in the past 30 days, so we are closing the issue.
If you still have questions, feel free to reopen the issue.


Can we consider to install openssl 1.0.2 and FIPS module

We are requested to install openssl FIPS module in the image, is it possible to upgrade openssl (from 1.0.1 to 1.0.2) and install FIPS module in Base OS or add a stage to do that? Thanks.
http://openssl.org/docs/fips.html

Currently BOSH team is waiting for an ok on FIPS compatibility work from CF foundation. Since Ubuntu does not have pre-built packages for openssl FIPS that they maintain this would be non-zero maintenance cost for the BOSH team. Once CF starts moving towards FIPS compatibility then BOSH can also take it on.

Any update on this @cppforlife?


Lost GameWindow focus while using this in openlevel a new level

I change a SProgressBar to replace origin SCircularThrobber, but I found that it lost focus while switch level to a new level(use openlevel method), thus the SProgressBar  not change smoothly

I also use SetFocusToGameViewport and SetInputModeUIOnly or (SetInputModeGameOnly , SetInputModeGameAndUI), nothings change.

I test in my project, I found that the window also is foreground ,but the switchlevel's slate not update quickly, which result the material SProgressBar not change smoothly.

I use command
Slate.AllowSlateToSleep 0
which let SprogreeBar change smoothly


Azure bicep – Find an item in an array of objects

Hi,
Hope that you are doing well.
I am stuck trying to filter out a specific value from Output.
Here my config
param groupUsage array = [
  'admins'
  'businessadmins'
]

resource entraIdGroup<EMAIL_ADDRESS>= [for usage in groupUsage: {
  displayName: toUpper('appgrp_${RegionCode}${productName}${usage}${env}')
  mailEnabled: false
  mailNickname: toUpper('appgrp${RegionCode}${productName}${usage}${env}')
  securityEnabled: true
  uniqueName: toUpper('appgrp${RegionCode}${productName}${usage}_${env}')
}]

output aadGroupPro array = [for (usage, i) in groupUsage: {
  aadGroupUsage: usage
  aadGroupName: entraIdGroup[i].uniqueName
  aadGroupId: entraIdGroup[i].id
}]

I'm trying to filter the output to get the group Id where the groupUsage equal "admins".
I try this lambda expression with no success
first(filter(entraIdGroup.outputs.aadGroupPro, x => x.aadGroupUsage == 'admins')).id
module readerassignment 'roles.bicep' =  {
  scope: Reg
  name: 'readerrole'
  params: {
    principalId: first(filter(entraIdGroup.outputs.aadGroupPro, x => x.aadGroupUsage == 'admins')).id
    roleDefinitionId: 'acdd72a7-3385-48ef-bd42-f606fba81ae7'
  }
}

Can you help please.

@marcelin I can't spot anything obviously wrong with your usage of first and filter. Could you provide more information about what is not working?

@anthony-c-martin thanks for your quick response.
As you can see in this Screenshot bicep underlines in red the expression 'outputs'

when i run the bicep file i have this error message
Error BCP055: Cannot access properties of type<EMAIL_ADDRESS>An "object" type is required.

Thanks @anthony-c-martin . It work now


FIDE rules

Analyze; create user stories
Extend backlog below.
Create separate issues per item TODO

Paragraph 9.6 is new altogether:
9.6
If one or both of the following occur(s) then the game is drawn:
the same position has appeared, as in 9.2b, for at least five consecutive alternate moves by each player.
any consecutive series of 75 moves have been completed by each player without the movement of any pawn and without any capture. If the last move resulted in checkmate, that shall take precedence.


Chrome Dinosaur Game added

Fixes #165

@artimanputra I added the screenshot of the game. Can you please check and merge it with GSSoC23 and the level 2 label?

@artimanputra Please review this PR too.

Please merge this with the main code.
@artimanputra @Dharmesh177 @Shreyaar12 @cleveranu

@Dharmesh177


Adding Package.json for NPM

This adds a package.json so this can go into npm
I've added the package here:
https://www.npmjs.com/package/highlightjs
Let me know any npmjs usernames that you'd like added to contributors

https://www.npmjs.com/package/highlight.js ?
Usually the syntax for a component on npm is component-%name.

@thepauleh Could you add myself and a few other maintainers to the package? 9.2.0 is out.

HI Rob - i've added a couple of people to the project - can't find you on npmjs?

@thepauleh Thanks! I should be robloach on there https://www.npmjs.com/~robloach
Thanks again! Glad this got pushed forwards.


Document issue: wrong object structure for rgba format color

if you use rgba as the default color format, and pass it with v-model, you should use:
<vue-color v-model="color">
...
data(){
    return {
        color: {
            r: 1,
            g: 1,
            b: 1,
            a: 1,
        }
    }
}
...

What the document told you is wrong:
color:{
  rgba: {
    r: 25,
    g: 77,
    b: 51,
    a: 1
  },
}

Because tinycolor.toRgb() dosen't support this documented object format.

We will fix the document in the next release, estimated in next week.
Thanks for your feedback.

Have updated document. Thanks!


shallowMounted tests with globally registered components that have slot content display component registration errors

Version
1.0.0-beta.28
Reproduction link
https://github.com/robodude/shallow-test
Steps to reproduce

Create a globally registered component (A) that has a slot.

Use that component (A) in another component (B) which passes information via slot to (A)

Test the component (B)

Run unit tests

or

Pull from repo link.

Navigate to directory using command prompt

npm ci

npm run test:unit

What is expected?
Test passes without errors
What is actually happening?
Test passes with an error
console.error node_modules/vue/dist/vue.runtime.common.js:603     [Vue warn]: Unknown custom element: <HelloWorld> - did you register the component correctly? For recursive components, make sure to provide the "name" option.

The generated snapshot looks how I would have expected it to.

We could auto stub every non-DOM element, but that would stub any custom elements that people use.
Ultimately we decided to require components to be registered explicitly before they are stubbed. As you suggested, you will need to install the components explicitly on the localVue constructor.

@eddyerburgh I've just been having the same problem. Is there no way to register the global components for the suite instead of explicitly for every component?
We've just made all Base components global and now the tests are complaining about unregistered components. This localVue approach works but it means increased setup for your tests so we're kind of deferring the benefits of doing it. 😕

Any updates? I'm having headaches with this, it gets really filthy and verbose to manually set each custom component.

I am able to just setup a components.js file where I register all my global vue components:
// example
// components.js
import Vue from 'vue'
import MyHeader from './components/MyHeader'
import MyNav from './components/MyNav'
Vue.component('MyHeader', MyHeader)
Vue.component('MyNav', MyNav)

// ... etc

I import this file both into my app and into a setupFile for my jest tests
// jest.config.js
modules.exports = {
moduleNameMapper:{
    "~(.*)$": "<rootDir>/resources/js/$1",
},
setupFilesAfterEnv: ['<rootDir>resources/js/tests/setupTests.js'],
// ...etc
}

//setupTests.js
import '~/components'

This seems to negate the need to register the component dependencies with the localVue approah above.

@danielkellyio are those Vue components .js files or .vue files?

I found that if you are using shallow mount, there is no need to import the real components and register them to localVue, you can simply register an empty component
localVue.component("MyComponent", {});

what's the equivalent of doing this these days? I'm not seeing any function like createLocalVue


Bookmarks - Rename bookmarks to favorites

Renaming Bookmarks to Favorites would be consistent with our existing marketing materials.  (If this is moved out of backlog please add tickets to webtop and center to change their dropdown menus to say Favorites).

We had a very detailed discussion about whether these would be "Bookmarks" or "Favorites" at the beginning of this project. In the end, it was decided that we would use "Bookmarks" for now, reserving "Favorites" for future functionality. If users have a lot of bookmarks, a second tier of organization will be added called "Favorites" and those would appear in many places as a small / quick set of the apps and allow easy, speedy access.
All of the icons and wording ("Bookmarks" and "Your Library") were chosen accordingly. If we need to switch it to "Favorites" it will be very difficult to implement and will probably necessitate another terminology change.
Also, I strongly disagree with changing thoroughly thought-out decisions and re-doing all the terminology and iconography of the platform to reflect marketing materials which should be describing the platform.

Thanks, Sarah.  Tony just told me that we had already had this conversation.
This request came from program management.  We'll see what we can do to convince them this isn't the correct approach.
I'm adding a "User Submitted" tag to any issues that were reported by users (or management) of the system.  This does not mean that we have to accept all of these requests, but we do need a way to prove that we received their request and considered it.

I agree we should not implement this at this time.


nuxt 3.7.0 - "getActivePinia()" was called but there was no active Pinia

Environment

Operating System: Linux
Node Version:     v16.20.0
Nuxt Version:     3.7.0
CLI Version:      3.7.3
Nitro Version:    2.6.2
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-h8hokc?file=app.vue
Describe the bug
Hello,
since 3.7.0 it seems that the pinia instance is initilized in a different order. With nuxt 3.6.5 everything works fine.
Additional context
There is also an issue on my repository for this topic https://github.com/CodeDredd/pinia-orm/issues/1585
Logs
[🍍]: "getActivePinia()" was called but there was no active Pinia. Did you forget to install pinia? const pinia = createPinia() app.use(pinia) This will fail in production.

at Query.commit (./node_modules/pinia-orm/dist/index.js:630:91)
at Query.storeFind (./node_modules/pinia-orm/dist/index.js:833:23)
at Query.select (./node_modules/pinia-orm/dist/index.js:897:23)
at Query.internalGet (./node_modules/pinia-orm/dist/index.js:871:25)
at Query.get (./node_modules/pinia-orm/dist/index.js:850:19)
at Repository.all (./node_modules/pinia-orm/dist/index.js:1566:25)
at setup (./app.js:15:28)
at _sfc_main.setup (./app.js:30:23)
at callWithErrorHandling (./node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:156:18)

Possibly related: https://github.com/nuxt/nuxt/issues/22918

Yes. I think below works
import { defineStore, createPinia, setActivePinia } from "pinia"
const pinia = createPinia()

export default { store: setActivePinia(pinia) }
export const useMainStore = defineStore("main", {

Yes. I think below works
import { defineStore, createPinia, setActivePinia } from "pinia"
const pinia = createPinia()

export default { store: setActivePinia(pinia) }
export const useMainStore = defineStore("main", {

It's not about a general vue project there i need to create Pinia store by my own. Here @pinia/nuxt is used which already creates the pinia instance.
But somehow the pinia instance isn't created before setup is called which was different in nuxt 3.6.5.
The workarounds that work are already mentioned in this ticket: https://github.com/CodeDredd/pinia-orm/issues/1585

Yes. Says in nuxt only. In store folder.

Yes. I think below works
import { defineStore, createPinia, setActivePinia } from "pinia"
const pinia = createPinia()

export default { store: setActivePinia(pinia) }
export const useMainStore = defineStore("main", {

It's not about a general vue project there i need to create Pinia store by my own. Here @pinia/nuxt is used which already creates the pinia instance. But somehow the pinia instance isn't created before setup is called which was different in nuxt 3.6.5. The workarounds that work are already mentioned in this ticket: CodeDredd/pinia-orm#1585

nuxt 3.11.2, had to use the same workaround, it doesn't seem to be fixed with more current versions

nuxt 3.11.2, had to use the same workaround, it doesn't seem to be fixed with more current versions

Same situation here. It works on 3.10.2 and fails on 3.10.3.
I have created a new bug report here: https://github.com/nuxt/nuxt/issues/27447


concurrent write exception

The client socket closes after a few seconds, and the server returns a concurrent write-on connection error.

The issue is now resolved & it's been fixed on #PR-9


File Manager: download broken in Firefox with no error in console

See: https://www.virtualmin.com/node/56365
I can't find the issue, I suspect changes in the javascript are to blame, but it seems only the minified versions of the javascript is present here.
Is the original source for the file manager javascript for recent versions available somewhere so I can examine the files to try and find the issue?

It's been already fixed.

Cool! Thanks for the update!


bug in the gcc lib

We found the gcc lib has a bug, call the system call futex without the FUTEX_PRIVATE_FLAG  flag. This would block the thread after mesh, because the phys address changed. clang' lib without this bug.
eg. std::future would trigger this bug.

yikes!  do you have links to the bug or a small reproducer? is it in libgcc or glibc? its surprising to me that freeing the meshed page wouldn't clear/purge the waiting futex, otherwise it seems like a process could get notified about a futex in a different process (if/when the memory is reused)

https://github.com/gcc-mirror/gcc/blob/df3e7e7eba31450357b3773f5fb028b5ec2d3669/libstdc%2B%2B-v3/src/c%2B%2B11/futex.cc
here:
const unsigned futex_wait_op = 0;
const unsigned futex_wake_op = 1;
they should be
const unsigned futex_wait_op = 0 | 128;
const unsigned futex_wake_op = 1 | 128;