Migrate all API endpoints to Fastify as secondary server

Discussed in https://github.com/verdaccio/verdaccio/discussions/2155
The current progress could be tested via
pnpm install
pnpm build
pnpm debug:fastify

or
npm i -g verdaccio@6-next
verdaccio fastify-server

The main idea is launch fastify with next major but under a flag, so could be tested for a period of time until replaces completely Express server.
The main blocker of this migration, the fact some important logic still remains on express endpoints, should be ported to another place were could be reused from both servers and avoid duplication and easy testing.
Steps to migrate

API (cli)

user (easy) https://github.com/verdaccio/verdaccio/pull/2301 @dianmorales
packages (high)

metadata, manifest ( medium). Requires refactor  #2750
~tarball~ https://github.com/verdaccio/verdaccio/pull/2600   @juanpicado

~dist-tags~ ( medium) https://github.com/verdaccio/verdaccio/pull/2638 @dianmorales
~ping (easy)~ @dianmorales
~whoami (easy)~ #2599
stars (medium)  #2767 @juanpicado
search (v1 only) ignore /-/all (high)  https://github.com/verdaccio/verdaccio/pull/2432 (uses undici + undici-fetch) @juanpicado
publish  (high) #2767 @juanpicado

Web API

~readme~  (easy) #2613 @dianmorales
packages  ( medium) #2750
~user~ (easy) https://github.com/verdaccio/verdaccio/pull/2624 @dianmorales
~sidebar~  (easy) https://github.com/verdaccio/verdaccio/pull/2618  @dianmorales
~login~ (easy) #2624 #2653 @dianmorales

Improvements

[ ] Apply hooks for Header Authorization validation
[ ] Apply validations, schema etc #2653
[ ] Apply require host header on some endpoints, (packages and manifest)
[ ] Improve Types (remove @ts-ignore
[ ] UserRemote requestDecodarator
[ ] Improve integration with @verdaccio/node-api, currently is not able to enable from this API fastify
[ ] Review compatibility with Express (it should remains as primary server, Fastify as secondary opt-in)
[ ] Integrate Fastify server to E2E CLI and E2E UI
[ ] Add integration basic test on fastify server package
[ ] Enable fastify via environment variable as also replace verdaccio fastify by verdaccio --fastify via CLI (this will enable Docker nightly build to use Fastify)
[ ] Document on server page the possibility to using fastify (This is the very last task to be done)

@juanpicado - how do you envision this impacting the middleware plugins? Would something like fastify-express be leveraged for compatibility?

tbh, I have no answer yet, I haven´t experienced enough with it, but the idea is extend compability longer as possible as I try to do with all features. Express won´t go anytime soon as a default server, a lot of things still to refactor 😓 .

Thanks. Maybe when we eventually migrate to v6 I'll see if we can use the fastify server and take a look at what is entailed in switching the middleware plugins to be compatible with v6. I'll open a discussion if/when that occurs and if a discussion hasn't already been started.

Thanks. Maybe when we eventually migrate to v6 I'll see if we can use the fastify server and take a look at what is entailed in switching the middleware plugins to be compatible with v6. I'll open a discussion if/when that occurs and if a discussion hasn't already been started.

No discussions open yet regarding plugin future, that´s also an open topic, my position on v6 with plugins is leave it as it is (with minor improvements) and back to it on v7.


Calcite 763

The test case would throw "cannot translate rel" exception without the fix. Now it can run through, but with result check failure because:
The plan using the materialized view is not chosen or is not even matched (not sure yet).
So shall we address this in another JIRA issue?

It's this extra sort, looks like the result with Materialization is not in the right order. But you are right, I shouldn't have removed it.


chore(deps): update terraform terraform-ibm-modules/observability-instances/ibm to v2.12.2

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/observability-instances/ibm (source)
module
patch
2.12.1 -> 2.12.2

Release Notes

terraform-ibm-modules/terraform-ibm-observability-instances (terraform-ibm-modules/observability-instances/ibm)
v2.12.2
Compare Source
Bug Fixes

added missing depends_on for s2s auth policy creation when creating ATracker route with COS target (#475) (17df111)

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

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

This PR has been generated by Renovate Bot.

/run pipeline

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


FilePond field does not seem to be removing the files on error

Laravel Form Components Version
v6.0.0
Laravel Version
v8.62.0
Alpine Version
v3.3.5
Bug description
First of all, thank you so much for this amazing package! 👌🏻💯
I'm using the FilePond field with Livewire with pretty much the same code from the demo which fires the browser event file-pond-clear if there was a validation error. However, the file is still there and I have to manually remove it before trying another file.
How can I get it removed if there is an error?

Steps to reproduce
No response
Relevant log output
No response

This may be a possible bug and I'll have to look into it more when I get the chance to.

@pktharindu
I had the same problem, apparently the condition is wrong in resources/views/vendor/form-components/components/files/file-pond.blade.php
I had to create a new listener called file-validation-single-file-error which will be executed (in my case) only when a specific file fails, so I don't need such conditions
I just added
     x-on:file-validation-single-file-error.window="
         pond.removeFiles();
         "

right before
     x-on:file-pond-clear.window="
         if (! this.wireId || $event.detail.id !== this.wireId) {
             return;
         }

         @if ($multiple)
            pond.getFiles().forEach(file => pond.removeFile(file.id));
         @else
            pond.removeFile();
         @endif
     "

and in LiveWire I replaced the old browserEvent for
$this->dispatchBrowserEvent('file-validation-single-file-error', ['id' => $this->id]);

NOTE:
apparently pond.removeFiles(); removes all files, so it would not be necessary to do a forEach and then remove the files one by one as the original listener does

I've released v8 of this package, which has slightly tweaked how filepond handles when files should be removed. It won't detect if there are validation errors however, so you'll need to clear filepond out manually if that's the desired effect. In v8 in a livewire component, you can clear it out like this:
$this->emitSelf('file-pond-clear');


Create Hotspot SVGs for Nebra

Nebra has an open PR for app integration and will need help creating Hotspot SVGs for the onboarding process.
There will be 2 Hotspots: the Indoor edition and Outdoor edition.
https://github.com/NebraLtd/hotspot-app/tree/nebra-hotspots

Nebra Icons.zip


How to use dio to request data in protobuf format

I don't know how to use dio to request protobuf format data. Can you help me？ thank you very much

I have the same question. Has dio support protobuf yet?


Respect user-supplied quoting

Currently it's impossible to have 'lazily' evaluated environment
variables like
  FOO='${BAR}'

where ${BAR} is evaluated at runtime.
This is because we strip any quotes (single or double) and replace them
with double quotes, and then ${BAR} is expanded.
Instead, we can check if the value is already quoted, and only supply
our own double quotes if it's not.

Thank you @iainlane !

@iainlane I've released version 4 with your addition.
It's been a while since I have done a github action release, so hopefully I did everything correctly 🤞


Fixed Firefox 35 not able to open local files (#588)

Tested on Firefox, Chromium (with --disable-web-security option) and IE11.

Beautiful, thank you!


UNDERTOW-1823: Add test coverage for TLSv1.3

I've created a sample change for discussion. I'm not attached to the code or config itself, just trying to provide more validation with TLSv1.3.

I've updated this PR, but it looks like a newer PushResourceRSTTestCase test fails when the client SSLContext allows TLSv1.3 (regardless of the server-side configuration). This may be a legitimate client failure, or a test issue -- I haven't had a chance to debug particularly deeply.
Would you like me to add a @TLSv13Ignore on this test to unblock everything else, allowing that investigation to occur later?


Question about the memory use

Hi,
Thank you for your wonderful work.
I have a question about the memory requirements.
The Qwen_1.5_1.8B-Chat model has 1.8B parameters. In your code, you load the model in BF16 precision. Your experiment shows that the total memory requirement is 11-12GB.
However, based on the calculations from their paper, there seems to be a significant gap between the reported memory requirement (11-12GB) and the calculated requirement (3.4GB + 0.2GB + 0.6GB).
I am not sure if this is the right place to ask, but I would like to have your help in understanding this discrepancy.
Here is a breakdown of the memory calculations:
| Parameters | Gradient(FP32) | Optimizer(FP32) | Total
-- | -- | -- | -- | --
M | 2M(FP16), 4M(FP32) | 4M/D | 12M/D | 18M
Llama2-7B, D=32 | 13.4GB(FP16) | 0.8GB | 2.4GB | 21.8GB
Qwen1.5_1.8B-Chat, D=32 | 3.4GB(FP16) | 0.2GB | 0.6GB | 11-12GB
Thx,

@yoyostudy Apologies for my late response. Tomorrow evening I have some spare time and will verify again with the script. I'll let you know the results.

@yoyostudy I did some more testruns with the notebook. When the model is just loaded into GPU memory it uses around 4.6 GB of GPU RAM.
When I start training the model - with max sequence length of 1024 and a batch size of 2 - the used GPU Memory hovers around 10 GB.
I don't think the values are a discrepancy between the paper and reality. The memory calculations they provide are general guidelines I presume but the real values can slightly change based on different model architectures. When actually training the model the memory consumption will increase depending on the used batch size.
I will make some minor updates in my notebook and the description provided and will publish those in the coming days.
If you still have some doubts about the values you notice when training with it I would suggest to contact the researchers that created the BAdam method.


Template pipeline

Should use mapping file to create pipeline

Opting to use a map for the relationship map, will abstract to a file later.


Feature: Added connection state check

Relates to #193.  Added method that exposes underlying amqp.Connection.IsClosed() function in both Conn and ConnectionManager

@wagslane thoughts?


”Stolniceni-Prăjescu”, ”Brătești”, ”Cozmești” are listed in the drop-down menu

Description:
When user writes ”Stolniceni-Prăjescu”, ”Brătești”, ”Cozmești” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Stolniceni-Prăjescu"

Expected results
The location appears in the drop-down menu as "Sat Stolniceni-Prăjescu, IAȘI (Stolniceni-Prăjescu)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Brătești"

Expected results
The location appears in the drop-down menu as "Sat Brătești, IAȘI (Stolniceni-Prăjescu)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Cozmești"

Expected results
The location appears in the drop-down menu as "Sat Cozmești, IAȘI (Stolniceni-Prăjescu)".


Perhaps typedef RequestInfo should include TrustedURL

Perhaps RequestInfo should be @typedef {!Request|string|!TrustedURL}
/**
 * @typedef {!Request|string}
 * @see https://fetch.spec.whatwg.org/#requestinfo
 */
var RequestInfo;

https://github.com/google/closure-compiler/blob/v20190709/externs/browser/fetchapi.js#L145

On further investigation, the above suggestion is not currently applicable.

for now fetch() is not guarded

28 May 2019, https://github.com/WICG/trusted-types/issues/176#issuecomment-496533726


allow to run smug inside tmux session without nesting

if I run smug from inside a session I get sessions should be nested with care, unset $TMUX to force

v0.1.2 is released now and this should be fixed. feel free to reopen or create another issue if you have any problems 🙂

Tried it again, but still the same message. Also tried it with --attach.
➜  ~ smug start poppygo --attach
Starting a new session...
sessions should be nested with care, unset $TMUX to force

Please make sure you have the latest version. The best way is to delete an old one & reinstall

If it doesn't help, try to run with --debug option and then paste content from the ~/.config/smug/smug.log

Sorry, did not look right. It now creates the session correctly. It complains when the command tmux attach -d -t poppygo: is executed. Could it be possible to run tmux switch -t poppygo when the environment var TMUX is set?

It should switch if you provide the —attach flag.
I will test it again tomorrow’s morning.
On Wed, Dec 30, 2020 at 00:56, Pim Snel<EMAIL_ADDRESS>wrote:

Sorry, did not look right. It now creates the session correctly. It complains when the command tmux attach -d -t poppygo: is executed. Could it be possible to run tmux switch -t poppygo when the environment var TMUX is set?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.

@mipmip can you please update to the latest version and send the output of those commands:
$ smug --help
When inside tmux session:
$ echo $TERM
And also the OS you're using.

I'm using void-linux on x86_64.
➜  ~ echo $TERM
xterm-256color

➜  ~ smug --help
Smug - tmux session manager. Version v0.1.4

Usage:
        smug <command> <project> [-w <window>]... [--attach] [--debug]

Options:
        -w List of windows to start. If session exists, those windows will be attached to current session.

Examples:
        $ smug start blog
        $ smug start blog:win1
        $ smug start blog -w win1
        $ smug start blog:win1,win2
        $ smug stop blog

The previous version worked better for me. The last command in the debug-log kills the just created session.
➜  ~ smug start poppygo --debug --attach
Starting a new session...
sessions should be nested with care, unset $TMUX to force
Oops, an error occurred! Rolling back...
Cannot run "tmux attach -d -t poppygo:". Error exit status 1

➜  ~ cat  ~/.config/smug/smug.log

tmux has-session -t poppygo:
exit status 1
tmux new -Pd -s poppygo -n poppygo -c /home/pim/cPoppyGo/poppygo-eletron-app
tmux send-keys -t poppygo:poppygo vim ./ Enter
tmux select-layout -t poppygo:poppygo main-horizontal
tmux attach -d -t poppygo:
exit status 1
tmux kill-session -t poppygo

@mipmip thanks, I've updated the code to use the TMUX variable as well, to detect the tmux session. Please, upgrade and let me know if that works for you.
_, tmux := os.LookupEnv("TMUX")
insideTmuxSession := os.Getenv("TERM") == "screen" || tmux

Yes, it works! Thanks! Finally a fast dependency-free tmux-session-manager.


Log uncaught global errors.

/botio test

/botio test

/botio test

Do we still need this PR now that the bots are stable again?

Thanks


Audio widget uses numpy array and sample_rate  as input

Fixes the issue : https://github.com/pyviz/panel/issues/351

Thanks again @Karamya, much appreciated!


Add support for parsing cgroup

We've encountered a case where we needed to know wether a particular process was running inside a container or not. One way to achieve this is to check for cgroups inheritance from the container runtime via /proc/<pid>/cgroup.
It would be great to have visibility on the hierarchical dependencies per cgroup resource controller.

Happy to take this one on. I already have a bit of related code+tests from my work on systemd-exporter that I was hoping to upstream. I may need some help with building the best possible API in go, so if anyone would be willing to act as a sounding board please comment here and I will tag you when there is a specific code question. (As an example - cgroups v1 and v2 APIs differ. Should we try to expose one master cgroups API, or two specific APIs?)
At the least, I can at least knock out the bulk of 'setup the fixtures, parse files into structs' boilerplate to get this moving


Show stats on Streamer page

Right now we show the stats on the Receiver page -- but this information would be a lot more useful on the Streamer page, where we are potentially doing P2P streaming to multiple different users.

I can work on this!

I started implementing this but ran into problems when trying to access "stats.video.recv" because the streamer is not receiving the data from the video feed. When #92 is solved this will be a lot easier to implement.
Other than that the stats code from streamer and receiver are likely to be very similar so it might be beneficial to make this a component.


[github-pr-review] Mark outdated review comments as obsolete or delete such comments

With github-pr-review, if a reported line is changed to fix the reported problem, GitHub marks the comment as outdated. However, if the fix doesn't change the reported line, the previously reported comment still remains as is.
github-pr-review reporter fetches reported comments first to prevent posting the same comments, so it should be possible to detect which comments are fixed by comparing the new reported results.
It should be better to resolve fixed reported results (or mark them as obsolete or delete such comments?).
IMPACT: This problem can happen  for type check based on comment. e.g. phpstan, jsdoc type check of Typescript.
Just for reference  (in Japanese though): https://twitter.com/about_hiroppy/status/1258699418796097537

Note that we may want to implement similar feature to gitlab-mr-duscission reporter as well.
Also, it should be no problem with github-check (and github-pr-check) reporters because it always reports all results to a specific commit hash, not as PR comments.

I don't plan to work on it immediately, so please feel free to take this issue if you're interested in it. cc/ @reviewdog/reviewdog

It's quite nice to delete linting comments to avoid leaving long trails of old PR activity. We've been using houndci.com for linting and when they switched from resolving to deleting comments our PR's got a whole lot cleaner. Maybe this behaviour needs to be configurable since I don't know if deleting makes sense for all kinds of reviewdog checkers?

GitLab has the idea of resolve thread or discussion. That would be great to see. Not to delete the discussion, but to make it resolved.
Having this cleaned up for fixed issues would be awesome!

As far as I checked quickly, the REST API has no method to resolve review comments; only the GraphQL API has.

REST: https://docs.github.com/en/rest/reference/pulls
GraphQL: https://docs.github.com/en/graphql/reference/mutations#resolvereviewthread

reviewdog currently using REST API, so we can't mark review comments as "resolved".
The only choice currently available seems to be deletion.

To elaborate on the deletion behaviour of HoundCI: It only deletes comments if there was no follow up. So "discussions" would be left in place. In my experience it's quite uncommon for discussions to occur on linting comments though.

Any updates on this? I'm moving from CodeClimate to reviewdog + GH actions and I'm trying to decide between github-pr-review or github-pr-check.
github-pr-check is nice in that it marks the build as a failure, and once you resolve linting issues locally and push again, the stale annotations are removed and the build is now a success. The downside here is there's no way to manually dismiss an annotation (akin to CodeClimate's options for "invalid" or "wontfix" for certain warnings).
github-pr-review allows you to manually resolve a warning, but because stale comments aren't deleted or resolved, it kinda floods the PR with comments.
I'm thinking github-pr-review is the lesser of the evils here, but it'd be nice to have the best of both worlds.

https://github.com/Ardiannn08/resolve-outdated-comment

Not a fix for reviewdog in general. But for people using reviewdog for running Rubocop in particular, we made an alternative GitHub action which does proper deletion of outdated comments: https://github.com/reclaim-the-stack/rubocop-action
NOTE: This is cleaner than "resolving" comments since it fully clears the PR history of linting noise.


Add l10n community mktg kit to misc folder

create the new folder for web projects.
add l10n community mktg kit to "misc" folder.

I'll check the content next week, in the meantime:

This content needs to be linked in the main README.md and SUMMARY.md, unless you want it quite hidden (like the doc about Pootle)
No colons in titles

I would really suggest to create one branch (and pull request) for topic/issue/document.
Can we use this one for the marketing kit alone?


Version 16.0.0 The given test sources target multiple frameworks/platforms that are incompatible

Description
WE have both .net core and standard tests.
in version 15.9.0 we could run them in a single VSTest task,
Now, in 16.0.0 it fails with:
The given test sources target multiple frameworks/platforms that are incompatible. Please make sure the sources target the same framework and platform.
In VS , running all the tests also works.
We would expect this to work in VSTS also.

We don't have multiple targets; all of our projects are currently targeting .NET Framework 4.7.1, and we're also seeing this issue. We do have the .NET Core tools installed on the server, and other projects in other solutions do use them for what it's worth.

I started getting this error today as well. My solution has a single project:
<TargetFramework>net462</TargetFramework>
<RuntimeIdentifier>win7-x86</RuntimeIdentifier>

My unit tests are UI tests (Selenium), so the code it is testing is running on a different server.
If I force version 16.0.0-preview-20190203-03, then my tests run as expected. Only when targeting 16.0.0 do I get this error.
Looking back at previous logs, I do see output stating which assemblies are not compatible with my chosen framework, but this was information text (not even a warning):

Test run will use DLL(s) built for framework .NETFramework,Version=v4.6.2 and platform X86. Following DLL(s) do not match framework/platform settings.
Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.Interface.dll is built for Framework 4.5 and Platform AnyCPU.
Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.dll is built for Framework 4.5 and Platform AnyCPU.
Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.dll is built for Framework 4.5 and Platform AnyCPU.
Microsoft.VisualStudio.TestPlatform.TestFramework.dll is built for Framework 4.5 and Platform AnyCPU.

I tried to update my project to net45, but I have dependencies that only work with .NETStandard 2, so I can't do that. I also can't upgrade to .netcoreapp2.0 because the version of the TestContext that I get with .NET Standard doesn't allow me to add attachments.
For now I can work around by forcing the most recent preview version of the tools.

Since Yesterday we have the same issue :
Visual Studio Test Platform Installer 1.*  on latest
Visual Studio Test : Installed by Tools Installer

The given test sources target multiple frameworks/platforms that are incompatible.
##[error]VsTest task failed.

All projects are in .net 4.7.
For now forcing the Installer on Specific Version 15.9.0  solved the issue.

We have a breaking change in 16.0.0 regarding running tests targeting multiple framework dlls and incompatible target framework given by the user which is mentioned in the release notes here.
According to the associated changes, (1) multiple dlls targeting different frameworks (version irrespective) will throw a framework/architecture incompatible error.
Explanation : if there is one dll targeting .NETFramework and the other targeting .NetCore or vice-versa, the run would be aborted. Earlier we used to spit out warnings and run tests from one of the target dlls (skipping the other dlls).
The scenario where all dlls are of either .NetFramework or .NETCore (even if versions are different) should work seamlessly as before.
(2) The other change we made was that if a dll is .NetCore and user explicitly sets /Framework to .NETFramework or dll is .NETFramework and /Framework is set to .NetCore(i.e. frameworkcore10), this gets aborted due to mismatch between given dlls and user provided input.

@bogdancucosel Can you please check if your scenario is the one I mentioned first? That is you are targeting both .NetFramework and .NetCore test dlls which would break?
@shortstuffsushi This shouldn't break if all projects are targeting .NET Framework 4.7.1. Can you please check if your scenario is the one I mentioned first? If not, can you please provide a sample repro project?

@vagisha-nidhi yes, you are right. It seems we have misread the test result before. We now have 2 VSTest tasks, targeting each framework individually , and everything works as expected. (we were also confused because the release notes were not posted together with the 16.0.0 release ). As far as we are concerned, the issue can be closed.
Thank you very much,
Bogdan

@vagisha-nidhi I don't believe the explanation you provided addresses my issue. I provided details in my first post on this issue.
Short version: My tests target .NET 4.6.2. I do have some dependencies that target .NET Standard (NOT .NET Core).

I am also seeing this issue, and my tests target .NET 4.7.2, and all dependencies target versions of .NET Framework except one which targets both .NET 4.7 and .NET Standard.

The change we made for aborting test run when there are multiple targets was failing on CI because the vstest.console task was picking up Adapter dlls(present in output folder), that is, PlatformServices.dll and PlatformServices.Interface.dll, of which one target .NetPortable and the other .NetStandard.
We have reverted the change and released Test Platform 16.0.1 with the revert.
Can you please check if you are still facing the issue with 16.0.1?

My configuration was similiar to @mmyk (only .NET 4.7). Using version 16.0 failed with the reported error message.
Using version 16.0.1 solved it.

16.0.1 resolve my issue as well.

@vagisha-nidhi Using 16.0.1 resolved the issue for me.

Closing the issue as 16.0.1 worked.

I am still facing this issue with version 16.0.1.
`Microsoft (R) Test Execution Command Line Tool Version 16.0.1
Copyright (c) Microsoft Corporation.  All rights reserved.
vstest.console.exe
"D:\a\r1\a_PEG UI Tests Build\drop\E2E_test_automation\bin\Release\E2E_test_automation.dll"
"D:\a\r1\a_PEG UI Tests Build\drop\E2E_test_automation\bin\Release
\Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.Interface.dll"
"D:\a\r1\a_PEG UI Tests Build\drop\E2E_test_automation\bin\Release\Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.dll"
"D:\a\r1\a_PEG UI Tests Build\drop\E2E_test_automation\bin\Release\Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.dll"
"D:\a\r1\a_PEG UI Tests Build\drop\E2E_test_automation\bin\Release\Microsoft.VisualStudio.TestPlatform.TestFramework.dll"
/logger:"trx"
/TestAdapterPath:"D:\a\r1\a"
Starting test execution, please wait...
Test run will use DLL(s) built for framework .NETFramework,Version=v4.0 and platform X86. Following DLL(s) do not match framework/platform settings.
E2E_test_automation.dll is built for Framework 4.6.2 and Platform AnyCPU.
Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.Interface.dll is built for Framework 4.5 and Platform AnyCPU.
Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.dll is built for Framework 4.5 and Platform AnyCPU.
Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.dll is built for Framework 4.5 and Platform AnyCPU.
Microsoft.VisualStudio.TestPlatform.TestFramework.dll is built for Framework 4.5 and Platform AnyCPU.
Go to http://go.microsoft.com/fwlink/?LinkID=236877&clcid=0x409 for more details on managing these settings.`

@nmfernandes
have you solved this issue?

Although those messages about "DLL(s) do not match framework/platform settings" keep being displayed, the tests run as expected, so after all this is not an issue for me.

@nmfernandes can you see then the results of your tests? It seems there is no test available cause of the platform incompatibility

@anaprikho, and are you sure that the platform incompatibility is the reason for which the test results are not being displayed?


ARM-support for Cartographer

Description of problem
Cartographer doesn't install on arm-based OS today. This means that M1 users especially cannot use Cartographer.
Proposed solution
Given I want to use Cartographer and I am on an M1 laptop
When I install Cartographer on my Kubernetes cluster
Then it installs successfully

Example
Additional Context
This also enables the story of TCE users being able to use Cartographer in TCE on arm-based operating systems.

@idoru to recreate on his M1.
Any details about what's not working @kartiklunkad26

After this discussion https://kubernetes.slack.com/archives/C02GY94A8KT/p1655396822733659
I feel it's best to frame this as supporting ARM architectures on cloud platforms, those which lack a compatibility layer such as Rosetta.


Charlie McKenzie Joins Harvest Connect - Harvest Connect

Georgia Cannabis Pioneer
Chief Host at Crop Talk Media
Founding Partner at CropWalk LLC
Chief of Ops at Harvest Connect

Cannabis Industry best kept secret y'all!

Source URL:
https://www.myharvestconnect.com/2020/03/meet-charlie-mckenzie/
BrowserChrome 84.0.4147.68OSWindows 10 64-bitScreen Size2560x1080Viewport Size2560x888Pixel Ratio@1xZoom Level100%

charlie-mckenzie-harvest-connect-georgia-cannabis-pioneer.pdf


How to wait for .flushEvents() to finish in AWS lambda function

Hi, I'm trying to use the SDK to send events on demand (not using batch)
But the function ends without executing the AmplitudeCallbacks
Here is my code:
public class AmplitudeAPI {
    private static final Logger log = LoggerFactory.getLogger(AmplitudeAPI.class);
    private Amplitude client;
    private AmplitudeApiAwsConfiguration awsConfiguration;

    public AmplitudeAPI() {
        client = Amplitude.getInstance();
        awsConfiguration = new AmplitudeApiAwsConfiguration();
        AmplitudeSecret secret = awsConfiguration.getAmplitudeSecret();
        client.init(secret.getApiKey());
    }

    public void sendEvent(String eventType, String userId, JSONObject eventProperties, JSONObject userProperties) {
        Event event = new Event(eventType, userId);
        if (eventProperties != null) {
            event.eventProperties = eventProperties;
        }
        if (userProperties != null) {
            event.userProperties = eventProperties;
        }

        AmplitudeCallbacks eventCallbacks =
                new AmplitudeCallbacks() {
                    @Override
                    public void onLogEventServerResponse(Event event, int status, String message) {
                        log.debug(String.format("parameters sent " + eventType + userId + eventProperties + userProperties)); // In the logs this line doesn't execute
                        log.debug(String.format("Before if Status: %d, Message: %s", status, message));
                        if (status == 200) {
                            log.debug(String.format("Status: %d, Message: %s", status, message));
                        } else {
                            log.error(String.format("Failed to send event to Amplitude Status: %d, Message: %s", status, message));
                        }
                    }
                };
        log.debug("About to send event");
        client.logEvent(event, eventCallbacks);
        log.debug("Flushing...");
        client.flushEvents();
        log.debug("Finished flushing"); // In the logs it reaches this line
    }

Update: It seems that after 30 minutes it fails with this message "Status: 400, Message: Request missing required field"

cc: @bohan-amplitude

Update: The api key was incorrect, but after fixing it and trying again it didn't post an event in Amplitude or log an error after 50 minutes
I'm trying to solve it using http api but i'd like to know what happened

Hi @dario-jahnel ,
The amplitude client with default configuration usually sent out events in 10 seconds. Even with errors and retries, it won't take 30 minutes to trigger callback. Did you customize the config like client.setEventUploadThreshold() or client.setEventUploadPeriodMillis()? Can you use client.setLogMode(AmplitudeLog.LogMode.DEBUG) to collect some debug info and provide more detail? Thanks very much.


Problem specifying threads to be used

How to config madmax to use specific threads? For example, I have 12 threads and want it to skip 0 and 1. If I run -r 10 it will use 0-9

That's something your OS has to handle / you have to handle via your OS


add pink color variable for aponjon stories

@peterp can you please review
There's styles cleanup and fixed indentation.
Added the banner

Added pink section and updated footer page and footer nav

👍


Offset option

Hello! Thanks for a good tool for downloading laracasts series!
Is there an option or a way to set offset for downloading a series?
For example i have to download a series starting from second or third lesson.
Thanks.

Hi,
That option does not exist yet.
I don't have the time to do it now, but i accept pull requests :smiley:.


As a user, I would like controlled vocabularies in Figgy Ephemera to function like authorities in the online catalog .

If I have a controlled vocabulary in a geo-subject, for example, like India--Gujarat, I would like to be able to select EITHER India or Gujarat from the link in DPUL. Currently the only way to have both items searchable via a link is to have 2 separate entries in figgy:
India--Gujarat (This would be the ideal construct, if we move forward in this way)
India
This use case has come up with Ellen and David M., in reference to the SAE materials. However, it would benefit all of the ephemera, if selectors would like to be more granular and include state-level information for items, where possible.  This would also clean up the controlled vocabs for specific area studies using ephemera projects.
Sample of what is envisioned (from online catalog):
Ethnology—India—Nāgāland [Browse]
See: https://catalog.princeton.edu/catalog/12245799

This came up in the recent ephemera curators meeting. The data entered in figgy is in a different format than the data in the catalog. In figgy you would see "Gujarat (India)" as a geographic subject. The example given from the catalog is a 650 subject term in LCSH format, with the dashes. So these examples aren't really analogous in either format or field type.
Notes from that meeting: https://docs.google.com/document/d/1ZO6ghfgL5VbrSUFI7Wy8zAIsy5V2s_UMQMTaLE9z0j8/edit#
The geo subjects are coming from the Geo Names vocabulary. More discussion needed to specify this ticket.

Keep using Geo Names?
Keep this format?
Rely on this Geo Names format to index the terms differently in DPUL?
Query the geo names API at index time to pull in geographically broader terms?


No deepslate ores

Bug Description
The bot only checks for normal ores, not the deepslate ores.
Expected Behavior
The bot checking for both deepslate and normal stone ores.
Actual Behavior
only checks normal stone ores, not deepslate

I'm unable to replicate this, it seems like the bot handles deepslate ores when running things like get iron_ingot or get diamond. Is there a reproducible command or video you can show where this happens?

I think this was probably on the release and not nightly (yay me for not keeping things up to date), I believe this has been fixed.


Sliding / dropping buffer

Thanks for this awesome library. Personally, I've found it to be the most friendly csp implementation for JS and enjoy using it.
Is it already possible to have a channel with a sliding or dropping buffer? If not, any plans to add it?
I haven't seen anything mentioned in the docs and didn't dig too deep into the source code yet, so apologies if this is already mentioned somewhere.
Thanks!

Thanks!
It's certainly on the wishlist. I had a partial implementation at one point during early development and opted to drop it because it was adding unnecessary complexity during the development cycle, but I definitely kept it in the back of my mind while developing the core functionality of the library.
Now that the spec has started to settle and a lot of the internal complexity is hammered out, it may be a good time to revisit these sometime soon. Or, as always, pull requests are welcome if anyone else would like to take a swing at it.
So the tl;dr is -- not yet, hopefully some day soon!

Awesome, thanks for the quick response! Depending how urgently I need it,
I'll take a crack at it.
On Apr 10, 2016 9:54 AM, "dvlsg"<EMAIL_ADDRESS>wrote:

Thanks!
It's certainly on the wishlist. I had a partial implementation at one
point during early development and opted to drop it because it was adding
unnecessary complexity during the development cycle, but I definitely kept
it in the back of my mind while developing the core functionality of the
library.
Now that the spec has started to settle and a lot of the internal
complexity is hammered out, it may be a good time to revisit these sometime
soon. Or, as always, pull requests are welcome if anyone else would like to
take a swing at it.
So the tl;dr is -- not yet, hopefully some day soon!
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/dvlsg/async-csp/issues/16#issuecomment-208015100

Of course! If you beat me to it, take a look at the data structures. I tried to intentionally abstract away all of the required data interface methods (full(), put(), etc) so it would be easier to integrate with additional data types. There is definitely some dead code / unused classes in there (still on my todo list for cleanup), though. Just a heads up.
I'm sure there's still something else that will need to be refactored, though, once we really dig into it.

Alright, turns out this isn't too bad (for the most part). I have a DroppingBuffer complete locally.
SlidingBuffer is more or less complete. Getting a SlidingBuffer to work with the unbounded unshift (for the multiple-value-transforms) is fairly complicated, though.
Here's a quick rundown of the issue. Assume we have the following:
let buffer = new SlidingBuffer(3); // size of 3
let transformer = (x, push) => {
    push(x);
    push(x + 1);
    push(x + 2);
};
ch.put(1);
ch.put(4);
ch.put(7);
// current state => Channel(1, 4, 7)
await ch.take(); // => 1
// should Channel hold (2, 3, 4, 7)?
// or should it hold (2, 3, 7)?
// what happens if we ch.put(10) here?
// should it end up outputting (2, 3, 4, 7, 10)?
// or should it output (3, 4, 7, 10)?
// or maybe even (4, 7, 10)?

I have committed an implementation of DroppingBuffer and SlidingBuffer to the development branch. DroppingBuffer should be fully functional, but SlidingBuffer won't work with multi-return-value transforms until we figure out / decide how they should actually function (if at all).


[Do not merge] feat: documented invokable scripts within tasks functionality in the CLI

This PR documents the new flags available to the create and update subcommands for tasks in the CLI tool. PRs implementing these new features are here: https://github.com/influxdata/influx-cli/pull/432, https://github.com/influxdata/influx-cli/pull/433.

[x] Signed the InfluxData CLA
(if necessary)
[x] Rebased/mergeable

Sorry I totally broke my branch with a force push, addressed feedback and fixed the branch here: https://github.com/influxdata/docs-v2/pull/4295


NTR: cell integrity signalling rename cell wall integrity MAPK cascade, obsolete stress grouping term

Please provide as much information as you can:

Suggested term label:

"cell integrity signalling" OR "cell integrity pathway"
(102 PubMed results)

Definition (free text)

The series of molecular signals regulates  multiple biological processes including  cell wall maintenance  (in yeast), cytokinesis, vacuole fusion, and ionic homeostasis by fine tuning  the  cell integrity MAPK cascade** .

Reference, in format PMID:#######
PMID:24498240
PMID:34198697

Gene product name and ID to be annotated to this term

All
https://www.pombase.org/term/GO:0000196
(note that I intend to change the name of
https://www.pombase.org/term/GO:0000196
cell wall integrity MAPK cascade  to cell integrity MAPK cascade (cell wall is too restrictive, because it also covers plasma membrane integrity, I will put this in another ticket)

Parent term(s)

intracellular signal transduction (GO:0035556)

Children terms (if applicable) Should any existing terms that should be moved underneath this new proposed term?

https://www.pombase.org/term/GO:0000196
cell wall integrity MAPK cascade

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)

Cross-references

For enzymes, please provide RHEA and/or EC numbers.

Can also provide MetaCyc, KEGG, Wikipedia, and other links.

 Most of these have only
intracellular signal transduction (GO:0035556)

Can we adjust the definition and/or label of cell wall integrity MAPK cascade?

It sounds like it should be is_a MAPK cascade. Please have a look at the definitions in the branch, @hattrill and I tried to be a lot more specific about the component of the signaling cascade/pathway, otherwise it's my end up being annotated for proteins somehow upstream of the pathway.

I would love that , but my understanding is that the MAP kinase cascades begin with the MAP kinase.
I am trying to capture these upstream components of the CIP pathway.

If I could annotate these directly to "cell wall integrity MAPK cascade" that would be OK
(note that I don't want to annotate these as "positive regulation of MAPK cascade" I want to annotate them as part_of the pathway)

i.e the definitiions states
" It starts with the activation of an upstream MAP4K, and continues with the consecutive activation of a MAP3K and a MAP2K."

cell integrity signalling begins with Rho, proceeds through protein kinase C and the MAP kinase cascade.
I want to:
i) avoid annotating a single pathway steps  to different terms, and have a collective term for all steps (under signalling)
ii)  capture the direction of regulation by relations between activities

I think you are saying that I annotate all to "positive regulation of cell wall integrity MAPK cascade"
I can do that.
I still want to revise the label and definition of cell wall integrity MAPK cascade to
"cell integrity MAPK cascade" which is more correct  (and update the definition, and remove the regulation of cell wall parentage,  downstream regulation need to be added as concurrent annotations)

The  other issue is thet MAPK are all defined as "stress-activated" and have a "stress activated parent". However
"The evolutionarily conserved MAPK signaling pathways orchestrate multiple and fundamental cellular functions in eukaryotic organisms that assure cell proliferation and survival during unperturbed conditions and in response to environmental stimuli. "

stress-activated can probably go under stress (not sure)
but CIP   is not always stress  (although some inputs are stress activated)

Hi @ValWood Removing stress-activated MAPK sounds like a good idea - we look at this before and moved some of the MAPK cascades out from under it. I will make a note to look at this term with Pascale again, as there are some direct annotations to this term but I need to remind myself why we still have it. Think if CIP is moved from under it, then even more reason to question its existence.
As for the intended use of 'cell wall integrity MAPK cascade' then only the MAPKs and the scaffolds should be annotated with this term, to keep it inline with other cassettes. For all components, a separate 'encompassing pathway' term which begins with a Rho and ends with XXXX would fit the pattern of the other MAPK-containing pathways. Are there other pathways where the cassette is used?

I see, I thought p38MAPK was the canonical stress cascade, but that isn't even under stress-activated. In that case I think it should go.
So in summary, actions

Obsolete  GO:0051403   stress-activated MAPK cascade grouping term

Should we also obsolete "GO:0031098  stress-activated protein kinase signaling cascade"
even if a pathway is called stress-activated, that doesn't mean it is only stress activated?

Relabel "cell wall integrity MAPK cascade" to "cell integrity MAPK cascade"

Tweak the definition of "cell integrity MAPK cascade"

Existing def is OK but could add
The cell integrity MAPK cascade regulates multiple biological processes including cell wall maintenance (in yeast), cytokinesis, vacuole fusion, plasma membrane integrity and ionic homeostasis.
How close is this def to ERK1/2  ? (see 5)

Note that the core effector of the CIP is MAPK Pmk1, an ERK1/2 ortholog which makes sense, so these are orthologous signalling pathways. ) We should probably really consider merge and call it  ERG/cell integrity MAPK cascade? but I can ignore that for now.

I have  annotated upstream of  CIP MAPK to positive  regulation...

I just re read this

For all components, a separate 'encompassing pathway' term which begins with a Rho and ends with XXXX would fit the pattern of the other MAPK-containing pathways. Are there other pathways where the cassette is used?

So we should have the encompassing pathway.
I am not aware of other pathways which use this cassette.
At least , I think they all go through protein kinase C pck2.....

@ValWood I think that if you are ok with "I have annotated upstream of CIP MAPK to positive regulation..." then we don't have to make an encompassing pathway.
This means that the "cell integrity MAPK cascade" still fits in the definition of the "MAPK cascade" even if it is a bit of an 'special case' in that it is only used in that context it does mean that it is useful for those wanting to see all the types of MAPK cascade units. Which I think, on balance is the stronger argument here.
So, unless you really needed an encompassing pathway, which from the above comment you don't, then let's not make one.

As for " GO:0031098 stress-activated protein kinase signaling cascade" - there are some direct annotations to this term, but looks like it could/should go.
Same for GO:0051403 stress-activated MAPK cascade .
Also, cell integrity MAPK cascade has a part_of rel to 'cell wall organization or biogenesis' which based on "The cell integrity MAPK cascade regulates multiple biological processes including cell wall maintenance (in yeast), cytokinesis, vacuole fusion, plasma membrane integrity and ionic homeostasis." should be removed.
On point 6, Pascale and I talked about this before and decided against merging these as we thought making the functional distinction makes sense from the perspective of users. So, I would keep cell integrity MAPK cascade separate.

OK, I had noted the incorrect parent, but forgot to include it in the list!

Updated the action items

see also https://github.com/geneontology/go-ontology/issues/26810


Pick a next class to tackle, after Message, do cdefs, start fixing tests, ....

transport #25
ssl
connect
engine


Add statistic settings to region form again

Short description

Add statistic settings to region form again
Resolved issues

Fixes: #1471

I am not really sure how and why there have been stripped from the template

Actually, it was my own fault, when I redesigned the region form in #1412, I initially wanted to use a separate box for statistics, but then I didn't like it and removed it, but apparently forgot to add the fields to the original box again :see_no_evil:


Change default Android theme color meta tag

Before
After

Don't have a strong opinion on that, so LGTM

-- File Changes --
M conf/app.ini (2)
M modules/bindata/bindata.go (1922)

I think bindata changes should be avoided as that file should not
even be in the repository.

There are only changes to a single file. Beside that this change needs an update of the bindata to take an effect

I don't really care about the android theme, so LGTM

LGTM


version: post-release bump

Signed-off-by: Jiri Kuncar<EMAIL_ADDRESS>

Coverage remained the same at 99.571% when pulling 68816c4ba20512f3f9a30fa9baf6baeca430f174 on jirikuncar:version-bump into 5aa73e4a5001739e68651b90d6630d5f5822d92a on inveniosoftware:master.


insertCSS/executeScript option?

looks like insertCSS/executeScript option were missed?
when I used "loadstop", the alert can be executed, but the insertCSS/executeScript not working, I have the files in the "www" folder.
.addEventListener('loadstop', function() {
alert('load stop');
ref.insertCSS({file: "css/style.css"});
ref.executeScript({file: "js/test.js"});
});
plz advise!

These two methods were inherited from InAppBrowser. I have made no improvements on those so far.
As far as I can tell, file path is relative to the URL you opened, not your local www directory. So for example if you loaded up http://apache.org, {file: 'js/test.js'} will actually be linking to http://apache.org/js/test.js. In order to link to local files, you have to use the file:/// protocal. While this is pretty straightforward on Android, unfortunately this does not work well on iOS, where the URL to a local file looks like this file:///Users/initialxy/Library/Developer/CoreSimulator/Devices/EB775856-C9A6-408D-AC40-31469FF5623B/data/Containers/Bundle/Application/C6A211E9-F074-4BEA-944A-20839E50F168/Test.app/www/js/test.js, which is actually device specific. So in order to support loading local files with executeScript and insertCSS there will need to be some native improvements especially for iOS.

thx! this totally clear for me ;)

