[gts] Build failure on x64-osx

Host Environment

OS: macOS Monterey 12.2
Compiler: Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)

To Reproduce
Steps to reproduce the behavior:
./vcpkg install gts
Failure logs
console
❯ ./vcpkg install gts
Computing installation plan...
The following packages will be built and installed:
    gts[core]:x64-osx -> 0.7.6#4
Detecting compiler hash for triplet x64-osx...
Restored 0 packages from /Users/xxxx/.cache/vcpkg/archives in 31.49 us. Use --debug to see more details.
Starting package 1/1: gts:x64-osx
Building package gts[core]:x64-osx...
-- Note: gts only supports dynamic library linkage. Building dynamic library.
-- Using cached finetjul-gts-c4da61ae075f355d9ecc9f2d4767acf777f54c2b.tar.gz.
-- Cleaning sources at /Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /Users/xxxx/vcpkg/downloads/finetjul-gts-c4da61ae075f355d9ecc9f2d4767acf777f54c2b.tar.gz
-- Applying patch fix-M_PI-in-windows.patch
-- Applying patch support-unix.patch
-- Applying patch fix-pkgconfig.patch
-- Applying patch glib2.patch
-- Using source at /Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean
-- Configuring x64-osx-dbg
-- Configuring x64-osx-rel
-- Building x64-osx-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:157 (message):
    Command failed: /Users/xxxx/vcpkg/downloads/tools/cmake-3.21.1-osx/cmake-3.21.1-macos-universal/CMake.app/Contents/bin/cmake --build . --config Debug --target install -- -v -j5
    Working Directory: /Users/xxxx/vcpkg/buildtrees/gts/x64-osx-dbg
    See logs for more information:
      /Users/xxxx/vcpkg/buildtrees/gts/install-x64-osx-dbg-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_build_cmake.cmake:106 (vcpkg_execute_build_process)
  scripts/cmake/vcpkg_install_cmake.cmake:44 (vcpkg_build_cmake)
  ports/gts/portfile.cmake:22 (vcpkg_install_cmake)
  scripts/ports.cmake:142 (include)

Error: Building package gts:x64-osx failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `git pull` and `./vcpkg update`.
Then check for known issues at:
  https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+gts
You can submit a new issue at:
  https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[gts]+Build+error
including:
  package: gts[core]:x64-osx -> 0.7.6#4
    vcpkg-tool version: 2021-12-09-724cea8b33cbf06c645f5095fa29773697da9761
    vcpkg-scripts version: 07e508359 2022-01-27 (17 hours ago)

Additionally, attach any relevant sections from the log files above.

logs
[21/38] /Library/Developer/CommandLineTools/usr/bin/cc -DGTS_COMPILATION -Dgts_EXPORTS -I/Users/xxxx/vcpkg/buildtrees/gts/x64-osx-dbg -I/Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean -I/Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean/src -isystem /Users/xxxx/vcpkg/installed/x64-osx/debug/lib/pkgconfig/../../../include/glib-2.0 -isystem /Users/xxxx/vcpkg/installed/x64-osx/debug/lib/pkgconfig/../../../include -isystem /Users/xxxx/vcpkg/installed/x64-osx/debug/lib/pkgconfig/../../lib/glib-2.0/include -fPIC -g -arch x86_64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX12.1.sdk -fPIC -DPCRE_STATIC -MD -MT CMakeFiles/gts.dir/src/predicates.c.o -MF CMakeFiles/gts.dir/src/predicates.c.o.d -o CMakeFiles/gts.dir/src/predicates.c.o -c /Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean/src/predicates.c
FAILED: CMakeFiles/gts.dir/src/predicates.c.o
/Library/Developer/CommandLineTools/usr/bin/cc -DGTS_COMPILATION -Dgts_EXPORTS -I/Users/xxxx/vcpkg/buildtrees/gts/x64-osx-dbg -I/Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean -I/Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean/src -isystem /Users/xxxx/vcpkg/installed/x64-osx/debug/lib/pkgconfig/../../../include/glib-2.0 -isystem /Users/xxxx/vcpkg/installed/x64-osx/debug/lib/pkgconfig/../../../include -isystem /Users/xxxx/vcpkg/installed/x64-osx/debug/lib/pkgconfig/../../lib/glib-2.0/include -fPIC -g -arch x86_64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX12.1.sdk -fPIC -DPCRE_STATIC -MD -MT CMakeFiles/gts.dir/src/predicates.c.o -MF CMakeFiles/gts.dir/src/predicates.c.o.d -o CMakeFiles/gts.dir/src/predicates.c.o -c /Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean/src/predicates.c
In file included from /Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean/src/predicates.c:129:
/Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean/src/rounding.h:28:14: fatal error: 'fpu_control.h' file not found
#    include <fpu_control.h>
             ^~~~~~~~~~~~~~~
1 error generated.

/Users/xxxx/vcpkg/buildtrees/gts/src/f777f54c2b-f8f38b8ae0.clean/src/rounding.h
/* GTS - Library for the manipulation of triangulated surfaces
 * Copyright (C) 1999 St�phane Popinet
 *
 * This library is free software; you can redistribute it and/or
 * modify it under the terms of the GNU Library General Public
 * License as published by the Free Software Foundation; either
 * version 2 of the License, or (at your option) any later version.
 *
 * This library is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.	 See the GNU
 * Library General Public License for more details.
 *
 * You should have received a copy of the GNU Library General Public
 * License along with this library; if not, write to the
 * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
 * Boston, MA 02111-1307, USA.
 */

#include "config.h"

/* disable _controlfp on 64bit machines */
#ifdef _WIN64
#  define FPU_ROUND_DOUBLE
#  define FPU_RESTORE
#else
#  ifdef HAVE_FPU_CONTROL_H
#    include <fpu_control.h>
#    ifdef _FPU_EXTENDED
#     if !defined(__alpha__) || !defined(__GLIBC__)
#      if defined(__arm__)
         static fpu_control_t fpu_round_double = _FPU_DEFAULT;
#      else
         static fpu_control_t fpu_round_double =
           (_FPU_DEFAULT & ~ _FPU_EXTENDED)|_FPU_DOUBLE;
#      endif
       static fpu_control_t fpu_init;
#      define FPU_ROUND_DOUBLE  { _FPU_GETCW(fpu_init);\
                                  _FPU_SETCW(fpu_round_double); }
#      define FPU_RESTORE       {_FPU_SETCW(fpu_init);}
#     else /* __alpha__ && __GLIBC__ */
#       define FPU_ROUND_DOUBLE
#       define FPU_RESTORE
#     endif /* __alpha__ && __GLIBC__ */
#    else /* not FPU_EXTENDED */
#      define FPU_ROUND_DOUBLE
#      define FPU_RESTORE
#    endif /* not FPU_EXTENDED */
#  else /* not HAVE_FPU_CONTROL_H */
#    ifdef __FreeBSD__
#      include <floatingpoint.h>
#      define FPU_ROUND_DOUBLE  (fpsetprec(FP_PD))
#      define FPU_RESTORE       (fpsetprec(FP_PE))
#    else /* not __FreeBSD__ */
#      ifdef WIN32
#        ifdef _MSC_VER
#          include <float.h>
           static unsigned int fpu_init;
#          define FPU_ROUND_DOUBLE (fpu_init = _controlfp (0, 0),\
                                   _controlfp (_PC_53, MCW_PC))
#          define FPU_RESTORE      (_controlfp (fpu_init, 0xfffff))
#        elif __MINGW32__
#          include <float.h>
           static unsigned int fpu_init;
#          define FPU_ROUND_DOUBLE (fpu_init = _controlfp (0, 0),\
                                    _controlfp (_PC_53, _MCW_PC))
#          define FPU_RESTORE      (_controlfp (fpu_init, 0xfffff))
#        else /* not _MSC_VER or __MINGW32__ */
#          error "You need MSVC or MinGW for the Win32 version"
#        endif /*  not _MSC_VER or __MINGW32__ */
#      else /* not WIN32 */
#        ifdef __CYGWIN__
           typedef unsigned int fpu_control_t __attribute__ ((__mode__ (__HI__)));
           static fpu_control_t fpu_round_double = 0x027f;
           static fpu_control_t fpu_init;
#          define _FPU_GETCW(cw) __asm__ ("fnstcw %0" : "=m" (*&cw))
#          define _FPU_SETCW(cw) __asm__ ("fldcw %0" : : "m" (*&cw))
#          define FPU_ROUND_DOUBLE  { _FPU_GETCW(fpu_init);\
                                    _FPU_SETCW(fpu_round_double); }
#          define FPU_RESTORE       { _FPU_SETCW(fpu_init);}
#        else /* not __CYGWIN__ */
#          ifdef CPP_HAS_WARNING
#            warning "Unknown CPU: assuming default double precision rounding"
#          endif /* CPP_HAS_WARNING */
#          define FPU_ROUND_DOUBLE
#          define FPU_RESTORE
#        endif /* not __CYGWIN__ */
#      endif /* not WIN32 */
#    endif /* not __FreeBSD__ */
#  endif /* not HAVE_FPU_CONTROL_H */
#endif

It is strange that HAVE_FPU_CONTROL_H is defined in mac.
Additional context

@acd1034
Thanks for posting this issue.
Currently this port is broken on osx in vcpkg. Please see
https://github.com/microsoft/vcpkg/blob/6f1d72a4bd98b635c98691054864229aed61fa82/scripts/ci.baseline.txt#L375

@acd1034 Is still work for this issue?

@acd1034 Thanks for posting this issue. Currently this port is broken on osx in vcpkg. Please see
https://github.com/microsoft/vcpkg/blob/6f1d72a4bd98b635c98691054864229aed61fa82/scripts/ci.baseline.txt#L375

This means gts cannot be built with vcpkg because of gts’s side, right?

@acd1034 Yes, currently gts does not support on tripletx64-osx

@acd1034 Of course, if you need to build on osx, you can submit an issue to the Upstream:(https://github.com/finetjul/gts) to support it. After the upstream is fixed, we will update it in the next version. :)

I will try submit a PR  and add a feature to support osx. :)

@acd1034 Sorry, gts source include a lot of file <fpu_control.h>, it looks like they don't plan to support Apple, we can't change the upstream support architecture. But I found a way to disable this function, hope it can help you. see here :)

@acd1034 I will close this issue and if there are any other questions please contact me and I will reopen it. :)


Does the AMD EPYC CPU support QAT_ENGINE？

