"HOMEBREW_DEVELOPER" description in man page could use an update

The man page info for HOMEBREW_DEVELOPER seems a bit out of date these days, compared to it's actual functionality:
If set, Homebrew will print warnings that are only relevant to Homebrew developers
(active or budding).

It seems like there's some kind of auto-update mechanism around now, which forcibly resets /usr/local back to master branch.  And blows away all uncommitted changes at the same time.
Guess how I found out? 😁
@tdsmith mentioned on IRC that the HOMEBREW_DEVELOPER variable should probably stop that from happening. (thanks @tdsmith!)
It would be pretty useful to have that documented better so others can avoid this same mishap. :wink:

You need to git stash pop if you don't have HOMEBREW_DEVELOPER set. It's running git stash, not blowing your work away.

You need to git stash pop if you don't have HOMEBREW_DEVELOPER set. It's running git stash, not blowing your work away.

(which a printed message told you).
I'm not convinced I want to get into the habit of documenting everything HOMEBREW_DEVELOPER does but we could tweak it to be a bit more encompassing.

Guys, if a printed message told me that, I would have seen it. 😦

Thinking about it more, it's possible some message was printed out and lost in the --verbose output.
But I didn't run --verbose every time, and I really don't remember seeing a message. 😦
Anyway, the point is more that this variable should be a bit more prominent in the man page, along with it's purpose and what it does. :smile:

I don't think we want most users ever setting it, so making it prominent in the man page seems somewhat counterproductive since the effect will be more users setting it.

I don't think we want most users ever setting it, so making it prominent in the man page seems somewhat counterproductive since the effect will be more users setting it.

I agree.

Heh Heh Heh
I guess I'm misunderstanding the target audience for that variable then.
So far my understanding is that it's for people working on Homebrew or the various formulas.
Is that wrong?

For most contributors, setting HOMEBREW_NO_AUTO_UPDATE while editing files in-place is probably a less sledgehammery approach.

Yep, reading the description of that in the man page, it sounds like the right kind of thing:
HOMEBREW_NO_AUTO_UPDATE
              If set, Homebrew will not auto-update before running brew install and brew upgrade.

Is tweaking to say something like this feasible?
HOMEBREW_NO_AUTO_UPDATE
              If set, Homebrew will not auto-update before running brew install and brew upgrade.

              If you are making manual changes in formulas, such as editing them prior to submission,
              you'll want to set this.  Otherwise Homebrew will `git stash` your changes, and
              they'll seem to disappear.

@justinclift I don't think so. brew update tells you when it's stashed your changes and this is unrelated to autoupdate as brew update itself has the same behaviour.

k.  Is this a recent change, or is it just something I've managed to miss through luck?

It's related in that someone working on a formula upgrade is likely to be running brew install multiple times while doing that work, and wouldn't want to have to keep popping the changes, so it'd be one or the other, HOMEBREW_DEVELOPER or HOMEBREW_NO_AUTO_UPDATE that they'd want to set for the duration of the work.
But I agree that these subtleties don't belong in the man page. More so, it would belong in the docs on how to contribute to Homebrew, etc.

https://github.com/Homebrew/brew/pull/679#issuecomment-239663569
2 days ago is an eternity, so no not recent ;)

Ahhh got it.  Feel free to change the title of this issue to reflect "in the docs" (etc) rather than in the manpage.
It's just the man page was one of the first things I checked after the conversation on IRC.  For someone coming at this from a different direction (eg not hitting the problem at the time), yeah, the man page may not be the right place for them to learn about it before experiencing it. 😁

The brew-update-stashes-but-doesn't-pop-unless-developer thing has been around for months though. It's only auto update that's new.

Ok.  As a data point, for me it's extremely rare for me to run brew update.  I may have only run it 5 times (total) ever if that.
That's because I blow away all of my formulas very often:
$ brew remove `brew list` --force

... generally prior to doing new formula development, Qt development, etc.  So... they don't last long enough to need an update.  Rarely more than 2-3 days.
brew install though, I run all the time. :smile:

Do you mean update or upgrade?

Ahhh, yeah, upgrade.

brew update I run often... but don't think I've ever done that with any manual changes in the tree.
Not trusting git to not getting mixed up in that situation.

Not trusting git to not getting mixed up in that situation.

I suggest you set HOMEBREW_DEVELOPER=1 for a while and see how it goes. brew update is pretty smart about the whole stash/pop dance.

k, will do. :smile:


Enable HTTPS for passportjs.org

Please enable HTTPS for www.passportjs.org
A site dedicate to a security middleware shouldn't be using http.
It's ironic and makes it seem not trustworthy.

The website is now available with TLS at https://www.passportjs.org.


feat: avm/res/network/virtual hub

Description
Migrated module from CARML to AVM. Closes #547

@ChrisSidebotham don't want to dismiss your review. Hence please check as well ;)


Support 3D in ArcLayer

For #3869

Change List

Use 3D positions for source/target
Calculate parabola between 3D points
Render test
What's new

Coverage decreased (-0.02%) to 80.347% when pulling 657c9228fd066b18dc5c7110403a650ce3135124 on x/arc-3d into 429218a7bec9a934fe775fcfaa332c06339f6030 on master.


How can I use this plugin with no base layer?

Thanks for this great plugin! But I have tried all kinds of ways to use this plugin without base layers , I have another control managing the base layers, can you add the ability to use no base layer?

Is this example what you are trying to implement (only showing the groupedOverlays/options in the control)?
Have you tried declaring the baseLayers variable, but leaving it empty?
For example:
var baseLayers = {
};

Yes, you can do: L.control.groupedLayers(null, overlays, options).addTo(map);

I am sorry, both of these work, and my application is working now. the error was actually cased by a misconfiguration of my overlays which caused me to think my empty baselayer object or use of null was causing errors. Thanks  both of you.


Fix socket2 dependency.

Fix socket2 dependency issue.

@ShaleXIONG, can you rebase?

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 3fa5023284399a49db631ec9abd790a378f50449
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: ef6dfd9c835b3878d7928c220b12f3b92aa8e906
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

+1 reached, merging


@lit/localize-tools/0.6.6 regression with variables

Which package(s) are affected?
Localize (@lit/localize)
Description
The extract function is adding an extra <x> tag to the beginning of a string.
<?xml version="1.0" encoding="UTF-8"?>
<xliff version="1.2" xmlns="urn:oasis:names:tc:xliff:document:1.2">
<file target-language="es-419" source-language="en" original="lit-localize-inputs" datatype="plaintext">

<trans-unit id="h71437edc6d0d1c12">

  <source><x id="0" equiv-text="${this.who}"/>Hello <x id="0" equiv-text="${this.who}"/></source>
</trans-unit>

</file>
</xliff>

Reproduction
See this minimal reproduction which uses the "my-greeter" example.
git clone git@github.com:pmcelhaney/lit-localize-bug.git
npm test
cat ./xliff/es-419.xlf

Note line 6 of that file.
Expected:
<source><x id="0" equiv-text="${this.who}"/>Hello <x id="0" equiv-text="${this.who}"/></source>

Actual:
Every time you run lit-localize extract (npm test), it alternates between these two.
<source>Hello Hello <x id="0" equiv-text="${this.who}"/></source>

<source><x id="0" equiv-text="${this.who}"/>Hello <x id="0" equiv-text="${this.who}"/></source>

Workaround
I have not found a workaround.
Is this a regression?
Yes. This used to work, but now it doesn't.
Affected versions
Failing in @lit/localize-tools/0.6.6, worked in 0.6.5
Browser/OS/Node environment
MacOS 13.1
npm: '8.5.0',
node: '16.14.2',

Thanks for filing, investigating now. Likely related to https://github.com/lit/lit/pull/3514.

Fixed in https://github.com/lit/lit/pull/3576

The fix was just released in<EMAIL_ADDRESS>Thanks again for the clear bug report and repro!


gzip all the minified JS, CSS, and HTML

Consider gzipping all the minified JS, CSS, and HTML after the minification process based on the doGzip flag!

Introduced gzip all the minified JS, CSS, and HTML


Create Docker image for use with Google Cloud Workstations

Background
We want to provide a Docker image that enables Posit Workbench within Google Cloud Workstations. This image will likely closely mirror the default PWB image that we provide, but with a few significant differences:

PWB needs to be listening on port 80 instead of 8787
The image should be built on a base image provided by Google. Perhaps us-central1-docker.pkg.dev/cloud-workstations-images/predefined/base:latest.
Cloud Workstations are single-tenant environments. Therefore, users shouldn't need to navigate the Workbench login screen when the image starts. Instead, users should be taken straight to the homepage.

A PoC image was developed with @colearendt and can be found here: ghcr.io/colearendt/rsw:james
Definition of Done

Write a new dockerfile to https://github.com/rstudio/rstudio-docker-products that is pretty much a clone of our workbench image except:

We'll open port 80 not 8787 at this line
And instead of this being the ubuntu 22.04 image, it'll be the cloud-workstations-images/predefined/base:latest flavored-image.

Which this will probably entail the most work

Ask the workbench team to produce a special single-tenant version that skips login

Swap this into our image instead

They may have some security scanning requirements that we will need to talk about.

From James: if we start seeing that having this different base image is too painful, I can go back to Google to see if we can do this differently.

@colearendt @blairj09 do we still have the dockerfile produced for ghcr.io/colearendt/rsw:james laying around? That code could be helpful to look at it, if we know it was mostly acceptable to gcp at the time.

I talked with Cole about the dockerfile, and he said this was essentially just the workbench dockerfile.  He believes he had to make some port change (I think?) but doesn't have anything he saved around for this.

I have a loose plan in mind for how we could tackle this issue, at least to start with:

Write a new dockerfile to https://github.com/rstudio/rstudio-docker-products that is pretty much a clone of our workbench image except:

We'll open port 80 not 8787 at this line
And instead of this being the ubuntu 22.04 image, it'll be the cloud-workstations-images/predefined/base:latest flavored-image.