can you give an example on how did you used the {code: ...} ?
i'm sorry i'm new to ionic ans i can't seem to figure out how to use this .insertCss({code: ...})
i'm using it to change the size of the title options
hope you can help me
thanks in advance


⚠ Bofrost.de has degraded performance

In ecd2b68, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1042 ms

Resolved: Bofrost.de performance has improved in ce48166.


SOLR - add tests using Docker image

Similar to what we have for ES, the tests could spin a Docker container for SOLR and check that documents are indexed as it should be.

See https://www.testcontainers.org/modules/solr/

Initial code in https://github.com/DigitalPebble/storm-crawler/tree/851
Work in progress

@mvolikas since you use this module, any chance you would be able to have a look at this one? Or any of the other issues for SOLR?

Sure, thx! I was actually having a look earlier today :)

Sure, thx! I was actually having a look earlier today :)

That would be fantastic!

Hi there!
I have opened a PR to branch 851. I fixed the StatusBoltTest and added a very simple IndexerBoltTest (just checks if the tuple is acked). Note that I used a slightly different approach: Now the solr/cores/ configuration files are used for the tests, and the cores are created explicitly. For the Indexer, I could also add a query to Solr that checks that the document is actually in the core. What do you think?

Fixed by #1240


CDS 2022 Value Set Updates

