Error SET Clause not supported

I am trying to write the results of spark processing back to neo4j using CAPS.
I looked at the example Neo4jMergeExample.scala
https://github.com/opencypher/cypher-for-apache-spark/blob/master/spark-cypher-examples/src/main/scala/org/opencypher/spark/examples/Neo4jMergeExample.scala
and was able to successfully write the results back to Neo4J. This required us to create a copy of the original Neo4J Graph , do updates on this copy and then use the
Neo4jGraphMerge.merge() method to update the results back to Neo4J.
This method worked for my use case. I am trying to explore how can we do an update to Neo4j using CAPS without creating a copy of the original graph.
I found one way to do this as mentioned in the example Neo4jWorkflowExample.scala
https://github.com/opencypher/cypher-for-apache-spark/blob/master/spark-cypher-examples/src/main/scala/org/opencypher/spark/examples/Neo4jWorkflowExample.scala.
I tried this example and am getting the error stating SET Clause is not supported.
code below gives an error stating that SET clause is not supported.
val rankRecords = capsession.cypher(
"""
|FROM GRAPH ranks
|MATCH (r)
|return id(r) as id, r.rank as rank
""".stripMargin
)
rankRecords.records.collect.foreach(record =>
capsession.cypher(
s"""
|FROM GRAPH neo4j
|MATCH (f:node {id:${record.get("id").get.toCypherString}})
| SET f.rank = ${record.get("rank").get.toCypherString}
""".stripMargin)
)
I am using spark-cypher 0.2.0
libraryDependencies += "org.opencypher" % "spark-cypher" % "0.2.0"
Stack trace
Exception in thread "main" org.opencypher.okapi.impl.exception.IllegalStateException: Error during IR construction: IRBuilderError(Clause not yet supported: SetClause(List(SetPropertyItem(Property(Variable(f),PropertyKeyName(rank)),Parameter(  AUTODOUBLE1,Float)))))
at org.opencypher.okapi.ir.impl.IRBuilder$.getTuple(IRBuilder.scala:61)
at org.opencypher.okapi.ir.impl.IRBuilder$.extract(IRBuilder.scala:66)
at org.opencypher.okapi.relational.api.graph.RelationalCypherSession.cypherOnGraph(RelationalCypherSession.scala:158)
at org.opencypher.okapi.relational.api.graph.RelationalCypherSession.cypher(RelationalCypherSession.scala:120)
at com.example.graphx.LoadNeo4jGraphx$$anonfun$3.apply(LoadNeo4jGraphx.scala:103)
at com.example.graphx.LoadNeo4jGraphx$$anonfun$3.apply(LoadNeo4jGraphx.scala:102)
at scala.collection.IndexedSeqOptimized$class.foreach(IndexedSeqOptimized.scala:33)
at scala.collection.mutable.ArrayOps$ofRef.foreach(ArrayOps.scala:186)
at com.example.graphx.LoadNeo4jGraphx$.delayedEndpoint$com$example$graphx$LoadNeo4jGraphx$1(LoadNeo4jGraphx.scala:102)
at com.example.graphx.LoadNeo4jGraphx$delayedInit$body.apply(LoadNeo4jGraphx.scala:16)
at scala.Function0$class.apply$mcV$sp(Function0.scala:34)
at scala.runtime.AbstractFunction0.apply$mcV$sp(AbstractFunction0.scala:12)
at scala.App$$anonfun$main$1.apply(App.scala:76)
at scala.App$$anonfun$main$1.apply(App.scala:76)
at scala.collection.immutable.List.foreach(List.scala:381)
at scala.collection.generic.TraversableForwarder$class.foreach(TraversableForwarder.scala:35)
at scala.App$class.main(App.scala:76)
at com.example.graphx.LoadNeo4jGraphx$.main(LoadNeo4jGraphx.scala:16)
at com.example.graphx.LoadNeo4jGraphx.main(LoadNeo4jGraphx.scala)
18/11/04 19:15:16 INFO SparkContext: Invoking stop() from shutdown hook

Hej @shyamrjoshi !
In CAPS all graphs are immutable (just like Dataframes), so once loaded you can not change or update a graph as you would in Neo4j, that is why the SET clause is not supported. However in CAPS you have the opportunity to create new graphs using the CONSTRUCT clause.
In addition in CAPS it is possible to query from different graphs in a single query, which actually gives you the possibility to combine your two queries into a single one like this:
  val updatedGraph = capsession.cypher(
    """
      |FROM GRAPH ranks
      |MATCH (r)
      |FROM GRAPH neo4j
      |MATCH (f:node)
      |WHERE f.id = id(r)
      |CONSTRUCT
      |   CREATE (f {rank: r.rank})
      |RETURN GRAPH
    """.stripMargin
  ).graph

The resulting graph only contains the nodes bound to f (all nodes with label node that have a partner in r). Having this graph you can now again use the Neo4j merge feature to merge the updated nodes into your Neo4j database.

Note that in order for Neo4jGraphMerge to work for you, the rank property must uniquely identify nodes in the Neo4j graph (it is a key for the node), specified by an EntityKeys object. If you need more properties to get a well-defined key, these need to be added in the CREATE clause as well.

Hi Max and Mat,
Thank you for the quick reply.
I am able to write back to Neo4J successfully using the Neo4jGraphMerge.merge().
I am also looking at the Neo4jWorkflowExample.scala example where a SET clause is executed using the neo4j.execute().
https://github.com/opencypher/cypher-for-apache-spark/blob/master/spark-cypher-examples/src/main/scala/org/opencypher/spark/examples/Neo4jWorkflowExample.scala
// Query for product recommendations
val recommendations = recommendationGraph.cypher(
"""|MATCH (person:Person)-[:FRIEND_OF]-(friend:Person),
|(friend)-[:IS]->(customer:Customer),
|(customer)-[:BOUGHT]->(product:Product)
|RETURN person.name AS for, collect(DISTINCT product.title) AS recommendations""".stripMargin)
// Use Cypher queries to write the product recommendations back to Neo4j
recommendations.records.collect.foreach { recommendation =>
neo4j.execute(
s"""|MATCH (p:Person {name: ${recommendation.get("for").get.toCypherString}})
|SET p.should_buy = ${recommendation.get("recommendations").get.toCypherString}""".stripMargin)
}
How can we use the neo4j.execute() to run the SET query on a remote Neo4j instance. (Neo4j running on a machine outside the spark clusters).
neo4j.execute() is part of the Neo4jHarnessUtils.scala, is this something used for testing and not a recommended practice ?

@shyamrjoshi The most straight-forward way would be to just instantiate a Neo4j Java Driver instance to your remote server and execute queries from that.
But if you already have a Neo4jConfig object created (which you will have in most CAPS apps that use Neo4j somehow) you can also do import Neo4jHelpers._ and then you'll get an implicit .cypher which you can call directly on the Neo4jConfig object itself. See for example SchemaFromProcedure.
I'll make a note to show this in one of the examples.

I updated the example to not use SET statements, but the (recommended) Neo4jGraphMerge feature: https://github.com/opencypher/cypher-for-apache-spark/pull/710

Hi Mats,
Thank you for the reply. I am trying to explore CAPS and I also posted a question on Neo4j Online Community . Thanks for your reply on the Neo4j online community. I will continue that thread here (Did not want to open a new issue for the basic question).
I wanted to get some more insights on how capssession reads from the remote Neo4j Server.
I see that CAPS will only load the nodes/data from remote Neo4j Server (based on the cypher query), when a spark action is performed, like .show() or .store() or any other action on the dataframe is called. However, When a spark action is called, does this load data from remote Neo4j into one spark node (one JVM machine) ? For example, I am querying remote Neo4j server using the below cypher query. This cypher query returns 100,000 vertices.
capsession.cypher("""
|FROM GRAPH neo4j
|Match (n)
|RETURN n
""")
Does all the vertices get store in memory of a single machine (spark master node/cluster manager)? or the capssession reads from the remote Neo4j Server in batches ?
Thanks,
Shyam

Thanks @s1ck . I will continue to use Neo4jGraphMerge.merge() for my use case. It works smooth. Thanks.


9.0.0-rc.1.24511.1 does not detect podman

To detect podman, I need to set the app host DOTNET_ASPIRE_CONTAINER_RUNTIME configuration to podman.
Is that the intended behavior?

Do you have Docker installed?

If your host environment has both Docker and Podman installed, .NET Aspire defaults to using Docker. You can instruct .NET Aspire to use Podman instead, by setting the DOTNET_ASPIRE_CONTAINER_RUNTIME environment variable to podman

see: https://learn.microsoft.com/en-us/dotnet/aspire/fundamentals/setup-tooling?tabs=windows&pivots=visual-studio#container-runtime

@Alirexaa, I don't have Docker installed, but I have docker CLI working with podman.

The detection is based on the CLI interface, not the backend daemon, so we're detecting and defaulting to the Docker CLI. If both are installed, we have to pick one and chose to default to the Docker CLI (we do attempt to detect the scenario where docker is just a symlink to the podman binary and choose Podman in that scenario).

The detection is based on the CLI interface, not the backend daemon, so we're detecting and defaulting to the Docker CLI. If both are installed, we have to pick one and chose to default to the Docker CLI (we do attempt to detect the scenario where docker is just a symlink to the podman binary and choose Podman in that scenario).

That doesn't seem to be a fail-safe approach.

The detection is based on the CLI interface, not the backend daemon, so we're detecting and defaulting to the Docker CLI. If both are installed, we have to pick one and chose to default to the Docker CLI (we do attempt to detect the scenario where docker is just a symlink to the podman binary and choose Podman in that scenario).

That doesn't seem to be a fail-safe approach.

We don't use any daemon API directly; our Docker vs. Podman support just represents our ability to run commands and parse the output from the different CLI implementations. The Podman CLI can be used to access a Docker daemon backend and the Docker CLI can target a Podman daemon's Docker compatible API and our tooling will still work in those cases, but we also see those as edge cases compared to Docker users running the Docker CLI and Podman users running the Podman CLI. We've been using this same approach since the first GA release of Aspire without any major issues.


Who's using the pod?

I'd love to know how you are using UICollectionViewLeftAlignedLayout, please leave a comment here with a link to the app, and/or a screenshot.
Thanks :smile:

Hey, just came along here. I'm not using the pod but would - the only reason I'm moving on is because what I REALLY want is a CenterAlignedLayout. :)

That's a great suggestion for a pod @crrobinson14 :D I don't know if I'll have the time to do it shortly, but if you do post a link here ;)

ss

I use it for the app and translate to a xamarin version. Thanks for your code. :)

@KanLei my pleasure :smile:


Question:  can you get back just a (flattened) json array instead of the csv string?

we have a need for a nice library to transform complex nested json into tabular form for display in a simple table/datagrid in a React UI
playing with json2csv it appears to do a nice job with the flatten/unwind features that we need
ironically we don't really want a csv string back though what we really need is just the transformed json in the form of an array of objects whose keys are what I see you return as first row of headings in your csv string
I was imagining maybe such an array of objects per row might already be used within json2csv, but I'm not seeing any exported functions that return other than the csv as a string?
(also looking at examples I was trying to figure out if maybe using a custom "formatter" or the streaming interface, or something might let me get at the intermediate json data as I want?  maybe?)

