[Event Request] Application “Tests-CRM Integration”, Codeunit 139173 "CRM Synch. Helper Test"

[Event Request] Application “Tests-CRM Integration”, Codeunit 139173 "CRM Synch. Helper Test"

Could you please insert an OnTestInitialize-Raiser at the beginning of the following procedures?

TestUpdateCRMInvoiceStateActive
TestUpdateCRMInvoiceStatePaid
TestUpdateCRMInvoiceStatePartial
TestUpdateCRMInvoiceWithCreateBillsPaymentMethod

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


no luck w/ running on Arch Linux

Description
There are no easy way to run these patches on Arch linux at all.
e.g. Crash on launching latest .AppImage with suggestion to update the latest linux kernel to use sandbox.
also, i have crash on building/compiling ungoogled-chromium w/ failing write files.
i saw that u have no responsibility for installing ungoogled-chromium-bin from aur,but anyway: crash when trying to launch fresh-installed ungoogled-chromium-bin
(fish: “chromium” terminated by signal SIGSEGV (Address boundary error))
Console output and logs
s1nceri7y@s1nceri7y ~/Downloads>  ./ungoogled-chromium_69.0.3497.100-2_linux.AppImage  [9540:9540:1024/214814.525529:FATAL:zygote_host_impl_linux.cc(116)] No usable sandbox! Update your kernel or see https://chromium.9oo91esource.qjz9zk/chromium/src/+/master/docs/linux_suid_sandbox_development.md for more information on developing with the SUID sandbox. If you want to live dangerously and need an immediate workaround, you can try using --no-sandbox. /tmp/.mount_ungoogJhm06p/AppRun: line 4:  9540 Trace/breakpoint trap   (core dumped) "${HERE}"/opt/google/chrome/chrome --password-store=basic $@ s1nceri7y@s1nceri7y ~/Downloads> uname -a Linux s1nceri7y 4.18.16-arch1-1-ARCH #1 SMP PREEMPT Sat Oct 20 22:06:45 UTC 2018 x86_64 GNU/Linux
chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/lower_bound1.fail.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/count.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/lower_bound3.fail.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/find.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/count1.fail.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/count_transparent.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/equal_range3.fail.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/equal_range1.fail.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/count0.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/find1.fail.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.ops/lower_bound.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/compare.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/index_rv_key.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/index_tuple.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/at.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/empty.fail.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/index_key.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/size.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/iterator.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/max_size.pass.cpp: Write failed chromium-69.0.3497.100/buildtools/third_party/libc++/trunk/test/std/containers/associative/map/map.access/empty.pass.cpp: Write failed ^C
Environment details

ungoogled-chromium version: latest available from site and aur (69.0.3497.100-2)
OS and version for building: Linux s1nceri7y 4.18.16-arch1-1-ARCH #1 SMP PREEMPT Sat Oct 20 22:06:45 UTC 2018 x86_64 GNU/Linux
Compiler toolchain: dunno. most probably i have latest version of gcc.

i enabled users namespaces, but no luck
`> sudo sysctl kernel.unprivileged_userns_clone=1
kernel.unprivileged_userns_clone = 1

./chrome-wrapper
fish: “./chrome-wrapper” terminated by signal SIGSEGV (Address boundary error)
bash
[s1nceri7y@s1nceri7y ungoogled-chromium_69.0.3497.100-2_linux]$ ./chrome-wrapper
Segmentation fault (core dumped)
`

