fix(api): @bable/node is not the part of cli

IDK why, but I used to get an error while building. Re-installing this seems to fix the error. I also ran npm audit fix while I was at it.

Hey @raisedadead.  What wasn't working?  I can't see anything wrong with this, but I don't really know what I'm looking for.

The error was really cryptic:
sh: babel-node: not found

All I did was update it to the latest version and ran the npm audit fix while I was at it.

That is weird.  babel-node gets installed when I npm ci this branch and seems to work fine (i.e. npm run babel-dev-server runs without complaint) on my machine.
By the way, do you want me to take a look at the audit or is that under control?

...when I npm ci this branch and seems to work fine...

That is the point of the fix, unless you meant it works fine on master?

By the way, do you want me to take a look at the audit or is that under control?

Nope the audit is fine as well. You should merge this if you see the build / dev is fine.

That is the point of the fix, unless you meant it works fine on master?

Yep, it's fine for me on this branch and on master.

Nope the audit is fine as well. You should merge this if you see the build / dev is fine.

I'm still seeing 4 vulnerabilities (though 3 are dev only) on this branch which couldn't be resolved by npm audit fix.

Hmm, maybe it's down to the binaries for the dependencies that I have for my OS (macOS). I don't think bumping the version is going to hurt (you have already confirmed, it works for you on this branch).
As for the remaining sec issues:
~/DEV/freeCodeCamp/api-server fix/add-bable-node
❯ npm audit fix
up to date in 9.477s
fixed 0 of 4 vulnerabilities in 890669 scanned packages
  3 vulnerabilities required manual review and could not be updated
  1 package update for 1 vuln involved breaking changes
  (use `npm audit fix --force` to install breaking changes; or refer to `npm audit` for steps to fix these manually)

I get the same checked that these are in the explorer (swagger-ui) and will not be available in production. So we can worry about them later?

I'm not worried about the swagger ones either - as you say, they're not used in production.  The one that concerns me is the googleapis package, since that's used in production in the 'About' model.

Regardless, this PR is not adding to the risk that we already have. The major changes will have to come in a separate PR? Unless you want to address it here? Right now, the build are pretty much broken for me.

Regardless, this PR is not adding to the risk that we already have. The major changes will have to come in a separate PR?

That's a very good point, I'll stop holding this PR up - I wasn't thinking straight. I'll create an issue about the security updates in a sec.

@raisedadead I can't merge this, but it's ready.

Thanks @ojeytonwilliams and @ValeraS !


time.clock() deprecated in python

time.clock() has been removed in python, causing test_image.py to crash. It has been replaced with timeit.default_timer.
It can be fixed with sed -i 's/time\.clock/timeit\.default_timer/g' test_image.py.

@nirimal Oh, thank you. I'll check this error right away.


ImportError: DLL load failed while importing _wenet: 找不到指定的模块。

Describe the bug
(voice) PS E:\XXXX> python src/main.py
Traceback (most recent call last):
  File "src/main.py", line 3, in <module>
    import wenetruntime as wenet
  File "E:\ENV\anaconda\envs\voice\lib\site-packages\wenetruntime\__init__.py", line 1, in <module>
    from .decoder import Decoder  # noqa
  File "E:\ENV\anaconda\envs\voice\lib\site-packages\wenetruntime\decoder.py", line 17, in <module>
    import _wenet
ImportError: DLL load failed while importing _wenet: 找不到指定的模块。
>>>

To Reproduce
conda create -n voice python=3.8
conda activate voice
python -m pip install wenetruntime
python
>>> import wenetruntime

I tried this on Windows 10 and Windows 11 and got the same error. I tried uninstalling and reinstalling wenetruntime with pip, still doesn't work.
Screenshots
In order to take a screenshot, I recreated a virtual environment called voice2, the result is as follows

Desktop (please complete the following information):

Win10 x64 and Win11 x64, same error

Additional context
https://github.com/wenet-e2e/wenet/issues/1180 I found a similar question and tried to follow his way out, but it didn't work.

问题解决，除了 wenetruntime, 还需要再装一个 wenet （但是文档上没写，最好可以补充一下
python -m pip install wenet

it didn't work too

wenet

where did you found this package? I cant find it in pip

wenet

where did you found this package? I cant find it in pip

https://pypi.tuna.tsinghua.edu.cn/packages/18/90/e2677c94896a1fcb96011e4c57b0607e0b9d34cc6797724583e5be6fb52e/wenet-1.0.6-cp39-cp39-win_amd64.whl


Highlight Words

I tried to highlight multiple words like this:
```sh /"Nenad<EMAIL_ADDRESS>
And also like this:
```sh /"Nenad Novakovic"/<EMAIL_ADDRESS>
But neither approach has worked. In the docs, I saw an example with a single word only. Is this not a possibility yet and may I know if the regex going to be an option?
Good day!

