Fix wrong xorm get usage on migration

Fix the bug on try.gitea.io
2023/09/18 01:48:41 ...ations/migrations.go:635:Migrate() [I] Migration[276]: Add RemoteAddress to mirrors
2023/09/18 01:48:41 routers/common/db.go:34:InitDBEngine() [E] ORM engine initialization attempt #7/10 failed. Error: migrate: migration[276]: Add RemoteAddress to mirrors failed: exit status 128 - fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
 - fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

Caused by #26952

Isn't the problem that there are mirrors in the database which do not exist in the file system? I don't think we should work around that in the migration.

I don't see how the change should be related to the error message.

I don't see how the change should be related to the error message.

This PR will ignore the empty repository and repositories which status is not ready. So reading git repository should not be failed.

If there is a mirror the repo path must exists:
pull mirror
https://github.com/go-gitea/gitea/blob/323135b97b219d7fb10557fb9d9156c6bef3ae62/services/repository/create.go#L241
push mirror
https://github.com/go-gitea/gitea/blob/323135b97b219d7fb10557fb9d9156c6bef3ae62/services/mirror/mirror_push.go#L29

That PR should not have merged at that point...

That PR should not have merged at that point...
See also https://discord.com/channels/322538954119184384/322910365237248000/1153441247940321391 and the following messages.

Please review https://github.com/go-gitea/gitea/pull/27132


Please include release notes so there is consistency with the rest of the product set.

It would be really useful to have a page for release notes ( agent versions, when they were released, what has been fixed in each version of the agent).  For some customers, this is a necessity.
This will bring some uniformity across the Azure Arc product set.

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

ID: 9b49520e-430a-723e-c284-4cb6b38ca418
Version Independent ID: 1b8ea006-328b-b22d-b0f5-57f5455f68d8
Content: Upgrading Azure Arc-enabled Kubernetes agents - Azure Arc
Content Source: articles/azure-arc/kubernetes/agent-upgrade.md
Service: azure-arc
Sub-service: azure-arc-kubernetes
GitHub Login: @shashankbarsin
Microsoft Alias: shasb

