[TS] Handle adding async for multiple await in SourceFixAll properly

Str:

"editor.codeActionsOnSave": { "source.fixAll": true }

const func = () => {
  await 5;
  await 5;
}

After save:
const func = async async () => {
  await 5;
  await 5;
}

This pr should fix so only one async is added at location.
Code action (reference ) fixAwaitInSyncFunction always output async edit per function where needed (and possibly second to change return type), thats why I chose to use this workaround.

Sorry I missed this PR when it original came in
I'd really like to move fix all to the TypeScript side. Our fix all code is already complicated and buggy. TypeScript should have a much better understanding of the code and how to generate valid edits. Here's the TS issue tracking this: https://github.com/microsoft/typescript/issues/29452

I can’t support the reasoning behind your decisions. This feature is widely used, and the change was a clear bugfix for very annoying issue that we could use while waiting for the ideal solution to arrive. Anyway, will be tracking that issue then…


Support for ARM instance on AWS

As AWS has already supported ARM instance, I think we can try to make an AMI for ARM.
https://aws.amazon.com/cn/blogs/aws/new-ec2-instances-a1-powered-by-arm-based-aws-graviton-processors/

Is there any update on this work?


Planned release date

Hi,
I was wondering when is the planned date for the release of your code? Your paper is quite interesting and wanted to reproduce the results.
Thanks

I am currently spending the Chinese New Year in wuhan city, China. Due to the impact of the new coronavirus, the city of wuhan has shut down public transportation, and I'll be stuck here until the virus is contained. The server of our lab also has been suspended, so I cannot access the original program. But I promise I'll upload the code as soon as I get back to the lab. In the meantime, you can try to reproduce our paper. Much of the code for this paper is based on centernet, so we believe this can be easily replicated.
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2020年2月10日(星期一) 晚上10:52
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>主题: [Banconxuan/RTM3D] Planned release date (#1)
Hi,
I was wondering when is the planned date for the release of your code? Your paper is quite interesting and wanted to reproduce the results.
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

 Thanks for your answer and I hope this situation ends soon for all the people over there.

Did you have plan to release your code?


add custom url input to pass to docker

This lets us pass a preview URL to the cli
  ...
  rainforest:
    name: Rainforest QA GitHub Action
    needs: wait_for_preview_deployment
    runs-on: ubuntu-latest
    steps:
      - name: Display the preview url
        run: "echo ${{ needs.wait_for_preview_deployment.outputs.preview_url }}"
      - name: Running the Rainforest Test
        uses: taplytics/github-action@master
        with:
          environment_id: 27323 # Note: This is our test environment ID
          run_group_id: 9915
          crowd: automation
          token: ${{ secrets.RAINFOREST_API_TOKEN }}
          custom_url: ${{ needs.wait_for_preview_deployment.outputs.preview_url }}

@nikolasleblanc Thanks for opening this PR! 🙇🏼 I made some slight tweaks and added documentation for this to the README in #5

Awesome thanks @magni-!


Upgrade to cloudpickle 3.0.0

Upgrade to cloudpickle 3.0.0 https://github.com/cloudpipe/cloudpickle/releases/tag/v3.0.0
This version official adds support for Python 3.12.
Most of the changes in cloudpickle are removing dead code.

Need to adopt patch from https://github.com/numba/numba/pull/7388

Note for reviewers:
Best to read this commit by commit. First commit is purely syncing code to cloudpickle 3.0.0.


Standard operating Procedure for QD and (Q applications)

Is your feature request related to a problem? Please describe.
Provide SOP documentation related to Q to enable the customer deployment and to fit their practices and processes.  We will figure out what this should be.

Closing. We have deployment docs now; it'll be improved on a need basis.


avoid FutureWarnings related to multiindexing in Numpy1.15

This PR fixes additional cases not previously addressed by #378.
Lists of slices have been converted to tuples of slices prior to use for multiindexing.  After this PR, the PyWavelets tests complete without any warnings being raised under Numpy 1.15.0rc1.

Thanks @grlee77, all looks good to me. Double checked that the FutureWarning you're silencing here is covered by an issue (gh-81) and assigned to the next release. I was confused by the diff, but that actually didn't have anything to do with numpy 1.15.


ERROR: Memory: bad allocation size

I'm trying to export assets from two games.
i keep getting the "ERROR: Memory: bad allocation size"  followed by a changing string of numbers each time no matter which settings i use. i have studied the Compatibility Table for both games and no setting seem to work.
and before i'm pointed to the Forums (https://www.gildor.org/smf/index.php/topic,6385.0.html) / Google, like Similar past issues. Nada no help there.
I'm using the new relaese:
UE Viewer for Win32 | Size944.06 KB | Updated20 May, 2020 - 12:28
the two games i am trying to export:
Deliver us the Moon
The Turing Test
*** ERROR: Memory: bad allocation size -482914636 bytes
appMalloc:size=-482914636 (total=2 Mbytes) <- FArray::Empty:1391412879 x 12 <- FPackageFileSummary::Serialize4 <- FPackageFileSummary<<:Ver=0/0 <- UnPackage::UnPackage:Engine/Content/BasicShapes/BasicShapeMaterial.uasset, ver=413/0, game=ue4.6 <- UnPackage::LoadPackage:Engine/Content/BasicShapes/BasicShapeMaterial.uasset <- UIMenuItem::HandleCommand <- UIMenu::Popup <- UIBaseDialog::WndProc <- UIBaseDialog::CustomMessageLoop <- UIBaseDialog::ShowDialog:modal=1, title="Choose a package to open" <- CUmodelApp::ShowPackageUI <- Main:umodel_build=1190

"Deliver to the Moon" has specified version 4.16, you're setting 4.6. What's wrong?
Anyway, in #113 I've asked to not create issues of this kind, this is a user error - not an issue in UModel.

I’ve tried numerous settings. both Overriding auto detection and setting it to 4.16 & 4.22 that is listed on the compat table.
Not setting the override gets me the unversioned pop up in which there is no option for either of the two stated. It only has
4.0, 4.1, 4.2, 4.3, 4.4, 4.5 And 4.6. Nothing else.
All ways still gets me the same final “ ERROR: Memory: bad allocation size”
I had no problem in the Past with “Deliver Us the Moon“ however that was with previous versions of Umodel which I no longer have. Thus me thinking it’s a change in code.

Older UModel versions are available here, as a history of umodel.exe (it was mentioned in FAQ). If older version does work, but new one - not, then it's probably a bug.
Anyway, I always suggesting using forum for this kind of conversation, because I'm very rarely checking games myself - other people do that job.

Seems issue author lost interest to the problem. Closing. Feel free to reopen if still needed.

I don't know if I'm doing something absolutely wrong, But it pops up whenever I extract a model.
SO I wouldn't know if I was doing anything wrong. Probably am, But I just need a bit of guidance is all.

Probably you've set up a wrong engine version, or the game is not supported. You should check the compatibility table and the forum thread about this game.

Me pasa algo parecido, me aparece esto
Memory: bad allocation size<PHONE_NUMBER> bytes
appMalloc: size=1673760496 (total=33 Mbytes) <- FArray::Empty: 855307924 x 12 <- FPackageFileSummary::Serialize4 <- FPackageFileSummary<<: Ver=0/0 <- UnPackage::UnPackage: /Engine/Animation/DefaultAnimCurveCompressionSettings.uasset, ver=518/0, game=ue4.24 <- UnPackage::LoadPackage(info): /Engine/Animation/DefaultAnimCurveCompressionSettings.uasset <- ScanContent <- UIMenuItem::HandleCommand <- UIMenu::Popup <- UIBaseDialog::WndProc <- UIBaseDialog::CustomMessageLoop <- UIBaseDialog::ShowDialog: modal=1, title="Choose a package to open" <- CUmodelApp::ShowPackageUI <- Main: umodel_build=1556

Hey, I'm receiving the "bad allocation size" error whenever I'm trying to export anything from the valorant game files using umodel. I've set up numerous engine versions, Downloaded a variety of umodel versions to see if the problem was located in the version I'm using but apparently it's not. I'm not too sure if I'm missing out on some specific settings located in umodel or if there is a problem with the software. As you can see in the images, when the Error message pops up it's still pretending to load in the background while the export has clearly failed. Would love to get some help/know how to solve this.
Best Regards, Tristan

I see you're using UE4.27 setting. I think, if game (you didn't say the NAME of the game) is REALLY using release version of UE4.27 - it might not work because I didn't do final support for this engine version yet.


Switch card image added to nidec giraffe readmes

Remove the switch card image from the main giraffe readme and add it to readme for the nidec giraffe. People are still getting confused by the Toyota giraffe switch image. This removes it so they can discover  the flowchart. I also updated the https://community.comma.ai/wiki/index.php/Installing_Giraffe wiki page to remove a link to the old image.

looks like you have some broken links here?

The links are right for when you merge it, but they don't work on my repo.

Ahh, duhh, ok, will merge.


x86 letsgo error

[hook_mem_unmapped]:  21 0xffa6971d 1 0
[hook_code] address: 0xFFA6971D size: 0xF1F1F1F1 what?
Traceback (most recent call last):
File "D:\tool/ida/plugins\novmpy\ui.py", line 17, in activate
vtil_graph.show_graph(ea)
File "D:\tool/ida/plugins\novmpy\views\vtil_graph.py", line 462, in show_graph
lifter.lift_il(None, VMState(current_handler=ea))
File "D:\tool/ida/plugins\novmpy\vm_lifter.py", line 92, in lift_il
assert parse_result
AssertionError

I have the same problem when I use it


Support for self-signed SSL cert

Goal:
Set VSTS_SSL_NO_VERIFY=1
Agent will not verify self-signed ssl cert for connection back to VSTS.

https://github.com/dotnet/corefx/commit/31336c936584d21426a467096e64dd02940307f6
The dotcore framework level support is there, we need wait for it release, so we can consume it.

Target when dotnet core 1.2.0 release.

Code completed. #1274
Updating doc


Update needed gradle

The Android Gradle plugin supports only Kotlin Gradle plugin version 1.5.20 and higher. The following dependencies do not satisfy the required version: project ':restart_app' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.50

