shaka.util.StringUtils.fromCharCode throws call stack limit error for low end device

Hi team,
I am running shaka player (v2.4.3) on low spec set top box and I got a runtime exception for call stack limit reached when parsing the manifest. I was able to go around it changing the max cycle iteration value in shaka.util.StringUtils.fromCharCode from 16k to 8k.
I don't want to maintain a separate shaka player just for this change. I wonder what will be the proper solution here so can we include this change in shaka? Don't think exposing config param is worth it ...
There is one more thing that I want to ask. VOD playback (DASH + Widevine)  look stable but we have issues with Live playback where the device crash because it reaches its memory limits. Is there a way to optimize memory usage?
Thank you in advance!

You could certainly reduce it to 8k in a PR, with a comment explaining why that is the limit.  That seems like a reasonable solution to me.  Would you be willing to send us a PR for that?
As for the memory usage in live, it depends on where that memory is going.
You could try configuring the player with lower values for bufferingGoal, bufferBehind, and rebufferingGoal.  That could reduce memory used by the media stack to store segments and frames and such.
If that doesn't help, we'll need to dig into a memory profile to find out what the issue is.  That would require access to the device and a debugger.  If this is a device we can get off the shelf somewhere, you can give us details and we can look into adding one to our lab environment.  If not, then you may have to help us out by doing some profiling on our behalf.
But hopefully the buffering config will make a difference.  Try that first and let us know how it goes.

Thanks for the quick reply,
I will make the PR after I try bufferingGoal, bufferBehind, and rebufferingGoal. Will check with the box provider how we can get memory profile. Please can you provide an email that I can contact you for the eventual device delivery.

Hello,
I have decided to use this conversation since we are seeing a regression in 2.5.3 regarding the video playback on Samsung Tizen TV devices.
We are using HLS (mux.js 5.1.3), Live Broadcast, with 4 seconds segment duration. We see that even Premium devices would crash after some time of playing.
We are conducting tests and collecting more information but in 2.4.6 we did not see any application crashes on Premium devices:
Test Results for 2.5.3 (In Progress)

Test Results for 2.4.6

Let me know if you need more information about device models.
Regards,
Nicolas

@NicolasSiver, please file a new issue.  This one is closed, and your comments do not appear to have anything to do with this issue's topic.
The issue templates will guide you on providing the information we need to address your issue.  Please answer all questions in the template.  Thanks!

@joeyparrish sure. Created separate ticket - https://github.com/google/shaka-player/issues/2059

Thanks!


MISC: use bls12 prefix instead of bls

Thanks Danny! I have pushed a commit fixing this (there were one mention in the README that was overlooked). Was planning to do it yesterday but lost track of time with other work 😅 Thanks again anyway!


Gracefully handle resource types with no instances in the FHIR server.

In the list of resource types that are configured to be fetched, if a type has no resources in the FHIR server, we get an exception in the controller code when it tries to create a table (if createHiveResourceTables is set). This case should be gracefully handled. Note the actual pipeline handles this case and is only a problem in the controller.

We ran into this as well and would love it to have it skip resource types that have zero records.


Dotnet --version doesn't seem to work

Description
For some reason, the option --version doesn't exists. When executing dotnet --version, it produces the following output:
The command could not be loaded, possibly because:
  * You intended to execute a .NET application:
      The application 'info' does not exist.
  * You intended to execute a .NET SDK command:
      No .NET SDKs were found.

Download a .NET SDK:
https://aka.ms/dotnet-download

Learn about SDK resolution:
https://aka.ms/dotnet/sdk-not-found

Configuration
I'm running on: Ubuntu 22.04
Here is the output of dotnet --info:

global.json file:
  Not found

Host:
  Version:      6.0.8
  Architecture: x64
  Commit:       55fb7ef977

.NET SDKs installed:
  No SDKs were found.

.NET runtimes installed:
  Microsoft.AspNetCore.App 6.0.8 [/usr/lib/dotnet/dotnet6-6.0.108/shared/Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 6.0.8 [/usr/lib/dotnet/dotnet6-6.0.108/shared/Microsoft.NETCore.App]

Download .NET:
  https://aka.ms/dotnet-download

Learn about .NET Runtimes and SDKs:
  https://aka.ms/dotnet/runtimes-sdk-info

Regression?
dotnet --version seemed to work in the past. I'm not 100% sure, but dotnet might have updated since I last used the command. I don't think I've uninstalled anything relating to dotnet since then.
Additional Info
As a note, I performed the following commands to previously install dotnet:
    ## Microsoft package signing key.
    curl -O https://packages.microsoft.com/config/"$1"/"$2"/packages-microsoft-prod.deb
    sudo dpkg -i packages-microsoft-prod.deb
    sudo rm -f packages-microsoft-prod.deb

    ## Install the SDK.
    sudo apt-get update
    sudo apt-get install -y apt-transport-https \
        && sudo apt-get update \
        && sudo apt-get install -y dotnet-sdk-6.0

Resolved using https://github.com/dotnet/core/issues/7699


[Feature Request ] Song Lyrics

Is it possible to add a song lyrics options in Apple Music Electron?
Example:

I could definitely look into it. I don't know about live lyrics but I could definitely add a button that opens a genius lyrics window or something.

i concerned that apple is gonna implement it at some point

I wonder if it would be possible to pull time-synced lyrics from Musixmatch or even .lrc files from Deezer and use Genius as a fallback for non-synced lyrics.

It is possible to query time-synced lyrics from MusicKit: MusicKit.getInstance().api.lyric(songId)

👀
https://genius.com/developers

Lyrics from AM is from Musixmatch, it's possible to parse .lrc from them however user need to provide API keys to access it. (and it's would be illegal to distribute the API keys)
A better choice would be kugou:
"http://music.163.com/api/search/get?s="+songname+"_"+artistname+"&limit=1&type=1&offset=0" to get the songid
"http://music.163.com/api/song/media?id=" + x to get the .lrc from songid

This looks surprisingly good for a start. Good Job!
I have one question:
Does the lyric get selected or did you do that?


"make env" is broken

