Opened tooltips not visible until "reset zoom" on the tray icon.

When I use Ctrl + D to search items, sometimes the tooltip does not appear. Once in this state, it will not start working again unless I relaunch the application, or hit "reset zoom" on the tray icon. It breaks again pretty quickly. When I use the "reset zoom" trick I often have several tooltips show up that I have to click to close. My suspicion since I don't see anyone else complaining is that my non-standard monitor size is contributing.
To Reproduce
One way to guarantee it breaks for me is to open the browser by clicking on something on the tooltip, however it also frequently breaks if I have clicked anything in the tooltip.
Expected behavior
I expect the tooltips to not stop appearing.

OS: Windows 10 x64
PoE:  Steam 3.10.0/e English
Monitor: 3440x1440 Ultrawide (dual monitors)

same thing is happening to me. thought i was crazy and it kept crashing. have you found a workaround for this?

just downgraded to old version. works fine

I do not know if it is related, but I am having problems with the overlay staying on top of the PoE window after alt-tabbing with the newest version. Before everything was fine.

0.6.13 (2020-03-25)

fix tooltips not visible by disabling keyboard support (#464)

I'm on the latest, with keyboard support disabled, and I'm still getting the alt-tab refocus issue.


v0.6rc2 improvements/clarifications

Hi Vince and Nathan,
I had a go with v0.6rc2 and since I encountered several hiccups (now in retrospect I think mostly caused by old docs), I just to be sure none of the issues was caused by me and I just took some time to go over them.
My comments follow:

Logging:

Could there be an easy way to let the user see some of the logging options in the CLI without cluttering too much the --help message? Right now one has to look at the code to activate more verbose logging.
Similarly, could we add Vince's way of logging in interactive section (which I somehow cannot find anywhere in the pull/issue discussions) to the docs? #131 <- already an issue
the --dbg option doesn't seem to do anything for me, I still get the same amount of detail. When using the interactive session however (as described in #129), I do get more detail.
[really minor thing: can the argparse arguments that receive values (e.g. --logfile ) have a dest= keyword making explicit the destination variable? This makes it easy to track where certain variables come from/are used again.]

Sample subclasses (this took me quite a while to understand - some of the comments actually don't apply, but since I wrote them as I explored, they reflect my steps in trying to understanding how this works and can be useful to make a few things more explicit/documented):

How is this implemented exactly now? This section of the docs probably needs to be updated. I couldn't figure out how to let looper know about my classes.
Importantly, this should be independent of looper and just handled by models. Not entirely sure if it is like that right now the flow now is build_submission_bundles -> fetch_sample_subtype -> _import_sample_subtype but build_submission_bundles is only called by looper and not somewhere at Project initialization as I'd have thought. In fact there is no logic in Project to accommodate sample subclasses, which now only uses _make_basic_samples.
I got some confirmation of the above when I saw the following message in my looper log: "'/home/arendeiro/workspace/pipelines/pipelines/rnaBitSeq.py' appears to attempt to run on import; does it lack a conditional on 'main'? Using base type: Sample" <- so it is trying to use subclasses! (This pipeline in particular does not use classes and does not extend Sample though). When I inspected my Yaml files of another data type/pipeline, my additional attributes were there! Maybe the logger should be more refined here is saying something like "Using subclass X for sample A" or maybe just my logging from CLI doesn't work.
But despite working, this means that Sample subclasses are only triggered by looper and are not present in the samples of the Project. This would be really useful for the future use of models outside of looper! (and my current use too) - can we please have this implemented in this way?

Sample-Project interaction/link:

Can we reverse the removal of Sample-Project linking? It is true these don't need to exist, and probably shouldn't be used in the looper workflow but are extremely useful outside of it.

TODO:

[x] Logging CLI help (#145)
[x] Explicit CLI option names (#146)
[x] Document Sample subclasses (#147)
[x] Logging options docs (#148)
[x] relink samples/projects (for now -- to be revisited later) (#150)

Ok, lots of stuff here.

Logging:
Could there be an easy way to let the user see some of the logging options in the CLI without cluttering too much the --help message? Right now one has to look at the code to activate more verbose logging.

I propose: a detailed documentation page in readthedocs, with a note in the CLI help message.

the --dbg option doesn't seem to do anything for me, I still get the same amount of detail. When using the interactive session however (as described in #129), I do get more detail.

make sure you put --dbg right after looper (so, looper --dbg NOT looper run config.yaml --dbg) because this parameter is parsed before the command. It's a python thing. Any suggestions for documenting this better?

[really minor thing: can the argparse arguments that receive values (e.g. --logfile ) have a dest= keyword making explicit the destination variable? This makes it easy to track where certain variables come from/are used again.]

I agree with you on this one, but @vreuter does not :stuck_out_tongue:

How is this implemented exactly now? This section of the docs probably needs to be updated. I couldn't figure out how to let looper know about my classes.

Yeah. sorry about that. we didn't write any docs for it yet. Oops. The only place it's discussed is in issue 130. it's quite simple, you just add a line to the pipeline interface -- and this is actually not needed unless you have more than 1 sample subtype. If you specify only a single child class that extends Sample, looper will automatically find it and use it.

Importantly, this should be independent of looper and just handled by models. Not entirely sure if it is like that right now the flow now is build_submission_bundles -> fetch_sample_subtype -> _import_sample_subtype but build_submission_bundles is only called by looper and not somewhere at Project initialization as I'd have thought. In fact there is no logic in Project to accommodate sample subclasses, which now only uses _make_basic_samples.
I got some confirmation of the above when I saw the following message in my looper log: "'/home/arendeiro/workspace/pipelines/pipelines/rnaBitSeq.py' appears to attempt to run on import; does it lack a conditional on 'main'? Using base type: Sample" <- so it is trying to use subclasses! (This pipeline in particular does not use classes and does not extend Sample though). When I inspected my Yaml files of another data type/pipeline, my additional attributes were there! Maybe the logger should be more refined here is saying something like "Using subclass X for sample A" or maybe just my logging from CLI doesn't work.
But despite working, this means that Sample subclasses are only triggered by looper and are not present in the samples of the Project. This would be really useful for the future use of models outside of looper! (and my current use too) - can we please have this implemented in this way?

well -- good points. key thing to remember: a Sample subclass is_not_ inherent to a sample, but is relative to a sample-pipeline combination. Different pipelines could specify different subtypes for a given input sample. So, if you want sample subclasses, you need a pipeline interface. In other words it will not be handled by Project -- but it is handled by the pipeline interface classes, which are also in models. It should be possible to do what you're saying while still maintaining the child subtypes as relative to a pipeline... if only 1 pipeline is specified in a pipeline interface it should default to that child class, but if more are present, it must default to a basic class unless a particular subclass is specified somehow. This may take a bit more thought.

Sample-Project interaction/link:
Can we reverse the removal of Sample-Project linking? It is true these don't need to exist, and probably shouldn't be used in the looper workflow but are extremely useful outside of it.

I propose: make the samples project-aware, but don't use this anywhere in the code (so that the dev can use it outside models).

I'll address the other points separately, but I want to first focus on the samples/project deal. Why is that good/desirable? I see value in a sample knowing about certain attributes of a project that are sample-independent, but it would be bizarre for a sample to know about other samples via a project, especially with inter-sample independence advertised as a bedrock looper principle. I think it'd be good to facilitate access by each sample to a subset of project metadata, but I strongly oppose the existence of a direct reference of a sample to its project. Imports, function calls, object references -- I think circular is bad.

Sample/Project link: I see both sides. I don't have strong feelings at this point. I think if you have a sample, you probably got it through a project, so why do you need the pointer? Just remember where you pulled the sample. But @afrendeiro if you can give a specific use case for when you'd have a Sample, need it's Project, and not have accessed the sample through that Project in the first place (necessitating a link to Project from Sample), that would be interesting.

LOGGING
TLDR -- I like the idea of reserving detail for docs but a note in the CLI about the logging options, and I can hook up the debug/more detailed stuff to the interactive session/module. @nsheff apparently knows me too well; I think the ease of renaming offered by using the same option name on the user and internal sides is nice, and so then I just omit dest, but if you both want it, then I'm fine with adding it.
More thorough -- What would be the mechanism for the note for the logging? An addendum to the text passed as the argument to the epilog parameter in the argument parser? I like that idea, and then there we could also note the positional requirement for the --dbg. Otherwise, I think we'd have to not argparse.SUPPRESS the --dbg help text and explain it there. I agree with @afrendeiro that the specific placement of the --dbg is mildly annoying--and it's tripped me up a couple of times before I remember this. I think that it'd be nice to eventually see if there may be a way to remove that requirement through a little sparring with argparse, but for now it sounds good to just make note of it in the CLI.

What would be the mechanism for the note for the logging? An addendum to the text passed as the argument to the epilog parameter in the argument parser?

Sure, something at the end -- I think something simple like: For advanced options like increasing debug output verbosity and changing logging message formats, please see the documentation -- or better yet, could there be a --adv-help command that describes the advanced help stuff? I like that!
Then we add a new page for usage that would have an 'advanced usage' section. And this should be automatic. As outlined in #73

this would be really great; then we only maintain the docs right in place and then auto-generate the readthe docs; it's suppressed by default but accessible on the CLI if someone wants. keeps general use help uncluttered. I think this solves all issues. name ideas? --adv-help ? Meh...

SAMPLE SUBCLASSES ...

Great explanation. This post is the beginning of the documentation for this...

make sure you put --dbg right after looper (so, looper --dbg NOT looper run config.yaml --dbg)
+
I agree with @afrendeiro that the specific placement of the --dbg is mildly annoying--and it's tripped me up a couple of times before

Thanks for pointing it out. This is a bit annoying, but completely fine if you do give the user the chance to know that there is a --dbg option and its placement by seeing that it only appears when you do looper --help and doesn't when you do looper run --help.

But @afrendeiro if you can give a specific use case for when you'd have a Sample, need its Project, and not have accessed the sample through that Project in the first place (necessitating a link to Project from Sample), that would be interesting.

Well I generally don't ask for features if I'm not using them:
My case is in when running a pipeline. Imagine your pipeline could make use of a few project-centric variables like a specific directory where you want to group a certain type of output for all samples (e.g. bigwig files, some pdfs, etc - this is not the norm for all of the sample's output but just one or two). This variable is common to all samples of the project and could simply be defined by extending the project's config with it. Instead of always having this passed as a parameter, one could simply read the sample's config (which is the only thing I require as input) and get that variable though the project's.
I do agree though that there must be a way to contain circularity and recursiveness specially in the serialization of the objects to yaml (I believe something was/is in place to assure this), so yes it is not straightforward to do this out of the box - it would be good if possible though.

SAMPLE SUBCLASSES

Also agree with everything.
By the way, this also accomodates further class inheritance too:
ChIPSeqSample(Sample)
ChIPmentationSample(ChIPSeqSample)

Thanks @afrendeiro  for pointing out the grandchild-and-beyond subclassing ability available; I'll add this to the docs. The way that I've written the subclasses section of the docs in https://github.com/epigen/looper/pull/149 doesn't suggest that it isn't possible, but it also doesn't drawn attention to this capability, and I think that'd be good to do.

Intuitively, it seems like the portion of a Project that's most useful and logical to share among / distribute to each Sample is the metadata section, and it sounds like the directory scenario that you're describing would fit into this framework; is that right? If so, would it be acceptable to allow each Sample to access the metadata section components rather than the entire Project object. I think that could fit nicely with the mental model of what's going on with the objects and avoid circularity concerns, so if it will meet usage needs, that's what I'd like to implement.

would it be acceptable to allow each Sample to access the metadata section components rather than the entire Project object.

@vreuter  I would suggest either an all-or-nothing here. let's not add the complexity of figuring out what it can or can't do... this is a slippery slope and I think it's not worth our time to go down that route. either it's useful for a sample to know its project, or it's not. let's keep it simple...

Instead of always having this passed as a parameter, one could simply read the sample's config (which is the only thing I require as input) and get that variable though the project's.

@afrendeiro could you instead add this variable to the sample object itself at object creation?

Agreed, I think we're suggesting almost the same thing. That said, I do think there's a major anti-complexity win in that small difference as you alluded to. Attaching the raw paths that would be in metadata directly to the Sample instead--as simple string-valued attributes and accession them directly rather than through metadata--would achieve this, so I'm all for it!

I think for now, I don't want to delay this release any further and I think we still should think further about this. So I think we should add the link back in (just add a pointer at Sample.project) so @afrendeiro can continue to use it in his use case. Let's think about this and continue to discuss and then revisit this in a few months at version 0.7. I'd like to finish off 0.6 which I feel is already overdue.

Ok let's continue the sample/project discussion in #150 and @vreuter can you just set a simple link for now to close this master-ticket?

For Sample-to-Project link, I've implemented a workaround that includes the metadata section. Inclusion of the entire Project reignited the circularity/recursion issues during serialization that I experienced amid the removal of the link a couple weeks ago and that we anticipated in this discussion. This felt acceptable given the use case discussed and the previously mentioned preference for simplicity and expediency here. An attempt to include the entire Project seemed substantially more complex than the approach now in dev, and mostly unneeded, so I went the this (metadata) route.

For Sample-to-Project link, I've implemented a workaround that includes the metadata section. Inclusion of the entire Project reignited the circularity/recursion issues during serialization that I experienced amid the removal of the link a couple weeks ago and that we anticipated in this discussion. This felt acceptable given the use case discussed and the previously mentioned preference for simplicity and expediency here. An attempt to include the entire Project seemed substantially more complex than the approach now in dev, and mostly unneeded, so I went the this (metadata) route.

Ok sounds good.

I also left in a commented-out section with an explanation of a slight implementation change that could be made depending on preferred interaction on the usage side. As it is now, linked attributes are accessed via something like  sample.prj.output_dir, but it's certainly possible to make them available directly, like sample.output_dir. I went this route since that sounded more like what may be in place in existing pipeline and is the current desire. I'm happy to change it around if the other way is preferred. Maybe @afrendeiro has a thought/preference.


Error saving the model in Section 8

Section 8, video 189.
I'm getting this error when saving the model:
model.save("drive/MyDrive/101_food_classes_10_percent_saved_big_dog_model")

WARNING:absl:Found untraced functions such as _jit_compiled_convolution_op, _jit_compiled_convolution_op, _jit_compiled_convolution_op, _jit_compiled_convolution_op, _jit_compiled_convolution_op while saving (showing 5 of 81). These functions will not be directly callable after loading.
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
[<ipython-input-17-7b06e6431aba>](https://localhost:8080/#) in <cell line: 1>()
----> 1 model.save("drive/MyDrive/101_food_classes_10_percent_saved_big_dog_model")

2 frames
[/usr/lib/python3.10/json/encoder.py](https://localhost:8080/#) in iterencode(self, o, _one_shot)
    255                 self.key_separator, self.item_separator, self.sort_keys,
    256                 self.skipkeys, _one_shot)
--> 257         return _iterencode(o, 0)
    258
    259 def _make_iterencode(markers, _default, _encoder, _indent, _floatstr,

TypeError: Unable to serialize [2.0896919 2.1128857 2.1081853] to JSON. Unrecognized type <class 'tensorflow.python.framework.ops.EagerTensor'>.

I found out that tf downgrade to 2.9 should do the job, but I'm unable to install 2.9 on colab.

I think this is because your model includes custom layers or functions that are not tracked by Keras.
try using saving h5 format:
model.save("saved_model", save_format='tf')

or you can save weight and biases and then import them
model.save_weights("model_weights.h5")

model.load_weights("model_weights.h5")

@ostadnavid I already opened another issue with saving weights and biases :)

Hi Ivan, can you link to the notebook you're working with here?
I'm currently working through trying to find a fix.
Seems to be older versions of TensorFlow working (e.g. 2.9.0) but later versions are causing errors with tf.keras.applications.efficientnet models.

Hi @ivanthecrazy , this may help , The problem is with the models rescaling layer.
At the top:
IMAGENET_STDDEV_RGB = [0.229, 0.224, 0.225]
IMAGENET_STDDEV_RGB = [1/math.sqrt(i) for i in IMAGENET_STDDEV_RGB]
Then on build just do:
x = layers.Rescaling(IMAGENET_STDDEV_RGB)(x)

!pip install tensorflow==2.9
pip install --force-reinstall -v protobuf==3.20.3
from tensorflow.keras import mixed_precision
mixed_precision.set_global_policy(policy="float32") # set global policy to mixed precision

Thank you guys, with correct downgrade it seems to work


Confirming support for MSI X470 GAMING PRO

Hey there! I wanted to make this to confirm that the program works well with the MSI X470 Gaming Pro motherboard. Thanks for making this program!

Thanks!


Config flag for not including source code into output html

I'd love to use esdoc to generate public documentation for a project at Work. But unfortunately, since we are not open-sourcing the code, I cannot use esdoc, as long as it includes the source code.
The idea would be to have a config flag includeSource that would default to true. If its set to false, the output html should not contain any source code.
I'd be happy to submit a pull request, If you can give me some pointers as to the best way of implementing this :)

@Weitklick Hi! Thanks for this issue!
I consider way to disabled including source code. Please wait a little.

I published ESDoc v0.2.5. This version support config.includeSource.
You can config:
{
  ...,
  "includeSource": false
}

see https://esdoc.org/config.html for more information.

Wow, you are fast! Much appreciated, thanks a bunch! :+1:

duplicate https://github.com/esdoc/esdoc/issues/67


settings.json location

My mup.json file is located in ${project_root}/.deploy directory and
settings.json in ${project_root}/
I run mup deploy while in ${project_root}/.deploy and Meteor.settings.public is empty on the client side. Where must I put the settings.json file? How to get the settings properly run on the server via mup?

I have found out by trial and error, that settings.json must be in the same dir as mup.json


Add auto user id generation to fauna adapter

Reading through the Fauna doc I just found the solution for the auto user id generation.
No hurry on merging this PR :)

Thank you, like 4th time in the last 24 hours!


Error: Error: docker: invalid reference format.

Hello,
I am attempting to use Azure CLI to upload to a blob storage. I have followed the documentation and am receiving the error message displayed in the title of this bug request. "Error: Error: docker: invalid reference format."
Here is the github action yaml that I am using.
- name: Upload To Blob Storage
   uses: azure/CLI@v1
   with:
     azcliversion: 2.34.1
     inlineScript: |
       az storage blob upload-batch -d $web -s ./dist/portal --acount-name documentuploadstaticsite --overwrite

I have attempted several things to try to resolve this.

I have verified the source directory has files in it.
I have tested the az cli command locally and verified it works properly with no exceptions.
I have tested using different versions of Azure CLI.

I am all out of options, and any assistance would be greatly appreciated.
Thank you.

Hi @Norrch2 This looks like a docker issue. I suspect the issue is due to $web, can you try these steps:-

replace $web with random hardcoded value. If this works then we will be sure that the issue is due to $web.
Then try ${web} or $(web)

Hello @t-dedah
I have tried changing it to a hard coded value and the problem still persists. I have included two screenshots, 1 being the error and the second being the change I made to the yaml.

I have tried using different action versions and cli versions.

is there a way I can run the docker command line on my local machine to be able to troubleshoot that way? I am not certain as to what the CLI command that is being provided by docker is to test.

Hi @Norrch2 I tried to reproduce the error but failed to do in the call cases. I used runs-on: ubuntu-latest.
Can you please share your runs-on field? If its ubuntu-latest too then can you please help me reproducing this error?
I tried this and got an error related to --acount-name not being a valid parameter but not the docker error.
- name: Upload To Blob Storage
   uses: azure/CLI@v1
   with:
     azcliversion: 2.34.1
     inlineScript: |
       az storage blob upload-batch -d $web -s ./dist/portal --acount-name documentuploadstaticsite --overwrite

Hi @t-dedah
Please try the following: copied and pasted. (account was misspelled)
  runs-on: ubuntu-latest
    defaults:
      run:
        working-directory: ./Web

- name: Upload To Blob Storage
      uses: azure/CLI@v1
      with:
        azcliversion: 2.34.1
        inlineScript: |
          az storage blob upload-batch --account-name documentuploadstaticsite -d '$web' -s ./dist/portal

Hello. May I have an update on this?

Hi @Norrch2, I tried multiple combinations and still not able to reproduce the issue.
I tried this and it worked perfectly fine.
- name: Upload To Blob Storage
  uses: azure/CLI@v1
  with:
    azcliversion: 2.34.1
    inlineScript: |
      az storage blob upload-batch --account-key ${{ secrets.ACCOUNT_KEY }} --account-name deepakstorage -d test -s ./src

Is there a way to see the docker command that is executed so perhaps I can execute it locally and get an error I can work with?

Okay so it looks like the problem was due to environment variables on the Environment scope instead of the job scope.
The following environment variables were defined and the action didn't like them:
AZURE_FUNCTIONAPP_NAME: randomappname
AZURE_FUNCTIONAPP_PACKAGE_PATH: randomappname\published
AZURE_FUNCTIONAPP_PUBLISH_PROFILE: ${{ secrets.randomappname_FFFF }}
CONFIGURATION: Release
DOTNET_CORE_VERSION: 6.0.x
WORKING_DIRECTORY: randomappname

@Norrch2 Thank you for the info. As the issue is resolved now, I am closing the issue. Feel free to reopen if you face any more issues.


Vectorize StateMachine::ProcessString

Performance of printing enwik8.txt at the following block sizes:
4KiB (printf): 54MB/s -> 57MB/s
128KiB (cat): 97MB/s -> 106MB/s
The rewrite of StateMachine::ProcessString into an alternating
loop alone gets it to above 100MB/s.
Validation Steps Performed

Works on x64 ✅
TBD: Works on ARM ❓

I'm so friggin excited


[Q] Custom attributes from fact

Hello,
Is it possible to use ansible fact as a custom attribute? I would like to see my attribute (fact_download_speed_in_bytes_per_second) in the node overview.
  - name: Set persistent fact
    set_fact:
      fact_download_speed_in_bytes_per_second: download_speed

With PR #279 you'll be able to add attritbutes from an ansible inventory.
Will that do the trick for you?

I think so, thank you.


Selective components do not open (MudSelect, MudMenu, MudDatePicker, etc.)

Bug type
Component
Component name
MudSelect, MudMenu, MudDatePicker and more
What happened?
In my Blazor application (.NET 8) the selective menu components do not open. For example MudSelect as in the example below. When clicking on the component there is no reaction. It works without any problems in TryMudBlazor.
<MudItem sm="3">
     <MudStack AlignItems="AlignItems.Center">
         <MudSelect T="String" Label="My selection" Variant="Variant.Filled" AnchorOrigin="Origin.BottomCenter" Margin="Margin.Dense">
             <MudSelectItem Value="@("Selection 1")" />
             <MudSelectItem Value="@("Selection 2")" />
             <MudSelectItem Value="@("Selection 3")" />
             <MudSelectItem Value="@("Selection 4")" />
         </MudSelect>
    </MudStack>
</MudItem>
I have noticed this faulty behaviour with the following components so far:

MudSelect
MudMenu
MudDatePicker

I have already added the two lines in the App.razor file:
<head>
    <HeadOutlet @rendermode="RenderMode.InteractiveServer" />
</head>
and
<body>
    <Routes @rendermode="RenderMode.InteractiveServer" />
</body>
Does anyone have a workaround for this problem? The problem should also be fixed, as it has existed for some time.
Expected behavior
I expect behaviour like in the MudBlazor documentary. The MudSelect component should open and display the defined elements. After selection, the value should be updated.
See here:
MudSelect in MudBlazor Doc
I think the behaviour is clear.
Reproduction link
https://try.mudblazor.com/snippet/wEmeOocnSUpqVAwV
Reproduction steps

Create Blazor projekt with .NET 8
Set MudSelect with MudSelectItem components
Try to open Select component

Relevant log output
No response
Version (bug)
6.19.1
Version (working)
No response
What browsers are you seeing the problem on?
Firefox, Chrome, Edge
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

You are most likely missing something in your setup. Since you've stated that you added the render modes globally, the next possible issues is missing the mud providers in your main layout.
<MudThemeProvider/> - you likely already have this
<MudDialogProvider/> - do you have this as well?

@Anu6is Yes, I have declared the two lines  and  in the MainLayout.razor.

Why are you setting UsePopoverProvider to false? You should only need that if you are already registering a PopoverProvider somewhere else in your code usually registered within the MudThemeProvider

@Anu6is This was probably a copy paste error on my part. I have removed this and now it works without any problems. Many thanks for your support.


Worker.Extensions.RabbitMQ: How to output a message with headers?

We are currently in the process of migrating from in-process to isolated. First it wasn't really clear as to how to access BasicProperties in the new way but it seems that they can be accessed through context.BindingContext.BindingData. However we seem to be a bit blocked as to how to output a message with headers? It seems like the current implementation can only return a string as per the documentation on https://learn.microsoft.com/en-us/azure/azure-functions/functions-bindings-rabbitmq-output?tabs=isolated-process&pivots=programming-language-csharp.
Previously we used the binding to access IModel like in the samples in https://github.com/Azure/azure-functions-rabbitmq-extension/blob/dev/extension/WebJobs.Extensions.RabbitMQ.Samples/RabbitMQSamples.cs

@JatinSanghvi Could you help with this question?

It may not be possible to output message with headers in .NET isolated function apps. We too suggest binding to an IModel object to users who need to add headers to output messages.

@JatinSanghvi how would you go about this? Are you talking about a total custom solution here?
Cause the Microsoft.Azure.Functions.Worker.Extensions.RabbitMQ package does not reference any RabbitMQ-library.

Hi @kshyju, @fabiocav, I will need input regarding what solution we can provide to solve this. The support for referencing IModel inside the user function must be added in RabbitMQ extension to cover all scenarios that were not made available otherwise by directly binding to byte[]/string/POCO types. However, IModel has internal states (it's not serializable) and it's not the one that can be assigned the output message directly. Instead, user is supposed to call .BasicPublish() method on it to publish messages (see https://github.com/Azure/azure-functions-rabbitmq-extension/issues/197#issuecomment-1165458553). Personally, I do not think about it fits as an output binding as user is mostly free to do anything with IModel even including sending messages to a different RabbitMQ queue.
Does it make sense to introduce a new type for output binding in RabbitMQ extension that can be used for passing message properties like headers e.g. IRabbitMQMessage<string/byte[]/POCO>. Along with the message body, this class can have another member with type similar to IBasicProperties with all properties that the users of RabbitMQ extension are looking for. We can also provide trigger binding support for this type.

Does it make sense to introduce a new type for output binding in RabbitMQ extension that can be used for passing message-properties like headers e.g., IRabbitMQMessage<string/byte[]/POCO>. Along with the message body, this class can have another member with type similar to IBasicProperties with all properties that the users of RabbitMQ extension are looking for. We can also provide trigger binding support for this type.

In some cases, it makes sense to introduce something like this. It's ultimately up to the extension and the experience you want to enable. This may benefit from a quick design sync.

Any updates on this? We still very much dependent on this in order to move to isolated process.


ask for help again

Sorry to bother you again, as I really need your help urgently.
Recently, I really need to run through and understand this project in a short period of time, but I am a novice in this field. I have used it for a long time but have not yet run through the code. I urgently need your help.
Details are as follows:
1 As a novice, although I have been doing it for a long time, I can't understand the problem of environment matching at all.
My computer is an ordinary notebook with a Windows system with slightly stronger performance. I run your code on pycharm, using the python3.6.9 you were using, and working with tensorflow1.15.0. I have changed the "train. py" inside the directory problem, now encountered "(ModuleNotFoundError: No module named 'tensorflow.contrib')" problem

2 I have installed the corresponding tensorflow version, but it still reported this error. After consulting some information, I think it may be that the application of the "contrib" module in the windows system is somewhat different, and I don't know how to solve it.
3 Recently, I really need to quickly run through this code and try to understand all the content of the code. I also admire your project and want to study it. So I implore you to take the time to pay attention to it recently, and give me some practical guidance for this newcomer novice, at least let me run through this code as soon as possible, thank you.
4 please save me！！！

Please don't close this issue, I hope it can be saved until I finish learning this project completely, and at the same time it can give some encouragement to other newcomers.
begging for your help again

Hello
I am really happy to help you in this project, so you can ask whenever you need!
I really haven't tried it with windows I was working with Linux and currently with WSL, so I highly recommend to you to do so.
But if you are not aware of Linux it may be more complicated and may waste more time trying to configure your environment. In this case you can still work with windows and try to search for your tensorflow issue.
TensorFlow had a major migration lately to Tensorflow 2, however this project was based on TF1 so you should not get problems with your TF1 version.
However if this issue still exists, you may work with TF2 and switch to the branch I have created lately named "main_migrated" however there might be some other issues you may face as I haven't tested it completely but as far as I remember it was working to the end, but sometimes fails while saving.
Unfortunately, migration from TF1 to TF2 is a huge change that needs a lot of effort and time to finish. The problem is that I barely have time lately and in the upcoming period to solve these issues, specially considering the fact that the project was made 2 years ago and there are a lot of concepts and implementation to remember and navigate through.
I promise I will try to specify a period at every weekend to finish the migration, in the meanwhile you can ask me at any time and I will help you once I read the comment and try to provide short solutions if any.
Thanks

Thank you very much for your reply. Next, I will continue to actively explore and discuss with you, trying to solve this problem as soon as possible.


Add basePath option in a non-breaking way

This patch adds support for the basePath option, but unlike the PR #160 this one does it a non-breaking way. :-)

This should be merged. I had to fork this repo that doesn't seem to be maintained anymore.


Idea for logo generation.

🚀 Pull Request
Description

[Done in my spare time] Just wanted to get this idea out of my head. Up to @bjlittle whether this is actually what we're after, or if we want something different.
I doubt this will end up in the examples/ directory, that was just an easy place to put it while showing/discussing.
Problems I couldn't solve

Thicker "GV" text - a bold font would do it but that isn't configurable.
Might need an entirely different way of generating the text mask, not using Text3D.
Better z levels. Low poly land mass means that the sea base-layer must be set much lower (I even had to use radius= instead of zlevel=), and you can see the gap at the outer edges of the G and V.
https://github.com/bjlittle/geovista/blob/974da780594dde9fa5bf3468f24500863b43cf29/src/geovista/examples/generate_logo.py#L126-L127

Ideas for configuring

Choose the 'official' rotation frame for the static logo.
Colours. I just chose the ones from the Iris logo.
#aec928, #156475
Lighting of the various layers.
Tilting the globe, as is done in the Iris logo.
Level of triangulation/subdivision of the land mass.
https://github.com/bjlittle/geovista/blob/974da780594dde9fa5bf3468f24500863b43cf29/src/geovista/examples/generate_logo.py#L75-L79
Size of the "GV" mask.
https://github.com/bjlittle/geovista/blob/974da780594dde9fa5bf3468f24500863b43cf29/src/geovista/examples/generate_logo.py#L36-L37
A border for the "GV" mask? I tried one using PyVista edge detection but it didn't render smoothly.

@trexfeathers Looks like you've been having fun! Very cool! :smile:
Regardless of what we decide about the geovista logo, it would be kinda awesome to bank this example as a pattern to share with the community.
I was thinking about having an eye-candy section in the forthcoming documentation where the community can contribute and share cool examples that demonstrate various geovista/pyvista/vtk techniques.
Also, I've got some further thoughts on the geovista logo that I'd love to chat to you about offline.

Wanted to finish my ideas for posterity's sake. @bjlittle I remember you had some of your own ideas so I'm posting without the expectation that we run with it.

Code block
"""
Script to create a logo - rotating triangulated globe beneath "GV" text mask.
"""

from tempfile import TemporaryDirectory
from warnings import filterwarnings

import cartopy.io.shapereader as shp
import numpy as np
import pyvista as pv

import geovista as gv

TEXT_SCALE_FACTOR = 0.5
"""Size of the "GV" text relative to the full globe."""

CENTRAL_LONGITUDE = -30
"""Sets where the land mesh is centred in the logo."""

SEA_COLOUR = "green"
"""The colour that will be seen beneath the text mask."""

TEXT_COLOUR = "#005900"
"""The colour of the 'GeoVista' name at the bottom of the 'title' logos."""

LAND_MESH_COLOUR = "white"
"""The land is shown as a wireframe, in this colour."""

globe_bounds = np.array([-180, 180, -90, 90])

def get_dim_size(bounds: tuple, dim: int) -> float:
    """
    Get a PolyData's max size in the given dimension, by parsing the bounds.
    """
    idx_lo = dim * 2
    bound_lo = bounds[idx_lo]
    bound_hi = bounds[idx_lo + 1]
    return bound_hi - bound_lo

def get_centered(polydata: pv.PolyData) -> pv.PointSet:
    """Make a version of the `polydata` that is centered on the canvas."""
    center: tuple = polydata.center
    return polydata.translate(np.array(center) * -1)

def land_polydata() -> pv.PolyData:
    """
    Create PolyData of the globe's landmasses, based on the Nat Earth shapefile.
    """
    land_shapefile = shp.natural_earth(
        resolution="110m", category="physical", name="land"
    )
    reader = shp.Reader(land_shapefile)

    polydata_list = []
    for geometry in reader.geometries():
        # Less complex coastlines.
        geometry = geometry.simplify(1, True)
        coords = geometry.exterior.coords
        n_points = len(coords)
        z_coords = np.zeros(n_points)
        coords_xyz = [*coords.xy, z_coords]

        # Convert geometry to PolyData.
        coords_array = np.array(coords_xyz).T
        face_nodes = [n_points + 1] + list(range(n_points)) + [0]
        polydata = pv.PolyData(coords_array, faces=face_nodes, n_faces=1)

        # Give land some internal structure.
        # Triangulation allows further PyVista operations.
        polydata = polydata.triangulate()
        # Subdivide gives more aesthetically pleasing triangles.
        polydata = polydata.subdivide(2)

        polydata_list.append(polydata)

    polydata_full = polydata_list[0]
    for polydata_sub in polydata_list[1:]:
        polydata_full = polydata_full.merge(polydata_sub)

    return polydata_full

def globe_texture() -> pv.plotting.texture.Texture:
    """
    Create a 'texture' image of the globe that can be mapped to a sphere.
    """
    land = land_polydata()
    land_z = land.points[:, 2]
    assert land_z.min() == land_z.max() == 0
    # Add a sea below the land.
    sea = pv.Box((list(globe_bounds) + [-1e-3, -1e-3]))
    texture_plotter = pv.Plotter(
        off_screen=True,
        window_size=[
            get_dim_size(globe_bounds, 0) * 10,
            get_dim_size(globe_bounds, 0) * 10,
        ]
    )
    texture_plotter.camera_position = "xy"
    texture_plotter.set_background(SEA_COLOUR)
    texture_plotter.add_mesh(
        land, color=SEA_COLOUR, show_edges=True, edge_color=LAND_MESH_COLOUR,
        line_width=5
    )
    texture_plotter.add_mesh(
        sea, color=SEA_COLOUR
    )
    texture_plotter.zoom_camera("tight")

    with TemporaryDirectory() as temp_dir:
        tmp_path = f"{temp_dir}/texture.png"
        texture_plotter.show(screenshot=tmp_path)
        texture = pv.read_texture(tmp_path)

    return texture

def make_line_polydata(
    sequence_x: list[float],
    sequence_y: list[float],
) -> pv.PolyData:
    """
    Create a 3-dimensional line from a series of X and Y coordinates.
    """
    assert len(sequence_x) == len(sequence_y)
    n_lines = len(sequence_x)
    sequence_z = [0] * n_lines

    points = np.stack([sequence_x, sequence_y, sequence_z], dtype=float).T
    conn = np.hstack([[n_lines], np.arange(n_lines)])

    return pv.PolyData(points, lines=conn).tube(radius=0.4)

def globe_text_mask() -> pv.PolyData:
    """
    Create a sphere with the 'GV' text cut out.
    """
    line_g = make_line_polydata([1, 0, 1, 2, 1], [2, 1, 0, 1, 1])
    line_v = make_line_polydata([2.5, 3.5, 4.5], [2, 0, 2])
    text = line_g + line_v
    text = text.triangulate()

    # Create a PyVista box that will have the text cut out of it.
    # Scale the box appropriately based on the existing text.
    #  (Done this way because PyVista had problems upscaling the text).
    text_xy_sizes = np.array([get_dim_size(text.bounds, i) for i in (0, 1)])
    globe_xy_sizes = np.array(
        [get_dim_size(tuple(globe_bounds), i) for i in (0, 1)]
    )
    scaling = np.max(globe_xy_sizes / text_xy_sizes) * TEXT_SCALE_FACTOR
    box_bounds = list(globe_bounds / scaling)
    # Give the box a z-thickness less than text - to facilitate boolean
    #  differencing.
    box_bounds += list(np.array(text.bounds)[-2:] / 2)
    box = pv.Box(box_bounds, level=100).triangulate()

    # Cut the text out of the box, then scale back up to lon-lat size.
    text = get_centered(text)
    box = get_centered(box)
    diff = box - text
    diff = diff.scale([scaling, scaling, 1])

    # Use the cut-out as a text mask for a globe.
    text_mask = diff.project_points_to_plane()
    text_mask.points = gv.common.to_cartesian(
        text_mask.points[:, 0], text_mask.points[:, 1], zlevel=15
    )

    return text_mask

def main():
    logo_plotter = gv.GeoPlotter(off_screen=True)
    pv.global_theme.transparent_background = True
    filterwarnings(
        action="ignore",
        message=".*no coordinate reference.*",
        category=UserWarning,
    )
    # Add text mask as the highest zlevel.
    logo_plotter.add_mesh(
        globe_text_mask(), color="white", zlevel=2
    )
    # Add the globe texture at the zlevel below.
    logo_plotter.add_base_layer(texture=globe_texture(), zlevel=1)
    base_mesh = logo_plotter.mesh
    base_mesh.points = base_mesh.rotate_z(-CENTRAL_LONGITUDE).points

    logo_plotter.camera.tight()
    logo_plotter.camera_position = "yz"

    def write_rotating_gif(file_stem: str):
        logo_plotter.open_gif(f"{file_stem}.gif")
        for _rot in range(60):
            base_mesh.points = base_mesh.rotate_z(12).points
            logo_plotter.write_frame()

    # TODO: poor SVG support limits ability to produce a modern, professional
    #  logo - no transparency, and the globe is actually a poorly rendered
    #  raster, only the text is truly SVG. See pyvista/pyvista#331 .

    # logo_plotter.save_graphic("geovista-logo.svg")
    logo_plotter.screenshot("geovista-logo.png", transparent_background=True)
    write_rotating_gif("geovista-logo")

    logo_plotter.add_text(
        text="GeoVista",
        position="lower_edge",
        font_size=52,
        color=TEXT_COLOUR,
        name="logo_text",
    )
    # logo_plotter.save_graphic("geovista-logo-title.svg")
    logo_plotter.screenshot("geovista-logo-title.png", transparent_background=True)
    write_rotating_gif("geovista-logo-title")

if __name__ == "__main__":
    main()

Just FWIW, I think lower-case letters "gv" might actually look nicer in this context.
Not least because I think the overall outline would be "rounder" than it is with caps.


boards: mec1501modular_assy6885: Add support for UART0

Add options to configure UART0 and pins based upon device
tree settings.
Signed-off-by: Scott Worley<EMAIL_ADDRESS>Change requested by<EMAIL_ADDRESS>

Thanks for adding UART0 support


discussion: check-tests-for-flakes-lane should not run on release branches

The lane pull-kubevirt-check-tests-for-flakes is currently defined for branches main and the release branches 1. I'm opening the discussion since I have seen the lane run here 2.
The main goal for this lane is to find flaky tests by running changed tests five times, each time in random order. This shows order dependencies in tests and reveals 88% of flaky tests. It has proven to be very effective in revealing problems in PRs where tests have been introduced or changed.
This lane is one of the heavier lanes resource-wise, since it requires the same resources as other e2e lanes, probably averaging similar to a sig-storage lane with three nodes. Runtime varies, since it depends on the amount of tests changes, averaging around 30 mins I guess 5.
However, advancing development happens only in the main branch, while release branches only receive backports of fixes. Fixing tests should happen in main branch first and foremost, since it will be carried over into all new development and then eventually backported to older branches. If the above lane runs on release branches, it merely checks again what has already been checked in the main branch. Possibly it might reveal a backport missing, but these cases should be rare.
Since the capacity on KubeVirt CI is reached every day as the graph prowjobs in queue shows 4, we might ask ourselves whether having this is worth the cost - admittedly this is only run per backport PR of which we don't have that many.
Proposal is to remove the lane definition for all release branches 1 and remove the annotation that leads to forking the job definition for release branches 3.
@brianmcarey @jean-edouard @xpivarc WDYT?
EDIT: adding the cost of the lane

If it were free we would definitely want to keep it. It's a loss/gain tradeoff, and I feel like you only touched on the loss part. What's the gain? Do we need the capacity to do something else?
If the company that owns the test clusters just wants to reduce costs, then I agree this is a good place to start.

Actually we are reaching max capacity of jobs to run every day: https://grafana.ci.kubevirt.io/d/asdfare79we8r/prow-job-load-1h-resolution?orgId=1&refresh=1d
the topmost graph shows whenever jobs are waiting for execution.

As said in the meeting, since this is only one lane of many it might not make much sense to shave it.

If it were free we would definitely want to keep it. It's a loss/gain tradeoff, and I feel like you only touched on the loss part. What's the gain? Do we need the capacity to do something else? If the company that owns the test clusters just wants to reduce costs, then I agree this is a good place to start.

@jean-edouard I've updated the issue with some more data and context, hope that explains it better.

The question is, do we look actively on the backports? The lane is not required so the chance of spotting the failures is low. I am not sure about the cost but I am pretty sure we can have a quick look how this lane performs for release branches(# of failures in the past) and see the benefits of keeping it.

/close
since we don't seem to be up to this. Reopen if you think otherwise.


fix1366: Uncommented the title string

fixes 1366: Docusaurus appends the title with url. In absence of
title 'undefined' was shown.
Signed-off-by: Jayesh Kumar<EMAIL_ADDRESS>The page has now the attached view. Please let know if this will be fine?

cool! Thank you @k8s-dev !


Allow to run functions with .NET 7 (SK VS Code Extension)

Describe the bug
When attempting to execute a skill within the Visual Studio Code extension for Semantic Kernel, an error occurs, prompting the installation of .NET 6 when .NET 7 is present on the machine.
To Reproduce
Steps to reproduce the behavior:

Install .NET 6 and .NET 7 on the machine.
Run a skill in SK extension for VS Code

Expected behavior
Should run without any error.
Screenshots

Desktop (please complete the following information):

OS: Windows
IDE: VS Code
Latest version from main branch

Additional context
C:\Dev>dotnet --list-sdks
6.0.118 [C:\Program Files\dotnet\sdk]
6.0.411 [C:\Program Files\dotnet\sdk]
7.0.304 [C:\Program Files\dotnet\sdk]
7.0.400-preview.23274.1 [C:\Program Files\dotnet\sdk]
8.0.100-preview.4.23260.5 [C:\Program Files\dotnet\sdk]
C:\Dev>dotnet --version
8.0.100-preview.4.23260.5

@jpiquot  - thank you for bringing up this bug.  The team is looking into it - @matthewbolanos

Hi @jpiquot
I'm trying to reproduce this issue.
I have tested with just .Net 7 installed i.e.
PS C:\> dotnet --list-sdks
7.0.203 [C:\Program Files\dotnet\sdk]
7.0.304 [C:\Program Files\dotnet\sdk]
PS C:\> dotnet --version
7.0.304

And I am not seeing the issue. If I add .Net 6 I get the same behaviour.
In your instructions to reproduce the say Install .NET 6 and .NET 7 on the machine This seems to conflict with the ask in the title of the issue.
Is there any other information you can share to help me to reproduce this issue?

Can't repro this issue. Closing after weeks without further input. @jpiquot Feel free to re-open with more detail if you encounter this problem again.


feat: state channel details hints

Description
Resolves #232
Implements state channel details hints without the transactions section (as agreed with @michele-franchi)
Demo

Checklist:

[x] I have read and followed the Contributing Guide
[x] My change does not require a change to the documentation.

Thanks @michele-franchi @janmichek for the great suggestions. All of them are included


feat(hocon_schema): add schema engine (with typerefl)

looks good in general.
I'd like to once again bring up the binary vs. atom discussion though.
It's less of a concern if hocon lib parses keys into atoms when it is only to deal with config files.
however, the scope is now extended, we'll also allow REST APIs to POST config changes.
I'd propose to use binary() keys in HOCON parse results.
Only convert to atom() keys after schema check.
to make it easier when hand-crafting the schema,
we perhaps only need atoms for keys and namespaces.
the mapping target can be a list of atoms instead of dot separated strings.
e.g.
fields() -> [foo, a].

fields(foo) ->
    [ {setting, fun setting/1}
    ...;
fields(a) ->
    [{a, fun(type) -> {ref, b}; (_) -> undefined end}];
fields(b) ->
    [{birthdays, fun birthdays/1},
     {someint, fun some_int/1}
    ].

birthdays(mapping) -> [app_foo, a, b, birthdays];
birthdays(type) -> .....

some_int(mapping) -> [app_foo, a, b, someint];
some_int(type) -> ...


MobileNet V2 can't be compiled toward valhall-g77-unknown-android10 in Linalg on tensors

Describe the bug
The model can be compiled with --iree-vulkan-target-triple=qualcomm-adreno640-unknown-android10, but not --iree-vulkan-target-triple=valhall-g77-unknown-android10.
I found this when I tried to add new metrics for Linalg on tensors path to benchmark infra. This is not urgent. I filed this for tracking the issue. Assign to @antiagainst since he has been working on MobileNet V2.
To Reproduce

Download the model from https://storage.googleapis.com/iree-model-artifacts/mobilenet-v2.tar.gz
Run iree-translate --iree-mlir-to-vm-bytecode-module --iree-hal-target-backends=vulkan-spirv -o a.vmfb --iree-vulkan-target-triple=valhall-g77-unknown-android10 -iree-flow-dispatch-linalg-on-tensors -iree-codegen-spirv-experimental-linalg-on-tensors mobilenet-v2/iree_input.mlir

I think you missed -iree-spirv-enable-vectorization in the command line options. Note that we will turn on vectorization by default in the new tensor-based path.

Verified that with vectorization turned on it works.

I was not aware that turning vectorization off would affect the success of compilation, but I guess maybe #5187 fixed it... Anyway, thanks for looking into this.


account controller redux

Proposed changes

Switch to account service
Add new data service and make working with the data layer easier
Add more functions to errors to make working with the dce errors package easier

Next steps include rewriting the accounts get and list controller functions
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Refactor (changes to code, which do not change application behavior)

Checklist

[ ] I have filled out this PR template
[ ] I have read the CONTRIBUTING doc
[ ] I have added automated tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (README.md, inline comments, etc.)
[ ] I have updated the CHANGELOG.md under a ## next release, with a short summary of my changes

Relevant Links

Further comments

Taking a 👀 at this now

latest changes look good. Just ping me when the build is passing, and I'll give it a 👍

Looks like my terraform state file got messed up. Going to do some more manual cleanup to get it green


Saving stage damage and opening for edit threw exception

I saved computed stage damage curves and then tried to open for edit and the following exception was thrown:

I will work on this right now.

Aha. So we need an empty constructor on Histogram and ThreadsafeInlineHistogram.

Sure. You can do it.

hmmmm. we do have empty constructors in Histogram and in ThreadsafeInlineHIstogram

Ok, let me take a look.

Looks like it is histogram data provider

HistogramDataProvider doesn't have an empty ctor. @rnugent3 do you still want to make the change on this one?

I'll let you make the change - I imagine working in that class will be easier for you than for me.

Ran into another one. I don't like all this Activator stuff. I am guessing the histogram class use to live at Statistics.Distributions.Histogram. It doesn't live there anymore.

Looks like it is in Statistics.Histograms.Histogram. I will try that and see what happens.

I was just going to change line 55 but i see that all the distributions seem to use this method. Can we move histogram back to distributions or I can write some custom logic.

After looking through the git history on that class, it looks like it has always been where it is. I'm guessing we just haven't tried this logic before because they stage damages weren't saving before today.

I think i will pass this off to @Brennan1994 I think Brennan understands this logic a little better than I do.
I added the following logic hoping that it would solve the problem but it just kicked the can down the road a bit. I really hate that logic because it will break if the class name or the namespaces ever changes but I couldn't think of anything else to do.

After the above change I get this exception:

The histogram data provider gets its data cleared and then it tries to evaluate the zeroth item. This makes me think that there must be special logic needed in the AddUnlinkedRow(i) call.

When i go to that method it says this:

I am on branch stage_damage_saving.

Ran into another one. I don't like all this Activator stuff. I am guessing the histogram class use to live at Statistics.Distributions.Histogram. It doesn't live there anymore.

statistics.histograms now

@Brennan1994 It looks like your fix is working well. I ran into a null pointer and needed to add a name to the empty ctor in the data provider.

I am now able to open up the stage damages in edit mode. The values in the table with plot are not correct when opening up for edit, but I think that is my problem and I will make that another issue.
This is completed in branch stage_damage_saving.

This work is finished and ready to be tested.

We can return to this issue after #686 is resolved

this issue has been resolved


Let users choose which golden tests are generated in plutarch-test

I'm using plutarch-test in a project where I don't care about tracking the pre eval UPLC, I only care about the benchmark and the resulting UPLC. I think it'd be nice if we could have a config to choose which goldens are actually generated.
The change would be quite minimal I'd assume, something like-
data GoldenConf = GoldenConf
  { trackPreEval :: Bool
  -- ^ Whether to track UPLC golden.
  , trackPostEval :: Bool
  -- ^ Whether to track evaluated UPLC golden.
  , trackBench :: Bool
  -- ^ Whether to track benchmark golden.
  }

-- | Like 'pgoldenSpec' but takes a 'GoldenConf' to determine which goldens to track.
pgoldenSpec' :: HasCallStack => GoldenConf -> PlutarchGoldens -> Spec
pgoldenSpec' (GoldenConf {trackPreEval, trackPostEval, trackBench}) map = do
  base <- currentGoldenKey
  let bs = runListSyntax map
      goldenKeys = [(trackPreEval, UPLCPreEval), (trackPostEval, UPLCPostEval), (trackBench, Bench)]
      goldenTests = [ t | (True, t) <- goldenKeys ]
  -- Golden tests
  describe "golden" $ do
    goldenTestSpec base bs `mapM_` goldenTests
  -- Assertion tests (if any)
  let asserts = flip mapMaybe bs $ \(k, v) -> do
        (k,) . (\f -> f (goldenValueEvaluated v) $ goldenValueBenchVal v) <$> goldenValueExpectation v
  unless (null asserts) $ do
    forM_ asserts $ \(k, v) ->
      it (goldenKeyString $ "<golden>" <> k <> "assert") v

and pgoldenSpec would turn into pgoldenSpec' $ GoldenConf True True True
cc @srid

Sounds good to me. In Plutarch itself we won't be using pgoldenSpec', but user projects can. Is there anything else that should be made configurable in GoldenConf?
This would also require adjusting the unused goldens check accordingly.

Ah yeah. noUnusedGoldens would also need to take a GoldenConf then.
I'm thinking of putting a goldenBasePath :: FilePath in GoldenConf as well - that'd allow users to choose where to store the goldens.


type object 'gtsam.gtsam.Pose3' has no attribute 'identity'

First of all, thank you for publicizing your project.
When I run the command： python. / examples/slam_demo py -- dataset_dir =. / datasets/up/office0 - dataset_name = nerf - buffer = 100 - slam --parallel_run --img_stride=2 --fusion='nerf' --multi_gpu --gui
The following error will occur:
Running with GPUs: 0,1
Running pipeline in parallel mode.
I0105 19:11:41.273782 58239 pipeline_module.py:85] Module: VioSLAM - Spinning.
Traceback (most recent call last):
File "./examples/slam_demo.py", line 200, in
run(args)
File "./examples/slam_demo.py", line 143, in run
slam_module.spin() # visualizer should be the main spin, but pytorch has a memory bug/leak if threaded...
File "/workspace/NeRF-SLAM/./examples/../pipeline/pipeline_module.py", line 88, in spin
self.initialize_module()
File "/workspace/NeRF-SLAM/./examples/../slam/slam_module.py", line 19, in initialize_module
self.slam = VioSLAM(self.name, self.args, self.device)
File "/workspace/NeRF-SLAM/./examples/../slam/vio_slam.py", line 81, in init
world_T_imu_t0, _, _ = initial_state()
File "/workspace/NeRF-SLAM/./examples/../slam/vio_slam.py", line 65, in initial_state
naive_pose = gtsam.Pose3.identity()
AttributeError: type object 'gtsam.gtsam.Pose3' has no attribute 'identity'
I'm running your project on a server without a visual interface, so I guess it doesn't matter. Can you help me with this problem? Thanks!

You can use this fork to recompile NeRF-SLAM. It fixes issues with new version of GTSAM. I did the same and worked for me.

You can use this fork to recompile NeRF-SLAM. It fixes issues with new version of GTSAM. I did the same and worked for me.

Hello, when I recompile the relevant file with your fork. appear this error:
CMakeFiles/Makefile2:32332: recipe for target 'python/CMakeFiles/gtsam_py.dir/all' failed
make[1]: *** [python/CMakeFiles/gtsam_py.dir/all] Error 2
Makefile:165: recipe for target 'all' failed
make: *** [all] Error 2
The same error occurred when I compiled the previous version. pass order: "ti  go it clone https://github.com/ToniRV/gtsam-1.git" solve it.
Now I find that your fork is an improvement on the original version, mainly cuda version and gtsam.
But there are still problems. How can I solve it? If it is resolved in the same way, I think there will still be problems raised by this issue.

I check my version of gtsam and the commit of branch used is f427b0411. So, could you do git checkout f427b0411 for gtsam? I am also testing the code myself so do not know much. Hope this solves your issue.

f427b0411

I check my version of gtsam and the commit of branch used is f427b0411. So, could you do git checkout f427b0411 for gtsam? I am also testing the code myself so do not know much. Hope this solves your issue.

Are you sure your branch number is f427b0411? I get nothing.

And I compare two branches to see what’s changed, don't find  f427b0411.

I met the same problem with you. Has this problem been solved? Thx.

I used your gtsam version and recompiled, but still got the same error.

Try this.

I used your gtsam version and recompiled, but still got the same error.

@543630836 If you are getting type object 'gtsam.gtsam.Pose3' has no attribute 'identity' error, then the problem is with NeRF-SLAM not GTSAM.
Try cloning this and follow the install instruction here.

@543630836 If you are getting type object 'gtsam.gtsam.Pose3' has no attribute 'identity' error, then the problem is with NeRF-SLAM not GTSAM.
Try cloning this and follow the install instruction here.

Thanks for your reply.
However, when I run "cmake --build build_ngp --config RelWithDebInfo -j"
The following error will occur:
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/testbed.dir/build.make:118: testbed] Error 1
make[1]: *** [CMakeFiles/Makefile2:204: CMakeFiles/testbed.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[100%] Linking CXX shared library pyngp.cpython-37m-x86_64-linux-gnu.so
[100%] Built target pyngp
make: *** [Makefile:91: all] Error 2

Thank you for your advice, I have compiled successfully, there are no gtsam related errors. But when I run demo's command: "./examples/slam_demo.py --dataset_dir=./Datasets/Replica/office0 --dataset_name=nerf --buffer=100 --slam --parallel_run --img_stride=2 --fusion='nerf' --multi_gpu --gui", It has the following error:
Traceback (most recent call last):
File "/opt/conda/envs/NeRF_SLAM/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/opt/conda/envs/NeRF_SLAM/lib/python3.8/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/workspace/NeRF_SLAM/./examples/../pipeline/pipeline_module.py", line 88, in spin
self.initialize_module()
File "/workspace/NeRF_SLAM/./examples/../datasets/data_module.py", line 37, in initialize_module
self.dataset = NeRFDataset(self.args, self.device)
File "/workspace/NeRF_SLAM/./examples/../datasets/nerf_dataset.py", line 18, in init
self.parse_metadata()
File "/workspace/NeRF_SLAM/./examples/../datasets/nerf_dataset.py", line 39, in parse_metadata
with open(os.path.join(self.dataset_dir, "transforms.json"), 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: './Datasets/Replica/office0/transforms.json'
I downloaded the related data set from README.md, but I did not find the related transforms.json file. Do you have any suggestions. Thanks a lot.

@543630836 If you are getting type object 'gtsam.gtsam.Pose3' has no attribute 'identity' error, then the problem is with NeRF-SLAM not GTSAM.
Try cloning this and follow the install instruction here.

Thank you for your advice, I have compiled successfully, there are no gtsam related errors. But when I run demo's command: "./examples/slam_demo.py --dataset_dir=./Datasets/Replica/office0 --dataset_name=nerf --buffer=100 --slam --parallel_run --img_stride=2 --fusion='nerf' --multi_gpu --gui", It has the following error:
Traceback (most recent call last):
File "/opt/conda/envs/NeRF_SLAM/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/opt/conda/envs/NeRF_SLAM/lib/python3.8/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/workspace/NeRF_SLAM/./examples/../pipeline/pipeline_module.py", line 88, in spin
self.initialize_module()
File "/workspace/NeRF_SLAM/./examples/../datasets/data_module.py", line 37, in initialize_module
self.dataset = NeRFDataset(self.args, self.device)
File "/workspace/NeRF_SLAM/./examples/../datasets/nerf_dataset.py", line 18, in init
self.parse_metadata()
File "/workspace/NeRF_SLAM/./examples/../datasets/nerf_dataset.py", line 39, in parse_metadata
with open(os.path.join(self.dataset_dir, "transforms.json"), 'r') as f:
FileNotFoundError: [Errno 2] No such file or directory: './Datasets/Replica/office0/transforms.json'
I downloaded the related data set from README.md, but I did not find the related transforms.json file. Do you have any suggestions. Thanks a lot.

I tried all the methods mentioned above, but the same problem

I tried all the methods mentioned above, but the same problem
you can try this :https://github.com/ToniRV/NeRF-SLAM/issues/21#issuecomment-1373536626. It does work for me.

@543630836 Make sure you have extracted the dataset in correct folder. The dataset might be extracted to ./datasets rather than ./Datasets.
Try:
./examples/slam_demo.py --dataset_dir=./datasets/Replica/office0 --dataset_name=nerf --buffer=100 --slam --parallel_run --img_stride=2 --fusion='nerf' --multi_gpu --gui

I changed the file decompressing path and sent the Replica to datasets, but there was still no transforms.json file.
May I ask if this file is generated externally, not as COLMAP or other

@szgy66 I am using x11 forwarding from server for visualization.

Any progress on this issue? (dont wanna another fork, sorry guys).  It was really frustrating that after a long time install process and finally nothing worked. :-(

@ToniRV can you help us?

To be clear, following @bkhanal-11 fork works just fine. I guess your git instruction is :"git clone https://github.com/ToniRV/NeRF-SLAM.git --recurse-submodules". Its wrong. You should use:  "git clone https://github.com/jrpowers/NeRF-SLAM.git --recurse-submodules". The rest of the operation can follow this.
The git address of the first step must be correct. Hope that can help you.

@ToniRV can you help us?

To be clear, following @bkhanal-11 fork works just fine. I guess your git instruction is :"git clone https://github.com/ToniRV/NeRF-SLAM.git --recurse-submodules". Its wrong. You should use: "git clone https://github.com/jrpowers/NeRF-SLAM.git --recurse-submodules". The rest of the operation can follow this.
The git address of the first step must be correct. Hope that can help you.

@ToniRV can you help us?

To be clear, following @bkhanal-11 fork works just fine. I guess your git instruction is :"git clone https://github.com/ToniRV/NeRF-SLAM.git --recurse-submodules". Its wrong. You should use: "git clone https://github.com/jrpowers/NeRF-SLAM.git --recurse-submodules". The rest of the operation can follow this. The git address of the first step must be correct. Hope that can help you.

thanks @szgy66 , just for g++, which version should be changed to? Currently, I am working with <= 11, is that sufficient? Because I got an error while building instant-ngp.

@ToniRV can you help us?

To be clear, following @bkhanal-11 fork works just fine. I guess your git instruction is :"git clone https://github.com/ToniRV/NeRF-SLAM.git --recurse-submodules". Its wrong. You should use: "git clone https://github.com/jrpowers/NeRF-SLAM.git --recurse-submodules". The rest of the operation can follow this. The git address of the first step must be correct. Hope that can help you.

thanks @szgy66 , just for g++, which version should be changed to? Currently, I am working with <= 11, is that sufficient? Because I got an error while building instant-ngp.

you can use pip install gtsam==4.2a9 to solve all problems about gtsam
if there is no gtsam==4.2a9 you can change the pip source to   https://pypi.mirrors.ustc.edu.cn/simple/

这是来自QQ邮箱的假期自动回复邮件。

您好，已经收到您的邮件，无法亲自回复，还请见谅。

@ToniRV can you help us?

To be clear, following @bkhanal-11 fork works just fine. I guess your git instruction is :"git clone https://github.com/ToniRV/NeRF-SLAM.git --recurse-submodules". Its wrong. You should use: "git clone https://github.com/jrpowers/NeRF-SLAM.git --recurse-submodules". The rest of the operation can follow this. The git address of the first step must be correct. Hope that can help you.

thanks @szgy66 , just for g++, which version should be changed to? Currently, I am working with <= 11, is that sufficient? Because I got an error while building instant-ngp.

I meet a problem：ImportError: /home/mei/anaconda3/envs/nerf-slam2/lib/python3.10/site-packages/torch/lib/../../../../libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /home/mei/python/NeRF-SLAM/build_ngp/pyngp.cpython-310-x86_64-linux-gnu.so) how can I solve this?

这是来自QQ邮箱的假期自动回复邮件。

您好，已经收到您的邮件，无法亲自回复，还请见谅。

@543630836 If you are getting type object 'gtsam.gtsam.Pose3' has no attribute 'identity' error, then the problem is with NeRF-SLAM not GTSAM.
Try cloning this and follow the install instruction here.

Not using NeRF-SLAM, but the error is same

这是来自QQ邮箱的假期自动回复邮件。

您好，已经收到您的邮件，无法亲自回复，还请见谅。

Solved  it.
just modifyslam/visual_frontends/visual_frontend.py as below:`# def gtsam_pose_to_torch(pose: gtsam.Pose3, device, dtype):
t = pose.translation()
# q = pose.rotation().quaternion()
q = pose.rotation().matrix()
return torch.tensor([t[0], t[1], t[2], q[1], q[2], q[3], q[0]], device=device, dtype=dtype)
def gtsam_pose_to_torch(pose: gtsam.Pose3, device, dtype):
t = pose.translation()
R_matrix = pose.rotation().matrix()  # 3x3 rotating matrix
r = R.from_matrix(R_matrix)
q = r.as_quat()  # transfer to  [qx, qy, qz, qw]
 remember to import like this:from scipy.spatial.transform import Rotation as R`


redirect to restore complete

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Does this PR require documentation?

[sc-61561]


SQLite Library

It would be nice to have a SQLite library to interface SQLite with LDPL. I've found this library that seems to do just that. Maybe something can be done with it.

I think this is another one where we’d benefit from custom types. You could do RUN SQL QUERY “blah” IN var where var would be a special SQL result type that you can run other statements against. Like the # of rows returned, names of fields as a list of strings, etc. Possibly.

Yes, that'd be lovely :thinking:


Update develop-web-apps-with-aspnet.md

Fixes VSTS bug requesting more SEO on Azure.

Thanks for making the additional changes, @Rick-Anderson. This looks really good. I'll merge now.


Issue with Reloadium in PyCharm+WSL

Describe the bug
When attempting to use the remote debugging feature in PyCharm 2023.3.5 on Windows 11, targeting a Python environment under Anaconda in WSL, an error occurs with Reloadium. The error message displayed is java.lang.NoSuchMethodException: java.lang.ProcessImpl.addRemoteTunnel(int,java.lang.String,int).
To Reproduce
Steps to reproduce the behavior:

Set up remote debugging in PyCharm 2023.3.5 on a Windows 11 system.
Target a Python environment within Anaconda installed on WSL.
Start remote debugging with Reloadium.
Observe the error message: java.lang.NoSuchMethodException: java.lang.ProcessImpl.addRemoteTunnel(int,java.lang.String,int).

Expected behavior
I expected to be able to remotely debug my Python code in the WSL environment using Reloadium without encountering any exceptions.

@ByteMelodist What is your PyCharm version?

2023.3.5

Iam also encountering the same issue.

Still happens with PyCharm 2023.3.5


Using TS with Material UI and Styled Components

Hi everyone. Sorry if this isn't the best place to post this, but I'm pretty new to TS and I'm not sure if it was a MUI or Styled Component issue (I think it's a MUI one)
I'm having trouble with this component where checked is throwing the following error:
`No overload matches this call.
  Overload 1 of 2, '(props: CheckboxProps, context?: any): ReactElement<any, any> | Component<CheckboxProps, any, any> | null', gave the following error.
    Type 'Boolean' is not assignable to type 'boolean | undefined'.
      Type 'Boolean' is not assignable to type 'true'.
  Overload 2 of 2, '(props: PropsWithChildren<CheckboxProps>, context?: any): ReactElement<any, any> | Component<CheckboxProps, any, any> | null', gave the following error.
    Type 'Boolean' is not assignable to type 'boolean | undefined'.
      Type 'Boolean' is not assignable to type 'true'.`

<CheckboxDiv>

Hide all

        <Checkbox
          label="Check this box"
          onChange={() => dispatch(switchAll())}
          checked={!status.hideAll}
          style={{
            color: "#262626",
          }}
          classes={{ root: "custom-checkbox-root" }}
        />
      </CheckboxDiv>

      <CheckboxDiv>

Show completed

        <Checkbox
          onChange={() => dispatch(switchCompleted())}
          checked={status.showCompleted}
          style={{
            color: "#262626",
          }}
          classes={{ root: "custom-checkbox-root" }}
        />
</CheckboxDiv>

and I don't know if it changes something, but I also have defined the size of the checkbox on the index.html file. It looks like this:

can anyone spot the error of why TS is throwing an error?

The Checkbox component doesn't support the label prop, the TypeScript error is expected. Regarding the second, we can't help without a reproduction. Closing for now


Add builds for M1 Macs

Tested locally and works just fine:
$ file sectionctl
sectionctl: Mach-O 64-bit executable arm64
$ ./sectionctl version
1.11.0 (darwin-arm64)
$ ./sectionctl whoami
Looking up current user... done
|   ATTRIBUTE   |         VALUE          |
|---------------|------------------------|
| Name          | Lindsay Holmwood       |
| Email         |<EMAIL_ADDRESS>|
<snip>

should we have builds for both?

Builds for both what?

@auxesis for older macs and for m1?

Yes, this PR doesn't remove any existing builds, it just adds arm64-darwin.


fresh flutter app is 129 mb

creating fresh flutter app on android studio  is 129 MB

That's fine. When releasing you are going to use app bundle which reduces the code size.

Can you post the output of
flutter create test
cd test
flutter build appbundle

It should say sth like
✓ Built build/app/outputs/bundle/release/app-release.aab (16.8MB).

at the end.

Closing as this not an issue and WAI. You may read about app size here for further reference: https://docs.flutter.dev/perf/app-size


alt-b and alt-f differ from readline when_moving_over_underscores

Is there a configurable list of characters where alt-f and alt-b will stop? On GNU readline I need to press alt-f 3 times to_get_over that, but in ppt, it seems to jump the whole lot, only stopping on spaces.
Sorry if this is documented somewhere, I couldn't find it.

Same as https://github.com/jonathanslenders/python-prompt-toolkit/issues/458

Thanks.


[FR] Auto-complete Fragments

✅ Viability Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion
Currently, unlike other tags, fragments can't be auto-completed, would be nice to have them supported.
📃 Motivating Example
Entering <> won't auto-complete </>:

Strange, it seems this completion is not handled by tsserver.
I set a breakpoint on executeCommand(in session.ts), then add '>' for '<foo'. When the breakpoint works, the completion has already been added to the file. The log does not have any related info too, I think.
@RyanCavanaugh would you mind give a kind guide about possible related code? I do not have any idea now...


Question: Considering .eslintignore

Description
It sems that with this configuration:
"lint-staged": {
   "*.js": ["eslint --fix", "git add"]
}

eslint does NOT consider any .eslintignore, thus I get errors for files which are in folders that have an .eslintignore.
I have corrected this by doing this:
"lint-staged": {
   "*.js,.eslintignore": ["eslint --fix", "git add"]
}

Is this the correct way to configure this?
Should this be a default behaviour?
Maybe worth adding this to the example configs?
Thanks.
Steps to reproduce
Debug Logs
Environment

No, it should not affect how eslint works. 🚫💩 Lint-staged only calls eslint with arguments that are staged files so I don’t think this is a valid solution. Are you sure it’s not considering it? If so that must be some isssue with eslint since I’ve been using eslint with ignores like this before.

Closing of inactivity

@okonet So strange, I'm also having the same error as @hepiyellow. I run eslint --fix with lint-staged but it completely ignores my .eslintignore. I also tried to the ignore option by passing in the same files/folders from .eslintignore, but that didn't work.
@hepiyellow's solution works for me! :)

That's weird indeed. I never had such issues with it before. If that's an issue with lint-staged, when we'd need a reproducible case.

@okonet Of course! Hopefully it's just something in my configuration.
Let me give some background:
Description
When running husky with lint-staged, my .eslintignore file is omitted. The files and folders to be ignored instead show up.
I've temporarily removed lint-staged to keep on coding. :)
Steps to reproduce
This is my configuration:
"lint-staged": {
		"linters": {
			"*.js": [
				"eslint --fix", // Also tried npm run lint:fix
				"prettier --write",
				"git add"
			]
		}
	}

I tried this as well:
"lint-staged": {
		"linters": {
			"src/**/*.js": [
				"eslint --fix",
				"prettier --write",
				"git add"
			],
			"ignore": ["**/src/config/*", "**/src/crons/*", "**/src/db-postgres/**/*.js", "**/src/db-postgres/**/*.js", "**/src/middleware/*", "**/src/public/*", "**/src/routes/*", "**/src/services/*", "**/src/tests/**/*.js", "**/src/index.js"]
		}
	}

Folder structure
root
|-- src
|-- package.json
|-- .eslintignore

Debug Logs
Environment
Node v10.13.0
NPM v6.9.0
Thanks for your help! 😄

Please investigate the root cause of this and submit a fix.

I also have this is issue. I have added *d.ts files to ignore and it works well with my eslint but fails with lint-staged

This issue is still an issue, should it be open instead?

Please post debug logs, otherwise there is nothing we can do to debug this issue.

I have the same problem. Even if I explictly pass --ignore-path it won't work. When running eslint with execa it can't see the .eslintignore file. It seems to have to do with that my project path contains a space. But didn't have the time to verify that.

@okonet I'm having a similar issue: I have an .eslintignore file.

If I run eslint on it's own with --max-warnings 0, it respects the .eslintignore and I get no warnings

However, if I run the same command in lint-staged, it does not take into account the .eslintignore and throws the warnings

Here's the command I'm running:
eslint --fix src/ --ext .ts --max-warnings 0

same error +1

same error +1

@1o1w1 - I solved it by adding a .lintstagedrc.js file: https://github.com/okonet/lint-staged#eslint--7

@1o1w1 - I solved it by adding a .lintstagedrc.js file: https://github.com/okonet/lint-staged#eslint--7

For those traveling through the issues in future (like me for last couple hours):
@kunalnagar answer above was very helpful, but I had to adjust it to consider 2 .eslintrc configs I had (root and for cypress):
const { ESLint } = require('eslint')
const filterAsync = require('node-filter-async').default

// All of this below is caused by eslint dropping warnings (File ignored because of a matching ignore pattern.) on ignored files
// solution taken from: https://github.com/okonet/lint-staged#eslint--7

// two eslint initializations because we have two .eslintrc, one in root, another one in cypress/.eslintrc
const rootEslintCli = new ESLint()
const cypressEslintCli = new ESLint({ overrideConfigFile: './cypress/.eslintrc'})

const removeIgnoredFiles = async (files, eslintCli) => {
  const filteredFiles = await filterAsync(files, async (file) => {
    const isIgnored = await eslintCli.isPathIgnored(file)
    return !isIgnored
  })
  return filteredFiles.join(' ')
}

module.exports = {
  'src/**/*.{ts,tsx,js,jsx}': async (files) => {
    const filesToLint = await removeIgnoredFiles(files, rootEslintCli)
    return ['tsc', `eslint --fix --max-warnings=0 ${filesToLint}`]
  },
  'cypress/**/*.{ts,tsx,js,jsx}': async (files) => {
    const filesToLint = await removeIgnoredFiles(files, cypressEslintCli)
    return ['tsc --project cypress', `eslint --config cypress/.eslintrc --fix --max-warnings=0 ${filesToLint}`]
  },
}

Hi everyone,
To address this, I've published a tool called @skarab/eslint-staged.
This CLI tool enhances lint-staged by ensuring it respects the ignored files, avoiding the common issue where ignored files still get processed.
It's straightforward to use, first install:
pnpm add @skarab/eslint-staged --save-dev

And add the following to your lint-staged config:
{
  "*": [
    "pnpm eslint-staged --fix --max-warnings=0",
    "pnpm prettier --write --ignore-unknown"
  ]
}

I believe this could be a helpful solution for those experiencing similar issues. The package is open for contributions, and I'd love to hear your feedback!
You can find it here: https://github.com/skarab42/eslint-staged
Happy coding!

ESLint now has a command line flag --no-warn-ignored that makes it not care about ignored files:

Suppress warnings when the file list includes ignored files. Flat Config Mode Only

https://eslint.org/docs/latest/use/command-line-interface
This also documented in our readme: https://github.com/lint-staged/lint-staged#how-can-i-ignore-files-from-eslintignore


Vulnerability in criterion dependencies

nom-kconfig $ cargo audit
    Fetching advisory database from `https://github.com/RustSec/advisory-db.git`
      Loaded 556 security advisories (from /Users/bnc8/.cargo/advisory-db)
    Updating crates.io index
    Scanning Cargo.lock for vulnerabilities (74 crate dependencies)
Crate:     atty
Version:   0.2.14
Warning:   unsound
Title:     Potential unaligned read
Date:      2021-07-04
ID:        RUSTSEC-2021-0145
URL:       https://rustsec.org/advisories/RUSTSEC-2021-0145
Dependency tree:
atty 0.2.14
└── criterion 0.4.0
    └── nom-kconfig 0.1.0

warning: 1 allowed warning found

Could be fixed by using criterion 0.5.1, which has no vulnerabilities as of today!

Updated!
Don't worry it's a dependency only used for unit tests!


Small wording patch

Thanks to @Donistivanov on aspnetcore Slack for reporting. ✈

I see the Department of Redundancy Department was at work again

Closing and reopening to trigger the build


add instructions to use money market protocol

The money market protocol is in phase 2.

Closing this PR. Will wait for docs from zklend team and link to it.


Create A Loginup Form

Create A Loginup Form Using HTML, CSS And Javascript.

I would like to take this up.

@deepak2431 you can add whatever you want to add in login form. It's all up to you.

I would also like to take it up

I have made a pull req, please check

Can I Contribute?

@ramkrishn123 Yes!
Good luck!

I want to contribute for this.
Sir,Can I??

@ShravanMeena
Sir I have uploaded the folder. Please review PR #95

@ShravanMeena Sir my pull request has been merged. But on  hacktoberfest profile, its showing pending.

@soumyadip007 this is just a guide don't get panic.

created a PR please check

Is this still available?


Color enum: Add support for inherit, transparent and surface color

Description

Adds the possibility to set the color to Surface, Inherit or Transparent
Fixes #2961
How Has This Been Tested?

only visually (You can't check css rules with bUnit)
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)

Checklist:

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.

Allot of components have their own CSS for color, and i think even specific lines for transparent, inherit and so on.
How will this affect them? i guess we should clean it up?


Add Slug field type

TBD

We have Text type which supports pattern and unique features. and we don't need to have a dedicated field type for Slug.
I'll close this issue, if anyone thinks we need it, please open this issue and add a comment.


[348] Add reference-to-unknown-keyword tests to earlier drafts

Also renaming "reference of" to "reference to".
Closes #348

Left a comment on https://github.com/json-schema-org/JSON-Schema-Test-Suite/issues/348#issuecomment-619439510 that's relevant.

Turns out I don't fully understand this yet. I found this: https://github.com/json-schema-org/json-schema-spec/pull/713, the bug it's fixing (https://github.com/json-schema-org/json-schema-spec/issues/687), and other things that refer to it.

Are you sure this is correct, despite your comments and my response? (https://github.com/json-schema-org/JSON-Schema-Test-Suite/issues/348#issuecomment-619439510)

Sounds like #361 will resolve this one, so I think we can move the discussion there.


finish the training doc and code

[ ] upgrade de contrat /storage de function lambda
[ ] tickets : blocked by destructuring ligo v0.43?

done


dependency updates

Title
dependency refreshments
Description
ratatui and uci was a bit outdated, so I updated them
with this version of ratatui, they reexport crossterm, so it was removed as a direct dependency
there are some new pretty convenient functions as well in ratatui reducing boilerplate when initializing and reseting terminal: made use of these
How Has This Been Tested?

[x] cargo test
[x] played a basic game

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes

Perfect checking tomorrow


Vine tj550 thermostat cant see if fan is running

Description
the tuya local is unable to use the fan as an automation. it shows "heating" or "cooling" when it is working, but there is no entity to use from it. just if the whole thermostat is on or off.
Steps to reproduce
set thermostat to higher than current temp
Expected behaviour
entity should show "fan on" but instead it doesnt. (tuya app does let me)
Additional context
No response
Confirmation

[ ] I am sure this is a bug or improvement, that is well enough described that it can be implemented.

[X] If this is for a specific device, the device config is mentioned above, or diagnostics are included.

[ ] Log messages or diagnostics relevant to the issue are included.

[X] This is not requesting addition of a new device.

It seems you need to review the Home Assistant documentation to see how to do whatever it is that you are trying to do within Home Assistant. Don't expect it to work exactly as the Tuya app does, that is not the goal of Home Assistant or this integration.

I just know the device has the ability to show if the fan is running or not, it would have just been nice to be able to turn on other fans when my furnace is running without using additional thermostats or tuya(online). I'll just have to find one of the thermostats you already have supported that does do that
Yahoo Mail: Search, Organize, Conquer
On Sun, Dec 22, 2024 at 8:43 PM, Jason @.***> wrote:
It seems you need to review the Home Assistant documentation to see how to do whatever it is that you are trying to do within Home Assistant. Don't expect it to work exactly as the Tuya app does, that is not the goal of Home Assistant or this integration.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

You said 'it shows "heating" or "cooling" when it is working', if that is the information you need to trigger off, then why not use that?

It's physically shows it on tuya local (like on the display) but it does not give you any sort of entity or any actual state change to run anything off of. It just SHOWS it's working lol. That's why I was hoping it would be something super simple 😅. I can see the state changing, but it's not giving me an actual entity that changes if you follow what I mean
Yahoo Mail: Search, Organize, Conquer
On Sun, Dec 22, 2024 at 8:54 PM, Jason @.***> wrote:
You said 'it shows "heating" or "cooling" when it is working', if that is the information you need to trigger off, then why not use that?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

It shows heating, but on the actual entity or log book, there isn't a state change on anything.

"HVAC mode changed" is just if you set it to heat, or AC, or Off. But not what the furnace is actively doing.

You need to use the Entity triggers, not the Device triggers. Device triggers are just limited to the major ones as decided by Home Assistant, entity triggers give you access to all the attributes, including the hvac_action which you see in the UI.

Yup....I'm a moron 😅 sorry for bothering you.
Yahoo Mail: Search, Organize, Conquer
On Sun, Dec 22, 2024 at 10:04 PM, Jason @.***> wrote:
You need to use the Entity triggers, not the Device triggers. Device triggers are just limited to the major ones as decided by Home Assistant, entity triggers give you access to all the attributes, including the hvac_action which you see in the UI.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


[5.7][Serialization] Harden the compiler tag check on swiftmodules

When two different serialization formats share a version number but are different enough, it can defeat the check to restrict loading swiftmodules built by the same compiler. Add a backup check in case the REVISION block is not seen to detect such issues. This applies to swiftmodules only and not to swiftdoc or sourceinfo files.
rdar://93188070
Cherry-pick of #58880

@swift-ci Please test


Question about tailwindui

Hi Christine,
I'm still learning about Astro and Tailwind and I noticed that you mention TailwindUI in the README.md. I was looking into tailwindui.com and reading about their licenses, but I'm not sure exactly how/if hello-astro uses tailwindui. Do you know if there are any licensing issues related to tailwindui that would affect users of hello-astro?
Another option I'm looking into is building something similar using https://daisyui.com/, since it uses the MIT license. If I wanted to do that, would I need to replace the contents of components/ with similar components from daisyui?
Thanks for any guidance you can provide and for sharing such a beautiful theme with us!

I'm not sure exactly how/if hello-astro uses tailwindui. Do you know if there are any licensing issues related to tailwindui that would affect users of hello-astro?

Hello
hello-astro only uses the "free" components provided as a preview in this link:
https://tailwindui.com/components/preview
I don't think there should be any licensing issues as the source code of these components are provided as a "sampler" for tailwindui. hello-astro does not use any of the "paid" components. In any case, these components were not used "as is" but were modified (for example, AlpineJS is used for interactivity rather than HeadlessUI).

Got it, thanks for clarifying!


[CT-2338] [Bug] {{this}} evaluates incorrectly with Snowflake when used in a macro

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
{{ this }} evaluates to the correct full path of the table in Snowflake when used directly in a model's pre-hook/post-hook.
When it is passed from a model as an argument to a macro, only the table name information is retained and the Snowflake schema and database information is lost.
Expected Behavior
When {{ this }} is passed from a model to the macro, the schema and database info is not lost.
Steps To Reproduce
When using {{ this }} directly in a post_hook, it correctly evaluates to the full path of the table in Snowflake.
{{ config(
    post_hook=[
        "grant select on {{this}} to role DW_GL"
    ]
)}}

In the above example, {{this}} would evaluate to PRESENTATION.B2B.MY_DBT_MODEL_NAME, which is as intended.
Now I want to take this logic and put it into a macro.
I created a macro as below:
{% macro grant_select(table_name, roles) %}
    -- Macro to grant select on table to one or more roles.

    {% for role in roles %}

        grant select on {{ table_name }} to role {{role}}

    {% endfor %}
{% endmacro %}

In the model I call the macro as below:
{{ config(
    post_hook = grant_select(this, ['DW_GL'])
)}}

Now when the macro runs, this is passed to the macro as the table_name variable.
However it now evaluates to RAW.PUBLIC.MY_DBT_MODEL_NAME.
RAW.PUBLIC is my default schema in Snowflake. The actual database and schema info is lost when passing through to the macro.
Relevant log output
08:00:50  Database Error in model vw_bfn_d365dimretailstore (models/presentation/bfn_presentation/vw_bfn_d365dimretailstore.sql)
08:00:50    002003 (42S02): SQL compilation error:
08:00:50    Table 'RAW.PUBLIC.VW_BFN_D365DIMRETAILSTORE' does not exist or not authorized.

Note: Does not exist because table is created at `PRESENTATION96447.BFN_PRESENTATION.vw_bfn_d365dimretailstore`

Environment
- OS:Mac OS 12.6.3
- Python: 3.10.4
- dbt: 1.4.5
- dbt-snowflake: 1.4.5

Which database adapter are you using with dbt?
snowflake
Additional Context
N/A

@rory-donaldson Unfortunately, you need to "nest your curlies" in this case:
{{ config(
    post_hook = "{{ grant_select(this, ['DW_GL']) }}"
)}}

That way, the grant_select macro will be re-rendered at runtime, when the full configuration for {{ this }} is available.
I know this is a bit ugly & a bit odd. It is documented behavior:
https://docs.getdbt.com/docs/building-a-dbt-project/dont-nest-your-curlies#an-exception
(though we could do a better job of documenting it: https://github.com/dbt-labs/docs.getdbt.com/issues/2818)

Also - did you know that, since v1.2, dbt has had built-in support for grants?
https://docs.getdbt.com/reference/resource-configs/grants
{{ config(grants = {'select': ['DW_GL']}) }}

Thanks, nesting the curlies fixed it! And yeah an update to the docs to make this more clear would be awesome


request and request-promise-native vulnerability

These two packages are no longer maintained, and Snyk has reported an issue with the request package related to Server-side Request Forgery.
jsforce went through a similar analysis and remediation here: https://github.com/jsforce/jsforce/issues/1312
I wish I could help, but this is out of my wheelhouse. I do think this is an important fix to push through.
https://github.com/dxatscale/sfpowerscripts/blob/185b6fbcbd2d847619b021eaf48544506191cba2/packages/sfprofiles/package.json#L74-L75

@azlam-abdulsalam I was looking into this a bit more. Looks like the only usage of request / request-promise-native in the entire sfp library is in this method:
https://github.com/dxatscale/sfpowerscripts/blob/5ab19002f66fa843a2a4c6605829c41ad87cea65/packages/sfprofiles/src/utils/scratchOrgUtils.ts#L32-L47
It doesn't look like this method is referenced anywhere in the codebase. (PoolCreateImpl is using ScratchOrgLimitsFetcher, the latter not having a dependency to either request / request-promist native, nor sfprofiles):
https://github.com/dxatscale/sfpowerscripts/blob/5ab19002f66fa843a2a4c6605829c41ad87cea65/packages/core/src/scratchorg/pool/PoolCreateImpl.ts#L57
Perhaps the vulnerable method is an artifact of the past, and can simply be removed (which would also eliminate the dependency on request and request-promise-native)?? 🤞

Wohooo! Thanks @rygramer  Checking this one


implement kurtosis udaf

Which issue does this PR close?

Closes #12250
Rationale for this change

I followed the changes in PR: https://github.com/apache/datafusion/pull/12273
What changes are included in this PR?

Are these changes tested?

yes
Are there any user-facing changes?

Given #12357 we may want to move this function to some other repo rather than the core

FYI #12357

Thank you @jatin510 .
As @Weijun-H  says, is there any chance you can retarget this PR to https://github.com/datafusion-contrib/datafusion-functions-extra instead?

sure, I will make the changes.

Closing as this PR is going to be retargeted