endpoint: Make regenerateBPF() more resilient when panic()

If something panics during endpoint regeneration endpoint's lock may remain locked, which will then cause a panic due to trying to lock the endpoint again in a deferred function.
This can be fixed by always deferring the endpoint unlock rather than trusting that the program flow will reach the unlock invocation.
The end result is still a crash due to the panic, but at least it is attributed to the actual reason rather than the secondary effect.
This commit also removes the definition of ExecTimeout as can never fire due to a shorter timeout already being ticking.
Signed-off-by: Jarno Rajahalme<EMAIL_ADDRESS>

This change is

test-me-please

The timeout logic change is pretty severe. It causes the outer timeout of the API to kick in which will result in different behavior.
@jrajahalme Is the change in completion timeout really needed? I would appreciate if we could keep changes to the minimum in this code.

test-me-please

Changed the endpoint generation timeout back to 55 secs.

@jrajahalme Where exactly can we panic?

@tgraf The panic that I saw was due to a bug in my development branch that caused by redir.implementation being nil when it should not have been, which was caused by a simple mixup of C and Go syntax regarding switch/case fallthrough. The panic happened while holding the endpoint lock, while the deferred functions expect the lock not be held. This caused double locking, hiding the root cause (null pointer dereference).
The fix proposed here is not essential if we do not introduce bugs, but helps when we do.

The fix proposed here is not essential if we do not introduce bugs, but helps when we do.

OK, thanks for the context. We are not doing this anywhere else right now and we should be careful to change this function much so late in the development cycle. I'd rather merge this into 1.4 if it is not essential and it is not required to recover panics from the new l7 parsers.

@jrajahalme this needs rebasing.

Unfortunately there is too much bit-rot for this PR to be salvaged by rebasing. This needs to be done from scratch (maybe using this PR as a reference), if we want this.


Parameter path isn't recognized when there is another path

Here is my controller
@Controller('api/authors/')
export class AuthorController {

    @Get(':id')
    async private get(req: Request, res: Response) {
       //Logic
    }

    @Get('random')
    async private get(req: Request, res: Response) {
       //Logic
    }

    @Get('')
    async private getAll(req: Request, res: Response) {
       //Logic
    }

}

The controller should work as follows:

/api/authors/authorId, e.g. /api/authors/3 should return author with the specified id
 /api/authors should return all authors
 /api/authors/random should return a random author

When, in Postman, I make a request to either /api/authors or /api/authors/random it works fine. But when I try to make a request to /api/authors/3 it says:

Cannot GET /api/authors/3

However If I remove the code associated with @Get(random) it is back to normal,  /api/authors/3 works fine again.

Hello,
With express, the expected behavior here is a little different than you have stated.

/api/authors/3 should hit the method labeled @Get(':id')
/api/authors/random should also hit the method labeled @Get(':id')

A request to /api/authors/random matches both /api/authors/:id (where id is "random") and /api/authors/random. Since /api/authors/:id is declared first, it takes priority. If you switch the order that the two methods are declared, then a request to /api/authors/random will match /api/authors/random rather than /api/authors/:id.
Relevant stackoverflow question here. See the more upvoted answer, not the accepted one.
Some assumptions I'm making about the provided code:

The keywords async and private need to be switched.
The first two methods need to have different names (both are currently get)

There is a gist here with the code I am using to test. It looks like it's working as expected.

Hello, thanks for such a fast response. I changed the names of these methods and swapped them in code:
    @Get('random')
    async private getRandom(req: Request, res: Response) {
    }

    @Get(':id')
    async private get(req: Request, res: Response) {
    }

    @Get('')
    async private getAll(req: Request, res: Response) {
    }
}

And everything seems to be working just fine!


Amplitude scale

