Added gaussian_blur_op

This PR is aimed at putting a gaussian blur in in an image. It reduces the noise in an image.
See Issue #1333

@seanpmorgan @gabrieldemarmiesse @WindQAQ @autoih Please provide a review

@ghosalsattam This is PR requires a lot of work to get to an acceptable state:

You can't put the file in the base directory, put in under tensorflow_addons/image.
Update BUILD in tensorflow_addons/image.
The naming scheme of files should be file_name.py, not fileName.py.
You've implemented the op as an object, whereas it should actually be implemented as a function. See some of the other ops in tensorflow_addons/image to get a better idea.
You should use tf for the implementation instead of np. (ones, expand_dims).
All core TF ops should be used via the public API. So use tf.nn.convolution directly. Remove all tensorflow.python.* imports.
The testing code should run using pytest.
The test should compare the output with other standard image processing libraries like PIL or sklearn-image.

@Squadrick I am trying to use tf.Tensor to find Gaussian Kernel.But when I am using the function assign in a variable of type tf.Variable I am getting the error 'tensorflow.python.framework.ops.EagerTensor' object has no attribute 'assign'
Can you please help me with the problem as I am new to tensorflow open source community.
Thank You

@ghosalsattam Can you post the code that's causing the error.

Ok

def findKernel(sigma,kSize,gaussianFilter):
"This function creates a kernel of size [kSizekSize]"
for i in range(-kSize//2,kSize//2+1):
for j in range(-kSize//2,kSize//2+1):
gaussianFilter[i+kSize//2][j+kSize//2].assign(1/(2np.pi*(sigma)2)*np.exp(-(i2+j2)/(2*sigma2)))
print(gaussianFilter)
return
gaussianFilter=tf.Variable(tf.zeros([5,5]))
findKernel(1,5,gaussianFilter)

`def findKernel(sigma,kSize,gaussianFilter):
"This function creates a kernel of size [kSizekSize]"
for i in range(-kSize//2,kSize//2+1):
for j in range(-kSize//2,kSize//2+1):
gaussianFilter[i+kSize//2][j+kSize//2].assign(1/(2np.pi*(sigma)2)*np.exp(-(i2+j2)/(2*sigma2)))
print(gaussianFilter)
return
gaussianFilter=tf.Variable(tf.zeros([5,5]))
findKernel(1,5,gaussianFilter)

@Squadrick Please help with the code I provided above

Ok I solved it. Thank you for the support and sorry for the inconvenience caused.

@Squadrick plaese provide a review.

@seanpmorgan @gabrieldemarmiesse @WindQAQ @autoih @Squadrick  Please provide a review

Fixes Issue #1353

@ghosalsattam we usually provide review when all the tests in the CI are passing. Here you have some failed tests. You can look at the logs by clicking on "details".

@Squadrick @gabrieldemarmiesse One test case is not running Build release wheels (ubuntu-18.04, 3.5) (pull_request) Failing after 9m — Build release wheels (ubuntu-18.04, 3.5)
Can you help me with the issue.

@ghosalsattam The failure is due to the added dep of OpenCV. Since we already use PIL for other tests, why not use PIL.ImageFilter.GaussianBlur as the ground truth for testing your function?

@ghosalsattam @gabrieldemarmiesse Please give a review

@Squadrick @gabrieldemarmiesse Please provide a review

@Squadrick @gabrieldemarmiesse @facaiy Please provide a review

@ghosalsattam Maintainers are nearly all volunteers working on Addons during their free time, and the list of pull requests is quite long, it may take a few days, sometimes weeks to get a review depending on the size of the diff. No worries, as long as the build is passing and the pull request is opened, you'll get a review at some point.

@Squadrick I messed up pushing commits n the master branch. Now I want to create new branch and push commits. But I am unable to do so because master branch has commits. When I am making new PR from the new branches the PR contain the older commits.I am having few more files fixing other issues. Can you help me out of the situation.

@Squadrick Any More changes you suggest?

@gabrieldemarmiesse @Squadrick   I have implemented all the changes you suggsted.
But I am getting few errors which were not coming earlier. All of them related to publish wheels PyPi.
Can You help. Link-https://github.com/ghosalsattam/addons/

@ghosalsattam Pushing to pypi requires a login and password that is only available in the original addons, not it's normal that it fails for this step in your fork. I'll re-open the pull request, that should work out.

@gabrieldemarmiesse I have implemented a gaussian kernel which is sequence of 2 1d convolution one in x direction and other in y-direction. This is implemented in other libraries like scipy, opencv etc as this method is faster. All the tests are running as suggested by you. Please provide a review.

@gabrieldemarmiesse any more changes you suggest.

@gabrieldemarmiesse Please provide a review or any more suggestions.

@WindQAQ @gabrieldemarmiesse @Squadrick I have done all the changes suggested. Kindly provide a review at the earliest.

@seanpmorgan I have done all the changes suggested by you. Please provide a review.
I just have one query in my mind. First I had opened the PR at tensorflow/tensorflow repository as GSOC contribution. There I was asked to open the PR here. But I have seen tf.image which containg similar operations and Gaussian_Blur() functionality is missing even over there. Is it that if the PRs opened here, if they are working properly are migrated to tensorflow main repository later. I am a new guy to tensorflow/addons and sorry if the question is not making sense. Thank you for the help.

@ghosalsattam just few more changes in the main file. Almost there!

@ghosalsattam just few more changes in the main file. Almost there!

@AakashKumarNain what more changes are you suggesting?

@ghosalsattam just few more changes in the main file. Almost there!

@AakashKumarNain what more changes are you suggesting?

Check my comments in tensorflow_addons/image/filters.py file. I will put the question here as well.
def _get_gaussian_kernel(sigma, filter_shape, channels):
    "This function creates a kernel of size [filter_shape]"
    x = tf.range(-filter_shape // 2 + 1, filter_shape // 2 + 1)
    x = tf.math.square(x, tf.float64)

Is there any reason why we are making float64 default here? Is there any overflow? If not, I would suggest to change this to float32 as well

Thanks LGTM! @seanpmorgan thoughts?

@seanpmorgan @autoih @AakashKumarNain I have updated the files addressing the issues raised by you. Is it in a state to be merged or need some more refinements.

@seanpmorgan Is the PR in a state to bemerged now?

@bhack @WindQAQ Sorry for delayed response.
I had run the program on GPU and noted the time for 1D-sequence and 2D convolution. In most of the cases 2D Convolution was faster.
Here are few outputs:
filter size       Image size                     1d-Sequence time              2d filter time(s)
(5,5)            (20,1600,1600,1)                 4.25  s                                  4.12
(5,5)            (20,1600,1600,1)                .00065 s                                .00073
(7,7)            (20,1600,1600,1)                 2.91   s                                  2.49
(7,7)            (20,1600,1600,1)                 .00045 s                               .00043
So I have updated the program. If the program is running on GPU then 2D convolution technique is used. Else the sequence of 1D convolution is used. I am sorry that I earlier did not have the knowledge of the working of GPU. Thanks for your support.

And if is It TPU?

@bhack For that what are you suggesting? Shall I pass a parameter that shall act as a checker to use '2D kernel' or sequence of 1D kernel. I dont have that experience with TPUs and I surfed the net, but got no valuable article to check if tenorflow is running on TPU. Can you please help me out? Thank you

Check https://colab.research.google.com/notebooks/tpu.ipynb

@bhack I have gone through your code. From the code it seems that I have to pass a variable_flag responsible for passing device hint(since the tpu variable would be created somewhere else, if at all created). Shall I pass a new parameter giving device information?

@WindQAQ @bhack What are your suggestions on passing an extra parameter to choose the type of convolution?

I have the impression the we don't need to handle all these things. But in some way to inform DepthwiseConv2D that we have a rank=1 kernel param and all the other stuff needs to be handled by the compilation/transformation stack.
I don't know why we don't have this interface.

So as per now what should be done? Leave it as it is or pass another parameter to select the type of convolution(as done in case of switching between kmeans and kmeans++ algorithm).

@karmel Do you know anyone internal that could comment about why the DepWhiseConv2d don't handle the kernel rank=1 case extra parameter for down the stack compiler/transformations?
@ghosalsattam in the meantine can you try to benchmark It again activating experimental_compile param in tf.function?

@bhack experimental_compile activation is not recommended for public APIs in tensorflow/addons. I am reverting the changes.

experimental_compile activation is not recommended for public APIs in tensorflow/addons

@ghosalsattam I don't find a reference to this.
But in any case it was just to see your new results as in https://github.com/tensorflow/addons/pull/1450#issuecomment-620530971

@bhack check addons/tools/testing/source_code_test.py::test_no_experimental_api

@karmel Do you know anyone internal that could comment about why the DepWhiseConv2d doesn't handle the kernel rank=1 case extra parameter for down the stack compiler/transformations?

@reedwm or @sanjoy - can you advise, or recommend someone else?

check addons/tools/testing/source_code_test.py::test_no_experimental_api

@ghosalsattam Check also @alextp post at https://github.com/tensorflow/addons/issues/1752#issuecomment-621322109

@bhack  I had checked the GPU time as said by you(in experiment_compile=True). It took .9sec for 1D convolution and .6sec for 2D convolution for shape(1,1600,1600,1).Please provide a review.

@ghosalsattam and how do those compare with the hand-written kernel times? And are you excluding warm-up costs (measuring warm-up costs for both approaches independently would also be nice)

@alextp I don't think that we had an hand-written kernel here for gaussian blur. We are just talking about a twice call to DepWhiseConv2d TF ops when we have rank=1 kernel (separable filter) vs a single call to DepWhiseConv2d with 2d kernel.
I suppose that is would be hard to fuse on trace two DepWhiseConv2d on the separable filter right?
So I was asking why the DepWhiseConv2d in TF doesn't handle the kernel rank=1 case with an parameter for down the stack compiler/transformations?

Ah, sorry, I misread the thread.
@rmlarsen do you know who understands our depthwise conv kernels enough to answer this?

Somehow I cannot comment on that thread? Here is what I would like to say:
"I believe @bhack is correct that this could be accomplished with two
1-dimensional convolutions, which would reduce the number of flops.
However, if the kernel is small and the image large, reading the image
twice might slow down the overall computation, compared to using a single
Conv2D call. We should really have a specialized op for separable kernels
like this. (Do we?)"
On Wed, Apr 29, 2020 at 10:25 AM Alexandre Passos<EMAIL_ADDRESS>wrote:

Ah, sorry, I misread the thread.
@rmlarsen https://github.com/rmlarsen do you know who understands our
depthwise conv kernels enough to answer this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/1450#issuecomment-621352450,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEA72DUQWBZTWI4GSSMS2BTRPBPJLANCNFSM4LUXGCNA
.

Correct me if I'm wrong, but from my experience, I think there are (at
least) 3 distinct regimes for separable kernels (assuming a large image):

Tiny kernels: Use brute-force conv2d.
Medium kernels: Use "brute force" kernel, but take advantage of
separability, one block at a time.
Large kernels: Use FFT.

On Wed, Apr 29, 2020 at 10:34 AM Rasmus Munk Larsen<EMAIL_ADDRESS>wrote:

Somehow I cannot comment on that thread? Here is what I would like to say:
"I believe @bhack is correct that this could be accomplished with two
1-dimensional convolutions, which would reduce the number of flops.
However, if the kernel is small and the image large, reading the image
twice might slow down the overall computation, compared to using a single
Conv2D call. We should really have a specialized op for separable kernels
like this. (Do we?)"
On Wed, Apr 29, 2020 at 10:25 AM Alexandre Passos <
<EMAIL_ADDRESS>wrote:

Ah, sorry, I misread the thread.
@rmlarsen https://github.com/rmlarsen do you know who understands our
depthwise conv kernels enough to answer this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/1450#issuecomment-621352450,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEA72DUQWBZTWI4GSSMS2BTRPBPJLANCNFSM4LUXGCNA
.

@rmlarsen I don't know if also the device specific knowledge it is going to play a role in the transform stack.

@rmlarsen P.s. I don't know if we have already this device specific knowledge or we need to waiting for something like a more advanced machine retargetability in MLIR

Yes, the device will definitely play a role and, if nothing else, determine
where the boundaries of the regimes are.
On Wed, Apr 29, 2020 at 11:31 AM bhack<EMAIL_ADDRESS>wrote:

@rmlarsen https://github.com/rmlarsen P.s. I don't know if we have
already this device specific knowledge or we need to waiting for something
like a more advanced machine retargetability in MLIR
https://llvm.discourse.group/t/rfc-enhancing-machine-retargetability-in-mlir/885
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/1450#issuecomment-621386365,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEA72DTYKIVGATV6UVTUNWTRPBXAVANCNFSM4LUXGCNA
.

@rmlarsen @alextp Do you think it makes sense to track part of this in a new issue on the TF repo or is it not interesting enough for other use cases?

@bhack For now what should I do to this PR in order for it to be brought to merging state?

@ghosalsattam I think that you could use the 2d kernel here for merging and that you could track this on a DepthwiseConv2D specific issue on the TF repo after waiting for TF team feedback to see if the topic has a "general enough public".

And for CPU?

@ghosalsattam I think you can ignore the device and that it is better that DepthwiseConv2D will have directly this knowledge instead of having conditions here in the op so it knows how to propagate down the stack.
In proposal ticket on TF I hope that you could ask to pass the rank=1/separability arg directly (or something similar to inform it about the separate filter).

Ok!!!
On Thu, 30 Apr 2020 at 00:42, bhack<EMAIL_ADDRESS>wrote:

@ghosalsattam https://github.com/ghosalsattam I think you can ignore
the device and that it is better that DepthwiseConv2D will have directly
this knowledge instead of having conditions here in the op so it knows how
to propagate down the stack.
In proposal ticket on TF I hope that you could ask to pass the
rank=1/separability arg directly (or something similar to inform it about
the separate filter).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/addons/pull/1450#issuecomment-621407601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJMBF2CHLHIFYJ5HJS2QNMTRPB3ZXANCNFSM4LUXGCNA
.

@bhack CAn it be merged now?

You need to wait for a reviewer with write power.

@WindQAQ  Can it be merged now?

@WindQAQ I have addressed all your changes. Can it be merged now?

@rmlarsen @alextp The feature request for Tensorflow is at https://github.com/tensorflow/tensorflow/issues/39050. Please close it if you evaluate that it is not relevant enough.

@bhack According to me this should be implemented in all other conv ops where this situation may arise like along wiith depthwise conv operation-  Conv2D, Conv3D,...

@all I learnt many new things from this contribution and cant wait for having new contributions. Thanx for your support.

@ghosalsattam Thanks for the contribution!


Error: namespace "cooperative_groups" has no member "labeled_partition", when build gsplat from source

I clone the newest gsplat code, and try to install it by:
pip install -e .
But i meet the following error:
gsplat/cuda/csrc/projection.cu(485): error: namespace "cooperative_groups" has no member "labeled_partition"
auto warp_group_g = cg::labeled_partition(warp, gid);
                         ^

gsplat/cuda/csrc/projection.cu(510): error: namespace "cooperative_groups" has no member "labeled_partition"
          auto warp_group_c = cg::labeled_partition(warp, cid);
                                  ^

gsplat/cuda/csrc/projection.cu(825): error: namespace "cooperative_groups" has no member "labeled_partition"
      auto warp_group_g = cg::labeled_partition(warp, gid);
                              ^

gsplat/cuda/csrc/projection.cu(869): error: namespace "cooperative_groups" has no member "labeled_partition"
          auto warp_group_c = cg::labeled_partition(warp, cid);
                                  ^

gsplat/cuda/csrc/projection.cu(1423): error: namespace "cooperative_groups" has no member "labeled_partition"
          auto warp_group_g = cg::labeled_partition(warp, gid);
                                  ^

gsplat/cuda/csrc/projection.cu(1469): error: namespace "cooperative_groups" has no member "labeled_partition"
          auto warp_group_c = cg::labeled_partition(warp, cid);

It seams the namespace cooperative_groups did not implement the function labeled_partition.
How can i solve this problem?

Hi! gsplat requires a modern GPU that has compute capability >= 7.0.
You can check your GPU's compute capability at here

Im seeing the same error using a 3090 with compute capability 8.6
311.2       gsplat/cuda/csrc/projection.cu(1469): error: namespace "cooperative_groups" has no member "labeled_partition" 311.2 311.2       6 errors detected in the compilation of "gsplat/cuda/csrc/projection.cu". 311.2       error: command '/usr/local/cuda/bin/nvcc' failed with exit code 1 311.2       [end of output] 311.2 311.2   note: This error originates from a subprocess, and is likely not a problem with pip. 311.2   ERROR: Failed building wheel for gsplat 311.2   Running setup.py clean for gsplat 312.9 Failed to build gsplat 312.9 ERROR: Could not build wheels for gsplat, which is required to install pyproject.toml-based projects

@realimposter Sorry I think also CUDA>=11.0 is required. labeled_partition is a new feature introduced in CUDA 11: https://www.cs.nthu.edu.tw/~ychung/slides/CSC4005/CUDA_C_Programming_Guide.pdf

ahh ok thanks I added ENV TORCH_CUDA_ARCH_LIST="8.0;8.6", and it works now

Hi! gsplat requires a modern GPU that has compute capability >= 7.0.
You can check your GPU's compute capability at here

My GPU has compute capability >= 6.1 and CUDA==11.8, so gsplat==1.0.0 and nerfstudio==1.1.3 do not work anymore?
Thanks for your reply!

@JerryDeepl  Unfortunately yes.

@JerryDeepl Unfortunately yes.

What are the differences between the gsplat 1.0.0 version and the 0.1.12 version in nerfstudio? If the code for gsplat 1.0.0 in nerfstudio is implemented with 0.1.12, can the same training results be achieved?
Thanks for your reply!

1.0.0 improves the speed and memory usage by a large margin. It also fixes the gradients w.r.t cameras. So if you are not optimizing camera and are not sensitive to runtime/GPU usage. You should be able to get the same training results.

1.0.0 improves the speed and memory usage by a large margin. It also fixes the gradients w.r.t cameras. There are also a long list of more features supported in 1.0.0. But if you are not optimizing camera and are not sensitive to runtime/GPU usage. You should be able to get the same training results.

Got it, thanks a lot

Edit your shell configuration file: Open your shell configuration file using a text editor. Commonly used shell configuration files include:

vi ~/.bashrc (for Bash shell)

Add the environment variable: At the end of the file, add the following line:
export TORCH_CUDA_ARCH_LIST="8.6"

Replace "8.0;8.6" with the desired CUDA architecture list for your GPU. gpu

Save and close the file.

Activate the changes: Run the following command to activate the new environment variable:
source ~/.bashrc

https://github.com/nerfstudio-project/nerfstudio/issues/2727#issue-2066103535


seamless switching and reconnect

fixes #1638

a few small things! looks good!!!

Reviewed 9 of 11 files at r1, 2 of 2 files at r2.
Review status: 11 of 13 files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

src/generic_core/uproxy_core.ts, line 236 [r2] (raw file):
will it be a problem here if we startGet before we stop any existing remote instances?

src/generic_ui/scripts/ui.ts, line 229 [r2] (raw file):
effectively, does null mean stop whatever proxy is running? could we do instanceId: this.instanceGettingAccessFrom_ (or do you feel like that seems to verbose? I think it makes it a little more clear but it also feels kind of long)

src/generic_ui/scripts/ui.ts, line 542 [r2] (raw file):
I'm confused by when we would run this piece of code instead of the else if (this.instance...) just below.
If data.instanceId != this.instanceGettingAccessFrom_ then I feel like this.mapInstanceIdToUser_[data.instanceId].isSharingWithMe should already = false.

src/generic_ui/scripts/ui.ts, line 717 [r2] (raw file):
super super nit: "instanceId: null" (moving the ":")

Comments from the review on Reviewable.io

ptal

Review status: 11 of 13 files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

src/generic_core/uproxy_core.ts, line 236 [r2] (raw file):
Currently that's not technically possible.

src/generic_ui/scripts/ui.ts, line 229 [r2] (raw file):
I agree with this but I believe the reason for not giving that explicitly was that I'd have to make that field public and this way it's sort of internal.
I don't actually feel strongly about either approach though.

src/generic_ui/scripts/ui.ts, line 542 [r2] (raw file):
You start getting access form instanceId1;
You start getting access from instalceId2;
You stop getting access from instanceId1.

src/generic_ui/scripts/ui.ts, line 717 [r2] (raw file):
Done.

Comments from the review on Reviewable.io

Review status: 11 of 13 files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

src/generic_core/firewall.spec.ts, line 12 [r3] (raw file):
Can you comment on why these changes are needed?  It doesn't look like you are otherwise changing the firewall in this pull request, seems more like you might be hitting issues related to https://github.com/uProxy/uproxy/issues/1503

Comments from the review on Reviewable.io

Yes, and I filed  #1737.

Review status: 3 of 13 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

src/generic_ui/scripts/ui.ts, line 229 [r2] (raw file):
sounds good; can you add a comment to stopGettingInUiAndComment that says null means stop existing proxy, because the instance isnt public?

src/generic_ui/scripts/ui.ts, line 564 [r2] (raw file):
aha! thanks

Comments from the review on Reviewable.io

Review status: 3 of 13 files reviewed at latest revision, 3 unresolved discussions, some commit checks pending.

src/generic_ui/scripts/ui.ts, line 247 [r2] (raw file):
Done.

Comments from the review on Reviewable.io

:+1:


undefined is not an object (evaluating '_module.getCurrentPosition')

Description
when i use baidu map, we have a tabbar in activity, there are four fragments ,two native and two use react, when switch to MatchFragment ,the err come.
undefined is not an object (evaluating '_module.getCurrentPosition')

Geolocation.js:88:16

Reproduction Steps and Sample Code

This is react-native-baidu-map issue / StackOverflow - Not a RN issue.
Make shure you have linked the library and import it correct!


Issue with key file address being misinterpreted

Checklist

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

[tcp @ 0000025f1d62d300] Starting connection attempt to <IP_ADDRESS> port 443
[tcp @ 0000025f1d62d300] Successfully connected to <IP_ADDRESS> port 443
[hls,applehttp @ 0000025f1d62bd40] HLS request for url 'https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-10800.ts', offset 0, playlist 0
[hls,applehttp @ 0000025f1d62bd40] Opening 'https://www.***.com/*****/*****/ratatouille-the-musical/./ttBearer-1080.key' for reading
[tcp @ 0000025f1dac3d80] Starting connection attempt to <IP_ADDRESS> port 443
[tcp @ 0000025f1dac3d80] Successfully connected to <IP_ADDRESS> port 443
[https @ 0000025f1dae4040] HTTP error 403 Forbidden
Unable to open key file https://www.***.com/*****/*****/ratatouille-the-musical/./ttBearer-1080.key
[hls,applehttp @ 0000025f1d62bd40] Opening 'crypto+https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-10800.ts' for reading
[tcp @ 0000025f1dcc0480] Starting connection attempt to <IP_ADDRESS> port 443
[tcp @ 0000025f1dcc0480] Successfully connected to <IP_ADDRESS> port 443
[AVIOContext @ 0000025f1dc5a780] Statistics: 349104 bytes read, 0 seeks
[hls,applehttp @ 0000025f1d62bd40] HLS request for url 'https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-10801.ts', offset 0, playlist 0
[hls,applehttp @ 0000025f1d62bd40] Opening 'crypto+https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-10801.ts' for reading
[tcp @ 0000025f1deb6840] Starting connection attempt to <IP_ADDRESS> port 443
[tcp @ 0000025f1deb6840] Successfully connected to <IP_ADDRESS> port 443
[AVIOContext @ 0000025f1d6db340] Statistics: 348176 bytes read, 0 seeks
[hls,applehttp @ 0000025f1d62bd40] HLS request for url 'https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-10802.ts', offset 0, playlist 0
[hls,applehttp @ 0000025f1d62bd40] Opening 'crypto+https://www.***.com/*****/*****/video/ratatouille-the-musical/ratatouille-captions-10802.ts' for reading
[tcp @ 0000025f1dcc0a40] Starting connection attempt to <IP_ADDRESS> port 443
[tcp @ 0000025f1dcc0a40] Successfully connected to <IP_ADDRESS> port 443
[AVIOContext @ 0000025f1dac3d80] Statistics: 347040 bytes read, 0 seeks
[hls,applehttp @ 0000025f1d62bd40] HLS request for url 'https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-10803.ts', offset 0, playlist 0
[hls,applehttp @ 0000025f1d62bd40] Opening 'crypto+https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-10803.ts' for reading
[tcp @ 0000025f1d6f9080] Starting connection attempt to <IP_ADDRESS> port 443
[tcp @ 0000025f1d6f9080] Successfully connected to <IP_ADDRESS> port 443
[hls,applehttp @ 0000025f1d62bd40] Error when loading first segment 'https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-10800.ts'
[AVIOContext @ 0000025f1d6fd840] Statistics: 36848 bytes read, 0 seeks
[AVIOContext @ 0000025f1db813c0] Statistics: 26018 bytes read, 0 seeks
https://www.***.com/*****/*****/ratatouille-the-musical/ratatouille-captions-1080.m3u8: Invalid data found when processing input

Description

Exact URLs redacted for copyright purposes, I purchased this myself and wish to download an offline copy. I can provide them as well as full verbose logs for testing privately if necessary.
I'm unable to download this video due to youtube-dl misinterpreting the URL within the .m3u8 file.
One of the arguments in the m3u8 file is URI="./ttBearer-1080.key". This should eventually lead to the key file held at https://www.***.com/static/*****/ratatouille-the-musical/ttBearer-1080.key. However, youtube-dl seems to instead try to download the key file from https://www.***.com/static/*****/ratatouille-the-musical/./ttBearer-1080.key.
Using both a web browser and VLC player I am able to open and watch the video in full. However, youtube-dl is unable to download the video. I believe the issue here is with youtube-dl incorrectly interpreting the m3u8 file, and failing to download the key file.

What exactly am I supposed to do? I can't go linking to content that's behind a paywall in my post.

What exactly am I supposed to do? I can't go linking to content that's behind a paywall in my post.


New SEM Page Inline Issues

Example: https://main--shredit--stericycle.aem.live/en-us/marketing/shred-it-document-destruction-inline
o	:shred-it-a-20stericycle-20solution-logo-white: showing on the page
o	Form should be inline to the right of the content not below

o	25 Off Badge and text missing

@MarquiseRosier the 25% off badge is rendering a bit off. Text should be centered and no line spacing between words. Yellow block should be perfectly aligned to the icon above.
EDS:

Live:

will be fixed in the referenced PR


[Feature]: Support Guided Decoding in LLM entrypoint

🚀 The feature, motivation and pitch
Currently we support guided decoding of (JSON, Regex, Choice, Grammar, and arbitrary JSON) in OpenAI inference server. It would be great that we expose the same functionality in the offline interface as well.
https://docs.vllm.ai/en/latest/serving/openai_compatible_server.html#extra-parameters
Concretely this would mean adding the support here as a new parameter to generate call. Using methods introduced in #2819.
Do make sure to add test and examples.
Alternatives
No response
Additional context
No response

Will work on this.

Any update on this?

still working on this, sorry about the delay

I'm trying to understand why this support is considered missing if you can already do this:
from vllm import LLM, SamplingParams
from outlines.serve.vllm import JSONLogitsProcessor
from pydantic import BaseModel, conlist
import datetime as dt

class Output(BaseModel):
    names: conlist(str, max_length=5)
    organizations: conlist(str, max_length=5)
    locations: conlist(str, max_length=5)
    miscellanous: conlist(str, max_length=5)

llm = LLM('mistralai/Mistral-7B-v0.1', max_model_len=10_000, gpu_memory_utilization=0.9)
logits_processor = JSONLogitsProcessor(schema=Output, llm=llm.llm_engine)
logits_processor.fsm.vocabulary = list(logits_processor.fsm.vocabulary)
prompt = """
Locate all the names, organizations, locations and other miscellaneous entities in the following sentence:
"Charles went and saw Anna at the coffee shop Starbucks, which was based in a small town in Germany called Essen."
"""
sampling_params = SamplingParams(max_tokens=128, temperature=0, logits_processors=[logits_processor])

t0 = dt.datetime.now()
llm.generate([prompt] * 256, sampling_params=sampling_params)
time_elapsed = (dt.datetime.now() - t0).total_seconds()
print(f"Generation took {time_elapsed:,} seconds.")

(Example taken from https://github.com/vllm-project/vllm/issues/3087).
Is the point of this issue to make the use of guided decoding more intuitive for the user?

Yes just for ease of use, and a way to better reset the processor.

how to use in openai api（[OpenAI Compatible Server]）？

Please use the extra parameters listed here

Please use the extra parameters listed here

Is there a version limit? Exception, is there a complete call example, thank you.

Sorry I don't have time to come up with an full example. You can take a look at the tests (e.g. tests/entrypoints/openai/test_chat.py) for some inspiration. As for the version limit, you can set the version of the docs via the bottom right menu and find the earliest one that documents these parameters

Sorry I don't have time to come up with an full example. You can take a look at the tests (e.g. tests/entrypoints/openai/test_chat.py) for some inspiration.
As for the version limit, you can set the version of the docs via the bottom right menu and find the earliest one that documents these parameters.

ok,thanks.
If version 0.4.2[https://docs.vllm.ai/en/v0.4.2/serving/openai_compatible_server.html] is used, an error is reported：
{'object': 'error', 'message': "[{'type': 'extra_forbidden', 'loc': ('body', 'extra_body'), 'msg': 'Extra inputs are not permitted', 'input': {'guided_json': {'type': 'object', 'properties': {'name': {'type': 'string'}, 'age': {'type': 'integer'}, 'skills': {'type': 'array', 'items': {'type': 'string', 'maxLength': 10}, 'minItems': 3}, 'work_history': {'type': 'array', 'items': {'type': 'object', 'properties': {'company': {'type': 'string'}, 'duration': {'type': 'number'}, 'position': {'type': 'string'}}, 'required': ['company', 'position']}}}, 'required': ['name', 'age', 'skills', 'work_history']}, 'guided_decoding_backend': 'outlines'}}]", 'type': 'BadRequestError', 'param': None, 'code': 400}
input：
{'model': 'qwen1_5_7b', 'temperature': 0.3, 'top_p': 0.3, 'max_tokens': 1024, 'messages': [{'role': 'system', 'content': 'you are a helpful assistant'}, {'role': 'user', 'content': "Give an example JSON for an employee profile that fits this schema: {'type': 'object', 'properties': {'name': {'type': 'string'}, 'age': {'type': 'integer'}, 'skills': {'type': 'array', 'items': {'type': 'string', 'maxLength': 10}, 'minItems': 3}, 'work_history': {'type': 'array', 'items': {'type': 'object', 'properties': {'company': {'type': 'string'}, 'duration': {'type': 'number'}, 'position': {'type': 'string'}}, 'required': ['company', 'position']}}}, 'required': ['name', 'age', 'skills', 'work_history']}"}], 'extra_body': {'guided_json': {'type': 'object', 'properties': {'name': {'type': 'string'}, 'age': {'type': 'integer'}, 'skills': {'type': 'array', 'items': {'type': 'string', 'maxLength': 10}, 'minItems': 3}, 'work_history': {'type': 'array', 'items': {'type': 'object', 'properties': {'company': {'type': 'string'}, 'duration': {'type': 'number'}, 'position': {'type': 'string'}}, 'required': ['company', 'position']}}}, 'required': ['name', 'age', 'skills', 'work_history']}, 'guided_decoding_backend': 'outlines'}}

Only guided_choice is supported in that version.

How are you accessing the endpoint? extra_body is only for OpenAI client.

How are you accessing the endpoint? extra_body is only for OpenAI client.

yes，my endpoint is using vllm openai server，like ：xxxx/v1/chat/completions

Can you show your code?

@DarkLight1337
SQL_GRAMMER = """
    ?start: select_statement

    ?select_statement: "SELECT " column_list " FROM " table_name (join_clause)? (where_clause)? (order_by_clause)? (limit_clause)?

    ?column_list: column_name ("," column_name)*

    ?table_name: identifier

    ?join_clause: "JOIN " table_name " ON " condition

    ?where_clause: "WHERE " condition

    ?order_by_clause: "ORDER BY " column_name ("ASC" | "DESC")?

    ?limit_clause: "LIMIT " INT

    ?condition: identifier "=" value

    ?column_name: identifier
    ?value: STRING | INT

    ?identifier: /[a-zA-Z_][a-zA-Z0-9_]*/
    ?STRING: /'[^']*'/
    ?INT: /[0-9]+/
"""

chat_completion = client.chat.completions.create(
    messages=[
        {"role": "system", "content": system_prompt},
        {
        "role": "user",
        "content": [
            {
                "type": "text",
                "text": user_prompt
            }
        ],
        }
    ],
    model=model,
    temperature=0,
    top_p=0.1,
    extra_body={
        "guided_grammer": SQL_GRAMMER
    },
)

I'm having the same issue with it. I'm using 0.6.4post1

@DarkLight1337
SQL_GRAMMER = """
    ?start: select_statement

    ?select_statement: "SELECT " column_list " FROM " table_name (join_clause)? (where_clause)? (order_by_clause)? (limit_clause)?

    ?column_list: column_name ("," column_name)*

    ?table_name: identifier

    ?join_clause: "JOIN " table_name " ON " condition

    ?where_clause: "WHERE " condition

    ?order_by_clause: "ORDER BY " column_name ("ASC" | "DESC")?

    ?limit_clause: "LIMIT " INT

    ?condition: identifier "=" value

    ?column_name: identifier
    ?value: STRING | INT

    ?identifier: /[a-zA-Z_][a-zA-Z0-9_]*/
    ?STRING: /'[^']*'/
    ?INT: /[0-9]+/
"""

chat_completion = client.chat.completions.create(
    messages=[
        {"role": "system", "content": system_prompt},
        {
        "role": "user",
        "content": [
            {
                "type": "text",
                "text": user_prompt
            }
        ],
        }
    ],
    model=model,
    temperature=0,
    top_p=0.1,
    extra_body={
        "guided_grammer": SQL_GRAMMER
    },
)

I'm having the same issue with it. I'm using 0.6.4post1

You made a typo. The field is called "guided_grammar", not "guided_grammer".


Question about MaxProcessingTime timeouts

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.

Sarama
Kafka
Go

v1.29.1
2.6.0
1.17

Configuration
What configuration values are you using for Sarama and Kafka?
Default configuration.
Problem Description
The documentation mentions for MaxProcessingTime:
		// The maximum amount of time the consumer expects a message takes to
		// process for the user. If writing to the Messages channel takes longer
		// than this, that partition will stop fetching more messages until it
		// can proceed again.
		// Note that, since the Messages channel is buffered, the actual grace time is
		// (MaxProcessingTime * ChannelBufferSize). Defaults to 100ms.
		// If a message is not written to the Messages channel between two ticks
		// of the expiryTicker then a timeout is detected.
		// Using a ticker instead of a timer to detect timeouts should typically
		// result in many fewer calls to Timer functions which may result in a
		// significant performance improvement if many messages are being sent
		// and timeouts are infrequent.
		// The disadvantage of using a ticker instead of a timer is that
		// timeouts will be less accurate. That is, the effective timeout could
		// be between `MaxProcessingTime` and `2 * MaxProcessingTime`. For
		// example, if `MaxProcessingTime` is 100ms then a delay of 180ms
		// between two messages being sent may not be recognized as a timeout.
		MaxProcessingTime time.Duration

When this timeout is crossed (~25s with defaults) the log line
			Logger.Printf("consumer/broker/%d abandoned subscription to %s/%d because consuming was taking too long\n",
				bc.broker.ID(), child.topic, child.partition)

is logged, but processing happens normally.
Question
I cannot understand what is the downside of this timeout being triggered, nor why it exists in first place.
Any chance for some clarification? I can make a PR to update the documentation afterwards.
Thanks!

I guess the purpose of abandoning the subscription is that the slow consumer must release assigned topics/partitions to allow faster consumers in a group to take over it.

if a partition consumer consume too slow, the channel is full， no need to fetch the partition data from broker

Closing as presumed to have been answered. Please reopen if not


T558: Hyperconnected ultraconnected are not biconnected

Add T558: hyperconnected + ultraconnected + $|X|\ge 4$ ==> not biconnected.
Also add a ref for T551, which I had missed before.

T558 can instead replace T555.

Good point.  That's what I had intended to do, and then forgot about it in the haste to get this out.


update retryable and viable/strict workflows

cc @seemethere

Hmmm, I checked that the API key exists. There were no changes in the code meanwhile. I guess that there might be some issue while trigger this workflow at the PR level. The changes look correct. Let's merge this PR and follow the workflow.


[163] Issue #163 - July 2, 2020

To contribute to this issue, simply leave a comment here. Please also review our contributing guidelines.
The current draft for this issue in _drafts/. If you want to contribute directly, feel free to open a pull request.

@fassko would you be interested in writing this one? Note that this one will probably be packed, given it includes everything WWDC... ;-)

Yes I would like to 🙂

https://spec.fm/podcasts/swift-unwrapped/fxMk4ipF

https://developer.apple.com/forums/

Returning to an old hobbyhorse: Migrating higher order function names to comply with API guidelines
https://forums.swift.org/t/returning-to-an-old-hobbyhorse-migrating-higher-order-function-names-to-comply-with-api-guidelines/37728

Extend SwiftPM PackageDescription to introduce metadata
https://forums.swift.org/t/extend-swiftpm-packagedescription-to-introduce-metadata/37722

Revisiting the source compatibility impact of SE-0274: Concise magic file names
https://forums.swift.org/t/revisiting-the-source-compatibility-impact-of-se-0274-concise-magic-file-names/37720

https://swiftrocks.com/benefits-of-throwing-functions-try-swift-underrated-feature

Override implicit default access modifier as fileprivate per file
https://forums.swift.org/t/override-implicit-default-access-modifier-as-fileprivate-per-file/37775

https://twitter.com/slava_pestov/status/1274914543630782464

https://developer.apple.com/documentation/xcode-release-notes/xcode-12-beta-release-notes

https://twitter.com/s4tfnews/status/1275157350287134724

Fun one https://twitter.com/Valzevul/status/1275164878056103936

https://developer.apple.com/wwdc20/10644
Check out Use Swift on AWS Lambda with Xcode from #wwdc20

https://twitter.com/Lukasaoz/status/1275167892510826502

https://www.swiftbysundell.com/podcast/75

Another "finally" option: https://twitter.com/jckarter/status/1275423569552400384

https://twitter.com/mousiechika/status/1275547535206166531

Swift on Server Workgroup June 10th, 2020 meeting notes
https://forums.swift.org/t/june-10th-2020/37863

https://www.wwdcnotes.com/notes/wwdc20/10170/

https://twitter.com/AirspeedSwift/status/1276235796429721600

https://twitter.com/tomerdoron/status/1276616136541396992

https://forums.swift.org/t/compound-variable-names/37963

https://oleb.net/2020/as/

https://twitter.com/slashmodev/status/1277730788226859010

https://github.com/slashmo/awesome-swift-nio

https://forums.swift.org/t/introduce-anonymouskeypath/38080

https://forums.swift.org/t/accepted-se-0282-interoperability-with-the-c-atomic-operations-library/38050

https://forums.swift.org/t/swift-package-registry-service-security/38088

https://twitter.com/stephencelis/status/1278347619794911232
https://twitter.com/stephencelis/status/1278347623359971328
https://github.com/apple/swift/pull/32451


add timeout multishot flag

It is part of the liburing 2.4 since
this commit.

@ianic Thank you for your contribution


UI update causes modification of client side transforms/entity hierarchy

I encountered a bug where using a UI hook would modify transform of the prefab model instance.
Video
I believe it is forcing some update from server causing the child-parent relationship be modified.
Repo Branch: https://github.com/devjobe/ambient_third_person_character/tree/TransformBug
Spawning prefab on client bugs:
https://github.com/devjobe/ambient_third_person_character/blob/917cfddb26fce3da75059ae7548987cc321a1652/src/client.rs#L30
Moving it to server side works:
https://github.com/devjobe/ambient_third_person_character/blob/917cfddb26fce3da75059ae7548987cc321a1652/src/server.rs#L18
I believe animation controller is innocent it's just causing the transforms to be updated. Nothing happens when it's removed due to the transforms not being touched by anything.

@FredrikNoren one for you when you get back

Hm that's bizarre. @devjobe can you try dumping the client side ecs before and after it breaks, and then run a diff on them to see what changed?

I don't know when but I can get to it eventually if you want to assign it to me. It's just not a priority for me since I can spawn it on server side and I've more urgent things to deal with atm. thesis is top prio until monday atleast.

@devjobe Thanks, assigned

this has nothing to do with UI update I guess. I got this when there is no UI at all

@VinhTruongAmbient Is taking a look, we suspect it's got something to do with gpu ecs or gpu mesh buffers. There's a minimal example here: https://github.com/AmbientRun/fbxbug/tree/main


Include VIZ images in repo.

VIZ images are currently GitHub "user images". Due to a recent VIZ change, these images no longer load in VIZ due to their cross-origin headers. Images committed to the repo should work. So these images must (once again) become part of the repo.

Looks resolved.


[Power Automate] Increasing PA visibility from VBA differences doc

Fixes #292 and #295.

Docs Build status updates of commit 56f9a60:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/resources/vba-differences.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit af9e326:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/resources/vba-differences.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add screen orientation for Android

http://caniuse.com/#feat=screen-orientation
It's more accurate when the keyboard shows up on the screen.

@neolwc Definitely makes sense, I actually have a version of the RotateScreen component using orientation (I can send it over if you want it). I'm going to give matchMedia a test (see how it handles the android keyboard) and based on that, will either integrate this or matchMedia.
One thing to note, some devices, specifically some Android Tablets, report their orientation incorrectly, 0 is actually landscape instead of portrait. For that case, i found its useful to compare the orientation and w/h on the first load, to see if orientation is correct.

Confirmed, this works well, thanks!


Update btp-operator resources to the newest version regardless of the BtpOperator CR state

Reason
We have a situation where the newest BTP Manager still operates with the old btp-operator version.
AC

[ ] 1) Write an e2e test proving the lack of upgrade (should we run it only when releasing or on every PR, right now e2e upgrade tests are executed only in the release pipeline)
[ ] 2) Improve reconciliation loop to upgrade always btp-operator if a new version is provided in the manager

PR with e2e upgrade test: https://github.com/kyma-project/btp-manager/pull/591

PR with reconciliation fix: https://github.com/kyma-project/btp-manager/pull/596


chore: Adjust Application update tutorial to match UI onboarding

Ticket: MEN-6681
External Contributor Checklist

🚨 Please review the guidelines for contributing to this repository.

[ ] Make sure that all commits follow the conventional commit specification for the Mender project.

The majority of our contributions are fixes, which means your commit should have
the form below:
fix: <SHORT DESCRIPTION OF FIX>

<OPTIONAL LONGER DESCRIPTION>

Changelog: <USER-FRIENDLY-CHANGE-DESCRIPTION> or <None>
Ticket: <TICKET NUMBER> or <None>

[ ] Make sure that all commits are signed with git --signoff. Also note that the signoff author must match the author of the commit.

Description
Please describe your pull request.
Thank you!

To go with https://github.com/mendersoftware/mender-demo-artifact/pull/268 and https://github.com/mendersoftware/gui/pull/3870


Can't obtain any information from opensea due to cloudflare

I've added addtionalWait (up to 1 minute) to get html.content(), but it seems that opensea blocks me from it.
Note, it works when getting nft information from opensea testnet. Any tips?

Hi @dcts , Thanks for your reply. I just tried that AWS works for this part


Test ACTSTracking Processors Using GitHub Actions

This depends on #1. Marking as draft until it is merged. However the code/methology is ready for review.
Implements automated testing of different Processors in the built images. This ensures that all packages included in a release are working together in a consistent manner.
This PR only tests the ACTSTracking package. However the framework proposed here can be extended to other packages.
The new workflow is as follows:

Build image (as in #1)
Generate a small number of events (10 single muons for now)
Run Marlin steering files on 2.

Image Building
The images are built as in #1. The only addition is uploading an additional label for the final image with the git SHA: mucoll-ilc-framework:${{github.sha}}-centos8. This is to have a uniquely identifiable image for each run of a workflow to be used in subsequent jobs. DockerHub is used as storage to share the images between jobs. A possible downside is pollution of the DockerHub repository.
Event Generation
This stage runs ddsim on sim steering files that are stored in the test/ directory. For now this is 10 events of single muons. However more configurations can be matrixed into this job.
Events are transferred between sim and test jobs using GitHub Action caching mechanism. The hash of the sim steering file is used as simulation key. This saves time by only running simulation only when changes are detected in the steering file. However it is not sensitive to changes in the DD4hep and GEANT libraries.
Running Marlin
Runs Marling on several steering files provided by individual packages. The location of steering files is matrixed into the job. This allows to scale this stage to multiple steering files and run the jobs in parallel.

Apart from this being relatively old/outdated, I think this is great and we should set up these kind tests systematically.
I wonder though if this is the right place for this test to live in. Shouldn't it rather go to the ACTSTracking repo?

Apart from this being relatively old/outdated, I think this is great and we should set up these kind tests systematically. I wonder though if this is the right place for this test to live in. Shouldn't it rather go to the ACTSTracking repo?

Potentially both? This was moving towards having an encompassing set of tests for everything that we want to check when making a release.


Feat/cmdrequest handler

Pull Request Test Coverage Report for Build 295095966

43 of 57 (75.44%)  changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 58.239%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/newrelic-infra/newrelic-infra.go
0
1
0.0%

pkg/integrations/v4/manager.go
30
33
90.91%

internal/agent/cmdchannel/handler/featureflag_handler.go
0
4
0.0%

internal/integrations/v4/runner/runner.go
3
9
33.33%

Files with Coverage Reduction
New Missed Lines
%

internal/integrations/v4/runner/runner.go
1
84.57%

Totals

Change from base Build 293152317:
0.02%

Covered Lines:
11024

Relevant Lines:
18929

💛  - Coveralls

Duplicated: https://github.com/newrelic/infrastructure-agent/pull/163


Fix bugs that cause the chapter examples to fail

Becuase it now calls self.expand_meta_graph instead of MetaMolecule.expand_meta_graph. The first works with subclassing, the second doesnt.


Improve module based on new bitcore submodule standard

requires https://github.com/bitpay/bitcore/pull/955 to be merged first
fixes https://github.com/bitpay/bitcore/issues/930

LGTM save nit

nit addressed


Fixed excel import to GingerDataSource

Import from excel wizard added checkbox to take empty columns or not

please ping me to review the PR togather

Merged, please update relevant ticket.


Afrikaans Language File

Added Afrikaans translation file
This pull request includes a

[x] Translation

I think you have to add your file in the src directory, not in the dist one

You are correct :(, i will move the file. Thanks


Added a blog on emotion detection

Checklist

Please note that in case of below checklist not updated accordingly, the maintainers have the right to immediately close your pull request.

[x] You created a branch from dev branch.
[x] You PR is raised on dev branch and not onmaster.
[x] Blog follows Blog Guidelines.
[x] You have read the Contribution Guidlines before creating this PR.
[x] Checked for any duplicate blog entry.
[x] Checked for all grammatical errors in the blog.
[x] You have not published the blog content anywhere else on the internet on platforms like medium, dev.to etc.
[x] The content is not copy-pasted from anywhere else on the Internet.

Description of change

Author Details for Blog Post Entry
name - mayank chaudhary
bio - I'm a final-year student at Maharaja Surajmal Institute of Technology, New Delhi pursuing Electronics and
Communications Engineering. I exhibit an eagerness to learn new things and adapt. Being a technology
enthusiast, I love to explore new technologies and contribute to different writing/blogging platforms. I'm
also involved with college societies and in extracurricular activities. Currently working on enhancing my
analytical skills.
twitter - rado_mayank

name - mayank chaudhary
bio - I'm a final-year student at Maharaja Surajmal Institute of Technology, New Delhi pursuing Electronics and
Communications Engineering. I exhibit an eagerness to learn new things and adapt. Being a technology
enthusiast, I love to explore new technologies and contribute to different writing/blogging platforms. I'm
also involved with college societies and in extracurricular activities. Currently working on enhancing my
analytical skills.
twitter - rado_mayank

it would be great if you could review the blog and suggest changes

@mayankchaudhary26 This is being rejected and closed as it didn't meet our guidelines and quality standards.
For more details, please refer to:
https://github.com/LoginRadius/engineering-portal/blob/master/GUIDELINES.md
https://github.com/LoginRadius/engineering-portal/blob/master/CONTRIBUTING.md


Transpose sequence

I implemented transpose sequence function which transposes list of Variables. It requires to make new LSTM function.
related to #1146

LGTM!


Support Scala 2.13.

Bump Scala 2.12.8 -> 2.12.10
Bump sbt 1.2.8 -> 1.3.2

There are a couple of additional deprecation warnings, but can't be fixed without either dropping 2.11 support, or adding separate sources for 2.11 so decided to leave as is.

Thanks! We don't need to support 2.11 any more, it's too old. Please feel free to just remove 2.11 support.

Thanks for your PR.
I've created a commit based on your PR:
https://github.com/xitrum-framework/scala-xgettext/commit/cf701b855bea15c8a106e5c0eeb10a7fa46df1e8
I've released version 1.5.4 which supports Scala 2.13.

Thanks for your PR.
I've created a commit based on your PR:
https://github.com/xitrum-framework/scala-xgettext/commit/cf701b855bea15c8a106e5c0eeb10a7fa46df1e8
I've released version 1.5.4 which supports Scala 2.13.


[Hydrogen] Remix VisualEditing is performing a full reload on change

Describe the bug
In Presentation mode, the Remix VisualEditing component is performing a full page reload when the content changes.
I'm using the VisualEditing component in a Hydrogen app.
To Reproduce
https://github.com/frontvibe/hydrogen-vite-sanity
Screenshots
https://github.com/user-attachments/assets/b5917edb-6dbb-4802-b8a9-991be1a23227
Which versions of Sanity are you using?
@sanity/cli (global)         3.62.2
@sanity/preview-url-secret    2.0.0
@sanity/react-loader        1.10.13
@sanity/visual-editing        2.4.1
sanity                       3.62.2
What operating system are you using?
macOS 15.0.1
Which versions of Node.js / npm are you running?
10.5.0
v20.12.0

I am seeing the same exact behavior on Next 15.


bug: DI and CommonModule directives not working in Ionic components

Bug Report
Ionic version:
6.10.0 using angular 9.1.7
Current behavior:

Can't bind to 'ngIf' since it isn't a known property of 'any element' --> after trying to use the ngIf & ngFor directives on a 'component' routed by a page (all generated by the ionic CLI using "ionic generate page pageName" & "ionic generate componentName"). Work fine on a Page

Component elements will not render injected components. These work fine on a Page

Expected behavior:

** First Issue **
<element *ngFor = "let x of abc">{{x.name}}</element> should not output an error
** Second Issue **

<customElement></customElement>

in a routed component on a page should just work.
Steps to reproduce:

Create a new page with "ionic generate page page-test"
in the page-test.html insert a router-outlet
create a page-test component with "ionic generate component page-test" (& configure route)
in the newly created page-test-component.ts file, insert a simple "bool = true" line just above the constructor
in the page-test-component.html file, simply insert something like "
This should work"
After reloading, the code will not work and a warning with the text "Can't bind to 'ngIf' since it isn't a known property of " will be displayed.

File and Folder structure

page-test (the page)
---- page-test.html
---- page-test.ts
---- page-test.module, etc
---- page-test (the component)
-------- page-test.component.ts
-------- page-test.component.html, etc

==================================================
For the second error i am encountering, simply create another component and inject it in the page-test-component.html. It will be completely skipped as if it were not there without any error.
Related code:
No code required as the steps outlined above are eXhaustive and concise.
Other information:

When implemented directly in a page's template (html file), the issues do not arise. Everything is rendered and all directives will be fully functional and accessible.
Ionic info:

Ionic:

   Ionic CLI                     : 6.10.0 (/home/chinx/.nvm/versions/node/v10.16.0/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 5.1.1
   @angular-devkit/build-angular : 0.901.7
   @angular-devkit/schematics    : 9.1.7
   @angular/cli                  : 9.1.7
   @ionic/angular-toolkit        : 2.2.0

Capacitor:

   Capacitor CLI   : 2.1.2
   @capacitor/core : 2.1.2

Utility:

   cordova-res                          : not installed
   native-run (update available: 1.0.0) : 0.3.0

System:

   NodeJS : v10.16.0 (/home/chinx/.nvm/versions/node/v10.16.0/bin/node)
   npm    : 6.14.5
   OS     : Linux 4.15

I would like to close this issue (face-slaps self)... The output error is a little cryptic, but basically... the issue was to do with imports. The page-test.component needed to be imported into the page-test.module declaration array. Sorry!! I Totally overlooked that.


🐛 [Bug]: Otelfiber blocks stream writers

Bug Description
The stream writer is attempting to flush events, but the initial response is never received on the client, neither are the subsequent events.
How to Reproduce
Steps to reproduce the behavior:

Use go-fiber recipe example Link
Add otel-fiber middleware to this example
Immediately the requests gets blocked
get

Expected Behavior
Otelfiber shouldn't block stream writers.
Contrib package Version
v1.0.10
Code Snippet (optional)
package main

import (
	"bufio"
	"fmt"
	"log"
	"time"

	"github.com/gofiber/contrib/otelfiber"
	"github.com/gofiber/fiber/v2"
	"github.com/gofiber/fiber/v2/middleware/cors"
	"github.com/valyala/fasthttp"
)

func main() {
	// Fiber instance
	app := fiber.New()

	// CORS for external resources
	app.Use(cors.New(cors.Config{
		AllowOrigins: "*",
		AllowHeaders: "Cache-Control",
	}))

	app.Use(otelfiber.Middleware(
		otelfiber.WithServerName("test"),
	))

	app.Get("/sse", func(c *fiber.Ctx) error {
		c.Set("Content-Type", "text/event-stream")
		c.Set("Cache-Control", "no-cache")
		c.Set("Connection", "keep-alive")
		c.Set("Transfer-Encoding", "chunked")

		c.Context().SetBodyStreamWriter(fasthttp.StreamWriter(func(w *bufio.Writer) {
			fmt.Println("WRITER")
			var i int
			for {
				i++
				msg := fmt.Sprintf("%d - the time is %v", i, time.Now())
				fmt.Fprintf(w, "data: Message: %s\n\n", msg)
				fmt.Println(msg)

				w.Flush()
				time.Sleep(5 * time.Second)
			}
		}))

		return nil
	})

	// Start server
	log.Fatal(app.Listen(":3000"))
}

Checklist:

[X] I agree to follow Fiber's Code of Conduct.
[X] I have checked for existing issues that describe my problem prior to opening this one.
[X] I understand that improperly formatted bug reports may be closed without explanation.

I also encountered this problem. How can I solve this problem while using otel-fiber?


Running time, when it is not running

In desktop client and web client timer is not running
In Toggle Button it is running
http://take.ms/UAORC

Toggl Button uses websocket for live updates. At times the socket connection gets lost. This is when the timer can go out of sync. Clicking "Sync" in the menu would get everything up to date again.
Never the less I would like to look into this to see if I can replicate the issue. How did you stop the entry? From web, desktop?

Found the troublemaker.


CI Testing PR(Dummy)

this PR is for CI testing for https://github.com/denoland/fresh/pull/892, but there are problems remain...
It appears that status checking is enabled, so it is not possible to merge into main through the www test only. However, it seems that disabling status checking to speed up CI is not a good idea🤔

There seems to be a work-around to create a task with the same name.
https://docs.github.com/ja/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/troubleshooting-required-status-checks

We have stopped adopting this workaround and changed the Required status setting. Hopefully someday GitHub will introduce a mechanism to better handle this issue. We need to check CI status when merging for now.


[Bug]: il2cpp_init failed: Failed to load hostfxr. Please make sure you have installed the .NET 6.0 runtime

All of the following criteria must be met

[X] All Requirements must be installed.
[X] Changed the title so that it doesn't just says "[Bug]: "
[X] I have searched the GitHub issues for my bug, even in the closed issues.

All of the following are optional to answer

[X] Tried reinstalling the Game.
[X] Tried reinstalling MelonLoader.
[X] Tried restarting PC.
[ ] Was able to see the Start Screen.

Describe the issue.
il2cpp_init failed: Failed to load hostfxr. Please make sure you have installed the .NET 6.0 runtime says this every time I try to launch btd6 even though I have already installed the .NET 6.0 runtime
There is also no latest . log file
Did you attach your log file?
No, I could not find a log file at {Game_Directory}\MelonLoader\Latest.log

not a bug. Install the .NET 6 ruintime. (Desktop Runtime x64)

I'm experiencing the same issue. I installed .NET 6.029 and am still getting this error. I did the entire process as prescribed. When it did not work, I restored my computer to a previous restore point and tried again. Same error each time. Would love some other troubleshooting steps.


Incorrect word boundary detection

What version of regex are you using?
1.4.3
Describe the bug at a high level.
When scanning the string " abc를" with the pattern r"\babc\b", it returns 0 matches. It appears that the word boundary operator is failing to detect a boundary between "abc" and "를"; testing this string with https://util.unicode.org/UnicodeJsps/breaks.jsp indicates that a word boundary should be present there.
What are the steps to reproduce the behavior?
See above
What is the actual behavior?
The regular expression returns 0 matches
What is the expected behavior?
The regular expression returns a single match, "abc"
Additional information
This may be a duplicate of #579

Never mind; regex's behavior is correct in this case (see https://github.com/unicode-rs/unicode-segmentation/issues/90)

Right, this is indeed correct. The regex crate implements word boundaries in accordance with UTS#18 RL1.4. That is, a word boundary occurs when one side is \W and the other is \w. 를 matches \w.
Digging deeper, \w is defined as the union (according to RL1.4) of the following properties:

\p{Alphabetic}
\p{Join_Control}
\p{gc:Mark}
\p{gc:Decimal_Number}
\p{gc:Connector_Punctuation}

This program shows that 를 is in the Alphabetic property. Alphabetic is itself generated from several other properties.
This program shows that 를 is in the Lo or Other_Letter general category (which is one of the constituents of the Alphabetic property). According to UnicodeData.txt in the UCD 13.0.0 archive, 를 is a Hangul syllable codepoint:
AC00;<Hangul Syllable, First>;Lo;0;L;;;;;N;;;;;
D7A3;<Hangul Syllable, Last>;Lo;0;L;;;;;N;;;;;

And each one of those Hangul codepoints is assigned to the Other_Letter general category.
Thus, regex satisfies the UTS#18 spec here correctly as far as I can tell.
More generally though, Unicode does permit custom tailoring for particular locales. I don't know enough about Hangul syllables to say what is appropriate here, but custom tailoring is definitely beyond the scope of the regex crate.


Add ThreatIngestor

Hey Herman. :)
This PR adds ThreatIngestor, a threat intel automation tool.

Hey Ryan!
Looks like a nice one 😄
I will keep an eye on ThreatKB too 😉


add method to set date value in <input type="date">

The problem
Usual $("input[type=date]").sendKeys("24.02.2021") might not always work.
The date format can very between browsers. It may be "mm/dd/yyyy" or "dd.mm.yyyy" on different machines. I didn't find a stable way to make all browsers use the same format.
It makes our tests unstable.
Solution
But internally all browsers hold the date value in format "yyyy-mm-dd".
Selenide could set the date using JavaScript code in format "yyyy-mm-dd", independently of browser locale etc.
Tell us about your environment

Selenide Version: 6.3.4
Chrome\Firefox\IE Version: Chrome 99, Firefox 98
OS Version: MacOS

Implemented in PR https://github.com/selenide/selenide/pull/1770


Text -> Remove -> Remove Selection doesn't work as expected!

If you select a block of text then it's working as expected. But, what if you don't select anything? Well, it will delete the entire line!

Hi, this feature was removed and Delete key issue has been fixed in v1.15. Hope they are gone now.


[bazel][python] Support _virtual_imports input for py_proto_library and py_grpc_library rules

(This change is needed by the googleapis repository)
The proto_library targets are used as deps for py_proto_library and py_grpc_library rules. The proto_library targets can be configured using import_prefix and/or strip_import_prefix (which essentially move original location of the proto file and, as result, affects import path within proto files themselves and in the generated language-specific stubs).
The biggest question to answer when generating stubs from moved protos is where to put the result (this decision affects all downstream rules as well, because the location of a file affects its import path).
tl;dr; This PR tries to follow same logic as the native cc_proto_library (created and maintained by Bazel team).  More details below.
For example, if we have firestore.proto file, which is located under google/firestore/v1beta1 Bazel package and want to move it under google/cloud/firestore_v1beta1/proto (this is axactly what happens in googleapis repository), it can be configured the following way:
proto_library(
    name = "firestore_moved_proto",
    srcs = ["firestore.proto"],
    import_prefix = "google/cloud/firestore_v1beta1/proto",
    strip_import_prefix = "google/firestore/v1beta1",
)

The rule above will first generate virtual .proto files (under new location) and only after that generate a binary descriptor from them.
Specifically it will generate the following "virtual" file under _virtual_imports subdirectory of same package:
bazel-bin/google/firestore/v1beta1/_virtual_imports/firestore_moved_proto/google/cloud/firestore_v1beta1/proto/common.proto

When supplied to cc_proto_library, like the following:
cc_proto_library(
    name = "firestore_moved_cc_proto",
    deps = ["firestore_moved_proto"],
)

The rule will generate .cc and .h files like the following:
bazel-bin/google/firestore/v1beta1/_virtual_imports/firestore_moved_proto/google/cloud/firestore_v1beta1/proto/firestore.pb.h
bazel-bin/google/firestore/v1beta1/_virtual_imports/firestore_moved_proto/google/cloud/firestore_v1beta1/proto/firestore.pb.cc

Notice, that each generated .cc and .h file is prefixed with _virtual_imports/<name_of_proto_library_target>.
The python rules try to do same thing, so for the following py_proto_library rule:
py_proto_library(
    name = "firestore_moved_py_proto",
    deps = [":firestore_moved_proto"],
)

It wil generate the following file:
bazel-bin/google/firestore/v1beta1/_virtual_imports/firestore_moved_proto/google/cloud/firestore_v1beta1/proto/firestore_pb2.py

I.e in same path as cc_proto_library.
This all woks, but an annoying part is that to use the generated library in some other py_library, in needs to specify the "_virtual_imports/proto_target" path as its "includes" parameter (and I can't make it better than that).
Another option would be to skeep the _virtual_imports/<name_of_proto_library_target> the suffix for the generated python stubs, which will make the path look like the following:
bazel-bin/google/firestore/v1beta1/google/cloud/firestore_v1beta1/proto/firestore_pb2.py

That will make using generated stubs simpler and cleaner (no need to specify imports argument), but it will make it inconsistent with the other rules (like cc_proto_library) and also more susseptible to naming conflicts (if there is already something under the generated path).

@mhaidrygoog

@gnossen PTAL (i'm happy to schedule a gvc to discuss the details)

@eoogbe FYI

rubygems.org appears to be down, causing just about all the tests to fail... Trying again.


Updated visual tests

Connected to https://github.com/vaadin/vaadin-core/issues/127
Updated visual tests screenshots.

This change is

Reviewed 6 of 6 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add ssh -i flag to SCP command

Many users save their ec2 keypairs to nonstandard locations and aren't using
ssh-agent, this suggests the -i flag to use the path to their key.
Signed-off-by: Ken Simon<EMAIL_ADDRESS>

Ugh, turns out the $\{ isn't the way to get a literal like I thought it was... I'll reopen when I fix this


Solarized dark: customizable color of constants

Adding a customization option for solarized dark theme that allows one to make the color of the constants blue instead of magenta. The reason for it is that I work with C++ alot and in C++ code, a lot of things (primary example: namespace names) are considered constants. This results in a lot of magenta on screen, which is a bit hard on the eyes.

Thanks for the suggestion but I cannot accept it. Requests to fix syntax highlighting in a specific mode are too niche and can have an unpredictable effect on other major modes that use the same faces. I also want to avoid adding variables for every little change. I'm working on a system so folks can change these color variables, but until then, use custom-set-faces to customize the theme. e.g.
(load-theme 'doom-solarized-dark t)

;; must be used after the theme as loaded
(custom-set-faces `(font-lock-constant-face (( t (:foreground ,(doom-color 'blue))))))

If you use Doom, load order isn't important if you use the custom-set-faces! macro:
(custom-set-faces! `(font-lock-constant-face :foreground ,(doom-color 'blue)))

Hope that helps!


"The transaction fee has expired" on all transactions

coinbase-node 2.0.5
Every time I attempt to send BTC:
        account.sendMoney({
          to: payment.recipient,
          amount: payment.amount,
          fee: '0.0006',
          description: payment.note,
          currency: 'USD',
          idem: payment.idem
        }, cb)

I get this error:
 ValidationError: The transaction fee has expired.
     at handleHttpError (/app/node_modules/coinbase/lib/errorHandler.js:46:15)
     at Request.onPost [as _callback] (/app/node_modules/coinbase/lib/ClientBase.js:174:10)
     at Request.self.callback (/app/node_modules/coinbase/node_modules/request/request.js:344:22)
     at emitTwo (events.js:106:13)
     at Request.emit (events.js:191:7)
     at Request.<anonymous> (/app/node_modules/coinbase/node_modules/request/request.js:1239:14)
     at emitOne (events.js:101:20)
     at Request.emit (events.js:188:7)
     at IncomingMessage.<anonymous> (/app/node_modules/coinbase/node_modules/request/request.js:1187:12)
     at emitNone (events.js:91:20)
     at IncomingMessage.emit (events.js:185:7)
     at endReadableNT (_stream_readable.js:974:12)
     at _combinedTickCallback (internal/process/next_tick.js:74:11)
     at process._tickDomainCallback (internal/process/next_tick.js:122:9)

I thought at first perhaps the transaction fee was too low, so I upped it from 0.0001 to 0.0006.  Also tried 0.0016.  Same result.
Coinbase Status is all green when this is happening.

do you mean .0006 usd or btc?

It turns out this is simply because specifying a fee is no longer supported by the Coinbase API.  Coinbase automatically assigns the fee as of 21 March 2017.

Can this please be re-added to the API? Or a way to qoute what the fees will be? There is no way to send an exact amount of btc/eth using the API, which makes the entire API worthless to me...
Would love this!


small changes so that it could build

Once it can build we can rebuild it :)

Thanks for the work! I really appreciate you helping out! It gone done by design through this PR : #26 and it refactored a lot to reduce what is needed for the application. I'll be closing your PR, but I wanted to take a moment to commend you for your efforts to helping me with this automation :)


JukeAlert not on Civtest

It's not loaded on civtest, that has been the case for a while. Did something break?

Probably oversight. I'll toss it on unless someone beats me to it.
On Aug 9, 2015 9:25 AM, "Maxopoly"<EMAIL_ADDRESS>wrote:

It's not loaded on civtest, that has been the case for a while. Did
something break?
—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/JukeAlert/issues/36.

on cIvtest now.


Seems (but not sure) that the specfile does not work

Hi there, tried to build the rpm using the provided specfile (centos 7) and landing to an error related to path ... since i'm not realy an expert in rpm build, may be i've done it wrong, but with this patch it work
--- statsite.spec.orig  2016-04-27 13:41:10.346946752 +0000
+++ statsite.spec   2016-04-27 13:41:32.746945900 +0000
@@ -48,7 +48,7 @@
 install -m 755 rpm/statsite.initscript $RPM_BUILD_ROOT/etc/init.d/statsite
 %endif

-install -m 755 statsite $RPM_BUILD_ROOT/usr/sbin
+install -m 755 src/statsite $RPM_BUILD_ROOT/usr/sbin
 install -m 644 rpm/statsite.conf.example $RPM_BUILD_ROOT/etc/%{name}/statsite.conf

@david-guenault I think this was fixed by @ROunofF in #198!


Allow infinite scroll in templates

I'm really not sure how you might do this as it seems like they added this in a relatively recent update, but out-of-the-box Remarkable templates allow you to scroll infinitely, with the lines/grid patterning to reach the new scroll thresholds. Would be nice if we could do this in custom templates.

@robinovitch61 , have you tried, what the current behavior with custom template is? If you keep scrolling? Is the template repeated or something?

@robinovitch61 , have you tried, what the current behavior with custom template is? If you keep scrolling? Is the template repeated or something?

It becomes blank once you scroll past the template


Introduce school order state

Depends on #458, rebase this PR after that one is merged.
Context
Schools and responsible bodies need to know when they can start ordering devices, and whether that's for specific circumstances or local coronavirus restrictions. This will be tracked on a per-school basis.
Changes proposed in this pull request

Introduce an order state on the school's pre-order information.
Update the responsible body's school details summary table to show appropriate guidance in the "Can place orders?" section.

Guidance to review
It's awkward having the order state on the preorder information, but it feels like that's the right place for the information. It's worth considering whether we want to rename PreorderInformation, to reflect its increased scope, as a subsequent change.
Screenshots
Is the microcopy correct? cc @emmajhf
Cannot order

Can order under local lockdown

** Can order for specific circumstances**

@benilovj I think the microcopy is good. I have one suggested change to 'Can order for specific circumstances': I'd shorten it to 'Yes, for specific circumstances'.

@benilovj I think the microcopy is good. I have one suggested change to 'Can order for specific circumstances': I'd shorten it to 'Yes, for specific circumstances'.

Thanks @emmajhf, I've incorporated your suggestion.

Looks great. I wonder whether the order_state would fit better on the School if you think of it more as the state of the school is in - has shielding pupils or wider restrictions - rather than directly associated with ordering as such - might need to be renamed to covid_state or something better?
Also I think some of the copy you're using looks very similar to what I need so might be better off in the en.yml?

I've moved the order_state to schools, as discussed.
So as to not block further work, I'll do a follow-up PR to extract the copy bits to en.yml.

Oh, I should maybe rename the field as you suggested @tonyheadford


Imported js modules not being run through babel

Laravel Mix Version: 0.11.4
Node Version (node -v): 4.6.1
NPM Version (npm -v): 6.10.3
OS: Ubuntu 16.04

Description:
Running npm run production throws an error
/js/app.js from UglifyJs
Unexpected token: name (storedTransformPropertyName_) [./~/@material/drawer/util.js:20,0][/js/app.js:157,4]

that leads me to believe that Mix is not applying babel transformation to imported modules in the main js file. If I comment out the imports in the app.js file then the script runs without error. I don't have a .babelrc file and I've tried removing the node_modules folder and reinstalling everything.
contents of package.json
{
  "dependencies": {
    "material-components-web": "^0.11.0",
    "material-design-icons": "^3.0.1",
    "normalize.css": "^7.0.0"
  },
  "devDependencies": {
    "laravel-mix": "^0.11.0"
  },
  "scripts": {
    "dev": "cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "watch": "cross-env NODE_ENV=development node_modules/webpack/bin/webpack.js --watch --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
    "hot": "cross-env NODE_ENV=development node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot --config=node_modules/laravel-mix/setup/webpack.config.js",
    "production": "cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js"
  }
}

contents of webpack.mix.js
const { mix } = require('laravel-mix');

mix.sass('resources/assets/sass/app.scss', 'public/css', {
     includePaths: ['node_modules']
   })
   .js('resources/assets/js/app.js', 'public/js/app.js');

if (mix.config.inProduction) {
    mix.disableNotifications();
}

contents of app.js
import {MDCIconToggle} from '@material/icon-toggle';
import {MDCTemporaryDrawer} from '@material/drawer';
import {MDCTextfield} from '@material/textfield';

let drawer = new MDCTemporaryDrawer(document.querySelector('.mdc-temporary-drawer'));

Remove includePaths: ['node_modules'] from the mix.sass()
You should not be doing this.
Instead, inside of app.scss, import the sass from node_modules directory.
Example:
@import "~/package/sass/package.scss"

Where package refers to the package of the one that you want to use.

Thanks for your response, but it doesn't address my initial issue (which was about js), and your suggestion unfortunately doesn't work for me. If I remove the includePaths option and use the kind of @import to import my sass from node_modules that you gave in your example, I get a File to import not found or unreadable error.
Would you mind explaining why I shouldn't be using includePaths: ['node_modules']?

Each package should come with tools that you can build it yourself or have already been built inside the dist directory. If you want to rebuild the package with some of your edits, you should go into that particular root folder of the package and build it using the instructions provided from their Github repository.

Ok, thanks for the suggestion, but it's not exactly appropriate for my project.  I'm currently starting to integrate the new Material Components for Web and I am importing only the components that I need so I've got a large number of imports in the app.scss file.

Try this: https://github.com/material-components/material-components-web/#css

Thank you once again for your response, but css is not my problem, that is working fine.  The problem is with uglifyjs not understanding es2015.

Right. Sorry for going off track with your issue. Guess I misinterpreted.

Not a problem.

So what I ended up doing is copying the main Laravel Mix setup/webpack.config.js file to my root directory and removing the exclude parameter from the js test on this line https://github.com/JeffreyWay/laravel-mix/blob/master/setup/webpack.config.js#L148.  This means that I'll have to maintain my own version of this file, but at least npm run production is working now.  Hope this helps someone else.

@robbyrice The reason why @JeffreyWay wrote it this way to exclude node_modules because the package have their own way to build the assets and should not let Webpack do the building.

Yeah, I understand but this is what I need to get my project to work.

Might be worth looking into https://github.com/JeffreyWay/laravel-mix/blob/master/docs/quick-webpack-configuration.md. Maybe you could be able to override this particular configuration.
I would also use this project myself if this is possible.

I've tried just overriding this particular configuration, but for the moment I'm not having any luck.  For now the quickest and simplest approach is maintaining my own webpack.config.js file.


Add : Up / Down arrow keys to zoom.

Thinking of less usage of mouse.
Right / Left keys work nice, zoom with up / down arrow keys would be a more complete Viewer with out grabbing the mouse.

Yeah.. Shall plan for the next version.

hey man, great work with this project. came here for the same suggestion.


Quickstart

Motivation
Which issue does this fix? Fixes #issue number
If no issue exists, what is the fix or new feature? Were there any reasons to fix/implement things that are not obvious?
Checklist

[ ] Code follows coding conventions held in this repo
[ ] Automated tests have been added
[ ] Tests are passing
[ ] Docu has been updated (if applicable)
[ ] Temporary settings (e.g. project ID used during development and testing) have been reverted to defaults

How to test
If manual testing is required, what are the steps?

Hi @umeshdhauni . There's no added value in this pull request.
If you want to contribute to hacktoberfest, please feel free to pick any of the hacktoberfest labeled issues.


Feature Request - Need Horizontal Scroll functionality on Resource Views

Currently, If we add more resources the page shrinks.
Having a scrollbar would help in navigating page with ease.
Currently, This is how the resourceTimeGridWeek view looks.

In the current version of the calendar, such a scroll is not possible. I will think about making this scroll in the future major version of the library. For now, the solution may be to use the resourceTimeline view.


Upgrade next js up to 13

What
I've upgraded nextjs to 13 and react up to 18 version
Why
Because I don't wanna add <a> tag always when I add some link tag ;)
###How
Doing only things that are needed and that are not in the beta.
https://nextjs.org/docs/upgrading
###Testing
Manually testing, but honestly its rather simply change for images and links - so it isn't much :)
BTW thanks for that starter - I speeded up my work on the project (at least in the beginning)

Hi @kasperkristensen. I see you have accepted other pull requests. Is everything OK? Maybe I should add or correct something?

@rafalpetryka Thanks for the PR. Nothing wrong, I just haven't had the time to look at your PR yet, since it's not a fix it's not a super high priority, but I will get to it when I have the time. You will need to resolve the merge conflicts, but I most likely won't be reviewing your PR before next week so no rush :)

@kasperkristensen Thanks for the answer! I resolved merge conflicts and will wait patiently ;)


Fix links by removing slash between site.baseurl and post.url

This avoids generating URLs with two leading slashes which results
in unexpected behaviour. Example:
//2018/01/29/

was converted to
<IP_ADDRESS>/01/29/

Fixes #617

@leonklingele Hmm, I merged @rehrar 's fix, or "fix". If this is the right one, just rebase. :)

This fix is still needed. @leonklingele thanks for doing this. Please rebase so we can merge asap. :)

Terribly terribly sorry. This was a front page issue and I was very concerned that the front page wasn't working as it should, so I pushed the fix in #631.
Not trying to steal thunder.

Rebased


[deliver] Sort and comment options

While working with deliver I noticed that the list of options was not very organized.
I moved some options around and added some code comments to at least get some structure in the source.

@janpio Lint failed 😊

Fixed it @joshdholtz (Sorry)


shopware-cli: 0.4.50 -> 0.4.51

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/FriendsOfShopware/shopware-cli/releases.
meta.description for shopware-cli is: Command line tool for Shopware 6
meta.homepage for shopware-cli is: https://github.com/FriendsOfShopware/shopware-cli
meta.changelog for shopware-cli is: https://github.com/FriendsOfShopware/shopware-cli/releases/tag/0.4.51
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.51 with grep in /nix/store/fmi09sl60g34440df3f3gz177c4g7x4b-shopware-cli-0.4.51
found 0.4.51 in filename of file in /nix/store/fmi09sl60g34440df3f3gz177c4g7x4b-shopware-cli-0.4.51

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

shopware-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fmi09sl60g34440df3f3gz177c4g7x4b-shopware-cli-0.4.51 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A shopware-cli https://github.com/r-ryantm/nixpkgs/archive/1c53ad68fa1fb33648b9b2a84e021d241b9f4836.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/1c53ad68fa1fb33648b9b2a84e021d241b9f4836#shopware-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fmi09sl60g34440df3f3gz177c4g7x4b-shopware-cli-0.4.51
ls -la /nix/store/fmi09sl60g34440df3f3gz177c4g7x4b-shopware-cli-0.4.51/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

shopware-cli

Maintainer pings
cc @shyim for testing.

@NixOS/nixpkgs-merge-bot merge


Move all builders into their associated built interfaces

The following builders are not following the new convention to have the builder interface within the target interface they build:

[ ] org.spongepowered.api.data.manipulator.DataManipulatorBuilder
[ ] org.spongepowered.api.data.manipulator.ImmutableDataManipulatorBuilder
[ ] org.spongepowered.api.data.value.ValueBuilder
[ ] org.spongepowered.api.data.DataTransactionBuilder
[ ] org.spongepowered.api.data.ImmutableDataBuilder
[ ] org.spongepowered.api.item.inventory.transaction.InventoryTransactionBuilder
[ ] org.spongepowered.api.service.pagination.PaginationBuilder
[ ] org.spongepowered.api.service.persistence.DataBuilder
[ ] org.spongepowered.api.text.selector.SelectorBuilder
[ ] org.spongepowered.api.text.title.TitleBuilder
[ ] org.spongepowered.api.text.TextBuilder
[ ] org.spongepowered.api.util.ban.BanBuilder
[ ] org.spongepowered.api.world.WorldBuilder

Data related stuff will stay as is.

OK I'll remove them from the list

Actually, what is with the AI task builders?


VS Code, format HTML code with Twig or Swig tags

I've been using VSCode and I love it but I have some projects where I deal with twig tags and formatting code for TWIG templates is a nightmare..
Lets say for example I have:
{% for sushi in dinner.sushiArray %}
{% if sushi == 'New York' %}

Yum

{% else %}

iew...

{% endif %}
{% endfor %}

This is what I get when formatting
 {% for sushi in dinner.sushiArray %} {% if sushi == 'New York' %}

Yum

{% else %}

iew...

{% endif %} {% endfor %}

Is there a way to format correctly HTML in VSCode when we use Twig or Swig tags, like
{% if ... %} {%else%} {%endif%} {% for %} {%endfor%} {% include %}, etc...
For now code formatters remove all line breaks before and after those tags.
Was not able to find an suitable extension for that.. Neither a way to configure the internal code formatter.
Please, I need my sushi.
Thanks

We use http://jsbeautifier.org/ as the underlying formatter. I suggest to file this enhancement request there.

I have finally solved the issue of formatting Twig in VSCode!
Over the past week I have put together a VSCode extension to include everything you would need in a Twig extension: https://marketplace.visualstudio.com/items?itemName=mblode.twig-language
This includes a WORKING Twig file formatter using PrettyDiff with a similar implementation as Atom-Beaitify. Moreover, my plugin includes syntax highlighting and language support, snippets and information on hover for Twig. Hopefully this fixes all of your issues with VSCode and Twig. If you run into a problem, simply open an issue on GitHub.
Thanks and happy coding :)

@mblode Awesome!!


Con 1762 - Increase timeout since cv-csp takes longer to provision

@datamattsson - This effects clone. Do we need this for 1.5-beta?

@suneeth51 can we close this PR?

We are not supporting CV-CSP


Fix version tests in geecheck.html

http://localhost/admin/geecheck.html

Following two tests are failing:
Test class:TestVersion
Description:Check Fusion and server versions are aligned.
Test module:fusion_version_test
Test status:ERROR
Test name:testFusionVersionsMatch
Error message:local variable 'version' referenced before assignment
Error type:UnboundLocalError
Error traceback: File "/usr/lib/python2.7/unittest/case.py", line 329, in run testMethod() File "/opt/google/gehttpd/cgi-bin/geecheck_tests/user_tests/fusion_version_test.py", line 59, in testFusionVersionsMatch gee_server_version = common.GetGeeServerVersion() File "/opt/google/gehttpd/cgi-bin/geecheck_tests/common.py", line 89, in GetGeeServerVersion return version
Test class:TestVersion
Description:Check if GEE Server release is the latest available.
Test module:fusion_version_test
Test status:ERROR
Test name:testGeeServerVersion
Error message:local variable 'version' referenced before assignment
Error type:UnboundLocalError
Error traceback: File "/usr/lib/python2.7/unittest/case.py", line 329, in run testMethod() File "/opt/google/gehttpd/cgi-bin/geecheck_tests/user_tests/fusion_version_test.py", line 44, in testGeeServerVersion gee_server_version = common.GetGeeServerVersion() File "/opt/google/gehttpd/cgi-bin/geecheck_tests/common.py", line 89, in GetGeeServerVersion return version

@tst-awankhede This is something we'll need to do for every version, right? If so, we need to update the release process as well.

yeah, geecheck tests should be run and verified for all releases. However, this test was failing even before we had a version change in our sources.

I mean about the version number change. We will be updating the version number in every release. So, should the tests be updated for every release too?

In release process,
the step that updates all the version numbers in all applicable files  takes care of this.
The test is failing because of file name fusion_Server_version ( real) vs fusion_server_version( in test)

The process currently lists these files:
` The following files need to be updated with the latest version number:

earth_enterprise/src/support/geecheck/geecheck.pl (2 places)
earth_enterprise/src/third_party/apache2/apache-ge-index.shtml
earth_enterprise/src/fusion_version.txt (2 places)
earth_enterprise/src/fusion/portableglobe/cutter/cgi-bin/geecheck_tests/common.py
earth_enterprise/src/installer/common.sh (2 places)`

Shouldn't we add the file that the test is looking at? If it's already updated, is it just a bug in the test?

I verified and yes, files are up-to-date. Although they are not listed here.some process is updating both the version files test looks for.
It's just test issue.

Closed via PR #463.


Feature request: duration formatting

E.g. something like
let interval: NSTimeInterval = 60 * 60
print(interval.toFormatString()) // “1 hour”

Should it work only with NSTimeInterval? So you can pass an arbitrary number of seconds and you get it using specified unit of times.

I think so!

Can be done with NSDateComponentFormatter too.

Can be done with NSDateComponentFormatter too.

Can you provide an example? I was trying to figure out how to do this but could not.

let interval: NSTimeInterval = 63
let formatter = NSDateComponentsFormatter()

formatter.unitsStyle = .Full
formatter.allowedUnits = [.Hour, .Minute, .Second]

formatter.stringFromTimeInterval(interval)  // "1 minute, 3 seconds"

However seems it does not work fine with negative intervals

NSTimeInterval now has a toString(style:) function you can use to convert a duration into a readable string.
Example:
let duration = NSTimeInterval( (60*60*24) ).toString() // print "1d"

You can also specify the style of the formatted output by passing a FormatterStyle struct:
let duration = NSTimeInterval( (60*60*24) ).toString(FormatterStyle(style: .Full)) // print "1 day"

FormatterStyle is a struct where you can set:
public var style :NSDateComponentsFormatterUnitsStyle = .Abbreviated

Configure the strings to use for unit names such as days, hours, minutes and seconds. Use this property to specify the style of abbreviations used to identify each time component. By default the value of this property is .Abbreviated; so for example, one hour and ten minutes is displayed as “1h 10m”.

public var units :NSCalendarUnit

The bitmask of calendrical units such as day and month to include in the output string. By default all available values are used; this means each unit of time is part of the output string if the relative value is not zero. You can use:

Year
Month
WeekOfMonth
Day
Hour
Minute
Second
Any other value of the NSCalendarUnit enum will result in an exception.

public var maxUnits :Int = 0

This property can be used to limit the number of units displayed in the resulting string. For example if set to 2, instead of “1h 10m, 30s” the output string will be “1h 10m”. You can use it to provide a round up version of the output when you are constrained.

public var approximate :Bool = false

Setting the value of this property to true adds phrasing to output strings to reflect that the given time value is approximate and not exact. Using this property yields more correct phrasing than simply prepending the string “About” to an output string. By default is set to false

public var approximatePast :Bool = false

Setting this property to true results in output strings like “30 minutes remaining”. The default value of this property is false.

public var collapsesLargestUnit :Bool = false

An example of when this property might apply is when expressing 63 seconds worth of time. When this property is set to true, the formatted value would be “63s”. When the value of this property is false, the formatted value would be “1m 3s”. By default is set to false

public var zeroBehavior :NSDateComponentsFormatterZeroFormattingBehavior = .Default

This specify the behavior of the formatter when it encounter a zero value for a particular unit of time.

Should not weekOfMonth be weekOfYear?
Otherwise we get into trouble when the time interval crosses into another month.

NSDateComponentsFormatter supports only NSCalendarUnitWeekOfMonth in  allowedUnits  property. In fact  FormatterStyle it's just a wrapper around it.