Summary
Updates our Cancer Disease Status code lookups to include 2 new codes added in the STU2 Condition Status Trend Value Set.
New behavior
The extractor now supports displayText mappings for the two new requested codes in the aforementioned VS.
Code changes

New codes added to the CDS lookup file.
Code System check added to the CDSExtractor to ensure the proper system is used for the two additional codes.
CDSExtractor test fixtures updated to test the new changes.

Testing guidance

Ensure unit tests pass.
Run the extractor with the two new codes (USCRS-352236 or USCRS-352237) in the diseaseStatusCode column of the cancer-disease-status-information csv and the diseaseStatusText column blank. The resulting JSON should include the new code, the corresponding display, and the corresponding system.

@julianxcarter This looks like it's gonna merge into main, is there a reason for that or should we switch that to develop?

@julianxcarter This looks like it's gonna merge into main, is there a reason for that or should we switch that to develop?

It should absolutely go into develop, thanks for catching that!


inline-completions example is not work

The inline-completions  example is not working. I was able to run this demo successfully last month, but it stopped working after the recent update.

I get the following error:
Activating extension 'vscode-samples.inline-completion-sample' failed: Extension 'vscode-samples.inline-completion-sample' CANNOT use API proposal: inlineCompletionsAdditions.
Its package.json#enabledApiProposals-property declares:  but NOT inlineCompletionsAdditions.
 The missing proposal MUST be added and you must start in extension development mode or use the following command line switch: --enable-proposed-api vscode-samples.inline-completion-sample.

