Unit Tests for Models

PHPUnit tests should be written to cover the models of the application.
At the most basic level, these tests should cover the relationship between the models.
Example:
A post can have many tags.
A tag belongs to a post.
A user can have many posts.
The tests can be further fleshed out to protect the application from a pretty granular level.

I wrote a package that attempted to create a test case specifically for this purpose. In Rails, I would say it is a lot more common to test models, relationships, and their corresponding database table. From your description, I think it would take care if this and I wouldn't mind some feedback on it/would like to see if it's something that could be used easily by others (https://github.com/erikgall/eloquent-phpunit). I can create PR that pull's it in and uses it if you would like.

Awesome @erikgall. Let me look into it a little bit and I'll get back to you!

@erikgall Really like the package that you wrote there, it's exactly the kinda thing I was envisioning. If you wanted to create a PR that brings this into Canvas, that'd be fantastic as it'll monitor a deeper level of code that hasn't been covered yet.

@austintoddj Awesome. I'll pull that in and create a PR once I'm done. Thanks for taking a look at it.


Wrong indentation for passed scriptblocks in pipe sequence (PSUseConsistentIndentation)

Steps to reproduce
Get-ChildItem *.yml |
    Get-Content -Raw |
    ConvertFrom-KubeYaml |
    ForEach-Object {
        switch ($_.Metadata.Name) {
            # ...
        }
    } |
	Update-KubeResource

Expected behavior
Should not cause a rule violation, this is the expected formatting
Actual behavior
Formats to:
Get-ChildItem *.yml |
    Get-Content -Raw |
    ConvertFrom-KubeYaml |
    ForEach-Object {
    switch ($_.Metadata.Name) {
        # ...
    }
} |
    Update-KubeResource

Note how the switch is not indented and the closing brace of the scriptblock is on the wrong level, while the next pipeline element is correct.
Environment data

> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.1.0-rc.1
PSEdition                      Core
GitCommitId                    6.1.0-rc.1
OS                             Darwin 17.7.0 Darwin Kernel Version 17.7.0: Thu Jun 21 22:53:14 PDT 2018; root:xnu-4570.71.2~1/RELEASE_X86_64
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
> (Get-Module -ListAvailable PSScriptAnalyzer).Version | ForEach-Object { $_.ToString() }
1.17.1

Thanks for raising it. This has annoyed me sometimes as well. I will start to look into this so that we hopefully have a fix in the next version of PSSA

I had a look at this and it seems that the way the rule was written is that it currently takes only opening/closing parenthesis and braces into account to determine the expected indentation. A slightly more simplified example for this issue is:
Get-ChildItem |
    ForEach-Object {
        $_.ToString()
}

I am thinking of enhancing the rule and therefore fixing this issue by adding 1 level of expected indentation when the pipe is at the end of a newline. If it turns out to be successful, there will be a PR in the next days.


Feature request: linking to terraform documentation

This one is a feature request. It would be nice to be able to press shift+F1 on a resource (and maybe even data, backend, provider, etc) and have it take you to straight to the Terraform docs. This feature is called External Documentation in the keymap. It's supported pretty well for Java and Python methods.
Example:
resource "aws_instance" "my-machine" {

Putting the cursor on aws_instance and hitting shift+F1 would take you here.
Another nice-to-have would be ctrl+q, or Quick Documentation. This pops up a window right in the IDE.
I'm not sure whether Terraform has enough metadata about the resources to make this possible. Keeping a manual list of URLs would not be fun. Either way, this is on my wishlist :)

This would be so helpful! Terraform docs are difficult to navigate and locate (especially for AWS, where I have to scroll for days to find anything), so being able to jump right to the documentation would save a lot of time

VSCode has this feature:

However they don't support all providers like this plugin does. They only support the "big three" (AWS/Azure/GCloud). I did a tiny bit of research on their implementation. Here is their data source. It was generated with this script (written in Node/TypeScript).
I wonder if the links couldn't be generated with just dumb string concateneation. If you look at the URL format, it seems very standard:
https://www.terraform.io/docs/providers/aws/r/cloudfront_distribution.html
Maybe just assume https://www.terraform.io/docs/providers/$provider/$type/$name.html?

This patch seems to be very helpful https://github.com/VladRassokhin/intellij-hcl/pull/221#issuecomment-500211731 but not yet merged into https://github.com/VladRassokhin/intellij-hcl/blob/master/src/kotlin/org/intellij/plugins/hcl/terraform/config/TerraformDocumentationProvider.kt

@saiya in one of recent updates I've added next faeture:

'Quick Documentation' is now able to show link to terraform.io.

@saiya in one of recent updates I've added next feature:

'Quick Documentation' is now able to show link to terraform.io.

@VladRassokhin I'm using version 0.7.9 and cannot seem to get quick documentation to work.
With the following in place I assume I should be able to invoke "Quick Documentation" on aws_lambda_function, but nothing happens:
resource "aws_lambda_function" foo {
}

Any idea what might be wrong?

If you put caret/cursor on foo it would work 100%, on other places - may not, yet nothing could be done as foo is identifier of a element (resource) and IntelliJ shows documentation for identifiers.
Although if you reference element somewhere, e.g. output x {value=aws_lambda_function.foo<caret>}, quick documentation should work.

If you put caret/cursor on foo it would work 100%
Wow, can't believe I missed that. Thanks!

If you put caret/cursor on foo it would work 100%, on other places - may not, yet nothing could be done as foo is identifier of a element (resource) and IntelliJ shows documentation for identifiers.

Here for a "google_service_account" resource it does not display a quick documentation text and the link displayed when pressing ctrl+q points to https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/service_account while the documentation is actually at https://registry.terraform.io/providers/hashicorp/google/latest/docs/resources/google_service_account. Also it uses "latest" as the version, which is not correct in my case: .terraform.lock.hcl does not point to the "latest" version as of today.

This stopped working for me. I am using IntelliJ IDEA 2021.3.1  and the current version of the plugin (0.7.14). Anybody else experiencing this?
Instead of opening the documentation in the browser an empty quick documentation popup opens.

@carlduevel I have the same problem recently. Opened an explicit issue in https://github.com/VladRassokhin/intellij-hcl/issues/372 so that this doesn't get lost.

I just installed version 0.7.14 and it works, BUT it appends .html to the webpage which breaks the link as the TF Registry docs don't support the .html


[SPARK-48157][SQL] Add collation support for CSV expressions

What changes were proposed in this pull request?
Introduce collation awareness for CSV expressions: from_csv, schema_of_csv, to_csv.
Why are the changes needed?
Add collation support for CSV expressions in Spark.
Does this PR introduce any user-facing change?
Yes, users should now be able to use collated strings within arguments for CSV functions: from_csv, schema_of_csv, to_csv.
How was this patch tested?
E2e sql tests.
Was this patch authored or co-authored using generative AI tooling?
No.

thanks, merging to master!


Fix comment in test

Found this while making sure that my parser could handle every .opt file under souper/test/**.
@regehr, please take a look, thanks!

LGTM, thanks Nick!


Use twox64_concat to map address to balance

https://github.com/apopiak/pallet-stablecoin/blob/e4074e6753fdc9257922826a7a1218735d25e030/src/lib.rs#L191
See https://github.com/paritytech/substrate/issues/4917

Thanks for the info! Users can transfer stablecoins to any valid AccountId, though, so I wonder whether blake2_128_concat might be the better option?

Yes you are right!

addressed in 431608a59afd9cfcca1f2684efe1d1b13fcf438d


Payload size validates multiple queries and fallback queries

As far as I understand we will store almost all the body in most of the cases, except a small part of it: the root {query: ''}. So to keep it simple I decided to check against the full body instead of just what is in the query argument.
This will also work with fallback queries.
It uses a middleware to check the body size.
cc @dgaubert

Oh! middlewares... I love them 😍 😍 😍
👍


research cargo build steps to replace precommit.py

Is it possible to update the README.md file on every build using cargo? That'd make sure it was always up to date.

Could use cog:
https://til.simonwillison.net/python/cog-to-update-help-in-readme
or Pollen:
https://docs.racket-lang.org/pollen/
Both require extra tools though. Would be nice to have something that worked with minimal additional dependencies.


Add legend for debugging view of graph

Short description
Details
Subtasks

[ ] ...

Dependencies
This task depends on ... to be finished

Hello, we tried to solve the issue.
This is what we did:
Modified the task.md file in the .github/ISSUE_TEMPLATE directory to include the specified sections: Short description, Details, Subtasks, and Dependencies. The changes maintain the existing structure and styling while incorporating the requested modifications.
You can review changes in this commit: https://github.com/mnmnk43434/precice-forschungsprojekt-config-graph-7/commit/86810bdc195f1c699744406e06603eda38a07871.

[!CAUTION]
Disclaimer: The concept of solution was created by AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


Typo corrected - Overview.md

Before submitting a pull request, please make sure the following is done...

Fork the repo and create your branch from master.
If you've added code that should be tested, add tests!
If you've changed APIs, update the documentation.
Ensure that:

The test suite passes (npm test)
Your code lints (npm run lint) and passes Flow (npm run flow)
You have followed the testing guidelines

If you haven't already, complete the CLA.

Please use the simple form below as a guideline for describing your pull request.
Thanks for contributing to Draft.js!

Summary
[...]
Test Plan
[...]

Thanks for this fix @joerlop and welcome to Draft.js!


Real Pause

At the moment, the first image is displayed when paused (or so it seems to me). Would it be possible to show the current frame?

Hi @skerbis, showing the current frame is a lot more complex (which is out of scope for this library). The closest library to achieve that would be this older library created by Buzzfeed called libgif.js

thx


porting of pileups infrastructure.

Pileup is going to be needed for variant annotations. Ported over the new read interface.
@droazen please review

Back to @akiezun for a refactoring before I review

First-pass review complete. Back to @akiezun

I cleaned up the class a bit - removed all specialized filtering methods in favor of generic predicate-based access. If we find repeated uses in client code, we can move those here.
Regarding tests, can you list the conditions that you think are inadequately tested?

The problem with the ReadPileupUnitTest class was that most of the specialized filtering methods were only exercised in trivial ways (eg., assert that the return value is 0 or null). You have mostly resolved this by just deleting the majority of the specialized filtering methods, but it is still a problem with methods like getNumberOfElements().

Second-pass review complete -- back to @akiezun

addressed comments + added some tests for pileups with indels.
back to @droazen

:+1: Rebase/merge at will.


Make link to documentation in params

hi,
first, fantastic job. KB is great!
would it be possible to have all links to kb.net/documentation in a config param to be able to host the proprer documentation ?
thx

I already plan to have the documentation embedded into Kanboard itself.

The doc is now part of Kanboard.

I know this is closed but in 1.0.18 there is only al link to the doc in the admin settings. There is no prominent link on the dashboard. That would make it easier for normal users to find the documentation.
Maybe you can add a small question mark on the top right?


Himox H06 detected as "simple_switch"

I have a Himox H06 and it's only detected as a simple_switch - anything I can do about this?
(I also have a H05 that is detected correctly in localtuya - does it make any sense to try to set it up in tuya-local?)
I've removed the integration from localtuya before setting it up in tuya-local.

Additional information - my H06 payload looks like this:
    "1": True,
    "4": "high",
    "5": 92,
    "8": True,
    "11": False,
    "18": "cancel",
    "19": 0,
    "22": "severe",
    "101": 2,

Himox H06 support was added to the source recently. It is not yet in a release.  To try it if using HACS, you need to select "Redownload" from the tuya-local menu inside the HACS Integrations screen, and select "main" as the version to install.
Himox H13 is not yet supported.  As long as you do not have the same device configured in both, localtuya and tuya-local can co-exist side by side.

I blithely assumed its presence in the documentation meant it was already released. Apologies and thanks for the help!

The documentation you see on github is taken from main.  I have not made a release for some time because there are major changes in the next release (vacuum and RGBW lighting support) which still need some debugging. And also I have not had time for the past month due to other life events.

Understood. If there's anything I can do to help add support for the H13 - if you're interested - please let me know. No hurry, of course.

It seems to me like the dps that match have the same meaning.  dp 2 is new, and looks like it is in the right range to be a current temperature sensor.  dp 21 looks like it may have just moved from dp 22 (air quality indicator).  dp 7 is a new switch or indication, or maybe it has also simply moved from dp 8.   I guess figuring out those three (and whether my speculation about them having simply moved is correct) is all that is needed.

I went back and checked and it's the H05, not H13 - I've updated the comments.
The H05 does not have a light on/off switch but it does have a child lock.
The other buttons are a timer (1/2/4/8h), on/off, reset filter, and speed (night/low/high/auto).

Reopening for the H05

Support for H05 is in 0.16.0

Tried this out and it connects and turns on and off, but I think it's not 100% right:
Temperature displays at 108°C
Timer offers 1/2/4/8 hours on the device, but only 4 and 8 in the config.
Thanks for adding it!

The timer values were copied straight from H06, which does apparently only support 4 and 8 hours.  As for the temperature, it is strange, as there are no transformations applied to it, and the value you first posted was 21, which looks like a reasonable temperature.  Unless the original was when it was 4 degrees and there is actually a 5x multiplier on that...

No, it was approximately the same temperature then. But it's also not changed by even 0.1° since I commented:

Are you sure the entity is correctly configured in the UI?  As there is no transformation on the temperature value, I cannot see how this could go wrong like this.

Closing due to no feedback


Corporate Dashboard Getting Started

Getting started guide for the corporate-dashboard project.
Future

[ ] Centralize SST setup docs

@ecxyzzy Is it possible to get deploy previews on these PRs?


Cannot build Docker container using provided Dockerfile

Hey there,
when I run the Dockerfile using
docker build -t env4scripter .
as explained in the documentation I get the following error.
 > [10/10] RUN pip3 install pyzmq:
0.580 error: externally-managed-environment
0.580
0.580 × This environment is externally managed
0.580 ╰─> To install Python packages system-wide, try apt install
0.580     python3-xyz, where xyz is the package you are trying to
0.580     install.
0.580
0.580     If you wish to install a non-Debian-packaged Python package,
0.580     create a virtual environment using python3 -m venv path/to/venv.
0.580     Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
0.580     sure you have python3-full installed.
0.580
0.580     If you wish to install a non-Debian packaged Python application,
0.580     it may be easiest to use pipx install xyz, which will manage a
0.580     virtual environment for you. Make sure you have pipx installed.
0.580
0.580     See /usr/share/doc/python3.12/README.venv for more information.
0.580
0.580 note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
0.580 hint: See PEP 668 for the detailed specification.
------
Dockerfile:14
--------------------
  12 |     COPY ailice/modules/AScrollablePage.py /scripter/ailice/modules/AScrollablePage.py
  13 |
  14 | >>> RUN pip3 install pyzmq
  15 |
  16 |     EXPOSE 59000-59200
--------------------
ERROR: failed to solve: process "/bin/sh -c pip3 install pyzmq" did not complete successfully: exit code: 1

I managed to bypass this issue by creating a virtual environment.
However, I encountered several issues even after creating a working Docker container.
First of all, several Python packages are missing now, like numpy.
Furthermore, after installing those missing libraries, access to the AIlice modules is not working either and needs to be fixed.
So far, I have not found a solution to this issue.
Thus, I'm not able to use the dockerized method right now.

I still can't reproduce your issue. You can give this cmd a try
docker build --no-cache -t env4scripter .

Still running into the same issue with docker build --no-cache -t env4scripter . which also gives me the error.

[+] Building 0.0s (0/0)  docker:default
2024/05/06 10:02:09 http2: server: error reading preface from client //./pipe/docker_engine: file has already been close[+] Building 56.6s (14/14) FINISHED                                                                      docker:default
 => [internal] load build definition from Dockerfile                                                               0.0s
 => => transferring dockerfile: 712B                                                                               0.0s
 => [internal] load metadata for docker.io/library/ubuntu:latest                                                   0.5s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => CACHED [ 1/10] FROM docker.io/library/ubuntu:latest@sha256:3f85b7caad41a95462cf5b787d8a04604c8262cdcdf9a472b8  0.0s
 => [internal] load build context                                                                                0.0s
 => => transferring context: 360B                                                                                0.0s
 => [ 2/10] RUN apt-get update && apt-get install -y python3 python3-pip                                          55.4s
 => [ 3/10] WORKDIR /scripter                                                                                0.0s
 => [ 4/10] COPY ailice/__init__.py /scripter/ailice/__init__.py                                                   0.0s
 => [ 5/10] COPY ailice/common/__init__.py /scripter/ailice/common/__init__.py                                     0.0s
 => [ 6/10] COPY ailice/common/lightRPC.py /scripter/ailice/common/lightRPC.py                                     0.0s
 => [ 7/10] COPY ailice/modules/__init__.py /scripter/ailice/modules/__init__.py                                   0.0s
 => [ 8/10] COPY ailice/modules/AScripter.py /scripter/ailice/modules/AScripter.py                                 0.0s
 => [ 9/10] COPY ailice/modules/AScrollablePage.py /scripter/ailice/modules/AScrollablePage.py                     0.0s
 => ERROR [10/10] RUN pip3 install pyzmq                                                                           0.4s
------
 > [10/10] RUN pip3 install pyzmq:
0.374 error: externally-managed-environment
0.374
0.374 × This environment is externally managed
0.374 ╰─> To install Python packages system-wide, try apt install
0.374     python3-xyz, where xyz is the package you are trying to
0.374     install.
0.374
0.374     If you wish to install a non-Debian-packaged Python package,
0.374     create a virtual environment using python3 -m venv path/to/venv.
0.374     Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
0.374     sure you have python3-full installed.
0.374
0.374     If you wish to install a non-Debian packaged Python application,
0.374     it may be easiest to use pipx install xyz, which will manage a
0.374     virtual environment for you. Make sure you have pipx installed.
0.374
0.374     See /usr/share/doc/python3.12/README.venv for more information.
0.374
0.374 note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
0.374 hint: See PEP 668 for the detailed specification.
------
Dockerfile:14
--------------------
  12 |     COPY ailice/modules/AScrollablePage.py /scripter/ailice/modules/AScrollablePage.py
  13 |
  14 | >>> RUN pip3 install pyzmq
  15 |
  16 |     EXPOSE 59000-59200
--------------------
ERROR: failed to solve: process "/bin/sh -c pip3 install pyzmq" did not complete successfully: exit code: 1

I'm using Docker Desktop 4.29.0 (145265) on Windows 11. If you need any further information, please let me know.

Due to some network issues in my area, I've been unable to establish a suitable working environment for addressing your question. However, the cause of this problem should be clear, and you can refer to this post:
https://stackoverflow.com/questions/77028925/docker-compose-fails-error-externally-managed-environment
If you want a short answer, you might consider changing 'RUN pip3 install pyzmq' in the Dockerfile to 'RUN pip3 install pyzmq --break-system-packages' and see if it works.
Please let me know the result. If the problem is solved, I will consider updating the Dockerfile.

No worries.
I used RUN pip3 install pyzmq --break-system-packages as you suggested.
With the additional statement, the Docker container was created successfully.

Then I followed your descriptions in the documentation
However, when running the container I still get the numpy is missing error message:
2024-05-10 08:15:10 Traceback (most recent call last):
2024-05-10 08:15:10   File "<frozen runpy>", line 198, in _run_module_as_main
2024-05-10 08:15:10   File "<frozen runpy>", line 88, in _run_code
2024-05-10 08:15:10   File "/scripter/ailice/modules/AScripter.py", line 8, in <module>
2024-05-10 08:15:10     from ailice.common.lightRPC import makeServer
2024-05-10 08:15:10   File "/scripter/ailice/common/lightRPC.py", line 11, in <module>
2024-05-10 08:15:10     import numpy
2024-05-10 08:15:10 ModuleNotFoundError: No module named 'numpy'

Dockerfile extension:
RUN pip3 install numpy --break-system-packages
Adding this install to the Dockerfile works with no issues.
Leading to the final error:
2024-05-10 08:21:01 Traceback (most recent call last):
2024-05-10 08:21:01   File "<frozen runpy>", line 198, in _run_module_as_main
2024-05-10 08:21:01   File "<frozen runpy>", line 88, in _run_code
2024-05-10 08:21:01   File "/scripter/ailice/modules/AScripter.py", line 8, in <module>
2024-05-10 08:21:01     from ailice.common.lightRPC import makeServer
2024-05-10 08:21:01   File "/scripter/ailice/common/lightRPC.py", line 17, in <module>
2024-05-10 08:21:01     from ailice.common.ADataType import *
2024-05-10 08:21:01 ModuleNotFoundError: No module named 'ailice.common.ADataType'

Ah, I forgot to update some new dependencies and files in the Dockerfile. Please try with the latest code from the master branch, it should work now.

Yes, now the Docker container is running 😄
I think updating the documentation and explaining the Docker usage in more depth would be beneficial.


[WFLY-8418] Enhance the way licenses are presented

issue: https://issues.jboss.org/browse/WFLY-8418

This may need to be reverted. It creates a problem when the prepare-package goal, e.g. package, install, etc., is executed. It will update the license.xml file and create a new untracked license.xml file. We can't do this as it will mess up the commit the developer is working on.
This is from a mvn clean install -DskipTests on master with no other changes.
On branch master
Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

	modified:   feature-pack/src/main/resources/content/docs/licenses/licenses.xml

Untracked files:
  (use "git add <file>..." to include in what will be committed)

	feature-pack/src/main/resources/content/docs/licenses/licenses.html

no changes added to commit (use "git add" and/or "git commit -a")

The general rule is, don't modify things under src/ during build.  There is a generated-sources directory which might be appropriate (or might not be).

This won't need to be reverted as https://github.com/wildfly/wildfly/pull/10148 will fix it.

@dmlloyd I'm not happy to find out I did it wrong, but there was reason for doing it this way.  There is now plenty of time to solve with the way that would satisfy all requirements.


Handle subscription cancellations and error handling

This PR adds:

checking relevant StatusResponses for subscription datareports that should lead to a cancellation according to specs
still catch errors from subscription update sending to prevent crashing with SmartLife app because this happens when Andoid devcie goes to sleep/offline and would crash our device right now

Unfortunately no test but tested locally with iOS and google.

"proof":
2023-04-03 00:55:27.213 DEBUG SubscriptionHandler Sending subscription update message for ID<PHONE_NUMBER> with 0 values
2023-04-03 00:55:27.213 DEBUG MessageExchange new MessageExchange 1 56156 300 5000 5
2023-04-03 00:55:27.213 DEBUG SubscriptionHandler Sending subscription changes for ID<PHONE_NUMBER>: []
2023-04-03 00:55:27.213 DEBUG ExchangeManager sending id:44380/56156/1346505783 t:1/5 reqAck:true payload: 1526005c5bcfa5360118290424ff0118
2023-04-03 00:55:27.226 DEBUG MessageExchange onMessageReceived 1 id:20397/56156/94391172 t:1/1 acked:1346505783 reqAck:true payload: 1524007d24ff0118
2023-04-03 00:55:27.227 INFO SubscriptionHandler Subscription<PHONE_NUMBER> cancelled by peer.

tested with puttion my computer to sleep and wake up long time afterwards then next subscription message is declined by iOS (in this case) with error 7d (Invalid subscription) ... so we cancel in this case sending too


AMIs went missing  in AWS region eu-west-1 for xenial stemcells 621.5 ,170.3,

Hi all,
Currently We are having issues with our AMIs present in AWS region eu-west-1 for xenial stemcells 621.5 ,170.3,
Suddenly the AWS AMI went missing form our AWS account and when we try to upload the stemcell for that it says it already exists.
But when we use --fix option to fix this it gives below error :
L Error: CPI error 'Bosh::Clouds::CloudError' with message 'Stemcell does not contain an AMI in region eu-west-1' in 'create_stemcell' CPI method (CPI request ID: 'cpi-433859')
Please let us know what is the issue, why the AMI went missing and what could be the cause?
Also we are able to upload any new stemcell and find its AMI on AWS like for version 621.123.
Please let us know how can we resolve this issue.
We raised it with AWS on why the AMIs are missing bt we received the response as we are not owner of this AMIs and to contact the owner of these AMIs

That's the exact same stemcells that were already handled with the slack thread

So is there no answer to this issue? I am blocked by this very same problem.

So is there no answer to this issue? I am blocked by this very same problem.

Did you try to upload a full stemcell instead? Heavy stemcells do not depend on AMIs that are published on AWS.
I’ve checked that the link is still downloadable: https://bosh.io/stemcells/bosh-aws-xen-hvm-ubuntu-xenial-go_agent#v621.125


How to convert ROI coordinates from relative to pixels

nd2 version: 5.42.06
Python version: 3.9
Operating System: W10

Description
This is not really an issue, just a suggestion to add to the documentation
What I Did
The code below converts the ROI coordinates from relative fractions to pixel coordinates.
The Ellipse ROI width and height are before rotation over the indicated angle.
The only missing detail is the inner circle of the Ring ROI.
f = nd2.ND2File(sys.argv[1])

mid_x = f.shape[1] / 2.
mid_y = f.shape[0] / 2.

def RelToPixSizeX(x):
	return mid_x * x

def RelToPixSizeY(y):
	return mid_y * y

def RelToPixPosX(x):
	return mid_x + mid_x * x

def RelToPixPosY(y):
	return mid_y + mid_y * y

for id in f.rois:
	roi = f.rois[id]
	header = str(roi.id) + ";" + nd2.structures.RoiShapeType(roi.info.shapeType).name + ";"
	pnts = ""
	info = roi.animParams[0]
	if (roi.info.shapeType == nd2.structures.RoiShapeType.Rectangle) or \
			(roi.info.shapeType == nd2.structures.RoiShapeType.Square) or \
			(roi.info.shapeType == nd2.structures.RoiShapeType.Circle) or \
			(roi.info.shapeType == nd2.structures.RoiShapeType.Ellipse) or \
			(roi.info.shapeType == nd2.structures.RoiShapeType.Ring):
		pnts = "(" + \
			"{:#.1f}".format(RelToPixPosX(info.centerX)) + "," + \
			"{:#.1f}".format(RelToPixPosY(info.centerY)) + "),(" + \
			"{:#.1f}".format(RelToPixSizeX(info.boxShape.sizeX)) + "," + \
			"{:#.1f}".format(RelToPixSizeY(info.boxShape.sizeY)) + ")"
	elif (roi.info.shapeType == nd2.structures.RoiShapeType.Line) or \
			(roi.info.shapeType == nd2.structures.RoiShapeType.Polygon) or \
			(roi.info.shapeType == nd2.structures.RoiShapeType.PolyLine) or \
			(roi.info.shapeType == nd2.structures.RoiShapeType.Bezier):
		for pnt in roi.animParams[0].extrudedShape.basePoints:
			if len(pnts) != 0:
				pnts += ","
			pnts += "(" + "{:#.1f}".format(RelToPixPosX(pnt.x+info.centerX)) + "," + "{:#.1f}".format(RelToPixPosY(pnt.y+info.centerY)) + ")"
	print(header + pnts + ";" + "{:#.4f}".format(info.rotationZ))

f.close()

WR,
Kees

hey @Kees-van-der-Oord-Nikon
Thanks for your note.  I agree, it's pretty cumbersome to use ROIs straight from the file metadata :joy:
This is great and i'd be happy to add a convenience like this to the code base.  Would you be interested in opening a PR to that effect?  (Happy to do it myself if not)

Dear Talley,
Thanks for your response.
I think you are more fluent in github than me, so please do the PR.
The biggest challenge was that f.shape array has the dimensions in an unexpected order (height,width).
I think the interface should support moving ROIs as well.
I expect that the animsParam array has more entries in that case (one for every control frame ?)
WR,
Kees
Dr. Kees van der Oord
European Support Specialist
T: +31207099246
M: +31623367589
E: @.***
Website
Nikon Europe B.V.
Stroombaan 14
1181 VX Amstelveen
Netherlands
Registered Chamber of Commerce Amsterdam No. 34139593
VAT No. NL 8091.99.889.B.01
This e-mail may contain information that is PRIVILEGED and/or CONFIDENTIAL. This e-mail is intended solely for the individual or entity to whom it is addressed. If you are not, or have reason to believe you are not the intended recipient of this e-mail please contact us by reply e-mail as soon as possible. Immediately after contacting us, please destroy this e-mail (including attachment(s)) from your computer system(s). Any unauthorized copying, disclosure or distribution of this e-mail (including attachment(s)) is not allowed. Nikon Europe BV hereby disclaims any and all liabilities which may arise in relation to any of its electronic communication. No obligation is entered into by virtue of this e-mail, unless confirmed in writing and duly signed by an authorized representative of Nikon Europe BV
From: Talley Lambert @.>
Sent: 20 June, 2024 16:29
To: tlambert03/nd2 @.>
Cc: Oord, Kees van der @.>; Mention @.>
Subject: Re: [tlambert03/nd2] How to convert ROI coordinates from relative to pixels (Issue #232)
hey @Kees-van-der-Oord-Nikonhttps://github.com/Kees-van-der-Oord-Nikon
Thanks for your note. I agree, it's pretty cumbersome to use ROIs straight from the file metadata 😂
This is great and i'd be happy to add a convenience like this to the code base. Would you be interested in opening a PR to that effect? (Happy to do it myself if not)
—
Reply to this email directly, view it on GitHubhttps://github.com/tlambert03/nd2/issues/232#issuecomment-2180853580, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXSSBOSB2W2GBE4FDVE7EPTZILRJZAVCNFSM6AAAAABJUA6XNOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBQHA2TGNJYGA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

please do the PR.

no worries, will do

The biggest challenge was that f.shape array has the dimensions in an unexpected order (height,width).

yes (height, width) (row major) is the standard convention for arrays in python, numpy, etc.., so nd2 sticks with this convention.  it's true that needs to be taken into consideration when comparing to the file metadata


error: no associated item named get_screen found for type gtk::WindowExt in the current scope

Just wondering if anyone has experienced the above when compiling on Windows, using
Nightly 1.16
error: no associated item named `get_screen` found for type `gtk::WindowExt` in
the current scope
    --> src\hymntoninkasi.rs:2086:17
     |
2086 |         let w = WindowExt::get_screen(&window).unwrap();
     |                 ^^^^^^^^^^^^^^^^^^^^^
     |
     = note: the method `get_screen` exists but the following trait bounds were
not satisfied: `gtk::WindowExt : std::marker::Sized`, `gtk::WindowExt : glib::Is
A<gtk::Widget>`, `gtk::WindowExt : glib::IsA<glib::Object>`, `&gtk::WindowExt :
glib::IsA<gtk::Widget>`, `&gtk::WindowExt : glib::IsA<glib::Object>`, `&mut gtk:
:WindowExt : glib::IsA<gtk::Widget>`, `&mut gtk::WindowExt : glib::IsA<glib::Obj
ect>`

error[E0038]: the trait `gtk::WindowExt` cannot be made into an object
    --> src\hymntoninkasi.rs:2086:17
     |
2086 |         let w = WindowExt::get_screen(&window).unwrap();
     |                 ^^^^^^^^^^^^^^^^^^^^^ the trait `gtk::WindowExt` cannot b
e made into an object
     |
     = note: method `add_mnemonic` has generic type parameters
     = note: method `remove_mnemonic` has generic type parameters
     = note: method `set_attached_to` has generic type parameters
     = note: method `set_default` has generic type parameters
     = note: method `set_focus` has generic type parameters
     = note: method `set_icon_from_file` has generic type parameters
     = note: method `set_titlebar` has generic type parameters
     = note: method `set_transient_for` has generic type parameters
     = note: method `connect_activate_default` has generic type parameters
     = note: method `connect_activate_focus` has generic type parameters
     = note: method `connect_enable_debugging` has generic type parameters
     = note: method `connect_keys_changed` has generic type parameters
     = note: method `connect_set_focus` has generic type parameters

error: aborting due to 2 previous errors

error: Could not compile `Hymn to Ninkasi`.

It was duplicate (by signature) with Widget's and removed. So you can use window.get_screen().unwrap();

Cheers, that works fine in windows, when I do that, however when I compile in Linux then, I get:
error[E0034]: multiple applicable items in scope
    --> src/hymntoninkasi.rs:2088:17
     |
2088 | 	let w = window.get_screen().unwrap();
     | 	               ^^^^^^^^^^ multiple `get_screen` found
     |
     = note: candidate #1 is defined in an impl of the trait `gtk::WidgetExt` for the type `_`
     = note: candidate #2 is defined in an impl of the trait `gtk::WindowExt` for the type `_`

(I'm writing a cross platform program)

May be on linux you use packages? Or just don't updated gtk?
If you want it work on both version use WidgetExt::get_screen

That did the trick! Thanks a lot.


Hi! Where we can see the minted nfts?

Is there any marketplace we can see the minted NFT's.

As far as I know there is no testnet/devnet marketplace to run some tests. But some wallets make you see your minted NFTs like martian wallet.


Data loss when the page was inactive for some time

Describe the bug
I left a HIT open on a fresh document then came back and annotated it.
My strategy is to assign all scores and then click all Submit buttons.
When I did this after that break, I lost all my annotations because the Submit buttons started complaining about an error, asking me to reload and use a relatively modern browser.
The data loss is extremely annoying and it should be clearly stated that people should never continue annotation without first reloading. Ideally, the page should go gray asking to reload after such a timeout.

Thank you for reporting this. I can think of three possible explanations:

A Django validation/security token may have expired (actually, I don't know if it can).
The server instance has been restarted in the meantime.
Someone else made annotations from the same account in the meantime.


Consider relationship callback in select option label/s

Thanks!

Hello. Just updated to this ver and select field with relationship is broken:
Before -> selected value displays correctly on view and edit pages just as should be
After -> selected value is actually the first value from table or if (in my case I'm using sorting by name) it shows me the first option from table and not the one I chosen
To confirm that, I reverted all changes by this PR in latest build


Jaccard

This PR adds the Jaccard similarity function at character level: jaccard('ab', 'aaabbbb') = 1.0.

Thanks, this looks good now.


java.lang.IllegalArgumentException: Image byte buffer must be allocated as 'direct'

Hi, I was testing the sample in my Nexus 5. When i press "Read Barcode" button i get following exception:
java.lang.IllegalArgumentException: Image byte buffer must be allocated as 'direct'. See ByteBuffer.allocateDirect().
at com.google.android.gms.vision.Frame$Builder.setImageData(Unknown Source)
at barcodereader.camera.CameraSource$FrameProcessingRunnable.run(CameraSource.java:930)
in this snippet:
outputFrame = new Frame.Builder()
.setImageData(mPendingFrameData, mPreviewSize.getWidth(),
mPreviewSize.getHeight(), ImageFormat.NV21)
.setId(mPendingFrameId)
.setTimestampMillis(mPendingTimeMillis)
.setRotation(mRotation)
.build();
Please, help me resolve this problem!

I've solved this problem:
setImageData(ByteBuffer.allocateDirect(mPendingFrameData.capacity()), mPreviewSize.getWidth(),
mPreviewSize.getHeight(), ImageFormat.NV21);
I think it's correct solution.

That demo requires compiling with Google Play Services 8.1.  I'm guessing that you compiled with Google Play Services 7.8.
This was the result of a bug fix that occurred between the two versions.  The issue is that using direct byte buffers with the CameraSource class will result in a pixel shift of the images sent to the detector (e.g., the position reported for the barcode may be shifted to the left several pixels).  In 8.1 we added support for indirect byte buffers to address this, but the 7.8 library doesn't have that ability.
Long story short:  Compile with 8.1 if you can.  If you can't, you'll need to change to direct byte buffers as suggested above, and positions reported back will be off by several pixels.

Thanks alot for your fast and useful answer. It's helped me.

It would be helpful if the Barcode codelabs course that references this sample project were updated to reflect the correct version.  https://codelabs.developers.google.com/codelabs/bar-codes/#2


Edit pipeline name in Pipeline Settings popup

User should be able to edit the pipeline name in Pipeline Settings popup, even when is in the process of creating a new pipeline

Working as expected


Adding Harvard Program

We are in contact with partners at Harvard that wish to collect data using openPATH. This PR will add the config for their study. They only wish to support English as of right now, and want to include a custom survey. The file is in draft state for now, as we await clarification and submission of a custom form.

Sample of the intro screen in the emulator:

I can't get past accepting permissions, but I think I'm remembering that happening last time? If that's expected behavior, let me know, and I'll add a note in the documentation for testing!

I can't get past accepting permissions, but I think I'm remembering that happening last time? If that's expected behavior, let me know, and I'll add a note in the documentation for testing!

That is likely because the stage is set up to require one of the valid tokens and the autogenerated ones are not.

That is likely because the stage is set up to require one of the valid tokens and the autogenerated ones are not.

I'm not sure I understand what you mean. Should I try logging into the emulator again with a different token? I'm not getting an invalid opcode popup, which I have sometimes when I structure an opcode I make for testing differently than the config expects. The app just won't progress when I tap "accept" after turning the permissions green, and I see log statements about the consent being stored.

The app just won't progress when I tap "accept" after turning the permissions green, and I see log statements about the consent being stored.

It is because the server has not yet been created. But they can't create the server until the config is ready.
So let's go ahead with this config (after the rename) and then re-test once the server is ready


added date picker

Mentioned in https://github.com/charmbracelet/bubbles/pull/521

For people as curious and lazy as me
https://github.com/EthanEFung/bubble-datepicker


"Extras & Help" Pop-up Immediately Closes Itself

This link opens an informational popup about HRT, but the popup immediately closes without any indication as to why.
As far as I can tell this is affecting all movie pages, irrespective of being logged in / not logged in.

Thanks for alerting me!  I'm seeing this too.  I'll try to fix ASAP.

Fixed as of 3.0.8.  It can take up to an hour from now for the new version to finish publishing.  @JKelner, many thanks for your report!
https://github.com/jmattthew/HeirloomRottenTomatoes/commit/3ef65a762fd99e7e4182f38b766c80ad08abfb91


Umbrella Issue: refactor commands to split flags from options

Currently most or even all of the commands have an *Options structure holding all of the data needed to run a command and three main methods Complete (finishing *Options struct), Validate (validating *Options struct) and Run (actual code behind the command). The current problem is that these *Options struct is tightly coupled with flags, but we'd like to split these.
A perfect example is wait command.
This issue should serve as a synchronization point.
/cc @eddiezane @seans3

/triage accepted

/remove-help
/remote-good-first-issue
Holding off on this until we scope better and trial a few to confirm it's what we want.

@soltysh looking at log and get

@rikatz will take basic create

@KnVerey to look at drain or describe

@eddiezane will look at a simple config command

/assign
So I wont forget :)

/assign Will do create with @rikatz synchronously

/assign I'll try to do for apply command

/assign

i can do attach.

A simple summary:

[x] -  wait  ✅    a perfect example
[ ] -  [WIP]create  @lauchokyip & rikatz  https://github.com/kubernetes/kubernetes/pull/101736
[ ] -  logs and get @soltysh
[ ] -  apply @SaiHarshaK
[ ] -  config @eddiezane
[ ] -  drain/describe @KnVerey
[ ] -  attach @ihcsim

/remove-lifecycle stale

/remove-lifecycle stale
On Jan 9, 2022, 06:32 -0500, Kubernetes Triage Robot @.***>, wrote:

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.
This bot triages issues and PRs according to the following rules:
• After 90d of inactivity, lifecycle/stale is applied
• After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
• After 30d of inactivity since lifecycle/rotten was applied, the issue is closed
You can:
• Mark this issue or PR as fresh with /remove-lifecycle stale
• Mark this issue or PR as rotten with /lifecycle rotten
• Close this issue or PR with /close
• Offer to help out with Issue Triage
Please send feedback to sig-contributor-experience at kubernetes/community.
/lifecycle stale
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>

/lifecycle frozen
Considering this is a long term tracking issue :)

/assign
I'll refactor label :)

@KnVerey Have you started to refactor the describe command ?

@soltysh Based on the slack conversation, I'm picking up the describe command, thanks!
cc @KnVerey
/assign
Refactoring describe command

Hey! I was wondering if you guys needed more help on this issue. I'm happy to pick up a command and start working on it.

/assign
Talked to owner of issue. Grabbing drain to refactor.

I'll refactor create
/assign

@harshitasao , I think I finished create  a while ago (https://github.com/kubernetes/kubernetes/pull/101736),  just waiting for review, would you be able to pick other command to refactor instead?

/assign
I'll do diff

/unassign

/assign
I'll do diff

I can work on the refactoring of expose command.

@soltysh @eddiezane  I am working on refactoring of Run command. Currently id does not follow Complete, Validate, Run. Validate seems to be being handled within the Run function. Should i make that refactor also, to make Validate function and remove this functionality from the Run function?


JDK17 EA downloads ambiguous response from /v3/binary/latest/17/ea/linux/aarch64/jdk/hotspot/normal/eclipse

Hello,
This URL used to work just fine for downloading the latest early access builds. It returns a selection of some JDK builds. I cannot see it documented on https://api.adoptium.net/q/swagger-ui/#/Binary/getBinary though.
Reproducer
 curl -OJL 'https://api.adoptium.net/v3/binary/latest/17/ea/linux/x64/jdk/hotspot/normal/eclipse'

Expected
Tarball is downloaded.
Actual
The response is this text:
{"errorMessage":"Multiple binaries match request:
[OpenJDK17U-jdk_x64_linux_hotspot_2022-05-27-19-32.tar.gz,
OpenJDK17U-sbom_x64_linux_hotspot_2022-05-26-23-30.tar.gz]"}

Similar with curl -OJL 'https://api.adoptium.net/v3/binary/latest/17/ea/linux/aarch64/jdk/hotspot/normal/eclipse'.
JDK 11 seems to download just fine.
Thx for hints.

/cc @johnoliver
Might be related to a new sbom artefact masking as jdk image?

Has this been deployed yet (I still see the issue)? I'm guessing this only will work again after the next nightly pipeline run?


Task executer: Handle task artefacts

This has been split from #52
This ticket focuses on the Task Executer side. Ticket 52 is reserved for the Workflow Executer section.
Copied from #52
Workflow tasks use artefacts in two ways - as input, and as output. Artefacts can be files or directories, either the original payload that came from the Informatics Gateway or files that were generated by tasks in the workflow.
For more information about artefacts and how they're supposed to be used see the documentation: https://github.com/Project-MONAI/monai-deploy-workflow-manager/blob/develop/guidelines/mwm-workflow-spec.md#artifacts.
We need to support artefacts in the following ways:

[ ] The task executer will then generate temporary credentials (not in scope for this ticket, see #112 ) and provide these to the task plug-in
[x] The task executer will receive output artefacts from the task plug-in and save their path in the task update status event

Acceptance criteria

GIVEN a task definition contains input artefacts WHEN the task is dispatched THEN the artefact values should be converted to a file path and included in the task dispatch event
GIVEN the task definition contains an input artefact WHEN the artefact value references a non-existent artefact THEN it shouldn't be included in the artefact map (note: in the future we'll add the notion of mandatory artefacts, see #99)
GIVEN a task update event contains an output artefacts map WHEN the task is updated THEN the output artefacts and their paths should be stored in MongoDB

Implementation notes

Artefact values can be one of the following:
1.1. context.input should be converted to the folder where the original payload has been stored. In the case of a DICOM input, that is the study folder.
1.2. context.executions.TASK_ID.artifacts.ARTIFACT_NAME should be converted to the path provided by the task plug-in for this artefact name in the output artefact map it provided when TASK_ID was wrapped up. These are "registered artefacts".
1.3 context.executions.TASK_ID.output_dir should be translated to the output directory of the referenced task
See #95 for more information on the task status update event. It is as part of handling the task status update that output artefacts should be registered.

File structure
The following structure should be used within MinIO:
BUCKET
- {{PAYLOAD_ID}}
-- dcm
-- fhr
-- workflows
--- {{WORKFLOW_INSTANCE_ID_1}}
---- {{execution_id_1}}
----- Files generated by execution 1
--- {{WORKFLOW_INSTANCE_ID_2}}

Int tests:

Send task dispatch message with artefact
Receive task update event with same artefact

Data:

Correct artefact (Happy path)
Null

@RemakingEden can we close this as complete?


AMP not showing delete button with full rights

Bug Report
System Information

Ubuntu 22
AMP version v2.4.4, built 17/05/2023 17:54 (Network Edition)
AMP release stream Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

What are you trying to do?
So I'm trying to add a role with all rights for my friends. But when I did this and tested it I've seen, that they cannot delete any instances because most of the buttons are just not there.
What are you expecting to happen?
That with that role (who has all rights I can set) they can delete instances.
What is actually happening? ('Nothing' is not an acceptable answer!)
The buttons to delete a instance are not shown.

Reproduction
Enter reproductions steps here.
adding a role: "testadmin"
giving the role "testadmin" all rights I can set
assign the role to the user "testuser"
logging in as the testuser
click on an instance
Here some screenshots:

Does someone have an idea what I may did wrong or what happend?
Kind regards,
Constantin

I noticed this same issue, it appears only Super Admin - the built in role, can assign those permissions to a user. We noticed this when one of our users who had our Admin and Super Admin roles both assigned ( this had been working for more than a year) and all of a sudden they complained about not seeing these buttons. After troubleshooting I found that if I gave them Super Admin only it would show, but giving them Admin ( which has everything checked just like yours) it wont show, even logging out and back in.
Something in the permissions changed recently, and I'm not sure if it is currently possible to give the permission to modify the instances other than giving "Super Admin" at this time. Would love to know what changed, and if there was a reason behind it.

This has been fixed in a subsequent update.

Hello @PhonicUK
Thank you for your Work.
I've updated AMP and tried on the same user as before.
Somehow I still can't see the delete button but the start and stop buttons.
I've made a screenshot of it:

I am seeing the same


Create OpBeat Release notification script

OpBeat Requires notification when a server is updated.
We should create a script that can be run whenever the server is updated. This script will need to require the .env environ vars using dotenv
Here is the actual URL that most be hit when servers are updated using cURL:
curl https://opbeat.com/api/v1/organizations/<organization-id>/apps/<app-id>/releases/ \
    -H "Authorization: Bearer <secret-token>" \
    -d rev=`git log -n 1 --pretty=format:%H` \
    -d branch=`git rev-parse --abbrev-ref HEAD` \
    -d status=completed

We should use something like Request or the http module to make this request as port of the script
For more info here is the associated docs: https://opbeat.com/docs/articles/get-started-with-release-tracking/

@BerkeleyTrue we have a live version thats suppose deal with that. but i cant figure out what was wrong. Can you look at prod server ?

we have a live version thats suppose deal with that.

I'm not sure what you mean by this. Can you clarify?

@BerkeleyTrue its running  at production

@jrogatis What is running in production? OpBeat?

@BerkeleyTrue yes. at least is the same code that works at my local and at heroku... but not at the production.


Cant Use TS in VS 2013 Existing Projects

I have VS 2013 Community Edition Update 4 and have installed TS 1.14 from download today.
Went into VS and created a new TS project.  All worked fine, just like in the playground.
Then, I went into several of my existing projects in VS 2013 and tried to add new item, typescript file.  No such menu choice was presented.  In fact, when in the EXISTING projects, it is as though Typescript was never installed for my VS 2013; however, the fact that I can create a NEW typescript project and run all the demos OK negates that conclusion.
Why am I not able to "see"/use Typescript in my existing VS 2013 projects but AM able to do so in a new project in the same instance of VS 2013?

What kind of existing projects? Can you upload a sample project that reproduces the problem?

I discovered my problem.  It was one of comprehension and understanding.
I added a NEW PROJECT (using the typescript template) to my solution.  I had been trying to shoehorn typescript functionality into the existing project in my solution, which, up to that point was a single-project solution.  I did not understand that typescript functionality has to be in its own project, with the overall solution benefitting from the js files generated out of that project.
Am I correct that you always need a two(or more)-project solution if you want to use typescript with an existing project?

It depends on your existing project. For example, if you create a normal ASP.NET MVC app and then Add New Item -> TypeScript File to it you'll get this dialog:

You can see the changes to the project file that occur after this that enable TypeScript support if you wanted to try hacking them into some other project type (no guarantee it would work in some arbitrary project type). What is your existing project exactly?

Project started out in WebMatrix.  It is a WebPages  Website project.

I took a brief look at WebMatrix and it doesn't appear to use .csproj files that would allow this scenario to work. I don't know the product well but it appears to have some ASP.NET project templates or no project file at all.


Media Queries with Webpack 2

I cannot get to work with webpack 2 for example.
@custom-media --small-viewport (max-width: 30em); to work with webpack 2.

Set the postcss options in module like this in webpack config file:
{
            loader: 'postcss-loader',
            options: {
              modules: true
              // postcss: [
              //   postCssImport({
              //     addDependencyTo: webpack
              //   }),
              //   precss(),
              //   autoprefixer({
              //     browsers: [
              //       'last 3 version',
              //       'ie >= 10'
              //     ]
              //   })
              // ]
            }
          }

and also in plugin
new webpack.LoaderOptionsPlugin({
      test: /\.css$/,
      options: {
        postcss: [
          require('postcss-import')({ addDependencyTo: webpack }),
          require('postcss-url')(),
          require('postcss-cssnext')({
            browsers: 'last 2 versions'
          }),
          precss(),
          require('postcss-browser-reporter')(),
          require('postcss-reporter')()
        ],
        context: __dirname
      }
    }),

No idea what is going on and why it's not working

This has nothing to do with cssnext. You just didn't read the doc enough for postcss-loader.

There is no "module: true" option for it
addDependencyTo: webpack require "webpack" to be an argument from former "postcss" function , see removed doc https://github.com/postcss/postcss-loader/commit/2b3e8a4c59a41b0f3497f26931bb41f1e7fae186 (poke @ai)
for webpack 2 config, the key is "plugins" not "postcss".

Thanks
I was actually getting some concepts from here:
https://github.com/MoOx/phenomic/blob/290976455fe11a5d4afd666f832e1ddf1dee3a56/themes/phenomic-theme-base/webpack.config.js
Sorry all the documentation is confusing as hell.

I totally agree, but cssnext docs can't handle the doc of others... Well... Maybe we should add something...

i am actually trying this config but complaining about the plugins array:
 module: {
 +    rules: [
 +      {
 +        test: /\.css$/,
 +        use: [
 +          'style-loader', {
 +            loader: 'css-loader',
 +            options: { importLoaders: 1 }
 +          }, {
 +            loader: 'postcss-loader',
 +            plugins: function () {
 +              return [
 +                require('precss'),
 +                require('autoprefixer')
 +              ];
 +            }
 +          }
 +        ]
 +      }
 +    ]
    }

Also do I need to do this?
https://github.com/michael-ciniawsky/postcss-load-config
I am not understanding this postcss-load-config

I for it to work...here it is...
const webpack = require('webpack');
const path = require('path');
const HtmlWebpackPlugin = require('html-webpack-plugin');
const ExtractTextPlugin = require('extract-text-webpack-plugin');

module.exports = {
  entry: {
    app: path.join(__dirname, './src/'),
    vendor: ['react', 'react-dom']
  },
  output: {
    filename: '[name].[hash].js',
    path: path.join(__dirname, './dist/')
  },
  module: {
    loaders: [
      {
        test: /\.js$/,
        use: [{
          loader: 'babel-loader'
        }]
      },
      {
        test: /\.css$/,
        loader: ExtractTextPlugin.extract({
          fallbackLoader: 'style-loader',
          loader: [
            {
              loader: 'css-loader',
              options: {
                modules: true,
                importLoaders: 1
              }
            },
            {
              loader: 'postcss-loader'
            }
          ]
        })
      },
      {
        test: /\.(gif|png|jpg|jpeg\ttf|eot|svg|woff(2)?)(\?[a-z0-9]+)?$/,
        loader: 'file'
      }
    ]
  },
  plugins: [
    new webpack.optimize.CommonsChunkPlugin({
      name: 'vendor',
      minChunks: Infinity,
      filename: '[name].[hash].js'
    }),
    new webpack.LoaderOptionsPlugin({
      test: /\.css$/,
      options: {
        postcss: [
          require('postcss-import')(),
          require('postcss-url')(),
          require('postcss-cssnext')({
            browsers: 'last 2 versions'
          }),
          require('precss')
        ],
        context: __dirname
      }
    }),
    new ExtractTextPlugin('[name].css'),
    new HtmlWebpackPlugin({
      template: path.join(__dirname, './src/index.html'),
      filename: 'index.html',
      inject: 'body'
    })
  ]
};


CheckBox pressed signal does not use a value.

I tried to check but searching for checkbox issues returns obscure results.
Godot version:

3.02
Issue description:

When we connect a CheckBox node Signal with an object, the function to be used with the Signal does not take or have any arguments.
We need a way to retrive the state of the CheckBox in the function called by the Signal and searching for the CheckBox inside the tree is not ideal.
This may be marked as a future request, but in reality does not make sense to have a signal for pressed without taking an argument with the pressed value of the CheckBox or any other Node that have Signal pressed.
Steps to reproduce:
Create a CheckBox and connect the Signal pressed to a Node with script.
the created function looks like this:
func _on_CheckBox_pressed():
we need something like this:
func _on_CheckBox_pressed(value):

Do checkboxes have a signal for 'unpressed' though? (That could explain the absence of argument)

Yes indeed you are right, toggled signal seems to work as expected.
Now I just realized that this pressed signal is in this way cause it inherits from the button.
So there is no issue... closing it.
Thanks.


server "IPv6"?

I'm using Yggdrasil, and want to host capsule without regular domain name.
By defining in config IPv6, e.g. server "[xxx:xxxx:xxxx:xxxx:xx]" -  get incomplete header error.
With /etc/hosts it works using domain alias but with IP not

Without knowing how Yggdrasil works, I think the issue is that the client connecting to gmid is not providing any SNI (server name indication).
gmid relies on the client to use SNI to understand to which virtual host the client wants to talk to, and from my understanding IPv4 or IPv6 bare addresses can't be used as host names in TLS.
I'm not sure whether gmid should gracefully handle the omission of the SNI, as it's mandatory per the gemini specification.  On the other hand, I see how this could be useful for some niche use-cases.
I guess that for your use-case with Yggdrasil you can't use an hostname?

Well, the gemini specification says that

Client and server implementations MUST support TLS SNI (Server Name Indication) and clients MUST include hostname information when making requests for URLs where the authority section is a hostname.

and I probably missed the "where the authority section is a hostname", which seems to imply that it's also OK to make requests without SNI if you want to talk to a bare IPv4 or IPv6 address.
At the moment gmid always require a SNI from the client, I'll see what can be done to relax this requirement.
Thanks!

I believe this is now fixed.  I've relaxed the requirements for SNI in 1ef0cd0cdb6512fad96ecf0830e581af677d5947 and then added support for IPv6 addresses in the IRI parsing code in b00f71ba97cf4448446fc5ccf715ca0dfda3a93b, and added some tests.
I've tested this configuration
server "::1" {
        listen on ::1 port 10965
        cert "..."
        key  "..."
        root "..."
}

and works for me.  Don't wrap IPv6 addresses in braces [...] in the server and listen directive.

I've tagged 2.0.3 which also includes this fix, so I'm closing this for now.
Thanks for the report!


ARROW-17786: [Java] Read CSV files using org.apache.arrow.dataset.jni.NativeDatasetFactory

Support CSV file format in java Dataset API

@lwhite1 Would you like to review this?

@davisusanibar Why is only write support tested? Shouldn't you test support for dataset reads as well?

@davisusanibar Why is only write support tested? Shouldn't you test support for dataset reads as well?

Hi, write support is doing by Java native libraries, then Dataset module is reading that CSV file with (FileFormat.CSV): new FileSystemDatasetFactory(rootAllocator(), NativeMemoryPool.getDefault(), FileFormat.CSV, writeSupport.getOutputURI());
Currently we are only offering Read support for Parquet. ORC and now CSV, ... write support it not implemented

@davisusanibar Why is only write support tested? Shouldn't you test support for dataset reads as well?

Hi, write support is doing by Java native libraries, then Dataset module is reading that CSV file with (FileFormat.CSV): new FileSystemDatasetFactory(rootAllocator(), NativeMemoryPool.getDefault(), FileFormat.CSV, writeSupport.getOutputURI());
Currently we are only offering Read support for Parquet. ORC and now CSV, ... write support it not implemented

Oops, sorry, I had misread the tests.

This line https://github.com/apache/arrow/blob/91e3ac53e2e21736ce6291d73fc37da6fa21259d/cpp/src/arrow/json/parser.cc#L1030 was not changed long time ago, I don't know what is the error:
[1/1] cd /tmp/arrow-lint-5g90mrbe/cpp-build && /usr/local/bin/python /arrow/cpp/build-support/run_cpplint.py --cpplint_binary /arrow/cpp/build-support/cpplint.py --exclude_globs /arrow/cpp/build-support/lint_exclusions.txt --source_dir /arrow/cpp/src --source_dir /arrow/cpp/examples --source_dir /arrow/cpp/tools --quiet
FAILED: CMakeFiles/lint
cd /tmp/arrow-lint-5g90mrbe/cpp-build && /usr/local/bin/python /arrow/cpp/build-support/run_cpplint.py --cpplint_binary /arrow/cpp/build-support/cpplint.py --exclude_globs /arrow/cpp/build-support/lint_exclusions.txt --source_dir /arrow/cpp/src --source_dir /arrow/cpp/examples --source_dir /arrow/cpp/tools --quiet
/arrow/cpp/src/arrow/json/parser.cc:1030:  Lines should be <= 90 characters long  [whitespace/line_length] [2]
Done processing /arrow/cpp/src/arrow/json/parser.cc
Total errors found: 1

This line
https://github.com/apache/arrow/blob/91e3ac53e2e21736ce6291d73fc37da6fa21259d/cpp/src/arrow/json/parser.cc#L1030
was not changed long time ago, I don't know what is the error:
[1/1] cd /tmp/arrow-lint-5g90mrbe/cpp-build && /usr/local/bin/python /arrow/cpp/build-support/run_cpplint.py --cpplint_binary /arrow/cpp/build-support/cpplint.py --exclude_globs /arrow/cpp/build-support/lint_exclusions.txt --source_dir /arrow/cpp/src --source_dir /arrow/cpp/examples --source_dir /arrow/cpp/tools --quiet
FAILED: CMakeFiles/lint
cd /tmp/arrow-lint-5g90mrbe/cpp-build && /usr/local/bin/python /arrow/cpp/build-support/run_cpplint.py --cpplint_binary /arrow/cpp/build-support/cpplint.py --exclude_globs /arrow/cpp/build-support/lint_exclusions.txt --source_dir /arrow/cpp/src --source_dir /arrow/cpp/examples --source_dir /arrow/cpp/tools --quiet
/arrow/cpp/src/arrow/json/parser.cc:1030:  Lines should be <= 90 characters long  [whitespace/line_length] [2]
Done processing /arrow/cpp/src/arrow/json/parser.cc
Total errors found: 1

Just rebase and working fine now

Please, when you have some time if you could review this, thank you

Is this for release 10?

Sorry for not getting to this sooner. Is this for release 10?

Not problem for that. Yes, trying to merge for 10 to then could create cookbooks for this CSV Reader case

@github-actions crossbow submit java

CI shows a flake in testTable:
Error:  org.apache.arrow.c.RoundtripTest.testTable  Time elapsed: 0.031 s  <<< ERROR!
java.lang.IllegalStateException: Cannot import released ArrowSchema
	at org.apache.arrow.util.Preconditions.checkState(Preconditions.java:458)
	at org.apache.arrow.c.SchemaImporter.importField(SchemaImporter.java:63)
	at org.apache.arrow.c.SchemaImporter.importField(SchemaImporter.java:56)
	at org.apache.arrow.c.Data.importField(Data.java:246)
	at org.apache.arrow.c.Data.importSchema(Data.java:266)
	at org.apache.arrow.c.Data.importVectorSchemaRoot(Data.java:377)
	at org.apache.arrow.c.RoundtripTest.testTable(RoundtripTest.java:683)

CI shows a flake in testTable:
Error:  org.apache.arrow.c.RoundtripTest.testTable  Time elapsed: 0.031 s  <<< ERROR!
java.lang.IllegalStateException: Cannot import released ArrowSchema
	at org.apache.arrow.util.Preconditions.checkState(Preconditions.java:458)
	at org.apache.arrow.c.SchemaImporter.importField(SchemaImporter.java:63)
	at org.apache.arrow.c.SchemaImporter.importField(SchemaImporter.java:56)
	at org.apache.arrow.c.Data.importField(Data.java:246)
	at org.apache.arrow.c.Data.importSchema(Data.java:266)
	at org.apache.arrow.c.Data.importVectorSchemaRoot(Data.java:377)
	at org.apache.arrow.c.RoundtripTest.testTable(RoundtripTest.java:683)

@lwhite1 have you seen this? Might be worth some investigation

@davisusanibar it may be worth investigating if we can not build/run tests in java-jars? It seems rather redundant (or maybe that's on purpose)

@davisusanibar it may be worth investigating if we can not build/run tests in java-jars? It seems rather redundant (or maybe that's on purpose)

Yes, sound redundant at first review, but, let me check what part could be improved

CI shows a flake in testTable:
Error:  org.apache.arrow.c.RoundtripTest.testTable  Time elapsed: 0.031 s  <<< ERROR!
java.lang.IllegalStateException: Cannot import released ArrowSchema
	at org.apache.arrow.util.Preconditions.checkState(Preconditions.java:458)
	at org.apache.arrow.c.SchemaImporter.importField(SchemaImporter.java:63)
	at org.apache.arrow.c.SchemaImporter.importField(SchemaImporter.java:56)
	at org.apache.arrow.c.Data.importField(Data.java:246)
	at org.apache.arrow.c.Data.importSchema(Data.java:266)
	at org.apache.arrow.c.Data.importVectorSchemaRoot(Data.java:377)
	at org.apache.arrow.c.RoundtripTest.testTable(RoundtripTest.java:683)

@lwhite1 have you seen this? Might be worth some investigation

I haven't noticed it before but there are so many failures with any given commit I probably don't pay as much attention as I should to something that doesn't look related to the changes.  I will take a look now.

CI shows a flake in testTable:
Error:  org.apache.arrow.c.RoundtripTest.testTable  Time elapsed: 0.031 s  <<< ERROR!
java.lang.IllegalStateException: Cannot import released ArrowSchema
	at org.apache.arrow.util.Preconditions.checkState(Preconditions.java:458)
	at org.apache.arrow.c.SchemaImporter.importField(SchemaImporter.java:63)
	at org.apache.arrow.c.SchemaImporter.importField(SchemaImporter.java:56)
	at org.apache.arrow.c.Data.importField(Data.java:246)
	at org.apache.arrow.c.Data.importSchema(Data.java:266)
	at org.apache.arrow.c.Data.importVectorSchemaRoot(Data.java:377)
	at org.apache.arrow.c.RoundtripTest.testTable(RoundtripTest.java:683)

@lwhite1 have you seen this? Might be worth some investigation

I haven't noticed it before but there are so many failures with any given commit I probably don't pay as much attention as I should to something that doesn't look related to the changes. I will take a look now.

It looks like a bug in the test. I will push a fix. IDK why it passes locally.

It looks like the test crashes on another attempt.

@davisusanibar can you rebase here to see if Larry's fix makes CI pass?

Ok, looks all good. Thanks!


add tree nav and pubdocs

Coverage remained the same at 96.396% when pulling ba302fbac145931eeb446dda422bd9a2167bfb8b on prod into 2d3ec02ca12e83b83db2999d2e7348283da221d7 on master.


Changes in History

Add HistoricUserOperation Object
Alter HistoricTask (Start and End Time)

Please have a look at #91

Please have a look at #91

Hello,
I just had a look. It seems to me that the new version does not include some features of the History API, that I am currently using. I will try to have a look with bigger detail.
Thank you

I've added a new issue here: https://github.com/komsa-ag/Camunda.Api.Client/issues/3
Please add missing features here or just add an issue and a pull request


Fix typos for CALL, CALL_PLT and GOT_HI20

CALL and CALL_PLT relocations refer auipc and jalr instruction pairs.
jalr is not a J-type instruction; it's I-type.
As to GOT_HI20, G means the offset inside the .got in the other ELF specs.
So the correct expression is not G+A-P but G+GOT+A-P where GOT means
the address of the .got.
I found these errata when I was trying to implement a RISC-V support to my linker.
https://github.com/rui314/mold/commit/e76f7c0d474e667cc0eccac5281709664539b714

Thanks for the patch. This makes sense to me.

As to GOT_HI20, G means the offset inside the .got in the other ELF specs. So the correct expression is not G+A-P but G+GOT+A-P where GOT means the address of the .got.

Table 4 (Variables used in relocation calculation) defines the terms. That table should also be updated.

The indirect jump instruction JALR (jump and link register) uses the I-type encoding.

LG

As to GOT_HI20, G means the offset inside the .got in the other ELF specs.
So the correct expression is not G+A-P but G+GOT+A-P where GOT means
the address of the .got.

This isn't true. In the AArch64 ELF ABI G (well, G(sym) because they're overly pedantic) is defined as the address of the GOT entry. Given we only ever want the address of the GOT entry, not its index, we should just fix the definition of G in the table to be as intended.

As to GOT_HI20, G means the offset inside the .got in the other ELF specs. So the correct expression is not G+A-P but G+GOT+A-P where GOT means the address of the .got.

This isn't true. In the AArch64 ELF ABI, G (well, G(sym) because they're overly pedantic) is defined as the address of the GOT entry. Given we only ever want the address of the GOT entry, not its index, we should just fix the definition of G in the table to be as intended.

x86-64, i386,  MIPS and SPARC psABI documents all use G to indicate "the offset into the global offset table at which the address of the relocation entry’s symbol will reside during execution". AArch64 is indeed an outlier, but it's as you pointed out not G but G(expr) (the psABI says "G(expr) is the address of the GOT entry for the expression expr"). To my knowledge, just G is exclusively used to indicate the offset in the .got. Therefore, I think it  is better to follow that convention in this document for the sake of consistency.

I added GOT to Table 4. Please take a look at this again.

As to GOT_HI20, G means the offset inside the .got in the other ELF specs. So the correct expression is not G+A-P but G+GOT+A-P where GOT means the address of the .got.

This isn't true. In the AArch64 ELF ABI, G (well, G(sym) because they're overly pedantic) is defined as the address of the GOT entry. Given we only ever want the address of the GOT entry, not its index, we should just fix the definition of G in the table to be as intended.

x86-64, i386, MIPS and SPARC psABI documents all use G to indicate "the offset into the global offset table at which the address of the relocation entry’s symbol will reside during execution". AArch64 is indeed an outlier, but it's as you pointed out not G but G(expr) (the psABI says "G(expr) is the address of the GOT entry for the expression expr"). To my knowledge, just G is exclusively used to indicate the offset in the .got. Therefore, I think it is better to follow that convention in this document for the sake of consistency.

 The same is true for i386. For x86_64 it makes sense for it to inherit its 32-bit ancestor's definitions. It does not make sense for us to mess about defining G and GOT for everything to then be written as GOT + G, it's unnecessarily verbose, we should just define G as what we actually want it to be, which is how it's currently used. If AArch64 can define G(expr) to be what makes sense for them we can define G to be what makes sense for us; I disagree that the additional argument makes a difference.

 The same is true for i386. For x86_64 it makes sense for it to inherit its 32-bit ancestor's definitions. It does not make sense for us to mess about defining G and GOT for everything to then be written as GOT + G, it's unnecessarily verbose, we should just define G as what we actually want it to be, which is how it's currently used. If AArch64 can define G(expr) to be what makes sense for them we can define G to be what makes sense for us; I disagree that the additional argument makes a difference.

You can freely define G as the address in the GOT in your psABI spec, but that's simply too confusing. AArch64 redefines G as a function rather than a variable, so there's no chance to confuse G(expr) with G. But if you redefine G to have a different meaning without changing the notation at all, I'm sure that that'll confuse lots of readers of this spec.
This very document is a good example of the confusion of the notation conflict. You guys defines G as the offset in the GOT in Table 4 but uses G as the address of the GOT entry in Table 3. Even the authors of the psABI can make such mistake by redefining G, I think it is unavoidable for readers to make the same mistake.
The easiest way to avoid that confusion is to write it as G + GOT. Just adding + GOT in one place can make it clear what that means to those who are familiar with other psABIs. Or, you can introduce a new symbol other than G to indicate the address in the GOT. But please do not redefine existing, established notation for a similar but different purpose.

My first impression is fix the G notation become Address of the GOT (Global Offset Table) plus the offset of the symbol into the GOT, then the new symbol is not needed anymore but I realized @rui314's point is G already widely used in another target, the exception is AArch64, but AArch64 redefine that as function rather than a symbol, so that could prevent confusion when implement the linker.
The major reader/user of this part would be the system software developers like @rui314, they well know lots of system software knowledge amount different architecture, but they might not RISC-V expert, so redefine those symbol might misleading those developer.
So I think current we have following options:

Introduce GOT symbol (@rui314's proposal)
Re-define the G to a function rather than symbol like AArch64.
Define another symbol other than G to prevent confusion, maybe GO or ?

I don't have strong opinion for those options, but I would prefer made a change here rather than just update the definition of G.

@rui314 We've discuss this issue in last psABI call, and we agree redefine that is too ambiguous, and prefer to define G as function as AArch64, did you mind update with that? Otherwise this PR is LGTM :)
[1] Meeting minutes: https://github.com/riscv-admin/psabi/blob/master/MINUTES/meeting-20220207.adoc

OK, great! But what does the G take?

I would prefer using same term as AArch64, take an expression (expr), thanks :)

Do you want me to introduce another symbol GDAT just like AArch64? Here is the AArch64 psABI. https://developer.arm.com/documentation/ihi0056/g/

For RISC-V, G's argument will always be S+A, so we'll always write G(S+A) to indicate the address of the GOT entry. But I'm not sure if it is any better than G + GOT.
Is redefining G as a function and always using it as G(S+A) better than always writing G + GOT (which is in line with other psABIs)? I honestly think that we are creating a bigger problem than the one we are solving with that...

Oh, I got your point, but I am concern about the addend (A) is kind of weird used in current expression, although we are always emit zero addend for now, but that is broken if addend is non-zero, e.g. A = 8 in RV64 means point to next GOT entry of S?
The problem is not introduce by your PR and existing here for long time, but I think using G(S+A) - P could fix that issue, but G + GOT + A - P isn't, that's why I prefer introduce G(expr), or we can fix that issue by changing G + GOT - P that to fix that issue, did you have suggestion on this issue?

As an expected reader of the document, I'd appreciate if it would just be G + GOT - P. I'm familiar with these symbols and their definitions already, and they are easy to understand. I actually use these symbols as macros as you can see here: https://github.com/rui314/mold/blob/main/elf/arch-x86-64.cc#L284. It wanted to keep the code in sync with the spec, so a simpler notation would be appreciated.


Add server env connect timeout

When we need to connect to a SCDF env either directly from extension(axios) or from LS(RestTemplate) it'd be good if user could define a timeout as now we either hardcode that value or use some defaults. Timeout is highly dependant on a system where things are running and i.e. if env is not routable(i.e. vpn being doing) UI kind kinda breaks. Using low timeouts then may cause other issues if user env is just slow. So atleast allow them to define these.

dup of #18


Problems with gpu instancing: location, scale, model integrity

@bertt, hello,
I created tileset with single object (tree) to demonstrate the problem I encountered with. Without gpu instancing, generated tileset looks as expected:

But when I enable gpu instancing, then object scale and location are invalid:

Also the model is corrupted. Original:

From tileset:

Model and tileset are available at https://drive.google.com/drive/folders/1oO32_vzYSDY18r3n0mh-dj7q4U03gAcP?usp=drive_link

Quick inspection: The used model is a bit complicated (has multiple meshes) and uses another extension like KHR_materials_ior. Can take a look if this can be fixed, but for now better use a more simple tree model.

ok, thank you!

Issue is in the multiple meshes, could make it work but then getting the attributes in CesiumJS was not longer working... Will look again later.
Anyhow, its quite a large model, creating/visualizing tiles will be much slower... I think using a simpler model will make more sense.

hello @bertt

some questions:

This is with I3dm's?

no, i3dm's are generated with --use_gpu_instancing false produce no warnings. Warnings appear when tileset is generated with --use_gpu_instancing true, tiles have .glb extension

Does this warning also occur with valid glTF ?

yes, even with Box.glb, if gpu instancing support is enabled

Validate the tileset with https://github.com/CesiumGS/3d-tiles-validator (it also does glTF validation);

here's validation output:
Validation result:
{
  "date": "2024-10-16T09:22:54.668Z",
  "numErrors": 0,
  "numWarnings": 0,
  "numInfos": 1,
  "issues": [
    {
      "type": "CONTENT_VALIDATION_INFO",
      "path": "content/0_0_0.glb",
      "message": "content/0_0_0.glb caused validation infos",
      "severity": "INFO",
      "causes": [
        {
          "type": "CONTENT_VALIDATION_INFO",
          "path": "content/0_0_0.glb",
          "message": "Content content/0_0_0.glb caused validation infos",
          "severity": "INFO",
          "causes": [
            {
              "type": "CONTENT_VALIDATION_INFO",
              "path": "/extensionsUsed/0",
              "message": "Cannot validate an extension as it is not supported by the validator: 'EXT_structural_metadata'.",
              "severity": "INFO"
            },
            {
              "type": "CONTENT_VALIDATION_INFO",
              "path": "/extensionsUsed/1",
              "message": "Cannot validate an extension as it is not supported by the validator: 'EXT_mesh_gpu_instancing'.",
              "severity": "INFO"
            },
            {
              "type": "CONTENT_VALIDATION_INFO",
              "path": "/extensionsUsed/2",
              "message": "Cannot validate an extension as it is not supported by the validator: 'EXT_instance_features'.",
              "severity": "INFO"
            },
            {
              "type": "CONTENT_VALIDATION_INFO",
              "path": "/accessors/3",
              "message": "This object may be unused.",
              "severity": "INFO"
            },
            {
              "type": "CONTENT_VALIDATION_INFO",
              "path": "/accessors/4",
              "message": "This object may be unused.",
              "severity": "INFO"
            }
          ]
        }
      ]
    }
  ]
}

Sample tileset with Box.glb, producing warnings:
output.zip

you can try to set the tags column (all features will have attribute id=0):
update your_table set tags = json_build_array(json_build_object('id',0))
and create the tiles again and see if the warning disappears

you can try to set the tags column (all features will have attribute id=0):
update your_table set tags = json_build_array(json_build_object('id',0))
and create the tiles again and see if the warning disappears

This didn't help, so I've experimented a bit and here's what I can observe:

warningInvalid accessor for instance scales arises if all rows in db table have scale=1
warning Invalid accessor for instance translations arises when number of rows in db table is equal to 1

So it's not a problem, as this will not happen with real data.


Add -[IGListAdapter sectionControllerForSection:]

Before the diff, you can lookup the object for a given section, and then lookup the section controller for that object, but this seems like a pretty valuable/common operation.

I put the changelog in the 2.2.0 release, I hope you guys will drop it in.


transformer: improve the implementation of eliminating type-only exports

#3820 was added IdentifierReference to ExportSpecifier. This means we no longer need to collect type-only import binding names.
https://github.com/oxc-project/oxc/blob/445603444f98402fb66a4a956b1023611bc66133/crates/oxc_transformer/src/typescript/annotations.rs#L143-L146
We just need to check if the import binding has a type reference.
But, unfortunately, we're currently stuck due to issue #3826. Once it's fixed, I'll be able to start working on this.

https://oxc-project.github.io/oxc/playground/?code=3YCAAICjgICAgICAgIC0G8rnONK883rgbiRJi4tneyT%2FrTHJB6mX%2Ba%2BqVzpzEyct039%2FXqCAgA%3D%3D

We may also need a different flag for the type/value import

I think we need to resolve the questions in #3799 before tackling this.

According to comment there, #3827 closes this. @Dunqing please re-open if that's not correct.


Bright tile 23163 marked as unsure

Tile 23163 has a cluster of spurious redshifts in petal 7
https://data.desi.lbl.gov/desi/spectro/redux/daily/tiles/cumulative/23163/20240609/tile-qa-23163-thru20240609.png
I marked the tile as "unsure" during QA.

This is in the same situation as https://github.com/desihub/desisurveyops/issues/197; let's go ahead and mark these as good for now, and do the archiving & MTLing.

Marked as good.


Fix space in IMG link

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


generate multicorner spefs for each block

Generate multicorner spefs for blocks that didn't get updated

Some blocks without views harder to generate spefs.
POR, XRES, mgmt_protect_hv and mprj logic high. Discussion today.

related to #199

Solved by pull request #209

@RTimothyEdwards PR #209 is related to PrimeTime signoff of the re-implemented blocks. PT doesn't generate spef, only lib and sdf

This is the classification of blocks that don't have multi-corner spef files:

needs a def update:

xres_buf
gpio_defaults_block_0403
gpio_defaults_block_1803
gpio_defaults_block_0801
spare_logic_block
mprj_logic_high
mprj2_logic_high
mgmt_protect_hv

They either do not have a def view or the mag view belong to a different commit than the def view.

can be generated:

constant_block

This was done by openlane at a time when it didn't save all spef corners.

openroad crash/fail:

gpio_signal_buffering*
chip_io*

these have updated def views but when attempting extraction with openroad it leads to crashes similar to this -> https://github.com/The-OpenROAD-Project/OpenROAD/issues/2429

@RTimothyEdwards I am unable to extract a def using latest version of magic. Perhaps you can update the views in (1) and add the script to generate them. I believe most of them are simple and will not cause the same crash with openroad but we need to investigate the cause crash.

For any block that just has standard cells (which is everything above except for chip_io), use
load <cellname>
extract do local
extract no all
extract all
def write <cellname>

For chip_io and chip_io_alt, there is a script in the caravel repository scripts/chip_io_prep.sh that generates the DEF files (and also GDS;  you may want to modify the script to just run the DEF generation part).

The problem that I faced is that def write took too much time. It was almost running indefinitely. I killed magic before it got to finish but I am going to try again.

Status of this ticket was marked implementation complete incorrectly. The team identified this issue as incomplete while working on issue #247 yesterday and I've marked it back to TO DO as there are still issues in the SPEF extraction processes of two cells that must be resolved before tapeout. (gpio_signal_buffering and dffram).

Tracker for chip_io: https://github.com/efabless/caravel/issues/372
Tracker for gpio_signal_buffering: https://github.com/efabless/caravel/issues/369


Fix join issue

Fixed an issue with mege keys

Codecov Report
Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (59fcbd6) 82.22% compared to head (7f2fb80) 82.82%.

Files
Patch %
Lines

tablite/nimlite.py
82.35%
9 Missing :warning:

tablite/joins.py
90.47%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #135      +/-   ##
==========================================
+ Coverage   82.22%   82.82%   +0.60%
==========================================
  Files          27       27
  Lines        4197     4210      +13
==========================================
+ Hits         3451     3487      +36
+ Misses        746      723      -23

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


An option for the user to choose that whether "enter" or "shift+enter" should be used to produce line breaks

Hello @isteinbrecher,
I'm happy here to initiate a new request as you recommended.
Some users may be accustomed to using ”enter“ for line wrapping in other LaTeX editing environments, they are prone to triggering the compilation process incorrectly when using LaTeX2AI and needing line wrapping. Sometimes, this can lead to tedious operations.
If possible, It would be better to set a configuration option that allows users to choose whether to use ”enter“ or ”shift+enter“ for switching lines.
By the way, can the default height of the code editing area be increased? When inputting large multi line formulas, the number of lines displayed at the current height is very limited, and frequent operation of the slider is not very convenient. Perhaps this height value can also be used as an option for users to set themselves.

Thanks for the feature request @MGG1996 . With #180 this functionality was added to LaTeX2AI. Now enter works as expected in a standard editor, additionally SHIFT+ENTER can be used as a hotkey to close the item create / edit form. The user can select the behaviour in the settings dialog. I played around with this a little bit and it feels much more natural this way, so the new behaviour is the default. This will be in the next release.
@MGG1996 The option to change the window size is for sure a good idea, unfortunately this is not very straight forward. Could you open a separate issue on that?

OK @isteinbrecher,
As your wish, I will open a new issue for the second advice.

With the release of v1.0.0 it is now possible to select this behaviour in the options (this is actually the default now, since it feels more natural). Thanks for the suggestion @MGG1996


Remove pow/nonce - python

fixes https://github.com/iotaledger/iota-sdk/issues/932

Not sure about removing the nonce from the test values.

Not sure about removing the nonce from the test values.

Why? Would fail if we leave them.


update logging-demo to use API v1beta3; modify the way to access Elasticsearch and Kibana services

update logging-demo to use API v1beta3; modify the way to access Elasticsearch and Kibana services.
Fixes #7818.

@nikhiljindal @mbforbes

@satnam6502 what do you think?

The reason I have not updated this yet is that I am furiously working away on the logging blog article and in the process making a lot of changes to our logging setup and examples. We could merge this -- but the changes would be short lived since they are likely to be eclipsed by my work in progress.

Thank you very much for your help.

No problem : )

@satnam6502 shall I merge this in for the interim until your overhaul is complete?


Add prefetch instructions

Add instructions for safely doing prefetches and writing data without polluting the cache. Useful when processing large files in a single pass.

The prefetch seems fine, but the streaming writes can cause trouble if they're used and then the reference is sent to another thread without a fence instruction. I think that we can only add the prefetch functions as safe functions.

Should be out now in version 0.7.1, sorry about the delay!


Make sure mul happens before div

otherwise the expression evaluates to 0

This doesn't look right, and even feels malicious. Can you check again what you were trying to do?

So the expression 44100 * 2 / 3 evaluates to 0 becuase the interger division 2 / 3 = 0. This means that the statement is equivalent to Vec::new. With (44100 * 2) / 3 the expression correctly evaluates to (sample_rate * channels) / frequency

Nah, that's not how math operators work. https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=29abc9ef177c7ae93e790b29aeae8631

You are right.
Gosh I've been thinking that operators worked differently for decades!
https://en.wikipedia.org/wiki/Operators_in_C_and_C%2B%2B#Operator_precedence


Fix More dropdown (html, aria, bootstrap classes)

Fixes #1363.
Note that this PR specifically does not use the combobox role or pattern because a dropdown in a nav bar is not, semantically speaking, a select-only combobox. Accessibility expert Adrian Roselli advises against using such patterns for navigation, and proposes a disclosure widget navigation pattern instead.
The ARIA APG Patterns site also has an example of the disclosure pattern in navigation, which this PR uses as a reference for the HTML markup. Essentially this meant replacing aria-haspopup with aria-controls.

hover styling is incorrect I think (see "API") in screenshot:

Also might be nice to unify the styling of this dropdown and the version switcher? Version switcher has borders between items and the hover background color change affects the whole area of the container.  I think I'd opt for making this one look like the version switcher rather than vice-versa, but I don't have super strong feelings about it.

@drammock, you're too fast! :-D
You got to this PR before I finished self-reviewing it, haha.
Yeah, the dark theme styles will need to be fixed. I think it looks okay in light theme, though — minus one minor issue where the hover color clips the focus ring, which I was going to submit a separate PR for.
As for unifying the style of the nav links dropdown with the version switcher, I would only be a fan of doing that if it simplifies the code. Otherwise, I don't feel strongly that the styles for the two things need to be the same, since they are not semantically the same, as I mention in the PR description.

ah, yeah, I see the focus ring issue now. FWIW that happens on the version switcher too, please do fix both

I created an issue (#1415) referencing your comment so we can keep track of the bug.

I also think that it's OK for two things that are technically semantically different to have similar appearance given that their interaction mechanisms are the same

I totally agree.
I'm wonder why that padding (gutters) you mention is there. At first, I thought it was just default Bootstrap styling, but then I saw from the Bootstrap dropdown docs page that that is not the case. It looks like the gutters were added in #754 but I'm not sure why.

It looks like the gutters were added in #754 but I'm not sure why.

I think it's safe to assume "no good reason" for adding the padding and you should feel free to remove it.

Okay, my last two commits should do the trick!

Thanks @drammock for bringing up the point about colors. I totally agree. I remapped the Bootstrap dropdown colors in my latest changes.

Seems you don't have our pre-commit hooks set up?

No, I do but it was the code comment I committed via GitHub suggestion that caused the CI lint job to fail. :facepalm:

at a quick glance I think the test failure relates to changing the dropdown from a div to a ul.  @gabalafou can you fix? If you can't sort it LMK and I can try.

All checks pass now, I think this is ready to go


Multi-lined environments with \*

Issue Summary
MathJax v3.
With multline and similar, \\ and \\[2ex] work as expected, but \\* and \\*[2ex] fail.  The star and trailing optional argument are not absorbed.
Steps to Reproduce:
<!DOCTYPE html>
<html lang="en-US">
<head>
<script
         id="MathJax-script"
<EMAIL_ADDRESS>></script>
</head>
<body>

<p>

\begin{multline}
\tan a+b < c \\
\cos (d)+sin(e) \\*
\cos (f)+sin(g) \\*[2ex]
h+i=j
\end{multline}

</p>

</body>
</html>

You are correct, MathJax doesn't process \\*.  This is because the star is about controlling page breaks, which are outside the scope of MathJax output.  But I do suppose it should process and ignore the star.  I've marked it as a "feature request" for that.