How to jump to lib header files?

This plugin works well, but I find I can't jump to any function in lib. like socket, printf..etc.(I mean just the header files, like socket.h)
So, is there any ways can do that? Something like adding a lib path.. But I don't know how to..

Hi!
There's no easy way to do this all in Gutentags. It would require for example a hacky g:gutentags_file_list_command that lists both your project's files and the system files, or some even more hacky stuff with options inside .gutctags.
One reason that this type of scenario isn't supported out of the box is that if you told Gutentags to re-parse your project's files, it wouldn't know to not re-parse all the system libraries (unless we make the code needlessly complicated). Another downside is that if you have 10 different projects, your computer would parse the system libraries 10 times.
For now, the recommended way to do this is to run ctags /usr/lib/whatever once, and then tell Vim to load that tags file (set tags += /usr/lib/whatever/tags or something). Vim supports multiple tags files and Gutentags will only add to this list.


[Extension types] Allow @immutable on extension types

The following should be legal IMO:
@immutable
extension type  E(int i) {}

but is currently disallowed by our TargetKind validation (edit: INVALID_IMMUTABLE_ANNOTATION may just need broadening).
Related: #52274

The motivation for this is a desire to enforce prefer_const_constructors_in_immutables, but maybe we should just treat extension types as immutable (and effectively annotated as such). Unless there are times when someone would enable this lint but NOT want it to flag a non const constructor in an extension type? 🤔

(A response to the original question.)
I'm not sure I see the value in adding the annotation. Extension types are immutable by definition (they can only have one field, and that field is final), and the only subtypes of an extension type are also extension types, so the subclass case is also already covered. I think that allowing the annotation wouldn't allow us to catch any problems that aren't already covered by the spec. Am I missing something?

(A response to the second comment.)
Maybe. My understanding is that the lint was motivated by widgets to allow more of the widget structure to be const. I don't know how often an extension type is likely to show up in a widget tree. But if there's another motivation, or if we think extension types will be used in widget trees, then that might increase the utility of doing so. (I'm not necessarily opposed to doing so.)
As for wanting a non-const constructor, I can't really think of any reason why anyone would want to disallow const instances of a candidate class. In my mind it's more a question of value. If I have to add const to every extension type and don't plan on using them in a constant context, then I might find it annoying. But I also might not, especially if extension types are not used very often outside of JS interop support.

Yeah. These are good points. I think the only value is if for some reason we don't want prefer_const_constructors_in_immutables firing on all non const extension type constructors.  Not at all sure about that.
@goderbauer?

The extension type itself is "immutable", but I think that's a red herring. It's a view on another object, which holds all the state. If the view mutates that state, then it's not really immutable. At runtime, it's like calling extension methods on the representation object, which mutates it.
I think it makes sense to allow the annotation, depending on what it actually means, maybe then check whether the representation type is also immutable.

If the view mutates that state, then it's not really immutable.

I agree. The same is true for a class with a final field: if the state of the object stored in the field can change then the class declaring the field isn't really immutable either. The annotation isn't currently enforcing that the fields contain immutable objects, only that the fields are final.
It isn't clear to me why we'd want it to do more checking for extension types than for classes.
And if the only purpose served by the annotation (irrespective of the original intent) is to ensure that all of the fields are final, then given that the single field of an extension type is always final it isn't clear to me what value allowing the annotation adds.
If we want to extend the behavior of the annotation for classes, then it would make sense to me to allow it on extension types. I just don't know whether that's what the community is asking for.

@pq wrote:

I think the only value is if for some reason we don't want prefer_const_constructors_in_immutables firing on all non const extension type constructors

I certainly think extension type constructors can be non-const for good reasons. For instance, they can have a body, or they can have an initializer list containing expressions that aren't constant.


Something like EarTrumpet

Description of the new feature / enhancement
I use an app called EarTrumpet to manipulate the volume of specific apps (vs. only a master volume). https://eartrumpet.app/
Scenario when this would be used?
It's nice to mute or lower the volume of certain apps while keeping others at full volume. For example, I typically mute sound from Teams or Outlook while in a video call, so notifications don't interrupt the meeting.
Supporting information
https://eartrumpet.app/

Why not just use ear trumpet?  :)

#1011 duplicate


Allow to proxy from HTTPS to HTTP

Is there an existing issue for this?

[x] I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.
When using .WithHttpEndpoint() Aspire automatically proxies, from an HTTP endpoint to target HTTP endpoint.
When using .WithHttpsEndpoint() Aspire automatically proxies from an HTTPS endpoint to target HTTP endpoint. In this scenario, it leverages .NET dev cert, which is trusted, a nice to have when working with web apps locally, so the user doesn't get security alerts.
However, there is no way to have an HTTPS endpoing proxying to a target HTTP endpoint. For instance, when serving a web site and working with OAuth, one might need to use HTTPS.
On Azure, the platform takes care of proxying from HTTPS to HTTP.
Aspire doesn't offer the same, so one has to implement SSL locally with custom certs, to be created and configured, leading to more work. Another approach is spinning up, only locally, a reverse proxy app/container like stunnel, grok, and similar, to do that, something like this:
var httpApp = builder
                 .AddProject<MyApp>()
                 .WithHttpEndpoint();

var httpsApp = builder.AddProject<Proxy>()
                 .WithEnvironment("TARGET_ENDPOINT", httpApp.GetEndpoint("http"))
                 .WithHttpsEndpoint();

Describe the solution you'd like
Allow to map from HTTPS to HTTP, something like
var httpsApp = builder
                 .AddProject<MyApp>()
                 .WithHttpsEndpoint(httpsTarget = false); // the internal endpoint will be http://...

This way, the HTTPS endpoint will leverage .NET dev cert, and the internal endpoint is a simple HTTP endpoint that doesn't require and SSL configuration.

The proxying that Aspire does during the local dev inner loop today is TCP-level, not HTTP level, so it can't easily proxy from HTTPS to HTTP. There are other issues covering HTTP ingress and better support for HTTPS using the dev cert though that would likely improve your scenario:

#5508
#836
#4319
#5453
#6890


bcdec: Fix logic for detecting incomplete blocks

Fixes the logic behind detecting whether a block can be safely decompressed into a 4x4 pixel buffer

