Why cant i download?

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

python azure_downloader.py -i https://videoengine.punjabcollege.net/f594a712-3ecf-4fdc-a234-f8bf4eaed83c/phy1ch2lect10-new.ism
[generic] manifest(format=mpd-time-csf): Requesting header
WARNING: Falling back on generic information extractor.
[generic] manifest(format=mpd-time-csf): Downloading webpage
[generic] manifest(format=mpd-time-csf): Extracting information
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 815, in wrapper
return func(self, *args, **kwargs)
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 836, in __extract_info
ie_result = ie.extract(url)
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\extractor\common.py", line 534, in extract
ie_result = self._real_extract(url)
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\extractor\generic.py", line 2525, in _real_extract
self._sort_formats(info_dict['formats'])
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\extractor\common.py", line 1374, in _sort_formats
raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\Abdullah\Downloads\azure_video_downloader\azure_downloader.py", line 260, in
ydl.download([inputs])
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 2068, in download
res = self.extract_info(
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 808, in extract_info
return self.__extract_info(url, ie, download, extra_info, process)
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 824, in wrapper
self.report_error(compat_str(e), e.format_traceback())
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 628, in report_error
self.trouble(error_message, tb)
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 598, in trouble
raise DownloadError(message, exc_info)
youtube_dl.utils.DownloadError: ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output
AND the --verbose is:-
PS C:\Users\Abdullah\Downloads\azure_video_downloader> python -m youtube_dl -F https://videoengine.punjabcollege.net/6c8bdf75-c6bf-4c1c-80ef-477cc59fd3a9/chem1ch1lect1-new.ism/manifest --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-F', 'https://videoengine.punjabcollege.net/6c8bdf75-c6bf-4c1c-80ef-477cc59fd3a9/chem1ch1lect1-new.ism/manifest', '--verbose']
[debug] Encodings: locale cp1252, fs utf-8, out utf-8, pref cp1252
[debug] youtube-dl version 2021.06.06
[debug] Python version 3.9.5 (CPython) - Windows-10-10.0.19043-SP0
[debug] exe versions: none
[debug] Proxy map: {}
[generic] manifest: Requesting header
WARNING: Falling back on generic information extractor.
[generic] manifest: Downloading webpage
[generic] manifest: Extracting information
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 815, in wrapper
return func(self, *args, **kwargs)
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\YoutubeDL.py", line 836, in __extract_info
ie_result = ie.extract(url)
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\extractor\common.py", line 534, in extract
ie_result = self._real_extract(url)
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\extractor\generic.py", line 2508, in _real_extract
self._sort_formats(info_dict['formats'])
File "C:\Users\Abdullah\AppData\Roaming\Python\Python39\site-packages\youtube_dl\extractor\common.py", line 1374, in _sort_formats
raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Description

WRITE DESCRIPTION HERE
Cant download, say that No supported formats found.
why python -m youtube_dl -F 'https://videoengine.punjabcollege.net/f594a712-3ecf-4fdc-a234-f8bf4eaed83c/phy1ch2lect10-new.ism/manifest(format=mpd-time-csf)' is not working but python -m youtube_dl -F "https://amssamples.streaming.mediaservices.windows.net/3b970ae0-39d5-44bd-b3a3-3136143d6435/AzureMediaServicesPromo.ism/manifest(format=mpd-time-csf)" is working just fine??
Help PLZ

I need help

it says "403 - forbidden" for me. are you sure you're supposed to have access to it? The app might not be able to see any formats because it can't access the website itself

I think its Geotagged to my country

incognito in the browser i get

Yes, because when you open incognito, none of your history passwords, cookies etc. are not present, and when you close the window, it deletes all the information from your computer about the activity.

it says "403 - forbidden" for me. are you sure you're supposed to have access to it? The app might not be able to see any formats because it can't access the website itself

I think its Geotagged to my country

if it is geotagged you might need to specify the --geo-bypass argument. click the link for help. it leads to README.md

same here, just download the latest version
youtube-dl --verbose "https://www.youtube.com/watch?v=y6120QOlsfU"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.youtube.com/watch?v=y6120QOlsfU']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.02.10
[debug] Python version 3.9.2 (CPython) - Linux-5.10.0-5-amd64-x86_64-with-glibc2.31
[debug] exe versions: ffmpeg 4.3.2-0, ffprobe 4.3.2-0, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] y6120QOlsfU: Downloading webpage
[youtube] y6120QOlsfU: Downloading API JSON
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
WARNING: unable to extract player URL; 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.
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see https://yt-dl.org/update on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 806, in wrapper
return func(self, *args, **kwargs)
File "/usr/lib/python3/dist-packages/youtube_dl/YoutubeDL.py", line 827, in __extract_info
ie_result = ie.extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 532, in extract
ie_result = self._real_extract(url)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/youtube.py", line 1632, in _real_extract
self._sort_formats(formats)
File "/usr/lib/python3/dist-packages/youtube_dl/extractor/common.py", line 1367, in _sort_formats
raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see https://yt-dl.org/update on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

the dump says your app version is 2021.02.10. try updating first via youtube-dl --update

Not sure if it was some stupidity on my side or geo-bypass.
Either way its working... thanks
dave@dave-pc:~/Desktop/dnb/dance$ ~/Downloads/youtube-dl --geo-bypass --verbose "https://www.youtube.com/watch?v=y6120QOlsfU"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--geo-bypass', u'--verbose', u'https://www.youtube.com/watch?v=y6120QOlsfU']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 2.7.18 (CPython) - Linux-5.10.0-5-amd64-x86_64-with-debian-bullseye-sid
[debug] exe versions: ffmpeg 4.3.2-0, ffprobe 4.3.2-0, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] y6120QOlsfU: Downloading webpage
[youtube] y6120QOlsfU: Downloading player 68cc98b3
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'https://r3---sn-q0c7rn76.googlevideo.com/videoplayback?expire=1623246515&ei=U3LAYJWOG8H6xN8P0qSEiA8&ip=2a02%3A8084%3A983e%3Ad380%3Ab8fe%3Abcee%3A463%3A39d4&id=o-AHgTNsYSeiK3lUrXwSBQssb9HMsywbg3oy7lO-nEMVNB&itag=134&aitags=133%2C134%2C160%2C242%2C243%2C278%2C394%2C395%2C396&source=youtube&requiressl=yes&mh=Yy&mm=31%2C26&mn=sn-q0c7rn76%2Csn-aigzrne7&ms=au%2Conr&mv=m&mvi=3&pl=30&initcwndbps=1782500&vprv=1&mime=video%2Fmp4&ns=Q3MZG9e7WjWgjSYwXqIYJ7kF&gir=yes&clen=14579306&dur=232.280&lmt=1575013127368986&mt=1623224679&fvip=3&keepalive=yes&fexp=24001373%2C24007246&c=WEB&txp=5532432&n=2OmufHbbD42F0ip_&sparams=expire%2Cei%2Cip%2Cid%2Caitags%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIhAKzb7u9HMgJQn-5IRSBzK79X2lx3sTfFr0cg05FxznfvAiBAsr_W_uqx82qPZiKQIU0Jv-hhxyTSlYETNIMaLa3fNw%3D%3D&sig=AOq0QJ8wRAIhALlk9_M0Xwq1wyqwN02qjfC0X8iHY_bZsqxB4MqaBo47Ah8XH4hwkOBWExrcr63vLq8sXTg2iVkSQk-Wta1tf-jO'
[download] Destination: Darude - Sandstorm-y6120QOlsfU.f134.mp4
[download] 100% of 13.90MiB in 00:00
[debug] Invoking downloader on u'https://r3---sn-q0c7rn76.googlevideo.com/videoplayback?expire=1623246515&ei=U3LAYJWOG8H6xN8P0qSEiA8&ip=2a02%3A8084%3A983e%3Ad380%3Ab8fe%3Abcee%3A463%3A39d4&id=o-AHgTNsYSeiK3lUrXwSBQssb9HMsywbg3oy7lO-nEMVNB&itag=140&source=youtube&requiressl=yes&mh=Yy&mm=31%2C26&mn=sn-q0c7rn76%2Csn-aigzrne7&ms=au%2Conr&mv=m&mvi=3&pl=30&initcwndbps=1782500&vprv=1&mime=audio%2Fmp4&ns=Q3MZG9e7WjWgjSYwXqIYJ7kF&gir=yes&clen=3760893&dur=232.338&lmt=1575013035271523&mt=1623224679&fvip=3&keepalive=yes&fexp=24001373%2C24007246&c=WEB&txp=5531432&n=2OmufHbbD42F0ip_&sparams=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cvprv%2Cmime%2Cns%2Cgir%2Cclen%2Cdur%2Clmt&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRQIgFkIBJNB9dHbg1SZFc6KNjklqKxzNnEU0IXFeoMuS8CICIQDuD_Pp1Czbi6jword1um2lU-YTM8WIYcBp9Nc-nESFKw%3D%3D&sig=AOq0QJ8wRQIgTO7TzRoImDtGx7eLIJqRhInF_tI9hmlUROfLOIXy19YCIQC_GbwVeu-waLVZ9PBs_a6HERbG42Ts4Ocq2veeAULrHQ=='
[download] Destination: Darude - Sandstorm-y6120QOlsfU.f140.m4a
[download] 100% of 3.59MiB in 00:00
[ffmpeg] Merging formats into "Darude - Sandstorm-y6120QOlsfU.mp4"
[debug] ffmpeg command line: ffmpeg -y -loglevel 'repeat+info' -i 'file:Darude - Sandstorm-y6120QOlsfU.f134.mp4' -i 'file:Darude - Sandstorm-y6120QOlsfU.f140.m4a' -c copy -map '0:v:0' -map '1:a:0' 'file:Darude - Sandstorm-y6120QOlsfU.temp.mp4'
Deleting original file Darude - Sandstorm-y6120QOlsfU.f134.mp4 (pass -k to keep)
Deleting original file Darude - Sandstorm-y6120QOlsfU.f140.m4a (pass -k to keep)

It's seriously weird that packagers aren't using automatic fetching of source code to auto-compile.

@rautamiekka Rogério Brito has passed away due to corona virus.  I talked with his wife via email. i will look into picking up the package

it says "403 - forbidden" for me. are you sure you're supposed to have access to it? The app might not be able to see any formats because it can't access the website itself

I think its Geotagged to my country