After running ./scripts/setup/, then make env, I see:
Preparing transaction: done
Executing transaction: done
/media/tim/GIT/clean/CFU-Playground/env/downloads/Miniconda3-latest-Linux-x86_64.sh: 438: [[: not found

That line is
    if [[ "${env_name}" == "*" ]]; then

I'm seeing this both locally with Ubuntu 20.04 and also in Github Actions (also 20.04?) -- see the failing tests in #751.

FYI @PiotrZierhoffer @proppy

Not sure if this is expected?
tim@tim-ubuntu:/media/tim/GIT/clean/CFU-Playground$ ls env/conda/envs/
total 12
4 '*'/  4  ./  4  ../

It looks like the Miniconda3-latest-Linux-x86_64.sh installer has just started using "bashisms" in their shell script?
The instructions at https://docs.conda.io/projects/conda/en/latest/user-guide/install/linux.html do say bash Miniconda3-latest-Linux-x86_64.sh

just started with a fresh ubuntu installation and git cloned the current repo. still getting the error
/CFU-Playground/env/downloads/Miniconda3-latest-Linux-x86_64.sh: 438: [[: not found
shouldn't the bash fix be merged already? if not, where is Miniconda3-latest-Linux-x86_64.sh being called in order to add the bash command manually?

@haaslukas looks like the submodule in https://github.com/SymbiFlow/make-env/tree/59adb0f248cc4a5d764b6b06224bf6adea7fa36e would need to be updated to point to a rev that includes https://github.com/SymbiFlow/make-env/pull/31
as a workaround you should be able to update the submodule manually with:
git submodule update --remote


Automated cherry pick of #1599: fix max session login issue

Cherry pick of #1599 on release/2.11.0.
#1599: fix max session login issue

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11.0@fc0eca3). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.11.0   #1600   +/-   ##
================================================
  Coverage                  ?   7.67%
================================================
  Files                     ?     411
  Lines                     ?   63033
  Branches                  ?       0
================================================
  Hits                      ?    4837
  Misses                    ?   57786
  Partials                  ?     410

Flag
Coverage Δ

#aFlag
7.67% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/storagecachedimages.go
0.36% <0%> (ø)

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 fc0eca3...65a6896. Read the comment docs.

/lgtm
/approve


HHH-11294: check for null interceptor in PersistenceUtilHelper.isInitialized

https://hibernate.atlassian.net/browse/HHH-11294

@barreiro thanks for pointing that out, I've updated the PR with that change.

Applied upstream. Thanks.


add vnua domain

add Vietnam National University of Agriculture domain

@manhhdtop Pull request merged. Thank you!


FullNat

Under the fullnat mode, LVS replaces the source ip and port with its ip and port, replaces des ip and port with RS's ip and port. In order not to lose client ip, fullnat add an option to TCP package to store client ip. When realserver gets the package, it gets the client ip through toa module of kernel. It seems to meet the requirement? @zoumo
I will set up fullnat with kubernetes to test whether it can work in a couple of days.

It's workable in theory. But real server can't get real src ip (See taobao/toa).

It puzzles me... Do you mean The fullnat alone can't ensure that real servers get the real ip, but if we do what just taobao/toa does, real servers can get the real src ip?

I means the workflow is workable. But the main concern is how to get real source ip in real server.
FYI:

For TCP, toa module can help you to get source ip from TCP packets. You just need to install it to kernels of real servers.
For UDP, https://github.com/yubo/ip_vs_ca may help you.

Got it. I will try on it.

:-D


Improve Sail code formatting and syntax highlighting

The Sail code in the RISC-V manuals suffers from poor readability and alignment issues. I discovered a repository (https://github.com/Alasdair/asciidoctor-sail) that offers a solution to enhance the presentation of Sail code by aligning it and highlighting keywords with colors. This approach significantly improves the clarity of the Sail code in the manual.
To implement this, it's necessary to add a folder to the RISC-V ISA manual repository with the Ruby files that will handle the formatting. While not all of these files may be useful immediately, for now in simple sail codes only highlighter.rb file is used, they could be valuable for future enhancements.
I noticed that @wmat has previously interacted with this repository, which raises the question of why this solution hasn't been added yet.

And this is how the sail code will look like in the manual.
To implement this improvement, it will be necessary to include these files in the "REQUIRES" section of the Makefile in the RISC-V ISA Manual.

I'm confused.  Is this PR adding the asciidoctor-sail extension that Alasdair wrote directly to the docs-resources?


Minecraft FabricMC

Dears,  Developers!
My issue is about minecraft launching other launcher example forge. I want to run FabricMC but is not supportable around CmlLib module.

3.2.0 support fabric


Large-scale refactor

Getting this to look like Gala.
What do you say we loop Adrian in on this and coordinate a shared repo. With not too much more work this can be nearly a gala clone.
Sorry for the stuff that says my name on it. I used https://github.com/scientific-python/cookie to set up a clean new repo and then copied the git history into it. We should fix the attribution, etc.
I also deleted all the notebooks. Might want to copy-paste them elsewhere before merging!
Also, this is a WIP. I used equinox to make this stateful, but I haven't finished all the stuff.
In particular, I haven't separated out the stream generation stuff into a separate class, like in gala.

I haven't looked at the code here, but I love the idea! I started a similar project (gala-jax) to collect some ideas I've been thinking about and just invited you. Would be great to collaborate over there on an evolution of gala!

If we're going to move this to gala-jax, then I'll close this PR and use my fork as a separate playground.


export/download tutorials as a pdf. feature request

selfish request, but the current layout can be cumbersome to travel
i personally always scrape documentations to pdf so i can either print them to highlight them or have them on my ipad where i can highlight them, study them etc
would be good if the whole of the documentation could be exported as a pdf keeping the order of all the fields since sitting in the computer and going link by link is cumbersome

I think it's a good idea to have a PDF version.
I wonder if it is finished now ?
What should I do to convert it?
Is there any command line can do it? like
$ make latex
$ pdflatex

Thanks！

In theory, you should be able to do:
python3 -m sphinx -M latexpdf source build -c .

To have the sphinx documentation generator output PDFs.  Unfortunately, I have yet to be able to make it work; I keep getting errors while Latex is running.  We'll leave this on the backburner to look at eventually, but if you find a solution in the meantime we'd be happy to review a PR.


Fix object property syntax in the readme example

I think this is correct syntax. It's being destructured with David Baszucki as the default name (e.g. it will set David Baszucki as the name if no name is provided in the props)


Request: Add edge based KWS to  the framework

It would be great to also have satellite KWS added to the project as with pytorch & pytorch audio it seems to enforce some vendor considerations.
I would be really interested in a common MFCC method for Brain and Edge or maybe 'Shelf' is a better description for simple devices that connect to a brain to create solutions.
Having a MFCC source reduces Brain process but also makes an extremely low bandwidth audio representative stream.
I have been playing with the Google KWS from there research repo and the quantised TFL models can run on a little as 20% of a single core of a Pi3 which gives a lot of headroom for some interesting implementations.
I have my convenience repo https://github.com/StuartIanNaylor/google-kws which is all google work with just some quick scripts to test inference.
It would be really great to have, something similar, Brain addons as my current interest with the above is to  use simple multi-mic unidirectional distributed mics and use KW hit to position the near/far stream and maybe try blind source separation as a relatively low load feed to the brain.
But also in network mic situations MFCC as a stream is also another consideration but having identical processing is essential, but my tests the methods of torchaudio seem strangely locked to intel_mkl (maybe that has changed?)
The dataset set side of Google-kws could do with some work but the models for TFL are extremely lite and not saying use it but as an example its exactly what I would envisage if I was to look at distributed edge devices to connect to a brain.
The google methods use 2 MFCC methods a DCT method that should run with any inference engine and then there TF FFT based one.

Maybe you could put this on our Discourse and try to foster some discussion from there. I am not super familiar with this topic though.


Remove poetry version limit for new versions

Related to #16

The version restriction is deliberately included to prevent failure (including silent failure) if a Poetry release breaks the internal structures that this plugin depends upon. Ideally this would be handled in some automated way (or through compatibility guarantees in Poetry itself), but for now it'll have to remain as-is.


Add reset function

The SBI specification provides a shutdown function, but no reset function. Reset support has been added recently to the QEMU "RISC-V VirtIO Board" machine. Would it be possible to add such a function to the SBI specifications? It's not clear to me what process has to be followed to update the specifications.

We are in the process of sending a patch to add the reset function. I will
send the PR in couple of days.
On Sun, Oct 6, 2019 at 5:16 AM aurel32<EMAIL_ADDRESS>wrote:

The SBI specification provides a shutdown function, but no reset function.
Reset support has been added recently to the QEMU "RISC-V VirtIO Board"
machine. Would it be possible to add such a function to the SBI
specifications? It's not clear to me what process has to be followed to
update the specifications.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-sbi-doc/issues/26?email_source=notifications&email_token=AA6CNH64HUKQU64CQTRQVDTQNHJIVA5CNFSM4I53XWV2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HP4SZTQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6CNHYVMFCTUGXYEBOGNFLQNHJIVANCNFSM4I53XWVQ
.

--
Regards,
Atish

@aurel32 : Reset function added
https://github.com/riscv/riscv-sbi-doc/pull/27

@atishp04 Thanks a lot, this looks all good (I only checked the shutdown/reboot part).

Looks like this got removed before that PR was merged?

We had intentionally kept reset extension out of SBI v0.2.
Now that SBI v0.2 is tagged and ready to be ratified, I will send SBI reset extension patch again for review.
Regards,
Anup

The reset extension is under discussion and there is a PR available #39 . Closing this issue.


create a third type of NodeFlvSession, one that sends the flv to a stream

Instead of a transcode of ffmpeg connecting back and playing, this allows a one to save an incoming flv to an arbitrary node stream.

Thank you for your advice.
There are several problems.

The app.js in the repository is a use case. There is only one project entry file "node_media_server.js". Therefore, other classes cannot be exposed.
This feature is not required by everyone, so a configurable option should be required.
The folder is not checked and created, may causing an exception
:8000/api/streams causes an exception

grr... I guess I messed up merging your commits in to fix the conflict. Let me know and I can close this PR and just make another


feat: fancy toggle switch

Adds a more stylized toggle switch than the default bootstrap one. I adapted the code from a tutorial.
⚠ Merge this after #62.
📸  Before
https://user-images.githubusercontent.com/10026790/152847768-eb5db0b3-eba6-4463-91dd-568d9157e084.mov
📸  After
https://user-images.githubusercontent.com/10026790/152847657-7e14b97e-9ce1-431a-a400-f880df59616b.mov

Keeping this a draft until #62 is merged.

Resolved some conflicts with master and addressed feedback. Will merge now as this is just a small thing anyway.


curl: (35) error:02FFF036:system library:func(4095):Connection reset by peer

Operating system and version:
nvm debug output:

nvm ls output:

How did you install nvm?

What steps did you perform?
What happened?
What did you expect to happen?
Is there anything in any of your profile files that modifies the PATH?

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

Please fill out the issue template.


Java PackedRTree inefficient streaming search

The streaming search re-sorts the entire Stack it's using after each insertion, which makes it rewrite the entire backing data structure. A PriorityQueue is designed to insert new items in the right order without as much overhead.

After some recollection and thinking I think that sort was added by mistake for some reason. I do not believe it is needed as the flatgeobuf index is guaranteed to be sorted already.
I've made https://github.com/flatgeobuf/flatgeobuf/pull/277 that removes the sort and replace Stack with Deque.

I'll try to get this out in a patch release later today and then do downstream patches.


weechat: Add version and meta to buildEnv package

Motivation for this change
Currently the output from nix search and similar tools are lacking important meta data
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: weechat
Partial log (click to expand)
copying path '/nix/store/r7d37i8dlw3rb6sdp6nz5xxwvbygv2rq-weechat-2.2' from 'https://cache.nixos.org'...
copying path '/nix/store/7b5agj8f7pgwihnabyilkjgvjpp2vp24-weechat-2.2-guile' from 'https://cache.nixos.org'...
copying path '/nix/store/2rdfxb7y5c5dh702lsdhmnbay7rc6i1i-weechat-2.2-ruby' from 'https://cache.nixos.org'...
copying path '/nix/store/sr1fvic96g9a3wdc4yh63fw3v61fgiyw-weechat-2.2-tcl' from 'https://cache.nixos.org'...
copying path '/nix/store/ki52hv6gf7720wn159hlh5pa0bwmyisz-weechat-plugins' from 'https://cache.nixos.org'...
building '/nix/store/gssm6ysd69hn28mixsq81v0l63qgpvxg-weechat-headless.drv'...
building '/nix/store/jr3rpv41bl9y93cqrw2lnh191c07z1cn-weechat.drv'...
building '/nix/store/1v2nms2ih07g36zy8hmz594gc9grw4wg-weechat-bin-env-2.2.drv'...
created 2 symlinks in user environment
/nix/store/8lkxwf68w3zcsjhxn2gbiq3fsjpykkw1-weechat-bin-env-2.2

Success on x86_64-darwin (full log)
Attempted: weechat
Partial log (click to expand)
patching script interpreter paths in /nix/store/9b09ymwjgg2jrgx4r6yvhylvyzi7fll3-weechat-2.2-lua
strip is /nix/store/df6k4mgdjxciy0f637lryp7c9ln7n1m3-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/cx7ji5mnarbmrd9qm1navashv3c3lbzd-weechat-2.2-python/lib
patching script interpreter paths in /nix/store/cx7ji5mnarbmrd9qm1navashv3c3lbzd-weechat-2.2-python
building '/nix/store/z9zi7k78xwx2n0pzb9qhg42pcjzlryyl-weechat-plugins.drv'...
building '/nix/store/3d5fjddpysjs5sjj91k61g27n2j9xxgz-weechat-headless.drv'...
building '/nix/store/jqawwvzqxf863wkr7vqfai10zi4qbpjb-weechat.drv'...
building '/nix/store/k6cdggrwkhm5yx4sszkfk13bj1yzsciv-weechat-bin-env-2.2.drv'...
created 2 symlinks in user environment
/nix/store/isxmw84qf3kw8kzfzn1lcn6i5r9rqj0b-weechat-bin-env-2.2

Success on aarch64-linux (full log)
Attempted: weechat
Partial log (click to expand)
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/1c3drc6cfx3w0xa4hai4ii3gnyb8fhf2-weechat-2.2-python/lib
patching script interpreter paths in /nix/store/1c3drc6cfx3w0xa4hai4ii3gnyb8fhf2-weechat-2.2-python
checking for references to /build in /nix/store/1c3drc6cfx3w0xa4hai4ii3gnyb8fhf2-weechat-2.2-python...
building '/nix/store/qbw49k6igzpsydnqffayybq2pjsi1wdz-weechat-plugins.drv'...
building '/nix/store/xs6rzb3q2d1gs04nfs5z5wwk0p185x6y-weechat-headless.drv'...
building '/nix/store/sqk93cr4z3b5wsq2f1yh2raxyq7dm8j2-weechat.drv'...
building '/nix/store/r3x6i7vvk9cs76plgs3wdr6aj5y36q9m-weechat-bin-env-2.2.drv'...
created 2 symlinks in user environment
/nix/store/0x4rc2jliykv076m0sddvsg6lilkk6z2-weechat-bin-env-2.2


Importing and Running React Spectrum

🐛 Bug Report
I am working on React/electron application (the React portion of the application was created using create-react-app) and followed the directions from the getting started page to run react-spectrum, but keep getting this error:
TypeError: __WEBPACK_IMPORTED_MODULE_1_react___default.a.createContext is not a function
I installed react-spectrum using npm and wrapped my App.js component with a Provider to test it out but it fails to run. Did I miss a step in the installation process?
🤔 Expected Behavior
The application should run without issue as I did not add any components only wrapped App.js with a provider
😯 Current Behavior
This error appears: TypeError: __WEBPACK_IMPORTED_MODULE_1_react___default.a.createContext is not a function
and application does not run
💁 Possible Solution

🔦 Context
I am trying to use react-spectrum to style my application
💻 Code Sample
import {Provider, defaultTheme} from '@adobe/react-spectrum';

export default class extends Component {
  constructor() {
    super();
  }
    render() {
        return (
           <Provider theme={defaultTheme}>
                 <Some JSX>
                 </SomeJSX>
            </Provider>
         );
    }
 }

🌍 Your Environment

Software
Version(s)

react-spectrum
v3

Browser
chrome

Operating System
windows

🧢 Your Company/Team

🕷 Tracking Issue (optional)

Figured it out just need to update react to he latest version


Dynamically generated route path does not work for snake case page names

If I create a new nuxt page called sign_up.vue, createRouter() in router.js from nuxt generates the following route:
routes: [
{
  path: "/sign:up",
  component: _1afcc2c0,
  name: "signup"
},

With this bug, I wouldn't be able to create another page called sign_in.vue as it collides with the dynamically generated route.
https://spiky-part.glitch.me is an example of this bug.

Duplicate of #2716

Thanks for noticing the duplicate. I'll go ahead and close mine


pull request for e2e test at 11/11/2016 00:15:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Will it work without GPU?

hey, recently I was trying to run this code on my computer. Sadly, I don't have any GPU. So I was wondering if this is still gonna work. If not, what changes should I do?

Thanks for your question.
If you don't have a  GPU, you can use a CPU instead of it with the command "THEANO_FLAGS="floatX=float32". But it will be very slow.

Wow, thanks for your reply. And this is a relief. I tried this command once, but it didn't have any responding for a long time. So I thought maybe I didn't use it right.


[Question] Does setSources(true) works with Cucumber tests ?

Hello,
I have a project with cucumber tests (features, steps and actions files) and I am trying to get the Sources tab within the traces file.
To which location should the environment variable PLAYWRIGHT_JAVA_SRC points to ?
How to set up multiple locations, could you give me an example ?

I have a project with cucumber tests (features, steps and actions files) and I am trying to get the Sources tab within the traces file.

I don't know how exactly cucumber works but I'd expect it to first translate Cucumber scripts to .java files which are compiled and run on a jvm. In that case you should be able to add the directory with the generated .java files to PLAYWRIGHT_JAVA_SRC and they will appear in the stack traces and sources tab. This will not give you original cucumber sources though only the generated java sources.

To which location should the environment variable PLAYWRIGHT_JAVA_SRC points to ?

It should point to the parent directory of the root package, see e.g. this test on our CI:
https://github.com/microsoft/playwright-java/blob/447578c5824688e5209dadbb233867ed0b20385c/.github/workflows/test.yml#L37-L41

How to set up multiple locations, could you give me an example ?

The value is a list of directories separated by File.pathSeparator, so e.g. on Linux:
PLAYWRIGHT_JAVA_SRC=/home/me/foo/java/src:/home/me/bar/java/src

Thank you very very much @yury-s
I was able to add the sources tab to the traces, working directory/src/test/java did the trick.
For adding multiple paths in windows we can use ; => linux is with :

Let me also update the docs to mention this.


Multiple components in single file

Is your feature request related to a problem? Please describe.
<AbellComponent name="Footer">
<template>
</template>
<style>
</style>
<script>
</script>
</AbellComponent>

<AbellComponent name="Navbar">
<template>
</template>
<style>
</style>
</AbellComponent>

Rough idea.

I don't plan to support a lot of new syntax in abell component starting v1. Will recommend using components from some existing framework like React or Solid for such usecases.


Hub to Hub secure Vwan hub traffic

Please include the caveate that secure hub to secure hub traffic does not pass through the firewalls. This page has a diagram that is misleading in support for that traffic path. While this is clear in our main page, this page which includes routing details and diagrams leads the customer to believe this is possible.

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

ID: 4cae3cbe-02bc-755b-792b-64923b32eb13
Version Independent ID: bef0a2cf-9567-c46b-04c1-7ed7476e788b
Content: Virtual WAN routing deep dive - Azure Virtual WAN
Content Source: articles/virtual-wan/routing-deep-dive.md
Service: virtual-wan
GitHub Login: @erjosito
Microsoft Alias: jomore

@JohnPoetzinger,
Thank you for the feedback. Can you please point out the scenario and diagram you are referring to above? it will us pinpointing the issue. Thanks.

Good point @JohnPoetzinger let me add that caveat

PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/217535

@JohnPoetzinger, the suggested changes were made.
Thank you @erjosito for making the PR.

@JohnPoetzinger. Can you please advise the following as I still find this misleading. If you have two secure hubs in different regions, does traffic still flow between the regional hubs(routers) just not via the azure firewall?

I would agree, upon reviewing the updated content you state that connecting 2 hubs is unsupported. This is not accurate. In the case of 2 secure hubs being connected hub to hub traffic will not be inspected by the firewall in either hub.

Thanks for that John. I don’t think I’ve seen this clearly defined in any document (but may have missed it)

The feature to correct this is in preview.

https://learn.microsoft.com/en-us/azure/virtual-wan/virtual-wan-global-transit-network-architecture
" Note
Inter-hub processing of traffic via firewall is currently not supported. Traffic between hubs will be routed to the proper branch within the secured virtual hub, however traffic will bypass the Azure Firewall in each hub."

@JohnPoetzinger. Fantastic!!! Thanks for that. That’s what I was looking for. I was aware of the preview feature 👍

@JohnPoetzinger It actually depends on how you configure routing. If you configure routing to go through the firewalls, traffic will be dropped and kitties will die. If you configure routing to bypass the firewalls, it will be pink unicorns and rainbows.

@erjosito Hi Jose. So in the following scenario can you please advise. Two secure hubs in different regions. In the same region, the requirement is for B-to-V and V-V traffic to go through the firewall. In the above configuration will inter-hub traffic (across hub regions) still be routed, albeit not via the firewall?

With the default configuration that you get out of the Firewall Manager, VHHV traffic will be dropped.


Block-mapped resample with the help of flox

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #xyz

[x] Tests for the changes have been added (for bug fixes / features)

[x] (If applicable) Documentation has been added / updated (for bug fixes / features)

[x] CHANGELOG.rst has been updated (with summary of main changes)

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added

What kind of change does this PR introduce?
Implements resample_map. This function is meant for all da.resample(...).map(...) calls. These, flox cannot improve automatically so we use some flox logic to help. The idea is to map the resample-map construct on each block in parallel. This is possible by first rechunking the array so that chunks boundary fit with resampling period boundaries (this is a flox function).
The main improvement should come from the fact that map_blocks hides much of the complexity to dask, so the resulting graph is much lighter.  I still have to better test the performance of this. My goal would be to have some short text in xclim's doc that highlights when the option is useful and when it is not. The option is activated through  set_options.
The current function works only when the input object is of the same type as the output one. So some functions couldn't be wrapped with this yet. The most important untouched code for the moment is the missing checks where I think this could help a lot.
Does this PR introduce a breaking change?
It should not. This is completely optional.
Other information:
In progress, I still need to prove the performance boost.
This depends on #1845 because I need all improvements for PC.

coverage: 89.447% (+0.01%) from 89.436%
when pulling d5d638a95058300ed9fb142d6e3d87cdf1fafb46 on resample-map
into 0644f3ad9ee68fed9061559daf3524742ab3510f on main.


feat(notfound): add PageNotFound component

Please read and mark the following check list before creating a pull request (check one with "x"):

[x] I read and followed the CONTRIBUTING.md guide.
[x] I read and followed the New Feature Checklist guide.

Short description of what this resolves:
Redirects users who hit an unknown page to a custom component
Fixes #1410

A couple of things I was not too sure about :

I've designed it so that only errors within /pages/* are captured by this. I think that for most single page applications, you probably don't want to catch 404's in a global manner (it's ambiguous if the user is not authenticated). I expect most people to have protected their pages directory with a guard of some sort.
Should this component be within nebular?
Should we have a direct link in the demo menu to show this page?

Hey @dizco, great addition!

yes, I agree, let's keep it for /pages/* only
probably not, as this is more of a page rather than component
yes, let's add a Pages section and this one as a sub-page. Just for demo purposes.

Also, I would suggest a bit of a refactoring - let's move the comonent from @theme/ to the pages/other (didn't think of a better folder name rather than other...) where it belongs better. As this component more of a 'page' component, refferenced in the routes.

@nnixaa what would you think of Miscellaneous instead of Other? Also thought of Extra.

Miscellaneous sounds good!

Choice of icons isn't my particular strenght, so please feel free to suggest something better if there is haha!

@dizco cool! Merging now.


Meta backup/restoration.

Output of restic version
0.16.4
What should restic do differently? Which functionality do you think we should add?
Backup/Restore meta, like date modified, and date created, etc.
What are you trying to do? What problem would this solve?
None
Did restic help you today? Did it make you happy in any way?
Yes, it allows me to backup files much more easier and faster. But it's a pity that restic can't store meta.

Please properly fill out the issue template and reopen the issue afterwards. There's no information at all what operating system you're using and what commands you've run, so it's not possible to investigate this issue. Restic already supports restoring file modification times.


Email Address Regex in Controller for Invite

Match submitted email addresses against a regex filter in the controller to eliminate invalid email addresses.

Already implemented.


Resize Docker Volume with ontap-nas and ontap-nas-economy

Describe the bug
Volume size cannot be updated to a new size.
Environment

Trident version: 23.01
Trident installation flags used:
Container runtime: Docker CE 20.10.16
Kubernetes version: N/A
Kubernetes orchestrator: N/A
Kubernetes enabled feature gates: N/A
OS: AmazonLinux2
NetApp backend types: AWS FSx

To Reproduce
Steps to reproduce the behavior:

Install docker yum -y install docker
Create Trident config file gp01.json

    {
          "version": 1,
          "storageDriverName": "ontap-nas-economy",
          "allowVolumeExpansion": true,
          "managementLIF": "<IP_ADDRESS>",
          "dataLIF": "<IP_ADDRESS>",
          "svm": "svm03",
          "clientCertificate": "adadsadsadadaad........",
          "clientPrivateKey": "LS0tLS1CRUd.........",
          "aggregate": "aggr1",
          "storagePrefix": "gp01_"
    }

docker plugin install docker plugin install --grant-all-permissions --alias gp01 netapp/trident-plugin:23.01 config=gp01.json
docker plugin ls

ID             NAME               DESCRIPTION                             ENABLED
410f33f4a3a5   gp01:latest   Trident - NetApp Docker Volume Plugin   true

Expected behavior

Create a docker volume docker volume create -d gp01 --name gp01_vol01 --opt size=10G
Update volume size docker volume create -d gp01 --name gp01_vol01 --opt size=50G
docker volume should be resized to 50G

Additional context
This is needed for some workloads where we need to run an application as a docker container and persist data in NFS share.

Hi @chaudhariatul,
Docker doesn't officially support expanding a volume via that CLI. The docker volume create command only supports creating the volume.
As a work around, you can expand your AWS FSx volume. For an NFS volume this is non-disruptive and the increased volume capacity will immediately be available for use.

@chaudhariatul Please let us know if this issue still exists with the newer versions of Trident. If not, please close the issue.


[$1000] Android - Copy icon is missing in Profile/Contact method

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open NewDot app
Go to profile \ contact method

Expected Result:
Copy icon should be displayed near<EMAIL_ADDRESS>email
Actual Result:
Updated copy icon is missing in Profile/Contact method
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/231621215-5a52fd49-070f-47df-9f58-5c862fa3b290.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0129c789975d0ac1e3
Upwork Job ID:<PHONE_NUMBER>154647552
Last Price Increase: 2023-04-13

Triggered auto assignment to @lschurr (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Confirmed, the copy option isn't there on Android, but it is on iOS.

Should this be internal or external @iwiznia?

Triggered auto assignment to @NikkiWines (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

I think this is something related to the nested Text element issue in android
https://github.com/facebook/react-native/issues/31826

SVG

📣 @Balabhadrad! 📣
Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork.
Please follow these steps:

Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.

 Thank you for contributing to Expensify!

@NikkiWines, @lschurr, @aimane-chnaif Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Proposal
Please re-state the problem that we are trying to solve in this issue.
Android - Copy icon is missing in Profile/Contact method
What is the root cause of that problem?
I suspect this behavior is unique to Android and is caused by nesting a View inside a Text component, as is done so here in CopyTextToClipboard.js.
https://github.com/Expensify/App/blob/c144663fcad41a69344311a27dd6a147a28e7191/src/components/CopyTextToClipboard.js#L56-L71
The Tooltip component has a condition which renders its children whenever the device doesn't support hovering, therefore, the Icon component is rendered which wraps the svg inside a View.
https://github.com/Expensify/App/blob/c144663fcad41a69344311a27dd6a147a28e7191/src/components/Tooltip/index.js#L145-L149
https://github.com/Expensify/App/blob/c144663fcad41a69344311a27dd6a147a28e7191/src/components/Icon/index.js#L54-L65
Interestingly enough, the Copy icon isn't missing - it's rendered in the wrong place.

Screenshot of 'copy to clipboard' icon location

However, in order to see it, you'll first need to save while in the CopyTextToClipboard.js file to trigger a re-render of the component. This leverages React Native's fast refresh feature to re-render just the default-exported component, otherwise, saving in a file higher in the component hierarchy might not re-render the component of interest.
The icon even responds to click events like it's supposed to whether visible (after it's re-rendered) or invisible (current behavior when app is loaded).

Invisible 'copy to clipboard' icon click interaction
https://user-images.githubusercontent.com/79470910/233067383-ec1147f0-2c39-40ce-97cb-5746ee394804.mov

Visible 'copy to clipboard' icon click interaction
https://user-images.githubusercontent.com/79470910/233067541-c9175fa6-b6a7-4d93-a656-385968ed651f.mov

To demonstrate further, I tried nesting a different View inside a Text component in ContactMethodsPage.js and got similar behavior to the current bug.
Here's the code I used.
//...
{loginMenuItems}
<View>
    <Text>
       Hello, there
           <View>
               <Text>I'm nested in a View</Text>
           </View>
    </Text>
</View>
//...

My result after nesting a View inside a Text component

What changes do you think we should make in order to solve the problem?
This looks like a bug in React Native that may or may not be related to this currently open React Native Android bug discussing nested Views in Android.
I'd suggest not including the 'copy to clipboard' icon on Android devices for the time being, however, the docs forbid contributors from writing platform-specific code, so I think it would be best to leave implementing this at the discretion of Expensify's internal team.
Opening an issue with React Native also sounded like a good idea to me.
What alternative solutions did you explore?
I noticed one of the Views wrapping the svg was absolutely positioned, so I tried to play around with left and right style values to see what I could find.
One of the things I did was add a backgroundColor and adjust height & width of both Views wrapping the svg and noticed the outermost wrapper's color i.e. red, doesn't show on Android but shows on web (presumably on the other platforms where everything is working too).

Web result

Android result

Hopefully, this provides more insight on Android's tricky behavior.

@Victor-Nyagudi are you able to reproduce same in the snack? I am not facing this there, so doubt of bug is with forked RN repo.

@Victor-Nyagudi are you able to reproduce same in the snack? I am not facing this there, so doubt of bug is with forked RN repo.

Take a look at this Snack I made on Android and compare it to iOS and Web.
The text in the nested View  is misaligned.
Do you think it could be some default View behavior I'm not aware of?

Please re-state the problem that we are trying to solve in this issue.
Android - Copy icon is missing in Profile/Contact method
What is the root cause of that problem?
In React native CLI, android you can not render <View> component inside the <Text> components, this behavior is unique to Android and is caused by nesting a View inside a Text component,
You can check inside CopyTextToClipboard.js file, The Tooltip component has the condition to render Icon inside the Text components, but Icon has wrapped inside View components.
Check below file:
https://github.com/Expensify/App/blob/c144663fcad41a69344311a27dd6a147a28e7191/src/components/Icon/index.js#L54-L65
not working code like below:
<Text>
    Hello, there
    <View>
        <Text>I'm nested in a View</Text>
    </View>
</Text>

What changes do you think we should make in order to solve the problem?
There is only one solution I tested and seems appropriate. You need to render <View> component outside from <Text> Components otherwise Copy Icon is not rendered in android.
"Add more ways for people to find you, and forward receipts to<EMAIL_ADDRESS>+ <copy icon>"

@NikkiWines, @lschurr, @aimane-chnaif Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@aimane-chnaif can you review the recent proposal?

Still no acceptable proposals, right @aimane-chnaif?

This has the same root cause with this https://github.com/Expensify/App/issues/17509

Both issue comes from this PR https://github.com/Expensify/react-native/pull/37. From my testing, applying this solution will fix both issues.

@NikkiWines @lschurr @aimane-chnaif this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks!

Ideas for this one @NikkiWines @aimane-chnaif?

@lschurr still looking for valid proposals I believe, going to update the price on this one!

Proposal
Please re-state the problem that we are trying to solve in this issue.
Copy icon is missing in profile/contact page and workspace/reimburse expenses page.
What is the root cause of that problem?
This issue happens only on Android. Below is the code segment related to copy-clipboard icon.
https://github.com/Expensify/App/blob/d120f6a9aa13dc842e265202996f8fe32019faca/src/pages/settings/Profile/Contacts/ContactMethodsPage.js#L122-L131
I've tested the same code in a new react-native project(with the latest version of react-native) and that works fine.

And I've found that svg icons inside the Text element isn't visible in the current version of react-native through a lot of tests.
<View>
    <CopyIcon width={16} height={16} />
</View>

<View>
    <Text>Sample text</Text>
    <CopyIcon width={16} height={16} />
</View>

Above code works fine but below code doesn't work.
<Text>
    <CopyIcon width={16} height={16} />
</Text>

Through these considerations, I verified that the root cause of this issue is the bug of the current version of react-native.
What changes do you think we should make in order to solve the problem?
A tricky solution exists for this bug. Throughout the tests, I was able to find that numberOfLines props of Text component does the tricks. We need to set this value large enough to show all contents and this props should be set at the top-most Text component of the component tree.
Below are the code lines related.
https://github.com/Expensify/App/blob/d120f6a9aa13dc842e265202996f8fe32019faca/src/pages/settings/Profile/Contacts/ContactMethodsPage.js#L123-L130
https://github.com/Expensify/App/blob/d120f6a9aa13dc842e265202996f8fe32019faca/src/pages/workspace/reimburse/WorkspaceReimburseView.js#L237-L244
Replacing the <Text> at the first line with <Text numberOfLines={5}> fixed the issue successfully.

Result
https://user-images.githubusercontent.com/126839717/236634192-81cccc82-3e96-48a5-9523-0509bdbc5ee1.mp4

What alternative solutions did you explore? (Optional)
NA

Could you review this new proposal @aimane-chnaif?

How are we doing on this one @aimane-chnaif?

Nice, @s-alves10 - thanks for the thorough investigation.
If this is a bug in RN, we should also probably try and get that fixed in the main repo. So once we decide on a solution let's submit a change to our fork and also create an issue in the RN repo.
However, rhe fix of modifying the top level <Text> to havenumberOfLInes={5} seems a little hacky to me.

However, the fix of modifying the top level <Text> to havenumberOfLInes={5} seems a little hacky to me.

Yeah. It's a little bit tricky. We can track the bug from this solution in the main RN repo, I think.

Hello @NikkiWines! QA team is noticed that Copy icon is missing on Workspace Setting> any workspace > Reimburse expenses. Please let me know if we need to create a new ticket for this issue.
https://github.com/Expensify/App/assets/93399543/29008cf1-0ab3-45e5-b72d-7e94cfb14de0

still no proposals. we should wait for upstream fix

My proposal doesn't work?

I'm heading OOO from May 17-23rd. I've re-added the Bug label to reassign the issue and kept myself assigned as well. If action is still needed on May 24th, I'll take it back from you @anmurali.
Latest update: still no accepted proposals.

Still waiting on proposals here

@aimane-chnaif can you review the recent proposal?

That's still not acceptable.

You can't use the View component inside Text component. This is not acceptable form native android.
https://github.com/facebook/react-native/issues/25197
So if you can state the reason for not accepting my proposal, will help me improve my technical knowledge for the same, please! 🙂 And perhaps I can make new proposal based on that.

Un-assigning @anmurali since I'm back from OOO.

Still waiting on proposals for this one @NikkiWines @aimane-chnaif?

Not acceptable. There's upstream PR but not sure how much longer we should wait.

@NikkiWines @lschurr @aimane-chnaif
May I ask you a question?
If we have to wait this PR for this issue, I'm not sure why you are waiting for proposals.
I think we need to have some get-around until the PR is merged.
My solution here works perfectly for both contact page and reimburse page though it's a little bit tricky.

Thoughts on that last comment @NikkiWines @aimane-chnaif?

@aimane-chnaif that's true, we'd be beholden to a PR that we have no influence over. I'm hopeful that it will get merged since it's pretty active at the moment - but it's impossible to know how long that'll take.
Given that, we should probably go with option 2 that you listed above, where we update our own fork and put in a (hopefully) temporary fix until it's fixed in the RN repo.
However, whatever fix we put in place should still have a high standard and avoid hacky practices. Unfortunately, that leaves us back where we started 😅
@s-alves1, since your proposal involves hardcoding numberOfLines to a specific value, have you tested it across various platforms with different viewport sizes to confirm that it fully resolves the issue and doesn't cause any regressions?

@s-alves1, since your proposal involves hardcoding numberOfLines to a specific value, have you tested it across various platforms with different viewport sizes to confirm that it fully resolves the issue and doesn't cause any regressions?

Yes. I've tested both pages(contact and reimburse)

@NikkiWines
Okay. Then I'll tested on various platforms and upload the video

@NikkiWines
I'll test on various platforms and upload the video

Any updates @NikkiWines @aimane-chnaif?

Proposal
Updated
@aimane-chnaif
I uploaded the demo videos. Tests were done on smaller devices(Android 480 x 320) as well

Proposal
Updated
@aimane-chnaif
I updated the proposal for social icons. And updated the videos for android native and chrome(320) reflecting the social icons

Proposal
Updated
@aimane-chnaif
I updated the proposal according to your feedback. But it has a lot of changes. Please review and let me know what you think is the best choice.

@aimane-chnaif
Yeah, that makes sense.
I thought creating a PressableLink component for future use would be better.

@s-alves10
Yes. I've checked them on 1080p/320p devices. I think you checked them already

Are we assigning to @s-alves10 or what's the next step?

Oh sorry, missed this ping - looking now

@s-alves10 can you confirm that this change works even if you modify the text size (i.e. zooming in and out and also manually changing the text size on the mobile device)
If that works, then I think we can go ahead and use this solution. Sorry for the delay 🙇

@NikkiWines
I can confirm.
numOfLines defines the maximum number of lines. If the text is longer or larger, ellipsis will appear. If the text is shorter, numberOfLines doesn't affect the component size.
So we can set the numberOfLines to large number, i.e. 100. This works perfect.

Ok cool, thanks @s-alves10. In that case, let's move forward with your proposal.

@aimane-chnaif
PR is ready for review

@NikkiWines
The upwork job link is no longer available.

@lschurr can we re-post the upwork job for this please 🙇

New open job posting is here: https://www.upwork.com/jobs/~01ac7e80f8af912eb4

I submitted a proposal. Thank you

@aimane-chnaif - please apply for the job here: https://www.upwork.com/jobs/~01ac7e80f8af912eb4
Do we need a regression test for this?

Payment sent to @s-alves10.

Thank you

Be sure to apply for the job @aimane-chnaif - https://www.upwork.com/jobs/~01ac7e80f8af912eb4

Paid with bonuses. Closing!


Master branch default and no context setting

Hi there, a bit confused:

Why is master the default branch?
Further, why is a remote context apparently forced on the build process? Why can't we use a local checkout like most other build actions? Allows to specify build context seems inaccurate here.

Thanks.

I ran into this issue as well
I just added
          branch: main

into the with: section and it worked after!


Directwrite, Keyboard Input, and some more things

We have:

Windows rendering can also be done with Directwrite and Direct2D
Keyboard input works
Some bug fixes (resizing)
etc.

(Not intentional)


Modify papermill to use jovyan home directory as execution context

Since repo2docker converts the whole repository into the contents of the /home/jovyan directory inside the Docker image, instruct papermill to run from this directory. This ensures that any additional code in the repo used by the notebook can be accessed.

This has the added consequence that the stage-out directory now needs to be an absolute path.


Filter member email recipients based on the newsletter subscriptions

refs https://github.com/TryGhost/Team/issues/1524

We need to fetch the post newsletter to grab the slug as it's needed for the member NQL filter.
We can then use the newsletter slug and append it in the existing member NQL filter.

This is the backend implementation only, there is still the need for an admin PR to update the members counts that are displayed in the Publish flow.

@tpatel you'll also need to update the transform to stop using the member's subscribed flag.

I'm working on some changes here with the fixtures from https://github.com/TryGhost/Ghost/pull/14563. Still some failing tests but now they should be easy to solve with these fixtures

There is still a bug somewhere when sending without the newsletter_id option


Adding support to es6 Set collection

Hi,
It's seem like object-inspect is identifying a set and map by it's size property. Which when using a babel-polyfill doesn't distinguish between a Set and Map. This cause the inspect to try do a mapForEach on the set and cause the error.
Any idea what should be done?

If there's a bug in object-inspect, please file it there, and I'll address it ASAP.

(the object-inspect bug is filed here, and it turns out it's a bug in core-js/babel-polyfill)

object-inspect has a workaround for core-js's broken Map/Set implementation in v1.3.0, and separately, core-js fixed it in v2.5.0.

This package is no longer being updated; v21+ is maintained here


Detailed Completion Label Support

I'm looking at the new detailed completion label support, and would it be possible to make it a new property instead of a new union type? Or perhaps a new type entirely?
Every union type causes a binary breaking change for some compiled languages, like C# (and Java I think as well).  This forces a breaking on our consumers, and all downstream code.  A new property would make it much easier to gracefully support this feature, and avoid adoption delays.
cc @ntaylormullen @ryanbrandenburg @TylerLeonhardt
Proposed changes:
/**
 * A more detailed label for a completion item.
 *
 * @since 3.17.0 - proposed state
 */
export interface CompletionItemLabel {
	/**
	 * The name of a function or variable.
	 */
	name: string;

	/**
	 * The parameters without the return type.
	 */
	parameters?: string;

	/**
	 * The fully qualified name, like package name or file path.
	 */
	qualifier?: string;

	/**
	 * The return-type of a function or type of a property/variable.
	 */
	type?: string;
}

export interface CompletionItem {

	/**
	 * The label of this completion item.
	 *
	 * If the label is of type `string` it is also by
	 * default the text that is inserted when selecting
	 * this completion.
	 */
	label: string;

       /**
	 * @since 3.17.0 - proposed state -  support for CompletionItemLabel. See
	 *  also the client and server capability
	 * `completionItem.detailedLabelSupport`
	 *
	 * If the detailedLabel is of provided the `name` property
	 * is also by default the text that is inserted when selecting
	 * this completion.
	 */
        detailedLabel?: CompletionItemLabel;

        // ...
}

As one of the maintainers for LSP4J, I can confirm that these kinds of changes are breaking changes for us as well.

The problem with that approach is that a user now has to fill in a label and a detailedLabel and that the value of label is basically never used.
I better approach would be:
/**
 * Additional label details (observe no name property
 *
 * @since 3.17.0 - proposed state
 */
export interface CompletionItemLabelDetails {
	/**
	 * The parameters without the return type.
	 */
	parameters?: string;

	/**
	 * The fully qualified name, like package name or file path.
	 */
	qualifier?: string;

	/**
	 * The return-type of a function or type of a property/variable.
	 */
	type?: string;
}

export interface CompletionItem {

	/**
	 * The label of this completion item.
	 *
	 * If the label is of type `string` it is also by
	 * default the text that is inserted when selecting
	 * this completion.
	 */
	label: string;

       /**
	 * @since 3.17.0 - proposed state -  support for CompletionItemLabel. See
	 *  also the client and server capability
	 * `completionItem.detailedLabelSupport`
	 */
        labelDetails?: CompletionItemLabelDetails;

        // ...
}

Would that work for you?

Thanks @dbaeumer

Done. Let me know if this now works for you.

So is the label now the unqualified name? (while the labelDetails.qualifier is the FQDN?)
Does this mean the label is a substring of labelDetails.qualifier? Are there any "is substring of" relations?

I think that depends. Without labelDetails it is the old behavior whatever servers filled in in the past. Wil a label details it should be the unqualified name.

@dbaeumer  Should parameters be a string[]? How do we specify multiple parameters for eg a function?

Making it a string allows server to format the string according to the language. If we get a string[] we can't simply concatenate the pieces using ,. There are other languages that are not separating params with ,

How about using one large string, while having a list of (beginOffsetUtf16, endOffsetUtf16) that denote the parameter starts/ends like in signatureHelp?

Aahhh so the format is (param1, param2) for a function with 2 parameters?
If there are no params it should be () and for non-functions it should be
null?
On Wed, Feb 17, 2021, 6:07 AM Raoul Wols<EMAIL_ADDRESS>wrote:

How about using one large string, while having a list of
(beginOffsetUtf16, endOffsetUtf16) that denote the parameter starts/ends
like in signatureHelp?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/language-server-protocol/issues/1193#issuecomment-780481729,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBACRFUA3VINLRNNFRA223S7OPPTANCNFSM4XDNHQKQ
.

In SignatureHelp this is mostly used for highlighting since the label must be a subpart of the out label. With completion items we want to allow clients to render a more detailed label if wanted and it felt easier to send the parts.


Add initial hooks for future contact library development

As discussed, follow these steps:

Enable Axom building in our TPLs to be used by Tribol developer
Add FindTribol.cmake that uses TRIBOL_DIR given on cmake/config-build.py command line
Add SeracConfig.hpp that will have SERAC_USE_TRIBOL defined


Small fix regarding Jumpboot recipe

About the PR

Why / Balance

Technical details

Media

Requirements

[ ] I have read and am following the Pull Request and Changelog Guidelines.
[ ] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

Well I suppose they wont have a price for a while, fin.


How to collect and analyse tweets from a generic list of accounts?

I'd like to collect and analyse tweets from a generic list of accounts. What would be the preferred way to achieve this? How should the list of accounts look like? Where should the list be saved?
In your file code/02_get_accounts.R in line 11, your refer to a csv file "../data_polit_twitter/raw/german_politicians_twitter_raw.csv". I guess this would be the input for further steps? How is this csv created and what's its scheme?
Best,
Moritz

Hi Moritz,
I had forgotten to include this account list data file - it's now uploaded under data/.
This file was created more or less manually. It's a csv file with the columns
screenName,name,party,human
human is simply indicating whether it's a person (ie., politician) or a batch account (eg., for general notes from a given political party). I only included accounts from real individuals in my analysis.
HTH!


get_historical_klines return different values for the same close candle

Describe the bug
Im trying to get the most recent candle after close. For example 18:01, I want the 18 candle closed (interval 15 min). And the value that it is return, is different each time i run the script, until the next quarter (when the bug starts to be in the newest one)
To Reproduce
client.get_historical_klines('BTCUSDT','15m',
start_str='15 min ago',
end_str='tomorrow',
limit=1)
Expected behavior
Every time the same close value
Environment (please complete the following information):

Python version: 3.10.6
OS: windows 10
python-binance version: latest

Logs or Additional context
Add any other context about the problem here.

IMHO, it is not a bug and the code works as expected. Check open time and close time in return value to find out why.

Thanks for the answer halfelf, but there is no too much complexity in the open and close time to review.
Here is how you can reproduce it, and the console output. You can see that the time is after the candle suppose to be closed
from binance.client import Client
import datetime as dt
import pandas as pd
import time
config = open('config','r').read().splitlines()
client = Client(config[0], config[1])
print('Current time', dt.datetime.now())
data = pd.DataFrame(client.get_historical_klines('BTCUSDT','15m', limit=1,
start_str='15 min ago',
end_str='tomorrow'),
columns=['Open Time','open', 'high','low','close','volume','6', '7', '8', '9', '10', '11'], dtype=float)
data = data.drop(columns=[ '6', '7', '8', '9', '10', '11'])
data['Open Time2'] = [dt.datetime.fromtimestamp(int(x)/1000.0) for x in data['Open Time']]
print(data)
time.sleep(3)
data = pd.DataFrame(client.get_historical_klines('BTCUSDT','15m', limit=1,
start_str='15 min ago',
end_str='tomorrow'),
columns=['Open Time','open', 'high','low','close','volume','6', '7', '8', '9', '10', '11'], dtype=float)
data = data.drop(columns=[ '6', '7', '8', '9', '10', '11'])
data['Open Time2'] = [dt.datetime.fromtimestamp(int(x)/1000.0) for x in data['Open Time']]
print(data)
`
and here is the output console
Current time 2022-09-07 09:31:59.312941
Open Time      open        high       low           close         volume     Open Time2
0  1.662536e+12  18758.13  18759.0  18742.95  18743.76  282.31399 2022-09-07 09:30:00
Open Time      open        high       low           close        volume    Open Time2
0  1.662536e+12  18758.13  18759.0  18742.95  18745.03  291.1601 2022-09-07 09:30:00

Ok, i was analyzing and the thing is that the candle opened 9.30 is not close until 9.45. Thats why the close value is changing all the time.


add a tls test case for psql

add a psql test case that ensures we're tickling the tls paths.
leaving this as draft, as this is expected to fail until we fix the openssl3.2 compatibility issues with postgres.
ref: https://github.com/chainguard-images/images/issues/1898

As part of a Chainguard effort to reduce code duplication and provide higher quality images, we have stopped accepting pull requests for changes to images in this repository. Due to this, we are closing all pull requests today in order to start fresh.
If you think this pull request was closed in error, please feel free to re-open this pull request or open an issue and we will try to help you out. Thank you!


AST: Inherit access level of an opaque type decl from its naming decl

When an OpaqueTypeDecl is constructed, the access level attributes of the decl that names the opaque type are copied on to it. However, the @usableFromInline attribute is not permitted on every decl, so it does not get copied. This in turn causes effective access level computations for opaque types to fail to take @usableFromInline into account and that results in the emitted symbol getting the wrong linkage during IRGen. The fix is to make the effective access computation take this quirk of opaque types into account directly, instead of relying on copying of attributes.
Resolves rdar://110544170

@swift-ci please test

Can you remove the access level copying too, then?

Good idea, I think it makes sense to do that. However, I'd like to make that part of the change on main only as I think it increases the chances of regression slightly (I don't know whether any other part of the compiler queries for the modifiers/attributes and might be affected).

@swift-ci please test

@swift-ci please test

@swift-ci please test

Removing the attribute copying revealed that my previous approach to the fix was incomplete.

That's why I enjoy these kinds of cleanups. Thank you for seeing this through!


python311Packages.fiona: 1.9.4.post1 -> 1.9.5

Description of changes
Update and fix fiona.
Changelog: https://github.com/Toblerity/Fiona/releases/tag/1.9.5
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 262886 run on x86_64-linux 1

8 packages failed to build:

python310Packages.bsuite
python310Packages.bsuite.dist
python310Packages.plotnine
python310Packages.plotnine.dist
python311Packages.bsuite
python311Packages.bsuite.dist
python311Packages.plotnine
python311Packages.plotnine.dist

20 packages built:

python310Packages.fiona
python310Packages.fiona.dist
python310Packages.folium
python310Packages.folium.dist
python310Packages.geopandas
python310Packages.geopandas.dist
python310Packages.osmnx
python310Packages.osmnx.dist
python310Packages.wktutils
python310Packages.wktutils.dist
python311Packages.fiona
python311Packages.fiona.dist
python311Packages.folium
python311Packages.folium.dist
python311Packages.geopandas
python311Packages.geopandas.dist
python311Packages.osmnx
python311Packages.osmnx.dist
python311Packages.wktutils
python311Packages.wktutils.dist

Result of nixpkgs-review pr 262886 run on x86_64-darwin 1

4 packages marked as broken and skipped:

python310Packages.bsuite
python310Packages.bsuite.dist
python311Packages.bsuite
python311Packages.bsuite.dist

24 packages failed to build:

python310Packages.fiona
python310Packages.fiona.dist
python310Packages.folium
python310Packages.folium.dist
python310Packages.geopandas
python310Packages.geopandas.dist
python310Packages.osmnx
python310Packages.osmnx.dist
python310Packages.plotnine
python310Packages.plotnine.dist
python310Packages.wktutils
python310Packages.wktutils.dist
python311Packages.fiona
python311Packages.fiona.dist
python311Packages.folium
python311Packages.folium.dist
python311Packages.geopandas
python311Packages.geopandas.dist
python311Packages.osmnx
python311Packages.osmnx.dist
python311Packages.plotnine
python311Packages.plotnine.dist
python311Packages.wktutils
python311Packages.wktutils.dist

4 packages built:

python310Packages.scikit-misc
python310Packages.scikit-misc.dist
python311Packages.scikit-misc
python311Packages.scikit-misc.dist

@sikmir are you able to build netcdf on master ?

Indeed, it seems to fail on darwin:
FAIL: run_nc4_tests.sh
======================
*** Testing ncgen3 for netCDF-4.
*** creating netCDF-4 file c0_4.nc from c0.cdl...
dyld[87361]: Library not loaded: @rpath/libhdf5_hl.310.dylib
  Referenced from: <no uuid> /private/tmp/nix-build-netcdf-4.9.2.drv-0/netcdf-c-4.9.2/ncgen3/.libs/ncgen3
  Reason: tried: '/private/tmp/nix-build-netcdf-4.9.2.drv-0/netcdf-c-4.9.2/liblib/.libs/libhdf5_hl.310.dylib' (no such file), '/nix/store/w2vc36964qpm78gxi85x10fkkx2i23yi-swift-corefoundation-unstable-2018-09-14/Library/Frameworks/libhdf5_hl.310.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/nix/store/w2vc36964qpm78gxi85x10fkkx2i23yi-swift-corefoundation-unstable-2018-09-14/Library/Frameworks/libhdf5_hl.310.dylib' (no such file), '/nix/store/w2vc36964qpm78gxi85x10fkkx2i23yi-swift-corefoundation-unstable-2018-09-14/Library/Frameworks/libhdf5_hl.310.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/nix/store/w2vc36964qpm78gxi85x10fkkx2i23yi-swift-corefoundation-unstable-2018-09-14/Library/Frameworks/libhdf5_hl.310.dylib' (no such file), '/usr/local/lib/libhdf5_hl.310.dylib' (no such file), '/usr/lib/libhdf5_hl.310.dylib' (no such file, not in dyld cache)
./run_nc4_tests.sh: line 13: 87361 Abort trap: 6           ${NCGEN3} -k3 -b -o c0_4.nc ${ncgen3c0}
FAIL run_nc4_tests.sh (exit status: 134)

Looks like hdf5 needs to be fixed:
$ nix-build -A hdf5
/nix/store/0fvdgdvpv92a7nwrfvzh7c3aippwl9a7-hdf5-1.14.2
$ otool -L result/lib/libhdf5_hl.310.dylib
result/lib/libhdf5_hl.310.dylib:
        @rpath/libhdf5_hl.310.dylib (compatibility version 310.0.0, current version 310.0.2)
        @rpath/libhdf5.310.dylib (compatibility version 310.0.0, current version 310.2.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1292.60.1)

While @rpath/libhdf5_hl.310.dylib should be gone by adding fixDarwinDylibNames, fixing @rpath/libhdf5.310.dylib would need some manual install_name_tool changes.

Looks like hdf5 needs to be fixed:

This is out of the scope of this PR though, right ?

Looks like hdf5 needs to be fixed:

This is out of the scope of this PR though, right ?

Definitely. python311Packages.fiona was already failing on staging-next due to netcdf, so on master, it's not a regression from this PR.

@ofborg build python311Packages.fiona python311Packages.fiona.passthru.tests python311Packages.scikit-misc python311Packages.scikit-misc.passthru.tests

Result of nixpkgs-review pr 262886 run on aarch64-darwin 1

4 packages marked as broken and skipped:

python310Packages.bsuite
python310Packages.bsuite.dist
python311Packages.bsuite
python311Packages.bsuite.dist

4 packages failed to build:

python310Packages.plotnine
python310Packages.plotnine.dist
python311Packages.plotnine
python311Packages.plotnine.dist

24 packages built:

python310Packages.fiona
python310Packages.fiona.dist
python310Packages.folium
python310Packages.folium.dist
python310Packages.geopandas
python310Packages.geopandas.dist
python310Packages.osmnx
python310Packages.osmnx.dist
python310Packages.scikit-misc
python310Packages.scikit-misc.dist
python310Packages.wktutils
python310Packages.wktutils.dist
python311Packages.fiona
python311Packages.fiona.dist
python311Packages.folium
python311Packages.folium.dist
python311Packages.geopandas
python311Packages.geopandas.dist
python311Packages.osmnx
python311Packages.osmnx.dist
python311Packages.scikit-misc
python311Packages.scikit-misc.dist
python311Packages.wktutils
python311Packages.wktutils.dist

https://github.com/ofborg build python311Packages.fiona python311Packages.fiona.passthru.tests python311Packages.scikit-misc python311Packages.scikit-misc.passthru.tests

@GaetanLepage , if you are interested in maintenance of some geospatial packages you are welcome to join Geospatial team.

@GaetanLepage , would you please extract change of source for Fiona package to separate commit ? Sorry for bothering with this, but I think that change of source (especially when changing from Pypi) is quite important change which shouldn't be lost between some other changes. Thank you.

I am not sure to get what you mean.
In the fiona commit, I did the following:

Update dependencies:
a. cython -> cython_3
b. +oldest-supported-numpy
Remove now useless postPatch section
Update the package version:
a. version: 1.9.4.post1 -> 1.9.5
b. Change the hash accordingly
format = "pyproject" -> pyproject = true (now recommended)

You want me to split those 4 sub-changes ?
If so, how would you cut it ?

I am not sure to get what you mean.

I am very sorry for confusion. I see, there is no change of source in Fiona. It is good as it is.
But for scikit-misc, there is a change of source from Pypi to GH and I see following commit message:
python311Packages.scikit-misc: fix build
Changelog: https://github.com/Toblerity/Fiona/releases/tag/1.9.5

I suggest to replace fix build text with change package source from Pypi to GitHub and remove next line Changelog: ....

I suggest to replace fix build text with change package source from Pypi to GitHub and remove next line Changelog: ....

My bad. I was 100% sure that I had changed the scikit-misc commit message already, but I have overwritten it by mistake.
Now it should be good !

Thanks @GaetanLepage .


Adding header values into request when values are present in req_args

Given an OpenAPI specification:
...
path:
  /foo:
    get:
      parameters:
        - in: header
          name: bar
          schema:
            type: string
          required: true
...

SwaggerClient.foo(bar="abcd"), a functions generated after parsing this specification will not apply `bar="abcd" to the header of the request. The following PR fixes this.

Codecov Report

Merging #447 into master will decrease coverage by 0.37%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #447      +/-   ##
==========================================
- Coverage   85.47%   85.09%   -0.38%
==========================================
  Files          39       39
  Lines        1824     1825       +1
==========================================
- Hits         1559     1553       -6
- Misses        265      272       +7

Impacted Files
Coverage Δ

hca/util/__init__.py
86.92% <100%> (+0.03%)
:arrow_up:

hca/dss/__init__.py
90.13% <0%> (-1.92%)
:arrow_down:

Continue to review full report at Codecov.

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


Request: Time tracker idle detection

One feature I really miss from other time trackers I've used is idle detection.  When it detects that you haven't been using your computer for a user-specified length of time, it pops up a dialogue asking you what to do.  You can either:

revert the timer to when you went idle, and stop the timer.
revert the timer to when you went idle, and start the timer.
continue the timer.

Of course this would require a resolution to #5 to be terribly useful, as well as recording explicit start/stop times for each interval.

Although I implemented the intervals stuff already (haven't pushed yet) I won't implement this since I don't need it.
People are free to make a pull request if it doesn't complicate the code base too much.

Btw, ou can prevent the tracker from running when idle by simply letting the screen lock. This will stop the tracker.


YouTube Channel extraction isn't working

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.02.10
[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

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-J', u'--flat-playlist', u'https://www.youtube.com/c/TechLead']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2021.02.10
[debug] Python version 2.7.17 (CPython) - Linux-4.15.0-132-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: none
[debug] Proxy map: {}
{"extractor": "youtube:tab", "_type": "playlist", "description": "Ex-Google/ex-Facebook tech lead, multi-millionaire app entrepreneur, digital nomad traveler, software engineer. \u2615\ufe0f \ud83d\udc4c\n\nIf you're looking for help landing a job as a software engineer, resume critique, and mentorship, join me at:\nhttp://techinterviewpro.com/\nhttp://coderpro.com/\n\nLearn the secrets of YouTube success with me here:\nhttp://youtubebackstage.com/\n\nAdd me here:\nhttp://instagram.com/techleadhd\nhttp://twitter.com/techleadhd\nhttps://www.linkedin.com/in/patrickshyu/\n\nBusiness Inquiries<EMAIL_ADDRESS>"title": "TechLead - Home", "extractor_key": "YoutubeTab", "webpage_url": "https://www.youtube.com/c/TechLead", "entries": [{"url": "https://www.youtube.com/c/TechLead/videos?view=0&sort=dd&shelf_id=1", "_type": "url", "ie_key": null, "title": "Uploads"}, {"url": "https://www.youtube.com/c/TechLead/videos?view=0&sort=p&shelf_id=2", "_type": "url", "ie_key": null, "title": "Popular uploads"}], "id": "UC4xKdmAXFh4ACyhpiQ_3qBw", "webpage_url_basename": "TechLead"}

Description

Hi, youtube-dl is not getting the JSON info for channels. This used to work before but now it just gets the playlists of the channel and not the list of videos. Here's a sample link:
https://www.youtube.com/c/TechLead
It only fetches the 2 playlists in the channel and not the entire videos list. It used to work using --flat-playlist but now it doesn't. Has something changed?

Use youtube.com/c/TechLead/videos and close the issue.

I would like to jump on this before it is closed: I have been having a similar issue for several months. Ever since YouTube-dl got restored from the RIAA takedown, I have been able to download videos again, but I can't seem to download playlists or channels anymore, for playlists, it will just download the first video and stop, and for channels it won't do anything at all. I am also using version 2021.02.10.

It downloads exactly what's located on the link's webpage. As already noted above, to download all videos pass videos URL.

This didn't help at all


Support for erb statements within deployment manifests?

With the ruby cli, it was possible to embed ERB statements in deployment manifest that were executed when deploying.
Example: https://github.com/pivotal-cf/cf-rabbitmq-release/blob/v229.2.0/docs/bosh_rabbitmq.md#unclustered-rabbit-with-customised-configuration
Currently, the new golang cli does not execute ERB code in deployment manifest which makes constructs as the one above impossible. Are there any plans to support this?

That particular behavior was intentionally removed from the new CLI. We wanted to avoid the extra-dynamic ruby behavior in favor of a more declarative syntax. We also didn't want to be encouraging release authors to be distributing manifests where ruby/erb was a dependency.
We're not bringing back ERB support into the CLI commands, so now we're recommending the usage of ops files. The combination of variables and YAML manipulations can hopefully provide you with sufficient functionality for the use cases you need.
If you have any questions on those ops files, or have scenarios where they don't appear to work for your needs, please feel free to bring them up here or in slack so we can figure out recommendations/improvements we can make.

Thank you @dpb587-pivotal for your explanation.
That sounds reasonable to me. I don't see a lot of cases where one would really rely on this behavior so I guess this can be fixed in the respective releases rather easily. Will bring it up with them.


Build failed on Windows，“stdio.h”: No such file or directory

Getting error when execute gluonfx:build, and I found error meesage "stdio.h": No such file or directory in file client-debug0.log. How can I solve this problem?
By the way, the cl command was added to the path manually, I don't know if it's related to the error or what.

client-debug0.log
[周四 10月 06 17:32:32 CST 2022][信息] ==================== COMPILE TASK ====================
[周四 10月 06 17:32:33 CST 2022][信息] We will now compile your code for x86_64-microsoft-windows. This may take some time.
[周四 10月 06 17:32:33 CST 2022][详细] Extracting native libs to: D:\workspace\projects\git\termio\desktop\target\gluonfx\x86_64-windows\gvm\lib
[周四 10月 06 17:32:33 CST 2022][详细] Looking for resource: /native/windows/launcher.c
[周四 10月 06 17:32:33 CST 2022][详细] PB Command for compile-additional-sources: cl -c -DSUBSTRATE /MD /D_UNICODE /DUNICODE /DWIN32 /D_WINDOWS -ID:\workspace\projects\git\termio\desktop\target\gluonfx\x86_64-windows\gvm\desktop launcher.c
[周四 10月 06 17:32:33 CST 2022][详细] Start process compile-additional-sources...
[周四 10月 06 17:32:33 CST 2022][详细] [SUB] 用于 x64 的 Microsoft (R) C/C++ 优化编译器 19.33.31629 版
[周四 10月 06 17:32:33 CST 2022][详细] [SUB] 版权所有(C) Microsoft Corporation。保留所有权利。
[周四 10月 06 17:32:33 CST 2022][详细] [SUB]
[周四 10月 06 17:32:33 CST 2022][详细] [SUB] launcher.c
[周四 10月 06 17:32:33 CST 2022][详细] [SUB] launcher.c(28): fatal error C1083: 无法打开包括文件: “stdio.h”: No such file or directory
[周四 10月 06 17:32:33 CST 2022][详细] Result for compile-additional-sources: 2
[周四 10月 06 17:32:33 CST 2022][严重] Process compile-additional-sources failed with result: 2
Check the log files under D:\workspace\projects\git\termio\desktop\target\gluonfx\x86_64-windows\gvm\log
And please check https://docs.gluonhq.com/ for more information.
[周四 10月 06 17:32:33 CST 2022][信息] Logging process [compile-additional-sources] to file: D:\workspace\projects\git\termio\desktop\target\gluonfx\log\process-compile-additional-sources-1665048753580.log
[周四 10月 06 17:32:33 CST 2022][严重] Compiling failed.
Check the log files under D:\workspace\projects\git\termio\desktop\target\gluonfx\x86_64-windows\gvm\log
And please check https://docs.gluonhq.com/ for more information.

OS info
os.detected.name: windows
os.detected.arch: x86_64
os.detected.bitness: 64
os.detected.version: 10.0
os.detected.version.major: 10
os.detected.version.minor: 0
os.detected.classifier: windows-x86_64

Maven output
[ERROR] Failed to execute goal com.gluonhq:gluonfx-maven-plugin:1.0.15:build (default-cli) on project desktop: Error, gluonfx:build failed -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.gluonhq:gluonfx-maven-plugin:1.0.15:build (default-cli) on project desktop: Error, gluonfx:build failed
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:215)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
at org.apache.maven.cli.MavenCli.execute (MavenCli.java:972)
at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:293)
at org.apache.maven.cli.MavenCli.main (MavenCli.java:196)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:77)
at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke (Method.java:568)
at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:282)
at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:225)
at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:406)
at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:347)
at org.codehaus.classworlds.Launcher.main (Launcher.java:47)
Caused by: org.apache.maven.plugin.MojoExecutionException: Error, gluonfx:build failed
at com.gluonhq.NativeBuildMojo.execute (NativeBuildMojo.java:82)
at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:210)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
at org.apache.maven.cli.MavenCli.execute (MavenCli.java:972)
at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:293)
at org.apache.maven.cli.MavenCli.main (MavenCli.java:196)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:77)
at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke (Method.java:568)
at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:282)
at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:225)
at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:406)
at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:347)
at org.codehaus.classworlds.Launcher.main (Launcher.java:47)