Thanks!


Crash reporting now takes optional argument for log level

Fixes https://github.com/delvtech/elf-simulations/issues/967


[Translation] Missing translation for lost password

When a user is submitting the lost password form,
a modal is displayed right after to confirm the submission.
A translation is missing, that display the "home link".

Done


Stream too long when trying to download very large file

I am attempting to host a very large file for HTTP download via Pode and am encountering the following exception
`PS E:\git\navision-docker-container\navlive-backup-copied> pode start
VERBOSE: Adding Route: [Static] /backup[/]{0,1}(?.*)
Listening on the following 1 endpoint(s) [1 thread(s)]:
- http://xxx.xx.x.xxx:xxxx/
Date: 2022-09-13 12:34:39
Level: Error
ThreadId: 1
Server: xxxxxxx
Category: NotSpecified: (:) [Set-PodeResponseAttachment], MethodInvocationException
Message: Exception calling "Write" with "3" argument(s): "Stream was too long."
StackTrace: at Set-PodeResponseAttachment, C:\Program Files\WindowsPowerShell\Modules\Pode\2.7.1\Public\Responses.ps1: line 115
at , : line 97
Date: 2022-09-13 12:34:39
Level: Error
ThreadId: 1
Server: xxxxxx
Category: System.Management.Automation
Message: Exception calling "Write" with "3" argument(s): "Stream was too long."
StackTrace:    at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
at System.Management.Automation.Interpreter.ActionCallInstruction2.Run(InterpretedFrame frame)    at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)    at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)    at System.Management.Automation.Interpreter.Interpreter.Run(InterpretedFrame frame)    at System.Management.Automation.Interpreter.LightLambda.RunVoid1[T0](T0 arg0)    at System.Management.Automation.PSScriptCmdlet.RunClause(Action1 clause, Object dollarUnderbar, Object inputToProcess)
at System.Management.Automation.PSScriptCmdlet.DoEndProcessing()
at System.Management.Automation.CommandProcessorBase.Complete()
Date: 2022-09-13 12:34:39
Level: Error
ThreadId: 1
Server: xxxxx
Category: mscorlib
Message: Stream was too long.
StackTrace:    at System.IO.MemoryStream.Write(Byte[] buffer, Int32 offset, Int32 count)
at CallSite.Target(Closure , CallSite , Object , Object , Int32 , Object )
<IP_ADDRESS> - - [13/Sep/2022:12:34:39 -04:00] "GET /backup/NavLive-Daily-Full.bak HTTP/1.1" 500 1148 "-" "-"`
The file I am trying to serve is huge +10GB. Based on the stacktrace this issue may be related to the open PowerShell issue
https://github.com/PowerShell/Microsoft.PowerShell.Archive/issues/19
I've tried disabling compression but that does not seem to help. Any suggestions on what to try next?
server.ps1
`
$HostIPAddress = (Get-NetIPAddress -InterfaceIndex 10).IPAddress
Write-Host $HostIPAddress
$Port = 9898
Write-Host $Port
Start-PodeServer {
Add-PodeEndpoint -Address $HostIPAddress -Port $Port -Protocol Http

Add-PodeRoute -Method Get -Path '/' -ScriptBlock {
    Write-PodeJsonResponse -Value @{ 'value' = 'Hello, world!' }
}

Add-PodeStaticRoute -Path '/backup' -Source './backup' -DownloadOnly -Verbose

Use-PodeLogging -Path './log'

New-PodeLoggingMethod -Terminal | Enable-PodeRequestLogging

New-PodeLoggingMethod -Terminal | Enable-PodeErrorLogging

}
`
server.psd1
@{     Server = @{         Request = @{             Timeout = 86400                     }     }     Web = @{         Compression = @{             Enable = $false         }     } }

Hi @fazleskhan,
The issue comes down to MemoryStream using a Integer, which limits the max size to ~2GB. I believe it was at the beginning of the year, I actually fixed the opposite issue, whereby file uploads to the server were capped at 2GB. I might be able to use similar logic that fixed that issue here 🤔
One other workaround for now could be to split the file up into multiple parts, and then reform the file on the client side?

Thank you for letting me know. I've been pulled into another project for the next couple of weeks. When I get a chance I'll give the splitting up the file try.


not work this script

not work this script

It's unclear what you are trying to explain. Please write a bit more.

Vaiya bujtesi na eita ki auto mail? Gotokal o dekhsilam not work name e
email ashche. Ami ekhono using shuru korini.
On Wed, Nov 25, 2020, 7:31 PM Aminul Islam<EMAIL_ADDRESS>wrote:

It's unclear what you are trying to explain. Please write a bit more.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/NawabHost/bkash-checkout-for-whmcs/issues/1#issuecomment-733707705,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANJAHKNUFDCU6J4GIVOBCSLSRUBKLANCNFSM4UAS3FGQ
.


Updating example

Kai, trying the example: did not work. I think the code is an integer. Changed to that and worked fine.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix Page Templates usage

Context

It was reported by us and also by users that the "Saved Templates" function is broken in the last release due to multiple bugs.
Summary

Since this is actually three bugs in one, this PR contains some JavaScript changes but also some PHP changes.
In essence, it makes saving custom page templates work again.
See below for details.
Relevant Technical Choices

Note: see also individual commits for a better understanding.
Don't fetch page templates if nextTemplatesToFetch is false
This is self-explanatory. Basically, this would lead to REST API requests being made with ?page=false, which will be rejected by the API because false clearly is not an integer.
Ensure TemplateList doesn't crash when pages is null
This is self-explanatory. pages is always null at the beginning. And when REST API fetching fails, it will stay null. However, the component didn't support this and crashed when pages wasn't an array. So this adds some extra hardening.
Always show TemplateSave button
Again, self-explanatory. Always show the button - even if templates haven't been loaded yet.
Improve service registration to ensure correct order
This change allows us to clearly define dependencies between services using HasRequirements, without having to use different action priorities at the same time.
Example: Settings, Story_Post_Type and Page_Template_Post_Type are all delayed to be registered on init, with priority 10.
However, because Story_Post_Type depends on Settings, it will have priority 11 to ensure it runs after.
And because Page_Template_Post_Type depends on Story_Post_Type, it will have new priority 12 to ensure the Story_Post_Type service is properly registered first.
This, together with extended usage of HasRequirements throughout the code base makes dependency resolution more robust.
Remove unnecessary abstraction for REST controllers
Autosaves_Controller and Lock_Controller were both abstract classes, theoretically making it easier to implement them for multiple post types. However, there are no plans for ever using those for any post types other than web-story, so having abstract classes is overkill.
The code has been merged into Stories_Autosaves_Controller and Stories_Lock_Controller, respectively.
E2E tests
To prevent regressions, this PR adds an e2e test verifying that custom page templates can be loaded & saved.
To-do