Ask the workbench team to produce a special single-tenant version that skips login

Swap this into our image instead

One additional observation that came to light during some exploration yesterday:
Cloud Workstations are divided into two components: configurations and workstations. Administrators setup configurations, and then individual users request workstations based on defined configurations. So individual users will, over time, build up a collection of workstations they use:

These workstations can be stopped and started, either manually or based on time-out settings specified in the configuration. Whenever a workstation is stopped, the infrastructure behind it is terminated and when the workstation is started again, new infrastructure is provisioned and the underlying Docker image is pulled (so, effectively, updates to the Docker image are automatically applied to individual workstations on restart). Throughout this process, there is a filesystem unique to the workstation that persists across restarts. This is helpful because it means things like user-installed R packages or local files on the workstation are preserved. However, since Workbench writes session status to the filesystem, this means that when a Workstation with Workbench is restarted, it will show previously started sessions, even though they don't exist any longer:

In the screenshot above, several sessions are listed but none of them are real sessions. Selecting any of them results in a 502 error.
In order to improve the user experience, we'll need to think of how we can signal all sessions to quit when the workstation is shut down so that these "ghost sessions" aren't preserved across Workstation restarts.

It looks like the shutdown process is not getting run properly, per:
https://github.com/rstudio/rstudio-docker-products/blob/ff278dbeaa93503c3ab61024374e78e3cb993b49/workbench/startup.sh#L9
This could also cause issues with license activations too. Definitely some digging / refinement to be done!

I'm attempting to get RStudio working on Cloud Workstations, and have been attempting to use gcr.io/cloud-workstations-external/rstudio but it's been failing at launch with Unable to forward your request to a backend Couldn't connect to a server on port 80.
Did the team here not write this particular rstudio image?

Hey @TomKlimovski! Thanks for your interest in this project. We're still early in the process of building out images that will support Posit Workbench on Cloud Workstations. The image that you linked to is not one that we've produced. We'll be sharing more details about this integration over time as we get the image built out.

Yes, I think we can call this one good!


Updates as I use library for Selenium testing

Tried to update on the original developer but they seem not to be responsive.

There are other errors in other methods that retrieve the token I was unable to understand how to fix.
The code is a bit hard to follow. Would be clearer if it used RestTemplate instead.


Attack animation timing

Seems like the Sfx plays immediately when you press ctrl but the animation doesn't happen till half a second later.  I haven't paid too much attention to the sfx timing so I'll take another look at it.


Implement initial version of rewrite for df.getitem as attr

@densmirn Cool feature! 🥇 Have you felt yourself a magician during writing this?

At the least I liked working on the :smile:


GH-156 Collapse nodes to function and expand node feature

Fixes #156
Given a graph like this one:

There is a new Organization section on the node context menu:

With nodes selected, pick "Collapse to Function" to move the selected nodes to a new user-defined function:

On the context-menu, a user-defined script function can be expanded, which will replace the call script function node with the nodes from the child function, as follows:

When expanding the following are not performed and left as exercises for the user:

Previous connections to the call function node are not preserved.
The function is not removed automatically; it's left in case it is used in other places.

Applied to main (=2.1) and branch 2.0.


Add Dependabot

Pull Request
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#12
#11  👈
#10
#9
#8
#7
#6
#5
#4
#2
#1 : 1 other dependent PR (#3 )
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @JackPlowman and the rest of your teammates on  Graphite

Merge activity

Oct 21, 12:42 PM EDT: A user merged this pull request with Graphite.


Patch simpeg dependencies

Patch simpeg for numpy upper bounds, next version will support 2.0
Checklist

[x] Used a static YAML file for the patch if possible (instructions).
[x] Only wrote code directly into generate_patch_json.py if absolutely necessary.
[x] Ran pre-commit run -a and ensured all files pass the linting checks.
[ ] Ran python show_diff.py and posted the output as part of the PR.
[x] Modifications won't affect packages built in the future.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Upper bound was added to recipe: https://github.com/conda-forge/simpeg-feedstock/blob/main/recipe/meta.yaml#L24

i feel like azure isn't running.... we have to wait...

Silly me using the .yml extension... It should be good to go now.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-recipe-manager. Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11961048133. Examine the logs at this URL for more detail.


Fix stats API doc

Current coverage is 43.86% (diff: 100%)

Merging #173 into master will decrease coverage by 0.14%

@@             master       #173   diff @@
==========================================
  Files            16         16
  Lines          8950       8950
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           3939       3926    -13
- Misses         3980       4011    +31
+ Partials       1031       1013    -18

Powered by Codecov. Last update 0970bad...5d0b99e


How to change delimiters?

Can I change de default delimiters?
https://vuejs.org/v2/api/#delimiters

@ebertti as described in the documentation:

This option is only available in the full build, with in-browser compilation.

Nuxt.js use vue runtime only build so this option is not available.


Aurelia Skeleton for Phonegap

Skeleton to have running quicky a phonegap application with Aurelia. I modified some paths and serve gulp task in order to host the www application folder.
Regards.

@jedd-ahyoung You took a look at this recently. Can you provide some feedback? @joelcoxokc may also have some input.

@hmarcelodn I attempted to run this with no success. gulp serve loaded the correct page, but Aurelia could not load. I also tried to run this with phonegap run in the iOS simulator after doing phonegap build for iOS, but the skeleton navigation page did not appear (the loading page never disappeared, meaning there may have been a script error). I am using Phonegap 6.0.0.
Did this work for you when you tried it?

@jedd-ahyoung I was running same environment like you when I pushed the code and I had this working on my local, my tablet and my phones with Aurelia.
I got the code again and now it is not working to me either. It is not loading aurelia-bootstrapper when doing System.import('aurelia-bootstrapper');
I will take a look and check what changed.

@jedd-ahyoung I already fixed and it is working for me: http://screencast.com/t/KlRjlKepaGU
It was the Jspm updating config.js when you performed jspm install -y .
You should run this command into www/ folder as I specified in the readme.md file.
Thanks a lot.

Hi, is it possible to create a script to install aurelia into a cordova (typescript) application ?

@valeriob  probably you should think on starting with the typescript skeleton. This one is a cordova application skeleton pre-configured using raw Ecma6. I tested it on my devices and it was working with Aurelia.

@hmarcelodn Looks like you have done some interesting work!

I'm closing this for now since I'm not sure it's 100% working and we may have a slightly different direction we want to go for the official skele. Thanks for the contribution! Even though this isn't being merged, it still has helped us out!


76/docopt subcommands

Fully implements Remy's DocOpt subcommand pattern. Fixes #76.
Simplifies CLI logic around allowed commands.
Uses get() to access CLI args throughout since they're not longer guaranteed to be present.
Fixed a bug where molecule test --platform xxx wouldn't preserve platform in state file throughout entire run.
Fixed a bug where specifying an invalid platform and provider simultaneously would blow up with stack trace.
Updated test to look for different SystemExit response code when using DocOpt's built-in --version.

:+1: looks like a good cleanup.  I'm still an argparse fan tho :p

I definitely like it better like that! :) :+1:
@retr0h out of here! :stuck_out_tongue_winking_eye:

@rgreinho @retr0h Updated. --tag and --tags work without defining --tag as an alias. And I'm not sure I want to change the name of that flag this far into the life of molecule.

For now I think we should stay with the existing style of handling tags since it matches what ansible-playbook uses.


WIP: test excluding unfixable CVE issues raised by ECR

Fails

:no_entry_sign:

Danger failed to run dangerfile.ts.

Error TypeError
Cannot read property 'length' of null
TypeError: Cannot read property 'length' of null
    at githubChecks (dangerfile.ts:21:31)
    at Object.<anonymous> (dangerfile.ts:131:3)
    at Module._compile (internal/modules/cjs/loader.js:1072:14)
    at Object.requireFromString [as default] (/home/circleci/transcom/mymove/node_modules/require-from-string/index.js:28:4)
    at /home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:157:63
    at step (/home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:52:23)
    at Object.next (/home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:33:53)
    at /home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:27:71
    at new Promise (<anonymous>)
    at __awaiter (/home/circleci/transcom/mymove/node_modules/danger/distribution/runner/runners/inline.js:23:12)