Any idea how to make it work?

I got the same issue here but it's my first time trying this completion API. I copied the whole thing and still can't get it work.

yes，the inlineCompletion example is out of date，this api is now official already。i fixed the compile problem，but  i found the inline completion not work sometimes, very weird。
U can use registerInlineCompletionItemProvider this api  as common.
i hope examples and api  description may more clear and concrete later.

@brookylin , it seems it should be solved by adding this additional parameter --enable-proposed-api your-extension-identifier to your launch args:
"args": [        "--extensionDevelopmentPath=${workspaceFolder} --enable-proposed-api your-extension-identifier",        "${workspaceFolder}/playground.js", ],

@brookylin I have the same issue as you. Do you solve this problem?

bump

The sample worked for me after applying the changes suggested in #879.


1.2 additional changes

Codecov Report
Base: 66.12% // Head: 99.83% // Increases project coverage by +33.70% :tada:

Coverage data is based on head (f820122) compared to base (7477c1e).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##              1.2     #825       +/-   ##
===========================================
+ Coverage   66.12%   99.83%   +33.70%
===========================================
  Files         104       27       -77
  Lines        4198     1221     -2977
  Branches      540      140      -400
===========================================
- Hits         2776     1219     -1557
+ Misses       1402        1     -1401
+ Partials       20        1       -19

Flag
Coverage Δ

foundry
?

production
99.83% <ø> (+0.48%)
:arrow_up:

reference
?

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

Impacted Files
Coverage Δ

contracts/lib/OrderValidator.sol
98.38% <ø> (+0.48%)
:arrow_up:

contracts/lib/OrderCombiner.sol
100.00% <0.00%> (ø)

contracts/lib/CounterManager.sol
100.00% <0.00%> (ø)

contracts/lib/OrderFulfiller.sol
100.00% <0.00%> (ø)

contracts/lib/ZoneInteraction.sol
100.00% <0.00%> (ø)

contracts/helpers/TransferHelper.sol
100.00% <0.00%> (ø)

contracts/lib/CriteriaResolution.sol
100.00% <0.00%> (ø)

contracts/lib/GettersAndDerivers.sol
100.00% <0.00%> (ø)

contracts/lib/ConsiderationDecoder.sol
100.00% <0.00%> (ø)

contracts/lib/ConsiderationEncoder.sol
100.00% <0.00%> (ø)

... and 88 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Enhance Documentation for Using Custom Docker Images with Covalent Remote Plugins

What should we add?
We have sporadically mentioned how to use custom Docker images with Covalent executors in our documentation, particularly in the context of AWS Lambda Executors (link). However, we believe there's a need for more comprehensive and dedicated documentation to guide users through the process of utilizing custom Docker images as environments for Covalent remote plugins. This enhanced documentation should also outline the constraints and best practices for creating and using custom images effectively.
Proposed Changes:

Create a dedicated how-to guide for using custom Docker images with Covalent remote plugins.
Detail the constraints and requirements that custom Docker images must follow to work seamlessly with Covalent.
For plugins that have unique requirements or considerations, provide separate documentation pages for each plugin.
Implement a central "How-to use custom Images for plugins" guide that describes the concept and links to individual plugin-specific documentation.

Hey, could you please assign this issue to me under hacktoberfest

I would like to work on this

Please feel free to work on this @neelshah2409 and pass on a PR.


New version: Julialang.Julia version <IP_ADDRESS>

https://github.com/microsoft/winget-pkgs/issues/137698

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

hold

@SpecterShell This has been marked ready for review 👀 Has the discussion in https://github.com/microsoft/winget-pkgs/issues/137698 reached to a conclusion? Should we start merging these in?

@SpecterShell This has been marked ready for review 👀 Has the discussion in #137698 reached to a conclusion? Should we start merging these in?

Not yet. I have converted them back to draft.


Amend base filter with new metadata filter

https://trello.com/c/2BGHlD7M/13-make-it-possible-to-publish-finder
We'd like to include any document tagged to a business readiness
filtering facet, so use an internal metadata value as the base
filter.

I think this will pass CI when #1327 is merged.


vendoring

I highly recommend to start using vendoring.
For example using govendor you would just:
govendor init
govendor add +external

Otherwise sooner or later you will bump into a situation where your code will not compile due some API changes of your dependencies. With vendoring you can compile any commit in the history of repository..

Done.


docs: update communtity resources

add quicknode articles
fix some links in projects
add Flowdiver historical API to projects
re-order wallets, prioritize frw.
add LearnWeb3 Flow Faucet

we'll be removing articles from the docs until we refactor the view
cc: @Aliserag

we'll be removing articles from the docs until we refactor the view cc: @Aliserag

So are we supposed to update the community resources after the articles view is refactored it?

yes, you still can add new articles into the list, we'll be refactoring the view and remove some obsolete articles, but the articles page will be restored in the future

@btspoony could you please resolve the merge conflict?
src/data/pages/learn.ts has been removed until further refactoring

@btspoony could you please resolve the merge conflict? src/data/pages/learn.ts has been removed until further refactoring

Resolved


Handle sciety-like docmaps that are generated by the canonical ETL tool

Currently, render-rev's handling of Sciety-style docmaps will error rather than emit a (limited) visual rendering in certain cases:

if the first step in the docmap has no inputs. This is common in our use case where we instead have a Step for the production of the preprint itself  rather than it being the first input to the first step.
if no publisher is provided in the docmap. This is just a display issue but it's healthy to have the widget still display and show where data is missing rather than an error in the system.
if no output.content. Now, show src: null (matching case where output.content has no text entry) rather than failing to display at all.
When using journal-article rather than journal-publication as the output type. This is canonical, and the PR doesn't break existing behavior, but leaves a comment to deprecate/change that in future.
if a preprint has a DOI but no URL. now, construct the doi.org url for that preprint.

Additionally, there is a small fix to date sorting logic.
I have added one test case matching this structure (using an actual example I generated using our tool).

Thank you very much! I see you've also indicated some TODOs, I'll try and fix those on our (server-) side.


More converters.

Merge after #10
Adds converters for the following catalog items:

EsriMapServerCatalogGroup (esri-mapServer-group)
MapboxVectorTileCatalogItem (mvt)
CartoMapCatalogItem (carto)

Converts the trait forceProxy

@nf-s I have added the tests. Let me know if it is good to merge.
Thanks!


use buildpg to prepare function

ref #68
@bitner let me know if that's what you envisioned.
~We now have another problem, we need to cast additional argument because at the application level we will get string. I'm not sure what's the best way to do this:~

~parse SQL code in python (seems possible using sqlparse)~
~use run a SQL code to get argument+type 🤷♂~

Last commit changed the way we handle function layers. This is now similar to https://github.com/urbica/martin#function-sources, where the application doesn't know the SQL function parameters but forward additional query parameter as a JSON param. This forward the responsibility of knowing how to parse the query to the user.
IMO this is the best option we have now. As mentioned before, we don't want to register the function to the database nor we want to parse the SQL code using python.
To Do

[ ] review from @bitner
[ ] update documentation


asd

Coverage remained the same at 100.0% when pulling 18c8d7555ee1f7ba439049f7e44683c22f7ed22a on master into c185fb56ec7beb7685c4d92cc42dbcaf98a67fe0 on test1.


Add test of dist structure

What
Ensure the dist tree matches expected contents (no additions either!)
Why
https://github.com/OpenAssetIO/OpenAssetIO-Test-CMake/issues/8 removed explicit exhaustitive header checks.

NB: consider all files including stray additions


chore: allow deps scope

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #895 (ecdf38a) into main (0730a34) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #895   +/-   ##
=======================================
  Coverage   92.96%   92.96%
=======================================
  Files         117      117
  Lines        4975     4975
  Branches     1446     1446
=======================================
  Hits         4625     4625
  Misses        329      329
  Partials       21       21

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 0730a34...ecdf38a. Read the comment docs.


Call IsActive for resource owner requests

IsActive is called in the token request validator for authorization code and refresh token requests.
It is not called for resource owner password requests

Should an inactive user return invalid_grant or invalid_request ?


web.config reset to default values

We saved the web.config using App Services Editor and Kudu with our parameters for IIS.
The configuration is ok.
But, a new day, a new web.config (by default).
How to avoid this behavior?
Thanks!

Put web.config at root of project,

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Redirect all users on HTTP to use HTTPS

Work with GA to implement it.

@steve9164 check for datasets loaded over http


Serializing member objects

In the example.dart file, class Person has a List of Orders with both classes annotated as serializable.
However the member List doesn't get serialized in Person.toJson(), is that intentional?
Map<String, dynamic> toJson() {
    var val = <String, dynamic>{
      'firstName': firstName,
    };

    void writeNotNull(String key, dynamic value) {
      if (value != null) {
        val[key] = value;
      }
    }

    writeNotNull('middleName', middleName);
    val['lastName'] = lastName;
    val['date-of-birth'] = dateOfBirth.toIso8601String();
    val['last-order'] = lastOrder?.toIso8601String();
    val['orders'] = orders; // <-- Shouldn't this be: val['orders'] = orders.toJson();
    return val;
  }

Nevermind! found this solution: dart-lang/json_serializable#57

Is there a better way to generate a map instead of doing JSON.decode(JSON.encode(entity)); ? I'm using mongo-dart and when inserting data, just JSON.encode is not useful since it generates a string and I'm not sure if doing JSON.decode(JSON.encode(..)) is great especially if you have frequent operations and potentially big json/entity sizes?

Have you tried entity.toJson(). I haven't tried json_serializable myself, but from having a quick look at the example code it should work.


EoS #200 idempotent producer

Resolves #200
Add an idempotent flag to the producer factory method to support creation of idempotent producers. Documented this as an "experimental" feature, since I haven't had a chance to observe its behavior in practice yet.
Implementation details

Idempotent producer will init the PID & epoch during the producer.connect() call
Idempotent producer uses the transactionManager to track sequences across messages and include in produce request
Idempotent producer enforces config optionsacks=-1 and retries>0. Default retry of MAX_SAFE_INTEGER

This PR additionally includes a few tests verifying the recent work allowing the firstSequence option to be provided in record batches.

@tulios @Nevon Thanks for the feedback! Made the requested changes for updating sequence


Dont update status/rating/thoughts if not changed

Currently if you update (for example) the rating of a show to the same as current rating, it will actually send a request to the server.
Frontend shouldn't send a request if changing to same as current and backend should reject the request if value hasn't changed.

Closed in favor of #114


Need ability to "not continue" an inline style or mutable entity

