Update tf2_saved_model.md

URL fixed for export_mnist_cnn.py
The docs has a incorrect url, here is the correct one

@googlebot I signed it!

Thank you, Hrishikesh! The PR looks good. I'll shepherd this change through our internal copy of the repo and back into GitHub.

Thank you, Hrishikesh, for the contribution! Your contribution was merged into master in 0670048.


Update parse.go

The error text when you don't have a trailing slash isn't clear. This is to make it super-obvious!

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

:)


SwipeTableViewCellDelegate add canBeginEditing

If you don't want to swipe action to begin simply return no swipe items via the editActionsForItemAt: delegate callback.


FOOTER ENHANCEMENT

MADE THE FOOTER LOOK MORE PROFESSTIONAL AND ALSO MADE IT EASIER TO USE

please add the labels

your pr has conflicts kindly resolve them

your pr has conflicts kindly resolve them


Fix operator regex

Since django operators aren't words we should use the negated version of \b: \B.

That would only match if there are additional non-word characters on the sides of the token, so you couldn't put an operator directly next to a word. Maybe the proper fix is to drop these boundary checks entirely?


Replace UTC_Now() workaround with MSRest.UTC

use msrest.serialization utc instead of utc_now.
https://github.com/Azure/azure-sdk-for-python/issues/13405
@KieranBrantnerMagee  let me know what you think.

@lmazuel hope you can check again, this is after a merge conflict.

/azp run python - servicebus - tests

Thank you!


docs: fixed issue in chatbot.ipynb

Description: The tutorial would return ValueError: SystemMessage should be the first in the history since the prompt wasn't defined
Issue: N/A
Dependencies: N/A

existing notebook runs for me. prompt is defined higher up (same with model)!


Refactor gatk germline (resolves #280)

resolves #280 #426 #425 #401
@jvivian  @fnothaft @hannes-ucsc

@jpfeil — Ping me when tests pass and I'll review

@jpfeil do you want us to review this? If not, please set the "needs work" label.
Come review time, I will also ask for the commit titles to all be prefixed in such a way that the connote the pipeline they refer to.

WRT https://github.com/BD2KGenomics/toil-scripts/commit/14a268a, alignment with bwa is already implemented in the end-to-end germline pipeline --> https://github.com/BD2KGenomics/toil-scripts/blob/master/src/toil_scripts/adam_gatk_pipeline/align_and_call.py#L98, https://github.com/BD2KGenomics/toil-scripts/blob/master/src/toil_scripts/adam_gatk_pipeline/align_and_call.py#L183.

@jpfeil Just dropping a reminder to add a list with the required/complete features to this PR.

[x] Alignment
[x] Pre-processing
[x] Variant Calling
[x] Hard Filtering
[ ] VQSR
[ ] Clinical Variant Annotation

@jpfeil please make the list more specific. What are the changes you are making? What features were requested to be added, removed, changed, etc.

So this PR greatly expanded in scope from what we discussed at the start: eliminate dupe code between germline and somatic pipelines, focus on validation. We need this to merge for the ADAM recompute, so I'd like to triage the Oncotator/annotation and Synapse bits out to a follow-on PR, and set a deadline for validating this code and merging the PR. Specifically, I'd like to propose freezing the features, wrapping up development by Tuesday of next week, and shooting to validate the pipeline on a single sample by the Tuesday after.
I think right now, if we triage the annotation/Synapse work out to another PR, that leaves joint calling as the last feature. At a cursory glance, that code is implemented already in this PR. I am concerned about the implementation---the GVCFs we are using for SGDP are O(25GB), so joint calling on a single node is out of the question---@jpfeil, do you have a plan for addressing that issue?
I'm proposing the following action items:

[ ] @fnothaft: make review pass by end-of-day today
[ ] @jpfeil: open ticket to triage out annotation/Synapse work
[ ] @jpfeil: write up validation strategy and timeline on this PR (can you do this in time for Monday scrum?)
[ ] @jpfeil: explain joint calling architecture and how the joint calling process will get around excessive data volumes

CC @briandoconnor for review/work planning.

Okay that sounds like a good plan. I will add a modified job function that handles running a single sample through the pipeline with VQSR.

Shouldn't the single sample case fall under the hard filtering? Also, I thought we had an override that allows running <30 samples through VQSR.

I can add a check to see if there are enough variants in the sample before running VQSR. Can you send me a description of your samples?

I can add a check to see if there are enough variants in the sample before running VQSR. Can you send me a description of your samples?

What's the current validation plan? IMO, for a single sample, we should validate against GIAB/NA12878 high coverage from 1KG.

My plan is to run NA12878 for validation.

Just finished my review pass. @jpfeil let's plan to have a timeline for closing out this ticket by scrum on Monday, and we can then discuss at scrum.

Ready for review @fnothaft @jvivian

@jpfeil where do we stand on the action items in https://github.com/BD2KGenomics/toil-scripts/pull/335#issuecomment-237884145? Can you update those?

Also, how is the validation going? Do you have results/a timeline for results?

Also, why did all of the commits to address review comments get squashed down, and why is the work from #418 (to resolve #401) now over here? Unless I missed a conversation where we decided to change the scope of this PR, commit https://github.com/BD2KGenomics/toil-scripts/pull/335/commits/5158c26c346493b604c6df788d1246e3cb168d82 shouldn't be on this branch.

Most of the changes were consistent across #418 and #280, so I thought it would be easier to review in one PR (https://github.com/BD2KGenomics/toil-scripts/pull/418#issuecomment-239998323).

Ah, sorry, forgot that I OK'ed squashing. Nevermind there—didn't mean to be inconsistent—forgot what I said.
That being said, my comment on #418 that you linked to was saying that I wanted #418 rebased on this PR so #418 it was cleaner to review, not saying that I wanted #418's changes pulled over here.
My preference is guided by the timeline for verifying the pipelines. How is that going? From my side, I need #280 to close ASAP. If the timeline continues pulling out, I will need to look at alternatives for the adam_gatk_pipeline, and yes, I am working on alternatives as we speak. So for example, if you're expecting verification of the changes to resolve #280 to complete by Wednesday, but the changes to resolve #401 will push out this PR by another 2 weeks, then we can't have those changes in here. If verification of #208 is more than a week out, then I will need to move to an alternative.
TL;DR here is that I need schedule guidance on this PR. Will you be on scrum today?

I would still like 5158c26 removed from this PR (unless it has also been verified). Making a pass on the first commit now...

I modified the exome variant code because the tests were failing. @jvivian

Rough pass complete.
My biggest issue with this PR are all of the changes to common lib. I would prefer to see common lib additions being made in their own PR with review from @hannes-ucsc and @fnothaft, but I'll defer to them about what they'd prefer.
Aside from that, nj! This has obviously been a ton of work.

Explain joint calling architecture:
HaplotypeCaller is run per sample and I collect the g.vcf file store ids in a dictionary. The GenotypeGVCFs function then merges the g.vcfs into a single vcf file. If run-vqsr is set to True, I pass the joint vcf through VQSR. Otherwise, I use the default hard filters.
Link to code
Based on: https://software.broadinstitute.org/gatk/documentation/article?id=3893
@fnothaft @jvivian

Hi @jpfeil
That sounds a-OK and all, but that explanation is somewhat cursory. My question about the joint caller has been about scale. I'm not 100% sure what the size of a gVCF for WES is, but for a WGS run it varies from 5-35GB. Typical cohort sizes can be >>100 samples (I think ExAC joint called ~250 cohorts of ~250 samples), so do we have a strategy for dealing with said input data volume? Do we have limits to what we can call? Have we documented those limits? How are we making sure that the disk requirement for the final joint calling job isn't out of bounds --> if we sum across the promised values for all sample gVCFs, we could get a disk requirement that's larger than the free disk on our largest available node, and that'll just hang forever. Do we do anything to log/warn/pre-validate this? Etc. Those are the sorts of architectural questions I'm interested in.

Thanks @fnothaft ! The best practices pipeline recommends batches of ~200 samples, but of course it depends on the type of samples you are running. I think the user should specify a batch size that is appropriate for their samples and compute resources.

Hi @fnothaft !
After talking with @jvivian  and @hannes-ucsc , I think a better solution is to batch samples  based on the number of VCFs that can comfortably fit on a single node. I am working on a job function that reserves all cores for a node (to prevent other jobs from running and taking up disk space), queries the free disk space, and finally batches the GVCFs such that the batches fit on the node and are approximately the same size. There is the problem of having a batch that is too small for VQSR, so we should also check that there is a minimum number of variants per batch. GATK recommends around 25,000 variants for VQSR. I think we should also have a maximum number of VCFs per batch so we do not increase our memory requirements too much. Please let me know what you think about this plan!

I think a better solution is to batch samples based on the number of VCFs that can comfortably fit on a single node.

The main problem with that approach is that your variant calls will depend on how you structure your cohorts. I.e., if your pipeline gets split into 2 sub-batches instead of 4 sub-batches, you wouldn't expect to get the same variant calls at a sample level at the end, unless you had incredibly uniform batches. Additionally, if you have a batch of samples that you split into n sub-batches, you won't wind up with a "squared off" variant matrix at the batch level, since the n sub-batches will all (likely) have distinct sets of variants. Ideally, if we were going to split anything, we'd split by locus, which should be safe (with the "minor" issue of splitting up gVCF blocks if not working with BP resolution gVCFs).

NA12878
Threshold
True-pos-baseline
True-pos-call
False-pos
False-neg
Precision
Sensitivity
F-measure

Hard Filter
None
3374152
3382820
16969
18055
0.9950
0.9947
0.9948

VQSR
None
3088970
3094529
13874
303237
0.9955
0.9106
0.9512

I also made a small BAM file for testing VQSR (https://www.synapse.org/#!Synapse:syn7211902).
It takes about an hour to run through the entire pipeline.

I have extracted resource requirement information from the pipeline here

Ready for review @fnothaft @jvivian

The test passed @jvivian


Pagination returns to initial index when data updated

so I am have a function inside a useEffect hooks
  useEffect(() => {
    getData('post')
  }, [data])

and this is my table component
<DataTable columns={column} data={data} />

since data is alawys being updated my pagination would not let me pass to the second page and it always returns me to pageIndex 0.

solution
is found in this link.
closing this


Highlighted Color is not Showing back in epub

Highlighted Color is not Showing back in epub after clicking rows in HighlightList !!

should be fixed now with v0..3.1..


Unable to make changes to torque config after first using the wizard

When applying the torque wizard, the initial torque animation runs with cartodb_id as the time column, but this error appears in the console:
Uncaught Error: TrackJS Caught: Cannot read property 'options' of undefined y.wrapError @ tracker.js:26 l.wrapAndCatch.d.util.hasFunction.a.(anonymous function).d.util.hasFunction.f.(anonymous function) @ tracker.js:4
Any other attempts to modify the torque config, either in the wizardor the CartoCSS pane do not work until the page is reloaded.

this is cartodb, right? cc @fdansv

Yes, if it's related to the wizard or tabs it goes under cartodb/cartodb

I'll move it.
On Wed, Jun 17, 2015 at 9:09 AM, Francisco Dans<EMAIL_ADDRESS>wrote:

Yes, if it's related to the wizard or tabs it goes under cartodb/cartodb
—
Reply to this email directly or view it on GitHub
https://github.com/CartoDB/cartodb.js/issues/529#issuecomment-112793844.


Foundry VTT Version 10

Update Challenge Tracker to Foundry VTT version 10.

Added in release 2.0.


[REQ] import into rancher

Our clusters are managed via rancher. The created K3s cluster can be imported successfully with a few modifications.
I could either offer a PR that optionally does full registration in rancher at the same time ( create rancher cluster and import k3s ) or just customize the few places (set the rights of kubeconfig).
What do you think about this?

Yes, Sir! :) OOTB support for Rancher would be awesome. Why not even an option of automatically deploying it. Do as you see fit, this project needs to support real world scenarios like the ones you are exposing.

Both solutions would be very useful. (I am currently planning on moving some Clusters from our rancher/bare-meta/datacenter -> hcloud ) and still trying to wrap my head around the ui-node-driver or importing the cluster afterwards (besides the hetzner-controller etc..).
Im guessing deploying rancher ist very simple, since it is just a helm deployment, just have to figure out the ssl/letsencrypt part.
helm install rancher-stable/rancher \
  --name rancher \
  --namespace cattle-system \
  --set hostname=rancher.xyz.net \
  --set ingress.tls.source=secret

The importing although needs a token from the rancher. I'd be very interested if there are any additional steps besides running the agent @batthebee .

@strowi  I had a few problems with the ui-node-driver+dockermachine setup, one i could not fix or work around unfortunately. When creating a cluster via terraform I always had the error message: "default-token already in use".
I then switched to setting up the clusters externally and only importing them. That worked right away.
I am currently building a module to set up rancher via terraform. When it's ready, I'll post the link to the repo here. I hope, I am Sunday ready.

@strowi  There was almost nothing else to do. Additionally only the rights for the node kubeconfig must be set (see k3s docs). I hope that I can also make a PR until Sunday.

Yes, Sir! :) OOTB support for Rancher would be awesome. Why not even an option of automatically deploying it. Do as you see fit, this project needs to support real world scenarios like the ones you are exposing.
@mysticaltech
Do I understand you correctly that you want to have an optional Rancher installation within the module? My feeling is that this is better off in its own repo. Currently the module does exactly one thing, set up a k3s cluster on top of hetzner, which I personally like.

