build: Configure PR title validation

Signed-off-by: Gábor Lipták<EMAIL_ADDRESS>https://github.com/marketplace/actions/conventional-pr-title
#17

Thanks for the contribution @gliptak, but it does not appear that the Conventional Commits parser used by that action supports item 3 in the standard. Specifically, the use of an ! after the type to indicate a breaking change. Maybe we can get the author to support the use and then revisit? Thoughts?

They both use ultimately use the same parser it seems, but action-semantic-pull-request seems to be more configurable. I'm assuming this is the one you landed on as well given it was the one used in the
The only question I have is regarding the use of pull_request_target vs pull_request. It seems that the latter is safer in general to use as the credentials are read-only vs read-write. Thoughts?
on:
  pull_request_target:
    types:
      - opened
      - edited
      - synchronize

@pkazmier thank you for your analysis. https://github.com/amannn/action-semantic-pull-request seemed to have been more popular
reading https://securitylab.github.com/research/github-actions-preventing-pwn-requests/ pull_request might be better fit
here is a failing run https://github.com/gliptak/awsrun/actions/runs/4179413368/jobs/7239331269

thank you for the review, updated
https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request

Thanks @gliptak. I've not really explored Github Actions yet, so your contributions have been much appreciated. It's also provided me the motivation to learn more about Actions.


Magic byte error

Hey,
I am using the ConsumerGroup to consume messages.
After several messages (2 - 50) I suddenly get the following error:
Error: Not a message set. Magic byte is 2
The messages are produced using a different client producer.
Messages are In binary format, (I've set the Client format accordingly)
Kafka version is: 2.1.0
Thanks.

I noticed this happens when the fetchMaxBytes is set too low the broker will start sending fetch responses in RecordBatch format instead of MessageSet.

Thanks @hyperlink , looks like this solves the issue.
I would expect a more explanatory message for this type of error though :)

I noticed this happens when the fetchMaxBytes is set too low the broker will start sending fetch responses in RecordBatch format instead of MessageSet.

@hyperlink what is the minimum number that we can set ? I have to consume the messages slowly.

If so what is the min value that we can set for 'fetchMaxBytes'

Is there some specific configuration for Kafka broker in order to fix this error and allow kafka-node working with low fetchMaxBytes value?

@IharKrasnik you have to edit some items in server.properties files

I wonder which configuration option we can change exactly if there is any indeed.

server.properity -> socket.request.max.bytes=104857600

In docker-compose.yml
image: wurstmeister/kafka
environment:
   KAFKA_MESSAGE_MAX_BYTES: 200000000
   KAFKA_MAX_PARTITION_FETCH_BYTES: 200000000
   KAFKA_FETCH_MAX_BYTES: 200000000
   KAFKA_MAX_REQUEST_SIZE: 200000000
   KAFKA_SOCKET_REQUEST_MAX_BYTES: 200000000

in node.js:
consumer = new Consumer(
      kafkaClient,
      [
        {
          topic: 'my-topic',
        },
      ],
      {
        fetchMaxBytes: 200000000,
      },
    );

In docker-compose.yml
image: wurstmeister/kafka
environment:
   KAFKA_MESSAGE_MAX_BYTES: 200000000
   KAFKA_MAX_PARTITION_FETCH_BYTES: 200000000
   KAFKA_FETCH_MAX_BYTES: 200000000
   KAFKA_MAX_REQUEST_SIZE: 200000000
   KAFKA_SOCKET_REQUEST_MAX_BYTES: 200000000

in node.js:
consumer = new Consumer(
      kafkaClient,
      [
        {
          topic: 'my-topic',
        },
      ],
      {
        fetchMaxBytes: 200000000,
      },
    );

Thx you! This config helps for me!


[TACHYON-522] Implement LRU with new Evictor interface

This PR is a rebase of PR https://github.com/amplab/tachyon/pull/990 into rebased_worker_wip branch

Minor comments left. Thanks for rebasing the PR!

@uronce-cc +1 Looks good to me, I opened another PR with the merge conflicts resolved.

@calvinjia thx.


change the self attribute declaration sequence

In order to resolve the following error:
============
2020-11-17 09:40:43 [ADK:INFO] Initializing
2020-11-17 09:40:43 [ADK:ERROR] ADK not initialized
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/apeer_dev_kit/_core.py", line 31, in _read_inputs
with open(self.wfe_input_file_name, 'r') as input_file:
AttributeError: '_core' object has no attribute 'wfe_input_file_name'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "./apeer_main.py", line 1, in
from apeer_dev_kit import adk
File "/usr/local/lib/python3.6/dist-packages/apeer_dev_kit/adk.py", line 3, in
_adk = _core._core()
File "/usr/local/lib/python3.6/dist-packages/apeer_dev_kit/_core.py", line 15, in init
self._input_json = self._read_inputs()
File "/usr/local/lib/python3.6/dist-packages/apeer_dev_kit/_core.py", line 40, in _read_inputs
raise IOError(message)
OSError: ADK not initialized

Oh boi! Thanks you found this. This one slipped through even though we have tests for it...

@thomasirmer Should we then release a new version of the ADK for the bug fix then? :-)

@mullerlele already done! Version 1.2.1

Actually we need to bump up our tests. If you look carefully, recent changes have actually dropped the branch coverage.  This library is pretty critical for every module we run. Thanks @mullerlele for your quick fix!


