Wamp Client.php read() does not return actual payload

I tried to use the Wamp Websocket Client in order to push a notification from our webserver to the websocket server (I use web-socket-bundle v2.0.0-rc). The $this->read() does not extract the payload properly when connecting to the websocket server.
As a fix I suggest the following solution in the Wamp Client:
  protected function read()
    {
        $stream = $this->socket;

        $stream_meta_data =  stream_get_meta_data($stream);

        $max_length = $stream_meta_data['unread_bytes'];

        $stream_data_string = stream_get_contents($stream, $max_length);

        $startPos = strpos($stream_data_string, '[');

        $endPos = strpos($stream_data_string, ']');

        return substr($stream_data_string, $startPos, $endPos);
    }

Can I make a pull request for this?

Yes, please make a pull request.
On Thu, Dec 19, 2019 at 7:05 AM kasual1<EMAIL_ADDRESS>wrote:

I tried to use the Wamp Websocket Client in order to push a notification
from our webserver to the websocket server (I use web-socket-bundle
v2.0.0-rc). The $this->read() does not extract the payload properly when
connecting to the websocket server.
As a fix I suggest the following solution in the Wamp Client:
protected function read()
{
$stream = $this->socket;
    $stream_meta_data =  stream_get_meta_data($stream);

    $max_length = $stream_meta_data['unread_bytes'];

    $stream_data_string = stream_get_contents($stream, $max_length);

    $startPos = strpos($stream_data_string, '[');

    $endPos = strpos($stream_data_string, ']');

    return substr($stream_data_string, $startPos, $endPos);
}

Can I make a pull request for this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GeniusesOfSymfony/WebSocketPhpClient/issues/2?email_source=notifications&email_token=AACZ7INREMAR77Z7GFOMUPTQZNWRFA5CNFSM4J5FX3Q2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IBULKHA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACZ7ILYWLYIE2MHUSSSQFDQZNWRFANCNFSM4J5FX3QQ
.

--

Michael Please pardon any errors, this message was sent from my iPhone.


The documentation provided here is wrong.

The button onclick event binding seems wrong. It does not seem to accept method groups, but instead wants a method call with the parantheses included.

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

ID: 3de31c7e-7895-9d7c-0d9a-5bdfe83b60a9
Version Independent ID: 334b7206-d383-40c3-b346-b54df9d87293
Content: ASP.NET Core Blazor event handling
Content Source: aspnetcore/blazor/event-handling.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Dup of #18345.


Consider trailingSlash: never

Fixes https://github.com/otoyo/astro-notion-blog/pull/187

When base is not set and trailingSlash is set as 'never', import.meta.env.BASE_URL becomes /.


[FLINK-9742][Table API & SQL] Expose Expression.resultType to public

What is the purpose of the change
This patch proposes exposing Expression.resultType to public, to enable custom TimestampExtractor outside of Flink acessing Expression.resultType.
There is some use cases of TableSource which requires custom implementation of TimestampExtractor, and to ensure new TimestampExtractor to cover more general use cases, accessing Expression.resultType is necessary, but its scope is now defined as package private for "org.apache.flink". It would be better to just make Expression.resultType public to cover other cases as well.
In order to avoid changing public API side, this patch adds new method to ExpressionUtils to let it access resultType and return it instead.
Please refer https://issues.apache.org/jira/browse/FLINK-9742 for more details.
Brief change log

adds ExpressionUtils.getReturnType to let it access resultType and return it instead

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: no
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for opening this PR @HeartSaVioR! (cc @twalthr)
I thought about this again and I don't think we should make the method public. The problem is that Expression is one of the core classes of the Table API. By making the method public it becomes visible to all users of the API.
A better approach is to provide a publicly accessible util object (in org.apache.flink.table.api...) that provides access to the result type (and possibly other properties) of ab Expression.
What do you think?
Best, Fabian

I think it can also be a new public method in org.apache.flink.table.expressions.ExpressionUtils.
Thanks, Fabian

@fhueske
Ah I didn't notice the object is already existing. Thanks for letting me know!

@fhueske Addressed. Please take a look again.

Looks good. Just one last comment.

Addressed.

Thanks for the update @HeartSaVioR.
I'll merge this


pre_training and fine-tune

Hi, thank you so much for sharing your code！
I have a question. I'd like to transfer the model to medical images.
But I don't I retrain directly or I need load the pre-training model and fine-tune it?
What do you think about it? Looking forward to your reply!

Thanks for your interest of our work. In my opinion, there might be a large gap between the distribution of medical images and our pretrained datasets. As a result, I think it might be more appropriate to retrain our model using your dataset. However, you could also try to fine-tune the model to see if it is feasible.

Ok, thank you for your answer. I'll try it.


[FR] Track number of bookmarks over time

Probably quite easy to do, just store in config and increment each time a bookmark is added (and decrement when removed)
Could be displayed on the mention embed

Implemented in #4 Thanks for the suggestion :)


Small UPDATEs using & leaking a lot of memory

Dear Crate team,
I have many tables (one per month) with about 200M docs and 4 shards / 1 replica.
Crate 0.54.5 on Ubuntu 14.04
Am doing small updates (modifying few hundreds / thousands of documents) like:
UPDATE my_table SET field_id=1234 WHERE field_id=-1;
The first UPDATE are going well (still a bit slow).
Then it fails with errors such as:
CircuitBreakingException: [FIELDDATA] Data too large, data for [_uid] would be larger than limit of [22705287987/21.1gb]]
CircuitBreakingException: [PARENT] Data too large, data for [_uid] would be larger than limit of [22705287987/21.1gb]]
I increased indices.breaker.fielddata.limit and indices.breaker.total.limit : it works for few more UPDATEs.
But the HEAP Usage on a unique machine (it doesn't seem to be the master) increases at each UPDATE and doesn't decrease : I have to restart it.
It looks like each UPDATE is fetching a lot of data into memory and won't release it.
Ben

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/31014282-small-updates-using-leaking-a-lot-of-memory?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F505659&utm_medium=issues&utm_source=github).

Hi @benjaminrigaud, we are looking into this on Monday morning.

hi @benjaminrigaud, as @chaudum told me this issue might be solved. Is it okay for you to close this issue? If this is happening again and you have more detailed information you can open a new one.

Hello,
From our last discussion, @chaudum couldn't reproduce the memory leak issue.
The high memory usage is the most concerning in my case: the UPDATE (of few thousands of documents) requires approximately the size of the table (20-25Go for ~ 200M documents) in memory: I don't know if this is expected. Do you have any idea?
I still have to TRACE the circuit breaker to get a better view: I should be able to do that this week.

no. it is not expected and as @chaudum said we aren't able to reproduce it.
for you this is still an issue?
so i leave this open. when you have any further findings we're happy to investigate.

It is still an issue for me at this time:
I'll try to build a docker / sandbox env with some fake data thanks to cr8 / faker.
I should be able to do it this week or next week.
Thank you.

hi @benjaminrigaud, have you been able to try this?

Not yet: too much work at this time. It is still on my TODO list :)

hi @benjaminrigaud, this should also be fixed with 0.54.9 and 0.55.0 as we refactored a lot of stuff in this part of crate. therefore i'm closing this issue. feel free to open it again, whenever you hit it again.


Default page labels are displayed for PDF files that specify empty string page labels for all pages

Attach (recommended) or Link to PDF file here:
1.pdf
Configuration:

Web browser and its version:  Firefox 121.0.1
Operating system and its version:  Windows 10
PDF.js version:  The version included with browser.  Maybe 4.0.240?
Is a browser extension:  No?

What is the problem?
Default page labels are displayed for PDF files that specify empty string page labels for all pages.
Steps to reproduce the problem:

Open the attached PDF file which specifies empty string page labels for all pages.

What is the expected behavior?
Page input field in the PDF.js toolbar is blank for all pages.  Text to the right of the page input field is (1 of 2) for the first page and (2 of 2) for the second page.
What went wrong?
Page input field in the PDF.js toolbar contains 1 for the first page and 2 for the second page.  Text to the right of the page input field is of 2.

This is done on purpose, since having the page-number input be completely blank doesn't seem particularly helpful; please note https://github.com/mozilla/pdf.js/blob/ae627870801eb758180e1db0c51b5a778c61af7a/web/app.js#L1590-L1607


Add Nong Song:

Song Name
grasspiano
Artist Name
fauxlite
Source
Youtube
Youtube Link
https://youtu.be/M1O169s1DzU
Direct File Link
No response
Song ID
3333
Start Offset [ms]
0
Extra
grasspiano is my own personal work, and i would like to get a temporary auto-nong until im whitelisted on newgrounds.

reject
invalid YouTube link

reject
invalid YouTube link


Webpack Caching Error

Bug report

What is the current behavior?
Shows Deprecated Warning
If the current behavior is a bug, please provide the steps to reproduce.
See screenshot

https://webpack.js.org/guides/caching/. - Change the moduleIDs from hashed to deterministic

What is the expected behavior?
After running npm run build it works. It's only a notification for deprecation.

Other relevant information:
webpack version:  5.3.1
Node.js version: 12.9.0
Operating System: macOS Catalina 10.15.7
Additional tools:

Webpack support long term cache out box, maybe we should remove this guides now

Webpack supports long term cache out box, maybe we should remove this guides now

@evilebottnawi seems like the problem of the contenthashes gettting changed, described in the "Module Identifiers" section of the Caching guide, is no longer there. After importing the new print.js module into index.js, upon rebuilding, only the main bundle's hash changed.

Yes

Fixed


pthread allow Stack on Spiram (IDFGH-7045)

when you have a lot of Tasks,
internal Memory might be used up fast.
now,
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/kconfig.html#config-spiram-allow-stack-external-memory
allow use of Stack on SPIRAM.
but not via C++ Pthreads.
https://github.com/espressif/esp-idf/tree/master/examples/cxx/pthread
would be a nice Extension right ?
Thanks,
Franz

@f-hoepfinger-hr-agrartechnik

would be a nice Extension right ?

Certainly! But I'm afraid that the constructor of std::thread does not allow any additional parameters than the actual function to be executed in the thread.
Maybe I missed something, so if you have another idea, please let me know!

esp_pthread_set_cfg(&cfg)
The cfg could be extended.

@f-hoepfinger-hr-agrartechnik @franz-ms-muc I see now (Should have looked at the example more thoroughly). Just to clarify: you would like a possibility to configure std::thread via the esp_pthread API? If that's correct, then I think it's possible. We would add this to our backlog but we currently have some other high-priority tasks. So it might take some (> 1 month). If you eventually implement something like this yourself, feel free to open a pull request and then we can discuss there.

Hi,
we did one implementation here:
https://github.com/hr-agrartechnik/esp-idf/blob/HR_4.4_ram/components/pthread/pthread.c
but i would not want to open a Pull Request as i feel the Quality is not good enough for a Pull request.
maybe later when we have done a bunch more test.
it is "generally" running OK, so we use it in a Prototype.

Ported same to 4.4.2
https://github.com/hr-agrartechnik/esp-idf/tree/HR_4.4.2_ram
any outlook that we can get such in ESP-IDF ?

will be here shortly.
https://github.com/espressif/esp-idf/pull/10623


CASSANDRA-19448 followup

Thanks for sending a pull request! Here are some tips if you're new here:

Ensure you have added or run the appropriate tests for your PR.
Be sure to keep the PR description updated to reflect all changes.
Write your PR title to summarize what this PR proposes.
If possible, provide a concise example to reproduce the issue for a faster review.
Read our contributor guidelines
If you're making a documentation change, see our guide to documentation contribution

Commit messages should follow the following format:
<One sentence description, usually Jira title or CHANGES.txt summary>

<Optional lengthier description (context on patch)>

patch by <Authors>; reviewed by <Reviewers> for CASSANDRA-#####

Co-authored-by: Name1 <email1>
Co-authored-by: Name2 <email2>

The Cassandra Jira

I will close this just to have the links and JIRA ticket numbers right in favor of https://github.com/apache/cassandra/pull/3581


[Feature Request]: To add the contact form and contact page

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The contact page is missing.
I will add it and to it I will add a contact form
Use Case
It will help the user to get connected.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@VaibhavArora314 Assign this issue to me. I will complete it within a day or two

This issue already exists #73


Merge t5 cut

T5 rz chi2 calculation upgraded from the previous used linear cut into helix calculation. With fake rate reduced.

This is a midway work of revisiting the rz chi2 cut when building the T5 object. Motivation of this task is based on changed track pT threshold is lowered from 1GeV to 0.8GeV. And we are always confused why do we have high fake rate for T5 objects with 10 hits. This work is based on optimizing the rz chi2 cut of T5. Previously on the rz plane, linear approximation is used when connecting the hits. Now helix approximation is used to better describe the trajectory, inspired by the previous study on pT3 object. Efficiency is saved while fake rate is reduced for PU200 samples. However, the efficiency for displaced cube samples is a bit affected.

helix approxi for T5 chi2 summary update (add linear for high pT) part1.pdf

part 2.pdf

There are 2 slides explaining the procedure, and the efficiency comparisons are shown below

This is a version of using helix approximation for rz chi2

this PR now has 85 commits with changes in 42 files. I don't think that the rebase wend well


Fix typos

Changed isSuccess parameters to isSuccess and IsSuccsess properties to IsSuccess.
Note: This is breaking change since IsSuccsess is public property.

Thanks! It isn't actually a breaking change since IsSuccsess only appears in README =)


Fix multiple history issues

Fixes:
#1607
#1285
#1376
#1374
#626

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Trivial change (affects only documentation or cleanup)

Checklist:

[X] I have tested this change with a run of GEOSgcm (if non-trivial)
[X] I have added one of the required labels (0 diff, 0 diff trivial, 0 diff structural, non 0-diff)
[X] I have updated the CHANGELOG.md accordingly following the style of Keep a Changelog

@bena-nasa Are we sure the new alias checking works well with the field splitting. My concern is the timing of the check - before we do the splitting, the alias field might "look the same' until is expanded during the splitting

I'll check that I haven't messed up the field splitting

@bena-nasa Are we sure the new alias checking works well with the field splitting. My concern is the timing of the check - before we do the splitting, the alias field might "look the same' until is expanded during the splitting

A split collection was zero-diff when I checked a split collection vs the control.


Just A Question

I want to know whether the picture after cropped has saved on SD card or we just get the result Uri?

see wiki


Cleanup require import

So far, I've identified a few "util is not defined" errors:

ReferenceError: "util" is not defined
org.openjdk.nashorn.internal.runtime.ECMAException: ReferenceError: "util" is not defined
E.g. in the Dashboard, "Kommende publiseringer" and "Spørringer" -> "Andre". Custom selectors have "Service error: Unknown error" due to the same util error.

So far, I've identified a few "util is not defined" errors:

ReferenceError: "util" is not defined
org.openjdk.nashorn.internal.runtime.ECMAException: ReferenceError: "util" is not defined

Can be found in:

Dashboard, "Kommende publiseringer" and "Spørringer" -> "Andre". When manually updating fetch statreg statistics under "Spørringer".
Best-bet.
Custom selectors have "Service error: Unknown error" due to the same util error.
Highcharts.
Possible other places as well?

Resolved. Alle these placese used arrayUtils which was where the bug was.

Highcharts with excel as their data source get the following error: TypeError: _striptags is not a function


primary color not changing

for the past week or so i've been trying to change my primary color to pink it seems to always turn back to white in game.

Cannot reproduce.


add token UCH

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

upload

new upload

Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwallet.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp


Feat/add faq page

This PR add the F.A.Q page (section Builders Program).

:tada: This PR is included in version 1.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Enable requester pays google storage remote objects

Issue: Current snakemake.remote.GS.remote() does not (to my knowledge) allow specifying a billable user project for accessing files in buckets with requester-pays set, e.g. GTEx v8.
This pull request updates snakemake.remote.GS.RemoteObject to allow specifying named parameter user_project. This allows remote downloads of files from google storage with requester pays enabled. Remote input files are specified using GS.remote(url, user_project=example_project), where example_project is a string specifying the project that will be charged for accessing the data.

@vsoch, since you are also working on the specification of a google project (globally, outside of the workflow) it might be a good idea to coordinate. I think a project name should not be part of the workflow definition itself, because others would need to alter the workflow definition to make it run with their google account. Rather, this should be taken from @vsoch's global option.

Another question @jaicher: doesn't this automatically happen when configuring via gcloud init?

Taking a look! In case you are also having formatting errors with black (different results here vs locally) I opened an issue here https://github.com/psf/black/issues/1140.

@johanneskoester as @vsoch mentions, this doesn't happen when configuring using gcloud init (or gcloud auth application-default login either as described in the docs).
I agree that we should not be hard-coding in project names in Snakefiles. I saw the example_project variable being derived from an input configuration file. I agree that it is unlikely that someone would want to bill different projects in a single workflow -- I think that using a boolean flag as @vsoch proposes is an excellent alternative if we are fine with restricting billing to a single project. Really like the ideas of how to report when requester pays is being used.
@vsoch if the bucket requires requester pays it raises google.api_core.exceptions.BadRequest with the message google.api_core.exceptions.BadRequest: 400 GET {... url ...}: Bucket is requester pays bucket but no user project provided.
@vsoch With the CI formatting issue -- I'm fairly new to contributing to these projects. I was looking at the docs on contributing and only saw information about the nosetests. I was trying to enable CircleCI/run locally but I thought it needed a .circleci/config.yaml file? Would appreciate any help setting up CI appropriately for testing future contributions to snakemake. I was trying to keep flake8 happy with my changes.
As for my original use-case, I was looking at processing cloud-only data from GTEx using a Snakemake pipeline -- the data are stored in a requester-pays only bucket. It turns out that this is a far from ideal way for downloading remote objects (downloads appeared to be single-threaded/blocking on the pipeline without compiled crcmod), so I've moved to an alternative approach.

@jaicher the google.api_core.exceptions.BadRequest would be perfect to catch (and then raise) and the error message provided is spot on to alert the user what is going on.
For formatting - let me see if I can help out! The repository uses black (which you can install with pip) and I would recommend that you use the same version as I see in tests:
pip install black==19.3b0

I was having issues with consistency between local and remote, so I wound up installing from the GitHub master branch, and although the first time it failed, the second time seemed to go okay. The way you can check locally is to mirror the CI and do:
conda create --quiet --name black black
source activate black
black --check snakemake tests/*.py

The command says to check the snakemake module, and files that end in .py in the tests folder. The CI itself isn't CircleCI, but rather GitHub CI / workflows, so you can find the entire recipe under .github/workflows/main.yml. You should be able to click the check links at the bottom of the PR to see the results.
For the testing, it would be nuts to try and run all of the tests that Snakemake has, but we still should have tests for this GS.remote setting that you can write. What I did is to:

Find an already existing folder that uses GS.remote commands
Create a new folder (named by issue associated likely, make sure you include the prefix test_github_issue* as the issues in question are from bitbucket originally (numbers don't match up).
Modify the Snakefile to include your variable, and possibly write additional Snakefiles that test other cases (e.g., one might have the flag for remotes, and the other not). You'd likely also need to use a remote bucket that you know has / doesn't have it.

You'll probably end up with something like this:
@pytest.mark.skip(reason="also need to figure out how to provide Google Cloud Project")
def test_github_issueX():
    workdir = dpath("test_github_issueX")
    tmpdir = run(
        workdir,
        use_conda=True,
        Snakefile="Snakefile.1"
        .... #other args
    )

    tmpdir = run(
        workdir,
        use_conda=True,
        Snakefile="Snakefile.1"
        .... #other args
    )

And note that you don't have to inspect the tmpdir after, that's only there if cleanup=False and you want to look at it for some reason.
Then to actually test, what I did is to open up ipython from the root of the repo, and basically import everything from tests
from tests.tests import *

And then you got it - I copy pasted the various tests. And what I would do to interact / debug is insert this snippet into relevant sections where I wanted the code to stop and interact (breakpoints)
import code
funcname = "GS.remote"
code.interact(local=locals())

You can set different funcnames as variables to give yourself hints about where you are, in the case there is more than one. Then you can at least run all your tests locally with your credentials to make sure things are working, and note that the function is marked for pytest to skip so it wouldn't run on CI.
Ah interesting! The test cases I've been doing thus far are using fairly small data files, and I suspect that's not going to be ideal for many projects. What would make sense to think about a little further down is (given an executor on Google Cloud) how to possibly mount a read only bucket as a drive (only works for read only) but yeah, for local runs where you need to download to your machine that's not even possible :P
Hope that helps!

@jaicher, please just call the test such that we know what is tested: test_gs_requester_pays.

@johanneskoester Okay, made update to the name of the test -- please let me know if there are any other changes you need.


x1066 As PO (Minal), I want horizontal scroll bars at the top of the tables on the dashboards, history page, search page and SGP management page for ease of use

@sabrine33 this is working great on the history, search and management pages but the scroll bar is on the bottom of the table on the dashboard pages, can you update these as well please?

Thank you @KatyTudor for the feedback, the top scroll bar is present at my end in the dashboard  page (Summary dashboard page) or may be we are looking into different pages; could you please provide more details about the pages that you are looking into, thanks

@sabrine33 I think on the test summary dashboard I see theres not a scroll bar o nthe top:

is my screen is too big and the scroll isn't needed?
The other dashboard is the main dashboard you come to on the home page:

@KatyTudor , yes, so basically the scrolling bars are only present when they are needed, in the case of the test summary dashboard, you need to minimise the width of your window to make it show up. Also, I think I missed the main dashboard page; I will update it now. Thanks.


Avoid having the linker generate debug symbols if turned off

fixes #7957

Do the same for Unix here: https://github.com/dotnet/corert/blob/master/src/BuildIntegration/Microsoft.NETCore.Native.Unix.props#L76 ?

I was considering changing the -g for unix too but was uncertain if that was effective at all. The PE file (6MB) at least got smaller by 512 bytes without /DEBUG, the ELF file (8MB) didn't, but stripping it reduced its size by 3MB.

Hmmm, looks like the SDK is using DebugSymbols as well and it is colliding with our definition. I believe that it is what's causing the CoreFX tests leg to fail. We should probably have our own property for this - like NativeDebugSymbols.

the ELF file (8MB) didn't, but stripping it reduced its size by 3MB.

Maybe we need to pass -Wl,--strip-debug to the linker to the omit the debug symbols?

-Wl,--strip-debug, yep, awesome, that did it.

Except on OSX where ld does not recognize --strip-debug


Parse escaped characters \x??, \u???? and \U????????

It seems that greenery does not support escaped characters:
import greenery
greenery.parse(
    '^[\\x09\\x0A\\x0D\\x20-\\uD7FF\\uE000-\\uFFFD\\U00010000-\\U0010FFFF]*$'
)

... throws
greenery.parse.NoMatch: Could not parse '^[\\x09\\x0A\\x0D\\x20-\\uD7FF\\uE000-\\uFFFD\\U00010000-\\U0010FFFF]*$' beyond index 1

However, Python's re module works with the escapes:
import re
re.compile(
    '^[\\x09\\x0A\\x0D\\x20-\\uD7FF\\uE000-\\uFFFD\\U00010000-\\U0010FFFF]*$'
)

I would expect greenery to match built-in module re in this regard. Or is this behavior by design?

The parser is intentionally very simple (note also the mostly useless parsing error it emits) because parsing all possible regular expression syntax wasn't really the core problem greenery was designed to solve. One workaround is to stop double-escaping those characters:
import greenery
greenery.parse(
    '^[\x09\x0A\x0D\x20-\uD7FF\uE000-\uFFFD\U00010000-\U0010FFFF]*$'
)

I will consider enhancing the parser to be able to handle double-escaped characters.


Looping single and multiple songs

How to do looping using this player?

I handled looping by adding some condition in onCompletion () of PlaybackManager.

The latest version supports looping + shuffle out of the box by calling transportControls.setRepeatMode(repeatMode) and transportControls.setShuffleMode(shuffleMode) respectively.
There isn't a UI associated with that at the moment, but those methods work via the MediaSessionConnection class, so shouldn't be difficult to connect up.


Refactor MQTT gateway and LWT unit tests to run without router

The unit tests for mqtt-* components currently start up the router in a docker image. This causes some test unstability on travis, and I'm not sure how much value those tests add when we have systemtests as well.
Wdyt @ppatierno ?

My opinion is that we can remove to run this tests on Travis CI (so no starting up the router image) but not removing the tests from the projects. I have used them a lot during development and I think that it could be useful during the development process (because I don't use systemtests in that phase).

Closing as this was fixed a while ago.


Update to curve25519-dalek-4.0.0-rc.1

Current dependencies on/for this PR:

mobilecoin

PR #6   👈

This comment was auto-generated by Graphite.


To be able to attach two or more windows showing side by side for easier Alt-Tab Switching

Hi,

So today i was having two windows side by side (vscode and outlook) . Then when i tried to switch to chrome with alt tab . After i was finished with chrome i pressed alt tab was hoping that the split view i created (vscode & outlook) should come back with 1 click of alt tab but i had to find both of the apps with alt tab to bring them back into the required position. If we could somehow attach or pin the two apps together with a shortcut that would be great.This could be extended to four side by side attached windows or custom positioned windows to be attached together for alt-tab.
This can also happen in another way where an extra key with alt tab can smartly detect all the windows placed on screen and can be locked for easier alt+tab+(an appropriate key) switching

And also while typing the above i had another idea for having alt-tab having two different modes. Where a user can add in a personalized alt-tab section all the  applications the person frequently switches . And the user can switch between these modes with a keyboard shortcut.

So what this will do is person can switch to non frequent mode when the user wants to with a keyboard shortcut . and switch to frequent application mode the user wants to so that the user doesnt have to keep searching for the frequently needed apps because some non frequent apps came in front of the alt tab switcher.
And both the non frequent and frequent modes can be seen on the alt tab overlay easily.
Dont know if the 2) one would be possible with power toys but please implement the first one.
Thanks
PS: Dont know if all the above  is currently possible.

Thank you for the suggestions, which would be approppriate or a Window Manager features.
Regarding having set views that you could switch between, wouldn't Virtual Desktops serve this purpose?

Like using virtual desktop in my work flow was kind of cumbersome and was not feeling efficient. I ll explain.
So i work on three different things simultaneously. So i have android studio work on one virtual desktop.
Vscode native development on one desktop and vscode-insider remote development on another desktop.
So i have lets say 6 windows which i can split half and half making 3 set of 6 windows. If two windows of a set could be attached together. then its easier 3 set switching with alt tab.
If i try to achieve this with virtual desktop i would need to first create 3 virtual desktop for 1 work flow and with 3 work flows it makes (9 desktops) . Then switching between work flows would be excess work if i switch between work flows often. And also some other apps i use support only one instance in only 1 desktop. So if i  click on that 1 instance app icon (currently open in 1st virtual desktop) and i am on the 9th virtual desktop the switch to 1st virtual desktop happens and then to switch back to the 9th virtual desktop is 9 cumbersome swipes.
So if we have attachable split windows or attachable custom window positioning available the number of virtual desktops could be reduced for me . So the power of alt-tab and virtual desktops could be used in unison.
Thanks

I also feel the need of linking two or more windows together.
Many times, I have to open multiple windows side-by-side and switch between them.
A simple example is when I switch via outlook, slack and explorer, if for example I am reading an email and on chat with someone on slack, then have to look for a doc to attach to emails and then go back to slack. Other scenarios is using VS and Terminal or a browser open at same time side-by-side.
Having the option to link the windows would be amazing.
Virtual Desktops is an alternative but takes much longer to find and switch to other windows

In Windows 11, this is basically fixed. Snap one window to the left and another to the right, and hover the icon for either one of them on the task bar. (difficult to make a screenshot, or I would've attached it)

Moving our hands to arrow keys while coding is tiring. I'd rather have alt + tab option or if there is another shcortcut for changing desktops?f


Installation command creates a folder named --help

:wrench: Summary
When I run the command lefthook install (using pnpm exec) a folder named --help is created.

Lefthook version
1.7.2

Steps to reproduce

Install lefthook
Use the command lefthook install
See the --help folder created by lefthook

Expected results
Some .lefthook folder or something like that and not a folder named --help.

Actual results
A folder named --help

Possible Solution

Logs / Screenshots
https://github.com/user-attachments/assets/f6904342-a17c-4b7a-967a-21d282395dda

LEFTHOOK_VERBOSE=true git ...

Hey! Could you share your lefthook.yml config?

Hi @mrexox, here my lefhook.yml file
pre-commit:
  parallel: true
  commands:
    check:
      glob: "*.{ts,tsx}"
      run: npx lint-staged --allow-empty && git update-index --again

commit-msg:
  scripts:
    "commitlint.sh":
      runner: bash

post-commit:
  commands:
    git-update-index:
      run: git update-index --again

Thank you. Could you please show what this git command displays?
git rev-parse --git-path hooks

Hi @mrexox, the response of the command is this:


Secure Rest API

Currently REST API is open, it would be nice to have some basic authentication.

PR welcome :)


Translation in Header

Hi
We need a translation dropdown in header for https://refugies.info/fr (we use this tool from dataesr implementation).
How can we implement this with codegouvfr/react-dsfr ?
Thanks

Il sill preprod we have it as well: https://sill-preprod.lab.sspcloud.fr/

@garronej
Hello!
After some quick testing I wanted to point out that apparently the locale selector works great in desktop mode but the drop-down menu is not triggered when clicked in mobile mode.
Thank you :)

😢 Yes indeed


Include / after hostname in generated links

The "Copy link to message" option in the message action sheet produces links that look like this:
https://chat.zulip.org#narrow/stream/48-mobile/topic/Copied.20message.20links.20do.20not.20linkify/near/1904591
Note there's no / after the hostname, before the #. This is still a valid URL, I believe — but it looks funny, and more concretely it means that if you post it in a Zulip message it doesn't get linkified. We should fix it so that the link instead looks like, for example:
https://chat.zulip.org/#narrow/stream/48-mobile/topic/Copied.20message.20links.20do.20not.20linkify/near/1904591
It looks like none of our existing tests for this functionality spell out the full URL that gets produced — they're all in terms of pieces of the parsed URL, or compute the expected URL with things like realmUrl.resolve(…). Those are fine sorts of tests to have in order to keep those tests focused on the details they're meant to be about; but as part of a PR fixing this issue, we should also add a few tests that include, verbatim, the full URL they expect as the output.
Implementation
A PR to fix this issue should do two things:

Add some tests of the narrowLink function, alongside its existing tests, that specify the verbatim full URL they expect.
Adjust that function so that its results look like https://chat.example/#narrow/… instead of https://chat.example#narrow/….

This issue is open to be worked on. There was a previous PR attempting it:

#874

but that PR didn't work out and I've closed it.
That PR thread is long, and most of what's there probably isn't useful to read before sending a fresh PR. After rereading the thread just now, I've edited the issue description above to lay out more explicitly what should go into an implementation.
This was in the "Beta 3: Summer 2024" milestone because it was actively being worked on at the end of the summer. Since the thread of work on that PR is over, I've moved it to the current milestone, "Beta 4: Fall 2024".


Configuring reverse proxy for Apache 2

Hi,
First of all, thanks a lot for bringing this nice tool and keeping it up to date, very usefull.
I've just a little problem completing the reverse proxy configuration on Apache 2 to access it throught TLS.
Here's my virtualhost configuration (I've hide some variables):
<VirtualHost *:80>
        Define SERVER_NAME filezilla.domain.tld
        ServerName ${SERVER_NAME}

        RewriteEngine On
        RewriteCond %{SERVER_NAME} =${SERVER_NAME}
        RewriteRule ^ https://%{SERVER_NAME}%{REQUEST_URI} [END,NE,R=permanent]

        CustomLog ${APACHE_LOG_DIR}/${SERVER_NAME}_access.log combined
        ErrorLog ${APACHE_LOG_DIR}/${SERVER_NAME}_error.log
</VirtualHost>

<VirtualHost *:443>
        ProxyRequests Off
        ProxyPreserveHost On

        ServerName ${SERVER_NAME}
        SSLProxyEngine On

        ProxyPass /websockify ws://filezilla.domain.tld/
        ProxyPassReverse /websockify http://filezilla.domain.tld/

        ProxyPass / http://localhost:5800/
        ProxyPassReverse / http://localhost:5800/

        RequestHeader set X-Forwarded-Proto "https"

        SSLEngine On
        SSLCertificateFile /etc/apache2/certs/live/domain.tld/fullchain.pem
        SSLCertificateKeyFile /etc/apache2/certs/live/domain.tld/privkey.pem

        CustomLog ${APACHE_LOG_DIR}/${SERVER_NAME}_access.log combined
        ErrorLog ${APACHE_LOG_DIR}/${SERVER_NAME}_error.log
</VirtualHost>

I've load module 'proxy_wstunnel' and I've no error in the Apache logs, the only error I've got is from my browser in console mode:
WebSocket connection to 'wss://filezilla.domain.tld/websockify' failed: Error during WebSocket handshake: Unexpected response code: 301
WebSocket on-error event
Msg: Server disconnected (code: 1006)

Any ideas to correct this ?
FYI I can access the container GUI through classic port without reverse proxy in front of it.
Regards.

Hi @jlesage,
Any idea ?

Sorry for the delay on this.  I'm not really familiar with Apache config...  However, you could look at logs under log/nginx/, under the directory you mapped to /config.  You can see if the requests are reaching the container and if yes, if they are correct/expected...

Hi,
No problem for the delay, thanks for the tip, I'll take a look into ~/config/log/nginx.
Keep you in touch.

No more informations on this logs...
I'll keep searching on my own, if I find the solution, I will come back to share it.

Closing this issue.  Please re-open if needed.


Handle inline functions better

A lot of xed's interface uses inline functions which are not well supported by bindgen. It would be good to come up with some kind of solution to this.

rust-bindgen has support for exposing static inline functions now: https://github.com/rust-lang/rust-bindgen/pull/2335


hidding table names on query results and data tab

I'd like to show just the column name on query results and when navigating over a object data tab. Couldn't find how to. Follow some examples that show the results with table names:

I ca't reproduce such behavior with any available database/driver. DBeaver doesn't show table names as a part of column names. This should be some specific behavior of the driver.
What database do you use?
(Sorry for so long reply)

This is Hive-specific behavior. By some reason it returns column names in such weird form.
Potentially we could make some workaround but I'm not sure we should. Maybe we should file a bug report to Hive dev..

Run this set command in your dbeaver SQL Editor and you should be good to go.
set hive.resultset.use.unique.column.names=false;

It works when I set the variable with :
set hive.resultset.use.unique.column.names=false;
But it resets to true everytime I reopen DBeaver. How can I do to keep it that way ? I didn't find in the configuration,

Try using Bootstrap Queries like this:
1.) Enter the Connection Wizard to re-create your Hive Settings (hit "next" three times)
2.) On the screen "Finish General Connection Settings",
--> Click the url hyperlink to enter: "Connection initialization settings"
--> Hit the "Configure" button under "Bootstrap Queries:" and add your SQL queries.
...
Note: I didn't test this (we use a labor-intensive, bastardized config that isn't wizard-friendly for hive),
but its something to chew-on and may work out of the box for you. Try the "Shell Commands" url if bootstrap goes nowhere.
I actually stopped using the SET method workaround I posted.
In the record results-window, the Filter button stops listing the distinct entries per field, meaning you'd have to eyeball thousands of records of categorical data to spot the one stray/undefined entry that shouldn't exist in that column and was inserted incorrectly. Definitely a big dealbreaker for me.
My new "Learn a better habit" fix is: Alias tables that use asterisk queries to something short.
Query:      Select * FROM  ExtraLongNamespaceTableHere  t  ;
Returns:    t.foo  , t.bar ,  t.bat ,  t.baz ...


Index out of Range Error After Reducing number of class_names

Hi @ahmetozlu, we are getting list index out of range after reducing no. of class_names while running single_image_object_counting.py file in mask_rcnn_counting_api_keras_tensorflow folder. Can you please help us? Thanks!

Since you are using coco trained model, there have to be 80 classes in default. If you need to update classes, you have to perform transfer learning with your own data with your custom classes.


Setup scripts: use AWS_PROFILE

Description of changes:
The AWS_PROFILE field isn't used in two of the setup scripts for the ECS demo. This change causes the scripts to observe the profile field, but only if it is set.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for this.
Maybe we want this in delete.sh and route_canary.sh too, to be consistent.


Cache Jurisdiction Loads

This PR address a small performance improvement when gathering a list of Jurisdictions for drop down menus.
Some Benchmarks on a patient with 5 assessments.
ActiveRecord Cached (with Redis):

(Views: 1296.9ms | ActiveRecord: 86.7ms | Allocations: 983307)
(Views: 1361.4ms | ActiveRecord: 90.6ms | Allocations: 983259)
(Views: 1288.1ms | ActiveRecord: 83.6ms | Allocations: 983306)

AR Average: 86.96ms
ActiveRecord Not cached (with Redis):

(Views: 1478.4ms | ActiveRecord: 344.1ms | Allocations: 1435569)
(Views: 1303.8ms | ActiveRecord: 299.9ms | Allocations: 1433634)
(Views: 1425.8ms | ActiveRecord: 325.7ms | Allocations: 1435565)

AR Average: 323.23ms
ActiveRecord Cached (without Redis):

(Views: 1405.7ms | ActiveRecord: 89.7ms | Allocations: 984925)
(Views: 1461.1ms | ActiveRecord: 90.0ms | Allocations: 984917)
(Views: 1495.0ms | ActiveRecord: 92.2ms | Allocations: 984915)

AR Average: 90.63ms
ActiveRecord not Cached (without Redis):

(Views: 1369.6ms | ActiveRecord: 310.9ms | Allocations: 1435857)
(Views: 1464.3ms | ActiveRecord: 341.5ms | Allocations: 1435836)
(Views: 1419.9ms | ActiveRecord: 327.4ms | Allocations: 1433901)

AR Average: 326.59ms
Benchmarks on the actual call:
Redis
irb(main):004:0> Benchmark.measure { Jurisdiction.all_paths }
=> #<Benchmark::Tms:0x00007f823ffed1c0 @label="", @real=0.0009510000236332417, @cstime=0.0, @cutime=0.0, @stime=0.00017000000000000348, @utime=0.0007169999999999954, @total=0.0008869999999999989>
irb(main):005:0> Benchmark.measure { Jurisdiction.all_paths }
=> #<Benchmark::Tms:0x00007f824a825f40 @label="", @real=0.0012610000558197498, @cstime=0.0, @cutime=0.0, @stime=0.0002400000000000041, @utime=0.0009089999999999931, @total=0.0011489999999999972>
irb(main):006:0> Benchmark.measure { Jurisdiction.all_paths }
=> #<Benchmark::Tms:0x00007f824a84e4b8 @label="", @real=0.0010719997808337212, @cstime=0.0, @cutime=0.0, @stime=0.0001709999999999906, @utime=0.0007979999999999932, @total=0.0009689999999999838>
irb(main):007:0> Benchmark.measure { Jurisdiction.all_paths }
=> #<Benchmark::Tms:0x00007f824a8773e0 @label="", @real=0.000898000318557024, @cstime=0.0, @cutime=0.0, @stime=0.00016599999999999948, @utime=0.0006359999999999977, @total=0.0008019999999999972>

No redis
irb(main):009:0> Benchmark.measure { Hash[Jurisdiction.all.where.not(name: 'USA').pluck(:id, :path).map { |id, path| [id, path] }] }
   (1.0ms)  SELECT `jurisdictions`.`id`, `jurisdictions`.`path` FROM `jurisdictions` WHERE `jurisdictions`.`name` != 'USA'
=> #<Benchmark::Tms:0x00007f823ebff898 @label="", @real=0.015306999906897545, @cstime=0.0, @cutime=0.0, @stime=0.003240999999999994, @utime=0.010511999999999994, @total=0.013752999999999987>
irb(main):010:0> Benchmark.measure { Hash[Jurisdiction.all.where.not(name: 'USA').pluck(:id, :path).map { |id, path| [id, path] }] }
   (1.1ms)  SELECT `jurisdictions`.`id`, `jurisdictions`.`path` FROM `jurisdictions` WHERE `jurisdictions`.`name` != 'USA'
=> #<Benchmark::Tms:0x00007f8249965f28 @label="", @real=0.004269000142812729, @cstime=0.0, @cutime=0.0, @stime=0.0006750000000000089, @utime=0.002910999999999997, @total=0.003586000000000006>
irb(main):011:0> Benchmark.measure { Hash[Jurisdiction.all.where.not(name: 'USA').pluck(:id, :path).map { |id, path| [id, path] }] }
   (1.0ms)  SELECT `jurisdictions`.`id`, `jurisdictions`.`path` FROM `jurisdictions` WHERE `jurisdictions`.`name` != 'USA'
=> #<Benchmark::Tms:0x00007f823e4e81e0 @label="", @real=0.0041519999504089355, @cstime=0.0, @cutime=0.0, @stime=0.0006770000000000109, @utime=0.002812000000000009, @total=0.00348900000000002>
irb(main):012:0> Benchmark.measure { Hash[Jurisdiction.all.where.not(name: 'USA').pluck(:id, :path).map { |id, path| [id, path] }] }
   (1.1ms)  SELECT `jurisdictions`.`id`, `jurisdictions`.`path` FROM `jurisdictions` WHERE `jurisdictions`.`name` != 'USA'
=> #<Benchmark::Tms:0x00007f82482df768 @label="", @real=0.004271000158041716, @cstime=0.0, @cutime=0.0, @stime=0.0006239999999999857, @utime=0.002919000000000005, @total=0.0035429999999999906>

This seems to be working well locally. I have a few questions about this setup though:
1. Why should these expire every 24 hours? Technically this information won't ever need to expire until it's explicitly invalidated during a jurisdiction update.

2. The [docs](https://guides.rubyonrails.org/caching_with_rails.html#activesupport-cache-rediscachestore) for using a RedisCacheStore highly recommend having a dedicated cache-only Redis server - we don't have that yet, correct? I realize this isn't memory intensive at the moment but still.

I figured I'd just set a value. Can remove it if we want to. We can add another Redis specifically for cache and then set the config on it too...

Closing and will be updating with ticket for tracking this work and bringing into future sprint.

Closing and will be updating with ticket for tracking this work and bringing into future sprint.


Readme Improvements

Mention contribution guidelines
Add more images and gifs to project it better
Update new logos and UI of the readme

Can I work on this?

The enhancement I would be adding to the documentation is:

Adding images and gifs so that the developer would have ease in reading the readme
Updating the logos so that the UI looks good.
And Mention the contribution guidelines so that developer who is contributing like me would have an idea about the specific criteria of contribution.

I'll give my best


Suggestion: Prevent squishing of panes

As per @mrleblanc101's comments on #12 I'm creating a new issue for separate discussion.
I think it would be nice if you could set a 'minimum' dimension in absolute value that a pane will be reduced to. This way you can try to resize a pane split to 90% / 10% but the smaller pane will not go below 200px.
I personally set my separation to around 60% so this isn't a big issue for me, but this seemed like a straightforward way to fix the problem for others.

The real problem is that you squish the content making it look really unpleasant and distracting.
There should be an absolute mode where the pane overlaps each other instead of being squished.
It also causes conflict with the minimal plugin as you can see in the screenshot below.

Then setting pane looks good because it's content doesn't get squished or wrapped over multiple lines

I would also find this helpful.

Also to address @mrleblanc101's original issue, I think his problem could be fixed by simply disabling wrapping.

Good idea! Is that something that plugins can modify on the fly, pane-by-pane?

Because the issue is moved, I'll just copy my answer from the original Issue #12.

That's not possible due to the way Panes work in Atom. The only way to not get squished content is by disabling soft wrap at the end of the Window and instead only wrap content at the 80char mark.

I checked the Docs of Panes in Atom if it's possible to disable wrapping per pane, but I didn't find anything.
Also I saw in my config file that the setting for Text Wrapping is scoped to editor (it's editor.softWrap and editor.softWrapAtPreferredLineLength), which also makes me believe this is only a global editor setting and not adjustable per Pane.
Maybe someone has an idea how to achieve it nevertheless?

Is it worth asking @atom to see if they'd be willing to implement pane-specific wrap overrides? That would seem to fit their goal of making the editor hackable.

Opening a feature request is always a possibility, although I think supporting configs per pane is quite a big feature request. Also activating/deactivating soft wrap would only be a hack. For example, as you can see in the first screenshot of https://github.com/timomeh/hey-pane/issues/15#issue-271970920, the minimap is also squished, because the pane is so small.
We would need the possibility to let panes overlap, which also is quite a big feature request, because it would totally change the way panes are positioned.

I think people shouldn't use such extreme percentages to begin with. I personally care more about animated resizing than this story lol

I personally use a "extreme" percentage, too, and I think many people also do.

I would use an extreme percentage, but I can't because the wrapping doesn't work, and I want to be able to at least see a bit of the small column's file without it scrolling down super far when it compresses. If there were wrap disabling, I'd use an extreme percentages in a heartbeat.
Just my 2c and my usecase.


Upgrade linkerd v2.14.3

Upgrade Linkerd v2.14.3
Linkerd Release notes:
2.14.3 - https://github.com/linkerd/linkerd2/releases/tag/stable-2.14.3
2.14.2 - https://github.com/linkerd/linkerd2/releases/tag/stable-2.14.2
2.14.1 - https://github.com/linkerd/linkerd2/releases/tag/stable-2.14.1
2.14.0 - https://github.com/linkerd/linkerd2/releases/tag/stable-2.14.0
Also verify you have:

[X] Read the contributions page.
[X] Submitted the CLA form, if you are a first time contributor.

Thanks @balchua !


[Release] sdk/resourcemanager/hybridcompute/armhybridcompute/2.0.0-beta.4

https://github.com/Azure/sdk-release-request/issues/5334

beta breakingchange from: https://github.com/Azure/azure-rest-api-specs/pull/29510


fix(docs): incorrect description for refund api

Type of Change

[ ] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[x] Documentation
[ ] CI/CD

Description
The current description for refund - list is incorrect. Newer one looks more apt.
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

[ ] I formatted the code cargo +nightly fmt --all
[ ] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible

@shawshankkumar Could you generate the OpenAPI spec files using the following commands, and commit them?
cargo run -p openapi --features v1
cargo run -p openapi --features v2

Also, while you're at it, can you create an issue for the same and link the PR to the issue?

Closes - #6751 6751

Hey @SanchithHegde can you please check this again?


Output plots

Graphical output can help users to understand HIVIntact's decisions, and call alternative judgments.

As helpful as it is, this feature is outside the scope of CFEIntact.


8.0.0 updates

Remove unnecessary configuration of assets.rb
Doc updates

This change is

Coverage decreased (-0.02%) to 97.992% when pulling d45ca7a11620bab22f886a9039b502aa65ff2df9 on 8.0.0-updates into e57b058f8cfa2b7897846d753c8bfc906372ec46 on master.

Coverage decreased (-0.02%) to 97.992% when pulling d45ca7a11620bab22f886a9039b502aa65ff2df9 on 8.0.0-updates into e57b058f8cfa2b7897846d753c8bfc906372ec46 on master.


BROOKLYN-272: disable more failing non-deterministic tests

Only affects tests, disabling those that failed in apache jenkins. Merging now.


fix: Sort Workflows by finish time in UI

Fixes some changes from #2863:

Sorts Workflows by FinishedAt time as previously done in workflow-list and archive-workflow-list.

Adds Finished field to lists.

We should add more sorting options in the UI: opened https://github.com/argoproj/argo/issues/2895 for this.

Yes, of course. I'll rework.


csrf token example for Laravel Integration

To use vue-dropzone with laravel, we need to add csrf token to it and this example helps with that.

Great @codegeek1001,
It's not good practice to maintain docs related to every or multiple frameworks in single README file. File gets bulky to understand to user. No worry meanwhile your efforts are welcome. V3 version of this component is in progress and address this point shortly.
Thanks for your contribution. I'm merging this to docs.
Happy coding :man_technologist:

Using
 headers: { "X-CSRF-TOKEN": window.axios.defaults.headers.common['X-CSRF-TOKEN'] }

worked for me as csrf token was assigned in bootstrap.js

headers: { "X-CSRF-TOKEN": document.head.querySelector('meta[name="csrf-token"]').content }
This worked for me


Deprecated?

I noticed this repo hasn't been updated for 4 years. Is it deprecated? Is there a superseded project?

I am no longer interested.

Wrong status


Database migration fails if collection has an TTL index

Hi,
When we tried to upgrade to the latest Hangfire.Mongo version in one of our projects, on startup we get an  InvalidCastException (Unable to cast object of type 'MongoDB.Bson.BsonDouble' to type 'MongoDB.Bson.BsonInt64'). This only happens in one project where we have added a TTL index to the jobGraph collection and that is causing the database migration to fail.
The exception is being thrown by this line: https://github.com/sergeyzwezdin/Hangfire.Mongo/blob/7c3ae992719a0747d71380f13c97af1ef0769660/src/Hangfire.Mongo/Migration/Strategies/Backup/CollectionMongoBackupStrategy.cs#L105
We tried running the backup strategy code in a simple console app and in fact the BsonType for the "expireAfterSeconds" being return is Double thus causing the exception

@hmsantos,
I added a check for the type.
Could you elaborate on what you are trying to fix with the TTL index? Hangfire.Mongo comes with a "MongoExpirationManager" which deletes expired records. Is something not working correctly?

@gottscj
Thanks for adding the check.
We added the TTL index because for some reason in that service the jogGraph documents expiration date isn't always being set and because of that the database kept growing. We didn't find any pattern of which documents failed to set the expiration date so we created the index to control the database size and upgraded to the latest Hangfire.Mongo version at the time. Will try to have another look when possible


CSS Formatter for VS Code

Hello Mutian,
sneaky introduction, I'm a web developer and I'm switching from Sublime text to VS Code.
I gradually managed to migrate everything, only the best / amazing package remained for me :)  CSS Formatter for Sublime Text. https://packagecontrol.io/packages/CSS Format
Many packages contain "beautifiers", but I code on desktop, not on mobile. Aligned css to the column is unergonomic and difficult....
I've tried almost all CSS "formatting" packages for VSCode, but none of them contain what yours contains for Sublime Text.
Unfortunately, I cannot find a replacement for this package. :/
That's why I want to ask, have you ever considered preparing it for VS Code?
Thank you very much for the answer.
Have a nice day
Matthew

I moved a copy to vscode
css-format-st3


new release available for mpc-hc-clsid2

Hello,
There is a new version for the package https://community.chocolatey.org/packages/mpc-hc-clsid2
The update is available at this address: https://github.com/clsid2/mpc-hc/releases/tag/1.9.18
Current version: 1.9.16
New version: 1.9.18
Kind regards,

hmm, looks like there is something going on when we try to update the package.
I'll have a look at it as soon as I can.

A new update of this package is now queued on our update CI, a new package version should be submitted shortly and will be available once it has completed the moderation checks.


Fix a couple of UI tests flakes

Description
Fixing two flaky tests:

increased timeout to 10 seconds for ViolationsByPolicyCategory.test.tsx;
add days to the exact start date, use ISO date format for VulnRequestedAction.test.tsx;

Checklist

[ ] Investigated and inspected CI test results
[x] Unit test and regression tests added
[ ] ~Evaluated and added CHANGELOG entry if required~
[ ] ~Determined and documented upgrade steps~
[ ] ~Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
cd ui
make test

Removed timeout increase as couldn't reproduce the flake locally with no parallel load.

/test ui-unit-tests

/test grouped-static-checks

/retest-required


After split, Show pages' items list includes the filesets of child works

Summary
NOTE: This issue is caused by/related to this Hyrax bug
Per Katharine,
I tested with my own 6 page PDF.
https://adl.s2.adventistdigitallibrary.org/concern/generic_works/538a6bcf-e2b7-429e-b15a-f19530a9a7d9?locale=en
BUGS:
The splitting worked, but we have double the number of child works showing in the items list. On Production we usually only see the child works with page numbers attached. However, on staging, there are a string of JPEGs in the item list.
All child works are showing in the UV, so the work is duplicated in the UV.
Acceptance Criteria

[ ] Feature parity with production
[ ] After split, the items section should only show the child works with page number attached.
[ ] After split, the parent work should not also show the child work's filesets in the items section.
[ ] No duplication in the UV

Screenshots or Video
STAGING

PRODUCTION

Testing Instructions
To be filled out by dev
Notes
related issue: https://github.com/samvera/hyrax/issues/6767

Thoughts on an override to the indexers? Will it have negative consequences?
# hyku_indexing.rb and maybe pcdm indexer?
        solr_doc['member_ids_ssim'] = filter_member_ids(object.member_ids.map(&:to_s))

private

  def filter_member_ids(member_ids)
    member_ids.reject do |id|
      member = Hyrax.query_service.find_by(id: Valkyrie::ID.new(id))
      member.is_a?(Hyrax::FileSet)
    end
  end

Per LaRita, dassie works as expected.
She will test Koppie next.

QA Results: ✅  Passed
Tested with work https://adl.s2.adventistdigitallibrary.org/concern/generic_works/9854b255-38ee-4cdd-95cf-004b9f8d324a?locale=en
Screenshots
UV does not duplicate pages

Before a reindex, child work filesets are included in items

After edit/save (to reindex) child work filesets are not included in items

Working as expected when tested in ADL staging. Sample work: https://adl.s2.adventistdigitallibrary.org/concern/journal_articles/lake_union_herald_2023_05_18_19_how_shall_we_go?locale=en
Viewer shows the PDF once, and the items list has only page number child works.


Generalize the CDD downloading/processing process

Try to move away from code being built around our previous narrow use case (one specific search from one CDD vault, always being processed with the same fields in the same way).
Todos

[x] Modularize URL in downloading process
[x] Split asapdiscovery.data.utils.filter_molecules_dataframe into two separate functions for filtering and processing
[x] Tests!

Status

[x] Ready to go

@kaminow should we include this in 0.1.0? My guess is yes. You can remove it if you disagree.

@hmacdope @ijpulidos should be ready for a review now. not really sure why that one test is failing as it passes on my machine. I can try and recreate it in the morning


add the ability to set the authMethodName

Is your enhancement request related to a problem? Please describe.
Many authN/authZ setups use tokens. But the way pulsar relies on determining which authentication class to call requires a user to configure the authMethodName on the Authentication class on the client side. This is fine for java, but for the python client and many others this leads to difficulties and duplicating much of the AuthenticationToken class. If we had the ability to set the authMethodName that would be easier for customized authentication/authorization in Pulsar
Describe the solution you'd like
the ability to change the authMethodName on AuthenticationToken to tell pulsar which custom classes are to be used.
Describe alternatives you've considered
I've been building pulsar-client libraries that extend the AuthenticationToken class.

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.


Can you add the paper references for each of the implementations?

Instead of labeling the constants and functions with description text, could you please add the paper references?
Thanks!

What are you referring to?

Closing due to lack of activity.  Feel free to reopen if more information is available.

@martinwicke: DId you read the question or did you just ignore it?
@girving: Why did you close the question without a clear response?
Tensorflow is not written in a pedagogical manner but rather a opaque manner.  This should change with some paper references and a glossary section so people can drill-down on the vocab with sources cited.  I cannot believe that an open-source project would have two unrelated responses to my question in the last three months.  Where have the Tensorflow developers gone?

@shyamalschandra: Martin asked you for specifics and you didn't reply, so I closed it due to lack of activity.  We would be happy to add specific references to papers to specific places in the code, but a blanket "Add more documentation" bug is hard to work with.

@girving and @martinwicke: Okay, can you start by adding paper references to all the "implementations" inside the Tensorflow repository?  The version number is now around 2.0 preview.  Can you please tell the technical writers to add this information?

There are many papers referenced in the docstrings. If you would like to add more, please send PRs.


hallo toolbar misplacement within StreamField in a MultiFieldPanel

Issue Summary
The toolbar overlaps content in the text editor when a RichTextBlock is used inside of a MultiFieldPanel.
The workaround is to wrap the RichTextBlock in a StructBlock - as per #1511 fixed in #3415 - or not to use MultiFieldPanel.
Steps to Reproduce
Set up a model as follows:
body_de = StreamField([('paragraph', RichTextBlock())])
content_panels = [
        MultiFieldPanel([
            StreamFieldPanel('body')
        ])
]

Observe the toolbar overlapping like this:

Technical details

Python version: 3.5.3
Django version: 1.11
Wagtail version: 1.10
Browser version: tested in current stable versions of IE, Firefox, Chrome

It seems to me that this fancy positioned toolbar is way to complex to be bugfree, and every fix for one position issue seems to cause other position issues in other contexts.
Would it be possible use a toolbar that are not positioned but placed statically just above the text-area, instead?

Had the same problem and solved it using a hacky workaround. I'm sure that the toolbar will make some problems if we integrate the css rule I've added here in other edge cases.
I'm not suggesting you should use the following, but it solves the problem (using a not maintainable way, I admit).
Maybe we can build a proper solution based on it:
In some_app/wagtail_hooks.py:
from django.contrib.staticfiles.templatetags.staticfiles import static
from django.utils.html import format_html

from wagtail.wagtailcore import hooks

@hooks.register('insert_editor_css')
def editor_css():
    return format_html(
        '<style>{}</style>',
        'body div.hallotoolbar:not(.stream-field) {min-width: 850px !important;transform: translate3d(-60px, -75px, 0px) !important;}'
    )


Cannot find 'assign_taxonomy.py'

hello, I install qiime2 throught conda:
conda env create -n qiime2-2019.7 --file qiime2-2019.7-py36-linux-conda.yml
source activate qiime2-2019.7
I want to use the 'assign_taxonomy.py' but I cannot find it in the
qiime enviroment.
Where can I find this script?
Thanks!!!!

assign_taxonomy.py is not a part of QIIME 2, it is a script from qiime 1, hence why it was not installed.
In the future, please use the QIIME 2 forum for all support questions. Thanks!


Two unrelated nested issues regarding inference with NeuronStableDiffusionPipeline

First issue
When using the latest diffusers library (Diffusers > 0.19.0, after this commit: 27062c3631b7011a5df45782b8e3d01349d1f3e9)
Some attributes are missing in NeuronStableDiffusionPipeline, causing "components" method to fail
Reproduce:
from optimum.neuron import NeuronStableDiffusionPipeline
model_id = "stabilityai/stable-diffusion-2-1-base"
input_shapes = {"batch_size": 1, "height": 512, "width": 512}
import pdb
pdb.set_trace()
stable_diffusion = NeuronStableDiffusionPipeline.from_pretrained(model_id, export=True, **input_shapes)
save_directory = "sd_neuron/"
stable_diffusion.save_pretrained(save_directory)
stable_diffusion('whatever')

Error obtained:
Traceback (most recent call last):
  File "test.py", line 8, in <module>
    stable_diffusion('horse')
  File "/opt/aws_neuron_venv_pytorch/lib/python3.8/site-packages/optimum/neuron/modeling_diffusion.py", line 509, in __call__
    return StableDiffusionPipelineMixin.__call__(self, *args, **kwargs)
  File "/opt/aws_neuron_venv_pytorch/lib/python3.8/site-packages/optimum/neuron/pipelines/diffusers/pipeline_stable_diffusion.py", line 178, in __call__
    device = self._execution_device
  File "/opt/aws_neuron_venv_pytorch/lib/python3.8/site-packages/diffusers/configuration_utils.py", line 137, in __getattr__
    raise AttributeError(f"'{type(self).__name__}' object has no attribute '{name}'")
AttributeError: 'NeuronStableDiffusionPipeline' object has no attribute '_execution_device'

Workaround patch (not a proper fix, just to expose the cause):
diff --git a/optimum/neuron/modeling_diffusion.py b/optimum/neuron/modeling_diffusion.py
index 8967e16..242a780 100644
--- a/optimum/neuron/modeling_diffusion.py
+++ b/optimum/neuron/modeling_diffusion.py
@@ -24,6 +24,7 @@ from tempfile import TemporaryDirectory
 from typing import TYPE_CHECKING, Any, Dict, List, Optional, Tuple, Union

 import torch
+from diffusers.pipelines.stable_diffusion import StableDiffusionSafetyChecker
 from huggingface_hub import snapshot_download
 from transformers import CLIPFeatureExtractor, CLIPTokenizer, PretrainedConfig

@@ -73,14 +74,16 @@ class NeuronStableDiffusionPipelineBase(NeuronBaseModel):
         unet: torch.jit._script.ScriptModule,
         vae_encoder: torch.jit._script.ScriptModule,
         vae_decoder: torch.jit._script.ScriptModule,
-        config: Dict[str, Any],
         tokenizer: CLIPTokenizer,
         scheduler: Union[DDIMScheduler, PNDMScheduler, LMSDiscreteScheduler],
         feature_extractor: Optional[CLIPFeatureExtractor] = None,
+        config: Optional[Dict[str, Any]] = {},
         configs: Optional[Dict[str, "PretrainedConfig"]] = None,
         neuron_configs: Optional[Dict[str, "NeuronConfig"]] = None,
         model_save_dir: Optional[Union[str, Path, TemporaryDirectory]] = None,
         model_and_config_save_paths: Optional[Dict[str, Tuple[str, Path]]] = None,
+        safety_checker: StableDiffusionSafetyChecker = None,
+        requires_safety_checker: bool = True,
     ):
         """
         Args:
@@ -141,7 +144,8 @@ class NeuronStableDiffusionPipelineBase(NeuronBaseModel):
         self.tokenizer = tokenizer
         self.scheduler = scheduler
         self.feature_extractor = feature_extractor
-        self.safety_checker = None
+        self.safety_checker = safety_checker
+        self.requires_safety_checker = requires_safety_checker
         sub_models = {
             DIFFUSION_MODEL_TEXT_ENCODER_NAME: self.text_encoder,
             DIFFUSION_MODEL_UNET_NAME: self.unet,

Second issue
Then nested behind, another error regarding tensor invalid shapes:
from what I understand this is related to the fact that when exporting the model to Neuron, we need to freeze the shapes in advance and while exporting the model one assume some options to be deactivated (example: guidance_scale /guidance free classification -> during export it is indirectly assumed to be deactivated in inference this one gets activated and the input shape differs from the exported one multiplied by 2 on tensor dim 0)
Reproduce:
Same script as before after applying the previous patch, the new error appears
Error obtained:
RuntimeError:
    Incorrect tensor shape at input tensor #0: received 2 4 64 64, expected 1 4 64 64.
    Incorrect tensor shape at input tensor #1: received 2, expected 1.
    Incorrect tensor shape at input tensor #2: received 2 77 1024, expected 1 77 1024.

Reason of this shape unconsistency here:
https://github.com/huggingface/optimum-neuron/blob/main/optimum/neuron/pipelines/diffusers/pipeline_stable_diffusion.py#L182
do_classifier_free_guidance = guidance_scale > 1.0 -> True, guidance_scale is 7.5 by default
Hence the latent shape on dim 0 should be multiplied by 2 if I understand correctly what's here:
https://github.com/huggingface/optimum-neuron/blob/main/optimum/neuron/pipelines/diffusers/pipeline_stable_diffusion.py#L125https://github.com/huggingface/optimum-neuron/blob/main/optimum/neuron/pipelines/diffusers/pipeline_stable_diffusion.py#L125
Workaround (not a fix either, just to prove the point above):
diff --git a/optimum/neuron/pipelines/diffusers/pipeline_stable_diffusion.py b/optimum/neuron/pipelines/diffusers/pipeline_stable_diffusion.py
index 831676a..d6311c6 100644
--- a/optimum/neuron/pipelines/diffusers/pipeline_stable_diffusion.py
+++ b/optimum/neuron/pipelines/diffusers/pipeline_stable_diffusion.py
@@ -179,7 +180,10 @@ class StableDiffusionPipelineMixin(StableDiffusionPipeline):
         # here `guidance_scale` is defined analog to the guidance weight `w` of equation (2)
         # of the Imagen paper: https://arxiv.org/pdf/2205.11487.pdf . `guidance_scale = 1`
         # corresponds to doing no classifier free guidance.
-        do_classifier_free_guidance = guidance_scale > 1.0
+
+        # do_classifier_free_guidance = guidance_scale > 1.0
+
+        do_classifier_free_guidance = False

         # 3. Encode input prompt
         text_encoder_lora_scale = (

Thank you both for the feedback @oOraph @philschmid, and especially those details on the errors @oOraph !
The first issue, it seems to come from recent changes in diffusers. I will just remove device related code from our custom pipeline, which is useless in our case.
And for the second issue, we need to set the do_classifier_free_guidance=False or turn on the flag --dynamic-batch-size during the export. I will try to optimize that and avoid this occurring.
You can track the patch in this PR: #156. Could you try it out?

Hey @JingyaHuang :), I tried the patch, I can confirm the reported issues are now fixed and inference is working on Inferentia2
But this raises a new question:
Default guidance scale is set to 7.5, meaning that when we perform diffusion inference on a GPU, do_classifier_free_guidance ends up at True
Setting this parameter to True seems to lead to better image generated: closer to the prompt and better image quality (I did not measure it through any clip or fid scores though to prove it, just repeated the operation several times and "looked" at the generated to get this impress). But for Inferentia, it ends up at False for default parameters. So, may be we should enable dynamic batch size at export for Inferentia by default and provide a way to disable it, otherwise images generated with inf2 with default params will have a lower quality than those generated with GPUs.  What do you think ?

@oOraph Have you tried disabling the dynamic batch size but running with two neuon cores by passing device_ids=[0, 1]?
Under that case, even when we do not enable the dynamic batching while compiling, latent_model_input with doubled batch size will be run on two neuron cores, each one with the input of batch size 1. So I wonder would it be better to disable the dynamic batching by default but using two neuron cores by default.

@JingyaHuang  indeed, if I understand correctly what you said above here is what I did:

Reuse my model compiled with dynamic batch size to false
Replace the patched line:

do_classifier_free_guidance = guidance_scale > 1.0 and self.dynamic_batch_size

with the original condition
do_classifier_free_guidance = guidance_scale > 1.0

Run inference by specifying the two neuron as device ids
-> If I do so, I can also confirm inference works and we get back to the do_classifier_free_guidance True case with the best image quality
-> so you're right, using two neurons by default could also be an option

The issue should solved now, although for those default value I need to make a PR to make it more new-user-friendly. I will close this, feel free to reopen it.


creating morphology crashed

Create a new morp
Add pattern
Create
Doesn't display (HTML) or crashed (Local)

closing as it is no longer issue


npm ERR! missing script: start

npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "start"
npm ERR! node v7.7.1
npm ERR! npm  v4.1.2
npm ERR! missing script: start
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\zestard16\Desktop\trello\abc\react-trello-master\npm-debug.log

having posted error while npm run start after npm install, and while npm install having
C:\Users\zestard16\Desktop\trello\abc\react-trello-master\src\components\Board.js
19:9   error  'data' PropType is defined but prop is never used              react/no-unused-prop-types
20:17  error  'onLaneScroll' PropType is defined but prop is never used      react/no-unused-prop-types
21:16  error  'onCardClick' PropType is defined but prop is never used       react/no-unused-prop-types
22:19  error  'eventBusHandle' PropType is defined but prop is never used    react/no-unused-prop-types
23:21  error  'laneSortFunction' PropType is defined but prop is never used  react/no-unused-prop-types
24:14  error  'draggable' PropType is defined but prop is never used         react/no-unused-prop-types
25:20  error  'handleDragStart' PropType is defined but prop is never used   react/no-unused-prop-types
26:18  error  'handleDragEnd' PropType is defined but prop is never used     react/no-unused-prop-types
27:17  error  'onDataChange' PropType is defined but prop is never used      react/no-unused-prop-types
✖ 9 problems (9 errors, 0 warnings)
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "run" "lint"
npm ERR! node v7.7.1
npm ERR! npm  v4.1.2
npm ERR! code ELIFECYCLE
npm ERR! react-trello@ lint: eslint src/**
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the react-trello@ lint script 'eslint src/'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-trello package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     eslint src/
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-trello
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-trello
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\zestard16\Desktop\trello\abc\react-trello-master\npm-debug.log
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"ia32"})
npm ERR! Windows_NT 10.0.14393
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v7.7.1
npm ERR! npm  v4.1.2
npm ERR! code ELIFECYCLE
npm ERR! react-trello@ prepublish: npm run lint && npm run build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the react-trello@ prepublish script 'npm run lint && npm run build'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the react-trello package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm run lint && npm run build
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs react-trello
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls react-trello
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\zestard16\Desktop\trello\abc\react-trello-master\npm-debug.log


 Zephyr replies correctly to the same UDP datagram in IPv4. Attached also Linux pcap for this case.
zephyr-udpv4-checksum-good.gz
zephyr-udpv6-checksum-bad.gz
linux-udpv6-checksum-good.gz

linux-udpv4-checksum-good.gz

It seems that Linux sets the checksums in both cases, IPv4 and IPv6, to 0xffff_

In this case as well as in #16375, I think it is the same root cause in UDP side.


Add content validation

Client::stream_content should validate the content, returning an error after the last chunk if the digest doesn't match.
Could maybe do this as a new PackageSource::validate_stream_content method with default impl.

6231c0512f6a1cd4a1e6fe2a317f0a58298f87ba


boyer_moore algorithm

I have added boyer moore algorithm for pattern searching.

I have added boyer moore algorithm for pattern searching


Make it easy to confirm configuration cache was reused

Expected Behavior
It would be nice if there were an easy way for a build to programmatically confirm that the configuration cache had been reused, and if the cache couldn't be reused, to see a brief reason why.
Current Behavior (optional)
At the moment it seems that it is possible to search the output of Gradle for the message Reusing configuration cache. although hard to be sure that that message will be stable over time (and it also requires some effort to programmatically access the build logs).
Context
In AndroidX we run our CI Gradle builds twice in a row to check for out-of-date tasks. When we updated to Gradle 8.1, we noticed our configuration cache entries weren't being used in the second build anymore. It was sort of a coincidence that we noticed this, because the text of the message was noticed as suspicious by another system.
It could be nice if we could validate that the configuration cache entries get used in our up-to-date builds, to make sure that we can detect configuration cache invalidations even if the text of the message isn't one we've seen before.
This issue is similar to https://github.com/gradle/gradle/issues/26019 since they both relate to the idea of making it easy to verify that the configuration cache wasn't unexpectedly invalidated - implementing one might make the other one less important.
Thanks!

Also interesting - it looks like other workarounds include making a task (which uses as an input the timestamp at which it was configured) for verifying that the configuration cache was reused. See for example https://r.android.com/2951925 . This doesn't necessarily also show the reason that the configuration cache entry wasn't reused, but it does seem to avoid relying on a specific log message to decide whether to fail the build

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team and is prioritized by them.


Rename libreddit to redlib

Is there an existing issue outlining your problem?

[X] I have searched the existing issues and they do not solve my problem.

Describe your issue.
This probably should be a userstyle issue but what the hell.
libreddit is dead. It has been forked off into redlib.
So we should rename the userstyle accordingly
lbl:libreddit

Would it be better to lean into the fork or to have an array of names like anilist?


Adds suport custom collect API endpoint

As discussed in #1255 and https://github.com/umami-software/umami/discussions/1026#discussioncomment-2624639, ad-blockers are blocking API calls to /api/collect endpoint.
This PR adds support for custom collect API endpoint using Next.js middleware.
A new environment variable COLLECT_API_ENDPOINT has to be added to enable this feature.
Example variable assignments,
COLLECT_API_ENDPOINT = '/api/endpoint'

COLLECT_API_ENDPOINT = '/endpoint'

Can we get a new release please?


Add package.json

Add package.json to fix error install from cordova 7

Thanks for the PR! I left one tiny comment. If you could address that, we would love to merge this 👍

Updated.

Thanks!


Use groupified OCP API

oapi is deprecated and will be removed in OCP4

OSBS-5497

Signed-off-by: Martin Bašti<EMAIL_ADDRESS>Maintainers will complete the following section:

[x] Commit messages are descriptive enough
[x] "Signed-off-by:" line is present in each commit
[x] Code coverage from testing does not decrease and new code is covered
[x] JSON/YAML configuration changes are updated in the relevant schema
[x] Changes to metadata also update the documentation for the metadata
[x] Pull request includes link to an osbs-docs PR for user documentation updates
[x] New feature can be disabled from a configuration file

LGTM

Waiting with merging for osbs-client counterpart  https://github.com/containerbuildsystem/osbs-client/pull/878

No RN, just small doc change


UTF-8 encoded content files with Byte Order Mark break metadata parser

When you save a content file (including the metadata section) encoded as UTF-8 with Byte Order Mark (BOM) and recompile the site all your metadata will end up rendered in the layout.
I'm sorry I can't provide a failing test as I'm a noob Ruby developer, but I would really love to see this issue fixed.

Good catch. I’ve fixed the issue as you can see, but can you check out the release-3.2.x branch and test it to make sure it works for you as well?

It works (given I set the LANG as stated in issue #50)! Thank you for quickly providing a fix!


CARLA Simulator Timeouts for Data Collection

I'm running the Data Collection scenario, and I've set the WANDB syncing to offline.
The commands I ran to start data collection:
export WANDB_MODE=offline
bash run/data_collect.sh /home/user1/carla_0_9_11/CarlaUE4.sh /home/user1/mile/dataset_collect 2000 lb_data

However, I receive the following Runtime Error regarding the simulator timing out
RuntimeError: time-out of 60000ms while waiting for the simulator, make sure the simulator is ready and connected to localhost:2000
Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.

wandb: Waiting for W&B process to finish... (failed 1).
wandb: You can sync this run to the cloud by running:
wandb: wandb sync /home/user1/mile/outputs/2024-05-01/18-16-17/wandb/offline-run-20240501_181626-213560f2
wandb: Find logs at: ./wandb/offline-run-20240501_181626-213560f2/logs
PYTHON_RETURN=1!!! Start Over!!!

Does the data collection scenario require me to run it with a GUI? I've SSH'd into my machine, so it's running the scenario in headless mode.

Hello,
You do no need to run CARLA with a GUI, I've also collected the data by ssh'ing into my machine and ran the script from there. I've also encountered that error in the past, here are some possible solutions:

For some unknown reason, sometimes simply retrying to launch the script fixes it..
Is your port 2000 free and unused?

Hi Anthony,
My port 2000 is free and unused. I tried to use a different port too but still receiving the same error after multiple tries.
Not too sure if this matters, but does wandb syncing need to be set to online? I've set it to offline.

I've picked up the same project as Jordan. Same error online:
(mile) dihadley3-adas-0614@dihadley3-adas-0614:~/mile$ bash run/data_collect.sh $CARLA_ROOT/CarlaUE4.sh /home/dihadley3-adas-0614/mile_data/ 2000 lb_data
rm: cannot remove 'outputs/port_2000_checkpoint.txt': No such file or directory
rm: cannot remove 'outputs/port_2000_ep_stat_buffer_*.json': No such file or directory
[2024-06-14 19:46:51,891][utils.server_utils][INFO] - Killed Carla Servers on port 2000!
[2024-06-14 19:46:52,902][utils.server_utils][INFO] - Killed Carla Servers on port 2000!
[2024-06-14 19:46:53,907][utils.server_utils][INFO] - CUDA_VISIBLE_DEVICES=None bash /home/dihadley3-adas-0614/carla_0_9_11/CarlaUE4.sh -fps=25 -quality-level=Epic -carla-rpc-port=2000
4.24.3-0+++UE4+Release-4.24 518 0
Disabling core dumps.
wandb: (1) Create a W&B account
wandb: (2) Use an existing W&B account
wandb: (3) Don't visualize my results
wandb: Enter your choice: 2
wandb: You chose 'Use an existing W&B account'
wandb: Logging into wandb.ai. (Learn how to deploy a W&B server locally: https://wandb.me/wandb-server)
wandb: You can find your API key in your browser here: https://wandb.ai/authorize
wandb: Paste an API key from your profile and hit enter, or press ctrl+c to quit:
wandb: Appending key for api.wandb.ai to your netrc file: /home/dihadley3-adas-0614/.netrc
[2024-06-14 19:47:21,435][mile.agents.rl_birdview.rl_birdview_agent][INFO] - Resume checkpoint latest ckpt/ckpt_11833344.pth
[2024-06-14 19:47:22,191][mile.agents.rl_birdview.rl_birdview_agent][INFO] - Loading wandb checkpoint: ckpt/ckpt_11833344.pth
[2024-06-14 19:47:22,389][__main__][INFO] - Start from env_idx: 0, task_idx 0
wandb: Currently logged in as: dianehadley27 (diane-hadley). Use `wandb login --relogin` to force relogin
wandb: wandb version 0.17.1 is available!  To upgrade, please run:
wandb:  $ pip install wandb --upgrade
wandb: Tracking run with wandb version 0.13.9
wandb: Run data is saved locally in /home/dihadley3-adas-0614/mile/outputs/2024-06-14/19-46-51/wandb/run-20240614_194722-9sgwl85s
wandb: Run `wandb offline` to turn off syncing.
wandb: Syncing run mile_data
wandb: ⭐ View project at https://wandb.ai/diane-hadley/mile
wandb: 🚀 View run at https://wandb.ai/diane-hadley/mile/runs/9sgwl85s
Error executing job with overrides: ['carla_sh_path=/home/dihadley3-adas-0614/carla_0_9_11/CarlaUE4.sh', 'dataset_root=/home/dihadley3-adas-0614/mile_data/', 'port=2000', 'test_suites=lb_data']
Traceback (most recent call last):
  File "data_collect.py", line 175, in main
    env = gym.make(env_setup['env_id'], obs_configs=obs_configs, reward_configs=reward_configs,
  File "/home/dihadley3-adas-0614/miniconda3/envs/mile/lib/python3.8/site-packages/gym/envs/registration.py", line 145, in make
    return registry.make(id, **kwargs)
  File "/home/dihadley3-adas-0614/miniconda3/envs/mile/lib/python3.8/site-packages/gym/envs/registration.py", line 90, in make
    env = spec.make(**kwargs)
  File "/home/dihadley3-adas-0614/miniconda3/envs/mile/lib/python3.8/site-packages/gym/envs/registration.py", line 60, in make
    env = cls(**_kwargs)
  File "/home/dihadley3-adas-0614/mile/carla_gym/envs/suites/endless_env.py", line 10, in __init__
    super().__init__(carla_map, host, port, seed, no_rendering,
  File "/home/dihadley3-adas-0614/mile/carla_gym/carla_multi_agent_env.py", line 31, in __init__
    self._init_client(carla_map, host, port, seed=seed, no_rendering=no_rendering)
  File "/home/dihadley3-adas-0614/mile/carla_gym/carla_multi_agent_env.py", line 155, in _init_client
    self._world = client.load_world(carla_map)
RuntimeError: time-out of 60000ms while waiting for the simulator, make sure the simulator is ready and connected to localhost:2000

But I think the error may be with the CARLA package install:
(mile) dihadley3-adas-0614@dihadley3-adas-0614:~/mile$ easy_install /home/dihadley3-adas-0614/carla_0_9_11/PythonAPI/carla/dist/carla-0.9.11-py3.7-linux-x86_64.egg
WARNING: The easy_install command is deprecated and will be removed in a future version.
Processing carla-0.9.11-py3.7-linux-x86_64.egg
creating /home/dihadley3-adas-0614/miniconda3/envs/mile/lib/python3.8/site-packages/carla-0.9.11-py3.7-linux-x86_64.egg
Extracting carla-0.9.11-py3.7-linux-x86_64.egg to /home/dihadley3-adas-0614/miniconda3/envs/mile/lib/python3.8/site-packages
Adding carla 0.9.11 to easy-install.pth file

Installed /home/dihadley3-adas-0614/miniconda3/envs/mile/lib/python3.8/site-packages/carla-0.9.11-py3.7-linux-x86_64.egg
Processing dependencies for carla==0.9.11
Searching for carla==0.9.11
Reading https://pypi.org/simple/carla/
No local packages or working download links found for carla==0.9.11
error: Could not find suitable distribution for Requirement.parse('carla==0.9.11')

This Carla 0.9.11 package depends on Python 3.7 but this repo uses Python 3.8. Should I be using a different CARLA egg file for 3.8?

Hello Diane,
The carla package error is more of a pip warning. It is trying to find the 0.9.11 version on pip to find out dependencies on this package but does not find it. The carla package itself however should be correctly installed.
Have you tried running the CARLA simulator by itself? Can you see the CARLA graphical interface and move in the environment?

Hi @anthonyhu,
I was able to run the data collection scenario. The solution was switching to a machine that has a dedicated NVIDIA GPU with at least 6gb VRAM to be able to run the CARLA simulator.
Do you have any recommendations on which test suite to use that doesn't take too long to collect the data where I can still run the training scenario?
Initially, I ran the data collection scenario with the lb_data test suite but it takes a while.

Great to see you made it work. If collecting data takes a while, you can limit yourself to Town01 for experimentation (https://github.com/wayveai/mile/blob/main/config/test_suites/lb_town01.yaml), and only evaluate on Town02 (by removing Town05 in the lb_test.yaml, which is a smaller version of Town01 with a different layout).


Enum injection

Hi guys,
These are the supported values:
java.lang.String
• java.lang.Long
• java.lang.Boolean
• java.util.Set<java.lang.String>
• javax.json.JsonValue.TRUE/FALSE
• javax.json.JsonString
• javax.json.JsonNumber
• javax.json.JsonArray
• javax.json.JsonObject
Shouldn't we also support Enum classes? Strings could be transformed to a Enum using valueOf if the injection point is an Enum class.
What do you guys think?

We should have identical conversion capabilities as the Config API.  Last I checked they were something like the following:

public TheType(String) produce an instance using public string constructor on the type itself.
public static TheType parse(String) produce an instance using a static factory on the type itself.

There's an order of precedence and if I recall correctly at least one supported more static factory pattern.

I think this makes sense.

Is it a duplicate of #100 ?

Hi @ivanjunckes all, copying your requirement to #100, to keep it all centralized in one place;
CC @dblevins, @radcortez
thanks


Add resend verification email function

This is for the dashboard.
Not sure whether auth.ts is the right file to put this :/ Help! :smile:

Looks good! Would put it lobby/index.ts instead eyes

Ah! Perfect. Thanks.


Merge core

Merge https://github.com/src-d/core into https://github.com/src-d/core-retrieval
Now src-d/core is used exclusively by Data Retrieval projects so we merge both together under a single project and maintainership.

We should also decide if we want the package name to be core_retrieval or just core.


Add warning for not having .Net Framework 3.5 installed

I've asked the developer of sdb2xml to change their .NET requirements for the application. .NET 3.5 is quite old, so requiring something newer only makes sense. Understandably, if you're running this internally from a single device you could just have .NET 3.5 on the one and not all of them. However, in an MSP environment, that's not at all possible (or at least a bit more difficult to control). It seems more that the intention of his application was to be run repeatedly on a single device, perhaps, so it would make sense that you can control the framework more, but hopefully he'll be willing to modernize the application to support newer, native .NET Framework installs.

With help from the developer of sdb2xml, all you need to do is add a config file to the same directory as the sdb2xml executable (named "sdb2xml.exe.config"). This will get rid of the .NET warning.
Here is my config file:
<?xml version="1.0"?>
<configuration>
  <startup>
    <supportedRuntime version="v4.0"/>
  </startup>
</configuration>

Timothy

It appears I spoke too soon. The config file will silence the prompt to install .NET 3.5, but the application does not run. Not sure where to go from here.
Timothy

Just deploy .net 3.5 like every other org on earth?


update readme to support gradle as well

This way I got it running with gradle.

That's even better. Thanks a lot.


Add an insights page

Would be great to have an 'Insights' page.  Similar to 18F:  https://micropurchase.18f.gov/insights (LW)

Is this html copyright or forkable? // cc @markwooff

Found their old insights page: https://micropurchase.18f.gov/insights.html

Hi all, I've added this to a new project Kanban board (on this repo) with some other future enhancements.  Thinking of closing this as an issue unless there are any objections? @MaryBethBaker  @markwooff


[Feature Request] Custom Events Support

Use case
I'm trying to implement Amazon Publisher Services Ads for a Flutter app but I don't see anywhere that custom events are supported.
Proposal
Something similar to getMediationExtras perhaps, but it would need to be able to call addCustomEventExtrasBundle on the AdRequest so that I can attach some parameters to the AdRequest and get Amazon ads.

Hi @Reprevise
I'm not sure about the feasibility of this feature since I see there is only a identifier parameter for custom event on Flutter plugin and Android native plugin:

Flutter: https://github.com/googleads/googleads-mobile-flutter/blob/184f5e8e4a1a9f520d777092f063980d63f7167f/packages/google_mobile_ads/lib/src/ad_containers.dart#L231

Android: https://github.com/googleads/googleads-mobile-android-mediation/blob/ddd519d92a227034e956037d3f8c4a7623a5bf08/Example/customevent/src/main/java/com/google/ads/mediation/sample/customevent/SampleBannerCustomEventLoader.java#L70-L71

Labeling the issue for further inside from the team!

@Reprevise, how much revenue is being blocked for your business by Google not supporting Custom Events in the plugin yet? Can you share your app store page? We will use this information to determine how to prioritize this request.
If you want to share this info privately, feel free to send me an email -><EMAIL_ADDRESS>

Hey there, I'm on @Reprevise's team building Tracker Network Stats. You can find our app on the App Store and Google Play:
https://apps.apple.com/ca/app/tracker-network-stats/id1287696482
https://play.google.com/store/apps/details?id=network.tracker.app
In terms of a revenue estimate.. it's hard to say, could be a 5 figure difference. It just depends on what competitive edge mediators like Amazon can provide. We understand the importance of mediation on Admob and in advertising in general, so we're a bit surprised to see it isn't supported.

We’re closing this issue due to inactivity. If you’re still impacted, please create a new issue via the Developer Forum.