If you want to support rancher here, I would suggest we do that in a second repo.
In this repo only the read-write to the node kubeconfig necessary for rancher goes.
In the second repo I would build two modules, one that registers a cluster with terraform and a second that sets up terraform on such a cluster (and registers itself at the same time).
What do you think?

I did not understand exactly what you meant, but compartmentalizing stuff is a good idea.
If we can have a second repo that can run rancher to manage a kube-hetzner cluster, it would be awesome!
You have great ideas @batthebee!

@batthebee Have invited you to join the org so that you can initiate the second repo for the above if you are inspired.
Welcome to Kube-hetzner ! :)

Thanx! :)

Hey folks, just wanted to share that in https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/pull/173, we actually deployed a helm chart with a simple YAML HelmChart definition. So no need to use the command line cli, that is a major simplifier IMHO!

I understand that probably it's not wise to install Rancher on the cluster itself, but just felt I needed to share the above info. It may be useful in another context.

Ah, I retract what I said above, as the author of this video does, it's pretty nice to have rancher run on the cluster itself too. But definitely not mandatory!
Probably that is a use case different from what you are proposing here @batthebee. Both seem very interesting!
Probably just deploying Rancher into the cluster can be done very easily with a HelmChart declaration, and a flag variable. Will try that in a separate PR as soon as I can.

Finally, will just install it via the helm cli, that way it's easier to get the whole ingress config right. It's not really something that can be automated to the point of an HelmChart YAML file.
Basically, the only thing required here @batthebee is to make this cluster compatible to be imported in a third-party Rancher install. As you said, formatting the kubeconfig well... etc.  That would indeed be awesome!

Good work! ;)
Just to clean this up (hopefully) - having worked with rancher for +5y, the the difference between same cluster and extra-cluster is "just" for security and stability issues.
It is best practice to have a separate management-cluster running only rancher. This prevents high workloads etc, from interfering with it (or developers from deleting the rancher-daemonset :) ).
It might be possible to get the rancher-agent token via the terraform-module (haven't checked this yet) and deploy the agent accordingly.

Thanks for clarifying the matter @strowi! That makes perfect sense.
In that case, the ideal scenario would be a single node cluster for Rancher only (the beauty of single-node clusters is that it uses Klipper as LB, so removes the need for a Hetzner LB, hence dropping the cost even more).
And now that we have support for Longhorn, we can set up an automatic backup of PVCs to any S3 compatible endpoint, and make everything even more robust.

@mysticaltech depends on definition of "ideal scenario" ;) In production you'd probably want at least a 3-node rancher cluster for high-availability if this is a developers only access to the downstream clusters. But that setup really wouldn't differ i guess.
Backups of pvcs are neat and definitely useful! Although i've had a lot of trouble with longhorn itself (very high cpu usage for keeping the disks in sync, etc), i think it is very easy to setup. (Otherwise i can recommend restic/velero for backups)

Awesome @strowi!! Thanks for sharing your production experience. It's invaluable!

happy to help;)

@batthebee @strowi Have added the possibility to deploy Rancher locally. Please do not hesitate to suggest changes. https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/pull/176
It does not answer the requirement of that issue, but at least it's great to have Rancher!


Virtual + real dom manipulation simultaneously is not a good idea.

Virtual + real dom having issue which is difficult to debug.

Can you elaborate?
We don't modify the VDom directly, besides using public APIs exposed by react or react-dom. Same goes for Angular.


Doesn't pick the correct cards on turn 3

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
NA
FGA build number
2690
Describe the bug
FGA Ignores card priority and servant priority on turn 3.

Video
https://github.com/user-attachments/assets/3679cc80-0bc0-47d9-8db4-6bf6f625ec04
Device model
LDPlayer9 "SM-G9880"
Android version
7
Screen size
No response
RAM
No response

Are you using spam? Card priorit, servant priority and spam option doesn't work well together

Are you using spam? Card priorit, servant priority and spam option doesn't work well together

Yes, I'm using spam from turn 3 and onwards. I suppose that's most likely it, when combined with an NP, but it still seems sort of random.

I am having the same issue, and I have spam turned off. It very consistently chooses spishtar's cards over morgans cards only on wave 3 (although, it seems to pick the correct priority order on turn 7 weirdly enough). This bug happens every run on both my bluestacks emulator and my galaxy s23 plus phone, and has also been verified by others to happen with other servants as well.

https://www.youtube.com/watch?v=eQwUjVCpsXo
https://www.youtube.com/watch?v=O4VLRL1DqrI
https://www.youtube.com/watch?v=8ioVDMlh4J0
https://www.youtube.com/watch?v=CwwKzRx4RwY

Same issue with both of these settings.


Visually Appealing Loading Animation.

Visually Appealing Loading Animation
Description:

I would like to request a smooth and visually appealing loading animation to appear after user actions (e.g., submitting their birthdate, retrieving campaigns, or navigating between pages). This will enhance the user experience by adding an engaging transition during the wait times.

Key Benefits:

Keeps users engaged and entertained during loading times, preventing frustration or boredom.
Improves the overall aesthetic and flow of the website by creating seamless transitions between actions.
Enhances the brand’s visual identity by incorporating creative animations aligned with the site's design theme.

Expected Functionality:

The loading animation should be triggered during critical moments, such as:

After the user submits their birthdate.
When campaign data is being loaded.
When navigating between sections or pages of the website.

The animation should be smooth, lightweight, and not impact the website's performance.

Possible ideas include:

A progress bar with a sleek design.
A spinning logo or birthday-themed animation (e.g., balloons, cake, or confetti).
Minimalist animations with fading effects, matching the site's overall aesthetic.

Additional Considerations:

Ensure the loading animation works smoothly on both desktop and mobile devices.
The animation should have a short duration to keep users engaged without causing delays.
Ensure that the animation is non-intrusive, and the user can see it only when necessary, without interrupting the experience.

@brgkdm may i be assigned this issue please.
Thanks!

@brgkdm may i be assigned this issue please.
Thanks!

Added.


fix(optimizer): fix translate apply rule

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

According to the paper Unnesting Arbitrary Queries, we can calculate the domain directly by projecting the correlated columns and make a distinction.
Previously, we support rewrite the left side of the apply operator if it belongs to SPJ type which only fetch the correlated columns and can avoid some join, but in reality no one can guarantee the left side of the apply can only be SPJ type, so we need a general way to deal with it.

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)
https://github.com/risingwavelabs/risingwave/issues/6350
https://github.com/risingwavelabs/risingwave/issues/6499

If I understood correctly, previously, we will not apply the translate apply rule correctly, and we will essentially not perform translate apply when LHS is not SPJ and hence, we will end up with the wrong result on LHS when performing the apply.
To my understanding, our objective in this translateapply step is simply to extract the distinct column values of the correlated indices on LHS.
However, rather than returning a top-level None for non-SPJ in rewrite, isn't it more efficient to modify it so that opaque types that do not match the pattern will simply be wrapped in a project onto the correlated indices? If I'm not wrong, if the operators above it are simply joins and projects, then we will still get the right result by taking the distinct agg of the top level operator's correlated indices.
Well, the only advantage to doing so would be to reduce the amount of intermediate state by pruning away unnecessary columns in project and join. The cost would be additional project operators at the leaves of SPJ.
Well, it would be nicer if we knew every simple projection can be pushed down into an output_indices-type parameter.

Actually, even the LHS is SPJ form, we don't handle properly. It ignore the project which makes no sense at all and leads to bugs.


Need additional HD space on the D drive for embedding h5 files

To save the embeddings from the SQuAD 2.0 task, a lot more HD space is necessary on the D drive.

Completed

Completed


Bad request

When trying to register with the "Login with Tumblr" button, I get an error page that says "Bad request. The browser (or proxy) sent a request that this server could not understand."

same

Tumblr started re-directing their auth endpoints to https, which the oath2 library didn't like. The issue should now be fixed.
Sorry it took me so long to get to this one.


ios parameters

how to new IOS flops,kernels....

how to know IOS memory,flops,kernels....

Hi,
You can use the flops(), memory_access() and kernels() member function of ios.ir.Graph object to get the memory flops, memory access, and memory access number. The following is code snippet is an example.
import os
import numpy as np
import ios