@dmc-tech, it appears that the Azure Arc release notes include Kubernetes specific updates for Azure Arc (@shashankbarsin, let me know if I'm incorrect here):

https://docs.microsoft.com/en-us/azure/azure-arc/data/release-notes
https://docs.microsoft.com/en-us/azure/azure-arc/data/version-log

If anyone is looking for the AKS release notes, they can be found here:

https://docs.microsoft.com/en-us/azure/aks/supported-kubernetes-versions
https://github.com/Azure/AKS/releases

The connected k8s agent has a number of versions that do not have release notes attributed to them.  The example you show is for Arc enabled Data services, not the Kubernetes Azure Arc agents.
I can see a list of the agents that have been released, but that is through me digging around MCR: https://mcr.microsoft.com/v2/azurearck8s/batch2/stable/azure-arc-k8sagents/tags/list

@dmc-tech, I'm going to assign this to the document author who will be able to provide more insight.

#reassign:JnHs

Hi @dmc-tech, thank you for taking the time to share your feedback and I apologize for the extremely long time to respond. We appreciate your input; I've created a work item to consider this suggestion and we'll triage it against other priorities in the future. Again, I'm sorry for the long delay in addressing your feedback. Thanks again!
#please-close


The argument 'path' must be a string or Uint8Array without null bytes.

Environment

Operating System: Windows_NT
Node Version:     v14.18.0
Nuxt Version:     3-3.0.0-27234368.47ea41d
Package Manager:  Yarn
Bundler:          Webpack
User Config:      -
Runtime Modules:  -
Build Modules:    -

Describe the bug and expected behavior
Getting error:
Nuxt CLI v3.0.0-27234368.47ea41d                                                                                14:37:33
                                                                                14:37:33
  > Local:    http://localhost:3000/
  > Network:  http://<IP_ADDRESS>:3000/
  > Network:  http://<IP_ADDRESS>:3000/
  > Network:  http://<IP_ADDRESS>:3000/
  > Network:  http://<IP_ADDRESS>:3000/
  > Network:  http://<IP_ADDRESS>:3000/

i Vite warmed up in 980ms                                                                                14:37:36
√ Generated nuxt.d.ts                                                                                14:37:36
√ Vite server built in 1025ms                                                                                14:37:36
√ Nitro built in 212 ms                                                                                14:37:37
Server Side Rendering Error: TypeError [ERR_INVALID_ARG_VALUE]: The argument 'path' must be a string or Uint8Array without null bytes. Received 'C:\\nodesites\\nuxt-vue-3\\nuxt3-app\\\x00virtual:C:\\nodesites\\nuxt-vue-3\\nuxt3-app\\.nuxt\\css.mjs'
    at statSync (fs.js:1124:10)
    at tryStatSync (internal/modules/esm/resolve.js:109:13)
    at finalizeResolution (internal/modules/esm/resolve.js:301:17)
    at moduleResolve (internal/modules/esm/resolve.js:731:10)
    at Loader.defaultResolve [as _resolve] (internal/modules/esm/resolve.js:842:11)
    at Loader.resolve (internal/modules/esm/loader.js:89:40)
    at Loader.getModuleJob (internal/modules/esm/loader.js:242:28)
    at Loader.import (internal/modules/esm/loader.js:177:28)
    at importModuleDynamically (internal/modules/esm/translators.js:114:35)
    at exports.importModuleDynamicallyCallback (internal/process/esm_loader.js:30:14) {
  code: 'ERR_INVALID_ARG_VALUE'
}

Steps to reproduce

npx nuxi init nuxt3-app
cd nuxt3-app
yarn install
yarn dev
open browser at http://localhost:3000

Additional context
Message only shows when I open the browser

The error is not being thrown due to the "vue" word but because the path contained null byte (\x00). Null byte is a special byte which terminates the string.
This was the path which was giving error: '/Users/..../nuxt3-app/\x00virtual:/Users/.../nuxt3-app/.nuxt/css...
You can clearly the presence of null byte right after "nuxt3-app/". Just remove that null byte and you will be good to go.


vcpkg hung on update

Today I update and build vcpkg (before I dont update about 3 weeks).
Then I launch command "vcpkg update" as usual update. And the vcpkg loads cpu and don`t stop about 30 minutes. I break the execution because usually it finich update in about 20 seconds and then I try again.
The vcpkg loads cpu again and now it works almost 8 hours.

What kind of information should I add?

Hi @aPSopt
Can you try running:
git pull
./bootstrap-vcpkg.bat (./bootstrap-vcpkg.sh on Linux)
./vcpkg update
``

Hmmmm, now it works


Individual Project - MERN (Mongoose, Express, React + Redux, Node.js)

Create Project With Your Team Using React
BE CREATIVE
Rules
Frontend

Create Landing Page. (Explaining about your web)
Handle all state using Redux.
Features:

Register
Login
Logout
Create Blog
Blog have a comment
User can have an avatar
Blog can have an image
Use Filestack to handle uploaded image

Please use React UI Framework (i.e Reactstrap, Material-UI, ANTDesign  Emotion And Styled-Components)
Deploy to Netlify

Backend

Serve your backend with Node.js

Use Express.js

Save your data in MongoDB

Use Mongoose as an ORM
Uploaded to Atlas

@thatguyarsya @thomasfebrianseiei

GitHub Repo (WIP)

backend
frontend
https://mern-front.netlify.com

frontend
backend

backend

https://github.com/thomasfebrianseiei/MERN-frontend
https://github.com/thomasfebrianseiei/MERN-backend

Frontend
Backend
Deploy


[BUG] White screen when opening the installer

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.8.0
What problem did you encounter?
When ever I try opening the bloxstrap installer it always pops up as a white screen then goes to (not responding) Not sure if the log below is helpful I'll also attach a vid of it happening
https://github.com/user-attachments/assets/31eb0e90-0708-4954-b7ef-a364d4f22e7f
note after a thing called bloxstrap menu opens up but its the same white screen but it only sometimes happens
here is the log the other text box wasn't working
UE4SS.log
Bloxstrap Log
No response

Try reinstalling the .NET 6 runtime. Find it by searching for "desktop runtime" in the add/remove programs list, and then reinstall it at https://dotnet.microsoft.com/en-us/download/dotnet/thank-you/runtime-desktop-6.0.33-windows-x64-installer?cid=getdotnetcore

Still doesn't work.

this may be caused by your hardware being too slow to load the application. i advise waiting for about 5 minutes since i've personally experienced this on old pc's running on hard disks. hope that helps!

here is the log the other text box wasn't working
UE4SS.log

Where did this log come from and what makes you think it's relevant to what's happening here?

this may be caused by your hardware being too slow to load the application. i advise waiting for about 5 minutes since i've personally experienced this on old pc's running on hard disks. hope that helps!

Yeah my pc is pretty new but I still tried your suggestion and it didn't work out. Thanks for trying to help your time is greatly appreciated

Looks like this log is relevant to the issue. Do you know what UE4SS is and do you know how to not make it apply to Bloxstrap?

No clue I'll try doing some research.

I didn't mean to close it mb


chore: V1.14.0 release patch

Issue #, if available:
N/A
Description of changes:
Patches a dependency update and test updates for the v1.14.0 release.
Description of how you validated changes:
make pr passes
Checklist:

[x] Write/update tests
[x] make pr passes
[x] Update documentation
[x] Verify transformed template deploys and application functions as expected
[x] Add/update example to examples/2016-10-31

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

Codecov Report

Merging #1081 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1081   +/-   ##
========================================
  Coverage    94.81%   94.81%
========================================
  Files           69       69
  Lines         3276     3276
  Branches       639      639
========================================
  Hits          3106     3106
  Misses          89       89
  Partials        81       81

Continue to review full report at Codecov.

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


Editing a note isn't intuitive

Affected version
0.1.1
Description
Editing a note requires a double click, but this isn't intuitive or mentioned explicitly in the UI. I think an edit button (icon-only) would be better.

would it be better if it only required one click? but what about the misclicks...
to be honest the point about sticky notes is to be able to edit them easily so I think the current way is half as good

thanks. I'll work on this soon...

Sticky now doesnt require a double click to open a note


Tracking issue for bot problems

Note that these problems may or may not be related to recent bot updates.

[ ] Development add-on (XPI) is not signed anymore (perhaps unzip is missing on the bots, as that process is being spawned?)
[ ] Linux bot appears to be dead (see https://github.com/mozilla/pdf.js/pull/8028#issuecomment-277520822)
[ ] Issue #8005
[x] Issue #8015
[ ] IRC shows pdf.js/gh-pages b269a97 Brendan Dahl: gh-pages site created via make.js script... instead of pdfjsbot

Development add-on (XPI) is not signed anymore

yeah, maybe unzip or path to it, or @brendandahl do you still have amo.json file from old windows bot? on_push still has signxpi.js call https://github.com/mozilla/botio-files-pdfjs/blob/master/on_push.js#L23

unzip is there. I didn't copy over amo.json (fixed now).

Git user should be fixed now too.

Closing as fixed. The remaining problem is tracked in a separate issue. Thanks!


Reticle Eval

Evaluation showing we can import reticle kernels and use them in Filament designs.

That's pretty rad, TBH.


Release V1.0.0

Overdue V1 release of hyp3-isce2 🥳! Adds merge_tops_bursts CLI tool for merging burst InSAR products and adds some new datasets to burst InSAR products to facilitate merging.

Golden tests are good (after accounting for known changes) so merging!


chore: Fix typos in the codebase.

SubTasks
There are multiple typos in the production codebase. E.g. method and variable names in Java files, test files etc.

@Nikhil-Nandagopal can I work on correcting the typos in codebase and create a PR?

@shubham88fru no thanks this will likely cause more problems that it will solve


Added option for passing through the ssl options

Now you can pass SSL options to pino-elasticsearch and he will pass to the Client. #44

How is this going to be used? I think we should document it in the readme and possibly expose it to the CLI?

From the documentation on the nodejs client from elasticsearch:
https://www.elastic.co/guide/en/elasticsearch/client/javascript-api/16.x/auth-reference.html
you can pass this object
 ssl: {
    ca: fs.readFileSync('./cacert.pem'),
    rejectUnauthorized: true
  }

In my case I am passing only the  rejectUnauthorized: false in the object.
I can try to expose to the CLI, tomorrow.

thanks, that’d be handy. I would also recommend adding a test using http://npm.im/proxyquire.
Il 9 feb 2020, 14:18 +0000, Fernando Derkoski<EMAIL_ADDRESS>ha scritto:

From the documentation on the nodejs client from elasticsearch:
https://www.elastic.co/guide/en/elasticsearch/client/javascript-api/16.x/auth-reference.html
you can pass this object
ssl: {
ca: fs.readFileSync('./cacert.pem'),
rejectUnauthorized: true
}
In my case I am passing only the rejectUnauthorized: false in the object.
I can try to expose to the CLI, tomorrow.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I am also in need of this feature.  Is this still being actively worked on?

I don't think so. Would you like to send a fresh PR?

Sure thing. I'm going to wrap up the other PR I have first and then would be happy to

Closed for no activity, feel free to reopen if you are still interested.


fix: html block test

Fix RM-XYZ

🧰 Changes
unskipping and fixing one of the html block tests
🧬 QA & Testing

Broken on production.
Working in this PR app.

This PR was released!
🚀 Changes included in v6.75.0-beta.66


Apply the same pre-processing to my own scanning sessions ?

Hello,
First of all thanks for the great work! The progress we are seeing in the field of fMRI-to-Image is a direct consequence of your work, so big congrats.
I'm working on a research project where we do our own scanning sessions, with a similar protocol but for different tasks. The scans are done on a 3T MRI.
I wonder what is the best practice to apply the exact same pre-processing as the NSD paper but on my own scanning sessions? I don't see how this could not be done theoretically, but in practice I would like you opinion and advice on how to do it and avoid mistakes.
The goal for me would be to go from my nifti raw sessions to a dataset with pre-processed (masked, beta, etc) voxels for each event of my protocol.
I hope you'll have a bit of time to provide a bit of advice, otherwise that's completely ok and don't hesitate to close the issue.
Thanks again!

Hello,
Regarding applying exactly the same pre-processing, that is a very high threshold to cross. The preprocessing performed in NSD is quite custom/tailored, so practically speaking it would require substantial effort to exactly replicate. That being, the code used is indeed available...
If you wanted to "more or less" replicate the pre-processing to the level of what is practically important, I think that is a much easier bar to cross.  Many flavors of pre-processing out there are not that different from what NSD involved.  It's just that there are details to consider, and maybe you care a lot about the details?
The NSD paper describes in full detail exactly what was done.  For the most part you can think about the analysis in terms of preprocessing (e.g. head motion, spatial distortion, registration, etc.) vs. analysis (e.g. the time-series analysis that we implemented using GLMsingle).   Depending on what you care about, maybe you are more concerned with the time-series analysis?

Thanks for the prompt answer!
Thank you for pointing out the difficulty of exactly replicating the pre-processing. It wasn't sure that it would be that difficult and hearing that from you definitely confirms me that it is not the way to go for me.
In my research, I'm trying the following:

start from MindEye v2, which came out recently: paper. This fMRI-to-Image model can be trained on all scanning sessions from 7 out of your 8 subjects for image reconstruction from brain data. Then, it only requires one hour of scanning data from subject 8 to learn the model how to reconstruct images from brain data, which the core contribution of the paper.
in my case I would like to use their pre-trained model on the 7 subjects, and then fine-tune it on my data.
preprocessing vs analysis: in my case i'm calling pre-processing everything that is applied to the data, from its raw acquisition state to its format when inputted in the neural net (head motion, spatial distorsion, registration, betas computation, masking, etc). Sorry about that I come from a more Machine Learning background and I might not have the right vocabulary.
every subject has a different number of voxels as input, they handle that by mapping the voxels to a shared latent space (a shared representation by the neural net).
my guess is that if the preprocessing for my data is somewhat different from the preprocessing of the 7 subjects on which the model was pre-trained on, it'll be harder to obtain good results. That is why I'm trying to be as close in terms of preprocessing.

Hence, what do you recommend in terms of pre-processing steps? This what I had in mind :

motion correction
spatial distorsion
co-registration
compute a glm with a simple model like glover with a design matrix of the events in my sequence related to my tasks
compute the betas for each event
mask the betas using a mask similar to the nsdgeneral mask
use that as input to my model

Let me know what you think!

Thanks for the context...

Hence, what do you recommend in terms of pre-processing steps? This what I had in mind :
motion correction
spatial distorsion
co-registration
compute a glm with a simple model like glover with a design matrix of the events in my sequence related to my tasks
compute the betas for each event
mask the betas using a mask similar to the nsdgeneral mask
use that as input to my model

The steps you describe here sound generally fine. (Except that I don't quite know what "glover" is...)
I guess, one question to consider is the space that your data will be prepared in.  Like, native volume space, or surface space, or some group space like fsaverage or MNI.   Since you are trying to bridge different subjects and datasets, presumably, fsaverage is probably a good choice.
But if yyou want to lean heavily on the idea you described of mapping to some latent space, then I guess the preprocessed space won't really matter (but then of course, the details of your mapping procedure are important to think carefully about).

Thank you for your answers, I'll be working this.
I'll close the issue for now, and maybe re-open it if there are further discussions to have :)
Thanks again for answering promptly, and congrats again on the great work you did.

👍


DBZ-5907 Fix Oracle transaction event undo

https://issues.redhat.com/browse/DBZ-5907
This PR correctly performs transaction event undo by only reversing the last event in the transaction event queue that matches the row-id, rather than undoing all events with the same row-id.
@jpechane I would suggest we consider backporting this to 2.0 and/or 2.1.

@Naros Applied, thanks.

@Naros 2.1 is the new stable so I backported it only to that version.


Temporary table names should be generated independently

In extraction and grounding, DeepDive creates temporary views and tables by extending user's names specified in application.conf.  Most DBMS have length limit, so this can cause problem when names become too long, e.g., #274.  We should fix relevant parts to generate temporary names completely independent from user's input, e.g., _dd_tmp_1234567 instead of very_very_very_very_long_user_table_name_unload_very_very_very_very_long_user_function_name.

We don't have temp tables like the above anymore, though we still have tables like dd_categories_$usertable. 14 chars longer isn't a big concern (even though we could shorten it to dd_cat_). Either way, as long as the DB consistently panics and gives meaningful error messages, that should enough not to frustrate the user for now.


Passing model to the fields

What is your recommendation for being able to access the resource calling the action button from within the fields() function?
Attempts for reference below:
https://ibb.co/wNBYdLV
Video for reference:
https://share.getcloudapp.com/d5ugW6JY

Oh, great dialog with ChatGTP))
Try this:
// TorrentResource.php
ActionButton::make('')
  ->icon('<svg></svg>')
  ->action(new UpdateTorrentAction($this->resource), $this->resource->id)

// UpdateTorrentAction

/**
 * @param Torrent $torrent
 */
public function __construct(Torrent $torrent)
{
    $this->torrent = $torrent;
}

/**
 * Get the fields available on the action.
 *
 * @param NovaRequest $request
 * @return array|null
 */
public function fields(NovaRequest $request): ?array
{
    // $request is empty if action is called from index row (or inline)
    // so use instance injected to action constructor
    $torrent = (fn(): ?Torrent => $request?->selectedResources()?->first())();
    $torrent = $torrent ?? $this->torrent;

    if ($torrent) {
         ...
    }

    return null;
}

And register your action in resource class this way:
// TorrentResource.php

/**
 * Get the actions available on the entity.
 *
 * @param Request $request
 * @return array
 */
public function actions(Request $request)
{
    return [
        UpdateTorrentAction::make(),
    ];
}

Hi,
Thank you for your quick response.
Please find below my modified files as per your instructions:
Touchpoint.php - https://gist.github.com/HeadStudios/9a6befc480556aeb295edc983c4bed4f (resource model)
OppHit.php - https://gist.github.com/HeadStudios/7c1eada4472d65d32a4eaabe0203118d (Action)
Unfortunately I am unable to get the argument passed between the ActionButton and the to the Action's constructor method, I get the error below:
Error:
[2023-03-17 10:12:36] local.ERROR: Too few arguments to function App\Nova\Actions\OppHit::__construct(), 0 passed in /var/www/vendor/laravel/nova/src/Makeable.php on line 14 and exactly 1 expected {"userId":1,"exception":"[object] (ArgumentCountError(code: 0): Too few arguments to function App\Nova\Actions\OppHit::__construct(), 0 passed in /var/www/vendor/laravel/nova/src/Makeable.php on line 14 and exactly 1 expected at /var/www/app/Nova/Actions/OppHit.php:39)
[stacktrace]
Video for reference:
https://p147.p4.n0.cdn.getcloudapp.com/items/kpuA54PP/7cb3b7a0-5b8d-4f5f-9110-1c41ed4e4745.mp4?v=9fc795e0fe507589e34595f09ccd0f96
Anything I'm missing please let me know.
Note: in the video I didn't add the UpdateTorrentAction::make() updated syntax as per your suggestion however I have done this after and it still hasn't worked for me.

@HeadStudios
As I understand your situation:
Error on frontend is caused by actions method of your resource class (Touchpoint.php)
Use OppHit::make() in actions array, because static make method uses Laravel  resolve for dependency injection in your OppHit.php constructor.
Because your OppHit action is constructed on Touchpoint resource initialization — you can not pass any parameter to constructor on this stage. Use default value (__construct(int $number = 0) or type hint for Laravel DI (__construct(Touchpoint $touch))
So, 69 line in your OppHit action will get Touchpoint instance on default Nova action use (by selecting on index row by checkbox or by executing action from dropdown on detail page ) and 70 line will be use on ActionButton press (when your OppHit action will be initialized with provided instance in constructor)
I hope this will help you!

Thank you, this has fixed my issue!  Marking this closed. For anyone interested full code is below (for creating SMS send action button that populates name in textarea)
https://gist.github.com/HeadStudios/f4235359f00315e548ab7034bef8bec5
https://gist.github.com/HeadStudios/c5b93a1e32ed1b7b0c05e146ef6958c3


Custom styles not being applied

Describe the bug
Hello, I am currently trying to override the colors for code snippets using prism highlighter.
The documentation is limited and doesn't say much.
I already have applied the changes with what the prism-theme-vars repo say, and also based on some theme styles I found. But it isn't working as expected, as my custom styles are not being imported.

Desktop (please complete the following information):

OS: macOS
Browser: Brave
Slidev version: 0.22.7

Solved by putting all the --prism variables again like so:
.dark {
    --prism-foreground: var(--text-primary);
    --prism-background: var(--primary);
    ...
}


Extract tests from sub-chapters

I've started looking into using mdbook-keeper with Comprehensive Rust 🦀, but apparently tests in sub-chapters aren't found. This adds a test-case demonstrating the problem and refactors get_tests_from_book to recursively discover tests.

One wart: this doesn't include directories in the generated name/slug. I haven't dug into whether this is due to something in mdbook (i.e. does path include only the file name??) or mdbook-keeper.
I think this PR is worthwhile despite this.

Not including the path is due to src/lib.rs:43, I believe.
There's two options I can think of:

Escape slashes in that path, so that it's clear what the file is.
ignore it

I tried to think of a scenario where not having the full path might cause a bad result with caching; but I can't because it only cares about hashes (not pathes); so if two files are identical they should hopefully both compile/fail the same.
If someone is using this with rand to occasionally cause failures... maybe there'd be some weird behaviour. But also, I think doing that in the first place could also be classed as "weird behaviour"....
I'll merge this, but if you (or anyone else) can think of an way this might cause issues, I'd be more than happy to merge another fix.


Issue #21: merge the issue and pull request prompt templates into one with a new parameter [AI-generated]

AI-generated implementation by CodeWhisper
fix: #21

AI-generated commit information
Summary
This change merges the issue-implementation-prompt.hbs and pr-diff-prompt.hbs templates into a single unified template. It introduces a new parameter to distinguish between issue and pull request contexts. The pull-request-workflow.ts and task-workflow.ts files are updated to use the new unified template.
Files we looked at

/Users/gregoryguterman/code/CodeWhisper/src/ai/pull-request-workflow.ts
/Users/gregoryguterman/code/CodeWhisper/src/ai/task-workflow.ts
/Users/gregoryguterman/code/CodeWhisper/src/templates/issue-implementation-prompt.hbs
/Users/gregoryguterman/code/CodeWhisper/src/templates/pr-diff-prompt.hbs
/Users/gregoryguterman/code/CodeWhisper/src/templates/task-plan-prompt.hbs

Potential Issues

Existing code that relies on separate templates may need to be updated.
The unified template may be slightly longer, which could impact readability.
Care must be taken to ensure that all use cases from both original templates are covered in the unified version.

Next steps
You can reply with instructions such as:

"Revert the last commit"
"Fix the typo in line 10"
"Add a new function to the file"


Choosing "Skip" doesn't skip version update

When an update is available and the "skip" option is offered and selected, the program again shows the update dialogue within x amount of hours. In actuality, we are not allowed to skip a version update. Selecting the "skip" option should result in no further update dialogues (nags) until there truly is an update available beyond the current update. "Skip" should skip. This has happened forever (or as far back as I can recall, through numerous version updates).
Actual behavior
Steps to reproduce the behavior
Beeftext version (e.g. Beeftext v7.0 Portable Edition).
Latest and at least several versions prior.
Operating system version (e.g Windows 10)
Windows 10.

You're right. The wording is incorrect. The label should be Not Now, instead of Skip. I will first fix the wording, this will be included in 12.1. And I will implement a new feature to offer a Skip This Version option (see issue #437)


spelling fix

@mjosephmiller thanks! ✌

Will try a make a better contribution next time...how about making a Munich map?! bitmoji

Will try a make a better contribution next time...how about making a Munich map?!

So far there is only a static one  http://blog.honeypot.io/munich-tech-map/
but it's definitely an option to have a interactive munich tech map in the future.


Enable global scope, disable labels for global scope

Implements https://github.com/snypy/snypy-frontend/issues/1096
A new global scope is enabled, which adds new funtionality:

allows to search for all snippets a user has access to (personal, team assignment or public snippets)
textual search based on snippet content and filtering by file extension (as in personal or team scope)

Labels are not available in the global scope as these are entities that are bound to a user or a team.

Codecov Report

Merging #1245 (075313c) into master (8bc3aa4) will increase coverage by 0.43%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #1245      +/-   ##
==========================================
+ Coverage   43.82%   44.26%   +0.43%
==========================================
  Files          56       56
  Lines         721      723       +2
  Branches       47       47
==========================================
+ Hits          316      320       +4
+ Misses        399      397       -2
  Partials        6        6

Impacted Files
Coverage Δ

src/app/state/label/label.state.ts
19.35% <0.00%> (-1.34%)
:arrow_down:

src/app/components/teams/teams.component.ts
50.00% <0.00%> (+6.25%)
:arrow_up:

src/app/layout/fullscreen/fullscreen.component.ts
66.66% <0.00%> (+11.11%)
:arrow_up:

...omponents/auth-activate/auth-activate.component.ts
77.77% <0.00%> (+22.22%)
:arrow_up:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

Please fix linter errors 🙂

Also the private snippets are visible in the global scope

Also the private snippets are visible in the global scope. I think this should be fixed in the Backend.

Only private snippets inside an assigned team and personal scope should be visible. Are you able to see others?


fake lag stops working after recharging dt

cant double tap bots with fake laag is sad

attach video

It's not possible to use DT and Fakelag together.


Fixed hashes for cabal2nix.

Looks like somebody did a push --force that broke the build ;)
https://github.com/NixOS/cabal2nix/commit/d131b2b2db1bc37a10bbc40c3adea3f006633a5e
https://github.com/NixOS/cabal2nix/commit/33c83599330d67929bef9b3bf10131e352bd108c

I just hit this issue, thanks! Just merged :+1:

Thank you for catching this bug!


Plumb through tensor.pack e2e execution for llvm-cpu backend.

All the tensor.pack ops with static inner_tile_sizes are vectorized, which are all covered by e2e tests.

Can we make this test separate so that it is tested on vmvx path where it is used and vectorization isnt important.

Supporting the op on VMVX is my next step. Although vectorization is not used on VMVX side, it requires a memref version of pack/unpack. In this PR, I'd like to make sure that tensor version covers all the cases for llvm-cpu side.
I'm preparing a separate PR that will bufferize tensor.pack/unpack into iree_linalg_ext.pack/unpack; will enable the test for vmvx. After that, I'll switch data tiling approach to use tensor.pack/unpack version, and deprecate iree_linalg_ext ops and transforms as much as possible. WDYT?

Can we make this test separate so that it is tested on vmvx path where it is used and vectorization isnt important.

Supporting the op on VMVX is my next step. Although vectorization is not used on VMVX side, it requires a memref version of pack/unpack. In this PR, I'd like to make sure that tensor version covers all the cases for llvm-cpu side.
I'm preparing a separate PR that will bufferize tensor.pack/unpack into iree_linalg_ext.pack/unpack; will enable the test for vmvx. After that, I'll switch data tiling approach to use tensor.pack/unpack version, and deprecate iree_linalg_ext ops and transforms as much as possible. WDYT?

Sounds good!


Flow Diagram page: add entity named notes

At NWM, we auto generate flow diagrams for impact analysis, and there is a requirement display notes on Flow Diagrams for additional data (eg: impact statements etc).

would these appear on the diagram itself or on the flow diagram page in a section?

Flow diagram page, in a section. Just need to add an entry in dynamic-section-definitions.js

you ok to take this one?

Yep, I'll take it up, unless someone else wants it.

Making of:  https://youtu.be/-wiZK3jOZ2E  :)


All day events show up as completed the day after

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When there was an all day event yesterday, the event shows up as a completed event today after in event mode.
Expected behaviour
I do not expect to see the event from the previous day as a completed event for today.
Steps To Reproduce
I am connected to a Google calendar.
Whenever I have an all day event yesterday, it shows up as a completed event today.
I have enabled to include all day enemy's and to show completed events as grayed out.
My time zone is GMT+1 in both HA and browser.
Card Version
7.2.0
Home Assistant Version
2033.1.1
Configuration
type: custom:atomic-calendar-revive
enableModeChange: true
defaultMode: Event
firstDayOfWeek: 1
maxDaysToShow: 10
refreshInterval: 60
showNoEventDays: true
showWeekDay: true
entities:
  - entity: calendar.family
    color: orange
    name: Familjen
  - entity: calendar.6a_22_23_lax_och_provschema
    name: Läxor
    color: red
    icon: mdi:owl
  - entity: calendar.helgdagar_i_sverige
    name: Helgdagar
    color: green
    icon: mdi:party-popper
  - entity: calendar.hammarliden11_gmail_com
    name: Skolan
    icon: mdi:town-hall
dateFormat: L
hoursFormat: HH:mm
showRelativeTime: false
showDate: false
showLocation: false
disableCalLink: true
showMonth: false
disableEventLink: true
showNoEventsForToday: true
disableLocationLink: true
showEventIcon: true
showCalendarName: false
showEventDate: true
showDatePerEvent: false
showWeekNumber: false
showFullDayProgress: true
showCurrentEventLine: false
sortByStartTime: true
noEventText: Inga planer
untilText: fram till
showLoader: false
showProgressBar: false
fullDayEventText: hela dagen
showMultiDayEventParts: false
showMultiDay: false

Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

The screenshot shows today's events. The event "Rebecka fyller år" was a full day event yesterday.

Today I had a full day event yesterday that does not show up as a completed event today. So it is not all cases, but there are some more conditions

I think this is the same issue as https://github.com/totaldebug/atomic-calendar-revive/issues/754
But it's still happening for me too on 7.2.0

It could be the same as #754, but please note that I did not see the event spill over to the next day until after it completed.
/Thomas O

From: ajobbins @.>
Sent: Tuesday, January 10, 2023 12:06:40 AM
To: totaldebug/atomic-calendar-revive @.>
Cc: Thomas Olsson @.>; Mention @.>
Subject: Re: [totaldebug/atomic-calendar-revive] All day events show up as completed the day after (Issue #901)
I think this is the same issue as #754https://github.com/totaldebug/atomic-calendar-revive/issues/754
But it's still happening for me too on 7.2.0
—
Reply to this email directly, view it on GitHubhttps://github.com/totaldebug/atomic-calendar-revive/issues/901#issuecomment-1376461363, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD2V23DO4RY523ZXMSOAT63WRSKYBANCNFSM6AAAAAATUOCFSA.
You are receiving this because you were mentioned.Message ID: @.***>

Now it happened again. Is there anything I can do to troubleshoot this? Any additional info I can provide?

It doesn't look like anyone is looking at any issues currently.  Unfortunate.

Now it happened again. Is there anything I can do to troubleshoot this? Any additional info I can provide?

This has been an issue for some time, it's to do with timezones, I just haven't had time to figure it out as in my timezone it works as expected. But I assume it's because Im GMT


[Feature][Manager] Add file and auto_push extract node

Description
[Feature][Manager]Add filesource and sdk extract node
Use case
[Feature][Manager]Add filesource and sdk extract node
Are you willing to submit PR?

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

Code of Conduct

[X] I agree to follow this project's Code of Conduct

File source and auto push source are already supported in StreamSourceOperator#getSourcesMap, so close this issue.


How to migrate slot (CLUSTER SETSLOT)

this is doc: https://redis.io/commands/cluster-setslot#redis-cluster-live-resharding-explained
I want to migrate some slots from one to another.
before:
 cluster nodes
43a1af446b289000fb2d187215eeceab356efa86 <IP_ADDRESS>:7002 master - 0<PHONE_NUMBER>128 2 connected 10922-16383
659c45f1be91a41bbcacc895e500e39aff4b767e <IP_ADDRESS>:7000 myself,master - 0 0 1 connected 0-5460
ab58196edbfb0861fd73846353e464c30ae2cabb <IP_ADDRESS>:7001 master - 0<PHONE_NUMBER>131 0 connected 5461-10921

I want :
 cluster nodes
43a1af446b289000fb2d187215eeceab356efa86 <IP_ADDRESS>:7002 master - 0<PHONE_NUMBER>128 2 connected 10922-16383
659c45f1be91a41bbcacc895e500e39aff4b767e <IP_ADDRESS>:7000 myself,master - 0 0 1 connected 0-8000
ab58196edbfb0861fd73846353e464c30ae2cabb <IP_ADDRESS>:7001 master - 0<PHONE_NUMBER>131 0 connected 8001-10921

And  I do not find any function can use, like CLUSTER SETSLOT  IMPORTING 、 MIGRATING  and so on.
So, how to do it?
thanks!

We should add that function to the commands list but for now you can use:
client.Do("cluster", "setslot", slot, "importing", node).Err()

@vmihailenco   OK  Thanks！


Catch for Invalid .inputrc

Installed phpsh globally (tried locally as well), got the following error:
Traceback (most recent call last):
  File "/usr/local/bin/phpsh", line 47, in <module>
    verbose=opts.verbose)
  File "/Library/Python/2.6/site-packages/phpsh/__init__.py", line 477, in __init__
    readline.read_init_file(input_rc_file)
IOError: [Errno -1] Unknown error: -1

My .inputrc (from https://github.com/mathiasbynens/dotfiles):
# Make Tab autocomplete regardless of filename case
set completion-ignore-case on

# Append a slash when autocompleting symbolic links to directories
set mark-symlinked-directories on

# List all matches in case multiple possible completions are possible
set show-all-if-ambiguous on

Added try-catch block and error output when python encounters an invalid inputrc (although not sure what was causing the above inputrc to fire the readline error).

I don't think the inputrc is invalid…
>>> readline.parse_and_bind(open("/Users/myfreeweb/.inputrc").read())
# success
>>> readline.read_init_file("/Users/myfreeweb/.inputrc")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
IOError: [Errno -1] Unknown error: -1


Remove Spotifree recommendation

I'd like to have a discussion about the Spotifree recommendation.

Dislike ads and Spotify Membership although very useful does not give me a reason to switch from free version

I too dislike ads. I love Reddit. I could easely block ads on Reddit. Instead, they provide me with the option to buy "Gold" and remove ads. And that's what I do. I buy reddit gold to remove ads I don't like having.
Same goes for Spotify. They provide a way to remove advertisment. I don't find ethical to recommend people to bypass the system instead.
For personal reasons one might not be able to afford the Spotify membership, just like one might not be able to afford Reddit Gold. All I'm saying is maybe to avoid recommending people to do the same.

You are right and I definitely agree with you on it. I have removed it.


feat(proto/subgraph): added labelled subgraph

Closes #26

@nikitawootten  @erictg what if an edge is already present in the Knowledge Graph?


Update documentation to consistently use new BidsDataset syntax

In particular, any mentions of BidsDataset.entities[<component_name>] need to be changed to BidsDataset[<component_name>].entities. The same applies to all of the following attributes:

BidsDataset.entities
BidsDatset.wildcards
BidsDatset.zip_lists
BidsDatset.path

Waiting on

[ ] #197

Before updating the docs, I'd like to implement the pretty-print version of the dataset/component to make it easier to explain what's happening interactively


Enable validation of individual containers / components within an EditForm e.g. individual tabs in tab control.

In "details" documents like TreeDetail and ListDetail if any fields on the currently selected model have been modified the component e.g. textbox will have a green border indicating it has been modified. If the model is replaced with a new instance then the green border must be cleared.


Adding more disables.

DISABLE_NODE_MONITOR should be set in edge yaml files as well (both init container as well as hpe-csi-driver)


Create a stack for rocket web framework

We need a stack for rocket web framework based on rust language.

related to #356


Lint for reserved HTML classes

Description
This linter checks for HTML classes that we consider "internal" to Primer ViewComponents.
Integration

Does this change require any updates to code in production?

Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Added/updated previews

@neall is this one ready to go out? 🚀


Verify correctness of the required browser features pages

On https://github.com/mozilla/pdf.js/wiki/Required-Browser-Features we mention required browser features for PDF.js, but I have the feeling that not all of them are necessary nowadays since we moved to using more modern features over the years. Moreover, as indicated in https://github.com/mozilla/pdf.js/issues/11950#issuecomment-637149085, some features may be missing such as range requests or async/await.
In short, we should verify which are still necessary and update the wiki page and the automated feature detection if possible.

There is also the idea to replace the supported browsers overview with a verification based on which demo works; see https://github.com/mozilla/pdf.js/pull/12170#issuecomment-687562858. If we use that, we might also consider dropping the required browser features page since I think it's not relevant then anymore.

Given that this page hasn't received any meaningful updates since 2014[1], and that first of all fixing it and secondly keeping it up-to-date will take quite a bit of effort, I'd suggest that we move forward with https://github.com/mozilla/pdf.js/issues/11954#issuecomment-687587562 instead.

[1] There's been changes since then, but that's when the last feature-test was added.

Given that this page hasn't received any meaningful updates since 2014[1], and that first of all fixing it and secondly keeping it up-to-date will take quite a bit of effort, I'd suggest that we move forward with https://github.com/mozilla/pdf.js/issues/11954#issuecomment-687587562 instead.

[1] There's been changes since then, but that's when the last feature-test was added.


Inject Rust into nonterminal types

Idea: take something like
Comma<T>: Vec<T> {
    // ...
}

and inject Rust into the type using something like:
type __intellij_lalrpop_Comma<T> = Vec<T>;

Does the name really matter? If not I think it would be a better idea to inject that for all type_refs

I don't think the name of the type alias matters, but it probably should be different enough that it won't be referenced by the user. Also, each injection site should be independent of each other, so names shouldn't clash anyways.
That's a good idea, injecting for all non-recursive type_refs.

There's one small catch here:
Nonterminal<Rule>: Box<Rule> = {/*...*/};

Trying to rename "Rule" from the injection leads to


Versioned not ignoring obsolete fields 2

Copy of #2163 from updated branch - as requested have squashed all commits to affected code into one and added a unit test that fails prior to the fix suggested.
Useful to prevent weird errors in development where fields have been renamed and in production where old databases have been updated and old fields made obsolete, and the Versioned extension has later been applied to the dataobject. In both these cases the obsolete fields present in the core table cause an error when writing Versioned records.
To Reproduce:

Create a DataObject with at least one database field
Perform a build
Remove or rename a field from the dataobject, and apply the Versioned extension to the dataobject
Perform another build
Write a new or old but changed DataObject of this type
A database exception is thrown

Resolution
The branch to pull fixes the problem by filtering the list of fields returned from the core table by those known to be present in the code via a call to DataObject::database_fields()

Merged in 6212b4bd4bd9adcc79042ffffe88144796239b57


Using chart.getModel() private method

getModel() is a private method and it can break in the future! https://github.com/apache/echarts/issues/16479. Using the method works fine, it just throws an error: "Property 'getModel' is private and only accessible within class 'ECharts'."
Workarounds might be possible. I started using the method because it was used in some of the examples in the ECharts own examples and provided simple and efficient solutions.

+1


View the bug in converting the whiteboard to mp4

hi @C0D3D3V
used your code to convert the session record in BBB.
But the items that were presented in the whiteboard section of the meeting were displayed as a white page in the converted file.
Have you ever encountered this bug?

Test v1.0.8. Open a new issue if it still exists. And send a session where this happens.


Fixes #602: user configurable listener backlog

Make a new attribute 'backlog' for entity tcpListener.
User can enter integer >= 1, or "max".
Default is "max" -- which means it uses SOMAXCONN.
Invalid value logs error, the uses old value of 50.
I tested setting it to 'max' ,  small integers, and 'foo'.
And then used my connection rate test on it.
Difference in connection rate is very substantial using 'max' vs 5 or 10.

Codecov Report

Merging #649 (288d733) into main (ebff9eb) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #649      +/-   ##
==========================================
- Coverage   25.90%   25.90%   -0.01%
==========================================
  Files         128      128
  Lines       31195    31207      +12
  Branches     4973     4975       +2
==========================================
+ Hits         8081     8083       +2
- Misses      22059    22071      +12
+ Partials     1055     1053       -2

Flag
Coverage Δ

unittests
25.90% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/adaptors/adaptor_common.c
0.00% <0.00%> (ø)

src/adaptors/adaptor_listener.c
4.84% <0.00%> (-0.03%)
:arrow_down:

src/adaptors/http1/http1_client.c
0.00% <0.00%> (ø)

src/adaptors/http2/http2_adaptor.c
2.07% <0.00%> (ø)

src/adaptors/tcp_adaptor.c
2.24% <0.00%> (ø)

src/router_core/route_tables.c
16.62% <0.00%> (+0.27%)
:arrow_up:

src/router_core/address_watch.c
74.71% <0.00%> (+1.14%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.


hypergraph_construction is not defined

hello there is no place in your code that define the hypergraph_construction function.

same, and there's the import issue here: a custom package that doesn't exist is imported


Out of Memory on a 24G GPU Card from InstantMesh

Hello, I'm getting memory allocation error with every workflow that involves this instant-mesh reconstruction node. I'm running on a machine with 24G gpu memory. Should this happen? How could I fix it?
Traceback (most recent call last):
  File "/home/zling/Documents/projects/ComfyUI/execution.py", line 151, in recursive_execute
    output_data, output_ui = get_output_data(obj, input_data_all)
  File "/home/zling/Documents/projects/ComfyUI/execution.py", line 81, in get_output_data
    return_values = map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True)
  File "/home/zling/Documents/projects/ComfyUI/execution.py", line 74, in map_node_over_list
    results.append(getattr(obj, func)(**slice_dict(input_data_all, i)))
  File "/home/zling/anaconda3/envs/comfyui/lib/python3.8/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/zling/Documents/projects/ComfyUI/custom_nodes/ComfyUI-3D-Pack/nodes.py", line 2272, in run_LRM
    mesh_out = lrm_model.extract_mesh(
  File "/home/zling/Documents/projects/ComfyUI/custom_nodes/ComfyUI-3D-Pack/instant_mesh/models/lrm_mesh.py", line 357, in extract_mesh
    mesh_v, mesh_f, sdf, deformation, v_deformed, sdf_reg_loss = self.get_geometry_prediction(planes)
  File "/home/zling/Documents/projects/ComfyUI/custom_nodes/ComfyUI-3D-Pack/instant_mesh/models/lrm_mesh.py", line 165, in get_geometry_prediction
    sdf, deformation, sdf_reg_loss, weight = self.get_sdf_deformation_prediction(planes)
  File "/home/zling/Documents/projects/ComfyUI/custom_nodes/ComfyUI-3D-Pack/instant_mesh/models/lrm_mesh.py", line 110, in get_sdf_deformation_prediction
    sdf, deformation, weight = torch.utils.checkpoint.checkpoint(
  File "/home/zling/anaconda3/envs/comfyui/lib/python3.8/site-packages/torch/_compile.py", line 24, in inner
    return torch._dynamo.disable(fn, recursive)(*args, **kwargs)
  File "/home/zling/anaconda3/envs/comfyui/lib/python3.8/site-packages/torch/_dynamo/eval_frame.py", line 451, in _fn
    return fn(*args, **kwargs)
  File "/home/zling/anaconda3/envs/comfyui/lib/python3.8/site-packages/torch/_dynamo/external_utils.py", line 36, in inner
    return fn(*args, **kwargs)
  File "/home/zling/anaconda3/envs/comfyui/lib/python3.8/site-packages/torch/utils/checkpoint.py", line 494, in checkpoint
    ret = function(*args, **kwargs)
  File "/home/zling/Documents/projects/ComfyUI/custom_nodes/ComfyUI-3D-Pack/instant_mesh/models/renderer/synthesizer_mesh.py", line 132, in get_geometry_prediction
    sdf, deformation, weight = self.decoder.get_geometry_prediction(sampled_features, flexicubes_indices)
  File "/home/zling/Documents/projects/ComfyUI/custom_nodes/ComfyUI-3D-Pack/instant_mesh/models/renderer/synthesizer_mesh.py", line 76, in get_geometry_prediction
    grid_features = torch.index_select(input=sampled_features, index=flexicubes_indices.reshape(-1), dim=1)
torch.cuda.OutOfMemoryError: Allocation on device

HI @iszihan, same issue here, have you been able resolve it?

Hi all,
I'm getting the same error.
I noticed that if I try running the pipeline again immediately afterwards it completes without any issues, however. Is this the case on your end as well @SergioKingOne and @iszihan ?
Also, if you scroll to the beginning of the ComyUI console do you see a message that says:
"UserWarning: TORCH_CUDA_ARCH_LIST is not set, all archs for visible cards are included for compilation.
If this is not desired, please set os.environ['TORCH_CUDA_ARCH_LIST']."
I noticed this message appeared immediately after installing the ComfyUI 3D Pack. I'm wondering if setting this value correctly will resolve the issue, but I don't know what to set it to.

Hi @tcarneyjohnson. That env variable is for setting the available compute capability architectures you can run. That should not be a problem.
A workaround for me was to directly run instant mesh without ComfyUI (i.e. download instant mesh repo and run it with a command).
Another possible solution is to unload the MVs model once its done its job but I could not achieve it.

This also happens to me on the "Triplane_Gaussian_Transformers_to_3DGS_and_3DMesh.json" workflow.
I'm running a 3080 series 24gb vram which is minimum listed req for gaussian to marching cubes. Everything else I use works but the 3d marching cubes.
I get the same error:
  File "D:\Code\ComfyUI_windows_portable\python_embeded\Lib\site-packages\diff_gaussian_rasterization\__init__.py", line 92, in forward
    num_rendered, color, depth, alpha, radii, geomBuffer, binningBuffer, imgBuffer = _C.rasterize_gaussians(*args)
                                                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: Allocation on device

I get the same warning:
"UserWarning: TORCH_CUDA_ARCH_LIST is not set, all archs for visible cards are included for compilation.
If this is not desired, please set os.environ['TORCH_CUDA_ARCH_LIST']."

It crashes on this node:

Fixed this by reinstalling diff-gaussian-rasterization
pip uninstall diff_gaussian_rasterization
pip install "git+https://github.com/ashawkey/diff-gaussian-rasterization.git"

Fixed this by reinstalling diff-gaussian-rasterization
pip uninstall diff_gaussian_rasterization pip install "git+https://github.com/ashawkey/diff-gaussian-rasterization.git"

using the python.exe embedded or ?? Inside the custom nodes folder?

This is also occurring for me on any workflow that uses the InstantMesh node.  Here's the callstack from running the Era3D_to_InstantMesh workflow:
[Comfy3D] [Load_InstantMesh_Reconstruction_Model] loaded model ckpt from D:\ComfyUI\custom_nodes\ComfyUI-3D-Pack\Checkpoints\InstantMesh\instant_mesh_large.ckpt
!!! Exception during processing !!! Allocation on device
Traceback (most recent call last):
  File "D:\ComfyUI\execution.py", line 323, in execute
    output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
                                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ComfyUI\execution.py", line 198, in get_output_data
    return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ComfyUI\execution.py", line 169, in _map_node_over_list
    process_inputs(input_dict, i)
  File "D:\ComfyUI\execution.py", line 158, in process_inputs
    results.append(getattr(obj, func)(**inputs))
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\JeffSipko\miniconda3\envs\py311\Lib\site-packages\torch\utils\_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "D:\ComfyUI\custom_nodes\ComfyUI-3D-Pack\nodes.py", line 2496, in run_LRM
    mesh_out = lrm_model.extract_mesh(
               ^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ComfyUI\custom_nodes\ComfyUI-3D-Pack\Gen_3D_Modules\InstantMesh\models\lrm_mesh.py", line 357, in extract_mesh
    mesh_v, mesh_f, sdf, deformation, v_deformed, sdf_reg_loss = self.get_geometry_prediction(planes)
                                                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ComfyUI\custom_nodes\ComfyUI-3D-Pack\Gen_3D_Modules\InstantMesh\models\lrm_mesh.py", line 165, in get_geometry_prediction
    sdf, deformation, sdf_reg_loss, weight = self.get_sdf_deformation_prediction(planes)
                                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ComfyUI\custom_nodes\ComfyUI-3D-Pack\Gen_3D_Modules\InstantMesh\models\lrm_mesh.py", line 110, in get_sdf_deformation_prediction
    sdf, deformation, weight = torch.utils.checkpoint.checkpoint(
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\JeffSipko\miniconda3\envs\py311\Lib\site-packages\torch\_compile.py", line 31, in inner
    return disable_fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\JeffSipko\miniconda3\envs\py311\Lib\site-packages\torch\_dynamo\eval_frame.py", line 600, in _fn
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "C:\Users\JeffSipko\miniconda3\envs\py311\Lib\site-packages\torch\utils\checkpoint.py", line 488, in checkpoint
    ret = function(*args, **kwargs)
          ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ComfyUI\custom_nodes\ComfyUI-3D-Pack\Gen_3D_Modules\InstantMesh\models\renderer\synthesizer_mesh.py", line 132, in get_geometry_prediction
    sdf, deformation, weight = self.decoder.get_geometry_prediction(sampled_features, flexicubes_indices)
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "D:\ComfyUI\custom_nodes\ComfyUI-3D-Pack\Gen_3D_Modules\InstantMesh\models\renderer\synthesizer_mesh.py", line 76, in get_geometry_prediction
    grid_features = torch.index_select(input=sampled_features, index=flexicubes_indices.reshape(-1), dim=1)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
torch.OutOfMemoryError: Allocation on device

My configuration from Comfy startup:
## ComfyUI-Manager: installing dependencies done.
** ComfyUI startup time: 2024-11-25 13:14:39.768854
** Platform: Windows
** Python version: 3.11.9 | packaged by Anaconda, Inc. | (main, Apr 19 2024, 16:40:41) [MSC v.1916 64 bit (AMD64)]
** Python executable: C:\Users\sipkode\miniconda3\envs\py311\python.exe
** ComfyUI Path: D:\ComfyUI
** Log path: D:\ComfyUI\comfyui.log

Prestartup times for custom nodes:
   1.1 seconds: D:\ComfyUI\custom_nodes\ComfyUI-Manager

Total VRAM 24564 MB, total RAM 65254 MB
pytorch version: 2.4.0
xformers version: 0.0.27.post2
Set vram state to: NORMAL_VRAM
Device: cuda:0 NVIDIA GeForce RTX 4090 : cudaMallocAsync
Using xformers cross attention

Re-installing diff_gaussian_rasterization as suggested above did not work...
Maybe I should try with the LOW_VRAM option?

Fixed this by reinstalling diff-gaussian-rasterization
pip uninstall diff_gaussian_rasterization pip install "git+https://github.com/ashawkey/diff-gaussian-rasterization.git"

Could you help please, I don't understand where exactly to enter these commands. Wherever I tried it says that it is not installed (diff-gaussian-rasterization)


pull request for e2e test at 01/06/2017 13:26:14

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Adding edge detection constructs

In the Filters namespace, a filter for edge detection can be added, preferably a canny edge detector.
As per the programming style, the expected syntax is
Bitmap bitmap = ...//Load Image

bitmap.Filter(new EdgeDetectingFilter());

The algorithm can be looked up at this Wikipedia article.

I might probably add it myself, but I won't be free for a few days. So any contribution would be greatly appreciated.


Support to monitor fs.access ?

should  the fs.access be supported ?
const fs = require('fs')
const fsp = require('fs/promises')

fs.accessSync('package.json')

for above test, it wont works now

Yeah, that's definitely reasonable to have. This project is pretty much a quick PoC for now, feel free to make PR and refactorings that you feel would be useful.


Update API

Fixes #https://github.com/viewthespace/shapeable/issues/2, but doesn't include an app level configuration.
@shaw3257 @CharlesMcMillan
We still need to update the README
Also, the error message should be more descriptive.

OMG SHAPEABLE

@shaw3257 I updated the README. Mind taking a look?

Also changed the version number as we discussed.


Sounds don't play when soundboard is "popped out"

Just installed and noticed that there are no sounds when I pop-out the soundboard.  When it is pop-in again, they work as normal.

Hi, I've not tested PopOut!, so will need to take a look and see if I can make it compatible.
Thanks for the feedback

Had a look at this - clicking a sound before popping out causes the sound to be available after popping out...
I was able to have the whole SoundBoard work after altering the popout code to stop it from stripping out a bunch of extra header information. I assume KaKaRoTo does this for performance reasons (otherwise everything is being re-loaded into the popout window)
My gut feeling is that SoundBoard is doing something wrong, and should be passing some more data into the Application that PopOut! will handle correctly - I'll try to find some time to have a chat with KaKaRoTo, but I'm aware he's very busy at the moment.

There's a new experimental option in 1.4.0 that should allow PopOut! to work, though it may cause some performance issues, and might not play nice with certain modules.
Give it a try and let me know if it does the job

Closing as no feedback received


Demo page doesn't work as expected content in IE11

Attach (recommended) or Link to PDF file here:

Configuration:

Web browser and its version: IE11
Operating system and its version: Windows 10
PDF.js version:
Is a browser extension:

Steps to reproduce the problem:

Open https://mozilla.github.io/pdf.js/legacy/web/viewer.html in IE11

What is the expected behavior? (add screenshot)
Pdf should be available for preview
What went wrong? (add screenshot)
Pdf is not available for preview
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
https://mozilla.github.io/pdf.js/legacy/web/viewer.html

As clearly mentioned in the FAQ, IE 11 is no longer supported; see https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#faq-support


chore: update phpstan-baseline.php

Description
Because of PHPStan upgrade.
Error: Ignored error pattern #^Return type \(CodeIgniter\\Database\\MySQLi\\PreparedQuery\) of method CodeIgniter\\Database\\MySQLi\\PreparedQuery\:\:_prepare\(\) should be covariant with return type \(\$this\(CodeIgniter\\Database\\BasePreparedQuery\<TConnection, TStatement, TResult\>\)\) of method CodeIgniter\\Database\\BasePreparedQuery\<mysqli,mysqli_stmt,mysqli_result\>\:\:_prepare\(\)$# in path /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/MySQLi/PreparedQuery.php was not matched in reported errors.
Error: Ignored error pattern #^Return type \(CodeIgniter\\Database\\OCI8\\PreparedQuery\) of method CodeIgniter\\Database\\OCI8\\PreparedQuery\:\:_prepare\(\) should be covariant with return type \(\$this\(CodeIgniter\\Database\\BasePreparedQuery\<TConnection, TStatement, TResult\>\)\) of method CodeIgniter\\Database\\BasePreparedQuery\<resource,resource,resource\>\:\:_prepare\(\)$# in path /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/OCI8/PreparedQuery.php was not matched in reported errors.
Error: Ignored error pattern #^Return type \(CodeIgniter\\Database\\Postgre\\PreparedQuery\) of method CodeIgniter\\Database\\Postgre\\PreparedQuery\:\:_prepare\(\) should be covariant with return type \(\$this\(CodeIgniter\\Database\\BasePreparedQuery\<TConnection, TStatement, TResult\>\)\) of method CodeIgniter\\Database\\BasePreparedQuery\<PgSql\\Connection,PgSql\\Result,PgSql\\Result\>\:\:_prepare\(\)$# in path /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/Postgre/PreparedQuery.php was not matched in reported errors.
Error: Ignored error pattern #^Return type \(CodeIgniter\\Database\\SQLSRV\\PreparedQuery\) of method CodeIgniter\\Database\\SQLSRV\\PreparedQuery\:\:_prepare\(\) should be covariant with return type \(\$this\(CodeIgniter\\Database\\BasePreparedQuery\<TConnection, TStatement, TResult\>\)\) of method CodeIgniter\\Database\\BasePreparedQuery\<resource,resource,resource\>\:\:_prepare\(\)$# in path /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/SQLSRV/PreparedQuery.php was not matched in reported errors.
Error: Ignored error pattern #^Return type \(CodeIgniter\\Database\\SQLite3\\PreparedQuery\) of method CodeIgniter\\Database\\SQLite3\\PreparedQuery\:\:_prepare\(\) should be covariant with return type \(\$this\(CodeIgniter\\Database\\BasePreparedQuery\<TConnection, TStatement, TResult\>\)\) of method CodeIgniter\\Database\\BasePreparedQuery\<SQLite3,SQLite3Stmt,SQLite3Result\>\:\:_prepare\(\)$# in path /home/runner/work/CodeIgniter4/CodeIgniter4/system/Database/SQLite3/PreparedQuery.php was not matched in reported errors.
Error: Ignored error pattern #^Return type \(CodeIgniter\\HTTP\\DownloadResponse\) of method CodeIgniter\\HTTP\\DownloadResponse\:\:noCache\(\) should be covariant with return type \(\$this\(CodeIgniter\\HTTP\\ResponseInterface\)\) of method CodeIgniter\\HTTP\\ResponseInterface\:\:noCache\(\)$# in path /home/runner/work/CodeIgniter4/CodeIgniter4/system/HTTP/DownloadResponse.php was not matched in reported errors.
Error: Ignored error pattern #^Return type \(CodeIgniter\\HTTP\\DownloadResponse\) of method CodeIgniter\\HTTP\\DownloadResponse\:\:noCache\(\) should be covariant with return type \(\$this\(CodeIgniter\\HTTP\\Response\)\) of method CodeIgniter\\HTTP\\Response\:\:noCache\(\)$# in path /home/runner/work/CodeIgniter4/CodeIgniter4/system/HTTP/DownloadResponse.php was not matched in reported errors.

https://github.com/codeigniter4/CodeIgniter4/actions/runs/7147604463/job/19467408417?pr=8310
Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[] User guide updated
[] Conforms to style guide

Please review first.

@paulbalandan Thank you!


APPS-1421 Update dir output for WDL 2.0

This PR is dependent on https://github.com/dnanexus/wdlTools/pull/235

All in principal in the scope - I'll modify accordingly


Unable to locate package miopenkernels

Hi,
When I run my training code, I saw a warning: "MIOpen(HIP): Warning [SQLiteBase] Missing system database file: gfx90878.kdb Performance may degrade. Please follow instructions to install: https://github.com/ROCmSoftwarePlatform/MIOpen#installing-miopen-kernels-package", so I was trying to download miopenkernels packages. By running this script: https://github.com/ROCmSoftwarePlatform/MIOpen/blob/develop/utils/install_precompiled_kernels.sh, the package cannot be found:
using rocminfo at /usr/bin/rocminfo
sudo apt install -y miopenkernels-gfx908-120pkg
Hit:1 https://packages.microsoft.com/ubuntu/18.04/prod bionic InRelease
Hit:2 http://archive.ubuntu.com/ubuntu focal InRelease
Hit:3 https://repo.radeon.com/amdgpu/22.10.1/ubuntu focal InRelease
Get:4 http://security.ubuntu.com/ubuntu focal-security InRelease [114 kB]
Hit:5 http://ppa.launchpad.net/criu/ppa/ubuntu focal InRelease
Get:6 https://repo.radeon.com/rocm/apt/debian ubuntu InRelease [2,603 B]
Hit:7 http://archive.ubuntu.com/ubuntu focal-updates InRelease
Hit:8 http://archive.ubuntu.com/ubuntu focal-backports InRelease
Get:9 https://repo.radeon.com/rocm/apt/debian ubuntu/main amd64 Packages [26.3 kB]
Fetched 143 kB in 1s (143 kB/s)
Reading package lists... Done
Building dependency tree
Reading state information... Done
10 packages can be upgraded. Run 'apt list --upgradable' to see them.
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package miopenkernels-gfx908-120pkg

My image uses rocm/dev-ubuntu-20.04:5.1.1-complete as base image. Is there any missing apt repository for miopenkernels packages?
Thanks

Most likely the pre-compiled kernels are not uploaded to the public place for downloads.
@cderb  do you know how to ping Zhongyu for this issue?

@junliume If it's not public, should the doc update this?

@xiaoyu-work  could you try apt install -y miopenkernels-gfx908-120kdb?
@JehandadKhan It seems there could be an issue with https://github.com/ROCmSoftwarePlatform/MIOpen/blob/develop/utils/install_precompiled_kernels.sh
or something has changed and this script is not updated


Need an OWNERS file for openshift-ci

We should add an OWNERS file to comply with reqs of using openshift-ci.

Added a PR for the owners file.


Add py.typed files to released packages

Is your feature request related to a problem? Please describe.
The current package does not include the google/cloud/secretmanager/py.typed file. As such, mypy (and other type checkers) consider this package to be untyped (as specified in PEP 561):
# venv & package install
$ python3 -m venv venv && source venv/bin/activate
$ pip install google-cloud-secret-manager mypy
...
Successfully installed cachetools-4.2.1 certifi-2020.12.5 chardet-4.0.0 google-api-core-1.25.1 google-auth-1.24.0 google-cloud-secret-manager-2.2.0 googleapis-common-protos-1.52.0 grpc-google-iam-v1-0.12.3 grpcio-1.35.0 idna-2.10 libcst-0.3.16 mypy-0.800 mypy-extensions-0.4.3 proto-plus-1.13.0 protobuf-3.14.0 pyasn1-0.4.8 pyasn1-modules-0.2.8 pytz-2021.1 pyyaml-5.4.1 requests-2.25.1 rsa-4.7 six-1.15.0 typed-ast-1.4.2 typing-extensions-<IP_ADDRESS> typing-inspect-0.6.0 urllib3-1.26.3

# no py.typed file in google/cloud/secretmanager
$ find . -iname "py.typed"
./venv/lib/python3.8/site-packages/mypy/py.typed
./venv/lib/python3.8/site-packages/libcst/py.typed

# mypy complains about the untyped library
$ echo "from google.cloud.secretmanager import SecretManagerServiceClient" >> main.py
$ mypy main.py
main.py:1: error: Skipping analyzing 'google.cloud.secretmanager': found module but no type hints or library stubs
main.py:1: note: See https://mypy.readthedocs.io/en/latest/running_mypy.html#missing-imports
Found 1 error in 1 file (checked 1 source file)

# adding the py.typed file in both secretmanager/ and secretmanager_v1/ , mypy doesn't complain anymore
$ touch venv/lib/python3.8/site-packages/google/cloud/secretmanager{,_v1}/py.typed
$ mypy main.py
Success: no issues found in 1 source file

# dummy test case to check mypy finds errors
$ echo "SecretManagerServiceClient.parse_secret_path(1)" >> main.py
$ mypy main.py
main.py:2: error: Argument 1 to "parse_secret_path" of "SecretManagerServiceClient" has incompatible type "int"; expected "str"
Found 1 error in 1 file (checked 1 source file)

Describe the solution you'd like
Have the packages on pypi contain

google/cloud/secretmanager/py.typed
google/cloud/secretmanager_v1/py.typed
google/cloud/secretmanager_v1beta/py.typed

Describe alternatives you've considered
Have mypy ignore those imports, but it's a bit of a shame since they're typed.
Versions
python: 3.8.5
pip: 20.0.2
google-cloud-secret-manager: 2.2.0
mypy: 0.800
Thanks

These have been included (see linked issues) and are here:
https://github.com/googleapis/python-secret-manager/blob/main/google/cloud/secretmanager/py.typed
https://github.com/googleapis/python-secret-manager/blob/main/google/cloud/secretmanager_v1/py.typed
https://github.com/googleapis/python-secret-manager/blob/main/google/cloud/secretmanager_v1beta1/py.typed
Closing out, thank you for filing!


Git Error When Updating fzf-lua

Info

Operating System: Ubuntu 22.04

Shell: Bash

Terminal: Alacritty

nvim --version: NVIM v0.10.0-dev
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3

fzf --version: 0.29 (devel)

[ ] The issue is reproducible with mini.sh

fzf-lua configuration

In my plugins.lua
  use { 'ibhagwan/fzf-lua',
    -- optional for icon support
    requires = { 'nvim-tree/nvim-web-devicons' }
  }

In init.lua
require('fzf-lua').setup({'fzf-vim'})

Description
When updating fzf-lua with :PackerUpdate I get an error
 ✗ Failed to update ibhagwan/fzf-lua
  Errors:
    fatal: Not possible to fast-forward, aborting.
    5f1f4b2

This means you probably caught a rare force push I did, you have to either delete the plug-in and reinstall or change your package manager to git pull --rebase instead.

ref: https://github.com/wbthomason/packer.nvim/issues?q=pull+rebase++
Btw, highly recommend switching packer to lazy, last commit on packer was 7 months ago…

I went to the plugin directory and pulled with rebase and it's fixed, I'll keep in mind about packer/lazy as well, thank you.

I went to the plugin directory and pulled with rebase and it's fixed, I'll keep in mind about packer/lazy as well, thank you.

Great, ty for the update.


Reuse an already successful proxy on next requests

RequestHedgingProxyProvider doesn't reuse the successful proxy to send
the next request, but instead it broadcast the latter to all namenode.

LGTM


Multivariate regression

Draft PR
Started working on adding multivariate regression support
Not finished yet

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Does your contribution introduce a new dependency? If yes, which one?

PR checklist for new estimators

Any other comments?

Thanks!
Does this relate to an issue?
If not, would you be able to outline what you are planning?
We may also want to align on an interface design first, to avoid incompatibility between workstreams.
We have a tentative API design proposal for multivariate:
https://github.com/alan-turing-institute/sktime/wiki/Design-proposal:-forecasting-API-re-work-including-support-for-datetime-indices
(but we haven´t started with implementation yet, since the design still needs hammering out)

@fkiraly @SebasKoel got involved with sktime during the PyData sprint and is now in the mentorship programme and mentored by @big-o. 🙂

Ah, nice to make your acquaintance, and welcome onboard!
:wave:
(not sure whether I saw you on slack)

Please merge into master (see #360)

Closed in favour of #390


Unnecessary args not being utilised in main.tf.

@syedrakib I believe we have a PR in review that has these changes
https://github.com/terraform-google-modules/terraform-google-sql-db/pull/100/

@syedrakib I believe we have a PR in review that has these changes #100

i see. Ok. Any word on the ETA?

Closing this PR as currently being addressed in #100


streaming/setup.sh fails to recognize iot hub in subscription despite azure portal showing it has been created

Module: DP-900
Lab/Demo: 05 - Streaming Analytics
Task: 00
Step: 00
Description of issue: streaming/setup.sh fails to recognize iot hub in subscription despite azure portal showing it has been created.
Steps to issue:

I am using the Azure Free Subscription - and am following the steps in this streaming analytics exercise
I ran bash setup.sh  and ran into the below error

Creating IoT hub...
ERROR: Unable to find IoT Hub: iothubb7fd109c1c2b488b88 in current subscription 30bxxxxxxxxxxxxxx.
Unable to find IoT Hub: iothubb7fd109c1c2b488b88 in current subscription 30bxxxxxxxxxxxxx.
ERROR: Unable to find IoT Hub: iothubb7fd109c1c2b488b88 in current subscription 30bxxxxxxxxxxxx.

Appears to be due to line 25 in the setup.sh

marc [ ~/dp-900/streaming ]$ az iot hub connection-string show --hub-name iothubb7fd109c1c2b488b88 --output tsv
Unable to find IoT Hub: iothubb7fd109c1c2b488b88 in current subscription 30bxxxxxxxxxxxxxx.

What's more confusing is when I click on the iothub7fd... it says it is part of the azure subscription:

I thought this might have originally had something to do with the resource provider registration, but the device is registered (and luckily MSFT would have thrown a different error)
Moreover, if this isn't tied to this subscription - what is it tied to?

looking into this issue thanks for flagging

I also experience this problem.
Not sure about the Authors problem, but for me, I was able to resolve this by making sure that the resource group is created somewhere near me.
The setup.sh script explicitly creates the resource group in the location westus. Me being in europe, if I change it to westeurope everything works fine.
I guess this has to do with the azure portal taking some time to propagate changes made in the us region when querying from a European region. A fix could be to get the region of the cloud shell and creating the rg in that region or a region nearby

Hi
I also experienced this issue today, and managed to workaround it using @mettke 's suggestion. E.g. changing lines 13 and 34 to a local region (uksouth in my case) allows the script to work fine.
Thanks.

The issue has been fixed. We changed the setup script to choose a region from a random list, rather than staying fixed with westus. This worked when testing, eliminating all errors.


Switch not working with strict enums enabled

switch() doesn't seem to work when combined with BETTER_ENUMS_STRICT_CONVERSION. This is because it's using BETTER_ENUMS_ENUM_CLASS_SWITCH_TYPE_GENERATE, which creates a "copy" scoped enum with the same enumerators. The problem is that enumerators from _enumerated (which are also specified in each case section) cannot be compared to the like-named enumerators in the "copy" version.
Any reason that in the strict case, the implicit typecast operator in the Enum class isn't just returning _enumerated and not this dummy scoped enum? Oddly though, the code compiles fine on MSVC 14 but on Android NDK GCC 4.9, it does not work:
MSRData.cpp: In constructor 'CMSRData::CMSRData(const char*, MsrEncryptionStatus)':
MSRData.cpp:42:36: error: could not convert 'Ksn' from 'MsrEncryptionStatus::_enumerated' to 'better_enums::_data_MsrEncryptionStatus::_EnumClassForSwitchStatements'
          case MsrEncryptionStatus::Ksn:
                                    ^
MSRData.cpp:51:36: error: could not convert 'None' from 'MsrEncryptionStatus::_enumerated' to 'better_enums::_data_MsrEncryptionStatus::_EnumClassForSwitchStatements'
          case MsrEncryptionStatus::None:
                                    ^
MSRData.cpp:55:36: error: could not convert 'Dukpt' from 'MsrEncryptionStatus::_enumerated' to 'better_enums::_data_MsrEncryptionStatus::_EnumClassForSwitchStatements'
          case MsrEncryptionStatus::Dukpt:
                                    ^

Any thoughts on this? ATM I have to do:
switch (static_cast<SynapseSubmissionStatus::_enumerated>(m_status._to_integral()))

Yes, you have to follow these instructions and use + in your cases:
switch (bar) {
    case +Foo::A: break;
    case +Foo::B: break;
    case +Foo::C: break;
}

Any reason that in the strict case, the implicit typecast operator in the Enum class isn't just returning _enumerated and not this dummy scoped enum?

Yes, that is to avoid implicit conversions to _enumerated, which then allows implicit conversions to integers. The purpose of STRICT_CONVERSION is not to allow that.
As mentioned elsewhere, there is also the possibility to add a + operator to the enum instead, so it would look like:
switch (+bar) {
    case Foo::A: break;
    case Foo::B: break;
    case Foo::C: break;
}

That would do your typecast.

Sorry for the late response. Unfortunately your first example doesn't compile on MSVC 2015 Update 3. However it does work like this:

   auto command_from_string = PaymentServiceCommand::_from_string_nocase_nothrow(
      context.GetArgByKey("command").c_str());

   // If this argument is unspecified, assume we are processing the next device
   // interaction as a payment
   auto command = +PaymentServiceCommand::MakePayment;

   if (command_from_string)
   {
      command = *command_from_string;
   }

   switch (command)
   {
      case PaymentServiceCommand::GetSourceData:
      case PaymentServiceCommand::MakePayment:
      {
         SetEnabled(true, command);
         break;
      }

      case PaymentServiceCommand::StopService:
      {
         SetEnabled(false, command);
         break;
      }

      default:
      {
         LOG_ERROR << "Unknown command provided to PaymentServiceHandler";
         break;
      }
   }


Graphics drawRoundedRect method radius parameter

http://jsfiddle.net/adireddy/qoxsr8t2/
In graphics.drawRoundedRect method, If I set the radius value with half the height (30 in this case), it's not working.
graphics.drawRoundedRect(10, 90, 200, 60, 30); //not working`
graphics.drawRoundedRect(10, 90, 200, 60, 20); //working`

Uncaught TypeError: Cannot read property 'length' of null

Heres another fiddle: http://jsfiddle.net/f1gm72wa/
The error is gone without fill, but the rounded rect is not being drawn correctly if the radius is too big.

I also have this on v2.2.9

Is this both canvas and webgl?

webgl only.

This will get fixed by using #1772, this was a bug in PolyK.

:+1:

Is there any update as to whether anyone is working on a fix for this issue? I am still seeing it in 3.0.9.


not work in my Xcode7.1.1

I use  Alcatraz install , but not succeed,  and i  immediate download ,and install ,but not succeed, and i see the UUID is the same , why my xcode is not succeed

+1

xcode7.2 work fine

No for Swift

Delete old plugin, reinstall, restart Xcode，then Xcode will show a dialog asking you load plugin bundle or not, be sure click "load".

I am closing this since you can find more information in Xcode Version of README. Try to reset the "Load Bundle" option if you happened to forbid third party bundle.


[Discussion] Converting GAPIC based libraries to TypeScript

Greetings folks!  I wanted to start a conversation about converting our GAPIC based libraries to TypeScript, and figured this was as good a spot as any :)   Just to get the ball rolling, I tried converting the nodejs-logging repository over to TypeScript.  You can see the initial results (with most of the tests passing) here:
https://github.com/googleapis/nodejs-logging/pull/111
I think if we want to do this, we need to address the following issues:

