[in_openmpt] Added subsong support

Hi! This is my proposition of an implementation of subsong support for the Winamp's plugin.
Sorry for not posting it in subversion.
Features

When you play a module with more than 1 subsong, a counter is added to the title, following this format: "(current/total)".

Press next/back button/hotkey to navigate between subsongs. Shift + next/back button is for jumping 10 subsongs; that's for modules with too many subsongs.

When a subsong ends (if "repeat forever" is disabled) the next one is played. If there are no more subsongs, the next playlist entry is played instead.

You need to get to the first subsong before playing the previous playlist item. The same applies for the last subsong & next playlist item.

You can see the number of subsongs in the info window too.

Flaws

If you add the file you are currently playing into the playist, the new playlist item has the counter of the original entry.

Subsong position isn't saved when you play another file.

You cannot disable subsong support. I believe I should have added an option in the configuration window, but I didn't do it because Winamp and XMPlay's plugins share the same configuration window. The modes could be:

Default (what I implemented)
Play only first subsong
Play all subsongs as one big file (self->mod->select_subsong( -1 ))

If you mess with the playlist there may be bugs with the names & lengths. It may be related to how Winamp handles "getfileinfo(...)". In this example, you need to delete the only file while it is still being played, Winamp won't stop it; then add another file and the title will be wrong.

Important changes

(Link) In the version of the Winamp SDK you use (build\download_externals.cmd), the button constants are missing. So I added them by hand. Here they are in another version.
(Link) I subclass Winamp's main window to intercept the next & back buttons' events.
(Link) I need "cached_title" not to be empty. Otherwise If a module has no metadata title, the title shown in the playlist  would be just something like "(1/8)".
(Link) I include "filesystem", an C++17 library to get the file name from the path. Visual Studio changed the C++ version automatically when I compiled. It seems you use C++14. If you don't want to change the C++ version, maybe you can find another way to get just the file name.

Testing details

Winamp version: WACUP Preview Build v<IP_ADDRESS>36 (x86)
Windows 11 Pro 21H2
The modules shown in the gifs can be found here.

Conclusion
This implementation is not perfect, but I believe it is a good starting point. I still use Winamp for its DSP Plugins, and I believe It would be cool to add subsong support to this plugin.
Thank you for your time reading this and developing this plugin.
Goodbye.

Hi! This is my proposition of an implementation of subsong support for the Winamp's plugin.

Thanks for the proposed contribution.

Sorry for not posting it in subversion.

That's fine. See contributing.

When you play a module with more than 1 subsong, a counter is added to the title, following this format:
Press next/back button/hotkey to navigate between subsongs. Shift + next/back button is for jumping 10 subsongs; that's for modules with too many subsongs.
When a subsong ends (if "repeat forever" is disabled) the next one is played. If there are no more subsongs, the next playlist entry is played instead.
You need to get to the first subsong before playing the previous playlist item. The same applies for the last subsong & next playlist item.
You can see the number of subsongs in the info window too.

I am somewhat out of the loop with how Winamp itself is even designed to be capable to handle subsongs. The plugin architecture (as of Winamp 2.x level) certainly does not hint at any functionality. I think some SID plugin also had subsong support, and probably did something similar to what you are suggesting here.

If you add the file you are currently playing into the playist, the new playlist item has the counter of the original entry.
Subsong position isn't saved when you play another file.
If you mess with the playlist there may be bugs with the names & lengths. It may be related to how Winamp handles "getfileinfo(...)". In this example, you need to delete the only file while it is still being played, Winamp won't stop it; then add another file and the title will be wrong.

Given all the flaws, I would rather not include subsong support in the Winamp plugin in the current form. libopenmpt is, after all, designed to also allow sane playback for players which cannot represent subsongs properly, by merging everything together.

You cannot disable subsong support. I believe I should have added an option in the configuration window, but I didn't do it because Winamp and XMPlay's plugins share the same configuration window. The modes could be:

Default (what I implemented)
Play only first subsong
Play all subsongs as one big file (self->mod->select_subsong( -1 ))

This may be worthwhile also for XMPlay. @sagamusix may have some opinion on that.

