Auto calculated width in GridView columns doesn't take care of placeholder text in search columns

Hello,
It's my first report, so excuse me if not done in the perfect manner.
Here my code:
['attribute' => 'category',
. 'filter' =>  Html::activeDropDownList($searchModel, 'language', array_combine(Yii::$app->languageSelector->supportedLanguages,Yii::$app->languageSelector->supportedLanguages),['class'=>'form-control','prompt' => Yii::t('SLc', 'str_categories_filter_all')])
]

So I add a placeholder text, but it is larger than the rest. So I need to specify the width with this code:
'headerOptions' => ['style' => 'width:95px']

But it could be an issue with different devices, screen size...
Would be great if it could include placeholder text width in auto width calculation.
Thank you.

This is  out of the scope of the GridView - you have to take care of it by yourself by using css / javascript - everything you need is already available with headerOptions like you mentioned.

Hm, I'm confused, is GridView adding any code to set this auto width minimum?

No, I use this minimum to prevent GridView column width from being calculated under the width of the placeholder.
Because GridView DataColumn auto calculated the width of each column, but forgot to check with the placeholder.

Please explain this "Because GridView DataColumn auto calculated the width of each column".

Sorry, my English seem not enough.
When you create a GridView, you don't specify column width. DataColum will set them base on string length for each column.
So when you use another screen size, column width are automatically recalculated from space available.
Thank you.

AFAIK GridView does not specify columns width in any way. It just generates <table> and it is browser responsibility to generate it in most optimal way. If you think that it is not optimal, you can either report it as bug in browser, or create an extension that would calculate widths using JS and overwrite browser behavior.

OK, so there is definitely something I'm missing in the base core here.
I will look further at it.
Thank you.


Add test for the shipped widgets + fixes

I've noticed that there are some changes in sniplates/django.html needed (see also https://github.com/funkybob/django-sniplates/issues/36).
This initial PR adds tests for the default widgets with the correct expected output. The following commits will fix the failing test(s).

I've fixed the widgets so far - all environments are passing with tox locally.
There should be some tests added with a bound form and initial data, but I think I've covered a big deal already.

I noticed that the Travis setup doesn't use Tox, is there a particular reason? Would be nice to keep the testing config in one place as much as possible, and I'd vote for Tox for that:

it runs setup.py install, so any packacking errors turn up
easy to run locally to check if you broke any versions
easy Travis integration: https://github.com/bigjason/django-choices/blob/master/.travis.yml#L11

Would like to hear opinions :-)

@schinckel  added the tox.ini ... I never used it.
It's not hard to go either way -- add the features you suggest, or wrap around tox.

Yeah, I didn't do it yet, in case there was a though out reason and it would pollute this PR. I'll have a new PR with the Travis infrastructure wrapped around tox during this week.
I'll also add coverage tracking to the mix, since that's not set up yet :)

The same conversion was done for django-classy-settings ... so it's not a difficult process, from what I've seen.

No, exactly, it's a bit of rinse-and-repeat for reusable apps, nothing exciting.


Add cache-control support

With these PRs merged:

https://github.com/apache/incubator-opendal/pull/1806
https://github.com/apache/incubator-opendal/pull/1804
https://github.com/apache/incubator-opendal/pull/1803

Can this be closed?

@baszalmstra Thanks a lot!


Support for elegoo mars 2 pro

Hello,
This Project is what I have been looking for weeks now.
Is there a way to support elegoo printers or export to stl file for bigger support on other printers ?

Great Project!! Congratulations, and thanks for sharing!
+1 for STL Support, allowing use with third-party slicers like Chitubox

Interested in support for Elegoo Mars 3 as well, so Chitu-based printers and/or .stl export support for a more generic approach :)

Hey folks, STL support is something I considered. However, I think the existing tooling is pretty close for getting you there. If you open Gerber files with Fusion 360, you can render them into 3D models and export to STL relatively easily. Thomas Sanladerer demonstrates this technique in his video on this process.
My goal with this project was a "one click" workflow for going straight from gerber to printable file, which I have achieved for the Anycubic printers (or I at least I hope to shortly). I'm happy to consider other printers if there's demand for it.
What is the file extension used by the the Elegoo Mars printers?

Yes, I agree Andrew, STL support seems like a step away from the underlying concept. Going from a Gerber file to a 3D model to a flat/sliced file is a bit backwards.

I tried the process from Thomas to convert Gerber > SVG > Fusion STL.
But the particular file generated from KiCad somehow was garbage already converted to SVG.
After import to Fusion 360 the sizes are not correct anymore and pads are incomplete.
Also tried to use different software i could find to convert Gerber to SVG and nothing did the job right.
So if your tool could also support Elegoo printers would be very helpfull to remove the errors introduced due to rescaling etc.
Mars 2 Pro details from chitubox and attached also file example with 3D-benchy.
Format is .CTB

3DBenchy_CTB.zip

@helmet86 Does the board preview in photonic-etcher look correct for your boards? I've never tried using KiCad gerbers, maybe they have a different encoding?
Photonic-etcher uses a third-party JS library called tracespace to identify gerber layers, render board previews, and convert to SVG. I suspect tracespace handles any peculiarities of the KiCad gerber exports, but like I said, I've never tried it.

Love this idea.  And great work getting it to work on the anycubic machines. I'd also love to do this with the Elegoo machines.

What is the file extension used by the the Elegoo Mars printers?

    If there is any thing else i can do to assist or test just let me know.
I've attached an example of a simple 40mm cube and a screen shot from chitubox

_cube.zip

+1 for Elegoo

+1 for Elegoo
Support for Chitubox files would be great. I've tried the Thomas Sanladerer way and like others I found there were issues subtracting the traces to create the stencil. Your one click approach looks good.

The problem for the Saturn through Mars 4 DLP is the .ctb format, which requires requesting the SDK from Chitu Systems to break through the encryption. Elegoo has moved on to .GOO format starting the the Saturn 3/Mars 4 6k, so proper open source. The positive about opening it up to .ctb would be possible coverage for quite a few brands that were bound to the encrypted Chitu boards... Phrozen 4ks and onwards are using them. It's about 3 years worth of consumer printers.


[Style]: Enhance Hero Section Background in Homepage

Description:
The current background in the homepage's hero section appears plain black, lacking visual appeal. To improve the aesthetics, consider adding a gradient, grid, or dotted pattern to enhance the overall look and feel of the hero section.
Tasks:

Explore and decide on an appropriate background style such as gradient, grid, or dotted pattern.
Implement the chosen background style using CSS to enhance the visual appeal of the hero section.
Ensure that the new background complements the overall design and does not negatively impact readability.
Test the changes across various screen sizes to guarantee a responsive and consistent appearance.

Additional Information:

Consider color choices that align with the overall theme of the homepage.
Seek feedback from stakeholders or team members to ensure the proposed background style aligns with the project's visual identity.

Acceptance Criteria:

The hero section background in the homepage should be updated with the chosen style (gradient/grid/dotted pattern).
The new background style should enhance the overall aesthetics without sacrificing readability.
The changes should be responsive, providing a consistent appearance across different devices.

Screenshots

Record

[x] I agree to follow this project's Code of Conduct
[x] I'm a SWOC 4.0 contributor
[x] I want to work on this issue

Hi @harmeetsingh11,
Thank you for raising this issue. Your insights into enhancing the hero section background are much appreciated.
However, I kindly request your patience in assigning the task at the moment. We have some pending issues that need to be merged, and as you know, our project workflow is not yet streamlined like CI/CD. I would appreciate it if you could wait until the tasks are assigned. Don't worry I'll assign this task to you ASAP! ✌
Once again, thank you for your effort and understanding.


Using component in Repeater within a Builder

Package
filament/forms
Package Version
2.10.10
Laravel Version
9.1
Livewire Version
2.10.3
PHP Version
8.0.14
Bug description
Given the follwing:
    public static function form(Form $form): Form
    {
        return $form
            ->schema([
                TextInput::make('title'),
                Builder::make('content')
                    ->blocks([
                        Block::make('test')
                            ->schema([
                                Repeater::make('test1')
                                    ->schema([
                                        FileUpload::make('image')
                                    ])

                            ])
                    ])
            ]);
    }

When adding multiple repeater items, only the last one is capable of adding items. Repeater elements before the last one result in the following error:
Undefined array key "f5d9a73a-2b09-41c9-afc9-16af07a582f1"

I thought this was unique to FileUpload initially but seems to affect all components.
Steps to reproduce
No response
Relevant log output
No response