[ ] Eliminate the lazy constructor pattern.  We've discussed this at great length, but before doing anything, we need to make the breaking change to get rid of this pattern:

function Logging(options) {
  if (!(this instanceof Logging)) {
    return new Logging(options);
  }
  ....
}

We can make this change today, independent of the TypeScript transition. It will likely be semver major.

[ ] Create a TypeScript GAPIC generator..  I was really hopeful that I could use the [allowJs](https://www.typescriptlang.org/docs/handbook/compiler-options.html) compiler flag in TypeScript to fake out the generated code.  Sadly, you cannot mix the allowJs: true and declaration: true flags in tsconfig.json.  This means that we can't have a d.ts file, which really eliminates half the TypeScript appeal. By creating a separate TypeScript focused generator, we can also upgrade our repositories one at a time, ensuring we do things slowly and carefully.

[ ] Do the normal TypeScript conversion dance After we've fixed the constructor pattern issue, and generated a TypeScript focused GAPIC layer, the rest is well understood.

[ ] Convert all *.js files to *.ts files
[ ] Install typescript and gts, get the initial compilation passing
[ ] Convert var to let and const
[ ] Convert to es6 classes
[ ] Add initial types where possible
[ ] Enable noImplicitAny
[ ] Enable noImplicitThis
[ ] Eat cake 🍰

I'm really just putting all this here as a conversation starter.  Thoughts? Concerns? Opinions?
@GoogleCloudPlatform/node-team @alexander-fenster @theacodes @ofrobots @crwilcox

Missed a step:

[ ] - Can generate API reference documentation (to be published on cloud.google.com) for libraries written in TypeScript. Current docs

I personally am a big fan of TypeScript. I have used it quite a bit and
find it does help as the code base expands and helps to guard against
coding mistakes.
Right now I can think of two questions:

Since I don't have the background, is there data to support doing this
work? Are users asking for d.ts files today? I am guessing yes as there are
some typings files out in the wild today.
Is there a benefit to making a new TypeScript generator that we wouldn't
get from extending the generator to emit a .d.ts file to go with the JS we
generate today? As we provide libraries, at least on the surface, it seems
to me providing .d.ts files is what client library users need. But maybe
there is a pro to full conversion I am glossing over?

I think for the handwritten clients could benefit from transitioning to
TypeScript, but I am not sure the generator output matters as much.
On Sun, May 13, 2018 at 1:51 AM Jason Dobry<EMAIL_ADDRESS>wrote:

Missed a step:

Can generate API reference documentation (to be published on
cloud.google.com) for libraries written in TypeScript. Current docs
http://cloud.google.com/nodejs/docs/reference/libraries

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-node/issues/2821#issuecomment-388603415,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAm_TfSA7eVjrUBInfMEC6ibJZSp-nn2ks5tx8nRgaJpZM4T8g6N
.

I'm starting to think it might be a good idea to connect this request and the in-language generator initiative by @lukesneeringer (see the Python generator for example). I would say, regardless of converting the existing GAPIC generator templates to TypeScript, if we fund the in-language generation for JavaScript, it should definitely emit TypeScript code.

@alexander-fenster @lukesneeringer @timburks just wanted to check - any chances of TypeScript landing in the generator in Q3?  We are getting a lot of customer demand.

We have a plan going here 👍


Recursive random photo is not recursive

Describe the bug
Files tree in the PiGallery (with /app/data/images/ as root index folder) :
/app/data/images/galleries/
/app/data/images/galleries/0.jpg
/app/data/images/galleries/dir1/
/app/data/images/galleries/dir1/1.jpg
/app/data/images/galleries/dir2/
/app/data/images/galleries/dir2/2.jpg
Expected

With http://mygallery.local/api/gallery/random/ : get alternatively 0.jpg, 1.jpg or 2.jpg when refreshed
With http://mygallery.local/api/gallery/random/?directory=galleries : get 0.jpg even when refreshed 10 times
With http://mygallery.local/api/gallery/random/?directory=galleries&recursive=true : get alternatively 0.jpg, 1.jpg or 2.jpg when refreshed

Reality

OK
OK
Got only 0.jpg
3.bis If I delete 0.jpg, I get the following error :
{"error":{"code":12,"message":"No photo found","detailsStr":""},"result":null}
I get the same error with 2. after 0.jpg deletion (but it is expected in this case).
So it seems the recursive parameter does not work or I missed something.

Additionally (you may want me to create another bug issue for this), the "&recursive=true" in the URL field is not updated when I switch "recursive" to on/off in the Random Link menu in the GUI. It is updated when I change another field (Date or Resolution) though.
Environment

Latest docker version without nginx proxy
Browser Firefox and Chrome

Thanks for the report. I'll look into it.

The last commit should solve this.


WebXR mode does not time out when it fails

Description

Go to https://modelviewer.dev/ with affected device
click the AR button for the default model
Console logs: Attempting to present in AR with WebXR...
Nothing happens. It never times out to try the SceneViewer. Clicking the AR button again does not do anything.
When tested with SceneViewer as a first option it works.
Expected:
Underlying issue is related to Samsung (or Chrome), but it should still fail gracefully and try SceneViewer in case there's some issue with WebXR.
Problem will be somehwere here, that this await never times out: https://github.com/google/model-viewer/blob/4a313508895ebac912e95d81f0c6f972dcaba2ad/packages/model-viewer/src/features/ar.ts#L278

Live Demo
https://modelviewer.dev/
Version

model-viewer: v3.4.0
Tested also with 3.5.0 and occurs as well

Browser Affected

[x] Chrome, version: 124.0.6367.82

OS

[x] Android v14
Replicated on Samsung S24+ and S22+
Affects OS with One UI 6.1. Tested on one device, it started occuring ater updating from One UI 6.0 to 6.1.

AR

[x] WebXR

At the moment I only have Pixel devices on hand, but I can confirm that the Pixel 6, 7, and 8 are not seeing this regression on Android 14, so it's likely something to do with the Samsung software like @petr-hajek theorized.
That said, there's a few different places in the ARRenderer that the await could be getting hung up on, and it would be helpful to know which is was. Some would be more practical to put timeout logic around than others.
rAF: https://github.com/google/model-viewer/blob/4a313508895ebac912e95d81f0c6f972dcaba2ad/packages/model-viewer/src/three-components/ARRenderer.ts#L186
Session: https://github.com/google/model-viewer/blob/4a313508895ebac912e95d81f0c6f972dcaba2ad/packages/model-viewer/src/three-components/ARRenderer.ts#L207
Reference Space: https://github.com/google/model-viewer/blob/4a313508895ebac912e95d81f0c6f972dcaba2ad/packages/model-viewer/src/three-components/ARRenderer.ts#L219

Can you also provide the full chrome://version string including the bitness of Chrome? (32 vs 64)?

@alcooper91 Chrome: 124.0.6367.113 (Official Build) (64-bit)


Update Composer dependencies (2023-06-26-00-12)

Visual regression test passed!
View the visual regression test report


Stripe for frontend. SMS and email service also implemented

Stripe for payment, Twillio for SMS and sendgrid for email implemented


Support Redis service with sentinel mode

Some changes may need to be done for supporting Redis service with sentinel mode.

duplicated with epic: https://github.com/goharbor/harbor-operator/issues/158


bytebuf reference count bug?

Expected behavior
private int addComponent0(boolean increaseWriterIndex, int cIndex, ByteBuf buffer) {
    assert buffer != null;
    boolean wasAdded = false;
    try {
        checkComponentIndex(cIndex);

        int readableBytes = buffer.readableBytes();

        // No need to consolidate - just add a component to the list.
        @SuppressWarnings("deprecation")
        Component c = new Component(buffer.order(ByteOrder.BIG_ENDIAN).slice());
        if (cIndex == components.size()) {
            wasAdded = components.add(c);
            if (cIndex == 0) {
                c.endOffset = readableBytes;
            } else {
                Component prev = components.get(cIndex - 1);
                c.offset = prev.endOffset;
                c.endOffset = c.offset + readableBytes;
            }
        } else {
            components.add(cIndex, c);
            wasAdded = true;
            if (readableBytes != 0) {
                updateComponentOffsets(cIndex);
            }
        }
        if (increaseWriterIndex) {
            writerIndex(writerIndex() + buffer.readableBytes());
        }
        return cIndex;
    } finally {
        if (!wasAdded) {
            buffer.release();
        }
    }
}

Actual behavior
executed this line "buffer.release();"
Steps to reproduce
called  "Unpooled.wrappedBuffer(headerBuf, metaBuf);"
Minimal yet complete reproducer code (or URL to code)
Netty version
4.1.30.Final
JVM version (e.g. java -version)
OS version (e.g. uname -a)

@wenweihu86 can you please add a unit test that shows the problem ?

@normanmaurer unit test can not recurrent the problem.
it has very small probability to happen under large pressure test.

@wenweihu86 then you will need to give me more details... I dont understand what the problem is.

my code is as following:
ByteBuf outBuf = Unpooled.wrappedBuffer(headerBuf, metaBuf, bodyBuf);
ctx.channel.writeAndFlush(outBuf);
and then it throws exception at this line:
CompositeByteBuf.java:285
the exception is
io.netty.util.IllegalReferenceCountException: refCnt: 0, decrement: 1

@wenweihu86 could you provide a bit more detail than that.. specifically

how are headerBuf, metaBuf, bodyBuf created, do more than one of them derive from a common buffer?
are any of them or any buffer they are derived from used in any way after being wrapped?
a more complete stacktrace of the exception

@njhill headerBuf, metaBuf, bodyBuf is created as following:
ByteBuf headerBuf = Unpooled.buffer(12);
ByteBuf metaBuf = Unpooled.wrappedBuffer(new byte[128]);
ByteBuf bodyBuf = Unpooled.wrappedBuffer(new byte[256]);
ByteBuf outBuf = Unpooled.wrappedBuffer(headerBuf, metaBuf, bodyBuf);
ctx.channel.writeAndFlush(outBuf);
outBuf may be send multi times when need to retry rpc request.
originally, i forgot to retain the outBuf;
latterly, I have retain the outBuf when send outBuf repeatedly.
I confused why it throws IllegalReferenceCountException at the following line code:
if (!wasAdded) {
buffer.release();
}

@wenweihu86 so it sounds like you found the source of the issue and have now resolved it (writing multiple times without retaining), is that correct?

I confused why it throws IllegalReferenceCountException at the following line code

I can't see how that could happen given the code example you provided. It is likely you are referring to some different code where you wrap the same buffer(s) more than once without retaining them first.

@njhill Yes, I have solved the problem.
But I just don't know why it crashed at that line, maybe as your said reason.

closing


Run dump and outline actions offline

Build does not need access to AWS when run with dump or
outline options so make it run offline for those actions.

I somehow messed up this PR, replacing it with PR https://github.com/remind101/stacker/pull/594


Parameter name is incorrect

Field name of PostMessageParameters.Username is "username", not "user_name".
https://github.com/nlopes/slack/blob/master/chat.go#L34

will gladly accept a PR making the change. nice catch.

thank you for your reply :)
check my change, please.

