No validate kernels

Same as https://github.com/microsoft/vscode-python/pull/13326 (slightly refactored).
Was unable to re-open that PR.
For #12189
Validation moved to when we run a cell.

The kernel information returned to VSC from kernel provider now returns a stateless class
No more validation is performed when a kernel is changed (this happens when executing code)
Cleaned up IKernel interface
Move all execution of code & managing state of cells and notebook into IKernel class

Hence deleted the old ExzecutionService

TODO (will probably file issues for these)

[ ] Update trust code to create new interface/function to check if a VSC Notebook is trusted, instead of relying on INotebookModel, else this old interface will never go away even though we want to stop using this.

Codecov Report

Merging #13402 into master will decrease coverage by 0.03%.
The diff coverage is 13.33%.

@@            Coverage Diff             @@
##           master   #13402      +/-   ##
==========================================
- Coverage   59.94%   59.91%   -0.04%
==========================================
  Files         670      670
  Lines       37186    37197      +11
  Branches     5309     5310       +1
==========================================
- Hits        22293    22285       -8
- Misses      13771    13773       +2
- Partials     1122     1139      +17

Impacted Files
Coverage Δ

src/client/datascience/notebook/helpers/helpers.ts
40.88% <0.00%> (-0.93%)
:arrow_down:

src/client/datascience/notebook/types.ts
100.00% <ø> (ø)

...datascience/notebookStorage/nativeEditorStorage.ts
67.47% <0.00%> (-0.67%)
:arrow_down:

src/client/datascience/types.ts
100.00% <ø> (ø)

