Changing conditional checking of object properties to optional chaining

While going through the volto/plone repository I found that one of the components in src/components/themes/Icon/Icon.jsx
There was conditional checking of object properties,
xmlns={name.attributes && name.attributes.xmlns}
viewBox={name.attributes && name.attributes.viewBox}
It can be replaced with optional chaining.

@nilootpal please post text, not images of text. See https://meta.stackoverflow.com/questions/285551/why-should-i-not-upload-images-of-code-data-errors please.

Thanks @stevepiercy for the comment and I have changed as per your advise

Hi nilootpal,
I want to work on this issue. Is it open?

@saadkhan011 it is open (see the green Open indicator?). However @nilootpal did not link their PR to this issue, nor did they assign themselves to the issue, thus you would have had to search the pull requests to determine that any work had been done on the issue.
@saadkhan011 before doing any work, please read contributing guidelines and follow their instructions:

https://6.docs.plone.org/contributing/index.html
https://6.docs.plone.org/volto/developer-guidelines/contributing.html


[MINOR] Update NOTICE file to reflect the correct date

Codecov Report

Merging #271 into master will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #271      +/-   ##
============================================
+ Coverage     68.18%   68.23%   +0.05%
- Complexity      945      946       +1
============================================
  Files           102      102
  Lines          5899     5899
  Branches        896      896
============================================
+ Hits           4022     4025       +3
+ Misses         1303     1301       -2
+ Partials        574      573       -1

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/org/apache/livy/rsc/driver/RSCDriver.java
80.75% <0%> (+0.83%)
46% <0%> (+1%)
:arrow_up:

...c/main/scala/org/apache/livy/repl/ReplDriver.scala
33.33% <0%> (+2.56%)
7% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


[Jaeger-Agent] Use RawConn.Control to get fd instead of Fd()

Which problem is this PR solving?
Resolves #4448
Short description of the changes
Jaeger agent gets stuck when closing with SocketBufferSize set. This is because Close() of net.UDPConn will be blocked if Fd() is used to get the file descriptor.
Use RawConn.Control instead to get fd to set the socket buffer.
Same issue was discussed here: https://github.com/golang/go/issues/29277
The fix refers to here: https://github.com/brucespang/go-tcpinfo/pull/3

please make sure commits are signed

Signed


Use powershell as a default CMake/Launch terminal

Your issue may have an existing solution
-->
If I configure cmake project and launch using Shift + F5 ( Build without debugging). I can see the output from vscode terminal.
I set my default terminal of vscode as powershell, but CMake/launch terminal is always using cmd.
Is there any way I can set up default terminal for CMake/Launch?

Has this been implemented?


Support Google Play Billing Library v6 is Required by Aug 31, 2024

According to this link all new apps and updates to existing apps must use Billing Library version 6 or newer by Aug 31, 2024.
This library is currently at v5.2.1
https://github.com/chargebee/chargebee-react-native/blob/daeb1747a90e0fea091f42d943124b8b199062fc/android/build.gradle#L88

Hi @MarkAuderli
This is on our radar and we would be updating this library soon. We will communicate once its out.

Hi @MarkAuderli This is on our radar and we would be updating this library soon. We will communicate once its out.

waiting for the update.

Please update the play library.

Hi chargebee team, will you please update this at the earliest, we have used your SDK in our react native app. Waiting for your response

Hi @talhatsk , @sahanap92 , @chethana-shastry-ps  -
We are working on it and expect it to be out for all relevant sdks by 20th Jul.

Release 3.0.0-beta.1 (beta) and 2.5.0 (main) are updated with google billing client 6.2.1.


enable internal kv bucket in llama

What does this PR do?
To enhance throughput in scenarios with long new tokens, break down the KV cache into multiples of the bucket width. Use this to compute attention rather than using the entire KV cache.

Add --bucket_size=128 --bucket_internal to the commands to enable the feature.

@dvarshney-habana - check comment. Once addressed then we can merge.

@puneeshkhanna updated, thank you!

@xt574chen  - Thank you. Hope you also verified the changes and we are not missing any corner cases.
Changes look good to me.
@dvarshney-habana  - lets merge it so that we can start testing in nightly jobs too and we can see an impact of improved performances with bucketing.

https://github.com/huggingface/optimum-habana/pull/720


Overlapping text when changing different languages and helpboard butt…

…on fixed
What does this pull request aim to achieve? 💯
_Please be concise and link any related issue(s):_This PR Resolves #2233

[x] I have checked that no one else is working on similar changes.
[x] I have read the latest README and understand the development workflow.
[x] The name of this branch is: feature/<branch_name> for a regular feature, hotfix/<branch_name> for a hotfix, and service/<branch_name> for a microservice, where <branch_name> briefly describes the issue(s) resolved and is prefixed with the issue number(s) (e.g. 1127-1130-update-git-branching-model). The branch is created in a cloned version of this repo, not a forked repo.
[x] This branch is rebased/merged with the latest staging (or the latest production if it is a hotfix).
[x] There are no merge conflicts.
[x] There are no console warnings and errors.
[x] On the right hand side of this PR, I have linked the appropriate issue(s) under "Issues" and added a project progress under "Projects".
[x] The title describes the issue(s) being addressed using format: <issue_numbers> - <brief_issue_description> (e.g. #1127/#1130 - Update Git Branching Model)

@sujatha-m The build link  https://feature2233-overlapping-text.fightpandemics.xyz. for the
PR#2251 works as expected, i did not observe any overlapping of text in the languages translated in the Help board. Since this issue was only for desktop, review has been done for desktop only. Below is the observations made:
Browsers: Chrome, Safari, Edge
OS: Windows 10 and Mac OS Big Sur
Resolution: 1920x1080, 1366x768
Desktop
https://user-images.githubusercontent.com/78573508/109733881-dd0d6f00-7b74-11eb-99d4-82dd6d2cf277.mp4


Contributing - Maven Dependency

While setting up my environment for development (macOS), Maven was required to complete make test which required certain mvn commands.
Should Maven be added as a dependency under the #Setting up an Environment header in the Contributing tab?

Should Maven be added as a dependency under the #Setting up an Environment header in the Contributing tab?

Thanks for the analysis.  We should add it.  I'll do!


Ability to create lists in Env

Code of Conduct

[X] I agree to follow this project's Code of Conduct

What would you like to see changed/added?
The ability to create lists, similar to Path, easily cross-platform, like this:
env:
  - name: USERNAMES
    value: |
      dylan
      steve
      bob ross

# Windows
$Env:USERNAMES = "dylan;steve;bob ross"
# Linux
$Env:USERNAMES = "dylan:steve:boss ross"

Or, with an optional delimeter parameter:
env:
  - name: USERNAMES
    value: |
      dylan
      steve
      bob ross
    delimeter: ","

$Env:USERNAMES = "dylan,steve,bob ross"

@RuiNtD there's no such thing though. Environment variables are always strings, this is a very specific use-case where I fail to see the use-case for as it assumes you're actually having different env var logic in an application per platform.

There are many applications that take lists of strings, usually paths, in an ENV, like how the OS does with Path. It would just be nice to be able to format them in multiple rows instead of just one.

Implementing this is actually feasible, I would allow the delimiter to be templatable though.

I mean, I did mention a delimiter option in the original post. Lol

@RuiNtD I know, and upon re-read I went with that option.


fix: Remove trustanchor.crt value in favor of existing chart value na…

…med identityTrustAnchorsPEM.

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

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[8.x] Add reduceMany to Collections

Sometimes you need to compute multiple aggregate values from a collection. In many cases, it's fine to perform separate operations, but in cases where these values are interdependent, or there's a material performance cost to iterating the collection multiple times, this is not ideal.
This PR introduces a reduceMany operation that behaves exactly like reduce but allows for carrying as many values as needed. Rather than using the simplest example (see the added tests for this case), here's a slightly more complicated "real-world" example:
// Figure out how many images we can send to our image processor, given a fixed
// number of "credits" available (where each image may require more than 1 credit)
[$credits_remaining, $batch] = Image::unprocessed()->get()
    ->reduceMany(function($credits_remaining, $batch, $image) {
        if ($credits_remaining && $credits_remaining >= $image->creditsNecessary()) {
            $batch->push($image);
            $credits_remaining -= $image->creditsNecessary();
        }

        return [$credits_remaining, $batch];
    }, ImageProcessor::creditsAvailable(), collect());

ImageProcessor::processImages($batch);

if ($credits_remaining < 100) {
    Log::alert("Only $credits_remaining image processing credits remaining!");
}

(See this Twitter thread for another real-world use-case).
In the end, the API is exactly the same as reduce except that you can pass in more $initial values and receive a corresponding number of $carry values before the $value and $key. The only other difference is that you must return an array in your reducer that corresponds to the number of values you're reducing.
I've intentionally left this method off the Enumerable interface for backwards-compatibility reasons. If this PR gets merged, I would suggest adding it to the interface in 9.x and possibly updating the signature of reduce to behave the same way (which would be backwards compatible but change the interface and thus affect anyone who has a custom implementation).

Have you seen a similar method on any other client libraries in other languages?

Not exactly. JS benefits from destructuring assignment in the function parameters. And there are functional programming approaches like transducers that try to perform multiple operations in a single pass, but nothing with this API.
This came up because @DanielCoulbourne asked a question about it on Twitter and I had very recently had a conversation about exactly this with @cheyner — it just felt like too much of a coincidence :) There are "fine" approaches that you can use right now, but nothing that really feels right.


Any way to increase/decrease the render distance of 3D buildings?

I am thinking, if its possible to increase / decrease the distance from where the building starts rendering, so that I don't have to scroll too far and get too zoomed in but I can see a skyline view of the buildings and of course it would be help if there is a way to increase how many buildings get rendered, basically the radius of rendering

