test modification of static_ips in Terraform configs

About this change—what it does

fixes PostgreSQL acceptance test with static_ips to actually check for their existence after service's creation
adds acceptance test coverage for modification of static_ips in Terraform configs (via PostgreSQL)
fixes CustomizeDiffCheckStaticIpDisassociation behavior

Why this way

to make acceptance tests better
to improve user experience

Are we not able to do it in one step by dropping that customize diff check?

no, the API fails

Are we not able to do it in one step by dropping that customize diff check?

no, the API fails

upd: I was able to achieve it, it's a bit tricky, but it works really good:
2022/07/08 20:05:36 [DEBUG] Creating an instance of TopicCache ...
=== RUN   TestAccAivenPG_static_ips
=== PAUSE TestAccAivenPG_static_ips
=== CONT  TestAccAivenPG_static_ips
--- PASS: TestAccAivenPG_static_ips (415.09s)
PASS


Fix Windows issues

Fix issues unique to Windows such as adding support for non-POSIX concepts of user ID and file system differences that are not directly supported.

Successfully tested on Linux, macOS, and particularly Windows.  Awaiting confirmation testing and conflict resolution.

Issues #35 and #44 should also be solved by this pull request.

This may also resolve Issue #14, but will need to be tested.


Disable ctrl+w closing the entire app

Hi,
Thanks for making an excellent piece of software, I'm really enjoying both Jupyterlab desktop and the browser app.
I know these are still early days for the Desktop app, so don't take this as a hard critique or anything like that, I just would like to eliminate one of the few frustrations I'm having with the Jupyterlap app.
How do I prevent ctrl+w from closing the entire JupyterLab desktop app?
The expected behavior from any other app is: ctrl + w will close open tabs or documents. In this case, close the currently-selected notebook. While Alt+F4 will close the entire app. Jupyterlab does not conform to this UI norm, so I would like to fix that. What is the best way to go about that?
I checked the "Keyboard shortcuts" in the Settings, but I only saw "Close tab", which was assigned to Alt+w. Nothing about closing the entire app.
On a related note, how do I completely remove a shortcut? For instance, I would like to remove "Ctrl+P" shortcut for printing. But it doesn't seem possible? I can select the keyboard shortcut field, and I could type in another keyboard shortcut, but I can't remove the keyboard shortcut.
Note: Unlike with the browser app, closing the Jupyterlab desktop app accidentally is really annoying. In the browser app, I can just "ctrl+shift+t" to re-open the jupyter app, and have it back up in a few seconds. With the app, because it has to start the server, it takes much longer to re-open the app. So I would like to make it much harder to accidentally close the app.
One more thing, while we are at it: The symbols for Ctrl/Alt/Cmd are misaligned, making them hard to read. Personally, I would prefer to just have "Ctrl", "Alt", "Shift", "Cmd", instead of symbols, as I can never remember which symbol is which (except for CMD). So instead of reading "Alt+w", I'm reading "<some symbol I don't know>+w" - then referring to the symbols list to figure out what the actual shortcut is.

There are multiple issues in this report. Going through them in order:
How can I disable CTRL+W closing the entire app?
Is it possible to change the shortcut for "close notebook" from ALT+W to CTRL+W? What happens when you do this?
How can I disable a shortcut?
The JupyterLab shortcut editor https://github.com/jupyterlab/jupyterlab/issues/7597 doesn't have an option in the UI to disable a shortcut. In addition, the documentation for editing shortcuts in JupyterLab is out of date: https://github.com/jupyterlab/jupyterlab/issues/7570
@marthacryan can add an enhancement in JupyterLab to provide the option to disable shortcut keys in the new settings editor.
Symbols misaligned for shortcut keys
See https://github.com/jupyterlab/jupyterlab/issues/11988 for more on this issue.

Just for clarity, which OS are you running jupyterlab-desktop on?

This is on Ubuntu 22.04... actually Pop_OS 22.04 (but I don't think there's much difference).
% uname -a
Linux hilbert-space 5.17.5-76051705-generic #202204271406~1651504840~22.04~63e51bd SMP PREEMPT Mon May 2 15: x86_64 x86_64 x86_64 GNU/Linux

% lsb_release -a
No LSB modules are available.
Distributor ID:	Pop
Description:	Pop!_OS 22.04 LTS
Release:	22.04
Codename:	jammy

I installed it through the .deb (JupyterLab-Setup-Debian.deb), which came from this link yesterday (May 11, 2022): https://github.com/jupyterlab/jupyterlab-desktop/releases/latest/download/JupyterLab-Setup-Debian.deb
This computer did not previously have JupyterLab-Desktop installed, though my base conda environment (only channel: conda-forge) does have the JupyterLab package installed. When JupyterLab-Desktop first came up, it prompted me for a Python executable corresponding to an environment with a new-enough JupyterLab, and I pointed it at my base conda environment so that I'd have only one environment (which I always keep up to date, and its JupyterLab is easily new-enough).
I think that describes my entire set-up.

+1 on this, I would like to be able to customize / disable this ctrl w behavior, as I am using "a la vim bindings" (ie ctrl w i/j/k/l) to move between my tabs...

For a long time, I have been using one browser (Safari) for JupyterLab and another (Chrome) for everything else on my Mac.  But I have persistent annoyances with Safari, including accidentally closing the JupyterLab window.
So I thought to myself, wouldn't it be a good idea to make an Electron desktop app for JupyterLab?  And lo and behold, I found that it already exists.  Great job!  I love this app.  Except for one thing.
I keep hitting command-W  instead of option-W when I want to close a tab within JupyterLab.  This closes the whole window and everything needs to be restarted from scratch.  This is really annoying, and it is one of the things I was hoping would be an improvement over Safari.  But it's worse than Safari, since the kernels are killed as well as the browser tabs.
Would it be possible to provide an option to disable this keybinding?  I also think the app should ask the user to confirm before doing something so drastic.

@mbektas
Thanks a lot for the update! I've been really looking forward to the fix. Can I ask a really simple question? I'm following https://github.com/jupyterlab/jupyterlab-desktop/pull/606 but I can't find the setting in my JupyterLab-Desktop. I'm using Version 3.6.2-1 (3.6.2-1). See below for screenshots of my "Advanced Settings Editor" under "Settings".

Could you please let me know where to find the setting for this?

Thanks a million for adding this feature -- it works perfectly.

@mbektas
Thanks a lot for the update! I've been really looking forward to the fix. Can I ask a really simple question? I'm following #606 but I can't find the setting in my JupyterLab-Desktop. I'm using Version 3.6.2-1 (3.6.2-1). See below for screenshots of my "Advanced Settings Editor" under "Settings".

I searched for "Bahavior" in settings but couldn't find it either.
Could you please let me know where to find the setting for this?

Never mind. Finally found it!


Update smithy4s-aws-http4s, ... to 0.18.6

About this PR
📦 Updates

com.disneystreaming.smithy4s:smithy4s-aws-http4s
com.disneystreaming.smithy4s:smithy4s-sbt-codegen

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

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.disneystreaming.smithy4s" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.disneystreaming.smithy4s" }
}]

labels: library-update, early-semver-major, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #106.


(packages) Styles to Support Icon Backgrounds

Adds a light background to the package icons so they can be easily seen
while in dark mode.

@designxsteph thanks for getting this added!


Invalid  tag: turtlebot3_description Please Help!

ISSUE TEMPLATE ver. 0.2.0

Which TurtleBot3 you have?

[ ] Burger
[ ] Waffle
[ X] Waffle Pi

Which SBC(Single Board Computer) is working on TurtleBot3?

[ X] Raspberry Pi 3
[ ] Intel Joule 570x
[ ] etc (PLEASE, WRITE DOWN YOUR SBC HERE)

Which OS you installed in SBC?

[ ] Ubuntu MATE 16.04 or later
[ X] Raspbian
[ ] etc (PLEASE, WRITE DOWN YOUR OS)

Which OS you installed in Remote PC?

[ X] Ubuntu 16.04 LTS (Xenial Xerus)
[ ] Ubuntu 18.04 LTS (Bionic Beaver)
[ ] Linux Mint 18.x
[ ] etc (PLEASE, WRITE DOWN YOUR OS)

Write down software version and firmware version

Software version: [16.04.5]
Firmware version: [1.10]

Write down the commands you used in order
roslaunch turtlebot3_bringup turtlebot3_remote.launch

Copy and Paste your error message on terminal

while processing /home/fish/catkin_ws/src/turtlebot3/turtlebot3_bringup/launch/includes/description.launch.xml:
Invalid  tag: turtlebot3_description
ROS path [0]=/opt/ros/kinetic/share/ros
ROS path [1]=/home/fish/catkin_ws/src
ROS path [2]=/opt/ros/kinetic/share.
Arg xml is
The traceback for the exception was written to the log file

Please, describe detailedly what difficulty you are in
I'm trying to load TurtleBot3 on RViz, but I have this error all the time. I've tried to solve it by myself but I could not figure it out what I'm doing wrong.
I'm mentioning the TURTLEBOT3_MODEL=waffle_pi

Please help!
Link: http://emanual.robotis.com/docs/en/platform/turtlebot3/bringup/#load-a-turtlebot3-on-rviz

I managed to solve it. I've reinstalled the turtlebot3 packages on the PC and delete the turtlebot3 folder


Pre-authorize service level methods on org membership

Currently many service-level methods make an in-line call to getCurrentOrganization() and then do custom validation to make sure that the caller's current org matches the org of the entities they are trying to access/edit. All of these methods should instead have pre-authorization on the org ID in question, where the pre-authorization checks the caller's membership in that org.

