Implements a multivariate normal distribution

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
#2222
What are the significant changes in functionality due to this change request?
Implements a multivariate normal distribution in python that uses scipy.stats.multivariate_normal and scipy.stats.norm classes as a backend.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Job Test qsubs sawtooth on 415048b : invalidated by @joshua-cogliati-inl
Failed with: DirectoryNotACondaEnvironmentError: The target directory exists, but it is not a conda environment.

Job Test qsubs sawtooth on 04f284a : invalidated by @j-bryan
Failed with DirectoryNotACondaEnvironmentError: The target directory exists, but it is not a conda environment.

The last several commits, starting with this commit, add a Crow MultivariateNormalDistribution object to the constructor of the MultivariateNormalPCA class added by this pull request to check the results of the SVD of the covariance matrix of the distribution against the existing Crow implementation. This is done for now as a way to check the remainder of the implemented code. The idea here is that if inconsistencies in the SVD (e.g. signs in the U and V matrices that are not unique) are the only discrepancy between my added class and the Crow implementation, all of the tests should pass as long as the Crow classes SVD matrices are used. Rather than replace the entire matrices calculated by the RAVEN python code, I only adjust the signs of the left and right singular vectors to match those from the Crow SVD.
After applying this sign correction, there are three tests that are failing across the test machines:

tests/framework/MCMC/MetropolisTenDimMVN
tests/framework/MCMC/MetropolisTenDimPropMVN
tests/framework/pca_sparseGridCollocation/attenuCorrelation

The decomposed covariance matrix in the MetropolisTenDimMVN contains several repeated singular values, and the order of their associated vectors is not consistent between my implementation and the Crow result. Something similar happens with the MetropolisTenDimPropMVN test, but the covariance matrix in question has many more repeated singular values (9 of the 10). The vector associated with the unique singular value matches between implementations but the remaining vectors from the repeated singular values do not. This is not too surprising though, since eigenvalues of multiplicity greater than 1 do not have unique eigenvectors. Both implementations produce a valid SVD. Finally, the attenuCorrelation test is failing with a very small diff in an output XML file (difference only in the 8th or 9th significant digit). I believe this may be stemming from small numerical differences in the SVD results between implementations. I believe this test is still working as intended but needs either a regold or a tolerance added to the test.
While this isn't as nice as a script showing that the tests aren't diffing, I do believe this shows that the added code is working as intended, and the diffs in the failing tests that occur when the Crow SVD sign convention is not used are the result only of that difference in sign. @joshua-cogliati-inl, in your opinion, is this thorough enough to green light regolding effected tests?

Job Test qsubs sawtooth on 972af8b : invalidated by @joshua-cogliati-inl
Failed with: DirectoryNotACondaEnvironmentError: The target directory exists, but it is not a conda environment.

Job Test qsubs sawtooth on e1eab7f : invalidated by @joshua-cogliati-inl
Failed with: DirectoryNotACondaEnvironmentError: The target directory exists, but it is not a conda environment.

Job Mingw Test on e1eab7f : invalidated by @joshua-cogliati-inl
failed in fetch

Job Mingw Test on e1eab7f : invalidated by @j-bryan
Error when fetching Raven

Job Test qsubs sawtooth on e1eab7f : invalidated by @joshua-cogliati-inl
FAILED: Diff tests/cluster_tests/test_mpiqsub_long

Job Test qsubs sawtooth on e1eab7f : invalidated by @joshua-cogliati-inl
Failed with: DirectoryNotACondaEnvironmentError: The target directory exists, but it is not a conda environment.

Job Test qsubs sawtooth on e1eab7f : invalidated by @j-bryan
Something broke but results won''t load. Same error as before? Trying again.


There's a 5 second blocking wait on issuing the credential in the controller

Will have to remove or make the 5 second wait on issuing the credential in issues.controller.ts on the post '/' route.

Also remove the "mime-type" from the controller when the next version of Street cred is available.

This is actually required.. if you don't instantiate a wait streetcred and the agent don't form a connection in some cases.. the polling will push it through and credex issuance will error out.

Assuming this can be closed based on the last comment.


[New Extension] Spellbinding Weaver - ChatGPT Completion for Raycast

Description
Spellbinding Weaver
This Raycast extension utilizes the OpenAI ChatGPT language model to generate human-like text based on user input. The extension allows users to send prompts to the ChatGPT model and receive a response. The response is automatically copied to the clipboard, allowing users to auto paste it into other applications when completion is received.
Screencast
Spellbinding in action
https://user-images.githubusercontent.com/10308616/227592174-715310e2-d971-46ae-b5f5-557b1bd094b5.mov
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Just found out that Raycast have Raycast AI, sorry.


Synchronous Version

Please make a synchronous version of this module.

Synchronous Mysql access would be very helpful as in most cases I have to wait for the database answer to continue the current method.

I'm locking this, because this is impossible; Node.js does not provide anyway to make sync network calls.


Fix url: ...triage-bugzilla#what-do-you-triage

Coverage remained the same at 31.613% when pulling 7d9f3531dd329a7a9a13b042bd4ce793ee4df28e on calixteman:fix_anchor into 2eb6c4f54aa1f7ad69fe559f08aafe9c3d17f025 on mozilla:master.


Int parsed as float when loading a dictionary from JSON.

Operating system or device - Godot version:
Ubuntu 16.04 64b, Godot version fc70824 (probably any previous version as well)
Issue description (what happened, and what was expected):
Integers in a dictionary loaded from JSON are unusable to reference indices in arrays, apparently, despite looking like an int, sounding like an int, etcetera, it's not an integer. This is a very serious issue that can cause endless hard-to-spot bugs in games if not handled with utmost care.
Steps to reproduce:
I am loading some data from json files. That data contains some integers that are used to reference an index in a normal array.
If done from a regular, gdscript dictionary, (such as var mydict  = { value = 5 } then I can do print(myarray[mydict.value]) and as long as there's an index 5 in the array, that will work, no problem.
The problem comes if I acquire the same dictionary (with "value = 5") from json. Then it will fail. After spending some time debugging the issue, I came to the conclusion that the only way to get it to work is to "cast" the json-provided integer (like print(myarray[int(mydict.value)])). Meaning that, somehow, value doesn't contain an integer despite it having no decimal part in the json file.
Link to minimal example project (optional but very welcome):
Demonstration of this bug as-is: Node.tscn.zip
Demonstration that casting to int solves it: Node2.tscn.zip

After discussing it on IRC, it could be convenient for the built-in json parser to automatically handle numbers without decimal parts as integers. That would prevent this issue and make results more consistent.

Please note the JSON standard doesn't defines int and float, just number. Externally-generated JSON may or may not do the difference, and possibly round floats making them look as ints or vice versa.

Since the whole issue came from the problem described in #6361, I'd better close this up to clean up spurious issues.


Compatibility with react-leaflet v3

Is this plugin compatible with react-leaflet v3?
When using<EMAIL_ADDRESS>I get this error from npm list:
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>

Hey @Vacilando thanks for pointing this out. I've just published a new major version to npm that should solve this for you. Let me know if it doesn't work.

Hey @Vacilando thanks for pointing this out. I've just published a new major version to npm that should solve this for you. Let me know if it doesn't work.

It's fine now, thanks!

It's fine now, thanks!


What is license of jqlistbox

I am impressed with jqlistbox as it is supported in iphone. May i know what is the license of this plugin.

Hi, thank you! :) I have just added an MIT license in the master branch.


chore: Create codeql-analysis.yml

That's what I understood, yes. I only did a very quick research and there are community solutions for circle, but they don't integrate with GH well. one of the points of this security scan is to give our security department some control and peace of mind


fixed value error

fixed #22 partly.

I think it's simpler to just do this
https://github.com/google/enjarify/commit/ad8d64f6e564922bbe1f08f488cf7f51c961422d
Although in the long term, the proper solution would be to move to memoryview.


Tab in Scenario text breaks it

Expected Behavior
Tab of string should have no effekt
Actual Behavior
But it does
Steps to reproduce error
Tests Green

Tests Red

Estimation: 5


Link shortcut

On the top of the page, you can introduce links to have shortcuts

Developed on 5183d69b42e51af6143f4b74f2471990fb5d9ac8


404 page resonsiveness issue

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
v2.2.1
Issue Description
Bagisto Demo Link-https://demo.bagisto.com/bagisto-common/arctic-frost-winter-accessories-bundle-j
There are some padding issue between 404 and Page not found senetence. ( Responsivess issue )
Preconditions
No any condition , check your 404 error page.
Steps to reproduce
One any error page (404 error page).
Expected Result
Make this fully responsive.
Actual Result
404 page responsive
Screenshots

Additional context
No response

@ajeetkesherwani
Hello there,
Thank you so much for your contribution towards Bagisto.
Upon checking the issue, it was found that the URL attached from your end is incorrect, which is causing the issue.
Issue URL: https://demo.bagisto.com/bagisto-common/arctic-frost-winter-accessories-bundle-j
Correct URL: https://demo.bagisto.com/bagisto-115-113-191-18/arctic-frost-winter-accessories-bundle
I have checked and found it working. I am attaching a video for your reference. Please have a look at it. If you are still facing this issue, I would really appreciate it if you could share the steps to reproduce the issue.
Thank you.
Video
https://github.com/user-attachments/assets/9949d999-b55d-404d-8ebe-2f6c695b8dab

You once again check the issue title on Geet. I raised an issue for
the responsiveness of the 404 error page. Your error page is not
responsive.
On Thu, 25 Jul 2024 at 9:38 AM, Ashish Kumar Bhai @.***>
wrote:

@ajeetkesherwani https://github.com/ajeetkesherwani
Hello there,
Thank you so much for your contribution towards Bagisto.
Upon checking the issue, it was found that the URL attached from your end
is incorrect, which is causing the issue.
Issue URL:
https://demo.bagisto.com/bagisto-common/arctic-frost-winter-accessories-bundle-j
Correct URL:
https://demo.bagisto.com/bagisto-115-113-191-18/arctic-frost-winter-accessories-bundle
I have checked and found it working. I am attaching a video for your
reference. Please have a look at it. If you are still facing this issue, I
would really appreciate it if you could share the steps to reproduce the
issue.
Thank you.
Video
https://github.com/user-attachments/assets/9949d999-b55d-404d-8ebe-2f6c695b8dab
—
Reply to this email directly, view it on GitHub
https://github.com/bagisto/bagisto/issues/10041#issuecomment-2249335994,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2WKACHXZPAJTEONRZHQZ7LZOB25DAVCNFSM6AAAAABLM5RFCKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBZGMZTKOJZGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Prepare for ReqAthena v0.2

