Update HTTPDigest to match RFC 7616

🎯 Aim
Closes #1476
🤨 Summary of changes

Commit 1/4 introduces a utility to create Nonces. This PR elects to use the format recommended in RFC 7616, sec 3.3 MD5(timestamp_in_ms:resource:secret). Note, the RFC recommends the header field ETAG instead of resource/URI, however since there is no mentions of ETAG in FastAPI, the PR chooses to use the more readily available request.url.path
Commit 2/4 is a quick fix to make the HTTPDigest error code 401 UNAUTHORISED instead of 403 FORBIDDEN, see RFC 7616, sec 3.3 'the server responds with a "401 Unauthorized" status code and a WWW-Authenticate header field with Digest scheme'
Commit 3/4 adds another utility to calculate the digest response field consistent with RFC 7616, sec 3.4.1
Commit 4/4 updates the HTTPDigest class to correctly follow the challenge-response framework laid out by RFC 7616.

📋 Implementation notes

Authentication via the Dependent HTTPDigestCredentials: The resolved-dependency will be of type HTTPDigestCredentials, which will have an authenticate(username: str, password: str) -> bool method. Because the Digest response is a hashed field, the server can only authenticate a user by comparing the hashed credentials. This method makes this easier for developers.
Stateless Nonces: A key motivation for the digest nonce is to prevent against replay attacks, "An implementation might choose not to accept a previously used nonce or a previously used digest, in order to protect against a replay attack." (RFC 7616, sec 3.3), however this would require the server to store some state to track already seen nonces. To avoid this, this PR creates a naive stateless nonce within which only the timestamp is checked.

Not implemented

Domain directive: This is optional, since RFC 7616, sec 3.3 states "If this parameter is omitted or its value is empty, the client SHOULD assume that the protection space consists of all URIs on the web-origin."
QOP=auth-int: RCC 7616, sec 3.4.3 allows for integrity protection by setting the QOP directive to auth-int. However, this requires hashing the entity-body of the request for which we do not have readily available. "the hash of the entity body, not the message body -- it is computed before any transfer encoding is applied by the sender and after it has been removed by the recipient.  Note that this includes multipart boundaries and embedded header fields in each part of any multipart content-type."
Nonce counter checks: RFC 7616, sec 3.4 specifies a nc field to allow servers to detect request replays by maintaining its own copy of this count. However, because FastAPI is primarily for RESTful APIs, this PR chooses to not implement this as it would require stateful servers.
Unquoted usernames: RFC 7616, sec 3.4.5 allows for non-quoted, non-hashed usernames that are not allowed in ABNF quoted-string production via a username* digest field. This PR does not support this.

✅ Testing: Manually with CURL
We can validate this implementation is consistent with the RFC by using curl --digest.
First we create a simple server:
# srvr.py
from typing import Annotated

import uvicorn

from fastapi import Depends, FastAPI
from fastapi.exceptions import HTTPException
from fastapi.security import HTTPDigest, HTTPDigestCredentials

app = FastAPI()

security =<EMAIL_ADDRESS>

@app.get("/")
async def read_root(digest: Annotated[HTTPDigestCredentials, Depends(security)]):
    if not await<EMAIL_ADDRESS>"password1"):
        raise HTTPException(status_code=401, detail="Invalid user or password")

if __name__ == "__main__":
    uvicorn.run(app, host="<IP_ADDRESS>", port=51122)

Then CURL the running server
python srvr.py &

curl -v http://localhost:51122/ --digest -u "jamie@ordinarylab.dev:password1"
# INFO:     <IP_ADDRESS>:64092 - "GET / HTTP/1.1" 401 Unauthorized
# INFO:     <IP_ADDRESS>:64092 - "GET / HTTP/1.1" 200 OK

# digest -u "jamie@ordinarylab.dev:password1"

# *   Trying <IP_ADDRESS>:51122...
# * Connected to localhost (<IP_ADDRESS>) port 51122 (#0)
# * Server auth using Digest with user<EMAIL_ADDRESS># > GET / HTTP/1.1
# > Host: localhost:51122
# > User-Agent: curl/7.88.1
# > Accept: */*
# >
# < HTTP/1.1 401 Unauthorized
# < date: Fri, 07 Jul 2023 06:52:25 GMT
# < server: uvicorn
# < www-authenticate: Digest<EMAIL_ADDRESS># < content-length: 30
# < content-type: application/json
# <
# * Ignoring the response-body
# * Connection #0 to host localhost left intact
# * Issue another request to this URL: 'http://localhost:51122/'
# * Found bundle for host: 0x14f708b60 [serially]
# * Can not multiplex, even if we wanted to
# * Re-using existing connection #0 with host localhost
# * Server auth using Digest with user<EMAIL_ADDRESS># > GET / HTTP/1.1
# > Host: localhost:51122
# > Authorization: Digest<EMAIL_ADDRESS><EMAIL_ADDRESS>nonce="MTY4ODcxMjc0NjQzNzo2NjY2Y2Q3NmY5OmQ0MWQ4Y2Q5OGY=", uri="/", cnonce="NWRkMDMyZWE4ZDBhZWIxMDE5ZTY3MTQ2MjA4MDNjYmU=", nc=00000001, qop=auth, response="0e08634b73edaaec3e264cdc0a908b73", opaque="17fa8f50160163eb0a955678d6f0c526", algorithm=md5
# > User-Agent: curl/7.88.1
# > Accept: */*
# >
# < HTTP/1.1 200 OK
# < date: Fri, 07 Jul 2023 06:52:25 GMT
# < server: uvicorn
# < content-length: 4
# < content-type: application/json
# <
# * Connection #0 to host localhost left intact
# null%

References

Wikipedia: https://en.wikipedia.org/wiki/Digest_access_authentication
RFC: https://datatracker.ietf.org/doc/html/rfc7616

LGTM!!


Fixed name of generated comments file

File is generated for Swagger documentation purposes. File has .xml
extension written in lowercase. In source code extension of the file
was written in uppercase and UNIX system did not recognized the file.

Hi @PiotrWachulec - thanks! I use Windows OS so I was not aware of that issue.


[SPARK-11515][ML] QuantileDiscretizer should take random seed

cc @jkbradley

Merged build triggered.

Merged build started.

Test build #45277 has started for PR 9535 at commit bfabeb2.

Test build #45277 has finished for PR 9535 at commit bfabeb2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Jenkins, test this please.

Test build #47692 has started for PR 9535 at commit 888b0f0.

Test build #47692 has finished for PR 9535 at commit 888b0f0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47692/
Test PASSed.

Merged build finished. Test PASSed.

I have rebased this PR with master.

@yu-iskw thanks for fixing this issue. It looks good to me, but can you please resolve the conflicts?

@yu-iskw Apologies for the delay.  We can get this merged now, once you can update it.  Thank you!

Test build #49683 has started for PR 9535 at commit 604cd60.

Test build #49683 has finished for PR 9535 at commit 604cd60.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49683/
Test FAILed.

Test build #49740 has started for PR 9535 at commit c2ca9b9.

Test build #49740 has finished for PR 9535 at commit c2ca9b9.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/49740/
Test PASSed.

Merged build finished. Test PASSed.

@jkbradley I have rebased with master. Can you review it?

@jkbradley ping

LGTM. Merged into master. Thanks! @yu-iskw I created https://issues.apache.org/jira/browse/SPARK-13292 for the Python API and assigned to you.

Thank you for merging it. I will do the issue. Thanks!


AWS Marketplace Integration

### Tasks
- [ ] https://github.com/awslabs/sbt-aws/pull/43

Completed as part of https://github.com/awslabs/sbt-aws/pull/43


Cannot compile shaderc on windows (VS2015)

Hi,
I'm having trouble figuring out how to compile shaderc on windows and how to use the provided makefiles in the examples.
I generated the project like so:
C:\Libs\bgfx>..\bx\tools\bin\windows\genie.exe --with-tools --with-shared-lib vs2015
However, I see these errors:
1>------ Build started: Project: shaderc, Configuration: Release x64 ------
1>  shaderc_hlsl.cpp
1>C:\Libs\bx\include\bx/os.h(297): error C2079: 'st' uses undefined struct 'bx::stat::stat'
1>C:\Libs\bx\include\bx/os.h(298): error C2039: 'stat': is not a member of '`global namespace''
1>C:\Libs\bx\include\bx/os.h(298): error C3423: Use of incomplete type 'bx::stat::stat'
1>  C:\Libs\bx\include\bx/os.h(297): note: see declaration of 'bx::stat::stat'
1>C:\Libs\bx\include\bx/os.h(304): error C2228: left of '.st_mode' must have class/struct/union
1>  C:\Libs\bx\include\bx/os.h(304): note: type is 'int'
1>C:\Libs\bx\include\bx/os.h(304): error C2065: 'S_IFREG': undeclared identifier
1>C:\Libs\bx\include\bx/os.h(308): error C2228: left of '.st_mode' must have class/struct/union
1>  C:\Libs\bx\include\bx/os.h(308): note: type is 'int'
1>C:\Libs\bx\include\bx/os.h(308): error C2065: 'S_IFDIR': undeclared identifier
1>C:\Libs\bx\include\bx/os.h(313): error C2228: left of '.st_size' must have class/struct/union
1>  C:\Libs\bx\include\bx/os.h(313): note: type is 'int'
========== Build: 0 succeeded, 1 failed, 36 up-to-date, 0 skipped ==========
I basically am wondering, what's the correct way to build shaderc, and what's the recommended way to run makefiles on windows (gnuwin32? cygwin?)
Thanks

Grab latest bx&bgfx. It should be fixed.


Add custom fields to every log

After a lot of searching the internet, I was ecstatic to see logz.io working on log aggregation via the lambda extensions. I believe this approach is much cleaner and a more scalable solution over the traditional cloud watch stream approach.
One question, is it possible to add custom field / value pairs that would exist outside of each log line, ex setting app_name: cool app explicitly? The benefit being one could deploy the layer and organize the logs without having to change the log lines across all their lambda functions.
To note, I tried setting an explicit app_name field/value in GROK_PATTERNS: "{\"app_name\":\"cool app\",\"my_message\":\".*\"}", and omitting it from LOGS_FORMAT, ex LOGS_FORMAT="%{app_message:my_message}", but that didn't send through app_name.
One feature proposal to solve this could be to add a new env var, ex LOGZIO_STATIC_FIELDS="{\"app_name\":\"cool app\"}", which would merge in that json into the parsed log