I want to use the ipp and ipsec in the qat_engine SW to accelerate openssl, my CPU is AMD EPYC series, is it supported?

Hi @damoncui1993 , QAT Engine does not support IPP-crypto and IPsec operations on AMD CPUs.


Impossible to get object for element if it's inside an iframe

Hi, @cheezy, I've trapped into the issue described here.
I need to have access to generated methods available for TableRow objects, and the only way is to get them using 'each' (or some other) for Table object.
Table object is available after using some_name_element like method. And my table is inside iframe, so I'm getting the error: 'Element belongs to a different frame than the current one – switch to its containing frame to use it'

any update on this ?

Any update?
----O--(_)---O---

I was able to get around it in Selenium by first assigning the element object to a local variable and then using 'switch_to' to get around the framing problem. This could possibly be fixed by overriding the find_element method for Selenium...? I haven't tried that yet.
ele = page.name_element
browser.switch_to.frame('your-frame-name')
puts ele.text  # or whatever method for the element that is needed
browser.switch_to.default_content
You must switch back to default_content. If you write helper routines, you may need to have rescue and possibly ensure blocks to do the switch back to default_content, else the next find_element will fail.

This issue is fixed with the recent page-object changes. If anyone experiences these issues with the newest version of page object, feel free to reopen.


Remove or change the Batch Gift Entry reference text in NPSP Settings

Issue Summary
Batch Gift Entry in NPSP was retired in version 3.202 - effective in production orgs from 13 July 2021. See announcements here:

https://powerofus.force.com/s/feed/0D57y00000JHFm5CAH
https://powerofus.force.com/s/feed/0D51E00005acDV1SAM

With the retirement, users can still access Batch Gift Entry records but they are no longer able to process these records using BGE.
The issue is the STG_PanelDataImportAdvancedMapping page within NPSP Settings (NPSP Settings --> System Tools --> Advanced Mapping for Data Import & Gift Entry) still contains the following text which is confusing and misleading for users, as it suggests it is possible to continue to use Batch Gift Entry instead of Gift Entry.
"If preferred, you can disable Gift Entry and instead use older Batch Gift Entry (or Single Gift Entry, if using GEM)."
The text is stored in the geBodyGiftEntryHelp label and associated translation files:
https://github.com/SalesforceFoundation/NPSP/blob/main/force-app/main/default/labels/CustomLabels.labels-meta.xml#L6737

W-9968552

Please follow this Known Issue in the Trailblazer Community for updates.


Freezing on progress bar (booting off USB, not SSD)

Taken all of the bios settings, the OpenCore guide, and your EFI folder (along with generating an SMBIOS) into account, and got the Apple logo up. Makes a small amount of progress then just, reboots back into my currently installed OS. No error message, only black screen then back to the Diagnostic boot screen, and systemd-boot. Tried both Ventura and Monterey, provided by macrecovery as DMG and chunklist files.
The operating systems that I currently have installed are; Windows 10 22H2, and Arch Linux.
Obviously using an L13 Yoga, with an i5

Sorry for late response on this. Can you turn on the verbose option by adding -v to your boot-args. Also do you know what make/modle NVME drive you have installed. MacOS has issues with some of them.
Best

That's all good. I can try that later today/in a few hours due to me not being at my laptop right now.
The NVME drive appears to be a SKHynix HFS256GD9TNG-L5B0B (long name, I know) and it is the drive that came with my ThinkPad when I bought it