@mythrnr sorry just seeing this. the change looks good, mind actually submitting a pull request?

@james-lawrence I think that this change was merged, but isn't it?
https://github.com/nlopes/slack/pull/337

ah, sorry =) i was wondering why this was still open and thought I imagined that we had merged it.


Incremental writing?

I don't know if this is something typical to do, but I think it could kind of make sense? It seems at least like the MIDI file format should lend itself perfectly to writing incrementally to a MIDI file, if I am not mistaken. I could help develop it, I just don't know the best API.

What I do is I incrementally build the tracks and then I create the SMF and save it.
Any reason why that wouldn't be sufficient? Are you concerned about RAM usage?
(Are you processing multi-GB "black midi" files?)
My main issue is the ergonomics of constructing the track data, because I can't just append an owned sysex or MetaMessage, because they only borrow: https://github.com/negamartin/midly/issues/7#issuecomment-632147145

What I want to do is to forever listen to MIDI to save whatever is going on always. Could be into multiple midi files or one big file. Anyhow, I think it is best to write to the MIDI file periodically in order to not lose any data. Every minute or so. That's why I'm looking into incremental writing. But maybe it's unnecessary to worry about losing data in my case... and I suppose I could as well write the entire file ever so often to backup the "WIP" midi.
I thought track chunks could be interleaved, looking briefly to the MIDI file specification.