Hi @sirakc,
I am not sure I get your issue, maybe this helps:
You can basically adjust the Style of the extruded buildings by extending the "Theme", e.g like following
theme: {
extends: "resources/berlin_tilezen_base.json",
// end:harp_gl_hello_custom_theme_0.ts
styles: {
tilezen: [
{
id: "extrudedBuildings",
description: "extruded buildings",
technique: "extruded-polygon",
when: ["ref", "extrudedBuildingsCondition"],
minZoomLevel: 16,
renderOrder: 2000,
height: ["get", "height"],
color: ["ref", "defaultBuildingColor"],
roughness: 1,
metalness: 0.8,
emissive: "#78858C",
emissiveIntensity: 0.85,
footprint: true,
maxSlope: 0.8799999999999999,
lineWidth: 1,
lineColor: "#172023",
lineColorMix: 0.6,
fadeNear: 0.4,
fadeFar: 0.5,
lineFadeNear: 0.4,
lineFadeFar: 0.5
}
]
}
}
note the changed fadeNear, fadeFar parameters which will fade out the buildings earlier (in direction of the horizon) on the map.
Is this the case you want to achieve?
Otherwise if you refer to distance as in the height the camera has over the map, which is defined as zoomlevel, this can also be configured, but is internally limited to I think show earliest at zoom-level 14 or 16 or similar, due to performance issues if we would allow it before.

@FraukeF yea first I was talking about how can I see the buildings even from a far distance, then I asked if I can increase how many buildings get rendered once the buildings start appearing, and the solution answered both the questions :D


TeamCity: ERROR The Chrome 73.0.3683 / Linux 0.0.0 browser disconnected

What is your Test Scenario?
I'm running TestCafe tests in TeamCity, no concurrency/no quarantine mode.
Few tests are running one by one.
The issue was not reproduced locally.
Some other tests are running along with Test Cafe, but the load is too small to cause resource draining or application issues.
What is the Current behavior?
Tests started randomly fail last week with "ERROR The Chrome 73.0.3683 / Linux 0.0.0 browser disconnected"
No tests hanging, no browser sessions hanging. Sometimes run fails even before the first test. TeamCity agents restart fixes problem for a few runs.
What is the Expected behavior?
The browser should be opened for each test.
Your Environment details:

testcafe version:   1.1.0
node.js version: v8.14.0
testcafe chrome --no-sandbox --reporter teamcity --screenshots ./screenshots --screenshots-on-fails --video ./videos -e
browser name and version: Chrome 73.0.3683
platform and version:          Linux
other:                                   Runnin in TeamCity

Please advise on ways to troubleshoot it or minimize occurrences.

We already had some reports about issues with Chrome and we are working on it (see https://github.com/DevExpress/testcafe/issues/3671). Since it's very difficult to reproduce and debug this issue I cannot give you any estimate on when it will be fixed. However, of course, we hope to provide the fix in the near future. I'll close this issue, follow our progress in https://github.com/DevExpress/testcafe/issues/3671
In the meantime, try updating your Chrome version and check how it works.

@AlexKamaev we changed our run to "chrome:headless --no-sandbox" and updated agents configuration. It's more stable now. Hope it helps.

@apoluian, thank you for the update. We will take it into account when examining this behavior.


Feature request: opacity transition

Just like with parallax transitioning i would like to be able to define an opacity transitioning during slides.
Swiper argument:
{
    opacity: true
}

Data attributes on elements inside a slide element, due to the nature of opacity negative values would make no sense.

Title

Subtitle

Text

Thx

I think I understand what you mean. What you would like to accomplish is fading in/out individual elements within a slide and to also animate these opacities at different speeds. I did something pretty similar within the onSlideChangeStart callback. In the callback you can look for elements within the current slide that has either certain classes or data attributes and apply the desired effects.

+++

@mikebonds can you send me your code?

@Tusko That code has was written probably close to 2 years ago. I believe that project has changed since then. I'll try to dig something up.

@mikebonds thank you so much)

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


fix: _cross_xAx does not find dateAx

ValueAxis._cross_xAx does not find c:dateAx crossing axis (neglected when dateAx was added).
Outline of problem with fix is here:
https://stackoverflow.com/questions/52271154/problem-with-setting-value-at-which-category-axis-intersect-value-axis/52315841#52315841

Fixed in release v0.6.14 on Sep 24, 2018.


Wrap scheduler tasks in Open Telemetry spans

Hey, I just made a Pull Request!

This PR changes the scheduler so that tasks are wrapped in Open Telemetry spans.
Before this change, when trying to troubleshoot a scheduled task through an OTEL trace, spans would be incorrectly grouped. This lead to the scheduled task being split in many traces. By adding a span around the task itself, all the work a task does can now be grouped into that task in a single span.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

👋 @Rugvip It's been a little while this PR hasn't seen updates. Can you take a look. I believe that I've addressed your comments.


Default navigation: map. Missing cache context

When selecting map as default navigation, all caches in the area are displayed when selecting the default navigation for a cache

Yep, just wanted to write the same issue.


Handle unsupported boards gracefully along with micro:bit support

test this please

test this please

test this please

+1, however dont have access to CI at the moment to see the failure

test this please

test this please

test this please


Need advice using custom dataset

Hi this is really good application. I would like to have your advice to use my own dataset. I have images that are very different with widely known dataset such as imagenet containing common object (cars, cat, dog, etc). this is very technical related, it is dynamometer cards that commonly used in oil industry as shown below :
https://knepublishing.com/index.php/KnE-Engineering/article/download/3083/6588/15176
Unlike common object, my dataset always comes in neutral position, there is no rotated, no flipped, no color distortion (just B&W) and always comes in the full part (unlike cars that might exist only part of it in images), etc that might exist during implementation in the real world. So in the common object recognition, I assume such data augmentations are needed to introduce many variations to the model.
Do I need to do same data augmentations used by default in this repo ? or is it ok for BYOL to use no augmentations or minimum augmentations ? what do you think the best solution needed for my own dataset especially in term of augmentation.
thank you

Hi, @ramdhan1989 . Can you please share the source of dataset that you are using ?
I am also working on similar problem. It will be very helpful for me , if you can share your dataset of dynacards.


App crash: $s7Amplify23AWSUnifiedLoggingPluginC7defaultAA6Logger_pvg

Describe the bug
Application crashes with the following call stacks:

Crashed: com.apple.main-thread
0  Amplify                        0x3b674 $s7Amplify23AWSUnifiedLoggingPluginC7defaultAA6Logger_pvg + 48
1  Amplify                        0x3b638 $s7Amplify23AWSUnifiedLoggingPluginCAA0C22CategoryClientBehaviorA2aDP7defaultAA6Logger_pvgTW + 20
2  Amplify                        0x7c114 $s7Amplify15LoggingCategoryC4infoyySSyXKF + 140
3  AmplifyPlugins                 0x1ea64 $s14AmplifyPlugins28AuthorizationProviderAdapterC13setupListener33_9F56DFBCB2FE52BF5B4A87DA0573D416LLyyFy15AWSMobileClient9UserStateO_SDyS2SGtcfU_ + 116
4  AWSMobileClient                0x166e0 $s15AWSMobileClient9UserStateOSDyS2SGIegyg_AcDytIegnnr_TRTA + 28
5  AWSMobileClient                0x2a4fc $s15AWSMobileClientAAC7signOutyyF + 2348
6  AWSMobileClient                0x2aec4 $s15AWSMobileClientAAC15internalSignOut33_92B29A0229F7802B642D31E4C9FB126DLL18presentationAnchor7options17completionHandlerySo8UIWindowCSg_AA0dE7OptionsVys5Error_pSgctF + 1380
7  AmplifyPlugins                 0x1a744 $s14AmplifyPlugins29AuthenticationProviderAdapterC13signOutWithUI33_8CDF2480850D24387F6F51D10DDDA818LL15isGlobalSignout17completionHandlerySb_ys6ResultOyyt0A09AuthErrorOGctF + 328
8  AmplifyPlugins                 0x1aea8 $s14AmplifyPlugins29AuthenticationProviderAdapterC7signOut7request17completionHandlery0A008AuthSignG7RequestV_ys6ResultOyytAG0K5ErrorOGctFyyScMYccfU_TA + 88
9  AmplifyPlugins                 0x6a2c0 $sIeg_IeyB_TR + 28
10 libdispatch.dylib              0x1e6c _dispatch_call_block_and_release + 32
11 libdispatch.dylib              0x3a30 _dispatch_client_callout + 20
12 libdispatch.dylib              0x11f48 _dispatch_main_queue_drain + 928
13 libdispatch.dylib              0x11b98 _dispatch_main_queue_callback_4CF + 44
14 CoreFoundation                 0x51800 CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE + 16
15 CoreFoundation                 0xb704 __CFRunLoopRun + 2532
16 CoreFoundation                 0x1ebc8 CFRunLoopRunSpecific + 600
17 GraphicsServices               0x1374 GSEventRunModal + 164
18 UIKitCore                      0x514b58 -[UIApplication _run] + 1100
19 UIKitCore                      0x296090 UIApplicationMain + 364
20 Breeze                         0xd76c main + 42 (AppDelegate.swift:42)
21 ???                            0x103865da4 (Missing)

Steps To Reproduce
Not reproducible. Happens randomly.

Expected behavior
Show not crash.
Amplify Framework Version
1.22.1
Amplify Categories
Auth
Dependency manager
Cocoapods
Swift version
5.0
CLI version
Don't know
Xcode version
14.1
Relevant log output
<details>
<summary>Log Messages</summary>

INSERT LOG MESSAGES HERE

```
Is this a regression?
No
Regression additional context
No response
Device
iPhone XR
iOS Version
iOS 15.6.1
Specific to simulators
No response
Additional context
No response

Hi @zhouhao27, thanks for opening this issue.
While we investigate what's going on, could you please provide the following?

Your app's amplifyconfiguration.json file, with sensitive information removed.
Is it a regression starting on version 1.22.1, or does it happen for older versions as well?
Have you tried updating to the latest 1.x (v1.28.2) version?
Can you pinpoint if the crash happens in a particular moment (e.g. during start up, after signing in, etc)?

Thanks!


Fix Github "Used by" metric for this repository

Right now the Github used by the metric for this repository is trying to aggregate usage of @prisma/generator-helper.

It should track usage of prisma2 package instead.

https://github.community/t5/How-to-use-Git-and-GitHub/quot-Used-by-counter-quot-not-showing-all-projects-and-main-one/m-p/51098#M11010

Fixed via manual action from Github: https://github.community/t5/How-to-use-Git-and-GitHub/quot-Used-by-counter-quot-not-showing-all-projects-and-main-one/m-p/51446#M11053


Update GA4 docs

Include information on what dimensions are available from events sent to GA4

#2419  👈 (View in Graphite)
main

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

🚀 Deployed on https://6761f5d4a9eaf503fb89d101--cozy-fox-0defec.netlify.app


Citing the project

Hello,
I am writing this issue/question on how to cite this repository. Is there a preffered way from you guys on how to do this?
I am using the code of this repository in my Bachelor Thesis on structure mining. I am citing the paper you are referencing in the projects readme.
Thanks for your answer and best regards.

Hi,
No preferred way. Just cite this repo in your way.

Thanks for the reply,
I went for the following style in BibTex:
@misc{gSpan-github,
  author = {QY, Nikolay Karpov},
  title  = {betterenvi/gSpan; Version `0.2.3'},
  url    = {https://github.com/betterenvi/gSpan},
  note   = {last access: 07.09.2020}
}

