Added tsv output format for bookmarks.

Does this look correct to you? It is functioning.

Hold off on this. I believe it would be more useful to also allow filter by columns, but I'm trying to avoid duplicating whole functions. Also a column for Parent IDs would be useful.

looks mostly good  -  I added 2 comments.
One is about the lens array in your PrintPrimaryOutputTSV method (I think you can remove that part)
The other is about using csv.Writer (but if you say you are not familar enough in go, we can merge it like this and I'll change it later).
Also would be nice if you could also update the README (there is a section that is just the --help output and teh Usage section is listing the available output formats).
Otherwise looks good, and thank 👍

I removed the lens code, and added to help.
Escape codes are already ignored in the values. I tested it by adding a \t to a title in my bookmarks. The output just shows \t and does not tab.

okay, looks good, I'm merging it 👍.
I think I will perhaps add tsv output to a few more commands later today and then built a new version (1.4)

@peterjschroeder Oh - one question: we are currently only printing the fields ("ID", "PARENT ID", "TYPE", "DELETED", "TITLE", "URI").
Is there any problem with printing all available fields in the tsv output (stuff like URL, Keyword, Type, etc).
If tsv is a format that intended to get processed by another system anyway (and not directly read by a user) we can just output all potentially useful data. That way the next person already has everything

Update: okay I added the csv/tsv output to the other xxx-list subcommands. ( and also the list-tabs command from #1 )
If you want you can try yourself at the column filtering - but I'm going home for today :)

Excellent. I will do the filtering tomorrow, I also ran out of time for today :P

Nevermind. Filtering is all you. I'm lost on how to iterate through the column filters.

Ok, I implemented a version of it with --csv-filter. If you pull you should get it

Thank you.
I was trying to approach it by just using the existing column filters, but this works too. The types in go is what throws me off. I know C, but I think python may have spoiled me away from declaring them properly. Go seems like its closer to C in the sense.


[RL-baseline-NM] Adding more linears to each head

We'll add more linear layers to each head of the model to give more capacity to each head. Allowing them to learn better.

What!?!?! 1k episodes - max running reward ~600


TAXII 2.1 / 2.0 in medallion 3.0

Hello
I can get my TAXII server to work with TAXII 2.1 clients.
But TAXII 2.0 clients cannot discover any collections whatsoever, what am I missing in my configuration or have overlooked? I would like to support both versions in my server.
Appreciate any help.
Thanks

Current Medallion supports TAXII 2.1 only.
https://github.com/oasis-open/cti-taxii-server/blob/46c89eb56b681003a4399cce6214f500176e86cc/medallion/views/__init__.py#L21-L22

Current Medallion supports TAXII 2.1 only.
https://github.com/oasis-open/cti-taxii-server/blob/46c89eb56b681003a4399cce6214f500176e86cc/medallion/views/__init__.py#L21-L22

Aha. Understood. Unfortunately I missed that. I need to think of an alternative then, because I need to support both client versions.

Closed.


Short writeup on Gaussian Processes

Progress on #135

Coverage remained the same at 26.741% when pulling c6cf5e91090db2e798b0ebe255aa6c9563eb8d3c on jakubnabaglo:master into 8b752bbc6f3c2d941fc8e946bacfd9a3b89f9af9 on chengsoonong:master.


loginAsync crashes when the network is down

I disabled my WiFi to test the behaviour of my tiny script, and I got this:
/Users/ec/Dropbox/Code/generate-tesla-token/node_modules/teslajs/TeslaJS.js:320
        callback(error, { error: error, response: response, body: body, authToken: loginResult.access_token, refreshToken: loginResult.refresh_token });
                                                                                ^

TypeError: Cannot read property 'access_token' of undefined
    at Request._callback (/Users/ec/Dropbox/Code/generate-tesla-token/node_modules/teslajs/TeslaJS.js:320:95)
    at self.callback (/Users/ec/Dropbox/Code/generate-tesla-token/node_modules/request/request.js:188:22)
    at emitOne (events.js:115:13)
    at Request.emit (events.js:210:7)
    at Request.onRequestError (/Users/ec/Dropbox/Code/generate-tesla-token/node_modules/request/request.js:884:8)
    at emitOne (events.js:115:13)
    at ClientRequest.emit (events.js:210:7)
    at TLSSocket.socketErrorListener (_http_client.js:400:9)
    at emitOne (events.js:115:13)
    at TLSSocket.emit (events.js:210:7)

Thank you.  Yes, the library does not currently test for network connectivity as that is pre-requisite to the libraries function.  Is your hope mainly that the library would handle that scenario more gracefully?

Circling back to review this one.  There is no great way to test for network connectivity in Node.  Common solution is a DNS resolve request which is noisy on the network.  And every single library call requires a working connection to the network.  I can look at better error handling if/when an exception occurs.


Add support for user supplied command-line args

Sometimes users might want to add additional flags to VS code to support experimental features like Wayland support or fractional scaling. In such case, users will have to modify the .desktop file. But this will not work when the user launches code-insider from a terminal.
This PR attempts to solve that issue by adding a wrapper script that reads flags from a file and adds it to the actual binary. This is also how it is done in the AUR package.

Thanks!


Connection test

Notes for Reviewers
This PR fixes #8147
This is the unit test for the connection_handlers.go
Signed commits

[ ] Yes, I signed my commits.

check out issues


PHP 7.4 Preload Support

PHP：rfc：preload

opcache_reset() is not going to reload preloaded files. It's just not possible using current opcache design, because during restart, they may be used by some process, and any modifications may lead to crash.

Still we can provide opcache:compile as an entrypoint for opcache.preload.

Should work if you put this code in a php script in your Laravel project, and point to the file.
<?php

echo shell_exec("php artisan opcache:compile --force");


Reimbursements / coverage / donation of needed $ for accounts

@anyapatel27  Is this still relevant Or can we close it?

Keeping this open as it's relevant for Paterson, NJ


Exception details not logged

After upgrading to 2.0.0 I noticed app.UseApplicationInsightsExceptionTelemetry(); is marked as deprecated.  Now in the portal I see failed 500 requests but digging in doesn't show any exception details, I am pretty sure they were there before.

Do we have to use a custom exception filter now to log the details?  Couldn't find anything in the docs or getting started guide mentioning this.

The deprecated method should be redundant to the listener that is automatically wired up internally now.
The code inside each handler function is identical.
It would be redundant to still have the old method that uses a middleware, and you would get two of each exception reported, but are you saying that with that obsolete method you are seeing details and without it you are not?  Or are you saying that with version 1.0.* you saw details and with version 2.0.0 you do not?
Just to confirm, if you downgrade to version 1.0.* of the SDK do you see more details?

Looks like when I go back to 1.0.0 the exception details are not logged either, I must've been mistaken with a non-core project that we use app insights on.  So what is the standard way to get exceptions logged?

Exceptions should be tracked automatically by default.
Can you see any in the search results or if you query through analytics?

I ran a simple .NET Core app with 2.0.0 of the AI SDK for .NET Core and I see the exceptions and requests in the portal.

I have seen some similar issue with AI 2.0 and .net framework. As a tempfix I copied the  OperationCorrelationTelemetryInitializer that's in the works for 2.1 and is setting OperationContext.ParentId and OperationContext.Id to requestTelemetry.Id. Now exceptions show up on the request's blade properly.

@dnduffy Yes, I can see the individual exceptions and details but they aren't tied to the failed requests.  Sounds like @andrasg is on the right track for what the problem is.

We really need to get an update out for aspnetcore - even if it's pre-release. This has been fixed in develop for months now.

We plan to release a 2.1-beta1 soon to address getting the checked in fixes out for people to use.


PartialTagHelper with null model results uses the incorrect right model type

Essentially a variant of https://github.com/aspnet/Mvc/issues/7295 but now with the model property.
Consider
@* _StatusMessage.cshtml *@
@model string
<alert>@Model</alert>

@* Page.cshtml *@
@model SomeModel
<partial name="_StatusMessage" model="SomeModel.Property" />

The application fails with
System.InvalidOperationException: The model item passed into the ViewDataDictionary is of type 'SomeModel', but this ViewDataDictionary instance requires a model item of type 'System.String'.
   at Microsoft.AspNetCore.Mvc.ViewFeatures.ViewDataDictionary.EnsureCompatible(Object value)
   at Microsoft.AspNetCore.Mvc.ViewFeatures.ViewDataDictionary..ctor(ViewDataDictionary source, Object model, Type declaredModelType)
   at lambda_method(Closure , ViewDataDictionary )
   at Microsoft.AspNetCore.Mvc.Razor.Internal.RazorPagePropertyActivator.CreateViewDataDictionary(ViewContext context

One possible fix would be to detect when the Model property is assigned to, rather than check if it's non-null:
Here: https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.TagHelpers/PartialTagHelper.cs#L100-L103

cc @mkArtakMSFT \ @DamianEdwards we should take this for rc1. It's pretty easy to run in to.


Loading static assets (defaults.css) is very slow on local development (admin + client)

In the current version it may take many seconds to load defaults.css to browser. This problem does not exists on a project that was created based on the template 5th of November 2018.
I already tried upgrading some libs, and also downgrading node.js and all libs but it didn't seem to resolve the issue.

https://github.com/webpack/webpack-dev-server/issues/1418


other jvm-based language completion?

there is no scala completion for vim yet, it would be a great improvement for vim if there was a scala completion plugin.
I'm not familiar with vim-scripts, but I think javavi can be a common backend for all jvm-based languages.

+1

@artur-shaik what about put javavi on his own repository? So we can improve and anothers projects can use the library.

@adelarsq I'm not really disposed to break project, because javavi still contains java source parser, and viml code still can be used for another languages.
But, still we need more investigation on it.
By the way, can we compose list of existing parsers of other languages?

I do not think split javavi is a good idea.because javavi also need javaparser.and I do not think people who use vim like git submodel.that is why I do not use ycm,too many third_part lib

@wsdjeg Good point. I was thinking about use submodules.

@artur-shaik Actually I don't know about parsers for anothers languages. For now what languages will be desired between Scala, Groovy, Clojure, Kotlin, Ceylon, Xtend, Fantom and X10 (or anothers)?

@artur-shaik what about use ANTLR and grammars-v4 for the parsing?

@adelarsq thank you for information, may be it really make sence to use this tools, just need more investigation on it.

If you use clojure, I suggest to use neoclojure

If it's only grammar parsing , maybe we can try something like compiler plugin. writing scala compiler plugin , this can gives you a compiler based completion just like clang.

I am writing a scala compiler plugin that may help to do completion.
https://github.com/Misfit-John/Scala-completion-Base
but I don't know how to integrate it with vim.... anyway, this is still a base , it can only print out members in a very tricky way.

@Misfit-John can you get completion from source code which contains syntax errors?

@richox What do you mean by syntax error? Almost all of the compiler base completion tools don't allow syntax error so the compiler can make you a right completion. If you are saying that you are using some special scala syntax, you can make a plugin to tell compiler what does them means so compiler can understand your code and give you right completion.

@Misfit-John most completion tools feature a "loose parser" that is forgiving of errors so you still get completions when some of your code is wrong. Ecj is an example and vim-javacomplete2 here has a strict parser combined with a Vimscript loose parser to gather completions for in progress code

@DonnieWest I can understand that, but this plugin is based on a scala compiler, it's kind of hacking, so you can't expect it to be a loose parser because I am not rewriting a parser. I can understand that when you are typing codes, your file will be syntex error so my first step is just to make sure that when you type a "." after a symbol, you can get the member list of this symbol.  Basically your file is syntax correct before you type the dot and so you can get the member list, this will be enough at current stage.

With LSP introduction I think this is not relevant anymore.


Preview: A new nucleotide should not be added to the end of a cyclic sequence, if the last nucleotide of the sequence does not have a free AP R1 or R2

Steps to Reproduce

Switch to the Macro mode

Draw a cyclic sequence, consisting of any 4 nucleotides, for example:

Switch to the Sequence mode - the Text-editing mode

Add any nucleotide to the end of this sequence

Switch to the Flex mode and move all nucleotides apart from each other so that all bonds are visible

Actual behavior
A new nucleotide is added to the end of a cyclic sequence. The bond, that provided and displayed cyclicity, disappears in the Sequence mode and the Flex mode

Expected behavior
A new nucleotide should not be added to the end of a cyclic sequence, if the last nucleotide of the sequence does not have a free AP R1 or R2. An error message appears on the canvas (From BA):
Error Message:
It is impossible to merge fragments. Attachment point to establish bonds are not available.
Screenshots
https://github.com/epam/ketcher/assets/139227567/a8d29b43-c951-4a10-9e64-3676a8d50006
Desktop (please complete the following information):

OS: Windows 10 Pro (version 22H2) x64
Browser: Сhrome
Version: (125.0.6422.61) x64

Ketcher version 2.21.0-rc.2, Build at 2024-05-28; 14:29:04
related with

https://github.com/epam/ketcher/issues/4548

Should be verified with next Issues: (4708, 4575, 4506,4706) + 4328
Error Message should appear:
Error Message:
It is impossible to merge fragments. Attachment point to establish bonds are not available.

Should be verified with next Issues: (4708, 4575, 4506,4706) + 4328
Error Message should appear:
Error Message:
It is impossible to merge fragments. Attachment point to establish bonds are not available.

Verified.

Indigo Toolkit Version 1.21.0-rc.1.0-g9194599b2-wasm32-wasm-clang-19.0.0
Ketcher Version 2.22.0-rc.2 Build at 2024-06-03; 18:43:00
Chrome Version 125.0.6422.142 (Official Build) (64-bit)
Windows 10

Automated.


Error out in case given file not found

Description
Currently, the following error means that checksums.txt does not found in the current directory, which is a bit confused message to see what's going on actually:
$ rekor-cli search --artifact checksums.txt --sha 852dc7cfc62513324debede3cdc7a159843b7a05ae0eb740d905b074e4798d21
Error: invalid argument "checksums.txt" for "--artifact" flag: Key: '' Error:Field validation for '' failed on the 'url' tag

We can revise the error to smth like: <FILE> does not found
$ rekor-cli version
GitVersion:    v0.3.0
GitCommit:     e4303a87b92081e162e080ff69f36830c1035c09
GitTreeState:  clean
BuildDate:     '2021-07-28T08:08:56Z'
GoVersion:     go1.16.6
Compiler:      gc
Platform:      darwin/amd64

@developer-guy

This looks handled - https://github.com/sigstore/rekor/blob/main/cmd/rekor-cli/app/search.go#L128


the generated jar file has wrong name xxx-unspecified-runner.jar

it seems the quarkus plugin generate wrong filename. ie: xxx-unspecified-runner.jar in stead of xxx-runner.jar
what i should have to change to generate good filename?
this is the output of a build:
./gradlew xxx:jib

> Task :xxx:quarkusGenerateCode
preparing quarkus application

> Task :xxx:jib FAILED
Running extension: com.google.cloud.tools.jib.gradle.extension.quarkus.JibQuarkusExtension
Running Quarkus Jib extension

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':xxx:jib'.
> error running extension 'com.google.cloud.tools.jib.gradle.extension.quarkus.JibQuarkusExtension': /home/lfarkas/work/test/java/xxx/build/xxx-runner.jar doesn't exist; did you run the Qaurkus Gradle plugin ('quarkusBuild' task)?

anyway if i rename xxx-unspecified-runner.jar to xxx-runner.jar then the result is working.

It's the Quarkus plugin that generates *-runner.jar. I am not sure if it is a bug in Quarkus that the name contains "unspecified", it is working as intended, or there is a way to control it. I looked in their code, and this code seems to determine the filename, while the "baseName" seems to be set here. @geoand any idea?
In any case, the Jib Quarkus extension simply uses the filename based on the jar task, if you configure the jar task so that the artifact name becomes xxx-unspecified.jar, I think at least it will work. Not that this is a good workaround.

Hh... I don't see how that could happen TBH.
Is there any quarkus configuration being set?

not. this is default everything.

Do you have an example application I can run to see this issue?

after further investigation the quarkus jib plugin only use the jar name and rename it. the unspecified refer to the version. and our quarkus project has no version. after we add a version it's works as the jar task and everything working. good to know that the version is a must! same as the mainClass (even if it's bogus). imho it'd be also useful to document it.

There may be something the Jib extension can do to make it work when the version is unspecified. I'll leave this issue open for tracking purposes, so that we may look into it when we have time (low priority).

For others coming here:
./gradlew build -Dquarkus.native.enabled=true -Dquarkus.package.jar.enabled=false -Dquarkus.application.version=1.0.0'

Won't do the job. unspecified comes from Gradle's version property. So instead, you want:
./gradlew build -Pversion=1.0.0 -Dquarkus.native.enabled=true -Dquarkus.package.jar.enabled=false -Dquarkus.application.version=1.0.0'

-Dquarkus.application.version=1.0.0 might be redundant.


Use builtins instead of inline asm for readlane

I currently have a compiler patch up for review (https://github.com/llvm/llvm-project/pull/89217) which will break with inline asm. This patch is a prerequisite

ping..

@vikramRH Can this PR be made in rocALUTION-internal instead?

@jsandham , I dont seem to have permissions to rocALUTION-internal and no merge permissions here. could you help me land these changes ? (Im okay with either creating new PR directly to internal repo or cherry picking this, whichever seems proper to you)


Question: SQLite

This is exactly what I have been looking for. I've always wanted to know how long I work on projects or files. I see there is WakaTime but that requires an account to be made and it goes to their cloud. No no not for me. I like local
Question: As of now in order to get the data out I just query the sqlite table ?

Hey!! I'm really happy to hear this, I agree that I also don't like the idea of having to share my data also (we already worry to much about that nowadays lol). This is the way.
Yes that's right, but actually today I was thinking that maybe adding some queries to neovim would be really cool!! I just cooked the pr, but here is a screenshot:

Otherwise you can use the queries as an example.
I also fixed a bug today where neovim was pushing some random fileype of nil (telescope buffers or neotree are a good example). Will push the pr today :)

Img broken in past comment

Great,
I noticed those nil values and filtered them out.
I' am writing a script that takes the .db file and is going to fill an MS SQL database with this information. Then I can really have fun with the data...
Thank you again!!
Love this plugin....


How to set id for each marker?

I need to set an ID for each marker to be able to use them later on when selected, any help?

If you need a reference to the actual map Marker object you should be able to get this from MarkerManager using this method:
/**
 * Get the the real marker object.
 *
 * @param marker marker which to obtain its real marker
 * @return real marker
 */
public Marker getMarker(T marker) {
    return markerCache.get(marker);
}


npm install runs all the time

What happened?
I have pulumi-kubernetes-operator managing 4 stacks from 2 TypeScript programs (fetched from git).
Even if there are no changes to the git repo, the pod keeps continuously running npm install
This does not seem desirable.
Example
All screenshots were taken around the same time.

I have the following dependencies in my package.json in the root of the repo where the pulumi programs reside:
...
    "@pulumi/keycloak": "^5.2.1",
    "@pulumi/kubernetes": "^4.5.2",
    "@pulumi/pulumi": "3.91",
...

Output of pulumi about
Probably not relevant for this case;
I'm using version 0.5.0 via the Helm chart.
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Thanks for filing the report @v-yarotsky. Have you checked if the issue reproduces on a fresh cluster? It might also be interesting to try to run a manual preview for the stacks to see if there is and to check the interval for the reconcile loop.

After taking a brief look at the code, it seems like this is by design.
On each iteration of the Reconcile loop^1, reconcileStackSession.setupWorkspace^2 is called, which in turn calls reconcileStackSession.InstallProjectDependencies^3. All of that is happening before checking whether the current commit matches the last applied one^4.

Added to epic https://github.com/pulumi/pulumi-kubernetes-operator/issues/586

Good news everyone, we just release a preview of Pulumi Kubernetes Operator v2. This new release has a whole-new architecture that uses pods as the execution environment, and seeks to retain the pod across stack operations.  Installation happens once whenever a new commit is observed by the system. It is also now possible to create a custom docker image with your bits pre-installed, to further increase performance.
Please read the announcement blog post for more information:
https://www.pulumi.com/blog/pulumi-kubernetes-operator-2-0/
Would love to hear your feedback!  Feel free to engage with us on the #kubernetes channel of the Pulumi Slack workspace.
cc @v-yarotsky @rodolfototaro @ainthapanya-sqc


Off-chain indexing is independent from workers

Currently the --enable-offchain-indexing CLI option would be taken into consideration iff off-chain workers are enabled as well.
This is incorrect, these two features serve different purposes and are designed to work separately, so it's not necessary to run off-chain workers for anything at all.

bot merge


Add generic typing to multicore_utils.py

This change adds generic typing into the functions from multicore_utils.py. This allows for better type checking.

Is the failing test flaky?

Hey @bryant1410; thanks for the PR!!. The S3 tests here seem flaky (@savingoyal ?). The 7 failures can be checked internally (I'll DM you the link) and they are all flaky so this should be good to go from a testing pov.


Cannot retrieve contents of public folder with anon key

The following only returns files using the admin key.
  Dim AllFiles = Await client.Storage.From(BUCKET).List(FOLDER, opt)

Do you have a policy set for anon users that allows for SELECT (list) for that particular bucket?


support as Jupyterbook template

Firstly I got to say how much I love this theme. Hands down one of the best-looking themes in the eco-system and thank you for bringing this up :)
I was wondering if support for jupyterbook would be something on your roadmap. I tried installing it but there seems to be a few small issues with some style elements.
Would love to work with you on this one with your assistance too

this can be general support for jupyter notebooks too but I feel like jupyterbook has a few handy features

Seems like little theme bugs are the main issue with jupyterbook too.  Jupyterbook is build on top of sphinx doc. Only a couple of theme settings are missing.

I was able to fix most of the theming issues with using Myst with this them (you can see it in action at docs.ragas.io.
I'm sharing the custom CSS file I used and was wondering if we can add it to the theme too, maybe even improve it a little too. I would be more than happy to help but I don't the enough CSS to contribute.
/* Base Theme */
div.cell_input {
    background: var(--background) !important;
}

.cell_output .output.text_plain,
.cell_output .output.stream {
  background: var(--background);
  border: 1px solid grey;
  border-radius: var(--mystnb-source-border-radius);
}

.cell_output .output.stderr {
  background: var(--background);
  border: 1px solid #fbc7c7;
  border-radius: var(--mystnb-source-border-radius);
}

.cell_output .output.traceback {
  background: var(--background);
  border: 1px solid #fbc7c7;
  border-radius: var(--mystnb-source-border-radius);
}

/* Pandas tables. Pulled from the Jupyter / nbsphinx CSS */
div.cell_output table {
  display: block;
  overflow-x: auto;
  white-space: nowrap;
  border: none;
  border-collapse: collapse;
  border-spacing: 0;
  color: var(--foreground);
  font-size: 1em;
  table-layout: fixed;
}

div.cell_output thead {
  border-bottom: 1px solid black;
  vertical-align: bottom;
}

div.cell_output tr,
div.cell_output th,
div.cell_output td {
  text-align: right;
  vertical-align: middle;
  padding: 0.5em 0.5em;
  line-height: normal;
  white-space: normal;
  max-width: none;
  border: none;
}

div.cell_output th {
  font-weight: bold;
}

div.cell_output tbody tr:nth-child(odd) {
  background: var(--backgound);
}

div.cell_output tbody tr:hover {
  background: rgba(66, 165, 245, 0.2);
}


Add translation service

Implement translation services on Kauri wallet

[ ] Investigate options for implementing translation into Kauri Wallet
[ ] Ensure that OSS contributors can easily add translations
[ ] Ensure that developers can type in english
[ ] See if we can auto build translation files
[ ] Coverage report for translate
[ ] Implement translation all current screens
[ ] Practise creating translation
[ ] Confirm with team we are happy with this translation implementation
[ ] Ensure translations for things like Time and Currency

Closed by new issues.


[FlowBroker] Not working properly after connection with RabbitMQ unexpected close

Flowbroker gets messages from Kafka but cannot en-queue them on RabbitMQ. These messages are discarded and not processed.

<00:00:23 27/04/2019> -- |producer -- DEBUG: Error while creating producer: Error: broker transport failure
<05:40:23 28/04/2019> -- |producer -- DEBUG: Error while creating producer: Error: broker transport failure
<09:10:23 28/04/2019> -- |producer -- DEBUG: Error while creating producer: Error: broker transport failure
<21:11:12 28/04/2019> -- |undefined -- ERROR: uncaughtException: write EPIPE
Error: write EPIPE
at WriteWrap.afterWrite [as oncomplete] (net.js:868:14)
<21:11:21 28/04/2019> -- |undefined -- ERROR: uncaughtException: write EPIPE
Error: write EPIPE
at WriteWrap.afterWrite [as oncomplete] (net.js:868:14)
<21:11:21 28/04/2019> -- |undefined -- ERROR: uncaughtException: write EPIPE
Error: write EPIPE
at WriteWrap.afterWrite [as oncomplete] (net.js:868:14)
<21:11:25 28/04/2019> -- |undefined -- ERROR: uncaughtException: Unexpected close
Error: Unexpected close
at succeed (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:272:13)
at onOpenOk (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:254:5)
at /opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:166:32
at /opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:160:12
at Socket.recv (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:499:12)
at Object.onceWrapper (events.js:313:30)
at emitNone (events.js:106:13)
at Socket.emit (events.js:208:7)
at emitReadable_ (_stream_readable.js:513:10)
at emitReadable (_stream_readable.js:507:7)
at addChunk (_stream_readable.js:274:7)
at readableAddChunk (_stream_readable.js:250:11)
at Socket.Readable.push (stream_readable.js:208:10)
at TCP.onread (net.js:601:20)
<21:11:25 28/04/2019> -- |undefined -- ERROR: uncaughtException: Unexpected close
Error: Unexpected close
at succeed (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:272:13)
at onOpenOk (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:254:5)
at /opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:166:32
at /opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:160:12
at Socket.recv (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:499:12)
at Object.onceWrapper (events.js:313:30)
at emitNone (events.js:106:13)
at Socket.emit (events.js:208:7)
at emitReadable (_stream_readable.js:513:10)
at emitReadable (_stream_readable.js:507:7)
at addChunk (_stream_readable.js:274:7)
at readableAddChunk (_stream_readable.js:250:11)
at Socket.Readable.push (_stream_readable.js:208:10)
at TCP.onread (net.js:601:20)
<21:11:34 28/04/2019> -- |undefined -- ERROR: uncaughtException: Heartbeat timeout
Error: Heartbeat timeout
at Heart. (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:427:19)
at emitNone (events.js:106:13)
at Heart.emit (events.js:208:7)
at Heart.runHeartbeat (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/heartbeat.js:88:17)
at ontimeout (timers.js:498:11)
at tryOnTimeout (timers.js:323:5)
at Timer.listOnTimeout (timers.js:290:5)
<21:11:37 28/04/2019> -- |undefined -- ERROR: uncaughtException: Heartbeat timeout
Error: Heartbeat timeout
at Heart. (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:427:19)
at emitNone (events.js:106:13)
at Heart.emit (events.js:208:7)
at Heart.runHeartbeat (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/heartbeat.js:88:17)
at ontimeout (timers.js:498:11)
at tryOnTimeout (timers.js:323:5)
at Timer.listOnTimeout (timers.js:290:5)
at listOnTimeoutNT (timers.js:369:26)
at _combinedTickCallback (internal/process/next_tick.js:136:11)
at process._tickCallback (internal/process/next_tick.js:181:9)
<21:11:37 28/04/2019> -- |undefined -- ERROR: uncaughtException: Heartbeat timeout
Error: Heartbeat timeout
at Heart. (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:427:19)
at emitNone (events.js:106:13)
at Heart.emit (events.js:208:7)
at Heart.runHeartbeat (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/heartbeat.js:88:17)
at ontimeout (timers.js:498:11)
at tryOnTimeout (timers.js:323:5)
at Timer.listOnTimeout (timers.js:290:5)
at listOnTimeoutNT (timers.js:369:26)
at _combinedTickCallback (internal/process/next_tick.js:136:11)
at process._tickCallback (internal/process/next_tick.js:181:9)
<21:11:37 28/04/2019> -- |producer -- DEBUG: Error while creating producer: Error: timed out
<21:13:33 28/04/2019> -- |undefined -- ERROR: uncaughtException: Heartbeat timeout
Error: Heartbeat timeout
at Heart. (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:427:19)
at emitNone (events.js:106:13)
at Heart.emit (events.js:208:7)
at Heart.runHeartbeat (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/heartbeat.js:88:17)
at ontimeout (timers.js:498:11)
at tryOnTimeout (timers.js:323:5)
at Timer.listOnTimeout (timers.js:290:5)
<21:13:37 28/04/2019> -- |undefined -- ERROR: uncaughtException: Heartbeat timeout
Error: Heartbeat timeout
at Heart. (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/connection.js:427:19)
at emitNone (events.js:106:13)
at Heart.emit (events.js:208:7)
at Heart.runHeartbeat (/opt/flowbroker/orchestrator/node_modules/amqplib/lib/heartbeat.js:88:17)
at ontimeout (timers.js:498:11)
at tryOnTimeout (timers.js:323:5)
at Timer.listOnTimeout (timers.js:290:5)

Ticket analyis (Dec, 27th): Move to test (Rafael is responsible for testing)

Done.


Trying to log into LDAP even when user is not LDAP user

Hi Andreas,
we've spotted that when I log in as admin who has only WP account, I get the following message in the error log:
[26-Feb-2021 11:47:20 UTC] [AuthLDAP] User 'karolina' logging in
[26-Feb-2021 11:47:20 UTC] [AuthLDAP] about to do LDAP authentication
[26-Feb-2021 11:47:20 UTC] [AuthLDAP] connect to LDAP server
[26-Feb-2021 11:47:20 UTC] [AuthLDAP] No bind successfull. Exception thrown in line 70
[26-Feb-2021 11:47:20 UTC] PHP Notice:  No bind successfull. Exception thrown in line 70 in C:\laragon\www\getmantaportal\wp-content\plugins\authldap\authLdap.php on line 467
[26-Feb-2021 11:47:20 UTC] PHP Stack trace:
[26-Feb-2021 11:47:20 UTC] PHP   1. {main}() C:\laragon\www\getmantaportal\index.php:0
[26-Feb-2021 11:47:20 UTC] PHP   2. require() C:\laragon\www\getmantaportal\index.php:17
[26-Feb-2021 11:47:20 UTC] PHP   3. require_once() C:\laragon\www\getmantaportal\wp-blog-header.php:13
[26-Feb-2021 11:47:20 UTC] PHP   4. require_once() C:\laragon\www\getmantaportal\wp-load.php:37
[26-Feb-2021 11:47:20 UTC] PHP   5. require_once() C:\laragon\www\getmantaportal\wp-config.php:123
[26-Feb-2021 11:47:20 UTC] PHP   6. do_action() C:\laragon\www\getmantaportal\wp-settings.php:579
[26-Feb-2021 11:47:20 UTC] PHP   7. WP_Hook->do_action() C:\laragon\www\getmantaportal\wp-includes\plugin.php:484
[26-Feb-2021 11:47:20 UTC] PHP   8. WP_Hook->apply_filters() C:\laragon\www\getmantaportal\wp-includes\class-wp-hook.php:311
[26-Feb-2021 11:47:20 UTC] PHP   9. WC_Form_Handler::process_login() C:\laragon\www\getmantaportal\wp-includes\class-wp-hook.php:287
[26-Feb-2021 11:47:20 UTC] PHP  10. wp_signon() C:\laragon\www\getmantaportal\wp-content\plugins\woocommerce\includes\class-wc-form-handler.php:963
[26-Feb-2021 11:47:20 UTC] PHP  11. wp_authenticate() C:\laragon\www\getmantaportal\wp-includes\user.php:95
[26-Feb-2021 11:47:20 UTC] PHP  12. apply_filters() C:\laragon\www\getmantaportal\wp-includes\pluggable.php:549
[26-Feb-2021 11:47:20 UTC] PHP  13. WP_Hook->apply_filters() C:\laragon\www\getmantaportal\wp-includes\plugin.php:212
[26-Feb-2021 11:47:20 UTC] PHP  14. authLdap_login() C:\laragon\www\getmantaportal\wp-includes\class-wp-hook.php:287
[26-Feb-2021 11:47:20 UTC] PHP  15. trigger_error() C:\laragon\www\getmantaportal\wp-content\plugins\authldap\authLdap.php:467

That means it tries to log me in LDAP even when get_user_meta($uid, 'authLDAP') returns null since it's not set. Maybe I'm seeing wrong, but there is no check for this parameter before the connection to LDAP?
The login itself goes as expected and I log in, but this is weird since I would expect to skip the authLdap process entiery.

Now I had formulated a very thought through response why we need to do the LDAP authentication first. And then I reread your post and completely deleted it as you could be right. When `get_user_meta($uid, 'authLDAP') returns 'null', we should not need to do the LDAP authentication you could think.
I'm just thinking about whether there can be cases where someone logs in with name  but the username retrieved from LDAP would be different and so they might be using a different account then.
As currently it is i.e. possible to authenticate via the LDAP field wordpressUid which contains "me". That ldapuser has a uid of userA so for wordpress that user will have the uid userA. I'm not 100% sure whether that might open a loophole that can be exploited.
Nevertheless: After searching the ldap for that user I can either see that there is no user with that identifier so I take the literal one of I find a user with that identifier, then I take the converted one and check that against the users meta-data. If the user has no ldap-flag set, I can skip the rest of the authentication process.
I might implement that later...

@heiglandreas You're right with the rest of the user login process, no troubles on that side.
I haven't tested any other case than logging in with the email and there could be something going wrong. And it's probably better to be safe than sorry and I definitely don't want to open a loophole for an exploit (that's why I asked first not tried to fix it). Thanks for your feedback


Update release for current branch

Signed-off-by: Jenny Li<EMAIL_ADDRESS>

/lgtm

/kind improvement


Add plain text option to translate method

Q
A

Branch?
"master"

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Tests pass?
yes, but no new tests written

Currently, the translate method always treats the source and target text as HTML.  Although the Google API behaves this way by default, they provide a "format" option for cases in which a plain text translation is preferred.  This PR implements that option.
Without this option, the method may return HTML escape sequences for some characters.  For example, the apostrophe in "je n'ai pas".  If we're saving the translation result in a database, we want to save "je n'ai pas" not "je nl'pas".  There may be other cases in which this option is useful.

Hi @jeffclemens,
Thank you for this great addition!
Can you please add a little unit test to check that the format parameter is correctly sent?
Thank you!

@eko , is that what you meant?

@jeffclemens Yes, thank you :)

Is it possible to get this tagged as a new version?

@jeffclemens tag 1.1.3 is now available with your feature. Thank you and sorry for the delay.


☂ eslint-plugin-vitest

[!WARNING]
This comment is maintained by CI. Do not edit this comment directly.
To update comment template, see https://github.com/oxc-project/oxc/tree/main/tasks/lint_rules

This is tracking issue for eslint-plugin-vitest.
There are 51(+ 1 deprecated) rules.

7/8 recommended rules are remaining as TODO
43/43 not recommended rules are remaining as TODO

To get started, run the following command:
just new-vitest-rule <RULE_NAME>

Then register the rule in crates/oxc_linter/src/rules.rs and also declare_all_lint_rules at the bottom.
Recommended rules

  ✨: 1, 🚫: 0 / total: 8

Status
Name
Docs

vitest/no-identical-title
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-identical-title.md

vitest/expect-expect
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/expect-expect.md

vitest/no-commented-out-tests
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-commented-out-tests.md

✨
vitest/no-import-node-test
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-import-node-test.md

vitest/valid-title
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/valid-title.md

vitest/valid-expect
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/valid-expect.md

vitest/require-local-test-context-for-concurrent-snapshots
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/require-local-test-context-for-concurrent-snapshots.md

vitest/valid-describe-callback
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/valid-describe-callback.md

✨ = Implemented, 🚫 = Not supported

Not recommended rules

  ✨: 0, 🚫: 0 / total: 43

Status
Name
Docs

vitest/prefer-lowercase-title
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-lowercase-title.md

vitest/max-nested-describe
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/max-nested-describe.md

vitest/no-focused-tests
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-focused-tests.md

vitest/no-conditional-tests
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-tests.md

vitest/consistent-test-it
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/consistent-test-it.md

vitest/prefer-to-be
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be.md

vitest/no-hooks
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-hooks.md

vitest/no-restricted-vi-methods
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-restricted-vi-methods.md

vitest/consistent-test-filename
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/consistent-test-filename.md

vitest/max-expects
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/max-expects.md

vitest/no-alias-methods
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-alias-methods.md

vitest/no-conditional-expect
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-expect.md

vitest/no-conditional-in-test
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-conditional-in-test.md

vitest/no-disabled-tests
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-disabled-tests.md

vitest/no-duplicate-hooks
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-duplicate-hooks.md

vitest/no-large-snapshots
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-large-snapshots.md

vitest/no-interpolation-in-snapshots
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-interpolation-in-snapshots.md

vitest/no-mocks-import
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-mocks-import.md

vitest/no-restricted-matchers
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-restricted-matchers.md

vitest/no-standalone-expect
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-standalone-expect.md

vitest/no-test-prefixes
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-test-prefixes.md

vitest/no-test-return-statement
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-test-return-statement.md

vitest/prefer-called-with
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-called-with.md

vitest/prefer-to-be-falsy
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-falsy.md

vitest/prefer-to-be-object
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-object.md

vitest/prefer-to-be-truthy
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-be-truthy.md

vitest/prefer-to-have-length
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-have-length.md

vitest/prefer-equality-matcher
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-equality-matcher.md

vitest/prefer-strict-equal
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-strict-equal.md

vitest/prefer-expect-resolves
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-expect-resolves.md

vitest/prefer-each
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-each.md

vitest/prefer-hooks-on-top
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-hooks-on-top.md

vitest/prefer-hooks-in-order
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-hooks-in-order.md

vitest/prefer-mock-promise-shorthand
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-mock-promise-shorthand.md

vitest/prefer-snapshot-hint
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-snapshot-hint.md

vitest/require-top-level-describe
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/require-top-level-describe.md

vitest/require-to-throw-message
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/require-to-throw-message.md

vitest/require-hook
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/require-hook.md

vitest/prefer-todo
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-todo.md

vitest/prefer-spy-on
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-spy-on.md

vitest/prefer-comparison-matcher
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-comparison-matcher.md

vitest/prefer-to-contain
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-to-contain.md

vitest/prefer-expect-assertions
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/prefer-expect-assertions.md

✨ = Implemented, 🚫 = Not supported

Deprecated rules

  ✨: 0, 🚫: 0 / total: 1

Status
Name
Docs

vitest/no-done-callback
https://github.com/veritem/eslint-plugin-vitest/blob/main/docs/rules/no-done-callback.md

✨ = Implemented, 🚫 = Not supported

Does vitest/no-commented-out-tests seem to be duplicated with jest/no-commented-out-tests?

We need to map jest to vitest rules in tasks/lint_rules, they have the same implementation.

Justfile seems to not have the command just new-vitest-rule. Should we add it or use just new-jest-rule? @Boshen

Justfile seems to not have the command just new-vitest-rule. Should we add it or use just new-jest-rule? @Boshen

Add this is welcome.

Rules Implemented but Not Listed in the Table
Some rules are already implemented but not marked in the table. They are located in src/rules/jest because they share similar logic between Jest and Vitest.
The following files mention Vitest, suggesting they may also apply to Vitest:
List of files mentioning Vitest

> grep -rl "vitest"  src/rules/jest
./expect_expect.rs
./no_identical_title.rs
./no_conditional_in_test.rs
./valid_expect.rs
./consistent_test_it.rs
./no_disabled_tests.rs
./no_commented_out_tests.rs
./valid_describe_callback.rs
./no_focused_tests.rs
./no_alias_methods.rs
./no_restricted_jest_methods.rs
./no_test_prefixes.rs
./no_conditional_expect.rs
./prefer_hooks_in_order.rs

Unable to Register Rules with the Same Name Across Different Plugins
Some rules have the same name as those in the Jest plugin, which prevents them from being registered in Vitest. This is because the macro cannot differentiate identical rule names between plugins.
For more details, see: #6520

@no-yan I've opened a PR which will automatically keep this issue in sync with that list. Thanks for making a note of it. Hopefully that will make this more clear.
https://github.com/oxc-project/oxc/pull/6667
cc @DonIsaac @Boshen


Fix: File Watcher Ignore List

We were feeding the ignore crate the wrong data and it would build an empty ignore list. Fixing this greatly speeds up serve start times as we aren't checking every file in folders like target.

THANK YOU! I figured we were doing something wrong here.


[hardening] cheaper storage fee and rebate conservation checks to be …

…called in prod
Currently, we have check_sui_conservation, which actually checks two different things

We conserve SUI in the Move part of an object
We conserve SUI in the storage_rebate part of an object w.r.t the storage fees, storage rebate, and non-refundable storage rebate of a tx

All of the conservation bugs we have identified have been in (2), and (1) is much, much more expensive to check because it involves computing type layouts for all input/output objects, re-serializing them, and recursively iterating through every field and vector entry looking for SUI.
This PR exposes a separate function that only checks (2), which should be cheap enough to call in production, and will save us from future issues that look like the ones we have seen in the past. This raises an invariant_violation for the tx currently being executed if we cannot verify conservation.

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Data migration is far ideal than data wipe imo


[SPARK-30495][SS] Consider spark.security.credentials.kafka.enabled and cluster configuration when checking latest delegation token

What changes were proposed in this pull request?
Spark SQL Kafka consumer connector considers delegation token usage even if the user configures sasl.jaas.config manually.
In this PR I've added spark.security.credentials.kafka.enabled and cluster configuration check to the condition.
Why are the changes needed?
Now it's not possible to configure sasl.jaas.config manually.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing + additional unit tests.

Can one of the admins verify this patch?

Test build #116638 has started for PR 27191 at commit 13c444f.

Test build #116638 has finished for PR 27191 at commit 13c444f.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/116638/
Test FAILed.

Test build #116642 has started for PR 27191 at commit 44fdb5e.

Test build #116642 has finished for PR 27191 at commit 44fdb5e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Test build #116698 has started for PR 27191 at commit d874a5a.

cc @vanzin

Test build #116698 has finished for PR 27191 at commit d874a5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Test build #116710 has started for PR 27191 at commit 541e986.

If possible it's better to have another E2E test on Kafka security without delegation token.

The end to end test is really heavy weight and hard to maintain. Even after 2-3 fixes we've seen another failure most probably because of the environment. I would keep the number of such tests low not to irritate others with failures.

On a somewhat unrelated note (not to be addressed in this PR), does the producer side need checks related to delegation tokens expiring too?

If possible it's better to have another E2E test on Kafka security without delegation token.

The end to end test is really heavy weight and hard to maintain. Even after 2-3 fixes we've seen another failure most probably because of the environment.

For me, it sounds as the test is really heavy weight and flaky because of the "environment", right? If they can be tested in same suite with setting up environment only once, it won't hurt. Doesn't mean to address in this PR, but E2E test would avoid introducing regression - this bug is a regression for the valid use case. I'll try to deal with the test addition.

On a somewhat unrelated note (not to be addressed in this PR), does the producer side need checks related to delegation tokens expiring too?

Once delegation token is updated, SASL_JAAS_CONFIG will be different which leads to initialize and use different Producer instance. I've updated it in doc as well. https://github.com/apache/spark/commit/eefcc7d762a627bf19cab7041a1a82f88862e7e1#diff-291798b0ad8f1d2c5c21dff52922e3d8R813-R814


Juttle CLI detects Juttle errors incorrectly

Juttle CLI assumes that any error with location info is a Juttle error and any error without it is not. This assumption is invalid. Juttle errors are those descending from JuttleError, all other errors are internal errors (and should generally be treated as bugs). Most Juttle errors have location info, but some do not (see e.g. #519).
The CLI needs to be fixed to detect Juttle errors using instanceof and display them correctly whether they contain location info or not.

Probably something for you, @mstemm.


Check on Windows, esp. file separator behavior

I'm on windows, so if you have a test, I can run it.

Nothing specific; mostly I was concerned about using / as a file separator, but if the library is working for you then it's probably not an issue.
After further reading, I think separators might just be a problem for scripts that actually run command-line processes since cmd.exe does have trouble there.


feat: add @sapphire/lexure package

TODO

[ ] Docs
[ ] Tests

omg omg omg omg
I can write the docs at a later point (more like help with docs)

PR still has TODOs so converted to draft


Add improved input facilities

Aaron Clarke was asking about how to get input in Harlan programs. Currently there's not a lot of support for this, although you might be able to get some done by using extern to call C functions. Still, a better API would be better. Aaron would like to be able to do something like this:
(let ((temperatures (load-vector "temp.dat")))...)

or
(let ((features (load-matrix (argv 1))))...)

1f18a0d3e8030b4e3c98084b9d6014c02ae389b8 includes some more useful input functions, although still not quite as high level as suggested by Aaron.


get RuntimeError when use getattr() to get a non-exist attribute on asgiref.local.Local

python version: 3.9.18
nuitka version: 2.1.5
pip list result:
Package           Version
----------------- -------
asgiref           3.8.1
Django            4.2.11
Nuitka            2.1.5
ordered-set       4.1.0
pip               23.0.1
setuptools        58.1.0
sqlparse          0.4.4
typing_extensions 4.11.0
zstandard         0.22.0

[notice] A new release of pip is available: 23.0.1 -> 24.0
[notice] To update, run: pip install --upgrade pip

code example:
from asgiref.local import Local

if __name__ == '__main__':
    a = Local(thread_critical=True)
    getattr(a, 'default')

nuitka package cmd: python -m nuitka --onefile --show-progress test.py
expected result:
AttributeError: '_thread._local' object has no attribute 'default'

what i get:
Traceback (most recent call last):
  File "/tmp/onefile_81632_1712738386_558915/test.py", line 6, in <module>
    getattr(a, 'default')
  File "/tmp/onefile_81632_1712738386_558915/asgiref/local.py", line 118, in __getattr__
RuntimeError: No active exception to reraise

This is looking like an issue of compatibility with getattr on the storage while an exception was active in the context generator function used here:
    def __getattr__(self, key):
        with self._lock_storage() as storage:
            return getattr(storage, key)

With Python 3.10, this is not happening.

I believe, we finally need to generate the getattr, hasattr, variations that are floating around, where we also can have a look at taking advantage of the ability to cache attribute lookups starting with 3.11, this probably a failure to save and restore an exception, dropping the failure of getattr attempt on some object method. It seems this one is confusing it:
    def __getattr__(self, key):
        storage_object = self._data.get({})
        try:
            return storage_object[key]
        except KeyError:
            raise AttributeError(f"{self!r} object has no attribute {key!r}")

I am confused to see that you report using 3.9, but alas, maybe there is that too, not sure now. You were not using the proper reporting output to be sure what it really was.

2.1.5
Commercial: None
Python: 3.9.18 (main, Mar 13 2024, 11:01:03)
Flavor: Unknown
Executable: /root/pycharm_project_593/web/venv/bin/python
OS: Linux
Arch: aarch64
Distribution: Kylin V10
Version C compiler: /usr/bin/gcc (gcc 7.3.0).

It's strange that my 3.10 didn't do it, but totally possible the versions of asgiref just didn't align, and then this error would be more general indeed. Once I get to it, I will check it out.

Wanted to jump in here - I am also getting this issue with Django and asgiref:
2.3.11
Commercial: None
Python: 3.10.0 | packaged by conda-forge | (default, Nov 10 2021, 13:20:59) [MSC v.1916 64 bit (AMD64)]
Flavor: Anaconda Python
Executable: C:\Users\scott\anaconda3\envs\exe\python.exe
OS: Windows
Arch: x86_64
WindowsRelease: 11
Version C compiler: C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.40.33807\bin\Hostx64\x64\cl.exe (cl 14.3).
asgiref==3.8.1
Django==5.0.6
Traceback (most recent call last):
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\startup.py", line 38, in
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django_init_.py", line 24, in setup
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\apps\registry.py", line 116, in populate
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\apps\config.py", line 269, in import_models
File "importlib.py", line 126, in import_module
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\contrib\auth\models.py", line 3, in
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\contrib\auth\base_user.py", line 59, in
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\db\models\base.py", line 143, in new
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\db\models\base.py", line 371, in add_to_class
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\db\models\options.py", line 243, in contribute_to_class
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\utils\connection.py", line 15, in getattr
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\django\utils\connection.py", line 58, in getitem
File "C:\Users\scott\website\exe_only\site\STARTU~1.DIS\asgiref\local.py", line 117, in getattr
RuntimeError: No active exception to reraise
Command used:
python -m nuitka startup.py --include-data-files=params.txt=params.txt --enable-plugin=tk-inter --standalone --include-package=django --include-package-data=django --include-package=django.views --include-package-data=django.views
To reproduce
import django
from django.conf import settings
import os
os.environ.setdefault("DJANGO_SETTINGS_MODULE", "settings")
settings.configure(INSTALLED_APPS=[
"django.contrib.admin",
"django.contrib.auth",
"django.contrib.contenttypes",
"django.contrib.sessions",
"django.contrib.messages",
"django.contrib.staticfiles",
])
django.setup()

There has been some Django progress recently, but on an empty Django project with 3.12, I am now seeing this for myself easily, and will be able to reproduce and fix this hopefully. I noticed that this is in a spot, where I recently added an assertion, because of issues seen in other places, hopefully it turns out to be triggering with --debug in my current development version too.

So, this seems to be about the with statement, and either its re-formulation or try/finally in generally, being able to become confused. It seems generated code doesn't fetch the exception when checking it in the exception handler, while it ran code to change things there, so the re-raise is triggered, although it ought not to, since there wasn't an actual exception.
This can be indicative of 2 things, the re-raise is checking for the wrong value, and/or the clearing in getattr sets the "wrong" value, and there is maybe a need to re-factor the exception handling code to fetch the exception so it cannot get changed behind its back.


Pass TFJob API version instead of hardcoding it

Related https://github.com/kubeflow/kubeflow/issues/852
/assign @jlewi

This change is

/lgtm
/approve


Regarding open source

@alfredgu001324 Great and meaningful work on online mapping and behavior prediction! I would be very appreciative if you could release the code soon.

Thanks for your interest in our work! We are planning to release it by the end of the week or early next week!


fix this: mulitiple things

[x] 1. link the routes properly
[x] 2. add form validation in login page and sign up page
[x] 3. improve the css of mat card

successfully completed this issue fixing


Gradient Ascent for Caffe and Torch + Layer Viz

Do Not Merge!
 Gradient Ascent For Torch and Caffe + Layer Visualizations.
Things left to do:

Add receptive field calculation for Torch . Perhaps this step could be moved out of tools, and into the views before rendering (giving the user options to dev # deviations, size of bounding box etc.

Customizeable parameters: Currently these are hardcoded into the python and lua scripts. It would be nice to have a few set of options available from the UI like: "Best Alexnet" or "Best Googlenet" etc.

Additional Regularization Options (I didn't have time to implement all the params from Deep Vis Toolbox)

Use mean file during Gradient Ascent. (The mean file is sent as a param for both jobs but not used, as it seems to make certain black and white jobs like LeNet(MNIST) return odd regularizations) . Perhaps this could be toggles with a checkbox in the UI

Layer Visualizations - Caffe

Layer Visualizations - Torch (Notice Lack of Receptive Field Calculations)

Gradient Ascent - Torch

How does this PR relate to #968 and #937?

Closing for now (there are merge conflicts anyway). I'd like to revisit this.
I've backed up this work at https://github.com/lukeyeager/DIGITS/tree/backup/lucas/gradientAscentWithLayerVis.

@lukeyeager @TimZaman Sorry school has been super busy. But hopefully soon I'll get some time to revisit this and create some smaller pull requests (such as layer visualizations for Torch models, etc.) once I have a machine up and running.


bug: most .wav files playback corrupted

version: 0.9.2
some, though strangely not all wav files play completely mangled versions of the audio, typically as an incredibly loud screech or static.
many sounds are still vaguely recognizable, leaving me to believe it may be one or two bits off, but again, only for some files and not others.
i dont know enough about the wav format to know what would be causing this.. differing header length maybe? some other incorrect assumption about the file format?
aside from that, this is an otherwise excellent program.

Unfortunately that's a known problem with the audio import lib I use. Unfortunately, my knowledge on that stuff is non-existent so I can't fix it myself :(


Forecast chart

Forecast chart

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (dev@6d90088). Learn more about missing BASE report.
Report is 1 commits behind head on dev.

Additional details and impacted files
@@          Coverage Diff           @@
##             dev     #564   +/-   ##
======================================
  Coverage       ?   70.16%
======================================
  Files          ?      129
  Lines          ?     1354
  Branches       ?       78
======================================
  Hits           ?      950
  Misses         ?      404
  Partials       ?        0

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


Allow all members to see and participate in conflict resolution

It seems useful to allow all members to participate in conflict resolution processes.
From https://community.foodsaving.world/t/conflict-resolution-with-possibility-to-remove-user-from-group/201

While I understand why, some people found it weird that not everyone was able to vote but only the ones with editing rights. Especially as the notifications were not that visible, we reminded all that there is a process going on, but then not all could see it as they didn’t have these rights, which gave an unfair feeling.

I agree, it is a good idea to involve all active members. This way people also get involved in the group dynamics and see what is going on.
In Östersund I guess that some people don’t really understand the trust karrots and don’t know about the conflict feature. Thus they might be irritated if others talk about ongoing conflicts. But this is only a guess.

I started working on this yesterday and noticed some open questions:

who should be in the conversation by default?
new potential loophole: while an issue is ongoing, somebody adds fake users to the group that vote in their favor


date issue in osx

Checked out and run on OSX.
$ make
date: illegal option -- I
usage: date [-jnu] [-d dst] [-r seconds] [-t west] [-v[+|-]val[ymwdHMS]] ...
            [-f fmt date | [[[mm]dd]HH]MM[[cc]yy][.ss]] [+format]
▶ running gofmt…
▶ setting GOPATH…
▶ retrieving dependencies…
▶ building golint…
▶ running golint…
▶ building executable…

I have pushed a commit and I believe this should fix this issue. Tell me if not.

Thanks worked fine.


[tidy-html5] Added libtidy version 5.4.0

More information at http://www.html-tidy.org/.
I am not involved with the project, just wrote the port.
The #pragma execution_character_set("utf-8") lines conflicted with the /utf-8 flag, I figured they pretty much enable the same thing so I wrote a patch to remove the pragmas from the code rather than find a way to remove /utf-8 from c and cxx flags... which seems impossible from a port file anyway.

Thanks for the PR!
I added support for static builds as well, along with moving the tool into a more dedicated subfolder (tools\tidy-html5\).


Update top trends prompt

Updating top trends prompt to be more prescriptive and achieve better results across industries

LGTM -- tested and results looked good. Thanks for the suggestion and for the offline discussion on this improvement.


When skill is moved into a group it still appears under the old group on skill display and self reporting requests fail

When skill is moved into a group it still appears under the old group on skill display and self reporting requests fail
the underlying issue is that SkillDef.groupId is not properly updated when a skill is moved

PR: https://github.com/NationalSecurityAgency/skills-service/pull/1695


Run in parallel

Especially for CREST this would be a significant UX improvement.

For xtb this is done with the -P <processes> or --parallel <processes> flags.
For CREST this is done with the --T <n_threads> flag.
For CREST this definitely means threads not cores, and I assume "parallel processes" for xtb means threaded processes too.
The number of logical cores (i.e. threads) available on the system can be found with os.cpu_count().
People probably will want to leave e.g. conformer searches running in the background so probably a good idea to use 1 or 2 fewer than maximum by default so that they don't find their machine becomes extremely slow.


Tor Browser crashing on startup

Describe the bug
Tor browser crashes on start with the following output. I installed tor-browser-bundle-bin as a system package. Is there something I need to do in addition or something I need to do differently?
Steps To Reproduce
Steps to reproduce the behavior:

Start tor-browser from the command line.
Crashes with:

$ tor-browser
Crash Annotation GraphicsCriticalError: |[0][GFX1-]: glxtest: libpci missing (t=4.7125) [GFX1-]: glxtest: libpci missing
Fontconfig warning: "/nix/store/8s3w3vy7pvkvxhjjzndbqxybzsgj62i3-tor-browser-bundle-bin-11.0/share/tor-browser/TorBrowser/Data/fontconfig/fonts.conf", line 85: unknown element "blank"
Dec 05 17:44:09.539 [notice] Tor <IP_ADDRESS> (git-d5efc2c98619568e) running on Linux with Libevent 2.1.12-stable, OpenSSL 1.1.1l, Zlib 1.2.11, Liblzma N/A, Libzstd N/A and Glibc 2.33 as libc.
Dec 05 17:44:09.539 [notice] Tor can't help you if you use it wrong! Learn how to be safe at https://www.torproject.org/download/download#warning
Dec 05 17:44:09.553 [notice] Read configuration file "/nix/store/8s3w3vy7pvkvxhjjzndbqxybzsgj62i3-tor-browser-bundle-bin-11.0/share/tor-browser/TorBrowser/Data/Tor/torrc-defaults".
Dec 05 17:44:09.553 [notice] Read configuration file "/home/username/.local/share/tor-browser/TorBrowser/Data/Tor/torrc".
Dec 05 17:44:09.582 [notice] Opening Control listener on /home/username/.local/share/tor-browser/TorBrowser/Data/Tor/control.socket
Dec 05 17:44:09.582 [notice] Opened Control listener connection (ready) on /home/username/.local/share/tor-browser/TorBrowser/Data/Tor/control.socket
Dec 05 17:44:09.582 [notice] DisableNetwork is set. Tor will not make or accept non-control network connections. Shutting down all existing connections.
Dec 05 17:44:09.000 [notice] Parsing GEOIP IPv4 file /nix/store/8s3w3vy7pvkvxhjjzndbqxybzsgj62i3-tor-browser-bundle-bin-11.0/share/tor-browser/TorBrowser/Data/Tor/geoip.
Dec 05 17:44:09.000 [notice] Parsing GEOIP IPv6 file /nix/store/8s3w3vy7pvkvxhjjzndbqxybzsgj62i3-tor-browser-bundle-bin-11.0/share/tor-browser/TorBrowser/Data/Tor/geoip6.
Dec 05 17:44:10.000 [notice] Bootstrapped 0% (starting): Starting
Dec 05 17:44:10.000 [notice] Starting with guard context "default"
Dec 05 17:44:10.000 [notice] Delaying directory fetches: DisableNetwork is set.
Dec 05 17:44:10.000 [notice] New control connection opened.
Dec 05 17:44:10.000 [notice] New control connection opened.
Dec 05 17:44:10.000 [notice] New control connection opened.
Dec 05 17:44:10.000 [notice] New control connection opened.
Dec 05 17:44:11.000 [notice] DisableNetwork is set. Tor will not make or accept non-control network connections. Shutting down all existing connections.
Fontconfig warning: "/nix/store/8s3w3vy7pvkvxhjjzndbqxybzsgj62i3-tor-browser-bundle-bin-11.0/share/tor-browser/TorBrowser/Data/fontconfig/fonts.conf", line 85: unknown element "blank"
Dec 05 17:44:13.000 [notice] New control connection opened.
Dec 05 17:44:13.000 [notice] New control connection opened.
fatal allocator error: invalid uninitialized allocator usage
Redirecting call to abort() to mozalloc_abort

Dec 05 17:44:15.000 [notice] Owning controller connection has closed -- exiting now.
Exiting due to channel error.
Dec 05 17:44:15.000 [notice] Owning controller connection has closed -- exiting now.
Dec 05 17:44:15.000 [notice] Owning controller connection has closed -- exiting now.
Dec 05 17:44:15.000 [notice] Catching signal TERM, exiting cleanly.

Expected behavior
tor-browser starts.
Additional context
Installed tor-browser-bin-bundle in the configuration.nix file.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
these paths will be fetched (0.06 MiB download, 0.30 MiB unpacked):
  /nix/store/0wxn3wnk6qiv5kzl0w8abv9jzh8szgqz-bash-interactive-5.1-p8-dev
copying path '/nix/store/0wxn3wnk6qiv5kzl0w8abv9jzh8szgqz-bash-interactive-5.1-p8-dev' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.81, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.16`
 - channels(root): `"nixos-21.11.333896.a640d8394f3"`
 - channels(usename): `"home-manager-21.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Appears to be the same as https://github.com/NixOS/nixpkgs/issues/146401 so closing.


Please remove social media tracking

Hello I am one of the maintainers of Hugo Themes.
As described in https://github.com/gohugoio/hugoThemes/issues/564 your theme features share buttons from social media networks that track users in \simpleit-hugo-theme\layouts\partials\share_buttons.html:
I kindly request that you remove the tracking code.
I understand that your theme has been approved with the tracking code but currently the guidelines for maintaining a theme on the Hugo Themes list are about to change and the requirement to not include tracking in Hugo Themes will be included in the new guidelines.
I suggest that you re-write to the above partial as described in this resource: https://sharingbuttons.io
If you have any questions then please do not hesitate to ask.
Looking forward to your reply.
cc: @digitalcraftsman

Hello @onedrawingperday , I am already using a static version for share buttons by default.
After reading https://github.com/gohugoio/hugoThemes/issues/535 and
https://github.com/gohugoio/hugoThemes/issues/564 I understand that the requirement is to not have them enabled in the Hugo Themes site.
By default this theme demo uses a static set of buttons partial based in https://sharingbuttons.io which is the one published in Hugo Themes site.
The partial you referenced uses a configuration parameter that let the user decide the static or dynamic version:
  # Render Share buttons static or with javascript
  shareButtonsType = "static" #<static|javascript>

which is being used at share_buttons.html
    {{ if (eq .Site.Params.shareButtonsType "static")}}
    {{ partial "share_static.html" . }}
    {{else}}
    //  load dynamic ones
    {{end}}

Please let me know if you have any questions about it.

@marcanuy
Thank you for clarifying.


Download Complete Message creates new timestamp

Expected behavior
When executing "Download complete message" the message should retain its original timestamp and position in the message list.
Actual behavior
When executing "Download complete message" the message picks up a new timestamp (the current time) and is refiltered to the top of the message list, losing its position in the "sorted list.
These attachments show screenshot of the same message before and after downloading complete message.

Steps to reproduce

"Download complete message"

Environment
K-9 Mail version: 5.600
Android version: Oreo 8.1.0
Account type (IMAP, POP3, WebDAV/Exchange): POP3

See #1356


MP GO:0000212 - meiotic spindle organization

should be part_of
GO:0140013 - meiotic nuclear division
@Antonialock  do you agree?

The mitotic one isn't either. It this true though? The organization of the spindles also dictates where cytokinesis will occur. Is that part of nuclear division?

I don't think the organization of the spindles determines where cytokinesis occurs. A  signalling pathway emenating from the SPB/centrosome determines where/when cytokinesis occurs.

so endomeiosis might be a thing in parthogenically developing eggs..
https://www.tandfonline.com/doi/pdf/10.1080/11250009309355818

but does that affect that the meiotic spindle   would only exist in M-phase (as in mitosis)?
(I don't understand what it is!)

the point was that those organisms seem to do meiosis without nuclear division? It is probably a very obscure example, and very old.

I'm a little hesitant to make changes in this area since I know that people with more knowledge than I have about meiosis & mitosis have worked in this area. Eurie Hong did her PhD in a meiosis lab.
I will point out that the term "meiotic spindle organization" has a child term of "spindle assembly involved in meiosis" which is already a  part_of "meiotic nuclear division".
-- meiotic spindle organization
--- is_a meiotic spindle disassembly
--- part_of meiotic spindle elongation
--- part_of meiotic spindle pole body organization
--- is_a spindle assembly involved in meiosis
-- meiotic nuclear division
--- part_of meiotic chromosome segregation
--- part_of meiotic spindle midzone assembly
--- part_of spindle assembly involved in meiosis
Also note that the term "meiotic spindle organization" has "meiotic spindle disassembly" as an is_a child term.
Thus, I am inclined so say that this has already been carefully considered and should not be changed in the way you suggest.

Hi Karen,
It might not be  correct because the meiotic spindle  is also involved in forsepore membrane formation.
However, most of the mitotic spindle terms have been completely overhauled since Eurie left (mainly by feedback from input from spindle experts).  This is the final open ticket:
https://github.com/geneontology/go-ontology/issues/13461
For mitotic spindle organization, the  spindle  does not form until  mitotic prophase (i.e. is  part of mitotic nuclear division), and is desassembly at the end of nuclear division.
I therefore think that   "mitotic spindle organization" should be part of "mitotic nuclear division" (currently it isn't).  So  I think that could be added, but it  doesn't really matter as the term is not much used for direct annotation.
I noted it here in case it comes up again. No action required though.
Val


Create containerized option for running docker examples

In order to cultivate an easy-to-view running example, we should use Docker.

Node and NPM version are controlled
Storybook version is controlled
User's machine does not get polluted with any dependencies

@mtheoryx Thanks for the contribution! I tried it and it works perfectly.
Can you also add volumn mapping in docker environment? As the user has live reload ability by running npm run storybook to play the examples, we want to keep this ability for the docker way as well.

@artecher Great idea, I added that now, and hot reload is working!
Additionally, I added the docker configuration files to the .npmignore since we probably don't want to be shipping that around in a published module.
Cheers!


compliant with devise

It will be simpler to run any rails app as web interface instead js web if we will store authentication info compliant with devise defaults. What are you think about it?

I haven't used devise, but it's easy to use custom authentication mechanisms in Vines.  The storage interface is designed to allow this type of customization.  You could subclass one of the existing Vines::Storage classes and override the authenticate method to use devise.

Ok, I will do it. Thanks for your job, it's really awesome.


Using the front camera for detection causing boxes to be mirrored

Thank you for the great work here. I am constantly trying to experiment with variations of your work for my own purposes and for one of my applications I am trying to use the object dectection through my front camera (on my iPad) so the I can see what being detected while moving in front my my screen.
I have made the following change in VideoCapture.swift to capture the video stream through the front camera:
Original code chunk:
 guard let captureDevice = AVCaptureDevice.default(for: AVMediaType.video) else {
      print("Error: no video devices available")
      return false
    }

My changed chunk:
    guard let captureDevice = AVCaptureDevice.default(.builtInWideAngleCamera, for: AVMediaType.video, position: .front) else {
        print ("Error: no video devices available")
        return false
    }

All works well and good except that the bounding boxes are mirrored, meaning that if I am holding an apple in my left hand and a banana in my right hand, the boxes are swapped around these objects. I understand that this might have to do something with the iPhones front camera mirroring the video display but I am not sure how to tackle this issue. Any guidance will be very useful. Thank you

I'm not sure where this goes wrong or why but an easy fix is to flip the coordinates of the bounding boxes too: box.x = screen width - box.x.

That sounds like a neat solution, it would be great if you could advice on where this code modification is to be made? I went through most of the .swift files but wasn't able to find a "box.x" variable. Thank you

On this line,
https://github.com/hollance/YOLO-CoreML-MPSNNGraph/blob/master/TinyYOLO-CoreML/TinyYOLO-CoreML/YOLO.swift#L94
change it to:
let x = blockSize - ((Float(cx) + sigmoid(tx)) * blockSize)

This happens because the image from the front camera is mirrored. There are other solutions to this, but mirroring the x coordinate of the bounding box is probably easiest (and fastest).

Thank you very much, I will test this on my iPad tomorrow and close the issue accordingly if all goes well :)

Hi there! So I attempted to make the above mentioned change with no luck. There are no more visible bounding boxes on the screen anymore (only see edges of a box at the left end of the iPad screen). Is there another option, or is there an additional step that needs to be taken in combination to the change in x? Thank you

I think you need to subtract from the width (416), not from block size in order to mirror. This line did it for me:
let x = 416 - ((Float(cx) + sigmoid(tx)) * blockSize)


FancyZones Editor - Snap rulers to current windows

Description of the new feature / enhancement
Rulers should snap to the edges of existing windows.
Maybe it could be that it only happens when you hold shift while dragging.
Scenario when this would be used?
It's common to already have your windows arranged in the layout you want.
Currently it's hard to precisely align a ruler to the edge of an existing window.
Supporting information
No response

Could you please explain what you mean by "ruler"? Thanks!
/needinfo

Could you please explain what you mean by "ruler"? Thanks!

The zone dividers


#122 closes #123 white and off white for sections completed numbers i…

…n case study section have less whitespace

Current coverage is 47.18%

Merging #145 into master will not affect coverage as of 54e7692

@@            master    #145   diff @@
======================================
  Files           18      18
  Stmts          284     284
  Branches         0       0
  Methods          0       0
======================================
  Hit            134     134
  Partial          0       0
  Missed         150     150

Review entire Coverage Diff as of 54e7692

Powered by Codecov. Updated on successful CI builds.


abseil-cpp: lock in provided vocabulary types

Description of changes
Abseil provides a special header that allows software distributions
to override compile option dependent features. If this file is left
as-is downstream packages will usually fail to link due to undefined
references.
See the upstream documentation for a more comprehensive explanation:
https://github.com/abseil/abseil-cpp/blob/lts_2021_11_02/absl/base/options.h
Up till now we allowed to bulid abseil itself with different C++
Standards to overcome this problem, but that approach didn't fully
solve it:

The override is an optional argument, so its easy to overlook.
Processes that see multiple variants of the library (becausse
they get pulled into the closure by different dependencies) are
subject to undefined behavior.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I built up to libtensorflow without problem, but nixpkgs-review is too much for my hardware.
resolves #185865

There is at least another reference at pkgs/applications/networking/instant-messengers/telegram/kotatogram-desktop/default.nix:72:19.

There is at least another reference at pkgs/applications/networking/instant-messengers/telegram/kotatogram-desktop/default.nix:72:19.

Thanks, I removed the remaining referencces.

Part of the reason cxxStandard was introduced as an overridable option was to avoid breaking or-tools (https://github.com/NixOS/nixpkgs/pull/143967), and indeed, this PR breaks or-tools.

Apparently absl::string_view doesn't implement a conversion operator to std::string_view, which would be needed to assign to std::string. I pushed a patch for or-tools to fix that failure.
I think we can try to switch over to the std provided implementations with 202206 and start phasing out the older LTSs soon.

@andersk would be more open to accepting this if I added myself as co-,maintainer?

Apparently absl::string_view doesn't implement a conversion operator to std::string_view, which would be needed to assign to std::string. I pushed a patch for or-tools to fix that failure.

I mean, of course it doesn’t, because you’ve now configured it to never use std::string_view. Since that isn’t a configuration or-tools supports, I’m somewhat worried that we’ll need to maintain a growing number of patches like this over time.
What do you think of allowing cxxStandard to vary as long as static = true (which it is for or-tools)?

I think we can try to switch over to the std provided implementations with 202206 and start phasing out the older LTSs soon.

Does 202206 change something relevant here?
Abseil’s options.h documents that forcing the std implementations “requires that all code using Abseil is built in C++17 mode or later”, which was evidently not the case at least as of #143967.

I mean, of course it doesn’t, because you’ve now configured it to never use std::string_view. Since that isn’t a configuration or-tools supports, I’m somewhat worried that we’ll need to maintain a growing number of patches like this over time.
What do you think of allowing cxxStandard to vary as long as static = true (which it is for or-tools)?

I think it's better to patch or-tools than to complicate the interface to abseil. I wouldn't expect this to require too many patches like this, but the risk of it is the reason I suggested to switch to the standard implementations in a somewhat timely manner.

I think we can try to switch over to the std provided implementations with 202206 and start phasing out the older LTSs soon.

Does 202206 change something relevant here?

No, but in my opinion a new version is a good opportunity to do the switch.

Abseil’s options.h documents that forcing the std implementations “requires that all code using Abseil is built in C++17 mode or later”, which was evidently not the case at least as of #143967.

grpc and dependents work fine with C++17. Fedora, Arch, and Homebrew all do this already, so we wouldn't be the only ones.

Personally I like the ability to choose cxxStandard of abseil. This allows to build with recommended settings of upstreams.

What do you think of allowing cxxStandard to vary as long as static = true (which it is for or-tools)?

@andersk does this offer still stand? I'd like to get the ball rolling again.

Sorry about the mass-notification. I made a mistake while re-targeting.

Please open a new pull request as people who were requested will now get a notification for any action


proposal: cmd/go: support cgo in test files

Currently it's impossible to use cgo in x_test.go files.
Given that we support coverage tests on cgo using packages,
there really is no reason that we can't support using cgo in
test files (and besides, we don't need to rewrite any x_test.go
files, so I don't know why we have such a limitation in the first
place. perhaps it's because we doesn't has a TestCgoFiles
field in go/build.Package?)
The main problem of not supporting cgo in test file is:
because there is no test build tag, there is no way to
export a cgo function for test that is not also included in the
package (true, deadcode elimination might remove those
functions, but what about C helpers needed by such function?
our linker can't do deadcode elimination on C functions).

Allowing cgo in tests adds complication. Why is it important to do this?

From the outside, I was surprised when I learned that cgo cannot appear in tests and thought it felt arbitrary. I still don't even really understand why, I just work around it.
I do think it can be rather annoying as well. If I define a Go function which takes a C.size_t argument, I can't write a table test for it very simply because I can't define a struct that contains C.size_t within my test file.
It would be great if test files could call C functions, but I really be glad if I were just able to reference C types.

@minux, do you know how many lines would need to change in the go command for this? If it's a small change then maybe it's OK to do.

I understand that this is handy, but I started to look into it and got stuck pretty quickly. The info about the package is https://golang.org/pkg/go/build/#Package and you'd need to add TestCgoFiles and maybe XTestCgoFiles, but then what about CgoCFLAGS? Does that collect all of them? What about CFiles, HFiles, etc?
It seems like a lot of trouble when there's a reasonable workaround for this not-too-common case already.

Closing since it is very difficult and doesn't seem worth it.


carify 'environment name' in challenge 3

Script requires an enviornment name, which was originally used to create the fhir resources. picking the same thing puts all new resources for anonymization and export into same RG. Probably want them in new rg, so should leave a hint that this is a New environment name

Added to Github


bug: Robot crashes and reboots after homing gantry

Overview
When running any protocol or even just the home gantry command, the robots starts to align the position of the pipettes above the trash bin, then just crashes / looses power and automatically reboots. video attached. it can be clearly seen when the blue led goes silent. (Opentrons OT2)
https://github.com/user-attachments/assets/6e7d0cf1-1abc-479f-8239-3c593cfa1311
Steps to reproduce
Run home gantry command.
Current behavior
The robot crashes and reboots when I try to start any protocol, or even just run the home gantry command (Opentrons OT-2)
Expected behavior
The home gantry command or whole protocols should execute, instead of the robot crashing and rebooting.
Operating system
Windows
System and robot setup or anything else?
Ethernet cable connection

I am experiencing the same thing combined with an error in the opentrons app
This robot's API server is not responding correctly to requests at IP address <IP_ADDRESS>
Did you find any fixes thus far?


⚠ Bofrost.at has degraded performance

In 62d0785, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2073 ms

Resolved: Bofrost.at performance has improved in b04098a.


Adding Column type to the column property

Could you kindly add the table column type ?
Thank you in advance.
Abdoessalam

My suggestion is :
To add Column type property - the column attribute in EF core is as follows :
Column Attribute: [Column (string name, Properties:[Order = int],[TypeName = string])
ex
[Column("DoB", Order = 1, TypeName="DateTime2")]
<the suggested steps to add it :
a-  open  Craftsman\Models\EntityProperty.cs file, append the following property:
    public string ColumnType { get; set; }

b-  open Craftsman\Builders\EntityBuilder.cs file and replace the following:
     if (!string.IsNullOrEmpty(entityProperty.ColumnName))
        attributeString += @$"        [Column(""{entityProperty.ColumnName}"",TypeName=""{entityProperty.ColumnType}"")]{Environment.NewLine}";

    with the following:

     if (!string.IsNullOrEmpty(entityProperty.ColumnName))
        {
            if (!string.IsNullOrEmpty(entityProperty.ColumnType))
                attributeString += @$"        [Column(""{entityProperty.ColumnName}"",TypeName=""{entityProperty.ColumnType}"")]{Environment.NewLine}";
            else
                attributeString += @$"        [Column(""{entityProperty.ColumnName}"")]{Environment.NewLine}";
        }

Sounds reasonable. I'll put it in my backlog

Btw, if you're up for a PR it would be more than welcomed. Not sure how soon I'll get to this, but it should be a pretty low lift and good first ticket!

new property on EntityProperty
updating the attribute builder in the EntityBuilder class


This is more a question then a issues

Can I use this framework to talk to the sensor, in case the sensor is more then just a temperature reader my can open relay?

At this point you cannot do this with these samples which are mainly focused on telemetry data sent from devices up to Azure.
We are working on adding this capability to the samples and you should start seeing some of these in a couple weeks from now.
In the meantime, you can look into options such as Azure Mobile services, Azure Notification Hubs or event Service Bus Topics.
Stay tuned

I am making good progress on the Azure IoT Hub support (see the IoTHub branch) which is bringing the ability to send messages back to devices from Azure through Azure IoT Hub. I'll label this issue as a feature request and will close once the V2 is pushed into Master (should be real soon!)

IoTHub support is now full there. I have added a Cloud to Device Message from the backend down to the devices each time there is an alert triggered. You can check how it's don't in the latest website code. I will update the devices code as well, for now only the NodeJS desktop sample does something out of the incoming message.


tinyproxy: 1.10.0 -> 1.11.0

Motivation for this change
Upgrade to latest version.
Things done

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

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

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

Result of nixpkgs-review pr 122757 at 99979d20 run on aarch64-linux 1

1 package built successfully:

tinyproxy

1 suggestion:

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/tools/networking/tinyproxy/default.nix:33:5:
   |
33 |     license = licenses.gpl2;
   |     ^

@GrahamcOfBorg eval


mass listing followup - modal background

please apply the same style as other modals have. That means black color with low opacity overlay. It should cover the navbar as well.

i tried to fix this one but seems currently it's little difficult. maybe after we upgrade to nuxt 3 then we can use <Teleport> to fix this one.

i tried to fix this one but seems currently it's little difficult. maybe after we upgrade to nuxt 3 then we can use <Teleport> to fix this one. it can be a common fix for all the NeoModal

possible, thanks!

can continue this now

👋


Will there be a wechat group for discussion?

We have created a discord channel for discussion. Feel free to ask questions there: https://discord.gg/jsedWbwkff


Fix for #26

All of the sensors are actually sensor_class MEASUREMENT instead of TOTAL_INCREASING, except for intervalReadings

Sorry, won't merge as this will delete and disable long term statistics for many users.


SMD19 goal #1 met

What's this PR do?

Updates the list of SMD goals to indicate the first goal of $15,000 has been met.

Why are we doing this? How does it help us?

To encourage visitors to the landing page to donate.

How should this change be communicated to end users?
I'll tell Sarah
Are there any smells or added technical debt to note?
No
What are the relevant tickets?
No
Have you done the following, if applicable:
(optional: add explanation between parentheses)

[ ] Added automated tests? ( )
[ ] Tested manually on mobile? ( )
[ ] Checked for performance implications? ( )
[ ] Checked for security implications? ( )
[ ] Updated the documentation/wiki? ( )

TODOs / next steps:

[ ] your TODO here

Not that this really needs review, but LGTM!


docs: Add display text and style text sections 📚

@Nickiel12 thanks for the review! I made a few fixes. Do you want to resolve the conversations as you see fit?