Fix flakiness in authenticateWithChunkedStreaming().

We were prefering to return the networkResponse before the request
had completely failed. Instead we should return the network response
only on failure, and only for APIs that return failed responses.
Closes: https://github.com/square/okhttp/issues/2997

Good find 👍


Renovator Publish Functionality

Required Information

AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM 6.5.6
ACS AEM Commons Version: 5.0.2
Reproducible on Latest? yes

Expected Behavior
When we use the renovator to rename an asset, and select publish option as self-managed or Queue, I am assuming the renamed asset to be republished, as the references of the asset is getting published.
Actual Behavior
Whenever, we are renaming an asset using renovator and selecting the publish option as self managed or queue, the renamed asset is not getting published again. Although, the references are getting published.
Steps to Reproduce

Publish an asset. Example: /content/dam/we-retail/en/features/support.png
Use the renovator tool to rename this asset with Publish option as "self managed" or "queue".
Observe the name of asset which is renamed in author.
Observe the name of asset in publish environment, the renamed asset is not published.

I'm seeing a similar issue for folder moves. The folders are published, but not the assets within them.

@Collingr are there any errors in the logs? Do you know if its just not trying to replicate at all? Or if it issues the replicate call but the replication never makes it out?


rename things to align with golang style conventions

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: greghanson
Assign the PR to them by writing /assign @greghanson in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ZackButcher: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-v1alpha3.sh
c08bd79560ad367f1c362df9d745f8c98c3f628f
link
/test istio-pilot-e2e-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ZackButcher: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-v1alpha3.sh
c08bd79560ad367f1c362df9d745f8c98c3f628f
link
/test istio-pilot-e2e-v1alpha3

prow/istio-pilot-e2e.sh
c08bd79560ad367f1c362df9d745f8c98c3f628f
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add ChangeLogLevel to DebugService

Motivation
https://github.com/spacemeshos/go-spacemesh/issues/5674

bors merge

bors cancel

bors merge


Enhance CreateVolumeSnapshot function to snapshot vSphere CSI 2.4.0 Alpha volumes

Is your feature request related to a problem? Please describe.
Use CreateVolumeSnapshot function to snapshot vSphere CSI 2.4.0 Alpha volumes
Describe the solution you'd like
CreateVolumeSnapshot function creates snapshots for vSphere CSI 2.4.0 Alpha volumes
Describe alternatives you've considered
Environment
Kubernetes Version/Provider: Rancher RKE2 1.21.5-rke2r1
Storage Provider: vSphere
Cluster Size (#nodes): 6
Additional context

@ron1 If these snapshots are done through the CSI VolumeSnapshot APIs, you should be able to do it with the CreateCSISnapshot and the CreateCSISnapshotStatic functions.

Thanks


[FEATURE] Static analysis: Preemptive cost estimation/profiling

Apalache currently implements a version of profiling, which allows users to visualize parts of the specification which result in a large number of SMT constraints (and typically, longer runtimes).
While helpful, it cannot be used as part of a quick-iteration cycle, because it is tied to actually running the model checker.
We should implement a static analysis, that preemptively computes costs: Since we know, for each encoding, how many constraints are introduced by each rewriting rule, we can implement a cost estimator for each TLA+ operator, and a pass which aggregates these costs without actually performing model-checking. These aggregates can then be made into a heatmap, like the ones obtained from using the --smtprof flag and heatmap.py.
This could be run either as a standalone mode ($APA profile Spec.tla), or simply as additional output to the standard model-checking mode.

If we could integrate that with a REPL similar to TLC REPL, that would give us a very quick feedback loop.

I believe that is one more use case for the server mode @shonfeder


Test failure

Hey!
I got the following test failure while trying to package get_blessed_rs for the AUR:
running 1 test
test content_parser::test::general_table_has_expected_data ... FAILED

failures:

---- content_parser::test::general_table_has_expected_data stdout ----
thread 'content_parser::test::general_table_has_expected_data' panicked at src/content_parser.rs:14:9:
not yet implemented
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

failures:
    content_parser::test::general_table_has_expected_data

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

error: test failed, to rerun pass `--bin get-blessed`

I realized it is just an empty test with a todo!() - it might be a good idea to remove it or add #[ignore] until it is functional.

Hi yes at the moment there is no test I don´t know how to do proper tests

I took a look at the codebase, maybe you can create a new ContentParser, fetch the contents and assert the result there. But I'm not too sure.

Closed #11


Watcher hidden files

If you haven't already, please read through our contributing guidelines before opening your PR
What is the purpose of this PR?
Closes #189.
Edit watcher to give a warning instead of an error when encountering weird files, and will continue to run.
How did you implement your changes
Detect for hidden files and give warning that it is not a valid file and won't be processed.
If path passed no longer exists, give a warning.
Remaining issues

test notebook with actual hidden file in the bin_file_dir directory

We discussed this in our one-on-one yesterday, but it looks like list_files already does this by default.


NGINX Root Location

Linking to an issue that was lingering on the archived php-web repo: https://github.com/paketo-buildpacks/php-web/issues/207

I was deploying a Laravel app with paketo-buildpacks/php and BP_PHP_SERVER = nginx and then I realized that URLs were not working (nginx returns 404).
I had to create a .nginx.conf.d/location-server.conf file with the following content:
location / {
    try_files $uri $uri/ /index.php?$args;
}

Redirecting every request to the index.php is a common practice in many PHP Frameworks/CMS like Laravel, Symfony, WordPress, etc.

Symfony nginx config: https://symfony.com/doc/current/setup/web_server_configuration.html#nginx
Laravel nginx config: https://laravel.com/docs/9.x/deployment#nginx
WordPress nginx config: https://wordpress.org/support/article/nginx/

Since this piece of config is very common maybe it would be a good idea to add a new environment variable, something like BP_PHP_ENABLE_ROOT_TRY_INDEX_PHP (example name, it could be other), which would add this location config block. The default value would be false, so it is backwards compatible and won't break any existing scenario.
I would like to know what you think about this change. If you are open to merge something like this I can work on this and I would send a PR with the changes needed.

@dvigueras this seems like a reasonable thing for the buildpacks to do. I wonder if there is a way to introduce this functionality without needing an environment variable.
Could you clarify what your app structure looks like that you needed to add that configuration?

Hi @sophiewigmore
I'm building a container with a Laravel application (a blog). I'm using the paketobuildpacks/builder:full builder and the paketo-buildpacks/php buildpack.
Laravel needs an nginx directive (https://laravel.com/docs/9.x/deployment#nginx) to direct the incoming requests where the requested file doesn't exist to the index.php.
location / {
        try_files $uri $uri/ /index.php?$query_string;
    }

This way the Framework can intercept the request and the look for the requested route in the list of routes defined in the Laravel application.
e.g.: nginx receives an incoming request for the /login path. a login file doesn't exist in the filesystem, so nginx passes the request to the index.php file. Then this request is forwared to php-fpm and therefore Laravel is able to process the request. Laravel matches /login with an entry in the application route list and finds which Framework Controller will process the request.
If Laravel is not able to match the requested path then it will return the 404 Framework page.

Also I've noticed of another issue with my Laravel project and the current implementation of the buildpack.
In my project I am using Livewire (https://laravel-livewire.com/). This library adds a route to /livewire/livewire.js which is processed by a Livewire PHP Controller.
What is happening right now in nginx is that the incoming request is matching this location block.
location ~* \.(?:ico|css|js|gif|jpeg|jpg|png)$ {
    expires         max;
    add_header      Pragma public;
    add_header      Cache-Control "public, must-revalidate, proxy-revalidate";
}

instead of my location / block. Then nginx tries to serve an existing livewire/livewire.js file which doesn't exist and then it  returns 404. This happens because nginx location precedence matches the previous block first.
A possible idea would be to add a new environment variable to be able to disable this location block (it would be similar as the work done with the BP_PHP_ENABLE_HTTPS_REDIRECT env var / DisableHTTPSRedirect variable)


Add functions for emitting/listening events

Currently when we emit events, we simply call eventHub.emit(eventName, ...), which does not offer any type protection. Similarly, we call eventHub.on(eventName, callback) which does not offer any type protection for the callback.
We should use functions to emit events and listen to them, so we get type protection. PanelEvent.OPEN is being wired up with #2144, we should wire up all the other events as well.

We also have the option of better typing eventHub.emit. One way to do this is to use a type map of eventName to function args. Something like:
interface EventPayload {
  [key: string]: unknown[];
  ['test']: [number, Date];
  ['blah']: [string];
}

and then can genericize the eventHub
class EventEmitter<T extends Record<string, unknown[]>> {
   ...
   emit<K extends keyof T>(eventName: K, ...args: T[K])

declare const emitter: EventEmitter<EventPayload>;
declare const emitter: EventEmitter<EventPayload>;

// types enforced of event name mapped to proper args
emitter.emit('test', 4, new Date());


Implement import and cleanup

@ushitora-anqou I guess we can remove a code snippet for testing.

@ushitora-anqou I guess we can remove a code snippet for testing.

Fixed in https://github.com/cybozu-go/mantle/pull/63/commits/23710581842ba60c067359c7e3d6cabab5fe3468

@satoru-takeuchi I've fixed your points. Could you review this again?

@satoru-takeuchi I squashed my commits. Could you review this again? The code included in the commits shouldn't have been changed: https://github.com/cybozu-go/mantle/compare/afc7d1fa1bbb7c2f8300abdea63c4a8f7f2e5a87..2da741f78b859c867c7699fae40db6a477da95f7

Thank you!


Tobias Meixner suggestions

Dear Stefan,
thanks for the offer.
Though I am not interested. I think I will give your theme a few more release cycles to consider again.
I will be happy to promote is once its mature or has a new great feature.
For me for example you are missing alt-attributes on author images, tags are not linked in post list, author names are not linked, footer colors look mixed up, few images are missing in your demo blog, disqus container missing padding and please consider using gulp+scss to make it more flexible for user.
Thank you.

These are fixed:

[ ] Missing alt-attributes on author images
[ ] Tags should be linked
[ ] Author should be linked
[ ] Disqus container missing padding

About the missing images in my demo, it has to do with my blog, there is no issue in theme.


Command+B for Play the project shortcut sometimes takes two presses to activate

Godot version:
3.2.rc2.official
OS/device including version:
Macbook Pro 2014, macOS 10.14.16
Issue description:
Since updating to 3.2rc1, sometimes, but not always, usually when initially loading a project and trying to play it with Command+B keyboard shortcut, it takes two presses of the shortcut to actually start the project.
I did not see this happen with the latest 3.1 version. Reporting this for now,maybe will look into it more by compiling the engine and seeing what happens, but unfortunately don't have more info about this.
Steps to reproduce:
I don't know how to 100% reproduce this, as sometimes the Command+B works perfectly, but usually when initially I have loaded a project, and try to play it with Command+B, it needs two presses to start.
This might have something to do with the mouse focus, although I'm not sure, sometimes moving the mouse to a different location on the UI seems to help, this might be totally bogus though, I don't know.
Minimal reproduction project:
This seems to happen on any project I've already made.

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?

Can anyone still reproduce this bug in Godot 3.2.3 or any later release?

Hey! Still can see in 3.2.4 beta4 ! Feels like sometimes I have to click the main view to make it work. Not sure though, after I moved to using F5 i have not seen it happen, so it has something to do probably with Cmd+B being bound to multiple things or having to handle a key combination differently than a single key ? Just guessing.

Might be fixed in master, needs testing.

I was having issues running the game by pressing command + B on my Mac (M1, Ventura 13.2.1).
To fix it I had to remove a shortcut that was already using the same command as noted on a comment above.
Godot version: 4.0.1.stable

Still happening in 4.1.3.  Cmd-B is also bound to Bookmark by default, and when the cursor is sitting in an editor, that seems to take precedence. Removed that binding and Cmd-B to Run Project works reliably.

I guess the solution would be to change the shortcut for bookmark to Cmd-shift-B in mac?

I went looking for where this is defined and it seems like the current default is Option-B.
So maybe this issue is only hitting people who have a keymap that was created before this change?
https://github.com/godotengine/godot/blob/4c5a1484527ec5f8e320a24d0ee4c4fde0514776/editor/gui/editor_run_bar.cpp#L381C2-L381C2

I went looking for where this is defined and it seems like the current default is Option-B.

KeyModifierMask::META is Command not Option, Option+B can't be used since it's used to input some special characters (like ∫).

yeah the fix is as you said, change the shortcut for Go to Next Bookmark to Cmd-Shift-B.

It's already used as well as most others stuff (and already have other conflicts as well):
https://github.com/godotengine/godot/blob/4c5a1484527ec5f8e320a24d0ee4c4fde0514776/editor/plugins/script_text_editor.cpp#L2484-L2486
https://github.com/godotengine/godot/blob/4c5a1484527ec5f8e320a24d0ee4c4fde0514776/editor/plugins/script_text_editor.cpp#L2495-L2495
https://github.com/godotengine/godot/blob/4c5a1484527ec5f8e320a24d0ee4c4fde0514776/editor/plugins/canvas_item_editor_plugin.cpp#L5352
It's quire hard to find free combinations (at least without using 3+ modifiers) and avoid conflicts with input and system shortcuts. So it's definitely not the only one with this issue.

Yeah no kidding.  "Run Project" is probably one of the most important shortcuts and the current behaviour where it works sometimes is maddening.

I would be in favor of removing the bookmark shortcut for mac users just to get the running one by default


onChange event will trigger event onDayCreate

I just want to call onDayCreate  event when it render,but when i change the day will trigger event onDayCreate.

Hello,
I don't quite understand the problem.
Please elaborate.
Thank you

Nothing need to redraw,why the onDayCreate() hook?

@chmln Can you see the images?
I don't think it is reasonable to trigger onDayCreate(),when I just picks a date.
Like this onDayCreate().
flatpickr("#dayCreate", {
onDayCreate: function(dObj, dStr, fp, dayElem){
// Utilize dayElem.dateObj, which is the corresponding Date
	// dummy logic
	if (Math.random() < 0.15)
		dayElem.innerHTML += "";

	else if (Math.random() > 0.85)
		dayElem.innerHTML += "";
}

});

The calendar days are redrawn after selection, in a matter of milliseconds.
What's the problem?
If you make expensive API calls, they certainly don't belong to onDayCreate()

Yeah,of course.Actually,i run it well.Er,maybe,just maybe it can be better.


network-manager: link-local ip connections do not work

To reproduce: create new (ethernet) connection, with ip4 set to link-local.
When uping the connection, the following errors are thrown:
~> nmcli con up crossover
Error: Connection activation failed: Active connection removed before it was initialized

Jul 15 02:01:54 katara NetworkManager[19954]: <warn>  (enp0s25): Activation: Stage 3 of 5 (IP Configure Start) failed to start avahi-autoipd: not found
Jul 15 02:01:54 katara NetworkManager[19954]: <warn>  (enp0s25): Activation: failed for connection 'crossover'

An strace reveals the problem:
access("/sbin/avahi-autoipd", X_OK)     = -1 ENOENT (No such file or directory)
access("/usr/sbin/avahi-autoipd", X_OK) = -1 ENOENT (No such file or directory)
access("/usr/local/sbin/avahi-autoipd", X_OK) = -1 ENOENT (No such file or directory)
access("/bin/avahi-autoipd", X_OK)      = -1 ENOENT (No such file or directory)
access("/usr/bin/avahi-autoipd", X_OK)  = -1 ENOENT (No such file or directory)
access("/usr/local/bin/avahi-autoipd", X_OK) = -1 ENOENT (No such file or directory)

I thought the following would fix it (network-manager/default.nix):
    substituteInPlace src/devices/nm-device.c --replace '("avahi-autoipd", NULL, NULL)' '("avahi-autoipd", &"${avahi}/sbin/avahi-autoipd", NULL)'

but this creates a new problem:
Jul 15 01:56:30 katara NetworkManager[17223]: Failed to find user 'avahi-autoipd'.
Jul 15 01:56:30 katara NetworkManager[17223]: <error> [1436918190.768417] [NetworkManagerUtils.c:568] nm_utils_kill_child_sync(): kill child process 'avahi-autoipd' (18999): failed due to unexpected return value -1 by waitpid (No child processes, 10) after sending SIGKILL (9)
Jul 15 01:56:30 katara NetworkManager[17223]: <warn>  (enp0s25): Activation: failed for connection 'crossover'

@Profpatsch add that new user avahi-autoipd when NM is enabled in the nixos module.

@Profpatsch Did @lethalman's suggestion work?  I was just bitten by this issue.

@meteficha Did not try it futher yet. If you want you can try it.


Bugfix for TRUE_FALSE_PATTERN and LibreOffice/Numbers

Two small bug fixes. One results in strings like "True, this is not boolean" and "This is false" being interpreted as booleans. The other prevents the .xlsx files from opening in Numbers and LibreOffice.

+1
This would be enormously helpful!

@spekode would you create a separate pull request for the LibreOffice/Numbers fix?
thank you!

No problem! All set.


Sharing the pre-survey results needs to be part of the pre-workshop briefing

To make sure all instructors and helpers know this in advance.

fixed in 03ed5a6d


tour: memory issues

Context: https://tour.golang.org/concurrency/4
Attempting to run the sample code, I get this error /usr/local/go/src/sync/pool.go:9:2: read /usr/local/go/src/runtime/symtab_test.go: read /usr/local/go/src/runtime/symtab_test.go: cannot allocate memory

Seems to work ok for channel buffer size < 10

It works for me, even with buffer 100. Please try again later and confirm? Maybe it was a temporary server glitch.

Hello! Random contributor here. :smile:
Doesn't seem to work for me too:

It seems random, can you try again later?
More users complain, on different pages at different times.

Didn't work again. Can someone check whether the server is actually running out of memory?

I have the same problem, but if i add newline after for loop its working.

It's random, probably there are many containers (servers) that receive the Run calls, and some of them remain out of memory, depending on the online users probably.
I posted the issue on the Go developer group, I'm waiting for someone to investigate it (who has access to the production env).

Talk about crazy! @bgadrian I can verify that adding a newline after the loop (as indicated by @KraProgrammer) indeed works. Weird, eh?

Ah yes, I forgot, I discovered a difference between calls here https://github.com/golang/tour/issues/505
When you load the page for the first time, it loads (in the textarea) a bugged source code.
Any modification you do, like adding a new line, "it fixes" some invalid, invisible characters.
The issue is that the code is remembered in your browser, so to reproduce it you have to use a private browser session.

My guess is that it is a tour thing, if it was a bigger problem (related to the Go language) most likely was already fixed.
We have to fix it, noone is in charge of this project, officially.
I think next week I can find some time to further investigate the issue, hopefully then replicate it on a local env, and then make a pull request.
Hopefully it is something simple, like the source code formatting issue, or some incompatibility between the newer browser versions (most of the tour was not touched since 2-3years ago), or the servers are old and broken :)

Closing as a dup of  #541


[PDI-18139] Copy tables Wizard is broken and errors out with "There is no repository connection available" when connected to repository.

Fix for the https://jira.pentaho.com/browse/PDI-18139
The error is happening due to empty name of the transformation and invalid format of the name when uploading to the repository.
Proposing to set the name to transformation in advance in format that is acceptable by the pentaho-server.
Original transformation name contains '[',']' symbols that are not acceptable by pentaho-server REST API. Replacing it to alternative ' symbols that are maybe fine.
With the fix locally the transformation is getting created.
I am not so familiar with the code - if the the should have initialized somewhere else and reason is different - let's discuss.
Also not sure how to write the test for this component since it is a big component and I am not what is the design for the transformation name initialization.

:white_check_mark: Build finished in 18m 6s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1144, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

I am not able to access sonarqube report. I will be gratefull if someone will give me a note where is "Code Smells" rules are violated in those few lines of code that I have submitted.
Thanks!

:x: Build failed in 3h 1m 39s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message

:x: Build failed in 2h 41m 28s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:exclamation: No tests found!
Errors:
Filtered log (click to expand)

script returned exit code 1

:information_source:  This is an automatic message


Don't coerce counters to floats, avoid metrics handles

What does this PR do?
A brief description of the change being made with this pull request.
Motivation
What inspired you to submit this pull request?
Related issues
A list of issues either fixed, containing architectural discussions, otherwise relevant
for this Pull Request.
Additional Notes
Anything else we should know when reviewing?

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1144  👈 (View in Graphite)
#1143
#1141
#1140
#1139
#1138
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Copy dotfiles from assets to public

I apologize for posting in ost.io/chaplin, obviously it is related to brunch.
Is there any way to copy hidden files from assets to public other than .htaccess? It seems that has been hardcoded in, but there are plenty of other reasons to include hidden files in my output folder.
Thanks,
Daniel
brunch 1.7.13 (with chaplin/exoskeleton)
node v0.10.0
OS Mac 10.8.5

I'm running into this today in a Phoenix app I'm building. It defaults to having you put assets in a web/assets folder and uses brunch to copy them over.
I need to have a .well-known directory that has some files in it that I want brunch to copy over so they are served by my app.
It seems like that's currently not possible due to the hardcoding of all dotfiles being excluded with (it seems?) no way to override that

+1 on @jakecraige's comment above for the Phoenix issue.
We're also trying to add an Apple Pay domain verification file to our app and Brunch won't copy the .well-known/ directory nor its contents by default. A special config would do the trick because I can definitely see the issue copying all dotfiles by default.
See the Stripe docs.
Thanks!

Hey @markhealey @jakecraige, thanks for reporting.
We will expose chokidar's ignored as config.watcher.ignored in 2.11 release and make .well-known not ignored by default. Progress in #1664.


[FEAT]  How to use it in umi

Describe the feature
I want to try to use onlook in umi, which is part of my umijs configuration.

However, it was not detected that I used onlook in the application, and the application could not edit the code.
So I want to know the correct configuration method or when cli can support umi.
Thank you very much！

Hey @shancheng-77, there will soon be a new version where there is no longer a configuration step. This will be released in a few days. https://github.com/onlook-dev/onlook/pull/782
In the meantime, can you verify that the babel plugin is installed in package.json? Something like this
 "devDependencies": {
    "@onlook/babel-plugin-react": "latest",
  }

https://github.com/onlook-dev/onlook/blob/main/demos/babel/package.json#L40
Thank you for your patience :)

Hey @shancheng-77 try out the latest version of Onlook when you get a chance – there should be an update button that appears after 30 seconds when you start the app up again!

Hey @shancheng-77, there will soon be a new version where there is no longer a configuration step. This will be released in a few days. #782
In the meantime, can you verify that the babel plugin is installed in package.json? Something like this
 "devDependencies": {
    "@onlook/babel-plugin-react": "latest",
  }

https://github.com/onlook-dev/onlook/blob/main/demos/babel/package.json#L40
Thank you for your patience :)
I'm sorry that I just saw the message. At that time,I carried out this order. ： npm install -D @onlook/babel-plugin-react,
So there should be this dependence.

Hey @shancheng-77 try out the latest version of Onlook when you get a chance – there should be an update button that appears after 30 seconds when you start the app up again!

It seems to be okay. good job！


Add action field to petition

Added action attribute to petition model, with form field on front end. Also added rspec and cucumber tests for it.

Looks like the schema.rb got changed, currently investigating...

Apart from the already mentioned schema changes, this looks good to me.  Fix those and we can commit.  Separately we should work out what version of Postgres we're planning to deploy to, make sure we're all using that one, and regenerate a canonical new db/schema.rb from that version.


-V flag should be -v

C:\Users\jk23\dev\my_projects
λ adonis -v

  error: unknown option `-v'

C:\Users\jk23\dev\my_projects
λ adonis -V
2.1.8
```

Yup will get it fixed. Thanks for reporting

Just realized it is from commander instead of ace. https://www.npmjs.com/package/commander#automated---help.
Closing since Adonis makes use of commander


feat(backend): ip lookup, atomic asset creation, document updates.

Changes proposed in this pull request

https://github.com/interledger/rafiki/issues/2756

Context

Checklist

[x] Related issues linked using fixes #number
[x] Tests added/updated
[x] Documentation added
[x] Make sure that all checks pass
[x] Bruno collection updated

@koekiebox Most recent doc updates look good to me. Not sure if I should be the one to approve the PR given all the other updates.


CircleSnail.js - Exposing animation duration

CircleSnail has hard coded duration values which makes the animation a bit slow for me.
Exposing the duration and spinDuration values, with defaults to the current values.

Thanks for your PR! Would you mind adding these props to propTypes and the README?

Added to propTypes and README.md :-)

Thanks!


Derived types

Describe the bug
Some _idx and iXX variables are described using type(bounds_info) instead of the type(int_bounds_info) derived type. Possibly unexpected behavior can occur.
I suspect the first six occurrences below have this problem, plus the last one:
MFC/src $ grep -iR '(bounds_info)' ./*
./post_process/m_global_parameters.f90:    type(bounds_info) :: cont_idx                  !< Indexes of first & last continuity eqns.
./post_process/m_global_parameters.f90:    type(bounds_info) :: mom_idx                   !< Indexes of first & last momentum eqns.
./post_process/m_global_parameters.f90:    type(bounds_info) :: adv_idx                   !< Indexes of first & last advection eqns.
./post_process/m_global_parameters.f90:    type(bounds_info) :: internalEnergies_idx      !< Indexes of first & last internal energy eqns.
./post_process/m_global_parameters.f90:    type(bounds_info) :: stress_idx                !< Indices of elastic stresses
./post_process/m_derived_variables.f90:        type(bounds_info) :: iz1
./pre_process/m_global_parameters.fpp:    type(bounds_info) :: x_domain, y_domain, z_domain !<
./pre_process/m_initial_condition.fpp:    type(bounds_info) :: x_boundary, y_boundary, z_boundary  !<
./simulation/m_global_parameters.fpp:    type(bounds_info) :: stress_idx                !< Indexes of first and last shear stress eqns.

While adding compatibility for Cray and GNU 13 + OpenACC, I believe I have already corrected these occurrences in my fork!

Great, I will wait for that PR to fix it.

Fixed via https://github.com/MFlowCode/MFC/pull/52


Request to add an MX record to my domain.

Hi there,
How can I add a MX record to my domain hide.localplayer.dev so that I can use it for forwarding mail to my main inbox.
Thanks,
Kind Regards,
HID-E

It appears you have a CNAME record. As per DNS, you cannot include other records with a CNAME record. You would have to change your record in order to have additional records including MX.
https://www.rfc-editor.org/rfc/rfc1034#section-3.6.2


RFC: Travis caching

Problem:
A Travis builds job takes around ~20 minutes with 3 jobs per build, hence ~60 minutes of build time. Timing the builds I find that the big part of the time is spent on:

pip install (~4 min)
bower install (~30 secs)
``inveniomanage assets build``` (~1 min)
inveniomanage database create (3m - 3m 30s)

Proposal:
From above commands, we can potentially shave of something like 6 minutes for each build job by caching:

[x] lib/python/site-packages + bin/
[x] bower_components
[ ] database

I've created a small test script https://github.com/lnielsen-cern/zenodo/blob/master/traviscache.py which will create a .tar.gz ball out of a list of directories/files and stick it in Amazon S3, as well as start each build with trying to get the tar.gz an extract it. The tar-ball is named according to the pattern <TRAVIS_PYHTON_VERSION>-<HASH of requirements.txt, ...>.tar.gz, hence if one of the files in the hash changes content, the cache is invalidated, and it can also support the Travis build matrix (i.e. different python versions).
Additionally S3 will automatically delete objects older than 7 days to prevent it from growing out of hand, and make sure we test new package versions. The overhead of the cache is quite small:

download 200mb from S3 and extract it: 5 secs
compress 200mb and upload to S3: 9 secs

Using this on Zenodo I've managed to save 4m 30s.
Issues to address:

Amazon S3 costs money (and while we likely won't store lots of data, we might potentially use lots of bandwidth). Any other good alternatives?
Database: Ideas are welcome how we can optimise this. Current ideas include 1) mysqldump/load 2) cache MySQL data directory 3) use mixer. The hard part is how we're going to invalidate the cache.

I'm closing this one, as I think it's superseded by both Travis own caching as well as docker images.


Document Process for Debugging Local Bot on Channels

Document exact steps needed to connect a locally running bot to the Emulator while interacting with it using an external app or channel like Teams, Skype, Slack, etc.

#Run Your Bot in a Channel While Debugging Locally
If you have configured your bot to run in 1 or more channels and would like to test a locally running bot using the Emulator while interacting with it in a communication app (Teams, Skype, Slack, WebChat, etc.), these instructions will show you how.
Since chat messages, adaptive cards, and other features have notable differences across the many available channels, the Emulator cannot reasonably account for all of them. This guild show you how to have a locally running bot provides where changes can be made quickly, breakpoints can be set and the project can be rebuilt and restarted while interacting with it via an installed app like Teams or Slack.
#How it works
The tunneling software ngrok is used to create a tunnel to your locally running bot. The tunnel's URL is provided to your web app bot in Azure. You build your bot with an Emulator aware Adapter, run it locally then chat with it in Teams or any other available channel. The Emulator still receives the conversation's message exchange as usual.
#Let's get Started
If you haven't already, get the latest Emulator, ngrok and update your bot's dependencies to use BotBuilder v4.4+
1. Update Your Bot's Code
Your bot will need to become "Emulator Aware" in order to connect to and send the conversation exchange to the Emulator. Include the BotDebugger class somewhere in your bot's code:

2. Run ngrok
Open a terminal and run ngrok with the following command to create a new tunnel:
./ngrok http 3979 --host-header=localhost

The output in the terminal should look something like this:

3. Update Azure to Point to the Tunnel
In the azure portal, navigate to your bot's settings and paste the
url provided by the ngrok terminal in the Messaging endpoint field and save the changes. Do not forget to use /api/messages. It's best to create a Bot in Azure that is used specifically for this purpose. Do not overwrite
the messaging endpoint of a deployed production bot.
4. Connect to a Channel
If you haven't already done so, connect your bot to a channel. You may also need to download and install the app
associated with the channel if you have't already.
5. Debug!
Run your bot locally, then open the Emulator. In the Emulator choose file > Sidecar Debug Mode.

 This will place the Emulator in a read only mode an enable inbound
  connections from your locally running bot. Begin chatting with your bot in Teams, Skype, Slack or WebChat. Once the first message is received, your bot will connect
  to the Emulator and send it the conversation exchange.


Handle on cancelled

If this has been discussed in an issue, make sure to mention the issue number here.  If not, go file an issue about this to make sure this is a desirable change.
Issue #71

Thanks for this, I like this idea.
Not sure why the build failed, looks like a Travis issue to me.  I can manually verify if it keeps doing that.

Thanks for the PR! I really appreciate it.
Both Sam and I left a few comments. Just respond if something is unclear or if you want to discuss further..

And we're green!  Thanks for sticking through all of the comments,  Merging this now, and I will release a 0.5.2 release soon that includes this change.
Thanks for your contribution.

Finally!

@SUPERCILEX by the way I used the 'squash and merge' option so you may need to update your fork for future work.

Thanks for the tip!


RESTEASY-1418 RESTEasy should be able to provide trace information about request

It's just a preview of the feature. Here are the todo list:

We need to put log all the events in RESTEasyServerTracingEvent
We need to have ON_DEMAND tracing supported
Client side tracing events need to be designed and implemented
resteasy-tracing module needs to be removed, because tracing features need to be inside resteasy-jaxrs module.
tests...

I'll keep updating this branch.

tests on WildFly, not just UndertowJaxrsServer
Documentation, documentation, documentation ;)
WebConsole / file-system-log-file support to see the tracing data (web console probably via log view facility, I would suggest to have dedicated resteasy-tracing.log file, not to mix server & tracing stuff into server.log)
Have simple option for admins to disable tracing server wide - to avoid tracing requests processing at all and to save CPU
[optional] management api support to fetch tracing data
[to be enterprise level feature] management api support for deployment / server-wide configuration

When I look at https://issues.jboss.org/browse/RESTEASY-1418 it speaks about returning returning tracing logs in response headers. Also request for tracing details is client side driven via special header to request.
When I first looked into this PR I had feeling this is quite server side focused

to enable / disable it user needs to reconfigure stuff on the server
to get the tracing data user needs to have access to server logs

Second look revealed that the header support is there.
I miss analysis document on https://github.com/wildfly/wildfly-proposals/pulls with end user scenarios descriptions. Some good older examples can be https://developer.jboss.org/people/brian.stansberry/content Analysis of/Design notes documents, https://developer.jboss.org/wiki/AddingSupportForGitBackendForConfigurationFiles or https://developer.jboss.org/wiki/CLI-BetterCompletionForArguments
Another important aspect is possibility to enable and disable this feature without restart, just for investigation in real traffic. Many users have long time running instances without outages allowed. Just planned restarts in specified periods.

@rsvoboda Thanks for checking and feedback! And I'll check the documents you provided.
Still a long way to complete this feature ^_^

The ON_DEMAND mode needs to be supported so users could send tracing request in HTTP header, and get the tracing they want.
/**
+     * Enable tracing support.
+     *

+     * It allows service developer to get diagnostic information about request processing by RESTEasy.
+     * Those diagnostic/tracing information are returned in response headers ({@code X-RESTEasy-Tracing-nnn}).
+     * The feature should not be switched on on production environment.
+     *

+     *

+     * Allowed values:
+     *

+     *
{@code OFF} - tracing support is disabled.

+     *
{@code ON_DEMAND} - tracing support is in 'stand by' mode, it is enabled on demand by existence of request HTTP header

+     *
{@code ALL} - tracing support is enabled for every request.

+     *

+     * Type of the property value is {@code String}. The default value is {@code "OFF"}.
+     *

+     *

+     * The name of the configuration property is <tt>{@value}</tt>.
+     *

+     */

In addition, we need to define Client side events. Currently we have RESTEasyServerTracingEvent that implements RESTEasyTracingEvent, and we need the similar thing for client apis.

The current design should support run-time switching on tracing levels(including switch it on/off), in addition with ON_DEMAND mode. I'll digging into more details on this.

@rsvoboda , thanks for the feedback. Let me comment a bit on your previous message topics (@liweinan FYI):

tests on WildFly, not just UndertowJaxrsServer

+1, it should simply be a matter of having few tests within the testsuite/integration-tests submodule.

WebConsole / file-system-log-file support to see the tracing data (web console probably via log view facility, I would suggest to have dedicated resteasy-tracing.log file, not to mix server & tracing stuff into server.log)

IMHO the only thing we should be doing here is make sure the logs from the new tracing facility have a dedicated log category. Then the user can properly tune the logging subsystem and the console can be enriched to show/filter desired logs. In any case, any effort beyond having a dedicated log category should probably go under a separate jira.

Have simple option for admins to disable tracing server wide - to avoid tracing requests processing at all and to save CPU

Yep, ideally the goal is for the mechanism to have no impact on performances, when no trace message is written (default behavior). Additionally, it should be possible to enable/trigger logging on demand without having to restart the server.

[optional] management api support to fetch tracing data

Probably something for the console project or an external tool that can process logs.

[to be enterprise level feature] management api support for deployment / server-wide configuration

I'd say this should go under an additional jira for future enhancements.

RESTEasyTracingEvent interface is used to define tracing events, and RESTEasyServerTracingEvent is implemented(and events are defined, the next step is to log these events properly).
On the other hands, JBoss Logger has its own logging categories like:
21:09:08,083 DEBUG [request] Matched default handler path /logger
21:09:08,084 DEBUG [security] Attempting to authenticate HttpServerExchange{ GET /logger request {Connection=[Keep-Alive], User-Agent=[Apache-HttpClient/4.5.4 (Java/9.0.1)], Host=[localhost:8081]} response {}}, authentication required: false
21:09:08,084 DEBUG [security] Authentication outcome was NOT_ATTEMPTED with method io.undertow.security.impl.CachedAuthenticatedSessionMechanism@956019b for HttpServerExchange{ GET /logger request {Connection=[Keep-Alive], User-Agent=[Apache-HttpClient/4.5.4 (Java/9.0.1)], Host=[localhost:8081]} response {}}
21:09:08,084 DEBUG [security] Authentication result was ATTEMPTED for HttpServerExchange{ GET /logger request {Connection=[Keep-Alive], User-Agent=[Apache-HttpClient/4.5.4 (Java/9.0.1)], Host=[localhost:8081]} response {}}
21:09:08,085 DEBUG [general] REQUEST_FILTER Filter by [io.weli.tracing.HttpMethodOverride @150aede3] [ 0.01 ms]
21:09:08,085 DEBUG [general] REQUEST_FILTER Filter by [org.jboss.resteasy.core.AcceptHeaderByFileSuffixFilter @320916e1] [ 0.00 ms]
21:09:08,085 DEBUG [i18n] RESTEASY002315: PathInfo: /logger
21:09:08,085 DEBUG [general] REQUEST_FILTER Filter by [org.jboss.resteasy.plugins.providers.sse.SseEventSinkInterceptor @7c7ad254 #2147483647] [ 0.00 ms]
21:09:08,085 INFO  [general] METHOD_INVOKE Resource [org.jboss.resteasy.plugins.server.resourcefactory.SingletonResource @f4a49] method=[[org.jboss.resteasy.spi.metadata.DefaultResourceMethod @39b74c96]] [ 0.23 ms]

Currently we can see the tracing info is under [general] category, and we can divide them into finer categories. Let me add this requirement into my TODO list.

According to our discussion, let me create a series of subtasks under [RESTEASY-1418], so we can further  and cleanup this PR

@liweinan sure, go ahead

@asoldano Thanks for confirmation! I'll create subtasks under RESTEASY-1418, and finish this PR asap.

@asoldano According to team discussion, I have created several sub-tasks under RESTEASY-1418 to go on working on the task, and I guess this Pull Request can be treated as an initial commit for review.

The next sub-task I'll work on is RESTEASY-1890

This is the new PR for reviewing: https://github.com/resteasy/Resteasy/pull/1516

I'm closing this as it's superseded by #1533


Make EnumerableDataReader public

It was marked internal in #46 when it should have been left public.

Decided to keep this class internal.


Treat components as native HTML elements ✨

Summary
This PR aims to fix two issues related to types:
1) Treat and type components as HTML elements
Many times ( not always ) we were spreading the rest props on our components in order to enable the client pass native HMTL attributes to them:
const Foo = ({ children, ...rest }) => <span {...rest}>{children}</span>
<Foo className="bar" />

Now the intend is for every component to allow this behavior and get them typed correctly with React.HTMLAtrributes 💆🏻♂
2) Only wrap styled-components when necessary
Styleguidist, which uses react-docgen internally to locate components and generate documentation, wasn't able to locate components when declared just as a styled-components:
const Text = styled.span`
  color: pink;
`
export default Text

There's suggested fixes for this but not supported by react-docgen-typescript yet ... 🤯
A temporal solution is to export wrappers as a named export that Styleguidist can locate.
Nudge me in case you want more details on this, the solution is tricky to summarise... 😛
Try it out
$ yarn add @zopauk/react-components@next
$ yarn tsc --noEmit

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add data constructor/deconstructor blocks

Data constuctors are not quite functions, and easy recognizable in Haskell as they start by a capital letter.
They deserve a specially shaped variant of a function block.
And for use in pattern match/case expression the same sort of block could be used but then upside down.
Also data constructors seem to be missing in the catalog.
Related issue is #158 and #154.

If I understand it correctly, that means you'd have Just 10 as a data constructor block of Just and a value block of 10, instead of as a single value block?

Yes, but using a single value block for them would still be possible.
Now value blocks are a bit abused for everything that is not supported or inconvenient

Deconstructor blocks in general should wait until there is a case block, since you can't just deconstruct a sum type.
A deconstructor block for the (n-)tuple would be nice for now, although this should probably be a special shape on its own, together with the (n-)tuple constructor.

Not all constructors are sumtypes, and then you have the special case of constructors that are newtypes.

Deconstructors seems to be finished by #414.


Contributing guide(?)

I don't know how the project is running and if you guys need help with anything, but I've noticed that there is some features in TS/JS that we do not have in AssemblyScript like the JSON parser and stringify. Is there some special reason or it's just because no one implemented it yet? I mean, I've read on an issue that you guys don't want to implement Array.push because you don't want to deal with dynamic arrays, etc.
What would be the best approach: Create a PR adding the feature or create a separated lib like a pollyfill?
I'm trying to study the project to understand how could I help, so a contribution guide and a list of what is needed would be nice.

Just one more question: Whats exactly the difference between this repo and the original one? I mean, would that JSON integration work with the old compiler? I keep asking about the JSON integration because I'm trying to migrate a lib to AssemblyScript as experiment... I want to know what are the limitations and benefits in a real-world project today. Anyway, thanks for the fast answer and great explanation!

The prototype compiler has some, though limited, support for strings. String (or more precisely: class) support there is based on linking your code with a memory management runtime compiled from C (basically dlmalloc) and requires manually freeing no longer needed memory.
Other notable differences are that the prototype compiler is based on TypeScript (written in TypeScript), while this one here is standalone (written in "portable" AssemblyScript) so it can be compiled to WebAssembly eventually. In the new one I am avoiding using/linking with non-AssemblyScript code. The other issue from above has some thoughts on memory management in the new compiler.

Hey @dcodeIO, thanks again for the answer and explanation!
I'm actually migrating the elasticlunr library to AssemblyScript and it uses the JSON during the indexing process.
I will keep my research and maybe try to isolate the slower but simpler things and migrate them to AssemblyScript. I think this will help me understand how far along the project is and how should I start my studies over the compiler so I'm able to contribute.

Closing this issue for now as it hasn't received any replies recently. Feel free to reopen it if necessary!


Remove the binary-buildpack link from the front page of docs site

This link shouldn't be present on the main page.
#99518888]

Thank you, aemengo!


Feature/fix convert real

What problem does this PR solve?
[2021-03-15T06:50:51.604Z] - Test Convert from java.lang.Long to REAL *** FAILED ***

[2021-03-15T06:50:51.604Z]   BaseDataSourceTest.this.compSqlResult(sql, jdbcResult, answer, false) was false (BaseDataSourceTest.scala:195)

What is changed and how it works?
fix comparison between long and fload, because of Precision problem
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update the tidb-ansible repository
Need to be included in the release note

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: marsishandsome:x: ti-srebotYou have signed the CLA already but the status is still pending? Let us recheck it.


Lazy Loading support

Hi 😃 I'm here again

I just found this with the plugi takes sooome of time any idea how improve the lazy loading?

I just found this solution but if if have other idea 👍🏼 https://github.com/TeoDev1611/AstroVim/blob/astro/lua/plugins.lua#L99

Hm what exactly is being profiled here? All I see is just numbers! In my next todo I was actually planning on taking a look at performance and loading times, so would be interesting to know what exactly happens inside "nvim-lsp-installer config"

Is the nvim-lsp-installer plugin and config @williamboman the numbers are the startup time of the plugin

So no interactions with any Lua APIs? Just neovim loading the plugin itself? I have profiled that particular scenario before and there is no bottlenecks that would cause a 100ms load time (it's consistently one of my fastest plugins to load, averaging <0.04ms). Are you sure you're not calling on_server_ready or get_installed_servers() etc? These are highly i/o-bound and I'd not be surprised if they produce such high numbers

No this use some functions and configurations:
local lsp_installer = require 'nvim-lsp-installer'
local lsp_servers = require 'nvim-lsp-installer.servers'
local M = {}
local capabilities = vim.lsp.protocol.make_client_capabilities()
capabilities = require('cmp_nvim_lsp').update_capabilities(capabilities)

-- Use an on_attach function to only map the following keys
-- after the language server attaches to the current buffer
local on_attach = function(client, bufnr)
  local function buf_set_keymap(...)
    vim.api.nvim_buf_set_keymap(bufnr, ...)
  end
  local function buf_set_option(...)
    vim.api.nvim_buf_set_option(bufnr, ...)
  end

  -- Enable completion triggered by <c-x><c-o>
  buf_set_option('omnifunc', 'v:lua.vim.lsp.omnifunc')

  -- Mappings.
  local opts = { noremap = true, silent = true }

  -- See `:help vim.lsp.*` for documentation on any of the below functions
  buf_set_keymap('n', 'gD', '<cmd>lua vim.lsp.buf.declaration()<CR>', opts)
  buf_set_keymap('n', 'gd', '<cmd>lua vim.lsp.buf.definition()<CR>', opts)
  buf_set_keymap('n', 'K', '<cmd>lua vim.lsp.buf.hover()<CR>', opts)
  buf_set_keymap('n', 'gi', '<cmd>lua vim.lsp.buf.implementation()<CR>', opts)
  buf_set_keymap('n', '<C-k>', '<cmd>lua vim.lsp.buf.signature_help()<CR>', opts)
  buf_set_keymap('n', '<space>wa', '<cmd>lua vim.lsp.buf.add_workspace_folder()<CR>', opts)
  buf_set_keymap('n', '<space>wr', '<cmd>lua vim.lsp.buf.remove_workspace_folder()<CR>', opts)
  buf_set_keymap('n', '<space>wl', '<cmd>lua print(vim.inspect(vim.lsp.buf.list_workspace_folders()))<CR>', opts)
  buf_set_keymap('n', '<space>D', '<cmd>lua vim.lsp.buf.type_definition()<CR>', opts)
  buf_set_keymap('n', '<space>rn', '<cmd>lua vim.lsp.buf.rename()<CR>', opts)
  buf_set_keymap('n', '<space>ca', '<cmd>lua vim.lsp.buf.code_action()<CR>', opts)
  buf_set_keymap('n', 'gr', '<cmd>lua vim.lsp.buf.references()<CR>', opts)
  buf_set_keymap('n', '<space>e', '<cmd>lua vim.lsp.diagnostic.show_line_diagnostics()<CR>', opts)
  buf_set_keymap('n', '[d', '<cmd>lua vim.lsp.diagnostic.goto_prev()<CR>', opts)
  buf_set_keymap('n', ']d', '<cmd>lua vim.lsp.diagnostic.goto_next()<CR>', opts)
  buf_set_keymap('n', '<space>q', '<cmd>lua vim.lsp.diagnostic.set_loclist()<CR>', opts)
  buf_set_keymap('n', '<space>f', '<cmd>lua vim.lsp.buf.formatting()<CR>', opts)
end

local function setup_handlers()
  vim.lsp.handlers['textDocument/publishDiagnostics'] = vim.lsp.with(vim.lsp.diagnostic.on_publish_diagnostics, {
    virtual_text = {
      spacing = 5,
      prefix = '',
    },
    signs = false, -- rely on highlight styles instead, don't want to clobber signcolumn
  })
end

function M.setup()
  setup_handlers()
  vim.cmd [[ command! LspLog tabnew|lua vim.cmd('e'..vim.lsp.get_log_path()) ]]

  local ok, sumneko_lua = lsp_servers.get_server 'sumneko_lua'
  if ok then
    if not sumneko_lua:is_installed() then
      sumneko_lua:install()
    end
  end

  lsp_installer.on_server_ready(function(server)
    local opts = {
      on_attach = on_attach,
      capabilities = capabilities,
    }

    if server.name == 'eslintls' then
      opts.settings = {
        format = { enable = true },
      }
    end

    server:setup(opts)
    vim.cmd [[ do User LspAttachBuffers ]]
  end)
end

return M

Ah and with the configuration shown above takes this:

Yeah the heavy stuff is actually scheduled already, so just measuring that code it'll be blazing fast :P. I'm currently experimenting a bit with speeding up the parts that are scheduled

So I've done some work to defer require calls as much as possible. These are by far the biggest culprits in terms of performance. On my Windows machine a single require took ~5ms 🤢 (multiply that for each available server...). The problem today is that all the metadata for a given server exist exclusively inside each server's Lua module - which forces us to require all servers to do things like identify which ones are installed etc. I've started separating this a bit and colocating all the essential metadata in one location, and deferring all the "ad-hoc" metadata to the server modules themselves. This allows us to very quickly identify which servers are installed by scanning the root install directory once (via libuv), and then only require the installed servers.
The important metric is what's called when a user registers a on_server_ready callback, because that's the first touchpoint. Here are some profiling results I did across 3 different machines and operating systems:
macOS (11 servers installed)
37ms -> 10ms (~73% faster)

macOS (1 server installed)
37ms -> 1ms (~97% faster)

Windows (11 servers installed)
230ms -> 48ms (~80% faster)

Low-end Ubuntu machine (11 servers installed)
80ms -> 24ms (~70% faster)

I went a bit too far down the rabbit hole however and started looking at some other unrelated things at the same time (which resulted in for example https://github.com/neovim/neovim/issues/15648). Now it's all tangled together, will have to do work to extract the parts that are not ready yet 🙈

:o that's interesting i have a idea is possible call only one server when y call a lua function this could be make the startup time more faster like this:
:lua require('nvim-lsp-installer.lazy').load('sumneko_lua')

or something like this but can be have a setup option for if don´t want call to the server manually can be made this automatically example:
require('nvim-lsp-installer').setup{
       lazy = {
                  enable = true
       }
}

No need to make this opt-in behavior (would probably be a headache to even implement in the first place)! It'll be default going forward, just finishing up the last bits and pieces at the moment

@TeoDev1611 Would you mind trying out the io-speedups-new branch (https://github.com/williamboman/nvim-lsp-installer/pull/93) and see if it runs fine for you?

Yeaaaaaaah
This amazing works

Re: https://github.com/neovim/neovim/issues/15648 I have looked into it, I just haven't decided what to do about it yet :)

Re: neovim/neovim#15648 I have looked into it, I just haven't decided what to do about it yet :)

Hey! Thanks for the update! I posted another comment with how I could imagine things working in https://github.com/neovim/neovim/issues/15648.

Lazy loading added in #93


Turn ci.yml a callable workflow.

Updates the ci.yml adding the configuration to allow other workflows call it to run the tests.

I would expect a new make e2e-test, that runs things similar to what there is right now in e2e-tests/.

Yes, I think we should run e2e-tests as well. I don't think there's anything wrong with running make test though, just an extra check that needs to pass before we call a release, besides the e2e-tests (that as said, I think we should add).
Thanks for realizing about that!

Oh, from the conversation we had, I was thinking about reusing the workflow from the end-to-end-tests repo, not just reusing within this repo on the release workflow. Makes sense.


ipp everywhere Canon Pixma Pro 10 wrong print size

Hi, printing jpeg using ipp everywhere on Canon Pixma Pro 10 gives wrong print size (very small) and on top left corner.
Contrary to a few years back #5238, now setting the printer works out of the box using the ipp everywhere driver however format is wrong and printing a test page from cups server seems to be occupying more space on the page, but colors are wrong.
Using the below ppd used to be working perfect.. not anymore. I must admit I haven't been using the printer for a long time so I can't really know when it started not to work correctly.
*PPD-Adobe: "4.3"
*FormatVersion: "4.3"
*FileVersion: "1.19.0"
*LanguageVersion: English
*LanguageEncoding: ISOLatin1
*PSVersion: "(3010.000) 0"
*LanguageLevel: "3"
*FileSystem: False
*PCFileName: "ippeve.ppd"
*Manufacturer: "Canon"
*ModelName: "Canon PRO-10 series"
*Product: "(Canon PRO-10 series)"
*NickName: "Canon PRO-10 series"
*ShortNickName: "Canon PRO-10 series"
*ColorDevice: True
*cupsVersion: 2.2
*cupsSNMPSupplies: False
*cupsLanguages: "en"
*APSupplies: "http://000000F8A378.local./index.html?PAGE_AAP"
*cupsFilter2: "image/urf image/urf 100 -"
*cupsFilter2: "image/jpeg image/jpeg 0 -"
*OpenUI *PageSize/Media Size: PickOne
*OrderDependency: 10 AnySetup *PageSize
*DefaultPageSize: Letter
*PageSize 3.5x5/3.5 x 5": "<</PageSize[252 360]>>setpagedevice"
*PageSize 4x6/4 x 6": "<</PageSize[288 432]>>setpagedevice"
*PageSize 5x7/5 x 7": "<</PageSize[360 504]>>setpagedevice"
*PageSize 8x10/8 x 10": "<</PageSize[576 720]>>setpagedevice"
*PageSize A3/A3: "<</PageSize[841.889763779528 1190.551181102362]>>setpagedevice"
*PageSize A4/A4: "<</PageSize[595.275590551181<PHONE_NUMBER>79528]>>setpagedevice"
*PageSize A5/A5: "<</PageSize[419.527559055118<PHONE_NUMBER>51181]>>setpagedevice"
*PageSize B4/JIS B4: "<</PageSize[728.503937007874 1031.811023622047]>>setpagedevice"
*PageSize B5/JIS B5: "<</PageSize[515.905511811024<PHONE_NUMBER>07874]>>setpagedevice"
*PageSize Legal/US Legal: "<</PageSize[612 1008]>>setpagedevice"
*PageSize Letter/US Letter: "<</PageSize[612 792]>>setpagedevice"
*PageSize Postcard/Hagaki: "<</PageSize[283.464566929134<PHONE_NUMBER>55118]>>setpagedevice"
*PageSize Tabloid/11 x 17": "<</PageSize[792 1224]>>setpagedevice"
*CloseUI: *PageSize
*OpenUI *PageRegion/Media Size: PickOne
*OrderDependency: 10 AnySetup *PageRegion
*DefaultPageRegion: Letter
*PageRegion 3.5x5/3.5 x 5": "<</PageSize[252 360]>>setpagedevice"
*PageRegion 4x6/4 x 6": "<</PageSize[288 432]>>setpagedevice"
*PageRegion 5x7/5 x 7": "<</PageSize[360 504]>>setpagedevice"
*PageRegion 8x10/8 x 10": "<</PageSize[576 720]>>setpagedevice"
*PageRegion A3/A3: "<</PageSize[841.889763779528 1190.551181102362]>>setpagedevice"
*PageRegion A4/A4: "<</PageSize[595.275590551181<PHONE_NUMBER>79528]>>setpagedevice"
*PageRegion A5/A5: "<</PageSize[419.527559055118<PHONE_NUMBER>51181]>>setpagedevice"
*PageRegion B4/JIS B4: "<</PageSize[728.503937007874 1031.811023622047]>>setpagedevice"
*PageRegion B5/JIS B5: "<</PageSize[515.905511811024<PHONE_NUMBER>07874]>>setpagedevice"
*PageRegion Legal/US Legal: "<</PageSize[612 1008]>>setpagedevice"
*PageRegion Letter/US Letter: "<</PageSize[612 792]>>setpagedevice"
*PageRegion Postcard/Hagaki: "<</PageSize[283.464566929134<PHONE_NUMBER>55118]>>setpagedevice"
*PageRegion Tabloid/11 x 17": "<</PageSize[792 1224]>>setpagedevice"
*CloseUI: *PageRegion
*DefaultImageableArea: Letter
*DefaultPaperDimension: Letter
*ImageableArea 3.5x5: "18.141732283465 14.173228346457<PHONE_NUMBER>83465<PHONE_NUMBER>53543"
*PaperDimension 3.5x5: "252 360"
*ImageableArea 4x6: "18.141732283465 14.173228346457<PHONE_NUMBER>83465<PHONE_NUMBER>53543"
*PaperDimension 4x6: "288 432"
*ImageableArea 5x7: "18.141732283465 14.173228346457<PHONE_NUMBER>83465<PHONE_NUMBER>53543"
*PaperDimension 5x7: "360 504"
*ImageableArea 8x10: "18.141732283465 14.173228346457<PHONE_NUMBER>83465<PHONE_NUMBER>53543"
*PaperDimension 8x10: "576 720"
*ImageableArea A3: "18.141732283465 14.173228346457<PHONE_NUMBER>62992 1176.377952755905"
*PaperDimension A3: "841.889763779528 1190.551181102362"
*ImageableArea A4: "18.141732283465 14.173228346457<PHONE_NUMBER>34646<PHONE_NUMBER>33071"
*PaperDimension A4: "595.275590551181<PHONE_NUMBER>79528"
*ImageableArea A5: "18.141732283465 14.173228346457<PHONE_NUMBER>38583<PHONE_NUMBER>04724"
*PaperDimension A5: "419.527559055118<PHONE_NUMBER>51181"
*ImageableArea B4: "18.141732283465 14.173228346457<PHONE_NUMBER>91339 1017.637795275591"
*PaperDimension B4: "728.503937007874 1031.811023622047"
*ImageableArea B5: "18.141732283465 14.173228346457<PHONE_NUMBER>94488<PHONE_NUMBER>61417"
*PaperDimension B5: "515.905511811024<PHONE_NUMBER>07874"
*ImageableArea Legal: "18.141732283465 14.173228346457<PHONE_NUMBER>83465<PHONE_NUMBER>53543"
*PaperDimension Legal: "612 1008"
*ImageableArea Letter: "18.141732283465 14.173228346457<PHONE_NUMBER>83465<PHONE_NUMBER>53543"
*PaperDimension Letter: "612 792"
*ImageableArea Postcard: "18.141732283465 14.173228346457<PHONE_NUMBER>12598<PHONE_NUMBER>08661"
*PaperDimension Postcard: "283.464566929134<PHONE_NUMBER>55118"
*ImageableArea Tabloid: "18.141732283465 14.173228346457<PHONE_NUMBER>83465 1209.826771653543"
*PaperDimension Tabloid: "792 1224"
*HWMargins: "18.141732283465 14.173228346457 17.858267716535 14.173228346457"
*ParamCustomPageSize Width: 1 points<PHONE_NUMBER>66929 1008
*ParamCustomPageSize Height: 2 points 360 1916.220472440945
*ParamCustomPageSize WidthOffset: 3 points 0 0
*ParamCustomPageSize HeightOffset: 4 points 0 0
*ParamCustomPageSize Orientation: 5 int 0 3
*CustomPageSize True: "pop pop pop <</PageSize[5 -2 roll]/ImagingBBox null>>setpagedevice"
*OpenUI *InputSlot/Media Source: PickOne
*OrderDependency: 10 AnySetup *InputSlot
*DefaultInputSlot: Rear
*InputSlot Auto/Automatic: "<</MediaPosition 0>>setpagedevice"
*InputSlot Rear/Rear: "<</MediaPosition 18>>setpagedevice"
*InputSlot Manual/Manual: "<</MediaPosition 4>>setpagedevice"
*CloseUI: *InputSlot
*OpenUI *MediaType/Media Type: PickOne
*OrderDependency: 10 AnySetup *MediaType
*DefaultMediaType: Stationery
*MediaType Photographic/Photo Paper: "<</MediaType(Photographic)>>setpagedevice"
*MediaType Stationery/Stationery: "<</MediaType(Stationery)>>setpagedevice"
*CloseUI: *MediaType
*OpenUI *ColorModel/Print Color Mode: PickOne
*OrderDependency: 10 AnySetup *ColorModel
*ColorModel RGB/Color: "<</cupsColorSpace 19/cupsBitsPerColor 8/cupsColorOrder 0/cupsCompression 0>>setpagedevice"
*ColorModel Gray/Grayscale: "<</cupsColorSpace 18/cupsBitsPerColor 8/cupsColorOrder 0/cupsCompression 0>>setpagedevice"
*DefaultColorModel: RGB
*CloseUI: *ColorModel
*DefaultResolution: 600dpi
*OpenUI *cupsPrintQuality/Print Quality: PickOne
*OrderDependency: 10 AnySetup *cupsPrintQuality
*DefaultcupsPrintQuality: 5
*cupsPrintQuality 4/Normal: "<</HWResolution[600 600]>>setpagedevice"
*cupsPrintQuality 5/High: "<</HWResolution[1200 1200]>>setpagedevice"
*CloseUI: *cupsPrintQuality
*OpenUI *print-scaling/Print Scaling: PickOne
*OrderDependency: 10 AnySetup *print-scaling
*Defaultprint-scaling: auto
*print-scaling none/None: ""
*print-scaling fill/Fill: ""
*print-scaling fit/Fit: ""
*print-scaling auto-fit/Auto Fit: ""
*print-scaling auto/Automatic: ""
*CloseUI: *print-scaling

@mfauvain What operating system/Linux distribution are you using?
What version of CUPS are you using?
How are you submitting print jobs?

Hi Michael, Arch, kernel 5.11.6, cups 2.3.3
I tried submitting the job with different gnome apps, same result. I also tried to print a test page from cups webserver, page size seems to be better reflected, however colors are not I believe (I actually never obtained a colored test page from the cups webserver before)

@mfauvain Hmm, can you do the following:

Enable debug logging (cupsctl --debug-logging will do this)
Print a page from whatever application
Attach a copy of the error_log along with a picture of the resulting output

Thanks!

Here are the print configuration, the output (scanned) on a US letter paper (one can see the test page from cups webserver on the back) and the log.

error_log.txt

@mfauvain Weird, the log shows a grayscale ("auto-monochrome") print and grayscale raster data being sent. Will try to reproduce with my Canon TS-series printer.
Can you print in color?
If so, have you checked for firmware updates for your printer (just in case this is a grayscale printing bug)?

I confirmed that my Canon PIXMA TS5120 does not reproduce this issue...

Hi Michael, thx, I just tried to print a color pdf and it works perfect. Tried also (again) a color jog and it does print in color (I might have left grayscale by mistake in my latest try) but scale is still wrong (unlike for the pdf that prints full page with margins (I just stopped the process to save ink)..
logs and scan attached.
log.txt
Scanned Document.pdf

@mfauvain Everything looks correct in the log, and even the JPEG file is coming across as a PDF...
Can you grab the "d" file from /var/spool/cups for a JPEG job after you print?  It will be the PDF coming from your application.

The d file is as the printed output.. small image on top of the page. So I guess cups is not to be blamed..
I'll continue investigating. thx for the help.

No worries, thanks for letting us know!


Problem at the end of List

Hi, I made a page with an infinite loop structure in my project by using your project. but when I come to the end of the list, it still works as if there is an element in the list and tries to load it. Can we put a limit on lazyLoad?

@kaydeniz can you explain in detail? couldn't get your point.

`
import {getQuestionDetailsInTest} from "../../../../../_apis/addQuestionApi/addQuestionApi";
import useLazyLoad from "../utils/lazyLoad";
import {Stack} from "@mui/material";
import QuestionView from "./QuestionView/QuestionView";
import clsx from "clsx";
import React, {useRef} from "react";
import CircularProgress from "@mui/material/CircularProgress";
import PropTypes from "prop-types";
function LazyComponent(props) {
const {questionList} = props;
const triggerRef = useRef(null);
const onGrabData = (currentPage) => {
const param = {questionIds: [questionList[currentPage].questionId]}
return new Promise((resolve) => {
let data = {}
getQuestionDetailsInTest(param,
(responseData) => {
data = responseData
},
() => {
}
).then(() => resolve(data))
});
};
const {data, loading} = useLazyLoad({triggerRef, onGrabData});

return (
    <Stack spacing={2}>
        {data.map(question => {
            return <QuestionView key={question.questionId} questionData={question}/>
        })}

                <CircularProgress/>

    </Stack>
);

}
LazyComponent.propTypes = {
questionList: PropTypes.array
};
export default LazyComponent;
`
I prepared the above component to show the questions in my project. As I scroll down on the page, I call a question and display the incoming data. I also show the last question in the list, but still onGrabData works as if there is a question in the list and I get an error. I wonder if I have to limit the size of my list in the useLazyLoad hook.

@ashutosh1919 Hi, Have you had a chance to look into the problem?


Implement reversion logic in MySQL record set repo for failed changes

Fixes #500.
Changes in this pull request:

Handle failed record set changes in MySQL record set repo
Add integration test

Note for reviewers:
To test this, your modules/api/src/main/resources/application.conf should be updated to remove the record-set from vinyldns.dynamodb.repositories and add record-set {} to vinyldns.mysql.repositories.

👍


Redo storage interface and architecture

Implement TransactionData class for producer which will incapsulate all required logic.

Closed.


Remove leading indentation from (some) pod code blocks.

There are numerous instances of =for and =begin code blocks marking already indented code peppered through the documentation. This indentation is preserved in the rendered output, which is typically unintentional. So, for example:
=for code
    say 'hello';

is rendered as
    say 'hello';

rather than
say 'hello';

Although this is a fairly boring commit, it touches a lot of files—hence the PR rather than direct push. Note that If you ignore whitespace (git diff master -w), no content has changed. There're also no test or xtest regresssions.

Thanks!


SRE is now installable

I have only tested this on linux.

SRE is now installable. (Thanks to Kasper)


Interface-to-interface type assertion

This feature require:

Each type full method table storage available at run-time due to
method set matching.
Global method matching cache.

It is implementable, but additional global state seems obnoxious.
Most (99.9%) of Emacs code will not ever reference that metadata.
This issue state may change when reflect package will be developed.

In the current implementation, only precomputed method tables are used.


Implement the QKA runtime

Runtimes are being replaced with Session. See https://github.com/Qiskit/qiskit-ibm-runtime/ for more details.


Typography Helper

Port the existing helper over.
Development Checklist:

[ ] Component(s)
[ ] ... with PHPdocs
[ ] Unit testing
[ ] ... with >80% coverage
[ ] User guide updated
[ ] Classmap integration?
[ ] Securely signed commits
[ ] Conforms to style guide

In CI3 we have Typography Helper and Typography Library. The helper just loads the library for two of its functions and Security Class for entity_decode function.
Would it make sense to merge helper and library and have just a Typography class (library) with static methods, no helper needed?

Initial pull request made. Don't know how to create tests. :-(

Sorry. Just now getting a chance to look at this. My initial reaction was a No to static methods, but your pull request doesn't use any, so that's all good. :)
I'll make some comments on that PR, since there's some formatting and style issues that need to be corrected, etc.
This article at SitePoint goes over the basics of using PHPUnit so that should get you started. Didn't you work with tests on the previous PR you made? Between that article and the existing tests, that should get you going, but don't hesitate to ask questions if you're unsure of something. I'll help you through it, but I'd really like to see tests with your PR's.


Can't use an undefined value as an ARRAY reference

Since upgrading to 2.8 (and still on 2.9), I'm getting the following error when trying to extract cited refs to a .bib file.  (this runs fine in my normal latexmk workflow, without the --output_format=bibtex, producing a .bbl file).
Omitting some warnings (which I think are about ISBNs and duplicate keys and I think unrelated):
➜ biber --output_format=bibtex talker-variability.bcf
INFO - This is Biber 2.9
INFO - Logfile is 'talker-variability.blg'
INFO - Reading 'talker-variability.bcf'
INFO - Found 76 citekeys in bib section 0
INFO - Looking for bibtex format file 'talker-variability.bib'
INFO - LaTeX decoding ...
INFO - Found BibTeX data source 'talker-variability.bib'
WARN - BibTeX subsystem: /tmp/cRfLN_Zqdw/talker-variability.bib_23427.utf8, line 8, warning: 194 characters of junk seen at toplevel
Can't use an undefined value as an ARRAY reference at /tmp/par-64617665/cache-975de9a2a2c44ccaa5329d9695a54026ffc29f68/inc/lib/Biber/Internals.pm line 1100.

Apologies for the lack of a MRE, but I've uploaded the bcf and bib files here as a gist.  If I find the time I'll try to chase down a MRE.

That .bcf file you are using - was this generated with biber <2.8? The format has changed in 2.8 which I suspect is your problem.

Nope, it's generated with 2.8.

Please try biber 2.10 DEV version from the development folder on Sourceforge. This was a bug.

Works great! Thanks for the quick fix.

I had same problem with 2.9, and 2.10 fixed it. Here is the link to the binaries
https://sourceforge.net/projects/biblatex-biber/files/biblatex-biber/2.10/binaries/


Neighbours ordering

According to the docs for DAGs:  neighbours are listed in reverse order of their addition to the graph. Is it possible to modify edges, rather than delete/insert and somehow to guarantee certain ordering of the neighbours (maybe according to the edge weight or smth like that).

It would be like sorting a linked list.


cannot find symbol import io.airlift.http.client.AsyncHttpClient

While building airpal I am getting following error.. Please help me !
I am trying to build airpal by using latest version of presto
warning: [options] bootstrap class path not set in conjunction with -source 1.7
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airlift/http/client/OldJettyHttpClient.java:13: error: cannot find symbol
import io.airlift.http.client.AsyncHttpClient;
^
symbol:   class AsyncHttpClient
location: package io.airlift.http.client
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airlift/http/client/OldJettyHttpClient.java:66: error: cannot find symbol
implements AsyncHttpClient
^
symbol: class AsyncHttpClient
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airlift/http/client/OldJettyHttpClient.java:227: error: cannot find symbol
public <T, E extends Exception> HttpResponseFuture executeAsync(Request request, ResponseHandler<T, E> responseHandler)
^
symbol:   class HttpResponseFuture
location: class OldJettyHttpClient
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airlift/http/client/OldJettyHttpClient.java:381: error: cannot find symbol
implements HttpResponseFuture
^
symbol:   class HttpResponseFuture
location: class OldJettyHttpClient
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/core/execution/InputReferenceExtractor.java:14: error: cannot find symbol
import com.facebook.presto.sql.tree.UseCollection;
^
symbol:   class UseCollection
location: package com.facebook.presto.sql.tree
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/core/execution/InputReferenceExtractor.java:113: error: cannot find symbol
protected CatalogSchemaContext visitUseCollection(UseCollection node, CatalogSchemaContext context)
^
symbol:   class UseCollection
location: class InputReferenceExtractor
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/core/execution/ExecutionClient.java:29: error: package org.jetbrains.annotations does not exist
import org.jetbrains.annotations.NotNull;
^
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/presto/QueryRunner.java:6: error: cannot find symbol
import io.airlift.http.client.AsyncHttpClient;
^
symbol:   class AsyncHttpClient
location: package io.airlift.http.client
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/presto/QueryInfoClient.java:11: error: cannot find symbol
import io.airlift.http.client.AsyncHttpClient;
^
symbol:   class AsyncHttpClient
location: package io.airlift.http.client
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/presto/QueryRunner.java:19: error: cannot find symbol
private final AsyncHttpClient httpClient;
^
symbol:   class AsyncHttpClient
location: class QueryRunner
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/presto/QueryRunner.java:21: error: cannot find symbol
protected QueryRunner(ClientSession session, JsonCodec queryResultsCodec, AsyncHttpClient httpClient)
^
symbol:   class AsyncHttpClient
location: class QueryRunner
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/presto/QueryRunner.java:42: error: cannot find symbol
private final AsyncHttpClient httpClient;
^
symbol:   class AsyncHttpClient
location: class QueryRunnerFactory
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/presto/QueryRunner.java:44: error: cannot find symbol
public QueryRunnerFactory(ClientSessionFactory sessionFactory, AsyncHttpClient httpClient)
^
symbol:   class AsyncHttpClient
location: class QueryRunnerFactory
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/presto/QueryInfoClient.java:39: error: cannot find symbol
private final AsyncHttpClient httpClient;
^
symbol:   class AsyncHttpClient
location: class QueryInfoClient
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/presto/QueryInfoClient.java:42: error: cannot find symbol
public QueryInfoClient(AsyncHttpClient httpClient, JsonCodec queryInfoCodec)
^
symbol:   class AsyncHttpClient
location: class QueryInfoClient
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/modules/AirpalModule.java:61: error: cannot find symbol
import io.airlift.http.client.AsyncHttpClient;
^
symbol:   class AsyncHttpClient
location: package io.airlift.http.client
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/modules/AirpalModule.java:69: error: package org.jetbrains.annotations does not exist
import org.jetbrains.annotations.Nullable;
^
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/modules/AirpalModule.java:158: error: cannot find symbol
public AsyncHttpClient provideQueryRunnerHttpClient()
^
symbol:   class AsyncHttpClient
location: class AirpalModule
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/modules/AirpalModule.java:194: error: cannot find symbol
@Named("query-runner-http-client") AsyncHttpClient httpClient)
^
symbol:   class AsyncHttpClient
location: class AirpalModule
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/sql/beans/TableRow.java:21: error: cannot find symbol
import static com.google.inject.internal.util.$Preconditions.checkNotNull;
^
symbol:   class $Preconditions
location: package com.google.inject.internal.util
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/sql/beans/TableRow.java:21: error: static import only from classes and interfaces
import static com.google.inject.internal.util.$Preconditions.checkNotNull;
^
/Users/sumit/Documents/workspace/airpal/src/main/java/com/airbnb/airpal/modules/AirpalModule.java:309: error: cannot find symbol
@Nullable
^
symbol:   class Nullable
location: class AirpalModule
22 errors
:compileJava FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':compileJava'.

Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED

same issue for me

I was encountering a similar issue but was able to get things running by using JDK7 and ./gradlew clean shadowJar to build.

happens to me when trying to use different version of presto to build, e.g.
./gradlew clean shadowJar -Dairpal.prestoVersion=0.87

I have the same problem when I set pretoVersion=0.144.1, anyone could help us?

https://github.com/airlift/airlift/commit/d93edf1895e3953c1c06018586a2757a86da497b
AsyncHttpClient removed from airlift at 2014, replaced by HttpClient.

so how can we fix it?

./gradlew -Dairpal.prestoVersion=0.77 clean shadowJar
Use presto version 0.77 as Airpal source uses this version .
You can find dependency jars inside Airpal folder ...presto-spi-0.77 ,presto-main-0.77,presto-client-0.77 etc. check also build.gradle file which contains presto version
ext {
prestoVersion = "0.77"
if (System.properties['airpal.prestoVersion'] != null) {
prestoVersion = System.properties['airpal.prestoVersion']
}
}

If you want to use latest versions of presto you need to change the Airpal source code to resolve compilation errors by referring to presto api.


feat :sparkles:: Load and save models functions

I opened this PR as per our Issue #24 , to load and save models
I followed the same conventions as much as possible, please feel free to edit the repo or shoot me a comment
Thanks and have a great day @xuhongzuo

Hi @elsheikh21 ,
Thanks so much for your interest, and I apologize for my late reply.
I reviewed your commits. It is a little weird to implement load_model as an inner-class function.
Suppose that the class of a detection model is instantiated as a variable model1, and has been trained and saved.
If we want to use this saved model, then we should instantiate a new variable model2, and use load_model, but this function does not return the saved model to this variable model2, but produces a new instantiated variable model3. Of course, we can directly return this new variable to replace the model2.
model1 = Class()
model1.fit(X)
model1.save_model(path)

model2 = Class()
model3 = model2.load_model(path)

This implementation is totally right, but personally, it might be not very elegant.

@xuhongzuo
thanks and no need to apologize.
Anyway, I totally understand your point and I do agree with you,
I could not implement it only inside the class DeepSVDD, as I also require the functionality from all other models. So, It was not very practical to rewrite the same method in all classes, that's why I went to BaseClassAD
# An example of the Implementation of the Save and Load Model inside children's classes
Class DeepSVDD(BaseClassAD):
      # Existing code
      def save_model(se;f, path):
            with open(f"{path}.pkl", mode="wb") as f:
                  pickle.dump(self, f)

      def load_model(self, path):
            model = None
            with open(f"{path}.pkl", mode="rb") as f:
                     model = pickle.load(f)
             return model

I tried changing the load_method(self, path) instead of doing self = pickle.load(f) as follows, but it did not work obviously, I failed to point out the problem source,
# An Example of implementation of the same functionality inside the Parent Class
Class BaseClassAD(BaseClassAD):
      # Existing code
      def save_model(se;f, path):
            with open(f"{path}.pkl", mode="wb") as f:
                  pickle.dump(self, f)

      def load_model(self, path):
            with open(f"{path}.pkl", mode="rb") as f:
                     self = pickle.load(f)

So the solution provided was mainly focused on functionality and not code elegancy, but let me know how to approach the problem so we have an elegant functional code
Thanks again and regards

After searching a simple obvious idea came to mind which is adding the load_model() as a class method of BaseDeepAD in this case, we can load the model easily
class BaseDeepAD:
# Existing code
     @classmethod
     def load_model(cls, path):
         with open(f"{path}.pkl", mode="rb") as f:
             return pickle.load(f)

so now it is tested locally and functions as following
model1 = DeepSVDD() # or any other child model (of BaseDeepAD)
model1.fit(X)
model1.save_model(path)

model2 = DeepSVDD.load_model(path)

Please let me know wdyt?

I just tried this solution. It seems very fantastic.


Update readme

I want to update readme file. I will add information about Hacktoberfest and contributors of this repository.

Please assign me this issue.


Conway spec prose: Ratification

Description
Add prose explaining the ratification procedure introduced in the Conway era and formalized in the Ledger.Ratify module.
This is a new version of #159.  (I tried to resurrect the old PR, but was not able to because I had recreated the branch william/145-Ratification on which the PR was based.)
This PR addresses one of the items of issue #145.
Checklist

[X] Commit sequence broadly makes sense and commits have useful messages
[X] Code is formatted according to CONTRIBUTING.md
[X] Self-reviewed the diff

Thank you, this is looking much better. I left a bunch of comments, here are two more things:

isCC, isDRep, isSPO, getStakeDist, verifyPrev and delayingAction could all use some aligning

Okay, I'll fix that today.

I like the macros file & I think it's ok to have it committed while it still needs manual tweaking, but we should get it automatically generated as part of the build process if we can

We could integrate the generation of some of, or perhaps most of, the macros file.  However, we would still need to hard-code the names of some macros, since we can't use unicode or numbers in macro names.
For now, I'll leave the macro file fls-macros.tex in the repo.

Now that we're using macros for this (Ratification) section of the pdf, we should make an issue to go back over the PParams and GovernanceActions sections and use macros so that the typesetting of symbols is consistent.  I'll make the issue after I finish incorporating @WhatisRT's requested changes into this PR.

Also, since this almost done, please don't force push this branch anymore, just put commits on top. Otherwise I'll have to fully review this again to merge it.


Choose/Click Attributes and Drag to Query Window

Desired Functionality:
-- From the "Connections" pain, normally on the left, expand a connection to view individual columns/attributes
-- In a table/view, choose one or more attributes and drag to the query window
-- Fully qualified column/attribute names populate to the query window. (table_name.attribute_name)
-- Normal functions of the ctrl and shift keys work with no surprises
-- Attributes are comma delimited
-- Ability to select: database name, table name, view name, column name, etc.
-- Reference the functionality in the Teradata SQL Assistant, and others.
It's time consuming and  unproductive to manually type all the attribute names.  I can be so much more productive by semi-automating the chosen attributes.
Dragging and Dropping is normal behavior in 2019, even in Chrome/Electron!
Thank you...

Closing as Dupe: https://github.com/microsoft/azuredatastudio/issues/3630


Unexpected behaviour when using mini.ai in html

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
mini.ai
Description
When using mini.ai on a html file the usual ways of interacting with html tags behave incorrectly. I noticed this with all actions that involve tags for ex: "vat", "dat", "cit", etc.
Neovim version
NVIM v0.9.5
Steps to reproduce
With mini.ai installed open any html file and try to select around any tag, the behaviour is very inconsistent also.
It seems to occur more  on tags that have multiple parents.
Expected behavior
Should not alter the default behaviour of vanilla nvim
Actual behavior
https://github.com/echasnovski/mini.nvim/assets/44908537/f9fd113b-8862-4d83-a815-9307641f04ae
This is me creating an index.html file adding a couple of tags and trying to select around a tag.

'mini.ai' works using Lua patterns which are not designed to deliver 100% compatibility with parsing structured nested data.
If there is a concrete example of something not working as expected, please provide a step-by-step reproducible example. The attached video is not reproducible for me. Also, chances are that this was already discussed in one of the previous issues.
If you want to disable particular built-in textobject, you can disable it by setting to false entry in custom_textobjects. Like this.
Closing as there is not enough information.


commands/repos; leftovers from old integration tests?

I didn't find anything that uses the repositories in commands/repos.  There are the cleanup scripts script/clean-git-text-fixture and script/clean-git-text-fixtures which clean them up.  The cleanup scripts are invoked from script/fmt, so no hint there either.
Can the repos and the cleanup scripts be removed?
If so, just merge in/cherry pick the commit mentioned below.  (In branch michael-k/git-lfs@cleanup)

Yup, I made a mental note this weekend to clean them up. Thanks for the reminder.


Create a harness for measuring thread times and dropped frames in flutter

@yjbanov has created flutter drive which can do functional integration tests.
This bug is about extending such to tell us about thread times and dropped frames.
Until we have something like this, we can't easily know (in an automated fashion) if we're fast yet.

Fixed by https://github.com/flutter/flutter/pull/3058. Next steps: create some benchmarks, create dashboard showing benchmark results.


dangerouslyGetAll breaks all prismic queries

Versions

gatsby-source-prismic: 5.2.5
gatsby-plugin-prismic-previews: 5.2.5
node: v14.17.6

Steps to reproduce

Upgrade to "gatsby-plugin-prismic-previews": "^5.2.5" and "gatsby-source-prismic": "^5.2.5"
gatsby clean just in case
yarn run dev

Nothing is loaded when not in preview (I have ~100 Content Post live documents):

When I try to preview a document, I get a more helpful error message

I had to upgrade the prismic gatsby due to another preview bug, which I'll document in another issue. I'll downgrade to an earlier version (5.2.0) in the meantime
What is expected?
Prismic content to be queried and loaded correctly.
What is actually happening?
No Prismic content is being loaded.

Hey @kb1995, apologies for the late reply. The version of @prismicio/client has been updated to the latest current version, v6.4.2, which will ensure the correct version is used.
This update to the Gatsby plugins will be published soon with this fix.


BUG | default schema not being applied in CI

Describe the bug
I am attempting to query a table in DB and getting an error in GitHub Actions that I am not getting running locally.
Query being attempted:
supabase.table('meeting_types').select('*').execute()

I get the error:
requests.exceptions.MissingSchema: Invalid URL '***/rest/v1/meeting_types?select=%2A': No schema supplied. Perhaps you meant http://***/rest/v1/meeting_types?select=%2A

Screen shot of error stack trace:

I'm doing lots of querying and inserting locally with no issues. First time trying to replicate in CI and am getting this error. I'm using v0.0.3 of the supabase package and like I said I've been successfully running this locally and have replicated that setup in CI.
Let me know of any additional context you may need 👍 thanks in advance!

Apologies for the late response; does the bug still exist? I'm around 90% sure it should be fixed in newer versions of the library, but I haven't tried reproducing this in a CI environment yet.

I'll update to the latest version of the library and give it another try and report back.

Thank you so much!
Just seeing the traceback, I can guess what caused it: in the past, postgrest-py only supported async requests, so here at supabase-py we monkey-patched it to use the requests library to provide sync support (Source)
I'm guessing some headers, like the one for default schema were missed in this patch

Closing due to a lack of activity. I believe this should be fixed in the newer versions but if the issue persists, feel free to reopen.


Add compile time function to compute the size of AdditiveShare

The benefit of using it is it does not require trait bounds math as Serializable trait and it can be used in const context

It seems that coverage is codecov issue: https://github.com/private-attribution/ipa/actions/runs/11379712512/job/31657642117?pr=1357#step:6:1295. This code is covered


Email configuration options set via ENV

Implement the appropriate changes required to enable email sending in Spring Boot. Values should be set via environmental variables via the application.properties file.

Already complete


Version up to 1.4.6

Description
Current code will "rotten" on 14.12.2022. It requires new release.

Version up to 1.4.6

How has this been tested?

[ ] GitHub workflow "rottenness" should pass after new release

Please, review but merge after main branch action "rottenness" fail on 14.12.2022


wasi: moves test-only overrides out of config, simplifies error tests

This moves test-only overrides out of config in preparation of
normalized configuration. This also simplifies error tests by using the
API directly for all cases except exit.

thanks for the 👁 @mathetake


Hard-coded namespaces in istiod

Bug description
In istiod, there are hard-coded namespaces in "istiod.istio-system", e.g.,

https://github.com/istio/istio/blob/master/pilot/pkg/bootstrap/certcontroller.go#L123
https://github.com/istio/istio/blob/master/pilot/pkg/bootstrap/certcontroller.go#L124
if hostname == "istio-pilot.istio-system.svc" {
names = append(names, "istiod.istio-system.svc")
}
https://github.com/istio/istio/blob/master/pilot/pkg/bootstrap/certcontroller.go#L128
https://github.com/istio/istio/blob/master/pilot/pkg/bootstrap/certcontroller.go#L129
if hostname == "istiod.istio-system.svc" {
names = append(names, "istio-pilot.istio-system.svc")
}
https://github.com/istio/istio/blob/master/pilot/pkg/bootstrap/istio_ca.go#L164
caServer, startErr := caserver.NewWithGRPC(grpc, ca, maxWorkloadCertTTL.Get(),
false, []string{"istiod.istio-system"}, 0, spiffe.GetTrustDomain(),
true)
https://github.com/istio/istio/blob/master/pkg/istio-agent/sds-agent.go#L204
discHost = "istiod.istio-system.svc"
https://github.com/istio/istio/blob/master/pkg/istio-agent/sds-agent.go#L354
serverOptions.CAEndpoint = "istio-citadel.istio-system:8060"
https://github.com/istio/istio/blob/master/pkg/istio-agent/sds-agent.go#L359
https://github.com/istio/istio/blob/master/pkg/istio-agent/sds-agent.go#L362
serverOptions.CAEndpoint = "istiod.istio-system.svc:15010"
} else {
log.Info("Using default istiod CA, with K8S certificates for SDS")
serverOptions.CAEndpoint = "istiod.istio-system.svc:15012"

[X ] Configuration Infrastructure
[ ] Docs
[X ] Installation
[X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[X ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)

Would be better to use priority when tracking internal work item.

Not stale - we should add a test outside of istio-system

it seems code here(https://github.com/istio/istio/blob/master/pilot/pkg/bootstrap/certcontroller.go#L121-L124) is not ever hardcoded could we close it?

All usages mentioned above and as detected by rg "istio-system" **/*.go -l | grep -v test | xargs rg istio-system -C 2 seem appropriate or resolved. Closing


child object filtering / sorting does not work

Please see the attached POC of the issue.  I discussed briefly on gitter chat about this issue with @oskardudycz .  It will work fine as long as you don't have to do any type of querying on the child object.
MartenSubObjectTest.zip

Looks like a duplicate of #870, closing this one.


[feat] add ignore filetype setting to config

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
trailspace
Description
I have highlighted trailspace in Lazy buffer, when updating plugins. How can i disable this behaviour?

Red color

This is already possible via buffer-local variables. See this section in help file.
Adding something like this should remove highlighting in 'lazy.nvim' buffer:
vim.api.nvim_create_autocmd('FileType', {
  pattern = 'lazy',
  callback = function(data)
    vim.b[data.buf].minitrailspace_disable = true
    vim.api.nvim_buf_call(data.buf, MiniTrailspace.unhighlight)
  end,
})

Somewhat a duplicate of #52 and others.


fix vulnerabilities

Vulnerabilities fixed by dependabot:
https://github.com/EnMarche/en-marche.fr/pulls?utf8=✓&q=dependabot