I'm not convinced this is the right direction to take our security enforcement.

Okta


fix: breadcrumb & show code button collision

Description
Fix breadcrumb & show code button collision

Task Item
close #7705
close #7762

Screenshots

@hatpick @srinaath To implement above behavior in fluentui, seems the only way is to use javascript  calculate the size of breadcrumb element, do you think it make sense, or have better suggestions?

@zhixzhan, So the example with no maxDisplayedItems we should be using that option. Essentially, the way u would do something like this, pseudo coding here

          <Breadcrumb ariaLabel="Breadcrumb with no maxDisplayedItems"
           overflowAriaLabel="More links" />

         Show Code

So, the breadcrumb starts shrinking and always occupies only a particular width in percentage. Does that work?

@mewa1024 I agree with @zhixzhan, if we simply say always truncate the middle items and keep first and last item in the view, we can still have the following issue based on the size of the window and length of texts:
FIRST ITEM > ... > LAS [BUTTON]
FIRST ITEM > .. [BUTTON]
FIR [BUTTON]
etc.
Even if we do calculations as @zhixzhan suggested, these situations can happen.
My suggestion is doing horizontal scrolling + Breadcrumb capabilities of moving items in overflow menu, so in case any of the above happens, user can still scroll horizontally to find breadcrumb first and last item, as well as the overflow menu in between.
Thoughts?

I like @srinaath's suggestion.

@srinaath @hatpick @mewa1024 PR updated, according to the codepen example, thanks for your help.


Deploy hooks API docs

Existing documentation URL(s)
https://developers.cloudflare.com/api/operations/pages-deployment-get-deployments
https://developers.cloudflare.com/pages/configuration/deploy-hooks/
What changes are you suggesting?
Hello
Just wanted to ask if the is any reason why Deploy hooks are not documented in Cloudflare API docs and thus there is no cloudflare-go and cloudflare terraform provider support? I mean I'm considering contributing to the library (which version should I start with main or v2?) and then integrate it to terraform provider, but wondering why it's not even documented. Shouldn't be considered as stable so contributions are not good idea?
The endpoint I meant
https://dash.cloudflare.com/api/v4/accounts/<ACCOUNT_ID>/pages/projects/<PROJECT_NAME>/deploy_hooks
Thanks in advance
Additional information
No response

Thanks for the feedback @pawelhenek! I've raised this internally and we're working on it.

CC @crwaters16

Hey @pawelhenek, thanks for raising this - we've let the Pages team know this endpoint is missing and are tracking the work to address internally. They'll follow up once this has been handled!


Page becomes unresponsive

Challenge http://www.freecodecamp.com/challenges/waypoint-bring-your-javascript-slot-machine-to-life has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
When trying to write out the following code, the browser becomes unresponsive.
$($(".slot")[0]).html(slotOne);
It seems natural to start typing out $($. When the open paren is typed, the editor auto-completes the closing paren $() then when typing the second $, the browser stops responding: $($)

Json

I have the same issue I think, but my page freeze when I type "function" on another challenge

Duplicate of #1960


Contact Page - New positions

Please Add 4 More Positions While Connecting Contact with Company
Group Chairmen
Group CFO
Group CS
Group HR
By client
https://community.easycloudbooks.com/t/Questions or Bugs/78fa26ea-9655-4263-a2c4-ef9ef008ad12

https://www.berrycast.com/conversations/316ebf74-01f6-5dcb-9e1a-5dd7a0b9c07c


Questions regarding adversarial trainer

Hello!
I've been trying to adversarially train some medical DL classifiers lately and have been running into some questions.

when do you recommend using AdversarialTrainer vs MadryPGD? How should we determine how many epochs to train for? I've been getting poor adversarial training results on some of my classifiers and good ones on others.
For adversarial training, your code examples all showed loading a new uninitialized model. Can I load in an already trained model for adversarial training?
I've been trying to replicate https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/examples/adversarial_training_data_augmentation.py but with replacing the model with my own vgg16 model architecture. After training, I got poor training and testing accuracy for the new classifier (~10%). I've included my model architecture below.

Thank you for your help!
def build_model(input_shape):
vgg_base = VGG16(
include_top=False,
#weights=None,
weights='imagenet',
input_tensor=Input(shape=input_shape),
input_shape=input_shape,
pooling=None
)
x = vgg_base.output
dropout1 = tf.keras.layers.Dropout(0.5)
dropout2 = tf.keras.layers.Dropout(0.5)
x = dropout1(x)
x = Flatten(name='flatten')(x)
x = Dense(4096, activation='relu', name='fc1',kernel_constraint=max_norm(2), trainable = False
)(x)
x = dropout2(x)
x = Dense(1024, activation='relu', name='fc2', kernel_constraint=max_norm(2), trainable = False
)(x)
predictions = Dense(10, activation='softmax', name='predictions',trainable = False)(x)
model = Model(inputs=vgg_base.input, outputs=predictions)
opt = SGD(lr=0.002)
model.compile(loss="categorical_crossentropy", optimizer=opt, metrics=["accuracy"])
return model

Hi @joelma1 I think these are all good questions and I hope my colleagues will join the discussion.
1.) It depends on what you'd like to do. MadryPGD is probably the most popular standard approach with many evolutions in the literature, AdversarialTrainer is similar but focuses on using one or more attacks (that can but don't have to be PGD) for generating the adversarial training examples. Not all models and data combinations are straightforward, some approaches for challenging cases of adversarial training include for example increasing the fraction of adversarial training examples on a schedule over the training epochs, there are many more adversarial training recipes in the literature.
2.) I think the codes/tools should allow you to load and use a trained model for adversarial training. You could try the fraction of adversarial training example schedule mentioned in 1.) to see if you can save training epochs that way.
3.) The linked example is for the CIFAR10 10-class classification task. I've noticed that your model is a binary classifier (loss='binary_crossentropy). Did you adapt the dataset for binary classification?

@beat-buesser
Thank you for the advice!
3) I accidentally pasted the wrong model. I have updated my post with the correct model I used.

@beat-buesser
I ran the code in https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/examples/adversarial_training_data_augmentation.py with no modifications.
I got these results:
Accuracy test set: 10.00%
Accuracy on original PGD adversarial samples: 10.00%
Accuracy on new PGD adversarial samples: 10.00%
I'm using tensorflow 1.15.3 and python 2.7. Please advise thank you!

@joelma1
The 10% accuracies mean your model doesn't learn anything.
I've noticed that you are loading Imagenet weights (weights='imagenet',), does the same also happen for random initialisation?
It looks like you are adding additional layers on top, I guess for transfer learning, but are freezing them (trainable = False). Are you loading pretrained weights into these layers? I think these layers will remain unchanged (with random weights) during the training later on in the example which might explain the 10%.

@beat-buesser
For those results, I used the model in https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/main/examples/adversarial_training_data_augmentation.py so there were no imagenet weights loaded (I ran the code exactly as is and got 10% acc on everything)
Thanks for letting me know! I have changed the layers to be trainable

@joelma1 Interesting, I somehow thought it was with your model. Let me try to run the original example.

@beat-buesser great thank you!

Hi @joelma1 Sorry for the long delay. I think I have finally identified the reason for the unexpected results in this example. The attack used to generate the adversarial examples define in line 67 https://github.com/Trusted-AI/adversarial-robustness-toolbox/blob/3776d66ae3139afc280c7847e5600f115160659f/examples/adversarial_training_data_augmentation.py#L67
is defined with eps and eps_step in the range [0, 255], but the images in x_train and x_test are in range [0, 1]. Updating the attack in line 67 to
pgd = ProjectedGradientDescent(classifier, eps=8/255, eps_step=2/255, max_iter=3, num_random_init=1)

makes the example work as expected. I'll update the example accordingly.

Fixed with 7cc6fe6171b92475b16f35a1d70fc429e49b12f2


rasterize.js: gs encountered an error

I get many error:
rasterize.js: gs encountered an error:
GPL Ghostscript 9.18: Some glyphs of the font CAAAAA+LiberationSerif-Bold requires a patented True Type interpreter.
How to fix ?

The problem is probably what the error message says and thus a problem of ghostscript with the PDF you are using.  Nothing much scissors (which is a wrapper around PDFTK and ghostscript) can do, I am afraid.


Update Code and Fixed Some Warning ⚠

Overview

Fixed redundant code.
Fixed Possible System.InvalidOperationException warning.
Change some public method to private because it's not being use in the other class and protected as well.
Remove unused variables.
Remove virtual method that hasn't been override.
Fixed Possible System.NullReferenceException warning.

Changes

Fixes: Warning Code
Fix Possible System.NullReferenceException warning.
Fix Possible System.InvalidOperationException warning
Fix redundant code

/azp run

Why do the pipelines make some errors?

@BillyFrcs , neither Unity 2018 nor Unity 2019 support switch pattern matching.
https://dev.azure.com/aipmr/MixedRealityToolkit-Unity-CI/_build/results?buildId=20950&view=logs&j=9a40d849-de61-58a5-b9fb-0035311f3ef5&t=40a1ca36-8c03-582a-2c45-de22c125c2c7&l=11115

Alright understood, Thanks for the correction @timGerken I've already revert it.

/azp run

/azp run

Hey @BillyFrcs, I really appreciate you taking the time to address our feedback. We need all of the public fields that were changed to private/protected be reverted back to public, since they are breaking changes to our api.
We'd also like you to retarget this branch to prerelease/2.8.0 to get this in if you have the time.