This is in line with https://github.com/livebook-dev/req_athena/pull/40

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Fix floating input style

Ticket Link
https://mattermost.atlassian.net/browse/MM-49386
NONE

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/27174
https://circleci.com/gh/mattermost/mattermost-mobile/27173

Artifact links:
https://output.circle-artifacts.com/output/job/efa62c06-5b7f-4619-bce2-3e0bebe21a61/artifacts/0/tmp/artifacts/Mattermost_Beta.ipa
https://output.circle-artifacts.com/output/job/3838e73b-0338-4b45-8a79-3cb457f045a6/artifacts/0/tmp/artifacts/Mattermost_Beta.apk

Thanks @enahum
Tested on Android (and iOS) no longer reproduces - looks good to merge.


Fail to import the MolDataset

Hi,
I would like to use the MolDataset class shown in your document (https://gnina.github.io/libmolgrid/python/index.html?highlight=merge_coordinates#molgrid.torch_bindings.MolDataset), but it fails to import this class. I have tried both installing from the source (version 0.2.1) and installing by pip. They all show the ImportError as the following:
ImportError: cannot import name 'MolDataset' from 'molgrid.torch_bindings' (/usr/local/lib/python3.7/dist-packages/molgrid-0.2.1.dev35+g281b3c3-py3.7.egg/molgrid/torch_bindings.py)
(pip install version: ImportError: cannot import name 'MolDataset' from 'molgrid.torch_bindings' (/opt/tiger/miniconda/lib/python3.7/site-packages/molgrid/torch_bindings.py))
Could you please help me with this issue?
Thanks!

We haven't included this feature in a release yet - it isn't quite clear what the right use case and API for this is.  Can you explain what you are doing and how you want to use this?

@dkoes Prof.Koes, I plan to use moldataset for pytorch distributed data parallel, @drewnutt replied me that I can use it by building from source. Actually, I don't know how to recompile it from source, are there any documents for tutorial? Thanks a lot.

The main README.md has a step by step instruction on how to build from source.

thanks for quick reply. OK, I will first try that.


Login

Login allows access with only a partial username.

Working on this - Implementing Search User (Garry) will help this along better.

Error still exists; I believe there might be a context issue, or an issue with the loading i'm not sure.

Login function has been fixed an i've marked it complete


failed to execute torch.compile

I have made the custom_ops according to pytorch-tutorials: https://pytorch.org/tutorials/advanced/python_custom_ops.html#python-custom-ops-tutorial.  But it can't work.
Here is my reproduction code:
import torch
import torch.nn as nn
import flashinfer

@torch.library.custom_op("mylib::flashinferattn", mutates_args=())
def flashinferattn(q_tensor: torch.Tensor, k_tensor: torch.Tensor, v_tensor: torch.Tensor, scale: float) -> torch.Tensor:
    return flashinfer.single_prefill_with_kv_cache(q_tensor, k_tensor, v_tensor, sm_scale=scale)

@flashinferattn.register_fake
def _(q_tensor, k_tensor, v_tensor, scale):
    return torch.empty_like(q_tensor)

class Attention(nn.Module):
    def __init__(
        self, inner_dim, head_num,
    ):
        super(Attention, self).__init__()

        self.num_heads = head_num
        self.head_dim = inner_dim//self.num_heads

        self.scale = self.head_dim**-0.5

        self.to_q = nn.Linear(inner_dim, inner_dim, bias=False)
        self.to_k = nn.Linear(inner_dim, inner_dim, bias=False)
        self.to_v = nn.Linear(inner_dim, inner_dim, bias=False)

        self.to_out = nn.Linear(inner_dim, inner_dim)

    @torch.no_grad()
    def forward(self, hidden_states):

        q = self.to_q(hidden_states)
        k = self.to_k(hidden_states)
        v = self.to_v(hidden_states)

        batch, seq_len, hidden_size = q.size()

        attn_output = torch.cat(
                                [
                                    flashinferattn(q[i,:,:].view(-1, self.num_heads, self.head_dim),
                                                    k[i,:,:].view(-1, self.num_heads, self.head_dim),
                                                    v[i,:,:].view(-1, self.num_heads, self.head_dim), self.scale) for i in torch.arange(0,batch)
                                ], 0).view(batch, seq_len, hidden_size)

        attn_output = self.to_out(attn_output)

        return attn_output

torch.compiler.reset()
attn = Attention(inner_dim=640, head_num=10).cuda().half()
attn_compiled = torch.compile(attn, mode="reduce-overhead", fullgraph=True)
#batch=2, seq-len=128, hidden-size=640
inputs = torch.rand((2,128,640),device="cuda:0", dtype=torch.float16)
attn_out = attn_compiled(inputs)
print(attn_out)

partial log:
RuntimeError: Failed running call_function <built-in function getitem>(*(FakeTensor(..., device='cuda:0', size=(2, 128, 640), dtype=torch.float16), (FakeTensor(..., size=(), dtype=torch.int64), slice(None, None, None), slice(None, None, None))), **{}):
aten._local_scalar_dense.default

During handling of the above exception, another exception occurred:

    raise Unsupported(msg)
torch._dynamo.exc.Unsupported: data dependent operator: aten._local_scalar_dense.default; to enable, set torch._dynamo.config.capture_scalar_outputs = True

from user code:

SGLang used torch.compile and flashinfer together without any issues.
You can check out some end-to-end examples here https://github.com/sgl-project/sglang/issues/1008

@merrymercy
It seems that SGLang didn't open fullgraph=True in torch.compile : https://github.com/sgl-project/sglang/blob/main/python/sglang/srt/model_executor/cuda_graph_runner.py#67.
If I use mode="max-autotune-no-cudagraphs" in my above code like sglang, it can also succeed. But I don't know how these settings affects performance

Hi @bianxuxuxu , by looking at your code I think the problem exist in expressions like q[i,:,:], and you are trying to run batch prefill with single_prefill_with_kv_cache:
        attn_output = torch.cat(
                                [
                                    flashinferattn(q[i,:,:].view(-1, self.num_heads, self.head_dim),
                                                    k[i,:,:].view(-1, self.num_heads, self.head_dim),
                                                    v[i,:,:].view(-1, self.num_heads, self.head_dim), self.scale) for i in torch.arange(0,batch)
                                ], 0).view(batch, seq_len, hidden_size)

I don't recommend doing this (the single_prefill_with_kv_cache is not designed for batch prefill and not cudagraph compatible) but I can do it anyways,  you don't have to batch like this actually, there is a easier way (by fusing B and H dimension) where you only have to call one kernel:
q_hnd = q.view(batch_size, seq_len, num_heads, head_dim).transpose(1, 2).contiguous().view(-1, seq_len, head_dim)
k_hnd = k.view(batch_size, seq_len, num_heads, head_dim).transpose(1, 2).contiguous().view(-1, seq_len, head_dim)
k_hnd = k.view(batch_size, seq_len, num_heads, head_dim).transpose(1, 2).contiguous().view(-1, seq_len, head_dim)
attn_output = single_prefill_with_kv_cache(q_hnd, k_hnd, v_hnd, layout="HND", scale=scale).view(batch, num_heads, seq_len, head_dim).transpose(1, 2).contiguous().view(batch_size, seq_len, head_dim)

You are encouraged to use https://docs.flashinfer.ai/api/python/prefill.html#flashinfer.prefill.BatchPrefillWithRaggedKVCacheWrapper instead for batch prefill where we support variable length and is designed to be CUDAGraph compatible.

Hi @yzh119 . Thanks for your advise.
But the q_hnd layout is incorrect, it must be in shape of [qo_len, num_qo_heads, head_dim] according to the comments ：https://docs.flashinfer.ai/generated/flashinfer.prefill.single_prefill_with_kv_cache.html#flashinfer.prefill.single_prefill_with_kv_cache

I see, sorry about the confusion, then I think the following way should work:
q = q.view(batch_size, seq_len, num_heads, head_dim).transpose(0, 1).contiguous().view(seq_len, -1, head_dim)

The general idea is to fuse batch_size with num_heads, they are equivalent when seq_len is a constant.

@yzh119 . Yes. This is correct and can run correctly.
Is there any difference in performance between the implementation and using [BatchPrefillWithRaggedKVCacheWrapper]

BatchPrefillWithRaggedKVCacheWrapper was designed for variable length (using Ragged Tensor which don't need padding) and has a load-balancing scheduler, and it's designed to be compatible with CUDAGraph (PyTorch's CUDAGraph requires a fixed kernel configuration, and single_prefill_with_kv_cache uses dynamic kernel configuration).
Performance-wise, if the input length is not variable, they should be similar.

@bianxuxuxu In sglang, we use our own code to capture the cuda graph at the outermost level. We compare our perf with gpt-fast and find there is no performance degradation.
The latest sglang is faster than gpt-fast in most cases.

Close this issue for now, feel free to re-open it if there are still other concerns.


[KBM] long pressing the Win key while remapping a key results in no mouse input

transfered from:https://github.com/microsoft/terminal/issues/3325
Environment
Windows build number: [version 10.0.19041.329]
PowerToys version: 0.18.2
PowerToy module for which you are reporting the bug (if applicable): shortcut guide

Steps to reproduce

start a fresh runtime of powertoys (don't press win key until stateted)
open pt settings and make sure shortcut guide is active
go to keyboard manager and click remap a key
click the plus button and then the type key button
you should be in this screen now:

hold down the win key for at least the delay set for shortcut guide to come up

Expected behavior

shorcut guide should not try to come up
Actual behavior

shortcut guide try to come up and gets stuck in an transparent state stealing the majority of mouse inputs
Screenshots

Hi @LuanVSO
thanks for reporting the bug.

Had seen the same bug. But didn't find the key. Now I know the key.

If KBM is up, SG key should be disabled.

@LuanVSO
can you still repro this problem with 0.29.3? I tried the repro steps but the mouse input kept working regularly.

@LuanVSO
can you still repro this problem with 0.29.3? I tried the repro steps but the mouse input kept working regularly.

@LuanVSO
can you still repro this problem with 0.29.3? I tried the repro steps but the mouse input kept working regularly.

seems to be working correctly here too

@LuanVSO
can you still repro this problem with 0.29.3? I tried the repro steps but the mouse input kept working regularly.

seems to be working correctly here too


XP-336 Fix docs badge

References
XP-385 Fix the documentation badge in xain-fl
Summary

fixed badge link

Are there any open tasks/blockers for the ticket?
No

Reviewer checklist
Reviewer agreement:

Reviewers assign themselves at the start of the review.
Reviewers do not commit or merge the merge request.
Reviewers have to check and mark items in the checklist.

Merge request checklist

[ ] Conforms to the merge request title naming XP-XXX <a description in imperative form>.
[ ] Each commit conforms to the naming convention XP-XXX <a description in imperative form>.
[ ] Linked the ticket in the merge request title or the references section.
[ ] Added an informative merge request summary.

Code checklist

[ ] Conforms to the branch naming XP-XXX-<a_small_stub>.
[ ] Passed scope checks.
[ ] Added or updated tests if needed.
[ ] Added or updated code documentation if needed.
[ ] Conforms to Google docstring style.
[ ] Conforms to XAIN structlog style.

Closed because of the wrong ticket number


redundant libraries

why using from lstchain.io.data_management import * when the data_management library of the LST_scripts repo is going to be used anyway ?
Just in case I'm loosing a point - can this be modified so that only the library from LST_scripts is used ?

I think it's done that way only for historical reasons, when there were only scripts in LST_scripts and function needs to be imported from somewhere.
In any case, yes, we should be the library self-consistent if possible.

Resolved in #31


Alternatively return redis result as ES6 Map/Set in callback/promise.

Hi there, I wonder if anyone would +1 if the returned value of some certain method like smembers  and hgetall could be ES6 Set and Map?
This could be alternative like using redis.smembers().asSet() and wouldn't break the original structure.
I'd like to make a PR if you guys agree to it.

I think you should simply use response transformers, that would lead to the desired result

On Aug 29, 2017, at 9:01 PM, kzz555<EMAIL_ADDRESS>wrote:
Hi there, I wonder if anyone would +1 if the returned value of some certain method like smembers and hgetall could be ES6 Set and Map?
This could be alternative like using redis.smembers().asSet() and wouldn't break the original structure.
I'd like to make a PR if you guys agree to it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/luin/ioredis/issues/515, or mute the thread https://github.com/notifications/unsubscribe-auth/ABol0X9WNlGfxI_T0jbkzqhezIa8K_3Qks5sdN6TgaJpZM4PG1qv.

Thank you for replying!
I'm a little confused about the response transformers you kindly mentioned. Did you mean the wrapping work I did my self after getting the result?
IMHO, wrapping the result in a way that meets the ES standard could be a little more better than getting the 'raw' data.

It's pretty straight to wrap the result of smembers in a Set:
const set = new Set(await redis.smembers())

Does this work for you?


How to delete a file share?

I am sure it is explained somewhere else, but I could not locate it and I could not find a link to it from this page.
IMHO it would make sense to include a link to the document about deleting a file-share.

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

ID: ad2c9646-a4bc-aa28-de27-5fce4d4b4e44
Version Independent ID: 1288f4c1-e871-8bc7-9693-2e62d5074a09
Content: Create an Azure file share - Azure Files
Content Source: articles/storage/files/storage-how-to-create-file-share.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

@szabgab we'll review your feedback and get back to you shortly. Thanks.

@szabgab I don't see documentation specifically for deleting a file share. You can select the share and hit the delete button in the portal, or use CLI, or Powershell.
Sending to the author to see if this is something they think should be added.

Adding a paragraph such as this:
"In order to remove a file share using the Azure CLI use the delete command  or the DELETE API method.
This paragraph includes both the "remove" and the "delete" keywords that people often use for this action and will make it easier for them to find.
However please also note the instructions there are also seem to be missing some information: https://github.com/Azure/azure-cli/issues/20232

#reassign: @khdownie

@szabgab You have the same name as one of my favorite guitarists. The Sorcerer! Anyway, I added a "Delete file share" section per your request:
https://learn.microsoft.com/en-us/azure/storage/files/storage-how-to-create-file-share?tabs=azure-portal#delete-a-file-share

#please-close


wait returns unexpected end of JSON every 5 minutes

In a loop, I watch for changes in a directory
p.client.Watch(storesKey, 0, true, nil, nil)

All changes are picked up and handled by my app.
However, every 5 minutes I receive an invalid response, which I currently just ignore (skip this event).
W1110 11:16:35.776132 49235 etcd_store_dao.go:71] watch failed, skip this event:unexpected end of JSON input
W1110 11:21:35.777721 49235 etcd_store_dao.go:71] watch failed, skip this event:unexpected end of JSON input
W1110 11:26:35.779467 49235 etcd_store_dao.go:71] watch failed, skip this event:unexpected end of JSON input
W1110 11:31:35.779922 49235 etcd_store_dao.go:71] watch failed, skip this event:unexpected end of JSON input