Hi @brandonwestcott thank you for the feedback, happy to hear that you like it!
We will add your request to our backlog, but currently we don't have an ETA for this.
If you want to, you are more than welcome to contribute to this project!
In the meantime, you can add custom fields from your code, by using a logger module.
Not sure which language you're using in your Lambda, but for example in python you can add to your logs some fields by using extra when logging (see python documentation).
The grok pattern is not for adding custom fields, it's only to parse fields. After using a logger that formats logs, you can use the grok feature to parse those fields correctly.


Include Aurelia in comparisons guide

On the Vue.js site in the comparison section, Aurelia is missing from the comparisons and benchmarks. You include the likes of Knockout and Ember and I think Aurelia should definitely be in that list.

Saw this duplicated an existing issue.


Should Ruff linter parse and react to pylint disable waiver comments?

Probably many people that start to use ruff linter have been using pylint previously. These users might already have some pylint rules disabled in their codebase:
class ComplexBanana:  # pylint:disable=too-many-public-methods

Could and should ruff read those disable comments and treat them as noqa comments? I think it would ease the crossover.

This ties in to https://github.com/astral-sh/ruff/issues/1773


RKT does not conserve directory owner on render

When loading dependencies or while rendering a container
Every folder get it's owner:group changed to root:root
While the file in those folder keep their correct attributes.
ie:
ls -ll ~/rkt-volume-rendered/app
-rw-rw-r--  1   33   33     45 Sep  7 14:13 Gemfile
-rw-rw-r--  1   33   33   2085 Sep  7 14:13 Gruntfile.js
-rw-rw-r--  1   33   33    210 Sep  7 14:13 Makefile
-rw-rw-r--  1   33   33   1300 Sep  7 14:13 README.md
-rw-r--r--  1   33   33      1 Sep  7 15:43 TIMESTAMP
drwxr-xr-x  6 root root   4096 Sep  7 15:47 app

The app Folder should also be
drwxr-xr-x  6 33 33   4096 Sep  7 15:47 app

Is it wanted or not ?

@sgotti would you mind taking a look at this one?