@Joezeo run gluonfx:build in windows Visual Studio x64 Command Prompt e.g. x64 Native Tools Command Prompt for VS 2019
or in Cmd first run somethings like this:
%comspec% /k "C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Auxiliary\Build\vcvars64.bat"

@ctoabidmaqbool It works now, thanks a lot!

Having the same problem, but running from the Native Tools Command Prompt.
I installed Visual Studio with all the components mentioned in the Gluon guide. GraalVM is also installed and the path set.
When running "mvnw gluonfx:runagent" after cd to my project dir inside the native tools cmd prompt, it works. But the compile task first wouldn't run because of missing cl.exe (#195), which I fixed by adding the path to cl.exe from the new VS install to my PATH, but after that the compile command gives same error as user Joezeo reports above, even though stdio.h is present where it should be (C:\Program Files (x86)\Windows Kits\10\Include\10.0.19041.0\ucrt)...adding that to PATH didn't solve the problem.
Should I open a new issue or do you want to reopen?

Okay so after modifying the Visual Studio installation to include the "latest" build tools, it will compile.
Please consider changing your guide appropriately, as I would have intuitively installed the "latest" option if it wasn't for your guide specifically mentioning an older version. The guide should tell people to install the option "latest" when it comes to build tools, because, as the Microsoft answer in the above post's link says, only then will the vcvars[].bat be installed; otherwise compilation will not be possible.
Unfortunately my program still does not run with Graal, because it seems to package the resources differently than jpackage, but that's a different issue...


