Table updates

@davenquinn Don't merge, last commit is not a great one. Lets discuss at 1.

Awesome ish.
Just focusing on the front page:

Does not work in dark mode
You lost the numbers, which are important for us (internally) as a quick reference. There should be some way from that page to get the ID numbers or the slugs. And the lower rows without metadata have no way to see which map you're looking at
You're using a "log-out" icon for a "log-in" action
If you do link cards, ideally the whole card should be clickable
The toolbar should not scroll with the page (although the whole concept of having a toolbar on this page is debatable)

Overall, the login/logout functionality is needed, but I think this is a bit of a lateral move in terms of effective user interface TBH.

In the future, it may be useful to separate "design" and "functionality"-oriented PRs to the extent possible.

Also, having a separate entry page for map editing that puts the login front and center, and then lists only/primarily "incomplete" maps would be more ideal, I think.

I'm also getting 500 errors on clicking any of the newly imported maps. Could it be that the fields were not prepared for them properly?


Tricky Ghost Splits

[x] Cloth: implement the Traitor Lord dead on entry thing, shouldn't be too hard.
[ ] Revek: ~try using gladeGhostsKilled.~ try using SceneData for all other glade ghosts.
[x] Vespa: figure something out that works without relying on Hive Knight, and without relying on Hiveblood if it's possible to get Vespa without that.

Ideas from cerpin and slaurent about Vespa:

a hacky fix for the vespa one: check knight coords
a timer after hive knight death
both are gross

it looks like there's a box around the hiveblood pickup you can enter and leave to trigger vespa
a timer could work tbh
but would be jank soooo

On patches without Hive Knight, it can just split when essence increments while in Hive_05.
On patches with Hive Knight... I should test how much time can pass between when the player data shows Hive Knight killed, and getting Blessence from Hive Knight. If it's instant, then the "timer" solution could just be checking that at least one tick has passed with Hive Knight dead, and then check for essence incrementing while in Hive_05. I should be able to test that using the EssenceAlways mod.

It looks like it is pretty instant, as in
  ---
PlayerData dreamOrbs: 903
PlayerData killedHiveKnight: true
  ---
PlayerData dreamOrbs: 904
  ---

The gladeGhostsKilled field doesn't seem to be working though. With the dream tree there in addition to the ghosts, how else can it be done properly?

In the Online Hollow Knight Save Editor here: https://bloodorca.github.io/hollow/
{
  ...
  "sceneData": {
    ...
    "persistentBoolItems": [
      ...
      {
        "id": "Ghost revek",
        "sceneName": "RestingGrounds_08",
        "activated": true,
        "semiPersistent": false
      },
      ...
    ],
    ...
  }
}

Is there a way I can access that?

The possibility of using Scene Data for Revek was already looked at by cerpin and slaurent:

scenedata only updates once you leave the scene completely, aka transition

So I can't use this scene data for Revek directly. I might be able to use it to determine whether every ghost except Revek has been dream nailed on previous visits to the scene, and then just count any essence after that while in the scene as Revek.
That technically would wrongly split it when a runner dream nails all the ghosts except Revek, leaves, re-enters, and then goes past a Revek ready to be dream nailed, activates the dream tree, and then gets essence from the tree instead, but, I think that's the best I can do.

Video by kamakiller: https://www.youtube.com/@kamakiller, karina%: https://www.youtube.com/watch?v=LbIiqaISSQY
Apparently Karina is not like the other Glade Ghosts. So given that, what should the OnObtainGhostRevek split do if someone Dream Nails all the ghosts in the glade except Karina, exits the room, and then Dream Nails Revek? Should it only split when all Glade Ghosts have been Dream Nailed including both Revek and Karina?

At least Dream Nailing Karina alerts Revek so he starts attacking, so she's not completely separate.

But Dream Nailing all Glade Ghosts except Karina and Revek, and then Dream Nailing Karina, does not alert Revek to attack.

Idea: track both the number of glade ghosts killed including Karina, and the number of glade ghosts killed excluding Karina. If the number excluding Karina is 17, then Revek is obtainable.
But if the number of glade ghosts killed including Karina is only 17, then it shouldn't split until 2 more essence have been obtained in the Glade, while if the number including Karina is 18, then it can split on 1 more essence obtained in the Glade.


Visualization issues

Why is the visualization result of demo.py and kitti_object_vis very different?

I haven't compare the result of kitti_object_vis. I notice that the visualization result of demo.py miss some cases in the distant region. Could you check the score of those missed cases? Maybe the reason is our model filter the predicted boxes with scores less than 0.1 (i.e., MODEL.POST_PROCESSING.SCORE_THRESH). However, this just my guess, since I haven't compare them before.

Thanks for your reply, I will try it. Wish you all the best.


Strong typedef col_id to avoid confusion. Associated minor fixes

col_id and projection_list_index are both uint16_t but mean different things. This is very error prone since the two classes of values often coexist and are simply named generally as col or i in the code,
With strong typedef, the compiler shall slap you if you try to use one for the other.
StrongTypeAlias is also modified to have most of the integer operators, because it's pretty clear now that we almost always wrap things transparently (they still make sense as integers), as opposed to originally where we thought maybe we want to stop people from not only using them as integer types, but also do integer operations on them.
I have verified that this has no performance impact on benchmarks, as expected, since strong typedefs are compile-time only.

Coverage increased (+0.2%) to 98.367% when pulling 272a96b9989d3510fd076b37035269a45dc83192 on tli2:tianyu-col-id into b56f08feaa2c9782b8a6ce269afc4d6847427ab2 on cmu-db:master.