Hi all,
It was suggested to me to post my feature request/suggestion on github instead of via the mailing list, so here it goes:
There is currently no indication of the amplitude scale when inspecting individual epochs. I am doing visual research and I sometimes use the VEOG and HEOG channels to identify eye movements. Oftentimes there is a clear relationship between e.g. the size of the eye movement and the increase/decrease in amplitude of the step function. So for situations like these, and others, it would be handy to have an indicator of the scale (for an example of implementation in EEGLab see bottom right in this figure http://cognitrn.psych.indiana.edu/busey/temp/eeglabtutorial4.301/rejtut/rawpink1.gif ).
Thanks.

I think it's a useful feature indeed
we would need to have such scale bars for each visible channel type

FYI we have this for raw plotting, we need to port it to Epochs. Ideally we would also unify our Epochs and raw plotting a bit before then.

Porting the scale bars should have automagically happened when @drammock unified the raw and epochs browsers


link resources to projects

how can you return docs that are relevant to generated project?

https://github.com/learntocloud/azure-project-generator/pull/18


⚠ Bofrost.ch has degraded performance

In ffe5201, Bofrost.ch (https://www.bofrost.ch) experienced degraded performance:

HTTP code: 200
Response time: 1242 ms

Resolved: Bofrost.ch performance has improved in 3723eac.


Metric pgcat_pools_sv_idle reports incorrect values

Describe the bug
We run pgcat in k8s with 3 replicas. We scrape the prometheus metrics and visualize them in grafana. The pgcat_pools_sv_idle metric seems to be reporting incorrect numbers as the connections on the postgres side stay stable and are not reflecting what pgcat is reporting. In the application, we are using https://github.com/lib/pq Go driver to connect to pgcat. The pgcat_pools_sv_idle just seems to go up over time.
To Reproduce
Config is here https://gist.github.com/jbub/ef1a1d5436aefca06e51c59c1b8a1150. I noticed that bumps in the metric happen especially when the pool is full and application receives this error pq: could not get connection from the pool.
Expected behavior
The pgcat_pools_sv_idle metric reports correct numbers.
Screenshots
The huge drop in the metric is restart/redeploy.

@drdrsh Thanks, just deployed the new release, will report how it behaves over time.


Allows collector-only servers to disable the UI and query apis

This adds the ability to set QUERY_ENABLED=false for reasons including
security policy.
Fixes #1591

polished up thx to @shakuzen for the advice

Nice guys! That's pretty much how I would have done it :dancer:


Fix last two examples by removing hypen

nuget is the command, not nuget-trusted-signers.

Docs Build status updates of commit 11ed99d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/cli-reference/cli-ref-trusted-signers.md
:white_check_mark:Succeeded
View

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


⚠ ClamAV has degraded performance

In 5c54f17, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 563 ms

Resolved: ClamAV performance has improved in 2726d76 after 6 minutes.


Limit tags to 50 characters and only 10 tags at maximum and update UG for upcoming features

Resolves #315
Note that we need to add this to the UG

LGTM


RemoteRead or Thanos sidecar support

Hello, our company is running a distributed prometheus setup across cloud providers and data centers using Thanos.
We would like to support GCP managed prometheus setup in our Thanos setup.
I've tried several things:

Deploy the forked GCP prometheus: Seems like it's only used for ingesting data into GCP managed prometheus not querying or could not find any info.
Deploy the thanos sidecar with GCP prom frontend and getting error from sidercar for external labels fetch:

/v1/projects/PROJECT_ID/location/global/prometheus/api/v1/status/config was not found on this server.

Try remote read from a native prometheus to GCP prom frontend but getting:

GET /graph HTTP/1.1
Host: frontend.devops-gcp-monitoring.svc.cluster.local:9090
User-Agent: Prometheus/2.41.0
Accept-Encoding: snappy
Content-Encoding: snappy
Content-Type: application/x-protobuf
Referer: http://frontend.devops-gcp-monitoring.svc.cluster.local:9090
X-Prometheus-Remote-Read-Version: 0.1.0

HTTP/1.1 200 OK
Date: Fri, 22 Dec 2023 19:00:32 GMT
Content-Length: 752
Content-Type: text/html; charset=utf-8

<!doctype html><meta charset="utf-8"/><link rel="shortcut icon" href="./favicon.ico"/><meta name="viewport" content="width=device-width,initial-scale=1,shrink-to-fit=no"/><meta name="theme-color" content="#000000"/><link rel="manifest" href="./manifest.json" crossorigin="use-credentials"/><title>Google Cloud Managed Service for Prometheus</title><link href="./static/css/main.4858e794.css" rel="stylesheet">You need to enable JavaScript to run this app.

POST / HTTP/1.1
Host: frontend.devops-gcp-monitoring.svc.cluster.local:9090
User-Agent: Prometheus/2.41.0
Content-Length: 201
Accept-Encoding: snappy
Content-Encoding: snappy
Content-Type: application/x-protobuf
X-Prometheus-Remote-Read-Version: 0.1.0

HTTP/1.1 302 Found
Location: /graph
Date: Fri, 22 Dec 2023 19:00:45 GMT
Content-Length: 0

Is there any options to achieve this ? Otherwise are they any plans to support this pattern ?
Ideally having the frontend proxy work with a thanos sidecar would be ideal/fantastic with a specific external label, Thank you.

Hi! Great question. We don't currently support this (not officially), but there might be some workarounds here, plus future experiments. Let's unpack it:
Sounds like you want to connect your GCP monitoring to Thanos (specifically your GKE clusters with managed collection, right?). Are you interested specifically in "read" mode? (remote-read/sidecar/Thanos StoreAPI) or remote write would solve your problem too?
From a high level point of view your attempts were solid. In theory, we could try getting metrics from your GKE clusters to Thanos in two ways:

Use local cluster metric collection -> Thanos
Use local cluster metric collection (e.g. recommended GMP Managed Collection) -> Google Cloud GMP service (part of Google Cloud Monitoring) -> Thanos

1. Cluster -> Thanos
In this case we could try getting metrics directly from the cluster metric collection either via remote-write or read (with sidecar) to Thanos. In the simplest scenario you can skip GMP totally and use classic methods like manual Prometheus setup, Prometheus Operator or anything else that supports Thanos sidecars or remote-writing to Thanos.
I assume you added this issue, because for some reason you would like to use:
a. collection that is managed, default on GKE or...
b. GMP operator with scalable daemonSet pattern or..
c.  perhaps you want ingest metrics to both GMP/GCM service AND Thanos
.. or mix of those (: (which one?)
For (a), indeed we don't support this case with GMP managed collection and currently we don't plan to add it.
For (c), you can totally deploy our forked GMP Prometheus (our fork simply adds GCM API support, nothing else) without operator or with Prometheus Operator and configure it to export to GMP, as well as add sidecar/remote-write. All of this should be possible with Prometheus Operator flexible configuration (extra arguments). We also work hard to support native remote write in GCM, which will get rid of fork.
For (b), unfortunately you can't configure remote-write yet (this requires operator code changes). We plan to add that sooner or later. However, you should be able to use self-deployed GMP operator with Thanos sidecar, if it makes sense for you. We might make it more easy in the future (ideas or even contributions are welcome!), but you should be able to do it now e.g. (theory, not tested):

Disable managed collection on your cluster
Go to this repo manifests/operator.yaml and adapt Prometheus DaemonSet  to your needs. E.g. add Thanos sidecar, expose port, add necessary HTTP service that will route to each sidecar (not load balance!) change limits etc.
Apply manifests/setup.yaml then manifests/operator.yaml YAMLs.
Done, you can use PodMonitoring etc.

Question is, is (b) + sidecar really a good idea. Thanos will have to discover and query each Prometheus pod per node for potentially every PromQL query if you don't setup and use external labels correctly. For multiple clusters with hundreds nodes (e.g. 10k StoreAPIs), without proper label matcher enforcing this might be tricky.
Remote-write with (b) might be better, let us know if you would like us to add it for the self-deployed operator scenario.
2. GMP/GCM -> Thanos
This is even more interesting, because there are various (free) system metrics that go through different mechanisms than GMP managed collection. You might also add metrics from other GCP systems using managed flows (Cloud Run, GCE etc). This is not currently possible without some additional custom code on top.
frontend proxy (as you noticed) is not helpful for this case at the moment. External labels are not a huge problem, we could find a way (e.g. allow configuring them), but GCM/GMP cloud service does not support remote-read API.
Potential solutions in future or for brave programmer:
a. One could implement remote-read (or even directly StoreAPI) against GCM API e.g. using ListTimeSeries gRPC/HTTP API inside frontend or other proxy. Perhaps possible with some limitations (e.g. ListTimeSeries allows retrieving only one metric type per call).
b. One could utilize a new (a bit hidden for now) QueryAPI in Thanos. I don't want to spoil too much, but we might have something to show about this in the upcoming ThanosCon in Paris in March. With some extra code and Thanos setup this allows similar flow.
cc @macxamin @fpetkovski @pintohutch @lyanco

We use multiple Prometheus servers and employ Thanos as a single endpoint to query metric data from these Prometheus servers. Additionally, we use the GMP frontend to query metrics native to the GCP cloud, such as GCP load balancer metrics. Implementing a Thanos sidecar with the GMP frontend would be highly beneficial. This setup would enable Thanos to query metrics both from our local Prometheus servers and from the Google Cloud Managed Service for Prometheus (GMP).

👋🏽
I believe this use case is now solved thanks to @maxamins and @fpetkovski work, but without using classing Thanos sidecar or remote read.
Essentially it is now possible to query GMP/GCM from Thanos Querier and join data from other Thanos (or any other) sources
How:

Add extra Thanos Querier on top of your Thanos cluster (or any other PromQL enabled endpoint) with --query.mode=distributed flag. It will pushdown PromQL queries to all your endpoints and aggregate on the top level.
Deploy light proxy (https://github.com/thanos-community/thanos-promql-connector) next to that top level Thanos Querier that can translate any HTTP PromQL endpoint to Thanos gRPC Query API, allowing (for example) to connect GMP/GCM HTTP PromQL endpoint to Thanos system.

This was explained in the KubeCon EU ThanosCon talk (slides, recording available soon). We are discussing adding proper documentation and blog posts about this too.
Feedback welcome @r0bj @arthurh4 etc, but in the meantime I will close this issue as fixed 🤗

Thank you @bwplotka.
It seems that recording "Connecting Thanos to the Outer Rim via Query API " you mentioned is already available: https://www.youtube.com/watch?v=E8L8fuRj66o


Lib is unable to find config resources

Installed library is missing the config resources to create the classes in domain.py, as shown in the IPython notebook.

Attempted to fix with PR #16


Correct Formula Cookbook (missing ?)

Correct Messaging section:
There is a missing ? in the example code:
Currently:
if build.with?("qt") && build.with("qt5")
  odie "Options --with-qt and --with-qt5 are mutually exclusive."
end

There should be a ? in the second build.with:
build.with?("qt5")

[+] Have you followed the guidelines in our Contributing document?
[+] Have you checked to ensure there aren't other open Pull Requests for the same change?
[+] Have you added an explanation of what your changes do and why you'd like us to include them?
[N/A] Have you written new tests for your changes? Here's an example.

[N/A] Have you successfully run brew typecheck with your changes locally?
[N/A] Have you successfully run brew tests with your changes locally?
[N/A] Have you successfully run brew man locally and committed any changes?

Thanks again, @psobolewskiPhD!

Thanks again, @psobolewskiPhD!


Add CRS codes for OSGB36

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.30%. Comparing base (bfc85b9) to head (86671a3).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #218   +/-   ##
=======================================
  Coverage   96.30%   96.30%
=======================================
  Files          32       32
  Lines        1382     1382
=======================================
  Hits         1331     1331
  Misses         51       51

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


WIP: Port to Fabric 1.19 and use the new fabric-message-api decorator

This makes the mod work on 1.19.
Since the whole chat infrastructure has changed a bit, these changes consist of the following:

Update fabric/loom/etc versions to 1.19 compatible ones.
Update Mixin references to new method signatures.
Update to the new Text class layout (things got moved around a lot in the game's code).
Use the newly established fabric-message-api to apply the server side formatting rather than a Mixin. The game now supports the concept of a "chat decorator" for this very purpose. However, as it only allows a single decorator to exist at the same time, we can make use of Fabric's abstraction here to have our implementation be more robust and compatible with other mods.

Left to do:

Ensure no further changes are necessary between 1.19-rc2 and 1.19.
Update fabric.mod.json to use ~1.19 once the version has been fully released.

I understand the refactor of new LiteralText("") to Text.empty() but why are many (but not all) instances of new LiteralText(x) replaced with Text.empty().append(x) and not Text.of(x)?

I have to double check later, but I believe it is because Text.of() returns an instance of Text which is not mutable. Text.empty() on the other hand creates a new MutableText instance that can then be modified (in our case, mostly appended and styled).
This pattern also seemed awkward to me at first, but it appears to be used throughout the game's own code base now.

After reviewing the codebase and mappings, the prior instances of new LiteralText(x) should be now using Text.method_43470(x) (currently unmapped in Yarn). This method creates a new MutableText object wrapping a LiteralText parent. The mapping of Text.of is rather ambiguous. I intend to migrate the codebase to official mappings for 1.19, you are welcome to do this in your PR but there is no expectation, I will happily do it myself once this has been merged.

Ah, I must have missed that. I can convert to Text.literal.
Migrating mappings feels out of scope for this, I'd rather not deal with that.


fix tokio thread_name

pulled out of and continuing discussion from https://github.com/shotover/shotover-proxy/pull/1315#discussion_r1311109557
After enabling "show custom thread names" in htop (its disabled by default) we get output like this:

It looks truncated because linux does not support thread names that long.
This PR fixes the issue by using a shorter thread name.
It now looks like this:

While there could theoretically be some obscure case out there, at our current scale I dont believe this will cause any issues due to the change of name.
Scripts that invoke things like killall shotover-proxy will be unaffected because the main thread is still called shotover-proxy.

It is the Linux API that limits to 16 characters not htop https://man7.org/linux/man-pages/man3/pthread_setname_np.3.html


API: consistent float_format specification

For now just some observations:

The docstring of different funtions about float_format:

the documentation of to_csv (as well as to_excel) states: string, Format string for floating point numbers

Example of this usage: df.to_csv(float_format='%.3f')

The docstring of to_string (as well as to_html) says:  one-parameter function, Formatter function to apply to columns’ elements if they are floats

Example of this usage: df.to_string(float_format='{:.3f}'.format)

The option display.float_format also accepts a callable following the docs

The narrative docs are consistent with that:

http://pandas.pydata.org/pandas-docs/stable/io.html#writing-to-csv-format
http://pandas.pydata.org/pandas-docs/stable/io.html#writing-a-formatted-string

Question:

Should we make this consistent? Eg that all methods can accept both a format string as a format function?
Should we make the usage of '{:.3f}'.format) more convenient?

There are a lot of different issues already about formatting, but I didn't find anything about this in specific.

Aha, related issue: #2502 (about the format of the format string), but this was closed in favor of #4668

put he reference on #4668

This is probably a separate issue, but:

There are a lot of different issues already about formatting

Just to be sure: there is still no way to provide a per column float_format, as styler offers?
If so, and if we're going to shoot for consistent float_format, then might I suggest it also aligns with styler?


figure out why SpiritPylonGhosts are colliding with player

Description
Ghosts from the Spirit Pylon have the SpiritPylonGhostMinion component which inherits UndeadMinion, which should stop player collision. But somehow, in this particular case, it does not.

I think your prefab has an extra collider on it, and since it's on a child object of a Character, collision.gameObject.GetComponent<Character>() returns null in UndeadMinion::OnCollisionEnter.

It's probably this. I'll test removing it tomorrow

Maybe its removal breaks the ghost though. We'll see. I don't know what it's for

Doesn't appear to break it. Did you copy it from a creature that hurts you just by touching you?
I used this mod to find it and test its removal:
https://valheim.thunderstore.io/package/sinai-dev/UnityExplorer/

It's just a copy of the vanilla ghost with some minor changes to its color. I don't know if removing the attack collider will have any effect. I'll test tomorrow

Yeah removing that fixes the problem. Thank you


Google Colab not working, default is now Python 3.8

Describe the bug
The tinycudann install is failing because Colab recently changed to python 3.8, not 3.7
To fix, we should either update the wheel or add an if-case depending on what python version colab is using right now. This would be in case the roll-out isn't immediate for everyone.

In the meantime, you can use the runtime fallback version.
This is available from the Command Palette via the "Use fallback runtime version"
Ref: https://github.com/googlecolab/colabtools/issues/3246

Actually, I also had to comment out the condacolab installation, simply reverting to the fallback runtime version did not work for me.
See my fixed version of the demo notebook.
The condacolab installation restarts the runtime, which falls back to Python 3.8 per default, hence the tiny cuda installation still fails.

@puzzlefish  I just made a PR for this. I spend a good while trying to get a new wheel to work, but with no dice. But luckily the default pip install works fine. Let me know if it works for you!

The tinycudann install works in your fix, thanks!
The only thing is that it takes considerably longer (maybe 13min longer), but given its a temporary fix I guess that's not really an issue.
Unfortunately, the notebook as a whole still fails at the data download part, though due to the unrelated issue #1062, which is, in my opinion unjustly, marked as closed.

The only thing is that it takes considerably longer (maybe 13min longer), but given its a temporary fix I guess that's not really an issue.

@puzzlefish  just created a PR to fix this. Had to bring the tinycuda version down to v1.5, but now its a wheel which should be much faster.
On the data downloading, we are looking into possible solutions to this outside of just reposting the data on google drive. Sorry for the delay on this!


tests: add more e2e test cases

Codecov Report

Merging #14 (71bfdae) into develop (f557980) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #14   +/-   ##
========================================
  Coverage    88.52%   88.52%
========================================
  Files           12       12
  Lines         2352     2352
========================================
  Hits          2082     2082
  Misses         270      270

Flag
Coverage Δ

unittests
88.52% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Update domains.json gov.richquack.com to richquack.eth

Hi! What is your PR about?

[ ] Add or edit a skin
[ V] Add custom domain for your space
[ ] Add alias for migrate your space
~Add images for your space~ (deprecated) Add the image URL on the space settings directly
~Edit your space with GitHub?~ (deprecated) Please migrate your space to ENS https://docs.snapshot.page/spaces/migrate
~Create your space with GitHub~ (deprecated) Create your space with ENS instead https://docs.snapshot.page/spaces/create

Hi @gokaiorg how about the domain? Did you approve it yet? anything wrong with the custom domain?

We have a new method to insert your images without creating a Pull Request. You can now use the Avatar field as your space settings.

yes just update from settings.


Base cl3 on a fork of cl-sys as it is no longer actively maintained.

@vmx proposed creating a fork of cl-sys (cl3-sys?) in issue https://github.com/kenba/opencl3/issues/30 in the opencl3 repo.
The rational is that cl-sys is no longer actively maintained.
This would involve making a new ffi repo, fixing bugs in the cl-sys repo, adding new OpenCL  features  and attempting to handle features properly between the new repo, this repo and the  opencl3 repo.
Now would be a good time to do this since Rust is stable at Edition 2021 and the OpenCl SDK is also relatively stable.
However, I would appreciate Volke's help to design and debug the changes.

Changes pushed to crates.io.


Update installation instructions

Now that the app is rewritten to flask, the current installation instructions are obsolete.  We should update the installation instructions.  (using virtualenvs etc)

Doing it right now

https://github.com/minimalparts/PeARS/pull/20 addresses this issue.


WslRegisterDistribution failed with error: 0xe000020b

Version
Microsoft Windows [Version 10.0.19042.1586]
WSL Version

[X] WSL 2
[X] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
No response
Other Software
No response
Repro Steps
Unsure of how this happened, I'm working off a VDI. I am the only one getting this issue at the moment and support doesn't know why. I've tried installing the Ubuntu distro on the MS Store and it gives me this error:
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0xe000020b
Error: 0xe000020b (null)
Press any key to continue...

Expected Behavior
I was expecting it to install and run just fine like it did on my local machine.
Actual Behavior

I found another ticket with the same error, I tried following the steps he listed to fix it, but that didn't work for me.
Diagnostic Logs
No response

/logs

I can't seem to open the feedback hub and github won't let me upload the .etl file that's mentioned in the last step of how to attack a log link the bot gave me.

Another user had same problem and solved it resetting the networking adapters. See https://github.com/microsoft/WSL/issues/6089

@saykin If resetting the network adapters didn't work you can upload your .etl file to your gist and post a link here.

@saykin If resetting the network adapters didn't work you can upload your .etl file renaming it to .txt first.

If feedback hub doesn't work for you, can you follow the Records logs manually section and share the log file ?

Does anyone able to resolve this 0xe000020b error? I have the same error when installing version 20 and 22 of WSL.

I'm still experiencing the same issue

I'm also experiencing the same issue.


Removed empty numbers and fixed italics

in "Re-sign the application and deployment manifests"
Also added underscore before "MostRecentPublishedVersion"

 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.:x: Falthazar sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/vsto/deploying-an-office-solution-by-using-clickonce.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks for taking the time to fix these things.


No scam-domain label

The ISSUE_TEMPLATE to report scam links calls out the label scam-domain, but it doesnt exist in this repo.

ok just fixed it

ok just fixed it

Thanks

It works now, as expected.


ItBit orderbook missing date

Wouldn't it be better to leave it null instead of artificially setting to local time?
I mean, if the server returns some value, than OK, it's a time of the last operation, or in worst case, current server time, but with new Date() the field is worthless.

Yes, I agree. It was decided at the beginning to only allow a date that comes from the server and not to call new Date(). Maybe we should call the field serverTime to make that clearer.

BTW, it still wouldn't be clear if it's the timestamp of the call or the last modification of order book. We don't need to rename the field, javadoc should suffice.

Ok makes sense. Quite a few imps use polling date so we should change it
there as well.
On 12 Jan 2015 19:48, "Tim Molter"<EMAIL_ADDRESS>wrote:

Yes, I agree. It was decided at the beginning to only allow a date that
comes from the server and not to call new Date(). Maybe we should call
the field serverTime to make that clearer.
—
Reply to this email directly or view it on GitHub
https://github.com/timmolter/XChange/pull/816#issuecomment-69632959.


Add Lupusec Adapter to stable Version

Version 0.2.7

Hm,... can you please merge current master in your fork? Solve conflicts. Or need to check later from real pc

@Apollon77 , I have no idea what I have to do. I hope you can help me

fixed


OCPBUGS-35522: Copy infra-volumesnapshot-class-mapping to core binding

What this PR does / why we need it:
The infra-volumesnapshot-class-mapping was only part of the developer binding but should have been a core binding so it ends up in hcp as well as hypershift.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes https://issues.redhat.com/browse/OCPBUGS-35522
Checklist

[ ] Subject and description added to both, commit and PR.
[X] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/jira refresh

/area cli

/jira refresh

/test e2e-aws

/test e2e-aws

/jira refresh

/jira refresh


Created nx_version file

Hello, I have completed the first outreachy task. Kindly review the PR.

Thanks @AnneO1!


coap-gateway.md: update deployment doc based on yaml config.

@ondrejtomcik @jkralik
I newly updated this guide based on yaml config as well as making certificates, getting/editing config file, running image without source download & build. Review and then give me comments. If good, approve plz.

@ondrejtomcik
Do you think document about oauth-server for mock test in docs/guide/deployment ?

@ondrejtomcik
Do you think we document about oauth-server for mock test in docs/guide/deployment ?

It's not used only in test but also in bundle. Yes I would document it as well. But with note that it's just for tests, not suited for production.

@ondrejtomcik
Do you think we document about oauth-server for mock test in docs/guide/deployment ?

It's not used only in test but also in bundle. Yes I would document it as well. But with note that it's just for tests, not suited for production.

Do I create doc for oauth-server on the test/oauth-server as a README.md ? Not in docs/guide/deployment.


team-invite allows to invite the same mail address twice

When you invite the same mail address twice on the team page (/orga/event/foo/settings/team), the invitations are treated and displayed separately. Instead the page should probably resend the invitation without duplicating the entry. In addition a helpful message like

Whoops, this mail address is already on the list. We gave sending the invitation link a second time a try instead.

could be shown.

yay! 👍


Level 20 Barrage has wrong damage multiplier

In POB, level 20 barrage has damage multiplier of 48% but in game and on wiki it is listed as 47.6%.
This is important because POE rounds down so it's means the damage at level 20 barrage is wrong. It also means that POB shows leveling Barrage to 21 has no effect even though that's when Barrage's damage multiplier should hit 48%.
Thanks!

Actually, it is also 47.6% in PoB, but it is currently rounded to 48% when displayed in the gem tooltip; I'll fix the inconsistency in the next update.

Hmm, in that case, I'm surprised because POB shows no DPS change when leveling the gem to 21 even though it should be effective 1% increase in damage multiplier as far as I can tell.
Glad to hear about tooltip!

I tried changing the gem level in a simple Barrage setup and did see a DPS change from 20 to 21; what is your build?

https://pastebin.com/jZcq7w1B barrage six link.
I thought it was the rounding issue since DPS changes from 19 -> 20 and then 22 -> 23.
Though perhaps something about the build is making damage multiplier not do anything and change is just from damage effectiveness?

The change is indeed from damage effectiveness alone, and that's working as intended; you don't actually have any damage on your weapon for the base damage multiplier to apply to.

Right, I was thinking it was just no physical damage but was misreading. Thanks for point it out. Super useful as always.


Remove fields from Container edit form

The list boxes on the /core/ctnr edit form are extremely dangerous and allow a user to completely blow away available domains/ranges in a container easily (yes, this has already happened).  Please remove the Domains,Ranges,Workgroups fields from this form.

Resolved by https://github.com/OSU-Net/cyder/pull/952


github icon link doesn't work without projectLink key

version: 2.0.0-alpha.29
expected: if projectLink key isn't specified, clicking on github icon will go to the link provided in the github key.
what's happening now:
// theme.config.js
export default {
  github: 'https://github.com/neovim/neovim',
}
// => github icon link goes to https://github.com/shuding/nextra

and
// theme.config.js
export default {
  github: 'https://github.com/neovim/neovim',
  projectLink: 'https://github.com/neovim/neovim',
}
// => github icon link goes to https://github.com/neovim/neovim

Report
On Tue, Mar 1, 2022, 21:28 Khang @.***> wrote:

version: 2.0.0-alpha.29
expected: if projectLink key isn't specified, clicking on github icon
will go to the link provided in the github key.
what's happening now:
// theme.config.jsexport default {
github: 'https://github.com/neovim/neovim',}// => github icon link goes to https://github.com/shuding/nextra
and
// theme.config.jsexport default {
github: 'https://github.com/neovim/neovim',
projectLink: 'https://github.com/neovim/neovim',}// => github icon link goes to https://github.com/neovim/neovim
—
Reply to this email directly, view it on GitHub
https://github.com/shuding/nextra/issues/383, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXT4H7UETA3C4APAPZLQU2DU5Y5AXANCNFSM5PUOGRPA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Still happening now, was quite confusing.

will be fixed in the next release with changed theme config options!


Bump libmdns to 0.7

On my machine I'm getting spammed with warnings of this type:
[2022-05-19T16:56:38Z WARN  libmdns::fsm] couldn't parse packet from <IP_ADDRESS>:5353: type 47 is invalid
[2022-05-19T16:56:45Z WARN  libmdns::fsm] couldn't parse packet from <IP_ADDRESS>:5353: type 47 is invalid
[2022-05-19T16:56:45Z WARN  libmdns::fsm] couldn't parse packet from <IP_ADDRESS>:5353: type 47 is invalid

This is because some devices in my network use mDNS to talk to each other, broadcasting messages to other mDNS devices like the machine running librespot. But this packet type 47 hasn't been implemented in libmdns, so the warning is printed.
As I use librespot as a systemd daemon all those messages end up in my journal and accumulate to >1000 warning per day. This warning has been fixed 11 days ago in libmdns (see https://github.com/librespot-org/libmdns/issues/19).
I've locally compiled with the new libmdns version 0.7 instead of 0.6 and the messages disappear, so the bug seems to be resolved

Thanks!


remove device

Good day,
First of all, thank you very much for this beautiful addon. I have a question about removing a device.
Can this also directly in hassio?
I tried something but unfortunately I'm doing something wrong probably? (see

attachment)

Greetings justin


Rebuild search

As we had huge performance problems with pg_search and want to get rid of elasticsearch we decided to build a "simple" postgres search.

[x] add base search for search term
[x] add grouping for facets
[x] add weights for different fields --> we decided to solve that by pg_search
[ ] optimise performance by having only one connection to the database (putting all queries in one exec_query-call)
[x] Decide if you want to show cities with count 1 (I added this to reduce noise and make handling the resulting hash in ruby faster, but now that I think about it: After a search you probably want to see the 1's as well, right?)
[x] Proper documentation and tests for the ProfileGrouper(!) so the queries can be maintained easier
[x] remove elasticsearch
[x] remove search.rb if we decide to stick with pg_search
[x] optimize performance of search (The search page should be rendered within 1 second - if possible ;) )

We decided to use pg_search for implementing the weights. It provides an easy to understand wrapper of postgres  ts_vector.
As the grouping is now managed via the ProfileGrouper we already resolved some of the performance issues we had in the first run.

Deployment on production failed with a
at=error code=H10 desc="App crashed" error.
During deployment I got the following warning, I'm not sure if this has something to do with it:
remote: ###### WARNING:
remote:
remote:        Installing a default version (20.9.0) of Node.js.
remote:        This version is not pinned and can change over time, causing unexpected failures.
remote:
remote:        Heroku recommends placing the `heroku/nodejs` buildpack in front of
remote:        `heroku/ruby` to install a specific version of node:
remote:
remote:        https://devcenter.heroku.com/articles/ruby-support#node-js-support
remote:
remote: -----> Installing node-v20.9.0-linux-x64
remote: -----> Detecting rake tasks
remote: -----> Preparing app for Rails asset pipeline
remote:        Running: rake assets:precompile
remote:        Yarn executable was not detected in the system.
remote:        Download Yarn at https://yarnpkg.com/en/docs/install
remote:        Yarn executable was not detected in the system.
remote:        Download Yarn at https://yarnpkg.com/en/docs/install
remote:        autoprefixer: /tmp/build_76c02237/app/assets/stylesheets/application.scss:17661:3: end value has mixed support, consider using flex-end instead
remote:        I, [2023-11-14T20:08:59.684899 #303]  INFO -- : Writing /tmp/build_76c02237/public/assets/application-607630b1440c36b8e455400bb35cef1d55b1ef874516cb22e1821bf86880999c.css
remote:        I, [2023-11-14T20:08:59.685481 #303]  INFO -- : Writing /tmp/build_76c02237/public/assets/application-607630b1440c36b8e455400bb35cef1d55b1ef874516cb22e1821bf86880999c.css.gz
remote:        Asset precompilation completed (5.10s)
remote:        Cleaning assets
remote:        Running: rake assets:clean
remote:        I, [2023-11-14T20:09:00.869012 #316]  INFO -- : Removed application-ad39eea2292e99be5a7010267c1bf6ff707dfca842e415f01510e9b583b59dcf.css
remote: -----> Detecting rails configuration
remote:
remote: ###### WARNING:
remote:
remote:        Installing a default version (20.9.0) of Node.js.
remote:        This version is not pinned and can change over time, causing unexpected failures.
remote:
remote:        Heroku recommends placing the `heroku/nodejs` buildpack in front of
remote:        `heroku/ruby` to install a specific version of node:
remote:
remote:        https://devcenter.heroku.com/articles/ruby-support#node-js-support

Besides check and fix behaviour of pagy while clicking on another page. Instead of using the paginated profiles it creates a new DB query.

Done and deployed. Thanks a lot @A5308Y 🥳


add some missing return values

That was somehow missed by the last commit.

Merged manually in adfe7f293a41da4ba0188e6a9688af9eaaf0726e.  Thanks.


Fix scrollPosition and added different alignments

elem.offsetTop is not working if the element to be scrolled to is inside an relative placed container. In that case it makes more sense to use getBoundingClientRect() to get the real offset relative to the viewport.
Added 2 more ways to align the element in the viewport : bottom and middle
And also added all the options to the readme

I just noticed there is still one issue, that this only works when scrolling from top. Need to fix that first.

Okay fixed the last remaining issues. Now the function is always calculating the absolute scrollPosition.

Merged!


Clean up C APIs for calling objects

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.11%. Comparing base (70980f8) to head (9b464af).
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

:exclamation:  There is a different number of reports uploaded between BASE (70980f8) and HEAD (9b464af). Click for more details.
HEAD has 18 uploads less than BASE

Flag
BASE (70980f8)
HEAD (9b464af)

19
1

Additional details and impacted files
@@             Coverage Diff             @@
##            master     #501      +/-   ##
===========================================
- Coverage   100.00%   94.11%   -5.89%
===========================================
  Files           20       20
  Lines         2311     2311
===========================================
- Hits          2311     2175     -136
- Misses           0      136     +136

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Cralify flow for tests

Please clarify the flow for tests. Which calls should be called and in which order?
There are POST /test/$id/submit and GET /project/$id/tests, but it's not obvious how to work with them.

This might help a little bit.


Remove join, nick and whois inputs, they are cleanly handled by the server

Fixes #201

:+1:

I'll test tomorrow and will merge then.

Tested locally and all of the removed commands (well, apart from /query) still work.
:+1: and merging.
When releasing this, I will specify in the release introduction that the /query command was removed.
My understanding is that usually, people expect that:

/whois displays the whois data of the user. In our case, opening a message window that begins with that info.
/msg sends a message to another user and opens a chat window with displaying that message.
/query simply opens a chat window with a user.

Am I correct?
We are definitely losing the last option with this PR. Would it be possible to re-add /query so that it only opens an empty chat? Something custom like the /clear command The Lounge has for example. Ideally early enough so we can add it to the release that ships this PR.
Also, @dgw rightly mentioned that /msg doesn't open a chat window. It's another problem indeed, but was here before this PR.


Saving custom-lists.ps1 fails silently when the path has special characters like '['

It seemed to work after I replaced every -FilePath option with -LiteralPath in $SaveList.Add_Click( ... ). Maybe this could be a solution?
And maybe you want to wrap a try/catch block around to show the user an error message when the file cannot be created?
Thanks for creating this script, kind regards :)

I will certainly do this! Thanks for the suggestions.


Multicluster app - API should not allow removing of roles

rancher/rancher:master 2/28
The api should throw an error when attempting to remove roles from a multicluster app.
Additionally, when removing roles in the current implementation, the associated rolebindings for the multicluster service account are not removed.
To reproduce:

Create a multicluster app and add a role
Upgrade app and remove the role

Actual:

Role can be removed

Expected:

Role cannot be removed

@bmdepesa  I don't think we ever wanted this to throw an error. It was just going to be a no-op earlier. But now along with allowing this, we will also be removing the rolebindings for the sys acc from the target projects.
cc @alena1108 @cjellick

rancher/rancher:master 38fd13a8 3/7
HA installation

Create cluster(s)
Deploy multicluster app
Remove all roles from multicluster app
HTTP 422 - No roles provided


Fix the legacy driver path construction on non-Windows platforms

swift-driver tries to invoke swiftc-legacy-driver. (trailing dot) on non-Windows platforms when legacy driver is requested since https://github.com/swiftlang/swift-driver/pull/1746
This is because URL.pathExtension returns an empty string if the URL does not have an extension but URL.appendPathExtension appends a period before the extension even if given an empty string. Thus url.deletingPathExtension().appendingPathExtension(url.pathExtension) will not be equal to url if url does not have an extension.

@swift-ci test

Wait, it seems like a swift-foundation side issue:
import Foundation
let url = URL(fileURLWithPath: "/tmp/x")
print(url.deletingPathExtension().appendingPathExtension(url.pathExtension))
// Swift 5.10: file:///tmp/x
// Swift 6.0: file:///tmp/x.

@compnerd The team said it was an unintentional change and fixed in the main branch. https://github.com/swiftlang/swift-foundation/issues/1080

@swift-ci smoke test

@swift-ci please test macOS platform

@swift-ci please test Windows platform


background-clip不生效，文字渐变，项目需要

Please make sure you are testing with the latest release of html2canvas.
Old versions are not supported and issues reported for them will be closed.
Please follow the general troubleshooting steps first:

[ ] You are using the latest version
[ ] You are testing using the non-minified version of html2canvas and checked any potential issues reported in the console

Bug reports:
Please replace this line with a brief summary of your issue AND if possible an example on jsfiddle.
Specifications:

html2canvas version tested with:
Browser & version:
Operating system:

最后解决了么


tabs click not trigger frequently

Ionic version:  (check one with "x")
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
Current behavior:
Tabs click not trigger frequently，you can see below：

Expected behavior:
Click event always trigger even though tap on the top of tab
Steps to reproduce:
1.
npm install -g ionic

ionic start myProj tabs

npm run ionic:serve

Related code:

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
global packages:

    @ionic/cli-utils : 1.2.0
    Ionic CLI        : 3.2.0

local packages:

    @ionic/app-scripts              : 1.3.7
    @ionic/cli-plugin-ionic-angular : 1.2.0
    Ionic Framework                 : ionic-angular 3.3.0

System:

    Node       : v6.10.0
    OS         : Windows 10
    Xcode      : not installed
    ios-deploy : not installed
    ios-sim    : not installed

Dup: https://github.com/driftyco/ionic/issues/10985

Hello, thanks for opening an issue with us. Closing this as a duplicate of https://github.com/ionic-team/ionic/issues/10985.


Dapper doves do dastardly deeds quest not working

The quest glitched out, I picked up each bird and it is not letting me have the keys. It says I poked up 7/5 birds and everytime I talk to Maribel she just says “have you had any luck finding my keys?” and then sort of shoo’s me to go back to picking up the birds.
Expected Behavior
Since I picked up all of the birds I am assuming she should have responded differently. My guess is by saying I have found her keys and thanking me? I cannot find a video anywhere to see how it should properly work or what exactly I am doing wrong here.
Current Behavior
It says I picked up more birds than there are and there are no keys, even though Maribel insists that one of those birds has them and that I need to keep picking them up until I find said keys.
Possible Solution
I am not sure, maybe figure out if it was a mistake of not giving one of the birds the keys on the xbox version?
Steps to Reproduce (for bugs)

Ask Maribel if she needs help
she will tell you the birds have stolen her keys as well as accessories
pick up the birds
It will go up to 7/5
speak to Maribel
she will ask you if you had any luck with finding her keys.

Context
I am trying to accomplish finishing this mission, I don’t really like having incomplete tasks on my quest log. I like to complete them as soon as possible and I feel bad for completing every other task except hers.
Your Environment

Calico version: no idea where to find this.
Orchestrator version (e.g. kubernetes, mesos, rkt): Not sure where to find this.
Operating System and version: Xbox one s

I think you've found the wrong issue tracker.  This one is for the Project Calico networking and security tool; I think you're looking for this discord server: https://discord.com/invite/2RfZQpx

I think you've found the wrong issue tracker.  This one is for the Project Calico networking and security tool; I think you're looking for this discord server: https://discord.com/invite/2RfZQpx


file doesn't exists

https://github.com/jamesgleave/DD_protocol/blob/9c842f6d946a97a5018c899993c70712cbc095fe/scripts_2/simple_job_models.py#L80
Hello, I don't understand where "Mol_ct_file_%s.csv" is generated.  I have checked all your scripts and I didn't find any command that create such file.
Should the variable t_mol be just the total number of compounds in the general database (i.e. ZINC20) ? Supposing that such database has 1 billion, the t_mol is supposed to be 1000? If not, why then divide by 1 million?

Ok, sorry, I have solved. I didn't figured out that that file is actually this https://files.docking.org/zinc20-ML/fingerprints_count.txt
the file name was different.

Mol_ct_file_x.csv, where x is the name of your project, is generated in molecular_file_count_updated.py, line 70. The "updated" file with the number of molecules to sample per smiles file is then generated from it (see lines after 70 in the same script).

Oh, perfect! I don't know why by running that previous code, Mol_ct_file_x.csv was not generated.
Thank you very much!


Fix monospaced font 404

What is the context of this PR?
This is to fix 404 console log. This PR will
How to review

Check fonts all look correct
Check list items prefixed by "1" is still aligned with others

 Fix 404 font console error


Route for checking duplicate vocabs

We need a route to checki if vocab already exists in groups/packages when creating them. The check only has do be done client-side, so no additional checking in the vocab creation functions.
Needed Routes:

Route to check if vocab already exists in package
/api/check-duplicate?languagePackageId={languagePackageId}&vocab={vocab}

Response:
{
    "duplicate": false,
    "group": {
        "id": "fbd8ee4a-8105-4716-zt72-65d6ef411234",
        "languagePackageId": "81ec68fc-843e-4a76-9fzu-5ae0b1gb05c2",
        "name": "Unit 1",
        "description": "This is a test",
        "active": true,
    },
}

Route to check if vocab already exists in group
/api/check-duplicate?groupId={groupId}&vocab={vocab}

Response:
{
   " duplicate": false,
    "group": {
        "id": "fbd8ee4a-8105-4716-zt72-65d6ef411234",
        "languagePackageId": "81ec68fc-843e-4a76-9fzu-5ae0b1gb05c2",
        "name": "Unit 1",
        "description": "This is a test",
        "active": true,
    },
}

For a better overview I have split up the two needed cases. But overall there is only one endpoint, where you can give your needed query parameters to get the result. This includes adding groupId and packageId in one query too (although this is a bit useless).

@luwol03 I have updated the route response

what about making groupId and languagePackagId optional?

what about making groupId and languagePackagId optional and have one controller handler?

Yeah this was my intention I tried to explain here
<
For a better overview I have split up the two needed cases. But overall there is only one endpoint, where you can give your needed query parameters to get the result. This includes adding groupId and packageId in one query too (although this is a bit useless).

Or did you think of something else

oh ok. And what is with the third case without any parameters?

I don't know if not parameters make that much sense. No parameters would mean that it checks in every group and package. But I think you only need this function to check a specific package, not all your packages

I don't know if not parameters make that much sense. No parameters would mean that it checks in every group and package. But I think you only need this function to check a specific package, not all your packages

I was thinking of showing that this vocab already exists in package XXX group XXX

@luwol03 I have added a vocab parameter to the route, as it is a GET. Would you agree with this?

What should this be for? And I would also respond with the found vocab so you can show it in the create anyway dialog.

I was thinking of showing that this vocab already exists in package XXX group XXX

Yeah understand what you mean. But when you create a new package you usually don't care if a vocab you want to add is already existing in another package, as it is completely new and created for another learning purpose

What should this be for? And I would also respond with the found vocab so you can show it in the create anyway dialog.

The vocab parameter includes the name (foreign word) that has to be checked. As this is a GET request I wouldn't send it in the body.
Yeah you could add the found vocab I'm the response. But you could just use the foreign word of the input field in the client that is saved there anyways


[W6.2][W13-B1]Vivek Lakshmanan

Ready for Review

I see that you had completed all LOs of week 6. Keep up the spirit.


[Feature]: Add translations/localization/internationalization to RulesView

Feature Summary
With the foundations for localization in place in the app, we can start adding translations to individual pages by pulling their text out of the html and into the local files in src/translations. We can now add this to the RulesView.
Detailed Description
Look at src/views/LoginView for an example of a page that is correctly using $i18n to localize/translate content. The user-visible strings have been removed from the <template> code and placed into a "login" section of the translation files in src/translations.
The page then imports and consumes the useI18n() composable to grab the t() function used to render/translate text by pulling from the appropriate language's json file. This is done using a setup() block of the page like so:
  setup() {
    const { t } = useI18n();
    return { t };
  },

From there, we can grab a given string by name in the appropriate locale using {{ t('login.joinDiscord') }} since each translation json in this case has a login attribute for all the text in the login page, with a joinDiscord child attribute for the text to show in the 'Join our discord community' button.
We should do the same thing to the RulesView, by extracting all the text in the page into a rules section of each translation json file (e.g. src/translations/en.json) and then configuring RulesView with the useI18n() composable and rendering all its text with t() to translate it to the user's selected locale.

Hi, I will work on this issue, can you assign it to me please 😄

@Rafajrg21 have at it! We can only "assign" issues to org members but go ahead and take a crack at this one


ng generate without specified blueprint produces incorrect warning

OS?
Ubuntu 16.04
Versions.
angular-cli: 1.0.0-beta.25.5
node: 6.9.1
command ng generate without specified blueprint/name produces warning below
The ember generate command requires a blueprint name to be specified. For more details, use ember help.

Are you absolutely sure you have the latest version? You can double check your local version of the CLI with npm ls angular-cli.
We fixed this in beta.25.5 so I find it weird you're still getting it:
kamik@T460p MINGW64 /D/sandbox/master-project (master)
$ ng generate
The `ng generate` command requires a blueprint name to be specified. For more details, use `ng help`.

Oh my bad, it's actually fixed on master only via https://github.com/angular/angular-cli/pull/4026 by @clydin. The fix will be out on the next release.

hi, experiencing the same... :(
`

ng generate
The ng generate command requires a blueprint name to be specified.


add typescript declarations

a minimal src/types/shell-quote.d.ts:
declare module 'shell-quote' {
  type ParseOptions = {
    escape?: string;
  };
  export function quote(xs: any[]): string;
  export function parse(s: string, env?: Record<string, string>, opts?: ParseOptions): string[];
}

would you like to contribute these types to DefinitelyTyped?

nope sorry, i moved to node-shlex for its prettier output (prefer quotes over escapes)


Updated README and updated file tree

Reply if i accedientelly missed or deleted something. Willing to make a site, wait for that

This looks great! I'm going to merge it in then make some small tweaks ("challenges" is spelled incorrectly for example.) I'm also noticing that some of my examples don't match the YouTube numbering so would be great to fix these up sometime soon. Thank you for this help!!