Is this really an issue or should this be ignored as I do because it is some hearbeart ping?

@leverly @GeertJohan @emicklei
etcd 2.0 server will not generate empty json in normal case. The connection will only be closed if the client has actually gone.
#2049


Add a button to import libraries from a path file.

Is your feature request related to a problem? Please describe.
We need to inform the user the Capability to import libraries from a local path file ftp file or http.
Describe the solution you'd like
we need to add example or a check box or drop box, discussion need to be generated around this issue.
Describe alternatives you've considered
A message as a tool tip can be implemented.
Additional context
N/A

Hi, for do these changes we need modify this part of the code in apps>Products>views.py file in CreateSourceView class:

Also we need modify this part of the code in apps>Products>forms.py file:

Hi, I think you can add the button to upload a zip file, the this way:
apps>Products>forms.py

apps>Products>views.py

Under review by Ulises ( need to block the field that is not being used)


Youtube integration error

To avoid RuntimeError: Maximum call stack size exceeded(…)

Good commit message here would be:

Handle 'timeupdate' event only once a second
To prevent infinite loop caused by calling setCurrentTime() inside a handler.


Directions are lacking

Open a PowerShell terminal as an administrator on your Windows machine.
Run the following command to download the Windows Docker installer PowerShell script file: Invoke-WebRequest https://adaprodconsole.blob.core.windows.net/public-files/LogCollectorInstaller.ps1 -OutFile (Join-Path $Env:Temp LogCollectorInstaller.ps1)
To validate that the installer is signed by Microsoft, see Validate installer signature
To enable PowerShell script execution, run Set-ExecutionPolicy RemoteSigned
Run: & (Join-Path $Env:Temp LogCollectorInstaller.ps1) This installs the Docker client on your machine. While the log collector container is installed, the machine will be restarted twice and you will have to log in again. Make sure the Docker client is set to use Linux containers. -Elaborate? Why does this need to be done? Is it not done by default? Where can this be done? Screenshot would go a long ways here.
After each restart, open a PowerShell terminal as an administrator on your machine, re-run: & (Join-Path $Env:Temp LogCollectorInstaller.ps1)
Before the installation completes, you will have to paste in the run command you copied earlier. - Which command there were multiples.... after banging my head I realized it was in the prior step, would be helpful to indicate that further. When will this pop up? I don't remember ever seeing this to begin with
Deploy the collector image on the hosting machine by importing the collector configuration. - Can you elaborate?  Import the configuration by copying the run command generated in the portal. Oh -- so this is what you should have just said in the first place.
If you need to configure a proxy, add the proxy IP address and port number. For example, if your proxy details are <IP_ADDRESS>:8080, your updated run command is: - N/A
Container shows up in Docker, you access it and run this command Verify that the collector is running properly with the following command: docker logs <collector_name> - This doesn't work, and your screen shot shows the command with sudo - which also doesn't work
It does show as "connected" in the portal of CAS.
Over all this documentation needs work, usually those of us trying to make this log collector have 0 experience with Docker and probably only minor Linux experience which is why we had to go this route anyway. I had this working in the past with a Ubuntu but for security reasons had to go with a Windows box to do this now.
EDIT: Figured even more stuff out, the command listed for step 8: apparently needs to be done in powershell or cmd not within the docker container cli, this screenshot in the documentation isn't even relevant so someone just recycled a Linux snapshot to be lazy.
Thanks.
[Enter feedback here]

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

ID: e4f7c304-9777-e8d6-1e49-e80ea333c072
Version Independent ID: c659f6d0-9011-fd40-d33e-090a719e82da
Content: Roll out continuous reports for Cloud App Security using a Docker on Windows
Content Source: CloudAppSecurityDocs/discovery-docker-windows.md
Service: cloud-app-security
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thank you for your comments. We'll investigate and get back to you.


URL blocking is not working properly

Steps to reproduce:

Enable Social Media Filter
Go to http://progorodsamara.ru/news/view/207467

Actual behavior:
Social widget is not blocked
Screenshot:

The same with Android version with CL.

Another example - cannot block VK comments
https://mister-robot.tv/1-sezon/15-6-seriya.html

||vk.com/widget_comments.php$important

Duplicate of https://github.com/AdguardTeam/CoreLibs/issues/771


DF Hotkeys: Add ability to map macros to custom Hotkeys

Add a way for users to add their own custom Hotkeys that map to macros.
Maybe an extra checkbox on the macro Config that registers a hotkey for the macro.

I've decided to not do this feature.


check if 'warnings' and/or 'upgradeableComponents' return values when not null

these on arb have both
"0x2d6ced12420a9af5a83765a8c48be2afcd1a8feb000000000000000000000500"
"0x4b3af34eb1135d59df8b9cdc2ff07d30d05334c400000000000000000000050d"
"0xd0ec47c54ca5e20aaae4616c25c825c7f48d40690000000000000000000004ef"

The arbitrum pools mentioned above should have warnings and/or upgradeableComponents under priceRateProviderData for the specific rate providers but the api currently returns null for both props.

@alter-eggo – all looks good on the API side. The GBP.e token doesn't have the "safe" argument set based on the review. Same for rETH.


[zabbix-community/zabbix] Mysql missing

Is your feature request related to a problem? Please describe.
No mysql documentation, has it been dropped or just missing
Describe the solution you'd like
Update docs to have mysql support for example:
https://artifacthub.io/packages/helm/fermosit/zabbix-server-mysql/0.0.5

Hi @seanfariacustomink!
Despite knowing that Zabbix officially supports MySQL and other types of databases, this helm chart has only focused on PostgreSQL since the first commits. Supporting many databases in a helm chart requires a lot of technical effort and time.
If you have the proper knowledge and time to tweak this helm chart and make it flexible to support multiple database types, starting with MySQL, we'd be happy to review and test your changes.
Merry Christmas and have an excellent 2023 for you, your family and friends.

Hi @aeciopires,
Seems like a lot of this work is already done, i.e. mysql-zabbix-helm-chart

Hi @aeciopires,
Rather than support MySQL or etc databases can the instructions be provided for someone to point to an existing MySQL DB?

Hi @seanfariacustomink!
Good idea. It's a better way. Do you agree @sa-ChristianAnton?

Support  for external dbs would be highly appreciate it.

Support for external dbs would be highly appreciate it.

Also for us, currently most useful would be a helm-chart variant where the persistence (mariaDB or mysql) is outside k8s - any hints for this one ?

External DBs would help our team a lot. I've seen PRs that in part address this issue already https://github.com/zabbix-community/helm-zabbix/pull/25
We have tried to replace zabbix/zabbix-server-pgsql with a mysql one and our own DB but the server could not start. Is there some volume misconfiguration that we need to fix?

Hello @marijusGood @ralfhauser @xeberth @inggaboluna @seanfariacustomink!
I understand the importance of supporting MySQL/MariaDB and other databases supported by Zabbix.
But unfortunately I don't have the knowledge and time to implement this feature.
I made a lot of changes to the helm chart to unlock unique PostgreSQL configurations, but that's about it.
Another problem for us is running multiple tests on each database type with each new version of the chart. We do not have automated tests.
But if anyone has time to contribute, I'd appreciate it.
Have a nive week.


[Beta] Navbar color/contrast issues

App version: 8.0-beta1
OS version: Android 9
Device: Motorola moto g⁶
There are some color combinations that it very difficult (or literally impossible) to see tabs on the bottom navbar.
In light mode, setting the primary color to something dark makes it hard to see the non-selected tabs:
And black makes it literally impossible:
Likewise, setting the primary color to something bright makes the current tab hard to see:
Dark mode does not respect the primary color setting, and forces a color that makes the current tab hard to see:

I changed it to use the accent color when selected and just white otherwise (see below). Looks decent IMO


fix a grammer error

In the first paragraph of section Automatic Parallelization, there are two "technologies ".

Tppppppppp thanksssssssss.
-Zackkkkkkkkkkkk


Upgrade molo and molo.profiles

Relates to the following tickets:
Fixing mobile required feild
https://praekeltorg.atlassian.net/browse/MOLO-378
https://praekeltorg.atlassian.net/browse/TUNE-469
Adding media upload/download functionality
https://praekeltorg.atlassian.net/browse/TUNE-244

👍 🍰


Fixed TYP of values argument of the pandas.DataFrame.pivot_table

This pull request addresses issue #885 by modifying the pivot_table stub definition in frame.pyi to allow lists of strings as the values argument. Previously, the stub definition only supported single strings or None for values.
Changes Introduced:
Updated the values argument type in the pivot_table stub definition to include _Sequence[str]. This allows lists of strings to be used for values.
Test Case:
No test case added as there was already an existing test case in the file test_frame.py on line 922 :
check(
assert_type(
df.pivot(index="col1", columns="col3", values=["col2", "col4"]),
pd.DataFrame,
),
pd.DataFrame,
)

Which already test the addressed issue and updated fix.
[ ] Closes #885

Hello @Dr-Irv , I have included the test case. Can you please have a look and check if it is alright.

Changed per your suggestion. Thanks!

Hopefully, no more formatting issue now.


[DS-2502] Correct some dependencies to prevent pulling in servlet-api

This needs some testing.  It leaves servlet-api.jar out of all webapp.s but includes it in [DSpace]/lib.

+1 I was testing a jspui instance on tomcat8 and it wouldn't appear to start. I cherry-picked this patch, and JSPUI is working fine on tomcat8 now.

+1, I've been running this in production for months, I appreciate not hearing about servlet-api every time I watch the log files.


take multiple snapshots for previews

instead of single image:

generate 4 for each preview in previews

fixed in: https://github.com/paulz/SwiftUI-snapshot-testing/commit/7f2bb16e959f974461176d666b146b7b8db50673


ng generate route command doesn't respect the --default flag

Mac OSX (El Capitan)
ng --version:

angular-cli: 0.0.33
node: 5.10.1
os: darwin x64

Steps:
ng new my-app
cd my-app
ng g route home --default

The generated route config is:
{path: '/home/...', name: 'Home', component: HomeComponent}`

The expected config is:
{path: '/home/...', name: 'Home', component: HomeComponent, useAsDefault: true}

I saw this parameter/argument the other day and wondered if it was implemented, apparently not.


Error while importing fsf-api-access package

After creating a new virtual environment, and installing fsf-api-access package using pip, I am unable to successfully import the package using python. Following is the error I receive:

It seems that there is an issue with importing shapely through fsf-api-access package installed through pip. Can you provide the conda install documentation for fsf-api-access package?

Hey Pranj, what version is your Python? And are you on a Windows?
It looks like for Windows + Python 2.7 combination, binary wheels aren't available. https://gis.stackexchange.com/questions/62925/shapely-not-installing-correctly
If you have Python 3, you can also try to install shapely manually with pip install shapely
The linked post also suggests conda install shapely if you are using Anaconda or Miniconda

That works.
I am now able to import fsf-api-access package.
Thanks


Request: Add Update Order Method

currently we can only use
`
$user = User::first();
$user->newOrder()
->setRecipientName('John Doe')
->setCountryCode('FR')
->create();
`
It would be fantastic if we could update an existing order using the API too.

@benhamlett makes sense. I will try to find some time soon to add this should be fairly easy.
PRs are always welcomed :)
Thanks

@gpanos Cheers mate,
I think the only issue I am having with this atm is that there is no way to do a newOrder() if you have extra fields on the table that are not nullable. currently our process is create the order locally and on approve we want to create the order on Pwinty. So even a method in the OrderBuilder to update the pwinty_id & pwinty_status only would be great :)

@benhamlett tagged a new release that provides a updatePwintyOrder  method. https://github.com/Wingly-Company/pwinty#updating-an-order.


Missed iteration with an optional component of a composed entity

The following should trace, but doesn't:
final g = new EntityGroup();
g.createEntity({ x: "" });
g.foreachEntity(function(e : { ?b:String }) e.b = "");
g.foreachEntity(function(e: { b:String, ?c:String }) {
	trace("missed");
});

It seems the component x needs to be there for the test to fail.

Seems to be introduced by 1ba4e9487c047f27198940f5bc58f2b57cd2d685.


Problem on replicating LLFF results

Dear Authors,
Thanks for your attention. I have met some problem on replicating the results of forward facing dataset. I have trained a VM-96 model with the configuration under configs/flower.txt. The PSNR I get is around $28.2$ ~ $28.3$, which has a gap against $28.6$ reported by the paper. The depth maps rendered also contain more floater and aren't as clean as the results you provided. Would you mind suggesting whether the configs/flower.txt is the correct experiment setting to replicate your results? If not, how could I correctly configure the experiment to replicate the results?
Anyway, I'm really impressed by your marvelous work, as well as the clean implementation :)

have you figure it out?


New Plugin: PatchCom4360

Adds a required patch for several broadcom cards that work with apples 4360 kext
link: https://github.com/coderobe/PatchCom4360

For unknown reasons, this plugin is unstable, likely because of kext loading / lilu injection race conditions - closing this for now until i've got more info and/or a fix

The only thing I could think of is non-bootloader loading. It is explicitly stated that it is a bad idea to load Lilu and its plugins from /L/E or /S/L/E, they will randomly fail unless you use apple id & security module flag. Be sure not to do that.

i am using clover kext injection for lilu and this plugin, but even that appears to be unreliable.
This is the first plugin this kind of issue occurs with though


Add recurse import test

Currently import test files are hardcoded. In-need to find all python files in recurse way and add them to test.


BREAKING: Remove virtual on methods that are being called from constructors

Continuation of fixes with virtual calls being made from constructors. The issue originally reported by SonarCloud scans: https://sonarcloud.io/project/issues?resolved=false&rules=csharpsquid%3AS1699&id=apache_lucenenet and referenced in this issue: https://github.com/apache/lucenenet/issues/670
This time I focused on cases where a class is public, we are breaking the API, BUT, the method that was marked virtual was not calling any other virtual methods, was not being overridden in Lucene repo, and/or looked like a method that was not meant to be overridden to change the behavior of the class. Similarly to #798 , preferring explicit/clear/no surprises break in API over introducing locking/lazy initialization/additional allocations when the public consumers have a way to safely extend the class behavior.
This should eliminate the majority of "easy" changes, leaving us with a few "hairer" cases that we will tackle separately.

Taking a look at the failures, somehow locally I did not trigger them...

Thank you for the review. I went ahead and addressed the issues with separate PRs. I left out DirectoryTaxonomyReader.cs and will take it in another PR. Closing this one without merging.


sometimes SeekBar is not showing in the Notification

Hi
I'm currently writing a Music Player that streams the music from the network
The issue I am facing is sometimes seekBar is not showing in the Notification
is there a solution to set it in Service? or It's a bug?
also, MediaMetadata.Builder.setArtworkUri seems not working. I had to set NotificationBuilder.setLargeIcon manually to show artwork.

A media needs to be seekable. Can you check whether the media that doesn't show the seekbar is seekable? Alternatively please provide a media URI for such a stream.

Yes, It's seekable. the problem is sometimes randomly seekbar not showing but the solution is to pause the Player and then it shows up
the same song playing that seekbar was hidden before:

This is such a great API that you guys developing, Thanks.

MediaMetadata.Builder.setArtworkUri is still not working in the 1.0.0-beta01

MediaMetadata.Builder.setArtworkUri is still not working in the 1.0.0-beta01

Yes, it's not working for me either
Unfortunately, in 1.0.0-beta01 after seeking another media item, the title and artist do not change to new MediaMetadata values. The manual solution, as I earlier pointed is a pause and then play again.
SeekBar does not show the first time, but after pausing and resuming after. It's visible and works like a charm.

@Kyant0
The session demo app is using MediaMetadata.Builder.setArtworkUri and I am pretty sure I was looking at notifications that display a background image.
Can you check whether this works for you with the session demo app, then check whether you do the same with your app? If you still see that this is not working, please open a different issue that is about that specific problem and explain in a bit more detail what you are doing when building the media item and setting it to the player.

@graymind75 I migrated from Service to MediaLibraryService, it worked! Splendid!


Fix dragging color picker sliders

Summary
It looks like the issue for the slider not working was that the pointer was positioned below the slider area, and the dragging is being listened in the slider area an not on the pointer.
This PR adds an additional wrapper around the slider area and assigns padding for not allowing the pointer to go out of the slider area.

Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

Add a Text
Click on the color picker for the text
Start dragging the Hue and the Opacity sliders in the color picker
Verify that they work right away when dragging.

Fixes #1901

@barklund This is not ready for review yet, testing only, asked it without intention, sorry!


Avoid heap allocation on MacOS

This forgoes allocation in favour of stack allocation - and also avoids a leak if the call to proc_pidinfo fails for some reason.
I don't have a Mac to test on so I don't know if it even compiles, but this certainly seems a better approach than heap allocating.

I really missed a trick by not calling the branch macos-the-mallocless :(

If anyone asks, I got this right first time.

Thanks!


Support pinned memory pooling

I read your article at https://pingcap.com/blog/use-gpu-boosts-tidb-analytics-performance-by-10-to-150-times
Really great work.
I work on a related project https://github.com/NVIDIA/spark-rapids/
In the article you mentioned that you cannot get above 3 GiB/s PCIe transfers.  This is probably because you are not using pinned memory for the transfers. For the java API we added in a pinned memory pool similar to RMM. It is not a great implementation and https://github.com/rapidsai/rmm/issues/618 is to have RMM also handle pinned memory at some point. The pinned pool lets us get about 12-13 GiB/sec on PCIe gen 3.
Be aware that it is rather expensive to allocate the pinned memory, which is why we pool it. If you really have a 1:1 ratio for data transfer to CPU compute, this could be a really big win.

Hi Bobby, thank you for the comments. That's very helpful!
Yes I believe a pinned memory pool could definitely be a big win. I'll keep watching the progress of rmm's implementation.
Besides what you've mentioned about the expensiveness of pinned memory allocation, a pool is also significant for our pipeline model. We intend to achieve higher GPU utilization by concurrently executing multiple CPU threads, each executes a pipelined cuDF calls. IIRC, a raw cudaMalloc or cudaFree call will force sync the device thus brutally kill the potential parallelism among the GPU jobs submitted by these concurrent CPU threads.

Having multiple threads helps, especially for larger devices that have enough memory and compute to do everything multiple things at once. I am not sure how deep you can get your pipelined calls to be. For a lot of operations in cuDF (like aggregations and joins) the output size is not know ahead of time. This means that they have to do a calculation on the GPU and then pull back a result to the CPU, allocate memory on the GPU and finally compute the answer.  This can result in a lot of synchronization.  This is one of the main reasons we put in per thread default stream so each thread is not stuck waiting on the other threads. Also in CUDA an asyncMemCopy is not actually async unless both sides are either on the GPU or against pinned memory.  If you use non pinned memory the CUDA driver will use a small pinned bounce buffer to do the transfer (because the GPU can only access pinned memory directly). This is why the transfers are slower when not using pinned memory, but it also means that it is harder to get a deep pipeline, because the thread cannot start to issue other calls while doing the transfer.
I think there is a lot of overlap with what you are trying to do here and what we have done with our plugin to Apache Spark. Let me know if you have any questions or just want to meet up some time to talk.

I am not sure how deep you can get your pipelined calls to be.

Since our pipeline model will aggressively pipe as many cuDF calls as possible, a pipeline could be very deep. Consider an N-way join, the optimizer may decide to compute N-1 hash tables and do a chained probe to each of them. The pipeline executing this chained probe will have a chained N-1 calls to cuDF's hash table probe function.

This can result in a lot of synchronization. This is one of the main reasons we put in per thread default stream so each thread is not stuck waiting on the other threads.

Yes we observed these lot of synchronizations as well, and we used ptds too. Our story is, when we firstly enabled ptds, we saw that the GPU tasks from different CPU threads were decoupled as expected. But we were still using cuda_memory_resource back then, so the device sync introduced by raw cudaMalloc and cudaFree APIs were still in the way. It only got better after we switched to other pooled memory resources. I just want to be sure that both ptds and a pooled memory resource are critical to achieve the utilization level we are expecting.

Also in CUDA an asyncMemCopy is not actually async unless both sides are either on the GPU or against pinned memory.

I mean, WOW, this might be the answer to the question that we haven't figured out. If I understand correctly, a copy to/from an unpinned memory still requires CPU's involvement to move it between the "bounce buffer" and the real host memory? We expect CPU to go forward immediately after making an asyncMemCopy call but it somehow gets busy with managing the implicit "bound buffer"? If that's the case, it probably explains why we weren't getting high enough GPU utilization.

I think there is a lot of overlap with what you are trying to do here and what we have done with our plugin to Apache Spark. Let me know if you have any questions or just want to meet up some time to talk.

Yes I can imagine that we have a lot of work in common and you must have dealt with many problems that I haven't even met. Your help will be really helpful. Thanks a lot, Bobby! I appreciate that!

If I understand correctly, a copy to/from an unpinned memory still requires CPU's involvement to move it between the "bounce buffer" and the real host memory?

Yes that is 100% correct.
The documentation is not clear about this at all, and it might not be that way for all systems, but from what we have tested it is the case for x86 servers running Linux.


Unity 6 support

I tested Unity 6 and found that recording doesn't work in the editor, and the microphone isn't recognized on the web either.

Need to open https server for connecting on the web.


Cookbook ideas for reqwest crate

Come up with ideas for nice introductory examples of using reqwest, possibly in combination with other crates, that would be good to show in the Rust Cookbook. Please leave a comment here with your ideas! You don't necessarily have to write the example code yourself but PRs are always welcome!

[x] Make an HTTP GET request (reqwest)
[x] Download a file to a temporary directory (reqwest + tempdir)
[x] Query the GitHub API (reqwest + serde)
[x] Create and delete Gist with GitHub API (reqwest + serde)
[ ] ???

Crate evaluation thread on internals.rust-lang.org

In addition to already created issues i would suggest:

[ ] using HEAD request to test endpoint/file existence (or just check headders)
[ ] use PUT / PATCH requests (GitHub has some apis like that) although it might be a little redundant
[ ] test if links on webpage are stale - html5ever or select

I guess that following might be out of scope and a little redundant:
Possibly just show some interesting web apis (twitter) but most of these would be to involved due to oAuth (or require very high level crate oauth-client  )

[ ] Interact with twitter API - make a tweet
[ ] google translate
[ ] google geocoder / reverse geocoder
[ ] stack overflow

I am open to showing a variety of common interesting web APIs. OAuth is a little more involved than I would like for the cookbook. I would prefer to handle it by linking from the basic-auth GitHub example to the oauth-client documentation: #94.

Make a partial download with range headers

This is a bit of a more advanced idea, but I recently wrote a wrapper around the github API for dealing with pagination by giving you an iterator which automatically requests the next page and deserializes it using serde.
Would something like that be a useful cookbook recipe, or is it too much of a niche problem?

Hi @Michael-F-Bryan. Thanks for taking an interest!
We already have an example showing paginated response and we are trying to avoid pulling non LibzBlitz crates at the moment but you are welcome to submit your crate for inclusion in the official thread or contribute in any way such as:

to fix the existing example
implement new examples
propose new examples for any crate
improve cookbook in any way (including ui/ux)

We'd love to have you on board!

Ah, looks like you guys are one step ahead of me! I was thinking you might be able to copy the guts out of my Pagination struct and adapt it to use reqwest instead of the github-rs client (which itself is just a thin wrapper around hyper). But you probably wouldn't gain much by altering the existing example to take inspiration from what I did other than showing how it's almost trivial to make the Pagination wrapper generic over anything which is deserializable using serde.

I like the pagination idea.
may I also suggest the different authentication methods as cookbook examples for reqwest?

[ ] Basic authentication
[ ] Basic hidden authentication
[ ] Digest authentication
[ ] Digest authentication with algorithm

we can use httpbin for those, as in this one for digest:
curl -vv --digest -k -u "user:pass"  "https://httpbin.org/digest-auth/auth/user/pass/MD5

Created remaining Reqwest recipes, closing tracking issue.


Feature request: hide reset button

Please add a property hideResetButton to hide the reset button:

Thanks

In the newest version 2.9.1 this property is included.


Godot from master branch don't build in Gentoo

Godot version:

ba51af7904819b3739173d24567e79ee5d3600f9
OS/device including version:

Gentoo Linux AMD64
Issue description:

[Initial build] Compiling ==> platform/linuxbsd/display_server_x11.cpp
platform/linuxbsd/display_server_x11.cpp:809:2: warning: #warning Need to get from proper window [-Wcpp]
  809 | #warning Need to get from proper window
      |  ^~~~~~~
platform/linuxbsd/display_server_x11.cpp:3990:2: warning: #warning Forcing vulkan rendering driver because OpenGL not implemented yet [-Wcpp]
 3990 | #warning Forcing vulkan rendering driver because OpenGL not implemented yet
      |  ^~~~~~~
platform/linuxbsd/display_server_x11.cpp: In member function 'String DisplayServerX11::_clipboard_get_impl(Atom, Window, Atom) const':
platform/linuxbsd/display_server_x11.cpp:541:11: error: 'LONG_MAX' was not declared in this scope
  541 |        0, LONG_MAX, // offset - len
      |           ^~~~~~~~
platform/linuxbsd/display_server_x11.cpp:76:1: note: 'LONG_MAX' is defined in header '<climits>'; did you forget to '#include <climits>'?
   75 | #include <X11/XKBlib.h>
  +++ |+#include <climits>
   76 |
platform/linuxbsd/display_server_x11.cpp: In member function 'void DisplayServerX11::_handle_selection_request_event(XSelectionRequestEvent*) const':
platform/linuxbsd/display_server_x11.cpp:2501:81: error: 'LONG_MAX' was not declared in this scope
 2501 |   if (XGetWindowProperty(x11_display, p_event->requestor, p_event->property, 0, LONG_MAX, False, atom_pair, &type, &format, &len, &remaining, &data) == Success) {
      |                                                                                ^~~~~~~~
platform/linuxbsd/display_server_x11.cpp:2501:81: note: 'LONG_MAX' is defined in header '<climits>'; did you forget to '#include <climits>'?
scons: *** [platform/linuxbsd/display_server_x11.linuxbsd.opt.tools.64.o] Error 1
scons: building terminated because of errors.

Steps to reproduce:

Checkout latest sources.
Build then on linux with scons platform=linuxbsd target=release_debug

Also happens in Ubuntu 20.10
@  cpp/godot   master  scons platform=x11
scons: Reading SConscript files ...
Platform "x11" has been renamed to "linuxbsd" in Godot 4.0. Building for platform "linuxbsd".
Enabling ALSA
PulseAudio development libraries not found, disabling driver
Checking for C header file mntent.h... (cached) yes
scons: done reading SConscript files.
scons: Building targets ...
[  2%] Compiling ==> platform/linuxbsd/display_server_x11.cpp
platform/linuxbsd/display_server_x11.cpp:809:2: warning: #warning Need to get from proper window [-Wcpp]
809 | #warning Need to get from proper window
    |  ^~~~~~~
platform/linuxbsd/display_server_x11.cpp:3990:2: warning: #warning Forcing vulkan rendering driver because OpenGL not implemented yet [-Wcpp]
3990 | #warning Forcing vulkan rendering driver because OpenGL not implemented yet
    |  ^~~~~~~
platform/linuxbsd/display_server_x11.cpp: In member function 'String DisplayServerX11::_clipboard_get_impl(Atom, Window, Atom) const':
platform/linuxbsd/display_server_x11.cpp:541:11: error: 'LONG_MAX' was not declared in this scope
541 |        0, LONG_MAX, // offset - len
    |           ^~~~~~~~
platform/linuxbsd/display_server_x11.cpp:76:1: note: 'LONG_MAX' is defined in header '<climits>'; did you forget to '#include <climits>'?
75 | #include <X11/XKBlib.h>
+++ |+#include <climits>
76 |
platform/linuxbsd/display_server_x11.cpp: In member function 'void DisplayServerX11::_handle_selection_request_event(XSelectionRequestEvent*) const':
platform/linuxbsd/display_server_x11.cpp:2501:81: error: 'LONG_MAX' was not declared in this scope
2501 |   if (XGetWindowProperty(x11_display, p_event->requestor, p_event->property, 0, LONG_MAX, False, atom_pair, &type, &format, &len, &remaining, &data) == Success) {
    |                                                                                ^~~~~~~~
platform/linuxbsd/display_server_x11.cpp:2501:81: note: 'LONG_MAX' is defined in header '<climits>'; did you forget to '#include <climits>'?
scons: *** [platform/linuxbsd/display_server_x11.linuxbsd.tools.64.o] Error 1
scons: building terminated because of errors.


[README] use GH url for the demo image

it is recommended to use GH raw urls, or use any other hosting service (e.g. GH pages.) if the REAMDE.md is displayed on other sites (like npm) the relative links might be broken.

Well, there are other opinions on that, too.

Thanks!


Various fix for finder's prompt

This PR includes three commits, explained here.
1. Change the default prompt_prefix value
The default value of the prompt_prefix is an indicator " > ":
https://github.com/NeogitOrg/neogit/blob/0d0879b0045fb213c328126969a3317c0963d34a/lua/neogit/lib/finder.lua#L152
But whenever the Finder was created, it added an indicator as a postfix also:
https://github.com/NeogitOrg/neogit/blob/0d0879b0045fb213c328126969a3317c0963d34a/lua/neogit/lib/finder.lua#L182
So the default value of prompt_prefix for users will be " > > ", I change the default value to a more meaningful value in this commit.
2. Remove redundant prompt_prefix postfix
The behavior of adding a postfix for every prompt_prefix has problems too. Because neogit do it whenever the Finder is created:
https://github.com/NeogitOrg/neogit/blob/0d0879b0045fb213c328126969a3317c0963d34a/lua/neogit/lib/finder.lua#L182
And when the finder is not telescope.nvim, it will add the postfix again:
https://github.com/NeogitOrg/neogit/blob/0d0879b0045fb213c328126969a3317c0963d34a/lua/neogit/lib/finder.lua#L224
https://github.com/NeogitOrg/neogit/blob/0d0879b0045fb213c328126969a3317c0963d34a/lua/neogit/lib/finder.lua#L233
So this behavior is redundant in this situation. There are two ways to fix this indeed,

Do not add the postfix whenever the Finder is created;
Do not add the postfix when the Finder is not telescope.nvim also.

I selected the second way to complete the third commit.
3. Modify postfix for finders
For telescope.nvim, maybe the prefix space is needed, but for fzf-lua, it is redundant, so I removed it, and for vim.ui.select, even though there's no standard to specify what the prompt postfix is, most plugins select a colon. Most importantly, the popular plugin dressing.nvim which is dedicated to customizing vim.ui.* will trim if the end is a colon in some situations to be more beautiful.

Nice.


Update node packages

Motivation and Context
Update dependencies
What has changed
Updated the non-breaking change dependencies.  The changes address both dependabot PRs.
How to test?
All tests should pass and everything should work as normal
Links
https://github.com/ONSdigital/ssdc-rm-support-tool/pull/250
https://github.com/ONSdigital/ssdc-rm-support-tool/pull/251
https://trello.com/c/v77XGnEP

 More dependabot PRs need to be looked at


Transfer this repo to dwyl org?

@iteles is it worth transferring this repo to DWYL? (we will be using it for Hansel ...)

If it's not relevant to any of your personal projects and you don't feel you need it to form part of your individual reputation, then it makes sense :blush:


Viruses were found in 2.00 version

I downloaded the w64devkit-x64-2.0.0.exe file from https://github.com/skeeto/w64devkit/releases/download/v2.0.0/w64devkit-x64-2.0.0.exe and ran Avast Antivirus, which found 2 viruses.

There are no viruses. These are false positives. See #154, #158, #167, #176.

@cpgitgig135 You should probably get rid of Avast anyway...
https://www.theverge.com/2024/2/22/24080135/avast-security-privacy-software-ftc-fine-data-harvesting


Wordcloud Generator not working

In the section Wordcloud Generator there is a link to an application that should generate wordclouds. When clicking on it, the page just displays "Not found"

Hi @mathiashoeld, I haven't had time to update that project yet, but thanks for the flag :) Will comment and close the issue once it's fully updated!

Hey @mathiashoeld, I spent a week building a fun/lightweight version of this.  Feel free to check it out here: https://wordcloud-generator.netlify.app/#


workflow to run on pull request

specify when workflow runs (push to master, pull request)

Working with Derek to make PR merge depend on successful workflow

Implemented in https://github.com/pravega/presto-connector/pull/50


Airflow webserver is slow

This is probably something weird with my environment setup, but I'm opening this in case anyone has any ideas as to what might cause this.
Recently (like, since yesterday afternoon) the airflow webserver has been very, very slow. This persists no matter how many times I restart. It's particularly slow when visited via an AWS load balancer, moderately slow over an SSH port forward, and also very slow via local curl.
System is not under particularly high load, is able to contact the database server, the database server is responding promptly.
Running on redhat 7.
I'd add more details if I knew what was germane.

Hello,
I have a similar experience after upgrading to 1.5.2. I downgraded back to 1.5.1 for the moment to fix the issue. My guess would be it is somehow related to one of the following changes:

MySQL uses mysqlclient lib instead of mysql-python
Using gunicorn instead of tornado as the wsgi web server

@pascalknapen That's awesome. I didn't realise either of those changes happened. Do you have any plans to try out reverting either of those changes?

mysqlclient is a fork of mysql-python so I would not necessarily expect a huge regression there.
Quick question, are you running a single webserver? (We run several of them behind a load balancer which means that regressions for a single server may be harder for us to detect, and 4 workers (= processes) as per the default configuration).
Note that the debug server runs on a single thread, and so might be slower as well. Usually one recommends 2N+1 gunicorn workers for N vCPUs. I have a PR out that exposes the ability to use multithreaded workers. Maybe playing with those options might help?

@artwr I just run the server with airflow webserver. This preforks 4 workers and one master process. We do use a single server on a single host. It's very slow even with only one person accessing it. It's much faster when accessed directly over an SSH tunnel than when accessed over an AWS load balancer.

@mtustin-handy May I ask why you would use a load balancer if it runs on a single node? It seems that this setup would just increase latency, significantly so if the load balancer ends up being in a different availability zone?
If you have chrome, can you take a look at the developer tools on the Network tab when you load the server URL, and post a screenshot of what you see there?

@artwr Like Amazon itself, we use loadbalancers to make webservers available across network security/availability zones.

@mtustin-handy @artwr This seems indeed to be related to the interplay between gunicorn and ELB. This article explains it a bit: https://forums.aws.amazon.com/thread.jspa?messageID=419138. As the reply suggests, using async workers might fix the problem. I'll try to find out tomorrow.

Another short term solution might be to bump the number of workers erronously named threads in our current config.

Made the changes to support both sync and async workers in gunicorn. Just
finished testing it and working great now behind ELB. Will send a pull
request later today.
On Tue, Nov 10, 2015 at 10:51 PM, Arthur Wiedmer<EMAIL_ADDRESS>wrote:

Another short term solution might be to bump the number of workers
erronously named threads in our current config.
—
Reply to this email directly or view it on GitHub
https://github.com/airbnb/airflow/issues/531#issuecomment-155578019.

That is so awesome. I've been testing out the new scheduling stuff that
@mistercrunch merged yesterday.

--
In the news https://www.handy.com/press:
The Hottest Startups of 2014 (Forbes
http://www.forbes.com/sites/briansolomon/2014/12/17/the-hottest-startups-of-2014/
)
Handy Hits $1 Million A Week In Bookings (TechCrunch
http://techcrunch.com/2014/10/14/handy-hits-1-million-a-week-in-bookings-as-cleaning-economy-consolidates/
)
Handy Raises $50 Million in Series C (Handy Blog
http://blog.handy.com/50m/) http://blog.handy.com/50m/

@pascalknapen : Does your PR look anything like https://github.com/airbnb/airflow/pull/610 ?

Closed by #618. Feel free to reopen if needed.


Add StreetComplete icon

Description
Added icon for StreetComplete.
Type of change

:x: Bug fix (non-breaking change which fixes an issue)
:white_check_mark: Icon addition (non-breaking change that adds/modifies Lawnicons's icons)
:x: General change (non-breaking change that doesn't fit the above categories like copyediting)

Icons addition information
Icons added

StreetComplete (de.westnordost.streetcomplete)

The icon wasn't quite at its 160px x 160px border so I just made I minor tweak here that you could just swap out the file with: streetcomplete.zip. Mind also updating the drawable file (streetcomplete.xml)? Other than that great job!


Stars Without Number: Initiative not using correct die

From a user:
"Rolling for initiative off of the character sheet uses 1d10+dex, but the rulebook (at least the free version from DriveThruRPG) says it's 1d8+dex."

I'll fold this into my PR #982, it's def. wrong (I've been playing DH which is 1d10-based).


fix: now list activity graphs only load data client-side

Description
Now only list info is loaded on the list activity page. All graph data is pulled on the client-side only now.
Here's a public list loading data for graphs client-side only now, https://deploy-preview-2215--oss-insights.netlify.app/lists/8b371419-6219-4448-9de1-f6293a25f98e/activity?range=30
What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Fixes #2213
Mobile & Desktop Screenshots/Recordings
https://github.com/open-sauced/app/assets/833231/75e7e0ae-3c7f-47f1-bff0-98b9743d06f8
Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[x] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

I confirmed with @isabensusan that for her 200 contributors list, it's all good now.


Some grammar improvements in scopes-oidc.md

Learn Build status updates of commit 8486831:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/develop/scopes-oidc.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Looks good; thanks, @noamtd


Florina has a room with three stacked two-wide doors

Unsure if there are any other doors like this or if this was a glitch, but I noticed those specifically

this got fixed I believe


feat(tests): adds test management

Adds neotest setup based on my very early understanding of neotest.
We can then add test adapters in the following way.
return {
  {
    "nvim-neotest/neotest",
    dependencies = {
      "nvim-neotest/neotest-go",
    },
    opts = function(_, opts)
      -- Would like "cleaner" way than this..
      opts.adapters = vim.list_extend(opts.adapters or {}, {
        require("neotest-go")({ args = { "-tags=integration" } }),
      })
    end,
  },
}

Can we somehow only add the "Debug Nearest" keybind if nvim-dap exists?

FixCursorHold may need to be added as a dependency even though part of its functionality has already been merged into neovim:

Neotest uses the CursorHold event. This uses the updatetime setting which is by default very high, and lowering this can lead to excessive writes to disk. It's recommended to use https://github.com/antoinemadec/FixCursorHold.nvim which allows detaching updatetime from the frequency of the CursorHold event. The repo claims it is no longer needed but it is still recommended (See this issue)

antoinemadec/FixCursorHold.nvim may need to be added as a dependency even though part of its functionality has already been merged into neovim:

Neotest uses the CursorHold event. This uses the updatetime setting which is by default very high, and lowering this can lead to excessive writes to disk. It's recommended to use https://github.com/antoinemadec/FixCursorHold.nvim which allows detaching updatetime from the frequency of the CursorHold event. The repo claims it is no longer needed but it is still recommended (See this issue)

I am unsure of the impact - decided to not add it as I do not understand it fully.
It should be easy to add from userland. But I am definitely not against adding it either.
Just need to be told what is "correct" 😄

@folke What do you think about this? Do you prefer me opening an Idea in discussions?

I would love to have this tbh! Something else i would add to the default configuration is neotest as it's a best effort plugin to run test automatically. see my config for neotest https://github.com/kevinrobayna/dotfiles/blob/c699b32f7493e0caf44dc94db738447fd41b03f9/config/.config/nvim/lua/plugins/neotest.lua
				require("neotest-vim-test")({
					ignore_file_types = { "ruby", "go" },
				})

I would love to have this tbh! Something else i would add to the default configuration is neotest as it's a best effort plugin to run test automatically. see my config for neotest https://github.com/kevinrobayna/dotfiles/blob/c699b32f7493e0caf44dc94db738447fd41b03f9/config/.config/nvim/lua/plugins/neotest.lua
require("neotest-vim-test")({
  ignore_file_types = { "ruby", "go" },
})

Although i think is also makes sense to leave it to the consumer so that they do something like this
  {
    "nvim-neotest/neotest",
    dependencies = {
      "nvim-neotest/neotest-vim-test",
    },
    opts = function(_, opts)
      opts.adapters = vim.list_extend(opts.adapters or {}, {
        require("neotest-vim-test")({
          ignore_file_types = { "ruby", "go" },
        }),
      })
    end,
  },

I would personally prefer not to have neotest-vim-test as a dependency, it seems like that is only there because there is not a specific adapter for all languages yet, so this work as backwards compatible bridge to vim-test.
I am not sure how this relates to auto running tests - unless vim-test does that?
But if I understand correctly, you can auto run tests using neotest with an auto command.
Not sure if we want to include that in a default configuration either though?

Sorry for the delay. Will have a look into this tomorrow!

I would love to have this tbh! Something else i would add to the default configuration is neotest as it's a best effort plugin to run test automatically. see my config for neotest https://github.com/kevinrobayna/dotfiles/blob/c699b32f7493e0caf44dc94db738447fd41b03f9/config/.config/nvim/lua/plugins/neotest.lua

require("neotest-vim-test")({

ignore_file_types = { "ruby", "go" },

})

Although i think is also makes sense to leave it to the consumer so that they do something like this

{

"nvim-neotest/neotest",

dependencies = {

  "nvim-neotest/neotest-vim-test",

},

opts = function(_, opts)

  opts.adapters = vim.list_extend(opts.adapters or {}, {

    require("neotest-vim-test")({

      ignore_file_types = { "ruby", "go" },

    }),

  })

end,

},

I would personally prefer not to have neotest-vim-test as a dependency, it seems like that is only there because there is not a specific adapter for all languages yet, so this work as backwards compatible bridge to vim-test.
I am not sure how this relates to auto running tests - unless vim-test does that?
But if I understand correctly, you can auto run tests using neotest with an auto command.
Not sure if we want to include that in a default configuration either though?

Yeah I think the default configuration should be as minimal as possible to be honest

I recently expedited some effort in setting up neotest with language plugins broken up into language layers:

Core config with ships with neotest-vim-test: https://github.com/ditsuke/nvim-config/blob/main/lua/ditsuke/extras/code/neotest.lua

Why do you want neotest-vim-test? It seems like a "catch-all" / "bloated" package.
We want to stay minimal, so we should really only have language specific adapters. neotest-vim-test seems easy to add in userland anyway?

I recently expedited some effort in setting up neotest with language plugins broken up into language layers:

Core config with ships with neotest-vim-test: https://github.com/ditsuke/nvim-config/blob/main/lua/ditsuke/extras/code/neotest.lua

Why do you want neotest-vim-test? It seems like a "catch-all" / "bloated" package.
We want to stay minimal, so we should really only have language specific adapters. neotest-vim-test seems easy to add in userland anyway?

I'm fine either way wrt vim-test. My concern is having a clean way to enable neotest adaptors for imported language layers.

@folke Did you have time to give feedback here? 😄

Let me know when it's ready to merge

Let me know when it's ready to merge

Just copied into my personal config and it works as expected.
Unsure if there is a good way to test other than that.. 😄
Do we need any documentation / examples on how to configure a test adapter/plugin?
I only have neotest-go as a working example currently.
  {
    "nvim-neotest/neotest",
    dependencies = {
      "nvim-neotest/neotest-go",
    },
    opts = function(_, opts)
      opts.adapters = vim.list_extend(opts.adapters or {}, {
        require("neotest-go")({
          args = { "-tags=integration" },
        }),
      })
    end,
  },

Looks good. I don't really use neotest myself, but may give it a try :)

Merged. Thanks!


feat: support IP limitation for user entry pages

fix: https://github.com/casdoor/casdoor/issues/3254

@dacongda plz review


Search result by Grouped by Section Title

Hello @gwleclerc
You look like a JS killer
Are you able to  implement something like the search result pane in the new gohugo website ?

I can try :)

If you need help, more data in the search results, tell me !


Seems that the cypress tests does not work correctly

Hey, I just made a Pull Request!
It seems that when the upgrade to cypress 10 were made the config files were not updated
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hi! @pbnPierre seems there was a issue with the changeset bot. Can you rebase from master so that the bot will spamming your PR? 🙏

Yes I will thanks

It seems good now, thanks! @pbnPierre

Seems my linter issue also

Not really sure what i should put in the changeset to be honnest

Thanks for the PR!
I was not aware that we had cypress tests being broken, where/how are they failing currently?

I think they even are note launched because it did not failed. But Cypress > 10.x does not support cypress.json configuration files anymore. We have tu use a cypress.config.ts
I saw this here
https://docs.cypress.io/guides/references/configuration
seems that there is one also on the root of the folder i forgot.

Seems that the integrationFolder is legacy configuration also.
I'm wondering now are they actually runned ?

I tried to update also the cypress/cypress.json also let's see what it does

If i'm not mistaken, I dont see the command "test:cypress": "cypress open" used in CI

Oh there are three more
./packages/techdocs-cli-embedded-app/cypress.json
./packages/techdocs-cli/cypress.json
./packages/create-app/templates/default-app/packages/app/cypress.json```

Seems that someone is also on it https://github.com/backstage/backstage/pull/15368

./packages/create-app/templates/default-app/packages/app/cypress.json this one is in a template with cypress 9.7 so seems ok

the last one is more touchy ./packages/techdocs-cli/cypress.json

@jhaals could i have some advice to what i should do with this ? Do you have inputs
I see some cypress refrences in github actions yaml
      - name: cypress run
        uses: cypress-io/github-action@v2
        env:
          CYPRESS_baseUrl: ${{steps.get-status-env.outputs.result}}
        with:
          config-file: ./cypress.json
          working-directory: ./cypress
          browser: chrome
          install: false
          headless: true

but nor reat cy:run executed elsewhere and the config-file seems not good though

I don't think cypress is used any longer for app(packages/app). The only references I can find only point to https://github.com/backstage/backstage/blob/master/.github/workflows/verify_e2e-tugboat.yml which is no longer in use and should probably be removed. I know @benjdlambert worked on this previously and might be able to speak to the usage of the cypress today

I can remove all trace of cypress everywhere if you want in this PR.

Seems a lot of work though. I can leave it to you also. Shame i could not officially contritube to backstage using my professionnal mail @deezer.com :D

There's clearly some overlap with https://github.com/backstage/backstage/pull/15368 currently. I'd suggest we make sure that we separate the techdocs related tests from the other cypress tests. I was not able to run the tests from the cypress folder checking out the branch
CYPRESS_baseUrl="http://demo.backstage.io" yarn cypress run                                                                                1s [11:33:32]