Made all the getUserMedia standard.

You are using the deprecated version of getUserMedia, so it's making a warning.
I made all the getUserMedia standard with my MediaStreamHelper.
If you want this library to fully support IE, then it needs polyfill codes for Promises(I didn't add it.),
because IE does not support Promises.
P.S.:
On the line 108: is })(window);; a correct code or a typo?

If you adopt this, Wad supports all the browsers that have a deprecated version or a standard version of getUserMedia.

Thanks for submitting this pull request. This looks like a great contribution, however it looks like you've edited the build version, and left the source version unchanged. If you could apply these changes to src/wad.js instead of build/wad.js, and then rebuild the library with grunt, that would be awesome.
I don't particularly care about supporting Internet explorer, though it would be nice to support Edge.
The extra semicolon on line 108 was put there by the build system. Grunt adds a semicolon between each file when they are concatenated, though I guess it wasn't necessary in this case.

Thank you and check my new pull-request: https://github.com/rserota/wad/pull/54!


Approve a pipeline to run on a specific agent pool, avoiding to manually approve on the UI

Preconditions

[X] No need to upgrade Python SDK or the Python SDK is ready.

Related command
az pipelines run \
  --commit-id "xxx" \
  --name "xxx"

Resource Provider
N/A
Description of Feature or Work Requested
I'm automating integration testing with Azure DevOps Pipelines for on-premise agent containers. This requires me to create projects & pipelines, execute them, and confirm the result.
Each time a new pipeline is created, and a first run is programmed, the pipeline never runs and a message "This pipeline needs permission to access a resource before this run can continue" is displayed. This requires a manual intervention on two buttons.
This behaviour disrupts a lot of the workflow. I would need a way to authorize a pipeline to run on a specific agent pool.
Minimum API Version Required
N/A
Swagger PR link / SDK link
N/A
Request Example
No response
Target Date
2023-03-01
PM Contact
N/A
Engineer Contact
N/A
Additional context
In the Azure DevOps web interface, this call is executed.
HTTP call:
PATCH
https://dev.azure.com/azure-pipelines-agent/a4830acb-f3a3-45b5-b555-d21a240eeb36/_apis/pipelines/pipelinePermissions/queue/80
Authorization: Bearer xxx

