Idea: Chia Plot Planner function

Is it possible to expand the script function with a "Chia plot planner" to avoid wasting space? So I choose a disk, and the plotter apply the best k32, k33, k34 compilation based on free space.

There's already a line command you can use called "Get-ChiaKPlotCombination"
Example:
Get-ChiaKPlotCombination | Show combinations from all drives
Get-ChiaKPlotCombination -DriveLetter d | Show combinations from just drive d

Thanks 👍

It would be nice for the GUI to do this automatically, but unfortunately that bring in too much complexity for the current moment. As phpfrog mentioned there are a powershell command to find the best optimizations of space, and from there you will have to setup how you want to plot those combinations. Thank you for the idea :)


Timeout Pattern

This is used so heavily, it would be good to have a creator for it, or at least document the convention that we use for it.

We got a decorator somewhere along the way....it works great and has a test for it now too.

Went in with f4e2ef69 and got bugfixes for the test in 88ef1b0e


Make GaussianChiEffChiP jittable

The if statement in the GaussianChiEffChiP causes jittability issues. This removes it. If computing the normalizations becomes a bottleneck (I do not think it will), we should consider adding a flag for disabling normalizations during likelihood calculations and enabling them for things like rate calculations.

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

Project coverage is 94.55%. Comparing base (573a084) to head (6ddefc8).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
- Coverage   94.57%   94.55%   -0.02%
==========================================
  Files          11       11
  Lines         811      808       -3
==========================================
- Hits          767      764       -3
  Misses         44       44

Flag
Coverage Δ

python3.10
94.55% <100.00%> (-0.02%)
:arrow_down:

python3.11
94.55% <100.00%> (-0.02%)
:arrow_down:

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

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


new password length is not checked on recovery

Steps to reproduce:
register account
recover account
click link
the new password can be any >0 length, was able to use 1 character only
Suggestion:
should check for the same length requirement as on registration

I think kratos supports password rules natively. No need to code it here
Sent from my iPhone

On 25. Aug 2022, at 14:52, openoms @.***> wrote:

Steps to reproduce:
register account
recover account
click link
the new password can be any >0 length, was able to use 1 character only
Suggestion:
should check for the same length requirement as on registration
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.


SCRD-3497 Avoid passing list in extra vars

When calling the install playbook, the extra-vars should contain a node
list.  The node list should always be a simple string, rather than a
python list.  Note that in the event that multiple nodes are to be
supplied (not relevant in this change), the string should be a
comma-separated list of nodes.

Note that this problem was discovered when testing on our environment in the provo lab.


CatalogExportSkillsUnderGroupSpec.export skills under a group to the catalog can fail due to the results' order

https://github.com/NationalSecurityAgency/skills-service/runs/6848320829?check_suite_focus=true

PR: https://github.com/NationalSecurityAgency/skills-service/pull/1396


Added all italian provinces

I've added all italian provinces "santo patrono", wrote in comments the reference links about the dates and the list.
Implemented 100% of the tests about provinces and cities of composites provinces (like Balretta-Andria-Trani).
Edited provinces list in the readme to reflect all the provinces implemented.

I hope to have not made mistakes as this is my first PR...

I forgot to say that I didn't add the note in the readme.rst file about "Santo Patrono" of provinces (now called subdivisions) are only valid in "Capoluogo di provincia" and not in the rest of province territory, if not in some exceptions. Just to ensure an informed use of library by users.

@dr-prodigy can you please suggest me what's the problem with the quality assurance checks? So the next time I can do what I did not this time.

Hi @energywave
thank you for your work!
I've just merged it on beta (just a few code format fixes were required.. good job!).
Regarding README notes, I considered not to explicit this: all around the lib (and world too) there are a good number of specific caveats/not obvious behaviours, but none of them appears in the README (aiming to stay short and synthetic).
Those are usually clarified in the code with comments.
Thx again, best 👍

Wonderful, all clear. Thank you and keep on the good work :)


Doesn´t work over a polygon layer

When any polygon (GeoJSON) layer is switched on I am not able to open context menu by right-click. Outside a layer or when layer is switched off it works great. Any idea? Thanks

Hi, I haven't seen that issue but then again I've never used a geojson layer. Do you have a test case that demonstrates the problem?

Hi, the example is available on: http://rostanetek.cz/_geocentrum/v6/prikazy/test.html . Over the layer it doesn´t work, while outside it is OK.

Hi, the example is available on: http://rostanetek.cz/_geocentrum/v6/prikazy/test.html . Over the layer it doesn´t work, while outside it is OK.

I think you'll need to use a combination of the GeoJSON.onEachFeature() method and the bindContextMenu() method that the plugin mixin adds to the L.Path layer. Can you try that and let me know how it goes?

Amazing! It is neccessary only add "contextmenu: true"  into  onEachFeature e.g. "onEachFeature:popUp, contextmenu: true". Thanks

Great! Can you share the complete code you're using for the GeoJSON layer so I can look at adding an example to the docs?

Hi,
I have a problem with polygon (GeoJSON) layer - is it possible to know which polygon was selected in the function deletePolygon,()?
contextmenu: true,
contextmenuItems: [{
text: 'Delete polygon',
icon: 'images/delete.png',
callback: deletePolygon,
index: 0
},
{
separator: true,
index: 1
}]
If yes, could you, please provide an example using that type of event?

See https://github.com/aratcliffe/Leaflet.contextmenu#contextmenushow.  Add a listener for this map event and the event object's relatedTarget property will reference the L.Path instance the context menu was opened for.

I'm aware this is a closed issue but striking the same problem with wanting the context menu working on GeoJSON features, the following code worked for me:

function onEachFeature(feature, layer) {
          // enable context menu
          layer.on({ contextmenu: true });
 }

onEachFeature is well documented at http://leafletjs.com/reference.html#geojson


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-watsonx-saas-da
module
patch
v1.7.8 -> v1.7.9

terraform-ibm-modules/cos/ibm (source)
module
patch
8.14.3 -> 8.14.4

Release Notes

terraform-ibm-modules/terraform-ibm-watsonx-saas-da (github.com/terraform-ibm-modules/terraform-ibm-watsonx-saas-da)
v1.7.9
Compare Source
Bug Fixes

added new input enable_cos_kms_encryption to toggle KMS encryption with default value of true.

NOTE: When value is set to true, a value must be passed for the cos_kms_crn input.
This change is required to fix a bug where terraform was iving the error: The "count" value depends on resource attributes that cannot be determined until apply, so Terraform cannot predict how many instances will be created. (#213) (53a4e2a)

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v8.14.4
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#778) (19ca0d7)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 2.5.25 :tada:
The release is available on:

GitHub release
v2.5.25

Your semantic-release bot :package::rocket:


Not enough detail

This is very brief. It doesn't describe the purpose of a TextConst and why they are used in BC. It does not give an example of how to use it or where to put it on the AL object.

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

ID: 5430cd34-1b14-3cea-c29e-f161b6793b78
Version Independent ID: 36251966-e872-e2f8-d462-5efbec2f66ba
Content: TextConst Data Type - Business Central
Content Source: dev-itpro/developer/methods-auto/textconst/textconst-data-type.md
Service: dynamics365-business-central
GitHub Login: @solsen
Microsoft Alias: solsen

Yes, you're absolutely right. I've added some more content and will be pushing the topic live soon. The TextConst datatype is typically used for UI messages, below you can see a code example of the data type with local and global declaration:
codeunit 50100 MyCodeunit
{
procedure MyProcedure(): ErrorType
var
localTextConst: TextConst ENU = 'My text', DAN = 'Min tekst';
begin
Message(localTextConst);
end;
var
    globalTextConst: TextConst ENU = 'My text', DAN = 'Min tekst';

}


fix: Include the generator for mount run commands

Already implicity used by https://github.com/jdx/usage/blob/main/cli/src/cli/generate/fig.rs#L343

Still not working perfectly need to dig a bit deeper, don't merge yet

I'm trying to fix your PR because the job is failing by merging, you can feel free to overwrite. I can only test this using a fork.

It seems to work now locally.
If, out of nowhere,  it breaks tomorrow I'll investigate if it's something related to caching or the likes.


[BUG] Pipeline task returns InvalidPassFailCriteria when yaml has more than 5 failureCriteria

Describe the bug
AzureLoadTest@1 - Azure Load Testing v1 task  fails pipeline with error code: 'InvalidPassFailCriteria' when  failureCriteria contains more than 5 items.
To Reproduce
Steps to reproduce the behavior:
pipeline.yml:
 # variables are set before this, other details removed for brevity
stages:
    - stage: LoadTest
      displayName: Load test
      jobs:
          - job: Run
            displayName: Runs a fixed load test in azure
            steps:
              - task: AzureLoadTest@1
                displayName: Azure load test
                inputs:
                  azureSubscription: $(serviceConnection)
                  loadTestResource: $(loadTestResource)
                  resourceGroup: $(loadTestResourceGroup)
                  loadTestConfigFile: src/HomePage/HomePage.yml

src/HomePage/HomePage.yml:
version: v0.1
testName: BHB_Home_Page
testPlan: HomePage.jmx
description: BHB Home page
engineInstances: 1
splitAllCSVs: False
failureCriteria:
  - AppointmentType: percentage(error) > 1
  - AppointmentType: p90(response_time_ms) > 1000
  - PatientLocationById: percentage(error) > 1
  - PatientLocationById: p90(response_time_ms) > 1000
  - Provider: percentage(error) > 1
  - Provider: p90(response_time_ms) > 1000
env:
  - name: JM_RampTime
    value: 30
  - name: JM_EndRpsA
    value: 100
  - name: JM_EndRpsB
    value: 100
  - name: JM_HoldFor
    value: 300
  - name: JM_CoolDown
    value: 10
  - name: JM_PauseFor
    value: 5

result from DevOps pipelines
Starting: Azure load test
==============================================================================
Task         : Azure Load Testing
Description  : Automate performance regression testing with Azure Load Testing
Version      : 1.2.9
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/load-testing/tutorial-cicd-azure-pipelines#azure-load-testing-task
==============================================================================
{ error:
   { code: 'InvalidPassFailCriteria',
     message: 'Invalid pass fail criteria "Invalid PassFailCriteria" .',
     target: null,
     details: null } }
##[error]Error in creating test: bhb_home_page
Finishing: Azure load test

Expected behavior
Azure load testing task  should allow create/update test with the same amount of test criteria as the tests created by portal.
Screenshots
Error from pipeline:

Additional context
Add any other context about the problem here.
AB#1642808

@leandrodotec ... Noted. we will update once we have rectified the issue.

We are having the same issue


Running tasks broken dependencies with poetry

Hi
I'm having an issue where I try to create tasks on installation for various ubuntu packages (to be run in a docker build process).
The issue is that in the pyproject.toml there are dependencies which are relative, for example:
lib1 = {path = "../../lib1"}
but those packages aren't available anymore because I've dropped them in the building process.
Trying to run any task with poetry results in:
Directory ../../lib1 does not exist
error
and the task isn't executed.
Task:
[tool.poe.tasks]
[tool.poe.tasks.install-lib1]
help = "Install lib1"
shell  = "echo test"
I'm not sure why running the plugin tries to access the dependencies but I assume it shouldn't.
Not sure if there is a way to avoid this issue at the moment.

Hi @OceanManOne,
Hard to say what's happening here, but maybe I can offer some clues.
Does the same thing happen when running other poetry commands? If so it's probably not to do with poe.
Does the same thing happen if you just call the poe command on poetry without a task? (like poetry poe)
Poe doesn't know anything about declared dependencies, but it does try to use the poetry managed venv for running tasks so that tasks benefit those dependencies, and for this it will call poetry env info and maybe also poetry run .... My best guess is that when calling poetry to get details about the env poetry raises that error.
I'm curious what you're trying to achieve here... why don't you bring all the dependencies into the container?


Top navigation disappears when deploying custom web part

I have created a custom web part using SPFX version 1.10.0. The custom webpart display list of items from the sharepoint list. I have Material UI framework to develop this report. The issue is when I add my webpart in a site page and publish I could observe the top navigation menu disappears as shown in the below screenshot. Please advise to resolve the issue

Hi, I am having trouble reproducing this in my environment. Can you confirm if your material styles are conflicting with Microsoft styles in some way?
Also, would it be possible if you could share your SPFx solution with us? That may  make it easier to see what is happening.

Attached my spfx solution.
WaitingOnMeGrid.zip

Hi, unfortunately, I am unable to load this. It looks like the solution is dependent on a library component that wasn't included in your solution.


Expose CARGO_FEATURE_ environment variables to tests

This can be useful if you need to enumerate the active features within an integration test which spawns builds.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

r? @alexcrichton I'm unsure if this has been intentionally unimplemented!

Isn't this possible via routes like:
pub fn foo_activated() -> bool { cfg!(feature = "foo") }

It is possible, but it can be error prone as cfg!(feature = "foo") doesn't check for the feature to exist afaik so it's subject to typos.
More concretely, the intended use case is to have some infrastructure to test examples within a project, shelling out to cargo for the builds. It does it this way, so as if the user does cargo test --test foo then foo will rebuild it's dependant example and have quicker development cycles. This might actually be really misguided and I'd love to hear of better suggestions for ensuring examples perform as expected!
Currently we have something like the above in place for nickel. We currently pull the active features from an env-var which we set in CI, but that's a real footgun when testing locally. I'd like to be able to extract it out into something more general and if we were to do that it would be great if we didn't have to require the user to have boilerplate to determine active features (especially if they could make errors easily) and make it just work from within #[test] functions.
#[test]
fn foo() {
    // the Example lib knows to enumerate CARGO_FEATURE_* and add it to the sub-builds
    let process =  Example::run("bar");
    assert!(process.stdout.contains("foo"));
}

// vs

fn active_features() -> Vec<Option<&'static str>> {
     vec![
         if cfg!("baz") { Some("baz") } else { None }
     ]
}