if it is geotagged you might need to specify the --geo-bypass argument. click the link for help. it leads to README.md

@didotb  Its doesnot show 404 error, as you are expeirenceinng, because its geotagged to my country. So, rather you have to use that comand and check whether it can be downloaded or not.
On my side it first processes Manifest, and then say No supported formats are found. and the file is .ism (Azure Service)
why python -m youtube_dl -F 'https://videoengine.punjabcollege.net/f594a712-3ecf-4fdc-a234-f8bf4eaed83c/phy1ch2lect10-new.ism/manifest(format=mpd-time-csf)' is not working but python -m youtube_dl -F "https://amssamples.streaming.mediaservices.windows.net/3b970ae0-39d5-44bd-b3a3-3136143d6435/AzureMediaServicesPromo.ism/manifest(format=mpd-time-csf)" is working just fine??

it says "403 - forbidden" for me. are you sure you're supposed to have access to it? The app might not be able to see any formats because it can't access the website itself

I think its Geotagged to my country

if it is geotagged you might need to specify the --geo-bypass argument. click the link for help. it leads to README.md

@didotb Its doesn't show 403 error on my side; as you are experiencing, because its geotagged to my country. So, rather you have to use that command and check whether it can be downloaded or not.
On my side it first processes Manifest, and then say No supported formats are found. and the file is .ism (Azure Service)
why python -m youtube_dl -F 'https://videoengine.punjabcollege.net/f594a712-3ecf-4fdc-a234-f8bf4eaed83c/phy1ch2lect10-new.ism/manifest(format=mpd-time-csf)' is not working but python -m youtube_dl -F "https://amssamples.streaming.mediaservices.windows.net/3b970ae0-39d5-44bd-b3a3-3136143d6435/AzureMediaServicesPromo.ism/manifest(format=mpd-time-csf)" is working just fine??

What is your country code? so I could try it on my end

What is your country code? so I could try it on my end

If you are talking about the country dialing code, then that's +92 other wise the code is 92

Issue should be closed, URL is dead


Add Missing Performance Tests for Monitor Query

Add performance tests for

Making a simple query to the service "requests | summarize count() by bin(timestamp, 1h)", "timestamp = P1D"
Use $batch to run 5 queries in 1 api call
Get query statistics (cpu usage / memory / execution time)

Fixed in https://github.com/Azure/azure-sdk-for-java/pull/23280


feat: add useAnime composable

add useAnime composable to module.
add useage example to playground.
add useage example to readme

Type of Change

[ ] Bug fix
[x] New feature
[x] Documentation
[ ] Breaking change
[ ] Other

Request Description

This PR will add the useAnime composable once the module is installed. I added it because in Nuxt3, i dont like importing stuff manually (The Nuxt team spoiled us lolol)

This is related to the discuession started here: https://github.com/hypernym-studio/nuxt-anime/discussions/12

Looks good so far.
Regarding the auto-import, we also need an explicit import for composable, since the user can opt-out of the built-in Nuxt feature.
So if the Nuxt autoImport is disabled:
// nuxt.config.ts

export default defineNuxtConfig({
  imports: {
    autoImport: false
  }
})

the user will be able to manually import the composable like this:
<script setup lang="ts">
  // Explicit import (optional)
  import { useAnime } from '#anime'

  useAnime({ targets: '.title', translateX: 250, duration: 800 })
</script>

Also, it would be nice if the module had an autoImport option to be more consistent with the framework and give the user a choice, since auto-import is a opinionated feature.
Example:
// nuxt.config.ts

{
  anime: {
    autoImport: true // default is true
  }
}

Here are types for ModuleOptions:
export interface ModuleOptions {
  /**
   * @default undefined
   */
  composable?: boolean
  /**
   * @default undefined
   */
  autoImport?: boolean
}

@ivodolenc  So i am a bit confused  when it comes on to the moduleOptions. What will the two  options do differently?
If i pass composable: true, what should happen in the module? If i pass autoImport: true, what happens then? aren't both used to add the useAnime composable to the app?

@BayBreezy Sorry if I wasn't clear enough, it might be best to split my suggestions into a new PR since they have a separate function from your initial proposal.
I will merge your PR that implements the new useAnime composable.
Also, before the final release, I will add a new PR with these additional suggestions.
Thanks for your time and contribution.

@ivodolenc Ok, cool. Sorry I couldn't add it as needed. I am looking forward to your implementation 😄 . I think I might learn something

@BayBreezy No problem, I'm glad you're interested in this module 😀


WIP Landsat Collection 2

For #37. This is a very early WIP to start talking about the architecture of the Landsat Collection 2 Reader. (Also because I haven't kept up with the myriad changes to rio-tiler for v2)

Presumably we'll want to support level 1 and level 2 of collection 2. But since level 2 is atmospherically corrected, we don't need necessarily need the complexity of the existing L8Reader it seems? The Sentinel 2 reader is much less complex
Can we support everything in one class? In terms of fetching the COGs themselves, constructing accurate paths is possible for every dataset given the _prefix format string and metadata from sceneid_parser

Todo:

[ ] Add sceneid_parser tests for a variety of collection 2 ids (it seems these are technically called product_ids)
...

Presumably we'll want to support level 1 and level 2 of collection 2. But since level 2 is atmospherically corrected, we don't need necessarily need the complexity of the existing L8Reader it seems?

Well, yes for level2 you don't need a much complex reader but for level1 it would still be nice to have TOA reflectance as output. you could create a small L8L1Reader (subclass of COGReader) as https://github.com/cogeotiff/rio-tiler-pds/blob/master/rio_tiler_pds/landsat/aws/landsat8.py#L41 🤷♂

The Sentinel 2 reader is much less complex

🤷♂

Can we support everything in one class? In terms of fetching the COGs themselves, constructing accurate paths is possible for every dataset given the _prefix format string and metadata from sceneid_parser

I think it's possible!


Android push.on('notification') event handlers not being called when using FCM

Expected Behaviour
Android only:
1.1  Push message arrives with app in background (payload does not contain content-available: 1)
1.2  User clicks on notification in notification center
1.3  The app resumes and the plugin calls all of the notification event handlers
2.1  Push message arrives with app closed (payload does not contain content-available: 1)
2.2  User clicks on notification in notification center
2.3  The app starts and the plugin calls all of the notification event handlers
Actual Behaviour
Android only. This applies to both the app being in the background and the app being closed.

Push message arrives
User clicks on notification in notification center
The app starts but does NOT call the notification event handlers

Behavior that is working correctly:

(Android) Push message arrives while app is in foreground.  notification event handlers are called.

(iOS) Push message arrives while app is in background or closed.  User clicks on notification in notification center.  The app starts and notification event handlers are called.

(iOS) Push message arrives while app is in foreground.  notification event handers are called.

Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
Android: 5.1
iOS: 9.3.2
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Motorola: Moto E (2nd Generation)
Cordova CLI version and cordova platform version
cordova 6.2.0
cordova android 4.1.1
cordova ios 4.1.0

Plugin version
phonegap-plugin-push 2.0.0 "PushPlugin"
Using the most recent commit of the 'FCM' branch
07fa81b7a3a62400915ac70b9a24a802465a6f95

Sample Push Data Payload
I am using FCM for push notifications, not GCM.
{
  to: '/topics/news',
  data: {
    message: 'this is a custom param',
    signal_id: 'this is a custom param'
  },
  notification: {
    title: 'Test title',
    body: 'Test body',
    sound: 'default',
    badge: '1',
    icon: 'ic_launcher'
  },
  priority: 'high'
};

Sample Code that illustrates the problem
This is the initialization code:
let config = {
  android: {
    senderID: '85XXXXXXXXX',
    icon: 'icon',
    sound: true,
    vibrate: true,
    clearNotifications: true,
    topics: ['news']
  },
  ios: {
    senderID: '85XXXXXXXXX',
    gcmSandbox: true,
    alert: true,
    badge: true,
    sound: true,
    clearBadge: true,
    topics: ['news']
  }
};
this.push = PushNotification.init(config);

this.push.on('registration', (data) => {
  console.log('registration complete');
  console.log(data.registrationId);
});

this.push.on('notification', (data) => {
  console.log('notification event handler invoked');
});

Logs taken while reproducing problem
There were no errors.

@lukebpowell the plugin does not currently support FCM. It is being worked on in the v2.0.x branch of this plugin. When issue #929 is finished then support will be official at this time we're not opening bugs on unreleased code.

@macdonst alright, thanks for the reply.

@lukebpowell np, feel free to comment on #929 with your findings. I'm on vacation this week but I'll be getting to FCM with Julio and hopefully another committer soon.


App is closing instantly in Task Manager

Trying to start RPW but it closes instantly, i was playing with it yesterday and everything was good

Same situation here


Maven Central

Hi,
do you have plans to publish Artifacts to Maven Central. As an enduser I would expect to find them there.
Regards
Andreas

@weand - Totally. We're in the process of pushing the Maven Central artifacts for version 1.0.0 These should be available shortly. Thanks for your patience :-)

One further question regarding the artifacts that will be deployed to maven central. Are you going to compile them with JDK 12, like the main plugin? Would that mean, that SPI developers would also have to use JDK 12.
From my perspective it would make sense to better compile against the latest JDK LTS version, which is JDK 11 at the moment. I assume that newer non-LTS versions aren't widely adopted by the community.

Hi @weand ,
Sorry for the delayed response.
Yes, we use JDK 12 to build this JobScheduler plugin, because Elasticsearch 7 requires JDK12 to build plugins. Even we use JDK 12 as our compiler, we still specify the source and target compatibility to 1.8 in our build logic, so that this plugin can run on JDK 8, 10, 11 and 12
Hope this answers your question.

JobScheduler spi has been released to MavenCentral starting OpenDistroForElasticsearch 1.0. https://mvnrepository.com/artifact/com.amazon.opendistroforelasticsearch/opendistro-job-scheduler-spi


Add LICENSE file

The README.md makes the license clear for humans, but computers can benefit from having it in its own file.

Thanks - I'm not accepting this pull request since I'm going to change the license to MIT.


Azure Data Studio CLI

We are currently working on an ADS extension and our build out a release environment in Azure DevOps. Is there a way to automate the .vsix download? I know that ADS is built of the VSCode framework and VSCode has a CLI but I have not seen any CLI from ADS. Is there a CLI for ADS or when is it coming out soon? Please let me know if you have any additional questions.

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