[C#] Added vector constructors to allow create them from one scalar

Allows to create a vector like:
var v = new Vector2i(64); // 64x64 vector declaration
I know it's a syntax sugar but I found it's useful for myself.

Also, this shouldn't be in C# unless it's also in core. Currently print(Vector3(5)) is not valid in GDScript.

I know, but I need to found a way to implement it properly since it suddenly has compilation conflicts with other constructors in GDScript...

Need to think about how to do it for GDScript, currently, I lose interest to do it, maybe later I create another PR with all the requested changes been made.


Cross reference metric list for further customization

Useful to list https://docs.microsoft.com/en-us/azure/azure-monitor/platform/metrics-supported#microsoftsqlserversdatabases as a linked or relevant article for consuming non-DTU percentage metric as provided as example in sample script.

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

ID: edd8f2cb-6ecf-6669-b643-3af38429a044
Version Independent ID: d41cceb1-4e97-3533-5e75-c1bbd686659e
Content: PowerShell example-monitor-scale-single Azure SQL database
Content Source: articles/sql-database/scripts/sql-database-monitor-and-scale-database-powershell.md
Service: sql-database
GitHub Login: @julieMSFT
Microsoft Alias: jrasnick

@sqlazure Thank you very much for providing this detailed feedback. I see the sample script is built upon the dtu_consumption_percent metric. The link you provided (https://docs.microsoft.com/en-us/azure/azure-monitor/platform/metrics-supported#microsoftsqlserversdatabases) is an entire list of valid metrics that could be used for MetricNames in the sample script. This link would be handy so you would not have to go search for it.
I am forwarding this on to the content author (@julieMSFT) to evaluate and update as appropriate.

@sqlazure Thank you for your suggestion. I have submitted the change to the document.

#70464

#please-close

@sqlazure We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Elegoo Mars 5 Ultra compatibility ?

I'd like to know if the Mars 5 Ultra is compatible with SdcpMonitor.
I think it should be, but I don't have a Mars 5 Ultra.
If someone has one, please test this out and leave some feedback.

Awesome !


Trouble retrieving text

Hi,
First, thank you for creating this package. I am a beginner when it comes to using API. I am trying to extract data but not receiving any text.
I ran this code:
mport epo_ops
anonymous_client = epo_ops.Client()  # Instantiate a default client
response = anonymous_client.published_data(  # Retrieve bibliography data
reference_type = 'publication',  # publication, application, priority
input = epo_ops.models.Docdb('1000000', 'EP', 'A1'),  # original, docdb, epodoc
endpoint = 'biblio',  # optional, defaults to biblio in case of published_data
constituents = []  # optional, list of constituents
)
registered_client = epo_ops.RegisteredClient(key='abc', secret='xyz')
registered_client.access_token  # To see the current token
response = registered_client.published_data(…)
Replaced the last line:
response = registered_client.published_data('publication', epo_ops.models.Docdb('1000000', 'EP', 'A1'), endpoint='biblio', constituents=None)
Output I got : <Response [200]>
How should I modify the code to retrieve text?  I would appreciate your help.

Hi @alokgrover,
"response" variable is an object and you have to check "response.content" attribute for downloaded data

@alokgrover See http://docs.python-requests.org/en/master/user/quickstart/#response-content and http://docs.python-requests.org/en/master/api/#requests.Response for more information on the response object.


Package broken by import

Cannot import _request_ctx_stack from quart.  This import prevents the import of the entire package:
Traceback (most recent call last):
  File "M:\source\bandz-padline-webserver\main.py", line 6, in <module>
    import quart_login
  File "M:\source\bandz-padline-webserver\venv\lib\site-packages\quart_login\__init__.py", line 12, in <module>
    from .login_manager import LoginManager
  File "M:\source\bandz-padline-webserver\venv\lib\site-packages\quart_login\login_manager.py", line 5, in <module>
    from quart import _request_ctx_stack
ImportError: cannot import name '_request_ctx_stack' from 'quart' (M:\source\bandz-padline-webserver\venv\lib\site-packages\quart\__init__.py)

The proper import at this time:
from quart.globals import _request_ctx_stack

Also, I don't think you can _request_ctx_stack  on the general import of quart_login.  I think the server and context have to exist for _request_ctx_stack to load

I get the same error

from quart import _request_ctx_stack
this method is not in quart. There is a ctx module, but nothing that relates to the import. Additionally, I cannot find any history of the above mentioned module.

Same issue here, is this package even still a thing? Also having issues using flask patch with flask_login

Same issue, no fix in sight...


CB-5385 Flow API check in SDX might return with false result

tested with latest 2.16 build locally using AwsSdxTests.testSDXMultiRepairIDBRokerAndMasterWithTerminatedECInstances, will test with latest 2.17 build soon

tested with latest 2.17 build locally using SdxTests.disabledTestSDXRepairIDBRokerWithStoppedEC2Instance (which was enabled locally)

We should also reenable the tests right? This should fix that issue?


get_state should return an encrypted and signed value (TLS into enclave?)

the client should be able to verify that what a worker sends him is indeed coming from the enclave.
Moreover, the payload should be encrypted such that the worker doesn't learn anything about the query

the enclave doesn't have a key agreement with the user. We should actually have a TLS connection right into the enclave for worker-api

This issue should be fully covered by issue #202.
@brenzi should we keep this issue open either way?

We can close this one as it is fully covered.


undo help inside click-argument

move help from click argument to the fuction doc-string - to fix cooltools cli
quick fix  - for https://github.com/mirnylab/cooltools/commit/1dbdd3dbf01ef7e1180544ef531dd30c97b3a5b4#commitcomment-39538718

Ouch!  Thank you!!!


Add .apply_layout()method  to Pauli class

Summary
This PR adds an apply_layout method to the Pauli class, following what was done for SparsePauliOp in https://github.com/Qiskit/qiskit/pull/10947. This was the agreed action to partially solve #11824 in the short term until a more permanent/extensible solution is designed.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

18 of 19 (94.74%)  changed or added relevant lines in 1 file are covered.
9 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.009%) to 89.332%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/quantum_info/operators/symplectic/pauli.py
18
19
94.74%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/expr.rs
1
94.03%

crates/qasm2/src/lex.rs
2
92.37%

crates/qasm2/src/parse.rs
6
97.61%

Totals

Change from base Build<PHONE_NUMBER>:
0.009%

Covered Lines:
59813

Relevant Lines:
66956

💛  - Coveralls


feat: added the customdata or init script to azure and civo vms

Tasks description
Issues
Related Issue(s)

#302
#277

Solution
Sub-Tasks

[ ] milestones or sub-task which needs to be coverted

Note to reviewers

[ ] Ran Tests locally
[ ] Checked Contribution's guidelines

cc @Horiodino

some issues can happen if the apt update is going on and hostname has not changed and it can cause the issue with the k3s script

so this can happen

and got this error

add debug log for the initscript


Watchtower gets a 404 when trying to update

"Could not do a head request for "xavierh/goaccess-for-nginxproxymanager:latest", falling back to regular pull.
Reason: registry responded to head request with "404 Not Found", auth: "not present""
Recently change detection had the same issue with buildx, https://github.com/docker/setup-buildx-action/issues/187
It seems to be related to "provenance: false"
Is this the same issue for go access, or a different reason watchtower is failing to update?
Thanks

Thanks for opening this issue and looking into it.
This does appear to be an issue with watchtower. I did recently update the actions on GitHub because there are phasing out the old actions in May/June which is why we're probably getting this error now.
Looks like they have pull request out there to fix the problem, https://github.com/containrrr/watchtower/pull/1526 . I'll monitor it for a few days to see what progress they've made on approving it.
For now... docker pull xavierh/goaccess-for-nginxproxymanager:latest will work for you.

@wrench12435678 can you post a Watchtower debug log snippet, from Trying to load authentication credentials or Got image name to Pulling image? Pulling images from Docker Hub isn't currently broken afaik.

watchtower  | time="2023-01-29T00:09:45+07:00" level=debug msg="Trying to load authentication credentials." container=/neededimg-moby-buildkit-1 image="moby/buildkit:buildx-stable-1"
watchtower  | time="2023-01-29T00:09:45+07:00" level=debug msg="No credentials for moby found" config_file=/config.json
watchtower  | time="2023-01-29T00:09:45+07:00" level=debug msg="Got image name: moby/buildkit:buildx-stable-1"
watchtower  | time="2023-01-29T00:09:45+07:00" level=debug msg="Checking if pull is needed" container=/neededimg-moby-buildkit-1 image="moby/buildkit:buildx-stable-1"
watchtower  | time="2023-01-29T00:09:45+07:00" level=debug msg="Building challenge URL" URL="https://index.docker.io/v2/"
watchtower  | time="2023-01-29T00:09:46+07:00" level=debug msg="Got response to challenge request" header="Bearer realm=\"https://auth.docker.io/token\",service=\"registry.docker.io\"" status="401 Unauthorized"
watchtower  | time="2023-01-29T00:09:46+07:00" level=debug msg="Checking challenge header content" realm="https://auth.docker.io/token" service=registry.docker.io
watchtower  | time="2023-01-29T00:09:46+07:00" level=debug msg="Setting scope for auth token" image=moby/buildkit scope="repository:moby/buildkit:pull"
watchtower  | time="2023-01-29T00:09:46+07:00" level=debug msg="No credentials found."
watchtower  | time="2023-01-29T00:09:47+07:00" level=debug msg="Parsing image ref" host=index.docker.io image=moby/buildkit normalized="docker.io/moby/buildkit:buildx-stable-1" tag=buildx-stable-1
watchtower  | time="2023-01-29T00:09:47+07:00" level=debug msg="Doing a HEAD request to fetch a digest" url="https://index.docker.io/v2/moby/buildkit/manifests/buildx-stable-1"
watchtower  | time="2023-01-29T00:09:49+07:00" level=warning msg="Could not do a head request for \"moby/buildkit:buildx-stable-1\", falling back to regular pull." container=/neededimg-moby-buildkit-1 image="moby/buildkit:buildx-stable-1"
watchtower  | time="2023-01-29T00:09:49+07:00" level=warning msg="Reason: registry responded to head request with \"404 Not Found\", auth: \"not present\"" container=/neededimg-moby-buildkit-1 image="moby/buildkit:buildx-stable-1"
watchtower  | time="2023-01-29T00:09:49+07:00" level=debug msg="Pulling image" container=/neededimg-moby-buildkit-1 image="moby/buildkit:buildx-stable-1"
watchtower  | time="2023-01-29T00:09:53+07:00" level=debug msg="No new images found for /neededimg-moby-buildkit-1"

not goaccess image, but tecnically every image that build using buildx v0.10.0+ will result 404 with watchtower. I saw a PR supporting new OCI manifest https://github.com/containrrr/watchtower/pull/1533 , but cant really tell if it working, as no watchtower development container image to test it out.

cant really tell if it working, as no watchtower development container image to test it out

Try containrrr/watchtower:latest-dev :)

That appears to have worked. :)
I set the check interval to 10 seconds temporarily.
BEFORE
# docker container logs watchtower
time="2023-01-28T20:00:10-05:00" level=info msg="Watchtower 1.5.1"
time="2023-01-28T20:00:10-05:00" level=info msg="Using notifications: pushover"
time="2023-01-28T20:00:10-05:00" level=info msg="Checking all containers (except explicitly disabled with label)"
time="2023-01-28T20:00:10-05:00" level=info msg="Scheduling first run: 2023-01-28 20:00:20 -0500 EST"
time="2023-01-28T20:00:10-05:00" level=info msg="Note that the first check will be performed in 9 seconds"
time="2023-01-28T20:00:20-05:00" level=warning msg="Could not do a head request for \"xavierh/goaccess-for-nginxproxymanager:latest\", falling back to regular pull." container=/goaccess image="xavierh/goaccess-for-nginxproxymanager:latest"
time="2023-01-28T20:00:20-05:00" level=warning msg="Reason: registry responded to head request with \"404 Not Found\", auth: \"not present\"" container=/goaccess image="xavierh/goaccess-for-nginxproxymanager:latest"
time="2023-01-28T20:00:20-05:00" level=warning msg="Could not do a head request for \"xavierh/goaccess-docker-console:latest\", falling back to regular pull." container=/goaccess_docker_console image="xavierh/goaccess-docker-console:latest"
time="2023-01-28T20:00:20-05:00" level=warning msg="Reason: registry responded to head request with \"404 Not Found\", auth: \"not present\"" container=/goaccess_docker_console image="xavierh/goaccess-docker-console:latest"
time="2023-01-28T20:00:21-05:00" level=info msg="Session done" Failed=0 Scanned=5 Updated=0 notify=no

AFTER
# docker container logs watchtower
time="2023-01-28T20:00:44-05:00" level=info msg="Watchtower v1.5.1-26-g8464e0d"
time="2023-01-28T20:00:44-05:00" level=info msg="Using notifications: pushover"
time="2023-01-28T20:00:44-05:00" level=info msg="Checking all containers (except explicitly disabled with label)"
time="2023-01-28T20:00:44-05:00" level=info msg="Scheduling first run: 2023-01-28 20:00:54 -0500 EST"
time="2023-01-28T20:00:44-05:00" level=info msg="Note that the first check will be performed in 9 seconds"
time="2023-01-28T20:00:54-05:00" level=info msg="Session done" Failed=0 Scanned=5 Updated=0 notify=no
time="2023-01-28T20:01:04-05:00" level=info msg="Session done" Failed=0 Scanned=5 Updated=0 notify=no
time="2023-01-28T20:01:14-05:00" level=info msg="Session done" Failed=0 Scanned=5 Updated=0 notify=no
time="2023-01-28T20:01:24-05:00" level=info msg="Session done" Failed=0 Scanned=5 Updated=0 notify=no
time="2023-01-28T20:01:34-05:00" level=info msg="Session done" Failed=0 Scanned=5 Updated=0 notify=no
time="2023-01-28T20:01:44-05:00" level=info msg="Session done" Failed=0 Scanned=5 Updated=0 notify=no

As Watchtower has fixed the issue on their side (Latest-Dev) and recently have pushed out an update on Latest (Not sure if its part of it), I will close this issue as it's been resolved


Option to change field display (dashed border-bottom)

Would be nice if could change/define (from within editable(options) or data-) the way a editable field is displayed prior to being selected. Yes, CSS is the best way to make fine tuned changes, but it would be nice to make less fine tuned changes field by field. Such as simply choosing between dashed or nothing, or even just entering a color code that will override the default dash color
On a page with lots of editable fields, that many dashes is overwhelming.

I agree with jansportw - I'm in exact this situation. I need to disable this underline only for a specific textarea because this effect is to massive and distracting.

@Hexodus you can just use a little jquery

edit me!

$('#myeditable').editable();
$('[data-underline="false"]').css('border-bottom', 'none');

Has this, perchance, been implemented? It seems like a really useful customization.

#myEditableTextArea.editable-click {
border-bottom-color: #cccccc;
border-bottom-style: solid;
}


ATTiny85 support

Hello and thank you for your library.
Could you please port it to support ATTiny85 using TinyWireM.h lib for I2C?
Thanks in advance.

Thanks for your request.
I am not familiar with the TinyWireM library and have almost no hands on experience with tiny, it is hard to say how much time it will take. Furthermore I am quite busy at the moment so have very little time to dive into this on short term.
Do you have a link to the TinyWireM library you use?

Thank you for your quick reply.
Here is the library in question:

If you can find a spare time, I and probably many other users will be more than grateful for your efforts.

how should it be called?
TinyKT0803 or KT0803_tiny or ???

TinyKT0803 it will be


feat #103 #118 support. provide default data structure for edge \ vertex \ path \ sub-graph, and their result handler.

feat: provide default data structure for edge \ vertex \ path \ sub-graph, and their result handler. #103 #118
for example:
List<NgSubgraph<String>> selectSubgraph();

<select id="selectSubgraph" resultType="org.nebula.contrib.ngbatis.models.data.NgSubgraph">
    GET SUBGRAPH WITH PROP 3 STEPS FROM "startVid"
    BOTH like
    YIELD VERTICES AS nodes, EDGES AS relationships
</select>

alias must be nodes and relationship.