Can't be specific due to OpenCore not producing a proper log file that I can read, but after I select the USB in the OpenCore menu, the text shows a few errors, then spits out a bunch of stuff that goes by too fast to read before restarting the computer. I saw "secure-boot" in there (which I forgot to re-disable after giving up on this for a bit), but once I did, it did the same thing.
Unless you know how I can generate a log file (I can't find any info on doing that), I can't be any more specific. Sorry! Also, sorry about the late reply, was busy with school.

You can enable debugging by following the guide here. That should provide a log file on you EFI partition and additional information during boot.
If you have an external drive available it may be worth while to try and install on it first to see if it is your SKHynix drive causing the problem?
Do you know if you're using a USB 2 or USB 3 key for the install? Some reported issues with USB 3.

Okay, thanks. I'll enable debugging and get back to you with a log file. For now, I can't install to anything because I can't get into the installer in the first place, though once I do I will certainly try to install to an external drive.
As for the USB, It is USB 3, and I have tried a USB 2 drive before, and it made no difference to the results. I can try again if you wish, but as far as I know, it doesn't help

Here we go, There's the log file.
opencore-2023-06-01-090502.txt

Try replace your config.plist file with the one attached. You will have to update the serial numbers in the PlatformInfo section.
config.plist.zip

No visible difference when booting.
New Log File here:
opencore-2023-06-01-085909.txt

Hi @bladehorton I just updated the EFI folder to OC-0.9.3. If you still have issues installing, try changing the SecureBootModel value to Default instead of the j223 that I have it set to in the config. You can always change it back to j223 afterwards. Your Log file shows "07:625 02:108 OC: Loading Apple Secure Boot with j223 (level 1)" it looks like it was still set to j223. Not sure if that will help, it's been a long time since I did the install. I also just tested booting Ventura from an external drive with the 0.9.3 EFI and it worked.

Yeah, no difference. The new EFI folder (whith the serial numbers i previously generated copied over) did not make any visual changes. Neither did using the new folder while changing SecureBootModel to default.

Hi @seven-of-eleven
Any updates? I can't see anything in the log file that would cause it to freak out and crash personally. I have tried on and off for a little bit since writing my last comment, but nothing has changed

Hi @bladehorton I don't see anything that would cause an issue. I'm wondering if it is related to the Multiple OSs you're running. I don't multi-boot so I can't really help in this area. You can review the Opencore Multiboot guide. It may have some tips for you. Alternatively I would try install on an external drive to see if that works.
Can you provide an updated log file?

I cannot provide an updated log file as my previous attempts after making my comment on June 17th have been using the Stable build, which while less useful for troubleshooting it is more stable and I thought it could help. Thankfully I am getting a new laptop this afternoon, and once I have cloned my Linux installation onto the new SSD, we won't have to worry about multibooting, as my ThinkPad will remain blank until we get the MacOS installation disk booting

Hello again,
It's been a little bit since I've revised this issue, for a while I've been working on my new computer and my L13 Yoga has been left blank for some time. I started over again, and used your EFI folder, and this time tried to use Big Sur, instead of Montery or Ventura. Still no luck. Awaiting on some kind of fix still, or something to pop up to me in a log file perhaps. But for now, I'm still stuck.

I haven't included all the kexts for Big Sur so it likely wouldn't work 100%. Monterey or Ventura should work without issue.
I'll be updating the EFI to the latest over the next week and including Sonoma support. You're welcome to try again at that point. Without logs/errors it's hard to trouble shoot. Everything works on my end and the only change I make to the EFI uploaded here is removing the serial numbers.
Best

I've tried the latest EFI folder upload, even without any changes and with a Sonoma partition on my USB (I followed the "Making the installer in Linux, Method 2) and it seems to have gotten worse? The progress bar does not even show up anymore. Out of curiosity, would you be able to provide a disk image of your USB stick that you used to install macOS? If so, I could flash that image onto a USB stick and try it again. If you can get an installation disk running with Sonoma and the EFI folder provided in the releases of this GitHub repo (meaning, still lacking serial numbers). In short, get a disk image of your working install USB, and try it myself.

Actually, here's an extra question I probably should have asked ages ago. Are you running this on a generation 2 or 3 L13 Yoga? My current system is only a generation 1, and I'm not sure if that would cause any problems when running something meant for a gen 2, but maybe a gen 3


fix: ensure AWS_REGION matches CLI_REGION in all tests

Description of changes

Issue #, if available

Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #12837 (654378b) into dev (ec9a2ba) will increase coverage by 48.42%.
The diff coverage is 61.68%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##              dev   #12837       +/-   ##
===========================================
+ Coverage    0.00%   48.42%   +48.42%
===========================================
  Files        1296      835      -461
  Lines      149743    37992   -111751
  Branches     1296     7745     +6449
===========================================
+ Hits            0    18397    +18397
+ Misses     148447    18006   -130441
- Partials     1296     1589      +293

Impacted Files
Coverage Δ

...ify-category-function/src/events/prePushHandler.ts
33.33% <0.00%> (+33.33%)
:arrow_up:

...ider-utils/awscloudformation/utils/layerHelpers.ts
21.80% <0.00%> (+21.80%)
:arrow_up:

...ib/S3AndCloudFront/helpers/configure-CloudFront.js
87.06% <ø> (+87.06%)
:arrow_up:

...lify-category-hosting/lib/S3AndCloudFront/index.js
89.65% <ø> (+89.65%)
:arrow_up:

...ifications/src/commands/notifications/configure.ts
0.00% <0.00%> (ø)

...e/src/amplify-node-pkg-detector/lock-file-types.ts
100.00% <ø> (+100.00%)
:arrow_up:

.../src/amplify-node-pkg-detector/yarn-lock-parser.ts
100.00% <ø> (+100.00%)
:arrow_up:

packages/amplify-cli-core/src/constants.ts
100.00% <ø> (+100.00%)
:arrow_up:

...es/amplify-cli-core/src/context/plugin-platform.ts
33.33% <0.00%> (+33.33%)
:arrow_up:

...s/amplify-cli-core/src/errors/amplify-exception.ts
82.35% <ø> (+82.35%)
:arrow_up:

... and 36 more

... and 1258 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


⚠ Bofrost.at has degraded performance

In 5629423, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1085 ms

Resolved: Bofrost.at performance has improved in 6821c5c after 58 minutes.


Update documentation link

Description
Hi there, I believe the documentation on overriding Expo payloads through the template editor might be linking to wrong expo docs. Instead of the Push API docs it links to client side types and so there's a bit of a mismatch in terms of fields that can be specified :))

Thanks @matonny!! Including this in #599 🙏


Implementing SSR Integration

In any case note that the above password authentication function could be slow if executed every time due to the bcrypt hash verification.
A better approach for admin authentication is to have a single server global admin instance that you can just import in your server-side actions. For example:
// src/adminClient.js
import PocketBase from "pocketbase"

const adminClient = new PocketBase(process.env.dbURL);

await adminClient.admins.authWithPassword(process.env.dbUsername, process.env.dbPassword, {
  // This will trigger auto refresh or auto reauthentication in case
  // the token has expired or is going to expire in the next 30 minutes.
  autoRefreshThreshold: 30 * 60
})

export default adminClient;

And then in your serverside actions you can just import it and use it directly:
import adminClient from "src/adminClient.js";

export async function voteEmoji(ratingId, emoji) {
    ...

    await adminClient.collection("emojis").create(data);

    ...
}

Originally posted by @ganigeorgiev in https://github.com/pocketbase/pocketbase/issues/3903#issuecomment-1847842771

Hi, I was trying to implement your suggestion but there is a some kind of issue: I set my single file of instance but I got an error that says:
SyntaxError: await is only valid in async functions and the top level bodies of modules.
Which I think it makes sense since there is not an async function in my "adminClient.js" how is this going to work? Am I missing something?

import PocketBase from "pocketbase";

const pb = new PocketBase(process.env.dbURL);

await pb.admins.authWithPassword(
  process.env.dbUsername,
  process.env.dbPassword,
  {
    // This will trigger auto refresh or auto reauthentication in case
    // the token has expired or is going to expire in the next 30 minutes.
    autoRefreshThreshold: 30 * 60,
  }
);

export default pb;

```js

I'm getting tired by this... If you have questrion please file a Q&A discussion.


DATAJDBC-614 - Fix the bug in Query.with(Pageable pageable) . The Sort in Query is r…

Fix the bug in Query.with(Pageable pageable) . The Sort in Query is repeat and the Sort in PageAble is not available

Please create a ticket in Jira so we understand what problem you are trying to solve.
https://jira.spring.io/projects/DATAJDBC

Thanks, that's merged.


How can I drag entire code in iPhone?

I want to select and copy the entire code, but I can't drag the entire code on my iPhone.

Please refer to https://github.com/thebaselab/codeapp/issues/169. I will bring better touch support in the next update.


fix: cypress test added for forking critical issue

Description
This PR adds cypress test to ensure the fix made for issue #24702
PR fixes following issue(s)
Fixes #24702

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] Jest
[x] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test

Testing will not be required for this PR as it simply adds cypress test for forking functionality @prapullac @btsgh

/ok-to-test

/ok-to-test

@sneha122 - Always merge latest manually & then start oktotest. Have seen the auto merge with release not really consistent every time.

@sneha122 - Will add TestPlanApproved label - after the case is a pass in CI run!

/ok-to-test


easy way to set secondary colors, etc.

I mean you just have to change the Blockly.colours_ or whatever, it shouldn't be too difficult to add
There's no reason to limit users to CSS. CSS is a valid option but not the only one.

but the color is how we determine category... we may nee to find a different way to achieve that

Do we need to update all existing blocks whenever the colors change?

Is this just as easy as mapping Blockly.Colours_ to something?

but the color is how we determine category... we may nee to find a different way to achieve that

What if when a color updates, we add it to the known category colors 📦
(of course, if multiple categories can be the same color then it opens up a whole 'nother can of worms)


Follow up fix for buffer overflow

Start the outer buffer with the capacity of initialSize + 1 so later we can null terminate on the index of initialSize, assuming the return length is smaller or equal to initialSize. We don't bother to verify the assumption since that'd be a user error.
Also fix a terrible copy-pasta error: We should null-terminate the inner buffer, not the outer buffer.
Fixed 129806508

@swift-ci please test


[FEA] Add explain api for Spark 2.X

Is your feature request related to a problem? Please describe.
Investigate adding an explain api that would run on a Spark 2.x plan.
This would be a user callable api because there is no plugin point in catalyst until Spark 3.0.
Task breakdown:

[x] Core initial implementation using separate module
[ ] update deploy scripts

Follow ons for another release: https://github.com/NVIDIA/spark-rapids/issues/4522

still has deploy scripts that need to be done.


Abilitity to clear all alerts and messages at once

There needs to be some design around this. Were the button would be to placed and the language on it.
If I had to implement it today, it would just go at the top right and say "Clear all".
UX and design could be improved.

i will try make this feature 😃


Avoid using CUlinkState unless absolutely necessary

I noticed ls.add_ptr_data() took about 20% of time in _compile_with_cache_cuda(), but it's actually not needed unless we are doing separate compilation and need to link with other input files/data. Otherwise, we can just pass the generated PTX to cuModuleLoad() which should work just fine.

Jenkins, test this please

Jenkins CI test (for commit 3c16c76478d9ebc3b1f3fdf7e84a8cdbc031e5b6, target branch master) succeeded!

LGTM!


Unexpected EOF when making request with Safari

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you do?
Sent request to https://nyu.a1liu.com/ with Safari
Used Traefik with Docker Compose, here's the docker-compose.yml contents:
Expected the server to respond with a webpage, which is the behavior on Firefox and Chrome.
What did you see instead?

Safari responds with the following:

Traefik has the following log message:

reverseproxy.go:505: httputil: ReverseProxy read error during body copy: unexpected EOF

What version of Traefik are you using?

Docker compose file
version: '3.3'

services:
reverse-proxy:
  # The official v2.1 Traefik docker image
  image: traefik:v2.9.6
  restart: always

  command:
    # You need to actively say traefik.enable=true
    - "--providers.docker=true"
    - "--providers.docker.exposedbydefault=false"

    - "--entrypoints.web.address=:80"
    - "--entrypoints.websecure.address=:443"

    - "--certificatesresolvers.mytlschallenge.acme.tlschallenge=true"
    -<EMAIL_ADDRESS>    - "--certificatesresolvers.mytlschallenge.acme.storage=/.letsencrypt/acme.json"

  ports:
    - "80:80"
    - "443:443"

  volumes:
    # So that Traefik can listen to the Docker events
    - "/var/run/docker.sock:/var/run/docker.sock:ro"
    - "/home/ubuntu/schedge/.letsencrypt:/.letsencrypt"

postgres:
  image: postgres:14.1-alpine
  restart: always
  environment:
    - POSTGRES_USER=postgres
    - POSTGRES_PASSWORD=postgres
  logging:
    options:
      max-size: 10m
      max-file: "3"
  expose:
    - "5432"

  volumes:
    - "/home/ubuntu/schedge/.build/postgres:/var/lib/postgresql/data"

schedge:
  image: ghcr.io/a1liu/schedge:latest
  restart: always
  environment:
    - DB_USERNAME=postgres
    - DB_PASSWORD=postgres
    - SCHEDGE_ADMIN_PASSWORD=$SCHEDGE_ADMIN_PASSWORD
    - JDBC_URL=jdbc:postgresql://postgres/postgres

  expose:
    - "4358"

  labels:
    - "traefik.enable=true"
    - "com.centurylinklabs.watchtower.enable=true"
    - "com.centurylinklabs.watchtower.stop-signal=SIGKILL"

    - "traefik.http.services.schedge.loadbalancer.server.port=4358"

      # HTTPS
    - "traefik.http.routers.schedge-https.rule=Host(`nyu.a1liu.com`)"
    - "traefik.http.routers.schedge-https.service=schedge"
    - "traefik.http.routers.schedge-https.entrypoints=websecure"
    - "traefik.http.routers.schedge-https.tls.certresolver=mytlschallenge"

      # HTTP
    - "traefik.http.routers.schedge-http.rule=Host(`nyu.a1liu.com`)"
    - "traefik.http.routers.schedge-http.service=schedge"
    - "traefik.http.routers.schedge-http.entrypoints=web"

watchtower:
  image: containrrr/watchtower:1.4.0
  volumes:
    - /var/run/docker.sock:/var/run/docker.sock
  command: --interval 300 --cleanup --label-enable
  restart: unless-stopped

If applicable, please paste the log output in DEBUG level
No response

As a note, other paths work, like for example, https://nyu.a1liu.com/api works on safari as expected

Hello @A1Liu ,
Do you see the same discrepancy with http://nyu.a1liu.com VS http://nyu.a1liu.com/api (with http) or do you only see the problem with https ?

The problem only seems to come up with https, http://nyu.a1liu.com works fine on all platforms i've tested

Ok, I'm also seeing a problem specific to safari in the https case, but my error is different. Mine is "cannot parse response. NSURLErrorDomain:-1017".
So atm I'm not sure that I've reproduce the same error.

@A1Liu oh btw, I had not thought of it earlier, but with my safari (Version 15.2 (176<IP_ADDRESS>.6)), I can actually access your server, as shown on the screenshot.
So tbh, I don't think this is a Traefik problem, but rather a safari one.
Could you please maybe retry with another version of Safari?

Hmmm, I'm on a more upgraded version of Safari: 16.2 (186<IP_ADDRESS>.5)
I will see what I can do

And I can also access your server from Firefox Mobile (Android, 108.2.0).

Also though, Traefik is still outputting an error log when responding to Safari, so I don't think it's soley a Safari thing.
reverseproxy.go:505: httputil: ReverseProxy read error during body copy: unexpected EOF

Firefox Mobile uses a different engine on Android and iOS; I found it works on Android Firefox but not on iOS Firefox or iOS Safari, but I am updating my phone now to see what happens with a newer Safari.

Sure, I had noticed that the log line was happening when requesting with Chrome as well. However there's no guarantee that this log error as anything to do with your problem (but we certainly would have to make sure of it).
btw, one possible hint (that we will look into) is:
https://apple.stackexchange.com/questions/409055/safari-iphone-macbook-gives-nsurlerrordomain-1017

btw, it looks like we're not doing anything wrong regarding sending an HTTP/2 upgrade when not needed (the lead I mentioned above), because it seems to be going fine with curl:
mbp:9668 mpl$ docker run -t --rm badouralix/curl-http2 -I -v https://nyu.a1liu.com
*   Trying <IP_ADDRESS>:443...
* Connected to nyu.a1liu.com (<IP_ADDRESS>) port 443 (#0)
* ALPN: offers h2
* ALPN: offers http/1.1
*  CAfile: /etc/ssl/certs/ca-certificates.crt
*  CApath: none
* [CONN-0-0][CF-SSL] TLSv1.0 (OUT), TLS header, Certificate Status (22):
* [CONN-0-0][CF-SSL] TLSv1.3 (OUT), TLS handshake, Client hello (1):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Certificate Status (22):
* [CONN-0-0][CF-SSL] TLSv1.3 (IN), TLS handshake, Server hello (2):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Finished (20):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.3 (IN), TLS handshake, Certificate (11):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.3 (IN), TLS handshake, CERT verify (15):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.3 (IN), TLS handshake, Finished (20):
* [CONN-0-0][CF-SSL] TLSv1.2 (OUT), TLS header, Finished (20):
* [CONN-0-0][CF-SSL] TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
* [CONN-0-0][CF-SSL] TLSv1.2 (OUT), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.3 (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / TLS_AES_128_GCM_SHA256
* ALPN: server accepted h2
* Server certificate:
*  subject: CN=nyu.a1liu.com
*  start date: Nov 25 03:27:01 2022 GMT
*  expire date: Feb 23 03:27:00 2023 GMT
*  subjectAltName: host "nyu.a1liu.com" matched cert's "nyu.a1liu.com"
*  issuer: C=US; O=Let's Encrypt; CN=R3
*  SSL certificate verify ok.
* Using HTTP2, server supports multiplexing
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* [CONN-0-0][CF-SSL] TLSv1.2 (OUT), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.2 (OUT), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.2 (OUT), TLS header, Supplemental data (23):
* h2h3 [:method: HEAD]
* h2h3 [:path: /]
* h2h3 [:scheme: https]
* h2h3 [:authority: nyu.a1liu.com]
* h2h3 [user-agent: curl/7.87.0]
* h2h3 [accept: */*]
* Using Stream ID: 1 (easy handle 0x7fd30b2a6a90)
* [CONN-0-0][CF-SSL] TLSv1.2 (OUT), TLS header, Supplemental data (23):
> HEAD / HTTP/2
> Host: nyu.a1liu.com
> user-agent: curl/7.87.0
> accept: */*
>
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
* Connection state changed (MAX_CONCURRENT_STREAMS == 250)!
* [CONN-0-0][CF-SSL] TLSv1.2 (OUT), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
* [CONN-0-0][CF-SSL] TLSv1.2 (IN), TLS header, Supplemental data (23):
< HTTP/2 200
HTTP/2 200
< cache-control: max-age=0
cache-control: max-age=0
< date: Wed, 18 Jan 2023 17:23:43 GMT
date: Wed, 18 Jan 2023 17:23:43 GMT
< etag: W/"z5UTP/K38mEz5USukpV0II"
etag: W/"z5UTP/K38mEz5USukpV0II"
< content-length: 4179
content-length: 4179

<
* Connection #0 to host nyu.a1liu.com left intact

As a note, other paths work, for example, https://nyu.a1liu.com/api works on safari as expected

Btw, the most likely reason why you're not seeing any problem on /api is because you have a redirect to http on that endpoint, so your browser actually ends up trying on http://nyu.a1liu.com/api , and as you said, you didn't see any problem with http.
$ curl -I https://nyu.a1liu.com/api
HTTP/2 302
cache-control: max-age=0
date: Wed, 18 Jan 2023 17:44:47 GMT
location: http://nyu.a1liu.com/api/
content-length: 0

finally, if you replace your whole schedge section in your docker compose, with a dumb server, like
  whoami:
    image: traefik/whoami
    labels:
      traefik.enable: true
      traefik.http.routers.whoami.entrypoints: web
      traefik.http.routers.whoami.rule: Host(`nyu.a1liu.com`)
      traefik.http.routers.whoami-https.entrypoints: websecure
      traefik.http.routers.whoami-https.tls: true
      traefik.http.routers.whoami-https.rule: Host(`nyu.a1liu.com`)
      traefik.http.routers.whoami-https.tls.certresolver: mytlschallenge

then it looks like even with safari there is no problem (a coworker has a higher safari version, like yours, and tried for me).
So we're inclined to think there's something funky going on with schedge.


New Character Sheet Cepheus Deluxe

Created on behalf of the creators of Cepheus Deluxe

Submission Checklist
Required

[ X] The pull request title clearly contains the name of the sheet I am editing.
[ X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ X] The pull request makes changes to files in only one sub-folder.
[ X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: < Cepheus Deluxe  >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <  Stellagama Publishing > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <  Cepheus Engine > (i.e. Dungeons & Dragons, FATE)

[ X] I have followed the Character Sheets Standards when building this sheet.

[ X] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

@roll20deploy Is there anything else I need to do? I see it is blocked because it hasn't been added to the approved.yaml. Do I need to do anything for this?

Nothing else to do, this is normal for newly submitted sheets, just wait human review

Greetings @n8midd,
We no longer accept characters sheets with layouts built with table tags (see article.) These elements will need to be replaced to meet the required programming standards.
Thank you

Thank you for the review @mperes. I have updated the code to no longer use table tags. Let me know if there is anything else.


bugfix: beatport has stopped using their 'pro' subdomain, using just …

…beatport.com instead to fix bug where salmon was unable to fetch metadata from beatport

merged


[Bugs] Data Network Name value in DNN Configurations cannot be changed from the default "internet"

Hi @free5gc-org
When I registered the subscriber information using WebConsole, even if I changed the value of Data Network Name of DNN Configurations from the default internet to another string (eg. internet2), when I checked the contents of mongodb, it was still internet.
Also, the function to change subscriber information does not work properly.
When will the these major bugs in WebConsole are fixed?
Thank you very much in advance.

Hi @free5gc-org
For your reference, I will tell you a little more about the bug.
For example, when SST=1 and SD=101023,  even if registered internet2 to DNN (Data Network Name)  using WebConsole, DNN=internet (default) is always set in MongoDB as follows.

free5gc.policyData.ues.smData.smPolicySnssaiData.01010203.smPolicyDnnData.internet.dnn=internet

Please fix it.
Thank you very much in advance.

Hi @free5gc-org
Now, I'm using free5GC to confirm the operation of the facility of ULCL.
In the process, first of all, I have confirmed the operation of free5GC with the following simple configuration (no ULCL).
In addition, as this above I modify the data in MongoDB registered using WebConsole and use it.
https://github.com/s5uishida/free5gc_ueransim_sample_config
FYI.

@s5uishida
Thanks for your report.
We're in CNY holidays.
We'll try to fix it ASAP.

@s5uishida
We fixed the issue, please update webconsole to v1.0.1:
https://github.com/free5gc/webconsole/releases/tag/v1.0.1

Hi @free5gc-org
Thank you very much for your quick response!
I have confirmed that the data is registered correctly in MongoDB and free5GC works properly.
By the way I have two questions.

Please allow this modified webconsole to be installed using the normal installation procedure for free5GC v3.0.5.
Currently, when v3.0.5 was git cloned, the source version before this fix was got.

The registered data editing function does not work properly, so please correct this as well.

Anyway, this fix is very helpful.
Thank you very much again!!

@s5uishida

We updated the wiki: https://github.com/free5gc/free5gc/wiki/Installation#d-install-webconsole
We'll check it.

@s5uishida
About registered data issue, please follow steps as below:

Drop mongoDB:

mongo --eval "db.dropDatabase()" free5gc

Run free5gc

./run.sh

Run WebConsole

./webconsole/bin/webconsole

Check "Registered UEs" page

It should be workable.

Hi @free5gc-org
OK.
As an installation procedure, I edited my sample configuration to refer to the above URL.
Also,

The registered data editing function does not work properly, so please correct this as well.

Please do not forget to deal with this matter. Perhaps this bug has existed unfixed since free5GC v3.0.0.
Thank you very much in advance.

The registered data editing function does not work properly, so please correct this as well.

Please do not forget to deal with this matter. Perhaps this bug has existed unfixed since free5GC v3.0.0.

@s5uishida
Please follow here to resolve it.

Hi @free5gc-org
The following steps describe the situation here.

After deleting the data of MongoDB, login to WebConsole and register the data.

Logout of the WebConsole and restart the browser.

Login to the WebConsole again and click SUBSCRIBERS.

Select UE ID(imsi-001010000000000) and click the Modify button.

The edit screen of<PHONE_NUMBER>00003, which is different from<PHONE_NUMBER>00000, is displayed.

In addition, UEs are not connected to free5GC yet.
I cannot modify the various data of the selected IMSI. If select another IMSI and press the Modify button, the same screen will be displayed.
Is this situation reproduced there?

@s5uishida
There are some issues for subscriber modification.
If you want to modify subscriber, please delete first and new again for now.

Hi @free5gc-org

There are some issues for subscriber modification.
If you want to modify subscriber, please delete first and new again for now.

Could you please write the above in the following URL?
https://github.com/free5gc/free5gc/wiki/New-Subscriber-via-webconsole
I think that registering subscriber data is a basic task when using free5GC. At this time, not being able to edit the registered data is important information for the user, so please make something clear this.
After that, please close this issue.
Thank you very much in advance.

Could you please write the above in the following URL?
https://github.com/free5gc/free5gc/wiki/New-Subscriber-via-webconsole
I think that registering subscriber data is a basic task when using free5GC. At this time, not being able to edit the registered data is important information for the user, so please make something clear this.
After that, please close this issue.

Wiki has been updated. Thanks.


Allow empty FLAGS argument

We conditionally set the FLAGS argument to corrosion_import_crate() when we want to pass in -Zbuild-std but corrosion_import_crate() isn't happy to receive an empty FLAGS in the case that we don't and errors out:
-- Using Corrosion as a subdirectory
CMake Error at build/_deps/corrosion-src/cmake/Corrosion.cmake:1125 (message):
  Invalid arguments: the following keywords had no associated value(s): FLAGS
Call Stack (most recent call first):
  cmake/Rust.cmake:22 (corrosion_import_crate)
  CMakeLists.txt:27 (include)

Passing in a single space as FLAGS obviously breaks the build since it's passed as significant whitespace to cargo, which doesn't recognize the argument and complains that it has two input file names. There are obvious workarounds, but this feels like the natural approach.

Thanks!


tm.shared.licensing.activation support delete

Currently Python SDK does not support delete for a license activation
https://github.com/wojtek0806/f5-common-python/commit/c69ad3e14c1bb2a560f72f9cba4a844eb60a6c5a

details on how to use this?


Campaign scheduling

Hi,
When I send a big contact list (~10K contacts), the newsletter stay in draft status, instead of the scheduled mode. Whereas if my contact list is small, everything works fine.
It seems that if the contact list import process is too long, the scheduled parameters is not saved.
Do you have any idea to resolve that ?
Regards

Hi @TheExcalibur,
First of all, thank you for using Mailjet!
Could you provide us with a bit more information (some code would be great!) regarding the process you go through to upload your contact list and how it is related to said newsletter?
Gaëtan

Thanks for your attention. Here is the code I use :
                // Génération du fichier des contacts à envoyer
				$filename = 'files/to_mailjet.csv';
				$data = '"email","civilite","nom","prenom"'."\r\n";
				foreach ($contacts as $contact) {
					$data .= '"'.$contact['Lead']['email'].'","'.$contact['Contact']['civilite'].'","'.$contact['Contact']['nom'].'","'.$contact['Contact']['prenom'].'"'."\r\n";
				}
				$fichier = fopen (WWW_ROOT.$filename, 'w');
				fwrite($fichier, $data);
				fclose($fichier);

				$mj = new MailjetV3Component();
				// Création de la ContactList
				$params = array(
					'method' => 'POST',
					'Name' => $contact_list,
				);
				$result = $mj->contactslist($params);
				$liste_id = $mj->_response->Data[0]->ID;
				// Envoi des contacts
				$params = array(
					"method" => "POST",
					"ID" => $liste_id,
					"csv_content" => file_get_contents(WWW_ROOT.$filename),
				);
				$result = $mj->uploadCSVContactslistData($params);
				$data_id = $result->ID;
				// Affectation des contacts à la liste
				$params = array(
					"method" => "POST",
					"ContactsListID" => $liste_id,
					"DataID" => $data_id,
					"Method" => "addnoforce"
				);
				$result = $mj->csvimport($params);
				// Création de la newsletter (campaign)
				$params = array(
					'method' => 'POST',
					'Title' => $titre_campagne,
					'Subject' => $export['Campagne']['sujet_mailing'],
					'Locale' => 'fr_FR',
					'EditMode' => 'html',
					'PolitenessMode' => 1,
					'Sender' => $sender,
					'SenderName' => $sender_name,
					'SenderEmail' => $sender_email,
					'ContactsListID' => $liste_id,
				);
				$result = $mj->newsletter($params);
				$newsletter_id = $mj->_response->Data[0]->ID;
				// Mise en place du contenu du mail
				$params = array(
					'method' => 'PUT',
					'ID' => $newsletter_id,
					'html_content' => $html_content,
				);
				$result = $mj->addHTMLbody($params);
				// Envoi des mails
				$date_scheduled = mktime(date('H'), date('i') + 10, 0, date('n'), date('j'), date('Y'));
				$date_for_mailjet = gmdate('Y-m-d\TH:i:s.uP', $date_scheduled);
				$params = array(
					"method" => "POST",
					"ID" => $newsletter_id,
					"date" => $date_for_mailjet
				);
				$result = $mj->newsletterSchedule($params);

The mailjet library is up to date regarding the github repository and I'm using the CakePHP v2 framework.

Hi @TheExcalibur,
Sorry fo the slight delay.
Could you be a bit more precise on the the scheduled parameter is not saved part of your original message?
Thanks in advance.
Gaëtan

Hi @Gormador,
According to the MailJet Interface, the campaign has the draft status and when I manually send the newsletter, the scheduled parameter is not configured.
Regards

Hi @TheExcalibur,
I hope you had a nice weekend!
One thing that could help would be to wait for the completed upload before going ahead with the newsletter part of your code.
You can do as follows:
do
{
    sleep(5);
    $importcheck = $mj->csvimport(array('method' => 'GET', 'ID' => $result->Data[0]->ID));
} while($importcheck->Data[0]->Status != "Completed");

Also, when you say "I manually send the newsletter", do you mean through the API or via your Mailjet Dashboard ?
Cheers.
Gaëtan


Feature: Add service worker for client-side caching

Add a workbox service worker for more aggressive client-side caching

Added as of 2c526c1


diagnostics partially empty for serial get

Undesired behaviour
I am invoking a local cluster to start realtime gathering of diagnostic information with something like:
client = Client()
dsk = .... # A custom graph of tasks that use nested dask computations
dask.get(dsk,['pull_m20'])

What I find is that the /tasks diagnostic is empty. The system pages show current system resources.
Alternative approaches
I understand  #2397 has fixed the nested callbacks issue so I can use:
with Profiler() as prof, ResourceProfiler(dt=0.25) as rprof, CacheProfiler() as cprof:
    dask.get(dsk,'pull_m20')
visualize([prof,rprof,cprof])

Desireable Feature
As far as I understand, it is not possible to do:
client = Client()
dsk = .... # A custom graph of tasks that uses nested dask computations
client.get(dsk,['pull_m20']) # daemon having children ERROR

Is there a way of using the distributed scheduler actually, and not just the data collection abilities with sub-dasks (possibly also using multiprocessing or distributed) with minimal overhear?
Verison
Master 1c0113d

What I find is that the /tasks diagnostic is empty. The system pages show current system resources.

I suspect that you are just hosting pages on a couple different ports.  Tasks should show all of the information here.

As far as I understand, it is not possible to do:

Your second example looks like your first.  I'm not sure what the difference is.
I apologize that no one responded to your issue when you raised it originally.  I'm going to go ahead and close it now.  If you still have issues and can add some new information please respond and we'll reopen.


Update PostEditorHelper.php

The constant "FILTER_SANITIZE_STRING" is deprecated since PHP 8.1. It will be removed in the future, causing the Yoast SEO plugin to crash in the admin panel. This code just effectively does the same as if($_GET['action'] === 'elementor'), filter_input is used to stop IDE complaints, and FILTER_SANITIZE_STRING is cargo-cult as it serves no purpose.  Changed to use no filter (will use the default UNSAFE_RAW), which will satisfy IDE.

Thanks @npr-gh!


move walla to RSS; refactor files

One non-refactoring change, fix #1290, complete #1270.
Added test network-accessing test.
Refactor, nearly completing #1260

I suggest reviewing the resulting files, not the diffs, particularly rss_sites.py.
new_news_check.py is removed as it is not needed - we simply break when we find an old item.
Exceptions are not handled explicitly. The existing handling is definitely overkill. I think Letting them propagate is better, at least for now.
Terminology (not consistent yet with untouched files):

Fetch: get html/xml from the source (web, local file in tests)
Parse: take relevant textual fields from xml/html, convert to format when there's no decision to make (datetime)
Scrape: fetch + parse
Extract features: add features to parsed items: is it an accident, geo locations, relevant tags maybe
Insertion to database is a distinct step
rss_sites: walla, ynet
news_flash: twitter and rss sites

@Mano3 please review

@elazarg please resolve conflicts


DateTime values smaller than 1-1-1601 fails

The minimum supported DateTime value for table storage is 1-1-1601 as described here https://docs.microsoft.com/en-us/rest/api/storageservices/Understanding-the-Table-Service-Data-Model?redirectedfrom=MSDN#property-types
This means that whenever a log entry contains a DateTime with a value less than that, the logging fails.
There doesn't seem to be any way to fix this through configuration, since the DateTime is treated as a scalar, and not properly handled in the implementation.
Is there something I'm missing or is this a bug?
I would expect some sort of DateTimeHandling configuration options somewhere, where you could either specify an object to handle unsupported DateTime values, or a simple option to be TreatAsNull or UseMinimumSupportedValue.

Not sure why there would ever be a DateTime that small.   In version 8, you can completely customize the document created if you need to by implementing IDocumentFactory.   Please open a new issue if this is still an issue.   Thanks


[FLINK-32429][table] Support lazy initialization of catalogs and persistence of catalog configurations

What is the purpose of the change
Support lazy initialization of catalog and persistence of catalog configuration
Brief change log
(for example:)

Introduce CatalogStore in CatalogManager
Support config CatalogStore using Table API
Support sharing CatalogStoreFactory resource between multiple FlinkSQL gateway sessions
Add built-in FileCatalogStore

Verifying this change

Add some unit test to verify the CatalogManager work with CatalogStore
Add unit test to verify config CatalogStore using Table API
Add unit test to verify resource sharing between sessions
Add unit test to verify FileCatalogStore save/remove/read related operations

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no )
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no )
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (JavaDocs)
There will be a document pr later.

@ruanhang1993 @leonardBang Please take a look this pr when you have time. Thanks

The configuration of the table API for the CatalogStore module in Python is temporarily not supported. Therefore, we are skipping the checks related to CatalogStore in EnvironmentSettingsCompletenessTests and EnvironmentSettingsBuilderCompletenessTests. We will address this issue separately in a dedicated Jira ticket.

Let's wait the CI green

@leonardBang Thanks for reviewing the code.


Corrections: Tramway arrivals at EINA's tramway stop

@Guillerm097 don't close pull request #49!!!! Just push your changes!!!


Pulsar SQL is not able to read the latest message

Describe the bug
Pulsar SQL is always lacking one message when querying pulsar topics.

Reason
The problem is due to broker doesn't advance LAC, so when pulsar sql bypasses broker to query the data, it can only read entries up to the LAC that all the bookies learned.
Fixes
Enable periodically write lac on broker.

@sijie yup have discussed this issue before with @merlimat but we decided not to take any action at that time because we reasoned that Pulsar is a streaming system thus we expect users to continuously pump data into pulsar and not just one time batch thus this wouldn't be an issue since LAC will eventually get updated if there is continuous data

@jerrypeng this is not just about not being able to read the event, but also about the latency (how fast it can query the data). current assumption is assuming there are are events "eventually" arrive and advance the LAC, which will significantly delay the messages being visible to users if the events are delayed. so I think we still need to enable explicit write lac on broker, so the lac will be advanced within a SLA.
The feature is already in bookkeeper. It is just a matter to enable it and make sure it is working as expected. If pulsar broker doesn't want this feature to be turned on by default, we can make it configurable and turned off by default. If people want this feature, they can turn it on. I don't think we should just ignore this fact and try to explain to people who don't really understand how the underlying stuff works.

@sijie I totally agree we should add something like this.  I was just presenting some previous thoughts we have had on this problem.

Yes, we should have a way to configure the explicit LAC update on the topics and have some reasonable default.
Though, should this be for 2.3.1? I think it would more be appropriate for 2.4 since it's not a bug.. but rather a current limitation.

@merlimat we can move this to 2.4.0

close via #5822

This issue is fixed by #6908 and #6908 is released in 2.5.2, So I add release/2.5.2 tag for this issue.


test

the idea is to get main repo from datafaker-gen and build it and then run tests against latest datafaker version to be able to detect issue appearing only in downstream projects like e.g. #1170 or #1168
currently this check is done only in case of jdk21 and ubuntu (in fact i do not see a real reason to test it with the whole matrix)

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

Project coverage is 92.34%. Comparing base (b37c566) to head (fabc92a).
Report is 77 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1171      +/-   ##
============================================
- Coverage     92.35%   92.34%   -0.01%
- Complexity     2821     2834      +13
============================================
  Files           292      293       +1
  Lines          5609     5615       +6
  Branches        599      598       -1
============================================
+ Hits           5180     5185       +5
- Misses          275      276       +1
  Partials        154      154

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

//cc @bodiam since you were asking about something like this

Sorry I missed the request on this one. It does look good 🙂


Issue with Block Syncing in Graph Node on Local Hardhat Network

Description:
I am currently trying to set up a local Graph Node for a local Hardhat network. I have followed the instructions provided in the graph-node repository and utilized the same docker-compose.yml file. However, I am encountering an issue where the graph-node fails to sync blocks and produces the following error message:
May 07 11:19:12.142 DEBG Requesting 0 block(s), sgd: 3, subgraph_id: QmYiWnkuSSD7jtwpx3CXFNMCYLcJdR3eHHRox3vyR36rpc, component: BlockStream

May 07 11:19:12.144 DEBG Block stream produced a non-fatal error, error: Failed to get parent pointer for #0 (f995baa7a24fc3f8823659fc8751eb8660e86cf0f0551f53f50890af08282394) (nonfinal), sgd: 3, subgraph_id: QmYiWnkuSSD7jtwpx3CXFNMCYLcJdR3eHHRox3vyR36rpc, component: SubgraphInstanceManager

Steps to Reproduce:

Set up a local Hardhat network.
Use the provided docker-compose.yml from the graph-node repository.
Start the Graph Node using the specified configuration.
Monitor the logs for any error messages.

Expected Behavior:
I expect the Graph Node to sync blocks successfully without encountering any errors.
Additional Information:
I have verified that my local Hardhat network is running correctly and accessible.
I have reviewed the documentation and troubleshooting guides related to the graph-node setup, but couldn't find a solution for this particular error.
I have attempted restarting the Graph Node and the local Hardhat network, but the issue persists.
Environment:
Operating System: [MacOS monetery 12.6.5]
Docker version: [Docker Desktop 4.5.0 (74594)]
Hardhat version: [2.14.0]
Graph Node version: [Latest docker image]

Is this with a vanilla hardhat network? (i.e. it is not forked)


chore: upgrade oak deps to 10.5.0

It seems that oak itself and oak_sessions must be using the same version of oak to avoid some type error. So I update the oak from deps. Wondering if there's a better way than having to upgrade libraries depending on oak every time oak upgrades.

Yeah, I agree we should find a better way. Tried asking the Deno community a while back, haven't gotten guidance. I'm very new to TypeScript, any guidance here would be valuable.


Allow interpreter to detect reboot and finish working on cell

When things really get hung on waiting for an x04 this can be the only way out, yet it still hangs

There's now a 5 second timeout


Add ability to display all rows

A suggestion would be that when number of rows is set to 0 or null, all rows are displayed.

This replaces #138


Not all characters work in names of drinks

Especially: '+'

The system was only planed for alphanumeric characters which is neither checked nor enforced. Need to simply url-escape everything

This has been fixed with rewrite of drinklist.


URLs unavailable in Field types.

Scenario & Reproduction Steps
When retrieving an item, it seems to only include username and password field types. According to the documentation, URLs should also be supported, but they are not present in the Item.Fields within the 1Password item structure.
// Item represents a 1Password item.
type Item struct {
	// ID is the unique identifier of the item.
	ID string `json:"id"`
	// Title is the title of the item.
	Title string `json:"title"`
	// Category represents the item's category.
	Category ItemCategory `json:"category"`
	// VaultID is the identifier of the vault where the item is stored.
	VaultID string `json:"vaultId"`
	// Fields contains the item's fields.
	Fields []ItemField `json:"fields"`
	// Sections holds the sections of the item.
	Sections []ItemSection `json:"sections"`
	// Version represents the item's version.
	Version uint32 `json:"version"`
}

func getItem(token string, vaultID string, itemID string) onepassword.Item {
	client := createClient(token)

	item, err := client.Items.Get(context.Background(), vaultID, itemID)

        // This will only print username or password and never URL
	for _, field := range item.Fields {
		print(field.ID)
	}

	if err != nil {
		panic(err)
	}

	return item
}

Actual Behavior
URLs does not appear in Item.Fields
Expected Behavior
URLs should appear in Item.Fields
SDK version
v0.1.1
Additional information
No response

Hey @sebastianlivoni! Thanks for reporting this bug.
I think I was able to reproduce the issue you are encountering from the details that you provided. It seems that the websites associated to an item are not returned by the SDK.
Is this the issue you were running into?
URL-type fields are supported and returned correctly by the SDK (based on my testing), but they are different from the websites associated to an item.
Can you confirm if this is the issue you're running into? If so we will look into fixing it soon.

I think I was able to reproduce the issue you are encountering from the details that you provided. It seems that the websites associated to an item are not returned by the SDK.
Is this the issue you were running into?

Hi. Yes, I'm not able to see associated websites from items ☺

@sebastianlivoni A very relevant question for us: do you see it as a requirement that URLs are available to you fast? For example, in a context where you have 2000 items in a vault, do you want to list all items and quickly look have the URLs available to you prior to seeing the rest of the item details, such as other secret fields and sections?

Hi @AndyTitu. Thank you for looking into it :)

How do you plan to use the URLs in your workflow?

I'm implementing an Autofill Credential Provider Extension App on macOS for personal use in order to provide "macOS Native Autofill".

What type of app are you writing and how does this app use the websites it retrieves from 1Password?

The app is designed to provide autofill functionality. :)

A very relevant question for us: do you see it as a requirement that URLs are available to you fast?

No, not for my use case, as I need all the item details, including the username and password, for my implementation. However, I can see how that approach might make sense in other contexts.

For example, in a context where you have 2000 items in a vault, do you want to list all items and quickly look have the URLs available to you prior to seeing the rest of the item details, such as other secret fields and sections?

Currently, I'm fetching all items and saving them in the Keychain for fast autofill (currently with sample URLs, as they are unavailable 😊) and I think I keep this approach instead of only fetching Title, Username and URLs and then retrieving the password later.
For my use case, I'd prefer a query system similar to GraphQL, allowing me to request only the specific data I need, such as Title, Username, Password, URL(s), and 2FA, without fetching unnecessary information.

@sebastianlivoni version 0.1.3 of the SDK adds support for websites.
I'll be closing this issue, but do feel free to reopen if this doesn't address your use-case. Thanks!


Android Basics: tip-calculator

URL of codelab
Calculate the tip
In which task and step of the codelab can this issue be found?
4 Calculate the tip -> Display the tip
Describe the problem
binding.tipResult.text = getString(R.string.tip_amount, formattedTip)
should be
binding.tipAmount.text= getString(R.string.tip_amount, formattedTip)

@qvmzs
Thank you for reaching out to us.
The code snippet is correct in the codelab. Text view ID is tip_result
 <TextView
        android:id="@+id/tip_result"
<EMAIL_ADDRESS>        ...
        tools:text="Tip Amount: $10"
        />

Please verify your activity_main.xml.
https://github.com/google-developer-training/android-basics-kotlin-tip-calculator-app-solution/blob/3662e275a97e6e577287613b10fcbf20c2ff0ac8/app/src/main/res/layout/activity_main.xml
Please provide your activity_main.xml file, if you still face the issue.


Kubernetes-Ceph Issues

I'm deploying Ceph by the Kubernetes Helm Example.
The default configuration with
values.yaml |
daemon: docker.io/ceph/daemon:latest issues with  Issue #573
values.yaml |
daemon: docker.io/ceph/daemon:tag-build-master-jewel-ubuntu-14.04 starts up the Cluster.
Next Issue, when i stop the Ceph Cluster and start it again, the OSD's keeps rebooting in Crashloop with Issue #510
disabling the Auth will solve the Issue.
But when auth is disabled, my Ceph-Mons arent able to connect by PV/C to my containers.
So im not able to set-up a persistent CephFS storage for my cluster.

I need to build a k8s env to reproduce that, every installers I've found failed for me... Any hint on how to install k8s with vagrant + libvirt?
Thanks!

@leseb alternativley to quickly set up a testing kubernetes single node cluster you can use https://github.com/kubernetes/minikube it comes with dns and dashboard

@osixia I'll give a try, thanks!

@J-Ha Did you solve the problem? I have met the same problem.


Avoid hidding exceptions

then() catches exceptions, which is not desirable when its result is not returned.
When using the RetryExecutor, the exception thrown by the following code is caught, whereas it should not:
$executor = new Executor($loop, new Parser(), new BinaryDumper());
$executor = new RetryExecutor($executor);
$resolver = new Resolver('<IP_ADDRESS>:53', $executor);
$executor->resolve(null)
    // Once the promise is rejected, `done()` throws an exceptions, since there is no `onRejected` callback.
    // However, the exception is caught by RetryExecutor's `then()`.
    // This PR fixes this.
    ->done(function ($result) {
        ...
    });

The PR fixes this by changing some then to done.

Thanks for spotting! These could certainly introduce some subtle and hard to trace issues.
I'm currently in the process of refactoring quite a bit of the dns component, so I'm considering how I'm going to introduce this.
FWIW: These changes only work with the latest Promise API, so we would also have to raise our minimum requirement.

Thanks.
Currently, catching issues in user code is made very hard because of this, as soon as this then is part of the call path.
Just for my information, what kind of issues could this change introduce ?

FWIW: These changes only work with the latest Promise API, so we would also have to raise our minimum requirement.

What are the rules for raising the requirements against a dependency that's also part of reactphp ? Could this be done in a MINOR (i.e. not PATCH) release of react/dns ?

I'm currently in the process of refactoring quite a bit of the dns component, so I'm considering how I'm going to introduce this.
Just for my information, what kind of issues could this change introduce ?

After reconsidering this, I'm leaning towards getting your changes in first. After all you've filed your PR first :-) I'll have to resolve a few merge conflicts because I'm looking into introducing quite a bit of new structure, but that's nothing you have to worry about.

What are the rules for raising the requirements against a dependency that's also part of reactphp ?

This will likely be tackled once it comes to tagging the next release. In this particular case it does not affect the public API, so this will likely end up in a patch release.
Can you update the composer.json?
Otherwise LGTM :+1:
ping @WyriHaximus, @jsor

LGTM :+1:

Can you update the composer.json?

Ping @arnaud-lb, can you update this to require react/promise: ~2.2?
Otherwise LGTM :+1:

LGTM :+1:

done seems to have caused a test to fail for PHP 5.3

Yes, because react/promise ~2.0|~1.1 is required and it installs ~1.1 on PHP5.3 (~2.0 requires 5.4) whoch has no done.

This will also cause a merge conflict with #35. Is this still relevant?

@clue Yes it is :)

@clue: I've rebased the PR against master, to fix the conflicts.
Also, I've updated the composer requirements (react/promise ~2.2), and removed PHP 5.3 from travis' php targets.

@arnaud-lb Thanks for filing this PR, keeping it updated and sparking the discussion! :+1:
I'm not opposed to getting this in, but I'm currently unsure about how much value this PR (still) provides.
I think the original PR actually addressed a relevant issue of possibly discarding exceptions in the RetryExecutor, but this was actually already addressed as part of #48 in the meantime.
This means that now, the current PR only avoids discarding exceptions in a single location (cache) that never throws exceptions in the first place. To get this PR in, we'd have to sacrifice backwards compatibility with completely outdated versions (which I'm okay with).
As an alternative, I would rather address the cache issue in #81 instead, at which point this update is no longer required in the first place.
What do you think about this? Every way, again thanks for your contribution! :+1:

@arnaud-lb Thank you for working on this! I'm closing this for now as it hasn't received any input in a while and I believe this has been answered. Please come back with more details if this problem persists and we can reopen this :+1:+


Dispatch: Add craftsman search & filter by issue statistics

Proposals:

Filter by statistics fields
Search by Craftsman. Autofocus input on navigation.

No construction site has sufficiently many craftsman to make this useful


Another interfaces.php text change to IPv6

This one was missed in recent fixes around this area.

@sbeaver-netgate This should be any easy one to review and commit in 5 seconds :)

Already committed by @sbeaver-netgate separately.


Chorded shortcuts should support context

Issue Type: Bug
Currently it is possible to specify context conditions on keyboard shortcuts, but not on the first element of a chord. This means that if I have, for instance, Ctrl+A as a first chord element in /any/ shortcut, even if the shortcut is only active in a specific context, then Ctrl+A will be "reserved" for that chord even in irrelevant contexts.
(Usecase in point: I have several editor-bound actions tied to chords that start with Ctrl-A, but this prevents me from using the normal  meaning of Ctrl+A (i.e. select all) in other contexts such as in search fields.)
 Here is an example demonstrating that this works:
    {
        "key": "ctrl+a ctrl+x",
        "command": "closeFindWidget",
        "when": "findWidgetVisible"
    }

open a code editor
press ctrl+a; it will be dispatched to the builtin select all command
open the find widget (ctrl+f). This action turns findWidgetVisible to be true
press ctrl+a; it will enter chord mode

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


VMSA_GPA = 0xFFFFFFFFF000 is 48-bit,  confused....

https://github.com/IBM/sev-snp-measure/blob/90f6e59831d20e44d03d5ee19388f624fca87291/sevsnpmeasure/gctx.py#L40
in the code, VMSA_GPA = 0xFFFFFFFFF000, but 0xFFFFFFFFF000 is 48-bits, not 52-bits
when launch sev-snp vm, we use cbitpos=51,reduced-phys-bits=1, so i think it use 51-bit (0-50) physical address space, with cbit=1, so should VMSA_GPA = 0xFFFFFFFFFF000 ?

I agree, this value is only 48-bits. I also see the same value in snp-digest-rs:
https://github.com/slp/snp-digest-rs/blob/0e5a787e99069944467151101ae4db474793d657/src/main.rs#L86
Note that with the current value I'm able to successfully verify launch digests of SNP guests, so the only question is why the PSP is using this 48-bit value when filling the PAGE_INFO (SNP spec table 67).
Section <IP_ADDRESS> of the SNP spec says:

• GPA: The gPA of the 4 KB page. The firmware uses the RMP.GPA of the page.

And in the host kernel, the RMP setting of the VMSA page is set with gpa = (u64)-1 (second argument in this call):
https://github.com/AMDESE/linux/blob/3a15da8647b99d147ed3ae8daf7cfdb19b20d83c/arch/x86/kvm/svm/sev.c#L2188

Section 8.17.2 provides some further detail. The input GPA must be page aligned (CMDBUF_SNP_LAUNCH_UPDATE structure, byte offset 0x10, bits 11:0 must be zero) and the second from the last paragraph of that section states "the guest physical address space is limited according to CPUID Fn80000008_EAX and thus the GPAs used by the firmware in measurement calculation are equally limited".
Given that information and that CPUID Fn80000008_EAX[PhysAddrSize]=0x30:
-1 == 0xFFFFFFFFFFFFFFFF => 0xFFFFFFFFF000 is used as the VMSA GPA during measuring.
I think the hypervisor patches should be made a bit clearer by performing this actual transformation of the supplied address.

Sorry, I confused the SPA and the GPA in the first part of that last response. The spec should probably be updated to clarify that the GPA will be page aligned and thus bits 11:0 will be zero.


Compatible with openApi 3.1

Hi. I know the bug is not on your side.
The bug come for the package OpenApiPSR7Validator.
I open a PR for it but, for what I can see, there is no more activity on it.
Do you plan to change the package for another one like opis/json-schema ?

Salut Philippe,
I have no plan to change the underlying library no, nor do I have the bandwidth to look into it, if I'm honest.
Plus, OpenApiPSR7Validator currently does a lot of the heavy lifting, while from what I gathered after a very quick look at the library you've suggested, it would be quite a bit of work to adapt it to the particular purpose of validating PSR7 messages

Hi.
Thanks for the response.
What is the best for you between :

Add a fork of the Validator to update to be compliant  with OpenApi 3.1 and use it instead
I see to work on the migration to the other library

I reckon the latter, since otherwise you go from maintaining a single library to maintaining a library + a fork ^^


Add ADR for decision to copy Apply from CAS1

@tahb I want to leave this for a bit then come back to it with fresh eyes. Any feedback in the meantime would be appreciated


Add OS Image features to Az images

What this PR does / why we need it:
Adds the following image features to images published to Azure to support latest machine types with Garden Linux:
IsAcceleratedNetworkSupported: True
DiskControllerTypes: "SCSI, NVMe"
Which issue(s) this PR fixes:
Fixes #58
Special notes for your reviewer:
Release note:

NVMe and Accelerated Features are added to Garden Linux images on Azure

I'm assuming the removed imports were unused

Exactly, they were.


Power Platform Licensing Article Changes

Article issue
After some discussion with my Microsoft counterparts, we'd like to slim down the article to remove the costs and limits and replace those sections with links to the official Power Platform documentation. While one of the goals of this article is to make the licensing more understandable, we won't be able to stay up to date with any changes in the details reliably.
By doing this, we can also shift the focus of the article to better discuss more about why the Power Platform tools are important in Microsoft 365 and what we can do with the tools.
Yes, this does feel a bit like a retreat, but I think it is a reasonable one. @bigpix2000 - let's discuss this here (out in the open) and we can come up with the best way to make the changes.

Thank you. I originally want to use links instead of direct displays of tables with the limits and costs but the main source appears to be the PDF and no good way to link to them. Can we ask Microsoft to put that into into linkable doc form then we can definitely work out a more spartan (and focused) version of the article.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Marc D Anderson<EMAIL_ADDRESS>Sent: Friday, September 11, 2020 11:50:36 AM
To: MicrosoftDocs/microsoft-365-community<EMAIL_ADDRESS>Cc: Ralph Rivas<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: [MicrosoftDocs/microsoft-365-community] Power Platform Licensing Article Changes (#315)
Article issue
After some discussion with my Microsoft counterparts, we'd like to slim down the article to remove the costs and limits and replace those sections with links to the official Power Platform documentation. While one of the goals of this article is to make the licensing more understandable, we won't be able to stay up to date with any changes in the details reliably.
By doing this, we can also shift the focus of the article to better discuss more about why the Power Platform tools are important in Microsoft 365 and what we can do with the tools.
Yes, this does feel a bit like a retreat, but I think it is a reasonable one. @bigpix2000https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbigpix2000&data=02|01||f4fafdabf2f042433e0108d85672cfe1|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637354398391460212&sdata=07p2V90rn7Adr15Nw6X14Nd3FknxDKMSl9QHik5y1OQ%3D&reserved=0 - let's discuss this here (out in the open) and we can come up with the best way to make the changes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fmicrosoft-365-community%2Fissues%2F315&data=02|01||f4fafdabf2f042433e0108d85672cfe1|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637354398391470187&sdata=BPXrr0vprdXgZ9NL09QxcGxtOGqqb1xk3g%2FQ6xN1GVA%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAB2OOTTFYAELIIIBQH3TYODSFJINZANCNFSM4RH4AF5A&data=02|01||f4fafdabf2f042433e0108d85672cfe1|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637354398391480176&sdata=u3AK%2FE0h6tc5tRla7iRhgQKiM3XFN4RQQRDkRkVhdXU%3D&reserved=0.

Great point! Let me look into that.

Back to this issue as we will address the bullet item bug #342 along with this change.  Here is what it will look like:

Updated intro section into what I came up with as the "half page summary" (tested with Ryan Cunninghman
{https://twitter.com/rc_says}   Prod Mgr power platform as to validity) then a list of the key licensing levers (capacity related items, specific services, specific bundles), the lookups (although this may be copies of the pdf tables if we truly can't find links) and finally the scenarios (there would be one to three common scenarios at the intro for quicker access leaving the "nuance" ones below.  adding @LuiseFreese to this activity and for her feedback as well on this organization.  thoughts?  let's discuss openly!

few thoughts: Hosted this QnA Licensing session and really got some good insights - working right now on a summarizing blog post with some key-take aways. Would love to point out my maker perspective on Power Platform and why this is not some "nice to have" thingy at the end of M365 adoption road.

@LuiseFreese , the perspective you mention is EXACTLY the spirit of the document and also a reason why every effort to "optimize" the information is in order.  On that note, let me repost the "proposed" sections for where the information we have now will find new homes (or be removed because it is superfluous because of a link, etc.)

Overview: why this doc, who is it for, how to use this doc
Core licensing: my "half page" piece which I summarized at the end of the meetup with Ryan but hopefully more artfully written with perhaps three "scenarios" that exemplify the core concepts in action (e.g. app plus flow to o365, flow to premium, AI\RPA with Flow)
Licensing Levers: organized by Feature (App, Automate, Virtual Agents) and Lever Type (Trigger, Capacity, AI, RPA, Portal for Apps
The "lookups" :  This one is a discussion point because here is where @sympmarc suggested links which would be great if they were not hiding in a PDF so some way to optimize without making it obsolete as soon as it is posted needs to be determined.  We might not have a choice and I will have to set a flow to remind me to update it monthly ;-)
Guidance ( missed this one in my last post) This one is another discussion as it is the key contribution of my experience and John L. and it was supposed to entice practices or additional guidance from the community. For folks making decisions, they would take this section to heart.
the Scenarios:  My vision originally was that I would start off with a few scenarios that mix up the pieces above but also entice the community to "submit their scenario" and say what they pay for or had to get (e.g. we got the stand along apps license and we are paying for extra CDS capacity) . This would include a preface section for Automate vs Azure.  Scenarios are not recommendations as much as they are "if you do X, your consequence is Y"

looking forward to the thoughts!!

Good to see the discussion here. I'm fairly ignorant about the details of all this, so I leave it in your capable hands!

hi there @bigpix2000 - let's work on that and see how it goes. I would like to write overview and happy to rewrite  the core licensing thing. I think the way I explained it in my blog post here is pretty much that? my blogpost about PP licensing QnA  -
I agree that PDF are usually a way to hide information..
guidance should be a pretty decent when to use what / most of the time it will be an AND and not an OR. complexity doesnt come with simple answers :-)
scenarios and examples are a good idea in addition.
So if you agree I will start with overview and a draft for core licensing... where shall I do that? (Here in the comments? shall I share an md file via OneDrive? Still new to this here.

So ..... for authoring, technically you can author in place and we can go back and forth on it like a project but you may also use this space for the content to be reviewed as the same MD tools work here as well.  The overview re-write is a good place to start. As for the core licensing pieces, your blog does cover the discussion and can be used as the model for the "Core" section.  Because of the links issue, I believe we still need to have the additional sections I noted above.  Think of it like this: depending on what the person has come here to see, having the "summary" so they can go away sooner is great so the detail is built up from the overview (to perhaps immediately answer a question) to the Core (the "fast" FAQ in line with your blog) and then "the nitty gritty" where items that took many pages in the PDF files are covered in short and then, at the end the "Scenarios" which I envisioned would grow with community contributions.
Action item then will be for you to (re) Create the Overview as a draft here in notes. Mention me to follow up and we go back and forth there until its good.  Then same thing with the Core.  After that, depending on if we go Live with Oakdale, I can work on the other sections and you can help review or take up a specific on a section even.   Sound like a plan?  we can have this done before our Chicago M365 Virtual Conference and my session for licensing will highlight our work :)  Danke!! @LuiseFreese

ok, as I wanted to start SOMEWHERE, I did this:
Please feel free to correct, add, elaborate.
Power Platform Licensing
Overview
This article shall give you a  detailed and easy to understand overview about licensing/pricing regarding Power Platform.
Power Platform consist of Power Apps, Power Automate, Power BI and Power Virtual Agents and has Portals and AI Builder as add-ons.
Core licensing
We distinguish between Power Apps for Office 365 and Power Apps Standalone
Power Apps and Power Automate for Office 365
Power Apps and Power Automate for Office 365 means customizing and extending Office with low code. Users can build and run unlimited amount of apps and flows using SharePoint, Excel and dozens of other standard connectors. Now in public preview: Project Oakdale in Teams (“CDS light”). For reference, find a list of all Power Automate Standard Connectors.
Project Oakdale
Project Oakdale (in public preview) is basically CDS speaking of database and environment but with a more simple security model, but it’s the basic structure of a high scale relational database with tables and the ability to use them in an app.
About Project Oakdale environment
Project Oakdale
Power Apps Standalone
per user
Power Apps "per user" plan is the equivalent to an "All You Can Eat" Buffet. It means, that users can build and run as many apps using Premium Connectors as they wish.
per app
Power Apps "per app" plan is the equivalent to an à la carte menu. It means, that users can build and run a single app.
Any flow that’s working on the same data as the app, doesn’t require an additional license for the flow: We don’t need multiple licenses just to run one use case.
Licensing Levers
organized by Feature

App
Automate
Virtual Agents

organized by Lever Type

Trigger

Capacity

AI
For AI Builder, there is a gorgeous calculator in which you can calculate the monthly costs.

RPA

Portal for Apps

The "lookups"
This one is a discussion point because here is where @sympmarc suggested links which would be great if they were not hiding in a PDF so some way to optimize without making it obsolete as soon as it is posted needs to be determined. We might not have a choice and I will have to set a flow to remind me to update it monthly ;-)
Guidance
( missed this one in my last post) This one is another discussion as it is the key contribution of my experience and John L. and it was supposed to entice practices or additional guidance from the community. For folks making decisions, they would take this section to heart.
the Scenarios:
My vision originally was that I would start off with a few scenarios that mix up the pieces above but also entice the community to "submit their scenario" and say what they pay for or had to get (e.g. we got the stand along apps license and we are paying for extra CDS capacity) . This would include a preface section for Automate vs Azure. Scenarios are not recommendations as much as they are "if you do X, your consequence is Y"

So @LuiseFreese , now that we are back in the game, the obvious first change will be to Dataverse and away from Oakdale.  Based on the notes so far and the kind of questions and feedback we have seen in just the last couple of months, consider the following organization:
Overview
summarizing what the doc is about and its purpose and that it does NOT replace the licensing detail, or all those licensing bits posted all over docs
Core Licensing Concepts
We call out specifically in key bullets or sub headers, the following main concepts:

Standalone versus "Seeded"  (what they call in in the docs and, yes, we translate this to standalone versus in Office/Dynamics ... I would probably put in here the case of "is it included or will it cost)
Premium versus Standard (again, the case of if it will cost)
Capacity (what you get "included" when you have a license and the idea that you can always pay for more)
The Dataverse exception: the concept of when you are using Teams, some premium items are no longer premium but included .... with the catch
Power Apps is based on Apps you run, Power Automate is based on Flows you Create.

And then we break this down for additional information:
Licensing Levers
(as noted above and the goal here is to unpack the core concepts and yes, include the link to the AI calculator)
Guidance
(There will be some major bullets I see here as dataverse totally changed much of what was put up before!)
Here are some bullet highlights (just examples but perhaps maybe ...?)

Do your App\Flow in Teams wherever possible
Dataverse over Other Data "premium connections", and free version over SP Lists for functionality
know your data to pick the right sized data repo and the repo that will cost the lease when you push it through to Power BI
Take advantage of any free thing wherever possible (reminder of the community environment!)
Think Governance early not after (control costs, set up "samples" for others to follow etc) corollary:  Adopt best practices or implement the COE
The license docs are updated often (lately its been monthly) Take time to review changes as often as you can to catch those not heavily publicized.
Think of your scale and what that is worth to you or your organization.

Scenarios
initially at least one to exemplify the core concepts and then a call to action for contributions.
Links
The (hopefully not to be changed) direct links to the sources such as the main PDF and the starting points for the various license details
Once we are final on the organization, I will make the change in a workspace and we can share that for editing .... we will advise @sympmarc when we agree we are "done" :-)

So @LuiseFreese , now that we are back in the game, the obvious first change will be to Dataverse and away from Oakdale.  Based on the notes so far and the kind of questions and feedback we have seen in just the last couple of months, consider the following organization:
Overview
summarizing what the doc is about and its purpose and that it does NOT replace the licensing detail, or all those licensing bits posted all over docs
Core Licensing Concepts
We call out specifically in key bullets or sub headers, the following main concepts:

Standalone versus "Seeded"  (what they call in in the docs and, yes, we translate this to standalone versus in Office/Dynamics ... I would probably put in here the case of "is it included or will it cost)
Premium versus Standard (again, the case of if it will cost)
Capacity (what you get "included" when you have a license and the idea that you can always pay for more)
The Dataverse exception: the concept of when you are using Teams, some premium items are no longer premium but included .... with the catch
Power Apps is based on Apps you run, Power Automate is based on Flows you Create.

And then we break this down for additional information:
Licensing Levers
(as noted above and the goal here is to unpack the core concepts and yes, include the link to the AI calculator)
Guidance
(There will be some major bullets I see here as dataverse totally changed much of what was put up before!)
Here are some bullet highlights (just examples but perhaps maybe ...?)

Do your App\Flow in Teams wherever possible
Dataverse over Other Data "premium connections", and free version over SP Lists for functionality
know your data to pick the right sized data repo and the repo that will cost the lease when you push it through to Power BI
Take advantage of any free thing wherever possible (reminder of the community environment!)
Think Governance early not after (control costs, set up "samples" for others to follow etc) corollary:  Adopt best practices or implement the COE
The license docs are updated often (lately its been monthly) Take time to review changes as often as you can to catch those not heavily publicized.
Think of your scale and what that is worth to you or your organization.

Scenarios
initially at least one to exemplify the core concepts and then a call to action for contributions.
Links
The (hopefully not to be changed) direct links to the sources such as the main PDF and the starting points for the various license details
Once we are final on the organization, I will make the change in a workspace and we can share that for editing .... we will advise @sympmarc when we agree we are "done" :-)

hi there, I struggled with commenting here and scrolling up and down so I put my takes into my blog repo, its public and you will find it here:
Power Platform Licensing
Let me know what you think @bigpix2000

hi there, I struggled with commenting here and scrolling up and down so I put my takes into my blog repo, its public and you will find it here:
Power Platform Licensing
Let me know what you think @bigpix2000

Got busy and will be deep diving the link above .... interestingly I got pinged by the community on License questions that actually gave some validation for the pieces we are working on as well as a note for one additional (or modified) sample that is current.

Got busy and will be deep diving the link above .... interestingly I got pinged by the community on License questions that actually gave some validation for the pieces we are working on as well as a note for one additional (or modified) sample that is current.

Got busy and will be deep diving the link above .... interestingly I got pinged by the community on License questions that actually gave some validation for the pieces we are working on as well as a note for one additional (or modified) sample that is current.

Got busy and will be deep diving the link above .... interestingly I got pinged by the community on License questions that actually gave some validation for the pieces we are working on as well as a note for one additional (or modified) sample that is current.

Got busy and will be deep diving the link above .... interestingly I got pinged by the community on License questions that actually gave some validation for the pieces we are working on as well as a note for one additional (or modified) sample that is current.

Got busy and will be deep diving the link above .... interestingly I got pinged by the community on License questions that actually gave some validation for the pieces we are working on as well as a note for one additional (or modified) sample that is current.