Hey @BillyFrcs, I really appreciate you taking the time to address our feedback. We need all of the public fields that were changed to private/protected be reverted back to public, since they are breaking changes to our api.
We'd also like you to retarget this branch to prerelease/2.8.0 to get this in if you have the time.

Okay, I'll try to take it back.

Sorry for the delay on accepting this PR. Left some last comments, after which I think this PR should be good to go!

Sorry for the delay on accepting this PR. Left some last comments, after which I think this PR should be good to go!

Alright, no problem!

/azp run


Task HTM

This PR adds a task on H-transport-materials

In order for the tests to pass, we need to be able to install scipy==1.8 in the fenics docker container. See this issue

I switched to a conda install but another issue appears (see https://github.com/conda-forge/fenics-feedstock/issues/164)


Fixed #30004 -- Set default FILE_UPLOAD_PERMISSION to 0o644

Updated FILE_UPLOAD_PERMISSION settings default value
to 0o644 to avoid inconsitency in file permissions.
Updated test case to test against new default value and
added documentation regarding this breaking change.

Hi @claudep. Can I ask for your opinion on this change please? For me it solves the issue, and it's nice and straight-forward, but I wanted to check you were in agreement. Thanks.

@Himanshu-Lakhara Did you see Tim's comment on the ticket, re adjusting the checklist discussion? Do you fancy having a look at that?

Hi @carltongibson I've shared my thoughts on ticket regarding checklist discussion.

Thank you everyone for helping me land my first PR to Django.

@Himanshu-Lakhara,  Could you mention the Ticket Link in the first comment for SWE bench users?


Could not restart bluealsa

Hello,
I am getting sound stopped sometimes and could not get it restart.
I am trying to kill bluealsa, then cannot start it back:
bluealsa: Couldn't initialize controller thread: Address already in use
There is no bluealsa process running. Can anyone help?

Try to stop and then start the hciuart service. Also make sure permissions are correct on /var/run/bluealsa dir and files.

I am trying to kill bluealsa, then cannot start it back

If bluealsa hangs (due to bug), it is not possible to kill it without repercussions :) After pkill bluealsa make sure that there is no "files" in the /var/run/bluealsa/ directory (if there is any, remove it).
Also this "I  am getting sound stopped sometimes and could not get it restart" might require a little bit of debugging, because it seems like a nasty bug :/ .

Removing files under  /var/run/bluealsa helps alot. How would I help with debugging?

Now, the /var/run/bluealsa directory is gone. BlueALSA uses D-Bus for communication.


updated copy command to template command

Updated copy command to template command so that google-cloud-ops-agent config files can be dynamically generated.
For eg: using GCE instance hostname in the ops agent receiver configuration

Would you mind signing the CLA?


Add armcore module for ARM-specific content

This adds the armcore module with the automatic RP registration policy
which is specific to ARM.

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] Apache v2 license headers are included in each file.

Fixes https://github.com/Azure/azure-sdk-for-go/issues/9038

Can one of the admins verify this patch?


Medical Prescription System

Describe the project you want to add with tech stack
In the Medical Prescription System, patients can receive a prescription by entering their basic information and symptoms. The system is designed in Java. The patient provides their first name, last name, age, gender, and symptoms and get the system then generates prescription using a predefined list of medicines and directions.
Expected behavior
Its is very useful for patient get immediately prescription.
Screenshots (optional)
I have some attached the Screenshot of it:-

@MohitGupta121 Sir Kindly assigned me this issue

@MohitGupta121 @TusharKesarwani @pranjay-poddar Sir kindly assigned the project


Added spelled-out aliases for abbreviated predicate cases.

I've been waffling back and forth, but I agree with you. Should we bother with deprecation warnings or just rip the band-aid off and make an @channel on the Slack?

I think a single point-release containing a set of migration aliases will be fine.  I doubt anybody is using fcmp heavily enough at this point that it will be a problem.

Will merge once CI passes

🚢 :it:


Yank line and paste not working with osc52

Thank you for making a great plugin! It's so great to be able to copy text even over an ssh session!
However, after setting up the plugin, I have noticed that my usual workflow of copying and pasting lines (yy and p) no longer works. I get the error:
E353: Nothing in register "

I have set the following in my config:
vim.opt.clipboard = "unnamedplus"

local status_ok, osc52 = pcall(require, "osc52")
if not status_ok then
    return
end

osc52.setup({
    max_length = 0, -- Maximum length of selection (0 for no limit)
    silent = false, -- Disable message on successful copy
    trim = false, -- Trim text before copy
})

-- set this plugin as the clipboard provider
local function copy(lines, _)
    require('osc52').copy(table.concat(lines, '\n'))
end

local function paste()
    return {vim.fn.split(vim.fn.getreg(''), '\n'), vim.fn.getregtype('')}
end

vim.g.clipboard = {
    name = 'osc52',
    copy = {['+'] = copy, ['*'] = copy },
    paste = {['+'] = paste, ['*'] = paste },
}

Is there anything I can do to re-enable my copy-line-and-paste workflow? I am new to Neovim so apologies if I am missing something obvious.

Something I just noticed: yy does copy because I see the message from osc52 when I do it. It's just the paste with p that doesn't work.

When you set vim.opt.clipboard = "unnamedplus", you instruct Neovim to put all your yanked text into the + register, which in turn calls the copy() function with the given text. But Neovim still puts the text into your unnamed register " as well. So p should work fine. Did you remap y or yy in your config?
Also note that if you don't change any of the default options, you don't need to call setup().

I've updated the README to address your question, but you can re-open this issue if you still have problems.

I was actually referring to text copied from inside neovim and pasting inside neovim. I already use Kitty's keybinding to paste from external applications. Thanks for suggesting that there may be a conflicting remap ... I removed whichkey from my plugin list and yy+p works again! So, whichkey must be remapping it somehow.

I think this issue may be related: https://github.com/folke/which-key.nvim/issues/202

Good to know! Glad that your problem is resolved.


ncurses: include release number in version

Description of changes
Included the release number in the version.
I'm putting it into staging because it might rebuild 500+ packages.

ZHF: #309482

Fix the following Hydra failures (nixpkgs.tests.pkg-config.defaultPkgConfigPackages.[name]):

check-pkg-form: 258531998, 258583946, 258533184, 258579848
check-pkg-formw: 258532352, 258586816, 258532178, 258583614
check-pkg-menu: 258532451, 258577307, 258532460, 258580353
check-pkg-menuw: 258533067, 258577737, 258533385, 258586402
check-pkg-ncurses++: 258532586, 258578414, 258533428, 258577901
check-pkg-ncurses++w: 258532475, 258581669, 258533149, 258586927
check-pkg-ncurses: 258532386, 258582876, 258533151, 258576435
check-pkg-ncursesw: 258532416, 258586091, 258532121, 258583304
check-pkg-config-panel: 258532669, 258578618, 258532963, 258576839
check-pkg-config-panelw: 258532410, 258582510, 258532663, 258582194

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[X] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build nixpkgs.tests.pkg-config.defaultPkgConfigPackages.{check-pkg-form,check-pkg-formw,check-pkg-menu,check-pkg-menuw,check-pkg-ncurses++,check-pkg-ncurses++w,check-pkg-ncurses,check-pkg-ncursesw,check-pkg-config-panel,check-pkg-config-panelw}

@ofborg build ncurses.passthru.tests.pkg-config

i've set the wrong badge, whoops

I'd prefer not to merge this one before branch-off, as it has a treewide rebuild while only fixing "nixpkgs.tests" instead of packages.


UTXO Consolidation

Description
The wallet library should expose a function to consolidate funds spread across addresses and UTXOs into a single address/UTXO.
Motivation

Ledger Nano has max 17 inputs/outputs in a single message
All messages have an upper limit of 127 inputs/outputs

Requirements

Are you planning to do it yourself in a pull request?
No.

@Thoralf-M can you also implement the max 17 inputs when the account signer type is Ledger?

Closed by https://github.com/iotaledger/wallet.rs/pull/483


Update existing draft permit if it exists

Description
Always update existing draft permit if it exists for the given vehicle.
Context
PV-895
How Has This Been Tested?
Manually.

LGTM!


Build and run the game from the root of the source code

Inspired by Minetest

Whoa we're finally getting this PagChomp

Invalidated by #1114


fix typo in efuse.rs

I fixed typo that I found in efuse.rs

Thanks for the PR.
Since we frequently regenerate the source files, we use patching to make modifications rather than editing source files directly. See the Patching the SVDs section of the README for more details.

I didn't see them, sorry.
So, What I should do is to add the patch in https://github.com/esp-rs/esp-pacs/blob/main/esp32c3/svd/patches/esp32c3.yaml, according to svdtools format, right?

I didn't see them, sorry.

Not a problem, now you know! 😁

So, What I should do is to add the patch in https://github.com/esp-rs/esp-pacs/blob/main/esp32c3/svd/patches/esp32c3.yaml, according to svdtools format, right?

Yes that is correct! The patching format specification should contain all the information you need, I hope.

Please check it! is this ok?

Your patches look good, but that SVD should not have been included. Not sure what's going on there, the .gitignore should have covered that 😅 Also looks like not all re-generated sources files are included.
If you remove the SVD which you had previously committed, and then from within the xtask/ directory run the command cargo run -- generate esp32c3 you should be in good shape, I think. Doing this locally resulted in the following modified files, all of which should be included in this PR along with the YAML patches:

esp32c3/src/efuse.rs
esp32c3/src/efuse/clk.rs
esp32c3/src/efuse/conf.rs
esp32c3/src/efuse/wr_tim_conf1.rs
esp32c3/src/efuse/wr_tim_conf2.rs
esp32c3/src/system.rs
esp32c3/src/system/edma_ctrl.rs

I'm sorry that this process was not more clear. In the meantime I have updated the repository's README which I hope makes things more clear now. If you have any feedback on that I'd love to hear it.

I’m sorry but I misunderstood that I should use svdtools directly when I generate svd patch file. This is the reason that my commit contained svd file because svdtools generate *.svd.base.patched, which is not covered in .gitignore.
Now the process is also so clear, Thank you so much!
Anyway, I added all my files!


Paragraph about changes in Coex pix version 1.2

The quality of the image is really poor. What the original looks like?

I completely agree, but they said that while we are posting such

Waiting for new schemes with better quality


Update build.gradle

This PR adds safeExtGet function to use sdk versions from the root project
Resolves: Issue 77
If the root project uses a minSdkVersion which is higher than 16, building the root project which uses react native (com.facebook.react:react-native:0.65.1) fails
The error message:
> Task :react-native-community_progress-bar-android:processDebugAndroidTestManifest FAILED
[androidx.vectordrawable:vectordrawable-animated:1.0.0] /Users/UgurGumushan/.gradle/caches/transforms-3/f66d8f3946e5dc0709e169eabc5d4aea/transformed/vectordrawable-animated-1.0.0/AndroidManifest.xml Warning:
        Package name 'androidx.vectordrawable' used in: androidx.vectordrawable:vectordrawable-animated:1.0.0, androidx.vectordrawable:vectordrawable:1.0.1.
/Users/UgurGumushan/Code/kontist/app/node_modules/@react-native-community/progress-bar-android/android/build/intermediates/tmp/manifest/androidTest/debug/manifestMerger8441565954767347173.xml:5:5-74 Error:
        uses-sdk:minSdkVersion 16 cannot be smaller than version 21 declared in library [com.facebook.react:react-native:0.65.1] /Users/UgurGumushan/.gradle/caches/transforms-3/154779808e92101f84642acdf5bc7bdf/transformed/jetified-react-native-0.65.1/AndroidManifest.xml as the library might be using APIs not available in 16
        Suggestion: use a compatible library with a minSdk of at most 16,
                or increase this project's minSdk version to at least 21,
                or use tools:overrideLibrary="com.facebook.react" to force usage (may lead to runtime failures)

@StebanDev thanks for the reminder! just published update with this fix as v1.0.5 :tada:


Rdm 7650 excel json

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Could we also translate the other Excel files into json in the repository, just like it's done for 3 jurisdictions? Also, the excel files should be either removed or put under a folder old-excels-never-edit. Otherwise people may accidentally edit them or try to keep them in synch with the edited json files, which is not good.


replace license with file instead of link

Replace the license with the license file instead of a link

Fixed by #1


add markdown linting

Lets do markdown with standards.
Thoughts on what should be included to describe what they are in the README.md ?

Superseded by #38


[RFE] Allow import of EKS Cluster without having to provide AWS Access Key and Secret Key

Is your feature request related to a problem? Please describe.
Yes.  In our AWS GovCloud Account, they do not allow AWS Keys to be used.  We must use either MFA, IAM instance roles, or use Service Roles, which  is able to assume a role.  This will not allow me to import an EKS.  The EKS being imported is running from a Rancher on another EKS with the node instance role that has the rights to manage both cluster.
Describe the solution you'd like
A clear and concise description of what you want to happen.
The better way is to not make AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY required parameter to import the cluster and allow it to use the native VM instance role that Rancher is running under.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
If we provide any AWS Access Key and Secret Key it will override any node instance role native on the VM.  Some how if we can get into the imported cluster and remove the AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY and let it use the node instance role we should be fine.
Additional context
Add any other context or screenshots about the feature request here.
My understanding is that Rancher is big in the DoD, and Intel community, but I dont see how if it's not able to support this lock down requirements for AWS in the Gov Cloud.

Also trying to figure out if this could be done with Rancher CLI as well.  Appears so, but have not been able to get it to work yet.

Figured it out.
Best way to import EKS cluster and not require AWS keys is to import it as a generic Kubernetes.  Make sure the following properties are set in your helm chart which is how I solved the problem
My list of issues.
In settings the following was wrong

Cacerts was blank.  It was not ingesting my CA certs cause I had privateCA: false in stead of true
server-url did not have the https://. It only had the host name.  Make sure you Put in the entire URL
I did not create the tls-ca-additional.  Created that and set additionalTrustedCAs: true

got it all working.  Better documentation on how to import cluster with screen shots would have been very helpful.


Release page for 21.3.1 has dbeaver-ce-21.3.0-macos-aarch64.dmg as first entry

System information:

Operating system (distribution) and version: macos 11.6.1, Intel + M1
DBeaver version: 21.3.1
Additional extensions: aarch64

Connection specification:

Database name and version: n/a
Driver name: n/a
Do you use tunnels or proxies (SSH, SOCKS, etc)? n/a