ID: 12bf49db-fd24-7f4d-6792-6825f4e08faa
Version Independent ID: 5f524d58-816a-812f-e43f-1916f1b4f199
Content: Add extensions - Azure Data Studio
Content Source: docs/azure-data-studio/extensions.md
Product: sql
GitHub Login: @stevestein
Microsoft Alias: sstein

@jclark0909 - Hi Jared, Apologies for the slow reply. I'm trying to find out if there is any plan for this but haven't been able to get a solid answer yet.
You might try posting the question at UserVoice
@yualan - Do you have any additional info or insight you can provide on this?

@jclark0909 - This area is primarily for specific documentation issues, and I haven't been able to get an official statement on this so I suggest either posting as an Azure Data Studio issue, or posting on the UserVoice  site.
-Thanks


Remove commit instructions. Closes #127

What:
Removed section on committing and pushing from CONTRIBUTING.md

Why:
Now that commitizen has been removed from the project (see 8abbc685c0e909f8b9c2788bf8bbe2bd703f1f5c) these instructions are no longer relevant.

How:
Deleted section.

Codecov Report

Merging #140 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #140   +/-   ##
======================================
  Coverage    96.2%   96.2%
======================================
  Files          57      57
  Lines         211     211
  Branches       20      20
======================================
  Hits          203     203
  Misses          7       7
  Partials        1       1

Continue to review full report at Codecov.

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

Awesome, thanks :)


"bad huffman code under subscribed" error on valid PNG

Hi,
I'm unable to load a valid PNG (see attached file) using wuffs, but this PNG loads with libpng (zlib), stb_image.h, lodepng, and validates OK with pngcheck. After some investigation it appears that if the Deflate distance table only has a single used code (with a bit length of 1), wuffs refuses to load the image.
This is incorrect behavior, as far as I can tell. From the Deflate spec 3.2.7. Compression with dynamic Huffman codes (BTYPE=10):
https://datatracker.ietf.org/doc/rfc1951/
If only one distance code is used, it is encoded using one bit, not zero bits;
in this case there is a single code length of one, with one unused code.

The problem is in function wuffs_deflate__decoder__init_huff():
if (v_remaining != 0) {
    if ((a_which == 1) &&
        (v_counts[1] == 1) &&
        (self->private_data.f_code_lengths[a_n_codes0] == 1) &&
        ((((uint32_t)(v_counts[0])) + a_n_codes0 + 1) == a_n_codes1)) {
      self->private_impl.f_n_huffs_bits[1] = 1;
      self->private_data.f_huffs[1][0] = (WUFFS_DEFLATE__DCODE_MAGIC_NUMBERS[0] | 1);
      self->private_data.f_huffs[1][1] = (WUFFS_DEFLATE__DCODE_MAGIC_NUMBERS[31] | 1);
      return wuffs_base__make_status(NULL);
    }
    return wuffs_base__make_status(wuffs_deflate__error__bad_huffman_code_under_subscribed);
  }

Thanks for the great bug report.
release/c/wuffs-v0.3.c picked up the fix in the follow-up commit 58608c4e.
https://github.com/richgel999/fpng is also a cool project. :-)


Array spread operator does not resolve type correctly (for javascript in vscode)

Bug Report

🔎 Search Terms

array spread, jsdoc spread operator, spread operator type
🕗 Version & Regression Information

This happens in every version I tried, I searched through the FAQ and issues for related issues.

⏯ Playground Link

Does not seem to occur in typescript.
💻 Code

const mapOfStrings = new Map([["foo", "bar"]])

const arrayOfStrings = [...mapOfStrings.values()]
//    ^ should be type string[]

🙁 Actual behavior

The type shown in VSCode is any[] while it should be string[].

VSCode 1.63.2 (commit: 899d46d82c4c95423fb7e10e68eba52050e30ba3)
OS: MacOS 12.0.1 (21A559)
Typescript 4.5.4

🙂 Expected behavior

The type should be string[] as shown below (this does work with Array.from()).

Interesting, works for me both locally and in TS playground

Strange, I tried running in safe mode and seems to work fine.
Just tried disabling all extensions one by one but it's still showing any[] type for me...
For some reason this only happens for saved files in javascript projects.
Unsaved file:

Saved file:

I can't repro this either. We'll need some specific (probably very specific) steps to see this

I tried making a test file in a different project and worked fine too...
Update as I'm typing this, it looks like an empty jsconfig.json caused the problem


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
INFO: /tmp/abs_d8ykm9kjsb/clone/recipe/meta.yaml:0: gui_app: This may be a GUI application. It is advised to test the GUI.
ERROR: /tmp/abs_d8ykm9kjsb/clone/recipe/meta.yaml:17: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_d8ykm9kjsb/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
Errors were found


Problem while installing NNI on windows

I'm facing some problem while installing NNI. I don't have much knowledge on shell script.
Windows 10 environment
nni Environment:

nni version: v0.9
nni mode(local|pai|remote): local
OS: windows 10
python version: 3.6.7

script : powershell -ExecutionPolicy Bypass -file install.ps1
Error :
Rename-Item : Cannot bind argument to parameter 'Path' because it is null.
At G:\NNI\nni\install.ps1:100 char:21

    Rename-Item $unzipYarnDir "nni-yarn"

                ~~~~~~~~~~~~~

CategoryInfo          : InvalidData: (:) [Rename-Item], ParameterBindingValidationException
FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Microsoft.PowerShell.Commands.RenameItemC
ommand

Copy-Item : Cannot find path 'G:\NNI\nni\src\webui\build' because it does not exist.
At G:\NNI\nni\install.ps1:133 char:1

Copy-Item "src\webui\build" $NNI_PKG_FOLDER_STATIC -Recurse

CategoryInfo          : ObjectNotFound: (G:\NNI\nni\src\webui\build:String) [Copy-Item], ItemNotFoundException
FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.CopyItemCommand

Any lead will be appreciated
Thanks.

Thank you for your question  @anandh02.
@SparkSnail will follow this question.

Hi, have you tried using python -m pip install --upgrade nni to install nni? could you please show the result of command $PSVersionTable?

This worked for me python -m pip install --upgrade nni but installing from source code didn't work for me.
Anyways thanks. You can close the issue.

I'll close this issue, if you have any problem , you could reopen it again, thanks.


image picker saves picture every time

image picker saves picture every time I select and compress a picture, is there anyway to prevent it from saving pictures or auto-delete them?

@yamin8000 This looks like a project-specific requirement. The thing, I will recommend is to delete the picture after your use case has been fulfilled.
For example, If you're using this library for uploading profile image. You can delete the image after you get a successful response from API calls. The only thing you need to take care of is to check the image source before delete. Make sure that you don't delete the gallery image which the user has chosen. The easiest option is to check the file modified time. Else you can specify the saveDir to store the captured or compressed image and later check the file path of it.
Hope this makes sense.  Let me know if you have any queries regarding the same.
Thank you!


feat(scan-pkg-manifest) Adds new scan-pkg-manifest, refactor container scan job

New scan-pkg-manifest job for integration with Lacework Host vulnerability capability
Refactor vuln-scan-run job to ctr-vuln-scan to be more explicit about the job funtionality
Update base container image to use lacework/lacework-cli

Signed-off-by: Scott Ford<EMAIL_ADDRESS>

#!/bin/bash -eo pipefail
yamllint .

./src/jobs/ctr-vuln-scan.yml
  23:15     warning  too many spaces after colon  (colons)

./src/commands/scan-pkg-manifest.yml
  3:78      error    trailing spaces  (trailing-spaces)


[Language Text] Delay release

Fix samples to use the correct env vars
Delay release to Thursday

API change check
API changes are not detected in this pull request.


Usage with pyvisa

Hi!
How can you use this script with pyvisa, where you directly import data in form of bytes and not in the binary form you get when loading file?
Br Kristian

Hi,
I would take the byte-array and throw it into a io.BytesIO class.
From the readTrc point of view, this should look exactly like a file, only
that it is completely in memory.
Best,
Michael
2016-12-15 3:18 GMT-08:00 mkkb<EMAIL_ADDRESS>
Hi!
How can you use this script with pyvisa, where you directly import data in
form of bytes and not in the binary form you get when loading file?
Br Kristian
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yetifrisstlama/readTrc/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGzmK82dI5fqbTfOyjwL07alH-fipfRoks5rISHvgaJpZM4LN_Eu
.

Thank you very much for swift answer, problem solved! [😊]
BR Kristian
Sendt fra Outlookhttp://aka.ms/weboutlook

