Try to reproduce voc 10-1 results

I am trying to reproduce 10-1 results as shown in the table below. I notice a large gap of the old class mIoU between my reproduce result (38.82) and your reported one (44.03), roughly 5 percent points. I am wondering what will cause this problem. I run the experiments with 2 x RTX 3090 GPU. I follow your original implementation except for the cuda version. I am using cuda 11.3 because cuda 10.2 does not support RTX 3090. Does it matter?
Btw, may I know what GPU model do you use? I think it requires to have at least 16G to hold a batch of 12 on each device and needs to support cuda 10.2 as well. V100? I guess.
Meanwhile, I notice a weird phenomenon that background performance drops drastically starting from the 8-th step and becomes 0 at 9-th step. I think this harms the old class performance a lot. Do you have a similar issue?
Thanks.

step
background
aeroplane
bicycle
bird
boat
bottle
bus
car
cat
chair
cow
diningtable
dog
horse
motorbike
person
pottedplant
sheep
sofa
train
tvmonitor
0-10
11-20
all

0
95.19%
89.84%
41.01%
89.74%
72.24%
85.02%
95.29%
88.26%
93.30%
43.15%
92.25%
x
x
x
x
x
x
x
x
x
x
80.48%
-
80.48%

1
92.01%
87.25%
35.76%
82.88%
67.51%
79.44%
94.49%
82.55%
88.44%
46.25%
90.74%
37.38%
x
x
x
x
x
x
x
x
x
77.03%
37.38%
73.73%

2
89.16%
84.00%
34.32%
83.28%
63.44%
68.45%
92.90%
82.54%
82.37%
31.68%
84.48%
24.68%
62.59%
x
x
x
x
x
x
x
x
72.42%
43.64%
67.99%

3
83.97%
79.07%
31.92%
73.50%
53.19%
51.06%
91.18%
80.90%
76.82%
17.40%
65.26%
18.69%
3.93%
22.11%
x
x
x
x
x
x
x
64.02%
14.91%
53.50%

4
79.89%
77.61%
35.83%
74.77%
47.67%
49.70%
90.73%
78.02%
79.41%
13.21%
65.81%
17.77%
9.43%
20.52%
33.81%
x
x
x
x
x
x
62.97%
20.38%
51.61%

5
84.56%
84.34%
37.88%
81.72%
53.81%
56.75%
90.72%
82.48%
82.98%
13.77%
69.20%
0.57%
0.00%
26.10%
58.37%
69.71%
x
x
x
x
x
67.11%
30.95%
55.81%

6
82.18%
80.07%
37.93%
69.65%
49.38%
55.32%
89.74%
81.63%
77.15%
14.10%
61.90%
0.48%
0.00%
23.42%
51.28%
64.77%
5.04%
x
x
x
x
63.55%
24.16%
49.65%

7
80.42%
77.61%
36.88%
50.15%
42.00%
53.55%
75.10%
80.31%
74.55%
10.30%
20.94%
0.05%
0.00%
24.42%
50.90%
57.63%
0.00%
19.33%
x
x
x
54.71%
21.76%
41.90%

8
44.45%
71.53%
35.75%
51.99%
44.17%
50.20%
80.98%
76.45%
69.19%
29.32%
40.46%
0.18%
0.00%
22.80%
51.03%
71.25%
0.00%
15.88%
3.26%
x
x
54.04%
20.55%
39.94%

9
0.55%
61.80%
35.53%
51.15%
44.62%
50.52%
69.98%
77.75%
57.82%
9.92%
16.76%
0.05%
0.00%
23.56%
46.89%
62.95%
0.00%
7.37%
1.49%
2.27%
x
43.31%
16.06%
31.05%

10
0.00%
50.48%
32.40%
38.14%
40.64%
51.90%
62.86%
69.76%
56.53%
17.47%
6.83%
0.02%
0.00%
23.25%
53.95%
66.92%
0.00%
1.73%
1.67%
0.04%
2.60%
38.82%
15.02%
27.49%

I was using 2 V100 GPUs.
Do you also use the same batch size as me?
I know that the mixed precision could something give different results depending on the GPU/CUDA. Have you tried without it?

Do you have a similar issue?

Unfortunately I have left my lab as I have finished my lab, and I don't have anymore those intermediary results.

I have tried it on 2x RTX 2080Ti with cuda 10.2 (see environment.txt for the full environment). However, I got almost similar results as running with 2x RTX 3090. I think the cuda version and pytorch version does not affect the results that much. It still exists some gap (4.5 percent point) in the old class performance. Looking forward to your advice. Thanks.

step
background
aeroplane
bicycle
bird
boat
bottle
bus
car
cat
chair
cow
diningtable
dog
horse
motorbike
person
pottedplant
sheep
sofa
train
tvmonitor
0-10
11-20
all

0
95.20%
89.52%
40.69%
89.28%
71.95%
84.31%
95.54%
88.19%
92.76%
46.13%
91.65%
x
x
x
x
x
x
x
x
x
x
80.48%

80.48%

1
92.01%
85.67%
36.34%
84.94%
69.18%
78.48%
94.33%
82.29%
87.31%
48.07%
90.68%
37.13%
x
x
x
x
x
x
x
x
x
77.21%
37.13%
73.87%

2
88.68%
81.91%
35.53%
83.66%
63.85%
66.64%
93.23%
79.39%
81.74%
30.81%
85.40%
28.75%
61.16%
x
x
x
x
x
x
x
x
71.90%
44.96%
67.75%

3
83.42%
81.38%
32.47%
77.93%
55.47%
54.16%
91.72%
79.14%
73.56%
15.42%
67.04%
20.92%
13.42%
23.46%
x
x
x
x
x
x
x
64.70%
19.26%
54.96%

4
77.20%
73.92%
35.52%
72.78%
46.47%
52.34%
89.25%
77.28%
75.36%
10.86%
71.35%
17.17%
15.28%
21.92%
32.24%
x
x
x
x
x
x
62.03%
21.65%
51.26%

5
82.93%
85.78%
38.60%
78.68%
54.56%
58.62%
89.83%
81.99%
81.21%
11.34%
72.56%
0.38%
0.00%
29.06%
56.57%
69.71%
x
x
x
x
x
66.92%
31.14%
55.74%

6
80.73%
79.88%
37.65%
67.48%
50.95%
55.54%
90.07%
81.10%
78.17%
11.73%
65.35%
0.70%
0.00%
25.87%
47.94%
64.26%
5.08%
x
x
x
x
63.51%
23.98%
49.56%

7
76.70%
78.55%
37.95%
60.99%
40.11%
54.74%
71.47%
79.51%
73.48%
8.12%
36.72%
0.06%
0.00%
26.92%
41.19%
56.38%
0.00%
20.49%
x
x
x
56.21%
20.72%
42.41%

8
27.93%
71.21%
35.10%
49.39%
41.93%
52.80%
76.69%
76.92%
71.46%
23.04%
38.82%
0.08%
0.00%
25.62%
47.53%
70.07%
0.00%
0.00%
2.51%
x
x
51.39%
18.22%
37.43%

9
0.00%
65.28%
34.59%
52.03%
41.87%
54.80%
68.75%
76.32%
62.90%
6.32%
30.98%
0.01%
0.00%
25.35%
51.62%
59.17%
0.00%
0.00%
1.46%
6.82%
x
44.89%
16.05%
31.91%

10
0.00%
59.74%
23.64%
41.97%
37.19%
56.25%
58.29%
74.52%
59.88%
11.13%
11.89%
0.02%
0.00%
24.02%
55.30%
64.20%
0.00%
0.00%
1.70%
0.00%
3.38%
39.50%
14.86%
27.77%

@arthurdouillard May I know do you use different hyperparameter settings for different tasks, e.g., 10-1, 15-5, 15-1, etc.? Because I can reproduce the results for 15-1.


MUMUP-2508 Screen reader does not read account menu

As a screen reader user, I would like my screen reader to let me know the account dropdown menu exists, so that I can exercise that menu.
add 'role' and 'aria-label'

Coverage remained the same at 34.738% when pulling fec119f22b3a18f2b4c67085f840e51b451d03c8 on jiayinjx:account_dropdownMenu into dc5af3b90dcd85961ef06d7f0ba6e13019d78771 on UW-Madison-DoIT:master.

:+1:


Ensure same order of CAs for adminkubeconfig subresource and ca-cluster secret

How to categorize this PR?

/area security
/kind enhancement
What this PR does / why we need it:
Today, when there are multiple CAs then the order differs between the certificate-authority-data in the <shoot-name>.kubeconfig or ca.crt in the <shoot-name>.ca-cluster secret compared to the certificate-authority-data in the kubeconfig issued via the shoots/adminkubeconfig subresource.
While this shouldn't matter theoretically, practically it leads to issues when gardenctl-v2 is used. gardenctl-v2 uses gardenlogin which fails when trying to communicate to the shoot cluster in case there are multiple CAs (i.e., after phase 1 of the CA rotation):
$ ks get po
Error: failed to get ExecCredential: could not find matching auth info from shoot kubeconfig for given cluster: no matching cluster found for server https://api.my-cluster.example.com and certificate-authority-data
Usage:
  gardenlogin get-client-certificate [flags]

...

Error: failed to get ExecCredential: could not find matching auth info from shoot kubeconfig for given cluster: no matching cluster found for server https://api.my-cluster.example.com and certificate-authority-data
Unable to connect to the server: getting credentials: exec: executable kubectl failed with exit code 1

gardenlogin internally has an extra check to ensure the CA data of the kubeconfig issued via the shoots/adminkubeconfig subresource matches the CA data of the kubeconfig used for gardenctl-v2: https://github.com/gardener/gardenlogin/blob/c6814bddc4c307db6db930122563131605d3c04a/cmd/get_client_certificate.go#L411-L416
Since the order is different in the <shoot-name>.ca-cluster secret compared to the certificate-authority-data in the kubeconfig returned via the shoots/adminkubeconfig, above check fails.
There is https://github.com/gardener/gardenlogin/issues/28 to improve this check (by not depending on the order of the CAs in the bundle). However, to ensure backwards-compatibility Gardener can also always generate the same order (which is nicer and "more consistent" anyways).
Which issue(s) this PR fixes:
Part of #3292
Special notes for your reviewer:
/cc @petersutter
Release note:

NONE

/test pull-gardener-integration

/assign @plkokanov @ialidzhikov @ary1992
/cc @plkokanov @ialidzhikov @ary1992

/test pull-gardener-integration


App is always deployed at the root?

It seems like the runner always deploys a given application at the root. For instance, if my application is called hello.war, I would expect it be available under heroku for example as https://hello.herokuapp.com/hello. Instead it becomes available under https://hello.herokuapp.com
How can one ensure the context path is preserved?

Never mind. I see the -path  option.


[Node.js] Adding record & playback functionality of sensor data

@halton PTAL, thanks.

@halton PTAL again, thanks


update opentelemetry

update opentelemetry
bump version 0.6.0
update changelog

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.716%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1667

Relevant Lines:
2040

💛  - Coveralls


RN 0.76 Regression - RCTWindowFrameDidChangeNotification is not called when dark mode is toggled

Description
I’ve noticed another discrepancy between RN 0.75.x and RN 0.76. With the new bare starter, my iOS native notification listener is no longer called when the user changes the color scheme. Previously, I was using the RCTWindowFrameDidChangeNotification notification to receive information about screen and native color scheme changes.
RN 0.75 repo - https://github.com/jpudysz/rn-0.75-jsi-reproducer/tree/feature/repro2
Steps to reproduce