Request body:
{
    "resource": {
        "type": "queue",
        "id": "80"
    },
    "pipelines": [
        {
            "id": 9,
            "authorized": true,
            "authorizedBy": {
                "displayName": "Clémence Lesné",
                "id": "970e1056-90cc-66a2-aa4e-1b4b9efc4967",
                "uniqueName"<EMAIL_ADDRESS>                "descriptor": "aad.OTcwZTEwNTYtOTBjYy03NmEyLWFhNGUtMWI0YjllZmM0OTY3"
            },
            "authorizedOn": "/Date(1703957638150)/"
        }
    ]
}

Thank you for opening this issue, we will look into it.

I succeeded in executing the call with az devops invoke.
Nevertheless:

It could be useful to create a dedicated command, like az pipelines permissions --name=xxx --queue=xxx
At minimum, procedures should be documented somewhere, I was forced to retro-engineer the Azure CLI to make that working

Script:
#!/bin/bash
set -e

pipeline_id="xxx"
project_id="xxx"
queue_id="xxx"

# Store body in a temp file
tmp_file=$(mktemp -t XXXXXX.json)

# Add body content
cat <<EOF > ${tmp_file}
{
  "pipelines": [{
    "authorized": true,
    "id": "${pipeline_id}"
  }]
}
EOF

# Execute command
az devops invoke \
  --api-version 7.1-preview \
  --area pipelinePermissions \
  --http-method PATCH \
  --in-file ${tmp_file} \
  --resource pipelinePermissions \
  --route-parameters project=${project_id} resourceType=queue resourceId=${queue_id} \
    > /dev/null

# Cleanup
rm -f ${tmp_file}


You should add warning anti-injection may cause bugs in your application

Because anti-injection avoids some applications to run correctly. Actually it's not a bug it's just avoids application to inject python311.dll

python311.dll

Can you explain better?

Because anti-injection avoids some applications to run correctly. Actually it's not a bug it's just avoids application to inject python311.dll

Can you explain better?

Because anti-injection avoids some applications to run correctly. Actually it's not a bug it's just avoids application to inject python311.dll

Can you explain better?

Example if you want notify from notifypy import Notify then you try Notify it doesn't allow to inject python311.dll so it blocks notify the application.

Ok that's a bug related with me. Please look the  #9


the mojo generated libraries don't have library level docs

See
http://build.chromium.org/p/client.flutter/builders/Linux/builds/663/steps/dartdoc packages/logs/stdio. If the source material has some docs, we should generate library level docs for the *.mojom.dart files.

Probably should file in the domokit/mojo.git repo.

Closing as a dupe of https://github.com/domokit/mojo/issues/624


CDN links for Media Library doesn't work yet after #340

The definition o w3tc_attachment_url (https://github.com/szepeviktor/w3-total-cache-fixed/blob/v0.9.5.x/Cdn_Plugin.php#L776) is wrong.
If idea is to work only in ADMIN, and  cdn.admin.media_library=true, the if statement is wrong.

merged https://github.com/szepeviktor/w3-total-cache-fixed/issues/516