Dangerfile
16|
17| const githubChecks = () => {
18|   if (danger.github) {
19|     // No PR is too small to include a description of why you made a change
20|     if (danger.github.pr.body.length < 10) {
---------------------------------^
21|       warn('Please include a description of your PR changes.');
22|     }
23|     // PRs should have a Jira ID in the title
24|     jiraIssue({

  Generated by :no_entry_sign: dangerJS against 842f3bbb7dbf9a659facbb04cb824e4c0df7f228


Fix: Environment build logs sometimes are failed to render.

Description
Environment build logs sometimes failed to render because socket.io failed to reconnect immediately after being disconnected. This connect and disconnect is actually not necessary, as FE could just continue using the same connection.
This PR made the following changes to remove the unnecessary disconnect:

BE doesn't send logs on connect.
FE sends a request to get logs when needed.
FE doesn't disconnect until the browser tab is closed.

Checklist

[x] I have manually tested my changes and I am happy with the result.
[x] The PR branch is set up to merge into dev instead of master.

FE doesn't disconnect until the browser tab is closed.

Note there are limits to the total number of concurrent websocket connections one can have. As long as we hold at most one connection per namespace we should be in the clear.


Import fails on Services in clusters which have custom node port range

If a cluster is using a custom node port range (e.g., --service-node-port-range=30000-36000 set in kube-apiserver args), the import on Services using non-default port range will fail:
Failed to import "ns/name" (/v1, Resource=services) with error: failed to import resource: failed to create resource: Service "name" is invalid: spec.ports[0].nodePort: Invalid value: 33814: provided port is not in the valid range. The range of valid ports is 30000-32767

@some-things the fix is included in https://github.com/mhrabovcin/troubleshoot-live/releases/tag/v0.0.18


Code Review

Code review ensures the implementation of @mentions in comments adheres to coding standards, validates user existence, and efficiently stores and retrieves mentions. Focus on database schema integrity, frontend-backend integration, and error handling for invalid mentions.

“Thorough code review ensures that the @mentions feature not only meets coding standards but also verifies user existence and optimizes the mechanisms for storing and retrieving mentions. Key focus areas include preserving database schema integrity, streamlining frontend-backend communication, and implementing resilient error handling for invalid mentions, guaranteeing a robust and user-friendly implementation.”

The code review thoroughly addresses key aspects like user verification, database schema integrity, and efficient storage and retrieval of mentions. Strong focus on error handling and frontend-backend communication ensures a robust implementation. This approach sets a solid foundation for a seamless user experience.

The code review effectively covers essential aspects such as user verification, database integrity, and efficient mention management. With strong attention to error handling and frontend-backend communication, this approach lays a solid foundation for a seamless and robust user experience. Great work overall!


Zotero not working after Better BibTex Installation

Debug log ID
D581890365
What happened?
Zotero is basically showing a blank screen with no entries saying " load entries" after I installed BBT. Therefore I also cannot provide an entry specific log file. Attached the one I get from Zotero.
One speciality though: My Zotero files are located on a mounted drive. Not sure whether that may have an impact.
BBT-log.txt

Hello @retorquere, log file is attached
BBT-log.txt

I reinstalled Zotero and Better BibTeX from scratch: That solved the problem.


Use inline image for Mermaid

Describe the issue
Is is possible to use inline images when using Mermaid like it is done with Graphviz?
To not overload the browser when opening the adapters status page I would suggest to not generate the images until a user is clicking on an adapter to see more information.
Reporter
Jaco de Groot

What happens when somebody clicks the 'expand-all' button? The browser screen should not crash when there are more then 250 adapters. (This was the case and the reason it's not inline. Not sure if browsers are 'better' or Angular has dramatically improved over AngularJS...)

Maybe in that case we can only show a limited amount (or none) inline and show a button in that area which the user can press to show the image inline?

So instead of opening a popup/modal you click on a button or text to make it generate inline instead?
I guess if you click on the generated mermaid image it should still open the modal

Yes, that would be handy. Say you open a number of adapters and you want to see/demo them and scroll a bit through the page it would be nice to have them show the inline images while scrolling up and down and see the images instead of having to open them one by one and after closing you can't see them again until you open the popup again.


Delete product from PriceList

Goal
We want to be able to delete a product from a PriceList
Should use medusa-react hook useAdminDeletePriceListProductPrices
Add in <PricesTable /> in this action.

Solved in #599


Remove Default full path

Default is already included in the standard std prelude

seems to have glitched, let's restart the CI
bors r-

I'll retry once the other PR's CI run is in order too.

Weird CI ಠ_ಠ

it has its moments :smiley:
bors r+

seems that it's hanging again? maybe it's due to that canceled job? I guess I'll rerun all of them again and retry then :shrug:.
bors r-

bors r+


[BUG] Dependencies of published integration tests jar are missing

Attempting to download the integration tests jar as a dependency fails with missing dependencies:
$ mvn dependency:get -B -Ddest=snap -DgroupId=com.nvidia -DartifactId=rapids-4-spark-integration-tests_2.12 -Dversion=21.08.0 -Dpackaging=jar
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-dependency-plugin:2.8:get (default-cli) on project rapids-4-spark-parent: Couldn't download artifact: Missing:
[ERROR] ----------
[ERROR] 1) com.nvidia:rapids-4-spark-tests_2.12:test-jar:tests:21.08.0
[ERROR]
[ERROR]   Try downloading the file manually from the project website.

We need to do at least one of the following:

Publish the missing artifacts
Fix the dependencies to remove the missing artifacts (if they indeed should be removed)
Stop publishing this broken artifact

Issue #3932 will remove aggregator dependency from this integration module, and will remove tests module, so it's better to fix  #3932 first.

@jlowe
The current version is working fine if we specify all the necessary parameters.
// Will download successfully if providing the classifier and packageing parameters:
mvn dependency:get -B -Ddest=snap -DgroupId=com.nvidia -DartifactId=rapids-4-spark-integration-tests_2.12 -Dversion=22.02.0-SNAPSHOT -Dpackaging=tar.gz -Dclassifier=pytest
mvn dependency:get -B -Ddest=snap -DgroupId=com.nvidia -DartifactId=rapids-4-spark-integration-tests_2.12 -Dversion=22.02.0-SNAPSHOT -Dclassifier=spark301
mvn dependency:get -B -Ddest=snap -DgroupId=com.nvidia -DartifactId=rapids-4-spark-sql_2.12 -Dversion=22.02.0-SNAPSHOT -Dpackaging=jar -Dclassifier=spark301

// this will fail without classifier parameter
mvn dependency:get -B -Ddest=snap -DgroupId=com.nvidia -DartifactId=rapids-4-spark-sql_2.12 -Dversion=22.02.0-SNAPSHOT -Dpackaging=jar

Are there any other considerations?

The current version is working fine if we specify all the necessary parameters.

Those commands are only working for you because it is picking up artifacts that you built locally and installed in your local Maven repository (or you are using Maven settings that add an internal repository that has these artifacts available).  It will fail on a fresh Maven repository (e.g.: add -Dmaven.repo.local=/tmp/mynewlocalrepo to any of the command-lines above) with no additional Maven settings that add non-public repositories.

Currently, we do not public the internal jars like sql-plugin, shuffle, see the https://repo.maven.apache.org/maven2/com/nvidia, only published dist, tool, integration.
Integration_tests depends on the aggregator, udf-example and the tests modules. I checked, all the dependencies are necessary.
The aggregator depends on sql-plugin and shuffle.
"mvn dependency:get" will pull all the dependencies and some of the dependencies are not published.
Is it necessary to publish all the dependencies?
By sepcifing -Dtransitive will only download the solo jar without the dependencies.
see the following:
mvn dependency:get -B -Ddest=snap -DgroupId=com.nvidia -DartifactId=rapids-4-spark-integration-tests_2.12 -Dversion=21.10.0 -Dclassifier=spark301 -Dmaven.repo.local=/tmp/.m2 -Dtransitive=false

Is it necessary to publish udf-example and tests test-jar?

The plan is to move the udf-examples into a separate repository like spark-rapids-examples.  Currently that project isn't a very good example project, as it depends on the rapids-4-spark-sql jar rather than the dist jar.  Users trying to copy the example will get a rude surprise, as the rapids-4-spark-sql jar is not published, nor do we intend to publish it as it contains classes user code should not access directly (and those classes are hidden in the dist jar).
The problem with publishing the tests jar is that it just moves the problem being described here from the integration tests jar to the tests jar.  The tests jar will then have dependencies that are unpublished in Maven Central, and I don't think we want to publish all those dependencies (per discussion above about classes we do not want to be directly accessed).

Use -Dtransitive

I'm -1 on this being considered a real "fix."  IMO that is a workaround to a bug in the published dependencies of the artifact, so it does not address the problem being described here.  It just provides a workaround for those dealing with this bug.
AFAICT the only reason we're publishing this artifact is as a convenience to get this to our nightly test pipelines.  So one option is to stop publishing this to the public repo and only publish to an internal repo that is accessed by the nightly pipelines.

Move the following to a project

Moving these out of the tests module, which has problematic dependencies for public publishing per the discussion in #3932, seems like it could be a good idea.  That should be able to be cleanly published as a separate module or moved to the integration tests module.
To sum up, we either need to stop publishing this publicly and just use the internal repo for out pipelines or work on the following (which I think we should do even if we stop publishing the integration tests jar to Maven Central):

Getting the udf-examples moved to a separate repository along with its corresponding tests.  We will need to setup nightly test pipelines for it and we should start publishing that artifact, as it has useful UDFs that users could use as-is.
Move the code identified above out of the tests module to make it easier to use without pulling in the problematic dependencies of the tests module.

After discussion:
Stop publishing the integration tests jar to Maven Central.  @NvTimLiu  Please remove it.
Remove the following dependency in the integration_tests module pom.xml. @NvTimLiu
//   it's not needed.
<!-- use aggregator jar because accessing internal classes -->
        <dependency>
            <groupId>com.nvidia</groupId>
            <artifactId>rapids-4-spark-aggregator_${scala.binary.version}</artifactId>
            <version>${project.version}</version>
            <classifier>${spark.version.classifier}</classifier>
            <scope>test</scope>
        </dependency>

[blocking] Move the udf-examples module to the spark-rapids-examples repository; Publish the udf-examples snapshot jar and rapids-4-spark snapshot jar to Maven Central; Let udf-examples depends on rapids-4-spark; Keep the rapids_udf_test.py in the integration tests. This step is blocked because rapids-4-spark snapshot jar is not published to Maven Central currently.
@jlowe any other info?

The Scala tests in the integration-tests module depend on SparkQueryCompareTestSuite resides in the test module.
Attemped to extract SparkQueryCompareTestSuite to a separate project, and then let the integration-tests module and the tests module both depend on the new project, but failed.
SparkQueryCompareTestSuite actually depends on RapidsConf and ShimLoader which are internal.
The dependency paths are:
SparkQueryCompareTestSuite ->
    RapidsConf
    ExecutionPlanCaptureCallback ->
        PlanUtils ->
            ShimLoader.loadClass ->
               ShimLoader.getShimClassLoader()

All of this is a good start but not sufficient.  As long as the integration tests jar depends on the tests jar, it will also be transitively depending on the aggregation jar.

The Scala tests in the integration-tests module depend on SparkQueryCompareTestSuite resides in the test module.

I think those tests should be moved to the tests module then.  There are many tests in the tests module that are similarly constructed, so I don't see why these tests would require special treatment and be placed in the integration tests module.

Reopen, there are PRs for the issue

Reopen, there are PRs for the issue

Ropen

Fixed:

Stop publishing the integration tests jar to Central Maven.
Remove the dependency from the integration module to the tests module.


 On my iPad mini (first generation, Dual-core PowerVR SGX543MP2), attaching the following filter to the stage halfs the fps once like lets say 100 containers are in play:
precision highp float; // intended

varying vec2 vTextureCoord;
varying vec4 vColor;
uniform sampler2D uSampler;
uniform vec4 dimensions;

void main() {
    vec3 color = texture2D(uSampler, vTextureCoord).xyz * vColor.xyz;
    gl_FragColor = vec4(color, 1.0);
}

Is this normal / simply due to the limitations of the device? Running 3.0.9 here.
If this is expected, is there anything I can do to improve this - like replacing the default shader with a custom one so that I don't have to use a filter at all?

Not sure what specifically is happening, but here is some information that might help:
When using a filter, it adds at least 1 more framebuffer to the mix along with at least 1 more shader pass (depending on the filter). Say for example you were just barely getting exactly 60 fps, then doubling the shader passes could in fact cause it to drop.
Using highp on certain devices can be seriously performance impacting, just as an experiment try only messing with the precision there. On desktop you rarely (if ever) notice, and most modern mobile devices are fine; but older devices can easily choke.
Last thing to try is profiling and WebGL inspector to ensure the bottleneck is actually the shader itself and not some weird js we have instead.
Hope that helps!

Thanks, that definitely helps. I tried mediump and lowp, but still the same.
Now I am replacing TextureShader.defaultFragmentSrc by editing the pixi source as I can't seem to find a way to access that properly. Isn't that exposed somewhere on the PIXI namespace?

Make sure also, that your sprites are getting batched. To be batched they need to use the same shader and have the same base texture.
If you have a bunch of sprites, and they all have the same shader (either default or .shader property all the same) and the same base texture (can be different texture objects, but needs to be the same underlying baseTexture object) then they will all be drawn in a single draw call.
 That could also be an issue.

Thanks again! Just reduced draw calls from 10 to 4 by rearranging some stuff. Probably not a significant improvement, but an improvement, isn't it.
I wonder if I could actually use ParticleContainer here.

If they are all sprites, and the hierarchy is only a single layer deep, you should try it. All sprites within a particle container must share a base texture, and by definition of the container they will share a shader (the particle container's shader).

 While I get like 10 more frames by replacing the standard shader, doing
color *= 0.95 + 0.05 * sin(0.01 * time + vScreenCoord.y);

to emulate scanlines seems to be too much (considering I want 60 instead of 40fps) for my iPad ^^

Noticed that ParticleContainer (performs great btw!) weirdly stretches animated sprites when using a spritesheet with trimmed whitespace around the individual frames. There's no issues with non-trimmed spritesheets. Is that simply a general restriction?

Not really an intentional one, we probably just forgot to implement support :P

Well then, will just use non-trimmed spritesheets for now :)
New hotness:

Not sure if

a) what I am doing here simply is super optimized already so that bufferSubData became the heaviest call
b) putting lots of moving, rotating and fading stuff into a ParticleContainer is stupid or
c) that's just something that needs to be optimized ^^

Yeah tuning the size to be closer to your batch sizing will yield much better performance since we don't have to do expensive sub arrays and such.
Glad it is working out!

Next part of dc's adventures in ParticleContainer wonderland: When generating mipmaps of my spritesheet, and I scale a sprite (like a healthbar) on the X axis only, it picks the corresponding mipmap level, even if scale on Y axis is still 100%. Basically it's: The lower the health, the more awkward the sprite. Currently trying to figure out how to prevent this - any advise? ^^

Alright, I am cropping the image now. This is a bit awkward, though, and requires all textures to be unique, which sums up quite a bit for healthbars:
this._texture.crop.width = newWidth
this._texture._updateUvs() // which is "protected" and requires touching pixi.js.d.ts

I could also assign a new frame of course, but this goes through that many ifs that I'd rather avoid it. Having crop functionality on sprites would be ideal.

So, I am trying to crop the frame of a texture, represented by a Sprite within a ParticleContainer, from frame to frame.
What's happening is this: https://jsfiddle.net/79zgL7gr/2/
Not sure if I am doing something wrong here, but what's rendered is obviously wrong. The expected picture is a 10 px height sprite on the left side, and a 40 px (full sized) sprite on the right.
If I comment out the contents of the else-clause, the the left sprite is in fact 10 px in height, but if it is updated dynamically, it's also 40 px.
Any hints how to do this right? Tried assigning frames etc., but nothing seems to work.

Found a workaround: https://jsfiddle.net/79zgL7gr/6/
Calling addChild/removeChild for a dummy sprite each frame seems to make it work. This an issue?

You changed height (scale) but didn't tell the particle container that scale is dynamic.

Hmm, thanks! Really thought this is not scale. Hope I turned not too annoying already, seems that I still have to learn a lot of stuff :)

