3dskullstrip libGL.so.1?

Hi,
I have been running mriqc by converting docker --> singularity, but now that singularity 2.3 supports pulling directly from dockerhub, I'm giving that a try.
Command I'm running:
singularity exec -B $base:/mnt docker://poldracklab/mriqc:latest mriqc --participant_label $subj --verbose-reports --n_procs 10 --mem_gb 10 -w $WORKDIR $DATADIR $OUTDIR participant

Everything seems to be running fine, until getting to afni SkullStrip, where I run into a missing library causing that node to fail:
3dSkullStrip: error while loading shared libraries: libGL.so.1: cannot open shared object file: No such file or directory

I haven't tried using docker2singularity for mriqc 0.9.6 yet, but 0.9.3 was working fine.

After pulling the docker image and converting with docker2singularity, no issue is encountered - @vsoch potential bug?

A few possibilities:

If 3dSkullStrip (and it's respective libraries) are somewhere in root's home (/root) as is typical of these images, and loading the shared library requires more than read permission, then the file is not found or openable because of that. The error message I would have expected is with regard to permission, but if an entire directory isn't readable at all, then it would come across as not found.
If the library is defined in some environment variable that is not being properly passed (for example, see this issue) then the library isn't being found because it's not on the path.
it could be some issue with a library that (for some reason) was not properly linked, or can't be properly linked given some permission. For example, with python wheel files python sometimes tries to unpackage them the first time you run, and if you don't have write, that can't happen (you get an error on the first run and then you might have a silent bug).

The approach I would take is to first figure out how/where the library is loaded in the working image, and then compare 1) locations, 2) environment variables to them, and 3) permissions for those things. My thinking is that one or more combinations of those things is seeing the error that you report. If you haven't been using singularity before 2.3, I'm not sure that we can attribute the error to the specific version, it very well could have been there with 2.2 too.

@vsoch thanks for the response, I have been using singularity since 2.1, but this is the first time I've tried to run this image from pulling it straight from dockerhub (a really nice addition!).
However, the conversion is not that same as using the docker container docker2singularity since I don't have root access when pulling/running the singularity. This is what you meant, yes?

yes, it's not the same.

FWIW 3dSkullStrip is not located in /root
Singularity.poldracklab_mriqc_0.9.6-2017-06-03-99db97c9be2e.img> $ which 3dSkullStrip
/usr/lib/afni/bin/3dSkullStrip

It seems that even though the pull feature in singularity is amazing and the team is doing an outstanding job the feature is still a little rough around the edges.
@mgxd I would suggest moving this issue to https://github.com/singularityware/singularity/issues since I am not sure what we could do within MRIQC to resolve it.

@chrisfilo but where is the library? It would be very helpful if someone that uses this image could look into these issues, and figure out if it is related to the things above. We do our best to meet the needs of the users for Singularity, but without some guidance about "this is the /root of the problem" (har har I just made a pun) there isn't much we could do.

The library is in /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1. Let me know if there is anything else I could help with.
I was not suggesting @mgxd or me should not be involved in investigating this issue. Just wanted to move it to singularity repo to expose it to broader singularity development community.

@chrisfilo looks like /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1 links to another file with limited permissions:
root@26412170ebeb:/# ls -l /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1
lrwxrwxrwx 1 root root 14 Jan 26 00:17 /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1 -> libGL.so.1.2.0
root@26412170ebeb:/# ls -l /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1.2.0
-rw-r--r-- 1 root root 471680 Jan 26 00:17 /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1.2.0

Perhaps this lack of permission results in it not being copied over?

Could you confirm that /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1.2.0 is present in the Docker image and the Singularity image created by docker2singularity but missing in the Singularity image created by singularity pull?

@chrisfilo /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1.2.0 is present in all images, but /usr/lib/x86_64-linux-gnu/mesa/libGL.so.1 is not present in the image generated from singularity pull

here are the layers that singularity is dumping:
[u'sha256:a3ed95caeb02ffe68cdd9fd84406680ae93d633cb16422d00e8a7c22955b46d4',
 u'sha256:8d47a6b78090d68b17786e22b92816f61974abea1ed267dd2ab6dc705b47495a',
 u'sha256:71cc1f897f9182469542083e31668fe2e9ef6a29bd34a642994098c792453823',
 u'sha256:7580416535f0c3dbdace804bed0213a3701debb79e34ea099f8849bf8dccb535',
 u'sha256:5bd848b357d35ec62bd6c13341b2193230c0d98155df403cee5b601c45e58c20',
 u'sha256:7700f5f11bcd210e6758a53d17dca9df692740731fc3e3805ac48e9f840a31d0',
 u'sha256:4404026f166c19230e6e66654ee7a558d538c20e8063192d36f13f26ddb0f3b5',
 u'sha256:1f75aa614800db092e54925a55a594b38d2d86637caa6f73b74c5efb433a6d94',
 u'sha256:37ed49cdeab67b45d3ddb44c25335bf6a7b13dbf48f09977061728017ac9edd0',
 u'sha256:fe74f1e296f66f8dbdea2d0f2344453dc660717a8bb502d0e62ba72d98ead94d',
 u'sha256:37773e97f2be732e3c478b498098b728e3c09b7ad7bd821202a4ed12bbe80c8b',
 u'sha256:3035235ca7e3b602ff8305dc50444232674143ca0968e8ea040a5a21b56972a7',
 u'sha256:34e9c7f09808029d900e60a3c239a70c6851c225bfdab24e6bf59042168cee08',
 u'sha256:6353ca1d3ed97743ee8a084d1d67356dc2f22807508907ec82131b338d1f2810',
 u'sha256:148a7d8ec5ddd119658f990e52dd4a2a2b11c92067c17afb51aaa783546d08d1',
 u'sha256:a144dee688a05a33cc6159fa173830645631c3c2e2cd39f79be2931f3399fc75',
 u'sha256:84e7d3c3db999d016da96fb629a9a2d557b38ff692b40ca89498866fb90f99c9',
 u'sha256:45da7bb9c607ee46c06e6c9d38cffe4a19b1f42a202dcb3698dce92eb42b011d',
 u'sha256:bd842a2105a862f62ba4ad726531029be49046c290307e9c07a688c89f4606e6',
 u'sha256:4a70713c436f2350e8a9f1c1e6c823290f331254c6c21d5657056499d8575608',
 u'sha256:63fe8c0068a811103c702452985368fc99014f83b5f0151cf631644f13f383b2',
 u'sha256:4daad8bdde31f5969d6a44800e993a911bed25853b2f89417b07ce7f0ac67714',
 u'sha256:b3e1c725a85f0953e81815b7c7aabfad9ebfd90af53f99248981282b8045d787']