after trying to launch from root w/ --no-sandbox option( which was failed too) and reassigment the unprivileged userns clone variable, i have another error (w/ delaying on ~0.5 sec before printing "Segmentation fault":
[s1nceri7y@s1nceri7y ungoogled-chromium_69.0.3497.100-2_linux]$ ./chrome-wrapper  [5145:5205:1025/184834.633308:ERROR:data_store_impl.cc(131)] Failed to open Data Reduction Proxy DB: 3 Segmentation fault (core dumped) [s1nceri7y@s1nceri7y ungoogled-chromium_69.0.3497.100-2_linux]$

@s1nceri7y In the first post, you were using an AppImage. Are you now using the Portable Linux build? What happens if you try the AppImage again?

at the beggining i was looking for easy ways, but then i downloaded portable build.
here's what i have for now  when trying to launch an appimage build
s1nceri7y@s1nceri7y ~/Downloads>  ./ungoogled-chromium_69.0.3497.100-2_linux\(1\).AppImage  [7203:7219:1025/190231.321316:ERROR:data_store_impl.cc(131)] Failed to open Data Reduction Proxy DB: 3 /tmp/.mount_ungoog2VYYdS/AppRun: line 4:  7203 Segmentation fault      (core dumped) "${HERE}"/opt/google/chrome/chrome --password-store=basic $@ s1nceri7y@s1nceri7y ~/Downloads>

Same error with both builds... stick with the portable version to debugging...

Do you have the same issue with chromium ?
Do you have same issue with v67 portable ?
How did you disable namespaces in the first place
Are you using a custom kernel ? what's the output of "uname -a" ?
Try to run it with a different user or with a new profile

Only one of them, but i still able to run it.
I'm a little bit busy. i'll try later
They are disabled on Arch Linux by default.
I'm using original kernel (i mentioned my kernel in Enviroment info)
`Linux s1nceri7y 4.18.16-arch1-1-ARCH #1 SMP PREEMPT Sat Oct 20 22:06:45 UTC 2018 x86_64 GNU/Linux
s1nceri7y@s1nceri7y ~>
I'll try later, but i just noticed i haven't removed my profile from Chromium before uninstalling (i was logged in google account). i'll remove it and try.
Thanks for helping me!

yes, that was a deal: i wasn't able to launch ungoogled-chromium cause of remaining google acount i hadn't log out and disabled namespaces by default.
thanks.
i think it's a good idea to add removing existing cookies and etc before installing ungoogled-chromium

Cool :) i am glad that helped, can you close the issue please

Opened #568 to discuss crashing due to existing Google account.


Add send_payment and send_zkapp mutations to limited GraphQL server

This is the only mutations offered by that server.

!ci-build-me

Internal discussion #1.
Internal discussion #2.

!ci-build-me

Rampup is retired, this PR is now orphaned. Closing.


Component Taxonomy match with Fiori

Description
The Fundamental UI Components should have the same name as their Fiori counterpart
First matching
https://wiki.hybris.com/display/yintfdes/Comparison+of+Fundamental+and+Fiore+Components
Tasks

[x] Fundamental Components comparison against Fiori Components
[x] Define the nomenclature and Taxonomy changes
[ ] ~Identify missing components in Fundamental UI.~ Posponed

@jeannewalters
I have reviewed the results of the investigation and there is some improvements that can be made:

"All of what is labeled as a Button in Fundamental UI is part of "Action" submenu in Fiore." - This statement is not so clear, we have different types of Buttons, The Dropdown Menu Fundamental have is a different component.
The screenshot of the "Lists" under "Cards" is not really Cards that's a normal table where you can select/check the rows. See the example: http://sap.github.io/techne/table.html#table-with-checkbox
The Dropdown component is not exactly the same as the ComboBox in fiori. That is still to be decided if a component like that will be part of the Fundamental UI Dropdown or a new component with that functionality.

All others are fine, the similarity is there.

been done on the fundamental repo


[nit] Upgrade the minimum Python version to 3.8 in docs

I noticed that the minimum Python version mentioned in the documentation has not been updated. It is updated by https://github.com/z3z1ma/dbt-osmosis/commit/e59809093ba1710a3c878ba53473495969545994.

Thanks!


Fix code scanning alert - Potentially unsafe external link

Tracking issue for:

[ ] https://github.com/mertturkmenoglu/horizon/security/code-scanning/3

fixed with the latest commit


Add common messagebus api

Add common messagebus/ipc api and adapters for browser (Default), Electron and OpenFin
resolves #8

The codacy/pr check is good.  It was based off of old settings while setting up (and is now turned off for PRs)
https://www.codacy.com/app/brian.ingenito/desktopJS/pullRequest?prid=672887


support buildinfo in version sub-command

Intent of this PR is to include VCS version info as described in #1697. Note that the behavior of the command is unchanged in versions of Go before 1.18, since the embedding of VCS information appeared in Go 1.18.
Fixes #1697
Signed-off-by: Dave Josephsen<EMAIL_ADDRESS>

Imported as https://review.gerrithub.io/c/cue-lang/cue/+/540297; should be merged this week, once I've got an approval there.


Build fails on Dart 3.6.0/Flutter 3.27.0

[        ] [+8967 ms] ../pub_cache/hosted/pub.dev/emoji_extension-1.1.1/lib/src/emojis/emoji_parser/emoji_parser_getters.dart:305:13: Error: The property 'shuffled' is defined in multiple extensions for
'List<String>' and neither is more specific.
[        ] [   +1 ms]  - 'List' is from 'dart:core'.
[        ] [        ] Try using an explicit extension application of the wanted extension or hiding unwanted extensions from scope.
[        ] [        ]     extract.shuffled.forEachIndexed((i, e) {
[        ] [        ]             ^^^^^^^^
[        ] [        ] ../pub_cache/hosted/pub.dev/collection-1.19.0/lib/src/iterable_extensions.dart:69:11: Context: This is one of the extension members.
[        ] [        ]   List<T> shuffled([Random? random]) => [...this]..shuffle(random);
[        ] [        ]           ^^^^^^^^
[        ] [        ] ../pub_cache/hosted/pub.dev/emoji_extension-1.1.1/lib/src/extensions/iterable_extensions.dart:241:15: Context: This is one of the extension members.
[        ] [        ]   List<E> get shuffled => ([...this]..shuffle()).toUnmodifiableList();
[        ] [        ]               ^^^^^^^^

Hi!
I've adjusted the code to work with newest version of package collection (1.19.1)
Newest version of Flutter (3.27.0) uses package collection (1.19.0)
So when using emoji extension with Flutter 3.27.0 dependency ovveride is needed:
See flutter_examples:
https://github.com/Nikoro/emoji_extension/blob/75ac83b67a713402d974b3f98e5bbbe9072f3af4/flutter_examples/emoji_chat_app_example/pubspec.yaml#L15-L17


feature(form-field) create new component to handle errors and hints

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our
guidelines: https://github.com/goetzrobin/spartan/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Which package are you modifying?

[ ] accordion
[ ] alert
[ ] alert-dialog
[ ] aspect-ratio
[ ] avatar
[ ] badge
[ ] button
[ ] calendar
[ ] card
[ ] checkbox
[ ] collapsible
[ ] combobox
[ ] command
[ ] context-menu
[ ] data-table
[ ] date-picker
[ ] dialog
[ ] dropdown-menu
[x] form-field
[ ] hover-card
[ ] icon
[ ] input
[ ] label
[ ] menubar
[ ] navigation-menu
[ ] pagination
[ ] popover
[ ] progress
[ ] radio-group
[ ] scroll-area
[ ] select
[ ] separator
[ ] sheet
[ ] skeleton
[ ] slider
[ ] sonner
[ ] spinner
[ ] switch
[ ] table
[ ] tabs
[ ] textarea
[ ] toast
[ ] toggle
[ ] tooltip
[ ] typography

What is the current behavior?
None
Closes #294
What is the new behavior?
Create a new component to handle errors and hints like the material functionality but with the shadcn styles.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

I added some comments! I really like the idea of this and the overall approach seems fine to me, too. One big question I have: Is there a way to support this with uncontrolled form controls, too? e.g. just add a (host) directive to any form control that's Angular form compatible and automatically get the functionality, instead of having to manually update the form state on every doCheck

When you say uncontrolled form controls, you mean inputs or selects without any of these (FormControl or formcontrolName).
Also I think it is not possible to use this validation as HostDirective because only some components must have the errorState, another point is each component that will be used inside the FormFIeld component should be use the BrnFormFieldControl in the providers, to be identified by the parent(form-field), additionally using the update of the error in the ngDoCheck is the only way, at least that I know of, where I can reevaluate on every change detection when the error is triggered.

Hi @thatsamsonkid and @goetzrobin thanks for your time, i think i resolve each doubt, please let me know if you have any question.
One point I would like to understand, is how can I test the generator for this new component, I have created it but I am not sure how to test it.

any thoughts here?, can we merge it? @goetzrobin


[BUG] Deadlock in AzureCliCredential when executed concurrently.

This issue and the following repro was reported by @NeilMacMullen, and appears to be a different deadlock than that described in #14691, as it only reproduces when multiple requests are made in parallels.
Repro:
  public class SimpleKeyVaultAccess
  {
      public static readonly SimpleKeyVaultAccess Primary = new(KeyVaultNames.SensizePrimary);
      private readonly string _vaultName;
      private SimpleKeyVaultAccess(string vaultName) => _vaultName = vaultName;
      public async Task<string> FetchSecret(string secretName)
      {
          var vault = new Uri(_vaultName);
          var client = new SecretClient(vault,
              new ChainedTokenCredential(new ManagedIdentityCredential(), new AzureCliCredential()));
          var response = await client.GetSecretAsync(secretName);
          return response.Value.Value;
      }
      public async Task DemonstrateTimeout(int n)
      {
          Console.WriteLine($"Running with {n} parallel tasks..");
          var tasks = Enumerable.Range(0, n).Select(_ => FetchSecret(SecretNames.TablesAccount)).ToArray();
          var all = await Task.WhenAll(tasks);
          Console.WriteLine("Succeeded");
      }
  }

  await SimpleKeyVaultAccess.Primary.DemonstrateTimeout(50);

The code is called from a .net5 console app running on my dev PC in the UK.  The key vault itself is in the West US2 region (so network latency may be relevant).  It reliably throws
Azure.Identity.AuthenticationFailedException: The ChainedTokenCredential failed due to an unhandled exception: Azure CLI authentication timed out.
after approximately 25 seconds

I am not able to repro this. Duplicate of #23135


ERROR: Exception on thread Thread on Huawei devices

[ ] I have read the FAQ.
[ ] I have searched in existing issues.

Environment

OS: Windows 10
scrcpy version: v1.17
installation method: Windows release
device model: Huawei Y7p & Huawei Nova 5i
Android version: 9

Describe the bug
It work fine with my other devices but not for this Huawei devices, is it because the devices using HMS Core? I did try using the server fix from previous comment which the Meizu one but still same problem occurred. It appeared error as can be seen in the screenshot below.

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#exception

ok cool! Thanks.


Import two legacy mapping tables into the legacy stylx

SQL script, versions.csv, and README changes.

Repeated the legacy mapping stylx creation process successfully! Stylx has two legacy mapping tables now:

I'll go ahead and merge
@abouffard @csmoore


HDR Resize Bug

Render: OpenGL
Hardware: i7 4790K,GTX970

Start Engine in window
Run Editor
Do resize(using in windows 7 style-move window pressed mouse on header)
See HDR bug

Thanks for the report!

Hey guys, just testing something out.
Here you can find a small bounty on this issue. If you fix it you can collect it :P

But first you have to find a way to reproduce it ;P.

Didn't test it but seems like it's a 4-step process to repro. Make sure to notice that it is in OpenGL :P

@galek , can you post a comparision screenshot? For have a better idea about the problem.

I think @Azaezel reproduced it too.

yes and no. Managed to sporadically get white flickers that seemed to change as the sun went down while the console was open on outpost by resizing it. Couldn't consistently reproduce it however, nor reduce it down to hdr being on or off.

On Thursday I will do this until there is no time.
For Note:Render was modificated for support Deferred Shading from some fork(forgot name)

ok, 2 things.
A) That hasn't been rolled in yet, so you want to fill a ticket either https://github.com/andr3wmac/Torque3D/issues or https://github.com/Azaezel/Torque3D/issues depending on which one you grabed.
B) Tried the same reproduction outlined here a couple of times, and again, came up with no result. Whichever version you've got, could you include a log and a sceeenshot to clarify the bug?

Ok so if this isn't an actual stock issue, then I'll be revoking the bounty :P

Please open a new issue at the relevant repository, linking to this one if you need to :).


added windows 11

after reading this article, i confirmed windows 11 is supported

Learn Build status updates of commit 8d17f04:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows/security/threat-protection/security-policy-settings/shutdown-allow-system-to-be-shut-down-without-having-to-log-on.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


⚠ SolarQuery has degraded performance

In 9fa1c63, SolarQuery ($SOLARQUERY_URL) experienced degraded performance:

HTTP code: 200
Response time: 6932 ms

Resolved: SolarQuery performance has improved in e9e8469 after 51 minutes.


Add 1.X server compatibility with allow_unencrypted attribute

Description
Add the ability to set the allow_unencrypted attribute in the push-jobs config.
Issues Resolved
#83
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Looks good to me, assuming tests pass. One comment on one of the updates to 14.04 where it looks like it might have been left at 10.04 by accident.

Fixed the last 10.04

Thank you, this corrected the 1.x error.


count in any_xxx works

but cwtools tells me, its an error

does this work only with stuff like any_owned_xxx or any_controlled_xxx, or does it work as well with general stuff like any_ship or any_war?
anyhow, i suggest you make the changes here and submit as pull request:
https://github.com/tboby/cwtools-stellaris-config/blob/master/config/triggers.cwt

Well, I guess it works simply always, but I would have to test to be sure and I will not have time for it, sry.

So its confirmed. Can you add:
##cardinality = 0..1
count = int
To all any_ triggers?

yes

done

Great 😃

Done


Add elements plugins feature

"The time of the start has been passed.. away.
The chance to overcome conservative fears is born.
Like unexpected emotions, the potential features come.
Like in the book has been dreamed, the elders idioms are translated."
closes #19

I forked this into https://github.com/nullstack/nullstack/tree/experimental-plugins 💖


Fix typo in duplicating-a-repository.md

Why:
Changing /OLD-REPOSITORY.git/OLD-REPOSITORY.git to /OLD-REPOSITORY/OLD-REPOSITORY.git, probably due to a faulty copy/paste.

@dsotirakis Thanks for noticing this and submitting a PR! ⚡
We had actually just received another PR earlier for this same fix from the same article. So I'm going to go ahead and close this one as a duplicate. 💛
Thank you for passion and interest in updating the docs! 💖


[Merge Accounts] Add Menu Item, Route, and MergeAccountsPage Components

Part of the Merge Accounts project
Main issue: https://github.com/Expensify/App/issues/47073
Doc section: https://docs.google.com/document/d/1QhfgsJ0w-KoWX7r3RYNSk25DICp4C8XIfrFmULZgEgk/edit?tab=t.0#bookmark=id.bkjxhsrclw01
Project: https://github.com/Expensify/App/issues/47073
Feature Description
Steps to Complete:
1. Update Text and Add Icon:

Change text to:
"Keep your account even safer with two-factor authentication and other security features."
Add a new icon arrow-collapse.svg to the assets/images folder with the provided SVG content.

2. Add Translation Entry:

Add the following entry to translations:mergeAccountsPage: {
    mergeAccount: 'Merge accounts',
},

3. Create a New Route:

Add the following to the SCREENS file:MERGE_ACCOUNTS: 'Settings_Merge_Accounts',

Add to the ROUTES file:SETTINGS_MERGE_ACCOUNTS: 'settings/security/mergeAccounts',

Add the linking config:[SCREENS.SETTINGS.MERGE_ACCOUNTS]: {
    path: ROUTES.SETTINGS_MERGE_ACCOUNTS,
    exact: true,
},

4. Add New Menu Item:

Add the "Merge accounts" option in the menu with:{
    translationKey: 'mergeAccountsPage.mergeAccount',
    icon: Expensicons.ArrowCollapse,
    action: waitForNavigate(() => Navigation.navigate(ROUTES.SETTINGS_MERGE_ACCOUNTS)),
},

5. Create MergeAccountsPage Component:

Create a new component: pages/settings/Security/MergeAccountsPage.
Link it to the route by adding the following:[SCREENS.SETTINGS.MERGE_ACCOUNTS]: () => require<ReactComponentModule>('../../../../pages/settings/Security/MergeAccountsPage').default,

6. Define Steps for the Flow:

Add the following steps in the CONST.ts file:MERGE_ACCOUNT_STEPS: {
    ACCOUNT_DETAILS: 'ACCOUNT_DETAILS',
    ACCOUNT_VALIDATION: 'ACCOUNT_VALIDATION',
    MERGE_SUCCESS: 'MERGE_SUCCESS',
    MERGE_FAILURE: 'MERGE_FAILURE',
},

7. Create MergeAccountsContext:

Implement MergeAccountsContext to manage the state of the merge flow with properties:

currentStep
accountToMerge
result

Use useContext, useCallback, and useMemo for state and step transitions.

8. Create Subcomponents:
AccountDetailStep Component

Create a form for entering the account to merge, with a checkbox for confirmation.
Validate input and proceed to the ACCOUNT_VALIDATION step.

AccountValidateStep Component

Include a magic code validator form.
Make the API call to validate the merge and navigate to either MERGE_SUCCESS or MERGE_FAILURE.

MergeSuccessStep Component

Wrap around ConfirmationPage to display a success message.

Manual Test Steps
Offline Account Merge
Action To Take: Log into the test account and navigate to Security in the left-hand menu.
Action to Take: Go offline
Action To Take: Click on Merge accounts.
Verify: A full page offline screen displays since we cannot take this action while offline.
Successful Account Merge
Action To Take: Log into the test account and navigate to Security in the left-hand menu.
Verify: The Security page displays options:
Two-factor authentication
Merge accounts
Close account
Action To Take: Click on Merge accounts.
Verify:
A screen appears showing the target account email (e.g. danny@dannysdonuts.com).
The input field for “Email or phone” is visible.
The checkbox "I understand this is not reversible" is present.
Action To Take: Enter a valid email into the input field.
Verify: No error message.
Action To Take: Check the "I understand this is not reversible" checkbox.
Action To Take: Click on the Next button.
Verify: The screen transitions to the Magic Code Input step.
Action To Take: Enter the magic code sent to<EMAIL_ADDRESS>Verify:
A message displays confirming that a magic code was sent to the correct email address.
The input field for the magic code is visible.
Action To Take: Input the correct 6-digit magic code.
Verify: The “Merge accounts” button can be tapped.
Action To Take: Click the Merge accounts button.
Verify: The screen transitions to the Accounts Merged Confirmation page.
Action To Take: Review the confirmation message.
Verify:
The message confirms: “You've successfully merged all data from<EMAIL_ADDRESS>into<EMAIL_ADDRESS>The Got it button is displayed.
Action To Take: Click the Got it button.
Verify: User is redirected back to the Security page.
Account merge blocked due to trying to merge an account with SAML into a personal account
Prerequisite: Set up a new private domain with domain control and require SAML. Create a new domain member email to use for this test.
Action to Take: Sign into New Expensify with a public domain account.
Action to Take: Attempt to merge the SAML domain member account into the public domain account.
Action to Take: Check the “I understand this is not reversible” checkbox.
Action to Take: Tap the “Next” button
Verify: The “Can’t merge accounts” screen appears and explains why the action cannot be taken i.e. “because your domain admin has set it as your primary login”.
Account merge for users on domain with SAML enabled and required
Prerequisite: Set up a new private domain with domain control and require SAML. Create two new domain member emails to use for this test.
Action to Take: Sign into New Expensify with a SAML required domain controlled email account.
Action to Take: Attempt to merge a domain email account on that same domain into the SAML required account.
Action to Take: Check the “I understand this is not reversible” checkbox.
Action to Take: Tap the “Next” button
Verify: The “We’re working on it” screen appears and explains why the action cannot be taken.
Action to Take: Tap “Go to Expensify Classic”
Verify: We are brought to Expensify Classic and land in the “Settings > Account details” section.
Action to Take: Tap the “Got it” button in New Expensify
Verify: We are brought back to the “Security” screen.
Merge Failure: 2FA is enabled on the oldAccount
Prerequisite: Create two accounts. One with 2FA enabled and one without.
Action to Take: Sign into the account which does not have 2FA enabled.
Action to Take: Attempt to merge the account which does have 2FA enabled into the account that does not have 2FA enabled.
Verify: The screen to enter the magic code appears and a magic code is sent.
Action to Take: Enter the 6-digit magic code.
Verify: “Can’t merge accounts” screen appears. With a message to “disable 2FA for [email] and try again”.
Merge Failure: User is trying to merge an uncreated account
Prerequisite: Create one account with no domain control or 2FA.
Action to Take: Sign into the created validated account in NewDot.
Action to Take: Attempt to merge an account that does not yet exist into an account that does exist.
Verify: “Can’t merge accounts” screen appears. With a message: “[email] doesn’t have an Expensify account”.
Merge Failure: SmartScanner account
Prerequisite: Create one account with no domain control or 2FA.
Action to Take: Sign into the created validated account in NewDot.
Action to Take: Attempt to merge an account that is one of our SmartScan vendor domains.
Verify: “Can’t merge accounts” screen appears. With a message: “You can’t merge [email] into other accounts”.
Merge Failure: Invoiced Billing account
Prerequisite: Create one account with no domain control or 2FA. Create another account with a policy set up for invoiced billing.
Action to Take: Sign into the created validated account in NewDot.
Action to Take: Attempt to merge an account that has invoiced billing.
Verify: “Can’t merge accounts” screen appears. With a message: “You can’t merge [email] into other accounts because it’s the billing owner of an invoiced account”.
Automated Tests
We'll be creating the unit tests for the following frontend components:

AccountDetailsPage
AccountValidatePage
MergeResult Page

@allroundexperts can you provide an update here, since you're assigned?  I see you're assigned to Add Merge Account to NewExpensify too.  Mainly wanting to know what next steps are, I haven't followed the tracking issue.


Change api endpoint for listing service keys

CLI should use the endpoint /v2/service_instances/:fake-guid/service_keys
to list service keys instead of using endpoint /v2/service_keys?q=service_instance_guid:fake-guid
[#87481016]
Signed-off-by: Hua Zhang<EMAIL_ADDRESS>

@shalako @maximilien this PR is to change the api endpoint to list service keys.
@simonleung8 @jberkhahn : please help us review the code and merge it. Thanks!

@zhang-hua This new endpoint only exists in CF release 206 and up, the code path should only use this endpoint with the newer releases, else it should revert back to the old endpoint.
Here is an example of CLI checking the CC API version
https://github.com/cloudfoundry/cli/blob/master/cf/commands/service/update_service.go#L149
We are happy to take another look once this change is made.
Thanks

@simonleung8 : I have checked the API doc of CF release 206 and didn't found service key feature support. So it doesn't make sense to revert to old endpoint for service key. In addition, both of the endpoints are supported since service key features are developed. This change is just related to special use case to behave differently for SpaceAuditor and SpaceManager.  Please clarify your concern. Thanks!

@zhang-hua , you are right, thanks for pointing that out.
The PR looks good, merged.


Update requirements

Requests and Pillow aren't included in a clean install, had to install them myself on new computer

Specific Error Tracebacks:
InsecurePlatformWarning
Downloading IndicoIo-0.7.3.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 20, in
File "/private/tmp/pip-build-cM5kE3/indicoio/setup.py", line 4, in
from indicoio import VERSION
File "indicoio/init.py", line 12, in
from indicoio.text.sentiment import political, posneg, sentiment_hq
File "indicoio/text/sentiment.py", line 1, in
from indicoio.utils.api import api_handler
File "indicoio/utils/api.py", line 5, in
import json, requests
ImportError: No module named requests
Downloading IndicoIo-0.7.3.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 20, in
File "/private/tmp/pip-build-lBIWIp/indicoio/setup.py", line 4, in
from indicoio import VERSION
File "indicoio/init.py", line 16, in
from indicoio.images.fer import fer
File "indicoio/images/fer.py", line 4, in
from indicoio.utils.image import image_preprocess
File "indicoio/utils/image.py", line 7, in
from PIL import Image
ImportError: No module named PIL

Ah, figured out the problem. importing VERSION has requirements.

Fixed in #73.


cargo: Fix cargo clippy doc warnings

This'll fix the newly arrived clippy warnings and a doc warning.

Some breakage in different places ;)

one or two :) working on it...

thanks for squashing them!


QTs: differentiate between DNF and completed

When a candidate fails to hit the submit button but runs out of time (Did Not Finish), their responses so far are still valid and are still marked and scored.
We need to show in admin when a candidate has failed to complete the test, but has run out of time. This is to help teams with marking and arranging resists.

Hello @warrensearle this ticket makes a lot more sense after the 051 QT. When you get to it, please let me know and we can discuss.

Hello @warrensearle this issue came into play during the 051 QT. It helps admins to know whether all answers are completed and there is an auto-submit because canidates didn't press submit or if the test wasn't completed before an auto-submit.

Hello @lloback am I right in thinking that this work was not completed in QT pt II and is now to be swept up in QT pt III? I can see the ticket allocated to me in the 2021 and 2022 pipeline.


Getting rid of scanpy.api

This distributes the reexports in scanpy/api/ in two modules: the core functionality goes into the root (scanpy/__init__.py) and the external functionality goes into scanpy/ext/. The structure of the latter mirrors the one of scanpy.api so that users simply need to adjust their imports
import scanpy as sc
import scanpy.ext as sce

instead of previously
import scanpy.api as sc

Backward compatibility for use of scanpy.api is fully maintained, but from now on, its use and development are deprecated.

@fidelram FYI, this defines an explicit plotting submodule in plotting/__init__.py, which also contains the docs for it. There is no need to reexport to api/pl.py anymore. Everything is backwards compat and if you want to use new functionality, simply import scanpy instead of import scanpy.api...

can you give me access to https://icb-scanpy.readthedocs-hosted.com?
I just have access to the public readthedocs.org

I sent you an invitation for readthedocs.com about 2 months ago already - I just resent it. :)

I've tested this quite a bit in the past 5 days and am merging it into master. In essence, it's a superficial change.

I sent you an invitation for readthedocs.com about 2 months ago already - I just resent it. :)

Well, doesn’t seem like it worked in the past: What I got now was not an invitation that I needed to click, but simply a notification that I’m now member of the team on rtd.com (which I wasn’t before)
The changes look good! I would however prefer to do things via .. include:: instead of duplicating code for the scanpy and scanpy.api sections. Except if you plan to not update the scanpy.api module and docs section.

Except if you plan to not update the scanpy.api module and docs section.

Yes, that's the plan. scanpy.api is completely phased out an simply there for backwards compatibility.


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2298

Questions ?
Please refer to the Backport tool documentation


how to read and print @rid ?

hello
i want to print @rid after inserting to database
i have tried the following
$user["@rid"] // didn’t work
$user->@rid // didn’t work

is there a work around?

Hello @mshanak, echo $user["@rid"]; is working perfectly for me.

I can not print out @rid too
I'm using laravel 5.2.
Please help me fix it.


LHCb video content as well?

we decided to do a video with a backdrop not to go to LHCb, so we need to find the backdrop


Update resources in index.yml

Change SO to Q&A
Change UV to TC Ideas forum

Docs Build status updates of commit 6e1747a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/index.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit bcf4a8f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch lindalu-update-resources has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


READY : Implementing & covering option streamSizeLimit.

Implemented option streamSizeLimit.
streamSizeLimit : null, /* qqq for Yevhen : implement and cover the option. look option maxBuffer of spawn */
1.1 Added asserts for the wrong usage of streamSizeLimit.
Wrote test routine startOptionStreamSizeLimit.
Wrote test routine startOptionStreamSizeLimitThrowing.
Fixed according to the review.

no loossy conditions


Regex Timeout are too short, not consistent, and not configurable in RewriteMiddleware

Regex timeouts that are used across UrlRewrite, ApacheModRewrite, and code rules are not consistent. The are set at 1 second or 1 millisecond in different places. Secondly, 1 millisecond is too short for a regex expression; it should be 1 second as the timeout is mostly for making sure your server isn't locked. Also if the expression timeouts, it returns a 500 server error as the exception is unhandled. Thirdly, the timeout should be configurable in RewriteOptions.
Plan of action:

Timeouts should be increased to 1 second across the board.
For 2.1, we should make Regex timeouts configurable a
The timeouts should be backported to 2.0 and 1.1

@Eilon @muratg this is a patch candidate. Workarounds are very difficult.

Need to consult with @blowdart on a good timeout. I agree 1ms is waaaay too small.

1.1 milliseconds.
Merry Christmas ⛄
Oh ok, 1 second is fine as a default, as long as it's configurable.

@blowdart but in a patch, we might not make it configurable. Would 1sec be reasonable in the patch? (With a switch to restore the original bizarre 1ms timeout.)

The patch is a simple fix. However the redesign would require more work. When calling options.AddRedirect(), options.AddIISUrlRewrite, etc, we eagerly create the Regex object, which is too early as options would now store the regex timeout. Someone could do:
options.AddRedirect(...); // Creates a regex object with default timeout of 1 ms
options.RegexTimeout = TimeSpan.FromSeconds(1);
options.AddRewrite(...);

which would set the timeout of 1 ms instead of 1 second for the regex expression. We shouldn't have options apis that need to be ordered.
We could either create the regex objects on first request or use a different pattern to know when rules are done being added.

Config would be better, but if we can't do that for a patch, then, ugh, ok fix it at 1 second.

options.RegexTimeout = TimeSpan.FromSeconds(1);  looks like a simple, piratical solution. Changing it to DefaultRegexTimeout might make it's usage a little more clear. The ordering restriction is not a hard thing to explain to users.
Passing in the timeout to every .Add API looks like overkill, you'll add a lot of overloads that very few people will use, and they'll have to specify it on every rule.

Adding timeouts through each of the .AddXyz() calls would be bloated for sure. I guess we can add it to options and throw if someone does:
options.AddRule();
options.RegexTimeout = TimeSpan.FromSeconds(1); // throw

Or don't throw and make it work: have every call to .AddXyz() capture the "current" RegexTimeout and use it.


initialize column in create_column

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Initialize the column in ColumnHelper::create_column.

Could you add more background about this issue? What is the problem you are going to resolve via this issue.

@Mergify rebase

@Mergifyio rebase

@Mergifyio rebase

run starrocks_be_unittest


Support a component prop (or as prop) for ArchetypeEntities

It would be nice to have a prop to render directly a component for each entity, instead of using the children:
<ArchetypeEntities archetype="enemy" as={Enemy} />

Instead of:
<ArchetypeEntities archetype="enemy">
  <Enemy />
</ArchetypeEntities>

This is going to be in 2.0 as discussed, so I'm closing this issue.


[SPARK-24960][K8S] explicitly expose ports on driver container

https://issues.apache.org/jira/browse/SPARK-24960
What changes were proposed in this pull request?
Expose ports explicitly in the driver container. The driver Service created expects to reach the driver Pod at specific ports which before this change, were not explicitly exposed and would likely cause connection issues (see https://github.com/apache-spark-on-k8s/spark/issues/617).
This is a port of the original PR created in the now-deprecated Kubernetes fork: https://github.com/apache-spark-on-k8s/spark/pull/618
How was this patch tested?
Failure in https://github.com/apache-spark-on-k8s/spark/issues/617 reproduced on Kubernetes 1.6.x and 1.8.x. Built the driver image with this patch and observed fixed https://github.com/apache-spark-on-k8s/spark/issues/617 on Kubernetes 1.6.x.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@felixcheung Done I think, is that OK?

@mccheah Done https://issues.apache.org/jira/browse/SPARK-24960
Re: Exposing ports, my understanding of Kubernetes' networking model is it is possible for a cluster to be setup such that Pod ports are closed, perhaps as a security measure. At least in our Kubernetes cluster (tested both on 1.6.x and 1.8.x) the SparkPi example failed with a connection error without this change, and succeeded after. (I've also added this information to the JIRA)

At least in our Kubernetes cluster (tested both on 1.6.x and 1.8.x) the SparkPi example failed with a connection error without this change, and succeeded after. (I've also added this information to the JIRA)

@adelbertc Just to clarify, by "succeeded later", do you mean after this change was applied? Or do you mean "succeeded later" as in a retry worked with the same code?

@mccheah After the change was applied

Can you quickly change [Kubernetes] to [K8S] in the PR title? Will merge after that is done.

Actually can you also update the unit test? Thanks!

@mccheah Done and done

ok to test

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/1565/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/1565/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1565/
Test PASSed.

jenkins, test this please

(Not sure why the unit test build didn't run just now, only the integration tests did)

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/1575/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/1575/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1575/
Test PASSed.

@shaneknapp the unit test build doesn't seem to be triggering on this PR, any idea as to what's going on?

@mccheah nope no idea why the unit tests didn't trigger.  the build config hasn't changed, and the worker that this build ran on is one of the original ubuntu workers (not freshly deployed), so i'm unsure.
i'll try and take a closer look later today.  got a bunch of other stuff on my plate.
maybe @ssuchter can help?

jenkins test this please

Test build #93901 has started for PR 21884 at commit 4215fc2.

@mccheah the tests didn't run because you're not an admin for the regular (non-k8s) pull request builder.
i added you to that list and you'll be able to trigger tests now.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/1581/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/1581/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1581/
Test PASSed.

Test build #93901 has finished for PR 21884 at commit 4215fc2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93901/
Test PASSed.

Thanks @shaneknapp, I'll merge this now.

Thanks @adelbertc for the contribution!

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/1583/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/1583/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1583/
Test PASSed.


SPFX app calling AadHttpClient returns idToken is null or empty

starting last week, we are getting an error similar to this bug
https://github.com/SharePoint/sp-dev-docs/issues/4840
we have a SPFX app that calls API which is secured with AD authentication. The app has been working for months. There were no changes to the code. This errors just suddenly happens on both production and development sites on the same tenant.
The app calls the API in two places, one on the entry screen and one on the detail screen. It seems that it works only the first call, then on the second call it returns “ClientAuthError: The idToken is null or empty. Please review the trace to determine the root cause. Raw ID Token Value: null”.
Here is the code
On the webpart file
public onInit(): Promise {
return super.onInit().then((_) => {
let appID =
  this.context.aadHttpClientFactory
    .getClient(appID)
    .then((client: AadHttpClient): void => {
      this.apiClient = client;
    });
  sp.setup({
    spfxContext: this.context,
  });
});

}
Then the apiClient is passed down via props to the components and calls this way
this.props.apiClient
.fetch(“api endpoint”, AadHttpClient.configurations.v1, {
method: "GET",
headers: {
Accept: "application/json",
"Content-Type": "application/json",
},
})
.then((response) => response.json())
.then((data) => {}
.catch((error) => {
alert("Error loading data: " + error);
});
The SPFX version used is 1.9.1
Does anyone know what happening here?
thank you !

I tried to upgrade the framework to 1.11 as instructed here https://docs.microsoft.com/en-us/sharepoint/dev/spfx/release-1.11.0
still getting the same error

@manlamcheng said:

let appID =
  this.context.aadHttpClientFactory
    .getClient(appID)
    .then((client: AadHttpClient): void => {
      this.apiClient = client;
    });
  sp.setup({
    spfxContext: this.context,
  });
});

What's `appID`? Is that a real GUID or endpoint? If so, that's not supported. At some point in the previews, you could use a GUID, but that wasn't the documented way of doing it once it went GA. While it worked for a while, that's not what was supported so if it worked until just recently, very well could be that support was dropped. See this for a reference: https://docs.microsoft.com/en-us/sharepoint/dev/spfx/use-aadhttpclient#connect-to-azure-ad-applications-using-the-aadhttpclient

BTW, didn't notice this before, but in the future, please don't clear out the issue template. It includes important context information we need to help. In the future, new issues will be automatically rejected if they omit this information.

Hello Andrew, thanks for the reply,
sorry, I did see the template that was used on other issues, but when I open one by clicking on the "..." on other issue and chose "Reference in new issue", I didn't see the template, just a blank text box. I will make sure to include the template next time
as for the appID, it's the Application ID of the app that's registered in Azure AD. It's used in the Azure function app for authentication

@manlamcheng said:

as for the appID, it's the Application ID of the app that's registered in Azure AD. It's used in the Azure function app for authentication

As I said in my reply, that's not supported and won't always work. In your case, you should use the URL of your Azure Function App as the referenced doc I provided shows.

yes, I just tested with the url instead of the appID as indicated on the link you provided, but still getting the same error
.getClient("https://{appname}.azurewebsites.net")

The AadHttpClient object is created on the web part file and passed to main component
main component then passed to 2 other components (A, B) via props
if A open first, it will work, then from A switch to B, it gives error on B
if B open first, then it works.
is this approach supported? or should I create a separate AadHttpClient object on each component ?

still struggling here
I tried to create a service class for aadHttpClient service, and imported to components. still facing the same problem "The idToken is null or empty."
also tried to turn off the AD authentications on the function app, no help
copy and paste the function url into browser works
I am running out of options, please please let me know if you have any hints/directions that I can follow
thank you very much

@manlamcheng said:

still facing the same problem "The idToken is null or empty."

That sounds more like an issue with your custom AAD app than your code or the config. There's a setting within the AAD app where you can select if an ID token is returned.

thanks,
I think I may have a fix, it's related to the session being cleared when switching components, however, it's not an issue for past months, perhaps there was a recent change with SharePoint online that affected how session is handled

There's active work going on to migrate the internals of how tokens are obtained, but the current state of that work isn't well known. They are moving from ADAL.js => MSAL.js v1 & also working to fix numerous known issues with the process. That may be an impact to what you're seeing, but maybe not... can't tell.


Merge ReturnMacros.h into CodeUtils.h

Problem
The ‘Exit’ family of error handling macros are in support/CodeUtils.h
but the ‘Return’ macros are in a separate file support/ReturnMacros.h,
which may discourage use.
Summary of Changes

Move the ‘Return’ macros to support/CodeUtils.h, above the ‘Exit’ ones.
Remove support/ReturnMacros.h.

@kpschoedel - could we move to something named differently than utils? CodeUtils is super vague - when do I include them? I am coding, so do I need "Utils"? ReturnMacros is a much better name for what it contained.

@kpschoedel - could we move to something named differently than utils? CodeUtils is super vague - when do I include them? I am coding, so do I need "Utils"? ReturnMacros is a much better name for what it contained.

The concern is having the return macros and the goto-exit macros in different places, with the exit ones being available ‘free’ because so many things already include CodeUtils. We could move both to a new file.


[Feat]: Add Enum field support

What new feature are you looking for / expect?
We only support Optional but other Enums would be great.

#[derive(GeekValue)
enum UserType {
    Admin,
    User,
    Guest
}

#[derive(GeekTable)]
struct Users {
    // ...

    usertype: UserType
}

Expectations
Nice to have

GeekValue or now called Data is supported now


Release 0.5.1

<!-- Make sure you have read http://napalm.readthedocs.io/en/latest/contributing/index.html --!>

Coverage remained the same at 90.726% when pulling 1a700410f7741f29ef52a8b5a9fae989c2d21daf on develop into 4423b1f40b3646a5a1d3a905b891619ce96e9c22 on master.


added note about acceptedResourceRoles limitation

This is pretty minimal, so if there's more you'd add or an example, please let me know.

@sascala Thanks. This updates the deprecated api documentation. Would be great, if we can also add this to our knowledge base?


Hallowed Sepulcre items not equipable

Hello guys,
I tried equipping the Hallowed Grapple and Hallowed ring in my 'skilling' setup but was presented with the error:
Command used: +equip skilling hallowed ring
Return: You don't have enough of this item to equip, or its not equippable.
Command used: +equip skilling Hallowed grapple
Return: You don't have enough of this item to equip, or its not equippable.
I managed to equip the Hallowed symbol and Halloed Focus with no issues.
P.S great with with the bot as a whole, I see so many people and servers using it, it's great!
Take care
James

resolved


*: Update to Prometheus v0.10.0

We just pushed a new release candidate for the Prometheus crate: https://github.com/tikv/rust-prometheus/pull/340
Opening up this pull request to test the release candidate for now. Once it is promoted to a release we can proceed with this pull request.

Nothing weird on Grafana.

Let's not merge yet as it is still set to the release candidate. I will promote the release candidate to a release soon (https://github.com/tikv/rust-prometheus/pull/341).

bot merge

The PR is currently unmergeable.

Guessing it is missing another core-dev approval. @tomaka would you once more do the favor?

bot merge


C#: Add "Use actual type" context action for generically-typed Node::GetNode calls to resolve broken references

@van800 Thank you for requesting this on Discord.
As a long time developer but relatively new to Godot, this would have saved me hours of pain.
Pseudocode example:
Before:
Player.cs (Attached via Godot editor to a KinematicBody2D Node named 'Player'):
public class Player : KinematicBody2D
{
  ...

  // My custom method
  public void Attack (Enemy enemy)
  {
    // Implement my uber-cool attack!
  }
}

Combat.cs (Attached via Godot editor to some sibling node):
...

// In some collision method:

var player = GetNode <KinematicBody2D> ("../Player"); // Incorrect: Obtain reference to
                                                      // Player class Node instance
                                                      // which extends KinematicBody2D

player.Attack (enemy); // Error: Cannot resolve symbol 'Attack'
                       // Huh? How do I fix this?
                       // Everything "just works" in GDScript... >:(
...

In the above example, the type KinematicBody2D should be hint-underlined, whereby ALT + Enter should show a context action along the lines of Use actual type, in order to fix broken references to fields and methods after a mis-typed call to Node::GetNode.
Additionally, if practical, add a context action to the broken method call itself, such as Fix reference by using actual type.
After:
Combat.cs (Attached via Godot editor to some sibling node):
...

// In some collision method:

var player = GetNode <Player> ("../Player"); // Correct: Obtain reference to
                                             // Player class Node instance
                                             // which extends KinematicBody2D

player.Attack (enemy); // Error resolved; FINISH HIM!!!

...

Probably related implementation-wise to #4 & #39.


Add notice to docs for migrating from more recent versions of Gogs

This PR adds a reference to #4286 to the migration docs.
(We should extract the info from the issue and put them directly in the docs, but just putting a reference to the issue is fine for now - helps finding it quickly)

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@09b281a). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #8724   +/-   ##
=========================================
  Coverage          ?   41.38%
=========================================
  Files             ?      537
  Lines             ?    69239
  Branches          ?        0
=========================================
  Hits              ?    28652
  Misses            ?    36913
  Partials          ?     3674

Continue to review full report at Codecov.

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


TOC and shortcode

I noticed an unpleasant feature related to shortcode and TOC
If the chapter ends up in shortcode, like this:
{{< short >}}
## Test
{{</ short>}}

Then the header is missing in the TOC. This gives me a lot of hassle, had to use 2 shortcodes to get around this. But this is very inconvenient to maintain.
{{< short >}}
## Test
{{< short-end >}}

It is very hidden in the documentation (so that I didn't find it) but to have your headers from shortcode in the TOC you need to load the shortcodes via {{% shortcode %}}.

https://gohugo.io/content-management/shortcodes/#shortcodes-with-markdown

Shortcodes using the % as the outer-most delimiter will now be fully rendered when sent to the content renderer (e.g. Blackfriday for Markdown), meaning they can be part of the generated table of contents, footnotes, etc.

@davidsneighbour Unfortunately it didn't work for me. Perhaps there is something else to be done?

From my previous comment:

Shortcodes using the % as the outer-most delimiter will now be fully rendered

Your shortcode template need only contain {{ .Inner }}.
Please do not use GitHub to ask questions; that's what the forum is for:
https://discourse.gohugo.io/


Next.js Build Fails with Unhandled node:crypto Module Error

Description
Build fails when running bun run build (Next.js 15.0.3) due to an unhandled node:crypto module import. The error occurs when webpack attempts to process imports from Gleam crypto modules.
Error Message
Module build failed: UnhandledSchemeError: Reading from "node:crypto" is not handled by plugins (Unhandled scheme).
Webpack supports "data:" and "file:" URIs by default.
You may need an additional plugin to handle "node:" URIs.

Import Trace
node:crypto
../core/build/dev/javascript/gleam_crypto/gleam_crypto_ffi.mjs
../core/build/dev/javascript/gleam_crypto/gleam/crypto.mjs

Environment

Next.js: 15.0.3
Build Tool: Bun
Project: Using Gleam crypto modules

Possible Solution
It seems that just removing the node: from the protocol imports would fix it. This would allow the package to run against multiple JS engines.

Hello! You need to add a handler for the node protocol, as the error message says

If anyone else also hits the same issue, this fixed it for me


Is linuxPackages.perf dependable on non-NixOS hosts?

Question: Is it safe to use the package linuxPackages.perf in Nix expressions that will run on non-NixOS Linux hosts (for example Ubuntu or RHEL)?
My concern is that perf may interface with the kernel using an unstable private interface and be sensitive to the exact running kernel version i.e. be an impure dependency that will be prone to breakage.
Great if that is not a problem in practice but I don't have the platforms available to test with so I am looking for crowdsourced wisdom...

It should be safe, at least in theory. There aren't (or at least shouldn't be) any userspace-kernel interfaces that aren't stable. Or at least Linus shouts at people who try to break any interface. :)

@dezgeg On the one hand that's true, on the other hand I think that most distros distribute a separate version of perf userspace for each supported kernel? (and that it doesn't work unless you use the right one?) Seems like that has been my experience although I don't know how to reconcile that with well-known kernel policies.

(Closing to avoid clutter but don't know the answer yet. Possibly have to try running it in a bunch of VMs of different distros.)


[RFC] Discuss New Features of AOT Runtime

As we move to the NNVMv2(relay), there is a clear separation of compiler and runtime. The compiler IR is maximumly flexible, while the runtime is a virtual machine(interpreter) that the executes the code which compiler generates.
This is an RFC to discuss what kinds of runtimes do we have. Given the separation of compiler and runtime, it might make sense to have multiple runtimes since there is a tradeoff on how rich the feature we want to support(e.g. JIT) vs the minimalism we need on an embedded device. Likely we will need an AOT runtime and JIT one.
There are also questions on what data structures we want to expose in the runtime. Likely the TVM runtime and PackedFunc is already our friend, but we may need a few more things to accommodate control-flow and other applications

One thing that would be useful (that I think is orthogonal to JIT vs AOT) is considering relaxing the requirement on fully-specified shapes in graph_runtime. That is, we'd like to allow variable dimensions (consider supporting e.g. variable batch size) and allocate (at least some) memory dynamically.

@ajtulloch can you give some specific application examples. For example, dynamic batch size in image modeling, dynamic length in language models etc. We do support certain forms of type inference with dynamic variables (the current type inference support symbolic integer in batch), and likely that would help in certain cases.

@tqchen  yes, a simple case would be supporting using a graph in graph_runtime with requiring fully static shapes. For example, say we have a ResNet trained on ImageNet-1k classes, we'd like to be able to pass an (N, 3, 224, 224) tensor and recieve an (N, 1000) tensor, where N can change between calls. This shouldn't require JIT, we can just partially shape-specialize when compiling.

I agree with @ajtulloch.
More dynamic is necessary for future application. We should list a few tasks for each field, including CV/NLP/RL/ASR/TTS etc, check one by one to see what kind of feature will be necessary and what is the trend. In this phrase domain experts' input will be very helpful.

For embedded applications it is often important not to use dynamic memory. For example, some tiny systems may miss memory manager at all. In my opinion, user should always have an option to generate static-memory code or the code where upper bound for memory usage is well-defined.
@ajtulloch  Is there a possibility to relax static shape restrictions and still use static memory? For example, could we generate static memory code, which would allow users to vary one dimension of one input tensor?

@grwlf I don't envision this would require dynamic memory allocations, more that it would make it possible. If you know all shapes statically then of course you can just statically allocate. This is more enabling new cases where you don't know all dimensions for all tensors statically.

Is training in the scope of discussion?

Another thing that could be useful in runtime is to support graph partition in case certain operators are not supported by accelerators or resources at runtime doesn't permit.

I would list three most demanding features of our runtime: 1) handling inputs in dynamic shape, 2) handling execution on multiple compute resources (CPU, GPU, DSP, NPU, etc.), and 3) enabling operator-level parallelism.
I think having multiple runtimes makes sense as the scenarios of server and edge are vastly different. On the server it is relatively flexible due to the sufficient compute resource and less constraint of power consumption. For example, we can use JIT to handle dynamic shapes, execute sample runs to determine the resource availability beforehand.
The critical part is how to design runtime in the constraints of edge devices. I think a minimal static interpreter @ajtulloch suggested makes sense. However, an edge device may not have much space to store many pre-compiled object files.

Several concerns:

Minimizing data movement by laziness. In a word, just dump data anywhere in global memory is fine, do not waste time moving them around before they are actually needed. For example, in concat, can we avoid memory allocation, kernel launching, etc, which just move data to somewhere, even if there is no actual computation? We could maintain discontiguous memory chunks lazily, and only indexing them when real computation happens. Although memory access cost in this kernel is possibly increased, it does eliminate overall memory accesses. I guess this could be implemented by an extra pass.
Dynamic batching. If 1) is enabled, dynamic batching could be realized very easily. I do believe dynamic batching will be a nice feature so that NLPers could focus on the model itself rather than batching data, while get performance gain automatically. I think we have some usecases in industry  as well @yidawang .
Fine-grained streaming. It is possible to stream data on CUDA GPU, but kind of complicated. We do hear about usecases when doing heterogeneous FPGA-CUDA acceleration, streaming row by row from GPU to FPGA helps a lot in reducing latency to nearly zero.
The ability to get rid of a runtime. Having a heavy runtime is repeating the failure of TensorFlow. What's more, any extra dependency means extra plain on edge devices. I think it could be great to dump an AOT version which allows us to call a subset of computation (which contains no control flow, only dataflow), and let users themselves assemble customized version containing branches, loops etc.

I agree with @szha that we should have a scope of discussion. In this thread, should we talk about future integration into upstream deep learning framework (e.g. MXNet), should we talk about inference v.s. training, should we consider distributed stuff, and should we consider thread safety? These are all concerns from different scopes.

To limit the scope of the discussion, let us first focus on the low resource and pure AOT scenario. The multiple target execution is already supported in the current graph runtime as #1695, and we only need to build compiler support for that. My guess is that jit and training is its own beast and would deserve another thread.

#1695 is great but doesn't cover all I meant by "handling execution on multiple compute resources". Given multiple targets, how to schedule the execution on them in parallel would be an interesting research/engineering topic to explore. Also, TVM runtime may call a third-party runtime (e.g. TensorRT) for a particular target in the heterogenous environment.

@grwlf This sounds pretty cool! So how could we do manual (or automatic) scheduling on a automatically generated backprop?
CC: @were seems to be interested in this as well.

@grwlf I don't envision this would require dynamic memory allocations, more that it would make it possible. If you know all shapes statically then of course you can just statically allocate.

@ajtulloch I meant the case where we have to allocate memory statically, but still want to vary one of input dimensions. If I am correct, in dynamic batching, as mentioned by @junrushao1994, we typically vary a batch size dimension. Having this feature implemented, TVM may attempt to enter the domain for resource-critical embedded applications.

+1 to assigning graph partitions to threads instead of axes. There are some convincing benchmarks and discussions which suggest that cache locality is a primary performance booster.

+1 to assigning graph partitions to threads instead of axes. There are some convincing benchmarks and discussions which suggest that cache locality is a primary performance booster.

Normally if workload is big enough, our past experience in MXNet suggest that parallelization within op has better potential than bind parts to threads. The pipeline partition would be useful though on small ops

In inference, graph-level parallelism does not help that much (at least on CPU/GPU), because normally an operator has been big enough to occupy all CPU threads or GPU streaming processors. About small workloads, @tqchen @nhynes do you have any specific example of such workloads, i.e. that could be accelerated by multiple issuing, but couldn't be fused into a single kernel?

Glow is a framework that does not support multi-threading (at least at the time of writing this post), i.e. all their operators are single-threaded. This could somehow explain why multiple issuing helps in Glow (imho).

Glow [...] does not support multi-threading

Right, but that's what they're planning on doing now. They're going the graph partitioning route and have expressed that cache locality makes this approach more efficient than op-parallelism.
I can't quite think of an example of a "small" op in the world of fusion, though.

Right, but that's what they're planning on doing now. They're going the graph partitioning route and have expressed that cache locality makes this approach more efficient than op-parallelism.

Looks very interesting. @were do you have some bandwidth looking into this?

related #2810


Location of lya_forest mocks has changed

I'm trying to run the minitest notebook using (mostly) release 18.3, and the select_mock_targets stage failed.  Digging into the log files, I found this:
./53/5352/build-64-5352.log:WARNING:mockmaker.py:1088:readmock: Mock file /project/projectdirs/desi/mocks/lya_forest/v2.0.2/master.fits not found!
Apparently the arrangement of the lya_forest directory was changed at some point, and not in a backward-compatible way.
I'm using the version of desitarget from release 18.3, 0.20.1.

It looks like a bunch of subdirectories (develop, london, saclay, uncorr) were added by @andreufont the past couple days.

@julienguy, thank you!

This should be a more stable structure, at least for a while. Sorry for
lack of communication, I wasn't aware that this would break any test.
Cheers,
Andreu
On 16 May 2018 at 20:59, Benjamin Alan Weaver<EMAIL_ADDRESS>wrote:

@julienguy https://github.com/julienguy, thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/desihub/desitest/issues/4#issuecomment-389629381, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ADscR_5E9snkBsKfCC6nPGsfwhKJASBWks5tzHchgaJpZM4UB02P
.

OK, let's keep this open until we've confirmed that the test notebook(s) can work with the new layout.

For the record, I was able to complete a test with the symlinks  installed, but I don't think anyone has run the test(s) using the new layout explicitly.

@sbailey agrees it is sufficient that the test notebook passes with just the symlink.


MM-22845 Fix quick switcher results

Summary

Change existing searchprofiles selector to be searchProfilesStartingWithTerm
Add a new selector searchProfileMatchingWithTerm for substring matches

Ticket Link
https://mattermost.atlassian.net/browse/MM-22854

Cherry pick is scheduled.


[Event Request] Table 2158 "O365 Document Sent History" - Event OnBeforeNewInProgressFromRecRef

Hello,
Can we please get "O365 Document Sent History" as reference in this event "OnBeforeNewInProgressFromRecRef"?
procedure NewInProgressFromRecRef(RecRef: RecordRef) Result: Boolean
var
SalesHeader: Record "Sales Header";
SalesInvoiceHeader: Record "Sales Invoice Header";
IsHandled: Boolean;
begin
IsHandled := false;
OnBeforeNewInProgressFromRecRef(Rec, RecRef, Result, IsHandled);
if IsHandled then
exit(Result);
case RecRef.Number of
DATABASE::"Sales Header":
begin
RecRef.SetTable(SalesHeader);
exit(NewInProgressFromSalesHeader(SalesHeader));
end;
DATABASE::"Sales Invoice Header":
begin
RecRef.SetTable(SalesInvoiceHeader);
exit(NewInProgressFromSalesInvoiceHeader(SalesInvoiceHeader));
end;
else begin
Session.LogMessage('000028D', StrSubstNo(UnrecognizedParentRecordErr, RecRef.Number), Verbosity::Error, DataClassification::SystemMetadata, TelemetryScope::ExtensionPublisher, 'Category', DocSentHistoryCategoryTxt);
exit(false);
end;
end;
end;
[IntegrationEvent(false, false)]
local procedure OnBeforeNewInProgressFromRecRef(var O365DocumentSentHistory: Record "O365 Document Sent History";  RecRef: RecordRef; var Result: Boolean; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[GEOS] Update GEOS-Dycore

We need to keep the @FVDycoreGridComponent and @fvdycore up-to-date with GEOS mainline.
For each new commit we need:

Merge in dsl/develop
Create tasks associated with any code changes
Create a new dataset for translate test if necessary

We will make task per update


Update swap towards latest exchanger version.

Improved the readme. Bumped the exchanger dependency version.

Thanks for you PR @ddinchev :)
Would be fantastic if you could update the Fixer usages in the documentation too
https://github.com/florianv/swap/blob/master/doc/readme.md
Exchanger 1.1.0 was just released so it should work straight once the build runs again!

Also waiting for this PR to be merged.

@ddinchev Can you please update this PR to use exchanger 1.1 ?

@florianv it already is updated but exchanger:1.1 wasn't released yet. I can push a dummy change to trigger the rebuild if you can't do that manually?

@ddinchev I restarted the build but the tests are failing!

@florianv all green now ;)

Thanks @ddinchev


types (value & reference)

Might be worth it to dedicate a section to this information, as it is basic and should be explained properly.
This is especially important for new people to C# (or programming in general) as they might not understand what an int is for example.

I might start working on this as soon as Monday approaches


WIP: TS Typings

Closes #4

Hey @giulioz, would you mind adding the eslint config from drei and setting that up? 🙏🏼

@joshuaellis I just made an attempt at importing the drei eslint config, I'll make a PR into this branch shortly :smile:

Seeing that FirstPersonControls.js hasn't gotten an update yet, cool if I try converting it?

@DylanTackoor yeah go ahead if you want :)

is this ready to merge? controls currently have side effects in the constructor which is a no go, they should add event listeners in a manual connect function. see https://github.com/mrdoob/three.js/pull/21224 should i make this change now to the js files?

@giulioz do we merge little by little or as a whole? there are some changes still made to the js files, does that complicate things for you? controls currently have side effects in the constructor which is a no go, they should add event listeners in a manual connect function. see mrdoob/three.js#21224 should i make this change now to the js files or in here?

Yeah this could be a problem, we already have 6 conflicting files, maybe it's better to merge ASAP before continuing

I think that'd be a good idea, I can look post work at resolving merge conflicts unless you get it to it before me


Add content

yea I think it's gud enough for now. I can always add more content later. I'll address the lag in the other branch


Go to next/previous sibling commands do not work in zoomed edit mode

When one zooms into a block hierarchy with the built-in Zoom in editing block command (or clicking a block's bullet), the Vim Shortcuts plugin commands for moving to siblings do not seem to function.
How to Reproduce
It's not convenient for me to make a recording at the moment, so I'll describe reproduction this way:

Create a hierarchy of blocks on a page like the following:
- Parent
    - Child A
        - Grandchild A.1
    - Child B
        - Grandchild B.1

Zoom in to Parent, so it is the top level viewed.

Navigate to Child A in Normal mode

Try to move to Child B with Shift-J

This does not work for me, it works fine at the level of the page containing Parent but not in zoom.
Thanks
Thanks very much for your Vim plugins! They are enhancing my experience with Logseq greatly, I'll be sending a coffee your way ☺

I have done some fixes. but not perfect, Logseq API can not keep you stay on zoom mode, it will be redirected to the page and go to sibling blocks. I have contacted the Logseq dev team to see if they have some solutions. But for now, at least it works more or less.

My last comment issue has been fixed also.

I'm curious whether this issue has been fixed. It seems like the moving to sibling functionality is still not working. Thank you very much for this plugin.


Milestone 7 feedback

Team K4
Tyler:
I'm really not sure why you are using hexagons. Is that the actual size and shape of surge zones? It looks cool but seems arbitrary. Also, It would be nice if I could just click on a hexagon 'price zone' and it would automatically direct me to the very edge of that zone so that I don't have to walk as far or be as precise in my click/tap. Not a big change but would be nice to see. Also the point count is an unnecessary abstraction. I'd rather see actual dollar fare estimates.
Drew:
I really like dragging routes but its not very clear how to remove pins on the map. To finish the features of the app, it would be nice to have pins snap to the route. The next feature I want to see is an automatic total of points used by taking the current route.
Cynthia:
You should do the following to complete core map/geo:
Change km and k to probably feet/miles since those are the units we use in the US. For the last functionality, I think you should include the estimated fare pricing for the region. I'm also not too sure what "point count" means in the legend.
Andrew:
The app looks good, good choice of color on the location pins are easy to use. You can take the information elements on the map and move them off the map to the side, but that is just a personal preference.  I think you should integrate the Uber and Lyft functionality, when showing a route and pricing for it, I would just have both showing at the same time.

TEAM JOANNA (K3)
Anish:
You should a walking route from the user’s current location to the location with the cheaper prices. That would provide a sense of direction as well as let the user see if walking the distance is even worth the price difference in Ubers.
Ashkon:
You should add some sort of display for a calculation on the most effective route. It should display the cost for one of the ride sharing services instead of just the directions between two locations.
Joseph:
You should consider finding another source of data to give fair estimates for rides. Hearing from the group that there is a long process to use Uber’s API, maybe there is another option to base the use as price calculations. For instance, I think it would be cool to see a shortest route estimate based on the nearby locations you can walk to. A shorter drive can save users money on these apps.
Nic: You should add the ability to enter in your final driving destination, so that the app can suggest pickup locations that are along the way and cheap. For example, if I’m ubering all the way to Solana Beach (north of UCSD), it would be better to walk to a cheaper spot that is north of where I currently am. Otherwise, the map interactions look very slick and I think this wille be amazing if you can get real uber price data.


[ISSUE] Unrecognized configuration parameter after chart upgrade

Upgrade from v0.8.2 -> v0.9.0 fails due to new major version of Postgresql does not recognize wal_keep_segments parameter in /var/lib/postgresql/data/postgresql.conf.
To Reproduce
Deploy chart version v0.8.2 and upgrade to v0.9.0.
Expected behavior
Postgresql major version should update from 12 to 13 and wal_keep_segments should be replaced with wal_keep_size.
Deployment
values.yaml:
secretNames:
  credentials: "timescaledb-creds"
  certificate: "timescaledb-cert"
backup.enabled: false
patroni.log.level: INFO
postInit:
  - configMap:
      name: db-init-script
      optional: false
service.primary:
    type: ClusterIP
    port: 5432
loadBalancer.enabled: false
persistentVolumes.data.size: 8Gi
prometheus.enabled: true
affinity:
  podAntiAffinity:
    requiredDuringSchedulingIgnoredDuringExecution:
    - topologyKey: "failure-domain.beta.kubernetes.io/zone"
      labelSelector:
        matchLabels:
          app: "timescale-timescaledb"
          release: "timescale"
          cluster-name: "timescale"

Kubernetes environment: EKS
Deployment
helm list:
NAME     	NAMESPACE  	REVISION	UPDATED                                	STATUS	CHART                    	APP VERSION
timescale	visioncraft	12      	2022-01-27 18:31:34.146896014 +0100 CET	failed	timescaledb-single-0.9.0

Logs
Logs from last pod from stateful set:
2022-01-27 17:51:35.848 GMT [1359] LOG: unrecognized configuration parameter "wal_keep_segments" in file "/var/lib/postgresql/data/postgresql.conf" line 42
2022-01-27 17:51:35.848 GMT [1359] FATAL: configuration file "/var/lib/postgresql/data/postgresql.conf" contains errors
2022-01-27 17:51:35,860 INFO: postmaster pid=1359
/var/run/postgresql:5432 - no response
2022-01-27 17:51:35,870 INFO: Lock owner: timescale-timescaledb-1; I am timescale-timescaledb-2
2022-01-27 17:51:35,871 INFO: failed to start postgres

Which causes: Readiness probe failed: /var/run/postgresql:5432 - no response
Additional context
Running on EKS with 3 timescale pods. Only the last one fails with an error.
As a workaround version: 12 can be set so that Postgresql 12 will be used instead.
I tried to edit /var/lib/postgresql/postgresql.conf manually, but when pod is recreated this file reverts back to the previous state with an unrecognized option.
For now, we upgraded to v0.10.0 with version: 12 (Postgresql major version 12).

Same issue here!


Pyftmerge crash on large GSUB Tables

A crash in pyftmerge ... I think it is an overstuffed GSUB table that is causing this crash in pyftmerge ... yes?
Traceback (most recent call last):
  File "C:\App\Python37\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 271, in getData
    items[i] = packUShort(item.pos - pos)
  File "C:\App\Python37\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 526, in packUShort
    return struct.pack(">H", value)
struct.error: 'H' format requires 0 <= number <= 65535

...

  File "C:\App\Python37\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 276, in getData
    raise OTLOffsetOverflowError(overflowErrorRecord)
fontTools.ttLib.tables.otBase.OTLOffsetOverflowError: ('GSUB', 'LookupIndex:', 1240, 'SubTableIndex:', None, 'ItemName:'
, None, 'ItemIndex:', None)

A combined GSUB table of between 99K and 124K bytes seems to cause this (I have a working example at 98K and it fails at 124K).
Any guidance / suggestions / ideas would be appreciated!
A reproducer package has been sent to Behdad at his request ...

The original report I submitted was done against FontTools 3.38.0. I have just tried the same script against 3.41.0 and get (essentially) the same results.
The fonts are mostly Noto fonts and most have been scaled from 1000 UperEm to 2048 by my code on top of perl FontUtils (the "KM2K" in the file name). All fonts have then been subset by pyftsubset to a single script (but mulitple blocks) - the "_SS..." in the file name.
The GSUB tables of the merged files are rather large (NotoSerifTibetan and Myanmar, in particular) ... but that's all the likely useful info I have.
Running on Win7x64 produces this STDOUT/STDERR:
D:\Dev\MergeCrash>pyftmerge NotoSansRejang-Regular_2019_OFL_KM2K_SSRjng.ttf NotoSansSamaritan-Regular_2019_OFL_K
M2K_SSSamr.ttf NotoSansSaurashtra-Regular_2019_OFL_KM2K_SSSaur.ttf NotoSansSharada-Regular_2019_OFL_KM2K_SSShrd.ttf Noto
SansSoraSompeng-Regular_2019_OFL_KM2K_SSSora.ttf NotoSansSundanese-Regular_2019_OFL_KM2K_SSSund.ttf NotoSansSyriac-Regul
ar_2019_OFL_KM2K_SSSyrc.ttf NotoSansTagbanwa-Regular_2019_OFL_KM2K_SSTagb.ttf NotoSansTaiTham-Regular_2019_OFL_SSLana.tt
f NotoSansTakri-Regular_2019_OFL_KM2K_SSTakr.ttf NotoSansTirhuta-Regular_2019_OFL_KM2K_SSTirh.ttf NotoSansVai-Regular_20
19_OFL_KM2K_SSVaii.ttf NotoSansWarangCiti-Regular_2019_OFL_KM2K_SSWara.ttf NotoSansYi-Regular_2019_OFL_KM2K_SSYiii.ttf N
otoSerifSinhala-Regular_2019_OFL_KM2K_SSSinh.ttf NotoSerifTamil-Regular_2019_OFL_KM2K_SSTaml.ttf NotoSerifTelugu-Regular
_2019_OFL_SSTelu.ttf NotoSerifThai-Regular_2019_OFL_KM2K_SSThai.ttf NotoSerifTibetan-Regular_2019_OFL_KM2K_SSTibt.ttf Ta
iHeritagePro-Regular_SIL_2018_OFL_SSTavt.ttf
Traceback (most recent call last):
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 271, in getData
    items[i] = packUShort(item.pos - pos)
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 526, in packUShort
    return struct.pack(">H", value)
struct.error: 'H' format requires 0 <= number <= 65535

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "c:\app\python37\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "c:\app\python37\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\App\Python37\Scripts\pyftmerge.exe\__main__.py", line 9, in <module>
  File "c:\app\python37\lib\site-packages\fontTools\misc\loggingTools.py", line 375, in wrapper
    return func(*args, **kwds)
  File "c:\app\python37\lib\site-packages\fontTools\merge.py", line 1129, in main
    font.save(outfile)
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\ttFont.py", line 173, in save
    writer_reordersTables = self._save(tmp)
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\ttFont.py", line 212, in _save
    self._writeTable(tag, writer, done, tableCache)
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\ttFont.py", line 633, in _writeTable
    tabledata = self.getTableData(tag)
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\ttFont.py", line 651, in getTableData
    return self.tables[tag].compile(self)
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 71, in compile
    return writer.getAllData()
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 401, in getAllData
    tableData = table.getData()
  File "c:\app\python37\lib\site-packages\fontTools\ttLib\tables\otBase.py", line 276, in getData
    raise OTLOffsetOverflowError(overflowErrorRecord)
fontTools.ttLib.tables.otBase.OTLOffsetOverflowError: ('GSUB', 'LookupIndex:', 284, 'SubTableIndex:', None, 'ItemName:',
 None, 'ItemIndex:', None)

Would it be helpful if I posted the reproducer package here?

I could be wrong but this error
  struct.error: 'H' format requires 0 <= number <= 65535

makes me think that the resulting font would have more glyphs than what the format allows.

I debugged this some.  Gets stuck trying to fix overflow in one of the  lookups, keeps retrying on the same lookup.  Maybe it is number of glyphs getting too large, donno.  Needs more debugging.

Glyph count should be within (official) limits, but the combined GSUB is large (I think NotoSerifTibetan-Regular is the worst offender). I've got 20+ years of perl history, but no Python experience, so I'm stumbling around here ... not sure what to do. Been trying to track down the handling of GSUB LookupType 7 / Extension Substitution subtables, but I'm mostly stumbling ...
Any suggestions as to where I should look / what I can do would be appreciated. As a last resort, I can carve the project into multiple smaller fonts, but that would be a bit sad ...

This pyftmerge crash is still an issue for me. I have carved multi-script fonts into separate "variants" to accommodate scripts that carry large GPOS tables. However (and I cannot see how this is done), is there use of / implementation of GPOS lookup type 9 (Extension Positioning)?


merge package from arlac77/npm-package-template

.travis.yml

chore(travis): merge from template .travis.yml

Coverage remained the same at 90.909% when pulling 072a4a1a2eab75682239c48a550f2199b8750398 on template-sync-1 into 322a079b0df7996f9ca052814d69b48266aae3d6 on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[IN]

You do not need to put in anything here :)

Just open the issue by clicking the green "Submit new issue"; within 10-15s, a message instructing you how to upload your files should appear!

vasp
VASP.zip

cif
Uploading VASP.zip…

vasp
LiZn.zip

Hi @lyw998111

The LiZn.zip passed through pySIPFENN and ALIGNN happily, but for some reason, CHGNet cannot parse it and returns An error occurred: string indices must be integers, not 'str', as you can see in the action log.

You can see the partial output (pySIPFENN and ALIGNN) in the logs, if that works for you:

name
ALIGNN-JARVIS Formation Energy [eV/atom]
ALIGNN-JARVIS MBJ Bandgap [eV]
ALIGNN-JARVIS Shear Modulus [GPa]
ALIGNN-JARVIS Bulk Modulus [GPa]
ALIGNN-JARVIS Exfoliation Energy [meV/atom]
ALIGNN-MP Formation Energy [eV/atom]
ALIGNN-MP PBE Bandgap [eV]

POSCAR.VASP
-0.2516
-0.0304
31.4424
42.5166
526.1657
-0.2194
0.0143

name
SIPFENN_NN30-OQMD [eV/atom]

POSCAR.VASP
-0.1876


feat: change Array.set to take a Nat and a tactic provided bound

This PR changes the signature of Array.set to take a Nat, and a tactic-provided bound, rather than a Fin.
Corresponding changes (but without the auto-param) for Array.get will arrive shortly, after which I'll go more pervasively through the Array API.

Why?


Output just the frames?

Hello all, not a bug report but a general question. Right now, the output format only supports some form of mp4, webm or gif. Is it possible to output just the frames without assembling them?

Disregard, I just used a different "save image" node


updated image_type defaults

It seems like the meta module no longer produces pFgA_z images.  This results in the decoder and the meta module crashing when not specifying a correct image.  I have updated the default to pFgA_pF=0.50.  Feel free to change it if you want to go with something else, but definitely update it to something.  This could be really annoying for beginning neurosynth users.

Awesome, thanks for the PR--this is long overdue. But it looks like the tests are failing now and need to be updated; can you take a look? Probably just need to update a string to reflect the change.

Oh, hold on, sorry. The issue is actually the the _z images are now called something else  ("association_test" maybe? or "specificity"?). I think you just need to rename the target. Don't change it to the pF image, because that will give pretty different results. Sorry about the confusion--I need to a bunch of maintenance on this package...

There are two new ones that I don't recognize [consistency_z, specificity_z].  What are these and which one should I use?  Is it the same as before with just a new name?

Consistency is the previous pAgF_z, specificity was previously pFgA_z. Sorry about the confusion. I actually never meant to push those changes to master (was still mulling names over), but a year later, here we are... let me know if you have other suggestions.

ok, switched the names to specificity_z.  I think those names are fine, would be good to have some documentation somewhere.  The tests are passing on my end for these changes in the analysis module.  There is another test failing on the base module, which I don't think is related to these changes.  The tests seem to have been failing before today, but I don't have access to your travisci account to check for sure.
______________________________________ TestBase.test_selection_by_peaks ______________________________________

self = <neurosynth.tests.test_base.TestBase testMethod=test_selection_by_peaks>

    def test_selection_by_peaks(self):
        """ Test peak-based Mappable selection. """
        ids = self.real_dataset.get_studies(peaks=[[0, 20, 40]])
>       self.assertEquals(len(ids), 3)
E       AssertionError: 1 != 3

neurosynth/tests/test_base.py:181: AssertionError```

Looks good, thanks!

This probably deserves its own issue, but due to changing the name of these outputs, the docs and tutorials are now out of date. e.g.: http://neurosynth.readthedocs.io/en/latest/neurosynth.analysis.meta.html#module-neurosynth.analysis.meta


Validate weights in VirtualService sum up to 100

Describe the feature request
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[X] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

This already exists: https://github.com/istio/istio/blob/master/pkg/config/validation/validation.go#L2088-L2091
https://github.com/istio/istio/blob/master/pkg/config/validation/validation.go#L2104-L2107


Add site url, fixes RSS feeds

The XML feed templates prefix the site URL, e.g. here:
https://github.com/LineageOS/www/blob/39dbf0bce5798aebe163ef1b94730a7c028793b6/feed.xml#L15
However, the URL wasn't set and the feeds contained just relative paths as links, which is not a valid RSS feed.

Apparently this has been fixed on the server side, thank you!


Optimize for layer store: Decouple layer digest and layer id

We've been taking digest(layer data) as layerid, it's ok at initiation period and keep the info in image.yaml, which is unique and consistent. But we encountered the problem of different compress digest in different env(majorly derived from header info), and there are no awesome approach to fix the issue well.
So I decide to change the decouple between layer id layer digest,  layer id would be consistent after initiation, but layer digest may change for some reasons. In the meanwhile, I use tar-split util to make sure the filesystem of the layer consistency, even though the layer digest has changed.
And the layer orders in image yaml and in registry description are equal, to make sure each layer can retrieve correct layer filesystem.

And we might need to add a parent record in each layers metadata to make build cache mechanism easy to implement.


troubleshooting: fix issue with missing spldev key

This patch describes how to fix the failed OTA update when spldev key is
missing from the factory
Signed-off-by: Milosz Wasilewski<EMAIL_ADDRESS>

Docs for dba246804e6686ba25907028af86b98f3ccc5ee9 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1386/docs/artifacts/html/index.html

@kprosise all fixed and updated in the branch

Docs for a6fd6ae1582c67b9d5cec55dbdfc192bdb0c18fc are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1389/docs/artifacts/html/index.html

LGTM. Once the checks finish, I'll go ahead and merge.

Docs for 64b4e217599a54d7e34b8ef4e15ab14f15673687 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1490/docs/artifacts/html/index.html

Link-check worked when I checked locally, issue seems to have been transient. I can not recall why this had not been merged, but as of right now, everything seems fine. @mwasilew, if it all looks good to you, you can go ahead and merge it.

Thanks, merged. The link issue is most likely unrelated as it's in a different part of the docs.


uncaught (in promise) timeout

Every time a component containing <Reaptcha /> is unmounted I get (in 5–10 sec) that annoying error in recaptcha__en.js:105. Handlers onError and onExpire are not called. I tried to call reset and _stopTimer in componentWillUnmount but it didn't help.
I use invisible Recaptcha v2. Actually, when component with <Reaptcha /> is mounted again after the error, Recaptcha works without problems. So the only problem is that error falling to console and error tracking service.
I also tried several solutions (e.g. clearing cookies) but no one worked.
Is there any way to avoid it?

I've played around a little bit with this error and can reproduce it. Unfortunately this is an error in Google's recaptcha script, which - for some reason - has the reset function delayed and expects the widget to still be available in the DOM (which isn't, because React cleans it up).
There's a workaround for this introduced in [react-google-recaptcha(https://github.com/dozoisch/react-google-recaptcha/pull/120) but I don't really like fixes that add some unclear logic just to get rid of some console error that actually has no impact on the overall functioning.
Unfortunately I don't have much time right now to look for some other solution, but I'd be happy to review any PR's on this. If no one will be willing to take this, I'll take a look at this issue in 1-2 months.

@sarneeh useful thread on this on SO.
First answer:

I had this error too and I found is related with the recaptcha callback (in your case data-callback="callback").