Not compatible with Phaser v3

Hi,
I'm not sure if this is a bug or just a poor setup but no matter what I do I cannot get this imported. I have tried using both a plain script tag from HTML as well as an import using webpack but nothing seems to work. I am getting two errors, the first, from using
a simple html script tag
<script src="dist/phaser-navmesh.min.js"></script>
as well as a normal require or import from phaser-navmesh
import { PhaserNavmesh } from "phaser-navmesh";
const phaserNavmesh = require('phaser-navmesh');
all three of these methods of importing produce this error for me in Chrome and Firefox

Uncaught TypeError: Super expression must either be null or a function, not undefined

Perhaps a more telling error comes from trying to import from the raw es6 library, using
import PhaserNavmesh from "phaser-navmesh/src/library";
produces the following error

Uncaught TypeError: Class extends value undefined is not a constructor or null

debugging in chrome for this second error takes me to  line 14 of nav-mesh-plugin.js
class NavMeshPlugin extends Phaser.Plugin {
it seems that Phaser.Plugin is undefined and this is what is causing the error to be thrown, this leads me to believe that phaser-navmesh doesn't have access to Phaser for some reason but i'm not sure if this is the case.

Hey @cour64, which version of Phaser are you trying to use? This was written for v2, though I'm planning on updating it to work with v3 soon. I suspect you are using v3.

Hah realised that was probably the issue after I went to bed last night, sorry it was late! Thanks for adding the note to clarify.

No worries - thanks for doing all that detective work to diagnose the issue. I'm going to leave this issue open until v3 compatibility is in place.

Should be v2 and v3 compatible now - they are separate libraries within the repo. I'm still working on testing and documenting, but if you want to give the v3 nav mesh wrapper a go, you can follow along with:
packages/examples/main.js - sets up the scene plugin in the game's config
packages/examples/scenes/start.js - scene that uses the plugin for pathfinding


Frontend Hands On 14

ToDo App Using Redux
Drift Link : https://video.drift.com/v/abawmv64zfQSFHdDOhSzhgRczNuB4dmbBXatktO2cek/

Metric
Value

alert_status
:white_check_mark: OK

bugs
1

code_smells
16

reliability_rating
C (at least 1 Major Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


need siunitx or \pu support for drawing circuits

Can't use siunitx package for using SI units with circuitikz. Obsidian supports \pu command for physical units natively. But that doesn't work either.

For now, I am using this macro to get around the issue:
\def\u#1{\ua{#1}#1\end O\end\eend}
\def\ua#1#2O\end#3\eend{%
	\ifx\end#3%
	\end\ensuremath{\textrm{#1}}%
	\else\ensuremath{\textrm{#2}\Omega}%
	\fi%
}

How to use: \u{1 kO}. This should be rendered as: $\textrm{1 k}\Omega$

I wanted to draw a circuit and write on the labels of my resistors the ohm value, but there is no support for siunitx
\usepackage{circuitikz}
\usepackage{siunitx}
\begin{document}
\begin{circuitikz}\draw
    (0,0) to [battery] (0,3)
    (0,3) to [R=2\si{\ohm}, l_=$2\ohm$] (3,3)
    (3,3) to [R=3\si{\ohm}, l_=$R_2$] (3,0)
    (3,0) to [R=5\si{\ohm}, l_=$R_3$] (0,0)
;
\end{circuitikz}
\end{document}


Analytics (update):

Overview
Tool: <Explorer>
Feature: <Filters, Panels, >
Description: Engagement is currently classed by any user who opens a dataset and selects color by cell type. Change the current exploring, interaction, engagement thresholds for Explorer. Most specifically Explorer engagements will now be measured at a higher level than previous. We believe this change will allow us to identify specific actions the users undertake, align with our new understanding of what engagement means for users vs old explorer analytics
Color by category events
EXPLORER_COLORBY_CATEGORY_EXPAND

visitor clicked to color by a category and either (1) a category/categories are already expanded OR (2) visitor then expands a category

EXPLORER_COLORBY_CATEGORY_HIGHLIGHT_HISTOGRAM

visitor clicked to color by a category and either (1) a histogram/histograms are already highlighted OR (2) visitor then highlights one of the histograms

EXPLORER_COLORBY_CATEGORY_SHOW_LABELS

visitor clicked to color by a category and THEN turned on labels

EXPLORER_COLORBY_CATEGORY_CHANGE_EMBEDDING

visitor switched to a different embedding type and a category was colored by

Color by histogram events
EXPLORER_COLORBY_HISTOGRAM_EXPAND_CATEGORY

visitor clicked to color by a histogram and either (1) a category/categories are already expanded OR (2) visitor then expands a category

EXPLORER_COLORBY_HISTOGRAM_HIGHLIGHT

visitor clicked to color by a histogram and either (1) a histogram/histograms are already highlighted OR (2) visitor then highlights one of the histograms

EXPLORER_COLORBY_HISTOGRAM_CHANGE_EMBEDDING

visitor switched to a different embedding type and a histogram was colored by

Select using lasso
EXPLORER_LASSO

visitor uses lasso to select a portion of data

EXPLORER_LASSO_CHANGE_EMBEDDING

visitor switched to a different embedding type and a portion of data was selected using lasso

Add gene
rename EXPLORER_SUGGEST_MENU_ITEM_CLICKED event to EXPLORER_ADD_GENE

adjust so that visitor adds a gene either by (1) selecting from the drop down menu or (2) pressing enter in the quick gene search bar

rename EXPLORER_COLORBY_GENE_BUTTON_CLICKED  to EXPLORER_COLORBY_GENE

visitor colors by an individual gene whether the gene is part of a gene set or not

rename EXPLORER_MAXIMIZE_GENE_BUTTON_CLICKED  to EXPLORER_GENE_VIEW_DISTRIBUTION

visitor expands an individual gene to see the distribution whether the gene is part of a gene set or not

rename EXPLORER_ADD_GENE_AND_SELECT_HISTOGRAM  to EXPLORER_GENE_HISTOGRAM_HIGHLIGHT

visitor highlights a portion of the histogram for an individual gene whether the gene is part of a gene set or not
adjust EXPLORER_SELECT_HISTOGRAM event to only fire if a histogram in the left side panel is highlighted but not any histograms that have to do with genes

rename EXPLORER_DISPLAY_SCATTERPLOT  to EXPLORER_GENE_DISPLAY_PLOT

visitor clicks X and Y for an individual gene to display the plot whether the gene is part of a gene set or not

rename EXPLORER_GENE_INFO_BUTTON_CLICKED  to EXPLORER_VIEW_GENE_INFO

visitor clicks the info button for a gene

Add gene set
rename EXPLORER_SUBMIT_GENESET_BUTTON_CLICKED to EXPLORER_ADD_GENE_SET

visitor successfully creates a gene set

rename EXPLORER_COLOR_BY_ENTIRE_GENESET_BUTTON_CLICKED to EXPLORER_COLORBY_GENE_SET

visitor colors by an entire gene set

rename EXPLORER_GENESET_EXPAND_BUTTON_CLICKED  to EXPLORER_GENE_SET_VIEW_DISTRIBUTION

visitor expands the gene set heading to see the distribution of the entire gene set

EXPLORER_GENE_SET_HISTOGRAM_HIGHLIGHT

visitor highlights a portion of the histogram for the entire gene set
adjust EXPLORER_ADD_GENE_AND_SELECT_HISTOGRAM event to not fire if the histogram for an entire gene set is highlighted
adjust EXPLORER_SELECT_HISTOGRAM event to only fire if a histogram in the left side panel is highlighted but not any histograms that have to do with gene sets

Other
EXPLORER_CLIP

visitor clips all continuous values to percentile range

Events to remove
EXPLORER_ADD_GENE_AND_COLORBY
EXPLORER_ADD_GENE_AND_DISPLAY_SCATTERPLOT
Checklist

[ ] Once event updates are deployed to Staging ping @ainfeld to check analytics
[ ] Verify events are showing up in the Staging space in Plausible correctly
[ ] Once event updates are deployed to Prod ping @ainfeld
[ ] Confirm Plausible events are still showing up in the Prod space in Plausible correctly

People to cc
PM: @niknak33
DS: @ainfeld
Privacy policy: https://cellxgene.cziscience.com/privacy

Hey @ainfeld thanks for all the event descriptions! One clarifying question - is clipping a histogram the same action as highlighting a histogram? If not, I might need help knowing the difference 🙏 Thank you!

Hey @tihuan thanks for checking!! The EXPLORER_CLIP event has to do with the button highlighted in the photo below.

OH!!! I never knew that functionality existed LOL Makes sense now thank you so much 😆!


[SPARK-15916][SQL] JDBC filter push down should respect operator precedence

What changes were proposed in this pull request?
This PR fixes the problem that the precedence order is messed when pushing where-clause expression to JDBC layer.
Case 1:
For sql select * from table where (a or b) and c, the where-clause is wrongly converted to JDBC where-clause a or (b and c) after filter push down. The consequence is that JDBC may returns less or more rows than expected.
Case 2:
For sql select * from table where always_false_condition, the result table may not be empty if the JDBC RDD is partitioned using where-clause:
spark.read.jdbc(url, table, predicates = Array("partition 1 where clause", "partition 2 where clause"...)

How was this patch tested?
Unit test.
This PR also close #13640

Test build #60729 has started for PR 13743 at commit 2f1ada3.

Test build #60729 has finished for PR 13743 at commit 2f1ada3.

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/60729/
Test PASSed.

LGTM

Merging to master and branch-2.0.


Add Wiki access request procedure to contributor guildelines.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

It will help us expedite review of your Pull Request if you tag someone (e.g. @username) to look at it.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

CC: @Ardagan

@swegner website build fails for reasons unrelated to this change. Can you please take a look?

Yes, this is https://issues.apache.org/jira/browse/BEAM-5681. I have this on my plate and will investigate today.
In the meantime, no need to block website PR's on this. Since @Ardagan has LGTM'ed, I'll merge this.


[Enchancement] Ignore based on app - possible fix

Possible way to ignore copied items is based on App bundle would be to monitor Pasteboard for changes when Maccy loads (part of the code is based from https://github.com/Rawnly/Pasty) :
NotificationCenter.default.addObserver(self, selector: #selector(onPasteboardChanged), name: .NSPasteboardDidChange, object: nil)
Then use NSworkspace for frontmost application (focusable apps):
(after the at symbol there is zero-white space character so it would display correctly if decide to copy the code below)
@objc
func onPasteboardChanged(_ notification: Notification) {
    let runningApp = NSWorkspace.shared.frontmostApplication

    if runningApp?.bundleIdentifier == "com.agilebits.onepassword7" {
        NSLog("ignored 1password")
    } else {
        guard let pb = notification.object as? NSPasteboard else { return }
        guard let items = pb.pasteboardItems else { return }

        print(items.count)

        guard let item = items.first?.string(forType: .string) else { return }

    }

}

That is great. I was not aware it's possible to have a notification for pasteboard changes, so I had to use a timer and periodically check it. Because of the timer, checking frontmostApplication would not be reliable.
I think this can be done in 2 iterations:

Replace the timer with notification. This will already be useful.
Introduce a preference to customize ignored apps list.

Actually, as I'm looking through Pasty code they also use timer :(
https://github.com/Rawnly/Pasty/blob/master/Pasty/AppDelegate.swift#L24-L29

Correct, sorry if that wasn’t clear. I saw you used a timer. I figured you could attach it to the polling to see the count with out capturing the sensitive data (as pasty codedid). Since at The time of checking  it could validate the app that had focus you can create set of apps to ignore.
I tested it in the pasty code used above and it appear to work for ignoring apps.  Do you still see it worth trying?

One other thought is to set notification using  didactivateapplicationnotification and it’s partner for deactivate:
https://developer.apple.com/documentation/appkit/nsworkspace/1535049-didactivateapplicationnotification
This returns the active app as NSRunningApplication representation. Each time the active app changes Maccy could be notified what is the active app and there by disabling the timer if it matches certain app bundle. And when that app resigns(Deactives) as the active app the timer checking pasteboard turns back on.

Thanks for great suggestions! My only problem with all these approaches is that it might fail in some cases. For example, when something is copied from long-running background process (e.g. bash script). Getting currently active application doesn't guarantee that pastboard is changed by it. However, I guess it might be enough in 95% of cases.
I believe the way to go is to make timer much faster (e.g. 100ms instead of current 1s) and then check for NSWorkspace.shared.frontmostApplication. I'll also need to build a preference pane with application selector to customize the list.

I did some testing using didActiveApplicationNotfication. I couldn't get it to work reliably. I think it is best to use the NSWorkspace.shared.frontmostApplication
I would just add a note where you add apps  the user wants to ignore that it will only ignore the focused/active application and not background tasks/applications.


Does UMAP accept a probability matrix as input?

Thank you!

Not currently -- what sort of probability matrix is it? If you want to shortcut some of UMAPs computations you can just apply the embedding algorithms directly to a similarity matrix. It would involve reaching in to some of the lower level functions and you'll have to find sensible values for some of the parameters, but it can certainly be done.

Hello, Mr. McIness.
Thank you for your promptly response!
That's exactly what I am trying to do. I am trying to use UMAP to embed data using my own matrix of transition probability. I'll take a closer look at the code.
Thanks.


Generate docs

Hello,
Are there instructions somewhere on how to generate readable docs versus the erb?  Like to read offline.
Regards.
Ron

We hope to remove the *.erb extension this week so they are nicely rendered by GitHub and local editors. In the meantime, there is a pull request which documents how they are built for bosh.io/docs if you want to try running it locally.

Well that is serendipitous!  Will keep a look out for that.


TTN Console might need different default for Router when selecting Australia frequency plan?

This is a bug report for the console.

What do you want to do?

Add a new gateway in Australia.

What steps did you take?

Select frequency plan "Australia 915MHz". According to the Server Addresses page on the Wiki, this apparently needs router.au.thethings.network in the gateway configuration, which (today) is an alias for thethings.meshed.com.au.

What went wrong or what is missing?

The Router dropdown defaults to ttn-router-asia-se (asia-se.thethings.network:1901). However, it seems meshed-router (thethings.meshed.com.au:1901) is a better choice?
(Also, "pick a router that is in a region which is close to the location of the router itself" could use some different wording, as "router" seems to be used for two different things here?)

Do you have Screenshots?

Done.


Publishing v0.4.x

@RolfBremer
First up: this package is great! Thank you for making and maintaining this.
Could you please upload version v0.4.x to the typst universe? I am already relying on the display parameter and currently forced to use a local copy since the latest version on the universe is v0.3.0.
Thanks! :)

OK, version 0.4.2 is submitted to typst universe. Waiting for approval.

Ok, it is published!


Add/self vhf only

Allow function to only compute self part of the VHF.

I think this could be implemented in a better way, but would require edits to compute_rdf_t in MDTraj directly.  For the sake of time, I may merge this as tests are currently passing. Then raise an issue to clean this up at a further time.


Update FAQ.md

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/Alluxio-Pull-Request-Builder/3905/
Test PASSed.

alluxio-bot, merge this please


Packaging ogg & vorbis with conan

Hello,
I've been using ogg and vorbis for a while. Recently, I have packaged them with conan, an open source C/C++ package manager and placed the packages on bintray:
https://bintray.com/dimi309/conan-packages/ogg%3Adimi309
https://bintray.com/dimi309/conan-packages/vorbis%3Adimi309
I hope you don't mind :)
The packages are in the process of being evaluated for inclusion into a curated repository on bintray, which is to only contain good quality conan packages. I am writing to ask you if you would be interested in maintaining the packages directly. I can provide you with the packaging code (actually, you can just copy it, since it's open source) or even introduce a pull request with some non-intrusive changes that would allow you to package them directly from the main ogg and vorbis repositories.

I don't speak for Xiph, but I am involved in maintaining a Xiph library (FLAC).

I hope you don't mind :)

As long as you are abiding by the license, I really don't think anyone at Xiph would mind. Quite the contrary. This is open source software.

I am writing to ask you if you would be interested in maintaining the packages directly.

Almost certainly not. Bintray serves a similar purpose to Linux distributions. Xiph is not involved with packaging Xiph libraries for the different Linux distributions. Similarly, Xiph is probably not interested in packaging Xiph libraries for Bintray.

I understand. Thank you very much for writing back with this information.
I have read through the license and I will always make sure I abide to it. Nevertheless, since mistakes happen, I will just go ahead and mention that if (now or ever) you or anyone else sees something wrong while having a look at the two above mentioned packages, I would be more than willing to make any required modifications.
One last question that I have, in case someone knows: Would it be ok if I add the xiph logo to the packages? (not to misrepresent them in any way, just to make them more recognisable)
Please feel free to close this request whenever you think that would be appropriate.

For whoever is interested, I have moved the vorbis and ogg conan packages to bincrafters, since it's better to work in a team:
http://bintray.com/bincrafters/public-conan/ogg%3Abincrafters
http://bintray.com/bincrafters/public-conan/vorbis%3Abincrafters


ImportError: /tmp/_MEIg8WrG2/libcrypto.so.3: version `OPENSSL_3.3.0' not found

Running on arch linux, downloaded release and after adding sunshine credentials, received this error:
> ./lutristosunshine
Error executing command: Traceback (most recent call last):
  File "/usr/bin/lutris", line 61, in <module>
    from lutris.gui.application import Application  # pylint: disable=no-name-in-module
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/lutris/gui/application.py", line 37, in <module>
    from lutris.api import get_runners, parse_installer_url
  File "/usr/lib/python3.12/site-packages/lutris/api.py", line 20, in <module>
    from lutris.util import cache_single, http, system
  File "/usr/lib/python3.12/site-packages/lutris/util/http.py", line 6, in <module>
    import ssl
  File "/usr/lib/python3.12/ssl.py", line 100, in <module>
    import _ssl             # if we can't import it, let the error propagate
    ^^^^^^^^^^^
ImportError: /tmp/_MEImEh0j8/libcrypto.so.3: version `OPENSSL_3.3.0' not found (required by /usr/lib/python3.12/lib-dynload/_ssl.cpython-312-x86_64-linux-gnu.so)

No games found in Lutris, Heroic, or Bottles.

Not entirely sure how to go about fixing this....

Can you test this and see if it works?
https://github.com/Arbitrate3280/LutrisToSunshine/actions/runs/11818704339/artifacts/2182139912

https://github.com/Arbitrate3280/LutrisToSunshine/actions/runs/11830255043/artifacts/2185641306
give this one a try

If this is still an issue let me know

Hey, sorry for the delay!

this artifact will expire soon, but I've updated the release binary to the new one:
https://github.com/Arbitrate3280/LutrisToSunshine/releases/download/2.0.1/lutristosunshine

Yep! This one seemed to work! Thank you!


Introduce constants for 'magic strings' in ManageController.

Forgive me if this is an insignificant number of changes for a single pull-request. Lot of firsts for me here: contributing to an open source project, using git and github, so I just wanted to make sure I get the workflow down before I start making bigger changes.
Please let me know if I should come back with a different pull-request when there is more to review.
#340

I :heart: small PRs

This is a great start, but I think you could take string abstraction a little further in ManageConrroller. There are a few more strings that could use constants.

Okay, something like creating a constant for every string, similar to EMAIL_CONFIRMATION_SUBJECT ?
Or something more robust for things like Controller names? Keep in mind nameof() won't work with Controller classes as MVC expects the "Controller" part of the name to be removed when being passed around.
Otherwise, I can list all the strings throughout ManageController as consts fields.

I was thinking a constant for each string

Sure thing! Thanks for the feedback and I'll get back to implementing it now.

@mheggeseth each string has been constant-fied. Please let me know what you think.

Once @mheggeseth does a review I can merge it in.  thx

@mheggeseth literal quotes added as per your feedback.

Looks good :shipit:

@mheggeseth @tonysurma @BillWagner
Why are we changing strings to constants in files that only use the constant once?
IMO, it's easier on the eyes to "read a human readable string" then a BUNCH_OF_CONSTANT_NAMES in a file.. I can see where a constant comes in handy when it's used more than once in a given file, but if it's used ONLY once, what's the benefit?
I understood Issue #340 to be related to getting rid of magic strings for things like using the new nameof operator, that would give us compile time checks against items being renamed in the system instead of relying on runtime failures. To me, we get value out of a change like that.
Unless I'm missing something

@mgmccarthy I don't feel strongly about it. Personally, I probably wouldn't have made constants for the single-use strings. Having to jump to another place in the file to read the string value is a little annoying but having a constant for every string serves as a good reminder to consider refactoring strings. It also makes it easier to do so when a single-use string becomes multi-use.


Remove nested rst formatting from contributing

The formatting in the bottom sections of the 'Contributing' page on readthedocs is broken, resulting in the display of formatting characters and full urls. The contents should instead mirror the github contributing page. The problems mostly stem from the fact that (unlike markdown) rst formatting cannot be nested, so you can't have something that's both italic and a link.
This PR is an example in one possible way to restructure things such that the contents appear more in line with the github based 'contributing' page, plus a couple of minor typo fixes.

This is great! Thanks, @bluemoo


address VisData does not contain firstFrameTime, defaulting to 0 error log

Description
when I call goToTime from the binding simulator I get an error VisData does not contain firstFrameTime, defaulting to 0
Expected Behavior
no error log. I'm not sure when/how firstFrameTime gets set or why it's not set in this case, but there is a comment with this function that indicates this was not a finished feature
Reproduction
in the education app, change the input concentration using the slider
Environment
Any additional information about your environment. E.g. OS version, python version, browser etc.

here's the comment:
// Instead of requesting from the backend the time passed into this
// function, we request (time - firstFrameTime) because the backend
// currently assumes the first frame of every trajectory is at time 0.
//
// TODO: Long term, we should decide on a better way to deal with this
// assumption: remove assumption from backend, perform this normalization
// in simulariumio, or something else? One way might be to require making
// firstFrameTime a part of TrajectoryFileInfo.


Toc - Table of Contents

https://github.com/rootzoll/raspiblitz/issues/2942
add script to build toc to base
add built tocs to markdown files

Looks good in:
https://github.com/nyxnor/raspiblitz/blob/toc/README.md
https://github.com/nyxnor/raspiblitz/blob/toc/FAQ.md
https://github.com/nyxnor/raspiblitz/blob/toc/FAQ.cl.md


Update async to ~1.2

Hi,
I want to install Kue in my project but i used async in version 1.2.1. But, Kue needs async in version 0.9.0 and i have an error on install (if I downgrade my async version, it works fine). It's possible to migrate Kue to async 1.2 ?
Thanks

My mistakes, the error provides from kue-scheduler : https://github.com/lykmapipo/kue-scheduler


Gradle publish plugin shouldn't trigger realization of all tasks

Expected Behavior
Applying the com.gradle.plugin-publish should only configure the minimum amount of work.
Current Behavior
Applying the plugin triggers the realization of all tasks, killing all laziness and preventing a legit build from running.
Context
I figured out that the culprit is at:
        at com.gradle.publish.PublishTask.useAutomatedPublishing(PublishTask.java:319)
        at com.gradle.publish.PublishPlugin$4.execute(PublishPlugin.java:223)
        at com.gradle.publish.PublishPlugin$4.execute(PublishPlugin.java:219)

by adding the following code snippet to my build script:
tasks.withType(DefaultTask).configureEach {
    if (it.name == 'wrapper') {
        throw new RuntimeException()
    }
}

This affects me because it triggers configuration of unnecessary tasks in a composite, and the build fails because of missing toolchains that it shouldn't need because the tasks should never be realized.
Steps to Reproduce
Use the following script:
plugins {
    id 'java-gradle-plugin'
    id 'maven-publish'
    id 'com.gradle.plugin-publish' version '0.15.0'
}

tasks.withType(DefaultTask).configureEach {
   if (it.name == 'wrapper') {
      throw new RuntimeException("oh noes, why did you get configured?!")
   }
}

Run ./gradlew help and see the build fail. Comment out the maven-publish plugin, run again and see the build pass.
Your Environment
The build scan below shows the problem on a real world build
Build scan URL: https://scans.gradle.com/s/nnklofn2aw2gc/performance/configuration?details=code-unit-com.gradle.plugin-publish!:native-gradle-plugin&openScriptsAndPlugins=WyJjb2RlLXVuaXQtY29tLmdyYWRsZS5wbHVnaW4tcHVibGlzaCJd

This will be fixed in the next version of the plugin-publish plugin although the chain of tasks for generating Gradle Module Metadata will still be realised. This will only apply when the plugin is used in combination with the maven-publish plugin, although that is now the recommended way.

Version 0.16.0 of the plugin-publish has some improvements in that area.
It will no longer causes eager realization of all tasks but still creates a number eagerly itself.


[MRG]: BUG: fix MPI child communication, add several improvements w/ unit testing

This should address the issues with #190, finally (I've tested on a rebased branch with code from #190). The key fix was simple: 2e425c53ad60fe223ede18050a52d047c2202447. When the system is slow (e.g. Travis) is might accumulate more than 4k in the stderr buffer and only the first 4k would get read.
During the search for this bug, I made many improvements that should make life easier for everyone going forward. The changes to the MPI child communication scheme have a bunch of unit tests. The largest change for discussion is the testing scheme code for parallel backends... @jasmainak @rythorpe @ntolley
It drove me mad trying to fix Travis issues when the time to complete tests and print errors is ~10 minutes. While we are still validating the MPIBackend and adding new simuatlion outputs, I think having the full simulation is useful. So I updated test_parallel_backends.py to make my life easier, and it's included in this PR.

There is a custom pytest marker incremental that is defined in conftest.py and declared in pytest.ini. It allows a failing test (e.g. a reduced mpi simulation) to cancel subsequent tests (e.g. the full lengthl mpi and joblib comparison test). This was an example from pytest's documentation and it doesn't require any additional modules.
If mpi4py isn't installed, then MPI tests are skipped, but that does cause subsequent tests to be skipped (as long as they don't require mpi4py). This seems to be the appropriate solution to the issue @ntolley ran into when testing locally without mpi4py.
The test test_mpi_nprocs is pretty important to make a prerequisite for later mpi tests. If there is truly only one processor available, then mpi tests aren't useful tests (and the user should be aware).
Each test in test_parallel_backends.py performs a specific purpose. Code for running simulations is made as modular as possible. I find this easier to follow than the previous test_compare_hnn that ran reduced and full-length simulations all together.

Ugh, I need to redo this yet again.

okay, let us know when ready!

I took a brief look and it looks good from a superficial look. I'd like to set aside some time and mess around with the code / review it more thoroughly though. Might push a commit or two. Just to absorb what's going on. This is turning out to be quite the saga. Thanks for your efforts @blakecaldwell

I took a brief look and it looks good from a superficial look. I'd like to set aside some time and mess around with the code / review it more thoroughly though. Might push a commit or two. Just to absorb what's going on. This is turning out to be quite the saga. Thanks for your efforts @blakecaldwell

Sure, would recommend starting with my review comments to see the important changes. Also, the addition of conftest.py.

Apologies if I've been a bit absent lately @blakecaldwell. I'll try to get caught up with this PR this weekend.

Looks very promising! Code gets better and better with each PR.
So, I did not get through the entire PR. I still need to do test_mpi_backends.py. Please interpret all the flurry of comments as my excitement to see interesting progress, specially with the testing procedure!

Thanks for reviewing the code. I'm in a very busy period for the next two weeks, so I probably won't be able to implement significant code changes in the meantime. Also if I don't reply to comments for a bit, that's why :)

One last question, wondering how all this compares to Popen.communicate? If there are system library alternatives, we should try to use them.

I used Popen.communicate() before #185. That summarizes one reason. Additionally, the issues seen recently with interpreting data/error messages cannot be resolved with Popen.communicate() because it blocks until the process completes.

Hi all, starting my update to this PR today.

@jasmainak @ntolley I think that I've addressed all comments above and this PR is ready once it goes green. Thanks for reviewing... hope the code is reasonable to follow now.

Codecov Report

Merging #198 (e0220fd) into master (737739e) will decrease coverage by 1.11%.
The diff coverage is 68.91%.

@@            Coverage Diff             @@
##           master     #198      +/-   ##
==========================================
- Coverage   68.06%   66.95%   -1.12%
==========================================
  Files          20       21       +1
  Lines        2123     2106      -17
==========================================
- Hits         1445     1410      -35
- Misses        678      696      +18

Impacted Files
Coverage Δ

hnn_core/mpi_child.py
0.00% <0.00%> (ø)

hnn_core/parallel_backends.py
16.97% <7.14%> (-2.37%)
:arrow_down:

hnn_core/tests/conftest.py
55.55% <55.55%> (ø)

hnn_core/tests/test_mpi_child.py
100.00% <100.00%> (ø)

hnn_core/tests/test_parallel_backends.py
100.00% <100.00%> (+3.94%)
:arrow_up:

hnn_core/tests/test_cell.py
96.87% <0.00%> (-3.13%)
:arrow_down:

hnn_core/basket.py
100.00% <0.00%> (ø)

hnn_core/pyramidal.py
100.00% <0.00%> (+1.67%)
:arrow_up:

hnn_core/cell.py
77.08% <0.00%> (+7.85%)
:arrow_up:

Continue to review full report at Codecov.

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

PR is good enough to merge. Let's move on and make improvements in future PRs, I realize your time is limited @blakecaldwell . Thanks for the efforts so far!

@blakecaldwell is the "notes for contributors" section in this document still up-to-date: https://github.com/jonescompneurolab/hnn-core/blob/master/doc/parallel.rst ? A quick PR to update it would be very much appreciated!


Additional Administrator email(s) field not working

Using the same as the example<EMAIL_ADDRESS><EMAIL_ADDRESS>is marked as invalid

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

ID: 6f795f00-1229-2646-c18e-948b493647b4
Version Independent ID: 09a2b3c1-1d3e-4df5-d2c1-2efb76f1413f
Content: Use the Azure portal to create classic alerts for Azure services
Content Source: articles/monitoring-and-diagnostics/insights-alerts-portal.md
Service: azure-monitor
GitHub Login: @rboucher
Microsoft Alias: robb

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

@titobf thanks for bringing this to our attention. the issue here is that there's an empty space after the semicolon. It works as expected if the space is removed. Kicked off a PR to remove the empty space in the example. Thanks for helping make our docs better. closing this as resolved.
FYI  - @rboucher


jsDoc: Bug detecting periods in description

This function description is being flagged erroneously. The preceding line does indeed end in a period. Perhaps it's getting tripped up by the HTML?
jsDoc: You started a new line with an upper case letter but previous line does not end with a period at main.js :
539 |     *
540 |     * Any parameter value without a name is assigned the boolean value true.
541 |     * The first such parameter, going from left-to-right, is also assigned to a field named "_".
------------^
542 |     *
543 |     * @private

This happens when separating paragraphs using
 as well. The suggestion in another issue to strip out HTML would solve it.
/**
 * Does this thing.
 *

 * Also does something else.
 */

/cc @zxqfox

@zxqfox I would love to work on this bug.
Can could guide me on how to take this one?

@kepta It's a rule requireDescriptionCompleteSentence
https://github.com/jscs-dev/jscs-jsdoc/issues?utf8=✓&q=is%3Aissue+requireDescriptionCompleteSentence
The problem here that this rule based on regular expressions. I know this is not the best thing here but don't see any better solution atm.
Would be great if you'll tackle this. I'm even agree if you'll find some good library for that. I'm really confused atm how this rule should work internally.

Btw, we have another rules that close to this problem:

http://jscs.info/rule/requireCapitalizedComments
https://github.com/jscs-dev/jscs-spellcheck

Probably we just should move out this rule to separate package with independent release cycle and spec.


Create multiple agent stream

Create a graph that contains multiple agents communicating with each other

I think a good start is to draw inspiration from the crypto agent example in the agent workshop from A*

Currently when accesing a tool, it goes into an infinite loop. Maybe something in the way the graph is constructed?


Authorization Support

Is your feature request related to a problem? Please describe.
Need to implement field level authorization
Describe the solution you'd like
The solution should allow user's of this library to customize the logic.

Already implemented by @CNAChino.


compress-Images not getting installed

node version: 8.10.0

@khushbu-maurya
Try install compress-images 1.7.4

Or maybe node 10

@khushbu-maurya
You do not can install - npm install gifsicle --save

@khushbu-maurya
Try install compress-images 1.7.4

still same issue

@khushbu-maurya

Remove compress-images

Try install compress-images 1.7.6

You get error from gifsicle but not from compress-images

This working for me:
If you run for install:
npm i -D gifsicle
npm i -D compress-images
npm i -D pngquant-bin
If you run for solve posible errors:
npm audit fix
npm install
package.json
...
"compress-images": "^1.7.7",
"gifsicle": "^5.0.0",
"pngquant-bin": "^5.0.2"
...

There's an issue with gifsicle as it's 5.0.0 version doesn't contain bin for Linux :(
https://github.com/imagemin/gifsicle-bin/issues/113

I am here because I can't install it in docker linux.
It worked well for me on local with OSX but production with linux in docker

error /app/node_modules/jpeg-recompress-bin: Command failed.
Exit code: 1
Command: node lib/install.js
Arguments:
Directory: /app/node_modules/jpeg-recompress-bin
Output:
⚠ spawn /app/node_modules/jpeg-recompress-bin/vendor/jpeg-recompress ENOENT
⚠ jpeg-recompress pre-build test failed

Anyone faced this issue?


lang: Make stack frames slimmer on ATA creation

Initializing many associated token accounts in the same instruction can result in too large stack frames at compile time:
    #[account(
        init,
        payer = payer,
        associated_token::authority = system_program,
        associated_token::mint = mint,
    )]

Error: Function _ZN137_$LT$pda_derivation..AssociatedTokenResolution$u20$as$u20$anchor_lang..Accounts$LT$pda_derivation..AssociatedTokenResolutionBumps$GT$$GT$12try_accounts17hddc314f570fb4fb5E Stack offset of 7696 exceeded max offset of 4096 by 3600 bytes, please minimize large stack variables

The problem has been fixed similar to #2913 by inlining local variables created while generating associated token account creation code.

@acheroncrypto When will a new Anchor version be released? This fix is quite important for me.

Not sure, but we might need to make a patch release depending on the resolution of https://github.com/coral-xyz/anchor/issues/3044.
Do you publish your crates? If not, you can just depend on the git version until the next release.

@acheroncrypto When approximately will a new Anchor version be released?
Depending on git versions for dependencies can sometimes be seen as an unsafe habit. I see there is very active development on main branch, so I expect it to take some time.

@acheroncrypto When approximately will a new Anchor version be released?

~3 weeks

Depending on git versions for dependencies can sometimes be seen as an unsafe habit. I see there is very active development on master branch, so I expect it to take some time.

This PR was merged slightly after the last release, so there won't be many changes if you depend on this PR's commit hash 5727865.


nan while doing inference in C++ with half precision

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
No
Source
source
Tensorflow Version
2.11
Custom Code
Yes
OS Platform and Distribution
Debian GNU/Linux 10
Mobile device
No response
Python version
3.7
Bazel version
6.1.0
GCC/Compiler version
LLVM 12
CUDA/cuDNN version
11.6
GPU model and memory
Tesla T4
Current Behaviour?
I've trained a model in python using mixed precision. However, when loading the model in C++ and trying to perform inference, The output is all `nan`.

I've tested loading the model in python and it works.

Some ideas might be:

- Toolchain is different.
- Tensors in C++ are not zero-initialized.

Standalone code to reproduce the issue
Python

class Model(tf.keras.Model):
  def __init__(self):
    self.conv = tf.keras.layers.Conv2D(
        128,
        activation='relu')

  def call(self, x):
    return self.conv(x)

x = tf.convert_to_tensor(np.random.random(1, 8, 8))
model = Model()
model(x)
model.save('/tmp/model')

C++
using namespace ::tensorflow;

const static std::vector<std::string> kInputNames = {
    "serving_default_args_0:0",
};

const static std::vector<std::string> kOutputNames = {
    "StatefulPartitionedCall:0",
};

int main(int argc, char** argv) {
  SavedModelBundleLite model_;
  LoadSavedModel(SessionOptions(), RunOptions(), "/tmp/model",
                 {"serve"}, &model_bundle_);

  Tensor input_features(DataType::DT_FLOAT,
                        {1, 8, 8});
  std::vector<Tensor> cast_output;
  Status status =
      session.Run({ops::Cast(scope, input_features, DataType::DT_HALF),
                   ops::Cast(scope, input_state, DataType::DT_HALF)},
                  &cast_output);
  std::vector<std::pair<std::string, Tensor>>{
      std::make_pair(kInputNames[0], cast_output[0]),
  };

  std::vector<Tensor> output_buf = {
      Tensor(DataType::DT_HALF, {1, 6, 6}),
  };

  // this is nan
  model.GetSession()->Run(inputs, output_names, {}, output_buf);
}

### Relevant log output

```shell
2023-03-14 05:08:59.641765: I cc/scripts/play_model.cc:94] -nan
2023-03-14 05:08:59.641818: I cc/scripts/play_model.cc:95] -nan
2023-03-14 05:08:59.655568: I cc/scripts/play_model.cc:119] ------- Model Stats -------
2023-03-14 05:08:59.655609: I cc/scripts/play_model.cc:120] Top Move: Loc(0, 0)
2023-03-14 05:08:59.655621: I cc/scripts/play_model.cc:121] Win: -nan Loss: -nan
2023-03-14 05:08:59.655677: I cc/scripts/play_model.cc:123] -----Board-----
0  ○ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
1  ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
2  ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
3  ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅
4  ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
5  ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
6  ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
7  ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
8  ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
9  ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅
10 ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
11 ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
12 ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
13 ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
14 ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
15 ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅ ⋅ ⋅ + ⋅ ⋅ ⋅
16 ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
17 ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
18 ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅ ⋅
   A B C D E F G H I J K L M N O P Q R S

@p3achyjr  Thanks for reporting this issue.
Looks like inputs are not being provided to the model's session.  Also, the minimal reproducible code for the issue is incomplete.
Can you please provide the steps you have followed and also did you try setting up the dtype policy to mixed_float16.
Thanks.

Thanks for getting back--the dtype policy is set to mixed_float16 in python. Not sure how to do that in c++.
I also ended up solving this as it turns out my tensors were not zero-initialized. I wonder if there's a way to do that automatically?
Also let me know if there is a better way to do half precision in C++, as right now I need to construct and run a graph to cast my tensors from float -> half -> back to float every time I want to call into my model.

Hi, Could you please check the ZerosLike function https://www.tensorflow.org/api_docs/cc/class/tensorflow/ops/zeros-like which returns tensor of zeros.
For half precision in C++, to the best of knowledge I can think of Requantize OP https://www.tensorflow.org/api_docs/cc/class/tensorflow/ops/requantize, check if that helps you. Thanks!

using namespace tensorflow;

const static std::vector<std::string> kInputNames = {
    "serving_default_args_0:0",
};

const static std::vector<std::string> kOutputNames = {
    "StatefulPartitionedCall:0",
};

int main(int argc, char** argv) {
  SavedModelBundleLite model_bundle_;
  LoadSavedModel(SessionOptions(), RunOptions(), "/tmp/model",
                 {"serve"}, &model_bundle_);

  Tensor input_features(DataType::DT_HALF,
                        {1, 8, 8});
  input_features.flat_half().setZero(); // Initialize to zero
  std::vector<std::pair<std::string, Tensor>> inputs{
      std::make_pair(kInputNames[0], input_features),
  };

  std::vector<Tensor> output_buf = {
      Tensor(DataType::DT_FLOAT, {1, 6, 6}),
  };

  // Run inference with mixed precision
  Status status = model_bundle_.GetSession()->Run(
      {ops::Cast(scope, input_features, DataType::DT_HALF)},
      {kOutputNames[0]}, {}, &output_buf);

  if (!status.ok()) {
    LOG(ERROR) << "Inference failed: " << status;
    return -1;
  }

  // Print the output tensor values for debugging
  auto output_flat = output_buf[0].flat<float>();
  for (int i = 0; i < output_flat.size(); i++) {
    LOG(INFO) << "Output[" << i << "] = " << output_flat(i);
  }

  return 0;
}

Note that the code now initializes the input_features tensor values to zero using the setZero() method on the flat_half() accessor. The input tensor is also now created with DataType::DT_HALF, and the output tensor is created with DataType::DT_FLOAT to match the output of the Python model.
During inference, the code now uses mixed precision by casting the input tensor to half-precision using ops::Cast(scope, input_features, DataType::DT_HALF). The output tensor is also created as a std::vector so that the Run() method can fill in the output values.
After running inference, the code prints out the values of the output tensor for debugging purposes.

Thanks all! I think we can close this :)


Second version-like option

I would like to have a second version-like argument (--git) that just prints a hash and quits
myprog
    Lorem ipsum.

Usage:
    myprog [options] <file>

Argument:
    <file>      Some file.

Options:
    -h, --help
        Show help.

    --version
        Show version.

    --git
        Show git commit hash as the time of installation.

How can I do this?
Currently I cannot even do this manually without adding a usage case:
myprog [options]

(which I don't want)

What is the problem with handling it as
if args['--git']:     print(hash)     sys.exit(0)
AFAIK only --version and --help are mandatory according to GNU standards

The real question is: what good reason do you have for not wanting --git to show up in usage?
--git isn't some almost universal, well-established convention, like --help and --version are.
If I ever use your program, in almost any context or circumstance, I will not even suspect that it takes a --git option.
So then for example if I ever file a bug report, if I knew it took a --git option to print a precise version like a git hash I would proactively put that output in the bug report without being asked because that's obviously useful, but if we waste a round-trip of human interaction before the bug gets looked into because I didn't know about --git so you have to ask me "what's the output of foo --git?" then I will be rather annoyed.
For that matter I don't think it's such a good idea to omit `--help` and `--version` from help text.I think the `Usage` section definitely *should* explicitly document the `--help` and `--version` options, and the `Options` section should document their `-h` and `-V` (that's a capital V) short aliases. Part of how people learn conventions is by actually seeing it documented repeatedly. New computer users first learning the shell often don't know `--version` is a big convention for months or even years after they know to try `--help`. Plus, we should be looking forward to a future of tooling that parses docopt help strings! And exactly how many conventions should that tooling assume are implemented? There's such a simple elegance in just leaving nothing implicit, across the whole stack. And all it costs you is one usage line `foo (--help | --version)` and a couple options lines.

Oh I guess you're saying you just don't want it to show up in usage, rather than in the help text at all. Okay, next point:
When I look at
myprog
    Lorem ipsum.

Usage:
    myprog [options] <file>

Argument:
    <file>      Some file.

Options:
    -h, --help
        Show help.

    --version
        Show version.

    --git
        Show git commit hash as the time of installation.

You know what I see? I see that tooling would have no way to tell if myprog --git some-file is a good or bad invocation.


Post examples

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?
Add posts example from RTK Query guide
[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Need more examples from RTK Query guide
Issue Number: #2
What is the new behavior?
Closes #2
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

:tada: This PR is included in version 1.4.0-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support Sqlite file locations

It doesn't seem to be possible (at least not in a straightforward way) to locate an Sqlite database in the user's document directory. Specifying an absolute path results in error 14 unable to open file (and this is not due to a lack of permissions).
Being able to store the file in arbitrary locations is important to apps where the file is being used as a document format or where it is otherwise desired to allow the user to move, copy or backup the file, or even just because the user wants to be able to see it in with their other files.
I know the file should be on a local drive but I think we can do better than having it hidden away in a directory that most users can't find.
All the best,
sjb

@sjb-sjb absolute paths are already be supported.  If you are still having issues with this in the 1.0.0 release, please open an issue on https://github.com/aspnet/Microsoft.Data.Sqlite along with your error message and repro steps.

Are you referring to 7.0.0-rc1-final? Because in that version, you can specify an absolute path but it has to be a path to the app dir.


[Bug]: IBrowserType does not have ConnectAsync method

Playwright version
1.15.4
Operating system
Windows
What browsers are you seeing the problem on?
No response
Other information
No response
What happened? / Describe the bug
Guys, in the BrowserType class you have a method for connecting to an existing browser instance:
public async Task<IBrowser> ConnectAsync
but I can't use it, cause the only thing I can get is the IBrowserType interface using:
            using var playwright = await Playwright.CreateAsync();
            var chromium = playwright.Chromium;

And the IBrowserType interface doesn't have this method. Is it a bug?
Code snippet to reproduce your bug
No response
Relevant log output
No response

Folding into https://github.com/microsoft/playwright-dotnet/issues/986


Dart binary on windows should set it's process name to include the entry point?

When we get reports of dart taking up too much memory, it would be nice to know which dart:

https://github.com/flutter/flutter/issues/33862#issuecomment-507174340
Maybe there are other, better ways than changing the process name?  FYI @devoncarew

I can recommend Process Explorer where you can get a lot more details about each process (e.g. command line parameters):
https://docs.microsoft.com/en-us/sysinternals/downloads/process-explorer

I ran into this again today while doing some local server development.   :/  It seems we do support this on Mac at least, there I see "dart:flutter.snapshot" as the name in Activity Monitor, which is helpful.  FYI @kevmoo @clarkezone

To be clear, I'm not at all sure this is possible on windows.  Other things I found while investigating:
https://stackoverflow.com/questions/23783985/set-child-process-name-in-windows
https://github.com/dvarrazzo/py-setproctitle

One thing we may do here is ship a dart bug command - a tool to help gather some SDK setup and diagnostic info; that would likely include stats on Dart process (though may not include process IDs). Here's some potential output:

General info

Dart 2.19.0-edge.29c3fea3bf3aee14c22e628748fafbd2c33cc792 (be) (2022-11-12) on "macos_arm64"
on macos / Version 13.0.1 (Build 22A400)
locale is en-US

Project info

sdk constraint: '>=2.17.0 <3.0.0'
dependencies: cloud_firestore, collection, firebase_core, fl_chart, flutter, provider, pub_semver, url_launcher, url_strategy, yaml
dev_dependencies: flutter_lints, flutter_test

Process info

Memory
CPU
Elapsed time
Command line

75 MB
0.0%
00:04
dart devtools --machine --try-ports 10 --allow-embedding

792 MB
0.0%
17:52:09
dart language-server --protocol=lsp --client-id=VS-Code --client-version=3.52.1

588 MB
207.9%
00:04
dart language-server --protocol=lsp --client-id=VS-Code --client-version=3.52.1

A tool that could tell me what the dart processes were on my machine would be awesome (even if I couldn't used the OS ones, although that OS ones would be preferred where possible).  Maybe someone has already written a dartps package or similar, that does sound possible to do...

Windows executables can have a file section for application resources and that contains the version information for the program, including its display name and info. This information is used by Task Manager and the Properties panel when displaying information on a executable file.
I compiled @timsneath tetris example as a win32 app with dart compile exe and added a resource section with the following version info.
1 VERSIONINFO
FILEVERSION 1,0,0,0
PRODUCTVERSION 1,0,0,0
FILEOS 0x40004
FILETYPE 0x1
{
BLOCK "StringFileInfo"
{
    BLOCK "000004B0"
    {
        VALUE "FileVersion", "<IP_ADDRESS>"
        VALUE "ProductVersion", "<IP_ADDRESS>"
        VALUE "OriginalFilename", "tetris.exe"
        VALUE "InternalName", "tetris.exe"
        VALUE "FileDescription", "My Awesome Tetris App"
        VALUE "CompanyName", " "
        VALUE "LegalCopyright", "Copyright (c) 2022"
        VALUE "ProductName", " "
    }
}

BLOCK "VarFileInfo"
{
    VALUE "Translation", 0x0000 0x04B0
}
}

And Task Manager shows this:

You can use the the BeginUpdateResource/UpdateResource/EndUpdateResource APIs to make these changes. The data used in UpdateResource is the compiled script from the above. Microsoft Resource Compiler takes the script and outputs the data with a little header. The format of the raw resource data is spec'd but I forget where.
Ref:

https://learn.microsoft.com/en-us/windows/win32/menurc/adding-deleting-and-replacing-resources
https://learn.microsoft.com/en-us/windows/win32/menurc/using-resources#updating-resources
https://learn.microsoft.com/en-us/windows/win32/menurc/resource-compiler

Additionally from my understanding there is no runtime way of altering the displayed titles, as, Task Manager opens and memory maps in the executable files into memory to process them and read their resources.
However there is a trick 🤔 If your application has a window it will show its title. You might be able to create an off-screen window or something. Quite hacky 😢


Add automated interpretability metrics

Add automated interpretability metrics (using either a large local model, or e.g. GPT) to the validation stage of the pipeline.

@alan-cooney can I work on this issue? or any other issue you need help with.

@StatKumar that would be great! Btw feel free to slack me if you have any questions on the open source mech interp group
As a quick starting thing - I think it'll want to extend AbstractValidationMetric and you may also want to update that abstract class a bit

Thank you @alan-cooney

@StatKumar are you working on this. I've got some previous code from https://github.com/hoagyc/sparse_coding that I'd like to clean up and add as a metric.

Thinking about this a bit further:

It shouldn't be a validation metric - it costs quite significant money to run (I think ~5p per feature interp at current prices, haven't checked recently) so you don't want to run on unfinished runs often.
There's nothing that's specifically sparse autoencoder-related about doing automatic interpretability - all you need to do is define some function of the activations and evaluate it across a big range of inputs. I think having an easy-to-use version is good but might not make sense here. It might make more sense in TransformerLens?
The way I did this before depended on https://github.com/openai/automated-interpretability which is not a package, just a normal git repo which might not be in line with something that wants to be a mature package? Maybe if doing this it would go into a separate part of the poe dependencies, like [dev, demos]?


Bogus transition object returned for not-found routes in Router.run

Given the following routes:
    <Route>
      <Route path='/found'/>
    </Route>

Calling Router.run(routes, '/missing' (error, initialState, transition) => {...}) provides a surprising set of arguments to the callback.
error is undefined, which maybe makes sense.
initialState is null, makes sense.
transition is this object: Transition{isCancelled: false, redirectInfo: null, abortReason: null}
IMO transition should be null as well, since there's no transition happening here. This makes it harder to distinguish redirects from not-found (initialState == null && transition.redirectInfo == null vs. just initialState == null, or checking for error).

SSR rendering is still baking, this is what it will look like when done, hopefully a lot less bogus than this :P
https://github.com/rackt/react-router/blob/master/docs/ServerRendering.md


Only update timestamps on set

The timestamp on "get" should not be the current time, but the last time it was set

only fixed for getValue of single data, not for multiple data. Fix for multiple data will be done in #158


Error launching package on Raspberry Pi

Hello,
I'm trying to use the package in a Raspberry Pi 3B+ with Raspbian OS and ROS noetic. When I launched the package I got the following messages:
[ WARN] [1550139515.070712783]: Parameter 'broker/tls/enabled' not set, defaulting to '0'
[ WARN] [1550139515.084676755]: Parameter 'client/id' not set, defaulting to ''
[ WARN] [1550139515.085581642]: Client buffer can not be enabled when client ID is empty
[ WARN] [1550139515.092249230]: Parameter 'client/clean_session' not set, defaulting to '1'
[ WARN] [1550139515.097297460]: Parameter 'client/keep_alive_interval' not set, defaulting to
'60.000000'
[ WARN] [1550139515.102260794]: Parameter 'client/max_inflight' not set, defaulting to '65535'
/opt/ros/noetic/lib/nodelet/nodelet: symbol lookup error: /opt/ros/noetic/lib//libmqtt_client.so: undefined
symbol: _ZNSt10filesystem7__cxx114path14_M_split_cmptsEv
[mqtt_client-2] process has died [pid 2174, exit code 127, cmd /opt/ros/noetic/lib/nodelet/nodelet
standalone mqtt_client/MqttClient __name:=mqtt_client __log:=/home/pi/.ros/log/dcd419a6-3041-11e9-
93a2-0013252229ad/mqtt_client-2.log].
log file: /home/pi/.ros/log/dcd419a6-3041-11e9-93a2-0013252229ad/mqtt_client-2*.log
I believe the problem is during the compilation a flag/symbol was not set properly, but I don't know how I can fix it.

Have you built the package yourself or have you installed it via a package manager?

I did it by myself. Since the Raspberry Pi has a limited computing capacity. I installed the package on the Raspberry Pi in the following steps:
1- rosinstall_generator to generate the list of the package dependencies.
2- wstool to fetch all the remote repos specified from the dependencies.
3- catkin_make_isolated

The error message undefined symbol: _ZNSt10filesystem7__cxx114path14_M_split_cmptsEv indicates that the issue is related to std::filesystem that is used in the implementation.
As far as I am aware, std::filesystem is only available starting with C++17. Note that the CMakeLists.txt is configured to build the mqtt_client with C++17. However, the cxx114 in the error message sounds like perhaps you only have C++14 installed.
Have you tried building from source the way it is described in the README (rosdep, catkin build)?
Alternatively, have you tried installing it from the package manager, i.e., sudo apt-get install ros-noetic-mqtt-client.

Hello, thank you for your reply. Based on what you wrote, I tried different approaches. First, I installed the package from the package manager, but I encountered the same error as when I installed it from the source. Then, I focused on the C++ version. I realized that my OS (Debian Buster) has GCC 8 by default, and Ubuntu 20, where I could install the package without any issues, has GCC 9. I attempted to upgrade the compiler, but my system got corrupted in the process.
Next, I tried to force GCC 8 to use C++17 standards. Finally, I found that, in addition to "add_compile_options(-std=c++17)," I also needed to add "link_libraries("$<$<AND:$<CXX_COMPILER_ID:GNU>,$<VERSION_LESS:$<CXX_COMPILER_VERSION>,9.0>>:-lstdc++fs>")" in the CMakeLists.txt file. After making these changes, I compiled again from the source using catkin_make_isolated, and finally, I could launch the package without any errors.

Hello, thank you for your reply. Based on what you wrote, I tried different approaches. First, I installed the package from the package manager, but I encountered the same error as when I installed it from the source. Then, I focused on the C++ version. I realized that my OS (Debian Buster) has GCC 8 by default, and Ubuntu 20, where I could install the package without any issues, has GCC 9. I attempted to upgrade the compiler, but my system got corrupted in the process.
Next, I tried to force GCC 8 to use C++17 standards. Finally, I found that, in addition to "add_compile_options(-std=c++17)," I also needed to add "link_libraries("$<$<AND:$<CXX_COMPILER_ID:GNU>,$<VERSION_LESS:$<CXX_COMPILER_VERSION>,9.0>>:-lstdc++fs>")" in the CMakeLists.txt file. After making these changes, I compiled again from the source using catkin_make_isolated, and finally, I could launch the package without any errors.

Good to hear! Feel free for posting a pull request to fix the issue!


Roll Flutter Engine from 3964cf62cdf8 to 175f4b1bc49d (1 revision)

https://github.com/flutter/engine/compare/3964cf62cdf8...175f4b1bc49d
2022-07-25<EMAIL_ADDRESS>Roll Skia from f39d68a2adcc to fff7fb2ae1fb (1 revision) (flutter/engine#34892)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Update ed25519-dalek to allow rust-analyzer to work properly

From #3, this is currently blocked as-is.
I'm converting that PR into an issue to track this problem, as ed25519-dalek has gone two years without a release, with some improvements on master branch that should be in a normal release.
Original issue, which is now closed as fixed, but not in a release yet: https://github.com/dalek-cryptography/ed25519-dalek/issues/173
Issue on their repo calling for a new release: https://github.com/dalek-cryptography/ed25519-dalek/issues/192

Alternatively, the following could be added to .vscode/settings.json to fix this for everyone using vscode;
  "rust-analyzer.cargo.unsetTest": [
    "core",
    "ed25519-dalek"
  ],

This is now presumably fixed since we do depend on a new release of ed25519-dalek as of: 11e81cda6473392aba1e4435c7f9b814cc31bdcc.


feat: added cloud-event support

@davidB  i added the cloudevent here! let me know what you think

Look into the "Files change" view of the PR, you'll notice useless change or not clean revert. Also I push a big reorganisation of the project layout that could impact you (on the rebase,...), eg:
* remove of justfile (replaced by taskfile)

* remove of the cargo workspace, no cdviz-collector is a standalone cargo project (I suggest that you import cdviz-collector as a root project into your IDE, editor)

Thanks for your contribution

Thank you for the reply. Sorry,  i use a custom linter in my IDE i will remove it, close this pr and make a new simple a short one.


rocketchat-server fails to start after install

With 0.50.1 , the rocketchat-server does not restart.  Instead, the following causes it to stop the service:
Feb 03 01:26:53 vps105588 snap[16633]: /snap/rocketchat-server/x1/node_modules/fibers/future.js:313
Feb 03 01:26:53 vps105588 snap[16633]:                                                 throw(ex);
Feb 03 01:26:53 vps105588 snap[16633]:                                                 ^
Feb 03 01:26:53 vps105588 snap[16633]: MongoError: no primary found in replicaset
Feb 03 01:26:53 vps105588 snap[16633]:     at Object.Future.wait (/snap/rocketchat-server/x1/node_modules/fibers/future.js:
Feb 03 01:26:53 vps105588 snap[16633]:     at new MongoConnection (meteor://💻app/packages/mongo/mongo_driver.js:219:27)
Feb 03 01:26:53 vps105588 snap[16633]:     at OplogHandle._startTailing (meteor://💻app/packages/mongo/oplog_tailing.js:200:
Feb 03 01:26:53 vps105588 snap[16633]:     at new OplogHandle (meteor://💻app/packages/mongo/oplog_tailing.js:76:8)
Feb 03 01:26:53 vps105588 snap[16633]:     at new MongoConnection (meteor://💻app/packages/mongo/mongo_driver.js:222:25)
Feb 03 01:26:53 vps105588 snap[16633]:     at new MongoInternals.RemoteCollectionDriver (meteor://💻app/packages/mongo/remot
Feb 03 01:26:53 vps105588 snap[16633]:     at Object.<anonymous> (meteor://💻app/packages/mongo/remote_collection_driver.js:
Feb 03 01:26:53 vps105588 snap[16633]:     at Object.defaultRemoteCollectionDriver (packages/underscore.js:784:19)
Feb 03 01:26:53 vps105588 snap[16633]:     at new Mongo.Collection (meteor://💻app/packages/mongo/collection.js:103:40)
Feb 03 01:26:53 vps105588 snap[16633]:     at AccountsServer.AccountsCommon (meteor://💻app/packages/accounts-base/accounts_
Feb 03 01:26:53 vps105588 snap[16633]:     - - - - -
Feb 03 01:26:53 vps105588 snap[16633]:     at /snap/rocketchat-server/x1/programs/server/npm/node_modules/meteor/npm-mongo/
Feb 03 01:26:53 vps105588 snap[16633]:     at null.<anonymous> (/snap/rocketchat-server/x1/programs/server/npm/node_modules
Feb 03 01:26:53 vps105588 snap[16633]:     at null.g (events.js:260:16)
Feb 03 01:26:53 vps105588 snap[16633]:     at emitOne (events.js:77:13)
Feb 03 01:26:53 vps105588 snap[16633]:     at null.emit (events.js:169:7)
Feb 03 01:26:53 vps105588 snap[16633]:     at /snap/rocketchat-server/x1/programs/server/npm/node_modules/meteor/npm-mongo/
Feb 03 01:26:53 vps105588 snap[16633]:     at /snap/rocketchat-server/x1/programs/server/npm/node_modules/meteor/npm-mongo/
Feb 03 01:26:53 vps105588 snap[16633]:     at nextTickCallbackWith0Args (node.js:436:9)
Feb 03 01:26:53 vps105588 snap[16633]:     at process._tickCallback (node.js:365:13)
Feb 03 01:26:53 vps105588 systemd[1]: snap.rocketchat-server.rocketchat-server.service: Main process exited, code=exited, s
Feb 03 01:26:53 vps105588 systemd[1]: snap.rocketchat-server.rocketchat-server.service: Unit entered failed state.
Feb 03 01:26:53 vps105588 systemd[1]: snap.rocketchat-server.rocketchat-server.service: Failed with result 'exit-code'.
Feb 03 01:26:53 vps105588 systemd[1]: snap.rocketchat-server.rocketchat-server.service: Service hold-off time over, schedul
Feb 03 01:26:53 vps105588 systemd[1]: Stopped Service for snap application rocketchat-server.rocketchat-server.
Feb 03 01:26:53 vps105588 systemd[1]: snap.rocketchat-server.rocketchat-server.service: Start request repeated too quickly.
Feb 03 01:26:53 vps105588 systemd[1]: Failed to start Service for snap application rocketchat-server.rocketchat-server.

Mongo will not be ready for at least 40 seconds but RC no longer retires.

A  lot of work has been on-going to solve this problem behind the scene.  Thanks totally to @elespike
He has been in communications with Ubuntu's snapcraft team and has made a PR that will make our startup attempt more resilient to RC's seasonal code changes (sometimes we fail fast on first error, sometimes we re-try forever. sometimes ....).
https://bugs.launchpad.net/snapcraft/+bug/1662240
https://github.com/snapcore/snapcraft/pull/1109
We will wait for snapcraft's 2.27 release (where his PR will become part of distro) and subsequent testing before closing this issue.


[5.5][CodeCompletion] Don't suggest underscored module name

Cherry-pick #37527 into release/5.5 originally reviewed by @benlangmuir
rdar://78232229

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - c76484bcd24b3c7bc93f7a3610e07f00bbf99dce

@swift-ci Please test macOS


Crash with my setup

Grab my branch : https://github.com/MathieuDuponchelle/slate/tree/crash
bundle exec middleman server
Go to <IP_ADDRESS>:4567
The crash should be reproduced.
I have bisected it to that faulty commit :
bb24528f851fdcb7db7016baa3a376926817dbf1 "Update gems" (of course :)
I don't know ruby and it seems the crash is there, if you can't reproduce I'll be happy to follow instructions for providing a relevant stack trace

Yup, the new version of RedCarpet has a segfault. Thanks for bisecting the faulty commit! The dev branch has a fix that should be merged into master soon. If you just bundle install in that branch, I think your problem should be fixed? Let me know if that works for you.

(Closing because I think it's a known bug, but LMK if dev doesn't work for you and we can reopen!)

Yop no problem glad to know it was fixed, I'll try later and check if that fixed my issue, thanks :)


[RFC] Better displaying of errors.

Closing, since another PR (see #29) passes the ISpoofaxResult<?> to the client to provide errors in however way the client needs.


(OraklNode) Add conditions to skip deviation job

Description
minor update to skip job on certain situations
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Deployment

[ ] Should publish npm package
[ ] Should publish Docker image

@coderabbitai review


Feature/263

https://github.com/OAGr/foretold/issues/263

Please fix the conflicts and I'll merge.


[R] There is any way to install with pre downloaded sources

Hi,
I can't use any installation who need a internet connection. My only solution is to download my file before in an other computer. Is it possible to download sources and install it after in an other commputer ?
In advance thank you

@louvetg this is currently not supported in spark-install, it assumes internet connection is available. Keeping this open as a feature request.


remove unused assert option from cmake, docs

As per title, this is no longer used now that the size is passed in to polygonToCellsExperimental.

coverage: 98.784%. remained the same
when pulling 6f51a8ef6f9afc33accb5be07e7b7dd8b1eef3de on isaacbrodsky:remove-unused-assert-option
into e05a9028406210c9f777638032cb6c4f939908f3 on uber:master.


Kotlin demo support

Code needed by https://github.com/pusher/chatkit-android/pull/15

Codecov Report

Merging #28 into master will increase coverage by 0.38%.
The diff coverage is 36.36%.

@@             Coverage Diff              @@
##             master      #28      +/-   ##
============================================
+ Coverage     12.86%   13.24%   +0.38%
- Complexity       28       30       +2
============================================
  Files            24       24
  Lines           863      868       +5
  Branches         88       90       +2
============================================
+ Hits            111      115       +4
- Misses          749      750       +1
  Partials          3        3

Impacted Files
Coverage Δ
Complexity Δ

.../src/main/kotlin/com/pusher/platform/BaseClient.kt
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...m/pusher/platform/subscription/BaseSubscription.kt
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ary-core/src/main/kotlin/com/pusher/util/Result.kt
42.85% <66.66%> (+2.85%)
6 <2> (+2)
:arrow_up:

Continue to review full report at Codecov.

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


New starttls implemenation

Coverage remained the same at 96.212% when pulling efd1ff0c86dc4559f049dbf333dd7173d63466c1 on starttls into d6fef333a3e51ed440c07db9b8fe48d18313ee5d on master.


New version release plan?

It's been almost a year since the last release, so I wonder when the new version will be released. Is there any plan for release?

This library is very solid and generally a rather complete implementation of RFC 4122.
What features or improvements would you be looking for in a new release?

Thank you for the response!
When I use this package with my Next.js project, the warning message like Anonymous function declarations cause Fast Refresh to not preserve local component state. is printed out. So I think it would be nice that PR #545 is released.

Thanks for pointing this out again. Looks like we should make a new release soon then.
@broofa do you think we should craft a patched 8.x release that contains the specific bug fix or would you go for 9.0 directly (as node 8 support was dropped)?
Maybe for 9.0 we could create a list of things we want to include (like dropping IE 11 support), to not have too many major version bumps?

Let's roll a 9.0 release.  Enough stuff has changed.  In addition to already-committed work, let's include the following:

[ ] #597
[ ] #590
[ ] #469

Maybe you could pull the types directly into this repo? https://www.npmjs.com/package/@types/uuid

Maybe you could pull the types directly into this repo? https://www.npmjs.com/package/@types/uuid

According to the typescript documentation this is still not recommended for packages that don’t auto-generate the typescript declarations: https://www.typescriptlang.org/docs/handbook/declaration-files/publishing.html
I would prefer following the official typescript guidance and keep the types separate.

This is usually meant for those who add types and does not maintain them. For consumer this is burden to add one more package with possibly outdated types (this is the feeling I always have about any @types/* package).
For this project I don't see an issue in improving DX.

This is usually meant for those who add types and does not maintain them. For consumer this is burden to add one more package with possibly outdated types (this is the feeling I always have about any @types/* package).
For this project I don't see an issue in improving DX.

I.e. you would prefer to have the types shipped directly with the uuid package?
My main concern is that I'm personally not a TypeScript expert so I don't feel comfortable maintaining/supporting the types. I think we would at least need a soft commitment from some TypeScript-aware contributors (you, @TrySound ? And maybe @LinusU ?) to provide support, if we wanted to ship them with the npm package.

I've created the Version 9 project to track tasks for this upcoming release.
Re: TS types, I'm in favor of that, but as @ctavan says we need someone on the team to own that.  While I've been doing more work in the TS space of late, and could probably whip something together, I'm not confident that it would be well enough baked to not require ongoing maintenance, which I don't currently have the bandwidth for.

I'm open to owning the TS types, but I think that before we commit to it we should see how it integrates with our somewhat complex situation with shipping multiple different versions (ESM/browser/CJS) of the library.
It seems like TS supports placing a "types" key in the "exports" map, next to e.g. "node" and "default", but we are also using main, module, and browser. This might be a non-issue but we should probably just check how the setup would look ☺
It would also be amazing to have some kind of tests that they are in sync, but I'm not sure how that would look 🤔
One approach is to generate the typings using --allowJS and TSDoc comments. If that's feasible with our setup I think it would be the best approach, I can take a look at prototyping this!

btw. for the next major version I think we should investigate just shipping ESM. I'm thinking that the shipped package just have node.js and browser.js and just uses the exports field in package.json. All supported Node.js versions already have support, we just need to make sure that that works with rollup/webpack/etc ☺

If you’re dropping IE 11 it might be worth dropping old node versions too. Node 10 is no longer supported by node itself, and 12 is also quite old now.

If you’re dropping IE 11 it might be worth dropping old node versions too. Node 10 is no longer supported by node itself, and 12 is also quite old now.

So far we have always been supporting Node LTS versions plus one older version. I think we should keep up with this given the abundant use of this library in the Node.js world. Except for ESM, I don't see anything in this library that is incompatible with Node.js 10.x, so there's also not a big benefit of dropping Node.js 10.x support (except for maybe ESM, but I feel like this is a larger discussion).

If you’re dropping IE 11 it might be worth dropping old node versions too. Node 10 is no longer supported by node itself, and 12 is also quite old now.

So far we have always been supporting Node LTS versions plus one older version. I think we should keep up with this given the abundant use of this library in the Node.js world. Except for ESM, I don't see anything in this library that is incompatible with Node.js 10.x, so there's also not a big benefit of dropping Node.js 10.x support (except for maybe ESM, but I feel like this is a larger discussion).

Active LTS is v16, maintenance LTS is v14, if you went one version older it would be v12. So by that rule v10 would still be be dropped. I’m not saying you need to drop it, but it’s usually unhealthy having to support so many versions at the same time.

According to https://nodejs.org/en/about/releases/ v12 is still in Maintenance until May 2022. With "old" I intended to say "officially unsupported". That currently applies to v10.

Good question.  9.0.0-beta.0 has been published, and there's no open issues remaining on the v9 project board.
@ctavan: Are we just in "let it simmer for a while" mode at this point?  Thoughts on when we should make it official?

I was wondering if we should try to get the TS types packed with uuid before we ship v9? However I think for this we'd have to rely on @LinusU's help. On the other hand, adding them would also not be a breaking change, so we can still ship that with 9.1.
I was also hoping to see the jest@29 release before releasing uuid@9, it's supposed to come very soon.
So let's say that we're O(days) away from v9 and not O(weeks).

<EMAIL_ADDRESS>has been released, see changelog.
It supports jest@29 out of the box again but we don't ship TypeScript types yet.

Sorry, missed the comment 25 days ago, I can submit a PR to add the types to this repo. It should be a non-breaking change so I think it could go out as 9.1?

Sounds good to me.

Started work on typings here: #654

Hey Guys, this may be a little late (because I've just stumbled on the discussion), but as a committed TS hater I'd hate to see TS types packaged as part of the main app. If you really want them, can't you just release a different package with them?
For every TS developer, there are 3 JS developers who hate that language. Because of their instance on adding their types to our libraries, my colleagues and I now have to suffer larger libraries than I would otherwise. I know you're going to say something like, "but it's only 1 or two files," and it is, but it's an extra file or two for every one of the thousands of libs that are imported in a typical non-trivial project. Files that 3/4 of us don't want and won't use.
If the TS guys want this, and you're happy to provide it, please, provide it in a separate package, like some other libraries do.

For every TS developer, there are 3 JS developers who hate that language.

Very interesting, where did you get these statistics?
It was my understanding that the benefits certainly outweighs the very small drawback of shipping an extra file. E.g. even if you don't use TypeScript, having a typings file will provide better autocomplete in your editor...

It was my understanding that the benefits certainly outweighs the very small drawback of shipping an extra file. E.g. even if you don't use TypeScript, having a typings file will provide better autocomplete in your editor...

In a personal perspective, I don't like Typescript because ads itself as a EcmaScript extension while it isn't, and it's implementation has some bugs left for historical reasons, but types are something really useful, that's why I'm desiring so much that the proposal of optional typings annotations as comments moves forward, defining types with JsDoc is painfully verbose.

If the TS guys want this, and you're happy to provide it, please, provide it in a separate package, like some other libraries do.

So you'll be terribly inconvenienced by your IDE knowing the types of your dependency even when you yourself use Javascript.
If you need a super small bundle you can just use webpack/rollup - then no types get shipped with your application. Most packages ship a ton of useless data that's simply much much much larger than any typescript definitions file.
It's really quite silly what you're saying.

I understand where you're coming from, @BryanDollery.  5 years ago, I despised TS.  I saw it as yet another embrace-extend-extinguish effort by M$FT.  But I've since become a convert.  So much so that I've been leading the effort to port CodePen's codebase to TS.   I like to think this gives me a pretty good basis for having opinions on this matter.

can't you just release a different package with them?

That is how it currently works.  @types/uuid has been a thing for a few years now.  But there are a couple problems with that:  1) It's an additional installation step, with additional clutter in a projects package.json#dependencies, and 2) they're maintained separately by people unaffiliated with this project.
That last point is really the biggest issue.  Having two separate projects defining the same API invariably causes problems.
Basically the @types project is an enormous band-aid to help with TS adoption, but it's not an ideal solution by any means.

suffer larger libraries than I would otherwise

How does that translate to actual pain for you, though? Yes, node_modules/uuid is grows slightly, but the node_modules directory is largely opaque to developers, by design.
You're going to have to do a better job articulating how the presence of type declaration files actually inhibits your ability to use uuid.  At present, I'm just not seeing it.
(FWIW, the most compelling argument against TS that I've been able to come up with is that, in an open source project, it reduces the number of developers who are comfortable reviewing and writing code.  But that only applies if the actual codebase is written in TS.  That's not what is being proposed here, however.... yet.  😈 )

For every TS developer, there are 3 JS developers who hate that language.

Very interesting, where did you get these statistics?
It was my understanding that the benefits certainly outweighs the very small drawback of shipping an extra file. E.g. even if you don't use TypeScript, having a typings file will provide better autocomplete in your editor...

StackOverflow developer survey 2022

I understand where you're coming from, @BryanDollery. 5 years ago, I despised TS. I saw it as yet another embrace-extend-extinguish effort by M$FT. But I've since become a convert. So much so that I've been leading the effort to port CodePen's codebase to TS. I like to think this gives me a pretty good basis for having opinions on this matter.

can't you just release a different package with them?

That is how it currently works. @types/uuid has been a thing for a few years now. But there are a couple problems with that: 1) It's an additional installation step, with additional clutter in a projects package.json#dependencies, and 2) it's maintained separately by people unaffiliated with this project.
That last point is really the biggest issue. Having two separate projects defining the same API invariably causes problems. Basically the @types project is an enormous band-aid to help with TS adoption, but it's not an ideal solution by any means.

suffer larger libraries than I would otherwise

How does that translate to actual pain for you, though? Yes, node_modules/uuid is grows slightly, but the node_modules directory is largely opaque to developers, by design.

It's not that opaque when the size of your container images are growing by the day. It's not just uuid that's causing a problem with this solution -- it's a pandemic :) It is in aggregation that we see the problem emerging. Everyone is doing this.
My build is slowed down too because there are now extra files and dependencies I have to scan. Again, individually they might be harmless, but with this trend a large majority of us could be spending more on storage and compute than we need to at a time when we should all be doing our best to reduce such things so as to promote economic prudence and environmental sustainability.

You're going to have to do a better job articulating how the presence of type declaration files actually inhibits your ability to use uuid. At present, I'm just not seeing it.
(FWIW, the most compelling argument against TS that I've been able to come up with is that, in an open source project, it reduces the number of developers who are comfortable reviewing and writing code. But that only applies if the actual codebase is written in TS. That's not what is being proposed here, however.... yet. 😈 )

Fortunately, it doesn't stop me from using uuid (presumably).
I get that it might add some typing support for an ide, but half the time I just want to make a quick edit and open the file in vim, which I'm not going to configure as a full ide just to get that benefit.
I don't want to go into depth about why I hate TS. I have significant experience (years) with that language and know it well enough to have developed my beliefs. And, it's not important. What is important is that if this isn't a typescript project, then it doesn't need the extra baggage.
For this project though, where is the benefit? uuid produces strings, not complex classes. I've checked out the TS bindings, and they're are mind-numbing. I know that uuid does more than just produce a string, btw, but I honestly believe that 99% of people using uuid do so just to generate a unique id as a string. So, not only are you producing bindings for 1/4 of developers, you're producing them for the 1% of those guys who need to do something more complicated than simply gen a unique string id. Not worth the effort of alienating potential contributors for. Not worth the effort of further bloating my container images and slowing down my processes for.
Look, if it saves you some hassle then yeah, go ahead. I'm already grateful that you're doing this. It has saved me hours of work and I'm happy and I wouldn't want to suggest that you sign up for more effort than absolutely necessary. But, if the external lib already exists and someone else is maintaining it, then why would you go through the effort at all? The existing solution works fine for most of us already.

Belated apologies for the off-topic discussion.  I'm going to close this out since we've pushed uuid@9 out months ago.
If anyone wants to continue the TS-bloat conversation, please open a discussion for that and reference the conversation here.


[Clean] Eliminate registrations moved upstream.

Remove registrations moved into ember core via https://github.com/emberjs/ember.js/pull/13459
Cleanup of patches moved upstream into ember core will be done on the clean branch so that master can continue to accept patches and work in a backward compatible way as much as possible. Once all the patches are moved upstream, we'll merge clean to master and bump the minor version.

Left one nit-pick/tweak (non-blocking), but looks good to me!

Thanks @rwjblue!


add attributes to join frame, sign and support

@3nids I don't get why it is not finding siro_od.frame

the tables were not created in correct order, fixed in 38b9b6f

nice, thanks


[SCB-617] Graceful shutdown with standalone tomcat

Coverage increased (+0.03%) to 87.552% when pulling 8c7a3188384b72729ac198b823fac064f25d5a81 on wujimin:graceful-shutdown-with-standalone-tomcat into 6ce5280969d1949fff9cb26c6925a1774a409af8 on apache:master.


Custom background for the playground

Hi, thanks for your work on Docz, I'm giving it a try and it's fantastic! 🎉
How would one go about changing the background of the preview in the <Playground> for only one component ? It doesn't look like it's possible at the time of writing.
Description
I know you can create a custom theme for the entire Docz site, but in our case it's only a specific subset of components that don't work well on a white background, with the vast majority looking great.
It would be great if we could just pass in a CSS colour as a property of the playground that would only affect that specific instance.
Thanks,
Alvin

Hi @alvinsight
We did run into a similar problem. For now we do:
<Playground>
  <div style={{ backgroundColor: "#eeeeee", margin: '-2em', padding: '2em' }}>
    <Button handleClick={ () => alert('Working') } >
      Accept
    </Button>
  </div>
</Playground>

I did recheck and the width is the problem. This works great:
<Playground style={{ backgroundColor: "#eee", margin: '-2em', padding: '2em', width: 'auto' }}>
  <Button handleClick={ () => alert('Working') } >
    Accept
  </Button>
</Playground>

Note that there's a media-query which sets the the padding to 1.5em if the viewport is smaller than 420px.

What about having a toggle for some kind of black and white theme in this situations? In some cases you can set what should be the default, but in other you can allow user to select it. @pedronauck what do you think?

@thebergamo I liked it, Do you send to roadmap as a feature?
http://feedback.docz.site/
Thanks man

Will do it. Maybe I can open a PR as well sooner.

@thebergamo nice man 🚀

Fixed on release v0.12.10 ✅

style doesn't seem to work on <Playground> anymore?

style doesn't seem to work on <Playground> anymore, and I can't wrap playground in a <div>. Seems the only way to style is within playground, which means people hitting the clipboard btn will get the wrapping element/styles as well. :(

I can confirm this, my inline styles into Playground component isn't working anymore.

The <Playground> component use some colour codes from theme-ui config, as far as I can see for the background and border.
colors: {
    playground: {
        bg: 'white',
        border: 'rgb(220, 220, 220)',
    },
}

But this only affects the upper part of the component, the lower part with the live editor has it's bg style inside the component.

I would love to have a consistent way to change the look of the <Playground>. As it is now, the live editor background color doesn't look good with the rest of our theme and it will also be impossible to add a proper dark theme for our docs.


Remove IPC from WS implementation

It actually doesn't really help at all. One could manually listen for the change event and implement IPC that way.

Done https://github.com/fortunejs/fortune/commit/2326ec83f5fc1ad63bc1aa6451dbb80ed1e8118c


Mobile support for the official website

Seeing as this project is about best practices in Css, one could argue that having a poor mobile experience on the official website; would reflect poorly on the entire project.

I agreed, supporting mobile is one of top priorities!


Allow AdapterModels to have custom tokens

PEFT has a feature for adapters to add tokens to a model: https://github.com/huggingface/peft/blob/main/examples/causal_language_modeling/peft_lora_clm_with_additional_tokens.ipynb
When using an AdapterModel with new tokens in LightEval, the script fails because:

AdapterModel._create_auto_tokenizer always uses the base model path, without checking config.tokenizer
the new tokenizer and base model will have a mismatched number of embeddings

Notebook with error: https://colab.research.google.com/drive/1AMJ6_MiZGFTBf8KdRn-zj7soKyZrzpbf?usp=sharing
This PR would create the tokenizer from config.tokenizer or config.base_model
and run base.resize_token_embeddings(...) before PeftModel.from_pretrained(base, adapter_weights)
This is based on my fix for llm-evaluation-harness: https://github.com/EleutherAI/lm-evaluation-harness/pull/1828
Notes:

+1 please merge #298 (cherry-picked here), their one-line fix passed model_config_path to AdapterModel and this is necessary for most advanced models
opinion: I think that adapter_weights and delta_weights should be optional so I do not need delta_weights: false in my config

Hi!
That works for me, but I would add a more detailed hlog message at the resize step: if you're adding tokens, we need to know how many, since they are likely to behave weirdly, and if you're removing tokens from the vocab, we would want to know which ones.
So a log like "You're using the adapter model's tokenizer, which has less/more tokens than its base: adding/removing X tokens" + in case of removing specifying which, would be useful.
You'll also need to run ruff to fix the style.

Re your comment, adapter_weights and delta_weights should be skippable by default, what's the error message you're getting when not specifying them?

I rebased, reformatted, and created a smaller demo to trigger the adding/removing token messages https://colab.research.google.com/drive/1iZhdL-HTuStKGZ3ezspd53eDt_lhXbvm
Unfortunately I can't test due to a Torch/TorchVision compatibility issue. If I install and run lighteval on CoLab I get this error:
RuntimeError: Failed to import transformers.models.bert.modeling_bert because of the following error (look up to see its traceback):
operator torchvision::nms does not exist

After upgrading torch and torchvision, I get:
  File "/content/lighteval/src/lighteval/data.py", line 28, in <module>
    from torch.utils.data.distributed import DistributedSampler, T_co
ImportError: cannot import name 'T_co' from 'torch.utils.data.distributed' (/usr/local/lib/python3.10/dist-packages/torch/utils/data/distributed.py)

OK ✔ I changed how the dependencies are installed and was able to run lighteval on the larger and smaller tokenizers
Updated CoLab, same URL: https://colab.research.google.com/drive/1iZhdL-HTuStKGZ3ezspd53eDt_lhXbvm
At the end of the notebook, I have added the error if I leave out delta_weights: false from the config
File "/usr/local/lib/python3.10/dist-packages/lighteval/models/model_config.py", line 439, in create_model_config
    if config["merged_weights"]["delta_weights"]:
KeyError: 'delta_weights'

I added a commit to this PR to change the line to if config["merged_weights"].get("delta_weights", False): and the same for where it checks adapter_weights

Rebased and looks like that config["merged_weights"] issue moved to main_accelerate

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Set goals for next assignment not very visible

This button is not very visible and I almost missed it.
Suggestion:

Make the button orange (also grey is used to indicate 'completed')

Centre align the "Set your goals" button with others (as the user will naturally be drawn towards right)

needs a padding beneath the button

Give access to goal setting functionality from the reporting area - see the mock up below (but make the button orange)

Also see https://github.com/theodi/ODMAT/issues/224


It would be nice to have some sort of -o type option to tell kubernaut where to store your cluster config

gcloud, kops and maybe kubeadm respect the value of KUBECONFIG when retrieving a cluster config. Would you want that UX or something like --output=<path>?


[backmerge] hotfix 4.0.3

color customization readme
Payment recovery call for auth
loadings mejorados para vending
fixes destroy activity

Codecov Report

Merging #1255 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##             master   #1255      +/-   ##
===========================================
- Coverage      2.56%   2.56%   -0.01%
  Complexity      125     125
===========================================
  Files           371     371
  Lines         15108   15110       +2
  Branches       1303    1303
===========================================
  Hits            387     387
- Misses        14696   14698       +2
  Partials         25      25

Impacted Files
Coverage Δ
Complexity Δ

...paymentmethodsearch/PaymentMethodSearchOption.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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