There can be license-specific output lines before parameters (e.g., WLS)

This doctest fails with a WLS license.

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

Hmm, without WLS the fix fails again. I thought that "..." includes no line, obviously not.

please note that only the first commit is relevant for this issue.

Hmm, without WLS the fix fails again. I thought that "..." includes no line, obviously not.

Yes it's annoying finnicky that ellipsis can't match nothing. I couldn't find a way around yet, but surely there's an option we're missing.

I did a quick search but found nothing. closing this PR for now until we found a useful solution.


AttributeError: type object 'CogVideoXImageToVideoPipeline' has no attribute 'save_lora_weights'

  File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 967, in <module>
    main(args)
  File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 876, in main
    CogVideoXImageToVideoPipeline.save_lora_weights(
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: type object 'CogVideoXImageToVideoPipeline' has no attribute 'save_lora_weights'
[rank0]: Traceback (most recent call last):
[rank0]:   File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 967, in <module>
[rank0]:     main(args)
[rank0]:   File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 876, in main
[rank0]:     CogVideoXImageToVideoPipeline.save_lora_weights(
[rank0]:     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[rank0]: AttributeError: type object 'CogVideoXImageToVideoPipeline' has no attribute 'save_lora_weights'

I kept getting this with tests, when I did a long run it looked like this when checkpointing
[2024-10-20 13:27:18,842] [INFO] [logging.py:96:log_dist] [Rank 0] Saving model checkpoint: ./resultstenti2v/cogvideox-lora/checkpoint-500/pytorch_model/mp_rank_00_model_states.pt
[2024-10-20 13:27:18,842] [INFO] [torch_checkpoint_engine.py:21:save] [Torch] Saving ./resultstenti2v/cogvideox-lora/checkpoint-500/pytorch_model/mp_rank_00_model_states.pt...
[2024-10-20 13:27:59,017] [INFO] [torch_checkpoint_engine.py:23:save] [Torch] Saved ./resultstenti2v/cogvideox-lora/checkpoint-500/pytorch_model/mp_rank_00_model_states.pt.
[2024-10-20 13:27:59,097] [INFO] [torch_checkpoint_engine.py:21:save] [Torch] Saving ./resultstenti2v/cogvideox-lora/checkpoint-500/pytorch_model/bf16_zero_pp_rank_0_mp_rank_00_optim_states.pt...
[2024-10-20 13:27:59,961] [INFO] [torch_checkpoint_engine.py:23:save] [Torch] Saved ./resultstenti2v/cogvideox-lora/checkpoint-500/pytorch_model/bf16_zero_pp_rank_0_mp_rank_00_optim_states.pt.
[2024-10-20 13:27:59,962] [INFO] [engine.py:3536:_save_zero_checkpoint] zero checkpoint saved ./resultstenti2v/cogvideox-lora/checkpoint-500/pytorch_model/bf16_zero_pp_rank_0_mp_rank_00_optim_states.pt
[2024-10-20 13:27:59,962] [INFO] [torch_checkpoint_engine.py:33:commit] [Torch] Checkpoint pytorch_model is ready now!
Traceback (most recent call last):
  File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 967, in <module>
    main(args)
  File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 794, in main
    accelerator.save_state(save_path)
  File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/accelerate/accelerator.py", line 3081, in save_state
    hook(self._models, weights, output_dir)
  File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 391, in save_model_hook
    raise ValueError(f"unexpected save model: {model.__class__}")
ValueError: unexpected save model: <class 'deepspeed.runtime.engine.DeepSpeedEngine'>```

Need to install diffusers from the main branch for the CogVideoXImageToVideoPipeline to be able to load/save lora weights. DeepSpeed related fixes were made in #38, so be sure to pull latest changes. Also completed a full training run here, so not sure why you see the DeepSpeed error

Need to install diffusers from the main branch for the CogVideoXImageToVideoPipeline to be able to load/save lora weights. DeepSpeed related fixes were made in #38, so be sure to pull latest changes. Also completed a full training run here, so not sure why you see the DeepSpeed error

I am now on latest pull and using the provided prepare_dataset.py with --use_tiling, --save_image_latents, --save_latents_and_embeddings
I'm now getting this memory_allocated=20.011 GB max_memory_allocated=20.011 GB max_memory_reserved=20.027 GB ***** Running training *****   Num trainable parameters = 132120576   Num examples = 59   Num batches each epoch = 59   Num epochs = 1   Instantaneous batch size per device = 1   Total train batch size (w. parallel, distributed & accumulation) = 1   Gradient accumulation steps = 1   Total optimization steps = 10 Steps:   0%|                                                                         | 0/10 [00:00<?, ?it/s]Traceback (most recent call last):   File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 978, in <module>     main(args)   File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 652, in main     for step, batch in enumerate(train_dataloader):   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/accelerate/data_loader.py", line 547, in __iter__     dataloader_iter = self.base_dataloader.__iter__()                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 440, in __iter__     return self._get_iterator()            ^^^^^^^^^^^^^^^^^^^^   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 388, in _get_iterator     return _MultiProcessingDataLoaderIter(self)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 1083, in __init__     self._reset(loader, first_iter=True)   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 1116, in _reset     self._try_put_index()   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 1350, in _try_put_index     index = self._next_index()             ^^^^^^^^^^^^^^^^^^   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 620, in _next_index     return next(self._sampler_iter)  # may raise StopIteration            ^^^^^^^^^^^^^^^^^^^^^^^^   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/sampler.py", line 288, in __iter__     for idx in self.sampler:   File "/home/cubeyai/cogvideox-factory/training/dataset.py", line 410, in __iter__     self.buckets[(f, h, w)].append(data)     ~~~~~~~~~~~~^^^^^^^^^^^ KeyError: (49, 720, 960) [rank0]: Traceback (most recent call last): [rank0]:   File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 978, in <module> [rank0]:     main(args) [rank0]:   File "/home/cubeyai/cogvideox-factory/training/cogvideox_image_to_video_lora.py", line 652, in main [rank0]:     for step, batch in enumerate(train_dataloader): [rank0]:   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/accelerate/data_loader.py", line 547, in __iter__ [rank0]:     dataloader_iter = self.base_dataloader.__iter__() [rank0]:                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [rank0]:   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 440, in __iter__ [rank0]:     return self._get_iterator() [rank0]:            ^^^^^^^^^^^^^^^^^^^^ [rank0]:   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 388, in _get_iterator [rank0]:     return _MultiProcessingDataLoaderIter(self) [rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [rank0]:   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 1083, in __init__ [rank0]:     self._reset(loader, first_iter=True) [rank0]:   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 1116, in _reset [rank0]:     self._try_put_index() [rank0]:   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 1350, in _try_put_index [rank0]:     index = self._next_index() [rank0]:             ^^^^^^^^^^^^^^^^^^ [rank0]:   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/dataloader.py", line 620, in _next_index [rank0]:     return next(self._sampler_iter)  # may raise StopIteration [rank0]:            ^^^^^^^^^^^^^^^^^^^^^^^^ [rank0]:   File "/home/cubeyai/cogvideox-factory/vvv/lib/python3.12/site-packages/torch/utils/data/sampler.py", line 288, in __iter__ [rank0]:     for idx in self.sampler: [rank0]:   File "/home/cubeyai/cogvideox-factory/training/dataset.py", line 410, in __iter__ [rank0]:     self.buckets[(f, h, w)].append(data) [rank0]:     ~~~~~~~~~~~~^^^^^^^^^^^ [rank0]: KeyError: (49, 720, 960)

This is because the bucket sampler initializes its internal dictionary using the values provided in --height_buckets, --width_buckets and --frame_buckets. If you've run prepare_dataset.py at 49 x 720 x 960 resolution, please make sure to pass the same buckets configuration to the training scripts too. Maybe will make this clearer in the README soon

Is now working, thank you. is training at 720x960 lead to better quality?


Update cats-effect, cats-effect-std to 3.2.2

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-std

from 3.1.1 to 3.2.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (3.1.1).
You might want to review and update them manually.
build.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

labels: library-update, semver-minor, old-version-remains

Superseded by #333.


[cherry-pick] [XIONE-3049, DELIA-48850] Merge pull request #955

Adding support for more resolutions

https://github.com/rdkcentral/rdkservices/pull/975 merged. Abandonning this


Change to GPL compatible license?

I was wondering if there is any particular reason the Ms-PL was picked as the license. Would it be possible to change to a GPL-compatible permissive license like MIT, so unfortunate people who have been forced into the GPL license can use this awesome library?

Sadly there isn't - we would have to get permission from all 105 contributors, which is overly difficult

Wait, this is owned by the .NET Foundation now, 🤔

I wouldn't be against changing to the MOST license. Will need to speak with .NET foundation and involve legal thou for guidance.

Yo. @reactiveui/dotnet-foundation can we please start motions to change ReactiveUi's license to MIT? What's involved and what do you need from us.
.cc @reactiveui/owners

This would be handy for those of us who work at large corporates. At my company MIT/Apache 2 is on our automatic pre-approved distribution license list so would make the library much easier to work with.

Good suggestion, I'll review and update the issue on Monday (I'm traveling and speaking over the weekend). High level sounds do-able, I'll need to check no the process to change the license.

Should have answer in the next 24-48hours.

Considering there has been no community objection, we're okay from .NET Foundation side. Ready when you are.

Fantastic! Thanks a bunch.


Higher requirements for NSLocationWhenInUseUsageDescription

When uploading app to App Store, Apple did not accept the simple Usage Description provided here. Maybe it would be a good idea to add a note, that people should write something about why they are actually requesting position.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f0acf3fa-b705-f0cc-b676-eeca60176dfa
Version Independent ID: 1788a5d5-a3cb-15c8-927c-b6a24810245a
Content: Xamarin.Essentials: Geolocation - Xamarin
Content Source: docs/essentials/geolocation.md
Product: xamarin
Technology: xamarin-essentials
GitHub Login: @jamesmontemagno
Microsoft Alias: jamont

See: https://github.com/MicrosoftDocs/xamarin-docs/pull/1802


Tables: CombiTimeTable with spline interpolation and linear extrapolation

Reported by beutlich on 14 Mar 2013 12:34 UTC
Consider the following table model
model Unnamed
  Modelica.Blocks.Sources.CombiTimeTable t_old(
    table=[0.3,0;0.4,1;0.5,0;0.6,-1;0.7,0],
    smoothness=Modelica.Blocks.Types.Smoothness.ContinuousDerivative);
end Unnamed;

Should the extrapolated values of intervals [0,0.3) and (0.7,1] be taken from the boundary spline polynoms or from the linear extrapolation?

Migrated-From: https://trac.modelica.org/Modelica/ticket/1028

Comment by otter on 26 Mar 2013 10:33 UTC
According to the documentation of CombiTimeTable (and also to common sense, since extrapolation with polynomials is usually not what the user desires), the correct behavior is linear extrapolation outside of the definition domain (the Dymola behavior is an error).

Comment by dietmarw on 26 Mar 2013 19:17 UTC
This ticket is invalid as it's really an issue of Dymola and not the MSL. And hence there was nothing to fix here.

Modified by dietmarw on 26 Mar 2013 19:18 UTC

Comment by beutlich on 26 Mar 2013 20:38 UTC
The expected behavior (see t_new.y[1] above) is already implemented in the revised implementation of the MSL tables (and I guess will make it in MSL 3.2.1).

Comment by hansolsson on 19 Jun 2013 10:25 UTC
Looking more I believe that neither the previous behavior nor the current one is the correct one (yes, should have found this earlier).
The smoothInterpolation guarantees that you have continuous first order derivatives inside the interval; and it seems natural to have at least a choice for extrapolation that guarantees this for the entire interval.
If you want a first order extrapolation outside the interval the correct approach would naturally be to keep the derivative-value at the end-point.
Modelica.Blocks.Sources.CombiTimeTable combiTimeTable1(table=[0.0,8; 1,-1; 2,
        6], smoothness=Modelica.Blocks.Types.Smoothness.ContinuousDerivative);

simulate 0 to 3 seconds and note that we get a discontinuous first order derivative at time=2.
[[Image(DerTest.png, 600px)]]

Comment by beutlich on 19 Jun 2013 10:48 UTC
At least the current behaviour is correct according to the documentation
Extrapolate linearly through the last two table points outside of the table scope

and the name Extrapolation.LastTwoPoints. ;-)
So if I will try to fix this, should we add a new option LastPointAndSlope to Extrapolation enum or handle this implicitly?

Modified by beutlich on 19 Jun 2013 11:13 UTC

Comment by beutlich on 20 Jun 2013 15:04 UTC
attachment:T1028.patch solves this extrapolation issue on source level for all four external tables. 1D case is pretty clean, but 2D case really is a mess. I am open for any better implementation suggestions. That's the reason I did not yet commit the changes. Thank you.
TODO:

Comment correction of Extrapolation.LastTwoPoints
Documentation adaptions

Comment by beutlich on 21 Jun 2013 13:49 UTC
Resolved by aa9a4ae54580586b4147653fedf02ff6de98488d (1D case), 7b1c1a5f5ae0507be71d6c2acd6cfb691377a9c4 (2D case) and ddf5648f171310577284f24010de96943dbbbff7 (comment, doc).

Modified by otter on 26 Jun 2013 09:18 UTC


How to modify FGD to run yolox-s with yolox-l as the teather?

How to modify FGD to run yolox-tiny with yolox-l as the teather? In FGD/configs/distillers/fgd, I can't find the config of yolox.
Thank you so much!

Please read README. The config is in branch yolox. Besides, Yolox-tiny has been fully trained, it is hard to benefit it more with distillation


Block/Unblock Users logic change

The logic for adding suspicious users to black list must be slightly changed.
as for now: we check wether all roles for users were deleted.
as must be: check wether all users were blocked (mithril.users.is_blocked=true) (there is no verification for roles)

@danakonovalenko changed & deployed


fix(image-annotator-react): add custom clear confirm

:tada: This PR is included in version 1.3.0-alpha.3 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.2.3-alpha.3 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-alpha.14 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0-alpha.15 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:


Run aware test in test-support path

This should merge the test-support/* with the consumers test/helpers directory, enabling users to take advantage of this run loop aware test function.

@snewcomer first I'm curious what we gain by altering that array test I see in the diff. Could we do without that change? Was it for any specific reason (consistency/ etc) or just a one off?
Also - am I correct to assume the unit tests use this helper? 2 examples below
https://github.com/toranb/ember-cli-simple-store/blob/master/tests/unit/arrays-test.js#L4
https://github.com/toranb/ember-cli-simple-store/blob/master/tests/unit/store-test.js#L5
If we move this to test support how does this still pull in from the dummy app? Just curious :)
Also -could we land on a name that is store specific (instead of plain ol' qunit) ? Maybe just using a namespace folder like simple-store first would help avoid some crazy conflict. In that case the import I'm thinking of would look something like ...
import { test } from 'my-app/tests/helpers/simple-store/qunit'

I like that idea.  Another option I found is under addon-test-support.  ember-native-dom-helpers uses this pattern.  So now I would just import ember-cli-simple-store/test-support/qunit, which I think is exactly what you were going for above.
Also added a comment in the test-support file about passing the env to the callback.apply in order to solve the case where we need to maintain the test scope - i.e. this.store setup in the beforeEach.  So if you delete that line let env = , the array-test will fail.
And I think your question is about how does the test-support get utilized for a test run?  In both the addon/external application, the test-support gets merged automatically with the /tests folder.

@snewcomer love the naming choice of addon-test-support you are using here. The only question that remains is that callback.apply mention (why you had to use this.store in the array test). Am I correct in assuming that because this test helper (qunit) was never public api we are free to alter this and push a v5.8.0 instead of something more drastic like v6.0.0 (ie: breaking api change)
If this is the case -anything else we should consider before merging this and doing a release?

@toranb yeah I believe it is true that this was never a public api.  So without passing testEnvironment it will not apply any context from, say the beforeEach which may setup this.store/this.ticket etc.  So that is what I tried to emulate in the array test.
And I think given our test suite, we had something very similar you did for us before.  I refactored it to exactly what it looks like in this PR and all tests passed!  So if our test suite was an indication, I think 5.8 is good if you are good with it.  (right now our master is pointing at this PR).

Sorry actually I believe the previous /tests/helpers/qunit was merged into the user's test folder...if they knew it was there.  So technically it was public.  So upgrading would require a path change is all.

@snewcomer ah in that case ... could we do a quick deprecation + export to stay backward compat friendly? Example below of how we did this for another addon I'm supporting
https://github.com/ember-redux/ember-redux/blob/release/2x/addon/components/connect.js
also -should we avoid the refactor'd helper now that we are trying to stay backwards compatible? Still trying to understand that this.store change in the array test and if the older version of this qunit function would have left that test as-is (no change)

@toranb Ah great idea!  Added.
So the qunit helper as-is would not work for any test that assigned variables to the testEnvironment.  The original helper would not work for any test that assigned variables to this in the beforeEach.  Thus if this PR only changed the array-test to use this.store, it would actually fail using the existing helper.  So I see it as a +1 👍.  Lmk if you have other thoughts!
In terms of backwards compat, it did two things: 1. added support for the testEnvironment. and 2. refactored to use es6.  Given we replaced our test suite with the new helper, I am led to believe that this should be entirely backwards compat, but for the change in path addon-test-support.

@snewcomer thanks for all the work on this dude! I just tagged/published v5.8.0


Post body images not responsive.

Describe your problem
I noticed that while inserting an image into a page, it dose not automatically resize to fit the screen, instead it remains bigger and the actual text is wrapped, resulting in wide screen half white half text. Not sure is that CSS related or CMS related.
Bludit version
3.4.0     Mediumish 1.1

Would be good to see an example, can you provide some web test or image ?

The theme is to blame here. Bludit itself doesn't add width and height attributes to image tags when inserting them, and even if it did, that wouldn't account for different devices. It's up to the theme to specify a CSS property of max-width: 100% or some such for images added to a post. Blekathlon and Blog-X get it right, for instance. Striped also fails.


Add option to set STAR memory limit

Hi,
I've been trying out this package, I think it's a very nice idea!
I've run into the following issue: by default, STAR limits the memory size to 30 GB. If it passes that treshold, it exists with the message:
Apr 01 14:43:41 ..... Started STAR run
Apr 01 14:43:41 ... Starting to generate Genome files

EXITING because of FATAL ERROR: limitGenomeGenerateRAM=31000000000 is too small for your genome
SOLUTION: specify limitGenomeGenerateRAM not less than<PHONE_NUMBER>0 and make RAM available

However, if I run imfusion with  --limitGenomeGenerateRAM, this option causes an error in the Python parser and is not passed on to STAR.
As a workaround, I added this option to /usr/lib64/python3.5/site-packages/imfusion/external/star.py:
--- star.py.backup      2018-05-10 12:04:31.000000000 -0000
+++ star.py     2018-05-09 13:51:38.000000000 -0000
@@ -33,7 +33,8 @@ def star_index(fasta_path,
     args = [
         'STAR', '--runMode', 'genomeGenerate', '--genomeDir', str(output_dir),
         '--genomeFastaFiles', str(fasta_path), '--sjdbGTFfile', str(gtf_path),
-        '--sjdbOverhang', str(overhang), '--runThreadN', str(threads)
+        '--sjdbOverhang', str(overhang), '--runThreadN', str(threads),
+        '--limitGenomeGenerateRAM', '50000000000'
     ]

     run_command(args=args, log_path=log_path)

It would be nice if imfusion would pass additional command-line parameters to STAR or tophat on the command-line instead.

Thanks for the tip, I'll have a look at including the extra parameter!


Add support to connect multiple wallets

Added WalletConnect which supports most of the widely used wallets. Thus, closing this for now and will integrate more wallets as required


How to attach UISearchController

Hi Ahmed,
I'd like to attach a UISearchController to a PaginatedTableView, is it the right way ?
In delegate search result update method, save search text for future call of load data, and send a loadData with refresh true.
func updateSearchResults(for searchController: UISearchController) {
if let searchText = searchController.searchBar.text {
        self.currentSearchText = searchText
        tableView.loadData(refresh: true)

    }
}

Use currentSearchText in loadMore ?\
Thanks for help

Yes that is how you will do this, I hope it works fine.


Revert "Standardize the FAQ"

Reverts mhsengineering/mhsengineering.github.io#15
This fixes the site and makes it readable again until #16 is merged.
Tested locally and everything is back to normal.
/cc @johntheawesomeguy, @threedot14 (not waiting for review because this is pretty major)

Because many people seem to be merging pull requests without waiting for feedback, it may be helpful to implement a service that requires another to review a pull request before allowing it to be merged.
I recently discovered PullApprove which does just this. I don't know much about it, other than it does exactly what we want, and am reluctant to give read/write access to our repos to a website that isn't in the mainstream line of Github applications. Still, it might be worth looking into.

sorry my bad, I didn't think it necessary because I was only matching it
with the rest of the site and had already tested it locally
On Tue, Dec 8, 2015 at 7:44 PM, Eric Roberts<EMAIL_ADDRESS>wrote:

Because many people seem to be merging pull requests without waiting for
feedback, it may be helpful to implement a service that requires another to
review a pull request before allowing it to be merged.
I recently discovered PullApprove https://pullapprove.com/ which does
just this. I don't know much about it, other than it does exactly what we
want, and am reluctant to give read/write access to our repos to a website
that isn't in the mainstream line of Github applications. Still, it might
be worth looking into.
—
Reply to this email directly or view it on GitHub
https://github.com/mhsengineering/mhsengineering.github.io/pull/18#issuecomment-163069533
.


GCP docs have random HTML tags strewn about

For example:

Obviously this is not great - we should figure out what's going on here.

Fixed by https://github.com/pulumi/docs/pull/886.


Fix namespace of invoked clobber task

Hey, when using webpacker_lite in my rails project I recognized rake assets:clobber failed bacause of a wrong namespace in the clobber task of webpacker_lite.
I made a small fix which worked for me locally.

This change is

@dpuscher Awesome!
Be sure to see the latest updates to the README!


TensorFlowOnSpark  How does it end?

        with tf.train.MonitoredTrainingSession(master=server.target,
                                               is_chief=(task_index == 0),
                                               checkpoint_dir=logdir,
                                               hooks=[tf.train.StopAtStepHook(last_step=100)],
                                               save_checkpoint_secs=60,
                                               config=tf.ConfigProto(log_device_placement=True)) as sess:

Last_step does not stop the distributed task until it arrives,  it keeps running.
--epochs=50
cluster.train(dataRDD, args.epochs)

But when the data is finished, the state of spark is done

I think that's just due to how epochs are implemented in InputMode.Spark.  Since Spark doesn't have a notion of repeating RDDs, we actually implement epochs as a Spark UnionRDD of epochs copies of the RDD.  This, combined with the fact that Spark doesn't support early termination of an RDD.mapPartitions() operation (used to feed data to TF), means that the entire UnionRDD must be completed "processed" in order for Spark to successfully complete the operation.  So, in this mode, even though the TF side has "completed" by steps, the Spark side must still "process" the entire dataset... it just does so by consuming the iterator but skipping the feeding part.
So basically, you should try to match your steps to your epochs fairly closely to minimize this overhead.  As an alternative, you can use InputMode.TENSORFLOW and just use TensorFlow's native support for reading files from HDFS directly.

ok  ,Thank you very much for your patient explanation. i get it


Failed to download media file.

Hi. Thank you for very useful Library.
Error will occur when gatsby develop commands.
related isuue https://github.com/gatsbyjs/gatsby-source-wordpress-experimental/issues/53.
Environment to reproduce
Repo: https://github.com/kawamataryo/practice-gatsby-source-wordpress-experimental
Wordpress URL: https://makoto-acu.com
Wordpress: v5.51
gatsby-source-wordpress-experimental: 1.4.4
WP Gatsby: 0.4.17
WP GraphQL: 0.12.3
log
success open and validate gatsby-configs - 0.037s
success load plugins - 0.780s
success onPreInit - 0.033s
success initialize cache - 0.014s
success copy gatsby files - 0.041s
success onPreBootstrap - 0.021s
success  gatsby-source-wordpress  ingest WPGraphQL schema - 0.857s
success createSchemaCustomization - 1.668s
success  gatsby-source-wordpress  Category - 0.372s - fetched 4
success  gatsby-source-wordpress  Comment - 0.155s - fetched 0
success  gatsby-source-wordpress  ContentType - 0.239s - fetched 3
success  gatsby-source-wordpress  MenuItem - 0.403s - fetched 24
success  gatsby-source-wordpress  Menu - 0.248s - fetched 3
success  gatsby-source-wordpress  Page - 0.360s - fetched 8
success  gatsby-source-wordpress  PostFormat - 0.163s - fetched 0
success  gatsby-source-wordpress  Post - 1.631s - fetched 70
success  gatsby-source-wordpress  Tag - 0.207s - fetched 0
success  gatsby-source-wordpress  Taxonomy - 0.248s - fetched 3
success  gatsby-source-wordpress  UserRole - 0.124s - fetched 0
success  gatsby-source-wordpress  User - 0.347s - fetched 2
info pushing /wp-content/uploads/2017/03/5642fae96ade14100be5-1.jpg to the end of the request queue.
info Previously retried 2 times already.
info pushing /wp-content/uploads/2017/02/466bd28e5ab06680a011.jpg to the end of the request queue.
info Previously retried 2 times already.
info pushing /wp-content/uploads/2017/03/5642fae96ade14100be5-1.jpg to the end of the request queue.
info Previously retried 3 times already.
info pushing /wp-content/uploads/2017/02/466bd28e5ab06680a011.jpg to the end of the request queue.
info Previously retried 3 times already.
info pushing /wp-content/uploads/2017/03/5642fae96ade14100be5-1.jpg to the end of the request queue.
info Previously retried 4 times already.
info pushing /wp-content/uploads/2017/02/466bd28e5ab06680a011.jpg to the end of the request queue.
info Previously retried 4 times already.
info

already re-queued /wp-content/uploads/2017/03/5642fae96ade14100be5-1.jpg 5 times :( sorry.
Try lowering process.env.GATSBY_CONCURRENT_DOWNLOAD.
It's currently set to 1

 ERROR

url passed to create-remote-file-node is either missing or not a proper web uri: /wp-content/uploads/2017/03/5642fae96ade14100be5-1.jpg

not finished source and transform nodes - 300.573s
not finished  gatsby-source-wordpress  fetching nodes - 300.547s - 119 total
not finished  gatsby-source-wordpress  creating nodes - 295.972s
not finished  gatsby-source-wordpress  MediaItem - 295.973s - fetched 2

I set GATSBY_CONCURRENT_DOWNLOAD=1. but I get this error.
Is there a solution?
Thanks.

I'm having the same problem after updating from 1.4.0 to 1.4.4. @kawamataryo downgrading to 1.4.0 helps in my case.

@gustavpursche
Thank you for your comment.
I tried downgrading to 1.4.0.
$ yarn remove gatsby-source-wordpress-experimental
$ yarn add<EMAIL_ADDRESS>
But, I got same error :_(

Same here but after downgrading to 1.4.1 in the package.json (don't forget to do a yarn after that) it worked !

Hey, everyone, this regression was introduced when fixing a bug related to JSON fields which have image tags in embedded html. I've added a bunch of inline html gatsby image integration tests, but haven't yet added one for relative image paths like "/wp-content/uploads/2017/03/5642fae96ade14100be5-1.jpg". I'll get this fixed asap and add a test for this to prevent future regressions. Thanks for reporting this!

This is fixed in<EMAIL_ADDRESS>Please feel free to re-open or comment here if this didn't solve your issue. I tested this on the provided repo/gql api and it worked great. This bug also uncovered a couple other semi-related bugs which always nice :)

@TylerBarnes
wow!! great!! It worked.  I’m grateful to you :)

Hi @gustavpursche and @enx1998 it appears you're encountering a separate issue. The original issue was related to media items that had paths stored as source urls (like /wp-content/uploads/2020/09/jmm-1.png).
Can you open a new issue with a reproduction I can use to debug?


X.W

Am i crazy or does the matrix multiplication in this slide not work given that the nrows = 100 and ncols = 10???
matrixMultiply.pdf

nrows=784, ncols=784 so yes, the matrix multiply works
On Fri, 6 Sep 2019 at 00:03, Ragy Ibrahim<EMAIL_ADDRESS>wrote:

Am i crazy or does the matrix multiplication in this slide not work given
that the nrows = 100 and ncols = 10???
matrixMultiply.pdf
https://github.com/GoogleCloudPlatform/tensorflow-without-a-phd/files/3582563/matrixMultiply.pdf
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/tensorflow-without-a-phd/issues/45?email_source=notifications&email_token=AAHKKZ4QRCJUQSVPNLQ52G3QIH6FTA5CNFSM4IUFWSTKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HJXKYMA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHKKZZOWNWJIWBXI6Q745LQIH6FTANCNFSM4IUFWSTA
.

--
Martin Görner | Developer Relations |<EMAIL_ADDRESS>| +1<PHONE_NUMBER>


Implement single-level lookups on RM3754

Single-level lookups
Add FDL for RM3754, which is the first framework to have a one-level lookup implemented by a case_insensitive_inclusion validator.
Define the lookup with a lookup block like this:
Lookups {
  PaymentProfile [
    'Monthly'
    'Quarterly'
    'Annual'
    'One-off'
  ]
}

Then reference this lookup as a user-defined lookup type like this:
optional PaymentProfile Additional2 from 'Payment Profile'

Add support for an integer field
This framework has our first non-string type; implement it
Prove that the validations from this FDL result in no errors compared to the existing class for our current submissions
➜  DataSubmissionServiceAPI git:(898-lookups-rm3754) ✗ be rake fdl:validations:test[RM3754,9000]
W, [2019-03-12T12:22:23.841419 #29661]  WARN -- Skylight: [SKYLIGHT] [4.0.0-alpha] Running Skylight in development mode. No data will be reported until you deploy your app.
(To disable this message for all local apps, run `skylight disable_dev_warning`.)
[####################################################################################################] [8761/8761] [100.00%] [03:25] [00:00] [  42.56/s]

Rake task fdl:transpile[short_name]
Add a rake task fdl:transpile that takes a framework short name and verifies that the FDL transpiles correctly. In the event that it doesn't, print a Parslet error tree showing what went wrong.

@rgarner @lozette Is there any way that the lookups could error if they aren't used within the definition?
If I renamed PaymentProfile to something else and run rake fdl:validations:test[RM3754,10] it still passes because it is an optional field, but the validation isn't actually working properly.

@leeky this comes under the heading of "semantic checks".
We kind of don't have those right now. There's syntax, and then there's ... not much else. Certain things will fail, like if you use a type that's neither a Lookup or a KnownField, you'll get hard failures, but yes, this is under the heading of "stuff to do if there's time"


Feature request: disable label layout magics

My labels are being laid out "smartly" but not to my liking - I'd rather take some minor overlapping. It'd be neat to have an override.

Ah: Theme(label_placement_iterations=0) does this... maybe I could do a PR to advertise this in the geom_label manual page?

Even better, now you can get static label layouts like Geom.label(position=p), where p is one of :left, :right, :above, :below, :centered.

Neat :+1: !


npm install fails without --legacy-peer-deps

I have tried installing v13.1.26.
When running with npm i I get this error message:

Tried on node v16.20.0 and v18.8.0
As explained here it seems to be a problem with peer dependencies, in this case with<EMAIL_ADDRESS>as far as I can understand the error message.
It only worked with npm install --legacy-peer-deps.
@aldefouw I think this is caused by using rather outdated cypress version, that do not keep in sync with peer dependencies. Not sure what is the best way to resolve this issue, also for the future:
Some possibilities that come into my mind:

Upgrade to Cypress 12.X: I know this is a lot of work. Especially, since there are a lot of breaking changes regarding authentication flow.
Switch to yarn instead of npm. Not sure about it, but it seems like to be more stable.

Thank you for your detailed response.
I will try to use latest Node version (with nvm) - I am glad your setup is working stable.
I like the idea to go with BDD a lot.


fix: add AIRFLOW_CONN_SFTP_TEST env var and stfp service inlint-and-test.yml

Description
Fix test adding in inlint-and-test.yml:

STFP service
AIRFLOW_CONN_SFTP_TEST environment variable.

@victor-centelles-dkl I also could not get the tests to work like this. I think we can close this, in my latest PR I used an additional step to run the SFTP container in docker and it seems to work.

@victor-centelles-dkl I also could not get the tests to work like this. I think we can close this, in my latest PR I used an additional step to run the SFTP container in docker and it seems to work.

No problem. I apologize because I have been with other issues and have not been able to work so much time in this PR, but it seems to me correct what you propose. But I think we continue having to add the AIRFLOW_CONN_SFTP_TEST env var, right?

We close this PR because does not solve the problems with tests in GHA (problem with ports).


[Changelog]: Updating to 1.4.7

VSIX to be published (zipped to allow upload to GitHub): vscode-edge-devtools-1.4.7.zip

@mliao95 @hkal, I have this PR targeting a new servicing branch I created where we can cherry-pick any fixes we want to release before the next major update is ready.


Support PUG files (plain text/source code files in the PUG template format)

Glance currently does not provide a preview for PUG files (see https://pugjs.org/api/getting-started.html for details). This is the UTI:
kMDItemContentType = "dyn.ah62d4rv4ge81a7ph"

I would gladly provide a pull request, but I have no idea how to setup the build environment to build and test the change. If you could give me some pointers on how to setup the dev environment, I could also write up a DEVELOPMENT.md file for other contributors.
Anyway, thanks for your great work, it is much appreciated!
Best, Ole.

Hi Ole, there are some instructions in README.md about adding new file types. Building should just be opening the project in Xcode, updating the code signing configurations and clicking the build button.
Unfortunately in this case, it seems that the library we use for highlighting, Chroma, does not support PUG syntax. This means we can't support PUG too.

Hi chamburr, thanks for responding so quickly. You are right, it was just as easy as downloading xcode, opening the project , adjusting the config and rebuilding. And although pug is not supported by the source code highlighting lib you're using it is still very helpful for me to be able to quicklook pug files. So if you are interested in a pull request for pug files (without color coding), just let me know.
Anyway, thanks again, all the best –
Ole.

Given that every other source code file has syntax highlighting support, I don't really want to add pug files without syntax highlighting. Maybe you could look into adding or requesting pug support from the upstream library?


Presence rapidly toggling between states by itself

This issue was first identified as https://github.com/conversejs/converse.js/issues/1117
It now happens frequent enough to be fixed. Reproduce and fix.

In the meantime, modify the auto-presence Timeout slider to go all the way down to zero in order to disable auto-presence for away and xa states.

I was able to reproduce this issue. I have not found the root cause. Running the chrome.idle event handler in the browser extension at the same time as Converse auto-away that was responsible for my reproduced issue. This has been fixed in version 1.1.4.
The hunt continues.

I think I found it. This has been fixed in version 1.1.5


npn not found when building linux/386 based on ubuntu-18.04-v3.0.0-pre.6

Hi
I'd like to run a GUI based container on an old i386 machine running ubuntu-18.04.
Seeing that you have recently started to publish jlesage/baseimage beta images with linux/386 target, I tried to use ubuntu-18.04-v3.0.0-pre.6 this way:

copying versions/ubuntu-18.04 to versions/ubuntu-18.04-v3.0.0-pre.6 changing the tag from ubuntu-18.04-v2.4.5 to ubuntu-18.04-v3.0.0-pre.6
launching the build with ubuntu-18.04-v3.0.0-pre.6 on the i386 machine

However, on the i386 machine itself, the build fails on
https://github.com/jlesage/docker-baseimage-gui/blob/799f7c52011bee4a42b3c3749f2c604fa5c100e9/Dockerfile.debian#L143
with 'npm': No such file or directory
On the amd64 machine, the build work but then fails when starting the tests as bats is not found.
I understand that you are in total rework of both baseimage and baseimage-gui so hopefully this feedback may help.
Let me know if more details would be usefull.
Maybe I should have used the v4 branch ?
Thanks for all this.

Multi-arch images are not quite ready to be used, as development is not terminated yet.
But yes, jlesage/baseimage:ubuntu-18.04-v3.0.0-pre.6 should be used with the v4 branch.
Note that I should start to publish pre-release versions of this v4 very soon.

thanks.
looking forward updates

Official v4 images now available.

Thanks a lot


Editing imported plan does not work on testing-farm

Executor: https://github.com/scikit-build/scikit-build-core/pull/518
Imported plan: https://github.com/packit/tmt-plans/pull/7
Could it be a clash with testing-farm's setting of environment, i.e. the -e @tmt-environment-plans-rpminspect.yaml in
$ /usr/bin/tmt --root .distro -c arch=x86_64 -c distro=fedora-rawhide -c initiator=packit -c trigger=commit run --all --verbose --id /var/ARTIFACTS/work-rpminspectb7tmges9 -e @tmt-environment-plans-rpminspect.yaml provision --how connect --guest <IP_ADDRESS> --key /etc/citool.d/id_rsa_artemis --port 22 plan --name '^/plans/rpminspect$'

Looking at the code, it shouldn't:
https://github.com/teemtee/tmt/blob/7552550ff21ee3864814159be587e4604c125137/tmt/base.py#L1668-L1683
But the log.txt file indicates otherwise 🤔
Or myabe it involves:
https://github.com/teemtee/tmt/blob/7552550ff21ee3864814159be587e4604c125137/tmt/base.py#L1558-L1565

@LecrisUT could you bit expand more what is exactly the problem here pls?

Hi @thrix. Basically, if we call a tmt plan with:
# /plans/rpminspect
plan:
  import:
    url: https://github.com/LecrisUT/tmt-plans
    ref: doc/rpminspect
    name: /plans/rpminspect
environment:
  RPMINSPECT_EXCLUDE: disttag

The configuration in environment is not picked up when it is being executed on testing-farm


[SDK Automation - Java] Improve the logging to be concise and actionable

Refine the error logs to enhance clarity and conciseness, provide guidance on corrective actions, and include relevant document links as necessary. The logging should include three parts information if possible.

error code or error message
next steps or suggested actions
associated document links if have

@weidongxu-microsoft

@raych1
Do you plan to have some consistency on error?
E.g. do you want something consistent, if automation find that tspconfig is not correct.

Part of the logging and "recommendation on failure" is included in the tspconfig validation PR https://github.com/Azure/azure-sdk-for-java/pull/39953

As synced, we will monitor the errors reported in the log and improve them if necessary next two months.


[mempool] rate-limit outgoing broadcasts and concurrent txn validation

Motivation
This PR introduces rate-limiting on outgoing mempool broadcasts.
Running cluster test experiment for high-traffic/DDoS-like scenario showed mempool broadcasts being dropped and high CPU usage. Broadcasting too much also slows down other processes like consensus and mempool processing incoming transactions, so overall the node's performance is negatively impacted.
This rate-limiting functionality:

helps reduce unnecessary broadcasts that are dropped
lowers CPU used in DDoS scenario from 69% to 33%
lowers the traffic to overloaded remote peers (slow responses can be interpreted as that peer being under heavy load)

The rate-limiting config values are tested to ensure optimal protection of CPU/other processes
Test Plan
Cluster test experiment for high-traffic showed CPU usage dropped from 69% to 33%.
Ran other cluster test experiments to ensure no perf regressions are introduced

/land

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

/land


Add support for routes

Hi, @adleong - can you link to any issues or add text to this PR to go into more detail about the intent here? I think that will make it easier to get reviewers - thanks!

Thanks for taking a look, @bridgetkromhout.  I tried to give sufficient motivation and context for this change in the description in traffic-metrics.md.  Please let me know if anything is unclear or requires further explanation.
Due to the repo move, I've lost access to this branch so I have re-opened a replacement PR: https://github.com/servicemeshinterface/smi-spec/pull/129  I also copied the relevant section from the spec into the PR description to make it easier to review.


Warnings from plugin-vue when it tries to resolve xml tags

@vite/plugin-vue tries to resolve the xml+svg paragraph icon as vue template tags. This obviously fails and creates warnings in the dev console of the browser.
I already tried to configure the plugin-vue to exclude xml files but to no avail. Now, I don't have any idea, how to resolve that.
Actually, the implementation of the menu doesn't really looks like how vue components should be implemented IMO. How about using simple <router-link>s with an icon instead of the menu? We don't currently need the features of the <n-menu>, and I think it will never be necessary.

Id not swap the menu on the top (as it works well and looks good), but in the footer, simple router links are probably sufficient.
I'd just not ~waste~ invest any energy into  non-functional stuff (aka "quality") at the moment ;-)

Agreed.
We could also have a look at vite-svg-loader. That would simplify things a lot.

I've updated the footer with #71 - I think we should close this ticket now


Common name (CN) characters validation

Hi All,
I have a question related to the common name (CN) of a x509 certificate.
As per the specification https://datatracker.ietf.org/doc/html/rfc5280, the CN can have only
"The character string type Printable String supports a very basic Latin   character set: the lowercase letters 'a' through 'z', uppercase   letters 'A' through 'Z', the digits '0' through '9', eleven special   characters ' = ( ) + , - . / : ? and space."
But when I use openssl to create the x509 certificate, it will not do any kind of validation on the characters I can use for the CN.
Is there any reason for not validating?. Or am I referring the wrong specification?. Please help to understand.
Thanks,
Pradeep

RFC 5280 is the correct specification, but I believe you are looking at the wrong definition for comonName. The definition for CommonName at the bottom of page 120, which specifies the use of PrintableString, appears to be an extension attribute value, which is part of the definition of the X.400 address syntax.
The CN attribute that may appear in the issuer or subject field of a certificate is defined at the bottom of page 112:
-- Naming attributes of type X520CommonName

id-at-commonName        AttributeType ::= { id-at 3 }

-- Naming attributes of type X520CommonName:
--   X520CommonName ::= DirectoryName (SIZE (1..ub-common-name))
--
-- Expanded to avoid parameterized type:
X520CommonName ::= CHOICE {
      teletexString     TeletexString   (SIZE (1..ub-common-name)),
      printableString   PrintableString (SIZE (1..ub-common-name)),
      universalString   UniversalString (SIZE (1..ub-common-name)),
      utf8String        UTF8String      (SIZE (1..ub-common-name)),
      bmpString         BMPString       (SIZE (1..ub-common-name)) }

As noted in Sections <IP_ADDRESS> and <IP_ADDRESS>, CAs conforming to RFC 5280 may use either PrintableString or UTF8String. (Note that Section <IP_ADDRESS> indicates that the common name attribute is defined in X.520.)

Thanks for the quick answer. So, it's ok to use all the UTFStrings characters for the common name right?

So, it's ok to use all the UTFStrings characters for the common name right?

Yes.

Marking as inactive, to be closed at the end of 3.4 dev, barring further input


Create RFID Server

The RFID server has 3 endpoints:

GET /getRFIDs - gets all RFID users' info
POST /createRFID - toggles between validating and creating
DELETE /deleteRFID - deletes an RFID user

For validating cards, we used pub/sub (AWS-IoT). ESP32 will publish to a topic that Core-v4 subscribes to. The validation results are sent back to ESP32 similarly. For creating new RFID cards, the RFID create API will toggle from validating to creating. So now when ESP32 sends the RFID data, it will create a new RFID user.

[x]  Create APIs to get and delete RFID users
[x]  Create AWS-IOT folder with info
[x]  Create pub/sub to create and validate RFID cards
[x]  Update RFID Schema
[x] Add API tests

I added function documentation for the RFID helper functions and removed all the unnecessary comments. As for the readme.MD file of AWS-IOT dir, I will add that to the Getting Started Page later tonight

Everything looks great.


Who is responsible for the ARL GitHub site

In the DISTRIBUTION METHODS section, it is stated:

The ARL GitHub site is the RECOMMENDED method, but is not the only method.

It is not clear who the PD needs to contact to make that happen. Please be more specific, for example:

Contact Bob at<EMAIL_ADDRESS>to set up a GitHub repo for you.
Contact your XYZ manager, they know who to contact to set up your GitHub account.
Contact ABC department to get access to the USArmyResearchLab GitHub repository so you can create a repo yourself.

Nevermind, the T2O2 is responsible for GitHub release, as specified further below in the document.


Set individual TTL times (Min TTL, Max TLL, Default TTL)?

Looking at  https://github.com/serverless-components/aws-cloudfront/blob/master/lib/getCacheBehavior.js
I see the input is ttl which then gets assigned to all TTL values.
MinTTL: ttl,
DefaultTTL: ttl,
MaxTTL: ttl,

But I want to set MaxTTL only to 31536000

@eahefnawy is this repo active?


Various fixes across whole package

Mainly adapted template so the legacy suffix 'f' is dropped everywhere and added support for go generate.
Also restructured tests, since these usually belong to the same package as the tested code resides in.

Thank you.


Create code playground for origin.js

@joshfraser had excellent idea to demo the "raw" origin.js (not in npm) via a code playground site.
Goal is to demo only basic functionality; about what we do in tests:

Getting details of listing
Creating a listing
Buy a listing

Possible playground sites:

https://codepen.io/
https://jsfiddle.net/
https://runkit.com/ (support for npm too)
More here: https://www.sitepoint.com/7-code-playgrounds/

Docs is a good place (though we could also link from origin-js readme).
Closing this issue in favor of docs.


Book minor edit

Deleted a repeated word in chapter 12.

can you clean your PR and the git history?

Will do. I just found out the mess now.


oci_ons_notification_topic.bastion_notification is empty tuple

fresh install
setting bastion_enabled = false
Error: Invalid index
on .terraform\modules\base\modules\bastion\datasources.tf line 69, in data "oci_ons_notification_topic" "bastion_notification":
69:   topic_id = oci_ons_notification_topic.bastion_notification[0].topic_id
|----------------
| oci_ons_notification_topic.bastion_notification is empty tuple
The given key does not identify an element in this collection value.
should not continue the bastion notification setup if no bastion

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version and Provider Version

Affected Resource(s)

Terraform Configuration Files

# Copy-paste your Terraform configurations here - for large Terraform configs,
# please use a service like Dropbox and share a link to the ZIP file.
# Please remove any sensitive information from configuration files before sharing them.

Debug Output

Panic Output

Expected Behavior

Actual Behavior

Steps to Reproduce

terraform apply

Important Factoids

References

Hi, it's now fixed in the base module. Can you please test and confirm?

@scanlonjts can you please check?

Closing due to lack of response


[docs] Add a how-to guide on running IWYU on the Swift project.

As mentioned in https://github.com/apple/swift/pull/32080#issuecomment-636419901.

@swift-ci please smoke test


Fix build error with unused import

Fixes a build error with unused import
Problem
Just a build fix
Solution
Remove the unused import
Notes
Checklist

[X] Unit test all changes
[X] Update README.md if applicable
[X] Add [WIP] to the pull request title if it's work in progress
[X] Squash commits that aren't meaningful changes
[X] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

👍


Define 0.00000001 XMY

Some comments are here:
https://www.reddit.com/r/myriadcoin/comments/cl3wqo/000000001_xmy_request_for_names/

Although 8bit was popular, I think the readability would be an issue. Roarde I think had the best suggestion with obi.


Remove unused imports

remove extern crate XXX
remove unused dependencies
simplify feature sets
remove allow(unused_imports)
code formating in some places: avoid too long lines

Signed-off-by: Artem Mironov<EMAIL_ADDRESS>

Codecov Report

Merging #595 (6259397) into main (9c9ec55) will increase coverage by 0.16%.
The diff coverage is 83.49%.

:exclamation: Current head 6259397 differs from pull request most recent head 6e896ed. Consider uploading reports for the commit 6e896ed to get more accurate results

@@            Coverage Diff             @@
##             main     #595      +/-   ##
==========================================
+ Coverage   59.09%   59.26%   +0.16%
==========================================
  Files         198      198
  Lines       19047    19121      +74
  Branches     4358     4353       -5
==========================================
+ Hits        11256    11332      +76
- Misses       3458     3462       +4
+ Partials     4333     4327       -6

Flag
Coverage Δ

integration-aries-vcx
47.91% <83.33%> (+0.25%)
:arrow_up:

unittests-agency-client
29.33% <ø> (ø)

unittests-aries-vcx
47.59% <66.66%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

agency_client/src/lib.rs
61.11% <ø> (ø)

agency_client/src/messages/create_key.rs
60.71% <ø> (ø)

agency_client/src/messages/update_com_method.rs
52.63% <ø> (ø)

agency_client/src/testing/test_utils.rs
84.21% <ø> (ø)

agency_client/src/utils/validation.rs
55.88% <ø> (ø)

aries_vcx/src/global/settings.rs
68.47% <ø> (ø)

...ries_vcx/src/handlers/proof_presentation/prover.rs
57.04% <ø> (ø)

aries_vcx/src/indy/credentials/encoding.rs
69.04% <ø> (ø)

aries_vcx/src/indy/credentials/holder/mod.rs
73.80% <0.00%> (ø)

aries_vcx/src/indy/primitives/credential_schema.rs
25.00% <ø> (ø)

... and 34 more

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


"ImageUpload Upload an image and start a VMI Should succeed" fails often

/kind bug
What happened:
CI fails:
 (root).Tests Suite.Invoking jobs / check-patch.k8s-1.11.0-dev.el7.x86_64 / ImageUpload Upload an image and start a VMI Should succeed
4
2471, 2469, 2467, 2466
(root).Tests Suite.Invoking jobs / check-patch.k8s-genie-1.11.1-release.el7.x86_64 / ImageUpload Upload an image and start a VMI Should succeed
3
2471, 2469, 2467
(root).Tests Suite.Invoking jobs / check-patch.k8s-multus-1.11.1-release.el7.x86_64 / ImageUpload Upload an image and start a VMI Should succeed
3
2472, 2469, 2466
(root).Tests Suite.Invoking jobs / check-patch.openshift-3.10-crio-release.el7.x86_64 / ImageUpload Upload an image and start a VMI Should succeed
3
2471, 2469, 2467

https://jenkins.ovirt.org/job/kubevirt_kubevirt_standard-check-pr/test_results_analyzer/
What you expected to happen:
CI completes.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

@fabiand @rmohr looks like we've got a lot of dups on this, see also #1659 and #1679.

making progress on this in #1682

#1682 is fixing it - eventually.


handle scenario where the keyboard may not be showing or shown in time.

fixes #40

Thanks for this PR @ericlewis
I just ran your branch on simulator and noticed that as the keyboard is animating the passcode dashes also perform a rise up and in animation as the center of the frame is changing. The dashes shouldn't be animating (they should stay in place) as the keyboard appears.
There may be a better way to solve this issue in order to attain the desired behavior. Definitely will look into this soon, unless you figure it out before I do! :smiley:


Fix Selecting "All Threads" and Display That Track on Linux

With recent changes, we started using "-1" instead of "0" for special
purpose thread/process ids (no selection and all threads). By these
changes, we missed a couple of places.
This change uses SamplingProfiler::kAllThreadsFakeTid on all these
places and thus fixes the issue that scheduler track was not highlighting
anything, when clicking on the "all threads" track and also displays that
track on Linux at all.
Bug: http://167391339 http://167393367
Test: In capture, select different tracks (including all threads), and
click on the background (to reset). Also check the lable in
TopDownView and the SamplingViews.

As this is targeting the release branch, I will wait for a second approval here.


Update node.md

describe your changes...

[ ] Read and sign the CLA. PRs that haven't signed it won't be accepted.
[ ] Make sure your PR complies with the writer's guide.
[ ] Review the diff carefully as sometimes this can reveal issues.
Remove these instructions from your PR as they are for your eyes only.

NodeStuffPlugin  --〉NodeStuffPlugin


Putting message in queue silently fails to add content with Python 3.4

Using queue_service.put_message('queue', 'message') fails using Python 3.4
Instead, it places an empty message in the queue.
edit:
This is using the example from: http://azure-storage.readthedocs.org/en/latest/queue.html?highlight=put_message

I'm not able to reproduce that with 3.4 and the code from the docs. Specifically I ran the below:
queue_service.create_queue('taskqueue')
queue_service.put_message('taskqueue', 'Hello World')
messages = queue_service.get_messages('taskqueue')
queue_service.delete_queue('taskqueue')

On Windows I used Fiddler to verify what was being sent via HTTP and I also looked at messages under the debugger. I can see the message being sent to the server and that messages[0].message_text contains 'Hello World'. How are you checking the message text?

Maybe I should have added that I get it to work by using Python 2.7, and 'Hello World'.encode('Base64'), but only in that case. Without first encoding in Base64, it also produces an empty message. I see the messages in the queue in all cases, but its "message" is empty. I check by using Azure Storage Explorer, on an existing queue. The app that takes messages from that queue also sees nothing in what is queued when using Python 3.4.
Any suggestion as to what I could try?
Also, this is executed under an Azure VM running Ubuntu Server 14.04

When you tried Base64 encoding on Python 3.4, was it like this (which works for me):
from base64 import b64encode

service.create_queue('testqueue1')
encoded_msg = b64encode(b'bonjour world')
encoded_msg_txt = encoded_msg.decode()
service.put_message('testqueue1', encoded_msg_txt)

You just solved my issue! A big thank you for the quick help.


[WIP] enhanced colour support

I found it a bit limiting that khal can use only 16 colours when modern terminals often support 256 or even 24-bit colours.
So, I hacked at it and implemented support for more colours.
Unfortunately, click does not seem to support more than 16 colours. Instead of relying on click for colours, I changed the colored function and use it instead of click.style. As a positive side-effect, colouring is now more consistent, as it uses only one function.
In addition to implementing support for more colours, I also changed handling of the 16 "base" colours a bit. I believe this provides a better fix for issue #314.
Apart from that, my changes should be fully compatible with existing config files and code.
The new colours can be specified similar to colours in git: 0-255 for colours from the 256-colours palette and HTML-style colours for 24-bit colours. Does this seem sensible to you?
My work is not complete, yet. I am aware of the following open issues:

Clearly, this needs more testing. I always forget something...
I implemented support for using bright "base" colours without using the terminal's bold attribute. This has the advantage, that backgrounds can be coloured brightly, too. However, the ANSI codes for non-bold bright colours may not be supported by all terminals. The best solution seemed to make this configurable. However, I could not figure out how to access configuration options from within the colored function. I'd be grateful for any hints.
The documentation needs to be updated, of course. I'd rather leave this to someone more familiar with the project.

So, what do you think? Is this useful? Are there any issues I forgot?

The vdir spec supports adding rgb hex-encoded colours in a colour file inside the vdir (we haven't been supporting this yet). I do believe that this is a good chance to add support for that, by:

Reading from that file rather than the config file.
Updating the docs to reflect this.

I'm okay with this PR as-is though, if others feel that that should be implemented separately.
Thanks for this, by the way, it's a very welcome change!

I thought about getting the calendar colour from the vdir, too. It seems like a logical next step to me. However, it is not trivial to implement. Therefore, I think it is best to focus on one step at a time.

Hello and welcome to khal! This is certainly useful and I don't mind getting rid of click's style. If I see this correctly, the extended palette currently doesn't work in ikhal/urwid, does it? If so, we should have a look if something can be done about that.
But I really need to install a terminal which supports more colors to properly test it first.

You are right, apparently I missed ikhal completely. I will look into that.

Unfortunately, urwid does not support 24-bit colour. So I only implemented 256 colour support. 24-bit colours are converted to hopefully similar palette colours.
The current approach of using colours as attributes for styling the interface does not scale for 256 colours. So I switched to using the calender names as attributes instead. They are generated at run-time with the correct colours.
I think I found all places where colours were used as attributes. But maybe I didn't. Is there any way to make urwid not silently ignore undefined attributes? That would help tremendously.
On a side note: ikhal currently does not support grey colours, as the khal configuration calls it grey while urwid calls it gray.
I resolved this by adding a simple replacement rule.
However, using the American spelling "gray" might be better for consistency with the American spelling "color". But that is not for me to decide and I did not want to break existing config files with this PR.

Looks good too me, could anybody else have a look if this works for them? @dzoep or @languitar perhaps?
@black2754 Two more things: 1) Could you add yourself to AUTHORS.txt please?
2) the config spec file (from which the documentation is generated) does mention the "named colors" but does not specify them, could you put them into that "doc" string?
Also let's use the American spelling then, I think it's ok to break compatibility here, I'll put a note in the CHANGELOG. We could of course use both for the time being, issue a deprecating warning for BE spelling and so on, but I think that it's really worth it.

I changed the spelling of grey and updated the documentation in khal.spec. Is this what you meant?
I also made khal actually use the new config option I introduced earlier. I changed seven function definitions just to have that option available in one function. This seems somewhat inelegant and like a lot of overhead to me. Is there a better way to do this?
Anyway, I think this PR is complete now. It could probably still use more testing and I'd be happy to fix any issues that might come up.
Thank you both for all your feedback.

Haven't tried it yet, but the code looks much cleaner than how it looked when I hacked the initial color support in :). I'd definitely merge it.

@black2754 yep, that's what I meant (regarding the doc).
This could be made a lot prettier I agree.
Thank you for the PR again!


Only remove titles in non-fastboot mode

This PR moves the initializer into an instance initializer and makes sure it runs before ember-cli-head adds in its <title> tag. See the discussion in #51 for more information

@pgrippi nice! I think this is probably a better way to go than mine in #53 as making this an instance initializer makes sense. Does this mean that fastboot users don't need to remove the <title> tag as well?

Sadly no, fastboot users will still need to remove the title tag in index.html


sendMultiple not working properly with transactional Templates

Issue Summary
I'm trying to send a transactional template to multiple users without revealing all users in the "to" field when the email is received. In the docs it says to use the method sgMail.sendMultiple(). I have tested but still showing all emails addresses to all recipients.
here is my code
  const msg = {
      from: {
         email<EMAIL_ADDRESS>         name: "My Org"
     },
      reply_to: {
        email<EMAIL_ADDRESS>        name: "Admin"
      },
      bcc<EMAIL_ADDRESS>      categories: ['Transactional', 'SpecialOne'],
      template_id: 'd-xxxxxxxxxxxxxxxx',
      personalizations: [{
        to: [
            { email<EMAIL_ADDRESS>name: username1 },
            { email<EMAIL_ADDRESS>name: username2 },
            { email<EMAIL_ADDRESS>name: username3 },
         ],
        dynamic_template_data: data
      }]
    };
sgMail.sendMultiple(msg);

As the documentation said for sendMultiple only the user's email address should be showing when the email is received, however this is not the case.
Thank you
Technical details:

sendgrid-nodejs "@sendgrid/mail": "6.4.0",
Node.js Version: 10.15.1
express": "4.16.4",

I would use BCC instead of To.

Thanks for the reply but i do not think the BCC would not work for me for the reasons below,

the template has some customization where {{name}} and some other variables are replaced for each receiver. Question will the BCC replace the variables in the email content?
from the documentation it says that "if you wish to distribute BCCs to multiple addresses you will need to create a distribution group or use forwarding rules." and futhermore i would have to turn the feature on and as per the documentation "With this setting turned on, you will be charged an extra email for every email you send.".

My issue is that this is a transactional email meant to be an invitation email coming from one user to other friends. no way of defining a distriution list ahead.
So my best bet is to use the sendMultiple() and not showing all email addresses in the "to" field.
So is this a bug for the sendMultiple not able to work with the templates? or am i missing something?
Thank you for the help

I'm also having issues with sendMultiple() not working as expected. It shows all emails.

The solution provided by @sergeyfilimonov should work. Another way to do this with a single API call (using the OP's sample code):
const msg = {
    from: {
        email<EMAIL_ADDRESS>        name: 'My Org'
    },
    reply_to: {
        email<EMAIL_ADDRESS>        name: 'Admin'
    },
    bcc<EMAIL_ADDRESS>    categories: ['Transactional', 'SpecialOne'],
    template_id: 'd-xxxxxxxxxxxxxxxx',
    personalizations: [
        { to: { email<EMAIL_ADDRESS>name: 'username1' } },
        { to: { email<EMAIL_ADDRESS>name: 'username3' } },
        { to: { email<EMAIL_ADDRESS>name: 'username2' } }
    ],
    dynamic_template_data: data
};
sgMail.send(msg);

I'm not sure why you closed this, seems to me that this is a workaround not a solution. If this is a bug and it is not working as documented in your documents, then remove it from your docs or update it to reflect how it should work.

I don't consider this to be a workaround as it's the proper way to accomplish what you're looking to achieve. The sendMultiple implementation simply wasn't built to work with personalizations and is more of a convenience function when the top-level to value is an array. That said, I'm open to reviewing a PR to enhance the documentation/samples with how to achieve what I illustrated above.

ok thank you for the clarification and the follow-up.


Split out arrow-array

~Draft as I wish to perform another pass, and double-check the benchmarks~
Which issue does this PR close?

Part of #2594
Rationale for this change

Continues the process of splitting apart the crate, so that components can depend on just what they need, compilation parallelizes better, etc...
What changes are included in this PR?

Moves the array, array builders, and record batch definitions into a new arrow-array crate
Are there any user-facing changes?
The deprecated RecordBatch::concat is removed, otherwise there are no breaking changes :tada:

marking as api-change due to removal of RecordBatch::concat

FWIW it occurs to me we probably need to update the github workflow triggers to reflect this new code organization:
For example:
https://github.com/apache/arrow-rs/blob/master/.github/workflows/arrow.yml#L21-L29
SHould probably include arrow-array and arrow-buffer

Running the benchmarks, some of the faster benchmarks do show the odd ~10% regression, but we're talking 10s of microseconds here. I'm inclined to think this is not an issue, and if it transpires to be so, we can revisit those kernels.


new log error that I do not understand

Running check: PomBase::Check::DuplicateInteractions
Running check: PomBase::Check::FeatureCount
Running check: PomBase::Check::PhenotypesNotInCategory
Wide character in print at /local-homes/pbuild/git/pombase-chado/lib/PomBase/Check.pm line 148, <$website_config_fh> line 1.

The message has gone away but I think I've found and fixed the problem.  I'll close this issue.  Let me know if you see it again.


Restyle categories section

Close #2599

@Bonifacy1 Few general remarks about this PR.

Please make sure that you follow guidelines about writing commit messages - the commit history looks cleaner if we follow the same rules.
I'm not sure if you're using any linters and code formatters (I see 37 issues on codeclimate) - we use Eslint and Prettier.

@Bonifacy1 Few general remarks about this PR.

Please make sure that you follow guidelines about writing commit messages - the commit history looks cleaner if we follow the same rules.
I'm not sure if you're using any linters and code formatters (I see 37 issues on codeclimate) - we use Eslint and Prettier.

Actually we use tslint here.

Blocked by #3123

Sure we can

It would be nice if PRs changing the UI could contain some sort of screenshots to present the changes to the wider audience


User experience on mobile is terrible

@penberg what about the user experience on emacs?

Fixed by https://github.com/penberg/librescholar/commit/123b4b485227a17d123713c8144e970fda61a4c0


Simple x64 binary: There is no dead variable elimination? Memory is versioned for no reason? Return variable is missing?

In this tiny binary: http://cyberjes.us/retn

Constants propagated without squashing useless intermediate variables: zero is rax = 0; return 0 for some reason
Spurious memory versioning: call zero apparently versions memory without touching memory
Return variable missing: The 0 returned by zero is propagated into the syscall argument, but there is no rax = zero().

All calls version memory, and dead variables are more aggressively eliminated in HLIL #1383 which is currently being developed.


Emit language specific topic sections in the markdown representation of automatic curation

Bug/issue #, if applicable: rdar://124527905
Summary
This builds on #854 to emit language specific topic sections in the markdown representation of automatic curation from docc process-catalog emit-generated-curation
Dependencies
#854
Testing
Run docc process-catalog emit-generated-curation on a project with language refinements (for example " Tests/SwiftDocCTests/Test Bundles/GeometricalShapes.docc"
Symbols that appear in different places in different languages should be listed in language specific topic sections.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[ ] Updated documentation if necessary

I opened a PR on the upstream here instead https://github.com/apple/swift-docc/pull/856


Config File

Hi!
First for all, this mod is really good!
My only concern is the mod miss a config file for:
[ Note: all I am going to comment on are suggestions that I would like with the intention of "improving in my opinion" the mod, I would love to get into the world of Minecraft mod development and even make a pull request on this topic.
In any case, if you manage to implement something of what I am going to say I would be very grateful and if not, no problem!
Also add that I'm playing on 1.20.1 and this features I would like to see them at least on that version if it's possible]

Select what type of fish do you want the boids effect to have:
I think only cod, salmon and tropical fish are affected by boids (vanilla mobs) and also, end fish & cubozoa (jellyfish) from BetterEnd are affected too, which is strange to see boids effect in jellyfishes, spinefish from Aquamirae and Lantern Fish from Alex's Caves (sinytra connector is necessary for this because this mod is fabric-only).
I think it could be good to have in the config file a option for disable an specific mob for the boids mod.
I would add this option to deactivate the effect on cubozoa and tropical fish.

Speed of each type of fish:
The other option could be that for each type of fish you can choose a movement speed because it feels that with this mod the fish go much faster than normal (I have seen another Issue that already commented on this and that for aquariums it is something strange to see).

The config file could be editable and so each user could add the fish they wanted (to extend compatibility with other mods and avoid having to do it on your own) configure their boids in a certain way (aka speed and disable), writing it in the config file with certain format.
Thank you very much if you took the time to read this!

Hi,
I have added a basic configuration in the latest version that implements a few of your suggestions. I will keep this issue open, as I might implement some of the other ideas in the future as well.

Hey!
I am very glad that you were able to read my post and that you wanted to apply my suggestions!
I just tested the update and the excluding entities function certainly works great.
I have also tried the max and min speed thing and I THINK it works well.
Here is a screenshot of the changes I have made to my config:

Not counting Cod and Salmon, I have found all of those entities that are affected by mod boids.
An interesting detail is that at the end the lanternfish has its own boids algorithm probably made by Alex.
The cool thing is that I have discovered that the Aerglow Fish of the Deep Aether Mod is also affected by the algorithm, in addition to the perch of the Upgrade Aquatic Mod.
You are doing it very well!
Now I will be able to play with this mod again in my world.
Thank you!


Update ZkProgram proof detection during deployment

Fixes #628

This is weird This branch has conflicts that must be resolved. This branch does not have any conflicts.

This is weird This branch has conflicts that must be resolved. This branch does not have any conflicts.

This is strange.


Suggestion: add this function since it was in FastBitBuffer docs but not in the actual module

function BitBuffer.BitsRequired(Value: number) 	return math.ceil(math.log(Value,2)) end
Just a neat function that returns how many bits is a number, which was also in FastBitBuffer docs but never in the actual module.

I don't think this belongs in the library. I'd prefer to keep the library as minimal as possible, so adding niche features like this doesn't seem appropriate. If you need this function, you can just add it to your scripts.


Support custom prop validation errors

What problem does this feature solve?
Sometimes prop validation error messages are to vague, and one would like to specify his own custom error message, instead of the default "Invalid prop: custom validator check failed for prop" message.
What does the proposed API look like?
I suggest supporting the validator to be an object. Instead of:
export default {
  name: 'MyComponent',
  props: {
    type: {
      type: String,
      validator: v => ['positive', 'negative', 'warning', 'info'].includes(v)
    }
  }
}

Have the following:
export default {
  name: 'MyComponent',
  props: {
    type: {
      type: String,
      validator: {
        handler: v => ['positive', 'negative', 'warning', 'info'].includes(v),
        message: "Type must be one of 'positive', 'negative', 'warning', 'info'"
      }
    }
  }
}

Also message as functions should be supported:
export default {
  name: 'MyComponent',
  props: {
    type: {
      type: String,
      validator: {
        handler: v => ['positive', 'negative', 'warning', 'info'].includes(v),
        message: val => `Invalid value ${val}; Type must be one of 'positive', 'negative', 'warning', 'info'`
      }
    }
  }
}

Duplicate of https://github.com/vuejs/vue/issues/7186


libbpf support in CI

What would you like to be added?
ref to https://github.com/sustainable-computing-io/kepler-action/pull/65
I found that in our CI we don't have libbpf installed. Hence in CI related code, I want to enable libbpf support as

install libbpf via kepler-action during image build.
install libbpf via kepler-action during integration test.

@rootfs , do we need two images as one for bcc and one for libbpf?
if so, we also need enable libbpf during image build.
and to save CI running time.
My suggestion is one as bcc and kind and one as libbpf with microshft
Why is this needed?
to add test coverage for lbbpf

fixed via #951


API and capabilities research

Research what capabilities and APIs would be required and feasible in an embedded environment.
Some things to consider:

Streamed transaction signing (Trezor proposal)
Automatic signing (any transaction or only non-spending?)

Resume can be found in design document


Bugfix/130 item batcher validation

I made a mess of the rebase on the old PR (https://github.com/ChristopheBougere/asl-validator/pull/131), so this is a clean implementation using the path property to specify nested items.

Thank you for your contribution!

@ChristopheBougere thanks for maintaining this, really useful tool :)


Fix Cloudflare support for Dynamic DNS Updates (Todo #4737)

Resubmit of #1812
The current implementation isn't working due to API change. Credits: det0nat3 @ https://forum.pfsense.org/index.php?topic=87436.msg534817#msg534817

PR closed per https://github.com/pfsense/pfsense/pull/1917#issuecomment-142041673


Passing data

how to send data from service to template engine ( pug, handlebars etc.. )

Hi, after a lot of guessing, I discovered that this way is possible to send data to handlebars and pug:
 this.pdfService.toStream('template', {
      locals: {
        title: 'My title',
      },
    });

Hi, after a lot of guessing, I discovered that this way is possible to send data to handlebars and pug:
 this.pdfService.toStream('template', {
      locals: {
        title: 'My title',
      },
    });

@smatioli if you could , please post a sample template as well where this variables are replaced.

doctype html
html(lang="en")
  head
    title= title
    script(type='text/javascript').
  body
    h1 #{locals.title} - node template engine
    #container.col
      if youAreUsingPug
        p You are amazing
      else
        p Get on it!
      p.
        Pug is a terse and simple templating language with a
        strong focus on performance and powerful features.

try these sample template to get the title passed in variable in your pdf

Hello, if you have to make an iteration, how do you pass the array?

`ul

var count = 0;
each item in items
if count < 3
li= item.name

count++;`

try this please

ul
  - var count = 0;
  each item in items
    if count < 3
      li= item.name
    - count++;

try this please


Validate Template Error Message

A small notification will pop up from the top when successfully validating a template.
But when the validation is unsuccessful, you will get a big notification that takes up a large portion of the screen. This is an older version of our error alerts, and this should also be refactored to be a notification that pops up from the top. To change this, update the checkTemplateReference() function in TemplateStore.tsx.

hey @KaradzaJuraj can i work on this

Of course @duplixx!

@duplixx can you I take up this issue in case you have not started working on this

No problem!
@KaradzaJuraj @petar-cvit can you please assign this issue to me?

@KaradzaJuraj  can you assign me this issue ?

@Wasiim105 you can work on this issue, since @s-vamshi is already assigned to issue  https://github.com/cyclops-ui/cyclops/issues/464


[multi tenancy] the influence range at the time of configuring a wrong OIDC issuer.

[provide a description of the issue]
If a tenant admin configures a wrong OIDC issuer for only 1 service of 1 tenant in the Integration view in 3scale Admin Portal, other tenant admins cannot change their service configurations with the error message "auto-updating configuration failed with: Cannot serialize table: excessively sparse array" in configuration_loader.lua.
After that when we try to redeploy the apicast-production, we fail in deploying with no error message.
Version
tag: 3scale-2.2.1
Current Behavior
The apicast-production loads all services of all tenants in one time, and if there is a wrong OIDC issuer configuration, the apicast-production fails in loading all services of all tenants.
Expected Result
Even if there is a wrong OIDC issuer configuration in 1 tenant, other tenant's admin should be able to change their service configurations.
Or even if there is a wrong OIDC issuer configuration, the apicast-production should load the service configuration successfully.

This has been fixed in https://github.com/3scale/apicast/pull/893 and released as 3.3.0-cr1.


refactor the unified-docs experience

Closes #1008
Code Changes

[x] Combined ops and ctl docs for one unified experience
[x] Updated uses of fidesctl and fidesops to use "Fides", though some code-related uses were left (fidesctl-meta, fidesops-meta)
[x] Reorganized docs IA
[x] Removed duplicated documentation
[x] Unlinked tutorials from the nav to be rewritten

Steps to Confirm

[x] overview of images still work
[x] docs build process shows no broken links
[x] docs build process shows no (unexpected) pages missed in the navigation

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation Updated:

[x] documentation complete, or draft/outline provided (tag docs-team to complete/review on this branch)

[x] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

@ThomasLaPiana I think this should be in a good enough state for review!

@conceptualshark thank you!! I'll take a look

@conceptualshark this looks pretty dang awesome, thank you! I'm sure we'll hammer more on the docs as we get closer to an official release, but this looks great! Kind of awesome how seamlessly they look when put together like this


Way to get the line number in custom overlays

Is there a way to get the line number in an overlay token method?
I did something similar here but the way I did it looks not scalable. That's why I did not make a PR.
I need this feature to highlights parts of code not depending on string matching but on line and column indexes.
Thanks

You should probably use markText for this use case. The way overlays (and modes in general) work relies on them being stateless, and oblivious to line numbers.

Thank you! I'll try :wink:


ValueError: Cannot process type as image: <class 'pathlib.WindowsPath'>

I'm getting this error when trying to use the "Add selection" or "Add ALL Displayed" buttons in the "Filter by Selection" tab.
Using the latest commits of automatic1111 and this extension as of today.
OS is Windows 11.

Thank you for reporting!


Applies multiple performance improvements

Applies the following performance improvements:

in Application Insights disables auto-loading unnecessary instrumentation for third party tools
in auth changes adal-node and node-forge to be lazy-loaded
in cli core and command base class changes chalk to be lazy-loaded
introduces the CLIMICROSOFT365_NOUPDATE environment variable to disable automatic checking for updates

Merged manually

Merged manually


Add streamlit chat interface tutorial

Description
This PR adds a streamlit chat interface tutorial.

I added patches that clear the text input field when a text is sent and also added a patch that lets users send messages by pressing Enter instead of pressing Send.
This PR is awaiting following PRs to be merged:

[x] #112 (this PR is based on it because they both use API endpoint and to preserve the order of tutorials)
[ ] https://github.com/AI-Yash/st-chat/pull/36 to allow installation of the streamlit-chat package on python 3.7. I tested that this tutorial works on python 3.7 so the only problem here is the setup requirement of that package.

This PR fails tests/tutorials/test_format tests because tutorials/messengers/web_api_interface/4_streamlit_chat.py uses # instead of """ for markdown cells. This is done because Streamlit treats """ strings as markdown statements and displays them on its page. Changing them to # solves this.
The doc building job succeeds while the format test fails so we should do something about it. I propose removing test_format tests: I don't think they are necessary.

As I mentioned in the body of this pull request, this markdown is treated by Streamlit as markdown cells that it has to display:

As I mentioned in the body of this pull request, markdown cells in this format are treated by Streamlit as markdown cells that it has to display:

Can we test it too?

In order to fix dependencies issue, can we switch to streamlit-chat~=0.0.1? Are there any breaking changes?

Can we test it too?

Theoretically, yes.
But I don't know if we should (in that it is worth the time spent figuring out how to do that).
This tutorial is weakly dependent on dff (it only uses Message and includes an implementation without it). So changes to dff will likely not affect this tutorial.
I think if we are to test something out of the web_api_interface tutorials it should only be the 1_fastapi one which would also cover this tutorial (because 4_streamlit_chat uses the FastAPI endpoint to communicate with pipeline and the version of streamlit is frozen).


Add --exclude-existing option to ls-remote

The git-show-ref command has a --exclude-existing option to take a list of references and remove them from the final list.  This allows you to run a command like this:
git ls-remote --tags origin | git show-ref --tags --exclude-existing

This shows all of the tags that exist on the remote that aren't in the local repo.  However, there is no equivalent for git-ls-remote.  It would be nice to be able to do this:
git show-ref --tags | git ls-remote --tags origin --exclude-existing

That would show all of the tags that exist on the local repo that aren't on the remote.  This is handy for figuring out which tags might need to be pruned from the local repo.

Hey, this is the issue tracker for the git-scm.com website, not Git itself. If you still want to discuss this issue, you should reach out to the community.


creating a private room is not e2ee by default

Duplicate of #3166?

Thanks @tulir.
We are working on it.


Question: parameters for running the container on a NAS with lower end CPU

Hi,
Thanks for the docker image! I have configured it to run on my QNAP NAS with a weak J4125 CPU, and would like to understand how to tune the container:

I started Docker container with the following parameters from the example without knowing what and why they are needed? Could those options have any impact on the performance of other services on the NAS?
--ulimit nproc=65535
--ulimit nofile=32000:40000 \
What is the recommended settings for memory? I currently configured 4GB memory but it uses all. I only have 16G in total so can't assign more memory. Any suggestion?

Any input is appreciated!

You should read performance tunning in rtorrent rakshasa wiki.
This is base on peers and torrent contents. If your torrent have thousands of peers and the torrent content a lot of files in it. You probably need to limit max file open and max peer connected in rtorrent.rc.

You should read performance tunning in rtorrent rakshasa wiki.
This is base on peers and torrent contents. If your torrent have thousands of peers and the torrent content a lot of files in it. You probably need to limit max file open and max peer connected in rtorrent.rc.

Tried that - I found the problem was high read io when seeding with upload 1-3MB the disk read is always above 50MB. I saw many have this issue and since rTorrent is not maintained for a long time I decided to give qBitTorrent a try.

You should read performance tunning in rtorrent rakshasa wiki.
This is base on peers and torrent contents. If your torrent have thousands of peers and the torrent content a lot of files in it. You probably need to limit max file open and max peer connected in rtorrent.rc.

Tried that - I found the problem was high read io when seeding with upload 1-3MB the disk read is always above 50MB. I saw many have this issue and since rTorrent is not maintained for a long time I decided to give qBitTorrent a try.

I did that too, but lately found that qbittorrent is a monster eating all of resource, especially CPU. With only 200 torrents and 5 of them have more than 2000 peers. My CPU freezed in every 3 days. rtorrent manage CPU performance better then qbit, I think.

You should read performance tunning in rtorrent rakshasa wiki.
This is base on peers and torrent contents. If your torrent have thousands of peers and the torrent content a lot of files in it. You probably need to limit max file open and max peer connected in rtorrent.rc.

Tried that - I found the problem was high read io when seeding with upload 1-3MB the disk read is always above 50MB. I saw many have this issue and since rTorrent is not maintained for a long time I decided to give qBitTorrent a try.

I did that too, but lately found that qbittorrent is a monster eating all of resource, especially CPU. With only 200 torrents and 5 of them have more than 2000 peers. My CPU freezed in every 3 days. rtorrent manage CPU performance better then qbit, I think.

I run both rTorrent/ruTorrent and Qbittorrent in docker and so far I haven't notice any CPU issue with Qbittorrent. I will try to add more torrents to it. I do find with Qbittorrent it also reads 10MB from HDD even though it only uploads 1-2MB, it is better than rTorrent but I'm still not very happy. The disk IO is the bottleneck of my NAS and I'd definitely like to minimise it if possible. I have tried the following with rTorrent without luck:

Disable preload
Assign more memory (8G)

You should read performance tunning in rtorrent rakshasa wiki.
This is base on peers and torrent contents. If your torrent have thousands of peers and the torrent content a lot of files in it. You probably need to limit max file open and max peer connected in rtorrent.rc.

Tried that - I found the problem was high read io when seeding with upload 1-3MB the disk read is always above 50MB. I saw many have this issue and since rTorrent is not maintained for a long time I decided to give qBitTorrent a try.

I did that too, but lately found that qbittorrent is a monster eating all of resource, especially CPU. With only 200 torrents and 5 of them have more than 2000 peers. My CPU freezed in every 3 days. rtorrent manage CPU performance better then qbit, I think.

I run both rTorrent/ruTorrent and Qbittorrent in docker and so far I haven't notice any CPU issue with Qbittorrent. I will try to add more torrents to it. I do find with Qbittorrent it also reads 10MB from HDD even though it only uploads 1-2MB, it is better than rTorrent but I'm still not very happy. The disk IO is the bottleneck of my NAS and I'd definitely like to minimise it if possible. I have tried the following with rTorrent without luck:

Disable preload
Assign more memory (8G)

Tried both without luck. Switch to qbittorent still experience similar problem but reading is now below 10MB so much better. Qbittorrent also seems to allow stop gracefully plus better features I think I will settle with it for now.

Suggest to open an issue on rTorrent repo.


Cow: Watch vistasadindia.com

Cow requests the watch of the watch_keyword vistasadindia\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Makyen in Charcoal HQ

vistasadindia\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.


Allow prefixes with multiple characters

Currently, prefixes can only consist of one character, which is very restrictive. And you cannot even use letters as prefixes, as every letter could be a prefix for some other application.
Also, this kind of prefixes  should activate the plugin only if there is a space after the prefix, similar to how Alfred, popular Mac OS launcher, works.  This way you won't miss other entries and can have unlimited plugins.

If the prefix is 1 character long, the behavior should probably remain the same, so you can still use things like "=3+3", "/app" e.t.c.

Ah yes, this seems much better and flexible. Added in 0.3.0.


No "size" field in resulting JSON when requesting empty file

[{"id":"ad17a845-542d-4d04-b35f-8fef4e49f639","hash":{"md5":"d41d8cd98f00b204e9800998ecf8427e"},"name":"subuw.py","owner":["launcher"],"filename":"subuw.py"}]

Fix in dev


OWIN cookie

Hi,
I've used this sample successfully in the past when consuming a forms authentication cookie. I'm trying to do the same thing, but consume a cookie that is generated by an app using OWIN middleware. I've tried to add an OWIN startup class but the code is never called. Do you know if what I'm trying to achieve is possible?

You can't inject OWIN pipelines into the Reporting Services Portal

Oh well, plan B then. Thanks


Proxy not showing

VSCode Version: 1.49.0
OS Version: Windows 10.0.19041.508

Steps to Reproduce:
The montly (3rd month in succession) proxy break is there.
This time the proxy authentication windows shows for less than a second and disappears without a chance to enter credentials.
After that a info tip with error 407 is showing.
Its really annoying to have proxy problems every month 👎
The Changelog says nothing about a change in proxy settings

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:
* [Proxy window stopped working (#104637)](https://www.github.com/microsoft/vscode/issues/104637)

* [Can NOT Clear Proxy (#103693)](https://www.github.com/microsoft/vscode/issues/103693)

* [Save Proxy Credentials (#32566)](https://www.github.com/microsoft/vscode/issues/32566)

* [Proxy Auth (#104726)](https://www.github.com/microsoft/vscode/issues/104726)

* [Save Proxy Authentication (#56439)](https://www.github.com/microsoft/vscode/issues/56439)

No, its none of those. Its something new. But I have gone through all of them to, (for me) they where fix with the next .1 release

Recovery release is nearly here. See #106671

Fixed in Recovery Release, dup of #106434