Open both the 0.75 and 0.76 repos
Navigate to Pods/Development Pods/Unistyles/ios/NativePlatformListener+ios.swift
Add a breakpoint at line 29

For RN 0.75 - the event is reported
For RN 0.76 - the breakpoint is not hit
React Native Version
0.76.0
Affected Platforms
Runtime - iOS
Output of npx react-native info
System:
  OS: macOS 15.0
  CPU: (12) arm64 Apple M3 Pro
  Memory: 728.19 MB / 36.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.15.0
    path: /usr/local/bin/node
  Yarn:
    version: 1.22.22
    path: /opt/homebrew/bin/yarn
  npm:
    version: 10.7.0
    path: /usr/local/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 24.0
      - iOS 18.0
      - macOS 15.0
      - tvOS 18.0
      - visionOS 2.0
      - watchOS 11.0
  Android SDK:
    API Levels:
      - "23"
      - "28"
      - "29"
      - "30"
      - "33"
      - "34"
      - "35"
    Build Tools:
      - 30.0.3
      - 34.0.0
      - 35.0.0
    System Images:
      - android-23 | ARM 64 v8a
      - android-28 | ARM 64 v8a
      - android-29 | ARM 64 v8a
      - android-30 | ARM 64 v8a
      - android-30 | Google APIs ARM 64 v8a
      - android-34 | Android TV ARM 64 v8a
      - android-34 | ARM 64 v8a
      - android-35 | Google APIs ARM 64 v8a
    Android NDK: Not Found
IDEs:
  Android Studio: 2024.2 AI-242.23339.11.2421.12483815
  Xcode:
    version: 16.0/16A242d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.11
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli":
    installed: 15.0.0-alpha.2
    wanted: 15.0.0-alpha.2
  react:
    installed: 18.3.1
    wanted: 18.3.1
  react-native:
    installed: 0.76.0
    wanted: 0.76.0
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: true
iOS:
  hermesEnabled: true
  newArchEnabled: true

Stacktrace or Logs
No crash

Reproducer
https://github.com/jpudysz/rn-0.76-jsi-reproducer/tree/feature/repro2
Screenshots and Videos
RN 0.75:
Unistyles theme event was applied.
https://github.com/user-attachments/assets/9b70c54c-a971-4b63-a68f-113809583cc5
RN 0.76:
Unistyles theme is not applied as event was not called.
https://github.com/user-attachments/assets/2f093ab3-b94e-4a4f-9779-3964aee8df02

One more note: both repros are running New Arch, so it's not about it

Issue Fixed in #47236.


Move product versions and build numbers to _config.yml

This PR proposes to migrate the management of the product versions to the top-level _config.yml to ease the maintenance, updates and prevent out-of-sync issues.
The first commit leaves the artifact file sizes hardcoded in the product downloads pages for now. Happy to hear thoughts on how to deal with those. They could also be migrated to the top-level configuration file as well if we want to manage everything  that way.

I'm not sure re: download sizes - that's quite a lot of extra parameters to add to the config file but on the other hand, they are easier to find there than scrolling through the html on the download page. So if we are keeping them, it might be worth migrating them.
If we move to manually updating the link to the release announcement being the only thing we need to update on the download page, that's at least near the top and easy to find.

Pushed the missing Windows change.
Re  artifacts sizes, I propose to postpone the migration to later. Part of the question is related to the maintenance of these values. Ideally it should be possible to either reuse/extend ome-release and/or write a local Python script generating a list of human-readable sizes using e.g. urllib2 as in:
>>> r=urllib2.urlopen("http://downloads.openmicroscopy.org/bio-formats/5.2.4/artifacts/bioformats_package.jar")
>>> float(r.headers['content-length'])/1000/1000
30.163378
>>> float(r.headers['content-length'])/1024/1024
28.76603889465332

As this has been approved and is conflicting with #69, merging so that we can move forward. As mentioned above the maintenance of download sizes will need to happen separately.


Avoid showing autoreload warnings

Description of your problem
Ran code and got this error in the console, not in the Internal Console or in the terminal, but directly after my printed output.
What steps will reproduce the problem?
Not sure.  I ran this code from the editor.  It probably sat for a few hours and then I ran it again.  When I ran it the second time, it produced the output and then printed the error in the IPython console window.
with open(file, newline='') as csvfile:
    reader = csv.DictReader(csvfile)
    open_items = [row['id'] for row in reader]
    for i in range(10):
        print(random.choice(open_items))

What is the expected output? What do you see instead?
I just expected to see my output and not a traceback.  If I click run again, I don't get the error.  I'll try to let it sit for awhile to see if that's the difference.
Please provide any additional information below
[autoreload of spyder.widgets.variableexplorer.utils failed: Traceback (most recent call last):
File "/home/cheryl/anaconda3/envs/spyder-dev/lib/python3.6/site-packages/IPython/extensions/autoreload.py", line 246, in check
superreload(m, reload, self.old_objects)
File "/home/cheryl/anaconda3/envs/spyder-dev/lib/python3.6/site-packages/IPython/extensions/autoreload.py", line 369, in superreload
module = reload(module)
File "/home/cheryl/anaconda3/envs/spyder-dev/lib/python3.6/imp.py", line 315, in reload
return importlib.reload(module)
File "/home/cheryl/anaconda3/envs/spyder-dev/lib/python3.6/importlib/init.py", line 166, in reload
_bootstrap._exec(spec, module)
File "", line 618, in _exec
File "", line 678, in exec_module
File "", line 219, in _call_with_frames_removed
File "/home/cheryl/spyder/spyder/widgets/variableexplorer/utils.py", line 33, in
required_version=PANDAS_REQVER, optional=True)
File "/home/cheryl/spyder/spyder/dependencies.py", line 73, in add
% modname)
ValueError: Dependency has already been registered: pandas
]
Versions and main components

Spyder Version: 4.0.0.dev0 (up to date)
Python Version:  3.6.4
Qt Version:  5.6.2
PyQt Version:  5.6 on Linux
Operating system:  Ubuntu 16.04

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
IPython >=4.0    :  6.1.0 (OK)
cython >=0.21    :  0.26.1 (OK)
jedi >=0.9.0     :  0.11.0 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
numpy >=1.7      :  1.13.3 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
psutil >=0.3     :  5.4.1 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pyflakes >=0.6.0 :  1.6.0 (OK)
pygments >=2.0   :  2.2.0 (OK)
pylint >=0.25    :  1.7.4 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
rope >=0.9.4     :  0.10.7 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
sympy >=0.7.3    :  1.1.1 (OK)

I experienced this as well with a clean/untouched console that had been left open from the previous day, after importing a few data columns from the clipboard (testing #7244 ) as pandas dataframes, and then running (as my first actual command in the terminal) just test = [1, 2, 3, 4, 5] without ever having directly loaded or called pandas. This seems very similar to the original report, which involved reading in a csv and a substantial delay between the console being opened and the followup code.
Unfortunately, I opened a new console in the same spyder instance, repeated the steps (aside from the delay), but did not get the same error. Therefore, either the delay was in fact significant, the error occurring is non-deterministic, or there was something seemingly very small that I did differently that didn't trigger it.

It probably happens when you edit and import a file in the console which contains import pandas. It's a very minimal thing but we can't solve it in our side but in IPython.

Hmm, well for me it happened in a clean console I'd opened the day before and never touched. It happened the on the very first command I entered, just creating a vanilla python list. The only thing I'd done was use the import from clipboard dialog to create a dataframe (which, however, would of course use pandas). But it didn't happen again when I tried the exact same thing in another fresh console, just not waiting a day to use it. So strange.


Unicode control caracters in parameters - java.lang.IllegalArgumentException: Illegal number of path parameters. Expected 1, was 2.

From<EMAIL_ADDRESS>on May 22, 2015 19:07:02
What steps will reproduce the problem? 1. Parameters in get() method that have unicode control caracters fail with an IllegalArgumentException
Example:
String request = "param1={param1Value}&param2={param2Value}";
String param1Value =  "Hello";
String param2Value =  "Hello\u0085";
System.out.println(param1Value);
RestAssured.get(request, param1Value, param2Value); What is the expected output? What do you see instead? Success of method call get() What version of the product are you using? On what operating system? 2.4.1 Please provide any additional information below. The error is in com.jayway.restassured.internal.RequestSpecificationImpl:
def hasAnyTemplateLeft = ~/.{\w+}./
This regex only takes words and not others caracters.
if (queryParams.matches(pathTemplate)) {
def hasAnyTemplateLeft = ~/.{\w+}./
def replacePattern = ~/{\w+}/
def definedParams
..... else {
throw new IllegalArgumentException("Illegal number of path parameters. Expected $numberOfUsedPathParameters, was $expectedNumberOfUsedPathParameters.")
}
}
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=404

From<EMAIL_ADDRESS>on May 26, 2015 02:15:19
Status: Accepted

From<EMAIL_ADDRESS>on May 26, 2015 02:15:49
Thanks for reporting. If you would like to help out I would very much appreciate a pull request.

From<EMAIL_ADDRESS>on May 26, 2015 04:41:50
I used another library as the performance of rest-assured was 50x to 100x slower than regular httpclient and json-simple.

From<EMAIL_ADDRESS>on May 26, 2015 10:47:36
Because of this issue?!

From<EMAIL_ADDRESS>on May 26, 2015 18:34:53
Because of this issue I found that rest-assured is super slow for the kind of work I do. I execute concurrent tests that run for hours (> 200K requests), Rest-assured is very inefficient because of groovy code I suppose.

From<EMAIL_ADDRESS>on May 28, 2015 11:35:55
I suppose that Groovy could be a reason but I'm sure that I could do things internally to optimize it more as well. Would be interesting to know where RA spends the most time? It would be great if you could run a profiler on it.

From<EMAIL_ADDRESS>on June 06, 2015 10:03:50
I don't think your issue has anything to do with unicode characters. I think it has to do with the fact that your request path is not a valid URI? If I change your example to:
String request = "/test?param1={param1Value}&param2={param2Value}";
String param1Value =  "Hello";
String param2Value =  "Hello\u0085";
System.out.println(param1Value);
RestAssured.get(request, param1Value, param2Value);
then it works!

From<EMAIL_ADDRESS>on June 06, 2015 10:05:00
The code (if I remember it correctly) looks for an initial "?" when searching for parameters and your request path doesn't include a "?". There should probably be a better exception message though.

From<EMAIL_ADDRESS>on June 06, 2015 10:14:19
I've fixed a better expection message now
Status: Invalid


b4557d64c8938e60298e5376157365c0c98a6b72

Describe the bug

Affects

[ ] Designer
[ ] Blocks editor
[ ] Companion
[ ] Compiled apps
[ ] Buildserver
[ ] Debugging
[ ] Other... (please describe)

Expected behavior

Steps to reproduce

Marking this as invalid and closing without more information.


"There is no interaction model for en-US. Select a different locale."

Describe the bug
After creating a skill using the web browser, I've found the possibility of using vs code to edit the skill. Then, I've installed the vscode extension and did the necessary configuration so that I could start using it to develop a sample skill. But, the option Test Skill -> Open simulator was throwing me the error "There is no interaction model for en-US. Select a different locale.". That's ok, because when I've created the skill I selected Brazilian Portuguese as the default language. So, as sugested by the error message, I tried to change the locale in the locale combobox (show below), but I couldn't, because it was stuck with a "Loading...":

To Reproduce
High level steps  to reproduce the behavior (of what I remember I've done so far, I have not tested it):

Create a custom skill using the web browser selecting as primary language Portuguese BR.
Add 3 more languages using web browser: English (US), Spanish (ES) and Spanish (US), but without their interaction model.
Install and configure VScode ASK extension.
Sign in using the extension.
Download the previously created skill using the option "Download and edit skill"
Install ask-sdk-local-debug for nodejs as described in the README
Configure the lauch.json as described in the docs.
Use the option "Test Skill -> Open Simulator" to see the error.

Expected behavior
The expected behaviour would be allowing me to select the Portuguese Language after the error is shown, even though the English model doesn't exist yet.

Desktop (please complete the following information):

OS: Windows_NT x64 10.0.19044
Visual Studio Code Version:  1.77.1
Alexa Skills Toolkit Version: 2.14.1
Git Version: git version 2.35.1.windows.2

Additional context
As a workaround to solve the problem, I've added a en-US locale to the skill using the web browser, saved and deployed. I had to close the vscode an open it again. Then, in vscode, I downloaded the en-US locale json configuration using the option "Interaction model"-> Download. After that, the error stopped showing and I could select any of the 4 languages shown in the second image above.

@cancastilho Thanks for the detailed report Carlos! I'll take a look into what's going on.

To clarify, after you worked around the issue and were able to select Portuguese (BR), were you able to test your skill?

Yes, alter the workaround I could use the simulator in  vscode to test the skill.

To clarify, after you worked around the issue and were able to select Portuguese (BR), were you able to test your skill?

re-opening.   we pushed the changes but we are holding back the actual release until Monday. I'll comment again here and close once the release is out.

released 2.14.2 to VScode marketplace.  Let us know if the problem persists after updating.
thank you for reporting the bug!


Fix missing includes in tapcell.{h,cpp}

Fixed 20 misc-include-cleaner issues. Only 12199 to go :)

IWYU says "for every symbol (type, function, variable, or macro) that you use in foo.cc (or foo.cpp), either foo.cc or foo.h should include a .h file that exports the declaration of that symbol.".  So it wouldn't be necessary to add vector to both the h and cpp file but that what happens in this PR.  Is the clang-tidy check stricter than IWYU ?

Correct, misc-include-cleaner makes sure that each file can be considered stand-alone it itself, importing all the things it needs. So if the header has something that needs std::vector it needs to include <vector>, but then also the corresponding compilation unit also uses std::vector, then it also needs to include <vector>. Μaybe the older IWYU rules were more lenient, but this stand-alone-ness makes it very easy not only for tools, but also humans that look at a file.
There are ways to allow for one level of transitive export. Sometimes there are headers that include others headers whose symbols we want to make look like as if they had been directly defined in that header, in that case we can use the IWYU pragma: export to tell it that this is the intent, and misc-include-cleaner will not warn about that header directly included anymore.
Should of course be used rarely and not for things such as <vector>, but sometimes there are 'collection' headers that combine some key headers where this might be useful.
#include "include/header/that/whose/symbols/we/want/to/export.h"  // IWYU pragma: export

The direct documentation of the check is a bit limited
https://clang.llvm.org/extra/clang-tidy/checks/misc/include-cleaner.html
... but the linked document in there
https://clangd.llvm.org/design/include-cleaner

The quote is directly from https://github.com/include-what-you-use/include-what-you-use so I imagine it to be the current rule.  Given the rule that the .h should always be included by the .cpp it is a bit redundant but not worth a lot of effort to resolve.

IIRC, IWYU is essentially a 15y old project that emerged from avoiding forward declares in the Google codebase back then to actually including most of the things, so they might've settled for a simpler goal.
If we could configure misc-include-cleaner to not worry about it, that would also be good ('complain about things that are not included unless they are included in a file whose basename looks like mine' :) ).
But even if not, the additional readability of the code is also a plus.
Using the clangd language server in emacs/vi/vscode is very helpful to fix these things while developing.
I think it is doable to get in a better state; I have gotten other projects from tens of thousand misc-include-cleaner warnings to zero fairly quickly with an overall improvement in readability.  I'll keep chipping away on it for OR.
Different topic: One questiion w.r.t. to rules to consider: There is a rule that seems to be somewhat noisy and probably not too interesting (at least at this point): performance-enum-size ... it suggests if an enum does not have too many values to change it ti uint8_t for instance instead of the default uint32_t. I think for now, we should probably disable it as it is somewhat noisy, wdyt ?

