text:
Modules that don't use the children issue a warning or error?

Should modules that don't use their children fail when children are passed?
I just spent a baffled several minutes before realizing that I had left out a semicolon, so my code was being passed to a module that didn't use it.

It's doable to add a bunch of assert($children==0);s in various modules.

Yeah, simple enough to do.   Just kind of tedious.

Are there any modules that don't use all their children?  Like they only use the first one or something?
I was wondering about making a module like no_children to try to streamline this process, though it's not clear how much it really helps if you have to do no_children($children).   I guess it avoids having to repeat the text of the error message.
But I noticed that there is now a warning message from regular openscad modules that don't take children, so it seems like to keep everything consistent, it would be best to use the same mesage text, namely:
assert($children==0,"Module modulename() does not support child modules")

I suppose there's no way to access the current module name so that this could be done generically, like str("Module ",$modulename,"() does not support child modules")

module no_children(count)
{
  assert(count==0, str("Module ",parent_module(1),"() does not support child modules"));
}

Call with no_children($children)

[ ] beziers.scad: bezier_polygon
[ ] beziers.scad: bezier_sweep_bezier
[ ] beziers.scad: trace_bezier
[ ] beziers.scad: bezier_polyhedron
[ ] common.scad: assert_approx
[ ] common.scad: assert_equal
[ ] debug.scad: trace_polyline
[ ] debug.scad: debug_polygon
[ ] debug.scad: debug_polyhedron
[ ] hull.scad: hull_points
[ ] involute_gears.scad: gear_tooth_profile
[ ] masks.scad: rounding_cylinder_mask
[ ] masks.scad: teardrop_corner_mask
[ ] partitions.scad: partition_mask
[ ] partitions.scad: partition_cut_mask
[ ] paths.scad: modulated_circle
[ ] regions.scad: region
[ ] screws.scad: screw_head
[ ] screws.scad: nut
[ ] shapes2d.scad: stroke
[ ] shapes2d.scad: arc
[ ] shapes.scad: nil
[ ] shapes.scad: slot
[ ] shapes.scad: heightfield
[ ] strings.scad: echofmt
[ ] structs.scad: struct_echo
[ ] torx_drive.scad: torx_drive2d
[ ] version.scad: bosl_required
[ ] vnf.scad: vnf_polyhedron
[ ] vnf.scad: vnf_validate
[ ] wiring.scad: wiring

I used grep to try to find the affected modules and have listed them above.  Some of these should probably call attachable and take children.  Others should have no_children($children) added.  No guarantee that my process for producing the list was correct or complete, but hopefully it's close.

I suppose there's no way to access the current module name so that this could be done generically, like str("Module ",$modulename,"() does not support child modules")

Well, there are the builtins parent_module(idx) and $parent_modules:

$parent_modules contains the number of modules in the instantiation stack. parent_module(i) returns the name of the module i levels above the current module in the instantiation stack. The stack is independent of where the modules are defined. It's where they're instantiated that counts. This can, for example, be used to build a BOM (Bill Of Material).

I already did it generically with no_children(), which already exists in common.scad, and indeed uses parent_module(1).  Note also the code example above.   Now we just need to add the no_children($children) invocation everywhere it's needed.

I added some of the checks and attachable() to vnf_polyhedron.  Probably hull_points should use attachable as well, but that one is complicated, because it's 2d and 3d.  Most of the other ones I'm not sure what they should do.

authors: adrianVmariano revarbat


text:
حالت تاریک

لطفا در صورت امکان حالت تاریک را به برنامه اضافه کنید.

سلام ممنونم از پیشنهادتون
این مورد به لیست پیشرفت پروژه اضافه شد
https://trello.com/b/90VwIm3M/loutos

source: gharchive/issue

authors: Monirzadeh rezaghz

repo: rezaghz/loutos

license: MIT

license_type: permissive

license_source: github-api


text:
Local shortcuts

This issue is opened in response to the thread at reddit.
Shortcuts are found in the file ~/.local/share/Steam/userdata/<your_id>/config/shortcut.vdf in a binary format. i.e.
00000000: 0073 686f 7274 6375 7473 0000 3000 0261  .shortcuts..0..a
00000010: 7070 6964 0023 4cde 9401 4170 704e 616d  ppid.#L...AppNam
00000020: 6500 4d79 2074 696d 6520 6174 2070 6f72  e.My time at por
00000030: 7469 6100 0145 7865 0022 656e 7622 0001  tia..Exe."env"..

And here is the appid (2497596451, or 0x94de4c23) that stored in hex: 7070 6964 0023 4cde 9401 4170 704e 616d
There is a binary vdf parser here with a summarized spec for reference if you would like to implement a deserializer.

Perfect. Thanks.

b7011d1 implements shortcuts files.

authors: heyrict rfvgyhn

repo: rfvgyhn/proton-usage


text:
Batch file doesn't allow URL with quotes

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.10.21.1. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.10.21.1

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Description of your issue, suggested solution and other information
youtube-dl --batch-file doesn't allow URL with quotes
Example:
test.txt
"https://www.youtube.com/watch?t=4&v=BaW_jenozKc"

youtube-dl --batch-file .\test.txt
WARNING: The url doesn't specify the protocol, trying with http
[generic] watch?t=4&v=BaW_jenozKc": Requesting header
WARNING: Could not send HEAD request to http://"https://www.youtube.com/watch?t=4&v=BaW_jenozKc": <urlopen error [Errno 11001] getaddrinfo failed>
[generic] watch?t=4&v=BaW_jenozKc": Downloading webpage
ERROR: Unable to download webpage: <urlopen error [Errno 11001] getaddrinfo failed> (caused by URLError(gaierror(11001, 'getaddrinfo failed'),))

Looks good: http://stackoverflow.com/a/1327476/3786245

license: Unlicense


text:
Unable to extract OpenGraph title

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.03.26.1. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2018.03.26.1

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones
[x ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.prosieben.de/tv/late-night-berlin-mit-klaas-heufer-umlauf/video/13-ganze-folge-late-night-berlin-3-ganze-folge']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2018.03.26.1
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.16299
[debug] exe versions: none
[debug] Proxy map: {}
[prosiebensat1] tv/late-night-berlin-mit-klaas-heufer-umlauf/video/13-ganze-folge-late-night-berlin-3-ganze-folge: Downloading webpage
ERROR: Unable to extract OpenGraph title; 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 "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmph9lcnopy\build\youtube_dl\YoutubeDL.py", line 785, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmph9lcnopy\build\youtube_dl\extractor\common.py", line 440, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmph9lcnopy\build\youtube_dl\extractor\prosiebensat1.py", line 450, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmph9lcnopy\build\youtube_dl\extractor\prosiebensat1.py", line 398, in _extract_clip
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmph9lcnopy\build\youtube_dl\extractor\common.py", line 920, in _og_search_title
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmph9lcnopy\build\youtube_dl\extractor\common.py", line 908, in _og_search_property
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmph9lcnopy\build\youtube_dl\extractor\common.py", line 808, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract OpenGraph title; 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.
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.prosieben.de/tv/late-night-berlin-mit-klaas-heufer-umlauf/video/13-ganze-folge-late-night-berlin-3-ganze-folge
Single video: https://www.prosieben.de/tv/late-night-berlin-mit-klaas-heufer-umlauf/video/12-ganze-folge-late-night-berlin-2-ganze-folge

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
Last week I was able to download the second link I gave above (with an older version of youtube-dl.exe). But today both links provided doesn't work (anymore), giving me the OpenGraph-Error.
Thanks for reading my issue!

I tried to download the given links again and it worked fine. I don't know why and I don't know if I should close this issue now...

I think i have a similar issue (I can file a different report if its something different):
$ youtube-dl --verbose 'https://www.prosieben.de/tv/germanys-next-topmodel/episoden/138-episode-8-magic-bride'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://www.prosieben.de/tv/germanys-next-topmodel/episoden/138-episode-8-magic-bride']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.03.26.1
[debug] Python version 2.7.12 (CPython) - Linux-4.4.0-116-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 2.8.11-0ubuntu0.16.04.1, avprobe 2.8.11-0ubuntu0.16.04.1, ffmpeg 2.8.11-0ubuntu0.16.04.1, ffprobe 2.8.11-0ubuntu0.16.04.1
[debug] Proxy map: {}
[prosiebensat1] tv/germanys-next-topmodel/episoden/138-episode-8-magic-bride: Downloading webpage
[prosiebensat1] 2847174: Downloading videos JSON
ERROR: An extractor error has occurred. (caused by KeyError(u'sources',)); 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 "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 440, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/prosiebensat1.py", line 450, in _real_extract
    return self._extract_clip(url, webpage)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/prosiebensat1.py", line 399, in _extract_clip
    info = self._extract_video_info(url, clip_id)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/prosiebensat1.py", line 35, in _extract_video_info
    source_ids = [compat_str(source['id']) for source in video['sources']]
KeyError: u'sources'
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 785, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 453, in extract
    raise ExtractorError('An extractor error has occurred.', cause=e)
ExtractorError: An extractor error has occurred. (caused by KeyError(u'sources',)); 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.


text:
exception in youtube-dl when parsing youtube video

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.09.10. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.09.10

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-o', 'C:\\Users\\ruinvtyu\\AppData\\Local\\WebDownloader\\json\\%(title)s', '--no-playlist', '--verbose', '--skip-download', '--restrict-filenames', '--write-info-json', 'https://www.youtube.com/watch?v=ENAwnVMwxhM']
[debug] Encodings: locale cp1251, fs mbcs, out cp1251, pref cp1251
[debug] youtube-dl version 2018.09.01
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.10240
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] ENAwnVMwxhM: Downloading webpage
[youtube] ENAwnVMwxhM: Downloading video info webpage
WARNING: unable to extract html5 player; 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.
[youtube] {22} signature length 41.42, html5 player None
[youtube] ENAwnVMwxhM: Downloading player https://www.youtube.com/yts/jsbin/player_ias-vflIVQ4xT/en_US/base.js
ERROR: Signature extraction failed: Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1210, in _decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1121, in _extract_signature_function
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1182, in _parse_sig_js
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\common.py", line 972, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; 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.
(caused by RegexNotFoundError('Unable to extract Initial JS player signature function name; 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.',)); 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 "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1210, in _decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1121, in _extract_signature_function
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1182, in _parse_sig_js
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\common.py", line 972, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; 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 "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1210, in _decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1121, in _extract_signature_function
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1182, in _parse_sig_js
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\common.py", line 972, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; 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.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\YoutubeDL.py", line 792, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\common.py", line 502, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1803, in _real_extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1220, in _decrypt_signature
youtube_dl.utils.ExtractorError: Signature extraction failed: Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1210, in _decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1121, in _extract_signature_function
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1182, in _parse_sig_js
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\common.py", line 972, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; 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.
(caused by RegexNotFoundError('Unable to extract Initial JS player signature function name; 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.',)); 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.
Error: Signature extraction failed: Traceback (most recent call last):
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=ENAwnVMwxhM

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
youtube-dl exists with exception when parsing youtube video. Please see youtube video link and logs above.