Hi,
I created a similar issue a few days ago that seems to have been fixed yesterday by @danharrin (#1402)
It just hasn't been released yet ;)

The fix has been released now 👍


change whse_fish.fiss_stream_sample_sites_events_sp to bcfishpass.fis…

…s_stream_sample_sites_events_sp
change bcfishpass.mean_annual_precip to bcfishpass.mean_annual_precip_load
I think that these names should be changed.  I could be wrong though - that's for sure.  Maybe be sure to test that the files are accessible in your system.  Everything was done in Rstudion server on Linux so the should be good to go (provided the changes make sense).

fixed with https://github.com/smnorris/bcfishpass/commit/af90d4b22168b4aa30c3c337cfa78f612a5af622

ok.  Thinking it is maybe because I am not through things yet but I don't see bcfishpass.mean_annual_precip ...  It's not meant to be bcfishpass.mean_annual_precip_wsd is it?


network: Include timestamps in failure output

Timestamps are critical for matching up failures with log files. When
there's a failure the output is now of the form:
.... (10:12:06 - 10:12:07) red.q:

@luise the other tests just log directly, so nothing else comes to mind with this problem. (the reason we can't log directly here is that we're running the tests in parallel).


After saving clinvar submission, cannot select variants any more

Describe the bug
The variant list on the left of the submission set editor cannot be used any more after clicking the "save" button. A workaround is to reload the browser tab.
To Reproduce
Steps to reproduce the behavior:

Go to a submission set in the clinvar export editor
Click "save"
Wait a few seconds
Go back to the submission set
Try to click on the list

Expected behavior
Variants should be selectable.
Screenshots
N/A
Additional context
The browser console says
TypeError: this.currentSubmissionSet is undefined
    get SubmissionList.vue:270
    VueJS 3
    G SubmissionList.vue:1
    VueJS 7
bundle.tracing.min.js:2:12567

Root Cause Analysis
Similar to #668, this also looks like a timing issue.
Closing as duplicate of #668.


Boolean values and Mathematical Expressions

We created information on both booleans and mathematical expressions.

You are out of bounds and trying to put your names on my Teacher files.


Renamed procs to steps in code

Renamed procs to steps in code for the issue #1288

@6543 plz review

@6543 @qwerty287  all breaking changes that are commented on PR are because of migration?
if I add the migration script then can I keep those changes? (no need to revert)

@qwerty287 @6543 migration script added & review comments resolved
plz review

I didn't get what fallback means, why it's required and what it will do
can u plz explain

Codecov Report
Base: 44.71% // Head: 44.99% // Increases project coverage by +0.27% :tada:

Coverage data is based on head (cbeeb1a) compared to base (da7d13a).
Patch coverage: 49.87% of modified lines in pull request are covered.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

diff --git a/web/components.d.ts b/web/components.d.ts
index 8a53c720..0600280b 100644
--- a/web/components.d.ts
+++ b/web/components.d.ts
@@ -75,8 +75,6 @@ declare module '@vue/runtime-core' {
PipelineItem: typeof import('./src/components/repo/pipeline/PipelineItem.vue')['default']
PipelineList: typeof import('./src/components/repo/pipeline/PipelineList.vue')['default']
PipelineLog: typeof import('./src/components/repo/pipeline/PipelineLog.vue')['default']

PipelineProcDuration: typeof import('./src/components/repo/pipeline/PipelineProcDuration.vue')['default']
PipelineProcList: typeof import('./src/components/repo/pipeline/PipelineProcList.vue')['default']
PipelineRunningIcon: typeof import('./src/components/repo/pipeline/PipelineRunningIcon.vue')['default']
PipelineStatusIcon: typeof import('./src/components/repo/pipeline/PipelineStatusIcon.vue')['default']
PipelineStepDuration: typeof import('./src/components/repo/pipeline/PipelineStepDuration.vue')['default']

:point_up:

Thank you @6543 @qwerty287 for all your support, you did a lot


Add email signup to footer

@ghuntley pointed out on Twitter that we don't have a way to sign up for email updates. Adding it back in here.

Thanks :) 🚢


[vm] Add execute_script_function and verification

Added execute_script_function entry point to the move-vm
Added verification of the script function call

Motivation

Necessary for #7580

Test Plan

New tests

I guess this one needs a rebase as it includes #7535 as well

Some unit tests are not happy, but apart from that, the changes LGTM.

/land

:exclamation: Land has been canceled due to this PR being updated with new commits. Please issue another Land command if you want to requeue this PR.

/land


⚠ ClamAV has degraded performance

In e12930a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 213 ms

Resolved: ClamAV performance has improved in 21df7db after 5 minutes.


FER_GO  and FER_DATA are too small

Reported by Jonathan S Callahan on 9 Feb 2005 01:11 UTC
FDS appends several, potentially long, paths to the FER_GO and FER_DATA
environment variables it inherits from LAS.  We have already encountered cases
where we overrun the 512 bytes assigned to FER_GO and FER_DATA.  In a typical
LAS installation this probably won't happen and can continue with packaging
LAS and announcing a beta release.  But we need a Ferret executable upgrade
for folks who encounter this problem and we need it to be available before we
make a general announcement about LAS.  I see no reason why we shouldn't have
8192 bytes allotted to the environment variables.  That should prevent this
problem from cropping up again for a while.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1169

Comment by<EMAIL_ADDRESS>on 18 Feb 2005 18:54 UTC
Increase the length of variable path_list in tm_inq_path.F


Broken command gulp package-solution --ship for web parts

Category

[ ] Question
[x] Bug
[ ] Enhancement

Expected or Desired Behavior
gulp task package-solution --ship should build a package prepared to be deployed to the prod environment.
Observed Behavior
gulp task throws an exception:

Steps to Reproduce

install web-part template by typing yo @microsoft/sharepoint
choose web-part template without react
when installation is finished type gulp build --ship
and then type gulp package-solution --ship

It doesn't matter whether property cdnBasePath in the file ./config/write-manifests.json is properly defined or not. The exception is always the same.
The workaround that I've found is to use command gulp serve --nobrowser --ship before executing gulp build --ship.
Any ideas? Am I doing something wrong?

gulp package-solution --ship depends on the gulp bundle --ship task. This dependency isn't explicit, but it is required for the package-solution task to run properly. If you don't run bundle first, you will the error that you're seeing.

Yep, that was the problem!
I've missed the gulp bundle --ship command.
It works now, thank you!

Hi waldekmastykarz ,
I am facing the same issue even after i tried gulp bundle --ship.
Thanks, Pal

@rpalanivel83 what error do you see? Can you reproduce it in a clean solution?

Hi Waldekmastykarz,
Thanks for your response. I have resolved this issue. This is due to I had included external js in config.json without “http://“.
Thanks,Pal

Hi @waldekmastykarz I have choose web-part template wtih react. Now I'm running gulp bundle --ship at that moment, I'm not able to find any additional file inside teams after that agian I run gulp package-solution --ship. So, it is throwing error. Please check it and try to resolve it.

Looking forward for your response.

Hello @waldekmastykarz The error is resolved. just I restart everything and try again now it is working fine.


Nice theme, but I encountered a problem "block in setup': You have already activated rouge 1.11.1, but your Gemfile requires rouge 2.0.7. Prepending `bundle "

I want to run your great theme, but got error. Could you help me?
here the log:
[sumarsono@mesin-fedora julia]$ jekyll serve
/home/sumarsono/.gem/ruby/2.3.0/gems/bundler-1.14.6/lib/bundler/runtime.rb:40:in `block in setup': You have already activated rouge 1.11.1, but your Gemfile requires rouge 2.0.7. Prepending `bundle exec` to your command may solve this. (Gem::LoadError)
        from /home/sumarsono/.gem/ruby/2.3.0/gems/bundler-1.14.6/lib/bundler/runtime.rb:25:in `map'
        from /home/sumarsono/.gem/ruby/2.3.0/gems/bundler-1.14.6/lib/bundler/runtime.rb:25:in `setup'
        from /home/sumarsono/.gem/ruby/2.3.0/gems/bundler-1.14.6/lib/bundler.rb:100:in `setup'
        from /home/sumarsono/.gem/ruby/2.3.0/gems/jekyll-3.4.3/lib/jekyll/plugin_manager.rb:36:in `require_from_bundler'
        from /home/sumarsono/.gem/ruby/2.3.0/gems/jekyll-3.4.3/exe/jekyll:9:in `<top (required)>'
        from /home/sumarsono/bin/jekyll:22:in `load'
        from /home/sumarsono/bin/jekyll:22:in `<main>'

thanks

Hello!
As the log file indicates you have rouge 1.11.1 installed in your system but the theme uses version 2.0.7
One way to fix this is to run bundle exec jekyll serve .
"bundle exec is a Bundler command to execute a script in the context of the current bundle (the one from your directory's Gemfile)", this will install the new gem for you and it should fix the problem.

thanks


cluster-api-provider-aws need to have default seccompprofile set.

As part of the roadmap issue https://github.com/giantswarm/roadmap/issues/259
Managed applications and core components need to have a default seccomp profile set.
Some repositories have already gotten the required changes. For team Hydra the one we'd like you ask you to do is:

cluster-api-provider-aws

Generally speaking step by step you'll need to at at least:

Set seccompprofile on RuntimeDefault in container and pod security context
Allow the use of said profiles in the PSP
Allow the use of the proper volumes in the PSP (Otherwise pods will have trouble running see: https://github.com/giantswarm/roadmap/issues/259#issuecomment-1436729770)

However, depending on the repository and nature of the component/application, this might differ.
Here's an example for step 1 and 2:
https://github.com/giantswarm/node-operator/pull/308/commits/d0c8f187b105ca1ccf019453466ea2e869afc032
Here's an example for step 3:
https://github.com/giantswarm/dns-operator-route53/pull/92/commits/e7513932827c933964b2b778063279abafa6ede6
NOTE: These are the volumes allowed by the restricted PSP as an example, your applications might not need them all, or needs others. We've seen most issues with the volumes Secrets, Projected, and ConfigMap.

done


Develop

it's been a while since there was a release. sorry! :/
let's get this out and then i'll migrate the backend to aws.

Hey, I actually found the time to do something on the app today and basically have a PR ready - nothing special just some housekeeping/minor changes Can we maybe get that in before?

sure!
On June 5, 2017 2:45:25 PM GMT+02:00, Christian Balster<EMAIL_ADDRESS>wrote:

Hey, I actually found the time to do something on the app today and
basically have a PR ready - nothing special just some
housekeeping/minor changes Can we maybe get that in before?
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/criticalmaps/criticalmaps-android/pull/188#issuecomment-306178592

This is ready to merge right?

currently testing it. but looks like it. :)


海豚调度器配合hdfs高可用

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
按照官方推荐的部署步骤安装之后，都使用默认配置，DS没办法做到开箱即用
因为hdfs是高可用的，所以我需要将hdfs-site.xml、core-site.xml，复制到DS的conf目录下，但是DS有4个server，具体复制到哪个生效就不得而知了，我都复制了进去。
所以希望以后可以更智能一点
谢谢！！！
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

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

Code of Conduct

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

Thank you for your suggestion，it will be added in version 1.1.0.

这功能是不是还没加上？


VRAM and cell numbers

Can you comment on the amount of cells that can be processed as a function of VRAM?  At 11GB, it seems like I'm running into memory limits with bigger samples.  (The tutorial ran fine).
What is the max number of cells that you have tested?

@JBreunig,
This is a great question. I admit we have been mostly testing on 32gb GPUs but I can generate some random data with known sparsity to get a feel for how far we can push different GPUs.
What is the ideal target size and sparsity for your GPU? Are there any public datasets with a similar size/sparsity?

Good examples would be here: http://mousebrain.org/downloads.html (either the aggregate loom https://storage.googleapis.com/linnarsson-lab-loom/l5_all.agg.loom or maybe you can see where you hit the limit with concatenating different subsets http://mousebrain.org/loomfiles_level_L1.html ?)
For example, I'm currently processing a dataset of 330,000 cells including all of the above and a bunch of our datasets combined with batch correction.

@JBreunig,
Wanted to provide a small update just to let you know that I have been looking into this. I think the most straightforward solution here might be to use the unified virtual memory allocator. I’ll get an example together.

Ok, thanks!  I'll close this.

@JBreunig,
I've made a modification to the notebook to enable the Unified Virtual Memory manager in RAPIDS & CuPy. Specifically, the change looks like this:
import rmm
rmm.reinitialize(
    managed_memory=True, # Allows oversubscription
    devices=0, # GPU device IDs to register. By default registers only GPU 0.
)

cp.cuda.set_allocator(rmm.rmm_cupy_allocator)

This should allow you to oversubscribe your GPU memory to use all available host memory and it will page memory onto the GPU as needed. While swapping pages can slow down the workflow, it does make it much easier to experiment and explore different datasets & sizes without having to think about available GPU memory.
For example, I was able to load the 10x 1.3M neuron dataset pretty easily onto a 32gb GPU and do all sorts of transformations to it without ever encountering an OOM.
If you get a chance, you should try this feature out and let us know if it allow you to scale higher than before. Specifically, we're also very curious to know if you still find it extremely fast. I didn't notice much hit to performance at all and I wasn't even paying attention to how much memory I was using (I'm sure it was well over 100gb by the time I was done).

