Make it easier to access contents of emitted assets

Feature request

What is the expected behavior?
Prior to futureEmitAssets being added in #8642, it was possible to easily access the final contents of all emitted assets at stats.compilation.assets.
What is motivation or use case for adding/changing the behavior?
This was useful for tracking the size of our bundled assets to avoid unintentional bundle size regressions. While size is still exposed on SizeOnlySource, it's now harder to get access to the actual contents in order to track gzip/brotli/etc. size. Reading in the final emitted files from the file system would work fine but is harder than it probably should be as various plugins like MiniCssExtractPlugin support different public paths for different asset types.
How should this be implemented in your opinion?
Including the location of the emitted asset as part of the asset stats would make things easier. Perhaps this is already available somewhere but I've been unable to find it if so.
Are you willing to work on this yourself?
Yes.

The publicPath doesn't matter here.
The key in compilation.assets is the output filename in the compilation.outputPath.

But it probably makes sense to add a Hook which allows to introspect the in-memory file content just after it was emitted to measure i. e. gzip size, etc.
You could add a hook just before this line
https://github.com/webpack/webpack/blob/75c340c3d1f0d807e0548133ccf8e70252d66cd1/lib/Compiler.js#L400
i. e. this.hook.assetEmitted.callAsync(file, content, callback)
and similar here
https://github.com/webpack/webpack/blob/75c340c3d1f0d807e0548133ccf8e70252d66cd1/lib/Compiler.js#L415

Send a PR

@sokra I would take this up :+1:

Thanks, I meant the output path and not public path. Joining with the output path from the output options did end up working for my use case, will leave the issue open to track that enhancement.


added flutter projects Ishorts_Clone app and Movie Rating App with bl…

…oc patten

@imSanjaySoni good projects.

@imSanjaySoni good projects.

Thank you 🙏


Rollup 4.x

Describe the bug
rollup-plugin-summary doesn't have rollup >= 4.0.0 as a valid peer dependency, preventing the use of the latest rollup release.
To Reproduce
Steps to reproduce the behavior:

Install latest<EMAIL_ADDRESS>Attempt to install rollup-plugin-summary

 WARN  Issues with peer dependencies found
.
└─┬ rollup-plugin-summary 2.0.0
  └── ✕ unmet peer<EMAIL_ADDRESS>found 4.0.2

Expected behavior
Being able to use rollup-plugin-summary with the latest rollup

@yousifalraheem Thanks for making this update!
It looks like the GitHub action that was meant to create the NPM release for this update failed. Are you able to get a new NPM release out for this?

@yousifalraheem bump

:tada: This issue has been resolved in version 2.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add TTL option for dht.request() and dht.ping()

The helper _requestToPromise is handling a few request options:
https://github.com/LuKks/dht-rpc/blob/c67630bf295d5de55ee328f8b6a8aba6a88ab4fb/index.js#L189
We could move it to inside of Request class, then we could simplifiy the opts part as we could pass it directly to createRequest, then just for the Promise part we can add a similar helper like toPromise()

(but this toPromise() will not touch options, just take care of the Promise part)

@mafintosh Ready!


Do not set 'display: block' on all s

Currently the theme's CSS sets display: block for all <table> elements. There are multiple reasons why not to do it (see for example https://stackoverflow.com/a/7456216), and doing that breaks other uses, like https://github.com/ansible-community/antsibull/pull/354#issuecomment-989201744. So please let's remove that faulty line :)
(That line was originally added to the theme in https://github.com/ansible/ansible/commit/373b1dcf5961f57702e9070bcedf6577bda996f4#diff-9a54ec88f2695401d17b8ea9a48d9970c13279b8c01dc902367d00c70c4e31e2R420 and has been carried around ever since then. I have no idea what the original intention was, but I don't think it ever did anything good.)

@webknjaz is there something wrong with CI here? Nothing seems to be running (except the readthedocs test).

@webknjaz is there something wrong with CI here? Nothing seems to be running (except the readthedocs test).

Sort of. Funny that I had to explain exactly this a few times recently :)
Basically, when there's no activity for 60 days, GH disables the cron workflows triggered on the schedule event. It really affects the whole workflow definition file, not just one trigger. So push and pull_request triggers become collateral damage if in the same workflow. If a workflow is disabled, no events will "wake it up". This won't happen to the workflows that don't have schedule, though (unless they are manually disabled by a human).
As a result, new PRs don't cause the workflow to be triggered and you only see the required checks listed :man_shrugging:. I've probably missed the GH email about disabling (they usually give a chance to click a button to skip turning it off).
For now, I've re-enabled the workflow. But in the future, it may be reasonable to think about really moving out the cron trigger. This doesn't affect active projects as much so people often don't realize that this mechanics exists.

Re-opened this PR to trigger CI.

@felixfontein thanks, https://github.com/ansible-community/sphinx_ansible_theme/actions/runs/1563168971 should make the release happen in a minute.

@webknjaz thanks a lot! Yeah, I'm somewhat familiar with the 60 days inactivity timeout, this regularly happens for some of my private repos :)