The new version [1.1.1] has been published.


nirlauncher installation issues [manifest outdated]

It looks like the download link has changed for nirlauncher. Along with that, the zip now appears to be password protected. The version has not changed:

This zip file is password-protected. The password for extracting the files is nirsoft9876$

Website:      https://launcher.nirsoft.net/downloads/index.html
Manifest:     https://github.com/lukesampson/scoop-extras/blob/master/nirlauncher.json
cc: @rasa

Fixed

Thanks, I missed this.


Added command "contributors"

Added a command to show the list of contributors using the GitHub API.

Fixes

Fixes #[issue number] by @[issue author]
Types of changes

[ ] Bug/Hot fix (non-breaking change which fixes an bug)
[x] New feature (non-breaking change which adds a functionality/feature)
[ ] Refactoring Old Code ( non-breaking change replacing already existing code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Preview Link Click here

Checklist:

[x] I have read the CONTRIBUTING guidelines in repository.
[x] My pull request has a descriptive title and is attached to an existing issue.
[x] I haven't repeated the code and it follows established code style of repository.
[x] My branch is up-to-date with upstream branch.

Preview link: https://subhranshuchoudhury.github.io/Terminal-Portfolio/

Great Work 👍

Great Work 👍

Thanks ❤😃✨


Updating classes on elements outside of the #swup container

Is there a way to update classes on DOM elements outside of the #swup container based on the new page? For example to reset the active link in a header menu.
Thanks again!

Of course! In the code that handles your header/menu (I'm guessing you separate your code in some way), simply update the state anytime after the URL is changed based on swup events.
For this example, you could use the current URL as an indicator for the active link, as the href of the link would be the same as window.location.pathname. You could do something like:
document.addEventListener('swup:contentReplaced', event => {
    // 1. remove active class from all links
    // 2. go through all links and add active class if href == current URL
});

Similarlly, you can for example use swup:clickLink  event to trigger close of menu on mobile.
If you would like to actually access the new page and do something based on the content, I would suggest to do that the same way, after the content is placed into DOM.

Nice, thanks — I ran this code on the barba.js new page ready event:
// Update classes for elements flagged with "data-barba-update" attribute
		var updateElements = $(newPageRawHTML).find('[data-barba-update]') // New ones
		$('[data-barba-update]').each(function(index) {
	        var newClasses = $(updateElements[index]).attr('class');
	        $(this).attr('class', newClasses);

Is the new page HTML accessible before the contentReplaced event fires? It's nice to get instant feedback (as on clickLink), but I'm hoping for something more flexible that could be used to update other elements outside of the #swup container as well.

No problem!
The content of any page is accessible in cache when it's loaded and you can detect that based on swup:pageLoaded or swup:pagePreloaded, but you can't jump in before the content is replaced in DOM and modify it (or you could directly in cache, but that would just create a mess).
I'm sensing that you would like to modify different parts of the page from the received one. In swup, you can define as many elements that you'd like to replace as you want, not just one container. That should solve your issue. In case that's not the issue, please be specific.

Ahhh, I totally missed that you can define multiple elements for swup to replace! That's exactly what I'm looking for. Elements that need to be swapped outside of the main container could be selected with a data attribute like data-swup-update, with no animation specified so the swap is instantaneous.

That's working great except for one thing — the change is not instant onclick, it's time with the reveal of the new page. Is there anyway to let some elements update instantly without waiting for animations to complete?

Nope, the animation is also there to buy some time for the next page to load. Page can't be replaced if it's not loaded and all of it's parts are replaced at the same time.
If you want to change some visual part of your page right after click (like the active link in menu), while the next page is loading, you can use classes assigned on html element, where classes based on routes are used as well.
For menu links you could also just force hide the active state for all of them during the process of page transition (based on is-changing class).

I'm guessing this is resolved. If not, feel free to reopen.

Thanks and sorry for the slow response, these are good solutions. I think binding to the swup:clickLink event will come closest to the experience I'm looking for.


Add support for the Bitwuzla solver and the timeout option

The boolector team appears to be developing a new solver, bitwuzla. I have a case where it is 3X faster than boolector, though another where it's 2X slower, so it is perhaps not always quicker. It seems to be where most of their development effort is going and was the tool they entered in the 2021 SMT competition, so I would expect it might continue to improve.
The interface unsurprisingly seems very similar to boolector, so I've added it to the same code path.
Both solvers support a timeout with the --time or -t options, so I've added support for that as well.

Thank you for the catch! I clearly hadn't tested the timeout portion. It looks like the wrapper code is expecting "timeout" rather than the more elaborate message being output by these solvers. I've redone the commit to just add Bitwuzla and remove the timeout change.


W2RC serialization fails due to semantic version of release

Using the CR2W command to serialize files to json with release 1.4 fails.
This seems to be because part of the json output writer attempts to read the assembly version of WolvenManager.UI.dll and parse it as a semantic version. The semver library is very strict and requires a patch version, without which it throws an exception. Changing the assembly info version attribute on WolvenManager.UI.dll to 1.4.0 using dnspy fixes the issue, which leads me to believe that the only thing needed is to change the project version attribute to 1.4.0, but I don't have a build environment configured to test this at the moment.
[assembly: AssemblyInformationalVersion("1.4.0")]

Thank you! I changed the version in https://github.com/rfuzzo/WolvenManager/releases/tag/1.4.1


Refactor integration/unit tests

Closes #22

Really need to fix https://github.com/05nelsonm/gradle-kmp-configuration-plugin/issues/33
Could not for the life of me figure out what it was. Needed to completely exclude androidUnitTest source sets because the connected check will run them in the emulator, but w/o any of the emulator runtime things... ridiculous.


There is a "base_url" bug

Hi, first thanks for this project.
But I found that
str_replace($dir, "\\", "/")
should be
str_replace("\\", "/", $dir)

in libs\Server\Server.php and in daux.phar\libs\Server\Server.php

You're right, this is incorrect, I will fix that ASAP


Use of angle brackets around file names for include statements

Would you like to replace any double quotes by angle brackets around file names for include statements?

Hm... since the tensorflow include files are not usually placed in the system directory (yet), that would be the wrong thing to do. However, as we create deb packages that actually install headers into system paths, we should do it. @josh11b, opinions?

I'd agree that we should use angle brackets outside Google.

I suggest to reconsider the consequences of the following wording from the section "16.2 Source file inclusion" in the standard specification for the programming language "C++".
…
The named source file is searched for in an implementation-defined manner. If this search is not supported, or if the search fails, the directive is reprocessed as if it read

#include <h-char-sequence> new-line
…

How do you think about to avoid "a duplicated file search" here?
Would you like to restrict the searched directories for header files of your software?

@josh11b: Opinions?  This might only matter in the future when the C++ API stabilizes.

I've been at Google too long to know what the rest of the world expects, but I could imagine the regexp that would do this if we decide it is a good idea.  Would this just be for examples, or inside the core library?  Manjunath would probably be a better person to ask.

@elfring Can you please clarify a few things? What ARE the "consequences" of the wording that you point to? i.e., can you give an example of what could go wrong if TensorFlow continues to use double quotes for header inclusion? Also, what do you mean by "duplicated file search"?

There are different opinions about the handling of the involved implementation-defined behaviour.

Will header files be also searched outside the specified include directories if double quotes are used for the discussed preprocessor statement?
Is there a speed difference measurable if a file is not found there and the search will be retried with "the angle brackets inclusion method"?

@elfring Thanks a lot for bringing this to our notice. Those two issues are not high priority for us now, and unlikely to be the case in the foreseeable future. I am closing this issue for now. If we do see a real bug due to the double quotes inclusion in the future, we will definitely re-visit this, and of course we know whom to consult :)

Can you imagine use cases where you would like to exclude the search in "local" directories?


Results table to occupied all the available width

Describe the solution you'd like
Really just an aesthetic change.. Let the result table stretch to the available width space, so all the columns are visible without scrolling
Right now its set to 80rem (max-w-7xl). Maybe better to be 80%? dunno

For later Alex to circle back around on this, will add it as a setting in the UI.
https://filamentphp.com/docs/2.x/admin/appearance#changing-the-maximum-content-width


No debug version (also, no default version)

Problem
The newest version of the .csproj lacks the 'Debug|AnyCPU' configuration:
<!--old version, was not replaced-->
  <PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
    <DebugSymbols>false</DebugSymbols>
    <DebugType>none</DebugType>
    <Optimize>false</Optimize>
    <OutputPath>..\Assemblies\</OutputPath>
    <ErrorReport>prompt</ErrorReport>
    <WarningLevel>4</WarningLevel>
  </PropertyGroup>

However, it's the default:
<Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>

The debug version should probably be added back.
When adding it, it's probably a good idea to change DebugSymbols to true and to add the DEBUG symbol, so that DEBUG-only code works:
#if DEBUG
....do debug stuff
#endif

The line that monodevelop uses for this is:
  <!-- inside 'Debug|AnyCPU'-->
   <DefineConstants>DEBUG;</DefineConstants>

Would pull requests be good?

That’s a good point, I had pulled it out for simplicity but if it’s confusing then I’m fine reverting that. A pull request would be welcome!
On Apr 28, 2020, at 12:47 AM, Little White Mouse<EMAIL_ADDRESS>wrote:

Would pull requests be good?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Using the correct buffer length when URI encoding keys

Issue : #463

looks good, thanks!

out in 1.5.15, thanks!


Netdata version bump and module permission fix

Motivation for this change
The version bump is minor, but the module changes allows one of the default plugins to properly read data from /proc to report more process stats. I also added a test to verify the setuid capability.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

I'm all ears if there's a better way to pull in a setuid binary. I tried libredirect but it didn't catch execve and there's no option in netdata itself yet to specify multiple directories for plugins so I could just use /run/wrappers/bin/apps.plugin directly.

There will be a fix upstream landing (probably) in the next release that will make most of the workarounds here obsolete. When that's out, I'll open a new PR.


fix: don't stash away original history methods so other libs can monkeypatch it

Some other libraries might hook into history.replaceState/pushState to do additional work. This no longer works since the introduction of shallow routing because we're stashing away the original history methods. This PR adjusts the DEV time check logic so that this is no longer necessary.
Fixes #11499
Fixes https://github.com/vercel/analytics/issues/121

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Edits

[x] Please ensure that 'Allow edits from maintainers' is checked. PRs without this option may be closed.

This PR, released as part of v2.3.4 results in a new warning that wasn't previously there: [Warning] Avoid using history.pushState(...)andhistory.replaceState(...)as these will conflict with SvelteKit's router. Use thepushStateandreplaceStateimports from$app/navigation instead. (client.js, line 2580)
I'm not using history as suggested by the warning. I'm using goto in a couple of places, from $app/navigation. I haven't changed anything. The warning is not present with v2.3.3.
From a quick test, commenting out all the goto calls still resulted in the warning.


While building DCNv2, an error occured

/home/amax/.conda/envs/yolact/lib/python3.7/site-packages/torch/include/c10/cuda/CUDAStream.h:6:30: fatal error: cuda_runtime_api.h: No such file or directory
compilation terminated.
error: command 'gcc' failed with exit status 1
YOLACT works fine, so I think the CUDA is correctly installed. But when bulid the DCNv2, This error occured.

@FinalFlowers did you manage to resolve this error? I get the same problem, CUDA 10.1.

@FinalFlowers did you manage to resolve this error? I get the same problem, CUDA 10.1.

The system path of CUDA may has some errors. Please check it.

@FinalFlowers  can you elaborate more ?

nvm I resolved my issue here #535


Fix NaN score issue in NN model

If NN detects NaN validation score during training, will break and revert to last checkpoint with non-NaN validation score (or raise RuntimeError if no checkpoint exists yet).
Issue #, if available: https://github.com/awslabs/autogluon/issues/932

Job PR-954-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-954/1/index.html

Job PR-954-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-954/2/index.html

Job PR-954-3 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-954/3/index.html


task/install: rm {proj}.list is idempotent

It does not matter if the {proj}.list file can't be removed because it
does not exist.
Signed-off-by: Loic Dachary<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1552/
Tests failed for this pull request.

test this please

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1579/
Tests passed for this pull request.


__dirname conversion not working in Windows

The __dirname is converted to /file:\/\/(.+)\/[^/]/.exec(import.meta.url)[1]
However, in Windows, the import.meta.url looks like this:
file:///D:/code/unional/clibuilder/packages/clibuilder/ts/loadPlugins.ts
So the regex gives /D:/code/unional/clibuilder/packages/clibuilder/ts/loadPlugins.ts.
Having an extra / in front of the path.

https://github.com/rhysd/dirname-filename-esm
Here is a package that that works

Thanks for the bugreport. The right way is definetelly to use fileURLToPath (as dirname-filename-esm does), but it also requires an additional import inject  and we have to make sure that it hasn't been loaded in some way before. So let's tfirst ry to patch the regex:
'/file:\\/{2,3}(.+)\\/[^/]/.exec(import.meta.url)[1]',

@unional, could you plz test 2.20.1-beta.0?

/file:\/\/{2,3}(.+)\/[^/]/.exec(import.meta.url)![1]

btw I think it should be like this. with escapes and ! to keep tsc quite. :)

Yes, I think that works. 🎉

2.20.1 is on the way

:tada: This issue has been resolved in version 2.20.1 :tada:
The release is available on:

v2.20.1
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

thx. Maybe you can also add the !. Right now we have to manually edit the file to add that in after transforming

np, I will.


Version 0.3.0

I have some changes ready and I'd like to release the recent changes made by @LukasKnuth:

Release https://github.com/martinthenth/goal/pull/50
Release https://github.com/martinthenth/goal/pull/53
Add :any field type and set it as default 🏗
Remove defschema in favor of defparams 🏗

Defaulting to :any would allow for proponents of the "check don't validate" parameter validation methodology to omit the types and simply check for presence. This change would make a breaking change for users of the basic syntax (not for the defparams users), because the current default is :string.
defschema is an earlier version of defparams that expands to the basic syntax. I'm not sure that anybody uses it anymore, because defparams is more convenient.

Applied in https://github.com/martinthenth/goal/pull/55


case 1 bike count issues

df.index = ... -> add new column, and then use set_index?
should we still use dayfirst?

Slicing with timestamps doesn't work because not monotonic .. -> error in the data (updating the dataset (https://github.com/jorisvandenbossche/course-python-data/issues/19) might already solve this)

Interim data -> provide in material (so at some point students can start doing the analysis exercises without having finished the preprocessing ones)


InsertOrReplace ignores PrimaryKey AutoIncrement

If I use InsertOrReplace to insert a new record, which has a primary key of 0, it inserts the record with 0 instead of using the next available id.

Sorry to dig that post but it appears that the problem is still there.
AutoIncrement is apparently not working with Insert.
I didn't find any answer to that on Google or here.
So i did a workaround before i execute the Insert, i check for the last Id of the table and increment it (if one row is present).
Stock lastStock = database.Table<Stock> ().OrderByDescending (u => u.Id).FirstOrDefault ();
if (lastStock != null) {
    newStock.Id = lastStock.Id + 1;
} else {
    lastStock.Id = 1;
}

Yes, the InsertOrReplace is still not working :(
My workaround is that the Update and Insert method is working good.
using (var dbConn = new SQLiteConnection(_dbPath))
{
    var item = dbConn.Table<DbType>().FirstOrDefault(r => r.Id == id);
    if (item != null)
    {
        item.Property = newValue;
        dbConn.Update(item, typeof(DbType));
    }
    else
    {
        item = new DbType() { Property = newValue, };
        dbConn.Insert(item, typeof(DbType));
    }
}

Thanks @ColonelDuddits making PK nullable made InsertOrReplace work correctly.

Making the ID nullable did work for me too.
I've changed something like this:
[Table(@"Categories")]
public sealed class ShopCategoryModel
{
    [PrimaryKey, AutoIncrement]
    public int Id { get; set; }

    // ...
}

to something like this:
[Table(@"Categories")]
public sealed class ShopCategoryModel
{
    [PrimaryKey, AutoIncrement]
    public int? Id { get; set; }

    // ...
}

But I think it is really "ugly" to use nullable IDs in my code (always using GetValueOrDefault() decreases readability, IMO).

Thanks @ColonelDuddits, @Kwoth  making PK nullable worked for me.

Well it is not working as is should, the null id get always incremented by 1, whats the point, it should work like insert or update, if it is the some record, don't update the id.

This is looking like a bug to me. I'll take a look...

I confirmed this bug is still present in v1.6.258-beta.
Following @UweKeim's work-around and using a nullable int, e.g. int?, for the [PrimaryKey] worked for me:
[PrimaryKey, AutoIncrement]
public int? ID { get; set; }

It's 2021 and this is still a problem and a barrier to new users. I would still be pulling my hair out had I not found this issue. Requiring the use of a nullable integer for an autoincremented primary key violates the concept of a primary key.

I've resorted to always using nullable int for primary key.

I found it more convenient to implement my own "SaveItem( Type obj)" function; My version tests the Id field for 0 and performs an Add; if not zero it performs an Update...
Regards,
Gary
From: Olumide Oyetoke @.>
Sent: Sunday, June 20, 2021 2:06 AM
To: praeclarum/sqlite-net @.>
Cc: Subscribed @.***>
Subject: Re: [praeclarum/sqlite-net] InsertOrReplace ignores PrimaryKey AutoIncrement (#327)
I've resorted to always using nullable int for primary key.

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpraeclarum%2Fsqlite-net%2Fissues%2F327%23issuecomment-864522730&data=04|01||d10de1fc00f84a5f5fb408d933ca8ffb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637597767357815234|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KVRxd7O1yxkjZkqbYH4pJcF0GHV3z%2F%2BYyO7Qa%2FfoU2A%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADFAH7BHXXJR54CBHI3SNQLTTWVN3ANCNFSM4AXIEK4A&data=04|01||d10de1fc00f84a5f5fb408d933ca8ffb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637597767357815234|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=LbjRq5maRRCTlTSZholJ5BJl8q7vF71spGvaEyrD9r8%3D&reserved=0.

No need for InsertOrReplace.
Wait for InsertAsync....
var T = DatabaseAsyncConnection.InsertAsync(object);
T.Wait();
return T;
Guaranteed to work.

+1 from me that this bug is very annoying.
InsertOrReplace behave differently with Id=0 compared to Insert: Insert replaces value with autogenerated one, while InsertOrUpdate does not. This is not consistent behavior.

+1 from me. Spent a whole day breaking my head to find out this was actually a very annoying bug.


should the minibroker run in not k8s env ?

should the minibroker run in not k8s env, but in location machine ？in other words, if the service broker implement the osb api, and want to deploy it, it must run in k8s env ？

This project is being archived, closing open issues and PRs.
Please see this PR for more information: https://github.com/kubernetes/community/pull/6632


README.md: fix vim-plug installation instructions

Thanks for the fix! Sorry I've been traveling for several weeks so am seeing this late, but the fix is applied now.


Convert repo to an actual API

Proposed Changes

Refactor directory structure
Implement a simple API
Add live-reload with docker-compose for easy local development

TODO

[ ] Implement JWT auth
[x] Fix live-reload crash on API crash
[ ] Ensure kube deployment works properly
[ ] List of 3rd party libraries in README
[ ] Ensure swagger docs are rendered properly
[ ] ^ Update swagger docs
[x] Update .gitignore to ignore .yaml files in new deployments path

Will implement the other TODO's in separate PR's. This PR is too large already, as basically a full refactor.


Feature/max size command

Explain your changes:
This PR adds generation of max-cost zkapp commands to zkapp scheduler tool
Explain how you tested your changes:
*
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet

We agreed to do the refactoring in a follow-up PR


Create a basic top navigation header.

It would be nice to use a sticky top header that shrinks a bit after you scroll down a bit
The content of the header could be something like, for a basic structure.

The flow of the header should be something like
https://www.rstudio.com/home/

@rlaverde could you fix this so that it reflects the current state (includes docs and community / and removes projects and about)


checker(dm): fix deadlock when check-task/start-task

What problem does this PR solve?

Issue Number: close #5759
What is changed and how it works?
ShardingTablesChecker: return an error so errgroup will cancel the context
TablesChecker: start consumer of channel earlier
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Fix a bug that start-task/check-task may stuck.

/run-all-tests

/run-all-tests

/merge


Fix for methods in routes.

Fix for methods in relationship routes. Add support of options "only" and "except" in resource routes

@MCMatters
Thanks for the pull request and good spot on the bug.
In terms of the "only" and "except" options, they're not strictly required. If you're using the JsonApiController then the controller will send 501 Not Implemented responses if you haven't defined the methods required for the route that is being dispatched.
I think that's correct because the JSON API spec defines that all those routes should be available, so from the client's perspective the route should exist. So a 501 response would be semantically correct.
Do you have a use for only and except that I'm not thinking of?

Yes, you are right. I'm beginner in the Laravel and I thought that would be good to have this functional

No problem. Can you update the pull request so that the options aren't in there? Then I'll be able to merge it.

Yes, updated


Receiving multiple Input from Web portal.

Is your feature request related to a problem? Please describe.
We are developing a Web frontend that drives dora-flows at the backend. It seems that we can only setup a dynamic node at the begining of the dora flow to get input. The middle nodes won't be able to take the additional input.
Describe the solution you'd like
Provide the middle-nodes capability to receive input from frontend.
Describe alternatives you've considered
N/A
Additional context
Thank you!
from MAE team

I'm not sure, I understand. Could you provide a simple example ?

I think that you might have been blocked by: #606 not being merged.
Now that it's merged it should be easier to have receive-only dynamic node

How should I use this configuration? Could you provide an example of how to configure synchronized_start in YAML or Python?

Do you need that all nodes are ready to start or do you want them to start independently from your web front end?

Do you need that all nodes are ready to start or do you want them to start independently from your web front end?

I want them to start independently from the web front end.

Yes, so you don't need to configure anything the new merged PR will allow you to do just that.

Yes, so you don't need to configure anything the new merged PR will allow you to do just that.

However, have you tried including multiple dynamic nodes in a dataflow? When I started one of the nodes, there was no response at all.
Can you take a look at this issue?
https://github.com/dora-rs/dora/issues/607

Yes, with the latest version that I have merged. You can include multiple dynamic nodes. You can even include them in the same python scripts.

Yes, with the latest version that I have just merged. You can include multiple dynamic nodes. You can even include them in the same python script.

How should I get your updates? Should I reinstall Dora-RS? If so, which version should I install?

Yes so, to update to latest main:
sudo rm $(which dora)
pip uninstall dora-rs

## dora binary
git clone https://github.com/dora-rs/dora.git
cd dora
cargo build --release -p dora-cli
PATH=$PATH:$(pwd)/target/release

## Python API
maturin develop -m apis/python/node/Cargo.toml

dora --help

Yes so, to update to latest main:
sudo rm $(which dora)
pip uninstall dora-rs

## dora binary
git clone https://github.com/dora-rs/dora.git
cd dora
cargo build --release -p dora-cli
PATH=$PATH:$(pwd)/target/release

## Python API
maturin develop -m apis/python/node/Cargo.toml

dora --help

Alright, thank you.


Testing CDK version to 1.116.0

Codecov Report

Merging #329 (204d2ba) into master (5659a9b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 204d2ba differs from pull request most recent head 67b453c. Consider uploading reports for the commit 67b453c to get more accurate results

@@           Coverage Diff           @@
##           master     #329   +/-   ##
=======================================
  Coverage   97.29%   97.29%
=======================================
  Files           3        3
  Lines          74       74
  Branches       15       15
=======================================
  Hits           72       72
  Misses          2        2

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 fe3283c...67b453c. Read the comment docs.


macos build

Amazing job. Here is a small update to run this on macOS.
Thank you for doing this project. It is really eye opening.

Seems good, just one question, shouldn't there be some install instructions for sdl using brew as well as OpenMP? Like "brew install sdl2" or something? I don't know mac at all so forgive me if I'm misunderstanding

That is very fair. I'll make sure that is the package in brew and add that.

updated readme to include brew install libomp sdl2

Wonderful, looks good!


Laser is not implemented

Pleas implement the lasercontroller, too @beniroquai

I have implemented in 4bbc8eb
Interestingly, I didn't get a message - maybe you have to assign me, too. Anyway. Will close the issue since it's done. Feel free to ask for more parts of the controllers :)


Add three levels of durability

The existing tests pass, but I didn't add any tests for the new durability level.
Why kinds of tests would be appropriate?

I'm kind of sad that I removed commit_helper_inner, it would have been more fun to add a third layer, commit_helper_inner_nested.

Why don't we duplicate a basic test that does a bunch of commits and change it to use eventually durable commits. It won't really test durability guarantees, but it will at least exercise the codepath, and it sounds like more testing infrastructure is needed to really exercise durability guarantees.

Ya, that seems good for now. I'd suggest integration_tests::persistence that tests that data is persisted across closing & re-opening the database

Added a test for eventual persistence, this should be ready to go.

Awesome, merged!


Starting client-only node

From Gitter thread. Not sure where these details might best live (troubleshooting?).

When doing PG connection pooling with Cockroach, should I have my app server connect to a single Cockroach instance in the cluster with multiple connections, or should I be doing one-or-more connections per Cockroach instance and round-robin scheduling, or…?
I have many app server instances (stateless, like a typical HTTP server)
kena @knz 13:46
Load balancing is probably appropriate
A TCP load balancer is known to work
Matt Jibson @mjibson 13:47
another approach is to host your database and application on the same machine. your app always connects to localhost. spin up more nodes as needed and connect to existing ones to make a cluster
RaduBerinde @RaduBerinde 13:47
^ Yeah. You can make the client a node as well, even if it doesn't actually store any data
David Taylor @dt 14:01
hm, would a very large number of 0 store nodes clutter gossip?
Andrei Matei @andreimatei 14:02
I guess, but on the flip side you get the benefit of the leader cache close to the client
but I agree that this no-storage node idea doesn’t seem fully thought through
David Taylor @dt 14:03
obviously we could optimize it a bit, I just don’t think we’ve done that
Erich Ocean @erichocean 14:05
I load balance prior to hitting my application servers, so it sounds like co-locating the app server and a Cockroach instance is the way to go, and then having my PG connection pool only connect to Cockroach over localhost.
David Taylor @dt 14:08
If that ends up being a common deployment pattern, we might want to optimize for those client-only nodes so that the cluster doesn’t have to maintain gossip/heartbeat/etc information about them
Erich Ocean @erichocean 14:11
for client-only Cockroach, do I just pass --store=type=mem,size=0% at startup?
Tamir Duberstein @tamird 14:11
i think it won’t let you
there’s a minimum store size IIRC
Erich Ocean @erichocean 14:18
hmm, so how does one launch a client-only node then?
FWIW, the way RethinkDB handles this is with a special "proxy" binary that's more-or-less a client-only node
Tobias Schottdorf @tschottdorf 14:20
I don’t think it’s currently possible. This type of deployment is clearly planned, but we don’t actually have it running.
jseldess @jseldess 14:22
Based on error messages, seems like currently you have to give it a minimum storage size of 640MiB:
JESSEs-MacBook-Pro:cockroach jesseseldess$ ./cockroach start --store=type=mem,size=640MiB
build:     beta-20160428-94-g1b81f2c @ 2016/05/02 15:00:43 (go1.6)
admin:     http://localhost:8080
sql:       postgresql://root@localhost:26257?sslmode=disable
logs:      /var/folders/5t/1rt6nzr17sjg87mz5n0fw15c0000gn/T/
store[0]:  type=mem,size=640 MiB
Erich Ocean @erichocean 14:26
okay, no worries then

Adding a memory store (with non-zero size) is dangerous: ranges will be rebalanced onto it (unless you're careful with zone configs), and be vulnerable to data loss. We need to support running with no stores for a client-only proxy. (and we should probably also suppress the generation of a node id and turn off any gossip).
Security for such a proxy is tricky - it would need certificates for the node user, which gives it full access to the cluster. This seems like an unacceptable risk for any multi-tenant cluster.
For now we should definitely be steering people towards network-level solutions (dns round-robin, haproxy, etc), or maybe something that understands the postgres protocol like pgbouncer.

Thanks, Ben. Wonder if we should add some details to the "Recommendations" section of Manual Deployment? Or we could create a standalone topic on this. @mberhault, any opinions?

Client-only nodes are really not a thing right now, but we probably do want a story eventually.
What is doable is running a node on the same machine as the client, but it will have the same security requirements as a regular node (certificates in secure mode). The only advantage it provides is that the client can use localhost to reach cockroach.

OK, in addition to that approach, should we call out using network-level solutions like the ones Ben mentions?

Closing this for now, until we have an approach, or stronger recommendations, for client-only nodes.


World Crashes upon loading

Mod Version: <IP_ADDRESS> for 1.7.10
Forge Version: <IP_ADDRESS>4
Link to crash log (if applicable): https://pastebin.com/KHhWi2Da
Description: Opened world, and it immediately crashed. I relaunched Minecraft and was able to enter world. Posted to /r/feedthebeastcrashes and someone said that it looks like it was Dynamic Surroundings, and that I should post it here.
In the case that it may be a conflict my mod list can be found here: https://www.modlister.com/user/beanaboston

No, DS didn't do that.  It just gets fingered for it because the mod's name is in the call stack.  In the cases I have seen this is due to another mod reaching in from a thread other than the client thread (such as from the server thread) and trying to do something with sounds.  I recently put in some changes into the 1.10.x+ versions that guard against this bad behavior just so I don't get these reports. :)  I may backport those changes once I finish up with my current release train.

Ah, all right. That makes sense. Sorry for bothering you with this. Thanks for explaining it.

No worries.  I would rather hear about problems than not, even if they aren't mine. :)

Pushed v<IP_ADDRESS> to CurseForge with several updates.  Let me know if it fixes things.


Applied fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

[Current coverage][cc-pull] is 65.52%

Merging [#24][cc-pull] into [develop][cc-base-branch] will not change coverage

@@            develop        #24   diff @@
==========================================
  Files             9          9
  Lines           293        293
  Methods          74         74
  Messages          0          0
  Branches          0          0
==========================================
  Hits            192        192
  Misses          101        101
  Partials          0          0

Powered by Codecov. Last updated by [523f1c2...4dea43d][cc-compare]
[cc-base-branch]: https://codecov.io/gh/tidal/WampWatch/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/tidal/WampWatch/compare/523f1c20936e685a59dd6f37d1604d04ab7f2ae1...4dea43d24dab5799e1f3b6afeba40cb6088cee8e?src=pr
[cc-pull]: https://codecov.io/gh/tidal/WampWatch/pull/24?src=pr


Downloads very large, not stripped

Please strip everything. Example:
squashfs-root/usr/bin/AppImageUpdate-Qt: ELF 64-bit LSB executable, x86-64, version 1 (SYSV),
dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 2.6.24,
BuildID[sha1]=3f3deedac2cec5d4741965a605a7cd9db929300d, not stripped

The point not to strip the binaries is it makes it way easier to debug our (still beta) software. Users will see stacktraces, and can provide them when asked to do so.
I am not sure where this overhead comes from, though. It's most likely not that the binaries aren't stripped (strip isn't perfect btw, e.g., static archives embedded aren't perfectly optimized, it'd be better to do that before linking the final executable).
I'll look into reducing the overhead. AppImageUpdate's binary used to be maybe around 5-6 MiB.

The reason the main AppImage (FLTK UI) is so large is that all the Qt libraries have been bundled in it, too, unnecessarily. I know why that happens, and I will push a fix.

WIP in #80.

Closed via #80.


setAutoCollectPerfarmance to false keeps sending performance metrics

I disabled the collection of performance metrics, however I can still see the data in the dashboard. Is this method setAutoCollectPerformance(false) meant to switch the performance data off?

setAutoCollectPerformance(false) is indeed intended to disable autocollection of performance counters when used like so:
appInsights.setup("<instrumentation_key>")
    .setAutoCollectPerformance(false)
    .start();

Is this what your code looks like?
To clarify, you're seeing new performance counter data show up in Application Insights even after setting this flag and re-deploying/restarting your app?

Closing this issue. Please feel free to open it again if you're still experiencing this!

Many thanks, this helped!


Conceptualisation 🖊 Concerning the Diseases of Women, by Metrodora

Title

Concerning the Diseases of Women, by Metrodora
Date or Period 📅

200 CE because she seems to have lived in the second century (Gregory et al., 2013).
Intro

Metrodora appears to be the first female author of a medical text.
This story has been published on Tiki-Toki, but please feel free to reopen this issue and make any suggestions below!

#142 :)


Implement a Kafka consumer to read data from CF Broker Topic

Basic implementation of CF Broker Topic is available. Need testing with portal. Will request actual topic once it's working.

Changed plan: use RESTful API to get ABQM data from Orchestrator instead. It's hard to use Kafka consumer within the client browser (FABRIC portal is a web application).


Make gazelle tests respect USE_BAZEL_VERSION

Checklist

[X] I have filed an issue about this change and discussed potential changes with the maintainers.
[X] I have received the approval from the maintainers to make this change.
[X] This is not a stylistic, refactoring, or cleanup change.

Discussion thread for this change
@mai93 brought it to attention.
Description of this change
Some downstream tests use the USE_BAZEL_VERSION env var to set up the Bazel executable. Normally, this would not be set up for tests, as most tests don't run Bazel themselves. However, the Gazelle integration tests do need to run Bazel, so we must set those global settings manually.

@blorente Unfortunately we are still getting errors with Bazel@Head: https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/2744#0184a2ac-9040-4254-a361-78cc4c1fb45f
I was thinking about mocking the build step, I think other sync tests do that https://github.com/bazelbuild/intellij/blob/master/base/tests/utils/integration/com/google/idea/blaze/base/sync/BlazeSyncIntegrationTestCase.java#L342.


Create options to support HeatMap and ClusterMap for the project

Basic support added in version 0.5.0


console.earlySetup does not build

Describe the bug
The following config fails during build:
{ config, lib, pkgs, ... }:
{
        console.earlySetup = true;
        console.keyMap = "de-latin1";
        console.font = "ter-v32b.psf.gz";
        console.packages = [ pkgs.terminus_font ];
}

'/nix/store/d1isq56iqbwwdj3fpa6clwc23scr1lxa-extra-utils/bin/fsck.ext4' -> 'e2fsck'
cp: missing destination file operand after '/nix/store/d1isq56iqbwwdj3fpa6clwc23scr1lxa-extra-utils/share/consolefonts/font.psf'
Try 'cp --help' for more information.
builder for '/nix/store/5dzsywbjgqzvd5mz6w91drpil7dfh69c-extra-utils.drv' failed with exit code 1

I may have a fix that I'm testing locally.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute: console.earlySetup
# a list of nixos modules affected by the problem
module:

Is doing console.font = "ter-v32b"; undesirable in your use case? I think that's the expected way to set console.font.

@collares: Yeah, I think you're not supposed to put the suffix in the font name.
   console.font
           The font used for the virtual
           consoles. Leave empty to use
           whatever the setfont program
           considers the default font.

           Type: string

           Default: "Lat2-Terminus16"

           Example: "LatArCyrHeb-16"

Is doing console.font = "ter-v32b"; undesirable in your use case? I think that's the expected way to set console.font.

That would have been far too easy compared to diving into nixpkgs, figuring out which line causes it and rewriting that following by opening a PR...
I guessed I couldn't find the wood due to all the trees. Thanks :)


Set default for manage_repo param to true (use packages.icinga.com)

Imho it should be best practice to use the official packages from packages.icinga.com, especially when you are using RPM based deployments. Debian users may still opt out from that.
Therefore I'm kindly asking to set manage_repo to true by default. If you agree, I'll happily send you a PR.

Discussed this offline. We don't want to manage the users repositories by default.


Fix delete and edit for Groups

References

Fixes #893
Fixes #911

Description
This PR implements the delete and edit features for groups that were incomplete up until now.
Instructions for Reviewers
You can test this PR by authenticating as an admin and trying to delete groups from both the list view and the edit group page.
Verify that the deletes work, that the groups aren't shown in the lists anymore afterwards and that you no longer get notifications saying the feature isn't implemented.
Also verify that you can edit the group's name and description and persist those changes
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes TSLint validation using yarn run lint
[x] My PR doesn't introduce circular dependencies
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.

Hi @artlowel , logging in as admin I cannot delete the groups: in the list view the buttons are not visible and in the edit page the button is disabled. Do I need to configure something?

as noted by @LucaGiamminonni not all goups have the delete button. Since the authorizations endpoint return at least one result, the button should be visible. Is our assumption correct?

@MarieVerdonck noticed that the authorization system currently claims the deleting of persistent groups is allowed. But it isn't. Since we can check for that using the persisent property on the group, we hide the delete buttons both for groups that are persistent and for groups for which you aren't authorized.

When I try to edit title of a group, both a success and an error notifications are shown. However the edit seems to work properly

That's an issue, we'll take a look

finally something that does not strictly concern this PR, but could be related. Indeed editing epersons metadata doesn't work as well. Maybe we can make a follow-up PR for this problem

Indeed, that should be a separate ticket and a separate PR.

I recreated the issue where both success and nameInUse error message was shown. NameInUse was meant to check if there was a group found with the same name as the one being attempted, but it just used a search on name => inUse if totalElements of result > 0, but the search isn’t on exact name.
Example: So when you rename a group to Test, it will succeed if Test name is not already in use, but the nameInUse check showed error because for example Test 2 group existed, and got returned with search ‘Test’.
It would be slow to check for all the groups in the search result if their names is an Exact match to the one we are attempting, since could be great number of results, and there is not an endpoint for get group by exact name… (Implemented a similar check with eperson with email but the search there with scope email is exact I believe)

A patch request with a group name that is already in use results in a 500 Internal Server error response with a stack trace (javax.persistence.PersistenceException: org.hibernate.exception.ConstraintViolationException: could not execute batch), which can’t really be used to form a meaningful message as to Why your edit request failed.
The check has been removed for now since as it is now it is not guaranteed to be correctly shown, and the only way for it to be correctly shown would be to check each search result group for an exact name match…

@tdonohue I asked @benbosman about why this split happened, and he said it's been that way since DSpace 6, and was done in order to make it harder to accidentally delete those groups using the rest api. If that's the case, then the UI hiding that added complexity for the user is likely not wanted, otherwise might as well simply allow it on the groups endpoint.
You can already delete these groups from the assign roles tab on edit collection and community pages
I noticed that editing them the same on the groups endpoint fails as well. I tried to PATCH one of them on the collection endpoint using postman, and I got a 405: method not allowed. So this likely means the edit form shouldn't be shown for these groups either. Adding and removing members does work though.
So perhaps the best thing we can do is: if a group has an object link, don't show the delete button in the list view or detail page. Disable the edit form on the detail page and add a message that explains this group can't be modified because it corresponds to a community or collection role, with a link to the "assign roles" tab for the linked object.
I can ask Marie to do that for this PR, but you should know that the scope has expanded multiple times since my original estimate of 4h. We already have 7 hours logged on this task. Doing this will likely add another one or two.

@artlowel : Your suggested approach sounds fine.  I'll update the estimates (which currently total 6hrs as this PR resolves two tickets...however, I'll update them to total 9hrs by adding a few hours to the delete task).
One minor tweak to this part of your suggested approach:

Disable the edit form on the detail page and add a message that explains this group can't be modified because it corresponds to a community or collection role, with a link to the "assign roles" tab for the linked object.

If possible, this same sort of message should be shown with regards to deleting the group.  So, I agree we shouldn't show the delete button (or show a disabled button), and it'd be nice if we provided a message describing why it cannot be deleted as normal & link the user to the "assign roles" tab where that group can be deleted.

The reason for the newGroup call is that the component is used for both editing existing groups and creating new groups. It tries to get the id for the current group from the route, and does a GET to the backend with that id. So it needs to take in to account the newGroup case
The reason for the object, epersons and subgroups calls is that the group list template contains methods that fetch those endpoints. When a group is deleted the alternative links to those objects are also removed from the index. For a split second the template is still showing the freshly deleted group though, so it tries to re-retrieve those links. It's bad for performance in any case to put non-trivial methods in a template, it's definitely a mistake to put methods that perform backend calls in a template as they may be executed many times a second. The solution here is to move the members and subgroup counts to the DTO, so the template has that data available without method calls.
Both of these issues are present in the main branch already, so I'd be in favor of solving them in a separate PR.
Fixing them would take an estimated 4h

Per @artlowel 's suggestion (above), I've created a separate ticket for the 404 errors.  See https://github.com/DSpace/dspace-angular/issues/962
Therefore, merging this as-is.  The 404 errors will be fixed separately.


misaligned MdInput placeholder w/ MdPrefix

BUG
Bug Report
The input placeholder height is off when using MdPrefix; the placeholder and the MdPrefix icon are not aligned.  Also, the placeholder font-size is larger that material2 using convalent platform.scss.  See screenshot below.
With covalent platform.scss

Without covalent platform.scss

Gonna check this out. We are trying to remove the dependency of platform.scss in the upcoming releases.

Hello, I took 2 screenshots with and without platform.scss.
I see no difference in the results.

With platform.scss
Without platform.scss

MatCard issue
I do see a difference on font-size when we render inputs inside a MatCard, but this isn't a platform.scss issue.
Font-size is smaller inside a MatCart

Questions
I see misalignment and font-size differences on the screenshots on the description of this task, but I found no misalignment or text-size differences regarding platform.scss on my tests.
The MdPrefix has no effect on the screenshots.
Are there specific steps to reproduce the text-size and misalignment issues?
Covalent commit
commit 3bbf97088eccbd1e98a139653e38ff05c8f4ffc2 (HEAD -> develop, origin/develop, origin/HEAD)
Author: emoralesb05<EMAIL_ADDRESS>Date:   Mon Oct 8 10:30:24 2018 -0700

This looks to be OBE


python310Packages.ocrmypdf: 14.2.1 -> 14.3.0

Description of changes
Diff: https://github.com/ocrmypdf/OCRmyPDF/compare/v14.2.1...v14.3.0
Changelog: https://github.com/ocrmypdf/OCRmyPDF/blob/v14.3.0/docs/release_notes.rst

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 240488 run on x86_64-linux 1

5 packages built:

ocrmypdf (python310Packages.ocrmypdf)
ocrmypdf.dist (python310Packages.ocrmypdf.dist)
paperless-ngx
python311Packages.ocrmypdf
python311Packages.ocrmypdf.dist


#74 do not use the self signed certificate when talking to the remote server

Signed-off-by: David Radley<EMAIL_ADDRESS>

If you think this code makes the ui better, I think it would be fine to merge. I am happy to then verify it independently as I want to fully understand that API, and the impact of changes to the certs in the base. We can handle any observations as a new issue and/or PR as needed.

If you think this code makes the ui better, I think it would be fine to merge. I am happy to then verify it independently as I want to fully understand that API, and the impact of changes to the certs in the base. I don't yet have all the gaps filled to be confident the solution is complete, but equally it could be my node knowledge. We can handle any observations as a new issue and/or PR as needed. As above any refs to the API would help.

OK I have merged, please test independently,


Problem entering text on Android

Issue summary
If you type text, on the second keypress terminal insert two characters and first character is dead.
Expected behavior
You can enter text normally
Actual behavior
The first key is dead.
Steps to reproduce

Open https://terminal.jcubic.pl/android.html on Android
Type some text

This is actually caused by the keypress event added in android.html file. The library works fine.


Improve content of README

Add more content to README, specifically, screen capture GIFs demonstrating:

syntax highlighting features
commenting command Ctrl+/
snippet insertion and fields

Done


Error Hydration failed because the initial UI does not match what was rendered on the server.

Unhandled Runtime Error
Error: Hydration failed because the initial UI does not match what was rendered on the server.
Warning: Expected server HTML to contain a matching  in .
See more info here: https://nextjs.org/docs/messages/react-hydration-error
Component Stack
path
svg
MoonFilledIcon
./components/icons.tsx (112:11)
div
label
ThemeSwitch
./components/theme-switch.tsx (19:11)
ul
eval
<EMAIL_ADDRESS>(20:11)
header

Just change the moon icon to another

Fixed


chore: update BPF_PROG_TYPE_SCHED_ACT.md

Hi @dylandreimerink
I noticed that the page of BPF_PROG_TYPE_SCHED_ACT has a to be improved mark.
So, I added some information in this page.
However,
I'm not sure the Kfuncs and helpers sections are 100% correct.
My checking way is:

search the kernel source with keyword register_btf_kfunc_id_set(BPF_PROG_TYPE_SCHED_ACT for kfuncs section.
read the comments in tools/include/uapi/linux/bpf.h for helpers section.

Do you have any suggestion on this?
Thank you!

I noticed that the page of BPF_PROG_TYPE_SCHED_ACT has a to be improved mark. So, I added some information in this page.

Thank you. Do you happen to know what sets the SCHED_ACT program apart from a SCHED_CLS in direct-action mode? I have personally never seen SCHED_ACT used, its always the classifier in DA.

search the kernel source with keyword register_btf_kfunc_id_set(BPF_PROG_TYPE_SCHED_ACT for kfuncs section.

This is also how I have done it in the past.

read the comments in tools/include/uapi/linux/bpf.h for helpers section.

Resolving helpers is more difficult in general. The kernel has a bunch of functions that call into each other. I modeled this with the data file.
To double check it you start at the verifier ops for the SCHED_ACT programs and work down:

https://elixir.bootlin.com/linux/v6.12.1/source/net/core/filter.c#L11014
https://elixir.bootlin.com/linux/v6.12.1/source/net/core/filter.c#L8133
https://elixir.bootlin.com/linux/v6.12.1/source/net/core/filter.c#L11864
https://elixir.bootlin.com/linux/v6.12.1/source/kernel/bpf/helpers.c#L1892

The data file contains "groups" for each of these functions that are squashed. So as long as the list of helpers in every group is correct, and the groups reference each other correctly the end result should be good.

Hi @dylandreimerink

Thank you. Do you happen to know what sets the SCHED_ACT program apart from a SCHED_CLS in direct-action mode? I have personally never seen SCHED_ACT used, its always the classifier in DA.

Me too, I have never seen the SCHED_ACT used. But I can find some example on GitHub link.

Resolving helpers is more difficult in general. The kernel has a bunch of functions that call into each other. I modeled this with the data file.
To double check it you start at the verifier ops for the SCHED_ACT programs and work down:
...

Thanks for your information, It's really helpful to me.

Me too, I have never seen the SCHED_ACT used. But I can find some example on GitHub link.

Thanks for the link!


Scaffolding page monster footer has link typos

javascript links go to javasript.html intstead of javascript.html. Simple fix!

Fixed!


:sparkles: Support for Amazon Load Balancer Ingress controller

Fixes #184
Partially fixes #167, still need to confirm 'auth' workflows on EKS, there may be additional PRs needed for auth.
Relates to #170
These changes allow us to deploy Konveyor onto an Amazon EKS cluster that uses the Amazon recommended Amazon Load Balancer ingress opposed to 'nginx'.
I deployed the EKS cluster for testing using this newly developed automation:
https://github.com/konveyor/hack_env_helpers/tree/main/aws/eks
I captured notes while exploring ALB Ingress here for debugging purposes:
https://gist.github.com/jwmatthews/d492a4944c83dc3f2d016187d53d13b2
Note at this stage, only 'non auth' workflows have been confirmed working.

@jmontleon this is ready for another review when you have time.
I confirmed we are good with the ALB on EKS with auth enabled.


Editing status, clicking on other areas of the map will cancel the editing status

Is it possible to click on other areas and still maintain the previous editing status?

We could potentially allow this as a feature, how would you want to deselect a feature? In this scenario you could only deselect by selecting another feature or exiting the mode. Would that be acceptable?

I don’t know if my suggestion is correct. In the state of editing, click on the blank area of the map, you should keep the editing status. If you cancel it at this time, the user needs to click the map to enter the editing status again. Obviously this is repeated operation. If you click on the blank area to keep Edit state, I think it is the best.
As you said, if you want to cancel the selection function, you can only cancel the selection by selecting another function or exit mode.
In addition, I hope that the author can optimize the round/rectangle, the editing zoom, and maintain the original shape. If this problem can be solved, I think it can release the official version. Of course, I am also a source code of 'fork' a source code. Try to modify.
Thank you very much, I hope Terra-DRAW can be more complete.

We could potentially allow this as a feature, how would you want to deselect a feature? In this scenario you could only deselect by selecting another feature or exiting the mode. Would that be acceptable?

OK

I think that this can be resolved with https://github.com/JamesLMilner/terra-draw/pull/194 - the approach here is to add allowManualDeselection which can be set to false preventing the user from deselecting by clicking on other places on the map.

Yes, it will be much simpler.

I think that this can be resolved with #194 - the approach here is to add allowManualDeselection which can be set to false preventing the user from deselecting by clicking on other places on the map.

Yes, it will be much simpler.

This change is available on 0.0.1-alpha.57 🎉 Hope it helps 👍🏻


Add support for logback-access v2

Describe the changes

upgrade dependencies
Add support for logback-access v2
only supports jetty 12

v4.2.0 has been released! It will be available in the Maven Central repository soon.


If iso has no parameter transfer, it needs to be released

https://github.com/rogchap/v8go/blob/da7f43c007a43d74523db42da75874ede77d22c3/context.go#L57
In all cases, it is not released
ctx := v8go.NewContext(nil)
ctx.Close()
ctx.Isolate().Dispose()

It can cause memory leaks

I think that is expected.  Although, the documentation could be updated to make this clearer.


Removed deprecated syntax in releases workflow

#141

PR-Codex overview

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@yegor256 I updated auto-pull.py script to use GITHUB_ENV between Actions steps. And I removed deprecated syntax from Github Actions according to GitHub Actions: Deprecating save-state and set-output commands. Please have a look

@yegor256 Reminder

@Graur thanks


Many documents for Architecture and design details needed！

Many documents for Architecture and design details needed！

@Apokleos We are sorting out them and will be released soon.

@Apokleos We are sorting out them and will be released soon.

Thx @jiazhang0, Waiting for them！！！

@Apokleos Sorry for late reply. You could refer to this slide for the details about rune's design. https://inclavare-containers.io/blog/oci-runtime-rune/


stable/mediawiki smtpusername is wrong configured

Describe the bug
Smtp username configuration is not configured correctly.
Which chart:
mediawiki
What happened:
$wgSMTP = array(
'host' => 'testidhost',
'IDHost' => 'testidhost',
'port' => 'testport',
'username' => 'testport',
'password' => 'testpass*',
'auth' => true
);
What you expected to happen:
$wgSMTP = array(
'host' => 'testidhost',
'IDHost' => 'testidhost',
'port' => 'testport',
'username' => 'testusername',
'password' => 'testpass*',
'auth' => true
);

Hi,
Given the stable deprecation timeline, this Bitnami maintained Helm chart is now located at bitnami/charts. Please visit the bitnami/charts GitHub repository to create Issues or PRs.
In this issue we tried to explain more carefully the reasons and motivations behind this transition, please don't hesitate to add a comment in this issue if you have any question related to the migration itself.


add http message creation detail & images

Changes proposed in this pull request
Describes the HTTP signature creation process, including diagrams.

Context
Explains, in fairly simple terms, how the OpenPaymets API has implemented part of the HTTP message signatures draft spec.

If possible, please change the font used in the images. Code examples should use a monospace font (Starlight uses a system monospace font by default). For "regular" text and diagrams we should use a sans-serif font that is easier to read and doesn't look handwritten (Starlight uses a system sans-serif font by default). I believe Mermaid uses Trebuchet for its sequence diagrams, but it's not my favorite. Regular Arial would work.

Related #321

If possible, please change the font used in the images. Code examples should use a monospace font (Starlight uses a system monospace font by default). For "regular" text and diagrams we should use a sans-serif font that is easier to read and doesn't look handwritten (Starlight uses a system sans-serif font by default). I believe Mermaid uses Trebuchet for its sequence diagrams, but it's not my favorite. Regular Arial would work.

Cool and noted, thanks for the thoughtful points @melissahenderson.
I wanted to use Excalidraw exactly for the handwritten 'look' for images.
But if this feels like it creates inconsistency or if it's not inline with the doc style-guide, then we can definitely change it.
From what I see in the Excalidraw docs, there is currently just one alternative font available: 'Helvetica'.
For more detail, check out the Excalidraw docs here

Do the graphics work on black and white background? I think the written part is good, but the technical writers may be able to make it even better 😉.

Thanks @sabineschaller - image updated to work with white or black backgrounds.
:laughing: I am with you about the written part.

Updates merged with #424


docs: Fix missing operator

Just a typo. Thank you for the other corrections and proofreading! 👍

Would you like to add yourself to the contributors table? Feel free to open a PR for that :smile:


Add hardcoded links to related resources that aren't indexed by Swifttype

Is your feature request related to a problem? Please describe
In the right rail, we show people related content based on sites indexed by Swifttype. However, there are resources not indexed by Swifttype that we may want to link.
Describe the solution you'd like
In front matter, make it possible for people to specify links to these non-indexed resources. For example:

Apps on the open source site
Repos on GitHub
Links in New Relic One (if the user is authenticated)

Describe why this important to you
We want to make sure the most relevant stuff gets elevated in the right rail and something like Swifttype is always going to have holes.

@djsauble Reading this again, I think we have all of this addressed. My question to you is are we going to have different visual treatments for any of these bits of content or are they all going to be a link with the source tag underneath it? If it is the latter, this should be addressed with the resources frontmatter item. Here is an example for the "Add custom attributes" page: https://raw.githubusercontent.com/newrelic/developer-website/main/src/markdown-pages/collect-data/add-custom-attributes.mdx
You'll see the resources frontmatter item that allows someone to specify a URL + title

Cool! I'll let @markweitzel chime in if there's anything I didn't capture in this ticket, but it sounds like we may be good here.

This this currently working. Feel free to reopen if there is something we missed!


Bug loading private key from file

When loading a private key from file the CLI throws an error when assigning the loaded file to a variable.
box configure:environments:add 107417548__config.json --private-key-path=private.pem

BoxCLIError: Could not read private key file /Users/cbetta/private.pem
Caused by: TypeError: Cannot set property 'privateKey' of undefined
    at FoldersGetCommand.getClient (/Users/cbetta/.nvm/versions/node/v14.5.0/lib/node_modules/@box/cli/src/box-command.js:534:35)
    at FoldersGetCommand.init (/Users/cbetta/.nvm/versions/node/v14.5.0/lib/node_modules/@box/cli/src/box-command.js:250:22)
    at FoldersGetCommand._run (/Users/cbetta/.nvm/versions/node/v14.5.0/lib/node_modules/@box/cli/node_modules/@oclif/command/lib/command.js:28:24)
    at Function.Command.run (/Users/cbetta/.nvm/versions/node/v14.5.0/lib/node_modules/@box/cli/node_modules/@oclif/command/lib/command.js:133:16)
    at async Config.runCommand (/Users/cbetta/.nvm/versions/node/v14.5.0/lib/node_modules/@box/cli/node_modules/@oclif/config/lib/config.js:151:9)
    at async Main.run (/Users/cbetta/.nvm/versions/node/v14.5.0/lib/node_modules/@box/cli/node_modules/@oclif/command/lib/main.js:21:9)
    at async Main._run (/Users/cbetta/.nvm/versions/node/v14.5.0/lib/node_modules/@box/cli/node_modules/@oclif/command/lib/command.js:29:20)

It seems the faulty line in question is this line where we assign the loaded file to a value.
https://github.com/box/boxcli/blob/master/src/box-command.js#L534
It seems like we try to assign the loaded file to configObj.appAuth.privateKey where the right syntax should be configObj.boxAppSettings.appAuth.privateKey.

Found bug, reported bug, fixed bug! Way to go!  Thank you!


Code to update v4 genome sample_qc metadata

This script is used to create the sample QC metadata HT for genomes. The main updates from v3.1 to v4.0 are:
Sample QC annotations for HGDP + TGP subset, this script directly merges the updated annotations from the subset meta HT to the full meta HT.

I put both of you as reviewer, but feel free to do it once you have time.

@ch-kr Test run is here: 0677fc8311b04852b9661913b6278822 if you want to see the naming of new_temp_file. Runtime is fine without repartitioning for this one.


Add missing debug implementations

This adds Debug derives to most of the library public API.  I have on
occasion missed not having these.
Derives are for now probably good enough, they can be improved as
needed.

This in on top of the edition-2018 branch so includes those commits as long as they're not merged.  Only the last commit here is relevant.  It's easy enough to rebase if that branch won't be merged so just let me know.

If you rebase this I'm happy to merge it (or you can do so yourself since I just sent you an invitation to be a collaborator).


Simple MongoDB query takes over 30 seconds to excecute

Bug description
I have GoogleOAuth authentication through Passport in my Express server, and I run the .upsert() function to check if a user exists, otherwise I create one. However it appears this query is taking upwards of 30 seconds, meaning the user is stuck on the OAuth consent screen for about that long waiting to be logged in. It seems this is happening on most queries I make, however I've only thoroughly tested with this scenario.
How to reproduce

Set up basic auth template with passport google oauth or copy from a demo repo I made and adjust to use Prisma accordingly.
Navigate to the login route which will eventually execute the upsert() function
Wait....

Prisma information

Schema file
datasource db {
  provider = "mongodb"
  url      = env("DATABASE_URL")
}

generator client {
  provider        = "prisma-client-js"
  previewFeatures = ["mongoDb"]
}

model User {
  id          String @id @default(dbgenerated()) @map("_id") @db.ObjectId
  googleId    String @unique
  displayName String
  avatarUrl   String
  role        String @default("free")
}

// query looks like this, however I've tried making a few adjustments with no success
      const user = await prisma.user.upsert({
        where: {
          googleId: profile.id,
        },
        update: {},
        create: {
          googleId: profile.id,
          displayName: profile.displayName,
          avatarUrl: profile.photos[0].value,
        },
      });

Environment & setup

OS: Windows 11
Database: MongoDB (Atlas)
Node.js version: 16.13.1

Prisma Version
3.7.0

Can you share same sample data that would we can use to create a reproduction without any additional software, just a simple route that executes exactly this query to confirm this is really just Prisma being slow? Thanks.

Here's a basic test just getting all users (of which there is only one):
router.get("/test", async (req: Request, res: Response) => {
  try {
    const users = await prisma.user.findMany();

    return res.json(users);
  } catch (error) {
    return res.status(500).json({ message: "Failed to get users" });
  }
});

It took 21.92 seconds for a total size of 210 bytes.

I can't imagine Prisma and MongoDB are really behaving that differently for you. Are you sure it is the Prisma Client query taking that time and not some other logic happening in your application?

I guess it's possible that something else is causing the slowness, however after switching out only the /test endpoint with the following, I get the same response in a rock solid 1.27 seconds.
import { MongoClient } from "mongodb";
if (!process.env.DATABASE_URL) throw new Error("DATABASE_URL is not set");
const client = new MongoClient(process.env.DATABASE_URL);

router.get("/test", async (req: Request, res: Response) => {
  try {
    await client.connect();
    const database = client.db("myFirstDatabase");
    const users = await database.collection("User").find({}).toArray();

    return res.json(users);
  } catch (error) {
    return res.status(500).json({ message: "Failed to get users" });
  } finally {
    client.close();
  }
});

Yep, that sounds much more realistic with creating a connection to the database and all - which indeed points to Prisma taking that time. Can you maybe add a timestamp logging before and after calling the findMany just to make sure it is really that line that takes so long? You can also enable logging in Prisma Client and use that and debugging to get log output which will allow similar things if you have output of timestamps for everything: https://pris.ly/d/logging + https://pris.ly/d/debugging
Alternatively: Can you maybe share the connection string of your MongoDB Atlas instance with us (in private of course<EMAIL_ADDRESS>so we can reproduce this manually?

    // generate timestamp
    const timestamp = new Date().toTimeString();
    const users = await prisma.user.findMany();

    // generate timestamp
    const timestamp2 = new Date().toTimeString();

    // log time
    console.log(`${timestamp} - ${timestamp2}`);

Logs: 14:09:29 GMT-0500 (Eastern Standard Time) - 14:10:00 GMT-0500 (Eastern Standard Time)
I'll shoot over the connection string shortly!

Received and reproduced - something is definitely weird with that cluster. This is probably something to figure out together with MongoDB Atlas and nothing Prisma can influence.

Can confirm, the cluster was causing the problem. Thanks for the support in finding out the issue here!

Awesome, thanks for playing along and nice we could figure this out.


[Core] Clique engine for Parity Ethereum

Parity Ethereum is a blockchain client and full-node implementation written in Rust programming language. It supports multiple different chains based on the Ethereum Virtual Machine (EVM) or Parity's WebAssembly VM (pWasm). The client comes with "pluggable consensus," so it has different consensus engines, among others the "Ethash" proof-of-work engine, the "AuRa" proof-of-authority engine, and some developer presets like "Instant Seal" and "Basic Authority."
We want to add the "Clique" proof-of-authority engine to that list and allow users to synchronize the Rinkeby, Görli, and Kotti Classic testnets. That breaks down into these major components/tasks:

The "Clique" proof-of-authority core engine with proper block validation logic as per EIP-225, full chain synchronization, and light/headers verification.
The sealing logic to enable the node to be run in "authority mode:" creating, proposing, and sealing new blocks; voting for new validators; unvoting validators to remove them.
The "Clique" JSONRPC-APIs that allow users to control the engine via HTTP/WS/IPC APIs.
Configurations and chain specifications for the Rinkeby, Görli, and Kotti Classic networks.
Tests! Everything should be covered. (This is actually part of all the tasks above, just making sure this is not forgotten about.)

... Profit? All tasks can be claimed via bounties. What is a bounty? That's a fixed amount of money that is claimable by anyone who is submitting a working solution to any of the tasks listed above and sufficient quality. What is sufficient quality? You certainly should have written Rust before and be familiar  with the concepts and best practices of the language. Knowing Parity Ethereum or blockchain experience in general is a bonus. How to claim a bounty? Reach out, name your price and agree on a date for code delivery.
Communication happens on Gitter: goerli/clique

@5chdn what's the best way to contact you right now?

@thefallentree , you may try contacting https://twitter.com/MPtherealMVP in goerli related issues.


pg_receivexlog on a replica

I have configured pghoard to be backing up from a replica, but it does not seem to be working.
When I trigger a pg_switch_xlog on the master, I see the replica switching over to the new log, but pghoard does not seem to register the fact he replica has switched to a new wal.
Is it possible that pg_receivexlog is not expected to work on a replica, and I should be using archiving only?

I haven't tried running it from standby in years but it should work just fine.

Ok maybe something I am doing wrong


More bad references in guide.dox

There are some other bad cross references in guide.dox. For example:
Descriptions of the overall structure of netCDF classic and 64-bit
offset files are provided later in this manual. See Structure.

I will fix and submit a PR.

These were fixed and merged.


Powershell References

I see that there are ways to SET dynamic group membership rules via power-shell as referenced in the accompanying articles, but I can't find a Get to view the rules as the currently are. Is that something I'm missing or not yet exposed to powershell?

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

ID: d390d5ea-4f2f-3458-6b26-16d80dc1a9ee
Version Independent ID: 7beffeee-37a3-1d95-0894-d626e5bd7fdc
Content: Update and manage a dynamic group rule and troubleshoot membership - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/groups-update-rule.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@tbogle You can use the "Get-AzureADMSGroup"  command to get this information.
Ref: https://docs.microsoft.com/en-us/powershell/module/azuread/get-azureadmsgroup?view=azureadps-2.0


Run the attack, but success rate is low

Hello, I am running your attack.py as instruction, but the success rate is low, can you give some advices? Thanks

@code-roamer
make sure the l_infty_norm is large enough. please check the documentation in attack.py


Fix dependencies

I was hitting several issues with dependencies compatibility when running
tests. The updated go.mod/go.sum were working for me. The k8s
constraints were taken from Thanos repo.

Thanks!


cmd/compile: "in prove, detect loops with negative increments" breaks blackfriday tests

CL 104041 (commit 6d379add0fefcc17ed7b763078526800a3c1d705) breaks a test in blackfriday:
go test -v github.com/russross/blackfriday/ -run TestReference
=== RUN   TestReferenceOverride
--- PASS: TestReferenceOverride (0.02s)
=== RUN   TestReferenceLink
--- PASS: TestReferenceLink (0.04s)
=== RUN   TestReference
--- FAIL: TestReference (4.82s)
	ref_test.go:32:
		panic while processing ["## Unordered\n\nAsterisks tight:\n\n*\tasterisk 1\n*\tasterisk 2\n*\tasterisk 3\n\n\nAsterisks loose:\n\n*\tasterisk 1\n\n*\tasterisk 2\n\n*\tasterisk 3\n\n* * "]: block input is missing terminating newline
=== RUN   TestReference_EXTENSION_NO_EMPTY_LINE_BEFORE_BLOCK
--- FAIL: TestReference_EXTENSION_NO_EMPTY_LINE_BEFORE_BLOCK (4.80s)
	ref_test.go:32:
		panic while processing ["## Unordered\n\nAsterisks tight:\n\n*\tasterisk 1\n*\tasterisk 2\n*\tasterisk 3\n\n\nAsterisks loose:\n\n*\tasterisk 1\n\n*\tasterisk 2\n\n*\tasterisk 3\n\n* * "]: block input is missing terminating newline
FAIL
FAIL	github.com/russross/blackfriday	9.694s

I haven't looked at all at what exactly is happening, sorry.
CC @dr2chase @rasky @aclements

I dug into this. The bug comes while compiling this loop in listItem:
https://github.com/russross/blackfriday/blob/11635eb403ff09dbc3a6b5a007ab5ab09151c229/block.go#L1131-L1134
	for i > 0 && data[i-1] != '\n' {
		i++
	}

Note that the i > 0 clause is totally irrelevant -- i is increasing, not decreasing, and it doesn't start out negative. Removing that clause fixes the test.
I don't understand what precisely is going wrong in findIndVar, but somehow the extraneous clause must be convincing it that both the upper and lower boundaries are defined when clearly only the minimum is.

I looked at this some more and I think I understand what's going on, but fixing it is way above my head. A self-contained repro:
package main

func main() {
	s := []int{0, 1, 2}
	i := 1
	for i > 0 && s[i] != 2 {
		i++
	}
	if i != 2 {
		panic("loop didn't run")
	}
}

I believe the problem is that we aren't checking that min and max move in the same direction as inc. Fixing this exact case, where the boundaries are constants, is easy; just check that the sign of inc is the same as the sign of max-min. But in the more common/important case, ranging over a slice, at least one of them will not be a constant. We need external information, maybe from the facts table, to check that. So maybe the fix is in prove, while calling addIndVarRestrictions?
Separately, while reading the code I noticed that we never set indVarMinExc. That almost has to be a bug.

Certainly the induction variable conclusion is wrong (compile with -d=ssa/prove):
/usr/local/google/home/khr/gowork/issue26116.go:6:10: Induction variable: limits [1,0), increment 1 (v17)

The upper limit is wrong.  It's treating i > 0 like it were i < 0.  Somehow we have to ensure the comparison direction matches the increment direction.


Argument of type 'void' is not assignable to parameter of type 'PiniaPlugin'

Are you using Nuxt?

[ ] Check this box if you encountered the issue using Nuxt and the included module.

Describe the bug
In Vue-TS project the following give an error:
import { createPinia } from 'pinia';
import createPersistedState from 'pinia-plugin-persistedstate'
const pinia = createPinia();
pinia.use(createPersistedState({ storage: sessionStorage })); // Compile Error
app.use(pinia);

Error:
Argument of type 'void' is not assignable to parameter of type 'PiniaPlugin'.ts(2345)

I was using the following docs: https://prazdevs.github.io/pinia-plugin-persistedstate/guide/advanced.html#global-persistence-options

Reproduction
https://codesandbox.io/p/devbox/c2nvhp?file=%2Fsrc%2Fmain.ts
System Info
System:
    OS: Windows 11 10.0.22631
    CPU: (4) x64 Intel(R) Core(TM) i3-8100 CPU @ 3.60GHz
    Memory: 13.19 GB / 23.92 GB
  Binaries:
    Node: 20.10.0 - C:\Program Files\nodejs\node.EXE
    npm: 9.2.0 - C:\Program Files\nodejs\npm.CMD
    pnpm: 9.1.3 - ~\AppData\Local\pnpm\pnpm.CMD
  Browsers:
    Edge: Chromium (130.0.2849.56)
    Internet Explorer: 11.0.22621.3527

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] The provided reproduction is a minimal reproducible of the bug.

Hi, the default export is an already configured plugin, if you want to create one, you want to use the named export:
import { createApp } from "vue";
import { createPinia } from "pinia";
import { createPersistedState } from "pinia-plugin-persistedstate";
import "./style.css";
import App from "./App.vue";

const app = createApp(App);

// Init Pinia

const pinia = createPinia();
pinia.use(createPersistedState({ storage: sessionStorage }));
app.use(pinia);

app.mount("#app");

also i recommend grouping imports at the top

Thanks


[CWS] make sure we do not fetch the hostname multiple times (especially in tests)

What does this PR do?
GetHostname can be pretty slow to call because it does some gRPC with the agent, and this can fail. To improve this, this PR adds a cache layer to ensure we do it only once when possible.
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


[#28] fix language detection

the travis CI is fixed in #31

Coverage remained the same at 100.0% when pulling 1febdb4c726aae9be4d88ed5833405bb791551d0 on alemata.28.fix_language_detection into b5267e96b4f3532ff843f886067c34ac976275c5 on master.