I will try the 1.3M neuron dataset myself and some internal datasets at some point today.
Using this new modification looks excellent so far.  Initial results are: 54 seconds (GPU) vs. 563 seconds (CPU) on a 32 core/ 64 thread Threadripper with a 1080 Ti and 128GB ram.

Just FYI, the 1.3M neuron dataset crashes the kernel on this line:

filtered = rapids_scanpy_funcs.filter_cells(sparse_gpu_array, min_genes=min_genes_per_cell, max_genes=max_genes_per_cell)

However, my other dataset of ~330K cells gets past this step.  It doesn't seem to be a RAM/swap issue as I don't seem to be approaching the ceiling.  Perhaps it's a VRAM issue?  I'll try again later.

@JBreunig,
I meant to respond to you earlier about this. Indeed, as I was playing around with this I noticed the crash as well. It's actually a known bug in Cusparse and we're waiting for them to fix it. I played around a little bit and managed to isolate the bug to entry 1057790 in the input data. The problem is in the conversion from the CPU array to the GPU.
If you slice off the first 1M records, or vstack everything up to 1057789 and above 1057791, the filter will work. I was able to run the 1M fairly easily all the way up to the regress_out. We're also very close to merging a PR on cuML that will enable sparse inputs for PCA (and doesn't require conversion to dense for the mean centering).

Just FYI, I'm consistently crashing the kernel here with your new notebook and rapids_scanpy_funcs.py file:

%%time
sc.pp.highly_variable_genes(adata, n_top_genes=n_top_genes, flavor="cell_ranger")
adata = adata[:, adata.var.highly_variable]

Any suggestions?

@JBreunig,
What is the shape of adata passed into the highly_variable_genes? Is it giving you any type of error at all before the kernel crashes?

(989838, 23781) right after this step (adding a line):

%%time
adata = anndata.AnnData(sparse_gpu_array.get())
adata.var_names = genes.to_pandas()
adata.shape

and there is no python error except for the kernal crash and a system request to send a report.
Let me try shaving off cells to see if it's a memory issue.

@JBreunig,
I believe we might have hit a similar issue today where our Jupyter kernel crashed without giving any type of useful error information. I’m pretty sure it’s because we were running on a system that didn’t have enough main memory.
While the benefit to using the managed memory option is the ability to oversubscribe the GPU memory, it does now increase the requirement on the amount of main memory needed.
Many of the CPU examples of the 1.3M cells dataset indicate a requirement of at least 30gb of main memory to do the processing end to end. I think you can get away with a smaller gpu and managed memory, but this comes at the expense of needing more main memory.

I can run it fine with 300K cells (266 seconds) but somewhere a little above that number it fails to work.  At 500K it gets stuck, never finishing but at 700K or above it seems to crash the kernel.  As I mentioned, I have 128 GB of RAM and 628GB set aside for Swap but it doesn't appear to get near that limit--especially with 500K.
We are ordering workstations with 256 GB of RAM and hopefully, I'll add more VRAM with the next gen of video cards.

Just FYI, I've upgraded to 256 GB of RAM and completely reinstalled drivers and CUDA (from 10.1 to 10.2) and still have problems with the code getting "stuck" perpetually in the regression or scaling steps (no kernel crashes lately but a few CPU cores are continually engaged by Python but no progression to completion).
This only happens above 350K cells.
Is there any way to troubleshoot this?

Unfortunately, I don't have any 2080 Ti's available to try and reproduce your problem on my end and the 1M cell notebook appears to be working w/ the T4 instances in AWS, which rules out the problem being exclusive to the Turing architecture. This behavior does sound very strange, though.

Is there any way to troubleshoot this?

Unfortunately, a lot of times when errors are printed, they end up displaying in the command-line that's running the Jupyter notebook and not in the notebook itself. Do you see any errors on the command-line?
You can set verbose=True in the call to rapids_scanpy_funcs.regress_out, which will print something after every 500 cells are processed. If that's not enough, you can add more prints to the regress_out and scale functions in rapids_scanpy_funcs.
If you have a command-line available, can you also run the nvidia-smi command? That should at least help us determine if the GPU is being actively utilized when the code gets stuck.

Sorry for the delay...coming back to this it seems like it's now hanging at

%%time
sparse_gpu_array, genes = rapids_scanpy_funcs.filter_genes(sparse_gpu_array, genes, min_cells=1)

I'm guessing this is related to: #53


Revise CLINT implmentenation

Changes the CLINT to use lowRISC's reggen tool.
Update UART device tree settings.

Yes, I think that would make sense. For the initial bring-up, I've moved it from the IP's directory mainly because we would have ended up with a lot of diff :-) The CVA6 repo is also re-organizing itself at the moment but I guess we should move it to the dedicated clint repo that we have :-)

I guess we should move it to the dedicated clint repo that we have :-)

I was wondering what to do with this convenient regbus CLINT. Well, don't mind if I do just that :+1:


Add social shares to the single node page

https://user-images.githubusercontent.com/54587893/172266873-8b8661bf-7c24-4158-8a10-8f016c29b28c.mov


Added the toStringList() function PG11

This function is inspired by the "tostringlist" function in OpenCypher.https://neo4j.com/docs/cypher-manual/current/functions/list/#functions-tostringlist
toStringList() converts a list of values and returns a list of string values. If any values are not convertible to string they will be null in the list returned.
A list containing the converted elements; depending on the input value a converted value is either a string value or null.
Also added the regression tests

Looks good.


JXplorer: init at <IP_ADDRESS>

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I did add myself as a maintainer in a previous pull request: https://github.com/NixOS/nixpkgs/pull/209776.
Should I add myself again to this pull request or create a separate pull request to add myself as a maintainer?

Considering that PR was never merged, you should add it in this PR :P
It does need to be a separate commit, though. maintainers: add benwbooth should be the commit message.

Done. Thanks!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2200

@ofborg build jxplorer


UI: Property Manager - Update Stepper

Issue:

https://github.com/bcgov/entity/issues/23747

Description of changes:

[x] Users will see 5 steps in the stepper - "Property Manager Information", "Host Information", "Property Details", "Principal Residence", and "Review and Confirm".
[x] "Property Manager Information" will be highlighted in the stepper.
[x] Create placeholder for the Step
[x] Update unit test

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-195-k2fjgmg4.web.app

/gcbrun

/gcbrun

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-195-k2fjgmg4.web.app

/gcbrun

Temporary Url for review: https://strr-ui-dev--pr-195-k2fjgmg4.web.app


feat: implement KongLicense controller and inject to translator

What this PR does / why we need it:

The PR implements a KongLicense controller which picks the newest KongLicense in the cluster and provides a GetLicense method for translator to inject Kong license into Kong configurations.
The controller is enabled when --enable-controller-kong-license is set to true (by default) and sync license with Konnect disabled. When sync license with Konnect is enabled, the licenses from Konnect takes the precedence.
Which issue this PR fixes:

Major part of #5437.
Special notes for your reviewer:

Not fully finished yet, but can reconcile KongLicense resources and ready to receive some reviews. Major remaining part:

When no KongLicense CRD in cluster, the controller will exit abnormally
Maybe add validation of KongLicense?
Disable applying to Kong gateway when it is OSS (in another issue/PR?)

PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

BTW it might make sense to not include integration tests and translator code in this PR to not make it bigger as it's already 2k+ lines. :)


Create an exit function for stopping the prompt and exiting

An exit function is required for stopping the prompt and exiting , it is time that use of CTRL-C for doing this should be depreciated

commit 4291da334f7360bf3ceda9907148236978a9ee9 closes this issue. :smile_cat:


Change database schema to accommodate different types of severity indicators

Repaste   from gitter :

@sbs2001
Mar 04 14:53
Btw, I wanted to start discussion regarding storage of severity indicators: The problem is different advisories provide severity scores in different formats, for example cvss v1,cvss v2,cvss v3,cvss v3.1 or just textual description(High,Low,Medium). I can think of 2 approaches:

Make separate columns for each type of severity indicator for model Vulnerability.
OR

cvss scores of different versions can be interconverted by manipulating the attack vector, this adds some error (upto 5%). This way we will have 2 columns to indicate severity one for cvss score of the latest standard and other for textual representation of severity.
What are your opinions?

Another approach I can think of is having 2 columns for cvss's , 1st column will store the version of cvss and 2nd column will store the score

What about having something  tad different?
I do not want us to be limited to CVSS which is a rather complex system and we know tat there can be several interpretation of them for the same vulnerability.
IMHO we should have instead three data points for each score:

like for a reference, some source URL of sorts where that score was published
a type of score (cvss, etc)
the score value itself.

And since there can be many of these, we make that a model of its own.
This is a tad heavier but this is the correct model IMHO.
It could be toned down to a single JSON field, but that does not help much with queries

A third an likely cleaner approach is based on the facts that we know that:

there are multiple scoring mechanisms aka. "scoring type" (some formal cvssversion, or informal high/medium/low)
for a "scoring type" there can be different interpretations from different "sources" leading to different "score"

So IMHO you could get these "type, source, score" in a Score table that has has a foreign key to a Vulnerability much like a Reference of sorts.
Single JSON columns get messy quickly and are best avoided, and limiting ourselves to the CVSS scores is not enough IMHO.

@pombredanne  I think the new  VulnerabilityReference  from https://github.com/nexB/vulnerablecode/issues/206  takes care of    this. Apart from different versions of cvss and sometimes indicators like MEDIUM , there seems no other scoring system, so IMHO having the severity indicators inside VulnerabilityReference seems more  appropriate, since it will contain all things you mentioned, so we won't lose on context.

from https://github.com/nexB/vulnerablecode/issues/18#issuecomment-590652923

We can also interconvert between cvss v2,cvss v3,cvss v1 . We need to do this since not all advisories use same cvss version. https://security.stackexchange.com/questions/127335/how-to-convert-risk-scores-cvssv1-cvssv2-cvssv3-owasp-risk-severity shows how to do this .

Here's the repaste of @pombredanne  's notes  of the chat we had recently about this issue
Vulnerability: CVE-2020-12100

References
- NVD ref1 URL:https://nvd.nist.gov/vuln/detail/CVE-2020-12100
- Mitre ref2 URL:https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2020-12100
- Gentoo ref3 URL:https://security.gentoo.org/glsa/202009-02
- OpenWall ref4 URL:https://www.openwall.com/lists/oss-security/2020/08/12/1
- RedHat ref5 URL:https://access.redhat.com/security/cve/cve-2020-12100

Score1
- For Vulnerability: CVE-2020-12100
- Type: CVSS2
- Value: base:5.2 vector: XXXXXXX/YYYY/ZZZZ
- From Reference: NVD ref1

Score2
- For Vulnerability: CVE-2020-12100
- Type: CVSS3
- Value: base:7.5 vector: XXXXXXX/YYYY/ZZZZ
- From Reference: NVD ref1

Score3
- For Vulnerability: CVE-2020-12100
- Type: CVSS3
- Value: base:7.5 vector: XXXXXXX/YYYY/ZZZZ
- From Reference: RedHat ref5

Score4
- For Vulnerability: CVE-2020-12100
- Type: Redhat-scale
- Value: Important Impact
- From Reference: RedHat ref5

Score5
- For Vulnerability: CVE-2020-12100
- Type: Gentoo-scale
- Value: normal
- From Reference: Gentoo ref3

Score6
- For Vulnerability: CVE-2020-12100
- Type: CVSS3
- Value: base:7.5 vector: XXXXXXX/YYYY/ZZZZ
- From Reference: OpenWall ref4

In the future we may get
Score7
- For Vulnerability: CVE-2020-12100
- Type: VCODE
- Value: 0-High
- From Reference: Vulnerable Code refYYYY

Say we first collect the JSON from NVD for CVE-2020-12100
1. we create a vuln
2. we create several refs including one for the NDV record itself
3. we relate the vuln to packages. We may create some packages
4. we create two scores: Score1 and Score2 that we relate to the current  NDV record ref

Later:
1. we collect Gentoo vulnerabilities and stumble on Gentoo ref3 URL:https://security.gentoo.org/glsa/202009-02
2. we try to create a new ref related to the existing vuln... (but in fact we find it exists because it was created above)
3. we create a score for Score5 and relate it to this ref

Still later:
1. A VulnerableCode contributor provides another score: High
For this we would have some data entered and stored in some models TBD.
This record and its URL becomes a reference
2. we create a new score pointing the vuln and that reference (that happens to be stored in our DB somehwere)

How do we use the scores?

1. used for filtering
- show me vulnerabilities above some score
- show me packages with vulnerabilities above some score

2. used for display
- give me all the scores for a package vulnerability

Typical workflow leading with packages:
1. detect packages in use --> get some PURL
2. lookup in VC for these packages and possibly filter filter on score

Or with vulnerability:
1. Filter vulnerabilities above some score (possibly new ones)
2. list all packages that are affected --> get some PURL
3. Search is these PURLs are in use anywhere

Some issues:
- filtering has to be for a type?
- what if we have more than of the same score type with different values?
- Could we have our own score average? based on what model?
- Our we just provide our own new scale?
- When querying by package the scores of that package type should be more prominent?


Deep-link related objects

UUID 9c4e33fa-f1e7-471c-a3d4-dd095e77e278
Links should not go into the instance, but switch the UUID of the component.

Done.


When resolving a non-get request that results in a 200, recommend the user to use content-location

Context
When posting on a form enabled with data-sg-visit, the url changes the the received response's URL. For example, if you're on posts/new and the form does a POST to /posts to create a new post, its possible for the response to respond with a 200 because we may render errors. For example, in the controller:
def create
  @post = Post.new
  if @post.save
    redirect_to post_path(@post)
  else
   # render errors
    render :new
end

In this case the URL will change to /post, but will show the render of /posts/new which is incorrect. This happens with normal rails html too.
Superglue has a solution for this. If you add content-location to the headers, superglue will use that as the actual url. For example:
def create
  @post = Post.new
  if @post.save
    redirect_to post_path(@post)
  else
+  response.set_header("content-location", new_post_path)
    render :new
end

this will tell Superglue.js to set the url to posts/new correctly when it receives the response from Rails.
issue
I'd like to be able to give the developer some console.info that they should use the above methodology when using data-sg-visit or visit.
Maybe here:
https://github.com/thoughtbot/superglue/blob/aaba7da87ab26c1b3db8593e1f79fdaddb10fc5e/superglue/lib/action_creators/requests.js#L96
We can detect the request method (non-Get), the response code, if it was a 200 (not redirected), check the headers for content-location, if its not there we console.info the following copy:
We recommend setting a content-location when using data-sg-visit or visit with a POST, PUT, or PATCH

Closing this as its no longer needed with this PR https://github.com/thoughtbot/superglue/pull/73


[Bug]: Polybar border disappears when compositor is enabled

Checklist

[X] I have read the appropriate section in the contributing guidelines
[X] I believe this issue is a problem with polybar itself and not a misconfiguration on my part
[X] I have searched for other open and closed issues that may have already reported this problem
[X] I have checked the known issues page for this problem.
[X] I have followed the debugging guide to narrow down the problem to a minimal config.

Steps to reproduce
Run polybar bar. You will see the border, as expected. However, when enabling a compositor (tested with compton and picom), this border disappears.
Minimal config
[colors]
background = #252737
background-alt = #383b54
foreground = #dfdfdf
foreground-alt = #555
primary = #ffb52a
secondary = #e60053
alert = #bd2c40

[bar/bar]
width = 100%
height = 27
radius = 6.0

background = ${colors.background}
foreground = ${colors.foreground}

border-size = 4
border-color = #00000000

font-0 = fixed:pixelsize=10;2

override-redirect = true
modules-left = text

[module/text]
type = custom/text
content = "foo"

Polybar log
No response
Expected behavior
Border not changing when running compositor.
Actual behavior
Border vanishes as soon as compositor is run.
Window Manager and Version
bspwm 0.9.10
Linux Distribution
Ubuntu 22.04.1
Polybar version
polybar 3.5.7

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags:  -Wall -Wextra -Wpedantic -O3 -DNDEBUG
Linker flags:

Additional Context / Screenshots
No response

This is expected. The border you defined is completely transparent, but if you don't have any type of transparency support enabled (compositor or pseudo-transparency), it cannot be rendered as transparent and is instead rendered as black.

Why is my border transparent if I specified color to be black?

The color is #00000000, which is black, but fully transparent (the first two hex digits are the alpha channel). Opaque black would be #ff000000 or #000000.

Thank you, I didn't notice it!


Include IPA with releases?

Hi, thank for creating this app, its possibly the best streaming application available for iOS.
Would it be possible to include an IPA with your future releases on github for those who would like to sideload? Thank you.

Do you know how to do it?

I'm not sure, not being an apple developer, but the links below should help:
https://github.com/MrKai77/Export-unsigned-ipa-files
https://forums.developer.apple.com/forums/thread/109122

Will postpone this.

I made a script that does this very easily. Simply place the script in the project directory and run it.
https://github.com/therysin/UnsignedIPAGen


License information missing

May I ask what is the license of GenIE codes and also data?

Sure! GenIE is licensed under the terms of the MIT License – I just added the information to the repository. Regarding the data, it has been adapted from prior work. So while the additions fall under the MIT licence, the original data might not, so please verify that your application complies with the other licences. You can find the respective references in the paper.
Hope this helps!


Proposal: Bazel Build Tool

Currently we are using go test. It works well and does caching too. In my team we recently tried bazel and noticed it's so fast and does caching and supports golang along with few other languages. The tests literally run in few seconds - by processing only files that have changed and their related files and using the cache for everything else. I would love to show a proof of concept for the same. We used https://github.com/bazelbuild/bazel-gazelle to generate necessary files and then that's it! It wasn't very tough after the setup. Let me know what you folks feel about it!