Anytime, not annoying at all :)

closing as looks resolved to me :+1:


Add an iterations marker and CLI option

It'd be really helpful if we could add an iterations marker and CLI option, so that the tests can be rerun multiple times.
I can work on this.

@andfoy Can we do a release with this? I can take care of it if you want.

Sure! I can do it in a few hours time, but if you want to go ahead, no problem at all!

Alright, let's make a decision on #12 first and I'll start the release process right after.


How to use/acess?

Hi Daandelange, I’m quite interested by your plugin, but I’m a real n00b!
I followed the Installation method (copy to /site/plugins/simplestats), but now I’m stuck. How do I access the plugin? Do I need to go to my Kirby panel?

Hello, your plugin should be up & running.

Any issues should be reported to site/logs/simplestats_errors.txt.
The panel is for viewing the collected stats only. After 24h there should be something visible. You can also view the raw database file with an SQL viewer (site/logs/simplestats.sqlite).
You can customise your setup up in site/config/config.php like any other kirby plugin. (available options to override are listed in options.php)

Hello, your plugin should be up & running.

Any issues should be reported to site/logs/simplestats_errors.txt.
The panel is for viewing the collected stats only. After 24h there should be something visible. You can also view the raw database file with an SQL viewer (site/logs/simplestats.sqlite).
You can customise your setup up in site/config/config.php like any other kirby plugin. (available options to override are listed in options.php)

Hi Daandelange, thanks for the quick support! It’s still a bit fuzzy for me, but I’ll wait 24h and see if something’s pops up.
Right now, I only have the plugin on my local installation (laravel valet). Is this a problem?

I don’t currently see the log folder.

And in my panel, it is unclear where I should go to see the stats 🤔

Hi Daandelange, thanks for the quick support! It’s still a bit fuzzy for me, but I’ll wait 24h and see if something’s pops up.
Right now, I only have the plugin on my local installation (laravel valet). Is this a problem?

I don’t currently see the log folder.

And in my panel, it is unclear where I should go to see the stats 🤔

It should be in your site's main menu. If not, there might be something wrong...

It should be in your site's main menu. If not, there might be something wrong...

Oh! This is so easy! 🤦♂
I have a new error message now 😬

Oh! This is so easy! 🤦♂
I have a new error message now 😬

And on my remote install, this is the error message Class 'SQLite3' not found. 😔

And on my remote install, this is the error message Class 'SQLite3' not found. 😔

For the Database issue, I'll try to reproduce it. Do your dev tools' network monitor reveal something more useful ? Is there something in simplestats_errors.txt ?
The 2nd one speaks for itself, you don't have SQLite3 enabled on your server. (worth looking if you can enable it trough php.ini or your hoster's admin panel)

For the Database issue, I'll try to reproduce it. Do your dev tools' network monitor reveal something more useful ? Is there something in simplestats_errors.txt ?
The 2nd one speaks for itself, you don't have SQLite3 enabled on your server. (worth looking if you can enable it trough php.ini or your hoster's admin panel)

There were some files missing for being plug-and play. Can you test with the latest version ?

There were some files missing for being plug-and play. Can you test with the latest version ?

Hey Daandelange, I reinstalled with your new version.
Will look into activating 'QLite3 on the remote one.
Same error message on the local. Dev console says [Error] Failed to load resource: the server responded with a status of 500 (Internal Server Error) (pagestats, line 0)

Hey Daandelange, I reinstalled with your new version.
Will look into activating 'QLite3 on the remote one.
Same error message on the local. Dev console says [Error] Failed to load resource: the server responded with a status of 500 (Internal Server Error) (pagestats, line 0)

I've tested the plugin on a fresh install, the latest version works on my side, but that's only 1 possible configuration.
If you're familiar with php code, you can enable Kirby debugging and trace back errors that don't appear in site/logs/simplestats_errors.txt.

I've tested the plugin on a fresh install, the latest version works on my side, but that's only 1 possible configuration.
If you're familiar with php code, you can enable Kirby debugging and trace back errors that don't appear in site/logs/simplestats_errors.txt.

Hey there, I tried on a fresh install as well. Same error Unable to open database: unable to open database file.
And on both install, even with Kirby debugging, the logs folder is missing.
If it might help, I’m running my local version through the latest version of Valet. Don’t know if it’s relevant.

Hey there, I tried on a fresh install as well. Same error Unable to open database: unable to open database file.
And on both install, even with Kirby debugging, the logs folder is missing.
If it might help, I’m running my local version through the latest version of Valet. Don’t know if it’s relevant.

Did you try creating the logs folder ? (after which the db and log file should appear on visiting your website ?)
If not, the plugin is probably not able to load itself correctly in Kirby, maybe it's something to do with Valet.

Did you try creating the logs folder ? (after which the db and log file should appear on visiting your website ?)
If not, the plugin is probably not able to load itself correctly in Kirby, maybe it's something to do with Valet.

Bingo, got it. Now works on my local installation :)
There’s no stats, but I guess this is because it’s local, duh.
Got some error thought. I’ll let you know how it goes on my remote install as soon as I got SQLite on it.
simplestats_errors.txt

Bingo, got it. Now works on my local installation :)
There’s no stats, but I guess this is because it’s local, duh.
Got some error thought. I’ll let you know how it goes on my remote install as soon as I got SQLite on it.
simplestats_errors.txt