N/A
User-facing changes

The "Save page as template" button now doesn't have a hover effect when disabled
The "Saved Templates" panel is no longer broken

Testing Instructions

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

Create a new story
Go to page templates tab
Verify that the "Save page as template" button is disabled
Add an element to the page
Verify that the button is no longer disabled and allows you to create a new page template

Reviews
Does this PR have a security-related impact?

No
Does this PR change what data or activity we track or use?

No
Does this PR have a legal-related impact?

No
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #9294

Created a follow up issue from this PR, as it opens up some great options https://github.com/google/web-stories-wp/issues/9316

Question: Would remove the delayed interface on some of the services help here? For example, not delay the settings service at all.


your model weight is unaccesible

Nice work! I'd like to run your work but your weigths for shallow, deep network are not accessible.

Dear @junhocho ,
thank you very much for your interest in our work and sorry for the inconvenience.
Links on the documentation are fixed now. We might have broken them accidentally during some update.
Best,
Xavi


Azure - IoTEdge EFLOW NTP configuration/errors

Dear all,
I've tried to configure chronyd to use a company owned NTP server.
Currently the command: "sudo iotedge check --verbose --ntp-server " gives the error message reported here:
https://learn.microsoft.com/en-us/answers/questions/1056524/azure-iotedge-eflow-ntp-configurationerrors.html
Same happens if you use pool.ntp.org ntp server.
I've restarted chronyd and iotedge service.
Also the DNS is ok. I have tested the name resolution of the NTP server and it is working.
Also the output of chronyc is fine regarding the time of the host.
Maybe this issue is related to a mis-configuration of EFLOW? Can you please give me a hint for checking NTP configuration in EFLOW?
Or can it be a problem of the check routine itself? In my case i do not have problem with timing and seems that chrony is working fine.
The version installed is: https://github.com/Azure/iotedge-eflow/releases/tag/1.1.2207.13102
on Windows Server 2019
Kind regards,
Alberto S.

@fcabrera23 Please reopen this issue I have installed latest 1.5 LTS version and I have the same issue, I tried things described by you but with no luck.


Initial commit

This component is a hof-behaviour, designed to be mixed into hof-form-controller or an extension. It extends functionality by running hooks before and after pipeline methods. This can be used to extend form/field functionality within the main controller pipeline.


Use the resolver for PSI References

closes #38

Rebase? There's some failures on the build which tests the would-be merged commit.

Someday Alec will send a 10 line PR... I'll look later... 🐧 !

beautiful code doesnt come cheap
or it just takes me 20 tries to write good code we'll never know


jpalmer108/updateEventApi/#156

Alaska Airlines Pull Request
After discussing in-issue thread, I'm satisfied with adding these doc notes and wrapping up this issue.
Fixes: #156
Summary:
More clear documentation regarding what can be gathered from the input event as well as the purpose for the validated event. This is so other's aren't also confused by it in the future.
Type of change:
Please delete options that are not relevant.

[X] Other (Docs)

Checklist:

[X] My update follows the CONTRIBUTING guidelines of this project
[X] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

@jpalmer108 please be sure to read the contribution guidelines for how to construct a commit message.

apiExamples

I'm not sure I have the context to understand what you're referencing with adding the apiExamples.md doc, however I can add this update to the demo.md and remove my updates from this file. Will someone else with context on the apiExamples.mddoc be able to implement your suggestion here?

@jpalmer108 This work needs to happen in the apiExamples.md file (which needs to be added). It's not obvious how to do it because this repo is a little out of date with our latest generator.
If you would like, I can make some time with you today and we can update this repo together. It will be a little bit of work though. Otherwise, I can take this change on. It's all documentation, so it shouldn't impact your ability to ship, correct?

It's all documentation, so it shouldn't impact your ability to ship, correct?

Correct, we are unblocked.


Make array of supported languages a first-class member of the Generator class

It can be important for consumers of cucumber-forge-report-generator to know what Gherkin dialects are supported.
We should just make this array a static field on the Generator class.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Don't expose internal model on the maker API

We don't have tests for the maker's API to create new offers.
We should not expose internal types that can change, but treat the HTTP API as a separate interface, otherwise we might accidentally break the API.
I really think we should stop exposing our internal model to the outside and keep proper API boundaries.
Some of these changes might be controversion, feel free to come up with other ideas if you have better ones.

Blocked until the new version is out because it is a breaking API change for the automation. Trivial to change but let's wait nonetheless.

Blocked until the new version is out because it is a breaking API change for the automation.

What part of this is a breaking change?

Closing this because the API changed.


meshoptimizer: init at unstable-2023-03-22

Description of changes
Added the package meshoptimizer, and its gltfpack binary.
packages homepage: https://github.com/zeux/meshoptimizer/
Closes #223406
Things done
Can't currently test it on linux, as my other machine is broken right now.

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

I can confirm, that it works on x86_64-linux.

Should solve #223406.

Also, if you use one of these described keywords over here https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword then the accompanying issue will be closed automatically, once this PR is merged.

Sorry, I can't figure out how to squash my commits together here (I think because I pulled from upstream in between it messed it up?)

If you have this checkbox "Allow edits from maintainers" in the sidebox ticked, I might be able to fix the commits on your branch.

Yes, I fixed that on my local copy :) just didn't want to push until I sorted out the commits

After these last two changes, it should be fine.


Fix for multiple presenters override breaks other things

https://github.com/intridea/grape/pull/784 causes https://github.com/dblock/grape-with-roar/issues/5. We need to rewrite the if that checks whether the representer supports merge.

Fixed in #1025


[FEATURE] State diff should always contain difference

The state DIFF should contain the true difference even if supression is active.
Suppression is marke using state supr anyway.

