[SPARK-38089][CORE][TESTS] Show the root cause exception in TestUtils.assertExceptionMsg

What changes were proposed in this pull request?
Improve assertion failure message in TestUtils.assertExceptionMsg to print out the exception tree in which it was searching (upon failure).
Why are the changes needed?
TestUtils.assertExceptionMsg is great, but when the assertion doesn't match, it can be challenging to tell why, because the exception tree that was searched isn't printed. Only way I could find to fix it up was to run things in a debugger and check the exception tree. This makes it easier to tell what went wrong just from the assertion failure message.
Does this PR introduce any user-facing change?
No. Easier for devs to see why their test failed.
How was this patch tested?
Used extensively while writing the tests for PR #34009. It was very useful!
For example, let's say I had a typo in the assertion for the test case AvroSuite.SPARK-34133: Writing user provided schema respects case sensitivity for field matching. The failure would look like:
java.lang.AssertionError: assertion failed: Exception tree doesn't contain the expected exception of type
org.apache.spark.sql.avro.IncompatibleSchemaException with message: Cannot find field "FOO" in Avro schema
	at scala.Predef$.assert(Predef.scala:223)
	at org.apache.spark.TestUtils$.assertExceptionMsg(TestUtils.scala:269)
	at org.apache.spark.sql.avro.AvroSuite.$anonfun$new$175(AvroSuite.scala:1409)
	...

So I know that I'm wrong, but I can't tell why. Now, after the change:
java.lang.AssertionError: assertion failed: Exception tree doesn't contain the expected exception of type
org.apache.spark.sql.avro.IncompatibleSchemaException with message: Cannot find field "FOO" in Avro schema
org.apache.spark.sql.avro.IncompatibleSchemaException: Cannot find field 'FOO' in Avro schema
	at org.apache.spark.sql.avro.AvroUtils$AvroSchemaHelper.$anonfun$validateNoExtraCatalystFields$1(AvroUtils.scala:265)
	at org.apache.spark.sql.avro.AvroUtils$AvroSchemaHelper.$anonfun$validateNoExtraCatalystFields$1$adapted(AvroUtils.scala:258)
	at scala.collection.immutable.List.foreach(List.scala:431)
	at org.apache.spark.sql.avro.AvroUtils$AvroSchemaHelper.validateNoExtraCatalystFields(AvroUtils.scala:258)
	...

	at scala.Predef$.assert(Predef.scala:223)
	at org.apache.spark.TestUtils$.assertExceptionMsg(TestUtils.scala:266)
	at org.apache.spark.sql.avro.AvroSuite.$anonfun$new$175(AvroSuite.scala:1409)
	...

I can easily see that I used the wrong type of quotes. Much better!

In addition, please include some example into the PR description. I have no idea how helpful it was to you.

Weird, GA is definitely enabled on my fork, I have used it many times. I pushed again a few minutes ago and it seems to be working again.
I also updated the description per your request.

Shouldn't be any change in log size unless there are test failures, since this will only print anything extra on assertion failure (which would cause a test failure). But of course let's wait for CI.

Looks like the test failures are some unrelated issues in the yarn tests:
[info] - run Spark in yarn-client mode with different configurations, ensuring redaction *** FAILED *** (3 minutes, 0 seconds)
[info]   The code passed to eventually never returned normally. Attempted 190 times over 3.0012781688500003 minutes. Last failure message: handle.getState().isFinal() was false. (BaseYarnClusterSuite.scala:204)
....
[info] - run Spark in yarn-cluster mode with different configurations, ensuring redaction *** FAILED *** (3 minutes, 0 seconds)
[info]   The code passed to eventually never returned normally. Attempted 190 times over 3.001207979333333 minutes. Last failure message: handle.getState().isFinal() was false. (BaseYarnClusterSuite.scala:204)
....
[info] *** Test still running after 2 minutes, 55 seconds: suite name: YarnClusterSuite, test name: yarn-cluster should respect conf overrides in SparkHadoopUtil (SPARK-16414, SPARK-23630).
[info] - yarn-cluster should respect conf overrides in SparkHadoopUtil (SPARK-16414, SPARK-23630) *** FAILED *** (3 minutes, 0 seconds)
[info]   The code passed to eventually never returned normally. Attempted 190 times over 3.0010310271500003 minutes. Last failure message: handle.getState().isFinal() was false. (BaseYarnClusterSuite.scala:204)
....
[info] - SPARK-35672: run Spark in yarn-client mode with additional jar using URI scheme 'local' *** FAILED *** (3 minutes, 0 seconds)
[info]   The code passed to eventually never returned normally. Attempted 190 times over 3.0009631187333334 minutes. Last failure message: handle.getState().isFinal() was false. (BaseYarnClusterSuite.scala:204)

I'll re-trigger.

Thank you for re-triggering. I agree with you. I'll merge this when the CI passes, @xkrogen .

Merged to master for Apache Spark 3.3.

BTW, for new feature and improvement JIRAs, Affected Version is master branch's version because Apache Spark doesn't allow backporting of new features and improvements.

Got it, thanks for the tip and for the review!


printToPDF crashes renderer if the shared memory is too small

When rendering PDF with latest Electron v0.36.7 running Debian Jesse on Docker v0.10.1,
the render process crashes.
This happens due to small 64MB (the default for Docker) /dev/shm shared memory.
Is it possible to improve the behaviour with some meaningful error message on Electron side?
Fortunately it is possible to increase the tempfs size in Docker 0.10 with --shm-size flag. So there is a workaround.
Stack:
Received signal 7 BUS_ADRERR 7fac3055e29c
#0 0x000001c8908b <unknown>
#1 0x7fac45d098d0 <unknown>
#2 0x7fac3fba0135 <unknown>
#3 0x0000020e0e4d <unknown>
#4 0x00000300bca3 <unknown>
#5 0x0000030784e0 printing::PrintWebViewHelper::CopyMetafileDataToSharedMem()
#6 0x00000307813d printing::PrintWebViewHelper::FinalizePrintReadyDocument()
#7 0x000003077b39 printing::PrintWebViewHelper::CreatePreviewDocument()
#8 0x000003077a5b printing::PrintWebViewHelper::OnFramePreparedForPreviewDocument()
#9 0x000003077957 printing::PrintWebViewHelper::PrepareFrameForPreviewDocument()
#10 0x000003076a74 printing::PrintWebViewHelper::OnMessageReceived()
#11 0x0000011fd3a6 <unknown>
#12 0x000002310623 <unknown>
#13 0x0000023105c9 <unknown>
#14 0x000001563604 <unknown>
#15 0x000002f35622 <unknown>
#16 0x000001c8a453 <unknown>
#17 0x00000054a4ae <unknown>
#18 0x000000548fee <unknown>
#19 0x000001c8a453 <unknown>
#20 0x000001ca5abb <unknown>
#21 0x000001ca5dc9 <unknown>
#22 0x000001ca5fad <unknown>
#23 0x000001ca72da <unknown>
#24 0x000001cb91b0 <unknown>
#25 0x000001ca53aa <unknown>
#26 0x00000121db94 <unknown>
#27 0x0000010cd8f7 <unknown>
#28 0x0000010ce691 <unknown>
#29 0x0000010cd570 <unknown>
#30 0x00000305027c main
#31 0x7fac3fb2fb45 __libc_start_main
#32 0x0000005354d9 <unknown>
  r8: ffff8f9520016e0c  r9: ffff8f9520016dfc r10: ffff8f9520016dec r11:<PHONE_NUMBER>121b40
 r12: 00007fac3055deac r13:<PHONE_NUMBER>728090 r14:<PHONE_NUMBER>000000 r15: 00000f414e877d80
  di: 00007fac3055deac  si: 00000f4150575018  bp:<PHONE_NUMBER>1151e4  bx:<PHONE_NUMBER>000400
  dx:<PHONE_NUMBER>000400  ax: ffff8f9520016d6c  cx:<PHONE_NUMBER>000800  sp: 00007fff6e3b35d8
  ip: 00007fac3fba0135 efl:<PHONE_NUMBER>010202 cgf:<PHONE_NUMBER>000033 erf:<PHONE_NUMBER>000006
 trp:<PHONE_NUMBER>00000e msk:<PHONE_NUMBER>000000 cr2: 00007fac3055e29c
[end of stack trace]

Unfortunately shared memory is a requirement of Chromium for passing the PDF data between processes, if you really care about this I would suggest reporting this to Chromium's issues list instead, but honestly I don't think anyone would look into it.
Closing as won't fix.


Issue on EventHeader serialization since it is internal class

Dear Denikson,
on revision commit 4d79afa76be937445ce1b82e71b7734d3788c9de , EventHeader is now internal class (instead of public class). And now SerializerUtils.Serialize fires "System.Runtime.Serialization.SerializationException : 'Non-public type 'ZeroRpc.Net.Data.EventHeader' cannot be serialized.'"
I tried to put public instead of internal and it solves the problem.
Why such a change?
Best Regards
Boris

No worries.  And suprisingly, this also solved a heartbeat error that appeared on my codes and that I was unabled to diagnose.
Thanks


FRDR repository expanded metadata

Display FRDR's datacite_contributor fields as part of "contributor". Related Identifiers are now part of domain metadata, but nameIdentifier and funder fields are excluded for now.
Bug fixes:

an exception other than AttributeError or StopIteration stopped all harvesting for an OAI repository; now it just fails for that specific record
in construct_local_url, don't reference local_url when it may not have been assigned yet

Expanding metadata for oai repository harvest to include "contributor" and fixed some bugs in DB Interface and and OAIRepository


Fix navbar graphql

Closes #5422
What's Changed

main-content area for the graphql editor is made collapse capable and expandable depending on the side navbar

Screenshots
Before:

After:

TODO

[x] Explanation of Change(s)
[x] Added change log fragment(s) (for more information see the documentation)
[x] Attached Screenshots, Payload Example
[x] Unit, Integration Tests
[x] Documentation Updates (when adding/changing features)
[x] Example Plugin Updates (when adding/changing features)
[x] Outline Remaining Work, Constraints from Design
[x] fix applied to /graphql/
[ ] fix applied to /api/
[ ] fix applied to /admin/