feat: NebulaDaoBasic shortest path support. #118
List<NgPath<String>> ngPaths = dao.shortestPath("startId", "endId");

Wow! 👍
BTW the build seems to fail in ci?

Wow! 👍 BTW the build seems to fail in ci?

The test depends on unpublished version.
CI flow may install ngbatis first and then build ngbatis-demo.

But we build ngbatis-demo directly now.
This results in that the test code and the new feature code cannot be committed together.

But we build ngbatis-demo directly now. This results in that the test code and the new feature code cannot be committed together.

OK, maybe we could revisit this in the future :)

But we build ngbatis-demo directly now. This results in that the test code and the new feature code cannot be committed together.

OK, maybe we could revisit this in the future :)

Yes, I have fixed the version time difference by this commit. ngbatis-demo depends on 'ngbatis' from local maven repository when building. https://github.com/nebula-contrib/ngbatis/pull/119/commits/31cac59371b535e5f339b7743d03b4b4cb16991c

But we build ngbatis-demo directly now. This results in that the test code and the new feature code cannot be committed together.

OK, maybe we could revisit this in the future :)

Yes, I have fixed the version time difference by this commit. ngbatis-demo depends on 'ngbatis' from local maven repository when building. 31cac59

This makes a lot more sense :) I saw this after replying to that :-P, thanks to the amazing @CorvusYe as always!


Add a LICENSE to this Repo

Hey @kmg7. Which type of license would fit to this project.

I think MIT would fit. Added already!


Bazel build fails downloading sqlite-amalgamation-3340100.zip

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Linux Ubunto 20.04
TensorFlow installed from (source or binary): source
TensorFlow version: v2
Python version: 3.8
Installed using: pip
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): gcc-14
CUDA/cuDNN version: no
GPU model and memory: no

Describe the problem
Building from sources from git clone https://github.com/tensorflow/tensorflow.git, using bazel fails
Provide the exact sequence of commands / steps that you executed before running into the problem
Command
bazel build //tensorflow/tools/pip_package:build_pip_package

fails with error:
Error in download_and_extract: java.io.IOException: Error downloading [https://storage.googleapis.com/mirror.tensorflow.org/www.sqlite.org/2021/sqlite-amalgamation-3340100.zip, https://www.sqlite.org/2021/sqlite-amalgamation-3340100.zip] to /home/attardi/.cache/bazel/_bazel_attardi/9e2174a9a29f23641a2f691abf17e76f/external/org_sqlite/temp4950223412785952933/sqlite-amalgamation-3340100.zip: connect timed out

Indeed:
https://storage.googleapis.com/mirror.tensorflow.org/www.sqlite.org/2021/sqlite-amalgamation-3340100.zip

is missing.
Any other info / logs
Strangely, the second alternative URL to download, does exists:
"https://www.sqlite.org/2021/sqlite-amalgamation-3340100.zip"

but bazel ignores it.

I downgraded to bazel 3.1.0 and switched to v2.4.1 and it worked:
sudo apt install bazel-3.1.0
git checkout tags/v2.4.1


no result after successfully import trivy scan report into defectdojo.

hello
i have implemented trivy tool to scan my docker images in gitlab ci/cd
then uploaded the report to defectdojo as the following:
curl --fail --location --request POST "${DEFECTDOJO_URL}/import-scan/"
--header "Authorization: Token ${DEFECTDOJO_TOKEN}"
--form "scan_date="${TODAY}""
--form "minimum_severity="${DEFECTDOJO_SCAN_MINIMUM_SEVERITY}""
--form "active="${DEFECTDOJO_SCAN_ACTIVE}""
--form "verified="${DEFECTDOJO_SCAN_VERIFIED}""
--form "scan_type="${DEFECTDOJO_SCAN_TYPE}""
--form "engagement="${DEFECTDOJO_ENGAGEMENTID}""
--form "file=@${DEFECTDOJO_SCAN_FILE}"
--form "close_old_findings="${DEFECTDOJO_SCAN_CLOSE_OLD_FINDINGS}""
--form "test_type="${DEFECTDOJO_SCAN_TEST_TYPE}""
--form "environment="${DEFECTDOJO_SCAN_ENVIRONMENT}""
variables:
DEFECTDOJO_SCAN_CLOSE_OLD_FINDINGS: "False"
DEFECTDOJO_SCAN_ACTIVE: "True"
DEFECTDOJO_SCAN_VERIFIED: "False"
DEFECTDOJO_SCAN_TYPE: "Trivy Operator Scan"
DEFECTDOJO_SCAN_TEST_TYPE: "GitLab-CI Trivy"
DEFECTDOJO_SCAN_FILE: "./gl-container-scanning-report.json"
When the pipeline is finished, I can see the report on the defectdjo web page but with 0 findings or results, note that the trivy report scan is generated successfully and it is not empty. it is a json file
Is there any thing wrong in the parameters that I defined in curl command.
defectdojo version: 2.26.4
trivy version: 0.46.1
PS:
DEFECTDOJO_URL, DEFECTDOJO_TOKEN, DEFECTDOJO_ENGAGEMENTID
I've already assigned values for those variables in my pipeline.

it works @kiblik,
thank you for your kind help.


Snapshot corruption after several saves

Found this in Bards Tale after playing for a few hours.   Saved snapshot would not load and results in crashing the emulator.
This may have occurred after several saves on the same emulator session - so the effect may have been cumulative (resulting in a few hours of game progress lost.)
Have yet to attempt loading the snapshot in the debugger.

Bards Talae I.clemens-sav.zip


Fix: Custom guard usage inside postLogin & authorize

$this->guard() still defaults to the project default guard through the AuthenticatesUsers trait but instead it should use the guard binded to VoyagerAuth.

Some more background info about this PR and issue it solves.
Our project's default Guard is set to use JWT tokens and api middleware. Voyager routes on the other hand use built-in sessions and web middleware.
So far previous developer had copied and modified some code to make everything work. Primarily seems to be:
VoyagerAuthController.php - To make login use the web guard instead of the default
routes.php                - To add web middleware to most of the routes

One day I stumbled upon Voyager's docs which stated that it's possible to just change the default auth guard (customization/custom-guard). This made it sound like our overrides were no longer needed. So I removed some of our overrides, changed the default guard for Voyager, but couldn't get it to work. After some digging found that:

VoyagerAuthController@postLogin still uses default Guard instead the one set for VoyagerAuth.
After v1.2 all ->authorize() calls also default to the project's default guard (AuthorizesRequests).

Edit: I'm fairly certain that following also need an update. If somebody can confirm I can add to this PR:

MediaPickerHandler@createContent:29 uses \Auth::user()
VoyagerMediaController@upload:209 uses \Auth::user()

Thats correct. Those files were created/changed after we implemented #3968.
Are you able to provide a phpunit test which tests login/logout and some other things with a custom guard?

Thats correct. Those files were created/changed after we implemented #3968.
Are you able to provide a phpunit test which tests login/logout and some other things with a custom guard?

👍🏻 Yes. Should be able to do this next week.

Just thought I'd add that an alternative approach for src/Http/Controllers/VoyagerAuthController.php would be to add the following override at the bottom of the class:
    protected function guard()
    {
        return app('VoyagerAuth');
    }
}

😃

@superhaggis Yep, that's better. Made the change.

Looks like you are done here?

Looks like you are done here?

Yes. Everything's done.


Authorizations > Front-end: Enhancement of the current concept (Sites)

@Fonsu What about another solution to have a pop-up mode property with enums read, update and create to display the Site or Company pop-ups.
And for the other kind of pop-up like Assign Users we can pass the whole Site object where we can directly read values.
What do you think?

Thank you for your review 🙂

I think we can pass the whole Site object to both pop-ups and if tomorrow we add new props, we'll benefit from getting them right from the object without changing anything in the dialog data / template.

That makes sense, that way we don't strip the site.id from site and pass it to the popup only to fetch the whole site from the backend again. Done in my last two commits.

@Fonsu What about another solution to have a pop-up mode property with enums read, update and create to display the Site or Company pop-ups.
And for the other kind of pop-up like Assign Users we can pass the whole Site object where we can directly read values.
What do you think?

Maybe 'enriching' site in sites-list-table-data-source.ts:220 so that there is also a mode that can be checked on site.component.html would be useful. I am not entirely sure if that is what you mean, though. With this approach there would be no need to 'force' canUpdate to false in the view action, which would definitely improve readability.

That makes sense, that way we don't strip the site.id from site and pass it to the popup only to fetch the whole site from the backend again. Done in my last two commits.

Thanks!

Maybe 'enriching' site in sites-list-table-data-source.ts:220 so that there is also a mode that can be checked on site.component.html would be useful. I am not entirely sure if that is what you mean, though. With this approach there would be no need to 'force' canUpdate to false in the view action, which would definitely improve readability.

You can check this when the action is triggered and set the PopupMode when calling it.


Automatic releases to MavenCentral & Github

Currently all our deployments require a manual release step through https://s01.oss.sonatype.org/#stagingRepositories.
It would be good to have a completely automated deployment pipeline that does the following.

Builds the binaries and runs tests
Deploys binaries to a publicly available state on MavenCentral
Creates the release in Github with release notes.

https://github.com/Apadmi-Engineering/Mockzilla/issues/7
These two issues can be done together.

https://github.com/vanniktech/gradle-maven-publish-plugin
Recommended by @KieronQuinn

This wouldn't allow fully automated releases FYI, you still need to go onto the control panel and click "Release", but it does get it to that stage (no need to close things)


setImageUri takes too long

When I get a photo from G+ Photos app, sometimes this need to be downloaded. In this case the app will hang for some time and I get an ANR.
With older devices, decodeBitmat takes too long too.
I think that setImageUri needs to be executed on a background thread. The method setImageUri can accept a callback and return after the job has done.

This seems like a problem worth solving.  @ArthurHub, what do you think the right answer should be here? Disabling the crop support and drawing some sort of loading indicator on the crop image view while downloading the Bitmap?

Yep, that sounds reasonable, maybe provide the client with optional widget to be placed while it is loading.

Hmm... how about an async overload of setImageUri with just a basic success/failure callback? That should let the client toggle whatever UI state they want.

@sethgho That's exactly what I did.
I will improve the code and make a pull request.

:+1:

:+1:

added setImageUriAsync(Uri) and getCroppedImageAsync() in v1.1.0


QR-decomposition through Givens rotations

Added a module that contains a method for computing the QR decomposition of a Hessenberg matrix through Givens rotations. Givens rotations are stored in a list.

So there's one issue: you should collect the rotations during the qr-factorization of H, cause they depend on each other. If you apply givens rotation number 1, you modify row number 2, so that influences the second givens rotation.


Fix Unstable Test

We have one test which fails randomly in GitHub CI. Example: https://github.com/flink-extended/flink-scala-api/actions/runs/6855224529

I would be glad to look at it, but the build has already expired, is this still relevant?

