Framework - Add Socket Support

This is a lower priority, but including this ticket for tracking.

A majority of the initial groundwork has been complete for this. Some things that are still outstanding though;

Keep alive message must be sent in order to keep the socket open. This should be sent at a regular interval, the server will close connections after around 5 mins of inactivity.

Added support for the keep alive messages in API #194

Assuming this is working as expected based on Oconn's comments, moving to closed.


Dates and Times are not datetime.datetime

Issue Description
for things like Submission.created_utc and ModmailConversation.last_updated and similar, it takes a bit of an effort to convert the provided items into something usable
# for Submission et al.
time = datetime.datetime.fromtimestamp(Submission.created_utc, tz=datetime.timezone.utc)
# for ModmailConversation et al.
# '2015-12-23T11:11:11.111111+00:00'
time = ModmailConversation.last_updated
time = datetime.datetime.strptime(time[:-3]+time[-2:], '%Y-%m-%dT%H:%M:%S.%f%z')

Both incantations are a bit unweildly if used inline, and would take up >8 extra lines if moved into functions (assuming proper coding style) or result in creation of yet another dreaded utils.py file!
It would be nice if this work is moved into the praw (or prawcore) library, instead of re-created by each user of the praw library (at least when those attributes are used)
perhaps under Submission.created_dt and ModmailConversation.last_updated_dt to avoid backwards compatibility issues?
System Information
PRAW Version: 5.0.1
Python Version: 3.6.X
Operating System: Windows for Workgroups 3.11

@Summertime you have a good suggestion as this might make working with PRAW a little easier. However, I am a little weary of modifying attributes returned from PRAW that don't represent one of PRAW's existing models. This is in effort to try to keep PRAW's functionality relatively minimal.
However, I am very open to the idea of addon-packages that dynamically add additional attributes. In fact, the prawdditions package does just that: https://github.com/praw-dev/prawdditions
Consider opening a PR there to add this functionality. Thanks for the suggestion.


md-sidenav-layout hide all my content

When I use  all my content is hidden, why ?!!
In this GIF you can check my litle strucutre od DOM (ide + chrome inspector)

Not enough information. Please follow the issue template.


successful signup functionality and debugging in session controller

Overview

signup functionality has been added and is working
fixed server-side bug for signup: session is now sending res.locals.sessionInfo rather than an html file
MyTrips component is in progress, still need to render actual trip data into the trip component
added pop-up alerts for both successful and unsuccessful user registration

Screenshots
-example of successful login
https://user-images.githubusercontent.com/99763928/177458009-41f3d01e-c073-44ba-a3c2-27914f9bc79b.mov

Love the video preview - very helpful.


improve Add Note button

The Add Note button is not looking good as Add and Note are coming in two different lines.
Also, copyright text is not looking good on the left, either aligning it to the center will be good or removing it.
Assign me to fix it.

I want to solve this bug. can you allow me ??

I think after hovering on Add Note button ,it does not change any transion like color etc. if we add feature on it . then it definatly looks awesome . and in this repo there is no copyright section. if we add it . then definatly increase the dashboard looks.
i want to contribute in this repo. can you allow me to do the above part.


[prim] Move unrelated constants out of prim_pkg.sv

prim_pkg.sv will be auto-generated and cannot be used as common file
for constants. Put these constants into a primitives-specific package.
The toplevel uses these constants, therefore we need to include the
relevant core there.
This is in preparation for the dynamic generation of primitives from technology libraries.

Indeed, these have to move, thanks for the cleanup.
I believe there are some more instances of prim_pkg in the FPV testbenches under opentitan/hw/ip/prim/fpv/, could you rename them as well?

I updated the patch to catch those cases (and a couple more) as well, let's hope I got them all now.


[KARAF-6408] Upgrade to Spring 4.3.25.RELEASE

retest this please

retest this please

retest this please


Fix code scanning alert - Runs with a root primary or supplementary GID

Tracking issue for:

[ ] https://github.com/bcgov/nr-spar/security/code-scanning/52

Handled by OpenShift.  Change not required.


Missing VCRUNTIME140_1.dll in 0.7.1 x64

Bug while using the x64 version of the lateast release, i can't launch the programm because VCRUNTIME140_1.dll missing.
x32 works fine :)

Fixed by installing
https://aka.ms/vs/16/release/vc_redist.x64.exe


Eric Lopez

It was meant to communicate it was man for a world wide communication to establish freedom the speech and freedom of expression to expand knowledge beyond our imagination and share equally beyond our limits that is to be fair and equal let's not try to be better than anybody let's be equal to everybody.

Let's not hide anything from the people let's not be better than anybody else this show equalty amongst each other nobody is different we're all the same so should we all be treated the same amen.

oooookay


Grid view not accessible, would like list view to persist

When in the library view/bookshelf, the default seems to be the grid view. This says cover for each entry. I select the list view and can walk through the titles in the Bookshelf. However, when I reopen Thorium, it reverts back to grid view. I would like to have the list view to persist, or perhaps a setting that allows the person to select grid view or list view.

Hi George, it looks like you closed this issue unintentionally after posting it?
First and foremost, we must ensure that the grid view is accessible. In my tests with VoiceOver, the title and author metadata is announced when browsing the cover images in the library's grid view. Maybe this is a problem with NVDA, let me check on Windows ...

I am not able to reproduce this bug on Windows 10 with NVDA either. Could you please confirm whether this problem occurs with the Windows Store version of Thorium, or the more up to date automated build from https://github.com/readium/readium-desktop/releases ?

Hi,
I did not mean to close this item.
I have replicated on both the Microsoft store version and the one with shortcut key modification support.
In Grid view, I get cover  image and then a graphic to expand. When expanded I have about, delete, and export. If I activate about, I get what I think is the information about this book.
Best
George
From: Daniel Weck<EMAIL_ADDRESS>Sent: Tuesday, March 10, 2020 2:50 PM
To: readium/readium-desktop<EMAIL_ADDRESS>Cc: George<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [readium/readium-desktop] Grid view not accessible, would like list view to persist (#968)
I am not able to reproduce this bug on Windows 10 with NVDA either. Could you please confirm whether this problem occurs with the Windows Store version of Thorium, or the more up to date automated build from https://github.com/readium/readium-desktop/releases ?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/readium/readium-desktop/issues/968?email_source=notifications&email_token=ABW4OSGZRH6RTBWBRITGQK3RG2RYHA5CNFSM4LDT6E32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEONDBSI#issuecomment-597307593 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABW4OSA55CMTNL2CIRLJJJLRG2RYHANCNFSM4LDT6E3Q .

Hi George, there is definitely an accessible label announcing publication title and authors, when keyboard-tabbing on the cover image link. However, there might also be some confusion because of the screen reader delay between announcing the generic cover image link, and its accessible label, because of nested link image. So, I have fixed that in the latest build, the announcement is now immediate (tested with Voice Over on Mac,  as well as NVDA on Windows).

Hello,
Grid view is working with build 426!!!
Best
George
From: Daniel Weck<EMAIL_ADDRESS>Sent: Wednesday, March 11, 2020 3:33 AM
To: readium/readium-desktop<EMAIL_ADDRESS>Cc: George<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [readium/readium-desktop] Grid view not accessible, would like list view to persist (#968)
Hi George, there is definitely an accessible label announcing publication title and authors, when keyboard-tabbing on the cover image link. However, there might also be some confusion because of the screen reader delay between announcing the generic cover image link, and its accessible label, because of nested link image. So, I have fixed that in the latest build, the announcement is now immediate (tested with Voice Over on Mac, as well as NVDA on Windows).
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/readium/readium-desktop/issues/968?email_source=notifications&email_token=ABW4OSGKGSPVT2HZ54FSBKDRG5LDDA5CNFSM4LDT6E32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOOZSLY#issuecomment-597530927 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABW4OSDPIXV4PU3WBWMWCNTRG5LDDANCNFSM4LDT6E3Q .


"left behind trailing whitespace" in long patterns

Code like this:
fn put_telescope_synctocoordinates(
    PutTelescopeSynctocoordinatesPathParams {

                        device_number,

                }: PutTelescopeSynctocoordinatesPathParams,
) {
}

triggers "left behind trailing whitespace".
I suppose it's because rustfmt tries to put entire param - both pattern and type - on the same line, fails in doing so as that exceeds the max width, and gives up. Instead, it should probably fallback to something like
fn put_telescope_synctocoordinates(
    PutTelescopeSynctocoordinatesPathParams {
      device_number,
    }: PutTelescopeSynctocoordinatesPathParams,
) {
}

Thanks for the report!.
I believe your assessment is correct and we're trying to put the pattern and the type on the same line. After looking into it I think what's happening is there's enough space to write the pattern on a single line so we format it on one line, but there isn't enough space to put both the pattern and the type on one line so rewriting fails, which leads to returning the span unchanged. I did some digging to try and pin that down:
starting at rewrite_fn_base, we eventually call rewrite_params
https://github.com/rust-lang/rustfmt/blob/ef91154250977b3b5d05448dafbca524a1168b47/src/items.rs#L2287-L2296
rewrite_params then calls the Rewrite impl for ast::Param, and if it fails to format the param we just return the original span unchanged, which explains the left behind trailing whitespace issue.
https://github.com/rust-lang/rustfmt/blob/ef91154250977b3b5d05448dafbca524a1168b47/src/items.rs#L2595-L2597
For anyone interested in taking this on I'd start looking at the is_named_param if-else block in the ast::Param Rewrite impl.
https://github.com/rust-lang/rustfmt/blob/ef91154250977b3b5d05448dafbca524a1168b47/src/items.rs#L2035-L2088
I'd assume we'll also need to make some changes to rewrite_struct_pat, which gets called as a result of rewriting a struct pattern (the self.pat.rewrite call above Line 2036).
https://github.com/rust-lang/rustfmt/blob/ef91154250977b3b5d05448dafbca524a1168b47/src/patterns.rs#L258-L260

Not sure if this is the right issue to add this to, but "if-let" chains are also "long patterns" and seem to create this issue that breaks formatting (yes, I know the actual formatting of said elements is undecided, but the rest of the formatting died as a result). For example:
            if let Some(ref r) = reverse_regex
                && let Some(c) = r.captures(s)
                && let Some(m) = c.get(0)
            {

Errors out as:
error[internal]: left behind trailing whitespace
   --> /path/to/file.rs:335:335:47
    |
335 |             if let Some(ref r) = reverse_regex
    |                                               ^
    |

error[internal]: left behind trailing whitespace
   --> /path/to/file.rs:336:336:65
    |
336 |                 && let Some(c) = r.captures(s)
    |                                                                 ^
    |

error[internal]: left behind trailing whitespace
   --> /path/to/file.rs:337:337:42
    |
337 |                 && let Some(m) = c.get(0)
    |                                          ^

Formatting works after the trailing spaces are deleted. Using: rustfmt 1.5.1-nightly (a6b7274 2022-10-10)

@sammysheep - thanks for sharing but that's unrelated to this issue so I'm going to minimize that comment as well as this one to keep the issue focused.
It's not that formatting is "broken" in your case; rustfmt is simply leaving exactly what you wrote in the first place, and if you happened to have trailing spaces in your original code then rustfmt is noting that to you for awareness. This happens in many cases, including the one that's the topic of this issue, but also in cases where there are no defined formatting rules, as in let chains which is off topic here.

I have a ¨left behing trailing whitespace" in a let statement, rustc 1.65.0, on arch armv7l, a raspberry pi.
The offending code has a whitespace just after the final ) on the first code line, with the else part added here to the error message:
error[internal]: left behind trailing whitespace
--> /home/ype/p1encrypt-rs/src/p1input.rs:159:159:58
    let Some(first_open_bracket_pos) = line.find('(')
    else {
        return Err("missing open bracket".to_string());
    };

A commit with the offending the code is on gitlab:
https://gitlab.com/enmprivaggr/enmprv/-/commit/437c1bf1e158f7fc5e3aa03cd91ffa566e0d8de2
Without the backspace, the code formats just fine:
https://gitlab.com/enmprivaggr/enmprv/-/commit/407661172e63bffd13c0c95ff5a9105440e106c9
Does that fit here, or does this look like a new issue?

Looking at the previous comment of @calebcartwright this is off topic here.
I'd expect end of line whitespace to be a target for change by fmt, so I think this is a case of a missing format rule.
Would that qualify as a new issue?


As a [end user], I want a checkbox in order to indicate that a receiving address is an exchange address

Problem
If a transaction exceeds slippage, the user would receive Next assets. If the receiving address is an exchange deposit address, the user is likely to lose their funds because they won't recognize the Next assets.
Proposed Solution
Ideal solution would be a checkbox that allows for a callback to happen if slippage is too high, and returns the funds to origin. But that involves a lot of dev work.
In the meantime, I suggest having a checkbox that when checked displays a message about the risk that exceeding slippage presents when sending to a CEX deposit address.
Acceptance Criteria
A simple checkbox that is able to display a message.

We have two options

bump slippage (waiting for eng capacity) https://github.com/connext/nxtp/issues/2094
receive local nextAsset (debating if we even allow this still because UX is so bad)

Does this hinge on if we allow user to choose to receive local? I think it's a great callout if it's to Coinbase or w/e and they get fucked
I would love to revert to sending wallet as you noted it was too complex

@Tommy-R8 are you aware of a way to see if a receiving address is an exchange deposit address?

I'm not aware of a way to see this but a disclaimer and a checkbox when the user fills in the receiving address field should suffice

@Tommy-R8 I'm not sure, how can we detect an exchange address?
Also, referring to the issue problem, the user will be able to change the slippage or revert the transfer if required. Both issues are in pipeline. Starting with slippage bump implementation.
closing this issue for now, let's re-open it if required.


Add trouble-shooting item about out of memory errors

Pulling for example the TensorFlow image from NGC with the default amount of memory for a Slurm job (10GB per CPU) results in out-of-memory errors. A solution is to increase the amount of memory, e.g. "--mem 32G" seems to be sufficient for the TensorFlow container.

A better solution is now available in "docs/singularity.md"


Katex parse error with VS code 1.85.1

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version:  1.85.1
OS Version: Windows 10 build 19045. 3803

Steps to Reproduce:

write this in a markdown file:

### Householder transformations
We seek an orthogonal transformation which annihilates targeted components of a vector. This is achieved by the Householder matrix

\begin{equation}
\mathbf{H}=\mathbf{I}-2\frac{\mathbf{v}\mathbf{v}^T}{\mathbf{v}^T\mathbf{v}}
\end{equation}
with $\mathbf{v}$ a nonzero vector.
It can be shown that $\mathbf{H}=\mathbf{H^{-1}}=\mathbf{H^T}$, which means that $\mathbf{H}$ is orthogonal and symmetric.

#### Annihilating all but the first component of a vector
We want to determine $\mathbf{v}$ such that it annihilates all the components of a vector $\mathrm{a}$ except the first, i.e.

\begin{equation}
\mathbf{Ha}=\begin{bmatrix}\alpha\\0\\\vdots\\0\end{bmatrix}=\alpha\begin{bmatrix}1\\0\\\vdots\\0\end{bmatrix}=\alpha\mathbf{e}_1
\end{equation}

Using the definition of $\mathbf{H}$ we find

\begin{equation}
\alpha\mathbf{e}_1=\mathbf{Ha}=\left(\mathbf{I}-2\frac{\mathbf{v}\mathbf{v}^T}{\mathbf{v}^T\mathbf{v}}\right)\mathbf{a}
=\mathbf{a}-2\mathbf{v}\frac{\mathbf{v}^T\mathbf{a}}{\mathbf{v}^T\mathbf{v}}
\end{equation}

and thus

\begin{equation}
\mathbf{v}=(\mathbf{a}-\alpha\mathbf{e}_1)\frac{\mathbf{v}^T\mathbf{v}}{2\mathbf{v}^T\mathbf{a}}
\end{equation}

The scalar factor is irrelevant as it cancels out in the expression for $\mathbf{H}$, so we find

\begin{equation}
\mathbf{v}=(\mathbf{a}-\alpha\mathbf{e}_1)
\end{equation}

To preserve the norm and avoid cancellation
\begin{equation}
\alpha=-\mathrm{sign}(a_1)\|\mathbf{a}\|_2
\end{equation}

I get this katex parse error (added as a screenshot).

If I use jupyter lab with the same block i don't get any katex parse error, but I would like to use VS code,
Thanks in advance for your help,
Regards
### Tasks

Try this in a markdown cell in a notebook. We currently only enable bare math block (without using $$) in notebooks

works great in a notebook! Nice support addition, I used to use this a lot in college.


go/runtime/txpool: Remove rechecked transactions from seen cache

In case a transaction is rejected because it fails a re-check pass, it should also be removed from the seen cache as it may be resubmitted later when it could become valid.

Why is this better than submitting a new transaction?

Because the same transaction may become valid in the future. The future may also be due to the async nature of the network where some nodes may be slightly behind others.

Do we simplify DoS attacks with these changes?

Just in the sense that there may be more load on transaction checks (which are very fast). But someone can already generate slightly different transactions and spam those.

Is it possible that rejected transactions will travel from node to node and never be removed from the network?

Nodes only forward transactions that pass checks. Also the p2p layer has its own deduplication logic.

Do we need to limit the number of retries?

You mean like having a separate cache that would count the number of times a tx hash was rejected and after a certain amount just immediately reject without passing any checks? We could consider this if it becomes a problem in practice.

some nodes may be slightly behind others.

This makes sense.

But someone can already generate slightly different transactions and spam those.

If he takes this action now, the impact will be more significant, as the rejected transactions will accumulate in the network. However, according to your answer, these transactions will not be forwarded so I guess this is not a problem.

You mean like having a separate cache that would count the number of times a tx hash was rejected and after a certain amount just immediately reject without passing any checks?

Yes.

We could consider this if it becomes a problem in practice.

Ok.


Pluralsight appears to be broken

So if I do:
tim@picknmix:~/Videos$ youtube-dl --verbose<EMAIL_ADDRESS>--password=nevergonnagiveyouup http://www.pluralsight.com/courses/angularjs-get-started
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose'<EMAIL_ADDRESS>u'--password=nevergonnagiveyouup', u'http://www.pluralsight.com/courses/angularjs-get-started']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.11.15
[debug] Python version 2.7.3 - Linux-3.2.0-4-686-pae-i686-with-debian-7.9
[debug] exe versions: avconv 0.8.17-6, avprobe 0.8.17-6, ffmpeg 0.8.17-6, ffprobe 0.8.17-6
[debug] Proxy map: {}
[pluralsight:course] angularjs-get-started: Downloading course JSON
[pluralsight:course] angularjs-get-started: Downloading course data JSON
[download] Downloading playlist: AngularJS: Get Started
[pluralsight:course] playlist AngularJS: Get Started: Collected 44 video ids (downloading 44 of them)
[download] Downloading video 1 of 44
[pluralsight] Downloading login page
[pluralsight] Logging in as<EMAIL_ADDRESS>[pluralsight] angularjs-get-started-m1-introduction-0: Downloading webpage
ERROR: Unable to extract modules; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 661, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 290, in extract
return self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/pluralsight.py", line 90, in _real_extract
webpage, 'modules'),
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 584, in _search_regex
raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract modules; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
If I reference the video more directly, the URL isn't detected:
tim@picknmix:~/Videos$ youtube-dl --verbose<EMAIL_ADDRESS>--password=nevergonnagiveyouup "https://app.pluralsight.com/training/player?course=angularjs-get-started&author=scott-allen&name=angularjs-get-started-m1-introduction&clip=0&mode=live"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose'<EMAIL_ADDRESS>u'--password=nevergonnagiveyouup', u'https://app.pluralsight.com/training/player?course=angularjs-get-started&author=scott-allen&name=angularjs-get-started-m1-introduction&clip=0&mode=live']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.11.15
[debug] Python version 2.7.3 - Linux-3.2.0-4-686-pae-i686-with-debian-7.9
[debug] exe versions: avconv 0.8.17-6, avprobe 0.8.17-6, ffmpeg 0.8.17-6, ffprobe 0.8.17-6
[debug] Proxy map: {}
[generic] player?course=angularjs-get-started&author=scott-allen&name=angularjs-get-started-m1-introduction&clip=0&mode=live: Requesting header
WARNING: Falling back on generic information extractor.
[generic] player?course=angularjs-get-started&author=scott-allen&name=angularjs-get-started-m1-introduction&clip=0&mode=live: Downloading webpage
[generic] player?course=angularjs-get-started&author=scott-allen&name=angularjs-get-started-m1-introduction&clip=0&mode=live: Extracting information
[debug] Invoking downloader on u'https://app.pluralsight.com/training/player?course=angularjs-get-started&author=scott-allen&name=angularjs-get-started-m1-introduction&clip=0&mode=live'
[download] Destination: Pluralsight Player-playercourse=angularjs-get-started&author=scott-allen&name=angularjs-get-started-m1-introduction&clip=0&mode=live.unknown_video
[download] 351.9% of 21.04KiB at  6.21MiB/s ETA 00:00ERROR: content too short (expected 21546 bytes and served 75824)
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1598, in process_info
success = dl(filename, info_dict)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1540, in dl
return fd.download(name, info)
File "/usr/local/bin/youtube-dl/youtube_dl/downloader/common.py", line 342, in download
return self.real_download(filename, info_dict)
File "/usr/local/bin/youtube-dl/youtube_dl/downloader/http.py", line 238, in real_download
raise ContentTooShortError(byte_counter, int(data_len))

I am getting the error "no video formats found"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', 'http://www.pluralsight.com/courses/skeet-async', '--username', 'PRIVATE', '--password', 'PRIVATE']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2015.11.15
[debug] Python version 3.4.3 - Windows-8-6.2.9200`
[debug] exe versions: none
[debug] Proxy map: {}
[pluralsight:course] skeet-async: Downloading course JSON
[pluralsight:course] skeet-async: Downloading course data JSON
[download] Downloading playlist: Asynchronous C# 5.0
[pluralsight:course] playlist Asynchronous C# 5.0: Collected 19 video ids (downloading 19 of them)
[download] Downloading video 1 of 19
[pluralsight] Downloading login page
[pluralsight] Logging in as<EMAIL_ADDRESS>[pluralsight] skeet-async-m1-intro-0: Downloading webpage
[pluralsight] skeet-async-m1-intro-0: Downloading webm-high URL
WARNING: Unable to download webpage: HTTP Error 400: Bad Request
[pluralsight] skeet-async-m1-intro-0: Downloading mp4-low URL
WARNING: Unable to download webpage: HTTP Error 400: Bad Request
[pluralsight] skeet-async-m1-intro-0: Downloading mp4-medium URL
WARNING: Unable to download webpage: HTTP Error 400: Bad Request
[pluralsight] skeet-async-m1-intro-0: Downloading mp4-high URL
WARNING: Unable to download webpage: HTTP Error 400: Bad Request
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Python34\lib\site-packages\youtube_dl\YoutubeDL.py", line 661, in extract_info
ie_result = ie.extract(url)
File "C:\Python34\lib\site-packages\youtube_dl\extractor\common.py", line 290, in extract
return self._real_extract(url)
File "C:\Python34\lib\site-packages\youtube_dl\extractor\pluralsight.py", line 149, in _real_extract
self._sort_formats(formats)
File "C:\Python34\lib\site-packages\youtube_dl\extractor\common.py", line 766, in _sort_formats
raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Looks like pluralsight changed all their links to APP.pluralsight.com.  Looking at the code I believe it parses for WWW.pluralsight.com or falls back to the generic extractor which of course fails.  Since they seem to ban accounts if you hit them like this too much, this should probably be fixed sooner rather than later before we all get ourselves banned. :)  Can the one line of regex in their extractor to allow for APP.pluralsight.com?

I created a pull request myself and proposed a fix to the code.  When I test against my test URLs, my new regex works properly.

Pluralsight should work again in the next version of youtube-dl.

Thanks for all your work and efforts - I really appreciate the time you've spent to try and get this working. :)
Should this have been fixed in 2015.11.24? It doesn't seem to have been for me.
tdobson@tragicomix:~/Videos$ youtube-dl --version
2015.11.24
tdobson@tragicomix:~/Videos$ youtube-dl --username=username --password=pw http://app.pluralsight.com/courses/angularjs-get-started
[pluralsight:course] angularjs-get-started: Downloading course JSON
[pluralsight:course] angularjs-get-started: Downloading course data JSON
[download] Downloading playlist: AngularJS: Get Started
[pluralsight:course] playlist AngularJS: Get Started: Collected 44 video ids (downloading 44 of them)
[download] Downloading video 1 of 44
[pluralsight] Downloading login page
[pluralsight] Logging in as username
[pluralsight] angularjs-get-started-m1-introduction-0: Downloading webpage
[pluralsight] angularjs-get-started-m1-introduction-0: Downloading mp4-high URL
[pluralsight] angularjs-get-started-m1-introduction-0: Waiting for 3 seconds to avoid throttling
ERROR: An extractor error has occured. (caused by KeyError(u'clipName',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
tdobson@tragicomix:~/Videos$ youtube-dl --verbose --username=username --password=pw http://app.pluralsight.com/courses/angularjs-get-started
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'--username=username', u'--password=pw', u'http://app.pluralsight.com/courses/angularjs-get-started']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.11.24
[debug] Python version 2.7.3 - Linux-3.2.0-4-amd64-x86_64-with-debian-7.9
[debug] exe versions: avconv 0.8.17-6, avprobe 0.8.17-6, ffmpeg 0.8.17-6, ffprobe 0.8.17-6, rtmpdump 2.4
[debug] Proxy map: {}
[pluralsight:course] angularjs-get-started: Downloading course JSON
[pluralsight:course] angularjs-get-started: Downloading course data JSON
[download] Downloading playlist: AngularJS: Get Started
[pluralsight:course] playlist AngularJS: Get Started: Collected 44 video ids (downloading 44 of them)
[download] Downloading video 1 of 44
[pluralsight] Downloading login page
[pluralsight] Logging in as username
[pluralsight] angularjs-get-started-m1-introduction-0: Downloading webpage
[pluralsight] angularjs-get-started-m1-introduction-0: Downloading mp4-high URL
[pluralsight] angularjs-get-started-m1-introduction-0: Waiting for 5 seconds to avoid throttling
ERROR: An extractor error has occured. (caused by KeyError(u'clipName',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 290, in extract
return self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/pluralsight.py", line 219, in _real_extract
'id': clip['clipName'],
KeyError: u'clipName'
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 663, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 296, in extract
raise ExtractorError('An extractor error has occured.', cause=e)
ExtractorError: An extractor error has occured. (caused by KeyError(u'clipName',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
tdobson@tragicomix:~/Videos$ youtube-dl --verbose --username=username --password=pw http://app.pluralsight.com/courses/angularjs-get-started
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'--username=username', u'--password=pw', u'http://app.pluralsight.com/courses/angularjs-get-started']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.11.24
[debug] Python version 2.7.3 - Linux-3.2.0-4-amd64-x86_64-with-debian-7.9
[debug] exe versions: avconv 0.8.17-6, avprobe 0.8.17-6, ffmpeg 0.8.17-6, ffprobe 0.8.17-6, rtmpdump 2.4
[debug] Proxy map: {}
[pluralsight:course] angularjs-get-started: Downloading course JSON
[pluralsight:course] angularjs-get-started: Downloading course data JSON
[download] Downloading playlist: AngularJS: Get Started
[pluralsight:course] playlist AngularJS: Get Started: Collected 44 video ids (downloading 44 of them)
[download] Downloading video 1 of 44
[pluralsight] Downloading login page
[pluralsight] Logging in as username
[pluralsight] angularjs-get-started-m1-introduction-0: Downloading webpage
[pluralsight] angularjs-get-started-m1-introduction-0: Downloading mp4-high URL
[pluralsight] angularjs-get-started-m1-introduction-0: Waiting for 2 seconds to avoid throttling
ERROR: An extractor error has occured. (caused by KeyError(u'clipName',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 290, in extract
return self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/pluralsight.py", line 219, in _real_extract
'id': clip['clipName'],
KeyError: u'clipName'
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 663, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 296, in extract
raise ExtractorError('An extractor error has occured.', cause=e)
ExtractorError: An extractor error has occured. (caused by KeyError(u'clipName',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
tdobson@tragicomix:~/Videos$ youtube-dl --verbose --username=username --password=pw https://app.pluralsight.com/library/courses/angularjs-get-started/table-of-contents
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'--username=username', u'--password=pw', u'https://app.pluralsight.com/library/courses/angularjs-get-started/table-of-contents']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.11.24
[debug] Python version 2.7.3 - Linux-3.2.0-4-amd64-x86_64-with-debian-7.9
[debug] exe versions: avconv 0.8.17-6, avprobe 0.8.17-6, ffmpeg 0.8.17-6, ffprobe 0.8.17-6, rtmpdump 2.4
[debug] Proxy map: {}
[pluralsight:course] angularjs-get-started: Downloading course JSON
[pluralsight:course] angularjs-get-started: Downloading course data JSON
[download] Downloading playlist: AngularJS: Get Started
[pluralsight:course] playlist AngularJS: Get Started: Collected 44 video ids (downloading 44 of them)
[download] Downloading video 1 of 44
[pluralsight] Downloading login page
[pluralsight] Logging in as username
[pluralsight] angularjs-get-started-m1-introduction-0: Downloading webpage
[pluralsight] angularjs-get-started-m1-introduction-0: Downloading mp4-high URL
[pluralsight] angularjs-get-started-m1-introduction-0: Waiting for 4 seconds to avoid throttling
ERROR: An extractor error has occured. (caused by KeyError(u'clipName',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 290, in extract
return self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/pluralsight.py", line 219, in _real_extract
'id': clip['clipName'],
KeyError: u'clipName'
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 663, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 296, in extract
raise ExtractorError('An extractor error has occured.', cause=e)
ExtractorError: An extractor error has occured. (caused by KeyError(u'clipName',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
tdobson@tragicomix:~/Videos$ youtube-dl --verbose --username=username --password=pw https://app.pluralsight.com/training/player?course=angularjs-get-started&author=scott-allen&name=angularjs-get-started-m1-introduction&clip=0&mode=live
[1] 31437
[2] 31438
[3] 31439
[4] 31440
[2]   Done                    author=scott-allen
[3]-  Done                    name=angularjs-get-started-m1-introduction
tdobson@tragicomix:~/Videos$ [debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'--username=username', u'--password=pw', u'https://app.pluralsight.com/training/player?course=angularjs-get-started']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.11.24
[debug] Python version 2.7.3 - Linux-3.2.0-4-amd64-x86_64-with-debian-7.9
[debug] exe versions: avconv 0.8.17-6, avprobe 0.8.17-6, ffmpeg 0.8.17-6, ffprobe 0.8.17-6, rtmpdump 2.4
[debug] Proxy map: {}
[pluralsight] Downloading login page
[pluralsight] Logging in as username
ERROR: Invalid URL
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 663, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 290, in extract
return self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/pluralsight.py", line 101, in _real_extract
raise ExtractorError('Invalid URL', expected=True)
ExtractorError: Invalid URL

I am using version 2015.11.21 on Windows and its working.


Update and rename simdjsone.app.src to simdjson.app.src

Make simdjsone works when building Erlang releases

Unfortunately have to revert the change - hex.pm already has sombody else's simdjson project, so I cannot push to hex with this change.

I was looking for more consistency for file naming. But it's fine @saleyn

Unfortunately, this was the reason for the initial inconsistency, as at the time I created the project, I ran into the naming clash at hex.pm.  The other way around it would be to rename the simdjson.erl to simdjsone.erl, which I am not too opposed of doing.

Unfortunately, this was the reason for the initial inconsistency, as at the time I created the project, I ran into the naming clash at hex.pm. The other way around it would be to rename the simdjson.erl to simdjsone.erl, which I am not too opposed of doing.

I think we can live with it like this.
Thanks for at least considering this.


set background color

Hi. How can I set background color for the dialog? Currently I see it's white on your screenshot, but in my app it turns black.

Can you add a screenshot of yours?


Add attention data

[x] create method to read in attention data on backend (from another json file)
[ ] test out how long computing on fly would take
[x] ultimately, connect to attention plots in single(#7) /matrix view(#4)

closing for now... loaded precomputed data seems possible atm


Issue1886 - Session files do not work if dataset path has spaces

The problem is that string vectors are space delimited in the params database/xml files. Since I'm guessing this is not something we can change due to backwards compatibility and potentially breaking things, I added escaping for spaces for string vectors in the params database.
Fix #1886

The previous behavior of StrToWordVec was to what you describe: you would
get a four element vector back. I changed StrToWordVec to only split at
non-escaped strings and when saving to params, spaces inside strings are
escaped then they are joined with non-escaped strings.
On Tue, Oct 22, 2019 at 9:38 AM clyne<EMAIL_ADDRESS>wrote:

@clyne commented on this pull request.
In lib/params/XmlNode.cpp
https://github.com/NCAR/VAPOR/pull/1887#discussion_r337594578:

StrToWordVec(s, vec);

for (string &e : vec)

Does this code do what we want? If I have a two element string vector:
a[0] = "string"
a[1] = "string with spaces"
that I encode with SetElementStringVec
will I get be able to get it back as a two element vector with
GetElementStringVec()? It looks like I'll get four element vector back
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NCAR/VAPOR/pull/1887?email_source=notifications&email_token=AAVE5T3LMQEA6VWO574ZUOTQP4M5XA5CNFSM4JDEGD42YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCIZKIMI#pullrequestreview-305308721,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVE5T3JZBAGYQ2WE24WKWLQP4M5XANCNFSM4JDEGD4Q
.


[...all] match problem

Describe the bug
I have file structure like:
├── pages
│   ├── [...all].vue
│   ├── hi
│   │   └── [...all].vue
│   ├── index.vue
│   └── README.md

localhost:5173/hi is supposed to display pages/hi/[...all].vue but display pages/[...all].vue
localhost:5173/hi/ and localhost:5173/hi/foo works as expected.
Not really sure this is a bug or feature, but src/pages/[...all].vue -> /* (/non-existent-page) described in the doc is a little bit ambiguous.
Reproduction
https://github.com/Amorites/vite-plugin-pages-all-issue
System Info
System:
    OS: Linux 3.10 CentOS Linux 7 (Core)
    CPU: (12) x64 AMD EPYC 7702P 64-Core Processor
    Memory: 18.63 GB / 23.39 GB
    Container: Yes
    Shell: 5.8.1 - /usr/local/bin/zsh
  Binaries:
    Node: 16.16.0 - ~/n/bin/node
    npm: 6.14.4 - /usr/bin/npm

Used Package Manager
pnpm
Validations

[X] Follow the Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A please open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

fixed in v0.26.0

@hannoer I upgraded the dependency, but the performance is still the same as before
repro: https://github.com/Amorites/vite-plugin-pages-all-issue

Hmm, after further testing, it seems children are not being sorted with dynamic routes last.
So to fix it, I had to do the following in my vite.config.ts
    Pages({
      routeStyle: 'nuxt',
      onRoutesGenerated(routes: Array<VueRoute>) {
        function isCatchAll(route: VueRoute) {
          return route.name === 'all'
        }

        function isDynamic(route: VueRoute) {
          return route.path.includes(':')
        }

        function fixAll(routes: Array<VueRoute>) {
          routes.sort((a, b) => {
            if (isCatchAll(a)) {
              return 1
            }

            if (isCatchAll(b)) {
              return -1
            }

            const aDynamic = isDynamic(a)
            const bDynamic = isDynamic(b)

            if (aDynamic !== bDynamic) {
              // Non-dynamic first
              return aDynamic ? 1 : -1
            }

            const aSize = a.path.length
            const bSize = b.path.length

            if (aSize !== bSize) {
              // Longest length first
              return bSize - aSize
            }

            return a.path.localeCompare(b.path)
          })

          routes.forEach(fix)
        }

        function fix(route: VueRoute) {
          if (route.children) {
            fixAll(route.children)
          }
        }

        fixAll(routes)

        return routes
      },
    })

@achaphiv This is not releated to this issue, please open a new issue and provide a minimal reproduction.


Images and assets omitted during production build

Version
3.4.0
Environment info
Environment Info:

  System:
    OS: Linux 3.10 CentOS Linux 7 (Core)
    CPU: (1) x64 QEMU Virtual CPU version (cpu64-rhel6)
  Binaries:
    Node: 10.15.1 - /usr/bin/node
    Yarn: Not Found
    npm: 6.4.1 - /usr/bin/npm
  Browsers:
    Chrome: Not Found
    Firefox: Not Found
  npmGlobalPackages:
    @vue/cli: 3.4.0

Steps to reproduce
Include any image in assets folder under src (/src/assets)
Run "npm run build" which is equivalent to "vue-cli-service build"
What is expected?
Build including all images and assets
What is actually happening?
Following message is displayed when running "npm run build":
"Images and other types of images are omitted"

I need to include my logo in the application, which I included in assets folder.
However running a production build is not including those images/logo.
Secondly, I have tried using NODE_ENV = Production while running the production build, result was same and problem was not fixed.

"Images and other types of images are omitted"

This just means that they are omitted from the display in the console.
All assets that you import into your application code are processed by webpack.
If however you need to have an asset copied into /dist that is not imported in you appliucation code, webpack will not process it. For these types of static assets, we have the /public directory - its contents will be copied into dist regardless of wether your app uses them.
Please eread the documentation: https://cli.vuejs.org/guide/html-and-static-assets.html#static-assets-handling

Well thats the thing, to check whether or not this is just in the console, I tried ti open those images in browser and download from the server in my PC but it appears to be those are just blank images and not the actual ones I placed in assets directory.
This could really mean that hose images and assets are indeed omitted.

I can't follow.
Please provide a reproduction as is required, then i can reconsider reopening this.

No spaces in the path


1.14.1 is incorrect

1.14.1 is not the correct version for armor enchantment compatibility. It is from 19w02a (1.14) to 1.14.3 pre-1, meaning 1.14.2 was the last release version to allow for "god armor".

Fixed the label - thanks!


Add main export

fixes #4879

See https://github.com/material-components/material-web/issues/4879#issuecomment-1712285128 for why we want to reserve the main entrypoint


Run out of Java heap space on executing org.jlleitschuh.gradle.ktlint.worker.KtLintWorkAction on JDK 8

The Stripe CI is running ktlint and we ran out of Java heap space when upgrading to ktlint 0.41.0
> A failure occurred while executing org.jlleitschuh.gradle.ktlint.worker.KtLintWorkAction
   > Java heap space

Upgrading the JDK from 8 to JDK 11, but JDK causes some build failures as well, so simply upgrading ktlint -> upgrading the JDK is a cascade of issues.

You could increase heap size for KtLint worker:
tasks.withType(org.jlleitschuh.gradle.ktlint.tasks.BaseKtLintCheckTask::class.java).configureEach {
    it.workerMaxHeapSize.set("512mb")
}

Next time please open related issue in ktlint-gradle project.

Hello,
I found the same issue and I could make it work using
    tasks.withType(org.jlleitschuh.gradle.ktlint.tasks.BaseKtLintCheckTask).configureEach {
        it.workerMaxHeapSize.set("512m")
    }

More details at https://kotlinlang.slack.com/archives/CKS3XG0LS/p1619163241002400
Thank you once again, @romtsn


Repeated dependencies in the JSON file

The generated JSON file has many redundant copies of the same plugins. In particular, the generated JSON file for one of my packages has 2200 lines. Is this expected behavior, or can this be avoided?
Example:
    {"path":"antlr/antlr/2.7.2/antlr-2.7.2.pom","sha1":"60b2b206af3df735765e8e284396bcfdbced5665"},
    {"path":"antlr/antlr/2.7.2/antlr-2.7.2.jar","sha1":"546b5220622c4d9b2da45ad1899224b6ce1c8830"},
    {"path":"antlr/antlr/2.7.6/antlr-2.7.6.jar","sha1":"cf4f67dae5df4f9932ae7810f4548ef3e14dd35e"},
    {"path":"antlr/antlr/2.7.6/antlr-2.7.6.pom","sha1":"1ef68a00c22016097c813d1aae66fdd486e34c74"},
    {"path":"antlr/antlr/2.7.7/antlr-2.7.7.jar","sha1":"83cd2cd674a217ade95a4bb83a8a14f351f48bd0"},
    {"path":"antlr/antlr/2.7.7/antlr-2.7.7.pom","sha1":"52f15b99911ab8b8bc8744675f5cf1994a626fb8"},

The way Mavenix works is it runs Maven and looks in the resulting repository that Maven created and generates a JSON from the contents.
So, the question is: why does Maven download all those duplicates?
You can try removing duplicates from the JSON manually and then run nix-build. If it still builds I can try to figure out if there is a way to get a slimmer JSON.
I can also try to look at your project if you have a link or something.

I just tested, and removing any JAR from the list makes the compilation to fail. It seems that Maven just works like that.
Thank you very much!


Prevent ash_postgres v2.0.10 from being used

Merging #9 into main.
Thanks for your contribution @hwuethrich, and thanks @zachdaniel for the rapid bugfix on ash_postgres 🚀

Released!


Improve support for tick-by-tick parsing

As it stands, parse_rate=1 is hard to operationalize due to the memory requirements imposed by the JSON.
We could add a flag that indicates player-states and frames should be parsed CSV.

Made https://github.com/pnxenopoulos/csgo-tick-parser


Create a Material theme and apply styles from the ODK-X style guide

This issue is to make changes in Android Library to update the 'opendatakit' theme to use Material design and apply styles from the ODK-X styleguide:
https://docs.google.com/presentation/d/1Ol7YhG4WPLOWCoQZUrKHLPguHSiDz8RC3m8dNyTGGLo/
Design items to apply:

Fonts
Heading styles
Spacing
Button styles (these will be updated later)
Setup light and dark themes

Updated styles will be universally applied to all ODK-X mobile apps: Services, Tables, and Survey

Submit PR with #402

@maprehensive You said Button styles will be added later? Means that I shouldn't worry about adding it to the theme?

@Redeem-Grimm-Satoshi, on the call, we were talking about how @Cveman1 had some work left to do on buttons and make a final presentation on it. You can start implementing a button theme now, knowing that they might have updates later. However, creating the theme and getting all the buttons to point at the theme can be a separate task.

@wbrunette Thanks for the clarification

From meeting discussion - references for upgrading to M3 / v1.7.0 https://m3.material.io/develop/android/mdc-android

@maprehensive Thanks

Dependency on #384

PR https://github.com/odk-x/services/pull/257

@Redeem-Grimm-Satoshi @wbrunette @Cveman1
I noticed that some dialogues are not rounded corners and without the button style. Were they missed in the application of the theme?

Hi @Redeem-Grimm-Satoshi, can we both take a look at this?

We've discussed and Redeem would fix @maprehensive

Apply to Tables and Survey

I would like to Apply it to tables and survey.

I am an outreachy applicant and would like to Apply it to tables and survey.

Part of this issue has been worked on, so it needs a review. Best not to work on it now.


Document diff between native env and kube env in the new integration test framework

Describe the feature request
Document the differences (if any) between native env and kube env, limitations of native env compared to kube env if any, etc. in order to make choosing the testing env easier.

The environment is not meant to be chosen by the test author. Ideally, the tests should be written in an envrionment-agnostic way (the components should hide the details of execution for each environment).
Only in an exceptionally small set of cases, it should be legitimate to constraint to a specific environment (i.e. when specifically testing integration with K8s, for example).

We just have a bunch of unimplemented components at this point.

We just have a bunch of unimplemented components at this point.

Any rough ETA of these unimplemented components? I think if the ETA is months, then we should at least have some document to mention the limitations of the native env at its current stage.

It should be a community/group effort. If you see something missing, you should help implement it. :)


I working with UE4 .15,But I only can get release BLUI 3.1

BLUI 3.1 is working with UE 4.10.  But I need working with UE 4.15. Who can give me a more advanced version? I see people working at 4.15。

You can add the plugin to a C++ project and recompile and it will update the plugin to 4.15. Just use the master branch code which has 4.15 code fixes.

@getnamo  thanks for your help. And I can't download BLUI 3.1 or 3.0.    Maybe  I  should to copy the master branch code of BLUI into Plugin Folder in my project?

I am  ashamed of it .I don't know much about Github.


Add safe mode

i think mgp can add a featrue. like safe mode.when you use mgp to refresh you package form git repo. if you had changed you some file in package.then alert " do you want to cover the changed file?".so this can give user the chance to stop the silly action to cover their code.
for examle:
mgp -s //run in safe mode then alert user when  some  file had changed.

if you don't mind ,i can implement it and send a  PULL REQUEST.
one time.i write some code in package and debug it.then i take a rest .when i come back.take a mistake to run mgp!!! so my code lost.i just wana to cut off my FUCKing finger.

We would love a pull request. Good idea!

lol

maybe have mgp -f instead to force a potentially breaking update and assume the user wants to be safe pr. default.

Closing for now. Happy to entertain PRs.


python-gcm does not support connection pooling

Hello! I observe that an SSL connection is being created for each request. This was true at least in 0.1.5 and was still true in 0.2. This appears to be a consequence of using requests.post which creates and closes an internal Session object. In Requests, connection pooling is bound to the Session object (http://docs.python-requests.org/en/latest/user/advanced/#keep-alive).
I tested a monkey patched version of python-gcm where gcm.gcm.requests is replaced with a gcm.gcm.requests.Session() object and observed that connections are reused. From my test server in AWS us-east-1, each GCM POST request took avg ~140ms without the patch and avg ~40ms with (7 qps vs 25) which at our volume is a significant help.
It wasn't clear to me how best to integrate connection pooling into python-gcm so I thought I'd post the issue for comment before sending a PR.  Also, the thread safety of requests.Session is an open question so that's a trade-off to consider: https://github.com/kennethreitz/requests/issues/1871.

@alexdej Hi! Thanks for raising this issue! Indeed this is a performance problem and we should reuse the underlying TCP connection/socket.
Currently, using json_request you can send push notifications in bulks of 1000, so if you have 1 million tokens you would open 1000 TCP connections to the GCM server.
I've just tested with a few million notifications something along the lines of: https://github.com/mnemonicflow/python-gcm/commit/00a93d69288bc786e30d43df89ef271ac54117ce which should be a good starting point, although this means forcing the clients of the library to use a context manager and existing clients need to update their code.
Would be better to add another level of indirection and run the cleanup code inside the GCM object by creating a static session object wrapper using the https://docs.python.org/3/library/contextlib.html#contextlib.contextmanager decorator or similar.
Regarding thread safety, I think it should be handled by the client of the library.

Great! Your suggestion would work fine for our case, though I agree you might want to make the new behavior optional to avoid changing existing clients (and to preserve thread safety of gcm library by default).

:+1: on reusing connections. @mnemonicflow your suggestion only works if the data is the same for all recipients, right? If the data is unique for everyone you'd have to call gcm.json_request() once per recipient.

@tgwizard Yes, the data is the same for all recipients you pass in the registration_ids parameter. If you want to send different data for different recipients you have to group the recipients (preferrably in bulks of 1000) and pass the specific data you want to send to them in a different json_request call.
The underlying TCP connection is still reused due to the session object but as the documentation says http://docs.python-requests.org/en/latest/user/advanced/ the data is NOT.
"Note, however, that method-level parameters will not be persisted across requests, even if using a session."

Thanks for the response @mnemonicflow. The session, and TCP connection, reuse will only be enabled when #96 being merged, as now there's a call to requests.post, which creates new sessions for every call. Or am I missing something?

@tgwizard Yes! #96 got merged in develop branch https://github.com/geeknam/python-gcm/tree/develop

Cool, thank you!


/model/predict: Handle invalid input gracefully

API endpoint /model/predict returns HTTP status code 200 and
{
"status": "error",
"result": null
}

if an invalid audio input is provided. It should return status code 400 if an error is caused by input that cannot be processed because it doesn't meet the [stated] requirements.

Hey @ptitzler I am unable to reproduce this problem. Maybe this was fixed in a recent commit. Can you please verify?

Looks like Karthik fixed this in a077ba55bb65f74843c11eba8593271c9dc84944


HDFS-15043. RBF: The detail of the Exception is not shown in ZKDelegationTokenSecretManagerImpl

JIRA: https://issues.apache.org/jira/browse/HDFS-15043

Logging IOException.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
docker
22m 34s
Docker failed to build yetus/hadoop:104ccca9169.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/1748

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1748/1/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m  0s
Docker mode activated.

-1 :x:
docker
0m  8s
Docker failed to build yetus/hadoop:104ccca9169.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/1748

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1748/2/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

Thank you @ayushtkn and @tasanuma


basisOfRecord [PreservedSpecimen] that does not match the Darwin Core Type Vocabulary

Hello,
On a fresh new 2.5.1 (docker) installation, we have the following error when we try to publish an occurrence dataset
Line #1 has basisOfRecord [PreservedSpecimen] that does not match the Darwin Core Type Vocabulary
Line #2 has basisOfRecord [PreservedSpecimen] that does not match the Darwin Core Type Vocabulary
...
It's really similar to https://github.com/gbif/ipt/issues/1615
Except that the proposed workaround is not helping.
We try:

the Synchronize button in Core Types and Extension page
manually deleting the config/.vocabularies folder, restarting the IPT

We don't get the http_rs_gbif_org_vocabulary_dwc_basis_of_record_xml.vocab file, like in my other working installations.
In the debug.log, when we click Synchronize, we only have successful messages:
INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/gbif/quantityType INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/gbif/unitOfMeasurement INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/dwc/pathway/ INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/gbif/occurrence_status/ INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/gbif/preservation_method INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/gbif/datasetType INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/dwc/basis_of_record/ INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/dwc/degree_of_establishment/ INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/gbif/rank INFO  13-Oct-2021 16:19:30 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://dublincore.org/documents/dcmi-type-vocabulary/ INFO  13-Oct-2021 16:19:31 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://iso.org/iso3166-1/alpha2 INFO  13-Oct-2021 16:19:31 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://iso.org/639-2 INFO  13-Oct-2021 16:19:31 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/eml/updateFrequency INFO  13-Oct-2021 16:19:31 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/dwc/establishment_means/ INFO  13-Oct-2021 16:19:31 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/gbif/agentRole INFO  13-Oct-2021 16:19:31 [org.gbif.ipt.service.admin.impl.VocabulariesManagerImpl] - Successfully loaded vocabulary: http://rs.gbif.org/vocabulary/gbif/datasetSubtype
I don't get how it can be successful, since most of the links are broken for me....
http://rs.gbif.org/vocabulary/gbif/quantityType ==> 404
http://rs.gbif.org/vocabulary/gbif/unitOfMeasurement ==> 404
http://rs.gbif.org/vocabulary/gbif/occurrence_status/ ==> OK
http://rs.gbif.org/vocabulary/dwc/basis_of_record/ ==> 404

I guess that occurrence extension at your IPT points to http_rs_gbif_org_vocabulary_dwc_basis_of_record_xml.vocab which is absent (for some reason). If it's following it should be fine:
<property group='Record-level' name='basisOfRecord' thesaurus='http://rs.gbif.org/sandbox/vocabulary/dwc/basis_of_record_2021-08-22.xml' namespace='http://rs.tdwg.org/dwc/terms/' qualName='http://rs.tdwg.org/dwc/terms/basisOfRecord' dc:relation='https://dwc.tdwg.org/terms/#dwc:basisOfRecord' dc:description='The specific nature of the data record.' comments='Recommended best practice is to use the standard label of one of the Darwin Core classes.' examples='`PreservedSpecimen`, `FossilSpecimen`, `LivingSpecimen`, `MaterialSample`, `Event`, `HumanObservation`, `MachineObservation`, `Taxon`, `Occurrence`, `MaterialCitation`' required='true'/>

Greetings,
I am getting the same exact error on 2.5.3 docker and older versions :
Line #1 has basisOfRecord [PreservedSpecimen] that does not match the Darwin Core Type Vocabulary
Line #2 has basisOfRecord [PreservedSpecimen] that does not match the Darwin Core Type Vocabulary
...
I have tried all the workarounds that I've reviewed in the basisOfRecord issues, as well as other attempts:
Synchronize Vocabulary
Installed extensions the added the http_rs_gbif_org_vocabulary_dwc_basis_of_record_xml.vocab
Moved the http_rs_gbif_org_vocabulary_dwc_basis_of_record_xml.vocab instal the config/.vocabularies folder.
Edited the http_rs_tdwg_org_dwc_terms_Occurrence.xml extension to disable the basisOfRerord requirement
Deployed every version of the docker from current(2.5.3) to 2.4.2. on a Local dockers deployed on Mac OS and on Dockers I built on a Linux VM.
Any suggestions would be appreciated.

Hi,
I did some debugging since I still have the issue on a new 2.5.5 instance in TEST mode (connected to GBIF UAT)
Here is what I found.
The "basis of record" vocab is initialiazed here, using the key Constants.VOCAB_URI_BASIS_OF_RECORDS whose value is "http://rs.tdwg.org/dwc/dwctype/"
In VocabulariesManagerImpl, the map has the following content:

As we can see, we don't have "http://rs.tdwg.org/dwc/dwctype/", but instead "http://rs.gbif.org/vocabulary/dwc/basis_of_record/"
As a result, the basis of record vocab is empty, and we have the reported errors

[PreservedSpecimen] that does not match the Darwin Core Type Vocabulary

For me, the code is compliant with the PROD vocab:
https://gbrds.gbif.org/registry/thesauri.json ==> "identifier": "http://rs.tdwg.org/dwc/dwctype/"
but not the UAT vocab:
https://gbrds.gbif-uat.org/registry/thesauri.json ==> "identifier": "http://rs.gbif.org/vocabulary/dwc/basis_of_record/"
I'm not used to all the vocab stuff, so I don't know how to fix it... (don't think it's a code issue)
Thanks,
Sylvain

Thank you for your time testing this @sylmorin-gbif
Yes, it was switched to tdwg term instead of gbif one recently. I'm not sure what we can do with this right now, but we'll get back to this very soon, since we already started adopting new Darwin Core stuff.

With https://github.com/gbif/rs.gbif.org/commit/6ac1902c93ef0238def343cbc37371ab5e750a09 I've changed the dc:URIs back to the original values, so this should now be working again.


Run all project E2E tests on merge to main

Describe the task
We are using nx affected to only run E2E tests in CI for projects that are affected by the current PR. This has a few benefits, namely reducing GitHub runner usage, mitigating issues from flakey tests and in some cases reducing CI running time.
nx affected tries to be as safe as possible and will run all tests if anything in shared folders changes, though there is still the possibility of bugs getting through. For example a breaking change in the backend as that isn't monitored by nx affected. We should run the entire suite of E2E tests on merge to main to be safe.
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Looks like this is in progress so I will add it to the current sprint.


JBPM-5716: Stunner - Shapes and controls broken features as for some mouse interactions.

Hey @manstis , @hasys
This commit includes a few fixes but that should be included for the release!
See the Jira ticket
@hasys  sorry if this time cannot add missing tests for existing classes, will keep adding tests on further iterations..
@manstis this commit includes the fix for the canvas preview type, as commented, a partial fix yet...
Thanks!

it includes two type renamings, that's why a few many files appear to be changed...

btw @manstis this fix about text makes the text edition more intuitive... as before, you had to dbl-click on the shape (not the text) in order to display the name editor popup, so I saw people trying to click on text for thousands of times with no success hehe... now, once you get over the text, the cursor changes, at that point, you can dbl-click and the name editor popup appears.... so will see how it goes... but I think it will be clear for users

@manstis new commit added. Thanks!

Jenkins please retest this

Jenkins please retest this.

@manstis tbh still not sure about what the build error is about :( it says all good, just a few warns...


Update uid2 dependency from 0.0.3 to 1.0.0

This updates the uid2 dependency from 0.0.3 to 1.0.0 (its latest release) which has the following changes (see its HISTORY.md for full details):

Uses crypto.randomBytes instead of crypto.pseudoRandomBytes https://github.com/coreh/uid2/commit/e492a9db2a9892dc13a5f55e4ae28ff9af2add64
Adds -_ to generated unique IDs https://github.com/coreh/uid2/commit/a3b8def77e211a7fd8f244666a0e1d3edacf0d16
Improves generation time of an ID by 15% https://github.com/coreh/uid2/commit/3cfd2cc9555b772f3e361ca60a98c0931d504b41

The first change would only matter to clients that are using a version of node < 4.0.0, but this repo only supports >= 10 so that's not really a problem. This does also future-proof this repo a bit where while the crypto.pseudoRandomBytes does work, it's undocumented in the crypto docs, as crypto.randomBytes should work essentially the same, unless you ran it exactly at system boot time, in which it'll then block execution till enough entropy has built up.
The second case could matter if someone was for some reason relying on the exact shape of the generated IDs, but that seems unlikely, and the internal code doesn't have any issues with the two additional characters.

@MasterOdin thanks :+1:


Add Database from a diagram

Implemented Changes:

Implement a Database based the diagram on the bottom.
Include Foreign keys
Include a link table for many to many relations between the medical_histories table and the treatments table.

Hi @Diegogagan2587 and @Javivy,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that is invalid because the collaboration is invalid, we do not count commits to the README file as a valid contribution. Students are required to make contribution to the actual code
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Add a registration_behavior column to AllocationToken

This is, as of now, unused but we can use it for b/237683906 and
b/237800445 in the future to allow for special behavior dictated by
allocation tokens rather than having to reserve specific domains.

This change is

oops, need to do the sql changes only first

this should be good to go hopefully


Initial Xcode port.

You can see a screenshot here: https://github.com/dciccale/badwolf-xcode#readme

Thanks!


CRC is unsupported using WSL2

I get the following error when installing openshift local on wsl2 Ubuntu 24.04
$ crc setup --log-level debug
DEBU CRC version: 2.43.0+268795
DEBU OpenShift version: 4.17.1
DEBU MicroShift version: 4.17.1
DEBU Running 'crc setup'
INFO Using bundle path /home/jadebayo/.crc/cache/crc_libvirt_4.17.1_amd64.crcbundle
DEBU resolv.conf file path: /run/resolvconf/resolv.conf
INFO Checking if running as non-root
INFO Checking if running inside WSL2
DEBU Running inside WSL2 environment
DEBU CRC is unsupported using WSL2
CRC is unsupported using WSL2
$ cat /etc/*release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=24.04
DISTRIB_CODENAME=noble
DISTRIB_DESCRIPTION="Ubuntu 24.04.1 LTS"
PRETTY_NAME="Ubuntu 24.04.1 LTS"
NAME="Ubuntu"
VERSION_ID="24.04"
VERSION="24.04.1 LTS (Noble Numbat)"
VERSION_CODENAME=noble
ID=ubuntu

@johnbayo you are trying to run crc on WSL environment and it need virtualization. Better to directly install CRC on windows and then run (this is the way we develop and test it).


pkg/fuzzer: optimize triage

pkg/fuzzer: refactor progTypes
pkg/fuzzer: smarter retry for corpus candidates
pkg/fuzzer: improve triage procedure
pkg/fuzzer: refactor deflake tests

Added special testbed mode for corpus triage specifically (impossible to measure with fixed time tests), and pushed few small improvements.
Benchmarking results:

So triage time is -40%, heap size is -50%.
Signal -6%, but taking into account that max signal +4% and corpus +2.4%. It should actually be an improvement, we kept more corpus inputs and will continue mutating them during fuzzing, it's just the coverage wasn't stable.

In any case, we can try to run longer experiments to see whether this -6% reduction disappear over time.

But what do we expect to see and what will be a good result?
I can explain both outcomes as positive:

If the diff remains, it is good b/c it proves it's indeed just a different measurement method (these 6% are already in max cover, so we won't rediscover them).
It may also disappear over time due to signal rotation. Which is also good.

Do you have any explanations for that? Is it caused by the changes to the deflake() method or by not triaging every corpus program two times?

I don't have concrete evidence for any of these hypotheses.
More likely it's not triaging every program twice? Currently we triage 2 times, but don't update max signal in deflake for the additional runs, so perhaps that somehow moves part of the coverage from signal to max signal...
I am not ready to change DB format as part of this (it will be whole lot of other changes).
Let's see if we can shuffle things a bit to get better results.

If the diff remains, it is good

I'm not really sure about this. Can the decrease of the signal/max signal ratio sound good?
The slightly increased corpus size may be related to the less efficient corpus minimization due to the smaller per prog signal returned by deflake() now.

I don't have concrete evidence for any of these hypotheses.

We can try to remove one of these commits and re-run the syz-testbed experiment (we don't need too many runs -- 15-20 per checkout should be enough to see if -6% remain).

PTAL.
The last commit changes deflake to deflake all calls at once, and also uses more runs for programs from corpus (see added comments).
This still reduces triage time by 33%, but more importantly increases max signal after triage by 17.5%, which means we will do less triage/minimization/smashing/hints later. Signal/coverage are also larger, but I guess this is just because we do more runs.


Extract amplicon sequences from downloaded reference database

Hi, I am testing crabs to get a reference library of 12S. I noticed that after step 4 I got some small size sequences (for example 20 bps) and in other cases some sequences that bind only by one of the primers (sometimes fwd and sometimes rev) which should not happen when I use the following specifications "--percid 0.95 --coverage 0.95 --filter_method strict". What could this problem be due to? Is there any way to retain only the regions between the 2 primers?

Hello @FabioGAmaral,
Thank you for your query.
It is a bit difficult to provide an answer based on the limited provided information. I'm guessing you are trying to run the pairwise global alignment command (function: pga)? Based on the parameters you mention, you should only receive sequences that are at least 95% similar and cover at least 95% of any of the database sequences. These database sequences should be retrieved through the insilico_pcr function and, therefore, should contain the full amplicon sequence without primer-binding regions. Can you verify that the output of the insilico_pcr function contains only full-length amplicons? Or does this file contain sequences of length 20 bps?
I'm not sure what you mean by "bind only by one of the primers". My apologies, but could you please expand? Could you also please provide the code you used and some examples of the sequences you are referring to?
Best regards,
Gert-Jan

Hello, sorry for not explain it well. The output of the insilico_pcr contains only full-length amplicons (at least all of them have the , however during the pga some amplicons have only 20 bps.
The command I'm using is:
crabs pga --input "output of crabs db_download" --output Teleostei_MiFishU_teste.fasta --database "insilico_PCR output" --fwd "primer" --rev "primer" --speed medium --percid 0.95 --coverage 0.95 --filter_method strict.

Hello @FabioGAmaral,
I have not come across this issue before and will investigate. For now, you could filter the short reads using the seq_cleanup function.
Best regards,
Gert-Jan

Hello, I have another issue. When I'm doing crabs visualization --method primer_efficiency. I find this error written 89916 sequences from Vert_Ib.fasta into a dictionary
Traceback (most recent call last):
File "/home/whoami/reference_database_creator/crabs", line 1428, in
main()
File "/home/whoami/reference_database_creator/crabs", line 1425, in main
args.func(args)
File "/home/whoami/reference_database_creator/crabs", line 981, in visualization
orig = str(raw_dict[id])
KeyError: 'AF402623'
What could it be?

Hello @FabioGAmaral,
Based on the error it seems that the seqID AF402623 is in the input file, but cannot be found in the original raw sequencing file. Is this correct?
Best regards,
Gert-Jan

Based on the error it seems that the seqID AF402623 is in the input file, but cannot be found in the original raw sequencing file. Is this correct?
Exactly.

I do have another doubt/question. During db_download using BOLD I noticed that the sequences that have gaps are eliminated. is there a possibility to remove the gaps from the seqs instead of remove the seqs?
Best regards,
Fábio

Hello @FabioGAmaral,
This would require me to change a bit of the code base and add this functionality in. I'll try to get this incorporated by the end of next week.
Best,
Gert-Jan

Hello @FabioGAmaral,
This would require me to change a bit of the code base and add this functionality in. I'll try to get this incorporated by the end of next week.
Best, Gert-Jan

Hello, any news about it?

Hello @FabioGAmaral,
Apologies for the delay, it has been a hectic time!
This functionality has now been made available in version 0.1.5. As of now, you can only download this version through GitHub. We will try to update the Docker and conda versions as soon as possible, but there might be some delay on this. To check your version of CRABS, please use the following code: crabs --version.
To incorporate BOLD sequences with gaps into the output file, you can set the -g, --boldgap parameter to INCORPORATE. This will remove the gaps of the BOLD sequence, i.e., remove the '-' symbols, and add the sequence to the output file. By default this parameter is set to DISCARD.
I tested the new parameter using the following 2 lines of code:
crabs db_download -s bold -db Elasmobranchii -o bold_elasmobranchii_20230607.fasta -d bold_elasmobranchii_20230607_discard.fasta

db_download -s bold -db Elasmobranchii -o bold_elasmobranchii_20230607.fasta -d bold_elasmobranchii_20230607_discard.fasta -g INCORPORATE

By setting the -g, --boldgap parameter to INCORPORATE, CRABS managed to add all 23,964 initially downloaded Elasmobranchii sequences to the output file for downstream analysis.
PLEASE NOTE: I have not looked further into how incorporating these sequences in the CRABS database might impact/influence downstream analyses and results.
If you encounter any issues or bugs when using the new parameter, please feel free to comment below.
Best,
Gert-Jan

Good afternoon,
It worked very well thanks.
Sorry for the inconvenience, but is there a possibility to have a mapping file with the information between the two files that the program creates in the initial step (download) because one of the files does not indicate the gene and for bold the CRABS program does not allow to filter by the gene.
Best,
Fábio Amaral

Hello @FabioGAmaral,
I'm not sure I understand what you're referring to. Could you please clarify, maybe with an example?
Thanks,
Gert-Jan

When i download sequences through BOLD, crabs creates 2 files with different headers. In one of them, more complete, it says which gene that sequence refers to, and in the other it doesn't. Is it possible to keep the header in both sequences and the during the following points.

Hello @FabioGAmaral,
I'll have to look into the formatting and gene option available to design a parameter for selecting which genes to keep. Please give me a couple days to investigate.
Thanks,
Gert-Jan

got

Got any news?

Hello @FabioGAmaral,
My sincere apologies for the long pause in the communication!
I have now added in the option of genetic marker selection for downloading sequences from BOLD via the -m, --marker selection. This will allow you to specify which barcoding gene to download from the BOLD website. Please find more information in the updated README document. Please install CRABS --version 0.1.7 via GitHub for this functionality. We'll try to update the Docker and conda versions in the near future, but this could take a while. I hope this is what you were after?
Unfortunately, changing the sequence headers to display this information will be difficult to implement, as the header structure is used downstream for taxonomy assignment. I'm currently working on a full overhaul of the code with additional functionality to CRABS and will keep header structure in mind for the next big version update.
Best regards,
Gert-Jan


Error message "you need to quote the scalar"

Trying to configure my suites and contexts as described here (http://behat.org/en/latest/user_guide/configuration/suites.html), I get the following error in my terminal:
The reserved indicator "%" cannot start a plain scalar; you need to quote the scalar at line 7 (near "paths: [ %paths.base%/features ]").
How do I "quote the scalar", since using actual quotes on the variable placeholder produces the same error?
Thanks.

I had to quote like this in my behat.yaml file:
'%paths.base%/features'.
It would be helpful to update the documentation. Tags also need quoting: '@tag'.

The doc source is available at https://github.com/Behat/docs


Adding favicon

Adding Favicon

The original commit wasn't signed. I'll make a new one that is signed.


There is no answer

Copilot Chat Extension Version:
VS Code Version:
OS Version:Windows 10
Logs:
2024-09-18 14:20:11.537 [info] copilot token chat_enabled: true, sku: free_educational
2024-09-18 14:20:11.537 [info] GitHub.vscode-pull-request-github extension is not yet activated.
2024-09-18 14:20:11.819 [info] Fetched model metadata in 316ms 74d9c04f-b151-4903-841a-d0c334c8edf7
2024-09-18 14:20:12.268 [info] activationBlocker from 'languageModelAccess' took for 1909ms
2024-09-18 14:20:27.861 [info] Request ID for failed request: 14fe6df9-dbcb-4158-b656-71f73764eab7
Steps to Reproduce:
You write something back to him: Unfortunately, your request was limited by the speed. Wait a bit and try to send it again later.
What could be the problem?

And most importantly, it seemed to work at first and then stopped
Copilot version: v1.229.0

It appears you got rate limited, we're actively investigating why that happened .

The rate limit logic has since been improved. Let me know if you have any issues


Don't allow empty lines at the end of the file

eol_last rule requires a newline character at the end of the file. But on the other hand one might not want empty lines on itself at the end of the file. We should have a rule for that as well. What would be its name?

I don't recommend adding any rules that you personally don't plan on using. It just adds to what needs to be maintained and we don't know if anyone will ever turn it on.
They can either just not turn on eol_last or come implement the rule.

I am plan to use it. :-)

I haven't published a new version yet, so eol_last isn't in the wild. What do you think about adding a mode that could either be require or forbid? And my preference would be to default to require.

I am not sure what you are talking now here. Require or forbid what? An extra empty line at the end? This sounds strange that one would require extra empty line.

Now I'm not sure what you are talking about here :). doesn't eol_last (and eol-last) require that the last line of the file is empty?
foobar\n mode: 'require' says this file is good.
foobar\n mode: 'forbid' says this file is bad.
foobar\n is a 2 line file

This rule here is to forbid: foobar\n\n.

foobar\n is a 1 line file, BTW. If you think it the terms of eol_last.

oh, that makes more sense. Instead of making it specific to eof I think no-multiple-empty-lines would be ideal for this. I do think it should have a configurable max value, basically exactly the same as eslint's rule :).
I was thinking in terms of what the editor will show and:
> "foobar\n".split("\n").length
2

Yea, so I am planing to do no-multiple-empty-lines as of #430. But this here is different. Sorry to be nitpicky, but that is probably normal for people who are using lints. ;-)
So, for me, eol_last is about:
Bad: foobar
Good: foobar\n
no-multiple-empty-lines (with max number 1) is about:
Bad: foo\n\n\nbar
Good: foo, foo\nbar, foo\n\nbar
And this ticket here is about:
Bad: foo\n\n
Good: foo\n
So this ticket is not no-multiple-empty-lines. So at least for me, I am planing to use it so that there is always \n required exactly at the end of the file, but no multiple of them. In the code itself, I want to allow zero or one visual empty lines between lines, but not more than that. So no-multiple-empty-lines with 1 does not catch the foo\n\n because it allows it. It would catch foo\n\n\n though.
One option is to simply change eol_last to catch foo\n\n. And then we do not need this rule here. And then #430 is about empty lines in the code.

I think it's best to just have one rule that handles limiting the number of empty lines. I can't see any reason people would want to keep more than 1 line at the end of the file, so I think it's worth just making it a special case. The last line can be limited to 1 even if you set a higher maximum for that rule.

The multiple empty lines should be covered in the no trailing whitespace. One empty line at the end is acceptable. Two is basically trailing space.
Something like this should be caught: \s\s+$

And that's why it's configurable.
I see your point about it being trailing space. I'll give that some thought.

I think we should just make three rules. eol at the end, number of empty lines at the end, and number of empty lines in between the code.
I do not think we are anywhere looking at newlines as trailing space? Trailing space rule does not catch foo\n\n\n\nbar as trailing space?

:+1: \n{2,}$ (regex syntax, $ being end of file, not end of line) is bad and that's it. For me a perfectly valid rule.


Can you please update this plugin to make it compatible to latest version of https://github.com/videojs/videojs-contrib-ads and support for midroll and post roll ads also. Thanks

Here is the commit causing current vast plugin to be incompatible:
https://github.com/videojs/videojs-contrib-ads/commit/0220ab425c20e5081ae32585010681ecf69fb7ce

Here is the commit causing vast plugin to be incompatible with current videojs-contrib-ads:
https://github.com/videojs/videojs-contrib-ads/commit/0220ab425c20e5081ae32585010681ecf69fb7ce

Just prefix standard html5 events like timeupdate, ended, etc in videojs.vast.js and it should work.

Thanks for reply. I will check it out. Please help me for midroll and postroll ad also.

What's the status on this guys?

how can i use it as a mid roll , ?
Please help its urgent

how can i use it as a mid roll , ?
Please help its urgent

how can i use it as a mid roll , ?
Please help its urgent


Should support react 17

https://github.com/Grsmto/simplebar/blob/a0611f7bc31a45970f2f3f645c5f6eee47812f77/packages/simplebar-react/package.json#L39
ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^15.3.0 || ^16.0.0-rc || ^16.0" from<EMAIL_ADDRESS>npm ERR! node_modules/simplebar-react
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Supporting React 17 might be more than just a peer dependency update. React 17 seems to introduce a small breaking change with the onScroll handler.
React 16 allows onScroll being attached to SimpleBar directly like:
<SimpleBar onScroll={...}></SimpleBar>

This breaks in React 17 for a reason that is not clear to me. In React 17 you need to use this form of the onScroll handler:
<SimpleBar scrollableNodeProps={{
  onScroll() {...}
}}></SimpleBar>

There is a reproducer here: https://codesandbox.io/s/trusting-flower-butid?file=/src/App.js. If you use React 16.x you can see that both forms work. In React 17.x the top version stops firing.

@Grsmto do you have any plans for update simplebar for react 17.x in near future?

The CoreUI team is using React 17, Someone needs to work on support on this.

I really want to see an update for this please make it happen 🙏

i also had this problem. I hope it can happen in the future

The reason for the issue @sanbornhilland mentioned is that the onScroll event in React v17 no longer bubbles. I would argue that this isn't even an issue with simplebar-react, it's just how React 17 works.
Other than that, I haven't run into any issues using simplebar on React 17, I think that it would be safe to update the package.json for simplebar-react to allow react and react-dom v17 without making any other changes.


[BUG] Dark Mode Toggle appearing twice in 'Dashboard' Page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The Dark Mode toggle button appears twice on the 'Dashboard', once in the Navbar and second, right next to search bar. It should be only once, in the Navbar.
Would request you to assign me to fix this bug under GSSoC '24 :)

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

It was a minor fix @yashmandi . I have already done it !


Simplifies event filtering in the operator

Refactoring of reconcilers

I'm a little uncomfortable because the change ranges over all of the controllers and I don't have a clue how to construct automated tests that give confidence in the conditions in which our controller Reconcile functions get called

I also do not know how to do it at the moment. We might want to look into sigs.k8s.io/controller-runtime/pkg/envtest. As far as I understand integration tests are recommended way to test controllers. This however sounds like a lot of work.

How confident are you in the PR / how well manually tested is it?

Not super confident because we do not have automated tests, but the confidence level is still high. I spent several hours yesterday going trough all the controllers on my test cluster and triggering the reconciliation by editing/deleting resources (this is how I found this which probably deserves a separate PR and routefix namespace issue fixed in #1363).

Are you ready for it to be merged?

Yes, I think so.

Do you see anything actionable to be done wrt the pullsecret_controller.go conversation?

Yep, I'll create a story to investigate and improve, if possible. Adding a finaliser to the object might help as we can not reconcile it inside of the cluster (becuase the data for secret is coming from the RP).


Localise log messages indicating SonarQube enabled/disabled on Gradle…

… and Maven.

Hi @HSAR, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

This PR has been rebased onto master and is current at the time or writing.

LGTM


feature/add possibility to set default maxParallelism for all extension methods

This lets you to set default maxParallelism for all of extension methods.

LGTM. Possibly too late, but couldn't "max parallelism" simply be "concurrency"?

I'm wrong, seems .NET docs https://docs.microsoft.com/en-us/dotnet/standard/parallel-programming/task-parallel-library-tpl prefer parallel. I assumed they would go the other way.  "Parallel" is more literal too.  Sorry 😊


[OData Function]'s URL should not use the prefix of [oData Resource].

Remove prefix when request is OData function, because he will confuse the boundaries resource and function.

Unnecessary.


Add an experimental CLI argument for passing a configuration value as JSON.

This PR adds a new CLI argument, --experimental-configuration-path, which can be used to specify the path to a file containing an instance of __CommandLineArguments_v0 encoded as JSON. /dev/stdin or a named pipe can be specified if desired.
Changes to SwiftPM will be needed to support passing this argument all the way through from swift test.
Checklist:

[x] Code and documentation should follow the style of the Style Guide.
[x] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci please test


Minor update to UI help text

Description:
Update the help text for API Properties in the Manage tab.
Suggested Labels:
2.2.0, Type/UX
Suggested Assignees:
Affected Product Version:
2.2.0
OS, DB, other environment details and versions:
Steps to reproduce:
Log in to the API Publisher, choose to edit an API and click the Manage tab. Expand the API Properties section.
Suggestion is to change the help text to something similar to the following:
The property name must not contain any white spaces and must not use any of the following reserved keywords: name, description, provider, version, context, doc, tags, Icstate, status, subcontext.
Also, it would be nice to update the prompt text to have the first letter in title case, i.e. Property name and Property value.

Related Issues:
N/A

This is fixed with the 3.x.


Test of new stack traces datastructure

I have created ParallelStacksProfiler, a quick clone of MergedCallStacksProfiler to test the new stack trace datastructure. Instead of refactoring MergedCallStacksProfiler I wanted to compare both implementation and output side by side.
TreeNode code is very easy to understand, it will certainly be simpler to maintain and to adapt. I really like the idea of using HashMap property to aggregate call stacks, so much cleaner! Overall I think we could simply trash the old implementation. Good job!!
A few remarks though:

To print stacks from top to bottom I still have to do a revert (line 132) as the do_stack_snapshot method gives us methods_ids as a stack (to read from bottom to top)... if you think the prefered way to read stacks are the other way around maybe we should do the revert in the build_from_sequences tree method
we loose the thread_ids... i know we don't make any use of them right now, so not a big deal at the moment, but I wanted to point it out
depth in TreeNode can be discarded I think

We don't have to merge this PR, I pushed it in case you wanted to have a look at how MergedCallStacksProfiler could look like with TreeNode... but something tells me you already have a more generic approach to print TreeNode and maybe already refactored both MergedCallStacksProfiler  and CpuHotpathProfiler XD

Nooo haha I just did this, so I was really happy to see your PR. Didn't had time to check it yet however

To print stacks from top to bottom I still have to do a revert (line 132) as the do_stack_snapshot method gives us methods_ids as a stack (to read from bottom to top)... if you think the prefered way to read stacks are the other way around maybe we should do the revert in the build_from_sequences tree method

Ah yesss didn't thought of that. I think you put it in the best place. Maybe the code to rebuild all sequences but reverted can be compacted a bit, will see if I can get something

we loose the thread_ids... i know we don't make any use of them right now, so not a big deal at the moment, but I wanted to point it out

Yeah true.... Made some changes so that it is a little more flexible: https://github.com/ogxd/dr-dotnet/pull/58

depth in TreeNode can be discarded I think

Ok !

merged call stacks profiler has been replaced by pstacks profiler that make use of the new tree structure. The output is exactly the same than before. Unfortunately I haven't yet been able to refactor the rendering of the tree and make it available to other profiler... it's a bit tricky


Public dice rolls (attributes or combined with abilities) are not shown in the chat

Steps to reproduce:

Log in with at least two different players.
Roll any combination as public roll

Expected:

All players see the result as chat message

Currently:

message only appears for this player.

fixed


Fix escaping in exclusion filter

The files that should be ignored are called simply logcat or 0000_logcat. The extra backslash is not working correctly, with the current version of GCloud SDK it's throwing an exception CommandException: Invalid exclude filter (".*\.txt$|.*\.apk$|.*\logcat$|.*\.mp4$|.*\.txt$")

@piotrmadry will you have time to check and merge it? This makes the download functionality broken by default (workaround is to include Logcat files in download, even if you only care about JUnit results)


Terminate stream when navigating away from stats page

Signed-off-by: Tom Barlow<EMAIL_ADDRESS>

LGTM


README-pt-br

Solves #10
Proposed Changes
README on portuguese-br on a separate file
Recommendations
in the README, "This repo is a fork of the same project, but for Japan:", the right should be "but for Brazil" or "This repo is a fork of a Japan project" to give the idea that the project is for brazil

@tiagobpires Thank for this PR! You're right, the sentence on the English README sounds confusing. Can you update that line with your recommendation and add it to this PR?

Of course!
Already added in this PR. Congrats for this project.


Daily temperature/precipitation forecast in forecast.io

Description:
The API response from forecast.io contains not only the current weather data that is presently used but also the actual forecast. I now made a few of these available, namely the daily high and low (apparent) temperatures and the daily maximum precipitation intensity. I think this can be useful for heating or garden automations. Note that this does not change the API calls.
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):
sensor:
  platform: forecast
  api_key: YOUR_APP_KEY
  monitored_conditions:
    - temperature_max
    - temperature_min
    - apparent_temperature_max
    - apparent_temperature_min
    - precip_intensity_max

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Looks good! :dolphin:


Message correction

In the receiver we have deploy 2nd , while we send deploy next.
fixed.

related to the Issue:
https://github.com/tact-lang/tact-by-example/issues/7


Help, can you share a link to this model?：Kataragi_inpaintXL-loral28 [47d4edb1]

https://huggingface.co/kataragi/controlnetXL_inpaint