def main():
    graph = ios.models.inception_v3()

    graph.sequential_schedule()
    latency = ios.ios_runtime.graph_latency(graph, batch_size=1, repeat=6)
    print('{:>30}: {}'.format('seq flops (flops)', graph.flops()))
    print('{:>30}: {}'.format('seq memory access (bytes)', graph.memory_access()))
    print('{:>30}: {}'.format('seq kernels', graph.kernels()))

    optimized_graph = ios.optimize(graph, batch_size=1)
    optimized_latency = ios.ios_runtime.graph_latency(optimized_graph, batch_size=1, repeat=6)
    print('{:>30}: {}'.format('ios flops (flops)', optimized_graph.flops()))
    print('{:>30}: {}'.format('ios memory access (bytes)', optimized_graph.memory_access()))
    print('{:>30}: {}'.format('ios kernels', optimized_graph.kernels()))

    os.makedirs("./outputs/", exist_ok=True)
    ios.draw(optimized_graph, fname=f'optimized_{graph.name}.png', label=f'Optimized Graph, Latency = {np.mean(optimized_latency):.3f}')
    print(f' Sequential schedule: {np.mean(latency):.3f} ms')
    print(f'  Optimized schedule: {np.mean(optimized_latency):.3f} ms')

if __name__ == '__main__':
    main()

FYI: If you want to benchmark IOS, I would suggest doing it on cuDNN 7.6.5. The kernels in the new version of cuDNN (i.e., 8..) are not good at inter-op parallelization. It might because they do some pre-processing when launching a new kernel (e.g., set the cache config) that would trigger the kernel serialization in the new version of cuDNN.


Test Issue created at Wed, 14 Apr 2021 08:31:31 GMT

Test issue body  Wed, 14 Apr 2021 08:31:31 GMT

Comment on issue at Wed, 14 Apr 2021 08:31:34 GMT


Is it possible to monitor child nodes by monitoring the parent nodes?

Since there sometimes are many child nodes underneath a parent node it would be helpful to be able to just monitor the parent node and getting al the data from the child nodes underneath. Is this possible?
Thanks!

Complex data types are now supported in 2.7.183.


Fix rsync on Windows (path conversion, return code checking) and add trailing /

Rsync is very curious about "/", so when you sync directories, we should always make sure that we add a trailing "/" to really sync the content of the directory. Return code checking was missing too.
Also on Windows with cygwinbased rsync, the path names must be /cygdrive/c, otherwise rsync complaines:
The source and destination cannot both be remote

@elconas what's up with the .gem file in your commit?

Ups, will remove and recreate Pull request

git push --force

perhaps we should also add .gem to our .gitignore

Opened Issue https://github.com/voxpupuli/librarian-puppet/issues/21 for tracking


uppercase / lowercase homogeneity in filenames

I would like that looking at the filename we now if we describe a class.
This means that it should be Image.js and Stack.js, ROI.js
Currently it is mixed

@targos, what do you think ?

+1 but I prefer camelCase everywhere so: Roi, RoiLayer, etc.

Most of the name are now correct.


Update community README table

A hyperlinked page making it easier to jump to a particular control family
An explicit "description" column for the policies
Tables entirely composed of Markdown (instead of an HTML table), which I think will be easier to maintain and understand

@dhaiducek Thanks for the PR!
Can you make sure your PR includes below two PRs just got merged?
https://github.com/open-cluster-management/policy-collection/pull/36
https://github.com/open-cluster-management/policy-collection/pull/37

Shoot, I knew something was weird when it said I had a merge conflict! I've added the rows back in. 🙂


First version of full documentation

Most of the text was taken from the README and existing markdown files (mainly about the integration test framework).
New parts are:

How does it work
Creating patches

Online version on Read the Docs: https://recipy.readthedocs.io/en/better-docs/
I also added a command to the cli to print the patched modules table (either in plain text or rst format): recipy pm --format plain or recipy pm --format rst
After this is merged, the next step would be to shorten the README and include a link to the online documentation.

Thanks for doing this - sorry I didn't get round to looking at it and merging earlier.


How to mock or by-pass SSO for testing

Hi folks,
I am using django-auth-adfs v. 1.13.0 with Django 4.2.0 and Azure Entra for single sign-on

django_auth_adfs.urls included in urlpatterns
LOGIN_URL = "django_auth_adfs:login"
Sign In pages links set to 'django_auth_adfs:login' in the base template.

I would like to test all of my views while completely ignoring django-auth-adfs and relying instead on the default Django authorization. What is the best way to accomplish this?
I have a test settings file where I have tried to prevent django-auth-adfs from loading:

INSTALLED_APPS = list(BASE_INSTALLED_APPS)
INSTALLED_APPS.remove("django_auth_adfs")
MIDDLEWARE = list(BASE_MIDDLEWARE)
MIDDLEWARE.remove("django_auth_adfs.middleware.LoginRequiredMiddleware")
AUTHENTICATION_BACKENDS: list[Any] = []
LOGIN_URL = "account_login"
I set the root url config to the application URLs that excludes the django-auth-adfs.urls

Even after all of this, however, debug shows that i am still getting redirectred to /oauth2/login?next=/orgs/1
Any suggestions? Could it be that somehow the top-level URLs are still being included?
Regards,
Justin

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

I think the main thing you'd have to change is AUTHENTICATION_BACKENDS so that it has the default value. It shouldn't be empty otherwise it won't know how to look up a user based on any type of authentication and it'll always be treated as logged out. Or I would assume as such.

That was the missing piece! Thank you, @tim-schilling

To expanding on my test.py settings above and explain with more detail for any users who might be wondering how to do this -
Instead of setting the AUTHENTICATION_BACKENDS to an empty array in my settings you should set this explicitly to the default as @tim-schilling suggested:
AUTHENTICATION_BACKENDS = ["django.contrib.auth.backends.ModelBackend"]

In addition, unless you specify Debug = True in your tests you should add the testserver to your ALLOWED_HOSTS:
ALLOWED_HOSTS = ["localhost", "<IP_ADDRESS>", "<IP_ADDRESS>", "testserver"]

Finally, don't forget to create a user (User if using the default Django User model or your defined user e.g. MySpecialUser if you use one) and login during your view test setup:
class MyViewTestCase(TestCase):

    test_external_credentials = {
        "username": "testuser",
        "password": "testpassword",
    }
    @classmethod
    def setUpTestData(cls):
        cls.user = MySpecialUser.objects.create_user(
            email=cls.test_external_credentials["email"],
            password=cls.test_external_credentials["password"],
        )

And then login successfully before testing your view:
    def test_view_exists_at_correct_uri(self):
        result = self.client.login(
            email=self.test_external_credentials["email"],
            password=self.test_external_credentials["password"],
        )

        if result:
            response = self.client.get("/expected_view_uri")
            self.assertEqual(response.status_code, 200)

I just realized that I was successfully running all of my tests - including the views - with my default settings rather than my test settings and that when i tried to run them using my test setting - which disable django-auth-adfs - the view test failed. But I suspect that is because I have the following tag littered through-out my base templates:
                                                {% translate "Sign In" %}

Any suggestions for abstracting this login URL out so that I can continue down this route of disabling ADFS for my unit tests?
Regards,
Justin

I'm not quite sure what you're looking for.

@tim-schilling do you know of any non-specific Tag I could use in place of django_auth_adfs:login that would allow the re-direct to happen when the package is enabled but not require that I replace the tag during testing? Or perhaps is there a way to point Django unit tests at alternate templates? I am a bit new to this (< year) so I probably should be using a technique of which I and just not aware.
Regards,
Justin

What about writing your own view that wraps django-auth-adfs so that you can mock it for testing and pass through to adfs in non-testing?

Hey @tim-schilling - that idea sounds interesting and seems possible but I don't know how I would do that at this point. I am going to ask a follow-up question and I think they are related - perhaps even the same.
In my settings for my non-testing environment (i.e. regular settings) I have
LOGIN_URL = "django_auth_adfs:login"

which I change to
LOGIN_URL = "account_login"

It seems to me that if there was a way to simply reference the value of LOGIN_URL in my templates  - like
{% translate "Sign In" %}

Then I would have jumped the last hurdle. Do I need to create a view-wrapper to do this or is there another way?

An example approach it to create a custom tag that allows me to access the login url setting like the one described in a StackOverflow post then I could access the LOGIN_URL setting like this:
{% settings_value "LOGIN_URL" %}

Or I suppose I might introduce a context processor that sets a variable login_location=settings.LOGIN_URL and then I could access it via the url tag (based on another StackOverflow post) like this:
{% translate "Sign In" %}

Am I overthinking this?

Sorry @vanderzielj, you should try asking on the django forum or django discord for generalized help. I don't have the availability to dive in deep into this problem with you. There are a few ways to solve this, but I'm not sure of the implications and they'll depend on your goals. That's why I can't say definitely which way is best.

@tim-schilling I was able to implement a context processor and make it work. I tried to explain what I did above in the hope that it will help some other user of the django-auth-adfs package and help others to use this fantastic app. Thank you for all of your work on this and for making it available for others to use.


Parity between linux, macos and windows

This is so we don't have to re-write all the docs etc.  Is a bit annoying as this will have to be updated along with MicroK8s but is the best I can think of currently.

Should we close this PR?

Yes :)


binder/chore: remove drop test

https://github.com/risinglightdb/risinglight/pull/809

At present, drop test is not necessary, one is that this code is not used
let catalog = Arc::new(RootCatalog::new());
        catalog
            .add_table(0, "mytable".into(), vec![], false, vec![])
            .unwrap();

cc @wangrunji0408


TSMixer: Lightweight MLP-Mixer Model for Multivariate Time Series Forecasting

Hello,
There is a paper which can be added to the library.
https://arxiv.org/abs/2306.09364v3

Hi, thanks for your recommendation.
We will add this to our todo list. But we have to point out that there are extensive papers in this area, it may take a while to find out the papers that we are interested in.

Great, I can add it too.
Iwill send a pull request when I finish it, but it may take a while.
Thanks,
Ali


missing memoize resolver