@buinauskas it would be great. I think it is still relevant. I believe there we have some race-condition. I have seen it mainly in CI job.

Do you know which test is that? The builds and their logs have already expired so it's difficult to troubleshoot with that little information.

I think it was this test: ExampleTest

Hopefully this is the right fix to close. Thanks again @buinauskas


Mutual auth between server and client

Hey!
I am trying to implement mutual authentication between my web and my client.
For the moment, I configured the server side auth, using:
-v /path/to/certs:/etc/nginx/certs
It works properly with the server side.
To enable the user auth I amb trying to modify the /etc/nginx/conf.d/default.conf mounting my own .conf file, which includes the following lines:
  ssl_client_certificate /etc/nginx/certs/ca/ca.crt;
  ssl_verify_client on;

But the nginx-proxy container don't start when I load the .conf.
How can I modify it to enable user authentication?
(I created the user cert and imported it to the browser)
(Using Docker 17.12.0-ce , docker-compose 1.18.0 and the latest version of jwilder/nginx-proxy)
Thanks!

The answer is in: https://github.com/jwilder/nginx-proxy/issues/1027#issuecomment-362042435


Not able to  Single sign-out using the EndSession

I have followed the instruction to set up the logout url for single sign out but it is not working. I have 2 apps with logout url setup. I login to the apps in the same browser tab and logout of one app but still the other app is logged in. If call the EndSession end point manually for the 2nd app then it logged out. Seems like AZURE AD is not calling the logout url automatically. Not sure if I am missing some other settings related to single sign out.

@ismail1971 for a single sign-out you might want to have a look at the following sample:
https://github.com/Azure-Samples/active-directory-dotnet-web-single-sign-out
Closing this issue as I think the question is answered
cc: @kalyankrishna1


add subplots with letter numbering

a function that augments plt.subplots by adding letter numbering to the subplots.

i added the example image to the docstring, it really helps to be able to see the example figure on readthedocs.
could you also update the docstring format to match the style guide? https://analyze.readthedocs.io/en/latest/documentation-guidelines.html


