Ouroboros Compile Issue with VOC

Hello,
So I've been trying a lot, since the last 4 days, to get my head into this. After a lot of problems, I thought, that I must first work on resolving the compile issue of a particular file of VOC, that's failing currently, so that things can become easier ahead. On the contrary, this is giving some more problems itself.
For debugging purposes, I am printing the node value, that's being traversed in the Visitor() class of ast.py.
I tried compiling 3 python scripts with VOC-:

example1.py - contains the exact same ouroboros.collections.py It's error log can be found at example1_log_with_node_names.txt . Please note the order in which the nodes are printed, and the final line number as well (it's mentioned in the last line), where the error appears.

example2.py - This is just a sample generator function which compiles perfectly with VOC. It's compile log can be found at example2_log_with_node_names.txt.

example3.py - Basically contains example2.py's generator method added to the top of example1.py. It's compile log can be found at example3_log_with_node_names.txt

This is the GitHub GIST link - https://gist.github.com/ashutoshsaboo/52be09c551b8ffaedf35b727378bf2d8 which can be referred to for all these files.
Certain observations:

If we notice the order of nodes that are visited in example2.py, and example3.py, what's the change is that, in example2.py - a BinOp is generated inside the Yield node , whereas in example3.py - a Attribute Node is generated inside the Yield node.

From this, we might presume that there could be some problem related to VOC's code base in general itself, but if the compile log of example3.py and example2.py is compared, we can see that, the method that we wrote in example2.py compiles perfectly. So there's some issue related to node generation with respect to the ast module for Classes in general itself.

I've been caught up in this problem, and it would be really great, if someone could help on this.
@eliasdorneles @cflee @freakboy3742 Any help from you would be really great!

Thanks for the report. At this stage, VOC is still very much a work in progress, but at this point, what you're trying to do is definitely not working yet. It's also a very large chunk of work.
Let's break things down a bit. You've been commenting in gitter recently about things "not working", but we're still not entirely sure what you want to achieve. Are you just trying to implement NamedTuples? This is functionality that is one small part of the entire Collection module. Once VOC is fully operational, we'll be able to use the Python implementation of the Collections module, which is defined as part of Ouroboros, and get all the benefits of the Collections module without needing to write anything specific.
However, you're trying to import ALL of collections before VOC is fully complete.
VOC, and the entire BeeWare suite, is a iterative process. Can you reduce the problem to a minimal example (or examples) that don't work? What we need are small, well defined demonstrations of problems - like what was done here: pybee/voc#382
If you encounter issues, it greatly helps the core development team if you explain three things with your bug report: A) What you're trying to achieve, B) the exact error you're getting as a text, and C) what you've tried to do to correct the error.
Thanks!