@karuppiah7890 Can you do a proposal outlining the details in this issue?

Sure @hickeyma . I'll incorporate Bazel into helm with minimal effort and show the details. I haven't tried Bazel a lot. But I have seen it in action and it's pretty fast. Want to highlight that with facts :)

To add some more context... Go 1.13 can cache test results as well. It will only run what has changed. In CI, we recently removed caching (which is needed to know the test results for things that have not changed). The reason the caching was removed is that the CircleCI caching system needs to store and restore the cache. The time to cache/restore was approximately equal to the time to run everything from scratch.
I am likely overthinking this but... If I understand it right, the modules pulled in would be coming from Google Cloud (where the default Go Proxy is at) and circle looks to be hosted out of AWS so the module ingress to CircleCI would be free while caching would cost them. I could be wrong as this is my rough look at it and would be open to correction.
I just ran go test ./... locally for a case with no changes. It took 2 seconds because all the results were cached. The machine I ran the tests on it working fairly hard at the moment, too.
How would bazel provide an improvement?

@mattfarina Hmm, makes sense. I'll still do the spike and see the results.
Circle ci storing and restoring cache thing - I hear bazel can take care of it's cache, given a storage service (like AWS S3, GCS). But yeah, for that we need to think about paying for the service.
Is it okay to still go ahead and demo Bazel? Will surely compare it with go test 😄
And yes, I tried running unit tests in my local too just now, after the first two, three runs, it was very fast, like 2-3 seconds.

I'm averse to introducing new tooling for the sake of using new tooling. If there's a clear benefit to using Bazel for local development over what Go provides, then we could consider it.
Feel free to experiment with Bazel, but unless there's some clear benefit for every developer over the existing tooling, then I don't see it becoming part of Helm's standard dev tooling.
Similar arguments were made when it was asked if we could "docker-ize" the dev build in #4172. We saw a significant increase in build times so we had to drop it from Helm 3 as more developers were frustrated with the slow build times compared to the portability gains.

I use bazel quite a bit but I think it raises a significant barrier to entry for new contributors as well as nontrivial cost of ownership for maintainers. For a project like helm that is a single binary written in a single language, the benefits are also typically minimal (as mentioned above go tooling already does a pretty good job with caching and change detection).
The main benefit of using bazel to build helm might be that external workspaces could import helm's go_binary target rather than writing their own main package and using helm3 as a library. I'm currently using the library approach in a separate bazel project without issue.

Thanks for the inputs. I agree to all of it. Like @bacongobbler mentioned, I'll experiment it, along with my friends and see if there are any clear benefits.
But yeah, it's a bit of a barrier for new contributors - especially if they have to do a lot. I hope it'll only be installing Bazel and generating Bazel build files and running tests and build only. Let's see

Considering we felt it will increase the barrier for new folks, I'm going to close this issues 👍 Just me trying to reduce the number of issues by removing some stale issues 😛  with enough inputs to say the feature is not needed


⚠ Bofrost.be has degraded performance

In 94f6361, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1155 ms

Resolved: Bofrost.be performance has improved in 6a35ac8 after 17 minutes.


Sidebar Fixed

Fixes Issue

Changes proposed

Screenshots

Note to reviewers

Code of Conduct

[ ] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

You can also join our Discord community.
Feel free to check out other cool repositories of the Thecyberworld.
Join the Thecyberworld GitHub Organisation by raising an issue (you will be sent an invitation).

Hey @sanketshevare , thanks for creating pr on issue no. #290
The closing drop is working
Selecting a nested list, won't close the sidebar itself
When we click on the nested item, it should close the whole sidebar, example set to the previous state

you also can restructure the sidebar

https://user-images.githubusercontent.com/69853783/205859605-bb2ae085-bb75-4ac8-a181-3c48fc630255.mp4
Like this?


priority in done task

Hi benjamonnguyen,
thanks for the great plugin (again), I really appreciate the thoughtful layout and the simplicity of the implementation!
And thanks also for fixing the update/install issue so swiftly!
I am wondering about how priority appears in a done task: when I create a task with priority, say
(C) 2024-09-01 test task
and then mark it complete using the plugin, I get
x (C) 2024-09-01 2024-09-01 test task
The retained priority at second position seems to make the item as open/not completed in other apps: e.g., SwiftDo doesn't show the task as complete, todd shows it as complete but fails at removing the completed mark.
Perhaps better to drop the priority on completion?

Could you do a test for me and create a line in your todotxt file x 2024-09-01 (C) 2024-09-01 test task and see if it renders correctly and is able to be uncompleted in those 2 apps?

Thanks for getting back to me so swiftly!
I did a few experiments with apps that are available to me:

obsidian: your plugin
iOS: SwiftDo & 2do.txt
MacOS/linux (python): todd & todo-txt-tui

The behaviour of how completed tasks deal with priority seems fairly inconsistent, at least, I wasn't able to detect any commonly compatible format/arrangement of priority/completion/creation dates.  Here is the test file:
(B) 2024-08-31 test prio-created
2024-08-31 (B) test created-prio
x (B) 2024-09-02 2024-08-31 test prio-done-created
x 2024-09-02 (B) 2024-08-31 test done-prio-created
x 2024-09-02 2024-08-31 (B) test done-created-prio

With each app, I tested how the file was displayed and then how undoing tasks (if possible) in the respective app reflected (for SwiftDo, I also checked how your plugin shows the undone tasks) - see screenshots below, the image titles (visible on hover) read .
Hope that helps a bit...

Obsidian codeblock plugin (on iOS): for reflection of how tasks undone in SwiftDo (below) mirror in the plugin

SwiftDo (iOS): this app reckognizes 2/3 tasks as completed, but restores only one correctly with priority

2do.txt (iOS): this app shows 3 tasks done, but only one restores correctly with priority and date, two restored show the creation date correctly

todd (python, on MacOS/linux): this shows 3 tasks done, two correct with completion date, two restore properly (one with correct priority)

todo-txt-tui (python, on MacOS/linux): this app seems to be pretty agnostic of the arrangement of priority/completion/creation dates, however, completion of the undone tasks yields consistently the same format as your plugin (if I recall correctly)