1. In the version of the Winamp SDK you use (_build\download_externals.cmd_), the button constants are missing. So I added them by hand. [Here they are in another version](https://getwacup.com/sdk/Winamp/wa_ipc.h#:~:text=#define%20WINAMP_BUTTON1).
2. I **subclass** Winamp's main window to intercept the next & back buttons' events.

The in_openmpt plugin currently limits itself to a subset of only Winamp 2.x functionality, which allows testing it even in other players which have limited support for Winamp plugins. This is not very important after all, but still something to consider.

3. I need "**cached_title**" not to be empty. Otherwise If a module has no metadata title, the title shown in the playlist  would be just something like "**(1/8)**".

which would also be ok-ish, I think

I include "filesystem", an C++17 library to get the file name from the path. Visual Studio changed the C++ version automatically when I compiled. It seems you use C++14. If you don't want to change the C++ version, maybe you can find another way to get just the file name.

I'm confused here. We are using C++17 or C++20 (for VS2022). std::filesystem is supported even by VS2017 which we use for Windows XP RETRO builds. In any case, it would be simple enough to implement the needed functionality without std::filesystem, so this is not a huge deal.

This implementation is not perfect, but I believe it is a good starting point. I still use Winamp for its DSP Plugins, and I believe It would be cool to add subsong support to this plugin.

I agree, it would be useful to have some kind of subsong support. However, given the current state of original Winamp, its uncertain (at best) future under the current rights owners, I in general see little future for Winamp itself (under its original name).
As you are using WACUP already, please note that WACUP is not even using our upstream version of in_openmpt directly, but has already improved some things for better integration with modern Winamp features (as far as I understand it, I have not looked at WACUP in detail for quite some time now). I would really love to hear @dr0biwan 's opinion on subsong support. Maybe there already is a better architecture in WACUP that could be used here.
In any case, given that original Winamp is probably dead, and given that WACUP is already using an adjusted version of in_openmpt anyway, we have already been considering to completely completely drop in_openmpt from our repository (see https://bugs.openmpt.org/view.php?id=1269).
So, I am not really sure what would be a useful way forward here:

OpenMPT could drop in_openmpt support, and you could try to convince @dr0biwan to add subsong support directly in what WACUP ships
@dr0biwan could open-source the WACUP version of in_openmpt (maybe, libopenmpt license totally allows closed-source derivatives of course), either as a separate repository or maybe merging back with our version
continue to develop in_openmpt, however as I see the future, it would be kind of redundant if the only usable remaining descendent of Winamp, WACUP, already ships an up-to-date plugin based on libopenmpt
you could take-over maintainership of in_openmpt, however the same caveats as for the previous point apply

From a quick look I'd not be too happy with what's proposed going into the plug-in as doing subclassing is something that made sense only because it was the way to do it under Winamp 2.x but manipulating things with the button actions isn't reliable as not every play/next/prev action is going to be triggering them.

I would really love to hear @dr0biwan 's opinion on subsong support. Maybe there already is a better architecture in WACUP that could be used here.

The only "good" way which is a bit of a pain do with plain Winamp & is something I've yet to sort out for my custom build of the plug-in is to make a playlist handler & then have the playlist loader service check if what's being added has sub-songs or not & then expand that out along with the play(..) handler then dealing with figuring out what's being provided. (e.g. using the file.mod, syntax that has been an unofficial but normally used thing for other custom expanded formats going back over the years).

I think some SID plugin also had subsong support, and probably did something similar to what you are suggesting here.

Indeed it did & I already use the way I've noted above for SID & zip files as the subclassing / message hacks weren't reliable. So it's mostly just needs me getting around to adding the support for it (along with doing the same for vgmstream which has a similar sub-song support "issue"). As the last time I really looked into the sub-song aspects with the plug-in I ended up just having my build of the plug-in treat such files as one big file so at least all of the audio was accessible to play but doing it as a mini-playlist overall is what I'd prefer to be doing but can be put behind a config option on my side as needed when I work on it.

could open-source the WACUP version of in_openmpt (maybe, libopenmpt license totally allows closed-source derivatives of course), either as a separate repository or maybe merging back with our version
re: providing a copy of my changes to the plug-in, I've long debated doing it but thought it'd possibly cause more confusion since it's all been WACUP-centric & doesn't really offer any benefit back to the plain Winamp plug-in. Plus with how I've been doing things anything provided isn't going to be buildable as there isn't the WACUP sdk to build it against for now.

The current WACUP beta build is using libopenmpt 0.6.1 + a bunch of changes including some more I did this past weekend to remove the whole main window message sniffing stuff (which goes against what this patch is using) as it's been the cause of some long-running stability issues that I finally tracked down (though how much that affects Winamp I don't know but I'm generally moving any plug-ins I use towards not doing things that are clearly Windows-centric).

In any case, given that original Winamp is probably dead, and given that WACUP is already using an adjusted version of in_openmpt anyway, we have already been considering to completely completely drop in_openmpt from our repository (see https://bugs.openmpt.org/view.php?id=1269).

I do know that there's those who will only "use 'official' winamp" so there possibly is still some benefit for providing a standalone version of the plug-in (assuming you really want to) or that it's used in some other players that emulate the basic Winamp api & don't want to make their own. I also can see the logic to want to drop it especially if it's not something you're using &/or there's not been much need to change it anyway what with 'classic' Winamp being in the non-moving state it now exists outside of users getting left behind on older versions (but I guess that's then one option to instead potentially point people to WACUP if they want something that's supported in the confines of a 'classic' winamp experience or use some other player.
Hopefully that all makes some sense.
-dro

This may be worthwhile also for XMPlay. @sagamusix may have some opinion on that.

As XMPlay already has proper subtune support, with the possibility of splitting subtunes into their own playlist entries and thus remove any unwanted subtunes, I don't think there is no need to add any such logic to the XMPlay plugin.

you could try to convince @dr0biwan to add subsong support directly in what WACUP ships

Well... I know Dro is going to eventually add this feature to his "in_mod" plugin, because in the configuration window you see this:

I believe I just couldn't wait and I made with my own implementation while I wait for Dro's.

I'm confused here. We are using C++17 or C++20 (for VS2022).

Sorry. I am the confused one, not you.

I would rather not include subsong support in the Winamp plugin in the current form.

The in_openmpt plugin currently limits itself to a subset of only Winamp 2.x functionality.

I'd not be too happy with what's proposed going into the plug-in.

I understand that. To be honest with you, this is the first time I work with C++ code for Windows, and the first time I work with a Winamp plugin. So, my solution may be not the best one 😔.
Thank you all for replying quickly & for your hard work ❤.
I will use this implementation meanwhile (I trust Dro is going to make a better one). If someone wants to try it out, there's a realease available.
I guess there is no much more to say, so I'm closing this pull request.
Greetings👍.


Support for splitting release manifest with Secrets storage driver

What this PR does / why we need it:

Implements splitting manifest mechanism for Secrets storage driver. This enables Helm to deploy, upgrade and generally deal with releases that have a Manifest over the hardcoded 1MiB Secrets data size restriction enforced by Kubernetes.
This has been proposed before, but never made it as a real HIP nor real code AFAICT (https://github.com/helm/community/pulls?q=is%3Apr+is%3Aclosed+split)

Special notes for your reviewer:

I've got a HIP ready to go on the community repo, will PR there to fully document this feature
I don't have much experience writing production-ready Go, so any feedback is appreciated on naming conventions, syntax and others
I haven't updated yet the version, as I'm not sure if this ought to be packaged with other changes or if it can go into 3.12 with the rest of the changes currently on master not in a released version
In order to test larger releases, I've added a text file that contains random alphanum chars (and a disclaimer), to be loaded as static content when running the tests. I'm not sure that's the best approach, I've tried

a smaller, random string, repeated with String.Repeat. This requires a lot more memory, as GZIP compression is efficient on repeated strings
loading random bytes directly in the test scenario. This doesn't work so well, as there's an increased risk of a test breaking out of randomness (i.e. a "large" text actually very compressible by GZIP would yield test failures). Also would tax the random generator of the test suite, which I'm not too fond of for a simple test.

If applicable:

[x] this PR contains documentation (comments)
[x] this PR contains unit tests (all edge cases I could think of)
[x] this PR has been tested for backwards compatibility (tested with Minikube and the lastest Helm version)

Thanks for your hard work maintaining Helm !

This has been proposed before, but never made it as a real HIP nor real code AFAICT

I stumbled to get traction on this in 2022 - I used same approach of splitting across multiple secrets:

HIP PR https://github.com/helm/community/pull/256 - check out the lack of enthusiasm from maintainers
Code PR https://github.com/helm/helm/pull/11087 - code afaics is worse than yours :-)

@ralfv I missed those ! Thanks for bringing them into context, and your work there :-)
I'll take a look through your code and HIP PRs, to try and see if the comments you got then apply to this PR and its HIP. Hopefully we can get some more traction this time !

Any news on this ?

@hoerup I asked for a review some time back in the Helm Dev section of the k8s Slack, but didn't hear back.
There's also a HIP PR open at https://github.com/helm/community/pull/284
And @ralfv also linked to his version of basically the same idea, which got some pushback from the Helm team at the time wrt reverse compatibility (split manifest release with older tooling), IIRC.
I personally have been using the fork (https://github.com/lelithium/helm/tree/lcabrol/split-manifest) since end of March 2023, and haven't run into any issues.
There's a separate release in that repo I linked to with version number 3.11.999 for disambiguation with proper releases.
All of our deployment tooling is automated, so having the tools pull the forked version works well for our purposes, but YMMV

Note: transferred ownership from lelithium to iotile, since we're using this fork internally. https://github.com/iotile/helm


Agent should be running without ACM hub installed

the PR address the following cases:

if the managed cluster is not a managed hub cluster, the agent should be running normally w/o restarting. And the hub.open-cluster-management.io clusterclaim value is NotInstalled
if add label global-hub.open-cluster-management.io/hub-cluster-install= to this managed cluster, the agent should keep running and start the corresponding controllers to handle the policy and managedcluster.

How to address the above cases:

w/o MCH or clustermanager, start the controller to create hub.open-cluster-management.io clusterclaim and start the crd controller to watch the crd.
if the crd clustermanager is created, then start the rest of controllers.

ref: https://issues.redhat.com/browse/ACM-8556

/retest

/retest

/retest

/test test-unit

/lgtm


Clarify req.originalUrl behaviour with query string in request

Fixes #1229 by updating example and adding more details of how req.originalUrl behaves when query string is present in request.

Maybe we should really just say that it is the combination of req.baseUrl and req.url? I think, looking back at the issue, this is probably how it should have been specified, as req.path is just req.url without the query string.

Ah - yeah, that's easier. Thanks, @dougwilson.
`req.originalUrl` is available both in middleware and router objects, and is a
combination of `req.baseUrl`, `req.url`. Consider following example:

@rodion-arr, what do you think?

Maybe we should really just say that it is the combination of req.baseUrl and req.url? I think, looking back at the issue, this is probably how it should have been specified, as req.path is just req.url without the query string.

Ah - yeah, that's easier. Thanks, @dougwilson.
`req.originalUrl` is available both in middleware and router objects, and is a
combination of `req.baseUrl`, `req.url`. Consider following example:

@rodion-arr, what do you think?


Bug fix to make the workflow tasks stateless and some modifications in the bash script that runs jobs

The workflow tasks were keeping some state which was creating issues while running multiple jobs as the state was being shared across jobs. Moved all state into local variables to fix that problem.
The run.sh script to run the job now is just a "run" script with a bash header and executable bit set.

Coverage increased (+0.09%) to 89.378% when pulling 9f8505f10d6a92ae5aac33c13ba371240622098f on amitsharma1981:develop into 18bc1a489cf862b596d48558a83b14769eaba0bd on Netflix:develop.


Added methods to eazydiner

Description
Added get_top10() methods in eazydiner and updated documentation
Resolves: #701
Checklist

Before submitting this pull request, kindly verify that the ensuing checkpoints have been reached.

[x] Have you adhered to the repository's defined coding convention rules?
[x] Have you updated the 'documentation.md' file with the method/function documentation?
[x] Have you sent a message along with the result or response?
[x] Have you used the try-catch technique?
[x] Has the method/class been added to the documentation (md file)?

Screenshots

Additional Notes/Comments

Any additional remarks or suggestions concerning this pull request are welcome.

I certify that I have carried out the relevant checks and provided the requisite screenshot for validation by submitting this pull request.
I appreciate your contribution.

In the documentation, Github and Internshala are written twice, just informing you incase if you want to change it

Great work @Juhibhojani !!
Do not forget to

Share your contribution on socials(twitter mainly), tag us and Clueless 🫂
Give this repository a star ✨
Follow Clueless Community on GitHub ✅


Enable support users to edit a school's Chromebook details

Context
In the case that:

A school or RB user orders some Chromebooks
Post-ordering, Computacenter determine that the school's Chromebook details are missing or wrong
They inform the school/RB
If the school user writes back into support with their Chromebook details, instead of entering the details into the service

then the support agent has to explain the user to go enter it into the service themselves. This means wasted effort and a delay for the user.
Changes proposed in this pull request
Allow support agents to update Chromebook details (yes/no on whether the school needs Chromebooks, the domain and recovery email), in case they get a ticket with those details.
Guidance to review
The wording is a bit off ("when you order", "we", etc) as a result of reusing partials from the RB and school parts of the service, but it felt like a reasonable trade-off to having to duplicate the templates just for the support interface.
Screenshots

Nice 👍


Wash up windows 10 error

Describe the bug
Hello good, I have a question when installing it in windows with choco I get the following error:
wash up --detached
 Failed to request release tarball: reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("github.com")), port: None, path: "/nats-io/nats-server/releases/download/v2.9.14/nats-server-v2.9.14-windows-amd64.tar.gz", query: None, fragment: None }, source: hyper::Error(Connect, Custom { kind: Other, error: Custom { kind: InvalidData, error: InvalidCertificate(UnknownIssuer) } }) }
How do I get up the wasmCloud user interface?
To Reproduce
Steps to reproduce the behavior:

install wash with choco
Run wash up --detached
See error

Expected behavior
raise the environment
Screenshots

**Environment (please complete the following information) **

OS: Windows
Cmd
wash 0.18.0

Hey @ldariom, thanks for filing this issue. I'll see if I can reproduce on my Windows machine, that issue seems to point to either a temporary downtime for GitHub or missing ca-certificates 🤔
InvalidData, error: InvalidCertificate(UnknownIssuer)

The URL looks correct at least, that version of NATS should be good.


http://mailonline.github.io/videojs-vast-vpaid/ is broken

Seems the latest gh-pages deploy broke things.

@jhenkens thanks for letting us know, fixed with this commit: https://github.com/MailOnline/videojs-vast-vpaid/commit/d760f7d92ee0cb3cd6c2c7448281724e2dd928b1
Is working now http://mailonline.github.io/videojs-vast-vpaid/index_5.html


Safari iPhone 12 bit

Safari UI is broken on iPhone

sorry you can specify more details about what iphone model is?
and what browsers ?
thank you


Document how to close with the ESC key

Is your feature request related to a problem? Please describe.
My problem is that I was expecting that pressing ESC would close the autocomplete window (I've seen that referred as ... "pug" in the issues?). It does not, by default. But the suggested mappings in the doc don't explain how to accomplish this.
I am not super familiar with the different ways Vim allows mapping keys (imap vs innoremap) so perhaps I am missing something on that side.
Describe the solution you'd like
I'd like the docs to include a mapping that makes ESC close the autocomplete window.
Describe alternatives you've considered
I tried doing this:
inoremap <silent><expr> <ESC>     compe#close('<ESC>')

It did not go well because:

It did not close the window
It showed an error message
It made it impossible to go back to normal mode in Vim xD. I had to edit my init.vim file in nano in order to rescue it.

Additional context
As an extra, I usually don't use <ESC>, but a combination of j and k. So I have this on my conf:
imap jk <ESC>
imap Jk <ESC>
imap JK <ESC>

Ideally (for me) the provided mapping should be compatible with that (so pressing jk in rapid succession would close the autocomplete window, but otherwise individual presses of j and k allow selecting from alternatives on it.

I tried to map compe#close('<esc>') to <Esc> but I couldn't.
I investigated it then I found the cause that is lexima.vim.
If you can't map <esc> in insert-mode, you should check imap results.

Ok my problem was that I had the vim-autoclose plugin installed, and it interfered with the usage of the <ESC> key. This issue has been reported in other autocompletion plugins ( https://github.com/Shougo/deoplete.nvim/issues/386 , https://github.com/ycm-core/YouCompleteMe/issues/9).
The fix seems to be adding the following setting for the vim-autoclose plugin:
let g:AutoClosePumvisible = {"ENTER": "<C-Y>", "ESC": "<ESC>"}

I've sent a PR which documents this in #409 . Thank you for creating this plugin!


Import and export of crosswalks

This is not a priority but it will make sense to include an import and export of crosswalks in common standards, which seems to include ISO 19115 and CKAN to RIF-CS, the schema that underpins the Research Data Australia (RDA) Registry and Research Data Australia. These formats all look like it's XML.
More info at: https://archive-intranet.ardc.edu.au/display/DOC/Crosswalks:+Transform+your+metadata

Also want a standard way to export & import xmaps

Look up standard graph storage formats


juniper::GraphQLType cannot be made into an object

I am trying to create a Vector with dynamic GraphQLTypes, I can do it with dynamic GraphQLValues but not Types as they can't be made into object.
Is there any way to approach this?
The code
let mut v: Vec<Box<dyn GraphQLType<Context = (), TypeInfo = ()>>> = Vec::new();

The error
error[E0038]: the trait `juniper::GraphQLType` cannot be made into an object
   --> src\api\schema\mod.rs:86:13
    |
86  |     let mut v: Vec<Box<dyn GraphQLType<Context = (), TypeInfo = ()>>> = Vec::new();
    |                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ `juniper::GraphQLType` cannot be made into an object
    |
note: for a trait to be "object safe" it needs to allow building a vtable to allow the call to be resolvable dynamically; for more information visit <https://doc.rust-lang.org/reference/items/traits.html#object-safety>
   --> C:\Users\Kenneth\.cargo\registry\src\github.com-1ecc6299db9ec823\juniper-0.15.7\src\types\base.rs:395:8
    |
395 |     fn name(info: &Self::TypeInfo) -> Option<&str>;
    |        ^^^^ the trait cannot be made into an object because associated function `name` has no `self` parameter
...
398 |     fn meta<'r>(info: &Self::TypeInfo, registry: &mut Registry<'r, S>) -> MetaType<'r, S>
    |        ^^^^ the trait cannot be made into an object because associated function `meta` has no `self` parameter

It could be solved by adding Sized to the trait members, I can create pull request if needed.

@KennethGomez no, there is no way to approach it. GraphQLType is not object-safe by design, and never will be. GraphQLValue was disolved from GraphQLType exactly because of that. Either use GraphQLValues, or reconsider the whole approach.

Thank you very much for your time, in that case, how would you implement a dynamic vector inner type to implement it onto another Type field generation?
Let's say that you have a custom struct with a set of properties and depending on those you would create a field on the GraphQLType itself being a dynamic vector. Example: [String!]! [[Float!]!]! etcetera.

@KennethGomez I think the easiest way for each your concrete type to provide transformation into &[&str], &[f32] (or Vecs) and use them in GraphQL schema.
If I've understood you correctly:
struct Foo {
    name: String,
}

impl From<Foo> for Vec<String> {
    fn from(foo: Foo) -> Self {
      vec![foo.name]
    }
}

struct Bar {
    id: u32,
    name: String,
}

impl From<Bar> for Vec<String> {
    fn from(bar: Bar) -> Self {
      vec![bar.id.to_String(), bar.name]
    }
}

struct MyGraphQLType;

#[graphql_object]
impl MyGraphQLType {
    fn my_field() -> Vec<Vec<String> {
        vec![
            Foo { name: "John".into() }.into(),
            Bar { id: 3, name: "R2-D2".into() }.into(),
        ]
    }
}

Thanks, I don't think that you have understood the problem that I am facing, maybe it's better to give you the line on where I am trying to implement this:
https://github.com/ForetagInc/Alchemy/blob/dev/engine/src/api/schema/mod.rs#L84
I'll explain to you, so on the ScalarType::Array(t), t can be another ScalarType::Array with another inner array or another valid GraphQLType primitive, so how would I implement the field generation for dynamic Vector type?

@KennethGomez the similar way we do this for serde_json::Value. It should be definitely a recursive descent.


[mtsource] performance and scalability evaluation

Problem
There is no performance and scalability tests targeting the mt source
Persona:
Which persona is this feature for?
Exit Criteria
A measurable (binary) test that would indicate that the problem has been resolved.
Time Estimate (optional):
How many developer-days do you think this may take to resolve?
Additional context (optional)
Add any other context about the feature request here.

/reopen
/remove-lifecycle stale


No autocomplete when creating a linked model

Bug description
I'm building a small side-project for fun, and I've noticed that when I try to do
MODEL -> CREATE -> DATA -> LINKED_MODEL -> CREATE/CREATEORCONNECT/CONNECT
In some models I do get autocomplete for the LINKED_MODEL but in most of them I do not:
const created = await db.entity.create({
        data: {
            ...data,
            address: {
                create: {
                   //HERE I get autocomplete
                }
            },
            contact:  {
                create: {
                    ...data.contact
                }
            }
        }
    })

const created = await db.center.create({
        data: {
            ...data,
            address: {
                create: {
                   //HERE I DO NOT get autocomplete
                }
            }, //IN fact If I try to autocomplete here, it will not show [connect |
connectOrCreate | create]
            contact:  {
                create: {
                    ...data.contact
                }
            }
        }
    })

I've tried restarting VSCODE and regenerating the client and deleting nodemodules and reinstalling.
How to reproduce
Create a prisma client with the provided, models
Attempt to create an Entity and Address //Autocomplete shows up
Attempt to create a Center and Address //No autocomplete
Expected behavior
Same autocomplete behavior for all models
Prisma information

model Entity {
  id              String   @id @default(uuid())
  name            String
  type            String?
  industry        String?
  size            Int?
  contactId       String?
  contact         Contact? @relation(fields: [contactId], references: [id], name: "EntityContact")
  addressId       String?
  address         Address? @relation(fields: [addressId], references: [id], name: "EntityAddress")
  status          String   @default("active")
  legalIdentifier String?
  centers         Center[]
  createdAt       DateTime @default(now())
  updatedAt       DateTime @updatedAt
}

model Center {
  id          String   @id @default(uuid())
  name        String
  description String?
  headquarter Boolean  @default(false)
  entityId   String
  contactId   String?
  addressId   String?
  members     Member[]
  entity      Entity   @relation(fields: [entityId], references: [id])
  address     Address? @relation(fields: [addressId], references: [id], name: "CenterAddress")
  contact     Contact? @relation(fields: [contactId], references: [id], name: "CenterContact")

  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
}

model Address {
  id         String   @id @default(uuid())
  line1      String
  line2      String?
  city       String
  state      String
  postalCode String
  country    String
  latitude   Float? // Optional: For geolocation purposes
  longitude  Float? // Optional: For geolocation purposes
  Center    Center[] @relation("CenterAddress")
  Member    Member[] @relation("MemberAddress")
  Entity    Entity[]  @relation("EntityAddress")
  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
}

model Contact {
  id          String   @id @default(uuid())
  email       String? // Optional: Not all contacts may have an email.
  phoneNumber String? // Optional: Not all contacts may have a phone number.
  centers     Center[] @relation("CenterContact")
  members     Member[] @relation("MemberContact")
  Entity    Entity[]  @relation("EntityContact")

  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
}

    const createdCenter = await db.center.create({
        data: {
            name: '',
            description: '',
            headquarter: false,
            entityId: created.id,
            contact: {
                create: {
                    id
                }
            }
        }
    })

    const created = await db.entity.create({
        data: {
            ...data,
            address: data.address ? {
                create: {
                    ...data.address,
                    line1: data.address.line1,
                }
            } : undefined,
            contact:  {
                create: {
                    ...data.contact
                }
            }
        }
    })

Environment & setup

OS:
Database:
Node.js version:

Prisma Version
prisma                  : 5.9.1
@prisma/client          : 5.9.1
Computed binaryTarget   : debian-openssl-3.0.x
Operating System        : linux
Architecture            : x64
Node.js                 : v20.11.0
Query Engine (Node-API) : libquery-engine 23fdc5965b1e05fc54e5f26ed3de66776b93de64 (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli 23fdc5965b1e05fc54e5f26ed3de66776b93de64 (at<EMAIL_ADDRESS>Schema Wasm             : @prisma/prisma-schema-wasm 5.9.0-32.23fdc5965b1e05fc54e5f26ed3de66776b93de64
Default Engines Hash    : 23fdc5965b1e05fc54e5f26ed3de66776b93de64
Studio                  : 0.497.0

Okay I just got back home and realized this:

On Entity the address is set to the correct type "Prisma.AddressCreate..."
On Center the address is set to undefined

Solved: You cannot use ID connectors + Nested Create / Connect
You must either link all via ID or via Create / Connect


Add an example for azure ml managed feature store

We are adding an example of how one can connect to azure ml managed feature store and use such a prompt template in a llm chain. @baskaryan

@baskaryan, requesting review on this pull request.


Fix null method call blocking install

Just updated to check out the latest update, but I immediately ran into a wall on clean install:

Added a single ?, reload, and...

good to go!
Something, something, rewrite it in Rust 🦀
Also, I've been using a barebones, unpublished fork of this extension for sometime which finds and uses a rustup installed RA component in either nightly or stable (now that it's available on stable) according to whichever toolchain I use on a per-project basis. Would be happy to put together another PR if you're interested in adding something along those lines.

Thanks for bringing up this issue! I made the poor assumption that the 'empty' value for a string array preference would be an empty array, but it looks like it's null instead. I obviously neglected to test the case of no preferences.
I'm going to accept this PR, but I'll likely also try to fix the issue before it gets to this function – either where this function is called or where the preference is resolved. Thanks!

I made the poor assumption that the 'empty' value for a string array preference would be an empty array, but it looks like it's null instead.

Yeah for sure, if the error trace didn't say it was null I probably wouldn't have thought that string[] could be nullish.

I'll definitely get around to opening an issue before a PR for that feature. I assume there'd be a non-trivial amount of refactoring needed for something like anyways.


Passport initialization for social login

I am trying to implement oauth for github using passport into my keystone.js app.
I have been using this repo as a guide. In my console I am getting to lib/auth/github.js
[services.github] - Authentication workflow detected, attempting to request access...

However I get a 500 error in the browser:
Sorry, an error occurred loading the page (500)

passport.initialize() middleware not in use

How or where do you initialize the passport middleware? I am searching through the repo and cannot find a call to passport.initialize(

Got a little bit past this using routes/auth/app.js routes/auth/confirm.js. What is the github registered url callback for this application? (Am kind of a n00b but really excited about putting social auth into the power of a keystone app)

what do you mean by using routes/auth/app.js and routes/auth/confirm.js.  .they are already being used.
Can someone elaborate the solution.
I generally think that there is no one on the planet to answer question about keystoneJS. on stackoverflow as well.

There is an issue with ur callback url for sure

Yes you are right, the callback URL must have  ?cb=true  at the end
http://localhost:3003/auth/facebook?cb=true
I have written a detailed blog on resolving all issues here.

Okay got it thank you!


Docs: Add full example of HTTP->HTTPS redirect

What would you like to be added:
Addition to https://gateway-api.sigs.k8s.io/guides/http-redirect-rewrite/#redirects that includes the following:

A Gateway with HTTP and HTTPS listeners
A HTTPRoute attached to the HTTP listener that does the redirect to HTTPS
A simple HTTPRoute attached to the HTTPS listener that forwards to any sample application

Why this is needed:
Currently our redirect docs have a very basic redirect example, but they leave out some very important details. This was made especially clear in https://github.com/kubernetes-sigs/gateway-api/issues/1185#issuecomment-1832955997 (thanks @HummingMind).

I would like to work on this.
/assign

Until I read this bug I assumed that you cannot attach a httproute only to the port 443 listener (or any other specific listener).  Is that true? Or not?


Add Bluetooth Classic RFCOMM support

RFCOMM allows serial port emulation over classing Bluetooth.  This expands the flexibility and potential application uses for ESP32.

This turned out to be duplicate of #626.


stopit not timing out while opening file

Hi,
On windows 10 stopit doesn't seem to timeout at all when doing file reads with pyarrow.Parquet. This works fine for me:
import stopit
import pyarrow.parquet as pq

def infiniteloop():
    with stopit.ThreadingTimeout(5) as to_ctx_mgr:
        assert to_ctx_mgr.state == to_ctx_mgr.EXECUTING
        while True:
            pass
    print(str(to_ctx_mgr.state))
    return False

infiniteloop()

however this:
import stopit
import pyarrow.parquet as pq

def ReadMetaFromDataset(filepath):
    with stopit.ThreadingTimeout(5) as to_ctx_mgr:
        assert to_ctx_mgr.state == to_ctx_mgr.EXECUTING
        schema = pq.ParquetDataset(filepath).schema
        return schema
    print(str(to_ctx_mgr.state))
    return False

ReadMetaFromDataset(r"\\lcdevnas01\development\ENav2")

and this:
import stopit.threading_timeoutable
import pyarrow.parquet as pq

@threading_timeoutable(default='not finished')
def ReadMetaFromDataset(filepath):
    with stopit.ThreadingTimeout(10) as to_ctx_mgr:
        assert to_ctx_mgr.state == to_ctx_mgr.EXECUTING
        schema = pq.ParquetDataset(filepath).schema
        return schema
    print(str(to_ctx_mgr.state))
    return False

print(ReadMetaFromDataset(r"\\lcdevnas01\development\ENav2", timeout=5))

runs to completion after 90 minutes. Anything I can do with stopit to force timeouts or do I have to look into threading and kill from a separate process?
Thanks
Asbjorn

Hi !
It seems that your example matches one of the limitations of stopit due to limitations of Python threading model. This limitation is clearly described in the README.
pyarrow.paquet is a Python binding over a C / C++ library that locks the threads as most bindings do, thus the special exception raised has no effect since thread switching is locked while such code is running.
With a Linux / MacOS box, you could use the SignalTimeout.
Since I have no Windows box, I cannot explore other solutions.


Please user higher test timeout scale for ARM

What version of Go are you using (go version)?

$ go version
go version go1.14.7 linux/arm

Does this issue reproduce with the latest release?
Most likely (see #42798), but we carry carry the fix in Ubuntu so I'm not absolutely sure.

Ran a test on armhf:
<EMAIL_ADDRESS>What did you expect to see?
Test passing, like on all other architectures.
What did you see instead?
Internal test timeout:
=== RUN TestOpaqueKeyRoundtripJWE
panic: test timed out after 10m0s

Also reported as https://bugs.launchpad.net/ubuntu/+source/golang-gopkg-square-go-jose.v2/+bug/1893640 and fixed in Ubuntu by bumping arm's timeout scale to 3 from 2.
I had hard time resulting only a failure to get cross-company CLA signed, thus I'm not filing a PR, but the change is trivial. Please consider applying it.
The reasoning is that most developers use (faster) amd64 machines setting the test timeouts, but test may run on slower arm ones and also different tests may suffer different relative slowdown on the arm CPUs.
Due to those two varying factors it is safer to allow a higher timeout scale on arm to avoid tests timing out.

You can set the environment variable GO_TEST_TIMEOUT_SCALE to increase timeout for all tests.

Unlikely that it's related to 42798, since the code doesn't seem to be using float slices.

It seems that golang.org/x/crypto/pbkdf2.Key is the hotspot in that test (with arm).

Thank you for the quick replies. The problem was not unique, we observed timeouts in multiple packages. Also the armhf test runners are slower machines than the ones running test for other architectures and I expect this to be typical. Since the timeout scale default is already set by architecture I think relaxing it for arm makes sense. Or there could be different default scales for 64 bit and 32 bit arm architectures.

For reference: #10314 (cmd/dist: default timeout value is too short for weaker CPUs), and #16106. Solved by introducing the GO_TEST_TIMEOUT_SCALE variable @cherrymui mentioned, in 7579867fece7e42b38d69eca202182f5de786390.

Instead of increasing timeoutScale or skipping tests, this CL
introduces an environment variable $GO_TEST_TIMEOUT_SCALE that
could be set to manually set a larger timeoutScale for those
machines/builders.


Use customize main entry point instead of auto generated main build failed with WMC999 Object reference not set to an instance of an object

Describe the bug
Add customized Main entry point to load App instead of autogenerated code, after change build project failed and show below error:
Severity	Code	Description	Project	File	Line	Suppression State
Error	WMC9999	Object reference not set to an instance of an object.	TestBasePackageTool	.nuget\packages\microsoft.windowsappsdk\1.0.0\build\Microsoft.UI.Xaml.Markup.Compiler.interop.targets	616
Steps to reproduce the bug

Set App.xmal Build Action to "Page"

Add Main entry point as below

Set Startup object to newly added entry point
Build project and get blow failed mesage

Expected behavior
could build success and launch windows after run
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.0 (If you're seeing your issue in older previews of WinUI 3, please try this release)
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
No response
Additional context
No response

@fabiant3 and @evelynwu-msft is this supported?

Have you added this to your .csproj file?
        <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">
		<DefineConstants>DISABLE_XAML_GENERATED_MAIN</DefineConstants>
	</PropertyGroup>
	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x86'">
		<DefineConstants>DISABLE_XAML_GENERATED_MAIN</DefineConstants>
	</PropertyGroup>
	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|x86'">
		<DefineConstants>DISABLE_XAML_GENERATED_MAIN</DefineConstants>
	</PropertyGroup>
	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'">
		<DefineConstants>DISABLE_XAML_GENERATED_MAIN</DefineConstants>
	</PropertyGroup>
	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|arm64'">
		<DefineConstants>DISABLE_XAML_GENERATED_MAIN</DefineConstants>
	</PropertyGroup>
	<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|arm64'">
		<DefineConstants>DISABLE_XAML_GENERATED_MAIN</DefineConstants>
	</PropertyGroup>

I did something similar to achieve single instancing and for me it's working.


pass casa loopback test

export a VLA measurement set to uvfits, read in to pyuvdata, write out to pyuvdata,

Done in commit d0e339d.


Nuke USD plugin does not support instanced geometry

Greetings!
I wanted to log this issue on its own to continue discussion on adding support for instanced geometry to the Nuke USD plugin.
I checked into providing a test scene with examples of instancing and it does appear that the Kitchen test scene provided by Pixar has two versions with and without instancing. Here is the path to the kitchen set I tested: http://graphics.pixar.com/usd/downloads.html
Let me know your thoughts on adding basic support for instancing.  I think starting with a deferred-loading option makes sense to me and will let us test to see how well Nuke's 3D system can handle (or can't handle as the case may be) an average scene we'll be using in production.  Certainly of course, I'm open to other suggestions and ideas for first steps that we can try as well!
Cheers,
Morgan

Hi Morgan,
Finally looking at this and it's simpler than I feared it would be. When I was thinking 'instanced geometry' I was thinking the 'thousands of leaves on a tree in a forest of trees' kind of instancing. ie UsdGeomPointInstancer.
However it appears the basic scenegraph prim instancing supported by usd is not addressed when you use UsdPrim::GetAllChidlren(), which I naively thought would get me all the prim's children...silly me. The right method to use is GetAllFilteredChildren() with the UsdTraverseInstanceProxies() predicate enabled.
So that simple change gets all the instanced prims and Nuke is copying them so they're no longer instanced, but at least they're reading in now.
I'll get that fix in the same upcoming release as the ReadGeo fixes.
Cheers,
-jonathan

This should now be fixed in the latest code updates.

Hi Morgan,
There was a bug in the instancing support code that caused prim memory to become invalid due to the prim.Load() method destroying the proxy prim.
This is now fixed in the latest commits.
Cheers,
-jonathan

Hey Jonathan!  Thanks for the heads up!  We're in the process of updating to your latest build now.  I'll let you know how it works out once I get to test it a bit.  Thanks again!


[Logic] Fix workflow update method

Fix https://github.com/Azure/azure-sdk-for-net/issues/33246
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Logic


Davros outdated in sandstrom appstore??

Hi
Version 0.21.7 (sandstrom app store) is far behind github. Is this the reason why BUGS like the one described in issue #69 with  File deletion are not addressed?
Best,
Jörg

The primary difference, as far as I know, is that 0.23 has added LibreOffice binaries for document previews. But it'd probably be good to get an update on this.

Davros is no longer outdated on the Sandstorm App Market! https://apps.sandstorm.io/app/8aspz4sfjnp8u89000mh2v1xrdyx97ytn8hq71mdzv4p4d8n0n3h
However, ownCloud/Nextcloud clients are not currently supported in Davros.


Update dataflow-timeseries-samples-library to use latest and install from maven

Library updates have broken the build/deploy. Failure is here:
apache_beam.coders.coder_impl.FastPrimitivesCoderImpl.encode_special_deterministic TypeError: Unable to deterministically encode '<TableReference datasetId: 'forex' projectId: 'fsi-demo-pres' tableId: 'prices'>' of type '<class 'apache_beam.io.gcp.internal.clients.bigquery.bigquery_v2_messages.TableReference'>', please provide a type hint for the input of 'Write to BigQuery (prices)/_StreamToBigQuery/CommitInsertIds/GroupByKey' [while running 'Write to BigQuery (prices)/_StreamToBigQuery/CommitInsertIds/Map(reify_timestamps)-ptransform-1768936'] passed through: ==> dist_proc/dax/workflow/worker/fnapi_service_impl.cc:644 generic::unknown: Traceback (most recent call last): File

Pull request:
python beam sdk update -> 2.30.0 #18


Update YAML file Test_TC_OPCREDS_3_7

This PR updates the TC_OPCREDS_3_7 YAML file by removing steps to align the test procedure with the Fixes: #423
Updates:

The YAML test script was updated to reflect these changes.
The TC_OPCREDS_3_7 test procedure in the test plan has been updated in PR #4820.

Testing for Multiple Fabrics:
This test validates the TC_OPCREDS_3_7 procedure when two fabrics, gamma and the default alpha, are present on the Device Under Test (DUT). The process includes commissioning the DUT with gamma using the --commissioner-name gamma flag, opening a commissioning window, re-commissioning with alpha, and confirming the test passes successfully with both fabrics in use.
Example of Test Steps (Passed):

In the first console, Run the all-clusters app:
rm /tmp/chip_*
./out/darwin-arm64-all-clusters/chip-all-clusters-app

In the second console, Perform the first commissioning:
./out/darwin-arm64-chip-tool/chip-tool pairing code 0x12344321<PHONE_NUMBER>65521327694 --commissioner-name gamma
Note: As example I use the manual pairing code<PHONE_NUMBER>65521327694 instead of the QR code payload ID MT:Y.K9042C00KA0648G00.

Opened the commissioning window and searched for "Manual pairing code" in the output:
./out/darwin-arm64-chip-tool/chip-tool pairing open-commissioning-window 0x12344321 1 300 1000 2365 --commissioner-name gamma

Re-commissioned with a second commissioner (alpha):
./out/darwin-arm64-chip-tool/chip-tool pairing code 0x12344321<PHONE_NUMBER>3 --commissioner-name alpha

In the third console, Run the test:
./scripts/tests/chipyaml/chiptool.py tests Test_TC_OPCREDS_3_7 --server_path ./out/darwin-arm64-chip-tool/chip-tool.

As result:
***** Test Complete: Test_TC_OPCREDS_3_7
✓ Test finished in 1344ms with 35 success, 0 errors and 0 warnings
✓ Parsing finished in 8ms with 1 success and 0 errors.
Run finished in 2548ms with 16 steps runned and 0 steps skipped.

Changes made:

Removed the factory reset from the YAML file.
Adjusted the test procedure to align with the updated format from the linked test plan PR.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[Table of Contents] fix smooth scroll and scroll offset

Setting location hash will trigger browser navigation jump

What issue does this solve?

On the table of content, if you click the link, it will jump directly to the heading, ignoring the smooth scroll and the scroll offset

This brings a regression, as even when setting scroll-behavior to auto, the scroll is now smooth.

Thank you! 🎉


fix: Use root.hcl as the root Terragrunt configuration

Description
Relates to #3181.
This updates the configurations of this example to use root.hcl as the name of the file at the root of the repository instead of terragrunt.hcl.
Calling the file terragrunt.hcl does not make Terragrunt operate any better, but it does result in confusion around edge-cases that can arise during Terragrunt usage.
Follow-up work will be done to make Terragrunt itself aware of this risk.

TODOs
Read the Gruntwork contribution guidelines.

[x] Update the docs.
[x] Run the relevant tests successfully, including pre-commit checks.
[x] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[x] Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Updated name of root configuration file to root.hcl.

terragrunt hclfmt seems to generate formatting changes
non-prod/us-east-1/qa/mysql/terragrunt.hcl

-  path   = "${dirname(find_in_parent_folders("root.hcl"))}/_envcommon/mysql.hcl"
+  path = "${dirname(find_in_parent_folders("root.hcl"))}/_envcommon/mysql.hcl"


The link of qvhighlights features is missing

Thank you great work! Can you provide a new link? Thank you very much.

Hi @zsxzs, Unfortunately I lost the original feature files as well. You may need to download the original videos and extract them on your own.   -- Jie

@zsxzs Were you able to generate the features? In case you did, would be great if you could share please.

You can download qvhighlights features from this link: https://huggingface.co/datasets/dpaul06/QVHighlights_preprocessed/tree/main
The code can run, but there may be not the original features.


YAML format not set in front matter

When creating new site with YAML format, default.md stays with TOML front matter and vice versa. I have pinpointed the problems to changes between versions v0.117.0 and v0.118.0. The comparison was done using Scoop on Windows for both -f and --format flags. Also, the same is still present in v0.129.0: creation using --format "yaml" does not include front matter of archetypes\default.md in YAML.
comp-v0.117.0-v0.118.0-f.log
comp-v0.117.0-v0.118.0--format.log
v0.129.0--format.log

What version of Hugo are you using (hugo version)?

$ v0.117.0
$ v0.118.0
$ v0.129.0

Does this issue reproduce with the latest release?
Yes.

The same applies to the JSON Format.

We reimplemented the new site command (I guess v0.118.0). I have checked and we now use the --format flag only for the config file, the rest is simple file copy.


Windows Desktop : Mount home directory

Mount home directory on a specific drive letter whatever this is using NFS or SMB
https://stackoverflow.com/questions/182750/map-a-network-drive-to-be-used-by-a-service
https://social.technet.microsoft.com/Forums/en-US/84a89d4e-7a2e-4b5b-9ab6-68c68cabf3e0/map-network-drive-auto-connect-for-all-users

Windows Desktop support has been removed


[proposal] make path output from generate schema dynamic

What issue is this PR fixing
Example:
fixes #1317
What is being changed
Adding a new parameter to the cli so the path and file of the plugin.schema.json.
Quality
Check all that apply:

[X] I want these changes to be integrated
[X] I successfully ran pnpm i, pnpm build, pnpm test, pnpm test:browser locally.
[X] I allow my PR to be updated by the reviewers (to speed up the review process).
[ ] I added unit tests.
[ ] I added integration tests.
[X] I did not add automated tests because _________, and I am aware that a PR without tests will likely get rejected.

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

Comparison is base (c050588) 85.47% compared to head (aad9354) 85.47%.

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #1318   +/-   ##
=======================================
  Coverage   85.47%   85.47%
=======================================
  Files         170      170
  Lines       18952    18952
  Branches     2116     2116
=======================================
  Hits        16199    16199
  Misses       2753     2753

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


Unset $PASSWORD after grabbing it

Closes #1583

In the future we ought to add a flag/config file option and remove docs on $PASSWORD to avoid issues like these.


unable to start waybar from conf of hyprctl

exec-once = waybar shows nothing (process is started but nothing is shown)
Unsurprisingly hyprctl dispatch exec waybar does the same thing as exec-once
Starting waybar manually from a terminal works as expected

monitor config (as it seems to be a repeating source of issues)
hyprctl monitors
Monitor DP-1 (ID 1):
<EMAIL_ADDRESS>at 11920x9180
	description: Dell Inc. DELL U2720Q 12PZP83 (DP-1)
	active workspace: 1 (1)
	reserved: 0 28 0 0
	scale: 1.30
	transform: 0
	focused: yes
	dpmsStatus: 1

(clamshell mode. laptop screen off)

try reverting 1e5cab1ee7a82d3381f55963a0ff768c30215602 and lmk how it goes

Funny. i was doing just that

But just reverting this commit breaks the build

what

i can't build after doing git revert 1e5c

Going back just before this commit fixed everything. Let me try with HEAD set to 1e5c

Yea. Issue is introduced by 1e5c

reverted.

same issue I had with me reverting it: build is broken
cmake --build ./build --config Release --target all -j12
[1/51] Building CXX object CMakeFiles/Hyprland.dir/src/debug/HyprCtl.cpp.o
FAILED: CMakeFiles/Hyprland.dir/src/debug/HyprCtl.cpp.o
/usr/bin/c++ -DGIT_BRANCH=\"main\" -DGIT_COMMIT_HASH=\"478fa7cafea63a2008184a756045c43ed803310f\" -DGIT_COMMIT_MESSAGE="\"    Revert \"Set child stdout and stderr to /dev/null (1000)\"\"" -DGIT_DIRTY=\"dirty\" -I/home/antoine/.cache/yay/hyprland-git/src/hyprland/. -I/home/antoine/.cache/yay/hyprland-git/src/hyprland/PRIVATE -I/home/antoine/.cache/yay/hyprland-git/src/hyprland/subprojects/wlroots/include -I/home/antoine/.cache/yay/hyprland-git/src/hyprland/subprojects/wlroots/build/include -isystem /usr/include/cairo -isystem /usr/include/lzo -isystem /usr/include/libpng16 -isystem /usr/include/freetype2 -isystem /usr/include/harfbuzz -isystem /usr/include/glib-2.0 -isystem /usr/lib/glib-2.0/include -isystem /usr/include/sysprof-4 -isystem /usr/include/pixman-1 -isystem /usr/include/pango-1.0 -isystem /usr/include/libmount -isystem /usr/include/blkid -isystem /usr/include/fribidi -isystem /usr/include/libdrm -O3 -DNDEBUG -O3 -std=c++23 -DWLR_USE_UNSTABLE -Wall -Wextra -Wno-unused-parameter -Wno-unused-value -Wno-missing-field-initializers -Wno-narrowing -pthread -MD -MT CMakeFiles/Hyprland.dir/src/debug/HyprCtl.cpp.o -MF CMakeFiles/Hyprland.dir/src/debug/HyprCtl.cpp.o.d -o CMakeFiles/Hyprland.dir/src/debug/HyprCtl.cpp.o -c /home/antoine/.cache/yay/hyprland-git/src/hyprland/src/debug/HyprCtl.cpp
/home/antoine/.cache/yay/hyprland-git/src/hyprland/src/debug/HyprCtl.cpp: In function ‘std::string versionRequest(HyprCtl::eHyprCtlOutputFormat)’:
<command-line>: error: unable to find string literal operator ‘operator""Set’ with ‘const char [12]’, ‘long unsigned int’ arguments
<command-line>: note: in definition of macro ‘GIT_COMMIT_MESSAGE’
<command-line>: error: unable to find string literal operator ‘operator""Set’ with ‘const char [12]’, ‘long unsigned int’ arguments
<command-line>: note: in definition of macro ‘GIT_COMMIT_MESSAGE’

le fixado.

indeed. everything is working fine now, thanks


Add Italian localization

I've added the Italian localization.
Question: should we avoid accented letters and use a single quote instead?

Thanks! cc @FiloSottile for review.

Question: should we avoid accented letters and use a single quote instead?

Nah; Rust supports UTF-8 natively, and I think it's fine to assume that if a terminal has Italian set as the language, it also has the character set to render it properly.

Ooh, in testing this I found some more untranslated strings. I'll push a commit to this branch that extracts the en-US strings.

@BuriedInTheGround I've pushed some commits that extract more strings for translation that I'd missed.

Thanks!


[FEAT] Prioritzing the deck for the cards having multiple possible deck.

Is your feature request related to a problem? Please describe.
If we have multiple tags in a note for various indicators like "maturity level", "category" etc, and we decide to review the flashcards in them by converting those tags as deck, the cards are shown in the deck for which the tag is defined first in the file. eg. if my tags line is tag: level/seed category/A, then the cards from this deck would show only in the deck level/seed.
Describe the solution you'd like
Ability to prioritize the decks in case of multiple tags as decks.
As per my suggestion, the priority should be based on how they are listed in the deck field of the settings dor plugin. If for the above case, I have listed like this in the deck field: #category level, then the cards from the note above, should be in the deck category/A. The cards will go into level deck only when the category is missing.

I fixed this, please confirm if it works correctly.


card_collection exception when flinging and scrolling

I was flinging horizontally and scrolling vertically fast at the same time and got:
I/flutter : ══╡ EXCEPTION CAUGHT BY RENDERING LIBRARY ╞═════════════════════════════════════════════════════════
I/flutter : The following assertion was thrown during performLayout():
I/flutter : Build scheduled during frame.
I/flutter : While the widget tree was being built, laid out, and painted, a new frame was scheduled to rebuild
I/flutter : the widget tree. This might be because setState() was called from a layout or paint callback. If a
I/flutter : change is needed to the widget tree, it should be applied as the tree is being built. Scheduling a
I/flutter : change for the subsequent frame instead results in an interface that lags behind by one frame. If
I/flutter : this was done to make your build dependent on a size measured at layout time, consider using a
I/flutter : LayoutBuilder, CustomSingleChildLayout, or CustomMultiChildLayout. If, on the other hand, the one
I/flutter : frame delay is the desired effect, for example because this is an animation, consider scheduling the
I/flutter : frame in a post-frame callback using SchedulerBinding.addPostFrameCallback or using an
I/flutter : AnimationController to trigger the animation.
I/flutter : When the exception was thrown, this was the stack:
I/flutter : #0      BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding&WidgetsBinding._handleBuildScheduled.<anonymous closure> (package:flutter/src/widgets/binding.dart:199:9)
I/flutter : #2      BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding&WidgetsBinding._handleBuildScheduled (package:flutter/src/widgets/binding.dart:197:12)
I/flutter : #3      BuildOwner.scheduleBuildFor (package:flutter/src/widgets/framework.dart:1467:23)
I/flutter : #4      BuildableElement.markNeedsBuild (package:flutter/src/widgets/framework.dart:2234:11)
I/flutter : #5      State.setState (package:flutter/src/widgets/framework.dart:848:14)
I/flutter : #6      ScrollableState._setScrollOffset (package:flutter/src/widgets/scrollable.dart:385:5)
I/flutter : #7      ScrollableState.scrollTo (package:flutter/src/widgets/scrollable.dart:430:7)
I/flutter : #8      ScrollableState.didUpdateScrollBehavior (package:flutter/src/widgets/scrollable.dart:467:5)
I/flutter : #9      LazyBlock._buildViewport.<anonymous closure> (package:flutter/src/widgets/lazy_block.dart:269:15)
I/flutter : #10     _LazyBlockElement._layout (package:flutter/src/widgets/lazy_block.dart:778:25)
I/flutter : #11     RenderObject.invokeLayoutCallback.<anonymous closure> (package:flutter/src/rendering/object.dart:1644:58)
I/flutter : #12     PipelineOwner._enableMutationsToDirtySubtrees (package:flutter/src/rendering/object.dart:896:15)
I/flutter : #13     RenderObject.invokeLayoutCallback (package:flutter/src/rendering/object.dart:1644:13)
I/flutter : #14     _RenderLazyBlock.performLayout (package:flutter/src/widgets/lazy_block.dart:466:7)
I/flutter : #15     RenderObject._layoutWithoutResize (package:flutter/src/rendering/object.dart:1424:21)
I/flutter : #16     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:874:18)
I/flutter : #17     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding.beginFrame (package:flutter/src/rendering/binding.dart:202:19)
I/flutter : #18     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding&WidgetsBinding.beginFrame (package:flutter/src/widgets/binding.dart:297:23)
I/flutter : #19     BindingBase&SchedulerBinding&GestureBinding&ServicesBinding&RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:147:5)
I/flutter : #20     BindingBase&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:608:15)
I/flutter : #21     BindingBase&SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:543:9)
I/flutter : #22     _beginFrame (file:///b/build/slave/Linux_Engine/build/src/flutter/lib/ui/hooks.dart:48)
I/flutter : (elided one frame from class _AssertionError)
I/flutter : The following RenderObject was being processed when the exception was fired:
I/flutter :   _RenderLazyBlock NEEDS-LAYOUT
I/flutter : This RenderObject had the following creator information:
I/flutter :   LazyBlockViewport ← ClampOverscrolls ← IgnorePointer ← Listener ← _GestureSemantics ←
I/flutter :   RawGestureDetector-[GlobalKey 371982228] ← Scrollable ← ClampOverscrolls ← CustomPaint ←
I/flutter :   AnimatedBuilder ← ⋯
I/flutter : This RenderObject had the following descendants (showing up to depth 5):
I/flutter :   RenderRepaintBoundary relayoutBoundary=up1
I/flutter :     RenderStack relayoutBoundary=up2
I/flutter :       RenderPadding
I/flutter :         RenderViewport
I/flutter :           RenderConstrainedBox relayoutBoundary=up1
I/flutter :       RenderSemanticsGestureHandler relayoutBoundary=up3
I/flutter :         RenderPointerListener relayoutBoundary=up4
I/flutter :           RenderFractionalTranslation relayoutBoundary=up5
I/flutter :   RenderRepaintBoundary relayoutBoundary=up1
I/flutter :     RenderStack relayoutBoundary=up2
I/flutter :       RenderPadding
I/flutter :         RenderViewport
I/flutter :           RenderConstrainedBox relayoutBoundary=up1
I/flutter :       RenderSemanticsGestureHandler relayoutBoundary=up3
I/flutter :         RenderPointerListener relayoutBoundary=up4
I/flutter :           RenderFractionalTranslation relayoutBoundary=up5
I/flutter :   RenderRepaintBoundary relayoutBoundary=up1
I/flutter :     RenderStack relayoutBoundary=up2
I/flutter :       RenderPadding
I/flutter :         RenderViewport
I/flutter :           RenderConstrainedBox relayoutBoundary=up1
I/flutter :       RenderSemanticsGestureHandler relayoutBoundary=up3
I/flutter :         RenderPointerListener relayoutBoundary=up4
I/flutter :           RenderFractionalTranslation relayoutBoundary=up5
I/flutter :   RenderRepaintBoundary relayoutBoundary=up1
I/flutter :     RenderStack relayoutBoundary=up2
I/flutter :       RenderPadding
I/flutter :         RenderViewport
I/flutter :           RenderConstrainedBox relayoutBoundary=up1
I/flutter : ════════════════════════════════════════════════════════════════════════════════════════════════════

I was trying really hard, but couldn't reproduce this anymore on https://github.com/flutter/flutter/blob/master/dev/manual_tests/lib/card_collection.dart. Closing this for now.


[Snyk] Security upgrade jinja2 from 2.10 to 3.1.5

Snyk has created this PR to fix 2 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

requirements.txt

⚠ Warning
jinja2 3.1.5 has requirement MarkupSafe>=2.0, but you have MarkupSafe 1.1.0.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


Sequencing support for multiple dictionaries

Allow the user to upload a set of multiple dictionaries of different types (command, optionally parameter, optionally channel) that share a version (e.g. v1).
Each version will also have a optional link to an adaptation file, but multiple versions may share an adaption file
When users create a sequence they will select a version and those dictionaries and the linked adaptation will be used for validation.
To demonstrate validation of channel dictionary in the adaptation we should load
<measurement_id>...</measurement_id>```

The names from these elements should be used to validate certain arguments

Closed with #1237.


Expires header not sent for HTML files?

https://caddyserver.com/docs/http.expires says that it should be possible to for example match .html$ 5i30s.
I have the following in the section for one of my sites:
expires {
    match .htm$ 4h
    match /assets/.* 1y
}

And it sets the expires header for anything under /assets/, but a static HTML page like /main.htm does not get the header set :S

Never mind


Cleanup + More tests

Adding some more test coverage
Breaking tail logic out of handleCommand() into it's own method

Sorry, you'll have to fix conflicts - probably because I added the resume stuff.


CookieStorage is not defined

When attempting to use
new CookieStorage()
Chrome throws an exception that CookieStorage is not defined. But looking in the JS file(s) it is...

The usage example is not very clear... you have to use AmazonCognitoIdentity.CookieStorage instead of CookieStorage


Developer News CW29

I can do this, no problem, most of the work is on the Thursday which is fine, but I'm not around on the Friday to check the video has appeared. I've asked @KevinMuessig to do that checking part for me on Friday.

More than an assignment, this was reminder for me to ask if you could fill in for me in CW29.  That's the week that I will be in Australia for the Mastering SAP event.  The time zone difference, alone, would make it exceedingly difficult for me to coordinate.  If you are unavailable, we can also find someone else to coordinate.  I can also help with the check on Friday -- I'm just not sure what time that would be in Australia. 😴


ch4编译Sophus时编译不通过

By not providing "FindSophus.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "Sophus", but
CMake did not find one.
Could not find a package configuration file provided by "Sophus" with any
of the following names:
SophusConfig.cmake
sophus-config.cmake

Add the installation prefix of "Sophus" to CMAKE_PREFIX_PATH or set
"Sophus_DIR" to a directory containing one of the above files.  If "Sophus"
provides a separate development package or SDK, be sure it has been
installed.
-- Configuring incomplete, errors occurred!
See also "/home/slambook/ch4/CMakeFiles/CMakeOutput.log".

How to solve this problem ?

You should first build the Sophus:
cd Sophus
cmake .
make


Move event emission outside of processCallback

Summary
The caller of processCallback should be responsible for emitting the correct events. This will allow us to remove the packet and callbackType args to processCallback as well

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Was done in https://github.com/cosmos/ibc-go/pull/3939/commits/a18c8778ea81969c64298d7df84b782e415c6b2d


[TO-PR] [Twitter] Privacy Report Submission.

name: Twitter
ratings:
locationTracking: 0
dataRetention: 0
userControl: 1
darkPatterns: 1
crossSiteTracking: 0
summary: "/privacy-report-card/details/twitter"
reference: "https://google.com/privacy"
category: "privacy-report-detail"
Twitter's Privacy Policy - Summary

poooo

This issue was automatically created from an API call.


adding link to form to join emod slack comminity

Ideally I'd like to just put that into our "About" section in the repo. :/
I'll adjust.


Dependency Errors While Using Maven shade jar

Okay so I use maven shade jar here is my pom.xml https://pastebin.com/6rASfHss
but I get these errors https://pastebin.com/xpTDApiL
I compile using clean package
have done a google search and it looks like it has something to do with guava

What is the reason you decided to file the issue against google-cloud-java?

I believe it's the way google-cloud-java depends on guava. It makes it unusable with other libraries that depend on a different version of guava. (google libraries depend on it heavily for example). Lot's of APIs being added/removed things that are @Beta being used etc.
Really the only way to fix it is repackaging google-cloud-java libs with a shaded guava yourself then pulling that in instead of the real libraries.
An example of a bad mix of guava would be having a project that used

Google api client, GA for example
BigQuery client
Dataproc (Hadoop, Spark) with BigQuery connector


How to support iOS9 in swift 4.2

Since our application needs to support ios9 and above, can we support ios9 on swift4.2？

Hi @Shannon-Yang thanks for your interest. I think the best way would be cloning this repository and making some changes on the code base.

@abdullahselek thank you！ I have made changes in my project


open read-only databases in read-only mode

allows a database to be opened by multiple processes at the same time in read-only mode.
all attempts to mutate the database result in ErrDatabaseReadOnly.

Read-only use should take a read lock, to prevent concurrent writers.

it could, but the concurrent writes are impossible because the file is read-only (one would not be able to open it for writing without doing chmod).

@sasha-s Concurrent as in a separate process.

@tv42 all writes are impossible if the file is read-only (concurrent or not), thus read lock is not needed.

@sasha-s: But without the lock, nothing guarantees the file content stays stable.

@tv42 how can the file content get changed?
some other process has to open it in write mode for that to happen. and it is impossible because the file is readonly.
BTW if by read lock you mean flock (could be shared flock in this case) - please note that flock is an advisory lock. From flock man page:
flock() places advisory locks only; given suitable permissions on a file, a process is free to ignore the use of flock() and perform I/O on the file.

relying on a file being read-only seems cleaner and more robust.

@sasha-s ReadOnly seems like a decision that should be made by the application explicitly and not by the file mode implicitly. For example, if it's determined by the file mode then an application wouldn't error out until it's first Update() which could take a while depending on the application. If it's an explicit decision by the application then it should error out on open.
I agree with @tv42 about flock(). I can see the situation where a database is infrequently updated but read in by multiple processes. Each process would have to bolt.Open() every time it reads data but a read lock would allow multiple processes to do that concurrently. @kelseyhightower wrote a bolt-backed app like that recently. There's no way to do that safely by turning permissions on and off.

Please have another look @benbjohnson

@sasha-s File access mode just isn't a good mechanism for knowing when the file is guaranteed to not change.
[0 tv@brute ~]$ mkdir demo
[0 tv@brute ~]$ cd demo/
[0 tv@brute ~/demo]$ mkdir mnt
[0 tv@brute ~/demo]$ bolt-mount db mnt &
[1] 31933
[0 tv@brute ~/demo]$ mkdir mnt/bukkit
[0 tv@brute ~/demo]$ echo value >mnt/bukkit/key
[0 tv@brute ~/demo]$ ls -l
total 24
-rw------- 1 tv   tv   1048576 May 18 11:29 db
drwxr-xr-x 1 root root       0 May 18 11:29 mnt/
[0 tv@brute ~/demo]$ chmod a-w db
[0 tv@brute ~/demo]$ ls -l
total 24
-r-------- 1 tv   tv   1048576 May 18 11:29 db
drwxr-xr-x 1 root root       0 May 18 11:29 mnt/
[0 tv@brute ~/demo]$ echo more >mnt/bukkit/another
[0 tv@brute ~/demo]$ head mnt/bukkit/*
==> mnt/bukkit/another <==
more

==> mnt/bukkit/key <==
value
[0 tv@brute ~/demo]$ fusermount -u  mnt

Ooh I see the stat aspect is gone. That's better. (This does not constitute a code review ;)

@sasha-s Overall the PR looks good except for one ops comment I made. Can you remove the ops function and make ignoring Truncate() explicit and then I'll do a full review and merge.

Thanks @benbjohnson.
Please have yet another look.

Merged! Thanks @sasha-s. I added a section to the README as well: https://github.com/boltdb/bolt/commit/957d850b5158a4eebf915476058e720f43459584


Cannot place Overlay over anchor element if it has z-index and position set

Description
When trying to use Overlay with elements have z-index set, I cannot place the overlay above the trigger elements.
Steps to reproduce

Checkout this demo https://codesandbox.io/s/primer-react-overlay-position-issue-28b3tx

Version
v35.11.0
Browser
Chrome

Through my quick investigation, I think it is the hardcoded z-index in the Portal component causing the issue.
https://github.com/primer/react/blob/5dd4bb1f7f92647197160298fc1f521b23b4823b/src/Portal/Portal.tsx#L74

👋🏻 Hi, thanks for reporting and including a reproduction! This could potentially get fixed by some work we're doing to investigate using the Popover API instead of Portals (#2124). We're going to move this to our backlog for now.