Hum, on dubious. On the remote, my hosting companies informed me SQlite 3 is in place, and the Terminal agrees.

Hum, on dubious. On the remote, my hosting companies informed me SQlite 3 is in place, and the Terminal agrees.

The error in your logfile has just been fixed. (update simplestats)
As for SQLite3... that screenshot looks like the unix command line binary, what you need is the PHP Module.
You could try removing the \ here.
Otherwise, as a temporary solution, upload the locally generated database to your online server, to bypass the SQLite3 requirement.

The error in your logfile has just been fixed. (update simplestats)
As for SQLite3... that screenshot looks like the unix command line binary, what you need is the PHP Module.
You could try removing the \ here.
Otherwise, as a temporary solution, upload the locally generated database to your online server, to bypass the SQLite3 requirement.

With the new version and the upload of the simplestats.sqlite file, I have a new error! could not find driver

With the new version and the upload of the simplestats.sqlite file, I have a new error! could not find driver

That seems to be related to Kirby's PDO/SQLITE driver used by this plugin.
Maybe this thread can give you some directions. (look at phpinfo() if the PHP PDO SQLite3 module is enabled, and try to install that composer dependency)

That seems to be related to Kirby's PDO/SQLITE driver used by this plugin.
Maybe this thread can give you some directions. (look at phpinfo() if the PHP PDO SQLite3 module is enabled, and try to install that composer dependency)

I’ll need a bit more help to go through these steps; it is quite confusing for me.

Add "ext-pdo_sqlite": "*" to my composer.json in the require block

We’re talking about the composer.json file located on the kirby folder, right?

Run composer update to update the composer.lock file

I don’t know how to do that

Deploy
It’s already deployed if it’s on my remote (FTP) server, right?

I’ll need a bit more help to go through these steps; it is quite confusing for me.

Add "ext-pdo_sqlite": "*" to my composer.json in the require block

We’re talking about the composer.json file located on the kirby folder, right?

Run composer update to update the composer.lock file

I don’t know how to do that

Deploy
It’s already deployed if it’s on my remote (FTP) server, right?

No, the one in the plugin folder.
When you then run the composer update, it will update the plugin filles and make the necessary changes to integrate that extra dependency.
So you might need to re-upload the plugin files after that. Refer to the Kirby Composer Guide for more detailed instructions.

No, the one in the plugin folder.
When you then run the composer update, it will update the plugin filles and make the necessary changes to integrate that extra dependency.
So you might need to re-upload the plugin files after that. Refer to the Kirby Composer Guide for more detailed instructions.

Got it, thanks for the steps. Same error on my remote install :/

Got it, thanks for the steps. Same error on my remote install :/

I've just updated the plugin, including a draft of a requirements checker in the Information tab (panel).
Could you check if all requirements are there ? Hopefully that gives a better insight.

I've just updated the plugin, including a draft of a requirements checker in the Information tab (panel).
Could you check if all requirements are there ? Hopefully that gives a better insight.

