Support for Xtensa architecture

First, I would like to thank you for your amazing work! We recently started working on HIL for esp-hal based on embedded-test which depends on semihosting.
As you may know, some ESP chips are Xtensa based, so I wanted to ask if you have any plans to add support for it.
Semihosting on Xtensa
I haven't found any documentation on the Xtensa semihosting standard, so I asked some of our colleagues, here is the information that I gathered:

When espressif added Xtensa support into OpenOCD, they chose to use ARM semihosting spec, instead, because OpenOCD already had good support for it. Besides, RISC-V also uses ARM semihosting spec.
For a reference of the "official" Xtensa semihosting standard (as used in Cadence tools and in QEMU), there is some  code in https://github.com/qemu/qemu/blob/master/target/xtensa/xtensa-semi.c, https://github.com/espressif/newlib-esp32/blob/esp_based_on_4_1_0/libgloss/xtensa/syscalls.c — look for the branches with !OPENOCD_SEMIHOSTING and in Linux https://github.com/torvalds/linux/blob/master/arch/xtensa/platforms/iss/include/platform/simcall-iss.h.
This is the part of OpenOCD where the architecture-specific semihosting stuff happens and then call the "common" (ARM) semihosting code: https://github.com/espressif/openocd-esp32/blob/master/src/target/espressif/esp_xtensa_semihosting.c

https://github.com/espressif/esp-idf/blob/master/components/xtensa/include/xtensa/semihosting.h

I tried to implement this about a year ago but gave up because the compiler distributed via espup did not support the simcall instruction. (I didn't know about OpenOCD semihosting that using break instruction)
https://github.com/taiki-e/portable-atomic/pull/86#issuecomment-1487488398

I mainly use QEMU and semihosting crate to test no-std targets, so I tried to do the same with Xtensa, but the compiler didn't seem to recognize the simcall instruction so I gave that up testing on CI and decided to test locally using your wokwi-server (it worked well).

If things haven't changed (and we need to use simcall instruction), implementing this would require some work on esp-rs/rust or espressif/llvm-project first.
If it is OpenOCD semihosting (that using break instruction) that needs to be implemented (does this not seem to work with QEMU?) ), it is probably relatively easy to implement, but which is what you need for your use case?

When implementing this on my end, perhaps the hardest part is setting up a proper test environment.
Looking at the working branch at the time (which unfortunately contains code that cannot be published), it appears that we tried to work around the above problem by calling SIMCALL via C FFI, but failed to properly set up the QEMU and startup code. (The implementation on the semihosting side or the testing side or both was incorrect.)
In other words, perhaps it will not be possible to implement this on my end in the near term unless you or someone else can provide a suitable QEMU setup to test the implementation of this, or test it in some other way.
Therefore, marking this as help-wanted.

If things haven't changed (and we need to use simcall instruction), implementing this would require some work on esp-rs/rust or espressif/llvm-project first. (Although technically we could use C FFI)

Things havent change afaik.

If it is OpenOCD semihosting (that using break instruction) that needs to be implemented (does this not seem to work with QEMU?) ), it is probably relatively easy to implement, but which is what you need for your use case?

I think this is the case, in our end, the break approach should do the trick, and we could help with the testing, either with QEMU or doing it manually if needed.
As @mabezdev suggested, looks Ike what we need to do to get it working is having a https://github.com/espressif/esp-idf/blob/master/components/xtensa/include/xtensa/semihosting.h version of https://github.com/taiki-e/semihosting/blob/main/src/sys/arm_compat/syscall/riscv.rs


CB-11383 adding r4 instances to aws enabled list

See detailed description in the commit message.

Billing?

They are adding their part