#[test]
fn foo() {
    // would need to respecify in every test since we don't have any kind of global init for test executables
    let process =  Example::run_with_features("bar", active_features());
    assert!(process.stdout.contains("foo"));
}

Since all of the tests will have been compiled with the same set of features and all of the example builds should be replicating the active set, all of the tests will have to have the same calls, or the user would have to implement their own wrapper to DRY it up themselves.
Perhaps an alternative would be to allow examples to have test code themselves (which cargo would know about), which would keep things simpler in many ways and add a form of additional documentation to examples (what's expected output, how to test). I have seen some projects which have their own Cargo.toml per example, which could imitate this kind of thing, but it would be nice to not have to split into a lot of subprojects and rely on makefiles.
Very open to better suggestions!

Oh these variables are also set at runtime for tests! Isn't that just another footgun in a sense? I think lots of tools like to be able to run binaries directly rather than through cargo test, and this means that they wouldn't work out of the box :(.
Could Example be defined in the test harness and it could enumerate all features via cfg!(feature = "...") as well, right? I don't think we've had many requests in the past for #[test] in examples, but it seems reasonable to me! It may want to be opt-in behavior, but seems easy enough to implement.

Sorry for the delayed response, lost track of this!

I think lots of tools like to be able to run binaries directly rather than through cargo test, and this means that they wouldn't work out of the box :(.

This is something I hadn't considered at all and it probably would have been broken for a lot of cross compilation stuff. Good point!

Could Example be defined in the test harness and it could enumerate all features via cfg!(feature = "...") as well, right?

I didn't quite follow this, what is the test harness in this case? The user's test code, or the supporting test library? If it's the latter then I don't know how it can work as it wouldn't know which features to check for as it's meant to be generic upstream library code. Perhaps something like CARGO_FEATURES (a list of all active features?) could be added at compile time which it could parse?
I've opened https://github.com/rust-lang/cargo/pull/2667 in relation to the example testing.

Oh I just meant that in your example the Example struct could have enumerated all of the features or rather somewhere in the snipped could have enumerated all features and worked like that. I can imagine that this is a bit of a pain though.
Something like CARGO_FEATURES sounds good to me though!

:umbrella: The latest upstream changes (presumably #2743) made this pull request unmergeable. Please resolve the merge conflicts.

Closing due to inactivity, but feel free to reopen with a rebase!


Update terms and conditions

Purpose

Closes #93
Changes

Added new paragraph  in terms and conditions

Checklist

[ ] My code follows the style of this project.
[ ] The code compiles without warnings.
[ ] I have performed a self-review of the changes.
[ ] I have documented my code, in particular the intent of the
hard-to-understand areas.
[ ] (If necessary) I have updated the CHANGELOG.

CLA acceptance
_Remove if not applicable.
By submitting the contribution I accept the terms and conditions of the
Contributor License Agreement v1.0

link: https://developers.concordium.com/CLAs/Contributor-License-Agreement-v1.0.pdf

[ ] I accept the above linked CLA.

Sure 👍🏻


Scripting events are not working on 64-bit build

Following script code works flawlessly on a 32-bit soldatserver but on a 64-bit build any assigned game event is not called up. This seems to be related to https://github.com/remobjects/pascalscript/issues/153 and this.
procedure MyOnClockTickHandler(Ticks: Integer); // this won't be called on 64bit
begin
  WriteLn('OnClockTick event called: ' + IntToStr(Ticks));
end;

begin
  WriteLn('This works');
  Game.OnClockTick := @MyOnClockTickHandler;
end.

Maybe this might be one of the reasons https://wiki.freepascal.org/Integer

For 64-bit calculation, FPC does support 64-bit Int64 though, which may be defined as a value of -263 through 263-1, or -9 223 372 036 854 775 808 ... 9 223 372 036 854 775 807.
This will cause code expecting an integer and a pointer to be the same size to fail as a 64bit machine uses 64-bit pointers. To allow you to write portable code, the FPC system unit introduces the types PtrInt and PtrUInt which are signed and unsigned integer data types with the same size as a pointer.

Thanks for the links and minimal example. I consider this a high priority issue but seeing as I am not very familiar with ScriptCore it will likely take me a while to dig into this.

Updating pascalscript to latest revision, applying https://github.com/ccy/pascalscript/commit/1013b4f7deb549808f21312fa3ddd722170e6892 and commenting out {$define empty_methods_handler} in uPSRuntime.pas seems to fix the problem on win64 but on linux the problem still occurs since the patch is for windows only.

I was able to get this working on Linux 64 bit, at least on my machine (Arch Linux, FPC 3.2.2)... I checked the disassembly of  OnClockTickWriteHelper and it expects 3 parameters in registers (Self of ScriptGame in rdi, code pointer of callback in rsi, and data pointer of callback in rdx). However PascalScript is always passing the procedure pointer (called a btProcPtr internally) on the stack. So I replaced the StoreStack call in x64.inc with 2 StoreReg calls and it worked.
In the end what I had to do wasn't super complicated but it took me a very long time to figure it out... I'd also be surprised if this is all we need to do, so I'll make a draft PR in a little bit and we can test it.


Multiple reporting formats

cannot specify multiple output formats in format parameter
an alternative is to specify -f <format> multiple times in args, but this creates a weird redundant structure where formats are being specified in two separate arguments
with:
  format: JSON
  args: >
    -f HTML

at this point I'd suggest just getting rid of format argument altogether and let people specify it in args directly
For that matter, why not just have an args argument and get rid of the rest? It's just a wrapper to the cli that people can then call directly?

Hi
I think this was not possible but if you want to try and do a PR, that would be welcome


Get light-viz to work with client-server connection

Eventually we want to do client-server where they are not on the same machine...

If I understand the request right then that is possible to do following the multi-user setup guide

It is possible to deploy for multi-server setup. The current limitation is related to the fact that the dataset listing is expected to be on the machine that do the web delivery. This means, we can not start a remote pvserver if the dataset can not be accessed from the Web server machine.


removing duplicate download note - already appears at head of remote sec...

...tion, not needed in this as well

Assuming Norwegian translation will work through the beginner section, have updated the en & bi file of 'introduction' and amended images referring to #306


Issue 4009 preview UI

Fixes #{issue number here}
Changes proposed in this pull request:

Hi @jinouy2, it looks like this pull request intends to tackle #4009, which has already been solved by #4309, so I suggest we close this, unless you want to improve on the solution proposed in #4009?

Closing per the comment above.


Handoff table Indexes can't fully  describe the storage entry address in the data vault (lossy conversion)

The address of an entry stored in the data vault is a pair  (entry_type, entry_variant) , where entry type can be WarmResetEntry4, ColdResetEntry48,  The entry type is an abstraction of the register type (sticky/non-sticky/scratch)
The entry type information is lost when conversion entry_type::entry_variant to u8 is performed forcing FMC to make assumptions about the entry type  where the information is stored and defeating the purpose of a handoff table to decouple FMC and ROM.
Example :
rt_fw_load_addr_idx: WarmResetEntry4::RtLoadAddr.into(),
fmc_pub_key_x_dv_idx: ColdResetEntry48::FmcPubKeyX.into()
pub enum WarmResetEntry4 {
RtSvn = 0,
RtLoadAddr = 1,
RtEntryPoint = 2,
ManifestAddr = 3,
}
pub enum ColdResetEntry48 {
LDevDiceSigR = 0,
LDevDiceSigS = 1,
LDevDicePubKeyX = 2,
LDevDicePubKeyY = 3,
FmcDiceSigR = 4,
FmcDiceSigS = 5,
FmcPubKeyX = 6,
FmcPubKeyY = 7,
FmcTci = 8,
OwnerPubKeyHash = 9,
}
pub enum ColdResetEntry4 {
FmcSvn = 0,
FmcLoadAddr = 1,
FmcEntryPoint = 2,
VendorPubKeyIndex = 3,
}

@FerralCoder , @vsonims

@bluegate010 , let's chat about this on slacker.

I see a couple possible solutions:

Make every dv index in the FHT larger to hold the type and data information
Merge the dv variants into a single enum. Make the code that handles them understand what reset domain they're in.

I think 2 is reasonable.
also cc @mhatrevi for input, who wrote much of the data vault driver

Current implementation sizes indexes (storage address identifiers) as 8-bit.
As per discussion during the meeting,  we will increase the size of indexes (re-name them to handles) and have an abstraction layer StorageLocator that understands the encoding format of the data handle and can return the information to the requester.
pub struct DataHandle(u32);
pub const FHT_INVALID_DATA_HANDLE: DataHandle = DataHandle(u32::MAX);

@jhand2 , @mhatrevi , @FerralCoder  : ready for review.

closes issue: https://github.com/chipsalliance/caliptra-sw/issues/159


Publish to Hackage

Learn more about how they do versioning -> it is called PVP and is not exactly the same as semver.

Done!


Determine the target platform and configure Babel accordingly

While working on #29, I ran into Babel complaining about missing transformations a few times, for example when using async/await and when using class properties. Instead of blindly adding any, we should probably think about what exact browsers we target with our bundle, first. Maybe we can even remove some. :thinking:

Just to confirm my suspicion: Here is the part of the docs that states that by default, the Babel preset env (which we use) is configured basically for maximum compatibility:

Since one of the original goals of preset-env was to help users easily transition from using preset-latest, it behaves similarly when no targets are specified: preset-env will transform all ES2015-ES2020 code to be ES5 compatible.
We don't recommend using preset-env this way because it doesn't take advantage of its ability to target specific environments/versions.


ll::SDL_Quit() not being called at exit

Instead of letting users call sdl2::quit(), ll::SDL_Quit() should run automatically at application exit.
On init you should be able to do std::rt::at_exit(...) to call ll::SDL_Quit().
I've tried here but I could not get it working for some reason, I've tried to do like glfw-rs: https://github.com/bjz/glfw-rs/blob/master/src/lib.rs#L408
For example, all piston samples leak memory because they use sdl2_window and do not call sdl2::init anywhere. To leak at exit is not really a problem, but it is not nice.

Hmn. Perhaps the solution is to make sdl_init take a callback, so it automagically releases?


Updated README.md

Related Issue
#336

Closes: #336
Description of Changes

Added a menu bar in the table of contents
Fix the typo in the contributing section

Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

@anmode Have you checked my PR ? 😄

yeah! please could you wait for 1 or 2 days?? I want to add imp info in the readme.. that's why thinking to update this PR. Thanks for patience

yeah! please could you wait for 1 or 2 days?? I want to add imp info in the readme.. that's why thinking to update in this PR. Thanks for patience

Ok you can ask me to add anything in the readme 😄


add http-01 validation

Lets encrypt disable tls validation for new domains
https://community.letsencrypt.org/t/2018-01-11-update-regarding-acme-tls-sni-and-shared-hosting-infrastructure/50188
It need to add http-01 validation.

Fixed by 4a0338e8c2fd75bc6613bd2f968655ca3a27d0ff


Build a website for interacting with the dApp

This will depend on #6 at the very least, and on #7 for any non-trivial functionality

Done with #17


Test action button that just has a URL, not a da event

Don't throw an error. Not sure what the encoded name would be, though.

I don't know if this is the exact purpose of this issue, but I couldn't find a closer one. I'm trying to just press an action button, that has a specific url_ask passed in.
Without using the manual id_tag attribute of action_button_html there's no easy way to find a given button. Here's an example of an action_button_html gets you by default:
${ action_button_html(url_ask(
    [{'recompute': ['create_attorney', 'show_resp']}]),
    label="Create Attorney") }

<a data-linknum="346"
  data-embaction="eyJhY3Rpb24iOiAiX2RhX2ZvcmNlX2FzayIsICJhcmd1bWVudHMiOiB7InZhcmlhYmxlcyI6IFt7ImFjdGlvbiI6ICJfZGFfdW5kZWZpbmUiLCAiYXJndW1lbnRzIjogeyJ2YXJpYWJsZXMiOiBbImNyZWF0ZV9hdHRvcm5leSIsICJzaG93X3Jlc3AiXX19LCB7ImFjdGlvbiI6ICJfZGFfY29tcHV0ZSIsICJhcmd1bWVudHMiOiB7InZhcmlhYmxlcyI6IFsiY3JlYXRlX2F0dG9ybmV5IiwgInNob3dfcmVzcCJdfX1dfX0"
  href="/run/EFSPIntegration/admin_interview/?action=eyJhY3Rpb24iOiAiX2RhX2ZvcmNlX2FzayIsICJhcmd1bWVudHMiOiB7InZhcmlhYmxlcyI6IFt7ImFjdGlvbiI6ICJfZGFfdW5kZWZpbmUiLCAiYXJndW1lbnRzIjogeyJ2YXJpYWJsZXMiOiBbImNyZWF0ZV9hdHRvcm5leSIsICJzaG93X3Jlc3AiXX19LCB7ImFjdGlvbiI6ICJfZGFfY29tcHV0ZSIsICJhcmd1bWVudHMiOiB7InZhcmlhYmxlcyI6IFsiY3JlYXRlX2F0dG9ybmV5IiwgInNob3dfcmVzcCJdfX1dfX0"
  class="btn btn-sm btn-success btn-darevisit">
    Create attorney
</a>

Where the data-embaction is {\"action\": \"_da_force_ask\", \"arguments\": {\"variables\": [{\"action\": \"_da_undefine\", \"arguments\": {\"variables\": [\"create_attorney\", \"show_resp\"]}}, {\"action\": \"_da_compute\", \"arguments\": {\"variables\": [\"create_attorney\", \"show_resp\"]}}]}}" encoded into base64.
Related conversation here: https://cfb-public.slack.com/archives/CQ8RWCH17/p1648761462993049

This issue was actually a much simpler one about plain old urls for action buttons. This description is much more interesting and I'm alright sharing this issue for both of these if you are.
Easiest solution for us is to require folks to use an id_tag. Beyond that, I don't know all the possible configurations that action_button_html(url_ask()) can construct, but it's json and we could parse it. For example, the innermost property I see in the above is ["create_attorney", "show_resp"]. That seems like something that a user could provide, though we'd have to think about what that sentence would look like.


Add nameof() to thrown ArgumentNull exceptions so that errors specify which arguments are null

Added nameof() to connectionString and entityPath variables when throwing exceptions.
Previously if these were null or whitespace the error message would say: "System.ArgumentException: The argument is null or white space."
Now it will state which argument it is referring to.

Can one of the admins verify this patch?

Hey @thomhurst, sorry for the long response can you please rebase this PR and we'll get it merged.

/azp run

Test failures are unrelated.


Serverless does not support global distribution but it is not mentioned on the documentation

Customer that  configure serverless accounts are confused and create support tickets to ask support to enable Global Distribution for serverless accounts which Microsoft Support is unable to help since it is not supported but it is not mentioned anywhere in the documentation, we need to provide clarity for customers so they do not create serverless accounts if they are looking for global distribution features.

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

ID: e87da282-6ace-4f90-e894-b41c7e50649e
Version Independent ID: 7702ff20-f29d-321c-2516-92d448a6c333
Content: Distribute data globally with Azure Cosmos DB
Content Source: articles/cosmos-db/distribute-data-globally.md
Service: cosmos-db
GitHub Login: @seesharprun
Microsoft Alias: sidandrews

@hecepeda
Thanks for your feedback! We will investigate and update as appropriate.

Hello @hecepeda,
We have a change going live soon with a note directing users to the correct documentation page (<https://>) to see that serverless accounts can only run in a single region.
Thanks for your feedback!


[Enhancement] New Mechanic:  Damage Type

Similar to damage mechanic, but lets you set the damage types from "spigot damage causes" using a drop down list
Spigot Damage Cause: https://hub.spigotmc.org/javadocs/spigot/org/bukkit/event/entity/EntityDamageEvent.DamageCause.html

or instead of new mechanic just putting that inside the existing damage mechanic

Changing the damage cause used by the plugin isn't possible directly, but maybe it's possible to "simulate" a different damage cause. For that I need to know though, what's the purpose of changing the damage cause?

Prorpgitems elements group specific damage causes under the elemental defense. You can technically do a workaround on this by making a skillapi atrribute under the same name of the defense and set it to reduce specific damage classifier

This was done using a hacky solution, but works up to Java 17. Might have to keep an eye out for incopatibilities with certain Java versions, in which case the exception will be printed in console. But works for now :)


Initial refactoring of parfor reduction lowering

Summary:

Breakup long functions to highlight the logic structure
Replace manually constructed Numba IR with jitted python code.

In the long run, we want to move parfor lowering outside of the LoweringPass.

@DrTodd13 please could you take an initial look at this as codeowner? Many thanks! I'm very much in favour of these changes in principle as it reduces reliance on raw IR generation.


[$125] [Search v1] Search page: Show dates for prior year transactions

Problem
We're on the cusp of launching a v1 Search page, which means it's possible to see transactions from prior years. What I mean by this is that we are now in 2024, and I have a transaction I can find on the Search from back in 2017. The issue is that I have no idea looking at the transaction that it's from that far back.
Here's the New Expensify transaction.

Here's the Expensify Classic transaction

Solution
When looking at a prior year transaction, show the year, in addition to the month and date.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017440ce6c14aca464
Upwork Job ID:<PHONE_NUMBER>608614800
Last Price Increase: 2024-06-06

Issue OwnerCurrent Issue Owner: @dukenv0307

P.S. We're going to need web and mobile mocks from the design team. I reached out to them.

@Expensify/design would love your ideas on how to display this

I'm thinking it might be best to just make the column wider:

Because it doesn't look nearly as good if we stack like OldDot:

Mobile seems pretty straight forward:

Looks dope! I'll go ahead and take this so we can open up to community. cc @luacmartins

I agree we should just keep it simple and make the column wider.

@ShridharGoel why use 96 for the column width? In the mockup we're using 80.

Perfect, 80 is all you need so you should be good there.

@mountiny can you keep this moving? We might be able to get it today. :)

Cool taking over, I think Carlos will have enough of other tasks

PR is up/in review

@ShridharGoel I'm not sure that I understand how the scope increased. We still just added the year to the date column if there are transactions from previous years which was the OP, no?

I think the OP was to add the year, which is where the logic change came from and definitely in scope. I think we would just keep the existing bounty as is.

This was a simple change, so I think the current bounty of $125 is applicable here.

PR was deployed to prod on the 17th, so the payment due date is actually the 24th. cc @greg-schroeder

Oh. Okay. I'll process this now then

@ShridharGoel please accept the offer in Upwork so I can pay you, thanks!

bump @ShridharGoel

Bump @ShridharGoel - moving this to Monthly given no response.

@greg-schroeder seems like the offer was accepted. Can we close this now?

Yes it does appear it was finally accepted! I'll close it after I confirm payment!

Paid


Add Link to logo

On the homepage, add a link to the logo so that whenever someone clicks on it, the page refreshes and they return to the same homepage.

It can simply be done using logo.img

@AryanNanda17 Looks like a good first issue, and I'll need to get the environment setup. Mind if I take this one?

Hello @fudgemasterultra,
Go ahead with it!

@fudgemasterultra, also, it would be really great and motivating for our team if you could star this repo so that this project reaches more people. Thanks again for your interest. I am assigning this to you

@AryanNanda17 Do you want it to refresh the page on clicking or redirect the user back to the / route?
Right now it looks like we use the Link element and have that guide the user to the / route, and the reason it isn't refreshing the page is because of the way the Link property works. We can change it to an anchor element, but there is a bit of a jarring white screen then back to the app.
Also I have stared the app. :)

@fudgemasterultra we want it to redirect the user back to / route on clicking the logo

@ParamThakkar123 This has been fixed in PR#56

Okay, @AryanNanda17 just take a look. Things are working as expected so maybe we can merge this after you approve these changes.


Add parSequence and parTraverse combinators to Result and Output

Current implementation of sequence/traverse functions is sequential which can very easily lead to accidental serialisation of operations. Given that resource constructors can be painfully slow in infra it should be possible for the user to decide whether their side effects have to be executed in given order or in parallel. We could just make sequence and traverse parallel by default (I think that's how Output.allOf works in pulumi-java and All/Output.all combinators in other SDKs  work given that all of them are strict!) but that could be misleading, surprising and not really helpful for people used to working with lazy monads in Scala.

Sounds good. For the record, you are right, Java uses
CompletableFuture.allOf
like this
public static <T> CompletableFuture<List<T>> allOf(Collection<CompletableFuture<T>> futures) {
        return CompletableFuture
                .allOf(futures.toArray(new CompletableFuture[futures.size()]))
                .thenApply(unused -> futures.stream()
                        .filter(ignoreNullValues())
                        .map(CompletableFuture::join) // join() is not blocking here, by the time this function is called, the future is guaranteed to be complete
                        .collect(Collectors.toList())
                );
    }

This will wait for all of the futures without any ordering. So it can be parallel in theory.

In java it is parallel by default because CompletableFuture is not lazy :)
Implemented and done.


Chapter 2 - hello-buffer.py - Failed to load BPF program b'hello': Invalid argument

Hello
I run the code from the ch2, but something went wrong.
Here is the ERROR generated by python:

bpf: Failed to load program: Invalid argument
last insn is not an exit or jmp
processed 0 insns (limit 1000000) max_states_per_insn 0 total_states 0 peak_states 0 mark_read 0

Traceback (most recent call last):
File "/home/zrj/learning-eBPF/hello_buffer.py", line 32, in
b.attach_kprobe(event=syscall, fn_name="hello")
File "/usr/lib/python3.6/site-packages/bcc/init.py", line 646, in attach_kprobe
fn = self.load_func(fn_name, BPF.KPROBE)
File "/usr/lib/python3.6/site-packages/bcc/init.py", line 384, in load_func
(func_name, errstr))
Exception: Failed to load BPF program b'hello': Invalid argument

but it works when I delete bpf_probe_read_kernel(&data.message, sizeof(data.message), message);
I'm new in eBPF, and I want to figure out what's wrong with this line
thanks a lot!
Here is my kernel version

Linux 6.6.1-1.el7.elrepo.x86_64

Did you download the code or typed it out (maybe you made a typo?)?
I didn't have any problems running it on 6.6.1 (openSUSE Tumbleweed), so maybe a distro specific bug?


typeahead: Replace no-break space (U+00A0) in query with `space (U+…

…0020)`.
Fixes #10039.
Typing "tim " did not did not produce any match when suggesting person
in composebox typeahead or user group typeahead as the space at the
end of the "tim " string passed by the browser was a
no break-space (U+00A0)  instead of space (U+0020).
Although there are unicode characters other than no break-space which
represent spaces, only U+00A0 is replaced as it was the only space
character encountered when testing this issue manually.
One of the follow up issue after this would be to use composebox_typeahead.query_matches_person in user group settings typeahead after extracting query_matches_* functions to somewhere appropriate (I'm not sure about what location is appropriate atm, would put some more thought into it.).

Nice debugging!  Merged, after modifying the comment to mention contenteditable inputs, since I think that's what causes the browser to produce this.  Thanks @shubham-padia!


AssemblyLoadContext.Default.LoadFromAssemblyPath doesn't load assembly with dot in name

Doesn't work:
AssemblyLoadContext.Default.LoadFromAssemblyPath(@"C:\X.Y.dll");
Works:
AssemblyLoadContext.Default.LoadFromAssemblyPath(@"C:\XY.dll");

Please file this at https://github.com/dotnet/coreclr/issues.


[EngSys] upgrade dev dependency @types/sinon to ^17.0.0

and sinon too to ^17.0.0 for packages that missed last upgrade.

API change check
API changes are not detected in this pull request.

same js - cosmosdb - ci failure since 10/30 https://dev.azure.com/azure-sdk/public/_build/results?buildId=3216277&view=results

/check-enforcer override


Prettier way of loading data.values

Right now in my starlark program I do:
load("@ytt:data", "data")

top = data.values

Is there a way to do this in load? Like:
load("@ytt:data", top = data.values)

If I really want to keep it terse (in practice, less often than I originally thought), I'll sneak in a semi-colon and make the assignment:
load("@ytt:data", "data"); d=data.values

The load() statement allows you to name values that are exported from the module and optionally manage the local identifier to bind to; but not whole expressions.
For more: https://github.com/google/starlark-go/blob/master/doc/spec.md#load-statements

Thanks!


Improve workflow for development and releases

From my point of view, the current workflow is not suitable for efficient development work. Especially the missing snapshots for the Java packages as well as for Docker images are problematic to me. I suggest that every merge to master deploys new development versions of the artifacts. And I suggest that we use dedicated release commits to build releases. With these changes applied, the current two workflows can probably be unified.
As it is not usual to create release commits in a PR, and as the direct push to the main branch is forbidden, I suggest to create a "create release" pipeline that takes two parameters, the release version and the new dev (snapshot) version. This pipeline could than use the Maven release plugin to create the required commits and tags and push them directly to the main branch.

In a meeting, we decided to use canary releases as a solution to create Docker images for snapshots. The other ideas are currently discarded.


Lost access to Storage Manager on DSM 7.2-64561 (DS923+)

Hi,
I have ran the syno_hdd_db.sh script on my DS923+ and after reboot I still couldn't create an M2 storage pool. I have then run the script again to restore the previous settings before trying anything else. After another reboot the Storage Manager was gone.
This seems similar to #73 but on the final version of 7.2.
The model is missing within the Info Center: https://imgur.com/a/B8SV4WU
I run get_syno_model.sh and this is the output:
unamee:
synology_r1000_923+

/etc/synoinfo.conf unique:
synology_r1000_923+

/etc.defaults/synoinfo.conf unique:
synology_r1000_923+

/etc/synoinfo.conf upnpmodelname:
DS923+

/etc.defaults/synoinfo.conf upnpmodelname:
DS923+

/proc/cmdline syno_hw_version:
DS923+

/rpco/sys/kernel syno_hw_version:
DS923+

I have also checked to see if the config file was corrupt in some way, but this is the only thing diff could see:
max-admin@SynologyNAS:~$ sudo diff syno_conf_after_first_attempt.txt /etc.defaults/synoinfo.conf
Password:
204c204
< support_memory_compatibility="no"
---
> support_memory_compatibility="yes"
328a329
>

I have tried the "Restore DSM configurations" from the "Update & Restore" menu but that didn't help...
Do you have any pointers on how to fix my config?
Thank you

As an additional note, these are the times I have ran the script.
If I remember correctly, after the restore and the reboot, the Storage Manager was still there.
The 3rd run was to include the "-n" since I thought the first run might have not worked because of that. I hope this is helpful.

sudo ./syno_hdd_db.sh --ram --showedits

Synology_HDD_db v2.2.47
DS923+ DSM 7.2-64561
Using options: --ram --showedits

HDD/SSD models found: 3
ST4000VN000-1H4168,SC44
ST8000VN004-2M2101,SC60
ST8000VN004-3CP101,SC60

M.2 drive models found: 1
CT1000P3PSSD8,P9CR40A

No M.2 cards found

No Expansion Units found

Backed up ds923+_host_v7.db.new
Added ST4000VN000-1H4168 to ds923+_host_v7.db
Added ST4000VN000-1H4168 to ds923+_host_v7.db.new
Added ST8000VN004-2M2101 to ds923+_host_v7.db
Added ST8000VN004-2M2101 to ds923+_host_v7.db.new
ST8000VN004-3CP101 already exists in ds923+_host_v7.db
ST8000VN004-3CP101 already exists in ds923+_host_v7.db.new
Added CT1000P3PSSD8 to ds923+_host_v7.db
Added CT1000P3PSSD8 to ds923+_host_v7.db.new

Backed up synoinfo.conf

Support disk compatibility already enabled.

Disabled support memory compatibility.

Max memory already set to 32 GB.

M.2 volume support already enabled.

Drive db auto updates already enabled.

    "ST4000VN000-1H4168": {
      "SC44": {
        "compatibility_interval": [
          {
            "compatibility": "support",
            "not_yet_rolling_status": "support",
            "fw_dsm_update_status_notify": false,
            "barebone_installable": true
          }
        ]
      },
      "default": {

    "ST8000VN004-2M2101": {
      "SC60": {
        "compatibility_interval": [
          {
            "compatibility": "support",
            "not_yet_rolling_status": "support",
            "fw_dsm_update_status_notify": false,
            "barebone_installable": true
          }
        ]
      },
      "default": {

    "ST8000VN004-3CP101": {
      "SC60": {
        "compatibility_interval": [
          {
            "compatibility": "support",
            "not_yet_rolling_status": "support",
            "fw_dsm_update_status_notify": false,
            "barebone_installable": true,
            "smart_test_ignore": false,
            "smart_attr_ignore": false
          }
        ]

    "CT1000P3PSSD8": {
      "P9CR40A": {
        "compatibility_interval": [
          {
            "compatibility": "support",
            "not_yet_rolling_status": "support",
            "fw_dsm_update_status_notify": false,
            "barebone_installable": true
          }
        ]
      },
      "default": {

DSM successfully checked disk compatibility.

You may need to reboot the Synology to see the changes.

sudo ./syno_hdd_db.sh --restore

Synology_HDD_db v2.2.47
DS923+ DSM 7.2-64561
Using options: --restore

Restored synoinfo.conf
Restored rx418_v7.db
Restored rx415_v7.db
Restored rx1214_v7.db
Restored ds923+_host_v7.db
Restored dx513_v7.db
Restored rx1217_v7.db
Restored dx1215ii_v7.db
Restored ds923+_host_v7.db.new
Restored rx1214rp_v7.db
Restored rx1216sas_v7.db
Restored rx1211rp_v7.db
Restored rx1211_v7.db
Restored dx517_v7.db
Restored dx1211_v7.db
Restored dx1215_v7.db
Restored rx1213sas_v7.db
Restored rx1217sas_v7.db
Restored rxd1219sas_v7.db
Restored rx1217rp_v7.db
Restored rx6022sas_v7.db
Restored dx213_v7.db

Restore successful.

sudo ./syno_hdd_db.sh -nr --showedits > 1_log.txt
output:

Backed up ds923+_host_v7.db
Backed up ds923+_host_v7.db.new
ST8000VN004-3CP101 already exists in ds923+_host_v7.db
ST8000VN004-3CP101 already exists in ds923+_host_v7.db.new

Backed up synoinfo.conf

log:
Synology_HDD_db v2.2.47
DS923+ DSM 7.2-64561
Using options: -nr --showedits

HDD/SSD models found: 3
ST4000VN000-1H4168,SC44
ST8000VN004-2M2101,SC60
ST8000VN004-3CP101,SC60

M.2 drive models found: 1
CT1000P3PSSD8,P9CR40A

No M.2 cards found

No Expansion Units found

Added ST4000VN000-1H4168 to ds923+_host_v7.db
Added ST4000VN000-1H4168 to ds923+_host_v7.db.new
Added ST8000VN004-2M2101 to ds923+_host_v7.db
Added ST8000VN004-2M2101 to ds923+_host_v7.db.new
Added CT1000P3PSSD8 to ds923+_host_v7.db
Added CT1000P3PSSD8 to ds923+_host_v7.db.new

Support disk compatibility already enabled.

Disabled support memory compatibility.

Max memory already set to 32 GB.

M.2 volume support already enabled.

Disabled drive db auto updates.

    "ST4000VN000-1H4168": {
      "SC44": {
        "compatibility_interval": [
          {
            "compatibility": "support",
            "not_yet_rolling_status": "support",
            "fw_dsm_update_status_notify": false,
            "barebone_installable": true
          }
        ]
      },
      "default": {

    "ST8000VN004-2M2101": {
      "SC60": {
        "compatibility_interval": [
          {
            "compatibility": "support",
            "not_yet_rolling_status": "support",
            "fw_dsm_update_status_notify": false,
            "barebone_installable": true
          }
        ]
      },
      "default": {

    "ST8000VN004-3CP101": {
      "SC60": {
        "compatibility_interval": [
          {
            "compatibility": "support",
            "not_yet_rolling_status": "support",
            "fw_dsm_update_status_notify": false,
            "barebone_installable": true,
            "smart_test_ignore": false,
            "smart_attr_ignore": false
          }
        ]

    "CT1000P3PSSD8": {
      "P9CR40A": {
        "compatibility_interval": [
          {
            "compatibility": "support",
            "not_yet_rolling_status": "support",
            "fw_dsm_update_status_notify": false,
            "barebone_installable": true
          }
        ]
      },
      "default": {

DSM successfully checked disk compatibility.

You may need to reboot the Synology to see the changes.

I notice that the ST8000VN004-3CP101 which was already in the .db files contains 2 new lines that weren't in previous DSM version's db files:
            "smart_test_ignore": false,
            "smart_attr_ignore": false

I wonder if the other drives not having those 2 lines is a problem.
There is 1 change that --restore does not undo:
echo 1 > /run/synostorage/disks/nvme0n1/m2_pool_support
You could try the following:
echo 0 > /run/synostorage/disks/nvme0n1/m2_pool_support
I've downloaded the DSM_DS923+_64561.pat file, unpacked it and attached the default synoinfo.conf and disk compatability db files for you - hoping they'll get your Synology working again.
synoinfo.zip
DiskCompatibilityDB.zip

I'll be able to try the suggested steps in a few hours. In the meantime I'm backing up some important data just in case.
Thank you!

Quick update, I have decided to reset the NAS and reinstall DSM.
I have backed up the settings locally (but they were backed up also on the cloud) and restored them right after the reset. Everything seems to be looking fine 🤞, and I didn't lose any data.
One thing I forgot to mention previously is that I was on 7.1 and I have [manually] updated to 7.2.
This time I have installed 7.2 directly.
Thanks for the support!

OK, wrapping this up.
I guess I didn't really understand how the script[s] worked...
After the reset, I have run the Synology_HDD_db again, restarted and the ran Synology_M2_volume one.
I have now a working volume on the two m2 drives 🎉
I guess the first time I didn't understand what the 1st script was supposed to do and so I reverted the operation. The "restore" operation must have broken something, but I guess the script did what it was supposed to do...
Thanks again, I bought you a beer 😉

The "restore" operation might have caused problems, but on a second run the script[s] worked without any problem.

The "restore" operation might have caused problem

Thank you for pointing that out.
I just tested the restore option on a DS720+ running 7.2 and it had the same issues as you. I found a couple of likely causes and edited them by hand. After that I rebooted and everything was back to normal.
I'll have an updated version of the script online soon (after some more testing).

I'm happy to hear that reporting the issue is helping. Thanks again for the work you do


python310Packages.doctest-ignore-unicode: drop

Description of changes
part of https://github.com/NixOS/nixpkgs/issues/311054
Has not had a new release for 11 years.

Things done

Built on platform(s)

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

[ ] 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
[ ] 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 eval


feat: bump spec version to 2.3.11

Checkmarx One – Scan Summary & Details – 0501d4ac-81b3-45bb-93c2-976b2f37c818
No New Or Fixed Issues Found


For image capability, need to extract from manifest what imagesize is, if its RGB BGR YUV or Grayscale

Need these parameters to reformat in app.
Needs to be send from
manifest -> SDK -> App
Links to https://github.com/hotg-ai/rune_vm/issues/3

According to image-rs/image#1217 the image crate doesn't support YUV images, but we can convert between everything else.


[BugBash]No max limit for hub in hub settings

Describe the bug
No max limit for hub in hub setting
To Reproduce

open settings blade
add any number of hub setting

Exceptions (if any)

the max number of hub should be fixed

Further technical details

SDK and its version you use

https://msazure.visualstudio.com/One/_git/OSSServices-SignalR-PortalExtension/pullrequest/5288762


heap-buffer-overflow in lit_read_code_unit_from_hex

the following poc can trigger a heap buffer overflow bugs.
 I've tested on gcc 5.4 and 4.8 and couldn't reproduce the issue on the latest master.


add cl_downloadfilter mapsonly to comp/maxperf presets

Could you add documentation like
// Only downloads maps from server custom files


Fix parsing of email gateway token string to support missed message format

As per discussion with Tim, fixing the reply-to-PM message format before moving on to stripping out quoted text.

Automated message from Dropbox CLA bot
@karamcnair, it looks like you've already signed the Dropbox CLA. Thanks!

Updated to remove accidental whitespace/newline changes (and shorten commit line)

Merged, thanks!


request: replace node-fetch for isomorphic-fetch

Node fetch was a problem when targeting for web in webpack. Changing to isomorphic-fetch fixed the issue. Even though this fix is working, we could refactor the request.js to use node built-in http.

@derekstavis I think your gitcop is somehow screwing travis

@MarcoWorms you could also use the same body you used in pull request instead of using a dot... It makes a lot more sense.

@derekstavis let's :shipit:?

@MarcoWorms I'd like to release the next version fixing the bundle size, so I'd prefer implementing request.js using native http package.


Add tags to images in pipelines

Checklist:

[x] Read the Code of Conduct and Contributing Guidelines.

[x] Followed the commit message guidelines.

Updates to the collections

Related Issues:

Same changes that went into master.


chore: update packages to latest | chakra-ui v2

📝 Changed routes:

/about
/indonesia

(view full size)
(view full size)

Commit d302e96ace444f4f54456c154e492e9c4ad35292 (https://covid-19-data-lo1a38ixd-sozonome.vercel.app).


Implement TryFrom

This fixes a TODO I left in from when it was originally accepted. I've deprecated from_external_form "from 2.0.1", but that's a bit speculative.

Thank you

Thank you


Improve editing of min/max particle properties

Adds this fancy widget for particles:
https://github.com/godotengine/godot/assets/2223172/fff400f3-bf86-42b5-9045-a01e11c95a13
It can edit min and max as part of the one slider, including dragging them together and scaling. Old sliders are still provided for more precise values and values outside the default range. There is also a new slider mode where you can edit pivot and deviation instead of min and max.
This required some internal changes to ParticleProcessMaterial, namely the min/max properties have now third property, which is a vector of both min and max. This was needed to add inspector plugin. The linked properties are editor only, while the old properties are code-only and stored in scene. I added a convenience macro to make binding min/max properties easier.
Closes https://github.com/godotengine/godot-proposals/issues/7583
Closes https://github.com/godotengine/godot-proposals/issues/4315
Builds on top of #79527, so putting as draft until that PR is merged.
EDIT:
Here's new style:
https://user-images.githubusercontent.com/2223172/284101133-14c016db-1bf0-43d0-80c8-7675d2190898.mp4

Looks great!
I wonder if a modifier could be added (such as Shift) to change deviation when dragging the min/max icon on the slider. This way, you can change deviation without having to unfold the inspector.

It's already implemented, you can see it in 0:12. Needs to be more obvious though.

This is a downgrade for me personally, as I usually use the line edits to write my values.
I suggest we change this:

I think instead of numbers, we should have an EditorSpinSlider that can be edited.
This means that this could be simplified:

Instead, the arrow could popup a context menu with the two options - Range, Pivot - to change which values are shown. This would save A LOT of vertical space.
What if you want to see min/max easily while editing pivot mode, for example? IMO, this should be a tooltip while you're hovering the slider.
Speaking of hovering, the slider needs hovering and dragging feedback. In general, I suggest making the colors similar to scrollbars in the editor.
I think the grabbers to the side of the slider are way too big and could be confused for part of the range. In my opinion, shrink their width, and expand the click area only programmatically. Make them a different darker color, maybe.

I'm back to working on this PR. Pushed a fix for EditorProperty.
@MewPurPur How do you imagine replacing the labels with EditorSpinSliders? There is enough space only to display min/max, with pivot/deviation it would get extremely cramped. Unless the sliders are displayed in 2 rows, which would mean that the editor would take 3 rows by default (currently it takes two). I wanted it to take as small space as possible.

We can also use icons, it would also help us not worry about localizations. If not, then I don't see much harm in doing µ and σ for pivot and deviation, or even just displaying some kind of icon to make it clear what each is.
Ugly mock-up:

Then I'd need icons for min, max, pivot and deviation. Not sure if µ and σ would be recognizable.
As for your mock-up, I don't think it's possible with EditorSpinSlider, unless I add a Label that would partially overlap it 🤔

If you think this is a good direction to pursue, just ping me to make the icons

I'll go with piv and dev for the other mode. The advantage of labels is that EditorSpinSlider has them built-in and they disappear while editing, so it should be fine.

Looks fantastic IMO! The next thing I think needs to be done in terms of GUI is to:

Make the handles only appear when you're hovering over the widget.
Add some kind of clear indication of where the pivot is.
Make the color more gray-ish generally, lighter gray when you're hovering, bluish while you're dragging. This also holds for the handles. When you're pressing Shift, highlight both handles (as they are what the operation would affect).

Ugly mock-up with the right handle being hovered:

Pushed the new layout (still need to apply other changes).

Finished.
https://github.com/godotengine/godot/assets/2223172/f5b86ad6-51cd-490a-ba50-da4ae2a8b932

Looks excellent now IMO. Probably my last UI suggestions before I test and review:

Looks like it's drawn a little too high, it pokes out of the dark box.
Left arrow shouldn't be drawn when min can't be reduced further, likewise for the right arrow.
Pivot should be drawn always (!), Otherwise you can't tell where the value range is if there's no deviation.
I like the outline's hover indication, but it should be subtler as it's not interactable.

Pivot should be drawn always (!), Otherwise you can't tell where the value range is if there's no deviation.

It is drawn always, when slider width is > 4. Otherwise there is no space (and sense).

You mean too low?

It is drawn always, when slider width is > 4. Otherwise there is no space (and sense).

From a usability standpoint, I think there's much to gain from drawing it, even if the there's no deviation. It would allow you to always drag the range and to see where it is. Otherwise, you'll have to hover the widget before you can see it.

Ah right, I was referring to this:

No idea what causes it.

From a usability standpoint, I think there's much to gain from drawing it, even if the there's no deviation.

Ok I misunderstood.

Ooh I forgot about that... My bad.

When you press Shift, IMO the hover highlight should immediately switch to the two arrows.

That's not possible, because gui_input does not receive key events unless the Control is focused (and in this case it isn't). Gradient editor does not immediately snap on Control press for the same reason.

FINAL FINAL v2
https://github.com/godotengine/godot/assets/2223172/14c016db-1bf0-43d0-80c8-7675d2190898

Needs tweaks for light theme. The labels will need to be darker, the pivot should be near-black, the grays darker, hovering should make them even darker. The new svg's are color-remapped automatically so they are fine.

The labels use vector colors:

The labels use vector colors:

Oof. Okay, this should be fixed on its own then.

Something like this?
https://github.com/godotengine/godot/assets/2223172/3732d63d-92d0-4af1-86e4-60c9f99f2b71
I also noticed that arrows are not modulated, because they are black 🤦♂

Some properties display with -0 by default, such as linear acceleration in ParticleProcessMaterial:

No idea why this would happen.

An empty inspector field displays on the right of each value that has a min/max slider

I use invisible Ranges to hold values and EditorProperty seems to place them like normal controls. Fixed.

Holding down Ctrl could snap the pivot/min/max values to integer values.

You mean while dragging the custom slider? The bottom spin sliders already support this.

In terms of visual design, I think the slider's middle area should be a bit darker.

Done.

Initial velocity property's range could be extended to negative values

That's unrelated to this PR.

I really like this UI!
but
We need to find solutions to handle gracefully values that are outside of the range for properties that are specifically marked "or greater" and "or lesser". Right now it's just buggy:
https://github.com/godotengine/godot/assets/7917475/884419a1-39ce-438f-9882-f3b9dc4a9c11
Having your values changed if you touch the bar is not ideal.

I know values outside range are broken, but I don't know how to improve them .-.
Though it somewhat reminds me of this slider issue: #34583 Maybe it should behave like sliders in GIMP 🤔That means you could drag from outside range without instant jumping, but only that (the "slider" would still be constrained).

Let me get back to you on this, I think there's a way to make it okay. Perhaps it could base the range on the min/max registered range: so if when you have the scene open you go below min, then it will remember it for as long as it's open? but im not sure when the inspectors get re-instantiated.

Ok I did this:
https://github.com/godotengine/godot/assets/2223172/c338e77f-7055-4e35-b68b-e3d6976004f7
The custom editor's min and max is determined by both property's min/max and current value. If the value is outside range, the min/max is adjusted to allow editing within the new range. The range is controlled using the sliders.

Nice!!

Tested the PR, feels good to use now. I think that piv and dev could use a bit more explicit and layman names, i dont have a precise suggestion right now. Let me come back to you on this

Regarding input - when there is no random it's quite challenging to grab the cursor at the center. @KoBeWi do you think it's possible to make it easier? I think one of the biggest perk of this PR would be an easy way to move min and max together also when min == max, but right now it's a bit difficult

@KoBeWi I have interrogated the hive mind and we reached some consensus on value + range

Is value correct though? It's not exact value, but the middle point of the range.

I think it makes sense in the context of value + range. Someone also proposed base instead of value. I'm trying to find something that conveys the meaning but it's also easier to understand for non-math people

do you think it's possible to make it easier?

It's possible, but not sure how. There is very little space between arrows. I could maybe add a modifier for dragging both min and max? (e.g. Ctrl)

Ah! looks good! Maybe grabbing outside of the range could already cause the central pivot to move without changin the interval, without holding down control or shift?

@QbieShay, @TheRensei and I discussed the naming of the parameters again and settled with value and the plus-minus sign (U"\u00B1"). What do you think?

Disclaimer: The following wasn't part of the discussion
To make it clearer you could call val base value and +/- maybe spread in the tooltips?

Maybe grabbing outside of the range could already cause the central pivot to move without changin the interval, without holding down control or shift?

Something like that?
https://github.com/godotengine/godot/assets/2223172/77b8adbf-1aec-4c51-8e4a-788ea528bfb8
I removed the modifier. Now clicking anything outside arrows will move the center.

I tested again and i had concerns about the backwards compatibility with animation player possibly animating min-max properties, but im happy to report it works now (it's also not a common usecase as far as i know).
Not necessary for this PR, but i think that the shift modifier for making smaller change would help also this feature :+1:

Going below limit with avg +- doesn't work, there's some quirks there. also the slider of the +- is being funny when moved :thinking:

Going below limit with avg +- doesn't work

Not sure what do you mean by that.

also the slider of the +- is being funny when moved

That's because the max value adjusts to be within the range.

Sorry - I shoudl articulate better.
In the avg mode (not minmax), i think the sliders are clamped to the min or max, even if the property specifies or_greater or or_lesser. it works in minmax mode.
I don't think it's a blocking issue anyway.

Ok I made lesser/greater allowed in the second mode. However the max value of ± can't be greater than max - min of the property. Also ± only allows "or greater" when both min and max can go outside range.

just tested, NICE :ok_hand:
I can't approve again but i APPROVE

Thanks! Amazing usability improvement for particles 🌟

This is really fantastic to use!! What a great UI improvement! Thanks KoBeWi and thanks also to everyone commenting with improvement suggestions, especially MewPurPur!
I would really love to use this slider (in countless places really) in my projects. Both for exported values in the Inspector as well as Control node.
I tried to figure out if it is already available, but it seems to be it is not. Would it make sense to open a proposal?

I think this could be exposed using a special property hint. The only problem is that right now it's basically hard-coded into the particle material, so the new editor needs some internal rework. It's not very difficult to replicate in GDScript though, I did a prototype before opening the PR.
But yeah, opening a proposal wouldn't hurt.


Change preview attribute to experimental.

Fixes #2918
Also changes the overloads with callbacks exposing CDK types to not be optional. In all cases we have an overload that does not expose the callback so we don't need it to be optional.

/azp run

Can you do this too while you're here https://github.com/dotnet/aspire/issues/2952


ILR data, minor fix for CS/Info

Summary

[x] Updated data for ILR school
[x] Updated Info major
[x] Updated CS major

Remaining TODOs:

[ ] need to break down one req for ILR in next PR (left a TODO comment)

Test Plan

Test ILR school reqs
Test Info/CS reqs (in particular, there are no longer any "self checks" for CS")
Nothing should break :)

close and reopen to retrigger WIP job


save mixins alongside declarations for symbol alternates

Bug/issue #, if applicable: rdar://133767750
Summary
The "alternate declarations" mechanism added in https://github.com/swiftlang/swift-docc-symbolkit/pull/60 only preserves the declaration fragments of an alternate symbol, but this can hide other information, for example the related functionSignature mixin that is now used by Swift-DocC to lint documentation. This PR updates the "alternate declarations" functionality to save all of a symbol's mixins and its doc comment when merging alternate symbols together.
Dependencies
None
Testing
This is a purely API change, so coverage is handled with automated tests. A forthcoming PR on Swift-DocC will use the new API.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

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

@swift-ci Please test

@swift-ci Please test


Dark font on dark background in queue view

OS: Arch Linux kernel 4.4.4
Issue Descriptions: Dark theme results in unreadable queue
Steps to Reproduce: Play music using the dark theme. Click the queue button in the bottom right - the song titles are barely readable as it's black text on a dark gray background.

Sorry, check #443


📃: (GenAI Beginners)Build a Generatively Pretrained Transformer (GPT) referring to "Attention is All You Need", OpenAI GPT-2, and Shakespeare Dataset

:red_circle: Title : Build a Generatively Pretrained Transformer (GPT) referring to "Attention is All You Need", OpenAI GPT-2, and Shakespeare Dataset
:red_circle: Aim : Generating the next word based on the context of the previous ones.
:red_circle: Brief Explanation : Make use of Self Attention mechanisms to process and generate text efficiently, without relying on recurrent neural networks. You have to refer to "Attention is All You Need" paper and learn about GPT models, like OpenAI's GPT-2, which are pretrained on large text corpora, learning to predict the next word in a sentence, which allows them to generate coherent and contextually relevant text. You can start with Shakespeare's Dataset, the model learns the patterns specific to the playwright’s language, enabling it to generate text in a similar style. This combination of large-scale pretraining and fine-tuning on domain-specific data makes GPT highly effective for creative text generation tasks. This will be your first step towards Generative AI.
Here are the links for your help:
Attention is All You Need: https://arxiv.org/abs/1706.03762
Shakespeare Plays Dataset: https://www.kaggle.com/datasets/kingburrito666/shakespeare-plays
For any issues you can reach me out via email<EMAIL_ADDRESS>Screenshots 📷

:white_check_mark: To be Mentioned while taking the issue :

Full name : Suraj Sahu
What is your participant role? (Mention the Open Source Program name. Eg. GSOC, GSSOC, SSOC, JWOC, etc.)
GSSOC'24 Extended and Hacktoberfest Mentor

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

@shaansuraj Hey! I found this interesting. Can I work on this issue? Could you please assign this to me.

Hey I found this issue interesting. Could you assign this issue to me.

@Abhicoder03 I am glad that you found this topic interesting. I have assigned it to you. If you have any doubts, you can reach me out through the email I provided above.

Hi @Abhicoder03
I’m interested in contributing to the Build a Generatively Pretrained Transformer (GPT) project. I plan to focus on:
Understanding Self-Attention Mechanisms, I will thoroughly study the "Attention is All You Need" paper and the architecture of OpenAI's GPT-2 to grasp how self-attention enables efficient text generation.
I will work with the Shakespeare Dataset to learn and emulate the unique language patterns of the playwright, enhancing the model's text generation capabilities.
I will implement the GPT model, ensuring it generates coherent and contextually relevant text based on the context of previous words.

@JahnaviDhanaSri Okay sure, I will assign you to this issue as well. You can work collaboratively with @Abhicoder03 on this.

Thank You!
On Wed, 2 Oct 2024 at 22:17, Suraj Sahu @.***> wrote:

Assigned #31 https://github.com/UTSAVS26/PyVerse/issues/31 to
@JahnaviDhanaSri https://github.com/JahnaviDhanaSri.
—
Reply to this email directly, view it on GitHub
https://github.com/UTSAVS26/PyVerse/issues/31#event-14492075246, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BCKE3GS7THMNH7UBNIVWWQ3ZZQPTNAVCNFSM6AAAAABPHWSRGCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGQ4TEMBXGUZDINQ
.
You are receiving this because you were assigned.Message ID:
@.***>

@JahnaviDhanaSri Hey, we can collaborate on this project.

Sure!
On Wed, 2 Oct 2024 at 22:58, Abhishek Choudhary @.***>
wrote:

@JahnaviDhanaSri https://github.com/JahnaviDhanaSri Hey, we can
collaborate on this project.
—
Reply to this email directly, view it on GitHub
https://github.com/UTSAVS26/PyVerse/issues/31#issuecomment-2389225786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCKE3GVMJD6PTCYDNRKDBQTZZQUNNAVCNFSM6AAAAABPHWSRGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBZGIZDKNZYGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


add new learning resource drawer layout

What are the relevant tickets?
Closes https://github.com/mitodl/hq/issues/5819
Description (What does it do?)
This PR is a first pass at implementing the new wider learning resource drawer design. The structure and styles have been altered to match what's included in the design, minus the out of scope items listed in the issue. As part of this, the dropdown UI for selecting runs that changes the information section has been removed. Instead, all possible values across all runs are shown in a pipe separated list.
Screenshots (if appropriate):

How can this be tested?

Spin up mit-learn on this branch
Ensure that your local database is sufficiently populated with enough learning resources for testing:

A variety of courses
Podcasts
Videos

Ensure that you have built a search index
Visit the search page at http://localhost:8062/search
Search for and click on learning resources to bring up the new drawer
Check all of the above resource type and ensure that the general structure and information presented matches what's expected and aligns with the provided designs

Note to self: fix alignment of bookmark buttons when "on platform" section is not present.

@abeglova Thanks for your review, this is ready for another look. The mobile video issue actually had to do with the description on that particular video causing the description field to overflow the flex parent because of the white-space: pre-line rule. I changed this to use white-space: pre-wrap and word-break: break-word to fix this. Really the issue was caused by a really long unbroken line of text that was a URL. In reality, URLs like this should be actual links but I don't think we're set up for that yet. Video playlists should now have a proper CTA as well as thumbnail image. Learning paths we will leave as-is, as we discussed.

@gumaerc I checked out the posthog stuff. Worked well.


Scale with large monorepos

I'm trying to use the action on a large monorepo and unfortunately it's spending 5 minutes processing the git fetch command here. From local testing on command line it seems like just adding additional arguments for origin and the ref makes is blazing fast. That doesn't work in practice though, since usually the PR branch is deleted after merge.
I figure the only way to solve this at scale is to rewrite the action to query the GitHub API for "what changed in the PR?" instead of relying on a local clone?

I had similar issues when using codenotify with a large repo, so I created my own GitHub action at https://github.com/tobyhs/codemention that uses GitHub's REST API to fetch a pull request's diff instead of using actions/checkout to clone the repo.

Thank you @tobyhs - that certainly looks very useful! I'll give it a try after my vacation.
Do you have any example of how the comment posted looks like? Similar to Codenotify?

Do you have any example of how the comment posted looks like? Similar to Codenotify?

You can see an example at https://github.com/tobyhs/codemention-test/pull/3#issuecomment-1627779765. Currently, it isn't as detailed as Codenotify (it doesn't have the commits used to obtain the diff; it only lists the matching file patterns instead of the files themselves) as I initially wanted to get something minimally viable going.

Nice, that's good enough for me. Thanks for the tip!


Turn down upstream private repository

@prattmic Please let me know when you've synced them.

Done

Repository deleted.


Add tables for inspector2 findings

Unable to get results from inspector queries when using inspector2. Need tables created so I can run queries against number of vulnerabilities detected that are High, Medium and informational as well as id, arn, asset type and numerical severity for a start.

Serendipitously, I started working on adding (some) inspector2 tables last week, findings and coverage among them.
@cbruno10, @misraved: I have some questions about best practices and the like; is this an acceptable place to ask them?  For example, is it okay to just add a few of the inspector2 tables (the ones I need) rather than attempting to flesh out the entire set of 10 (based on the aws inspector2 list-* CLI commands)?  Also, inspector2 is a little different from most services in that you can query one account, and get results back from more than one account... so the account_id behavior may be somewhat different from most resources.  I want to make sure that I get it right, so that folks aren't surprised by the results.  Some inspector2 CLI calls use awsAccountId to differentiate, but some do not, and I'm not sure about how closely to the AWS CLI I should stay, vs. having a more-consistent experience within steampipe.

@jaredreisinger-drizly Awesome, we're happy to hear you started working on them 😄
Yes, it's definitely OK to add some tables from a service initially. You'll find that for a lot of services we have tables for, we don't have all of the possible tables implemented yet.
I'm not deeply familiar with Inspector2 API results, but my suggestion would be to keep the account_id (which is a common column across all tables) as the account that the API request is being made against. If there's another field that the Inspector2 API returns specifically to represent which AWS account the finding/other resource is from, we can always add that field with an altered column name, e.g, finding_account_id. For instance, in many tables, we represent the Tags property as tags_src (https://github.com/turbot/steampipe-plugin-aws/blob/33ba2d0f6e1300912cca877b12d3a36fb8ff355d/aws/table_aws_ebs_volume.go#L138-L143).
If you have any other questions, please let us know!

Thank you so much for the quick response!
One other (hopefully quick and easy) question: In some cases, the returned value includes a nested structure, not a list of them, but just a nested object that hold several related fields.  This makes 100% sense in a JSON world, but isn't as useful in SQL/tables.  My current plan is to flatten these into columns, for example, to make:
{
  // ...
  scanStatus: {
    reason: "whatever",
    statusCode: "whatever",
  },
  // ...
}

into two columns, scan_status_reason and scan_status_code, omitting the scanStatus object itself.  This also helps because AWS allows filtering by those fields, and writing where scan_status_code = 'whatever' feels like very natural SQL.
Let me know if that isn't the recommended way to deal with simple nested structures.  😄

@jaredreisinger-drizly Sorry I missed your latest question until now! In these situations, if we do decide to add top level column support to nested properties, we'll usually keep the original column, e.g., scan_status, but then potentially add more top level columns.
We add columns instead of replacing to preserve the data we get from AWS, so it looks familiar to what users would see when using other SDKs or the AWS CLI.
In terms of how to decide if additional columns should be added, I've included some criteria we use when making similar decisions:

How key are the properties to the table? For instance, do most examples use these properties? If so, this may be good reason to add them as top level columns too.
How nested are the properties? One level nesting isn't usually too bad, but if there are 2+ levels, that can be difficult to query.
Would you be able to use these columns as key columns?

In this case, based on the questions above and the info you provided, I think it'd make sense to add 2 additional columns, scan_status_reason and scan_status_status_code. The scan_status_status_code column name is a bit long and redundant, but following this naming convention makes naming decisions more consistent across all plugins.
Hope that helps!

Added the tables in this PR


KeyVault JCA Tutorial: Using Jarsigner with KeyVault JCA provider

Context
In this issue: Can we integrate JCA provider with Jarsigner?, users asked a question about the usage of KeyVault JCA, and we found we didn't have many docs to describe how to use KeyVault JCA.
Goal
Write a KeyVault JCA Tutorial about using Jarsigner with KeyVault JCA provider.
Reference doc
Use Azure Key Vault to deliver TLS/SSL certificates to Apache Tomcat
Readme: Azure Key Vault JCA client library for Java

Draft Doc: integrate_keyvault_JCA_provider_with_jarsigner.md


Toast Properties

Add the ability to set properties for all the different properties in the UI.
This could easily be expanded to support lots more things, like separate sizes and custom fonts for the Title and the Toast message.

Fixed some bugs from previous request. Sorry for spamming you.

Thanks for this, @Liron-K. I've resisted adding customization like this for some time, but it's a frequent request and I really like your approach. Let me take some time to review/refactor/test, but I will mostly likely merge most of what you have here.
Thanks again!

No problem. There's room for exposing a lot more properties, but I decided to not do them for now. For example, you could have a separate property for the fonts and font sizes of the title and body text.

I took a slightly different route using styles in Toast 3.0.


ListRandomizer Problems

Currently, I think there are at least two problems regarding alfred's ListRandomizer, resulting in sessions ending with an exception and experiments not filling their randomizer slots as expected. I will discuss these problems in the same issue since I can not rule out that they are connected somehow.
In our latest set of online experiments, we see the following exception multiple times in our logfiles:

As far as I know, this has happened in every online experiment since the v2.3.2 patch for alfred. For my own experiment, I was able to trace the problem back to several experimental sessions starting at exactly the same time. From my experimental dataset and the log, I was able to identify 7 sessions starting in a span of 40 milliseconds. All of these sessions are trying to acquire a randomization slot at almost exactly the same time, which according to @jobrachem might lead to the above error message. The exception is a fallback for the improbable case that too many sessions try to access the randomization slot list at the same time. Of the 7 mentioned sessions, 5 exit the experiment with an exception, and only two sessions continue regularly.
Of course, it is highly unlikely, seeing the average number of sessions per day on the server (less than 100 for my experiment), that seven participants tried to start the same experiment at the same time, only 5 milliseconds apart. The underlying reasons for this seem to be performance issues in the university network, most likely with the outer firewall. Recently, I have run into a couple of performance issues (e.g., experiments taking a long time to start, pages loading with only 50 kb/s, javascript components not working because the script files are slowly loaded). These issues are mostly temporary and after a while (or a couple of hours), the performance normalizes. As far as I can tell from my logfile, the error might occur during such a performance bottleneck, during which multiple server requests build up (I suspect through one or more participants repeatedly trying to reload an experiment's landing page). I suspect the firewall since we've had similar issues with it in the past (remember when the firewall dropped some of the requests for resource files due to performance problems a couple of months back) and our server shows no indication of reaching anywhere near the maximum available resources (cpu, network, memory, etc. in the vm performance monitor). Also, I have seen some other network performance problems with vms yesterday and today (the above error is from yesterday).
In summary, I think we need to harden our Randomizer code against more sessions trying to access the randomization slot list at the same time, to prevent sessions from ending prematurely with the above exception.
In addition to the problem described above, I am convinced that the Randomizer still does not correctly manage randomization slots. While investigating the above problem in my experiment, I came across irregularities in the randomization. Before describing the problem itself, I will outline the general parameters of my current online experiment:
I am using a ListRandomizer.balanced() to assing two experimental conditions with 80 sessions in each condition. Hence, my experiment should remain open for new participants until I have acquired 160 finished datasets (inclusive set to False). In the experimental dataset, I am able to differentiate different types of unfinished sessions by the amount of stored data and some specific settings. I am calling the randomizer from my experiment's setup hook. Before initiating the randomization, I adjust the exp_session_timeout, which is the only thing happening prior to the randomization. Hence, I can differentiate states in which a session either starts regularly, or is aborted early. Specifically, there are two relevant types of sessions:

I have a relatively small amount of sessions (50 in 1000), for which a dataset exists, in which the exp_session_timeout remains at its default value (86400 seconds). These sessions have been saved once before running the code in the setup hook, otherwise we should see the changes I made to exp_session_timeout reflected in the saved dataset. In these datasets, some basic information is missing (e.g., client data and the starting time), while some critical initial information is present (e.g., save_time). These sessions have not received an experimental condition from the randomizer, at least there is no condition saved in the dataset.
I have a large number of datasets with assigned experimental conditions and all the initial information, but with no user input whatsoever. These datasets seem to originate from participants closing the experiment directly on the landing page, never clicking on any button in the navigation. These datasets are saved at a later stage in the startup process, after finishing the experimental setup hook, since both my changes to the exp_session_timeout setting and assigned experimental conditions are reflected in the dataset.

I assume, that the first type of dataset results from the randomizer aborting the session, but somehow this is not accurately reflected in the dataset (calling abort() should set the aborted flag to true, maybe create a log entry, and it should save a reason for the abortion of the session. None of this is happening). Moreover, the logged error message shown above can be traced to this type of session data (but not all of these sessions have an associated error message in the log). There are no other sessions in my experimental data that indicate a regular call to abort() by the randomizer. Also, these 50 sessions are not evenly distributed, but start to occur more often when the majority of randomization slots are already closed. Hence, I am convinced this type of session in the experimental data set reflects an irregular abortion by the randomizer. Interestingly, I am running a Matchmaker behind the randomizer, also in the session hook. When the MatchMaker aborts the experiment due to no available matching slots, the experiment is aborted normally, specifically, the aborted state is set to true and a reason for the abortion is saved in the dataset. Also, a log entry is created. Since both the randomizer and the match maker are run in the experimental hook, I think the problem is a specific randomizer problem.
In summary, one problem seems to be that the randomizer does not work as expected when aborting the experiment due to no available randomization slots. Maybe there is an exception that is not handled, yet? This could explain why we never see regular session abortions performed by the randomizer.
Moreover, I think there is another problem with the process of determining the number of open slots: As described above, my randomization should assign 160 participants to two experimental conditions, 80 participants in each condition. I have set the exp_session_timeout to 7200 seconds instead of 86400 seconds. Hence, I would expect the randomizer to assign slots to new sessions until all slots are finished or all slots have ongoing datasets with saving times not older than two hours. My dataset, however, shows that sessions are aborted well before these criteria are met. Here is an excerpt from my randomizer slot list for the experiment:
{
                "label": "sanitized",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "sanitized",
                "session_groups": []
            },
            {
                "label": "sanitized",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            },
            {
                "label": "natural",
                "session_groups": []
            }

I do not see how the randomizer could abort a session when there are slots in the randomization list to which no session was ever assinged (hence, no ongoing session for this slot). Nontheless, sessions were not assigned an experimental condition and aborted prematurely, according to the experimental dataset.
I think all of the above problems occur, at least in part, because of the large number of unfinished sessions in our online studies. The amount of unfinished sessions blocking randomization (and match making) slots are somehow causing problems that we have not encountered while testing. In my experiment, there are 10 unfinished datasets for every finished dataset and I think we should test both match making and randomization more thoroughly under such circumstances.

I think it is most sensible to handle the three issues that you describe here in separate issues. See

#206
#207
#208


Add form-request-submit-polyfill by default so we can rely on requestSubmit()

Since we can't use form.submit() with Turbo and have all the triggers played, we need to use form.requestSubmit(). Problem is that this requires a polyfill in Safari. Since Turbo is the one making it such that you can't just call #submit, I feel like we're also responsible for provide for a no-effort alternative. Thus, I think we should just include the form-request-submit-polyfill by default to make this so.

Closed via https://github.com/hotwired/turbo/pull/439


Fix DBZ-4413 MongoDB Extract Document State SMT overwrites id field

As discussed here: https://issues.redhat.com/browse/DBZ-4413

Based on my conversation with @Naros, I've added the instructions on how to run the MongoDB oplog ITs from an IDE as a readme section. I hope this saves some time for future contributors. Please drop me a line if I should squash the two commits.