add initial support for keyboard shortcuts (issue #38, issue #39, issue #31)

Work in progress. I've added inline comments of questions I have. Any pointers greatly appreciated. Thank you!

@cvan I'm going to do some refactoring next week to make cli.js easier to test and modify. And changes like this will be easier to land before the refactoring. So if you have time to finish it up in the next couple days, I recommend it!

will fix this up early next week, thanks. sorry for the delays!

@mykmelez: I simplified the code for the keyboard shortcuts significantly. with these changes, toggling the DevTools (Cmd + Alt + I on Mac, Ctrl + Shift + I on Windows) works and handling page reloads.

I have a question here about how handle key events when the DevTools window is focused? since such keyboard shortcuts as toggling Dev Tools, reloading the page, etc. should occur regardless of which window is focused, is it better to have an event listener in the chrome context, not browser (#content) and any child windows opened (e.g., the DevTools window)?
what's the correct way to force-refresh? I searched a bit on https://dxr.mozilla.org/, and I found some references to reloadSkipCache, but that doesn't seem to be an available method. I tried passing true as the first argument to browser.reload, but it doesn't work (or at least appear to work).
once I'm clear on how to handle the listening between windows (#1 above), I'll add – in an additional follow-up PR, so I can get this merged – the other shortcuts for the Inspector, Web Console, Debugger, Page Info, New Tab, Close Tab, New Window, Close Window, Minimize Application, Hide Application, Fullscreen Application.

feel much better about this PR now. let me know if you have any additional feedback. thanks!

I have a question here about how handle key events when the DevTools window is focused? since such keyboard shortcuts as toggling Dev Tools, reloading the page, etc. should occur regardless of which window is focused, is it better to have an event listener in the chrome context, not browser (#content) and any child windows opened (e.g., the DevTools window)?

@cvan I thought it might be possible to add a single handler to the shell window, then set the shell window as the opener of the DevTools window, such that the single handler would handle key events both in the browser (which is contained within the shell window) and in the DevTools window (which is opened by the shell window).
Alas, it looks like key events don't bubble across window.opener references. So a key event listener on the shell window will observe key events in the browser, but it won't observe key events in the DevTools window, even if it's that window's opener.
Thus I don't have a better strategy for this. I think your approach is as good as it gets. Note, however, that the DevTools window already handles "reload" events itself, so it isn't necessary for the shell to handle them for the DevTools window. Also, I found a bug and fixed it in https://github.com/cvan/qbrt/pull/3.

what's the correct way to force-refresh? I searched a bit on https://dxr.mozilla.org/, and I found some references to reloadSkipCache, but that doesn't seem to be an available method. I tried passing true as the first argument to browser.reload, but it doesn't work (or at least appear to work).

I took a look at what DevTools does, and it looks like it calls the reload method of the nsIWebNavigation object attached to the browser element as its webNavigation property, passing it the LOAD_FLAGS_BYPASS_CACHE flag.
So replacing browser.reload(true); with this statement should work:
    browser.webNavigation.reload(Ci.nsIWebNavigation.LOAD_FLAGS_BYPASS_CACHE);

I took a look at what DevTools does, and it looks like it calls the reload method of the nsIWebNavigation object attached to the browser element as its webNavigation property, passing it the LOAD_FLAGS_BYPASS_CACHE flag.

Note that the BrowserReloadSkipCache function also sets LOAD_FLAGS_BYPASS_PROXY: http://searchfox.org/mozilla-central/rev/a798ee4fc323f9387b7576dbed177859d29d09b7/browser/base/content/browser.js#2143-2147
I doubt it's worth setting that here, though.

Note that the BrowserReloadSkipCache function also sets LOAD_FLAGS_BYPASS_PROXY: http://searchfox.org/mozilla-central/rev/a798ee4fc323f9387b7576dbed177859d29d09b7/browser/base/content/browser.js#2143-2147
I doubt it's worth setting that here, though.

nice, thanks for the catch. I'm going to add it for good measure; if you want me to remove (or later), we can.

@mykmelez: updated PR per your changes, suggestions, a few small bugs I noticed, and made things a bit more structured. I may've went a bit overboard, so let me know how this feels to you. thanks!
here's a video recording of the flows working (with your fix for toggling DevTools multiple times):
https://vimeo.com/220904063

@cvan Is there anything particularly problematic in those review comments? Note that the nits are all optional, and the questions just need answers, not necessarily code changes!

on it, thanks 👍

@mykmelez updated. thanks for your feedback (and patience) 👍

The test failures are due to eslint wanting a space between function keyword and opening parenthesis for anonymous functions (i.e. function () …). I actually prefer the way you do it in this branch, however (i.e. function()). So I've updated the eslint rule in https://github.com/mykmelez/qbrt/commit/dce317093dd73d579cab1df65bf42fa4593de364 and will merge this once the CIs turn green on https://github.com/mykmelez/qbrt/commits/cvan-keys.

And merged. Thanks, @cvan!

awesome, thanks for your help and patience 👍


Version switcher build test

PR summary
RTD build test for version switcher.
PR checklist

[ ] Links to the related issue (#....)
[ ] Units tests have been run and/or modified and/or added
[ ] Docstrings are complete and documentation modified
[ ] Any new dependencies have been added to pyproject.toml and requirements.txt (in docs folder)
[ ] If new files have been added, make sure they aren't excluded by .gitignore
[ ] Any new data files have been added to manifest.in

First test result : Version switcher is shown on "latest" version and not on "stable" version. Switcher is operational when shown.


SyntaxError while running df05.py @ Lab ( Processing Data with Google Cloud Dataflow )

For lab ( Processing Data with Google Cloud Dataflow), one of the step was to run script df05.py

However, it returned SyntaxError coming from one of the function under another file timezonefinder.py

Pls help to check and amend. Thanks.

Seems to be the same problem as highlighted under #55. Change the install_packages,sh to:
#!/bin/bash sudo pip install --upgrade timezonefinder==3.0.0 pytz apache-beam 'apache-beam[gcp]'


Feat#90

Describe your changes
Addition of a responsive design on all screens to adapt it to a mobile version.
Issue ticket number and link
closes #90
Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] I have reviewed my code and used the established color palette for the project.
[x] I have reviewed my code and used the established font for the project.

Wouldn't it be more convenient if notifications appeared in the drop-down menu instead of notification?


Unable to use Microsoft Teams WalkieTalkie using custom PTT button on Bartec IS910.1

I'm unable to use Microsoft Teams WalkieTalkie using custom PTT button on Bartec IS910.1 Android tablet.
Is there any developer API available so that I can program the PTT button to work with Microsoft Teams?
Thank you.

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

ID: d91ddefc-3722-8bc2-bd9b-4a90ede28944
Version Independent ID: 35e56a9d-a01c-4126-3750-aae1ed984f55
Content: Walkie Talkie application in Microsoft Teams - Microsoft Teams
Content Source: Teams/walkie-talkie.md
Service: msteams
GitHub Login: @MicrosoftHeidi
Microsoft Alias: heidip

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Looks like the latest master is not copying pythonic/utils/pdqsort.h

I tried installing the latest master, and the install is missing pythonic/utils/pdqsort.h
I wouldn't mind fixing this myself, but I'm not sure how to.
J.

This got closed through #1604


Add test for hub tools and file manager

Signed-off-by: Kaiyuan Hu<EMAIL_ADDRESS>

Codecov Report

Merging #521 (a07b3ba) into main (f08a3c6) will increase coverage by 0.90%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##             main     #521      +/-   ##
==========================================
+ Coverage   71.75%   72.66%   +0.90%
==========================================
  Files         139      139
  Lines        5198     5198
  Branches      778      778
==========================================
+ Hits         3730     3777      +47
+ Misses       1234     1183      -51
- Partials      234      238       +4

Impacted Files
Coverage Δ

towhee/hub/bin/hub_tools.py
40.96% <77.77%> (+40.96%)
:arrow_up:

towhee/hub/repo_manager.py
90.09% <0.00%> (-0.91%)
:arrow_down:

towhee/hub/operator_manager.py
84.21% <0.00%> (+1.31%)
:arrow_up:

towhee/data/random_erase.py
90.90% <0.00%> (+4.54%)
:arrow_up:

towhee/hub/file_manager.py
67.08% <0.00%> (+5.06%)
:arrow_up:

Continue to review full report at Codecov.

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

/approve
/lgtm


[SMP on target test] Task Delete

[SMP on target test] Task Delete
Description

Test case "Task Delete" implementation.
Test Steps

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Will create another PR to consist of all SMP on target test implementations.


Add a script to benchmark generator

Summary
Add a new script to collect latency results for OPT models during generation.

Please see the updated PR for a script to benchmark the generator interface in terms of latency and peak GPU memory usage. We also add a function to collect GPU traces if needed.


How can I get the little color icon next to color names in my custom language?

Writing an LSP extension for our CSS preprocessor, I notice that when I get a color completion from CSS (I proxy the CSS service through mine) I get the little color indicator next to the value, but in the file itself there is no indicator (see pic).

I haven't found anything in the LS Protocol or in TM grammars to make this possible in my language.
Is it at all possible to get the color icon through an extension / grammar, or is it a special VSCode feature for CSS only?

VSCode Version: 1.18.1
OS Version: Ubuntu 16.04

Reproduces without extensions: Yes/No

It's a proposed protocol extension. See
https://github.com/Microsoft/vscode-languageserver-node/blob/master/protocol/src/protocol.colorProvider.proposed.md
Check out the css client and server on how to use it.

Is it possible the vscode.d.ts version that is downloaded doesn't have these features?
I can't find languages.registerColorProvider, and get multiple errors from vscode-langugeclient:
node_modules/vscode-languageclient/lib/client.d.ts(1,139): error TS2305: Module '"/home/wix/projects/stylable-intelligence/packages/client/node_modules/vscode/vscode"' has no exported member 'ProviderResult'.
node_modules/vscode-languageclient/lib/client.d.ts(1,272): error TS2305: Module '"/home/wix/projects/stylable-intelligence/packages/client/node_modules/vscode/vscode"' has no exported member 'CompletionList'.
node_modules/vscode-languageclient/lib/client.d.ts(1,648): error TS2305: Module '"/home/wix/projects/stylable-intelligence/packages/client/node_modules/vscode/vscode"' has no exported member 'DocumentLink'.
node_modules/vscode-languageclient/lib/client.d.ts(1,679): error TS2305: Module '"/home/wix/projects/stylable-intelligence/packages/client/node_modules/vscode/vscode"' has no exported member 'TextDocumentWillSaveEvent'.
node_modules/vscode-languageclient/lib/client.d.ts(1,706): error TS2305: Module '"/home/wix/projects/stylable-intelligence/packages/client/node_modules/vscode/vscode"' has no exported member 'WorkspaceFolder'.
node_modules/vscode-languageclient/lib/codeConverter.d.ts(11,46): error TS2694: Namespace 'vscode' has no exported member 'TextDocumentWillSaveEvent'.
node_modules/vscode-languageclient/lib/codeConverter.d.ts(36,31): error TS2694: Namespace 'vscode' has no exported member 'DocumentLink'.
node_modules/vscode-languageclient/lib/protocolConverter.d.ts(18,57): error TS2694: Namespace 'vscode' has no exported member 'CompletionList'.
node_modules/vscode-languageclient/lib/protocolConverter.d.ts(21,122): error TS2694: Namespace 'vscode' has no exported member 'CompletionList'.
node_modules/vscode-languageclient/lib/protocolConverter.d.ts(66,49): error TS2694: Namespace 'vscode' has no exported member 'DocumentLink'.
node_modules/vscode-languageclient/lib/protocolConverter.d.ts(67,53): error TS2694: Namespace 'vscode' has no exported member 'DocumentLink'.
node_modules/vscode-languageclient/lib/protocolConverter.d.ts(69,72): error TS2694: Namespace 'vscode' has no exported member 'DocumentLink'.
node_modules/vscode-languageclient/lib/workspaceFolders.proposed.d.ts(1,10): error TS2305: Module '"/home/wix/projects/stylable-intelligence/packages/client/node_modules/vscode/vscode"' has no exported member 'WorkspaceFoldersChangeEvent'.

It's in 1.18. Make sure that you have
"engines": {
    "vscode": "1.18.x"
  },

in your extension's package.jsonand do a npm install to get vscode.d.ts updated.

Thanks. My engine was outdated.
You can close #39036  as well, it's the same issue.


Starting in Passthrough on Meta Quest

When you are already in passthrough in the menu system, and start a Godot project that is an AR project, I'm encountering two issues.
The first is that the Quest will temporarily switch passthrough off during loading of the Godot project. A manifest entry is needed to keep the Quest in passthrough during loading. We can add this manifest entry based on checking the environment blend mode setting in the OpenXR project settings. If this is set to anything but Opaque we should add the following:
<!-- Enable passthrough background during the splash screen -->
<meta-data android:name="com.oculus.ossplash.background" android:value="passthrough-contextual"/>

The second is that when OpenXR starts and loads this setting, it checks if the blend mode is supported. It looks like at this point in time, our "alpha mode emulation" hasn't been checked yet and thus the setting reverts back to Opaque. You have to set it to Alpha again in code.
You can use this project to test this functionality: https://github.com/BastiaanOlij/godottoy

cc @dsnopek

Thanks, I'll take a look at it!
I don't know exactly how everything relates together, but we also have OpenXRFbPassthroughExtensionWrapper::is_passthrough_preferred() (via the XR_META_passthrough_preferences extension), which is supposed to allow the OpenXR runtime to tell us if we should default to passthrough.

@BastiaanOlij:

The second is that when OpenXR starts and loads this setting, it checks if the blend mode is supported. It looks like at this point in time, our "alpha mode emulation" hasn't been checked yet and thus the setting reverts back to Opaque. You have to set it to Alpha again in code.

Regarding this second issue, I just posted PR https://github.com/godotengine/godot/pull/94550 for Godot to fix it!

@BastiaanOlij:

The first is that the Quest will temporarily switch passthrough off during loading of the Godot project. A manifest entry is needed to keep the Quest in passthrough during loading.

And PR https://github.com/GodotVR/godot_openxr_vendors/pull/194 addresses this issue
With both those PRs together, I can comment out the code changing the blend mode in start_vr.gd in your Godot Toy app, and it will seamlessly stay in passthrough!


List all needed environment variables

I found 3 additional environment variables which were not listed in the README.md:
SUBREDDIT, MONTH_UPVOTE_THRESHOLD and ALL_UPVOTE_THRESHOLD. I'm unsure if there are more.
Please also list them in the README.md so self hosters can find the necessary resources more easily.

Fixed issue in README and clarified the 2 more obscure config variables:

ALL_UPVOTE_THRESHOLD
MONTH_UPVOTE_THRESHOLD

By changing README.md using @NNTin's image in his 1 Click Deployment pull request.
Joss


remove need for type casting by using runtime.Object

Signed-off-by: Ashley Schuett<EMAIL_ADDRESS>What this PR does / why we need it:
This changes some instances of interface{} to instead use runtime.Object to avoid type casting.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

None

/lgtm
/approve


Create edit a whole row insted a field

How do I create a button for edit whole data of row instead a field??

Yes, this feature should support. I also expect to have this one.

Let me think about this. It makes sense, but was not part of google material table concept.
Since we can have multiple types of cells (dropdown in edit mode, input field in edit mode), we'd need to have the dropdown edit field first as a new feature, and then have a full implementation for this. But also possible in the other way around.
Would it be enough for you guys to have only text input fields for now?

Great. I think control supports edit row with text input fields is ok. With dropdown field, we can work around to include a dropdown when user click on textbox to select an item then bind text to the textbox. Thanks.


TF >= 2.7 slowdown tf.data API

Click to expand!
Issue Type
Performance
Have you reproduced the bug with TF nightly?
Yes
Source
installed from pip
Tensorflow Version
tf >= 2.7
Custom Code
No
OS Platform and Distribution
Linux, CentOS Linux 7
Mobile device
No
Python version
3.8, 3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Since TF >= 2.7, I observed a significant slowness when running ds_iter.get_next() where ds_iter is an iterator of tf.data.Dataset defined as dataset.shuffle(...).cache().repeat(...).batch(..., drop_remainder=True).prefetch(tf.data.AUTOTUNE). The same code runs much faster on the same machine using TF <= 2.6. A reproducer is included below.
Looking at the change log from TF2.8, I see the following related to Dataset API (since v2.7) Stateful ops used in tf.data.Dataset. However I am not entirely sure if there is a workaround in higher TF version to reach the same speed.
Standalone code to reproduce the issue
import tensorflow as tf
import time

X = tf.random.uniform(shape=[21000, 373])
Labels = tf.random.uniform(shape=[21000, 1])
batchsize = 8192
n_shuffles = 500000
n_samples = 21000
ds = tf.data.Dataset.from_tensor_slices((X, Labels))
ds = ds.shuffle(buffer_size=n_samples).cache().repeat(n_shuffles).batch(batchsize, drop_remainder=True).prefetch(tf.data.AUTOTUNE)
ds_iter = iter(ds)

num_iter = 100
loop_start = time.time()
for i in range(num_iter):
    ds_iter.get_next()
loop_stop  = time.time()
print('loop time', f'{loop_stop-loop_start : .4f}')

Relevant log output
output of the reproducer:
- with TF >= 2.7: loop time  15.8203
- with TF <= 2.6: loop time  1.0519

device to run the above two tests:
NVIDIA A100-PCIE-40GB NVIDIA-SMI 530.30.02 Driver Version: 530.30.02 CUDA Version: 12.1

$$ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2021 NVIDIA Corporation
Built on Thu_Jan_28_19:32:09_PST_2021
Cuda compilation tools, release 11.2, V11.2.142
Build cuda_11.2.r11.2/compiler.29558016_0

Hi, @chnzhangrui!  I tried this provided code in TF v2.8,  TF v2.12 and tf-nightly using colab. The issue is not replicating as described here. Could you please have a look at these gists and confirm the same. Thank you!

Hi @sushreebarsa, thanks for the quick response. I checked your gists and I can also confirm your (fast) result. However, it is not clear to me if it was running on GPU or CPU for you. The problem persist only if I run on GPU and I don't know if I can connect to a GPU for test on colab. Let's focus on v2.8.

Hi @chnzhangrui I have tested the issue on colab using GPU runtime with TF v2.8 as well which is also not replicating the issue reported here. Please find the gist here for reference. Thank you!

Hi @sushreebarsa, thanks for the confirmation. I am trying to confirm this with my system again and it currently encountered an issue. Will update once I have some idea. I am a little concerned that the fast result you see is really from GPU or just CPU, because this code runs very fast on CPU. I may need to come up with a code that doesn't work on CPU.

@chnzhangrui Thank you for your response. Please let us know if there is any new update?
Thank you!

Sorry for the silence. I am still waiting for response from our IT support.

@chnzhangrui Is there any update on this issue?


IHttpBufferingFeature.DisableRequestBuffering is named incorrectly

This method currently actually disables dynamic compression on the response when on IIS. It has nothing to do with buffering the request. In fact, if you're desire is to stop buffering of outgoing streaming responses, you need to call this API when on IIS.
It should probably be called DisableResponseCompression (because that's what it actually does and is intended to do) but that might look odd on the IHttpBufferingFeature interface (why are there compression methods on the buffering feature?).
Perhaps the better approach is simply remove this method altogether and reduce the interface to just have DisableResponseBuffering and in IIS have it disable both the IIS response buffers and the dynamic compression module. After all, that's what you need to do to truly turn of response buffering on IIS anyway.
/cc @davidfowl @lodejard @Tratcher @GrabYourPitchforks

What's the decision on this one?
/cc: @glennc

We need to make sure we revisit this entire area. We started and took tons of notes:

https://github.com/aspnet/HttpAbstractions/wiki/Rolling-Notes
https://github.com/aspnet/HttpAbstractions/wiki/Rolling-Notes---Response-Styles
https://github.com/aspnet/HttpAbstractions/wiki/Rolling-Notes-Response-Stream-Contract

But we haven't finished it. We need to finish it and implement something.
/cc @lodejard @DamianEdwards @GrabYourPitchforks @Tratcher

@glennc Can we get this discussed in the next design meeting?

@davidfowl Do we need another API review for HttpAbstractions?

This will be covered by aspnet/Hosting#299 (according to @davidfowl)


Review guidance on links and prepare proposals

For 1 December Style Council meeting.
Tasks

Follow up issues raised in the content community GitHub issue on external links, plus issues around links (internal and external) raised on NHS.UK Slack channel.
Draft guidance with input from content designers to present to Style Council

Discussed at Style Council, December 2021. No agreement but comments noted in Style Council minutes.
Actions

Get a clearer idea from analytics to see if people are returning to a page after clicking rarely visited inline links to see if they’re justified.  (Asked for help in analytics Slack channel.)
Check IA UR re sideways links at bottom of page.

To do

Sara to review analytics sent by Nicola, email 13 January
Given team resourcing, I'm going to put this into the backlog to pick up again in coming months - i.e. not at next Style Council meeting.


IA 4520 persist app region

Jira Ticket: https://broadworkbench.atlassian.net/browse/IA-4520

Summary of changes:

What

Persist region to table: KUBERNETES_CLUSTER , field: region (which already exists in mysql) [This was done in the leonardo PR: #3700]
Use this value when displaying the 'location' in the Cloud Environments page in terra

Why

We had been defaulting to the 'default' region for Apps instead of using their actual region

Testing strategy

[ ]  Create an azure billing project and workspace. Run a cromwell app in the workspace. Check that the 'Cloud Environments' page show the correct location

DO NOT MERGE


Cannot display incremental flashcard on queue on RemNote mobile web.

https://media.discordapp.net/attachments/1003459309088284675/1144109119956058113/Screenshot_20230824_112052_com.microsoft.emmx.jpg?width=310&height=671

Oh yeah I need to test mobile more

Hello, I have the same problem, it cannot be displayed on the mobile terminal. (Android13, Mi 13)


Automated cherry pick of #2305: .travis.yml: 不使用docker

Cherry pick of #2305 on release/2.11.
#2305: .travis.yml: 不使用docker

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@13913fc). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.11   #2308   +/-   ##
==============================================
  Coverage                ?   7.07%
==============================================
  Files                   ?     452
  Lines                   ?   71318
  Branches                ?       0
==============================================
  Hits                    ?    5046
  Misses                  ?   65854
  Partials                ?     418

Flag
Coverage Δ

#aFlag
7.07% <ø> (?)

Continue to review full report at Codecov.

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

/lgtm

/approve


Disabling Blur from Code

We can enable the blur by setting the attached property in code with
AcrylicWindow.SetEnabled(this, true);
After that, we unfortunately can not disable it again. Nothing happens wenn calling this method with false as paramter

Hi @poochie89
Thanks for your feedback.This is same to #88, I'll close this issue.
If there are any problems, please reopen this issue.


Add times method

I like this useful method!
http://underscorejs.org/#times
Please review @robb .

Does this commit fit in your intentions ?

Nah, sorry if I wasn't clear. Let's go with NSInteger as the type as you originally suggested but have an NSCParameterAssert(n >= 0) instead so if users pass in -1 it will crash at runtime.

Thank you for explain! How about this commit?
I am sorry to trouble you, review it please. :dog2:

:bow:

Thanks a lot!


refactor: Create a module with smartrest message id consts

Proposed changes
Create a single module that contains ids for Smartrest messages and static templates.

user code is easier to read
can easily jump to definition
easier to see the dependency and find usages by grepping for the usage of the module

The next steps would be for thin-edge not to use these ids directly, but to provide a higher-level layer of abstraction for interacting with Cumulocity, much like go-c8y-cli.
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[x] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

I see that you've inherited the existing names from the template titles in the C8y documentation. I've given some renaming suggestions, especially around the operation templates to make them sound like verbs rather than nouns. Feel free to accept/reject the ones that you feel relevant.

I wanted to keep the names somewhat consistent with c8y documentation, and don't really want to spend more time revising the names, so I will merge them as they are now. We can always adjust them later.

I wanted to keep the names somewhat consistent with c8y documentation, and don't really want to spend more time revising the names, so I will merge them as they are now. We can always adjust them later.

Agree. Keeping the names consistent with c8y documentation is important even if some might be unclear.


Login is not remembered across the same session using once

I am using 2 different environements for auth.
I am performing login for both environments once. However, the 2nd call of the 2nd auth is not persisting the session correctly.
Perhaps it is using the auth for the wrong environment?
See MigrateTextGroups.kt

Resolved via fix to cookie jar in 0.13.1


Cherry-pick all the juice from the 0.0.x branch.

This should bring master's functionality on-par with 0.0.x's.

Thank you
/lgtm


Bug Found in Challenge "Change the Color of Text"

Coding Bug.docx
Hi! After completing a few challenges, I think I encountered a bug.  I followed the instructions on changing text color, and the text turned red, so I think I wrote this correctly.  But the system won't send me to the next challenge, claiming I didn't do this right.  See code input and the system response below.  Can someone from CodeCamp admin fix this so I can continue funfun coding? Thanks!

Response:
Your h2 element should be red.

CatPhotoApp

Hi. I skipped this one, and tried the CSS, plugged in the code, the iphone turned Blue, and the system is still saying that CatPhotoApp thing needs to be blue.  I think that one has a bug too.

Hello @ceciland62, there's a known issue on some challenges if you're using Firefox.
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/10595
Are you experiencing the same issues?

Closing as duplicate
Please see http://forum.freecodecamp.com/t/creating-a-new-github-issue/18392
Happy Coding!

Thanks @Marmiz, I was using Firefox.  It all works now on Chrome.

I am having this problem in Chrome currently.


Date Picker with time

Hi !
I'm having "Date picker" column on TIMESTAMP DB field in IG (format YYYY-MM-DD HH24:MI).
Minutes selector appears before hours selector.
And both time selectors disappear when i navigate to next/previous month.
I tried to do the same in universal theme and there everything works correctly.

Thanks for filling this bug. Do you need a temporary fix or can you wait until the next release?

Some temporary solution for this would be really nice to have !

Ok, I'll get you some JS and/or CSS to add to your page soon. Thanks.

Add this CSS to your page:
.ui-datepicker .ui-datepicker-buttonpane .ui-corner-all {
  display: inline-block;
}
.ui-datepicker .ui-datepicker-buttonpane .ui-corner-all .spanTime table {
  margin-bottom: 0;
}
.ui-datepicker .ui-datepicker-buttonpane .ui-corner-all .spanTime .datetimepicker_newMonth {
  display: inline-block;
  width: auto;
  margin-top: 0.2rem !important;
  margin-left: 0.2rem !important;
  margin-right: 0.2rem !important;
}
.ui-datepicker .ui-datepicker-buttonpane button {
  border: none;
  border-radius: 2px;
  display: inline-block;
  height: 36px;
  line-height: 36px;
  padding: 0 2rem;
  text-transform: uppercase;
  vertical-align: middle;
  -webkit-tap-highlight-color: transparent;
}

and also you'll have to run this JS before the materialize-init.js file is loaded:
$(".datetimepicker_newMonth").addClass("browser-default")

Thanks for filling this bug and it will be fixed automatically in the next release.


AsciiDoc parser improvements

Implemented support for offset calculation is document and section header text
Added JavaDoc to the more obscure methods in the AsciiDoc implementation
Added some of Ito-san's AsciiDoc header offset tests

Thank you very much as always @gerryhocks!

LGTM


Fixed: build warnings (#2d932bv)

Related Issues

Closes #
Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

IMPORTANT NOTICE - Remember to add changelog entry
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules

Outdated PR, closing


MacOS support planned/possible?

Hello. I appreciate that you have developed this module. Will there be support for MacOS?

Hey, thank you for using it, electron already added this feature for MacOS.

the only issue with their approach is you can't receive input or mouse movement

closing the issue, no response.


#1168 Add repo link to nuget package

Types of Changes
Prerequisites
Please make sure you can check the following two boxes:

[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project

Contribution Type
What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[x] All new and existing tests passed

Description
This change sets the repository url for the nuget package which will be shown on Nuget.org and can be used for analysis by third party tools.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Footer Links are not appearing as links  - appearing as simple non clickable text

Describe the bug
The footer section contains various sections such as Explore, About, Community which includes various sub topics or various services .
Ideally the sub sections should be clickable links but they are appearing as simple text without any interaction facility.
To Reproduce
Steps to reproduce the behavior:

Go to the footer section of the website and try to click on any of the sub categories of Explore, About, Community section.

Expected behavior
The expected behavior should be links highlighted and link clicks redirects to the related section of the website or the place where that content is there.
Logs
If applicable, add logs to help explain your problem (e.g. the error message and/or exception traceback).
Environment (please complete the following information):

OS:
Windows
Browser:
Chrome

Additional context
Add any other context about the problem here.

Hey @SohamPatel0810 I would like to work on this issue

Hello, Myself Ayesha Firdaus!
I am newbie at open source and I want contribute open source
Can you please assign me some tasks and give me an opportunity to become a contributor?
Thank you,
Waiting for a positive response :)

Hello, Myself Ayesha Firdaus! I am newbie at open source and I want contribute open source Can you please assign me some tasks and give me an opportunity to become a contributor?
Thank you, Waiting for a positive response :)

This issue is already resolved.


resource/aws_iam_role: Add role_last_used attribute

https://github.com/hashicorp/terraform-provider-aws/issues/30750

resource/aws_iam_role: The role_last_used attribute has been removed. Use the aws_iam_role data source instead. (https://github.com/hashicorp/terraform-provider-aws/issues/31656)


Implement user profile component

Completed on my personal branch.
https://github.com/tennitech/juke/commit/1008c2a1026631dfc93c86f3a1dc53f72ca5589e


intermediate status

intermediate status

Coverage decreased (-0.02%) to 88.049% when pulling 47879356bbf8ed9a0716ab8e36503a1779d2c1aa on development into 6058b8fddecab01c9b665ecc404e9f2bd27eed20 on master.


[chart] Replace Big Number with Big Card

Is your feature request related to a problem? Please describe.
I would like to show a single metric/dimension value from db in a report. Currently the only option for that is Big Number chart type, which can work with only Numbers.
Describe the solution you'd like
Instead of Big Number, a new chart type like 'Big Card' or 'Big Value' should allow any data type like datetime, string, %, etc. along with some subheader text as it allows now.
And ofcourse, d3 formatting to set the Big Value formatting style.
Describe alternatives you've considered
An alternative could be supporting SQL query results in Markdown component of dashboard. Users can setup their formatting style in markdown, and get the values from SQL query. But this solution will need extra effort to support filtering effects.

@maloun96 🙏

Also could use this feature!
Our current work around is to use a word map chat in a dataset that returns only one series, but obviously this is not its intended use.
Similar issues were raised in https://github.com/apache/superset/issues/1784, https://github.com/apache/superset/issues/2919 and https://github.com/apache/superset/issues/6771 but were never finished.

@kamalkeshavani-aiinside @inesplc can you send us the link to the library? thanks!

@junlincc what library are you asking about?

Looks like there is some good activity happening on apache-superset/superset-ui#1390. We would love to see this feature.

Looks like the new PR (in this repo) is still active: https://github.com/apache/superset/pull/17903

Here's a very minimal example of a Handlebars chart that makes a chart with a single text value:

Data is "Raw Records" with a single row.  I wanted the value to look exactly like this and in order to get both the $ prefix and B suffix I had to process the value into a string outside of Superset.

@sfirke Thank you.
With Handlebars I think we can achieve what is requested in this feature request, so closing this for now.


prevent image resolution loss

Render and copy images at the current scale, instead of downscaling first thus losing resolution, then rescaling them.
Summary
Render images without losing resolution.
This PR fixes/implements the following bugs

[ ] Bug 1: images lose resolution when the scale is greater than 1

Explain the motivation for making this change. What existing problem does the pull request solve?
Example image generated after the fix:

Original rendering with version 1.3.2:

Do you have an example repro for this issue?

Just created one: https://jsfiddle.net/6e170o3n/
Note: my devicePixelRatio is 2


breeze suitable only for non-commercial usage?

Breeze library depends on "net.sourceforge.f2j" % "arpack_combined_all" % "0.1",
(see https://github.com/scalanlp/breeze/blob/master/math/build.sbt#L11)
When you download this artifact from Maven central https://repo1.maven.org/maven2/net/sourceforge/f2j/arpack_combined_all/0.1/ and extract it, you can find org/j_paine/formatter/CJFormat.class file and the assumed source code file of this Java class is https://github.com/barak/f2j/blob/master/jlapack-3.1.1/src/util/org/j_paine/formatter/CJFormat.java#L8
CJFormat.java file has following License:

Permission to use, copy, modify, and distribute this
software and its documentation for NON-COMMERCIAL purposes
and without fee is hereby granted provided that this
copyright notice appears in all copies.

So does this mean that also breeze is not suitable for any commercial use?

try just deleting the dependency. should be fine

I will try that

Just wanted to add that I also am running into this for a work prototype. I am considering creating an internal build with this dependency removed. I wonder if there's an alternative for achieving the same functionality without the restrictive license? Very possible that there is not though -- just curious.

@Syyllinen -- were you able to get things up and running with this dependency removed?


[Performance] Import the load performace

Doris' load performance is very bad in some cases. Especially when using broker load to load large quantities of data, the system resources may be idle, but the load is very slow.
Currently I found the following problems:

The execution plan of Broker Load has only one concurrency on a single node, so even if the system resources are idle, the load speed cannot be improved by increasing the concurrency.

Memtable is written to disk serially. By default, a disk has two flush threads, and the current flush thread pool strategy is serial execution. In fact, each memtable can be written to disk in parallel.

A loadChannel may correspond to data sent by multiple OlapTableSinks, and because of the problem of LoadChannel lock granularity, the data receiving is a serial operation. Therefore, in some cases, even if the BE node is added, the load speed cannot be improved.

I am trying to solve these problem.

duplicate #5281

duplicate #5281


Is it possible: Embed into a video's "metadata" field the info that we can get with print-to-file

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Is it possible to have info that we can get with print-to-file embedded into the video file in the metadata field? I don't want to always have an extra file and having the info inside the video container works just as well.

also maybe https://github.com/yt-dlp/yt-dlp#modifying-metadata

I've found this adds the info I need into description. But it maxes out at 255 characters.
yt-dlp C0DPdy98e4c --embed-metadata --parse-metadata "%(title)s %(id)s %(duration_string)s %(upload_date)s %(format_note)s %(filesize_approx)s %(vcodec)s %(webpage_url)s %(thumbnail)s %(channel)s %(uploader)s %(uploader_id)s %(uploader_url)s %(is_live)s %(live_status)s %(ext)s %(duration)s %(aspect_ratio)s %(resolution)s %(fps)s %(format)s %(format_id)s %(width)s %(height)s %(tbr)s %(dynamic_range)s %(vbr)s %(acodec)s %(abr)s %(asr)s %(audio_channels)s %(display_id)s %(fulltitle)s %(description)s %(age_limit)s %(availability)s %(was_live)s %(categories)s %(tags)s %(playable_in_embed)s %(extractor)s %(extractor_key)s %(epoch)s:%(description)s"
I've manually added data to description field & it has successfully included all the data so it is possible. Is this a YT-DLP limit?
(this is the manual cmd I used exiftool -overwrite_original -Description="description_data" "input_video")

We use ffmpeg, not exiftool. Try with ffmpeg and if disaparity still exists, post verbose log.

Set description info manually with ffmpeg & all the data managed to show up.
Here is verbose log https://pastebin.com/Urqmw8hr
Output data of --print-to-file: https://pastebin.com/agh3S3TE
Running ffprobe -hide_banner "TEST VIDEO [C0DPdy98e4c].webm": https://pastebin.com/9mMy2ckU
Some values in DESCRIPTION show up as NA, but these values are present in the file I produced with --print-to-file

Set description info manually with ffmpeg & all the data managed to show up.

Could you show me the command you used for that? I can't reproduce.

Some values in DESCRIPTION show up as NA, but these values are present in the file I produced with --print-to-file

parse-metadata happens before format selection by default and so format-specific fields don't exist yet. Change it to --parse-metadata video:"%(title)s ...

Could you show me the command you used for that? I can't reproduce.

#!/bin/bash

input_video=$1
metadata_file=$2

description=$(cat "$metadata_file")

ffmpeg -i "$input_video" -metadata DESCRIPTION="$description" some_metadata.mp4

You are re-coding it to mp4, while in original, it's webm. This is what's causing the disparity. It appears webm has limited size for the metadata fields.

Ran the command again with -S ext to download it in mp4 (verbose logs: https://pastefs.com/pid/500130), the description field has same data as https://pastebin.com/9mMy2ckU.


fix: redirect on session reset

Redirect to REACT_APP_BASENAME instead of hardcoded /.
REACT_APP_BASENAME example added to the configuration example file
logo URL fixed when REACT_APP_BASENAME is defined.

Great job @PauloASilva ! Thanks for another contribution.


Grantmaster

Project Abstract
GrantMaster is a comprehensive web application tailored to managing grant applications and deliveries for the W3F Grants Program, leveraging data from GitHub repositories. The primary aim is to streamline the grant management process for the Web3 Foundation grant committee, providing an interactive and user-friendly interface for better navigation and data management. GrantMaster encompasses a range of components, including an efficient crawler for data extraction, a REST API for data access, and a visually engaging frontend. The application is built to support transparency, ease-of-use, and a seamless transition from the existing system, bolstering the growth and development of the Dotsama ecosystem.
Grant Level

[x] Level 1:  Up to $10,000, 2 approvals
[ ] Level 2:  Up to $30,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (project_name.md).
[x] I have read the application guidelines.
[x] Payment details have been provided (bank details via email or BTC, Ethereum (USDC/DAI) or Polkadot/Kusama (USDT) address in the application).
[x] The software delivered for this grant will be released under an open-source license specified in the application (Apache 2.0).
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[ ] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @_______:matrix.org (change the homeserver if you use a different one)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you for this application @Zaniyar. I am glad that you decided to tackle that RFP.
One question about your approach: you propose a button to trigger the data reading and updating the database. But can you elaborate on how exactly this will work? Will it read everything again and search for differences and make the updates? Cause that could be pretty slow and under-optimal right? Or does it remember the last state and do some kind of git fetch/ git update and work on the diff? Also how does it manage PRs and actions like comments, PR approvals, etc.?
Also, did you consider having the update scripts callable through an API and have us add some github actions in the repos so that the scripts can be ran automatically when the repos receive PRs or merges? That way, we won't have to click buttons to update things. And could make the update scripts focus on the changes only, triggered by the github repo itself.
Please provide as many details as possible about how the data update scripts will work and include them in the deliverables tables.
Also, can you please add to the application:

the programming languages and frameworks that you will use for the different components.
a documentation for the database schema as part of the deliverables.

Thank you very much!

@randombishop  thanks for the feedback. I've gone ahead and added the DB schema to the proposal as TypeScript interfaces. It's a rough draft and can change when things become clearer during development, but I see this as a good starting point.
The tech stack, including use of Mongoose for dealing with schemas, is detailed under the "Project Details" section. Let me know if this meets your expectations or if you need any additional information.
I also appreciate the suggestion of a push-based system (GitHub Actions) over a pull-based one (manually starting the crawler). It does make the data syncing simpler, so I'm planning to roll with that. The idea is to have a configurable crawler that can selectively update specific applications or deliveries and can parse the data from GitHub Actions for incremental updates.
Does this approach sound good to you? If it does, I'll get the proposal updated accordingly.
Looking forward to your thoughts!

Thanks @Zaniyar !
Yes, in your own words, "the idea to have a configurable crawler that can selectively update specific applications or deliveries and can parse the data from GitHub Actions for incremental updates." sounds very good to me.
Just add this to the deliverables then ping me and I will mark this application as ready to review for the rest of the committee.

@randombishop thanks, I've made the following changes:

I've added a chapter that describes the data synchronization approach
I've updated the project's milestones to reflect these changes

Please have a look again when you have the chance.

Thank you @Zaniyar !
The approach looks good to me now.
I have some final details to suggest to the specifications: the entities represented in your DB schema cover grants, milestones and deliverables; but not the actual deliveries and evaluations. Can you add these in your list of entities to make sure we are covering the full scope of the data to be parsed and loaded to the database. (It's ok if the definitions are not exact and will change as you implement, as long as the entity is accounted for and included in scope, like you already mention in that section.)
An optional good to have would be Github repositories too, the deliveries will often link to github repos and it would be nice if they were recorded in a structured way.

@randombishop thanks for the suggestions. I've extended the interfaces with the following attributes.

Grant interface:

applicationLink: string;: points to the application doc on GitHub
applicationDoc: string;: matches the content of the application doc

Milestone interface:

githubPrs: string[];
deliveryLink: string;: points to the delivery doc on GitHub
deliveryDoc: string;: matches the content of the delivery doc
evaluations: Evaluation[];: i saw that for some of the deliveries there are multiple evaluations, hence I'm using Evaluation[] here

Also, I've added the Evaluation interface:
typescript
interface Evaluation {
author: string;
link: string;
doc: string;
}
I hope that this matches your expectations.
Regarding the GitHub repositories, I think we should just create forks of them on GitHub instead of copying into a database. That way we can update the forks in an easy manner. On the other hand, if we persist them manually it would make the syncing process unnecessarily complex.
As for recording the links to the repos, we can certainly do that. I've added codeRepos: string[]; to the Milestone interface.
Note that for the added complexity I slightly increased the expected FTE as well as the costs while keeping the rate and expected duration unchanged.

Thanks for the update @Zaniyar !
Yes, I agree that recording github repos is simply for the links and not the data itself. We actually do fork all the repos in our organization so it could be good to keep the records in the database. FYI we also run a script to monitor github activity on these repos (commits, forks, stars, etc.) so the grant management app could be a good place to run these scripts, store this information and display reports; but no need to add it in current scope, could be something to add later on in V2...
Anyways, happy to go ahead with current version and tag this application as ready for review for the rest of the team.

@semuelle I understand. I adjusted the document with the changes I deemed neccessary. I moved the second milestone into the "Future Plans" chapter and updated the prices. Could you have a look?

Hey @Zaniyar, any updates on your milestone? Are you still working on it? Any issues we might be able to help with?

Hey @semuelle sorry, it's taking me a bit longer than expected, but I'm still working on it. I'm confident I can deliver by the end of October. Does this work for you? thx

Hey @semuelle sorry, it's taking me a bit longer than expected, but I'm still working on it. I'm confident I can deliver by the end of October. Does this work for you? thx

Hey @semuelle sorry, it's taking me a bit longer than expected, but I'm still working on it. I'm confident I can deliver by the end of October. Does this work for you? thx

Hey @Zaniyar. It's probably easiest if you submit a pull request with the updated timeline. We have no reason to rush anyone, we just like to stay on top of things.

Hey @semuelle here is the pull request: https://github.com/w3f/Grants-Program/pull/2037 - I hope this works for you.


Generate manifest at runtime

The promise of fresh is great: Just write code and push to GitHub. No build step required, as everything is handled at runtime by Deno Deploy. As no local setup is needed this opens the door to writing code everywhere where there is a text editor with Git integration like on mobile devices.
However, in practice fresh still has a build step even if tiny: generating the manifest. So, we’re back at square one requiring a local developer installation, a terminal, fiddling with shell scripts, PATH variables, etc. pp. Remote code environments like GitHub Codespaces offer only a limited workaround as they just shift the problem to another device, and often creating new problems in the meantime like being limited to a less performant Web client, no mobile support, no persistence in containers, paid, etc. which makes things even more difficult. My current workaround is running it in a GitHub action workflow that creates a pull request with the updated manifest.
Generating the manifest at runtime would eliminate this. Currently this will most probably give a performance hit, as it needs to generate it over and over again for each request, but this could be solved in the future when Deno Deploy gains persistence.
In the meantime until Deno Deploy gains persistence, fresh could support both, by using a pre-built manifest if available and if it isn’t generating it at runtime, so developers have the choice of less complexity for a performance hit.

We can unfortunately not generate the manifest at runtime, as Deno Deploy does not support dynamic imports. Once it supports those, the manifest no longer needs to be generated.
I am just waiting on one final issue in Deno to be resolved (https://github.com/denoland/deno/issues/14685), before being able to generate the manifest seamlessly during development in fresh.

Thanks. Let’s hope Deno Deploy supports dynamic imports soon so that fresh can generate the manifest at runtime.
Meanwhile automating the build step with  #223 is a nice first step. Though unlike mentioned it doesn’t fix this issue “Generating the manifest at runtime”, since a build step is still necessary, and with the above mentioned complexity of a local developer setup.

@lucacasonato Please reopen, since this is not resolved as mentioned in my previous comment.
Even the 1.0 announcement makes the incorrect claim that there is no build step.

The framework has no build step

There is no build step, if one can push code to Deno Deploy without ever having it run locally using the Deno CLI.

@vwkd That is not a build step.

@lucacasonato What do you understand by “build step”?
If not a build step, how would you call necessary code generation (manifest file) after development (writing code in a text editor) before runtime (pushing to Deno Deploy)?

This file does not change every time you change code. It changes only if you update the file structure of your project. You can make code changes to your project, not run it locally, and it will still deploy correctly using the correct code and correct output. As such it is not a build step.

So, you’re saying a “build step” that only happens sometimes is not a “build step”?
So, most transpilations like TypeScript to JavaScript are also not a “build step”, because the output doesn’t (need to) change for certain input changes (assuming the transpiler is smart enough), like adding an empty line to the input file.
Haven’t seen this concept of a “build step” anywhere else.


Handling when ans is assigned to user defined variable

Current implementation => assignment to user defined variable is binding of AST to the variable.
When ans is bonded to particular variable which is better?
(1) When ans change, the variable will change.
(2) When ans change, the variable will keep old value.

3d15fafcf4dc936b9b8662fa75fec4061633a662


Fixed incorrect meta charset value

Dear Max,
Thanks a lot for your pull-request.
Are you planning to further use the SOS client or even to provide
additional fixes? If you are interested in contributing further code
to the SOS client project (and the corresponding GitHub repository),
we need your official permission for publishing your code. For this
purpose we have a so called contributors license agreement (CLA) in
place. With this agreement you grant us the rights to use and publish
your code under an open source license.
The contributors license agreement is available here:
http://52north.org/images/stories/52nadmin/files/52north_contributor_license_agreement.pdf
Please note: as you are working on the Sensor Web implementations as
part of your employment it is important, that the CLA is signed by an
authorised representative of your company.
Further explanations about the contributors license agreement are
available here: http://52north.org/about/licensing/cla-guidelines
Finally, we have also prepared an FAQ about the contributors license
agreement: http://52north.org/about/licensing/cla-faq
We are really looking forward to include your modification into our
implementation. If you have any further questions or need for
clarification just let us know. We will be happy to help you.
Again, thanks a lot for looking at this repo and your
pull request.
Best regards
Jan


Unshift can corrupt array

The following prgram outputs 0 on some machines when compiled with optimizations.
actor Main
  new create(env: Env) =>
    let a = Array[U8].init(1, 5).>unshift(2).>unshift(3)
    try env.out.print(a(6)?.string()) end

The rest of the array is as it should be.
Works fine when there is only one call to unshift.
Works fine if the array is shorter than 5 intially.
Works fine if compiled with ponyc -d
$ ponyc -v
0.21.3 [release]
compiled with: llvm 5.0.1 -- cc (GCC) 7.2.1 20171224

ponyc is the one from the arch linux package manager.
x86_64

As of 0.37.0 I can't reproduce this bug. Closing.
If anyone can reproduce, it can be reopened.


[Bug report] Rename mysql table through api, if new name included uppercase, api response NoSuchTableException.

Version
main branch
Describe what's wrong

Error message and/or stacktrace
N/A
How to reproduce
1.Rename mysql table through api
2.New name included uppercase
3.API response NoSuchTableException ,but actually table name had been renamed to new name (convert to lower-case).
Additional context
No response

@mchades please check this issue.

mysql configuration:
docker run -p 3306:3306 --name mysql --detach -e MYSQL_ROOT_PASSWORD="mysql" -e MYSQL_ROOT_HOST=% -e MYSQL_DATABASE=mydb -d mysql/mysql-server:latest --lower_case_table_names=1 --init-connect='GRANT CREATE USER ON . TO 'root'@'%';FLUSH PRIVILEGES;'

Why are you set lower_case_table_names=1?
The process of renaming a table is to first alter and then load. Currently, jdbc-mysql-catalog assumes that the underlying MySQL is case-sensitive in table name, but your MySQL service is configured with lower_case_table_names=1. This results in using uppercase names during loading, causing an error indicating that the table does not exist, when in fact the table has been successfully modified to lowercase.
In most cases, MySQL document recommends using the default configuration(0 on Unix, 1 on Windows, 2 on macOS):

If you are using MySQL on only one platform, you do not normally have to use a lower_case_table_names setting other than the default.

Although MySQL document says:

lower_case_table_names can only be configured when initializing the server. Changing the lower_case_table_names setting after the server is initialized is prohibited.

and we can make case-sensitivity a configurable option for the catalog to solve this problem, it will make the code more complex and is not necessary in most cases. Therefore, I suggest setting lower_case_table_names to 0 in your tests and do the code changes when encountered in a production environment. WDYT? @danhuawang @jerryshao

Sure. I will apply changes on testing and create an issue to track [Improvement] Add --lower_case_table_names=0 parameter for mysql db instance in testing environment. #147
Reopen it if it is required in a production environment


URI decoder

For the STORE project, a URI decoder is needed.  It doesn't need to be extremely fancy, but needs to extract each major part of the URI (scheme, authority, path, query and fragment) into separate strings.
The URI decoder will be internal only.

@levitte can this be closed now? Moving to post 1.1.1 in any case.


Let me help

Topo of the summit of Timberwolf Mountain
coords.txt

Updated Coordinates For Timberwolf Mountain
coords.txt


Fix invalid renewal dates

Yes it is definitely worth, i'd like to get this out of the door though relatively quickly

Not working as expected - it should be possible to reject the visit simply by ticking the checkbox. Not mandatory to enter a renewal date. @StupidCodeFactory @alan

Also, it's currently possible to reject the request with only the Day and Month. This should create a validation error without the year.

It's currently possible to enter a VO renewal date in the past. I was able to process the visit with VO renewal date 1/1/2016.

@kyliemulholland the date in the past has always been the case, do you want to fix this is part of this bug?

It should not be possible to process a VO renewal date without checking the checkbox. This is possible currently.

@alan see above comment - Fine to push out the bug fix without this, and work on it afterwards.

@StupidCodeFactory just reviewed, looks like it is currently not possible to process a rejection by only selecting the checkbox.

@kyliemulholland yes just saw that as well

Entering a date without selecting 'no allowance' checkbox and another reason ie 'child protection measures the visit is rejected because of child protection. Think we want to force the date to be blank if the 'no vo allowance' is not checked. @kyliemulholland can you confirm?

Reviewed and good to go!


Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

I'd like to take on this issue.

I'd like to take on this issue.

I assigned you manually but you could use the dot take comment :D
Will check the PR shortly!
Thanks a lot! 🍕

@0-vortex Thanks for your patience. I created a PR for this issue.


[Playground] Examples slider improvements

When opening the examples slider we should focus on the search field
When selecting an example the slider should close

I can take care of that


Suggestion SOC residual

Description
This PR points to the bugfix/soc_resid branch and adds handling for the edge cases, taking inspiration from @rileyjmurray 's code in the issue and the mosek docs.
(The linters were failing here, so I merged the master into this branch. Merge master into bugfix/soc_resid to get rid of the noise in this PR).

The changes for SOC residual look great! I'll merge so that we can continue with the other PR.


ha_strava not running on 2021.12.0b2

I'm running 2021.12.0b2 and am getting the following error:
Logger: homeassistant.setup
Source: setup.py:153
First occurred: 1:41:17 PM (1 occurrences)
Last logged: 1:41:17 PM

Setup failed for ha_strava: Unable to import component: cannot import name 'HTTP_OK' from 'homeassistant.const'
(/usr/src/homeassistant/homeassistant/const.py)

I've opened pull request #36 which should fix the issue. Starting with 2021.12.0b0, some constants have been removed from HASS.
I don't know if this repo is still maintained (the missing version in manifest.json has not been fixed for month). However, the changes in this PR are minimal, only a copy of lines must be changed in __init__.py and config_flow.py.

It is not being maintained, perhaps @codingcyclist you could allow someone else to merge pull request, if you are not interested?

Or should someone create a fork where we can continue maintenance on this?


Change component name for consistency (TrackSearch=>SearchTracker)

Hey, I just made a Pull Request!
While looking at this #15717 issue, I stumble upon this component name inconsistency
:heavy_check_mark: Checklist

[ ] ~A changeset describing the change and affected packages. (more info)~
[ ] ~Added or updated documentation~
[ ] ~Tests for new functionality and regression tests for bug fixes~
[ ] ~Screenshots attached (for UI changes)~
[ ] ~All your commits have a Signed-off-by line in the message. (more info)~

I'm not sure if these tiny changes are acceptable or require any of the checkbox

Hi @alileza , I really liked your suggestion! This is something that bothered me a lot before.
I've created another draft with some suggestions could you take a look?
https://github.com/backstage/backstage/pull/15950
I noticed this patch is using a property (sx) present only in MUI 5 but the project is still using 4. The draft above contains a suggestion on how to adapt it and an animation of the behaviour.

@aitherios oh nice! you get it working, do you want to submit the PR? or should I cherry-pick the commit here?

@alileza please cherry pick the commit.


feat(curriculum): Add play scene keyboard shortcut

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #52696

Adds "ctrl + space" keyboard shortcut for playing scene as suggested in https://github.com/freeCodeCamp/freeCodeCamp/issues/52696#issuecomment-1884048129
Also updates the shortcut modal with the new shortcut info

 Screenshot attached

There is some issue with the translation, I have added the string to the english translation file, but its not getting translated and the key is directly printed

Running pnpm run clean:client should fix that.

Just checking up, is anything blocking the PR?

Heyy @Sembauke , They are not in scope for this PR , Maybe it would be better to fix them in different PR? But, yes I can include them if you want.


Enhancement gcp disks

The PR needs to have a better title and description.

Add GitHub README changes as well


[RHPAM-4026] - [OpenShift] Users and Groups are not displayed in Admin perspective

perspective
Cherry-picks: https://github.com/jboss-container-images/rhdm-7-openshift-image/pull/457
Signed-off-by: spolti<EMAIL_ADDRESS>Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[x] Pull Request title is properly formatted: [RHDM-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[x] Attached commits represent units of work and are properly formatted
[x] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[x] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

/lgtm


Text of clicked buttons not echoed to chat window

When a user clicks a button of a rich card sent by the bot in the webchat window, the text of the button on which the user clicked does not appear in the chat window as if the user would've typed it himself, unlike in messenger for example.
The problem with this is that if the user scrolls up in the conversation, all he sees is text written by the bot and it makes it hard for him to see what choices he's made (what buttons were clicked), and therefore what bot's answers correspond to what choices.
In Messenger, bot asks the question:

In messenger, user clicks on a button, the text is displayed as if he would've typed it:

In webchat, bot asks the question:

In webchat, user clicks on a button, the text is not displayed as if he would've typed it:

Thanks for your help!

This depends on the action type of the button. If it's "imBack" then the text is displayed in the history. If it's "postBack" then it's sent to the bot silently. Not every messaging client supports both, but WebChat does.
This appears to be correct behavior so I'm closing this issue.

Thank you, you are absolutely right, using imBack does exactly what you describe. Cheers!


No way to pass the HttpClient a CA for schema registry

Description
I'm not seeing a way to pass the CA for the schema registry. I found the source code that creates the HttpClient and it watches for "https" but it doesn't have a way to plumb the certificate through.
Unless I'm missing something, this means I either have to install self-signed certificates or turn verification off. I would rather not do either, and I think it would be a nice option to be consistent with the Kafka client anyway
How to reproduce
Use a self-signed certificate for the schema-registry, then it would be nice to have an additional config property like...
var schemaRegistryConfig = new SchemaRegistryConfig
{
  SchemaRegistryUrl = "https://localhost:8085",
  SslCaLocation = @"C:\self-signed-ca.crt" // this doesn't exist
};

using (var schemaRegistry = new CachedSchemaRegistryClient(schemaRegistryConfig))
{
  schema = schemaRegistry.GetLatestSchemaAsync("").Result;
}

Checklist
Please provide the following information:

Confluent.Kafka 1.0
Apache Kafka 5.2.1

thanks for the +1 - yep, this is on the todo list, particularly as SR is now in ccloud.

CachedSchemaRegistryClient still does not work with self-signed CA certificates.
It throws AuthenticationException: "The remote certificate is invalid according to the validation procedure." unless the certificate is installed into Trusted Root Certification Authorities, or EnableSslCertificateVerification parameter is set to false.

 No need to supply a project file:

var schemaRegistryConfig = new SchemaRegistryConfig()
{
    Url = "https://localhost:8081",
    SslCaLocation = @"C:\path-to-self-signed-crt\kafka-ca-1.crt",
    SslKeystoreLocation = @"C:\path-to-keystore\kafka.keystore.p12",
    SslKeystorePassword = "123456"
};
var schemaRegistry = new CachedSchemaRegistryClient(schemaRegistryConfig);

var x = schemaRegistry.GetLatestSchemaAsync("topic-name").Result;

[x] Confluent.Kafka nuget version: 1.6.2
[ ] Apache Kafka version.
[ ] Client configuration.
[x] Operating system: Windows 10 Pro 20H2
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[ ] Critical issue.


learn: Change instructions to be less restrictive in Slice and Splice challenge

In the Slice and Splice challenge, the following line tells users to use slice and splice, but the tests do not force the user to use slice and splice.  Since this is an algorithm challenge, I believe we should let the users write the code any way that solves the problem.  My suggestion is to change the line's text to the following:

Copy each element of the first array into the second array, in order.

@scissorsneedfoodtoo @thecodingaviator @Manish-Giri Any thoughts?

I'll try to solve problem

@PeterF521 Before creating a PR, let's wait for feedback from at least one of the users I tagged.

@RandellDawson, sure, that sounds like a good idea, especially since we're not testing for those methods.

@RandellDawson I think this sounds like a good improvement

ok

@PeterF521 Go for it.

Nice, I'm thinking use asset.CallTracker on the testString, but I first want to know what's the asset version used on this repository.
If you want that I use other way, please tell me. I also want to learn more about JavaScript

Or use typeof keyword?

Hello @PeterF521 ,
There should be no need for either of those. The idea is to change the wording of this line of text:

Use the array methods slice and splice to copy each element of the first array into the second array, in order.

To this:

Copy each element of the first array into the second array, in order.

There is no need to change the tests, as they do not check for the slice or splice methods.

Thanks. It's very much easy!


push triggers 4 deployments for git components

[kind/bug]
odo push triggers 4 deployments and 2 builds for git components
Actual behavior
4 deployments and 2 builds
Expected behavior
1 deployment and 1 build should be triggered
Any logs, error output, etc?
[mrinaldas@localhost /]$ oc get dc
NAME              REVISION   DESIRED   CURRENT   TRIGGERED BY
nodejs-dnuu-app   2          1         1         config,image(nodejs-dnuu-app:latest)
[mrinaldas@localhost /]$ oc get dc
NAME              REVISION   DESIRED   CURRENT   TRIGGERED BY
nodejs-dnuu-app   6          1         1         config,image(nodejs-dnuu-app:latest)

/king-bug

/kind bug

/assign

@mik-dass i am closing thsi issue as it has been addressed in pr #1684.

/close

https://github.com/openshift/odo/pull/1684 doesn't resolve this issue
/reopen


Error thrown when using Styletron inside storybook

Has anyone had any success using Styletron components inside storybook? I'm getting a an error thrown inside my storybook: Cannot read property 'injectDeclaration' of undefined

Haven't used storybook recently, but this error is probably due to the lack of a styletron instance being defined within context. You'll need to use StyletronProvider
A quick search looks like this might be the solution: https://github.com/storybooks/react-storybook/issues/76#issuecomment-226834760 looks

Something like this should work, I think
import React from 'react';
import { action, storiesOf } from '@kadira/storybook';
import { StyletronProvider } from 'styletron-react';
import Styletron from 'styletron';

const styletron = new Styletron();

storiesOf('Styletron components', module)
  .addDecorator(story => (
    <StyletronProvider styletron={styletron}>
      {story()}
    </StyletronProvider>
  ))
  .add('component', _ => (
    <Component />
  ));

👍  that works :) Thanks @rtsao :)
I wonder if there's a way to add that decorator globally rather then adding it to each each individual story...

This is our config using it as a global decorator:
import React from 'react';
import {configure, addDecorator} from '@kadira/storybook';
import Styletron from 'styletron-client';
import {StyletronProvider} from 'styletron-react';

const styletron = new Styletron();
addDecorator((getStory) => (
  <StyletronProvider styletron={styletron}>{getStory()}</StyletronProvider>
));

configure(() => {
  // ...
}, module);

Thanks @faddee ! That works.

This was my final config:
import React from 'react'
import {configure, addDecorator} from '@kadira/storybook'
import Styletron from 'styletron-client'
import {StyletronProvider} from 'styletron-react'

const styletron = new Styletron()
addDecorator((getStory) => (
  <StyletronProvider styletron={styletron}>{getStory()}</StyletronProvider>
))

const req = require.context('../components', true, /.stories.js$/)

function loadStories () {
  req.keys().forEach((filename) => req(filename))
}

configure(() => {
  loadStories()
}, module)

The apis for storybook and styletron have seemingly updated since feb 26, 2017.  @rtsao Do you have any insight on how to get styletron running in storybooks this year?
I tried
import Styletron from 'styletron';
import {Provider} from "styletron-react";
const engine = new Styletron();

storiesOf('Button', module)
  .addDecorator(story => (
    <Provider value={engine}>
      {story()}
    </Provider>
  ))

but I'm now getting an error of
styletron.renderStyle is not a function

@matthewharwood Try this (new storybook/styletron APIs):
import {addDecorator} from '@storybook/react';
import {Client as Styletron} from 'styletron-engine-atomic';
import {Provider as StyletronProvider} from 'styletron-react';

const engine = new Styletron();

addDecorator(story => {
  return (
    <StyletronProvider value={engine}>
      {story}
    </StyletronProvider>
  );
});

@rtsao, that code doesn't seem to work for me.
    in DevProvider

Still digging into it. Will post back if I can figure out what's going on.


Windows installation (2)

Hi again,
I started installing RaiseWikibase on a Mac parallel to Windows, so I can see if problems are Windows specific.
The jobrunner seems to have, again, an issue with (probably) encoding:

")syntax error: invalid arithmetic operator (error token is "

")syntax error: invalid arithmetic operator (error token is "

wait-for-it.sh: waiting for mysql.svc:3306 without a timeout

wait-for-it.sh: mysql.svc:3306 is available after 0 seconds

/jobrunner-entrypoint.sh: line 12: $'\r': command not found

/jobrunner-entrypoint.sh: line 13: syntax error near unexpected token `$'{\r''

'jobrunner-entrypoint.sh: line 13: `kill_runner() {

/jobrunner-entrypoint.sh: line 2: $'\r': command not found

/jobrunner-entrypoint.sh: line 5: $'\r': command not found

")syntax error: invalid arithmetic operator (error token is "

")syntax error: invalid arithmetic operator (error token is "

wait-for-it.sh: waiting for mysql.svc:3306 without a timeout

wait-for-it.sh: mysql.svc:3306 is available after 0 seconds

sleep: invalid time interval '1\r'

Try 'sleep --help' for more information.

Line endings are differently encoded. Is there a way to ensure they have the correct one for Windows as well?
Best,
Michael

Hi Michael, the jobrunner-entrypoint.sh comes from Wikibase Release Pipeline. May be you could report it there or even at Phabricator.
Do you run docker-compose -f docker-compose.yml -f docker-compose.extra.yml up -d --scale wikibase_jobrunner=2 ? What does it say if you run only docker-compose -f docker-compose.yml -f docker-compose.extra.yml up -d ?

Well that one is on me :(
I must have opened the file with a Texteditor and Windows converted linebreaks etc... and could not reproduce the issue.
I need to install the whole package on a different windows machine in a few days. I will reopen the issue if it shows again.


Update Loops in Progress

Add me so I know I can do a PR. (And to get on the list)

You'll need to reopen this if you'd like me to merge it.


Test 3 Issue

I may be doing something wrong, but I have pored over the code I wrote as well as the test code. The test code (for test 3) is looking for a return value of 1 when calling the indexOfMaxGrade function. If the function runs as it should, the return value, based on the provided array in test, should be 8. Index 8 points to the double "100.00" which is the highest grade in the array that is provided. Like I said, maybe I am doing something wrong, but I am almost certain my code is correct. Thanks!

I just realized I was literally covering the 100.00 at index 8 due to my mouse placement. It is there, I promise LOL

You're misunderstanding the intent of the test. maxGrade() should give you the index of course, i.e., of the row, that contains the maximum grade. Since the maximum grade of 100 is in the second row, i.e., the row indexed by the number 1, the expected value is, in fact, 1. You're looking at this as if it were a one dimensional array...

Ah I see. I was calculating it using pointer arithmetic when I originally made the function (I was passing by reference), and forgot to change my calculation whenever I switched parameters. Thank you for the info.

Thanks for raising the issue, and I'm glad we got that cleared up.


Be lenient about whitespace in pipeline parser

This makes it possible to also include whitespace characters like non-breaking spaces and newlines in pipelines, effectively allowing the user to write complex pipelines with formatting however they prefer. For example, if you'd like to get really funky with your formatting, this would now be possible:
vast export json <<EOF
identity
| drop ts |
select orig_h,
  orig_p
 ,resp_h
  | identity
EOF

@Dakostu I requested you for review since we had already talked about this one—and I have mistaken the char classes the last time around.

So parsers::space does in fact do consider more complex whitespace such as tabs and line breaks?
If that is the case, then OK.

Yes, I was mistaken about the two char classes last week in my review. See isspace and isblank. Sorry for the confusion.


Unable to run @SpringBootTest without WebMVC

Running a very simple Spring Boot Test fails due to Springfox:
@RunWith(SpringRunner.class)
@SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.NONE)
public class Foo {
    @Test
    public void test() throws Exception {
    }
}

The reason for this is related to #1611 as no WebMVC beans are available. However, these are required for Springfox.
Version 2.6.1
Caused by: org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'documentationPluginsBootstrapper' defined in URL [jar:file:/C:/LocalData/dev/repo/io/springfox/springfox-spring-web/2.6.1/springfox-spring-web-2.6.1.jar!/springfox/documentation/spring/web/plugins/DocumentationPluginsBootstrapper.class]: Unsatisfied dependency expressed through constructor parameter 1;
nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'webMvcRequestHandlerProvider' defined in URL [jar:file:/C:/LocalData/dev/repo/io/springfox/springfox-spring-web/2.6.1/springfox-spring-web-2.6.1.jar!/springfox/documentation/spring/web/plugins/WebMvcRequestHandlerProvider.class]: Unsatisfied dependency expressed through constructor parameter 0;
nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'java.util.List<org.springframework.web.servlet.mvc.method.RequestMappingInfoHandlerMapping>' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}

Can you exclude the @EnableSwagger2 annotation that is pulled in automatically into this test?

What do you mean with "exclude", is there a Spring property for that? I did not include any Springfox annotation in my own code, but I am using a Maven parent project that pulls it in (and most likely is the reason for its activation).

When you do a @SpringBootTest it includes your application configuration. You have to tell it to exclude specific configuration or give it a test configuration. The idea is to not include the swagger configuration in the test configuration.

OK, I understand what you mean. Explicitly excluding configurations does not work, as far as I know. Besides, having to do this in every test class seems wrong to me. Would it be possible to only start Springfox in a MVC environment, using conditional bean configuration (e.g. @ConditionalOnClass)? That's the way Spring Boot works under the hood.

@C-Otto couple of things to try. Create a configuration that you explicitly include for your tests that you can share across your tests. Also keep in mind that this is a spring library that is also used in spring boot applications. So the idea of bringing in a dependency on spring boot will not work. It has to be something in the client side testing that needs to handle this.

OK, I see. Under these circumstances I'd say my bug report is invalid. Thanks for the help!

Why is springfox behaves badly with junit?

How do I exclude @EnableSwagger2 from a test

This has not been fixed #2329

@dilipkrish
hi dilip,
I am facing the same issue even though I am using 2.9.2 version of Springfox.  Here is the error. can you please advise.
Expecting:
<Unstarted application context org.springframework.boot.test.context.assertj.AssertableWebApplicationContext[startupFailure=org.springframework.beans.factory.UnsatisfiedDependencyException]>
to have bean named:
<"customDocketProvider">:
but context failed to start:
org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'documentationPluginsBootstrapper' defined in URL [jar:file:/Users//.gradle/caches/modules-2/files-2.1/io.springfox/springfox-spring-web/2.9.2/ed2ed714a6cba8804d00f80f0534901e4c7a3211/springfox-spring-web-2.9.2.jar!/springfox/documentation/spring/web/plugins/DocumentationPluginsBootstrapper.class]: Unsatisfied dependency expressed through constructor parameter 1; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'webMvcRequestHandlerProvider' defined in URL [jar:file:/Users/.gradle/caches/modules-2/files-2.1/io.springfox/springfox-spring-web/2.9.2/ed2ed714a6cba8804d00f80f0534901e4c7a3211/springfox-spring-web-2.9.2.jar!/springfox/documentation/spring/web/plugins/WebMvcRequestHandlerProvider.class]: Unsatisfied dependency expressed through constructor parameter 1; nested exception is org.springframework.beans.factory.NoSuchBeanDefinitionException: No qualifying bean of type 'java.util.List<org.springframework.web.servlet.mvc.method.RequestMappingInfoHandlerMapping>' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}


Difference between .vscode and .config/Code/User settings

I know this isn't the correct place to ask questions, but I asked a couple of questions on Stack Overflow that never got answers, and when the auto cleaner bot deleted them for getting no answers, it counted them against me and I was temporarily banned from asking questions there for 6 months. (tbf the questions I asked were good ones, but they were for really obscure hackey things which I think most people didn't have experience with).
Here's my issue. I'm trying to sync all my settings using my personal NextCloud server. I have all my settings in a folder called Configs. I have a folder called .vscode in which I have been putting my custom settings and then symlinking to ~/.vscode on my machine. I just started moving over stuff from .config to my cloud repo and I noticed

Sensitive data I don't want to share to my public dotfile repo

Massive cache files I don't want to sync on my cloud service.

I've tried writing exclusion lists for both my cloud clients and the .gitignore files but haven't been able to really nail them down as I would wish, largely because I want to save some files and not others and it all gets really finicky.
I don't have that problem with the .vscode folder in my home folder though... but upon reading further it seems that this is just workspace setting?
So what's it doing in my home folder? ~/.vscode is associated with what workspace? Is that effectively a global workspace? Is that just a workspace for all projects which are saved in subfolders of my home folder?  Is that effectively being used as my local settings for all projects, or is that just some strange artifact from me opening a file in the ~/ folder at some point using vscode?
Also yes, I'm also already using the syncLocalSettings extension. But I want to do things my way because I'm using this as an experiment in containerizing my configs across machines (some of them being Windows, some of them being WSL, some being Mac, and some being Linux machines).

Lots of discussion about the two folders in #3884
.vscode isn't related to workspace settings at all, but to extensions. They are installed over there.
I suggest to simply sync certain text files in .config/Code/User, such as .config/Code/User/settings.json.

Thanks @joaomoreno I ended up doing basically as you said. I replaced settings.json with a symbolic link that pointed to my configs repo folder.


[Feature Request] Add an pad option log level text

What
~I would like a TextFormatter option that outputs in a 5 character / 4 character + 1 padding space output mode. I'm tentatively calling it FourAndFiveCharacterTruncation (feedback requested on that name)~
Taking in the feedback provided in the comments, I would instead like a TextFormatter option that outputs the level text with padding up to the max length. I'm tentatively calling it PadLevelText. It would look like so
INFO    some information
WARNING some warning
DEBUG   some debug information

Why
The above example is in contrast to the current default behavior of
INFO some information
WARN some warning
DEBU some debug information

And the DisableLevelTruncation behavior of
INFO some information
WARNING some warning
DEBUG some debug information

(FATAL, PANIC, and TRACE not explicitly mentioned above for the sake of conciseness, they would all output at 5 characters just like DEBUG)
Who
This feature request is essentially the "pad" option that mentioned in https://github.com/sirupsen/logrus/pull/407#issuecomment-274781317
I intend to create the PR for this myself, if you all don't mind ^^ Haven't had an OSS contribution in a too-long now. I'll start on that if I get >=1 👍 react from a Contributor

@dgsb, @sirupsen, @lavoiesl would any of you happen to have an opinion about this feature request? I'd love to help out with this package!

Indeed the truncated level looks weird to me.
But rather than pad to an arbitrary value and truncate beyond, why not pad to the largest level string representation ?
wdyt ?

Padding to the largest level string representation would look good and be less work to implement. I'm for it! @dgsb 👍

(tangentially: I think my ideal end state would be pad to the largest string representation and change the cli text of WARNING to WARN. But the warn change is a different matter, and possibly a breaking change, so I'll consider that separately.)

For the sake of completeness, here's what a pad option would look like
INFO    some information
WARNING some warning
DEBUG   some debug information

I'll edit the OP to reflect this instead (...)

Hi,
Just wondering if the "PadLevelText" option on text formatter is not yet released, though the docs say it is usable. Let me know if i am missing something.
Thanks

@gauravaditya my PR merged on July 1st and the most recent release is May 18. So no, it has no been released. The docs reflect master, not the most recent release. If you want to view the documentation for the most recent release, that's available here => https://github.com/sirupsen/logrus/tree/v1.4.2
In related news, @dgsb & @sirupsen this package could use more maintainers, IMO 🙂. I recently helped push out release 2.0 for https://github.com/urfave/cli/, and would be available to help out here! Let me know if you're interested, either here or via ✉<EMAIL_ADDRESS>


refactor(api,robot-server): move calibration check out of server

overview
This is purely a moving around calibration check code.
|--opentrons
    |-- calibration
         |-- check
              |-- __init__.py
              |-- session.py
              ...
         |-- __init__.py
         |-- util.py
         |-- session.py
         ...
   ...

changelog

Non-http code calibration check code moved out of server into a new package calibration.check.
calibration.check.Session broken up into two. A calibration session in calibration package and calibration check session in calibration.check.

review requests

[ ] Reasonable structure?
[ ] should deck_calibration also be moved to calibration?

risk assessment
Low. Nothing added and nothing deleted.

* [ ]  should `deck_calibration` also be moved to `calibration`?

for my money, no, since we're going to delete it soon - but that's definitely where we should implement the new robot calibration flows

Makes sense to me 🏞

Looks good to me!


UI BUG in Course Details Page

Summary:

Buggy UI

Correct UI Part 1

Correct UI Part 2

Steps to reproduce:
How can we reproduce the issue?
Open Course Details Page
Screen-shots:
Can be created by pressing the Volume Down and Power Button at the same time on Android 4.0 and higher.
Would you like to work on the issue?
No

fixing this