solved with 0.0.3-alpha.1


DISABLING spell checker

I find the spellchecker introduced recently really annoying and it's impossible to disable.
There should be a config option to disable it.
And in my opinion, it should be opt-in, but I wouldn't mind opt-out either.

I second this. Personally I use cspell for spell checking already, so I've disabled the native spellchecker. However, neogit seems to have ignored the global setting...

Fair - I added an option to toggle it. Feel free to make a PR if you think this could hook into nvim better. I have no time right now to do much more than this, I'm afraid.


About upgraded with XP on an enchanting table

I played in 1.9 version. When i put my wand in Enchanting table, there are nothing that i can press button. exactly empty. So i can't upgrade my wand to student. is it error from sever? or am i do something wrong?
PS, i asked admin, he said don't know

Unless they have changed configs, you should see a spell shop when you right-click an enchant table with wand in hand.
Wands don't go directly in the enchant table anymore, I did away with that a while ago.

So you mean I should buy some spell at spell shop for upgrade my wand?

Yes. The shop GUI should appear when right-clicking an enchant table while holding a wand. If it doesn't, something is broken or misconfigured.

Thanks :D and Thanks for making this fun plugin


Removed equation when partially solving vars instead of adding.

This change is

Fixes #3031  (wasn't linked)

This fixes the issue for me as well. A single file of 60+ tests is working, and the test suite of around 120 tests can be run together.  :100:


Folders can be owned by multiple users

Description

Currently only one user can own and publish a folder, however there is a need that multiple members of a research group can operate on a folder, as it involves members with different understandings of the datasets
It would be good to introduce the concept of a group or project to correspond to CSC naming scheme and instead of assigning folders, drafts to a user assign them to a group.
DoD (Definition of Done)

Folders can be modified and published by users of different research groups
Testing

Unit and integration testing

We will get information on what groups (or better yet projects ) a user belongs with with the AAI response.
If the group does not exist we will have to create it and add the user to it.


How to specify GPU?

I want to use GPU 1 to run your code and my running command is:
python train_and_eval_pytorch.py --gpu_ids 1

But it raises the following issue.
model = model.to(device)
RuntimeError: CUDA error: invalid device ordinal

My server has 4 gpus and they are all normal.

Sorry probably there was some silent bug here... I'd suggest setting the GPU using the environment variable in cmd if it doesn't work.
Jiancheng


fix when kernel version < 4.13rc1 by using index=off cause overlay te…

…st error
Signed-off-by: ningmingxiao<EMAIL_ADDRESS>

some os don't have /sys/module/overlay/parameters/index file


Add bind for doubleclick in productionpanel: select production AND close cityview

When the user clicks the "City needs something to produce" button in the lower right corner
And the map centers on the city and the city view pops up
After selecting an item to produce, the city view does not dismiss automatically (it does this without the mod)
It's annoying because the user has to dismiss out of the city view every time. And this has made me accidentally change what I was building in a city a few times.

The old behavior used to be that the cityview changed every time production was selected, this was changed because it made using the cityview a lot more clunky. I think the best solution to this is adding a bind to something like doubleclick to select production AND close. This way you can trigger the desired behavior from any situation where the cityview would normally be open instead of having a little offshoot of behavior for just one cityview use-case

I think the default behavior is when you selected production in a city, the next city ready for production would be selected if any, otherwise the production menu would just close.
I would request an option setting to make this the default, or the option to bind the double click action you describe above to single click, and have double click leave it open as currently implemented.

@SeanV12 What you ask is surprisingly non-trivial. I can attach handlers to clicks and doubleclicks, but not the absence of one. A click event always fires on the first click of a doubleclick event. If I bind close to a single click, it will always fire and close the cityview before the doubleclick event has a chance to attempt cancelling the first event. UI handling is single threaded so I can't just set up actors to handle this issue either. I can't just schedule closing it and conditionally cancel it later when it realizes the doubleclick event isn't coming either, because there is no event for a "not doubleclick". And no, this isn't as simple as setting up a wait loop, because the UI is single threaded and that would cause the game to hang until the wait loop ends.

Ah ok, I understand the limitation.
Could there be an option in your CQUI Mod Options panel to set the default behavior? Something like an "Auto-close on production" check box?

@SeanV12 Yes, but it's not a priority right now

Prioritize at your discretion. Thanks for your efforts on this mod

@chaorace Why not do the inverse? Leave open on a single click and close on a double?

@TehSeph You literally just described the original proposition from two days ago. In fact, It's literally the title of this ticket

Yeah my bad. I edited my comment almost immediately.

With the introduction of the PQ, this enhancement no longer makes sense, since it would be very trivial to accidentally back out of the menu altogether when spam adding units


UI Banners not displaying on dashboard

I use UI (classification) banners, which should display on every webpage. However, they do not show on the new dashboard UI. Tested on 2.5.8, will test on 2.5.9 and 2.6.0

These are (only) in dashboard in 2.6.

Considering that 2.6.0 is not stable, this is not a viable option. We will need the classification banners in the latest stable release (2.5.9 currently)


Opening the app shows the error on iOS.

Version 0.0.14
Open the app on iPhone.
It shows an error. See screenshot.

Thanks. This a cold start issue for the test app as the server is shutdown when idle and the warm up is around 10s. The 5s timeout error can be seen on first interaction in such scenario


feat: add setup.view_options.is_excluded

My intention is that I don't want to see macOS folder view file .DS_Store while keeping show_hidden = true.
Some points:

The title cannot be achieved by is_hidden_file without is_excluded added by this PR. (following the naming convention is_hidden_file)
During my changing the ordering of the conditions in M.should_display, I realized that the better name could be should_display_hidden. What do you think?

@Cottser Well that doesn't make sense to me as .DS_Store being a hidden file is a truth. It make more sense to extend the default behaviour as written by the author:
https://github.com/stevearc/oil.nvim/blob/415281090746fcf562ddd15ceaf6f047e50e1981/lua/oil/config.lua#L56-L58
i.e. it's used to define something apparently not hidden as an exception, e.g. node_module/. (Yeah, the front-end nightmare)