Describe the problem you're observing:
The release page for 21.3.1 (https://github.com/dbeaver/dbeaver/releases/tag/21.3.1) has dbeaver-ce-21.3.0-macos-aarch64.dmg as first entry, and this misguides our autopkg recipe for DBeaverCE for arm64; it will not 'see' the correct dbeaver-ce-21.3.1-macos-aarch64.dmg download. In the grand scheme of things not important, but on the other hand I assume it can also be easily fixed.
Steps to reproduce, if exist: n/a
Include any warning/errors/backtraces from the logs
n/a

Hello @gerardkok,
Thanks for the report.
We've removed the dbeaver-ce-21.3.0-macos-aarch64.dmg entry.
It should work now.

Thanks for the quick fix! Our autopkg recipe works again now.


feat: non revertible effects and tx phases

Please read contributing guidelines and remove this line.

Benchmark results
Metrics with a significant change:

tx_sequencer_processing_time_ms (0): 0.0287 (-94%)
note_history_trial_decrypting_time_in_ms (5): 163 (+86%)
note_successful_decrypting_time_in_ms (8): 321 (-26%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit e63f0226 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,956
181,604
724,196

l1_rollup_calldata_gas
227,144
888,860
3,536,192

l1_rollup_execution_gas
320,335
1,007,308
3,756,570

l2_block_processing_time_in_ms
1,206
4,527 (+1%)
17,889 (+1%)

note_successful_decrypting_time_in_ms
:warning: 321 (-26%)
1,003 (+3%)
3,794 (+5%)

note_trial_decrypting_time_in_ms
31.6 (-25%)
39.3 (-63%)
187 (+6%)

l2_block_building_time_in_ms
18,840 (+3%)
74,779 (+4%)
298,672 (+3%)

l2_block_rollup_simulation_time_in_ms
13,906 (+3%)
54,860 (+3%)
220,569 (+3%)

l2_block_public_tx_process_time_in_ms
4,908 (+5%)
19,612 (+5%)
77,862 (+5%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,215 (-5%)
29,016 (-5%)

note_history_successful_decrypting_time_in_ms
2,326 (+1%)
4,778 (+3%)

note_history_trial_decrypting_time_in_ms
:warning: 163 (+86%)
168 (-32%)

node_database_size_in_bytes
17,309,776
33,112,144

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
330 (+1%)
44,568
25,313

private-kernel-ordering
213 (+9%)
43,633
14,531 (-10%)

base-rollup
1,489 (+2%)
127,600 (-2%)
933

root-rollup
82.6
4,192
729

private-kernel-inner
431 (-1%)
70,859
25,313

public-kernel-app-logic
250
31,550
24,771

merge-rollup
7.84
2,712
933

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,197 (-10%)
33,874 (-5%)

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
568 (+3%)
1,468 (+2%)

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
:warning: 0.0287 (-94%)
600 (+4%)


Add function worker client auth config in cluster

Motivation
Similar to #4473, when it is setting to use kerberos authn, pulsar will failed to start because of function worker authentication fail.
Modifications
Add authn related configs, so user could config the client in worker, and make it work.

run java8 tests

run java8 tests


Update README.md

We created a logo to help the clarify the difference between js-ipfs and js-ipfs-api. Would you like to add to your client library as well? You can see how it looks here: https://github.com/ipfs/js-ipfs-api
This design is from @akrych 🌟

Thanks @diasdavid
I've been very neglectful of this project, but this logo will be helpful anyhow. :peace_symbol:


Application Authentication instead of an user

Please add the functionality to authenticate with the application itself and not delegated permissions.
This will enable authentication via certificate instead of username/password.

This would be really usefull

There is a much simpler thin. I will publish it in a few days. You can use the new Microsoft.Graph module which is much better and supports everything. I only continue to use the module here to encrypt the pfx password, so probably the module could just be reduced to create the correct object.
  $UserPfx = New-IntuneUserPfxCertificate -PathToPfxFile "$($env:ProgramData)\baseVISION-SMIME\temp\cert.pfx" -PfxPassword  $Password -ProviderName "Microsoft Software Key Storage Provider" -KeyName "SMIME" -IntendedPurpose SmimeSigning -UPN $Upn

      # creating hashtable as an object is not working
      $UserPfxBody = @{
          CreatedDateTime = $UserPfx.CreatedDateTime
          EncryptedPfxBlob  = $UserPfx.EncryptedPfxBlob
          EncryptedPfxPassword = $UserPfx.EncryptedPfxPassword
          ExpirationDateTime = $UserPfx.ExpirationDateTime.DateTime
          IntendedPurpose = $UserPfx.IntendedPurpose
          KeyName = $UserPfx.KeyName
          LastModifiedDateTime = $UserPfx.LastModifiedDateTime.DateTime
          PaddingScheme = $UserPfx.PaddingScheme
          ProviderName = $UserPfx.ProviderName
          StartDateTime = $UserPfx.StartDateTime.DateTime
          Thumbprint = $UserPfx.Thumbprint
          UserPrincipalName = $UserPfx.UserPrincipalName
      }
      Write-Log -Message "Start uploading cert to Intune" -Type Debug
      New-MgDeviceManagementUserPfxCertificate -BodyParameter $UserPfxBody


Update README version badge to point to releases

Fixes #61
Problem
See #61
Solution
Renamed tags/tag to releases/release (as per shields.io documentation)
Notes

[x] Remember to bump the VERSION file using semver 📝
[x] Make sure the CI tests pass on your branch 👍

@jeroenptrs Thought it would be easier for you to bump the version.

@revett done!


does not support whitespace control hyphens

consider a file consisting of only:
{%- set z = true -%}

It is a valid expression in Jinja but currently jinjalint will swear about it:
index.html:1:2: Parse error: expected one of 'a letter', 'autoescape', 'block', 'blocktrans', 'comment', 'filter', 'for', 'if', 'ifchanged', 'ifequal', 'ifnotequal', 'spaceless', 'verbatim', 'with' at 0:2

I wonder if it's an issue or something's wrong with my install of jinjalint..

This is an issue with jinjalint, currently I have tested it with Django's flavor of Jinja which doesn't support some Jinja features like this one. However this one is really easy to implement so I'll add support for it soon.

thank you, also, let's consider {{- at the same go, so both {%- and {{- would work

@revelt I just published fa60351f59e6eee82f52c3ff1a894f140768e781 in a new version on Pypi so these whitespace control hyphens and pluses should be handled properly.

I’ll close the issue, but please let me know here if anything is going wrong.

Good work! I just tried linting, it does pass, there's only a bunch of indentation errors (which I'll find out how to disable) but otherwise healthy files pass fine now. Thank you very much!

I’m glad to hear that :slightly_smiling_face:


Add INA

What type of PR is this? (check all applicable)

[ ] ✨ Feature
[ ] 🐞 Bug Correction
[ ] 📝 Documentation Update
[ ] 🚩 Other

Changes:

Testing

Depois tiro o .txt


Cannot set masquerade

Is it possible to set masquerade in zone using this module, I've not found any informatiom about that.
Example:
/usr/bin/firewall-cmd --permanent --zone=public --add-masquerade
As a workaround I'm doing this:
exec { 'firewalld_add_masquerade_public':
command   => '/usr/bin/firewall-cmd --permanent --zone=public --add-masquerade',
unless    => '/usr/bin/grep masquerade /etc/firewalld/zones/public.xml > /dev/null',
}
I need this to build a NAT Gateway.
I'm using CentOS 7.3

Sorry, there was something missing in my workaround, this is the correct version.
exec { 'firewalld_add_masquerade_public':
command   => '/usr/bin/firewall-cmd --permanent --zone=public --add-masquerade',
unless    => '/usr/bin/grep masquerade /etc/firewalld/zones/public.xml > /dev/null',
notify    => Exec['firewalld::reload']
}

This isn't currently supported - it's a fairly easy addition to the firewalld_zone resource type, will get this added


Simd sort

Enabling vectorization for sorting using 3rd party library (https://github.com/intel/x86-simd-sort)
(Work in Progress)

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)??-InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002023-08-19T13:19:41+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b2cd65a806000094b5f31cd3a6e9752009fb33a735ClusternamerostamrostamEnvfileDatetime2023-05-10T14:50:18.616050-05:002023-08-19T08:25:36.834748-05:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch=InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002023-08-19T13:19:41+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b2cd65a806000094b5f31cd3a6e9752009fb33a735ClusternamerostamrostamEnvfileDatetime2023-05-10T14:52:35.047119-05:002023-08-19T08:27:50.442351-05:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)(=)=Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002023-08-19T13:19:41+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b2cd65a806000094b5f31cd3a6e9752009fb33a735ClusternamerostamrostamEnvfileDatetime2023-05-10T14:52:52.237641-05:002023-08-19T08:28:07.336369-05:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)??-InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002023-08-19T18:38:27+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b224dfcd5c69aa95b68b4be8b2c3de23cea6aa6d99EnvfileCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ClusternamerostamrostamDatetime2023-05-10T14:50:18.616050-05:002023-08-19T13:45:18.850938-05:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch(=)InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002023-08-19T18:38:27+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b224dfcd5c69aa95b68b4be8b2c3de23cea6aa6d99EnvfileCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ClusternamerostamrostamDatetime2023-05-10T14:52:35.047119-05:002023-08-19T13:47:31.102556-05:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)(=)(=)InfoPropertyBeforeAfterHPX Datetime2023-05-10T12:07:53+00:002023-08-19T18:38:27+00:00HPX Commitdcb541576898d370113946ba15fb58c20c8325b224dfcd5c69aa95b68b4be8b2c3de23cea6aa6d99EnvfileCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ClusternamerostamrostamDatetime2023-05-10T14:52:52.237641-05:002023-08-19T13:47:48.033631-05:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduExplanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)??(=)InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b2b60c175dcead0a8576859035baf24bb81b4821b4HPX Datetime2023-05-10T12:07:53+00:002023-08-29T06:23:47+00:00ClusternamerostamrostamDatetime2023-05-10T14:50:18.616050-05:002023-08-29T01:31:05.004560-05:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduEnvfileCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch(=)InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b2b60c175dcead0a8576859035baf24bb81b4821b4HPX Datetime2023-05-10T12:07:53+00:002023-08-29T06:23:47+00:00ClusternamerostamrostamDatetime2023-05-10T14:52:35.047119-05:002023-08-29T01:33:17.791388-05:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduEnvfileCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale=(=)=Stream Benchmark - Triad(=)=(=)Stream Benchmark - Copy(=)-(=)InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b2b60c175dcead0a8576859035baf24bb81b4821b4HPX Datetime2023-05-10T12:07:53+00:002023-08-29T06:23:47+00:00ClusternamerostamrostamDatetime2023-05-10T14:52:52.237641-05:002023-08-29T01:33:34.692225-05:00Hostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduEnvfileCompiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1Explanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…

Speedup observed for simd-sort

@hkaiser There are multiple issues trying to add this integrate this feature into HPX

We need to add feature tests for AVX512
Some vector intrinsic functions are not supported in certain architectures, these seem to be throwing errors. In case of medusa a certain vector load instruction related to 16bit integers was not supported which leads to compile time failure even if i16 is not being sorted

Performance test reportHPX PerformanceComparisonBENCHMARKFORK_JOIN_EXECUTORPARALLEL_EXECUTORSCHEDULER_EXECUTORFor Each(=)??(=)InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b225b4e45923f079fe7367d5855b20ba1ac6d45f5eHPX Datetime2023-05-10T12:07:53+00:002023-08-29T21:58:26+00:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduEnvfileDatetime2023-05-10T14:50:18.616050-05:002023-08-29T17:07:47.004681-05:00ComparisonBENCHMARKNO-EXECUTORFuture Overhead - Create Thread Hierarchical - Latch(=)InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b225b4e45923f079fe7367d5855b20ba1ac6d45f5eHPX Datetime2023-05-10T12:07:53+00:002023-08-29T21:58:26+00:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduEnvfileDatetime2023-05-10T14:52:35.047119-05:002023-08-29T17:09:59.091186-05:00ComparisonBENCHMARKFORK_JOIN_EXECUTOR_DEFAULT_FORK_JOIN_POLICY_ALLOCATORPARALLEL_EXECUTOR_DEFAULT_PARALLEL_POLICY_ALLOCATORSCHEDULER_EXECUTOR_DEFAULT_SCHEDULER_EXECUTOR_ALLOCATORStream Benchmark - Add(=)(=)(=)Stream Benchmark - Scale(=)(=)(=)Stream Benchmark - Triad(=)(=)(=)Stream Benchmark - Copy(=)-(=)InfoPropertyBeforeAfterHPX Commitdcb541576898d370113946ba15fb58c20c8325b225b4e45923f079fe7367d5855b20ba1ac6d45f5eHPX Datetime2023-05-10T12:07:53+00:002023-08-29T21:58:26+00:00Compiler/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1/opt/apps/llvm/13.0.1/bin/clang++ 13.0.1ClusternamerostamrostamHostnamemedusa08.rostam.cct.lsu.edumedusa08.rostam.cct.lsu.eduEnvfileDatetime2023-05-10T14:52:52.237641-05:002023-08-29T17:10:15.975599-05:00Explanation of SymbolsSymbolMEANING=No performance change (confidence interval within ±1%)(=)Probably no performance change (confidence interval within ±2%)(+)/(-)Very small performance improvement/degradation (≤1%)+/-Small performance improvement/degradation (≤5%)++/--Large performance improvement/degradation (≤10%)+++/---Very large performance improvement/degradation (>10%)?Probably no change, but quite large uncertainty (confidence interval with ±5%)??Unclear result, very large uncertainty (±10%)???Something unexpected…


readme changes for submission

Description
Concise description of proposed changes, We recommend using screenshots and videos for better description
Additional Information

[ ] I have read the contributing docs (if this is your first contribution)
[ ] This is not a duplicate of any existing pull request

Related Issues
Closes #{issue number}
Note: If your changes are small and straightforward, you may skip the creation of an issue beforehand and remove this section. However, for medium-to-large changes, it is recommended to have an open issue for discussion and approval prior to submitting a pull request.
Your ENS/address:

https://twitter.com/MontyG10437/status/1672839438202707968


Fix return to summary link while editing services

At some point, our return to service summary links stopped displaying
properly.  Got them back and added a test.
>> Story on Pivotal

Whoops

Fixed

I had noticed this but decided not to fix it because the feature isn’t live. But now you have. So :tada: :+1:.


Calling Aws::initAPI() spawns multiple threads

It appears that a call to Aws::initAPI() spawns '# cpu cores / 2' threads but so far I've been unable to find any documentation on how or why this happens, or how to configure it.
We spawn multiple process that call Aws::InitAPI and on a high cpu count machine this results in more than 2000 threads spawned by the SDK. As threads/PID's are not an infinite resource, any advice on how to remove/limit this behaviour would be appreciated.
Minimal reproducable example:
#include <thread>
#include <chrono>
#include <aws/core/Aws.h>
int main(int, char**)
{
    Aws::SDKOptions options;
    Aws::InitAPI(options);
    std::this_thread::sleep_for( std::chrono::seconds( 600 ) );
    return 0;
}

Compile & run and then do a pstack on its pid, you'll see multiple threads like:
Thread 19 (Thread 0x7f70257e2700 (LWP 301201)):
#0  0x00007f70349fc923 in epoll_wait () from /lib64/libc.so.6
#1  0x00007f7035b73c1b in s_main_loop () from /path/to/aws/1.9.114/lib64/libaws-cpp-sdk-core.so
#2  0x00007f7035c2ed5f in thread_fn () from /path/to/aws/1.9.114/lib64/libaws-cpp-sdk-core.so
#3  0x00007f7034ccee25 in start_thread () from /lib64/libpthread.so.0
#4  0x00007f70349fc34d in clone () from /lib64/libc.so.6

Did a bit more investigation, and was able to limit the threads via:
    options.ioOptions.clientBootstrap_create_fn = []()
    {
        Aws::Crt::Io::EventLoopGroup eventLoopGroup( 2 );
        Aws::Crt::Io::DefaultHostResolver defaultHostResolver(eventLoopGroup, 8, 30);
        auto clientBootstrap = Aws::MakeShared<Aws::Crt::Io::ClientBootstrap>(ALLOCATION_TAG, eventLoopGroup, defaultHostResolver);
        clientBootstrap->EnableBlockingShutdown();
        return clientBootstrap;
    };

Hi @dbowman-ion ,
So.. it seems you've found a sort of workaround, it this now a question on why this happens or?

Yes, understanding why would be nice.
It may also be worth some documentation as neither of these mention anything about threads or how to limit them:
https://sdk.amazonaws.com/cpp/api/LATEST/struct_aws_1_1_s_d_k_options.html
https://sdk.amazonaws.com/cpp/api/LATEST/struct_aws_1_1_io_options.html
I ended up finding the answer in aws-sdk-cpp-core/source/Aws.cpp

Hi,
I guess I have the same issue in my case.
I have an executable that gets some objects from S3, when I call the executable multiple times (sequential about 20-30 times) I get the crash below.
The crash happens when calling Aws::initAPI()
code:
_#include <aws/core/Aws.h>
static Aws::SDKOptions options;
void init_aws(){
Aws::InitAPI(options);
}
main(){
init_aws();
Aws::ShutdownAPI(options);
}_
=5629==ERROR: AddressSanitizer: SEGV on unknown address 0x7fff65800002 (pc 0x7f3868b462bd bp 0x611000000400 sp 0x7f3861bfb0c0 T1)
==5629==The signal is caused by a READ memory access.
#0 0x7f3868b462bc in getenv (/lib64/libc.so.6+0x392bc)
#1 0x7f3867b5f863  (/path/libuv.so.1+0xc863)
#2 0x7f386993820a in __pthread_once_slow (/lib64/libpthread.so.0+0x620a)
#3 0x7f3867b71058 in uv_once (/path/libuv.so.1+0x1e058)
#4 0x7f3867b5fab8 in uv__work_submit (/path/libuv.so.1+0xcab8)
#5 0x7f3867b6a5a1 in uv_getaddrinfo (/path/libuv.so.1+0x175a1)

This problem occurs when the InitAPI and ShutdownAPI are not in the same thread。

Can you make sure you are using the sdk correctly as show in the docs:
#include <aws/core/Aws.h>
int main(int argc, char** argv)
{
   Aws::SDKOptions options;
   Aws::InitAPI(options);
   {
      // make your SDK calls here.
   }
   Aws::ShutdownAPI(options);
   return 0;
}


Rolling issue after upgrading from v0.11.4.

After I updated from v0.11.4 to any of the other versions higher, I have an issue on the character sheet.
If I, the GM, click on (IQ lets say), it rolls as if the GM clicked on it instead of saying the name of the character that rolled it, in the chat window.
In v0.11.4, it said the character name in the chat window.
I would like to roll as the character, as before. (Unless this is a bug/change?)
I have the same character sheet, just downgraded until it started working as desired...  v0.11.4.
I have a fresh install of Foundry VTT and the game aid, to latest versions. (That is how I found out.)
Thanks

@crnormand I reproduced this behavior. If there is no token selected, the roll is done as "Gamemaster". If a token is selected, the roll is done as that character regardless of who the character sheet belongs to.
In the image attached, all three of these rolls where made by clicking on Drex's character sheet.

Do you want to fix this, or shall I take a look?

Fixed, and will be in the next release.


Filter feature generating deprecation warning

When I use the filter function on an index (for example AccountsIndex::Account.filter{ subscription.plan == "Basic" }.total_count) , I'm getting the following warning in the logs:
The [filtered] query is deprecated, please use a [bool] query instead with a [must] clause for the query part and a [filter] clause for the filter part.

With a little debugging, the above query was generating the following JSON being sent to the server:
{:query=>{:filtered=>{:query=>{:match_all=>{}}, :filter=>{:term=>{"subscription.plan"=>"Basic"}}}}}

Yeah, this is simply an outdated Chewy queries code, we are going to update it, but didn't manage yet, sorry

@pyromaniac No need to apologize. Chewy is awesome. Just trying to do my little bit to help out.

Yeah, we are planning to introduce Query2 to support ES >= 2 properly, so hope it is only a matter of time.

Just hit this, seems like this has now been completely removed not just deprecated. Getting errors like this:
     Elasticsearch::Transport::Transport::Errors::BadRequest:
       [400] {"error":{"root_cause":[{"type":"parsing_exception","reason":"no [query] registered for [filtered]","line":1,"col":22}],"type":"parsing_exception","reason":"no [query] registered for [filtered]","line":1,"col":22},"status":400}

@jnicklas, if you go back to elasticsearch 2.4, you should be OK until the gem gets updated

@aselder that's what I did to work around this for the time being. Thanks :)

Any updates on fixing this issue ?

Any updates on fixing this issue ?

Any updates on this? seems like the gem doesn't work ES5 now

https://github.com/toptal/chewy/pull/493


Cbindgen problem

issue：Do I need to implement these undefined functions myself？

@c410-f3r Ok, thank you for your answer，I have another question that I would like to ask.
below is a screenshot of the generated filecoin_sign.h header file：

Executed command：

cbindgen --config /Users/kk/Downloads/filecoin-rs-master/signer-ffi/cbindgen.toml  --crate filecoin-signer --output filecoin_signer.h -v

Executed result image：

issue：There seems to be a lot of missing definitions in the header file,I want to generate a complete definition, how do I do that?

It is probably because the target crate should be filecoin-signer-ffi instead of filecoin-signer.
cbindgen --config /Users/kk/Downloads/filecoin-rs-master/signer-ffi/cbindgen.toml --crate filecoin-signer-ffi --output filecoin_signer.h -v

@c410-f3r
below is a screenshot(This header file that I generated yesterday):

The function in filecoin-signer ffi.h doesn't have the function interface I want, and I think the function interface I want should be the target in filecoin-signer

ExtendedKey is a opaque structure because we deal with it internally in the Rust side. It is possible and trivial to expose its field but maybe it should be desired to avoid conflicts.

@c410-f3r
Okay, thank you. Good night💤


add iso8601 grammar

[x] dates
[x] times
[ ] expanded forms
[ ] durations
[ ] time intervals (point + duration)
[ ] pass all tests

This addition has been pending for a while and while it would be nice to have it appears that we didn't get around to doing much for it, wherefore I'll close this pull-request for now with the understanding that we can re-open and re-activate it in the future.


"send_queue is full"    之后出现 Send.OSError   Connection reset by peer 错误

这个问题是 因为 python  与 ubutnu 冲突引起吗？

@alex @smn

Hello @vagabond1132 ,
I'm sorry, but it's not clear from the screenshot above what your problem is, and why do you blaming carbon for it?

Server 程序是python编写的   我使用客户端与之通讯的时候，发现不能接收到服务端数据， 查看服务端log， 发现 send_queue is full , 并且 Send.OSError   , 并不知道到底发生了什么？ 请求帮助， thanks

@deniszh  hi,  Server 程序是python编写的 我使用客户端与之通讯的时候，发现不能接收到服务端数据， 查看服务端log， 发现 send_queue is full , 并且 Send.OSError , 并不知道到底发生了什么？ 请求帮助， thanks

Hi @vagabond1132,
If you have this error in carbon - upgrade to Graphite 0.9.16 or 1.0.1
See https://github.com/graphite-project/carbon/issues/505
But as far as I see this message is not from carbon, but from some other program, it's just similar error message.