Added back Best Practices (req'd for Native FW GA)

@lauramthomas @sweeneyskirt-sl @spider-sauce hi guys! I saw that one of you had written Best Practices section, but it wasn't published yet. I published it because I'm writing Native Frameworks RDC error messages and pointing to these new docs rather than the Wiki. If it's a problem lemme know.

Thank you for contributing to the sauce-docs project!
A well described pull request helps maintainers quickly review and merge your change
Before submitting your PR, please check our contributing
guidelines, applied for this repository. Avoid large PRs, help reviewers by making them as simple
and short as possible.

Description

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[x] I have read the contributing document.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hey @kimsauce -- I think the reason the "best practices"section was not published is because Meredith wants that kind of content to ultimately go in the "Community" site, not the docs site. You might want to circle back with her, because I don't think we want this content published here. --N

@kimsauce @sweeneyskirt-sl Yes, Meredith wanted me to port the BP docs but not publish them. We should probably touch base with her to verify, but I think you might need to just point them to the CB pages for now. She wasn't sure where the BP content should live on the new site.

@sweeneyskirt-sl @lauramthomas -- thank you both for providing the background! That makes sense, I will check with Meredith.


Post-Merge Survey Experiment

As discussed during the SIG meeting Nov 18th we want to try out if we can get a survey established that asks new comers to answer one (or "a few") questions after their PR has been merged.
Assumptions:

New comers who worked through the effort of getting their PR merged, are either in one of the following categories (1) one time contributors that want to get that one own issue fixed, (2) paid contributors to continuously contribute to the project, (3) hobby or side-project contributor that want to help here and there in their free time or a small portion of their working hours.
After they got their PR merged, they have shown some serious effort that they want to contribute (vs "hi, I want to contribute" on slack or simply raising an issue), so they already went a step further.
After they got their PR merged, maintainers were hopefully helpful and they are more inclined to answer such a survey. If the experience was not good, they might also want to let us know about that.

In this survey we can identify to which one of the groups (1) - (3) they belong to and how their contribution experience was, and what feedback they may have for us to improve.
Next Steps:
@svrnm is going to draft a workflow for a post merge survey that gets triggered after a PR got merged and if the contributor is not a member of the github org
@kaylareopelle will draft the message + survey send

I was searching for a ready to use github action, what I found so far:
https://github.com/actions/first-interaction/issues/226


reuse individual backup list entries

A common sequence of operations is to first list backups, then restore one of the listed backups, and then monitor the list of available backups with regular polling.
Since both the restore operation and the uncached list operation can load manifests, we'd rather not have the combined memory usage of both operations at the same time.
If the restore operation takes longer than the list response TTL (one hour with the default configuration), then a regular polling of the backups list will cause that simultaneous memory usage right after the cached response expires.
There is value in expiring the entire response after some time, because the list of backups can change without astacus being notified of that change.
However, the details about a single backup are not supposed to change, backups are immutable (until they are deleted).
Even when the list response is expired by TTL, reuse individual entries from the old response (if the entry still exists in the new response).
With that modification, the cached list response will expire, but since the individual entries will not be expired, the new list response will be built by listing the backup names from object storage but without loading the manifests.
Ideally, we should pre-compute all the memory-intensive values when creating the backup instead of doing it again every time we list the backups. But we want to improve the restore operation for backups that already exist.
This change also improves the "normal" flow of operation even when restore are not involved: instead of re-loading all manifests every hour, we will only load the most recently created backup once, right after the backup is created.
[DDB-881]

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

Project coverage is 89.97%. Comparing base (8142a72) to head (78eff28).

Files
Patch %
Lines

astacus/coordinator/list.py
80.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #186      +/-   ##
==========================================
- Coverage   89.97%   89.97%   -0.01%
==========================================
  Files         145      145
  Lines       10441    10460      +19
==========================================
+ Hits         9394     9411      +17
- Misses       1047     1049       +2

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

I had to add an extra commit to fix CI, ZooKeeper 3.8.3 was removed from the CDN.


Resolve paths

I get an error when I want to extend a template file from another directory.
Folder structure:
templates
↳ layouts
  ↳ _base.twig
↳ home
  ↳ _entry.twig

Content for templates/home/_entry.twig:
{% extends 'layouts/_base' %}

{% block content %}
{% endblock %}

Error Message:
ERROR in ./templates/home/_entry.twig (./node_modules/html-webpack-plugin/lib/loader.js!./templates/home/_entry.twig)
Module not found: Error: Can't resolve 'layouts/_base' in '/Users/user/Projects/noyb/templates/home' @ ./templates/home/_entry.twig (./node_modules/html-webpack-plugin/lib/loader.js!./templates/home/_entry.twig) 1:0-24

Craft CMS (which uses Twig) does not complain about my setup.
I would love to keep my twig files untouched. Is there any way that twig-loader resolves all files inside a default_path (even without using namespaces)

A loader does not resolve files, twig-loader simply converts template references to require statements so webpack can resolve them.
You need to configure webpack to fit your resolving needs. Take a look at the docs here: https://webpack.js.org/configuration/resolve/

When resolving paths in webpack, twig is unable find the files.

Same here looks that resolve doesnt work

I also tried to use resolve.alias to resolve files absolute, but can't make it work. Anybody got this working with the twig-loader?

Same problem here. I added
  config.resolve.modules = [
    ...config.resolve.modules,
    path.resolve(__dirname, "../templates"),
  ]

to my webpack config, but something like {% import "components/Form/Input.html.twig %} as Input fails to be resolved. Anybody found a solution?

Still doesn't work as far as I can tell. Works in twigjs-loader however.


Fix for #148

This fixes a bug in Get-VSTeamBuildArtifact where an error is returned because the API returns an extra record along with the list of artifacts that is called 'build.sourcelabel' and contains a URL but no "properties" object.
This PR stops the error by excluding any build artifact records returned by the API that don't contain a "properties" object from getting the build artifact specific type names applied. The result is still returned to the user however.
Alternatively if we think it would be better to exclude this result entirely, then it could instead be filtered out in the Get-VSTeamBuildArtifact function.
PR Summary

PR Checklist

[ ] Write Help
[ ] Write Unit Test
[ ] Update CHANGELOG.md

@markwragg would you able to add a unit test for this? Do you need help? I am also now a maintainer of this repo. So I could help you much better with problems regarding your PR.

I've added a test for this in my PR #257
Feel free to copy it into your PR and to merge this one :)


(PUP-7451) Add handling for Retry-After headers

Prior to this commit, the HTTP client used by the Puppet agent had special
handling for 3xx error codes in order to handle HTTP redirects. This patch
re-factors the response handler so that 503 and 429 error codes with
Retry-After headers set will result in a sleep followed by a retry of the
request. In particular, this allows a Puppet Server to re-splay its agent
population when overloaded by sending a 503 response with a randomized
Retry-After header, as implemented in SERVER-1767.

This is a bit of a work in progress. The code is done, but the mocks in existing tests need to be adjusted and new tests added to exercise the new behavior. Also, it looks like the code path for handling HTTP responses may be duplicated over in:
https://github.com/puppetlabs/puppet/blob/5.0.1/lib/puppet/util/http_proxy.rb#L169
So, handling for Retry-After headers would have to be duplicated over there if appropriate.

CLA signed by all contributors.

Can i nominate this for the next PR triage? @joshcooper @Magisus

@Sharpie one other question here is ... does the server issue the same absolute future available time in the header to all clients? Or it skewed / distributed based on when the request was received?
See the bit about auto-DDoS at:
http://www.nurkiewicz.com/2015/02/retry-after-http-header-in-practice.html

The server-side bit functions similar to splay in that there is a configurable max delay which defaults to 30 minutes. The server multiplies this max by a uniform random float between 0 and 1 and uses the result as the value of the Retry-After header included with 503 responses.
So, each agent will get a different sleep time which should push stagger their runs apart and break up a herd.
We effectively suffer from the auto-DOS issue right now as a restart or failure of an overloaded Puppet Server will unblock all the waiting agents at once creating a herd of agents with synced runtimes.

Added the $runinterval setting as a limit on Retry-After time.

@Iristyle this look OK to you now? what needs to happen before merging?

The method for handling a request has grown in length and complexity to the point that I'm not 100% comfortable saying I can understand it. Ideally it would be more like 10 lines than 75. Would it be possible to refactor a bit and pull bits out to make the core logic clear?

@pcarlisle I've factored the header parsing out into a separate method. The remaining logic can't be moved due to the use of next to advance the outer loop that drives retries and redirects.

@smcclellan I've re-factored the handling for Retry-After headers out into a separate method and added some docs. Let me know what you think!

@smcclellan Added a test around the parse_retry_after_header method returning zero when given a date that has already passed. I also made a small addition that adds the hostname of the remote server to the log messages if it is available.

This looks clean, as much as it can be.
I don't really like the idea of just sitting and blocking on requests - it would be nice to allow higher levels of Puppet to continue to do other work. That requires much bigger refactors (which I think might be worthwhile for other reasons), though.
I'm fine seeing this land as-is, but I'm going to make sure I write up my thoughts on refactoring this.

🎉 Thanks a bunch for the review and feedback!


Failing to call CMD commands when INITSYSTEM=on

When using resin/odroid-ux3-python:latest
If on DOCKERFILE:
INITSYSTEM=on
CMD bash start.sh

then start.sh is not executed (it's only executed if called as ENTRYPOINT bash start.sh)
If on DOCKERFILE:
INITSYSTEM=off
CMD bash start.sh

then start.sh is exectuted

@ntzovanis one clarification, this is on resinOS (unmanaged) when one does rdt push, correct?

@shaunmulligan that's correct

@petrosagg or @nghiant2710 any ideas on why this would happen?

@ntzovanis can you please send us the Dockerfile? I can't reproduce your issue, it seems fine on my side

@nghiant2710 did you try reproduce this on pure resinOS with rdt ... the issue only appears there and not on resin.io

@shaunmulligan is right. The issue happens on pure resinOS (and rdt). I've tried it in different computers and odroids with the same result.

# Enable systemd
ENV INITSYSTEM on

ENV PYTHONUNBUFFERED=0

# Your code goes here

WORKDIR /usr/src/mqtt-client

COPY . .

RUN pip install paho-mqtt
RUN pip install requests
RUN pip install pytz

ENTRYPOINT bash start.sh```

Where start.sh can be anything and it's not called at all.


Cache seed improvements

changes

ignore order of bbox coordinates
allow 0 tile buffer to be used
test on sendTiles function
test on SeedTile function
corrected default value for max_zoom
memory cache is now registrable

should close #437 and partially #453

Also, this branch covers the changes in #444

Pull Request Test Coverage Report for Build 1279

15 of 66 (22.73%)  changed or added relevant lines in 4 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-4.2%) to 39.165%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/tegola/cmd/server.go
0
2
0.0%

maths/maths.go
0
3
0.0%

cmd/tegola/cmd/cache.go
13
59
22.03%

Files with Coverage Reduction
New Missed Lines
%

cmd/tegola/cmd/cache.go
5
22.93%

Totals

Change from base Build 1277:
-4.2%

Covered Lines:
3734

Relevant Lines:
9534

💛  - Coveralls

Just merged this branch into my fork and the cache-seed performance improvement seems to be huge!

LGTM. only request is to add the other test cases for the SeedTile switch.


ETIMEDOUT and lack of callbacks with remote MySQL server

This issue affects both node-mysql, and node-mysql2. It is a hole in the pool API that can allow a potential edge case to totally destroy your server. Here is a rough breakdown of the problem and my workaround...

Using remote MySQL server, a machine where your Node is not running.
Use a connection pool to connect, say 20 connections.
Issue a query every 5 - 10 seconds or something from a browser or several, doesn't really matter.
Wait a duration of time roughly equal to your MySQL connection timeout (mine was 90 minutes).
IF a query is in flight, and you are waiting on a callback whilst or after ETIMEDOUT error occurs on a connection... the query never seems to time out, or the callback you are expecting is never fired. Often, if not always, on my setup.
By placing a suicide timer alongside the query, and invoking the callback yourself with an error after killing the connection end/destroy will prevent deadlock...
Without the suicide timer, more deadlocks occur and eventually your node server will stop serving anything at all. If you reverse-proxy, you will see 503 errors.

General observation:
ConnectionPool doesn't seem to correctly handle connection.on("error",function(){}); and "end" events, or they escape/abort the query callback somehow so it is never fired. The particular error that messes everything up is:
{ [Error: read ETIMEDOUT]
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'read',
  fatal: true }

And whilst it's seen at the global connection.on handlers, it doesn't get returned by the query.
Hope this helps someone, somewhere approach this problem. I'll dig into the code sometime to see if I can help... but I've lost a lot of ground to chasing this down and need to play catchup first.

Closing due to lack of response. Please feel free to post back with answers to the questions and I can always reopen :)

Having the same issue with node 4.4.0

have the same issue

Please, if no one is going to actually provide any information that I requested above, nothing is going to change, so there is no point in commenting at all. I'm going to lock the issue since no one actually providing any information to help and is just creating noise.


added support for _mm_cvtpd_epi32 / cvtpd2dq

i am not a hundred percent sure if the last three test are what we really want here, just looks weird that it "defaults" to i32::MIN but maybe that's supposed to happen, can't really find docs about that behavior.

Thanks!
I'm not sure where that behavior is documented either. The Intel docs say that each float is converted using Convert_FP64_To_Int32, but the behavior of Convert_FP64_To_Int32 isn't documented anywhere. It does at least appear consistent with Rust's rules for converting f64 values to i32 values, so I guess that's good?
Anyway, this looks great to me! Awesome work. :-)


python code to split Dataset

Split Dataset into testing and training data

[ ] implementing cuda support

we did not retrain our model, but that could be a good starting point in the future
there is already some code - look at
https://github.com/nosqlgeek/ai-meets-museum/blob/imageClassification/src/tools/imageClassification/splitDataset.py
https://github.com/nosqlgeek/ai-meets-museum/blob/imageClassification/src/tools/imageClassification/trainModel.py


Cannot pass object with selected fields to resolution method

To select fields from an object, EntityGraphQL creates a proxy object for it. This proxy object is incompatible with the original type. The incompatibility causes a runtime error when the proxy object is passed to a method used to resolve a subsequent field. This PR tests this scenario.

I swear this was in the docs :)
Current recommendation is to pass in fully selected fields (like the test was doing with p.Id).
You can pass many
schema.Type<Project>().AddField("config", "Get project config")
    .ResolveWithService<ConfigService>((p, srv) => srv.Get(p.Id, p.OtherField, p.More));

This page explains what is happening. Basically it first selects the Id, OtherField & More fields than can pass those into the service.
You should be able to do something like this too
schema.Type<Project>().AddField("config", "Get project config")
    .ResolveWithService<ConfigService>((p, srv) => srv.Get(new MyArgType { Id = p.Id, p.OtherField = p.OtherField, More = p.More }));

Even though the "How EntityGraphQL handles ResolveWithService() with EF" you referenced is about EF, it looks like the creation of the proxy object is independent of EF, which is why the unit test, which doesn't use EF, fails when the full entity is passed to the helper method.
In the production case where I encountered this issue, I am using EF. I passed the entity to look up related records using the DB context. The database has a legacy structure with several multi-column primary keys, which means foreign key lookup using the navigation properties requires far less code. My equivalent to Get was like this:
from child in db.Children where child.Parent == parent

But that caused the proxy object error. Based on your feedback, I fixed the problem by passing and using each key column individually:
from child in db.Children where child.Parent.KeyColumn0 == parentKeyColumn0 && child.Parent.KeyColumn1 == parentKeyColumn1 && child.Parent.KeyColumn2 == parentKeyColumn2

The flow of the code is the same, as I suspect is the generated SQL. Thanks for pointing me in the right direction. If a clever solution comes to mind on day that would allow working with the full entities, it would be handy. For now, if there's no known practical way to do so, feel free to close this PR.

It'd be possible to add implicit cast operators and maybe override equality function. Just not sure how that would translate to SQL. Just throwing ideas in 🙂

It turns out that passing the key column values as separate parameters leads to a different problem:
The query is like this:
query GetCustomers {
  customers {
      locations {
        services {
            id
        }
      }
  }
}

Where services is added to Location via AddField, with Location.CustomerId being one of the key fields. The result is a runtime error like this:

"Field 'customers' - 'location_CustomerId' is not a member of type 'Location' (Parameter 'propertyOrFieldName')"

I couldn't repro this in unit test. (I tried using TestDataContext.Projects.Children to get an equivalent depth.) It may be the underlying problem is fixed. I was able to work around the issue by passing the entity to the service method (the Get equivalent) using dynamic. Thought I'd mention in case it's useful.

Thanks. Yes I believe the issue you hit is fixed in the main branch here

It turns out that passing the key column values as separate parameters leads to a different problem

@breyed can you confirm if this second issue is resolved in 4.x?

@lukemurray Yes, passing the key column values as separate parameters works in 4.0.0.


CLI: Signing keys

This was going to be put inside an RFC, but the reality is that it's pretty short.
The CLI needs to be able to support signing envelopes, otherwise it's not very useful as all gobl documents require at least one signature to be valid. The main complication around this is where and how to store the JSON Web Keys.
There are two main options we've come up with so far:

Support loading ~/.gobl/config.yaml file with a configuration containing private keys (this is what Invocli does)
An SSH-style approach whereby private and public keys are stored in a ~/.gobl directory like id_es256.jwk for the private key and optionally id_es256.pub.jwk for the public key.

The SSH-like option is perhaps the most interesting as it opens up a set of possible commands on the interface:

Implies there is a default key, useful for setting up gobl as a microservice.
Adding an ssh-like -i ~/.gobl/test_es256.jwk option to specific a new identify file works easily.
A gobl keygen command can create a key and output it to the ~/.gobl directory.

GOBL already contains everything required to generate signatures inside the dsig package, so this is just a question of creating an interface.
We're expecting users to be able to upload their public keys to Invopop for storage, they should be easy to get to either by looking at the ~/.gobl/id_es256.pub.jwk file, or by generating it from the private key via the command line.
Key files are always in JSON format. Is it useful to add a .jwk or .json extension to each?

I think this issue has now been implemented. Are there any details I've overlooked that still need attention, @samlown ?


Emoji Reaction

Hello, the stickers do not appear when users use them in a message, the emoji comes out as if it were a separate message.
In other words, if 30 people Like a message, there will be 30 messages that whaticker will receive.
Any Solution?
Thanks

There is a property in baileys that returns whether it was a reaction to a message or something else
following example in terminal:

Endpoint:

NOTE: If it is not a reaction, the value null is returned.
I hope I helped, enjoy.

There is a property in baileys that returns whether it was a reaction to a message or something else
following example in terminal:

Endpoint:

NOTE: If it is not a reaction, the value null is returned.
I hope I helped, enjoy.

How Solve in this code?

This looks like a whaticket-community issue, not a baileys one


Performance improvements 🔥 🔥 🔥

Hi. I wanted to share some ideas about docs performance improvements.
TL;DR:
I have some history, so let's go.
Using pwmetrics some measurements were done.
P.S. It’s taking measurements using Nexus 5 devices emulation and 1.6 Mbps network throttling.

So, looks like there some place for opportunity,
Below some inconsistent behavior of FOIC (flash of invisible content) and FOUC (flash of unstyled content) can be noticed.
FOIC

FOUC

It will be nice to show fallback font as soon as possible and add progressive enhancement strategy to the site.
But, first of all, scripts which are blocking First Paint should be managed.
Timeline trace

So, adding defer attribute to all these scripts (except vue.js, we will manage it later) brought us to

100ms for TTCI and TTFI not so impressive but it’s something.
Commit - b996ecf
Continue….

Adding defer to vue.js saved us
FCP -  3,528 sec -> 1,972 sec
FMP - 4,627 sec -> 3,419 sec
PSI - 6,116 sec -> 4,912 sec
TTFI - 6,066 sec -> 4,861 sec
TTCI - 6,066 sec -> 4,861 sec

Good, but we can do more.
Commit - 97ea5db
P.S. Adding defer requires to warp example codes into domContentLoaded listener, but I think it can be easily done in scope of hexo-renderer-marked fork.

Looks like fonts loading is blocking FMP
Timeline trace

Since browsers repaint after loading resources founded in body

we can cheat and move loading resources fonts there.
Commit - 7fba6b7
Results:

Really really nice results on chart.
FMP, FCP under 2 sec
TTCI, TTCI, PSI under 5 sec
How it looks in browser
FOIC

FOUC

Not nice behavior for browsers with FOIC strategy…

But, we have fontfaceobserver which detects fonts loading. Let's try it.
So, results are:
Timeline trace

FMP, FCP under 2 sec, still the same
TTCI, TTCI, PSI under 5,2 sec instead of 4,8 sec
Hense, 400ms looks like a good trade off for consistent behavior through all browsers.
Commit - 6b13c3
And, last gifs
FOIC

FOUC

Aaaaand to compare the before and after...

Thanks :)
P.S. In case supporting only evergreen browsers font-display can manage FOIC.

Nice work and detailed notes! The only immediate issue (that comes to mind at least) is the code demos, which are actually handled outside of the Markdown renderer (even though they're in MD files). I think the only solution might be to manually wrap every example in a DOMContentLoaded listener, which I don't have a problem with, if you're willing to make those updates.

@chrisvfritz thanks for the reply. I've wrapped all examples :)

@denar90 Hmm, I'm actually testing this out in the browser now and immediately, I noticed that the body font doesn't seem to be loading anymore. 😕  The short flash of the fallback font is also a little off-putting to me - I'm thinking it should at least be sans-serif, like the primary font.
I'm a bit torn, because you've obviously put a lot of work into this and I do appreciate it, but seeing a problem right away has made me more cautious. Since the performance of the site is already pretty good, I'm not sure how much time I'd like to spend troubleshooting this - and perhaps other, more subtle bugs in the future.
If there's a @vuejs/docs team member that wants to take responsibility for this PR and work with @denar90 to ensure it doesn't break anything, I'm happy to keep this open and eventually merge. In my personal priorities for my Vue work, I worry that it's just not high enough. 😞

@chrisvfritz I really appreciate that you spend time looking at it.
It's true that there really present some issue, I messed up with some code because of rebasing and stuff and wanna apologize for wasting your time.
Talking about this PR, in general, I can say that it became too big because of wrapping all examples.
Wrapping each example in DOMContentLoaded event listener might cause some bugs if not now but in future creating new vue example.
Another thing is about using js for detecting loaded fonts. If you are happy using font-display and have behavior with FOIC for browsers are not supporting it https://caniuse.com/#search=font-display then js logic can be reduced.
So, I'm closing it but if someone from @vuejs team is interested in this improvements I'll be more than happy finish it :)

@denar90 Thanks for your understanding. 🙂  Despite this not quite fitting in with our current priorities, I do want to thank you for opening this PR. It's definitely one of the most organized and well-communicated I've ever seen and I definitely hope we see more from you in the future! 👍


feat: export datasets as ZIP files

SUMMARY

This PR adds a feature flag VERSIONED_EXPORT that allows user to download datasets as ZIP files, according to https://github.com/apache/incubator-superset/issues/11167. When the feature flag is enabled, hitting the api/v1/datasets/export/ endpoint returns a ZIP file with the selected dataset(s) and associated database(s) represented as YAML.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N/A
TEST PLAN

Added unit tests to cover the new API and the new command.
ADDITIONAL INFORMATION

[X] Has associated issue: https://github.com/apache/incubator-superset/issues/11167
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[X] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11332 into master will decrease coverage by 10.51%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #11332       +/-   ##
===========================================
- Coverage   65.67%   55.16%   -10.52%
===========================================
  Files         835      405      -430
  Lines       39656    13434    -26222
  Branches     3604     3401      -203
===========================================
- Hits        26045     7411    -18634
+ Misses      13502     5833     -7669
- Partials      109      190       +81

Flag
Coverage Δ

#cypress
55.16% <ø> (-0.65%)
:arrow_down:

#javascript
?

#python
?

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

Impacted Files
Coverage Δ

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/dashboard/util/getLayoutComponentFromChartId.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

...d/src/views/CRUD/csstemplates/CssTemplatesList.tsx
3.57% <0.00%> (-93.73%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...uperset-frontend/src/utils/getClientErrorObject.ts
0.00% <0.00%> (-89.19%)
:arrow_down:

.../src/dashboard/components/FilterIndicatorGroup.jsx
11.76% <0.00%> (-88.24%)
:arrow_down:

...c/explore/components/controls/withVerification.jsx
9.09% <0.00%> (-87.88%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

... and 651 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 917d02e...622f2eb. Read the comment docs.


Slow redirection

When I access to the dashboard logged in (e.g. https://dashboard.stryker-mutator.io/repos/theofidry) and click on the stryker icon top left, I get redirected to the following page:

And then it takes a solid 1-2 seconds before redirecting me to https://dashboard.stryker-mutator.io/repos/theofidry because I'm logged in

This should be fixed.


MetaCoin example discrepancy?

So I've deployed the example with truffle init on two machines. Both are connecting to the same testrpc.
Now, when I open the index.html in development I would expect, since both examples are using account[0], that the balance I would see would be the same..
But somehow each instance shows its own META amount, not the same?
To recreate this, don't fill in any address, and fill in any amount, this will subtract an amount from the account ID, and update the META amount you have..
Either this is a bug, or I'm fundamentally misunderstanding how ethereum works?

Secondly, when I try changing the app.js of one instance to use account[1]. It will show 0 meta, correct.
But I would expect when I transfer x amount of META from the other client (account[0]) to this second address, it should receive it? But this does not happen, or at least does not show up in the other account. It's still 0
I do see the testrpc receiving a et_sendTransaction though..

Hi @TrySpace. I apologize for the late response. These types of issues are best asked on Truffle's Gitter, where it's very likely you'll get a quick response from the community: http://gitter.im/consensys/truffle If your problem is still unsolved, please discuss on Gitter or open a new ticket. Thanks!


feat: add websocket support for popcorn bidding on the auction page

The type of this PR is: Feature
This PR solves BID-377
Description
Uses the new websocket context to hook up the timer label + progress bar on the lot grid on the auction page. It is very similar to the lot page components augmented. Seems to work w/ a mocktion testing locally!

Alright, merging this in! Thanks for the feedback, I think the websocket stuff is pretty nice now.

Yeah all of this functionality is actually pretty sweet / simple 💯


Interactive Start

It would be cool if we had an interactive ionic start that prompted the user for some info about their app to make it easier to start projects and get them pre-configured:
Example output:
$ ionic start
Hello! Let's get you started with a new Ionic app.

Hint: try the beta of Ionic 2 by entering "2" for Ionic version.

name: (My App)
app id: (com.example.myapp)
ionic version: (1)
template: (tabs)

We should give them the options to choose from for version and template, similar to commitizen's prompt:


Function call didn't work. Because of Deserialization issue

I'm using Function. But deserializer choose wrong class ChatGPTChatCompletionMessage instead of ChatGPTAssistantCompletionMessage.
Function call return Assistant Completion response. But deserializer always return ChatCompletion class and missing some properties. This is actual returning json
{
  "id": "chatcmpl-8icO5Kj0IEqgDr6dU9J0E9Gecv4nQ",
  "object": "chat.completion",
  "created":<PHONE_NUMBER>,
  "model": "gpt-3.5-turbo-0613",
  "choices": [
    {
      "index": 0,
      "message": {
        "role": "assistant",
        "content": null,
        "tool_calls": [
          {
            "id": "call_Ww3ocoHqydw5xhnS5H1CZImu",
            "type": "function",
            "function": {
              "name": "getExchangeRate",
              "arguments": "{\n  \"currency\": \"USD\"\n}"
            }
          }
        ]
      },
      "logprobs": null,
      "finish_reason": "tool_calls"
    }
  ],
  "usage": {
    "prompt_tokens": 99,
    "completion_tokens": 16,
    "total_tokens": 115
  },
  "system_fingerprint": null
}

But after deserialized object looks like this
{
    "Id": "chatcmpl-8iKN8vlb6QEYiGukfWLAlJNG0jS2V",
    "Object": "chat.completion",
    "Created": "2024-01-18T10:57:14Z",
    "Model": "gpt-3.5-turbo-0613",
    "Choices": [
        {
            "Index": 0,
            "Message": {
                "Role": "assistant",
                "Content": null,
                "Name": null
            },
            "FinishReason": "tool_calls"
        }
    ],
    "SystemFingerprint": null,
    "Usage": {
        "PromptTokens": 97,
        "CompletionTokens": 16,
        "TotalTokens": 113
    }
}

How to fix this to deserialized by ChatGPTAssistantCompletionMessage class.

Please upgrade to version 1.8.0. This was an oversight in the definition of the response message.


/asset endpoint lists seek keys

this asset endpoint: http://refgenomes.databio.org/v2/assets
will yield a response not for every asset but for every seek key. This causes some confusion... shoudl the asset endpoint only list assets? why would we want seek keys listed here? should it be a separate endpoint?
raised in discussion with @natefoo

this is the endpoint that is used by refgenie listr and it lists the seek keys just as refgenie list does. Maybe we could add a separate endpoint that would list just the genome/asset:tag?


properly group SPI and test packages

PR can be tested by running mvn site

Coverage increased (+0.008%) to 81.203% when pulling 90666ce258fe8f6c7f854716e3931033a770b468 on pongad:doc-pkg-group into 60a796a404390f04dc13f8d5dc06a0d83b0dfd95 on GoogleCloudPlatform:master.


Add to onboarding reproduction logs

Setup:
MacOS Sonoma 14.4.1
Chip: Intel Core i5
Memory: 8GB
Python 3.12.3
Java Version: 21.0.3
Maven: 3.9.6
After cloning, JDK 11 was the default and caused issues with building anserini. Everything worked fine after switching to JDK 21, and reproduction was successful.

Please resolve conflicts and sort entries chronologically.

Should be good now.


fix(curriculum): correct and improve the instruction

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #47055

Sorry for my late response. Updated!


ZAP is no longer an OWASP project

@psiinon, thanks for the notice! I will also change the reference of ZAP in The OSS Fortress. (I had planned to add a new dedicated section for an introduction to web security scanning with ZAP.)


Add Marshal and Unmarshal YAML

This will add support for marshaling and unmarshaling YAML into HexColor.

Please remove the tests to prevent adding a YAML dependency to the project. Otherwise looks good, thank you!

Now the YAML tests are removed.

Thanks!


Depth sensor flickering in imseCAVE

When the Azure Kinect is recording in the imseCAVE, the depth sensor keeps flickering, whenever in a steady environment.
Here is the example https://youtu.be/h5wXIvYpunk
I suppose I can take a depth sensor snapshot for background evironment and checks if people (or any other things) enters the imseCave. If the result keeps flickering, there will be wrong result.
Any possible fix?
Thanks in advance.

If the flickering you are talking about is occurring at the 9 second mark in the lower left area of the depth image, that is probably caused by a speck of dust floating by the sensor. The ToF sensor is really good at picking up dust in the air.

The dust problem is reasonable to me because the infrared camera also recorded the dust.
I was talking about the situation like 0:13/0:17/0:33 in the depth image. You can see there seems nothing changed in the color camera (I dont think the projection effects the infrared camera), but part of the depth camera image suddenly goes black (no signal) a second and then return to normal for unknown reason.
And the flickering problem actually occurs throughout the video. The above 3 time marks are the most obvious we can see.
I have tried to use the Kinect in other environment and I think it is normal to have a few pixels unstable. I expect the normal result will be like 0:25~0:27 for the whole video. But it goes unpredictable when I use it in imseCAVE.

This looks a lot like interference from another IR light source. And it looks like it is happening in a repeating pattern every 2 or 3 seconds. Do you have another device in the room that is emitting IR light around 850nm?


Enable Open Here context menu on Windows to work with root of a drive

PR Summary
The fundamental problem is that Microsoft cpp command line parsing rules indicate that a backslash followed by a quote is a quote, otherwise, it is literal.  So this:
pwsh -noexit -workingdirectory "c:\"

Is received as c:" at argument parsing time.  Of course, this is a bit more complicated as Open Here also uses -Command.  So:
pwsh -noexit -workingdirectory "c:\" -command 1+1

Becomes c:" -command 1+1 as the argument value to -workingdirectory.  And -command must be last in the command line.
Fix is to add a hidden switch for the purpose of the Open Here context menu called -RemoveWorkingDirectoryTrailingCharacter.  This allows Open Here's command line to pad the path:
pwsh -noexit -removeworkingdirectorytrailingcharacter -workingdirectory "C:\!" -command 1+1

Now all the args can get parsed appropriately as the working directory path never ends with a trailing backslash.
Fix https://github.com/PowerShell/PowerShell/issues/8278
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Change is not breaking
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

User-facing changes

[x] Not Applicable
OR
[ ] User-facing Documentation needed

[ ] Issue filed - Issue link:

Testing - New and feature

[x] Not Applicable or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

[ ] Add [feature] if the change is significant or affects feature tests

@bergmeister good point, I'll update


[FEATURE] Adding a cross icon in search bar to remove the input we entered

Description
THis feature is self explainatory,does need a description,its just that it becomes a little bad ux to remove the search input using backspace,
I can add this feature if it is worth adding according to u
Screenshots
No response
Do you want to work on this issue?
None
Additional information
No response

Thank you for the suggestion.

THis feature is self explainatory,does need a description

I will rewrite the title and description as this does not give any context

@nidhish-srivastava you closed the issue as completed? but you also closed your pull request, so it is not completed I am confused


ul is an existing command in /usr/bin

ul is a pre-existing command-line utility you can find at /usr/bin/ul. It's probably not used a lot, but it's worth being aware of.

I know. install.sh is too and asks you if you want to overwrite it. I have never heard of it before, so I hope that it's not too popular. I'm more worried about ll :confused: Thanks for the issue anyway :+1:


[modify] ss model user validation

Coverage decreased (-15.09%) to 68.884% when pulling 504ead3a72995c2397c760d4e2502e25ade47c2a on ShinjiTanimoto:user into e5fda85ccd04a0e7217d36ed2b052aa8743a1e01 on shirasagi:master.


[FIX] Android build error

Pull Request Changes:

Fixed String syntax & conditional statement causing Android build to fail

👍 Thank you for the fix, hope it gets merged soon.


Update Cake.Transifex reference from 1.0.0 to 1.0.1

This pull request was manually created because Cake.AddinDiscoverer triggered AbuseException before it was able to submit this PR.
Resolves #826

This has been rebased and merged here: https://github.com/cake-contrib/Cake.Recipe/commit/65c717adeb075c1987badfb3098a6550ca5f59e3


move GLib and GObject inside Vips

It would be neater to have the GLib and GObject modules inside Vips, see https://github.com/jcupitt/ruby-vips/issues/119#issuecomment-325286577
Though I wonder if this would be useful in practice. You can't have two glib or gobject libraries loaded into your program at once, since they both have plenty of global state, so two GLib modules would probably be a disaster.

You're right, it should be possible. I'll have another go.

Aside: @mvz, I copied the ruby-vips inheritance stuff from your example (you probably noticed), so thank you very much!
https://github.com/jcupitt/ruby-vips/blob/master/example/inheritance_with_refcount.rb


Fletching fixes

Just need the last commit checked

Everything but the readme link looks good so far. I haven't merged a fork in before, should I close my PR or will that mess things up?

Not sure. I have my own local copy of your forked repo pointing to this as its origin instead. I think the two branches/PRs are unique.


Delete never used cli.rb

cli.rb is loaded by pry at startup, looking for cli options to add to pry command options.
This file is slowing pry startup by 50%, but it's never used.
It's ok to not exist, since pry checks it:
https://github.com/pry/pry/blob/master/lib/pry/plugins.rb#L38
The same was proposed to pry-byebug:
https://github.com/deivid-rodriguez/pry-byebug/pull/76

I don't remember why it was added, but I guess there's a good reason for it.  How did you figure out that it slows the startup time by 50%?

I think it was to ensure that interception loaded ahead of time. In todays modern times we can probably rely on bundler rather than pry to do this for us.

@kyrylo I just copy/paste from byebug. Rescue add about 0.1s to load time on my machine.
But if every plugins adds 0.1s, you got the idea.

So without requiring cli.rb it adds only 0.05s, doesn't it?

@kyrylo On my machine, pry takes 0.8s to load without plugins. With only rescue, it takes 0.9s.
So 0.1s.
With others plugins, its taking 4s.

Just to make it clear, cli.rb is required by pry even if you'll not use it.
For example, if you execute pry -h it will require cli.rb from all pry extensions installed on the system.
That's why the simple existence of this file, slow down pry shell commands.
I have 12 pry-* gems on my system, but only byebug, rescue and theme has cli.rb and none are used.

@neves The fact that it does a require "pry-rescue" is significant. It's being used (or arguably "abused") to ensure that pry-rescue loads early. It might be that we don't need to do this anymore, as this heralds back to the pre-bundler days (the whole pry plugins system doesn't work well with bundler) but someone would have to test that assumption.

@ConradIrwin, actually pry doesn't require if not exists:
https://github.com/pry/pry/blob/master/lib/pry/plugins.rb#L38
Since day one:
https://github.com/pry/pry/commit/9142aa173f63aadd3f0cec218479525cee9540e7#diff-7a2e80a6a01c9b6a8fe7bfd120264932R38


Status screen is disarranged

Where the bitcoin version should be there are line brakes and confusing text.
_cache.sh is producing the following output:
/home/admin/_cache.sh meta btc_mainnet_version
value="v23.0.0
Core
Bitcoin
the
from
license,"
initiated=1
lasttouch="1659615339"
outdatesecs="-1"
stillvalid="1"

This is solved. Update to v1.8.0 or on v1.7.2 run patch.


Add InfiniBand port error counters

Add InfiniBand port error counters in exporter and dashboard, including:

port_xmit_discards
port_rcv_errors
port_xmit_constraint_errors
port_rcv_constraint_errors

Can we schedule some time to discuss the additional metrics we would like to add. I also have some changes I would like to to the Nvidia exporter and net Exporter

Can we schedule some time to discuss the additional metrics we would like to add. I also have some changes I would like to to the Nvidia exporter and net Exporter

synced offline, we may add low frequent status counters separately with longer interval.


Test against CodeNarc versions supported by the toolchain JDK

Previously, this tested against the versions suported by the build JDK, which might have worked due to happy coincidence, but could break if the test fixture selected a different JDK to use for the toolchain.

@bot-gradle test this

I've triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.


periphery: bus api update

Keep up to date with api changes in rocket-chip.
Future work will add per-peripheral clock crossing options.

Please no. That’s so annoying.
On Fri, Feb 23, 2018 at 15:04 Wesley W. Terpstra<EMAIL_ADDRESS>wrote:

@terpstra approved this pull request.
In src/main/scala/devices/gpio/GPIOPeriphery.scala
https://github.com/sifive/sifive-blocks/pull/50#discussion_r170389803:

val gpioParams = p(PeripheryGPIOKey)

val gpios = gpioParams map { params =>
val gpio = LazyModule(new TLGPIO(pbus.beatBytes, params))
gpio.node := pbus.toVariableWidthSlaves

val gpios = gpioParams.zipWithIndex.map { case(params, i) =>
val name = Some(s"gpio_$i")
val gpio = LazyModule(new TLGPIO(pbus.beatBytes, params)).suggestName(name)
pbus.toVariableWidthSlave(name) { gpio.node }

Without suggestName they will be called 'gpios, gpios_1, gpios_2' ...
that's not ok?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/sifive/sifive-blocks/pull/50#pullrequestreview-99078781,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARCAJKfEPIEXesYojz9NO2pnJAR5xA-xks5tX0P6gaJpZM4SRmGi
.

--
Megan A. Wachs
Engineer | SiFive, Inc
1875 South Grant Street
Suite 600
San Mateo, CA 94402
<EMAIL_ADDRESS>


Investigate/update minimum numpy/scipy versions

A new user recently had a problem with running tutorial code (both of the first two tutorials numpy/scipy based simulations), getting the error
TypeError: ufunc 'nextafter' not supported for the input types, and the inputs could not be safely coerced to any supported types according to the casting rule ''safe''

Their numpy version was 1.19.2 (which is well above the minimum version listed for dynamics), but upgrading resolved the issue. We should find the true minimum and update the listing to avoid this problem.

All right. I'll work on this issue

Closing as this is open ended with no concrete plans.


[Languages] Creating A Hand Gesture Controlled Mouse Using OpenCV, Autopy And Python

Topic Suggestion
Creating  A Hand Gesture Controlled Mouse Using OpenCV, Autopy, And Python
Writing sample(s):
Sample
Proposal Submission
Proposed title of article
Creating a hand gesture controlled virtual mouse using OpenCV, autopy, and Python
Use keyword research to improve your article's SEO.
Proposed article introduction
We go through a lot of trouble washing and drying our hands before we operate our computers when our hands are dirty. This happens when we are in the middle of a task and we get an urge to use the computer. This may be to skip a certain song to the next one or to open that urgent email you just received.
A hand gesture-controlled mouse helps us solve this problem. This is because we can control our computer without having to physically touch it. In this tutorial, we will learn how to use OpenCV, autopy, NumPy, MediaPipe, and Python to create a hand-gesture-controlled mouse.
Key takeaways
After going through this tutorial the reader will:

Understand how the Python libraries mentioned above are implemented in the program.
Understand how computer vision is implemented in the program
Understand how to create a program that uses a virtual mouse to control a computer.

Article quality
The article will be more detailed and in-depth. It will also have a code that is explained step by step, not a copy-pasted one.
References
N/A

Good afternoon and thank you for submitting your topic to the EngEd program.
After some careful consideration, it struck us that this topic may be a bit over-saturated throughout other blog sites and official documentations.
We typically refrain from publishing content that is covered widely on the net or other blogs. We're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
But in order to approve the topic, it has to serve value to the larger developer community at large.
An option and a great way to write this as an in-depth article and make it more add value to the greater developer community at large would be to walk the reader through the USE of methods and functions by building a unique, different, useful project.

We would love to learn your thought process behind the solution you arrived at using these concepts and topics.

That way a developer could see them in action.
As mentioned above - we believe this topic is widely covered on other blog sites.
The best way for students to build a great portfolio is by building what does not exist and what can provide the most value.


Automated builds need to be reinstated (manual intervention needed)

The automated builds on travis-ci.org have ceased due to it being retired.
It looks like the builds can be migrated to travis-ci.com.

Currently, the CI is only used for running the jshint tests. In the long term, we might also use it for deployment. However, I would suggest to rely on GitHub Actions instead of Travis.

I second that.


In the Translation page，have a bug?

when choose Chinese of target language ,an error throw at line 43 in TranslationRepository.cs

Use discussions https://github.com/orgs/serenity-is/discussions
Provide the version info of your serene application.
Share exception details related the error.


Extension 'ESlint' cannot format file 'index.ts' caused by probing failure

I encountered the problem Extension 'ESlint' cannot format file 'index.ts' while trying to format my TypeScript code in a Vue project. Here is the snapshot:

And here is my settings.json in the workspace:
"eslint.format.enable": true,
"editor.codeActionsOnSave": {
  "source.fixAll.eslint": true
},
"[typescript]": {
  "editor.defaultFormatter": "dbaeumer.vscode-eslint"
}

Then, I try to turn on the trace log of plugin with "eslint.trace.server": "verbose". The plugin gives the following output:
[Info  - 下午10:17:00] ESLint library loaded from: D:\code\web\***\node_modules\eslint\lib\api.js
[Trace - 下午10:17:01] Received request 'eslint/probeFailed - (4)'.
Params: {
    "textDocument": {
        "uri": "file:///d%3A/code/web/***/src/backend/index.ts"
    }
}

[Trace - 下午10:17:01] Sending notification 'textDocument/didClose'.
Params: {
    "textDocument": {
        "uri": "file:///d%3A/code/web/***/src/backend/index.ts"
    }
}

[Trace - 下午10:17:01] Sending response 'eslint/probeFailed - (4)'. Processing request took 0ms
No result returned.

It seems that probing failure causes the problem. So, I use the settings "eslint.validate": ["typescript"] to enforce validating of TypeScript. Finally, the problem got solved, and soon it is available to format my TypeScript code with ESLint.
The question has been reported for multiple times (#1086, #850, #951), is it a bug or feature?
The following information may be useful:

The project is generated by @vue/cli v4
Problem can be reproduced on any kinds of TypeScript file, even a Hello World
The .ts file can be fixed by eslint --fix index.ts
OS System: Windows 10 20H2
Node: v14 globally installed, v12 used by VSCode
VSCode version: v1.52.1
Extension version: v2.1.14

Thank you for help!

@ipid actually the probing is "difficult" since it require me to inspect the parser and plugins used and if something changes there I might fail with the probing.
Can you provide me with a Github repository I can clone that demos your behavior. Then I can have a look and see why the probing fails.

@dbaeumer Here it is: ipid/vscode-eslint-probing-issue
Note: The problem seems to be reproducible only on Windows.
Again thank you for your help!

I had this issue just now, same error, not vue or anything, just a typescript project with no TSX files. I
I nuked my node_modules, ran npm install, then shift+cmd+p: Reload window / restart vscode. I didn't update the eslint.validate setting.
Not the solution we need, but perhaps the one that we deserve.

@mrvisser that is strange. May be a plugin was missing hence the probing didn't work correctly.

I'm getting this issue also
VSCode:
Version: 1.54.2
Commit: fd6f3bce6709b121a895d042d343d71f317d74e7
Date: 2021-03-11T00:56:09.783Z
Electron: 11.3.0
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0

VSCode ESLint
v2.1.17

Local settings.json
{
    "eslint.format.enable": true,
    "editor.codeActionsOnSave": {
        "source.fixAll": true
    },
    "eslint.trace.server": "verbose",
    "editor.defaultFormatter": "dbaeumer.vscode-eslint",
    "eslint.alwaysShowStatus": true,
    "eslint.debug": true,
    "eslint.packageManager": "yarn",
    "eslint.probe": [
        "javascript",
        "javascriptreact",
        "typescript",
        "typescriptreact",
        "html",
        "vue",
        "markdown",
        "json"
    ]
}

User settings.json
There is nothing in my user settings.json that relates to ESLint
Reproduction repo:
https://github.com/jamiehaywood/eslint-debug

@jamiehaywood I can't reproduce this using the repository you provided. Any special steps?

@dbaeumer thanks for taking the time to look into this. I'm not sure to be honest, all I know is that when trying to edit hello.ts inside my eslint-debug repo I'm getting this:

So there were two problems which I've since figured out and fixed:

I hadn't installed TypeScript... which may have been why it couldn't format the hello.ts file (duh...🤦🏻♂)
ESLint can't lint json files out the box (it's ES Lint double duh...🤦🏻♂🤦🏻♂)

Having spent a good couple of hours trying to figure out how to get VSCode, Prettier, & ESLint playing nicely together, I'm going to raise a PR with example setup & config instructions for my issue #1204.
Just want to say a big thanks to @dbaeumer - the plugin is awesome, and I think I speak for everyone who uses the tool - we would be lost without it.

@jamiehaywood great that you figured it out. I agree ESLint can be confusing especially you need some configuration to validate anything other than js.
Looking forward to your PR. I will close the issue
@maximumstock can you please provide me with a GitHub repository I can clone that demos what you are experiencing.


Request more elements when decoding a large result.

Signed-off-by: Giedrius Visokinskas<EMAIL_ADDRESS>Issue description
https://github.com/r2dbc/r2dbc-mssql/issues/216

Thanks a lot for your work and your help. I decided to use concatMapIterable(…) to fix the count issue. As outlined in #216, decoding could also lead to multiple items being produced and that can lead to overflows and since we addressed underflows, we had to fix the other issue as well.


Support Swagger Annotations for nested create

Greetings,
first of all: Great work. I really enjoy this generator in conjunction with nest g res and another crud-service generator to bootstrap basic crud functionality quickly.
I noticed that this generator does not include typings for nested writes in Swagger. I guess this makes sense because of the "Create/Connect" DTOs needed. Would it be possible however, to include these as API Properties during generation? This is so we have proper API documentation, should an entity require or allow for nested creation.
An example could be:

User Model
Media Model

On user creation, you can pass a media array (create or connect eg.).
I hope this was somewhat understandable. :) I would help as this helps me aswell. I have however never contributed anything before and have not worked with generators. How would get started about it? :)

I don't fully understand your issue and how the expected outcome should look like. Can you please provide a small example including: how does the input model look like (schema.prisma), how should the generated dto files look like?

Sorry for the late response. I will include an example later today. Great work on this library by the way!


Failing to deserialize from serde_json::Value

The NodeId can't deserialize from serde_json::Value object.
Here is a simple test (unwrap crashes because deserialization returns None)
#[cfg(feature = "serde")]
#[test]
fn test_serde_value() {
    let node = NodeId::random();
    let value = serde_json::to_value(&node).unwrap();
    assert_eq!(node, serde_json::from_value::<NodeId>(value).unwrap());
}

One way to fix it is to change deserialize function to:
let raw: String = serde::Deserialize::deserialize(deserializer)?;
let src = raw.strip_prefix("0x").unwrap_or(&raw);
hex::FromHex::from_hex(src).map_err(serde::de::Error::custom)

but this is not optimal in other cases.
If you are fine with this fix, I can create a PR.

Might be worth noting that maybe the best way is to try the current implementation (deserializing to byte slice) and only try my fix if that one fails.

thanks for the issue @morph-dev. I don't fully understand what's going on, could you explain from your understanding why this serialization is failing? A bit surprising.
Regardless, we definitely care first about correctness than performance so I would be on board with this change. But being honest, I first need to understand the root of the issue for a proper review

My understanding is that deserializing into byte slice works fine for serde_json::from_slice and serde_json::from_str because they accept borrowed parameter.
However, serde_json::from_value moves the Value object, so we can't deserialize it into byte slice, but deserializing it into String works fine.
I believe that previous implementation, that used serde-hex crate had the same issue. There is forked version, the stremio-serde-hex, which implemented the special serde::de::Visitor (link) to deal with it.
I will create a PR with suggested changes.

Also, we (Trin) would like to remove our own implementation of NodeId and use this one instead (which we already do in many places).
So we would appreciate if new release could be published soon after this is fixed.

( This is something rushed; please double-check and verify. )
The problem is that the types don't match. The current NodeId deserializing
code requires a str, but what Value can provide is a String.
I am afraid we cannot find a way to force serde_json::from_value to give us a
value with a lifetime such as str, because the trait involves is DeserializeOwned
, not Deserialize.
The good news is that we don't need to return the "string" itself, but to find a way
to handle the two possible input types (borrowed and owned).
I figure that Cow<str> could be used here, and the code below seems to work.
This way, we might be able to preserve the performance (avoiding string copying)
and at the same time support from_value.
"import" Cow:
/// Serialize with the 0x prefix.
#[cfg(feature = "serde")]
mod serde_hex_prfx {
    use std::borrow::Cow;
    ...

Main code:
    {
        let raw: Cow<str> = serde::Deserialize::deserialize(deserializer)?;
        let src = raw.strip_prefix("0x").unwrap_or(&raw);
        hex::FromHex::from_hex(src).map_err(serde::de::Error::custom)
    }

This was a good lead. It needs some work because since there is no explicit lifetime serde produces an owned string everytime. I've fixed the pr and will merge soon. A release is unlikely for the next week though

Thanks for fixing it.
Can you maybe ping this issue once new released is pushed. Thanks


No action at all since rpi upgrade to 'bullseye'

This plugin was working just fine up to an upgrade of the raspberry pi to 'bullseye' version:
Linux raspberrypi 5.15.56-v7l+ #1575 SMP Fri Jul 22 20:29:46 BST 2022 armv7l GNU/Linux
Now the plugin loads correctly, as before but nothing happens via homekit.
Tried the 'homebridge-magichome-dynamic-platform' plugin and it can control the lights but has less options than this current one (scenes: presetSwitches).
-- Patrice.

Dear all,
I think the issue has been found, in fact the file homebridge-magichome-platform/src/flux_led.py is using the latest python defined on the system (so python3 in my case) but the code is not compatible:
$ python3 flux_led.py    File "/home/magic/npm/node_modules/homebridge-magichome-platform/src/flux_led.py", line 124     print ''.join('{:02x} '.format(x) for x in bytearray(bytes))           ^ SyntaxError: invalid syntax $
So changed the header of the file to use: '#!/usr/bin/python2' and it is now working fine back !!!!
-- Patrice

How do you do that from homebridge? I can't seem to find the files

How do you do that from homebridge? I can't seem to find the files

Well, without python2 on your system, you may also want to 're-write' all the print functions if this flux_led.py file:
print "response too short!"
becomes
print("response too short!")

flux_led.py.txt

I managed  to  find  the  file,  but Im unsure  how  too rename the header, as Im using  SSH connection to access  my pi and doubt  I  could  open the file  from here?

I managed to find the file, but Im unsure how too rename the header, as Im using SSH connection to access my pi and doubt I could open the file from here?

Lokks good, just replace the file by the one sent in a post earlier (remove the .txt of course....)

Im not  sure if Im doing it correct as Im new to this, but
-I downloaded  your file,
-the  path seems to be correct,
-the file don't have the .txt anymore
-the  ssh  connection is on
and I  get  this  error when Im  trying  to replace

Im not sure if Im doing it correct as Im new to this, but -I downloaded your file, -the path seems to be correct, -the file don't have the .txt anymore -the ssh connection is on
and I get this error when Im trying to replace

That's just because you shall use an scp tool from your windows to transfer the file, something like WinSCP

I tried with PuTTY  gave  me the same  error :(

Try to launch:
curl -sL https://github.com/sahilchaddha/homebridge-magichome-platform/files/12706444/flux_led.py.txt > flux_led.py
from the home dir of user pi through your ssh session, the file should be written in your home dir.

I just did  that,  and it  give  the  same  error  as  in  your  first post,
I  downloaded  the file from  the link inside  the command to  check, and in the header it says
#!/usr/bin/env python  instead  of  #!/usr/bin/python2
Could  that be  the  problem?

Thank  you for the  help,  I figured  out the  rest,
So  I  had to  put  this
pscp c:\Users\csaba\Desktop\flux_led.py pi@<IP_ADDRESS>:/var/lib/homebridge/node_modules/homebridge-magichome-platform/src/flux_led.py
into  the  windows  shell while  I  have   the  connection  open  on  PuTTY
but  it  works!

please consider a pull request


feat!: This is a breaking change where we are removing the ExecRecipe.service_name, GetHttpRequestRecipe.service_name, and PostHttpRequestRecipe.service_name fields, we suggest users pass this value as an argument in the exec, request and wait instructions where this type is currently used. We are also deprecating the previous exec, request, and wait instructions signature that haven't the service_name field, users must add this field on these instructions call. Another change is that now the service_name field on the exec, request, and wait instructions is mandatory. (#93)

Description:
feat: Added ReadyConditions type which is used in ServiceConfig for defining how to check services readiness
Is this change user facing?
YES
References (if applicable):
Fixes https://github.com/kurtosis-tech/kurtosis-private/issues/740
Changelog picked up from commits here:
docs: Added ReadyConditions type documentation (#249)
docs: Remove broken docs links (#258)
feat: Added service readiness check in add_service and add_services instructions, through the ServiceConfig.ReadyConditions field (#148)
feat: Added ReadyConditions type which is used in ServiceConfig for defining how to check services readiness  (#151)
build(deps): bump github.com/docker/distribution from 2.7.1+incompatible to 2.8.0+incompatible in /core/server (#114)
build(deps): bump github.com/docker/distribution in /core/server
build(deps): bump github.com/opencontainers/image-spec from 1.0.1 to 1.0.2 in /core/server (#113)
build(deps): bump github.com/opencontainers/image-spec in /core/server
docs: add onboarding calendly links to docs (#14)
docs: create guide on installing historical versions (#131)
fix: fix changelog for versioned docs going forward (#142)
docs: pruned versions 0.68.1, 0.68.3 & 0.68.4 as the binaries were
docs: added changelog to 0.68.2, 0.68.5 & 0.68.6
docs: ignore docs built by docusaurus in docs checker (#141)
fix: sort services by name (#139)
fix: gramatical fix in analytics tracking logging (#138)
docs: Minify README language (#107)
chore(main): release 0.68.6 (#132)
docs: add kurtosis feedback cli command (#53)
feat: Added kurtosis feedback CLI command (#28)
docs: Simplify the docs landing page with latest "distribution application development platform" language (#110)
feat: publish versioned brew formula (#130)
build: Stop publishing go module to a separate repository (#124)
chore(main): release 0.68.5 (#128)
fix: Use version.txt for kurtosis_version instead of Git tags (#126)
chore(main): release 0.68.4 (#123)
fix: Pass correct latest tag to GoReleaser CLI build (#122)
chore(main): release 0.68.3 (#121)
feat: Use semver versioning for Golang API package (#119)
docs: remove WARN level log lines in readme (#112)
docs: add questions+help+feedback links (#111)
fix: remove mentions about github discussions (#95)
chore(main): release 0.68.2 (#120)
fix: fix push_cli_artifacts ci job (#118)
chore(main): release 0.68.1 (#115)
ci: fix push docs ci (#116)
feat: docs are versioned (#106)
refactor: remove dependence on private free ip address provider (#108)
refactor: Migrate IP Address and Hostname to runtime value (#100)
build(deps): bump golang.org/x/net from 0.0.0-20210913180222-943fd674d43e to 0.7.0 in /internal_testsuites/golang (#72)
build(deps): bump golang.org/x/net in /internal_testsuites/golang
docs: Fix NGINX port in quickstart (#109)
chore(main): release 0.68.0 (#102)
refactor!: Migrate Kurtosis Print instruction to Starlark framework. This restrict the use of print to a single argument only. (#80) (#87)
docs: fix a typo in exec assert example (#103)
docs: Add LICENSE (#99)
feat: enclave clean has both name and uuid (#101)
refactor: Migrate ConnectionConfig to Starlark Framework (#97)
refactor: Migrate packet delay distributions to Starlark Framework (#96)
refactor: Migrate PortSpec to Starlark Command Framework (#91)
feat: persist partition connection overrides to disk (#98)

Closing it because it was merged here: https://github.com/kurtosis-tech/kurtosis/pull/301


Fix java.lang.NumberFormatException: empty String & optimize polling

Codecov Report

Merging #587 into master will decrease coverage by 0.1%.
The diff coverage is 45.45%.

@@             Coverage Diff             @@
##             master    #587      +/-   ##
===========================================
- Coverage     76.81%   76.7%   -0.11%
- Complexity      608     609       +1
===========================================
  Files            81      81
  Lines          2251    2254       +3
  Branches        319     319
===========================================
  Hits           1729    1729
- Misses          309     310       +1
- Partials        213     215       +2


How to use the Path in Fragment

I created a TileFragment just like the TileActivity ,but I can't add the Path dynamic。4 example，I want to
creat a Path between two points which I click.You have any suggest? THX

The wiki page has links to the appropriate Java docs: https://github.com/moagrius/TileView/wiki/Paths

Here like we are trying to do.

Here like we are trying to do.


fixes incorrect CLI stats output

What kind of change does this PR introduce?
bugfix

Did you add tests for your changes?
yes

If relevant, link to documentation update:
n/a

Summary
fixes #4141
fixes #4118

Does this PR introduce a breaking change?
yes, but it's a bug fix

TODO: chunks should be hidden by default

easy rebase


Modular services inspired by amazonka.

Not sure if you planned for this; but I thought this org structure cleanly separates request encoding, response parsing and service invoking. This allows each service to declaratively describe itself sharing a single invoke function, and lets the user fully control every optional param and extract every fringe response value if they choose.
Made some additional changes to aid in quick prototyping:

switch to package.yaml
moving around common types (Storage level, and Blob level) to standalone modules

Next steps:

[x] get tests passing (when tested manually with a Live azure account, I keep getting Auth errors)
[x] complete the service description plumbing to self resolve response type based on the request type (ATM the user has to type hint that they expect PutObjectResponse for a PutObject request).

Would be happy to join this project as a collaborator to build out the rest of the services (though my immediate needs solely lie in the Blobs API).

Addressed auth error(s) with live Azure account. Issue was a missing slash between account name and resource path. Manually verified GetBlob and PutBlob.
The integration tests appear to fail locally using Azurite since it lacks the ability to properly enforce any access control (or verify auth -- hence the inability for the tests to catch the live auth error above).

At this point, the sheer depth and breadth of changes probably is too large to be merged here. I'm going to keep them in my fork.
I personally no longer have a need for an Azure SDK. So will take the development more slowly. Feel free to re-open this and/or reach out to me if you're interested in collaborating.


Confusing jQuery waypoint instructions

On the jQuery Waypoint's last instructions, the wordings might confuse users as codeacademy's UI does not have any "sections".
For Eg: on "Waypoint: Harness Dynamic HTML" (http://www.freecodecamp.com/challenges/waypoint-harness-dynamic-html) The last instruction reads:
Go to http://www.codecademy.com/courses/web-beginner-en-v6phg/0/1 and complete the third section.
But after visiting the link to Codeacademy, there is no mention of the word section anywhere and the number of steps in the lesson are 14.
Even I struggled to understand if I was to complete till 3/14 or 3rd Section meant completing till "Mastering Manipulations" which occurs only after clicking the lesson number.
Just thought to update the waypoint instructions to be clear like "Complete all lessons".
Thanks,
Kanad Godse

Closing as all codecademy courses will be removed in next iteration of curriculum.
Happy Coding!


I met error about dependency during npm install.

How to reproduce

Leaflet version I'm using: current repository version 1.3.4
Browser (with version) I'm using:
OS/Platform (with version) I'm using: macOS Mojave 10.14.1 beta
npm version : 6.4.1

git clone -> git clone https://github.com/Leaflet/Leaflet.git Leaflet_test
cd Leaflet_test
npm install

npm install command result
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
found 17 vulnerabilities (12 low, 4 high, 1 critical)
run npm audit fix to fix them, or npm audit for details
What I want to do...

I Want to build using rollup after change namespace.

Question
I Just cloned from https://github.com/Leaflet/Leaflet.git
After clone, I Executed npm install. But I met a message about dependency problem.
npm audit fix command couldn't solve this problem.
How can I do for this situation?

I have not solved it yet, but it seems to be due to my development environment. I will close this issue because it is a question about npm.


Prompting info and chord progression

I've seen some of the prompting tips from @Duemellon and from youtubers.
I'm trying to figure out if the model is capable of creating chord progressions based on prompts or even more structured melodied based on prompts (without the melody conditioning).
I've tried prompts like
"Hip hop beat with chord progression vi-V-IV-V"
"Chord progression melody with C Major G Major A Minor F Major" Top-K : 50 (this one was decent)
results are interesting but not quite accurate.
Anyway, if anyone else is experimenting with more low level melody creation and has some cool tips, please share.
It would be sweet if you could prompt Synth(Sine, 220htz), pattern[0,0,0,1] or something like that.
Ideally MIDI tags could be added to the future training.

https://drive.google.com/drive/folders/1GhXKot4LZSuDkKzZyqQO4JrqoW8v1hQE?usp=drive_link
Generic music my coollaboratory script, save music path audiocraft

https://drive.google.com/drive/folders/1GhXKot4LZSuDkKzZyqQO4JrqoW8v1hQE?usp=drive_link Generic music my coollaboratory script, save music path audiocraft

These sound great but I do not understand what you mean by your statement.
What is the prompt you used to get this result?
What will we have to edit to create this locally?

I think he used
_top_p = -5 _top_k = 50 _temperature = 1.0 _cfg_coef = 3.0  _extend_stride = 18  _duration = 10  promt_text = "44khz stereo Chord progression melody with C Major G Major A Minor F Major"
I've never used - in top_p.. what does that do? @BuBaLoM-JMP

You can

I think he used

_top_p = -5
_top_k = 50
_temperature = 1.0
_cfg_coef = 3.0
_extend_stride = 18
_duration = 10
promt_text = "44khz stereo Chord progression melody with C Major G Major A Minor F Major"
Large Model

I cant put - in top_p in the Gradio interface version.. is it supposed to be negative value?? @BuBaLoM-JMP

No, negative top_p is ignored. In fact any top_p is ignored if you have top_k.

So you can use top_p or top_k, not both, and neither negative.
_top_k is a pretty low value. The default is 250. So that is a big change.

else ))))

44khz  orchestral  fantasy music
44khz  instrumental  celtic music
Russian Language>)))
No, negative top_p is ignored.
Странная особенность если прогнать промт 3-4 раза с -5.0=top_p то получаешь идеальный результат на выходе))) почему?

https://github.com/facebookresearch/audiocraft/assets/163408/a9815a9b-b37a-4c85-b9bb-87d2fbb9366b
https://github.com/facebookresearch/audiocraft/assets/163408/6a110e0d-6c72-43da-ba7c-84d00b328cd6
Okay so your "man of mystery" routine got me curious, is there actually something you are hinting at?
So I took a guess at what 'negative top_p" means and made it work. Just grab the least likely tokens instead of most likely, until you go over p. And.... it actually does seem... maybe interesting! I am very happy so far, as a person who truly delights in atypical outputs.


Default to osd_activate if OSD_DEVICE is provided

This patch is to dynamically choose between osd_directory and osd_activate.
The currently implementation forces osd_directory always.

Because ceph-disk will wipe out the device and I want to re-use the data.
I plan to run the container like this:
ExecStart=/usr/bin/sh -c "/usr/bin/docker run \
  --privileged=true \
  --net=host \
  --name=ceph_osd \
  --volume /var/lib/ceph:/var/lib/ceph \
  --volume /dev:/dev \
  --env CLUSTER='ceph' \
  --env OSD_DEVICE=/dev/sda \
  --env OSD_FORCE_ZAP=1 \
  --env KV_TYPE=etcd \
  --env KV_IP=<IP_ADDRESS> \
  ceph/daemon osd"

And my understanding is that:

The very first time the container runs it will execute osd_disk (because if condition will be true)
From now on every time the container is restarted the if condition will be false
So now we have to choose whether we run osd_directory or osd_activate
If the user provides OSD_DEVICE I assume osd_activate is prefered

Let me know if that makes sense. I am just learning how Ceph works.

Ok I didn't quite catch that this was a reboot scenario.
Since you use OSD_FORCE_ZAP=1, yes ceph-disk will wipe the disk. If you don't set this there is a check here https://github.com/ceph/ceph-docker/blob/master/daemon/entrypoint.sh#L243. I created this variable mainly for convenience during my tests (useful when you want to re-use a device and start from scratch).
But then I understand that you need this flag in order to not exit the entrypoint. Unfortunately I don't see how we can detect that the container is issuing a stop/start or a reboot. So I guess your approach is valid, if a device was passed then we can assume activate is desired.
It is correct to assume that the entrypoint doesn't handle restart really well. And I'm happy to see that someone is working with us on it :), so thanks for that.
Would you mind updating the readme to give some instruction on how to issue a reboot. Simply highlight that  OSD_FORCE_ZAP=1 is required.

I have updated the README with a fourth case. What do you think @leseb?

Thanks @h0tbird!


Add an option to disable rendering of items in pipes

I was proposing to a friend to use Pretty Pipes instead of Simple Storage Network because I prefer it. Their response was "I can never use that, it draws the items inside the pipes so that means it will cause lag". So I added a config option to turn off item rendering.

Hi, thanks for this pull request!
Unfortunately, I am not interested in a config option like this. Items rendering in pipes is an essential part of the mod's design and style, and I don't want users to be able to disable it.
Also, due to the fact that block entity renderers use batched rendering, drawing items in pipes should have a rather insignificant effect on rendering performance. Has your friend done any performance tests with and without Pretty Pipes? A profiling test with a mod like Spark could shed light on the exact source of the lag.

Has your friend done any performance tests with and without Pretty Pipes?

They have not, and they aren't technically oriented enough to understand that, it's just one of the bad rumors that some people believe. I just figured a config option would be enough to dispel said rumors when they realize it does nothing.


Access to the path

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] My answer given in the checkbox above is a lie.

What problem did you encounter?
System.UnauthorizedAccessException: Access to the path 'C:\Users\secur\AppData\Local\Bloxstrap\Bloxstrap.exe' is denied. how i can fix that?

[x]  My answer given in the checkbox above is a lie.

😐


Add more complex search

[x] Implement Postgres fuzzy search

[x] Add migration for extension

[x] Rubygems search suggestions
[x] Frontend for search results
[x] Suggest project conditions (maybe use search term words "distance")

Branch feature/search-2.0

Successfully deployed!


Some methods fail if used from storage extension

Any method called from Storage::disk('cloudinary') which passes thru the League\Flysystem\Filesystem and hits the assertPresent method of the aforementioned class will fail to achieve anything (for most setups).
Example methods that will fail:

delete
update
copy
rename

That issue is being fixed by this #35 for some reason it hasn't been merged yet

This has been fixed. So closing this issue.


bump from 1.44.0

browser.json updated
firefox-asan dropped (see: https://github.com/microsoft/playwright/pull/30626)
version bump

as per #3 (not fully fixed yet)


Adjusted possible responses for getData for method to avoid errors.

Hello,
Thank you for this library, awesome work here.
Deeper in that method, we find a json_decode at the vendor/greg0/lazer-database/src/Classes/Helpers/File.php:85. One of possible returns for that function is null, which happens more than ideal. To avoid errors to be logged without necessity I added this small change. From PHP 8 forward we will be able to use union types (array|null).
Please, let me know if you have any concerns or if this is not ideal.
Cheers,
Savio

Let's hold on this. After some extra testing, further in the road it seems it affects the library consistency. I don't have control of labels here so I'm leaving a commeng.

@lotharthesavior You can create the new PHP Unit Tests and check the changes are working correctly.
Here you can find the existing PHP Unit Tests: https://github.com/Lazer-Database/Lazer-Database/tree/master/tests/src/Classes

Thanks, I'll do that when I change something. Logically that problem is possible, but it seems that it only happens when I have filesystem permission problems, so it might not be not a big deal. I'll be using it on a daily basis for the next few months, so I'll have opportunity to confirm if that really needs adjustment. @anant-svc


Uni v2 remaining events

This decodes the remaining events for Uniswap V2:

Burn
Pair Created
Sync

Obselete #20


CLOUDP-101436: Reworked docs for Atlas helm chart.

I've replaced this PR for #58 thinking that the errors were because I was not creating this PR from a fork. It was not the case, but still, please review #58 instead.
Summary
I've reworked the docs for the Atlas operator, and the top-level README and charts/README.

Make it clear which Helm charts can be installed from the mongodb repo. Also stating the Trial state of Atlas operator.
Removed outdated docs
Clearly stated the relation between atlas-operator and atlas-operator-crds charts, and how to install atlas-operator without crds dependency.
Improved narrative in almost all the documentation pages.

@fabritsius @leo-ri @jasonmimick please review #58 instead.


custom delay function

move from here https://github.com/keboola/syrup/pull/119
requires https://github.com/keboola/storage-api-php-client/pull/180

fixed by #251


Creating a manual endpoint to upload images

Is it possible to create a manual endpoint to upload files?
Instead of using the uploader, I want to set a drop zone for a specific use case that users this lib behind the scenes. (all the rest of the site uses the normal dialog uploader).

@streamtw any ideas?


chore(deps): use stable node.js

Please explain the changes you made here.
Does this close any currently open issues?

[ ] Individual Contributor License Agreement (CLA) signed

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

Comparison is base (93e25d2) 82.45% compared to head (6e5d7c4) 82.45%.

:exclamation: Current head 6e5d7c4 differs from pull request most recent head 8060eb1. Consider uploading reports for the commit 8060eb1 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             beta     #190   +/-   ##
=======================================
  Coverage   82.45%   82.45%
=======================================
  Files           2        2
  Lines          57       57
  Branches       15       15
=======================================
  Hits           47       47
  Misses         10       10

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


A simple program illustrating chaotic behaviour

This program also demonstrates the apparent unpredictability of chaos functions and how small initial differences can lead to large differences as the formulae is repeatedly applied.

Thank you for you contribution to the repo and the community

On 23 Sep 2017, at 07:31, Tanaka Chingonzo<EMAIL_ADDRESS>wrote:
This program also demonstrates the apparent unpredictability of chaos functions and how small initial differences can lead to large differences as the formulae is repeatedly applied.
You can view, comment on, or merge this pull request online at:
https://github.com/geekcomputers/Python/pull/238 https://github.com/geekcomputers/Python/pull/238
Commit Summary
A simple program illustrating chaotic behaviour
File Changes
A chaos.py https://github.com/geekcomputers/Python/pull/238/files#diff-0 (11)
Patch Links:
https://github.com/geekcomputers/Python/pull/238.patch https://github.com/geekcomputers/Python/pull/238.patch
https://github.com/geekcomputers/Python/pull/238.diff https://github.com/geekcomputers/Python/pull/238.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/geekcomputers/Python/pull/238, or mute the thread https://github.com/notifications/unsubscribe-auth/ABHa0o71mdDM-nVvgJPvXXvJo0SCaiN3ks5slKW-gaJpZM4Phdj4.


aq pxeswitch: allow to configure aii-installfe cdburl

This helps with sharing an AII server between several sources, in particular between SCDB and Aquilon.
Fixes #113 (https://github.com/quattor/aquilon/issues/113)
Requires https://github.com/quattor/aii/pull/291 (aii-installfe counterpart) to be useful.

I confirm that having an empty value works like a charm! Not sure what I made wrong in my initial tests...

You may have one broker, but multiple URLs that serve profiles from it: mirrors, reverse proxies etc.
As I said, I don't think it matters that much, you can configure the AII server however you want unless you use the override for a shared server.

I understand, I didn't have this configuration in mind... But as you say, if you are using the configuration you mention, probably the price to pay is that sharing the AII server with SCDB is more complicated... A service instance based approach may bring a more general solution but looks to me as much more complex as there is no way AFAIK to define data to be passed to the service by the broker (not in a template), as in this particular case it is the broker which is building the aii-installfe command.
If it is acceptable for you, I'd appreciate to see this PR merged to make SCDB transition easier.

Sure, I don't really have a problem with this PR, just wanted to flag it!

Can this be rebased on the current master please?
The tests used internally in Morgan Stanley changed a bit and requires commits that have been included in the last master. Thank you!

Done

@jouvin so actually, it was not working but because of a change in the version of the panc compiler (an error message that changed). With the new master it should work. Could you please rebase ? I will post that same message in other pull requests sharing the same issue.
Sorry for asking the rebase a second time. Looking at the rest of the tests, this should be fine this time, and I'll try to merge it asap :)

@XaF No problem, done!


Sometimes response body cut in half.

In my app sometimes huge json bodies I am trying to read is received half. Is there any limit or configuration I should make to prevent this?

I don't think so, I wouldn't expect that to happen. Maybe a bug in Isahc somewhere?
If you enable trace logging, when this occurs do any logs stand out as suspicious? Maybe there's something in those logs that can point us in the right direction.
Tip: to enable trace logging, hook up a logger crate like env_logger with the RUST_LOG=isahc=trace environment variable set.

I recently had time to review the errors.I pasted the partial log output above.
For some reason sockets are being closed half which started to happen a lot recently

any ideas why this might be happening @sagebind

Not sure, I've never seen this before. These lines seem curious:
TLSv1.3 (OUT), TLS alert, close notify (256):
transfer closed with 12025 bytes remaining to read

Not sure what the TLS alert means, does that mean that this is somehow TLS related?

huge json bodies

How huge is that in bytes? Maybe I can try to perform something similar in a debugger and try to reproduce.
Also, it would help if I knew what version of Isahc you are using.
Note to self: #77 may have helped here to isolate the logs for a particular failed transfer.

im using version  0.8.2 I am not exactly sure about the body size I will update here if I get a chance to update the running binary on production.

this is how I produce the request
and this is how I send it

Good news, I was finally able to reproduce this! I'm not sure if this is exactly what you're running into, but the log output sure looks similarto what you are seeing. I reproduced it by having .text() fail mid-stream with invalid UTF-8 and force the connection to abort.
I took this UTF-8 decoder stress test file, duplicated it 100,000 times to make a big version of it (so that downloading wouldn't asynchronously complete while decoding), and then ran this code:
use isahc::{
    http,
    prelude::*,
};

fn main() {
    env_logger::init();

    http::Request::get("http://localhost:8000/big-bad-utf8.txt")
        .body(())
        .unwrap()
        .send()
        .unwrap()
        .text()
        .unwrap();
}

Output:
2020-02-26T05:13:58Z TRACE isahc::handler] received 16384 bytes of data
[2020-02-26T05:14:00Z DEBUG isahc::handler] transfer closed with 221452056 bytes remaining to read
[2020-02-26T05:14:00Z DEBUG isahc::handler] Closing connection 0
[2020-02-26T05:14:00Z DEBUG isahc::handler] curl error: [18] Transferred a partial file
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Io(Custom { kind: InvalidData, error: "stream did not contain valid UTF-8" })', src/libcore/result.rs:1188:5

What's happening here is that text() is streaming the response bytes and decoding it as UTF-8 incrementally. If there are malformed UTF-8 bytes in there, then it will return an error early, potentially causing the transfer to abort and result in the types of logs you see above.
In your code, what sort of error result are you getting back? Are you sure you're not getting an error from .text()? Perhaps sometimes the response body contains a malformed character and that is giving you trouble.
I tried lots of other things that didn't help, but I at least I know what this isn't. The file size isn't a problem here; Isahc is able to transfer very large ISO files perfectly (and quickly) and the digests of the output are correct, so the problem isn't the size of the payload.

Interestingly, as I test this, I am noticing that enabling or disabling trace logs actually effects the results a bit as to how early things abort, which is rather strange. I'm not sure it is related to your issue, but I will look into that to be sure.

Finally found a fix for this in #246! This was a bug caused by uncaught errors during response body (such as timeouts) instead causing an unexplained EOF. A fix for this will be available in the next release.

This is now fixed in the 0.9.11 release!


Install maturin as part of pyenv since it's needed to build

Maturin isn't tracked as a dependency anywhere, but it's needed to build. Luckily we can just pip install it when we setup the pyenv in the first place.

@warthog9 is this the Makefile that you deleted in your packaging branch?

It is tracked as a dependency of the crate, where it's actually used:
https://github.com/tenstorrent/luwen/blob/main/crates/pyluwen/pyproject.toml
And yes @TTDRosen that makefile is deleted in the packaging branch as it causes a lot of packaging problems having a makefile the way that makefile was


BUGFIX-#193: User can access undefined routes.

Summary
Added 404 not found page with 3d links in the background.
Description
This PR  resolves issues for undefined routes.
Previously there was an empty page when user enters any undefined url.
But now a beautiful Error page is added.
Images

Issue(s) Addressed
Closes #193
Prerequisites

[✓] Have you followed all the [CONTRIBUTING GUIDELINES]

Also removed login concerns from Error Page

Sorry! @krishnaacharyaa  I'll not repeat commit mistake again.

@Mayur-666
couple of suggestions,
Can you reduce the padding or height of the appbar, ideally just give p-4 or p-8 don't hardcode any value for height of the appbar.
And can you reduce the pace of the things moving around so it looks subtle
and set opacity to the bg of the appbar, so the things which are moving around will move around behind appbar even similar to the footer.

And make sure the "wanderlust" is at the same place when in proper page and in error page, it shouldn't seem like it is shifting its position. basically have same px values

Ok

And bonus, if you create a new PR with proper PR and commit and all the required changes requested. I'll give 3level, because it involved a bit of work and I'm fine giving it anyways, as far as you are actively interested in doing things :)