I'd prefer to make this API a bit more generic. I've added a set_is_hidden_file API method that should allow you to change what constitutes a "hidden" file at runtime. You can use this as a replacement for toggle_hidden by defining your own action that swaps it from the default implementation to one that returns true if name == '.DS_Store'

Right, so for your use case you wouldn't want to use the toggle_hidden action at all, and would instead want to switch out the is_hidden_file function.
local hidden_prefix = "."
require("oil").setup({
  keymaps = {
    ["g."] = {
      desc = "Toggle hidden files",
      callback = function()
        hidden_prefix = hidden_prefix == "." and ".DS_Store" or "."
        require("oil").set_is_hidden_file(function(name)
          return vim.startswith(name, hidden_prefix)
        end)
      end,
    },
  },
})

This way it toggles back and forth between hiding all files that start with . and all files that start with .DS_Store (technically you wanted filename == ".DS_Store", but this is close enough in practice and is slightly simpler).

Sure, but could we rename is_excluded to is_always_hidden?

Thanks!


Only reRender in RAF

If the user calls reRender() inside an event handler
it will update the DOM synchronously and call all kinds
of lifecycle methods on any nested components that have been
updated.
This can interrupt the browser event lifecycle, for example
https://github.com/heapwolf/components/issues/32
Here we schedule the DOM updated with requestAnimationFrame
so that the browser can finish it's DOM event lifecycle before
we make any changes to the DOM.

very cool

this makes a lot of sense, I was thinking this as well, code lgtm pending conflict resolution


Adding Istanbul Ayvansaray University domain

Institution/School Name
Istanbul Ayvansaray University
Instiution/School Website
https://www.ayvansaray.edu.tr/en-US
Email Users
Please place an x between the square brackets if yes

[x] Students
[x] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[x] Post-graduate research
[x] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@pumpkin34 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host stu.edu.tr not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for understanding!


[ANCHOR-384] Use stellar/anchor-tests to run the stellar validation tool

PR Checklist
PR Structure

[ ] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[ ] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
[TODO: Short statement about what is changing.]
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Something went wrong with PR preview build please check

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/

Reference Server Preview is available here:https://anchor-ref-pr1000.previews.kube001.services.stellar-ops.com/SEP Server Preview is available here:https://anchor-sep-pr1000.previews.kube001.services.stellar-ops.com/


Modern Object Pascal - Read Section 3

Section 3 of Modern Object Pascal is read & understood.
Acceptance Criteria:
GIVEN the development environment has been setup
WHEN I want to develop using Pascal
THEN I can
As a developer, I want to read Modern Object Pascal so that I can write code using the Pascal language.
https://castle-engine.io/modern_pascal_introduction.html

Read section 3 on 10/23. Added code for section.


geosnap depends on not most update scikit-learn api

I recently updated my scikit-learn to 0.21.2 and I think it is now crashing my geosnap imports:
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-1-b37417093a5a> in <module>()
      6 import numpy as np
      7 import pandas as pd
----> 8 import geosnap
      9 from osnap.data import read_ltdb
     10 from tobler.area_weighted import *

c:\users\renan\desktop\geosnap\geosnap\__init__.py in <module>()
     29
     30
---> 31 from . import analyze
     32 from . import data
     33 from . import util

c:\users\renan\desktop\geosnap\geosnap\analyze\__init__.py in <module>()
----> 1 from .analytics import cluster, cluster_spatial
      2 from .dynamics import Sequence, Transition
      3 from .incs import linc
      4
      5 __all__ = ['linc', 'Sequence', 'Transition', 'cluster', 'cluster_spatial']

c:\users\renan\desktop\geosnap\geosnap\analyze\analytics.py in <module>()
      8 from libpysal.weights.distance import KNN
      9
---> 10 from .cluster import (azp, affinity_propagation, gaussian_mixture, hdbscan, kmeans, max_p,
     11                       skater, spectral, spenc, ward, ward_spatial)
     12

c:\users\renan\desktop\geosnap\geosnap\analyze\cluster.py in <module>()
      2
      3 import numpy as np
----> 4 from hdbscan import HDBSCAN
      5 from region.max_p_regions.heuristics import MaxPRegionsHeu
      6 from region.p_regions.azp import AZP

~\AppData\Local\Continuum\anaconda3\lib\site-packages\hdbscan\__init__.py in <module>()
----> 1 from .hdbscan_ import HDBSCAN, hdbscan
      2 from .robust_single_linkage_ import RobustSingleLinkage, robust_single_linkage
      3 from .validity import validity_index
      4 from .prediction import approximate_predict, membership_vector, all_points_membership_vectors
      5

~\AppData\Local\Continuum\anaconda3\lib\site-packages\hdbscan\hdbscan_.py in <module>()
     15 from warnings import warn
     16 from sklearn.utils import check_array
---> 17 from sklearn.externals.joblib.parallel import cpu_count
     18
     19 from scipy.sparse import csgraph

ModuleNotFoundError: No module named 'sklearn.externals.joblib.parallel'

Is there  a way to overcome this?

Can you import it directly from joblib or use multiprocessing.cpu_count()?
The sklearn website says it's recently deprecated, and with that it looks like they removed it from their sklearn.externals submodule.

Were not using it directly. I think it’s happening from hdbscan

i'm tempted to remove hdbscan, at least for now because (1) this issue where it's pinning to an older scikit, and (2) it has some weird cython installation issues when using pure pip that I'd prefer to avoid.
in the neighborhood context, its results also tend to be a bit wacky

hdbscan is now an optional dependency, so i think this is resolved. I'm closing but please reopen if you continue to experience this problem


Should the "OOM killer script" be added as default

Reviewing Choosing Memory Heap Settings and wondering if it makes sense to set the "OOM killer script" as a default. (Nothing else stood out as a default but there may be more.)
The option probably looks like -XX:OnOutOfMemoryError=/opt/docker-solr/scripts/oom_solr.sh.

Starting with Solr 8.9, the OOM killer should run by default: SOLR-14978
What version are you running?
Note: It will still be hard to have a default policy for heap dumps, as those will likely be deleted when the pod is killed. If the logs are mapped to a volume, then they may be saved though...

Thank you for the tip! I'm running a few different versions but they're all older than 8.9. I will configure this myself for these older versions.


Crash when activating pcons on loading screen.