This is now possible in v0.4.0 thanks to @ttwrpz:
```js /word/ /otherword/


Possible to bundle with production flag in fusion framework cli

I'm experiencing some pretty hefty bundlesizes would it be possible to add a flag to choose production mode?
https://github.com/equinor/fusion-framework/blob/dfa64e26241c08f7935eda71bd59bb861404d71e/packages/cli/src/scripts/create-config.ts#L41

@adukstad My workaround for this was to create a vite config and bundle the application without the use of the cli tool. Maybe you can take inspiration from here https://github.com/equinor/cc-components


Not publishing specific Pipeline metrics

buildkite-metrics seems to be not reporting all Pipeline specific metrics, I can get the overall metrics but not by the specific Pipeline. In this example the builds were occurring on hoteltonight.
Version is buildkite-metrics v1.2.0-16-g5e7c730
Invoked with the following in a init script:
AWS_REGION=us-east-1 /usr/local/bin/buildkite-metrics -interval 30s -token SOME_TOKEN -org hoteltonight

The token has the following permissions:
read_agents,read_builds,read_build_logs,read_organizations,read_pipelines,read_user

Job output:
2016/12/13 07:50:38 Builds page 1 has 30 builds, next page is 2
2016/12/13 07:50:38 Builds page 2 has 5 builds, next page is 0
2016/12/13 07:50:38 Collecting running and scheduled build and job metrics
2016/12/13 07:50:38 Builds page 1 has 1 builds, next page is 0
2016/12/13 07:50:38 Collecting agent metrics
2016/12/13 07:50:38 Agents page 1 has 13 agents, next page is 0
2016/12/13 07:50:38 Buildkite > RunningJobsCount = 8
2016/12/13 07:50:38 Buildkite > ScheduledJobsCount = 12
2016/12/13 07:50:38 Buildkite > UnfinishedJobsCount = 20
2016/12/13 07:50:38 Buildkite > BusyAgentCount = 8
2016/12/13 07:50:38 Buildkite > IdleAgentCount = 5
2016/12/13 07:50:38 Buildkite > TotalAgentCount = 13
2016/12/13 07:50:38 Buildkite > RunningBuildsCount = 1
2016/12/13 07:50:38 Buildkite > ScheduledBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [queue = default] > RunningBuildsCount = 1
2016/12/13 07:50:38 Buildkite > [queue = default] > ScheduledBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [queue = default] > RunningJobsCount = 8
2016/12/13 07:50:38 Buildkite > [queue = default] > ScheduledJobsCount = 12
2016/12/13 07:50:38 Buildkite > [queue = default] > UnfinishedJobsCount = 20
2016/12/13 07:50:38 Buildkite > [queue = default] > BusyAgentCount = 8
2016/12/13 07:50:38 Buildkite > [queue = default] > IdleAgentCount = 5
2016/12/13 07:50:38 Buildkite > [queue = default] > TotalAgentCount = 13
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > RunningBuildsCount = 1
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > ScheduledBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > RunningJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > ScheduledJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > UnfinishedJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = XYZ] > RunningBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = XYZ] > ScheduledBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = XYZ] > RunningJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = XYZ] > ScheduledJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = XYZ] > UnfinishedJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = ABC] > ScheduledBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = ABC] > RunningJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = ABC] > ScheduledJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = ABC] > UnfinishedJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = ABC] > RunningBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = DEF] > RunningBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = DEF] > ScheduledBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = DEF] > RunningJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = DEF] > ScheduledJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = DEF] > UnfinishedJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = GHI] > RunningBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = GHI] > ScheduledBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = GHI] > RunningJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = GHI] > ScheduledJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = GHI] > UnfinishedJobsCount = 0
2016/12/13 07:50:38 Extracted 41 cloudwatch metrics from results
2016/12/13 07:50:38 Submitting chunk of 10 metrics to Cloudwatch
2016/12/13 07:50:38 Submitting chunk of 10 metrics to Cloudwatch
2016/12/13 07:50:38 Submitting chunk of 10 metrics to Cloudwatch
2016/12/13 07:50:38 Submitting chunk of 10 metrics to Cloudwatch
2016/12/13 07:50:38 Submitting chunk of 1 metrics to Cloudwatch
2016/12/13 07:50:38 Finished in 978.43957ms

Sorry for the issues @callumj and thanks for the issue report! So that I am clear, are you saying that despite looking like it's submitting metrics for the hoteltonight pipeline, it's not?
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > RunningBuildsCount = 1
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > ScheduledBuildsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > RunningJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > ScheduledJobsCount = 0
2016/12/13 07:50:38 Buildkite > [pipeline = hoteltonight] > UnfinishedJobsCount = 0

Or are you saying there are values the should have been reflected in the above that aren't?

Hey @lox! Yes, at that time all those default metrics were only coming from hoteltonight but there were not reflected in the breakdown.

Thanks for the report @callumj, think I've found the issue. Will get this merged in asap.

Thanks @lox!


Fix for LRS definition

@kurtfurbush : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@kurtfurbush - Thanks for making that useful clarification.
#sign-off


[CAMEL-18737]: camel-kamelet parameter substitution does not work in …

…bean instantiation when constructor is used

Ah there is already tests for 3.18.x, thanks


New Workspace should be seeded with the original folder

Testing #30722
I have two folders carbon-core and carbon-extension and I want to create a workspace that contains these folders.

open the folder carbon-core
select 'Add Folder to Workspace'
the info box shows up

I confirm that I want to create a workspace and select 'New Workspace'
A folder dialog shows up and I select carbon-extension, the folder I wanted to add to the workspace
A new workspace opens with the status bar indication, that only shows the carbon-extension with the root folder hidden:

At this point I was confused. I would have expected that this new workspace also includes carbon-core, since I wanted to add folder to carbon-core. In addition, since the root folder is hidden the only differentiation to an open folder is the different color in the status bar.
I suggest that the info box wording is changed to:
"Do you want to create a new workspace for the open folder"
And the new workspace is then seeded with the currently opened folder.

@egamma I think you must have used an old build for testing, e.g. the experience changed a lot and now you will see:

Can you confirm?

Verified much better flow now - closing.


Initial Package Split and Cloud Networks

This is the first step to splitting the Rackspace .NET SDK out of OpenStack.NET. It builds upon the OpenStack Networking v2 API and demonstrates how we can customize our CloudNetworksService class to only expose what we support.

Rackspace doesn't support bulk creation
Rackspace does support paging
Rackspace doesn't support port allowed address pairs
Other miscellaneous resource definition differences etc

In order to prevent unsupported features from leaking through, we don't use inheritance and instead use composition when possible and otherwise we simply redefine the resource definition to match what is supported. At times this may look like some epic copy/paste but that appears to be unavoidable at the resource definition level if we don't want to end up exposing unsupported features/properties. Most logic from OpenStack that can be reused is in the NetworkApiBuilder (e.g. auth retries, the type of http verb used, query parameters, etc).
Suggestions are welcome! :smiley:

Fixes #1


[request] using a sha256 hex (64 chars) hash as a proxy session identifier leads to cryptic 407 error

this one took me literally hours to debug. I was constantly getting HTTP/1.1 407 Proxy Authentication Required without any indication of what was the issue, when setting the proxyUrl of requestAsBrowser using getApifyProxyUrl. My generated session id had 64 chars (the hex representation of a sha256).
After removing the session parameter, I noticed that connections were successful, then tried with a smaller id, like "test" and it worked.
it would be nice to forcefully throw when the max allowed character limit is trespassed, since this information is a shy note in the proxy docs https://docs.apify.com/proxy/datacenter-proxy (max 50 chars), but the SDK docs say:

Apify Proxy session identifier to be used with requests made by CheerioCrawler. All HTTP requests going through the proxy with the same session identifier will use the same target proxy server (i.e. the same IP address). The identifier can only contain the following characters: 0-9, a-z, A-Z, ".", "_" and "~". Only applied if the useApifyProxy option is true.

another way would to make the proxy return a 414 or 400 (although both aren't accurate, but 414 comes closer), but this wouldn't be in the scope of this repo I guess.

We're sorry about that @pocesar , that's a really annoying bug. We're trying to keep stuff in sync with the Apify Platform, but sometimes something slips through the cracks. Thanks for bringing this to our attention.


Argument for prisma studio that injects a model for _prisma_migrations so it can be reviewed and modified with Prisma Studio

Right now you can not easily use Studio to review _prisma_migrations and its content. By injecting a predefined model into the schema for Studio, we could enable that easily.
Tasks:

Figure out Prisma model reflecting _prisma_migrations (probably easiest via renaming an example created by Prisma into a different name so that Introspection/db pull does not ignore it any more)
Expand prisma studio CLI command to have a parameter that modifies the schema file in memory (add model for _prisam_migrations) before handing it over to Studio
Maybe: Handle errors when migration table does not actually exists

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
This issue was originally created for internal tracking purposes, but if you feel this is important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest.
For more insight into our priorities and how we’re thinking about the future of Prisma, please see our blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


Redraw highlight divs when the viewport is resized

When the viewport of the cloud reader is resized, the highlights show up at apparently random locations. This is because the absolutely positioned highlight-divs are not repositioned when the window is resized.
Digging into the annotation module in Readium I saw that the code for redrawing the highlight divs is available in the redraw()-method, it's just never called. This patch solves the problem in the cloud reader, but I'm not sure if calling redrawHighlights from js/views/reader_view.js:handleViewportResize() is enough.
Any help would be appreciated!
Jan

@JCCR @dmitrym0 I seem to remember you guys have a pretty good understanding of the execution flow for selection highlights (semi-transparent overlay divs). Could you please check @jansc suggested fix? Many thanks!

Hi, sorry I just got a chance to look at this now..
@jansc Thanks for this PR. Calling it on reader_view:handleViewportResize() is not enough, you are right.
I would much rather like exposing this redrawHighlights as a public function in reader_view, so then who ever needs to use it, they can call it from a PAGINATION_CHANGED event callback or manually when needed. Calling it when this event triggers should cover all cases when they should be redrawn.

Thanks for the feedback and sorry for the late answer. I'll update this pull request soon with redrawHighlights() as a public function in reader_view.

I'm sorry it took so long to implement your suggestions, but now I've committed an update. If I got you right, the new reader_view.redrawHightlights() function is not to be called in reader_views PAGINATION_CHANGED callback but somewhere else, e.g. in readium-js-viewer's PAGINATION_CHANGED callback.


Removing all the StreamExecute...2 methods.

Both in vtgate and vttablet services. They are unused,
and don't map to any real RPC difference.
@aaijazi

This change is

LGTM


I 38 watch namespace

Fixes #38
The idea is being able of restrict the controller to a particular namespace as proposed here:
https://github.com/hashicorp/vault/issues/7364

@fcgravalos this is awesome! I wonder if we should consider using an "include/exclude" approach instead though?
Current PR implementation allows for either all namespaces or one namespace to be watched, I could imagine use cases where we would like a single secrets-manager to handle a group of namespaces (or exclude just several ones).
WDYT? overkill? or perhaps could be a welcome flexibility?

@fcgravalos this is awesome! I wonder if we should consider using an "include/exclude" approach instead though?
Current PR implementation allows for either all namespaces or one namespace to be watched, I could imagine use cases where we would like a single secrets-manager to handle a group of namespaces (or exclude just several ones).
WDYT? overkill? or perhaps could be a welcome flexibility?

I was considering it too, but took as an example nginx ingress controller that seems to just watch either one or all and thought it would be simpler.
In any case it seems a good idea too! Thanks! Let me figure out what are the implications and I'll get back to you :)

Codecov Report

Merging #39 into master will increase coverage by 0.25%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage      84%   84.25%   +0.25%
==========================================
  Files           8        8
  Lines         425      432       +7
==========================================
+ Hits          357      364       +7
  Misses         51       51
  Partials       17       17

Impacted Files
Coverage Δ

controllers/secretdefinition_controller.go
77.03% <100%> (+1.25%)
:arrow_up:

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 b7b5ccc...ded5799. Read the comment docs.

@eduardogr Thanks for your comments and sorry, we went ahead and merged and didn't realize about them.
I think it would be great if you can file a PR with your suggestion and let's build snapshot-2.
We can also sit down together and discuss about versioning/release process.

i thinks it could be nice for my PR here, I'll do it!


Bugfix for signal handlers

[x] I incremented the version in the VERSION file
[x] I updated CHANGELOG.md
[ ] I added test coverage for any changes

Closes #179
Not easy to test, not going to bother

LGTM.  Can you clarify what happens if it gets a signal?  So it closes the consumer.  Does that mean it would continue to process any events it was already processing?

It won't finish processing any in-progress events. Adding that functionality would be a little bit bigger PR. It will just trigger the client to leave the consumer group.

Okay...  Don't worry about it for now.  I was mostly just wanting to understand.


Custom objectives

implements custom objective specification for Bayesian optimization e.g. using GP models of individual objective components

Bad rendering in the notebook:

should be fixed now


data-turbo-confirm doesn't work if the form action contains a period

The simplest way to understand is probably this Codepen, but here is a quick rundown:
When building a form with data-turbo-confirm, if the last part of the URL contains a period, Turbo does not fire the confirm. For example:
<form data-turbo-confirm="Are you sure?" action="/domains/example.com" ...>

I assume this has something to do with formats, because .html works as expected—i.e.: this is fine:
<form data-turbo-confirm="Are you sure?" action="/domains/example.com.html" ...>

I assume this is not expected behaviour. I had a quick look at the source code and couldn't see anything explaining why this might be, but I'm new to Turbo, so I'm not too sure I was looking in the right place.
Thanks! 🥳

see #385 discussion


other package seems to be installed if it is in the staged dependency

You can recreate the issue with osprey when adding random.cdisc.data into the upstream dependencies and then running
install_deps()

please fix that as soon as possible.

So this behaviour has changed since https://github.com/openpharma/staged.dependencies/pull/48 has been merged.
Currently on main if you add a package as an upstream dependency into the yaml file which is not an upstream dependency of the package (based on the dependency graph using the DESCRIPTION file) then you get a cryptic error

This is the case if you add rcd as an upstream dependency of Osprey as it isn't. Once all the PRs associated with #44 go in then we won't see this behaviour unless you change the yaml files.
There's a more fundamental point about having different actual dependencies to those specified in the yaml files. My opinion would be to see if there's a mismatch and if that's the case to fail and give a helpful error message rather than silently no installing certain packages? Thoughts @maximilianmordig ?

The yaml files are used to discover the packages that it will install. The DESCRIPTION files are used to build the true dependency graph to determine the installation order.
Here, osprey may erroneously list rcd as an upstream dependency. In the true dep graph, this is noticed and may lead to rcd appearing after osprey in the topological order which leads to this error. In this case, I would say that we install rcd as well although it is not a true upstream dependency.
We can write a function to check the integrity so that such things do not happen, similarly to the function update_with_direct_deps.


Portal-1135: Dropdown UI Component

Introduces a new UI component called the Dropdown component
Tests added for component

Future Improvements:
Allow tabbing and enter key down to select dropdown items

Make sure the commits that are part of this PR get squashed into one commit with a "feat" prefix.


Feature Request:  Snippet Fragments

I currently use MassCode but I've started transitioning to Snippet-Box. I'd like to be able to have multiple fragments of snippets but all under 1 common 'snippet' name rather than cluttering up my instance so it's easier to copy specific fragments without having to search a bunch of snippets
I hope I explained that well, but if not let me know and I can try explain better

Maybe as snippets-grouping ?


Quit micro with a non-zero exit code

Description of the problem or steps to reproduce
I am using Micro to edit Git commit messages. Is it possible to quit Micro with a non-zero exit code to signal Git that the commit should be aborted?
Specifications
Commit hash: f594686
OS: macOS 10.15.4
Terminal: Terminal app (version 2.10) included with macOS

I don't think that is currently possible (it will quit with a non-zero exit code if it crashes).

Good point, I was so used to VI's :cq that I didn't think of that. And yes, that works.
However, a key binding would be a tad more convenient than removing the commit message in this situation. But I can also understand the desire to keep the editor simple and basic.

You shouldn't need to remove the commit message manually. Just pressing CtrlQ and then n should be enough.

I regularly amend-commit to work with a code review system In these cases there already is a commit message. Just pressing Ctrl+Q exits the editor and continues the commit operation.
I'm aware that this may be an edge case that doesn't justify adding a new 'exit strategy'.

For easy aborting of commit if the commit message is already non-empty, you can use something like:
bind "Alt-q" "SelectAll,Delete,Save,Quit"
I've quickly tried it, works for me.
The annoying bit about this solution is that it sets this Alt+Q binding globally for any buffer, not just for Git commits. A better solution might be possible with some Lua scripting.
It would be nice to be able to set keybindings per filetype (in this case, for "git-commit" filetype). Currently it's not quite easy to implement, since Micro supports global keybindings only: #1656

Another use case for this feature is BASH's edit-and-execute-command feature:
  edit-and-execute-command (C-xC-e)
          Invoke  an  editor  on the current command line, and execute the
          result as shell commands.   Bash  attempts  to  invoke  $VISUAL,
          $EDITOR, and emacs as the editor, in that order.

By default, BASH will fill the last command (or the currently-entered command) in the text editor. If the text editor exits with a zero exit code with no saved changes, the previous/original command is executed. I would really like to be able to ctrl+e in micro and type quit 1 (for example) to exit with a code of 1, telling BASH not to run any command.
See the comment on this SE answer and this other SE answer.

Just to append to the list of possible use-cases: I use mblaze which cancels mail drafts if the ${EDITOR} returns with a RC != 0.


Minor Gif fixes

Animated Gif files will now pull the delay from the file itself
Non-animated gifs will be treated like normal image files.

@araten10 feel free to add me on discord, same as my username here.


Add USE_WEBP option to disable building WebP.

Description
Similar option as exists for other external libraries.
Tests
Not applicable.
Checklist:

[x] I have read the contribution guidelines.
[x] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[x] I have updated the documentation, if applicable.
[ ] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

LGTM. Merging, and I will backport to 2.0.


Update RNFSManager writeFile method to handle attributes.

You can now append data to a file with the 'append' attribute.

This is awesome, just what I was looking for! 😀 Hope this gets accepted to the main branch quickly.
Are you planning on doing an Android implementation too?
👍 👍 👍

@K0rdan great! As @IjzerenHein mentioned, it would be great to get this in for Android too, so we have feature parity on this one :) Is that something you could look into?

Done !
Use the 'options' parameter was much more easier on Android as you can see in my update of the RNFSManager.java.

Sweet 😀 Awesome work @K0rdan !!

@K0rdan @johanneslumpe I think we should accept however I'm wondering if this should be a separate appendFile method. It would be nice to make the library follow the same pattern as Node's fs module:
https://nodejs.org/api/fs.html#fs_fs_appendfile_file_data_options_callback
If no one objects I can make this change on top of this PR.

I totally agree with @johanneslumpe on trying the create API parity with nodejs 👍

Really looking forward to this new append feature. 👍  Do you have a target date when you want to release this?
Cheers

@IjzerenHein Sorry appendFile has already been added, but I forgot to update the readme!

@cjdell Are you sure about that? The latest official version is 1.5.1 and was released 29 days ago. The pull request was created 13 days ago.

@IjzerenHein Sorry, I meant it's on master. You can add the GitHub repo URL directly to your package.json if you need it now. I can't update the NPM package, only @johanneslumpe can do that.

@cjdell Is there anything else you want to merge in or should we cut a release?


Implement Alpha Indicator

Expected Behavior
Lean supports Alpha as a Lean Indicator which takes a Symbol object, a benchmark Symbol, and a look-back period.
Actual Behavior
Not implemented. See #83.
Potential Solution

Implement Alpha Indicator in a similar way to Beta
Tthe QCAlgorithm.B method doesn't follow the Beta class parameter overload order which breaks the pattern.

Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

This one doesn't have a talib implementation, but it could be as easy as alpha = return - riskfree rate - beta(), in which can be done with our existing indicators:
Return : RateOfChange(1)
Beta: Beta(same window)
Risk free rate can be implemented as an arbitrary number, or from some benchmark return series.

qqq_alpha.csv
spy_alpha.csv

Hi there, I'm interested in taking this on. Please be patient with me, as I'm new to this project 😃
Taking a look in QuantConnect.Benchmarks, I see two classes, FuncBenchmark and SecurityBenchmark, are these appropriate classes to use to calculate the benchmark return? Both have an Evaluate method and seem so, but just wanted to clarify.

@LouisSzeto I don't understand your notation :) By chance, expand it, please. Better, if you know the code implementation, share, please.

spy_alpha.csv
Columns: "SPY close" "benchmark(QQQ) close" "alpha"
qqq_alpha.csv
Columns: "QQQ close" "benchmark(SPY) close" "alpha"
script:
import talib

qb = QuantBook()

spy = qb.AddEquity("SPY", Resolution.Daily).Symbol
qqq = qb.AddEquity("QQQ", Resolution.Daily).Symbol

history = qb.History([spy, qqq], datetime(1998, 1, 1), datetime(2023, 2, 23))

close = history.loc[spy].close

# Assumes no risk free rate
history2 = pd.concat([close, history.loc[qqq].close], axis=1).dropna()
history2.columns = [spy, qqq]

alpha_spy = (ret.loc[history2.index, spy] - talib.BETA(history2[spy], history2[qqq], 252).values * ret.loc[history2.index, qqq]).to_frame()
alpha_spy["spy close"] = close
alpha_spy["qqq close"] = history2.iloc[:, 1]
alpha_spy = alpha_spy.iloc[:, [1, 2, 0]].dropna()
alpha_spy.to_csv("spy_alpha.csv", header=False)

alpha_qqq = (ret.loc[history2.index, qqq] - talib.BETA(history2[qqq], history2[spy], 252).values * ret.loc[history2.index, spy]).to_frame()
alpha_qqq["qqq close"] = history2.iloc[:, 1]
alpha_qqq["spy close"] = close
alpha_qqq = alpha_qqq.iloc[:, [1, 2, 0]].dropna()
alpha_qqq.to_csv("qqq_alpha.csv", header=False)


Do not use system freetype for arm64 (2-0-x)

Backports #473 to 2-0-x, refs https://github.com/electron/electron/issues/12057.

I just tested this change against 2-0-x on our ARM CI and everything is green:
https://arm64-ci.electronjs.org/blue/organizations/jenkins/Electron ARM64%2Felectron/detail/arm64-ci-for-2-0-x/3/pipeline


requirements file environment variables in path using -r do not work

If I have a requirements file including a path to an external requirements file using the -r option such as:
-r ${APP_PATH}/app_requirements.txt

this is the correct syntax for pip but does not work with this plugin. Only absolute paths work with the -r option with this plugin; environment variables in the supplied path are not processed.
Output during a deploy:
  Error: ENOENT: no such file or directory, open '/example/${APP_PATH}/app_requirements.txt'
      at Object.openSync (fs.js:458:3)
      at Object.readFileSync (fs.js:360:35)
      at getRequirements (/example/node_modules/serverless-python-requirements/lib/pip.js:354:6)
      at /example/node_modules/serverless-python-requirements/lib/pip.js:364:24
      at Array.reduce (<anonymous>)
      at getRequirements (/example/node_modules/serverless-python-requirements/lib/pip.js:358:23)
      at filterRequirementsFile (/example/node_modules/serverless-python-requirements/lib/pip.js:381:24)
      at generateRequirementsFile (/example/node_modules/serverless-python-requirements/lib/pip.js:88:5)
      at installRequirementsIfNeeded (/example/node_modules/serverless-python-requirements/lib/pip.js:501:3)
      at ServerlessPythonRequirements.installAllRequirements (/example/node_modules/serverless-python-requirements/lib/pip.js:635:29)

For the use case of nesting external requirements files with the -r option (files which are typically going to be maintained centrally) the processing of an environment variable as part of the path is surely very common, replying only on absolute paths (which one would never want to commit) or relative paths (which vary based on where each engineer's code exists) negates all the benefits of supporting the -r option.

I am having the same issue when I have this in my requirements file.
-r https://raw.githubusercontent.com/snowflakedb/snowflake-connector-python/v2.4.6/tested_requirements/requirements_38.reqs
` Error ---------------------------------------------------
Error: ENOENT: no such file or directory, open '/home/john/PycharmProjects/ml-pipelines/DptSnowTest/https:/raw.githubusercontent.com/snowflakedb/snowflake-connector-python/v2.4.6/tested_requirements/requirements_38.reqs'
at Object.openSync (node:fs:582:3)
at Object.readFileSync (node:fs:450:35)
at getRequirements (/home/john/PycharmProjects/ml-pipelines/node_modules/serverless-python-requirements/lib/pip.js:360:6)
at /home/john/PycharmProjects/ml-pipelines/node_modules/serverless-python-requirements/lib/pip.js:370:24
at Array.reduce ()
at getRequirements (/home/john/PycharmProjects/ml-pipelines/node_modules/serverless-python-requirements/lib/pip.js:364:23)
at filterRequirementsFile (/home/john/PycharmProjects/ml-pipelines/node_modules/serverless-python-requirements/lib/pip.js:387:24)
at generateRequirementsFile (/home/john/PycharmProjects/ml-pipelines/node_modules/serverless-python-requirements/lib/pip.js:92:5)
at installRequirementsIfNeeded (/home/john/PycharmProjects/ml-pipelines/node_modules/serverless-python-requirements/lib/pip.js:521:3)
at /home/john/PycharmProjects/ml-pipelines/node_modules/serverless-python-requirements/lib/pip.js:619:35
at Array.map ()
at ServerlessPythonRequirements.installAllRequirements (/home/john/PycharmProjects/ml-pipelines/node_modules/serverless-python-requirements/lib/pip.js:613:8)
at ServerlessPythonRequirements.tryCatcher (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/util.js:16:23)
at Promise._settlePromiseFromHandler (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/promise.js:547:31)
at Promise._settlePromise (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/promise.js:604:18)
at Promise._settlePromise0 (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/promise.js:649:10)
at Promise._settlePromises (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/promise.js:729:18)
at _drainQueueStep (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/async.js:93:12)
at _drainQueue (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/async.js:86:9)
at Async._drainQueues (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/async.js:102:5)
at Immediate.Async.drainQueues [as _onImmediate] (/home/john/PycharmProjects/ml-pipelines/node_modules/bluebird/js/release/async.js:15:14)
at processImmediate (node:internal/timers:464:21)
Get Support --------------------------------------------
Docs:          docs.serverless.com
Bugs:          github.com/serverless/serverless/issues
Issues:        forum.serverless.com
Your Environment Information ---------------------------
Operating System:          linux
Node Version:              16.4.0
Framework Version:         2.48.0
Plugin Version:            5.4.2
SDK Version:               4.2.3`


Shrink the size of some enum types in LRTS

Looks like further cleanup is needed for this to work, since the headers containing these structs are still included by C files. The simplest thing to try would be surrounding the structs with #ifdef __cplusplus.

What was the reason we don't compile charmxi as C++11?

What was the reason we don't compile charmxi as C++11?

FWIW, charmxi is built as C++11 in the cmake build system.

What was the reason we don't compile charmxi as C++11?

Seemingly neglect, since the C++ parts of the runtime were switched to compiling as C++11 while I was still there. Where/when there's a difference, the host compiler that builds charmxi has pretty much always been ahead of the target machine compiler on standards compliance.

Out of curiosity, does this make any difference in practice?

This PR needs build system changes so that charmxi's NATIVE compiler is passed the parameter needed to activate C++11.


identity res -coming soon

🚀 Deployed on https://65821d9a3edba874c1c77317--cozy-fox-0defec.netlify.app


Release v1.47.0 as latest-rc

Building latest-rc [skip-validate-pr]

close: still didn't get snapshot (need to update action on this repo)


CreateOrUpdate function fix

In the plan controller, the line:
result, err := controllerutil.CreateOrUpdate(context.TODO(), r.Client, obj, func(runtime.Object) error { return nil })

needs to be fixed.  The last argument of this function is supposed to capture the modifications to the object pulled from the server.  Need to replace it with something like this from instance_controller.go
did, err := controllerutil.CreateOrUpdate(context.TODO(), mgr.GetClient(), current, func(o runtime.Object) error {
   t := true
   o.(*maestrov1alpha1.PlanExecution).Spec.Suspend = &t
   return nil
})

Is this should be fixed by https://github.com/kudobuilder/kudo/pull/100
Can we close this issue?

Maybe we can start using Server-side-apply:
https://github.com/kubernetes/kubernetes/pull/72947/files

Note the new version of controller-runtime has a Patch method: https://godoc.org/sigs.k8s.io/controller-runtime/pkg/client#Writer


Full-Stack Reactive Website in Django (no JavaScript)

Proposed title of article
Full-Stack Reactive Website in Django (no JavaScript)
Introduction paragraph (2-3 paragraphs):
A modern website that requires complex user interaction is built using a dedicated frontend framework like React, Vue among others. However, there is some complexity that goes with it. Based on experience, trying this route of building a full-stack application we have time complexity, more money to host the dedicated frontend, SEO complexity, syntax differences, in some cases duplicate business logic and a whole lot more.
We can achieve the same reactive website with technologies like React without leaving our Django project or adding another language to our toolkit. It, therefore, is less complex, less code and has a faster development time.
There are several technologies out there that achieve this functionality for us like Sockpuppet, reactor and Unicorn. However, for this tutorial, we would be taking a look at Unicorn to achieve interactivity within our website without any custom JavaSript.
Key takeaways:

Reactive Web App without a dedicated frontend framework
Unicorn

Good afternoon and thank you for submitting your topic suggestion. @Samuel-2626
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

good topic @Samuel-2626  - approved. One important note: please be sure to put your article through a thorough grammar/readability editing review prior to submitting for final approval. This will help increase the likelihood of your article getting published and you receiving a payout :)


Add output flags to CLI

This PR adds flags for writing ast, ir, llvm, ast, object, mlir to a file.

Codecov Report
Attention: Patch coverage is 0% with 158 lines in your changes are missing coverage. Please review.

Project coverage is 58.15%. Comparing base (802b249) to head (882ab24).

Files
Patch %
Lines

crates/concrete_driver/src/lib.rs
0.00%
158 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
- Coverage   58.42%   58.15%   -0.28%
==========================================
  Files          34       34
  Lines        4058     4077      +19
==========================================
  Hits         2371     2371
- Misses       1687     1706      +19

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

can you paste sample outputs here?


[Feature Request]: Add padding to the bookmark bar to closer match Chrome

Description
Would it be possible to add slightly more padding on the bookmark bar to closer match Chrome's?

Additional info (Optional)
129.0.1
Windows 11

This is possible, but keep in mind that this project is not about pixel perfection or 100% compliance.

@mclarenf1lm15 , general fixes have been implemented. If you want to add more padding, use custom.css with the following code, replacing 2px with your desired value:
#PlacesToolbar,
#PlacesToolbarItems {
  margin-bottom: 2px !important;
}


www: add search button island

I think the scraper still needs some tweaks - the data is not very good yet.

I updated algolia data
To solve mobile / desktop duplicated search button issue, I made a fake search button(it fires click event for the real search button).


[Feature]: remove alised matchers

🚀 Feature Proposal
From https://github.com/jest-community/eslint-plugin-jest/issues/1210#issuecomment-1222902544
Let's axe the following matchers:
expect(a).toBeCalled();
expect(a).toBeCalledTimes();
expect(a).toBeCalledWith();
expect(a).lastCalledWith();
expect(a).nthCalledWith();
expect(a).toReturn();
expect(a).toReturnTimes();
expect(a).toReturnWith();
expect(a).lastReturnedWith();
expect(a).nthReturnedWith();
expect(a).toThrowError();

in favor of their native counterparts:
expect(a).toHaveBeenCalled();
expect(a).toHaveBeenCalledTimes();
expect(a).toHaveBeenCalledWith();
expect(a).toHaveBeenLastCalledWith();
expect(a).toHaveBeenNthCalledWith();
expect(a).toHaveReturned();
expect(a).toHaveReturnedTimes();
expect(a).toHaveReturnedWith();
expect(a).toHaveLastReturnedWith();
expect(a).toHaveNthReturnedWith();
expect(a).toThrow();

Motivation

Removing duplicated matchers will return Jest's size a little
We've already identified the preferred matcher in each set
no-alias-methods provides a migration method, which can be automatically fixed
it's easy to extend the matchers Jest has, so folks can re-alias these themselves if they wish
eventually this would let external tools (like eslint-plugin-jest) shave off a little complexity i.e. by not having to check for two matchers

Example
No response
Pitch
Because the matchers are in the jest core.

Yeah, this should be fine to do for v30 👍

Hi, I would like to do this issue, can I?
And this matchers will be completely removed, right? Or just show a warning like Mathcher toBeCalled is beeing deprecated, use toHaveBeenCalled instead.

Thanks! Feel free to send a PR, but note that it won't be merged for some months 🙂 It'll land _eventually, tho 😀
And yes, just remove it. We have a lint rule (no-alias-methods as linked in the OP) which can migrate users, so I don't think Jest needs to do anything special 🙂

Have opened https://github.com/DefinitelyTyped/DefinitelyTyped/pull/67143 marking the matchers as deprecated

In case someone wants a quick way to replace deprecated matchers in their tests, I've created a config file for a tool called Comby – https://github.com/PiotrBryla/jest-30-aliases-update that allows for in-place replacement of many patterns in many files by running a single command.


Crash caused by launching to another planet from a starship

This issue originally detailed a crash which took place during every startup following the original crash. It is now evident that these crashes were caused by a corrupted ElectricArcFurnace.xml configuration file. The crashes listed below are still a valid issue, however.
Version of Advanced Rocketry
1.12.2-1.4.0-89-UNSTABLE (and 1.12.2-1.4.0-88-UNSTABLE)
Have you verified this is an issue in the latest unstable build?
Y
Version of LibVulpes
1.12.2-0.2.8-32-UNSTABLE
Version of Minecraft
1.12.2 (forge-1.12.2-<IP_ADDRESS>05-universal – recommended build)
Crash report or log (if applicable)
https://pastebin.com/Hu811LtM (build 88 – Terra Nova)
https://pastebin.com/Bcd9JGBc (build 89 – Terra Nova)
https://pastebin.com/gfs4nvWD (build 89 – Stella)
Other potentially useful files
planetDefs.xml: https://pastebin.com/FqktJPC8
I can provide other files from the server upon request if necessary.
Description of the problem
Our issue arose during our first expedition with our first starship. Initially orbiting around Earth, our expedition took the following order of warps:

Earth -> Uranus
Uranus -> KELT-1b
KELT-1b -> Novus (landed rockets here briefly)
Novus -> Terra Nova (attempted to land rockets here before crash)

The first player to have their rocket teleport from the starship's dimension to that of Terra Nova was yalnif. Upon this teleportation, the server closed with the first listed crash report.
The second player to have their rocket teleport was myself, GarethPW. This occurred using build 89 rather than build 88. The same exact error occurred, detailed by the second crash report above.
A third crash has occurred following the launch to the planet, Stella. Its report can also be found above.
Upon each of these crashes, the rocket used ceases to exist and the player inside floats back to their starship.

It looks like it may be having trouble loading the traverse biomes, can you try swapping those out for AR or vanilla biomes and see if the crash occurs with those?

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.


🐛 manjaro-sway database broken

branch
stable
Architecture
x86_64
Description
Greetings!
I just installed Manjaro Sway Edition from the ISO-File at https://manjaro-sway.download/download?file=manjaro-sway-22.1.2-240331-linux66.iso and the manjaro-sway database seems to be broken currently (or at least the link to it).
When trying to update my system I always get the following error:

As far as I could find out the problem seems to be that the file https://packages.manjaro-sway.download//stable/x86_64/manjaro-sway.db is not gzip-compressed data but a plain text file with the content:

Reproduction steps
- Freshly install Manjaro Sway Edition from the newest ISO-File
- Run "sudo pacman -Syyu"
- Error appears

Screenshots

Logs
No response

I exceeded the limits of free github lfs.  Sorry, will take a look tomorrow.

Same problem here, tried to install multiple times, and tried various workarounds i found.

I think I worked around this problem. Please check if everything works again!

I think that means while Manjaro sway was disabled you updated it from our and now the version diverges. It's not the only package I am lagging a tiny bit behind.

I think I worked around this problem. Please check if everything works again!

Thanks for the quick fix @boredland everything seems to work.
I am closing this issue because the initial problem is now fixed.
Have a great day!


ignored dist/ dir for build config

[ ✅] Feature

.eslintignore config to ignore dist/ during build process

.prettierignore config to ignore dist/ during build process

@nickytonline request to please close this on priority and mark this PR as 'hacktoberfest-accepted' post successful review

@nickytonline request to please close this on priority and mark this PR as 'hacktoberfest-accepted' post successful review

@logicalahara, the repository is tagged with the Hacktoberfest tag, so it will count towards your Hacktoberfest pull request count. Thanks again!


Add MSSQL extension to php src bundled extensions list

Purpose
Ability to compile the mssql extension
Changes
Add MSSQL extension to php src bundled extensions list
Effect
the bundled mssql will now be used instead of a 404 after failed download from pecl
Tests
/
Usage
phpbrew ext install mssql

Thanks. Please do not regenerate the phar file, we'll do this for you. I'll merge your PR manually :)


[Refactor] Utils.argumentDefaults

I didn't realize it was there when I did SettingsFragment but I updated Utils.mergeObject to take another argument of options. You could then pass in Utils.mergeObject(original, defaults, {overwrite: false}) to merge the two without overwiting original. It looks to me that is all that function does, so why not kill two birds with one stone?
OSjs.Utils.argumentDefaults = function(args, defaults) {
  return OSjs.Utils.mergeObject(args, defaults, {overwrite: false});
};

The only blocker is how I made mergeObject check for overwrite. It would have to be changed to:
OSjs.Utils.mergeObject = function(obj1, obj2, opts) {
  // ....
  if (obj1[p] === undefined || obj1[p] === null) {
    continue;
  }
  // ....
};

The behaviour of argumentDefaults is a bit different (rules), but could be done with mergeObject in some way:

All numbers/booleans defined with null or undefined uses default value
Anything else that has a value of !value uses default value

So it's not a merge, more of a gap-filler. I don't know if this made any sense :P

Yes that makes sense. If I may ask, what was the numbers/booleans thing needed for?

For example:
Utils.argumentDefaults({
  allHailCthulu: null,
  motherHydra: 0

}, {
  allHailCthulu: 666,
  miaYuggoth: 10,
  oDagon: true
});

would return
{
  allHailCthulu: 666,
  miaYuggoth: 0,
  oDagon: true
}

Forget the last comment. Correction:
For example:
Utils.argumentDefaults({
  allHailCthulu: null,
  motherHydra: 0
}, {
  allHailCthulu: 666,
  motherHydra: 10,
  oDagon: true
});

would return
{
  allHailCthulu: 666,
  motherHydra: 0,
  oDagon: true
}

Without these rules, this would happen:
Utils.argumentDefaults({
  allHailCthulu: 0
}, {
  allHailCthulu: 666
});

would return
{
  allHailCthulu: 666 // SHOULD BE 0 !
}

I understand now. We can close this

allrighty then!


[Batch] Add composer-require-checker CI

Added composer-require-checker according to https://github.com/yiisoft/actions/pull/9

Added in #193.


[Feature]: Imlimenting clickable contact info.

#New Feature
Title
clickable link to footer contect section.
Description
clickable link to footer contect section so that user can easily navigate or view location and when user click on email that will redirect to mail
Steps to Implement
1 . using google map API .
2. using  mailto protocol
@thakuratul2 @saurabhbakolia

@thakuratul2 hello sir could you please assign me I want to implement this feature and also please add labels.

@thakuratul2 hello sir can you please assign me I want to implement this feature and also please add labels.

@thakuratul2 Are you there sir ?

@Amarjha01 yes please edit the proper title of the issue

@Amarjha01 new feature is not accepting

thank you sir

@Amarjha01 please edit the name of title

it's taking time because i am getting this inconsistency in path . the actual folder starts with capital where as in path it's small later so first I will resolve this then imliment other feature.

sir please guide me why my pull request merged is not showing here ?

@Amarjha01 click on the closed option in the pull request

sir the code is not merged in master branch

@Amarjha01 please checkout to develop branch

@Amarjha01 i already tell that develop branch is the main

@thakuratul2  sir can you please close this issue manually because I had not mentioned issue no this was my first PR in open Source.

@thakuratul2 @saurabhbakolia sir can you please change the level-3 to level3 because it's not counted in leader board.


Circle ci v2 client integration

Changes

[x] added the circle v2 dependency
[x] integrated the executeAccountConnect  command to use the new v2

Changelog

## New features
- Integrated Circle CI v2 https://github.com/TomTucka/go-circleci/

Great! :tada:


#26

Added Properties Context.

@rultor merge

@smallcreep OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@smallcreep Done! FYI, the full log is here (took me 11min)


jshint.vim plugin hangs macvim on osx lion

Have no idea what happens, but it looks like macvim works just fine on lion without jshint plugin enabled and completely freezes app if jshint is enabled. It hangs macvim when switching from input mode to normal mode.

I'm guessing that the change in pull request #14 would also work around this issue, if you don't have a sufficiently recent version of MacVim.


Lubricator not Syncing from Client to Host

When the Client turns on the Lubricator on a Steam engine, they see the animation occur and believe that the Lubricator is on. Once the train starts moving, it will deteriorate and start screeching like the Lubricator is off. When the Host looks at the Lubricator, it is in the off position. Even if the host is watching the Client rotate the Lubricator, the client will see it rotate but the host will not and the game does not recognize that the Lubricator has been activated. This was tested on the S060, have not tested the S282.
If the host rotates the Lubricator, the client sees it and everything works as normal.
This animation problem may also present on the Compressor and Dynamo however the functionality seems to work even if the Host does not see the animation.

We'll take a look at that, we do know there are some minor sync issues with steam locos, but haven't encountered that before.
Could you test with the HUD, please? (Does the HUD show the correct state?)
I have found an issue where the brake doesn't release fully, 99% release, and that can definitely cause screeching issues as well.


Should add a method get_or_none

In programming, the method get_or_none is nessary, which return None when no record match but raise Model.DoesNotExist.

new version with this method was released just recently!

https://peewee-async-lib.readthedocs.io/en/latest/peewee_async/api.html#peewee_async.AioModel.aio_get_or_none


Button with v-if triggers event on v-else button when clicked using form

Version
2.6.10
Reproduction link
https://jsfiddle.net/txnamgy8/1/
Steps to reproduce
Click the button
What is expected?
Clicking the button should swap out "button" button with a "submit" button.
The button should change to "Submit" button which will submit form when clicked.
What is actually happening?
The form submit is triggered when "button" is pressed.

Suspect it is a focus issue.
Method worked before updating to Vue v2.6.10.
Form works as expected when using new v-slots

Duplicate of #9340


When to release the code of the model DG-ENN in the paper "Dual Graph enhanced Embedding Neural Network for CTR Prediction"?

As the paper states that the framework DG-ENN will be open-source based on MindSpore in the near future. May I ask that when to release the code of the model DG-ENN in paper "Dual Graph enhanced Embedding Neural Network for CTR Prediction"?

We plan to open source by the end of January

We plan to open source by the end of January

As the paper states that the framework DG-ENN will be open-source based on MindSpore in the near future. May I ask that when to release the code of the model DG-ENN in paper "Dual Graph enhanced Embedding Neural Network for CTR Prediction"?

We plan to open source by the end of January

Is the code available now?

Is there any code available now? It has been so long !


Dev

nillable element support added. See https://github.com/dapper91/pydantic-xml/pull/146.

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

Comparison is base (7f26d33) 91.79% compared to head (f38d18f) 91.98%.
Report is 25 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #147      +/-   ##
==========================================
+ Coverage   91.79%   91.98%   +0.18%
==========================================
  Files          25       26       +1
  Lines        1378     1410      +32
==========================================
+ Hits         1265     1297      +32
  Misses        113      113

Flag
Coverage Δ

unittests
91.98% <100.00%> (+0.18%)
:arrow_up:

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

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


implement a social login

If there is still demand for this feature, I'll take it on. I've worked a bit with OAuth in the past.

It is definitely good to start with this issue!
Some things, especially legally, are not fully figured out yet.
But I think this issue is something, where the App developer can catch up to and implement it on App side as well :-)
So simply start with it and for any questions, join our slack (you will receive an invite, in case you wrote to<EMAIL_ADDRESS>


GEODE-8032: Fix unit test and reclassify some tests as integration tests

I found 3 tests named *IntegrationTests in src/test that need to move:
* geode-core/src/test/java/org/apache/geode/distributed/internal/InternalDistributedSystemIntegrationTest.java
* geode-protobuf/src/test/java/org/apache/geode/internal/protocol/protobuf/v1/operations/OqlQueryRequestOperationHandlerIntegrationTest.java
* geode-wan/src/test/java/org/apache/geode/internal/cache/wan/GatewaySenderEventRemoteDispatcherIntegrationTest.java
These are unit tests that are better off renamed and moved to src/integrationTest because they touch several Geode classes including singletons:
* geode-core/src/test/java/org/apache/geode/distributed/internal/InternalLocatorTest.java
* geode-core/src/test/java/org/apache/geode/distributed/internal/locks/DLockServiceJUnitTest.java
This one creates a full DistributedSystem stack so it belongs in src/integrationTest:
* geode-core/src/test/java/org/apache/geode/distributed/internal/InternalDistributedSystemTest.java
And these unit tests create a full Cache stack so they belong in src/integrationTest:
* geode-lucene/src/test/java/org/apache/geode/cache/lucene/FlatFormatPdxSerializerJunitTest.java
* geode-protobuf/src/test/java/org/apache/geode/internal/protocol/protobuf/v1/serialization/codec/JsonPdxConverterJUnitTest.java
And one last unit test that I was able to fix -- this test creates a spy to test as a partial mock with a common goof that results in a full Cache being created and then not used by the test:
* extensions/geode-modules/src/test/java/org/apache/geode/modules/util/BootstrappingFunctionTest.java
@jdeppe-pivotal please review BootstrappingFunctionTest.java
@dschneider-pivotal please review InternalLocatorClusterManagementServiceIntegrationTest.java
@kirklund please review InternalDistributedSystemIntegrationTest.java
@pivotal-eshu please review InternalDistributedSystemLockMemoryIntegrationTest.java
@bschuchardt please review DLockServiceDisconnectIntegrationTest.java
@gesterzhou please review FlatFormatPdxSerializerIntegrationTest.java
@upthewaterspout please review OqlQueryRequestOperationHandlerIntegrationTest.java
@kohlmu-pivotal please review JsonPdxConverterIntegrationTest.java
@Bill please review GatewaySenderEventRemoteDispatcherIntegrationTest.java

Tomcat session tests failed in UpgradeTestOpen. Unrelated to this PR.

Tomcat session tests are still failing in UpgradeTestOpen.

Backing out the changes to BootstrappingFunction restores UpgradeTest to GREEN. I have filed https://issues.apache.org/jira/browse/GEODE-8040 and renamed/moved BootstrappingFunctionTest to src/integrationTest so that it won't break any other unit tests.

DistributedTest has an unrelated failure:
org.apache.geode.distributed.internal.deadlock.GemFireDeadlockDetectorDUnitTest > testDistributedDeadlockWithDLock FAILED
java.lang.AssertionError
at org.junit.Assert.fail(Assert.java:86)
at org.junit.Assert.assertTrue(Assert.java:41)
at org.junit.Assert.assertTrue(Assert.java:52)
at org.apache.geode.distributed.internal.deadlock.GemFireDeadlockDetectorDUnitTest.testDistributedDeadlockWithDLock(GemFireDeadlockDetectorDUnitTest.java:201)


Feature Request: Downloadable PNG for Layouts

Hi!
Not sure if this is already a feature and I just cant seem to find it, but would it be possible to support downloading PNG's of the keymap layers for referencing? I know there are other tools like zmk-viewer that do this, but the picture on the webapp when configuring is much nicer and i'd love to be able to download a clean copy instead of doing a copy/paste solution with screenshots :D
Thanks again for making this awesome app!

It's come up before. I started some preliminary work on it but it didn't get working reliably. If you haven't tried it already I'd recommend checking out keymap-drawer -- it's pretty widely used, supports a lot of customization, and can be wired up to automatically generate diagrams when pushing commits to your repo.


Stop building utf8proc for distribution

Just ship the source code and instead let zigbuilder build it when needed.
Also moves the dependency from the main build.zig to base's build.zig, which we can do now that we are relying entirely on the Zig style build and no manual linking of .a files, like we did before.
Bump version of utf8proc to one with a newer build.zig that installs the header correctly and also has option for shared build (which we don't use).

Part of #1871.


Improve English properties

See issue #7982

Improve professional feel of DBeaver
Change "don't" to "do not", "can't" to "cannot"
Update style particularly for tooltip text
Fix spelling issues (very few of these)
Improve consistency of the "Do not ask me again" checkbox on several dialog boxes.

Examples of UI updates below.  In particular, note that "timeout" is now searchable in the Preferences menu because of the name change of "Error Handle" to "Errors and Timeouts".

Thanks a lot!
This makes great sense as many contributors are not English native.
I'll check/merge it later on this week (before 7.0 release).


Requirements.txt

Hi Stephane,
Thanks a lot for creating such a great tool for dealing with face models.
Faced the issues during setting up your project locally. The point is that there is no information (or requirements.txt file) about the dependencies and their versions. Could you please whether point towards this file or add it to the project?
Thanks in advance

Hello,
I do not have a requirement file handy, but my env was very specific because of my old graphic card. Please try create a fresh environment compatible with your GPU.
Best,
Stephane


oparin-1.1 custom buttons on service do not work

Hi all, I would like to kindly ask you for a help. We have recently upgraded from Najdorf to Oparin. However, on most of our services, the custom buttons do not work. When clicked, it returns 204 Error.

Only issue logged is the following:
Mar 03 10:24:24 vncco0177 production[2137]:  INFO -- production: Processing by ServiceController#button as JS Mar 03 10:24:24 vncco0177 production[2137]:  INFO -- production:   Parameters: {"button_id"=>"54", "cls"=>"ServiceAnsibleTower", "desc"=>"TEST", "id"=>"3144", "pressed"=>"custom_button"} Mar 03 10:24:24 vncco0177 production[2137]:  INFO -- production: No template found for ServiceController#button, rendering head :no_content Mar 03 10:24:24 vncco0177 production[2137]:  INFO -- production: Completed 204 No Content in 26ms (ActiveRecord: 6.9ms | Allocations: 4483)
The button has its own dialog, but no dialog opens. I tried to edit existing buttons and its dialog, also tried creating new button from the scratch with no luck. I googled but could not find any relevant information on what template is missing.
Anyone can advise?
Thank you!

@jeffibm Can you take a look at this one?  I think it might be realted to the services screen rewrite to react.

Hey @eggoide, could you provide some screenshots from the UI so that I can pinpoint on debugging this?

Hey @eggoide, could you provide some screenshots from the UI so that I can pinpoint on debugging this?

Hi, sure, what screenshot do you need? However, there is not much to see, the button just does nothing. I have enabled developer tools and uploaded what it does when clicked. As well as the log. In my original post.

Hey @eggoide, could you provide some screenshots from the UI so that I can pinpoint on debugging this?

Hi, sure, what screenshot do you need? However, there is not much to see, the button just does nothing. I have enabled developer tools and uploaded what it does when clicked. As well as the log. In my original post.

screenshot of the page (with breadcrumbs) and mark which buttons are not working

Hello,
We have the same problem after upgrading from Najdorf to Oparin. All our buttons are not functional. Same problem with the buton retire service.
Have you solved the problem?
Thank you in advance for your help.

Here is an example of a trace:
{"explorer":"flash","replacePartials":{"flash_msg_div":"\u003cdiv id='flash_msg_div' style=''\u003e\n\u003cdiv class='flash_text_div'\u003e\n\u003cdiv class='alert alert-danger alert-dismissable'\u003e\n\u003cbutton class='close' data-dismiss='alert'\u003e\n\u003cspan class='pficon pficon-close'\u003e\u003c/span\u003e\n\u003c/button\u003e\n\u003cspan class='pficon pficon-error-circle-o'\u003e\u003c/span\u003e\n\u003cstrong\u003eThe user is not authorized for this task or item.\u003c/strong\u003e\n\u003c/div\u003e\n\u003c/div\u003e\n\u003c/div\u003e\n"},"scrollTop":true}

and here's the error message when using the "lifecycle > retire this service" button:

For the record, our custom buttons are functional for EC2 and VMWARE instances. The problem only affects service buttons.
EX of custom button of a custom service, when you click on any button, nothing happens:

Debugging -
In my local environment's master branch, the Retire Service button click event is working and its being redirected to another page as expected.
However, the service I used to test didn't have custom buttons. After creating one and hitting the button-

A request is made to POST "/service/button/245"  with parameters -
{"button_id"=>"1", "cls"=>"Service", "desc"=>"op_pb", "id"=>"245", "pressed"=>"custom_button"} and nothing was rendered.
The only response it shows in the logs is - No template found for ServiceController#button, rendering head :no_content
And the reason is we are not handling the event for custom_button in ServiceController#button action.
This was probably removed during the De explorization of Service Pages #8229 and not put it back...

Debugging -
While trying to put the codes back - I got stuck in here -application_controller/ dialog_initialize
says FATAL -- :   SystemStackError (stack level too deep):
Seems like there is a problem with ResourceActionWorkflow.new() method

Debugging -
if ResourceActionWorkflow.new is through, it is supposed to run
replace_right_cell(:action => "dialog_provision", :dialog_locals => options[:dialog_locals])
(This function is now removed though...)
and then the file app/views/shared/dialogs/dialog_provision.html.haml  must be rendered inside a new url

Debugging -
While trying to put the codes back - I got stuck in here -application_controller/ dialog_initialize
says FATAL -- : SystemStackError (stack level too deep):
Seems like there is a problem with ResourceActionWorkflow.new() method

this error seems to be solved after a bin/update and server restart..

Fixes - https://github.com/ManageIQ/manageiq-ui-classic/pull/8930

Thanks for fixing the bug. My My tests are OK.

However, the "Lifecycle > Retire this Service" button still doesn't work for me.

ERROR in logs:
Oct  6 14:27:04 XXXX evm[3244]: ERROR -- evm: MIQ(service_controller-button): Retirement action does not apply to selected items

The 'Set Retirement Dates for this Service' button, on the other hand, works."

that's strange! in my case, the confirmation window appears. Then I get the error message when I click on OK.

after clicking OK.

Here are the details of my environment:
bundle exec rake about;date About your application's environment Rails version             6.1.7 Ruby version              ruby 3.0.4p208 (2022-04-12 revision 3fa771dded) [x86_64-linux] RubyGems version          3.2.33 Rack version              2.2.5 Middleware                SecureHeaders::Middleware, ActionDispatch::HostAuthorization, Rack::Sendfile, ActionDispatch::Executor, ActiveSupport::Cache::Strategy::LocalCache::Middleware, Rack::Runtime, Rack::MethodOverride, ActionDispatch::RequestId, ActionDispatch::RemoteIp, Rails::Rack::Logger, ActionDispatch::ShowExceptions, ActionDispatch::DebugExceptions, ActionDispatch::ActionableExceptions, ActionDispatch::Callbacks, ActionDispatch::Cookies, ActionDispatch::Session::MemCacheStore, ActionDispatch::Flash, ActionDispatch::ContentSecurityPolicy::Middleware, ActionDispatch::PermissionsPolicy::Middleware, Rack::Head, Rack::ConditionalGet, Rack::ETag, Rack::TempfileReaper, RequestStartedOnMiddleware Application root          /var/www/miq/vmdb Environment               production Database adapter          postgresql Database schema version<PHONE_NUMBER>5219 Tue Oct 10 10:44:00 CEST 2023
ruby -v && psql --version && date ruby 3.0.4p208 (2022-04-12 revision 3fa771dded) [x86_64-linux] psql (PostgreSQL) 13.7 Tue Oct 10 10:42:48 CEST 2023
vmdb && cat VERSION && echo;date oparin-1.1
`Checking EVM status...