and manifest for latest (v1):
{
    "container": "0a809f3915ae132eabddf66d390dc552b03d5c295ead9768450c226139b4576b",
    "parent": "5299ab26dec4a8a36a902a00ea215d5b52170d9cef59f068d3a1ba9d9e77fede",
    "created": "2017-06-03T19:09:32.682763788Z",
    "os": "linux",
    "container_config": {
        "Tty": false,
        "Cmd": [
            "/bin/sh",
            "-c",
            "#(nop) LABEL org.label-schema.build-date=2017-06-03T19:08:17Z org.label-schema.name=MRIQC org.label-schema.description=MRIQC - Automated Quality Control and visual reports for Quality Assesment of structural (T1w, T2w) and functional MRI of the brain org.label-schema.url=http://mriqc.readthedocs.io org.label-schema.vcs-ref=41313a5 org.label-schema.vcs-url=https://github.com/poldracklab/mriqc org.label-schema.version=0.9.6 org.label-schema.schema-version=1.0"
        ],
        "Volumes": null,
        "Domainname": "",
        "WorkingDir": "",
        "Image": "5299ab26dec4a8a36a902a00ea215d5b52170d9cef59f068d3a1ba9d9e77fede",
        "Hostname": "2866979c4d77",
        "StdinOnce": false,
        "Labels": {
            "org.label-schema.version": "0.9.6",
            "org.label-schema.schema-version": "1.0",
            "org.label-schema.vcs-ref": "41313a5",
            "org.label-schema.url": "http://mriqc.readthedocs.io",
            "org.label-schema.vcs-url": "https://github.com/poldracklab/mriqc",
            "org.label-schema.build-date": "2017-06-03T19:08:17Z",
            "org.label-schema.name": "MRIQC",
            "org.label-schema.description": "MRIQC - Automated Quality Control and visual reports for Quality Assesment of structural (T1w, T2w) and functional MRI of the brain"
        },
        "AttachStdin": false,
        "User": "",
        "Env": [
            "PATH=/usr/local/miniconda/bin:/opt/ants:/usr/lib/fsl/5.0:/usr/lib/afni/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
            "FSLDIR=/usr/share/fsl/5.0",
            "FSLOUTPUTTYPE=NIFTI_GZ",
            "FSLMULTIFILEQUIT=TRUE",
            "POSSUMDIR=/usr/share/fsl/5.0",
            "LD_LIBRARY_PATH=/usr/lib/fsl/5.0:",
            "FSLTCLSH=/usr/bin/tclsh",
            "FSLWISH=/usr/bin/wish",
            "AFNI_MODELPATH=/usr/lib/afni/models",
            "AFNI_IMSAVE_WARNINGS=NO",
            "AFNI_TTATLAS_DATASET=/usr/share/afni/atlases",
            "AFNI_PLUGINPATH=/usr/lib/afni/plugins",
            "ANTSPATH=/opt/ants",
            "LANG=C.UTF-8",
            "LC_ALL=C.UTF-8",
            "CRN_SHARED_DATA=/niworkflows_data"
        ],
        "Entrypoint": [
            "/usr/local/miniconda/bin/mriqc"
        ],
        "OnBuild": [],
        "AttachStderr": false,
        "AttachStdout": false,
        "OpenStdin": false
    },
    "architecture": "amd64",
    "docker_version": "1.9.1-circleci-cp-workaround",
    "config": {
        "Tty": false,
        "Cmd": null,
        "Volumes": null,
        "Domainname": "",
        "WorkingDir": "",
        "Image": "5299ab26dec4a8a36a902a00ea215d5b52170d9cef59f068d3a1ba9d9e77fede",
        "Hostname": "2866979c4d77",
        "StdinOnce": false,
        "Labels": {
            "org.label-schema.version": "0.9.6",
            "org.label-schema.schema-version": "1.0",
            "org.label-schema.vcs-ref": "41313a5",
            "org.label-schema.url": "http://mriqc.readthedocs.io",
            "org.label-schema.vcs-url": "https://github.com/poldracklab/mriqc",
            "org.label-schema.build-date": "2017-06-03T19:08:17Z",
            "org.label-schema.name": "MRIQC",
            "org.label-schema.description": "MRIQC - Automated Quality Control and visual reports for Quality Assesment of structural (T1w, T2w) and functional MRI of the brain"
        },
        "AttachStdin": false,
        "User": "",
        "Env": [
            "PATH=/usr/local/miniconda/bin:/opt/ants:/usr/lib/fsl/5.0:/usr/lib/afni/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
            "FSLDIR=/usr/share/fsl/5.0",
            "FSLOUTPUTTYPE=NIFTI_GZ",
            "FSLMULTIFILEQUIT=TRUE",
            "POSSUMDIR=/usr/share/fsl/5.0",
            "LD_LIBRARY_PATH=/usr/lib/fsl/5.0:",
            "FSLTCLSH=/usr/bin/tclsh",
            "FSLWISH=/usr/bin/wish",
            "AFNI_MODELPATH=/usr/lib/afni/models",
            "AFNI_IMSAVE_WARNINGS=NO",
            "AFNI_TTATLAS_DATASET=/usr/share/afni/atlases",
            "AFNI_PLUGINPATH=/usr/lib/afni/plugins",
            "ANTSPATH=/opt/ants",
            "LANG=C.UTF-8",
            "LC_ALL=C.UTF-8",
            "CRN_SHARED_DATA=/niworkflows_data"
        ],
        "Entrypoint": [
            "/usr/local/miniconda/bin/mriqc"
        ],
        "OnBuild": [],
        "AttachStderr": false,
        "AttachStdout": false,
        "OpenStdin": false
    },
    "id": "dcc5fbea4efd992f8df0e052fa24b6b7a54e417104c0d2007b92a265175cab87"
}

Hey @vsoch I think I'm missing something. Are you just sharing this info to highlight a potential solution?

nope just sharing information, in case it's helpful. For example, you might look at the layers being used and the ones singularity is using and say "aha, this is different!" or "look at that in the manifest, I didn't expect that to be parsed as such."

I see - so your theory is that docker pull and singularity pull are getting different layers from Docker Hub?

I don't know... I think it's most likely that a file is being extracted with incorrect permissions, as we've seen there are a lot of problems with things in /root, I'm looking around at manifests and layer ids (and trying to pull the image to take a look myself... it's still fat @chrisfilo ! :P)  and figured it wouldn't hurt to share in case it inspired anyone else's thinking.

I think you could do like:
docker images --digests --all | grep mriqc

or use inspect to look at the manifest being used for the image for docker2singularity, and just for a sanity check, make sure they are the same

I'm getting:
poldracklab/mriqc                    0.9.6               sha256:c7a79735153b75fb0eee027cce958f8bb7f402b5ff1be7f264061e589a4eef5c   a395b7f4ccc0        2 weeks ago         4.61 GB
which does not seem to correspond to any of the hashes in the data you shared...

To recap:
The symlink file
/usr/lib/x86_64-linux-gnu/mesa/libGL.so.1 -> libGL.so.1.2.0

is not copied over, but the actually library
/usr/lib/x86_64-linux-gnu/mesa/libGL.so.1.2.0

is present in the container made by singularity pull.

Moving this to singualrity repo to try to get more exposure from the singularity dev community: https://github.com/singularityware/singularity/issues/795

I am still having this problem when I run the latest container posted on the Dockerhub. Has the latest container pushed to Dockerhub after RUN ldconfig was added?

Yes, it does (https://github.com/poldracklab/mriqc/blob/980c7209ba717e2cb06f2a23f77de1c756e5e708/Dockerfile#L122).
Can you double check that the tag you are using is 0.9.7? Thanks!

Thanks. I am running the container as following
singularity run docker://poldracklab/mriqc input output participant --no-sub
I've also tried the specific tag (docker://poldracklab/mriqc:0.9.7), but ran into the same problem. Maybe that's not the right way to specify the tag?
I am seeing following error message.
RuntimeError: Command:
3dSkullStrip -input /home/hayashis/git/app-mriqc/work/workflow_enumerator/anatMRIQCT1w/AFNISkullStripWorkflow/_in_file_..home..hayashis..git..app-mriqc..input..sub-0..anat..sub-0_T1w.nii.gz/skullstrip/sub-0_T1w_conformed_corrected.nii.gz -prefix sub-0_T1w_conformed_corrected_skullstrip.nii.gz
Standard output:
3dSkullStrip: error while loading shared libraries: libGL.so.1: cannot open shared object file: No such file or directory

If I do following, everything works.
singularity exec docker://poldracklab/mriqc bash -c "ldconfig && /usr/local/miniconda/bin/mriqc input output participant --no-sub"

I will add some details on my testing to the newer issue #648


More color-coding in tables

To make the tables easier to scan, it would be nice if more of the tds were highlighted. For instance, it would be great if the "Block Breakdown" column cells were highlighted according to which candidate won the breakdown.

We don't want to add more color coding right now. I can see why this would be interesting, but we've had a few similar proposals and we're pretty much uniformly set against adding any more color indicators aside from who the current leader is.


Backport HHH-14529 + HHH-14530 to branch 5.4 - XML mapping improvements to allow for compile-time parsing in Quarkus

HHH-14530: Adding pre-parsed Binding<?> instance to MetadataSources
HHH-14529: Move from dom4j to JAXB for orm.xml parsing

Backport of #3884 , which must be merged first.

Closing, as it seems we will introduce JAXB-based XML mapping in 5.5 only, after all.


RuntimeError: CUDA error: an illegal memory access was encountered

Any idea why this might be happening?
`Starting initialization
Downloading: "https://download.pytorch.org/models/resnet18-5c106cde.pth" to /root/.cache/torch/checkpoints/resnet18-5c106cde.pth
Loading training dataset metadata:

Can use 57874 images from the kitti (kitti_split) train set for depth training
Can use 2975 images from the cityscapes train set for segmentation training
Loading validation dataset metadata:
Can use 1159 images from the kitti (kitti_split) validation set for depth validation
Can use 500 images from the cityscapes validation set for segmentation validation
Summary:
Model name: sgdepth_chetan
Logging directory: /cmudi001-sgd-1/SGDepth/Checkpoints/sgdepth_chetan_test/sgdepth_chetan
Using device: cuda (GeForce GTX 1080 Ti)
100%|██████████| 46827520/46827520 [00:00<00:00, 57113495.00it/s]
Epoch 0:
kitti_kitti_train_depth losses at epoch 0 (batch 0):

avg 0.1202

cityscapes_train_seg losses at epoch 0 (batch 0):

cross_entropy: 2.9558

Breakdown of time spent this epoch:

unaccounted: 0.000
loading: 9.133
optimizer: 0.000
transfer: 0.075
forward: 3.802
THCudaCheck FAIL file=/opt/conda/conda-bld/pytorch_1556653114079/work/aten/src/THC/THCCachingHostAllocator.cpp line=265 error=77 : an illegal memory access was encountered
Traceback (most recent call last):
File "train.py", line 372, in
trainer.train()
File "train.py", line 340, in train
self._run_epoch()
File "train.py", line 266, in _run_epoch
loss.backward()
File "/opt/conda/envs/torch_110/lib/python3.7/site-packages/torch/tensor.py", line 107, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph)
File "/opt/conda/envs/torch_110/lib/python3.7/site-packages/torch/autograd/init.py", line 93, in backward
allow_unreachable=True)  # allow_unreachable flag
RuntimeError: CUDA error: an illegal memory access was encountered`

Usually I do not get such errors, however cuda errors are somewhat cryptical in most cases. Did you change anything in the code with respect to the standard configuration?
Also: Did you already try to run the code on a CPU, just to verify that it is running? If you get an error on the CPU sometimes it is more readable than on the GPU. The code automatically detects if it can be run on GPU and if not available it runs on the CPU.

I did not change anything in the code except for correct paths to data.
I tried running on CPU and it trains without any errors. For now, I will keep this issue open and try to debug around. Thank You.

After following some solutions on PyTorch discussions forum, the error is changed to this:
Traceback (most recent call last):
File "train.py", line 372, in
trainer.train()
File "train.py", line 340, in train
self._run_epoch()
File "train.py", line 249, in _run_epoch
outputs = model(batch)
File "/opt/conda/envs/torch_110/lib/python3.7/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/cmudi001-sgd-1/SGDepth/models/sgdepth.py", line 292, in forward
x = self.seg(*x)
File "/opt/conda/envs/torch_110/lib/python3.7/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/cmudi001-sgd-1/SGDepth/models/sgdepth.py", line 79, in forward
x = self.decoder(*x)
File "/opt/conda/envs/torch_110/lib/python3.7/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/cmudi001-sgd-1/SGDepth/models/networks/partial_decoder.py", line 134, in forward
x = self.blocksf'step_{step}'
File "/opt/conda/envs/torch_110/lib/python3.7/site-packages/torch/nn/modules/module.py", line 722, in _call_impl
result = self.forward(*input, **kwargs)
File "/cmudi001-sgd-1/SGDepth/models/networks/partial_decoder.py", line 69, in forward
x_new = torch.cat((x_new, x_skp), 1)
RuntimeError: cuda runtime error (700) : an illegal memory access was encountered at /opt/conda/conda-bld/pytorch_1595629403081/work/aten/src/THC/THCCachingHostAllocator.cpp:278


Search results titles should be heading level 2 for easy access.

This is already happening for boxed audio results, but image results should have their sr-only titles updated to use h2 instead of span.

@obulat You :wave: reacted in Make slack when I asked about this issue. Are you planning on taking it on and if so can you assign yourself to it? If not I'll keep looking for another volunteer :slightly_smiling_face:


Is it possible to provide pre-built dist files in repo?

Hi there,
I'm wondering if you can provide pre-built dist files in repo? Just for the release git tags, so that we can host this lib on cdnjs, and it's good to have an official build for users, thanks.

It'll be great if you can also provide pre-built files for videojs-contrib-ads repo, thanks.

I have added the build files to the latest release tag. @incompl could you add the build files to contrib-ads when you get a chance?

@PeterDaveHello does it need to be in the tag or would having it be in github releases be sufficient?

I probably won't be able to get around to this for awhile, if someone else wants to do it.

Hi all, thanks for your comments, the pre-built files need to be in the tagged commits only, so that cdnjs auto updater can recognize and update the new versions automatically, thanks!

@forbesjo I only see the files in the latest release, but not in the tagged commit, is it possible to be in the tagged commit?

Would it be possible for cdnjs to pickup the npm package for this repo and videojs-contrib-ads? It seems like that's how videojs-contrib-hls is set up https://github.com/cdnjs/cdnjs/blob/master/ajax/libs/videojs-contrib-hls/package.json

Yes we can pickup npm packages if there are distribution purpose files.

Most of the videojs related projects publish to npm with the distribution files showing up in dist/. Picking the files up from npm would be our preferred method as well.

Since there is no npm package info in the homepage and readme, so I actually don't know that, is there a list of the videojs related npm packages?

There's https://www.npmjs.com/package/videojs-contrib-dash, https://www.npmjs.com/package/videojs-contrib-ads, https://www.npmjs.com/package/videojs-contrib-hls, and https://www.npmjs.com/package/videojs-swf

Thanks @gkatsev, I'll take a look at these packages.


[2.4] Add Istio 1.4.800

rancher/rancher#26784

Oh this is on 2.4 branch, won't actually matter for 2.4 clusters as long as start is pre-2.4, but good to have both branches be the same I guess.


Respect comment line

I have been experiencing some issue, when I use this extension with .scss it moves the comments line to the end of the previous line.
Example:
body{
width:100%;
}
//more rules

become
body{
width:100%;
}//more rules

Hi,
The extension merges the comments line with the code line only if the code is preceded of spaces or tabs. This is the behaviour of the js-beautify library used for indent the code.


Node list exported incorrectly

While it works perfectly fine on import, the behavior here does not match the original exporter.
The node list was written in a layered fashion. That is to say that we must write the children of the root node before moving on to their children.
child and sibling index appears to be incorrect. Almost as if it takes that layer and works backwards.

Fixed in e63a7042395c4e2e15910af62693609ba08faaad


Trakt TV collection does not display

Couple of days ago, I started to get the following error upon entering TV Shows - Trakt > My Collection.
Error message displayed:
BAD STATUS GETTING TRAKT COLLECTION FOR SHOWS: 500
Other menu entries related to trakt seem to work. Even My Collection for Movies in Elementum is working. Seems like the problem is only on TV Shows  collection menu entry.
Can you guys please check what could be wrong?

hi. i would suggest to create issue here - https://github.com/elgatito/plugin.video.elementum/issues - for better visibility.
it also has "issue template" with important info for debugging the issue. log file is a must (guide is inside of template).


hv.QuadMesh with 8196x6000 samples

I have to display a spectrogram of audio data and tried that with quadmesh. But it crashes my kernel. Is a maximum data size known with quadmesh? My RAM explodes.
Data ist stored as a xarray dataarray with time and frequency as dimensions.
Hover functionality would be great.

If you have uniform spacing in both axes, you should normally use hv.Image, which is more efficient than QuadMesh while still supporting hover in recent Bokeh versions (1.0+, I think). You can also combine Datashader with HoloViews to handle even larger images, and as long as you use rasterize() rather than datashade() hover should still work.  If that's still not suitable, we plan to add out-of-core rendering for rasters in Datashader, but we haven't started implementing that yet.

Thank you, I'll give hv.Image a try.

For a 8196x6000 sized array you will definitely want to use the rasterize operation.
from holoviews.operation.datashader import rasterize
rasterize(hv.Image(...))

I'll close since this is a usage question and a raw QuadMesh will never scale to these sizes in bokeh since that's too much data to send to your browser.


Ripme using default (?) api. Not the one I inputted.

Ripme version: 1.7.60
Java version:  version 10.0.2
Operating system: win10

Exact URL you were trying to rip when the problem occurred: https://overbarawatch.tumblr.com/
Please include any additional information about how to reproduce the problem:

This issue happens with any tumblr account.
Expected Behavior
Should rip the tumblr
Detail the expected behavior here.
Actual Behavior
Downloading http://api.tumblr.com/v2/blog/overbarawatch.tumblr.com/posts/photo?api_key=FQrwZMCxVnzonv90rg[redacted]&offset=0
Tumblr rate limit has been exceeded
Worth noticing that the api_key isn't the one I inputted on the rip.properties folder. Also, should the ripme.jar file be inside the source code folder?

Issue resolved itself after I downloaded the ripme.jar file and folder again.

This was most likely caused by you editing rip.properties well ripme was still running. Any changes made to rip.properties well ripme is running are overwritten when ripme exits


[prototype] New Landing Page (Sajen & Arnab)

The prototype for our new landing page (complete with value proposition, benefits, CTAs, and Unique Selling Points) is now complete. I'm asking mentors, volunteers, technical writers, and other community members to review the landing page prototype and provide feedback and suggestions; if there are any improvements that need to be made, we'll make them. You can view the new landing page here, and make a suggestion.
The above link is a prototype of the website; because there are no vertical scrollbars on this prototype, you must use the trackpad or mouse wheel to scroll down if you are using a mouse. We've also included the website design here. Please review and make sure to provide your feedback and recommendations regarding landing page designs.
Arnab has included a full explanation of the landing page design with value propositions in our mailing list, so please take a look at it in our mailing list here. You can also post your recommendations in the mailing list thread.
We'll start working on the implementation phase once the designs and value propositions are approved.

Hi @sajenjeshan1222,
I'm sorry for missing out this request. I wasn't aware of things outside pull requests. Please see my comments below and feel free to contact me if we need further talk. Thank you.

In features, please only capitalize the first letter each line, except for the technical terms. (e.g ~p~[P]rocess execution, Room ~M~[m]anagement)
why do we need a period at the end? btw, do you think if it's better if we align these three words to the same pattern? Like three adjectives? How about using "easy-to-use" to replace "open source" (I guess you could find a better word)?
In developers' portal, is there any reason that you remove the explanation section?
For each topics (developers' portal, talks, honors and awards, ...), I could see that you put the topics that are in green box on the left side, but sponsors is also on the left. In addition, I felt a bit confusing when I had to scroll my eyes left and right for the topics. How do you think if we centralized all of them?
Could you align the sequence of topics to the document order, which is: introduction, tutorials, how-to guides, reference, and explanations.

Simin

@proudofsimin Thank you for the review.

In features, please only capitalize the first letter each line, except for the technical terms. (e.g ~p~[P]rocess execution, Room ~M~[m]anagement)

I have capitalized the first letter of each line in the features and I have doubt, In the features section for example if we took the 1st point "User-interface interaction" should I capital only the first word or the upcoming word after the "-", other words it's fine I understood and capitalized the first word as you said but when there is a word with "-" should I capitalize only the first word or the upcoming word after "-" because in the case the 1st point "User-interface interaction" is an acronym (UI interaction). Like what do u mean by "except for the technical terms".

why do we need a period at the end? btw, do you think if it's better if we align these three words to the same pattern? Like three adjectives? How about using "easy-to-use" or "multi-language" to replace "open source" (I guess you could find a better word)?

I have removed the period at the end, then I have replaced the word "opensource" with "easy to use" and also I replaced the existing "Multiplatform" with "Cross platform". If you think any changes must be done here please let me know.

In developers' portal, is there any reason that you remove the explanation section?

Yes, the reason why we added the Developer's portal section in the first place because all the developers and chatbot makers and others come to wechaty to add wechaty chatbots to their projects or to use wechaty chatbots in the case that users needs it, so the developer's portal sections aims to help the users to implement the chatbot and to get started with wechaty as soon as possible, developers actually looks for information's like how to get started to use wechaty chatbots and the API references and to learn about wechaty but the explanation sections actually covers about what is RPA and what is conversational AI etc.., which I think that the developers actually knows about the AI, NLP things in the explanation section. If you think that the explanation section must be added to the developer's portal surely we can add it , please feel free to share your thought about this so that we can discuss and add the explanation section.

For each topics (developers' portal, talks, honors and awards, ...), I could see that you put the topics that are in green box on the left side, but sponsors is also on the left. In addition, I felt a bit confusing when I had to scroll my eyes left and right for the topics. How do you think if we centralized all of them?

I can understand, I have changed all the topics towards left side so that now I think users can find the topics easily.

Could you align the sequence of topics to the document order, which is: introduction, tutorials, how-to guides, reference, and explanations.

Done ! I have aligned it correctly now.
I have made the changes as I said above and you can view all the changes in the same link that you visited before to view the prototype. If there is any changes or any feedbacks please let me know. Thank you.

Hi @sajenjeshan1222,
Thank you for your reply. I really really love your design. It's just so cute!
For your question about user interface, I wasn't sure so I found Apple Style Guide to help me. Please use i for interface. Thank you.

@proudofsimin Thank you for the review, I have made the necessary changes and you can view the changes in the same link that you visited before to view the landing page prototype. If there is any changes please let me know. Thank you.

Hi @sajenjeshan1222,
No more question from my side. Appreciate your work :)

Hi Sajen and Arnab,
Thank you for creating this issue for discussing the landing page draft. After I review your great draft design image, I feel that the value proposition, benefits, CTAs, and Unique Selling Points look all good, appreciate all the efforts you have made so far!
I'd like to add the following comments to our draft:

Use minimum non-default CSS: I'd like to make our docs site re-use the default docusaurus CSS(style) as much as possible, which means that we should try our best to not create new CSS code as possible as we can. Only create CSS when there's a MUST, and make sure to manage that css in one css file.
Start wording: please start preparing/wording the text materials, like the "Features", "Voice of developers" (as well as the developer profile avatar, etc).
1 PR for 1 component: start creating the draft PR for the landing page, with each component per PR. For example, the "Features" should use one PR, and "Voice of developers" should use another PR, CSS related should use a separated PR as well.
Introduction video: start to design the video layout in our landing page, and make a placeholder for it.

I'd like to see a draft version after we have followed the above 4 comments with PR, and we can make it update to online docs to make it clear for everyone and continue to improve it and merge to master.
@proudofsimin Thanks to Simin for the styling suggestions, appreciate it!

Thank you for the review and feedback @huan

Use minimum non-default CSS: I'd like to make our docs site re-use the default docusaurus CSS(style) as much as possible, which means that we should try our best to not create new CSS code as possible as we can. Only create CSS when there's a MUST, and make sure to manage that css in one css file.

Yes, we will surely try our best to use the existing docusaurus CSS styles, we will also try to avoid not creating a new CSS code and use minimum non-default CSS. Thank you!

Start wording: please start preparing/wording the text materials, like the "Features", "Voice of developers" (as well as the developer profile avatar, etc).

In the "Voice of developers" section for the developer quotes we will be adding the quotes that you have mentioned in the blog post "Reconstruct Wechaty landing page with value propositions" as GitHub wiki page here (Voice of developer). We have the developer quotes to add in the "Voice of Developers" section but for the developer avatars we are unable to find some avatar and some developer don't have an avatar or it's missing, so I would like to ask that what can we replace in the place of missing developer avatars?
Sorry, I am unable to get you like what do u mean by start working/preparing text material like you mean to change the title of the "Features" section or you mean to add some more contents and texts under the "Features" section or you mean to start creating the PRs for those sections?

1 PR for 1 component: start creating the draft PR for the landing page, with each component per PR. For example, the "Features" should use one PR, and "Voice of developers" should use another PR, CSS related should use a separated PR as well.

Yes, I understood and we will make 1 PR for each section and for each component as you said. I think it will be really great and easy to do in this way and thank you for the instructions.

Introduction video: start to design the video layout in our landing page, and make a placeholder for it.

ok huan, we have started to design a placeholder for our introduction video on our new landing page. I will update you soon once the placeholder is ready for the introduction video in our landing page prototype, which is going to be placed in our new landing page and you can view the changes in the same link that you visited before to view the landing page prototype.

I'd like to see a draft version after we have followed the above 4 comments with PR, and we can make it update to online docs to make it clear for everyone and continue to improve it and merge to master.

Sure huan, Once the landing page prototype is completed from top to bottom with all your reviews and changes made accordingly, when this landing page prototype is finalized with all changes made then we will start working on the implementation process by producing PRs because when the landing page prototype is completed and finalized then we can just completely focus on making PRs as you said which will be best and perfect!
If there are any changes please let me know. Thank you so much for the valuable comments and review.

I feel it will be a good idea to show the 6 lines of code that we are talking about (Dingdong bot ig) on the landing page. Maybe like them https://reactjs.org/
You can also take help from some other landing page templates available on Github.

I feel it will be a good idea to show the 6 lines of code that we are talking about (Dingdong bot ig) on the landing page. Maybe like them https://reactjs.org/
You can also take help from some other landing page templates available on Github.

Yes, I understand, and once I finish the current prototype, I will add the six lines of code to the new landing page. The concept you provided is very great, and I will definitely work on it once I finish coding the current landing page, since if I finish coding the current prototype implementation, it will be easy for me to incorporate further ideas and corrections provided. Thank you very much.

Hi @sajenjeshan1222
You were talking about some voice of developers in the meeting, could you please elaborate here on what exactly you require? I mean how many voices do you want, etc

Dear @sajenjeshan1222 and @encodeArnab
Thank you very much for working on the landing page in the past weeks!
I'd like to suggest that we should start planning to create PRs for review. Please remember that we should follow the "1 PR for 1 component" rule: we start creating the draft PR for the landing page, with each component per PR.
For example, the "Features" should use one PR, and "Voice of developers" should use another PR, CSS related should use a separated PR as well.
On the other hand, I'd like to the progress of our introduction video on our landing page: have we started working on it? when could we expect it to be finished, and what's the budget for it? Please let me know and so that we can be well prepared with it.
P.S.: I love the below idea from @Rohitesh-Kumar-Jain : https://reactjs.org/ looks really good.

I feel it will be a good idea to show the 6 lines of code that we are talking about (Dingdong bot ig) on the landing page. Maybe like them https://reactjs.org/

BTW: I have found that @encodeArnab has not joined the Wechaty org as a team member yet, so I have just sent an invitation to @encodeArnab, please accept it by following the below message:

You've invited Arnab to Wechaty! They'll be receiving an email shortly. They can also visit https://github.com/wechaty to accept the invitation.

Another thing I think can be added is contrast, most landing pages have a contrast between different parts of the page, but the current landing page consists of a single color theme.
Here's an example of a website having contrast https://www.shopify.in/

Hi @sajenjeshan1222
You were talking about some voice of developers in the meeting, could you please elaborate here on what exactly you require? I mean how many voices do you want, etc

Hi @Rohitesh-Kumar-Jain I mean that I have all voice of developers and you can take a look at all the voice here. What I mean is that in the new landing page prototype there is a section called voice of developers and that section tells voice of developers and what developers think about Wechaty and you can see in the prototype, the layout that I have designed is that it has developer voice and corresponding developer photo, so I have the voice of them to add in the section but for the avatar in the voice of developer section I don't have photo to add to the corresponding voice.
Please take a look at voice of developer section in the the new landing page prototype here. [The link is a prototype of the website; because there are no vertical scrollbars on this prototype, you must use the trackpad or mouse wheel to scroll down if you are using a mouse]

Thank you for the response @huan

Dear @sajenjeshan1222 and @encodeArnab
Thank you very much for working on the landing page in the past weeks!
I'd like to suggest that we should start planning to create PRs for review. Please remember that we should follow the "1 PR for 1 component" rule: we start creating the draft PR for the landing page, with each component per PR.
For example, the "Features" should use one PR, and "Voice of developers" should use another PR, CSS related should use a separated PR as well.

Yes, we will create a single PR for a single component. We're going to finish the landing page, so once we finish some integration things in the new landing page, we'll run some tests to see if it's stable, and then we'll start producing PRs as per your instructions, and in the coming weeks you'll be able to see our PRs, I'll finish the whole landing page, and I'll also make sure the new landing page is integrated without any bugs, and then we'll start producing PRs.

On the other hand, I'd like to the progress of our introduction video on our landing page: have we started working on it? when could we expect it to be finished, and what's the budget for it? Please let me know and so that we can be well prepared with it.

For the introduction video on our new landing page we have created a placeholder for it and you can see it in our new landing page prototype link that you visited before.
The introduction video will be ready soon, and I am working with the ADVIDS team on it. I have shared the first storyboard draft of the introduction video in our community mailing list, and I have also sent you an email about the complete progress of the introduction video project. I am also ensuring that the new landing page and the introduction video are syncing properly.
Once the introduction video is ready I will let you know and the video will be ready soon in upcoming weeks, Valerie should decide on the budget, as you said to me before.

P.S.: I love the below idea from @Rohitesh-Kumar-Jain : https://reactjs.org/ looks really good.

I feel it will be a good idea to show the 6 lines of code that we are talking about (Dingdong bot ig) on the landing page. Maybe like them https://reactjs.org/

Yes, I think that this is a wonderful and good concept. Once I complete coding the new landing page and ensure that it is properly integrated and free of problems, I will begin working on this idea and implementing it, and I will create a separate PR for this task.

BTW: I have found that @encodeArnab has not joined the Wechaty org as a team member yet, so I have just sent an invitation to @encodeArnab, please accept it by following the below message:

You've invited Arnab to Wechaty! They'll be receiving an email shortly. They can also visit https://github.com/wechaty to accept the invitation.

I have informed Arnab regarding it and now he have joined.

This was the principle I was talking about in the meeting https://thelandingpagecourse.com/landing-page-design-principles-contrast/

Hi @huan when can we merge the PRs?

In #1271 you need to resolve my request, in that PR only after that we can proceed.

I have given you both quotes and images, still, it's not implemented


Introduce secured GET /api/processes and /api/deployments

For instance, POST .../list

Hi
Kudu will continue to run in Azure App Service. However, this repo will no longer be maintained.
If the problem persists and is related to running on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service


Text input

https://github.com/okp4/product/issues/468

I'll strat from scratch without icon management


Add tmfer

This is a small command that allows formatting of OCaml->OCaml.

Great! But I think refmt --parse ml --print ml would work if we make ocaml comment interleaving better!


jail.sh: uneeded (typo)

Spelling: uneeded should be unneeded.

Either correct the spelling, or use an alternative word.
https://www.thesaurus.com/browse/unneeded?s=t


Remove obsolete parameter in example code

Seems to be a remnant from the last version, right?

Thanks!


Any way to remove resource file?

For example, I want to remove a huge image file which full-screen in a slide. So I remove the shape, then remove the slides.
However, the huge image still lay in ./ppt/media.
I tried to remove or edit the Image object from the Picture shape. But I don't know how to remove an Image object.
shape._element.getparent().remove(shape._element)
slides = list(document.slides._sldIdLst)
document.slides._sldIdLst.remove(slides[page-1])

Now, how can I remove this file?

You need to remove the relationship, because that is what is traversed in the process of saving.
A relationship is between one "part" and another. In this case, between the Slide part and an Image part.

You get the slide part from the slide with something like: slide._part -> SlidePart.
You remove the relationship with something like: slide_part.drop_rel(rId)
You get the relationship-id (rId) from the original picture shape with something like: picture._pic.blip_rId.

The image part is only removed (upon saving) if there are no relationships to it. So if it is used more than once, you have to remove all those picture-shapes and their image relationships.


Swap order of parameters 'name' and 'namespace' for TPRs

Change as agreed upon, see https://github.com/kubernetes-incubator/client-python/pull/265#issuecomment-310298050.

LGTM. Thanks.


Can you sign in to a vue / cordova app?

I did the integration and everything worked, but it is not possible for the user to login to a Vue / Cordova application. Trying to log in opens a browser window

Checks if Disqus has any restrictions for this type of application.


Remove doc-comment

When I tried to make doc-comment a dev-dependency, I realized it actually only creates a mod with the file as doc string and can be replaced by two small macros.
CC @kngwyu @Alexander-N

Good catch, thanks.
I think ideally we should make it dev-deps to let the author know the crate is used.
But it's also important to make the install time shorter for users.


Added Spider Man Game

Hello Project Admin,
I have added a Spider Man Game which I mention in the issue no. #2579 .
Screenshot -

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

It is super ready to Merge

Hello @TusharKesarwani ,
please review this one.

Hello @TusharKesarwani ,
Kindly review this 👍🏻

@MohitGupta121 ,
Check now I have added drive links instead of  images.

@MohitGupta121 ,
It is ready to merge 👍🏻

@MohitGupta121 @pranjay-poddar this was already merged in another project https://github.com/kunjgit/GameZone/pull/2160 and https://github.com/GameSphere-MultiPlayer/GameSphere/pull/601

@MohitGupta121 @pranjay-poddar this was already merged in another project kunjgit/GameZone#2160 and GameSphere-MultiPlayer/GameSphere#601

so, what's the matter bro ?

@MohitGupta121 @Saurabh-kumar09 @pranjay-poddar Please remove the labels and mark this as spam as this very unfair to other contributers. Many other contributers are doing the same thing. Please take appropriate actions

Let me clear the point,
It's not about the addition of the project it's my own game and it is approved by the admin so I think no one can enter between our talks.
and Mr. Killer I saw your real profile too you're doing this because we both have same points so please do contribution by your own don't do this silly stuff if you're not a kid okay ?
& one more thing there is no mention about the contribution. We can contribute any in any tech stack we're comfortable in and it is allowed in GSSOC-23 it's just a friendly game for the users but you @killer2O making this is an issue.
So first check the guidelines as per GSSOC-23 then say anything you want.

@MohitGupta121 @Saurabh-kumar09 @pranjay-poddar Please remove the labels and mark this as spam as this very unfair to other contributers. Many other contributers are doing the same thing. Please take appropriate actions

Also I'm not spamming same game in the same project or repo. it legal.
Please go through the guidelines then contribute.

@MohitGupta121 @Saurabh-kumar09 @pranjay-poddar Please remove the labels and mark this as spam as this very unfair to other contributers. Many other contributers are doing the same thing.
I added the drive files all task is proper maintained and there is more games like snake mario and all which are repeating then why only me ? because I give my 100% and developed this by now own or just for this killer2op objection ???

For your info already blacklisted some of the contributors from leaderboard, I don't have time to check these things, this is the work of maintainers to check the PR, like you said you learnt online then do some own changes why are you copying the whole content, what if others contributors will say same like this & ranking on 1st then it will be ok for you & why are you spamming on different repos to make extra points, please check server rules.

For your info already blacklisted some of the contributors from leaderboard, I don't have time to check these things, this is the work of maintainers to check the PR, like you said you learnt online then do some own changes why are you copying the whole content, what if others contributors will say same like this & ranking on 1st then it will be ok for you & why are you spamming on different repos to make extra points, please check server rules.

Sure @gurjeetsinghvirdee no problem.
will be more careful from the next time.
@MohitGupta121 remove the lables will add different thing expect this now.
Thank You.

For your info already blacklisted some of the contributors from leaderboard, I don't have time to check these things, this is the work of maintainers to check the PR, like you said you learnt online then do some own changes why are you copying the whole content, what if others contributors will say same like this & ranking on 1st then it will be ok for you & why are you spamming on different repos to make extra points, please check server rules.

Sure @gurjeetsinghvirdee no problem. will be more careful from the next time. @MohitGupta121 remove the lables will add different thing except this now.
Thank You.

Yes be careful! I will be on lookout for you 😈😈

No Problem will see you in the leaderboard.

If you do these things here in others repo, I will ban you both.
@killer2O @SyedImtiyaz-1

Really Sorry @gurjeetsinghvirdee
But he is just doing too much.
Will be careful and I'm not going to reply on his any action now
Really Sorry

@Killer2-O But you're not a part of GSSoC 23 and I did this earlier by youtube course and I have canceled the all the PR which I have made so whats the issue now.
@gurjeetsinghvirdee you can report this guy.
He is targeting me and just stalking me with a new ID now.
Previously his ID was Killer2O and now he came up with Killer2-O
#help

lets talk on discord. give me ur discord id

give me your I'll add you.


Update in-app DDG logo if user changes app icon

Quick Design Idea:
• Small Design Idea For The Homescreen Whatever Icon You Choose Can Replace The Orange Icon On The Homescreen And Possible Privacy Tips To Help Others Out

Thank you for sharing this idea with us.
I've added a task in our backlog with this proposal and will share it with the team.
https://app.asana.com/0/0/1190561305733441/f
I'm going to close the issue since it's already documented internally.


WebView's source prop should not require whitelist for static HTML.

Description
Due to how HTML is opened in iOS, it seems that using source as {html: string} no longer works, without white-listing about:blank.
This is a possible regression/side effect of https://github.com/facebook/react-native/commit/634e7e11e3ad39e0b13bf20cc7722c0cfd3c3e28.

iOS only: Breaking Change: Restrict WebView to only http(s) URLs
Summary:
To prevent people from linking file:// or other URLs inside RN WebViews, default  to not allowing those types of URLs.
This adds the originWhitelist to specify other schemes or domains to be allowed.

Solutions
There are two potential solutions:

Add about:blank to the whiltelist.
Do not enforce the whitelist when loading about:blank for a static HTML source, passed through props.

If (1) is chosen I, @squirly, can write a fix for this.
Environment
  React Native Environment Info:
    System:
      OS: Linux 4.13 Ubuntu 16.04.5 LTS (Xenial Xerus)
      CPU: x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
      Memory: 586.73 MB / 15.55 GB
      Shell: 4.3.48 - /bin/bash
    Binaries:
      Node: 8.11.3 - ~/bin/node
      npm: 6.2.0 - ~/bin/npm
    SDKs:
      Android SDK:
        Build Tools: 23.0.1, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.1, 27.0.3
        API Levels: 23, 25, 26, 27
    npmPackages:
      react: 16.4.2 => 16.4.2
      react-native: 0.56.0 => 0.56.0

Reproducible Demo
Remove the line below and run tests.
https://github.com/facebook/react-native/blob/a286c0e75358dd3d917df51189ee306ef00a5e7e/IntegrationTests/WebViewTest.js#L67
Related Error
[warn][tid:com.facebook.react.JavaScript] Possible Unhandled Promise Rejection (id: 0):
Error: Unable to open URL: about:blank

Stacktrace
createErrorFromErrorData@http://localhost:8081/index.bundle?platform=ios&dev=true&minify=false:2268:26
__invokeCallback@http://localhost:8081/index.bundle?platform=ios&dev=true&minify=false:2656:23
__guard@http://localhost:8081/index.bundle?platform=ios&dev=true&minify=false:2559:15
invokeCallbackAndReturnFlushedQueue@http://localhost:8081/index.bundle?platform=ios&dev=true&minify=false:2406:21
invokeCallbackAndReturnFlushedQueue@[native code]

@react-native-bot This should be tagged iOS, and the Linux tag should be removed.

Migrated to https://github.com/react-native-community/react-native-webview/issues/26 and can be closed here.


New Updates

:warning: This is a Pull Request Template :warning:

Checkout all the things you've done by placing an *x*\ within the square braces.

Proposed Changes

[x] I've forked the repository.
[x] I've created a branch and made my chagnes in it.
[ ] I've read the CODE OF CONDUCT and abide to it.
[ ] I've read the CONTRIBUTING.md
[ ] I understand opening a PULL REQUEST doesn't mean it will be merged for sure.

Replaced with #539


build: upgrade to bazel 7.2

Changes are visible to end-users: no
Test plan

Covered by existing test cases

Same errors as 7.3 so lets do that instead: https://github.com/aspect-build/rules_js/pull/1969


Add staking.blue gentx

Please make sure your node is up and running. Join #validators Discord for help: https://discord.gg/QeJWCrE.


I think "space between" is not working as expected when using function components

Describe the bug
Space between (space-y-4) is not creating margins when using function components.
To Reproduce
Steps to reproduce the behavior:

https://codesandbox.io/s/xenodochial-panini-besf0x In this example, I have a margin-top
const MyComponent = (props) => {
  return
;
};

export default function App() {
  return (

Hello CodeSandbox

      <MyComponent />

Start editing to see some magic happen!

  );
}

https://snack.expo.dev/@markitu/hello-world In this example, I don't have a margin-top
import React from 'react';
import { withExpoSnack } from 'nativewind';

import { View } from 'react-native';
import { styled } from 'nativewind';

const StyledView = styled(View)

const MyComponent = (props) => {
  return (

      <MyComponent />

    </StyledView>
  );
}

export default withExpoSnack(App);

from #127

I think I can see your issue, you are using components that do not accept a style prop as children.
<View className="flex-row items-center space-x-4">
          <Text className="font-black text-3xl">{order.id.slice(-5)}</Text>
          <BadgeStatus status={order.status} />
          <BadgePayment paid={order.isPaid} />
</View>

This will not work as BadgeStatus does not accept style as a prop.

So in your example, spreading the props should fix the issue
const MyComponent = (props) => {
  return (


Does not handle missing or blank kubeconfig files gracefully

Describe the bug
If a file or path in KUBECONFIG env var is missing or kubeconfig file is blank, kdash exits with a crash report. It would be better if the missing kubeconfigs are either ignored or a cleaner error message is presented.
To Reproduce

Add a non-existing path, a path without a kubeconfig  or a blank kubeconfig file in KUBECONFIG environment variable
Start kdash

Expected behavior

Either: invalid config is ignored and kdash functions normally
Or: clean error output stating missing kubeconfig path, instead of crash report

Desktop (please complete the following information):

MacOS 11.5
iTerm2 3.4.8
Bash  5.1.8
Kubectl Client Version 1.21.0
Kubectl Server Version 1.19
KDash version 0.2.3
Installed from homebrew

Thanks for the report. And yes I agree that a graceful error would be nice here


Doc/upload jupyter meeting mins 9.5.22

Description
Pull Request to add Jupyter Notebook Meeting Minutes file for 9.5.22
Fixes # (issue)
Nil
Type of change

[X] Documentation (update or new)

How Has This Been Tested?
N/A
Testing Checklist:
N/A
Checklist:
N/A

Document has spelling and formatting issues. Will close this pull request and submit a new one later.


Add ZipCode Service and Controller

In this PR, we add an endpoint /api/zipcode/get that can be used to get information about the location of a particular zipcode.
dev deployment here: http://team01-rkibel-dev.dokku-07.cs.ucsb.edu
closes #17

lgtm


Preparation for Azure.Core.Experimental release 0.1.0-preview.30

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


JS Engine written in Rust runs on WASM

I recommend making some comment about this in the top page of this repo!
I feel it is very exciting.
https://maekawatoshiki.github.io/rapidus/

That's true. It can attract people.


Docker image auto-publishing failing to upload

Command is failing with following message:
#!/bin/sh -eo pipefail
echo "$DOCKERHUB_PASSWORD" | docker login -u $DOCKERHUB_LOGIN --password-stdin
docker push $IMAGE_NAME

Error response from daemon: Get https://registry-1.docker.io/v2/: unauthorized: incorrect username or password
Exited with code 1

cc @tomaka

I see. Unfortunately since I'm not really watching the repo anymore, I don't really want to give to put the password allowing access to all my Docker images. Anyone could change IMAGE_NAME to something else and overwrite my other images.

We could set up an organization on Docker hub? I think you get one repository for free.

I'd be fine with that.


Table of contents

Added links to every sections in table of contents.
This will make it easier to navigate in document.
This change also allows you to see all sections that are in document.

Thank you.
This might add some maintenance overhead - since every section requires a headline added to the top. It the text is more read than written, so it would be easier to read only rule that sb finds interesting.
So :+1:


Improving the Navbar file

I would like to improve the Navbar file of this project making it engaging for the users and make it more expressive.

I would like to work on this issue, please assign this issue to me under GSSoC'23.

I  can work on this issue under gssoc 23 contribution ?

I can work on this issue under gssoc 23 contribution ?

@harshsingh32 since you are already assigned to an issue, so I will assign it to you once you submit a PR for that issue

@harshsingh32 you can work on that issue


Can I scrape tweets filtering by user location / tweet location?

Is there any way to get only results from a query filtering by country?

Here's an example of tweets made within 1km of NYC's central park zoo https://twitter.com/search?q=geocode%3A40.7732108%2C-73.9773577%2C1km&src=typed_query


Colson tls

Move my updates and changes into main.

Ah. There are conflicts.
Maybe we can use the main branch as is if there wasn't anything new to the code in your branch?


Issue reported by RETIREJS does not reference CVE

Describe the bug
An issue reported by RETIREJS does not reference the CVE attached to the issue, instead it shows a bug related to the issue and CVE
Version of dependency-check used
The problem occurs using version 5.2.4 of the the gradle plugin
To Reproduce

Goto https://github.com/safetytrick/bug-dependency-check-cve-missing
View build/reports/dependency-check-report.html

OR run ./gradlew dependencyCheckAnalyze and view the newly created report

A screenshot in the repo shows the problems in the report. This issue was previously suppressed through CVE-2012-6708 but the duplicate report from RETIREJS is incomplete so the suppression doesn't apply.
Expected behavior
The vulnerability is reported together with the CVE

Maybe the only issue is minor and/or related to the quality of the data behind the RETIREJS findings? Because the finding isn't as complete as what the older analyzers discovered it reports duplicate findings and the report includes links to nowhere?

The 3rd party integrations like RetireJS and OSS Index may contain findings that do not have CVE information.  ODC does its best to bring in as much information as it can - but ultimately this is an issue with the RetireJS data.  You can still use the suppression mechanism:
<suppress>
   <notes><![CDATA[
   file name: jquery-migrate-1.2.1.js
   ]]></notes>
   <packageUrl<EMAIL_ADDRESS>   <vulnerabilityName>jquery-migrate bug: 11290</vulnerabilityName>
</suppress>

Thanks that seems right. There is minor report issue here, the report contains links relative to the current page that go nowhere. In this case it generated a link to href="11290"

Again - that is the data from RetireJS. I suppose we should put a check in for valid URLs....

The next release will have slightly better formatted output in the references section - the references without a link won't generate an anchor tag.


setup.sh: line 105 - command not found-netopeer2

Hi ,
i am getting below error
netopeer2/scripts//setup.sh: line 105: -l: command not found
netopeer2/scripts//setup.sh: line 59: : command not found
i can see issue in setup.sh as follows
optional env variable override
if [ -n "$SYSREPOCTL_EXECUTABLE" ]; then
SYSREPOCTL="$SYSREPOCTL_EXECUTABLE"
avoid problems with sudo PATH
elif [ id -u -eq 0 ]; then
SYSREPOCTL=su -c 'command -v sysrepoctl' -l $USER
else
SYSREPOCTL=command -v sysrepoctl
fi
i have not exported any env data for "SYSREPOCTL_EXECUTABLE" still it is entering if statement and "$SYSREPOCTL_EXECUTABLE" have no data.
can i know why ?

What version of Netopeer2 (which git branch, and which commit) are you using? There's been a recent fix in the devel branch that might fix this issue.

I am using Netopeer2 [2.0.35] master branch.

Please post the full log of the configure and build and installation of the devel branch.

yes sure,
root@labadmin:~sysrepo/netopeer2/build# cmake -DCMAKE_INSTALL_PREFIX:PATH=/sysrepo/install_libs ..
-- The C compiler identification is GNU 9.3.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Found PkgConfig: /usr/bin/pkg-config (found version "0.29.1")
Package libnetconf2 was not found in the pkg-config search path.
Perhaps you should add the directory containing `libnetconf2.pc'
to the PKG_CONFIG_PATH environment variable
No package 'libnetconf2' found
-- Unable to learn libnetconf2 thread support, check skipped
CMake Warning at CMakeLists.txt:170 (message):
valgrind executable not found! Disabling memory leaks tests.
-- Could NOT find CMocka (missing: CMOCKA_LIBRARIES CMOCKA_INCLUDE_DIR) (Required is at least version "1.0.1")
-- Disabling tests because of missing CMocka
-- Could NOT find Uncrustify (missing: UNCRUSTIFY) (Required is at least version "0.71")
-- Found OpenSSL: /sysrepo/install_libs/lib/libcrypto.so (found version "1.1.1d")
-- Found LibSSH: /sysrepo/install_libs/include (found suitable version "0.8.90", minimum required is "0.7.1")
-- Found LibNETCONF2: /sysrepo/install_libs/lib/libnetconf2.so
-- Looking for NC_ENABLED_SSH
-- Looking for NC_ENABLED_SSH - found
-- Looking for NC_ENABLED_TLS
-- Looking for NC_ENABLED_TLS - found
-- Looking for vdprintf
-- Looking for vdprintf - found
-- Looking for asprintf
-- Looking for asprintf - found
-- Looking for vasprintf
-- Looking for vasprintf - found
-- Looking for getline
-- Looking for getline - found
-- Looking for strndup
-- Looking for strndup - found
-- Looking for strnstr
-- Looking for strnstr - not found
-- Looking for strdupa
-- Looking for strdupa - found
-- Looking for strchrnul
-- Looking for strchrnul - found
-- Looking for get_current_dir_name
-- Looking for get_current_dir_name - found
-- Looking for pthread_mutex_timedlock
-- Looking for pthread_mutex_timedlock - found
-- Check if the system is big endian
-- Searching 16 bit integer
-- Looking for sys/types.h
CMake Warning (dev) at /usr/share/cmake-3.16/Modules/CheckIncludeFile.cmake:80 (message):
Policy CMP0075 is not set: Include file check macros honor
CMAKE_REQUIRED_LIBRARIES.  Run "cmake --help-policy CMP0075" for policy
details.  Use the cmake_policy command to set the policy and suppress this
warning.
CMAKE_REQUIRED_LIBRARIES is set to:
pthread

For compatibility with CMake 3.11 and below this check is ignoring it.
Call Stack (most recent call first):
/usr/share/cmake-3.16/Modules/CheckTypeSize.cmake:230 (check_include_file)
/usr/share/cmake-3.16/Modules/TestBigEndian.cmake:33 (CHECK_TYPE_SIZE)
CMakeModules/UseCompat.cmake:46 (TEST_BIG_ENDIAN)
CMakeLists.txt:236 (use_compat)
This warning is for project developers.  Use -Wno-dev to suppress it.
-- Looking for sys/types.h - found
-- Looking for stdint.h
-- Looking for stdint.h - found
-- Looking for stddef.h
-- Looking for stddef.h - found
-- Check size of unsigned short
-- Check size of unsigned short - done
-- Using unsigned short
-- Check if the system is big endian - little endian
-- Looking for stdatomic.h
-- Looking for stdatomic.h - found
-- Could NOT find CURL (missing: CURL_LIBRARY CURL_INCLUDE_DIR)
-- libcurl not found, url capability will not be supported
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Found LibYANG: /sysrepo/install_libs/lib/libyang.so (found suitable version "2.13.7", minimum required is "2")
-- Found Sysrepo: /sysrepo/install_libs/lib/libsysrepo.so (found suitable version "6.4.19", minimum required is "6.4.0")
-- Looking for eaccess
-- Looking for eaccess - found
-- Looking for mkstemps
-- Looking for mkstemps - found
-- Configuring done
-- Generating done
-- Build files have been written to: /sysrepo/netopeer2/build
root@labadmin:~sysrepo/netopeer2/build#
root@labadmin:~/ sysrepo/netopeer2/build# make
Scanning dependencies of target serverobj
[  4%] Building C object CMakeFiles/serverobj.dir/src/common.c.o
[  8%] Building C object CMakeFiles/serverobj.dir/src/netconf.c.o
[ 12%] Building C object CMakeFiles/serverobj.dir/src/netconf_monitoring.c.o
[ 16%] Building C object CMakeFiles/serverobj.dir/src/netconf_acm.c.o
[ 20%] Building C object CMakeFiles/serverobj.dir/src/netconf_nmda.c.o
[ 25%] Building C object CMakeFiles/serverobj.dir/src/netconf_subscribed_notifications.c.o
[ 29%] Building C object CMakeFiles/serverobj.dir/src/netconf_confirmed_commit.c.o
[ 33%] Building C object CMakeFiles/serverobj.dir/src/subscribed_notifications.c.o
[ 37%] Building C object CMakeFiles/serverobj.dir/src/yang_push.c.o
[ 41%] Building C object CMakeFiles/serverobj.dir/src/log.c.o
[ 45%] Building C object CMakeFiles/serverobj.dir/src/err_netconf.c.o
[ 50%] Building C object CMakeFiles/serverobj.dir/src/netconf_server.c.o
[ 54%] Building C object CMakeFiles/serverobj.dir/src/netconf_server_ssh.c.o
[ 58%] Building C object CMakeFiles/serverobj.dir/src/netconf_server_tls.c.o
[ 58%] Built target serverobj
Scanning dependencies of target netopeer2-server
[ 62%] Building C object CMakeFiles/netopeer2-server.dir/src/main.c.o
[ 66%] Building C object CMakeFiles/netopeer2-server.dir/compat/compat.c.o
[ 70%] Linking C executable netopeer2-server
[ 70%] Built target netopeer2-server
Scanning dependencies of target netopeer2-cli
[ 75%] Building C object cli/CMakeFiles/netopeer2-cli.dir/main.c.o
[ 79%] Building C object cli/CMakeFiles/netopeer2-cli.dir/commands.c.o
[ 83%] Building C object cli/CMakeFiles/netopeer2-cli.dir/completion.c.o
[ 87%] Building C object cli/CMakeFiles/netopeer2-cli.dir/configuration.c.o
[ 91%] Building C object cli/CMakeFiles/netopeer2-cli.dir/linenoise/linenoise.c.o
[ 95%] Building C object cli/CMakeFiles/netopeer2-cli.dir/__/compat/compat.c.o
[100%] Linking C executable ../netopeer2-cli
[100%] Built target netopeer2-cli
root@labadmin:~/ sysrepo/netopeer2/build#
root@labadmin:~/sysrepo/netopeer2/build# make install
[ 58%] Built target serverobj
[ 70%] Built target netopeer2-server
[100%] Built target netopeer2-cli
Install the project...
-- Install configuration: "Debug"
-- Up-to-date: /sysrepo/install_libs/share/yang/modules/netopeer2
-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Installing: /sysrepo/install_libs/bin/netopeer2-server
-- Set runtime path of "/sysrepo/install_libs/bin/netopeer2-server" to ""
-- Up-to-date: /sysrepo/install_libs/share/man/man8/netopeer2-server.8
-- Installing missing sysrepo modules...
/sysrepo/netopeer2/scripts//setup.sh: line 102: -l: command not found
/sysrepo/netopeer2/scripts//setup.sh: line 56: : command not found
CMake Error at cmake_install.cmake:85 (message):
scripts/setup.sh failed: 127
make: *** [Makefile:86: install] Error 1
root@labadmin:~/sysrepo/netopeer2/build#

What commit are you on? This looks like you haven't updated to 05d9ac4 yet.

Development Branch expecting  LIBNETCONF2_INCLUDE_DIR) (Required is at least version "3.1.0").  But i can see only available version is  2.0.24

Most recent versions are always found in the devel branch in our projects.


Pulling in Instant Articles changes from 0.8.x-release

LGTM

Coverage increased (+0.3%) to 85.369% when pulling 4abfd1182edc882381216e43926d4aac74e805a5 on 0.8-instant_articles into 46ab537e41f834f29fac2cd52dca11f37d94b75b on master.

Coverage increased (+0.06%) to 85.146% when pulling 6689f13e0055657441d94693f33c8b073893796e on 0.8-instant_articles into 46ab537e41f834f29fac2cd52dca11f37d94b75b on master.

Coverage increased (+0.3%) to 85.379% when pulling 296d85de9cebdb0efe3454730069e4f0cbad53c5 on 0.8-instant_articles into 46ab537e41f834f29fac2cd52dca11f37d94b75b on master.

@spra85 added the additional dfp_position targeting to this branch

👍 @MichaelButkovic

Coverage increased (+0.04%) to 85.409% when pulling f76c12c3fb3d13be45374abe29d72264ab26185f on 0.8-instant_articles into 18c6f3470ee0e3cef7956db0f53458bfb72e3985 on master.


Bug fixes

Changes

fix sum bugs

Related Issues/ PRs

Fixes #141
Fixes #138
Fixes #137

Codecov Report

Merging #143 (0933103) into develop (b148fe3) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop     #143      +/-   ##
=============================================
- Coverage      56.64%   56.55%   -0.10%
  Complexity       612      612
=============================================
  Files            127      127
  Lines           2468     2472       +4
  Branches         248      248
=============================================
  Hits            1398     1398
- Misses           972      976       +4
  Partials          98       98

Impacted Files
Coverage Δ

src/main/java/seedu/ibook/logic/LogicManager.java
0.00% <0.00%> (ø)

...in/java/seedu/ibook/model/util/SampleDataUtil.java
0.00% <ø> (ø)

src/main/java/seedu/ibook/ui/MenuToolbar.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/ibook/ui/table/ProductCard.java
0.00% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Refactor and fix lib_root computation.

@nshahan @natebosch
I accidentally closed #67 PTAL

@DrMarcII Can you bring in the recent changes to master #70 to make sure we keep the presubmit tests passing?

Rebased.


Handle error cases for mean and variance.

This resolves #20 with the following changes:

Calling mean on an empty matrix returns an empty matrix vector.
Calling variance on a matrix with one or fewer rows/columns in the working axis returns an Error with kind InvalidArg.

Thanks for getting this in! :D
I added a few inline comments. It would also be great if you could add some tests to cover the failure cases. If there are any non-doc tests we should make sure these are unwrapping too. I think the following cases should cover it:
These three can be covered in one test:

Variance with empty data should return an error.

Variance over row with 1 row should err.

Same with column

Matrix should return an empty Vector if matrix is empty.

I did not notice any non-doc tests for mean or variance so I only created doc tests.
I believe all but the "variance of one column" situation were in the doc tests, so I'll add that one along with the other fixes.

I think this should cover all the above changes. Let me know if I missed a spot.

All of the code looks good but I'm afraid I'm going to have to be difficult, sorry!
I'd prefer if the failure test cases were in unit tests in that module rather than docs. When I first started I stuck a lot of tests in the docs but have tried to move away from that a little more to keep the examples easy to read.
I think it's enough to have one test each for matrix and vector. The matrix one can just test the empty matrix case returns an empty Vector. Maybe it's worth documenting this behavior in the matrix docs too?
Sorry again for asking you to do more than you already have.

I'd prefer if the failure test cases were in unit tests in that module rather than docs. When I first started I stuck a lot of tests in the docs but have tried to move away from that a little more to keep the examples easy to read.

Not a problem, I was going to put it in tests/mat/mod.rs originally but I just wasn't 100% sure if you actually wanted them outside of docs.

I think it's enough to have one test each for matrix and vector.

Not 100% sure what you mean by "one test each for matrix and vector", since aren't these just methods on Matrix<T>?

The matrix one can just test the empty matrix case returns an empty Vector. Maybe it's worth documenting this behavior in the matrix docs too?

I was thinking the "empty mean" test would just test that case, yes. The variance test will be checking for errors in n == 0 and both n == 1 cases. As for documenting empty mean in the docs, yeah, I can toss a note in there noting that it does not fail, but rather returns an empty matrix.

Sorry the "matrix and vector" was just my tiredness. I meant mean and variance.

How is this?

Thanks for putting up with my nitpicks. Once you've cleaned up the examples I'll be more than happy to get this merged!

Just pushed. Let me know if there's any other minor changes!

This is perfect, thanks! Once the CI passes I'll merge.


run exhaustive dep check on referenced query key

fixes https://github.com/TanStack/query/issues/4475

@TkDodo Something odd just happened. The Lint job has failed on eslint errors that occurred before I made this PR. I fixed them in https://github.com/TanStack/query/pull/4480/commits/f14f185249bf0d4e7d5190bf3e8ba2ce7db60b48.

I merged the other PR first so now they are conflicts here

There seems to be another flaky test.
About the issue I wrote above, it seems like I was wrong. I had 1 problem from this PR and a few warnings which won't fail in CI. What's the reasoning for not marking all rules as problems when running as CI?

Codecov Report
Base: 96.36% // Head: 92.40% // Decreases project coverage by -3.95% :warning:

Coverage data is based on head (571b609) compared to base (eab6e2c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4480      +/-   ##
==========================================
- Coverage   96.36%   92.40%   -3.96%
==========================================
  Files          45       88      +43
  Lines        2281     3648    +1367
  Branches      640      956     +316
==========================================
+ Hits         2198     3371    +1173
- Misses         80      261     +181
- Partials        3       16      +13

Impacted Files
Coverage Δ

src/react/reactBatchedUpdates.ts

src/devtools/Explorer.tsx

src/react/useIsMutating.ts

src/devtools/tests/utils.tsx

src/react/utils.ts

src/react/setBatchUpdatesFn.ts

...rc/createWebStoragePersistor-experimental/index.ts

src/core/retryer.ts

src/core/focusManager.ts

src/core/queriesObserver.ts

... and 123 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

About the issue I wrote above, it seems like I was wrong. I had 1 problem from this PR and a few warnings which won't fail in CI. What's the reasoning for not marking all rules as problems when running as CI?

eslint rules should be either errors or warnings, independent of environment. We can make some rules from warning to error if they are important to be fixed.


remove unused unit test leftover

Codecov Report
Patch and project coverage have no change.

Comparison is base (dd73d2e) 60.47% compared to head (082c980) 60.47%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #404   +/-   ##
=======================================
  Coverage   60.47%   60.47%
=======================================
  Files          34       34
  Lines        5252     5252
=======================================
  Hits         3176     3176
  Misses       2076     2076

Impacted Files
Coverage Δ

zingolib/src/lightclient.rs
62.29% <ø> (ø)

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


feat(translations): add Slovak translation

Description
I have added a translation into Slovak.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Chore (non-breaking change which does not add functionality)

Looks good @MrFriggo thanks for your contribution!


[#540] Verify excluded paths on camel spring module

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/wildfly-camel-pullreq/395/
Test PASSed.


[ie/roosterteeth] support bonus features

Adds support for RoosterTeeth "bonus features" (as discussed in #9355)
Caveats:

bonus features are available on the show level, that means they do not have a season, season_id or season_number
the series title is usually the show title (available via parent_content_title)
legacy /bonus-feature/ urls redirect to /watch/ urls (example found here)
the series extractor only fetches bonus features when no season is specified

Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

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

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

I refactored the entries extraction. Check it out, test it, etc and let me know what you think

looks great! I didn't even know about yield from, pretty nice way to handle this case.

Thanks for your help, @bashonly!


Broken version 2.1.1: bump to cpptrace 0.7.2

The new version of libassert 2.1.1 is broken on MacOS and older versions of Ubuntu. Look to bump the cpptrace dependency to include the fix for https://github.com/jeremy-rifkin/cpptrace/issues/174 to solve this.

The exact fix is https://github.com/jeremy-rifkin/cpptrace/commit/54a3e6fdf7837c44d20436c77d3469f4524bf6a1, but no 0.7.2 of cpptrace has yet to be created to pull this in. Could open a further issue in cpptrace if wanted.

Thanks for opening this, I haven't had a chance to look into this yet. I should be able to tomorrow or Friday.
Another fix for anyone running into issues would be to use a newer version of cmake (with recognition that that's not always as easy as an apt install etc).

No worries. Just opened an issue to track its progress. Take it when you have time :+1:

Now handled in dev, will aim to do a release soon

2.1.2 is released with the fix


Publish to NPM

Such a great library, please make it possible to install from npm

@cheapsteak @mpetroff
+1

I wrote a package.json file for Pannellum a while back but never got any further than that. I can go look at it again.

I just published it to NPM:
https://www.npmjs.com/package/pannellum

thanks!


Upgrade to 0.1.2-RC1

Might not have been synced to maven yet though!

Let's relaunch test tomorrow morning.

👍


Minify XML?

Can we safely use Minify_HTML::minify($xml, array('xhtml' => true)); in order to minify any xml content ?

It was not intended/tested for this purpose. It may work, but will certainly modify markup embedded unescaped in CDATA sections, etc.