When zoning from an explorable area to an outpost.
Auto-disable pcons on map change ON.
Clicking "Enable" during the lapse where there is no counter on pcons produces the crash.

Sending a crash dump to me on discord would help debug this - should be one in the crashes folder of gwtoolbox

https://github.com/gwdevhub/GWToolboxpp/commit/b830b99e07a335d374d6b761ea012488fde34286


Add evernote support

I added evernote support for the contest. It shows up with the cards that display the title of the note.

It seems that if I have multiple notes all the start time links are attached to the first note. Clicking the link gets data of the correct note.

Evernote integration added with pull request #426
Thank you for your contribution and for taking part in the challenge.


Update twitter image meta tag

Update the meta tag from twitter:image:src to twitter:image
Looks like the Twitter docs are all using the latter now (https://developer.twitter.com/en/docs/twitter-for-websites/cards/overview/summary)

Thanks for the PR @chrisrng! Just curious, is LinkedIn using this addon or was this just a random fix?

@rwwagner90 I did it for the Ember Blog since we use Empress which uses this :)

Gotcha, thanks for the context and thanks again for the PR!


nit: fixed readme omission for helm install with custom values file

Description:
nit: fixed readme omission for helm install with custom values file
Related issue(s):
Fixes #
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Patch and project coverage have no change.

Comparison is base (ad9ebf9) 77.83% compared to head (eaf7751) 77.83%.

:exclamation: Current head eaf7751 differs from pull request most recent head 0eb2b75. Consider uploading reports for the commit 0eb2b75 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1414   +/-   ##
=======================================
  Coverage   77.83%   77.83%
=======================================
  Files          32       32
  Lines        2599     2599
  Branches      539      539
=======================================
  Hits         2023     2023
  Misses        402      402
  Partials      174      174

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


⚠ ESPN has degraded performance

In 4183577, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 1405 ms

Resolved: ESPN performance has improved in abfcf35.


Set default language for brazil and mexico admin

When a user is created with the role of "Brazil admin" or "Mexico admin" set the default language for the user to pt-br and mx-es respectively.

I'm looking at this more closely, and this ticket might be irrelevant since we're creating users with these roles manually. This should be more of a procedural step than having it in code. The user creation form allows for the selection of the user's language, and I can't think of a scenario in which brazil and mexico admins won't be created manually.


cesiumForUnreal is in compatible

I had compile the source code according to READ.ME；But there are still some problems；

when I open the project file， Pop-up prompts  ‘cesiumForUnreal is in compatible‘；I had changed the engineversion to match the
unreal engine version,but that's not  work；
so I tried to solve this problem accroding to  https://github.com/CesiumGS/cesium-unreal/blob/main/Documentation/packaging-guide.md
when i execute the commond in  the last article，many files cannot be found；
such as：
C:\project\CesiumForUnreal\HostProject\Plugins\CesiumForUnreal\Source\CesiumRuntime\Public\Cesium3DTileset.h(5): fatal error C1083: cannot open the file: “Cesium3DTilesSelection/ViewState.h”: No such file or directory;

What version of Unreal Engine are you using? And where have you installed Cesium for Unreal from? From the Epic Marketplace? From GitHub? Building it yourself?

What version of Unreal Engine are you using? And where have you installed Cesium for Unreal from? From the Epic Marketplace? From GitHub? Building it yourself?

the  Unreal Engine is 4.26.1;
I download the cesium for unreal from github; use the commond :'git clone --recursive https://github.com/CesiumGS/cesium-unreal.git;
yeah; I built it by cmake; the version of cmake is 3.23.0-rc2; the Visual studio  version is 2017 (15.9.45)
'

Building cesium-unreal from source can be a bit challenging. You need to follow these instructions, and slight tweaks may be needed depending on your system:
https://github.com/CesiumGS/cesium-unreal/blob/main/Documentation/developer-setup-windows.md
I'd suggest using the built plugin from the Epic Marketplace or the Releases page if possible:
https://github.com/CesiumGS/cesium-unreal/releases
If your Unreal Engine isn't an official release from Epic, but rather one you built yourself, you may see that message about the plugin being for a different version. You should be able to ignore it, though.
I don't think there's anything for us to fix in the plugin here, so I'm going to close this. If you're still running into trouble, please post to the Community Forum.

Building cesium-unreal from source can be a bit challenging. You need to follow these instructions, and slight tweaks may be needed depending on your system: https://github.com/CesiumGS/cesium-unreal/blob/main/Documentation/developer-setup-windows.md
I'd suggest using the built plugin from the Epic Marketplace or the Releases page if possible: https://github.com/CesiumGS/cesium-unreal/releases
If your Unreal Engine isn't an official release from Epic, but rather one you built yourself, you may see that message about the plugin being for a different version. You should be able to ignore it, though.
I don't think there's anything for us to fix in the plugin here, so I'm going to close this. If you're still running into trouble, please post to the Community Forum.

thank you very so much


Implement conda recipe in conda-forge

I have tried to port the conda package for qibo, the main issue is that tensorflow is not really well supported by conda-forge, the current versions are old, there is version mismatch between linux and osx, and furthermore the GPU support is only available from tensorflow-gpu which is outdated and supports only linux.
Given that tf is using pip as their default channel, I believe we can stick to pip until there is stable and updated version of tensorflow in conda-forge.

I have implemented a basic recipe for linux-cpu. The PR is in https://github.com/conda-forge/staged-recipes/pull/12509
However due to an issue with the tensorflow version the job fails, so I am waiting for feedback from the conda-forge maintainers.

Given that the main issue here is tf in conda, let me propose to close this PR and wait until (if) the situation improves.

Given that the main issue here is tf in conda, let me propose to close this PR and wait until (if) the situation improves.


Contact Kodadot before new version rolls out

New version will use a builder pattern and will be simpler to use as well as feature more compatible nodes.
Release of the new version will make SDK incompatible with the current implementation in Kodadot.

Plan is to create PR solving this issue for them once new version is ready (1-3months)
https://github.com/kodadot/nft-gallery/pull/4297

Closing issue as we kept backwards compatibility intact.


breadcrumbs and header improvements without adminstyles

This pull request fixes the following issues:

BEFORE
AFTER

:+1:


GCMetricsSysTest is partially failing due to being executed before agent initialization