@Ploppz In SMF type 1, the tracks have to be written one after another. In SMF type 0, all events appear on a single track, but the size of that track data is still stored before it.
In theory you COULD implement a streaming writer for type 0, which under the hood also seeks back to update the size field, like the streaming wav writer in the hound crate (for wav it makes sense since the data is large).
BUT: Unless you're dealing with a "black midi" stream, it doesn't really require incremental writing:
From experience, 5 minutes of normal midi song material is ~64 kb. That'd be 768 kb per hour. Surely you have enough RAM to spare? :) (Depending on your definition of "forever".)
I'm interested in your use case, what are you doing with midi? :)
In my application I store the recorded midi in RAM and save it every once in a while (atomically, using atomicwrites).
Right now I think there are more important things to spend time on.. Like the above mentioned ergonomic improvements for writing midi files.
And then https://github.com/negamartin/midly/issues/8

I see, thanks for correcting me!

And thanks for your experience with midi file size.
I'm writing a daemon that listens for midi inputs and saves everything. That way I can go improvise on piano without worrying about forgetting it or recording it if I do something good :D

@Ploppz Ah, in that case, your kb/min will be even less (and zero when not playing), because you will only have 1 channel of data, and very few non-note msgs. In my mix above, at song transitions, all channels contain msgs to switch (i.e. serialize the diff of) their state (prog chg, CCs, pitch bend etc.) due to how my midi mixer works, but you won't have that data..


 The comments are addressed by this commit. For override_threadpool_test, please find my answers here.

Just rebase this PR to solve the conflicts in tensorflow/python/data/kernel_tests/flat_map_test.py   with this line

@jsimsa Thanks for your suggestion! The test cases for testNumThreadsDeprecated are updated here. Could you please take a look?

@rthadur The failed test in Ubuntu CPU is not related. An error happened when migrating the change to the internal tests. Could you please help retrigger the internal tests?

@feihugis thank you very much for porting these test to use TF combinations!

@jsimsa Thanks a lot for your help with my PRs! Please feel free to let me know if there is anything else I can help with.