Would you mind addressing the DRF API views as well (https://github.com/nautobot/nautobot/issues/5422#issuecomment-1995766507)?

Would you mind addressing the DRF API views as well (#5422 (comment))?

Yes.  This is the /api/ endpoint.  (Takes you to Django rest framework “drf”. I added it to the TODO section of the PR and will most definitely address it.

@glennmatthews - This is ready to review again with the footer fixes along with the /api/, /admin, and /graphql/ pages.  I was also able to remove a lot of redundant code in some of the templates along with fixing a few errors in the javascript imports that was happening under the hood for the admin page


A question about meta-learning few-shot NER

Hi, I have a question about ablation study in Table 4 1) w/o MAML.
I run the code to get the results of w/o MAML and find that w/o MAML also uses learner.evaluate_meta_() to evalate the performance.
As you know, meta learning aims to fast adaption. It is reasonable for your method to use evaluate_meta_() to fast adaption, but w/o MAML uses evaluate_meta_() to evaluate is unreasonable.
From my view, w/o MAML also trained the model again during the meta-testing phase (fast adaption). However, this method is w/o MAML.

Hi @wjczf123, thanks for your questions.
First, I want to clarify that the scene is an N-ways K-shot few-shot task, which means it can only use k examples in each test-task support set after using all support/query training sets. So when we do the w/o MAML experiment, there're two ways.

Finetuning in support/query training set, and fine-tune in each k example.
Finetuning in support/query training set + k test support example for each task.

The second way has high training costs and is prone to catastrophic forgetting. As we mention in our paper, we use the first way to do the w/o MAML ablation experiment.
The second one I want to clarify is that the evaluate_meta_ function only fine-tunes several steps(both entity span detection and entity typing) in each support test set, and evaluates the performance in each query test set. And the evaluation logits are almost the same between w MAML and w/o MAML. So we reuse them as the same function evaluate_meta_. And we also search the hyperparameter by the validation set. According to our experience, there're also convergence fine-tuning steps for each module and each task.
And I agree with your opinion, that MAML has advantages in this scenario. It's also our motivation to explore a way to resolve the few-shot NER problem.

Thanks for your reply. My core point is that when MAML is not used, there should be no fine-tuning in meta-testing stage.
As you know, w/o MAML methods (i.e., NNShot and ESD) that usually do not use support set and fine-tune in meta-testing stage. These methods only use support set in meta-testing to construct prototype.
Due to MAML methods aim to fast adaption, they fine-tune in meta-testing. So I argue that w/o MAML may be not fine-tune in meta-testing stage. Fine-tuning on support test set will make the result better.

Can you report this result in this repository?

Hi @wjczf123, I understand your consideration. Sure, as I mentioned before, we search the fine-tune steps hyperparameter(including 0 steps, zero-shot learning) in the w/o MAML experiments. In fact, most of the time, the entity span module needs several fine-tuning steps, especially in Few-NERD Intra(the gap is big, more than 10 points f1-score) and Cross-Domain datasets. And in the entity typing module, most of the time, the optimized hyperparameter is 1 step, even 0 steps(the gap is very small, less than 1 point f1-score).

OK. Thanks for your detailed reply.


Make Add Text Note Bloc structure consistent

It currently uses the generic Note bloc directly, it should be made more like the other notes for consistency.

Resolved by #9


ios-deploy claims to be out of date when libimobiledevice is uninstalled

Steps to Reproduce
Ensure flutter installation is up-to-date, then:

brew uninstall --ignore-dependencies libimobiledevice
flutter doctor

ios-deploy reports being at version 1.9.1, but notest that it is 'out of date' and should be upgraded.
[-] iOS toolchain - develop for iOS devices (Xcode 8.3)
    • Xcode at /Applications/Xcode-beta.app/Contents/Developer
    • Xcode 8.3, Build version 8W143q
    ✗ ideviceinstaller not available; this is used to discover connected iOS devices.
      Install via 'brew install --HEAD ideviceinstaller'.
    • ios-deploy 1.9.1
    ✗ ios-deploy out of date (1.9.0 is required): upgrade via 'brew upgrade ios-deploy'.
    • CocoaPods version 1.2.0

Fixed by #10254.

I am experiencing the ios-deploy error message in flutter doctor after first-time install of flutter from git repo today. Is the fix not pushed?

The fix definitely would have been pushed since march 8th.  You can check your flutter doctor and see if you have a flutter more recent than march 8th, but I would suspect so.  Thanks for the updated report.

Added comments to #10360 requesting more info. Using the repro instructions above, I'm not seeing a regression, but that definitely doesn't rule out that there's another way to trigger this.


Add RPi's temperature info to web interface

As phonieboxes might lack of a proper cooling solution, temperature should be visible in the web interface. Great to have as an indicator in case of issues or simply for monitoring temperature in hot environments (kids room under the roof etc).

Could you resolve the conflicts?

Could you resolve the conflicts?

It's done.


3.0.0

Releasable status

Codecov Report

Merging #19 into master will decrease coverage by 0.62%.
The diff coverage is 56.25%.

@@             Coverage Diff              @@
##             master      #19      +/-   ##
============================================
- Coverage     51.25%   50.62%   -0.63%
+ Complexity      238      236       -2
============================================
  Files            51       51
  Lines          1198     1201       +3
  Branches        131      133       +2
============================================
- Hits            614      608       -6
- Misses          527      537      +10
+ Partials         57       56       -1

Impacted Files
Coverage Δ
Complexity Δ

...business/impl/service/FileResourceServiceImpl.java
8.33% <ø> (ø)
1 <0> (ø)
:arrow_down:

...mmons/file/backend/api/FileResourceRepository.java
75% <ø> (ø)
2 <0> (ø)
:arrow_down:

...commons/file/business/api/FileResourceService.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../file/backend/impl/FileResourceRepositoryImpl.java
51.19% <69.23%> (-6.31%)
30 <2> (-2)

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 2779f08...894d72b. Read the comment docs.


onDropFinish function is required, which fires once after onDrop is done

We are using vue-smooth-dnd with 3 containers and countless numbers of draggables inside. After an element changes it's position, we need to save all the items in all containers.
Right now the last function is onDrop, which is fired n times for every container there is on page. Since we have 3 containers, it fires 3 times, whilst once would more than suffice.
A huge improvement would be a new function onDropFinish, which would fire only once after onDrop finishes running n times.

I now realize my proposal makes no sense, due to how it all structured. If there were a wrapper, which were to hold containers, then my proposal would have been possible to implement. But right now it's practically impossible, not without massive restructuring...
The only solution I see is to declare a separate "hidden" container, in which it would be impossible to drop in, and declare an onDrop there.


expose Sentry's environment as an option

I need more than 2 environment options

Thanks @abensur !

Release v1.0.0


plugins/lsp: set correct package level for the ruff-lsp package

The CI is annoying to run.
Also, there is no problem with having just pkgs.ruff-lsp. I will then leave it as is.


Please provide more information about the usage of Data Compression

Add information about performance hit when using such approach vs using directly text files. Number of parallel readers, eventual overhead of uncompressing files?
Thank you

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

ID: 577e278d-55cc-bd42-7517-f1c48b3671fe
Version Independent ID: 42a3922c-a31f-d9e4-8d7d-a7d19ac42d32
Content: Migrate your data to SQL Data Warehouse
Content Source: articles/sql-data-warehouse/sql-data-warehouse-migrate-data.md
Service: sql-data-warehouse
GitHub Login: @jrowlandjones
Microsoft Alias: jrj

@djpirra Thanks for the feedback. We are actively investigating and will get back to you soon.

@djpirra Just to clarify, are you making a request for additional information or are you requesting that the documentation provide additional details about data compression? Thank you, Mike

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

I was referring to more detailed documentation about it. Was it improved?
Thank you
Mike Ubezzi<EMAIL_ADDRESS>escreveu em qua, 9/05/2018 às 22:17 :

Closed #8182 https://github.com/MicrosoftDocs/azure-docs/issues/8182.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/8182#event-1618600924,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APPYdh4YmllB0uYHMuUU9ai-I0UgRWdhks5tw1zmgaJpZM4T1XCn
.

--
Melhores Cumprimentos,
Luis Simões

@djpirra Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@Mike-Ubezzi-MSFT Mike, please assign this to @TwoUnder. Thank you.

This article is quite old and will be removed soon.

@djpirra
thanks for bringing this to our attention! This article has been removed from the documentation. Please see  Design decisions and coding techniques for SQL Data Warehouse.
I will proceed with closing out this Git Issue, but please feel free to respond should you need anything further. Thanks!

#please-close

Thank you all!
Em sex, 31 de mai de 2019 às 03:14, julieMSFT<EMAIL_ADDRESS>escreveu:

@djpirra https://github.com/djpirra
thanks for bringing this to our attention! This article has been removed
from the documentation. Please see Design decisions and coding techniques
for SQL Data Warehouse
https://docs.microsoft.com/en-us/azure/sql-data-warehouse/sql-data-warehouse-overview-develop
.
I will proceed with closing out this Git Issue, but please feel free to
respond should you need anything further. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/8182?email_source=notifications&email_token=ADZ5Q5T6JWJ7QHK7PB5HDRTPYBNVZA5CNFSM4E6VOCT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWTYOEQ#issuecomment-497518354,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADZ5Q5XFACW7APRSNY2JHO3PYBNVZANCNFSM4E6VOCTQ
.

--
Melhores Cumprimentos,
Luis Simões


Update SDK to handle user entity sync tables

Create user entity table to be used with user mapping
Require 2 fields (userId and userEmail) when on any sync table marked as the user table. Require userEmail to be email or person. Require userId to be string or number.

Should we limit packs to 1 and only 1 user entity table?


Chat after knowledge is not working

Created an agent and added our large knowledge base docs. Refreshed chat and asked a questions. It's not working at all. I see the following logs. Screen shot attached gives the full context of the agent i created
2024-09-03T17:53:41.520Z [server] [ERROR]  ⨯ unhandledRejection: Error: Run must in creating, continue or error state, not running
    at Run.nextChat (file:///Users/cjellick/projects/desktop/node_modules/@gptscript-ai/gptscript/dist/gptscript.js:306:20)
    at Socket.<anonymous> (file:///Users/cjellick/projects/desktop/server/app.mjs:372:37)
    at Socket.emit (node:events:519:28)
    at Socket.emitUntyped (/Users/cjellick/projects/desktop/node_modules/socket.io/dist/typed-events.js:69:22)
    at /Users/cjellick/projects/desktop/node_modules/socket.io/dist/socket.js:704:39
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
2024-09-03T17:53:41.522Z [server] [ERROR]  ⨯ unhandledRejection: Error: Run must in creating, continue or error state, not running
    at Run.nextChat (file:///Users/cjellick/projects/desktop/node_modules/@gptscript-ai/gptscript/dist/gptscript.js:306:20)
    at Socket.<anonymous> (file:///Users/cjellick/projects/desktop/server/app.mjs:372:37)
    at Socket.emit (node:events:519:28)
    at Socket.emitUntyped (/Users/cjellick/projects/desktop/node_modules/socket.io/dist/typed-events.js:69:22)
    at /Users/cjellick/projects/desktop/node_modules/socket.io/dist/socket.js:704:39
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
2024-09-03T17:54:18.709Z [server] [ERROR]  ⨯ unhandledRejection: Error: Run must in creating, continue or error state, not running
    at Run.nextChat (file:///Users/cjellick/projects/desktop/node_modules/@gptscript-ai/gptscript/dist/gptscript.js:306:20)
    at Socket.<anonymous> (file:///Users/cjellick/projects/desktop/server/app.mjs:372:37)
    at Socket.emit (node:events:519:28)
    at Socket.emitUntyped (/Users/cjellick/projects/desktop/node_modules/socket.io/dist/typed-events.js:69:22)
    at /Users/cjellick/projects/desktop/node_modules/socket.io/dist/socket.js:704:39
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
2024-09-03T17:54:18.709Z [server] [ERROR]  ⨯ unhandledRejection: Error: Run must in creating, continue or error state, not running
    at Run.nextChat (file:///Users/cjellick/projects/desktop/node_modules/@gptscript-ai/gptscript/dist/gptscript.js:306:20)
    at Socket.<anonymous> (file:///Users/cjellick/projects/desktop/server/app.mjs:372:37)
    at Socket.emit (node:events:519:28)
    at Socket.emitUntyped (/Users/cjellick/projects/desktop/node_modules/socket.io/dist/typed-events.js:69:22)
    at /Users/cjellick/projects/desktop/node_modules/socket.io/dist/socket.js:704:39
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)

I'm seeing this as working now. Kicking it to testing to reconfirm.

Tested with 18eaca7d
I am able to chat with knowledge after files are ingested. I do see other issues relating to this workflow which is tracked in
https://github.com/gptscript-ai/desktop/issues/491
https://github.com/gptscript-ai/desktop/issues/492


line of overlapping connections is not displayed when selecting

Hello, I have a small problem visible in the attached screen,
I have overlapping links and when I select a specific link, I would like the whole link to be visible but here a part of the link is missing. Do you have an idea how to fix this? I have tried a Z-index but it doesn't seem to work on a svg.

This is a problem of straight lines, which overlap.
You can try with this css:
.connection:has(.selected) {
    z-index: 3;
}

Perfect ! Work correctly :)


Google Pubsub Publisher (latency/throughput config not exposed)

We are using google pubsub for a banking/transaction system and are seeing messages sometimes take as long as 17 seconds to publish using the API that is available here.
I see in other languages we have batchsize, data size and other things exposed to decrease latency, but I haven't been able to use/locate/configure these settings in elixir.
We would like to keep gPubsub, but if we can't get a handle on these performance issues in elixir specifically with publishing google pubsub messages soon we will have to find a different solution for our message queues.
Any help or guidance would be greatly appreciated.
of course, sorry to mention but time is of the most importance.

It's likely that the difference is the use of the gRPC pubsub endpoints vs HTTP/REST pubsub endpoints. Some of the more common languages have high-performance handwritten pubsub clients targeting gRPC, which is how they are able to support very high throughput and low latency. We haven't had the resources to write one for Elixir, so we're limited to an autogenerated HTTP/REST client which doesn't have as many features.
My advice at this point, if you are encountering unacceptable latency issues with the existing REST client that do not seem to appear in other languages, is to look for a different solution.

@dazuma Where would I find the Google pub/sub spec if I wanted to contribute to the development of Weddell ? It is a gRPC lib for Pubsub in elixir.

The cjab/weddell project is a good alternative: https://github.com/cjab/weddell


Disable checked_cast_br jump threading in ossa temporarily

This was enabled in ossa recently.
There are 2 bugs:

In CheckedCastBrJumpThreading::Edit::canRAUW, we don't check for canFixUpOwnershipForRAUW.
Currently modifyCFGForSuccessPreds has to be called after modifyCFGForFailurePreds. But modifyCFGForFailurePreds can modify cfg in a way such that OSSA RAUW can no longer be done in modifyCFGForSuccessPreds.

Due to this, we will eventually crash in OwnershipRAUWPrepare::prepareReplacement
This will require an algorithmic change, disabling temporarily to unblock rdar://121484645

I don't have a reduced test case yet.

@swift-ci smoke test

@swift-ci smoke test


Avoid exposing secrets on requirements.txt for --[extra-]index-url with credentials

When running
pip-compile requirements.in --extra-index-url<EMAIL_ADDRESS>
the resulting requirements is
#
# This file is autogenerated by pip-compile
# To update, run:
#
#    pip-compile<EMAIL_ADDRESS>requirements_test.in
#
--extra-index-url<EMAIL_ADDRESS>
...==0.2.7  # via -r requirements.in
numpy==1.19.1             # via ...
scipy==1.5.2              # via ...

where both [USERNAME] and [PASSWORD] are shown in plain text. Since the requirements.txt shall end in version control, specially with post-commit hooks, this exfiltrates secrets to the git repository.
Environment Versions

OS Type: Mac
Python version: Python 3.7.3
pip version:  pip 20.2.1
pip-tools version: pip-compile, version 5.3.1

Steps to replicate
See above.
Expected result
The secrets should not be outputted to the compiled requirements.txt. Instead, they should keep their original names, as pip accepts.
In the example above, IMO we should only output
--extra-index-url<EMAIL_ADDRESS>
both on the header and on the actual requirements. It is the responsibility of the deployment team to ensure that they have the credentials to download dependencies from a pypi server.
Actual result
The secrets are outputted to requirements.txt.

This also happens with
<EMAIL_ADDRESS>pip-compile requirements.in

This also happens when
--extra-index-url https://${PYPI_USERNAME}:${PYPI_PASSWORD}@${PYPI_REPOSITORY}/simple

is added on requirements.in. This is likely the correct place place it, as we have the opportunity to not expand the variables here. All other options above, PIP_EXTRA_INDEX_URL and --extra-index-url give us no opportunity to not have the expanded variables, as they are already expanded by bash when we receive them.

@atugushev, together with git-hooks this easily causes password exfiltrations, which is considered a security vulnerability. Is there any timeline or acknowledgement of the validity of the issue?

Hello @jorgecarleitao,
Thanks for the issue! There is --no-emit-index-url option for the case when you have sensitive data in index URLs.

This also happens when
--extra-index-url https://${PYPI_USERNAME}:${PYPI_PASSWORD}@${PYPI_REPOSITORY}/simple

is added on requirements.in. This is likely the correct place to place it, as we have the opportunity to not expand the variables here.

Keeping unexpanded variables in requirements.txt sounds a good idea, see relative issue #966.

closing in favor of #966 .

The issue here is that the shell is expanding the environment variables, so pip-compile will get the --extra-index-url parameter value with the environment variables already expanded. A solution is to use single quotes, to prevent the shell from expanding them, like:
pip-compile requirements.in --extra-index-url<EMAIL_ADDRESS>
But that only works partially. See #1367.


finish the code

finish with ?> everytime or sometimes it could end in infinite loop

No, don't use php close tag: "The closing tag of a PHP block at the end of a file is optional, and in some cases omitting it is helpful when using include() or require(), so unwanted whitespace will not occur at the end of files, and you will still be able to add headers to the response later."


Duplicate key error in collection when using mongo or mongoose stores

I was working with the express-brute-mongo store, and encountered this error:
Error: Cannot increment request count
errmsg: 'E11000 duplicate key error collection: mydb.bruteforce index: _id_ dup key: { : "G/IAnx1Agvj9X0JzMvChzHFl6jKU0ps4dnfVvJ+ZYTc=" }' } }

Since I couldn't get this resolved in the the mongo store module, I've tried the
exact same implementation with the express-brute-mongoose store, and encountered
the same error. Since the error gets triggered in express-brute, I'm hoping to find answers here on how to resolve this.
This is the middleware I set up (same as defaults, but logging the dup key error):
var rateLimitMiddleware = new ExpressBrute(bruteStore, {
  failCallback: function (req, res, next, nextValidRequestDate) {
    var err = new Error('Too many attempts.');
    err.status = 429;
    next(err);
  },
  handleStoreError: function (error) {
    console.log(error);
  }
});

My implementation with express-brute-mongo was:
var MongoStore = require('express-brute-mongo');
var MongoClient = require('mongodb').MongoClient;

var bruteStore = new MongoStore(function (ready) {
  MongoClient.connect(process.env['DB_URL'], function (err, db) {
    if (err) throw err;
    db.collection('ratelimits').ensureIndex({expires: 1}, {expireAfterSeconds: 0});
    ready(db.collection('ratelimits'));
  });
});

My implementation with express-brute-mongoose is:
var MongooseStore = require('express-brute-mongoose');
var BruteForceSchema = require('express-brute-mongoose/dist/schema');
var bruteModel = mongoose.model('bruteforce', BruteForceSchema);
var bruteStore = new MongooseStore(bruteModel);

Any help why i'm hitting dup key MongoErrors would be greatly appreciated.

+1


Revert "Fix TypeScript GraphQL config file loading."

Reverts the bugfix graphql/graphiql#2658 to fix #2662, a repeat of the bug from last week that it seemed #2658 had introduced. This will break things for graphql.config.ts users again, but make the extension work for more users again

ah... and because the versions aren't pinned, this revert may not even work 😢

ah the beautiful red squigglies, we're golden


No need to use sudo during installation

OCD build http://ocd.appscale.net:8080/job/Daily Build/6523/


Panic when no port

Prevent panic when port is not passed to consul.NewConnection, re Panic when ANNOUNCE_ADDR passed #67
https://github.com/seatgeek/resec/issues/67

This fix was pulled into this fork: https://github.com/Aireuropa/resec/pull/2


can: Make frame and filter conversion function params const

One of the function parameter in conversion functions can be
make const (because we are not changing it).
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

ping @nashif, could you merge this as it just adds const to the function parameter.


Add pre-commit config

/lgtm
You can create a CI job to run it based on https://github.com/openshift/release/pull/35600 then most of to pull-request triggered github workflows can be removed as they would be duplicates.


Differences between this and original FP

Hi just wondering how close this is the the original Ole version of FoundationPress, have you customised it for your own needs or kept it as it was but just more up to date?

Hi,nitrokevin.
Thank you for your comment.
As you said, it's a fork of Ole's version of FoundationPress.
I originally forked it, but it was a lot of work, so I'm publishing it as a new project.
I'm not up to date with the read-me, but I've adjusted it to work with the latest web environment, including foundation sites and WordPress gutenberg. I haven't explained it so others don't know how to use it.
I also referred to your and jonbrockett's repositories. I would like to take this opportunity to thank you for your help.
I'm sorry for my poor English.

Hey! I just post a message on nitrokevin Github to thanks him.
Seems you also made a great work to update ul theme.
We have to make it live again!

Hi dantahoua.
I have reviewed the repository and both mpn start and yarn start seem to be working fine. That's strange.
However, the way yarn2 was installed was wrong, so I fixed that.
Thank you very much.

Maybe you should update the documentation a little

I'm sure it's a lot of work to explain how to use wordpress. Maybe you should update the documentation a little bit, it would be a lot of work to explain how to use wordpress, but I think I should definitely explain the Gutenberg affinity.
Thanks for the follow-up and encouragement.

Weird npm do not start, maybe my node version (12.1)...

Hmmm. I tried it with v12.1.0 and there seems to be no problem.
Currently, I am using v12.14.0. Is it possible to change the version?


refactor: enhance admin mod typing

:tada: This issue has been resolved in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Preamble affects included attributes

I see inconsistent behavior when using the extension with or without a pagebreak ('<<<<') included in the preamble. It appears that attributes set in a file that is included prior to the pagebreak are not being set, or perhaps are being scoped out of existence by the time those attributes are used in the following document markup. I haven't tried to further characterize if anything else might not work, but do have a trivial test case which I'll attach. The prebuilt outputs are included and just 'make compare' will show the differences. 'make clean compare' will regenerate the outputs. Building without the extensions and with the normal 'html5' target generates identical outputs with the attributes properly set (I haven't included a rule for this, as the differences in behavior of the extension WRT the page break are the issue).
test.tar.gz
This is using asciidoctor-multipage 0.0.12, asciidoctor 2.0.12, and Ruby 2.5 on Debian stretch.

This issue may require #46 to be done first.


Set SimpleTerminal properties based on system props

When running sbt 1.4.0 with -Dsbt.ci=true and -Dsbt.color=true, there is
no color output. This was because in this scenario, a SimpleTerminal was
used and isAnsiSupported and isColorEnabled were hardcoded to false
rather than reading the values from the system properties.

Fixes #5958


plotfigure.py

Hi cshjin, i'm currently exploring the model about federated learning. As i reviewing your work, i find the part for visualising the result is missing. Could you help provide some ideas about the part 'plotfigure.py'? Thanks in advance for your help.

Thank you very much!!!

Sorry for the late reply. The folder out was in .gitignore earlier, that's why I missed it.
I added the plot script under out again.

Thank you very much!!!


Upgrade Play! version from 2.4.10 to 2.4.11

No changes other than version numbers from Maven plugin or test projects point of view, but update anyway.

New 1.0.0-beta7-SNAPSHOT snapshot deployed, documentation updated.


Training Visualization

Description
DJL should include a NN web or GUI visualization tool like DL4J to help optimize NN parameters and NN layers.
Will this change the current api? How?
This will be a new API.
Who will benefit from this feature?
Users who want to monitor training jobs and help determine why their model is not training successfully.
References

See also: Tensorboard

I would like to propose this prototype for DJL Training visualization https://github.com/mgubaidullin/djl/tree/djl-ui-prototype/ui


Fix crash on save in rejuv'd Color Schemes page

Summary of the Pull Request
Fix a bug where if you pressed the "Save" button, WT would crash. This was caused by adding the possibility that no color scheme is selected in the main page. With no "current scheme", attempting to get its "name" would cause a null ptr exception.
The fix is simple: just check if we actually have a current scheme.
Bonus points: if we don't have a current scheme, don't bother looking throught the color schemes for a match because we'll never find one.
References
#13269 - Color Schemes Rejuv

FYI @DHowett as the 1.16 release guru

aw frick, i lied to you
Error C2451: a conditional expression of type 'const winrt::hstring' is not valid

check for .empty() >_<

@msftbot merge this in 1 minute


tvOS release

Any plans to make a release with tvOS?

This should already work in tvOS.

Yes, but the code is not released yet..

Fair enough.
I meant that the basic saving data methods etc should already work in your own project if you just copy the .swift file into your project and add it to your tvOS target.

Thought I'd made a release with tvOS, but I might have forgotten. Will make a new one tonight 👍

On 16 Jan 2016, at 3:41 AM, Dom<EMAIL_ADDRESS>wrote:
Fair enough.
I meant that the basic saving data methods etc should already work in your own project if you just copy the .swift file into your project and add it to your tvOS target.
—
Reply to this email directly or view it on GitHub.

2.0.5 should work now, but let me know if not and I'll investigate :)


Fix code scanning alert - Starting a process with a partial executable path

Tracking issue for:

[x] https://github.com/YourLabXYZ/PIP-Installer/security/code-scanning/20

With the release of committee https://github.com/YourLabXYZ/PIP-Installer/commit/bd9f67e257a25c81172bdfb5c3035e7729874e6b the problem should be resolved, we are waiting for the release of the branch release/v1.0.1 to be able to close the issue.


Release 1.6 prep

updated extension control, sql files, and documentation regarding 1.6 version

@pykello ping

(My status is that I am testing this briefly before we merge it).


Theming: Making things pretty

Material-ui has plenty of support for theming, and to some extend we've already made use of it. However, more work is needed to make things look pretty across a variety of screen sizes.
Basic typography tweaks should happen in src/config/typography.js, whereas components should be styled in the components themselves.
The rest of the theming should happen in the Sass files in src/data/sass. It's this that this issue is about. Essentially, make things pretty.

I think we can close this as we've got a basis up and running, and this is too vague to keep open now.


Haiku: some type fixes for ioctl() arguments and readv()/writev(), and some missing constants

These two commits fix some issues that became visible while building libstd.

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

So does this mean the current declaration won't work properly anyway?

So does this mean the current declaration won't work properly anyway?

I assume you are talking about readv() and writev()
With the caveat that I am not an expert what goes on at the linker level/CPU register level, we have indications that both varieties will work. The valid input of the readv() size parameter is between 0 and 1024 anyway, and in the libstd it was even maximized to 16. These values fit both in an 8 byte unsigned integer, as well as a 4 byte signed integer. Furthermore, on 32 bit systems size_t and int are both 4 bytes, so there is not even a mismatch of the size. My fellow Haiku contributors think that it won't matter anyway; on 64 bit systems the 4 byte int will be put in an 8 byte register anyway, so if we pass a 4 byte value from Rust to the 8 byte value to Haiku, it will not break. I am doing some extra tests right now to make sure that this is correct, but so far it seems like this is correct.
There are unwanted downstream effects when not conforming to the standards, meaning that we will have to put specific cases in libstd for Haiku. It seems easier to fix the API, both in Rust's libc and on Haiku itself.

@bors r+

:pushpin: Commit d4e4814f320a838b2a1a13ba6e2009b274e3f784 has been approved by JohnTitor

:hourglass: Testing commit d4e4814f320a838b2a1a13ba6e2009b274e3f784 with merge 32457ddd8f7446b336d87d6c5f2c64416021e055...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 32457ddd8f7446b336d87d6c5f2c64416021e055 to master...


feat(crypto): upgrade Amount to 128 bits

#1378

Even though Amount uses u128 internally, the Amount::value method casts it down to u64 unconditionally. This is so the PR can be used right away without propagating changes through out the component stack that we're reworking. However, this also means that we do not actually support bigger Amounts yet.

this LGTM, I think using all little endian and shifting when decoding is clearer

Let's hold off on merging this until we land the in-flight component changes? I think it will be easier and cleaner to rebase this on top of that than the other way around.

From discussion in a meeting, after this is merged the changes will need to be propagated through the staking rate calculations. This can be rebased and merged

Switched to #1819


Merge filter is missing columns and attributes

From @SimonbJohnson - not sure if this is expected behaviour or not.  The following link, I want to pull through all tags with #meta+attribute from another spreadsheet with the merge column functionality.  Unfortunately it only pulls through the first value and labels the tag ‘#meta’ rather than ‘#meta+attribute’
Example
Spreadsheet I am trying to merge
To clarify above, I put my tags to be merged as ‘#meta’. We are trying to Jekyll’ise our site and so will feed in HXL proxy json into Jekyll to generate the pages.  It means they will be more responsive and will only query the proxy every recompile

[x] reproduce bug with unit test
[x] fix bug
[x] document expected behaviour more clearly
[x] update documentation in Proxy wiki, if needed
[x] deploy to beta Proxy

The problem comes with the filter that's using generic #meta to match all #meta tags. The filter does not seem to be treating it as a pattern and adding all columns that match (or rather, it's creating an empty #meta column).
Expected behaviour is to find all the columns that match in the merge dataset and use them first. Not sure what to do when there are no matches.

Deployed successfully to beta, wiki page updated, and Simon's example looks correct now.  Waiting for verification from @SimonbJohnson to close issue.


trim front option to allow more than 30bp

Hello! I am very thankful for this tool as I use it all the time!
However, I am running into some issues as I need to cut the first 50 bp of my reads but the program does not allow me
ERROR: trim_front1 (--trim_front1) should be 0 ~ 30, suggest 0 ~ 4
I see that this issue was brought up before here but the issue was closed despite having some demand for this implementation.
Is there a way to allow this in the program? Or what alternatives do you recommend?

I agree this should be changed to a warning instead of an error, there are plenty of cases where you may want to trim more than the ranges allowed here.


Debugging to find out which user the script is running as.

Just opened to do a build, removing now.


Add CI validation

I just copied the workflow and dependencies from the docs repo.

Yeah. Looks like GitHub doesn't include those by default. Whoops


KAFKA-10706; Ensure leader epoch cache is cleaned after truncation to end offset

This patch fixes a liveness bug which prevents follower truncation from completing after a leader election. If there are consecutive leader elections without writing any data entries, then the leader and follower may have conflicting epoch entries at the end of the log. The JIRA explains a specific scenario in more detail: https://issues.apache.org/jira/browse/KAFKA-10706.
The problem is the shortcut return in Log.truncateTo when the truncation offset is larger than or equal to the end offset, which prevents the conflicting entries from being resolved. Here we change this case to ensure LeaderEpochFileCache.truncateFromEnd is still called.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@junrao Thanks for reviewing. I pushed a minor tweak. I think we need to truncate the epoch cache even if the truncation offset is larger than the end offset. This is what would have happened in the example from the JIRA.


fix image tagging with registry / registry_org

/cc @lmilbaum The last run of this action running the following command and resulting in the following error:
/usr/bin/buildah bud --platform linux/amd64 -f /home/runner/work/ai-lab-recipes/ai-lab-recipes/models/convert_models/Containerfile --format docker --tls-verify=true -t model-converter:latest-linuxamd64 /home/runner/work/ai-lab-recipes/ai-lab-recipes/models/convert_models
error evaluating symlinks in build context path: lstat /home/runner/work/ai-lab-recipes/ai-lab-recipes/models/convert_models: no such file or directory

This PR changes:

update the tagging of the image in the build command to be: quay.io/ai-lab/model-converter:latest
fix the pathing on the build step
add better build constraints to hopefully reduce our workflows once we drop the cronschedule
unrelated houskeeping on .gitignore

LGTM
@lmilbaum @sallyom PTAL

Seems like this wasn't reviewed and these changes were made in other PRs. Closing.


INmulti incoming audio meter not showing

I no longer see the incoming level amount/meter underneath the gain sliders -- was this change on purspose?

definitely an oversight. just fixed it.


handle illegal tag: field no 0 wire type 0 errors

Encountered in https://github.com/RelationalAI/rai-ux/issues/1498, when decoding proto metadata of empty transactions.
These should go away once an engine-side bug fix rolls out.
cc @denisgursky

See https://github.com/RelationalAI/rai-sdk-javascript/pull/47

Note: This issue has been migrated to https://relationalai.atlassian.net/browse/RAI-6342.
This link is only accessible to employees of RelationalAI.


How to tag streets named after multiple people?

Not an issue per se, and probably more an OSM-related topic: There are streets in our city that are named after multiple people, e.g. after siblings. It's either explicit ("Geschwister-Scholl-Straße") or implicit (just a surname, but the municipal archiv suggests the street is named after two brothers).
According to the OSM wiki, it could be tagged as name:etymology:wikidata=value1;value2, but there are alternative options, e.g. numbered suffix.
IMHO using the semicolon should be fine, I've tagged objects in this manner before in different contexts. But I suppose the processing scripts of this project wouldn't be able handle this correctly? I haven't taken a closer look, though.
What do you think?
I am happy to provide a PR if we decide on way to handle this.

Multiple values are already handled in the data process ; separating the Wikidata items identifier with a semicolon in name:etymology:wikidata is indeed the way to go!
Here is an example in Brussels: https://www.openstreetmap.org/relation/3393466

Wow, I really should have looked at the code first! Thanks for the quick answer and the example, we will map the affected streets accordingly. 🙂


Modified Session Model To Account For Proxied Requests

This PR is in regards to ticket CPF-867.
Rather than making a new whitelistedXForwardedForDAO, Joel and I decided that it would be unnecessary as we could just modify the Session model instead.
Here are the following changes that were made:

Rather than the Session model only including one remoteHost property which stores the IP returned from req.getRemoteHost(), I replaced that property with two new properties: sourceHost and proxyHost.

sourceHost and proxyHost were put in place to account for requests who have the X-Forwarded-For header:
* sourceHost would be the IP detailed in the X-Forwarded-For header
* proxyHost would be what ever req.getRemoteHost returns (which is always the last node that sent the request which in this case is the proxy)
* in cases without proxies, sourceHost would be req.getRemoteHost and proxyHost would be null

Modified the logic in AuthWebAgent.java and SessionServerBox.java which previously contained session.getRemoteHost() to account for sourceHost and proxyHost by factoring in cases for proxies and non-proxies

BONUS: Added "android/nanos_example_client/.settings/" to the foam2 .gitignore after talking to numerous people facing the same problem of always having to remove this file in particular from git in order to make a proper commit to foam2

Regarding whitelist, since the Session is updated and saved to the SessionDAO we can use the RuleEngine to enforce the whitelist.

This approach basically implements the same logic twice, once for WebAgents and once for Boxes, but if we instead just handled this in foam.nanos.jetty.HttpServer then we would only have to do this once. Also, that object is where we already configure other HTTP server options, so would be a reasonable place to configure the whitelist. Also, doing it there would ensure that unauthenticated services would also get the correct IP-address. Jetty already provides code to do this: https://www.eclipse.org/jetty/javadoc/9.4.8.v20171121/org/eclipse/jetty/server/ForwardedRequestCustomizer.html which we could install in HttpServer if the whitelist is set.

@tharmaman we can close this PR yes?


[FLINK-20830][k8s] Add type of Headless_Cluster_IP for external rest …

…service
This is backport for  PR

Please take a look when you are free @wangyang0918


Support media type subtype parameter as version source

There are benefits and drawbacks with each method of versioning relating to Web APIs.  Almost all take into account the "API" version.  An API is a representation of functionality or capability, if there's a major functional change to an API, it should have an endpoint that better describes the functionality.  In all other cases, the required versioning involves content.  Both request and response.
One benefit of putting the version in the subtype parameter (e.g. application/vnd.null.entity+json; v=2 ) is that it is the only method that supports different versions of content. application/vnd.null.entity+json; v=2 signifies version two of the JSON format of the content.  I would hazard to say that more often than not that if one format changes so does the other.  But, there are several scenarios where one format can change and not the other.  e.g. an API that supports JSON and XML has two different syntaxes to deal with.  XML, for example, has "attributes" and while there are conventions to have attribute-like things in JSON, the fact remains that two different syntaxes can have two different reasons to change.  Two different reasons to change means two different version lineages.

I'll start by marking this as a question because I believe you asking for versioning to be supported by way of media type negotiation using a media type parameter. It may not be super obvious, but this is already support today and has been for some time (🤯).
This is described in the Versioning by Media Type and Media Type API Version Reader wiki topics. The default parameter name is v, but you can use whatever parameter name you like. If you're interested in a more comprehensive view of how it works, you can take a peek at the acceptance tests that use this style of versioning.
The wiki mentions the headers involved, but I know it's thrown a few people for a loop. Here's a simple table for all to see:

Method
Header

GET
Accept

POST
Content-Type, Accept

PUT
Content-Type

PATCH
Content-Type

DELETE
Accept*

In short, if a request has content, then Content-Type is expected; otherwise, Accept is used. If both are specified, Content-Type is always preferred first as you shouldn't be sending that header unless you are sending content. DELETE is more of an enigma that is often not discussed in versioning. It typically neither sends nor receives content. You can use Accept or you can now mark that specific action as version-neutral since DELETE usually does not vary by API version and would allow no input or output media type to be specified.
If you don't fancy those, you can always create a custom IApiVersionReader or extend the default MediaTypeApiVersionReader. Custom media types containing the API version in the text is not supported out-of-the-box because there is no standard convention or format to extract it from. I've gone to great lengths to avoid parsing magic strings. While the API version type and parsing is supported, it's always performed on an unmodified, discrete string (e.g. no trimming, etc).
I hope that helps. If I totally missed the boat, let me know and I'll be sure to answer you question. Thanks.

Sorry, I noticed that after I posted the issue.  I was basing it on

Defining new, custom media types (ex: application/vnd.my.company.1+json) to drive API versioning is another variant of this approach that is accepted within the constraints of REST. This method of API versioning, however, is not supported out-of-the-box

In Versioning-by-Media-Type.
I'd do a PR for it, but I would imagine there are some specific examples that would be required?

Such an implementation would have to support all flavors of an API version. The API version has a prescribed format that can be used in whatever input. This would mean you example would have the form of: application/vnd.my.company.{version}+json. The following media type would, therefore, be completely valid: application/vnd.my.company.2019-04-01.1.0-beta1+json. No one would probably use that format, but it's still valid.
The reason this has never been supported out-of-the-box is that there isn't a clear way to reason about the format of custom media types. This can probably only be achieved with some gnarly regex. The other reason it wasn't intrinsically supported is that you really have to want to do it that way because it requires remapping media type formatters. The built-in logic doesn't consider media type parameters so application/json;v=1.0 works effortlessly without having to remap anything.
I'm not opposed to the idea of supporting this, but I'm a bit skeptical as to whether it can really be made generic and whether it's that valuable to anyone. I don't have metrics, but versioning by media type doesn't appear to be all that common (though I agree - it's the only true way to do it RESTfully).

Such an implementation would have to support all flavors of an API version. The API version has a prescribed format that can be used in whatever input. This would mean you example would have the form of: application/vnd.my.company.{version}+json. The following media type would, therefore, be completely valid: application/vnd.my.company.2019-04-01.1.0-beta1+json. No one would probably use that format, but it's still valid.
The reason this has never been supported out-of-the-box is that there isn't a clear way to reason about the format of custom media types. This can probably only be achieved with some gnarly regex. The other reason it wasn't intrinsically supported is that you really have to want to do it that way because it requires remapping media type formatters. The built-in logic doesn't consider media type parameters so application/json;v=1.0 works effortlessly without having to remap anything.
I'm not opposed to the idea of supporting this, but I'm a bit skeptical as to whether it can really be made generic and whether it's that valuable to anyone. I don't have metrics, but versioning by media type doesn't appear to be all that common (though I agree - it's the only true way to do it RESTfully).

Agreed.  When I used "subtype parameter" I mean application/subtype; parameter or application/vnd.null.entity+json;v=1.0

The built-in MediaTypeApiVersionReader doesn't care what the actual media type is. It only considers and extracts the configured media type parameter which defaults to v. This means that a media type such as application/vnd.my.company.entity+json;v=1.0 will work just fine. Of course you still have to map the media type formatters for ASP.NET Core, but that's your choice.


Netgen LVS run failed for example design because of mismatch in number of nets

I've run the example design, heartbeat, with DRC/LVS enabled and get a mismatch in the number of nets, meaning that LVS fails.  I see a similar problem with other designs I've run.
Circuit 1 contains 34 devices, Circuit 2 contains 34 devices.
Circuit 1 contains 53 nets,    Circuit 2 contains 104 nets. *** MISMATCH ***
Result: Netlists do not match.
Logging to file "outputs/heartbeat.lvs.out" disabled
LVS Done.
I've attached the lvs.log.
lvs.log
.

Could you please attach the outputs/heartbeat.lvs.out file? I think that will be helpful for debugging. It would also be helpful to see the Python code or CLI invocation you're using to drive SC.
I was able to get LVS clean on heartbeat using Netgen 1.5.192. I've noticed issues with different errors cropping up depending on version, so could also be worth trying to downgrade to that version and see if that helps the problem.

The LVS mismatches appear to come from the lack of power pins in the Verilog netlist file.
heartbeat.lvs.out.txt

I'm running with Netgen 1.5.219 and I'm running with the heartbeat example.  Attached is my .py file.
heartbeat.py.txt

One other question: when will DRC/LVS checks be available for PDKs other than sky130A?  Is there a reason that Magic/Netgen cannot be used with other types of PDKs?

Got it, thanks for sending this over! As a workaround I'd recommend trying Netgen 1.5.192 for now, I think this issue is related to #480. We still have to investigate why this behavior is different between Netgen versions.
To answer your other question, we don't yet have Magic runsets (the file(s) that codify the DRC rules) for the other PDKs. There may be some available somewhere (or someone could create them), but we don't currently have that on the roadmap. We'd certainly appreciate this as a contribution though!

I realized that my PATH was incorrect and was pointing to an old version of Netgen.  I've installed Netgen 1.5.192 and now I can get LVS to pass. Good news, though I noticed that the SiliconCompiler run didn't end cleanly.  There's an type error reported.

| INFO    | job0    | ---          | -   | Showing file None
Traceback (most recent call last):
File "heartbeat.py", line 16, in
main()
File "heartbeat.py", line 13, in main
chip.show()                                   # show layout file
File "/home/lekan/.local/lib/python3.8/site-packages/siliconcompiler/core.py", line 3582, in show
filepath = os.path.abspath(filename)
File "/usr/lib/python3.8/posixpath.py", line 374, in abspath
path = os.fspath(path)
TypeError: expected str, bytes or os.PathLike object, not NoneType

Thanks for pointing that out - #907 should fix once merged.


United Flarmnet Daily output?

I see you are building the binary daily. Would it be possible to run it too and provide the output on a stable url?
At least on my system (Debian 11) the binary provided, segfaults instantly.
A stable url i could also integrate directly into https://github.com/XCSoar/xcsoar-data-content/tree/master/data/remote/flarmnet/globe

I see you are building the binary daily

not intentionally. it only runs/builds whenever a dependency has been updated.

Would it be possible to run it too and provide the output on a stable url?

my goal would be to run and publish hourly, but I haven't had time yet to set that up. unfortunately the GitHub Actions cronjobs are not suited for something like this since they would be disabled automatically after a few days due to cryptobros abusing the service 🙄

Well i'd run daily. I think that would be sufficient. I have also checks for the repository running daily.

the information that is coming from weglide is updated a lot more often than the rest, which is why I think that hourly could be useful

Ok i studied the weglide plane pilot association, basically this is very dynamic and can change on a sub daily basis.
Would an API call be more efficient, than offline data?

sure, an API call might be better but given the spotty coverage in the air the offline data is still better than nothing. Ideally XCSoar would automatically download the current file on startup.

Well we do have an update mechanusm now, that shows which files are out of date.
I would like to start to checksum remote resources and increment the update tag autoatically.


🚀  Add maven-publish plugin

📜 Description
Configure project LbcTopAppBar in order to deploy it on Sonatype and Maven central.
Additional documentation here
I also removed all the Composable Preview from the project (as I think it is useless for the Demo App as it is already a preview, and for the library it will avoid to add unnecessary and unwanted dependencies).
💡 Motivation and Context
Publish the first library on Maven Central
💚 How did you test it?
Deploy the library here https://search.maven.org/artifact/studio.lunabee.compose/lbctopappbar
And on Snapshot repositories https://s01.oss.sonatype.org/content/repositories/snapshots/
Use implementation("studio.lunabee.compose:lbctopappbar:_") instead of     implementation(project(Modules.lbctopappbar)) in demo app.
If you want the release process, you will have to update the version. The last version on Maven Central is 0.9.1.
You can use any versioning on a Snapshot repository, as long as version has a -SNAPSHOT suffix.
📝 Checklist

[x] I compiled before submitting the PR
[x] I reviewed the submitted code
[x] I launched ./gradlew detekt

🔮 Next steps
Add CI for automatization

@rlatapy-luna Configuration on Team City should be OK! I added a trigger to develop and master for the Publish Library job.


Update set annotations usage

Close #199 and adds vignette. Let's merge this first, and then #186 will rebase to take advantage of it. This makes the necessary changes in:

R/annotation_qc.R
R/annotations.R
R/calculate_related_studies.R
R/register_study.R

While #186 makes changes in:

R/basic_utils.R
R/datasets.R
R/find.R
R/nextflow_annotation_utils.R
R/provenance.R

I'm happy to test this :)

Happy to test it

here is the manifest generated: https://docs.google.com/spreadsheets/d/1P4AyJyzafDRCq3sUclUdKt83mTAJIvzZPNvLKswGqU8/edit?gid=0#gid=0
Seems like most fields are still missing?

@aditya-nath-sage Use the other id: syn53975003

@aditya-nath-sage Reminder to 1) run through vignettes/annotate-data-intro.Rmd and 2) test annotate_with_manifest(manifest) as part of the Jira ticket NFINT-950

manifest_generate test with syn53975003 :https://docs.google.com/spreadsheets/d/1A17I4pJ9zmwf-0t1zDjVu_Ns9x20GS7ZoPL-R0EIjhk/edit?gid=0#gid=0
Seems to populate well!

Set annotations for dev project (syn26462036) and then created manifest:
https://docs.google.com/spreadsheets/d/1OrWpemuxs0unu2OOd23kbN5tEZyvxTXt3m1lv_8RlU8/edit?gid=0#gid=0

@aditya-nath-sage Great! Thanks, this seems good enough validation that R and these package functions are clear and can be applied by you for test scenarios as well as real scenarios.


[Summer 2021] SAPP Bug - Users cannot copy any text to clipboard from source code in traces

Attempting to copy + paste the highlighted text above will not work
Steps to reproduce
Download sapp.db attached to the task

pip install fb-sapp
git clone https://github.com/stephenbradshaw/breakableflask&& cd breakableflask
sapp server --db-name sapp.db
Go to http://localhost:5000
Click on any issue and try to copy any source code

I know how to do this!

Closed via https://github.com/facebook/sapp/commit/b7e87bc571eacc089dc2921d99c7ecba1aa7ee19


TypeLoadException with .NET Core RC4 and malformed stylecop.json

I added StyleCop.Analyzers to my project and managed to manually add a ruleset file to the csproj.
Creating the project
md test
cd test
dotnet new console

The ruleset file
<?xml version="1.0" encoding="utf-8"?>
<RuleSet Name="WebDAV-Server Ruleset" Description=" Rules for the WebDAV-Server project" ToolsVersion="15.0">
  <Rules AnalyzerId="StyleCop.Analyzers" RuleNamespace="StyleCop.Analyzers">
    <Rule Id="SA0001" Action="None" />
    <Rule Id="SA1101" Action="None" />
    <Rule Id="SA1300" Action="None" />
    <Rule Id="SA1309" Action="None" />
    <Rule Id="SA1400" Action="None" />
    <Rule Id="SA1503" Action="None" />
    <Rule Id="SA1600" Action="None" />
    <Rule Id="SA1633" Action="None" />
  </Rules>
</RuleSet>

The project file
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp1.0</TargetFramework>
    <CodeAnalysisRuleSet>test.ruleset</CodeAnalysisRuleSet>
  </PropertyGroup>

  <ItemGroup>
    <AdditionalFiles Include="stylecop.json" />
  </ItemGroup>

  <ItemGroup>
    <PackageReference Include="StyleCop.Analyzers" Version="1.1.0-beta001">
      <PrivateAssets>All</PrivateAssets>
    </PackageReference>
  </ItemGroup>

</Project>

The stylecop.json file
This file is malformed, because comments aren't allowed in the project.json file.
{
  // ACTION REQUIRED: This file was automatically added to your project, but it
  // will not take effect until additional steps are taken to enable it. See the
  // following page for additional information:
  //
  // https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/EnableConfiguration.md

  "$schema": "https://raw.githubusercontent.com/DotNetAnalyzers/StyleCopAnalyzers/master/StyleCop.Analyzers/StyleCop.Analyzers/Settings/stylecop.schema.json",
  "settings": {
    "documentationRules": {
      "documentInternalElements": false
    },
    "orderingRules": {
      "usingDirectivesPlacement": "outsideNamespace"
    }
  }
}

Commands
dotnet restore
dotnet build

Output
Microsoft (R) Build Engine version 15.1.545.13942
Copyright (C) Microsoft Corporation. All rights reserved.

Program.cs(1,1): warning SA1200: Using directive must appear within a namespace declaration [C:\temp\test\test.csproj]
CSC : warning AD0001: Analyzer 'StyleCop.Analyzers.SpecialRules.SA0002InvalidSettingsFile' threw an exception of type 'System.TypeLoadException' with message 'Could not load type 'System.Runtime.Serialization.Formatters.FormatterAssemblyStyle' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e'.'. [C:\temp\test\test.csproj]
  test -> C:\temp\test\bin\Debug\netcoreapp1.0\test.dll

Build succeeded.

Program.cs(1,1): warning SA1200: Using directive must appear within a namespace declaration [C:\temp\test\test.csproj]
CSC : warning AD0001: Analyzer 'StyleCop.Analyzers.SpecialRules.SA0002InvalidSettingsFile' threw an exception of type 'System.TypeLoadException' with message 'Could not load type 'System.Runtime.Serialization.Formatters.FormatterAssemblyStyle' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e'.'. [C:\temp\test\test.csproj]
    2 Warning(s)
    0 Error(s)

Time Elapsed 00:00:02.02

Expected output
Either a good error message that the project.json is invalid or accepting the json file and not throwing the TypeLoadException
Additional information
The error doesn't appear when run from inside VS 2017.

Sometimes I also see the following error:
CSC : warning AD0001: Analyzer 'StyleCop.Analyzers.DocumentationRules.SA1642ConstructorSummaryDocumentationMustBeginWithStandardText' threw an exception of type 'System.TypeLoadException' with message 'Could not load type 'System.Runtime.Serialization.Formatters.FormatterAssemblyStyle' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e'.'. [D:\Entwicklung\WebDavServer\FubarDev.WebDavServer\FubarDev.WebDavServer.csproj]

I am also getting the above error 100% of the time with the RC3 tooling.

Adding <Rule Id="AD0001" Action="None" /> to the ruleset suppresses the warning and the analyzers seem to be otherwise unaffected and work correctly.

This seems to be an issue with Json.NET. I noticed that we are using quite an old version of it as well. Maybe upgrading Json.NET to 9.0.1 will solve this issue.

@vweijsters I expect you are correct regarding cause and solution.

I've run into this too and i'm not using a ruleset file. I'm just using the stylecop.json

I have same issue on Linux as well.

@fubar-coder :memo: Comments aren't allowed in strict JSON, but the parser we use allows them.

I get TypeLoadException as well when trying to use stylecop.json. I am using the latest version of the .Net SDK 1.0 that was released with VS2017

Same here, failing when using stylecop.json (without comments) and VS2017.

Similar here, getting a buttload of warnings after migrating to new csproj:

CSC : warning AD0001: Analyzer 'StyleCop.Analyzers.DocumentationRules.FileHeaderAnalyzers' threw an exception of type 'System.TypeLoadException' with message 'Could not load type 'System.Runtime.Serialization.Formatters.FormatterAssemblyStyle' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e'.'.

It would be great if a 1.0.1 release of StyleCop could include the fix for this.

Getting the same error with a well formed JSON file.

CSC : warning AD0001: Analyzer 'StyleCop.Analyzers.SpecialRules.SA0002InvalidSettingsFile' threw an exception of type 'System.TypeLoadException' with message 'Could not load type 'System.Runtime.Serialization.Formatters.FormatterAssemblyStyle' from assembly 'mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e'.'.

Prepared a PR to fix this issue in 1.0.0.
@sharwell can you check if this is enough for a 1.0.1 release?

@vweijsters There doesn't seem to be a due date for 1.0.1 or 1.1.0-beta2. When can we expect it?

@sharwell Is there anything we can do to help get this fix out the door? We have a linux build server that cannot enforce our rules because of this error

This should be fixed with 1.0.2.


Can this be used on live streams from personal profiles?

This works great on Pages, would I be right in assuming that this wont be usable on a live stream from a personal profile (especially since I don't think such a stream is possible from the web)

I haven't tested it on personal accounts. So long as you have a stream key I can't see why it wouldn't work. Just add your profile ID to pageID and the ID of your livestream post to postID.


Convert interface to abstract fix doesn't work

Consider this code:
interface class JsonDeserializer<T> {
  T fromJson(Map<String, Object?> json);
}

It generates this error:

'fromJson' must have a method body because 'JsonDeserializer' isn't abstract.

And this fix:

Which creates this code:

interface abstract class JsonDeserializer<T> {
  T fromJson(Map<String, Object?> json);
}

Which doesn't compile.
The fix should put the abstract keyword at the beggining.

Fixed by https://github.com/dart-lang/sdk/commit/04aa5727bef315d9f34c95b7f334c5b961b726b5 which is in 3.2.0 stable.


Linux support

There's a lot in here, but this works in vanilla Debian (after following the wiki instructions) and works properly on my Mac.
I lack the windows environment, so if @jeffdm can take a look?
Huge thanks to @zedtux for all of the work on this - I only cleaned up a few things

#closing in favor of #696


⚠ Samsung has degraded performance

In d51972c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3034 ms

Resolved: Samsung performance has improved in f9a2d20.


Format named nodes correctly for the response

@deiu check it out

Coverage increased (+0.06%) to 90.574% when pulling 36395c5088bb635308bbd7f15904fb251202345a on named-nodes into ca6b595305fff944f1beed0e42c73cc499fe9a79 on master.


Add REST API Quick Start Guide from Technet Wiki

Docs Build status updates of commit 627a534:
:warning: Validation status: warnings

File
Status
Preview URL
Details

index.md
:warning:Warning

Details

index.md

Line 9, Column 10: [Warning: ms-prod-technology-invalid - See documentation] Invalid value for 'ms.prod': 'system-center-2016'.
Line 21, Column 287: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/system-center/scom/manage-overview-html5-webconsole' will be broken in isolated environments. Replace with a relative link.
Line 21, Column 482: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/system-center/scom/manage-create-web-dashboard-custom' will be broken in isolated environments. Replace with a relative link.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Added support to read diagnostics config from the path defined in environment variable.

Fix Issue https://github.com/microsoft/ApplicationInsights-dotnet/issues/2724.
Changes
Design assumption:
Function Apps environment will set the value of this environment variable: APPLICATIONINSIGHTS_LOG_DIAGNOSTICS to be a directory that

Users can create/delete diagnostics file and,
SDK has the privilege to write to
at host initialization.

The SDK reads the value of APPLICATIONINSIGHTS_LOG_DIAGNOSTICS, and parse settings of ApplicationInsightsDiagnostics.json file and write diagnostics log to the directory.
Checklist

[x] I ran Unit Tests locally.
[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue if available.

The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.
Notes for authors:

FxCop and other analyzers will fail the build. To see these errors yourself, compile localy using the Release configuration.

Function Apps environment will set the value of this environment variable: APPLICATIONINSIGHTS_LOG_DIAGNOSTICS to be a directory that
Users can create/delete diagnostics file and,
SDK has the privilege to write to
at host initialization.
The SDK reads the value of APPLICATIONINSIGHTS_LOG_DIAGNOSTICS, and parse settings of ApplicationInsightsDiagnostics.json file and write diagnostics log to the directory.

Bit confused with this description.
The following is my understanding:
The SDK needs read permission for the directory specified in APPLICATIONINSIGHTS_LOG_DIAGNOSTICS.
The SDK needs write permission for the directory specified in ApplicationInsightsDiagnostics.json, it could be different directory than specified in APPLICATIONINSIGHTS_LOG_DIAGNOSTICs.

@Yun-Ting please add an entry in the changelog.


Update Project Configuration

Latest version of UI5 runtime
Latest version of UI5 tooling extensions
Adopted the Karma coverage configuration (only ui5-transpile is needed as preprocessor)
Cleanup of project dependencies
Simplified tsconfig.json to the minimum necessary

@akudev: this PR is ready to be merged. OK with you?


community: add terraform roadmap

Added the Terraform roadmap from roadmap.sh to the Community section 😄

@dansholds looks good, thanks!!


led-manager: add CLI11 option and config loading

To facilitate localized testing of JSON config loading, add an option
that allows the configuration to be directly passed, rather than try to
determine it from entity-manager config off dbus.
Signed-off-by: Patrick Williams<EMAIL_ADDRESS>Change-Id: I80d7051bf312044c68f44bcdb9ac02364f9abac3

Closing this . Pushed with https://github.com/ibm-openbmc/phosphor-led-manager/pull/44


joinAndSelect does not include selected data in output

Describe the bug
joinAndSelect fields with ManyToOne relationships are not populated. Instead their PKs take their place.
Stack trace
None available
To Reproduce
Steps to reproduce the behavior:

In an EntityRepository add a method that uses qb() and joinAndSelect().
Use joinAndSelect to join and select a relationship (let's call it a).
Use joinAndSelect to join a select a manyToOne relationship field from that relationship (let's call it a.b).
Call the query.
a.b is not populated. Its primary key takes its place.

An example repository that does this is the one used in the example repository
https://github.com/tonyjaimep/mikro-orm-join-demo
import { EntityRepository } from '@mikro-orm/postgresql';
import { Author } from './author.entity';

export class AuthorRepository extends EntityRepository<Author> {
  async findAllWithRelations(userUuid: string): Promise<Author[]> {
    return this.qb()
      .select('*')
      .joinAndSelect('books', 'b')
      .joinAndSelect('b.template', 'bt')
      .leftJoinAndSelect('b.comments', 'bc', {
        'bc.userUuid': userUuid,
      });
  }
}

Expected behavior

 Expected output (up until v5.5.3)
template and comments are populated
[
  {
    "uuid": "00000000-0000-0000-0000-000000000000",
    "name": "Test Author 1",
    "books": [
      {
        "uuid": "44444444-4444-4444-4444-444444444444",
        "title": "Test Book 1",
        "template": {
          "uuid": "22222222-2222-2222-2222-222222222222",
          "foo": "Test Book Template 1"
        },
        "author": "00000000-0000-0000-0000-000000000000",
        "comments": [
          {
            "uuid": "7d36d725-ab94-4324-b8ce-f2fac30964c2",
            "userUuid": "88888888-8888-8888-8888-888888888888",
            "message": "Test comment 2",
            "book": "44444444-4444-4444-4444-444444444444"
          },
          {
            "uuid": "b35373bb-fb4e-4818-bc58-8ffe691d25c7",
            "userUuid": "88888888-8888-8888-8888-888888888888",
            "message": "Test comment 1",
            "book": "44444444-4444-4444-4444-444444444444"
          }
        ]
      }
    ]
  },
  {
    "uuid": "11111111-1111-1111-1111-111111111111",
    "name": "Test Author 2",
    "books": [
      {
        "uuid": "55555555-5555-5555-5555-555555555555",
        "title": "Test Book 2",
        "template": {
          "uuid": "33333333-3333-3333-3333-333333333333",
          "foo": "Test Book Template 2"
        },
        "author": "11111111-1111-1111-1111-111111111111",
        "comments": [
          {
            "uuid": "9bc6a377-4431-424c-8da7-1e3c15a3c91d",
            "userUuid": "88888888-8888-8888-8888-888888888888",
            "message": "Test comment 4",
            "book": "55555555-5555-5555-5555-555555555555"
          },
          {
            "uuid": "8ce51989-ce3e-4ac6-973d-1941d03870b9",
            "userUuid": "88888888-8888-8888-8888-888888888888",
            "message": "Test comment 3",
            "book": "55555555-5555-5555-5555-555555555555"
          }
        ]
      }
    ]
  }
]

 Actual behavior (as of v5.6.0)
template is not populated, but comments are
[
  {
    "uuid": "00000000-0000-0000-0000-000000000000",
    "name": "Test Author 1",
    "books": [
      {
        "uuid": "44444444-4444-4444-4444-444444444444",
        "title": "Test Book 1",
        "template": "22222222-2222-2222-2222-222222222222",
        "author": "00000000-0000-0000-0000-000000000000",
        "comments": [
          {
            "uuid": "7d36d725-ab94-4324-b8ce-f2fac30964c2",
            "userUuid": "88888888-8888-8888-8888-888888888888",
            "message": "Test comment 2",
            "book": "44444444-4444-4444-4444-444444444444"
          },
          {
            "uuid": "b35373bb-fb4e-4818-bc58-8ffe691d25c7",
            "userUuid": "88888888-8888-8888-8888-888888888888",
            "message": "Test comment 1",
            "book": "44444444-4444-4444-4444-444444444444"
          }
        ]
      }
    ]
  },
  {
    "uuid": "11111111-1111-1111-1111-111111111111",
    "name": "Test Author 2",
    "books": [
      {
        "uuid": "55555555-5555-5555-5555-555555555555",
        "title": "Test Book 2",
        "template": "33333333-3333-3333-3333-333333333333",
        "author": "11111111-1111-1111-1111-111111111111",
        "comments": [
          {
            "uuid": "9bc6a377-4431-424c-8da7-1e3c15a3c91d",
            "userUuid": "88888888-8888-8888-8888-888888888888",
            "message": "Test comment 4",
            "book": "55555555-5555-5555-5555-555555555555"
          },
          {
            "uuid": "8ce51989-ce3e-4ac6-973d-1941d03870b9",
            "userUuid": "88888888-8888-8888-8888-888888888888",
            "message": "Test comment 3",
            "book": "55555555-5555-5555-5555-555555555555"
          }
        ]
      }
    ]
  }
]

Additional context
This behavior started with v5.6.0. Specifically with https://github.com/mikro-orm/mikro-orm/commit/bfa4962764b2e65cb9ee559e98e38035063c1e43, which tried to solve https://github.com/mikro-orm/mikro-orm/issues/3812.
Versions

Dependency
Version

node
16.15

typescript
4.7.4

mikro-orm
5.6.x

your-driver
PostgreSQL

Thanks for the repro. The code you mentioned here looks very similar to the test case I've added in that problematic commit, maybe we could just extend it? Looks like it joins two properties on that collection, maybe it's caused by that?
One workaround now would be explicit serialization.

Hmm looks like the test we have there is reproducing this too, it only asserts the collection is populated, not the nested relations inside it.

Thank you! 🚀


fix(app): App fix drop tips wizard button clickable area issue

Overview
close RQA-1991
The issue is caused by div tag.

after removing the tag
https://github.com/Opentrons/opentrons/assets/474225/bb6238de-8d50-4921-be3b-2f622d616257

Test Plan

Changelog

Review requests

Risk assessment

[ ] fix linting error


Error: Could not generate GraphQL type definitions (web) on Redwood 7.4

I used the quick start command to bootstrap.
$ npx @redwoodjs-stripe/cli@latest setup
$ yarn rw generate types and $ yarn rw dev will yield the same error:
... done with errors.

Error: Could not generate GraphQL type definitions (web)

Error: GraphQL Document Validation failed with 2 errors;
  Error 0: Unknown type "StripeItemsParamsInput".
    at web/src/components/StripeItemsCell/StripeItemsCell.tsx:1:28

Error 1: Cannot query field "stripeItems" on type "Query".
    at web/src/components/StripeItemsCell/StripeItemsCell.tsx:2:3
    at codegen (node_modules/@graphql-codegen/core/cjs/codegen.js:83:19)
    ..........

@senpo Thank you for bringing it up. I know about this. It just shows up when starting up but it shouldn't affect the app. Regardless, I'll look into a way to fix this, I understand it definitely does look something broke.


cmd not working

The /m option is required when the [ResourceMetadata] is used in the MappingFile.txt

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

ID: 5ac7b115-1e78-bb78-b461-c5d2739524ea
Version Independent ID: 966a5c8e-f846-9448-9a31-e948733a3c2e
Content: Introduction to asset packages - MSIX
Content Source: msix-src/package/asset-packages.md
Product: msix
GitHub Login: @dianmsft
Microsoft Alias: diahar

Thanks for the catch @fmExcense


Add unit tests that document proper exception is thrown when service is not found #260

Should the new files have the copyright header?

Should the new files have the copyright header? @AndreyTretyak

Yes, totally missed that, @EpicGuy4000  Could you please add them into new files that you've created or put classes in a single file alternatively (sorry, we have not merged .editorconfig change to add it automatically)

Should the new files have the copyright header? @AndreyTretyak

Yes, totally missed that, @EpicGuy4000 Could you please add them into new files that you've created or put classes in a single file alternatively (sorry, we have not merged .editorconfig change to add it automatically)

Added the copyright header to all new files.
Sorry for not adding them in the first place.
I look forward to seeing this repository evolve further 😄


feat(service): Add Mircosoft Teams service

Info
Adding support for Microsoft Teams. With this change you will be able to send messages to multiple MS-Teams channels via their webhook.
See #8 for more.
Proposed changes

Add ms_teams service
Add notes about ms_teams to ReadMe

Verification
I personally don't use Microsoft Teams so unfortunately I can't verify the functionality of this service. Maybe someone who has access to such a channel could verify if sending messages works as intended? I will not merge until we have confirmation.

@nikoksr I use teams often I'll test this and report back

@nikoksr I use teams often I'll test this and report back

Closes #8

Closes #8


Is it possible to request loading of several globals?

Not at the moment,
The challenge is how to provide a list by URL parameters.
For repetitive use I would use hardcoded global names
and feed it directly into $system.OBJ.Export(....)
This list could also be stored in a preloaded global:
Actually it's a demo of the transport mechanics.
The export list is straight COS


sqlite3.OperationalError: disk I/O error

Hi Paul,
I am running FindFungi for the demo ERR675624 fastq on cluster(CentOS Linux release 7.4.1708).  I had remove bsub and got the following results.
There something error I can't fix, could help have a check? Thank you.

ouput files in ./ERR675624/FindFungi/Results

128K Oct 21 09:34 BLAST_Processing
24bytes Oct 21 11:32 ERR675624.WordCloud.R
0bytes Oct 21 11:32 Final_Results_ERR675624-lca.sorted.csv
21bytes Oct 21 11:29 Final_Results_ERR675624-taxids.txt
91K Oct 21 05:48 Final_Results_ERR675624.tsv
2.9K Oct 21 11:30 Final_Results_ERR675624.tsv_AllResults-taxids.txt
11M Oct 21 05:48 Final_Results_ERR675624.tsv_AllResults.tsv
0bytes Oct 21 11:32 Final_Results_ERR675624_AllResults-lca.sorted.csv

The CSV is always empty with the errors showing below

source script code

$ScriptPath/LowestCommonAncestor_V4.sh $Dir/Results/Final_Results_$z.tsv
$ScriptPath/LowestCommonAncestor_V4.sh $Dir/Results/Final_Results_$z.tsv_AllResults.tsv

error information (using ete3-3.1.1 )

Uploading to /users/username/.etetoolkit/taxa.sqlite
Traceback (most recent call last):
  File "/users/username/tools/FindFungi-v0.23.3/LowestCommonAncestor_V4.py", line 26, in <module>
    ncbi = NCBITaxa()
  File "/users/username/username/tools/conda3/lib/python2.7/site-packages/ete3/ncbi_taxonomy/ncbiquery.py", line 120, in __init__
    self.update_taxonomy_database(taxdump_file)
  File "/users/username/username/tools/conda3/lib/python2.7/site-packages/ete3/ncbi_taxonomy/ncbiquery.py", line 129, in update_taxonomy_database
    update_db(self.dbfile)
  File "/users/username/username/tools/conda3/lib/python2.7/site-packages/ete3/ncbi_taxonomy/ncbiquery.py", line 760, in update_db
    upload_data(dbfile)
  File "/users/username/username/tools/conda3/lib/python2.7/site-packages/ete3/ncbi_taxonomy/ncbiquery.py", line 791, in upload_data
    db.execute(cmd)
sqlite3.OperationalError: disk I/O error
Done

error information(using older version ete3)

/users/username/tools/FindFungi-v0.23.3//LowestCommonAncestor_V4.sh: line 14: 28870 Segmentation fault      python2.7 ~/tools/FindFungi-v0.23.3/LowestCommonAncestor_V4.py ${1} ${y}-taxids.txt ${y}-lca.csv
Done
/users/username/tools/FindFungi-v0.23.3//LowestCommonAncestor_V4.sh: line 14: 28875 Segmentation fault      python2.7 ~/tools/FindFungi-v0.23.3/LowestCommonAncestor_V4.py ${1} ${y}-taxids.txt ${y}-lca.csv
Done

Best
Keli

@GiantSpaceRobot
Thank you.
I fixed the qlite3.OperationalError: disk I/O error by moving the output back to my PC. We don't run Google Drive or any other sync tools  in our cluster. It seems the error might caused by Lustre file system in our cluster showing like this https://github.com/TApplencourt/EMSL_Basis_Set_Exchange_Local/issues/6.
However, when moving on, another error occur in the LowestCommonAncestor_V4.py.
Traceback (most recent call last):
  File "/u2host/findfungi_test/FindFungi-v0.23.3/LowestCommonAncestor_V4.py", line 56, in <module>
    for i in descendants:
NameError: name 'descendants' is not defined
Done

I got the Final_Results_ERR675624-lca.csv with one line title while no other information.

#Taxon name,Taxid,Reads mapping to taxid,Reads mapping to children taxids,Pearson skewness score,Percent of pseudo-chromosomes with read hits

Seems there is an indentation error in this python script?

Attached the first 5 line in Final_Results_ERR675624.tsv

C
ERR675624.1001391_FC61KB9AAXX:4:14:2367:14327#GATCAG/1
76775
100
100
0.042346
100

C
ERR675624.1001391_FC61KB9AAXX:4:14:2367:14327#GATCAG/2
76775
100
100
0.042346
100

C
ERR675624.101282_FC61KB9AAXX:4:2:6436:3574#GATCAG/1
1759314
100
100
0.471951
100

C
ERR675624.101282_FC61KB9AAXX:4:2:6436:3574#GATCAG/2
294747
100
100
-0.42256
100

C
ERR675624.1029318_FC61KB9AAXX:4:14:9179:15769#GATCAG/1
1759314
100
100
0.471951
100

Is the Final_Results_ERR675624.tsv output right?

@GiantSpaceRobot
Hi Paul.
Sorry, the same error occurs in the script LowestCommonAncestor_V4.py.
Thank you.
Best
Keli

Hi Paul,
Finally, I got the csv tables with your new script.
It was done in a new ubuntu 14.04 environment with python 2.7.13(without Conda) run in docker. While still running error in ubuntu 16.04 with python 2.7.15 (install by Conda). Maybe the script can't run in python 2.7.15 or a conda environment?
Thanks a lot.
Best
Keli


Are there errors in the NinePatchRect algorithm?

Godot version:3.2.1:
I have to talk about NinePatchRect again. I wonder if there is an error in the NinePatchRect algorithm in the control?
Below is an example of the NinePatchtexture plugin I wrote. You can see the obvious differences by comparing the effects.
NinePatchTexture-Demo

OK, this is normal, then you look at the second picture, I have updated the demo, you can download a new demo to see a new demo example.
In Jiugong's standard algorithm:
1.The four corners should be kept (tiled) to their original size
2. The four sides should keep one-way scaling
3. The middle area should maintain two-way scaling
But I think NinePatchRect's current algorithm has lost its practical significance.

You need to disable Filter in the settings of the Import tab for the texture.

Aah sorry I got confused...
But it looks like the problem is more clear with the Editor in the bottom panel.

Aah sorry I got confused...
But it looks like the problem is more clear with the Editor in the bottom panel.

Please download the new Demo again and observe its borders and corners.

Yep that looks like a problem.
So have you tried the master with the PR merged to see if it was fixed.

Yep that looks like a problem.
So have you tried the master with the PR merged to see if it was fixed.

Yep that looks like a problem.
So have you tried the master with the PR merged to see if it was fixed.

I just heard that it was normal before a certain version was merged, but I cannot test it temporarily, please check this post.
->
#32170

I ran into this issue too. These rectangles use the texture with the same margins, but you can see that the borders look different:

And here's a closer look, observe the bottom:

This should not happen.

Totally agree. Documentation claims NinePatchRect is a "Scalable texture-based frame that tiles the texture's centers and sides, but keeps the corners' original size."

When resizing the NinePatchRect, the corners are scaled, preserving their original aspect ratio. This is incorrect. The corners should simply not be scaled. (If there is a hidpi scaling adjustment, then it should not be based on the size of the NinePatchRect).

The texture's centers and sides are scaled, not tiled, but that seems to be awaiting an implementation for axis_stretch_horizontal and axis_stretch_vertical.

The issue occurs when sizing the NinePatchRect to a smaller size than the backing texture.
Workaround: make the center column & row extremely narrow, i.e. 1px. The frame is affected only on scaling down, not up.
This is not a great workaround, because it limits the amount of detail you can put in the center columns, but the effect is less significant until center column tiling is implemented.

When resizing the NinePatchRect, the corners are scaled, preserving their original aspect ratio.

I'm wondering if this is the only problem.
Here is another repoduction example: ProgressBar.zip + code on GitHub. This is an attempt to implement a custom progress bar. As you can see the bar degenerates to a fat blob when the center column is supposed to become zero size:

I have added some "debugging pixel" to the progress bar texture to show where the three columns are:

Basically the texture has a width of 18 pixels (6 + 6 + 6 pixels for the three columns), and a height of 16 pixel (8 + 0 + 8 pixels for the three rows). Accordingly, the left/right margins are set to 6 pixel, the top/bottom to 8 pixels.
I've set the minimum size to a width of 12 pixel, which should allow for the case of having only the left and the right column, but no center, resulting in a prefect circle (the 6 + 0 + 6 case). What happens instead is:

Note that it would be possible to "preserve the aspect ratio of the corners" just fine in this case: Their aspect ratio is 3x4 pixels, and that would fit into the 12x16 minimum size rect with the correct aspect ratio -- if the center would shrink down to zero.
Thus, I would say the bug is rather that the center fails to shrink below its original size. Instead the algorithm messes with the size of the corners, failing to satisfy what the documentation says "... keeps the corners' original size" (and what would be much more intuitive).
Also it is a bit strange that the stretched area actually shows a color gradient. The debugging pixel jump from green to red at the boundary of the left to the center patch. The smooth transition from green to red is not what I would have expected:

Any ideas if this is intended behavior / should be tracked as a separate bug?

Fixed by #42119.
(At least the OP, I don't know if @bluenote10's comment is handled, but if not it should be reported as a new issue.)


Restructure bazelrc for reproducible builds

Moves the bulk of our bazelrc to a separate directory and then imports
it in the workspace bazelrc. This allows us to remove the workspace rc
from CI build scripts, which means that when running them locally we're
actually running with the same flags for reproducible builds and cache
hits.

Ping @stellaraccident?