The GCMetricsSysTest is not waiting for the agent to be initialized, thus, it is failing because the data cannot be retrieved at this point in time.

This should not happen anymore, since the tests are now waiting for the agent to be initialized.
See ticket #157


fix: table formatting in IPFS for websites deep dive

Please review @doctorrobinson

Nailed it. Thank you Alan!


ARROW-17439: [R] Change behavior of pull to compute instead of collect

I could use some help with a couple of things:

[x] devtools::check warns about pull.ArrowTabluar being undocumented. I @exported it to stay consistent with other ArrowTabular generics defined in arrow-tabular.R and don't understand why checking doesn't warn on all of these. Does this one just not need exporting?

Relevant devtools::check() output
❯ checking for missing documentation entries ... WARNING
  Undocumented code objects:
    ‘pull.ArrowTabular’
  All user-level objects in a package should have documentation entries.
  See chapter ‘Writing R documentation files’ in the ‘Writing R
  Extensions’ manual.

[x] I found an inconsistency with dplyr::pull and Tables: Pulling an ungrouped Table produces a ChunkedArray whereas pulling a grouped Table produces a Table. This makes a subsequent call to as.vector produce an error of Error in as.vector(x, mode) : cannot coerce type 'environment' to vector of type 'any'

Example of the difference
> sw_table <- arrow_table(starwars)

> sw_table |>
+   filter(height > 100) |>
+   group_by(homeworld) |>
+   pull(name) |>
+   class()
[1] "Table"        "ArrowTabular" "ArrowObject"  "R6"

> sw_table |>
+   filter(height > 100) |>
+   pull(name) |>
+   class()
[1] "ChunkedArray" "ArrowDatum"   "ArrowObject"  "R6"

2. Pulling an ungrouped Table produces a ChunkedArray whereas pulling a grouped Table produces a Table. This makes a subsequent call to as.vector produce an error of Error in as.vector(x, mode) : cannot coerce type 'environment' to vector of type 'any'

This is due to ARROW-17738.
You can use as_arrow_table instead of compute here I think. https://github.com/apache/arrow/pull/14160#issuecomment-1268404022

Thanks for the link @eitsupi that change fixes the issue I was having.
Re:  @wjones127's comment:

On the CHECK issue, IIRC we don't @export dplyr S3 methods. We do call s3_register on them here:

I removed the export directive but I would eventually like to understand why the other exported functions in arrow-tabular.R (and probably elsewhere) don't need specific documentation (but I'll do that elsewhere).
Both the issues I raised are resolved and this can be reviewed.

As I mentioned in JIRA, do you implement an argument like as_data_frame?
For consistency with other functions, wouldn't it be better to return an R vector by default and an Arrow object optionally?

Since the dplyr documentation says that pull returns a R vector, I think the documentation should at least explain that pull to arrow objects does not return a vector and using with as.vector to become a vector.
Users likely know very little about Chuncked Arrays.

Thanks the look @nealrichardson. This should be good to merge now.

Since the dplyr documentation says that pull returns a R vector, I think the documentation should at least explain that pull to arrow objects does not return a vector and using with as.vector to become a vector. Users likely know very little about Chuncked Arrays.

@eitsupi Good call, I'll add a note about this in #14387


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210718153538577 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210718153538577 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.566%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
26721

Relevant Lines:
56896

💛  - Coveralls


How Does an Update Honor the LTS Channel?

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
According to this article, if a device has an LTS version of PowerShell installed, it won't be upgraded to a non-LTS version. However, if you run an MSI for a non-LTS version, it will upgrade.

When upgrading, PowerShell won't upgrade from an LTS version to a non-LTS version. It only upgrades to the latest version of LTS, for example, from 7.2.3 to 7.2.16. To upgrade from an LTS release to a newer stable version or the next LTS, you need to install the new version with the MSI for that release.
When the installed version isn't an LTS version, PowerShell upgrades to the latest stable version.

How is this logic handled during installation? Is there a Registry key or some other setting that is queried that controls this? Why doesn't an MSI honor this when an upgrade can?
Expected behavior
If you install the latest version PowerShell with the MSI, it should not overwrite an LTS version.

Actual behavior
When installing with an MSI, it will upgrade you to the latest version.

Environment data
Name                           Value
----                           -----
PSVersion                      7.2.16
PSEdition                      Core
GitCommitId                    7.2.16
OS                             Microsoft Windows 10.0.20348
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

OS Data
Version    Caption
-------    -------
10.0.20348 Microsoft Windows Server 2022 Standard

Windows update log
N/A

Visuals
No response

This is handled in Windows update.
Windows update knows what version you are on and will only offer you an LTS update.   So, when 7.4 comes out, 7.2 will start being offered 7.4 updates.  We decide when that happens, so, around 7.4.1.
cc @StevenBucher98


Count JS test pass/fails; return proper exit code

Currently test/javascript/run always returns an exit code of 0,
regardless of whether any tests fail or not. This makes any sort of
automated testing that expects to inspect the results of the run
invalid.
make javascript should now properly fail if any of the JS tests fail.
It also prints a numerical summary of passed/failed tests, similar to
the eunit results.
Fixes COUCHDB-3328

Ironically, the failed Travis CI run shows exactly what I mean. I don't know if the failing JS test (test/javascript/tests/replication.js) fails only on R16B03 because we have a bug there, or because the test case is flaky. This demands further investigation - after we merge this change. :)

+1 \o/ passing tests


Wording changes

Hi Michel,
Could we change the word "task" for "Activity"

Thanks

Hi, this is done too!


How to make this plugin work with rabbitmq 3.5.1?

