Partially apply Nullable state for KDsl in Provider

Fixes PARTIALLY #12388
Context
This PR partially fixes the issue linked above, specifically for the map case. I ran into a parsing bug in QDox trying to add @Nullable to the flatMap call as well, that I simply don't have the energy to investigate. I would really appreciate if someone from Gradle could assist in getting that working as well, possibly by replacing QDox since it seems unmaintained.
I did include JB annotations since the JSR305 ones cannot apply to TYPE_USE. I hope this is okay as it is the only proper solution.
I would still like to get the partial fix in if possible.
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[x] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

One downside I see with this is that after the change IntelliJ will offer up both @Nullables for auto-import.

Yes, I was not confident in doing a full move to JetBrains annotations as the initial spike, but if that would be potentially desirable to avoid that import issue, I can do that too.

ping for stale bot

I don't think introducing the double @Nullable annotation and the confusion it brings is worth it for just this use case. I can see a couple ways forward:

abandon this PR
figure out how to avoid the duplicate @Nullable annotation on the classpath. We could migrate completely to JetBrains annotations (and whether or not that would be good for us).

An option for the latter would be to somehow generate JSR305 annotations from JetBrains annotations during build. Just an idea.

Could we define our own Nullable annotation for that scenario?

@bamboo Do you mean to add something like a @NullableType that only applies to ElementType.TYPE_USE and add it to IntelliJ as one more nullable annotation?  I like it!
I think that might be the best course of action, as:

it allows us keep using @javax.annotation.Nullable in most places,
does not require massive changes throughout the codebase,
doesn't introduce a duplicate @Nullable annotation.

IIUC, we'd need to add it to .idea/misc.xml under NullableNotNullManager/myNullables. However, that file is not currently tracked in Git, though looking at the contents it probably should be.
If we add a @NullableType annotation, we should also add a matching @NonnullType annotation, and add that under NullableNotNullManager/myNotNulls in misc.xml as well.

The only problematic thing in misc.xml seems to be ProjectRootManager@project-jdk-name which I believe points to a machine-local JDK name... 🤔

Do you mean to add something like a @NullableType that only applies to ElementType.TYPE_USE and add it to IntelliJ as one more nullable annotation?

Exactly. We'd also need to configure the Kotlin compiler to recognize the annotation.

Seems like we have a plan. How do we get there?

I am not sure if introducing another annotation for something being nullable makes sense. Does this mean that build script editors also would need to add this annotation to their list of nullability annotations in IDEA? Or would that happen somewhat automatically? We could use this for stuff we have on our internal APIs, though I don't understand what the effect on the public API is and if there is some configuration necessary for the consumers of the public API (aka plugin and build authors).

We could use this for stuff we have on our internal APIs, though I don't understand what the effect on the public API is and if there is some configuration necessary for the consumers of the public API (aka plugin and build authors).

I'd do this for internal APIs only for now. I'm also wondering if there's any initiative to make JSR-305's @Nullable applicable to type parameters. 🤔 If that happens at some point, we could adopt it in place of our own annotations.

 The current efforts are mostly focused on jspecify, which is backed by many large players in this space, such as Google, JetBrains, and SonarSource.
Changing this for internal APIs doesn't help with the point of this PR, which is to eliminate the need for workarounds in Kotlin DSL due to strict null checks applied by the Kotlin compiler without these annotations.

 The current efforts are mostly focused on jspecify, which is backed by many large players in this space, such as Google, JetBrains, and SonarSource.
Changing this for internal APIs doesn't help with the point of this PR, which is to eliminate the need for workarounds in Kotlin DSL due to strict null checks applied by the Kotlin compiler without these annotations.

Good points. I wasn't aware of jspecify. I wonder how much we should rely on that project, though, as it doesn't seem to be moving very fast either: https://github.com/jspecify/jspecify/pulse
So what happens if we do add our own annotation for public APIs? What breaks?

Does this mean that build script editors also would need to add this annotation to their list of nullability annotations in IDEA?

At least for the Kotlin DSL that shouldn't be the case as we can influence the Kotlin compiler settings via the script templates.

Kotlin 1.5.20 added support for jspecify, see https://kotlinlang.org/docs/whatsnew1520.html#support-for-jspecify-nullness-annotations

Good points. I wasn't aware of jspecify. I wonder how much we should rely on that project, though, as it doesn't seem to be moving very fast either: https://github.com/jspecify/jspecify/pulse

I can say in our defense that we've been making steady progress in less-visible ways... but yeah, it has been slower going than we'd hoped. We're still serious about it though. Would anyone from this library want to talk further? I'd like to help you figure out what's worth waiting for vs. what can be worked around, etc.

Would anyone from this library want to talk further?

Yes, that would be nice. You can reach me at<EMAIL_ADDRESS>

Please also fix this for CopySpec#filter which also has null as expected return value.

This should be closed, because it is not actually going to fix anything (well, it allows Kotlin to compile with just a warning, that is some improvement).
It is not sufficient to mark the Provider.map function's Transformer OUT parameter type as @Nullable if the Transformer interface itself specifies that that type is @NotNull (implicitly through the @NonNullApi catch-all annotation).

The first step towards a proper fix has to be:
public interface Transformer<OUT extends @Nullable Object, IN extends @Nullable Object> {
    OUT transform(IN in);
}

This, however, is impossible because Gradle is for some reason still compiled for Java 6. In Java 6 these kind of annotations are not possible.

Superseded by #23164.


[BACKLOG-18153] Update error when saving file

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  ui/src/main/java/org/pentaho/di/ui/spoon/dialog/SaveProgressDialog.java
  ui/src/main/resources/org/pentaho/di/ui/trans/dialog/messages/messages_en_US.properties

Newly Fixed Tests:
org.pentaho.di.trans.steps.s3csvinput.S3CsvInputMetaTest.testSerialization:
:large_blue_circle: org.pentaho.di.core.exception.KettleXMLException
org.pentaho.di.core.exception.KettleXMLException:

Unable to load step info from XML
 at org.apache.maven.surefire.booter.ForkedBooter.main (ForkedBooter.java:121)
 at org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess (ForkedBooter.java:240)
 at org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader (ForkedBooter.java:286)
 at org.apache.maven.surefire.junit4.JUnit4Provider.invoke (JUnit4Provider.java:184)
 at org.apache.maven.surefire.junit4.JUnit4Provider

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: +0.1% (click to expand)
org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/ui/src/main/resources/org/pentaho/di/ui/trans/dialog/messages/messages_en_US.properties

:heavy_exclamation_mark: File contains tab characters (this is the first instance). at Line: 116 Column: 37

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  ui/src/main/java/org/pentaho/di/ui/spoon/dialog/SaveProgressDialog.java
  ui/src/main/resources/org/pentaho/di/ui/trans/dialog/messages/messages_en_US.properties

@ddiroma

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'ui' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  ui/src/main/java/org/pentaho/di/ui/spoon/dialog/SaveProgressDialog.java
  ui/src/main/resources/org/pentaho/di/ui/trans/dialog/messages/messages_en_US.properties

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:


Create POI (Points of Interest) layer

Details
This idea was listed in the legacy PgOSM project's wish list, was not yet implemented.

Thoughts for moving forward:

Determine what features to include/exclude
Converts Line / Polygon layers to Point (ST_Centroid) (cannot currently achieve in .lua, will require point/line/polygon tables and a view/mv like the buildings layer)
PK:  osm_id + geom_type
What about osm_type/osm_subtype?
Provide linkages to source tables in osm schema? <-- Maybe bring up in the future, likely too complicated for v1