yes,   it`s  just similar error message  but I don't know how to solve ;  What reason caused this situation ; so  Please help  thank you


[CPU] Enable mmt4d distribution for big reduction size cases.

The distribution was disabled when the reduction size is huge. It leads the max tile size being 0, which stops distribution.
  int64_t reductionSize = ShapedType::isDynamic(K1) ? 1024 : K0 * K1;
  auto getMatmulTileSize = [](int64_t targetTileBytes, int bitWidth,
                              int64_t reductionSize, int64_t Tile0Size) {
    int64_t targetRhsTileElems = targetTileBytes * 8 / bitWidth;
    int64_t targetRhsTileNSize = targetRhsTileElems / reductionSize;
    return llvm::divideCeil(targetRhsTileNSize, Tile0Size);
  };

I noticed the issue when I prototyped cache level tiling. The case is from Falcon7B; I replace fp16 with fp32.


Map Rotation heading issue

Hello @dceejay,
Than You for amazing work with implementing map rotation, however I've noticed that there is small issue whith heading of icon.
For example map rotation = 0, arrow heading 180 everything is as expected.
but if i rotate the map to 90 than speed vector is pointing in proper direction but arrow icon still points 180 witch in fact is 180 + 90 = 270 real degree.
Im attaching a small photo there you can see.

can you share the exact msg to recreate ? Thanks

Than You so much for this help :)
I've tried
var dir = parseFloat(data.hdg ?? data.heading ?? data.bearing ?? "0") + map.getBearing();
And it seams to work now as expected i will investigate it more when i'll reconnect GPS back (now i am testing without).

great, as I said it's not a complete fix as it won't update "old" markers... but should be fine for live...
Proper fix will be in v4 later this week.

Yes thats correct, it is not updating the markers but i made a litle work arround...
My inject node is at the moment parsing:
payload.heading as well as payload.command.rotate - both of them are the same.
and to force the map to have this point in the center i am parsing in the same inject node
payload.lon and payload.command.lon - both of them are the same.
payload.lat and payload.command.lat  - both of them are the same.

v4 now released


update to show Zero values as in the ticket SCT-439

Desc:
The code has been modified to show zero values as in the ticket(SCT-439).
If a configuration is not present in the default config then that value won't come from the template as it checks for null values.
plugins:
    trivy:
       exit-code: 0
       timeout : 5m0s
       severity: HIGH
       ignore-unfixed: true
       security-checks: vuln,config
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""

go run . trivy
steps:
  - command: ls
    plugins:
      - equinixmetal-buildkite/trivy#v1.18.2:
          exit-code: 0
          timeout : "5m0s"
          severity: "HIGH"
          ignore-unfixed: true
          security-checks: "vuln,config"
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""

go run . trivy --exit-code=0 --ignore-unfixed=false
steps:
  - command: ls
    plugins:
      - equinixmetal-buildkite/trivy#v1.18.2:
          exit-code: 0
          timeout : "5m0s"
          severity: "HIGH"
          ignore-unfixed: false
          security-checks: "vuln,config"
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""

go run . trivy --exit-code=42 --ignore-unfixed=false
steps:
  - command: ls
    plugins:
      - equinixmetal-buildkite/trivy#v1.18.2:
          exit-code: 42
          timeout : "5m0s"
          severity: "HIGH"
          ignore-unfixed: false
          security-checks: "vuln,config"
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""

@hnadiminti-equinix i've left a few minor comments
please also have a look and test - it's segfaulting now:
$ git status
On branch SCT-439
Your branch is up to date with 'origin/SCT-439'.

nothing to commit, working tree clean

$ go run . trivy
{"level":"debug","msg":"config path:conf.yaml","time":"2023-09-22T17:01:09+01:00"}
{"level":"debug","msg":"error while loading the configuration file. Loading the defaults","time":"2023-09-22T17:01:09+01:00"}
{"level":"debug","msg":"trivy plugin configuration not found in the config file or wrong config file. Proceeding with defaults from command line flags.","time":"2023-09-22T17:01:09+01:00"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x1046f27f4]

goroutine 1 [running]:
dynamic-buildkite-template/cmd.setFromIntFlag(0x0, 0x100000005?, {0x1046f4b89?, 0x1?}, 0x1?)
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/cmd/util.go:40 +0xa4
dynamic-buildkite-template/cmd.CreateGenerator(0x0?, {0x0?, 0x0?, 0x0?})
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/cmd/trivy.go:61 +0x12c
dynamic-buildkite-template/cmd.glob..func2(0x1400020c200?, {0x104b3c7a0?, 0x4?, 0x1046f36d0?})
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/cmd/trivy.go:25 +0x20
github.com/spf13/cobra.(*Command).execute(0x104b026e0, {0x104b3c7a0, 0x0, 0x0})
	/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:944 +0x640
github.com/spf13/cobra.(*Command).ExecuteC(0x104b02400)
	/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:1068 +0x320
github.com/spf13/cobra.(*Command).Execute(...)
	/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:992
dynamic-buildkite-template/cmd.Execute(...)
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/cmd/generate.go:50
main.main()
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/main.go:23 +0x28
exit status 2

i think that's because it's switched to pointers, and they need to be initialised - ie, non-nil, before value assignment is attempted

I'm going to add nil check as suggested ...WIP, Thank you @prezha

@hnadiminti-equinix i've left a few minor comments
please also have a look and test - it's segfaulting now:
$ git status
On branch SCT-439
Your branch is up to date with 'origin/SCT-439'.

nothing to commit, working tree clean

$ go run . trivy
{"level":"debug","msg":"config path:conf.yaml","time":"2023-09-22T17:01:09+01:00"}
{"level":"debug","msg":"error while loading the configuration file. Loading the defaults","time":"2023-09-22T17:01:09+01:00"}
{"level":"debug","msg":"trivy plugin configuration not found in the config file or wrong config file. Proceeding with defaults from command line flags.","time":"2023-09-22T17:01:09+01:00"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x1046f27f4]

goroutine 1 [running]:
dynamic-buildkite-template/cmd.setFromIntFlag(0x0, 0x100000005?, {0x1046f4b89?, 0x1?}, 0x1?)
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/cmd/util.go:40 +0xa4
dynamic-buildkite-template/cmd.CreateGenerator(0x0?, {0x0?, 0x0?, 0x0?})
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/cmd/trivy.go:61 +0x12c
dynamic-buildkite-template/cmd.glob..func2(0x1400020c200?, {0x104b3c7a0?, 0x4?, 0x1046f36d0?})
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/cmd/trivy.go:25 +0x20
github.com/spf13/cobra.(*Command).execute(0x104b026e0, {0x104b3c7a0, 0x0, 0x0})
	/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:944 +0x640
github.com/spf13/cobra.(*Command).ExecuteC(0x104b02400)
	/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:1068 +0x320
github.com/spf13/cobra.(*Command).Execute(...)
	/go/pkg/mod/github.com/spf13/cobra@v1.7.0/command.go:992
dynamic-buildkite-template/cmd.Execute(...)
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/cmd/generate.go:50
main.main()
	/go/src/github.com/equinixmetal-buildkite/dynamic-buildkite-template/main.go:23 +0x28
exit status 2

i think that's because it's switched to pointers, and they need to be initialised - ie, non-nil, before value assignment is attempted

I'm going to add nil check as suggested ...WIP, Thank you @prezha

I shifted to pointers for members of TrivyPluginConfig because we wanted to print zero values. In addition we want to check if a parameter/flag is not defined or has zero values. The pointer becomes a suitable choice as we can check for nil if the flag is not defined.
I've added code blocks like this to read default command line flag values in case of missing/wrong config file. It also checks if the command line flags override the conf, if conf file is present and correct.
if doLookup {
if cmd.Flags().Lookup(name).Changed {
*f = mustGetIntFlag(cmd, name)
}
} else {
*f = mustGetIntFlag(cmd, name)
}
}
These blocks dereference the input variable pointer to assign value to it. And this is causing nil pointer exception.
I have modified the code to handle these cases and it's working fine now.
If the config file conf.yaml is not present. It picks from the default command line flag values. It also supports overriding those.
dynamic-buildkite-template % rm .\conf.yaml
dynamic-buildkite-template % go run . trivy
{"level":"debug","msg":"config path:conf.yaml","time":"2023-10-02T12:51:13+01:00"}
{"level":"debug","msg":"error while loading the configuration file. Loading the defaults","time":"2023-10-02T12:51:13+01:00"}
{"level":"debug","msg":"trivy plugin configuration not found in the config file or wrong config file. Proceeding with defaults from command line flags.","time":"2023-10-02T12:51:13+01:00"}
{"level":"info","msg":"latest trivy plugin tag: v1.18.3","time":"2023-10-02T12:51:13+01:00"}
steps:
  - command: ls
    plugins:
      - equinixmetal-buildkite/trivy#v1.18.3:
          exit-code: 0
          timeout : "5m0s"
          severity: "HIGH,CRITICAL"
          ignore-unfixed: true
          security-checks: "vuln,config"
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""

If conf.yaml is present it reads and point the values/config from the file along with zero values, also it supports command line overrides.
dynamic-buildkite-template % cat conf.yaml
plugins:
  trivy:
    exit-code: 0
    timeout: 5m0s
    severity: HIGH
    ignore-unfixed: true
    security-checks: vuln,config
    skip-files: ""
    skip-dirs: ""
    image-ref: ""
    version: ""
    helm-overrides-file: ""

 dynamic-buildkite-template % go run . trivy
{"level":"debug","msg":"config path:conf.yaml","time":"2023-10-02T13:14:23+01:00"}
{"level":"info","msg":"latest trivy plugin tag: v1.18.3","time":"2023-10-02T13:14:24+01:00"}
steps:
  - command: ls
    plugins:
      - equinixmetal-buildkite/trivy#v1.18.3:
          exit-code: 0
          timeout : "5m0s"
          severity: "HIGH"
          ignore-unfixed: true
          security-checks: "vuln,config"
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""

dynamic-buildkite-template % go run . trivy --severity="HIGH,CRITICAL"
{"level":"debug","msg":"config path:conf.yaml","time":"2023-10-02T13:15:16+01:00"}
{"level":"info","msg":"latest trivy plugin tag: v1.18.3","time":"2023-10-02T13:15:16+01:00"}
steps:
  - command: ls
    plugins:
      - equinixmetal-buildkite/trivy#v1.18.3:
          exit-code: 0
          timeout : "5m0s"
          severity: "HIGH,CRITICAL"
          ignore-unfixed: true
          security-checks: "vuln,config"
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""

if a flag is not defined in the conf.yaml then you won't see that in the output.This allows the user to not define few parameters in the template.However, we can always add that using command line override. For example, I have commented the severity parameter.
dynamic-buildkite-template % cat conf.yaml
plugins:
  trivy:
    exit-code: 0
    timeout: 5m0s
    #severity: HIGH
    ignore-unfixed: true
    security-checks: vuln,config
    skip-files: ""
    skip-dirs: ""
    image-ref: ""
    version: ""
    helm-overrides-file: ""
dynamic-buildkite-template % go run . trivy
{"level":"debug","msg":"config path:conf.yaml","time":"2023-10-02T13:20:18+01:00"}
{"level":"info","msg":"latest trivy plugin tag: v1.18.3","time":"2023-10-02T13:20:19+01:00"}
steps:
  - command: ls
    plugins:
      - equinixmetal-buildkite/trivy#v1.18.3:
          exit-code: 0
          timeout : "5m0s"
          ignore-unfixed: true
          security-checks: "vuln,config"
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""
dynamic-buildkite-template % go run . trivy --severity="HIGH,CRITICAL"
{"level":"debug","msg":"config path:conf.yaml","time":"2023-10-02T13:20:34+01:00"}
{"level":"info","msg":"latest trivy plugin tag: v1.18.3","time":"2023-10-02T13:20:34+01:00"}
steps:
  - command: ls
    plugins:
      - equinixmetal-buildkite/trivy#v1.18.3:
          exit-code: 0
          timeout : "5m0s"
          severity: "HIGH,CRITICAL"
          ignore-unfixed: true
          security-checks: "vuln,config"
          skip-files: ""
          skip-dirs: ""
          image-ref: ""
          helm-overrides-file: ""

/lgtm
thanks @hnadiminti-equinix for fixing this!

/lgtm thanks @hnadiminti-equinix for fixing this!

Thank you @prezha !


chore(dep-check): bump a couple of RN versions to latest patch

Description
A couple of RN versions were not pointing to latest available, which in some scenarios might lead to baad behaviours when ex. the native side pick sup the version from the package.json but doesn't understand semver so it complains (ex. 0.63.2 v 0.63.4 that we are hitting in a project right now).
For some reason once again I can't seem to get the update-script to update the README, so @tido64 do you think you'd be able to do it? 😓

I don't think we need this anymore


Invalid documentation for fields_are_metrics

Hello,
it seems to me like the following statement is invalid(or out of date?)
# An array indicating that these event fields should be treated as metrics
  # and will be sent verbatim to Graphite. You may use either `fields_are_metrics`
  # or `metrics`, but not both.
  config :fields_are_metrics, :validate => :boolean, :default => false

So, what which type is actually expected? Boolean or array?
Thanks.

I'm properly confused. https://github.com/logstash-plugins/logstash-output-graphite/issues/13 shows fields_are_metrics as an array, yet even unit tests in .spec are written to pass boolean.

Bump, and ditto here:
https://github.com/logstash-plugins/logstash-output-graphite/blob/master/lib/logstash/outputs/graphite.rb#L70
It seems like the metrics filter plugin and the graphite output plugin are just out of sync with one another.  Metrics is emitting events that have nested hashes, and with graphite, the only way to access those is by specifying exactly which of those values you want, a la:
metrics => { "my_counter.rate_1m" => "%{[my_counter][rate_1m]} }
This gets cumbersome quickly when you want to track anything more than a few counters and timers.


Record organizer overflowing long texts

for today: add finishing touches and submit for code review in an hour

The issue is present in the fields:

record
email
url

for today: resolve comments and resubmit for code review

for today: continue implementing a solution and submit code for review


Define levels of data and code sharing that apply to fMRI

Each level will have its own "lesson" with examples.

Here is a first list of use-cases:

Documenting the experiment (no data shared)
Documenting the experiment + sharing of results data
Documenting the experiment + sharing of results data + sharing of raw data
Documenting the experiment + sharing of results data + sharing of raw data + sharing of preprocessed data

To do: define "results", "raw", and "preprocessed" data.

Discussed with @demitri: think about how to break down those use-cases into smaller pieces and how those smaller pieces can link to one another.

Hello @cmaumet, just inquiring about your work pipeline here. I have some experience with data extraction/cleaning/manipulation in python, however, I do not have any experience with neuroimaging. Is there a group dedicated to the neuroimaging process of the project? Is there any possibility that someone with zero experience with fMRI imaging can contribute to this project?

Hi @yungpadwan! Thank you for your interest in this project! Would you like to contribute to https://github.com/easyreporting/fmri_reporting/issues/7 in which we try to build a list of external resources with good practices for transparent research?

@cmaumet Sure!


Include ShelleyMA checks

Related to #7
Using the dev/pallas-applying-public-API pallas branch.
The first step is to classify the checks based on the information they need. Some checks only require transactions, while others require a subset of the UTxO set.

The validations from the Shelley MA era are listed below, in a block structure. Blocks start with a statement that describes requirements of the validations within that block. Nested blocks require their requirements and the ones from containing blocks.
Some validations require specific context to validate, those validations (7th to 10th validations) were implemented but not added to the validations list. There are some validations that require context, but were added to the validations list, using hardcoded values (such as ones that need protocol params, era, block slot, and network id).

- The following validation only requires the tx size and the protocol parameters
1. validate_byron_ma_size(size, prot_pps)

- The followiing validations require the transaction
2. validate_shelley_ma_ins_not_empty(mtx_sma)
3. validate_shelley_ma_metadata(mtx_sma)
4. validate_shelley_ma_minting(mtx_sma)

    - The following validations require the size and the protocol parameters
    5. validate_shelley_ma_fees(mtx_sma, size, prot_pps)

    - The following validations require the protocol parameters and the era
    6. validate_shelley_ma_min_lovelace(mtx_sma, prot_pps, era)

    - The followinf validation requires the tx utxos
    7. validate_shelley_ma_ins_in_utxos(mtx_sma, utxos)

        - The following validation requires the era
        8. validate_shelley_ma_preservation_of_value(mtx_sma, utxos, era)

        - The following validation requires the transaction witness set
        9. validate_shelley_ma_witnesses(mtx_sma, tx_wits, utxos)

    The following validation requires the block slot
    10. validate_shelley_ma_ttl(mtx_sma, block_slot)

    The following validation requires the network id
    11. validate_shelley_ma_network_id(mtx_sma, network_id)


Custom templatetag implementation

I am switching an existing Django project's templates to jinja2. I use crispy forms (https://github.com/django-crispy-forms/django-crispy-forms/)  to render all forms in the project but jinja2 does not understand the {% crispy form %} tag. I need help with an extension that registers the crispy form tags in jinja2
Environment

Python version: 3.6
Jinja version: 2.10

You cannot use a tag written for Django templates in Jinja - look at its code what it does, maybe you can re-implement it as a Jinja extension.


Can change istio-proxyv2 access log's START_TIME value according to timezone?

I used isito with envoy
Below red box part is  START_TIME , the whole record is an access log

I want istio access log's START_TIME value can be changed according to pod's timezone, e.g.

if change TZ to Asia/Shanghai, START_TIME will use CST time.
if change TZ to Asia/Seoul, START_TIME will use KST time.

But it seems it doesn't support, my test as below
After installed tzdata package to corresponding istio-proxyv2 image, now can configure any timezone using below TZ env, e.g.
- name: TZ
   value: Asia/Shanghai

I tried Asia/Shanghai, Asia/Seoul ,  America/New_York , above START_TIME still uses UTC value.
There has any method to change istio-proxyv2 access log's START_TIME  value according to TZ?
Since istio-proxyv2 based on envoy, so i opened a new issue here.

Please raise this issue with Istio.

@mattklein123 istio does nothing special to Envoy
AFAIK, %START_TIME% do not respect TZ env, can senior envoy maintainers confirm that.

Envoy always reports in UTC time zone, there's no option to customize it.  I'm not sure why we need it strongly, since that requires parsing TZ data, etc.

@kyessenov
AFAIK, loki logStorage system also print the raw log's timestamp filed using UTC, but when show in grafana, it added a another timestamp to user(Its value is insertion time of the log record, the value based on ENV: TZ).
show example as below


GH-1477: Reduce Log Noise While Broker Down

Resolves https://github.com/spring-projects/spring-amqp/issues/1477
Only log the stack trace on the first failure.
Tested with a Boot app and stop/start/stop the broker.
cherry-pick to 2.4.x

... and cherry-picked to 2.4.x