Hi @darrencruse ,
The standalone function that you are looking for is https://github.com/zemirco/json2csv/blob/master/lib/transforms/flatten.js
You can use it as:
const flattenedData = nestedData.flatMap(flatten({ objects: true, arrays: true, separator: '.' }));

Unfortunately, it's part of v6 which hasn't been released yet so there is no way to get it from npm yet.

Thanks @juanjoDiaz
So do I follow that "flatMap" function you mention is different than the "flatten" function I see in master right now?
I was snooping to see if I saw a v6 branch with the flatMap function but I couldn't find it is there a branch or PR or something somewhere that I could try (or at least see the code for) the new function?

Hi Darren,
flatMap is raw Javascript (see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/flatMap).
Actually, each row in the original dataset will produce a single row after flattening so you could simple use map instead of flatMap (see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/map).
const flattenedDataArray = nestedDataArray.map(flatten({ objects: true, arrays: true, separator: '.' }));

The function that I can offer is flatten which is passed to map and for each nested object returns a flattened object.
map simple applies it to each element in you original array.

Closing this since The question is already answered


Don't replace an existing request id header

Fix #6

Sorry for the late answer. This looks good but I want to properly review the changes. This might take a few days.

Ok I had a look at the changes and it seems fine.
I just think there should be a way to disable the behavior of using the supplied request id from the request. Something like a boolean flag use_incoming_request_id in the middleware, that when set to false, ignores the incoming value and mirrors the current behavior of the library.

Is the boolean to replace the map that I did to take the value from the request otherwise take the one generated by the middleware?
I don't see the usage of having one request id comming from the load balancer that we need to replace by the new one… it will break the entire tracking/observability

Some apps might not be running behind a load balancer. In this case any request coming from a user might include the request id header. In this case it might not be acceptable to use the supplied id value.
Also I'd like to fall back on the current mode of operation as to not break the library for projects that already use it as is.
And I found a bug in your PR: the RequestId extractor that can be used to access the request id in a route handler, is not properly set, when the incoming request id is used instead of a generated one.
Add the following lines to your testcase to reproduce:
let body: Bytes = test::read_body(resp).await;
let body = String::from_utf8_lossy(&body);
assert_eq!(body, uuid4);

Strange, I ran 50k call on an API and 0 error between request header and response header.
I'll try with your snippet

The bug happens here: https://github.com/vbrandl/actix-request-identifier/blob/master/src/lib.rs#L174
This line makes RequestId extractable in actix handlers, but it always uses the generated value, not the value extracted from the request headers.
I also think, calling the id generator should happen in the Service trait implementation, instead of the Transfer block. That way it can be called only when the request does not already supply a request id and the generated value is actually used.

I pushed code for both points:

The RequestId extractor now contains the correct id, when the incoming id is reused
It is now possible to configure the id reuse behavior

I'm still not sure, which one should be the default. Using IdReuse::UseIncoming might break the library for existing users. On the other hand, it might be time for a 1.0.0 release and I can document this as a breaking changes for consumers...

Maybe I broke something but implementing this version without .use_incoming_id(IdReuse::IgnoreIncoming) and running multiple times the same request, it's looping across the same request id generated by the middleware
To reproduce:

Source code: https://github.com/sycured/streaming_calc_actixweb
Docker image: https://hub.docker.com/r/sycured/scaw

pull the container: docker pull sycured/scaw
run the container: docker run --rm -it -p 8080:8080 sycured/scaw
execute the same request few times (at least the same number of workers) and find the same result in the log: curl http://<IP_ADDRESS>:8080/
look at the log:

[2022-12-28T04:59:29Z INFO  actix_server::builder] Starting 2 workers
[2022-12-28T04:59:29Z INFO  actix_server::server] Actix runtime found; starting in Actix runtime
[2022-12-28T04:59:31Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2022-12-28T04:59:31.354974383Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "10935814-84e0-49c8-8823-8965547ea362" 0.000751
[2022-12-28T04:59:32Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2022-12-28T04:59:32.469874447Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "d3daefd0-755a-4a48-bf2c-3529938bda11" 0.000122
[2022-12-28T04:59:33Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2022-12-28T04:59:33.15326365Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "10935814-84e0-49c8-8823-8965547ea362" 0.000112
[2022-12-28T04:59:33Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2022-12-28T04:59:33.667521365Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "d3daefd0-755a-4a48-bf2c-3529938bda11" 0.000054
[2022-12-28T04:59:35Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2022-12-28T04:59:35.156373314Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "10935814-84e0-49c8-8823-8965547ea362" 0.000062
[2022-12-28T04:59:35Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2022-12-28T04:59:35.598717559Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "d3daefd0-755a-4a48-bf2c-3529938bda11" 0.000052

Only 2 different request id on 6 requests:

10935814-84e0-49c8-8823-8965547ea362
d3daefd0-755a-4a48-bf2c-3529938bda11
It's equal to the number of workers so something is keeping it in memory also if nothing is coming from the request header.

You can try it on other endpoints (just look at the readme to have curl requests), it's the same thing when the middleware generates the value
If I put .use_incoming_id(IdReuse::IgnoreIncoming), the generation is working as expected: each request has a different id

I ran 26 requests and the request id is changing each time as expected (commit https://github.com/vbrandl/actix-request-identifier/pull/7/commits/51fd7dc47112df149e5ff94c564c2416713bd5c7)

I cannot reproduce the issue you described. I just pulled and ran the docker image and I get random ids:
[2023-01-05T14:07:30Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:30.308258672Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "b484c6b5-6507-459e-ae8b-8bd4aff2d678" 0.000073
[2023-01-05T14:07:31Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:31.332238917Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "57ca71d1-2666-4339-8897-5e364b83062c" 0.000079
[2023-01-05T14:07:31Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:31.867976823Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "dd96bafe-17f6-42a6-bd6d-7a809260262c" 0.000076
[2023-01-05T14:07:37Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:37.753107644Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "44dc76b6-8cff-4426-8481-8e54b9558915" 0.000090
[2023-01-05T14:07:38Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:38.212431011Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "cbd45845-bd94-4af1-b8c2-1e826d60dd0b" 0.000061
[2023-01-05T14:07:38Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:38.639412765Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "f2ede482-41aa-4803-b0b2-44e7789f4970" 0.000072
[2023-01-05T14:07:39Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:39.070253809Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "faf942e9-398e-43b3-9e10-03d95e0e5817" 0.000081
[2023-01-05T14:07:39Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:39.499249367Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "54f20b59-88e3-4453-b102-0910828baab5" 0.000087
[2023-01-05T14:07:39Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:39.930286734Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "dd0a9502-04a3-44c1-a8a9-4c0be1f80a38" 0.000067
[2023-01-05T14:07:40Z INFO  actix_web::middleware::logger]  <IP_ADDRESS> 2023-01-05T14:07:40.384608628Z "GET / HTTP/1.1" 200 23 "-" "curl/7.87.0" "ea8c3bfd-c62c-40f1-a225-844c7b2979b3" 0.000052

Removing/keeping the const modifier does not change this behavior (I tested this using the example code: cargo run --example actix-web. So I don't see any reason for removing the const modifier again but we should investigate what causes the error for you. Could it be a weak PRNG inside your docker container or something like that?

I did a cargo update to update the repository so the docker image was updated too
You need to keep the last - 1 commit

I just built sycured/streaming_calc_actixweb@fdc7bff0 with the const modifier for header_value and header_value and still can't reproduce.

On it, I'm running some hyperfine with services running on docker and bare-metal to verify without the last commit

Everything is fine on my tests with the commit removed

Weird and probably worth opening a bug report against docker desktop.
I'll do a final review, merge later and publish a new release of the crate

Would you also be ok with keeping the current default behavior but being able to configure IdReuse::UseIncoming when needed?

No problem for me

Thanks for your help and sorry, it took so long to finish and for being so picky but I merged your changes and just published the new release to crates.io: https://crates.io/crates/actix-request-identifier/4.1.0


Add SqueezePlay.app v7.8.0r984

After making all changes to the cask:

[ ] brew cask audit --download {{cask_file}} is error-free.
[ ] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

Additionally, if adding a new cask:

[x] Named the cask according to the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no open pull requests for the same cask.
[x] Checked the cask was not already refused in closed issues.
[x] Checked the cask is submitted to the correct repo.

I've created this new cask.
brew cask install squeezebox : works perfectly, downloads the right .dmg and installs it.
brew cas uninstall squeezebox : also works perfectly.
but
brew cask audit squeezebox complains about the Sourceforge URL which is not in the format specified in
https://github.com/caskroom/homebrew-cask/blob/master/doc/cask_language_reference/stanzas/url.md#sourceforgeosdn-urls
That also raises an issue on brew cask --style squeezebox.
However I've spent many tries to fix the URL without any success: when I use one of the recommended format in the cask_language_reference page, I don't get the binary file but an html file.
Any help would be appreciated so I can understand what URL to use in the future for such sourceforge downloads.
Sorry for the inconvenience, and thank you for reviewing this cask.
Bruno

In addition, I've just run the following command on the legit downloaded file, which gives me the exact URL (2nd one) I've put on the squeezebox.rb file:
$(brew --repository)/Library/Taps/caskroom/homebrew-cask/developer/bin/list_url_attributes_on_file ~/Downloads/SqueezePlay-7.8.0r984.dmg
https://netcologne.dl.sourceforge.net/project/lmsclients/squeezeplay/osx/SqueezePlay-7.8.0r984.dmg
https://sourceforge.net/projects/lmsclients/files/squeezeplay/osx/SqueezePlay-7.8.0r984.dmg/download

So, I don't see what is wrong when running the brew cask audit squeezebox command.

Thank you commitay.


Feat:Want to change color theme in dark mode #551

issue: #551

hey @mdazfar2 please review  this pr and let me know the changes

@nishant0708 please mention your issue

@nishant0708 please mention your issue

#551

@nishant0708, can you also implement dark mode on reload as well? Currently, when I switch to dark mode, it reverts to light mode on page reload.

@nishant0708, can you also implement dark mode on reload as well? Currently, when I switch to dark mode, it reverts to light mode on page reload.

Arey you talking about skeletons and wrappers right??

Arey you talking about skeletons and wrappers right??

No! @nishant0708 I am talking about this reload page 👇

Arey you talking about skeletons and wrappers right??

No! @nishant0708 I am talking about this reload page 👇

okay doing it

hey @mdazfar2  please check it now
https://github.com/mdazfar2/HelpOps-Hub/assets/101548649/dd032d4f-fcce-4139-bef6-99d269638192

@nishant0708 Why did you update to the Benjamin folder? Please remove changes from the Benjamin folder

@nishant0708 Why did you update to the Benjamin folder? Please remove changes from the Benjamin folder

i was trying to change color from there  but undo most of the changes . let me paste old file wait a sec

@nishant0708 You can contribute to the Benjamin folder once it is deployed

hey @mdazfar2  done

@nishant0708 You can contribute to the Benjamin folder once it is deployed

oo got it i don't knew that i was just looking for some class as multiple folder was have same class so i reached there somehow sorry for the trouble


Cosmetic. Progress item is moving few pixels on screen 4 of how it works

Issue: When going from screen 3 to screen 4, the progress indicator moves 4 px on the left for no reason.
Steps: Got to introduction screen, then How it works guide.
Solution: Make sure the position of the progress component is constant on all the pages.

Looks good.


[Feature] Exposing a ChatController

I think it'd be great if the chat exposed a controller for more flexible handling of the following scenarios:

Attaching files (For example ClipboardData) - right now this is possible with a custom messageSender, but the files aren't visible in the chat before submission
Call sendMessage programmatically. For example there might be predefined prompts, which should be executed on click.
Manage focus of the Chats Input. Especially on Desktop there are Scenarios, where manual focus management might be required (Also the current approach behaves kinda weird)

Attaching files (For example ClipboardData) - right now this is possible with a custom messageSender, but the files aren't visible in the chat before submission

Can you give me an example of what you'd like to accomplish here?

Call sendMessage programmatically. For example there might be predefined prompts, which should be executed on click.

You can do that today with the suggestions parameter of the LlmChatView constructor.

Manage focus of the Chats Input. Especially on Desktop there are Scenarios, where manual focus management might be required (Also the current approach behaves kinda weird)

Please log any bugs along with repro samples against any issues you find with the current approach to focus.
What kind of scenario would you like to handle with manual focus management?

Can you give me an example of what you'd like to accomplish here?

Well I'm filling in files from the Clipboard: https://github.com/DominikStarke/ogre
Right now I'm tracking them outside the chat window.

You can do that today with the suggestions parameter of the LlmChatView constructor.

That might work. What I want to achieve is if the user has something in his clipboard, he can invoke some "quick-actions", like writing a summary, or similar.

Please log any bugs along with repro samples against any issues you find with the current approach to focus.

I'll upload a video in a different issue. Soft-keyboards can't be hidden on mobile and on navigation once the keyboard loses focus and regains focus, just to be then dropped again, because the route is now out of focus. It makes for a very unpleasent route transition. That's on ios at least.

What kind of scenario would you like to handle with manual focus management?

When running on desktop my app has a simple hotkey, which brings it to front. When this happens I want to focus the input field (like in the video of the repo above). However right now it won't do that if the input isn't focused before the application is hidden.

Can you give me an example of what you'd like to accomplish here?

Well I'm filling in files from the Clipboard: DominikStarke/ogre Right now I'm tracking them outside the chat window.

I'm sorry. I don't understand. Can you frame it in user actions, the results you'd like to provide to them and how you'd like the AIT to help with that?

What kind of scenario would you like to handle with manual focus management?

When running on desktop my app has a simple hotkey, which brings it to front. When this happens I want to focus the input field (like in the video of the repo above). However right now it won't do that if the input isn't focused before the application is hidden.

Great. If you could log a feature request for that functionality, that would be great.

Let's say the user goes to a website, this page right here for example.
He sees there's already quite a lot of conversation going on and instead of reading everything he then:

Selects all the text on the website
Copies the selection to his clipboard
Opens my app
Now the app has the copied text as Context for the chat.

So far so good, that works already.
However I've had to create my own widget to visualize these kind of attachments.
Ideally I'd like them in the same area as the attachments opened with the file dialog (right over the chat input).
I hope this makes it more clear.
Here's a short Video:
https://github.com/user-attachments/assets/654b3c31-82c6-4ddb-8b25-fc52a152fc6e

so, so your request to add support for the clipboard as an attachment, either manually or automatically? or is it for the app to be able to define custom attachment types?

I'm lookiong for a way to add these attachments to the chat windows attached files, like normal attachments:

I don't expect the package itself to handle clipboard data.

have you tried doing this with a messageSender?

Yes, I mentioned that earlier. Submission with messageSender works (like a charm) and the attachments are displayed in the chat history correctly.
My issue is rather... reducing visual clutter:

understood. how should this work programatically?

Easy solution
Probably an attachments parameter to LlmChatView?
final controller = LlmChatViewController();
LlmChatView(
  attachments: [...],
  provider: EchoProvider()
);

Harder solution
I wanted to avoid calling setState in an LlmChatView parent however, so that's where the "controller" idea comes from...
final controller = LlmChatViewController();
LlmChatView(
  controller: controller, // Optional
  provider: EchoProvider()
);

We could then have something like controller.addAttachment(...), which would internally notify listening Widgets.
Same for requesting focus controller.requestFocus().
I also think controller.addSuggestions(...), controller.setHistory(...) would make sense, because otherwise its always necessary to call setState on the parent?
Or am I missing something?


Add version: Apifox.Apifox version 2.4.12

Pull request has been created with WinGet Automation 🪟📦🤖
Microsoft Reviewers: Open in CodeFlow

Duplicate of https://github.com/microsoft/winget-pkgs/pull/134035

Reset PR.

Close with reason: Unneeded pull request;


Re-invoke DismissIntent in Autocomplete if ignored

Ensures the DismissIntent used to dismiss the RawAutocomplete menu is re-invoked if the menu isn't visible.
Pre-launch Checklist

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

@justinmc that did the trick, made sure it kills the mutant as well.


DataGenerator on_epoch_end / shuffle_pair not functional

Disclaimer: I haven't actually tested it but can't see it working..
The shuffle_pair function doesn't shuffle the array in place as far as I can see, but is returning a shuffled view. Which in turn is not used by on_epoch_end.
    def shuffle_pair(self, a, b):
        # generate permutation index array
        permutation = np.random.permutation(a.shape[0])
        # shuffle the two arrays
        return a[permutation], b[permutation]

    def on_epoch_end(self):
        # shuffle dataset at each epoch
        if self.shuffle == True:
            if self.y is None:
                np.random.shuffle(self.x)
            else:
                self.shuffle_pair(self.x,self.y)

For an in-place shuffle I found the following StackOverflow answer useful (essentially shuffling each array in-place with the same seed):
def shuffle_arrays(arrays, set_seed=-1):
    """Shuffles arrays in-place, in the same order, along axis=0

    Parameters:
    -----------
    arrays : List of NumPy arrays.
    set_seed : Seed value if int >= 0, else seed is random.
    """
    assert all(len(arr) == len(arrays[0]) for arr in arrays)
    seed = np.random.randint(0, 2**(32 - 1) - 1) if set_seed < 0 else set_seed

    for arr in arrays:
        rstate = np.random.RandomState(seed)  # pylint: disable=no-member
        rstate.shuffle(arr)

Otherwise you could of course use the shuffled numpy array views.

The quickest fix would be to just use the view:
    def on_epoch_end(self):
        # shuffle dataset at each epoch
        if self.shuffle == True:
            if self.y is None:
                np.random.shuffle(self.x)
            else:
                self.x, self.y = self.shuffle_pair(self.x, self.y)

However here there is a shuffling at the end of each epoque, dose this means that we might take again the same date in the next batch that is generated?
Unless I'm missing something (again 💦) wouldn't be better to shuffle at the beginning?

However here there is a shuffling at the end of each epoque, does this means that we might take again the same date in the next batch that is generated?
Unless I'm missing something (again 💦) wouldn't be better to shuffle at the beginning?

The first epoch will use non shuffled data, then every epoch will take the shuffled data at the end of the previous epoch, so this is fine - every epoch will use different ordered data.
keras.utils.Sequence only implements a on_epoch_end so this is the way to do it I think!
About the error with the actual shuffling, I guess shuffling the array in-place should be a bit faster? I don't see a particular issue with modifying the original x, y for training/eval, although it is better to keep x untouched for the prediction.

The first epoch will use non shuffled data, then every epoch will take the shuffled data at the end of the previous epoch, so this is fine - every epoch will use different ordered data.
keras.utils.Sequence only implements a on_epoch_end so this is the way to do it I think!

yes, the shuffled data will be taken from the whole training set, I suppose, I wonder we should shuffle only the remaining data.

About the error with the actual shuffling, I guess shuffling the array in-place should be a bit faster? I don't see a particular issue with modifying the original x, y for training/eval, although it is better to keep x untouched for the prediction.

OK. Good point.
Although, currently, in the prediction (when self.y is None I suppose) it seems that the x array is shuffled in place:  np.random.shuffle(self.x)

I understand. If I'm not wrong the idea of shuffling is to have more random data from one epoque to the following. In this way, I assume, the model might require more time to converge but should get more robust.

No it's more that the order in training data is often not neutral. For instance the same entity will be repeated in a document, so if we take sentences in the order of the documents, they will tend to have a biased distribution. This is the case for CoNLL 2003 NER of FTB. It has no impact on training time I think.

I noticed that currently we shuffle the whole self.x, so the whole training set. There are possibility that the same data used for a previous epoque, is used again. I wonder we should shuffle only the remaining data instead. Or shuffle at the beginning when the data is loaded in the generator?

The same/complete data is always reused at each epoch (one epoch uses all traning data, divided in batch, each batch process in one "step"). So shuffling at the end of an epoch is correct (we don't shuffle at the end of a "step").

What about that we shuffle in-place both x and y (and features) only in training mode (when Y is not None)?

Actually thinking about it, I would never shuffle in place, in the case of n-fold training where we select a different dev set for each fold, I think it will mess everything. So the option "just use the view" is safer I think in every case (even if it's a bit slower, the impact should be negligible as compared to all the other crazy heavy computations elsewhere).

Generally I certainly prefer a functional approach and I would definitely think that the data passed to DataGenerator should not be changed. But it should be okay to modify a copy / view in place (as long as it's confined to the DataGenerator). I wonder whether the following might lead to a long chain of views with every epoch:
self.x, self.y = self.shuffle_pair(self.x, self.y)

I don't know enough about how numpy views work - whether a view of a view would reference the passed in view or whether it references only the original array. A simple solution (if it needed one) would be to always pass in the original x and y.


Integrate with nixos-generators

We already have flk iso|vm|doi, but users want more. And we want to rely on the best available upstream aggregate for that task: nixos-generators which shall become part of the familiy.
/cc @lassulus

This is accomplished via https://github.com/divnix/digga/pull/50


Remove parquet-scala dependency from dependencyManagement.

Follow up to #2140.
The parquet-scala dependency can also be removed from <dependencyManagement>.

Coverage remained the same at 78.989% when pulling abf889fc838aa8475f1b8a7f3abcf9ec6b374064 on heuermh:remove-parquet-scala-dep into e78ccebd0626ffe9678a4f040f93889c7195da3e on bigdatagenomics:master.

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


Support Dehumidifier

It would be great to support Dehumidifiers which are a subset of the controls.
If I try my Midea compatible dehumidifier, it currently comes back with the error:
Unsupported device type: 0xa1
Would be great to support these.

That will be great. I have the same Problem.

Me 2


Add support for scaling mesh x,y,z independently

Adds support for scaling mesh by x,y,z independently. In RobotSDF, reads scale parameter from urdf.

Requires https://github.com/UM-ARM-Lab/pytorch_kinematics/pull/21


error when rapidly deleting and undoing

prosesitter.nvim/lua/prosesitter/on_event/marks/marks.lua:55: end_col value outside range

undo in general is problematic its seems

to reproduce:
example:
// misspelled
// longmisspelled
//
fn main() {
}

set check scheduling to 3 seconds
wait till extmarks/hl are set
delete and undo and delete longmisspelled within 3s
wait 3s

this will give error:
[FATAL zo 05 sep 2021 14:58:50 CEST] ...rosesitter.nvim/lua/prosesitter/on_event/marks/marks.lua:20: could not place extmark
buf_id: 1
row: 1
col_start: 3
col_end: 16
[FATAL zo 05 sep 2021 14:58:50 CEST] ...rosesitter.nvim/lua/prosesitter/on_event/marks/marks.lua:55: mask_id: col value outside range
[FATAL zo 05 sep 2021 14:58:50 CEST] ...rosesitter.nvim/lua/prosesitter/on_event/marks/marks.lua:56: mark: { 1, 0, {
    end_col = 0,
    end_row = 1
  } }
[FATAL zo 05 sep 2021 14:58:50 CEST] ...rosesitter.nvim/lua/prosesitter/on_event/marks/marks.lua:57: hl: {
  buf_id = 1,
  end_col = 17,
  group = "SpellBad",
  hover_txt = "Did you really mean 'longmisspeled'?",
  row_id = 3,
  start_col = 5
}

Analysis:
placeholder extmark gets created
text on it is send for checking
line with placeholder extmark gets deleted
results from checking come back and get row via placeholder extmark
row has content of smaller length, hl extmark can not be placed

Solutions:

on line deletion remove any placeholder extmark leave any hl-extmarks alone.

Non-Solutions:

wrap highlighting in pcal; this will only fix the case the line width changes
add hash and only hl if hash of checked text matches current text; this will block any highlights appearing while editing a line

TODO hook into undo event to schedule a check


fix: increment build numbers

Signed-off-by: ankitm123<EMAIL_ADDRESS>

FWIW, this wont fix it completely, but does add the logic to always increment build numbers.
The case where it breaks is this
Someone deletes all the pipeline acitivites associated with a given branch, in that case the logic will make the next build number start from 1.
What this fix does accomplish is that when a build exists - say build number 2, the next build will be 3 and not 1!
For a full fix, we should store the pas/build numbers in a config map/s3 bucket! Then we can retrieve it even after the pa(s) have been deleted from the kubernetes cluster.


Add examples for both Promises.join and Promise.join

Ci failure seems unrelated, but worrying in general.

I've been trying to find a way to succinctly describe the differences between the two for the documentation, here are two use-cases:
In one of my projects I have an array of actors, I use itertools.map to get an iterator of promises from this input array which is then passed to Promises.join.
In another, I have a promise that I've converted from something else via a chain of .next calls which I then want to combine with a set of other similar promises. I this case I would use:
promise.[Array[String]]next(...)
  .next[String](...)
  .join(other_promises)
  .next[Array[String]](...)

Thanks @CandleCandle


Authorization Code Flow with PKCE

https://auth0.com/docs/authorization/flows/authorization-code-flow-with-proof-key-for-code-exchange-pkce#how-it-works
Is this supported? Trying to use this for my test suite, but auth0 is going to be my real production? If it's not supported, is it planned? I was doing some code autocompletion, but I didn't see it as an option, but I might be looking for the wrong thing, or in the wrong place.
If it's not possible and not on the road map, could it be?

Yes, this is in and working. See org.springframework.security.oauth2.server.authorization.authentication.OAuth2ClientAuthenticationProvider

Hi @xenoterracide, thanks for your interest in the project. As @jimreader mentioned it is supported during client authentication. It is not required by default, so if you're using a public client, you may want to look at making it required on the RegisteredClient.Builder, e.g.
.clientSettings(ClientSettings.builder().requireProofKey(true).requireAuthorizationConsent(true).build())

I'm going to close this, but if there's anything specific you're having trouble figuring out, feel free to add a comment with a link to a Stack Overflow question and I'll be happy to help you move past it.

Thanks guys, I'll see if I can't work it out here soon. It just wasn't an obvious API from what was shared in the blog post.
Can't wait for the reference documentation 😊😉

Can't wait for reference docs btw

@xenoterracide, stay tuned! 📘


Merging first code

Main development is finished, so it's time to merge it to master branch

TODO: - Fix decimals in post text (e. g. it should be 12/10, not 12.0/10)

TODO: - Remove mod-only limitations


crate revival and discussion

@henryjac and I (he/him pronouns) will serve as new maintainers for the crate, in @boxtown's words (on #201)

Thanks a bunch for stepping up! I genuinely wished I had the time/drive to give this crate the attention it deserves but I'm happy to see it transition to people who are interested in the maintaining health of the project. Best of luck!

We are each pretty new to this, but each have support from mentors as we transition into the role. We'd appreciate patience as we figure out how we'll each be filling this role - much of that will start as discussion here so it's public. I'd be open to directed feedback on how I do here; I can't speak for @henryjac; expecting similar.
Expect some updates in the docs to reflect what we'll be aiming for to support contributors and users as we figure out how to carry this crate forward. At the moment, I think our short term goals are to collect a few PRs for a new release which I've added a milestone for as well as tracker issue, #205. I'm planning to see which of these is more helpful, but milestones seem sparse in the repo, so maybe just the issue will suffice.
--
EDIT
In my haste, I didn't write how thankful I am for @boxtown, @troublescooter, and many others for getting us here!

Since posting this, I've started to accept a few PRs to show activity on the gh repo, but I realize I've not stated that intent, so that's what I'm up to now. For now, I'm happy to continue prioritizing

existing gaps:

helpful usage and contributing docs,
CI to support consistency,
changes to abide by simple API guidelines,
removing panic! where available or where inconsistent,

and fixes:

egregious numerical inaccuracies,
inaccurate docs,
broken tests

But I'm hoping for some input/feedback for

the 0.17 release on #205
general input on here (#206)

what do you use this for?
what would you want to use it for?
what other statistical tooling do you use in lieu or alongside with this?

I don't quite know if this is the proper place for this, but is there a way to make certain guarantees that unsafe code external to the crate can depend upon? I use this code a lot for code that is supposed to be relatively high performance, but it's relatively difficult to extract information from the struct safely and effectively so that I can manipulate it with unsafe code.
As a simple example, is there a way to add some kind of InfallibleDistribution trait which is basically the Distribution trait but for distributions that have the relevant statistics always defined regardless of their parameters? Currently, I'm using unwrap_unchecked() on the std_dev() output of the Normal distribution, but that's not technically guaranteed by the API to always be sound, even though the standard deviation of any Normal distribution is obviously always defined.

Hey! Thanks for asking. It may not be the best place for this issue, hoping this one could be more focused on what areas/contexts you use statrs.
Could you open an issue to discuss this?

Replying to @wangjiawen2013's https://github.com/statrs-dev/statrs/issues/201#issuecomment-2196219142
We're glad it's alive too, let us know what directions you're interested in using/contributing to!

@We are very interested in hypothesis test, such as t-test and wilcoxon-mann-whitney U test. As you can see, some other people also want to use it (https://users.rust-lang.org/t/mann-whitney-u-test/95005)


Display /db_xrefs from features

These /dbxrefs are in Chado but need to be passed to the web site.
From Val:

They support the "coordinates' of a feature so it would be nice longer term to make them a pop
up somehow on genomic location
I, | 3086472-3086930 (459nt)

Example from chromosome1.contig:
FT   ncRNA           complement(40916..41770)
FT                   /systematic_id="SPNCRNA.601"
FT                   /product="intergenic RNA (predicted)"
FT                   /db_xref="PMID:21511999"

It's now improved so that the publications from the feature /db_xrefs appear in the literature section.

They support the "coordinates' of a feature so it would be nice longer term to make them a pop
up somehow on genomic location
I, | 3086472-3086930 (459nt)

We can do that now that the feature db_xrefs are available to the web code.
Could we have a link beside or underneath the location with text like "Source ..." that's a link to the reference?  And/or we could have a popup when you hover over "Source ..."

I like "Source ..."

I like "Source ..."

OK, I'll try that and we can see how it looks.

I've added the "Source ..." link:
https://www.pombase.org/gene/SPNCRNA.57
but I wasn't sure how it should be formatted if there are two db_xrefs.  So currently it looks different.  There aren't that many with more than one ref so maybe it doesn't matter.  Here's an example:
https://www.pombase.org/gene/SPNCRNA.1187
Let me know what you think.

I think its fine. we can always rethink when we rearrange top matter later. Is it possible to do a single pop up with 2 independent links?

Is it possible to do a single pop up with 2 independent links?

Yep, but that would have taken longer.  :-)

I'm going to close this.  We can open new issues for changes.


Support for copying logs to alternate outputs

eg:

mail to some user
copy to S3 bucket (private)

I think the current (file, s3, sqs) is sufficient for now


Support AMQP as a medium for scraping (apart from http{s}).

Many distributed systems already have one control/management channel opened into the network, and opening another one (for example http{s} for scraping) may raise security or otherwise maintenance concerns.
One protocol that is extremely popular in this space is the AMQP protocol (RabbitMQ).
AMQP protocol supports the RPC use case, which can be easily used by prometheus to scrape the information needed.
My feature request is to implement such scraping mechanism.
Please note, that for now, I've implemented a "proxy" service that accepts HTTP{s} requests and sends an AMQP-RPC request to my remote servers, but this is very patchy and I'd prefer this to be integrated into Prometheus. Code can be found here.
I also volunteer to implement this in prometheus code base (and send a pull request), if it's something you see fit in the project.

A system where there's no HTTP and where it cannot be added at all sounds like a very niche use case. I don't think it would be appropriate to add an additional non-HTTP scrape protocol to support it.

I don't think it's that niche. As mentioned on the "Pushing metrics" page, they may just live behind a firewall (my case).
As, also mentioned on your site, push gateway is usually not the right solution. That's why I want to use AMQP.
With that said - I understand that adding AMQP might not be something you want to look into, and appreciate this.
So - a new suggestion: How about a plugin-like system for adding scrapers?
This can be done in two ways:

Actual plugin (some C/Go .so file to load with code doing the calls).
A more elaborate protocol and configurations for scraping using a proxy. So that we could have a single proxy process, which has no special configuration, it will be scraped by localhost http and forward the request in the target protocol. (Example changes needed: scraping of different processes with the same port. So that I could scrape two jobs by http scraping the same localhost:8080/metrics endpoint, but with different parameters (for example "?actual=amqps://whatever/")).

they may just live behind a firewall (my case).

We recommend running Prometheus on the same network as what you're monitoring.
We already support 2, see how the blackbox exporter works for example.

The one thing that I'm missing in the blackbox example, is a call for init.
I want my proxy to be called once, when prometheus loads, to create a single connection that I can use later.
I can obviously do it on first scrape call, but then it might take longer to connect than the timeout, and if it fails, I don't know if it's the scrape that failed or the proxy.
But - I think it's good enough for now. Thanks!

Closing this as we're not likely to add non-HTTP scraper support and a workaround has been found.

Greetings.
Even if this has been discarded I'd like to tell you why I need this today.
My company is going to deploy hundreds of servers at clients and we need to export the metrics of this servers and aggregate them in a single place. As prometheus provide no authentication we can't use it as it because we can not expose the push gateway on internet without any authentication.
So we thought about it and we already have an authenticated path between the servers at the clients and us, rabbitmq. So we are going to write something that is going to poll the node exporter locally on the servers and send the result to rabbitmq.
It would have been nice if promrtheus was able to natively poll rabbitmq queues, it would have spared us to develop something that polls queues and send the messages to the push gateway.
Regards.


Add option to read airlock password from env

Airlock password is read from AIRLOCK_PASSWORD env
If env is not set, user is prompted to input their password

Signed-off-by: Valtteri Valtia<EMAIL_ADDRESS>

some conflicts still need resolving otherwise lgtm


The Python Extension takes forever to load

Environment data

VS Code version: 1.50.1
Extension version (available under the Extensions sidebar): v2020.9.114305
OS and version:10.0.19042 Build 19042
Python version (& distribution if applicable, e.g. Anaconda):3.8.5 and Miniconda 3.8.3
Type of virtual environment used (N/A | venv | virtualenv | conda | ...):conda
Relevant/affected Python-related VS Code extensions and their versions: pylance ,itellisense
Value of the `python.languageServer setting: Pylance

Expected behaviour
To start normally
Actual behaviour
Takes forever to load

Steps to reproduce:
[not provided]

Logs

Thanks for the bug report! We investigate issues in order based on priority and severity, which includes the impact it has on your ability to use the extension to do productive work, and the number of people affected. If other users come forward and leave a comment demonstrating they are seeing/reproducing the problem then we will raise this issue's priority.
If you think your issue is more of a question or configuration problem rather than a bug, please ask on Stack Overflow with the visual-studio-code and python labels for help.
Thanks for your understanding and patience!

Hi @ericsnowcurrently  the issue is gone, I don't know how. So should I close this issue or let it be open. Since it may pop out in future.

Closing this. The extension went through a major change in past couple of months. So, if this issue occurs again the cause will like be entirely different.


Add functional tests for current express / node app

See for howto code:
https://github.com/caolan/nodeunit/pull/27
https://gist.github.com/621957

After much struggle have not managed to get nodeunti httputil to work with express (issue is starting the server).

After much struggle have not managed to get nodeunti httputil to work with express (issue is starting the server).

High priority as we cannot do other work (properly) such as #27 without this in place.

High priority as we cannot do other work (properly) such as #27 without this in place.


Have a version of toYAML that outputs pretty YAML.

generators.toYAML currently just outputs JSON:
https://github.com/NixOS/nixpkgs/blob/59b2ed7599c77be71f1d5a1bd05258c35aa28e89/lib/generators.nix#L198
While this is technically valid, it makes for some very hard to read config files.
To fix this, I've been copy-pasting this little package around:
to-yaml-file.nix
{ stdenv, writeText, haskellPackages }:
name:
config:
stdenv.mkDerivation {
  name = name;
  buildInputs = [ haskellPackages.json2yaml ];
  src = "${writeText "${name}.yaml" (builtins.toJSON config)}";
  buildCommand = ''
    json2yaml $src > $out
  '';
}

Is there any interest to upstream this?

It looks like this was addressed in PR #133807.


Update README.md

Added improvements in the text grammer.

Chenaged the grammer in order to correct it's grammer in english.


Add DevWorkspace-specific metrics

What does this PR do?
Adds DevWorkspace-specific metrics to the metrics server, tracking

Number of workspace starts
Number of successfully started workspaces (i.e. workspaces entering 'Running' phase)
Number of devworkspaces that enter the Failed state
A histogram of workspace startup time, bucketed in 10-second intervals

Currently, the metrics come with a source label. The value of this label is derived from the value of the controller.devfile.io/devworkspace-source label on the DevWorkspace (optional). If set, this allows metrics to be partitioned based on the tool creating the DevWorkspace (e.g. Web Terminal Operator, Eclipse Che, etc.)
Metrics are updated on phase changes in the DevWorkspace (i.e. going from phase 'Started' -> 'Running') to avoid double counting DevWorkspaces.
What issues does this PR fix or reference?
Additional nice-to-haves for https://github.com/devfile/devworkspace-operator/issues/241
Is it tested? How?
Changes can be tested in same way as https://github.com/devfile/devworkspace-operator/pull/405 (note metrics won't appear until they're updated at least once)
A simpler way to test these changes directly is by running the controller locally:

make run the controller in one window
kubectl apply -f samples/theia-next.yaml
curl -s http://localhost:8080/metrics | grep '^dev'

PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v7-devworkspaces-operator-e2e, v7-devworkspace-happy-path to trigger)

[ ] v7-devworkspaces-operator-e2e: DevWorkspace e2e test
[ ] v7-devworkspace-happy-path: DevWorkspace e2e test

/test v7-devworkspaces-operator-e2e, v7-devworkspace-happy-path


Item 11: Cloneable

Clone should return the same class of the defining class and do not  declare to throw CloneNotSUpportedException.
They should also not call not-final methods.

To do that we should be able to solve method calls


New version: Cthulhu v2.2.3

Registering package: Cthulhu
Repository: https://github.com/JuliaDebug/Cthulhu.jl
Created by: @aviatesk
Version: v2.2.3
Commit: b4bfe54153407a3953cb2cadada864e4dfe25ece
Reviewed by: @aviatesk
Reference: https://github.com/JuliaDebug/Cthulhu.jl/commit/b4bfe54153407a3953cb2cadada864e4dfe25ece#commitcomment-56290007

Closed by request


TypeError: Cannot read property 'extend' of undefined

Hi,
I was trying to use this today however when executing "yo aaal" the below TypeError is raised:
/usr/lib/node_modules/generator-aaal/app/index.js:13
module.exports = yeoman.Base.extend({
                            ^

TypeError: Cannot read property 'extend' of undefined
    at Object.<anonymous> (/usr/lib/node_modules/generator-aaal/app/index.js:13:29)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.get [as aaal:app] (/usr/lib/node_modules/yo/node_modules/yeoman-environment/lib/store.js:40:23)
    at Store.get (/usr/lib/node_modules/yo/node_modules/yeoman-environment/lib/store.js:64:35)

There was a fix (#5) merged today. Maybe you can try reinstalling the generator?

Thanks for the update, I had to install the generator directly from git:
npm install -g git+/https://github.com/johannesjo/generator-angular-auto-admin-loopback.git
npm couldn't find the update at the time
npm ERR! notarget No compatible version found<EMAIL_ADDRESS>npm ERR! notarget Valid install targets:
npm ERR! notarget 0.0.8, 0.0.7, 0.0.6, 0.0.5, 0.0.4, 0.0.3, 0.0.2, 0.0.1

Yes I didn't release a new version yet. I will do so!


[Bug]: verbatimModuleSyntax support

Version
v29.1.0
Steps to reproduce
I have upgraded to TS v5. I had to replace importsNotUsedAsValues: 'error' with verbatimModuleSyntax: true
Expected behavior
No error.
Actual behavior

ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled.

for import { xxx } from './xxx';
Debug log
https://gist.github.com/simPod/90d2228089b7a8b1b16a827f73aa71f2
Additional context
config
module.exports = {
  preset: 'ts-jest',
};

Environment
System:
    OS: macOS 13.2.1
    CPU: (10) arm64 Apple M1 Max
  Binaries:
    Node: 19.6.1 - /opt/homebrew/bin/node
    Yarn: 4.0.0-rc.42 - /opt/homebrew/bin/yarn
    npm: 9.4.0 - /opt/homebrew/bin/npm

Thanks for reply. I have never seen the docs since I've used jest without esm mode until now with no issues.
However, I've followed the docs. And getting the same error even wit modified config:
/** @type {import('ts-jest').JestConfigWithTsJest} */
module.exports = {
  moduleNameMapper: {
    '^(\\.{1,2}/.*)\\.js$': '$1',
  },
  preset: 'ts-jest/presets/default-esm',
  transform: {
    '^.+\\.tsx?$': [
      'ts-jest',
      { useESM: true },
    ],
  },
};

Sorry the doc was a bit outdated, there is warning about using transform and preset see https://kulshekhar.github.io/ts-jest/docs/getting-started/options
the doc for ESM needs to be updated accordingly.
the error you saw came from TS type checking similar to tsc which we can’t control which options require ESM

I have inspected default-esm preset which basically just sets extensionsToTreatAsEsm: ['.ts', '.tsx', '.mts'].
Not sure whether I'm missing something in your reply about we can’t control which options require ESM but I see the same error still.
/** @type {import('ts-jest').JestConfigWithTsJest} */
module.exports = {
  extensionsToTreatAsEsm: ['.ts', '.tsx', '.mts'],
  moduleNameMapper: {
    '^(\\.{1,2}/.*)\\.js$': '$1',
  },
  transform: {
    '^.+\\.tsx?$': [
      'ts-jest',
      { useESM: true },
    ],
  },
};

Would you please put a reproduce example with this https://github.com/kulshekhar/ts-jest/tree/main/examples/ts-only

on it

https://github.com/kulshekhar/ts-jest/pull/4082

Hi @ahnpnl ,
I'm still facing this issue.
I've tried the default preset:
{
  preset: 'ts-jest/presets/default-esm', // or other ESM presets
  moduleNameMapper: { '^(\\.{1,2}/.*)\\.js$': '$1' },
  testEnvironment: 'node',
  testMatch: ['<rootDir>/**/test/*.ts', '<rootDir>/**/test/types/*.ts'],
  collectCoverageFrom: ['src/**'],
}

and the manual config:
{
  extensionsToTreatAsEsm: ['.ts', '.tsx', '.mts'],
  transform: {
    '^.+\\.tsx?$': [
      'ts-jest',
      { useESM: true, isolatedModules: false },
    ],
  },
  moduleNameMapper: { '^(\\.{1,2}/.*)\\.js$': '$1' },
  testEnvironment: 'node',
  testMatch: ['<rootDir>/**/test/*.ts', '<rootDir>/**/test/types/*.ts'],
  collectCoverageFrom: ['src/**'],
}

However, none of them worked.
I keep getting jest.config.ts:12:1 - error TS1286: ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled.
Any suggestion?

@juanjoDiaz You can use this PR example to try out https://github.com/kulshekhar/ts-jest/pull/4083 note that, this option only works with ESM mode so you need to choose the correct jest config file in one of those examples

Hi @ahnpnl ,
Thanks for the quick response.
I tried
import preset from 'ts-jest/presets/index.js'

/** @type {import('ts-jest').JestConfigWithTsJest} */
export default {
  ...preset.defaultsESM,
  transform: {
    '^.+\\.tsx?$': [
      'ts-jest',
      {
        useESM: true,
      },
    ],
  },
}

as per the PR but I'm getting the same error.
Can you be a bit more specific on where the issue is and what setting in the config would allow me to treat the "jest.config.ts" as an esm module?

jest.config.ts is not handled by ts-jest but Jest itself does.

But Jest is a js-only solution, right?
If my jest config is a TS file, it must be parsed by ts-node isn't it?
That's where the issue seems to be.

Jest handles loading config while ts-jest handles transforming test files, source files which are used by test files. Indeed Jest uses ts-node to parse your config file.

Indeed.
So, the issue that I am seeing when parsing the jest config file using the verbatimModuleSyntax option is related to ts-node.
Isn't that related to ts-jest?
Or does jest include its own ts-node to parse config files?

I haven't been able to figure out the root cause of this issue, but after reviewing the /examples/ts-only folder I found that the config in jest-esm-isolated.config.mjs resolves the error for my projects.
Below is a summary of the changes that I made to get ts-jest working with TypeScript 5; hopefully this will help some folks who come across this issue in the future.
// ./package.json
// (trimmed to just the relevant parts)
"devDependencies": {
	"@types/jest": "^29.5.0",
	"jest": "^29.5.0",
	"ts-jest": "^29.1.0",
	"typescript": "^5.0.4"
}

// ./tsconfig.json
// (trimmed to just the relevant parts)
 		"esModuleInterop": true,
-		"importsNotUsedAsValues": "error",
-		"isolatedModules": true,
 		"module": "esnext",
 		"moduleResolution": "node",
 		"strict": true,
 		"target": "es2022",
+		"verbatimModuleSyntax": true
 	}
 }

// ./config/jest.config.js
 	transform: {
-		".ts": ["ts-jest"],
+		".ts": [
+			"ts-jest",
+			{
+				// Note: We shouldn't need to include `isolatedModules` here because it's a deprecated config option in TS 5,
+				// but setting it to `true` fixes the `ESM syntax is not allowed in a CommonJS module when
+				// 'verbatimModuleSyntax' is enabled` error that we're seeing when running our Jest tests.
+				isolatedModules: true,
+				useESM: true,
+			},
+		],
 	},

That solution doesn't work for me 🙁
I get the error when parsing the actual jest.config.ts file. So the solution has to be somewhere else.

this answer is useful.
But it seems to conflict with "moduleResolution": "bundler", it is invalid after adding

I haven't been able to figure out the root cause of this issue, but after reviewing the /examples/ts-only folder I found that the config in jest-esm-isolated.config.mjs resolves the error for my projects. More specifically, setting isolatedModules to true in the ts-jest transfom config object is what fixed the issue for me.
Below is a summary of the changes that I made to get ts-jest working with TypeScript 5; hopefully this will help some folks who come across this issue in the future.
// ./package.json
// (trimmed to just the relevant parts)
"devDependencies": {
	"@types/jest": "^29.5.0",
	"jest": "^29.5.0",
	"ts-jest": "^29.1.0",
	"typescript": "^5.0.4"
}

// ./tsconfig.json
// (trimmed to just the relevant parts)
 		"esModuleInterop": true,
-		"importsNotUsedAsValues": "error",
-		"isolatedModules": true,
 		"module": "esnext",
 		"moduleResolution": "node",
 		"strict": true,
 		"target": "es2022",
+		"verbatimModuleSyntax": true
 	}
 }

// ./config/jest.config.js
// (trimmed to just the relevant parts)
 	transform: {
-		".ts": ["ts-jest"],
+		".ts": [
+			"ts-jest",
+			{
+				// Note: We shouldn't need to include `isolatedModules` here because it's a deprecated config option in TS 5,
+				// but setting it to `true` fixes the `ESM syntax is not allowed in a CommonJS module when
+				// 'verbatimModuleSyntax' is enabled` error that we're seeing when running our Jest tests.
+				isolatedModules: true,
+				useESM: true,
+			},
+		],
 	},

Thanks, this answer is useful.
But it seems to conflict with "moduleResolution": "bundler" of tsconfig, and it will be invalid after adding.

If you use a jest.config.ts file name, you also need to fix the main  tsconfig :/
diff --git a/__tests__/tsconfig.json b/__tests__/tsconfig.json
--- /dev/null
+++ b/__tests__/tsconfig.json
@@ -0,0 +1,6 @@
+{
+	"extends": "../tsconfig.json",
+	"compilerOptions": {
+		"verbatimModuleSyntax": false
+	}
+}

diff --git a/jest.config.ts b/jest.config.ts
--- a/jest.config.ts
+++ b/jest.config.ts
@@ -55,6 +55,10 @@ const config: Config = {
 		// process `*.js` files with `babel-jest`
 		'^.+\\.js$': 'babel-jest',
 		'^.+\\.vue$': '@vue/vue2-jest',
+		'^.+\\.ts$': ['ts-jest', {
+			// @see https://github.com/kulshekhar/ts-jest/issues/4081
+			tsconfig: './__tests__/tsconfig.json',
+		}],
 	},
 	transformIgnorePatterns: [
 		'node_modules/(?!(' + ignorePatterns.join('|') + ')/)',

diff --git a/tsconfig.json b/tsconfig.json
--- a/tsconfig.json
+++ b/tsconfig.json
@@ -19,7 +19,8 @@
 		// these options are overrides used only by ts-node
 		// same as our --compilerOptions flag and our TS_NODE_COMPILER_OPTIONS environment variable
 		"compilerOptions": {
-			"module": "commonjs"
+			"module": "commonjs",
+			"verbatimModuleSyntax": false
 		}
 	}
 }

That solution doesn't work for me 🙁
I get the error when parsing the actual jest.config.ts file. So the solution has to be somewhere else.

yup - same here. the issue always happened when resolving any ESM jest config file (including ts config files), it gave the original error: ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled.
seems like the issue has to do with parsing the config file in ESM, but typescript is used (I believe) to do so, but TypeScript keeps complaining about "verbatimModuleSyntax" even when you specify a custom tsconfig config or config file in the "transform" key... soo the trick is to just disable the "verbatimModuleSyntax" TS config, BUT you can't use the new transform key, because that is exactly what is breaking, and so you have to use the now deprecated "globals" key instead, and it just worked for me:
// jest.config.cjs
/** @type {import('ts-jest').JestConfigWithTsJest} */
module.exports = {
+   globals: {
+     'ts-jest': {
+       useESM: true,
+       tsconfig: {
+         verbatimModuleSyntax: false,
+       },
+     },
+   },
- transform: {
-    '.ts': ['ts-jest', { tsconfig: './tsconfig.jest.json' }],
- },
  preset: 'ts-jest/presets/default-esm',
  testEnvironment: 'node',
};

when i run jest cli, I do get this annoying warning, but tests are running
> jest

ts-jest[ts-jest-transformer] (WARN) Define `ts-jest` config under `globals` is deprecated. Please do
transform: {
    <transform_regex>: ['ts-jest', { /* ts-jest config goes here in Jest */ }],
},
 PASS  test/test1.spec.ts
...

this method very simply disables the "verbatimModuleSyntax" tsconfig value to false when TS is used as a transformer in my tests files, but the normal tsconfig.json still works for the test files in my editor
it seems to me likely that ts-jest is the culprit here as there is strictly different behaviour happening with the globals and the transform keys. pretty clear ts-jest doesn't support "verbatimModuleSyntax" whatsoever in the like 12 permutations of settings i tried with ESM jest config file. i really did try eveyrthing in this thread and had to give up on "everything" being TS/ESM. is it so hard to use TS for everything and use jest? apparently yes it is haha
don't exactly have time to create a new issue from this, but is very reproducible if you just switch the place of the tsconfig object/path from transform to globals if anyone else wants to create a new issue for this ongoing bug

The solution from @seanblonien does not work for me as well. (It's February 2024). I decided to bury Jest and go with Vitest. In my Svelte project that works out of the box. The question remains as to how long this system will work.

If I have package.json containing "type": "module", node is complaining if .js is missing. Therefore tsconfig.jest.json has to have "verbatimModuleSyntax": true.
I have no cue how to fix this. Why is this issue closed?

That solution doesn't work for me 🙁
I get the error when parsing the actual jest.config.ts file. So the solution has to be somewhere else.

yup - same here. the issue always happened when resolving any ESM jest config file (including ts config files), it gave the original error: ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled.
seems like the issue has to do with parsing the config file in ESM, but typescript is used (I believe) to do so, but TypeScript keeps complaining about "verbatimModuleSyntax" even when you specify a custom tsconfig config or config file in the "transform" key... soo the trick is to just disable the "verbatimModuleSyntax" TS config, BUT you can't use the new transform key, because that is exactly what is breaking, and so you have to use the now deprecated "globals" key instead, and it just worked for me:
// jest.config.cjs
/** @type {import('ts-jest').JestConfigWithTsJest} */
module.exports = {
+   globals: {
+     'ts-jest': {
+       useESM: true,
+       tsconfig: {
+         verbatimModuleSyntax: false,
+       },
+     },
+   },
- transform: {
-    '.ts': ['ts-jest', { tsconfig: './tsconfig.jest.json' }],
- },
  preset: 'ts-jest/presets/default-esm',
  testEnvironment: 'node',
};

when i run jest cli, I do get this annoying warning, but tests are running
> jest

ts-jest[ts-jest-transformer] (WARN) Define `ts-jest` config under `globals` is deprecated. Please do
transform: {
    <transform_regex>: ['ts-jest', { /* ts-jest config goes here in Jest */ }],
},
 PASS  test/test1.spec.ts
...

this method very simply disables the "verbatimModuleSyntax" tsconfig value to false when TS is used as a transformer in my tests files, but the normal tsconfig.json still works for the test files in my editor
it seems to me likely that ts-jest is the culprit here as there is strictly different behaviour happening with the globals and the transform keys. pretty clear ts-jest doesn't support "verbatimModuleSyntax" whatsoever in the like 12 permutations of settings i tried with ESM jest config file. i really did try eveyrthing in this thread and had to give up on "everything" being TS/ESM. is it so hard to use TS for everything and use jest? apparently yes it is haha
don't exactly have time to create a new issue from this, but is very reproducible if you just switch the place of the tsconfig object/path from transform to globals if anyone else wants to create a new issue for this ongoing bug

Can you point at an example repository where this is working?

Just commenting here in case most of you are using verbatimModuleSyntax as a linting mechansim so that your eslint auto fixes your imports to type only imports.
https://johnnyreilly.com/typescript-5-importsnotusedasvalues-error-eslint-consistent-type-imports
If you are, then it appears that you only need:
      "rules": {
        "@typescript-eslint/consistent-type-imports": "error"
      }

can can avoid all these errors by not using that option.
yes i just suggested fixing the problem by turning the errors off. 😅

jest.config.ts is not handled by ts-jest but Jest itself does.

More precisely transpiled by 'ts-node'.

This is still an issue in Aug 2024 FYI. Specifically ts-jest does not properly parse jest.config.ts files when "verbatimModuleSyntax": true,

Why is this issue closed as "completed" ?
Suppose you have an ESM module (package.json contains "type": "module") with the following in tsconfig (no path options) :
    "module": "nodenext",
    "moduleResolution": "nodenext",
    "verbatimModuleSyntax": true,
    "esModuleInterop": true

Your jest config (defined in jest.config.js, ie. ts-node not involved), as recommended by the doc (except for the CJS syntax obviously, and without the unnecessary / redundant settings that are already [pre]set) :
export default {
  preset: 'ts-jest/presets/default-esm'
};

You run your tests and get the error : "ESM syntax is not allowed in a CommonJS module when 'verbatimModuleSyntax' is enabled" 😮.
Adding isolatedModules: true - thanks @dustin-ruetz - makes the error go away :
export default {
  preset: 'ts-jest/presets/default-esm',
  transform: {
    '.ts': [
      'ts-jest',
      {
        isolatedModules: true
      }
    ]
  }
};

😮
Would it not be a bug ?
We shouldn't have to set isolatedModules: true or verbatimModuleSyntax: false to get rid of an error that says our ESM module is a CJS module.
This is not a feature request.

The original report is about jest config in typescript doesn’t work with this option. We closed this issue because ts-jest doesn’t handle that.
However, your issue is a different one which is related to https://github.com/kulshekhar/ts-jest/issues/4198

We closed this issue because ts-jest doesn’t handle that.

Ok, great.
So why the error doesn't reflect that with a clear message : ts-jest doesn’t handle "verbatimModuleSyntax" ?
Rather than ESM syntax is not allowed in a CommonJS module when "verbatimModuleSyntax" is enabled ?

However, your issue is a different one which is related to https://github.com/kulshekhar/ts-jest/issues/4198

How is it related ?

verbatimModuleSyntax is not mentioned at all in the issue description nor in the comments.
people say they only encounter the issue with isolatedModules: true, the exact opposite of what people say here.

I think there are 2 things in this issue: one is about Jest config in typescript doesn't work with verbatimModuleSyntax and one is about transforming issue. At some point, I thought the original issue was about working with jest.config.ts
If you have issues with your jest.config.ts, ts-jest can't solve it unfortunately.
If you have transforming issue, for example your other ts files, would you pls provide an example in any of the projects here https://github.com/kulshekhar/ts-jest/tree/main/examples. Based on your input, we can repopen this issue and fix it properly.

I guess I found a solution.

Install ts-jest and ts-node.

Create a separate TypeScript configuration file for Jest. Let’s name it tsconfig.jest.json.
{
  "compilerOptions": {
    "esModuleInterop": true,
    "moduleResolution": "node10",
    "verbatimModuleSyntax": false
  }
}

(Alternatively, you can set moduleResolution to bundler. Strategies other than bundler or node10 won’t work.)

Assign the path to the file to the TS_NODE_PROJECT environment variable just before running Jest.
Bash:
TS_NODE_PROJECT='./tsconfig.jest.json' npx jest

CMD:
set "TS_NODE_PROJECT=./tsconfig.jest.json" npx jest

Alternatively, for a cross-platform implementation, add
"scripts": {
  "test": "cross-env TS_NODE_PROJECT=\"./tsconfig.jest.json\" jest"
}

to package.json, install cross-env and run npm test.

This solution will enable ESM syntax in jest.config.ts only. You should also follow this guidance to enable ESM syntax in test modules.


Add source_url_pattern to enable linking to source

Just adding the source_url_pattern option for exdoc to opentelemetry_oban, to enable the "Link to source" button in the docs.

Happy to merge this, but figured I should ask if you'd be able to do this for the other apps in the repo :)

Sure! It became a small yak shave expedition. I've now also

Add README.md as an "extra" where it wasn't already
Add a main setting. They all have a very obvious main module. Set
that as main, so a user is shown this immediately instead of a list
of usually only this module.


Namespace DYLD, Code 1 Library missing

How frequently does the bug occur?
Always
Description
I am having issues with my app crashing on launch on macOS. I have searched for a solution by looking through other issues on here and tried the recommended suggestions.
As suggested by in other places, only Realmswift is imported, I have deleted derived data and made sure realm swift is signed and embedded.
When the app is built and ran through Xcode, it works fine. But on TestFlight and debugging version created through Archive, it crashes on load, and I don't know what else to do to get it to work.

Stacktrace & log output
Termination Reason:    Namespace DYLD, Code 1 Library missing
Library not loaded: @rpath/RealmSwift.framework/Versions/A/RealmSwift
Referenced from: <D46A7415-2DF7-3D38-BDE9-D3C3F882F308> /Users/USER/Downloads/*/myApp.app/Contents/MacOS/myApp
Reason: tried: '/Users/*/Downloads/myApp 2024-07-06 15-43-00/myApp.app/Contents/MacOS/Frameworks/RealmSwift.framework/Versions/A/RealmSwift' (no such file), '/Users/*/Downloads/myApp 2024-07-06 15-43-00/myApp.app/Contents/MacOS/Frameworks/RealmSwift.framework/Versions/A/RealmSwift' (no such file)
(terminated at launch; ignore backtrace)

Can you reproduce the bug?
Always
Reproduction Steps
Archive a project and then upload to testflight or choose debugging as a means of distribution.
Version
10.52.1
What Atlas Services are you using?
Atlas Device Sync
Are you using encryption?
No
Platform OS and version(s)
Mac Silicon - Sonoma
Build environment
Xcode version: 15.4
Dependency manager and version: SPM

Am I really the only one still having this issue?

@dejiapps  did you find a solution? I just added realm-swift on macOS and see the same issue

@dejiapps did you find a solution? I just added realm-swift on macOS and see the same issue

So the initial solution that worked for me was reverting back to version 10.49.2 for the time being. But with the deprecation of realm sync, I have removed realm for the app I was working on and I'm just going with SwiftData. Hopefully in the future, I can figure out a good solution for ios-android syncing.
I'm just glad I had not released my app yet.


parse AV values as single values, not lists

Resolves #55.
Due to the library not using type hints, I'm not sure if I resolved all occurrences yet, so this is a draft for now.

I added fixes for the --gen-md option, should be good to go now :)


Crash in IDEA 2022.2.3

java.lang.NoClassDefFoundError: com/intellij/sh/SHIcons
at com.fapiko.jetbrains.plugins.better_direnv.language.filetype.DirenvFileType.getIcon(DirenvFileType.java:38)
Seems it needs adaption to the new IDEA version.

This is a duplicate of https://github.com/Fapiko/intellij-better-direnv/issues/16

@Fapiko Hi 👋🏽
I'd like to try out your plugin but I cannot as it doesn't work with the latest version of PyCharm, due to this issue 👆🏽
Could you cut a new version that includes the fix please?
Thanks,
Craig


Uniformative runtime error(s)

I feel like we addressed these types of issue; and it seems like the error message(s) are rather unuseful, e.g., in this trace the file exists on the HD so what exactly is the problem?

[provenance:info] Mnefun operation: apply_preprocessing_combined
Traceback (most recent call last):
File "/home/ktavabi/Projects/ilabs/analysis_scripts/shiva.py", line 163, in
print_status=True
File "/home/ktavabi/Projects/mnefun/mnefun/_mnefun.py", line 558, in do_processing
p.on_process(text, func, outs[ii], p)
File "/home/ktavabi/Projects/niprov/niprov/mnefunsupport.py", line 68, in handler
provenance.log(pcafile, trans, sssfile, provenance=customprov)
File "/home/ktavabi/Projects/niprov/niprov/context.py", line 26, in log
return niprov.logging.log(*args, dependencies=self.deps, **kwargs)
File "/home/ktavabi/Projects/niprov/niprov/logging.py", line 90, in log
listener.unknownFile(parents[0])
File "/home/ktavabi/Projects/niprov/niprov/commandline.py", line 45, in unknownFile
self.log(level, 'Unknown file: '+fpath, UnknownFileError)
File "/home/ktavabi/Projects/niprov/niprov/commandline.py", line 76, in log
raise exceptionClass(message)
niprov.exceptions.UnknownFileError: Unknown file: /mnt/ALAYA/data/ilabs/shiva/meg/shiva_1081/sss_fif/shiva_1081_a_raw_sss.fif

It means the parent file has not been added/discovered to provenance yet

The ass file in this case

Excuse me, the what file?

Haha oops

We could make the error about an unknown parent optional. Or check in the handler if the parent is known, and otherwise not specify a parent.

We could throw a warning the parent file is missing and search for it before stopping the process?

Search for it automatically? Or prompt the user to search for it?
Is it common in menfun to skip the early pipeline steps?

And I could actually do that if I could actually understand what it is your doing in the code. Everything is constantly referring to something else!

I'd say so, the processes are linear.


Ft person profile

What does this PR do?

creates Person Profile Page

Description of Task to be completed?

create profile component
create person profile route
create error toastify message
styled person profile for desktop view

How should this be manually tested?

clone the repo
npm install
npm start
go to http://localhost:3000/profile

What are the relevant Trello Card stories?  Trello Card Item

Person Profile Component

Pull Request Type (check one)

[ ] Feature

Screenshots (if appropriate)

I hope the comments helped, feel free to hit me up if you have any questions. You can also run yarn lint to see what linting rules need fixing or I think you can use yarn lint:fix ( I would review the changes that yarn runs after this comment)

I ran yarn lint and I am seeing some lint errors on some of the files that have changes, if we could fix those then we are set.
/Users/steven/personal/STN/web/src/components/profile/Button/Button.js
   5:22  error  Unexpected block statement surrounding arrow body; move the returned value immediately after the `=>`  arrow-body-style
  11:2   error  Missing semicolon

/Users/steven/personal/STN/web/src/components/profile/PersonProfile.js
  73:2  error  Missing semicolon  semi

/Users/steven/personal/STN/web/src/pages/PersonProfile.js
  41:2   error  Missing semicolon                              semi
  43:29  error  Newline required at end of file but not found  eol-last
  43:29  error  Missing semicolon

In the husky setup, should we be running yarn lint:fix instead of yarn lint so some of the

I ran yarn lint and I am seeing some lint errors on some of the files that have changes, if we could fix those then we are set.
/Users/steven/personal/STN/web/src/components/profile/Button/Button.js
   5:22  error  Unexpected block statement surrounding arrow body; move the returned value immediately after the `=>`  arrow-body-style
  11:2   error  Missing semicolon

/Users/steven/personal/STN/web/src/components/profile/PersonProfile.js
  73:2  error  Missing semicolon  semi

/Users/steven/personal/STN/web/src/pages/PersonProfile.js
  41:2   error  Missing semicolon                              semi
  43:29  error  Newline required at end of file but not found  eol-last
  43:29  error  Missing semicolon

This was caused by another pull request that was merged, we need to setup CI for linting I guess

Approved! Looks good, squash and merge.


[BugFix] fix dynamic pooling for evaluation

Codecov Report

Merging #609 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #609      +/-   ##
==========================================
+ Coverage   94.31%   94.32%   +0.01%
==========================================
  Files          60       60
  Lines        2165     2169       +4
==========================================
+ Hits         2042     2046       +4
  Misses        123      123

Impacted Files
Coverage Δ

matchzoo/models/match_pyramid.py
100% <ø> (ø)
:arrow_up:

matchzoo/layers/dynamic_pooling_layer.py
91.17% <100%> (+1.17%)
:arrow_up:

matchzoo/data_generator/dpool_data_generator.py
92.3% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

This will conflict with 2.0-dev. If you're changing the source code, could you please make the changes on 2.0-dev instead? (I thought you wanted to change the tutorials)


Include libav-tools in Docker image

Description:
The ffmpeg component requires either ffmpeg or avconv, so one of these should probably be included in the Docker container.
Related issue (if applicable): n/a
Pull request in home-assistant.github.io with documentation (if applicable): n/a
Example entry for configuration.yaml (if applicable): /na
Checklist:
n/a

I'm not fan of libav since it die and debian/ubuntu going back to ffmpeg. The liav stay to day of fork and ffmpeg make big features and most imported he close all secure holes and libav have big holes. But I don't use docker so the maintener of docker have to deside.

Indeed, ffmpeg is the right way to go. See this page: https://wiki.debian.org/Debate/libav-provider/ffmpeg

Closing this in favor of #5322 which installs ffmpeg from jessie-backports.


Power of Three

Problem Link
Mention the problem on which you want to work on
https://leetcode.com/problems/power-of-three/
Coding platform
Mention the coding platform from where you have picked the problem
LeetCode
Programming language

[ ] C
[ ] C++
[x] Java
[ ] Python

Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.

I would like to work on this issue in java as a part of Cross-Woc. Kindly assign it to me.

I would like to work it using python. Can you please assign it to?

I would like to work on this in C++


Nice to have: touch gestures

...for navigating between posts within a thread or report on mobile or tablet devices.
Will require extra JS.

do you mean like swipe for next post sort thing?
Joseph McArthur
Assistant Director: Right to Research Coalition ( http://righttoresearch.org )
Director / Founder: Open Access Button ( http://openaccessbutton.org )
Twitter: @Mcarthur_Joe ( https://twitter.com/Mcarthur_Joe )
Skype: joseph_mcarthur
On Wed, Jun 17, 2020 at 1:54 PM, Sophy <<EMAIL_ADDRESS>> wrote:

...for navigating between posts within a thread or report on mobile or
tablet devices.
Will require extra JS.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub (
https://github.com/sparcopen/infrastructure/issues/69 ) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/ABFZMZ2LTCLA26OIOGZNCTLRXC4JVANCNFSM4OASJPWQ
).

Yep!

neat, yeah, nice to have.
Joseph McArthur
Assistant Director: Right to Research Coalition ( http://righttoresearch.org )
Director / Founder: Open Access Button ( http://openaccessbutton.org )
Twitter: @Mcarthur_Joe ( https://twitter.com/Mcarthur_Joe )
Skype: joseph_mcarthur
On Wed, Jun 17, 2020 at 1:57 PM, Sophy <<EMAIL_ADDRESS>> wrote:

Yep!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
https://github.com/sparcopen/infrastructure/issues/69#issuecomment-645356725
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/ABFZMZ42UDEY6ENANFOLCZ3RXC4SDANCNFSM4OASJPWQ
).


Database setup about postgresql refering old third-party image built 5 years ago

Change Type
Cleanup (formatting, typos, etc.)
Area
Database Setup
Proposed Changes
Please update

@WestFarmer the project doesn't have enough resources right now to even cover critical priorities. For that reason, please don't submit a documentation update request that doesn't directly offer help with the solution. It's unproductive.
Thank you for your report.


Dashboard Error -- AD DNS Object Summary (7d)

I ahve this error on the Dahsboard -- AD DNS Object Summary (7d)
Cannot parse 'EventID:5137 AND ObjectClass:dnsNode AND created AND NOT (SubjectUserName:$ OR SubjectUserName:SYSTEM OR SubjectUserName:-)': '' or '?' not allowed as first character in WildcardQuery

Do you have leading wild card searches enabled?  If that resolved the issue I'll have to add it to the requirements
allow_leading_wildcard_searches = true


✨ Add include support

Changes

✨  Added support for #include directive.

Source code based on https://github.com/ricardomatias/esbuild-plugin-glsl-include.

✅ Added test support for #include directive.

Feel free to edit whatever you want, I'm no expert in TypeScript or Ava, but I've checked that it works for me.

Hello @vanruesc, what do you think about this change? If you don't like it, we're going to continue with our fork. We think this is a good enhancement for the project and we'd like it to upstream the patch, but of course if you don't want/need this feature we are ok to continue with our custom fork. Let us know what you think.
Thanks!

Hi, thanks for contributing!
I think this is a nice feature and the PR looks good. However, the syntax seems a bit too similar to three's include system (#include <shader_chunk>). I'm not sure if this would be a problem though; need to think about it. We could consider an alternative include syntax to reduce ambiguity or we could add a setting like resolveIncludes to let the user opt into the new behaviour :thinking:

Hi @vanruesc, it's a pleasure to contribute!
The include pattern used by ThreeJS is /^[ \t]*#include +<([\w\d./]+)>/gm. It's very similar to the one we are using, but the main difference is that they use < and > instead of " or ' as we do.
I've updated the regular expression to make it:

More robust. Like ThreeJS, more spaces are allowed between #include and " or '.
Less redundant. \w already contains _.
Case sensitive.

About the ambiguity, unless ThreeJs changes its pattern, I think there should be no problem because the patterns do not overlap. In fact, as far as I've seen, they are at different stages of the pipeline and our #include is processed before the ThreeJS's #include.

In fact, as far as I've seen, they are at different stages of the pipeline and our #include is processed before the three's #include.

Right, but that's just a coincidence. If someone else had a custom shader include system in place that used " instead, then this new behaviour would break their code. This feature introduces potential conflicts that wouldn't otherwise exist.
I'll merge this anyway because it's useful, but I'll add an option to disable the feature.

Hi @vanruesc, I want to use this new feature but I think that it won't be available until you make a release. I thought I could reference this repository instead of the npm package as below:
"esbuild-plugin-glsl": "https://github.com/vanruesc/esbuild-plugin-glsl.git",

However, as there is no dist folder it doesn't work. What is the best approach in your opinion?

Sorry for the delay! The changes are now included in<EMAIL_ADDRESS>

The unit test works as expected :thinking:  Can you specify what doesn't work exactly?

Oops sorry, I didn't see your reply and created the PR. Check it out if you want or feel free to do it by yourself @vanruesc!

#8

Should be fixed in<EMAIL_ADDRESS>I've also updated the unit test to catch this use case and noticed another bug that would sometimes cause the test to fail because the regular expressions weren't reset properly after the refactoring.

Yes, it works like a charm, thanks for the effort!


Please help me! I do not know how to solve this problem

Does your machine not have a cuda environment... do you want to try it with linux?

Does your machine not have a cuda environment... do you want to try it with linux?

I have a nvidia 3080 and cuda version is 12.3 run on windows 11. I do not know what happens in my installation.

windows not supported

You can use vLLM on Windows via WSL

You can use vLLM on Windows via WSL

Doesn't wsl2 not support nccl? I tried it on windows10, but it always reports errors related to nccl, even with libnccl installed.


Roadrunner Fully Tuned

PR REQUEST TEMPLATE V1
Please fill out all sections to the best of your ability to help approve PRs.
Q1. Description(Less than 30 words) - What did you change?
I tuned RR on the current robot.
Q2. Did this add/change functionality to the robot(ex. add something to teleop/auto, change gamepad mappings, etc)? If so, explain.
Yes. RR has been imported, and tuned.
Q3. If Q2 is "yes", was this added functionality tested? If so, how? Explain.
Yes, it was tuned completely. Testing will be done later today.
Q4. Is there anything that should be noted? Is this software complete, or only parts of it?
Needs to be retuned once robot is changed.

From your testing, how accurate was the tuning?

The testing wasn't rigorous, but it was around 4 percent.

I will accept this merge, but the tuning later on needs to be better. The error should be around 0.5%.


error[E0433]: unresolved import ipc

It should be imported for linux and windows, what os are you on?

Anyways its deprecaded since v0.2.0, use use DiscordIpcClient::new() instead

I still have the same problem, although the package works, it doesn't work when I compile to WASI
Is there a way to use it in WASI?


copr: Vectorize AddDatetimeAndString

Signed-off-by: Drumil Patel<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: Partially fixes #9016
Problem Summary: Added Vectorize version for AddDatetimeAndString
What is changed and how it works?
What's Changed: Implmented Vectorize vesion for AddDatetimeAndString
Check List
Tests

Unit test

Release note

Partially fixes #9016

/cc @Xuanwo @andylokandy

/cc @Xuanwo

/merge

Please don't use fixes #9016 in issue, or #9016 will be closed.


Issue and Pr_templates.md file

There are nor issue_templates.md template neither Pr_template.md can I add them??

@HarshCasper ???


Too many chained references for LINQ

LINQ statements always show the ReSharper warning "Too many chained references. This might be violating the law of Demeter)
Is there an easy way to avoid this warning, other than to "Disable with comment".
var cassettesQuery = _db.Denominations
            .Include(d => d.Currency)
            .Where(d => ContainsValidDispensingNotes(d, _dispenseDenominationValues))
            .Select(d => CreateCassette(d));

This is a revisit of the issues mentioned below
https://github.com/hhariri/CleanCode/issues/15
https://github.com/hhariri/CleanCode/issues/20
https://github.com/MO2k4/CleanCode/issues/23

Another way would be to increase the value in the configuration. I can take another look at this and see if it is possible to know if the methods are chained via linq, but as already mentioned in the other issue, that could lead to other nasty side effects.

I changed the config value from 3 to 4 and the warning on the above example went away.
Am working on my other LINQ statements to try and get it down to 4. And it is actually is making my LINQ more readable and modular.
Thanks for this extension.


fix: make pre-push opt-in

The pre-push package always runs npm test even if there is no pre-push
section in your package.json. By switching to the git-validate package,
running pre push hooks is completely opt-in. Nothing will be run if there
is no pre-push section.
This doesn't change the behaviour of repositories that have some pre-push
targets set, as git-validate uses the same syntax as pre-push.
This change relates to https://github.com/ipfs/aegir/issues/242.

I've tested what happens with the pre-push hooks when you upgrade AEgir with this patch applied. It will correctly remove the pre-push package hook and install the git-validate hook. So it seems to be the perfect replacement.

@VictorBjelkholm can you merge and release this one?


There is no reset button which can restore to the default sort.

@edwardwang0302, I think this feature is not very important and not necessary for most people. I think you can add a button and restore the sortin state by yourself. If you dont know how to do, feel free to ask me :)  👍