Generate pages from master

I propose that it would be better for future pull requests to have all proposals made against one branch (master). This pull request changes the process of building the documentation to enable that.
On a technical level it does three main things:

Generate the documentation entirely using scripts, assets, and markup kept on the master branch
Rewrites the scripts in ./build_tools to output the generated pages into the ./pages folder
Adds a script to clean the gh-pages branch, replace it with the (generated) contents of the ./pages folder on master and force push that to origin.

WARNING: Running the push_changes_to_github script will overwrite the contents of the gh-pages branch on origin. I have pushed a gh-pages-backup branch to alphagov/digitalmarketplace-frontend-toolkit as mitigation.

@tombye 9bbe884f hopefully addresses the discussion we just had about preserving history in the gh-pages branch.


updated error calls with module names

Description

Included Module names in channel callback errors
closes: #1728

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards.
[ ] Wrote unit and integration tests (does not apply)
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/) (does not apply)
[x] Added relevant godoc comments.
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Will update the code to fix for issue raised by @colin-axner

Thanks @alizahidraja! Unfortunately with middleware, these error messages can be misleading. If we have a middleware stack which is ICS29 and ICS20, if ICS20 returns an error in the application callback, the final error message will actually indicate that the module which failed was fee-middleware (which is incorrect).
I think it might make more sense to include the portID? We could also change the notion of the "module" variable to be a "middleware stack", though it'd be nice to think this approach out fully

I have mentioned both the portID & channelID in channel callback errors for better identification, lmk if that works


systemd fixes

@sabedevops 's systemd fixes and a script fix to delete the OTT JWT file after successfully enrolling

Summary of ZET resolvers logic
Check whether the libc resolver config file /etc/resolv.conf is managed by systemd-resolved. We'll know that's the case if it is a symbolic link to one of the four expected configuration files that are provided for this purpose by systemd-resolved.
If /etc/resolv.conf is managed by systemd-resolved then ZET will configure systemd-resolved to use and prefer the Ziti nameserver by way of the most modern available interface for systemd-resolved. That is, libsystemd, the DBUS socket API > /usr/bin/resolvectl CLI > /usr/bin/systemd-resolve CLI, in that order.
If /etc/resolv.conf is not managed by systemd-resolved and is a regular file then, we fallback to the legacy resolver CLI  /usr/sbin/resolvconf if it is available and valid. We know it's invalid if it is a symbolic link to systemd-resolved's resolvectl CLI because at this point in the logic we've already proved that systemd-resolved is not managing the resolver config. That link exists on newer systems for which resolvectl maintains a reverse-compatible personality when invoked as resolvconf.
As a last resort ZET will directly munge /etc/resolv.conf to have the Ziti nameserver as primary.


Example of how to do hardware-aware training with DDP

Description and motivation
Provide an example of how to train a model with DDP on multiple GPUs.

closed with #328


apply sprite size logic to textures

Currently certain size rules are only applied in the presence of sprites. From the docs:

Note: Since both percent-based and ratio-constrained scaling only make sense in the context of a sprite (not a simple colored shader point), specifying these for layers without both a texture and a sprite defined will generate warnings and draw nothing, for example:

However these should also apply in the case of a simple texture.
Includes:

https://github.com/tangrams/tangram/issues/644
https://github.com/tangrams/tangram/issues/642
https://github.com/tangrams/tangram/issues/641

Duplicate of https://github.com/tangrams/tangram/issues/638


Appstudio update test-component-pac-impb

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-impb-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
11 seconds
init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
30 seconds
build-container

✅ Succeeded
17 seconds
inspect-image

✅ Succeeded
18 seconds
deprecated-base-image-check

✅ Succeeded
34 seconds
clamav-scan

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
12 seconds
label-check

✅ Succeeded
6 seconds
show-sbom

✅ Succeeded
8 seconds
show-summary

Pipelines as Code CI/test-component-pac-impb-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
25 seconds
build-container

✅ Succeeded
16 seconds
deprecated-base-image-check

✅ Succeeded
17 seconds
inspect-image

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
39 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
17 seconds
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary


Restrict branches access

I want to propose new changes. I created set of rules(settings) for different branches in taraxa-node github repo. There is different settings for:

master,
develop,
feature/*,
release/*,
hotfix/*
/ (all other branches)

I would also propose to give everyone in here "Write" access and only 1 person with "Admin" access in both EU and US timezones. Based on the new roles and branches settings it would pretty easy and straightforward to keep our repo clean, structured and without any hacks like direct commits to develop or master -> can't think of any situation when this should be needed...
What we can achieve by this:

only admins can do any sort of hacks like direct commits to protected branches, etc... (but they really should not even if they can....)
master will have restricted write access - we will specify set of people who can write (even merge PR's) to master - let's say admins
people with write access can create and merge PR's to develop but only those PR's that have 2 approvals and all checks passed, no direct commits will be technically possible for them
linear history will be required for all custom branches - it means people wont be able to merge custom branches into develop if they re-merged develop into their branch instead of rebase (We can omit this as people have usually problems with rebasing)
if linear history on custom branches rule is required we can keep the message "your branch is not up to date" but disable 'update-branch' button which behaves as merge and not rebase (not possible to change behaviour of this button to rebase)

I understand the fact that this should be strictly required once we have some real stable version in master as our testnet is using images from master and at the moment we are simply hot-fixing things... But cannot we restart testnet with provided custom image tag so we dont need to merge all these "fix tries" into master as we did  few days ago ? @rattrap @agrebin that is question for you guys I guess...

@Taraxa-project/ledger please have a look at this.

Agree we should do this.

access restricted, repo settings updated...


Padding free dpo

What does this PR do?
New feature #2422
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[x] Was this discussed/approved via a GitHub issue? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines.
[ ] Did you write any new necessary tests?

For now this is just a draft will be continuing to work on it
@osanseviero

so the now it works where if we do padding_free =True it will not pad but the tests keep failing due to attention_mask still being there if padding_free =True

most of the stuff is done just some small stuff left like dealing with list and converting to tensor

Hey @osanseviero,
The main idea for using padding_free is mostly in place now, but there are still a few things that need to be done. It would be awesome if you could take a look at the code and let me know if there's anything else I should address or add.

@osanseviero  I think this fixes it sorry for the small mistake I have been making  , thanks for your patience

still more work to be done not really ready yet

the padding_free finally can train , it works but
it's not optimised
not really clean code yet
better refactoring of code
will do this by tommorow

I'm not sure that having the padding-free logic in the collator is the best option here, since we concatenate prompt and completions in the trainer. Maybe the easiest to have everything inside the concatenated_forward method.

I'm not sure that having the padding-free logic in the collator is the best option here, since we concatenate prompt and completions in the trainer. Maybe the easiest is to have everything inside the concatenated_forward method.

question is since the concatenated_inputs depends on the collator do i just create position ids and all those stuff in the concatenated_forward??

Is sound like the easiest way. Am I missing something?

yes you are right I think this will be done by today  hopefully

depends on the collator

I don't think it depends on the collator. How would it?

another question we get the inputs ids and such from concatenated_inputs but they are padded in this case so what do we do ?

So let me get this straight
the concatenated_Inputs pad our inputs_ids and such and in our case we do not want to pad them right so  what do we do here ?
do we create the position ids inside the concatenated_inputs  and also add not pad the input ids and such ??

Here
https://github.com/huggingface/trl/blob/6d4ed070f1f53a87fb3cff2eb82a56db093bccc6/trl/trainer/dpo_trainer.py#L1115-L1123
After the flushing left, we could remove pad tokens, and add position ids:
# Flush left to reduce the memory usage
# [[0, 0, x, x, x, x],  ->  [[x, x, x, x],
#  [0, x, x, x, 0, 0]]       [x, x, x, 0]]
for i in range(attention_mask.size(0)):
    first_one_idx = torch.nonzero(attention_mask[i])[0].item()
    input_ids[i] = torch.roll(input_ids[i], shifts=-first_one_idx)
    attention_mask[i] = torch.roll(attention_mask[i], shifts=-first_one_idx)
    loss_mask[i] = torch.roll(loss_mask[i], shifts=-first_one_idx)

if self.padding_free:
    # Flush left to reduce the memory usage
    # input =             input =            pos_ids =           input =                    pod_ids =
    # [[x, x, x, x],  ->  [[x, x, x, x], and [[0, 1, 2, 3],  ->  [x, x, x, x, x, x, x]  and [0, 1, 2, 3, 0, 1, 2]
    #  [x, x, x, 0]]       [x, x, x]]         [0, 1, 2]]

    ... # code here

all right awesome actually this make more sense 😭

@osanseviero  before I push my code again I want to benchmark this with padding and padding_free just to show the performance

You can push it, no worry we can still refine after

Thank you for your understanding! I wanted to let you know that I’m a bit tied up today and tomorrow, so I might not be able to push the code right away. I’ll try to get to it as soon as possible, but please feel free to let me know if there’s a hard deadline I should prioritize.
Thanks for your patience!
I'll keep working on it so I'll try to push it by Tommorow if i can

No rush on our side :)

all right so I think this does it I did check if we can train this on a single T4 gpu colab notebook
now using the examples scripts provided the file trl/examples/scripts/dpo.p with a bit of update
I was able to train a model using the padding_Free =True
python trl/examples/scripts/dpo.py \
    --dataset_name trl-lib/ultrafeedback_binarized \
    --model_name_or_path Qwen/Qwen2-0.5B-Instruct \
    --learning_rate 5.0e-6 \
    --num_train_epochs 1 \
    --per_device_train_batch_size 2 \
    --gradient_accumulation_steps 8 \
    --gradient_checkpointing \
    --logging_steps 1 \
    --output_dir Qwen2-0.5B-DPO \
    --no_remove_unused_columns \
    --use_peft \
    --lora_r 32 \
    --lora_alpha 16

without padding_free it kept saying OOM is this normal or what ?

@osanseviero Just wanted to follow up on this PR and see if there’s any feedback so far. I’m happy to clarify anything or make updates if needed. Let me know whenever you get a chance—thanks so much for your time! 🙌

You still need to revert the changes applied to PPO files. And apply pre-commits

This is the metrics for the padding_free training trained on a T4 gpu colab notebook
configs
python trl/examples/scripts/dpo.py \
    --dataset_name trl-lib/ultrafeedback_binarized \
    --model_name_or_path Qwen/Qwen2-0.5B-Instruct \
    --learning_rate 5.0e-6 \
    --num_train_epochs 1 \
    --per_device_train_batch_size 2 \
    --gradient_accumulation_steps 8 \
    --gradient_checkpointing \
    --logging_steps 1 \
    --output_dir Qwen2-0.5B-DPO \
    --no_remove_unused_columns \
    --use_peft \
    --lora_r 32 \
    --lora_alpha 16

still testing the not padding_free

The new push changes some code due to these problems
nice but there is a few problem
this is training without padding_free
{'loss': 0.6933, 'grad_norm': 28.669958114624023, 'learning_rate': 4.880000000000001e-06, 'rewards/chosen':

 -0.0004301070875953883, 'rewards/rejected': -0.00019319055718369782, 'rewards/accuracies': 0.5, 'rewards/margins':

-0.0002369165886193514, 'logps/chosen': -359.64996337890625, 'logps/rejected': -218.63882446289062, 'logits/chosen':

-3.0950357913970947, 'logits/rejected': -2.8118934631347656, 'epoch': 0.02}

and training with padding_free
{'loss': 0.6931, 'grad_norm':<PHONE_NUMBER>0039062, 'learning_rate': 4.960000000000001e-06, 'rewards/chosen': 0.0, 'rewards/rejected': 0.0, 'rewards/accuracies': 0.0, 'rewards/margins': 0.0, 'logps/chosen': -972.31005859375, 'logps/rejected': -1276.14697265625, 'logits/chosen': -2.9294216632843018, 'logits/rejected': -2.5271308422088623, 'epoch': 0.01}
{'loss': 0.6791, 'grad_norm':<PHONE_NUMBER>6826172, 'learning_rate': 4.92e-06, 'rewards/chosen': 0.008736038580536842, 'rewards/rejected': -0.019646836444735527, 'rewards/accuracies': 1.0, 'rewards/margins': 0.02838287316262722, 'logps/chosen': -656.6194458007812, 'logps/rejected': -656.0517578125, 'logits/chosen': -2.9904067516326904, 'logits/rejected': -2.4395949840545654, 'epoch': 0.02}
{'loss': 0.6568, 'grad_norm':<PHONE_NUMBER>9726562, 'learning_rate': 4.880000000000001e-06, 'rewards/chosen': 0.020291520282626152, 'rewards/rejected': -0.05392017588019371, 'rewards/accuracies': 1.0, 'rewards/margins': 0.07421170175075531, 'logps/chosen': -653.7568359375, 'logps/rejected': -713.071044921875, 'logits/chosen': -3.067491292953491, 'logits/rejected': -2.60148286819458, 'epoch': 0.02}
{'loss': 0.6229, 'grad_norm':<PHONE_NUMBER>9316406, 'learning_rate': 4.84e-06, 'rewards/chosen': 0.02659912034869194, 'rewards/rejected': -0.12024041265249252, 'rewards/accuracies': 1.0, 'rewards/margins': 0.14683951437473297, 'logps/chosen': -1233.0953369140625, 'logps/rejected': -761.375732421875, 'logits/chosen': -2.791830062866211, 'logits/rejected': -2.6060781478881836, 'epoch': 0.03}

the grad_norm is so high for padding_free and the reward/accuracies is always 1.0 which is not correct

I was able to decrease the grad norm significantly by just normalizing the sequence_logps
{'loss': 0.6931, 'grad_norm': 0.4152209758758545, 'learning_rate': 4.960000000000001e-06, 'rewards/chosen': 0.0, 'rewards/rejected': 0.0, 'rewards/accuracies': 0.0, 'rewards/margins': 0.0, 'logps/chosen': -2.002589702606201, 'logps/rejected': -2.268846035003662, 'logits/chosen': -2.9294216632843018, 'logits/rejected': -2.5271308422088623, 'epoch': 0.01}
{'loss': 0.6931, 'grad_norm': 0.6821983456611633, 'learning_rate': 4.92e-06, 'rewards/chosen': 5.119443085277453e-05, 'rewards/rejected': -9.413660154677927e-05, 'rewards/accuracies': 1.0, 'rewards/margins': 0.0001453310251235962, 'logps/chosen': -2.0501720905303955, 'logps/rejected': -2.262946605682373, 'logits/chosen': -2.990408420562744, 'logits/rejected': -2.439610719680786, 'epoch': 0.02}
{'loss': 0.693, 'grad_norm': 0.5559526681900024, 'learning_rate': 4.880000000000001e-06, 'rewards/chosen': 7.167309377109632e-05, 'rewards/rejected': -0.00018270165310241282, 'rewards/accuracies': 1.0, 'rewards/margins': 0.00025437475414946675, 'logps/chosen': -1.6646722555160522, 'logps/rejected': -2.071936845779419, 'logits/chosen': -3.0675406455993652, 'logits/rejected': -2.601555347442627, 'epoch': 0.02}

@osanseviero
Train a qwen model for only 10 steps  using both padding_Free =Trrue and padding_free =False  with a batch_size of 1 with no gradient accumuatlion
GPU peak memory

Metric
Padding Free=False
Padding Free=True

PEAK MEMORY
13.9
9.0

here is the  lasts steps  metrics when training

Metric
Padding Free=False
Padding Free=True

Loss
0.6939
0.6914

Grad Norm
4.1626
5.1364

Rewards/Chosen
-0.00055
0.00207

Rewards/Rejected
0.00093
-0.00152

Rewards/Accuracies
0.0
1.0

Rewards/Margins
-0.00148
0.00359

LogPs/Chosen
-57.3851
-57.3589

LogPs/Rejected
-29.9984
-27.7588

Logits/Chosen
-2.8899
-2.8894

Logits/Rejected
-2.7367
-2.3579

Epoch
0.01
0.01

There still appear to be noticeable differences between the Rewards/Chosen and Rewards/Rejected metrics. Despite my efforts to resolve this,I just could not fix it
with gradient accumulation of 8 you can fit upto 4 batch for padding_free =True

(Please stop tagging osanseviero? Unless you've a good reason. He is not involved here, please don't bother him 🙏)


Update teams-for-vdi.md

Citrix Shim <IP_ADDRESS> fixed this issue

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@HowlinWolf-92 PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


possible issue with dp

Hi Michael and thanks for bignumber.js!
I run into an issue I don't fully understand yet. When I have a result that does not have decimal places, .dp() returns 0:
new BN('1').mul('10000000000000000000000000000').dp(6)
// returns 0

new BN('1').mul('10000000000000000000000000000').toString()
// returns '1e+28'

BN.config({ EXPONENTIAL_AT: 999 })

new BN('1').mul('10000000000000000000000000000').toString()
// returns '10000000000000000000000000000'

new BN('1').mul('10000000000000000000000000000').dp(6)
// returns 0

new BN('1000000000000000000000000000000.122134').dp(6)
// returns 6

I would expect to get a result like<PHONE_NUMBER>0000000000000000000.000000

related to bnjs@4


Selection interacts with models outside prefab mode, while inside prefab mode

This is fixed in version 1.535, I will close this ticket once this is available from the asset store


Add support for FSHy Parents

This PR adds support for FSHy Parents. This means that if the Parent of a Profile or Extension is another Profile or Extension, respectively, then the exporter process will export in the correct order so that Profiles and Extensions can be based on one another.
In order to test this, just run npm run check on this branch. I added what I believe to be comprehensive tests to cover various cases. If you think I'm missing any, let me know!

Comments that have been addressed from a code review:

The tests have been updated to check the metadata of parents, in order to verify that the parent information is actually being processed correctly.
The exporters have been refactored so that they take in a tank as an argument to their constructor, so it doesn't need to be passed into the export function. This way, there will be one exporter per tank when we have the capability to process multiple.
Some utility functions have been added to the FSHTank class so that we have access to them elsewhere. They have also been used in places that existed where they would have been useful.
Moved extraneous resolution functionality in to the resolver function rather than the exporter function. This way, the resolver can also be used to resolve parents that don't yet exist by first exporting them.


GCP - cannot remove service account permissions when using a service account key with only "editor" permissions

The change introduced in issue #37 has caused some issues tearing down CF Toolsmiths environments.
Some background info: our environments app can create what we call "custom GCP environments", which are PAS, SF-PAS, or PKS environments deployed by our app into a(n internal) customer's GCP project. In order to do this, our customers provide us with a GCP service account key. We specify (and validate in our app) that this key have editor permissions, but we do not require owner permissions.
Before April 25th, 2019, we created an Ops Manager service account in our customers' GCP projects, and attached the required Ops Manager permissions to it. We have since disabled the creation of new service accounts, but we still have a large number of environments lingering from before April 25th (approximately half of our custom GCP environments are these long-running environments). These environments may have user-provided service account keys with only editor permissions, which cannot delete the Ops Manager service accounts using versions of leftovers after v0.42.0.
As a result, we're a bit stuck—we want to upgrade to the latest version of leftovers to solve issue #89, but doing so would cause failures when tearing down these older environments.
After brainstorming with the Toolsmiths PM (@mjj209) we thought of a few options, of which these two are the most attractive:

switch between using old and new leftovers versions based on when an environment was created. This requires the least (i.e., no) work for the leftovers maintainers while being slightly hacky and weird for the Toolsmiths team.
when failing to remove member permissions on service account keys due to insufficient permissions, leftovers could output a warning instead of failing and exiting.

Option 2 is my personal preferred approach, although it requires some changes to leftovers and might be actively counterproductive for other teams' use cases. Before I go ahead and attempt to implement it, I'd like some thoughts about whether that's the right approach and if there are any other major approaches that I've missed.

I like option 2 but would this problem also be solved by the ignore/skip flag we’ve talked about?

We could an MVP of the skip flag for gcp iam resources and extend it to other resources as the demand arises.

That's even better. I think it would solve our problem while also not introducing a special case for handling permissions failures (which could cause all kinds of questions of the form "why do you allow users to try to delete this resource without permissions but not that one?").

Cool. So we'll add the --skip or --ignore flag to leftovers and in the README just indicate that this is currently only available gcp iam resources (which at this time is only service accounts.) Seems like a pretty simple, straightforward change and allows us to iterate! Love it.

We circled back to this issue in IPM today, where the rest of the Toolsmiths team had a few questions/comments/concerns. Specifically, when we delete PKS environments, we definitely need to delete service account permissions. Currently, we use the exact same deletion logic for PAS and PKS. Introducing a --skip or --ignore flag would also introduce a difference between the deletion code for these two kinds of environments. So we were thinking of going with option 1 (from my original post) and tabling --skip/--ignore until we really need it. Alternatively, someone had the idea that we could add a flag to optionally ignore permissions issues.

If you're comfortable with option 1 for now, that's fine by me.
Happy to continue to explore  --skip anyway for future uses.

Do you want to close this issue and keep the --skip flag issue as the source of truth?

Sounds good.


[#158791033] Use AWS internal ntp service

What
We want to use the AWS Amazon Time Sync Service[1] rather
than rely on the default pool.ntp.org.
We configure it in bosh following the example for GCP in
bosh-deployment[2] but using the AWS IP <IP_ADDRESS>
[1] https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/set-time.html
[2] https://github.com/cloudfoundry/bosh-deployment/blob/9b02de6c1b24fa770271f9b9bef75b19d5642902/gcp/cpi.yml#L62
How to review
Deploy this branch in your environment:
make dev deployer-concourse pipelines DEPLOY_ENV=hector BRANCH=$(git rev-parse --abbrev-ref HEAD) SELF_UPDATE_PIPELINE=false

and run the create-bosh-concourse pipeline
Run the main pipeline
All VMs shall get updated to the new ntp server. ssh to any and check the
file /var/vcap/bosh/etc/ntpserver.
You can run ntpdate -q $(cat /var/vcap/bosh/etc/ntpserver) to test it.
Who?
not me

I've tested this change, and it looks to work, however it required me to do a bosh recreate of the VMs for the change to apply, a normal run of the pipeline didn't apply this.

You are right.
Maybe we can merge this with other story that updates stemcells.
Not sure, your call I guess :man_shrugging: :smile:

I've added a commit to this to bump the Concourse stemcell. I've also raised https://github.com/alphagov/paas-cf/pull/1437 to bump the stemcells in paas-cf to force this to apply.

I've added a commit to this to bump the Concourse stemcell. I've also raised alphagov/paas-cf#1437 to bump the stemcells in paas-cf to force this to apply.

Makes sense :+1: . I think it shall be OK


Cache effectiveBalanceIncrements as Uint8Array

Motivation
I propose to change effectiveBalance array we keep in the epochCtx from MutableVector to a Uint8Array
Description

Change it from effectiveBalance cache to effectiveBalanceIncrements cache
Since EFFECTIVE_BALANCE_INCREMENT is a hard constant in the preset the increment is always less that 1 byte. The size of the data will be ~300KB (1 byte per validator)
On every epoch transition we just clone the Uint8Array, cloning bytes is really cheap
We will have extremely fast writes and extremely fast reads

Since effectiveBalances don't change much at all using MutableVector makes sense. However a Uint8Array will be extremely cheap on memory and since we are not expected to have 10s or 100s of states per epoch.
From @tuyennhv

the reason we use MutableVector there is cheap clone and shared memory. Divide it by EFFECTIVE_BALANCE_INCREMENT so that we can use Uint8Array sounds good potentially. The other down side is we have to multiply by EFFECTIVE_BALANCE_INCREMENT in all calculation, hope it does not cause cpu time.

Actually I think it reduces the amount of multiplications because we end up dividing by EFFECTIVE_BALANCE_INCREMENT almost everywhere
May main worry is:

What's the chance of MAX_EFFECTIVE_BALANCE_INCREMENTS > 255

Right now it's not practically possible to increase MAX_EFFECTIVE_BALANCE_INCREMENTS in Lodestar since all parameters that affect it can't be changed by consumers. However if we wanted to change the source to support higher MAX_EFFECTIVE_BALANCE_INCREMENTS I've made it easy to be able to change from Uint8Array to another TypedArray or a regular array.

However if we wanted to change the source to support higher MAX_EFFECTIVE_BALANCE_INCREMENTS I've made it easy to be able to change from Uint8Array to another TypedArray or a regular array.

:+1: likely Uint16Array would be plenty, but we can cross that bridge if we need to


[Bug]: Large stacks plotting too tight

Alexey, I have become a Patreon supporter to support your work.
When plotting large stacks, the plot is too tight. I suggest altering plot width based on length of dates:

Thanks. I'm surprised you didn't before because my articles and notebooks might be so helpful for your projects.
All the plots can be tuned using the common Matplotlib approach, such as:
plt.rcParams['figure.figsize'] = [12, 4]
plt.rcParams['figure.dpi'] = 300

In case you need to change a single plot only, a context manager helps to do it:
from contextlib import contextmanager
import matplotlib.pyplot as plt
@contextmanager
def mpl_settings(settings):
    original_settings = {k: plt.rcParams[k] for k in settings}
    plt.rcParams.update(settings)
    yield
    plt.rcParams.update(original_settings)

with mpl_settings({'figure.dpi': 300, 'figure.figsize': [12, 4]}):
    sbas.plot_baseline(baseline_pairs)

It's better to maintain smaller sizes for most of the plots to limit the notebook size.


Adapt griffin steady inputs for PJFNKMO

Regold transient for new restart point
Refs #46

Regold was necessary because of fluids update in #62


Tweak JWT refresh request so it works against prod

I thought I'd testing this against prod with service account
credentials, but I must have had user credentials set in my environment
instead. Tested using the command below:
$ export GOOGLE_APPLICATION_CREDENTIALS=<PATH_TO_MY_JSON_KEYFILE>
$ cd build-output/google/cloud/storage/examples/ &&
./storage_bucket_samples insert-object <MY_BUCKET_HERE> obj1 "abc"
This is a follow-up to
https://github.com/GoogleCloudPlatform/google-cloud-cpp/commit/21f9eb75301b75605f706551a058c62dfb79e294.

I added a couple of links describing the structure of a JWT and how Google specifically uses them in the context of service accounts.  Those should come in quite helpful for anyone that needs to troubleshoot or extend this functionality - thanks for requesting an explanation :)


10027: add e2e test

Summary
Fixes #10027

Occurred changes and/or fixed issues

add e2e test where filtering by either repo, category or text query should not affect entries in the carousel, as per description of #10027

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

@richard-cox  if this goes in, we can remove the label automation-candidate and remove the label QA-manual-test

this test looks good to me but maybe we add some checks for the other filter methods (filter by repos and categories). what do you think?

test updated @yonasberhe23


Fix icon links

Current icon link gives 404 so here is the updated link.

I just noticed that there are other apps with same issue. I'll update all of them in one go.

Ow god, I haven't checked PRs for a while. Sorry. You're right, you can continue fixing icons, or I can do it myself.

Allright, I've updated all icon links.

Thank you!
Squashing and merging...


Toolkit v1.0 – Basic Cost Management automation

📝 Scenario
As a FinOps practitioner, I need to automate the deployment of Cost Management resources in order to streamline onboarding
🚀 Goal
Publish modules to the Bicep Registry for all Cost Management features to simplify template creation.
📋 Tasks

### v0.1 – Basics ([discuss](https://github.com/microsoft/cloud-hubs/discussions/103))
- [ ] https://github.com/microsoft/cloud-hubs/issues/116
- [ ] #107
- [ ] #1
- [ ] Address all feedback from the release discussion

### v0.2 – Bicep modules for billing accounts ([discuss](https://github.com/microsoft/cloud-hubs/discussions/105))
- [ ] Cost allocation rules bicep module for billing accounts
- [ ] Cost details bicep module for billing accounts (TBD)
- [ ] Exports bicep module for billing accounts
- [ ] Reservation details bicep module for billing accounts (TBD)
- [ ] Scheduled actions bicep for billing accounts
- [ ] Settings bicep module for billing accounts
- [ ] Views bicep module for billing accounts
- [ ] Address all feedback from the release discussion

### v0.3 – Scripts for resources ([discuss](https://github.com/microsoft/cloud-hubs/discussions/106))
- [ ] Review gaps in PowerShell and CLI scripts
- [ ] Brainstorm script generation for PowerShell and CLI
- [ ] Cost allocation rules scripts
- [ ] Cost details scripts
- [ ] Exports scripts
- [ ] Reservation details scripts
- [ ] Scheduled actions scripts
- [ ] Settings scripts
- [ ] Views scripts
- [ ] Address all feedback from the release discussion

### v0.4 – Scripts for read-only APIs ([discuss](https://github.com/microsoft/cloud-hubs/discussions/107))
- [ ] Review gaps in PowerShell and CLI scripts
- [ ] Advisor recommendations scripts
- [ ] Alerts scripts
- [ ] Benefit recommendations scripts
- [ ] Benefit utilization summaries scripts
- [ ] Dimensions scripts (TBD)
- [ ] Forecast scripts
- [ ] Insights scripts
- [ ] Pricesheet scripts
- [ ] Query scripts
- [ ] Address all feedback from the release discussion

🙋♀ Ask for the community

We'd love to get your feedback! Please leave any questions, comments, or change requests in the release discussions at the bottom of each task list above. We'll try to close each of those out before we finalize the release.

Moving to projects


name.md does not document title

You can use name now to generate a job title, but it's not documented in https://github.com/stympy/faker/blob/master/doc/name.md

Faker::Name.title_job has been deprecated. Please use Faker::Job.

Thanks a lot.
On Sat, Jul 14, 2018, 11:28 AM Vitor<EMAIL_ADDRESS>wrote:

Faker::Name.title_job has been depreceated. Please use Faker::Job.
On Sat, 14 Jul 2018 at 11:21 Kevin Chugh, Ph.D, Founder, Main Street
Computing (mainstreetcomputing.com<EMAIL_ADDRESS>wrote:

You can use name now to generate a job title, but it's not documented in
https://github.com/stympy/faker/blob/master/doc/name.md
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stympy/faker/issues/1312, or mute the thread
<
https://github.com/notifications/unsubscribe-auth/ABO5DBCIREKUkZLS0iwCMxghDCHX___9ks5uGf5mgaJpZM4VP34U
.

--
Vitor Oliveira
<EMAIL_ADDRESS>+55 (21) 96758-9336
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stympy/faker/issues/1312#issuecomment-405030476, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB_i2t_jKBINOEWyNcml-os7rmeFV-Kvks5uGg4EgaJpZM4VP34U
.


Quests Maven Dependency

Where else have you looked for answers to this question?

Spigot Quests Site
readme.md on repo
deprecated wiki pages
new wiki pages

Share your question with as much detail as possible?
Our project uses Quests and uses the maven dependency here:
<repository>
        <id>elmakers-repo</id>
        <url>https://maven.elmakers.com/repository/</url>
</repository>
<dependency>
	<groupId>me.blackvein.quests</groupId>
	<artifactId>quests</artifactId>
	<version>3.6.2</version>
	<scope>provided</scope>
</dependency>

Is this still the correct location? We are trying to build our project through GitHub actions/CI and we get this error:
Warning:  Could not validate integrity of download from https://maven.elmakers.com/repository/me/blackvein/quests/quests/3.6.2/quests-3.6.2.pom
org.eclipse.aether.transfer.ChecksumFailureException: Checksum validation failed, no checksums available
    at org.eclipse.aether.internal.impl.AbstractChecksumPolicy.onNoMoreChecksums (AbstractChecksumPolicy.java:64)
    at org.eclipse.aether.connector.basic.ChecksumValidator.validate (ChecksumValidator.java:107)
    at org.eclipse.aether.connector.basic.BasicRepositoryConnector$GetTaskRunner.runTask (BasicRepositoryConnector.java:460)
    at org.eclipse.aether.connector.basic.BasicRepositoryConnector$TaskRunner.run (BasicRepositoryConnector.java:364)
    at org.eclipse.aether.util.concurrency.RunnableErrorForwarder$1.run (RunnableErrorForwarder.java:75)
    at org.eclipse.aether.connector.basic.BasicRepositoryConnector$DirectExecutor.execute (BasicRepositoryConnector.java:628)
    at org.eclipse.aether.connector.basic.BasicRepositoryConnector.get (BasicRepositoryConnector.java:262)
    at org.eclipse.aether.internal.impl.DefaultArtifactResolver.performDownloads (DefaultArtifactResolver.java:514)
    at org.eclipse.aether.internal.impl.DefaultArtifactResolver.resolve (DefaultArtifactResolver.java:402)

and
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  36.526 s
[INFO] Finished at: 2021-09-24T19:06:23Z
[INFO] ------------------------------------------------------------------------
Error:  Failed to execute goal on project WHIMC-ScienceTools: Could not resolve dependencies for project edu.whimc:WHIMC-ScienceTools:jar:2.4.2: The following artifacts could not be resolved: org.bukkit:bukkit1132:jar:1.13.2-R0.1-SNAPSHOT, org.bukkit:bukkit113:jar:1.13-R0.1-SNAPSHOT, org.bukkit:bukkit112:jar:1.12-R0.1-SNAPSHOT, org.bukkit:bukkit111:jar:1.11-R0.1-SNAPSHOT, org.bukkit:bukkit110:jar:1.10-R0.1-SNAPSHOT, org.bukkit:bukkit194:jar:1.9.4-R0.1-SNAPSHOT, org.bukkit:bukkit192:jar:1.9.2-R0.1-SNAPSHOT, org.bukkit:bukkit186:jar:1.8.6-R0.1-SNAPSHOT, org.bukkit:bukkit183:jar:1.8.3-R0.1-SNAPSHOT, org.bukkit:bukkit18:jar:1.8-R0.1-SNAPSHOT, org.bukkit:bukkit1710:jar:1.7.10-R0.1, org.bukkit:bukkit179:jar:1.7.9-R0.3, com.gmail.nossr50.mcMMO:mcMMO:jar:1.5.00, com.codisimus:PhatLoots:jar:3.8.1, ro.nicuch:CitizensBooks:jar:2.4.7, com.live.bemmamin:GPS:jar:2.2.2: Could not find artifact org.bukkit:bukkit1132:jar:1.13.2-R0.1-SNAPSHOT at specified path /home/runner/.m2/repository/me/blackvein/quests/quests/3.6.2/lib/craftbukkit-1.13.2.jar -> [Help 1]
Error:
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
Error: Process completed with exit code 1.

However, when I build locally, it works just fine (using IntelliJ 2021.2.1 Community Edition). Any and all help would be appreciated.

I'm afraid I can't speak for the integrity of builds hosted on that repository. Jitpack is currently recommended.


Add idempotency keys where needed in Client and queries

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

#38  👈
#37
main

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

Join @jklein24 and the rest of your teammates on  Graphite

Merge activity

May 14, 7:38 PM EDT: Graphite rebased this pull request as part of a merge.


Allow rendering collection of components

When rendering jbuilder partials, the following can be used instead of explicitly writing a loop that renders each partial:
json.partial! "posts/post", collection: @posts, as: :post

It would be interesting if we could have something similar for components. The example below assumes a PostComponent is defined.
# posts_controller.rb

class PostsController < ApplicationController
  def index
    @posts = Post.scope_that_queries_the_right_posts(params)
  end
end

# app/views/posts/index.html.erb

<%= render @posts %>

In this case, for each record we would get the associated component and render it. I'd love some feedback and implementations ideas. I can work on some implementation suggestions.

It seems like this is related to #21 no?

@juanmanuelramallo @vinistock let's close this in favor of https://github.com/github/actionview-component/issues/21 ❤


Feat/VBA-PivotTable-Tabular-Repeat-Label-No-Subtotals

Excel macro example for changing all PivotTables in the workbook to Tabular form, repeats labels and disables Subtotals.
Resolves #1196
Resolves #1182

@dbrennand Thank you the example!


Introduced GRPC_{TRUE,FALSE}

So that we can start using these in lieu of int literals. And then eventually replace those literals with the constants as well.
The rest of the changes to the file are cosmetic white-space removal.

Prefer: GPR_TRUE, GPR_FALSE in core/support.
I also have a strong preference for doing a complete sweep of the codebase and updating everything before introducing these. Doing it incrementally means that the interim state will be more confusing than where we are now.

Yeah, GPR_* fits better as it's as foundational a thing as it gets. I the gpr_* namespace is as public as the grpc_ one.
Agreed on postponing. Closing for now.


Null ls not started with command tab new

FAQ

[X] I have checked the FAQ and it didn't resolve my problem.

Issues

[X] I have checked existing issues and there are no issues with the same problem.

Neovim Version
NVIM v0.10.0 Build type: Release LuaJIT 2.1.1716656478
Dev Version?

[ ] I am using a stable Neovim release version, or if I am using a dev version of Neovim I have confirmed that my issue is reproducible on a stable version.

Operating System
Darwin wisers-MacBook-Pro.local 22.3.0 Darwin Kernel Version 22.3.0
Minimal Config
return {
  "nvimtools/none-ls.nvim",
  dependencies = {
    "nvimtools/none-ls-extras.nvim",
  },
  lazy = true,
  event = { "BufReadPre", "BufNewFile" },
  config = function()
    local null_ls = require("null-ls")

    -- custom setup
    null_ls.setup({
      border = "single",
      sources = {
        require("none-ls.formatting.jq"), -- json formatter
      },
      -- configure format on save
      on_attach = function(current_client, bufnr)
        if current_client.supports_method("textDocument/formatting") then
          vim.api.nvim_clear_autocmds({ group = augroup, buffer = bufnr })
          vim.api.nvim_create_autocmd("BufWritePre", {
            group = augroup,
            buffer = bufnr,
            callback = function()
              vim.lsp.buf.format({ async = false })
            end,
          })
        end
      end,
    })
  end,
}

Steps to Reproduce

nvim
:tab new
:set filetype=json
:NullLsInfo

Reproducibility Check

[ ] I confirm that my minimal config is based on the minimal_init.lua template and that my issue is reproducible by running nvim --clean -u minimal_init.lua and following the steps above.

Expected Behavior
:NullLsInfo

Actual Behavior
:NullLsInfo

Debug Log
No debug log because of null ls is not started.

Help
No
Implementation Help
No response
Requirements

[X] I have read and followed the instructions above and understand that my issue will be closed if I did not provide the required information.

In this case, filetype is only set after null-ls initialized, you have to manually start the sources.
I consider this not an issue? I'm open to discussing.

In this case, filetype is only set after null-ls initialized, you have to manually start the sources.

I don't know when and how null-ls initialized, but lsp server is started after set filetype.
PS: If null-ls's sources could be attached manually, please tell me, thx.

You could  could force it to start by starting the buffer with a file name, even if it does not exist yet
:tabnew tmp-file.json


Imported cards sometimes don't add up correctly.

GPO.csv
he_dump.csv
effects_remigrate.csv
effects_migrate.csv
When importing cards, some cards get added to the collection by the wrong amount. Some are missing, but most of the errors I noticed are cards being added by a comically large number. I managed to replicate most of the errors.
The csvs given are my cards that I imported to the database. I did it from the bottom file up.

Not sure what's up with this, will ask Benji to take a look.

I just tried another sequence and things broke differently.

One piece of info I forgot: On my main and first test account, I bought cards and packs from the shop first and ended up with 22 sandpaper. In my second test account, I did the importing first and ended up with 17 sandpaper. I did the same import sequence on all 3 accounts.

I think this is fixed as I made some changes to how card importing works and cannot replicate anymore. Let me know if you find issues again.

Still broken (Anvil×19 etc.)


[stable/dex] Remove allowPrivilegeEscalation from PodSecurityContext

The Kubernetes spec for PodSecurityContext does not allow to define allowPrivilegeEscalation.

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @vi7

Is just a fix in the comment, should I need to bump Chart version?

Hi @pipo02mix ,
yes, please update patch version of the Chart, I assume it should be bumped to 2.12.1

done


specify container ip on user.server

I'm not 100% sure container_ip is always the right value for this, but I think it is.

I think this works, too.
In SwarmSpawner I apparently set self.container_ip = "<IP_ADDRESS>" and then set self.user.server.ip equal to the IP address of the machine that it was started on by swarm. I don't entirely remember why they're not the same, but I think it's just because self.container_ip = "<IP_ADDRESS>" is a useful catchall that allows the container to run publicly but self.user.server.ip needs to be the real IP so that the hub can communicate with the server. But, if the container IP were correctly set to the real IP, then that would still work I think -- the difficulty in that case is that the container IP isn't known until the container is started by swarm, but I'm handling that in any case, so yeah, I think it's fine.

I think 90% of the time setting self.container_ip is a right value for self.user.server.ip. The other 10% of the time, it would have to be handled in a subclass.
An option to handle more cases would be to have a container_ip (or container_ip_bind) and a container_ip_connect.   container_ip  would be used for binding the ports on the host side as is done today.  self.user.server.ip would be assigned container_ip_connect is specified, else it would be assiend container_ip.  Since container_ip defaults to <IP_ADDRESS> in the config, not specifying either should not change behavior.
With this option, DockerSpawner probably handles the cases outside of using Swam or specialized network/firewall configuration.

@minrk I implemented my idea at bigzachattack/dockerspawner@b72dc5276187e0e40484b877232f2d00dae52b39
I built it ontop of your commit and rebased from upstream.


Whait db close a little bit before failing test

Overview
Databases are closed asynchronously. However it was not accounted for in
the test case. This commit waits database to close before it makes a
decision to abort the test. It uses default timeout of 5 sec.
I.e. if test case detects that database is still opened after 5 sec
it considers it as failure.
Testing recommendations
make eunit apps=couch_index tests=compaction_test
Checklist

[ ] Code is written and works correctly;
[x] Changes are covered by tests;
[ ] Documentation reflects the changes;

Looks good to me, didn't know we have test_util:wait/1,2,3 function. Except maybe pass timeout explicitly? " 'wait' for how long?" was my first question when I've seen the change.

@eiri: What timeout value would you recommend? I am considering 500.

@iilyak hard to guess, to be honest, 500 or 1000 both looks good to me. test_util's wait default is 5000, but it's more fitting for the tests around os_process related modules, it's common default there.

@wohali: Sorry for the delay.

@iilyak Hey I'm just glad people are starting to help ;)


Bar chart does not show correct total number of screened records

Hello,
Seems to have a similar issue with visualisation.
Here's a descriptive summary of my project details:

...and here's accompanying ROC curve:

Two questions seem to be clear:

why the chart chunks off at "13" while the number of relevant records is 18?
why the number of reviewed records is 137 in the chart, while the actual value is 152?
Prior knowledge was 5 relevant and 10 irrelevant records. This would, I guess, resolve question no. 1 (am I correct?) but not quite question no. 2.
Is this a kind of a glitch, or rather, more likely, I did something wrong?
Appreciate any assistance! :))

Discussed in https://github.com/asreview/asreview/discussions/1902

Originally posted by RiannevanderFeen October 23, 2024
Hello,
I had hoped to use the figures from the analytics in my paper, but I noticed that it does not show the correct total amount of screened records.
I screened >10% in phase 2 (2738), but the figure (the bar chart!) states 2462 as you can see below:

Hello!
For the general analytics section, we added a switch to change between showing or hiding the prior knowledge. So in this case, Total Records would switch between 137 (hiding prior knowledge) and 152 (showing prior knowledge).
This is the case for the general analytics. For the graphs, however, it's a bit more tricky. We opted to exclude the prior knowledge because of two reasons:

The graphs have a time axis, so, they are only meaningful if the labelling order is preserved. In most cases, prior knowledge doesn't come with chronology data, making the visualization inaccurate and confusing.
From our point of view, the analytics section is a way to understand your progress while using ASReview, as it reflects the performance of the models and the screener. Even though the prior knowledge labelling can also be done inside ASReview before starting the actual process, they belong to a different part of active learning.

So, the exclusion of prior knowledge would explain 2 of the 3 differences in your case. The differences between 18 - 13 and 152 - 137 are because of this.
However, this doesn't explain the 2738 - 2462 difference in the bar graph. Is it possible that your dataset had prior knowledge before you started screening? If not, it might be a bug with the axis values of the graph, I'll take a look.
Let us know if you think it's integral to see the prior knowledge in the graphs as well, we can try to come up with a clever solution.

Many thanks for this!
This is very much in line with my own intuition: prior knowledge is not in the graphs. I'm absolutely OK with that.
Thanks, all the best! ;)


Add support for apple OSX builds

[ ] Prepare cmake config package export too

This looks like it overlaps a lot with https://github.com/beman-project/optional26/pull/75 ?

I will wait and rebase than

will be replaced with https://github.com/ClausKlein/optional26/pull/1 after https://github.com/beman-project/optional26/pull/75 is merged


IOS Audio stops if user switches receives a call

Bevy version
Bevy 0.13.0
System information
iPhone 11, ios 17.2.1
If your bug is rendering-related, copy the adapter info that appears when you run Bevy.
What you did
Was playing a cool bevy game on my phone, received a call. When I resumed bevy app, there was no audio until I manually restarted the application.
Normal Task switching does not seem to trigger this behavior

(For reference, I'm part of the team making the cool bevy game) It may actually be a problem with bevy_kira_audio, since we use that


lab3: (5) Create SNS and SQS resources to pass on S3 events post-processing

This is specific to linking of access cards

SNS topic already exists. Just need to create the SQS Queue and subscribe to the topic.


Add comamnd LeaderfFiletype

I tried to implement filetype which is one of #451.
Please let us know if any changes are missing.
Thank you.

Thank you!


Fixed reappearing scanLocation markers on page refresh

The scanLocation markers reappeared on the map when refreshing the page even if the coverage-checkbox was set to false.

Did this instead 0f76744a562aabb777a3821d29ddf4dfa4890441


[2.x] Rename package 'o.o.action.support.master' to 'o.o.action.support.clustermanager' (#3556)

Description
Backport PR https://github.com/opensearch-project/OpenSearch/pull/3556 / commit https://github.com/opensearch-project/OpenSearch/commit/e7679240e0524316d03387dcb8a35166232fd971 to 2.x branch.

Rename package org.opensearch.action.support.master to org.opensearch.action.support.clustermanager
Rename classes with master term in the package org.opensearch.action.support.master

For an easier review, the change of class definition in the above package is starting with the file AcknowledgedRequest.java, you could search for it in the "Files changed" tab. The other changes are most of the references of the renamed classes.
Issues Resolved
The first step of issue https://github.com/opensearch-project/OpenSearch/pull/3542
On 2.x branch, the next step is to backport the commit https://github.com/opensearch-project/OpenSearch/commit/223d472e6d8ea4454cb05fba7271d213430a1a4e / PR https://github.com/opensearch-project/OpenSearch/pull/3593
Check List

[ ] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@dblock Thank you so much for reviewing! 👍👍


cant get the mqtt option to work

i have home assistant installed and the mqtt addon, i can see the ip of the raspberry pi that solar-sis is installed and running on the log as connected but dont know how to add it or sub to a topic like battery_voltage

My latest PR had MQTT activated. I don’t know how it was implemented before, I just write every value to mqtt so you would just sub to what you need, or use node red or hass to calc if that’s required I have migrated over to solpiplog since it has a GUI and so
Before I had MQTT enabled on solar sis I was using node red to pull the last value from influx dB, and then writing it to a MQTT topic for hass. Was working fine as well. You can also do commands via the sol sis http port, had that working rather well also, from hass.io -> MQTT then MQTT to http with node red to parse it.
I still have some of the code around if you need.
Regards
Jaco Goussard

On 17 Mar 2020, at 21:37, Josuan<EMAIL_ADDRESS>wrote:

i have home assistant installed and the mqtt addon, i can see the ip of the raspberry pi that solar-sis is installed and running on the log as connected but dont know how to add it or sub to a topic like battery_voltage
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

hi cybertza thanks for your reply if you could post your node-red flow so i can import it that would be awesome to try, thanks in advance

Hi
Sorry i dont have the full one i used to use anymore,
But here is an EG that can help you get there.
[{"id":"86cd1122.6012","type":"mqtt out","z":"a4af1532.59d5e8","name":"","topic":"Batrium/SOC","qos":"","retain":"","broker":"d38b37e2.c1ab18","x":2510,"y":1060,"wires":[]},{"id":"c4c0f1e.10f2d1","type":"mqtt out","z":"a4af1532.59d5e8","name":"","topic":"Batrium/ShuntVoltage","qos":"","retain":"","broker":"d38b37e2.c1ab18","x":2540,"y":1200,"wires":[]},{"id":"c877528b.9c4ba","type":"mqtt out","z":"a4af1532.59d5e8","name":"","topic":"Batrium/ShuntCurrent","qos":"","retain":"","broker":"d38b37e2.c1ab18","x":2540,"y":1240,"wires":[]},{"id":"3bf8ab5d.5c9a84","type":"delay","z":"a4af1532.59d5e8","name":"","pauseType":"rate","timeout":"5","timeoutUnits":"seconds","rate":"1","nbRateUnits":"5","rateUnits":"second","randomFirst":"1","randomLast":"5","randomUnits":"seconds","drop":true,"x":2320,"y":1200,"wires":[["c4c0f1e.10f2d1"]]},{"id":"eaa4f51c.b12df8","type":"delay","z":"a4af1532.59d5e8","name":"","pauseType":"rate","timeout":"5","timeoutUnits":"seconds","rate":"1","nbRateUnits":"5","rateUnits":"second","randomFirst":"1","randomLast":"5","randomUnits":"seconds","drop":true,"x":2320,"y":1240,"wires":[["c877528b.9c4ba"]]},{"id":"cbd322bd.b900e","type":"influxdb in","z":"a4af1532.59d5e8","influxdb":"5cca6087.b8801","name":"Pylon","query":"SELECT last(*) FROM \"pylon\"","rawOutput":false,"precision":"","retentionPolicy":"","x":1850,"y":1080,"wires":[["8fbc68fd.49eaf8","70ce6918.1f1dc8","14106cdc.bad0f3"]]},{"id":"8fbc68fd.49eaf8","type":"function","z":"a4af1532.59d5e8","name":"","func":"msg.payload = msg.payload[0].last_Pack_voltage / 1000;\nreturn msg;","outputs":1,"noerr":0,"x":2110,"y":1200,"wires":[["3bf8ab5d.5c9a84"]]},{"id":"70ce6918.1f1dc8","type":"function","z":"a4af1532.59d5e8","name":"","func":"msg.payload = ((msg.payload[0].last_Pack_remains_mAh + msg.payload[0].last_Pack_remains_mAh_P2) / (msg.payload[0].last_Pack_total_mAh + msg.payload[0].last_Pack_total_mAh_P2 ));\nmsg.payload = msg.payload * 100;\nmsg.payload = Math.round(msg.payload);\nreturn msg;","outputs":1,"noerr":0,"x":2110,"y":1080,"wires":[["86cd1122.6012"]]},{"id":"14106cdc.bad0f3","type":"function","z":"a4af1532.59d5e8","name":"","func":"msg.payload = msg.payload[0].last_Pack_current / 10;\nreturn msg;","outputs":1,"noerr":0,"x":2110,"y":1240,"wires":[["eaa4f51c.b12df8"]]},{"id":"d38b37e2.c1ab18","type":"mqtt-broker","z":"","name":"","broker":"<IP_ADDRESS>","port":"1883","clientid":"","usetls":false,"compatmode":true,"keepalive":"60","cleansession":true,"birthTopic":"","birthQos":"0","birthPayload":"","closeTopic":"","closePayload":"","willTopic":"","willQos":"0","willPayload":""},{"id":"5cca6087.b8801","type":"influxdb","z":"a4af1532.59d5e8","hostname":"<IP_ADDRESS>","port":"8086","protocol":"http","database":"powerwall","name":"","usetls":false,"tls":""}]

just play around a bit with the data coming in, and so forth, if you don't know node red, you will have a good understanding after this.
you would need the influx db palette for node.

Here is a control EG as well.
[{"id":"bd886be7.199e68","type":"http request","z":"14ecd37c.c379ed","name":"Make Request To Inverter","method":"GET","ret":"txt","paytoqs":false,"url":"","tls":"","proxy":"","authType":"","x":930,"y":560,"wires":[["9b720545.9df338","4646e1a0.dedd1"]]},{"id":"bd204bc4.aaa4f8","type":"inject","z":"14ecd37c.c379ed","name":"","topic":"","payload":"57.54","payloadType":"str","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":230,"y":500,"wires":[["5cee94d8.8ae8dc"]]},{"id":"bdd06f37.da03c","type":"inject","z":"14ecd37c.c379ed","name":"","topic":"","payload":"57.54","payloadType":"str","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":230,"y":540,"wires":[["fcbd6351.c9961"]]},{"id":"56b6aa03.49cd74","type":"inject","z":"14ecd37c.c379ed","name":"","topic":"","payload":"48","payloadType":"str","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":230,"y":580,"wires":[["247e1a84.a726d6"]]},{"id":"7d82692e.4c3108","type":"inject","z":"14ecd37c.c379ed","name":"","topic":"","payload":"55","payloadType":"str","repeat":"","crontab":"","once":false,"onceDelay":0.1,"x":230,"y":620,"wires":[["fb86792.3522788"]]},{"id":"5cee94d8.8ae8dc","type":"function","z":"14ecd37c.c379ed","name":"Set Bulk Charge","func":"msg.url = \"http://<IP_ADDRESS>:3001/set/bulk_charging?voltage=\" + msg.payload;\nreturn msg;","outputs":1,"noerr":0,"x":400,"y":500,"wires":[["45e58317.415bcc"]]},{"id":"fcbd6351.c9961","type":"function","z":"14ecd37c.c379ed","name":"Set Float Charge","func":"msg.url = \"http://<IP_ADDRESS>:3001/set/float_charging?voltage=\" + msg.payload;\nreturn msg;","outputs":1,"noerr":0,"x":410,"y":540,"wires":[["45e58317.415bcc"]]},{"id":"247e1a84.a726d6","type":"function","z":"14ecd37c.c379ed","name":"Set Back To Grid","func":"msg.url = \"http://<IP_ADDRESS>:3001/set/back_to_grid=\" + msg.payload;\nreturn msg;","outputs":1,"noerr":0,"x":410,"y":580,"wires":[["45e58317.415bcc"]]},{"id":"fb86792.3522788","type":"function","z":"14ecd37c.c379ed","name":"Back to Batery","func":"msg.url = \"http://<IP_ADDRESS>:3001/set/back_to_battery?voltage=\" + msg.payload;\nreturn msg;","outputs":1,"noerr":0,"x":400,"y":620,"wires":[["45e58317.415bcc"]]},{"id":"45e58317.415bcc","type":"delay","z":"14ecd37c.c379ed","name":"","pauseType":"rate","timeout":"2","timeoutUnits":"seconds","rate":"1","nbRateUnits":"1","rateUnits":"second","randomFirst":"1","randomLast":"5","randomUnits":"seconds","drop":false,"x":730,"y":560,"wires":[["bd886be7.199e68"]]}]

these were working with the latest build i made, for Dual pip, you may have to adjust accordingly to your calls file.

hi cybertza thanks for your time and example! got it imported and modified the influx, function and mqtt nodes, :) now i got stuck on how to edit my configuration yml to add it under sensors: if you dont mind write a little example i would appreciate it very much thanks!

this is a flow i have to get the values from a sonoff s31 and populating a influxdb, i would like to do the opposite, read from a influxdb and send data to a broker topic and display it in ha

platform: mqtt
name: "Washer Voltage"
state_topic: "tele/Washer/SENSOR"
value_template: '{{value_json["ENERGY"]["Voltage"] }}'
qos: 2
unit_of_measurement: 'V'
icon: mdi:flash-circle
availability_topic: "tele/Washer/LWT"
payload_available: "Online"
payload_not_available: "Offline"

[{"id":"1f187af9.f646c5","type":"mqtt in","z":"32901379.04e97c","name":"Fridge","topic":"tele/Fridge/SENSOR","qos":"2","datatype":"auto","broker":"","x":410,"y":660,"wires":[["9208302b.4b169"]]},{"id":"9208302b.4b169","type":"json","z":"32901379.04e97c","name":"ENERGY","property":"payload","action":"","pretty":true,"x":560,"y":660,"wires":[["3bc1f56e.68302a","bbfa7224.abcca"]]},{"id":"3bc1f56e.68302a","type":"function","z":"32901379.04e97c","name":"function","func":"if (msg.payload.ENERGY) {\n    var Power = { payload: msg.payload.ENERGY.Power, topic: "Power"  };\n    \nreturn [ Power ,];\n}","outputs":1,"noerr":0,"x":720,"y":660,"wires":[["9d48d14.d4c043"]]},{"id":"bbfa7224.abcca","type":"debug","z":"32901379.04e97c","name":"","active":true,"tosidebar":true,"console":false,"tostatus":true,"complete":"payload","targetType":"msg","x":730,"y":580,"wires":[]},{"id":"9d48d14.d4c043","type":"influxdb out","z":"32901379.04e97c","influxdb":"","name":"write Power","measurement":"Power","precision":"","retentionPolicy":"","x":910,"y":660,"wires":[]}]

` - platform: mqtt
name: "BVM AH Consumed"
state_topic: "BMV/AH_Consumed"

platform: mqtt
name: "BVM Amps"
state_topic: "BMV/Amps"
platform: mqtt
name: "BVM Amps"
state_topic: "BMV/Amps"
platform: mqtt
name: "BMV State Of Charge"
state_topic: "BMV/SOC"
platform: mqtt
name: "BMV Voltage"
state_topic: "BMV/Voltage"
platform: mqtt
name: "BMV Watts"
state_topic: "BMV/Watts"
platform: mqtt
name: "BMV Time To Go"
state_topic: "BMV/TimeToGo"
platform: mqtt
name: "BMV Time Full"
state_topic: "BMV/TimeTo100"
platform: mqtt
name: "Inverter SOC"
state_topic: "Inverter/BatterySOC"
platform: mqtt
name: "Inverter Volts"
state_topic: "Inverter/BatteryVolts"`

platform: mqtt
name: "Inverter 2 Solar"
state_topic: "tele/inverter2/pvwatts"
unit_of_measurement: "W"

those are in my sensor.yaml file or under the sensor heading if you don't use files.

got it to work! thank you for your time much appreciated!


Attempt upgrade to UE4-21

Right now, I'm attempting to upgrade USharp to the new UE4.21. Luckily there doesn't seem to be a lot of errors when attempting to compile a 4.21 project. I'm considering commenting out most of the code that isn't compatible with the new version of ue4, but I did make some changes that'll hopefully work fine.

When you upgrade to 4.21, make sure you go into your project's managed folder, delete the .usharpGenerated and USharp.ProjectGeneratedVariables.props files, and reopen your project. You'll see some errors, but simply close the project and recompile the managed solution, then reopen your project.

I want to let users with ue4.20 be able to compile usharp with no issues. That would be nice if Unreal Engine 4 has a way to check the engine version at compile time, similar to Unity's preprocessor statements. We might be able to create an artificial preprocessor, one for 4.20 and another for 4.21. But without visual studio configurations, I have no idea how I would do that.

Users with version 4.20 should now be able to compile with these changes.

These changes have been adapted into https://github.com/pixeltris/USharp/commit/1a34d49fd7bef3a3a7dc40362ebeb3fed9fb7108. Thanks!


pull request for e2e test at 02/03/2017 10:26:05

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Inconsistent style of top right "x" icon in Treebeard modals

Reported on Trello by me
The Dashboard's Legend modal and all the modals within Treebeard have two different "x" icons than the typical small, light gray "x" in other modals, such as the Change Avatar modal and Collaborative File Editing modal.

This is a treebeard change. I'll be working on this now and update osf.

Fix is sent here: https://github.com/CenterForOpenScience/osf.io/pull/3733


Options.rs singularization in FDB-gen for 6.1 API

fdb-client 6.1.8 installed, can use 6.0 bindings. Had to move forward from release-3.0
to master since I am running into the issue fixed by both #120 #121
Issue occurs on attempting to compile foundationdb crate
in bindings.rs
pub const FDBTransactionOption_FDB_TR_OPTION_USE_PROVISIONAL_PROXIES: FDBTransactionOption = 711;

Error:

error[E0425]: cannot find value `FDBTransactionOption_FDB_TR_OPTION_USE_PROVISIONAL_PROXY` in module `fdb`
   --> /home/testuser/rust/test-project/target/debug/build/foundationdb-c9fdf7023e981c03/out/options.rs:368:48
    |
368 | TransactionOption::UseProvisionalProxy => fdb::FDBTransactionOption_FDB_TR_OPTION_USE_PROVISIONAL_PROXY,
    |                                                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: a constant with a similar name exists: `FDBTransactionOption_FDB_TR_OPTION_USE_PROVISIONAL_PROXIES`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0425`.
error: Could not compile `foundationdb`.

To learn more, run the command again with --verbose.

Seems like doing this fixes that issue
Assignee
if opt.name == "AppendIfFit" {
    opt.name = String::from("AppendIfFits");
} else if opt.name == "UseProvisionalProxy" {
    opt.name = String::from("UseProvisionalProxies");
};

Places to be edited for that to function: lines 64, 77, and 283
Reason for issue:
inflector::cases::screamingsnakecase singularizes plural words by default. I wonder if there's a way to turn that off.

A different solution could be to update Inflector in foundationdb-gen/Cargo.toml to use default-features = false and keep the same version. Because of that, classcase::to_class_case would need to be changed to pascalcase::to_pascal_case, because classcase depends on singularize


Ich habe ein paar Zigbee Sticks getestet. 2 mit denen es mit der alten Firmware (CC2531ZNP-Prod_20181024) funktioniert hat und einen neuen auf den ich nur die neuste (CC2531ZNP-Prod_20190109) aufgespielt habe. Sobald ich den serial Port hinterlege und der Adapter neustartet fängt der Zigbee Stick an rot zu blinken ab dem Log Punkt: Reset coordinator, mit der alten Firmware startet der Adapter normal durch und die LED geht aus bzw. leuchtet grün. (Ich kann zwar trotzdem keine Geräte verbinden, aber dieses Problem habe ich schon seit längerem auf ein mal...)
Hat noch jemand dieses Problem?
System:
rock64 mit OMV 4.X
ioBroker 1.4.2
Zigbee Adapter 0.9.2

Bitte englisch.

In English:
Hello, unfortunately the latest firmware has some problems. I have tested a few Zigbee sticks. 2 with the old firmware (CC2531ZNP-Prod_20181024) which has worked and a new on which has the newest (CC2531ZNP-Prod_20190109) on it. As soon as I enter the serial port and the adapter restarts, the Zigbee Stick starts flashing red from the log point: Reset coordinator, with the old firmware the adapter starts normally and the LED goes off or glows green. (Although I can not connect devices anyway, but I have this problem for a long time at a time ...) Does anyone else have this problem?
System:
rock64 with OMV 4.X
ioBroker 1.4.2
Zigbee Adapter 0.9.2

I have a CC2531 and also a Rock64 (though not with OMV but rather armbian). Could you try with the very latest firmware from Feb 23nd? It is from the development branch but works quite well for me:
https://github.com/Koenkk/Z-Stack-firmware/tree/dev/coordinator/CC2531/bin

I'm pretty new to ZigBee and CC2531, but I right away bought two pieces of CC2531. One I started to use with zigbee2mqtt the other I used for sniffing with TI Wireshark Packet Converter.
With the latest dev firmware, the CC2531 started to work very poorly, short distance reception (1-2 meters), often not even receiving anything. I thought that I could try the other CC2531 (which was flashed for packet capture), and re-flashed it with the same latest dev, and voilà, everything started to work perfectly. A startup I even saw some short term red lights coming from the CC2531, which I have never seen before on the first CC2531 dongle.
I don't know if the bad adapter worked better with previous firmware, I might do some tests with that adapter.
Anyway, it might be a bad quality CC2531 causing some issues.

The short term red lights are normal during first initialization.

The short term red lights are normal during first initialization.

Its longer than 10 Minutes (after 10 i gave up).
I now migrate my NAS to ESXi and will try it later again

With the newest version 23.02 the problem is solved. The red light is only for a very shot time and than it change to green.


Fix CI tests hanging

SQLite has issues with concurrent connections.
This line:
https://github.com/bitcoin-s/bitcoin-s-core/blob/50565ee4b1a0203f3d319a188041b80d9f001710/build.sbt#L344
Forces tests to run sequentially, fixing the problem.

Is there any reason this can't be fixed by just using in-memory dbs alongside persistent read-only dbs where needed?

@nkohen I swapped out the unit test DBs for in-memory databases. We got a nice speed improvement, but interestingly the deadlock issue still persisted. I'm guessing that the in-memory version of SQLite is a rather dumb implementation, so it probably has some limitations left over from file-based DBs that aren't changed.


Option to select different animations for FAB showing.

You can select different animations for showing the FAB buttons.
In fab_dialer.dart, I did create another constructor parameter (optional) that allow passing an enumeration item for different animations. Using this optional parameter we can avoid any breaking changes. If this parameter is omitted then the default animation (current animation = scale) will take place.
I added 2 new animations using "Staggered animation": fadeIn (fade all small FAB buttons and text) and slideInDown (slide each small FAB button from top to the correct position). More animations can be added later.
How to call:
// Default call without selecting the animation
new FabDialer( [ ... <items> ...], Colors.red, new Icon(Icons.add));
or
// Default call selecting a different animation
new FabDialer( [ ... <items> ...], Colors.red, new Icon(Icons.add), AnimationStyle.slideInDown);

Tested up to 9 items in an IPhone 7 and up to 9 items in and Nexus 6P.
Must update documentation, maybe add a gif with each different animations.

Thanks man, It's very helpfull. I will test it myself to see how it works.


Application error ERR_FILE_NOT_FOUND

Hi,
Am having an issue with livereload on an android device (haven't used an others) where after the app is installed to the phone (using "cordova run android -- --livereload") I get this alert:
Application Error
net::ERR_FILE_NOT_FOUND (file:///android_asset/www/homePage_0.5324000904802233omePage.html)
(assume the number is random)
If I do "cordova run android" it installs and runs fine.
Using

Windows 10
Node version v4.2.4
Cordova 5.4.1

Tested with blank install of cordova project and only plugin was livereload.
Tried to install as per the documentation, but running the "cordova plugin add cordova-plugin-livereload" line before the project was created created an error for me.
Any help would be appreciated to show me what I might be doing wrong.
Thanks very much,
Jarrod

Thanks for bringing this issue up, will take a look ASAP.

Thanks very much. Let me know if I can provide any more info. I didn't see
anyone else with this issue in my searches, so hope it isn't just me.
On 4 Jan 2016 22:11, "Omar Mefire"<EMAIL_ADDRESS>wrote:

Thanks for bringing this issue up, will take a look ASAP.
—
Reply to this email directly or view it on GitHub
https://github.com/omefire/cordova-plugin-livereload/issues/54#issuecomment-168826908
.

This could very well be the same issue as #47, console output is the same when using chrome's remote debugging, this could very well be a two birds with one stone solution. :smile:

Sorry, guyz. I'll try and have a fix by the end of this week. So far, I haven't repro'd this yet.

Is there anymore info I can provide to help?
On 12 Jan 2016 22:53, "Omar Mefire"<EMAIL_ADDRESS>wrote:

Sorry, guyz. I'll try and have a fix by the end of this week. So far, I
haven't repro'd this yet.
—
Reply to this email directly or view it on GitHub
https://github.com/omefire/cordova-plugin-livereload/issues/54#issuecomment-171088469
.

Getting similar error (see attached). Only happens on Android device (Note4 with OS 5.1.1). Over Firefox/Chrome it works fine from all devices on same network.
I'm using:

Win10 as my dev machine
NPM 3.3.12
Cordova 5.4.1
cordova-plugin-crosswalk-webview 1.5.0
cordova-plugin-livereload 1.0.0-dev

Does anyone have any idea how to fix this?
Thanks!


Failed to compile while performing Sync on Web App\App Service

When I try to sync the service application, I encounter the following error:
'Quick solution for the application service authentication issue (#1226)'
This implementation was done from scratch

Hi @rramos9771 ,
Thanks for raising the query. We are looking into this issue on priority and keep you posted

Hi @rramos9771 ,
Thank you for the patience. The app service sync failure issue is now resolved, and the changes have been pushed to Company Communicator official GitHub repository.
If current deployed version of the Company Communicator application is pointing to v5.5 (latest version), please follow the below procedure to resolve the issue -

Go to Azure Portal --> Go to Resource Group (where the CC deployment has been done) --> Select Web App (App Service) --> Go to Deployment Center --> Click on SYNC

Once the status shows successful within logs section of Deployment Center.

Note: Please make sure the Repo URL and Branch name is pointing to below values in Deployment Center Screen of app service

Try accessing the CC application on Teams web client to avoid any cache related issue.

If current deployed version of the Company Communicator application is pointing to v5.4.1 or previous versions, please upgrade your app to latest version by following the migration guide as the fix is pushed to the main branch in Company Communicator's official GitHub repo.
If it’s a fresh deployment then proceed with the remaining steps as per the deployment guide.
Please let us know if you are still facing the issue.


Fix incorrect code example in docs

Minor changes to docs:

Add return type to the slice function
Change erroneous arg names samples and properties in docstring code examples of slice and slice_block fxns to samples_to_slice and properties_to_slice.

Changes made and commits squashed


[3.1.0][DEVPORTAL] Header resource parameter (authorization) not send on Try Out

Description:
I've added on one of my API resource a parameter of type Header named Authorization. When trying to use it on the devportal this header is never sent when using the try out with a value on this header... It looks like this issue is only existing for this specific Header parameter "Authorization", if I do the same with another Header parameter name, it works.
Steps to reproduce:

On the publisher :

Create an new API
In "Runtime Configuration" :

Activate Oauth2, Basic Auth and API Key security
Replace the default Authorization Header in "Application Level Security" by something like "AuthorizationWSO2"

Add a new resource API with Security activated

Add parameter type "Header" with name "Authorization"

On devportal :

On Try Out, try to make a call to the resource with a value on the Header Parameter using an API Key

The Authorization parameter will never be send if you check on the network console and the curl example will not diplay it

If you add another Header parameter named for example "HeaderTest", and you try again, this one will be send and display on the curl sample...

Affected Product Version:
API Manager 3.1.0
Regards,
Jérôme.

Hello,
Can someone tell me if a fix will ne done on this ?
Regards,
Jerome

After a check on the 3.2.0 Beta, I saw that this issue is fixed


Update Appium to latest

This should solve the compliance issue

Abandoning this. We can't update JUnit as it hasn't been tested on App Center Test yet, even though it is the component that triggers the "Component Governance" issue.


Easy way to inject and test custom javascripts

It'd be nice if you either create a way, or if there already is one just describe it in README, to add custom JS and test it without recompiling whole app.
If it's too much to ask for, just enable Inspector in WKWebView (either by default or when some kind of preference or runtime option is passed) :)

When compiling the Swift project locally, a local JS-file is used. See https://github.com/danielbuechele/goofy/blob/master/Goofy/AppDelegate.swift#L43
But adding a hidden preference or something to change the path of the JS-file would also be nice!

I'm not sure if it's a good idea to keep .js files on external server, especially unversioned. If you ever switch from AWS to something else, everyone will have a very bad time :)

Loading the JS from an external server is super convenient for responding to changes on Facebook's side and quickly adding features. However, linking to a domain that I control and introducing version would make sense.

Then I guess there're two issues here:
[ ] moving away from AWS domain and into something like static.goofyapp.com
[ ] adding either advanced preference or --option to inject custom CSS/JS for debugging and development reasons

In the current version you can set a custom URL for the JS in Goofy's Info.plist using GoofyJavaScriptURL

The JS-file is now served from another host. There can be specific versions of the JS-file and there is a fallback if there is not JS-file for this specific version.


fix(consensus): avoid task starvation when when executing payload

Description
avoid task starvation when when executing payload
Motivation and Context
Execution can take anywhere from 200ms to a few seconds while blocking the thread and any tasks queued for execution on that thread. This PR signals to tokio that this will happen so that tokio requeues other pending tasks on another thread.
This is still far from ideal but should be better until we decide on the current concurrency model for consensus.
How Has This Been Tested?
Manually

Good start, as you say. I think it should be spawned, but let's not optimize just yet

Yeah definitely,  spawn_blocking would require a largish refactor so left it for now


Undefined error in PropTypes.node

Facing error on line 21 in the library.
_react3.default.PropTypes.node in undefined.
version
react: 16.0.0
react-native: 0.50.3,

I have this exact same error, might have to use a previous version of react native.

I have the same issue, I looked it a bit and I found that the problem with View.propTypes is deprecated and instead ViewPropTypes is used from RN44.
will you guys upgrade soon?

react-native-swiper works well for me though if you're looking for an alternative .

Should be fixed as of 0.1.8

@phil-r Too late broda

¯\_(ツ)_/¯


Not able to open a channel to a Tor node?

Getting a "HTTP error 500", when trying to open a channel to a node with Tor address.

This is a know bug, will push a fix sometime this week


Passing Data though Routes

Hey there,
I think the ability to pass data through Routes, without having the Data as a parameter, would be great. For example, if I wanted to pass through which Template I'd like different pages to use:
<Route path="/" handler={ App }>
	<DefaultRoute handler={ Dashboard } data={{ template: 'panel' }} />

	<Route name="welcome" handler={ Splash } data={{ template: 'splash' }} />
</Route>

Am I blind and there is a way of doing something like this? I couldn't find anything in the Docs.

Route doesn't pass props like that, try creating a factory function that will return a react element.
<Route name="Welcome" handler={SplashFactory({foo:bar})} />

export default (data) => {
 return <Splash data={data} />
}

master branch (1.0) lets you add whatever meta data you want to your routes and it will be available where you've got a handle on the route


Move disabled cars to bottom of list in trip entry

Trip Entry displays the names of the cars that are disabled still in the page.  This should be moved to the bottom of the list instead of seemingly randomly listed.

Code complete.


Sticky state DOM tree not being destroyed when leaving parent

I have my states laid out like so:
- flow1
- flow2 (sticky, named view)
    - flow2.substate

When I navigate from flow1 to flow2.substate and then go back to flow1 the DOM for flow2 is not being destroyed.  Given the following from the Sticky State documentation, this seems incorrect:

The DOM and $scope of a Sticky State (and all activated substates) is retained until the parent state of the sticky state is exited.

I created an example demonstrating this behavior: http://plnkr.co/edit/QIW3MNItJNtJPwwjgCFD?p=preview
I'm not sure if I have a fundamental misunderstanding, or if there is a legitimate issue.

Hi, the parent state of your sticky state is the unnamed root state (""), which is never exited.  That behavior is what I would expect out of sticky states.  cheers! :beer:


Enable Secureboot images on MS Azure (stage0)

[ ] Finish implementation based on research
[ ] Prepare and execute platform tests
[ ] Produce images

@yeoldegrove , one manual run of platform tests once in 2024 is OK if it saves us effort when it comes to going directly to openTofu with Azure for secure boot

@pnpavlov @mguertler  I verified that these flavors (not being built currently) are bootable via our new openTofu approach:
azure-gardener_prod_trustedboot-amd64
azure-gardener_prod_trustedboot-arm64
azure-gardener_prod_trustedboot_tpm2-amd64
azure-gardener_prod_trustedboot_tpm2-arm64

We could enable the build an publishing (to S3) of these images and deal with the platform tests later, e.g. when enabling platform tests with openTofu (#2335).
openTofu upstream support for secure bootable azure images is (with custom keys) is still WIP, though.


The Example of Restore the AdventureWorks database can not execute properly

the Example is wrong
kubectl exec sqltest1-0 -n arc -- -c arc-sqlmi /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P MyPassword! -Q "RESTORE DATABASE AdventureWorks2019 FROM  DISK = N'/var/opt/mssql/data/AdventureWorks2019.bak' WITH MOVE 'AdventureWorks2017' TO '/var/opt/mssql/data/AdventureWorks2019.mdf', MOVE 'AdventureWorks2017_Log' TO '/var/opt/mssql/data/AdventureWorks2019_Log.ldf'"
it should change to
kubectl exec sqltest1-0 -n arc -c arc-sqlmi --  /opt/mssql-tools/bin/sqlcmd -S localhost -U sa -P MyPassword! -Q "RESTORE DATABASE AdventureWorks2019 FROM  DISK = N'/var/opt/mssql/data/AdventureWorks2019.bak' WITH MOVE 'AdventureWorks2017' TO '/var/opt/mssql/data/AdventureWorks2019.mdf', MOVE 'AdventureWorks2017_Log' TO '/var/opt/mssql/data/AdventureWorks2019_Log.ldf'"

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

ID: 72dec991-e39b-2a9a-442b-5d65442874b8
Version Independent ID: 1278cd31-37f7-4baa-fcc7-b7103282d378
Content: Restore the AdventureWorks sample database into SQL Managed Instance - Azure Arc
Content Source: articles/azure-arc/data/restore-adventureworks-sample-db.md
Service: azure-arc
Sub-service: azure-arc-data
GitHub Login: @twright-msft
Microsoft Alias: twright

@tomleetaiwan We have verified and updated the document as appropriate. The change should reflect soon i.e., in less than 24 hours. Thanks again!


'Properties' Panel isn't refreshed after focusing next panel using F6

Storage Explorer Version: 1.2.0
Platform: All
Architecture: ia32/x64
Build Number: 20180605.1
Commit:f8373d5a
Regression From: Not a regression
Steps to Reproduce:

Launch Storage Explorer and select one account.
Focus on 'Properties' Panel using F6.
Press F6 again.
Check whether the 'Properties' Panel is refreshed or not.

Expected Experience:
The 'Properties' Panel is refreshed.(The focus of first property in 'Properties' Panel is lost)
Actual Experience:
The 'Properties' Panel isn't refreshed.
More info:
The issue doesn't reproduce after focusing previous panel using Shift+F6(Explorer Panel).

I'm a little confused. What is the basis of your expectation that the panel is supposed to refresh? Properties are generated only once when they are needed, which is when the node is selected for the first time. The only way to refresh the properties is to refresh the node itself.

Hi Craig,
For this issue, when we focus to 'Properties' Panel from 'Explorer' Panel using F6, the first property is focused by default like below.

Then press F6 again to focus the container editor Panel.
Expect:
The focuse of the first property in previous panel is lost.

Actual:
The focus of the first property in previous panel still exists.

OK, so this isn't about "refreshing", right? Just focus in the panel.
I tested it out myself, and it's behaving as expected. What you're seeing is not focus. It's selected inactive text. If you look at the tabs (or Activity Log if there aren't any tabs), you should see that the next panel does have focus. In fact, look closely, and you should see the text box in question no longer has a border around it.

Yes, This issue just invoke focus in the panel.
Inaddition, thanks for your sharing and updates.


Fix error when receiving events

#158 had a bug that prevented any events from being received. It was caused by the events getter, which was really just trying to support backward compatibility with projects using context.event. I don't see any public code that uses that, so removing the getter.
cc @hiimbex

0.7.1 is released


Make libssl provider aware

This PR implements the various changes required to make libssl provider aware.
Through this PR is will be possible to specify a libctx and/or property query string on a per SSL_CTX basis. Ultimately this will enable (for example) one SSL_CTX to be used with the default provider and another SSL_CTX to be used with the FIPS provider.
All the various algorithms (ciphers, digests, keyexch etc) are fetched using the specified libctx/propqueries.
Unfortunately it is not possible to entirely complete this process at the moment because keygen/paramgen is still not converted to be provider aware. Those things call the "RAND" subsystem using the default libctx - so even though you may have specified a different libctx with only the FIPS provider loaded they will still end up using the default ctx which may have the default provider available. This will be fixed when keygen/paramgen is finished.
I've marked this as WIP because it depends on #10845 and includes the commits from there. It also has  a number of other miscellaneous fixes and patches that I had to implement along the journey which are only loosely connected to the final objective. I plan to split a number of those out into separate PRs to make this easier to review. However I thought it was helpful to have this PR here to show the ultimate goal.

Travis is now green.
Ping?

Ping.

Fixup pushed addressing all comments. Please take another look.

Looks good now.

Pushed. Thanks.


Implement simple encryption

Allow the user to simply encrypt single notes using this library: https://www.dlitz.net/software/pycrypto/
>>> from Crypto.Cipher import AES
>>> obj = AES.new('This is a key123', AES.MODE_CBC, 'This is an IV456')
>>> message = "The answer is no"
>>> ciphertext = obj.encrypt(message)
>>> ciphertext
'\xd6\x83\x8dd!VT\x92\xaa`A\x05\xe0\x9b\x8b\xf1'
>>> obj2 = AES.new('This is a key123', AES.MODE_CBC, 'This is an IV456')
>>> obj2.decrypt(ciphertext)
'The answer is no'

Use gpg for that!


Change of State from Review to Call for Records

As a FOI Analyst, when a request has a completed call for records action, and the file is under review
I want to be able to revert the state back to CFR
so that I can trigger another records search

Assumptions & Scope
This story addresses the occasional need for a request to have another call for records while under review.
Acceptance Criteria
Scenario 1: Change Status to Call for Records from Review State

GIVEN a request is in the review state, after a completed call for records status
WHEN the status drop down is activated, AND Call for Records is Selected
THEN a prompt will appear asking if you are sure you want to change the state back to call for records, with the option of Yes or Cancel

Scenario 2: Status is Changed to Call for Records

GIVEN an analyst has changed the request state back to call for records
WHEN the yes option is selected
THEN a modal will pop out, asking if a new CFR Due date is needed
AND will give radio options of “Yes”, with a blank text space next to “Yes” for number of days which will not be active
AND There will be a two buttons – “Save new Due Date” (Inactive) and “Proceed without Due Date” (Active)

Scenario 3: Proceed without Due Date Selected

GIVEN an analyst has reverted the status back to Call for Records
WHEN the “Proceed without due date” button is clicked
THEN the modal will close, and the status of the request will change to “Call for Records” with no Call for Records deadline/countdown showing

Scenario 4: Due Date Selected as Yes

GIVEN an analyst has selected the due date option as “Yes” in scenario 2
WHEN the yes option is selected
THEN the text space for number of days will be editable

Scenario 5: Number of Days Entered

GIVEN an analyst has selected the due date option as “Yes” in scenario 2
WHEN the analyst enters a number into the text box
THEN the “Save new due date” button will become active.

Scenario 6: Save new Due Date Activated

GIVEN an analyst has entered the number of days in the text box
WHEN the analyst clicks “Save new Due Date”
THEN the modal will close, and the status of the request will change to “Call for Records” with the days remaining appearing for the Call for Records on the navigation panel.
AND the clock for the number of days will function the same way as in story #452

Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

@lmullane - this is the story out of refinement yesterday. I may have made it too big, I was thinking about the fact that most call for records triggered late wouldn't necessarily have a due date, but analysts may want to set one depending on the circumstances. Let me know if I should split this up, and we could consider establishing a due date later, and just start with a more simple change of state.
@arielleandrews - feel free to have a look as well!

After discussion, closing this ticket as not necessary. Application already handles this state change at a basic level and we are not prepared to add any sophistications to clock countdowns at this time.


[Products] Broken product comparison table and UI issues

https://franklin.moleculardevices.com/products/microplate-readers/absorbance-readers
https://franklin.moleculardevices.com/products/microplate-readers/multi-mode-readers
Fix the specifications table on both the pages.

Also,

Fix the large h2 titles
Under "applications and assays" fix bullets alignment, add arrow icon and fix spacing between the TOC bullets
Centre align the titles
Add anchor link on view products CTA in the banner

https://franklin.moleculardevices.com/products/microplate-readers/multi-mode-readers

https://franklin.moleculardevices.com/products/microplate-readers/absorbance-readers

Reduce img size

https://franklin.moleculardevices.com/products/microplate-readers/fluorescence-readers
https://franklin.moleculardevices.com/products/microplate-readers/luminescence-readers
Reduce font size and make it blue

For the "ImageXpress HCI system comparison", please make the order of products like the below screenshot from live. Also please keep the same title
https://franklin.moleculardevices.com/products/cellular-imaging-systems

@MeghnaTewari we can try to keep the same order of products.
Re. title: the titles on original page were handcrafted and not similar to anywhere else. In the new table we retrieve them from the product page. As agreed with @dev-rajneeshkumar and @kapildhiman before, we take this title from product page as the source of truth.

@duynguyen Any update on the comparison table issue? They are severely broken

https://franklin.moleculardevices.com/products/microplate-readers/multi-mode-readers
https://franklin.moleculardevices.com/products/microplate-readers/absorbance-readers

@MeghnaTewari yes because content was not published. I just did and table seems to display fine now.

@alinarublea addressing a couple of common issues for both the tables -
https://issue-873--moleculardevices--hlxsites.hlx.page/products/microplate-readers/absorbance-readers
https://issue-873--moleculardevices--hlxsites.hlx.page/products/microplate-readers/multi-mode-readers

Please remove extra space from here

Please increase the width to add some more space on the right

Add a little space above the product images

Fix the URL of id3 and id5, showing 404

https://issue-873--moleculardevices--hlxsites.hlx.page/products/microplate-readers/multi-mode-readers

Fix the URL for ABS, showing 404

https://issue-873--moleculardevices--hlxsites.hlx.page/products/microplate-readers/absorbance-readers

@MeghnaTewari fixed 4 and 5

@MeghnaTewari can you check again should be looking a little better

Yes, looks better!
Can we have the default scroll bar set to the left when the page is opened?
Like this

Please remove some more space

@MeghnaTewari the space should look better now, but the scroll is still starting from the other direction

@duynguyen @alinarublea I just noticed that this issue is marked completed but styling issues on category pages are still not fixed.
cc @mhaack

Also,

Fix the large h2 titles
Under "applications and assays" fix bullets alignment, add arrow icon and fix spacing between the TOC bullets
Centre align the titles
Add anchor link on view products CTA in the banner

https://franklin.moleculardevices.com/products/microplate-readers/multi-mode-readers

https://franklin.moleculardevices.com/products/microplate-readers/absorbance-readers
  Reduce img size
https://franklin.moleculardevices.com/products/microplate-readers/fluorescence-readers https://franklin.moleculardevices.com/products/microplate-readers/luminescence-readers
Reduce font size and make it blue

@MeghnaTewari I added more space to the flourescence and luminescence pages, and I also opened this pr for the bullet points if you can check https://github.com/hlxsites/moleculardevices/pull/998 .

@alinarublea

Please add arrow icons for all the application titles and apply bold

Move AlphaScreen/AlphaLISA below Western Blot
Please align these 3 titles together so it looks better


Model result

hello,
after training the Xception model over 1000 episode. the restarting episode always come after 2 step, the agent is not moving properly.
is the problem of my PC low performance.

Hey, can you tell me how you managed to run the code? Highly appreciated.

Hey, can you tell me how you managed to run the code? Highly appreciated.

Hey, can you tell me how you managed to run the code? Highly appreciated.

I follow the tutorial on YouTube Sendtex RL in the CARLA


Preserve error message from socket's error event

The close event's argument is simply true on error, but the error event gets a real error object.

Thanks, @Yogu.