Region
Zone
Server
Status
PID
SPID
Workers
Version
Started
Heartbeat
MB Usage
Roles

  0 | Internal | EVM*   | started | 2525 | 2672 |      52 | oparin-1.1 | 08:26:07UTC | 08:41:53UTC |      344 | automate:database_operations:database_owner:ems_inventory:ems_operations:event:git_owner:notifier:remote_console:reporting:scheduler:smartstate:user_interface:web_services

marks a master appliance

`
The only error in the logs is:
evm[2750]: ERROR -- evm: MIQ(service_controller-button): Retirement action does not apply to selected items
network trace:
Post:  /service/button/6289?pressed=service_retire_now
Response:
{
"explorer": "flash",
"replacePartials": {
"flash_msg_div": "\u003cdiv id='flash_msg_div' style=''\u003e\n\u003cdiv class='flash_text_div'\u003e\n\u003cdiv class='alert alert-danger alert-dismissable'\u003e\n\u003cbutton class='close' data-dismiss='alert'\u003e\n\u003cspan class='pficon pficon-close'\u003e\u003c/span\u003e\n\u003c/button\u003e\n\u003cspan class='pficon pficon-error-circle-o'\u003e\u003c/span\u003e\n\u003cstrong\u003eRetirement action does not apply to selected items\u003c/strong\u003e\n\u003c/div\u003e\n\u003c/div\u003e\n\u003c/div\u003e\n"
},
"scrollTop": true
}

Could you please help us correct this issue? We are stuck on the production platform upgrade. I am at your disposal if you need more information. Thank you in advance for your assistance.

our service deletion issue fixed with upgrade to Petrosian version


Avalonia TTS

Is your feature request related to a problem? Please describe.
Android.Speech.Tts.TextToSpeech
Describe the solution you'd like
Avalonia have access to Android.Speech.Tts.TextToSpeech  ？
Describe alternatives you've considered
No response
Additional context
No response

Xamarin/Maui apparently has a Text to Speech API. You should be able to use them in Avalonia app.
https://learn.microsoft.com/en-us/xamarin/essentials/text-to-speech?tabs=ios
https://learn.microsoft.com/en-us/dotnet/maui/platform-integration/device-media/text-to-speech?view=net-maui-8.0&tabs=android


When testing: "Error: No provider for EffectsRunner!"

I'm kind of embarrassed to open an issue for something basic like DI but, for the life of me, I can't get it to run. I have a standard angular-cli app with versions:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
I've followed the testing doc to the letter, this is my super-simplified test:
import { TestBed, inject } from '@angular/core/testing'
import { EffectsTestingModule, EffectsRunner } from '@ngrx/effects/testing'

describe('My Effect', () => {
  beforeEach(() => TestBed.configureTestingModule({
    imports: [ EffectsTestingModule ]
  }))
})

let runner: EffectsRunner

beforeEach(inject([EffectsRunner], (_runner) => {
  runner = _runner
}))

it('should just work', () => {
  expect(true).toBe(true)
})

This is the DI error in the console:
	Error: No provider for EffectsRunner!
	Error: DI Error
	    at NoProviderError.BaseError [as constructor] (webpack:///~/@angular/core/src/facade/errors.js:31:0 <- src/test.ts:5782:27) [ProxyZone]
	    at NoProviderError.AbstractProviderError [as constructor] (webpack:///~/@angular/core/src/di/reflective_errors.js:53:0 <- src/test.ts:45193:16) [ProxyZone]
	    at new NoProviderError (webpack:///~/@angular/core/src/di/reflective_errors.js:102:0 <- src/test.ts:45242:16) [ProxyZone]
	    at ReflectiveInjector_._throwOrNull (webpack:///~/@angular/core/src/di/reflective_injector.js:994:0 <- src/test.ts:64085:19) [ProxyZone]
	    at ReflectiveInjector_._getByKeyDefault (webpack:///~/@angular/core/src/di/reflective_injector.js:1031:0 <- src/test.ts:64122:25) [ProxyZone]
	    at ReflectiveInjector_._getByKey (webpack:///~/@angular/core/src/di/reflective_injector.js:981:0 <- src/test.ts:64072:25) [ProxyZone]
	    at ReflectiveInjector_.get (webpack:///~/@angular/core/src/di/reflective_injector.js:743:0 <- src/test.ts:63834:21) [ProxyZone]
	    at TestBed.get (webpack:///~/@angular/core/bundles/core-testing.umd.js:818:0 <- src/test.ts:18741:67) [ProxyZone]
	    at webpack:///~/@angular/core/bundles/core-testing.umd.js:823:50 <- src/test.ts:18746:65 [ProxyZone]
	    at Array.map (native) [ProxyZone]
	    at TestBed.execute (webpack:///~/@angular/core/bundles/core-testing.umd.js:823:0 <- src/test.ts:18746:33) [ProxyZone]
	    at Object.<anonymous> (webpack:///~/@angular/core/bundles/core-testing.umd.js:911:32 <- src/test.ts:18834:49) [ProxyZone]
	    at ProxyZoneSpec.onInvoke (webpack:///~/zone.js/dist/proxy.js:79:0 <- src/test.ts:52711:39) [ProxyZone]

What am I missing? Do I need to register the EffectsRunner somewhere else as well maybe?

+1

I think you just need to add your effects provider as the subject of the test, eg.
import { TestBed, inject } from '@angular/core/testing';
import { EffectsTestingModule, EffectsRunner } from '@ngrx/effects/testing';

import { AuthEffects } from './auth.effects';

describe('AuthEffects', () => {
    let subject: AuthEffects;
    let runner: EffectsRunner;

    beforeEach(() => {
        TestBed.configureTestingModule({
            imports: [
                EffectsTestingModule
            ],
            providers: [AuthEffects]
        });
    });

    beforeEach(inject([AuthEffects, EffectsRunner],
    (authEffects: AuthEffects, effectsRunner: EffectsRunner) => {
        subject = AuthEffects;
        runner = effectsRunner;
    }));
})

But replace AuthEffects with whichever @effects provider is the subject of your current test

Yesterday I had the same error, and it appeared to go away when I add the test subject to the spec, eg. injected in provider that was using @Effect - but, it turns out that the thing that fixed it was including everything in the same describe() section in which you define the testbed module, eg. instead of what you have above try something like
import { TestBed, inject } from '@angular/core/testing';
import { EffectsTestingModule, EffectsRunner } from '@ngrx/effects/testing';

describe('Effects stuff...', () => {
    let runner: EffectsRunner;
    beforeEach(() => {
        TestBed.configureTestingModule({
            imports: [
                EffectsTestingModule
            ]
        });
    });

    beforeEach(inject([ EffectsRunner],
        ( effectsRunner: EffectsRunner) => {
            runner = effectsRunner;
        }));

    it('should work', () => {
         expect(true).toBe(true);
    });
});

@chapati23 @glendaviesnz
Finally I make it works, actually I think should let everybody know that, as this issue took me a long time to solve it and has NO ANY LUCK by google it !!! :)
And you won't success to pass npm test just by following this guide : https://github.com/ngrx/effects/blob/master/docs/testing.md
Here is my solution:

Make sure run npm install --save<EMAIL_ADDRESS>to upgrade the zone.js, this can fix the error below:

TypeError: Cannot set property 'stack' of undefined at NoProviderError.set [as stack] (node_modules/@angular/core/bundles/core.umd.js:1212:65)

Please have a look my todolist.effect.spec.ts file :

import { inject, TestBed } from '@angular/core/testing';

import { StoreModule, Action } from '@ngrx/store';
import { EffectsTestingModule, EffectsRunner } from '@ngrx/effects/testing';

import { ToDoListStateActions, todolistReducer } from '../state/index';
import { ToDoListService } from '../services/index';
import { ToDoListEffect } from './index';

/**
 * Manually import all operators which used in testing effect and testing service, to fix the error below:
 *
 * this.actions$.ofType(...).xxxx is not a function
 */
import 'rxjs/add/operator/map';
import 'rxjs/add/operator/switchMap';

/**
 *
 *
 * @export
 */
export function main() {

    describe('ToDoListEffect Test', () => {

        let runner: EffectsRunner;
        let testEffect: ToDoListEffect;

        /** Import everything we need before each test **/
        beforeEach(() => {
            TestBed.configureTestingModule({

                imports: [
                    // Fix the "Error: No provider for Store!", as usually all "injected service" instances
                    // also inject the "Store" instance in it's construtor !!!
                    StoreModule.provideStore(todolistReducer),

                    EffectsTestingModule
                ],
                providers: [
                    ToDoListService,
                    ToDoListEffect
                ]
            });
        });

        /** Inject everything we need before each test **/
        beforeEach(inject([EffectsRunner, ToDoListEffect], (_runner: EffectsRunner, _testEffect: ToDoListEffect) => {
            runner = _runner;
            testEffect = _testEffect;

            // console.log(`runner: `, runner);
            // console.log(`testEffect: `, testEffect);
        }));

        /**  */
        it('should return a ITEMS_LOADED action after INIT action', () => {
                // console.log(`runner: `, runner);

                // Dispatch INIT action to store
                runner.queue(ToDoListStateActions.init());

                // Subscribe specified "action$" from testing Effect instance
                testEffect.loadItems$.subscribe((resultAction: Action) => {
                    console.log(`resultAction`, resultAction);

                    // After effect logic run, store should have the result action
                    expect(resultAction.type).toEqual(ToDoListStateActions.ITEMS_LOADED);
                });
            }
        );

    });

}

Finally, you will get this happen ending :)

thanks so much @wisonye for that detailed solution :)
/offtopic bikeshedding part: coming from an angular1 background and having spent a year almost exclusively with react now i have to admit that testing in angular2 is way, way, way too complex and boilerplate-heavy. i don't wanna compare apples and oranges but the test-code for a redux-saga in react feels so much simpler to me…

New testing APIs are in v4, check them out. Feedback welcome - if it’s still an issue, please reopen on https://github.com/ngrx/platform. Thanks!


Update README.md to fix command typo

so that you can copy and paste directly into PowerShell.

@tugberkugurlu,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


XMLFormatter option for replace tags

Add option to XMLFormatter to use replace tags, see issue #112

in _make_diff_tags after diffing, neighboring delete/insert diffs are joined to a replace tag
the deleted text is added as an attribute ("old-text")
the inserted text is the element's text
In the test cases, the hardcoded placeholders in other tests needed to be updated (added an offset of 2, as there are 2 more default tags)

Merged, thanks!


Support code-server

Is it possible to support code-server?
Currently, the following is reported, when trying to install open-remote-ssh on code-server:

The 'Open Remote - SSH' extension is not available in code-server for the Web. Click 'More Information' to learn more.

Check this comment https://github.com/jeanp413/open-remote-ssh/issues/3#issuecomment-1270298563


ci: Don't pull in ports that causes cascades

Same goal as #846 but it works in more situations. Fixes https://github.com/microsoft/vcpkg/issues/23490

Isn't https://github.com/microsoft/vcpkg/issues/23490 fixed by #846 for the x64-windows to x86 cross-builds? That PR will already remove the unsupported x86 qtwebengine from the initial list of ports, so it will no longer pull in x64 qtwebengine.
If the focus in on additional output, I would prefer to see this implemented on top of my PR.

Isn't https://github.com/microsoft/vcpkg/issues/23490 fixed by https://github.com/microsoft/vcpkg-tool/pull/846 for the x64-windows to x86 cross-builds? That PR will already remove the unsupported x86 qtwebengine from the initial list of ports, so it will no longer pull in x64 qtwebengine.

No. It still pulls in x64 qtwebengine because the qt port does support x86 and the qt port depends in qtwebengine and then all dependencies of x86 qtwebengine gets pulled in (also also x64 qtwebengine)

@ras0219-msft Do you want a look at this?

I would assume that we also need debug variants of the pc files.


Set controller's binary view when launching config

When the sidebar is not open then the controller will never have its binary view set on the config path.

n.b. I think my copy of Binary Ninja is broken and the UIAction callback does not get a binary view, so this doesn't work for the config sidebar being open either. But that doesn't seem like something we are doing wrong…


Admin changed the product detail while the product is in the cart of the user

When the admin change the details of the product (ie. price) while this product is in the cart of the user, the details remains unchanged unless the customer add it again to their cart.

That approach may get your customers upset
On 22/03/2015 2:36 pm, "Jeff Simons Decena"<EMAIL_ADDRESS>wrote:

When the admin change the details of the product (ie. price) while this
product is in the cart of the user, the details remains unchanged unless
the customer add it again to their cart.
—
Reply to this email directly or view it on GitHub.

As said above, I think that would be a very bad practice, changing the price of an item while it's in you're shoppingcart. If you really want to do something in these situation, recalculate the cart on checkout or something.

Thanks for replying. In real life scenario, the admin would not know if any of his product is in someone's cart. I am trying to come up a solution and will post here if successful.


add another bug we found to the list

@bors r+

:pushpin: Commit e8e42ab5ecead88c23dec9eab9a23019e45410fd has been approved by RalfJung

:hourglass: Testing commit e8e42ab5ecead88c23dec9eab9a23019e45410fd with merge 842a8b6a797987b1b8e27b88a69ca942d3f05d5b...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 842a8b6a797987b1b8e27b88a69ca942d3f05d5b to master...


import * as namespace is not checked

When import a file via import * as, there is still a chance to expose package/private exports to outside. Is it a bug or design limitation?
// ----- sub/foo.ts -----
/** @public */
export const public_var = 0;

/** @package */
export const package_var = 0;

// ----- baz.ts -----
import * as Foo from "./sub/foo";
Foo.public_var;
Foo.package_var;
    // ^ expect error here

Thanks for the report!
I don't remember any technical difficulty about this, so it should be just an oversight.
Want to work on this, but I have limited spare time now... Leaving a help wanted label for now.


Wrap facility search results with react-infinite via react-infininte-any-height

Overview
Wraps search facility results with react-infinite via react-infininte-any-height in preparation for loading more than 500 results.
Connects #705
Demo

Notes
React infinite requires that we specify the height of the list item container. We accomplish this by listening for window resize events, tracking the window size in the Redux state, then calculating the proper container size.
react-infinite-any-height is a wrapper around react-infinite that wraps each list item with a span and uses that span to calculate the height, preventing the need to force all list items to be the same height. There is a lot of variability in the length of facility names and addresses. On narrow screens some list items can be twice as tall as others.
Testing Instructions

Run ./scripts/update to install the new packages
Verify that the facility search UI works as before

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

We may not need to worry about them but it appears that react-infinity & react-infinite-any-height cause some peer dependency warnings on yarn install.

Thanks for the excellent suggestions.


chore(docs): update README

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


⚠ ClamAV has degraded performance

In cda710d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 70 ms

Resolved: ClamAV performance has improved in e501e49 after 5 minutes.


[DART 2] Henson not able to detect extras from external target class

When an activity defines its extras in an external class and injects them through Dart.inject(Object target, Activity source), Henson cannot find them.
A possible solution may be a new annotation. It could be used to do two things:

Set target class.
Let Henson know where to look for more extras.

For instance: @InjectExtraTarget private Model model = new Model();...

This has been solved by issue #67


add flag to skip reference pool mutex if the program doesn't use the pool

This is a tiny idea I had to try to move ReferencePool work off the critical path for programs which have avoided using the reference pool.
The idea is that after #4095 we end up with the reference pool only containing deferred drops. We can use a boolean flag without any atomic synchronization to record if we ever wrote to the pool. If this flag is false, we skip the work. This should hopefully be extremely friendly to branch prediction - if the program never touches the pool then the branch predictor should elide the (more heavy) mutex work.
I wonder if this might be a more gentle alternative to the config-flag in #4095 which might be good enough for most users. I would also support keeping the flag but make it abort-only as a debugging tool to get to the never-touches-pool state for this PR. cc @adamreichold
Measuring the same sample from https://github.com/PyO3/pyo3/issues/3787#issuecomment-1918721168 on my machine, this feature does indeed make quite a marked improvement:
# before - rust noop is slower
$ python test.py
py 2.3021166001853998e-08
rust 3.136266599904047e-08

# after - rust noop is slightly faster
$ python test.py
py 2.5457915995502846e-08
rust 2.360866600065492e-08

Regarding the encapsulation, I fully agree making this part of update_counts but I wasn't keen on conflicting with #4095

I fully agree making this part of update_counts but I wasn't keen on conflicting with https://github.com/PyO3/pyo3/pull/4095

I would be glad if we could finish #4095 first. I it is a soundness fix and even if we decide here to replace the flag entirely, I think doing it one step at a step would be preferable.

In https://github.com/PyO3/pyo3/pull/1608 I introduced a similar bool, but it got removed at some point. May be worthwhile to figure out why we removed it.

In #1608 I introduced a similar bool, but it got removed at some point. May be worthwhile to figure out why we removed it.

Note that this is slightly different: This bool is a one-time flag that is set when the program first touches the reference pool and never reset. Hence, this will end up always shared if it is able to say on a separate cache line (we should make sure to use alignment to prevent any false sharing here).
The flag that was removed did track whether the reference pool was dirty and hence had the same amount of coherence traffic as the atomic on which all futex-based mutexes depend on in any case, i.e. in the uncontended case it would be just as fast/slow as the mutex access to check if the pool is dirty itself.
Now, it might be that the mutex access got slower now that we have removed parking_lot which guaranteed these futex-like performance characteristics. Especially if @davidhewitt did run his benchmark on macOS which IIRC (c.f. https://github.com/rust-lang/rust/issues/93740) still uses pthread mutexes behind the scenes.

Looking at https://github.com/rust-lang/rust/blob/master/library/std/src/sys/sync/mutex/mod.rs, macOS really does appear to use pthread still, so I would expect moving away from parking_lot to a fair PI mutex to certainly regress this. But I also don't think that this functionality is worth a dependency like parking_lot.
I really wonder whether we should just use a lockfree stack or queue here as we really do not need mutual exclusion, we just need to steal a single pointer using a compare-exchange.

I prefer #4178 as a more complete solution, so will close this.


Test workflow secret extraction

Test workflow secret extraction from a forked repo

Found answer: https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#workflows-in-forked-repositories


Release v1.0.0

This is an Epic for the v1.0.0 release. Use https://www.zenhub.com/ to check the connected issues.

Link to this repo and epic at https://app.zenhub.com/workspace/o/freecodecamp/open-api/boards?epics=55006901_164&repos=55006901

Best. Label. Ever.


Fix a bug when multiple Accept lines are coming in an incoming request.

This was discovered while testing some integration with the Consul healtch check API.

Coverage increased (+0.001%) to 99.564% when pulling 680b0fce241ca7b13d228dc7bf043cd621a696db on multiple_accept_lines into 0cd5e2d8685ccd9a441d31d974688c4ab1e77bd4 on release-candidate.


Support for higher dimensional pointclouds

Have you read the documentation?

[ x] Yes

Thank you very much for the great paper and code!
I am working with 3D pointclouds which additionally to the spatial coordinates, also have field properties at every point. I would like to model those properties as extra dimensions.
Does either the python or the C++ bindings support point set registration in higher dimensions? If no, how hard (and how feasible) would it be to modify the code to work for higher dimensions?

Hi @OlfwayAdbayIgbay, thank you for your interests. Unfortunately we don't have plans to include the functionality for higher dimensional point clouds.

Thanks you so much! I've been re-writing some parts of TEASER in python for prototyping, and this code is incredibly helpful! Also, I might be mistaken here, but isn't the C++ code using the Surrogate Geman-McClure distance for rotation rather than the TLS? (see master/teaser/src/registration.cc line 249).

We have both GM and TLS (https://github.com/MIT-SPARK/TEASER-plusplus/blob/d79d0c678b973e052796bc75d3f77025363726d6/teaser/src/registration.cc#L314)


NOA View - Content Update

User Story
As a student I want to be able to see a breakdown of my assessed costs when viewing my assessment and more information about how the amounts were calculated.
Current State

There is a list of "This includes" which doesn't all apply to part-time students and will need to be updated.
Miscellaneous costs show as (Not eligible) when they are eligible and have an amount contributing to the total assessed cost.
Total Academic expenses shows as $0 when there is a value.
Total assessed costs shows as $0 when there is a value.

Expected
The values shown should reflect the values used in the assessment.
The additional information on the page should be accurate to what is used in the assessment.
Acceptance Critera

[ ] Miscellaneous costs should show the value of calculatedDataMiscellaneousAllowance from Camunda.
[ ] Total Academic expenses should show the value of calculatedDataTotalAssessedNeed from Camunda.

@HRAGANBC @Joshua-Lakusta
The "This includes" on the NOA page includes things that don't factor in at all for the PT family income value. The only thing that is included is the "Income of your partner (if applicable)". Do we want to scratch this or just include "Income of your partner (if applicable)" as the lone bullet point?

@CarlyCotton To your question above - I think we want to keep the NoA the same form for PT and FT if possible. That would mean noting wherever particular inclusions are specific to only Full-Time.
So the list could read:
"This includes:

Income of your partner, if applicable
Income of your parent(s), if applicable *Full-Time students only
Scholarships and bursaries reported *Full-Time students only
etc...

P.s. There's ACs related to NOA content on here, should we include the recent issue uncovered where the NoA isn't correctly showing the student's MSFAA number?


Fix for updating to Next 14.0

Hello !
I've just update my project to Next v14 but I had this error generated by upgrading :
Failed to compile.
--
08:53:51.047 |
08:53:51.047 |<EMAIL_ADDRESS>08:53:51.048 | Attempted import error: 'renderToReadableStream' is not exported from 'react-dom/server' (imported as 'renderToReadableStream').
08:53:51.048 |
08:53:51.048 | Import trace for requested module:
08:53:51.048 |<EMAIL_ADDRESS>08:53:51.048 | ./node_modules/resend/build/src/resend.js
08:53:51.048 | ./node_modules/resend/build/src/index.js
08:53:51.048 | ./src/app/api/send/route.tsx

Do you think that you can fix this error on your package ?
Thanks for all

just so you all know, we're currently working on adding support for Next 14

Any updates on this?

I have updated to "resend": "^2.0.0-canary.0", and it is fixed.
When can we have a stable v2?

We didn't find a fix for react-email render yet, but still working on this. The<EMAIL_ADDRESS>uses the<EMAIL_ADDRESS>version which fixes the issue for Next.js 14, but breaks for Next.js 13. Once we fix that issue for both versions, we're going to update the Resend package.

For reference, out tests with React Email passed with the latest canary version (https://github.com/resendlabs/react-email/issues/989#issuecomment-1787746290). We'll be working on comparability with Resend now

I'm not using the render/renderAsync functions and still running into the issue. I've also tried<EMAIL_ADDRESS>

I'm not using the render/renderAsync functions and still running into the issue. I've also tried<EMAIL_ADDRESS>
Can you share your setup and how you're doing? I can include in the tests once I start working on the resend-node

I'm not using the render/renderAsync functions and still running into the issue. I've also tried<EMAIL_ADDRESS>
Can you try the 2.0.0-canary.2?

@bukinoshita Tested with 2.0.0-canary.2 it also works.
in terms of setup:

pnpm workspaces

apps

web(Next14) <- use resend here

packages

resend

lib/index.ts => resend class with functions to send email Templates
emails => react-email templates

all our packages are close to latest version(weekly updated)

I'm not using the render/renderAsync functions and still running into the issue. I've also tried<EMAIL_ADDRESS>
Can you share your setup and how you're doing? I can include in the tests once I start working on the resend-node

Hi, thanks for the response, I just tried with 2.0.0-canary.2 and it didn't work then i realised the @react-email/components was outdated so i updated that and it works now! Thanks a lot

@bukinoshita the<EMAIL_ADDRESS>version fixed the issue for me, thanks a lot!

@bukinoshita 👍🏼

Works fine in api routes but it fails in Server actions:

We didn't find a fix for react-email render yet, but still working on this. The<EMAIL_ADDRESS>uses the<EMAIL_ADDRESS>version which fixes the issue for Next.js 14, but breaks for Next.js 13. Once we fix that issue for both versions, we're going to update the Resend package.

Works for me!

this issue is still not fixed , I have the error with "@"resend": "^2.0.0-canary.2", and "@react-email/render": "^0.0.9-canary.0",
I will take off this library and use another thing , my page dosent work

please suggest other libraries. nothing is working for me.