Wow! Thanks for the detailed tests. Referencing the official specs (https://github.com/todotxt/todo.txt?tab=readme-ov-file#rule-2-the-date-of-completion-appears-directly-after-the-x-separated-by-a-space), it looks like priority is typically discarded. What are your thoughts on this?

Yeah, the official specs seem to recommend (though not demand) to use the format pri:X to preserve priority in completed tasks - to me, it seems not necessary to preserve priority:

first reason: after completion, a task no longer has any priority;
second reason: in the rare cases, where a task may get reactivated, a new priority is usually assigned anyways.

The original bash script seems to discard priority on task completion (at least, that seems to be the default).  But, of course, other people may use priorities differently, hence feel differently about preserving those.
SwiftDo and 2do.txt, for example, offer three options: discard priority, preserve it in the format (X), preserve it in the format pri:X.  Thus, I deduce, that each of these options have their followers.

https://github.com/benjamonnguyen/obsidian-todotxt-codeblocks/releases/tag/0.3.1


Add an Apple privacy info file for OpenSSL.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Fixes #23494

Should we treat it as a bug fix and add it on all branches?

Or documentation actually.

As @fwh-dc is the community maintainer of iOS platforms I kindly ask him to review this.

LGTM.
I have one minor thing to consider:
Should we copy this file to the install location of ios build targets to make it more convenient when packaging an xcframework?

There is also a discussion about this at #23262

someone with better knowledge of the config/perl would need to add it to the install process.. At least this gives a reference file.

@fwh-dc Which directory it should be installed to?  The same where the libcrypto/libssl .dylib is installed to? BTW the iOS targets by default disable shared build so there is actually no .dylib built. So I would skip installing this anywhere for now.
The iOS targets really need some fine-tuning/cleanups IMO to be more comprehensive and useful.

@fwh-dc Which directory it should be installed to? The same where the libcrypto/libssl .dylib is installed to? BTW the iOS targets by default disable shared build so there is actually no .dylib built. So I would skip installing this anywhere for now.

Good question. I think it's fine to just have the file for reference as suggested. We can always improve from there.

Doesn't this need to be added to the appropriate directory in the podspec and packaging (resource bundle) ?

Based on https://github.com/openssl/openssl/issues/23494#issuecomment-2036134844, I take it this was derived from the BoringSSL one? Keep in mind that stat appears in this list:
https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api?language=objc
We have an empty one, but we'd also removed our remaining stat call. I haven't looked too carefully at this, but it's possible you all may need either code changes or a different manifest.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

I do not see a need for us to install this.  It's useful documentation for folks building for Apple products.
This is very unlikely to stand by itself as the only things an application needs.

Based on #23494 (comment), I take it this was derived from the BoringSSL one? Keep in mind that stat appears in this list: https://developer.apple.com/documentation/bundleresources/privacy_manifest_files/describing_use_of_required_reason_api?language=objc
We have an empty one, but we'd also removed our remaining stat call. I haven't looked too carefully at this, but it's possible you all may need either code changes or a different manifest.

Yeah maybe we need to declare C617.1 Although we never use the timestamp info obtained with stat.

OTC: We are OK with adding the file. The contents need to be right. It seems we need to declare C617.

C617 is probably not right, and we might need to ask apple for a new id.

I guess I was confused, it looks right

Apple's description is unclear to me, but it seems like the best match given the documentation. For instance, how do we check certificates on iOS, and do those files belong to the app as the documentation seems to imply? It's also not clear if that code means it can be transferred off the device or not, which we don't do.

Is there someone who actually has Xcode installed who can generate this file
I dont feel comfortable trying to add this entry manually - and I dont have access to Xcode + MAC to do this..
(Submit a new PR if you want)
Basically the NSPrivacyAccessedAPITypes section needs
NSPrivacyAccessedAPIType=APICategoryFileTimeStamp
NSPrivacyAccessedAPITypeReasons=C617.1

In light of #24260 can this be closed now?

I would think so.


BugFix Fix some regression bugs

Proposed changes
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments

run buildall

run buildall

run p0

run buildall

run buildall

run buildall

run buildall

run buildall


yaml run through jenkins downstrean

Can you show the o/p of the command if you run this playbook manually as jenkins user in jenkins machine

Naresh generate ssh-key for jenkins user and using ssh-copy-id command copy the public key to host machine.


Adding the Important Text highlighter and Note taking Extension

Description
Added a Chrome Extension which will help users to take notes, highlight and keep words or important lines on the extension itself rather than using an external note taking app. My motivation behind making this project was to increase the productivity of readers and coders for taking note and noting the important keyword and lines for future usability.
Important text highlighter #37

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[X] I have made this from my own
[X] I have taken help from some online resourses
[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[X] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

I have changed the directory name to Important Text Highlighter, kindly review this.Thank you


sumo-bandplot rgb mode fails

I run command
sumo-bandplot --project B,N --mode rgb
and get the following.
Traceback (most recent call last):   File "/home/bugaboo/.local/bin/sumo-bandplot", line 8, in <module>     sys.exit(main())   File "/home/bugaboo/.local/lib/python3.10/site-packages/sumo/cli/bandplot.py", line 880, in main     bandplot(   File "/home/bugaboo/.local/lib/python3.10/site-packages/sumo/cli/bandplot.py", line 415, in bandplot     plt = plotter.get_projected_plot(   File "/home/bugaboo/.local/lib/python3.10/site-packages/sumo/plotting/__init__.py", line 58, in wrapper     return get_plot(*args, **kwargs)   File "/home/bugaboo/.local/lib/python3.10/site-packages/sumo/plotting/bs_plotter.py", line 612, in get_projected_plot     lc = colorline(   File "/home/bugaboo/.local/lib/python3.10/site-packages/sumo/plotting/__init__.py", line 215, in colorline     c = get_interpolated_colors(color1, color2, color3, w, colorspace=colorspace)   File "/home/bugaboo/.local/lib/python3.10/site-packages/sumo/plotting/__init__.py", line 285, in get_interpolated_colors     basis1 * weights[:, 0][:, None] IndexError: too many indices for array: array is 1-dimensional, but 2 were indexed
The 'stacked' mode executes fine. Same happens with 3 selections for orbitals. The previous version 2.2.5 of sumo-bandplot worked fine with the rgb mode for the same VASP calculation.
Upon fiddling with the code I found that it always happens for branch index nd=1 for me. The definition is at the line 558 of bs_plotter.py.

Interesting. I can't reproduce on the band structures in sumo/tests/data/OsO2 or tests/data/Cs2SnI6/bandstructure. Are you able to upload a vasprun.xml?

vasprun.tar.gz
Here it is. It might be because my vasp version is 5.4.1.

Thank you. This should now be fixed in the latest version of sumo (v2.3.3)

Yes, that fixed it for me. Thank you!


Investigate font cache failures

There have been two cases reported where fonts don't show in game due to a corrupted cache. We need to investigate how this can happen and hopefully prevent it (potentially writing the initial cache to a different file before performing an atomic rename).
https://github.com/ppy/osu/issues/12174#issuecomment-806304053
https://github.com/ppy/osu/issues/12257#issuecomment-812537556

I've looked into this one but can't see how this could happen. Also I checked the second report and it doesn't look like there are zero filled files in this case, although there are some zero-byte files:

These were generated quite a while back, so it may have been something we've since fixed.

So I've done something after 2021.707.0 released.
I tried replacing my own old cache (only the cache, not online.db and client.db).
My issue is fixed. The star rating number does show without a problem.
ppy/osu#12257 (comment)
However, I also tried replacing my cache with the first report's uploaded cache as well.
ppy/osu#12174 (comment)
What's interesting is the fonts works fine. But some textures of doesn't show it all.

I believe you guys fixed it in a previous build but only partially.

I'm unsure what you are even stating. Loading broken caches in place is of course going to still be broken?

Closing this for now. I believe we fixed the underlying issue via #4777.


ElixirLS error

nvim --version: NVIM v0.5.0-765-g75996a2cd
nvim-lsp version(commit hash): 1f86bfd62d0ba78b0479373661be3fa13f1f8f35
:checkhealth result - everything OK
What language server(If the problem is related to a specific language server): ElixirLS 0.6.2 3acc1fdb2ef2c1167935f7b6e74300e039200ec5
Operating system/version: linux 5.7.12

How to reproduce the problem from neovim startup

Enter directory with mix.exs (a.k.a elixir project root)
$ nvim mix.exs

Actual behaviour
Error detected while processing function LspStatus:
line    2:
E5108: Error executing lua ...m/plugged/lsp-status.nvim/lua/lsp-status/diagnostics.lua:12: attempt to index field 'diagnostic' (a nil value)
Press ENTER or type command to continue
Error detected while processing function LspStatus:
line    2:
E5108: Error executing lua ...m/plugged/lsp-status.nvim/lua/lsp-status/diagnostics.lua:12: attempt to index field 'diagnostic' (a nil value)

Expected behaviour
No errors
Additional info

Happened after upgrade from nvim-lsp to lspconfig
All elixir_ls tests successfully passed

I think you just need to update https://github.com/nvim-lua/lsp-status.nvim that includes the recent diagnostic changes. See https://github.com/nvim-lua/lsp-status.nvim/issues/19 and https://github.com/neovim/neovim/pull/12655.

@lithammer thanks, I'll try it out right now

Also ensure you have the latest Neovim nightly for good measure.

Same issue with NVIM v0.5.0-828-g0a95549d6 (latest prebuilt package avaliable in AUR)
Guess I'll have to build from master

@hissssst Is this still an issue?

Fixed in current nighly release


Removed test launch dialogue , test cannot be accessed after EndDate

Fixed Issues
#Tes launch dialogue removed
Files Changed
TestConductRepository.cs - Added condition for test link cannot be accessed after EndDate
test-settings.component.ts- removed dialogue box
Checks

[X] Naming Conventions
[X] Server side code comments (proper English, grammar and no spelling mistakes)
[X] Client side code comments (proper English, grammar and no spelling mistakes)
[X] Unused variables, methods, blank spaces and name spaces.
[X] Optimal Code and code formatting
[X] Commit History is proper, no merge is done.
[X] Commit/pull request messages should be proper to justify your feature
[ ] All test cases are executed provided by tester.

Known Issues

issue 1
issue 2

Comments
NA

This change is

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 systemjs.config.js#L5: Remove the unused function parameter "global".

SonarQube analysis reported 1 issue
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 systemjs.config.js#L5: Remove the unused function parameter "global".


there's nothing when I open http://localhost:5173/

I installed all packages with pnpm and started npm run dev but there's nothing in http://localhost:5173/

I don't understand do you want to develop the plugin? if so you just open logseq and load the plugin after installing dependencies and running pnpn dev and you it should work and make sure you use pnpm in install and run.
If you don't want to develop the plugin you just download it from the marketplace

Sorry man I misunderstood, thanks for clarifying.


Are there scenarios where it's necessary to call gpioTerminate?

I have a JavaScript module that wraps the pigpio C library. This wrapper never actually calls the gpioTerminate function. The wrapper relies on the pigpio C library to implicitly call gpioTerminate. For example the pigpio C library calls atexit to register gpioTerminate as a function to call at normal process termination. Signals will also result in the pigpio C library calling gpioTerminate.
Is the assumption that it's not necessary to explicitly call gpioTerminate to stop the library correct as it will always be called implicitly?

I wish I could find a way to clear the socket(s) down properly.  The problem is you are relying on both ends, otherwise you get the long timeout.

The only reason you probably have to call gpioTerminate is if you want to change one of the configuration parameters  settable with the gpioCfg* functions.
E.g.
The following is valid (although I can't think of any reason for wanting to do it apart from testing.
while (condition)
{
   gpioCfgFunc1(...);
   gpioCfgFunc2(...);
   gpioInitialise();
   ...
   gpioTerminate();
}

OK. Thanks for the clarification.


Indexing doesn't finish.

I have used this plugin with a lot of success in the past. I did a complete reformat of my iMac earlier this week, installing everything from scratch. Since then, the plugin won't finish indexing, even if leaving it running all day.
iMac late 2014
32GB RAM
4GHz Core i7
macOS Sierra 10.12.1
Atom 1.11.2
Installed Atom Packages:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I've got the same problem with atom 1.18.0 and the plugin 0.25.6


Builds but  "Unable to load script from assets index.android.bundle"

Went through the README and did all the steps. After installing all the dependencies I have the react-native compiling to android with npm run android.
I get back "Unable to load script from assets index.android.bundle".

Anyone know how I can go about debugging?

Had the same issue, to solve this is a nonsense..Found it a nightmare to run react native on android. Issues on every single build.


bug: VimLeavePre is causing all buffers to be marked as unloaded

Did you check the docs and existing issues?

[X] I have read the docs
[X] I have searched the existing issues

Neovim version (nvim -v)
NVIM v0.10.0-dev-2064+g2f9ee9b6c
Operating system/version
arch, ubuntu
Describe the bug
Hi!
All buffers from the session .json file are marked as loaded: false when exiting neovim by closing the terminal window instead of executing :qa!.
Is my understanding that at least 1 of the buffers from that .json file should be marked as loaded: true for everything to work correctly later on load, right? I believe that is the case because this is the root of a problem I'm having and which is kinda painful to fully replicate.
What is the severity of this bug?
tolerable (can work around it)
Steps To Reproduce
Using the provided repro, open neovim nightly with nvim --clean +'so repro.lua'.
Put at least 2 files (of any type) under the same cwd as your repro.

Open a new instance of your terminal emulator.
Open neovim with nvim --clean +'so repro.lua'
Open some files and exit neovim with :qa!
Go to the "dirsession" folder (should be on ~...resession_issue/.repro/data/nvim/dirsession/_session_file.json) and inspect the .json file, at least 1 of the buffers is marked as loaded. This is good.
Now repeat step 3 but this time exit neovim by closing your terminal emulator window (I tested this with Kitty and Konsole)
Inspect the .json file again. Now all buffers are marked as unloaded. Bad (I think)

This doesn't seems to cause that much of a problem, until you have some complicated lazy-loading on your neovim config.
My workaround is using "UILeave" event instead of "VimLeavePre":
      vim.api.nvim_create_autocmd("UILeave", {
        desc = "Save a dir-specific session when you close Neovim",
        group = vim.api.nvim_create_augroup("resession_autosave_session", { clear = true }),
        callback = function()
          -- Only save the session if nvim was started with no args
          resession.save(vim.fn.getcwd(), { dir = "dirsession", notify = false })
        end,
      })

Expected Behavior
At least 1 buffer should be marked as loaded
Directory structure
.
├── file-1.md
├── file-2.md
└── repro.lua
Repro
-- DO NOT change the paths
local root = vim.fn.fnamemodify("./.repro", ":p")
root = root:sub(-1) == "/" and root or root .. "/"

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. name
end

--------------------------------------------------------------------------------

vim.g.mapleader = " "

--------------------------------------------------------------------------------

local plugins = {
  {
    "stevearc/resession.nvim",
    event = "VeryLazy",
    config = function(_, opts)
      local resession = require("resession")
      resession.setup(opts)

      vim.api.nvim_create_autocmd("VimLeavePre", {
        desc = "Save a dir-specific session when you close Neovim",
        group = vim.api.nvim_create_augroup("resession_autosave_session", { clear = true }),
        callback = function()
          -- Only save the session if nvim was started with no args
          resession.save(vim.fn.getcwd(), { dir = "dirsession", notify = false })
        end,
      })
    end,
  },
}

--------------------------------------------------------------------------------

local lazypath = root .. "/plugins/lazy.nvim"
---@diagnostic disable-next-line: undefined-field
if not (vim.uv or vim.loop).fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "https://github.com/folke/lazy.nvim.git",
    "--branch=stable",
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)

require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

Did you check the bug with a clean config?

[X] I have confirmed that the bug reproduces with nvim -u repro.lua using the repro.lua file above.

Now I realized UILeave also don't work when exiting neovim as :qa!. What always work is just remembering to manually save the session before leaving. I also can't use the autosave option from resession because it uses VimLeavePre under the hook.

Looks like when Neovim is killed all of the buffers are marked as unloaded during VimLeavePre. I added a hack to work around this


first part of azure docs

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[x] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screenshot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65f338e304fec79eb9840b9d--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65f41543fd027e3a312466e6--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65f8090f5ee8bc3c5bde8c09--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65f809945ee8bc3ceede8b0d--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65f80c7ca4d56546206a3119--harness-developer.netlify.app


feat(helm): allow an user defined existing secret

Hey there,
We are using Trivy for a long time in our CI, but we recently decided to give a try to the Trivy Operator.
As I noted for trivy in https://github.com/aquasecurity/trivy/issues/2586, all secrets for trivy-operator must be defined as values in order to get consumed by the secret (see https://github.com/aquasecurity/trivy-operator/blob/v0.5.0/deploy/helm/templates/config.yaml#L134).
It would be nice to have a way to define a secret created outside of the chart (we're using external-secrets but it could be something else).
I will submit a PR on this shortly since I haven't seen any.
Let me know if it was on purpose, I'll close.

Hi @chen-keinan I don't know how the project works, should I ping a codeowner or reviewer in the PR to get it reviewed ?
Or are PRs assigned by someone else in the project ?

@cebidhem its ok , I review new PRs on a daily basis.  your PR is on my queue


moved ding from static to bundled assets

#goodnessSquad

Doesn't work, no gong plays. Closing now (reopen if it works).


Volcano adapts to the k8s v1.30

fix https://github.com/volcano-sh/volcano/issues/3523
The volcano api need to adapts to the k8s v1.30。
The pr of volcano adapts to the k8s v1.29 :https://github.com/volcano-sh/volcano/pull/3534

Is the go.sum file not updated?

Is the go.sum file not updated?

I have run go mod tidy just now. I think that this operation updates go.sum file. Is it right?

Is the go.sum file not updated?

I have run go mod tidy just now. I think that this operation updates go.sum file. Is it right?

Yes. And please combine these commits into one.

Is the go.sum file not updated?

I have run go mod tidy just now. I think that this operation updates go.sum file. Is it right?

Ok , I have squash the commits

This PR https://github.com/volcano-sh/apis/pull/129, which updates the Kubernetes version based on your PR, has been merged. Your PR can now be closed. Thank you very much for your contribution.

/close


mini update 2: new table of contents

This is a new table of contents for the spec, designed to
add simplicity and organization.
Existing content will later be moved into one of these sections,
or removed entirely if considered superfluous.

This PR is built on top of #158, so the diff is misleading. Please see the following for a true diff:  https://github.com/bloodorangeio/distribution-spec/compare/mini-update-1...bloodorangeio:mini-update-2

@rchincha modified based on your suggestions, please take a look when you have a min

before merging this one do we want to make a copy of spec.md over to spec-new.md? or copy spec.md to spec-v.0.1.md Just thinking out-loud.

@mikebrow please see https://github.com/opencontainers/distribution-spec/pull/160#issuecomment-650621585
Notice the target branch for this PR (reorg vs. master)

@mikebrow please see #160 (comment)
Notice the target branch for this PR (reorg vs. master)

didn't notice that before thx.


how to modify HG630v2 To enable telnet ?

the firmware could be found here http://www.mediafire.com/file/y8a3tn97xmuhb6e/HG630+Firmware.rar/file

Hi. That tool does not support this(HG630) format.


Add ddev remove --fire-bazooka

Is your feature request related to a problem? Please describe.
I always recommend ddev rm -a and if that fails docker stop $(docker ps -q) && docker rm -f $(docker ps -aq), but that's kind of complicated and dockerish for people. Why not docker rm --fire-bazooka? Easy to do, safe.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Now that we have ddev poweroff I think we should drop this more exotic attempt.


Test: refine astToExecutor to ExecutorBinders

What problem does this PR solve?
Issue Number: ref #4609
Problem Summary:
What is changed and how it works?
Separate Executor structs from astToExecutor to Binders. Rename executors to binders.
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

/run-all-tests

lgtm

/merge

/run-all-tests

/rebuild

/run-integration-tests

/run-integration-tests


Fiks install: Update packages

We may want to update packages on fiks install, as in some scenarios we were using the wrong package version.
At the same time, maybe our repo was only missing a push with fiks update and the extra update step on fiks install may not be needed.

@acmb Most of the time, I just do fiks pull and I expect everything to be working with correct versions. When I do fiks install I only want to update all the packages. If I do fiks update then I expect to upgrade all packages to the latest versions.
Packages from git are not versioned, and should always be updated to the latest version. So:

fiks pull should also do fiks install
fiks install does npm install for normal packages and npm update for git packages
fiks update does npm-check-updates for normal packages, and npm update for git packages

Does this sound reasonable?


Fix CI, silence mappend warnings

Includes #201
Pacify warnings: define mappend canonically as (<>)
CI: update caching logic, add tests: Successful run at https://github.com/andreasabel/foldl/actions/runs/6721082783/job/18266009010?pr=1

Closes #200.

#200

@Gabriella439 : PR is ready!

Thank you everybody!


Binding MVVM command

private void listView_PullProgressChanged(object sender, Comet.Controls.RefreshProgressEventArgs e)
{
if (e.PullProgress < 0.5) transform.RotationX = 0;
else transform.RotationX = 180 * ((e.PullProgress - 0.5) * 2);
        refreshText.Visibility = e.PullProgress < 1 ? Visibility.Collapsed : Visibility.Visible;
    }

this code is not working for me. No animation is occuring when i pull down the list box. A default Refresh Indicator should be added to the control. Otherwise the control is working great

Thanks for the feedback Priom. I'll take a look at this asap.
There is a default content for RefreshIndicatorContent that should be there if you don't specify your own, does it not show up for you? It should say "Pull to Refresh" and "Release to Refresh" as you pull.

It is showing up but a visual element like a refresh logo with spinning animation would be great as default refresh indicator content because implementing and animating using path geometry is not so easy for normal developers like us. Anyway thanks for your control. It works seamlessly.

I tried your code above and it seems to be working ok, the element is rotating as I'm pulling the content. Do you have a repro project you can send me that I can take a look at?
re: spinning animation, good feedback, I'll see if I can add it.

I don't have one right now. If you update the library with a default spinning animation that would be of great help.


Carousel added in home page

Issue Number
fixes #131

Describe the changes you've made
I added a small carousel containing 3 demo images in the header section of the home page.
@Abhishek-kumar09 Please have a look and tell me any changes required.
Checklist

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

That looks good, thanks!


can't run postal bundle

when i try to install the ruby dependencies it shows this problem.. what could have gone wrong .. thank you
root@localhost:~# su - postal postal bundle /opt/postal/vendor/bundle
ruby: error while loading shared libraries: libruby.so.2.4: cannot open shared object file: No such file or directory
root@localhost:~#

You should verify that ruby has been installed correctly. Can you try running which ruby to see if and where it is installed?

no problems occurred during the installation and this is the informations you requested :
root@localhost:~# which ruby
/usr/local/rvm/rubies/ruby-2.4.1/bin/ruby
root@localhost:~# rvm -v
rvm 1.29.3 (latest) by Michal Papis, Piotr Kuczynski, Wayne E. Seguin [https://rvm.io]

Ah I'm not sure how well this works with rvm, I thought this had been discussed somewhere but I can't see it.
You could try switching into the postal user (e.g. sudo -iu postal) and seeing if it can run any ruby commands to prove it works for the user.
Failing that, the only other thing which might be related is whether the version of ruby has been chosen as default.

whenever i run a ruby command on postal user it gives a warning :
root@localhost:~# sudo -iu postal
postal@localhost:~$ rvm
Warning! PATH is not properly set up, '/usr/local/rvm/gems/ruby-2.4.1/bin' is not at first place.
         Usually this is caused by shell initialization files. Search for 'PATH=...' entries.
         You can also re-add RVM to your profile by running: 'rvm get stable --auto-dotfiles'.
         To fix it temporarily in this shell session run: 'rvm use ruby-2.4.1'.
         To ignore this error add rvm_silence_path_mismatch_check_flag=1 to your ~/.rvmrc file.
Ruby enVironment Manager 1.29.3 (latest) (c) 2009-2017 Michal Papis, Piotr Kuczynski, Wayne E. Seguin

Usage:

    rvm [--debug][--trace][--nice] <command> <options>

  for example:

    rvm list                # list installed interpreters
    rvm list known          # list available interpreters
    rvm install <version>   # install ruby interpreter
    rvm use <version>       # switch to specified ruby interpreter
    rvm remove <version>    # remove ruby interpreter
    rvm get <version>       # upgrade rvm: stable, master

Available commands:

Whilst your root user is set up for rvm, I doubt the postal user is hence the error. Can you actually run ruby as postal?
Is there any reason you aren't using actual ruby?

the reason why i wasn't using ruby directly is i didn't know that rvm and  ruby aren't the same thing, now i rested my vps and reinstalled following the instruction on your wiki and now everything works smoothly .. thanks will for all the support


How do I use my users Azure AD login for my function app to access a blob storage

I'm trying to create a function app that uses the users Azure AD authentication to access a blob storage container.
Currently, I'm trying to use the Bearer token with the "https://storage.azure.com/user_impersonation" scope that is passed in the authentication header. The Token Credential implementations currently don't support working with a bearer token directly so
I tried using a custom implementation that just puts the bearer token in an Access token.
This, however, doesn't work. I'm getting either authentication errors or query parameter errors.
Is there a better way to implement this, or is there something else I'm missing? This worked for the graph and management API's so far.
I'm using "Azure.Storage.Blobs 12.4.4"

Thank you for your feedback.  Tagging and routing to the team best able to assist.

Hi,
Could you provide a snippet of how you're attempting to authenticate (e.g. with the storage managment API)?
I've tried doing something like this before? Hopefully this helps. I've tested that the bearer token works before using it here.
    public async Task UploadBlobMethod()
    {
        string blobName = "blobnamehere";
        Uri uri = new Uri("StorageAccountUriHere");

        var client = new BlobContainerClient(uri, new BearerTokenCredential());
        var blobClient = client.GetBlockBlobClient(blobName);

        var blob = await blobClient.DownloadAsync();
    }

    public class BearerTokenCredential : TokenCredential
    {
        private string _bearerToken = "<BearerToken>";

        public override ValueTask<AccessToken> GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
        {
            return new ValueTask<AccessToken>(GetToken(requestContext, cancellationToken));
        }

        public override AccessToken GetToken(TokenRequestContext requestContext, CancellationToken cancellationToken)
        {
            return new AccessToken(_bearerToken, DateTimeOffset.UtcNow.AddHours(1));
        }

Hi,
Thank you for the response. I had about the same except I was using JWT to get the expiration time but the result is the same for me. When trying to download I got errors before but I figured out what the issue was, I needed a very specific role "Storage Blob Data" reader or contributor.
But listing the blobs in a container using the "GetBlobsAsync()" and then the "AsPages()" gives me the following error.
Value for one of the query parameters specified in the request URI is invalid.
RequestId:688d6b06-f01e-00ff-264d-871ace000000
Time:2020-09-10T08:39:14.2919653Z
Status: 400 (Value for one of the query parameters specified in the request URI is invalid.)
ErrorCode: InvalidQueryParameterValue

While calling the Rest API directly at "https://{storageAccount}.blob.core.windows.net/{containerName}?restype=container&comp=list" works fine. This also worked before when I was using a connection string.
I think this might be an issue with the implementation of the library itself.
Thank you for your help, for now, I'll just call the rest API directly for listing the files.

Hi,
I'm a little confused. So were you able to use your custom token to download with using the SDK? I guess a second question is that which contributor role do you have then, Storage Blob Data Reader, Storage Blob Data Owner, or Storage Blob Data Contributor?
When you say calling the REST API for the list call works with the connection string did you also mean it works for the SDK as well?
I wonder how the Identity package is taking in the webtoken when we attempt to send the token.
I would like to see a snippet of what you are trying to attempt still. If not feel free to close this issue.

Sorry for the confusion.
I'm able to download using the SDK and do most everything, I thought the issue was with my token and that it wouldn't work because of my token implementation but it was because my account did not have the right permissions. With Storage Blob Data Reader I can download the files and with Storage Blob Data Contributor I can delete the file or edit metadata as well.
The general roles of Contributor, Owner, or Reader do not work for accessing blob storage. That is where I went wrong.
The only thing that doesn't work is listing the blobs in a container. This gives me the previously mentioned error.
Value for one of the query parameters specified in the request URI is invalid.
While if I use the same OAuth token for the rest API it does work, so the token doesn't seem to be the issue.
When using the SDK with a connection string to the blob storage, this way of listing the blobs also works.
Here is a code snippet of how I'm trying to get the blobs out of a container.
When using a token credential it throws an exception on the "blobs.AsPages()"
       var uri = new Uri($"https://{accountName}.blob.core.windows.net",UriKind.Absolute);
       var token = new OauthTokenCredential(userBearerToken);
       var blobServiceClient = new BlobServiceClient(uri, token);

        var containerClient = blobServiceClient .GetBlobContainerClient(containerName);
        var blobs = containerClient.GetBlobsAsync(BlobTraits.All, BlobStates.All, prefix: file.FileNameWithPath);
        await foreach (var page in blobs.AsPages())
        {
            foreach (var item in page.Values)
            {
                //read the blob objects
            }
        }

I thought I mention it, in case it's an issue with de SDK or there is some issue with my code I'm missing.

Hi @kees2125,
You should be able to list blob in a container with the Blob Data Contributor role.
I believe the issue you are running into is that BlobTraits.All contains Tags = 4.  The tags feature is in public preview in select regions, and in other regions using BlobTraits.All will result in Value for one of the query parameters specified in the request URI is invalid.  As a work around, I recommend using BlobTraits.CopyStatus | BlobTraits.Metadata instead.  Blob Tags will become avalible in all regions in the next few months, mitigating this issue.
Please re-open if you have further questions.
-Sean

Random note for people ending up here:
The role Storage Blob Data Contributor does not grant Microsoft.Storage/storageAccounts/blobServices/containers/blobs/tags/read, so if your code is using BlobTraits.All you will experience that GetBlob works but ListBlob returns AuthorizationPermissionMismatch.
I guess a suggestion for Microsoft is to extend Storage Blob Data Contributor with Microsoft.Storage/storageAccounts/blobServices/containers/blobs/tags/read


Chore/add ngrx eslint plugin

Description
Add ngrx eslint plugin.
Motivation
Makes it easier to follow best practices regarding ngrx.
Fixes:

Autofix lint warnings

Changes:

Add new selectors to avoid using rxjs map operator in components
Refactor app-header component because selector subscriptions caused tests to fail
Refactor published data edit component because selector subscriptions caused tests to fail

Tests included/Docs Updated?

[x] Included for each change/fix?
[x] Passing? (Merge will not be approved unless this is checked)
[ ] Docs updated?
[x] New packages used/requires npm install?
[ ] Toggle added for new features?
[ ] Requires update of catamel API?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

625 of 698 (89.54%)  changed or added relevant lines in 65 files are covered.
135 unchanged lines in 24 files lost coverage.
Overall coverage increased (+1.6%) to 74.288%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/app/app.component.ts
3
4
75.0%

src/app/datasets/archiving.service.ts
6
7
85.71%

src/app/datasets/batch-card/batch-card.component.ts
1
2
50.0%

src/app/datasets/dashboard/dashboard.component.ts
17
18
94.44%

src/app/datasets/datafiles/datafiles.component.ts
3
4
75.0%

src/app/datasets/dataset-table/dataset-table.component.ts
4
5
80.0%

src/app/datasets/reduce/reduce.component.ts
2
3
66.67%

src/app/logbooks/logbooks-dashboard/logbooks-dashboard.component.ts
7
8
87.5%

src/app/state-management/effects/policies.effects.ts
11
12
91.67%

src/app/state-management/selectors/user.selectors.ts
10
11
90.91%

Files with Coverage Reduction
New Missed Lines
%

src/app/datasets/datafiles/datafiles.component.ts
1
74.29%

src/app/shared/MockStubs.ts
1
50.0%

src/app/state-management/effects/policies.effects.ts
1
90.24%

src/app/state-management/reducers/proposals.reducer.ts
1
84.62%

src/app/state-management/reducers/samples.reducer.ts
1
86.21%

src/app/state-management/selectors/datasets.selectors.ts
1
93.4%

src/app/datasets/dataset-file-uploader/dataset-file-uploader.component.ts
2
67.5%

src/app/datasets/reduce/reduce.component.ts
2
60.61%

src/app/datasets/admin-tab/admin-tab.component.ts
3
27.5%

src/app/datasets/archiving.service.ts
3
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
1.6%

Covered Lines:
3608

Relevant Lines:
4468

💛  - Coveralls

I am afraid I can not say much to that change due to lack of expertise. I asked Carlo to jump in ;-)

@henrikjohansson712 : in general, I think the code looks good, although I do not feel comfortable to express any opinion on the functionalities as I lack the right knowledge.


:memo: Feat: Update docs

Description
Update docs to focus on

Clientside usage
Viem usage

Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

[!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

#1017  👈
#1016
main

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

Join @roninjin10 and the rest of your teammates on  Graphite