In the modelValueUpdaterMap (here: https://github.com/davidkpiano/react-redux-form/blob/master/src/utils/sequence.js#L32) the model props is being passed as the map key.
This leads to the change action always passing the same cached value to the form and model reducers.
Here is a sample of the problem. As I typed 'abc' the change action was called three times. In the first call the modelValueUpdaterMap returned the input value and cached this same value, and in the next calls it returned the cached value from the memoize map object.

Creating a resolver like this (props, eventValue) => props.model + eventValue solved the problem. But is this the right solution?
Great lib BTW.

I ended up just removing memoization for now. Thanks for catching that!


BUG str_replace array

let filtered = "test_test";
let sanitize = ["/", "'", "\"", "?", "!", "%", "$", "&"];
    let filtered = str_replace(sanitize, "", filtered);

    var_dump(filtered);

Result Zephir: string(0) ""
Result PHP: string(9) "test_test"

Are you using the development branch?

I use the stable version

This doesn't happen when using development.

@steffengy Let's merge development in master

You have not managed to reproduce the bug ?

@nekulin Please change your branch to development
cd zephir
git checkout development

@andresgutierrez
I literally thought the same thing :)
It also maybe makes sense to exchange some kind of private contact information (Instant Messaging or IRC or similar), to discuss
some future steps.
Generally I think the next step after merging development into master would be to push the version and then getting cphalcon to compile.
(which I haven't done yet, since I'm mainly using windows and that's a bit tricky to set up)

Sure, please send me your gtalk to<EMAIL_ADDRESS>

I compiled a version with development, the problem remained and the version has not changed, also 0.7.1 b

That is indeed strange, which PHP version are you using?

PHP 5.5.28

I just added the code from your issue to the tests and they all pass for PHP5.X - PHP 7.
How did you declare filtered and sanitize ?

string filtered;
array sanitize;

zephir version - Sie welche Version ?

I cannot reproduce this with
string filtered;
array sanitize;
let filtered = "test_test";
let sanitize = ["/", "'", "\"", "?", "!", "%", "$", "&"];
let filtered = str_replace(sanitize, "", filtered);

var_dump(filtered);
exit();

results in string(9) "test_test" for me.
Maybe retry using master, after the merge now (and possibly a zephir clean / zephir fullclean)

i can confirm str_replace array works now

Possible refs #732


Missing device configuration: Eurotronic Comet Z-Wave

Checklist

[X] It is not in the configuration DB

[X] It was not merged recently or has a pending PR

Which device is missing?
Eurotronic Comet Z-wave
Manufacturer ID
0x0148
Product Type
0x0004
Product ID
0x0003
Firmware Version
11.26.4
Is the device listed on the Z-Wave Alliance website?
https://products.z-wavealliance.org/products/1601
Do you have a manual?
https://manual.zwave.eu/backend/make.php?lang=en&sku=EURECOMET&cert=
The device is detected but manufacturer does not show in Home Assistant, looks like a product ID increment as the config DB has product 1,2 and not 3
The parameters are enumerated in discovery, as below, but are not editable or named to edit once interview completes (attached)
2023-02-14T18:03:52.164Z CNTRLR « [Node 113] received information for parameter #1:
parameter name:      LCD_invert
value format:        UnsignedInteger
value size:          1 bytes
min value:           0
max value:           1
default value:       0
is read-only:        false
is advanced (UI):    false
has bulk support:    true
alters capabilities: false

@zwave-js-bot import config 1601 from zwa

Hm, automatically importing the config was not helpful. The IDs are a bit different than what we currently have for the Comet Z, but that can happen. We also don't have any config params defined at the moment, so they should be added too.

If it helps, this is a snippet from the interview log:
  "index": 0,
  "installerIcon": 4608,
  "userIcon": 4608,
  "status": 4,
  "ready": true,
  "isListening": false,
  "isRouting": true,
  "isSecure": false,
  "manufacturerId": 328,
  "productId": 3,
  "productType": 4,
  "firmwareVersion": "11.26.4",
  "zwavePlusVersion": 2

@AlCalzone Is there anything I can do to help get the config parameters added, so they can be edited in Home assistant?
I'm happy to help as I can't use these TRV's without the ability to alter some of the parameters (like invert display etc)

Yeah, see the comment above: https://github.com/zwave-js/node-zwave-js/issues/5464#issuecomment-1430241560

Thanks, I'll try to get what you need.
I don't use JS UI, just the supported non UI zwave-js, so I dont believe I have access to parameter edit.

@kpine Thanks!
great workaround until the device gets proper support :)

If I set manually in zwave-js-ui the Parameter 8 for the temperature offset, I get a “Parameter 8 set failed” in return. Is this because the device is not yet fully supported? I would have thought that setting parameters manually always works.
zwave-js-ui: <IP_ADDRESS>c5be
zwave-js: 11.5.3

Sorry, this was a mistake on my side. While the command still seems to be ignored, I cannot reproduce the error message any more. Probably I mixed up devices in the UI.

In case this info helps anyone who can make the config for this - I moved to JSUI as I couldn't get the parameters working otherwise .

2023-09-17T17:36:10.413Z - INTERVIEW STARTED 2023-09-17T17:36:10.472Z - INTERVIEW STAGE COMPLETED Arg 0: ProtocolInfo 2023-09-17T17:36:11.772Z - INTERVIEW STAGE COMPLETED Arg 0: NodeInfo 2023-09-17T17:36:11.827Z - VALUE ADDED Arg 0: └─commandClassName: Manufacturer Specific └─commandClass: 114 └─property: manufacturerId └─endpoint: 0 └─newValue: 328 └─propertyName: manufacturerId 2023-09-17T17:36:11.830Z - VALUE ADDED Arg 0: └─commandClassName: Manufacturer Specific └─commandClass: 114 └─property: productType └─endpoint: 0 └─newValue: 4 └─propertyName: productType 2023-09-17T17:36:11.833Z - VALUE ADDED Arg 0: └─commandClassName: Manufacturer Specific └─commandClass: 114 └─property: productId └─endpoint: 0 └─newValue: 3 └─propertyName: productId 2023-09-17T17:36:11.955Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: libraryType └─endpoint: 0 └─newValue: 3 └─propertyName: libraryType 2023-09-17T17:36:11.959Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: protocolVersion └─endpoint: 0 └─newValue: 7.15 └─propertyName: protocolVersion 2023-09-17T17:36:11.962Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: firmwareVersions └─endpoint: 0 └─newValue: 11.26, 1.4 └─propertyName: firmwareVersions 2023-09-17T17:36:11.965Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: hardwareVersion └─endpoint: 0 └─newValue: 1 └─propertyName: hardwareVersion 2023-09-17T17:36:13.044Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: sdkVersion └─endpoint: 0 └─newValue: 7.15.4 └─propertyName: sdkVersion 2023-09-17T17:36:13.050Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: applicationFrameworkAPIVersion └─endpoint: 0 └─newValue: 10.15.4 └─propertyName: applicationFrameworkAPIVersion 2023-09-17T17:36:13.058Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: applicationFrameworkBuildNumber └─endpoint: 0 └─newValue: 256 └─propertyName: applicationFrameworkBuildNumber 2023-09-17T17:36:13.062Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: hostInterfaceVersion └─endpoint: 0 └─newValue: unused └─propertyName: hostInterfaceVersion 2023-09-17T17:36:13.070Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: hostInterfaceBuildNumber └─endpoint: 0 └─newValue: 0 └─propertyName: hostInterfaceBuildNumber 2023-09-17T17:36:13.075Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: zWaveProtocolVersion └─endpoint: 0 └─newValue: 7.15.4 └─propertyName: zWaveProtocolVersion 2023-09-17T17:36:13.079Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: zWaveProtocolBuildNumber └─endpoint: 0 └─newValue: 256 └─propertyName: zWaveProtocolBuildNumber 2023-09-17T17:36:13.084Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: applicationVersion └─endpoint: 0 └─newValue: 11.26.4 └─propertyName: applicationVersion 2023-09-17T17:36:13.088Z - VALUE ADDED Arg 0: └─commandClassName: Version └─commandClass: 134 └─property: applicationBuildNumber └─endpoint: 0 └─newValue: 43707 └─propertyName: applicationBuildNumber 2023-09-17T17:36:13.217Z - VALUE ADDED Arg 0: └─commandClassName: Battery └─commandClass: 128 └─property: level └─endpoint: 0 └─newValue: 85 └─propertyName: level 2023-09-17T17:36:13.220Z - VALUE ADDED Arg 0: └─commandClassName: Battery └─commandClass: 128 └─property: isLow └─endpoint: 0 └─newValue: false └─propertyName: isLow 2023-09-17T17:36:13.720Z - VALUE ADDED Arg 0: └─commandClassName: Thermostat Mode └─commandClass: 64 └─property: mode └─endpoint: 0 └─newValue: 1 └─propertyName: mode 2023-09-17T17:36:13.946Z - VALUE ADDED Arg 0: └─commandClassName: Thermostat Setpoint └─commandClass: 67 └─property: setpoint └─propertyKey: 1 └─endpoint: 0 └─newValue: 8.5 └─propertyName: setpoint └─propertyKeyName: Heating 2023-09-17T17:36:14.009Z - VALUE ADDED Arg 0: └─commandClassName: Thermostat Setpoint └─commandClass: 67 └─property: setpoint └─propertyKey: 11 └─endpoint: 0 └─newValue: 16 └─propertyName: setpoint └─propertyKeyName: Energy Save Heating 2023-09-17T17:36:14.098Z - VALUE ADDED Arg 0: └─commandClassName: Protection └─commandClass: 117 └─property: local └─endpoint: 0 └─newValue: 0 └─propertyName: local 2023-09-17T17:36:14.270Z - VALUE ADDED Arg 0: └─commandClassName: Multilevel Sensor └─commandClass: 49 └─property: Air temperature └─endpoint: 0 └─newValue: 24 └─propertyName: Air temperature 2023-09-17T17:36:15.411Z - VALUE ADDED Arg 0: └─commandClassName: Multilevel Switch └─commandClass: 38 └─property: currentValue └─endpoint: 0 └─newValue: 0 └─propertyName: currentValue 2023-09-17T17:36:16.862Z - VALUE ADDED Arg 0: └─commandClassName: Configuration └─commandClass: 112 └─property: 1 └─endpoint: 0 └─newValue: 1 └─propertyName: LCD_invert 2023-09-17T17:36:16.936Z - VALUE ADDED Arg 0: └─commandClassName: Configuration └─commandClass: 112 └─property: 2 └─endpoint: 0 └─newValue: 5 └─propertyName: LCD_timeout 2023-09-17T17:36:16.999Z - VALUE ADDED Arg 0: └─commandClassName: Configuration └─commandClass: 112 └─property: 3 └─endpoint: 0 └─newValue: 0 └─propertyName: Backlight 2023-09-17T17:36:17.060Z - VALUE ADDED Arg 0: └─commandClassName: Configuration └─commandClass: 112 └─property: 4 └─endpoint: 0 └─newValue: 0 └─propertyName: Battery_report 2023-09-17T17:36:17.143Z - VALUE ADDED Arg 0: └─commandClassName: Configuration └─commandClass: 112 └─property: 5 └─endpoint: 0 └─newValue: 5 └─propertyName: Measured_temperature_report 2023-09-17T17:36:17.208Z - VALUE ADDED Arg 0: └─commandClassName: Configuration └─commandClass: 112 └─property: 6 └─endpoint: 0 └─newValue: 0 └─propertyName: Valve_opening_percentage_report 2023-09-17T17:36:17.294Z - VALUE ADDED Arg 0: └─commandClassName: Configuration └─commandClass: 112 └─property: 7 └─endpoint: 0 └─newValue: 1 └─propertyName: window_opening_percentage_report 2023-09-17T17:36:17.375Z - VALUE ADDED Arg 0: └─commandClassName: Configuration └─commandClass: 112 └─property: 8 └─endpoint: 0 └─newValue: 0 └─propertyName: Measured_temperature_offset 2023-09-17T17:36:17.600Z - VALUE ADDED Arg 0: └─commandClassName: Notification └─commandClass: 113 └─property: alarmType └─endpoint: 0 └─newValue: 0 └─propertyName: alarmType 2023-09-17T17:36:17.603Z - VALUE ADDED Arg 0: └─commandClassName: Notification └─commandClass: 113 └─property: alarmLevel └─endpoint: 0 └─newValue: 0 └─propertyName: alarmLevel 2023-09-17T17:36:17.654Z - VALUE UPDATED Arg 0: └─commandClassName: Notification └─commandClass: 113 └─property: alarmType └─endpoint: 0 └─newValue: 0 └─prevValue: 0 └─propertyName: alarmType 2023-09-17T17:36:17.656Z - VALUE UPDATED Arg 0: └─commandClassName: Notification └─commandClass: 113 └─property: alarmLevel └─endpoint: 0 └─newValue: 0 └─prevValue: 0 └─propertyName: alarmLevel 2023-09-17T17:36:17.658Z - VALUE ADDED Arg 0: └─commandClassName: Notification └─commandClass: 113 └─property: System └─propertyKey: Hardware status └─endpoint: 0 └─newValue: 3 └─propertyName: System └─propertyKeyName: Hardware status 2023-09-17T17:36:17.660Z - INTERVIEW STAGE COMPLETED Arg 0: CommandClasses 2023-09-17T17:36:17.663Z - INTERVIEW STAGE COMPLETED Arg 0: OverwriteConfig 2023-09-17T17:36:17.665Z - INTERVIEW STAGE COMPLETED Arg 0: Complete 2023-09-17T17:36:17.738Z - READY 2023-09-17T17:36:17.758Z - INTERVIEW COMPLETED