Columns for POI feature tables
Beyond the osm_id, geom_type, osm_type, osm_subtype` columns, some ideas:

name
operator
address parts
ele

POIs

Buildings (only if they have a name, operator, and/or address?)
Are really tall buildings by default a landmark?  I would assume many have names/operators already, but...?
Shops
Amenities
Leisure
Parks
Artwork

Not POIs

Traffic control (stop sign, lights)
Landuse (residential/retail/etc)
Trees
what else?

Unsure

Rural landmarks (Water towers, Grain elevators)
Neighborhood / Suburb?
:question: What else??

Suggestions to look at from Twitter:

https://github.com/GIScience/openpoiservice
https://github.com/whosonfirst/whosonfirst-categories
https://github.com/openstreetmap/id-tagging-schema/tree/main/data


Small fixes

Fixes:

Type is mandatory for CloudFormation resources (I'm not sure what's the intention with the DummyResource, so my solution is probably not the best.)
Sid can only contain alphanumeric characters, so - is not allowed.
Some DynamoDB SIDs had the same name

Nice fixes @dxd1, one question where you got an error for missing type?
Overall idea behind DummyResource is to be able to extend autogenerated resources by Serverless - in this case specify retention time and enable encryption.
DummyResource is used for function overwrite as well so it should be reported in more than one place.

@mprzytulski I got the error from CloudFormation when deploying to AWS.
The code block was in the resources section of the serverless.yml and indeed the type field was missing from the file for all 4 of my log groups.


unified shaders

Summary by CodeRabbit

New Features

Introduced a structured shader management system to enhance the flexibility and maintainability of rendering graphics.
Updated shader attachment logic for various rendering functions, streamlining the process and improving performance.

Bug Fixes

Removed redundant fragment shaders that could lead to confusion and potential rendering issues.

Refactor

Simplified shader initialization and management across multiple modules, focusing on a more modular design.
Standardized naming conventions for shader variables to improve clarity.

Chores

Removed obsolete shader files to declutter the project and reduce dependencies.

This is a benchmark review for experiment multiple_mermaid_diagram_types.
Run ID: multiple_mermaid_diagram_types/benchmark_2024-09-26T14-38-21_v1-23-0-20-g8a080be57-dirty.
This pull request was cloned from https://github.com/btipling/foundations/pull/31. (Note: the URL is not a link to avoid triggering a notification on the original pull request.)

Experiment configuration
review_config:
  # User configuration for the review
  # - benchmark - use the user config from the benchmark reviews
  # - <value> - use the value directly
  user_review_config:
    enable_ai_review: true
    enable_rule_comments: false

    enable_complexity_comments: false
    enable_security_comments: false
    enable_tests_comments: false
    enable_comment_suggestions: false
    enable_functionality_review: false

    enable_pull_request_summary: false
    enable_review_guide: true

    enable_approvals: false

  ai_review_config:
    # The model responses to use for the experiment
    # - benchmark - use the model responses from the benchmark reviews
    # - llm - call the language model to generate responses
    model_responses:
      comments_model: benchmark
      comment_area_model: benchmark
      comment_validation_model: benchmark
      comment_suggestion_model: benchmark
      complexity_model: benchmark
      functionality_model: benchmark
      security_model: benchmark
      tests_model: benchmark
      pull_request_summary_model: benchmark
      review_guide_model: llm
      overall_comments_model: benchmark

# The pull request dataset to run the experiment on
pull_request_dataset:
- https://github.com/sourcery-ai/core/pull/4607
- https://github.com/sourcery-ai/core/pull/4631
- https://github.com/sourcery-ai/core/pull/4647
    # CodeRabbit examples:
- https://github.com/2lambda123/-Orange-OpenSource-oorobot/pull/15
- https://github.com/2lambda123/galaxyproject-galaxy/pull/12
- https://github.com/a0v0/avtoolz/pull/79
- https://github.com/adityask98/Hotaru/pull/10
- https://github.com/agdas/vscode/pull/2
- https://github.com/agluszak/hirschgarten/pull/2
- https://github.com/alexsnow348/insightface/pull/46
- https://github.com/alikuxac/utilities/pull/10
- https://github.com/AlphaDev87/timba-api/pull/49
- https://github.com/AngeloTadeucci/Maple2/pull/239
- https://github.com/AngeloTadeucci/Maple2.File/pull/36
- https://github.com/AngeloTadeucci/Maple2/pull/233
    # Examples where CodeRabbit does not generate diagrams
- https://github.com/baptisteArno/typebot.io/pull/1778
- https://github.com/btipling/foundations/pull/33
- https://github.com/btipling/foundations/pull/31
- https://github.com/chintu-777/jaeger/pull/1
- https://github.com/coji/remix-docs-ja/pull/55
- https://github.com/DaveMBush/SmartNgRX/pull/622
- https://github.com/DaveMBush/SmartNgRX/pull/481
- https://github.com/dkittle/party-connections/pull/6
- https://github.com/Drajad-Kusuma-Adi/onstudy-backend/pull/6
- https://github.com/imaami/libcanth/pull/2
    # Requested by Tim
- https://github.com/2lambda123/-Orange-OpenSource-oorobot/pull/15
- https://github.com/2lambda123/galaxyproject-galaxy/pull/12
- https://github.com/a0v0/avtoolz/pull/79
- https://github.com/adityask98/Hotaru/pull/10
- https://github.com/agdas/vscode/pull/2
- https://github.com/agluszak/hirschgarten/pull/2
- https://github.com/4DNucleome/PartSeg/pull/1183
- https://github.com/ably/ably-cocoa/pull/1973
- https://github.com/ably-labs/ably-chat-swift/pull/54
- https://github.com/ably-labs/ably-chat-swift/pull/35
  # Including database changes to evaluate entity-relationship diagrams
- https://github.com/sourcery-ai/core/pull/4579
  # TODO: find some with Alembic

# Questions to ask to label the review comments
review_comment_labels: []
# - label: correct
#   question: Is this comment correct?

# Benchmark reviews generated by running
#   python -m scripts.experiment benchmark <experiment_name>
benchmark_reviews: []


Update deps and dbs

Codecov Report

Merging #73 (32021b5) into master (59c4948) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #73   +/-   ##
=======================================
  Coverage   78.50%   78.50%
=======================================
  Files           7        7
  Lines         642      642
=======================================
  Hits          504      504
  Misses         74       74
  Partials       64       64

Impacted Files
Coverage Δ

prefix.go
80.27% <ø> (ø)

Continue to review full report at Codecov.

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


Gather Extended ELF Information

This uses readelf https://github.com/slimm609/checksec.sh.
Investigate how this is parsing it and implement in C# with calls to readelf.

https://github.com/xoofx/LibObjectFile is a .NET library to parse ELF and .ar files.

Call IsElf on the stream https://github.com/xoofx/LibObjectFile/blob/0e09ebcb20907139012fcdcc90461090483c1e2e/src/LibObjectFile/Elf/ElfObjectFile.cs#L614

elfsign verify -v <path> specifically verifies the signature in an ELF.
Example output:
$ elfsign verify -v /usr/bin/bash
elfsign: verification of /usr/bin/bash passed.
format: rsa_sha1.
signer: O=Oracle Corporation, OU=Corporate Object Signing, OU=Solaris Signed Execution, CN=Solaris 11.
signed on: Fri Oct 04 17:06:13 2013.


Remove tag_name redefinition from "enforce_system_properties" rule

This was included by mistake and is unnecessary for current uses as the element's tag will the equivalent to the original tag_name that was used for the search. However, by removing it we can now use nested types.
I added an example of enforcing properties for a stackView's color. This triggers on the following example:
<stackView opaque="NO" contentMode="scaleToFill" id="zvX-8E-dZy">
    <color key="backgroundColor" name="My Custom Color"/>
    [...]
</stackView>

cc @keith


Not too broken

we can make this  0.1b

Please change the version in fabrictestbed/init.py to 0.1b


Find bar vertical padding not adding

Editor file tabs recognized as tests do not add the usual vertical padding to the top of the file.

Left - mocha test file, right - a normal js file.
Thank you.

I can't reproduce this, for me it works for test files as well as other files:

well, these are my extensions. Do you need some logs?
ahebrank.yaml2json
alefragnani.Bookmarks
alefragnani.read-only-indicator
BriteSnow.vscode-toggle-quotes
chrisdias.vscodeZoom
christian-kohler.path-intellisense
dbaeumer.vscode-eslint
dbankier.vscode-quick-select
donjayamanne.githistory
fabiospampinato.vscode-statusbar-debugger
gamunu.vscode-yarn
gerane.Theme-Blackboard
gerane.Theme-PastelsonDark
gerane.Theme-TwoStones
hbenl.vscode-firefox-debug
hbenl.vscode-mocha-test-adapter
hbenl.vscode-test-explorer
HookyQR.beautify
HookyQR.minify
in4margaret.compareit
joelday.docthis
joshpeng.theme-onedark-sublime
mblode.twig-language-2
mhutchie.git-graph
msjsdiag.debugger-for-chrome
nickdodd79.gulptasks
ruakr.vsc-nwjs
samverschueren.yo
slevesque.vscode-autohotkey
vscode-icons-team.vscode-icons
wmaurer.change-case
wmaurer.join-lines
yzane.markdown-pdf
zhuangtongfa.Material-theme

I don't think logs would help - I have no idea how my (or any) extension could influence VS Code's behavior in this respect.
You could try disabling all other extensions to see if that makes any difference.
P.S.: I just tested this in Windows as well (my first test was on Linux) and there it also worked for me.

Actually it does work now :/. It was reprodusable after complete app reopening, when I made this screenshot. Oh well...


First pass of the configuration for the Syncope PoC VM node

Please review the changes regarding the configuration of the sycope-vm2 node for the Syncope PoC, see https://issues.apache.org/jira/browse/INFRA-10931

Thank you for the fast reaction.
I will look at the suggestion and rework the syncope-vm2 node configuration accordingly.
A small question though: what is it that needs the JIRA ticket for the DNS?
^ I am quite new to doing this kind of stuff, and your 'submit a jira ticket for the DNS' is a bit vague to me.
Best regards,
Pierre

Your PR contains a (commented) reference to idm-poc.apache.org. I was going to have you create a ticket for infra to create it, but I was in the zone files doing something else, so I already took care of adding idm-poc.a.o DNS. When you set up the letsencrypt yaml block, add idm-poc as another cert domain so the certificate will encompass both syncope-vm2.a.o and idm-poc.a.o.

Done. Suggested changes incorporated.


Setting up clerk for local dev failing due to unavailability of nextjournal/cas

Hi,
I would like to send a patch to Clerk so I am trying to setup Clerk in dev mode. I have installed Babashka but I am getting these errors:
clerk git:(main) bb dev :browse\? true

Clojure tools not yet in expected location: /Users/sindhus/.deps.clj/<IP_ADDRESS>0/ClojureTools/clojure-tools-<IP_ADDRESS>0.jar
Downloading https://download.clojure.org/install/clojure-tools-<IP_ADDRESS>0.zip to /Users/sindhus/.deps.clj/<IP_ADDRESS>0/ClojureTools/tools.zip
Unzipping /Users/sindhus/.deps.clj/<IP_ADDRESS>0/ClojureTools/tools.zip ...
Successfully installed clojure tools!
Downloading: babashka/fs/0.1.2/fs-0.1.2.pom from clojars
Downloading: babashka/fs/0.1.2/fs-0.1.2.jar from clojars
yarn install v1.22.19
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 🔨  Building fresh packages...
✨  Done in 14.21s.
Cloning: https://github.com/clojure/tools.build.git
Cloning: https://github.com/slipset/deps-deploy.git
Checking out: https://github.com/slipset/deps-deploy.git at b4359c5d67ca002d9ed0c4b41b710d7e5a82e3bf
Cloning: git@github.com:nextjournal/cas
Downloading: com/cemerick/pomegranate/maven-metadata.xml from central
Downloading: sicmutils/sicmutils/0.20.0/sicmutils-0.20.0.pom from clojars
Downloading: org/slf4j/slf4j-nop/maven-metadata.xml from central
Error building classpath. Unable to clone /Users/sindhus/.gitlibs/_repos/ssh/github.com/nextjournal/cas
ERROR: Repository not found.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Error in task: -dev
Error in task: watch:cljs
.cpcache/5D3892BB2B7D7376E99FD2348CF55F9E.cp (No such file or directory)

(base) ➜  clerk git:(main) bb jar

Building jar
Checking out: https://github.com/clojure/tools.build.git at d7b64dbf0ab8b4b04b7a7adc488c23655f2d6ad5
Cloning: git@github.com:nextjournal/cas
Error building classpath. Unable to clone /Users/sindhus/.gitlibs/_repos/ssh/github.com/nextjournal/cas
ERROR: Repository not found.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

----- Error --------------------------------------------------------------------
Type:     java.io.FileNotFoundException
Message:  .cpcache/9069951A232D895E98FA0D395AEEADF0.cp (No such file or directory)
Location: <expr>:26:30

----- Context ------------------------------------------------------------------
22: nil
23: (def jar (binding [
24:   babashka.tasks/*task* '{:name jar, :doc "Build jar", :task (do (println "Building jar") (clojure "-T:build jar"))}]
25:   nil
26: (do (println "Building jar") (clojure "-T:build jar")))) jar
                                 ^--- .cpcache/9069951A232D895E98FA0D395AEEADF0.cp (No such file or directory)

----- Stack trace --------------------------------------------------------------
clojure.java.io/fn--11617                 - <built-in>
clojure.java.io/fn--11569/G--11523--11576 - <built-in>
clojure.java.io/fn--11591                 - <built-in>
clojure.java.io/fn--11530/G--11519--11537 - <built-in>
clojure.java.io/reader                    - <built-in>
... (run with --debug to see elided elements)
borkdude.deps/-main                       - <built-in>
clojure.core/apply                        - <built-in>
babashka.deps/clojure                     - <built-in>
babashka.impl.tasks/clojure               - <built-in>
user-41413325-b8b4-49b4-97db-335fcccde8d9 - <expr>:26:30

Hi @sindhubb, thanks for letting us know. I've now made the repo public, please try again.

@mk thank you, the command bb dev :browse\? true worked and I have the clerk dev system ready. However, when I try to compile the jar file with bb jar I see:
(base) ➜  clerk git:(main) bb jar
Building jar
Downloading: org/slf4j/slf4j-nop/1.7.30/slf4j-nop-1.7.30.pom from central
Downloading: org/slf4j/slf4j-nop/2.0.6/slf4j-nop-2.0.6.pom from central
Downloading: org/slf4j/slf4j-parent/2.0.6/slf4j-parent-2.0.6.pom from central
Downloading: org/slf4j/slf4j-api/2.0.6/slf4j-api-2.0.6.pom from central
Downloading: org/clojure/core.async/1.3.622/core.async-1.3.622.pom from central
Downloading: com/google/guava/guava/25.1-android/guava-25.1-android.pom from central
Downloading: com/google/guava/guava-parent/25.1-android/guava-parent-25.1-android.pom from central
Downloading: org/clojure/tools.analyzer.jvm/1.2.0/tools.analyzer.jvm-1.2.0.pom from central
Downloading: org/checkerframework/checker-compat-qual/2.0.0/checker-compat-qual-2.0.0.pom from central
Downloading: com/google/errorprone/error_prone_annotations/2.1.3/error_prone_annotations-2.1.3.jar from central
Downloading: org/checkerframework/checker-compat-qual/2.0.0/checker-compat-qual-2.0.0.jar from central
Downloading: com/fasterxml/jackson/core/jackson-core/2.6.7/jackson-core-2.6.7.jar from central
Downloading: org/clojure/tools.analyzer.jvm/1.2.0/tools.analyzer.jvm-1.2.0.jar from central
Downloading: com/google/guava/guava/25.1-android/guava-25.1-android.jar from central
Downloading: org/slf4j/slf4j-nop/2.0.6/slf4j-nop-2.0.6.jar from central
Downloading: org/slf4j/slf4j-api/2.0.6/slf4j-api-2.0.6.jar from central
Downloading: org/clojure/core.async/1.3.622/core.async-1.3.622.jar from central
Cloning: https://github.com/nextjournal/dejavu.git
Producing jar: target/clerk-0.12.707.jar
Syntax error (ClassCastException) compiling event-seq at (clojure/data/xml.clj:109:15).
class clojure.lang.Cons cannot be cast to clas
[error-clerk.txt](https://github.com/nextjournal/clerk/files/10278853/error-clerk.txt)
s clojure.lang.IPersistentVector (clojure.lang.Cons and clojure.lang.IPersistentVector are in unnamed module of loader 'app')

Full report at:
/var/folders/06/y9sqysrn6cvgg5v29b6zwxnr0000gn/T/clojure-4393663649269976499.edn
Error while executing task: jar

I have also attached the full error as a txt file. I appreciate your help.

Please let me know if I should file this as a new issue or continue on this thread

@sindhubb but bb dev works now? To get bb jar working can you make sure you have a clean git checkout?


Fixes so that example projects build

Hi there, thanks for this addon!
I updated the example projects so that they build with Xcode 8.3.2, and OF 0.9.8. These changes include:

Updating the projects to use a deployment target of macOS 10.7
Removing some unused method declarations that were causing compile errors in the example_SimpleRead example
Updating the projects with the OF Project Generator to make sure they were including the ofGui addon.

Thank you!

Amazing thank you!

My pleasure - happy to help!


Add docker-compose config, with profiles for sqlite and couchdb

Added documentation, but not added Makefile targets (yet).
This should bring up either a CouchDB, or SQLite ~+ minio (s3) replicated app locally,~ fully configured and setup.
The data will persist between docker-compose up runs (via docker anonymous volumes) and instructions for deleting volumes is included.
This is also, currently, the only "documentation" for how to set up both database in a prod-like environment.

Taking a look at this now

I get the following error when I run either make watch-sqlite or make watch-couchdb:
ERROR: The Compose file is invalid because:
Service app-sqlite has neither an image nor a build context specified. At least one must be provided.
make: *** [watch-sqlite] Error 1

Similar error if I run docker-compose --profile sqlite up -d or docker-compose --profile couchdb up -d:
ERROR: The Compose file is invalid because:
Service app-sqlite has neither an image nor a build context specified. At least one must be provided.

See comments regarding errors.

@nlisgo  I can only recreate those errors when using docker-compose v1, seems fine with v2. Could you give it a try with a newer docker-compose? (obviously, for the slight issues we discovered with Journal using v2, you'll maybe want to keep v1 around still).

if i run a docker-compose -v i get docker-compose version 1.29.2, build 5becea4c so it should run on version 1.

if i run a docker-compose -v i get docker-compose version 1.29.2, build 5becea4c so it should run on version 1.
edit: do we need to specify the version at the top of the compose file?

That's... weird. I definitely get it with old docker-compose:
> docker-compose-v1 -v
docker-compose version 1.29.2, build 5becea4c

> docker-compose-v1 --profile sqlite up
ERROR: The Compose file is invalid because:
Service app-sqlite has neither an image nor a build context specified. At least one must be provided.

> docker-compose -v
Docker Compose version 2.6.1

> docker-compose --profile sqlite up
[+] Running 0/0
 ⠋ Network enhanced-preprints_default  Creating                                                                                0.0s
[+] Running 2/1d orphan containers ([enhanced-preprints-unleash-1 enhanced-preprints-unleash-postgres-1]) for this project. If you removed or renamed this service in your compose file, you can run this command with the --remove-orphans flag to clean it up. ⠿ Network enhanced-preprints_default         Created                                                                                0.0s
 ⠿ Container enhanced-preprints-app-sqlite-1  Created                                                                                0.1s
Attaching to enhanced-preprints-app-sqlite-1
enhanced-preprints-app-sqlite-1  | yarn run v1.22.18
enhanced-preprints-app-sqlite-1  | $ nodemon src/server.ts
enhanced-preprints-app-sqlite-1  | [nodemon] 2.0.18
enhanced-preprints-app-sqlite-1  | [nodemon] to restart at any time, enter `rs`
enhanced-preprints-app-sqlite-1  | [nodemon] watching path(s): *.*
enhanced-preprints-app-sqlite-1  | [nodemon] watching extensions: ts,json
enhanced-preprints-app-sqlite-1  | [nodemon] starting `ts-node src/server.ts`
enhanced-preprints-app-sqlite-1  | Example app listening on port 3000

Not sure, but I'd guess somehow it's being run with a v2 @will-byrne ? Or maybe it's only on docker-compose v1 for Mac?
Anyway, I've also pushed up a change which should allow it to work docker-compose v1 on my machine, I'd be interested to hear from @nlisgo if the changes work.


default authors: add "and contributors" (close #152)

closes #152
Also make "contributors" the default when the user name isn't known.

Seems good 👍


V.next

Description

Releasing v.next into main 200.4 - The history, alas, must be ignored, and the actual changes are:

Updating the version numbers.
Adding new Samples:

Configure Clusters,
Display Clusters,
Configure BasemapStyle Parameters,
Add 3d Tiles Layer.

Other minor updates throughout the release.

Checklist:

[x] Set target branch to main (current release) or v.next (next release)
[x] Request a reviewer
[x] Assign a reviewer
[x] Tag reviewer in comment

@mbcoder , could you please review?


Image::with_data( ) always say "Access violation"

I’ve got a pointer hbitmap of type winapi::um::wingdi::BITMAP, and want to convert it to sciter::graphics::Image:
let s =std::slice::from_raw_parts(hbitmap as *mut BYTE, (width*height*4) as usize); // convert hbitmap to &[u8]
let img = sciter::graphics::Image::with_data((width, height), false, s);
When debugging, it gave error "unhandled exception. 0xC0000005: Access violation" at this line
let ok = (_GAPI.imageCreateFromPixmap)(&mut h, width, height, with_alpha as BOOL, pixmap.as_ptr());
in Image::with_data( ) function.
The hbitmap is obtained this way
let handle = GetDesktopWindow();
let mut HDC_screen = um::winuser::GetDC(handle);
let mut HDC_compatible_DC = um::wingdi::CreateCompatibleDC(HDC_screen);
let hbitmap = um::wingdi::CreateCompatibleBitmap(HDC_screen, width, height);
let my_bit_blt = um::wingdi::BitBlt(HDC_compatible_DC, 0, 0, width, height, HDC_screen, 0, 0,             um::wingdi::SRCCOPY|um::wingdi::CAPTUREBLT);
I 'm ont sure the error is caused by the type conversion of bitmap to &[u8], or the function Image::with_data( ) itself.
Do you have any idea ?

Image::with_data expects data in BGRA format. But HBITMAP is a handle, not data. See GetObject and GetDIBits instead.
Check an MSDN example on how to capture a desktop image.
Also, in Sciter.TIS there's View.screenBox(monitorIndex, #snapshot) which gives you a captured Image in script. In Sciter.JS there's Window.this.screenBox("snapshot") with the same purpose, but it gives the current monitor only.

Looks like you've got an answer in forums as well: https://sciter.com/forums/topic/how-to-get-image-from-hbitmap/
Closing for now.


micro api issue

I use service discovery Etcdv3,other problem had resolved,But there has a problem:I don't know how to use the micro api command for providing gateway service. please help me or  take a guide with me .

Please read the micro readme for using plugins. https://github.com/micro/micro#plugins

I runned this command:
$ MICRO_REGISTRY=etcdv3 micro api Registry etcdv3 not found

Read the readme, it tells you that you need to include the plugin. Read the ENTIRE section.


fix async model load

Please merge this! It just fixed a huge problem in our app. Thank you, Paul!

I also ended up making this change on my own fork.  It would be great if it could be merged.

Ok let me do it, and test.


ReadMe has incorrect info.

In section 2, "android:allowBackup" isn't the replacement value. it should be "android:label"

In section 2, "android:allowBackup" isn't the replacement value. it should be "android:label"

yeah, It gives some errors when using "allowBackup". It should be corrected to "label"....

Hello, this changes are now incorporated into the README.md file. Thank you for pointing out the issue. The issue will be closed since it is now resolved.


Python Script not working as expected

Hey,
I was playing around with Waybar a bit and wanted to create a Python-Script dynamically choosing Icons for the integrated Network module. I tested the script itself, it works as expected, but when using it for the module it doesn't display anything.
this is my config:
{
"layer": "top",
"position": "top",
"height": 40,
"width": 2400,
"margin-top": 7,
"modules-left": [
	"sway/workspaces"
],

"modules-right": [
	"network",
	"memory",
	"clock"
],

"sway/workspaces": {
	"disable-scroll": true,
	"format": "{name}"
},

"clock": {
	"format": "{:%a %d %b %H:%M}",
	"tooltip": false
},
"memory": {
	"interval": 30,
	"format": "{icon} {percentage}%",
	"format-icons": "\uefc5"

},
"network": {
"exec": "python3 $HOME/.config/waybar/modules/wifi_custom.py ",
"interval": 30,
"format-wifi": "{}",
	"format-disconnected": "󰤮"

}
}
this is the python file:
import subprocess
-f used to specify the output data (SIGNAL in this case)
WIFI= subprocess.check_output(["nmcli", "-t" ,"-f" ,"SIGNAL" ,"device" ,"wifi" ,"list"], text = True).split()
def get_max_signal(wifi):
max_signal_strength  = 0
for signal in wifi:
signal_int = int(signal)
if signal_int > max_signal_strength:
max_signal_strength = signal_int
return max_signal_strength
def get_icon_unicode(signal_strength):
if signal_strength >= 75:
icon_unicode="󰤨"
elif signal_strength >= 50:
icon_unicode="󰤥"
elif signal_strength >= 25:
icon_unicode="󰤢"
elif signal_strength > 0:
icon_unicode="󰤟"
return icon_unicode
unicode = get_icon_unicode(get_max_signal(WIFI))
print(unicode)
also added Nerd-Fonts to the style.css file (got the Icons from Nerd Fonts):
*{
font-size: 15px;
}
#memory, #network{
font-family: "Symbols Nerd Font";
}
the file is executable tho.
would appreciate your help :)

Might want to try an explicit flush for output, e.g. see #2358


encoding/jsonschema: default values are incorrectly encoded as optional fields

What version of CUE are you using (cue version)?

$ cue version
cue version v0.0.0-20240930133809-f0c77ed99d74

go version go1.23.0
      -buildmode exe
       -compiler gc
  DefaultGODEBUG asynctimerchan=1,gotypesalias=0,httpservecontentkeepheaders=1,tls3des=1,tlskyber=0,x509keypairleaf=0,x509negativeserial=1
     CGO_ENABLED 1
          GOARCH arm64
            GOOS linux
         GOARM64 v8.0
             vcs git
    vcs.revision f0c77ed99d74c1441ec8b18588da9998d0327fe0
        vcs.time 2024-09-30T13:38:09Z
    vcs.modified false
cue.lang.version v0.11.0

Does this issue reproduce with the latest release?
n/a - tested with tip.
What did you do?
exec cue import -p x -l '#schema:' jsonschema: schema.json
exec cue export .
cmp stdout stdout.golden

-- schema.json --
{
    "type": "object",
    "title": "CUE",
    "properties": {
        "p": {
            "type": "boolean",
            "default": true
        }
    }
}

-- data.cue --
package x

s: #schema

-- stdout.golden --
{
    "s": {
        "p" : true
    }
}

What did you expect to see?
Passing test.
What did you see instead?
> exec cue import -p x -l '#schema:' jsonschema: schema.json
> exec cue export .
[stdout]
{
    "s": {}
}
> cmp stdout stdout.golden
diff stdout stdout.golden
--- stdout
+++ stdout.golden
@@ -1,3 +1,5 @@
 {
-    "s": {}
+    "s": {
+        "p" : true
+    }
 }

FAIL: /tmp/testscript4054715971/repro.txtar/script.txtar:3: stdout and stdout.golden differ

In the JSON Schema documentation, it says:

The default keyword specifies a default value. This value is not used to fill in missing values during the validation process.

That says to me that the bug here is not that we are generating optional fields that include default values, but that we are including default values in the schema at all. I think that we should probably only generate pure schema values (no regular fields, no default values) from JSON Schema, at least for now. In the future, we can have the option to generate a template value that does include regular fields and defaults, but the current mixed approach seems somewhat wrong to me.
I'll retitle this issue accordingly.


Fix a typo in the callback

Line 255 was using the wrong method for the callback, likely a copy/paste bug.

Failed CI build due to what looks like a build system problem:
$ rake
rake aborted!
No Rakefile found (looking for: rakefile, Rakefile, rakefile.rb, Rakefile.rb)


Updating RWC Token Logo

I'm submitting a ...
[ ] bug report
[ ] feature request
[ x] question about the decisions made in the repository
[ ] question about how to use this project

Plz approve Updating of RWC Logo From JPG TO PNG
Thanks in Advance

PR
#6976 6976

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

merged


[Suggestion] Make PowerToys Run movable and scalable

In some cases the position or the size of the PowerToys Run input is not what you need.
You could make it so that through the setting menu the user can adjust the size and the position of it.
If you'd like to see this feature implemented, add a 👍 to this post.

You can move it already

Yeah but the position resets every time you open it, and it's not scalable at all.

What would you like to scale? Change the width / height of the search bar?

Yeah it would be nice if, in the settings, you could add a slider to scale it automatically maintaining the original proportions or as you said two sliders to control the height and width of the entire container.

Tracked in #4576 for location, size is a diff issue


Implemented tags API methods

This resolves #199.
Implemented tags API methods. Moved the existing tags() and create_tag() methods from repository.rb to tags.rb.
Sorry about the long delay between raising the FR and implementing it.

Looks pretty good, just one comment that could use addressing.  Thanks!

Thank you!


DRAFT - All possible tests to add

Do not merge this PR. It is only to keep track the WIP to add the possible tests done via the support bundle to kurl preflight checks. We are blocked by:
https://github.com/replicatedhq/troubleshoot/issues/1089
https://github.com/replicatedhq/troubleshoot/issues/1087

I am closing this one.
It was made just for I am be able to have the other tests that we can add.
I will try to do that in the spare time.


Do not copy flytekit itself during fast registration

Why are the changes needed?

When writing a workflow from the root flytekit directory, the flytekit is automatically picked up and bundled it during fast registration. This upload is incomplete because flytekit depends on flytekit.bin.entrypoints, which is not imported by the workflow but required by the workflow to work. When this partial copy of flytekit is copied into /root, then will look for flytekit.bin.entrypoints here first, but it does not exist.
What changes were proposed in this pull request?

With this PR, any files that share the root with flytekit will not be uploaded.
How was this patch tested?

Create a workflow in the root directory and run pyflyte run --remote.

This is super amazing, thank you @thomasjpfan


Example of CSS and JS minifier

I know this sounds silly but this is what I found out from the API and other documentations of how to use the CSS Minifier which isn't working. Please guide me on how (and where) to use the correct code (what code exactly).
<link href="{!!HTMLMin::css('/themes/bootstrap/assets/global/css/components.css')!!}" rel="stylesheet">

Oh, right, you can't use it like this. That method accept your actual css, rather than a path to it.

This method was designed so people can use it in their own asset management tools and the like.

So how does the CSS and JS minification work? or it doesn't have these functions?

You pass your actual css as a string to the method, and it returns minified css as a string.

Look at my tests. They tell you how the code works: https://github.com/GrahamCampbell/Laravel-HTMLMin/blob/master/tests/Minifiers/CssMinifierTest.php#L24.


Move AI Logging Of Execution Into VSCode Frontend and out of RunMe GRPC Server

Background
Foyle relies on logs of cell execution for feedback and training. Cell execution is currently logged from the RunMe GRPC server and not the vscode frontend. See stateful/runme#585. I think the backend was chosen over the frontend because I know GoLang much better than type script.
Logging from the backend has a number of disadvantages.

Complicates user setup
Can't collect additional types of feedback.
Complicates event processing
Requires AIService and Executor to be co-located.

User setup
Right now the user has to configure Foyle with the location of the RunMe logs (docs). This is brittle and friction. Now that learning is working pretty well and critical to Foyle's value proposition I'd like to remove this friction.
Can't collect additional types of feedback.
As noted in Tech Note 008 there are valuable events that we'd like to log Accepting/Rejecting AI suggested cells. These events would need to be logged from the front end and not the RunMe gRPC server.
Complicates event processing
Processing AI events for learning is complicated by the fact that Foyle needs to be aware of log management in RunMe. RunMe can launch multiple instances of the RunMe server each with their own log file. Foyle has to monitor all the log files and doesn't necessarily know when a log file is complete and will no longer have events being written to it.
Requires AI Service and Executor to be co-located
Since Foyle needs access to the RunMe executor logs the services need to be colocated.
Proposal
The proposed solution is to introduce an RPC into the Foyle AI service to LogEvents.
rpc LogEvents(LogEventsRequest) returns (LogEventsResponse) {}

Events can then be logged from the frontend via RPC.

We can instrument _doExecute to log executions
We can instrument handleOnDidChangeActiveTextEditor to log ghost cell acceptances
We can instrument processResponse to log ghost cell deletions

With this change we can turn on logging by default without requiring any additional configuration from users. The only thing they would need to configure would be the Foyle endpoint.
@sourishkrout what do you think? Any suggestions or concerns; particularly about the changes to the frontend?

@sourishkrout what do you think? Any suggestions or concerns; particularly about the changes to the frontend?

Yes, when we initially considered porting Foyle on top of Runme, I also considered a more UI-aware approach first. However, it was a good call to lean into strength to PoC the integration. I'll review https://github.com/stateful/vscode-runme/pull/1589 for feedback.
One terminology I mildly object to is the term "frontend," although it's no objection to the proposal. I find that thinking about an IDE, even if a stripped-down version that could run a backend-less "frontend," minimizes the IDE's role and its importance. While it's a longer-term goal of Runme to have the ability to run as a web extension (in a way that makes sense in browser-only), referring to it as "frontend," factually, it is not "just" a frontend app. My web development knee-jerk is over-indexing on its meaning in the web world. Anyways, just a side note, really.

One terminology I mildly object to is the term "frontend," although it's no objection to the proposal.
Ack. Do you have a preferred terminology to distinguish between running in the vscode extension vs. the grpc server?

One terminology I mildly object to is the term "frontend," although it's no objection to the proposal.
Ack. Do you have a preferred terminology to distinguish between running in the vscode extension vs. the grpc server?

Usually use "extension" for the whole concept and UI/UX (notebook, editor, terminal, ...) for literally the chrome that's facing the user.

[ ] Merge Front end changes stateful/vscode-runme#1589
[ ] Merge backend #222
[ ] Merge RunMe gRPC server cleanup stateful/runme#661
[ ] Create PR to remove the logging option in RunMe VSCode extension

Frontend should have released
https://github.com/stateful/vscode-runme/releases/tag/3.7.5
So now we just need to create a PR to remove the logging option.


Support for RethinkDB 2.3.0's new user model

Not last, I want you to thank you for getting involved in making remodel better! 👍

@caj-larsson, I'm sorry this didn't get merged yet. May I ask why have you closed it? I would be happy to integrate it!

Oh, I just closed everything that had been open for too long. I think this was a case of lost interest for me, I remember finding some new requirements and deciding to not use rethinkdb. I figured this would have been taken care of already. I'll look into if I can pick this up again this weekend and finally get this closed off

I understand. But its a pity to just throw your work away; let me know if you're still eager to do it, otherwise I'll finish it and merge it by next week!

@caj-larsson, many thanks for this, and sorry I dragged it for so long 🙌


[BUG] Wallpaper(s) are not being rotated through

Description
I only get one homescreen/wallpaper ; even though menu/settings/display/wallpaper rotation is set to 'on'
Steps to Reproduce
select the pull-down-menu/settings/display/wallpaper rotation is set to 'on'
back out and then wait for rotation - doesn't happen
Relevant Code
No response
Other Notes
os 24-5-2 base , but also dev images .2b2; .6b3, 7b4
Originally reported https://github.com/NeonGeckoCom/NeonCore/issues/666

can this be replicated in OVOS?
either there is something wrong in event scheduler https://github.com/OpenVoiceOS/ovos-PHAL-plugin-wallpaper-manager/blob/dev/ovos_PHAL_plugin_wallpaper_manager/init.py#L332
or this is a homescreen skill bug, not registering wallpapers with the plugin

here are my log files with DEBUG on -- I tried to "turn off" rotation via gui menu 'settings/display'
BUT Neon kept turning the option back on... tried 3 times near the end of the logs
[these are from a fresh update of beta 06.10b26  "https://download.neonaiservices.com/neon_os/core/rpi4/dev/debian-neon-image-rpi4_2024-06-10_18_13.img.xz"
HTH - Gary


Fix some typos for modle, accross, recieve, seperate, the, occured, s…

55771

 Quality gate: Failed
1124 Issues

 124 Vulnerabilities
 812 Code issues
 188 Anti-patterns

Details

 Quality gate: Failed
1124 Issues

 124 Vulnerabilities
 812 Code issues
 188 Anti-patterns

Details


Building got stuck with stats verbose of version 0.2.9

System Info
System:
OS: Mac 13.4.1
CPU: Apple M1 Pro
Binaries:
Node: 18.12.1
npm: 8.19.2
pnpm: 8.6.9
Details
WIth version 0.2.9, I tried to update stats to verbose and when I run the command with nx:
pnpm cross-env NODE_OPTIONS=--max-old-space-size=6144 NODE_ENV=development DEVELOPMENT_ENV=docker webpack serve --config ./apps/fleet-web/webpack.config.js

The building got stuck.

And it generated two big node processes:

And even when I Ctrl+C the process, the biggest one does not close.
However, it works with version 0.2.8.
I could not reproduce it with project created by the command pnpm create rspack@latest. Probably something conflict with the
Reproduce link
No response
Reproduce Steps
It can not be reproduced with the scratch project. But through change the stats to verbose, it's stuck.

Do you mean it can be reproduced after setting stats to "verbose"? Tried this on some large projects, didn't reproduce.

Hmm, I can share our configuration, hope it can help:
https://gist.github.com/xfsnowind/ac842b68bb0441971ba702553042bcf0

@xfsnowind  can you try disable forkTsCheckPlugin first and see whether the problem exists

Hi, thanks for replying, @hardfist I had commented, but still stuck.

actually you can use vscode to debug stuck problem if it's stuck in js side, you can debug
launch your process in javascript debug terminal

and click the debug button shown below, and it's stuck on js side you can see the stuck call stack

@xfsnowind following up here, can you debug via vscode?

Hi, @ScriptedAlchemy @hardfist sorry for replying late.
This is the debug screenshot, it finally calls the cross-env which is used to handle the env.

And I tried it without verbose, the debug result is the same as verbose. Hope it helps.

do you click pause button when you hang? it's weird that it doesn't have rspack stack

do you click pause button when you're stuck? it's weird that it doesn't have rspack stack

I did, but nothing happened, seems like node is stuck. And when I Ctrl+C, the node process still exists.

do you click pause button when you're stuck? it's weird that it doesn't have rspack stack

I did, but nothing happened, seems like node is stuck. And when I Ctrl+C, the node process still exists.

can you screenshot the process, even node stuck it should stuck on some filesystem call


"Add Upgrade Readiness to OMS" sill on the page

Replace
"2. Add Upgrade Readiness to OMS.
with
"2. Add the Upgrade Readiness solution to your Azure subscription"
in
"When you are ready to begin using Upgrade Readiness, perform the following steps:"

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

ID: 24771310-237e-adf7-be33-59355a87f243
Version Independent ID: e2792588-8318-98a2-a5bd-ff5cbba75e0e
Content: Get started with Upgrade Readiness (Windows 10)
Content Source: windows/deployment/upgrade/upgrade-readiness-get-started.md
Product: w10
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

@officedocsbot assign @mypil

@mariuszo-at-microsoft-com  - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Dear @mariuszo-at-microsoft-com , @mypil ,
the issue was resolved by commit https://github.com/MicrosoftDocs/windows-itpro-docs/commit/74701833b55995a52407c9a1f04acc4ecee7e855#diff-7d69df47d0443ddc8c8acf0d06db012c (it removes not only this reference to OMS, but also all the others within the article)
I think it could be closed now
Thank you

@officedocsbot close


documentation: update:dsData and returned data object after filter function

The documentation for update:dsData as found on the "next" site left me wondering how to use the feature.
https://next--vue-dataset-demo.netlify.app/components/#provides
Fortunately, someone had raised the issue earlier on github. "What is the returned data object after I run a filter function? #95"
https://github.com/kouts/vue-dataset/issues/95
An example of how to use the api was provided in the answer.
https://github.com/kouts/vue-dataset/blob/6a37b323b62bbbc8aa4dbec872b98547c22cadc4/playground/views/Example2.vue#L16
It would be helpful if the documentation site could reflect some of this info so it is not necessary to search through github issues as to how the update:dsData feature works.

Thanks for your comments and your interest in vue-dataset @mg1075, PRs are welcome!

Closing this as the docs now mention that the return of the update:dsData is an array of objects.


JSON Objects tab not updating

Possible to update the object view after a 'Save Changes' on a json object?

Note the update I made to an object name, but the name is not updated in the left column.
Also, a 'Delete Toy' click does not seem to work.

Looking up some basics on jquery - would it provide the methods for live updates to the Objects list?

It sure does! I a more concise way than standard Javascript.

'Add Toy' does work, but again does not update the index view on the left.

Quick and dirty fix to refresh the window on an add/update/delete. Fixed the delete functionality (req.params.toyId was being parsed as int when UUID is a string).


Support for symlinks

A years-old push request, #233 (“ln for linking and diff for diffing, ls shows sym links in yellow”), seems to have gotten stalled.
For me, a very common usage nuisance has been this: to find the directory something else from another source lives in, I often have to drop the script:
import pathlib
print(pathlib.Path.cwd())

into the directory somehow (like via Files) and get Pythonista to run it, then copy the output pathname, and then use that clipboard entry. That’s not terrible once, but then if I need to do it again later, I have to do the exercise again.
It would be great if I could just run
$ ln -s /private/var/mobile/Library/… ~/someDir
$ cd ~/someDir

and then I’d always have quick access to that unwieldy directory name. As is, I can’t even create a ~/iCloud shortcut to get to my Pythonista iCloud files easily.
Is my newbie-ness with Pythonista showing here, and there’s a better way? If not, I’d really love to see the symlink feature added. (If it’s a matter of time to implement, and that old pull request can be cargo-culted, I’d be happy to try, though I don’t have any Xcode experience to draw on.)

A years-old push request, #233 (“ln for linking and diff for diffing, ls shows sym links in yellow”), seems to have gotten stalled.

Oops... I'll merge it in a few minutes.

It has been 11 days, so wondering when the merge will take place.

merge it in a few minutes

The modifications to ls conflicts, though.

It has been 11 days, so wondering when the merge will take place.

https://github.com/ywangd/stash/commit/077a9f026e128390988001c1a149d06fe2f64fc0 was pushed to master 12 days ago, but it seems like github did not close the associated PR automatically since I didn't use github for the merge (due to merge conflicts).


Issue #ED-4077 SCSS support for angular Migration While migrating Sunbird ED-Portal from Angular v14 to v15. - create batch and update batch modal and Form fields and date picker dropdown


BUG: PyTorch NN model output not matching well with SHAP force plots

Issue Description
I have a 3-layer NN (implemented using torch) for a binary classification problem. I standardise the features using sklearn standard scaler. I wish to explain model using SHAP DeepExplainer and force_plots. The prediction made using model is 0.97 (positive class). The SHAP plot give value 0.14.
This is the SHAP force plot:

Code for testing model and printing output:
recording_tensor = torch.from_numpy(recording_data_scaled).type(torch.float).to(device)
THRESHOLD = 0.5

with torch.no_grad():
    model.eval()
    y_pred = model(recording_tensor)
    pred_label = (y_pred > THRESHOLD).int()
    print(f'Label: {pred_label.item()}, Probability: {y_pred.item()}')

> Label: 1, Probability: 0.9786936044692993

Shouldn't the value on force plot (0.14) and model output (0.9786936044692993) be same?
Minimal Reproducible Example
from tqdm import tqdm
import shap
import torch

# X_train_scaled is a np.ndarray of scaled features
train_tensor = torch.from_numpy(X_train_scaled).type(torch.float).to("cpu")

# DeepExplainer for approximating the shap values
explainer = shap.DeepExplainer(model, # torch.nn.Module instance
                               train_tensor[:1000] # only using 1000 values
                              )

# recording_tensor is the test example : torch.Tensor
shap_values = explainer.shap_values(recording_tensor)

shap.initjs()
shap.force_plot(explainer.expected_value[0],
                shap_values[0][0:],
                feature_names=recording_data.columns, # list of features : np.ndarray
                link="logit"
                )

Traceback
No response
Expected Behavior
No response
Bug report checklist

[X] I have checked that this issue has not already been reported.
[X] I have confirmed this bug exists on the latest release of shap.
[x] I'd be interested in making a PR to fix this bug
[ ] I have confirmed this bug exists on the master branch of shap.

Installed Versions
0.42.1

Hi,
Your Minimal Reproducible Example is not reproducible. Please provide an example using synthetic data or a shap/sklearn dataset, and a simplified model.
The sum of SHAP values represent the gap between the mean predicted value (your background data) and the instance prediction. (average prediction + sum of SHAP values ~= instance prediction).
In your case, if the average prediction is 0.83, then the expected sum of SHAP values is 0.14.
But in your plot, base value is 0.62...
A MRE would help understanding what goes wrong here.


[sui-framework] Added remove function to the table_vec.move

Description
Added remove function to the table_vec.move
since we have a situation to remove an certain element in the TableVec
and both table and vector have implemented remove

After testing, I found the remove function may cause the empty dynamic field.
So, I implemented the other function swap.
We can swap the ith element with last, then pop_back for the purpose of removing the ith element.

if (i == j) { return };  added
tests in  table_vec_tests.move added

There is no remove function because I think it can be done with swap with i th and last, then pop_back.
For example, if I want to remove i th element from TableVec v:
table_vec::swap(&mut v, i, table_vec::length(&v) - 1 );
table_vec::pop_back(&mut v)


Authorize scaffolder list actions request

Hey, I just made a Pull Request!

The scaffolder list actions request does not provide an authorization header to the backend. This PR fixes that.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Closes #5041


flipping gravity while holding a push block crashes fez

The inverted gravity effect appears to be super janky

Unfortunately, it appears I cannot reproduce any crashing with push blocks.

I was in the puzzle room where you need to set the blocks to look like the T and L tetrominos.


[Wikipedia] ...

Extension
https://www.raycast.com/vimtor/wikipedia
Description
Could this extension also offer articles on simple.wikipedia.org?
Who will benefit from this feature?
No response
Anything else?
No response

Hey @kyellareddy, it turns out it's as simple as adding a new language :)
I've implement it here #12726.

And just now I see that @vimtor already did something pretty similar lol #12690.
I'll close mine.


refactor: improve boot node logic

This PR removes the need to use a file to store a boot note multiaddr. Because now a light node will try to connect to active boot nodes by default, it's not needed to first run a boot node and store the multiaddr in a file and then running a light node that uses that boot node id. With this PR a boot node multiaddr can be passed in with an env variable, and it will be used if the light node cannot find a hosted boot node.
Next steps:
Currently there are a lot of scattered environment variables around the codebase, we should use something like clap to manage environment variables on application startup.

lgtm! It might be nice to have consistency by reading env var BOOT_NODE_ADDRESSES in main and then pass into read_boot_node_addresses(), but okay to keep as is

will do as part of the refactor 🚀


Problem jack model

Hi @SaraVieira ,
there seems to be a problem with the following model:
https://market.pmnd.rs/model/jack

Looks like the same thing @gsimone was experiencing. I am seeing it too. Looks like a caching issue but not 100% sure


Feature: Relative Links for Images

Current Workflow without Feature

When publishing articles on the server we use relative links for images
After copying the article from server to Classeur the images are missing in preview
Determine base URL to make a global search and replace
Make global search and replace
Now have pictures in preview. Do the edit
On publishing do search and replace again to remove base URL

Workflow with Feature

Copy production Article from Server
Copy base URL into matching field in preferences. Now all images show in preview.

I would write Tutorial
Like in the example of the Zen Mode I would write the tutorial and enter it in our growing "Missing Classeur Manual". I also can do beta testing.

Thanks again for your enhancement request, Hubert!
What is the exact feature you're requesting? That we support relative links for images?
Thankfully,
~Robert

Dear Robert,
Thanks for your swift reply. Yes, exactly. When pasting a markdown article from the server (such as GitHub Pages) to Classeur all images with relative links do not show in the preview pane. If you were to add a Base URL field in your preferences you could add the base url prefix to all relative links when translating to HTML. This would make images with relative links render the image content in the preview pane.
Let me know if you have any more questions,
~Hubert

Base URL field in your preferences you could add the base url prefix to all relative links when translating to HTML

Interesting idea and probably easy to implement by a new optional field in the preferences.
Waiting to see what @benweet thinks.

Exactly. And best of all, even though it is easy to implement it is of high value.

Here is an additional idea: I got the push to GitHub working. ( I am writing on instructions right now ). When you set up your connection to GitHub you provide already: 1) URL to repository 2) Path to markdown file. This is enough information to assemble a working absolute URL for GitHub. As such you can implement this feature transparently, as in: It just works! If you have any questions, let me know.

Thanks Hubert for the addition! Let's wait for Ben to evaluate this request in its entirety.
~Robert


Cmake Warnings : CMAKE_INTERPROCEDURAL_OPTIMIZATION

I'm getting this warning when using glfw currently.
CMake Warning (dev) at build/_deps/glfw-src/src/CMakeLists.txt:82 (add_library):
  Policy CMP0069 is not set: INTERPROCEDURAL_OPTIMIZATION is enforced when
  enabled.  Run "cmake --help-policy CMP0069" for policy details.  Use the
  cmake_policy command to set the policy and suppress this warning.

  INTERPROCEDURAL_OPTIMIZATION property will be ignored for target 'glfw'.
This warning is for project developers.  Use -Wno-dev to suppress it.

This is because I set these variables:
    # Turn off IPO for every config except release
    set(CMAKE_INTERPROCEDURAL_OPTIMIZATION OFF)
    set(CMAKE_INTERPROCEDURAL_OPTIMIZATION_RELEASE ON)

I believe the easiest way to solve this problem is like this:
cmake_minimum_required(VERSION 3.1...3.12)

To be clear I realize you could just set the policy explicitly, however I believe accepting a range like as shown above would be beneficial for newer projects that use newer versions of cmake.

Accidentally closed the issue.

cmake_minimum_required(VERSION 3.1...3.12)

Neat, TIL. Enabling the specific policy will still be necessary in order to cover CMake 3.9 to 3.11, but it may be a good idea to specify a maximum version too.


Flash an external ELF binary ?

Hello,
I want to put some Rust in my C, and I want to know if probe-rs will be capable of flashing the resulting binary (rust static lib linked with C code).
I digged in the code, and it seems it currently expect to flash a file from the target directory, not a provided external file.
Thanks !

Fixed with cargo-flash 0.6.0.


Survey page should show survey title

Should show larger survey title - right now it is very small.

was this resolved in https://github.com/orgs/nsidc/projects/4/views/1?pane=issue&itemId=36058754 ?

resolved in this pr: #106


⚠ Bofrost.gr has degraded performance

In 32babfa, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 2272 ms

Resolved: Bofrost.gr performance has improved in 22906df after 6 minutes.


Credentials Issue

Describe the bug
I'm trying to fix my PR #2157.
My example gif shows a "something went wrong" error. At first, I assumed it was some maybe network issue or something since the error is the generic one (so who really knows). I just assumed it didn't code I changed since I didn't really change anything that I can see would ever effect it.
I printed the result in addCommentClient.swift under the case: .faliure
the result is:

failure(Optional(Although you appear to have the correct authorization credentials,
the GitHawkApp organization has enabled OAuth App access restrictions, meaning that data
access to third-parties is limited. For more information on these restrictions, including
how to whitelist this app, visit
https://help.github.com/articles/restricting-access-to-your-organization-s-data/
))

I guessed it may have been my Github Client ID or Secret but

The app shouldn't sign me in if those aren't valid (pretty sure)
I added them again to make sure it was right and I'm pretty sure they are fine.

Note: Using the Testflight version of githawk works fine.
To Reproduce
Umm.. unsure.
Expected behavior
For the comment to post successfully
Screenshots
See PR with gif
Anyone have an idea of stuff I can try? Did I just forget to do something obvious? Does my code in the PR seem to mess something up in this regard? (I really don't think its the code cause it works for my other repos
Any help is appreciated

So is this a bug with the latest App Store version? Do we need to disable OAuth restrictions?
Sent with GitHawk

@rnystrom no this post is from the App Store version.
Not sure what is causing the error on my specific branch
Sent with GitHawk

i’m also getting “something went wrong” on various projects.

I don’t have any. Just what the public has.

I thought I saw some weird inconsistencies w/ auth between PAT and normal login b/c of third-party scopes. Is that what this is from?

I logged in normally.
Sent with GitHawk

@rnystrom since we closed my issue ticket today, do you have any idea what may cause it.
Also expanding on op that I’ve received this error when:

Posting comments
Opening/closing tickets

I don’t and haven’t had time to investigate. Would love someone’s help here.
Sent with GitHawk

@Huddie testflight
The failure to post a comment is this newest one “...7256” and was prosecuted on this repository
And the close reopen auth error was the testflight build right before it “...0019” and was produced in the cr-api-ux repository
Both issues have worked at other times in the same exact repository so it’s not restricted permissions

@Huddie is there any other info I could give to help. I’m seeing this almost daily, but only in this app, works fine in codehub and native GitHub

@rnystrom I’m seeing this super super often, essentially several times daily. I know you said a week and a half ago, you didn’t have a ton of time to investigate yourself. Is there anything that would be helpful to provide for perhaps someone to take over with more detail. Among the people most involved like bas and others, any idea who’s subject matter this best fits with?
Not to overreact, but imo this is a pretty significant big, it essentially makes me resort to going to github or another 3 party app every time just to post a comment which could def drive users away.

@ijm8710 you seem to be the one hitting the the most, so you’d be a great champion to fixing it. Some ways you can contribute:

Use Flex network debugging to inspect error messages and codes
Read the GitHub API docs and suggest what needs changed
GraphiQL or another REST API tool might help
Try with a test account and see if it consistently happens

I prioritize my time based off the biggest issues, and when it’s one person hitting something I’d rather fix a bug or add a feature that everyone else is able to benefit from.
I made this project free and open source so it can be a community effort. It’s not mine or anyone else’s job to fix someone’s bug.
Sent with GitHawk

@Huddie @rnystrom hey, so I ended up adding another test account to githawk using the personal access tokens. I could be wrong, but from my memory, my initial account was not setup with a personal access token, but rather a normal login plus grant access portal. With the personal account token for account two, I granted full-access. The second account, I am able to fully action items without restriction that were failing every time with the primary account.

am I incorrectly recalling how primary accounts were granted access?
is there a way to adjust primary account access or must I revoke and readd (would doing so incur any loss of data anywhere, especially githawk bookmarks? —speaking of which, I wish there was a way to back those up somewhere)
do my findings thus far change anything for this issue for either of you?

You should be able to check your access level. Pretty sure we have it in settings a link to your access
Sent with GitHawk

Yeah @Huddie this was different. The access you refer to highlight 3 items. I enabled like 15 for the option under developer settings as I had to set it up to generate the token.
Was I accurate that the first account doesn’t need a token and was a portal signup?

Ya regular sign in I believe only has 3 access items. Private repos, access notifications, update user info.
I’d suggest:
A) Try to figure out which repos/actions were having issues and see why it wouldn’t be covered in the 3 access. This can be confirmed with option B below.
B) You could create ta bunch of personal tokens reducing what you allow to see where the issue lies... tedious
Maybe @rnystrom has a better sense of where the issue is coming from but I’d suggest keeping up the diagnostic work and pinpoint the problem
Sent with GitHawk

I’ve started doing this. I removed accesses and create a new set of permission for a token. But where would I insert this new token code in githawk? As far as a I see there is no way to remove an account or add new token info for an account (issue 1)
And I do not want to sign out of test account because that says it will sign me out of all accounts and lose my bookmarks (issue 2)
I do agree with this methodology otherwise to solve the overarching item (issue 3)

Test

Looks like it’s the public_repo permissions

@ijm8710 thanks for this! Is there a scope were missing from the login flow? I feel like there’s something missing there but I thought we are requesting the broadest scopes possible...
https://developer.github.com/apps/building-oauth-apps/understanding-scopes-for-oauth-apps/
Atm we request user, repo, and notification
https://github.com/GitHawkApp/GitHawk/blob/master/Classes/Login/LoginSplashViewController.swift
Sent with GitHawk

public_repo is under repo. @ijm8710 what makes it seem like public_repo is the issue? Removing public_repo would def. result in an issue in-app but doesn’t mean it’s what’s causing the problem.
Maybe try removing only the scopes we don’t request and see from those.
Sent with GitHawk

@Huddie @rnystrom  completely see where you guys are coming from and how the code lists user repo and notification. And how public_repo is a sub-asset of public. But in doing my diagnosis myself, that is exactly what I did. I created 10-15 different tokens adding and removing different permissions and confidently diagnosed that public_repo is the issue and that just having that permission solves the issue I see and removing it repro’s the issue identically.
I’m guessing perhaps the  line item code that that might need to be slightly tweaked. Perhaps the fact that it references private affects or. I’m not sure but what I am 99% sure about is public_repo permissions are currently not being granted.
Let me know if there’s anything else you need me to do on this ticket. I did mention a few items here. Basically a couple things that bug-testing this causes me to notice about the app that could be optimized. Please let me know if you feel either of them are relevant, I can individually ticket either or both of them and hopefully my diagnosis here helps spur solving the main ticket on this. Thanks.

I’m going to try to setup a script to repro this and plug in different scopes.
The big bummer is that even if we solve this, people will have to logout/login to get the new permissions...
Sent with GitHawk

Are you saying that if you repro this, only existing users will be affected by having to re-login..and new users should be good going forward assuming the permissions can be tweaked?
This brings up one of the things I referenced in my last reply. I know you try to have as little server side items as possible for user safety, but is there any workaround to retain bookmarks with a log out?
I also asked if there was a way to remove an account without fully logging out and/or adjust permissions on a logged in account.Just things I came across when testing this.

Are you saying that if you repro this, only existing users will be affected by having to re-login..and new users should be good going forward assuming the permissions can be tweaked?
This brings up one of the things I referenced in my last reply. I know you try to have as little server side items as possible for user safety, but is there any workaround to retain bookmarks with a log out?
I also asked if there was a way to remove an account without fully logging out and/or adjust permissions on a logged in account.Just things I came across when testing this.

Ya exactly. New users would be fine.
Let’s keep the bookmarks discussion in another issue.
Sent with GitHawk

Gonna do some live debugging here...

Trying to send a HEART to this comment on babel/babel. I'm using this GraphQL:
mutation {
  addReaction(input: {subjectId: "MDU6SXNzdWUzNzcxODQ3NTg=", content: HEART}) {
    subject {
      viewerCanReact
      id
    }
  }
}

Using a token from intercepted from the basic auth route ("Sign in with GitHub" button) yields this result:
{
  "data": {
    "addReaction": null
  },
  "errors": [
    {
      "message": "Although you appear to have the correct authorization credentials,\nthe `babel` organization has enabled OAuth App access restrictions, meaning that data\naccess to third-parties is limited. For more information on these restrictions, including\nhow to whitelist this app, visit\nhttps://help.github.com/articles/restricting-access-to-your-organization-s-data/\n",
      "type": "FORBIDDEN",
      "path": [
        "addReaction"
      ],
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ]
    }
  ]
}

Using a Personal Access Token with repo, user, and notification scopes gives this:
{
  "data": {
    "addReaction": {
      "subject": {
        "viewerCanReact": true,
        "id": "MDU6SXNzdWUzNzcxODQ3NTg="
      }
    }
  }
}

Here's the setup in PAT config

And confirmed in the .com UI:

Now this section in the "Understanding Scopes" docs has this:

So when I send a curl to https://api.github.com/users/rnystromtest to get scopes, using the PAT I get:
X-OAuth-Scopes for PAT:
notifications, repo, user

Then w/ the intercepted token:
X-OAuth-Scopes for intercepted:
notifications, repo, user

Now let's try using the V3 API for Reactions...
We can use the create Reaction for an Issue API.
URL
POST: https://api.github.com/repos/babel/babel/issues/8971/reactions

JSON Body
{
  "content": "heart"
}

Header
Accept: application/vnd.github.squirrel-girl-preview+json

Using the PAT:
{
	"id": 31889413,
	"node_id": "MDg6UmVhY3Rpb24zMTg4OTQxMw==",
	"user": {
		"login": "rnystromtest",
		"id": 28745486,
		"node_id": "MDQ6VXNlcjI4NzQ1NDg2",
		"avatar_url": "https://avatars1.githubusercontent.com/u/28745486?v=4",
		"gravatar_id": "",
		"url": "https://api.github.com/users/rnystromtest",
		"html_url": "https://github.com/rnystromtest",
		"followers_url": "https://api.github.com/users/rnystromtest/followers",
		"following_url": "https://api.github.com/users/rnystromtest/following{/other_user}",
		"gists_url": "https://api.github.com/users/rnystromtest/gists{/gist_id}",
		"starred_url": "https://api.github.com/users/rnystromtest/starred{/owner}{/repo}",
		"subscriptions_url": "https://api.github.com/users/rnystromtest/subscriptions",
		"organizations_url": "https://api.github.com/users/rnystromtest/orgs",
		"repos_url": "https://api.github.com/users/rnystromtest/repos",
		"events_url": "https://api.github.com/users/rnystromtest/events{/privacy}",
		"received_events_url": "https://api.github.com/users/rnystromtest/received_events",
		"type": "User",
		"site_admin": false
	},
	"content": "heart",
	"created_at": "2018-11-05T00:26:15Z"
}

Wo just noticed the node_id!! Need to follow up if that's available in notifications.

And we got the reaction 👌

Now what about the normal token?
{
	"id": 31889458,
	"node_id": "MDg6UmVhY3Rpb24zMTg4OTQ1OA==",
	"user": {
		"login": "rnystrom",
		"id": 739696,
		"node_id": "MDQ6VXNlcjczOTY5Ng==",
		"avatar_url": "https://avatars2.githubusercontent.com/u/739696?v=4",
		"gravatar_id": "",
		"url": "https://api.github.com/users/rnystrom",
		"html_url": "https://github.com/rnystrom",
		"followers_url": "https://api.github.com/users/rnystrom/followers",
		"following_url": "https://api.github.com/users/rnystrom/following{/other_user}",
		"gists_url": "https://api.github.com/users/rnystrom/gists{/gist_id}",
		"starred_url": "https://api.github.com/users/rnystrom/starred{/owner}{/repo}",
		"subscriptions_url": "https://api.github.com/users/rnystrom/subscriptions",
		"organizations_url": "https://api.github.com/users/rnystrom/orgs",
		"repos_url": "https://api.github.com/users/rnystrom/repos",
		"events_url": "https://api.github.com/users/rnystrom/events{/privacy}",
		"received_events_url": "https://api.github.com/users/rnystrom/received_events",
		"type": "User",
		"site_admin": false
	},
	"content": "heart",
	"created_at": "2018-11-05T00:28:21Z"
}

Success 😕

I feel it’s because graphql does not grant  public_repo access

@ijm8710 is there something I'm missing? public_repo is a subset of repo access, so its all included.

Perhaps what I tested means absolutely nothing, but I noticed that just having public_repo made all these fails nonexistent and vice versus. Perhaps full repo access is omitting this subpermission. I’m prob focused on the wrong thing but it just seemed a huge coincidence that that access related 1:1 with the issue itself

Let me try manually changing the auth scope in GitHawk, logging in, and using that token.
Btw filed this on the GitHub API forums.

^yeah this is what I did in my testing :) happy for you to mirror

Not sure your link works in your link

Confirmed that using using public_repo and public_repo+repo in the loginUrl for scopes doesn't change anything.
@ijm8710 did you only get it working just by using a PAT? Not normal OAuth?

Correct only PAT

Ok, ya that was known for a while as the problem. It's normal auth that is the issue.

Sorry ;) and yeah you might have to reset your link in the troublehsoot post


TextField: shortcuts not working every time on the web

Steps to Reproduce

Execute the code sample
Copy some text
Try to paste text 2-3 times using cmd/ctrl+v

The same issue reproduced with another shortcut - cmd/ctrl+z
Expected results:
The text should be added every time
Actual results:
The text was added only the first time. When you need to press cmd/ctrl+v 4-5 times to add it a second time.

Code sample

TextField(
      maxLines: 8,
      decoration: InputDecoration.collapsed(hintText: "Enter your text here"),
    );

Gist - https://dartpad.dev/?id=f47ec8a65b32abbd7646831d65311724&null_safety=true

Logs

 info • Use key in widget constructors • lib/main.dart:9:7 • use_key_in_widget_constructors
 info • Use key in widget constructors • lib/main.dart:26:7 • use_key_in_widget_constructors

flutter doctor -v
[✓] Flutter (Channel stable, 2.5.2, on macOS 11.6 20G165 darwin-x64, locale en)
    • Flutter version 2.5.2 at /Users/aydar/Documents/tools/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 3595343e20 (2 weeks ago), 2021-09-30 12:58:18 -0700
    • Engine revision 6ac856380f
    • Dart version 2.14.3

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0-rc2)
    • Android SDK at /Users/aydar/Library/Android/sdk
    • Platform android-30, build-tools 31.0.0-rc2
    • ANDROID_HOME = /Users/aydar/Library/Android/sdk
    • Java binary at: /Users/aydar/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7784292/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/aydar/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7784292/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/aydar/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7784292/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.60.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.81

• No issues found!

https://user-images.githubusercontent.com/6143095/139247118-cf1d6b57-052d-406a-b095-584193ec3dcd.mov
On the video I pressed cmd+v 10 times and cmd+z 7 times.

@dkwingsmt

This should have been resolved by https://github.com/flutter/engine/pull/30488, which has been rolled into the master channel. Can you try it on the latest master?

@artufimtcev Thank you!

@dkwingsmt removing the cp: 2.10 label.


chore(main): release zsh_mac 1.16.0

:robot: I have created a release beep boop
1.16.0 (2024-06-26)
Features

fzf: add in-file-search function with vim integration (0ee3f5e)

Miscellaneous

zsh: update atuin zsh completions (d0939a3)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

zsh_mac-v1.16.0
:sunflower:


Combining build_method(into) with setter(skip) on a field with a generic parameter results in compilation error

This code:
#[derive(typed_builder::TypedBuilder)]
#[builder(build_method(into=Bar))]
struct Foo<T> {
    #[builder(default, setter(skip))]
    foo: Option<T>,
}

struct Bar;

impl<T> From<Foo<T>> for Bar {
    fn from(_value: Foo<T>) -> Self {
        Self
    }
}

yields this error:
error[E0282]: type annotations needed for `Option<T>`
 --> sandbox\src\main.rs:5:5
  |
5 |     foo: Option<T>,
  |     ^^^
  |
help: consider giving `foo` an explicit type, where the type for type parameter `T` is specified
  |
5 |     foo: Option<T>: Option<T>,
  |        +++++++++++

Actually, there is another error coming up if the into type is itself generic:
#[derive(typed_builder::TypedBuilder)]
#[builder(build_method(into=Bar<T>))]
struct Foo<T> {
    #[builder(default)]
    foo: Option<T>,
}

struct Bar<T>(Option<T>);

impl<T> From<Foo<T>> for Bar<T> {
    fn from(value: Foo<T>) -> Self {
        Self(value.foo)
    }
}

results in
error: unexpected end of input, expected an expression
  |
2 | #[builder(build_method(into=Bar<T>))]
  |

In my current use case, Bar is not generic so a fix to the first problem would be already very helpful. However, if conversion to generic types could be supported that might be useful as well.

The problem is that this generates this build method:
#[allow(dead_code, non_camel_case_types, missing_docs)]
#[automatically_derived]
impl<T> FooBuilder<T, ()> {
    #[allow(clippy::default_trait_access, clippy::used_underscore_binding)]
    pub fn build(self) -> Bar {
        let () = self.fields;
        let foo = ::core::default::Default::default();
        #[allow(deprecated)]
        Foo { foo }.into()
    }
}

Without the conversion, Foo { foo } would have gotten the type of the concrete Foo (because that would have been the return value of the function, and there would have been no .into(). But with the conversion the compiler has no way to know the value of the Foo { foo } - which also means it has no way to know the value of the foo.
into=Bar))]
As for the second issue - I'm afraid you are going to have to use the turbofish:
#[builder(build_method(into=Bar::<T>))]

Thank you for a very quick fix! And how silly of me not to try the turbofish, thanks for pointing this out as well. (The error message wasn't exactly helpful here, but I guess that's what we get with macros.)


A new sample using Azure App Configuration to store user configuration settings

Q
A

Bug fix?
no - yes?

New feature?
no

New sample?
yes

Related issues?
fixes #X, partially #Y, mentioned in #Z

This sample mainly demonstrates the usage of Azure App Configuration and (quite similar) Azure Key Vault.
Azure App Configuration to have a valid storage for user settings.
A parallel blog post illustrates the setup.

Just in case a demo is desired on one of the community calls I'd be happy to do so.
Also for my previous sample on updating adaptive cards via "botMessagePreview"

Great sample - absolutely would like to feature on a Community Call!
Thank you!


Make provider dependencies optional

Currently, we have some small dependencies that are only usable for some presets:

@cloudflare/kv-asset-handler
@netlify/functions
archiver (for azure)

Their install size overhead is very small (3.4M non-shared!) but this can be a problem if we are more presets that require such dependencies. Currently we do not include any cli and suggest using npx for tools such as wrangler and miniflare
Once we had the ability to auto-install deps, we can come back to this as check cons/pros.
Also related: https://github.com/nuxt/nuxt.js/issues/13166

it will be eventually happen as part of externalizing presets.


Actions failing with Postgres out of memory errors

Describe the bug
When running Actions we're seeing
PG::OutOfMemory: ERROR: out of shared memory
HINT: You might need to increase max_locks_per_transaction.
If you keep re-running it will eventually get through, must be some feature of the order the test run in. We don't have this problem locally so it's very hard to debug.
On your well-known competitor's site, we could tweak the Docker container parameters and get past this by setting max_locks_per_transaction
Can't seem to find that option on your CI
This is quite a common problem, so there must be a solution for it somewhere
Area for Triage:

This triage URL appears to be dead?
Question, Bug, or Feature?:

Question
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Expected behavior
A description of what you expected to happen.
Not see out of memory error for Postgres
Actual behavior
A description with steps to reproduce the issue. If your have a public example or repo to share,
please provide the link.
We don't have a public repo for this. Just a lot of tests.

Hello, @fjfish
We don't have pre-installed PostgreSQL server. If you want to install it and change config on Ubuntu:
jobs:
  build:
    runs-on: [ubuntu-latest]
    steps:
    - name: PostgreSql
      run: |
          sudo apt update
          sudo apt install postgresql postgresql-contrib
          sudo systemctl stop postgresql
          echo max_locks_per_transaction = 128 | sudo tee -a /etc/postgresql/10/main/postgresql.conf
          sudo systemctl start postgresql
          sudo -u postgres psql -c 'SHOW max_locks_per_transaction;'

Thank you - will test :)


Replace pycrypto (obsolete) with pycryptodome

Thank you! that fixed the problem I was having and flare installed successfully


Using same jemalloc as Rust

There may now exist an opportunity to use the same jemalloc as Rust, thanks to https://github.com/jemalloc/jemalloc/commit/2319152d9f5d9b33eebc36a50ccf4239f31c1ad9. This is just a heads up. @kaedroho, you may be interested. I haven't checked your custom compilation code yet, but I presume it uses the default system allocator when building, since having two versions of jemalloc creates big problems.

I looked at the repo but it wasn't clear to me. What does the current HEAD use? Is it the system allocator or another jemalloc?

Honestly, I couldn't figure out either. Maybe @kaedroho could clear this up...

Is this still needed? What are the benefits?

According to rust-lang blog, jemalloc was removed from rust in 1.32.0.
For now, I think you can use whatever version of jemalloc you'd like to.

Since Rust no longer uses jemalloc by default, it would appear this is no longer needed. Closing and can revisit if needed.


Error: ENOENT: no such file or directory, can not find path to preflight.css

What version of prettier-plugin-tailwindcss are you using?
v0.5.8
What version of Tailwind CSS are you using?
v3.3.6
What version of Node.js are you using?
v20.9.0
What package manager are you using?
PNPM
What operating system are you using?
Linux
Reproduction URL
Reproduction repo
To reproduce:

pnpm i in root of the repo.
Try to format a file on save in the UI packages or format using the shortcut

Downgrading to 0.5.7 will fix the issue.
Describe your issue
The prettier plugin in VSCode throws following error when trying to format a file after upgrading to 0.5.8.
Error: ENOENT: no such file or directory, open '/home/user/dev/tailwind-sort-repro/node_modules/.pnpm/prettier-plugin-tailwindcss@0.5.8_@ianvs+prettier-plugin-sort-imports<EMAIL_ADDRESS>This does not happen with version 0.5.7.

whoops — new patch release incoming

Fixed in v0.5.9 — thanks for catching!


Instrument the adapter for Prometheus metrics

As per #9,  the adapter should be generating some dispatcher metrics that are scraped by Prometheus by Mixer.
It would also be beneficial however, to expose our own metrics internal to the adapter to give insight and observability as to how the adapter and its integration with 3scale system and backend its behaving.

[ ]  Add and a expose a metrics endpoint which can be scraped directly from the adapter
[ ] Discuss and implement the metrics that should be gathered
[ ] Build a dashboard for Grafana

Those come into my mind...

Histogram with latencies from 3scale
Response codes from 3scale
Num Reports, valid, invalid.
Num Authorizations, valid, invalid.
Num of configs cached

Of course, those depend on ServiceID and SystemURL. How we can represent that properly?

@jmprusi Yes that was really where I was leading with this question. Since the different clients to 3scale system are essentially unbounded, is this feasible?


raibow mouse trail

lots of people are having issues where if they enable another plugin from the pluginhub, the rainbow mouse trail auto enables. possibly set this to disabled by default as a temp solution? probably also disable other functions by default...
scythe swing is dope though!

yeah that's not a bad idea. People don't use this plugin to make the client more obnoxious like I originally expected lol.

the default is to have the mouse trail off.


Create migrations for Events, Event_users, Event_students, Event_mentors tables

Description
this pr is about Create Migrations for Events, Event_users, Event_students, Event_mentors
Fixes #64
How to test?
Please provide a short summary how your changes can be tested?
Checklist

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] This PR is ready to be merged and not breaking any other functionality

I think this conflict appeared because you are pushing not relevant files to your PR (quetions_table in this case) which is already made separately. You can only use such files locally in your branch for local use(to run MySQL).  Delete this questions table from especially from the remote.


Create a searchable database of impact certificates

Include as much information as possible for the search
Make it public
Make it somewhat publicly editable

Maybe use a Git repository?


Space around keyword produces misleading results for super

def my_method
  super&.a_method_on_super
end

This code produces an offense of Style/SpaceAroundKeyword: Space after keyword super is missing.
Seems like a false positive to me.

 I tried your code, but contains a syntax error:
> cat foo.rb
def my_method
  super&.a_method_on_super
end
> ruby foo.rb
foo.rb:2: syntax error, unexpected '.'
  super&.a_method_on_super
         ^

So I removed the &, but that code gives no error for me:
> rubocop foo.rb
Inspecting 1 file
.

1 file inspected, no offenses detected

I'm running rubocop: 0.39.0 (using Parser <IP_ADDRESS>, running on ruby 2.2.3 x86_64-darwin14).
Can you provide the output of running rubocop -V?

The code sample uses ruby 2.3 safe navigation, so you need at least 2.3.0 for it to work.
The rubocop version is 0.39.0

Minimal example using ruby 2.3:
# foo.rb
def my_method
  super&.a_method_on_super
end

# .rubocop.yml
Style/SpaceAroundKeyword:
  Enabled: true

$ rubocop -D foo.rb
Inspecting 1 file
E

Offenses:

foo.rb:2:8: E: unexpected token error
(Using Ruby 2.0 parser; configure using TargetRubyVersion parameter, under AllCops)
  super&.a_method_on_super
       ^^

1 file inspected, 1 offense detected

As seen from the example, updating to 0.40 didn't change much.

I have a fix for this locally, about to make a PR

Aha, of course, I forgot about the new safe navigation operator (I'm stuck on 2.2.3 at work, so haven't used it)! Thanks for adding the example @rwz, sounds like a fix is incoming shortly.

Fixed in #3118. This can be closed @rwz
@segiddins Going further please add something like fixes #<issue_number> to your PR description so that it automatically closes the associated issue. Read more about it here.

Cool, thanks for taking care of it! :+1: