⚠ GitLab Server has degraded performance

In cfc23e1, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 10960 ms

Resolved: GitLab Server performance has improved in 137bf1f.


Add search activities

Add the search in roomS activity.
This activity should be improved later by adding the search by room and member names.


[Hidive] Can only download the account's default subtitle/dub option

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that a supported site is broken
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
United States
Provide a description that is worded well enough to be understood
I can only download the "Spanish LatAm" subtitles, when many other options (English, Portuguese, German, Arabic, and more) are available on the site.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['-u', 'PRIVATE', '-p', 'PRIVATE', '--list-subs', '-vU', 'https://www.hidive.com/stream/ahiru-no-sora/s01e001']
[debug] Encodings: locale cp932, fs utf-8, pref cp932, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[141a8dff9] (win_exe)
[debug] Python 3.8.10 (CPython AMD64 64bit) - Windows-10-10.0.22000-SP0 (OpenSSL 1.1.1k  25 Mar 2021)
[debug] exe versions: ffmpeg 5.0.1-full_build-www.gyan.dev (setts), ffprobe 5.0.1-full_build-www.gyan.dev
[debug] Optional libraries: Cryptodome-3.17, brotli-1.0.9, certifi-2022.12.07, mutagen-1.46.0, sqlite3-2.6.0, websockets-11.0.1
[debug] Proxy map: {}
[debug] Loaded 1791 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp-nightly-builds/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: 2e4ba37c2958203b7326e04fd2f73ee4cc326ea1ba863b53e4bf03ffb6d5d754
yt-dlp is up to date<EMAIL_ADDRESS>[HiDive] None: Downloading webpage
[HiDive] Logging in
[HiDive] Extracting URL: https://www.hidive.com/stream/ahiru-no-sora/s01e001
[HiDive] ahiru-no-sora/s01e001: Downloading JSON metadata
[HiDive] ahiru-no-sora/s01e001: Downloading m3u8 information
[HiDive] ahiru-no-sora/s01e001: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[info] Available subtitles for ahiru-no-sora/s01e001:
Language           Formats
spanish-latam-subs vtt

I think I have a fix for this, will try to create a PR for that in the next days.

It's worth noting that this is a regression in the extractor, it worked in an older version, here is a comparison from when it worked vs when it broke:
https://github.com/yt-dlp/yt-dlp/compare/2023.01.06...2023.02.17

cc @chexxor

per discussion on discord, the old subtitles extraction code still works and retrieves all available languages

Yeah, that looks like my change broke it. When I was making my change, I couldn't figure out where it was getting the subtitle info from, so I wasn't able to make that work well again. Feel free to copy-paste the previous subtitle extraction code into the main branch again.

FYI, above, I wrote::

I think I have a fix for this, will try to create a PR for that in the next days.

Turns out my "fix" looks pretty much like what the extractor was doing before the regression was introduced in 7708df8, so let me know if you still want me to open a PR with my code, or if someone wants to restore the old code.

Yes, pls make PR. Even if it's just reverting to old code, I don't have account to test.

Yes, pls make PR. Even if it's just reverting to old code, I don't have account to test.

Sorry, might take a while longer, I'm dealing with some personal stuff at the moment.

Hello, will the fix be pushed in soon? I am looking to download Hidive episodes, but have been stuck on this issue.

I’ll try to have a look soon. Am 03.10.2023 um 02:06 schrieb nagi-neko1 @.***>:
Hello, will the fix be pushed in soon? I am looking to download Hidive episodes, but have been stuck on this issue.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

Closing due to #9385


[no ticket] Cache pip deps in travis

Purpose

Make builds faster by caching pip dependencies in Travis
Changes

Enable pip caching in Travis. See https://docs.travis-ci.com/user/caching
QA Notes

Documentation

Side Effects

Ticket

No ticket

Thanks. I knew we were caching some stuff in $HOME/.cache, but I wasn't sure if we were caching all pip deps. As you said, this shouldn't hurt.


Tour Steps Remain Highlighted

I've implemented the aSimpleTour and noticed when you click on Previous in the controls the previous steps from the one you are currently on remain highlighted.  I thought I may have missed something in the configuration but I see the same result in the demo.

Thanks for your report. I will fix this soon..

Fixed on version 1.0.7


[Bug]: Button drag issue in firefox

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Button is not draggable if drag starts from empty area in button. If I start dragging from the text, button is draggable. Happens in Firefox 91.5.
This doesn't happen in chromium, button can be dragged from any area.
https://user-images.githubusercontent.com/1573771/151161388-e69fe29c-8f59-4c0f-9c6a-f0ffb59772d1.mp4
Steps To Reproduce

open firefox
drag drop a button
increase size of the button
drag from empty space, it is not draggable. (bug)
drag from text, it is draggable.

Environment
Production
Version
Cloud

We can verify is this happening because of the pointer-event:all as pointed out by @aswathkk https://github.com/appsmithorg/appsmith/pull/8206


Include benchmarks in CI

@jdoliner has added a benchmark suite (make bench).  It'd be nice if we can add it to CI so we don't lose track of changes in performance.
Partially solves #199

I like this idea in theory, I'm a bit worried that the performance of a travis box is so variable that it will be hard to separate the single from the noise when looking at CI results.
But given that the cost is adding a line to our travis file I can't see a reason not to do it.

In particular, TestWriteManyFiles in the fuse test suite is a stress test. In particular when doing 10k the CI build time is ~30 minutes. At 1k files its 17 minutes. We want to tighten the CI feedback loop, so I'm all for having a lower count for that test.
One thing we can do is run it as a post-job task. This gives us shorter test times but allows us to run benchmarks on CI

Does a post-job task run and get reported but not block the results from CI coming back? That would be perfect for our needs :)

duplicate


Unsatisfied dependency: nim (> 0.13.0)

Installing this with nimble gives:

Unsatisfied dependency: nim (> 0.13.0)

forgot an '=' ?

Thanks for the report.
Created the fix[https://github.com/nim-lang/python/pull/4].
@dom96, please merge!

Thanks dom96!
@enthus1ast, can you try it now, please?

jep works!
python installed successfully


A+C: add Jes Cok

Please take a look.

Codecov Report
Patch coverage has no change and project coverage change: -0.17 :warning:

Comparison is base (57e305c) 74.02% compared to head (8a17160) 73.86%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1880      +/-   ##
==========================================
- Coverage   74.02%   73.86%   -0.17%
==========================================
  Files         526      508      -18
  Lines       63123    62484     -639
==========================================
- Hits        46726    46153     -573
+ Misses      13833    13792      -41
+ Partials     2564     2539      -25

see 21 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update README.md

Signed-off-by: MileyFu<EMAIL_ADDRESS>

This PR's title doesn't match our requirement.
Please make sure your title includes the corresponding Jira id.
@MileyFu, please update it.


HumanEval/114 prompt disagrees with OpenAI's prompt

For HumanEval/114, the prompt provided in https://github.com/openai/human-eval is
def minSubArraySum(nums):
    """
    Given an array of integers nums, find the minimum sum of any non-empty sub-array
    of nums.
    Example
    minSubArraySum([2, 3, 4, 1, 2, 4]) == 1
    minSubArraySum([-1, -2, -3]) == -6
    """

whereas in EvalPlus, it is
import math

def minSubArraySum(nums):
    """
    Given an array of integers nums, find the minimum sum of any non-empty sub-array
    of nums.
    Example
    minSubArraySum([2, 3, 4, 1, 2, 4]) == 1
    minSubArraySum([-1, -2, -3]) == -6
    """

Is there a reason for this? Language models will typically generate different continuations for the respective prompts.

I think this is actually a bug here.
https://github.com/evalplus/evalplus/blob/c0598d8781801de28c94eca3e5ae1d7083ad8e6a/evalplus/data/__init__.py#L215-L218
I was hoping to apply the fix for "HumanEval/115" whose import statement is between the signature and docstring:

and that would affect some assumptions in our dataset making but generally should not affect the generated code.
I will double-check why the 114 index is mapped to "HumanEval/114" (I thought it was 0-indexed).

Thanks @ganler for clearing this up so fast!

HumanEval problems are 0-indexed so index 114 actually just point to 114. Working on a patch now.

Thanks for the report! The bug has been fixed. Please reinstall the nightly version to use the latest dataset:
pip install "git+https://github.com/evalplus/evalplus.git" --upgrade

The PyPI version will follow up later.


1.8: How to use list?

This is probably one for @mrjimenez
I've been experimenting with the extra headers (1.8) branch, and am a bit confused to add some items to the list from client code.
There is a method: UpnpFileInfo_add_to_list_ExtraHeadersList(UpnpFileInfo *p, list_head* head);
I was expecting to be able to add a ExtraHeaders* here, how do I make a list_head? Why is it needed?

Hi Ian,
I imagine you already know that, but for completeness, the relevant files are upnp/inc/TemplateSource.h and upnp/inc/TemplateInclude.h, where the methods are generated. And of course, upnp/inc/list.h, where the list implementation resides.
The "list" type used is a frozen version of the linux kernel list. I tried to keep the file as close as possible to the original, so that it would be easy to put patches and keep it up to date.
The clever mechanism in this list is that it can be a list of anything you want, it really does not care about the structure that you want to be in that list. All that matters is that this struct contains a "struct list head something" member anywhere inside it. And it is a pointer to this struct that you get back when you call the get method "const struct list_head *CLASS##get##MEMBER(const CLASS *p)". All you need to know is that there is a "member function" to retrieve that member object. The object returned is a pointer to a "struct list head". And that list is totally "struct agnostic". You can put anything on that list, as long as it has a field that is a "struct list head" (not a pointer, the struct itself).
To recover your original struct from the list, you must walk that list with the functions and macros of the "list.h" file. For example, two of the most useful macros:
/**

list_entry - get the struct for this entry
@ptr:        the &struct list_head pointer.
@type:       the type of the struct this is embedded in.
@member:     the name of the list_struct within the struct.
*/
#define list_entry(ptr, type, member) \
container_of(ptr, type, member)

/**

list_for_each        -       iterate over a list
@pos:        the &struct list_head to use as a loop cursor.
@head:       the head for your list.
*/
#define list_for_each(pos, head) \
for (pos = (head)->next; prefetch(pos->next), pos != (head); \
pos = pos->next)

There are several macros there that do the necessary magic. The file list.h is worth a read.
Back when I put this list code in pupnp, the intention was to use something proven and practical and that was written in C, not C++. I think this list code is still used today to implement lots of things inside the linux kernel.
I hope I was clear, but if there are any doubts remaining, please ask.
Best regards,
Marcelo.

I am probably doing something wrong but getting:
                                                    ^
/usr/include/upnp-1.8/list.h:360:26: note: in definition of macro ‘list_entry’
  container_of(ptr, type, member)
                          ^~~~~~
/usr/include/upnp-1.8/list.h:360:2: error: ‘container_of’ was not declared in this scope
  container_of(ptr, type, member)
  ^

You are doing it right. There is missing code that should be in an include file.
We must add it somewhere, maybe in list.h.
#undef offsetof
#define offsetof(TYPE, MEMBER) ((size_t) &((TYPE *)0)->MEMBER)
/**

container_of - cast a member of a structure out to the containing structure
@ptr:        the pointer to the member.
@type:       the type of the container struct this is embedded in.
@member:     the name of the member within the struct.

*/
#define container_of(ptr, type, member) ({
const typeof( ((type *)0)->member ) *__mptr = (ptr);
(type *)( (char *)__mptr - offsetof(type,member) );})

I have just committed a new version of list.h, please check.
Also, I did a rebase on the philipe44-master branch, it this is the one you are using, maybe you will want to delete it and fetch it again.

My personal opinion is that exposing list_head in the upnp-API is wrong. Something like this is better to be kept internal. The list implementation changed since libupnp 1.6.x, and we have problems like
Debian Bug #884243 where programs started to use the upnp list implementation for their own use and so are harder to convert to libupnp 1.8.

@mrjimenez thanks, I will try it later.
@ukleinek I do not see how we can avoid exposing some kind of list here?

@v00d00: I'd let the _add routine take either only a single new item or a C array (+ a parameter for the length).

Hmm again probably doing something wrong here, but I cant seem to use this from C++.
I guess this needs an actual C compiler to work.

I am still having troube here with 1.8.5
auto head = const_cast<list_head*>(UpnpFileInfo_get_ExtraHeadersList(fileInfo));
UpnpExtraHeaders* h = UpnpExtraHeaders_new();
UpnpExtraHeaders_strcpy_name(h, "example-header");
UpnpExtraHeaders_strcpy_value(h, "example-value");
UpnpExtraHeaders_set_resp(h, "lolcat");
UpnpExtraHeaders_add_to_list_node(h, head); // ?
// or
UpnpFileInfo_add_to_list_ExtraHeadersList(fileInfo, head); // ?

But nothing seems to be produced
Apolgies in advance...

Hi Ian,
Your code is ok, I don't know why nothing is produced. Both add_to_list's are fine, I just don't have it in my mind right now how the structures are supposed to be used.
In fact, these templates were meant to be used in a C compiler, but there is nothing stopping you from using it from a C++ compiler.
Designing this API for a C++ compiler would in fact simplify things a lot by using real classes.
No need for apologies, if I can help somehow, please tell me.
Regards,
Marcelo.

Not sure why the above was giving me trouble... PEBCAK for sure.
Example of use for adding headers:
auto head = const_cast<list_head*>(UpnpFileInfo_get_ExtraHeadersList(fileInfo));
for (auto header : *headers) {
    UpnpExtraHeaders* h = UpnpExtraHeaders_new();
    UpnpExtraHeaders_set_resp(h, header);
    UpnpExtraHeaders_add_to_list_node(h, head);
}

Do not include a trailing CLRF or you will destroy the HTTP payload :-1:
I wonder if we need to patch the const-ness? Perhaps make CLASS##_add_to_list_##MEMBER take a const value?

And probably make ExtraHTTPHeaders() in webserver.c call the UpnpExtraHeaders_add_to_list_node method rather using list_add directly

I think they are both good ideas. If you have a patch I will accept it happily.


Fixed a typo: describe => described

@rasmusgude,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@jimdial Please review these proposed changes. Thanks.

@rasmusgude  Thanks for catching, we'll merge it.  #sign-off


Igor/feature/tx status fix

Travis automatic deployment: https://76-pr-ching.surge.sh

Can you pad the "confirmed!" a bit?


error to document for replication

Seems like this error may surface when authorizing replication is not done timely.

"message":"Volume replication operation: 'Authorize' failed, reason: Timeout during peering infrastructure setup

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

ID: 404bbddd-25fa-e791-5019-d33c9956a363
Version Independent ID: 11d54df6-c231-5d6a-532a-8c76b4ae6141
Content: Troubleshoot cross-region replication errors for Azure NetApp Files
Content Source: articles/azure-netapp-files/troubleshoot-cross-region-replication.md
Service: azure-netapp-files
GitHub Login: @b-hchen
Microsoft Alias: anfdocs

@MrRitt
Thanks for your feedback! We will investigate and update as appropriate.

@MrRitt
I'm going to assign this to the document author so they can take a look at it accordingly.
@b-hchen
Please review it.

@MrRitt - thanks for raising this issue. I've consulted with engineering and product management. This error is related to timeout but the larger issue requires a support request to resolve.
#please-close

This situation doesn't require a case. Customers can authorize the volume within x amount of time before this is of issue.  Only after authorization has expired is this of issue.


Issue in ios push notification subscription

CB-SDK: : [subscribe] subscription ERROR: ResponseException: 422: {"errors":{"client_identification_sequence":["is invalid"]}}
Specify the sample to which the issue belongs (use [x]):
[ ] Chat sample
[ ] P2P Call sample
[ ] Conference Call sample
Platform (use [x])
[ ] Android
[ ] iOS
Device info
Manufacture:
Model:
OS version:
Describe the bug:
Steps to Reproduce:
1.
2.
3.
4.
Logs:
log here

or
Gist log
Actual result:
Expected behavior:
Additional info
Add any other context about the problem here.

This error means you set a not valid token for the subscription.
Please provide logs with the creation subscription request and response. It will help us to understand your issue.

requset :
{"notification_channel":"apns","device":{"udid":"787E56E1-1AE7-41D3-83A1-5556028E7449","platform":"ios"},"push_token":{"environment":"development","client_identification_sequence":"f6QVAvlWUU4Gorp8G3n5D3:APA91bECli8gjaU5nlTZ1UDrwvh3AaBGK5ig1CtByd3L35ZmOm2k-n1bWHsFk5bMkHxNvAwbQZYzMeLPLSgKAFI8t9DG3zFEnB9mMP2_UqNxxjH8Xdgef4K9jsIHT9aU9KQJL81kh8Hk","bundle_identifier":"com.packagename"}}
response:
{"errors":{"client_identification_sequence":["is invalid"]}}
flutter:
flutter: CB-SDK: : [subscribe] subscription ERROR: ResponseException: 422: {"errors":{"client_identification_sequence":["is invalid"]}}

requset :
{"notification_channel":"apns","device":{"udid":"787E56E1-1AE7-41D3-83A1-5556028E7449","platform":"ios"},"push_token":{"environment":"development","client_identification_sequence":"f6QVAvlWUU4Gorp8G3n5D3:APA91bECli8gjaU5nlTZ1UDrwvh3AaBGK5ig1CtByd3L35ZmOm2k-n1bWHsFk5bMkHxNvAwbQZYzMeLPLSgKAFI8t9DG3zFEnB9mMP2_UqNxxjH8Xdgef4K9jsIHT9aU9KQJL81kh8Hk","bundle_identifier":"com.packagename"}}
response:
{"errors":{"client_identification_sequence":["is invalid"]}}
flutter:
flutter: CB-SDK: : [subscribe] subscription ERROR: ResponseException: 422: {"errors":{"client_identification_sequence":["is invalid"]}}

resolve using apns token

resolve using apns token


updates

adds cleanup command and tests
update command description

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (b6c0d69) 52.38% compared to head (ce74e07) 53.62%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

cmd/cleanup.go
0.00%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   52.38%   53.62%   +1.24%
==========================================
  Files           6        7       +1
  Lines         231      248      +17
==========================================
+ Hits          121      133      +12
- Misses        107      112       +5
  Partials        3        3

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


How to realize the attention maps？

Hi, thanks for your wonderful work especially the detailed code! But I am confused and have some questions about the attention maps. I only have a general understanding of the mechanism of Tranformer. I want to know how to understand the tokens mapping between images and lidar. Specifically,

In the  supplementary, I think these are the same two scenarios, but the green areas appear in two different places when you query the traffic lights in the same location. How do you explain this phenomenon?

Also, in the circled scene above, does the green area in the lidar map change if the traffic light in the image is red? That is to say, does agent's focus change as the traffic lights change?

Finally, I just feel confused and can you explain why the lidar is mapped to this location in a situation like this?

Thank you in advance for your answer!

Thank you for your quick response. But I still don't quite understand about the third point that sides of the intersection are mapped to each. Could you please give me some other examples about the visualizations?

I see. I will try to run the code later and ask you when I meet a problem. Thank you! Btw, is there another way to download the dataset? I download it slowly on Amazon or sometimes I can't connect to it.

Unfortunately, there is no other way to download the dataset.

I see. Thank you :)


creating the first draft of the landing page

Added a Welcome header, text and three widgets without function.

@Melodycel Is this pull request ready for review or still a draft?


Remote database with secure connection by certificate

SQLSTATE[HY000] [3159] Connections using insecure transport are prohibited while --require_secure_transport=ON.
The website is functioning alright, PHP-CLI also works fine, and tested the connection to DB with the tinker.

Found and fixed it within the Database/SqlDatabase.php
    /**
     * Constructor.
     *
     * @param string     $dsn
     * @param string     $user
     * @param string     $password
     * @param null|array $options
     */
    public function __construct($dsn, $user, $password, $options = null)
    {
        if (is_null($options)) {
            $options = [
                PDO::ATTR_ERRMODE            => PDO::ERRMODE_EXCEPTION,
                PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
                PDO::MYSQL_ATTR_SSL_CA => 'some/ridiculous/filepath', // this is the really dirty fix, because the certificate will not be checked
                PDO::MYSQL_ATTR_SSL_VERIFY_SERVER_CERT => false // this is the really dirty fix, because the certificate will not be checked
            ];
        }

        $this->pdo = new PDO($dsn, $user, $password, $options);
    }

Too lazy to create a PR, sorry.
Also seems this repo is dead.


Google Tag Manager with turbolinks: true duplicates pageView when using Push

It looks like GTM with turbolinks: true (#88) works only for pushing basic page view to the Data Layer:
handler :google_tag_manager, container: 'GTM-123456677', turbolinks: true
which will render in HEAD section:
<script>
      document.addEventListener('turbolinks:load', function(event) {
        var url = event.data.url;
        dataLayer.push({'event':'pageView','virtualUrl': url});
      });
  (function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
  new Date().getTime(),event:'gtm.js'});var f=d.getElementsByTagName(s)[0],
  j=d.createElement(s),dl=l!='dataLayer'?'&l='+l:'';j.async=true;j.src=
  'https://www.googletagmanager.com/gtm.js?id='+i+dl;f.parentNode.insertBefore(j,f);
  })(window,document,'script','dataLayer','GTM-123456677');</script>

Turbolinks will not duplicate this script tag on other page views because it's always exactly the same.
However, when user navigates to another controller (with Turbolinks) where I call something like this:
t.google_tag_manager :push, ecommerce: {
    impressions: @products.map { |product| { id: product.id } }
}

...I end up with two script tags in HEAD section:
<head>
<script>
      document.addEventListener('turbolinks:load', function(event) {
        var url = event.data.url;
        dataLayer.push({'event':'pageView','virtualUrl': url});
      });
  (function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
  new Date().getTime(),event:'gtm.js'});var f=d.getElementsByTagName(s)[0],
  j=d.createElement(s),dl=l!='dataLayer'?'&l='+l:'';j.async=true;j.src=
  'https://www.googletagmanager.com/gtm.js?id='+i+dl;f.parentNode.insertBefore(j,f);
  })(window,document,'script','dataLayer','GTM-123456677');
</script>
<script>
      document.addEventListener('turbolinks:load', function(event) {
        var url = event.data.url;
          dataLayer.push({"ecommerce":{"currencyCode":"HRK","impressions":[ ........ ]}});
        dataLayer.push({'event':'pageView','virtualUrl': url});
      });
  (function(w,d,s,l,i){w[l]=w[l]||[];w[l].push({'gtm.start':
  new Date().getTime(),event:'gtm.js'});var f=d.getElementsByTagName(s)[0],
  j=d.createElement(s),dl=l!='dataLayer'?'&l='+l:'';j.async=true;j.src=
  'https://www.googletagmanager.com/gtm.js?id='+i+dl;f.parentNode.insertBefore(j,f);
  })(window,document,'script','dataLayer','GTM-123456677');
</script>
</head>

This unfortunately creates duplicate pageviews in Analytics etc.
Am I doing something wrong or this is a real issue?
Thanks! :)

@viktorzimmermann sorry that this got lost... do you have an update on this from your side?

I'm at the exact same point, I can't make it work 🤯 Did you find a solution to this after all @viktorzimmermann ?

Unfortunately I still didn't find any solution. 🙄

At the same point now myself. Renders rack-tracker useless for me sadly.

sorry, sadly I can not really help here.
Any ideas on how to fix this are welcome.

We released a hotwire/turbo related change, maybe you can check this out: https://github.com/railslove/rack-tracker/pull/160/files


net: context: Set priority based on DSCP

If a socket has DSCP set then the packets from the socket should also be marked with appropriate priority in case traffic classes are used in networking stack.

In the RX path, I see that most of the Ethernet drivers have duplicate code by deriving from VLAN tag or hardcode (PTP), so, should we just move the logic of setting priority to networking stack based on VLAN or DSCP in IP header if priority is not set by driver? Drivers can still overwrite.

In the RX path, I see that most of the Ethernet drivers have duplicate code by deriving from VLAN tag or hardcode (PTP), so, should we just move the logic of setting priority to networking stack based on VLAN or DSCP in IP header if priority is not set by driver?

If tehre's a standardized way to handle this, then I guess it'd make sense, however the handling should not extend beyond Ethernet L2.

Drivers can still overwrite.

Not sure I follow - on RX path, packet is processed at the driver level before it's passed to the network stack, so I don't see how drivers could overwrite the prio.


Release 3.0

I would like to release 3.0 very soon but I am a bit worried because I am 100% sure the cache/index format will change very soon for #54. But don't really want to wait for #54 because that could also introduce new bugs.

Maybe it would be better to disable cache for now? An additional argument for this is decrease of indexing speed with this simple cache (for large projects).


Enabling Dark Mode

Dark Mode enabled in all pages.
Issue encountered : flip not working in homepage.
(team femmetech) @sonalsk

change the logo in this section and try to add better colours to it.
and it's having this issue to just check it once
https://drive.google.com/drive/folders/1V3NW8SqzgRDPi4oBQbqgZc9PmKOGEDyu?usp=sharing

change the colors of them

Strangely, Its completely fine in my development environment. Can you please tell me which browser you are using? @anushbhatia

Chrome browser only

I am using mozilla, that is why the difference @anushbhatia

Can you verify it on Firefox? If fine, I'll add those changes in chrome as well

I saw the video, this is also not occurring in firefox. PS. The chrome rendering is slightly displaced too

There is an extra tile of image.What do you think should be done?

How have u converted the elements of the website by coding each of them or something different?

Actually, chrome recently started supporting mix-blend-mode so there might be bugs. What we can do is that we can force the image to remain same and also the header can be shrunk a little bit so its not visible. @anushbhatia If we can't do this, there is certainly no way to implement dark mode without npm packages then or react maybe.

check the website of dsc vit they have implemented dark mode too. I can give the code of the whole website if u want.
https://dscvit.com/

Send me the code, I'll see😊

https://drive.google.com/drive/folders/1V3NW8SqzgRDPi4oBQbqgZc9PmKOGEDyu?usp=sharing

@anushbhatia We can't do that.  For that to happen, we would have to makeover the whole website.

I will discuss bout this issue with my team and tell u.


Question: Is whitelisting attributes in allowedAttributes a good idea?

For example: If I use allowedAttributes: { img: ['src'] }, does it exclude the src tag from analysis?

If disallowed, "href" and "src" are completely removed. If allowed, "href"
and "src" are subject to the naughtyHref test. So yes, it is a good idea.
On Fri, Jul 15, 2016 at 8:46 AM, James Calfee<EMAIL_ADDRESS>wrote:

For example: If I use allowedAttributes: { img: ['src'] }, does it
exclude the src tag from analysis?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/punkave/sanitize-html/issues/117, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB9fTq6QQX516Cos0GCjKhAL9S04EE9ks5qV4ESgaJpZM4JNXV1
.

--
*THOMAS BOUTELL, *SUPPORT LEAD
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com


Readme update :)

Replace publish

php artisan vendor:publish --provider="Vinkla\Hashids\HashidsServiceProvider"

Instead of

php artisan vendor:publish

Because we only want to publish the config of your package

Thanks but this has already been up for discussion before (#24, #29).


Temp fix: Explicitly include stellar-base

Version update for: https://github.com/stellar/laboratory/pull/501 (I can't update the fork).

Preview is available here:http://laboratory-pr502.previews.kube001.services.stellar-ops.com/


Jd pull requests - Added remaining sections of the page

Updated index.html / mobile_first.css
Completed the sections that were missing from the main template. Fixing the styles for those sections and added one last img for the last section background.
Please, let me know if I missed to add something or if I did great. It's been plenty of hours of coding these days and I learned a lot but didn't sleep enough jeje 😅.
I'll patiently wait for a review.

@LionRouge1 Hello! Thank you for your feedback, I just woke up and I'm happy to read that I achieved the goal. I will also try to improve what you suggested me. I tried to make it as similar as possible and maybe in my perspective (screen sizes tested) looked good and I didn't notice the differences but I know I can do it better next time!


charmc: MPI build errors when trying to pass -Werror

Passing -Werror to charmc for MPI builds causes a compilation failure. It appears to have been introduced in this commit: https://github.com/UIUC-PPL/charm/commit/3261467e797a03883c685955d40bdaf50eb1f867
To reproduce, build charm with the following command:
./build LIBS mpi-linux-x86_64 --build-shared --with-production -j12 -O3 -DNDEBUG
Then attempt to compile examples/charm++/hello/array1d, but add -Werror as an argument to either the compile or link command:
../../../../bin/charmc -Werror -c hello.C
I got the following error:
g++: error: : No such file or directory
Fatal Error by charmc in directory /home/epmikida/Programming/charm/examples/charm++/hello/1darray
Command mpicxx -fPIC -DCMK_GFORTRAN -DMPICH_SKIP_MPICXX -DOMPI_SKIP_MPICXX -I/home/epmikida/Programming/charm/include -D__CHARMC__=1  -U_FORTIFY_SOURCE -fno-stack-protector -fno-lifetime-dse -ftls-model=initial-exec -std=gnu++11 -c hello.C -o hello.o returned error code 1
charmc exiting...

@evan-charmworks do you have any insight on this?


[Feature Request] change before-upload behavior

Existing Component
是
Component Name
upload
Description
I was using a function to prevent upload，code like this
onBeforeUpload() {
  doSomthingBeforeUpload()
  return false
}

then one-day doSomthingBeforeUpload became a promise
async onBeforeUpload() {
  await doSomthingBeforeUpload()
  return false
}

this won't stop the upload, because this function actually resolves with false, not rejected, so I have to throw an error in doSomthingBeforeUpload or onBeforeUpload.
I read the code https://github.com/ElemeFE/element/blob/1afce457e7a2ab3d5be83d0fc1973722ff6ba87f/packages/upload/src/upload.vue#L93-L111 and know that we thread Blob as a new upload file, and other values will upload the original file
Here is my propose: treat Promise<false> as a signal to prevent upload, I think it's better and will not break many use-cases.
If it's ok, I can make a pr to make this change

@ziyoung ping

async onBeforeUpload() {
await doSomthingBeforeUpload()
return Promise.reject() // false
}

@island205
island205 I still think my solution makes this much easier to use, image you have more than just a doSomthingBeforeUpload function but many lines with return false
Should we at least discuss?


⚠ Bofrost.es has degraded performance

In a0b261d, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1533 ms

Resolved: Bofrost.es performance has improved in acff1e1 after 6 minutes.


how to Build MXNet under windows? the official instructions link is unavailable

I download your repository of mxnet-ssd under windows. I met the error "MXNetError: Cannot find argument 'pooling_convention' when I ran the demo, Possible Arguments:". I guess the cause is that I didn't Build MXNet correctly because the link of the official instructions is unavailable. I

prebuild mxnet library are available here :
https://github.com/yajiedesign/mxnet/releases
I don't know if it's compatible with @zhreshold mxnet version. But you can use SSD code in official mxnet repository : mxnet/example/ssd.

Use pip install mxnet should be good enough now.


[backport] Make docs of connection functions refer ndarray

Backport of #5875

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit a3f2ca6683b2e48102bcb70d93ec7aea689f7798, target branch v5) succeeded!

This PR seems ready for merge.


[WIP] More tests, more coverage

Work in progress, more commits to come. Adds more testing and increases code coverage.

Marking as "ready for review" since Travis apparently won't run against draft requests, but not ready for review yet.

Codecov Report

Merging #100 into master will increase coverage by 0.53%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #100      +/-   ##
==========================================
+ Coverage   46.93%   47.47%   +0.53%
==========================================
  Files          45       45
  Lines        2237     2237
==========================================
+ Hits         1050     1062      +12
+ Misses       1187     1175      -12

Impacted Files
Coverage Δ

...l/plugins/reporting/write_file_reporting_plugin.py
33.33% <0%> (+4.16%)
:arrow_up:

...ins/reporting/print_to_console_reporting_plugin.py
100% <0%> (+61.11%)
:arrow_up:

Continue to review full report at Codecov.

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


feature: allow adding a sentence to exemplify how a word could be used

instead of just having a=b
consider a(example)=b
the example if exists will be shown under the translation box

implemented in commit 7334d24


Setup English community meeting

What type of PR is this?

/kind documentation
What this PR does / why we need it:
Which issue(s) this PR fixes:
Resolves #30,  resolves karmada-io/karmada#1560
Special notes for your reviewer:

/assign @RainbowMango


Fix hash from branding repo

Description
I merged another PR with an error on the branding repo :( Fixed on the other one and by changing the hash here.
This should have probably been caught by CI, I assume it didn't because CI runs with the celo branding rather than the valora branding. Maybe there's something we could do there to avoid this in the future.

Hi @gnardini Can you please let us know if anything need to test here from our side ?

No need to test anything here, thanks!


Accessing instance, PK, or criteria in lifecycle callbacks

I have a blog system that converts a post title into a slug (e.g. Hello World -> hello-world). Titles don't need to be unique, but slugs do, so I need to add logic to my model that determines if a post already has the hello-world slug and, if so, changes the slug to hello-world-2 and so on.
Waterline lifecycle callbacks don't receive an instance, only the attributes that are being changed. This is understandable, since the underlying implementation seems to call Model.update, not record.save.
The issue I'm having is that, if I have just one post called Hello World with slug hello-world, and I edit and update that post, it changes the slug to hello-world-2, because, of course, there already is a post in the database with slug hello-world. If I knew the ID of the post I was editing, I could add a condition in my slug conflict checker function that says if (potentialConflictPost.id !== postWeAreEditing.id). But without having access to the PK in the lifecycle callback, I have no way of knowing this information.
Basically, I think it's a crucial feature to get the instance, PK, or criteria passed into the beforeCreate and beforeUpdate lifecycle callbacks somehow.

For 0.11.x I am quite sure you'll at least find criteria passed to these methods.  See issue/request #1004.  Do you mind chiming-in on that thread then closing this as a duplicate?  You have a great example use-case!


Generator Function

While talking with @GottZ, they raised the idea to use generator functions in the core aspects of the sentence generation.
The creation of multiple objects over a small time period would put the tasks to a queue as the process computes each one synchronously. A proposition to use generator functions to create iterable interfaces is a reasonable solution to circumvent the current issue with word linking and sentence outcome.
The main focus points to this improvement is performance and project readability, both are equally important in this case as this is a new area for me as a student.

Hitch a point onto this thread.
Cluster / WebWorker, process.nextTick()


Timezone implementation error, case:  'America/La_Paz' and now() function

Laravel Version: 8.10.0
PHP Version: 7.4.10

Description:
i got an error when i set up config/app.php with 'timezone' => 'America/La_Paz'

if i code --> echo now()." - "; return(now());
i got output --> 2020-10-16 19:59:08 - "2020-10-16T23:59:08.868845Z"

so I got 2 different dates, but if I don't modify the timezone which has by default 'timezone' => 'UTC' in config/app.php i got 2020-10-17 00:28:08 - "2020-10-17T00:28:08.099556Z" this is a equivalent date.
Steps To Reproduce:

change 'timezone' => 'UTC' in config/app.php to 'timezone' => 'America/La_Paz'
code in any controller with index method --> echo now()." - "; return(now());

The echo usage converts to a common string representation and uses a timezone.
The return usage tried to convert to a JSON representation, which is UTC based, hence the Z in the end.
If you try to do this:
Route::get('test', function () {
    echo now()->format('Y-m-d h:i:s') . ' - ';
    return(now()->format('Y-m-d h:i:s'));
});

You will get both dates within the same timezone.

the problem is that im getting a list of users with timestamps created_at, updated_at powered by api and im getting the problem with dates distinct, i think that laravel config timezone does not act globally on the application.


Catch all kinds of Exception not StandardError only

Currently only StandardError exceptions are caught by Delayed::Worker#run, raising an Exception while processing a job causes that worker is crashing.

Coverage decreased (-0.29%) when pulling d9789f42cc11b03d7306171302f5d2cba777801d on knapo:master into 1f881569603e45b93b8b1f21ecfa85379645fe3c on collectiveidea:master.

Coverage decreased (-0.29%) when pulling d9789f42cc11b03d7306171302f5d2cba777801d on knapo:master into 1f881569603e45b93b8b1f21ecfa85379645fe3c on collectiveidea:master.


selector resolved to hidden error

while trying to click the checkbox using the following test, it fails with the following error:  selector resolved to hidden check
async checkActiveStatusdom(text) {
const header = await this.screen.findByTestId('erow-GroupCode-0');
console.log(" header" + header)
const base = this.within(header).getByText("LINEHOLDER")
console.log("base value" + base);
const check = this.within(header).getByText("check")
console.log(" value " + check)
await expect(base).toContainText(text);
console.log("text assertion successful")
await check.click();  ---> fails
}
output:
headerLocator@query-by-test-id=["erow-GroupCode-0"]
base valueLocator@query-by-test-id=["erow-GroupCode-0"] >> get-by-text=["LINEHOLDER"]
value Locator@query-by-test-id=["erow-GroupCode-0"] >> get-by-text=["check"]
text assertion successful

[chromium] › tests/web/vacation.bidaward.defaults.spec.po.new.js:14:9 › Bid and award Default page verification › Default page verification

Test timeout of 30000ms exceeded.

locator.click: Target closed
=========================== logs ===========================
waiting for selector "query-by-test-id=["erow-GroupCode-0"] >> get-by-text=["check"]"
  selector resolved to hidden <title>check</title>
attempting click action
  waiting for element to be visible, enabled and stable
    element is not visible - waiting...
============================================================

It looks like you're attempting to click on the SVG <title /> element, which is not a visible element. I suggest you review the following documentation to get a better handle on the general Testing Library philosophy and how it is intended to be used to help you test your application more closely to how your users interact with it:

https://testing-library.com/docs/guiding-principles
https://testing-library.com/docs/dom-testing-library/faq
https://testing-library.com/docs/dom-testing-library/api-accessibility

In this case, ideally, you'd instead be querying for the native checkbox control using an accessible label, like so:
const header = await this.screen.findByTestId('erow-GroupCode-0');
const check = this.within(header).getByRole("checkbox");

await check.click();

Once you're reviewed that, if you're still having this kind of problem, it's really an issue with how you're using Playwright, and it has nothing to do with Playwright Testing Library. I suggest you try to debug your tests using Playwright's robust debugging capabilities before opening an issue here:

https://playwright.dev/docs/debug#playwright-inspector
https://playwright.dev/docs/debug#actionability-logs

It looks like you're attempting to click on the SVG <title /> element, which is not a visible element. I suggest you review the following documentation to get a better handle on the general Testing Library philosophy and how it is intended to be used to help you test your application more closely to how your users interact with it:

SVG <title /> element - checkbox is visible on the webpage, and it works fine with regular playwright code, can you please clarify what you mean by saying that it is not a visible element while using
const check = this.within(header).getByText("check")
await check.click(); ---> fails

The <svg /> element is visible, but <title /> is an inherently invisible element (visible only to screen readers:
✅ Query + click SVG using <title /> as accessible name:

If you really want to click the <svg />, this is probably what you want, but since it's a stylized checkbox I think you really do want to query the checkbox itself as I recommended

https://github.com/testing-library/playwright-testing-library/blob/d5a665ba6d62c26e2dcdf83cca86a2a13ce219a6/test/fixture/locators.test.ts#L122-L128
❌ Query + click <title /> within <svg />:
https://github.com/testing-library/playwright-testing-library/blob/d5a665ba6d62c26e2dcdf83cca86a2a13ce219a6/test/fixture/locators.test.ts#L130-L139


There is a problem with the block browser

There is a problem with the block browser，we can find nothing use   http://explorer.starcoin.org/search/
just   http://explorer.starcoin.org/barnard/address/    can find our account address on explorer.

fixed by: https://github.com/starcoinorg/starcoin-explorer/commit/23d893c88a545d4d822c472b1be91b3180ff8e7f


fix(client): key order

Closes https://github.com/DHI-GRAS/terracotta/issues/314 as discussed in https://github.com/DHI-GRAS/terracotta/pull/317.

Awesome, thanks!


Lab 0: Getting to know you - Minasian

    {
        "preferred_name": "Haig",
        "csulb_email"<EMAIL_ADDRESS>        "pronouns": "he/him/his",
        "previous_gis_experience": "GIS Mapper/Project Coordinator for a homeless services non-profit working on collecting/analysis of homelessness data",
        "previous_remote_sensing_experience": "No remote sensing experience yet!",
        "previous_programming_experience": "Most of my programming experience has been through my education/classes.",
        "github_repo_link": "(https://github.com/carte-ophile/GISC606Minasian.git)" ,
        "github_project_link": "(https://github.com/users/carte-ophile/projects/2)",
        "what_you_hope_to_get_out_of_this_class": "I am very interested in remote sensing, but I am also really happy that we are",
        "utilizing GitHub to manage this work since it is what we are using in our Master's project",
    }

Hey @carte-ophile,
Thanks for sharing about yourself. Could you also share about your larger MS project taking place outside this class?
I'd like to curate the course content to the interests of the students (including you), so anything you can share about your project or interests in remote sensing will help me. It sounds like you are most interested in content related to GitHub (or source code version control) and it's use in Remote Sensing analyses/project management?
Along those lines, I'd like you to start thinking about your final project for this class. I'm encouraging all students to pick a topic related to their larger MS project, but you are free to select any apply remote sensing topic of interest.
Look forward to working with you this semester.
Best,
Alex

Hey Alex,
Sorry I didnt see your response, I dont know if I came across any sort of notification.
That being said, in short, our team is collecting microplastics data to help develop the Moore Institutes Open Data Portal on microplastics, specifically by building a webmap for it and linking it to their github portal. Check out the link to the github portal to learn more!
We can talk more about it, but like I had mentioned in our last class, learning about github is super relevant to our project, so this has been great so far.
Haig

Hey @carte-ophile
Could you share the links to your lab 2 and 3 here? Thanks.

Hey Alex!
I am sending you all my project links today! Sorry again for the lag :(
Here is Lab 2: https://github.com/carte-ophile/GISC606Minasian/blob/bf554ca29e9c898bd0cd30fe9072ae20d4531cd8/Minasian_GISC606_L2.ipynb

Lab 3
https://github.com/carte-ophile/GISC606Minasian/blob/a562245904ce3de7a14b23e282b46341144783fa/Minasian_GISC606_L3.ipynb

Lab 4
https://github.com/carte-ophile/GISC606Minasian/blob/febdf781eab132690e34aba81607de6b2d9d78e3/Minasian_GISC606_L4.ipynb

Lab 5!
https://carte-ophile.github.io/Minasian_Lab5/
Repository: https://github.com/carte-ophile/Minasian_Lab5/blob/main/README.md


Enable/Disable Anything Zoomer

I have added a button to enable/ disable AnythingZoomer so I can use d3 pan/drag/zoom on a chart when it is disable or use Anything Zoomer over the chart when it is enabled.  Is there a way to do this?  I've tried   $('#zoom3').data('zoomer').setEnabled(false); but it didn't work.

Hi @sjp700!
Hmm, that method does work for me (demo), so I'm not sure why there's an issue.
Anyway, the "official" method to enable or disable AnythingZoomer is to do the following (demo):
$('#zoom3').anythingZoomer('disable');

While making the demo, I noticed that after hovering over the disabled zoomers, the zoom window would reappear when the zoomer is re-enabled. I'll have that problem fixed soon.


How to upgrade Elasticsearch version?

Note: If your question is regarding the AWS Amplify Console service, please log it in the
AWS Amplify Console repository

Which Category is your question related to?
API
Amplify CLI Version
4.52.0
What AWS Services are you utilizing?
Elasticsearch
Provide additional details e.g. code snippets
I'm using @searchable to generate the Elasticsearch instance, but currently it using version 6 of Elasticsearch, how can I upgrade it to version 7?

Hi @yaquawa currently the only way to do this is to modify the cloudformation template that the CLI generates. You can copy the SearchableStack.json from <project root>/amplify/backend/api/<api name>/build/stacks/SearchableStack.json to <project root>/amplify/backend/api/<api name>/stacks/SearchableStack.json (copy it out of the build directory). Then you can update the ElasticsearchVersion property in that stack.
You'll need to be careful when making updates to your API that the override template stays in sync.
We have an RFC for several @searchable improvements, which includes upgrading to version 7: https://github.com/aws-amplify/amplify-cli/issues/7546


[skip ci] Add comments explaning different locations for specifying mappings

Fixes https://github.com/lotus/model/issues/158

@tomkadwill Thank you. Merging :+1:


Repacing optimalizer from 'Adam' to more powerful 'Adadelta'

Now Adadelta is better optimalizer, so maybe it should be improved like this:
Import the Adadelta optimizer
from transformers import Adadelta
Define the optimizer
optimizer = Adadelta(model.parameters())
Train the model
for epoch in range(num_epochs):
model.train()
for batch in train_loader:
images = batch["images"]
captions = batch["captions"]
# Encode the images
encoded_images = tokenizer(image=images, return_tensors="pt")

# Generate the captions
outputs = model(**encoded_images)

# Calculate the loss
loss = outputs.loss

# Backpropagate the loss
loss.backward()

# Update the optimizer parameters
optimizer.step()

Evaluate the model
model.eval()
with torch.no_grad():
for batch in val_loader:
images = batch["images"]
captions = batch["captions"]
# Encode the images
encoded_images = tokenizer(image=images, return_tensors="pt")

# Generate the captions
outputs = model(**encoded_images)

# Calculate the CIDEr score
cider_score = calculate_cider_score(outputs.captions, captions)

# Print the CIDEr score
print(cider_score)

Thanks for your suggestion.
But Adadelta is not regarded as a better optimizer as Adam. Rather, Adam tries to improve AdaDelta.


Implement null-pointer de-referencing detection for Cortex-M

Fixes #3634

The PR implements null-pointer dereferencing (R/W) detection for Cortex-M architecture

using the DWT peripheral where available
using the MPU peripheral when DWT is not available

It is recommended (and implemented) as a debug feature only: enabling null-pointer detection will:

increase flash footprint, as it will un-map some area starting from address 0x0 (1kB by default)
reserve 1 or 2 DWT comparator(s), depending on the architecture variant, or,
consume 1 MPU region, if DWT is not used/present.

Null pointer exception detection is enforced in the test suite, to be able to catch accidental null pointer dereferencing
It is emulated in QEMU (mps2_an385).

The feature is not recommended in production code.
The feature is useful for privileged threads/ ISRs, which normally have access to all flash / sram
It's also useful for user threads, as normally reading flash area is allowed in nPRIV mode.
The feature is implemented based on the same principle as the solution for x86 (see #3634)

Missing/Future-work

documentation [release note entry is now added] of the feature
possibly a new error code specific to null-pointer exception (e.g. K_ERROR_NULL_POINTER_EXCEPTION) (CC @andrewboie)

This is very cool, thanks @ioannisg . I tried out the test on mimxrt1050_evk, but unfortunately the exception doesn't fire and the test fails. I haven't debugged why yet.

This is very cool, thanks @ioannisg . I tried out the test on mimxrt1050_evk, but unfortunately the exception doesn't fire and the test fails. I haven't debugged why yet.

Thanks @MaureenHelm ! Please share some log of the test output, so I can take a look. I could also try to debug on mimxrt1015 (that has to wait until next week, though).
One thing that might help here is whether the flashing of the board leaves the Debug mode on (then, it is true that the DWT won't be able to raise exceptions). I've seen this at least with openOCD on SAME70 platform.

One thing that might help here is whether the flashing of the board leaves the Debug mode on (then, it is true that the DWT won't be able to raise exceptions). I've seen this at least with openOCD on SAME70 platform.

That appears to have been the problem. The test passes after POR via SW4.

One thing that might help here is whether the flashing of the board leaves the Debug mode on (then, it is true that the DWT won't be able to raise exceptions). I've seen this at least with openOCD on SAME70 platform.

That appears to have been the problem. The test passes after POR via SW4.

I will make a comment in the Kconfig option, stressing that the feature requires Normal mode. @MaureenHelm thanks.

One thing that might help here is whether the flashing of the board leaves the Debug mode on (then, it is true that the DWT won't be able to raise exceptions). I've seen this at least with openOCD on SAME70 platform.

That appears to have been the problem. The test passes after POR via SW4.

I will make a comment in the Kconfig option, stressing that the feature requires Normal mode. @MaureenHelm thanks.

@hakehuang Do you see any issues with this for board farm testing?

One thing that might help here is whether the flashing of the board leaves the Debug mode on (then, it is true that the DWT won't be able to raise exceptions). I've seen this at least with openOCD on SAME70 platform.

That appears to have been the problem. The test passes after POR via SW4.

I will make a comment in the Kconfig option, stressing that the feature requires Normal mode. @MaureenHelm thanks.

@hakehuang Do you see any issues with this for board farm testing?

Thanks Maureen. I see this problem but not find the root cause befer, because I can always run pass, when I go check which looks like I push the reset key when I check.

@MaureenHelm @hakehuang I added a comment in the Kconfig definition stressing that we need the core to be in Normal mode, to use this software debug feature (the null pointer exception detection), if we're using the DWT peripheral. I also added a run-time assert for this. Let me know if you have more concerns around it.

@MaureenHelm @hakehuang I added a comment in the Kconfig definition stressing that we need the core to be in Normal mode, to use this software debug feature (the null pointer exception detection), if we're using the DWT peripheral. I also added a run-time assert for this. Let me know if you have more concerns around it.

@ioannisg , I am ok with it, thanks a lot.

@carlescufi @galak would you like to take a quick look as well?


Bugfix data casting

In Rails 5.2, and maybe other versions, cast gets called during the assignment and sometimes is used to attempt to expand a valid UUID. This is to skip casting if the value already looks like a UUID.
I am not sure what tests to add, I only noticed this issue when running my app instance in development and not during any of my normal test coverage. Which is odd because I would expect requests or controller specs to have triggered this issue.

Thank you for your pull request! I recognise the issue you bring up (noticed it during development), and I think the place where you placed the additional comparison makes sense if you read the comments there as to why we're doing that comparison. I'm glad you mention as to why cast is sometimes mentioned.
It would be nice if we have a test for this indeed. Would you mind adding one? I think you should be able to use String#encode to change the encoding and make sure your comparison is hit.
Current version:
mysql-binuuid-rails $ bin/console
irb(main):001:0> MySQLBinUUID::Type.new.cast(SecureRandom.uuid.encode(Encoding::ASCII_8BIT))
=> "33623866-3530-3633-2d64-6262332d343631652d393861662d323565643537343161353761"
irb(main):002:0>

With your fix (works as expected!)
mysql-binuuid-rails $ bin/console
irb(main):001:0> MySQLBinUUID::Type.new.cast(SecureRandom.uuid.encode(Encoding::ASCII_8BIT))
=> "6095e58b-7dd2-4fda-b471-a1bda0ebf411"
irb(main):002:0>

@markoudev PR Updated

Wonderful, thanks! I'll merge it and an going to create a new version.

Release in version 1.2.0!


deploying chatwoot helm chart on digitalocean using managed postgres and redis

[x] deploying chatwoot helm chart on digitalocean using managed postgres and redis
[x] Readme documenting digitalocean

Marking closed. No extra documentation specific to DigitalOcean is needed.


Add Describe Reserved Instance Endpoints

There is an API call to describe all of the reserved instance offerings in every region.
https://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.Client.describe_reserved_instances_offerings
This should be added to moto.

The two other related reserved instance end points are
http://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.Client.describe_reserved_instances
and https://boto3.readthedocs.io/en/latest/reference/services/ec2.html#EC2.Client.purchase_reserved_instances_offering


Add version to component

Signed-off-by: Ashish Tiwari<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@Abhishek-kumar09  Use this PR to make those 3 changes we talked about

Codecov Report

Merging #5745 (c3c5a9a) into master (db89a03) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #5745      +/-   ##
=========================================
- Coverage    6.05%   6.03%   -0.03%
=========================================
  Files         107     107
  Lines        9796    9796
=========================================
- Hits          593     591       -2
- Misses       9046    9048       +2
  Partials      157     157

Flag
Coverage Δ

gointegrationtests
6.03% <ø> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/start.go
56.13% <0.00%> (-0.95%)
:arrow_down:

Continue to review full report at Codecov.

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


Add Guild.bots and Guild.humans

Summary

The title says it all

Checklist

[x] If code changes were made then they have been tested.

[x] I have updated the documentation to reflect the changes.

[ ] This PR fixes an issue.
[x] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

Reverted commit because check failed


Bug: Designer Canvas is obscured by surrounding panels

This causes the "zoom to fit" and "center" functionality to not work correctly (even though they do!). Example symptom:

Thank you @mohdali for this wonderful fix!


feat/sql null comparisons

Currently, a predicate checking a null value like myColumn.eq(null) translates to the following SQL:
my_column = null

which is valid SQL, but doesn't actually check if my_column is null.

I found out that IS [NOT] NULL is defined as a unary operator (1) so I implemented the corresponding UnaryOperator class for them.
I added a check on the SqlWhereBuilder to translate EQ/NE comparisons with a null value to a IS_NULL/IS_NOT_NULL.
I added predicates and extensions to make the null checks too.

Hi @MiguelMJ
I like your proposal, thanks a lot!
But I don't understand why you need the code in archimedes-data-jdbc/src/main/kotlin/io/archimedesfw/data/sql/criteria/builder/SqlWhereBuilder.kt in the is BinaryPredicate<*> -> {
I mean, If you added the UnaryPredicate, I think you should use it to compare if a column is null with the isNull() instead of the eq(null).
For example the "where" test could be:
    @Test
    internal fun `build with null comparison`() {
        where.predicates.add(tBook.pagesCount.isNull())

instead of your actual code
    @Test
    internal fun `build with null comparison`() {
        where.predicates.add(tBook.pagesCount.eq(null))

So, I would remove all the code:
            is BinaryPredicate<*> -> {
//                val isNullRight = predicate.right is ParameterExpression && predicate.right.value == null
//
//                if (isNullRight && predicate.operator == BinaryOperator.EQ) {
//                    appendPredicate(sb, UnaryPredicate(predicate.left, UnaryOperator.IS_NULL), parameters)
//                } else if (isNullRight && predicate.operator == BinaryOperator.NE) {
//                    appendPredicate(sb, UnaryPredicate(predicate.left, UnaryOperator.IS_NOT_NULL), parameters)
//                } else {
                    appendPredicate(sb, predicate.left, parameters)
                    sb.append(predicate.operator.sql)
                    appendPredicate(sb, predicate.right, parameters)
//                }
            }

What do you think? Why did you add these code in the is BinaryPredicate<*> -> { section? What was your intention?

Hello!
My reasoning was that when you have a nullable type T? then a dynamic comparison myColumn.eq(myNullableValue) would need that internal conversion for the comparison to work as one would intuitively expect when translated to SQL.
In other words, it acts as a shortcut for if (myNullableValue == null) myColumn.isNull() else myColumn.eq(myNullableValue). However, I understand that this code is precisely what the user would write in these cases. If  you think so, I will make the suggested changes and push again.
Thank you for the review!
PD: Excuse the late response, I'm just back from vacation 😅

Hi @MiguelMJ
In this case, please, I prefer If you remove that code in the BinaryPredicate case.
Thanks!

Done ;)


Add parallax scrolling

Because :)

First I would need to add images for the parallax scrolling to (even remotely) make sense :D

As I'm not seeing it happening in any conceivable future, I'm closing the ticket ;)


which branch should I choose to support multi GPUs on spark cluster

hi,dear friends:
which branch should I choose to support multi GPUs on spark cluster on the github,master,multigpu,multigpucore,etc,and what diffirences are they?
thanks a lot!

I can know it.


[Feature Request] VST Plugin Support

I'm not at all familiar with the inner workings of this program or how VST plugins work but I was setting up an Elgato Wave XLR and Wave Link for someone earlier and I noticed that Wave Link has support for not only Elgato's own plugins but all VST and VST3 plugins as well and I was wondering if that's even possible to do with GoXLR Utility. Please let me know if it's possible, planned, or not in the scope of the project. Thanks!

I did a little research and found these repositories, which may or may not be helpful.
https://github.com/RustAudio/vst-rs
https://github.com/jesnor/vst3-rs

Ah I understand. So devices like the Elgato Wave XLR and Wave Link must do the audio processing in the software or something? Since I don't want to necro the thread in Discord, I'll just put my musings here but if you were to theoretically rewrite the drivers for the GoXLR to handle VSTs and look for a specific folder through the USB connection that would house the VST plugins, would that be possible? Obviously that is outside the scope of this project but I'm just theorizing and speculating.

I've not personally used the devices, but my general understanding is that the device itself is simply a 1 input, 1 output device to bring audio in from the XLR port and send audio out to the headphone jack, and the driver itself is responsible for all the mixing and processing before it arrives at WASAPI and is presented to the end user.
In theory, if there was direct control over the driver, it would theoretically be possible to do something similar with the GoXLR, but live monitoring the changes would be tricky due to latency of needing a full round trip.

Thanks for the insight. Interesting to think about. I'm by no means a driver author/developer but I wonder if one day someone would take up that project especially since Music Tribe laid off the development team early this year if I remember correctly so there would be no more (official) development on official app or the driver, unless I missed some news on the topic.

Ah okay. Thank you for the timeline summary. I did notice in GoXLR Utility that it said there was a firmware update and when I opened the official app to do it, it said there was a software update too which I thought was strange. I'm surprised Music Tribe actually got another team on the job. So, this sort of goes off topic now but what are the pros and cons to using GoXLR Utility vs the official GoXLR app if the official app is being actively developed on again? I did a skim over both recently and it seems like there is feature parity between the two.


changed ariaToolTip based on size

@pelikhan excited to start contributing!
This PR will fix https://github.com/microsoft/microcode/issues/160
The two screenshots below are after I made my changes.

Welcome to the project!


SAI04536: Coordinator node does not have correct collection tagged for TracingSpan

DO NOT MERGE
This is for internal review. We will push this upstream first and then pull it

this is committed upstream https://github.com/apache/solr/pull/1962


Improving readability and making plot formats consistent

Going through all the plots and tweaking axes, titles, colors, etc to make them readable and consistent across pages.

Looks good! Merging now


HystrixObservableCollapser does not properly handle streams per request arg

Bug reported in https://groups.google.com/forum/#!topic/hystrixoss/etlb_JE90CU.
The root cause is that, upon receiving the first piece of a batch response which matches a request arg, that Collapser has onNext(data) and onCompleted() invoked.  This happens even if more data is available for that Collapser.

Released in 1.4.15 and merged into master to appear in first 1.5.0.rc1 release


bcc: init at 0.1.8-master

This is a work in progress
Opening PR to get some feedback/help.
Status: compiles, some of the bcc tools work (haven't tested all yet) but I've had to resort to some tricks to get there.
Questions:

currently builds off unreleased git commit because of certain requirements (like https://github.com/iovisor/bcc/commit/4da282918f3d3d650cbd0d8c813389db5c72cad8). What should the nixpkg version be?

package requires a kernel built with at least:
extraConfig = ''
  BPF y
  BPF_SYSCALL y
  NET_CLS_BPF m
  NET_ACT_BPF m
  BPF_JIT y
  HAVE_BPF_JIT y
  BPF_EVENTS y
'';

I've used nixpkgs.config.packageOverrides to accomplish this like so:
nixpkgs.config.packageOverrides = pkgs: {
  linux_4_4 = pkgs.linux_4_4.override {
    extraConfig = ''
      BPF y
      BPF_SYSCALL y
      NET_CLS_BPF m
      NET_ACT_BPF m
      BPF_JIT y
      HAVE_BPF_JIT y
      BPF_EVENTS y
    '';
  };
};

However, I need to reference the path to the running kernels module directory, I've done this using ${linux.dev}  but that doesn't work as it refers to the default kernel derivation, without the necessary configuration (see comment in prePatch hook). How should I be doing this? In general, what is the correct way to make a derivation for a program like this that requires a custom kernel?

Things done:

[ ] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[X] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

You should add this in all-packages.nix inside the linuxPackagesFor function, e.g. where the NVIDIA drivers reside: https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/all-packages.nix#L10562

@dezgeg thanks, updated PR. How do I then use it in my configuration.nix?

Like pkgs.linuxPackages_4_1.bcc. Now if you change the variable linux to kernel in your expression (same way in e.g pkgs/os-specific/linux/cryptodev/default.nix) it should automatically compile against the matching kernel version/headers.

@dezgeg thanks again, I got that part of this working now. will continue testing all the programs

Awesome, looking forward to this!

@obadz @pSub @dezgeg Picked this up today again and updated to latest bcc commit. IMO it is in a good enough state to merge. Need to work more with upstream to fix some things like the fs tools (ext4dist/ex4slower/...), mdflush segfaulting but most tools work and are useful. Thoughts?

Looks good, thanks.


WriteValue - add type param

According to Bluez API (https://github.com/hadess/bluez/blob/master/doc/gatt-api.txt) you can specify the write type (eg: Write without response)
This PR aim to add this type parameter to WriteValue
Here is the allowed types :
"command": Write without response
"request": Write with response
"reliable": Reliable Write

Thank you for your PR. Can you provide a test suite for that?


test dynamiclistener

Issue:

Problem

Solution

Testing

Engineering Testing
Manual Testing

Automated Testing

Test types added/modified:

Unit
Integration (Go Framework)
Integration (v2prov Framework)
Validation (Go Framework)
Other - Explain: EXPLAIN
None
REMOVE NOT APPLICABLE BULLET POINTS ABOVE

If "None" - Reason: EXPLAIN THE REASON

If "None" - GH Issue/PR: LINK TO GH ISSUE/PR TO ADD TESTS

Summary: TODO
QA Testing Considerations

Regressions Considerations

TODO
Existing / newly added automated tests that provide evidence there are no regressions:

TODO

validated.. new dynamic listener is working.


Possible to do field-level authorization for fields of Nexus-Prisma models?

Question about GraphQL Shield
Hey 👋
I can't tell if it's possible to do field-level read-authorization for a Prisma Nexus generated model/field. For example, if I have the prisma model
type User {
  id: ID!
  firstName: String!
  address: String!
}

and I expose that using Nexus-Prisma, how would I go about making custom read rules for the firstName and address string? If, for example, everyone authorized can read the first name, but  only allow reading the address if you are the user?
Not sure if the question belongs more here or with nexus-prisma, sorry if I came to the wrong place.

[x] I have checked other questions and found none that matches mine.

Hey 👋 ,
You'd do it the same way you'd do it in a legacy Apollo Server projects. The idea is that nexus returns a schema that you can further restrict using graphql-nexus. Besides the schema generation syntax that nexus uses, everything should work the same as in the examples.


Modules

Added module-home to home css and home jade files. Add module about to about css. Could not find about jade. Does it exist?

DO NOT merge. There are some mistakes.

Okay. Holding the review till you say you're done.

I merged the homepage commits as be1cbca and removed those commits, along with the login page and your commit-revert cycles, from this PR. This leaves the events page, which I'm reviewing now.

Closing as an alternate version already merged as #281


Gravity of zoomable images

When an image is zoomed, SketchZoomImageView's current behavior is to scale on all sides which makes sense for most image viewer apps where the image is displayed at the center. Will it be possible to customize this behavior by taking in a "gravity"? I am trying to display an image at the top of the screen. If the image is zoomed, I'd like it to scale up towards the bottom edge of the image until it reaches its maximum height.

I can't imagine such a usage scenario, please describe in more detail why there is such a requirement displayed at the top

I have a screen that shows an image as its header. The image can be zoomed, but its height remains fixed.

Closing in favor of telephoto.


EDA-and-Perform-Modelling-on-Ionosphere-Dataset

Have you read the Contributing Guidelines ?
EDA-and-Perform-Modelling-on-Ionosphere-Dataset
Description
The project aims to predict class using various technical specifications (features) as input to the logistic regression algorithms.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[ ] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#166

Have you read the Contributing Guidelines ?
EDA-and-Perform-Modelling-on-Ionosphere-Dataset
Description
The project aims to predict class using various technical specifications (features) as input to the logistic regression algorithms.
Checklist

[x]  I've read the contribution guidelines.
[x]  I've checked the issue list before deciding what to submit.
[ ]  I've edited the README.md and link to my code.

Related Issues or Pull Requests
#166

You are still 2 commits behind the original repo please first fetch this repo once then send the PR again thank you @suy1968


(#152) fix block backup bug cause by we only backup one level

Codecov Report

Merging #153 into master will increase coverage by 0.06%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master     #153      +/-   ##
==========================================
+ Coverage   59.55%   59.62%   +0.06%
==========================================
  Files          11       11
  Lines        1318     1320       +2
==========================================
+ Hits          785      787       +2
  Misses        429      429
  Partials      104      104

Impacted Files
Coverage Δ

src/elz/codegen/context.go
61.53% <75%> (+6.99%)
:arrow_up:

Continue to review full report at Codecov.

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


Black screen when using screenshot()

Hi, I have Fedora Linux in my device, and I recently used pyautogui screenshot() function and when I checked the image in my home directory, the png was black entirely.
I installed Scrot for the compatibility issues from rpmsphere repository using dnf.
https://fedora.pkgs.org/33/rpm-sphere-x86_64/scrot-0.8-21.x86_64.rpm.html

Followed instructions from this website.
Also, locateOnScreen and locateCenterOnScreen doesnt seem to work, It always returns None(I even decreased the confidence level all the way to 0.2 and made grayscale as True).
The program works fine in Windows.
Thanks in advance

Hi, I have Fedora Linux in my device, and I recently used pyautogui screenshot() function and when I checked the image in my home directory, the png was black entirely.
I installed Scrot for the compatibility issues from rpmsphere repository using dnf.
https://fedora.pkgs.org/33/rpm-sphere-x86_64/scrot-0.8-21.x86_64.rpm.html

Followed instructions from this website.
Also, locateOnScreen and locateCenterOnScreen doesnt seem to work, It always returns None(I even decreased the confidence level all the way to 0.2 and made grayscale as True).
The program works fine in Windows.
Thanks in advance

I have also found out that its the fault of pyscreeze. Scrot invalid option -z

Hi, I have Fedora Linux in my device, and I recently used pyautogui screenshot() function and when I checked the image in my home directory, the png was black entirely.
I installed Scrot for the compatibility issues from rpmsphere repository using dnf.
https://fedora.pkgs.org/33/rpm-sphere-x86_64/scrot-0.8-21.x86_64.rpm.html

Followed instructions from this website.
Also, locateOnScreen and locateCenterOnScreen doesnt seem to work, It always returns None(I even decreased the confidence level all the way to 0.2 and made grayscale as True).
The program works fine in Windows.
Thanks in advance

Exactly Same Problem. I have tested it in Ubuntu 18, Ubuntu 20, Windows-10 and MacOS Big Sur and it works perfectly everywhere but Fedora.
BTW, I am using Fedora 34 5.11.12-300.fc34.x86_64

Hi, I have Fedora Linux in my device, and I recently used pyautogui screenshot() function and when I checked the image in my home directory, the png was black entirely.
I installed Scrot for the compatibility issues from rpmsphere repository using dnf.
https://fedora.pkgs.org/33/rpm-sphere-x86_64/scrot-0.8-21.x86_64.rpm.html

Followed instructions from this website.
Also, locateOnScreen and locateCenterOnScreen doesnt seem to work, It always returns None(I even decreased the confidence level all the way to 0.2 and made grayscale as True).
The program works fine in Windows.
Thanks in advance

Exactly Same Problem. I have tested it in Ubuntu 18, Ubuntu 20, Windows-10 and MacOS Big Sur and it works perfectly everywhere but Fedora.
BTW, I am using Fedora 34 5.11.12-300.fc34.x86_64

yeah I use the same Fedora, the problem is with scrot, they removed an option in scrot, which i downloaded from dnf, maybe I'll need to install scrot from somewhere else like flatpak
Thank u for replying

Using Ubuntu 22.04 inside VMWare and I'm having the same issue.


Add copy color feature

Created the copied feature #241. Here is a loom of the changes https://www.loom.com/share/e02d326fa4884ec195252a48289c493a

Fantastic. Merging.


Support for OPERATION_UPDATE

2023-10-18 15:31:20.357 ERROR           substreams-sink-clickhouse      unsupported operation found in entityChanges: OPERATION_UPDATE

Substreams Example

Testing with block-meta (https://github.com/streamingfast/substreams-eth-block-meta/releases/download/v0.5.1/substreams-eth-block-meta-v0.5.1.spkg)

graph_out https://github.com/streamingfast/substreams-eth-block-meta/blob/41d6c420567bdc9e128b4e6732d99cff25bf7ef5/src/graph_out.rs#L34-L41

Implemented


Latency is overriden when response is cached

When a test uses cached results, the latency is currently set to that of the most recent run with cached data.
Ideally, the latency should reflect the value from the original execution when the results were first generated.

Thank you for filing this @trondrensel Latency caching is going into our backlog. This is a duplicate of https://github.com/promptfoo/promptfoo/issues/945 We can track progress there!


Datasets

Hello!
I have to notice that you create great work! Code is easy to read and use!
But I have issue about Syn90k and SynthText.
Both of this datasets after loading haven't folder named train or test. Did you provide some scripts for preparing this datasets into required form? Because I downloaded them with links which you provided and it aren't in required form.
Or maybe you can share some info which can help me with preparing.

For these two datasets you need to split them into train/test folders yourself with your own splitting strategy. You can refer to line 81 in train.py to see how it is handled for different datasets.

Thank you for the answer.


Can't extract files from .7z zipfile using download_and_extract

Describe the bug
I'm adding a new dataset which is a .7z zip file in Google drive and contains 3 json files inside. I'm able to download the data files using download_and_extract but after downloading it throws this error:
>>> dataset = load_dataset("./datasets/mantis/")
Using custom data configuration default
Downloading and preparing dataset mantis/default to /Users/bhavitvyamalik/.cache/huggingface/datasets/mantis/default/1.1.0/611affa804ec53e2055a335cc1b8b213bb5a0b5142d919967729d5ee23c6bab4...
Downloading data: 100%|█████████████████████████████████████████████████████████| 77.2M/77.2M [00:23<00:00, 3.28MB/s]
/Users/bhavitvyamalik/.cache/huggingface/datasets/downloads/fc3d70123c9de8407587a59aa426c37819cf2bf016795d33270e8a1d558a34e6
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/bhavitvyamalik/Desktop/work/hf/datasets/src/datasets/load.py", line 1745, in load_dataset
    use_auth_token=use_auth_token,
  File "/Users/bhavitvyamalik/Desktop/work/hf/datasets/src/datasets/builder.py", line 595, in download_and_prepare
    dl_manager=dl_manager, verify_infos=verify_infos, **download_and_prepare_kwargs
  File "/Users/bhavitvyamalik/Desktop/work/hf/datasets/src/datasets/builder.py", line 690, in _download_and_prepare
    ) from None
OSError: Cannot find data file.
Original error:
[Errno 20] Not a directory: '/Users/bhavitvyamalik/.cache/huggingface/datasets/downloads/fc3d70123c9de8407587a59aa426c37819cf2bf016795d33270e8a1d558a34e6/merged_train.json'

just before generating the splits. I checked fc3d70123c9de8407587a59aa426c37819cf2bf016795d33270e8a1d558a34e6 file and it's 7z zip file (similar to downloaded Google drive file) which means it didn't get unzip. Do I need to unzip it separately and then pass the paths for train,dev,test files in SplitGenerator?
Environment info

datasets version: 1.18.4.dev0
Platform: Darwin-19.6.0-x86_64-i386-64bit
Python version: 3.7.8
PyArrow version: 5.0.0

Hi @bhavitvyamalik, thanks for reporting.
Yes, currently we do not support 7zip archive compression: I think we should.
As a workaround, you could uncompress it explicitly, like done in e.g. samsum dataset:
https://github.com/huggingface/datasets/blob/fedf891a08bfc77041d575fad6c26091bc0fce52/datasets/samsum/samsum.py#L106-L110

Sure, let me look into and check what can be done. Will keep you guys updated here!

Initially, I thought of solving this without any external dependency. Almost everywhere I saw lzma can be used for this but there is a caveat that lzma doesn’t work with 7z archives but only single files. In my case the 7z archive has multiple files so it didn't work. Is it fine to use external library here?

Hi @bhavitvyamalik, thanks for your investigation.
On Monday, I started a PR that will eventually close this issue as well: I'm linking it to this.

#4672

Let me know what you think.


WIP: ci: test on Windows using Travis CI

Purpose of this pull request?
We can also test on Windows using Travis CI.

Other, please explain:

CI improvement, one less CI provider (AppVeyor is slow).
The pull request refers to the:

generator (generator-nitro)

What changes did you make?
Update Travis CI config.

Is there anything you'd like reviewers to focus on?

Probably related to the npm version. Normally it works (we test on all 3 platforms for zurb/foundation-sites using Travis CI).
I'll check if a new triggered build might solve this or if it is an issue with npm on Windows Server 2016 and the Travis CI image.
Just keep this PR open (I'll rebase if needed).

Closing for now, due to lack of activity.

Please reopen, I probaly just forgot this one.

Closed again - still unsuccessful windows build on travis


New W10 console Ctrl-key shortcuts not supported

The new console host that ships with Windows 10 introduced ^-based (Ctrl-key-based) shortcuts for access to functionality that (mostly) operates on the entire screen buffer:

PSReadLine currently doesn't support these keyboard shortcuts (the only way to access the functionality is via the console window's system menu):
PSReadline handles ^V itself with the same functionality, so there's no problem.
^A is also handled by PSReadline itself, but only partly has the same functionality:

PSReadline selects the entire input line without the prompt string (which is actually more sensible)
However, pressing ^A again has no effect; by contrast, the console's functionality would select the entire screen buffer in that event.

Furthermore, ^M (Mark (select) arbitrary parts of the screen buffer) and ^F (Find text anywhere in the screen buffer) do not work at all (only accessible via the window's system menu) - with the default configuration, PSReadline simply prints literal ^M and ^F when these shortcut keys are pressed.
Supporting keyboard shortcuts ^M and ^F and ^A (to select the entire screen buffer on second invocation of ^A) would be great (I don't know whether technical restrictions prevent that).

This is a duplicate of #280.

For workarounds, see https://github.com/lzybkr/PSReadLine/issues/280#issuecomment-441338880


onVolumeChange event needed

Current Behavior
Player does not have volume change event listener.
Expected Behavior
Player fires event every time volume control changes of the native player ( tag).
Steps to Reproduce

Enable controls in Demo: https://cookpete.com/react-player/
Choose HLS (m3u8) source
Change volume in the native player
Custom volume range element does not change its value.

Environment

URL attempting to play: https://cookpete.com/react-player/
Browser: Safari/Chrome
Operating system: MacOS

Other Information
volumechange event has support in all browsers: https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/volumechange_event
My case: I have a custom player controls. Switching player to fullscreen, the native player controls are shown. Changing volume there and returning back from fullscreen the volume control value is wrong, because it cannot be updated.

Duplicate of https://github.com/cookpete/react-player/issues/1109

I see, but thought it is not related. I'm talking about built-in  tag functionality which is cross-browser supported. Do you really think that if some CUSTOM player does not support common functionality, the player should not support it at all?


Don't feed UUID and pvector to WrittenMessage.from_dict

Fixes #226.

little nit, probably optional - otherwise looks good to merge.


Drop Python 3.9 support

@jl-wynen Any idea why the serialization tests are failing here? They were disabled for Python 3.9, but are broken now for 3.10.

@jl-wynen Any idea why the serialization tests are failing here? They were disabled for Python 3.9, but are broken now for 3.10.

I fixed it, but I was wondering if this was expected.