(for some reason 'add code' isnt formatting the below like it should)

It's a bit counter-intuitive. First paste code, then select it, then press "format as code".

Product desc : [https://manual.zwave.eu/backend/make.php?lang=en&sku=EURECOMET&cert=]

Thanks to everyone that contributed to adding this ;)

@cadwizzard did you manage to tune parameters of your Comet devices ?
We fail here, so would be pleased to have your report.
Thank you 👍

Hi.
I gave up I'm afraid, and moved to JSUI.
But I'm really unhappy with the CometZ.
Max reporting detail is 0.5c, unlike the Spirit (I have set to 0.3 normally)
The freeze setpoint is higher than the spirit
Also the spirits actually report detail to 0.01 degree, unlike the comet which is 0.5
Worst of all though is that they seemingly don't report temperature change sometimes reliably. I've had to request a multilevel sensor update to get the right info. This has left my boiler running for hours because the room temperature isn't reported.
I found a re-interview seems to fix that for a while and then they report every 0.5 degree again.
I had 20+ spirits and never had these issues. Upgraded 6 to CometZ. Had the problems on every one. Now can't trust them. Have returned 2 and now am trialling the ZigBee Bosch radiator thermostat 2 in z2mqqt. So far they have been brilliant. I didn't want to move away from z wave, but wanted to modernize the radiators.
I feel like Eurotronic have cheaped out with the hardware on the CometZ compared to the spirits.
Hope you have better luck.

Thank you for both your reports.
I plan to use them in Manufacturer specific mode (hence #6583), combined with a room-temperature sensor, and a smart Home Assistant thermostat (for ex. https://github.com/KartoffelToby/better_thermostat). So fully getting rid of the builtin temperature sensor and logic. Should give better results.
But for now stuck with parameters' configuration :|

But for now stuck with parameters' configuration :|

.... So was I. The only reason I moved to JSUI was to accommodate these devices and the reporting resolution and mirror parameters.
The other 50+ devices were well supported in the native z wave js implementation
But I've found the network map and device management super useful. Especially because of the weirdness I was trying to figure out with the CometZ.
It's such a shame they don't work right.
Also if you change the display to mirrored, the up is setpoint down and down is up as you look at the TRV and it doesn't switch.
The spirits don't have that issue

OK, parameters issues fixed in https://github.com/zwave-js/node-zwave-js/pull/6583 👍
May then be usable now 😉


extract pm_user from pm_api_token_id when GetUserPermissions

Using version v2.9.12 and configure provider using token method to communicate with PVE API, it complains
Error: cannot find provided user terraform-prov@pve!terraform-prov-token on proxmox

By checking the code, it seems that we should split user id from token_id string.

Pretty sure I tested this without any problem(it was the point of the commit in fact) 🤨

please provide the config your are using

btw I'm using PVE 7.3-4

# md5sum /home/ethinx/.local/share/terraform/plugins/telmate/proxmox/2.9.12/linux_amd64/terraform-provider-proxmox_v2.9.12
3ee7767cf6af88ebae6f197f1ae584c0  /home/ethinx/.local/share/terraform/plugins/telmate/proxmox/2.9.12/linux_amd64/terraform-provider-proxmox_v2.9.12

The terraform I'm using is here
ethinx@rome ~/r/t/pve-dev> export PM_API_TOKEN_ID='terraform-prov@pve!terraform-prov-token'
ethinx@rome ~/r/t/pve-dev> export PM_API_TOKEN_SECRET='the-secret-here'
ethinx@rome ~/r/t/pve-dev> terraform plan
╷
│ Warning: Provider development overrides are in effect
│
│ The following provider development overrides are set in the CLI configuration:
│  - telmate/proxmox in /home/ethinx/.local/share/terraform/plugins/telmate/proxmox/2.9.12/linux_amd64
│
│ The behavior may therefore not match any released version of the provider and applying changes may cause the state to become incompatible with published releases.
╵
module.pve-devbox.data.local_file.ssh-pub-key: Reading...
module.pve-devbox.macaddress.vm-macs[1]: Refreshing state... [id=52:54:00:57:a3:9a]
module.pve-devbox.macaddress.vm-macs[0]: Refreshing state... [id=52:54:00:5f:3d:2d]
module.pve-devbox.data.http.github-pub-keys: Reading...
module.pve-devbox.data.local_file.ssh-pub-key: Read complete after 0s [id=7be4259aa91d2c8d1ae5d39b54635cda36a46606]
module.pve-devbox.tls_private_key.cluster-ssh-secret-key: Refreshing state... [id=6d935301e547c1fdf0a961a47dd7248579d05123]
module.pve-devbox.local_file.cloud-init-network-config[0]: Refreshing state... [id=c90328565d88f9097bc1cfec1c80c7b03efa579b]
module.pve-devbox.local_file.cloud-init-network-config[1]: Refreshing state... [id=557db2dac9d481dd760d6b6c2cad4ca014e88f32]
module.pve-devbox.data.tls_public_key.cluster-ssh-pub-key: Reading...
module.pve-devbox.data.tls_public_key.cluster-ssh-pub-key: Read complete after 0s [id=6d935301e547c1fdf0a961a47dd7248579d05123]
module.pve-devbox.data.http.github-pub-keys: Read complete after 2s [id=https://github.com/ethinx.keys]
module.pve-devbox.local_file.cloud-init-user-data[0]: Refreshing state... [id=ef9daa76fbf6c4c1c29b40cff030c345aa25cfe2]
module.pve-devbox.local_file.cloud-init-user-data[1]: Refreshing state... [id=61af338aec7bb129c52649c5982b905a4885e15e]
╷
│ Error: cannot find provided user terraform-prov@pve!terraform-prov-token on proxmox
│
│   with module.pve-devbox.provider["registry.terraform.io/telmate/proxmox"],
│   on .terraform/modules/pve-devbox/main.tf line 14, in provider "proxmox":
│   14: provider "proxmox" {
│
╵

@ethinx Thanks for the fix, I'm using it as well on my local branch and it is working great
I can confirm this fix is required for v2.9.12 when using pm_api_token_id / pm_api_token_secret at least on my setup

@Dr-Shadow np, feel free to let me know if any further action required.

This error can also be due to #666


kernel won't start / undefined symbol: zmq_msg_init

(my environment is Ubuntu via WSL 2 )
So far, I haven't been able to locally install the mit-scheme kernel. I've successfully installed MIT Scheme from source, ZeroMQ, and this repo, but when I run the last step of
 jupyter console --kernel mit-scheme
I get the following:
;The primitive dld-load-file, while executing the dlopen system call, received the error: /usr/local/lib/mit-scheme-x86-64/zmq-shim.so: undefined symbol: zmq_msg_init.
;To continue, call RESTART with an option number:
; (RESTART 1) => Return to read-eval-print level 1.

2 error>
End of input stream reached.

I also get this problem, Did u fix it?

I got the same problem and I fixed in this way.

Run sudo apt install pkg-config
Do this again

$ git clone https://github.com/joeltg/mit-scheme-kernel
$ cd mit-scheme-kernel
$ make
$ sudo make install

Why and what happened?
I examine the step make carefully, and found /bin/sh: 1: pkg-config: not found
echo '(generate-shim "zmq" "#include <zmq.h>")' | /usr/local/bin/mit-scheme --batch-mode
;Including zmq.cdecl... done
;Generating "zmq-shim.c"...
;Warning: Unknown type: zmq_timer_fn
;... done
;Generating "zmq-const.c"... done
;Dumping "zmq-types.bin"... done
cc -I/usr/local/lib/mit-scheme-x86-64 -Wall -fPIC `pkg-config --cflags libzmq` -o zmq-shim.o -c zmq-shim.c
/bin/sh: 1: pkg-config: not found
cc -shared -fPIC -o zmq-shim.so zmq-shim.o `pkg-config --libs libzmq`
/bin/sh: 1: pkg-config: not found
cc `pkg-config --cflags libzmq`  -o zmq-const.o -c zmq-const.c
/bin/sh: 1: pkg-config: not found
cc  -o zmq-const zmq-const.o  `pkg-config --libs libzmq`
/bin/sh: 1: pkg-config: not found
./zmq-const
echo '(sf "zmq-const")' | /usr/local/bin/mit-scheme --batch-mode
;Generating SCode for file: "zmq-const.scm" => "zmq-const.bin"... done

zmq_msg_init is a function in Zeromq. The error  "undefined symbol: zmq_msg_init" implies that zeromq may not link to the binary file successfully when compiling. Ref: https://stackoverflow.com/questions/21381993/makefile-issue-linking-zeromq-libs-on-linux.
The makefile use the flag pkg-config to help link the libzmq, see http://wiki.zeromq.org/bindings:c.
How to fix the pkg-config: not found issue, just install pkg-config : https://stackoverflow.com/questions/21381993/makefile-issue-linking-zeromq-libs-on-linux.
After fix the mit-scheme kernel launched. So please check your log carefully, here is a no error log for your reference:
echo '(generate-shim "zmq" "#include <zmq.h>")' | /usr/local/bin/mit-scheme --batch-mode
;Including zmq.cdecl... done
;Generating "zmq-shim.c"...
;Warning: Unknown type: zmq_timer_fn
;... done
;Generating "zmq-const.c"... done
;Dumping "zmq-types.bin"... done
cc -I/usr/local/lib/mit-scheme-x86-64 -Wall -fPIC `pkg-config --cflags libzmq` -o zmq-shim.o -c zmq-shim.c
cc -shared -fPIC -o zmq-shim.so zmq-shim.o `pkg-config --libs libzmq`
cc `pkg-config --cflags libzmq`  -o zmq-const.o -c zmq-const.c
cc  -o zmq-const zmq-const.o  `pkg-config --libs libzmq`
./zmq-const
echo '(sf "zmq-const")' | /usr/local/bin/mit-scheme --batch-mode
;Generating SCode for file: "zmq-const.scm" => "zmq-const.bin"... done


npm packages: do not crash if an array is given for author

I encountered a crash
lib/license_finder/packages/npm_package.rb:92:in `author_name': no implicit conversion of String into Integer (TypeError)

which was caused by author being an array instead of string or dictionary in the following package.json:
https://github.com/romulomachado/ember-cli-string-helpers/blob/master/package.json#L55-L59
This PR makes sure we do not crash on such cases.

@tomcornelis Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@tomcornelis Thank you for signing the Contributor License Agreement!

Hey @tomcornelis ! Thanks for the PR. It looks like someone else made an issue and gave an example where the author object type is checked in the calling method and then parsed out before grabbing the name itself. I think it is a cleaner approach. Could you update it to that?


How can I get efficiency metric for my huggingface local model? (llama2)

I have tested the standard usage process of HELM with the original llama-2. On the helm-server generated website, I noticed that there are no efficiency metrics recorded in the leaderboard presented by HELM.

However, I found the Observed inference runtime (s) in the Predictions section for the corresponding dataset (cnn_dailymail as shown below).

Could you please clarify how I can obtain the efficiency metrics if the former is correct? Alternatively, why is there only one count in the Predictions section where I set the --max-eval-instances=100 if the latter is correct?

In general, we no longer support computing denoised runtime. You can still used observed inference runtime though.

why is there only one count in the Predictions section

The count is from the number of "train trials" i.e. the number of trials the evaluation is run with a different selection of in context learning examples using a different random seed. This can be set using --num-train-trials and defaults to 1.  The value you see is the mean for 100 instances in your single trial.

Thanks for your helpful relpy!


Fix the strange attractors app

Addresses https://github.com/holoviz-topics/examples/issues/93

Looks good! It worked well for me at https://attractors.holoviz-demo.anaconda.com/attractors_panel with reasonable settings. I was able to get the player buttons to become unresponsive, but only after yanking the number of points way up and turning on line drawing, so I think it's good now. Thanks, @maximlt and @Azaya89 !


[BUG] useTable from @refinedev/react-table causes infinite rendering

Describe the bug
I was trying to create a list view following up the example of useTable. However, when I got to the page, the page was rendering infinitely as the following:

It seems like the tableQuery's status remains its fetchStatus as "fetching", but there is no calling to the data provider's getList client action, and the data remains undefined, so no data can be displayed.

Steps To Reproduce
My code is mostly the same with the usage example. The difference is it is built on top of Next.js's app routing.
Here is my Refine's layout:
<Refine
  routerProvider={routerProvider}
  dataProvider={DataProviderClient}
  authProvider={authProvider}
  notificationProvider={useNotificationProvider}
  resources={[
    {
       name: "users",
       list: "users",
       create: "users/create",
       edit: "users/:id/edit",
       show: "users/:id",
    },
  ]}
  options={{
    syncWithLocation: true,
    warnWhenUnsavedChanges: true,
    useNewQueryKeys: true,
  }}
>
  {children}
</Refine>

Here is my page at /src/app/users/page.tsx
"use client";

import { HStack, Table, TableContainer, Tbody, Td, Th, Thead, Tr } from "@chakra-ui/react";
import { ColumnFilter } from "@components/DataDisplay/ColumnFilter";
import { Pagination } from "@components/Pagination";
import { capitalize } from<EMAIL_ADDRESS>import { DateField, EditButton, List, ShowButton, TagField, TextField } from "@refinedev/chakra-ui";
import { useTable } from "@refinedev/react-table";
import { ColumnDef, flexRender } from "@tanstack/react-table";
import { useMemo } from "react";
import { UserProps } from "./schema/user.schema";

const LIST_PROPS_AS_TEXT = ["name", "email"];

export default function UsersPage() {
  const columns = useMemo<ColumnDef<UserProps>[]>(
    () => [
      {
        id: "_id",
        header: "ID",
        accessorKey: "_id",
        meta: {
          filterOperator: "eq",
        },
      },
      ...LIST_PROPS_AS_TEXT.map((field) => ({
        id: field,
        header: capitalize(field),
        accessorKey: field,
      })),
      {
        id: "roles",
        header: "Roles",
        accessorKey: "roles",
        cell: (props) => {
          return (
            <HStack>
              {(props.getValue() as string[]).map((role: string, index) => (
                <TagField key={index} value={role} />
              ))}
            </HStack>
          );
        },
      },
      {
        id: "createdAt",
        header: "Created At",
        accessorKey: "createdAt",
        cell: (props) => <DateField value={props.getValue() as string} format="HH:mm DD/MM/YYYY" />,
      },
      {
        id: "actions",
        header: "Actions",
        accessorKey: "_id",
        cell: (props) => {
          return (
            <HStack>
              <EditButton recordItemId={props.getValue() as string} />
              <ShowButton recordItemId={props.getValue() as string} />
            </HStack>
          );
        },
      },
    ],
    [],
  );

  const {
    getHeaderGroups,
    getRowModel,
    refineCore: { tableQuery, pageCount, current, setCurrent },
  } = useTable<UserProps>({ columns });

  console.log("🚀 ~ UsersPage ~ Render", tableQuery);
  const total = tableQuery?.data?.total ?? 0;

  return (
    <List>

Total: {total}

      <TableContainer marginTop={12}>

            {getHeaderGroups().map((headerGroup) => {
              return (

                  {headerGroup.headers.map((header) => {
                    return (

                        {!header.isPlaceholder && (
                          <HStack spacing={2}>
                            <TextField
                              value={flexRender(
                                header.column.columnDef.header,
                                header.getContext(),
                              )}
                            />
                            <ColumnFilter<UserProps> column={header.column} />
                          </HStack>
                        )}

                    );
                  })}

              );
            })}

            {getRowModel().rows.map((row) => {
              return (

                  {row.getVisibleCells().map((cell) => {
                    return (

                        {flexRender(cell.column.columnDef.cell, cell.getContext())}

                    );
                  })}

              );
            })}

      </TableContainer>
      <Pagination current={current} pageCount={pageCount} setCurrent={setCurrent} />
    </List>
  );
}

Expected behavior

I expect to have the data rendered when visiting the page.

Packages
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>Additional Context
Btw, I saw in the source code that it may pass an empty array [] to the TanStack's reactTable here.
Wondering if it has any effect on the infinite rendering of useTable. Because there was a TanStack's issue that is quite similar to it.

Hey @khoaxuantu, I couldn't reproduce the issue from the code you've provided. Could you provide a repository with a minimal repro? That would be great! Thanks.

@BatuhanW Sure! It's quite complicated to reproduce minimally, but I just have created a branch of my project here to reproduce it. I have mocked the login and the data so that we can visit the /users page and look into the issue directly.
Would you mind checking it out?
Thanks

Just checked out the codebase you've provided @khoaxuantu. I was able to reproduce the same rendering issue with the initial setup. Then it resolved when I removed the onClick prop of the prev button in the <Pagination /> component. Can you check if this works? Instead of using current - 1, I've updated it to pagination.prev, not sure if this is something related with the caches but it started working on every test I make 😅

@aliemir It doesn't work 100% correct, sadly. I have tried both current - 1 and pagination.nev in the <Pagination /> component as you said. The issue in both occurred as follows:

From the root path /, I navigated to the  /users (via navigation sidebar), I got the issue.
Then I clicked on the pagination buttons, and it works. It did call the getList action, fetch the data successfully, and stopp rendering infinitely.
Then I navigated to /, then reloaded the whole site, and clicked on the Users navigation sidebar again, the issue occurred again

Another thing that I noticed is that if I visit the /users path by entering the url directly, the page is renderred successfully.
So I don't think the problem is from the pagination component

Hey @khoaxuantu you are right! My bad, I only tested out in the same page and hot reloading fixed the issue when I made changes in the <Pagination /> 🤦 I investigated a bit more into the issue, it looks like its only related to the useTable of @refinedev/react-table, when I switch to using useTable from @refinedev/core the issue did not occur.
I've found that the issue happens when syncing sorters and filters from React Table to Refine. We did not had a check if the current states are equal or not and left with repeated calls to setFilters. This caused queries to stuck at loading without properly calling the API. As a workaround I saw that setting syncWithLocation to false resolves the issue. For a fix, we need to add equality checks for filters and sorters in their effects.
When the issue is stuck at loading, Refine's overtime interval keeps running, this was causing your console.log to run repeatedly every second. It looks like an infinite rendering issue but its really logging due to overtime.elapsedTime getting updated 😅
In my local, I've tested out using lodash/isEqual before calling setFilters in a useEffect of useTable and it seems to resolve the issue 🤔
Let us know if syncWithLocation: false workaround is working for you, then we can discuss about the fix. Also let us know if you want to work on the fix, we'll be happy to see your contribution 🙏

Hey @aliemir, I just tried setting syncWithLocation to false and it worked as expected. I think I will go with this workaround for now.
Because of my limited time, I think it's better to let you fix the issue and I hope to be able to turn on the option in the future releases soon.
Thank you very much for the support :bow:

Hey @aliemir, were you suggesting something like this?
if (!isEqual(crudFilters, filtersCore)) {
      setFilters(crudFilters);
}

Hey @Anonymous961 yeah this is what I tried, would love to hear from you if you can spare some time to work on this 🙏

Hey @aliemir, were you suggesting something like this?
if (!isEqual(crudFilters, filtersCore)) {
      setFilters(crudFilters);
}

Ye something like that!

Hey hey, I've been using refine recently and really fell in love with this amazing product. Can I work on this issue?

Hey hey, I've been using refine recently and really fell in love with this amazing product. Can I work on this issue?

This is NOT hacktoberfest, if you can fix, fix and submit PR. Open source is ruined literally....

Hey hey, I've been using refine recently and really fell in love with this amazing product. Can I work on this issue?

This is NOT hacktoberfest, if you can fix, fix and submit PR. Open source is ruined literally....

Hey, first of all, if this repo isn't up for hacktoberfest, it's fine. I want to fix this issue genuinely out of my likeness towards the product. Secondly, please do not make this assumption about anyone that participates in the hacktoberfest event. It's hurtful to see your comment as someone who is literally interested in contributing to the product. Your message is very unwelcoming and opinionated to me.
I asked to be assigned with the task just to make sure that no one else has been working on the issue because it seems to have a long discussion going on.

@qianzhong516 Anyways It was my opinion indeed. And I didn't know you are a female. I don't know since what time you are contributing to OSS, but I have never seen someone getting assigned to a project to fix an issue until unless it's an enterprise application. Generally it's the internal team/staff gets assigned to priority issues, rest are open for everyone for first come first serve.
In the other hand, what you have to loose ? Why don't you fix it and submit a PR even though someone is working. It is not fixed that there is a particular solution to an issue. If more then one PR is raised, the quality/optimized solution will be merged. So, it doesn't matter if someone is working or not, from your profile I learnt that you are passionate, want to be a full stack dev, I don't think you need to be assigned to do things. You are born to fix issues and raise PR asap.

@qianzhong516 Anyways It was my opinion indeed. And I didn't know you are a female. I don't know since what time you are contributing to OSS, but I have never seen someone getting assigned to a project to fix an issue until unless it's an enterprise application. Generally it's the internal team/staff gets assigned to priority issues, rest are open for everyone for first come first serve.
In the other hand, what you have to loose ? Why don't you fix it and submit a PR even though someone is working. It is not fixed that there is a particular solution to an issue. If more then one PR is raised, the quality/optimized solution will be merged. So, it doesn't matter if someone is working or not, from your profile I learnt that you are passionate, want to be a full stack dev, I don't think you need to be assigned to do things. You are born to fix issues and raise PR asap.

And I didn't know you are a female. I don't know since what time you are contributing to OSS,

Does any of these facts validate that you can be unwelcoming to someone who wants to be a part of an open source project? Like because the person is a female, because you don't know when they started contributing? Is it that hard for you just to say "yep, feel free to go ahead"?
Are you sure your actions here adhere to the policies in this repo? I literally have never seen someone being this agressive for no reason. If this gets allowed, I would be pretty surprised.
I thought you were just having a bad day but now it just seems you are full of disrespect for other people. Please stop right now.

@qianzhong516
If I say, "No, you shouldn't be contributing to this project" will you go away ?

@qianzhong516 reasons for NOT being welcoming:

Haven't you seen up for grabs tag for this issue ?
Haven't you seen open status ?
Haven't you seen there is NO PR regarding this issue ?

Only if you have checked these things, instead of reading policy, you would have welcomed.
Secondly, I believe your understanding is not that sharp cause you have miss understood my previous comment. Anyone with a good understanding in english would know that was not disrespecting at all. Please read that again (with translator if needed).
Thirdly, I am no one to welcome you to a project, so why my comments are bothering you ? If you are truly here for your passion, you could have proved that with a PR instead of misinterpretation, instead you are trying to gain sympathy (women ☕)

@qianzhong516 reasons for NOT being welcoming:

Haven't you seen up for grabs tag for this issue ?

Haven't you seen open status ?

Haven't you seen there is NO PR regarding this issue ?

Only if you have checked these things, instead of reading policy, you would have welcomed.
Secondly, I believe your understanding is not that sharp cause you have miss understood my previous comment. Anyone with a good understanding in english would know that was not disrespecting at all. Please read that again (with translator if needed).
Thirdly, I am no one to welcome you to a project, so why my comments are bothering you ? If you are truly here for your passion, you could have proved that with a PR instead of misinterpretation, instead you are trying to gain sympathy (women ☕)

Reported.

@qianzhong516 You can do everything but fixing the issue. LOL

@qianzhong516 reasons for NOT being welcoming:
1. Haven't you seen `up for grabs` tag for this issue ?

2. Haven't you seen `open` status ?

3. Haven't you seen there is NO PR regarding this issue ?

Only if you have checked these things, instead of reading policy, you would have welcomed.
Secondly, I believe your understanding is not that sharp cause you have miss understood my previous comment. Anyone with a good understanding in english would know that was not disrespecting at all. Please read that again (with translator if needed).
Thirdly, I am no one to welcome you to a project, so why my comments are bothering you ? If you are truly here for your passion, you could have proved that with a PR instead of misinterpretation, instead you are trying to gain sympathy (women ☕)

@chankruze

Haven't you seen our Code of Conduct?
Haven't you seen Our Standards section?

Only if you have checked these things, you would understand what you are doing is unacceptable.

Can I take this bug? this would be my first issue

Can this bug be assigned to me?

@raulmar0 feel free to start working on this issue and submit a pull request when you're ready. No need to wait for it to be assigned to you.

Thank you for consistently providing such a valuable framework.
It seems no PR has been created for this issue yet, so I plan to work on it.


player.get.nightmode and player.get.dialoglevel always return "on"

Request with new Universal node for player.get.nightmode and player.get.dialoglevel always return status "on"
Either via msg.topic or by selecting the respective command in the node itself.
Meanwhile, set.nightmode works and sets the mode correctly.
Immediate check after setting to e.g. "off", returns "on"

What node/command (topic)/state (payload)?
node: Universal | My Sonos
command: player.get.nightmode and player.get.dialoglevel
In case of a bug: Are you able to reproduce the error?
yes
Versions and Infrastructure
what system:  docker on Ubuntu 18.04
node-red-contrib-sonos-plus version: 4.1.2
Node-RED version: 1.1.3

Hi. At my Sonos Beam it shows the right status.
What player do you use?
What does the SONOS App show: on/off?

Hi, I do have a Beam as well.
When I use set commands they do work and also correctly shown in the SONOS app.
However, when I try to get a status via a get node it always return "on".
When I am using old set of nodes it returns correct value e.g. off.

Result of all 3:

You are right - that's a bug.
Please try 4.1.3 - I just published it.

@hklages Thanks, checked and it works great. I will close the issue now.


[NuGet Symbol Server] Deployment Symbol Ingestion job

Deployment of Symbol Ingestion job.

@ryuyu - It looks like this job hasn't been working in INT. Need to fix it see the exceptions in AI. Missing certificates.
traces
| where cloud_RoleName == "NuGet.Jobs.Validation.SymbolsIngester" and timestamp > ago(10m)

Dev and Int are verified.
Let the task open for Prod and remove the MVP label,


release s->s3.handshake_dgst on error path to prevent memory leak.

release s->s3.handshake_dgst on error path to prevent memory leak.

This should be unnecessary since s->s3.handshake_dgst gets freed which the SSL object is freed.

Sorry, I just notice the reply. Thank you for your review. So I directly close it.


Youtube embed is hanging, not providing response

Describe the bug
问题描述
Youtube embedded chatgptbox not working.
To Reproduce
如何复现
Steps to reproduce the behavior:

Go to youtube.com
Click on a video
Click "Ask ChatGPT embedded box
See error

Expected behavior
期望行为
Generate a response with the transcript data.
Please complete the following information):
请补全以下内容

OS: Windows
Browser: Firefox, Edge
Extension Version: [e.g. v2.4.9]

fixed in https://github.com/josStorer/chatGPTBox/commit/1fc3cd8ee072d420fc54e4ce01a6853c71c82927
After conducting some investigation, I found that OpenAI seems to have adopted different strategies for different accounts. Since my account still adheres to the traditional SSE strategy, I mistakenly assumed that everything was still normal. I have now adapted to the new WebSocket strategy and automatically adopted the appropriate strategy according to OpenAI's response.


Registration | Upload Document : Error msg always display on screen even if no doc uploaded.

Current Behavior

An error message appears on the Upload Document section even when no document is uploaded.

Expected Behavior

Error should display only if the validation for the uploaded document failed.

Steps To Reproduce

Invite the user for registration
Complete step one
Move to step 4
Notice the error message, it is always there.
If the validation for uploaded docs fails, the user will have 2 error messages.

Please add an screenshot to ensure that there is no missunderstanding. We have error below the upload document component as well as popups. I guess you are talking about the 1st scenario. Please confirm with a small screenshot and the env. which you used for testing

Please add an screenshot to ensure that there is no misunderstanding. We have error below the upload document component as well as popups. I guess you are talking about the 1st scenario. Please confirm with a small screenshot and the env. which you used for testing
Additionally: if contributing with the fix, make sure that you automatically directly assign the ticket to yourself - otherwise its hard to manage the tickets

Hey @jjeroch ,
I've added the screenshot for this issue.


Add initial JS docs

Thanks!

Thanks!

I would also be nice to add screenshots like on the other usage pages.

I would also be nice to add screenshots like on the other usage pages.

I'll look into that 👍

I'll look into that 👍

There is no installation instructions in docs. It would be great.

There is no installation instructions in docs. It would be great.


Update torch audio for new stft signature

The stft signature is being changed in https://github.com/pytorch/pytorch/pull/9497 (landing rn) to be consistent with librosa. This patch updates transforms. SPECTROGRAM so the behavior still matches the doc. (actually behavior changed because doc says pad pads on both size but the previous behavior only pads at the end). Note that the new stft support richer options, so cc @dhpollack on whether to update SPECTROGRAM.
An identical patch is also landing to facebook internal torchaudio.

merged. now that 0.4.1 is released.


Need Update

Need Update
FuseOption need a type

I think this is the error @zyi103 refers to:
ERROR in node_modules/angular-fusejs/src/fusejs.service.d.ts:3:47 - error TS2314: Generic type 'FuseOptions<T>' requires 1 type argument(s).

export interface AngularFusejsOptions extends FuseOptions {
                                              ~~~~~~~~~~~

+1

You may need to see my fork, let me know if that works for you

You may need to see my fork, let me know if that works for you https://github.com/almothafar/angular-fusejs

Greetings. Unfortunately, it did not work on version 12.0.5. I get an error that the minimum version of Angular to run is 14


iOS: Hide Alerts due to navigation issue

TL;DR
Removed service alert priority and restricted alert display to Android platform only.
What changed?

Renamed LocalPlatformTypeProvider to LocalAppPlatformProvider for clarity
Removed priority field from ServiceAlert data class
Added platform check to only show alerts on Android devices
Modified alert-related functions to handle alerts as List instead of Set
Added TODO comment explaining the iOS limitation with ComposeNavigation

How to test?

Verify alerts are only visible on Android devices
Confirm alerts still display correctly with heading and message
Navigate through journey cards and check alert functionality
Verify iOS app doesn't crash when handling alerts

Why make this change?
The service alert priority was unused and added unnecessary complexity. Additionally, iOS devices were crashing when navigating with HTML content as string arguments (see JetBrains issue CMP-7180). This change provides a temporary solution by limiting alerts to Android while maintaining app stability on iOS.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#423  👈 (View in Graphite)
#422
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Dec 4, 5:04 AM EST: A user merged this pull request with Graphite.


Update Documentation

https://github.com/tushar5526/Autotype#run-it-as-follows-if-you-are-not-familiar-with-cli-apps
This is outdated. Update accordingly.

I am creating sphinx docs for Autotype.

@tushar5526 I would like to know a bit more about the issue in order to properly update the documentation.Also , I tried to run the simulate_keyboard.py but it executed without typing anything.There weren't any errors but the script just didn't work.I followed the documentation for simulate_keyboard.py , has the method to use the script changed?

You might had to put the text to be typed inside that file and then run.
Update the gif, showing cli and gui process

Also try to find any online platform to upload demo gifs to reduce repo size.

@Mr-Sunglasses PLease assign this to me sir

@Mr-Sunglasses Assign this issue to me sir

sorry sir @RyanWalker277

@tushar5526 Hello sir, I want to work on this issue please assign this to me.

Hey @Anubhavsoni001 ! @Bharadwajshivam28 is already working on this one , you can explore other issues in the meanwhile. #3 and  #4 can be a good start.


Maintenance: Convert deprecated set-output command on Github Actions workflows

Summary
We need to convert all the calls to set-output commands to the new format: https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-output-parameter
Why is this needed?
GH is deprecating set-output (no ETA yet): https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Which area does this relate to?
Automation
Solution
N/A

There's now an ETA:

Starting 1st June 2023 workflows using save-state or set-output commands via stdout will fail with an error.

In #1335 we have removed the set-output from some of the workflows.
Based on a quick manual scan, these workflows still use it and need updating:

Label PR based on title (example run)
On new PR (example run)
On PR merge (example run)
Measure package size (example run)

The fix in the reusable_export_pr_details was applied in the Python repo and merged in: awslabs/aws-lambda-powertools-python#1957.
I have also opened a PR in the 3rd party action (flochaz/pkg-size-action#3) we use, to patch the setOutput calls made via the @actions/core sdk.
Action items:

Apply the same changes as the PR in the Python repo
Test the "Measure package size" workflow once the PR in the other repo is merged


Use multi-stage to build the Intel bootc image

The NVIDIA bootc container is using multi-stage to avoid shipping build dependencies in the final image, making it also smaller. This change implements the same build strategy for the Intel bootc image.
The builder image is the same as for NVIDIA bootc. It is currently named after NVIDIA, but should be renamed in a follow-up change. The benefit is that a single builder image is maintained for all bootc images that require out-of-tree drivers.
The number of build arguments is also reduced, since most of the information is already present in the builder image. There is only one kernel package per builder image and one image per architecture, so we can retrieve the KERNEL_VERSION and TARGET_ARCH variables by querying the RPM database. The OS information is retrieved by sourcing the /etc/os-release file.
The extraction of the RPMs doesn't require storing the files, as rpm2cpio supports streaming the file over HTTP(S). This number of commands is smaller and the downloads happened already for each build, since the download was not in a separate RUN statement.
It is not necessary to copy the source of the drivers in /usr/src\, since we don't need to keep it in the final image. The Makefiles accept a KVERSION` variable to specify the version of the kernel and resolve its path. The other benefit is to build as non-root.
The .ko files can then be copied to the final image with COPY --from=builder. The change also ensures that the firmware files are copied to the final image.
This change also adds support for EXTRA_RPM_PACKAGES.

/cc @enriquebelarte


[BUG] - problem installing packages from git into environment

OS system and architecture in which you are running QHub
qhub 0.4.2
Expected behavior
environments containing lines like this work:
- hvplot
- pip
- pip:
  - git+https://github.com/zarr-developers/numcodecs
  - git+https://github.com/observingClouds/xbitinfo

Actual behavior

How to Reproduce the problem?
name: bitinfo
channels:
- conda-forge
dependencies:
- python=3.10
- requests
- aiohttp
- intake-xarray
- julia
- pyjulia
- matplotlib-base
- numpy
- pooch
- xarray
- dask
- distributed
- ipykernel
- ipywidgets
- s3fs
- netcdf4
- zarr
- cfgrib
- prefect[version='>=1.0.0']
- graphviz
- pytest
- nb_conda_kernels
- cmcrameri
- tqdm
- asv
- sphinx!=4.4.0
- sphinxcontrib-napoleon
- sphinx-copybutton
- sphinx-book-theme
- myst-nb
- ca-certificates
- certifi
- openssl
- datashader
- geoviews
- hvplot
- pip
- pip:
  - git+https://github.com/zarr-developers/numcodecs
  - git+https://github.com/observingClouds/xbitinfo

Command output
No response
Versions and dependencies used.
No response
Compute environment
AWS
Integrations
No response
Anything else?
No response

Hi @rsignell-usgs thanks for opening this issue, regarding the problem itself this is more related to conda-store than qhub, so if you don't mind I will raise an issue to address this in there as well.
The env build is falling because conda-store has an inbuilt validator to inspect the pip/conda dependencies. And based on the Requirement.parse() specifications, the "git+https://github.com/zarr-developers/numcodecs" string does not satisfies the PEP508 specifiers.
So, in short, conda-store needs to fix the parsing for such cases, we will try to address this asap. Thanks again for rasing this :slightly_smiling_face:

Hi @peytondmurray, what is the current status on this?

@viniciusdc See this comment for a bit more info, but in short packages installed via pip from VCS must follow the PEP440 format, e.g.
# environment.yml
name: test-env
dependencies:
- hvplot
- pip
- pip:
  - numcodecs @ git+https://github.com/zarr-developers/numcodecs@main
  - xbitinfo @ git+https://github.com/observingClouds/xbitinfo@main

where the pip dependencies are specified like
<name of package to use> @ git+https://<github repo URL>@<commit hash or tag>

In the example above this should work, except the current development version of xbitinfo depends on a specific version of numcodecs. That's is an issue with those specific pip dependencies, though, not qhub or conda-store.

Yup, @peytondmurray thats the expected format we do expect in the validation. Have you tested if the above environment builds using the expected pip syntax? This would be enough to close the issue and open a doc update one.

Well, the development version of xbitinfo depends on the development version of numcodecs. Specifying both dependencies explicitly breaks pip's dependency solver:
The conflict is caused by:
    The user requested numcodecs 0.10.0a4.dev5 (from git+https://github.com/zarr-developers/numcodecs@main)
    xbitinfo 0.0.1.post13 depends on numcodecs 0.10.0a4.dev5 (from git+https://github.com/zarr-developers/numcodecs)

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

failed

CondaEnvException: Pip failed

Specifying xbitinfo alone also doesn't work, because again, pip's dependency solver seems unable to handle development versions of packages:
# environment.yaml
channels:
- conda-forge
dependencies:
- gcc
- cython
- hvplot
- pip
- pip:
  - numcodecs @ git+https://github.com/zarr-developers/numcodecs@main
  - xbitinfo @ git+https://github.com/observingClouds/xbitinfo@main
- ipykernel
name: test-environment
prefix: null

:point_up: results in :point_down:
Looking for: ['gcc', 'cython', 'hvplot', 'pip', 'ipykernel']

Preparing transaction: ...working... done
Verifying transaction: ...working... done
Executing transaction: ...working... done
Installing pip dependencies: ...working... Pip subprocess error:
  Running command git clone --filter=blob:none --quiet https://github.com/zarr-developers/numcodecs /tmp/pip-install-wemh84w2/numcodecs_04b010a4fbf0486abef69fe5d3cb72d7
  Running command git submodule update --init --recursive -q
  Running command git clone --filter=blob:none --quiet https://github.com/observingClouds/xbitinfo /tmp/pip-install-wemh84w2/xbitinfo_9515a463c4bd4ea58e063e44be88398b
  Running command git clone --filter=blob:none --quiet https://github.com/zarr-developers/numcodecs /tmp/pip-install-wemh84w2/numcodecs_fa43f2582edc4eafaac6d501bb4caac1
  Running command git submodule update --init --recursive -q
ERROR: Cannot install -r /tmp/tmpqf8b31p8/condaenv.0r805g0r.requirements.txt (line 2) and numcodecs 0.10.0a4.dev5 (from git+https://github.com/zarr-developers/numcodecs@main) because these package versions have conflicting dependencies.
ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

Inside conda-store, I tried installing a different package from source:
# environment.yaml
dependencies:
- pip
- pip:
  - tqdm @ git+https://github.com/tqdm/tqdm@master
- ipykernel
name: test-environment-conda-store

But ran into another seemingly unrelated issue: https://github.com/Quansight/conda-store/issues/299. However, installing the environment above from conda running on my desktop succeeded, so I think this is just a conda-store problem right now.


Expose mp-syndicate-to to Nunjucks context

The rationale is better explained in #38, but the gist is to expose the mp-syndicate-to field to Nunjucks' rendering context for the generated file to link to those so that it can be actually syndicated by services like https://brid.gy.
I tried ran tests to see if anything broke, but it seems there are some unrelated broken tests (mostly around HTTP status codes).
This closes #38.

Hi @vhfmag, thanks for the PR which, as you can see, I seem to have completely messed up. Sorry, I clearly have no idea what I’m doing! ☺
Anyway, I thought it made sense to take a slightly different approach. I now allow any extended property (that is, one beginning mp-*) to be be pulled into the main properties object, thus exposing their values to your custom templates. That said, I have added support for syndication targets to the default templates, and they should appear under the syndicate-to key.
As an aside, I welcome any other feedback and ideas you might have. I’ve largely been building this project for my own needs, but wish for it to be used by other people too. If there’s anything that can be added or improved to make it easier to use, please do let me know.
Cheers!
Paul


Reloading django apps is problematic

In apps.py django_jsonfield_backport is added to INSTALLED_APPS and django is forced to reload the apps. This is problematic for projects that perform non-idempotent operations in their apps' ready method. 3rd party packages should not be doing this. Can't that check just report an error asking the user to add django_jsonfield_backport to INSTALLED_APPS (and a warning that reminds them to remove it if django>=3.1)? I'm happy to make a PR for that

@lociii what do you suggest?

Sounds good to me.
However this will be a breaking change as activity stream will stop working with JSON data field when the update is installed and django_jsonfield_backport  is not added to the INSTALLED_APPS .

go ahead and make a PR so we can see the solution


IKS: Multiple-Config API rewiring

Task Description
IKS: Realm schema update, storing multiple configs and API rewiring(config/policy update, requesting IKS etc.)
Dependencies

#3802

Closing this issue as discussed with dev.


lack of up-to-date files after upgrading to tensorflow 0.11

I just upgraded tensorflow to 0.11 by using conda install -c conda-forge tensorflow , but still lack of files in anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/examples/tutorials, such as mnist_softmax.py. The current files I have in this folder are:
__init__.py
__init__.pyc
input_data.py
input_data.pyc
mnist.py
mnist.pyc

Hi @kopxiong ,
0.11 was built a month ago. If you would like the latest greatest of everything, please install nightly.
 Because I am using python 2.7 in Anaconda environment, does Anaconda is not a good choice for tensorflow? Thanks a lot!

@martinwicke Should a page about the tensorflow nightly builds show up on the first page of https://www.google.com/search?sclient=psy-ab&espv=2&biw=1440&bih=801&btnG=Search&q=tensorflow+nightly&oq=&gs_l=&pbx=1#q=tensorflow+nightly+builds?

Ideally, but since we're completely replacing our website, SEO is not
useful at this point.
On Thu, Nov 17, 2016 at 8:56 AM, Geoffrey Irving<EMAIL_ADDRESS>wrote:

@martinwicke https://github.com/martinwicke Should a page about the
tensorflow nightly builds show up on the first page of
https://www.google.com/search?sclient=psy-ab&espv=2&biw=
1440&bih=801&btnG=Search&q=tensorflow+nightly&oq=&gs_l=&
pbx=1#q=tensorflow+nightly+builds?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/5544#issuecomment-261302620,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjO_ZV4rR0iGHbSnkZ8EQsVhlxFutYdks5q_IcmgaJpZM4Kv46S
.

@girving @sherrym
I've uninstalled tensorflow and build it from nightly binaries, it is now 0.11.0 version, but still doesn't contain other examples except for mnist tutorials.
/tensorflow
----/examples
--------/__pycache__
--------/tutortials
------------/mnist
------------/__pycache__
------------/__init__.py
--------/__init__.py

That's the only example that named in the pip_package/BUILD file. I have no objection to including other things from the examples/ folder.