...science/notebookStorage/notebookStorageProvider.ts
50.74% <40.00%> (-0.87%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/datascience/crossProcessLock.ts
79.41% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

... and 4 more

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 032ebce...911cbc2. Read the comment docs.


Contribution guidelines

This PR adds contribution guidelines (including COC and style guide) to the README.

Merging despite failing CI since this branch doesn't include the travis configuration.


CHRTOUT test files are not the same generated on Cheyenne as with development machine

Comparing CHRTOUT and CHRTOUT.TRTE test files in
/test/input/geo/NWM_2.1_Sample_Datasets/Pocono_TEST1/example_CHRTOUT
produced exact parity in the time series (i.e., the values in the CHRTOUT and CHRTOUT.TRTE files are exactly the same) in local testing.
Comparing to these same outputs when executing t-route on Cheyenne, there are small but significant differences and it appears that there is some kind of incompatibility in the combination of netcdf libraries and xarray as they are applied in t-route.
The test files in the repository were developed using WRF-Hydro on that platform and it is possible that if the test files were reproduced on Cheyenne, they would generate identical output again.
The NetCDF library on the development environment is <IP_ADDRESS> while on Cheyenne the testing has been attempted with version 4.7.4

@hellkite500 @rcabell

The error seems to happen here:
https://github.com/NOAA-OWP/t-route/blob/381ac6827cff606720da7812dd50a74216148e9c/src/python_framework_v02/troute/nhd_io.py#L337
We ran a sequence of steps -- just prior to writing, the datastream on both systems is identical (and identical to the expected output)
We tested this with dataset_list[0]["streamflow"].to_series().to_csv("APD_stream_0200.csv") executed in the debugger right at that line.
This produces csv files that can be compared with the expected output and the comparison shows that the values are identical. On Cheyenne, however, when the write happens, the resulting netcdf is no long identical.

Ultimately, it appears that the problem is exactly what was originally hypothesized -- the globbing sequence was not bringing in the files in the same order. As a temporary fix, there is a sorted call wrapping the glob and this will be augmented with the explicit file list in the V3/looped execution mode.


Make a good documentation on the use of selectors and counters

Hey, I am trying to figure out how to use the counters, specifically the pokemon counter. Can you add a demo asap?

Yeah, I can make a small demo shorty.

Here is a link to the demo on stackblitz.
The basic documentation can be found on the readme.
Do you have suggestions on how to improve the documentation? I am working on making a demo app that uses all selectors and counters.


Feature Request: Reference tracking

Extools had an ability to get forwards and backward references to an object, for debugging hard Dels.
It would be wonderful if auxtools had this same functionality.

The issues with this in extools were:

memory usage, it can't stay in ram (at least in the byond address space)
detecting deleted lists/datums. The code extools hooked to do this is not always used by byond, as various call sites to it have been inlined by the compiler

Fixing the latter is hard or impossible


Interrupts/registers do not match the H7 datasheet

The block reference manual (RM0433) lists different interrupts and registers than the current driver expects.
Found so far:

RM044 section 56.5.16 FDCAN_IE is different than what src/interrupt.rs lists.
RX FIFO (RXFnC.FnWM) watermark is never set which means it will default to 0. Any RAM index greater than 0 will be reported as 'overrun'.

These are differences in the STM32H7 / STM32MP1 FDCAN controller that aren't captured by the feature gates. Thanks for listing these two - have you found any more?

So for not, but I'm sure there are more.

RM044 section 56.5.16 FDCAN_IE is different than what src/interrupt.rs lists.

Fixed by #14

RX FIFO (RXFnC.FnWM) watermark is never set which means it will default to 0

I think that should be fixed in the H7 HAL implementation, specifically here

So fun fact should anyone come across this, but on G4 you need to reverse the interrupt lines in both the NVIC name and when enabling the line; this error goes all the way back to ST's reference manual. (Works as expected on H7)


Dropdown menu

Anchor tag inside dropdown menu is not working, it doesn't link to the href when clicked on .
it's only add  two classes like this ("active selected") to the anchor .
please advise .
Thanks for your help.

I just tested links are working fine in dropdowns. There was a bug where they did not work in sub-menus before 1.8 #1638
Please remember when posting bugs to add a test case, be as descriptive as possible, and to always search for issues before reporting.

Sorry for not being descriptive as possible .
here is a test case https://facebookstore2.shopgo.io/myapp/test.dev.shopgo.io/en
please advise .
Thanks for your help.

Your page isn't loading for me.
Please publish a reduced test case, that only shows the aspects of your page that are broken.
You can fork a link to create a JSFiddle from our readme.


DeployLog: jenkins toggle is wrong.

It resolved to true for ServiceLayer deploys but not for any of the other. Isn't this column obsolete by now?
@iliqportals can't we just remove it?

Yea, it was intended only for that use, jenkins
we can remove it


Adminpanel userlist is empty

Bug Report
After the newest version, I created the first user. I tried to view the admin panel and the userlist was empty. I was looking in the docker volumes a databasefile is created and more than 0 bytes. I shutdown the stack and deleted the file, cleared the cache from the browser and wanted to start over. I created a user again, but same result.
If I want to view the user list it is empty. When I want to go back I am logged out. It does not matter if I use lowercases or uppercases.
Environment

Operating System: [e.g., Windows 10, macOS Big Sur, Ubuntu 20.04]

Ubuntu server 22.04 latest docker via ofgicial ppa

Browser (if applicable): [e.g., Chrome 100.0, Firefox 98.0]

Latest Firefox and brave
Reproduction Details
Confirmation:

[ x] I have read and followed all the instructions provided in the README.md.
[x ] I have reviewed the troubleshooting.md document.
[ ] I have included the browser console logs.
[ ] I have included the Docker container logs.

Installation Method
[Describe the method you used to install the project, e.g., manual installation, Docker, package manager, etc.]
Docker container, volume for ollama web-ui is added
Docker-compose.yml appended
dockercompose.txt
Additional Information
[Include any additional details that may help in understanding and reproducing the issue. This could include specific configurations, error messages, or anything else relevant to the bug.]
Note
If the bug report is incomplete or does not follow the provided instructions, it may not be addressed. Please ensure that you have followed the steps outlined in the README.md and troubleshooting.md documents, and provide all necessary information for us to reproduce and address the issue. Thank you!

Hi, Thanks for creating this issue! That seems very strange, could you verify that you're on the latest version by pulling the latest container? Keep us updated!

Here's what it should look like

I tried again, the docker volume /docker/ollama/ollama-webui:/app/backend/data is for the web ui, the other one is for ollama itself, storing the modelfiles.
After retrying the problem remains. I append the output of dockerlogs. I am using the web ui behind a reverse proxy, if this helps.
logs.txt
It looks like this for me

The db file is created

Oh, I meant I think you might have to replace /docker/ollama/ollama-webui:/app/backend/data with ollama-webui:/app/backend/data. Are there any reasons for why you're using /docker/ollama/ollama-webui:/app/backend/data for the volume mount?

Yes I always use bind mounts in every docker project for easier backup, instead of docker volumes. /docker is easier to backup instead of /var/lib/docker/overlay2/64c9c0cf8c9cfb0e2168071df0652a317d49f58a68fe86e4a9a9a525ab9e365e/webui
And I always need my network settings, because the reverse proxy is not running in docker. I shorted the path to /docker/ollama/webui now, but no change
Are there any special permissions for the db file? It is created and owned by root, which is normal and it seems to store data

Tried to use the docker volume

Getting the same error in the logs

The log you see is just a warning log 😅 it shouldn't have any effect on the backend!

Could you verify that /users route is being redirected to/users/ like above? Thanks.

Also, if you could share the browser console logs or request logs from the network tab in the browser dev tool, that would help us a lot with narrowing down the problem!

When I append /user to the URL, I am redirected to /auth
Mixed Content warning for /admin
From Brave:

/user is throwing this:

I should've clarified myself, you wouldn't be able to access the users route directly!
If you could share the logs you get in the dev tool when you visit the admin page, that would be helpful.
Super late where I am currently, will take a look soon as I wake up, Thanks!

The first picture was from the adminpanel. Tried again, login and click on my username and then adminpanel
Log from Brave:
bravelog.log
There is no more information in the brave dev console and maximum verbose logging :(

Can confirm this user add issue. Posting only to share that it is reproducible.

I'm back, @mtompkins are you also serving the webui over https?

Just identified what's causing the issue, working on a fix! Thank you all for your patience!

Tested the fix over https with cloudflare tunnel, seem to have fixed the issue. Let me know if the issue persists, Thanks a lot!

Yes it works now for me, thank you ☺

I'm back, @mtompkins are you also serving the webui over https?

Yes, I use traefik as a proxy. WIll confirm the current change.

Can confirm this resolves.
Would love a delete user choice :)

And disabling registration completly via ENV 🤔

They're all on the roadmap! This is just the beginning, stay tuned for regular updates!


-x option fails to convert to audio file without specifying --audio-format

After reading the documentation, I assumed that I could simply pass the -x option and youtube-dl would figure out that I want the best quality and best format for the video (song) I'm downloading. However, passing the --audio-format [mp3] argument is required so that the following error does not occur:
[youtube] Post-process file videoname-<videocode>.m4a exists, skipping

What I think is happening is the -x flag is changing the video format to an audio format before finishing the download and the post-processing can take place.

YouTube serves m4a files with a very weird header that only some players can read. youtube-dl 2<IP_ADDRESS> and newer will automatically correct the header of the m4a file if ffmpeg is present on the system. I am therefore closing this issue. See our FAQ if you need help updating. Thank you for the report!

It works! Thanks, @phihag, you're my hero!


[LLVMGPU] Allow workgroup reordering on ROCm

This is a cleanup PR in preparation for adding more workgroup reordering strategies.
On ROCm we can't currently use dynamic workgroup counts, so we only apply swizzling when all the required counts are static. This is implemented with a filter function passed to the workgroup reordering pass.

Ping. I have another change stacked on top of this.


privacy-policy.js added.

fix for the issue #58

Privacy policy page added with tailwind CSS migration.

Still working on terms and condition page, I'll update that soon.

@anadisharma121 Please mention your Issue number in PR.
Example:

@maniyaom I have edited the comment regarding PR for issue number please refer to it. Moreover please let me know about the level assigned to this issue.


Support VsCode Extensions

I don't know how different are vscode VSIX Packages But I think it would be great if you add vsCode Extensions Support.

I'm not sure exactly how much work it would be to support VS Code extensions, but that's only half the problem. The other half is that VS Code doesn't have native support for private extension galleries. 😢
Check out this issue in the VS Code repo: https://github.com/Microsoft/vscode/issues/21839
Towards the bottom of that issue (from about here onwards) there are some solutions that people have come up with that you may find useful.

I'm not sure exactly how much work it would be to support VS Code extensions, but that's only half the problem. The other half is that VS Code doesn't have native support for private extension galleries. 😢
Check out this issue in the VS Code repo: microsoft/vscode#21839
Towards the bottom of that issue (from about here onwards) there are some solutions that people have come up with that you may find useful.

yes I'm aware of those but https://open-vsx.org/ is written in java. i prefer C# solutions.


Implement optional auto-refresh of folder when contents change

Checkbox under the Refresh Folder button, maybe?
Realistically can't be implemented until after #15 is done.

The Excel event model is sufficient to allow this without some extreme gymnastics.  WILL NOT IMPLEMENT.


[FIX] Issue Template for New Topics is not labeling new topics with the content label

If you use the new topic template, it is not adding the content label. This functionality most likely broke when porting from the original repo.
To Fix
Edit the yaml template file and add the content label to the labels section. If it already exists. I suggest removing and adding it (all lowercase)

Do we need this label now that Topics is its own repository? That should be the vast majority of stuff here, right?

We also have conferences. I don't think it's entirely necessary but if we wanted to do a similar GH Projects like we did in the previous repo would need the labels for filtering.

Cool, works for me, just making sure we're not keeping obsolete features around after transitioning to the new repo.


Night mode

It was working on the previous version, but isn't anymore. Could you add back support for a dark / night style?

Any update on this?


allow skipping author match + skip GPT scoring for author-matched papers

Minor changes:

Added a flag in the config file to allow users to skip the author-matching part if they want to.
For papers with matched author(s), skip the GPT scoring step (so the author match scores won't be overwritten).
Edited the README file to clarify this.

Wouldn't it be slightly nicer if we take the max of the author match score + the GPT score?

oh good point, edited in 3d83f52d94a1ff77a63400dc2562af203e1e3741.

LGTM. Thank you for your contribution


Outdate docs

Seems that you enable HTML markup in card messages, but didn't update your docs.

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

ID: 2fa51748-908b-c781-39b2-09c961b1637a
Version Independent ID: 51b88c0a-29d3-2ca8-0b54-d96be4779dd2
Content: Legacy actionable message card reference - Outlook Developer
Content Source: docs/actionable-messages/message-card-reference.md
Product: outlook
Technology: o365-connectors
GitHub Login: @jasonjoh
Microsoft Alias: jasonjoh

It's an oversight. HTML markup isn't/shouldn't be supported, and will definitely not work in Outlook for Windows.
Thanks for reporting this.


PX Backup 2.7.2 Sign off

PXB Sign-off-Template
Following are the tasks which has to be completed by following teams
NOTE: This issue should not be closed until PX-Backup build/version pushed to IBM Cloud Catalog
Sign-off process Step 1:
Portworx Team:

[x] Planned PX-Backup Release Version: https://docs.portworx.com/release-notes/portworx-backup-on-prem
[x] Lists of Tests executed:
[x] Screenshots/Summary of Test Execution:
[x] IKS/ROKS Support Matrix for new version: IKS: 1.29.x, 1.28.8, 1.27.x ROKS: 4.15.x, 4.14.x, 4.13.x, 4.12.x
[ ] Steps for existing users to migrate to new versions: Same as 2.7.0/2.7.1
[ ] Bug fixes V/S TestCase mapping:
[ ] Please attach logs from both the PX-Backup and stork(source and destination cluster) components that were used during the execution of the tests described above:

IBM Team:

[ ] Planned PX-Backup Release Version:
[ ] Provide information about the IKS cluster(including BOM version) where integration tests were conducted:
[ ] Provide information about the ROKS cluster(including BOM version) where integration tests were conducted:
[ ] Torpedo Test Result's Details from IKS Cluster:
[ ] Torpedo Test Result's Details from ROKS Cluster:
[ ] IBM Team need to verify tasks and sign-off PX_Backup build

Sign-off process Step 2:
NOTE: Once above tasks completed and marked as done including sign-off then PX-Backup and IBM Team need to complete following tasks as well
PX-Backup Team:

[ ] Push sign-off PX-Backup build to IBM Cloud Catalog in one region i.e us-east or so
[ ] Verify torpedo tests(via automation) on default IKS and ROKS version - In case of test failure build has to be rollback and discussed with IBM Team.
[ ] Push sign-off PX-Backup build to all IBM Cloud Catalog regions.

IBM Team:

[ ] Run PX-Backup automated tests on one region for IKS and ROKS default version
[ ] In case of tests failure discuss with PX-Backup team and take action accordingly
[ ] Close this issue in case of success test result.

System Testing:

Regression Testing:

@vpinisetti-px Can you mark the check boxes for PX team in comment https://github.com/IBM/signoff-pxb/issues/25#issue-2523867785.
If we are good then I will run torpedo tests on IBM side and verify.
Also can someone ping me so that I can be aware that there is build to be signed-off ?

@vpinisetti-px 1.29 torpedo job failed.
https://alchemy-containers-jenkins.swg-devops.com/job/Containers-Volumes/job/pxbackup-torpedo-stage-e2e/48/
Test Suite Passed !!
[3:05](https://ibm-argonauts.slack.com/archives/CLMDLHW13/p1726652120605829)
BasicBackupCreation | CancelAllRunningRestoreJobs | ScaleMongoDBWhileBackupAndRestore | BasicSelectiveRestore | BackupMultipleNsWithSameLabel
PX-BACKUP-VERSION: 2.7.2

ocp: 4.15.25_openshift

https://alchemy-containers-jenkins.swg-devops.com/job/Containers-Volumes/job/pxbackup-torpedo-stage-e2e/49/
Test Suite Failed !!
[4:46](https://ibm-argonauts.slack.com/archives/CLMDLHW13/p1726658190130489)
BasicBackupCreation | CancelAllRunningRestoreJobs | ScaleMongoDBWhileBackupAndRestore | BasicSelectiveRestore | BackupMultipleNsWithSameLabel
PX-BACKUP-VERSION: 2.7.2

k8s: 1.29.8
4:46

@sameshai All the results are uploaded here https://drive.google.com/drive/u/0/folders/1rNQdXowSDeQl9QLTMBKc7sw6f6kqOf_f from testrail for IKS and ROKS runs.


[hats-protocol-hat-id] Fix hats-protocol-hat-id strategy

Fixes hats-protocol-hat-id strategy.
Changes proposed in this pull request:

Fix hat ID conversion
Remove subgraph usage
Use a multi-call to get the wearers of the hat

Hi @gershido PR should be coming from hotmanics as they are the author

Hey @ChaituVR. We decided to let @gershido do the PR as he was the one that ultimately found and fixed the bug, so just wan't to give him credit where its due. Is it possible for him to follow through on this? Otherwise I'm sure I can just make the changes and a PR from my account.


Bug - Game scene very dark when played from splash

If you play from the game scene, the scene is bright, but if played from splash or start screen, the game scene is dark.

Fixed with #49


Have git ask password in the terminal as the default option.

Issue description
Currently git pops a window so that we give the password and username because it uses the SSH_ASKPASS variable (x11-ssh-askpass). Instead it should ask for them in the terminal.
https://superuser.com/questions/758039/git-push-pull-keeps-on-trying-to-produce-gui-window
Steps to reproduce
git push
Technical details

System: (NixOS: 17.03 )

Doesn't do so for me, which desktop environment are you using?

It does that for me too (GUI pop-up). I use GNOME 3.

Git does this for me under i3 with no desktop environment.

@xekoukou Why should git ask for username/password in the terminal? More specifically, why should this behavior be different for git vs SSH?
All of my repos are set up for SSH keys, but I can confirm that SSH_ASKPASS is set to some version of x11-ssh-askpass under xmonad.
I was able to identify the associated option in a couple of minutes once I knew to look for SSH_ASKPASS.
I do not consider this a bug as you can configure the program used...
Some code-crawling (linked from the nixos.org options page) indicates that this value is the default, enabled and defaulting to the package noted above if xserver is enabled for the system. It appears to me to be blank if xserver is disabled, but I don't have a VM handy to confirm.

@globin I use xmonad.
@calvertvl

If you already use the terminal to execute commands, there is no need for a gui.
It probably shouldn't be different but I do not know all the cases where the SSH_ASKPASS variable is used.

One can configure it to use to terminal, but the default option would be better to be the terminal.

I am also using XMonad and definitely don't have this.
I do have a global username set in my ~/.gitconfig, so I can only speak to the password part.
Does ssh-add do the same thing?

Any news on this issue?

The same behaviour is still present, as the default option value is unchanged. Also, ssh-add does not do the same thing. I would also prefer a non-popup default.

SSH_ASKPASS is set by programs.ssh: nixos/modules/programs/ssh.nix#L264
Setting programs.ssh.askPassword = ""; disables this and reverts git to use the terminal to ask for the password.

Any advice on this :/
I'm having to fight vscode

@operator-name programs.ssh.askPassword = ""; doesn't work for me anymore. I need to set SSH_ASKPASSWORD='' in .bashrc.
Furthermore, x11-ssh-askpass seems broken and freezes my X sesssion (a minimal setup with StumpWM). This used to work fine too.
I spent sometime looking into NixPkgs, but I can't find the commit that has caused this.

Same issue with a freezing StumpWM here, I suspect the x11 askpass is incompatible somehow - this is a bug with StumpWM of course, but I would like to disable the graphical askpass all the same.

SSH_ASKPASS is set by programs.ssh: nixos/modules/programs/ssh.nix#L264
Setting programs.ssh.askPassword = ""; disables this and reverts git to use the terminal to ask for the password.

Seems this works for me - @solna86 can you confirm?

@TLATER I can confirm it works for me now, apologies for the late reply.

@TLATER Aside from this potential change in Nix defaults, shall we report this bug upstream (to StumpWM)?

@solna86 If you collect logs and share a reproduction scenario, of course - but this isn't related to this issue, let's keep discussion relevant and move upstream.

I'm having this issue as well, I tried to add programs.ssh.askPassword = ""; in /etc/nixos/configuration.nix and to run sudo nixos-rebuild switch, but this didn't help.

Still relevant to me.

FWIW, my approach to preventing Git from trying to use SSH_ASKPASS is to prevent it from being set in the system-wide environment:
  environment.extraInit = ''
    # Do not want this in the environment. NixOS always sets it and does not
    # provide any option not to, so I must unset it myself via the
    # environment.extraInit option.
    unset -v SSH_ASKPASS
  '';

When you don't really want SSH_ASKPASS, this seems a little cleaner than defining programs.ssh.askPassword = "", which allows SSH_ASKPASS="" to remain in the environment, which might be undesirable, as #25290 also mentions.
With NixOS 21.05, MATE Desktop, Emacs, and Magit, when attempting to push to an upstream git repo, I encountered the problem that x11-ssh-askpass would fail to even show its GUI window for some reason, leaving Magit silently waiting, leaving me mystified.  With the problematic SSH_ASKPASS no longer in my environment, Magit now asks for my git credentials in the Emacs minibuffer, as desired.

Setting GIT_ASKPASS="" also seems to work, which may be the "correct" way of doing all this :) Setting that variable to an empty string should be much safer than setting/unsetting SSH_ASKPASS.
Personally I'll stick to un-setting SSH_ASKPASS, because I generally don't want a graphical askpass, but I still agree that the default for git should probably be to disable graphical askpass, since it's usually either used as a terminal application or through some porcelain in an editor that'll break if it's set to x11 (even if they shouldn't, it's probably worth raising an issue with magit that it should set the GIT_ASKPASS variable to empty as well). I believe defaulting to non-graphical will result in less headscratching, especially if this is exposed through an option.
@calvertvl would you accept a PR that sets GIT_ASKPASS to empty by default, and adds a programs.git option to enable graphical prompts?

After refreshing on the nature of SSH_ASKPASS and GIT_ASKPASS, here are some maybe relevant thoughts:

Git is misusing SSH_ASKPASS, which is the root of our problem.  As defined by the OpenSSH man pages, SSH_ASKPASS is intended to only be used when ssh or ssh-add are not run from a terminal, and it is not intended to always be used just because it is present in the environment, and when things are run from terminals then it is not used and the terminal is used for asking, unless you set SSH_ASKPASS_REQUIRE to prefer or force.  This is consistent with NixOS always setting a default for it, which I appreciate because otherwise you'd not be able to, and probably wouldn't even notice that you need to, enter passphrases for things without terminals (I think).  The Git man pages say it uses SSH_ASKPASS just because it is present, which is inconsistent with its intended use.  Seems to me that Git should not use SSH_ASKPASS at all, and GIT_ASKPASS=$SSH_ASKPASS could be done if you want that.  But it's probably too late to expect Git to change this.

Not having to mess with SSH_ASKPASS would be better for making Git ask on the terminal, but the Git docs do not specify how GIT_ASKPASS="" is handled, so I'd not want to depend on that into the future.  But I agree that's a better temporary workaround than nuking SSH_ASKPASS (I was angry at it delaying what I was originally trying to do with my day, and I'm used to it being unset by default in Ubuntu).

x11-ssh-askpass failing to show its GUI window for me with MATE Desktop is some other bug.

Alright, I know it's been a while and it's been marked as stale but I'll just go ahead anyways
How's the status on this issue? Has anyone found a workaround to it that does not include nuking SSH_ASKPASS?
Setting GIT_ASKPASS="" really just does a whole lot of nothing and I feel like that just got be a better way to do this.

Git is misusing SSH_ASKPASS, which is the root of our problem.

If this is true @DerickEddington, perhaps we should simply report this upstream to Git?

Not having to mess with SSH_ASKPASS would be better for making Git ask on the terminal

I agree, setting this to an empty string is ugly. Also if not set to an empty string, it freezes some WMs.

Git is misusing SSH_ASKPASS, which is the root of our problem.

If this is true @DerickEddington, perhaps we should simply report this upstream to Git?

I think reporting upstream to Git could be a good idea.


Fix AV1 and VP9 codec matching

Description
Currently, AV1 or VP9 formats are matched regardless of the profile parameter. This was not noticeable until browsers started advertising multiple VP9 formats at the same time, each with a different profile ID, in order to allow the counterpart to send different streams on the basis of supported profiles.
This causes two issues: first, the library includes in the SDP all formats passed by the browser, regardless of the fact that the profile ID is registered in the API or not. Then, the library is unable to choose the correct format for streaming, causing an intermittent failure.
This patch fixes the matching algorithm and also covers the case in which the profile ID is missing, by using values dictated by specifications.
Tests were refactored since previous ones covered the same lines multiple times.
Related to #1912

@edaniels  thanks for the review, the issue in editing this patch is that is has already been merged inside the v3 branch (#2773), therefore editing it would de-synchronize the master branch from the v3 branch.


Fixed compile errors (comparing signed value with unsigned).

Fixed two compile errors by changing loop variable type as defined in Freetype API reference.

Thanks!  I had forgotten I wrote this program!


Add Template for cal.com

New Template for cal.com
This PR adds a new template for deploying cal.com (Open Source edition) as a service.
It includes the following:

Automatically generates secrets, DB usernames and passwords using Coolify's magic
Adds all required Environment Variables
Adds useful environment variables for configuring email sending
svg with the cal.com logo

💡 It's required to set NEXT_PUBLIC_LICENSE_CONSENT=agree. To let users actively decide to agree, I have not set it statically.
✅ Successfully tested with docker compose deploy

I am trying to get this template to work but it is not working for me. Could you please have another look at your template and fix it?

Thanks for your PR. I have added cal.com with this PR https://github.com/coollabsio/coolify/pull/3251 (as the author fix it completely).


Enabling CORS for OAuth2 webapp

Is your suggestion related to an experience ? Please describe.
If someone is using WSO2 IS docker images wants to enable CORS for oauth2 endpoints, the webapp gets extracted after starting the container and we are not able to update the web.xml before that. Therefore, he/she has to go through a long process to add an extracted webapp to the docker image (custom-built one) with updated web.xml.
Describe the improvement
We have two options to improve the developer experience.

Ship all the webapps as exploded directories, so we can easily update the web.xml file
Enable CORS by default for oauth2 like webapps

Might be a duplicate of https://github.com/wso2/product-is/issues/5957

I disagree. Please double check this again. It's a major hinderance when working with Docker images in all deployments that are not naive.

This might be the solution: https://github.com/wso2/product-is/issues/8337

@cpesch yes it is https://github.com/wso2/product-is/issues/8337, previously I have added a wrong link.


Error message when I try to uninstall

Current Situation
I am running Homebridge in Raspberry Pi and I try to uninstall it, but I receive the following message.
The 'hb-service uninstall' command is not available when Homebridge is running as a system package.
I tries as root and the problem persists. Any ideas?
Logs
No response
Config
No response
Homebridge UI Version
v 4.50.5
Homebridge Version
v 1.6.1
Node.js Version
v 18.16.0
Operating System
Official Homebridge Raspberry Pi Image
Environment Info

[ ] Using Docker?
[ ] Using Hyper-V?
[X] Using hb-service?

Raspberry Pi Model
Raspberry Pi Zero W

Hi @avoud you may be running homebridge in a non-traditional or non-recent way.
There is a wiki article here about standalone mode

https://github.com/homebridge/homebridge-config-ui-x/wiki/Standalone-Mode

and also an article about switching from the above mode to hb-service-mode

https://github.com/homebridge/homebridge-config-ui-x/wiki/How-To-Swap-From-Standalone-Mode-to-Service-Mode

maybe one or both of these could help? or at least help you understand how you are running homebridge and so the correct method of uninstallation?


Unable to load existing ComfyUI

I'm using Ubuntu 22.04.4 LTS, ComfyUI is in /datadisk/ComfyUI/, SwarmUI is in /datadisk/swarmui/SwarmUI/.
I added ComfyUI backend like this:

But got error like this:

The output of  bash launch-linux.sh is as follow:
05:18:35.576 [Init] === SwarmUI v<IP_ADDRESS> Starting at 2024-07-22 05:18:35 ===
05:18:35.632 [Init] Prepping extension: SwarmUI.Builtin_ScorersExtension.ScorersExtension...
05:18:35.642 [Init] Prepping extension: SwarmUI.Builtin_ImageBatchToolExtension.ImageBatchToolExtension...
05:18:35.642 [Init] Prepping extension: SwarmUI.Builtin_GridGeneratorExtension.GridGeneratorExtension...
05:18:35.642 [Init] Prepping extension: SwarmUI.Builtin_DynamicThresholding.DynamicThresholdingExtension...
05:18:35.643 [Init] Prepping extension: SwarmUI.Builtin_ComfyUIBackend.ComfyUIBackendExtension...
05:18:35.643 [Init] Prepping extension: SwarmUI.Builtin_AutoWebUIExtension.AutoWebUIBackendExtension...
05:18:35.687 [Init] Parsing command line...
05:18:35.687 [Init] Loading settings file...
05:18:35.714 [Init] Re-saving settings file...
05:18:35.732 [Init] Applying command line settings...
05:18:35.763 [Init] Prepping options...
05:18:35.766 [Init] Current git commit marked as date 2024-07-19 05:44:18
05:18:35.783 [Init] CPU Cores: 24 | RAM: 216.25 GiB total, 212.45 GiB available
05:18:35.784 [Init] GPU 0: NVIDIA A100 80GB PCIe | Temp 34C | Util 0% GPU, 0% Memory | VRAM 80.00 GiB total, 78.87 GiB free, 270.00 MiB used
05:18:35.942 [Init] Loading models list...
05:18:35.947 [Init] Loading backends...
05:18:35.949 [Init] Loading backends from file...
05:18:35.952 [Init] Initializing backend #1 - ComfyUI Self-Starting...
05:18:35.952 [Init] Prepping API...
05:18:35.954 [Init] Backend request handler loop ready...
05:18:35.955 [Init] Prepping webserver...
fatal: detected dubious ownership in repository at '/datadisk/ComfyUI'
To add an exception for this directory, call:

        git config --global --add safe.directory /datadisk/ComfyUI
05:18:36.094 [Init] Swarm is up to date! You have version <IP_ADDRESS>, and 0.9.1-Beta is the latest.
05:18:36.148 [Init] Scan for web extensions...
05:18:36.152 [Init] Readying extensions for launch...
05:18:36.152 [Init] Launching server...
05:18:36.152 [Init] Starting webserver on http://<IP_ADDRESS>:7801
05:18:36.211 [Init] Program is running.
05:18:36.569 [Init] Self-Start ComfyUI-1 on port 7821 is loading...
05:18:36.711 [Init] Launch web browser...
05:18:37.271 [Warning] [ComfyUI-1/STDERR] Traceback (most recent call last):
05:18:37.272 [Warning] [ComfyUI-1/STDERR]   File "/datadisk/ComfyUI/main.py", line 78, in <module>
05:18:37.272 [Warning] [ComfyUI-1/STDERR]     import comfy.utils
05:18:37.272 [Warning] [ComfyUI-1/STDERR]   File "/datadisk/ComfyUI/comfy/utils.py", line 1, in <module>
05:18:37.272 [Warning] [ComfyUI-1/STDERR]     import torch
05:18:37.272 [Warning] [ComfyUI-1/STDERR] ModuleNotFoundError: No module named 'torch'
05:18:37.289 [Info] Self-Start ComfyUI-1 unexpectedly exited (if something failed, change setting `LogLevel` to `Debug` to see why!)
05:18:37.290 [Info] Self-Start ComfyUI-1 had errors before shutdown:
[ComfyUI-1/STDERR] Traceback (most recent call last):
[ComfyUI-1/STDERR]   File "/datadisk/ComfyUI/main.py", line 78, in <module>
[ComfyUI-1/STDERR]     import comfy.utils
[ComfyUI-1/STDERR]   File "/datadisk/ComfyUI/comfy/utils.py", line 1, in <module>
[ComfyUI-1/STDERR]     import torch
[ComfyUI-1/STDERR] ModuleNotFoundError: No module named 'torch'

update.go:85: cannot change mount namespace according to change mount (/var/lib/snapd/hostfs/usr/local/share/doc /usr/local/share/doc none bind,ro 0 0): cannot open directory "/usr/local/share": permission denied
update.go:85: cannot change mount namespace according to change mount (/var/lib/snapd/hostfs/usr/share/cups/doc-root /usr/share/cups/doc-root none bind,ro 0 0): cannot write to "/var/lib/snapd/hostfs/usr/share/cups/doc-root" because it would affect the host in "/var/lib/snapd"
update.go:85: cannot change mount namespace according to change mount (/var/lib/snapd/hostfs/usr/share/gimp/2.0/help /usr/share/gimp/2.0/help none bind,ro 0 0): cannot write to "/var/lib/snapd/hostfs/usr/share/gimp/2.0/help" because it would affect the host in "/var/lib/snapd"
update.go:85: cannot change mount namespace according to change mount (/var/lib/snapd/hostfs/usr/share/gtk-doc /usr/share/gtk-doc none bind,ro 0 0): cannot write to "/var/lib/snapd/hostfs/usr/share/gtk-doc" because it would affect the host in "/var/lib/snapd"
update.go:85: cannot change mount namespace according to change mount (/var/lib/snapd/hostfs/usr/share/libreoffice/help /usr/share/libreoffice/help none bind,ro 0 0): cannot write to "/var/lib/snapd/hostfs/usr/share/libreoffice/help" because it would affect the host in "/var/lib/snapd"
update.go:85: cannot change mount namespace according to change mount (/var/lib/snapd/hostfs/usr/share/sphinx_rtd_theme /usr/share/sphinx_rtd_theme none bind,ro 0 0): cannot write to "/var/lib/snapd/hostfs/usr/share/sphinx_rtd_theme" because it would affect the host in "/var/lib/snapd"
update.go:85: cannot change mount namespace according to change mount (/var/lib/snapd/hostfs/usr/share/xubuntu-docs /usr/share/xubuntu-docs none bind,ro 0 0): cannot write to "/var/lib/snapd/hostfs/usr/share/xubuntu-docs" because it would affect the host in "/var/lib/snapd"
MoTTY X11 proxy: No authorisation provided
Error: cannot open display: localhost:11.0
05:19:03.295 [Info] Creating new admin session 'local' for <IP_ADDRESS> (forwarded-for: <my public IP>)
^C05:39:58.933 [Info] Shutting down...
05:39:58.933 [Info] Backend request handler loop closing...
05:39:58.934 [Error] Error #1 while initializing backend #1 - ComfyUI Self-Starting - will retry

I already ran  git config --global --add safe.directory /datadisk/ComfyUI
I'm using nginx so that I can access to this from my own device

Hi @kungfudante
did you fix the issue?

Hi @fiftin , sry I didn't
Hi @mcmonkey4eva , sry I thought I posted in the new project, actually I deployed the new project, let me post this issue there

This issue is posted in the new project, please follow-up in https://github.com/mcmonkeyprojects/SwarmUI/issues/89#issue-2428763500


pretrained model and sample train data

Hi thank you very much for the code. But for an easier start, could you please provide your trained model and also some suggestions or samples that how should I arrange the training data? Thank you again!

Any update on this? I wanted to try out the project, but failed. What should training data look like? Why is there no pretrained model? Unfortunately the demo site is also offline. Any help would be appreciated.

need the pretrained model @BlueWinters
Do you have some progress? @XinDing5 @thedani

Does anyone have a model ?

Does anyone know how to prepare the trainset?


use dataclasses for configs

define base_config as a dataclass and then instantiate it with model_name, p, power_transform, etc. as needed

One pattern is to replace your base config with a dataclass:
In [8]: @dataclass
...: class ModelConfig:
...:     sources: list[str]
...:     model_name: str = "sarix"
...:     fit_locations_separately: bool = False
...:     # model parameters
...:     p: int = 1
...:     P: 0 = 0
...:     #etc
...:     power_transform: str = None
And create a helper function that returns a ModelConfig instance based on passed parameters:
In [9]: def get_model_config(sources: list[str], fit_locations_separately: bool, p:int) -> ModelConf
...: ig:
...:     model_config = ModelConfig(sources=sources, fit_locations_separately=fit_locations_separ
...: ately, p=p)
...:     return model_config
And then in the code that parses args:
mc = get_model_config(["hhs"], fit_locations_separately=True, p=5)
The dataclass attributes work in the same tidy way as the SimpleNamespace
In [11]: mc
Out[11]: ModelConfig(sources=['hhs'], model_name='sarix', fit_locations_separately=True, p=5, P=0, power_transform=None)
In [12]: mc.model_name
Out[12]: 'sarix'


Add top-level cmake scaffolding

Signed-off-by: JaredWright<EMAIL_ADDRESS>

Here's an attempt at some CMake that tries to do the following:

Compile everything as a single CMake project (no ExternalProject_Add anywhere)
Compile from Linux/Windows/MacOS using clang + lld
Allow you to do configure the build with cmake ../hypervisor no matter which environment you are on (I found some clever tricks to default the generator to Ninja on Windows, and look for clang as the default compiler, etc.)

Building this PR should produce the following:

A vmmctl executable that can print "hello world" (build/vmmctl/vmmct)
The base vmm library (build/vmm)
An example extension that builds a minimal "hello world" VMM (build/example/hello_world)

Could others try to build this on their own environments and tell me what happens? Also, try to audit the VMM executable with readelf/objdump to see if this is on the right track.

I got the following error when running from ubuntu 19.10:
❯ make
Scanning dependencies of target vmm
[ 20%] Building ASM-ATT object vmm/CMakeFiles/vmm.dir/src/entry/start.s.obj
[ 20%] Built target vmm
Scanning dependencies of target vmmctl
[ 40%] Building CXX object vmmctl/CMakeFiles/vmmctl.dir/src/main.cpp.obj
[ 60%] Linking CXX executable vmmctl
[ 60%] Built target vmmctl
Scanning dependencies of target hello_world_vmm
[ 80%] Building CXX object example/hello_world/CMakeFiles/hello_world_vmm.dir/src/main.cpp.obj
[100%] Linking CXX executable hello_world_vmm
clang: error: invalid linker name in argument '-fuse-ld=lld'
make[2]: *** [example/hello_world/CMakeFiles/hello_world_vmm.dir/build.make:86: example/hello_world/hello_world_vmm] Error 1
make[1]: *** [CMakeFiles/Makefile2:219: example/hello_world/CMakeFiles/hello_world_vmm.dir/all] Error 2
make: *** [Makefile:152: all] Error 2

@connojd On Ubuntu, try running a sudo apt-get install lld first, and then try again.

ok that worked! I can test on Windows tomorrow and Ill leave some code comments inline

All works from windows too. I tested from normal cmd and the x64 native vs2019 cmd prompt 👍


Add more workspace setup information

Motivation
As a newcomer, I found it difficult to make VSCode understand the code written in the migration folder.
It is because the migration is not imported in the main crate.
Proposed Solutions

Add functionality to sea-orm-cli to automatically add the following lines to the main Cargo.toml:

[dependencies]
sea-orm = {
    # ...
}

migration = { path = "/path/to/migration/crate" }

If 1 is not feasible, add some information about this in the documentation, preferably under the Installation & Configuration section.

Additional Information
The solution enables rust-analyzer to correctly identify the crate, while also providing auto-completion and code-checking.

I found the reference here, would be cool if it was highlighted a little more.

Hey @sungvzer, welcome!

As a newcomer, I found it difficult to make VSCode understand the code written in the migration folder.

Do you mean VScode didn't provide any linting for symbols inside migration crate?

it is because the migration is not imported in the main crate.

I wonder which examples are you referring to? Because all of the examples are with migration crate imported into the root carate - except for the basic example as it doesn't have migration crate in it.

I found the reference here, would be cool if it was highlighted a little more.

Do you have any thoughts on this? How should we highlight it and where? hahaa


Remove redundant information when deploy flannel on kubernetes include windows node

Because the official flannel deployment yaml already contains the affinity options for deploying to linux nodes,there is no need to configure affinity in additional
The official documentation is as follows
https://github.com/coreos/flannel/blob/960b3243b9a7faccdfe7b3c09097105e68030ea7/Documentation/kube-flannel.yml#L152-L164

/assign @inductor

Thanks.
/approve
/lgtm


Better support for C++ methods written outside class

If I have
class C {
   void foo();
}
void C::foo() {}

Then if my cursor is at the last line nvim-gps will just show the method foo. It would be very useful though if it also included the name of the class C. I tried adding this to the query via
((function_definition
	declarator: (function_declarator
		declarator: (scoped_identifier
                    namespace: (namespace_identifier) @class-name
			name: (identifier) @method-name))) @scope-root)

But then it just shows the class but not the method name anymore. Looks like currently it doesn't allow for multiple captures in a single query.

Yep, you are right! Right now only immedieatly next query can get captured. I had actually considered making a special capture group (inplace of scope-root) to retrieve next two captures for this specific case. However this was the only use case I found for it. And since the get_location function gets called at every cursor movement, I didn't want to add something that could it down.

Hey @tom-anders
I just added this feature since it doesn't seem to slow down the performance. Let me know if its working ok!

Works great, thanks again!


Use "then" method for this logic

https://github.com/rbrisita/fakeFourSquare/blob/ec0e41d3fa7ea9a2611d298c2e1a1f42924740a5/app/static/js/app.js#L139
I haven't used jQuery in a while, but I recall there being a .then() method which automatically resolves a promise
$.get('some_uri_.json').then((data) => {
// response logic goes here

return somedata;
}).done((data) => {
 // your data is ready to be binded to the DOM and the promise has been resolved.
})

NOTE: Syntax might be a little off, but that's the basic concept

You can and I would lean towards using that approach (when, then) if there were a series of calls. 'done' is a promise resolver (successful) just with a different function signature. I would agree that moving the anonymous function into a chained 'done' function would be more readable to someone unfamiliar with jQuery ajax request signature.


Add ENR JC Chain to v4 & v5 tutorials

Adds the Excitation Number Restricted (ENR) example to the tutorials for v4 and v5. The original notebook can be found here. It was previously not mentioned on the webiste.
Note: @nwlambert wanted to take a second look at the partial trace function. Its implementation in the notebook can probably be improved.

It all looks good to me!  Not much we can do about the failing tests, I will just merge.

decided to get the tests working before merging, we can update the branch and hopefully just check the pep8 checks and the tests work


Add info to KRRRecommended

This update introduced info for recommendations in KRR, currently used for explaining undefined recommendations ("?"):

HPA Detected
No data

Added them to the playbook for Robusta UI to utilize

do we want to add info to the pdf?

I think we have future plans on improving those PDFs, so we might leave that for then


feat: loop through sections on Tab keys

If the user is on the last section, another tab will jump to the first section. I like the UX, but I don't know if you do. Feedback welcome.
https://github.com/user-attachments/assets/423925ad-23c2-4ab6-b9a1-8dc0190ea4ae

I like it.
Just in time. Will fix the Shift Tab to jump to last of previous


streaming: Now Executor

Frontend:

[x] #6669

Streaming:

[x] #6408
[x] #6501
[x] #6889
[ ] https://github.com/risingwavelabs/risingwave/issues/6790

Misc:

[ ] #6813

Regarding TemporalFilter

I'm not sure whether we achieve a consensus on the design meeting.
I'd prefer to introduce a now table function as the input of dynamic filter, instead of introduce the temporal filter explicitly.

https://github.com/risingwavelabs/risingwave/pull/6377#issuecomment-1316461934

Could you link some of the design docs or context?

Could you link some of the design docs or context?

This is to implement temporal filter. ('Now' + projection) + dynamic filter == temporal filter

Hmm, but actually, we will implement a standalone TemporalFilter, right? https://github.com/risingwavelabs/risingwave/pull/6377 So I guess this operator is as a fallback for when we cannot optimize now() into the standalone TemporalFilter?
Anw, it seems that the now executor will also produce Watermark messages? So is this executor the only part of our system responsible for emitting this message? Else, I am quite confused why now operator would need to emit these messages?

Hmm, but actually, we will implement a standalone TemporalFilter, right? (#6377) So I guess this operator is as a fallback for when we cannot optimize now() into the standalone TemporalFilter?
Anw, it seems that the now executor will also produce Watermark messages? So is this executor the only part of our system responsible for emitting this message? Else, I am quite confused why now operator would need to emit these messages?
Because it seems that in the case of standalone temporal filter the state cleaning can be performed based on the updated timestamp derived from the epoch, rather than the watermark? Furthermore, we will not need to use the now operator in the standalone temporal filter, so where does the watermark play a role?

They cannot stand together. We can either have a temporal filter executor or have a now executor. I implement both in case and we can choose one later :-)

Oh, I see :sweat_smile: . Ok, I see the comments by others on introducing now operator instead.
Anyway, I kind of agree, there is a lot of overlap of DynamicFilter with TemporalFilter. I think we can refactor DynamicFilter with append-only optimization...
Well, we should decide on the direction before investing further...
Also probably a good idea to link the comment where the idea was suggested. I've changed the OP.

link #6670 as reference

Let's wait for #6813 and contents (rather than todo!s) of watermark framework


Functions that generate secrets should take &mut arguments rather than returning secret data on the stack.

I'm trying to build a crypto library that uses dalek and I'd like to avoid copying secret data around as much as possible. Ideally, functions like generate_secret should take a &mut [u8; 32] so that the data can be written to the exact place I want to keep it and so that I can zero the data after use.
This would be a very trivial change to the API and I'm happy to do a PR if you're down with it.
Thanks for the great library btw!

Hi @canndrew! Thanks for the suggestion!
When passing in an &mut T, I'm not sure that the underlying pointer remains constant? Rust of course guarantees that nothing else will mutate it simultaneously, but no guarantees that the data won't move in the process.
I think the long-term fix for this is to do the suggestion in #9 by implementing key types (similarly as in done in ed25519-dalek which impl Drop and use clear_on_drop::Clear to zero out secrets.


Re-render React-Bootstrap Table Add/ Remove Column

On my site there are 5 columns in my table. There is a function to hide and show columns. Let's say I hide two columns. The table looks fine. As soon as I show these columns again the table pushes these columns off the page. I want the table to refresh/ re-render when this happens so the user can clearly see all the columns. I cannot use JQuery for this purpose.

What are you using to hide/show your columns? are you using the hidden prop?

This is solved now.  There is a prop hidden in the docs https://github.com/AllenFang/react-bootstrap-table/blob/master/src/BootstrapTable.js
That prop is printable. Simply putting in printable into my BootstrapTable component fixes issue. As far as I know, by default, it re-renders when I remove columns but will not re-render when they are added until printable is set to true. This should be in docs or be amended.


NPE appears in Metadata panel when try to copy selection when no value is chosen

System information:

Operating system (distribution) and version win 10 x64
DBeaver version 6.1.1
Java version 11

Describe the problem you're observing:
NPE appears in Metadata panel when try to copy selection when no value is chosen

metadata.log

verified


[Plugin] History plugin for enhanced undo/redo functionality

History plugin
Enhanced Undo / Redo functionality by watching changes to the editor

How does it work?
The plugin listens to tbwinit and tbwchange  events on trumbowyg.$c (instance wise)
and then check if the change is something new:

If the focused node is a text node it watches for any whitespace character and will create a new stack entry if found, otherwise it will check if the focused node has changed since the previous change, if yes it will also create a new stack entry, otherwise it will exchange latest stack entry with newest one.

If it is not a text node it compares the html tags within the editor of the new state with the previous state. the note types, the count and the order of the elements has to be the same.

The last special rule is for the first stack entry, so if there will be the first change it will always create a new stack entry, so you can go back to the initial state.

Ehanhanced undo / redo ?
There are big advantages to the default undo / redo buttons:

it is not limited to browsers which support simple undo / redo
it saves all changes to the editor, not only the simple ones (e.g. inserted something through a plugin)

As you might noticed my table plugin is also enhanced. I know there is a open pull request to enhance the table plugin, saw this one too late and also I think my table plugin is a bit more user friendly. I think I will create another pull request for this one, but first I have to extract my icons to svgs i used here. If you want to have a short look, see here: enhanced table plugin

Seems really good!
I need to take time to review this.
And yeah, send this awesome table plugin update! :D #need


Big speedup

With these changes I could speed up execution a lot. On my machine instead of over 2 minutes it now takes about 14 seconds (including saving the plot as SVG, 12 seconds without that).
The changes are:
Because the expression tree is always exactly the same (when not batching) it can be re-used and instead of re-creating the whole tree every time the new refresh() method recalculates all the values in-place. This method also sets grad to 0.0 in every node, so zero_grad() isn't needed in the optimization loop anymore.
Note that for this improvement to work with batching you need a little bit more preparation. That is, you need to create Xb and yb dummy lists of batch size containing Value objects and use these to setup the calculations before the optimization loop. In the loss() function you then need make the random index selection and update the value.data attributes of all the objects in Xb and yb before running refresh(). I didn't include an example for that at the moment. I also added code to the demo that demonstrates batching.

Good idea but it also makes this project less intuitive with non-essential optimizations


Docs: fix install and usage for yarn and npm

@aichbauer how long it took this time to choose this gif?

too loooooooooooong 🤣


Admission controller not working on K8s 1.19.x

Need to investigate a bit, but admission controller is not working on k8s 1.19.x version as it complains about the self signed certificates

Fix provided in here https://github.com/CARV-ICS-FORTH/karvdash/commit/72422e613bfffd45ddbccf53f0dab1c5c3069633
Thanks @chazapis !

Fix provided in here https://github.com/CARV-ICS-FORTH/karvdash/commit/72422e613bfffd45ddbccf53f0dab1c5c3069633
Thanks @chazapis !

Tracked branch https://github.com/IBM/dataset-lifecycle-framework/tree/cert-fix

Tracked branch https://github.com/IBM/dataset-lifecycle-framework/tree/cert-fix


[Release 1.22] netpol: Add dual-stack support

Issue: #5386
Backport: #5387

Rebased to fix conflict in go.mod


Update documentation to clarify bundled node/io.js version

I think this is an important thing to have in the documentation. I was searching for the info myself because I wanted to know which api calls would be available in node. I assume some others also must look for this info.
Had I not made the issue #2624 I would have written off electron as not a viable option.
this should close #2624

Looks good to me, thanks!


[T4A5][T09-A3] Tang Di Feng

@e0011840
Some comments added. Please ack & close the PR after reading comments.
Good job on using RCS the right way :-) :+1:  :100:


Missing relations CHEBI:58342, CHEBI:57643 and CHEBI:64612

Please add the following relationships, we need them for RHEA. Thanks in advance for your help,
Lucila
CHEBI:57368 is_a CHEBI:58342
CHEBI:57383 is_a CHEBI:58342
CHEBI:57387 is_a CHEBI:58342
CHEBI:57379 is_a CHEBI:58342
CHEBI:61540 is_a CHEBI:58342
CHEBI:73862 is_a CHEBI:58342
CHEBI:74146 is_a CHEBI:58342
CHEBI:75034 is_a CHEBI:57643
CHEBI:74669 is_a CHEBI:57643
CHEBI:73001 is_a CHEBI:57643
CHEBI:75300 is_a CHEBI:57643
CHEBI:73007 is_a CHEBI:64612
CHEBI:73008 is_a CHEBI:64612
CHEBI:73009 is_a CHEBI:64612
CHEBI:73999 is_a CHEBI:64612

Hi Lucila,
There was no need to add any of these relationships as they are already present further up the ontology tree so are automatically inherited.
For instance 73999 has 71712 as its parent and 71712 has 64612, hence the relationship 73999 is a 64612 is inherited, so there is no need to add it.
Hope this makes sense,
Kind regards,
Steve


DOC: add docs on how to debug linear algebra related issues

We have a lot of bug reports where it's not clear if it's a SciPy issue or a bug in a BLAS/LAPACK library. This guide should help with:

Diagnostics: what library (including version and build options) is being used?
Document ways to switch between BLAS libraries without having to rebuild SciPy, in order to figure out if a problem shows up for only one or for all BLAS libraries (in the latter case, it's much more likely it's a SciPy bug).
Once it looks like an upstream BLAS library bug, how to write a standalone reproducer for it, so the bug can be reported upstream in a constructive manner.

Thanks for the detailed review @ev-br. I have addressed everything. Couldn't test locally because sphinx seems to suffer from a temporary SSL error issue right this moment, so I hope it builds without warnings.

Thanks Lucas, applied.
I'm probably just blind, but I added this to the first commit, since I included Evgeni's issue reproducers:
Co-authored-by: Evgeni Burovski<EMAIL_ADDRESS>
GitHub doesn't seem to recognize it though. Not sure what's up with that, but if someone sees a mistake I'd like to fix it.

Perhaps because there is a non-empty line of [docs only] below it? Do the co-author lines have to come at the very end?

Oh yes, of course - thanks!


Misc clean up

Motivation:
IntelliJ issues several warnings.
Modifications:

ClientCookieDecoder and ServerCookieDecoder:

nameEnd, valueBegin and valueEnd don't need to be initialized
keyValLoop loop doesn't been to be labelled, as it's the most inner one (same thing for labelled breaks)
Remove if (i != headerLen) as condition is always true

ClientCookieEncoder javadoc still mention old logic
DefaultCookie, ServerCookieEncoder and DefaultHttpHeaders use ternary ops that can be turned into simple boolean ones
DefaultHeaders uses a for(int) loop over an array. It can be turned into a foreach one as javac doesn't allocate an iterator to iterate over arrays

Result:
Cleaner code

@slandelle - DefaultHttp2Headers has a similar warning as DefaultHttpHeaders ... would you mind also taking care of this too?

DefaultHttp2Headers has a similar warning as DefaultHttpHeaders ... would you mind also taking care of this too?

Really? This only simplification I found is equals that can be turned into a single boolean expression.

DefaultHttp2Headers has a similar warning as DefaultHttpHeaders ... would you mind also taking care of this too?

Really? This only simplification I found is equals that can be turned into a single boolean expression (same thing in AbstractByteBuf).

@Scottmitch PTAL

@fenik17 Thanks! Was about to fix actually :)

@Scottmitch please merge once satisfied

Really? This only simplification I found is equals that can be turned into a single boolean expression (same thing in AbstractByteBuf).

Yes it looks like we are on the same page bcz you already took care of it, but just to clarify my request was just to make the same change as you did in DefaultHttpHeaders.

@Scottmitch done!

Build failure is due to a buffer leak tracked by https://github.com/netty/netty/issues/6057

4.1 (ba95c40)
Keep em coming @slandelle !

My pleasure, thanks for the review :)


feat(about): Adding npm version to JSON RPC About endpoint output

:tada: This PR is included in version 1.3.0 :tada:
Your semantic-release bot :package::rocket:


Docs: Including drafts and livereload in docs Makefile

Signed-off-by: Lee Calcote<EMAIL_ADDRESS>fixes #395

Thanks, @venilnoronha


nano ledger not connecting to MEW

hi guys, can anyone help, my nano does not connect with MEW anymore, everything is updated  and was workin fine, but now i get this message.:
We do not know the correct path for this network. Please open a github issue so we can discuss / be enlightened
any help will be appreciated

@sparker90 can you try hard refreshing your browser?


test if travis fail with process instead of thread

On my local machine starting the server in a process instead of a thread seems to make the tests fail as expected.

Don't merge this, it's really draft. If it works I'll create a PR with only modification in server_test.py.

It seems to work I'll do one more test.

I kept the two commits to keep reference of the two Travis build. When everything seems OK and if you want to merge let me know I'll squash the commit and give a better description.
@Tarrasch this should fix #759.

https://travis-ci.org/spotify/luigi/jobs/51936789#L388 ??? That shouldn't have passed and it fail on my local machine. Anyway I would say merge and try to see why it passes later. I'll also try to see why the redshift tests are failing on python3.3.

This is an awesome catch. Please squash and give description and I'll merge! :)

@Tarrasch Done! Travis should be red because of an issue in the redshift test on python3. I'll try to fix later. There is also some error at the shutdown of the test https://travis-ci.org/spotify/luigi/jobs/51953774#L736. It seems to be due to worker that stay alive after the tests and try to acquire a lock that is deleted before the worker. Not sure thou.

Thank you so much!!!! :)


Idea: asciinema

Hello! I have an idea!
https://asciinema.org
A xontrib that automatically records every session using aschiinema, ofc written in Python already 😄
Could their PTY implementation be useful for capturing output in general? If ran around xonsh it doesn't miss a byte pushed to the screen! 😸
For community
⬇  Please click the 👍 reaction instead of leaving a +1 or 👍  comment

Video showing vim (nvim) being recorded through asciinema while interacting with it: https://asciinema.org/a/ZZTwWlg8X8HGuqYi9WSYDz7WK


VRAM does not clear after video generation so goes OOM after generating 2-3 videos

Hey, I’m using the image to video model. I can generate the lowest resolution videos with a 24GB vram but with each video generation, the VRAM resources keep going up till it goes out of memory.
Is there a way to clear the memory after a video is generated?

Could you show your whole log, which contains each video generation.

Ok will try again and show the logs. I’m using the L4 gpu from Google Colab Pro.

A matching Triton is not available, some optimizations will not be enabled
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/xformers/init.py", line 57, in _is_triton_available
import triton  # noqa
ModuleNotFoundError: No module named 'triton'
2024-11-20 02:01:37.278124: I tensorflow/core/util/port.cc:153] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0.
2024-11-20 02:01:37.732939: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:485] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-11-20 02:01:37.943587: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:8454] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-11-20 02:01:38.001004: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1452] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-11-20 02:01:38.339352: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F AVX512_VNNI FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-11-20 02:01:40.220905: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
/usr/local/lib/python3.10/dist-packages/gradio/components/dropdown.py:188: UserWarning: The value passed into gr.Dropdown() is not in the list of choices. Please update the list of choices to include: none or set allow_custom_value=True.
warnings.warn(
/usr/local/lib/python3.10/dist-packages/gradio/utils.py:1002: UserWarning: Expected 2 arguments for function <function ui..upload_generation_method at 0x79620c475e10>, received 1.
warnings.warn(
/usr/local/lib/python3.10/dist-packages/gradio/utils.py:1006: UserWarning: Expected at least 2 arguments for function <function ui..upload_generation_method at 0x79620c475e10>, received 1.
warnings.warn(
Running on local URL:  http://<IP_ADDRESS>:7860/
Running on public URL: https://8c02f0e679cdfaaa3a.gradio.live/
This share link expires in 72 hours. For free permanent hosting and GPU upgrades, run gradio deploy from Terminal to deploy to Spaces (https://huggingface.co/spaces)
Update diffusion transformer
missing keys: 0;
unexpected keys: 0;
[] []
loaded 3D transformer's pretrained weights from /content/drive/MyDrive/EasyAnimate/models/Diffusion_Transformer/EasyAnimateV5-12b-zh-InP/transformer ...
missing keys: 0;
unexpected keys: 0;
[]
All Parameters: 11797.69248 M
attn1 Parameters: 1812.54144 M
Loading checkpoint shards: 100% 2/2 [02:17<00:00, 68.58s/it]
The config attributes {'snr_shift_scale': 1.0} were passed to EulerDiscreteScheduler, but are not expected and will be ignored. Please verify your scheduler_config.json configuration file.
Loading pipeline components...: 100% 7/7 [00:00<00:00, 234.55it/s]
You have disabled the safety checker for <class 'easyanimate.pipeline.pipeline_easyanimate_multi_text_encoder_inpaint.EasyAnimatePipeline_Multi_Text_Encoder_Inpaint'> by passing safety_checker=None. Ensure that you abide to the conditions of the Stable Diffusion license and do not expose unfiltered results in services or applications open to the public. Both the diffusers team and Hugging Face strongly recommend to keep the safety filter enabled in all public facing circumstances, disabling it only for use-cases that involve analyzing network behavior or auditing its results. For more information, please have a look at https://github.com/huggingface/diffusers/pull/254 .
Update diffusion transformer done
The config attributes {'snr_shift_scale': 1.0} were passed to EulerDiscreteScheduler, but are not expected and will be ignored. Please verify your scheduler_config.json configuration file.
We strongly recommend passing in an attention_mask since your input_ids may be padded. See https://huggingface.co/docs/transformers/troubleshooting#incorrect-output-when-padding-tokens-arent-masked.
You may ignore this warning if your pad_token_id (0) is identical to the bos_token_id (0), eos_token_id (2), or the sep_token_id (None), and your input is not padded.
/content/drive/MyDrive/EasyAnimate/easyanimate/pipeline/pipeline_easyanimate_multi_text_encoder_inpaint.py:1053: FutureWarning: Accessing config attribute enable_clip_in_inpaint directly via 'EasyAnimateTransformer3DModel' object attribute is deprecated. Please access 'enable_clip_in_inpaint' over 'EasyAnimateTransformer3DModel's config object instead, e.g. 'unet.config.enable_clip_in_inpaint'.
elif clip_image is None and num_channels_transformer != num_channels_latents and self.transformer.enable_clip_in_inpaint:
100% 30/30 [05:03<00:00, 10.11s/it]
The config attributes {'snr_shift_scale': 1.0} were passed to EulerDiscreteScheduler, but are not expected and will be ignored. Please verify your scheduler_config.json configuration file.
/content/drive/MyDrive/EasyAnimate/easyanimate/pipeline/pipeline_easyanimate_multi_text_encoder_inpaint.py:1053: FutureWarning: Accessing config attribute enable_clip_in_inpaint directly via 'EasyAnimateTransformer3DModel' object attribute is deprecated. Please access 'enable_clip_in_inpaint' over 'EasyAnimateTransformer3DModel's config object instead, e.g. 'unet.config.enable_clip_in_inpaint'.
elif clip_image is None and num_channels_transformer != num_channels_latents and self.transformer.enable_clip_in_inpaint:
100% 30/30 [05:04<00:00, 10.17s/it]
The config attributes {'snr_shift_scale': 1.0} were passed to EulerDiscreteScheduler, but are not expected and will be ignored. Please verify your scheduler_config.json configuration file.
/content/drive/MyDrive/EasyAnimate/easyanimate/pipeline/pipeline_easyanimate_multi_text_encoder_inpaint.py:1053: FutureWarning: Accessing config attribute enable_clip_in_inpaint directly via 'EasyAnimateTransformer3DModel' object attribute is deprecated. Please access 'enable_clip_in_inpaint' over 'EasyAnimateTransformer3DModel's config object instead, e.g. 'unet.config.enable_clip_in_inpaint'.
elif clip_image is None and num_channels_transformer != num_channels_latents and self.transformer.enable_clip_in_inpaint:
9% 3/35 [00:37<06:37, 12.43s/it]
After this step the UI shows and error and goes OOM.

It should be solved in https://github.com/aigc-apps/EasyAnimate/pull/158.

New model runs fine! Thanks. Closing issue.


adding note

This article is for commercial issues troubleshooting. We'd like to add the consumer issue troubleshooting page link in the begin of the article in case some consumer users get lost here.

#sign-off


scale down to 80 pods

Test failed for 100 pods with timeout, so scale down back to 80 pods.
https://prow.k8s.io/view/gcs/kubernetes-jenkins/logs/ci-kubernetes-e2e-windows-node-throughput/1225796067589623808

/cc @pjh @yliaog

/lgtm
/approve

Thanks Yang, also LGTM.
@wojtek-t do you have any ideas about why we can get up to 100 pods just fine in manual test runs but we can't seem to reach 100 pods yet in continuous e2e runs? We'll continue investigating to see if we can find any differences in the manual vs. e2e setup/environment, but if you have any ideas or past experience please let us know.

@wojtek-t do you have any ideas about why we can get up to 100 pods just fine in manual test runs but we can't seem to reach 100 pods yet in continuous e2e runs? We'll continue investigating to see if we can find any differences in the manual vs. e2e setup/environment, but if you have any ideas or past experience please let us know.

No idea - we generally get pretty consistent results across manual and continuous runs.
How do you run your manual tests? I guess there has to be some difference between how you setup your cluster there.


Add Haptic Glove Module

[ ] I still have to add README file to the module
[x] I have to fix the configuration files
[x] I have to check the possible changes outside the scope of this PR on other modules and remove those changes if are unnecessary
[x] After the latest changes, I have to run the code with the robot.

The PR is ready to be reviewed, except the points mentioned in the first comment.

I have tested many times during these days the code, and I applied bug fixes as well as updates on the configuration file. All the features are implemented. The only remaining point is the documentation, that I may do it later. In any case, the PR is ready to be reviewed.

Given the size of the PR, if @S-Dafarra is already reviewing it I would remove myself from the reviewers. If there is any specific point on which you need my input @kouroshD feel free to point it out, thanks!

With this https://github.com/robotology/walking-teleoperation/pull/72/commits/fc89789c4fabb663a023cae91fd3c22e512bb5b0, I have modified the writeStrict option of the remotecontrolboard from true to false.

another thing is that in the RemoteControlBoard the default carrier is set to udp as in https://github.com/robotology/yarp/blob/ce6f008c86ba58a652b04b6bffdc2b46e9b157e2/src/devices/RemoteControlBoard/RemoteControlBoard.cpp#L247-L250 and https://github.com/robotology/yarp/blob/ce6f008c86ba58a652b04b6bffdc2b46e9b157e2/src/devices/RemoteControlBoard/RemoteControlBoard.cpp#L307. And I remember, I could run the HapticGloveModule some months ago from the operator side as well, however with some difficulties.  In any case, now we are running it the robot head.

@S-Dafarra  let me know if the responses are OK, if it needs further declarations/modifications.

I guess you can also remove that line. By default oit should be off

I guess you can also remove that line. By default it should be off

It is not. Look at here:
https://github.com/robotology/yarp/blob/165a46c6e034fc27b90cc0bd80b35419679b22c3/src/devices/RemoteControlBoard/RemoteControlBoard.h#L96-L97

I guess you can also remove that line. By default it should be off

It is not. Look at here:
https://github.com/robotology/yarp/blob/165a46c6e034fc27b90cc0bd80b35419679b22c3/src/devices/RemoteControlBoard/RemoteControlBoard.h#L96-L97

Ah nice catch. It seems that it enables it when writing to multiple joints. I guess this is to make sure that the commands sent to all the joints is coherent in time :thinking: .

I guess you can also remove that line. By default it should be off

It is not. Look at here:
https://github.com/robotology/yarp/blob/165a46c6e034fc27b90cc0bd80b35419679b22c3/src/devices/RemoteControlBoard/RemoteControlBoard.h#L96-L97

Ah nice catch. It seems that it enables it when writing to multiple joints. I guess this is to make sure that the commands sent to all the joints is coherent in time 🤔 .

Yes, writeStrict is necessary for all the commands that require multiple messages for different joints, otherwise tipically the last one would be the only one considered.

So then, maybe it is worth leaving it with the default behavior :thinking:

Yes, writeStrict is necessary for all the commands that require multiple messages for different joints, otherwise tipically the last one would be the only one considered.

@traversaro  What do you mean by requiring multiple messages for different joints? Can you give an example?

Yes, writeStrict is necessary for all the commands that require multiple messages for different joints, otherwise tipically the last one would be the only one considered.

@traversaro What do you mean by requiring multiple messages for different joints? Can you give an example?

Let's say that for some reason you are sending velocity reference joint-by-joint with: https://github.com/robotology/yarp/blob/165a46c6e034fc27b90cc0bd80b35419679b22c3/src/devices/RemoteControlBoard/RemoteControlBoard.cpp#L1914 . In that case, if you do not have writeStrict, only the last command will be considered.

As I see in that snippet, it sends the joint i reference value with false write strict option. I am not seeing any part that is discarding other commands! booh! I can put it back in our code in default option, what is your suggestion? When I tried yesterday with off option, I did not see any strange behavior. In any case, in our code, we are always send vector of joint values not a single joint!

@S-Dafarra  @traversaro  If you do not have other comments or suggestion, I will merge the PR. Thanks.

@S-Dafarra @traversaro If you do not have other comments or suggestion, I will merge the PR. Thanks. I will keep the history of PR as it is.

Sure on my side, see https://github.com/robotology/walking-teleoperation/pull/72#issuecomment-1008733201 !

@S-Dafarra @traversaro If you do not have other comments or suggestion, I will merge the PR. Thanks. I will keep the history of PR as it is.

GO also for me!


Cleanup Types.h

We had type definitions in a lot of files, and we really should
have these in one place. This patch cleans this up. Also found a
race condition with the new VMCS unit tests which have been
cleaned up
Signed-off-by: “Rian<EMAIL_ADDRESS>

Coverage decreased (-0.02%) to 99.982% when pulling 9534420f25d9d1a8655422cec0a45a2362e60954 on rianquinn:cleanup_types into 4733dc70496131de410e882e22f5ff23edd59929 on Bareflank:master.

Coverage remained the same at 100.0% when pulling 52f8941bf7af19014c551009cae62de2947bf6d7 on rianquinn:cleanup_types into e6321f30dc5b37cf5e57d45c86b8116fd92cb3c9 on Bareflank:master.

Coverage remained the same at 100.0% when pulling 64b0c0caf38d6fde3ac7caa3055c38234b9d25a9 on rianquinn:cleanup_types into 2f8d31762ef6d88a9261ab1b3f931e8991e067a3 on Bareflank:master.


native_assets_cli 0.9 breaks flutter_gpu_shaders 0.1.3

native_assets_cli version 0.9 no longer requires cli_config as a dependency, which flutter_gpu_shaders 0.1.3 depends upon.
I'm wondering if this change is intentional, and thus I should open a PR to add cli_config to flutter_gpu_shaders as a required dependency?
Example error stack:
FormatException: No value was provided for required key: out_dir_shared
#0      MapJsonUtils.get (package:native_assets_cli/src/json_utils.dart:50:7)
#1      MapJsonUtils.path (package:native_assets_cli/src/json_utils.dart:25:48)
#2      new HookConfig (package:native_assets_cli/src/config.dart:88:38)
#3      new BuildConfig (package:native_assets_cli/src/config.dart)
#4      build (package:native_assets_cli/src/api/build.dart:99:18)
#5      main (file:///Users/brettmorgan/.pub-cache/hosted/pub.dev/flutter_scene_importer-0.2.0-0/hook/build.dart:7:9)
#6      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#7      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)

stdout:

../../../../.pub-cache/hosted/pub.dev/flutter_gpu_shaders-0.1.3/lib/build.dart:86:14: Error: Type 'BuildOutput' not found.
    required BuildOutput buildOutput,
             ^^^^^^^^^^^
../../../../.pub-cache/hosted/pub.dev/flutter_gpu_shaders-0.1.3/lib/build.dart:86:14: Error: 'BuildOutput' isn't a type.
    required BuildOutput buildOutput,
             ^^^^^^^^^^^
Bad state: Generating kernel failed!

Also https://github.com/bdero/flutter_gpu_shaders/issues/3

native_assets_cli version 0.9 no longer requires cli_config as a dependency
...
I'm wondering if this change is intentional

Yes, it's intentional to no longer use package:cli_config in those packages.

which flutter_gpu_shaders 0.1.3 depends upon.
...
and thus I should open a PR to add cli_config to flutter_gpu_shaders as a required dependency?

If package:flutter_gpu_shaders doesn't use package:cli_config, why should it get it as a required dependency?

Example error stack:

../../../../.pub-cache/hosted/pub.dev/flutter_gpu_shaders-0.1.3/lib/build.dart:86:14: Error: Type 'BuildOutput' not found.
    required BuildOutput buildOutput,
             ^^^^^^^^^^^
../../../../.pub-cache/hosted/pub.dev/flutter_gpu_shaders-0.1.3/lib/build.dart:86:14: Error: 'BuildOutput' isn't a type.
    required BuildOutput buildOutput,
             ^^^^^^^^^^^
Bad state: Generating kernel failed!

The 0.9.0 has made some breaking changes to the (still experimental) package. See also my comment here https://github.com/bdero/flutter_gpu_shaders/issues/3#issuecomment-2456987663. It will require minor updates to the code.

Build hooks that don't link a library are failing again (against 0.9.0):
Unhandled exception:
FormatException: No value was provided for required key: out_dir_shared
#0      MapJsonUtils.get (package:native_assets_cli/src/json_utils.dart:50:7)
#1      MapJsonUtils.path (package:native_assets_cli/src/json_utils.dart:25:48)
#2      new HookConfig (package:native_assets_cli/src/config.dart:88:38)
#3      new BuildConfig (package:native_assets_cli/src/config.dart)
#4      build (package:native_assets_cli/src/api/build.dart:99:18)
#5      main (file:///Users/bdero/projects/flutter/apps/flutter_scene/hook/build.dart:7:9)
#6      _delayEntrypointInvocation.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:295:33)
#7      _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:184:12)
Building native assets for package:flutter_scene failed.
file:///Users/bdero/projects/flutter/apps/flutter_scene/hook/build.dart returned with exit code: 255.

Build hooks that don't link a library are failing again (against 0.9.0):
FormatException: No value was provided for required key: out_dir_shared

That's probably on me: It seems the out_dir_shared support wasn't included in the recent release, which is probably why you see this: Newer flutter run/build and dart run/build will provide this, but not in older versions.
The refactoring I did to the BuildConfig made us read this eagerly and mandate it. I've setup an SDK constraint to the most recent release, but it may not have had that support yet.
If strongly needed, I could make a patch release that makes this out_dir_shared optional (for now). Then some release later can make it mandatory. Though it's easy to workaround: Just stick with 0.8.0 for now. Once newer Dart & Flutter SDKs have rolled out with this support, upgrade to 0.9.0. Would that work for you?

The way I usually handle this is I tell Dependabot to jump in a lake every time it sends me a PR that fails CI.

@domesticmouse @bdero Version 0.9.0 should work with flutter main / master branch.

Version 0.9.0 should work with flutter main / master branch.

Unfortunately, it doesn't. Yesterday I was running it against 5a3fd44ae.
I temporarily fixed this by pinning flutter_gpu_shaders to < 0.9.0.
Let's re-open this bug please.

Unfortunately, it doesn't. Yesterday I was running it against 5a3fd44ae.

Have you tried with https://github.com/flutter/flutter/commit/3a83e43ede5700dfe4ea92709fec2c2365cdef4f or afterwards?

That landed today, so nope. Trying it.

Yup, confirmed it works now. Thanks!

@mkustermann we are seeing some failures on cherry pick pr to the 3.27 stable. Is there is a fix that can be pulled into that branch?

@reidbaker Could you be more specific? Are you trying to cherry-pick my changes? If so: why? Do you have a link to failures?

I am following @bkonyi comment from https://github.com/flutter/flutter/issues/156962#issuecomment-2460925873


Highscores

Gists seem to be a viable solution, but github doesn't want me to commit an auth token in plain text. It's not possible to give our application access to the highscores without allowing anyone to do the same, since it's all in plain text javascript.
The only consideration is where where we want to save the highscores, if we can't use gists. The problem with using the IBM server is that github pages doesn't want to talk to any 3rd party sites, it considers it a security risk.

Done! Using Cloudant, and yes there is a small security risk; people can cheat the highscores! OH NO


Test failure: XdsMemberPermissionTest

XdsMemberPermissionTest fails on my PC
  ["foo"]
to contain only:
  ["foo", "@xds"]
but could not find the following element(s):
  ["@xds"]

java.lang.AssertionError:
Expecting ArrayList:
  ["foo"]
to contain only:
  ["foo", "@xds"]
but could not find the following element(s):
  ["@xds"]

	at com.linecorp.centraldogma.server.test.XdsMemberPermissionTest.shouldAllowMembersToAccessInternalProjects(XdsMemberPermissionTest.java:156)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at java.util.ArrayList.forEach(ArrayList.java:1259)
	at java.util.ArrayList.forEach(ArrayList.java:1259)

I noticed that if I add Thread.sleep or even some small operation, for example userClient.listProjects().join().forEach(System.out::println);, before the line where the error occurs, the test completes successfully.

That's a flaky test that we need to fix. Thanks for the report. 😉

I want to contribute, and so far I have 2 ideas on how to solve the problem:

Use a Watcher to wait for changes in the metadata, but I'm not sure if it's possible to monitor changes in the metadata through this API.
Use await().untilAsserted, but I think the first solution would be more preferable.
Could you let me know if any of the solutions I proposed are acceptable? Or should this problem be solved not at the testing level?

Probably just using await().untilAsserted sounds fine to me


Custom ARM resource for sovereign clouds

For US government cloud and China cloud the ARM resource should be different so allowing it to be modified through env variable instead of hardcoding it
Also change the agent pool because it is different now

Codecov Report

Merging #37 into main will decrease coverage by 1.84%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
- Coverage   59.45%   57.61%   -1.85%
==========================================
  Files           6        6
  Lines         365      335      -30
==========================================
- Hits          217      193      -24
+ Misses        117      110       -7
- Partials       31       32       +1

Flag
Coverage Δ

#unittests
57.61% <60.00%> (-1.85%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/authorizer/token_retriever.go
70.17% <60.00%> (-1.76%)
:arrow_down:

pkg/authorizer/types/accesstoken.go
51.72% <0.00%> (-4.53%)
:arrow_down:

pkg/authorizer/authorizer.go
54.54% <0.00%> (-2.60%)
:arrow_down:

controllers/acrpullbinding_controller.go
53.84% <0.00%> (-2.18%)
:arrow_down:

pkg/authorizer/util.go
100.00% <0.00%> (ø)

pkg/authorizer/token_exchanger.go
58.97% <0.00%> (+0.43%)
:arrow_up:

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 66c0dff...9ae1390. Read the comment docs.


Update graphmetrics submodule to add support for fullscreen graphs

not sure why travis is failing but accepting the change


Renaming tests directory

regions/io/crtf/test should be renamed to regions/io/crtf/tests

Thanks, I will rename it in one of the active PRs.

#186 solve here. It can be closed now.


WIP: feat: download artifact

#44 add download artifact API

Codecov Report

Merging #45 (4e63b77) into main (94e2330) will increase coverage by 0.32%.
The diff coverage is 93.10%.

@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   70.47%   70.80%   +0.32%
==========================================
  Files          36       36
  Lines        1968     1997      +29
==========================================
+ Hits         1387     1414      +27
- Misses        514      515       +1
- Partials       67       68       +1

Flag
Coverage Δ

unittests
70.80% <93.10%> (+0.32%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/artifact/artifacts.go
66.66% <71.42%> (+4.16%)
:arrow_up:

pkg/artifact/artifacts_test_common.go
93.02% <100.00%> (+7.30%)
:arrow_up:

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 94e2330...4e63b77. Read the comment docs.


Submitting Appel game V<IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] This PR only modifies one (1) manifest
[x] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.5 schema?

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

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

Hi @krishisrani61,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 684.)

Automatic Validation ended with:

Installation failed with exit code -1978335187 2024-01-30 20:52:29.665 [CORE] DeliveryOptimization error: 0x80190193, extended error: 0x00000000 2024-01-30 20:52:29.715 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(289)\WindowsPackageManager.dll!00007FFF8E2CA465: (caller: 00007FFF8E2CAD01) Exception(1) tid(b84) 80190193 Forbidden (403). 2024-01-30 20:53:03.072 [FAIL] Installer failed security check. Url: https://github.com/krishisrani61/exestorage/raw/main/Appel/setup.exe Result: 0x80004005

(Automated response - build 684.)


fix(spinner): hide content when spinning

(๑ˇ ῁̫ ˇ)˒˒
^ smug 🐛  catching face
the disabled spinner button on the account page wasn't actually working as expected b/c all the styles for it were nested under box .box-header button.

csscsscsscsscsscss lgtm!


Customize the JSHint

Hi,
I have created a java script editor which allows to write code in customise java script. I am annotating error and waning in js code using JSHint. It annotates the normal js code but I have some custom apis which I wants to validate.
Can anyone help me to change the logic of JSHint logic to validate my defined apis?
Thanks
Ishan Jain

We may be able to help, but we'll need some more information first. You mention
"custom" JavaScript, which may be an issue. Are you interested in linting code
written in a language that is itself an extension of ECMAScript? Or is your
editor simply providing custom APIs in a standard JavaScript runtime?

It is itself an extension of ECMASCRIPT and we use JShint for annotating the error in js source. It supports ECMASCRIPT standard and we have to customize it for annotation of our own custom methods.

Linting arbitrary language extensions would require an advanced plugin system
that exposed most of JSHint's internals. There are currently no plans to
implement such a system.
You do have a few options, though:

You may use the ignore directive to instruct
JSHint to skip over blocks of non-standard code
If you are compiling to standard ECMAScript, you may use the output of your
compilation process as input to JSHint

Hope this helps!


Implementing facet registry (PROPOSAL #2161)

Problem
We have a proposal for a facet registry, this PR introduces the foundations of the registry into the repo.
Closes: PR #2228
Solution
As described in the proposal.
After this will be approved, we will start transferring over facets.
- add facet registry folder structure
- add codeowners
- add list of core facets

Checklist

[ ] You've signed-off your work
[ ] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] Your comment includes a one-liner for the changelog about the specific purpose of the change (not required for changes to tests, docs, or CI config)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2024 contributors to the OpenLineage project

/spec/registry

It's part of the spec, so it should probably live there. can we maybe change the code generator file pattern?

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 85.52%. Comparing base (e2390d5) to head (c20d8c3).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2729   +/-   ##
=======================================
  Coverage   85.52%   85.52%
=======================================
  Files          58       58
  Lines        3413     3413
=======================================
  Hits         2919     2919
  Misses        494      494

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Establish better guidance on the build/OS/tooling requirements

With our latest branches, we now we require insider preview SDKs (along with insider OSes) in order to build the code.
This is troublesome when trying to build against an older device (i.e. HL1) which doesn't use any of the new APIs. There are a few possibilities here:

Make it easier to get access to those SDKs without requiring new OSes. (maybe possible, I feel like I've done this in the past but haven't tried this in recent history)
Have some code/build level flags that strip out later SDK requiring code
Something else.

Now that the SDK is no longer in preview mode, the recommendation is just to install the public SDK which handles this. I think for future cases (i.e. if there's some other preview API we need to depend on, we'll have to have some recommendation/wrapping of the binaries so that people don't need to install preview SDKs, but for now there's no real action here)


support tpl literal in defineMessage

Porting https://github.com/lingui/js-lingui/pull/1495/files

@thekip can we merge this PR?


Catlog of music Album.

In this PR:-
Created music album class.
Created genre class.
Created SQL queries.
Created test for all class.

Hi @calebchris000 ,
Thanks for Approval. :smile:


Image no longer available

The link to the image is no longer available
https://github.com/jmcantrell/vim-virtualenv/blob/b1150223cd876f155ed7a3b2e285ed33f6f93873/README.mkd?plain=1#L29
Showing this:

Thanks for letting me know. I've removed the image references. I may add back some new images later.


[Improvement] Make config type of RSS_CLIENT_TYPE as enum

Code of Conduct

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

Search before asking

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

What would you like to be improved?
Make config type of RSS_CLIENT_TYPE as enum
How should we improve?
The enum type of config option is introduced in https://github.com/apache/incubator-uniffle/pull/199, follow this PR to fix the RSS_CLIENT_TYPE config
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Because the change of this issue involves only one class change, I want to use this to verify whether my thinking is correct


fix: Helium API not returning

Why
Helium API was not returning values because of missing headers
How
Added headers specifying the user-agent field and it returned perfectly. Helium API is now happy with our request and returning a proper response
References
Closes: #232

That should do the trick!
thanks @SebastianMaj