Works great on the local instal. On the remote one, it shows nothing :(

Works great on the local instal. On the remote one, it shows nothing :(

Does it show anything useful on website.domain/api/simplestats/listdbinfo?csrf=YOURTOKEN ? (you need to set the CSRF in your config.php : 'api.csrf' => 'YOURTOKEN',)

Does it show anything useful on website.domain/api/simplestats/listdbinfo?csrf=YOURTOKEN ? (you need to set the CSRF in your config.php : 'api.csrf' => 'YOURTOKEN',)

{"status":"error","message":"could not find driver","code":500,"key":null,"details":[]}

{"status":"error","message":"could not find driver","code":500,"key":null,"details":[]}

Hmm, ok, probably the Kirby\Database\Database class that doesn't work in your configuration.
I've made the requirements check "autonomous", so it should work now. Please check the message in the panel again, or website.domain/api/simplestats/checkrequirements?csrf=YOURTOKEN.

Hmm, ok, probably the Kirby\Database\Database class that doesn't work in your configuration.
I've made the requirements check "autonomous", so it should work now. Please check the message in the panel again, or website.domain/api/simplestats/checkrequirements?csrf=YOURTOKEN.

New error. the pop out when I open the page in the panel says
Return value of daandelange\SimpleStats\SimpleStatsDb::database() must be an instance of Kirby\Database\Database, null returned
Database Requirements says
PHP Extensions=Core, date, libxml, openssl, pcre, zlib, filter, hash, pcntl, readline, Reflection, SPL, session, standard, cgi-fcgi, bcmath, bz2, calendar, ctype, curl, dba, dom, mbstring, ftp, gd, gettext, iconv, imap, intl, json, ldap, exif, mysqlnd, odbc, PDO, pgsql, Phar, posix, shmop, SimpleXML, soap, sockets, sysvmsg, sysvsem, sysvshm, tidy, tokenizer, xml, xmlwriter, xsl, zip, mysqli, pdo_mysql, PDO_ODBC, pdo_pgsql, xmlreader, xmlrpc --- PHP=OK --- SQLite3=ERROR --- SQLite3.try=ERROR Class 'SQLite3' not found --- CreateDB()=ERROR:could not find driver --- pdo_sqlite=ERROR --- sqlite3=ERROR
And the url https://www.jolinmasson.com/api/simplestats/checkrequirements?csrf=COUCOU  says
{"dbRequirements":"PHP Extensions=Core, date, libxml, openssl, pcre, zlib, filter, hash, pcntl, readline, Reflection, SPL, session, standard, cgi-fcgi, bcmath, bz2, calendar, ctype, curl, dba, dom, mbstring, ftp, gd, gettext, iconv, imap, intl, json, ldap, exif, mysqlnd, odbc, PDO, pgsql, Phar, posix, shmop, SimpleXML, soap, sockets, sysvmsg, sysvsem, sysvshm, tidy, tokenizer, xml, xmlwriter, xsl, zip, mysqli, pdo_mysql, PDO_ODBC, pdo_pgsql, xmlreader, xmlrpc --- PHP=OK --- SQLite3=ERROR --- SQLite3.try=ERROR Class 'SQLite3' not found --- CreateDB()=ERROR:could not find driver --- pdo_sqlite=ERROR --- sqlite3=ERROR"}

New error. the pop out when I open the page in the panel says
Return value of daandelange\SimpleStats\SimpleStatsDb::database() must be an instance of Kirby\Database\Database, null returned
Database Requirements says
PHP Extensions=Core, date, libxml, openssl, pcre, zlib, filter, hash, pcntl, readline, Reflection, SPL, session, standard, cgi-fcgi, bcmath, bz2, calendar, ctype, curl, dba, dom, mbstring, ftp, gd, gettext, iconv, imap, intl, json, ldap, exif, mysqlnd, odbc, PDO, pgsql, Phar, posix, shmop, SimpleXML, soap, sockets, sysvmsg, sysvsem, sysvshm, tidy, tokenizer, xml, xmlwriter, xsl, zip, mysqli, pdo_mysql, PDO_ODBC, pdo_pgsql, xmlreader, xmlrpc --- PHP=OK --- SQLite3=ERROR --- SQLite3.try=ERROR Class 'SQLite3' not found --- CreateDB()=ERROR:could not find driver --- pdo_sqlite=ERROR --- sqlite3=ERROR
And the url https://www.jolinmasson.com/api/simplestats/checkrequirements?csrf=COUCOU  says
{"dbRequirements":"PHP Extensions=Core, date, libxml, openssl, pcre, zlib, filter, hash, pcntl, readline, Reflection, SPL, session, standard, cgi-fcgi, bcmath, bz2, calendar, ctype, curl, dba, dom, mbstring, ftp, gd, gettext, iconv, imap, intl, json, ldap, exif, mysqlnd, odbc, PDO, pgsql, Phar, posix, shmop, SimpleXML, soap, sockets, sysvmsg, sysvsem, sysvshm, tidy, tokenizer, xml, xmlwriter, xsl, zip, mysqli, pdo_mysql, PDO_ODBC, pdo_pgsql, xmlreader, xmlrpc --- PHP=OK --- SQLite3=ERROR --- SQLite3.try=ERROR Class 'SQLite3' not found --- CreateDB()=ERROR:could not find driver --- pdo_sqlite=ERROR --- sqlite3=ERROR"}

Ok, thanks, so that clarifies a lot. They say the same thing : Your php extensions sqlite3 (for SQLite3) and pdo_sqlite (used by Kirby\Database\Database) are not available. And they are at the core of this plugin. IMO, the easiest solution would be porting SimpleStats to optionally use MySQL databases (which seem to be enabled on your host) instead of SQLite3; but that is out of my scope... but I can give some directions for such changes if you wish.
Of course, a very simple solution would be to ask your hoster to enable them, or enable them yourself if you have access to custom php.ini or via your hoster's site admin panel. Or change your hoster.

Ok, thanks, so that clarifies a lot. They say the same thing : Your php extensions sqlite3 (for SQLite3) and pdo_sqlite (used by Kirby\Database\Database) are not available. And they are at the core of this plugin. IMO, the easiest solution would be porting SimpleStats to optionally use MySQL databases (which seem to be enabled on your host) instead of SQLite3; but that is out of my scope... but I can give some directions for such changes if you wish.
Of course, a very simple solution would be to ask your hoster to enable them, or enable them yourself if you have access to custom php.ini or via your hoster's site admin panel. Or change your hoster.

I indeed ran the command in ter Remote terminal, and it does seams to be installed.

My domain is a secondary domain on my host. Could that be what causes the problem?

I indeed ran the command in ter Remote terminal, and it does seams to be installed.

My domain is a secondary domain on my host. Could that be what causes the problem?

Hello, yes, the command line sqlite3 is installed, but the PHP module with the same name is probably not, as it's not listed in your "PHP extensions", neither is "pdo_sqlite". You need to enable the PHP module.
This one --> https://www.php.net/manual/en/class.sqlite3.php

Hello, yes, the command line sqlite3 is installed, but the PHP module with the same name is probably not, as it's not listed in your "PHP extensions", neither is "pdo_sqlite". You need to enable the PHP module.
This one --> https://www.php.net/manual/en/class.sqlite3.php

They say it’s already installed 😑.

They say it’s already installed 😑.

Well, then something weird is happening on your host, it's not enabled on your Kirby installation.
In the panel's simplestats requirement check, the extensions that your Kirby php installation can use are listed after PHP Extensions=**your**extensions**, so then you might need to compare the output of php -m and the list in the panel. If they differ, your server setup is using a different PHP environment then the one in the console. Try creating an info.php (with <?php phpinfo(INFO_MODULES); ?>) file in your Kirby root and compare the extensions listed there.

Well, then something weird is happening on your host, it's not enabled on your Kirby installation.
In the panel's simplestats requirement check, the extensions that your Kirby php installation can use are listed after PHP Extensions=**your**extensions**, so then you might need to compare the output of php -m and the list in the panel. If they differ, your server setup is using a different PHP environment then the one in the console. Try creating an info.php (with <?php phpinfo(INFO_MODULES); ?>) file in your Kirby root and compare the extensions listed there.

Here’s my info.php
I tried to 'activate' sqlite by adding these three lines at the end of my php.ini, but it made my site crash (I have not a lot of clue about what I’m doing actually 😑)
.
extension=curl extension=pdo_sqlite extension=sqlite3
The output of the console and the site don’t match :/
SimpleStats output.txt
Terminal server.txt

Here’s my info.php
I tried to 'activate' sqlite by adding these three lines at the end of my php.ini, but it made my site crash (I have not a lot of clue about what I’m doing actually 😑)
.
extension=curl extension=pdo_sqlite extension=sqlite3
The output of the console and the site don’t match :/
SimpleStats output.txt
Terminal server.txt

Yes, that's the regular way to go. A crash means it doesn't accept your new lines it's very strict. Sometimes you need to add .so (or .dll on WIN). Try them one by one until it works, search online, or ask your hoster for support on how to enable it, read their FAQ.

Yes, that's the regular way to go. A crash means it doesn't accept your new lines it's very strict. Sometimes you need to add .so (or .dll on WIN). Try them one by one until it works, search online, or ask your hoster for support on how to enable it, read their FAQ.

Ooooof, I asked to activate the extension… so they switched my PHP to 5.6, breaking my site in the process. 😑 I asked a second time, and they reverted to 7.3 and activated the extension. It now works like a charm!
Geez, that was not so complicated, and entirely on their end. Thanks a lot Daandelange for the support during all this process!

You're welcome, I'm happy it works now. :)

Hey Maandenlange, I think I’m cursed. I did a clean install of the plugin + database last weekend, now that I have the requirements.
I used a VPN to visit my website, see if it worked, and 48 hours, I still have no stats 😭.
Any clue?

Hey Jolin, I guess that you're not connected to the panel using your VPN ?
Is the logfile empty ? Did you try deleting the db file and let it create a new one ? (looks like it's already the case, but you never know...)
Don't bother waiting 24/48H : If there's nothing in this view ( cf: No rows available), nothing will appear later.
Did you try if referrer tracking works (instantly appears) ?
Any other clue... try disabling the Kirby cache or other settings that could affect tracking ? (Try a clean plainkit/starterkit on your configuration?)


Feature request: uv wrappers for dup() and dup2() primitives

libuv should expose wrappers for the dup() and dup2() system calls.
See https://github.com/nodejs/node/issues/41733 for rationale.

I have these (in an old PR here), and been meaning to get around to this for years. You want F_DUPFD specifically though here.

I've read through nodejs/node#41733 but I don't understand why dup2() is needed for stdio. It's not going to solve the problem of intermingled logging.

Closing, no clear use case and no follow-up.


chore: add reusable include for extension guides

Notes for Reviewers
This PR adds a reusable include to be used from guides pages

reduces repeated markup and style

Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 801a24689c65f76b9b77b1c5b237a8d48630b477 at: https://66c485143014ec04e3a1d154--meshery-docs-preview.netlify.app


bioconductor packages:  3.8 -> 3.10

Motivation for this change
Bioconductor packages are out of date by over a year. Want to build bioinformatics environments without relying on conda.
Things done

followed instructions from the nix documentation.
updated the *-packages.nix files in r-modules/ Added 27 workflows packages from bioconductor and updated the machinery to track them.
ran nix-build test-evaluation.nix --dry-run. Succeeds on linux, fails on darwin.
Edited many of the customizations in r-modules/default.nix.
Removed defunct packages.
Added a test suite that I ran manually in r-modules/test-package-build.nix Tested an environment that uses rstudio and a number of packages. The tested packages all build, except for a few that are now marked as broken.

This PR probably supersedes #81307, except it does not  bump R from 3.6.3 to 3.6.3

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Superseded by PR #85522


Can't get websocket IP address through amazon ELB

Expected behavior
Hi, I use netty for websocket, and I want to get IP address from client.
Actual behavior
When I use ELB load balancer, the only IP that I can get is IP of ELB machine, how can I get real ip from client ?
I saw that someone use this, but  I can only get WebSocketFrame object, can I get header of Http Websocket Upgrade request ?
  @Override     public void channelRead0(ChannelHandlerContext ctx, HttpObject msg)             throws Exception {         if (msg instanceof HttpRequest) {             HttpRequest mReq = (HttpRequest) msg;             String clientIP = mReq.headers().get("X-Forwarded-For");             if (clientIP == null) {                 InetSocketAddress insocket = (InetSocketAddress) ctx.channel()                         .remoteAddress();                 clientIP = insocket.getAddress().getHostAddress();             }         }     }
Steps to reproduce
Minimal yet complete reproducer code (or URL to code)
Netty version
4.1.22_final
JVM version (e.g. java -version)
jdk 1.8.0_161
OS version (e.g. uname -a)
Amazon Linux EC2

You will need to put the handler in front of your upgrade handler. Please show your pipeline confidence.g

@thiennguyen1308 I think your balancer should add 'X-Forwarded-For' header and then you can get real ip in your own handler before websocket upgrade.

@normanmaurer Thank for your instruction, I have already get client IP now. 👍

@amizurov  Yup, it's enabled by default.


Mobx leaks memory on server side

Current example of mobx observer usage leaks memory on server side, https://github.com/mobxjs/mobx-react/issues/140
In order to fix this server side render needs to configure / use the static rendering
This PR fixes it.

Thanks @ptomasroos


JMS source session not transacted so that client acking works.

Previously the JMS Source was set to use a transacted session which caused the acknowledge mode to be ignored.
Fixes #226

thank you!


Service Problem

Hi,
I have installed the .deb File and follow the systemctl descrition.
Now I become these errors:
~/.config$ systemctl --user start hacompanion
Job for hacompanion.service failed because of unavailable resources or another system error.
See "systemctl --user status hacompanion.service" and "journalctl --user -xeu hacompanion.service" for details.
~/.config$ journalctl --user -xlf -u hacompanion
Dez 16 22:07:56 matzepc-mint systemd[1543]: hacompanion.service: Scheduled restart job, restart counter is at 44.
░░ Subject: Automatic restarting of a unit has been scheduled
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ Automatic restarting of the unit UNIT has been scheduled, as the result for
░░ the configured Restart= setting for the unit.
Dez 16 22:07:56 matzepc-mint systemd[1543]: hacompanion.service: Failed to load environment files: No such file or directory
Dez 16 22:07:56 matzepc-mint systemd[1543]: hacompanion.service: Failed to spawn 'start' task: No such file or directory
Dez 16 22:07:56 matzepc-mint systemd[1543]: hacompanion.service: Failed with result 'resources'.
░░ Subject: Unit failed
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ The unit UNIT has entered the 'failed' state with result 'resources'.
Dez 16 22:07:56 matzepc-mint systemd[1543]: Failed to start hacompanion.service - Home Assistant Desktop Companion.
░░ Subject: A start job for unit UNIT has failed
░░ Defined-By: systemd
░░ Support: http://www.ubuntu.com/support
░░
░░ A start job for unit UNIT has finished with a failure.
░░
░░ The job identifier is 1607 and the job result is failed.
HANDMADE: --> runs
~/.config$ hacompanion
2024/12/16 22:21:24 Starting Desktop Companion version 1.0.15

now it runs with little help from ChatGPT


Bug: The subtile can't render when using LIVE HLS.

#32

input_looped_file_config.yaml

# List of inputs.
inputs:
    # The type of input.
  - input_type: looped_file
    # Name of the input file.
    # This example can be downloaded from https://storage.googleapis.com/shaka-streamer-assets/sample-inputs/Sintel.2010.720p.mkv
    name: Sintel.2010.720p.mkv
    # The track number.
    track_num: 0
    # The media type of the input. Can be audio or video.
    media_type: video
    # Frame rate in seconds.
    frame_rate: 24.0
    # Resolution of the input.
    resolution: 720p

    # The type of input.
  - input_type: looped_file
    # A second track (audio) from the same input file.
    name: Sintel.2010.720p.mkv
    track_num: 1
    media_type: audio

    # The type of input.
  - input_type: looped_file
    # https://storage.googleapis.com/shaka-streamer-assets/sample-inputs/Sintel.2010.Chinese.vtt
    name: Sintel.2010.Chinese.vtt
    media_type: text
    track_num: 2
    language: zh

pipeline_live_config.yaml

# Streaming mode.  Can be live or vod.
streaming_mode: live

# A list of resolutions to encode.
resolutions:
  - 720p
  - 480p

# The number of audio channels to output.
channels: 2

# The codecs to encode with.
audio_codecs:
  - aac
video_codecs:
  - h264

# Manifest format (dash, hls, or both)
manifest_format:
  # - dash
  - hls

# Length of each segment in seconds.
segment_size: 4

# Availability window, or the number of seconds a segment remains available.
availability_window: 300

# Presentation delay, or how far back from the edge the player should be.
presentation_delay: 30

# Update period, or how often the player should fetch a new manifest.
update_period: 8

I found a couple of issues with this in the latest code.
First, we just introduced auto-detection of several input features and changed the definition of track_num in an incompatible way.  To run this on the latest version, you'll need to update your input config to drop the "track_num" field from all inputs.  You can also simplify and drop "frame_rate", and "resolution" fields from the video input.
Second, this triggers a bug that we introduced with the custom bitrates feature.  I'll have a fix for that soon.

Finally, with those issues fixed, the remaining issue is that we don't have a way to loop subtitles at the moment.  Subtitle inputs bypass the transcoder and go straight to the packager.
I just checked to make sure, and I can confirm that ffmpeg doesn't support looping subtitle inputs.
The subtitles are being packaged into the output, though.

We will add a restriction so that media_type: text combined with input_type: looped_file will be rejected with a clear error message.  We will also update the docs to reflect this.  The changes will be released in v0.3.0.

Changes have been made to clarify the docs and reject these unsupported combinations of options.  Please let us know if you have any further issues.  Thanks!


ttf or otf export

Is it possible to convert the output files to ttf or otf?

https://github.com/SilenceLeo/snowb-bmf/issues/61
I think this tool is what you want.
https://www.fontsquirrel.com/tools/webfont-generator

https://github.com/SilenceLeo/snowb-bmf/discussions/60
I think this tool is what you want.
https://www.fontsquirrel.com/tools/webfont-generator


Inherit from sklearn.base

To make this maximally compatible with scikit-learn, we should have the estimators inherit from sklearn.base, and then run sklearn's check_estimator on each as part of the test suite.
The only thing to keep in mind is that sklearn estimators can't do anything in __init__ except store the arguments by name (this is to enable cloning the estimator for grid search & parallelization). It seems like we're already in good shape as far as that requirement.

To Do: add check_estimator to embedding testing. Make sure it works.

I can take a look at that now

check_estimator is failing for a number of reasons: mainly our decision not to have a default value for the radius, and also with some issues in scikit-learn itself (e.g. https://github.com/scikit-learn/scikit-learn/pull/6510). We'll leave this as a "will not fix" for the time being.


Move GraphViz to its own plugin and do not include by default

GraphViz has proved a hindrance when it comes to adoption and using phpDocumentor. By moving it to a separate plugin that is not enabled by default we hope to gain the following benefits:

Increase ease of use after installation
Reduce number of required dependencies

Because this is a BC breaking change we intend to do this when moving to version 3.0

Closing for now

@mvriel GraphViz website is down. How can I install it?


[proposal] report sys usage in NodeMetric

What is your proposal:
Report sys usage in NodeMetric.
Why is this needed:
For example, the descheduler plugin LowNodeLoad uses nodeMetric.Status.NodeMetric.NodeUsage to determine whether the load of the node exceeds the high threshold. nodeMetric.Status.NodeMetric.NodeUsage does not change immediately after some pods are evicted by the plugin, because by default nodeUsage is an average over 5 minutes. NodeUsage in NodeMetric takes a while to update to the current latest actual utilization. This may cause LowNodeLoad to incorrectly evict some Pods.
There are several ways to alleviate the problem:

One solution is to reduce the aggregation calculation period of NodeMetric, for example, to 1 minute. However, this method may cause the total amount of resources that can be oversold to fluctuate.
Another solution is to adjust the run cycle of descheduler, which can alleviate some problems.
The third way is to add a sysUsage field, and then calculate nodeUsage = ∑usage(existing Pod) + sysUsage as needed.

Is there a suggested solution, if so, please add it:

@zwzhang0107

/assign


The specific TuneIn podcast playback is too slow to fail AQT Music performance

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

When playing or resume the program "Joe Rogan Experience" on TuneIn podcast, it needs the time to wait TuneIn to play in some time.
The time exceeds 13 seconds and above which not match AQT Music expected.
(Echo dot doesn't have this problem)
What is the expected behavior?

TuneIn should be played less then 13 seconds.
What behavior are you observing?

It needs to wait exceeds to 13 seconds to play some times.
Provide the steps to reproduce the issue, if applicable:

Alexa, play the program Joe Rogan Experience on TuneIn.
(Alexa will response playing the program Joe Rogan Experience on Tune. Resuming....)
(wait TuneIn play)

Log:
play_tunein_podcast_debug9.zip
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <1.19.1>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[*] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[*] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

Hi @yuan0258 , this is an issue with TuneIn podcasts as they are downloaded from 0:00 and must buffer all the data up until the desired timestamp.
Some ways to ensure this test proceeds smoothly is to make sure you have a fast and stable internet connection to your device.
Another step to do is to make sure that AQT testing is done from the start of the podcast. You can do this by running the steps before proceeding with AQT testing
‘Alexa, play the Joe Rogan Experience’
(Alexa announces she is resuming the Joe Rogan experience, then the podcast starts.)
‘Alexa, restart’
(Alexa announces the start of the podcast, which starts)
‘Alexa, stop’
For more help please contact your amazon representative

Hi @womw , I follow your steps to setup the position of podcast before AQT TuneIn Podcast testing. There are two podcast "Joe Rogan Experience" and "Up First". It is improve for first time playing but still failed in the after.
I checked the report. It is failed for delayed when playing "Joe Rogan Experience" in the third time(No.27) due to timestamp.
(RaspberryPi)

I uses Echo dot to test same test case. it doesn't have the delayed problem. Does is it use difference way to download TuneIn podcast?
(Echo dot)

I will also provide this information to our amazon representative.

Hi @womw , Is it possible to use "Range" header to speed up the content downloading?
I found gstreamer will return the offset bytes of seeking when resuming TuneIn podcast. (ex: AttachmentReaderSource:handleSeekData:offset=124415584)
And I check podcast response header, it is accept "Range" header. So, I think SDK should able to set "Range: bytes" header for content fetch to speed up when seeking. But UrlContentToAttachmentConverter can not update offset during content fetch.
If I want to use "Range" header in existing HTTP content fetch, I needs to modify flow of MediaPlayer and UrlContentToAttachmentConverter to support stop current and create new one with "Range" header.
Or maybe it has better way to implement.

Hi @oscar1372 , unfortunately what you propose isn't possible in the SDK right now. There's no directive to "resume", but rather the device would just get another PLAY directive with an offset, so we'd also need to put in some effort to see if it's even possible to use the last byte offset.
Implementing something like what you suggest constitutes a feature request, so I'll label this issue as such. I suggest you reach out to your Amazon business associate to continue the conversation if you'd like this feature request to be prioritized.

Hi @oscar1372 , unfortunately what you propose isn't possible in the SDK right now. There's no directive to "resume", but rather the device would just get another PLAY directive with an offset, so we'd also need to put in some effort to see if it's even possible to use the last byte offset.
Implementing something like what you suggest constitutes a feature request, so I'll label this issue as such. I suggest you reach out to your Amazon business associate to continue the conversation if you'd like this feature request to be prioritized.

I'm going to close this due to inactivity. Please open a new issue or get in touch with your Amazon business contact if you want to ask for this feature to be prioritized.


bug: ClusterAuthenticationTrustController ConfigMap listing error - "invalid context cluster value: cluster name is empty"

Describe the bug
On startup and intermittently afterwards, I'm seeing:
E0928 12:35:49.469806 1864903 store.go:247] invalid context cluster value: cluster name is empty in the request context - RequestInfo: &request.RequestInfo{IsResourceRequest:true, Path:"/api/v1/namespaces/kube-system/configmaps", Verb:"list", APIPrefix:"api", APIGroup:"", APIVersion:"v1", Namespace:"kube-system", Resource:"configmaps", Subresource:"", Name:"", Parts:[]string{"configmaps"}}

At startup, it occurs just after:
I0928 12:35:49.464212 1864903 cluster_authentication_trust_controller.go:440] Starting cluster_authentication_trust_controller controller
I0928 12:35:49.464233 1864903 shared_informer.go:302] Waiting for caches to sync for cluster_authentication_trust_controller

And sure enough this controller is listing configmaps
Steps To Reproduce
...
Expected Behaviour
...
Additional Context
No response

We need to fix this. We only need 1 instance of the CA trust configmap. Should put it in root and then have docs/library code so webhook server authors know where to get it. Also make sure RBAC is in place.

xref #1807 - sort of / maybe a duplicate

This has been fixed


Add python 3.5 to contributing guide

The tox environment specifies that we should test 3.5, so this guide should also say the same.

Makes sense, thanks @kragniz !

:sparkles:


Run CI tests on range of Go versions

Add matrix of Go versions we should support.
Rename go.yml to test.yml (more descriptive).
Add push.branches to lint.yml (not actually sure if this is needed).
Remove quotes around "master" and remove brackets.

One minute. Something's not quite right.

Okay cool, should be good now 👍


BackColor around text

I can set the back color of the scintilla control like this:
scintillaNet1.Styles[Style.Default].BackColor = Color.FromArgb(234, 242, 251);
this looks fine when there is no text in the control, but once I start typing text, the back color around the text is gray.
How can I have the back color remain consistent, even when text is typed?

Make sure the style you're typing in also has the same background color. Chances are you're typing in style 0 (the default). Try:
scintillaNet1.Styles[Style.Default].BackColor = Color.FromArgb(234, 242, 251);
scintillaNet1.Styles[0].BackColor = Color.FromArgb(234, 242, 251);

thank you very much Jacob!


Package issue in  pubsec.yaml

Package  package_info 2.0.0 requires SDK version >=2.12.0-259.9.beta <3.0.0 and no versions of package_info match >2.0.0 <3.0.0, package_info ^2.0.0 is forbidden.
So, because doclense depends on package_info ^2.0.0, version solving failed.

Can I solve this issue?
#gssoc21

Be careful while implementing backwards compatibility. The current versions should remain unaffected by this.

okay


{AKS} Fix azdev style issues for aks-preview

This checklist is used to make sure that common guidelines for a pull request are followed.

Fix azdev style issues for aks-preview

Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS

Will break down the fix to several smaller PRs.

Fixed by separate PRs.


Exclude contact cards (and other non-editable blocks) from HIX

Short description

The contact cards introduced in #3169 have a severe negative impact on the HIX score. This PR excludes them from the HIX calculations.
Proposed changes

before sending text to Textlab, remove all divs with contenteditable="false".
this currently only affects contacts, but makes it simple to exclude other blocks in the future, should the need arise

Side effects

html.text_content() was previously only used to check for empty pages. For convenience, I have changed the code to send the result of this operation to textlab instead of the raw HTML - but I am uncertain if there has been a decision against this in the past when you added the code in question @david-venhoff - are my changes OK?

Resolved issues

Fixes: #3268

Pull Request Review Guidelines

@david-venhoff - are my changes OK?

I fear that this might change the hix scores of pages again. The last time we did this our service team had quite a nightmare dealing with municipalities that suddenly could not translate their pages anymore because the hix score was too low.
If we do this, we should probably at least do some tests that this change does not decrease the hix score in comparison to right now.


Add helmchart monitoring for harvester-installer

Problem:
Enhanced Monitoring.
Solution:
add Monitoring manifests and enable by default
Related Issue:
https://github.com/harvester/harvester/issues/576

close in favor of https://github.com/harvester/harvester-installer/pull/126


0.6.6 breaks compatibility pipping to formatter in CI

I am unsure if this issue makes more sense here or on install-action so please let me know if I should close this issue and reopen over there.
At the risk of this sounding like xkcd 1172, I was wondering if an option could be provided to disable the grouping behaviour added in #219. In our CI we use cargo-action-fmt to provide Github Action formatting for clippy. This worked seamlessly with taiki-e/install-action@cargo-hack, but since v0.6.6 piping to cargo-action-fmt now breaks the pipe. For now we're pinning to v0.6.5.
Here's the job without pinning to v0.6.5 if that helps at all.
    steps:
    - uses: actions/checkout@v4

    - name: Sccache Setup
      uses<EMAIL_ADDRESS>      with:
        version: "v0.5.3"

    - name: Install nightly
      uses: dtolnay/rust-toolchain@stable
      with:
        toolchain: nightly
        components: clippy, rustfmt

    - name: Install cargo-hack
      uses: taiki-e/install-action@cargo-hack

    - uses: olix0r/cargo-action-fmt/setup@v2

    - name: Check clippy
      run: cargo hack clippy --all-targets --feature-powerset --message-format=json -Zlints -- -Dwarnings -Adeprecated | cargo-action-fmt

    - name: Check rustfmt
      if: success() || failure()
      run: cargo fmt --all -- --check

If this is not something you wish to support or if there is an existing workaround that I am not aware of, please do let me know.

Thanks for the report and sorry for the breakage!
Log grouping can be opt-out using the --log-group=none option, as explained in the changelog and help message.
Ideally, I would like to handle this using tty detection, but GitHub Actions is incompatible with tty detection (https://github.com/actions/runner/issues/241).
A reasonable workaround here would be to avoid using log grouping if an option is passed in which stdout is assumed to be used, such as --message-format.

Ah, I completely missed that argument; my fault for not reading closely enough.
Thanks for the quick response, and sorry for the trouble!

Fixed in v0.6.8. I have implemented the one mentioned in the above comment.

A reasonable workaround here would be to avoid using log grouping if an option is passed in which stdout is assumed to be used, such as --message-format.


Analyzer doesn't detect deprecated variables

I would assume this should create an analysis warning.
@Deprecated('')
List<String> stringList;

void main() {
  print(stringList);
}

Works for me.

Seems to work in 1.19.0 🎆


Remove the "Apollo" word from the api

Hooks renamed:

useApolloClient -> useClient
useApolloQuery -> useQuery
useApolloMutation -> useMutation

Also, warning messages were created

This closes #6

Hi @nicolaslopezj. Thanks for the PR. I've got a few suggestions.

I'd stay with useApolloClient. useClient seems to be too generic for me, especially as it's used less often than other function. Also react-apollo uses Query and Mutation, but ApolloConsumer and ApolloProvider.
Could you use a package like warning instead of console.warning() directly? It will ease locating of usage of the deprecated methods if they were imported under different names.
Could you leave the deprecated methods in typescript definition (marked as deprecated). Adding something like this in index.d.ts seems works:

/**
 * @deprecated use useQuery instead
 */
export const useApolloQuery: typeof useQuery;
/**
 * @deprecated use useMutation instead
 */
export const useApolloMutation: typeof useMutation;

I think useClient is good, because it’s being imported from a package that has the Apollo work in it. The idea was to mimic the name of the high order component withClient provided by Apollo.
Could you do N3? I don’t know how to write typescript
Thank you!

I think useClient is good, because it’s being imported from a package that has the Apollo work in it. The idea was to mimic the name of the high order component withClient provided by Apollo.

The HOC is actually withApollo. :) I'd say that useApolloClient is better than useApollo because is more consistent with render props react-apollo API (Apollo is a part of render props components used for providing and consuming the client).

Could you do N3? I don’t know how to write typescript

Sure, but it's actually enough to put the snippet from my previous comment at the bottom of index.d.ts.

@nicolaslopezj
I published a version 0.1.3 with useQuery and useMutation hooks.


Hue and hoobs 4.0 Lights and Scenes not showing up

Issue
Hue Lights and Scenes are not showing up in Hoobs 4.0 or Homekit
Log Messages
"platform":·"Hue",
"name":·"Hue",
"brightnessAdjustment":·100,
"heartrate":·5,
"lowBattery":·25,
"timeout":·1,
"waitTimePut":·50,
"waitTimePutGroup":·1000,
"waitTimeResend":·300,
"waitTimeUpdate":·20,
"users":·{
"001788FFFE24F66F":·"iaIFvggDeOen36Cq8Z1migw6LaL0
},
"groups":·true,
"group0":·true,
"lights":·true,
·"hueDimmerRepeat":·false,
"hueMotionTemperatureHistory":·false,
"linkButton":·true,
"stealth":·false,
"sensors":·false,
"schedules":·false,
"scenesAsSwitch":·false,
"scenes":·true,
"rules":·false,
"rooms":·true,
"resource":·true
Debug Files

See https://github.com/ebaauw/homebridge-hue/wiki/Configuration:

You need to set nativeHomeKitLights to false if you want Homebridge Hue to expose lights already exposed by the Hue bridge;
You need to use a decent HomeKit app to interact with scenes, or set scenesAsSwitch to see them in the Home app.

Was able to get the lights working thank you. Still having issues with the senes showing up in the home app. It worked great with hoobs 3.3.5 no luck so far with hoobs 4.0.92.

Please attach the debug dump file, and capture and attach a debug log file, see FAQ.

Attached is my debugging log, I hope i did this correctly
Debug.txt

Could you also please attach the dump file, /var/lib/hoobs/huehoobsbridge/homebridge-hue.json.gz?

ok I think i have it now. good thing is im learning alot :)
Debug second time.txt
homebridge-hue.json.gz