Unfortunately I don't have a full name for you QY :/

My full name is Qingying Chen.
Free to cite.
Thanks :-)


⚠ Secret Website 2 has degraded performance

In a2c6502, Secret Website 2 ($SECRET_WEBSITE_2) experienced degraded performance:

HTTP code: 200
Response time: 6387 ms

Resolved: Secret Website 2 performance has improved in 34b0a3e.


sf-password & sf-client-secret property of Salesforce outbound provisioning connector needs to be confidential

Is your suggestion related to an experience ? Please describe.
When we get Salesforce outbound provisioning metadata using Identity Provider Management API, for sf-password & sf-client-secret properties we get it as a not confidential value.
Describe the improvement
Following properties of salesforce outbound provisioning connector needs to be confidential.

[ ] sf-password
[ ] sf-client-secret
[ ] sf-password type also needs to be change.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Update Azure CCM, AzureDisk CSI, and AzureFile CSI for Kubernetes 1.26

What this PR does / why we need it:

Update Azure CCM to v1.26.0, v1.25.5, v1.24.11, v1.23.24
Update AzureDisk CSI to v1.25.0
Update AzureFile CSI to v1.24.0

Which issue(s) this PR fixes:
xref #2562
What type of PR is this?
/kind feature
Does this PR introduce a user-facing change? Then add your Release Note here:
- Update Azure CCM to v1.26.0, v1.25.5, v1.24.11, v1.23.24
- Update AzureDisk CSI to v1.25.0
- Update AzureFile CSI to v1.24.0

Documentation:
NONE

/lgtm
/approve


Further reduction of pod cold start time

I believe we could do < 20s for llama 2 7b models.

Not current priority. Canceling.


Allowing to name a message for later adjustment e.g. removing the message in extended pages

Implemented in release 0.2.0


added 'object.remove()' thanks to Allah

for #29 I issued
I added these lines
    /// removes true if object was removed
    /// and false otherwise
      pub fn remove(self: *Object, key: []const u8) bool {
          // I saw it in `deinit()` so
          // I tried to do the same
          if (self.hashmap.getEntry(key)) |entry| {
              self.allocator.destroy(entry.value_ptr);
              self.allocator.destroy(entry.key_ptr);
          } else return false;

          return self.hashmap.swapRemove(key);
      }

is there anything I should edit?

@IbrahimOuhamou This code looks good to me - would you mind adding a test ? See src/tests.zig for examples - you should be able to copy one from there and just verify that removing results in null when fetching a removed key.

Nice work, thanks a lot @IbrahimOuhamou
Each Jetzig request has its own arena allocator (and Zmpl itself has its own arena as well). Zmpl frees its arena in Data.deinit() and Jetzig frees the request's arena when the request has finished processing. So you can be pretty lazy with freeing memory I'm Jetzig and when using Zmpl by itself.
Thanks for the PR.

you're welcome!


Refactor codebase to async/await - m365 teams app, m365 teams cache, and m365 teams funsettings

Closes #5088

Thanks for another awesome PR we will review it ASAP 👍

Merged manually, thank you 👏🥳


replace unsupported and vulnerable satori/go.uuid with gofrs/uuid

Is your feature request related to a problem? Please describe.
Replace non-maintained and vulnerable satori/go.uuid with its successor - gofrs/uuid
https://github.com/satori/go.uuid
https://github.com/gofrs/uuid
satori/go.uuid security issue:
https://snyk.io/vuln/SNYK-GOLANG-GITHUBCOMSATORIGOUUID-72488
Describe the solution you'd like
PR: https://github.com/swaggo/swag/pull/798
Describe alternatives you've considered
Since gofrs/uuid is mature successor of satori/go.uuid - it's most compatible replacement of it
Additional context
https://snyk.io/vuln/SNYK-GOLANG-GITHUBCOMSATORIGOUUID-72488

Merged.
Thanks for your contributions.


CI: Add automated check for copyright headers

Proposal
I occasionally forget to add copyright headers to new files in PRs, and reviewers have had to manually comment on this. Such a check can be automated, saving reviewer time.
golangci-lint includes support for the go-header linter which can probably leveraged to implement this, at least for Go source files.
Comments? Happy to implement this if folk think it would be useful.

I've added a copyright profile in Goland that automatically adds the header in new files, but this is useful in general.

Voted for golangci-lint with go-header, one good thing is that we don't need to introduce more tools required as part of development.
PS: I might be biased here.

Quick update on this: adding the check is fairly straightforward, but it turns out that there are a lot of files that either don't have a copyright header, or have a slightly weird one. I have a WIP PR but there are still a lot of files to update. As there are other higher-priority tasks this is only getting idle cycles from me.


Update to Fleet 2.1.1

https://github.com/kolide/fleet/releases/tag/2.1.1
As part of this, implement the username using fleetctl so that admin can be used to login again: https://github.com/kolide/fleet/issues/1970

Fixed in #262


Use get_full_name() instead of incorrect full_name attribute

Fixes #83

Just noticed this is a duplicate of #80


Progressive JPEG support

Hi,
is the generation of progressive, interlaced JPG supported somehow?
Thanks

Not at present as the jpeg encoder in System.Drawing does not support it. I am however attempting to use fully managed encoders in v3 which would support progressive jpegs. There's no release date for that defined though as it is a lot of work.

@JimBobSquarePants thanks for the quick response

When I google 'Image Processor Progressive Jpeg' this issue appears. For other people looking for this feature, it has now been added - see this post

Thanks for updating here. That's only valid for the ImageProcessor.Web as of v4.4.0 The ImageFactory in the base library doesn't know about it so desktop applications will still be without..


Scrolling on Web UI

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

When Alexa Displays Info that overflows, it's impossible to scroll
What is the expected behavior?

Native and natural scrolling
What behavior are you observing?

Can't scroll (horizontally or vertically)
Provide the steps to reproduce the issue, if applicable:

Run the Sample app, and have index.html displayed on the touchscreen with the rectangle config.
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <x.y.z>

Latest version
Tell us what hardware you're using:

[ ] Desktop / Laptop
[x] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[x] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi @Piflyer,
Could you provide more details regarding what Alexa skill you're using and what's the screen size?
Thanks,
Benny

Hi, it's on the knowledge card, weather card, and just about anything that needs scrolling. I adjusted the config file to fit the Raspberry Pi Touchscreen (800x480).

Another issue is that it will not play videos on enabled skills at all. Tells me to check the Alexa App, and the device is not shown in the Video skills.

Hi @Piflyer , It sounds like touch input is not being passed through, are you able to use TouchWrappers (e.g. the playback controls presented by the music player)? Also when browsing other web sites are you able to scroll by dragging the content?
Regarding video playback, what skills are you testing against? Only APL based video skills can be played using the SDK, such as "Syfy Wire".

Closing this due to inactivity. Please feel free to re-open if you encounter further problems.
Thanks!


Add setProxy method

Proposed changes in this pull request
Just add the PhantomJS method setProxy
Checklist

[ ] New tests have been added
[ ] npm test passes successfully
[ ] Documentation has been updated

@amir20 to review

Where are the tests and docs?

It looks good, but you shouldn't commit the compiled code (i.e., don't add the 'lib' directory). You should probably add a test too.

Any idea on how we could test this?

Let me think about that. I couldn't find anything about set proxy documentation so I am not sure how to call it. At the very least, we should make sure that no errors are raised when it is called. I was also thinking maybe setting bad proxy to see if it fails.

@wesleymilan any update?

Uh, it looks like you have removed #setProxy now? If you want to add other methods you should probably do that in another pull request.

Hey @wesleymilan, unfortunately this pull request is not up to this repo's standards. Please review the guidelines for creating pull requests. Specifically, there must be new tests. Adding new functionality without tests will not be merged. There are also build errors in Travis and you have reversed the .gitignore file. Due to those reasons, I will close this pull request. Please create a new pull request with all these fixes.


Add private shim to fetch credentials

This is for internal use for retrieving resolved credentials off a client when a Session credential resolver isn't readily available. It is subject to unannounced breakages and shouldn't be accessed publicly.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3f645d1) 93.46% compared to head (3fb38a4) 93.38%.
Report is 52 commits behind head on develop.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3068      +/-   ##
===========================================
- Coverage    93.46%   93.38%   -0.08%
===========================================
  Files           66       66
  Lines        14007    14038      +31
===========================================
+ Hits         13091    13110      +19
- Misses         916      928      +12

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


boards: cc3220sf_launchxl: Make cc3220sf XIP by default

Previously, the cc3220sf platform could only be flashed using
TI UniFlash (or CCS) tools, requiring unique instructions.
Also, these procedures were not integrated with the standard Zephyr
build targets: 'flash', 'debug', 'debugserver'.
After installation of a TI OpenOCD v1.10 release, and other
one-time board setup, flashing and debugging can now be done
via OpenOCD/gdb, using the standard Zephyr build targets per
the Zephyr Application Development primer.
This also makes the cc3220sf an XIP platform by default, and removes
some now extraneous board support files.

Codecov Report

Merging #7611 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #7611      +/-   ##
==========================================
- Coverage   55.24%   55.24%   -0.01%
==========================================
  Files         468      468
  Lines       51791    51791
  Branches     9909     9909
==========================================
- Hits        28612    28611       -1
- Misses      19266    19267       +1
  Partials     3913     3913

Impacted Files
Coverage Δ

lib/posix/pthread.c
69.34% <0%> (-0.51%)
:arrow_down:

Continue to review full report at Codecov.

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

I believe OpenOCD is developed using Gerrit.   The TI patches are in review, and according to the developer, it will be several months before getting upstreamed, which is why the board.rst prerequisites instruct to git clone the TI OpenOCD repo (git.ti.com/sdo-emu/).

I believe OpenOCD is developed using Gerrit. The TI patches are in review, and according to the developer, it will be several months before getting upstreamed, which is why the board.rst prerequisites instruct to git clone the TI OpenOCD repo (git.ti.com/sdo-emu/).

I guess I was asking to see if we could possibly pull the various changes into the Zephyr SDK so its openocd would support the board.

I guess I was asking to see if we could possibly pull the various changes into the Zephyr SDK so its openocd would support the board

Given the patches are still under review, and the TI OpenOCD repo may be updated at a higher frequency than the Zephyr SDK, I would suggest it might be safer to take the TI OpenOCD install.
My concern is the patches could break something for a different platform, a scenario which only gets tested (presumably) on official OpenOCD releases.
This separate install would be a temporary prerequisite, until the patches finally get merged.


Error occurs in smiles encoding containing @

When I use the @ symbol in the smiles encoding for image synthesis, an error occurs.
File "D:\ProgramData\anaconda\envs\pytorch\lib\site-packages\rdkit\sping\pid.py", line 289, in stringBox
return self.stringWidth(s, font), self.fontHeight(font)
File "D:\ProgramData\anaconda\envs\pytorch\lib\site-packages\rdkit\sping\SVG\pidSVG.py", line 633, in stringWidth
return pdfmetrics.stringwidth(s, fontname) * font.size * 0.001
File "D:\ProgramData\anaconda\envs\pytorch\lib\site-packages\rdkit\sping\PDF\pdfmetrics.py", line 320, in stringwidth
w = w + widths[ord(char)]
IndexError: list index out of range
I checked that it should be that the chiral molecule is not well written. What should I do?

Use of the @ symbol to denote chirality is not supported at present.  You can add it if you want...


Raspian Lite for custom images?

I'm wondering why the docs for custom builds ask to start from a full RaspiOS image instead of the Lite version. Due to an unrelated issue, I connected a display on first boot and noticed that there's a full desktop environment installed on the card. Perhaps there's a reason this is necessary?

Just saw #2389, which would make this question obsolete, of course. (And I would also be in favor of the proposal there.)

Cause there isnt a Liter version that is 64bit, that can support channel db larger than 1GB.

I see. Thanks.


Finalized Torque Controller & Test Cases

@jcallaham With this addition we have the ability to input a torque command to control the Cylinder environment. This enables a more realistic action space for the RL agent to learn on.

Oh, I think I see what the formatting issue is @SamAhnert.  I made a lot of these changes on jc/testcase-validation but I haven't merged them to main yet.  Let's do this - I'll merge #50 first, then let me know when you're all set with any other changes you want to make here and I can resolve whatever conflicts come up between this and main.  Then we can go ahead and merge this.
Definitely get pre-commit set up when you get a chance though.

Sounds good, made my final changes and installed the precommit hooks. Should be ready to merge whenever!


Features: Job expiration and trip only options in Route Finder

This PR adds 3 features and 1 small fix to Route Finder that allows user to:

Select jobs with a minimum expiraton deadline;
Select Trip Only jobs;
Swap between From/To button; and
Fixes a small bug in route timeline where the total was showing "and <999>kg" when no cargo job present, refering pax weight.

Motivation
Finder could build a very profitable route with jobs expiring too soon. The user, being able to enter a minimum job expire deadline, adds a useful mechanism of route refining. Also, since FSE doesn't allow mix VIP and Non-VIP jobs or add VIP jobs to a group, introducing these, allow Finder to return routes with jobs entirelly compatible, asuring it's continuity.

Sorry for the late reply, and thanks a lot for your contribution.
Regarding the first feature (expiration time), I am not in favor of integrating it into FSE Planner for a very dumb reason. I perfectly understand why this would be very useful. However, for this to work, you need to save the expiration time of each individual job, which is stored into the browser Local Storage. The issue is that there is a limit on the amount of data a browser allow a website to store into Local Storage. When loading large areas, I have seen users reach this limit (and had to optimize the data to stay bellow this limit). Storing more data will unfortunately be an issue for those issues.
What could be done instead is to introduce a new global setting 'Expiration time'. The app would simply discard all jobs that expires before XXX when loading them from FSE. This way, no need to store the expiration date, the app does not even store those jobs (it means they will not be shown on the map of available jobs either). What do you think of that?
For the other features, could you split them into different merge requests? It would be easier to review them.
I made some changes in the code, you might have to deal with conflicts, sorry :(


Quick Favor: can we get a demo content ?

Hope you're doing awesome! 👋 I've been checking out your UnrealMvvm and UnrealDI plugins on GitHub, and it's looking pretty cool!
I noticed there's no demo available yet. Any chance we could get a little demo action going? It doesn't have to be fancy—just something to show off what both of them can do in action together.
 Nice suggestion!
I already thought about making some simple game for showcasing how one can approach using both plugins together. Looks like it is time to actually start doing it.
Thanks for the feedback. It is a great motivation to see someone having interest in stuff that I make :)

Hi there!
I'm thrilled to hear that you're considering doing something to showcase the integration of both plugins. While a full game isn't necessary for me, even a small project would be fantastic for illustrating the combined capabilities.
Looking forward to hearing back from you when you have something ready!
Thanks again for the update.

@druhasu Hope you're doing well!
Just wanted to check in and see if there have been any updates on the minimal demo project showcasing the integration of UnrealMvvm and UnrealDI plugins


Still can't scroll on iOS 11.3.1 (iPhone 6) when accordion is open

Hi Guys,
I am opening this issue in relation to issue 2197 which is now closed so I can't add to it
Hi, I wonder if you can help, I am still seeing this, I have pulled the latest version of f7 js and css from github. I am using the latest version of iOS 11.3.1 on an iPhone 6. This is exactly the same behaviour explained I am seeing in this case and the bug. I have checked framework7.js and it has the code fix above mentioned but still the issue persists.
I have uploaded a screenshot attempting to scroll the accordion, unfortunately I cant really provide much else as this works fine when I view through the browser or on Android.
https://drive.google.com/file/d/1kMVEzWDgrq9xWUXBOKna1j-w9bDsertu/view?usp=sharing
Any help would be much appreciated
UPDATE: I have another video showing the behaviour there are a couple of things, only when scrolling on iOS does the content of the other accordion seem to flicker, the button hidden in the other accordion flckers briefly on the screen, I do actually use hidden divs inside the accordion with tables that are unhidden with buttons, I am going to try today to remove the hidden divs to see if that has any effect. The new video also shows behaviour that other people reported i.e. the screen does not scroll and then after a couple of attempts completely disappears.
One other thing is when I open up the hidden div and close it again it then seems to enable the ability to scroll for some reason.
Many thanks again guys
https://drive.google.com/file/d/1TloXzo0762H2Uolx8F1oD7pgZu20Fn47/view?usp=sharing

another quick update, I have tried various things like not using hidden div's in the accorion-item-content, I have also enclosed the inner part of accorion-item-content in a div with a block class.

Have the same issue.
My layout is much more simple: accordion list with ordinary list as a content of each accordion item.

I insert the code generated by this template into page-content directly.
There are 3 problems with its behaviour:

Flickering when I close accordion items
Scroll freezing
(The most disgusting). Content disappearing when scroll

Note
Content disappearing takes place if I use "view-in-popup" layout (F7 popup with F7 view as content, described here. View content is generated by the code above.)
If I open page via router, there is no such behaviour. At least, I wasn't able to catch it.
Flickering and scroll freezing take place in both scenarios.
I use Framework7 v2.0.8, iPhone 6 Plus with iOS 11.3.1.

another update - I have tried the same on an iPhone 7 and it works fine.  Also no issues on various Android devices.

@goodfellasthai Do you use UIWebView  or WKWebView in your Cordova app?

I am using Phonegap and PhoneGap build for the apps, but checking the build logs there are a lot of entries for UIWebView and none for WKWebView so I am assuming it is being build with UIWebView.

In my case opening the accordion added "overflow: hidden" to the class ".page-content" for some weird reason....
So I added this to my app css file:
.page-content{ overflow-y: scroll !important; }
and the issue with not being able to scroll was fixed for me.

Hi Stargurl still no joy for me with that overflow setting in my app .css, just to add another issue to this it may be related or it may not and I have seen people having similar issues in this issue forum.  When clicking on the accordion item title the accordion appears quickly then disappears as though you have pressed it twice.  If you click it super-fast it works.  A lot of these issues seem to be linked I think, and some specific to the iPhone 6, I have tried a few source code mods but no joy :(

Guys,
Just as an update I had hunted round many forums for a couple of months for the solution to this, it seems as though it is not an issue with F7 but more a specific issue to how iOS in particular deals with overflow situations (I also found people having similar issues using ionic which is where I got some of the solution from).  I also found this error only seems to happen in F7 when using tabs and not on standard screens.
I added the below css lines to cover all the situations I was experiencing, one weird thing until adding the 'touch' css to the specific css areas iOS would would still work with double finger scrolling but not single finger scrolling.
After adding the below css to my app css file I had no issues with scrolling or disappearing screens on iOS, some of my colleagues also reported similar issues with my app during Android testing.
If I find any other details I will update this thread.
@stargurl16 thank you for your initial input that put me on the right track.
Regards
Paul
CSS
.page-content{ overflow: auto !important; -webkit-overflow-scrolling: touch !important; }
.tab { overflow:auto; -webkit-overflow-scrolling:touch; }
.tab .content-block {  overflow:auto;  -webkit-overflow-scrolling: touch;}
.tab .page-content {  overflow:auto;  -webkit-overflow-scrolling: touch;}
.accordion-item-content {-webkit-transform: none; transform: none; overflow:auto; -webkit-overflow-scrolling: touch;}

Hi @contfedorov yes I upgraded F7 to 3.1.0 also to see if that would resolve it but only the css changes did.
I am still experiencing the same as you with the flickering though - if I find any info on that I will update this thread again.

Hi Guys,
Scrolling is sorted but having a problem with content in the accordion disappearing now.
@Dev guys if I registered for support on one of the contribution pages could this be looked at as a priority for me? I have some tight project deadlines and would happily contribute to the F7 project for an elevated level of support for this issue.
Regards
Paul

@goodfellasthai can you please setup a minimal JSFiddle where I can reproduce this issue? And I will take a look somewhere in the beginning of next week

@nolimits4web - many thanks for your response it is much appreciated and I am going set up a recurring donation on your patreon page today I think you have developed an outstanding product with Framework 7.
The problem with setting up a JSFiddle is I can't replicate the issue on a web browser or even Android, the issue only occurs on iOS with Accordion content, I have uploaded a couple of videos previously of when the page could not scroll, then I added the above css and this fixed my scrolling issue, I suspect it was the touch part of the css that fixed that - however when I added that css the accordion item content disappears from the screen when scrolling now.  I will do another video of that issue happening.
My accordion item content is fairly complex in the sense it has buttons that then display/hide other div sections in the same accordion item so I am not sure if this might be having some impact on the reason for this.  Also I am experiencing the same issue as @contfedorov with the accordion items flickering when selecting.
Your support is much appreciated.
Kind Regards
Paul

@goodfellasthai problem with video is that it is hard to reproduce it. Is this issue replicated in iOS Safari? Or only Cordova app? Maybe you can then setup minimal reproduction repo on GitHub. It can be some minimal Cordova app if it is cordova-only issue so need to check it live to be able to fix it

@nolimits4web I think I have resolved this now using css it also seems to have resolved the accordion flickering issue, using the same css as above use webkit-overflow-scrolling: auto instead of touch @contfedorov maybe you could give it a go see if it works for you.

Guys, still need a some minimal reproduction repo. Would be extremely helpful if some of you can set it up

@contfedorov do you have the ability to do a reproduction, my main problem is all my content is generated from web services and I don't have the ability to do that externally.  If not I could try and mock something together that replicates (if it does on Safari) also I only use an iMac to upload my generated ipa files and not for the actual development process.
Regards
Paul

@goodfellasthai Yes, I will.

@goodfellasthai thank you for your solution, because dropping those lines into my CSS is all it took to fix the problem.
@nolimits4web if you have access to iOS Safari then I can publish something to the web that will recreate the error, unless @contfedorov gets to it first. Let me know if this is what you are looking for to reproduce the issue.

@mbplautz

I can publish something to the web that will recreate the error

That will be good, but i will be able to check it on iOS 12

@nolimits4web please check
https://mbplautz.github.io/app/index.html
I was able to reproduce the issue on my iPhone on iOS 10.

@nolimits4web
Sorry for being late.
Unfortunately, I can't reproduce flickering and content disappearing now. Even via F7 v2.0.8.
Scroll freezing is reproducible, but for v2.0.8 only (v3.1.0 works fine). Please take a look if needed:
https://github.com/contfedorov/f7-issues/tree/accordion_content_issue

I think I figured out why it was happening for me.
It was to do with the pull to refresh, after dynamically creating all my accordion content I issues my app.ptr.done() this is when there was a problem - starting the tab normally or re-visiting the tab everything worked fine but a pull to refresh the problem occurred.
I moved the app.ptr.done() to the start of the dynamic content generation, this means I do not get the ptr pre-loader but once the content is generated the accordion content works fine.

Hello all, there appears to be another bug that was closed with exactly this issue on pull to refresh, I have the latest version of F7 and this is still happening, Bug #1473, to test yesterday I removed the pull to refresh and the scrolling worked fine within the tabs, I added the pull to refresh and got the content stuck/disappearing issue on iOS, this behaviour is not reproducible on the browser.
@nolimits4web will chase up my PM today regarding the project sponsorship.  Can see they still havent done it.

All,
Another update if it helps.  In my case it seems to be the presence pif the ptr-preloader div, causing the scroll sticking and the content disappearing.  (below)

For me, last version of framework7 2.x, iOS 12, iphone 5s, the css did not work.
I had to use javascript to set my content (within .page-content) with height: 100% followed by a setTimeout with height: auto. I guess in my case it was a sync issue between the rendering of elements.

We fixed this problem in version 3.6.2 by adding "before" to .page-content, and giving it the height of the .page-content + 1px. This makes the page always scrollable, so it forces safari to always recalculate the overflowing.
See the following code:
`
.device-ios .page-content::before {
display: block;
content: "";
position: absolute;
z-index: -99999;
top: 0px;
left: 0px;
bottom: -1px;
right: 0px;
}
In the end we wrapped this css inside a .always-scrollable class, so we always have control over which page must be implement this behavior.

@mjwvb thank you! This solved a similar issue for me where the scrolling was frozen when navigating back to an infinite list presenting page.

@mjwvb @nolimits4web I think I found the reason why scrolling is not always reset correctly. .page uses an identity transform. If I remove that, the problem seems to be fixed. I added the following CSS to my project:
.page {
  transform: none;
  -webkit-transform: none;
}

It still seems to work fine with transitions as far as I can tell.

@holtwick nice finding, not sure why, but it works 👍  Pushed this fix to repo

@nolimits4web  Great! I believe it is because those transformations usually trigger hardware accelerated painting, and they get the scrolling out of focus sooner. Therefore, switching back to regular painting modes improves the rendering and layout. But that's just a guess ;) Thanks for adding it!

HI all
I am running my store on Opencart 3 and recently installed an extention for better performance of the website. Basicly it caches the webpages for faster loads. It works like when someone for the first time visits a spacific page it creates a cache and when somebody else visits that page for second, third ..... etc time it loads extremely fast. If a page is cached for the first time by android device iphone devices cannot scroll the page, but if the page was cached by iphone for the first time it can be scroledl without any issues by all other devices.


systemd: systemd-analyze plot doesn't include initrd units

Describe the bug
it's this bug which I've been informed doesn't happen to everyone https://github.com/systemd/systemd/issues/29389
@ElvishJerricco
Steps To Reproduce
The following unrelated reproducer may exhibit the bug, however I'm away from my computer and can't check if it actually does; merely that this bug was annoying me while developing something extremely similar:
https://gist.github.com/lf-/3ad92d06adfec348fa7090e4b5188c51
Just git clone it and nix run :)
Expected behavior
Should see initrd units
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
output here

Add a :+1: reaction to issues you find important.

I'll be taking a look at this...


[TASK] rector: set PHP rule-set to the same version as composer.json

This way rector always follows the minimal required version in composer.json, which currently is PHP 7.2.
This helps with #433.

This now needs a rebase.

And it's rebased.


Add force delete when cleaning up env before testing

First add force delete (graceperiod=0) before testing start, following by this suggestion: https://github.com/fabric8io/kubernetes-client/issues/1195#issuecomment-461041903
I will add more commits for minor updates in this PR.

Looking into today. Sorry for the long delay.

To be on the safe side, this PR is covering force delete options for the build and the buildrun. There is no explicit clean-up before the test, right?

Good suggestion, I've added force clean function in performance pipeline jobs before calling build-load.


Inserting elements from clipboard or library with the grid on, the positions of the elements are all messed up

Try to copy paste the following while having the grid on
https://excalidraw.com/#json=6234903687462912,mLfPmu5OpJnHVptTbI5_Sg
Also if you add this item to the library, same problem exists if add it back to the scene with the grid enabled.

The problem is in how we align items on grid on paste in addElementsFromPasteOrLibrary(). When grid is enabled, we normalize positions of every pasted element so it lands on grid.
But it takes x/y, which in case of a bottom-aligned bar graph will end up misaligning it in case a given bar x/y (which is the top coordinate of the bar) isn't already aligned.

If we top-align it, the pasted elements y positions are untouched (in the case below, x position are still messed up because they weren't on the grid to begin with):

Not sure what we can do here, since the algorithm doesn't know what your intent was.

We can actually do this: check if the elements were copied from a scene with grid enabled, and on paste don't re-align if that is the case.

Actually, scratch that. We shouldn't ever re-align anything but the group as a whole.

Not only if it's a group.. separate elements as well.


Support partially specified writes from case classes

Currently, we cannot use rdd.saveToCassandra on an RDD[CaseClass] if CaseClass does not contain all of the fields from the target table.  This will currently result in the Error:
java.lang.IllegalArgumentException: requirement failed: Columns not found in CaseClass: [missing_field1, missing_field2]
This can be limiting if the table has had columns added to it from an external process.  If this is the case, a job that writes to the table will automatically fail.  This PR proposes a resolution for this issue by making the failure case more permissive (if all of the first n fields of the target table are matched by the case class, then we will attempt the write).

@RussellSpitzer, any word on this?

Do we have a SPARKC ticket for this yet? I couldn't find one in jira?
https://datastax-oss.atlassian.net/issues/

Sorry, I hadn't created one until now.  Here it is: https://datastax-oss.atlassian.net/browse/JAVA-1676

Going to try to trigger jenkins from here

test this please

Tests running!

Build against Scala 2.10 finished with success

Build against Scala 2.11 finished with success

So are we sure we want to aim this patch at Master? I don't think we are going to do another big public release. So if we want to have this capability it would be nice if we could find a way of back-porting it to b1.6 or b2.0 I know we slightly change one method signature but since it's an internal api it should be ok even on the earlier branches... thoughts?

@RussellSpitzer, since it is an internal method with a default parameter, I'm guessing it shouldn't change much?  I'm also not sure about all of the ramifications of doing a backport vs. a normal release.  Would I need to make a new PR that reapplies these changes against b2.0?

Hey @RussellSpitzer, any update on this?

Sorry yeah, you need to change the PR target to b2.0

test this please


Changing settings should discard existing entries if applicable

Testing #145461
Should changing settings clean up local history entries?

Should setting workbench.localHistory.enabled to false delete the local history entries?
Should setting workbench.localHistory.maxFileEntries delete the local history entries that are beyond the limit?

Will collect some more feedback in April from users to see if there is demand. We can always treat this as feature request for the future.
Note that there is an easy way to discard all history entries from the F1 list:

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


License

Xelis-Go-SDK cannot be legally used, since it doesn't have a license.
Whether or not contributions are possible is also unclear.
A license is needed.
This website can help choosing a license. https://choosealicense.com/
For a library like Xelis-Go-SDK, perhaps a good license is MIT, since it's simple and permissive.

MIT added. Closing


Hyundai Ioniq 6: virtual resume press working intermittently

Describe the bug
Right around 15:11:36 – 7 in the shared route, the Jeep in front of me starts to move. I had to press the accelerator pedal to get the car to move even though open pilot was engaged.
Which car does this affect?
Ioniq 6 2023
Provide a route where the issue occurs
a3156f49719f7455|2023-09-18--15-04-23--0
openpilot version
0.9.5
Additional info
Stop and go appears to be effective about half of the time.

Those logs are gone already, can you send a more recent example so I can request upload on full logs?

@thealexbaron any new examples? I couldn't find any recent ones

Closing due to inactivity


Question: Addition of custom labels in the roadmap?

Hello,
Are you considering adding the ability to add custom labels to projects and/or resources?
e.g.:

environment = dev
owner = "joe smith"

etc.
Thanks!

How would Iris learn what the value of  'environment', or of 'owner', should be?

How would Iris learn what the value of 'environment', or of 'owner', should be?

I was thinking from a user-supplied config file.

The preferred way to do this is to launch resources with Terraform and give labels at that time. Typically the CI/CD system is best positioned to have this information.
To do it in Iris, after launching resources, could you please give me a sample of such a config file? Here is one (pseudeocode)
- instance:
   indentifier: //compute.googleapis.com/projects/my-first-project/zones/us-west1/instances/my-first-instance
   label_name: environment
   label_value: billing-department
   label_name: owner
   label_value: John Smith

I am not sure that this makes sense. It would require manually building labels for potentially hundreds of resources, across many projects. Or do you prefer fancier logic in the config files, and if so, what? Some sort of usage of regular expressions?

Sorry, I should have outlined my potential use case.
In a nutshell, it is:  to add required labels to resources that weren't labeled from the start (however those resources may have been created in the first place).
For example, a user stands up a VM using the console or even writes TF to deploy the VM, but forgets to label it. This tool could come along afterwards (say on a schedule) and label the VM with some core labels (env, owner, business unit/cost center). It could/would use the Project that the resource lives in to determine what values the labels should have.
Perhaps a config file could look like:
- project1
    owner: "John Smith"
    cost-center: 12345
    ...
- project2
    owner: "Jane Cooper"
    cost-center: 98765
    ...

(BTW, I completely agree that resources should be labeled from the get-go with your IaC tool!)

It occurred to me that I may be asking too much of Iris3 and that this isn't the intent at all. If so, I understand, but perhaps add some explanation in the README pointing this out.
PS> I really appreciate all the very quick replies!

Sorry for the late reply.
Thanks Joshua for this idea! This would certainly work if inheriting the labels from the Project itself.


Update Trace page layout, add TraceTable from Perses

update trace page layout per mockups
add TraceTable from Perses
add hide/show graph button for ScatterPlot
display Perses native loading and no result states for Perses panels
upgrade Perses from 0.47.0-rc0 to 0.47.0 (no significant changes between rc0 and final version)

Resolves: https://issues.redhat.com/browse/OU-489
Screenshots
Traces found:

No results:

No Tempo instance selected:

/lgtm


testing: output failure report to stderr as per docu instead of stdout

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?
1.6.2
(installed via go1.6.2.linux-amd64.tar.gz at https://golang.org/dl/)

What operating system and processor architecture are you using (go env)?
The OS is Fedora 23 (Workstation Edition).  The go env output doesn't quite capture that fully:

GOARCH="amd64"
GOBIN="/usr/src/go/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/usr/src/go"
GORACE=""
GOROOT="/usr/src/golang-1.6"
GOTOOLDIR="/usr/src/golang-1.6/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

What did you do?
go code https://play.golang.org/p/6GR6cTzVcM
go test case for that code https://play.golang.org/p/ZEJ2iNJMPV

Run a simple test case that fails, outputting a t.Fatal(err).

What did you expect to see?

https://golang.org/pkg/testing/#T indicates that testing logs are accumulated and output to stderr so I expect the t.Fatal() output report to be on stderr.

What did you see instead?

The report output is on stdout.
It seems like something as trivial as https://github.com/tpepper/go/commit/c60fccd1231de145ec81f35b8693000e91eafbe5 (which is on top of tip's changes to allow subtests) could bring the report output in line with the documentation.  But as noted in that tentative commit message, this might break users who have coded to the behavior without reporting the inconsistency.  Perhaps the answer is to change the documentation, except the enabling the distinction between stdout and stderr is actually useful.

./all.bash fails with my above referenced "fix", highlighting existing test cases are indeed coded to expect other than the documented behavior:
--- FAIL: TestVendorTest (0.13s)
	go_test.go:244: running testgo [test -v]
	go_test.go:259: standard output:
	go_test.go:260: PASS
		ok  	vend/hello	0.001s

	go_test.go:263: standard error:
	go_test.go:264: === RUN   TestMsgInternal
		--- PASS: TestMsgInternal (0.00s)
		=== RUN   TestMsgExternal
		--- PASS: TestMsgExternal (0.00s)

	go_test.go:349: missing use in internal test
	go_test.go:350: pattern TestMsgInternal not found in standard output
FAIL
FAIL	cmd/go	22.235s

https://github.com/tpepper/go/commit/bf45959548bb5ec50724bd7ef2583b2c0cdb4653 trivially fixes the test case to match the fixed testcase package.  Then ./all.bash passes.  But everything else in the golang universe might have failures.

It's been this way for a long time, so I think we should change the documentation.


osxphotos crashes instantly

** Before submitting a bug report, please ensure you are running the most recent version of osxphotos and that the bug is reproducible on the latest version **

If you installed with pipx: pipx upgrade osxphotos
If you installed with pip: pip install --upgrade osxphotos
If you installed the pre-built binary, download and install the latest release

Describe the bug
I run the software and it instantly crashes with the error log (attached).
To Reproduce
Steps to reproduce the behavior:

What's the full command line you used with osxphotos?

osxphotos export /Users/mikekenyon/Library/CloudStorage/OneDrive-Personal/Pictures/"Camera Roll" --directory "{created.year}/{created.month}" --exiftool --keyword-template "{label}" --finder-tag-keywords --download-missing  --skip-original-if-edited --db /Volumes/"Extreme SSD"/"Photos Library.photoslibrary" --update --cleanup --skip-live

What was the error output?

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Sonoma 14.1.2
osxphotos version: 0.66.0

Additional context
Add any other context about the problem here.
Please see the crash log here:
osxphotos_crash.log

I don't know what happened but the export database which is used by osxphotos to maintain state between exports appears to be corrupt. Try deleting the file:
"/Users/mikekenyon/Library/CloudStorage/OneDrive-Personal/Pictures/Camera Roll/.osxphotos_export.db"
and trying again.

Ok - that will mean it will re upload etc..?  Guess there isn't much else I can do?

Yeah, it's probably actually best to start over with a fresh export directory and let osxphotos re-export everything. I don't know what happened -- I've not seen that error before -- but it appears osxphotos is trying to open the export database which is malformed and causing the crash. Was this working before for you?

Here's the actual error:
sqlite3.DatabaseError: database disk image is malformed

Yes, it's been working fine for many months actually - this is my first error.   Just seen that error message - not good.  I have quite a large export so it takes quite a while.  Oh well, it is what it is.
Thanks for the quick response.
Mike

Before you do that, perhaps this is worth a try:
sqlite3 "/Users/mikekenyon/Library/CloudStorage/OneDrive-Personal/Pictures/Camera Roll/.osxphotos_export.db" ".recover" | sqlite3 "/Users/mikekenyon/Library/CloudStorage/OneDrive-Personal/Pictures/Camera Roll/.osxphotos_export_new.db"
If that works, then copy the new recovered database to the old database and start the export again.

If the above command is successful, you would then run:
cp "/Users/mikekenyon/Library/CloudStorage/OneDrive-Personal/Pictures/Camera Roll/.osxphotos_export_new.db"  "/Users/mikekenyon/Library/CloudStorage/OneDrive-Personal/Pictures/Camera Roll/.osxphotos_export.db"

Now then good sir! That has worked :-) You need to put that in the Wiki as a possible fix for that error.  It did some "processing.." steps when I re-ran osxphotos but it's catching up now.  Will let you know if it completes successful.
Mike

Great! I'll catch that specific error and have osxphotos give a suggestion then also put in the docs.

All completed successfully :-)
Cheers
Mike

Add this to exportdb as osxphotos exportdb --repair

Added osxphotos exportdb --check and osxphotos exportdb --repeair , will be released in v0.67.0

@all-contributors please add @mikekenyon99 for ideas, bug


[Epic] Expand OC Data model

What product(s) is this story for?
No response
As a user, I want
No response
Description
Tentative placeholder Epic made available to @gaiaandreoletti to track the tasks related to the development of the OC Data Model
Acceptance criteria
No response
Tasks
No response
Anything else?
No response
Have you linked this story to a GitHub Project?

[X] I have linked this story to a GitHub Project and set its metadata.

@gaiaandreoletti Let's discuss about this Epic and task tracking in our next 1:1.


Migration must not use try/catch in code

PHP Version: 8.2.23   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Expected behaviour:
SQL migration that is not executed correctly should display an error.
Actual behaviour:
The migration code uses try/catch
https://github.com/shopware/shopware/blob/59c811373a4a91fc5044a018575bc8d1516958f5/src/Core/Migration/V6_6/Migration1707064042CartRemoveFK.php#L72
After Update, all migrations correctly done. But column definitions null was missing.
How to reproduce:
Cart table with many thousand datasets

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39026
[created from NEXT-39026, comment 504212]


Migrate to Swift 3.0

The default migration tool to Swift 3.0 in Xcode 8 beta leaves a few errors in SwiftyJSON, most notably the use of DictionaryIndex. It'd be nice to have an updated SwiftyJSON source available for Swift 3.0.
Thanks!

See the code in swift3 branch

Thanks for that, @lingoer. Just wanted to give you a heads up for that branch's code: In "SwiftyJson.swift", for lines no. 739 and 740, there is a syntax error. Make sure you change that. Many thanks!


[QUERY] How to use Proxy with V12 sdk for blob storage?

Query/Question
How to pass Proxy information in new sdk blob storage?
Why is this not a Bug or a feature Request?
I'm hoping this to be present since it was present in V10.5.
This is how I setup proxy with V10.5
PipelineOptions po = new PipelineOptions();
HttpClientConfiguration configuration = new HttpClientConfiguration(
new Proxy(Proxy.Type.HTTP, new InetSocketAddress(ProxyHost, ProxyPort)));
po.withClient(HttpClient.createDefault(configuration));
Setup (please complete the following information if applicable):

OS: [e.g. iOS] iOS
IDE : [e.g. IntelliJ] Intellij
Version of the Library used: V12

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[x] Setup information Added

@rickle-msft  This option is deprecated in azure-core-1.0.0-preview.4 coming as part of azure-storage-blob-12.0.0-preview.3.jar.

Hi @vmaheshw,
During the last release of Azure Core and the Storage SDK we made the HTTPClient pluggable and released two plugin packages (Azure Core Http Netty and Azure Core Http OkHttp), our recommendation is to use the Netty implementation.
Using the Netty plugin add this dependency to your project
<dependency>
  <groupId>com.azure</groupId>
  <artifactId>azure-core-http-netty</artifactId>
  <version>1.0.0-preview.4</version>
</dependency>

And this is how you would create the same client that @rickle-msft mentioned above using the Netty plugin.
HttpClient client = new NettyAsyncHttpClientBuilder()
    .proxy(new ProxyOptions(ProxyOptions.Type.HTTP, new InetSocketAddress("localhost", 8888)))
    .build();

And for the OkHttp plugin
<dependency>
  <groupId>com.azure</groupId>
  <artifactId>azure-core-http-okhttp</artifactId>
  <version>1.0.0-preview.4</version>
</dependency>

HttpClient client = new OkHttpAsyncHttpClientBuilder()
    .proxy(new Proxy(Proxy.Type.HTTP, new InetSocketAddress("localhost", 8888)))
    .build();

If you have any additional questions on the new pattern please let me know.

@vmaheshw, it looks like this is sorted out now.  Feel free to re-open if you have further questions on this.

HttpClient client = new NettyAsyncHttpClientBuilder()
    .proxy(new ProxyOptions(ProxyOptions.Type.HTTP, new InetSocketAddress("localhost", 8888)))
    .build();

At the moment preview 4 the proxy on the httpclient has been removed
I guess you need to to something like this now:
Configuration config = new Configuration();
config.put(Configuration.PROPERTY_HTTPS_PROXY, "myproxyserver:8080");
...
        BlobContainerClient blobContainerClient = new BlobContainerClientBuilder()
                .endpoint(azureSasUrl)
                .containerName(azureContainerName)
                .configuration(config)
                .httpClient(client)
                .buildClient();


Hyundai: Add FW for 2021 Hyundai Ioniq PHEV

Add firmware for the 2021 Hyundai Ioniq PHEV. Expand supported model-year range.
Dongle ID: 853c3e8e72d9748c
Thanks to community Ioniq PHEV owner rambor.

Thanks for the PR, next time can you post the route instead of the dongle ID?


Disable issuing refresh token per service provider

In the service provider level, if refresh token grant is disabled, IS should not issue refresh tokens for that application.

Currently, even though you have disabled the refresh grant SP wise, you will get a refresh token but you can't use the retrieved refresh token.


Update L1CrossDomainMessenger to v2.3.0

:clipboard: Add associated issues, tickets, docs URL here.
Overview
Describe what your Pull Request is about in a few sentences.
Update L1CrossDomainMessenger to v2.3.0
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Deploy the contract and update the deployment file

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.62%. Comparing base (c339901) to head (5443071).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #152       +/-   ##
===========================================
- Coverage    18.95%   0.62%   -18.33%
===========================================
  Files          158      94       -64
  Lines         6226    2396     -3830
  Branches      1311     546      -765
===========================================
- Hits          1180      15     -1165
+ Misses        4971    2381     -2590
+ Partials        75       0       -75

Flag
Coverage Δ

chain-mon-tests
?

common-ts-tests
?

contracts-bedrock-tests
0.62% <ø> (ø)

contracts-ts-tests
?

core-utils-tests
?

sdk-next-tests
?

sdk-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 64 files with indirect coverage changes

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 28.21%. Comparing base (c339901) to head (1481d71).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #152      +/-   ##
===========================================
+ Coverage    18.95%   28.21%   +9.26%
===========================================
  Files          158      168      +10
  Lines         6226     7294    +1068
  Branches      1311     1311
===========================================
+ Hits          1180     2058     +878
- Misses        4971     5130     +159
- Partials        75      106      +31

Flag
Coverage Δ

cannon-go-tests
82.20% <ø> (?)

chain-mon-tests
27.14% <ø> (ø)

common-ts-tests
26.72% <ø> (ø)

contracts-bedrock-tests
0.62% <ø> (ø)

contracts-ts-tests
12.25% <ø> (ø)

core-utils-tests
44.03% <ø> (ø)

sdk-next-tests
38.59% <ø> (ø)

sdk-tests
38.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 10 files with indirect coverage changes


The codes of Cloud-Robotic AI Benchmarking for Edge-cloud Collaborative Lifelong Learning

What type of PR is this?
This PR provides codes for PR#62

What this PR does / why we need it:
It provides a benchmark example of semantic segmentation for lifelong learning in Ianvs.

@luosiqi might want to take a look at it

I suggest that the commits can be compressed by using "git rebase -i".

As you have modified core codes of ianvs, make sure it works for other paradigms such as single task learning and incremental learning.

I suggest that the commits can be compressed by using "git rebase -i".

OK, I have compressed the commits.

It seems that the CI testing is not passed.

/lgtm

Please modify your codes according to CI. It seems that your codes don't meet the coding standard.

Please modify your codes according to CI. It seems that your codes don't meet the coding standard.

I have modified the codes to meet the coding standard.

/lgtm


Larger "small" heuristics for smaller max_width

I'm not sure where to put this.  It is kind of related to #2299, #2447, and #2437.  But I might be happier with a simpler solution than whats suggested there, so I open a specific issue.
I use max_width = 78 (since I want a git diff of my code to fit within an 80 chars wide terminal), and I want "short" struct literals and if-else statements to be single line, so I leave use_small_heuristics to its default true value.
Since most of the the default WidthHeuristics are defined in relation to max_width, which defaults to 100, I think many of the heuristics simply get too small when using a smaller max_width.
So my intent was to suggest non-linear values in WidthHeuristics::scale (maybe simply leaving it as is for max_width >= 90 and otherwise using e.g. values 85, 25, 50, and 75 instead of 60, 18, 35, and 50).
But maybe something like #2447 is a better alternative?  Maybe it can be generalized to having different named set of heuristics?  On the other hand, less configuration is better, so I rather like the concept that the heuristics should attempt to adapt as good as possible to the given max_width.

We will restore the per-item width configuration in 2.0.


Log shim panic logs in containerd.

Currently hcsshim writes the shim panic logs in a file named panic.log inside the sandbox
directory. However, those logs are never logged in containerd and they get lost when the
sandbox container is removed. This change allows the shim to log these panic logs to
containerd before deleting them.
Signed-off-by: Amit Barve<EMAIL_ADDRESS>

Do we no longer need a containerd PR here?

Nope. Whatever is written on the stderr of the shim binary delete instance will be logged by containerd automatically.

Sweet ok!

Can you make sure you have tested by inserting panic at various points in the code, and made sure the stack is correctly picked up by delete and re-logged via ETW? We should be able to see an event from the runhcs ETW provider with the panic stack.

@kevpar I have tested this by adding panic at different locations in hcsshim code and verified that the panic log shows up in both ETW and containerd events. Can you please take another look?


Comment like "# " gets flagged as 100% valid code

<?php
# https://github.com/WordPress-Coding-Standards/WordPress-Coding-Standards/issues/new

gets flagged with This comment is 100% valid code; is this commented out code?. Hm?

This comes from PHP_Codesniffer.
WP code standard is to use the // style of inline comments, not the # format, which we may or may not actually pick up on.

WP code standard is to use the // style of inline comments, not the # format, which we may or may not actually pick up on.

We do.
@lkraav I'm going to close this since that sniff is maintained by PHPCS. What version of PHPCS are you using? This might have been fixed in https://github.com/squizlabs/PHP_CodeSniffer/commit/980ce82a133d31a47f5c6d720fc458f31e451d46

What version of PHPCS are you using?

After filing this issue, an upgrade 2.3.0 -> 2.5.1 became available in portage. I think upgrading may have solved this.


Feat: Load collected morphologies using NRN order

If I understood well, the test fails because loading the morph this way remoses the additional annotations in the .asc file, and the ExtractMarker does not produce the pia.yaml file with the extracted markers. Probably my PR is better for this. Why did you need this one originally?

Partially fixed in #104


#363 viewcolumn expression

Handles the main feature as well as 3 bug fixes:
#363 New feature to add "expression" capabilities to view columns
#396 Negative default values are not see by migration steps and cause migration to always occue over and over again.
#395 GroupBy logic for aggregate views change from column name-based to 1,2,3,4... based.
#394 XXX_Json.Write() method doesn't handle not-null columns with number defaults

Further fixes for #363 and new fixes for #398 where aggregate columns are not handled properly for realized tables.


Linear Regression Implementation from scratch

Hi. Thank for your contribution, but unfortunately, we already have an implementation of linear regression: https://github.com/rushter/MLAlgorithms/blob/master/mla/linear_models.py
If you want to improve it or add something that is missing, please update that file and follow the same code style and module organization.


[State Restoration] Adds remaining Restorable{Property}N

Description
Introduces RestorableNumN, RestorableDoubleN, RestorableIntN, and RestorableStringN.
RestorableBoolN was already added and figured we might as well add the remaining properties so that we're consistent throughout in the framework.
Related Issues
https://github.com/flutter/flutter/issues/62916, Gallery app uses workaround for RestorableInt for the choice chip and data table demoes when it makes more sense to use a RestorableIntN.
Tests
I added the following tests:

Modified existing tests to test for the added properties.

Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

Manually set Google testing check to pass because the test seems unrelated/is a flake.


RATIS-953. XML Parsers should not be vulnerable to XXE attacks

What changes were proposed in this pull request?
Prevent external XML entities attacks:

Turn on "secure processing" for XML
Disable external DTD/schema explicitly

SonarCloud and SonarLint display slightly different instructions for fixing XXE.  This change applies both.
https://issues.apache.org/jira/browse/RATIS-953
How was this patch tested?
Existing unit tests.
https://github.com/adoroszlai/incubator-ratis/runs/771645855

Reopen pr to trigger ci.

@adoroszlai   Hi, why this PR run jenkins, but the other PR do not run jenkins ?

@adoroszlai Thanks the patch. @dineshchitlangia @bshashikant Thanks for review. I have merged the patch.

Thanks @runzhiwang for triggering CI and merging it.


Fix invalid date selection issue for e2e_tests

Description
The cypress test is selecting a day from the date picker for the current month. This will fail when the date is a weekend or holiday. Assuming that it's not possible to hard code the date and enter the date and that have to interact via the date picker (as I think that would probably be the easiest way to resolve)?
Reviewer Notes
Wasn't sure the best way to resolve this. Also, not really sure what's been tried here, so if you all could weigh in that would be appreciated. Two commits with two different approaches:

https://github.com/transcom/mymove/compare/34b5ab1...e9188bc3109009fb1db986f11f9bcb31ea737b9f make a call to the available_move_dates endpoint and uses the result. Maybe overly complicated for what we're trying to do here, but uses our holiday calendar / don't have to maintain another calendar.
https://github.com/transcom/mymove/compare/34b5ab1...affb732cae516bded3e13daa2d080035d8362aa2 uses moment-business-days. This is simpler, but requires that you add a holiday calendar, that we'd have to update going forward, whereas that's already being done in the available_move_dates endpoint.

*** Updated ***
Was directed to this story. This specific test was failing due to the shipments model validation rules. Adding a third option (remove the actual delivery date DateIsWorkday validation rule), but didn't have the context to know if that was correct to remove it entirely.

https://github.com/transcom/mymove/compare/34b5ab1...23087f297feae34fc23a6dc1db67e2f1cebec0dc remove shipment model validation

Setup
make e2e_test

Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

I prefer the use of the available_move_dates endpoint because it keeps our maintenance burden low for the holiday stuff.

If we already have a way to calculate holidays, I'd use that... As Matt said, prevents us from having to maintain another holiday calendar.
However, w/ the update of the business rules seems like we could get rid of testing for weekend dates all together since the TSP can choose a weekend.
Question: Are TSPs allowed to choose a date that might fall on a holiday? Do we still need a check for that?

I'm of two minds about this:
The pragmatic mind says that we are moth-balling HHGs and should do the least amount of work possible to get this test passing. That is, we could remove the verification entirely.
The perfectionist says that we want to check for weekends and holidays if the request is from a service member. It also say that the business rules for pickup and packing date should be corrected too.
For this story, I prefer the available_move_dates, since a) you have already done the work and b) it will be resilient to any changes we make to that business logic going forward.

thanks for the feedback everyone. seems like the consensus is to give the available_move_dates approach a shot. updated PR to use that.


Warning when running Boxen on 10.11/El Capitan

I have installed El Capitan (OS X 10.11) on one of my Macs. Only I am getting the warning:
You must be running one of the following OS X versions: 10.8 10.9 10.10.
Any plans to support Boxen on 10.11?

What version of the boxen gem are you using? 2.8.0 added "support" for 10.11: https://github.com/boxen/boxen/blob/2.8.0/lib/boxen/preflight/os.rb

Yes, I noticed that too. Looks like our-Biden when you check it out now is using an older version. Made it work now!
Thanks
Sent from my iPhone

On 14 Sep 2015, at 13:50, Les Aker<EMAIL_ADDRESS>wrote:
What version of the boxen gem are you using? 2.8.0 added "support" for 10.11: https://github.com/boxen/boxen/blob/2.8.0/lib/boxen/preflight/os.rb
—
Reply to this email directly or view it on GitHub.


Form layout inside combo box problem

Straight problem when I place my form layout inside combo box, the first time when I click on combo box to show its items, everything is fine, then if I retry, every items in combo box have a small space between each other.
here is the first time if I click:

But then if click again:

When this problem happen, try to scroll fast to bottom of items list, it will back to normal:
https://user-images.githubusercontent.com/39910970/115985109-7b95d980-a5d4-11eb-9b74-0d2442023e06.mp4

My java code:
`private ComponentRenderer<VerticalLayout, Datum> createDesktopDelegateListComboBox() {
return new ComponentRenderer<>(delegate -> {
String formattedWeight = String.format("%.2f", Long.parseLong(delegate.getDelegateStatistics().getVoting_power()) / 100000000F);
        Element subRanking = new Html("" + delegate.getRank() + "").getElement();
        Span delegateNameWithRank = new Span(delegate.getName());
        delegateNameWithRank.getElement().appendChild(subRanking);
        Span payout = new Span(String.valueOf(delegate.getPayout_percent()));
        Span voteWeight = new Span(formattedWeight);

        delegateNameWithRank.setId("content-s");
        payout.setId("content-s");
        voteWeight.setId("content-s");

        FormLayout formLayout = new FormLayout();
        formLayout.setResponsiveSteps(
                new FormLayout.ResponsiveStep("0px", 1, FormLayout.ResponsiveStep.LabelsPosition.ASIDE),
                new FormLayout.ResponsiveStep("550px", 3, FormLayout.ResponsiveStep.LabelsPosition.TOP));
        formLayout.addFormItem(delegateNameWithRank, "Name");
        formLayout.addFormItem(payout, "Shares");
        formLayout.addFormItem(voteWeight, "Vote weight");

        return new VerticalLayout(formLayout);
    });
}

public void init() {
    myCombobox.setRenderer(createDesktopDelegateListComboBox());
}
`

workaround:
If I remove "new FormLayout.ResponsiveStep("550px", 3, FormLayout.ResponsiveStep.LabelsPosition.TOP)"
and become
formLayout.setResponsiveSteps(new FormLayout.ResponsiveStep("0px", 3, FormLayout.ResponsiveStep.LabelsPosition.TOP));
Then the problem is gone.

Hi, the FormLayout component is not intended to be used inside the dropdown of a ComboBox. While it's technically possible to do, it's not a supported use case. I suggest creating a more lightweight html-based template to achieve the same item rendering.


Launchcontainers:  A Python tool for launching containerized analysis on HPC

Title
Launchcontainers: A Python tool for launching containerized analysis on HPC
Leaders
Yongning Lei
email<EMAIL_ADDRESS>Collaborators
No response
Brainhack Global 2023 Event
Brainhack Donostia
Project Description
Launchcontainers is a Python-based program built to automatically launch containerlized MRI processing pipelines. This program takes one config.ymal file, one container.json file, and one subject-session-list.txt file as inputs. Using 1 line of bash command, it will automatically send jobs to HPC clusters regarding your computing demands.
This program is well-suited for multi-subject, multi-scan datasets. And it will save a lot of time if you need analysis your entire dataset with different parameters multiple times.
In future versions, we are trying to add more functionality to this repository so that you can launch all the MRI data analysis pipelines such as Heudiconv, fMRIprep, pRF pipelines and etc. Please keep track of this repo and if you have any questions or suggestions, don't hesitate to contact Gari<EMAIL_ADDRESS>and Tiger<EMAIL_ADDRESS>Link to project repository/sources
https://github.com/garikoitz/launchcontainers
Goals for Brainhack Global

Let more people get to know the advantages of using containers to process MRI data to improve the data availability and reproducibility
Let beginner level users get familiar with the UNIX operation and git operation
Add new features to launchcontainer so that it supports more MRI processing pipeline (ie. HeuDiconv, fMRIprep, PRF-analyze)
Make a better documentation on the project Wiki

Good first issues

issue one:
https://github.com/garikoitz/launchcontainers/issues/28
issue two:
https://github.com/garikoitz/launchcontainers/issues/32

Communication channels
https://mattermost.bcbl.eu/bhdonostia/channels/launchcontainer
Skills

Python intermediate
Unix intermediate

Onboarding documentation
https://docs.google.com/document/d/1XMmMpp3w7i2o8wVyHv_kUgoTgYVJ7n-qvgIQilfEBEU/edit?usp=sharing
What will participants learn?

How to use Unix to run python script, navigate to directory, and edit files.
How to use git to collaborate with other's, create branch to fix issues, pull-request
How to use launchcontainers to launch DWI pipeline on SGE, SLURM and local machine

Data to use
I will prepare the converted nifti DWI scan folder in BIDS format and zipped it for use
Number of collaborators
3
Credit to collaborators
Project contributers will be listed in the launchcontainer repo
Image

Type
documentation coding_methods
Development status
2_releases_existing
Topic
reproducible_scientific_methods
Tools
BIDS
Programming language
Python
Modalities
DWI
Git skills
1_commit_push, 2_branches_PRs
Anything else?
No response
Things to do after the project is submitted and ready to review.

[X] Add a comment below the main post of your issue saying: Hi @brainhackorg/project-monitors my project is ready!
[x] Twitter-sized summary of your project pitch.

Hi @brainhackorg/project-monitors my project is ready!

hey @yongninglei
could you add or tag at least 2 issues in https://github.com/garikoitz/launchcontainers that you want to work on during the hackathon ?
for example have issues that match your "Goals for Brainhack Global" section, so that people who want to help you can at elast say they want to work on this and discuss progress / problems...

Hey @Remi-Gau
I have add a tag in the repo   called for brainhackdonostia 2023 and there are 7 issues ready.

perfect

Hi @Remi-Gau , I attached the launchcontainer logo to this issue just now :DDD maybe it can be added to the web?

OK testing if relabelling the issue will just reopen a PR with image (that's me being lazy here).

ok that did not work: elbow grease seems to be required...

The image appears in the project page but not as a thumbnail on the listing of projects here: http://brainhack.org/global2023/project/project_44/
Not sure why

quick q: does the project need a localisation label? Such as donostia?

I think we had them on the previous years but I have not generated the labels


Abayol Peter

https://github.com/zuri-training/AuthWiki_Team4/issues/71
https://github.com/zuri-training/AuthWiki_Team4/issues/35
https://github.com/zuri-training/AuthWiki_Team4/issues/72
https://github.com/zuri-training/AuthWiki_Team4/issues/76


what's means of pts_3d, rot_angles, trans?

as i have test in DECA, i just get "pose(6),euler_jaw_pose,verts,trans_verts,landmarks2d,landmarks3d,landmarks3d_world and other for rendering resources" but have not found rot_angles, trans, and the dimension is not croresspond. can u explain its means, thx very much.

i have read the paper again, pts_3d is facial lamdmark in object coordinate, rot_angles, trans are head pose, rotation and tranlate, which also in object coordinate. is this right?

pts_3d is [-1, 1], rot_angles is [-\pi, \pi], trans is (so large)?

Yes
range is not important, it depends on your data and how you process it.