Using the clangd language server in emacs/vi/vscode is very helpful to fix these things while developing.

How does it help?

Using the clangd language server in emacs/vi/vscode is very helpful to fix these things while developing.

How does it help?

If you have the clangd language server enabled and linked the compile_commands.json in the toplevel directory for the language server to find, it will show wiggly lines on symbols that are not directly mentioned in included files and allows wit one click to add the the corresponding header at the top of the file, already properly sorted according to .clang-format.
Also it will highlight headers that are not actually needed in a file that can be removed.


Generated folder is incomplete

Hiya!
I've been trying to create a new project with the new version but it does not include Android and IOS folder etc Basically it does not complete. The new command ends with the output below
✔ Linking external libs
⠋ Linking external libs./PickUp/ios/PickUp/Info.plist: file does not exist or is not readable or is not a regular file (Error Domain=NSCocoaErrorDomain Code=260 "The file “Info.plist” couldn’t be opened because there is no such file." UserInfo=0x7f9df14152a0 {NSFilePath=./PickUp/ios/PickUp/Info.plist, NSUnderlyingError=0x7f9df1413f80 "The operation couldn’t be completed. No such file or directory"})
✔ Updating PList file
⠋ Cleaning up after messy guestscd: no such file or directory: PickUp/android
/bin/sh: ./gradlew: No such file or directory
✔ Cleaning up after messy guests

So I think it's broken

[x] Issue does not already exist
Identify exactly what aspect your issue is about

[ ] ignite base application
[ ] ignite the cli
[x] ignite generated code

Also, for the quickest assistance i run ignite doctor and the everything is below (including the lines with 3 backticks).
Computer
  Platform: darwin

Ignite
  Version: 1.8.0
  Path: /Users/osi/.nvm/versions/node/v4.2.1/bin/ignite

Node
  Version: v4.2.1
  Path: /Users/osi/.nvm/versions/node/v4.2.1/bin/node

NPM
  Version: 3.8.6
  Path: /Users/osi/.nvm/versions/node/v4.2.1/bin/npm

Yeoman
  Version: 1.8.4

React Native CLI
  Version: 1.0.0

App
  React Native Version: ¯\_(ツ)_/¯

Hey!  What happens when you create a react native project without Ignite?  Does it make those 2 folders?

I think i traced the problem. An npm package in my local cache had a corrupted package.json file, so that kept breaking the install process. It would be nice if there was a nice error stating that npm could not install dependencies or something

that's wild!  If you give us a good way to detect that, we can add that to our CLI

This issue is related to npm/npm#6084. I'll look into it


[css] show border around editable input while editing

Codecov Report

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

@@           Coverage Diff           @@
##           master    #5724   +/-   ##
=======================================
  Coverage   63.38%   63.38%
=======================================
  Files         361      361
  Lines       22995    22995
  Branches     2559     2559
=======================================
  Hits        14575    14575
  Misses       8405     8405
  Partials       15       15

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 54ae215...c447a97. Read the comment docs.


what is cmd?

Hello, i read your auto-gfqg in github .i have understand the steps of that project. But i never use scala before. when i run the Scoresentences.scala , an error occurs , " error : not found : value cmd , import cmd.RunnerHelpers " , and i want to know what cmd is?

Build sbt , the problem solved.


Improve article [arXiv:2104.03438]

https://ar5iv.labs.arxiv.org/html/2104.03438
https://arxiv.org/pdf/2104.03438
experiment section
crashes on result tables and its desciption
As always, Thanks!

We may want to use Table 2 here as a test case in latexml, as it currently produces malformed flexbox markup, missing the cell size class. Namely, for two of the subtables:
<div class="ltx_flex_cell
                  ">

It should have chosen ltx_flex_size_1, ltx_flex_size_2 etc, rather than this long whitespace padding.

Attaching the TeX code here:

\subsection{Performance evaluation}

\begin{table*}[t]
\centering
\begin{tabular}{|C{1.555cm}|C{1.5cm}|C{1.5cm}|C{1.6cm}|C{1.6cm}|C{0.9cm}|C{0.9cm}|C{0.88cm}|}
\hline
\multirow{2}{*}{Approach} & Top-1 acc.  & Top-5 acc. & Top-1 acc. & Top-5 acc. & Top-1 & Top-5 & FLOPs  \\
& baseline & baseline & after prune & after prune & acc. $\downarrow$ & acc. $\downarrow$ & $\downarrow$ \\
\Xhline{1.8\arrayrulewidth}
MW & 76.13\% & 92.86\% & 71.24\% & 90.38\% & 4.89\% & 2.48\% & 41.8\%\\
SFP &  76.15\% &  92.87\% & 74.61\% & 92.06\% & 1.54\% & 0.81\% & 41.8\% \\
GM &  76.15\% &  92.87\% & 75.03\% & 92.40\% & 1.12\% & 0.47\% & 42.2\% \\
GAL&  76.15\% & 92.87\% & 71.95\% & 90.94\% & 4.20\% & 1.93\% & 43.0\% \\
TAS &  - & - & 76.20\% & 93.07\% & 1.26\% & 0.48\% & 43.5\% \\
HRank & 76.15\% & 92.87\% & 74.98\% & 92.33\% & 1.17\% & 0.54\% & 43.8\% \\
SNN  &  - &  - & 74.90\% & - & 1.10\% & - & 43.9\% \\
\cellcolor{Gray}SRR-GR & \cellcolor{Gray}76.13\% & \cellcolor{Gray}92.86\% &  \cellcolor{Gray}75.76\% &  \cellcolor{Gray}92.67\% & \cellcolor{Gray}\bf 0.37\% & \cellcolor{Gray}\bf 0.19\% & \cellcolor{Gray}\bf 44.1\%\\
TRP & - & - & 74.06\% & 92.07\% & - & - & 44.4\% \\
AP  &  76.15\% &  92.87\% & 74.76\% & 92.15\% & 1.39\% & 0.72\% & 51.2\% \\
GM &  76.15\% &  92.87\% & 74.13\% & 91.94\% & 2.02\% & 0.93\% & 53.5\% \\
ABC  & 76.01\% & 92.96\% & 73.86\% & 91.69\% & 2.15\% & 1.27\% & 54.0\% \\
SCP & 75.89\% & 92.98\% & 74.20\% & 92.00\% & 1.69\% & 0.98\% & 54.3\% \\
CUP  &  - &  - & - & - & 1.47\% & 0.88\% & 54.5\% \\
GAL &  76.15\% & 92.87\% & 71.80\% & 90.82\% & 4.35\% & 2.05\% & 55.0\% \\
\cellcolor{Gray}SRR-GR & \cellcolor{Gray}76.13\% & \cellcolor{Gray}92.86\% &  \cellcolor{Gray}75.11\% &  \cellcolor{Gray}92.35\% & \cellcolor{Gray}\bf 1.02\% & \cellcolor{Gray}\bf 0.51\% & \cellcolor{Gray}\bf 55.1\%\\
\hline
\end{tabular}
\caption{ResNet50 results on ImageNet. Results of MW, APoZ, and Taylor are based on our own implementation.}
\label{tab:imagenet_single}
\vskip -0.1in
\end{table*}