BTW, do the GitHub releases (https://github.com/ansible-community/sphinx_ansible_theme/releases) need to be created manually?

Yep, I didn't integrate GH releases autocreation in this repo. Will probably do that some other time.


Codecov Fix

Description
Resolves the following issue when merging code coverage reports generated in self-hosted (aml) runners.
"CoverageWarning: Couldn't parse
 '/home/azureuser/runner/work/recommenders/recommenders/recommenders/evaluation/__init__.py': No source for code"

Also add -i flag to ignore the "No source for code" error.
Checklist:

[x] I have followed the contribution guidelines and code style for this project.
[x] I have added tests covering my contributions.
[x] I have updated the documentation accordingly.
[x] This PR is being made to staging branch and not to main branch.

Codecov Report

Merging #1665 (0fa2ccf) into staging (afd0597) will decrease coverage by 34.76%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           staging    #1665       +/-   ##
============================================
- Coverage    57.97%   23.20%   -34.77%
============================================
  Files           88       88
  Lines         9096     9101        +5
============================================
- Hits          5273     2112     -3161
- Misses        3823     6989     +3166

Flag
Coverage Δ

pr-gate
23.20% <100.00%> (-34.77%)
:arrow_down:

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

Impacted Files
Coverage Δ

recommenders/models/lightfm/lightfm_utils.py
62.50% <ø> (ø)

recommenders/datasets/spark_splitters.py
87.27% <100.00%> (+1.55%)
:arrow_up:

recommenders/models/sar/sar_singlenode.py
96.85% <100.00%> (-0.02%)
:arrow_down:

...enders/models/deeprec/models/sequential/sli_rec.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ders/models/deeprec/models/sequential/nextitnet.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...commenders/models/deeprec/models/sequential/sum.py
0.00% <0.00%> (-96.50%)
:arrow_down:

recommenders/models/ncf/ncf_singlenode.py
2.29% <0.00%> (-96.19%)
:arrow_down:

recommenders/models/newsrec/models/npa.py
0.00% <0.00%> (-95.59%)
:arrow_down:

recommenders/models/wide_deep/wide_deep_utils.py
0.00% <0.00%> (-93.55%)
:arrow_down:

recommenders/models/newsrec/models/naml.py
0.00% <0.00%> (-92.44%)
:arrow_down:

... and 34 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0f1f8b3...0fa2ccf. Read the comment docs.


fx error: symbolically traced variables cannot be used as inputs to control flow

Issue Type
Bug
Source
pip (model-compression-toolkit)
MCT Version
2.2
OS Platform and Distribution
Ubuntu 20.04
Python version
3.8.10
Describe the issue
Trying to get https://github.com/HRNet/HRNet-Facial-Landmark-Detection though mct. ptq quantization step throws:
...
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/logger.py", line 117, in critical
    raise Exception(msg)
Exception: Error parsing model with torch.fx
fx error: symbolically traced variables cannot be used as inputs to control flow

Understand that its probably a problem with conditionals in the forward pass - any ideas for how to proceed?
Expected behaviour
No response
Code to reproduce the issue
see above

Log output
see above

Hi @alexander-sony,
Usually those conditionals in the forward pass are static per block and can be modified to be torch.fx compatible. Can you send the full log, I'll try to be more specific.
Thanks
Idan

Start Model Reading...
Error parsing model with torch.fx
fx error: symbolically traced variables cannot be used as inputs to control flow
Traceback (most recent call last):
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/core/pytorch/reader/reader.py", line 90, in fx_graph_module_generation
    symbolic_traced = symbolic_trace(pytorch_model)
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/torch/fx/_symbolic_trace.py", line 1070, in symbolic_trace
    graph = tracer.trace(root, concrete_args)
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/torch/fx/_symbolic_trace.py", line 739, in trace
    (self.create_arg(fn(*args)),),
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 161, in forward
    inputs, kwargs = self.scatter(inputs, kwargs, self.device_ids)
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 178, in scatter
    return scatter_kwargs(inputs, kwargs, device_ids, dim=self.dim)
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/torch/nn/parallel/scatter_gather.py", line 52, in scatter_kwargs
    inputs = scatter(inputs, target_gpus, dim) if inputs else []
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/torch/fx/proxy.py", line 298, in __bool__
    return self.tracer.to_bool(self)
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/torch/fx/proxy.py", line 174, in to_bool
    raise TraceError('symbolically traced variables cannot be used as inputs to control flow')
torch.fx.proxy.TraceError: symbolically traced variables cannot be used as inputs to control flow

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "quantize.py", line 138, in <module>
    main()
  File "quantize.py", line 132, in main
    quant_model = quantization(model, N_ITER, SHAPE)
  File "quantize.py", line 96, in quantization
    resource_utilization_data = mct.core.pytorch_resource_utilization_data(model,
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/core/pytorch/resource_utilization_data_facade.py", line 83, in pytorch_resource_utilization_data
    return compute_resource_utilization_data(in_model,
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/core/common/mixed_precision/resource_utilization_tools/resource_utilization_data.py", line 64, in compute_resource_utilization_data
    transformed_graph = graph_preparation_runner(in_model,
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/core/graph_prep_runner.py", line 72, in graph_preparation_runner
    graph = read_model_to_graph(in_model,
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/core/graph_prep_runner.py", line 207, in read_model_to_graph
    graph = fw_impl.model_reader(in_model,
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/core/pytorch/pytorch_implementation.py", line 149, in model_reader
    return model_reader(_module, representative_data_gen, self.to_numpy, self.to_tensor)
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/core/pytorch/reader/reader.py", line 153, in model_reader
    fx_model = fx_graph_module_generation(model, representative_data_gen, to_tensor)
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/core/pytorch/reader/reader.py", line 92, in fx_graph_module_generation
    Logger.critical(f'Error parsing model with torch.fx\n'
  File "/home/SONY/s1000328194/git/HRNet-Facial-Landmark-Detection/.venv/lib/python3.8/site-packages/model_compression_toolkit/logger.py", line 117, in critical
    raise Exception(msg)
Exception: Error parsing model with torch.fx
fx error: symbolically traced variables cannot be used as inputs to control flow
make: *** [Makefile:2: quant] Error 1


[Bug]: Unable to edit the name of the datasource even through demo user

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Unable to edit the name of the datasource even on logging in with through demo user.
Steps To Reproduce
View the following video for steps to reproduce.
https://ddpa7mfuvv.vmaker.com/record/b4v6ozxmqW7LuJoY
Public Sample App
No response
Issue video log
No response
Version
http://<IP_ADDRESS>/

@btsgh please tag these issues to the RBAC epic as well

@btsgh please tag these issues to the RBAC epic as well

Will do.

This issue is fixed


pageserver: optimize disk usage eviction for large total number of layers

The first phase of disk usage eviction is to enumerate all layers across all tenants, so that the layers can then be globally ordered by LRU.  This generates an O(n_layers) data structure, which will be millions of layers.  This has a memory cost, and also will generate a lot of spurious atomics from cloning args into EvictionCandidate, etc.
Basic approach: avoid using O(N_layers) memory
We can make this much more scalable by accepting an inexact ordering:

First calculate how much space we want to free
Then consider the first 10% of tenants: order their layers, and then delete layers until we have reclaimed 10% of the target
...and so on for the next 10% of tenants, etc.

That way we avoid holding the entire list of layers in memory at a time.
Sampling approach: operate in constant size memory
A more sophisticated approach would be to use statistical sampling of the layer age distribution:

Make a histogram of layer ages
Sample a modest number of layers from a modest number of tenants, e.g. 100 layers from 100 tenants each.
To free 10% of the used space, take a 10th percentile sample from the histogram: that is our age threshold for deleting layers
Iterate through tenants & layers, evicting anything older than the age threshold.

Unfair but fast approach
Avoid iterating through all tenants at all, by accepting that some tenants will "take one for the team" so that we don't have to touch all of them.
For example, to touch only half the tenants:

Start with the sampling approach
If our target is 10%, then adjust it up to 20%
Pick a random 50% subset of the tenants, and apply eviction with the 20% threshold

This will work fine if eviction is somewhat common, as each iteration we'll pick different tenants.

Only now found this issue. The options presented do not work badly with #5304 which we are now in progress of rolling out.

Re: todo I found from the code: https://github.com/neondatabase/neon/blob/12e9b2a909d2382567b79fa4b27a879f0009b5ff/pageserver/src/disk_usage_eviction_task.rs#L758-L760
We do take the LayerMap rwlock which is tokio lock for each attached timeline which will make progress per tokio's coop facilities and so yield every now and then. This is not true for secondaries. I'll add a yield per secondary tenant.


Update MLX logo

The logo needs to be update in:

UI
READMEs
screenshots
presentation materials

Find them here: https://github.com/machine-learning-exchange/artwork

Thanks @ptitzler for pointing this out!

@drewbutlerbb4 -- Please update the UI code when you get a chance. Thanks!


Disable horizontal scroll

Seen here:
https://www.tidymodels.org/start/resampling/#intro
/*---fix code scrolling-- */
pre {
  white-space: pre-wrap;
}

https://github.com/rstudio/hugo-tourmaline/pull/43/commits/2064e0c3a22789dc8176ce3aeac98aabf356d09d
This should fix it once it's merged in


Evaluation assertions

Fixes #80 #81 #93 .
This is a PR to bring evaluation into the main branch.
Things that this includes

A promptfoo assertion for running evaluation metrics
A promptfoo assertion for checking if quotes are in chunks
A promptfoo assertion to check the format of generated output against expected
Some fixes to the evaluation script
A new test config for just checking quotes against chunks (doesn't need annotated data, just chunks)

Pull request checklist

[x] I have linked my PR to an issue
[x] I’ve used conventional commits
[x] My branch is up-to-date with main
[x] Where appropriate, I have added or updated tests
[x] Where appropriate, I have updated documentation to reflect my changes

I've only done a quick pass through and raised a few minor comments. Given that this is safely in the evaluation folder, it shouldn't affect the Raphael product, so I'm happy for it to be merged in relatively quickly.

Thanks! I've responded to the comments, and I'll merge this in this afternoon after retro, probably


Any settings to reduce z-fighting?

Hi,
I have a scene with serious texture flickering while the camera moves (the objects overlap.) I already tried adjusting the near and far clipping planes, but to no avail. Is there a way to reduce it without going to the rather extreme step of eliminating overlap?
Thanks for your time.

I just tried using a 300:1 depth plane ratio.
How would one use a higher precision depth buffer in BGFX?
Also of note: I am using a rather tricky combination of a triplanar shader and a gradient mapped texture that uses a blend of a given texture's r, b, and g channels as final result + basic directional lighting. Does that change anything?

I just tried using a 300:1 depth plane ratio.
How would one use a higher precision depth buffer in BGFX?
Also of note: I am using a rather tricky combination of a triplanar shader and a gradient mapped texture that uses a blend of a given texture's r, b, and g channels as final result + basic directional lighting. Does that change anything?

Default backbuffer is D24S8, you can't change it. But you can create framebuffer with any available depth format.

Default backbuffer is D24S8, you can't change it. But you can create framebuffer with any available depth format.


Cask TrueCrypt fails to install on Yosemite

https://github.com/caskroom/homebrew-cask/blob/master/Casks/truecrypt.rb
Due to a whacky version check, the installer thinks OSX 10.10.1 is not above 10.4:
==> installer: Error - TrueCrypt requires Mac OS X 10.4 or later.

Also see:
http://tips.tinyiron.net/yosemite-to-truecrypt-never-gonna-get-it/
https://stefansundin.com/blog/656
Adding this line to the installation script might fix it:
sed -i '' 's/<installation-check .*>//' 'TrueCrypt 7.1a.mpkg/Contents/distribution.dist'

PS: I am new with brew / cask and do not feel confident enough Cask to come up with a PR...

Perhaps I'll consider building this into alehouse and provide a Yosemite-compatible fork available via caskroom/unofficial.

I’ve made some tests with the sed hack, and it didn’t solve the issue. Following @DomT4’s suggestion might be the best solution, or even installing it manually. I’ve found numerous solutions for how to do it in this manner, and since this is a piece of software that will only decline in usage and whose only reason to install it is to move your data somewhere else, spending time on trying to make it work with automation is a bit of a waste.
Either way, I understand the desire to have this at least until the next OS X release, and if someone does indeed want to take a stab at fixing it (not going to happen, I bet), the issue is referenced in the appropriate repo.

@vitorgalvao:  How is TrueCrypt dangerous to use still?  Since the original developer's strange disappearing act and unsubstantiated claim that it was unsafe, it has now gone through an official 3rd party audit, and has come away with an overall clean bill of health.  Only a couple very minor flaws were found as reported on multiple news & blog sites (e.g.: ArsTechnica, The Register, Cryptography Engineering Blog, Is TrueCrypt Audited Yet, GRC).   As mentioned the largest potential flaw found was an unlikely special case where the Windows Crypto API may fail and TrueCrypt would not issue a security warning, but overall the other flaws found are minor and the consensus is that it's still safe to use.  Perhaps the best thing that happened to TrueCrypt is that the author abandoned it... for it gave reason for people to take notice & actually give it an audit.  We now have enough conclusive evidence to reasonably determine it is safe to use to provide sufficient protection for sensitive data.

@trinitronx I don’t remember everything from this thread, and quite frankly I don’t intend to reread it, or the links you’ve posted (thank you for taking the time for doing the research, though). I still read my last comment, to see if I can get some modicum of context. I doubt there’s anything left to add. As far as I’m concerned, this issue is closed (in all senses).
Truecrypt is a broken cask whose use will only decline. There is no desire from any of the (all volunteer) maintainers in wasting (and I do mean waste) time fixing it, and I do not think any of us should. The cask still exists in an alternate repo, and anyone can create a tap for it. If you wish to fix it, submit a PR and we’ll accept it (on that repo, not the main one) and may even keep it around longer; otherwise, the plan stays the same.

Even if the Cask itself is fixed, I must add that the software is no longer actively maintained. If you need a successor to TrueCrypt, I suggest looking into VeraCrypt, which seems somewhat promising (and we should probably get a Cask for it soon).

@alebcay:  I realize it's sadly no longer maintained.  However, it is still useful software and I'm sure not alone in still wanting to use it.  I haven't played around with VeraCrypt yet and it looks promising, but I was used to using Truecrypt from homebrew and really wanted something still installable via homebrew.  So I took @vitorgalvao 's advice.
For those stubborn folks like me who still want to use the trusty old Truecrypt:
I've managed to create a homebrew Cask Tap for Truecrypt 7.1a (after working around an installer bug on OS X Yosemite)

Thumbs up to that, your tap looks great! Way better than anything we’d have here, even if we fixed it. Very informative, as this case warrants. Any future truecrypt issues shall be redirect to your tap.
Thank you.

So there is still no cask for veracrypt?

So there is still no cask for veracrypt?

Clearly there’s no demand for it. You’re free to submit it for review, as no one is working on it (or even thinking about it, I’d say).

I see, this must mean that people use another software for encryption? Or
cask users do not encrypt at all?
On Mon, Mar 7, 2016 at 10:43 PM, Vítor Galvão<EMAIL_ADDRESS>wrote:

So there is still no cask for veracrypt?
Clearly there’s no demand for it. You’re free to submit it for review, as
no one is working on it (or even thinking about it, I’d say).
—
Reply to this email directly or view it on GitHub
https://github.com/caskroom/homebrew-cask/issues/7693#issuecomment-193489645
.

Or cask users do not encrypt at all?

This would be a big generalisation, and I do know it’s not the truth.
Why no one ever submitted that cask, however, I do not know.

Sorry to resurrect an old thread... but I thought I'd add this info here for those following or interested.
It appears that there was an opened pull request for VeraCrypt in caskroom/homebrew-cask#12736, but it could not be accepted due to unverifiable URL, and that it did not use GPG or PGP signature verification on the dmg.  This appears totally feasible, as the links are there for both VeraCrypt DMG, and VeraCrypt DMG Signature.  Although there is no guarantee by CodePlex that these /get/xxxxxx numbered URLs will always be reliable as permalinks... they appear to at the very least be specific to a single release, and therefore may be used together to guarantee that the Signature & DMG match, and should also match a hardcoded sha256sum.
I think the main point I would make given the discussion above (and the reality of how OSS is developed), is that although Vítor says above:

Clearly there’s no demand for it. You’re free to submit it for review, as no one is working on it (or even thinking about it, I’d say).

There is a very slight difference between "demand" for something and "supply" & willingness of people in the community to actually create the supply of it.  Luckily in software, only one person has to create it and then it can be shared with everyone & copied indefinitely and infinitely in a manner which is only limited by physical storage & machines to run the software.
So like Vítor says, anyone is free to work on it & collaborate to create it... it just hasn't been fully completed yet in a manner that passes quality & security concerns of the community.  It's totally doable, but due to limitations on time, and since I still use TrueCrypt from my tap, I have no need for it, there is no other reward financially or otherwise for me to do it (If you haven't noticed, I am paid $0.0 for any of the OSS I have worked on), and I have no desire to maintain yet another project which I won't use that much.
Just thought I'd help give the community & people interested in this the links & resources necessary for it's creation, in case there is still interest & demand.  Feel free to create a fork from my tap & use it as a reference on how to create one for VeraCrypt.  Who knows... if there's enough interest or if it gets put on bountysource, I may take a couple hours to see if I can do it.

@vitorgalvao :  On a side note... I wanted to ask what the process is to become a collaborator on homebrew & homebrew cask?

there is no guarantee by CodePlex that these /get/xxxxxx numbered URLs will always be reliable as permalinks... they appear to at the very least be specific to a single release, and therefore may be used together to guarantee that the Signature & DMG match, and should also match a hardcoded sha256sum.

Unfortunately, this is not the case.

@trinitronx We know what demand looks like. Some apps, people keep asking for them even after they’re added or submitting pull requests after they have already been rejected multiple times. This app has no demand. Simple.
It’s unlikely we will ever get boutysources for casks. For core features perhaps, but for casks that’s a bit much.
We have no managerial relationship with homebrew. They do it they’re way, we do it ours. We have different needs, different rules to conform to those needs, different teams, different approaches. We also agree the other is doing a good job and plan to keep this separation of how to do things. I have no idea how they pick people to join their team, but I guess they might do it the same as we do.
For us, quite frankly I currently don’t believe you’d be able to become a good fit. Quite briefly, to become a core maintainer we invite you, not the other way around.
For us to notice your good work, you actually need to do good continuous good work as a contributor, triaging issues, helping users, contributing to the core (which we need more core maintainers for) or to casks (which current maintainers have well covered). For that, you need availability, which at the moment it seems like that’s not something you have.

Good news, @nunocalaim. See https://github.com/caskroom/homebrew-cask/pull/20386.


Multiple Molecule Converge runs fail with 8989 Bus Error

Issue Type

Bug report

Molecule and Ansible details
user01@M1711732 [10:03:43] ~/Documents/development/Ansible/concourse_ci  [master]
$ ansible --version && molecule --version
ansible 2.7.6
  config file = /Users/user01/.ansible.cfg
  configured module search path = ['/Users/user01/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /Users/user01/.pyenv/versions/3.7.2/lib/python3.7/site-packages/ansible
  executable location = /Users/user01/.pyenv/versions/3.7.2/bin/ansible
  python version = 3.7.2 (default, Jan 22 2019, 09:35:39) [Clang 9.0.0 (clang-9<IP_ADDRESS>)]
molecule, version 2.19.0
user01@M1711732 [10:03:48] ~/Documents/development/Ansible/concourse_ci  [master]
$ sw_vers
ProductName:	Mac OS X
ProductVersion:	10.12.6
BuildVersion:	16G1710
user01@M1711732 [10:03:50] ~/Documents/development/Ansible/concourse_ci  [master]
$ uname -a
Darwin M1711732 16.7.0 Darwin Kernel Version 16.7.0: Sun Oct 28 22:30:19 PDT 2018; root:xnu-3789.73.27~1/RELEASE_X86_64 x86_64
user01@M1711732 [10:03:52] ~/Documents/development/Ansible/concourse_ci  [master]
$ vagrant version
Installed Version: 2.1.5
user01@M1711732 [10:07:16] ~/Documents/development/Ansible/concourse_ci  [master]
$ VBoxManage --version
5.1.28r117968

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Detail any linters or test runners used:
Desired Behavior

Please give some details of the feature being requested or what should happen if providing a bug report.

I wanted to run molecule converge multiple times to make sure that my idempotent thing was working, and that the playbook would succeed. I can get a successful first run to work, but subsequent runs fail with 8989 Bus Error then after that following converges stall out on the failed portion.
Actual Behaviour
Please give some details of what is actually happening.
Include a minimum complete verifiable example with
output of running molecule --debug.
Molecule Debug Output listed here.

Hey @predatorian3, thanks for the report.

I wanted to run molecule converge multiple times to make sure that my idempotent thing was working,

JFYI, molecule idempotence re-runs molecule converge for you and compares task results. So, running converge multiple times shouldn't make a difference (since it is done for you). That's the idea anyway :sweat_smile:
Anyway, this feels like something gone awry in the VM that Vagrant is managing. It's hard to give any further information here but I can recommend: can you molecule login into the instance and run the command that the task was trying to do? What is the output? Also, perhaps there is something going wrong in the tear down of the VM - have you tried running molecule converge --destroy=never multiple times? Does it succeed?
Any more information would be very welcome here! Hope it helps ...

OK, pushing off 2.21 right now as we try to slim down the backlog.
Not sure how to push this forward without more Vagrant related feedback.

Molecule Vagrant driver was moved to https://github.com/pycontribs/molecule-vagrant/issues and if this issue is still valid, please raise a ticket there. Thanks.


content: add a new blog post for the 2024 Mayan Sol cruise

This PR adds a new blog post and photo gallery for the Virgin Voyages Mayan Sol cruise I did in July 2024.

The deploy preview looks good. I'm going to merge this. 🚀


Expand examples_do_not_have_gitignore test

To examples_do_not_contain_forbidden_paths or something similar:
https://github.com/trailofbits/dylint/blob/dad4f73484a0c30596afd7a1834a02cc83fc7914/examples/src/lib.rs#L101
The test should check for:

.cargo/config.toml
.gitignore
rust-toolchain

Hi, can I work on this issue? @smoelius

@0x471 Yes, please do!

I just clarified the issue description a little bit. I think the changes are small. Hopefully, you agree.

Thank you for the details, feel free to check the PR!
it seems the test is currently failing because there’s a rust-toolchain file in the general examples folder.  According to the issue description, this file shouldn’t be there.
How do you suggest proceeding with this?

it seems the test is currently failing because there’s a rust-toolchain file in the general examples folder.

That's okay, actually.
In fact, my reason for wanting to expand the test is to ensure that no file in a subdirectory overrides those.
I'm not sure the easiest/best way to suggest allowing exceptions for the roots of the general, supplementary, and restriction subdirectories. Can I leave it to you to decide?
P.S. I just tweaked the description again. Sorry for being imprecise.

I modified the test accordingly, added a new array for root directories with exceptions: https://github.com/trailofbits/dylint/pull/1461/commits/f49cf2365dcd07fbe18e6ed4d021ca72cdae1c33


Specs: more controller and feature tests

Tests:

VoteUser
Vote
Sign up, sign in, confirm account

Changing size of header-tags

Good work! +1 except for the comment above :)

Looks good now!


Maintenance mode shows wrong Effective Date Range

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
Using the new maintenance mode with 1.19 Beta 2 shows the wrong Effective Date Range when UTC plus an offset is being used and the installation uses a 24 hour scheme. See attached screenshot.
👟 Reproduction steps
Set the Maintenance Window with UTC+1 and a 24 hour scheme, e.g. Europe/Berlin.
👀 Expected behavior
A clarification what exactly the Effective Date Range is and how to interpret this would be helpful on that site.
😓 Actual Behavior
See above.
🐻 Uptime-Kuma Version
1.19 Beta 2
💻 Operating System and Arch
Ubuntu 20.04
🌐 Browser
Firefox
🐋 Docker Version
Docker CE 20
🟩 NodeJS Version
No response
📝 Relevant log output
No response

Maintenance Time Window only activate if the current date is within the Effective Date Range

maybe add a option to set the effective date to always? so it never runs out

Yes, that would be a good idea ... Maybe making this a default also should be considered.


add flatten layer

when i use kaffe-tensorflow to convert (Network in Network ) to tensorflow,but when i load the converted net ,i get error,because avg_pool.shape (batch ,10,1,1) have not been fatten，and the next layer is softmax.(batch,10) so i add a flatten layer in network.py
@layer
    def flatten(self,input,name):
        dim = 1
        for d in input.get_shape()[1:].as_list():
                dim *= d
        return tf.reshape(input,[-1,dim])

Thanks for reporting this. It appears that the NCHW order in Caffe made the singleton dimensions broadcast-able for the softmax, but not TensorFlow's NHWC. The softmax layer now squeezes out the spatial dimensions, so an explicit flatten layer should be unnecessary (tested on the updated network in network model).

after I add @icodingc code in network.py, I met the error message "Output shape computation not implemented for type: Flatten". How can I fix these error and convert caffe model which contains Flatten layers to tensorflow model sucessfully?


[Bug] Remote play  with Internet is not working

Platform

Device: Phone,
OS: Android,
Browser: kiwi
Browser Version:
Better xCloud Version: 2.1.2

Describe the bug
...
Last version, is not possible to use remote play
Early version work very well, but this not..
Screenshots/Videos
If applicable, add screenshots/videos to help explain your problem.

Does it work in the Xbox app?

Hi, yes in the Xbox app, work ok.

Is something wrong, I try to open another port, but is not working, I degrade the version, but nothing, In the xbox app, work without problems.

Does your network use IPv6?

Hello, I have IP v6 enabled but the Xbox does not grab IP v6, only v4, the
strange thing is that before it works fine without problems from the
outside. Even the official app connects well. My Nat is open, internally
the better xcloud works very well, but from the outside it no longer
connects and the message appears saying to try again later.
El jue, 4 de ene de 2024, 8:26 p. m., redphx @.***>
escribió:

Does your network use IPv6?
—
Reply to this email directly, view it on GitHub
https://github.com/redphx/better-xcloud/issues/196#issuecomment-1878036386,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFATWGRGWQA6KXCIOZXBDITYM5QEXAVCNFSM6AAAAABBHZXIISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZYGAZTMMZYGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

@KaristoCR please try this version: https://github.com/redphx/better-xcloud/raw/main/better-xcloud.user.js

It seems this problem is still present. I cannot connect via the remote play feature from outside my network, but it works perfectly through the Xbox app.


Can't compile

I tried to play with this project, but, Visual Studio can't compile it because in the file "app.cpp"  the "shader.mainImage" function code is not defined anywhere, only the function signature is specified in the fragment_shader struct but nothing else.
To be more clear, for the next code in app.cpp (line 229), where is the function code definition of shader.mainImage?
// vvvvvvvvvvvvvvvvvvvvvvvvvv
// THE SHADER IS INVOKED HERE
// ^^^^^^^^^^^^^^^^^^^^^^^^^^
shader.mainImage(shader.gl_FragColor, shader.gl_FragCoord);

Sorry if the project is not better explained.
Basically you select between subprojects with macro defines. See app.cpp lines 94-108. So for example to run the raytracer define APP_RAYTRACER either directly in the file, in the project settings or the makefile.
APP_EGG is an distance field ray marcher animation.
APP_RAYTRACER is a PBR raytracer with simple reflection and refraction.
APP_SDF_AO is a test for ambient occlusion with signed distance fields.
APP_2D is an old-skool demoscene tunnel effect.
APP_TERRAIN is an ongoing test for modeling sky and cloulds (originally I wanted to do some terrain work but gave up)
Enjoy and I will be happy to help with more questions.

Thank you for your reply and the info, finally I could make it work.
Also I want to add a small note which might be useful to other developers trying this project. Initially it didn't compile with Visual Studio 2015 Community edition, the problem for this other issue was the linker complaining of missing externals on the SDL libs, the solution was as simple as to download the same SDL version sources, to recompile them and to replace the project SDL libs with the recompiled ones.
Thank you


2.1.0 installs multiple versions of React

> npm install
...
> npm ls react
<EMAIL_ADDRESS>/home/brad/Projects/casesandberg/react-color
├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>
This is a minor issue, but one that we've found can bloat our build size if we're not careful.
Repros on a fresh pull of master at f603122813b82eedfa45f04c86fb7f695c2e9b6f although interactions with external dependencies also influence the repro.
I'm less concerned about react-context because it's listed in devDependencies and therefore only affects development of this package.  However, reactcss is listed in dependencies and therefore gets installed by consumers of react-color - thus, consumers of react-color are also installing multiple versions of React at the moment.
The resolution is not a change in this repo, but it seems to impact react-color the most which is why I'm reporting it here.  reactcss/master supports React 15 as of https://github.com/casesandberg/reactcss/pull/52, but a new version of the package has not yet been released.  Once a new minor/patch version of reactcss is on npm, thanks to the ^0.4.3 rule consumers of react-color will no longer be installing two versions of React.

For anyone interested, here's our temporary workaround by depending on our own fork until updated reactcss gets published.

I was able to solve this issue by switching the OR declaration around in both react-color & reactcss. It's odd, because<EMAIL_ADDRESS>depends on<EMAIL_ADDRESS>which resolves to 0.4.5.
<EMAIL_ADDRESS>declares react deps in the same way as react-color does, with 0.14 coming first. However, when npm install does it's magic; it ends up changing the dep for reactcss to just ^0.14.0
This seems to work though...
"react": "^15.0.0 || ^0.14.0",
"react-dom": "^15.0.0 || ^0.14.0"

Anyways, I'm tired and sick of this dependency hell, and what seems like an NPM bug.  😪

Any news on this? reactcss has moved along and is on version 1.0.4 already supporting 0.14 and 15.1 as peerDependencies.

It looks like this is still an issue, as reported a few days ago in https://github.com/casesandberg/react-color/issues/199.  I just tried installing react-color 2.2.2 and I end up with multiple versions of React:
╰○ npm ls react
(my app)
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>
It looks like a fix is available in https://github.com/casesandberg/react-color/pull/201 though!

The fix is live<EMAIL_ADDRESS>


FormBuilderDropdown onChanged fires twice during form Reset

Is there an existing issue for this?

[X] I have searched the existing issues

Package/Plugin version
9.2.1
Platforms

[X] Android
[X] iOS
[ ] Linux
[X] MacOS
[ ] Web
[X] Windows

Flutter doctor

Flutter doctor

Minimal code example

Code sample

FormBuilder(
    child: Column(
      children: [
        Builder(
          builder: (context) {
            return ElevatedButton(onPressed: (){
              FormBuilder.of(context)!.reset();
            }, child: Text('reset'));
          }
        ),
        FormBuilderDropdown(
          name: 'test',
          initialValue: 'A',
          items: [
            DropdownMenuItem(child: Text('A'), value: 'A'),
            DropdownMenuItem(child: Text('B'), value: 'B')
          ],
          onChanged: (value) {
            print('Changed value to $value');
          },
        ),
      ],
    ),
  ),

Current Behavior
When resetting the form, the dropdowns "onChange" handler will fire twice in quick succession - one setting it to the "initialValue", then a second setting it back to the value it was before the reset button was clicked (essentially undoing the reset).
Expected Behavior
When resetting the form, the "onChange" should fire only once, and the passed value should be the "initialValue".
Steps To Reproduce
Using the above code snippet.

Run the code
Click the dropdown and change its value to "B"
Press the "reset" button

Note that it will not reset, but stay as "B", and the debug console will read
flutter: Changed value to A
flutter: Changed value to B

Aditional information
No response

Additional information:
In my actual app (I'm assuming this is what's going on inside the demo code I provided too) - I have 4 FormBuilder fields.
2 * FormBuilderTextField
2 * FormBuilderDropdown
When I breakpoint the onChange, I can see that the form actually contains 6 field state objects - 2 for the text fields, and 4 for the dropdowns... it looks like for each dropdown there is a "FormBuilderDropdownState" and a "DropdownButtonFormFieldState".
When 'reset' is called on the FormBuilderDropdownState, it correctly fires onChanged and passes the initialValue.
However, when 'reset' is then called on the DropdownButtonFormFieldState this is when it resets it back to its current value.
I suspect this second state class isn't supposed to be in the fields collection - as it's just a double-up, one that breaks the functionality at that.

Additional information:
In my actual app (I'm assuming this is what's going on inside the demo code I provided too) - I have 4 FormBuilder fields. 2 * FormBuilderTextField 2 * FormBuilderDropdown
When I breakpoint the onChange, I can see that the form actually contains 6 field state objects - 2 for the text fields, and 4 for the dropdowns... it looks like for each dropdown there is a "FormBuilderDropdownState" and a "DropdownButtonFormFieldState".
When 'reset' is called on the FormBuilderDropdownState, it correctly fires onChanged and passes the initialValue. However, when 'reset' is then called on the DropdownButtonFormFieldState this is when it resets it back to its current value.
I suspect this second state class isn't supposed to be in the fields collection - as it's just a double-up, one that breaks the functionality at that.

Yeah but DropdownButtonFormField also extends FormField, In FormField's build function we are registering this field. So even this gets registered
If we comment out the reset function for DropdownButtonFormFieldState it works.


Installing dependencies on fresh Raspbian OS Raspberry pi4 failed

after running the ./install.sh script i get this error:
Preparing to unpack .../39-libopencv-dev_3.2.0+dfsg-6_armhf.deb ...
Unpacking libopencv-dev (3.2.0+dfsg-6) ...
dpkg: error processing archive /tmp/apt-dpkg-install-TZ0gMF/39-libopencv-dev_3.2.0+dfsg-6_armhf.deb (--unpack):
 trying to overwrite '/usr/bin/opencv_annotation', which is also in package opencv-dev 4.4.0
dpkg-deb: error: paste subprocess was killed by signal (Broken pipe)
Errors were encountered while processing:
 /tmp/apt-dpkg-install-TZ0gMF/39-libopencv-dev_3.2.0+dfsg-6_armhf.deb
E: Sub-process /usr/bin/dpkg returned an error code (1)
make[1]: *** [Makefile:91: deps] Error 100
make[1]: Leaving directory '/home/pi/allsky/src'
make: *** [Makefile:30: deps] Error 2
Installing dependencies failed!
pi@raspberrypi:~/allsky $

It is something related opencv which i installed according https://github.com/dlime/Faster_OpenCV_4_Raspberry_Pi
(Opencv is required by the install script but is not included in Raspbian OS nor allsky install script)

Actually, since they are deb's ( I just found them in your provided linked repo )... You can roll them back.  Try this:
apt remove OpenCV*

If that succeeds, you can then try the ./install.sh script again.
Thanks!

Hi linuxkidd
I did a fresh install Raspbian and then run the install script again - now it works. Thank you.


Update docker-compose.yml

Added a friendly fix for someone who might be using version: '2' docker-compose.yml standards.

hi! thanks - ended up going with '3.2' locked specific version, this way it also works with portainer and others


fix: set window filetype earlier

I had an issue where neotest clashes with the window-resizing plugin windows.nvim when opening the the test summary window, despite explicitly telling window.nvim to ignore the neotest-summary filetype.
It turns out that neotest sets the filetype neotest-summary too late. windows.nvim triggers on BufWinEnter and checks the filetype, which wasn't yet set by neotest. I can imagine that more plugins that check the filetype on BufWinEnter (or similar events) would have the same issue.
In this PR the window filetype is set as early as possible on open, which fixes the issue. I'm not sure if my solution is the best way to solve it, so feel free to tweak it!

Thanks for the PR! :smile: Unfortunately setting the filetype happens after opening the window specifically to support users using a filetype autocmd or ftplugin to set window settings, otherwise they would have no effect. As that is built-in functionality I'd prefer to support that instead.
What about telling windows.nvim to ignore nofile buftypes?

I see, I'll go with the nofile solution, thank you!


feat: add options to replace file name on rename action

Closes #354
This is in my opinion a very important feature to have, and I'd have a further proposal:
use --replace as default behavior and add three different flags:

--append: puts cursor between name and extension (e.g. name|.ext)
--append-ext: puts cursor at the very end of the extension (current default behavior, e.g. name.ext|)
--prepend: puts cursor at the very beginning of the file name (e.g. |name.ext)

This would be a little breaking, but I think it's worth it since we are moving to v0.2, and it would also be more consistent with ranger's behavior.
Please do let me know your thoughts on this :smiley:

Awesome! I love this feature, it would resolve https://github.com/sxyazi/yazi/issues/354. Let me make some changes to it!

--append: puts cursor between name and extension (e.g. name|.ext)

--append-ext: puts cursor at the very end of the extension (current default behavior, e.g. name.ext|)

--prepend: puts cursor at the very beginning of the file name (e.g. |name.ext)

Can I start pushing these changes or do you want to change something else before?

Please merge replace and keep_ext into a single empty option with the following values:

all: "foo.jpg" => ""
name: "foo.jpg" => ".jpg"
ext: "foo.jpg" => "foo."
dot_ext: "foo.jpg" => "foo"

Then merge append, append-ext, and prepend into a single cursor option with:

prepend: set cursor to 0
append: set cursor to filename.len() after empty is applied

We can add it in the preset keymap.toml. Due to Yazi's configuration file policy, this will not be considered a breaking change.

Ok, so what should be the new default behavior (I mean rename with no flags)?

How about rename --cursor before_ext?

Maybe we can add another new R key for rename --empty name --cursor before_ext.

How about rename --cursor before_ext?

Agreed.

Maybe we can add another new R key for rename --empty name --cursor prepend.

As for --empty name I'd implicitly set the cursor at the start because it just makes sense, I think no one would remove the file name keeping the extension and start adding text at the end of it...

My idea is that each option should ideally do only one thing, to achieve more certainty.
If a user wants to empty a part of the file name, they can use --empty; if a user wants to change the cursor position, then it's --cursor. Of course, they can also combine them for use.

Yeah, that makes also total sense, and it's also easier to implement. Pushing soon so we can discuss some real code

Everything seems to work as expected now

Hi, I made some simplifications, would you like to give it a review?

Nice catch. Let me know if it's ready to merge!

Let me know if it's ready to merge!

I think so, now everything looks good to me!

Okay, let me merge it now! Thank you for contributing to this feature!

Thank you for contributing to this feature!

It was a pleasure! I'm very happy that this feature got merged, not having it could be a deal breaker for some users (such as myself).


POST /findings redirect you to a GET request /findings/

Bug description
If you create a POST request with the method /findings (without the character "/" at the end) It redirects you to a GET request /findings/
Steps to reproduce
Steps to reproduce the behavior:

If you have defect dojo in your localhost environment, run a POST request with Postman to the following url http://<IP_ADDRESS>:8080/api/v2/findings (without the "/" at the end) and send the request
The response is a GET request

This is a feature of Django Rest Framework. We could change the default settings for that, but unsure about the impact and what it would mean to existing scripts. Why is this an issue for you?

That is not the expected behavior I would expect if I missed a "/" in the request. If you are not going to fix, at least is going to be good to have this well documented.
Regards,

and what is the expected behaviour.


HTTP: sol_http_progressive_response_feed() should receive a blob. v3

Signed-off-by: Guilherme Iscaro<EMAIL_ADDRESS>CHanges since v2:

Minor fixes pointed by @barbieri
The user is now able to delete the sol_http_client_connection from the callbacks send_cb response_cb recv_cb

style-check will complain, however I did not touch those code sections

#2049


se-1 Add dedicated email

Adding a dedicated email - Does your project have an e-mail address or forum that is solely for supporting users? - create a dedicated email - Unnati (README Support)

Added the support email and discord community.


Support for cd_files

Thank you for a very useful project, I was able to use it as a great source of information to build Windows 2019 images using the hyperv-iso and hyperv-vmcx builders.
Packer 1.6.3 added support for the cd_files  token to a number of builders, including the virtualbox-iso one.
This allows you to specify a list of files which Packer will package up as an ISO and attach to the booting VM. For instance it can make it more convenient to make changes to autounattend.xml without having to rewrap it into an ISO, and is an alternative to the floppy_files token on platforms where a virtual floppy drive is no longer supported (for instance Generation 2 VMs on Hyper-V).
The drawback is that you need one of the supported command line ISO authoring utilities in the path, oscdimg.exe can be extracted from the Windows 10 ADK.

Hi @jfpanisset,
I'm glad you're enjoying it.
Thanks for letting me know about this new feature, I haven't been following packer lately and wasn't aware of it.
It does look appealing, however the fact that you have get oscdimg.exe from the ADK that has a non-negligible size and that you have to add it to your path, would in my opinion deter a lot of people from using these packer files. I've tried looking for info about whether or not I could redistribute oscdimg.exe, but I haven't found anything.
As for linux users, they'd still have to add one of the .iso generating tools to their path.
This is why I prefer not to switch to using the cd_files parameter.
If you'd like to, you can send a PR to add your Hyper-V builder with cd_files, and as it is the only supported way for using autounattend.xml files on gen 2 VMs I'd be more than glad to merge it.


Move to a unified framework target that can build for either ios or tvos

This change removes the separate tvos framework and testing targets, and changes the iOS target to build successfully against either the iOS or tvOS SDKs. This should help keep target maintenance down to a minimum, especially as we move to convert to swift 3.
This should only impact integration via carthage: some older versions of carthage can't handle this sort of framework (I think something like 0.14 or something is when this functionality was introduced, so it shouldn't have a major affect on most who keep carthage up to date).
I also removed the framework project reference from the example project, since xcode is able to infer dependencies if you add them from the sibling framework in the workspace instead.
This should wait to be merged, since there's one remaining issue I need to resolve to allow unit tests to run on an apple tv simulator for the framework scheme.

Ah scratch this, forgot there were xibs built for each platform included as well. I'm not sure how to get around that yet so I'll close this out.


Have coremanager tag legal and profile on final candidate release

I won't implement it for now, because it'd require quite some changes (because those are seperate repositories).


Release 2.3

@alexcheng1982 can you please add release tag for Magento 2.3 ?
Thanks

Hi DanGlz, have you been able to get a Magento 2.3 container up and running? If so have you got any tips as to how to do it? Many thanks.


[EN DateTimeV2] In couple of weeks isn't being recoginzed as two weeks datetimeV2 entity

For the text "Get back to me in couple of weeks", couple of weeks isn't being extracted as datetimeV2 entity. It should be two weeks

Thanks for


No fab actions

I need to be able to click on the main button and fire events without opening the smaller buttons.

Instead, when I use @click or @click.native on the main button, an error is logged
[Vue warn]: Error in callback for watcher "toggle": "TypeError: Cannot read property 'length' of undefined"
found in
---> <Fab> at node_modules\vue-fab\src\FAB.vue

Hi @blurdylan ,
We've just released a new package version with that fix.
Now you can use @click.native on the main button.

@Edujugon ugh the FAB rotates and doesn't un-rotate with this method? any tips?

Hi @jvb93 ,
I'm trying to reproduce the issue but it actually rotates and un-rotates just fine.
Could you share some code snippet to see how you're using the component?

Hey, I think I was not getting your point...But now I guess I do.
So basically when you click on the button it rotates and keeps that position.
What would be the desired behaviour?

no rotate at all and trigger that event
rotate, trigger the event and rotate back

@Edujugon Hmm yeah this is interesting. If there is no menu to display, then the animation to rotate doesn't make much sense at all, as it's not, say rotating to open. Here's what I propose:

Provide a prop that would allow the developer to choose whether or not to rotate
OR if there are no actions, do not rotate at all.

EIther way, I am down to code this and PR, no worries.


Not creating Private key & public key

Here i am running generatekey file but not generating private key and public key, please check it.
http://localhost/php-bitpay-client/examples/tutorial/001_generateKeys.php

@soravgarg123 what is your error? I just have this error when I tried to create private key:
PHP Notice:  Uninitialized string offset: 0 in [...]/Vendor/bitpay/php-client/src/Bitpay/Math/RichArbitraryPrecisionIntegerMath.php on line 230
And the page go into a infinite time of loading when I tried to create public key.
Then, I have installed 'php-bcmath' and it's ok now, the two keys are generated.

Hey, thanks @estebandelaf, this fixed my problem as well. For others curious, I was getting the NOTICE when trying to generate the public and private keys:
PHP Notice:  Uninitialized string offset: 0 in /etc/Bitpay/vendor/bitpay/php-client/src/Bitpay/Math/RichArbitraryPrecisionIntegerMath.php on line 230
To follow @estebandelaf 's advice for Ubuntu 17.04:
sudo apt-get install php-bcmath
sudo /etc/init.d/apache2 restart

I then reloaded the 001_generateKeys.php file, and all is well in the world again.

@kneeki finally, bcmath doesn't solve my problem. Yes, it works for creating public key. But, later, can't make the connection with Bitpay. I can't remember the exact error, but the problem was bcmath when I tried to create the invoice. I used gmp and removed bcmath.
Is another issue with the problem with bcmath: https://github.com/bitpay/php-bitpay-client/issues/237#issuecomment-326899419


Verifying that HC's lesion masks are empty

Opening this issue to check if the Healthy Controls' (HC) lesion masks are empty (meaning that there are no segmented lesions on the masks).
This is related to this comment.
Tagging @jcohenadad and @leelisae which were part of the initial discussion.

Spoiler alert: PROBLEMS found !
The code I used to track the HC and verify that their respective masks are empty is the following:

import sys
import os
import subprocess
import nibabel as nib
import warnings

# Print the file name
print("File name:", sys.argv[1])

# Check if filename is provided as argument
if len(sys.argv) < 2:
    print("Usage:", sys.argv[0], "<filename.tsv>")
    sys.exit(1)

# Extract participant IDs with HC pathology
with open(sys.argv[1], 'r') as file:
    participant_ids = [line.split('\t')[0] for line in file if line.split('\t')[3] == "HC"]

# Iterate over participant IDs
for participant_id in participant_ids:
    # print("Participant ID:", participant_id)

    # Find the files ending in lesion-manual.nii.gz in subfolders of derivatives/participant_id
    for root, dirs, files in os.walk(os.path.join(sys.argv[2], participant_id)):
        for file in files:
            if file.endswith("lesion-manual.nii.gz"):
                # print("File:", os.path.join(root, file))
                # Check if the file contains only zeros
                img = nib.load(os.path.join(root, file))
                data = img.get_fdata()
                if data.sum() == 0:
                    print(participant_id, ": Only zeros")
                # Else we raise a warning
                else:

                    warnings.warn(file + ": Non-zero values found in lesion mask", UserWarning,  stacklevel=2)

It is executable using the following command
python get_HC.py ~/Documents/data/canproco/participants.tsv ~/Documents/data/canproco/derivatives/labels

Here is the output:
File name: /Users/plbenveniste/Documents/data/canproco/participants.tsv
sub-cal110 : Only zeros
sub-cal163 : Only zeros
sub-cal186 : Only zeros
sub-cal187 : Only zeros
sub-cal194 : Only zeros
sub-cal195 : Only zeros
sub-cal199 : Only zeros
sub-cal200 : Only zeros
sys:1: UserWarning: sub-cal201_ses-M0_STIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sub-cal202 : Only zeros
sub-edm123 : Only zeros
sub-edm127 : Only zeros
sub-edm128 : Only zeros
sub-edm129 : Only zeros
sub-edm138 : Only zeros
sub-edm166 : Only zeros
sub-edm168 : Only zeros
sub-edm169 : Only zeros
sub-edm177 : Only zeros
sub-edm178 : Only zeros
sub-edm179 : Only zeros
sub-mon014 : Only zeros
sub-mon014 : Only zeros
sys:1: UserWarning: sub-mon027_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sub-mon031 : Only zeros
sys:1: UserWarning: sub-mon031_ses-M12_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sub-mon065 : Only zeros
sys:1: UserWarning: sub-mon096_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sub-mon105 : Only zeros
sys:1: UserWarning: sub-mon126_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sys:1: UserWarning: sub-mon155_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sys:1: UserWarning: sub-mon169_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sys:1: UserWarning: sub-mon172_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sub-mon202 : Only zeros
sub-tor038 : Only zeros
sub-tor038 : Only zeros
sys:1: UserWarning: sub-tor039_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sub-tor039 : Only zeros
sub-tor051 : Only zeros
sub-tor051 : Only zeros
sub-tor052 : Only zeros
sub-tor053 : Only zeros
sub-tor054 : Only zeros
sub-tor055 : Only zeros
sub-tor057 : Only zeros
sub-tor092 : Only zeros
sub-tor148 : Only zeros
sub-van005 : Only zeros
sub-van138 : Only zeros
sub-van139 : Only zeros
sub-van141 : Only zeros
sys:1: UserWarning: sub-van159_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sys:1: UserWarning: sub-van160_ses-M0_PSIR_lesion-manual.nii.gz: Non-zero values found in lesion mask
sub-van162 : Only zeros
sub-van163 : Only zeros
sub-van164 : Only zeros
sub-van165 : Only zeros

Therefore, we have mistakenly segmented some non-existing lesions in HC controls, which is really problematic.
Thanks @leelisae for pointing this out to us.
TODO:

[ ] Clear lesions segmentation of all HCs.

@plbenveniste Awesome! To clarify - I'll remove the lesion segmentations I received from you earlier (Issue #88) and replace this with the lesion segmentations above instead on my end.  Please let me know if I misunderstood this. Thanks!

@plbenveniste While working on this analysis, I found that there were some lesion masks that I expected but could not find in the zip folder you provided above. Could you please clarify? Thank you!
Missing lesion masks:
Site: Edmonton
sub-edm161_ses-M0_PSIR_lesion-manual.nii.gz (CAN-05-RRM-161-M0)
Site: Montreal
sub-mon015_ses-M0_PSIR_lesion-manual.nii.gz (CAN-03-CON-015-M0)
sub-mon052_ses-M0_PSIR_lesion-manual.nii.gz (CAN-03-RRM-052-M0)
sub-mon164_ses-M0_PSIR_lesion-manual.nii.gz (CAN-03-RRM-164-M0)

Hi @lisaeylee !
I checked the files using the following command :
git log --follow -- sub-mon015/ses-M0/anat/sub-mon015_ses-M0_PSIR.nii.gz

I found that the reason these files didn't have a segmentation is because they were added in January with the corrected M0 data. Therefore, we didn't take the time to annotate them. There are no particular reasons for the segmentations masks to be absent.

(1) Yes the lesion masks were created using the corrected M0 data.
(2) On it now. Issue #99

Hi @lisaeylee, here is the new batch of lesion segmentations. Thanks again for your feedback.
canproco_M0_lesion_segmentations.zip
Also, as I was saying in this comment some files are recorded in an exclude.yml file because they are too artifacted for us to segment lesions or the spinal cord. This might impact your registration task.
Good luck with the rest of your work 😃

Hi Lisa !
I am sorry for the late reply. The exclude.yml file can be found in this current repo.
The file is here: https://github.com/ivadomed/canproco/blob/main/exclude.yml

@plbenveniste - No worries - you actually had already answered this in another thread! All good :)


Static Environment List

Creating and pushing new branches/environments to our puppet server now requires to reload the uwsgi process as the environment list is assembled globally upon puppetboard start.
This is very unfortunate and kind of kills our workflow. Three "solutions" come to mind:

reload uwsgi every time we push to our puppet repo

add patch to globally disable environment based views

add patch to reload environment list on every request

What do you recommend we should do?

I found another "feature".
It looks like puppetdb keeps environments as long as there are resources or reports available even though there are no nodes left in that environment. I'm not sure if this is useful for some people. If not we could check /pdb/query/v4/environments/<ENVIRONMENT>/facts for each environment and skip it if the result comes up empty.

@octomike I do not think that this latest approach is the best way because it brute-forces the PuppetDB. Users will be better off adjusting the setting of the node-purge-ttl setting in their PuppetDB configurations.

As for the initial solutions I would rather update the code to reload the environment list on every request.
Reloading the uwsgi process is not possible programmatically because each environment may be set up different. Globally disabling environments is also not advisable because Puppet is completely environment based now and would not make much sense to reverting this approach (there are active PRs to update the environment functionality now)

You're right, I overlooked the pull requests, specifically #185.
The "stale" environment issue still exists though. Lowering node-purge-ttl does not seem to be a solution at a first glance. How would we be able to track unreported nodes for multiple days and at the same time purge empty (no nodes left) environments very quickly ?

PuppetDB has a Garbage Collector that runs every 60 minutes by default but configurable via gc-interval. This process will clean up any deactivated nodes after node-purge-ttl expires as well as any facts, catalogs and environments.
You can lower the value of this setting if you require it.

In our environment-centric workflow nodes move between environments quite quickly.
Even with a low setting of gc-interval our (temporary) development environments pollute puppetboard as they won't be removed because at least report-ttl is deliberately kept at the default (14d).
I guess I am questioning if it makes sense for puppetboard to keep listing reports and catalogs for empty environments.

No matter how we try to suppress environments with no active nodes it still doesn't hide the fact that they still exist in PuppetDB and PuppetBoard only reports on what's present in PuppetDB.
We will submit a code patch to check the environments on each page load so you won't have to reload uwsgi all the time.

@octomike I have created https://github.com/puppet-community/puppetboard/issues/189 to fix this issue. Please stay tuned for the 0.1.2 release.


[MM-12861] Improve sorting in emoji autocomplete and emoji picker

Summary
This PR resolves #9761 by sorting the emojis for the emoji autocomplete and emoji picker like so:

First, display recently used emoji that match the search term, in alphabetical order
Second, display emoji that start with the search term, in alphabetical order
Last, display emoji that contain the search term, in alphabetical order.

Ticket Link
JIRA | #9761
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

Sorry this one took me so long to get around to! 😳

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Thank you @amorriscode! Much appreciated.
I've queued the PR for PM review. Note that with the holidays, it might take a little longer to review than usual.

Spinmint test server created at: https://i-0aa6b2eb95a23d2d7.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-0aa6b2eb95a23d2d7

Spinmint test running for more than 7 days. This test server was terminated.

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-012e7008f551c3ca4.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Instance ID: i-012e7008f551c3ca4

Thank you for adding this @amorriscode!  Looks really good!  The only thing I see is when you use the thumbsup emoji, it does not work as expected.  Thumbsdown is placed before thumbsup even when thumbsdown has not been used and thumbsup was recently used

.


CLOUDSTACK-10333: Secure Live VM Migration for KVM

Description
This extends securing of KVM hosts to securing of libvirt on KVM
host as well for TLS enabled live VM migration. To simplify implementation
securing of host implies that both host and libvirtd processes are
secured with management server's CA plugin issued certificates.
FS: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Secure+Live+VM+Migration+for+KVM
Based on whether keystore and certificates files are available at
/etc/cloudstack/agent, the KVM agent determines whether to use TLS or
TCP based uris for live VM migration. It is also enforced that a secured
host will allow live VM migration to/from other secured host, and an
unsecured hosts will allow live VM migration to/from other unsecured
host only.
Post upgrade the KVM agent on startup will expose its security state
(secured detail is sent as true or false) to the managements server that
gets saved in host_details for the host. This host detail can be accesed
via the listHosts response, and in the UI unsecured KVM hosts will show
up with the host state of ‘unsecured’. Further, a button has been added
that allows admins to provision/renew certificates to KVM hosts and can
be used to secure any unsecured KVM host.
The cloudstack-setup-agent was modified to accept a new flag -s
which will reconfigure libvirtd with following settings:
listen_tcp=0
listen_tls=1
tcp_port="16509"
tls_port="16514"
auth_tcp="none"
auth_tls="none"
key_file = "/etc/pki/libvirt/private/serverkey.pem"
cert_file = "/etc/pki/libvirt/servercert.pem"
ca_file = "/etc/pki/CA/cacert.pem"

For a connected KVM host agent, when the certificate are
renewed/provisioned a background task is scheduled that waits until all
of the agent tasks finish after which libvirt process is restarted and
finally the agent is restarted via AgentShell.
There are no API or DB changes.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
Unsecured KVM hosts when are Up and connected show up as:

Button to secure host, or renew/provision certificate using configured CA plugin;

Error message when VM migration is between secured and unsecured hosts:

How Has This Been Tested?

Tested between a pre-upgraded secured KVM host (el7 based) and post-upgrade it secures libvirtd using same certificates.
Tested tls enabled live vm migration between two secured and unsecured el7 kvm hosts.

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly. PR https://github.com/apache/cloudstack-docs-admin/pull/50, https://github.com/apache/cloudstack-docs-install/pull/36
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1807

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian.  I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✖centos7 ✖debian. JID-1820

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✖centos7 ✖debian. JID-1821

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✖centos7 ✖debian. JID-1830

Although the iptables changes have been removed for Ubuntu/Debian I think we should also remove them from the RPM packages.
In my opinion a package should never be allowed to touch firewalls without the operating knowing it.
In the documentation we already tell users to open ports: http://docs.cloudstack.apache.org/projects/cloudstack-installation/en/4.11/hypervisor/kvm.html#configuring-the-firewall
If additional ports need to be opened we should put them in there, but not just open them in a RPM or DEB package.

@rhtyd there seems to be some build errors with the latests changes

@borisstoyanov yes there was an unused import.
@wido bear in mind that cloudstack-setup-agent will reconfigure network, libvirtd, iptables configuration. It's a little known fact, and not well documented; the docs are redundant (they tell you to add stuff, that cloudstack-setup-agent already does, you may test it yourself). I've removed the post-install steps, now you can re-review. I've sent doc PRs instead:
apache/cloudstack-docs-admin#50
apache/cloudstack-docs-install#36

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1836

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@rhtyd Yes, that is correct. That tool is allowed to make such changes, but we should not just do this in a postinst of a package.
The postinst of a package is to clean up old files related to the package, handle some things around the package, but it should not start touching firewalling on a host.
That is just not done when it comes to packages.

@rhtyd I'm trying to migrate a VM between unsecured hosts and got the following exception:
2018-03-28 12:31:52,414 ERROR [c.c.v.VmWorkJobHandlerProxy] (Work-Job-Executor-6:ctx-bc927fcc job-38/job-39 ctx-441b0b18) (logid:d8b1d686) Invocation exception, caused by: com.cloud.utils.exception.CloudRuntimeException: org.libvirt.LibvirtException: Cannot get interface MTU on 'breth0-769': No such device
2018-03-28 12:31:52,415 INFO  [c.c.v.VmWorkJobHandlerProxy] (Work-Job-Executor-6:ctx-bc927fcc job-38/job-39 ctx-441b0b18) (logid:d8b1d686) Rethrow exception com.cloud.utils.exception.CloudRuntimeException: org.libvirt.LibvirtException: Cannot get interface MTU on 'breth0-769': No such device
2018-03-28 12:31:52,415 DEBUG [c.c.v.VmWorkJobDispatcher] (Work-Job-Executor-6:ctx-bc927fcc job-38/job-39) (logid:d8b1d686) Done with run of VM work job: com.cloud.vm.VmWorkMigrate for VM 3, job origin: 38
2018-03-28 12:31:52,415 ERROR [c.c.v.VmWorkJobDispatcher] (Work-Job-Executor-6:ctx-bc927fcc job-38/job-39) (logid:d8b1d686) Unable to complete AsyncJobVO {id:39, userId: 2, accountId: 2, instanceType: null, instanceId: null, cmd: com.cloud.vm.VmWorkMigrate, cmdInfo: rO0ABXNyABpjb20uY2xvdWQudm0uVm1Xb3JrTWlncmF0ZRdxQXtPtzYqAgAGSgAJc3JjSG9zdElkTAAJY2x1c3RlcklkdAAQTGphdmEvbGFuZy9Mb25nO0wABmhvc3RJZHEAfgABTAAFcG9kSWRxAH4AAUwAB3N0b3JhZ2V0AA9MamF2YS91dGlsL01hcDtMAAZ6b25lSWRxAH4AAXhyABNjb20uY2xvdWQudm0uVm1Xb3Jrn5m2VvAlZ2sCAARKAAlhY2NvdW50SWRKAAZ1c2VySWRKAAR2bUlkTAALaGFuZGxlck5hbWV0ABJMamF2YS9sYW5nL1N0cmluZzt4cAAAAAAAAAACAAAAAAAAAAIAAAAAAAAAA3QAGVZpcnR1YWxNYWNoaW5lTWFuYWdlckltcGwAAAAAAAAAAXNyAA5qYXZhLmxhbmcuTG9uZzuL5JDMjyPfAgABSgAFdmFsdWV4cgAQamF2YS5sYW5nLk51bWJlcoaslR0LlOCLAgAAeHAAAAAAAAAAAXNxAH4ABwAAAAAAAAACcQB-AAlwcQB-AAk, cmdVersion: 0, status: IN_PROGRESS, processStatus: 0, resultCode: 0, result: null, initMsid:<PHONE_NUMBER>041, completeMsid: null, lastUpdated: null, lastPolled: null, created: Wed Mar 28 12:31:50 UTC 2018}, job origin:38
com.cloud.utils.exception.CloudRuntimeException: org.libvirt.LibvirtException: Cannot get interface MTU on 'breth0-769': No such device

I've made the hosts unsecured by deleting the executing the cloudstack-setup-agent script without the -s option.

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Maybe this ^^ needs further investigation since I've provisioned the certificates on both the hosts and still getting the same error.

Packaging result: ✔centos6 ✔centos7 ✔debian.  Please use the sed based unsecuring approach, and avoid cloudstack-setup-agent to speed up tests.
@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian.  I'll keep you posted as I make progress.

I'll kick new tests with the latest commit once the build comes out

Packaging result: ✖centos6 ✖centos7 ✖debian. JID-1850

@blueorangutan package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✖centos7 ✖debian. JID-1851

@blueorangutan package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1855

@blueorangutan test

@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-2459)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 107420 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2505-t2459-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_routers_network_ops.py
Intermitten failure detected: /marvin/tests/smoke/test_routers.py
Intermitten failure detected: /marvin/tests/smoke/test_ssvm.py
Intermitten failure detected: /marvin/tests/smoke/test_vm_life_cycle.py
Smoke tests completed. 65 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_04_restart_network_wo_cleanup
Failure
4.14
test_routers.py

test_01_secured_vm_migration
Failure
1069.92
test_vm_life_cycle.py

test_02_not_secured_vm_migration
Failure
118.28
test_vm_life_cycle.py

Looks like there is an outstanding/related issue to CA management, I'll have a look next week (on holidays now).

I've resolved the conflict, let me run tests again @rhtyd
@blueorangutan package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Okay @borisstoyanov
All - please hold merging this, I may include some keystore related changes reported in recent issues.

Packaging result: ✔centos6 ✔centos7 ✔debian.  I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian.  Tests pass:
Test secured VM migration ... === TestName: test_01_secured_vm_migration | Status : SUCCESS ===
ok
Test Non-secured VM Migration ... === TestName: test_02_not_secured_vm_migration | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_03_secured_to_nonsecured_vm_migration | Status : SUCCESS ===
ok
Test Non-secured VM Migration ... === TestName: test_04_nonsecured_to_secured_vm_migration | Status : SUCCESS ===

All looks good. I'll merge this based on code reviews and tests. Doc PRs have been merged as well.

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1962


Update sample2.xml

I think this will make it look better! It was confusing to me to see the closing /> on a separate line without the closing tag </connectionStrings>

@badersur This is a great formatting change. Thank you for the contribution.


feat: fix cli commando for sitemap

What does it do?
fix cli commando for sitemap
Why is it needed?
Describe the issue you are solving.
How to test it?
Provide information about the environment and the path to verify the behaviour.
Related issue(s)/PR(s)
Let us know if this is related to any issue/pull request

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 38.14%. Comparing base (e9d8c8d) to head (e606992).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #201   +/-   ##
=======================================
  Coverage   38.14%   38.14%
=======================================
  Files           4        4
  Lines         839      839
  Branches      181      181
=======================================
  Hits          320      320
  Misses        419      419
  Partials      100      100

Flag
Coverage Δ

unit
38.14% <ø> (ø)

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

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


rhel-84 builds fail on s390x

GenericError: Failed to create the compose request: {"message":"Failed to depsolve base base packages for qcow2/s390x/rhel-84: DNF error occured: MarkingErrors: Error occurred when marking packages for installation: Problems in request:\nmissing packages: efi-filesystem, efivar, efivar-libs, grub2-efi-x64, shim-x64"}

Yeah, S390x does not use UEFI but zipl.

I think UEFI packages are pulled for all arches if the image is qcow2, which should also break ppc64le.
https://github.com/osbuild/osbuild-composer/blob/8e86d9dcae30ee87730beecd79dcd66d2acf8f6d/internal/distro/rhel84/distro.go#L823

This works ok for RHEL-8.3 builds, but fails for RHEL-8.4.
It seems like we added some arch-specific packages to arches that should not have them. Should be easy to fix fingers crossed.

I got this error when building aarch64:
GenericError: Failed to create the compose request: {"message":"Failed to depsolve base base packages for qcow2/aarch64/rhel-84: DNF error occured: MarkingErrors: Error occurred when marking packages for installation: Problems in request:\nmissing packages: grub2-efi-x64, shim-x64"}

And ppc64le:
{"message":"Failed to depsolve base base packages for qcow2/ppc64le/rhel-84: DNF error occured: MarkingErrors: Error occurred when marking packages for installation: Problems in request:\nmissing packages: efi-filesystem, efivar, efivar-libs, grub2-efi-x64, shim-x64"}

Thanks @ondrejbudai for reporting this and pasting the error for all architectures!
Turns out, the x86_64 EFI-related packages slipped into the generic package list. Since we're only testing on x86_64, CI didn't catch it. I've filed #1104 to fix it.


Discuss and plan adding an unique identifier (and perhaps sequence number) for each request response

With the requests/responses being asynchronous this seems necessary to properly managing the asynchronous events/messages.

This functionality is added in polaris-web, each incoming request contains a requestId


Define feature list for this application (TODO)

Automatically, or using CLI, the benchmarker should be versatile enough to launch srun if there is no allocation, or inspect the available hosts to spawn MPI procs on them.
We could use a separated JSON for that configuration, so we can have it for different clusters.
It could be like Hyperfine, but for distributed (and local) applications, with boostratp error column and automatic plotting of profiled data. We just need to specify one instrumentation formating like:
STDERR> [timestamp in u seconds][name of the stacked bar][begin]
STDERR> [timestamp in u seconds][name of the stacked bar][end]

Closing as we moved this discussion to the Mattermost channel


Strange or wrong week number for Western

Hi,
weekNumber for 29th of December 2024 (Western) is equal to 53, however when I look at US online week calendars, they all show 1.
Looks like a bug or I don't understand what Western is used for.

hi @Nick-A2B,
Sorry for late reply. I forgot to enable github notification.
Yes, It seems it is a bug from the view of standard specification.
It is convenient for me (for my app) to return 53 because 29th dec 2024 is 53th week of 2024.
Anyway, I'll fix it soon.
Thank you for a  bug report.

hi @Nick-A2B, Sorry for late reply. I forgot to enable github notification.
Yes, It seems it is a bug from the view of standard specification. It is convenient for me (for my app) to return 53 because 29th dec 2024 is 53th week of 2024. Anyway, I'll fix it soon.
Thank you for a bug report.

Thanks for a quick fix!


court information does not populate on review screen until end of interview

When I'm still mid-interview (but after selecting court, etc.), the review screen looks like this:

When I get to the download screen, additional court information appears on the review screen:

Can the court information appear as soon as it has been entered?

(same result for all 3 types of courts)


Debug Spyder-parallel run issue

On Starsim commit 88d44fe, this code:
import starsim as ss

s1 = ss.Sim(diseases='sir', networks='random')
s2 = ss.Sim(diseases='sis', networks='random')

msim = ss.parallel(s1, s2)

sometimes fails with:
Exception in thread Thread-12 (_handle_results):
Traceback (most recent call last):
  File "/software/anaconda3/lib/python3.11/threading.py", line 1045, in _bootstrap_inner
    self.run()
  File "/software/anaconda3/lib/python3.11/threading.py", line 982, in run
    self._target(*self._args, **self._kwargs)
  File "/software/anaconda3/lib/python3.11/site-packages/multiprocess/pool.py", line 579, in _handle_results
    task = get()
           ^^^^^
  File "/software/anaconda3/lib/python3.11/site-packages/multiprocess/connection.py", line 253, in recv
    return _ForkingPickler.loads(buf.getbuffer())
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/software/anaconda3/lib/python3.11/site-packages/dill/_dill.py", line 301, in loads
    return load(file, ignore, **kwds)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/software/anaconda3/lib/python3.11/site-packages/dill/_dill.py", line 287, in load
    return Unpickler(file, ignore=ignore, **kwds).load()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/software/anaconda3/lib/python3.11/site-packages/dill/_dill.py", line 442, in load
    obj = StockUnpickler.load(self)
          ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cliffk/sciris/sciris/sciris/sc_odict.py", line 1207, in __setitem__
    odict.__setitem__(self, name, value)
    ^^^^^
NameError: name 'odict' is not defined. Did you mean: 'dict'?

...but only on Spyder.

Reinstalled conda and the problem went away ... not sure what it was, but hard to reproduce!


升级 xuperchain,xupercore依赖,调整maxMessagsize设置

Description
What is the purpose of the change?
平行链和群组功能需要使用最新的xuperchain和xupercore,调整maxMessagsize设置
Fixes # (issue)
Brief of your solution
平行链和群组功能需要使用最新的xuperchain和xupercore,调整maxMessagsize设置

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: springrain:x: chimengnanchimengnan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

谢谢您的贡献，我们内部尽快安排review。


Added adds and removes events

I'm adding the adds and removes events which fire an aggregated event with all the added/removed models. This is useful for batch changes that need to be listened for. Without this you'd have to listen to add and use debounce to collect all the changes within a certain period of time. This is difficult since you could collect changes with different at's specified and you'll need to keep track of those.
I did change what remove() returns. It will only return the models that were actually removed, which could be undefined, if you passed in a singular model that wasn't in the collection. I added a test to back up this new behavior since I didn't see any tests backing up the current way it was working. The docs also weren't clear which way it was supposed to work but since removes will only give you the models actually removed, I thought remove() should work the same way.

For reference: #2063 #2365 #2268 #2265 and others. What about reordering? What's so difficult about reconciling what models you've already rendered and just iterating over the list?

So I talked with @akre54 on gitter and stated that I'd like the adds/removes so we can get batched change events for efficiently rendering large collections (like your queue or library). Instead of getting 50 add events and hitting the DOM for each one, you could get one adds event and render all 50 (songs in our case) at once.
The other reason we'd like it is for real-time queue syncing. We can listen for adds,removes,reset and forward those events over the wire instead of having to forward potentially tons of the single events.
In response to the question above, we could just reconcile after every event (if I'm understanding you correctly) but some people have thousands of songs in their library/collection and always doing that loop (even when its not necessary) seems kinda crappy. I know looping isn't very expensive but if we can do it just based on change events I'd rather do that.
I also wouldn't be asking for this if the change to backbone was large or complicated.

Talked to @akre54 on gitter and we decided that not enough people would benefit from this for it to be worth adding to core. I'm stripping out the coll.remove changes though and making a new PR with just those.


md-ink-bar not visible using

Bug, feature request, or proposal:
Bug
What is the expected behavior?
md-ink-bar should be visible
What is the current behavior?

md-ink-bar is not visible
What are the steps to reproduce?
http://plnkr.co/edit/qXJdsDIr5pwXxCdbRyLL
The plunker includes a simple example, however I could not get the router to work correctly, because I am not familiar with plunker.
What is the use-case or motivation for changing an existing behavior?
<nav md-tab-nav-bar> should yield the same presentation as <md-tab-group>
Which versions of Angular, Material, OS, browsers are affected?
Tested on angular 2.3 and 2.4, material2 2.0.0-beta.1,  Chrome v56
Is there anything else we should know?
nope

This should be fixed by #3041


Repo sync for protected CLA branch

The pull request is created from master637357179179556728 to master to fix git push error for protected CLA branch

@huypub : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix the incorrect code sample

@tony-xia : Thanks for your contribution! The author, @mrbullwinkle, has been notified to review your proposed change.

@tony-xia Thank you for helping to make our docs better!

#sign-off


Ability to use xls/etc options via CLI

Hi,
first of all thank you for this tool. I would like to use xls options via cli, in example sheet or fill_merged_cells.
How to do it via command line?
Thank you

Please preserve this line to notify @roll (lead of this repository)

Hi @aborruso,
it's not exposed yet to the CLI.
Would you like to PR cli.py?

Hi @roll  unfortunately I'm not able to do it :(
Without it, it is not possible to use it properly coupled with multi sheet files. You know this well, I know. But it seems to me important to emphasize it.
Thank you

Hi @aborruso
No problem. Let me check if I can quickly jump on it today (should be a very simple update)

WOW, thank you @roll


Install go automatically when the app is deployed the first time

Would be nice if go is automatically installed during deployment. This would allow users to just git push a .go file and everything is wired up for them in the cloud.

Agreed. My plan for this has been to use the site extension I've built and have people deploy using a resource manager script. Thoughts?
http://www.wadewegner.com/2015/01/creating-a-go-site-extension-and-resource-template-for-azure/

Take a look at the new tutorial and steps. I think this is as easy as it gets until the Go bits are pre-deployed on the machine.


Nuget Package should be using a license expression

Summary
I wish for the nuget packages to have the licence expression property set correctly
Details
The licence expression property should be set to the correct licence type I.e. Apache-2.0 as this will enable analysis of licences in use to occur in external tools & the license type will be shown in Nuget etc.

Hi @thompson-tomo and thanks for your interest in this project!
I have no objection to adding it, do you want to send a PR to fix this aspect?
cc. @baywet


Example game packages an incompatible plasma

Description
It seems like recent changes to the plasma slider have packaged an incompatible plasma version that complains about missing initial fields on the slider widget.
See: https://discord.com/channels/385151591524597761/402843211707449354/1211850232036462652

We can't seem to replicate this issue. Closing for now.

We can't replicate this bug. The example game depends on the newest dependency of plasma at v0.4.3. It shouldnt be possible to have that bug since we do not index at line 37


Inconsistencies between functions for option flags

Problem:  Inconsistencies between functions for option flags.
Solution: Consistently use "unsigned int" as return type and rename
get_bkc_value() to get_bkc_flags().

Hm, personally I don't like dropping the int from unsigned int, since the latter is more explicit. It seems this is not used consistently here in the code base (sometimes with the int, sometimes without).

Hmm, then I'll drop those changes.

thanks!


Visual Studio 2017 support

Hello. Will MSVC 2017 support? I tried compile, but not compiling.

Hey i guess MSVC 2017 has not implemented most of C++17 yet, so it will not work for the moment. As soon as Microsoft ships a solid C++17 compiler I will probably see to it.


Any way to make the linter realize that the libs.versions.toml entry is NOT

Not sure how this got duplicated while I was typing. Sorry for the extra noise.


src/js/services/configService.js many changes needed

Moved to https://github.com/BitLox/copay/issues/10


Does this work with OAuth 2's implicit grant and Resource Owner Password Credentials ?

I am looking through this library, and while it does say it supports OAuth facebook and twitter login, it doesn't explicitly say if it supports the other types of OAuth. Is this library usable with those few things?

@moneytree-doug - the OAuth stuff happens on the server. I made a rails gem for this project that supports OAuth using omniauth. omniauth supports these strategies.

@moneytree-doug - closing this issue, please re-open if I haven't answered your question.

@lynndylanhurley Its not exactly what I was looking for, but I appreciate the answer! Thanks!


Resolved errors related to saving video from depth images

There was no problem saving the video from the rgb images, but an error occurred regarding saving the video from the depth images, so the code was modified.

Thanks!


uninstall

Hi!
How do I uninstall svviz?
Thanks,
Rita

This depends on how you installed it. Assuming you used pip, it'd be sudo pip uninstall svviz

Many thanks,
Rita

On 14 Oct, 2016, at 14:51, Noah Spies<EMAIL_ADDRESS>wrote:
This depends on how you installed it. Assuming you used pip, it'd be sudo pip uninstall svviz
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/svviz/svviz/issues/45#issuecomment-253804506, or mute the thread https://github.com/notifications/unsubscribe-auth/AK28CJoaKUGCHx0qG07XeFE4XUCBtYO_ks5qz4jygaJpZM4KWy6t.


No Connection Adapters error

Octoprint.log: https://gist.github.com/LewisSpring/a4938e7584959d5ad5b97ff92a6c50b9
OctoPrint 1.5.3 Python 3.7.3 OctoPi 0.18.0
I've tried HTTP and HTTPS ports
WebDAV hosted on Synology DSM

Turns out i'm not supposed to use the port number in the address:
lay it out like "https://<IP_ADDRESS>/path/to/folder"

Turns out i'm not supposed to use the port number in the address:
lay it out like "https://<IP_ADDRESS>/path/to/folder"


Json schema for config files

There are editors and command line validation tools that support json schema for nice auto-complete and validation of json files. It's often useful to provide a json schema when your utility takes a json file as input, as HTMLHint does with the .htmlhintrc json configuration files. I made a schema file for the .htmlhintrc file which includes HTMLHint specific options. It sets "additionalProperties" to true to allow jshint and csshint options, but doesn't include schema for them.

This file used for IDE?
Can you tell me, the name of ide support the Json schema.

@yaniswang Visual Studio Code, Visual Studio, WebStorm, Atom, ...

Support ide is too little!

Well, now when Visual Studio, Atom and WebStorm took more than half of market, I can't say that support is too little.


Snackbar calls setState on timeout, even if unmounted

The Snackbar component seems to be calling setState internally, when the timeout fires, even if the component is by then unmounted (possibly in the ActivableRenderer HOC)
The component should cancel the timeout on unmounting (componentWillUnmount).

Can you provide an example where this happens??

I will provide a proof of concept shortly

Ok, thanks!!