Missing baseUrl in compilerOptions. tsconfig-paths will be skipped
There were multiple support files found matching your supportFile pattern.

Your supportFile is set to: ./src/support

We found the following files:

 - /Users/jhaals/src/backstage/cypress/src/support/commands.ts
 - /Users/jhaals/src/backstage/cypress/src/support/index.ts

Please remove or combine the support files into a single file.


How does network.stricttransportsecurity.preloadlist update and deploy?

This is just for information, sorry opening an issue when it is not, but I've spent hours searching for an answer on the Web, unsuccessfully.
Concerns:
user_pref("network.stricttransportsecurity.preloadlist", true);  // Default=true
I've had this set to false since always. I've set it to true and am faced to two misunderstandings.
1- When, how is this preloadlist updated?
2- Where is it located?
I just cannot find any file in my profile, be it new, be it modified.
I am wondering if this setting, to be functional (to have it downloaded) would depend on another setting that I would have disabled., i.e. one of the numerous safebrowsing settings found in about:config and more specifically a whatever url that I would have set to "" or <IP_ADDRESS>
When this preload list is in place, how do I know it, how do I access it? Is it automatically merged to my SiteSecurityServiceState.txt file? If so that file shows nothing but the visited sites' HSTS ...
I cannot conceptualize the whole process!
Thanks.