\noindent{\bf CIFAR-10.} Results of pruning ResNet20 and ResNet56 on CIFAR-10 are presented in Table~\ref{tab:cifar_single}. Our approach prunes a large percent of FLOPs from both architectures without performance degradation, which outperforms the previous state-of-the-art with an obvious margin. For ResNet20, we prune $45.8\%$ FLOPs and the test accuracy is increased by $0.21\%$. Our pruned ResNet56 model reduces $53.8\%$ FLOPs and achieves a test accuracy of $93.75\%$, which outperforms the baseline by $0.37\%$.

\noindent{\bf ImageNet.} Results of ResNet50 on ImageNet are shown in Table~\ref{tab:imagenet_single}. Our pruned model with $44.1\%$ FLOPs reduction only loses $0.37\%$ top-1 accuracy and $0.19\%$ top-5 accuracy. When pruning comparable FLOPs, the top-1 accuracy of the previous state-of-the-art approaches usually drop by more than $1\%$. As the pruning ratio increases to $55.1\%$, the proposed approach can still achieve a promising test accuracy ($1.02\%$ and $0.51\%$ drop for top-1 and top-5 accuracy), which is the best performance compared with recent works. These results verify the effectiveness of our approach on the single-shot pruning scheme. It is worth mentioning that the MW approach can be considered as a baseline of our approach because we add a redundancy identification stage before using MW for filter pruning. It is observed that pruning filters uniformly with MW results in unsatisfactory results ($71.24\%$ top-1 accuracy). After identifying the redundancy and pruning the corresponding number of filters in each layer, the performance is significantly improved by around $4\%$.

\begin{table}[htb]
\centering
\begin{tabular}{|c|c|c|c|}
\hline
{Num of} & {1-covering} & {Time with} & {Time with} \\
filters & number & oracle method & our method \\
\hline
\multirow{4}{*}{64} & 1 & 0.001 & 0.0015\\
\cline{2-4}
& 2 & 0.045 & 0.0013\\
\cline{2-4}
& 3 & 1.384 & 0.0020\\
\cline{2-4}
& 4 & 28.21 & 0.0026\\
\hline
\multirow{3}{*}{192} & 1 & 0.023 & 0.0014\\
\cline{2-4}
& 2 & 0.974 & 0.0027\\
\cline{2-4}
& 3 & 90.29 & 0.0028\\
\hline
\end{tabular}
\caption{Average computation time of the oracle and our proposed method for the 1-covering number calculation/estimate (in secs).}
\label{tab:time}
\vskip -0.1in
\end{table}

Looks mostly ordinary, as long as the respective table environments can be balanced without error by latexml.


how to add click event on hyperlink?

I want to add click event on a hyperlink and need to execute some code to show/hide records.
I have the number of records and want to render as pagination, for that using Show more and show less hyperlink but, didn't get click event for the hyperlink, I am passing onclick attribute but editor removes those attributes.
Please suggest how I can achieve this.

@cs-ajay-chakkar Currently, the editor does not provide a click event. To bypass, I think you can use the event.focus option of the officially provided editor instance creation options, or informally use the mousedown event of editor.eventManager, but I don't recommend it.
I don't understand the function you are trying to create, but can you explain it in more detail? I would be grateful if you could explain all the features of the Markdown Editor and WYSIWYG Editor.


use tox-lsr 2.14.1

tox-lsr moved to use ansible-core 2.14 which does not support python 3.8 - rather than try to revert that, I moved some of the ansible related checks to use python 3.9
for some reason the latest ansible-test image does not work with docker - so install the podman-docker package to "fool" the system into using podman instead of docker - the package is only available on ubuntu-latest - in order to do this, we first have to remove moby-cli (aka docker-client)

This isn't pretty, and I have a big plan to replace all of this stuff with

individual github actions for yamllint, ansible-lint, etc.
removal of py, pylint, etc. tests for non-python roles

But for now this should get CI working again. If you guys approve, I'll roll this out to all roles.


Randomize P-Rep list order

Randomize P-Rep list order. Currently top voted P-Reps are displayed first.

I'd like to give this one a try. I was thinking of saving a fisher-yates function in the utils directory and using that here

Ah, I hadn't seen the note about the FA pro token being necessary. Assuming there's no way around that, I won't be able to work on this, sorry.

We've already got lodash-es as a dependency and it looks like _.shuffle uses fisher-yates so I'd recommend just using their shuffle than adding a util for it. But yeah, go for it.
If you have any trouble with the FA pro dependency, branch off use-fontawesome-free because I think we'll need to get that merged in today.

Sounds good, will do.

We just merged the use-fontawesome-free branch, so feel free to rebase to master.


Error: Cannot find module 'core-js/modules/es6.array.sort'

Appear this error when using this plugin with Webpack4. I dont know how to debug plugins very well, so if you need something please tell me or show me how to start with 😄.
× ｢wdm｣: Error: Cannot find module 'core-js/modules/es6.array.sort'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:548:15)
    at Function.Module._load (internal/modules/cjs/loader.js:475:25)
    at Module.require (internal/modules/cjs/loader.js:598:17)
    at require (internal/modules/cjs/helpers.js:11:18)
    at Object.<anonymous> (C:\Projects\my-project\node_modules\favicons\dist\index.js:9:1)
    at Module._compile (internal/modules/cjs/loader.js:654:30)
    at Module._extensions..js (internal/modules/cjs/loader.js:665:10)
    at Object.require.extensions.(anonymous function) [as .js] (C:\Projects\my-project\node_modules\babel-register\lib\node.js:152:7)
    at Module.load (internal/modules/cjs/loader.js:566:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:506:12)

Could you try adding "@babel/polyfill": "^7.0.0-beta.42" as a dependency?

Error: Cannot find module 'core-js/modules/es6.array.sort'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:548:15)
    at Function.Module._load (internal/modules/cjs/loader.js:475:25)
    at Module.require (internal/modules/cjs/loader.js:598:17)
    at require (C:\Projects\react-boilerplate\node_modules\v8-compile-cache\v8-compile-cache.js:159:20)
    at Object.<anonymous> (C:\Projects\react-boilerplate\node_modules\favicons\dist\index.js:9:1)
    at Module._compile (C:\Projects\react-boilerplate\node_modules\v8-compile-cache\v8-compile-cache.js:178:30)
    at Module._extensions..js (internal/modules/cjs/loader.js:665:10)
    at Object.require.extensions.(anonymous function) [as .js] (C:\Projects\react-boilerplate\node_modules\babel-register\lib\node.js:152:7)
    at Module.load (internal/modules/cjs/loader.js:566:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:506:12)
    at Function.Module._load (internal/modules/cjs/loader.js:498:3)
    at Module.require (internal/modules/cjs/loader.js:598:17)
    at require (C:\Projects\react-boilerplate\node_modules\v8-compile-cache\v8-compile-cache.js:159:20)
    at Object.<anonymous> (C:\Projects\react-boilerplate\node_modules\webapp-webpack-plugin\src\loader.js:2:18)
    at Module._compile (C:\Projects\react-boilerplate\node_modules\v8-compile-cache\v8-compile-cache.js:178:30)
    at Module._extensions..js (internal/modules/cjs/loader.js:665:10)
    at Object.require.extensions.(anonymous function) [as .js] (C:\Projects\react-boilerplate\node_modules\babel-register\lib\node.js:152:7)
    at Module.load (internal/modules/cjs/loader.js:566:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:506:12)
    at Function.Module._load (internal/modules/cjs/loader.js:498:3)
    at Module.require (internal/modules/cjs/loader.js:598:17)
    at require (C:\Projects\react-boilerplate\node_modules\v8-compile-cache\v8-compile-cache.js:159:20)
    at loadLoader (C:\Projects\react-boilerplate\node_modules\loader-runner\lib\loadLoader.js:13:17)
    at iteratePitchingLoaders (C:\Projects\react-boilerplate\node_modules\loader-runner\lib\LoaderRunner.js:169:2)
    at iteratePitchingLoaders (C:\Projects\react-boilerplate\node_modules\loader-runner\lib\LoaderRunner.js:165:10)
    at C:\Projects\react-boilerplate\node_modules\loader-runner\lib\LoaderRunner.js:185:6
    at context.callback (C:\Projects\react-boilerplate\node_modules\loader-runner\lib\LoaderRunner.js:111:13)
    at C:\Projects\react-boilerplate\node_modules\cache-loader\dist\index.js:133:7
    at ReadFileContext.callback (C:\Projects\react-boilerplate\node_modules\cache-loader\dist\index.js:200:7)
    at FSReqWrap.readFileAfterOpen [as oncomplete] (fs.js:434:13)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `bash scripts/build.sh "app" "production"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\THernandez\AppData\Roaming\npm-cache\_logs\2018-04-12T20_03_50_958Z-debug.log```

Is this after adding @babel/polyfil as a dependency and running npm install?

its solves if i install core-js dependency 🤔

Could you check if v2.0.3 solves the issue?

No. It didn't solve the problem. Also viewing the changes you made to favicons repo I tried to install @babel/core, @babel/register also but didn't work either

I share you an invitacion to see the project...

I appreciate it, I'll have a look

After some tinkering I'm convinced the only reasonable solution is indeed to add core-js as a dependency. I can't really explain what's going on, but my only guess is that there is some weird interaction going on between babel@6 and babel@7.
Anyway, since adding core-js as a dependency does solve the problem, I'd say we can close this issue for now and reopen it in case more people report similar issues.
Please let me know what you think.

I think its ok... i thought something similar, but i was not sure at all.
Do you think it's a good idea to put an issue on @babel repo directly ?

I don't think there's much we can do about it while projects that depend on different major versions of babel coexist, I guess that's the flip side of the paced js/node community, where there is such a thing as packages that evolve almost too fast.

yarn add babel-preset-env --dev helped me solving this (using nuxt-edge)

its solves if i install core-js dependency!!! yay!!!

I'm able to get this to work as long as I install<EMAIL_ADDRESS>The newer 3.0 version does not solve the problem for me.

@drewda's solution works.  Problem seems to be that core-js renamed es6 directory to es gatsby broke as a result.

yarn add core-js
如果还不行
删除 node_modules,然后使用 yarn 安装，不要使用 cnpm 安装；

in similar core-js/modules/xxx/yyy error, my solution is to delete node_modules folder and reinstall.

@ian same goes for Gridsome.


Fix installation scenario when the context is no instance of UpdateContext

A renaming would be useful

@JoshuaBehrens A renaming would be a breaking change. I agree that this would be more accurate, but I would like to avoid a major update here. This is supposed to be a small fix to ensure, everything works as it should. Btw: Are you still on this?

I am looking forward to merge this library with https://github.com/shyim/FroshSupportLibrary


Add metrocluster FC support

Description
Hello, it will be great to add somme option like auto vserver parsing on metrocluster to remove information with vserver ending by -mc
Additional Context
in metrocluster FMCC  we have vserver with name ending by -mc that are normaly stopped
with an new option we can try to autoremove vserver with -mc information.
perhaps we can use the $Options.Exclude.Vserver and add -mc vserver automatically
Before submitting

[X] I have read the documentation, and referred to the known issues before submitting this change request.
[X] I have checked for previously opened & closed issues before submitting this change request.

I can help, but i just want to have some guideline (how to add a new option ?) to avoid breaking all the project

Hello,
You need to edit the json template file:
https://github.com/AsBuiltReport/AsBuiltReport.NetApp.ONTAP/blob/dev/AsBuiltReport.NetApp.ONTAP.json
Something like this:

Also is needed to modify the filter logic in the report source code.

Any update on this?

Hello, yes i will made a new pull request on next days
thanks

A new version has been released with the changes you introduced and other new features such as NVME support. If you are able to test it and let me know the results I would greatly appreciate it.

hello,
i have test new version successfully.
just in release note, it's a vserver filter with -mc not volume
we have some little issue with efficiency. I will try to fix it and made a new pull request
thanks

Don't forget to sync the changes to your branch repo before making any changes back to upstream.


Use TMPDIR env vars

On some systems, like on termux, for example, /tmp is not available, using $TMPDIR or $TEMPDIR and defaulting to /tmp only if neither is set would be better.

Good suggestion. I pushed changes using TMPDIR or TEMPDIR or /tmp if neither is set. This needs to go through testing before a new release is created but if you have Lazyman installed then lazyman -U should pick these changes up. If you do not have Lazyman installed then installing from a cloned repository will pickup these changes:
git clone https://github.com/doctorfree/nvim-lazyman $HOME/.config/nvim-Lazyman
$HOME/.config/nvim-Lazyman/lazyman.sh

Awesome, thanks. I'll give it a try.

Gave it a try and it seems to be working as expected.  Thanks.


Is there a way to delete specific calls?

We are wondering if there is a method to delete specific calls that have been processed by the PCA solution and remove them from showing in the dashboard?

Hi hyde-mattholgate,
Are you using the pca-dashboards CloudFormation template? If so, there isn't a great way to delete records out of the parquet datalake. You would have to craft an athena query that locates the appropriate parquet s3 path for those records, and modify the parquet to delete them.   Here's an example with CSV files in S3: https://repost.aws/knowledge-center/find-s3-source-file-athena-table-row


Added support for SSL connections

@jacek-lewandowski  can you take a look at this?

please rebase it

All the relevant properties are exposed by default: http://docs.oracle.com/cd/B19306_01/java.102/b14355/sslthin.htm#BABJIHFJ.  The driver uses the default options when you call the default SSLOptions constructor, which results in the JVM respecting these system JSSE properties.

Also, do you mean for me to rebase against master?

@rstrickland yes - rebase against the master
For the SSL settings - so if you set them on the client host you will have to pass them also as spark.executor.extraJavaOptions. Don't you think that adding them to Spark configuration and distributing them this way is better?
Also - what about Thrift based connections?

I went down this road initially, but then it seemed like wheel reinvention.  But I can see it both ways.  I'm happy to add spark options if you'd like to go that route.

@rstrickland IMHO that would be better. Especially because we've recently moved all the connection parameters into Spark configuration. This has also one more benefit - it allows us to introduce namespace in Spark configuration to setup connections to different clusters.

@rstrickland are you going to work on this?

@jacek-lewandowski it's on my radar, but I've been distracted by other stuff lately.

@rstrickland could you create a ticket in Jira for this ticket and add a link to this branch?

I'm pretty sure this is done (since we have a lot of ssl options :P) but just incase i'm missing something to we still need this?

Closing this - we already have SSL support and client auth is covered by SPARKC-359


Where can i get the jQueryKeyframes?

See other plugins that allow for spritesheets & more complex movement paths:
https://github.com/jQueryKeyframes
But I can't find the jQueryKeyframes :)

https://github.com/Keyframes


compress images

the react.png found at
Challenge: Learn About Self-Closing JSX Tags
has a size of 49kb which can be compressed to 11.2kb
also you can bring it down from 880x880 to 500x500 pixels

Yes I think we will try to do something about this image before the final version.


General: Fix translation in exercise overview

Checklist
General

[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I translated all newly inserted strings into English and German.

Motivation and Context

Fix #6766
Description

The translation key was missing the plural s, which lead to the missing key.
Steps for Testing

Prerequisites:

1 Instructor

Log in to Artemis
Go to the exercise overview and check that the translation is correctly shown in German and Englisch

Review Progress

Code Review

[x] Code Review 1
[ ] Code Review 2

Manual Tests

[x] Test 1
[x] Test 2

Tested on TS6, works as expected


Generate changelog on deploy

We're using https://github.com/skywinder/github-changelog-generator.  When deploying master, this should just be regenerated.

Done


Updated oversampling, minor edits to reflect changes, minor edits to examples

Added set and get functions for oversampling, using 5 different word options to reflect a different amount of time. I updated keywords.txt and readme to reflect these changes. I also made minor edits to the examples, and added a short explanation of the available oversampling settings. I also tested the library with my sensor, and it works very well.

Thanks for the PR, I will try to look into it tomorrow in more depth.
(busy upgrading / cleaning all my libs for open commits so it might take some time"

quick first look seems good!

title of issue => exmaples  ?

@LyricPants66133
Typical math for oversampling is 4x as much samples gives 1 bit accuracy extra.
More info - https://en.wikipedia.org/wiki/Oversampling#Resolution

to be continued ....

@LyricPants66133
released 0.3.2
I fixed backwards compatibility, while keeping the functionality you proposed.
renamed the internal variabele name to sampleRate, added some more examples
Can you please verify it still works for you? Thanks.


get rid of confd in cnt

since we're only using confd for templating and extend templating capabilities for our needs.

Include a folder to have some go template ready.
To be able to call them in the other template.
cf Godoc
`{{define "T1"}}ONE{{end}}
{{define "T2"}}TWO{{end}}
{{define "T3"}}{{template "T1"}} {{template "T2"}}{{end}}
{{template "T3"}}`

Add a function to define a default value.
to avoid having a line like
{{if lol}} {{lol}} {{else}}pom {{end}}
could look like
{{default lol pom}}
That would make a lot of templates a lot cleaner.
Not sure of the usefulness of the next one but just in case.
currently the {{range}} output an empty line when the array is empty or the value ain't there.
Could we have thw possibility to add a default member when that happens?

support include of template in other templates

done


Jupiter 5.9.0 + Vintage 5.9.0 + Platform 1.9.0

To fix https://github.com/sbt/sbt-jupiter-interface/issues/52, upgrade to support Junit 5.9.0

CI passed
https://github.com/LuciferYang/sbt-jupiter-interface/runs/7593353074?check_suite_focus=true

thanks @maichler


grpcio-tools proto compilation fails

Expected behavior:
grpcio-tools install automatically includes all dependencies
Observed behavior:
grpcio-tools complains about missing "protoc-gen-grpc" when trying to compile protos
Steps to reproduce
Clone the googleapis repo: https://github.com/googleapis/googleapis
jgeiger@jgeiger:~/repos/gapi-dev/googleapis$ virtualenv venv
New python executable in /usr/local/google/home/jgeiger/repos/gapi-dev/googleapis/venv/bin/python
Installing setuptools, pip, wheel...done.

jgeiger@jgeiger:~/repos/gapi-dev/googleapis$ . venv/bin/activate

(venv) jgeiger@jgeiger:~/repos/gapi-dev/googleapis$ pip install grpcio-tools
Collecting grpcio-tools
  Using cached grpcio_tools-1.0.0-cp27-cp27m-manylinux1_x86_64.whl
Collecting grpcio>=1.0.0 (from grpcio-tools)
  Using cached grpcio-1.0.0-cp27-cp27m-manylinux1_x86_64.whl
Collecting protobuf>=3.0.0 (from grpcio-tools)
  Using cached protobuf-3.0.0-py2.py3-none-any.whl
Collecting futures>=2.2.0 (from grpcio>=1.0.0->grpcio-tools)
  Using cached futures-3.0.5-py2-none-any.whl
Collecting enum34>=1.0.4 (from grpcio>=1.0.0->grpcio-tools)
  Using cached enum34-1.1.6-py2-none-any.whl
Collecting six>=1.5.2 (from grpcio>=1.0.0->grpcio-tools)
  Using cached six-1.10.0-py2.py3-none-any.whl
Requirement already satisfied (use --upgrade to upgrade): setuptools in ./venv/lib/python2.7/site-packages (from protobuf>=3.0.0->grpcio-tools)
Installing collected packages: futures, enum34, six, protobuf, grpcio, grpcio-tools
Successfully installed enum34-1.1.6 futures-3.0.5 grpcio-1.0.0 grpcio-tools-1.0.0 protobuf-3.0.0 six-1.10.0

(venv) jgeiger@jgeiger:~/repos/gapi-dev/googleapis$ python -m grpc.tools.protoc --python_out=tmp --grpc_out=tmp -I. -I/usr/local/google/home/jgeiger/repos/gapi-dev/googleapis google/logging/v2/log_entry.proto
protoc-gen-grpc: program not found or is not executable
--grpc_out: protoc-gen-grpc: Plugin failed with status code 1

What you describe as expected behavior is what we describe as expected behavior. Do you happen to know what version of pip is getting used to install grpcio-tools? What happens if you pip install --upgrade pip immediately after creating your virtual environment?
What else can you tell us about the platform on which you're seeing this problem?

s/--grpc_out/--grpc_python_out/
Does that fix your problem, @geigerj?

@soltanmm-google I was encountering the same error, and your change appears to work.

@nathanielmanistaatgoogle My versions are as follows:
(venv5) brianwatson@brianwatson0:~/src/gapi-dev/googleapis$ python --version
Python 2.7.12
(venv5) brianwatson@brianwatson0:~/src/gapi-dev/googleapis$ pip --version
pip 8.1.2 from /usr/local/google/home/brianwatson/src/gapi-dev/googleapis/venv5/lib/python2.7/site-packages (python 2.7)

@bjwatson Any chance y'all remember where you might've read that flag being grpc_out as opposed to grpc_python_out (so we can fix it)?

@soltanmm-google I think that pre-dates my time at Google. We've been using grpc_out for all languages, and just now are starting to change it for certain languages (Python, PHP, and Go).
@garrettjonesgoogle Any idea?

@bjwatson no idea, I haven't really dealt with Python codegen.

@garrettjonesgoogle It was more of a question about using --grpc_out across the board for all languages, but maybe that's a better questions for @ethanbao or @geigerj when they return from vacation.

If there's no documentation mismatch then the question of normalizing all grpc plugin output directory names is kinda out of scope until 2.0...


nested container types

This PR implements support for nested container types.
Here's a quick demo:
from dataclasses import dataclass, field
from typing import Dict, List

from pytest import raises

from omegaconf import MISSING, OmegaConf, ValidationError

@dataclass
class NestedContainers:
    dict_of_dict: Dict[str, Dict[str, int]]
    list_of_list: List[List[int]] = field(default_factory=lambda: [[123]])
    dict_of_list: Dict[str, List[int]] = MISSING
    list_of_dict: List[Dict[str, int]] = MISSING

cfg = OmegaConf.structured(NestedContainers(dict_of_dict={"foo": {"bar": 123}}))

print(OmegaConf.to_yaml(cfg))
# dict_of_dict:
#   foo:
#     bar: 123
# list_of_list:
# - - 123
# dict_of_list: ???
# list_of_dict: ???

with raises(ValidationError):
    cfg.list_of_dict = [["whoops"]]  # not a list of dicts

The biggest change to the source is the addition of a _deep_update_type_hint function to the basecontainer.py file. The purpose of this new function is to deeply validate compatibility of an OmegaConf object with a given type hint, mutating the object's metadata if necessary. This is necessary when merging nodes that are typed as nested containers, or when assigning an OmegaConf object to a field that is typed as a nested container.
resolves #427

Very nice work on this, especially on the tests. What's the python 3.7 constraint on this? It should probably be mentioned in the docs given how common 3.6 is

Thanks Pádraig!
I've just pushed a commit documenting the Python3.6 limitation.
The issue is with pickling subscripted typing. Here are some examples:
$ python3.6 -q
>>> from pickle import dumps
>>> from typing import *
>>> dumps(Dict[str, int])
...
_pickle.PicklingError: Can't pickle typing.Dict[str, int]: it's not the same object as typing.Dict
>>> dumps(DictConfig({}, element_type=Dict[str, int]))
...
_pickle.PicklingError: Can't pickle typing.Dict[str, int]: it's not the same object as typing.Dict

I believe this was the original motivation for OmegaConf's storing optional and ref_type metadata separately: the type hint Optional[int] cannot be pickled (by py3.6), but the ref_type int can be pickled together with a boolean flag indicating optionality.
Related is the BaseContainer.__getstate__ function, which sidesteps some of the above errors by discarding information from subscripted dicts and lists appearing in ref_type.


LP instead of QP?

Hi Brandon! Thanks for the great code!
In example-cls-layer, I tried to use an LP layer instead of a QP one, i.e. Q = Variable(torch.zeros(self.nCls, self.nCls)).
 Please make sure that your Q matrix is PSD and has a non-zero diagonal.
Any simple modifications I could make to get the LP working? Perhaps PDIPM is not the best for this special case of QP, but I'm only interested in playing around with this.
Thanks!

Hi Elias! To use the solver with LPs you can add a ridge term by setting Q = Variable(eps*torch.eye(nx)) for some eps. The ridge term makes the Hessian invertible for the default PDIPM solver in qpth and is also useful analytically since the solution of an unregularized LP is piecewise constant w.r.t. the parameters of the problem which makes the gradients w.r.t. the parameters not very useful
-Brandon.


f-string formatting

Fix #3257
Replaced .format() and % by f-strings in the following modules :

[x] tests
[x] metrics
[x] benchmarks
[x] utils
[x] templates
[x] src/Datasets/*.py

Modules in src/Datasets/:

[x] commands
[x] features
[x] formatting
[x] io
[x] tasks
[x] utils

Module datasets will not be edited as asked by @mariosasko
-A correction of the first PR (#3267)-

Hello @lhoestq, make style is applied as asked. :)


fix: Bug where got assertion error and no-exception to re-raise error [APE-1341]

What I did
So I am minding my own business, running tests in Ape core and all of a sudden I get random assertions errors in ape-etherscan for now reason... ok.
I really am easily peeved by assertion errors in production code. That should never happen.
So I go and look and comes to find out we are not handling the empty list case at all. So i fix it, and guess what happens? I get something even more dreadful
RuntimeError: No active exception to reraise

what.... why would there ever be an empty raise statement in production code?
So i go and fix that too.
How I did it
How to verify it
Run this test in ape core:
 pytest tests/functional/test_project.py::test_track_deployment_from_unknown_contract_missing_txn_hash

Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

closing in favor or upstream PR https://github.com/ApeWorX/ape-etherscan/pull/103


Add getUnsafe() interface for PrimarySelector

What changes are proposed in this pull request?
Replace the lock with volatile for the state variable in PrimarySelector because the state assignment is atomic and doesn't depend on any previous values.
Provide a getUnsafe() method to read the variable without adding a lock.
Why are the changes needed?
Improve the read performance of the primary selector.
Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including
N/A

alluxio-bot, merge this please


Exercise book library.

Adding (trying) book library exercise 014 to master repository.

I noticed I did not include an "if" condition to check if the book already exists in the library in the addBook method, nor to check if the book is not on the library in the removeBook method.


How to serve a static file directory with uWS

So for this next project I'm doing, I really want to use uWS for https and wss. Most of my projects have the two split unfortunately. This project is much simpler but I don't see how with uWS to serve static content. I don't want to have to make a wildcard and do file lookups. I just want a simple thing like express, where I just do .static(route). I have a couple image directories, audio directory, and an angular build dir I want to make static. Does uWS have a simple way of doing this?

No but it could possibly be done at some point, maybe.

@JemiloII You can look at here for ways to serve file/directory. Or use Cloudflare CDN as suggested by @nickchomey

Can't use cloud flare as a cdn, my website generates images and other content. It would be too slow to upload to a cdn and then serve. Faster to just serve from the server.

App.static(path) could definitely be a feature to look at some point but not right now. uWS doesn't exist where it doesn't provide exceptional value, and so if it would have app.static, it would have to be one of the best performing such features. Right now, that's not the case, since it would require kTLS and sendfile which would lock it to Linux only, which is not the plan. So it's more than just an opinion - I don't want to add features which aren't obviously motivated. Esp. not since most companies use proxies and proxies have static file serving built-in


Incorrect syntax in devcontainer.json file for loading VSCode

Describe the bug
VSCode has changed the way that extensions get loaded in the .devcontainers/devcontainer.json file. There is a missing line in between "customizations" and "extensions" to get VSCode loaded.
To Reproduce
Steps to reproduce the behavior:

Go to VSCode
Click on starting with dev container
See error

Expected behavior
VSCode properly loaded a docker container for the project.
Observed behavior
VSCode does not properly loaded a docker container.
Additional context
The devcontainer.json file could be updated with the following:
{
"name": "NYU Project",
"dockerComposeFile": "docker-compose.yml",
"service": "app",
"workspaceFolder": "/app",
"remoteUser": "vscode",
"customizations": {
"vscode": {
"extensions": [
"VisualStudioExptTeam.vscodeintellicode",
"ms-python.python",
"ms-python.pylint",
"ms-python.vscode-pylance",
"alexkrechik.cucumberautocomplete",
"cstrap.flask-snippets",
"yzhang.markdown-all-in-one",
"bierner.github-markdown-preview",
"hnw.vscode-auto-open-markdown-preview",
"DavidAnson.vscode-markdownlint",
"donjayamanne.githistory",
"GitHub.vscode-pull-request-github",
"hbenl.vscode-test-explorer",
"LittleFoxTeam.vscode-python-test-adapter",
"njpwerner.autodocstring",
"redhat.vscode-yaml",
"wholroyd.jinja",
"Zignd.html-css-class-completion",
"bbenoist.vagrant",
"rangav.vscode-thunder-client",
"ms-azuretools.vscode-docker",
"ms-kubernetes-tools.vscode-kubernetes-tools",
"streetsidesoftware.code-spell-checker"
]
},
},
"postCreateCommand": "sudo pip install -r requirements.txt",
// Use 'forwardPorts' to make a list of ports inside the container available locally.
// "forwardPorts": 8080,

// Features to add to the dev container. More info: https://containers.dev/features.
"features": {
    "ghcr.io/devcontainers/features/docker-in-docker:2": {},
    "ghcr.io/devcontainers/features/kubectl-helm-minikube:1": {}
}

}
To Fix
Replace old JSON: https://github.com/CSCI-GA-2820-SP23-001/shopcarts/blob/master/.devcontainer/devcontainer.json
With updated JSON: https://github.com/nyu-devops/project-template/blob/master/.devcontainer/devcontainer.json

@yitao-zhou changed story point per instructions "Assign Story Points to all of the Stories in the first Sprint (stick with 3,5,8,13 = S,M,L,XL for now)"


FieldValue.serverTimestamp() Is returning FieldValue(Instance of 'MethodChannelFieldValue')

Hello everyone, I'm trying to use timestamp from firebase but when I set the value FieldValue.serverTimestamp().toString(); on database it returns FieldValue(Instance of 'MethodChannelFieldValue'). What can i do to return a timestamp?

@saiyajindont
Your question is better suited for support channels like StackOverflow, so please ask the same there.
If you think, this is a bug in plugin, please report it in dedicated repo using issue template : https://github.com/firebase/flutterfire/issues


Create dependabot.yaml (fixes #2195)

Fixes #2195
Description
This (should) enable https://github.com/dependabot, see https://docs.github.com/en/code-security/dependabot.
Alternative(s) considered

Have you considered any alternatives? And if so, why have you chosen the approach in this PR?

Yes, always! 😺  I am aware of other such tools, but this is the best one, because it's from GitHub themselves, and thus most natively integrated. It also work great on many other open source projects where I have enabled it! 😃
Type
Choose one: ~Bug fix | Feature | Documentation | Testing |~ Code health | Builds | Releases | Security
Screenshots (if applicable)
The expected result would be to start seeing automagical Pull Requests such as these (from another repo) on this repo.

@vorburger do we know if this works before merging the PR?
We have a centralised place to keep the dependency info: https://github.com/google/android-fhir/blob/master/buildSrc/src/main/kotlin/Dependencies.kt in the buildSrc folder. Is dependabot able to handle this?

@vorburger do we know if this works before merging the PR?

No, not easily, unfortunately. My recommendation is to just merge this as-is; we could always git revert later, if it's causing problems.

We have a centralised place to keep the dependency info: https://github.com/google/android-fhir/blob/master/buildSrc/src/main/kotlin/Dependencies.kt in the buildSrc folder. Is dependabot able to handle this?

Yes, I expect it will; note how, since #2208 for #2194, https://github.com/google/android-fhir/network/dependencies now shows this project's dependencies; my expectation is that when this PR gets merged, Dependabot will start proposing upgrades.
In order to easily see which upgrades proposed by Dependabot "work" , perhaps you would like to merge #2203 to fix #2197 before this?

Yes, I expect it will; note how ...

Actually... I see what you mean; I'm not sure if Dependabot is smart enough to propose a correct PR, for this project as-is. We may have to do some manual work. Let's try and see? If it doesn't work, perhaps we could rework Dependencies.kt?


Add installation of adapters directly inside sublime text

It lacks user feedback but I added a way to install the adapters directly inside sublime text.
Now you can add an adapter inside your settings and the plugin can download it for you.

Much better than before!


Adds new strings for transifex translations

Adds new strings for transifex translations

A new build of 9b3cb2ecf0fb7582bf3ee27dc39de60bd0860b35 has completed succesfully!
Binaries available at https://build.nav.community/binaries/translations-transifex


Doorbird ng

@brownad - big update here.
I restructured parts of the plugin here for readability and fixed a few bugs along the way. I've simplified user configuration for the plugin.

FFmpeg autoconfiguration, if available, is now used. No user intervention needed on supported OSs.
Updated many of the configuration parameters and simplified the setup.
Updated README to reflect the above.
Fixed some bugs with the way the motion sensor and lights were triggered and reset.
Updated the ffmpeg.js to the latest from homebridge-camera-ffmpeg, though I removed Google Drive support because I view it as non-core to this plugin.
Simplified audio configuration.
Added manufacturer and serial information from the Doorbird.

There's a lot that I touched and mucked with. I've been running this for a few days without issue. Give it a whirl.

Wow!
Ok I’m on it tonight
Sent from my iPhone

On 1 Jun 2020, at 19:34, hjdhjd<EMAIL_ADDRESS>wrote:
@brownad - big update here.
I restructured parts of the plugin here for readability and fixed a few bugs along the way. I've simplified user configuration for the plugin.
FFmpeg autoconfiguration, if available, is now used. No user intervention needed on supported OSs.
Updated many of the configuration parameters and simplified the setup.
Updated README to reflect the above.
Fixed some bugs with the way the motion sensor and lights were triggered and reset.
Updated the ffmpeg.js to the latest from homebridge-camera-ffmpeg, though I removed Google Drive support because I view it as non-core to this plugin.
Simplified audio configuration.
Added manufacturer and serial information from the Doorbird.
There's a lot that I touched and mucked with. I've been running this for a few days without issue. Give it a whirl.
You can view, comment on, or merge this pull request online at:
https://github.com/brownad/homebridge-doorbird/pull/58
Commit Summary
Simplify Doorbird configuration for end user. Add info to accessories. Fix light support. Streamline ffmpeg support.
Update the README.
Typos.
Update dependencies.
File Changes
M README.md (160)
D config.json (35)
M config.schema.json (120)
D drive.js (315)
M ffmpeg.js (181)
M index.js (626)
M package.json (3)
Patch Links:
https://github.com/brownad/homebridge-doorbird/pull/58.patch
https://github.com/brownad/homebridge-doorbird/pull/58.diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Yeah...it started small, and then I just kept going. :)
Some cool stuff for the future in here too. Once we get this one merged in and live, the next rev we can target creating individual relays for activation since I started pulling the config information direct from the Doorbird.
I'd like to get this merged in, and people using it, and then start adding some of the other capabilities. Up next, if I can get the time, is to figure out how to make the microphone work based on how ring and the other apps have taken it on.
The README.md has been completely overhauled in this PR. Take a look as a guidepost for how I approached thinking about the plugin updates I made.

Just taking a look now, so much better 🥇.  Do you want to fix the conflicts then I'll merge and publish it tonight.

Let me try this again...

Let me try this again...

Np


Consolidate dependency versions in ranger plugin code ported from ranger-gaian-plugin

Porting the ranger-gaian-plugin from a standalone project into egeria has resulted in dependencies being expressed directly within the poms for each specific component added (plugin & impersonation).
Some plugins may be inherited from parents. Some should be (if they are common across modules), and in any case versions should be declared at the parent level to reduce the change of mixed versions being used during the build
See also #22 which proposes use of  to make this more consistant.

pom refactoring has addressed this


Use custom namespace when creating Velero clients

The Velero deployment did not have a way of exposing the namespace it
was installed in to the API client. This is a problem for plugins that
need to query for resources in that namespaces, such as the restic
restore process that needs to find PodVolume(Backup|Restore)s.
While the Velero client is consulted for a configured namespace, this
cannot be set in the server pod since there is no valid home directory
in which to place it.
This change provides the namespace to the deployment via the downward
API, and updates the API client factory to use the VELERO_NAMESPACE
before looking at the config file, so that any plugins using the client
will look at the appropriate namespace.
Fixes #1743
Signed-off-by: Nolan Brubaker<EMAIL_ADDRESS>

@dymurray and @sseago Please take a look, I think this fixes the issue you were seeing with PVRs.

Tests are currently flaking, will be fixed by #1745.

I think using the downward API makes sense here, but I do want to reconcile this new code with https://github.com/heptio/velero/blob/master/pkg/cmd/server/server.go#L174-L186 -- in particular, how the --namespace flag works if specified for velero server

I seem to remember that maybe we used to support running the Velero deployment in a different namespace than the CRDs existed in, which may be what some of this code is for, but that's probably not valid anymore.

I seem to remember that maybe we used to support running the Velero deployment in a different namespace than the CRDs existed in, which may be what some of this code is for, but that's probably not valid anymore.

Yeah, sounds like the split namespace stuff we did in 0.7. I'll take a look at the server's argument.

Reviewing this, I'm not quite sure I understand the implications of different Basenames. I don't see them being set anywhere in our code or the vendor directory.
My current thought is to change getServerNamespace to use the environment variable instead of the serviceaccount file, but I'm not sure that's correct given I don't really understand what the comment's talking about.

Ok, I think I found it - it's used in the user agent at https://github.com/heptio/velero/blob/master/pkg/client/client.go#L42-L48.
It gets defined as the name of the binary at https://github.com/heptio/velero/blob/master/cmd/velero/main.go#L34

I think it would be fine to switch to using the env var there -- I don't think we really need to worry about supporting storing the CRDs in a different NS than the pod is running in. Still not exactly sure how the --namespace flag would relate here - maybe it shouldn't be an option for this command.

So, thinking through swapping some of this logic around, I believe current behavior would be retained. The command line flag would win. From what I can see, the namespace extracted from the service account logic isn't really used anywhere, it's just a way to query kubernetes info without the downward API.
It is indeed possible to point the Velero server to a different namespace with --namespace, and it will use that namespace for all informers and client operations.
It's then possible, though I don't know how likely, that someone has many Velero deployments in a single namespace, each one responsible for managing another namespace with its own collection of CRs. However, I'm not sure that's really practical, since it'd require running multiple namespaces anyway; better to keep the deployments co-located, in my mind.
If we get rid of --namespace on the server, I think I'd like to do a deprecation period, since it is currently public.

One more wrinkle that using the factory will have to address - the server has QPS and burst limits on its clients, which the factory doesn't currently allow for setting.
This might be mitigated by a method to set these values, or maybe a rest.Config on the factory.

I think I've addressed all comments on this, as well as adding some tests where I can without doing a package move.

OK I think this looks good, but I'm going to take a fresh look in the morning and maybe test a little too.

I believe this is ready for final testing,

@nrb are you still testing, or is this ready to merge?

@skriss I'm ready to merge if you are


Fix: Crash on opening BubbleActivity

This PR resolves the crash that occurs when opening BubbleActivity. The issue was caused by the missing @AndroidEntryPoint annotation on the BubbleActivity class, which is essential for integrating Dagger Hilt dependency injection

Duplicate of #24


Usage without forking / cloning

I'm considering using this, but put off by the need to fork / clone.  What do people think about publishing a reusable versioned artifact to npm?

Utilising npm would be great but how would you provide your specific configuration.  It's a problem that has been discussed online a little.  Using s3, dynamo or packaging it with the deployment seem the only options.  I prefer the later which either means forking which I admit is not great or creating a separate project and pulling this in as a dependency.  That sounds like a process more involved than doing the fork and so I haven't bothered so far to investigate it.

I've not been fond of storing the configuration in either of those as a default as it would increase the delay at the start of each run.  Everything else is in parallel but that would have to complete before anything can start.  However, I have put placeholders in the provisioner did people to extend as they wish.
However, note that lambda allows environment variables I would accept a PR which given no variable uses the default config, but when defined uses that value to lookup a potentially different compiled in configuration.
If also accept PRs to use both S3 and dynamodb which could use another environment variable to determine which one of those is used


Adding BTC-e as an exchange and its authRequest() function

Hi all,
I've been trying to learn a bit about cryptocurrencies, coding etc etc and thought that a nice way may be to work on something here.  Anyway, I've got the code compiling and running which was the first hurdle and thought I'd try to add BTC-e as another exchange.
I've been trying to figure out how to add BTC-e as an exchange but I have no clue  about API authentication so when it comes to the authRequest() function I am flummoxed.  My usual strategy of looking at the cpp files for the other exchanges and figuring out by trial and error (lots of those in the debug window!) is failing miserably, as presumably each exchange has its own requirements for the structure of the request.
I've looked at the BTC-e https://btc-e.com/tapi/docs#auth but it makes little sense. I simply don't have the knowledge to understand the structure of the API request, let alone write it. I've done all the necessary declarations for the functions in the header and config files, so a functino si ready to go.  Is there anyone who can give me any pointers at least as to the structure of the request I am aiming for?
thanks

I would start with googling how http rest-style request works. There are plenty of tutorials to help.
For blackbird, check out https://github.com/butor/blackbird/blob/master/src/utils/restapi.h and use that to send the request. You can also check out https://github.com/greatwolf/cryptofu/blob/master/exchange/btce.lua#L11 which shows a working auth implementation for btc-e using Lua.

Thanks greatwolf, I feel like I'm inching closer.  The thing I can't seem to understand is the structure of the string that is to be hashed, and I can't work it our from your Lua code (the "post_data" string).
I am currently concatenating the nonce with the request eg (nonce=1494912588255&request=/getinfo) which then gets hashed and signed, but the query still returns "success:0" (i.e. FAIL...!).  Its not obvoius to me from the BTC-e documentation what the structure of the string should look like for the getinfo request.  How does one determine what this string should look like?

It looks like you're using the wrong name for the parameter. It's suppose to be method not request. There's also no forward slash in the method name. eg. nonce=123456&method=getInfo

Just FYI, if I remember correctly, there is a quirk with BTC-e API. This is a comment in the spreadbot1000 code:
"Special of BTC-e as an order that has directly been fulfilled the order id returned is always 0"

After trying most of the night, I think I am going to have to (temporarily) admit defeat on this.  I simply do not know enough about computing in general as my knowledge only extends to hacking about with recognisable bits of code.  I think its also possibly to do with the fact that the URL for the private api at BTC-e (https://btc-e.com/tapi)  is different to the URL for the public api (https://btc-e.com).  I'll look again in a couple of days after my eyes can focus again.  Thanks for the help thus far.

Do one really have to do all of that @gbsilp ? Can't one just add the key info in the conf file? Like you do with the other?

@jw4wellness The connection to BTC-e hasn't been implemented in Blackbird yet


GQ Signature flagging

Reading through https://github.com/bastionzero/openpubkey/pull/14 I had the following thoughts I wanted record about possible ways to flag that a signature if GQ Signature.
Consider the following two approaches:
The unsigned header flag approach
We flag that a OP RSA signature has been replaced by a GQ Signature by adding a unsigned header field: "sig_type": "oidc_gq" Lets call
{
	"payload": "payload",
	"signatures": [{
		"protected": {"alg":"ES256","rz":"5","upk":{"crv":"P-256","kty":"EC","x":"RI","y":"Hk"}},
		"header": {
			"sig_type": "cic"
		},
		"signature": "user signature"
	}, {
		"protected": {"alg":"RS256","kid":"6f7254101f56e41cf35c9926de84a2d552b4c6f1","typ":"JWT"},
		"header": {
			"sig_type": "oidc_gq"
		},
		"signature": "GQ signature"
	}]
}

The altered protected header approach
Alternatively we could alter the protected header changing the alg field to GQ256 and then signing the altered protected header and payload with the GQ signature. A verifier would then reconstruct the message the OP signed with RSA by changing the algorithm field back. It would look like:
{
	"payload": "payload",
	"signatures": [{
		"protected": {"alg":"ES256","rz":"5","upk":{"crv":"P-256","kty":"EC","x":"RI","y":"Hk"}},
		"header": {
			"sig_type": "cic"
		},
		"signature": "user signature"
	}, {
		"protected": {"alg":"GQ256","kid":"6","typ":"JWT"},
		"signature": "GQ signature"
	}]
}

Comparison
The unsigned header flag approach
Pros:

Results in a simpler implementation of the the verifier and the signer.
None of the values which are signed by OP are altered or reserialized.

Cons:

Standard OIDC validators don't know about GQ signatures or the sig_type header so they will likely attempt to verify the GQ signature as if it was a very large RSA signature resulting in a hard to debug failure.
Uses an unsigned header, which means we have to worry about what happens if someone alters this header. Ideally there is no content which is not covered by a signature. This is a minor con.

The altered protected header approach
Pros:

Standard OIDC validators will see that the alg is set to algorithm they don't support and fail gracefully producing a meaningful error message: Signature algorithm GQ256 is unsupported.
If we want to standardize GQ signatures in JWS as alg is the standard way of flagging the signing algorithm

Cons:

Likely a source of bugs and a pain point for implementors as signed message content is very fragile. Changing the protected header requires serializing and deserializing it. Different JSON libraries make minor changes to whitespace or the ordering of keys that result in signatures not verifying. These signature verification issues can creep in both when generating the GQ signature and when verifying it, as they both involve deserializing the protected header, making a change and then serializing it again.

We could bypass this con by storing the base64 encoding of the OPs signature in the protected header. That seems a little abusive toward the JWS standard.

The altered protected header approach is very nice, but that JSON serialization con is a very real problem. My first GQ experiments had that exact bug :laughing:
I was thinking we could just string replace - s/"alg":"RS256"/"alg":"GQ256"/ - but TIL that the JSON inside a JWS can contain whitespace :unamused:

We could bypass this con by storing the base64 encoding of the OPs signature in the protected header. That seems a little abusive toward the JWS standard.

I'm not sure I follow this, could you rephrase?

We could bypass this con by storing the base64 encoding of the OPs signature in the protected header. That seems a little abusive toward the JWS standard.

I'm not sure I follow this, could you rephrase?

OP RSA protected header (rsaph): {"alg":"RSA256","kid":"6","typ":"JWT"}
Take the OP's RSA protected header, base64 encode it and shove it into the GQ protected header, making the GQ protected header: {"alg":"GQ256","typ":"JWT", "rsaph": base64({"alg":"RS256","kid":"6","typ":"JWT"})}

I like the altered protected header approach. If someone wants to take it off the stack, let me know,. Otherwise I can put a PR together late next week.

I've been looking at this and have some working code for the GQ256 alg approach. However, we've since started using the unprotected header approach for cosigner signatures as well.
I think it would be fine to simply move the claim signature type claim to the protected headers on cosigner signatures. At the moment the unprotected header is called sig_type but we might want something shorter.
Any thoughts @EthanHeilman @lgmugnier ?

I think it would be fine to simply move the claim signature type claim to the protected headers on cosigner signatures. At the moment the unprotected header is called sig_type but we might want something shorter.

One option would be to get rid of sig_type altogether and just inferring if a signature is OIDC, CIC or COS based on the protecter header parameters.

OIDC signature - If no upk or iss claim in protected header.
CIC signature - If upk in protected header.
COS Signature - If iss in protected header.

If we did want to move the sig_type into the protected header, I would propose the using the typ (type) header introduced in RFC-7515 JSON Web Signature (JWS) and used by RFC-7519 JSON Web Token (JWT) and defined in the JOSE registry in place of sig_type.
The beauty of this approach is that the OP's OIDC signature should already have typ=JWT. Technically the JWT RFC says, "Use of this Header Parameter is OPTIONAL", but I've yet to see any OIDC Provider that does not set typ=JWT in the protected header of their signature. I just checked and Google, Azure, and Github Actions all specify typ=JWT.
Even if an OP did not set typ=JWT in the protected header, we could infer a signature was the OIDC signature because either typ=JWT or no typ was set in the protected header.

OIDC signature - If typ=JWT or no typ set
CIC signature - if typ=CIC
COS signature - if typ=COS

The only downside to this approach is that technically the typ header is suppose to describe the media type of the payload. We would be using it in a slightly off-label manner. The JWS RFC does really think through how typ should be used when there are multiple signatures and I think using typ in this fashion is reasonable and fits with the spirit if not the letter of the RFC.
RFC-7515 JSON Web Signature (JWS) Section 4.1.9](https://www.rfc-editor.org/rfc/rfc7515.html#section-4.1.9)

The "typ" (type) Header Parameter is used by JWS applications to declare the media type [IANA.MediaTypes] of this complete JWS. This is intended for use by the application when more than one kind of object could be present in an application data structure that can contain a JWS; the application can use this value to disambiguate among the different kinds of objects that might be present.  It will typically not be used by applications when the kind of object is already known.  This parameter is ignored by JWS implementations; any processing of this parameter is performed by the JWS application. Use of this Header Parameter is OPTIONAL. [..]

RFC-7519 JSON Web Token (JWT) Section 5.1

The "typ" (type) Header Parameter defined by [JWS] and [JWE] is used by JWT applications to declare the media type [IANA.MediaTypes] of this complete JWT.  This is intended for use by the JWT application when values that are not JWTs could also be present in an application data structure that can contain a JWT object; the application can use this value to disambiguate among the different kinds of objects that might be present.  It will typically not be used by applications when it is already known that the object is a JWT.  This parameter is ignored by JWT implementations; any processing of this parameter is performed by the JWT application.  If present, it is RECOMMENDED that its value be "JWT" to indicate that this object is a JWT.  While media type names are not case sensitive, it is RECOMMENDED that "JWT" always be spelled using uppercase characters for compatibility with legacy implementations.  Use of this Header Parameter is OPTIONAL.

This sounds good, I've updated #66 to use this approach. Another option would have been cty (https://www.rfc-editor.org/rfc/rfc7515.html#section-4.1.10) but it is indeed nice that the OIDC tokens already have typ set to JWT.

Another option would have been cty

Oh, that is interesting. If I am reading the RFC correctly the cty only covers the "secured content" which means it doesn't refer to other signatures?

I think it means it describes the payload field only, but (like with typ) it doesn't cover what it means to have different values in different signature headers.


Correct check for color being the default color or not

This pull reqest updates the Color class to also take into account of wether one of the colors that are being compared is the default color or not.
Additionally, I added examples for background color in the Samples console project

@AdmiringWorm Thanks for this! Appreciated! 👍


Javascript Github Action Warnings

Deprecation warnings from the GHA in prune-pr-images.yml due to Node.js in the<EMAIL_ADDRESS>which uses Node 12 (deprecated).  v1.0.1 is the latest version of ghcr-delete-image-action:
https://github.com/bots-house/ghcr-delete-image-action
so suggest cloning this code locally and replacing

runs:
using: 'node12'
main: 'dist/index.js'

with

runs:
using: 'node16'
main: 'dist/index.js'

in action.yml
Describe the Task
Provide a description of the task here.
Time Estimate
Estimate the amount of work required here.
Issues should represent approximately 1 to 3 days of work.
Sub-Issues
Consider breaking the task down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[ ] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[ ] Select component(s)
[ ] Select priority
[ ] Select requestor(s)

Projects and Milestone

[ ] Select Repository and/or Organization level Project(s) or add alert: NEED PROJECT ASSIGNMENT label
[ ] Select Milestone as the next official version or Future Versions

Define Related Issue(s)
Consider the impact to the other METplus components.

[ ] METplus, MET, METdataio, METviewer, METexpress, METcalcpy, METplotpy

Task Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above, including the Time Estimate and Funding Source.
[ ] Fork this repository or create a branch of development.
Branch name: feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into development.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s) and Development issues
Select: Repository level development cycle Project for the next official release
Select: Milestone as the next official version
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

The original action I used for image pruning seems regrettably unmaintained - the last commit was in 2021. So we could try to contribute the change you mentioned back to the original action if the maintainer is still paying attention. If they aren't, we could also investigate switching to a regularly maintained action like this one: https://github.com/vlaurin/action-ghcr-prune

Fixed in #134


Ability to move the legend

Matplotlib supports passing a location argument to legend to indicate where to move the legend on the graph: https://matplotlib.org/api/_as_gen/matplotlib.pyplot.legend.html
See loc : int or string or pair of floats, default: rcParams["legend.loc"] ('best' for axes, 'upper right' for figures)
Was curious if this was a possibility for the CPP version

Not yet, the current implementation doesn't accept any arguments:
inline void legend() { ... }


Disable dex / staticwebsite in staging hub

This seems broken, and is blocking deployment to anything on the 2i2c hub :( Disabling it to unblocko deployments.

See deployment failures in https://github.com/2i2c-org/infrastructure/actions/runs/4168960897 and https://github.com/2i2c-org/infrastructure/actions/runs/4168594911.
The primary culprit seems to be that the hub can not connect to the services to add them to the proxy. My suspicion is something about networkpolicies and how enforcement of those changed with the recent k8s upgrade.


dataset split is invalid (because of reset_index)

In the main() function, you have the following:
train_dataset=df.sample(frac=train_size, random_state = config.SEED).reset_index(drop=True)

val_dataset=df.drop(train_dataset.index).reset_index(drop=True)

But since you reset the index, the dropped rows will drop based on the new index. So your train and eval datasets overlap. You can reset the index only after doing drop
train_dataset=df.sample(frac=train_size, random_state = config.SEED)   #<-- removed rop
    val_dataset=df.drop(train_dataset.index).reset_index(drop=True)
    train_dataset = train_dataset.reset_index(drop=True)

@jsrozner , thanks for the feeback. I have updated the notebook with correction to the split.


Swift 4 Compatibility

When trying this as a cocoapod in Xcode 9 using Swift 4, compiling fails with the following error:
<project>/Pods/PiwikTracker/PiwikTracker/PiwikTracker.swift:202:89: Struct 'MemoryQueue' is internal and cannot be referenced from a default argument value

Is this library supposed to be compatible with Swift 4 or do we need to use a specific branch?

Hi @pieterclaerhout,
thanks for asking. You are totally right. We haven't added Swift 4 Compatibility yet. I just checked what is necessary to change and pushed a compatible version to this branch swift-4. Please try and let me know if you encounter any issues.

Works for me!

Hi @Shukuyen, thanks for your reply. I just fixed the carthage build error. Once the build completed I will merge it. I will close this issue once it is released.

I am still getting this error in a newly installed instance - is the fix already public? Also, I am getting another warning: Argument of '#selector' refers to instance method 'dispatch()' that is not exposed to Objective-C for which a fix is recommended (Add '@objc' to expose this instance method to Objective-C). As I don't want to edit the file itself as it would be overriden with the next update I wonder whether I need to open a new pull request, and if the update will be relased soon?

Hi @BernhardWebstudio, thanks for bringing this up. Unfortunately i merged it into the wrong branch 🤦♂
I will merge it into the development branch and release a patch version right away.
For the warning: It would be great if you create a PR for it.

I checked the issue. I am not sure why it isn't working for you. It is released in Version 4.2.0 and I just checked in the iOS Example Application if it works. Can you check if you use the latest version?

I was using 4.1.0 as this is still the default version loaded by CocoadPods and referenced in the README. By specifiyng 4.2.0 as the desired version, it is working as expected. Thank you for the support and sorry for bothering.

Oh, sorry. The 4.1 in the README was irritating.


build-figma-plugin --typecheck --watch fails to utilize changes made to tsconfig.json

Fixed by https://github.com/yuanqing/create-figma-plugin/commit/f905809feeca678d7e3784b61b68e4c505a72d3a.