Fra: M. Betz<EMAIL_ADDRESS>Sendt: 16. desember 2016 01:27
Til: yetifrisstlama/readTrc
Kopi: mkkb; Author
Emne: Re: [yetifrisstlama/readTrc] Usage with pyvisa (#1)
Hi,
I would take the byte-array and throw it into a io.BytesIO class.
From the readTrc point of view, this should look exactly like a file, only
that it is completely in memory.
Best,
Michael
2016-12-15 3:18 GMT-08:00 mkkb<EMAIL_ADDRESS>
Hi!
How can you use this script with pyvisa, where you directly import data in
form of bytes and not in the binary form you get when loading file?
Br Kristian
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yetifrisstlama/readTrc/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGzmK82dI5fqbTfOyjwL07alH-fipfRoks5rISHvgaJpZM4LN_Eu
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/yetifrisstlama/readTrc/issues/1#issuecomment-267493674, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHJIHT1RX-j9Mwo79ktLIcitvTjpU5ksks5rIekJgaJpZM4LN_Eu.


Part 4 Section 2 step 8 Menuhax Problem

After completing steps 6 and 7 Menuhax would not load with the down button or L after rebooting. After accessing browser hax I reinstalled Menuhax and it still would not load. Did deleting the updates folder mess with this somehow?

NVM i just put the HBLauncher in my sd card root and menuhax worked again. IDK chalk this whole thing up as irrelevant


Add middleware lazy creating

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
#29

👍


IOS  just sound no shade

Hello,
With Version 1.4.3 I only getting sound on my iPad no shade?
however in-app notification is working just find.
my payload looks like this PHP:
because I'm using Intel XDK and they don't support gradle i'm using the above version.
 $payload['aps'] = array(
                               'alert' =>  array(
                                    'title' => $title,
                                    'body '  => $message
                               ),
                               'content-available'=> 1,
                               'badge' => '1',
                               'sound' => 'default',

                               'userid'         => $userid,
                               'familyid'       => $familyid,
                               'postid'         => $postid,
                               'type'           => $type,
                               'postuserid'     => $postuserid,
                               'commentid'      => $commentid
                            );

@colonna-mike that's weird. I'm not a PHP expert but you should see this working. Here are two things to try:

Remove content-available => 1 as it can do funny things if your push has a alert, badge or sound as per https://developer.apple.com/library/ios/documentation/NetworkingInternet/Conceptual/RemoteNotificationsPG/Chapters/TheNotificationPayload.html#//apple_ref/doc/uid/TP40008194-CH107-SW6

instead of alert being an array just send it as text:

'alert' =>  $message
and let me know what's happening.

I give it a go but i'm not sure what 'alert' => $message as string would look like?

okay I removed the content-available => 1 and when the app is in the foreground it stop working when I put it back I get  when the app is in the foreground it works just find. its when the app is in the background or needs a cold start is where the issue is.

@colonna-mike what I meant was:
$payload['aps'] = array(
                               'alert' =>  $message,
                               'content-available'=> 1,
                               'badge' => '1',
                               'sound' => 'default',

                               'userid'         => $userid,
                               'familyid'       => $familyid,
                               'postid'         => $postid,
                               'type'           => $type,
                               'postuserid'     => $postuserid,
                               'commentid'      => $commentid
                            );

Instead of using your PHP code can you try following:
http://macdonst.github.io/push-workshop/module4.html
https://github.com/macdonst/push-workshop/blob/master/server/apnService.js
to send your first test message. There is no reason why you shouldn't see the notification in the shade.

okay I could not really use that however, staying with the idea of stripping the payload down I created this payload.
   $payload['aps'] = array('alert' => 'Oh hai!', 'badge' => 1, 'sound' => 'default');
and it worked!  that's the good news the bad news is without the additional payload my app does not show the proper page. .. What  I need to figure out is how to set up the additional data.
thanks!

OKAY, I just reread you comment above and
and it worked! Thank so much for taking the time to help me...
I really do appreciate it.
$payload['aps'] = array(
                               'alert' =>  $message,
                               'content-available'=> 1,
                               'badge' => '1',
                               'sound' => 'default',

                               'userid'         => $userid,
                               'familyid'       => $familyid,
                               'postid'         => $postid,
                               'type'           => $type,
                               'postuserid'     => $postuserid,
                               'commentid'      => $commentid
                            );


Some useful JS techniques that you may not know

I complete reading JavaScript Enlightenment recently. The book is more about basics in JavaScript and suitable for beginners. Here are a list of my benefits and extensions from the book.
Math
JavaScript developers often capitalize the constructor name to distinguish the constructors from normal functions. Therefore, some developers may mistake Math as function since the name is capitalized while Math is really just an object.
console.log(typeof Math); // object
console.log(typeof Array); // function

Function
There are two ways to construct a function:

Function declaration/expression

function sum(x, y) {
    return x + y;
}
console.log(sum(3,4)); //7

Function constructor

let multiply = new Function("x", "y", "return x * y;");
console.log(multiply(3,4)); //12

In development, we often need to use call or apply to switch the function context. E.g.
function print(){
    console.log(this.a);
}

print.call({a: 'hello'}); //hello

Some interview questions will ask why print doesn't define call as its property but print.call() won't throw error. It's because print is an instance from Function constructor so it inherits all the methods defined in Function.prototype through prototype chain.
print.call === Function.prototype.call

How to do Array check
typeof can be used to determine the types for primitive datatypes. But it won't be able to distinguish between arrays and objects.
console.log(typeof [1,2]); //object
console.log(typeof {}); //object

There are several wasy to do Array check:

API: Array.isArray

console.log(Array.isArray([1,2])); //true

toString

console.log(Object.prototype.toString.call([1,2])
            .toLowerCase() === '[object array]'); //true

Note here we have to use Object.prototype.toString with call to switch the calling context, as Array.prototype.toString is overriden.
console.log(Object.prototype.toString.call([1,2])); //[object Array]
console.log([1,2].toString()); //1,2

instanceof

[1,2] instanceof Array === true;

Object.prototype.toString won't work for custom datatypes. In this case we can use instanceof to determine the type.
class Person {}

let mike = new Person();
console.log(Object.prototype.toString.call(mike)); // [object Object]
console.log(mike instanceof Person); // true

undefined vs null
undefined - no value
There are two cases where a variable is undefined.

The variable is deifned but not assigned any value.

let s;
console.log(s); //undefined

The variable is NOT defined at all.

let obj1 = {};
console.log(obj1.a); //undefined

null - special value
let obj2 = {a: null};
console.log(obj2.a); //null

If we aim to filter out undefined and null, we can use weak comparison with double equality sign(i.e. ==).
console.log(null == undefined); //true
let arr = [null, undefined, 1];
let fil = arr.filter(it => {
    return it != null;
});
console.log(fil); //[1]

Reference
JavaScript Enlightenment
Notice

If you want to follow the latest news/articles for the series of reading notes, Please 「Watch」to Subscribe.

Interesting notes.
The Function constructor way is safe for block scope, but it has inject risk.
function a(c){   return (new Function(c))(c); } a('any js code');
I'm thinking about 'null' and 'undefined'. When and how to chose one?

@libook
Good point about Function constructor.
In terms of null and undefined,  null is a special value that means 'no value' :-). So when you use null, it does have a value that you want to deliberately ignore or view as meaningless, e.g. invalid value.
Think about this use case, I'm doing a questionnaire asking people's age. For people who skip the question, the answer to the question is undefined. For people who answer the question but with an invalid answer e.g. 'abc' (which is invalid as age has to be a number), I will set the answer in the case to be null.

@n0ruSh In common, user interaction designing should avoid invalid answers.

In terms of null and undefined, null is a special value that means 'no value' :-). So when you use null, it does have a value that you want to deliberately ignore or view as meaningless, e.g. invalid value.

I think this may be a direction.
I've seen this on StackOverflow:

The difference between null and undefined is: JavaScript will never set anything to null, that's usually what we do. While we can set variables to undefined, we prefer null because it's not something that is ever done for us. When you're debugging this means that anything set to null is of your own doing and not JavaScript. Beyond that, these two special values are nearly equivalent.

If this is true, 'null' and 'undefined' could be very useful at improving code reliability.


Layer order in L.Control.Layers is not preserved with GeoJson/FeatureGroup

When I do
L.control.layers({
    'OSM': L.tileLayer('http://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png').addTo(map)
}, {
    'Circle': L.circle([53, 4], 111111),
    'Polygon': L.polygon([[48, -3], [50, -4], [52, 4]]),
    'GeoJSON': L.geoJson(geojson)
}, {
    collapsed: false
}).addTo(map);

I expect the order of the overlays to be as I supplied them, however, GeoJSON and FeatureGroup layers are moved up top. This is because the FeatureGroup gets its _leaflet_id while it's created, while the other layers get it while they're added to Control.Layers.
FeatureGroup is stamped because it's passed as the context to layer.on in FeatureGroup.js:19.
Not sure how to fix this:

stamping each layer while initializing? Seems dirty.
Different sorting mechanism in Control.Layers?

http://jsfiddle.net/km5H9/1/

I think this is still valid?
I have a geoJSON layer among several tileLayers. Initial placement works fine but on certain page events that may change layer availability I trigger a "refresh" by removing and re-adding all layers in the same order as before, but my geoJSON layer now appears at the top of the control.
I was able to work around this by removing the old geoJSON layer and replacing it with a new one (ostensibly with a new internal ID in a proper order) and it appears in the correct place.

Hi folks. I'm also experiencing this with geojson featuregroups. Lots of overlapping polygons and the zindex changes confuse the user. Can anyone suggest a canonical/somewhat-future-proof method for maintaining group display order ?

Afraid not. I build an array of layers in their correct order, and cycle through the whole array each time layers are added, bringing every layer in turn to the front. Horrible, but it works. I have no alternative at this stage.

I'm basically doing what tomchadwin has except more explicit, since I've just kind of grown into the code. If I get around to refactoring it I'll either use a similar array or probably make my own little Javascript snippet to manage my own control interface and stick it within the map.

If anyone here is using Leaflet 1.0.0 you should be able to use the new pane option to control this:
map.createPane('bottom');
map.createPane('middle');
map.createPane('top');

// now pane order controls layer ordering not the control!
L.control.layers({
    'OSM': L.tileLayer('http://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png').addTo(map)
}, {
    'bottom': L.geoJson(geojson, {pane: 'bottom'}),
    'middle': L.geoJson(geojson, {pane: 'middle'}),
    'top':  L.geoJson(geojson, {pane: 'top'}),,
}, {
    collapsed: false
}).addTo(map);

You can see this inaction with lots of overlapping polygons here.

patrickarlt thanks - this is helpful.

patrickarlt thought I'd let you know I implemented the pattern you outlined above. This does lay out the order of the panes properly such that panes and their associated layer group are added to the map such that the pane appears beneath successively added panes. However, when using l.control.layers.addOverlay to then associate L.control.layers with panes on the map --> zero order to the layer names as they appear in the control seems possible. Half way there! Any suggestions ?

I look a look at the code. It looks like layers are stored internally in the control by the layers internal Leaflet ID https://github.com/Leaflet/Leaflet/blob/master/src/control/Control.Layers.js#L115-L121. The internal ID generator gets called A LOT. For example in my JS Bin http://jsbin.com/vocigipive/1/edit?html,css,js,output it is already at 177 by the time everything gets done.
Since all browsers sort objects when looping like for (i in this._layers) by the order in which they are defined http://stackoverflow.com/questions/280713/elements-order-in-a-for-in-loop this means that you cannot really rely on any specific ordering when using L.Control.layers.
@mourner I'm wondering if this could be refactored to use an array of layers internally rather then an object.
// convert basemaps and overlays to ordered arrays internally
L.control.layers({
    'OSM': L.tileLayer('http://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png').addTo(map)
}, {
    'bottom': L.geoJson(geojson, {pane: 'bottom'}),
    'middle': L.geoJson(geojson, {pane: 'middle'}),
    'top':  L.geoJson(geojson, {pane: 'top'}),,
}).addTo(map);

 // add a new position property to control where the added layer falls in the order. Default would be at the bottom.
control.addOverlay(layer, name, position);

@patrickarlt I think that's a great approach. I would be up for a PR on this.

@patrickarlt up for working on this before 1.0 or should I postpone? :)

Huh, I already made a PR for this...


Added ability to use owner key if provided to the morph relation

In Laravel Polymorphic relations, you can specify the owner key to the relation
protected function getResultsByType($type)
{
    $instance = $this->createModelByType($type);

    $key = $instance->getKeyName(); // <--- Before the change
    $ownerKey = $this->ownerKey ?? $instance->getKeyName(); // <---- After the change

    $query = $instance->newQuery();

    return $query->whereIn($ownerKey, $this->gatherKeysByType($type, $instance->getKeyType()))->get();
}

Checklist

[x] Add tests and ensure they pass
[ ] Add an entry to the CHANGELOG.md file
[ ] Update documentation for new features

That sounds correct to me. Could you add a test?

I've tried to reproduce the error, but I can't find the condition in which the code you've fixed is called.
It seems to be with eager loading of relations, but I don't find how. Could you show an example of your model definition and your query?

Fixed by #3162


Maas 3231 generate start and end in prometheus check dynamically

Fixed bugs in logic of prometheus plugin:

generate start and end dynamically based on frequency
convert received data into a flat list of floats (without timestamps)

Coverage increased (+0.2%) to 91.135% when pulling 8f2cec822bf61c33d865b5f554030d7f336d9726 on MAAS-3231-generate-start-and-end-dynamically into b0214b6dc1f4b651953203b94b36d991a0a64277 on develop.

Coverage increased (+0.2%) to 91.135% when pulling 8f2cec822bf61c33d865b5f554030d7f336d9726 on MAAS-3231-generate-start-and-end-dynamically into b0214b6dc1f4b651953203b94b36d991a0a64277 on develop.

Coverage increased (+0.2%) to 91.128% when pulling<PHONE_NUMBER>943fb75976435e45722af045b46136 on MAAS-3231-generate-start-and-end-dynamically into b0214b6dc1f4b651953203b94b36d991a0a64277 on develop.

Coverage increased (+0.08%) to 91.057% when pulling da090a134cf2d88272ebb370f252fb5c928a5941 on MAAS-3231-generate-start-and-end-dynamically into b0214b6dc1f4b651953203b94b36d991a0a64277 on develop.


Fix/Documentation: Obfuscate Empire does not install properly

Empire Version
2.5-dev
OS Information (Linux flavor, Python version)
Fedora 28 RHEL, Linux 4.18.13-200.fc28.x86_64
Python: 2.7.15 / 3.6.6
Expected behavior and description of the error, including any actions taken immediately prior to the error.
On default installation, Empire's Obfuscate functionality failed repeatedly because of the vague error
[Errno 2] No such file or directory: '/opt/Empire/data/misc/Obfuscated.ps1'
The error does not really explain what is really going wrong, which is that the Invoke-Obfuscation module is actually missing from the Powershell Modules directory because it was not able to be written on install. This causes the Invoke-Obfuscation module to fail to execute, and never creates the Obfuscated.ps1 file, resulting in this error.
I actually could not find any documentation or indicators within the setup scripts or elsewhere in the Empire code that gave me any clues. I actually found the solution by going all the way back to the original Obfuscate-Empire repo from @cobbr on the following issue: https://github.com/cobbr/ObfuscatedEmpire/issues/2
Empire's "Obfuscate" functionality should be working out of the box, or at least be able to tell the user why it doesn't work. In its current state, it can tell the user when powershell is not installed, but not when the Invoke-Obfuscation module is missing from the Powershell modules themselves.
Solution
Some powershell error-handling and a check for Invoke-Obfuscate in the powershell installation directory should be able to resolve the issue. For instance, calling Invoke-Obfuscation when it does not exist produces the following [System.Management.Automation.CommandNotFoundException] exception similar to as follows.
PS /home/truedemon> Invoke-Idontexist
Invoke-Idontexist : The term 'Invoke-Idontexist' is not recognized as the name of a cmdlet, function, script file, or operable program.
Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1
+ Invoke-Idontexist
+ ~~~~~~~~~~~~~~~~~
+ CategoryInfo          : ObjectNotFound: (Invoke-Idontexist:String) [], CommandNotFoundException
+ FullyQualifiedErrorId : CommandNotFoundException

Catching this upon execution of the Invoke-Obfuscation call should be an immediate indicator to the user that, for whatever reason, it is missing from the /path/to/microsoft/powershell/6/Modules directory, and they should simply cp -r /path/to/empire/lib/powershell/Invoke-Obfuscation /path/to/microsoft/powershell/6/Modules/
If you wanted to go a step further and handle the issue for the user, you could follow which powershell || which pwsh to where it's symbolically linked and copy it for them or cowardly bail out with the OS.Error exception that would likely follow upon failure.
Either way, it took me hours to solve this problem, when the fix should have only taken 5 minutes or less. I saw how many times it has appeared in the issue tracker, so this should cut down on the number of occurrences significantly.

The most likely reason Invoke-Obfuscation wouldn't install correctly is it the install script wasn't run as root. Most of the commands that require elevated permissions are prefixed by sudo, but not all of them.
I've been looking at how to re-work the install script, since there have been so many issues with it lately, and this is one thing I'll keep in mind.
Empire already checks for either powershell or pwsh, so that's not related to this.


Timestamp filter not working as expected

Describe the bug
The timestamp filter does not appear to be working correctly - it is including items outside of the parameters.
Expected behavior
The timestamp filter should only include the items indicated.
Current behavior
I was trying to find unused rules older with a 180 days.  However, the results came back with timestamp-last-hit.fast sooner than that (in fact, rules with last hits of today).
Some background: this cluster was the replacement for several other clusters.  On the day of cutover (December 11, 2021), there are a lot of matches for that date as the last hit (there was a zone issue I had to fix).  These rules would have been marked unused otherwise.
Possible solution
It's possible that I'm not using the filter correctly.  However, I tried it both ways (less than or greater than in different runs).  Less than 180 days is closer to accurate than greater than.  Greater than 180 days appears to return all security policy rules (I say appears because even though I'm limiting the location & ruletype I get back more rules than exist - for example, this firewall only has 3,480 explicit rules (plus the 2 implied) but the results come back with 3,484 rules.  Less than 180 days returns 2,174 rules.
I also tried it using a date instead of 180, but I received the same results.
Steps to reproduce
<1>
I tried it this way first:
pan-os-php type=rule in=api://panorama location=DG1 ruletype=security actions=exportToExcel:last-hit-180-A.xls  'filter=!(timestamp-last-hit.fast < - 180 days)'
This produced 2,174 rules.  I spot checked a couple of rules and it looked OK so I ran my 'real' action (in this case, adding an UNUSED tag) and when I spot checked that I found rules that had been hit sooner than 180 days (one had a hit today).
<2>
So, I did greater than to see what would happen:
pan-os-php type=rule in=api://panorama location=DG1 ruletype=security actions=exportToExcel:last-hit-180-B.xls  'filter=!(timestamp-last-hit.fast > - 180 days)'
This produced 3,484 rules (basically, all of the rules in the security policy)
Screenshots
Can't post in public - happy to email stuff to you.
Context
We use the unused to mark rules unused.  With the last hit field being added, I had put in a feature request for this filter (thanks again!) so we could search some arbitrary time since last hit.  Otherwise, we'd have to manually clear the hit counts on all stale rules with hit indicators.  Also, if I recall correctly, the unused filter queries a field that gets reset when the firewall is rebooted.
Your Environment

Version used: PANOS 9.1.13-h3
Environment name and version: PAN-OS-PHP v2.0.38
Operating System and version (desktop or mobile): Windows 10 21H2 running in PowerShell

please allow me some questions regarding your filter:

'filter=!(timestamp-last-hit.fast < - 180 days)'
'filter=!(timestamp-last-hit.fast > - 180 days)'

example for 1)
if timestamp-last-hit-fast is 2021 Dec 11th 00::00::00;
then:

timestamp value for 2022 Jul 11th - 180 days => 2022 Jan 12
!(  163918088 <<PHONE_NUMBER> ) => timestamp-last-hit is lower then the timestamp before 180 days; but your negating it => so wrong

I assume your are searching for timestamp-last-hit before -180 days.
so you need for example 1) the following filter; which is also state in the list filters example:
pan-os-php type=rule list filters
(timestamp-last-hit.fast < -180 days)
or
(timestamp-last-hit.fast < 1 February 2022 )
or US syntax MM/DD/YYYY
(timestamp-last-hit.fast < 02/01/2022 )
or European syntax DD-MM-YYYY
(timestamp-last-hit.fast < 01-02-2022 )


Update README.md

Fixed missing link close tag

Super small fix to a closing tag

An eye for detail, I see! Thank you :wink:

😀 No problem, I'll sure I'll contribute something more substantial in the future.
Great collection of threat lists!


Spring Cloud Azure 4.0 auto configuration apply builder factory customizer

In Spring Cloud Azure Core we've implemented a service builder factory customizer interface, this pr will apply it in the auto-configuration and add tests.

/azp run java - spring - tests


\n does not work in HTML tables

We should probably replace '\n' in warn/fail with
 or similar?

https://github.com/CocoaPods/CocoaPods/pull/5607#issuecomment-231547745

Agreed 👍 - it should

I ran into this yesterday and was thinking shouldn't the ruby string be covered over to HTML, so it doesn't require the users to insert markdown that doesn't really represent the formatting of the string?

Hrm, I thought we did that, it's a good point.

We don't, I wanted this and instead got this

Or not, we have specs for this:
      it "some warnings with markdown, no errors" do
        warnings = violations(["a markdown [link to danger](https://github.com/danger/danger)", "second **warning**"])
        result = @g.generate_comment(warnings: warnings, errors: [], messages: [])
        # rubocop:disable Metrics/LineLength
        expect(result.gsub(/\s+/, "")).to eq(
          '

<thwidth="50">	<thwidth="100%"data-kind="Warning">2Warnings

:warning:	<tddata-sticky="false">amarkdown<ahref="https://github.com/danger/danger">linktodanger

:warning:	<tddata-sticky="false">secondwarning

<palign="right"data-meta="generated_by_danger">Generatedby:no_entry_sign:<ahref="http://danger.systems/">danger
'
        )
        # rubocop:enable Metrics/LineLength
      End

@DanToml - I spotted that CP uses the wrong ruby string to make newlines, newlines:
  fail('Please include a CHANGELOG entry to credit yourself! \nYou can find it at [CHANGELOG.md](https://github.com/CocoaPods/CocoaPods/blob/master/CHANGELOG.md).', :sticky => false)

We'd have to do our own parsing here

@orta Good spot! 👍


Cut a 0.0.1 release of image-rs

image-rs is currently being used as part of the CCv0 branch of Kata Containers. However, we're importing main there, and this is very much error prone and not suitable for the moment when we start working on reproducible builds (a hard requirement for Confidential Containers).
With this in mind, please, let's cut a 0.0.1 release with a commit that's known to be working, and we can use that as part of Kata Containers.
Kata Containers link: https://github.com/kata-containers/kata-containers/blob/cfa3e1e9334dedeb3e28440e6b3c3d268fc2ea14/src/agent/Cargo.toml#L68

Another option, if a release is not yet welcome, would be pinning a specific commit of image-rs as part of Kata Conatiners' agent.

We can close this one as we're going to pin a specific commit on the Kata Containers' agent side.


feat: implement new architecture

TLDR
Update the user flow to interact with marginEngine directly
See #105 for detail
Todos

[x] Implementation
[x] Fix all test
[ ] Cleanup docs, comments and libraries
[ ] Update README
[ ] Add user flow

Codecov Report

Merging #108 (85221cf) into master (4577fcd) will decrease coverage by 0.90%.
The diff coverage is 74.02%.

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
- Coverage   30.62%   29.71%   -0.91%
==========================================
  Files          26       27       +1
  Lines        1032     1040       +8
  Branches      155      161       +6
==========================================
- Hits          316      309       -7
- Misses        691      696       +5
- Partials       25       35      +10

Impacted Files
Coverage Δ

script/Deploy.sol
0.00% <0.00%> (ø)

...core/engines/advanced-margin/AdvancedMarginLib.sol
0.00% <ø> (ø)

src/core/engines/full-margin/FullMarginLib.sol
0.00% <ø> (ø)

src/libraries/ProductIdUtil.sol
0.00% <0.00%> (ø)

src/test/shared/AdvancedFixture.t.sol
0.00% <0.00%> (ø)

src/test/shared/FullMarginFixture.t.sol
0.00% <0.00%> (ø)

src/core/engines/BaseEngine.sol
66.66% <66.66%> (ø)

src/core/Registry.sol
88.23% <71.42%> (-11.77%)
:arrow_down:

src/core/OptionToken.sol
84.61% <77.77%> (-15.39%)
:arrow_down:

src/core/engines/full-margin/FullMarginEngine.sol
73.23% <79.24%> (-21.89%)
:arrow_down:

... and 5 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Improve UI of timezone fields

The timezone fields are pretty unmanageable. There are a million timezones, and they mostly don't use names people are used to. (i.e. BST is in there as Europe/London, PST is in there as America/Los_Angeles, etc.) It could help to provide a link to an explanation of timezones / a more human-friendly list. Either Wikipedia or one of our own.
Better yet, we should allow lookup of timezones based on location. :-p Maybe in a modal popover.

Oh, also, Chosen doesn't let you search for London to find Europe/London. Though maybe that's a setting?


Swap Dark Mode to Light Mode  with a shortcut

Description of the new feature / enhancement
Hi,
I think that it would be great if we are able to configure a shortcut that allow us to swap between Dark Mode and Light Mode.
There is already utilities that allow that, but I think it is a great feature to have in Powertoys.
Currently to be able to swap between theses mode, we need to do Windows+I and manually change the settings.
I think it should be a native feature on Windows or at least having the possibility to swap the theme from the bottom right menus like the night mode.
Scenario when this would be used?

You are currently working on a project with your favorite IDE in dark mode, configured to be synchronized with your operating system.
You have completed your work, but now you need to document your code on an external interface that allows you to retain the font style copied from your IDE. (For example: Word, Powerpoint or Outlook)
To have a clear style that is more readable according to the format in which it is written, you need to switch from dark mode to light mode.
So you type your favorite shortcut (For example alt+ctrl+m)
You copy and paste your code
Now you go back to coding, but you want your dark mode, so you type the shortcut again.

Supporting information
No response

/dup #1331


feat(docker/container): gpu support for containers

Close #3143
Note: I added the GPU line in container details, but I find it redundant as we already have all the Nvidia related informations inside the ENV just above.

@pull-dog down

@pull-dog down

@huib-portainer I have no display errors...
Tested on

Linux (archlinux) + Firefox 84.0.2 (64-bit)
Linux (archlinux) + Chromium Version 87.0.4280.141 (Official Build) Arch Linux (64 bits)
Windows 10 + Firefox 85.0.1 (64 bits)
Windows 10 + Chrome 88.0.4324.150 (Official Build) (64 bits)

/azp run

@pull-dog down

Hello, thanks for the update for the GPU support. The new feature looks great and I can create containers running on NVIDIA GPU! But I have a concern about the "use specific GPUs" option: I have two GPUs on my server and when I tried to use a specific GPU (either by adding index[0, 1] or UUID[returned by nvidia-smi -L], I got this error massage: "cannot set both Count and DeviceIDs on device request".

However, running "docker run --gpus '"device=0"' nvidia/cuda:9.0-base nvidia-smi" from command line works fine. Thanks in advance.

Thanks for the feedback @ShijieYan !
I believe we'll need to continue our work on that piece and try to address the issues raised here:

In the comment above: https://github.com/portainer/portainer/pull/4791#issuecomment-783578981
In the following comment on the original issue: https://github.com/portainer/portainer/issues/3143#issuecomment-782979259

Thanks for the feedback @ShijieYan !
I believe we'll need to continue our work on that piece and try to address the issues raised here:

In the comment above: feat(docker/container): gpu support for containers #4791 (comment)
In the following comment on the original issue: Support for nvidia-container-toolkit and docker 19.03 #3143 (comment)

refer to https://github.com/moby/moby/blob/master/daemon/nvidia_linux.go#L50
diff --git a/app/docker/views/containers/create/createContainerController.js b/app/docker/views/containers/create/createContainerController.js
index 2cc41231..2f1a3b73 100644
--- a/app/docker/views/containers/create/createContainerController.js
+++ b/app/docker/views/containers/create/createContainerController.js
@@ -437,7 +437,7 @@ angular.module('portainer.docker').controller('CreateContainerController', [
       if (gpuOptions.useSpecific) {
         const gpuIds = _.map(gpuOptions.selectedGPUs, 'key');
         deviceRequest.DeviceIDs = gpuIds;
-        deviceRequest.Count = gpuIds.length;
+        deviceRequest.Count = 0;
       }
       const caps = _.map(gpuOptions.capabilities, 'name');
       // we only support a single set of capabilities for now

I apologize if this is mentioned or discussed somewhere else, but is there any sort of rough estimate for when this might see a release? I'm unable to use portainer until this is supported, and I would love to have a timeline so I can plan around that in the meantime. Anything works, doesn't have to be exact.

@mdlockyer it's still on our roadmap, although we might end up adding it to the Business Edition. And we haven't solidified the scope yet, which could range from a rudimentary approach that allows all gpus and all capabilities, or a more fine grained approach where the parameters can be specified. What would the requirements roughly be for your use case?

It would be enough to add an "enable GPU" option since currently containers that need GPU do not work at all.
Allowing this only in the Business Edition at all would not be great to say this gently...

@huib-portainer Thanks for the update.
I use docker to run dev containers on a deep learning server. I need to be able to quickly access various different versions of CUDA and other dependencies. For this reason, I have a number of different containers that are ready to spin up depending on what I'm working on or prototyping. I would really love to use Portainer for this. I've used in for other things at home and found it to be very helpful.
Regarding this being a Business Edition feature; I really hope that doesn't end up being the case. This is a pretty widely used feature of Docker at this point, and at least for my use case, there's no way I could justify getting a Business Edition plan just for this one feature. I have no need for all the additional offerings of BE.
As for capabilities, I just need to be able to select a device index/ID to a container. Essentially exposing the same functionality as Docker run's --gpus argument would do it.

It would be nice to be able to display occupied gpus in the container interface.


AWS authentication

Add checks that the AWS environment variables or config files are property set

All all AWS checks to specify a custom profile.

Implemented in 1.1.1 with commit 1c96317b8bb66746075258e05475a5cd93001571


Entire project directory is published to NPM (cli/tauri.js)

Describe the bug
All the source, config files, and even a tarball of the same are shipped to NPM.
To Reproduce
Steps to reproduce the behavior:

Run npm pack tauri, and see that the downloaded tarball is 8.5 MB and contains all the source, config files, and a tarball of the uncompiled source directory.

Expected behavior
Only compiled javascript and .d.ts files, license, and bin/ should be shipped.
Output
/home/projects/oss/ex4/test
⟩ npm pack tauri

### [full file listing elided...]

npm notice === Tarball Details ===
npm notice name:          tauri
npm notice version:       0.12.0
npm notice filename:      tauri-0.12.0.tgz
npm notice package size:  8.6 MB
npm notice unpacked size: 13.4 MB
npm notice shasum:        0266befa124138ebb6eb5d8532371185eea37ae2
npm notice integrity:     sha512-cF6o2KoFe4rdH[...]YqvdAkKAl7L4g==
npm notice total files:   170
npm notice
tauri-0.12.0.tgz

/home/projects/oss/ex4/test
⟩ tar -zxf ./tauri-0.12.0.tgz && du -sh ./package/*
224K	./package/api
52K	./package/api-src
284K	./package/app-icon.png
4.0K	./package/babel.config.js
40K	./package/bin
12K	./package/CHANGELOG.md
1.1M	./package/dist
4.0K	./package/jest.config.js
4.0K	./package/LICENSE
8.0K	./package/package.json
4.0K	./package/README.md
4.0K	./package/rollup.config.js
6.4M	./package/scripts
244K	./package/src
4.0M	./package/tauri-0.12.0.tgz
984K	./package/templates
4.0K	./package/tsconfig.api.json
4.0K	./package/tsconfig.json
4.0K	./package/webpack.config.js

I think that this is something that will have been revised in the forthcoming beta release. If not, please feel free to reopen.


chore: Add rebase github action

:tada: This PR is included in version 6.1.3 :tada:
The release is available on:

v6.1.3
GitHub release

Your semantic-release bot :package::rocket:


[SPARK-49279] Revise observers package

What changes were proposed in this pull request?
This PR aims to revise observers package.
Why are the changes needed?
To fix comments and simplify the code.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass the CIs and manual review.
Was this patch authored or co-authored using generative AI tooling?
No.

Thank you, Liang-Chi. Merged to main.


[CP] [ CLI ] Fix issue where 'dart build' would fail to find the 'dart' when run from PATH

Commit(s) to merge
https://dart-review.googlesource.com/c/sdk/+/376860
https://dart-review.googlesource.com/c/sdk/+/377542
Target
stable
Prepared changelist for beta/stable
https://dart-review.googlesource.com/c/sdk/+/381041
Issue Description
dart build was exiting with an error stating that the dart executable could not be found and dart devtools was throwing an exception when run from PATH.
What is the fix
Use the resolved SDK path determined by DartDev for locating snapshots to locate the path of the dart executable rather than relying solely on Platform.executable.
Why cherry-pick
This is impacting users of dart devtools as it is completely unusable when running dart from PATH, which is a common workflow.
Risk
Low
Issue link(s)
https://github.com/dart-lang/sdk/issues/56080, https://github.com/dart-lang/sdk/issues/56306, https://github.com/dart-lang/sdk/issues/56499
Extra Info
No response

lgtm

Landed, but not released yet.


Python: common method for overriding of libraries

Issue description
Most Python library expressions are in python-packages.nix and some are called using callPackage.
We applied makeOverridable to buildPythonPackage so we can override the Python packages. However, callPackage = pkgs.newScope self; also sets a override attribute which is applied later thus taking precedence. We should provide a common method for overriding Python libraries.
Possible methods I can think of:

Define callPackage without providing a override attribute. The override attribute should now correspond to the one added to buildPythonPackage by makeOverridable.
Add a buildPythonPackage.overridePythonPackage method.

I suppose the first would be easiest and the second the most robust.

It would by the way be nice if makeOverridable would allow you to set the attribute used for overriding.

cc @copumpkin @lsix @Mic92 @abbradar
See also the implementation at https://github.com/NixOS/nixpkgs/issues/24155. Seems to work for me.

This went in already a long time ago.


Add StdoutSink to sinks interface

This PR adds a new Sink that prints raw JSON messages to stdout.
By default, the EFK stack does not support indexing of glog formatted messages output from containers.  By logging the raw JSON, EFK is able to index all of the Event object properties.

Going to merge now and double check prometheus metric in a bit.


TypeError: Cannot read property 'addEventListener' of null

When submitting a solutino to https://projectlovelace.net/problems/habitable-exoplanets/, the spinner is stuck forever, and the console displays the error "TypeError: Cannot read property 'addEventListener' of null". The debugger points to this part of the code, where dropdown contains null:
  <script>
    var dropdown = document.querySelector('.dropdown');
    dropdown.addEventListener('click', function(event) {
      event.stopPropagation();
      dropdown.classList.toggle('is-active');
    });
  </script>

Indeed, I can't find any node with the ".dropdown" class.
Firefox 87.0 in Ubuntu 21.04 but I get the same error in Chromium.

Looks like the TypeError is on other problems too and appears in the console before any code is submitted even.
It also looks like the stuck spinner is caused due to a 500 error possibly as another error pops up in the console

And surely enough the response it's trying to parse is an html template with the title HTTP 500 Internal server error | Project Lovelace
We'll need to work on a fix for that


Support for "--platform" tag to prevent multi-arch FROM image architecture mismatch

Hello,
I manage a repo with +/- 55 addons with local builds, and wanted to convert them to automatic builds. The github actions works very well and was easily implemented in my repo, thanks very much!
There is however one case of failure : addons builds succeed when building from multi-arch base images, but use a wrong arch which result in a "[FATAL tini (*)] exec // failed: Exec format error".
By looking at docker documentation, it seems the "--platform" tag is designed to avoid this.

“–platform” tag : The optional --platform flag can be used to specify the platform of the image in case FROM references a multi-platform image. For example, linux/amd64, linux/arm64, or windows/amd64. By default, the target platform of the build request is used. Global build arguments can be used in the value of this flag, for example automatic platform ARGs allow you to force a stage to native build platform (--platform=$BUILDPLATFORM), and use it to cross-compile to the target platform inside the stage. (Dockerfile reference | Docker Documentation)

I have tried to implement it in my build.json to reference a specific arch when pulling the image, but it fails due to REGEX validation.
Can't parse /data/addons/git/db21ed7f/addons_updater/build.json: does not match regular expression ^([a-zA-Z\-\.:\d{}]+/)*?([\-\w{}]+)/([\-\w{}]+)(:[\.\-\w{}]+)?$ for dictionary value @ data['build_from']['aarch64']. Got '--platform=linux/arm64 homeassistant/aarch64-base-python:3.9-alpine3.13'

Request : I believe that a solution would be to modify the REGEX to allow the plaform tag ; but a second solution could be to automatically support the "--platform" tag in the builder... It would make multi-arch source images work; and would make no impact on single-arch source images
Gain : this would greatly help to help additional users move from locally built to images-distributed addons.
Thanks very much for your consideration. Development is not my core expertise (and neither my full time work) so please bare with me if I was naive and misinterpreted the solution.

addons builds succeed when building from multi-arch base images

We don't publish multi-arch base images?
 I would recommend not using a single image that contains multi-arch for that reason. I will lead to troubles (I've painfully learned that as well).

Thanks very much for your quick answer. To facilitate my addons updates, I usually base myself on pre-existing images from Dockerhub, then add through my Dockerfile the Bashio & Tempio scripts to allow more relevant HA scripts. This means the majority are not based on your excellent single-arch base images, but either on single-arch images from Linuxserver, or on multi-arch images from other providers...
Actually it seems to work great for local builds (I've tested on armv7, aarch64 & amd64), but the github action workflow script fails to provide the right arch.
I understand though that it is not an optimal way of working... Just the only one at my disposal as I've got a full time job and am maintaining my addons + supporting their on the side so I was trying to base my work on preexisting images as much as possible...

Actually it seems to work great for local builds (I've tested on armv7, aarch64 & amd64), but the github action workflow script fails to provide the right arch.

But will go south on armhf/armv6 for example.

Indeed, I've removed armhf from my builds... Well then thanks I'll close the issue then and keep as is :)
I take this opportunity of having your ear to say that I am a big fan of your work. Thanks so much for everything you've developed and your continued support.
BTW I know it's not good manners to ask another question in a same thread but as it will be closed soon :) how can we upgrade from locally build addons to image based addons? Even for my addons based on HA bas arch images, if I try to update a locally built addon with a new version that has the image tag in its config.json, the supervisor seems to still try to build locally according to its log. When the local build is completed, the addon however doesn't start and fails with an "image not found" with the name of the image in the config.json. My understanding is that the addon doesn't see at update that the new version is based on an image so it doesn't download it. Or am I missing something? Thanks again very much :)

I take this opportunity of having your ear to say that I am a big fan of your work. Thanks so much for everything you've developed and your continued support.

❤

how can we upgrade from locally build addons to image based addons?

When the image tag is present in the add-on configuration, it is picked up.

the supervisor seems to still try to build locally according to its log.

I'm not aware of such an issue, maybe file a report @ Supervisor instead?

Thanks! I'll double check then file on supervisor with logs. Have a great day


Golang support

Like this theme!
It has almost the same color as my Goland Darcula when syntax matches. But many can be improved:

package name
package method/function/variable
type

Like this theme!
It has almost the same color as my Goland Darcula when syntax matches. But many can be improved:

package name
package method/function/variable
type

It will be very helpful if you post screenshots for your GoLand IDE to compare the colors and make some edits because I do not have GoLand IDE to compare.

Take an example from here:
Comparison 1
GoLand:

VSCode:

Diff:

struct type

LanguageFeature
Language
File

package name

base64
fmt
filepath
jsoniter

package method

DecodeString
Sprintf
Unmarshal
Printf
Println
Ext
Errorf
ToLower

constant

languages

built-in function

panic
len

From my observation,  package method call shares the same color as function call.
err is a mystery to me, some err are highlighted in GoLand, others are not.
Comparison 2
Goland:

VSCode:

Diff:

package name

processor

constant

type

stateType
machine

receiver

m in resetState method

character

 , \t, \n, \r at line 36

Thank you for your contribution.
Since VSCode relies on textmate scopes as a selector for theme which sometimes can not differentiate some syntax like Goland.
I cannot highlight these syntaxes:

struct type
constant
type
receiver

I downloaded GoLand and from my observation, it uses some machine learning for highlighting.
However, these elements are improved:

package name
package method
built-in function
operators
character

and a new version released, you can update the theme now ;)


[beta] Support of message update endpoint

Added new message update endpoint to MessageService class.

Mentioned in markdown doc
Simple demo created (MessageUpdateExampleMain.java). I'd like make it a bit more fun so I'll modify it before merging this PR

@symphony-mariacristina - Should we maybe also mention that we cannot update a message containing attachments?

Good point, we have different solutions:

throw an error if Message#attachments is not empty
create another Message that does not contain attachments and previews fields
just mention that in Javadoc

I personally prefer solution 3, as we plan to support attachments in a near future, it will avoid us to modify the BDK once message update works with attachments.

I personally prefer solution 3, as we plan to support attachments in a near future, it will avoid us to modify the BDK once message update works with attachments.

yes, I agree with point 3. The API is marked as experimental and will be changed anyway to support attachments.


sqlc generate runs into nil pointer dereference

My db.sql:
CREATE TABLE authors (
  id   BIGSERIAL PRIMARY KEY,
  name text      NOT NULL,
  bio  text
);

-- ...

My sqlc.json:
{
  "version": "1",
  "packages": [
    {
      "name": "db",
      "emit_json_tags": true,
      "emit_prepared_queries": false,
      "path": "db",
      "queries": "./db.sql",
      "schema": "./db.sql"
    }
  ]
}

Running sqlc generate causes a panic:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x88 pc=0x6b9249]

goroutine 1 [running]:
github.com/kyleconroy/sqlc/internal/dinosql.ParseQueries(0xc00012ffb0, 0xe038f1, 0x1, 0xc000116300, 0x1, 0x4, 0x0, 0x0, 0x0, 0x0, ...)
        /home/runner/work/sqlc/sqlc/internal/dinosql/parser.go:226 +0x689
github.com/kyleconroy/sqlc/internal/cmd.glob..func4(0xdfaea0, 0xe2d4d8, 0x0, 0x0)
        /home/runner/work/sqlc/sqlc/internal/cmd/cmd.go:156 +0x9aa
github.com/spf13/cobra.(*Command).execute(0xdfaea0, 0xe2d4d8, 0x0, 0x0, 0xdfaea0, 0xe2d4d8)
...

The eror occurs here - parser.go:221:
query, err := parseQuery(c, stmt, source)
if err != nil {
	merr.Add(filename, source, location(stmt), err)
	continue
}
query.Filename = filepath.Base(filename)

The code assumes that after checking for err != nil it is safe to access query. This is not correct, since parseQuery may also return nil for both the query and the error - parser.go:384:
default:
	return nil, nil

The other cases do not match because the statement seems to be an instance of pg_query.CreateStmt.
I'm not sure if the actual problem is that the statement's type does not quite match, or that there is no error indicating that the statement has an 'unexpected' type, or even that the returned query is nil.
In any case, the nil pointer dereference should be fixed IMO.

Got same error. It causing if path for migrations and queries is same.

It causing if path for migrations and queries is same.

So it seems - fixed with https://github.com/kyleconroy/sqlc/pull/179

Fixed by #180


Fixed typo

Description
There was a small typo in the docs

Thanks @franga2000!
Merging with failing pre-commit pipeline, this is due to an isort/poetry issue that's solved in master, and I'm happy that this PR doesn't affect linting.


WRQ-5672: Support m3u8 video format for enact video player

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
As a supplement for https://github.com/enactjs/samples/pull/216 PR,
sandstone/pattern-video-player-custom/README.md has to be updated.
Resolution
In README.md file, I added more sentences which explains how we support m3u8 video in our app.
Additional Considerations
Links
WRQ-5672
Comments
Enact-DCO-1.0-Signed-off-by: Jiye Kim<EMAIL_ADDRESS>

I think it would be better to update the resolution of the PR description
In README.md file, I added more explanations of how to support the m3u8 format with enact VideoPlayer while developing the app.

Update README in the https://github.com/enactjs/samples/pull/222


Study data should be stored in the AWS sub account of their project

if all study data is stored on Main account it defeats the all purpose of having subaccounts for billing, compliance, etc...

Maybe to unpack this a little bit. You are requesting study data to always be stored in the AWS sub account of their project? Or the ability to put the study data in sub accounts?
Study data and Projects are currently independent in service workbench. If study data is put in a sub account, then it’ll only be accessible from projects within that sub account. Changing the current functionality that there is only organizational study data.

Hi, it would depends on the type of users
For most users YES I would like to enforce that the study data MUST stay in the same sub account as the project
For some advance users then I would be ok to leave them the choice.
In biomedical research when we handle patient data we must control where data is stored.   Today all data is stored in Master account and that's not ok. I know that you are working right now on enabeling to bring your own bucket. how will this work?
Thanks!

Hi Paul, for bring you own bucket, the admin would be able to add any AWS account (aka. data source account) which contains the study files/folders in S3 bucket(s). A CfN template would be generated for the data source account owner to create a Cloudformation stack with (this allows SWB to access buckets and studies in that account). Once the stack creation is completed, the data source account and studies would be displayed with their reachability status.
For a quick first glimpse, please take a look at the screenshots in this PR description.

Hey Paul, would the bring you own bucket feature resolve your concern?

I hope! Let's see when it's released! March 8th, right? Thanks

With the release 🚀 of BYOB v2.1.0 this can now be resolved


Relax active support dependency

Depends on #1 being merged first.

@sobrinho were you able to assert that specs pass in a Rails 5 env?

@oehlschl yes, the specs are passing:
$ grep activesupport Gemfile.lock
      activesupport (>= 4.0, < 6)
    activesupport (5.1.3)

$ bundle exec rake test
MiniTest::Unit::TestCase is now Minitest::Test. From /Users/sobrinho/Developer/ebayr/test/ebayr/record_test.rb:6:in `<module:Ebayr>'
/Users/sobrinho/Developer/ebayr/lib/ebayr/request.rb:84: warning: shadowing outer local variable - http
/Users/sobrinho/Developer/ebayr/lib/ebayr/request.rb:114: warning: mismatched indentations at 'end' with 'case' at 111
/Users/sobrinho/Developer/ebayr/lib/ebayr.rb:145: warning: File.exists? is a deprecated name, use File.exist? instead
/Users/sobrinho/Developer/ebayr/lib/ebayr.rb:145: warning: File.exists? is a deprecated name, use File.exist? instead
/Users/sobrinho/Developer/ebayr/lib/ebayr.rb:145: warning: File.exists? is a deprecated name, use File.exist? instead
/Users/sobrinho/Developer/ebayr/lib/ebayr.rb:145: warning: File.exists? is a deprecated name, use File.exist? instead
Run options: --seed 41926

# Running:

..........................

Finished in 0.401172s, 64.8101 runs/s, 291.6455 assertions/s.

26 runs, 117 assertions, 0 failures, 0 errors, 0 skips

I tested against 4, 5.0 and 5.1.

To be clear, I tested against active support 4, 5.0 and 5.1, not rails because the gem doesn't depend on rails itself, just the AS.


feat(popover): export popover

Popover module is added to the base branch, but it's not exported, so it's not possible to use it in anything. So I added it to the entry so it will get exported.

Current coverage is 85.42% (diff: 100%)

Merging #1411 into development will not change coverage

@@           development      #1411   diff @@
=============================================
  Files               76         76
  Lines             2140       2140
  Methods             20         20
  Messages             0          0
  Branches           285        285
=============================================
  Hits              1828       1828
  Misses             213        213
  Partials            99         99

Powered by Codecov. Last update 92f82c0...749d1db

thanks for PR! same changes with https://github.com/valor-software/ng2-bootstrap/pull/1417

http://valor-software.com/ng2-bootstrap/#/popover

here is docs


Fix crush single message inside long message

Without this check we can crash here https://github.com/UAVCAN/libcanard/blob/legacy-v0/canard.c#L340-L357
This happen because when processing single frame transfer we check that state->buffer_blocks == NULL
But if inside multi frame transfer we receive  single frame transfer this assert can happen.
This problem happened in real environment.

I am not sure but this maybe applied also to v1. I found similar check here https://github.com/UAVCAN/libcanard/blob/master/libcanard/canard.c#L835

Looks good to me. @a1ien can you please add a test case to reproduce this error? (Let me know if you need assistance with that).
I'm still looking into the differences with the v1 code to see if we need to implement the same fix on v1 as well.

Add test. But I am not sure how correct is it.

ping @coderkalyan

Oh, sorry. Let me take a look.


Mock file pages

For example: http://calaccess.californiacivicdata.org/files/cvr_so_cd/

Gonne close this and move to #66 to keep our stuff in one place


Fix missing newline when compiling only with 'STATS_EXTRA'

Codecov Report

Merging #334 (969ea06) into main (567153c) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #334   +/-   ##
=========================================
  Coverage   97.388%   97.388%
=========================================
  Files           59        59
  Lines         3292      3292
=========================================
  Hits          3206      3206
  Misses          86        86

Impacted Files
Coverage Δ

src/adiar/statistics.cpp
85.714% <ø> (ø)

Continue to review full report at Codecov.

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


Outdated docs for struct Listening

Hey, the docs for struct listening are out of date. The stop function now works, as the linked issue is closed as fixed.

Fixed on master... sorry!


implemented setEncoding()

I implemented setEncoding(), as ReadableStreams have to be implemented.
Just delegating this instruction to "this.source"

Thanks


Ghidra dosen't work with directories that have spaces in them

There is an issue where ghidra won't launch when launching in a directory with a space in the directory name.
This is easy to reproduce.

download and extract ghidra
copy ghidra to c:\program files or any other directory with a space
run ghidra
and it won't launch!

Duplicate of #3201


Update which RPC tests are being run

What was wrong?
Trying to address #1755
How was it fixed?
Do not filter for GeneralStateTests when running the RPC tests.
To-Do

[ ] Clean up commit history

[ ] Add entry to the release notes

Cute Animal Picture

@carver @pipermerriam I believe this is the fix to #1755 (most rpc tests just running a single test...for months :sweat_smile: )


Issue with Map.

Actual Behaviour
Map is showing nothing for googlePlay flavor of the app.
Expected Behaviour
Map should show the location of the venue.
LogCat for the issue

Would you like to work on the issue?
Yes,
We need to update the Maps API key.
But that will link the app to my account on Google.
@the-dagger what should I do?

@murad I opened the same issue a few days back. It was working fine according to them. See #833 .

@free4murad @rohanagarwal94 It was a bad practice to have exposed the Google Maps API key in the code, so I had deleted the project. Now you can create your own key and add it to google_play.xml to get maps working

Sure


TensorFlow should support importing submodules, like a normal Python package

System information

TensorFlow version (you are using): 2.x
Are you willing to contribute it (Yes/No): No

Describe the feature and the current behavior/state.
TensorFlow 1.x supports importing submodules, e.g., import tensorflow.io
On TensorFlow 2.x this raises ModuleNotFoundError: No module named 'tensorflow.io'. Instead you have to import tensorflow, at which point the io module is directly available. This is both misleading and highly non-standard for Python.
If you not want to support direct importing of submodules, then perhaps they shouldn't be called modules in the online docs?
https://www.tensorflow.org/api_docs/python/tf/io
Will this change the current api? How? Yes, but it would be fully backwards compatible.
Who will benefit with this feature? Anyone who expects TensorFlow to work like a normal Python package.

In general, in Python if you can do
import foo
foo.bar.baz.foobar

that doesn't imply you can also do
import foo.bar.baz as m
m.foobar

It is not just something in Python that TensorFlow breaks, it's just the way things work.
In fact, to separate public and private APIs it is actually better to not allow both types of imports.

It's true that the former behavior does not strictly imply the later, but in practice it essentially always does for modules. If something is called the foo.bar.baz module, then it is expected for import foo.bar.baz to work. I cannot think of any exceptions to this rule, besides TensorFlow.
There are other ways to hide internal implementation details. For example, it is common to use undocumented private modules (name starting with _) for implementations, and expose only intentionally public functions/classes in documented APIs.

I agree with you. There is work on simplifying this structure, see https://github.com/tensorflow/community/pull/182

Closing this issue since we have other thread capturing this. Thanks!