@nyodas Does your ACI contains a tar entry for rootfs/app? (what's the output of tar tvf $ACI?)
From my tests this happens only if the entry for a directory is missing (before creating the files inside a directory, rkt needs to create a directory and uses some default permissions since it doesn't have any information on it).

I will check it more thoroughly latter and add some more examples.
So that i'm clearly understood.
Data-aci = the aci with the files tree
Master-aci = the aci that will have data-aci as a dependency.
But from what i remember the
 tar xvf data-aci.aci does have the correct permission.
The folder does not exist in the master-aci befire runtime.
However the full path from root is there in the the data-aci containing my
data.
Do you mean that i have to create an empty directory tree with the correct
owners in the master-aci?
It does seems far fetched in my mind.
As for the information about the folders they should be available in the
data-aci.
My example might have lacked clarity.
But i have a whole directory tree with that issue.
On Sep 8, 2015 9:11 PM, "Simone Gotti"<EMAIL_ADDRESS>wrote:

@nyodas https://github.com/nyodas Does your ACI contains a tar entry
for rootfs/app? (what's the output of tar tvf $ACI?)
From my tests this happens only if the entry for a directory is missing
(before creating the files inside a directory, rkt needs to create a
directory and uses some default permissions since it doesn't have any
information on it).
—
Reply to this email directly or view it on GitHub
https://github.com/coreos/rkt/issues/1368#issuecomment-138673131.

@nyodas sorry I missed that you're using an aci with dependencies. It should work as you intended (no need for empty directory tree in master aci). Tomorrow I'll try with something similar or it will be very useful if you can provide two ACIs (without important data) or the steps to create them will be enough so I can reproduce the problem. Thanks!

Ok so While trying to create a sample aci to give, i couldn"t reproduce the issue.
And find myself in the same situation as you are (everything is fine ^^ , correct ownership everywhere.)
I'm now looking into the tools that create the aci , to see if it's not him that add something weird to the ownership / attributes of my files.
However i do still wonder why the files did have the correct ownerships but not the directories.
Somewhere along the way tar lose the permission.
I'm sorry for the time lost.

@nyodas Don't worry. Please let us know if you're able to reproduce it or have any news about the problem. Thanks!


Public search

New PR for public search, after I messed up #256...
This one's a little messed up as well, as it includes bogus deletions in public/dist (I hate rebasing...), but hopefully we have a way to deal with that...

as it includes bogus deletions in public/dist (I hate rebasing...), but hopefully we have a way to deal with that...

We should just be able to regenerate the files once the front end is merged and add them in another commit before we squash.

Codecov Report

Merging #267 into master will decrease coverage by 0.02%.
The diff coverage is 30.76%.


[BUG] Add AWS Dimensions as Labels for Static Configuration

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I want to Export the AWS/RDS EBSByteBalance% Metric. Right now my configuration looks like this:
  static:
    - namespace: AWS/RDS
      name: aws_rds_byte_balance_percent
      dimensions:
        - name: DBInstanceIdentifier
          value: qa
      regions:
        - eu-central-1
      metrics:
        - name: EBSByteBalance%
          statistics:
            - 'Average'
          period: 300
          length: 3600

This results in a metric getting correctly scraped:
# HELP aws_rds_ebsbyte_balance_percent_average Help is not implemented yet.
# TYPE aws_rds_ebsbyte_balance_percent_average gauge
aws_rds_ebsbyte_balance_percent_average{account_id="<my-account-id>",dimension_DBInstanceIdentifier="qa",name="aws_rds_byte_balance_percent",region="eu-central-1"} 99

But I want to just add the DBInstanceIdentifier as a Label to my metric, instead of using it as a filter.
In Grafana+CloudWatch what I want looks like this:

I tried using value: \*, but with that the metric doesn't get exported at all. (Probably because there is no DB with such a Identifier)
Expected Behavior
It would be great to have a configuration like this:
  static:
    - namespace: AWS/RDS
      name: aws_rds_byte_balance_percent
      dimensions:
        - name: DBInstanceIdentifier
          value: *
      regions:
        - eu-central-1
      metrics:
        - name: EBSByteBalance%
          statistics:
            - 'Average'
          period: 300
          length: 3600

result in a metric where the DBInstanceIdentifier dimension is just added as a label while scraping all DBs.
I am 99.8% sure that what I want is possible with the right configuration. But I read the documentation multiple times w/o finding a hint to what I am doing wrong. Maybe someone can help and improve the docs a bit?
Thanks in advance; :)
Steps To Reproduce
No response
Anything else?
No response

Are you able to use this (in the config example for a static job:
customTags:
      - key: CustomTag
        value: CustomValue

I was able to get this working for, what sounds like, your same use case. I then remove the YACE prepended tag_aws_ and tag_ using Prometheus relabeling. I also drop the YACE prepended dimension_.
(prometheus config)
- action: labelmap #create new labels without YACE prepended "dimension_"
        regex: "dimension_(.*)"
        replacement: "${1}"
      - action: labeldrop #drop labels with YACE prepended "dimension_"
        regex: "dimension_(.*)"
      - action: labelmap #create new labels without YACE prepended "tag_aws_"
        regex: "tag_aws_(.*)"
        replacement: "${1}"
      - action: labeldrop  #drop labels with YACE prepended "tag_aws_"
        regex: "tag_aws_(.*)"
      - action: labelmap #create new labels without YACE prepended "tag_"
        regex: "tag_(.*)"
        replacement: "${1}"
      - action: labeldrop #drop labels with YACE prepended "tag_"
        regex: "tag_(.*)"


Show translucent icon for hidden apps  (-h)

[x] I made sure this command is not in the commands list

[x] I searched and didn't find this command in the listed issues

[x] I know this defaults command exists

[x] I know this command works on macOS current version

[ ] I know this command is deprecated on macOS current version

Folder: Dock

Command: defaults write com.apple.dock showhidden -bool true && killall Dock

Nice 👍 are you planning to work on this one?

What do you mean by "work on" ?

Adding the command to the defaults.yml file and maybe add a video as explained in the main readme 😉

It's alright if you don't, I just want to know if I should set you as an assignee!

I can add the command to defaults.yml but I let you do the video part ;-)


fix(chat): fix the message template editor (Issue #2275, #2268)

Description:
redesign of message template editor
Issues:

Issue #2275
Issue #2268

UI changes
https://www.figma.com/design/lwL5jneveZczAj5RLcLFH2/DIAL-Chat?node-id=12326-85343&m=dev
Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review


[stable/mysql] Run container in privileged mode

We need to modify sysctl parameters to allow MySQL pod to start successfully. To do that we need to allow privileged mode.
Is your feature request related to a problem? Please describe.
We've hit an aio-max-nr issue on a MySQL deployment on our testing environment. We're using the MySQL5.6 chart.
Describe the solution you'd like
Add to the MySQL chart the ability to add privileged and sysctl opts through values.yaml.

The specific can be solved by adding  innodb_use_native_aio=0, so I'm closing this issue.


[feature] hyper live migration based on qemu and kvm

We ( @xlgao-zju @crazykev ) are implementing hyper vm live migration based on existing kvm hypervisor and qemu. Just fire up this issue to let anyone interested know and close the old one: #35 .
For now, we have tested qemu savevm/loadvm on the same node, it works. And we also figured out virtfs is not available to support live migration. We now use devicemapper backend to hold docker image.
Here's our work to track:

[x] test qemu live migration with hyper & kvm & devicemapper
[x] test cross host live migration with qemu & hyper, need share storage of the two nodes (use NFS for now)
[ ] refactor hyper side to load a freshly migrated vm
[ ] add commands to start qemu and migrate vm in hyper

Will update this list whenever we have new progress.
[Update] 2015-12-2

This is very interesting, and we'd love to see the PR. If there is a working beta in the next two weeks, we could throw a demo at the coming DockerCon at Barcelona!

any update?

Hi, is there any progress? It is a fantastic feature!


[WIP] Move WebTorrent to its own process

Just curious, why create a new electron process and not use a webworker if you are not gonna use the UI part?

note to self: next time, google it first
http://stackoverflow.com/questions/22774076/using-web-workers-in-stand-alone-v8-javascript-engine

web workers also can't do WebRTC yet
https://bugzilla.mozilla.org/show_bug.cgi?id=922363

webrtc is also the reason why i'm using second, hidden window instead of running webtorrent in the main process. the main process is node, while the renderer processes (windows) are chromium -- so only the renderer processes have a good solid webrtc implementation available.

didn't know that, really a blocking issue!

@feross ready for review now

I'm looking at this now. I will merge it when I'm done reviewing.


when training the masked LM, the unmasked words (have label 0) were trained together with masked words?

According to the code
    def random_word(self, sentence):
        tokens = sentence.split()
        output_label = []

        for i, token in enumerate(tokens):
            prob = random.random()
            if prob < 0.15:
                # 80% randomly change token to make token
                if prob < prob * 0.8:
                    tokens[i] = self.vocab.mask_index

                # 10% randomly change token to random token
                elif prob * 0.8 <= prob < prob * 0.9:
                    tokens[i] = random.randrange(len(self.vocab))

                # 10% randomly change token to current token
                elif prob >= prob * 0.9:
                    tokens[i] = self.vocab.stoi.get(token, self.vocab.unk_index)

                output_label.append(self.vocab.stoi.get(token, self.vocab.unk_index))

            else:
                tokens[i] = self.vocab.stoi.get(token, self.vocab.unk_index)
                output_label.append(0)

        return tokens, output_label

Do we need to exclude the unmasked words when training the LM?

@coddinglxf I just solved that problem with nn.NLLLoss(ignore_index=0) which 0 is equal to pad_index. Even if we target the 0(unmasked_value), it doesn't affect to the loss of propagation

got it，thanks.
however，when batch size is large，the final LM output will be “batch_size * seq_len * vacab_size”, such matrix is too big. maybe we can record the index of masked works when preprocessing the text， and use index technology to save memory.

@coddinglxf that's what I thought it first, but can't implement it efficiently as much as GPU computation time. If you have any idea please implement and pull request plez :)
It would be really cool to do it 👍

@codertimo
the output of transformer block will be : (batch, seq,  hidden)
step 1:
we can reshape the transformer output as : (batch*seq,  hidden)
step 2: index the masked words
for 0-th sentence， the i-th masked word will have a index:  0 * seq + i
for j-th sentence,    the i-th masked word will have a index:   j * seq + i
combine step1 and step2:
we can get the hidden output of masked words (using fancy indexing)
the question is we can not do this with multi-gpu

@coddinglxf I just solved that problem with nn.NLLLoss(ignore_index=0) which 0 is equal to pad_index. Even if we target the 0(unmasked_value), it doesn't affect to the loss of propagation

why here output_label is 0, but not "tokens[i]"?  if we set output_label =0, that means, just 15% of training data is used to train MaskedLM?
tokens[i] = self.vocab.stoi.get(token, self.vocab.unk_index) output_label.append(0)

@leon-cas yes #36 it's solved with your question


Composite quadrature rule

In this pull request, I add

class CompositeTriangle(Q_base) which contain all quadrature points and weights in all uniformly subdivided triangles;
a file test_CompQuad.py to test this quadrature rule. There are two tests in this file, corresponding to the integral over the region below a straight line and inside a circle.

Take a look at how I refactored the doc and the functionality to work with our documentation generater (sphinx)  and our testing framework  (py.test).
It would be good to test something beyond just using the rule. See for example what @tridelat does in test_spatialtools.py. You could compute the order of converengence of the composite quadture rules with respect to hk ans assert that it's close to the expected value, for example. Also you could check that the rules are exact for functions for which they should be exact.

In this commit, several changes are made:

Unittest.Testcase is used. And one can use pytest to run it.
The expectation for test_line and test_circle is the average convergence > 1
For polynomials the usual quadrature rules should be equal to the composite rule. Low order polynomials like 1, x, xy, xy^2  are tested.

In this commit, I am trying to add a composite quadrature rule for the reference tetrahedron. The Freudenthal-Bey refinement algorithm is used to get 8 sub-tetrahedron for each tetrahedron. A test file is also added, where

Unittest.Testcase is used. And one can use pytest to run it.
The expectation for test_plane and test_sphere is the average convergence > 1. In face, for those two tests, the convergence is > 1.5
For polynomials the composite quadrature rule gives the same results as the usual gauss quadrature rules. Low order polynomials like 1, x, xy, xyz are tested.

This is looking good. We should move the tests into the root proteus/tests directory. You can do that with git mv on any files in the yy_pracitice subdir.  We don't need the  yy_practice subdir.

I agree with you. In this commit, I have moved the test_composite_quad folder to proteus/proteus/tests/test_composite_quad

This is looking good. Let's do a comparison on the 2D vortex test: proteus/tests/levelset/vortex2D

In this commit, I am beginning to use 2D composite quadrature rule for the 2D vortex test. One comparison about the solutiond at the final time is shown in this figure

The 0-level-set becomes much smooth. I will do more tests on this problem. One suggestion is that maybe we can use composite quadrature only around the level set curve by labeling cells before the loop and considering the composite quadrature rule for those particular cells. The cost of current computation does not increase a lot.


add BUILD file for grpc

Add bazel rule to generate census proto library that can be used by
other repos that use gRPC.

Do you need a BUILD file only for this? Why do you not use the main BUILD file from the root directory of this project.

The reason for not using main BUILD file has to do with bazel and grpc limitation (I've asked gRPC folks to file a bug). gRPC proto library will only build when the proto and BUILD files are in the same folder. I can move this BUILD file to the main BUILD when this limitation is removed.

@makdharma It looks like the second BUILD file is causing a build failure.  Would it be possible to access this repository using new_http_archive instead, and then specify the new BUILD file with the build_file field?

@makdharma Is this change still needed for gRPC?

Does this https://github.com/census-instrumentation/opencensus-proto/pull/38 resolve your issue?

I'm closing this PR for now, since it isn't active and #34 added a build file in the stats directory.


Unable to parse PDF

First of all, thanks for this library! I am see the following problem with a PDF that I am trying to open:
gems/pdf-reader-0.10.0/lib/pdf/reader/pages_strategy.rb:354:in `content_stream': Unknown font F1 (PDF::Reader::MalformedPDFError)
I would submit the PDF so you can test it, however, it contains alot of my personal information (financial data, SSN, etc).
Let me know what other information I can give you.  The file is an ADP generated paystub (iPay)
Thanks,
Benny

I'm going to close this issue for now. Please re-open it if the issue still exists in the 1.0.0.rc1 release


Width fix

Is there a specific reason you removed the @grant none?

Apologies! Greasemonkey gave me a notification about @grant none and
removed it automatically. The notification disappeared before I could read
it. The script still worked, so I didn't put it back.
On 4 Apr 2016 21:43, "Ventero"<EMAIL_ADDRESS>wrote:

Is there a specific reason you removed the @grant none?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/Ventero/kongregate-userscripts/pull/2#issuecomment-205486715

Squashed, whitespace changes removed and cherry-picked. Thanks!


Languages Implementation Issue?

When I inspect elements on the English pages, I see things as I would expect.
However, when I go to view source, the source is shown for the French page (french body text, etc..).


Update Font advanced import dialog character ranges to Unicode 14.

Updates character ranges to the latest Unicode release. Also removes extra empty column for the character range selector Tree.

Thanks!


Fix tabs shortcode CSS

It seems in a Docsy/Hugo update that some CSS values changed and modified the look of the tabs.
If you look at the left side of the tabs (and right on mouse hover) you can see they're not connected to the container and are floating above.
Before:

After:

/lgtm

/area web-development
/approve


powershell: disable dotnet telemetry

Motivation for this change
fixes https://github.com/NixOS/nixpkgs/issues/74516
cc: @stefano-m
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[X] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Thank you for this @marsam
Is it going to be merged into the 19.09 release branch too?


Token Saver Should not save to C:\

In rare cases there will be no access rights to C:\ to save the JSON.
Please make this Application.persistentDataPath by default.

I just added this to TokenSaver.cs
 private static string FilePathCorrected = Application.persistentDataPath + "/SpotifyTokens.json";
Then referenced FilePathCorrected Everywhere

By default, TokenSaver does save to Application.persistenDataPath
(TokenSaver.cs Line 17)
private static string FilePath { get { return !string.IsNullOrEmpty(FileDirectory) ? Path.Combine(FileDirectory, FileName) : Path.Combine(BackupFileDirectory, FileName); } }
That will return the directory that you want or the default one which is Application.persistantDataPath + "/SpotifyTokens.json". The "FileDirectory" property is set through SpotifyService as the "AuthTokenBaseDirectory" property so as long as you're not setting that to anywhere yourself, it will use it.

For some reason even if blank it was trying to save to C:\
Gerard Slee
Director
+2784 689 0159
<EMAIL_ADDRESS>117 Hatfield Street, Gardens, Cape Town, 8001, South Africa
Twitter | Facebook | Instagram | YouTube | LinkedIn
On 15 Feb 2019, 17:20 +0200, Josh Shepherd<EMAIL_ADDRESS>wrote:

By default, TokenSaver does save to Application.persistenDataPath
(TokenSaver.cs Line 17)
private static string FilePath { get { return !string.IsNullOrEmpty(FileDirectory) ? Path.Combine(FileDirectory, FileName) : Path.Combine(BackupFileDirectory, FileName); } }
That will return the directory that you want or the default one which is Application.persistantDataPath + "/SpotifyTokens.json". The "FileDirectory" property is set through SpotifyService as the "AuthTokenBaseDirectory" property so as long as you're not setting that to anywhere yourself, it will use it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Alright, this one was weird. Turns out using Path.Combine on a directory and a file name with a / at the start doesn't combine it correctly. This is fixed now


Optimize the output of portsdiff command

Fixes https://github.com/microsoft/vcpkg/issues/40030
The portsdiff command output lacks a space between the port name and version. This modification adds a space between the output name and version.
before fixing：

After：

Thanks!


Ensure that ignore works for all notify requests

Move the check to ignore into notify, this way when the :ignore key is set in config it will work for all requests, not just those handled through the plug.
Closes #38

It looks like the rest of the ebert errors are existing errors or intentional (it does not like the IO.inspect call!). Let me know if there's anything else that needs to be done to get this PR merged in :)

Thanks so much for merging this in!


Opt-in error reporting via Sentry.io

Closes #1730.
This pull request implements opt-in automated error reporting via Sentry.io.
Everything is fully documented, I've gone through and prevented some exceptions we certainly don't want to see from being reported, and done quite a bit of under-the-hood changes to exceptions and error handling in FOSSBilling to clean up various rough-edges.
Both the Sentry SDK and the Sentry.io server will automatically scrub sensitive information (such as PII), but we can at any time add to this list if we discover something that's not already being scrubbed.
Our Sentry.io organization is configured to require this scrubbing to be enabled on all projects including IP address scrubbing.
Changes to error handling

PHP's error handling is now completely disabled as FOSSBilling already logged items, resulting in duplicated log entries.
I've cleaned up and improved the actual error logging in FOSSBilling so that the messages are clearer and more informative.
Box_Exception has been replaced with FOSSBilling\Exception and temporarily acts as an alias to it.
I have added a new FOSSBilling\InformationException class which is intended for use with informational exceptions, such as informing the user a product is out of stock or that their password isn't strong enough. These will not be reported to Sentry.io and I have updated roughly 200 exceptions inside of FOSSBilling to use this new class.
Both error pages will show the newly introduced instance ID to aid in troubleshooting.
Some various changes under-the-hood to streamline the integration with Sentry.io.
Changes to the config structure.

Error-reporting specific changes / additions

We can exclude errors from being reported via the \FOSSBillingErrorPage class now.
The installer now has an optional selection for error reporting, and some basic info on it with links to the documentation where things are explained in more detail.
I've added in a new "instance ID" system which is very basic. More info is in a below header.
Sentry.io is now integrated (duh!).
The bug report form now asks for the instance ID, though I didn't make it a requirement since error-reporting is opt-in and otherwise the ID is of no help for us.
The system settings page now includes the instance ID under "about" and there's a new tab for error reporting.
There are now two new messages that will displayed after an update. One for old installs that didn't support error reporting and another for if we change the behavior of error reporting. (See the screenshots for these)
Our build workflow adds in the Sentry DSN, so preview builds and releases should already have it included. This step is only run on our own repo or on PRs that are from our repo.

What is reported
(This is just a copy & paste from the docs PR, which also means we have this listed for system administrators to see)

Information about the request such as the request headers and URL being request.
The PHP version in use.
All types of errors that may occur in PHP applications (This does not include anything sent via error_log).
Exceptions thrown within FOSSBilling (with some exceptions for informational exceptions).
The version of FOSSBilling that's in use.
The unique instance ID for your FOSSBilling install.
If we are able to detect it, what type of web-server that's being used.
Error-specific information such as the path where it occurred and the stacktrace that led to the error.
Information about the host server such as it's hostname, operating system, and OS version information.

Screenshots
FOSSBillng

Sentry.io

FOSSBilling instance IDs
This pull request will implement a very simple system to give each FOSSBilling instance a unique ID.
These IDs are RFC4122 version 4 UUIDs which are randomly generated and have an insanely low chance of collisions.
Instance IDs are stored in the config file under info->instance_id.
The instance ID is passed along to Sentry.io, so it can be used within the dashboard to track or filter issues for a specifically installed instance. I have also added a prompt to our bug report form to request this.
By having these IDs sent via bug reports, we will be able to efficient find captured errors for a given instance, look up stack traces, and even get basic info on the environment that is being used without needing to have back-and-forth conversations with the reporter. Historically it can be rather difficult to get this type of info from some people (either because they are lazy or don't understand) so hopefully being able to get all of that after only being given the instance ID will eliminate some pain points that can occur when trying to provide support.
Instance IDs are found both in the system settings and on the error page.
Error filtering and capturing
All PHP errors and exceptions are recorded excluding anything sent via error_log and what's listed below:
Errors that aren't reported

Errors in the FOSSBilling loader (these are almost exclusively issues specific to a given install)
2001: Incomplete server manager configurations.
3001: Incomplete domain registrar configurations.
4001: Incomplete payment gateway configurations.
InformationException exception classes are ignore as they are intended to be informational only (IE: "This product is out of stock") (I have changed ~200 of our exceptions to be this).
Errors marked to not be reported via the ErrorPage class.

Significant / breaking changes

All new FOSSBilling\Exception and FOSSBilling\InformationException exception classes. Box_Exception is temporarily aliased to FOSSBilling\Exceptionand our internal code has been adjusted to reference the new classes.
Modifications to the config layout regarding debug options. This change was done to make this a bit cleaner by grouping related items under a debug_and_monitoring key.
The salt config key has been moved under info rather than having no parent key name.

This PR is now ready to be reviewed and tested. The PR for the documentation should also be reviewed: https://github.com/FOSSBilling/fossbilling.org/pull/165
Here's a zip with a build that has the DSN configured for error reporting, the preview zip off of the action on GH won't include it:
FOSSBilling - Error Reporting.zip

I've attached the most up-to-date preview of the error reporting build here:
FOSSBilling Error Reporting RC2.zip

I've had to drop support for translations on the error page.
It was proving to be problematic in my testing and we can't have the error page being unreliable.
If someone can find a way to get it working reliably that would be great, but otherwise I think we will have to live with that page only being available in English (which is fine, really. It shouldn't be displayed all that often and most browsers have built-in translation tools)

This is now the most up-to-date preview build with error reporting. It incorporates the suggestions by @admdly, has a few additional fixes and improvements, and should better handle errors (mostly during the initial loading process)
FOSSBilling Error Reporting RC3.zip

Woo, there is a lot in this PR, it takes a while just to get through it all without actually writing any of it!
It's great work, and I can't find any issues :)
Regarding the translateability of the error pages, yes they shouldn't be seen very often (preferably never) but I do think it's a bit of an issue, and I personally hate it when I'm looking at anything in one language and then a page (error or not) pops up in another one. Yes this PR should be merged now, but I think that this should be added as an issue and roadmapped for a future release.

and I personally hate it when I'm looking at anything in one language and then a page (error or not) pops up in another one.

Indeed, but the problem is that it needs to work when FOSSBilling is functional enough for translations to be working and setup, but it also needs to work when we haven't gotten that far. Previously I was able to get that to be the case by defining a "polyfill" for the __trans function, however with the changes in this PR that started causing errors and I was unable to find a technique to implement it where it wouldn't cause things to break when __trans wasn't already defined.
One saving grace, however, is that actual translatable exceptions will still be translated if FOSSBilling is able to get past it's initial loading steps & those messages will be visible on the page in their translated state


CrateDB vector: Refactor SQLAlchemy data model to provide two storage strategies

About
This patch makes a stab at GH-12, aiming to implement a more advanced data model for storing collections vs. associated embeddings. As suggested by @ckurze, this will follow a "embedding-table-per-collection" strategy, instead of storing all embedding vectors into the same database table.
Installation
🚧 The preview version can be installed directly from the branch embedding-table-per-collection on GitHub. 🚧
pip install 'langchain[cratedb,openai] @ git+https://github.com/crate-workbench/langchain.git@embedding-table-per-collection#subdirectory=libs/langchain'

Usage
The default storage strategy is LANGCHAIN_PGVECTOR, which is the same as before. To configure an alternative storage strategy, invoke this snippet before doing any other operations on CrateDBVectorSearch:
from langchain.vectorstores import CrateDBVectorSearch
from langchain.vectorstores.cratedb import StorageStrategy

CrateDBVectorSearch.configure(
    storage_strategy=StorageStrategy.EMBEDDING_TABLE_PER_COLLECTION
)

Details
Those strategies are implemented. The default is LANGCHAIN_PGVECTOR.

StorageStrategy.LANGCHAIN_PGVECTOR
Reflects the vanilla way the pgvector adapter manages the data model: There is a single collection table and a single embedding table.

StorageStrategy.EMBEDDING_TABLE_PER_COLLECTION
Reflects a more advanced way to manage the data model: There is a single collection table, and multiple embedding tables, one per collection.

Backlog

[ ] Add test case exercising different vector store handles with different dimension sizes at the same time.
[ ] Multi-collection querying as introduced by GH-15 is currently not supported by the new EMBEDDING_TABLE_PER_COLLECTION storage strategy.
[ ] Think about amending the vanilla data model to also store the associated vector/embedding table name within the CollectionStore database entity, in a new column, embedding_table.
[ ] For the second strategy, get rid of the collection table at all, no index table is needed.
[ ] Also on the first strategy, get rid of the collection-level cmetadata field. It isn't used at all within the other implementations, so it is considered to be legacy.
[ ] Add documentation on behalf of Jupyter Notebook and .mdx file.
[ ] Collect and evaluate user feedback.

Other than that I did some monkey testing and found the strategy reliable.

When you delete a collection, would it make sense to drop the table of this collection when using this new storage strategy?

That's a good suggestion, thank you.


dependency mock_outputs workflow for output changes

Hi everyone!
We have experienced the following error while trying to run terragrunt plan-all --terragrunt-source-update --terragrunt-non-interactive --terragrunt-include-external-dependencies in a lambda folder, which have dependency with sqs.
Unsupported attribute; This object does not have an attribute named “Sqs-Two_name”., and 1 other diagnostic(s)
[terragrunt] 2019/11/01 17:53:59 Unable to determine underlying exit code, so Terragrunt will exit with error code 1
In the tfstate, we have the Sqs-One, but the plan is going to create a new resource (Sqs-Two).
Could there be a flag that always force you to use the mock_outputs in the plan?
Something like this
mock_outputs_forced_terraform_commands = [“plan”]
Our project has the following structure:
├── live
│   ├── lambda
│   │   ├── Lambda_Sqs-One_Dependent
│   │   │   └── terragrunt.hcl
│   │   └── Lambda_Sqs-Two_Dependent
│   │       └── terragrunt.hcl
│   └── sqs
│       └── terragrunt.hcl
└── terraform
    ├── lambda
    │   ├── Lambda_Sqs-One_Dependent
    │   │   └── main.tf
    │   └── Lambda_Sqs-Two_Dependent
    │       └── main.tf
    └── sqs
        ├── Sqs-One.tf
        └── Sqs-Two.tf

Sqs-One and Sqs-Two have:
module "Sqs-One" {
  source = "git@github.com:Enterprise/terraform-aws-sqs.git?ref=tags/2.1"
}

output "Sqs-One_name" {
  value       = module.Sqs-One.sqs_name
}

module "Sqs-Two" {
  source = "git@github.com:Enterprise/terraform-aws-sqs.git?ref=tags/2.1"
}

output "Sqs-Two_name" {
  value       = module.Sqs-Two.sqs_name
}

For each lambda terragrunt.hcl, we have:
terraform {
  source = "path_to_source"
}

dependency "sqs" {
  config_path = "../../sqs"

  mock_outputs = {
    Sqs-Two_name = "mock_name"
  }
}

inputs = {
  dlq_name  = dependency.sqs.outputs.Sqs-Two_name
}

That seems reasonable. Will be happy to review a PR!

Doing something like this won't help ?
dependency "sqs" {
  config_path = "../../sqs"

  mock_outputs = {
    Sqs-Two_name = "mock_name"
  }

  skip_outputs = true
  mock_outputs_allowed_terraform_commands = ["plan"]
}

With skip_outputs seted to true, Terragrunt won't pull in output from states.

IIRC, setting skip_outputs = true will skip pulling outputs for terraform apply as well.

We have the same issue! When we add a new output, consumed by another dependent one, it breaks our CI/CD pipeline because mock_outputs are not taken into account (because the state already exists).
We were thinking of another solution: create a partial_mock_outputs = true option. It would default to false to be compatible with the current behavior. When true, it would merge the actual outputs with the mocked ones.
What do you think?

When we add a new output, consumed by another dependent one, it breaks our CI/CD pipeline because mock_outputs are not taken into account (because the state already exists).
We were thinking of another solution: create a partial_mock_outputs = true option. It would default to false to be compatible with the current behavior. When true, it would merge the actual outputs with the mocked ones.
What do you think?

I think this has the element of surprise. It feels like a smell to me to be depending on mock_outputs to workaround that issue, and it won't be immediately obvious when things are coming from mock_outputs and when things are coming from the actual outputs.
In general, plan-all is simply broken for day to day use for various reasons including this one and we generally discourage it's usage now, especially in a CI/CD pipeline. We are actively investigating an alternative solution for the use case, but that is not going to come any time soon. It is simply not an easy feat to have plans on multiple state files because terraform doesn't give us a lot of functionality to work with that (since that is not really the main use case they are focusing on).
Nevertheless, if you need this kind of functionality right now, there are two alternative workarounds to this that doesn't require any changes to terragrunt:
1.) Use read_terragrunt_config to read in the dependency, and "pack" it using merge in the inputs block. E.g., if you had the following:
dependency.hcl
locals {
  default_outputs = {
    Sqs-Two_name = "mock_name"
  }
}

dependency "sqs" {
  config_path = "../../sqs"
}

inputs = {
  final_out = merge(dependency.sqs.outputs, local.default_outputs)
}

You can read this in in your other configs as follows:
locals {
  sqs_dep = read_terragrunt_config("/path/to/dependency.hcl").inputs.final_out
}

2.) Use the lookup terraform function to provide a default on use. E.g.,
inputs = {
  dlq_name  = lookup(dependency.sqs.outputs, "Sqs-Two_name", "mock_name")
}

@yorinasub17 The problem I see with the solution you proposed is that for whatever reason if you forgot the output, it will create a resource with mock_name in it and would be problematic to deploy.
I honestly think the solution should be 100% managed by TG with some options of the mock_outputs as it works well except during the apply.

That is a fair criticism, and we would like to improve mock outputs too, but it is a bit hard to come up with a viable solution that is maintainable and makes sense. If you have suggestions on how to improve mock_outputs to fit the workflows described, an RFC + PR with suggested fixes are welcome!
FWIW, the original intention of mock outputs is to allow running validate. That is the primary use case of the original design, and so all the other use cases people want out of mock outputs is going to be awkward by design.

Hi @yorinasub17
Is it possible to propose a PR for this ? does it need a RFC ?
For us, with this issue, it's impossible to use plan. I saw, the proposed workaround, I will try it and see if it works for our use case. Thanks for that ;-)
In our use case, it's clear in the data when mocks values are used.
And we only activate it, on some commands, and never on applies.
So for us, it's not confusing and it helps a lot to have the possibility to plan before apply.
As I understood, with the workaround, we will loose the protection on apply command.
I was thinking on something like proposed by @thomvaill

Adding a parameter, with explicit name => merge_state_and_mocks_outputs
As default is should be set to false
If true, terragrunt will add missing mocks, in the state outputs

I can propose a PR, but before working on it, I prefer check if it's something, you can be interested on.
Thanks

@gregorycuellar I'm still not a fan of introducing another configuration option, but we also don't have anything to fulfill this need in the pipe, so open to accepting a PR to add this option if you are up for it.

Hi @yorinasub17
I made the PR #1765, in order to fix this.
I think, I respected all the guideline and spirit of the project, if not, of course don't hesitate to point out what need to be fixed and I will do it.
Thank you

In my case I have the following scenario.
..<higher-level-tree>
                /apps
                     /app1
                        base/
                           terragrunt.hcl
                        top/
                           terragrunt.hcl
                     /app2

where /apps/app1/base/terragrunt.hcl looks for parent .hcls to inherit some configuration and /top/terragrunt.hcl depends on /base/terragrunt.hcl
From a CI/CD perspective we would like to be able to validate quickly using the mock_outputs feature but for the plan we really would like to exercise the values retrieved from the /base/terragrunt.hcl (after execution).  I am not sure if there are other ways to make the plan as close to reality as possible without mocking.

https://github.com/gruntwork-io/terragrunt/releases/tag/v0.31.4 was released with a new feature flag (merge_state_and_mocks_outputs) on the dependency block which will force terragrunt to merge mock_outputs with the dependency output that was fetched from state.

I have upgraded and just get the message Unsupported argument; An argument named "merge_state_and_mocks_outputs" is not expected here.:
dependency "root" {
  config_path = find_in_parent_folders()
  mock_outputs = {
    [...]
  }
  merge_state_and_mocks_outputs = true
  mock_outputs_allowed_terraform_commands = ["validate", "fmt"]
}

It looks like the correct name for the parameter is mock_outputs_merge_with_state. It's correctly listed in the docs, but was wrong in the release notes - I've just fixed that.

@infraredgirl Ah, thanks, using that works :)

mock_outputs_merge_strategy_with_state doesn't work. Still got error:
 Unsupported attribute; This object does not have an attribute named


chore: remove support for browser field in config

Summary
This allows us to remove the dependency on browser-resolve. As the added deprecation message states, people can specify it as resolver if they want to use it.
Should make sure to specify the upgrade path in the blog post when we release Jest 26.

Test plan
Green CI? 😀 This removes a feature, so not much to test.

Codecov Report

Merging #9943 into master will not change coverage.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##           master    #9943   +/-   ##
=======================================
  Coverage   64.23%   64.23%
=======================================
  Files         292      292
  Lines       12466    12466
  Branches     3075     3076    +1
=======================================
  Hits         8007     8007
- Misses       3810     3811    +1
+ Partials      649      648    -1

Impacted Files
Coverage Δ

packages/jest-config/src/Defaults.ts
100.00% <ø> (ø)

packages/jest-config/src/Deprecated.ts
57.14% <0.00%> (-9.53%)
:arrow_down:

packages/jest-config/src/Descriptions.ts
100.00% <ø> (ø)

packages/jest-config/src/ValidConfig.ts
100.00% <ø> (ø)

packages/jest-config/src/index.ts
11.59% <ø> (ø)

packages/jest-config/src/normalize.ts
76.66% <ø> (ø)

packages/jest-resolve/src/index.ts
44.78% <ø> (ø)

packages/jest-runtime/src/index.ts
57.93% <ø> (ø)

packages/jest-resolve/src/defaultResolver.ts
77.50% <100.00%> (+1.89%)
:arrow_up:

Continue to review full report at Codecov.

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

What does the upgrade path look like for folks who rely on this?


Update content score to a single value.

Add weight attribute.

Pull Request Test Coverage Report for Build 289

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

Totals

Change from base Build 287:
0.0%

Covered Lines:
454

Relevant Lines:
463

💛  - Coveralls


Crashing with Iris Shaders

Iris Shaders
The game crashes the instant you try to open a portal to the nether when using Iris.
Crash Report:

Portality.zs

Because the Iris compatibility PR hasn't been merged https://github.com/IrisShaders/Iris/pull/425


feat: create footer

I created a single-content type to footer bar application.

I canceled this request to not pollute the main branch


Update data-box-object-size-limits.md

Update block blob size limit to 14TiB for Data Box

@alkohli
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Court72 The Data Box content portfolio has transitioned to Akash Dubey's team. Can we please assign this work item to them as well? Thanks!

@akashdubey-ms
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

@stevenmatthew
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Court72 @vupadhye-msft  Howdy, this has actually been addressed in a different PR within our private repo.  Thanks for the reminder on this!

#please-close


Add a window rule to render while unfocused

Adds a renderunfocused window rule.
Needs testing
fixes #2701

Tried it on glxgears with windowrulev2 = renderunfocused, title:^(glxgears)$, the rule was recognized but the FPS still dropped after moving it to an inactive workspace.

it should drop to ~15

seems to be working for me
the point isnt to render at max fps all the time is it? The point is to avoid apps stalling

Oh ok - then it works, it was about 15fps.
Do you think we could get a configuration option for that?
I see that it's a timeout in milliseconds here: https://github.com/hyprwm/Hyprland/pull/7582/files#diff-dbcd1ce1e0b916536f17c999473ad9e22c014646ddb2df15c45d726884a3481cR131
I think it'd be enough to make that value configurable.

prolly yea

the point isnt to render at max fps all the time is it? The point is to avoid apps stalling

Yeah not max fps all the time but some games (like Elden Ring as mentioned in the issue) are fps sensitive and being able to configure it would be great.

misc:render_unfocused_fps

Works like a charm!

FWIW, I had Elden Ring in the background for about 15 minutes @ 6634117f868bf087b93d92d9349b5200b78cc609 with the hardcoded 15fps while switching back and forth between workspaces, and it didn't trigger the framerate detection. The ability to configure that value is very welcome though. Thanks!

great work! fixed!


Make documentation version ready

Closes #36

[X] Generate a new documentation for each version
[x] Create link to newest version

Will it work on any linux?
Xavier Detant
On 6 Dec 2016 17:00, "Vincent Demeester"<EMAIL_ADDRESS>wrote:

@vdemeester approved this pull request.
Just a nit/question but LGTM 🐸
In .travis/publish-documentation.sh
https://github.com/FaustXVI/junit5-docker/pull/40#pullrequestreview-11617971
:

@@ -0,0 +1,88 @@
+#!/bin/bash

#!/usr/bin/env bash for nix ? 👼
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/FaustXVI/junit5-docker/pull/40#pullrequestreview-11617971,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-EHyEy1u5GAdSFmjyd4b8d25PZlgRBks5rFYaUgaJpZM4LFTvd
.

@FaustXVI yes, more than /bin/bash :angel:


How do you make this plugin work

I've installed the plugin from joplin and it is active. But, when I edit/enter code I do not see any line number nor a black background. What needs to be done to make it work?

Please try updating to the latest version.


help cmd as per #141

Can you remove 5441068 please?

Looks good from a quick skim, but you didn't cargo fmt. I wonder if we could have something more generic (eventually?).

Can you please elaborate a little more about making it more generic? ( Something like importing commands from a separate file? )

I was more musing about the future, this is the right approach for now!
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
Le lundi 23 août 2021 à 7:04 PM, Vaibhav Garg @.***> a écrit :

Looks good from a quick skim, but you didn't cargo fmt. I wonder if we could have something more generic (eventually?).

Can you please elaborate a little more about making it more generic? ( Something like importing commands from a separate file? )
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Some more nits

made required changes & clubbed in single commit...

Looks good to me, utACK 9238ecb41752e097443d0bc117df35ebd4ac932e

I noticed just now that it's a bit hard to navigate - that's my fault, I didn't notice when I wrote the spec for the command. For the future, maybe we should switch to having the name of the command as the key, like:
[
  {
    "getinfo": { ... }
  },
  {
    "getrevocationtxs": { ... }
  }
]

But for now it's good as it is! Thanks :)
ACK acbc03f436ca112bc8addefbc1882c80064c118e


[Android] Support Android 34 (take 2)

Re-lands https://github.com/flutter/flutter/pull/137191.
The fix for the issue causing that PR to be reverted was tested in this PR but ultimately landed separately in https://github.com/flutter/flutter/pull/138093.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

The reason that the first PR seemed to have failed is due to these warnings:
[2023-11-07 10:28:15.561422] [STDOUT] run:stderr: /opt/s/w/ir/cache/gradle/caches/transforms-3/2febba88601fdfcd50f9b2931eeb9213/transformed/rules-1.2.0-api.jar(/androidx/test/rule/ActivityTestRule.class): warning: [classfile] MethodParameters attribute introduced in version 52.0 class files is ignored in version 51.0 class files
[2023-11-07 10:28:15.563314] [STDOUT] run:stderr: error: warnings found and -Werror specified
[2023-11-07 10:28:15.601735] [STDOUT] run:stderr: /opt/s/w/ir/x/w/recipe_cleanup/tmpfr_3x8zx/flutter sdk/packages/integration_test/android/src/main/java/dev/flutter/plugins/integration_test/FlutterTestRunner.java:23: warning: [rawtypes] found raw type: Class
[2023-11-07 10:28:15.601983] [STDOUT] run:stderr:   final Class testClass;
[2023-11-07 10:28:15.602130] [STDOUT] run:stderr:         ^
[2023-11-07 10:28:15.602277] [STDOUT] run:stderr:   missing type arguments for generic class Class<T>
[2023-11-07 10:28:15.602581] [STDOUT] run:stderr:   where T is a type-variable:
[2023-11-07 10:28:15.602765] [STDOUT] run:stderr:     T extends Object declared in class Class
[2023-11-07 10:28:15.604360] [STDOUT] run:stderr: /opt/s/w/ir/x/w/recipe_cleanup/tmpfr_3x8zx/flutter sdk/packages/integration_test/android/src/main/java/dev/flutter/plugins/integration_test/FlutterTestRunner.java:35: warning: [deprecation] newInstance() in Class has been deprecated
[2023-11-07 10:28:15.604502] [STDOUT] run:stderr:           Object instance = testClass.newInstance();
[2023-11-07 10:28:15.604594] [STDOUT] run:stderr:                                      ^
[2023-11-07 10:28:15.604674] [STDOUT] run:stderr:   where T is a type-variable:
[2023-11-07 10:28:15.604747] [STDOUT] run:stderr:     T extends Object declared in class Class
[2023-11-07 10:28:15.604911] [STDOUT] run:stderr: 1 error
[2023-11-07 10:28:15.605002] [STDOUT] run:stderr: 3 warnings

This LGTM, but lets try to land it at the same time as #138183

"At the same time" as in after?

I was hoping the checks would pass on that PR. I think its actually fine to land this now.
The motivation for the comment is that I believe that if one of them lands first, the tests will start downloading the other version, and it would be better to just have the correct version already installed instead of downloading it. For example (this is at least my understanding), every check we change to use android api 34 here will need to download it when it tries to run the test (because it won't have it pre-installed from CIPD).


Server crash if _ is enabled

Hey there.
I like this plugin but I'd like to be able to use _ in urls so I made a little plugin myself. https://github.com/DatDraggy/underscores-in-url
When you have both plugins enabled and enter an unknown yourls url, more and more apache processes pop up until the server dies!
This happens with any kind of url. sho.rt/swag and sho.rt/sw_ag
The same thing happens when you just add an underscore to the charset in functions.php
No error in my logs though.
Any ideas?
https://puu.sh/udeK1/91371d81a5.png
https://puu.sh/udeMH/e879c4325e.png

Okay so apparently that's why my https://github.com/DatDraggy/anti-youtube-preview isn't working either. Thats so strange...

I honestly don't know any reason why it won't work with this plugin.
Does the url exist? Tried your plugin with vanilla YOURLS?

For example, sho.rt/notexist doesnt exist in the database. When you have the 404-redirect and the underscore plugin active/underscore in charset and try to go to sho.rt/notexist, it will load indefinitely and it'll spawn more and more apache servers until it reaches the limit and crashes.
However if the URL exists, everything works fine. Even with _ in the shortened URL.
Yes. I tried a fresh install, cleared cache and tried to replicate. It just brings me to the root domain like it should when only my plugin is activated and I enter an invalid url. When I put your plugin in there and activate it too, the problem occurs again. When I remove mine, the 404 header is being sent.
That's really strange...

What are the response headers when the problem occurs?

None because the page just doesn't load until apache says "Fuck it. I'm not doing this anymore". https://puu.sh/uhNTM/0f27c52a2b.png
Screenshot of the processes on my test server when I enter an invalid url: https://puu.sh/uhNNR/e59642da66.png

Sorry for the late reply. Totally forgot about this.
I believe this is a problem with YOURLS though.

Lol no worries xP
Hmhmm that's the only thing I can think of as well. Really strange. I'll just leave it inside my code then and just add a little "incompatible with blabla" notice on my plugin.

Okay :)


Update request: aider-chat 0.66.0 → 0.69.1

Package Information
https://github.com/NixOS/nixpkgs/blob/nixos-unstable/pkgs/by-name/ai/aider-chat/package.nix#L186

Package name: aider-chat
Latest released version: 0.69.1
Current version on the unstable channel: 0.66.0
Current version on the stable/release channel: 0.62.0

Checklist

[ ] Checked the nixpkgs pull requests

Notify maintainers

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

Looks like something related to archlinux is blocking this version update - https://github.com/NixOS/nixpkgs/runs/34548069769


self.date is undefined

On mdPickers.js line 241 self.date.daysInMonth() is throwing an error.
It seems like CalendarCtrl is not getting the values from datepicker (which is the alias of DatePickerCtrl).
I could make it work by adding self.date = $scope.$parent.datepicker.date in CalendarCtrl.
versions:
angular 1.6.0
angular-material 1.1.0
mdPickers 0.7.5
moment 2.17.1

Sale versions, same issue...

Same issue

I believe the issue is a breaking change angular 1.6 made:
"Disabling (by default) the pre-assignment of bindings on controller instances, which helps with support for native ES6 classes."
https://docs.angularjs.org/guide/migration#migrate1.5to1.6-ng-services-$compile
All the logic that is currently in the constructor for the directive controllers needs to be moved into an $onInit function.

https://github.com/alenaksu/mdPickers/pull/168

same here

@danieloprado
I was able to temporary workarround installing the fork :
bower install -f --save https://github.com/andrea-vega/mdPickers.git#7c8f32fac12a01ad66e6728b603dcf8c7e86c0ce
@andrea-vega it worked like a charm. Thanks so much :)

@felipegouveiae great! Glad to help  :)

Great Thank you very much.

Does anyone know if this fixes the time picker, too? I've got the following error:
TypeError: Cannot read property 'hours' of undefined
    at ClockCtrl.init (mdPickers.js:692)
    at new ClockCtrl (mdPickers.js:706)
    at Object.invoke (angular.js:5003)
    at $controllerInit (angular.js:10862)
    at nodeLinkFn (angular.js:9739)
    at compositeLinkFn (angular.js:9048)
    at publicLinkFn (angular.js:8913)
    at boundTranscludeFn (angular.js:9091)
    at Object.controllersBoundTransclude [as transclude] (angular.js:9858)
    at angular.js:31797


requests.exceptions.MissingSchema: Invalid URL 'None': No schema supplied. Perhaps you meant http://None?

Not sure what I'm doing wrong, I have 2 notifications in a specific job (start and end), the first one works fine and the second one fails with this:
Traceback (most recent call last):   File "/app/main.py", line 200, in <module>     main()   File "/app/main.py", line 195, in main     notify_slack(payload)   File "/app/main.py", line 169, in notify_slack     requests.post(url, data=payload, headers=headers)   File "/app/requests/api.py", line 117, in post     return request('post', url, data=data, json=json, **kwargs)   File "/app/requests/api.py", line 61, in request     return session.request(method=method, url=url, **kwargs)   File "/app/requests/sessions.py", line 528, in request     prep = self.prepare_request(req)   File "/app/requests/sessions.py", line 466, in prepare_request     hooks=merge_hooks(request.hooks, self.hooks),   File "/app/requests/models.py", line 316, in prepare     self.prepare_url(url, params)   File "/app/requests/models.py", line [39](https://github.com/niio-art/one/runs/5212989943?check_suite_focus=true#step:16:39)0, in prepare_url     raise MissingSchema(error) requests.exceptions.MissingSchema: Invalid URL 'None': No schema supplied. Perhaps you meant http://None?
The second notification is as simple as this:
      - name: Slack notification         if: always()         uses: ravsamhq/notify-slack-action@v1         with:           token: ${{ secrets.GITHUB_TOKEN }}           status: ${{ job.status }}           notification_title: 'Build finished'

@gabiganam were you able to resolve this?

Yes, it was my mistake, when copying and pasting the first action, I didn't copy the env :(
This is the error you get when you don't set the required env, maybe worth improving this with a proper error message.


CircularProgressIndicator causes 100% CPU on IOS simulator

Steps to Reproduce

Create main.dart containing:

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: CircularProgressIndicator(
          value: .8,
        ),
      ),
    );
  }
}

Run it on the IOS simulator
Run top -o cpu

PID    COMMAND      %CPU
47633  Runner       58.6
46728  backboardd   43.4

Replace CircularProgressIndicator(value: .8), with Column(),

import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Column(),
      ),
    );
  }
}

Note Simulator CPU usage drops to zero.

Flutter Doctor
[✓] Flutter (Channel beta, v0.6.0, on Mac OS X 10.13.6 17G65, locale en-US)
    • Flutter version 0.6.0 at /Users/jeff/Dev/2_flutter_tool/flutter
    • Framework revision 9299c02cf7 (3 weeks ago), 2018-08-16 00:35:12 +0200
    • Engine revision e3687f70c7
    • Dart version 2.1.0-dev.0.0.flutter-be6309690f

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).
      If Android SDK has been installed to a custom location, set $ANDROID_HOME to that location.

[✓] iOS toolchain - develop for iOS devices (Xcode 9.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.4.1, Build version 9F2000
    • ios-deploy 1.9.2
    • CocoaPods version 1.5.3

[✗] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[✓] VS Code (version 1.26.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.18.0

[✓] Connected devices (1 available)
    • iPhone 7 • D840D735-FEF3-4486-940C-923FC8044EB5 • ios • iOS 11.4 (simulator)

I see high CPU usage when running flutter app using desktop-embedding on Ubuntu 18.04.
Here is an endless animation example that demonstrates this issue:
https://gist.github.com/kirgene/00362d280430df48513ee691b2149a9e

This is still a problem with latest flutter and iOS simulator.
@jason-simmons do you want me to file a new issue for it?
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.15.1 19B88, locale en-IT)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.1)
[✓] Android Studio (version 3.5)
[✓] IntelliJ IDEA Community Edition (version 2019.2.3)
[✓] Connected device (1 available)

• No issues found!

This issue still exist today:
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.2 19C57,
    locale en-SG)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.41.0)
[✓] Connected device (2 available)

This issue affects Android Emulators as well, CPU usage skyrockets and fans go on full blast after showing a CircularProgressIndicator for ~20+ seconds.
[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Mac OS X 10.14.5 18F132, locale en-SE)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 10.3)
[✓] Android Studio (version 3.3)
[✓] IntelliJ IDEA Ultimate Edition (version 2018.3.4)
[✓] Connected device (1 available)

Still exists
[√] Flutter (Channel stable, v1.12.13+hotfix.9, on Microsoft Windows [Version 10.0.18363.720], locale en-US)
[!] Android toolchain - develop for Android devices (Android SDK version 29.0.1)
    X Android license status unknown.
      Try re-installing or updating your Android SDK Manager.
      See https://developer.android.com/studio/#downloads or visit https://flutter.dev/setup/#android-setup for detailed instructions.
[!] Android Studio (version 3.5)
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
[!] IntelliJ IDEA Ultimate Edition (version 2019.2)
    X Flutter plugin not installed; this adds Flutter specific functionality.
    X Dart plugin not installed; this adds Dart specific functionality.
[√] VS Code (version 1.44.2)
[√] Connected device (1 available)

Still exists

I experience this regularly.

Also a problem on flutter web, pins the CPU in Chrome. Especially bad when dev tools is open and it looks like it's continuously logging the frame output to debug (hidden logs in dev tools, that you can unhide to see).

Still exists


TypeError: options.headers.hasOwnProperty is not a function

new Response(ctx.response)

nodejs.TypeError: options.headers.hasOwnProperty is not a function
    at new Response (/Users/cwfan/code/outbook/outbook-api-v2/node_modules/oauth2-server/lib/response.js:16:25)
    at /Users/cwfan/code/outbook/outbook-api-v2/app/middleware/oauth2_token.js:11:22

Is the ctx.response you are passing formatted correctly based on the docs? (https://oauth2-server.readthedocs.io/en/latest/api/response.html) What is the format of ctx.response.headers?

ctx.response is replaced by other code

I have same issue using koa2... how you solved it?

koa 2.5
node_modules/oauth2-server/lib/response.js:16:25


Custom deleted model class

This PR allows full usage of the custom deleted model class. Besides that, validation on the service provider level is added, to be sure that configured model is an instance of the package DeletedModel class.

Thanks!


INT/RAD switch not calling pushback

Hello,
I'm on RC3, aircraft is loaded, APU AVAIL and Bleed on (PCA removed by Fenix2GSX), Beacon ON (Fenix2GSX sets preparing for pushback and removes the jetway), moving the INT/RAD switch to INT is not starting the pushback sequence. It does work to call boarding during refueling, and I have not tested "During Push-Back to either Confirm or Abort the Push".

I have a different flow, I do not turn the beacon on. So my aircraft is loaded, APU AVAIL and Bleed on (PCA removed by Fenix2GSX),moving the INT/RAD switch to INT is not starting the pushback sequence,

So you removed the Ground-Equipment by the normal Trigger of Beacon+Brake+ExtPwr?
I think this is the Issue. The Int/Rad "Shortcut" was specifically added for another Issue, to allow removing the Ground-Equipment without the Need to set the Beacon. Since the Ground-Equipment was already removed by the normal Trigger, it currently won't listen to the Push-Request by Int/Rad.

So you removed the Ground-Equipment by the normal Trigger of Beacon+Brake+ExtPwr?
I think this is the Issue. The Int/Rad "Shortcut" was specifically added for another Issue, to allow removing the Ground-Equipment without the Need to set the Beacon. Since the Ground-Equipment was already removed by the normal Trigger, it currently won't listen to the Push-Request by Int/Rad.

Correct

So you removed the Ground-Equipment by the normal Trigger of Beacon+Brake+ExtPwr?
I think this is the Issue. The Int/Rad "Shortcut" was specifically added for another Issue, to allow removing the Ground-Equipment without the Need to set the Beacon. Since the Ground-Equipment was already removed by the normal Trigger, it currently won't listen to the Push-Request by Int/Rad.

I do not remove the equipment, I leave it there and after the boarding is completed, and I see the final loadsheet countdown in the Fenix2GSX log, i press the INT button on the radio panel and leave it on.

Then see above :)
Fenix2GSX does not look for the Trigger anymore, since the Ground-Equipment was already removed.

Ground-Equipment was already removed.

what do you mean by this? that the ExtrPwr was set to off (Avail)? that still means that the ground equipment (GPU) is tere, as well as the chocks are there. Should i leeve ExtrPower on ON and then press INT? Disconnecting GPU while ON can be life threatening to the poor GSX ground guy :P

There are two Conditions/Trigger when Fenix2GSX will remove the Ground-Equipment:

Beacon ON + Brake SET + ExtPwr OFF
GSX Push-Back was called (does not matter who & how)

Fenix2GSX is currently only listening to the Int/Rad Switch if the Ground-Equipment is still there. But when reach Condition 1) it will not listen for it anymore. Since the Int/Rad Switch and that additional Ground-Equipment-Trigger where specifically added for not having the Beacon turned ON.
I'm currently investigating what can be done to make the Behavior more "universal".

Try RC4

But I never reach condition 1, since my beacon remains off, so listening to INT should not stop. Will test RC4 tomorrow. Thanks for your support!

Oh sorry, misread your Post - I read "Beacon off" as "Beacon on" 😓

RC4 works for my use case, thanks!

I did a few tests but still no luck with the INT button. Tried pressing it for boarding (during refueling), also did not trigger. Starting to think that I might be doing something wrong. I press the INT button above the INT knob and leave it on (lit) while my beacon is on, GPU avail, parking brake on.

So the Issue persists for you with RC4?
Do you still have the Logs from these Tests?

Yes, RC4, still got the logs.

Problem lied between the keyboard and chair, it is now fixed. I was pressing the INT button, not INT/RAD switch. All works.


Improve run command

Currently "poetry run" command uses subprocess.call for starting
programs and scripts. This approach has a problem --- signals like
Ctrl-C are caught by poetry process itself and not by the process the
user has run.
This can be observed this way: run "poetry run python" and press Ctrl-C.
Normally Python interpreter will catch the signal and print
"KeyboardInterrupt" exception, continuing execution normally. But when
run by poetry, it exits immediately. This behavior is rather annoying,
as one has to restart interpreter after accidentially doing Ctrl-C.
Moreover, after python exits, it leaves terminal in a broken state,
requiring the user to use "reset" or similar command.
This commit fixes this behavior by using "os.execvp" instead of
subprocess. This replaces poetry process directly with target process,
eliminating an intermediate process.
This approach is used in other products like Pipenv and Haskell's build
system stack. See these links for examples and more info:

https://github.com/pypa/pipenv/blob/master/pipenv/core.py#L2462
https://github.com/commercialhaskell/stack/issues/527
https://hackage.haskell.org/package/rio-<IP_ADDRESS>/docs/RIO-Process.html#v:exec

Pull Request Check List
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes.  But please read our contribution guide at least once, it will save you unnecessary review cycles!

[ ] Added tests for changed code.
[ ] Updated documentation for changed code.

Note: If your Pull Request introduces a new feature or changes the current behavior, it should be based
on the develop branch. If it's a bug fix or only a documentation update, it should be based on the master branch.
If you have any questions to any of the points above, just submit and ask!  This checklist is here to help you, not to deter you from contributing!
Notes

I'm unsure if it makes sense to add tests for this functionality. Right now there are no tests for run command in poetry.
Windows runs the old way, this issue seems not to be present there.
Also, it seems that nothing has to be added to the docs, as this a simple fix.

Thanks for taking the time to make this PR!
However, this is already covered by #1236 so I am closing this one to avoid duplicates.

Oh, great!
Sorry for that. I'm sure I tried searching github before making my pr, but did not find that one for some reason.


⚠ ClamAV has degraded performance

In 9f4f22f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 61 ms

Resolved: ClamAV performance has improved in 0ca9ada.


Prepare for v1.0.4

Close #35

Codecov Report

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

@@            Coverage Diff            @@
##             master      #43   +/-   ##
=========================================
  Coverage     83.63%   83.63%
  Complexity       51       51
=========================================
  Files            20       20
  Lines           110      110
  Branches         14       14
=========================================
  Hits             92       92
  Misses           12       12
  Partials          6        6

Impacted Files
Coverage Δ
Complexity Δ

...m/github/shiraji/yaemoji/domain/EmojiCompletion.kt
84.21% <ø> (ø)
5 <0> (ø)
:arrow_down:

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 8157cef...e8e88e8. Read the comment docs.


MAINT: Fully display problem size in lsmr/lsqr.

Using %8g results in scientific notation when there are more than 1e8
rows (or columns), even though the number is still an integer.  Format
the matrix size as integers instead.

Reference issue

What does this implement/fix?

Additional information

thanks, merged


Is ContenField-change based event activity implementation possible?

Hi guys,
is there a built-in way to implement a workflow trigger based on a content field change (not for whole  content item change ) in orhcard 1.10.3? If currently there is no way then how could we implement it, any direction you guys can advice please?
Thanks in advance

AFAIK there's no out of the box way to do what you are describing.
What are you trying to achieve?
If I understand you correctly, given a ContentItem MyItem of ContentType MyType, you want to trigger a worfkflow on Updated/Published only if the value of ContentField MyField has changed. Is that right?
I am going half off memory here, so forgive me for any mistake and imprecision. I'll talk about Updating/Updated, but you may make similar considerations for Publishing/Published, I think.
You need a handler.
I would start from src/Orchard.Web/Modules/Orchard.Projections/Handlers/FieldIndexPartHandler.cs
That is the handler that takes care of updating the values for the field in the FieldIndexRecords (by setting what should be done when the Describe method of each field's driver is invoked).
I think you'd need a reliable way to test that the value you are monitoring has changed. I am not sure you can reliably do that in the Updated event. You may end up needing to memorize the old values in an Updating event, and compare them in an Updated event.
Supposing the condition is verified, you would then simply need to Trigger you new Workflow event, for which you'd have to have created a corresponding Activity. Look for example at src/Orchard.Web/Modules/Orchard.Workflows/Activities/ContentActivity.cs.
To Get the values in the Updating phase, I would probably recommend going through IDraftFieldIndexService. The "new value" you use for comparison, instead, is not guaranteed to be in the record yet when the Updated event runs in your handler.
I hope this helps.

Thank you @MatteoPiovanelli-Laser for your well written detailed answer. I appreciate your time and your  favor.
Indeed, I did something similar to what you described above.
Best

Can you share what you did on a PR?
How did you manage the UI side of things, where an admin should configure the event?

Can you share what you did on a PR?
How did you manage the UI side of things, where an admin should configure the event?


Add CompactArray::address_of(index: usize) -> *const u64

Since random memory access is generally slow, I'm speeding up my implementation by prefetching cachelines for a few (16) iterations ahead, see here and here. For Vec<u64> this brings the inverse throughput (time per iteration) down from 18ns to 7ns and for Vec<u8> it goes from 17ns to 6ns.
Would be cool if you could add a similar method here -- I don't think I will be able to get its performance close to that of Vec without being able to prefetch.

Just added to BitFieldVec (the artist formerly known as CompactArray) in the "compact" branch.


/etc/ptdb/config.toml appears to no longer be necessary

Description
/etc/ptdb/config.toml appears to no longer be necessary. Is this true? If so, remove it and update the code accordingly.
Steps to Resolve

[x] check to see if the file is no longer necessary (it is no longer necessary)
[x] update the code
[x] rm /etc/ptdb/config.toml

/etc/ptdb/config.toml is no longer necessary and I forgot to remove it. it is now deprecated and will be removed in the next release.

this was fixed by commit e04750a


Action stopped working

Describe the bug
Action doesn't work currently.
I wanted to use it to read REPOSITORY_DESCRIPTION but it seems that currently no variable is filled.
Logs the following when using it:

Run<EMAIL_ADDRESS>with:
repository: false
debug: false
env:
GITHUB_TOKEN: ***
(node:2521) UnhandledPromiseRejectionWarning: HttpError: Not Found
at /home/runner/work/_actions/varunsridharan/action-repository-meta/2.0/webpack:/action-repository-meta/node_modules/@octokit/request/dist-node/index.js:66:1
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at run (/home/runner/work/_actions/varunsridharan/action-repository-meta/2.0/webpack:/action-repository-meta/src/index.js:16:1)
(node:2521) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:2521) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

To Reproduce
Used as described in your readme.
Snippet out of my workflow-definition:

jobs:
build:
runs-on: ubuntu-latest
steps:
- name: Read repository metadata
uses<EMAIL_ADDRESS>env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Expected behavior
Run successfully and fill the variables like described.
Screenshots

Additional context
Found a pretty similar issue: https://github.com/bchr02/node-pre-gyp-github/issues/24
In this issue the github api is different (due to changes maybe).
If this is here also the issue I would suggest to improve the errorhandling so that only single variables aren't filled
when the data can't be gathered.
Also the error shows that node exited with an errorcode != 0 but the step isn't failing (of course you maybe defined it to act like that).

Sorry about the delay.
I have added a fix for it. please give it a try and let me know.

Thanks a lot!
Current version on main-branch works again.


tank update procedure

we have basically 2 options IMHO:

stop tank, saving chunks to disk, update tank, start it, which means it has to decode from disk, and work through a backlog of data. this is AFAIK why we did #40
3 issues with this approach:
a) the ordering problem with nsq causes data loss and gaps in data (messages from during downtime will come out of order and we have to drop metrics if they're older than previous)
b) if the upgrade comes with changes to data structures, it will be a bunch of extra work to accommodate migrating the old data into a new format (if possible), and possibly having to throw away the old chunks, creating an even bigger gap of data.
c) the datasource as a whole is effectively offline during this process.

launch a new tank instance parallel to the old one on a different ip/port/machine.
the downsides are:
a) it needs to run for a while and cover all data range between now until when we have data in cassandra, ideally up to numChunks*numspan and/or whatever range our dashboards are set to so cassandra doesn't see an increased read load when we switch.
b) it's operationally a bit more complex
c) we need to implement some additional logic to not have both instances be saving the same data to cassandra, i guess some kind of signal for an instance to tell it not to save anything, and then another one to tell it to start saving starting with data from a given boundary timestamp. haven't thought too much about this.

but the upside is the three problems from 1 don't apply here, so we can do the upgrade seamlessly without any data loss.
There's some hope for 1a though:

I'm hopeful that #48 + probably some parrallelized encoding/decoding will make things fast and the time gap between stop and start small, so that we're in fairly good shape until we fix ordering properly
@woodsaj was going to experiment with max-in-flight and concurency settings, which I think will result in mostly-correct-ordering out of NSQ. but only one way to find out...

though due do 1b and 1c i think perhaps we shouldn't be spending too much time on save/restore of chunks to/from disk, and just go for 2 ? I think its downsides may be easy to overcome.
@woodsaj thoughts?

I do like the idea of "2", and building on this idea i think we can solve a number of problems.
at present we have 2 channels (copies of metric data). 1 for metric-tank and 1 for nsq-metrics-to-es.  Lets add an additional 2 channels so that we have 3 channels for metric-tank (1 for each availability zone).  We can then spin up an additional metric-tank in each zone.
So we still need a way to have only 1 node as the "primary" node which writes data to cassandra. Other nodes should just keep data in memory and provide additional capacity for graphite queries.
A clustering solution would be ideal for handling the assignment of the "primary" role, but we can also just handle it manually for now.  What we need to address is when the primary role moves to a new node, how do we know which chunks have not yet been committed to cassandra?
When performing upgrades, we just need to perform them 1 node at a time.

What we need to address is when the primary role moves to a new node, how do we know which chunks have not yet been committed to cassandra?

how about: when we signal a secondary instance that it should become primary, it can iterate its list of metrics, and for each one query cassandra what's the latest ts? we can then mark all chunks up until that one as saved.  for simplicity we can assume we won't do this right when the month changes so that it's easy to get the proper row_key. doing this on a metric by metric basis might be slow though, maybe we can do the query for many at the same time? we could also have the old primary write out a file of metric-key t0-of-last-saved-chunk mappings, and load in that file when promoting the new one.

from hangout:

cassandra -> too inefficient, have to query all rows individual
file save / file load -> only works for manual maintenance, if primary crashes, you don't have the information.

AJ suggested commitlog idea: after every successful cassandra chunk save, primary writes a log to a nsq/kafka topic. you need at least one secondary instance that has been consuming from this for a while (>1 chunkspan) to have built-up the required state, which it can then use to know from which point on the chunks require saving. he will work on this.

just had a great call with dieter.  The approach we agreed on was.

continuously run 2 or more metric-tank instances with only 1 acting as primary and writing to cassandra (the additional instances will be used for handling graphite queries.)
The primary nodes pushes updates to a NSQ stream when ever a chunk is persisted to cassandra.  The other nodes read this stream and use it to update their own chunk save states.
After an outage or during maintenance, a new instance becomes the primary and can just persist new chunks as needed.  The GC() task will handle persisting any chunks that were skipped.

main issue to consider is that the first chunk of a fresh instance will only be a partial chunk. So the primary role should always be assigned to the instance that has been running the longest.

@woodsaj any news on this?

@woodsaj how is this going, or do you want me to start working on this?

This is mostly done. Was just waiting on PR#67 to be merged so i could incorporate those changes.

https://github.com/raintank/raintank-metric/pull/74

we decided on 2 so this can be closed.


Show message to user when builder feed url is wrong?

Currently on the builder, if a user typos a feed URL, the "Choose an episode" select menu will show up regardless, with no episodes available. Should we instead show a message to user saying to check their URL?
Related issue: when user has put in a correct feed URL and selected an episode, then changes feed URL to another URL, the "Choose an episode" menu options change to reflect the new podcast chosen, but the rest of the page below (all the customization options for the previously selected show/episode) remains the same. Do we want to handle this case?

2nd issue has been resolved!

live in prod!


Style/ArrayIntersect causes undefined method `intersect?' for #<ActiveRecord::Associations::CollectionProxy

rails 7.0.3
ruby-3.2.0
Rubocop corrected (accounts & other_accounts).any? to accounts.intersect?(other_accounts), however accounts and other_accounts are instances of ActiveRecord::Associations::CollectionProxy, not arrays, and intersect? is not present on ActiveRecord::Associations::CollectionProxy.
Fortunately this was caught by my test suite, but was a bit scary as I was about to deploy to production, and I traditionally have run rubocop after the tests.

Yeah, this cop is not safe - https://github.com/rubocop/rubocop/blob/1e5f19a9afddb0e5123c86efb0f5957431c7a6d4/lib/rubocop/cop/style/array_intersect.rb#L14-L16. So this is expected.

Yeah, this cop is not safe -
https://github.com/rubocop/rubocop/blob/1e5f19a9afddb0e5123c86efb0f5957431c7a6d4/lib/rubocop/cop/style/array_intersect.rb#L14-L16
. So this is expected.

But it's not marked Safe: false in config/default.yml so it's being autocorrected on a safe autocorrect.

Ah, I see. Would you like to open a PR with this change?

This issue fixed by https://github.com/rubocop/rubocop/commit/d02d3149231d67f5a7fd59e00fe4ede0c3e01de6.


Java imports need to be further adjusted for Scala

Since the Scala kernel is using the default JVM imports, two adjustments are needed beyond the already implemented changing of the wildcard.  They are:

Remove static
Escape the package name object, which is a Scala keyword.  (This is used in one of the default import paths, com.twosigma.beaker.jvm.object.OutputCell.)

On the second point, it's conceivable that other Scala keywords that aren't Java keywords might show up in package names, but no others seem very likely, so a one-off replacement is probably fine.

Thank you Joe.  Agreed.

I've got some code for this in my workspace.  Working on a unit test.


⚠ Bofrost.at has degraded performance

In 5d087d8, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1669 ms

Resolved: Bofrost.at performance has improved in 0b07b1f after 11 minutes.