Hi @glasnt . Thanks.
What I was trying to do with NamedTuple was basically start the native collections module for VOC itself. I didn't knew then the Ouroboros had a native implementation already. That was the problem.
I understand your point, for a minimal test case. I'll look into it. And the minimal thing here, is mostly that yield is not working in the same way with self - classes and without them. For the latter case, it works perfectly, on the other hand for the first one it doesn't work fine. (My point number 2 in the Issue's main post). I guess we need to look at the classes transpilation for VOC.
Though I'm not sure about this, but after all the debugging, that seems to be the root cause of the problem at-least. And that is what, we could also infer from example1.py , example2.py as well as example3.py . What I wanted to work on was basically work on atleast making VOC compile all the Ouroboros files, for which the methods are already defined. For instance, currently VOC can already handle yield instances, and hence it should be able to compile the yield methods within the collections.py file as well, i.e, the error shouldn't be at the yield statement i mean, which is happening now. For other instances like, lambda and others as well, which aren't supported by VOC yet, it's fine if it can't compile the files.
I hope to work on this more, so that we can move a step ahead towards the resolution for this issue! :smile: Would be great to have your points on this @glasnt @freakboy3742 @cflee @eliasdorneles ! :+1:


Cannot deliver jar file via the JSON api

The API currently blows up when trying to serve java problems, due to the non-utf-8 characters in the gradle archive.
It is perhaps a bit excessive to serve a jar file with each problem. Should we provide a link to the jar and have people download it and put it in their PATH?
Should we simplify the testing? (we really don't need anything fancy, the problems are all very tiny).

Spoke with @kytrinyx about this, this morning.  Since this discussion, we've (rather by default) apparently settled on using a dependency management tool (i.e. Gradle).  Effectively, issue closed.


[et] Failed to parse JSON output from gn desc

Seen at
14d27622b3144d1e14218cbb789846b00360416a
Reproduction
$ ./flutter/bin/et build -c host_debug_unopt_arm64 //flutter/impeller:impeller_unittests
[2024-05-15 15:03:50.299682] ERROR: Failed to parse JSON output from `gn desc`:
FormatException: Unexpected character (at character 1)
WARNING at build arg file (use "gn args <out_dir>" to edit):47:24: Build ar...
^

[2024-05-15 15:03:50.306646] ERROR: FatalError caught in main. Please file a bug
error: Failed to parse JSON output from `gn desc`:
FormatException: Unexpected character (at character 1)
WARNING at build arg file (use "gn args <out_dir>" to edit):47:24: Build ar...
^

stack: #0      Logger.fatal (package:engine_tool/src/logger.dart:125:5)
#1      Gn.desc (package:engine_tool/src/gn.dart:69:27)
<asynchronous suspension>
#2      BuildCommand.run (package:engine_tool/src/commands/build_command.dart:85:41)
<asynchronous suspension>
#3      CommandRunner.runCommand (package:args/command_runner.dart:212:13)
<asynchronous suspension>
#4      ToolCommandRunner.run (package:engine_tool/src/commands/command_runner.dart:97:14)
<asynchronous suspension>
#5      main (package:engine_tool/main.dart:86:19)
<asynchronous suspension>

I think maybe the error may be more apparent if the output wasn't elided.  I didn't add those elipses in the description.  I think after them may have been the GN error.

et works fine for my diff at 1ccd0c308b3a567fc41bc287a584f0946cca6c6b.  Something went wrong between those commits.

My diff added a file to GN, I'm not sure if that is required for reproducing the bug.  I can supply it if it helps.

That could be useful

Locally I can't reproduce:
flutter % et build -c host_debug_unopt_arm64 //flutter/impeller:impeller_unittests
[2024-05-15T15:57:18.043][macos/host_debug_unopt_arm64: GN]: OK
[2024-05-15T15:57:19.041][macos/host_debug_unopt_arm64: RBE startup]: Proxy started successfully.
[2024-05-15T15:59:05.520][macos/host_debug_unopt_arm64: ninja]: 100.0% (893/893) LINK ./impeller_unittests
[2024-05-15T15:59:05.524][macos/host_debug_unopt_arm64: ninja]: OK
[2024-05-15T15:59:05.893][macos/host_debug_unopt_arm64: RBE shutdown]: Actions completed: 374 (331 cache hits, 43 racing locals)

Reduced to P1: the problem is we're not giving actionable feedback, not my specific patch.
https://github.com/flutter/engine/commit/41f10275bf1e432671ade35cf13403c73ea68a50 removed concurrent_dart_jobs, requiring a ./tools/gn --unopt --mac-cpu=arm64 that et doesn't handle today. I'll fix the logging issue (1-line fix), and file another issue for improved error handling/recovery.


Updated links to REST literature

👍

👍

How about also adding: https://www.thoughtworks.com/de/insights/blog/rest-api-design-resource-modeling as a new resource here. I'm not sure whether we have mentioned this resource somewhere else, but I think it contributes alot.

How about also adding: https://www.thoughtworks.com/de/insights/blog/rest-api-design-resource-modeling as a new resource here. I'm not sure whether we have mentioned this resource somewhere else, but I think it contributes alot.

:+1:

:+1:

👍

👍

:+1:

:+1:


IndexOutOfRange

数组赋值失败，因为索引“1370248”超出范围。
所在位置 C:\Users\Administrator\Desktop\Invoke-PSImage\Invoke-PSImage.ps1:122 字符: 20
+         $rgbValues[ <<<< ($counter*3+1)] = ($rgbValues[($counter*3+1)] -band 0xf0) -bor $paybyte2
    + CategoryInfo          : InvalidOperation: (1370248:Int32) [], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : IndexOutOfRange

I am getting the same thing
`Index was outside the bounds of the array.
At line:100 char:9

    $rgbValues[($counter*3+2)] = ($rgbValues[($counter*3+2)] -ban ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (:) [], IndexOutOfRangeException
FullyQualifiedErrorId : System.IndexOutOfRangeException`

I am getting the same thing
Index was outside the bounds of the array.
At line:100 char:9

    $rgbValues[($counter*3+2)] = ($rgbValues[($counter*3+2)] -ban ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (:) [], IndexOutOfRangeException
FullyQualifiedErrorId : System.IndexOutOfRangeException

I am getting the same thing
Index was outside the bounds of the array.
At line:100 char:9

    $rgbValues[($counter*3+2)] = ($rgbValues[($counter*3+2)] -ban ...

    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (:) [], IndexOutOfRangeException
FullyQualifiedErrorId : System.IndexOutOfRangeException

It turned out to be the png file I was using. I grabbed another one and it worked fine


Product Drives -- allow filtration by category

Summary
Allow filtration of product drives by item category
Why?
Provides more useful presentation of information to the banks
Details
Add item category to the possible filters for product drives
When filtered on item category,
1/  Only those product drives that produced items in that category will be shown
2/  Any totals of items, values, etc will only count the item in the chosen category
What view, exactly are we talking about
Click on Product drives | All Product Drives in the left-hand menu
Original request info
Requested along with product drive export in support ticket by Judi M 20220808
Criteria for completion

[ ] Banks can filter the product drive list by item category as described above
[ ] tests to support this functionality

Hey @cielf I'd be happy to try my hand at this one

You got it!

Hey @dorner  working on this now and making my way through adding/adjusting tests for this functionality. I've noticed that there is 3 test cases from distribution_system_spec.rb that are failing independently of my changes. Have these been raised as their own issue? I've run the tests a number of times and its appears to be flaky test cases as they can occasionally pass on some runs.
Failures:

  1) Distributions When creating a distribution from a donation when editing that distribution User creates a distribution from a donation then edits it
     Failure/Error: click_on "Distributions", match: :first

     Ferrum::JavaScriptError:
       TypeError: Cannot read properties of undefined (reading 'response')
           at HTMLDocument.<anonymous> (http://<IP_ADDRESS>:36529/assets/controllers/turbo_controller-fccb6961e984a4e30ac5da2e1cf185bbfb9d9e8677a0bf2e284ec349e7b0db17.js:16:47)
           at S (http://<IP_ADDRESS>:36529/assets/turbo.min-a07af75140e849147b6b70f34d0918d27f2eff08dd6d957cd3c7211649928372.js:1:5013)
           at B.requestFinished (http://<IP_ADDRESS>:36529/assets/turbo.min-a07af75140e849147b6b70f34d0918d27f2eff08dd6d957cd3c7211649928372.js:1:13793)
           at H.perform (http://<IP_ADDRESS>:36529/assets/turbo.min-a07af75140e849147b6b70f34d0918d27f2eff08dd6d957cd3c7211649928372.js:1:7612)

     [Screenshot Image]: /home/sean/human-essentials/tmp/screenshots/failures_r_spec_example_groups_distributions_2_when_creating_a_distribution_from_a_donation_when_editing_that_distribution_user_creates_a_distribution_from_a_donation_then_edits_it_75.png

     # ./spec/system/distribution_system_spec.rb:381:in `block (4 levels) in <top (required)>'

  2) Distributions When creating a distribution from a donation when editing that distribution User creates duplicate line items
     Failure/Error: click_on "Edit", match: :first

     Capybara::ElementNotFound:
       Unable to find link or button "Edit"

     [Screenshot Image]: /home/sean/human-essentials/tmp/screenshots/failures_r_spec_example_groups_distributions_2_when_creating_a_distribution_from_a_donation_when_editing_that_distribution_user_creates_duplicate_line_items_872.png

     # ./spec/system/distribution_system_spec.rb:382:in `block (4 levels) in <top (required)>'

  3) Distributions When creating a distribution from a donation when editing that distribution User creates a distribution from a donation then tries to make the quantity too big
     Failure/Error: click_on "Edit", match: :first

     Capybara::ElementNotFound:
       Unable to find link or button "Edit"

     [Screenshot Image]: /home/sean/human-essentials/tmp/screenshots/failures_r_spec_example_groups_distributions_2_when_creating_a_distribution_from_a_donation_when_editing_that_distribution_user_creates_a_distribution_from_a_donation_then_tries_to_make_the_quantity_too_big_762.png

     # ./spec/system/distribution_system_spec.rb:382:in `block (4 levels) in <top (required)>'

Finished in 36 minutes 13 seconds (files took 15.89 seconds to load)
1989 examples, 3 failures, 2 pending

Failed examples:

rspec ./spec/system/distribution_system_spec.rb:386 # Distributions When creating a distribution from a donation when editing that distribution User creates a distribution from a donation then edits it
rspec ./spec/system/distribution_system_spec.rb:409 # Distributions When creating a distribution from a donation when editing that distribution User creates duplicate line items
rspec ./spec/system/distribution_system_spec.rb:395 # Distributions When creating a distribution from a donation when editing that distribution User creates a distribution from a donation then tries to make the quantity too big

Randomized with seed 58545

Coverage report generated for RSpec to /human-essentials/coverage. 5797 / 6214 LOC (93.29%) covered.

I'm happy to take a look at them as I'm working on my own test cases but not sure if you want to include them in the pull request for this issue or have them separated out to another issue.

cc: @awwaiid - can you take a look here?
@PhantomSean it's weird because we haven't seen failures with these tests on main. Do they happen "most of the time"?

cc: @awwaiid - can you take a look here?
@PhantomSean it's weird because we haven't seen failures with these tests on main. Do they happen "most of the time"?

They were happening "most of the time" the other day when I first noticed the issue. I've just run bundle exec rspec 10 times and hit these failures on the 1st and 3rd run. All other runs passed so maybe it's not happening as often as I first thought.
I've been hitting the failures using https://github.com/PhantomSean/human-essentials/tree/main which I believe is currently up to date with main.

@PhantomSean let's just get the PR up and see what happens on CI.

Hey @dorner / @awwaiid,
Have the pull request created and it is failing on the following system rspec test
rspec ./spec/system/distribution_system_spec.rb:108 # Distributions When creating a new distribution manually when the quantity is lower than the on hand recommended quantity should display an alert

Would one of you be able to take a quick look at my code changes? As far as I can see, the changes I've made shouldn't be affecting anything with the distributions or their pages.

Yeah this seems to be a flaky test. Not sure why it's happening, but it's unrelated to your changes.

Yes -- I have a fix at a6371f79ade7662e2b433c8b2add380c1289a43d over in another branch! Cuprite is faster than javascript, and one of the dropdowns is dynamically generated in that spec.


fix: use modern dialect registration

@machow are you able to confirm that these changes fix the issues you experienced (and remove the need for a special case for the dialect load)

Yes--it looks good on my side! Thanks for the quick turnaround (and sqlalchemy wrangling, which I realize involves a lot of moving parts...).
Here's the code I ran to check.
from sqlalchemy import create_engine
from sqlalchemy.dialects import registry

from types import ModuleType

# Check 1: Loading dialect directly ----

dialect_cls = registry.load("duckdb")
assert not isinstance(dialect_cls, ModuleType)

# Check 2: creating engine -----

engine = create_engine("duckdb:///:memory:")
engine.execute("SELECT 1").fetchall()

# Check 3: creating siuba lazy tbl ----

from siuba.sql import LazyTbl
engine.execute("CREATE TABLE test AS (SELECT 1 AS x)")

LazyTbl(engine, "test")


allocator/podcidr: fix formatting issue with debug message

No need to run the entire CI for this change.


Add base_conditional_with_lm function

PR content:

Title
Description
Minimal working example

Title
Allow Lm to be passed into utility functions for computation of conditionals.
Description
This PR adds a new utility method: base_conditional_with_lm in conditionals/util.py which accepts Lm instead of Kmm. This enables the performance of, for example, prediction to be improved by passing a pre-computed value for Lm into this function, instead of calculating it each time.
This is a hopefully (improved) version of #1514, which uses two functions instead of trying to put both Kmm and Lm into a single function.
Minimal working example
Here is a rough example of how I would use the new base_conditional_with_lm function in my model. This improves prediction time by about 10x.
class MySVGP(SVGP):

    def __init__(self, kernel, likelihood, inducing_variable, *, mean_function=None,
                 num_latent_gps: int = 1, q_diag: bool = False, q_mu=None, q_sqrt=None,
                 whiten: bool = True, num_data=None):
        super().__init__(kernel, likelihood, inducing_variable, mean_function=mean_function,
                         num_latent_gps=num_latent_gps, q_diag=q_diag, q_mu=q_mu, q_sqrt=q_sqrt,
                         whiten=whiten, num_data=num_data)
        self._Kmm_for_prediction = None  # [M, M]
        self._Lm_for_prediction = None  # [M, M]

    def train(self):
        # Do training here...
        self._Kmm_for_prediction = covariances.Kuu(self.inducing_variable,
                                                   self.kernel, jitter=default_jitter())
        self._Lm_for_prediction = tf.linalg.cholesky(self._Kmm_for_prediction)

    def predict_f(self, Xnew: InputData, full_cov=False, full_output_cov=False) -> MeanAndVariance:
        if self._Kmm_for_prediction is None or self._Lm_for_prediction is None:
            raise ValueError('Kmm or LM is not set. Use the train method to train the model.')
        mu, var = self.compute_conditional(Xnew, full_cov, full_output_cov)
        return mu + self.mean_function(Xnew), var

    def compute_conditional(self, Xnew, full_cov, full_output_cov):
        Kmn = Kuf(self.inducing_variable, self.kernel, Xnew)  # [M, N]
        Knn = self.kernel(Xnew, full_cov=full_cov)
        fmean, fvar = base_conditional_with_lm(
            Kmn, self._Lm_for_prediction, Knn, self.q_mu, full_cov=full_cov,
            q_sqrt=self.q_sqrt, white=self.whiten
        )  # [N, R],  [R, N, N] or [N, R]
        return fmean, expand_independent_outputs(fvar, full_cov, full_output_cov)

@st-- Based on your feedback on #1514 I have created a new PR to try and solve this.
I thought that instead of the approach you suggested of having a Kmm_is_cholesky flag it might be cleaner to have two separate functions (one of which calls the other). What do you think of this approach?
If you think this is a good approach then I can add some tests and hopefully we can get this merged.

@st-- Any thoughts on this approach? We'd like to incorporate this GPflow change into our models fairly soon, if we can.

I'd be happy to merge this but unfortunately codecov-project seems to be having issues... :S

@st-- Thanks. Do you know if there's any way to get codecov to try again? I could try and push an insignificant change this to PR to try and trigger it, but there might be a better way?


\!': KaTeX parse error: Got group of unknown type: 'internal'

Trying to move a prime symbol ' slightly to the left, e.g., to emulate arc seconds,
.\!''

gives the KaTeX error
KaTeX parse error: Got group of unknown type: 'internal'

Thanks for all your reports! However, I'm not sure you need to document every single context where super and subscripts currently can't be placed. In particular, this one seems redundant with #3878. It would be much more helpful to merge all of these examples into a single issue, as the root cause is identical.

Didn't realize this is a duplicate, the MWE doesn't involve super- or subscripts. Since the error message is the same, the issues probably share some code paths though. Will add this #3878.


use php 7.3 and not-outdated dependencies for phpstan

related #471

Can we get this merged please?

Sorry for the wait, there you go!


Real-Time usage

Hey I'm sorry but I am a bit confused, by online do you mean real-time? Because for a real-time use case we cannot store the psnr results before hand, and I am currently trying the code on a tesla p100, and the psnr results itself take time to generate (about 3 fps)

Hello, yes by online we mean real-time. I am not sure why it is taking so long for the psnr results to generate, could you maybe check if it is using the GPU? For reference, it took me a couple of minutes for the entire dataset using a RTX 2070.

Oh okay, thanks for your reply, not sure why its so slow then. Yeah I just checked, I am using a GPU. I was running it on colab.

@vishalned Hi Vishal, would you mind sharing with me your Colab Notebook link? I'm struggling to run it online.
Here is my email<EMAIL_ADDRESS>Thank you so much.

@mausLe Thank you so much! And no worries about the messy code! Lol.
Cheers!

@mausLe Hey! Hope you're doing well. I was going through your code and have found myself stuck on the following line of code and I am unable to locate/trace the file ** myTrainingResult.json**. I have searched for the file as well, but I could not find it.
Snippet of the Code:
!./darknet detector test cfg/coco.data cfg/yolov4.cfg /content/darknet/cfg/yolov4.weights -ext_output -dont_show -out /content/Data/Codes/myTrainingResult.json < /content/Data/trainImagePath.txt
Any type of help would be highly appreciated!
Cheers!

Hi @Zain-Gill123
The purpose of the above snippet of code is to export the YOLO detection result of every input image in trainImagePath.txt into the myTrainingResult.json
Check out this link: https://github.com/kevaldoshi17/Prediction-based-Video-Anomaly-Detection-/blob/main/result_train_ucsd.json
The reason I created the myTrainingResult.json was similar to [myTrainingResult.json](https://github.com/kevaldoshi17/Prediction-based-Video-Anomaly-Detection-/blob/main/result_train_ucsd.json), when he used the detected details to run the prediction directly without calling the YOLO model.
See the 2nd section where KevalDoshi loaded the predicted object details result_train_ucsd.json https://github.com/kevaldoshi17/Prediction-based-Video-Anomaly-Detection-/blob/main/MONAD.ipynb

Though I assume that I remove this file later I found it pretty useful if you need to save time to process the training object.
Try to review KevalDoshi's Notebook carefully, I suppose that my work is derived from his

@mausLe Thank You so much for the quick and informative response! Solved the problem.
Much appreciated!
Cheers!


OpenVPN User/password

I use a Synology NAS with OpenVPN. and you need to login with user/password.
is it possible you can add a VPNAUTH_USER and VPNAUTH_PASSWORD option?
<<<config
proto udp
script-security 2
comp-lzo
reneg-sec 0
cipher BF-CBC
auth SHA1
auth-user-pass
<<<config
2021-10-13 12:22:27 WARNING: Compression for receiving enabled. Compression has been used in the past to break encryption. Sent packets are not compressed unless "allow-compression yes" is also set.
2021-10-13 12:22:27 DEPRECATED OPTION: --cipher set to 'BF-CBC' but missing in --data-ciphers (AES-256-GCM:AES-128-GCM). Future OpenVPN version will ignore --cipher for cipher negotiations. Add 'BF-CBC' to --data-ciphers or change --cipher 'BF-CBC' to --data-ciphers-fallback 'BF-CBC' to silence this warning.
2021-10-13 12:22:27 OpenVPN 2.5.2 x86_64-alpine-linux-musl [SSL (OpenSSL)] [LZO] [LZ4] [EPOLL] [MH/PKTINFO] [AEAD] built on May  4 2021
2021-10-13 12:22:27 library versions: OpenSSL 1.1.1k  25 Mar 2021, LZO 2.10
2021-10-13 12:22:27 neither stdin nor stderr are a tty device and you have neither a controlling tty nor systemd - can't ask for 'Enter Auth Username:'.  If you used --daemon, you need to use --askpass to make passphrase-protected keys work, and you can not use --auth-nocache.
2021-10-13 12:22:27 Exiting due to fatal error

This seems to be implemented already, but I put my username and password in a file called "credentials.conf" and change the line where it says "auth-user-pass" to "auth-user-pass credentials.conf".
The credentials.conf file as follows:
username
password

@aseelert @lasharor I'm not understanding the problem. Can you not supply a file with the username and password? Are you using the built-in Synology OpenVPN client? If so, how are you using this image?

@wfg Running Syno here. I'm using this image with a container with a .ovpn file for the connection. The server needs credentials to log in, hence @Johan-Claesson 's suggestion. I however was not able to implement his suggestion

i am facing same error

OpenVPN configuration files have an auth-user-pass option that can be set to the name of a file containing a username and password in the following format:
username
password

So to be clear, if you have a file called my-credentials.txt in a directory with the following contents:
my-username
my-password

and you have my-config.ovpn in the same directory with the line
auth-user-pass my-credentials.txt

inside, when OpenVPN starts, it will authenticate against the remote with those credentials, and nothing should have to be passed in.
If this doesn't work for you, someone with a Synology will have to explain what the actual problem is here.

To add to this discussion, here's the error that is returned on my Synology:
"2022-04-30 21:17:56 | stdout | 2022-04-30 21:17:56 Exiting due to fatal error
2022-04-30 21:17:56 | stdout | 2022-04-30 21:17:56 neither stdin nor stderr are a tty device and you have neither a controlling tty nor systemd - can't ask for 'Enter Auth Username:'.  If you used --daemon, you need to use --askpass to make passphrase-protected keys work, and you can not use --auth-nocache.
2022-04-30 21:17:56 | stdout | 2022-04-30 21:17:56 library versions: OpenSSL 1.1.1n  15 Mar 2022, LZO 2.10
2022-04-30 21:17:56 | stdout | 2022-04-30 21:17:56 OpenVPN 2.5.6 x86_64-alpine-linux-musl [SSL (OpenSSL)] [LZO] [LZ4] [EPOLL] [MH/PKTINFO] [AEAD] built on Mar 24 2022
2022-04-30 21:17:56 | stdout | 2022-04-30 21:17:56 DEPRECATED OPTION: --cipher set to 'AES-256-CBC' but missing in --data-ciphers (AES-256-GCM:AES-128-GCM). Future OpenVPN version will ignore --cipher for cipher negotiations. Add 'AES-256-CBC' to --data-ciphers or change --cipher 'AES-256-CBC' to --data-ciphers-fallback 'AES-256-CBC' to silence this warning."
My setup seems to be correct: credentials in file and reference to credentials file in configuration file. The "neither stdin nor stderr are a tty device" part of the message seems to be the culprit. Can't figure this one out either.

@silentioetfide can you share your config file?

The proposed solution works for me:
creating a 'credentials.conf' in the same folder as the OpenVPN.conf
touch credentials.conf
adding on line 1 the username
adding on line 2 the secret
saving the bunch
starting the container

https://github.com/wfg/docker-openvpn-client#vpn-authentication


test_out_file: fix file open mode for binary file

Which issue(s) this PR fixes:
Fixes #4727
What this PR does / why we need it:
Fix the file open mode so that gz files can be read properly.
Docs Changes:
Release Note:

Excellent!


Resolve to oui.opensearch.org

Is your feature request related to a problem? Please describe.
We need the hosted OUI document site for our community to be accessible via a human readable URL.
Describe the solution you'd like
https://oui.opensearch.org/ should resolve to the doc site.

:tada:


Update README.markdown

We're removing the Puppet version of the CP docs from the documentation, and housing CP content with PE docs. This is the guide that used to live with the Puppet docs, and might be of use to Puppet users.

Found in housekeeping pass and cloud provisioner is deprecated.


Bugdown: Duplicate links in embeds

When you send multiple duplicate links in a single chat message there are two embeds created for the exact same content.
Example: (https://www.youtube.com/watch?v=4bfECHhoBBQ https://www.youtube.com/watch?v=4bfECHhoBBQ) would create two embeds for the exact same video

@zulipbot claim

Already working on a fix for this. Will claim and create PR once my previous issue is fixed.

@aayushagra are you still working on this? If not, I will like to take this up

@shreyanshdwivedi The issue already has a pending PR


DCOS-17939: Blacklist historySummary

historySummary field prevents you from saving a metronome job, so we should just blacklist it as it is a meta information. Please refer to the ticket to come up wit a testing scenario.

LGTM! 👓 👍


No ticking after resuming from non-FlutterActivity

Steps to Reproduce
When passing a result from Android back to Flutter through MethodChannel.Result directly from a FlutterActivity::onActivityResult without waiting for the call to FlutterActivity::onResume(), some things go wrong. For example, showing a snack bar or starting an animation doesn't work.
I've attached a repro. The app is supposed to increment both counters at the same rate, but the second one only increments at half the rate.
Logs

flutter run output:
Launching lib\main.dart on Pixel 2 XL in debug mode...
Initializing gradle...
Resolving dependencies...
Running 'gradlew assembleDebug'...
Built build\app\outputs\apk\debug\app-debug.apk (25.4MB).
Installing build\app\outputs\apk\app.apk...
I/FlutterActivityDelegate( 8137): onResume setting current activity to this
Syncing files to device Pixel 2 XL...
D/AppActivity( 8137): onCreate()
D/MainActivity( 8137): onActivityResult()
I/FlutterActivityDelegate( 8137): onResume setting current activity to this
I/flutter ( 8137): Started animation after 'startActivity'
I/flutter ( 8137): Started animation after 'none'

I tapped the FAB twice and the screen now says (2 forward / 1 completed).

flutter analyze output:
No issues found!
Ran in 7.0s

Flutter Doctor

flutter doctor -v:
[√] Flutter (Channel beta, v0.1.5, on Microsoft Windows [Version 10.0.16299.309], locale en-US)
• Flutter version 0.1.5 at C:\Users\Christian\Documents\Visual Studio 2017\ThirdParty\flutter
• Framework revision 3ea4d06340 (4 weeks ago), 2018-02-22 11:12:39 -0800
• Engine revision ead227f118
• Dart version 2.0.0-dev.28.0.flutter-0b4f01f759

[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
• Android SDK at C:\Users\Christian\AppData\Local\Android\sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-27, build-tools 27.0.3
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[√] Android Studio (version 3.0)
• Android Studio at C:\Program Files\Android\Android Studio
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)
[√] Connected devices (1 available)
• Pixel 2 XL • 710KPYR0349426 • android-arm64 • Android 8.1.0 (API 27)
• No issues found!
animation.zip

The attached activity waits for onPostResume() to return the result, which makes the counters increment synchronously.
MainActivity.java.txt

This is possibly related to https://github.com/flutter/flutter/issues/11906

similar: #13818 #14619 #14665

@jonahwilliams @najeira I'm guessing the issue I've raised https://github.com/flutter/flutter/issues/16636 is a dupe of this?

@MaikuB that does sound similar to the issue that I fixed - the changes have been rolled into master so you should be able to confirm.  Otherwise I can look at your issue as a follow up.

@jonahwilliams brilliant, confirmed it's fixed on master so i'll close the issue i've raised. thanks


UXsim: Add new get_all_distances method to RouteChoice

get_all_distances is a new helper method that returns distance array in the same format as s.dist. This way you can look up distances the same way as you look up travel time.
Usage:
distance_array = w.ROUTECHOICE.get_all_distances()

some_distance = distance_array[o_node.id][d_node.id]
some_travel_time = uw.ROUTECHOICE.dist[o_node.id][d_node.id]

One consideration could be saving this matrix internally in the RouteChoice class, instead of returning it.
This PR was validated that it outputs the same distances as NetworkX.

what is this for?

ok, so this is not for simulation itself, but for users' convenience.
Then it is better to add a similar method to World class itself, not here. I will do that very soon
Idea behind it: RouteChoice class and node.id are basically encapsulated (or hidden) for basic users. Advanced users may access them to do advanced things, but they are recommended to define their own helper functions outside (downstream?) of uxsim, because they may not be very general. But distance info is general and useful as you pointed out, so it worth adding

Closing this PR as almost equivalent function World.get_shortest_path_distance_between_all_nodes(return_matrix=True) is implemented and tested by https://github.com/toruseo/UXsim/pull/136
For an example, see the added test.
One difference is that the return value in this PR is a tuple of 1d arrays as specified by
distances = *dijkstra(csr_matrix(distances), directed=directed, return_predecessors=False),
return distances

, which seems not common, so World.get_shortest_path_distance_between_all_nodes(return_matrix=True) returns simple 2d array instead.

Thanks!


add docker actions

manual & auto, based on implementation in CHARLIE
see new dockers repo: https://github.com/CCBR/Dockers2

resolved by #31, #33


Symptom Checker- Desktop and Mobile-Can proceed to next question without making selection

For each question on the Symptom checker:
Current Behavior: I can click next arrow without selecting any choices on the page
Expected Behavior: I must choose an option before clicking on the next arrow

@minsu3 are you still working on this issue?

@ralbach this bug is still active. Let me know if any questions. CC @tejasm2

Thanks for heads up will fix tonight
On Thu, Aug 20, 2020 at 6:09 PM Dhruv Patel<EMAIL_ADDRESS>wrote:

@ralbach https://github.com/ralbach this bug is still active. Let me
know if any questions. CC @tejasm2 https://github.com/tejasm2
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FightPandemics/FightPandemics/issues/353#issuecomment-677982322,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWQXQSWCBGLAGQ4AYAH6P3SBXCMLANCNFSM4NBJEMYA
.
--

Rylan Albach
<EMAIL_ADDRESS>


Fix snarky codeowners

We really wanted snarky to be owned by crypto, but it didn't work - it blocks on protocol review.
This is the reason!
How do I know? We had the same problem for snarkyjs when it was a submodule, and https://github.com/MinaProtocol/mina/commit/24b7486559fd6e01c6f5093c9e8903cba85ef3d9 fixed it

!ci-build-me

!ci-build-me


Introduce a simpler API for building kernels

This patch introduces KernelBuilder, which is a tiny wrapper around
SignatureBuilder for building kernels, simplifying the creation of a
kernel, especially when creating the kernel outside the Kernel trait.
In addition, it makes the memory initialization strategy explicit using
an appropriate MemInit enum instead of using naked booleans.

bors r=nicoTolly,andidr


Better callback documentation

The callbacks' arguments should be documented.

Unfortunately the @callback JSDoc tag is not supported in Markdox so the callback documentation will be a little messier.


can we get traffic stat display in client?

The issue tracker is not for questions. Please ask questions at<EMAIL_ADDRESS>how to get traffic stat from local shadowsocks-libev? so we an display on the client

Hi @rchen9012, we are open to implementing this feature once we migrate the clients to a shared Go network stack.
In the meantime, could you (and others who would like this feature) share why it would be useful?

What's the benefit of using Go? It seems like you guys have to rework all the codes again

since the VPS has limited bandwidth, we would like to encourage the fair usage policy.  So if we can show their usage in client they would be aware of their usage.

Now that Data Limits is fairly stable, perhaps this feature request could be picked up as well?

I want to restrict the user with specific range of data per day for that I need to monitor how much he/she has used the internet while connected to VPN.

Hello, I also would like to see that feature implemented because of major number of requests that comes from my family members like "hey, how much GBs are left for me?.... ok, thanks, and for today?.... what about today?..... ok, and now?....."


Update version of Chai.js and is-ip

https://david-dm.org/BigstickCarpet/postman-bdd
Looks like these dependencies are out of date. (I just wasted a day trying to chase down a "bug" that was just I was looking at the update docs while using the outdated lib) Any release dates on when it will have the new version? Are there issues with the new version that other's can help resolve?

I just attempted to build it myself with Chai 4.0.2 I only got one failed test.
✖ should not throw
    -------------------
      operator: throws
      expected: |-
        undefined
      actual: |-
        { actual: '35', expected: undefined, message: 'expected \'35\' to be a number', showDiff: false }
      at: Test.test (/Users/cparrish/Desktop/Sandbox/postman-bdd/test/specs/response/body.spec.js:55:5)

  Failed Tests: There was 1 failure

    XML response body

      ✖ should not throw

  total:     951
  passing:   950
  failing:   1
  duration:  539ms

Chai.js 4.0 just released last week, and it includes many breaking changes, so I haven't had a chance to figure out a clean upgrade path for Postman BDD.  I can't just update the Chai dependency, because that will break many people's existing tests that use Postman BDD.  So I need to think about the best way to expose a backward-compatible Chai 3.x version of Postman BDD and a new Chai 4.x version as well.
In the meantime, you should be able to use the existing version of Postman BDD, as long as you aren't relying on any of the new or changed functionality in Chai 4.0.

@carl-parrish - I've merged your PR and released Postman BDD v5.0.0 with Chai 4.0.2.  I solved the backward-compatibility problem by adding version-specific subfolders under the dist folder (e.g. dist/v3, dist/v4, dist/v5)


compatibility with React v19

Hey there,
any plans on supporting upcoming React v19? It's currently a release candidate, and your library gives several errors when building. For the upgrade guide, see here.
Thanks!

I haven't had a chance to try the RC yet, but I'll look into adding support for it in a future version

All I can ask, much appreciated! 🙏

Will it support React 19? React 19 was released a couple of weeks ago.


Adding the ability to import consumption data from a CSV file

The current Earl is restricted to only reading consumption data from a Cloudwatch. This enables the import from a CSV file too. Handy for when you need to put in metrics from other types of DBs that you are considering moving to DynamoDb

Wrong account :(


file_upload: Remove unusable UI elements if file uploading is disabled.

If MAX_FILE_UPLOAD_SIZE is set to 0, then UI elements like the upload icon in the compose and message edit UI and "Attachments" menu in "/#settings" are not displayed.
A different error message is also displayed if a user tries to drag and drop or paste a file into the compose message box.
Fixes #12152.
GIFs or Screenshots:

Hiding the "upload" icon in the compose and message edit UIs

Hiding the "Attachments" menu in "/#settings"

A special error message if someone tries to paste or drag a file into those widgets UIs.

@rishig Is the text for the error message okay?

That's pretty good. Maybe "File and image uploads have been disabled for this organization."?

That does sound better. I'll make the required change.

@vinitS101 this merge conflicts with another PR that I just merged; can your rebase?

I posted a few quick comments; otherwise, this looks great!

There didn't seem to be any merge conflicts after rebasing. Made the other changes as requested.

I haven't run into a codecov issue before and if I'm reading it correctly, it is because of low test coverage for  static/js/message_edit.js (which definitely seems feasible having looked at frontend_tests/node_tests/message_edit.js). The changes I made, though, were relatively small and I'm not sure if I should try to extensive test coverage here.
Would love to get some guidance here. Thanks! :)

Looks great, merged, thanks @vinitS101!


FAQ: No data is displayed on the list control

Below is the 1-star review on Visual Studio Marketplace with some possible remedies (everything is a guesswork, because the author never did not attempt to help me out):
It doesn't seem to be working in VS2019 (16.4.0). (edit: I can see the Enable Profiling menu). VS complains about missing some DLL (micro-profiler_x86.dll) from the extension and I have to copy it to the target exe folder (edit: later found that I have to start the program without debugging to avoid this problem). Nothing can be shown on the list control.
Kenny

If the frontend appears, but nothing is displayed in the report - the most probable cause is that the configuration different from the profiled one is started (in the Additional Compiler options it may not have /GH /Gh switches on).

Reply from the user: "12/25/2019 Updated: I just tried this on another computer (version 1.5.611.0) and I noticed this profiler seems to be working for release build application only, it can't show the function names on the list control for debug build. By the way, when I have two VS running at the same time and start profiling on one of them, the profiler's tab appears on the other instance of VS."


Update sbt-github-actions to 0.13.0

Updates com.codecommit:sbt-github-actions from 0.9.5 to 0.13.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.codecommit", artifactId = "sbt-github-actions" } ]

labels: sbt-plugin-update, semver-minor

Superseded by #66.


fix missing connect timeout and make tests safer

Fix missing connect timeout and make some tests faster and safer.
Motivation:

During pool implementation/refactoring we lost connection timeout, this PR fixes that
testConnectionFailError is slow, 10 seconds, due to not having a timeout set
testConnectionFailError and testTLSVersionError use dangerous do-try-XCFail-catch pattern

Modifications:

Set connection timeout if specified
Set connect timeout to 150 ms to make testTLSVersionError faster
Use XCTAssertThrowsError in testConnectionFailError and testTLSVersionError
Add general connection timeout test, since testConnectionFailError is NIOTS-specific.

Result:
Fixes an issue where connection timeout configuration was not respected, hopefully makes tests more robust

failed due to #265

@swift-server-bot test this please


stylish-haskell-<IP_ADDRESS> fails (tests)

Issue description
expected: "canDrink mbAge = case mbAge of\n    Just age | age > 18 -> True\n    _                   -> False\n"
 but got: "canDrink mbAge = case mbAge of\n    Just age | age > 18 -> True\n    _        -> False\n"
Language.Haskell.Stylish.Step.Tabs.Tests:
  case 01: [OK]
Language.Haskell.Stylish.Step.TrailingWhitespace.Tests:
  case 01: [OK]
Language.Haskell.Stylish.Step.UnicodeSyntax.Tests:
  case 01: [OK]

         Test Cases   Total
 Passed  62           62
 Failed  1            1
 Total   63           63
Test suite stylish-haskell-tests: FAIL
Test suite logged to:
dist/test/stylish-haskell-<IP_ADDRESS>-stylish-haskell-tests.log
0 of 1 test suites (0 of 1 test cases) passed.
Wrapping './bin/texmfstart' and patching shebang '/bin/sh'
Wrapping './bin/texosquery' and patching shebang '/bin/sh'
builder for '/nix/store/sjasilbpmzvkkzhv5ra262xvjzxl136n-stylish-haskell-<IP_ADDRESS>.drv' failed with exit code 1

Technical details

system: "x86_64-linux"
host os: Linux 4.14.33, NixOS, 18.09pre135256.6c064e6b1f3 (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.0
channels(philipp): ""
channels(root): "nixos-18.09pre138871.af55a0c3002"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Now fixed.
https://hydra.nixos.org/job/nixpkgs/trunk/haskellPackages.stylish-haskell.x86_64-linux


[ENHANCEMENT] Add a predefined prefix to variables

Description
Currently, bargs exports variables UPPER_CASE and lower_case.
I'm using a very strict naming convention, see Writing Bash Scripts Like A Pro - Part 1 - Styling Guide

TypeScopeConventionEnvironmentGlobalMY_VARIABLEGlobalGlobal_MY_VARIABLELocalFunctionmy_variable
Using bargs forces me to redefine variables
# Assuming my_global_var was declared in bargs_vars
# MY_GLOBAL_VAR will be available as an environment variable in my script
# So now I need to do this to define it as a "global var"
_MY_GLOBAL_VAR="${MY_GLOBAL_VAR:-"my-default-value"}"

Possible solution
The file bargs_vars contains metadata options for running bargs, so adding options, such as vars_perfix, won't break anything in bargs as it ignores unknown keys.
I'm thinking of adding four options:

vars_prefix="" - Defaults to "", adds a prefix to all exported variables.

Enables isolating the app from other predefined environment variables
Makes it easier to grep all variables that were exported by bargs with, so I used vars_prefix=MY_APP_PREFIX_ in bargs_vars, I could do the followingenv | grep "MY_APP_PREFIX_"
# results
MY_APP_PREFIX_GOOGLE_CLIENT_ID
MY_APP_PREFIX_GOOGLE_CLIENT_SECRET

vars_uppercase="true" - Defaults to "true" which avoids breaking previous bargs.sh versions. Setting to false will drop the assignment to UPPERCASED env vars.
vars_lowercase="true" - Defaults to "true" which avoids breaking previous bargs.sh versions. Setting to false will drop the assignment to LOWERCASED env vars.
vars_type=environment - Defaults to "true" which avoids breaking previous bargs.sh versions. An alternative value would be global, so the variables are not exported with export _MY_VAR="my value", but only declared as global variables `_MY_VAR="my value"

A concrete example implementing the above solution
bargs_vars
---
name=person_name
default="Willy"
---
name=age
default=99
---
name=bargs
description=bash example.sh -n Willy --gender male -a 99
vars_uppercase=false
vars_lowercase=false
vars_prefix=_APP_
vars_type=global
default=irrelevant
---
---

Should result with the following variables
_APP_PERSON_NAME="Willy"
_APP_AGE="99"

I need to estimate the effort, though I don't think it's a big deal. Would love to get some feed to see if this type of feature will enhance your experience with bargs.

I've realized it's not that important to work on so dropped it


doc: Improved Tetragon metrics guide

doc: Improved Tetragon metrics guide

Thanks a lot for the review, @lambdanis. I addressed it in https://github.com/cilium/tetragon/pull/2840/commits/52267210ef38ee5de3310c545067129fa0ce93cc.

Perfect, thanks for the review, @lambdanis. I just squashed and force-commit'ed them.


Social Platform icons

It would  be great to have social platform icons available that can be used to link to social platforms. E.g. when building a blog or company webpage it would be great to create some nice stylish links to the companies or personal social profiles.

[ ] Facebook
[ ] Twitter
[ ] LinkedIn
[ ] Keybase
[ ] Instagram
[ ] Github
[ ] Youtube
[ ] StackOverflow
[ ] Reddit
[ ] 500px
[ ] Flickr
[ ] Unsplash
[ ] ...

That would prevent me from including font-awesome so I can stick completely with the tailwind stack.

+1

+1

+1 as well ;)


Change developer bio text

Pull request checklist

Issue-560

[x] My code contains tests covering the code I modified
[x] I linted and tested the project with bin/check
[x] I added significant changes and product updates to the changelog

@joemasilotti Please review

Thanks for the contribution, @kaushikhande! Where did you source the non-English translations?

Thanks for the contribution, @kaushikhande! Where did you source the non-English translations?

Google translate

Got it. Can you please remove those translations then? I'd rather have no translation at all vs. something from Google Translate right now.

Got it. Can you please remove those translations then? I'd rather have no translation at all vs. something from Google Translate right now.

Okay. Removing

@joemasilotti Please review. Translation removed

Awesome – thanks for your contribution!


[Demo site]When I use Edge's webpage translation, the webpage will use the translated text.

Describe the bug
When Edge translates the word 'true' in this webpage to '真' in Chinese, it will use this text.
To Reproduce
Steps to reproduce the behavior:

Go to the website.
Click on 'Translate' in Edge.
You can see this:

The generated website address will look like this (This is just a part) :

https://readme-typing-svg.demolab.com? (...) &center=%E7%9C%9F&vCenter=%E5%81%87&repeat=%E7%9C%9F&random=%E7%9C%9F& (...)

Smartphone:

Device: Mi 10 Lite Zoom
OS: MIUI 13.0.7
Edge 128.0.2739.82

Possible Workarounds:
Check if the translation can be manually adjusted or disabled for specific terms.


Merged main dev 1.31

dev branch sync

@salaxander We do occasionally see Netlify build failures. Could you try to trigger the build by force-pushing empty commit using git commit --amend --no-edit && git push --force-with-lease

/approve


Rename logstash formatter to comply with golint.

Before: logstash.LogstashFormatter, after: logstash.Formatter.
Complies with Golint which complains for now and makes it more readable.

I've asked @stevvooe if we can move it to his personal Github, then he can make this decision. Cheers!

LGTM
This is a breaking change, but easy to fix.

@stevvooe do you mind moving this to your own Github? This is the only community hook left in Logrus core. All others have been moved to be maintained by the actual users of the hooks.

@Sirupsen I'm still planning on taking up some work in logging but have found myself busy with other projects. You may want to find another maintainer for the logstash plugin, as I haven't had time to catch up with the API changes.

@stevvooe yeah, I haven't had time either to implement what we talked about at DockerCon unfortunately :(

@Sirupsen Some day...


Unclear description

[Enter feedback here]
"SR-IOV allows a device, such as a network adapter, to separate access to 'its' resources among various PCIe hardware functions. "
What does "its" means here, network adapter or SR-IOV ?

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

ID: 19b1f1a3-dd47-4d16-c9db-f52009fcf8ff
Version Independent ID: 3e95c1d5-b5e7-2928-7e6a-3e725e18b061
Content: Overview of Single Root I/O Virtualization (SR-IOV) - Windows drivers
Content Source: windows-driver-docs-pr/network/overview-of-single-root-i-o-virtualization--sr-iov-.md
Product: windows-hardware
Technology: network
GitHub Login: @aviviano
Microsoft Alias: amvivian

Hi @jasonchou519,
Thanks for opening this issue. "Its" refers to "a device, such as a network adapter".
From https://learn.microsoft.com/en-us/windows-hardware/drivers/network/single-root-i-o-virtualization--sr-iov-:
The SR-IOV specification from PCI-SIG defines the extensions to the PCI Express (PCIe) specification suite that enable multiple virtual machines (VMs) to share the same PCIe physical hardware resources. This section describes the NDIS SR-IOV interface and describes the techniques for writing an NDIS miniport driver for an SR-IOV capable network adapter that implements the PCIe SR-IOV specification.
If you have further questions or comments, please feel free to open a new GitHub issue.
Thanks again.


Add support for serialization of hashes

Currently LWS only supports the serialization of objects. Often it is easier to use hashes though, especially for small data structures.
For example, we'd like to do the following:
countries = ISO3166::Country.all.map do |country|
  {
    code: country.alpha2,
    label: country.translation(I18n.locale)
  }
end

render json: countries, serializer: CountrySerializer

class CountrySerializer < ApplicationSerializer
  attribute :code
  attribute :label
end

If you currently try to serialize a hash:
NoMethodError:
       undefined method `code' for #<Hash:0x00005578e8dd3450>

This PR solves the above error. So CountrySerializer.new({code: 'DE', label: 'Germany'}) then works fine.

Nice!


Restart ECS Service Command

Description
Script to restart an ECS Service.  This is a graceful redeploy, so will take a few minutes to drain and swap out the running containers.  Could add an optional hard restart option, but would take more time to find the right aws api calls.
Reviewer Notes
N.A.
Code Review Verification Steps
N.A.
References
https://docs.aws.amazon.com/cli/latest/reference/ecs/update-service.html
Screenshots
N.A.

@dynamike updated with waiter


add Keil.gitignore

The Keil.gitignore is fit for being used on Keil Project which files need to be ignored.

Closing due to inactivity and pending feedback.


Fix(web-react): Replacing backticks in storybook #DS-468

Replacing backticks in Storybook DS-468
What was done

Replace backquotes with regular ones

Please, consider using different commit prefix. You are not fixing existing bug so Style or Refactor would be better, I think.


Support alternate voice aliases

Users sometimes want to use a more expensive voice for certain redeems, but have other redeems like RememberThis and RememberThisAboutMe use a cheaper or free voice.
So I think I need to create a main setting called "Alternate Voice Alias" and then write a method  called UseAlternateVoiceAlias(), this would then set an argument on run called alternateVoiceAlias. If the method for Speak(), RememberThis(), or AskGPT() sees the argument alternateVoiceAlias set, it would then use the other voice alias. You would then add the UseAlternateVoiceAlias method right above any action that speaks in the sub-actions. If this method runs at all, it would use the alternate voice, if the argument is not there, it would use the regular voice. This doesn't handle every edge case, but I made the design choice to focus on ease of use over flexibility here. It's better to have the single place to set voice aliases, over having them on each and every action for users to have to update. I'm open to feedback here though.

After review, it was found it was best to implement a Voice Alias option in the menu. Speaker.bot can have a default voice alias, and you can simply assign a different voice alias in settings to the bot. This allows for alerts to have the cheaper voices, while allowing the PNGTuber to use a different voice for GPT redeems. This is enough functionality for the moment; so I am closing this as a won't fix.


use the loaded object to check for hide stats

use the loaded object to check for hide stats better than running another query

Hello! Why did you close the pull request? There was some problem?

i created a new one because tests were failing in this one


Bumbed some dependencies, works on node 17.3

fixes https://github.com/bem/html-differ/issues/173

This also fixes two high security vulnerabilities that are found in 'diff' package
@tadatuta
Any chance this can get in?


feat(client): send logs to syslog

closes #2255
Describe your changes
Issue ticket number and link
Checklist

[ ] Is it a bug fix
[ ] Is a typo/documentation fix
[x] Is a feature enhancement
[ ] It is a refactor
[ ] Created tests that fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

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

hey, go is not really "my thing" yet but I think this changes should do the trick. any suggestions from reviewers more than welcome

thanks for merging @mlsmaycon ! here is a follow-up PR that fixes some sonar qube issues https://github.com/netbirdio/netbird/pull/2276


add url encoding parameter

To prevent wrong encoding, when default coding is non-utf8

Hi @liziwl Thanks for the PR. Can you please update the PR by removing the formatting changes i.e. new line and spaces.

Ok, I have removed the formatting.


Update .dockerignore (#180)

This PR updates the repo's .dockerignore file to ensure unnecessary and sensitive files are not included in Docker images built locally. Changes include use of the **/ Docker-specific syntax to properly find files with certain patterns in any directory; widening of the env pattern to capture HJSON configuration files; and the removal of the .vscode directory from the list (as it might be needed in our deployment environments). The PR aims to resolve issue #180.

Thanks for the reviews! I'm merging now.


Migrate from Hallo.js to Draftail

[Short description explaining the high-level reason for the pull request]
Additions

Removals

Changes

Testing

Screenshots
Notes

Todos

Checklist

[ ] PR has an informative and human-readable title
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the CFPB development guidelines
[ ] Passes all existing automated tests
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Visually tested in supported browsers and devices (see checklist below :point_down:)
[ ] Project documentation has been updated
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist
Browsers

[ ] Chrome on desktop
[ ] Firefox
[ ] Safari on macOS
[ ] Edge
[ ] Internet Explorer 9, 10, and 11
[ ] Safari on iOS
[ ] Chrome on Android

Accessibility

[ ] Keyboard friendly
[ ] Screen reader friendly

Other

[ ] Is useable without CSS
[ ] Is useable without JS
[ ] Flexible from small to large screens
[ ] No linting errors or warnings
[ ] JavaScript tests are passing

How about we keep the scope of this PR constrained to the final removal of the Tip and HTML plugins?

@Scotchester should this change have it's own PR?
I'm thinking that as long as there's no change in functionality, we can keep the changes in one PR. So far this makes no user-facing changes.

@Scotchester should this change have it's own PR?

That and I was assuming that that same piece, the table RTE, would need more changes to get fixed.

I'm thinking that as long as there's no change in functionality, we can keep the changes in one PR. So far this makes no user-facing changes.

But I can get behind this principle.

Yeah. Part of the reason why this is a draft. I'm not sure what it'll take :)


After restart SQL Server when Shutdown error message occured connection stay in CLOSE_WAIT in alipne 3.9 docker container

I have problem with App that is hosted on official asp-dotnet-core-alipne container and SQL Server on other server. After restart server on wich we have MS SQL 2014, conainers gets high CPU and some internal thread hangs. Helps only restart of container. We investigated that there are some tcp connection in state CLOSE_WAIT when this situations occured. Some information about app and server:
Details about app:

.net core 2.2 (C#)
based on official alpine container (mcr.net.core.asp:2.2.5-alpine3.9)
used Dapper as well as ADO.NET
used Async method with async/await
used Quartz.NET for job scheaduling
used System.Data.SqlClient 4.6.1 download by Nuget

Details about Docker

hosted on Centos 7
used Docker 19.03

Details about SQL Server:

MS SQL 2014 Standard x64
Windows Server 2012R2
hosted on Virtual Machine

Detalied problem description:
Container with app works 24/7 on linux server (Centos 7) on wich docker is installed. On same server there is virtual machine with Windows Server and MS SQL Server 2014 installed on it. App work fine if there some network issue and other stuff but after restart this server i get error:
[19-08-09 04:15:44.15 ERR -- SSI.Pojazd 0216NPIK] Job SSI.Pojazd.retry_Sms_RetryJob`1 threw an unhandled Exception:
System.Data.SqlClient.SqlException (0x80131904): SHUTDOWN is in progress.
Login failed for user 'XXXX'.
Cannot continue the execution because the session is in the kill state.
A severe error occurred on the current command.  The results, if any, should be discarded.
   at System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   at System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   at System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   at System.Data.SqlClient.TdsParser.Run(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj)
   at System.Data.SqlClient.TdsParser.TdsExecuteTransactionManagerRequest(Byte[] buffer, TransactionManagerRequestType request, String transactionName, TransactionManagerIsolationLevel isoLevel, Int32 timeout, SqlInternalTransaction transaction, TdsParserStateObject stateObj, Boolean isDelegateControlRequest)
   at System.Data.SqlClient.SqlInternalConnectionTds.ExecuteTransactionYukon(TransactionRequest transactionRequest, String transactionName, IsolationLevel iso, SqlInternalTransaction internalTransaction, Boolean isDelegateControlRequest)
   at System.Data.SqlClient.SqlInternalConnection.BeginSqlTransaction(IsolationLevel iso, String transactionName, Boolean shouldReconnect)
   at System.Data.SqlClient.SqlConnection.BeginTransaction(IsolationLevel iso, String transactionName)
   at Transport.Core.Abstractions.Database.TransportStoredProcedureWithResult`2.Execute(TInput input)
   at Transport.Jobs.RetriesJobs.RetryJob`1.Execute(IJobExecutionContext context)
   at Quartz.Core.JobRunShell.Run(CancellationToken cancellationToken)
ClientConnectionId:1d7f1d2d-f80b-42cf-906f-f0ee57b14f59
Error Number:6005,State:1,Class:14

After this error everything start's to behave wierd:

CPU of container rise from 0.9 CPU to 100 CPU and more
dotnet process consume CPU (on hosting linux processes ) rise from 1-2 to 12-14 cores (defends of how many apps are working)
HealthCheck sometimes hangs -> no anwser from server when used simple CURL query
there are CLOSE_WAIT tcp connection to SQL

I think problem is created by this CLOSE_WAIT connection. But I don't have idea how to closed them.
TCP connection log:
tcp        0      0 <IP_ADDRESS>:44920        <IP_ADDRESS>:5672        ESTABLISHED
tcp        0      0 <IP_ADDRESS>:47488        <IP_ADDRESS>:1433       ESTABLISHED
tcp        0      0 <IP_ADDRESS>:47246        <IP_ADDRESS>:1433       ESTABLISHED
tcp        0      0 <IP_ADDRESS>:46785        <IP_ADDRESS>:5672        ESTABLISHED
tcp        0      0 <IP_ADDRESS>:45556        <IP_ADDRESS>:1433       CLOSE_WAIT
tcp        0      0 <IP_ADDRESS>:45520        <IP_ADDRESS>:1433       CLOSE_WAIT

###What i tried to do:
Use CancellationToken as some way of cancel DbOperation in SqlClient.dll
[Dapper code]
public async Task Execute(TInput input, IScope scope,CancellationToken token)
{
    await scope.GetConnection().ExecuteAsync(new CommandDefinition(GetStoredProcedureName(), GetParameters(input), scope.GetTransaction(),
    commandType: CommandType.StoredProcedure,cancellationToken:token)).ConfigureAwait(false);
}

[ADO.NET CODE]
public async Task CheckAsync(string connectionString, int timeout, CancellationToken cancellationToken)
{
    try
    {
        SqlConnectionStringBuilder connectionStringBuilder =
            new SqlConnectionStringBuilder(connectionString) {ConnectTimeout = 2};

        using (var conn = new SqlConnection(connectionStringBuilder.ToString()))
        {
            try
            {
                if (cancellationToken.IsCancellationRequested)
                    cancellationToken.ThrowIfCancellationRequested();
                await conn.OpenAsync(cancellationToken).ConfigureAwait(false);
                using (var cmd = conn.CreateCommand())
                {
                    cmd.CommandText = "Select 1";
                    cmd.CommandTimeout = timeout;
                    cancellationToken.ThrowIfCancellationRequested();
                    await cmd.ExecuteNonQueryAsync(cancellationToken).ConfigureAwait(false);
                }
            }
            catch (TaskCanceledException tce)
            {
                TransportLogger.Log.Debug("Task cancelled " + tce.Message + tce.StackTrace + " " + (tce.InnerException == null ? string.Empty : tce.InnerException.Message));
                throw;
            }
            finally
            {
                if (conn.State == ConnectionState.Open) conn.Close();
            }

        }
    }
    catch (Exception ex)
    {
        TransportLogger.Log.Error(ex, "Cannot check db assebility");
        throw;
    }
}

Result -> i get information about cancellation but connection stay in Close_wait and CPU is rising to the roof.
Clear pool using ClearPool and ClearAllPool method
public class CheckDatabaseIsAccessible : ICheckDatabaseIsAccessible
{
    public async Task CheckAsync(string connectionString, int timeout, CancellationToken cancellationToken)
    {
        try
        {
            SqlConnectionStringBuilder connectionStringBuilder =
                new SqlConnectionStringBuilder(connectionString) {ConnectTimeout = 2};

            using (var conn = new SqlConnection(connectionStringBuilder.ToString()))
            {
                try
                {
                    if (cancellationToken.IsCancellationRequested)
                        cancellationToken.ThrowIfCancellationRequested();
                    await conn.OpenAsync(cancellationToken).ConfigureAwait(false);
                    using (var cmd = conn.CreateCommand())
                    {
                        cmd.CommandText = "Select 1";
                        cmd.CommandTimeout = timeout;
                        cancellationToken.ThrowIfCancellationRequested();
                        await cmd.ExecuteNonQueryAsync(cancellationToken).ConfigureAwait(false);
                    }
                }
                catch (TaskCanceledException tce)
                {
                    TransportLogger.Log.Debug("Task cancelled " + tce.Message + tce.StackTrace + " " + (tce.InnerException == null ? string.Empty : tce.InnerException.Message));
                    throw;
                }
                finally
                {
                    if (conn.State == ConnectionState.Open) conn.Close();
                }

            }
        }
        catch (Exception ex)
        {
            TransportLogger.Log.Error(ex, "Cannot check db assebility");
            throw;
        }
    }

    public void Check(string connectionString, int timeout)
    {
        try
        {
            SqlConnectionStringBuilder connectionStringBuilder =
                new SqlConnectionStringBuilder(connectionString) {ConnectTimeout = 2};

            using (var conn = new SqlConnection(connectionStringBuilder.ToString()))
            {
                try
                {
                    TransportLogger.Log.Debug("Open connection");
                    conn.Open();
                    TransportLogger.Log.Debug("connection open");
                    using (var cmd = conn.CreateCommand())
                    {
                        cmd.CommandText = "Select 1";
                        cmd.CommandTimeout = timeout;
                        TransportLogger.Log.Debug("Command executing...");
                        cmd.ExecuteNonQuery();
                        TransportLogger.Log.Debug("Command executed");
                    }
                }
                catch (TaskCanceledException tce)
                {
                    TransportLogger.Log.Debug("Task cancelled " + tce.Message + tce.StackTrace + " " + (tce.InnerException == null ? string.Empty : tce.InnerException.Message));
                    throw;
                }
                finally
                {
                    if (conn.State == ConnectionState.Open) conn.Close();
                }

            }
        }
        catch (Exception ex)
        {
            TransportLogger.Log.Error(ex, "Cannot check db assebility");
            throw;
        }
    }
}

Pool is cleared but nothing happend, problem still exists:
  [19-08-09 04:16:37.16 DBG -- SSI.Pojazd 0216NPIK] Task cancelled A task was canceled.   at Transport.Core.Abstractions.Database.CheckDatabaseIsAccessible.CheckAsync(String connectionString, Int32 timeout, CancellationToken cancellationToken)
  [19-08-09 04:16:37.16 INF -- SSI.Pojazd 0216NPIK] Try to clear pool
  [19-08-09 04:16:37.16 INF -- SSI.Pojazd 0216NPIK] Pool cleared

I try to reproduce this on test env but i failed. We have aproximatly 400 servers and this situation is quite common (i think is about time of server is shutdown). I observe this situation occured only when

System.Data.SqlClient.SqlException (0x80131904): SHUTDOWN is in progress.
is thrown

Mayby someone had this problem and she/he know solution? Any help will be apprecieated.

Hi @szymo0
We're investigating the behavior, will update you soon!

Hi there, is there any progress in this case?

@szymo0 Could you provide the netstat from the server as well?

@szymo0 I was doing some more searching and found that your issue looks very similar to this one, which makes me wonder if it's a .net core issue in System.Net.Sockets. Would you mind trying on .net core 2.1 and 3.0 to see if that can solve your issue?

Hi @szymo0. any update on this issue? Were you able to reproduce the problem after updating to Microsoft.Data.SqlClient v2.0.0 and dotnet 3.1?

Close this due to no activities on this.

Experiencing a similar issue with .NET Core 3.1 and EF Core.
Application runs inside a Container in a Kubernetes Cluster and connects via EF Core to a MS SQL Server outside the cluster.
Occasionally when there are backups/updates/restarts of the SQL Server, the application is unable to query anything from the SQL Server with the same exception and stacktrace as @szymo0  posted.
CPU usage goes up to 100%, all EF Core Queries fail with the following stacktrace until I restart the whole application:
Microsoft.Data.SqlClient.SqlException (0x80131904):
System.Data.SqlClient.SqlException (0x80131904): SHUTDOWN is in progress.
Login failed for user 'XXXX'.
Cannot continue the execution because the session is in the kill state.
A severe error occurred on the current command.  The results, if any, should be discarded.
   at Microsoft.Data.SqlClient.SqlCommand.<>c.<ExecuteDbDataReaderAsync>b__164_0(Task`1 result)
   at System.Threading.Tasks.ContinuationResultTaskFromResultTask`2.InnerInvoke()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread)
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.EntityFrameworkCore.Storage.RelationalCommand.ExecuteReaderAsync(RelationalCommandParameterObject parameterObject, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Storage.RelationalCommand.ExecuteReaderAsync(RelationalCommandParameterObject parameterObject, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Storage.RelationalCommand.ExecuteReaderAsync(RelationalCommandParameterObject parameterObject, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.AsyncEnumerator.InitializeReaderAsync(DbContext _, Boolean result, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.SqlServer.Storage.Internal.SqlServerExecutionStrategy.ExecuteAsync[TState,TResult](TState state, Func`4 operation, Func`4 verifySucceeded, CancellationToken cancellationToken)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.AsyncEnumerator.MoveNextAsync() ClientConnectionId:****************** Error Number:6005,State:1,Class:14

.NET Core 3.1
EntityFrameworkCore 3.1.9
Docker Base  image: mcr.microsoft.com/dotnet/core/aspnet:3.1

@Michriko @vasicvuk
Could you please try our latest released versions (v2.1.3 and v3.0.0-preview3) to validate this still occurs?
https://www.nuget.org/packages/Microsoft.Data.SqlClient

@cheenamalhotra can you confirm if this issue fixes by upgrading to Microsoft.Data.SqlClient? We are facing the same issue and we use an entity framework 3.1.13 which references Microsoft.Data.SqlClient 1.1.2) with .NET Core 3.1. I am not able to simulate in a lower environment to confirm it and would be helpful if someone has already confirmed this fix.


fix: Enable argocd to manage kube-prometheus-stack

What does this PR do?
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Motivation

I had to manually set the kube-prometheus-stack enable variable in my ArgoCD App of Apps config. This was defaulting to false and not being updated since the value wasn't past.

Resolves #

More

[YES] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[N/A ] Yes, I have added a new example under examples to support my PR
[NO ] Yes, I have created another PR for add-ons under add-ons repo (if applicable)
[ YES] Yes, I have updated the docs for this feature
[NO ] Yes, I ran pre-commit run -a with this PR

Note: Not all the PRs require a new example and/or doc page. In general:

Use an existing example when possible to demonstrate a new addons usage
A new docs page under docs/add-ons/* is required for new a new addon

For Moderators

[YES ] E2E Test successfully complete before merge?

Additional Notes
--- PASS: TestEksBlueprintsE2E (0.00s)
--- PASS: TestEksBlueprintsE2E/TF_PLAN_VALIDATION (31.20s)
--- PASS: TestEksBlueprintsE2E/TF_OUTPUTS_VALIDATION (1.86s)
--- PASS: TestEksBlueprintsE2E/TF_OUTPUTS_VALIDATION/vpc_cidr (0.39s)
--- PASS: TestEksBlueprintsE2E/TF_OUTPUTS_VALIDATION/vpc_private_subnet_cidr (0.37s)
--- PASS: TestEksBlueprintsE2E/TF_OUTPUTS_VALIDATION/vpc_public_subnet_cidr (0.37s)
--- PASS: TestEksBlueprintsE2E/TF_OUTPUTS_VALIDATION/eks_cluster_id (0.37s)
--- PASS: TestEksBlueprintsE2E/TF_OUTPUTS_VALIDATION/eks_managed_nodegroup_status (0.37s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION (4.52s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/MATCH_EKS_CLUSTER_NAME (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/MATCH_TOTAL_EKS_WORKER_NODES (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION (0.54s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION/MATCH_REPLICAS_VS_READY-REPLICAS/aws-load-balancer-controller (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION/UNAVAILABLE_REPLICAS/aws-load-balancer-controller (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION/MATCH_REPLICAS_VS_READY-REPLICAS/cluster-autoscaler-aws-cluster-autoscaler (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION/UNAVAILABLE_REPLICAS/cluster-autoscaler-aws-cluster-autoscaler (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION/MATCH_REPLICAS_VS_READY-REPLICAS/coredns (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION/UNAVAILABLE_REPLICAS/coredns (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION/MATCH_REPLICAS_VS_READY-REPLICAS/metrics-server (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DEPLOYMENTS_VALIDATION/UNAVAILABLE_REPLICAS/metrics-server (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DAEMONSETS_VALIDATION (0.39s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DAEMONSETS_VALIDATION/MATCH_DESIRED_VS_CURRENT_PODS/aws-node (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DAEMONSETS_VALIDATION/UNAVAILABLE_REPLICAS/aws-node (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DAEMONSETS_VALIDATION/MATCH_DESIRED_VS_CURRENT_PODS/kube-proxy (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DAEMONSETS_VALIDATION/UNAVAILABLE_REPLICAS/kube-proxy (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DAEMONSETS_VALIDATION/MATCH_DESIRED_VS_CURRENT_PODS/aws-cloudwatch-metrics (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_DAEMONSETS_VALIDATION/UNAVAILABLE_REPLICAS/aws-cloudwatch-metrics (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_SERVICES_VALIDATION (0.49s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_SERVICES_VALIDATION/SERVICE_STATUS/cluster-autoscaler-aws-cluster-autoscaler (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_SERVICES_VALIDATION/SERVICE_STATUS/kube-dns (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_SERVICES_VALIDATION/SERVICE_STATUS/kubernetes (0.00s)
--- PASS: TestEksBlueprintsE2E/EKS_ADDON_VALIDATION/EKS_SERVICES_VALIDATION/SERVICE_STATUS/metrics-server (0.00s)
--- PASS: TestEksBlueprintsE2E/eks-cluster-with-new-vpc (2950.38s)
PASS
ok  	github.com/aws-ia/terraform-aws-eks-blueprints/src	2950.770s

hey @jaydeland , I believe additional changes are required for this to work with our argo approach.
Please take a look at https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/modules/kubernetes-addons/locals.tf#L8-L54 , we should also add a line there and I don't see any PRs in https://github.com/aws-samples/eks-blueprints-add-ons regards this, were you using this repo or your own version for testing?
The testing that you provided if for the E2E, can you please share details of your e2e testing showing that kube-prometheus-stack is enabled and deployed via argo? a picture of how argo looks like will be good enough.

https://github.com/aws-samples/eks-blueprints-add-ons

@Zvikan - I have updated the locals.tf. Also the samples repo already has kube_prometheus_stack added: https://github.com/aws-samples/eks-blueprints-add-ons/blob/ee022fd998bb8faf035c3798729cc3758e63d9e5/chart/values.yaml#L218
I am using the samples repo as fork and had to hard code the enable variable to true since it wasn't being passed by blueprints.

@Zvikan - any update on this PR?

Think this breaks the module, unfortunately. I'm getting the following error:
│ Error: Invalid index
│
│   on .terraform/modules/eks_blueprints_kubernetes_addons/modules/kubernetes-addons/locals.tf line 36, in locals:
│   36:     kubePrometheusStack       = var.enable_kube_prometheus_stack ? module.kube_prometheus_stack[0].argocd_gitops_config : null
│     ├────────────────
│     │ module.kube_prometheus_stack is empty tuple
│
│ The given key does not identify an element in this collection value: the collection has no elements.

I'm yet to absorb the context of this PR so sorry if I've jumped the gun here :+1:

I see this adds a parameter that doesn't really affect us. I already have kube_prometheus_stack enabled as an addon and this breaks terraform apply.
I tried removing the new kube_prometheus_stack line from the file, but that makes terraform want to destroy the resource :man_shrugging:
Any ideas how to work around this or fix?

thank you for reporting this @sondrelg ,taking a look and fixing asap.

@sondrelg @jaydeland I was trying to deploy kube stack prometheus but currently facing an issue where nothing is being deployed and in argo I can see the server could not find the requested resource , I need to spend more time checking why but previous experience tells me it might be related to CRDs.
Did any of you faced this issue before?

Thank you @Zvikan !

Can confirm that after installing the CRDs manually (kubectl apply...) and then re-syncing the application in argo UI, everything is working properly.
Need to understand why Argo is not installing the addon CRDs first.

I am currently getting this error, but I haven't switched to using Blueprints to enable kube-prometheus-stack. My app of apps has the enabled flag set to true.

Was the fix merged or is it still branched? I'd be happy to test on my repo :slightly_smiling_face:

@sondrelg I believe you can follow this: https://github.com/argoproj/argo-helm/tree/main/charts/argo-cd#custom-resource-definitions

Was the fix merged or is it still branched? I'd be happy to test on my repo 🙂

I dont have a fix for the CRDs as of now besides suggesting. to install the CRDs manually before argo, the fix for the object argocd_gitops_config which was missing is part of https://github.com/aws-ia/terraform-aws-eks-blueprints/pull/1384
I'll update once I better understand how to to make the CRDs be installed by Argo first.

@Zvikan - Reinstalling the CRDs did not fix the problem for me ...

Quick update - https://github.com/aws-ia/terraform-aws-eks-blueprints/pull/1384 merged, you can pull and test it again and the failure with argocd_gitops_config object doesnt exists is fixed now.
As for the CRDs issue with Argo - I found this https://github.com/aws-samples/eks-blueprints-add-ons/issues/97 , same issue I was explaining, answer is the pretty much the same, either CRD use replace or install them manually and then re-sync.
What is the issue you are facing @jaydeland ? please provide screenshot and details so I can better help you.

I have upgrade Argo Helm to v5.19.8 (ArgoCD 2.5.8) and reinstalled the CRDS:
kubectl apply -k "https://github.com/argoproj/argo-cd/manifests/crds?ref=v2.5.8"
customresourcedefinition.apiextensions.k8s.io/applications.argoproj.io unchanged
customresourcedefinition.apiextensions.k8s.io/applicationsets.argoproj.io unchanged
customresourcedefinition.apiextensions.k8s.io/appprojects.argoproj.io unchanged

Still getting:
one or more synchronization tasks are not valid (retried 1 times).

Oh - looks like its the kube-prometheus-stack CRDs

Yes, sorry if I wasn't clear, when I was saying the CRDs, I was talking about this specific addon - kube-prometheus-stack => https://github.com/prometheus-community/helm-charts/tree/main/charts/kube-prometheus-stack#from-43x-to-44x

I fixed this by adding the details from this post:
https://blog.ediri.io/kube-prometheus-stack-and-argocd-23-how-to-remove-a-workaround
To the eks_blueprints_addons/chart/templates/kube-prometheus-stck.yaml file
{{- if and (.Values.kubePrometheusStack) (.Values.kubePrometheusStack.enable) -}}
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: kube-prometheus-stack
  namespace: argocd
  finalizers:
  - resources-finalizer.argocd.argoproj.io
spec:
  project: default
  source:
    repoURL: {{ .Values.repoUrl }}
    path: add-ons/kube-prometheus-stack
    targetRevision: {{ .Values.targetRevision }}
    helm:
      skipCrds: true
      values: |
        kube-prometheus-stack:
          prometheus:
            prometheusSpec:
              podMonitorSelectorNilUsesHelmValues: false
              serviceMonitorSelectorNilUsesHelmValues: false
          grafana:
            podLabels:
              sidecar.istio.io/inject: true
        {{- toYaml .Values.kubePrometheusStack | nindent 10 }}
  destination:
    server: https://kubernetes.default.svc
    namespace: kube-prometheus-stack
  syncPolicy:
    automated:
      prune: true
      selfHeal: true
    syncOptions:
    - CreateNamespace=true
    retry:
      limit: 1
      backoff:
        duration: 5s
        factor: 2
        maxDuration: 1m
---
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: kube-prometheus-stack-crds
  namespace: argocd
  finalizers:
  - resources-finalizer.argocd.argoproj.io
  annotations:
    argocd.argoproj.io/sync-wave: "2"
spec:
  destination:
    name: in-cluster
    namespace: kube-prometheus-stack
  project: default
  source:
    repoURL: https://github.com/prometheus-community/helm-charts.git
    path: charts/kube-prometheus-stack/crds/
    targetRevision: kube-prometheus-stack-44.3.0
    directory:
      recurse: true
  syncPolicy:
    syncOptions:
      - CreateNamespace=true
      - Replace=true
    automated:
      prune: true
      selfHeal: true
{{- end -}}

I fixed this by adding the details from this post: https://blog.ediri.io/kube-prometheus-stack-and-argocd-23-how-to-remove-a-workaround
To the eks_blueprints_addons/chart/templates/kube-prometheus-stck.yaml file
{{- if and (.Values.kubePrometheusStack) (.Values.kubePrometheusStack.enable) -}}
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: kube-prometheus-stack
  namespace: argocd
  finalizers:
  - resources-finalizer.argocd.argoproj.io
spec:
  project: default
  source:
    repoURL: {{ .Values.repoUrl }}
    path: add-ons/kube-prometheus-stack
    targetRevision: {{ .Values.targetRevision }}
    helm:
      skipCrds: true
      values: |
        kube-prometheus-stack:
          prometheus:
            prometheusSpec:
              podMonitorSelectorNilUsesHelmValues: false
              serviceMonitorSelectorNilUsesHelmValues: false
          grafana:
            podLabels:
              sidecar.istio.io/inject: true
        {{- toYaml .Values.kubePrometheusStack | nindent 10 }}
  destination:
    server: https://kubernetes.default.svc
    namespace: kube-prometheus-stack
  syncPolicy:
    automated:
      prune: true
      selfHeal: true
    syncOptions:
    - CreateNamespace=true
    retry:
      limit: 1
      backoff:
        duration: 5s
        factor: 2
        maxDuration: 1m
---
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: kube-prometheus-stack-crds
  namespace: argocd
  finalizers:
  - resources-finalizer.argocd.argoproj.io
  annotations:
    argocd.argoproj.io/sync-wave: "2"
spec:
  destination:
    name: in-cluster
    namespace: kube-prometheus-stack
  project: default
  source:
    repoURL: https://github.com/prometheus-community/helm-charts.git
    path: charts/kube-prometheus-stack/crds/
    targetRevision: kube-prometheus-stack-44.3.0
    directory:
      recurse: true
  syncPolicy:
    syncOptions:
      - CreateNamespace=true
      - Replace=true
    automated:
      prune: true
      selfHeal: true
{{- end -}}

This looks promising, I am going to give this a try and if it works well I'll create a PR.
Thank you for sharing this @jaydeland !


[ENG-15812] Updated config properties to allign with older versions of hudi

Task linked: ENG-15812 Add lakeview sync tool in lakeview repo


[lecture10[0619] Batch Normalization

Hello,
I have a question regarding batch normalization(lec 10_2, 33p):

The slide mentions that in some cases, we cannot increase the batch size due to huge computational requirements. How does batch normalization perform with very small batch sizes (e.g., 1 or 2)? Can other normalization techniques, such as group normalization, instance normalization, or layer normalization, mitigate any negative impacts in such scenarios?
Thank you for your time.

@jwemilee Thank you for the comment!
As you mentioned, there are scenarios in training generative models where we cannot use a batch size larger than 1 or 2. In such cases, we consider instance normalization, layer normalization, or group normalization, as they do not calculate the batch-wise mean, so the batch size does not affect performance.
However, if we have sufficient GPU memory, it is desirable to increase the batch size, and using batch normalization is a widely adopted approach. Of course, there are exceptions: for instance, transformers use layer normalization, and the use of group normalization with large-scale ResNet architectures has shown promising results compared to batch normalization.


Timeout when installing Fedora CoreOS on iPXE

Bug
Environment

Platform: bare-metal
OS: fedora-coreos
Release: 1.17.4
Terraform:

Terraform v0.12.24

provider.ct v0.4.0
provider.local v1.4.0
provider.matchbox v0.3.0
provider.null v2.1.2
provider.template v2.1.2
provider.tls v2.1.1

Problem
iPXE receives a timeout while attempting to download the Fedora CoreOS image.
Interestingly, my web browser has no problem downloading the same URL.
I compiled iPXE from source, and the only hint I have that it isn't working is that it hangs while (seemingly) reporting the certificate authority. It doesn't actually start downloading (like a percentage counter). Flatcar over HTTPS does not have the same issue.
Desired Behavior
iPXE Fedora CoreOS
Steps to Reproduce

Compile iPXE from latest source, enabling HTTPS, install on router, confirm booting from said iPXE image
terraform apply as per the docs for Fedora CoreOS on bare metal
Boot any machine, see it attempt to download correctly, and time out.

Reports for iPXE fetch problems can go to the upstream OS vendors.
At various points I've had issues with CoreOS Container Linux, Flatcar Linux, and Fedora CoreOS. They've all turned out to be either ephemeral upstream hosting issues or related to iPXE firmware -  test with debug enabled, check if its their root OSCP cert expiring again, etc.

I provisioned a Fedora CoreOS stable 31.20200310.3.0 cluster in the course of upgrading all clusters to Kubernetes v1.18.0. So I think the issue will turn out to be in your iPXE setup and I'll close this out.
If you can test out iPXE with debug still have issues, probably best to note the iPXE sha, compilation options, and Fedora CoreOS version to report to https://github.com/coreos/fedora-coreos-tracker


Fields added to the model but not allocated to a panel appear outside the tab interface

When fields are added to the model but not allocated to a panel (e.g not added to BlogPage.content_panels), they're displayed without label or js enhancement at the bottom of the page outside the tabbed interface. It's an ugliness I'd like to be tackled if possible.
Perhaps the new field(s) should appear in the first tab, "Content", or they shouldn't appear at all and perhaps display a warning.

https://github.com/gasman/wagtail/tree/fix/stray-form-fields-2 now passes against the improved test suite in #933 - there's still a latent references to rendered_fields in the InlinePanel tests, though, so this should wait until that test has been updated too.


Allow for displaying tabs as 4 spaces

Gogs currently shows tabs as \t, which is rendered as eight spaces on every browser.
Given that displaying tabs as four spaces is the default in many editors and environments, could there be an option to display them as such?

As I said here, one possible way is using what is specified in the .editorconfig file. This is how it work on GitHub.

Could be set as a user-setting as well, with default to the current
behaviour. Regexp replace should be fast enough on page-views :)

The proper way to solve this would be through CSS, so tab characters are preserved:
     tab-size: 2 !important;
-moz-tab-size: 2 !important;

Could be easily put in a userstyle as well ;)

@silverwind Why not send in a PR with your proposed solution?

I think it is more nature to set when Gogs supports custom static resources. (have we already supported somehow?)

.editorconfig could be another kind of support, too. @andreynering would you mind send a PR for this?

@Unknwon I'll try to implement editorconfig support when have a little free time.
But maybe I should wait for the web editor PR first? #3151

@andreynering @Unknwon Yeah, this is something I have worked with and fiddled with in the editing feature, and while I separated the JS call to the Wiki editor from the File editor, they are in the same JS file and similar in function. We can change how tabbing works.

PR for editorconfig: #3241

Patch has pushed to fix this issue, please test on develop branch or https://try.gogs.io.

Looking good!
https://try.gogs.io/silverwind/with-editorconfig/src/master/testtab.js
https://try.gogs.io/silverwind/without-editorconfig/src/master/testtab.js


test: changing waitForEndStream to fail if it takes 30s

Turns out the envoy default timeout of 30s means we had 2 tests waiting for a response that were just hanging 30s.
Fixing both tests (1 overload, 1 quic) and futureproofing.
Risk Level: n/a (test only)
Testing: yes
Docs Changes: n/a
Release Notes: n/a

cc @danzh2010 this and the follow-up doing the same for waitForReset should speed up the quic test runs substantially :-)

OK, CI happy.  PTAL!


Donate link broken

Thanks for reporting this. Is the issue still exist? It's working now on my side.
https://palletsprojects.com/donate
It seems Netlify can't handle the redirection occasionally (#88). Maybe we could hardcode the PSF link?

Issue still exists. Interestingly enough, the link works on Chrome 99 on Android, but not Chrome 99 on Mac 🤷


Incorrect description of existing known issue

Preconditions:
Mark at least one failed test with known issue (using existing JIRA ticket).
Steps to repro:

Find additional failed test scenario
Click on "Mark as known issue"

Expected result: User is able to mark this test with existing JIRA ticket (with correct description)
Actual result: Incorrect description of existing issue.

cannot reproduce


Release v1.17.1

Release v1.17.1
Features:

#1636
feat: change hmr sock path to /webpack-hmr
feat: 热更新 socket 请求的 path 修改为 /webpack-hmr

#1636
chore(server): export DevServerOptions type
chore(server): 导出 DevServerOptions 类型

#1645
feat: add response headers docs
feat: 添加自定义响应头的文档

#1642
fix: update upgrade command descrption
fix: 更新 upgrade 命令描述信息

#1643
feat(server): export DevServerHttpsOptions type
feat(server): 导出 DevServerHttpsOptions 类型

#1639
feat: add source.enableAsyncEntry config
feat: 新增 source.enableAsyncEntry 配置项

#1637
feat: upgrade support monorepo sub-project
feat: upgrade 命令支持升级 monorepo 子项目依赖版本

#1644
chore: add inspector plugin
chore: 增加 Inspector 插件

Bug Fix:

#1634
fix: upgrade generator error when not exist dependencies
fix: 修复升级工具在不存在依赖时报错

Codecov Report
Base: 57.52% // Head: 57.52% // No change to project coverage :thumbsup:

Coverage data is based on head (20abf87) compared to base (20abf87).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage   57.52%   57.52%
=======================================
  Files         630      630
  Lines       16549    16549
  Branches     3543     3543
=======================================
  Hits         9520     9520
  Misses       6481     6481
  Partials      548      548

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

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


[build] fix warning in SLAAC module in Keil

Fix warning under Keil:
compiling slaac_address.cpp...

..\..\..\..\openthread\src\core\utils\slaac_address.cpp(341): warning:  #1-D: last line of file ends without a newline

  #endif // OPENTHREAD_CONFIG_ENABLE_SLAAC

..\..\..\..\openthread\src\core\utils\slaac_address.cpp: 1 warning, 0 errors

Is it possible to add a check in script/clang-format-check.sh and script/clang-firmat.sh so that this issue will not come again?

@bukepo good idea! I think someone was faster than me :smile:


Revert "[Android] Enable pull-to-refresh"

Reverts crosswalk-project/crosswalk#3621
This will fail tests515, because this introduces android support library's dependence. While tests515 doesn't include this dependence.
Currently the only direct reference to android.support.v* is core_shell, xwalk has no other direct reference to android support library.  So we are not intending to introduce this dependence and will revert this commit. We will re-implement this and remove dependence like android.support.v*.
BUG=XWALK-6781, XWALK-6277

@lincsoon @fujunwei @xzhan96 @wuhengzhi  @rakuco

lgtm

Please be more verbose in your commit message, which is what matters the most in the end: it does not mention the hash of the commit being reverted (which should've been added automatically by git revert), nor does it mention XWALK-6781 and the original pull-to-refresh ticket or explain why this is being done.

Hi, @rakuco, just update the comments. BTW, I revert this by click the revert button.

Again, only the pull request message has been updated, not the actual commit message.

@rakuco

lgtm, thanks


[Postman Activity 1] Intro to API endpoints: Content Context

2.md

explain why cURL is bad and why postman exists to solve cURL's problems
I didnt get the point of this slide because it does not come up anywhere in this activity. Also I did not attend the workshop so I do not have the context through that

3.md

_after 4th line have the user click on create new request so they can enter the request _
lines that start with "![Creating New Collection]" is confusing. What is it supposed to do?

@Sibbons Congrats on your first pull request 👏 👏