You didn't set scenes in config.json.  Nor groups, nor group0, nor rooms.

it finally worked. at one point i had them all turned on expect rooms this was the missing piece. thank you so much for your help


Script failed with error: Error: The process '/usr/bin/bash' failed because one or more lines were written to the STDERR stream

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: AzureCLIV1
(Version: 1.164.0)
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines
Agent name: 'Hosted Agent'
Agent machine name: 'fv-az86-569'
Current agent version: '2.190.0'
Ubuntu
20.04.2
LTS
Environment: ubuntu-20.04
Version: 20210816.1
Included Software: https://github.com/actions/virtual-environments/blob/ubuntu20/20210816.1/im
Image Release: https://github.com/actions/virtual-environments/releases/tag/ubuntu20%2F2021081
<IP_ADDRESS>-master-20210802-2
Current image version: '20210816.1'
Agent running as: 'vsts'

Issue Description
Our pipelines started failing with the error: "Script failed with error: Error: The process '/usr/bin/bash' failed because one or more lines were written to the STDERR stream".
In the logs we see azure cli command started logging a Warning that was previously Info
We tried the same tasks/scripts on different machines and we pinpointed the issue on the latest image version of Ubuntu 20.04
Version: 20210816.1
Task logs
az aks get-credentials  --resource-group  --name  --overwrite-existing --admin
Output ubuntu-20.04 version 20210816.1
WARNING: Merged "" as current context in /home/vsts/.kube/config
Output ubuntu-20.04 version 20210810.1
Merged "" as current context in /home/vsts/.kube/config
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
##[error]Script failed with error: Error: The process '/usr/bin/bash' failed because one or more lines were written to the STDERR stream

Potential workarounds:

Configure pipeline task not to fail task on STDERR
Request an older version on Ubuntu
Demand capability for the agent with a specific image version -> i am not sure that this is possible

Seems to work fine if using:  az aks get-credentials --resource-group --name **--only-show-errors** --overwrite-existing --admin

I was experiencing this too and as @radulacatus said configuring the pipeline task to not fail on Standard Error fixed it for me.
My YAML pipeline step looked like this;
    - task: ShellScript@2
        inputs:
          scriptPath: '.run.sh'
          failOnStandardError
        displayName: 'Configure App environment variables'`

I'm having the sabe issue with this task

The failOnStandardError flag could result in false positive errors when there is actually no error in the pipeline's result.
As confirmed by @radulacatus  and @MFredX, setting the failOnStandardError flag to false could solve this issue.
Turn on:

Turn off:


Formatting and type hints

Use Black to format code, and use new style type hints (colon separated) to pass typing checks.

Coverage remained the same at 90.566% when pulling 53aecc2431e4164271a1e15815ae526a2cdf26be on Hellzed:format-type-hints into 2977814a20f5729fbe3fbaf97f437cc7a73878e9 on syrusakbary:master.

@syrusakbary Any news on this?