@dstftw - have you considered new label liar or similar yet?

Why is it outdated. I have just updated it before I submitted the bug. It's the latest version.
I downloaded the latest youtube_dl.exe.
Here is the command I run in command line:
youtude_dl.exe -o "C:\Users\my_user\AppData\Local\WebDownloader\json%(title)s" --no-playlist --skip-download --restrict-filenames --write-info-json "https://www.youtube.com/watch?v=ENAwnVMwxhM"
And here is the output:
[youtube] ENAwnVMwxhM: Downloading webpage
[youtube] ENAwnVMwxhM: Downloading video info webpage
[youtube] ENAwnVMwxhM: Downloading js player vflvABTsY
ERROR: Signature extraction failed: Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1210, in
_decrypt_signature
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1121, in
_extract_signature_function
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\youtube.py", line 1182, in
_parse_sig_js
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpkzpz548c\build\youtube_dl\extractor\common.py", line 972, in _
search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract Initial JS player signature function name; 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.
(caused by RegexNotFoundError('Unable to extract Initial JS player signature function name; 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 --ver
bose flag and include its complete output.',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the lat
est version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Please assist me with this issue solving

From your log:

[debug] youtube-dl version 2018.09.01

From issue template:

[x] I've verified and I assure that I'm running youtube-dl 2018.09.10

authors: dstftw pzygielo vtyutin


text:
Export HTML5  tags?

When exporting a video, is it possible to auto-generate an HTML5 <video> snippet which includes the thumbnail, codecs, etc?
I was thinking something like
<video poster="thumbnail.jpg" >
  <source src="file.mp4" type="video/mp4; codecs=mp4a.40.2, avc1.42001E">
  <source src="file.webm" type="video/webm; codecs=vorbis, vp8.0">
  <track kind="subtitles" src="file.srt">
</video>

I thinkt that's out of scope of youtube-dl, use the --write-info-json or --dump-json and write a script that process the json dict and creates the snippet.

authors: edent jaimeMF


text:
[Tubepatrol] Add new extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x ] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x ] Searched the bugtracker for similar pull requests

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x ] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x ] New extractor
[ ] New feature

Description of your pull request and other information
Added a new extractor for yet another porn site [Tubepatrol.sex]
This site largely serves as a mashup of video content from other providers - Pornhub, xHamster etc.
The basic extractor function has been provided for the Video ID, Display ID, Title and URL parameters only.
Cheers and thank you!
Parmjit V.

Both test cases are borfos.com generic embeds thus:

This extractor should be removed.
Extractor for https://borfos.com/embed/\d+ should be added.
Generic extractor should be modified to match such embeds.

Thanks for the prompt review.  Can you please elaborate?
The [borfos.com] embed URLs do not provide the required extractor metadata, namely the video title text and display ID.
As it is, extracting from [tubepatrol.sex] gives us access to this required metadata, and will allow us to also composite some optional fields in a subsequent commit - specifically 'categories' and 'view_count'.  The current implementation also provides for a fallback video URL.
Cheers,
Parmjit

As already said generic embed must be processed by generic extractor. As per additional metadata it can be added using url_transparent extraction entity result, but I wouldn't bother with that since it's not feasible to write extractors for all sites embedding this kind of videos.

OK, thank you for the valuable feedback.  Not too bad for a first pull request :)
Seeing as this new extractor appears to be redundant, shall we close this pull request?  I can update the generic extractor as you request and submit a new PR for review at a later date.
Yours,
Parmjit

source: gharchive/pull-request

authors: dstftw parmjitv


text:
Add Hooktube to supported sites

https://github.com/rg3/youtube-dl/commit/8b561bfc9d15bf487be0e0bb5bb1d7248fd321d3

This file is autogenerated.

authors: HeatedTea dstftw


text:
[20min.ch] Add new extractor (closes #5977)

[20min.ch] Add new extractor (closes #5977)

This most common pattern is not recognized: http://www.20min.ch/videotv/?vid=425123&cid=2.

@dstftw Works now

Cherry-picked, thanks.
Do you want to be credited as one of the authors of youtube-dl? If so under what name?

Thank you.

Do you want to be credited as one of the authors of youtube-dl? If so under what name?
No, thanks.

authors: dstftw pingtux


text:
Dominant Color Sync Zones

I think, for the Dominant Color effect to really show its potential, the user should be able to assign zones of the display to be sampled and be able to use different RGB devices for each of them.
For simplicity's sake, those zones could be predetermined, and I think the best way to go about it would be "top", "bottom", "left", "right" and "center".
This would create a more effective color reproduction and would enable a "color bleeding" kind of thing.
Heres roughly the ratios that I think would yield the best results:

Great idea - and is also planned! And thanks for reminding me, now I've put this on the projects list. :)

authors: augustepierre deankevorkian

repo: rgb-master-team/RGBMaster


text:
🛑 Sito Web is down

In d47cbe2, Sito Web (https://www.comune.preganziol.tv.it) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Sito Web is back up in a39aa13 after 5 minutes.

repo: rglauco/upptime


text:
License of this repository

Hi!
The tutorial code is pretty straightforward, and was a great start on this pretty hard topic. (Mostly the testing part)
Although, I noticed that there is no explicit license on this repository, and I'm curious as to which license is this repository, in case I would like to use it.
Thank you for your work! 👍
@rglsk

Hey @D0rs4n, thanks for reporting, I will add it soon!


text:
🛑 gstephens is down

In c921aa4, gstephens (https://gstephens.org) was down:

HTTP code: 0
Response time: 0 ms

Resolved: gstephens is back up in 0b46576.

repo: rgstephens/upptime


text:
🛑 RISbot REST endpoint is down

In 9ae8d98, RISbot REST endpoint (https://risbot.rasabot.us/webhooks/rest) was down:

HTTP code: 0
Response time: 0 ms

Resolved: RISbot REST endpoint is back up in bcfbe5b.


text:
Add mandatory root_disk

Description
Adds mandatory root_disk configuration to the examples, documentation and preflight verification

LGTM

Merging as the part for the disk validation is done at the beginning of the pipe

authors: iranzo jparrill


text:
deleted TFOD file

deleted TFOD file
(non-functional, accidental commit to master)

Yeah please delete pretty please


text:
Fix typo in docs

ChangLightState example -> ChangeLightState example

Thanks!

repo: rhasspy/rhasspy


text:
Mentions not serializing when posting from a Bot

This issue looks similar to #30 and #31. Posting from a non-bot works fine and calling as_dict() works fine as well. I think the changes made in 0.7.1 just need to be replicated in the Bots class although I haven't tested it.

This is no longer an issue in the new v0.8.0 available on PyPI.

license_source: bigquery


text:
MonitoringStack errors on quota for thanos-sidecar pod

Hello,
We have an issue with Cluster Observability Operator, in particular when creating MonitoringStack.
For example when using this yaml:
apiVersion: monitoring.rhobs/v1alpha1
kind: MonitoringStack
metadata:
  labels:
    mso: test-prometheus
  name: test-monitoring-stack
  namespace: test-monitoring
spec:
  alertmanagerConfig:
    disabled: true
  namespaceSelector:
    matchExpressions:
      - values:
          - test-monitoring
        key: kubernetes.io/metadata.name
        operator: In
  logLevel: debug
  resourceSelector:
    matchLabels:
      app: test-metrics
  retention: 1d

The Monitoring Stack is created and in Resources tab we can see:
"test-monitoring-stack-prometheus"
"test-monitoring-stack-prometheus-additional-scrape-configs"
"test-monitoring-stack-thanos-sidecar"

The problem is, that prometheus pods cannot start, because we get the following error:
"create Pod prometheus-test-monitoring-stack-0 in StatefulSet prometheus-test-monitoring-stack failed error: pods "prometheus-test-monitoring-stack-0" is forbidden: failed quota: test-monitoring-quota-compute-resources: must specify limits.cpu for: thanos-sidecar; limits.memory for: thanos-sidecar; requests.cpu for: thanos-sidecar; requests.memory for: thanos-sidecar"
We cannot find any documentation/instructions/guide on how to setup the limits for thanos-sidecar, because changing "resources" in the monitoringstack yaml only changes the limits for PROMETHEUS pods, but not thanos-sidecar.
Additionally, in our case we don't even need thanos-sidecar and would like to disable it completely like we can disable "alertmanager".
Thank you!

You should be able to patch the Prometheus resource controlled by the MonitoringStack resource following this page: https://github.com/rhobs/observability-operator/blob/main/docs/user-guides/server-side-apply.md

Additionally, in our case we don't even need thanos-sidecar and would like to disable it completely like we can disable "alertmanager".

The reason why COO automatically deploys a Thanos sidecar is to support integration with the ThanosQuerier.

authors: grigassukys simonpasquier

repo: rhobs/observability-operator


text:
COO-356: remove link to rhobs-handbook.netlify.app

The page isn't really up-to-date. Also removed sthaha from the list of maintainers.

/cc @danielmellado

This seems to have a trivial conflict in the csv version, @simonpasquier mind rebasing this? I'll get it in afterwards ;)

rebased

rebased again

/lgtm

authors: danielmellado rexagod simonpasquier


text:
🛑 BOS2 is down

In d4df622, BOS2 ($URL_BOS2) was down:

HTTP code: 0
Response time: 0 ms

Resolved: BOS2 is back up in 9f784ec after 31 minutes.

repo: rholak/upptime


text:
🛑 BOS2 is down

In b4c40f1, BOS2 ($URL_BOS2) was down:

HTTP code: 0
Response time: 0 ms

Resolved: BOS2 is back up in 8648fe8 after 1 hour, 36 minutes.


text:
🛑 BOS2 is down

In 5cf8029, BOS2 ($URL_BOS2) was down:

HTTP code: 0
Response time: 0 ms

Resolved: BOS2 is back up in 9e72755 after 12 minutes.


text:
Issues with Multi-GPU Inference

Hi,
When I try to run multi-GPU inference, it errors out here : https://github.com/nightsnack/Allegro-VideoSys/blob/master/videosys/models/transformers/allegro_transformer_3d.py#L66C35-L66C40 ; is there a known issue?

Oops, it's a bug. I forget to rename it. Let me fix it.

fixed. Simply remove this line solves the problem.

thanks! multi-gpu works well now. Is there some working following up which reduces the latency? takes about ~5 mins on 8x A100 for 1 video.

Yes. I can provide some ideas.

Use cudnn backend as attention backend seems faster than flash-attn.

with sdpa_kernel(SDPBackend.CUDNN_ATTENTION):
                hidden_states = F.scaled_dot_product_attention(
                    query, key, value, dropout_p=0.0, is_causal=False
                )

In current code, I only include flash-attn and xformers as attention backend in attention processor.

Torch.compile
If you generate several videos at a time, spending some time to compile the model can further speed up.

repo: rhymes-ai/Allegro


text:
Report bug and suggest feture

both lead to google form, but cant fill out form

Hello!
I'm not quite sure what you mean.
If the google form page is not letting you type anything, I'm not sure there's much that can be done.
Otherwise, if the google form page is telling you that you don't have access then google form, then it probably thinks you aren't logged into your Harker account (since I restricted it the form to only be open to Harker students).

K
Thx
On Sat, Nov 11, 2017 at 5:20 PM, rianadon notifications@github.com wrote:

Hello!
I'm not quite sure what you mean.
If the google form page is not letting you type anything, I'm not sure
there's much that can be done.
Otherwise, if the google form page is telling you that you don't have
access then google form, then it probably thinks you aren't logged into
your Harker account (since I restricted it the form to only be open to
Harker students).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rianadon/CheckPCR/issues/15#issuecomment-343706612,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYUQvmjvZp2FXP4ORSrxAbPLb-jJCiq5ks5s1kfFgaJpZM4QXd1d
.

Did that help fix the problem or is the form still not working for you?

repo: rianadon/CheckPCR

license: mit


text:
System properties

Configurable java system properties to access in the checkstyle file.

Thanks for this :grin:!
Sorry it took so long to realize someone would make a PR on this somewhat-dead project (imo the other VS Code checkstyle extension which exists now is much better), but your changes have been merged!

authors: rianadon thomdabeast

repo: rianadon/vscode-java-checkstyle


text:
Doc updates

I’ve been maintaining and restructuring docs in my fork (I can’t push to this repository and I need to be able to test how docs appear on project site after markup changes while working on visual styling and formatting). It seems like we hit the merge nightmare scenario now 😂

I’ve done a manual merge/rebase before making this PR, the docs should be in the right state in my fork, incorporating the work @opoudjis done in meantime where applicable

Yes. Clearly I need to coordinate with you better about documentation refactoring; I've just done a major exercise (https://github.com/riboseinc/metanorma-iso/issues/238) of genericising the quickstart and guidance documents, and the iso and standoc readmes. This will have downstream implications for you.

@strogonoff, should I give you write access to this repo, or should the documents under /docs move to a separate repo?

@opoudjis

I’ve just done a major exercise (#238) of genericising the quickstart and guidance documents, and the iso and standoc readmes

I’ve done a rebase based on your latest changes just before making this PR.

should I give you write access to this repo, or should the documents under /docs move to a separate repo?

I’d say either choice is OK.
It would be slightly less confusing to potential docs contributors if docs are in the same repo (in which case I’d need to have push access to it, I’ll only touch the docs).
That said Open Project sites can work if docs are in a separate repo, we can keep fresh docs in my fork, I believe I can give you push access to my fork or you can make PRs if you have corrections on docs.

Actually, I've updated asciiiso-syntax.adoc with the new content, so everything is ok.

authors: opoudjis strogonoff

repo: riboseinc/metanorma-iso


text:
Can you add support for "TestCaseSource"?

Support TestCaseSourceAttribute to run test cases with a custom source.
NUnit API Reference

https://docs.nunit.org/articles/nunit/writing-tests/attributes/testcasesource.html

Sample
using NUnit.Framework;
using System.Collections.Generic;

public class TestsCaseSource
{
    public static int[] CasesSource = new[] { 1, 2, 3 };
    [TestCaseSource(nameof(CasesSource))]
    public void CasesSourceTest(int i)
    {
        Assert.True(i > 0);
    }

    static IEnumerable<int> CasesSourceMethod()
    {
        yield return 1;
        yield return 2;
        yield return 3;
    }
    [TestCaseSource(nameof(CasesSourceMethod))]
    public void CasesSourceMethodTest(int i)
    {
        Assert.True(i > 0);
    }

    static IEnumerable<int> CasesSourceMethodWithParameters(int start, int count)
    {
        for (int i = 0; i < count; i++)
        {
            yield return start + i;
        }
    }

    [TestCaseSource(nameof(CasesSourceMethodWithParameters), new object[] { 1, 4 })]
    public void CasesSourceMethodWithParametersTest(int i)
    {
        Assert.True(i > 0);
    }

    [TestCaseSource(typeof(AnotherClass), nameof(AnotherClass.CasesSource))]
    public void CasesSourceAnotherClassTest(int i, int j, int k)
    {
        Assert.True(i > 0);
        Assert.True(j > 0);
        Assert.True(k > 0);
    }
    public class AnotherClass
    {
        public static object[] CasesSource =
        {
            new object[] { 1, 2, 3 },
            new object[] { 2, 3, 4 },
            new object[] { 3, 4, 5 }
        };
    }
}

Edited by @ricaun

What TestCaseSource means?
Please add some description with a sample.

Of course, this would be a sample test:
`        public static IEnumerable Names = new string[] { "Foo1", "Foo2", "Foo3" };
    [TestCaseSource(nameof(Names))]
    public void Foo(string name)
    {
        Console.WriteLine(name);
        Assert.True(true);
    }`

https://docs.nunit.org/articles/nunit/writing-tests/attributes/testcasesource.html

Interesting, is basically a compact TestCase with multiple source.
Never used this TestCaseSource and looks great, lets add support for that.

That would be really helpful. We run a test for all our Revit families, this takes about an hour. During this time I do not see any console output, as this is intercepted by NUnit and then output in the test result. In addition, the test is currently aborted after 10 minutes, I have not yet found a reason for this. With TestCaseSource I can run a separate test for each family.

We need to support Revit families for multiple versions of Revit. We open the families and then save them again after they have been updated by Revit. This allows the end user to use the family without the update process. Parameters are also filled in. In future, the families may be translated into several languages.
I also find it useful that the console output is summarized, especially when tests are executed in parallel, because this is the only way to assign which console output refers to which test. Unfortunately, in NUnit 3.13.3 there is no way to avoid this.

We have all families and test cases in one repository. All families are in one folder and are all updated together in one test. However, as soon as TestCaseSource is implemented, I will use one TestCase per family.

We have all families and test cases in one repository. All families are in one folder and are all updated together in one test. However, as soon as TestCaseSource is implemented, I will use one TestCase per family.

Could you explain how you gonna do that using TestCaseSource?
I suppose TestCaseSource gonna select each file name in a folder on the fly.
If you could share a sample would be really useful.

        public static string[] GetFileNames()
        {
            return Directory.GetFiles(@"C:\Users\fabian.oswald\Downloads");
        }

        [TestCaseSource(nameof(GetFileNames))]
        public void TestFile(string fileName)
        {
            Console.WriteLine(fileName);
            Assert.True(true);
        }

@FabianOswald-WolfSystem I prerelease a version to support TestCaseSource.

https://www.nuget.org/packages/ricaun.RevitTest.TestAdapter/1.4.2-alpha

Should support the basic stuff, here is a sample. If you could give a quick test and feedback I appreciated.
using NUnit.Framework;
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;

namespace RevitTest.Sample
{
    public class Tests_Files
    {
        public static IEnumerable<string> GetFileNames()
        {
            var folder = Directory.GetCurrentDirectory();
            return Directory.GetFiles(folder, "*RevitTest*.dll").Select(Path.GetFileName);
        }

        [TestCaseSource(nameof(GetFileNames))]
        public void TestFile(string fileName)
        {
            Console.WriteLine(fileName);
            Assert.True(true);
        }

        public static string[] GetRevitFileNames()
        {
            var downloadFolder = Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.UserProfile), "Downloads");
            return Directory.GetFiles(downloadFolder, "*.rvt");
        }

        [TestCaseSource(nameof(GetRevitFileNames))]
        public void TestRevitFile(string fileName)
        {
            Console.WriteLine(fileName);
            Assert.True(true);
        }
    }
}

Thank you very much for the quick implementation. I have tested it and it works perfectly.

However, I have noticed one thing. A maximum of 370 tests are executed, although several are stored.

Now the version 1.5.0 have support for the TestCaseSource

https://www.nuget.org/packages/ricaun.RevitTest.TestAdapter/1.5.0

authors: FabianOswald-WolfSystem ricaun

repo: ricaun-io/RevitTest


text:
chore(core): Remove unnecessary slice.

Since it's already calling map, there doesn't seem to be any purpose to call slice to make a shallow clone.

oh you're absolute right. sorry i totally missed this PR

authors: evantrimboli ricokahler

repo: ricokahler/pool


text:
Custom Header in MasterPage

Hi there, regarding MasterDetail page, can we set a custom header in the MasterPage something like below:

Found the answer at comment: https://github.com/rid00z/FreshMvvm/issues/37#issuecomment-194536483


text:
offset not working

I'm on nuxt and want to set an offset because of my top navbar.
However when I set it like this in nuxt.config.js nothing happens:
['vue-scrollto/nuxt', { offset: -50 }]
Is this a bug or am I doing something wrong?

Offset itself should work, and I assume the nuxt config part should also work. I haven't used Nuxt, and the module was a community effort, so I don't know much about it to pinpoint the issue. Can you try a different offset (200 for example) to see if it actually does something...

I've tried many different offsets (positive and negative, high and low), but it didn't seem to change anything.

Could you try setting the offset from the directive?
<button v-scroll-to="{ el: '#target', offset: -50 }">
    ...
</button>

That should definitely work...

Could you try setting the offset from the directive?
<button v-scroll-to="{ el: '#target', offset: -50 }">
    ...
</button>

That should definitely work...

It does work as a directive on the button, however it doesn't appear to work when set as a default option.
I'm also using Nuxt with vue-scrollto as a plugin.

For some reason it seems other changes influenced the scroll behaviour, the global offset is now working.

authors: kstadler rigor789 sixjapick


text:
Show navigation sidebar on the error page

This PR moves the error page from the root layout into the (application) layout group. This lets the user navigate away from the error page by using the asset navigation sidebar.

integrating the same change in #974, which needs this to land! Thanks for filing

repo: rilldata/rill-developer


text:
ngoi3 〔ŋɔːi˧〕→愛 撳唔出

「愛」呢個字有兩個發音
ngoi3 同 oi3
oi3→愛　可以撳出
但係ngoi3 →愛 就唔可以
請更正，多謝！

我哋嘅 README 上面寫咗「唔支持任何懶音同模糊音」，「愛」字本身就凈係得 oi3 一個發音。如果你想打 ngoi3 都出「愛」，可以按照README上面嘅步驟添加模糊音支援。

repo: rime/rime-cantonese


text:
Add build/coverage badge

Show build/coverage badge in the README, and coverage should be above 90% to deliver more confidence, especially for those who writing server apps.

The coverage part is tracked by #2 .

FYI coverage is now at ~91%. I will add badges.

authors: kirill-konshin zengfenfei

repo: ringcentral/ringcentral-js


text:
some bug fixes

about trajectories, attractor() and more

Awesome work Xuan. Keep them coming!

authors: rionbr xuan-w

repo: rionbr/CANA


text:
Can't run in PhantomJS

I use Ripple in combination with PhanomJS and Capybara to write automated tests for our Phonegap app.  This worked great up through Ripple version 0.9.28.  0.9.29 introduced a new technique
for overriding native browser properties that doesn't work on PhantomJS.  At
https://github.com/ripple-emulator/ripple/blob/0.9.29/lib/client/emulatorBridge.js#L81
it uses Object.defineProperty (before it just used =).  One of the properties that is overridden is 'name'.  This works in Chrome, but not in PhantomJS.  (There is an open issue for PhantomJS here https://github.com/ariya/phantomjs/issues/13895, but no progress has been made on it in 6 months).
Is there any way Ripple could use a different property name other than name, or possibly go back to using = for name?  Then I could start using PhantomJS again for tests.

Unfortunately we found that to consistently patch the properties Ripple needs to patch, it needs to the Object.defineProperty (various properties fail otherwise).
I'm not sure in what context name is getting set (in the situation you're looking at, what object is it a property of?), but pretty much all the properties Ripple sets are replacing existing properties or defining properties that plugins expect and so can't be renamed.

authors: CyborgMaster TimBarham

repo: ripple-emulator/ripple


text:
why rippled sync node lost last synced ledger?

My ripple sync node completed ledger is 46435033-46455809, and then I restart rippled service, the completed ledger is 46455839-46455876. Where are these ledger? How can I restore the lost ledger while I restart rippled service at next time?

They will re-appear in complete_ledgers once you have fetched enough history to cover the gap from 46455839 to 46455809.

Is the gap amount limited? Such as no more than ledger_history value?

https://github.com/ripple/rippled/blob/fa57859477441b60914e6239382c6fba286a0c26/cfg/rippled-example.cfg#L599-L619

thank you!

authors: MarkusTeufelberger qiluge

repo: ripple/rippled

license: ISC


text:
add known_failure annotation

@ptnapoleon

+1

authors: c-kodman ptnapoleon

repo: riptano/cassandra-dtest


text:
Can riscv-ctg support vector ISA?

Hello,
I didn't see vector ISA in sample_cgfs. What should I do if I want to test vector ISA in this test architecture?
Looking forward to answering, thanks！

The basic infrastructure is certainly equipped to support generation of tests for the vector ISA but active development is needed to add functionality specific to vector and enable support. There isn't any active support for the Vector ISA at the moment.

Thanks, I will take a look at the infrastructure to support the floating point and the P extensions first.

authors: pawks zhangguog

repo: riscv-software-src/riscv-ctg


text:
section 4.3 Changes to xie/xip CSRs: specify clearly that writes in clic mode to xie/xip are blocked

Hi,
In section 4.3: Changes to xie/xip CSRs, it states:
The xie CSR appears hardwired to zero in CLIC mode, replaced by separate memory-mapped interrupt enables (clicintie[i]).
The xip CSR appears hardwired to zero in CLIC mode, replaced by separate memory-mapped interrupt pendings (clicintip[i]).
I assume this to mean that csr reads of xie/xip return zero.  I also assume this to mean that csr writes to csr registers xie/xip in CLIC mode do not change them.   Is this correct?  If so, can we state that more clearly what the behavior on writes is?  e.g., the "underlying state of the xie/xip csr registers is not writeable in clic mode" or something like that.

Writes will be ignored and will not trap (no access exceptions). xie/xip always appear to be zero in CLIC mode.

authors: Kevin-Andes dansmathers

repo: riscv/riscv-fast-interrupt


text:
Ambiguous sentence in 4.1.10 of supervisor ISA

The first sentence of the second paragraph of the supervisor ISA states:

The Interrupt bit in the scause register is set if the contains a code identifying the last exception.

At a minimum a word or a phrase is missing after the 'if' and the sentence looks unfinished. Secondly I think the text could be made more explicit to state that the interrupt bit is used to distinguish between an exception and an interrupt as per the following comment in entry.S in the linux port:
/*
     * MSB of cause differentiates between
     * interrupts and exceptions
     */
    bge s4, zero, 1f

s4 contains the scause register.

Thanks; this error was fixed last year in commit b43657ca5a4266c39eb99b186a5af5449e913eb1

authors: aswaterman lorddoskias

repo: riscv/riscv-isa-manual


text:
BITREV[I] partially overlaps REV (GREVI) in Zbp

BITREV[I] is equivalent to REV followed by a shift.  This operation is useful for bit-reversed addressing, but the question is whether the savings is important enough to justify the additional, more complex instruction.  The answer may well be "yes", but I thought it was worth raising the issue.

After some performance study on the use of extra shift for bit reverse operations, we think that the extra shift operation will not impact performance too much. So I think it is ok to remove BITREV[I] and add REV from bitmanip extension.

Release v0.9.7 (0560ad6) adds rev to replace bitrev/bitrevi.

authors: aswaterman chuanhua

repo: riscv/riscv-p-spec


text:
Use a3 for widening multiply in ch6.4

This change makes the assembly code more clear and consistent, as x10 (a0) is already used to hold the total number of elements to process, while a3 is an argument register in the function calling convention that is suitable for the multiplier. This way, the programmer can easily understand the source and purpose of the multiplier for the widening multiplication.

I agree x10 needs to be replaced with another register, but a3 is already used to hold the current vector length in this loop. You'd need to change all existing uses of a3 to e.g. t0 for this to work.

Addressed. Thanks!

repo: riscv/riscv-v-spec


text:
Import operation ceases on the UI once the browser tab is closed

When the user initiates an import by going to Import > Start Import the import endpoint is hit, and the actual import job queue starts processing the import jobs.
However:

If the user closes the browser tab and re-opens it, the import state is reset
If the user quits the browser and re-opens it, navigating back to ThreeTwo's import page, the import state is reset

Both are poor UX patterns, misleading the user into thinking that the import operation stalled or errored out.

This PR: https://github.com/rishighan/threetwo-core-service/pull/6 is meant to address the queue automation somewhat

https://github.com/rishighan/threetwo/pull/87 as well.

@nstephenh Pending your verification, I can close this issue.

Closing it for now, please re-open if this is an issue still... @nstephenh

repo: rishighan/threetwo


text:
bug: compactor generate a big sstable file with a lot of delete-range-tombstone

Describe the bug
In some case, if watermark state does not insert any new data, it would still generate delete-range for every watermark. And a large number of range-tombstone all locate in one big sstable file because compactor-node does not cut them by size.
It would cause another problem that is the meta of this sstable file is also very large because all range-tombstones store in meta-block instead of data-block. So compute-node will evict this large file again and again because there is not enough meta-cache for it.
Error message/log
No response
To Reproduce
No response
Expected behavior
No response
How did you deploy RisingWave?
No response
The version of RisingWave
No response
Additional context
No response

After discussing this with wallace, we think we need to customize a tombstone selector, in which case we can push the sst to the last_level as far as possible and recycle it
I'll be tracking this requirement

repo: risingwavelabs/risingwave


text:
refactor: file cache receives Buf and BufMut api

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
File cache receives Buf and BufMut api to support non-continuous buffer in future.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Trying another way to solve alloaction tail latency, close.


text:
refactor(metrics): remove actor_id label from back-pressure metrics based on metrics level

Signed-off-by: Bugen Zhao i@bugenzhao.comI hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
See background in #18108. Resolve https://github.com/risingwavelabs/risingwave/issues/18062.
Following the idea described in https://github.com/risingwavelabs/risingwave/issues/18108#issuecomment-2303836241, i.e., split avg into sum and count, this PR reduces the cardinality of the back-pressure metrics stream_actor_output_buffer_blocking_duration_ns by masking the actor_id label if the metrics level is less verbose than Debug. This is achieved by utilizing RelabeledMetricVec.
Tested locally and the results are the same.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Can we verify that the amount of metrics ingested by prometheus is in fact less than before, after this PR?

Can we verify that the amount of metrics ingested by prometheus is in fact less than before, after this PR?

Sure. This is what the metrics look like when metrics_level is set to debug:
# HELP stream_actor_output_buffer_blocking_duration_ns Total blocking duration (ns) of output buffer
# TYPE stream_actor_output_buffer_blocking_duration_ns counter
stream_actor_output_buffer_blocking_duration_ns{actor_id="10",downstream_fragment_id="1",fragment_id="2"} 192042
stream_actor_output_buffer_blocking_duration_ns{actor_id="11",downstream_fragment_id="1",fragment_id="2"} 2252220657
stream_actor_output_buffer_blocking_duration_ns{actor_id="12",downstream_fragment_id="1",fragment_id="2"} 1167584
stream_actor_output_buffer_blocking_duration_ns{actor_id="17",downstream_fragment_id="2",fragment_id="3"} 5269707750
stream_actor_output_buffer_blocking_duration_ns{actor_id="18",downstream_fragment_id="2",fragment_id="3"} 5389916
stream_actor_output_buffer_blocking_duration_ns{actor_id="19",downstream_fragment_id="2",fragment_id="3"} 5196666
stream_actor_output_buffer_blocking_duration_ns{actor_id="20",downstream_fragment_id="2",fragment_id="3"} 4986791
stream_actor_output_buffer_blocking_duration_ns{actor_id="25",downstream_fragment_id="2",fragment_id="5"} 4126665207
stream_actor_output_buffer_blocking_duration_ns{actor_id="26",downstream_fragment_id="2",fragment_id="5"} 4813958
stream_actor_output_buffer_blocking_duration_ns{actor_id="27",downstream_fragment_id="2",fragment_id="5"} 4993415
stream_actor_output_buffer_blocking_duration_ns{actor_id="28",downstream_fragment_id="2",fragment_id="5"} 5631874
stream_actor_output_buffer_blocking_duration_ns{actor_id="33",downstream_fragment_id="3",fragment_id="4"} 968528994
stream_actor_output_buffer_blocking_duration_ns{actor_id="33",downstream_fragment_id="5",fragment_id="4"} 2211583543
stream_actor_output_buffer_blocking_duration_ns{actor_id="34",downstream_fragment_id="3",fragment_id="4"} 516835
stream_actor_output_buffer_blocking_duration_ns{actor_id="34",downstream_fragment_id="5",fragment_id="4"} 2560459
stream_actor_output_buffer_blocking_duration_ns{actor_id="35",downstream_fragment_id="3",fragment_id="4"} 667207
stream_actor_output_buffer_blocking_duration_ns{actor_id="35",downstream_fragment_id="5",fragment_id="4"} 602543
stream_actor_output_buffer_blocking_duration_ns{actor_id="36",downstream_fragment_id="3",fragment_id="4"} 553044
stream_actor_output_buffer_blocking_duration_ns{actor_id="36",downstream_fragment_id="5",fragment_id="4"} 2605332
stream_actor_output_buffer_blocking_duration_ns{actor_id="9",downstream_fragment_id="1",fragment_id="2"} 2575042

While the following is under info level, which is the default in production.
# HELP stream_actor_output_buffer_blocking_duration_ns Total blocking duration (ns) of output buffer
# TYPE stream_actor_output_buffer_blocking_duration_ns counter
stream_actor_output_buffer_blocking_duration_ns{actor_id="",downstream_fragment_id="1",fragment_id="2"} 281855115997
stream_actor_output_buffer_blocking_duration_ns{actor_id="",downstream_fragment_id="2",fragment_id="3"} 983293
stream_actor_output_buffer_blocking_duration_ns{actor_id="",downstream_fragment_id="2",fragment_id="5"} 649668
stream_actor_output_buffer_blocking_duration_ns{actor_id="",downstream_fragment_id="3",fragment_id="4"} 321543
stream_actor_output_buffer_blocking_duration_ns{actor_id="",downstream_fragment_id="5",fragment_id="4"} 3485792

authors: BugenZhao kwannoel


text:
fix(compaction): disable fast compact for split sst

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
related to https://github.com/risingwavelabs/risingwave/issues/18323
By design, fast compaction only handles single table tasks, so I've rejected it for split sst cases in this pr as well.
In this context, we no longer need to deal with sst boundaries, consisting of：

table_ids
key_range

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

https://github.com/risingwavelabs/risingwave/issues/15973


text:
feat(batch): support a new schedule way for distributed lookup join

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Extend UpstreamHashShard to accept a Option<TableId> which represent the exact data distribution of a Table/MV. (Maybe we need to discuss further which way is better to express this distribution)
Schedule all the LookupJoin to its inner side corresponding CN.
Optimize RowSeqScanExecutor for multi point get pattern.

Performance Test:
In distributed,  This query select count(pad) from sbtest1 where k between 1 and 100000; run 100% faster than before even in a single machine with 3 CN. It takes about 900ms after this PR while 2000ms before.
dev=> describe sbtest1;
 Name |            Type
------+----------------------------
 id   | Int32
 k    | Int32
 c    | Varchar
 pad  | Varchar
 k_1  | index(k) distributed by(k)
(5 rows)

dev=> select count(*) from sbtest1;
 ?column?
----------
   100000
(1 row)

dev=>set query_mode = distributed;
dev=> explain select count(pad) from sbtest1 where k between 1 and 100000;
                                                   QUERY PLAN
----------------------------------------------------------------------------------------------------------------
 BatchSimpleAgg { aggs: [sum0(count(sbtest1.pad))] }
 └─BatchExchange { order: [], dist: Single }
   └─BatchSimpleAgg { aggs: [count(sbtest1.pad)] }
     └─BatchLookupJoin { type: Inner, predicate: k_1.sbtest1.id IS NOT DISTINCT FROM sbtest1.id }
       └─BatchExchange { order: [], dist: UpstreamHashShard(k_1.sbtest1.id) }
         └─BatchScan { table: k_1, columns: [sbtest1.id], scan_ranges: [k >= Int32(1) AND k <= Int32(100000)] }
(6 rows)

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)
https://github.com/risingwavelabs/rfcs/pull/6
https://github.com/risingwavelabs/risingwave/issues/6089

According to our discussion, we should use a new plan node and new executor to distinguish current lookup join.


text:
Autocomplete multiSelect (tags) - backspace deletes tag even if there is text

Backspace deletes tag even if there is text.
The expected behavior is backspace should delete text before deleting the previous tag. If there is no text, the latest tag should be deleted.

Thanks for the PR. This repo is still a WIP so let me know whenever you find any issue. 😄

No worries @ritz078, it's a great project!


text:
Optional support for smallvec

It would be cool if there was a feature flag for providing rkyv support for the very popular smallvec crate ( https://crates.io/crates/smallvec ).
I saw that tinyvec is already supported, but smallvec I think is more popular and has some advantages, such as that the element type does not have to impl Default.

This seems reasonable, I'll take a look at getting this done.

Fixed by fd4e54c93a36ea0af0c032eb65cdda474cdab2bc, this will be in the next release.

That was fast. Thanks!

authors: djkoloski rklaehn

repo: rkyv/rkyv


text:
Implement *With traits via derive macros

This adds the ability to put #[rkyv(remote = Type)] on container definitions T which implements

ArchiveWith<Type> for T with the Archive macro
SerializeWith<Type, _> for T with the Serialize macro
DeserializeWith<Archived<T>, Type, _> for T with the Deserialize macro

The #[with(...)] attribute on fields is extended to accept a comma-separated list of either Type (just like before) or remote(...) which configures the *With implementations.
The remote(...) item accepts a comma-separated list of

Type which needs to be specified if the remote field's type is different
with = Type to specify a with-wrapper to convert from the remote field's type
getter = Path to specify a path to a function of signature fn(&RemoteType) -> &RemoteFieldType or fn(&RemoteType) -> RemoteFieldType which is required if the remote field is private.

The namings are inspired by serde's way of handling remote types.
Deriving Deserialize for T with #[rkyv(remote = R)] requires a From<T> for R implementation.
Checklist

[x] Add tests (/rkyv/tests/derive.rs)
[x] Parse macro attributes
[x] Implement traits
[ ] Add documentation

Am happy to comply with an requests for change and also no rush by any means 😃
Will add some documentation and examples next. Perhaps I could write a page in the book too.

While checking on implementations for partial types I was wondering how enums should be handled.
I've settled on

All variants of the remote enum must have a variant in the new type
Any variant field may be omitted
All variants keep the same type e.g. a tuple variant in the remote enum must stay a tuple variant, even if all fields are removed

enum Remote {
    A,
    B(i32),
    C { a: bool, b: [u8; 2] }
}

#[derive(Archive, Serialize, Deserialize)]
#[rkyv(remote = Remote)]
enum Example {
    A,
    B(),
    C { b: [u8; 2] }
}

// Fill in missing fields manually for the `Deserialize` impl
impl From<Example> for Remote {
    fn from(value: Example) -> Self {
        match value {
            Example::A => Remote::A,
            Example::B() => Remote::B(1234),
            Example::C { b } => Remote::C { a: false, b }
        }
    }
}

Worth noting that serde doesn't seem to be supporting partial types at all with it's remote implementation.

A current issue is that the compiler complains if the native with wrapper is the same as the remote one.
struct Remote { byte: Option<NonZeroU8> }

type OtherNiche;

#[derive(Archive)]
#[rkyv(remote = remote::Remote)]
struct RemoteRef {
    #[with(Niche, remote(with = OtherNiche))]
    pub byte: Option<NonZeroU8>,
}

This results in the errors
overflow evaluating the requirement `<Niche as ArchiveWith<Option<NonZero<u8>>>>::Archived == _`
overflow evaluating the requirement `<Niche as ArchiveWith<Option<NonZero<u8>>>>::Resolver == _`

due to the added trait bound
OtherNiche: ::rkyv::with::ArchiveWith<
    Option<NonZeroU8>,
    Archived = <Niche as ::rkyv::with::ArchiveWith<Option<NonZeroU8>>>::Archived,
    Resolver = <Niche as ::rkyv::with::ArchiveWith<Option<NonZeroU8>>>::Resolver,
>

Do you have a use case in mind for keeping with and with(remote(with)) separate? I think it may be clearer for us to not generate Archive/Serialize/Deserialize impls when the type is marked remote.

I think it may be clearer for us to not generate Archive/Serialize/Deserialize impls when the type is marked remote.

That actually sounds fine to me.
The main downside I can think of is that the 0.8 utility functions don't work well with ArchiveWith types anymore so when going doing Archived -deserialize> Remote -serialize> Bytes they're basically required to use a self-defined wrapper to use the derived implementations.
It would make macro usage a little more simpler though since it would just be #[with(Wrapper, getter = path::to::fn)] and make our macro implementations a little easier too.

Do you have a use case in mind for keeping with and with(remote(with)) separate?

Had roughly a scenario like this in mind:
struct Remote {
    field: u32,
}

struct MaybeNonZero;

impl ArchiveWith<u32> for MaybeNonZero {
    type Archived = <Niche as ArchiveWith<NonZeroU32>>;
    ...
}

#[derive(Archive)]
#[rkyv(remote = Remote)]
struct Local {
    #[with(Niche, remote(u32, with = MaybeNonZero))]
    field: Option<NonZeroU32>,
}

But yea I wouldn't mind dropping the Archive/Serialize/Deserialize impls 👍

The main downside I can think of is that the 0.8 utility functions don't work well with ArchiveWith types anymore

I agree, and I actually think the whole crate would benefit from _with variants of those utility functions. Would adding those help solve this pain point? It would be so nice to just add back the With type but I really don't want to regress on the bad error message.

I actually think the whole crate would benefit from _with variants of those utility functions.

Initially I expected the deserialize_with and serialize_with functions to do just that 😄
Their names are justified though so it's tough to also add functions for with-utility. Perhaps an rkyv::with::api module or rather rkyv::api::with? Or maybe the wrapper workaround is sufficient / it's not actually that much of a pain point so it's fine to maybe put utility functions into rkyv_util? Am unsure myself.

Continued in #544 since a rewrite was simpler than rebasing

authors: MaxOhn djkoloski


text:
🛑 Library is down

In ab8f31f, Library (http://library.cvsu.edu.ph) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Library is back up in eb1654a after 5 minutes.

repo: rlvillacarlos/cvsu-site-monitor


text:
Handle ambiguous bases

This PR closes #294. I also brings master up to date with dev.
The methodology here is that we split a Seq's internal sequence into a vector of strings that don't contain ambiguous bases - i.e., splitting the original sequence on ambiguous bases.

Okay, so I've implemented the first part of what you suggested @leoisl - i.e., split the sequence string into a vector of strings with no ambiguous bases.
However, the issue we face now (the failing test in seq_test.cpp) is that we lose the position of each minimizer on the read...

Last commit records the offsets of each non-ambiguous substrings so that we can recover the position of the minimizers on the original read. All tests passing. Could you please check? Would like to do a round of refactoring before merging though...

Alright, I've added some more tests and tested it on the sample in #294.
The example read file from #294 still fails to map anything though. But I think this is now due to the decreased cluster size when filtering hits. For example, this read
AACTCGCGCGCGCCGAGCTGACCGACATNNNNCGGCTCACCGGCGAGATCAACACCCTGGCCCAGC

There are 5 minimizers (8 total, but there are some duplicates and the sketch is stored as a set) as a result of the Ns in the middle wiping out a bunch of space that would be filled by minimizers. Our default minimum cluster size (-c) is 10, so we filter out these minmizers. The same thing happens for all of the other reads.
This is somewhat of a corner case as this sample was sequenced on an Illumina Genome Analyzer II, which is quite old. But it is still informative I guess.

Would like to do a round of refactoring before merging though...

Sure. What were you thinking of refactoring? Feel free to refactor whatever you like.

Alright, I've added some more tests and tested it on the sample in #294.
The example read file from #294 still fails to map anything though. But I think this is now due to the decreased cluster size when filtering hits. For example, this read
AACTCGCGCGCGCCGAGCTGACCGACATNNNNCGGCTCACCGGCGAGATCAACACCCTGGCCCAGC

There are 5 minimizers (8 total, but there are some duplicates and the sketch is stored as a set) as a result of the Ns in the middle wiping out a bunch of space that would be filled by minimizers. Our default minimum cluster size (-c) is 10, so we filter out these minmizers. The same thing happens for all of the other reads.
This is somewhat of a corner case as this sample was sequenced on an Illumina Genome Analyzer II, which is quite old. But it is still informative I guess.

Would like to do a round of refactoring before merging though...

Sure. What were you thinking of refactoring? Feel free to refactor whatever you like.

I've also made a binary https://github.com/rmcolq/pandora/pull/295/commits/f3662695568edc9d251363dae25e9cbfecb1f9a0 (current tip of this PR) of here: https://github.com/mbhall88/pandora/releases/tag/0.9.2-rc.1

I think it's 100% acceptable to entirely lose short reads

I think it's 100% acceptable to entirely lose short reads

I would normally agree, but the machine this was sequenced on states

Yield greater than 85% of bases higher than Q30 at 2 x 50 bp

So, all the reads are 50bp. So we'd potentially throw away the entire file when the bases that are there are very good.
I'm definitely not saying we should change the default minimum cluster size though. But it's good this is documented in case others come across it.

Personally i'm ok to lose high quality 50bp read files.

It's hard enough supporting nanopore and good/modern illumina, I wouldn't want to expend dev effort or make code more complex to handle legacy data.  I think it would even be ok to explicitly say we target reads longer than 99bp

I think we pull 07048fb. It is way longer, I would argue harder to read, and brings boost into the picture without any real benefit…

Agreed

Feel free to benchmark the two implementations though? Any differences will be quite important as we are running this over every base in the fastq.

I am sure your implementation is as fast as we can get

I've just added 3 commits. 2 refactoring, mostly variable renaming: https://github.com/rmcolq/pandora/pull/295/commits/2e5f583fa1bd0b356e907c16cce1ebbb82eb735e and https://github.com/rmcolq/pandora/pull/295/commits/a5eb55dec4ad911a992f4bff36a30686df7c8d6e. And 1 that basically errors out if we find an ambiguous base in Seq::add_letter_to_get_next_kmer() (we should not have ambiguous bases anymore here): https://github.com/rmcolq/pandora/pull/295/commits/f14cbcec0f8961b00341225ba50793bb3c4a5b9d

PR fine to be merged by me, please check if you agree with the changes. If yes, feel free to merge. Otherwise, we can discuss further.

authors: iqbal-lab leoisl mbhall88

repo: rmcolq/pandora


text:
RuntimeError: CUDA error: an illegal memory access was encountered

falcontune generate     --interactive     --model falcon-40b-instruct-4bit
--weights gptq_model-4bit--1g.safetensors     --max_new_tokens=50
--use_cache     --do_sample
--instruction "Who was the first person on the moon?"

...
RuntimeError: CUDA error: an illegal memory access was encountered
While trying to generate on a multiple GPU machine encountered the above error

So just using 1 GPU with the following, export CUDA_VISIBLE_DEVICES=1

Same here, two 3090 and falcontune crashes with the same CUDA illegal memory access error.

authors: gpravi hvico

repo: rmihaylov/falcontune


text:
bootstrap with environment flag

Is there not a flag when creating a droplet and bootstrapping to specify an environment? If not can that be something added to as a future feature?

My bad, just wasnt listed in the readme.


text:
Enhancement: Add Text explainer tab & "Summarize this" tab

Title.
Tab where the user can dump in text and have it explained to them (Technical vs non-technical content)
Tab where the user can just dump in a body of text and have it summarized.

Done.


text:
Generate transpiled loader code for older versions of Node

Src currently targets Node 5+.

Will be covered by #6.

repo: rnicholus/web-components-loader


text:
Warning: componentWillReceiveProps has been renamed

Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.

Move data fetching code or side effects to componentDidUpdate.
If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state
Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run npx react-codemod rename-unsafe-lifecycles in your project source folder.

Please update the following components: RevealBase

Seems like the library has been mostly abandoned. Not sure if we'll ever see this change.

Please use react-awesome-reveal instead of this

OR - You can use https://www.npmjs.com/package/patch-package to patch this yourself - in your own project, yay!

authors: YujingGao96 null-char rizkyramadhan

repo: rnosov/react-reveal


text:
Stream of current user location.

Hello @jothipriyadharshanr
I have flow all the steps adding SDK to flutter. and it working but when I startTracking there are not methods available to track the user's current location. (if there are I did not find them on docs.).
Can you please let me know if there are any methods or streams available for continuous user tracking?
Thanks in advance.

Hello @jothipriyadharshanr
I have flow all the steps adding SDK to flutter. and it working but when I startTracking there are not methods available to track the user's current location. (if there are I did not find them on docs.).
Can you please let me know if there are any methods or streams available for continuous user tracking?
Thanks in advance.

Hey @jigarfumakiya Thanks for your message.
We do have methods within startTracking which will tracking user location continuously.
The link which tells about the different tracking modes are here
You can also try our Example Application here to refer the implementation.
Please let me know if that helped.

Yes, I have tried your example it crashes at multiple button clicks. (Like start tracking and User Update many other)
About Tracking Yes I have added start tracking methods but where is the callback
Here is the code.
 @override
  void initState() {
    // TODO: implement initState
    super.initState();
    Permission.locationAlways.request();

    setupLocation();
  }

Future<void> setupLocation() async {
    await Roam.getUser(
      userId: '611e0dc751efb03cbddb2cc2',
      callBack: ({user}) {
        print(user.toString());
      },
    );

    Roam.getListenerStatus(callBack: ({user}) {
      print('getListenerStatus');
      print(user);
    });

    Roam.getCurrentLocation(accuracy: 20, callBack: ({location}) {
      print('getCurrentLocation');
      print(location);
    },);

    Roam.subscribeUserLocation(userId: '611e0dc751efb03cbddb2cc2');

  }

void myButtonClick(){
    Roam.startTracking(trackingMode: 'active').then((value) {
      print('Tracking Started');
    });
  }

Let me know where I can get the location every time when is changed

Kind reminder @jothipriyadharshanr
Any update on this?

authors: jigarfumakiya jothipriyadharshanr

repo: roam-ai/roam-flutter


text:
option to add extra namespace for reading and writing

we should be able to add an extra namespace (URI and prefix) on the command line that should be used when reading and writing RDF.

see v1.4.0


text:
🛑 Temasili is down

In d0d146b, Temasili (https://temasili.dgdc.unam.mx) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Temasili is back up in dc51d43.

repo: robertormzg/upptime-dgdc


text:
@daily is skipping the 1st time ie., its running from day after tomorrow instead of tomorrow

Program:

package main
import (
"fmt"
"gopkg.in/robfig/cron.v2"
"time"
)
func main()  {
c := cron.New()
schedule := "@daily"
c.AddFunc(schedule, func(){})
c.Start()
defer c.Stop()
fmt.Println(time.Now())
fmt.Println(c.Entries())
}

Output

2019-02-01 15:01:52.333788618 +0530 IST m=+0.001285534
[{1 0xc00009e180 2019-02-03 00:00:00 +0530 IST 0001-01-01 00:00:00 +0000 UTC 0x5fc410}]

Hi, I added a test for the exact case you printed, but it did not reproduce. If you can provide a reproducer, I would happily debug it. Thanks!
diff --git a/spec_test.go b/spec_test.go
index 1b8a503..d28212c 100644
--- a/spec_test.go
+++ b/spec_test.go
@@ -1,6 +1,7 @@
 package cron

 import (
+       "fmt"
        "strings"
        "testing"
        "time"
@@ -179,6 +180,9 @@ func TestNext(t *testing.T) {
                // Monthly job
                {"TZ=America/New_York 2012-11-04T00:00:00-0400", "0 0 3 3 * ?", "2012-12-03T03:00:00-0500"},

+               // Test for Issue #172, @daily in IST
+               {"TZ=Asia/Kolkata 2019-02-01T15:01:52+0530", "@daily", "2019-02-02T00:00:00+0530"},
+
                // Test the scenario of DST resulting in midnight not being a valid time.
                // https://github.com/robfig/cron/issues/157
                {"2018-10-17T05:00:00-0400", "TZ=America/Sao_Paulo 0 0 9 10 * ?", "2018-11-10T06:00:00-0500"},
@@ -191,6 +195,10 @@ func TestNext(t *testing.T) {
                        t.Error(err)
                        continue
                }
+               if strings.Contains(c.time, "Kolk") {
+                       fmt.Println(getTime(c.time))
+                       fmt.Println(sched.Next(getTime(c.time)))
+               }
                actual := sched.Next(getTime(c.time))
                expected := getTime(c.expected)
                if !actual.Equal(expected) {

$ go test -run=TestNext
2019-02-01 15:01:52 +0530 IST
2019-02-02 00:00:00 +0530 IST
PASS
ok  	github.com/robfig/cron/v3	0.014s

Please try v3 and let me know if you still see any issue. Thanks

authors: karthikeyang-gommt robfig

repo: robfig/cron


text:
Gemini adapter Refractoring/Testing

[x] Test initializing with and without a db
[x] Test test_gemini_db
[x] Test case_mixin

[x] Test _get_individuals
[x] Test _get_cases
[x] Test individuals
[x] Test case
[x] Test cases

[x] Test varant_mixin

[x] Test variant
[x] Test variants
[x] Test _get_genotypes
[x] Test _get_transcripts
[x] Test _variants
[x] Test _format_variant
[x] Test _is_variant

Awesome! Can this be closed then?

When merged I guess

Nice :-D

authors: guillermo-carrasco moonso

repo: robinandeer/puzzle


text:
Game replays

Describe the solution you'd like
Curious if there's any way to record / replay games from the server side?
Describe alternatives you've considered
Right now I have a semblance of recording via some custom python rendering in bot PoV. Would be cool to see it from the server side though.

Currently, this is not supported.
But it should not be hard to implement, as an observer can be made to receive both game state and events, which can record the battle into e.g. xml, json or something else.
Next step would then be to extend the GUI to read the file, and display the battle. 🙂

repo: robocode-dev/tank-royale


text:
glyph.unicodes questions

glyph.unicodes currently returns a tuple, not a list. Bug?
When setting glyph.unicodes, shouldn't duplicate values be removed? Right now they're not.

glyph.unicodes currently returns a tuple, not a list. Bug?

No, that's by design. Back in RoboFab it confused people that attributes that returned lists weren't mutable. For example:
components = glyph.components
del components[1]

That wouldn't even raise an error. It would just silently do nothing. So, in FontParts I have these returning a tuple. Does that make sense to you? It was one of the early decisions that I made really quickly and didn't think about again.

When setting glyph.unicodes, shouldn't duplicate values be removed? Right now they're not.

Yes. Good catch. This should be changed in fontParts.base.normalizers.normalizeGlyphUnicodes.

I understand the reasoning, but the current situation — setting with a list and getting a tuple — doesn't seem that intuitive to me. What about changing the API to require using a tuple to set glyph.unicodes. Would that be too much of a breakage or even worth it?
I'll take care of the change in fontParts.base.normalizers.normalizeGlyphUnicodes.

I understand the reasoning, but the current situation — setting with a list and getting a tuple — doesn't seem that intuitive to me. What about changing the API to require using a tuple to set glyph.unicodes. Would that be too much of a breakage or even worth it?

In general, I want to be very flexible about incoming data types. Given that this is a scripting API for designers who often don't know a ton about programming, I don't want to be make them have to worry about data types beyond the basics*. So, when setting ordered sequence values they can use list or tuple, for numerical values they can use float or int, there's no difference between str and uni (fingers crossed), etc. Internally, fontParts normalizes these values to standard types before passing them to the low-level environment implementation. This allows app developers get reliable incoming data while the scripters get to be fast and sloppy and still have things work. When values are coming from the low-level environment implementation, they are passed through the normalizers so that the scripter is getting a consistent output.
*This is what really interests me about your question. Maybe the output tuple is more of an unexpected thing than not being able to manipulate a glyph with something like del glyph.components[1]. What did you need to do with glyph.unicodes that you couldn't do with the tuple? How did you even notice that it was a tuple? :)

I'll take care of the change in fontParts.base.normalizers.normalizeGlyphUnicodes.

Thanks!

OK, I get your point.
I noticed while writing tests. The documentation needs a little tweak. Where are those source files again?

That documentation will be in base/glyph . Looks like the tuple bit should be in the normalizer, not the class

On Mar 21, 2018, at 8:07 PM, Miguel Sousa notifications@github.com wrote:
OK, I get your point.
I noticed while writing tests. The documentation needs a little tweak. Where are those source files again?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Fixed by https://github.com/robofab-developers/fontParts/commit/569af683fbbfbe9d403ab6cd60b9752f03045803

authors: benkiel miguelsousa typesupply

repo: robofab-developers/fontParts


text:
[QUESTION] Why .robot files icons are now grey ?

I was wondering why icons switched to grey ? It was much easier before to find .robot files at first glance, especially using dark mode.
This was also part of reasons why I chosed this extension, because others won't give green icons.
Before :

Now :

On topic of feedback. I do think it's a good question to ask. While I personally don't mind a 'dark mode' colouring. I do think that the gray colour we have right now puts me on a bit of a back foot because the gray colour is in the same colour scheme as the same as when they are part of the 'git ignore' colour scheme.
Also in white it's pretty hard to look I find, personally.
Example Dark:

Example White (I did turn it back to black as quick as possible after this)

I do think it helps spotting the Robot files easier if they're in the green spectrum. I would recommend light colours as opposed to low-contrast combinations. This will also help those visually impaired. If you like, you could consider changing .resource to a different colour if you want to separate them out more clearly.
Chat GPT told me these can be considered as a palette, though I'd defer to an actual expert on these things:
.robot file icon - A robot-related theme is typical, so a combination of blue and white can work well:

Light blue (e.g., #4D9FC1) for the robot parts or accents.
White (e.g., #FFFFFF) for the robot face or features, ensuring visibility.
Dark blue (e.g., #003B4F) for outlines or shadows to create a clear structure.

.resource file icon - For a resource file, you could use green or yellow as a color that suggests something like resources, modules, or scripts:

Light green (e.g., #64DD17) for the main part of the icon to signify resources or libraries.
Dark green (e.g., #2C6B28) for accents or outlining to ensure visibility.
White for text or other important details to make it stand out clearly.

So it seems green wasn't a bad choice overall according to the LLM, although I think it was thinking of an actual robot icon there. 😄

Hey, thanks for your feedback! 😊
I changed the icons a bit to align them more with the standard design of VS Code. The idea was to make them less bright and flashy, blending more naturally with the overall look. Personally, I prefer it this way, but I know tastes can differ.
Now, honestly: Is the new color a dealbreaker for you? Would it make you consider switching to something else, or is it fine for you? I’d just like to get a sense of how this change is landing with users.
By the way, there might be an option to create a custom VS Code File Icon Theme or ask existing File Icon Themes to add icons for robot files, for those who prefer a different style. Let me know if that’s something you’d be interested in! And if there’s enough interest, we could also set up a quick poll in Slack to gather more opinions.
Thanks again for sharing your thoughts! 🚀

First of all, thanks for your answer !
I wouldn't say it's a real dealbreaker but I loved the way it was and I'm a bit frustrated now.
If it's possible to add such an option (personalisation of icons), I would definitely like that !
Thanks again for the time you take to answer each time !

ok, I decided to change the colors to a the official RF colors, it's something like green, but not neon green. this will be published with the next version.

Thank you !
I really appreciate the way you consider our feedbacks.
In fact, it is the main reason why I stick to your extension.

thanks @BriceBoy if you want you can give me some stars at the marketplace  ;-)

You are right I never thought about it, but I think you truly deserve it so I just did so 🎉

authors: BriceBoy GerwinLaagland d-biehl

repo: robotcodedev/robotcode


text:
Use of yarpmanager within assignment_make-it-roll

Hi,
I've come up with my solution to the make-it-roll assignment and I want to test it using yarpmanager. Thus, I have launched yarpmanager and imported the two applications available in the app folder.
The assignment_make-it-roll-app app works just fine, whereas the assignment_make-it-roll-system app does not. In fact, only the yarplogger gets started correctly, while the other modules go in timeout waiting for resources.
What resources do I need?
Thanks

@Arya07 @Iaxama @gmmarconi @massimoregoli @towardthesea @giuliavezzani
anyone willing to help?

While the yarplogger does not have any dependency, all of the other modules depend on the iCubSim to be up and running to start. Try to run iCubSim from a terminal and check that you can run it properly, then try again.

Ok, now it works. I have also sent rpc command to the robot, in this way the robot look down.
Now the robot should search the blue ball or I have to move the head to the right?

You can use the getCog method provided in the main.cpp to get the center of the ball in the two images. Then you have to write the code to fixate at that point (Hint: use the IGazeControl to do that)

No ok, I already use getCog and IGazeControl in the code, but when I run it the robot looks forward.
I run yarpmanager with the two modules, yarpserver and the compiled main.cpp code

For the getCog method to work the ball should be in the visual range of the cameras. The robot has to look down towards the table first, then get the ball position in the two images with the getCog method, then you need to retrieve the ball position using the iGazeControl.triangulate3dPoint method, then ask the robot to approach the ball position with the hand and push it.

Ok, thanks!

Closing for now; feel free to open it again if needed.

authors: Iaxama accateo pattacini

repo: robotology/QA