Hi there, thanks for this great plugin. Recently I tried to install it with an old rabbitmq cluster running 3.5.1 but it seems not working.
At first I tried the latest version (following instructions on https://www.rabbitmq.com/prometheus.html) but the plugin can't be enabled (showing some error messages about "loading accept"). I guess it's because of 3.5.1 was built with an old erlang release, so I resolved to use this special build for R16B03.
With the special build the plugin can be enabled, but when I tried to access the /metrics endpoint it just returns 404:
curl -v http://localhost:15672/api/metrics
* Hostname was NOT found in DNS cache
*   Trying <IP_ADDRESS>...
* Connected to localhost (<IP_ADDRESS>) port 15672 (#0)
> GET /api/metrics HTTP/1.1
> User-Agent: curl/7.35.0
> Host: localhost:15672
> Accept: */*
>
< HTTP/1.1 404 Object Not Found
* Server MochiWeb/1.1 WebMachine/1.10.0 (never breaks eye contact) is not blacklisted
< Server: MochiWeb/1.1 WebMachine/1.10.0 (never breaks eye contact)
< Date: Wed, 29 May 2019 11:22:38 GMT
< Content-Type: application/json
< Content-Length: 55
<
* Connection #0 to host localhost left intact
{"error":"Object Not Found","reason":"\"Not Found\"\n"}(shuai)

My question: is it true that this plugin isn't compatible/tested with rabbitmq 3.5.1? If so, since I know a bit erlang,  would there be much work involved to make it work with 3.5.1?

btw this is the output of  rabbitmq-plugins list:
[E*] accept                            0.3.3
[e*] amqp_client                       3.5.1
[E*] cowboy                            0.5.0-rmq3.5.1-git4b93c2d
[e*] mochiweb                          2.7.0-rmq3.5.1-git680dba8
[E*] prometheus                        3.4.0
[E*] prometheus_httpd                  2.1.4
[E*] prometheus_process_collector      1.1.0
[E*] prometheus_rabbitmq_exporter      v<IP_ADDRESS>
[  ] rabbitmq_amqp1_0                  3.5.1
[  ] rabbitmq_auth_backend_ldap        3.5.1
[  ] rabbitmq_auth_mechanism_ssl       3.5.1
[  ] rabbitmq_consistent_hash_exchange 3.5.1
[  ] rabbitmq_federation               3.5.1
[  ] rabbitmq_federation_management    3.5.1
[E*] rabbitmq_management               3.5.1
[e*] rabbitmq_management_agent         3.5.1
[  ] rabbitmq_management_visualiser    3.5.1
[  ] rabbitmq_mqtt                     3.5.1
[E*] rabbitmq_shovel                   3.5.1
[E*] rabbitmq_shovel_management        3.5.1
[  ] rabbitmq_stomp                    3.5.1
[  ] rabbitmq_test                     3.5.1
[  ] rabbitmq_tracing                  3.5.1
[e*] rabbitmq_web_dispatch             3.5.1
[  ] rabbitmq_web_stomp                3.5.1
[  ] rabbitmq_web_stomp_examples       3.5.1
[  ] sockjs                            0.3.4-rmq3.5.1-git3132eb9
[e*] webmachine                        1.10.3-rmq3.5.1-gite9359c7

Yea I think I started with 3.6. No idea about the effort but 404 likely happens because registering handler done differently - a good starting point!


Fix css animation

Closes #2

Awesome. Thanks man!

Like to see the blue gear spinning 🔄  😄

Oh already shipped. Thanks!


switch fuzzer infrastructure to use cargo-bolero

I actually added the comments while in the process of fixing the tests this morning, so hopefully they turn green this time and it’ll be good to go :)


Migrate forward plugin to v0.14 api

This change is to migrate in/out_forward to v0.14 APIs, except for socket/server plugin helpers, to keep commits few and diff small.

Pull requests to add socket/server plugin helpers will follow this change.
@repeatedly Could you review this?

Okay. I will review it later 👍

@repeatedly ping?

Commented. Others looks good.

I pushed a commit to close sockets in #close, and will merge this after CI green.


More lenient TypeScript types

When SerpApi adds support for a new engine or search parameter, the TypeScript types in this library are not immediately updated. For TypeScript devs, they will see warnings that the engine or search parameter does not exist.
This PR looks into making the types more lenient so no warnings appear, while ensuring autocomplete works and providing a way to maintain type safety.
Related to https://github.com/serpapi/serpapi-javascript/issues/5
Effects

Factor
Works?

Does not show warnings for arbitrary engines
Yes

Does not show warnings for arbitrary search params
Yes (because of getJson’s P extends EngineParameters<E>)

Autocomplete for engine
Yes

Autocomplete for search params
Yes

Type-checking for inline params
Yes

Types for safety
Yes via EngineParamaters<"...">

Thanks @zyc9012!

Should we update the examples as well because it's showing warning for arbitrary search params? (The warnings are not consistent with inline usage)

That warning is expected because params have explicitly been set to satisfy GoogleParameters. This provides a way for users to have full type safety. I can see why this could be confusing though. What do you think if we add an additional block in the example without the satisfies GoogleParameters?

If we add arbitrary params to the above example. The warning becomes confusing. (It says asd is not in GoogleParameters, but we are supposed to support arbitrary params)

Yea, that's unexpected behavior. Super strange. I've added a Record<string, unknown> to resolve this. The only warning that should appear is that the required params are missing.
--- a/src/serpapi.ts
+++ b/src/serpapi.ts
@@ -71,7 +71,9 @@ const SEARCH_ARCHIVE_PATH = `/searches`;
  */
 export async function getJson<
   E extends EngineName = EngineName,
-  P extends EngineParameters<E> = EngineParameters<E>,
+  P extends (EngineParameters<E> & Record<string, unknown>) = EngineParameters<
+    E
+  >,
 >(
   engine: E,
   parameters: P,

That warning is expected because params have explicitly been set to satisfy GoogleParameters. This provides a way for users to have full type safety. I can see why this could be confusing though. What do you think if we add an additional block in the example without the satisfies GoogleParameters?

I think the idiomatic way may be use some type wrappers? Such as SomeTypeWrapperAllowingArbitraryParam<GoogleParameters>, and use that after satifies. Just a suggestion, you can decide.

I think the idiomatic way may be use some type wrappers? Such as SomeTypeWrapperAllowingArbitraryParam, and use that after satifies. Just a suggestion, you can decide.

That's a great idea, didn't think of that. I've used it for getJson's parameter type too.


recommend inclusion of high pass filter or not?

for users plugging confounds in say nilearn or conn, it would make more sense to not have the discrete cosines, as they have their own solutions, unless there is compcorr in which case you would need to have them as they are part of the generation.

Leaving high pass regressors, as this is an integral part of a number of strategies (compcor, ica_aroma). I still added a note on high pass filtering in the documentation.