Bonjour Tom, the list is compiled into FF from the file security/manager/ssl/nsSTSPreloadList.inc on DXR. I'm not sure it ever gets updated between releases but if so then it's probably done with services.blocklist.pinning.* but I think that's only used for HPKP updates. Atm the update data for that is empty.
You can find more information about services.blocklist.pinning if you search for it here, ie the bugzilla, where it gets the updates from, how to disable it etc. I guess if there's ever any data to be updated it would likely get stored in SecurityPreloadState.txt, judging by the name and the fact that the file is empty atm.

Bonjour @earthlng , thanks. So it appears I had totally misunderstood the process -- even after havig read all I could find about the setting -- as I believed the file, a file, was regularly updated with latest domains having been accepted for https : hence, I was searching for it everywhere. I did know that it was included in Firefox builds, I thought that basis was hardcoded and then updated regularly ...
OK, that's clear now. Before opening this thread I searched here but found no immediate topic related to mine which is why I opened a new "issue".
I'll close the door. Have a nice, sunny week-end!

Have a nice, sunny week-end!

Thanks! You too buddy

Thanks! You too buddy
Please : mate, dude, pal ... but not buddy

Pants wasn't fond of "just fine" and zymase isn't of "buddy" :)

Lol, okay, I realized it sounded kinda bad but it was already too late xD
Buddy is more fitting for dogs but dude to be fair your profile pic is a dog with what I assume are your eyes and mouth - what do you expect :)
btw you don't need to re-open an issue to comment again

I know some people actually talk to dogs but this is the first time I discover someone stating he writes to them!

And guess what - they answer too xD
All I'm saying is that your avatar must have subconsciously influenced me to think that "buddy" is an appropriate word to use. I usually never use that word. DAMN I'm easily influenced - bad earthlng, bad ;)

An avatar doesn't answer :)
In fact I was talking about avatars and she proposed (she's an artist) to imagine something original given "tu as du chien". In French "avoir du chien" mean "to "to be sexy". She was playing on words as you did, but I guess her subconscious had been quite differently fed.
Computing leads to everything nowadays.


Broken trailing comment tags,

Challenge Uncomment HTML has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.118 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

Hello World

CatPhotoApp

Hello Paragraph

@Chiemychanga There is still one test left to pass, unless I am missing something?
Please see the test suite under the Run test button

@Bouncey The test checks for the absence of -->. But since the OP had comments all over, the tests fail.
I think we should take care of that. What do you think?

Now I understand.
cc @FreeCodeCamp/issue-moderators Is the test for trailing comments redundant/unnecessary?

I think the test is necessary because without that test, a camper can get through with code like this.

Hello World

CatPhotoApp

Hello Paragraph

-->

The first three tests would still pass. So, the test which checks for trailing comments is not redundant and actually needed.


Completely handle block asynchronously

Description

This PR implements a txs module to store txs asynchronously to follow up mainnet network easily.
Checklist

[x] Targeted PR against correct branch.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit tests.
[ ] Re-reviewed Files changed in the Github PR explorer.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@234d65b). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head ab3a346 differs from pull request most recent head 266d60d. Consider uploading reports for the commit 266d60d to get more accurate results

@@            Coverage Diff            @@
##             master     #110   +/-   ##
=========================================
  Coverage          ?   68.31%
=========================================
  Files             ?       29
  Lines             ?     1256
  Branches          ?        0
=========================================
  Hits              ?      858
  Misses            ?      370
  Partials          ?       28

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 234d65b...266d60d. Read the comment docs.


API key is null

Getting this error for the demo:
/Users/jozefw/Desktop/chatbase-codelab-io-2017-master/node_modules/@google/chatbase/lib/MessageFactory.js:118
newMessage (apiKey=null, userId=null) {
^
SyntaxError: Unexpected token =
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:414:25)
at Object.Module._extensions..js (module.js:442:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:311:12)
at Module.require (module.js:366:17)
at require (module.js:385:17)
at Object. (/Users/jozefw/Desktop/chatbase-codelab-io-2017-master/node_modules/@google/chatbase/lib/ApplicationInterface.js:19:24)
at Module._compile (module.js:435:26)
at Object.Module._extensions..js (module.js:442:10)

Hey @Jozefw please make sure you are using Node V6+ while using this library. Thanks for pointing this out though -- will update the codelab and package.json accordingly.
To see which version you have currently installed you can use the following command:
node -v
If this doesn't help please lmk.
Thanks,
Cris

Engine field has been updated in e63242d80477170f708d7addd1df3d8c73d097fc

Chris,
thanks for your speedy reply.
Took some time to get the most recent install of node in place as node 4.1 kept
persisting.  I installed the node pkg and it went well.
will see if this works tomorrow.
Thanks again,
btw, I clicked on your link in github that took me to the youtube page “this goes up to 11”
NICE!
Jozef

On Sep 17, 2017, at 5:12 PM, Cristian Cavalli<EMAIL_ADDRESS>wrote:
Engine field has been updated in e63242d https://github.com/google/chatbase-node/commit/e63242d80477170f708d7addd1df3d8c73d097fc
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/google/chatbase-node/issues/3#issuecomment-330104276, or mute the thread https://github.com/notifications/unsubscribe-auth/AFueaxXCYrnKgszKoO37V2vHqPdNi_PRks5sjbV2gaJpZM4PaUem.


Bug: Popover

Hey! Thank you for your work on this framework. I have recently started to use into on one of my projects and was very pleased with the documentation and good developer experience.
Since I primarily use Firefox (cause someone's gotta use it) for developing, I noticed that the popover component doesn't work on at least the latest version. It works fine on chrome-based browsers, though is shown all the time on Firefox.
I can't really pin down the reason for the bug, but I will try to look into it.
Cheers,
Cobalt
System

Browser: Mozilla Firefox 78.13.0esr
OS: Parrot OS 4.11 x86_64 / 5.10.0-8parrot1-amd64 / awesomewm (also not working on Debian plain AFAIK and most likely Ubuntu)
kahi<EMAIL_ADDRESS>(also not working on official doc page when opening with Firefox)

Thanks for the bug report! Never used ESR before, but normal Firefox on Manjaro (Arch) is my primary so I try to make sure everything works there.
So I downloaded ESR and looked into it. Seems to be :not support for selector lists wasn't until FF 84+ (2020/12/15). It seems to be breaking multiple things not just Popover. I'll look into working around that with postcss-selector-not to flatten the selectors. Just need to test and make everything works there.

Thank you for the quick response and work on finding the bug. Didn't know that ESR lacked this feature compared to stable Firefox.

Did a quick test with the PostCSS plugin. Worked with most Components, but a couple of the multi-part Components had some issues with specificity.
I could probably take another crack at it by breaking out the selectors like .xyz:not(a, b, ...) -> .xyz:not(a), .xyz:not(b). However that's a bit more manual work than just using a plugin. So will be something I'll take a look at in a later date.
In the mean time, updated the Browser Support documentation to reflect the proper minimums.

Just tested it and it looks a lot better. Thank you for work.


dffml: model: Update the accuracy method from the models

Addresses #732
Signed-off-by: Sudhanshu<EMAIL_ADDRESS>

Nice!