Right now, the editor simply looks to the left of the cursor to figure out what the style should be when inserting text (https://github.com/facebook/draft-js/blob/master/src/model/immutable/EditorState.js#L597). This works for "continuing" bold/italic/etc which is usually what you want. But sometimes you might not. It also uses the similar logic for mutable entities.
This might be complicated to describe, so I will explain with a use case: Imagine you want to implement inline code like this. So when you type a tick, some text, and another tick, it converts 'foo' into foo, by replacing the text and applying an inline style. However, as you continue typing, it will extend that code snippet because the cursor is picking up the inline style.
There are workarounds like: using immutable/segmented entities (mutable also exhibit this behavior) or adding an extra space when replacing the text and applying the inline code. But, really, what I want is the ability to edit "inside" of the snippet, but not have it continue at the boundaries.
I currently have a hacky workaround which handles onBeforeInput globally for certain inline styles/entities and manually inserts text when it borders on one of these types of inline styles.
Not sure if this is a widespread, known or otherwise specific use case, but worth logging.

I haven't looked at exactly how it's implemented, but if you go to the example on the homepage https://facebook.github.io/draft-js/, click Bold, type some text, click Bold again, then type more, the second piece of text is not bold. Is that what you want?

Hm, that is a good point. I suppose applying a style to the collapsed, new range may solve it, though I am certain I tried that. Let me see. Thanks @spicyj

Ah, yes. It uses toggleInlineStyle which uses setInlineStyleOverride (https://facebook.github.io/draft-js/docs/api-reference-editor-state.html#setinlinestyleoverride).
I've switched to using this as it's exposed. Thanks for the tip.

Actually, re-opening because I think it's still valid.
In the above case, I actually never want it to continue from the edge. Otherwise, you end up with weird edge cases such as being unable to get out of it without some hotkey or UI affordance. I can solve the case for continuing to type after the transformation, but once I move the cursor left and right again (for example), it picks up the style and you can't get out of it.

Here's an example:

Are you sure you never want to continue the style from the edge? If the user toggles a style override and begins typing, the cursor remains at the edge of the styled span -- shouldn't that override continue to be valid as the user continues to type?
Do you have examples of rich editors that support this behavior without a UI affordance or key command? I think your use case makes sense, I just want to evaluate whether this is something that should be fixed at a framework level, or whether the current APIs allow the appropriate behavior.

To answer your question, yes, that's correct -- it should be valid. I handle this case, and it's a little hairy. The code hasn't been battle-tested yet, but the basic idea seems to work.
As for the punchline, I'm not sure whether or not it should be made available at the framework level or not. It seems fairly isolated, but this will require ongoing maintenance as the framework progresses. I don't have the historical knowledge to make a judgment call here, but happy to take a shot at integrating.
I don't have a specific example in mind. Medium (for example), does what I am doing here, which is that a new block is a "reset" and it doesn't look upward. But it doesn't have inline code snippets like this.
handleBeforeInput(chars, {getEditorState, setEditorState}) {
  const state = getEditorState();
  const selection = state.getSelection();

  if (!selection.isCollapsed()) {
    return false;
  }

  const startOffset = selection.getStartOffset();
  const block = getCurrentBlock(state);

  let style = state.getInlineStyleOverride();
  if (style === null) {
    style = block.getInlineStyleAt(startOffset);
  }

  let insertManually = false;

  if (startOffset == 0) {
    // We check the style here because Draft's inherent behavior is such that if there
    // is no inline style, it "carries" from the previous line or block. We want it to
    // "reset" for each block. However, if there is a style override, we should
    // let the default behavior occur.
    insertManually = style.isEmpty();
  } else if (style.intersect(rigidInlineTypes).isEmpty()) {
    // Some inline types should not be continuable at the far edge. That is,
    // if you put the cursor at the end of the style and start typing, it should
    // not carry that style. Draft does this be default, so we need to defeat it
    // just as we do above.
    const prevStyle = block.getInlineStyleAt(startOffset - 1);

    // Check if we're bordering on a rigid type.
    insertManually = !prevStyle.intersect(rigidInlineTypes).isEmpty();
  }

  if (insertManually) {
    const entity = block.getEntityAt(startOffset);

    // Insert the text manually so we can control the inline style behavior.
    let content = state.getCurrentContent();
    content = Modifier.insertText(content, selection, chars, style, entity);
    setEditorState(EditorState.push(state, content, 'insert-characters'));
    return true;
  }

  return false;
}

Of course, this has ramifications. For example, the default behavior of toggleInlineStyle will not work with this implementation, as I have to handle the isCollapsed case myself:
/**
 * This is our own custom version of toggle inline style. It has one special
 * case where we want to toggle a style at the start of the block. The default
 * behavior for Draft is to look upward to a previous block, but we don't
 * want that, so we manually set the override.
 */
export function toggleInlineStyle(state, inlineStyle) {
  const selection = state.getSelection();
  if (selection.isCollapsed()) {
    const currentStyle = (selection.getFocusOffset() == 0)
      ? getCurrentBlock(state).getInlineStyleAt(0)
      : state.getCurrentInlineStyle().subtract(rigidInlineTypes);

    return EditorState.setInlineStyleOverride(
      state,
      currentStyle.has(inlineStyle)
        ? currentStyle.remove(inlineStyle)
        : currentStyle.add(inlineStyle));
  }

  // Fallback to default beahvior.
  return RichUtils.toggleInlineStyle(state, inlineStyle);
}

Example:

And sorry, to be clear, there are a couple things going on in this code (sorry for conflating):

inline styles don't carry from the previous blocks
"rigid" inline styles don't continue at the far edge

We have this need as well.

I'm not sure if our use-case warrants a different issue than this one because it seems to be very related so I'll just ask here.
We're applying a LINK entity to a range (with MUTABLE as a mutability strategy) like the link example shows but do not want the link to be applied to subsequent character insertions. If you look at Google Docs for example, once you insert a link then type anything (not necessarily a space character), the entity doesn't apply to those characters.
How can we achieve such an effect?
I thought of appending a whitespace character with no entity to the block's charlist but it seems hackish and has side-effects on the UI as the user shouldn't see a space since they didn't insert one... :confused: Another thought was to provide a custom strategy (or just tune the MUTABLE one, but could not find such an API).
Would love your input on this, kind of stuck on this now. Thank you very much.

@hellendag @amireh I am facing the same issue, creating a link that i want to be editable inside but non contiguous. We should add some kind of contiguous: boolean option to the mutable entity or something. This is a pretty standard UX for WYSIWYG editors. Down below is an example with Redactor.js :

Update, i made it work in #510

Grunt work is there: /src/model/entity/getEntityKeyForSelection.js
    if (offset > 0) {
      entityKeyBeforeCaret = contentState.getBlockForKey(key).getEntityAt(offset - 1);
      entityKeyAfterCaret = contentState.getBlockForKey(key).getEntityAt(offset);
      caretOutsideEntity = (entityKeyBeforeCaret !== entityKeyAfterCaret);

      return filterKey(entityKeyBeforeCaret, caretOutsideEntity);
    }

function filterKey(
  entityKey: ?string,
  caretOutsideEntity: ?bool,
): ?string {
  if (entityKey) {
    var entity = DraftEntity.get(entityKey);

    // if entity is mutable and caret is inside it, return it
    if (entity.getMutability() === 'MUTABLE' && !caretOutsideEntity) {
      return entityKey;
    }

    // entity is mutable and the caret is outside of the entity
    // if it is contiguous, return it, else null
    if (entity.getMutability() === 'MUTABLE' && caretOutsideEntity) {
      return (entity.getContiguity()) ? entityKey : null;
    }
  }
  return null;
}

There's a few todo's left and i'm busy with work so if anybody can help out on what's left so we can get this merged in sooner rather than late i'd be grateful :)

I was thinking about adding a zero width space as first character in the desired entity type and right after outside of it so that moving the cursor can "enter/leave" the entity. So when pressing left/right at entity edges the cursor wouldn't visually move but its position would change and allow typing before/after outside and at start/end inside of entity. Is this an acceptable hack?

@davidbyttow Hi, davidbyttow.
I have the same need with you. Thank you for your code above and now I'm using it . Just find that "getCurrentBlock" and "rigidInlineTypes" is not mentioned in documentation.
I think that "getCurrentBlock" is implemented by yourself, but I really have no idea about how to get current block and what "rigidInlineTypes" is.
Could you help me?

Now I find "getBlockForKey" in document and implement my "getCurrentBlock". So, I still don't know what "rigidInlineTypes" is.  Is it a string or something?
Uncaught ReferenceError: rigidInlineTypes is not defined

Here is my workaround (used like a plugin in draft-js-plugins), which overrides this behavior only for mutable entities:
  handleBeforeInput: (chars, { getEditorState, setEditorState }) => {
    const state = getEditorState();
    const selection = state.getSelection();

    if (!selection.isCollapsed()) {
      return false;
    }

    const startOffset = selection.getStartOffset();
    const content = state.getCurrentContent();
    const block = content.getBlockForKey(selection.getStartKey());

    const entity = block.getEntityAt(startOffset);
    if (entity === null) {
      let style = null;
      if (startOffset > 0) {
        style = block.getInlineStyleAt(startOffset - 1);
      }
      const newContent = Modifier.insertText(content, selection, chars, style, null);
      setEditorState(EditorState.push(state, newContent, 'insert-characters'));
      return 'handled';
    }

    return 'not-handled';
  },

Here is how I have implemented the logic for MUTABLE entity which are not  contiguous.
Entity creation
// create the entity which is mutable but not contiguous
const data = { href: "https://www.domain.com" }
contentState.createEntity(
    "LINK",
    "MUTABLE",
    { ...data, url: data.href, target: "_blank", contiguous: false }
)

Utility to handle the contiguous entity
import {
    EditorState,
    Modifier
} from "draft-js"
// method to handle the contiguous entity for the newly inserted chars in the editor state
const handleContiguousEntity = (chars, editorState) => {
    const selectionState = editorState.getSelection()
    let contentState = editorState.getCurrentContent()
    const startKey = selectionState.getStartKey()
    const block = contentState.getBlockForKey(startKey)
    // handle the extending of mutable entities
    if (selectionState.isCollapsed()) {
        const startOffset = selectionState.getStartOffset()
        if (startOffset > 0) {
            // we are not at the start of the block
            const entityKeyBeforeCaret = block.getEntityAt(startOffset - 1)
            const entityKeyAfterCaret = block.getEntityAt(startOffset)
            const isCaretOutsideEntity = (entityKeyBeforeCaret !== entityKeyAfterCaret)
            if (entityKeyBeforeCaret && isCaretOutsideEntity) {
                const entity = contentState.getEntity(entityKeyBeforeCaret)
                const isMutable = entity.getMutability() === "MUTABLE"
                const { contiguous = true } = entity.getData()
                // if entity is mutable, and caret is outside, and contiguous is set the false
                // remove the entity from the current char
                if (isMutable && !contiguous) {
                    // insert the text into the contentState
                    contentState = Modifier.insertText(
                        contentState,
                        selectionState,
                        char,
                        editorState.getCurrentInlineStyle(),
                        null
                    )
                    // push the new content into the editor state
                    const newEditorState = EditorState.push(
                        editorState,
                        contentState,
                        "insert-characters"
                    )
                    return newEditorState
                }
            }
        }
    }
    return false
}

The Editor Component's handleBeforeInput prop
// method inside the editor component
handleBeforeInput = (char) => {
    const { editorState } = this.state
    const newEditorState = handleContiguousEntity(char, editorState)
    if (newEditorState) {
        this.setState({ editorState: newEditorState })
        return "handled"
    }
    return "not-handled"
}

@tomconroy Thank u so much!  I'm very happy to get your solution

I use @sudkumar piece of code and implement it for my needs.. So if someone wants to look on that code and find inspiration how to handle some extra cases like convert HTML to State with extra Entity data etc... you will find it here:
https://gist.github.com/bultas/7f551efbc9054f8cf227f42db55eec07

@tomconroy I don't think setEditorState is available in latest version of Draft-js. Any work around for it ?

In my testing of other editors, lots/all have the behavior to continue inline styles, but none continue inline mutable entities. IMHO this issue should be restricted to this case and marked as a bug, unless there is a use case I'm overlooking where it's desirable to continue entities.
Note: tested Google Docs, Dropbox Paper, Gmail, Apple Pages.

@prashantagarwal
  handleBeforeInput = (chars, editorState) => {
    const selection = editorState.getSelection()
    const startOffset = selection.getStartOffset();
    const content = editorState.getCurrentContent();
    const block = content.getBlockForKey(selection.getStartKey());
    const entity = block.getEntityAt(startOffset);

    if (entity === null) {
      const style = editorState.getCurrentInlineStyle();
      const newContent = Modifier.insertText(content, selection, chars, style, null);
      const newEditorState = EditorState.push(editorState, newContent, 'insert-characters');

      this.setState({ editorState: newEditorState });

      return 'handled';
    }

    return 'not-handled';
  }

I just implemented this in the draft-js-markdown-plugin based on the code above by @tomconroy, but heavily adapted to do as little manual changes as possible. You can see the full diff here: https://github.com/withspectrum/draft-js-markdown-plugin/pull/83

I'm surprised the v0.10.5 changelog doesn't mention this issue, this got fixed for entities:

Fix issue where typing at the end of a link caused the link to continue. (Ian Jones in d16833b3)

For inline styles, still works the same as before 🙂.

@thibaudcolas thank you so much 👍 it works after updating to v10.0.5.
I think this issue can be closed now

@haikyuu not really since you still have to use the workaround I posted to get it working for inline styles?

@mxstbr i think it's the expected behaviour for inline styles. Check notes app, email app ... in mac

i think it's the expected behaviour for inline styles.

The point is that it depends on how users can type inline styles. If you have an inline toolbar, sure the current behavior makes sense, but if you have markdown shortcuts it's very annoying since you can't exit an inline style at the end of the editor.
That's why we need this behavior to be configurable, so you can set it to work the way it makes sense in your application! Nobody's saying to change it to only be non-sticky, but it should be configurable out of the box.

Is there a recommendation for the best way to get the previous contiguous entity behavior?  For my entities use case, that was the preferred behavior so the 0.10.5 upgrade is posing some issues.

@skinandbones you should be able to implement this in handleBeforeInput, the opposite way from what was discussed in this thread before the 0.10.5 change (e.g. https://github.com/facebook/draft-js/issues/430#issuecomment-269815367).
Something like:
handleBeforeInput(char) {
  const { blockTypes } = this.props;
  const { editorState } = this.state;
  const selection = editorState.getSelection();
    if (selection.isCollapsed()) {
      const block = getSelectedBlock(editorState);

      if (hasSelectionStartEntity(selection, block)) {
          this.onChange(
              insertTextWithEntity(editorState, char),
          );
            return HANDLED;
      }
    }

    return NOT_HANDLED;
}

  hasSelectionStartEntity(selection, block) {
      const startOffset = selection.getStartOffset();
      return block.getEntityAt(startOffset) === ! null;
  },

  insertTextWithEntity(editorState, text) {
    const content = editorState.getCurrentContent();
    const selection = editorState.getSelection();
    const style = editorState.getCurrentInlineStyle();
    const entity = block.getEntityAt(selection.getStartOffset());

    const newContent = Modifier.insertText(
      content,
      selection,
      text,
      style,
      entity,
    );

    return EditorState.push(editorState, newContent, 'insert-characters');
},

Taken from the implementation of the inverse behavior over at https://github.com/springload/draftail/pull/106.


Modify Code to talk to new instance

@geneorama

I'm not 100% sure what this means, but I think this refers to connecting to the postgres server.
This has been accomplished, and the configuration is in the file default_profile (which is not committed publicly)

This is to connect to the new Postgres instance with the encrypted drives. @hoatrieu has the new connection string.

I put the new server address into lead/default_profile today, I thought it was already in there.
I'm waiting to test because according to Eric "I would hold off on running drake right now because we made some small fixes and they're not all merged."

So far this seems to be working, @Hoatrieu do you want to terminate the previous instance of postgres?

I locked the access from analytics servers to the postgres database server (dc21pgl50)

Thanks @Hoatrieu, looks like transfer to the new DB is complete. Closing issue.


Document third configuration variable $PL_RPROMPT

Great project! I did discover that a third variable $PL_RPROMPT needs to be set if you want to control the powerline that appears on the far right after a command is entered. By default, this powerline shows the history number and the timestamp.
The README.rst mentions the other two prompt controlling variables $PL_PROMPT and $PL_TOOLBAR. Metioning $PL_RPROMPT would complete the trifecta.
Thanks again for providing this to the xonsh community!

..and, while that's happening, perhaps document all of the available commands?


$TM_SELECTED_TEXT overtyping not functioning as expected in snippets for latex files

Does this issue occur when all extensions are disabled?: Yes

Version: 1.74.2 (user setup)
Commit: e8a3071ea4344d9d48ef8a4df2c097372b0c5161
Date: 2022-12-20T10:29:14.590Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: No

Steps to Reproduce:

Create a snippet for latex that uses $TM_SELECTED_TEXT. The following is an example that wraps selected text in auto-sized brackets:

"big ()": {
   	"prefix": ["lr"],
   	"body": [
   		"\\left( $TM_SELECTED_TEXT \\right)",
   	],
   	"description": "Begin auto-sized braket"
 },

Type 'lr' over a selected text and hit TAB. The selected text is not included in the output.

Using F1 + Insert Snippet, however, produces the desired output.

I have only encountered the problem in latex environment, including inline math in markdown files.
The issue was raised in #167364 and was fixed in 1.73 update via #165871 (I think). But came back in 1.74.

The problem is the latex word definition which requires two characters to be typed... This is what happens

overtyping happens, we capture the overtyped text, and "await" the next suggest session to start
this doesn't happen because suggest only auto starts at word ends and a single character is no word
the next character is typed, erasing the overtyped, captured value
suggest starts because the 2nd character makes this a word

@alexr00 looping you in here and wonder if you know why such a word definition was chosen and if it can be changed

I don't know why the definition specifies 2 characters. I'm not familiar enough with latex to know if there's a reason for it. Given that no other language does this I would be fine to remove it.

+1 for changing the word definition. FYO @jlelong who originally added this via https://github.com/microsoft/vscode/commit/0e52a7574f2df629d57fe65fbdb1d3e9b6b83861

I confirm that changing the word definition to 1 character is fine.

This bug has been fixed in the latest release of VS Code Insiders!
@Walker-Xin, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 0a6c277e4faa44a6b6eb82ddca8bfeccf7fecef6 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

/verified

This bug is back with the latest update
Version: 1.75.0 (user setup)
Commit: e2816fe719a4026ffa1ee0189dc89bdfdbafb164
Date: 2023-02-01T15:23:45.584Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: No


Remove static vector from GetModuleOrder

Fixes #53
Not as clean but follows the Google style guide.

For the record, I prefer the new form of this, i.e. switch statements over the use of containers.  Two reasons:

the STL containers version(s) of this induces concern for leaks, races, destructor order, etc. as evidenced on an earlier MR and in https://github.com/KhronosGroup/SPIRV-Tools/issues/53      I much prefer to KISS and have everyone move on to implementing new functionality.
the containers form creates extra failure modes, e.g. if allocation fails.  The switch form will fail only at compile or link time.

+2
I'll merge.


add name

#203 was split into general asdf support in #213 and adding support for Tabular model here.

Coverage decreased (-0.06%) to 93.701% when pulling dd808bbf975133896550874dc493defbdae114ae on nden:tabular-model into 50546cfbc7987bb2ff92614ab04c4a95daa36f5d on spacetelescope:master.

Coverage decreased (-0.006%) to 93.753% when pulling 594cddf6ab795828332809245c3ddc56cac655f9 on nden:tabular-model into 50546cfbc7987bb2ff92614ab04c4a95daa36f5d on spacetelescope:master.


full Tuple implementatation

The List implementation should follow this one.
Following cases should be handled.

lhs (x, y = 1, 2)
rhs((x, y))
star(packing) in lhs (x, *y, z = t)
star(unpacking) in rhs ((x, *y, *z, t))
delete case: del (x, y, z)

@SenZhangAI


SqlServerDsc: Rename all issues so the PR title reflect the new resource name

Details of the scenario you tried and the problem that is occurring:
So that it should be easier to find issue on a particular resource I suggest we rename all resource names in the PR titles. Then it's easy to search for the new name.
Renaming in all the PR description and comments will be to much work. Let's try to close the old issues instead: 😉
The DSC configuration that is using the resource (as detailed as possible):
n/a
Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
n/a
What module (SqlServer or SQLPS) and which version of the module the DSC Target Node is running:
n/a
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
n/a

Merry Christmas everybody!! 🎄  Have a great time with your families!
I think I have renamed all issue titles now. Also updated some new resource proposals to suggest a new name according to new naming convention. They are open for discussion of course.


storage: store column_iterators without Option

https://github.com/risinglightdb/risinglight/blob/8464ae49879b2745f31674dcda43caefd42c86fd/src/storage/secondary/rowset/rowset_iterator.rs#L22
Currently, we are storing column_iterators: Vec<Option<ColumnIteratorImpl>>. After some refactors recently, we will fill the vector will all Some. So we can safely remove the Option here.

I will take this


Incompatible with python 3.7, breaks Azure CLI

For example:
❯ az aks get-versions --location westeurope
Traceback (most recent call last):
  File "/usr/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/usr/lib/python3.7/site-packages/azure/cli/__main__.py", line 14, in <module>
    import azure.cli.core.telemetry as telemetry
  File "/usr/lib/python3.7/site-packages/azure/cli/core/telemetry.py", line 20, in <module>
    import azure.cli.core.telemetry_upload as telemetry_core
  File "/usr/lib/python3.7/site-packages/azure/cli/core/telemetry_upload.py", line 20, in <module>
    from applicationinsights import TelemetryClient
  File "/usr/lib/python3.7/site-packages/applicationinsights/__init__.py", line 1, in <module>
    from .TelemetryClient import TelemetryClient
  File "/usr/lib/python3.7/site-packages/applicationinsights/TelemetryClient.py", line 5, in <module>
    from applicationinsights import channel
  File "/usr/lib/python3.7/site-packages/applicationinsights/channel/__init__.py", line 7, in <module>
    from .TelemetryChannel import TelemetryChannel
  File "/usr/lib/python3.7/site-packages/applicationinsights/channel/TelemetryChannel.py", line 6, in <module>
    from .TelemetryContext import TelemetryContext
  File "/usr/lib/python3.7/site-packages/applicationinsights/channel/TelemetryContext.py", line 4, in <module>
    from applicationinsights.channel import contracts
  File "/usr/lib/python3.7/site-packages/applicationinsights/channel/contracts/__init__.py", line 7, in <module>
    from .RemoteDependencyData import RemoteDependencyData
  File "/usr/lib/python3.7/site-packages/applicationinsights/channel/contracts/RemoteDependencyData.py", line 256
    def async(self):
            ^
SyntaxError: invalid syntax

This is because python 3.7 was recently released, and async and await are now reserved keywords.
Please also enable check for 3.7 on Travis 😉

My bad, the version of this package in the package distribution I used was 0.11.1, the issue is not present in 0.11.5.


can I get  separate iso

Sir I want to install windows 10 S  on laptop   so that I need separate iso file or install.wim.
Will you please give me direct downlaod link for windows 10 S.
Also I need product key, could you tell where I can purchase

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

ID: 693c3223-df7f-9fbb-ce68-d3fb1ec04d53
Version Independent ID: 2e2a1281-750a-63f6-f412-c466f44d03c9
Content: Windows 10 Pro in S mode
Content Source: windows/deployment/s-mode.md
Product: w10
GitHub Login: @jaimeo
Microsoft Alias: greglin

Hi there. The docs explain how to enable S Mode (beginning with version 1803) here:
https://docs.microsoft.com/en-us/windows-hardware/manufacture/desktop/windows-10-s-enable-s-mode
I hope that helps.

thanks sir.  You given correct link to enable s mode


Unable to use MRC

Hello everyone,
I am not able to use MRC while invoking cmake for DSSP installation:
cmake --cifpp_DIR=/home/peter/libcifpp/build/cifpp/ ..

-- Could NOT find Mrc (missing: MRC_EXECUTABLE)
CMake Warning at CMakeLists.txt:128 (find_package):
  Found package configuration file:

    /home/peter/mrc/mrc-config.cmake

  but it set Mrc_FOUND to FALSE so package "Mrc" is considered to be NOT
  FOUND.

CMake Warning at CMakeLists.txt:133 (message):
  Not using resources since mrc was not found

-- Configuring done
-- Generating done
-- Build files have been written to: /home/peter/dssp/build

Could someone please highlight the issue here?
If I proceed without making any changes, I get the following errors in the next step:
cmake --build . --config Release

Scanning dependencies of target unit-test
[ 16%] Building CXX object CMakeFiles/unit-test.dir/test/unit-test.cpp.o
[ 33%] Building CXX object CMakeFiles/unit-test.dir/src/dssp.cpp.o
[ 50%] Linking CXX executable unit-test
/usr/bin/ld: CMakeFiles/unit-test.dir/src/dssp.cpp.o: in function `boost::date_time::month_formatter<boost::gregorian::greg_month, boost::date_time::iso_extended_format<char>, char>::format_month(boost::gregorian::greg_month const&, std::ostream&)':
dssp.cpp:(.text._ZN5boost9date_time15month_formatterINS_9gregorian10greg_monthENS0_19iso_extended_formatIcEEcE12format_monthERKS3_RSo[_ZN5boost9date_time15month_formatterINS_9gregorian10greg_monthENS0_19iso_extended_formatIcEEcE12format_monthERKS3_RSo]+0x4c): undefined reference to `boost::gregorian::greg_month::as_short_string() const'
/usr/bin/ld: dssp.cpp:(.text._ZN5boost9date_time15month_formatterINS_9gregorian10greg_monthENS0_19iso_extended_formatIcEEcE12format_monthERKS3_RSo[_ZN5boost9date_time15month_formatterINS_9gregorian10greg_monthENS0_19iso_extended_formatIcEEcE12format_monthERKS3_RSo]+0x6f): undefined reference to `boost::gregorian::greg_month::as_long_string() const'
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/unit-test.dir/build.make:105: unit-test] Error 1
make[1]: *** [CMakeFiles/Makefile2:106: CMakeFiles/unit-test.dir/all] Error 2
make: *** [Makefile:141: all] Error 2

Regards
Peter

Hi Peter,
DId you install mrc? It can be found at https://github.com/mhekkel/mrc.git
It is not strictly required, but makes the executable more portable.
The second issue is now fixed, you should checkout a fresh copy of the git repository and try again.
regards, -maarten