CI: Dependency checks (#176)

What this PR changes/adds
The CI verify workflow is extended with two Gradle tasks for analyzing EDC module dependencies:

A custom gradle plugin defines rules on which module dependencies are allowed within EDC (e.g. no module may depend directly on any of the core modules).
The Dependency Analysis Gradle Plugin used for detecting unused dependencies and dependencies declared on the wrong configuration (api vs implementation vs compileOnly, etc.).

The tasks are initially set to only emit warnings, but will be configured to fail builds once all warnings have been resolved.
Outputs can be seen in two tasks in the Test Code (Style, Tests) workflow -> Dependency-analysis:

Dependency rules report (408 violations, see "Dependency Rules" in CI run)
Dependency analysis report (~500 lines, see "Dependecy Analysis" in CI run)

Why it does that
The custom Gradle plugin contains logic implementing the rules described in #827 . These rules can be executed locally using:
./gradlew applyDependencyRules -Pdependency.analysis=warn -Pdependency.analysis.clear.artifacts=false

The Gradle Java library plugin defines a clear separation between dependency configurations. This allows controlling and limiting the use of transitive dependencies.
The Java library plugin introduces the concept of ABI (Application Binary Interface) which includes (among others) types used in public methods. Dependencies that define those types should be defined in the api configuration. Other dependencies should be defined in the implementation configuration so that they are not transitively exposed, and therefore do not leak into the consumers' compile classpath.
The Dependency Analysis Gradle Plugin analyzes the abstract syntax tree to determine the ABI. On that basis, it can recommend which import configurations should be changed. The dependency analysis can be executed locally using:
./gradlew buildHealth -Pdependency.analysis=warn -Pdependency.analysis.clear.artifacts=false

Further notes
Modified some of the project files to implement the recommendations from the tool for the core module. This validated that the recommendations were reasonable:

:core:contract:  scope of :common:state-machine-lib and :common:util changed from api to implementation.
:core:contract:  scope of :common:state-machine-lib changed from api to implementation. Removed unused dependency to :common:util

Increased compiler heap space to deal with a memory error.
Linked Issue(s)
Closes #827
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods?
[x] added/updated relevant documentation?
[x] added relevant details to the changelog? (skip with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #1000 (a94d541) into main (3b7d179) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1000   +/-   ##
=========================================
  Coverage     64.87%   64.87%
  Complexity     2879     2879
=========================================
  Files           677      677
  Lines         14821    14821
  Branches       1015     1015
=========================================
  Hits           9615     9615
  Misses         4740     4740
  Partials        466      466

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 3b7d179...a94d541. Read the comment docs.


The DB tables list. Only one table remains In the "Search" Tables list after XML Showplan opens.

Issue Type: Bug

Choose "Manage" on any database,
Click "Select Top 1000" on any table in the DB
In the opened query tab, click the "Explain" button,
Choose the "Results" tab
Click on the XML Showplan row to view XML.
Return to the DB Tables list tab.

Expected result:
The tables list contains all the tables present in the DB.
Actual result:
Only one table item is available in the DB tables list, the table name is not full, the "Refresh" button makes the whole name visible but doesn't show other DB tables.
Azure Data Studio version: azuredatastudio 1.6.0 (eb35dae1d1238e03f1cab3bd45e151f381b22aef, 2019-04-17T07:11:15.944Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2592)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.90GB (8.13GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Thanks for reporting!  I've seen this a few times but couldn't get repro steps.


vendor is not defined

On master branch, debian gnu linux, hot mode.
module.exports = vendor;

//////////////////
// WEBPACK FOOTER
// external "vendor"
// module id = 3
// module chunks = 0

external "vendor"?b940:1 Uncaught ReferenceError: vendor is not defined
    at eval (eval at <anonymous> (bundle.js:608), <anonymous>:1:18)
    at Object.<anonymous> (bundle.js:608)
    at __webpack_require__ (bundle.js:557)
    at fn (bundle.js:88)
    at eval (eval at <anonymous> (bundle.js:602), <anonymous>:1:19)
    at Object.<anonymous> (bundle.js:602)
    at __webpack_require__ (bundle.js:557)
    at fn (bundle.js:88)
    at eval (eval at <anonymous> (bundle.js:596), <anonymous>:256:89)
    at Object.<anonymous> (bundle.js:596)
(anonymous) @ external "vendor"?b940:1
(anonymous) @ bundle.js:608
__webpack_require__ @ bundle.js:557
fn @ bundle.js:88
(anonymous) @ module.js from dll-reference vendor?d6f0:1
(anonymous) @ bundle.js:602
__webpack_require__ @ bundle.js:557
fn @ bundle.js:88
(anonymous) @ client.js?5f51:256
(anonymous) @ bundle.js:596
__webpack_require__ @ bundle.js:557
fn @ bundle.js:88
(anonymous) @ bundle.js:587
__webpack_require__ @ bundle.js:557
(anonymous) @ bundle.js:580
(anonymous) @ bundle.js:583

Was because I broke it myself.


Is this normal to use 12G GPU memory to load the 200m model with default parameter?

Our code (following the sample)
model = timesfm.TimesFm(
    context_len=128,
    horizon_len=5,
    input_patch_len=32,
    output_patch_len=128,
    num_layers=20,
    model_dims=1280,
    backend=backend,
)
model.load_from_checkpoint(repo_id="google/timesfm-1.0-200m")

then our process gives the GPU memory usage around 12237MiB

that is a lot

I have 64gb of ram and 64 of swap and it oom XD sooooo

Can you try setting the environment variable: XLA_PYTHON_CLIENT_PREALLOCATE=false ?


[SPARK-36053][CORE] Extract a helper method to eliminate duplicate code related to delete abnormal disk block object file

What changes were proposed in this pull request?
The main change of this pr is extract a helper method named deleteAbnormalDiskBlockObjectFile to delete abnormal file write by DiskBlockObjectHelper and use this method to clean up the duplicate codes in BypassMergeSortShuffleWriter, ExternalAppendOnlyMap and ExternalSorter.
Why are the changes needed?
Clean up the duplicate codes
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass the Jenkins or GitHub Action

Test build #140803 has started for PR 33267 at commit e743e7d.

Test build #140804 has started for PR 33267 at commit 9dfc6a0.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45315/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45315/

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

Test build #140803 has finished for PR 33267 at commit e743e7d.

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

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

Test build #140804 has finished for PR 33267 at commit 9dfc6a0.

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

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

Test build #140894 has started for PR 33267 at commit fc14fb8.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/45406/

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

Test build #140894 has finished for PR 33267 at commit fc14fb8.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public final class SparkOutOfMemoryError extends OutOfMemoryError implements SparkThrowable
      throw new IllegalArgumentException(s\"Cannot find error class '$errorClass'\"))
class SparkArithmeticException(errorClass: String, messageParameters: Array[String])
  class RemoteBlockDownloadFileManager(
 class RocksDBStateStore(lastVersion: Long) extends StateStore
 class StateEncoder

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


fix(versions): update Activiti/activiti-cloud-dependencies versions into develop

UpdateBot pushed maven dependency: org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies to: 7.1.141

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.1.141


CharHits are showed incorrectly.

https://github.com/ferrywlto/EICS.Muddle/blob/c22e27e73f794b689b4029ff8f03809c0cfc104f/Game/Guess.cs#L28
For example, if the word contains one 'e', user can input "eeeee", CharHits are displayed wrong.

hi @ajlakanen thanks for creating issue. The char hit means the character is in answer but not in correct position. This is the intended behavior.


Retain TAB (\t) information on screen to recreate TAB on copy

This came out as a consequence of #982. But #982 should focus on retaining lost spaces (which is a bug to be fixed)
This is about explicitly adding the feature to retain any written TAB character when being translated to spaces when being written to the screen.
Expected is (would be nice to have) to recreate the TAB character when copying it back out (e.g. to the clipboard).
Test case
printf "A\tB\n"

While this yields A     B" on a single line on screen, when copying it back out, it must become A\tB` again.
I see pros and cons on both sides, so it might be interesting to have that feature configurable, such that users can decide what to do on tabs.
Configuration
Either via profile boolean flag or by extending the copy actions. But then the question would be how to distinguish this when using the vi normal mode.
Other Terminal's behaviour
didn't test much, but here we go:

Terminal
printf "A\tB\n"

gnome-terminal (vte)
retains TAB on copy

KDE konsole
TAB presented in spaces.

Oh interesting catch on Terminal.app. WRT roxterm, since it's libvte based, all TEs based on libvte should really behave the same on the lower end (that should include selection and TAB extraction). That makes sense.
We currently simply don't store the TAB information in the grid and directly move the cursor mod 8-style to the right. I didn't have the time to think of a proper way for implementing such a feature, sorry about this.

I'm currently doing something similar to vte in #999


'Connection' clarification

Recently I've come across an issue with multiple Laravel applications on a single server. These applications happen to share the same tube names, so it is possible for one application to create a queue job, and another application to execute it.  Obviously this is not good, and results in a lot of inadvertent results.
I opened a PR for Laravel to try and correct this by automatically adding a prefix to the tube name, but this proved a little more complicated than originally thought, and was reverted for the time being.
Other queue drivers have different ways of preventing this crosstalk. For example, the database driver will be connecting specifically to your application's database.  The SQS driver has a key/secret that connects to a specific instance on your Amazon account. Redis allows you to specify a "connection".
Looking through the Beanstalkd documentation, it appears Beanstalkd also has the concept of a "connection", but I'm having trouble figuring out how to utilize it.
Was wondering if you could help shed some light on if I'm understanding Beanstalkd connections correctly, and if they would be an appropriate solution to prevent that crosstalk that I've been seeing on my server.
Thanks so much for any help you can offer.

Other queue drivers have different ways of preventing this crosstalk. … Redis allows you to specify a "connection".

Note that a redis connection will not help address this issue — if you have multiple applications connecting to a single redis instance, they will be able to read and write the same keys. Beanstalkd connections are like memcached or redis connections — they're just regular TCP connections. I don't think they'll help you with your issue.
Your original approach was the right thing to do: put the names used by different applications into different namespaces. This is the same technique you'd use in memcached or redis as well.

thanks for the response! really appreciate the help.  i'll try and address it with the prefix again


Sync keyvault test resource template to the other languages(.net)

test resources json for keyvault and the post deployment ps1 script
sync up with the .net version


Config generator should return meaningful error for invalid service config

To reproduce:
Run ConfigGeneratorTool with --service_yaml flag pointing to a service config that cannot be parsed by the tools framework (for example, by adding an unrecognized section).
Expected behavior:
ConfigGeneratorTool checks DiagCollector after building its Model, and throws a meaningful exception if the collector has any errors. It should also report any other diagnostics that the DiagCollector has accumulated, even if building succeeds.
Observed behavior:
Hard-to-debug stack trace with exception that occurs in the api-compiler, rather than toolkit.
ERROR: toplevel: Unexpected exception:
java.lang.NullPointerException
    at com.google.api.codegen.config.ConfigGeneratorApi.process(ConfigGeneratorApi.java:74)
    at com.google.api.tools.framework.tools.GenericToolDriverBase.run(GenericToolDriverBase.java:85)
    at com.google.api.tools.framework.tools.ToolDriverBase.run(ToolDriverBase.java:85)
    at com.google.api.codegen.config.ConfigGeneratorTool.generate(ConfigGeneratorTool.java:74)
    at com.google.api.codegen.config.ConfigGeneratorTool.main(ConfigGeneratorTool.java:62)

The null pointer is because the tools framework returns a null Model rather than throwing an exception if an error occurs during initialization; the errors must be retrieved from the DiagCollector.
/cc @shinfan

On further investigation, the ModelBuildResult (from which the Model is drawn) never propagates its diags if there is an error... so it these errors get dropped before they ever make it to ConfigGeneratorTool. That's unfortunate because then we cannot check for errors during Model building.
@the8bit -- is that interpretation correct? (That is the Model cannot be built, then at this line, the Model is null, and so this.diag never gets access to the errors collected in the builder's DiagCollector, so the tool has no idea what error caused the failure.) If so, why do we want the if (buildResult.getModel() != null) condition at all?

I think the if() is not needed, it should always just set  this.diags = buildResult.getDiagCollector();
This is probably a bug I introduced when pulling the model building logic out of ToolDriverBase. There was some funky coupling between model / DiagCollector in ToolDriverBase that I was working on fixing and this seems like collateral damage.

Ok, let's change this internally and I can circle back to this once there's a new release of API compiler.

@geigerj Is this still relevant?

I think this was fixed by https://github.com/googleapis/api-compiler/commit/c6d2d33609b60241db80a4015cb0b91b45b28fac. If not, it should probably be tracked in the api-compiler repo, rather than here.


make tmpfile location handling more robust

check if require('os').tmpdir() is actually a directory
if not, set it to the install path of this module
if the temp file fails to write, throw with an error message that clearly indicates what v8flags is, why the error occurred and possibly provide suggested steps to resolve.

/ref tkellen/js-liftoff#45

I pushed gulp-npm-test once more to illustrate that setting the --gulpfile doesn't help and to get the tmp location for you which is /tmp, as expected.
I can give you a failing test if you'd like to have gulp and batshit as devDependencies - batshit will use bats to to run gulp.  That would compete with mocha though.  And would be a temporary measure (batshit will never be supported on windows), and you could achieve the same by cloning gulp-npm-test on a linux and npm linking the js-v8flags.  I also just encountered an issue with batshit that may cause it to fail on linux in some cases.  Not a problem with travis (as I use batshit to test other modules).  I'm guessing that you don't want neither gulp nor batshit to test this...
Or maybe some project that tests cli usage from an outside-of-node perspective?

Ok, I got the fail test done without extra dependencies.  The issue is with bats which changes $TMP to a dir that exists only temporarily.  For example if I create a test/tmp beforehand (in this case ./tmp, then all is ok.  Make sure you test this with linux, because somehow it not a problem on a mac.

/ref npm/npm#4531

resolved in cf81d5b6f801ad5c44c2559e21c636de69a6cdd9


Make pytest run in separate pods

Signed-off-by: Jenny Li<EMAIL_ADDRESS>/kind improvement

/lgtm


Remove Position Radio Buttons

Hello JiHong. For the images and videos i remove all things.
videoResizing: false,      videoHeightShow: false,      videoHeight: false,      videoRatioShow: false,      videoResizing: false,      imageResizing: false,      imageHeightShow: false,      imageWidth: false,      imageHeight: false,
It works! But how i can remove the positions radio buttons? I mean "standard", "left", "center", "right" ? Many thanks :) The option "Standard" is allways checked. I like only "standard".

The 2.42.0 version has been updated.
If this issue has not been resolved, please reopen this issue.
Thank you.


No padding in issue preview on hover

Description
When you hover over an issue number, popup has no paddings
https://try.gitea.io/stuzer05/test-preview/pulls/1
Gitea Version
trunk
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
No response
Operating System
No response
How are you running Gitea?
https://try.gitea.io
Database
None

It seems to be caused by
.tippy-box[data-theme="menu"] .tippy-content {
  padding: 0;
}

@silverwind I think it's a regression of making the theme=role for menus?

Yes, it's a regression. The code-line-menu adds it's own padding, so this tippy content needs to set a padding itself.

The default "theme" by "role" still seems a little confusing to me.
I tried to propose  Fix issue popup padding #25254  , use "default" for the theme (the "default" is a non-existing selector, so I think it does it work). Does it look good to you or do you have better approaches?


Update README.md

The docs have a bug. Under the usage it says to put the django-app into the INSTALLED_APPS in settings.py - but it has a wrong structure.
In the proposed change I have fixed it to the correct layout.

Your are correct. Thanks for the PR!


Website content audit planning

Objective:
Understand what we currently have on the website and make a plan on updating the website pages.
Tasks:

[ ] Take a look at all the website pages and update the Airtable inventory document.
[ ] Make a plan on when to update what and who we should assign the update task.
[ ] More to add.

Acceptance

[ ] Task issues for the update are created and ready for prioritization.

Content by section:
Guidelines
Layout foundations
https://ibm.box.com/s/1lm1zs6zcof4epd5k9i7wbk00o82glsj
Type scale
Spacing scale
Type pairing

Images update:
We need to do an audit and make a plan to update the images to meet IDL image ratio standards. [Jenny] - I might be pushing it.
FAQ page
We need to update or add more FAQ questions to the website page.

@shixiedesign The items you mentioned above are existed and have task issues to address them. The next step is for me to reset the priority so we can get started on them - FYI.

We have individual task issues created to address the needs, and the website enhancement work (next gen) will be done in a later time. This task issue can be closed.


TCP proxy route by domain name

Description:
I have two different PostgreSQL instance and i want route by domain name in port 5432 like below:

if use pg1.example.com (for X application) got to cluster pg1_cluster
if use pg2.example.com (For Y application) got to cluster pg2_cluster

and i don't know how to use filter or any other options in tcp_proxy
Config:
static_resources:
  listeners:
    - name: postgres_listener
      address:
        socket_address:
          address: <IP_ADDRESS>
          port_value: 5432
#      filter_chains:
#        - filters:
#            - name: envoy.tcp_proxy
#              config:
#                stat_prefix: pg1
#                cluster: pg1_cluster

  clusters:
    - name: pg1_cluster
      connect_timeout: 1s
      type: strict_dns
      lb_policy: MAGLEV
      hosts:
        - socket_address:
            address: pg1_server   # <IP_ADDRESS>
            port_value: 5432

   - name: pg2_cluster
      connect_timeout: 1s
      type: strict_dns
      lb_policy: MAGLEV
      hosts:
        - socket_address:
            address: pg2_server   # <IP_ADDRESS>
            port_value: 5432

This is not a feature of the TCP proxy, nor can it be done with the filter chain match as at L4 there is no  knowledge of domain name.
However, if you use the http connection manager you can create L7 aware routing rules like you describe.

If you use TLS you might get away with using the SNI to do that:
https://www.envoyproxy.io/docs/envoy/latest/configuration/listener_filters/tls_inspector
https://www.envoyproxy.io/docs/envoy/latest/api-v2/api/v2/listener/listener.proto#envoy-api-msg-listener-filterchainmatch

Sorry, some of the parts for me not understandable.
It's possible to take a sample configuration (how use filter)?

Maybe something like this can help
---
static_resources:
  listeners:
    - name: postgres_listener
      address:
        socket_address:
          address: <IP_ADDRESS>
          port_value: 5432
      listener_filters:
        - name: "envoy.listener.tls_inspector"
          config: {}
      filter_chains:
        - filter_chain_match:
            transport_protocol: tls
            server_names:
              - pg1.example.com
          filters:
          - name: envoy.tcp_proxy
            config:
              stat_prefix: pg1_cluster_stats
              cluster: pg1_cluster
          tls_context:
            common_tls_context:
                tls_certificates:
                - certificate_chain: { filename: "...." }
                  private_key: { filename: "..." }
        - filter_chain_match:
            transport_protocol: tls
            server_names:
              - pg2.example.com
          filters:
          - name: envoy.tcp_proxy
            config:
              stat_prefix: pg1_cluster_stats
              cluster: pg2_cluster
          tls_context:
            common_tls_context:
                tls_certificates:
                - certificate_chain: { filename: "...." }
                  private_key: { filename: "..." }

  clusters:
    - name: pg1_cluster
      connect_timeout: 1s
      type: strict_dns
      lb_policy: MAGLEV
      hosts:
        - socket_address:
            address: pg1_server   # <IP_ADDRESS>
            port_value: 5432
    - name: pg2_cluster
      connect_timeout: 1s
      type: strict_dns
      lb_policy: MAGLEV
      hosts:
        - socket_address:
            address: pg2_server   # <IP_ADDRESS>
            port_value: 5432

sorry, but i can't connect with psql, how to connect with client modules to proxy?
create tls file:
openssl req -x509 -nodes -days 365 -newkey rsa:2048 -keyout /my-patch/pg1.example.com/tls.key -out /my-patch/pg1.example.com/tls.cert-subj "/CN=pg1.example.com"

openssl req -x509 -nodes -days 365 -newkey rsa:2048 -keyout /my-patch/pg2.example.com/tls.key -out /my-patch/pg2.example.com/tls.cert-subj "/CN=pg2.example.com"

psql connection:
psql "host=pg1.example.com user=postgres dbname=postgres sslmode=prefer"

admin:
  access_log_path: "/dev/null"
  address:
    socket_address:
      address: <IP_ADDRESS>
      port_value: 8001

static_resources:
  listeners:
    - name: postgres_listner
      address:
        socket_address:
          address: <IP_ADDRESS>
          port_value: 5432
      listener_filters:
        - name: envoy.listener.tls_inspector
          config: {}
      filter_chains:
        - filter_chain_match:
            transport_protocol: tls
            server_names:
              - pg1.pooya.partdp.ir
          filters:
            - name: envoy.tcp_proxy
              config:
                stat_prefix: pg1_cluster_stats
                cluster: pg1_cluster
          tls_context:
            common_tls_context:
              tls_certificates:
                - certificate_chain: { filename: "/my-patch/pg1.example.com/tls.cert" }
                  private_key: { filename: "/my-patch/pg1.example.com/tls.key" }

#        - filter_chain_match:
#            transport_protocol: tls
#            server_names:
#              - pg2.pooya.partdp.ir
#          filters:
#            - name: envoy.tcp_proxy
#              config:
#                stat_prefix: pg2_cluster_stats
#                cluster: pg2_cluster
#          tls_context:
#            common_tls_context:
#              tls_certificates:
#                - certificate_chain: { filename: "/my-patch/pg2.example.com/tls.cert" }
#                  private_key: { filename: "/my-patch/pg2.example.com/tls.key" }

  clusters:
    - name: pg1_cluster
      connect_timeout: 1s
      type: strict_dns
      lb_policy: MAGLEV
      hosts:
        - socket_address:
            address: proxy-pg1
            port_value: 5432

    - name: pg2_cluster
      connect_timeout: 1s
      type: strict_dns
      lb_policy: MAGLEV
      hosts:
        - socket_address:
            address: proxy-pg2
            port_value: 5432

I guess you are be out of luck. Could not find anything useful regarding postgresql and sni.
I dont think this has to do with envoy anymore, and you might want to close this issue.

What is the solution for this?
A section of envoy.yaml
```yaml
admin:
  address:
```

@poyaz  hello, did you resolve the issue? I have the same one.

@dotw
no, i can't resolve the issue

The config pasted above should be sufficient to do SNI based routing of TCP connections made with TLS.
Based on some quick searching, the Postgres protocol starts unencrypted and then upgrades to SSL as part of the protocol. At that point it's too late for SNI to be of use for Envoy's tcp proxy.
Envoy would need an extension that understood at least part of the postgres protocol (and handled upgrade to SSL) in order route connections.

Community already working on it: #2861


Make this project a thing

This is an epic for making this project a thing. This will be a small wrapper around done-serve, but we will need a couple of changes to make it possible.

Last thing remaining is publishing the article. 🎇

It's out.


No signature of method: build_cs1dxgpbzjvnqu203tc6dv6un.bukkit() is applicable for argument types:

A problem occurred evaluating root project 'Hordes-1.8'.
> No signature of method: build_cs1dxgpbzjvnqu203tc6dv6un.bukkit() is applicable for argument types: (build_cs1dxgpbzjvnqu203tc6dv6un$_run_closure6) values: [build_cs1dxgpbzjvnqu203tc6dv6un$_run_closure6@4ad59117]
  Possible solutions: wait(), wait(long), mkdir(java.lang.Object), uri(java.lang.Object), split(groovy.lang.Closure)

When I add depend = asList('WorldGuard', 'PlaceholderAPI') to the bukkit task. been stuck on this for a bit. gradle version: 7.5.1

Issue resolved, asList was the issue. needs to be `["a", "b"]


Simple Auth example

Hey all, I'm having trouble wrapping my head around the new auth API update.
For my use case, I'd like to only authenticate that a user is the owner of a wallet. The way I'm envisioning the interaction is-

The app generates an auth token
A request is generated and opened in a browser (to the wallet connect uri? not sure)
The user signs into their wallet in the browser
the request is approved or denied and the answer (along with the wallet address) is passed back to the app.

-or maybe-

The user provides a wallet address
The app opens a browser with a request uri for the transaction/wallet address
The user signs in via the browser
The request is approved or denied and responds to the app

I'm pretty green working with wallet connect, so I may be misunderstanding some fundamentals with the interactions. Any help getting started would be appreciated. :)

Hello,
You can find an example of Auth API usage in the WalletConnect docs.

Hello,
You can find an example of Auth API usage in the WalletConnect docs.

Hey skibitsky, thanks for the response. I've run through the example on the site, but When I run the localhost example nothing comes up in the browser when navigating to http://localhost:3000/login address.

Bump- any updates on this? I’m still struggling with getting it set up.

Helo,
You don't need a server to use the Auth API. It's true that this can be confusing with native apps. The Aud and Domain fields can be filled with any URL. These values will show up in a wallet via the Auth API, but they can technically be any URL. In the example below, I use github.com. You could use your website address for both fields.
var dappOptions = new AuthOptions()
{
    ProjectId = "‼ YOUR PROJECT ID",
    Metadata = new Metadata
    {
        Description = "An example dapp to showcase WalletConnectSharpv2",
        Icons = new[] { "https://walletconnect.com/meta/favicon.ico" },
        Name = "WalletConnectSharpv2 Dapp Example",
        Url = "https://walletconnect.com"
    },
    Storage = new InMemoryStorage()
};

var dappClient = await WalletConnectAuthClient.Init(dappOptions);

var localhostAuthRequest = new RequestParams()
{
    Aud = "https://github.com/login",
    Domain = "github.com",
    ChainId = "eip155:1",
    Nonce = CryptoUtils.GenerateNonce()
};

Console.WriteLine("Making request...");
var uriData = await dappClient.Request(localhostAuthRequest);
var uri = uriData.Uri;

Console.WriteLine($"URI: {uri}");

void OnAuthResponded(object sender, AuthResponse args)
{
    var sessionTopic = args.Topic;
    var cacao = args.Response.Result;
    var signature = cacao.Signature;
    Console.WriteLine($"{sessionTopic}: {signature}");
}

void OnAuthError(object sender, AuthErrorResponse args)
{
    var sessionTopic = args.Topic;
    var error = args.Error;
    Console.WriteLine($"{sessionTopic}: {error}");
}

dappClient.AuthResponded += OnAuthResponded;
dappClient.AuthError += OnAuthError;

await Task.Delay(100000);

@skibitsky
Thanks for the update. I think my confusion lies primarily in how to get that popup to appear. I can run the code in a console app and have it generate a uri and wait for an event, but I don't know where to go to connect my wallet or where that connect request is meant to appear.
I'm using metamask as a browser extension if that helps.

@CodeRedCatto
MetaMask currently does not support the Auth Client SDK. MetaMask only supports a subset of the Sign Client SDK. The example you linked is using the Sign Client SDK, not the Auth SDK.
I recommend testing the Auth example using another wallet, such as Trust Wallet or a wallet that supports the Auth Client SDK.

@gigajuwels
Aha! I thought I was going crazy. I was able to successfully use the sign API for my use case. I only enabled the "personal_sign" flag with eth mainnet, and I'm able to derive the wallet address from the first namespace.
Are there specific teardown processes that should be run after I've found the wallet address?
Other than that, I'll help pester Metamask for auth api support.  :)

@CodeRedCatto did you manage to solve the issue with the WalletConnectSharp.Common.Model.Errors.WalletConnectException: '{params} was not initialized' ?
I'm having a similar issue. I authenticate the user end get the Access Token, but after around 5 minutes, as you also mentioned, I get the exception, although I think all my requests are resolved.

@CodeRedCatto manage to solve the issue by using the 2.1.2 version. That one had no timeout or params issues for the past 2 weeks i've been using it.
Can this be a bug that was resolved in that version and reintroduced after? I don't know who should i tag to take a look if it's a potential bug.

@levente-daczo I will look into it, thanks!

Hey @levente-daczo @CodeRedCatto, could you try v2.1.7 please? The timeout but have been fixed.

@skibitsky v2.1.7 works fine in my project, just like v2.1.2
👍 nice job


TypeError: e.referenceImages.entities.toReversed is not a function or its return value is not iterable

Everything seems to work fine, but clicking canvas gives this error every time and refreshing does nothing.

Most likely this is a problem of InvokeAI itself, this project only helps to host it on cloud servers. Today I will try to cause this error and write what the problem is.

I don't have enough information to reproduce this error, but it's definitely an issue with InvokeAI itself. You might want to wait for a release that fixes this. You can also submit this issue to InvokeAI to ensure it gets resolved in the next release.

Will do. It only did this by clicking the canvas at first. Now it's doing this every time I open the tunnel.

Have you tried restarting the environment?

Yes and I get the same error.

Got it. Now I'll be watching for the answer to the error you left at InvokeAI.

I get the typeError when opening the link for the ui, but an option to refresh to the model tab. Reloading loads up the ui and every other tab works except the cavas which causes the the same typeError. Very strange.

No matter how hard I try, I can't reproduce this error. Here are a few tips that might help (though they don't guarantee a solution).

Clear the cookies of the site where your webui is running.
Update your browser to the latest version.
If you can access your webui, go to the settings, clear the cache, and click restart webui.
Try launching your webui without loading data from Google Drive first.
If none of this helps, I'm afraid I’m out of options. I have no issues with launching or generating.

I tested out on my phone with kiwi. It indeed WAS my browser. That was embarrassing. Now I just need to figure out how to update chrome without screwing up my ublock.


Epoch-Changes tree pruning was lagging by one epoch

This PR improves the epoch changes tree with respect to pruning.
Apparently we were removing one level less than how much we may remove.
The consequence were really painless, we were only lagging by one epoch in our tree pruning strategy.

A test to exercise the fix has been added

NOTE: a babe test has been fixed. Apparently it was not checking for the right thing. Extra scrutiny and attention is thus required

Could you please check that it does not break warp sync?


docs: Add first draft of style guide

This commit adds the first draft of a style guide for documentation authors. The purpose is to help content authors produce more consistent documentation.
Closes #3641.
Signed-off-by: Vikram Vaswani<EMAIL_ADDRESS>

Adding many people as reviewers to increase awareness of this guide.


Demo fails with "expected key slot is undefined"

Describe the bug
The demo on the website (https://lodestar-lightclient.chainsafe.io/) fails with:
Error initializing from trusted checkpoint 0x858663a5f54e9cfc44c50bbfd3701dae9855429f1eecdf7536435f9d398abce1: JSON expected key slot is undefined

Expected behavior
Not sure since  I've never seen the demo
Steps to Reproduce

Go to https://lodestar-lightclient.chainsafe.io/
Click "Fill with trusted checkpoint"
click the initialize button

@Ivshti Is this still relevant? I can't reproduce this.

don't think so, safe to close I think


Adding_proxy_extension

Adding Proxy Extension
Fixes:  #818

[x] New feature (non-breaking change which adds functionality)

[x] I have made this from my own

[x] I have taken help from some online resourses

[x] My code follows the style guidelines of this project

[x] I have performed a self-review of my own code

[x] I have commented my code, particularly in hard-to-understand areas

[x] I have made corresponding changes to the documentation

[x] My changes generate no new warnings

Closes: #818

Change the directory name: Proxy Configuration/your files

Hey @Sulagna-Dutta-Roy , Kindly review the changes


[DO NOT MERGE] add exposition-only environment manipulation utilities

This PR adds some exposition-only utilities for creating and manipulation of queryable environment objects. They will make many of the algorithm specs shorter and more precise.

I failed to show in this PR that I also added a semantic requirement on queryable objects:

If tag_invoke(q, e, args...) is well-formed, then q(e, args...) is expression-equivalent to tag_invoke(q, e, args...).

Without this requirement, it is impossible to join two environments considering that for some query objects, operator() is always well-formed because it has a default.


Update the version for the current Flutter App as SVG files does not works in current version

/assign


[Bug] DOWN button & CHANGE button still working after Game Over in Tec]

What happened?

The DOWN ARROW button & the CHANGE/ROTATE button in the game is still working and is not disabled, when the game gets over. Moreover there is no reset button and the user has to refresh the whole game to play the game from beginning.
How can we reproduce this bug?
Uploading Tetris Game - Google Chrome 2024-05-19 01-08-06.mp4…
I have shown in the video, how to reproduce the bug
Desktop Information (Optional)
OS : Windows 11 Home
Browser: Google Chrome
Device: Laptop
Urgency (Optional)
Medium priority
Record

[x] I agree to follow this project's Code of Conduct
[x] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@mondalsurojit All the best 👍🏻

@mondalsurojit Any update ?

@swapnilsparsh Will be working on it...making a pr within 4 to 5 days


CPP Algorithm for Graph

Implement all type of Graph Functionalities with using different graph(Directed & Undirected Both)

Assign me this @siddharth25pandey

Some algos are added already like bfs dfs and prims, Ill add a few ones

Assign me this @siddharth25pandey

@anand4234 Go ahead
All the best :+1:

Can u plz assign this work to me?

Can u plz assign this work to me?

@Mohit123singh Go ahead
All the best 👍


7 changes to application process + form

@ricardomiron Can you please reassess the suggestions and update the list with the ones you want.

There should be Auto-verifiable emails

Email OTP log in instead of password

One email should be able to have only one account

The form should be auto-save instead of having a manual save button below each page

The first field - of application name - should be compulsory before users are able to move on to other questions.

The form + clarifications asked should have a note saying that their responses are going to be displayed publicly.

The form should have an ‘ask for help’ button on the side of each page that connects them to support id

@PelinSmines most of these are "nice to have" features.

Not sure what the implementation of "auto-verifiable" would mean, but I think it works ok as it is right now.
Would be great to explore a passwordless login like those "magic links" but definitely not a priority.
Yes this is something important I mentioned in our needed general changes, a unique issue for this should be opened.
Nice to have but not a priority.
This is already implemented when you create a new application being signed in.
Yes, this should be highlighted more clearly to applicants.
Great idea!


PMID:33483504

mitochondria absent
decreased number of ERMES foci
decreased phosphatidylethanolamine in mitochondria
decreased phosphatidylserine in mitochondria

decreased number of ERMES foci

Are ERMES foci the same as GO:0032865 'ERMES complex'?

Yes. I wasn't sure if there was a better way to describe this.
The title is:
Emr1 regulates the number of foci of the
endoplasmic reticulum-mitochondria encounter
structure complex
and  it isn't number of level of complex:

Can you think of a better way?

I still want to clarify the meaning of "ERMES focus" a bit more. Is it "a site where one or more ERMES complex(es) is/are located"? (Wouldn't it be lovely if the paper came out and said so?)
If so, the phenotype may reflect how much the ERMES complexes clump together ... it's probably worth including that the foci are bigger as well as fewer. Following the precedent we've set with vacuole terms, how about large ERMES foci present in decreased numbers (I'd include a "gloss" to include my made-up def of "ERMES focus")?

Yes, I think it is more  ERMES clumped together, good to mention that they are bigger too,
large ERMES foci present in decreased numbers
sounds perfect.

mitochondria absent from cell FYPO:0007712
large ERMES foci present in decreased numbers FYPO:0007713
decreased mitochondrial phosphatidylethanolamine level FYPO:0007714
decreased mitochondrial phosphatidylserine level FYPO:0007715


Apply button is far away

Instead of top left, Apply button should be closer to users, i.e bottom right

In 0.27 there isn't an Apply button


How do I upgrade from dev-master to 1.0?

Laravel Version: 5.5.1
Voyager Version: dev-master
PHP Version: 7.1.9
Database Driver & Version:

Description:
There's no information on how to upgrade from the dev-master (used it for Laravel 5.5 as Voyager wasn't released at the time).
How do I upgrade from dev-master to 1?
Steps To Reproduce:

There you go.
https://voyager.readme.io/docs/upgrading

Thanks but these instructions do not contain information on moving from the dev version to the release version. I did try them but none did the trick :-(

Can you describe the problem pls.

Seems I had a problem with my Git client. I fixed it and the upgrade worked but...
Even though I cleared cache and views there's a weird behavior:
Some stuff simply is missing on the menu/bread.
For example: in the menu builder I got menu items (for the admin) that appear on the builder just fine yet do not show on the actual menu at all (simply missing). I tried to refresh but they are still missing.
Another example is a recent Bread I added to a table isn't showing the edit/delete icons on each line (it did show them before moving from the dev version).
Any idea?

I think there are some problems in the FE since 1.0.*, I recently posted an issue I'm having with the media manager, so I guess we will have to wait for tcg team.

Thanks. That sucks... I hope it's fixed soon.


make fluentd:monitor-agent verbosity configurable

What would you like to be added:
The total volume of logging for the fluentd:monitor-agent sourcetype needs to allow for reduction in resolution to reduce daily ingest. I would like to see guidance on how to disable it and how to control the resolution based on how much of it you are willing to index per day.
Today, in practice, I just disable in the configmap, but would like to explore reduction in interval, as the data can be made very useful for a diagnostics dashboard :
@include system.conf
@include source.containers.conf
@include source.docker.conf
@include source.files.conf
@include source.journald.conf
#@include monit.conf
@include output.conf

Why is this needed:
some customers have millions of events for fluentd:monitor-agent across their 10s of collector pods. Adds up...Probably better used in metric store.
Looks like mine is doing about 30K events daily, to the tune of 3MB

The default emit_interval is 60 seconds.  You can reduce its log volume by specifying a higher value such as this.
<source>
  @id fluentd-monitor-agent
  @type monitor_agent
  @label @SPLUNK
  tag monitor_agent
  emit_interval 300
</source>

perfect! thanks!


Add a note that admin's project view changes to a table where there are at least 10 projects

could be added here

PR: https://github.com/NationalSecurityAgency/skills-docs/pull/117


 All the commits have already been merged in the master branch. These commits resolve the enum issue mentioned in https://forums.swift.org/t/fixing-enums-on-big-endian-systems/21730 and some other test failures seen on big-endian platforms, i.e., s390x.

Is there any particular reason you need these on the release branch?  As long as these have landed in master, you should be able to pull a toolchain that has them all in it.

Can one of the admins verify this patch?

Going to close this out. The 5.1 release has long since passed.


Feature/3536 upload video immediately to stream

This has been the branch we've been using in the SDK. I've merged the changes from master which include the code to stop streams when the recorder is closed. Should be tested thoroughly because there were many merge conflicts which had to be resolved.

Accidentally chose the upstream repo as target.


new term: MONDO:0100308 atactic disorder

reclassify children terms
close #2866

In case it is easier to view this way, here is a screenshot of the inferred view of the revised hierarchy:

I created new terms for sensory ataxia and vestibular ataxia. Can you comment if these annotations are incorrect:

(for vestibular ataxia, I used the UBERON class vestibular system

non-hereditary degenerative ataxia should go under acquired ataxia
Otherwise this seems reasonable!

thank you @pnrobinson!


Bug in Chain-based future's implementations of FusedFuture

https://github.com/rust-lang/futures-rs/blob/2f8943da9a0fcef208e86ae2f0f0152191629dcc/futures-util/src/future/future/chain.rs#L48
As you can see, once the second future returns ready it will return immediately without first transitioning to the Empty state, which is what the is_terminated function looks at:
https://github.com/rust-lang/futures-rs/blob/2f8943da9a0fcef208e86ae2f0f0152191629dcc/futures-util/src/future/future/chain.rs#L16-L18

I think this is fixed in #2128.

Yep.


build types before publishing to npm

fixes #43

[x] build the types before publishing to npm
[x] handle the two d.ts. files in src, and fix the duplicate types
[x] one bundled .d.ts file, or keep a hierarchy of directories and files?
[x] test the generated types with tsd? https://www.typescriptlang.org/docs/handbook/declaration-files/dts-from-js.html#tips -> no
[x] ensure it works as expected.

I ran npm pack and used that package in https://github.com/hyparam/hyperparam-cli components and hyparquet-demo. It showed some errors that I fixed with https://github.com/hyparam/hyparquet/pull/46/commits/8e62c07d9b9bdf22b2d9a2c3d96a77d81caccf3d (in these apps we directly import the types from hyparquet, not from hyparquet/src/types.d.ts).
We emit types/(file).d.ts AND types/(file).d.ts.map because I enabled https://www.typescriptlang.org/tsconfig/#declarationMap. I'm not 100% sure we need it, but it seems recommended.
Also, I'm not sure why the CI failed at some point (https://github.com/hyparam/hyparquet/actions/runs/12047750237/job/33590915755). I could not reproduce the errors locally, and they are weird (for example, we allow importing from JSON with "resolveJsonModule": true, in tsconfig.json, so the last error does not make much sense to me).
Update: I think the errors come from typescript 5.7:

import json: https://devblogs.microsoft.com/typescript/announcing-typescript-5-7/#validated-json-imports-in---module-nodenext
ArrayBuffer: https://devblogs.microsoft.com/typescript/announcing-typescript-5-7/#notable-behavioral-changes

The GitHub action was using tsc, ie the globally installed typescript binary, which happens to now be 5.7, and not 5.6.3 as locally installed. We could just have replaced tsc with npx tsc, or we could also upgrade to 5.7 and fix the issues.

Pushed changes to use new jsdoc @import syntax that was added in typescript 5.5. The upside is that typescript emits smaller declaration files with @import vs @typedef. 160kb -> 145kb unpacked size.

The generated declarations are a lot larger than before (117kb -> 143kb). Mostly because it's emitting types for a bunch of internal functions. This is understandable, and maybe even the right thing to do? But I experimented with various options:
@private / @internal jsdoc annotations
I wanted to not expose internal functions, since I don't want users to depend on them and then make it painful to evolve the internal code. In theory, you can annotate with jsdoc tag @internal and use tsconfig option "stripInternal": true but it doesn't work for me? [1] [2]
removeComments
If you set tsconfig option "removeComments": true it will strip the comments from the generated .d.ts files. This results in much smaller package (129.3kb). But when you try to use the package as a dependency in vscode, the hints no longer show the documentation. Not acceptable.
Combine js files and don't export
I did this for a few functions in header.js and datapageV2.js and asyncBuffer.js which saved a little bit, but I don't want to make one giant source file.
Conclusion
There's probably ways to improve this with tooling, but I don't want to complicate the build process more. If you have ideas @severo let me know. Otherwise this is probably good for now and will just eat the increased package size. (anyone who bundles it downstream won't really be affected by declaration size, but I'm still trying to keep the package as small as possible)

OK, thanks for sharing all the research.
https://api-extractor.com could help strip the declarations for the internal functions (as proposed in https://www.typescriptlang.org/tsconfig/#stripInternal) because I don't think we want to rely on another build tool. Anyway, we might want to add @internal as a hint for developers not to rely on these functions (and explain it in the README).
I have no specific idea to reduce the size

I'm getting a type error when using parquetRead and passing the utf8 option which appears to be related to this change. See #50.


Feature request: black compatibility mode

Ruff's website says:

The formatter is designed to be a drop-in replacement for Black.
Specifically, the formatter is intended to emit near-identical output when run over Black-formatted code.

However, there is an extensive list of differences between the output of the two formatters. On my project, I was not able to configure ruff and black in such a way that the two would result in the same output, or even allow the output of the other to go unchanged. The diffs I saw can be seen here.
If black-compatibility or gradual migration from black is a goal, it would be very helpful to offer a mode which allows the output of black to go unchanged. Without this, transitioning between the two formatters requires synchronized changes to both the formatter and the code being formatted.
I understand this may be a non-goal, but I figured it was worth raising here since the website explicitly lists Black compatibility as a goal/feature.

Is the right side of the diff there Ruff's output? If so, I think most of those are pretty clear improvements to the format. I understand it's tough to use both formatters at once though.
We are targeting Black compatibility, but it's very hard to reach 100% compatibility due to architectural differences. During the initial implementation, we spent time trying to close the gap from 99% -> 100% (e.g., https://github.com/astral-sh/ruff/pull/6343). I'm not sure how feasible it is to get closer. In the long term, we will probably diverge further from Black as we want to improve the produced format. Unfortunately, it's very hard to maintain multiple "modes" of the formatter, which is part of why we support a limited number of settings. I think this is a significant burden to a compatible mode as well.
cc @MichaReiser

We have goal to match Black closely so that migrating from black to ruff results in none or only very few code changes.
Achieving 100% compatibility is a non goal. It would require us to re-implement bugs in black or make our formatting worse just for compatibility.

transitioning between the two formatters requires synchronized changes to both the formatter and the code being formatted.

I recommend you against switching between formatters and instead advise you to commit to one (Black or Ruff).


Use FormSelect component

This PR changes:

Replaces Select to FormSelect component, so actual select tag with name attribute can be used
Adds RBAC permission check to display Create New Activation Button according to user permission

Codecov Report

Merging #40 (230466e) into master (0d29d4b) will increase coverage by 0.43%.
The diff coverage is 77.77%.

:exclamation: Current head 230466e differs from pull request most recent head 85fea11. Consider uploading reports for the commit 85fea11 to get more accurate results

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
+ Coverage   40.26%   40.70%   +0.43%
==========================================
  Files          45       45
  Lines         457      457
  Branches       80       82       +2
==========================================
+ Hits          184      186       +2
+ Misses        264      262       -2
  Partials        9        9

Impacted Files
Coverage Δ

src/Components/Forms/CreateActivationKeyForm.js
89.47% <ø> (ø)

src/Components/Forms/ActivationKeysFormSelect.js
80.00% <60.00%> (+8.57%)
:arrow_up:

src/Components/ActivationKeys/ActivationKeys.js
92.30% <100.00%> (+1.39%)
:arrow_up:

src/hooks/useRbacPermissions.js
92.30% <100.00%> (+1.39%)
: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 0d29d4b...85fea11. Read the comment docs.

LGTM


added OnOffLight

added wiser OnOffDevice

I think we need to adjust the api more than this as otherwise it will provide properties for on/off lights that don't exist.  Let me have a look when I get a chance.

This is now in v0.0.26


Redirection problem

Hi, i am having 2 issues with the multi-auth.
I have a user and admin login (2 separate login pages).

When i am not logged as an admin and i want to access a page (example : admin/test123)
It redirect me to the user login and i am not sure where to look to correct that. I don't want my admins to fall into the user platform.

When i login as an admin, it sometimes redirect me to the user login too. The login is working fine because if i manually do /admin, i can see the admin index. It only happen the first time i try like something was saved into a cache. I don't know.

Thanks!

Hi @hugo411,
My guess here without seeing code is that the routes for the various pages are protected by middleware / guards and therefore require a user to be logged in as either an admin or user.
Did you solve this?

 In my controller i have a construct method that require the method to be guard('admin')
  public function __construct()     {         $this->middleware('admin');     }
After removing that, it works but i am not sure if it is secure.
Thanks.

@hugo411 You'll most likely need to protect the routes, but before we dive into that, are there any methods in your controller that should be accessible by the other type of user? If so, simply adding the $this->middleware() to the constructor will block everything from being visible by the other type of user.
It might be that you need to protect each method individually, and in which case you might find it easier to do that from the routes file itself, rather than the controllers.
Does that help? Let me know if you need any help.

All the methods in that controller are for admin only so i had
public function __construct() { $this->middleware('admin'); }
But when i have this line, the admin login redirect me to user login.
Thanks for all the help.

hello there, my take for no.1 you might want to check the logic inside the admin middleware specifically if you have something like RedirectIfNotAdmin.php
for no.2 is it could be that the $redirectTo set in the admin login controller is pointing to a protected view by the middleware('auth'),
good luck!


Reimplement in1d

New implementation is based on numpy code. This is much much faster and less less memory hungry. Call to broadcast_to is avoided now. Additionally assume_unique keyword is handled.

Very interesting! Could you share insights on why the NumPy approach works better (which is not obvious to me), and also post performance benchmarks that you did before and with this PR?

Numpy uses approach with sorting input arrays, whereas cupy tries direct comparison by broadcasting inputs. Broadcasting is generally ok, it creates strided views on inputs, but the next comparison line tries to allocate MxN boolean array, where M, N are lengths of inputs. This is a recipe for out of memory error:
In [11]: import cupy

In [12]: a = cupy.random.randint(0, 1000000, size=500000)

In [13]: b = cupy.random.randint(0, 1000000, size=100000)

In [14]: cupy.in1d(a, b)
---------------------------------------------------------------------------
OutOfMemoryError                          Traceback (most recent call last)
<ipython-input-14-e28ce4babb52> in <module>
----> 1 cupy.in1d(a, b)

~/miniconda3/lib/python3.8/site-packages/cupy/logic/truth.py in in1d(ar1, ar2, assume_unique, invert)
     92     ar1_broadcast = cupy.broadcast_to(ar1[..., cupy.newaxis], shape)
     93     ar2_broadcast = cupy.broadcast_to(ar2, shape)
---> 94     count = (ar1_broadcast == ar2_broadcast).sum(axis=1)
     95     if invert:
     96         return count == 0

cupy/core/core.pyx in cupy.core.core.ndarray.__richcmp__()

cupy/core/_kernel.pyx in cupy.core._kernel.ufunc.__call__()

cupy/core/_kernel.pyx in cupy.core._kernel._get_out_args()

cupy/core/core.pyx in cupy.core.core.ndarray.__init__()

cupy/cuda/memory.pyx in cupy.cuda.memory.alloc()

cupy/cuda/memory.pyx in cupy.cuda.memory.MemoryPool.malloc()

cupy/cuda/memory.pyx in cupy.cuda.memory.MemoryPool.malloc()

cupy/cuda/memory.pyx in cupy.cuda.memory.SingleDeviceMemoryPool.malloc()

cupy/cuda/memory.pyx in cupy.cuda.memory.SingleDeviceMemoryPool._malloc()

cupy/cuda/memory.pyx in cupy.cuda.memory._try_malloc()

OutOfMemoryError: Out of memory allocating 50,000,000,000 bytes (allocated so far: 18,521,088 bytes).

Nothing like this happens with numpy's implementation, also for many times bigger inputs. In terms of speed current in1d is poor for larger inputs:
In [15]: a = cupy.random.randint(0, 100000, size=50000)

In [16]: b = cupy.random.randint(0, 100000, size=50000)

In [17]: %timeit cupy.in1d(a, b)
45.7 ms ± 37.6 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

In [18]: %timeit in1d(a, b)  # Numpy-like implementation
1.78 ms ± 72.4 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

For smaller arrays cupy.in1d is faster:
In [19]: b = cupy.random.randint(0, 100000, size=5000)

In [20]: a = cupy.random.randint(0, 100000, size=5000)

In [21]: timeit cupy.in1d(a, b)
490 µs ± 112 ns per loop (mean ± std. dev. of 7 runs, 1000 loops each)

In [22]: %timeit in1d(a, b)  # Numpy-like implementation
1.71 ms ± 59.5 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

So maybe for small inputs, here some heuristics have to be used, the current approach should still be used?

@mrkwjc Thank you for the PR!
The memory usage explosion in ar1_broadcast == ar2_broadcast seems to be avoidable by using cupy.ReductionKernel. I will compare the performance numbers between this PR and ReductionKernel version.

I confirmed that this PR improves GPU performance, but causes host/device synchronization.
master    :    CPU:   31.325 us   +/-37.739 (min:   27.504 / max:  662.737) us     GPU-0:16218.380 us   +/-52.691 (min:16190.464 / max:16840.704) us
this PR   :    CPU: 1228.504 us   +/-82.605 (min: 1207.951 / max: 5493.522) us     GPU-0: 1366.806 us   +/-80.935 (min: 1344.512 / max: 5547.008) us

Jenkins, test this please.

Jenkins CI test (for commit ae9705dcc8b59ed05a6c91fdfb401b71f7c2b224, target branch master) succeeded!

LGTM. Thank you for the PR!