Generate Github/Markdown documentation.

Currently this repository is set up to use Sphinx to build HTML documentation (or whichever other output format Sphinx provides). Considering this repo is hosted on Github, I think it follows naturally that we should be able to generate documentation in Markdown and publish the generated documentation here… 🤔
There are a few tools that generate Markdown from Python docstrings, with or without Sphinx (older Stackoverflow question):

NiklasRosenstein/pydoc-markdown
ml-tooling/lazydocs
clayrisser/sphinx-markdown-builder
mkdocs/mkdocs/

It would then make sense to integrate a “generate documentation” step into our release Action, and either find a way to publish the generated Markup directly into the repository or as a Github Wiki (which is its own repo), or use e.g. the uibcdf/action-sphinx-docs-to-gh-pages Action to publish to Github Pages.
Lastly, perhaps it would also make sense for this repository to opine/dictate a docstring style (e.g. based on Google’s Style Guide) and check docstrings using Sphinx’s Napoleon extension)?

Making it easy to get to the documentation, like auto-generate and push to the Github Wiki would be great imho.

See also the sphinx-notes/markdown-builder fork of the original sphinx-markdown-builder (background in issue https://github.com/clayrisser/sphinx-markdown-builder/issues/59).

PR https://github.com/jenstroeger/python-package-template/pull/224 is now open.

PR https://github.com/jenstroeger/python-package-template/pull/396, commit https://github.com/jenstroeger/python-package-template/commit/ef0a354185ee42a86308faf0bbe8aba24c2342ba.


Create an organization in GHES if the owner of the GitHub.com doesn't exist on the GHES instance

Right now, if an organization does not exist on the GHES appliance and we try to sync an action from GitHub.com to the GHES appliance, it will error out with a 404 Not Found since the user or organization does not exist. To remediate this, we can use the GHES API to create an organization with the exact same name as the owner of the GitHub.com repository. Even if the owner of the GitHub.com repository is a regular user and not an organization, we should still create it as an organization since its easier for individual users to create organizations via the API than other user accounts.

There isn't a public API to create an organization for GHES. DotCom has createEnterpriseOrganization. If there was a public API there might be some parameters we don't have good defaults for.
We won't be making this enhancement, but https://github.com/actions/actions-sync/pull/9 improves the error message when the organization doesn't exist.

I was mistaken, there is a public API for GHES: https://docs.github.com/en/enterprise/2.22/user/rest/reference/enterprise-admin#create-an-organization.


Reorganize ReadAtLocus and VariantRead

It seems difficult to count alleles other than the variant nucleotides using ReadAtLocus but we can't use VariantRead since by definition they only carry the variant allele.

Fixed by https://github.com/hammerlab/isovar/pull/31 (now have AlleleRead instead of VariantRead)


Disable cache refresh job on values and dev yaml files

Need to comment the value also in values.yaml since prod is still considering it as part of the deploy

I will merge this temporal change, later, will implement a better mechanism to enable/disable jobs


Add Typescript types

Hey, I need typescript types for my project. Would you accept pull request with .d.ts file?

Yes, of course

Hi, I migrated the whole library to typescript so no need to manually add .d.ts files.


proposed fix for issue #272. args were not correctly formatted.

Just changed the args formatting in start,sh
And also change a reference that was made in the development documentation (although it was correctly formatted, it could lead to mistakes)

Great! Thank you! :)


Unable to launch with sodium installed

Only the latest version is being supported. If you are using an outdated version, upgrade to the latest version.
Provide instructions for how to reproduce the issue. Include the Minecraft version and mod version.
Forge version issues belong to  https://github.com/qouteall/ImmersivePortalsModForForge/issues
If it's a mod compatibility issue, post that mod's link and open another issue on that mod
the game doesn't start
mod is CaffeineMC/sodium-fabric
apparently, it doesn't allow sodium
crash report:
Incompatible mod set!
net.fabricmc.loader.impl.FormattedException: Mod resolution encountered an incompatible mod set!
A potential solution has been determined:

Replace mod 'Immersive Portals' (immersive_portals) 1.2.5 with any version that is compatible with:

sodium 0.4.1+build.15
Unmet dependency listing:

Mod 'Immersive Portals' (immersive_portals) 1.2.5 is incompatible with any version before 0.4.0-alpha6 or any version after 0.4.0-alpha6 of mod 'Sodium' (sodium), but a matching version is present: 0.4.1+build.15!
at net.fabricmc.loader.impl.FabricLoaderImpl.load(FabricLoaderImpl.java:190)
at net.fabricmc.loader.impl.launch.knot.Knot.init(Knot.java:142)
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:71)
at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23)

use sodium 0.4.1

I am using the latest 1.18.1 version of sodium

It's recommended to use 1.18.2. In 1.18.1, the compatible sodium version is 0.4.0-alpha6


Rake task for unassigned or misassigned transcribers

Ultimately, all transcribers should be in a syndicate that reflects their transcribing. A rake task to identify those in:
Any Questions? Ask Us
Any County and Year
Technical
Wales Syndicate (?)
Yorkshire Syndicate (?)
And all old syndicates that are no longer in the dropdown list on Registration page

Task written by @AnneV-Learn and deployed to production.

task run and report sent off line to @AnneV-Learn and @DeniseColbert

Could this be used to identify users in the England syndicate, for the purposes of #1375 ?

Possibly


[SPARK-38234] [SQL] [SS] Added structured streaming monitoring APIs.

What changes were proposed in this pull request?

Add two monitoring REST API endpoints under SQL that provides details about the structured streaming queries. Even though a store exists for it and the data is presented in the UI under the "Structured Streaming" tab, this data is not exposed as a REST API.
Summary API
Returns the summary of all existing streaming queries.
GET /{appId}/sql/streamingqueries
Response is list of StreamingQueryData.
Progress API
Returns the progress events of a specific streaming query by runId. User can also specify how many of the most recent events needs to be retrieved by using the last query parameter. By default, we return the most recent progress event i.e. last is set to 1.
GET /{appId}/sql/streamingqueries/{runId}/progress?last={N}
Response is list of StreamingQueryProgress.
Note: We are not introducing new object definitions for the response since we are just returning the data from the store without aggregation, these are existing event structures - StreamingQueryData and StreamingQueryProgress.
Why are the changes needed?

This data can be used for monitoring, detecting streaming and to build custom dashboards. This monitoring API will be similar to the monitoring APIs that are present for DStreams - refer SPARK-18470.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Added 4 tests under SqlResourceWithActualMetricsSuite to test the endpoints when streaming code is executed. Tested various cases, parameters, boundary conditions

Added 2 tests under SqlResourceInvalidEndpointSuite.scala to test the endpoints when no streaming code is running.

As mentioned earlier, since we are just returning the objects from the store, there is no need to test them individually.

@dongjoon-hyun I've created this PR against master as per your suggestion. Can you please take a look? Thanks!

Apache Spark is using your GitHub Action to test your PR. Please enable GitHub Action on your Spark fork.

https://github.com/apache/spark/pull/35548/checks?check_run_id=5225282180

I have enabled it, can we re-trigger the build?

cc @xuanyuanking @viirya @HeartSaVioR @gengliangwang FYI

@HeartSaVioR can you please take a look at this PR?

cc. @sarutak as well

Gentle ping @dongjoon-hyun @HeartSaVioR @sarutak

Can one of the admins verify this patch?

@HeartSaVioR @viirya @sarutak @dongjoon-hyun @xuanyuanking   can you please take a look at this and provide your review? This is a small PR adding useful monitoring APIs to structured streaming. Thanks!

I'm not heavily involved on Spark UI - I'm afraid I'm the right person to make a decision about exposing REST APIs which are not used directly in Spark.
My general concern is that whether we consider REST APIs as public API or not. If we consider it as public API, we will have to worry about compatibility on StreamingQueryData and StreamingQueryProcess, since they are directly exposed to the REST API without any versioning. We wouldn't like to get blocked to change these models just because we expose them via REST API. Versioning would add non-trivial additional load to us - whenever we change the models, we have to retain the all models with versions so that we can serve all clients without having them to change their logic.
The realistic approach might be considering the REST APIs as developer API, meaning the output would be subject to change according to Spark version. I'd like to hear the actual use case of new REST APIs to see whether this could be acceptable or not, and also see we have alternative (like dropwizard metrics) or not.

@HeartSaVioR thank you for your response.

I understand the concern of sending the response as store objects which blocks the evolution of their structures between versions. However, these objects are also part of the existing programmatic monitoring APIs as mentioned here - https://spark.apache.org/docs/latest/structured-streaming-programming-guide.html#reading-metrics-interactively. Wouldn't the versioning issue also affect those APIs? Do the contracts apply only for REST APIs and not metrics objects or programmatic API responses?
On the other hand, what if we create another object with the same structure so that we do not need to break the API response objects but can evolve the StreamingQueryData / StreamingQueryProcess
We did have similar APIs for DStreams like /streaming/statistics, /streaming/receivers etc. How was this problem handled there?
I'm fine with marking this as Developer API - can you show me a precedence? How do we do that?
There are multiple use cases for these APIs - here are some that I can think of:

First one is real-time monitoring, these APIs can be used to build more sophisticated UIs than the one that is present under  the Structured Streaming tab. For example, we can build a live version of it that plots the data on the client side by frequent refreshes without reloading the page.
The other is to show the plot similar graphs about streaming metrics for a specific streaming query in 'Notebooks' under the query cell to enhance the experience.
The simplest of use case is to detect if a job contains streaming code which might be a variable in optimizing usage of resources for long running applications.
Achieving the same via DropWizard / other alternatives would add the overhead of building one more REST layer and we lose the real-time experience.

Gentle reminder to take a look a look at this @HeartSaVioR

@HeartSaVioR gentle reminder to take a look at this.

@HeartSaVioR @dongjoon-hyun @sarutak @viirya can you please take a look at this? This small PR adds monitoring APIs for structured streaming.

@HeartSaVioR can you please provide your review? Thanks!

Sorry I was too busy with handling other stuffs. I'm handling the blocker stuff for release and for others' works I'd focus reviewing on bugfixes. In addition, as I made it clear in previous comment, I'm not the one deeply involved in UI and I don't think I can sign off by myself for this one. If you'd like to move this forward, please try to get consensus about necessity of this feature and find some supporters.
My comments are all general one and subject to change if experts/another supporters have objects:

I understand the concern of sending the response as store objects which blocks the evolution of their structures between versions. However, these objects are also part of the existing programmatic monitoring APIs as mentioned here - https://spark.apache.org/docs/latest/structured-streaming-programming-guide.html#reading-metrics-interactively. Wouldn't the versioning issue also affect those APIs? Do the contracts apply only for REST APIs and not metrics objects or programmatic API responses?

We annotated all models with @Evolving and do not guarantee compatibility. We tend to not break backward compatibility, but we won't be blocked if it is unavoidable to make improvement. This brings less problem for customer listener since the compiler can notice the incompatibility when building a custom listener with new Spark version. Would it be OK for REST API as well?

Achieving the same via DropWizard / other alternatives would add the overhead of building one more REST layer and we lose the real-time experience.

I'm not sure I understand correctly - there is already a strong ecosystem being built among Dropwizard - time-series DB - UI. Once Spark produces the metrics via DropWizard to time-series DB, Spark can defer the concern to time-series DB and UI layer. The mechanism has been working well for a decade (I worked for Apache Storm which also produces the metrics to DropWizard).

First one is real-time monitoring, these APIs can be used to build more sophisticated UIs than the one that is present under the Structured Streaming tab. For example, we can build a live version of it that plots the data on the client side by frequent refreshes without reloading the page.

I agree with this, but if we think building a live version of the page would be helpful on monitoring and you have an idea, why not introducing such improvment to Spark UI and include this as a part of that?


Reassign partitions with replication factor option

The current feature for reassigning partitions doesn't allow you to update the replication factor, in case you've wrongly created a topic with an undesired replication. This offers you a way to change that in the Generate Partition Assignments step, by specifying the desired replication factor within the optional field.

You want to add a tool tip or note that this isn't required field?

It's an optional input field on the frontend side

@patelh, It's an optional input field on the frontend side with the desired replication factor.

@patelh, any follow up on this?

Thank you for creating this feature. I manually patched my installation.

can u fix conflicts? github isn't giving me option to resolve conflicts myself

@patelh, I fixed the conflicts :smiley_cat:

Somehow I missed your comment about fixing conflicts.  Sorry.  One more conflict to resolve.  If you want me to fix it, add me as contributor to your forked kafka-manager repo and I can update your branch.

@patelh, fixed the conflicts


Sort types

Bindgen generates types in the order they appear in a header file.
Consider following example:
typedef struct points points;
typedef struct point point;

struct points {
    point *point1;
    point *point2;
};

struct point {
    int x;
    int y;
};

It will generate types in the following order:
type struct_points = native.CStruct2[native.Ptr[point], native.Ptr[point]]
type points = struct_points
type struct_point = native.CStruct2[native.CInt, native.CInt]
type point = struct_point

In generated code points appear before point although points uses point.
To sort types we need to know what types are used by other types and then do topological sort.
There is also one problem that should be considered, types may have cyclic dependency:
struct b;
struct c;

struct a {
   struct b *b;
};

struct b {
    struct c *c;
};

struct c {
    struct a *a;
};

I think this should be moved to 0.4


How can I test the AP of each class in the coco dataset?

In addition, what is the difference between the proposal and bbox tested?
Thanks！

Please refer to the 5th example in the doc.

Please refer to the 5th example in the doc.
What IoU threshold is this AP for each class based on?

Please refer to the 5th example in the doc.
What IoU threshold is this AP for each class based on?


Build shared libraries as well

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, rerender

@conda-forge-admin, rerender

Do you want to use these for the compilers too? That could be awesome, currently installing clang on macOS takes ages since we statically link them (and bundle all the static libs).

Sure. rebuilding clangdev using this package would be enough.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Failed to even lint the recipe, probably because of a conda-smithy bug :cry:. This likely indicates a problem in your meta.yaml, though. To get a traceback to help figure out what's going on, install conda-smithy and run conda smithy recipe-lint . from the recipe directory.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I'm going to merge this as it's backwards compatible and will work on 8.0.1


JUnit5 migration

@rzo1 : a summary of the pull request so it's easier for you to review.
Regarding the build:

I noticed that the Spock tests weren't executed during the build.
The Groovy/Spock parts inside the Maven build script are not yet removed, that would be for a later commit (they are already no longer used).
surefire is upgraded to a milestone M7, because you mentioned M6 and I'm using M7 successfully in my other projects.

Regarding the tests:

The Spock tests contained code that was not up to date with the latest API of various classes.
At the moment all tests are migrated to JUnit5 and they get compiled and executed during the build, so that's an improvement :-).
Formatting will be off here and there, because the work was literally renaming .groovy files to .java and fixing coding problems using copy pastes while having non-compiling code (confuses the formatter).
I'm using AssertJ which is a continuation of fest-assert which was already way more useful (and readable) than JUnit's own Hamcrest library.
There are still 6 failing tests -> I would need an extra pair of eyes for those, it might help that you know more of the history of the project. I'll make a follow-up post with more explanation.
There is 1 test disabled ("ignored"). This is because of coding that changed after updating to the latest HttpClient version and the test can no longer be made to fail in the same way.

Regarding the code:

Based on the tests I learned that the exception handling inside WebCrawler could receive a partially filled in Page object. More specifically, when a timeout occurs it is possible that the Page object not only contains the webURL, but also the status code and response headers (so timeout occurs while transferring the page payload). This means that the new methods that replace the deprecated ones get an "upgrade" in the argument list, so they no longer receive WebURL and Exception, but rather Page and Exception (with a comment that the Page object will only be partially populated). This should be trivial to see and fix when upgrading after following the (removed) deprecated method upgrade guidelines of version 4.10.1.

All in all this pull request should already be a nice improvement, even when there are 6 tests failing (see follow-up post).

@rzo1 : about the failing tests.
BasicAuthTest
Not sure what changed to make this test fail. The code never allowed anything more than provide credentials, but using HttpClient manually, I would use following code to define that I want authentication to be done preemptively:
		final BasicScheme basicAuth = new BasicScheme();
		basicAuth.initPreemptive(new UsernamePasswordCredentials(userName, password));

So now I'm wondering how this used to work. This might require code changes unless you know of a use case where this (still) works. I expect the test to work the same as when it was written in Spock syntax (WireMock does the real magic in both the Spock and JUnit tests).
PublicSuffixTest
This detects the wrong "registered domain" and "subdomain". This code has never been touched, even in the previous repository, so I'm probably looking over something. It seems like AbstractWebURL.setURL(...) is not eagerly trying to detect the registered domain. It goes from right to left, and whenever a candidate satisfies following test:
if (tldList.isRegisteredDomain(candidate)) {
    rd = candidate;
}

... searching for a bigger possible registered domain name stops, and everything else is put as subdomain.
TLDListOnlineTest
I think this will be identical to the previous problem.

Implementation for issue https://github.com/rzo1/crawler4j/issues/113

It looks like the Groovy tests were never executed (at least for a long time) as the code only differs in the use of a StringBuilder and a missing InternetDomainName.isValid(domain) from guava, which we could add back by shading Guava and strip away the unnecessary garbage ;) Side note: This will not change anything, i.e. test will still fail.
Basically, I am wondering, if the test values are correct, i.e. are we sure we are testing the right things here?
It looks like the code assumes, that com is the domain and www.example is the sub domain.  But I guess, that the original intention of domain here is the registered domain. I think, that this "bug" / change in behaviour was introduced by switching to crawler commons instead of using Guava''s InternetDomainName.
I will push a fix to this PR.

Regarding BasicAuthTest, I guess it was just broke or broken earlier by moving to HTTP Client 5. Don't think, that this should be a blocker for this PR, so maybe we can just @Disable it and check for the BasicAuth part later.

I noticed, that the tests take a siginifcant amount of time (even locally). I guess, that we need to do anotheri iteration in order to speed things up ;)

It looks like the Groovy tests were never executed (at least for a long time) as the code only differs in the use of a StringBuilder and a missing InternetDomainName.isValid(domain) from guava, which we could add back by shading Guava and strip away the unnecessary garbage ;) Side note: This will not change anything, i.e. test will still fail.
It looks like the code assumes, that com is the domain and www.example is the sub domain. But I guess, that the original intention of domain here is the registered domain. I think, that this "bug" / change in behaviour was introduced by switching to crawler commons instead of using Guava''s InternetDomainName.

I'm not even sure crawler commons is the issue. Maybe the code in crawler4j should start testing with the biggest possible String first and start removing parts from the left. That way the largest possible registered domain returns true and everything else is subdomain.

I noticed, that the tests take a siginifcant amount of time (even locally). I guess, that we need to do anotheri iteration in order to speed things up ;)

The WireMock based tests are indeed a lot slower. It's possible this was always the case, but even then it's nice to have a possibility for a quick build and one with slow tests enabled (typically integration tests).

I'm not even sure crawler commons is the issue.

The problem originated from my misunderstanding of how the (undocumented) code should behave while doing the crawler-commons migration. I spend some time yesterday to debug and adjust accordingly to past the tests. The basic idea here is, that getDomain should return the private / registerable / pay-level domain while everything in front of it is the subdomain. This definition if things is okaish while working on a crawler but imho has some definition gaps ;)
Nevertheless, it now behaves as before (without doing a Guava shade in crawler4j).

The WireMock based tests are indeed a lot slower. It's possible this was always the case, but even then it's nice to have a possibility for a quick build and one with slow tests enabled (typically integration tests).

Yes. Perhaps we should make them real IT tests using failsafe + a profile, so we can run them in CI (only).


TensorBoard.dev is shutting down

Good morning, I'm on notebook 4 along with classes on the Udemy platform. I found a problem with Tensorboard.
Tensorboard

Through the documentation, I was able to solve the problem, in which it is not possible to open the external URL of tensorboard.
Using the example in the documentation, I adapted it to a notebook and now it runs normally.
Documentation
I forked the project and am sending the update.

Can you highlight specifically which area you updated?  The notebook / diffs are too large to display in the browser, and downloading locally I wasn't able to see it either.
I have been skipping the TensorBoard callbacks due to this issue myself.

Those locales:


if /tmp/chef is not a git repo recheck out

use git status to see if /tmp/chef is a git repo if it is not (returns non-zero error code) blow away and recheckout. Should solve most nightly cron failures when used repeatedly.

Fixed https://github.com/schubert/brewstrap/commit/3322f730cfbc4ed8334d5b95e3c70df86a2d85c4


Invalid CSS in xt256.css theme

The xt256 theme is missing a unit measurement for the padding property on the .hljs class. Existing code:
.hljs {
  ---
  padding: 0.5;
}

This should probably be:
.hljs {
  ---
  padding: 0.5em;
}

I can submit a PR if that's easier.

Would you kindly please send a pull request?

Absolutely! Opened as https://github.com/highlightjs/highlight.js/pull/1895.


wait for tenant to be active before polling for timeline absence

Problem
https://neon-github-public-dev.s3.amazonaws.com/reports/main/5692829577/index.html#suites/f588e0a787c49e67b29490359c589fae/4c50937643d68a66
Summary of changes
wait for tenant to be active after restart  before polling for timeline absence

test_concurrent_timeline_delete_stuck_on[in_progress_delete]: debug

Re-running.

Re: timeouts, adding this one here as well: https://neon-github-public-dev.s3.amazonaws.com/reports/pr-4857/5718388045/index.html#suites/f588e0a787c49e67b29490359c589fae/5ef37c7fda6cfb7f
TL;DR: another tenant load not completing in 2s, this time aiming for Broken. I think we should raise all timeouts to 10 seconds on tests which are also ran against real_s3.

I'll rerun workflow for several times to be sure that the fix helped


install script for develop branch installs master branch

After using the install script for the develop branch, Phoniebox works as expected. Good.
But checking the branch with git branch I am on master not develop.
@s-martin @ZyanKLee what do you think. Shouldn't the develop install be in the develop world?
https://github.com/MiczFlor/RPi-Jukebox-RFID/blob/4c7c9c7bc9eddf726346af96a7df9ce6cad56f6f/scripts/installscripts/buster-install-default.sh#L696

The idea was that you need to set a env variable before executing the install script, i.e.  GIT_BRANCH=develop bash ./buster-install-default.sh.

True, that's a good point (that I knew and forgot...). I have to check the install script again. My understanding was:

if you do not provide that variable (as it is correctly done in the wiki for develop) it will switch to master
the version number will be generated including the branch
But, I did the install with the conf file (non-interactive) and get the version number 2.0 %GIT_COMMIT% %GIT_BRANCH% in the WebUI

Hi @s-martin
All works as expected. My bad.
I thought the first time when installing with the config file non-interactive the version number was wrong. Now that didn't happen.


Fixes ProjectManager Buttons Not Updating

Fixes buttons in ProjectManager not updating to disabled after deleting projects.
Closes: #31377

Thanks!


Improve dir-compare tests

Changes
Realized that in our tests, we had the following line:
if (entry.type1 !== 'file') {
  return;
}

The original intent of that was to only compare files, not directories. But instead, it was skipping a crucial scenario. When dir-compare finds a file in one location, but not the other, it returns { type1: 'missing', type2: 'file' }. This means our snapshots were incomplete.
This updates the tests suites (and adds missing files) to ensure we’re testing the full snapshot.

Testing

assuming all of those snapshot updates are expected

😅 yeah unfortunately we weren’t testing many files to the snapshot


[EdgeTPU] HotFix Remove Advanced Option 'Help'

All options have codicon that describes each options.
Therefore, this option is useless.

ONE-vscode-DCO-1.0-Signed-off-by: hohee-hee<EMAIL_ADDRESS>

LGTM :-)


Fix for responses with empty body

Change description
PromEx.Plugins.Phoenix should return 0 as a measurment for "http.response.size.bytes" if the response body is empty.
What problem does this solve?
In my app I have a plug that checks session value and redirects to root page if the user id is missing or the user has no access due to role.
I noticed an error in PromEx when such redirects occurred:
01:10:30.111 [error] Handler {PromEx.TelemetryMetricsPrometheus.Core.EventHandler, #PID<0.3719.0>, [:my_app, :prom_ex, :phoenix, :http, :response, :size, :bytes]} has failed and has been detach
ed. Class=:error
Reason=:badarg
Stacktrace=[{:erlang, :iolist_size, [nil], []}, {PromEx.Plugins.Phoenix, :"-http_events/2-fun-0-", 2, [file: 'lib/prom_ex/plugins/phoenix.ex', line: 96]}, {PromEx.TelemetryMetricsPrometheus.Core.Eve
ntHandler, :get_measurement, 3, [file: 'lib/core/event_handler.ex', line: 63]}, {PromEx.TelemetryMetricsPrometheus.Core.Distribution, :handle_event, 4, [file: 'lib/core/distribution.ex', line: 71]}, {:telemetry, :"-execute/3-fun-0-", 4, [
file: '/app/deps/telemetry/src/telemetry.erl', line: 135]}, {:lists, :foreach, 2, [file: 'lists.erl', line: 1342]}, {Plug.Telemetry, :"-call/2-fun-0-", 4, [file: 'lib/plug/telemetry.ex', line: 76]}, {Enum, :"-reduce/3-lists^foldl/2-0-", 3
, [file: 'lib/enum.ex', line: 2111]}]

Checklist

[ ] I have added unit tests to cover my changes.
[ ] I have added documentation to cover my changes.
[x] My changes have passed unit tests and have been tested E2E in an example project.

Pull Request Test Coverage Report for Build c37d53e8c7d5b47b2e43fbdadbaa1cab9d701255-PR-54

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 84.488%

Totals

Change from base Build da3fe1ee6163f77f3e4169e65d34b85dc10b28c6:
0.2%

Covered Lines:
561

Relevant Lines:
664

💛  - Coveralls

Thanks for the fix and good catch!


FEAT: show progressbar and output only when not quite

Fixes #646

@jaapio please have a look at the integration tests?


Update Docker Image in DockerHub

Can we get the Updated Cachet Docker Image with Redis Driver?

I kicked off a build of master (:latest docker image) and backported changes to 2.3 branch (:2.3-latest docker image), which should include redis.

@djdefi These changes didn't make it into the 2.3:latest tag. They are in the 2.3.13 release of @cachethq/cachet, but the latest 2.3 version of the docker image is 2.3.12.
Can we get a 2.3.13 built and get the 2.3-latest tag updated?


How to get the message in events?

How to get the message in events?

If possible, can we include message label as well?


feat: add menubar component

close #39

There is a bug in the menu component that causes the class in the menu content to be rendered twice


Alternate TTS engine

Hi,
I see you have a list of possible TTS engines to add in the future.
I want to suggest to add this one to the list :)
https://github.com/serp-ai/bark-with-voice-clone
Regards,
Hyper

Awesome, I added it to the list :wink:


Update value sets for guidelines document v1.11 (to be adopted)

Hi,
as far as Im aware there is also new vaccines BionTech and Moderna Vaccines (BA.1) available soon-ish. Will this result in seperate/new valuesets for those or do they go under the current Comirnaty and Spikevax sets? Thanks

Hi @mrkobk ,
If it's decided by the eHN that it's necessary to be able to distinguish between variant-specific vaccines, then they will quite likely end up as separate entries in the appropriate value set. Yesterday, the v1.11 document was adopted during the eHN Coordinated Actions meeting, so that will also not be amended for variant-specific vaccines.

Hi @kruzikh ,
Apologies: I totally didn't see you had already made a release/2.9.0 branch for the value set update. I just merged a PR with the same content.
Now I just have to re-make a branch for the release 2.9.0 conforming to the release process (which I didn't follow precisely, or I'd seen your branch earlier). Would you mind if I deleted your 2.9.0 branch so I can hang the branch label on a commit that leads to main?

No problem, just remove it.

No problem, just remove it.

Thanks! Done, and re-created the branch from a commit leading to main.


IRGen: Use any field of fixed-layout structs for extra inhabitants.

This allows us to layout-optimize Optional when T is a struct with an extra-inhabitant-bearing field anywhere in its definition, not only at the beginning. Since runtime struct layout instantiation does not yet handle this, limit the effect only to structs whose layout is always completely known at compile time independent of their type parameters. rdar://problem/43019427

@swift-ci Please test

@swift-ci Please test source compatibility

@slavapestov Is there anywhere in RemoteAST/RemoteMirror or lldb that extra inhabitant counting for structs is hardcoded that I should update?

This is great and gets us a fair deal closer to optimal extra-inhabitant utilization!

@dcci It would be different in a case where there's an object reference in a struct in some position other than first, and you use an Optional of that struct. For instance:
struct Test {
  var a: Int
  var b: AnyObject
}

Previously, Optional<Test> would use an extra byte for its tag. With this change, Optional<Test> is the same size, and uses a null pointer in b to represent nil.

Build failed
Swift Test Linux Platform
Git Sha - 5cfc15071261ef2360e8d06ab1483cac6c511de4

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - 5cfc15071261ef2360e8d06ab1483cac6c511de4

Build failed
Swift Test Linux Platform
Git Sha - 5cfc15071261ef2360e8d06ab1483cac6c511de4

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility

@jckarter Remote mirrors do not know if the layout was static or dynamic. Look at stdlib/public/Reflection/TypeLowering.cpp.
Also what about resilience? If the struct is resilient, you can't use its extra inhabitants with your scheme.
I would suggest implementing the runtime support at the same time and getting rid of the special case. Other than that, this is a great idea!

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 8bcab803fe5abb87a3aae95684be5a0bfccc857f

@rjmccall @slavapestov I added handling for a runtime-selected extra inhabitant field as well. How does this look?
The annoying conditional dance here has me thinking we probably ought to just have a non-executable representation of the extra inhabitant set, since all of our extra-inhabitant-bearing types' sets are representable as an (offset, width, stride) triple. That would let us shed those value witnesses altogether, and would make forwarding the extra inhabitants from the chosen field at runtime here trivial. I'd like to investigate that separately from this PR, though.

Build failed
Swift Test OS X Platform
Git Sha - 8bcab803fe5abb87a3aae95684be5a0bfccc857f


Check if platform supports feenableexcept

feenableexcept is not supported by uclibc
Signed-off-by: Gilles Talis<EMAIL_ADDRESS>[Retrieved (and updated to add cmake support and simplify configure.ac)
from
https://git.buildroot.net/buildroot/tree/package/tesseract-ocr/0001-Check-if-platform-supports-feenableexcept.patch]
Signed-off-by: Fabrice Fontaine<EMAIL_ADDRESS>

These changes should not be needed. The GNU C library has feenableexcept, and other C libraries should not define __USE_GNU. Where does it come from when uClibc is used?

uclibc-ng defines __USE_GNU (cf. https://github.com/wbx-github/uclibc-ng/blob/98680eec548b921abecc641b70246587ffbbdc0a/include/features.h#L350) as well as __UCLIBC__. However, I would be more than happy to just disable tesseract with uclibc-ng. This PR (as well as #3717) is needed because tesseract is raising build failures on buildroot autobuilders since our bump from version 4.1.1 to 5.0.1.

I see two acceptable solutions:

If nobody uses Tesseract with uClibc I'd disable it. That does not require changes in the Tesseract code and reduces electrical power consumption.
If it is used, we could add conditional compilation using __UCLIBC__. Normally I prefer the way which you have implemented in your two commits, but this looks like a very special case needed because of an incomplete C library, so using that macro would be acceptable for me.

I always hesitate to add new checks to configure because each check increases the execution time and the log file a little bit more.

After further investigation, musl doesn't support feenableexcept either: https://github.com/ifduyue/musl/search?q=feenableexcept. As we can't really say that musl is incomplete (e.g. it is used by Alpine Linux), I would say that feenableexcept is a glibc specific extension.
Guarding the call to feenableexcept by defined(__GLIBC__) && !defined(__UCLIBC__) should work but this is really ugly compared to having the proper function checks in configure ...

You are right, feenableexcept is a glibc extension. From the documentation: "Since version 2.2, glibc supports the functions feenableexcept() and fedisableexcept() to set individual floating-point traps". So #if defined(__GLIBC_PREREQ) && __GLIBC_PREREQ(2,2) should be the right test.
uClibc claims to be glibc 2.2 by setting the relevant macros, but is obviously incompatible to glibc 2.2. I'd call that a bug.
We can use a modified test #if defined(__GLIBC_PREREQ) && __GLIBC_PREREQ(2,3) which should work for uClibc and musl (which correctly does not claim to be glibc).

Unfortunately, uclibc-ng also defines __GLIBC_PREREQ: https://github.com/wbx-github/uclibc-ng/blob/ab1dd83bec59c9e65c31efd6e887182948f627be/include/features.h#L391.
So the defines will have to be:
#if defined(__GLIBC_PREREQ) && !defined(__UCLIBC__)
#if __GLIBC_PREREQ(2,3)

It should be noted that defined(__GLIBC_PREREQ) can't be in the same if && condition than __GLIBC_PREREQ(2,3) as it will result in a build failure if __GLIBC_PREREQ is not defined.

This PR's function check is much simpler than ifdefs you are talking about.

libc in FreeBSD, NetBSD and OpenBSD also supports feenableexcept().

Then I assume that function checks is indeed the best solution.


Accessible color contrast

I rule and have completed some work on Case Manager that's ready for review!
I changed colors slightly to fix some accessibility issues!
I think the only entry I haven't addressed is on http://localhost:3000/accountants,             No patients found           | Element has insufficient color contrast of 4.47 (foreground color: #777777- background color: #ffffff- font size: 12.4pt- font weight: normal). Expected contrast ratio of 4.5:1
This pull request makes the following changes:

Change blue on sign-in page from #337ab7 to #23527c

change grey in nav-bar from #828282 to #525252

change blues on a/link tags. Only the lighter blue was failing, but to keep the hoover,etc UX decided to change both.

Overwrite form-text bootstrap grey on form sub-text

Overwrite btn-danger bootstrap background color. Didn't change border color, but happy to change that as well!

It relates to the following issue #s:

fixes (almost) #1704
Bumps #1704

For reviewer:

Adjust the title to explain what it does for the notification email to the listserv.
Tag this PR:

feature if it contains a feature, fix, or similar. This is anything that contains a user-facing fix in some way, such as frontend changes, alterations to backend behavior, or bug fixes.
dependencies if it contains library upgrades or similar. This is anything that upgrades any dependency, such as a Gemfile update or npm package upgrade.

If it contains neither, no need to tag this PR.

UX skim looks good! definitely wanna get that border color too!


Reimplement with zip and two DirWalkers

The current implementation has the problem that every matching file is opened and read twice. This is an alternative implementation that only checks every pair of files once.

I've made the changes in new branch https://github.com/mikerite/dir-diff/tree/walker_zip_pr3. How do I update this pull request with the new branch? Must I open a new one?

I've made the changes in new branch https://github.com/mikerite/dir-diff/tree/walker_zip_pr3. How do I update this pull request with the new branch? Must I open a new one?

You either need to modify your existing branch to include those changes or create a new PR.

I've opened pull request #13 with the changes.


Multiple issues with The Sims 2

Currently, The Sims 2 suffers from a variety of issues when running with D9VK.

[x] Antialising doesn't work (but if you play the trace using wined3d it works)
[x] The ground in the neighborhood menu is pitch black.
[x] The water reflection in the neighborhood menu does not work properly.
[x] The roads in the neighborhood menu are not displayed properly, they change color when being zoomed in and out.
[x] The family thumbnails in neighborhood menu and loading screens have exploding characters due to lack of ProcessVertices implementation (they get created when you click on a family for the first time and after that, the image gets cached). Occurs even when the game uses shaders.
[x] The bridges look way too bright (probably due to missing fixed function lightning implementation).
[x] The ground around the household lot during night time doesn't look dark, which is incorrect.
[x] Each individual ground tile around the lot appears to have a shadow map? on it in older versions of the game and when the new versions incorrectly detects that a GPU doesn't support shaders.
[x] The mirrors don't work properly (see the video in bottom section)
[x] The shadows in the neighborhood are misplaced and move when I zoom in and out. This did not happen before https://github.com/Joshua-Ashton/d9vk/commit/8d564f22a423f6eb9d67b8347221bfadb9c0ab2b
[x]  The water is flickering and blocky. This did not happen before https://github.com/Joshua-Ashton/d9vk/commit/8d564f22a423f6eb9d67b8347221bfadb9c0ab2b
[x]  Characters and flowers (if the game incorrectly detects that a GPU doesn't support shaders), certain objects like friendship icons that appear above sim heads and custom user-created haircuts (latter 2 occur even when the game uses shaders) explode due to lack of ProcessVertices implementation.
[x] The roofs during rain don't render correctly.
[x] Object shadows (like tree shadows) seem to only appear outside the lot when they are supposed to actually be cut away at the edge of the lot.
[x] The real-time rendered cut-scenes look completely black or with bits of UI that disappear very quickly.
[x] Specular highlights don't work on AMD - needed for some metallic objects (needs vendorID spoofed to Nvidia)
[x] Objects rendered in FF/SWVP mode appear darker than they are supposed to be.

System information

GPU: Nvidia GTX 1070
Driver: 430.40
Wine version: staging-4.13
D9VK version: master

Game demo
To reliably reproduce the flickering issue, it's best to use this game demo:
https://drive.google.com/open?id=1ZxjlzJi8cbD_RWABnHayrPp5JLwDxQs0
Apitrace file(s)
https://drive.google.com/open?id=1CmUlo3XG-34YTOK769rZ9LhWNB6j4C56
Recorded with Wined3d. The game was forced to use fixed function to workaround a tracing crash when loading the neighborhood menu and household mode. Purple sims should be ignored - it's some weird artifact then happened during tracing and it got baked into the trace, they look just fine with only D9VK in use.
An opengl trace of wined3d for reference:
https://drive.google.com/file/d/1Sudv_eeUGr_aJpVY4bDPd4x4f4i4QjaZ/view?usp=sharing
Log files
Sims2EP9_d3d9.log

Tracking flickering issue at https://github.com/Joshua-Ashton/d9vk/issues/324

Tracking broken shadows at https://github.com/Joshua-Ashton/d9vk/issues/325


Permissions not granted in Android 13

For Android 13, even after granting the app permissions for the gallery and camera, the permissions are not granted by Android. It shows "Camera permission not granted" and "Cannot access gallery" in the logcat.

No, I couldn't find a solution to it. I just had to do a custom implementation for the project.


[smoke tests] fix known flaky tests

Motivation
See commit messages for details.
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
Existing tests.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

:v: @zekun000 can now approve this pull request

@bors-libra r=metajack

:pushpin: Commit fb12b06 has been approved by metajack

:hourglass: Testing commit fb12b06e4bcf31a4aa98faf05ed986aa0498553d with merge 3b4e39490cf57cb9d03de60f60bf323ec738e9b5...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: metajack
Pushing 3b4e39490cf57cb9d03de60f60bf323ec738e9b5 to master...


Added Long Curry functions

I think you guys' Curry.swift is not enough for big projects.
I added long Curry functions with swift script.
@stephencelis

Hey @saltylight! Thanks for the PR!
I did some quick benchmarking and, on my computer, master builds in about 3 seconds, while this branch at 22 arguments takes about 12 seconds, so it's about 4x slower. By dropping down to 18 arguments, it's only marginally slower, at 4 seconds.
Are you cool with changing this to be 18 arguments for now?

@stephencelis Coooool. That's a good point! I dropped down to 18 arguments. 😄


GitHub actions

thanks @ronayadid!
can you please delete circleci config file config.yml?

@kobik circleci files removed


win: UV_PRIORITIZED should be ignored on Windows

CI: https://ci.nodejs.org/job/libuv-test-commit/2153/


MGDOBR-981: Action/Source configuration - browser saving/restoring form data

JIRA: https://issues.redhat.com/browse/MGDOBR-981
Unfortunately there's no standard way to prevent browsers from asking to store credentials.
This PR is focusing only on the autocomplete feature, to at least prevent form values from being auto-filled.
Steps to test it (on Chrome):

Create a processor with an action/source with a password field
Store the credentials
Create another processor and check that the saved credentials are not auto-filled.

Hi guys, I've had another idea and I'm still looking into this.
I'm converting to draft. Hold on your reviews for now, sorry!

Hi guys, I've had another idea and I'm still looking into this. I'm converting to draft. Hold on your reviews for now, sorry!

In the end my new ideas didn't work. Ready for review now.


Create 0000-01-02-Gimpyjoe.md

learning curve, this is very strange to me, created a branch, created a file, and made a commit, and opened a pull request.

It's still very confusing to me, but I am still learning.

It's still very confusing to me, but I am still learning.


Added detox smoke test's in the travis CI

The ios build run's fine on Xcode but fails in command line  xcodebuild -project ios/ZulipMobile.xcodeproj -scheme ZulipMobile -configuration Release -sdk iphonesimulator -derivedDataPath ios/build
** BUILD FAILED **
The following build commands failed:
	CompileC build/Build/Intermediates.noindex/ZulipMobile.build/Release-iphonesimulator/ZulipMobile.build/Objects-normal/x86_64/AppDelegate.o ZulipMobile/AppDelegate.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)
child_process.js:634
    throw err;
    ^
Error: Command failed: xcodebuild -project ios/ZulipMobile.xcodeproj -scheme ZulipMobile -configuration Release -sdk iphonesimulator -derivedDataPath ios/build
    at checkExecSyncError (child_process.js:591:13)
    at Object.execSync (child_process.js:631:13)
    at Object.<anonymous> (/Users/travis/build/kunall17/AutoTools/node_modules/detox/local-cli/detox-build.js:26:6)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Function.Module.runMain (module.js:605:10)
    at startup (bootstrap_node.js:158:16)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Any info how to resolve this error?

Automated message from Dropbox CLA bot
@kunall17, it looks like you've already signed the Dropbox CLA. Thanks!

You might want to try ZulipMobile.xcworkspace and maybe pod install (not sure if relevant)

@borisyankov Not working with these!

After a lot of work and trials the CI is hooked up with the basic detox UI test's!
Next step would be initiate the server and then use the server to execute more tests. :) 🎉

@borisyankov fixed this, LGTM
After this I'll write more test cases!

Do the Travis tests pass consistently? What are the risks if we merge this?
@Sam1301 have a look at this PR too (and comment). We will all be writing tests next.

Let's see, I have rebased!

@borisyankov the build passed after rebasing, so it would be stable if the detox test's passes locally!

Merging it then. 🌮


new rule: react: className starts with file name

Enforce React component's className starts with current file name.
Example
In MyComponent.ts
Ok
React.createElement("div", { className: 'MyComponent' });
React.createElement("div", { className: 'MyComponent-Icon MyComponent-Icon-hover' });

Wrong
React.createElement("div", { className: 'panel' });

Hey @slawomir, are you still interested in taking this one? It seems like enforcing at least one of a file's React components, if there are any, have the file's name would be useful.

💀 It's time! 💀
TSLint is being deprecated; therefore, it and tslint-microsoft-contrib are no longer accepting pull requests for major new changes or features. See #4534. 😱
If you'd like to see this change implemented, you have two choices:

Recommended: Check if this is available in ESLint + typescript-eslint ✅
Not Recommended: Fork TSLint or tslint-microsoft-contrib locally 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!


Extending DataObjects with CSVs

loading from dict or CSV now extends existing data instead of replacing it
Fixed framework/DataObjects.load_two_csv_histories
Fixed framework/DataObjects.load_two_csvs

Does this break any other test?


Feature Suggestion  - Colour Picker

Which area of IMT is your request about?  (ex:  Lines, Fillers, UI)
UI
Describe your idea clearly and concisely.
Colour picker -- the ability to select an existing colour from the screen / another network. I know there are other mods that implement a colour picker so it would probably be a fairly simple addition as that code is likely already open source.
Screenshots?
Provide a visual if possible what your idea would look like
Why would this idea be a good addition to IMT?
Add any other context or screenshots about the feature request here.
It would make for easy matching of various road's like bike or bus lanes, or non-standard lines.

Yes, we really need that.


I would be happy to take high-resolution die photos if you want.

Thank you for your kind offer, Ken.
The intention was to run this through our local university. I will get back to you if restrictions remain in place that prevent us from doing that by the end of summer. See also issue #8.


Could not perform discovery errors

Code that's throwing error:
clients.map(async (client) => {
      let tenant = client.tenant;
      let clientId = client.client_id;
      let clientName = client.name;

      const query = new QuerySpecification()
        .withStatement(
          "MERGE(c:Client {client_id: '$clientId', app_name: '$clientName'})-[r:CLIENT_OF]->(t:Tenant {name: '$tenant'})'RETURN c, r, t'",
        )
        .bind([clientId, clientName, tenant]);
      await this.persistenceManager.execute(query);

  Full error

  Error: Cannot run query in this transaction, because it has already been committed.
at Function.withRootCause (/home/taylor/modsquad/cubeless-backend/app/node_modules/@liberation-data/src/DrivineError.ts:8:16)
at Transaction.query (/home/taylor/modsquad/cubeless-backend/app/node_modules/@liberation-data/src/transaction/Transaction.ts:51:32)
at TransactionalPersistenceManager.execute (/home/taylor/modsquad/cubeless-backend/app/node_modules/@liberation-data/src/manager/TransactionalPersistenceManager.ts:28:9)
at /home/taylor/modsquad/cubeless-backend/app/src/clients/clients.service.ts:54:7

@tgod009 I have seen this error on recent Neo4j JS driver. Can you please try using the Neo4j driver directly? Does it work?

@jasperblues finally got around to trying with the neo4j driver directly and it worked. Any ideas why that is?

@tgod009 are you able to share a sample where it works directly but not with drivine? That would be much appreciated and should certainly lead to a solution.

I just initalized the driver in my service file and called the session like so
const session = this.driver.session();
    session
      .run('Merge (a:Application {name: $appName})', {
        appName: createUserApplications.app_name,
      })
      .then((result) => {
        return result.records.map((record) => {
          return record.get('a');
        });
      })
      .catch((error) => {
        console.log(error);
      })
      .then(() => {
        session.close();
      });

And the same thing does not work for you drivine @tgod009 ?

Running Drivine like this throws the routing table error
const query = new QuerySpecification()
      .withStatement('Merge(a:Application {name: $appName})')
      .bind(createUserApplications);
    return this.persistenceManager.execute(query);

Here's what I did to make it work:

Clone the drivine repository (this one).
Copy .env.sampe to .env and change the neo4j password to appropriate pass.
Modify the file RouteRepository. Add this code:

    @Transactional()
    async createApp(name: string): Promise<void> {
        const query = new QuerySpecification()
            .withStatement('Merge(a:Application {name: $appName})')
            .bind({appName: name});
        return this.persistenceManager.execute(query);
    }

^-- Note that the params are slightly different. Since we bind name parameters we supply the name.
Final Step
Modify the RouteRepository.spec.ts. Add this test:
    it('should create an app', async () => {
        await repo.createApp("foobar")
    });

The test passes. If you want to see the result in the DB, at the top of that file change the following:
RunWithDrivine({rollback: false}); <-- change rollback to false

If you follow the approach above, does it work for you? The may be an issue but the above works for me.

That's with the sample application?

No, I just added it to Drivine directly. You can clone that repo, create an env file, then run the tests.
Run this gist for most tests to pass: https://gist.github.com/jasperblues/7a6e774090c9ef0f18c44920b48b6b23
There are two tests that need additional setup steps. Don't worry about those, just add the code above and let me know what happens with your Neo4j server. If it works - it was a setup issue. If it does, its something environment related.
I was going to add to the sample app, but I think I need to freshen dependencies. (Will do shortly).

I gave it try and I got the same error using my db info
Error: Could not perform discovery. No routing servers available. Known routing table: RoutingTable[database=default database, expirationTime=0, currentTime=1679435963014, routers=[], readers=[], writers=[]]

Side note not sure if this effects it but I'm using Aura not a local DB with the encrypted url

@tgod009 I will see if I can reproduce with Aura.

Not sure if this was causing it but I did just notice that I was using the non encrypted protocol

If you can try with both and compare results that would be great. This is puzzling!


github-actions committer should be github-actions[bot]

Adding the [bot] makes it appear properly as a "user" in the commit history.

Irrelevant as of #35


fix(maker): detect out path of package step correctly

Check the 'name' property of electronPackagerConfig to determine the out path
ISSUES CLOSED: #102
Have you read the section in CONTRIBUTING.md about pull requests?
Yes
Are your changes appropriately documented?
N/A
Do your changes have sufficient test coverage?
N/A
Does the testsuite pass successfully on your local machine?
Yes
Summarize your changes:
See above

@malept Not without adding another 2 minutes to the tests just to test a fallback ternary.  I can add it but it seemed a bit heavy handed


Open pull request

Hi, i've forked the repo and for study i've working on it, if you want, i can open a pr with my changes

Hi, sure, any improvements are welcome!


Check artwork licenses of entries

That'll be a very unrewarding task, but still important to maybe replace copyrighted material with free material and make the games more indisputable.

During the import from Libregamewiki and OSgameclones I mostly copied the asset license information without checking them much. Ideally I want to put links into a code repository next to the license information to kind of prove the license.


[Bug] Plugin using remote library

Hey there your plugin was denied with reason:

This remotely links a library, see guidelines security & privacy #5.

This is referring to point number 5 of our Security & Privacy guidelines for plugins as seen here: https://docs.betterdiscord.app/plugins/introduction/guidelines#security--privacy
Please consider bundling the library you are using rather than remotely linking it. Once you do I'd love to see this plugin resubmitted! I think this is much better than our current LaTeX plugin.

Please consider bundling the library you are using rather than remotely linking it

I've been trying for a while but I'm not able to :/
Since you're experienced with this, would you mind looking at the related issue?  I'd be very glad. Maybe it's a common issue for BetterDiscord plugins. You know the internals better than me. I'm new to both BetterDiscord and JavaScript :D

Fixed in last version

Fixed* in latest version


[AutoPR containerregistry/resource-manager] ACR conf for 2018-09 and Python

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3824

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3140


Remove componentWillReceiveProps usages in examples

Summary
Resolves 1940
Change usage of componentWillReceiveProps in Tex examples to componentDidUpdate

Hi @chaitanya-deep, welcome to Draft.js and thanks for taking the time to look into this issue! This change looks good to me 👍🏼, sounds good to remove the sketchy setTimeouts also.


Add AlphaOmegaRecordingInterface

fix #211
Motivation
Adding a AlphaOmegaRecordingInterface.
Session start time is available and is fetched in the metadata.
Need to add a sister LFPInterface in subsequent PR (data on GIN has LFP stream as well as analog inputs).
Example output file: AlphaOmegaRecordingInterface_alphaomega.zip
How to test the behavior?
Test on GIN data added. Also for metadata fetching.
Checklist

[X] Have you thoroughly read our Developer Guide?
[X] Have you ensured the PR description clearly describes the problem and solutions?
[X] Have you checked to ensure that there aren't other open or previously closed Pull Requests for the same change?
[X] If this PR fixes an issue, is the first line of the PR description fix #XXX where XXX is the issue number?
[X] Did you update the CHANGLOG.md file on your branch to describe your changes?

Looks great! Needs an entry to the CHANGELOG.md


Started on e2e tests

Current coverage is 23.40% (diff: 14.28%)

Merging #17 into master will increase coverage by 10.12%

@@             master        #17   diff @@
==========================================
  Files            35         35
  Lines          1695       1705    +10
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            225        399   +174
+ Misses         1454       1267   -187
- Partials         16         39    +23

Powered by Codecov. Last update 0ea1fa9...7611179


ChoiceChip's default "selected" style in dark mode theme is unreadabl…

…e #49984
During the ChipThemeData creation the toggleableActiveColor is now used for the secondaryColor. This should adhere to the Material guide lines (https://material.io/design/interaction/states.html#activated). The following test app was used to test the behavior.
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        brightness: Brightness.light,
      ),
      darkTheme: ThemeData(
        brightness: Brightness.dark,
      ),
      themeMode: ThemeMode.dark,
      home: Scaffold(
        body: Center(
          child: Column(
            mainAxisAlignment: MainAxisAlignment.center,
            crossAxisAlignment: CrossAxisAlignment.center,
            children: <Widget>[
              ChoiceChip(
                label: const Text('Selected'),
                selected: true,
                onSelected: (bool selected) { },
              ),
              ChoiceChip(
                label: const Text('Idle'),
                selected: false,
                onSelected: (bool selected) { },
              ),
              ChoiceChip(
                label: const Text('Disabled'),
                selected: false,
              ),
              Switch(value: false, onChanged: (value) => false),
              Switch(value: true, onChanged: (value) => true)
            ],
          ),
        ),
      ),
    );
  }
}

This results in the following UI:
Light theme:

Dark theme:

Fixes:

https://github.com/flutter/flutter/issues/49984

No changes in the test repository were required.
Pre-launch Checklist

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

No new tests were added, @Hixie I hope this is fine. Also no changes to the documentation were made, I hope this is also fine. If not I will happily add everything required.

After clarification I will add tests for the change.

After clarification I will add tests for the change.

@Hixie added tests, as discussed in Discord.

@Hixie added tests, as discussed in Discord.


Force submissions to leaderboard

In spite of ndcatng that the last submission should be forced to the leaderboard:

It is possible for PARITICIPANTS to OVERWRITE this decision and submit whatever submission they want

This is not the desired behavior. In challenges, it is important to AVOID that participants can hide submissions and make such changes because then the leaderboard jumps around and people are not stimulated to make submssions by seeing the submissions of others.

This is the example competition: https://www.codabench.org/competitions/1145/

Nice catch, thank you for reporting.

@madclam Actually, this feature is working. It is just that, when you are admin of the platform, you can outpass this constraint.


fix(region): 避免从云上同步下来的主机，创建相同配置时network_id为空

这个 PR 实现什么功能/修复什么问题:

[x] 功能、bugfix描述
[x] 冒烟测试
[ ] 单元测试编写

是否需要 backport 到之前的 release 分支:

release/3.5

/area region
/cc @zexi

Codecov Report

Merging #9570 (61c25a5) into master (331e60e) will increase coverage by 10.59%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #9570       +/-   ##
===========================================
+ Coverage    5.91%   16.50%   +10.59%
===========================================
  Files         843       87      -756
  Lines      130850     9120   -121730
===========================================
- Hits         7736     1505     -6231
+ Misses     122413     7483   -114930
+ Partials      701      132      -569

Flag
Coverage Δ

aFlag
16.50% <ø> (+10.59%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/multicloud/aws/loadbalancer.go

pkg/cloudcommon/db/project_resources.go

pkg/compute/models/policy_definition_resource.go

pkg/compute/models/dbinstance_accounts.go

pkg/compute/models/dns_recordset_trafficpolicy.go

pkg/compute/guestdrivers/aws.go

pkg/util/seclib2/passwd.go

pkg/keystone/models/federation_protocol.go

pkg/cloudcommon/db/project.go

pkg/multicloud/aws/dbinstance.go

... and 745 more

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 331e60e...61c25a5. Read the comment docs.


Extend #parents with filter and block processing

KVDAG::Vertex#parents has been extended with the same API for
filtering and block processing as #children was in #8.

If we can be bothered caring at this stage in development: API changes should be accompanied by version bumps.


make txn fee adjustable

when calling + debugging app inner txns, sometimes we make want the outer txn to cover the inner txn fee. irulan does not currently support this.

support editable note field?

FYI can change default txn fee in the account settings modal (press purple account button), but this is more of a temp solution.
perhaps have a more options arrow down carrot next to "Call App" for "Call App w/Options" -> shows fields like start, end rounds, fee, note, etc (other options that we set as reasonable defaults)


⚠ Bofrost.es has degraded performance

In 478b497, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1826 ms

Resolved: Bofrost.es performance has improved in 158c1d2 after 13 minutes.


Spelling fixes

Fixes a typo in the README and makes other spelling corrections in the documentation.

Btw, I used https://github.com/crate-ci/typos to find these spelling mistakes.

But, thank you! With the intentionally misspelled word back where it belongs, I think this is good to go.
bors r+


Toast notifications

@Velua I like the idea of a toast notification system. Maybe we can come up with a system that monitors pending transactions and errors as well.
About sentry, I just checked and sofar no error in that regard has been logged. Will keep an eye on it.
Originally posted by @pingustar in https://github.com/bancorprotocol/webapp/issues/974#issuecomment-776539539

@ashachaf We were wondering about your thoughts toast notifications for certain errors or events to convey to the user when we don't have a dedicated Modal going for it.
If you're keen, can we get a design for it?

@ashachaf if my memory isn't failing me, I believe we have mockups for this somewhere? Are those designs still around?

https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=507%3A1911

@ashachaf The Notifications system has two parts:
Notification Alerts:

Notification History:

A notification currently has the following data:
interface INotificationView {
  id: number;
  status: ENotificationStatus;
  title: string;
  description: string;
  txHash?: string;
  timestamp: Date;
  showSeconds?: number;
  alertOnly?: boolean;
}

we currently have 5 statuses:
enum ENotificationStatus {
  success = "success",
  info = "info",
  warning = "warning",
  error = "error",
  pending = "pending"
}

@pingustar
https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2

https://www.figma.com/file/3yED7o7TgoLwoEjV2EkRLL/Bancor-V2?node-id=2844%3A2143
please note:

notification movement:
fly in - animate the notification as they fly in from the side (right side). on entry, allow 2seconds fade in.
keep notification visible for 7 seconds
fly out - fade the notification out for 3 seconds and in the last second close it to the right (move outside of the view).

menu item movemement:
appear - allow opening from the icon in a top to bottom movement (reference the settings and right side menu icon)
close - same as the settings and right menu

menu item deletion:
when clicking the 'x' on a specific notification, fade it out of view (moving to the right) for 1 second.
clear all items - fade all together in the same format as a single notification 'x'


Scoped CSS identifier is not applied to some html elements

Describe the bug
Scoped CSS generated identifier is not applied to body or header html tags. It is still applied to footer tag though.
To Reproduce

Create a new Asp.net razor pages project using dotnet new razor command.
Edit Pages\_Layout.cshtml.css file to add below code to the top of the file:

body {
  background-color: #dfdfdf;
}

header {
  background-color: #afafaf;
}

footer {
  background-color: #afafaf;
}

Run the project with dotnet run
Observe the opened browser page for the backgound color of page and header, it stays the default of white color but background color of footer changes.
Inspect the HTML using dev tools Elements panel, the generated scoped css identifier is not applied to body or header elements but it is still applied to footer element.
Look at the generated scoped css bundle in Sources panel, it adds the identifier to all three style rules.

Exceptions (if any)
None
Further technical details

Output of dotnet --info

.NET SDK (reflecting any global.json):
 Version:   6.0.101
 Commit:    ef49f6213a

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.19042
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\6.0.101\

Host (useful for support):
  Version: 6.0.1
  Commit:  3a25a7f1cc

.NET SDKs installed:
  5.0.403 [C:\Program Files\dotnet\sdk]
  6.0.101 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 5.0.12 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.1.20 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.12 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.20 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 5.0.12 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.1 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

To install additional .NET runtimes or SDKs:
  https://aka.ms/dotnet-download

The IDE (VS / VS Code/ VS4Mac) you're running on, and its version: For this repro, I did not use any IDE.

Looks like a duplicate of https://github.com/dotnet/razor/issues/9034. CSS scoping doesn't apply to tag helpers, and I think both body and head have tag helpers. If you opt-out these elements from tag helper discovery like <!body>...</!body>, CSS scoping should work.


Fixes to build.gradle for Gradle7.2

Made some changes to build.gradle file to fix bugs in Gradle 7.2.

@jackbayliss I have only tested with 7.2 and 7.3. Not sure if it works with later versions. It might be better to open a new branch to avoid breaking changes for those using 7.0.


[DOCS] Use HTTPS

Reference:

https://httpsiseasy.com
https://letsencrypt.org

Not sure what the point of this issue is tbh.


certbot/www is empty after running the script

Hi,
When running the script all goes well but the challenges are failing with the following error:
### Requesting Let's Encrypt certificate for mysite.io ...
Saving debug log to /var/log/letsencrypt/letsencrypt.log
Plugins selected: Authenticator webroot, Installer None
Obtaining a new certificate
Performing the following challenges:
http-01 challenge for mysite.io
Using the webroot path /var/www/certbot for all unmatched domains.
Waiting for verification...
Challenge failed for domain mysite.io
http-01 challenge for mysite.io
Cleaning up challenges
Some challenges have failed.

IMPORTANT NOTES:
 - The following errors were reported by the server:

   Domain: mysite.io
   Type:   unauthorized
   Detail: Invalid response from
   http://mysite.io/.well-known/acme-challenge/BcjIBO_5jNqGIj3aujUEILwPr916CXY4QCsVI6Mnaf0
   [ip.ip.ip.ip]: "\r\n<title>404 Not
   Found</title>\r\n\r\n
404 Not
   Found
\r\n
nginx\r\n"

   To fix these errors, please make sure that your domain name was
   entered correctly and the DNS A/AAAA record(s) for that domain
   contain(s) the right IP address.

This is my nginx.conf (notice that I am using load balancer):
events {}

http {
  upstream app {
    server app1:8080;
    server app2:8080;
  }

  gzip on;
  gzip_vary on;
  gzip_proxied any;
  gzip_comp_level 6;
  gzip_types text/plain text/css text/xml application/json application/javascript application/rss+xml application/atom+xml image/svg+xml;

  server {
    listen 80;
    server_name mysite.io;
    server_tokens off;

    location /.well-known/acme-challenge/ {
      root /var/www/certbot;
    }

    location / {
      return 301 https://$host$request_uri;
    }
  }

  server {
    listen 443 ssl;
    server_name mysite.io;
    server_tokens off;

    ssl_certificate /etc/letsencrypt/live/mysite.io/fullchain.pem;
    ssl_certificate_key /etc/letsencrypt/live/mysite.io/privkey.pem;
    include /etc/letsencrypt/options-ssl-nginx.conf;
    ssl_dhparam /etc/letsencrypt/ssl-dhparams.pem;

    location = / {
      return 302 /Mysite;
    }

    location / {
      proxy_pass http://app;
      proxy_set_header    Host                $http_host;
      proxy_set_header    X-Real-IP           $remote_addr;
      proxy_set_header    X-Forwarded-For     $proxy_add_x_forwarded_for;
    }
  }
}

When taking a look into /var/www/certbot there are no files there.
Any idea why it happens?

fixed: more details here
https://community.letsencrypt.org/t/challenge-failed-for-domain/129056


Fix problems with copying files and Docker-Compose on Windows

Copying files into the container did not work on Docker for Windows.
Also the docker socket needs to be mounted differently into the docker-compose container on Windows.
UNIX path separator needs to be used when building docker-compose env file path for containerized docker-compose.

Core will now build on Windows.


Errors when model doesn't contain any task

Using default settings (automatic width), some errors appears in console when no task are loaded in the model.

Maybe those got fixed with c24fbf2277422f1425eb9e583c412898a5c5e10b.

I think this was the issue, i'll give it a try again.

It is fixed thanks.


Twitter threads are ordered in the wrong order inside highlights

When syncing a thread, In the readwise highlights section, the threads are synced in the reverse order, which means it's basically unreadable.
The linked reference that appears in the Journal daily page, has sorting which is even weirder, it does not seem to follow any kind of logical pattern of sorting.
Example in the linked image.

Thanks for highlighting this!
Can I check if you are sorting the highlights by recent first or recent last?

On 29 Dec 2021, at 5:44 PM, silver-vlas @.***> wrote:

When syncing a thread, In the readwise highlights section, the threads are synced in the reverse order, which means it's basically unreadable.
The linked reference that appears in the Journal daily page, has sorting which is even weirder, it does not seem to follow any kind of logical pattern of sorting.
Example in the linked image.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Where would this setting be? I have not changed anything from the default neither in Readwise nor in the plugin.

Hmm, which version are you on currently?

On 29 Dec 2021, at 6:47 PM, silver-vlas @.***> wrote:

Where would this setting be? I have not changed anything from the default neither in Readwise nor in the plugin.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

Logseq Version: 0.5.5
Plugin Version: 3.01

Ok - this is my pluging json settings content:
{
"disabled": false,
"sortRecentFirst": true,
"latestRetrieved": "2021-12-29T08:26:13.115593Z",
"token": "kXXKaFJuDZbkP1w4H5KCjJCHLURDSFhlwUFjgOt02WXCvhwtLf",
"retrievedTime": true
}

Hey! I deleted your comment as you included your API token！

Can you uncheck the checkbox "Sort recent highlights first"? Then the recent tweets will be at the bottom.

Yes, that works. Thank you.

For Example:

Hmm I am not sure how Linked References are sorted though. It's not controlled by the plugin as far as I know.

I think perhaps that having the date as a property in every sub-block under "on" may be confusing logseq.
In roam, I had configured Readwise to add the date on the [[Readwise Highlights]] row as a header to the entire paragraph and thus the linked reference would be a single one for the entire thread.

Thanks for the suggestion! Will see if I can add this feature request for a future update!


Added a helper success case for Result<Void, Error> types

This change allows the following:
func foo(result: (Result<Void, Error>) -> ()) {
+  result(.success)
-  result(.success(()))
}

cc @jshier

I'd like to see this addition, as Result<Void, Failure> is fairly common and it feels awkward to keep having to write .success(()). However, I'm pretty sure that additions to the APIs of standard library types have to go through the evolution process. I have my own ideas for additions (tryMap, perhaps fold), so we may be able to combine all of the suggestions into a single pitch and proposal, but I won't have time to do it myself for a bit.

Thanks @jshier, I was hoping I could squeeze this change in but if it has to go through the evolution process then so be it 😄
In the meantime, should I close this PR and start writing a proposal?

Hey @bojanstef,
did you create a new pitch, to improve the Result type? If not, @jshier`s ideas and maybe my idea (https://github.com/apple/swift/pull/27908) could also be included.

Can one of the admins verify this patch?

@bojanstef Any movement here?

@bojanstef Any movement here?

I haven’t done anything.

Please update the base branch to main by Oct 5th otherwise the pull request will be closed automatically.

How to change the base branch: (Link)
More detail about the branch update: (Link)


Alert is not generating on P1 after PODFAILED

Hello Team,
we have configured the Alert rule using your code Podfailephase Alert via ARM template cluster stamp.json  after that we have configured the Alert with email configuration then we forcefully failed one pod  for testing purpose but we are not getting Alert email or any alert notification in portal  from podfailedphase alert rule.

could you please help us to figure it out the missing steps.

@v-fearam -- I think you might be the best person to answer this one. I haven't looked at the alert configuration close enough here to quickly answer.  Anything you might be able to spot from @rambabu0063's question that you can help with?

In the mean time, @rambabu0063 what happens when you execute the query itself?  Any results?  Does anything "look wrong" in the query (e.g. like clusterName not getting setting properly, or a parse error of some kind)
This is the query as defined in the ARM template looks something like the following as defined on the docs @ https://docs.microsoft.com/en-us/azure/azure-monitor/insights/container-insights-log-alerts
let endDateTime = now();
let startDateTime = ago(1h);
let trendBinSize = 1m;
let clusterName = '<your-cluster-name>';
KubePodInventory
    | where TimeGenerated < endDateTime
    | where TimeGenerated >= startDateTime
    | where ClusterName == clusterName
    | distinct ClusterName, TimeGenerated
    | summarize ClusterSnapshotCount = count() by bin(TimeGenerated, trendBinSize), ClusterName
    | join hint.strategy=broadcast (
        KubePodInventory
        | where TimeGenerated < endDateTime
        | where TimeGenerated >= startDateTime
        | distinct ClusterName, Computer, PodUid, TimeGenerated, PodStatus
        | summarize TotalCount = count(),
                    PendingCount = sumif(1, PodStatus =~ 'Pending'),
                    RunningCount = sumif(1, PodStatus =~ 'Running'),
                    SucceededCount = sumif(1, PodStatus =~ 'Succeeded'),
                    FailedCount = sumif(1, PodStatus =~ 'Failed')
                 by ClusterName, bin(TimeGenerated, trendBinSize)
    ) on ClusterName, TimeGenerated
    | extend UnknownCount = TotalCount - PendingCount - RunningCount - SucceededCount - FailedCount
    | project TimeGenerated,
              TotalCount = todouble(TotalCount) / ClusterSnapshotCount,
              PendingCount = todouble(PendingCount) / ClusterSnapshotCount,
              RunningCount = todouble(RunningCount) / ClusterSnapshotCount,
              SucceededCount = todouble(SucceededCount) / ClusterSnapshotCount,
              FailedCount = todouble(FailedCount) / ClusterSnapshotCount,
              UnknownCount = todouble(UnknownCount) / ClusterSnapshotCount
| summarize AggregatedValue = avg(PendingCount) by bin(TimeGenerated, trendBinSize)

Thank you @ckittel
will execute the query and share the result.
could you please validate the pod status.

Hi @ckittel
I have executed the query but did not get any exact log description.

That's a count per minute of whatever is in your last line of the query. The one I copied above was "pending count". What's yours? Also what is the threshold you have set? Does "2 a minute" cross that threshold?

I have used same code as given in  your ARM template

FailedCount, that's good. The threshold might need to be tweaked for your two failures every minute to be picked up. What we included was a sample for you to tweak based on your needs. Try adjusting the threshold to fit/tune to your test conditions.

@rambabu0063 how's is going?

Been about a week.  Going to close this one.  Feel free to reopen if the above guidance didn't help you solve your question.


Recognise windows that are in native-fullscreen when yabai starts

Bug report
Windows that are in native-fullscreen when yabai starts are currently not recognised at all.

This used to work fine. Probably something to do with the AXUnknown change.

Fixed again. This is working fine in 1.1.2 but was broken on master, thus, not adding this to the changelog and whatever.


Splitting / Combining Archives?

Hello,
I was wondering if it's possible to have one archive split into multiple smaller archives? Specifically, what I'm trying to do is serialize a lot of polymorphic pointers, but I'd like every pointer entry to be serialized in its own output file. Then later on when deserializing I'd like to load all those archives, smash them together, and have the pointers re-initialize themselves properly.
The end result would be that if I 1/10000 pointers changes, the whole archive doesn't have to be regenerated again, but rather, just one archive file for that specific pointer would be remade.

You’re likely to have better luck generating some kind of paginated archive
class which contains an array of “pages” each “page” object contains a list
of polymorphic objects.
Have manual load/save methods which call the archive load/save code and
rebuild the pointers after loading.
This isn’t something I think you can expect to be built into cereal itself
though (at least that’s my gut reaction based on your description.)
On Tue, Dec 31, 2024 at 9:13 PM Damir Halilovic @.***>
wrote:

Hello,
I was wondering if it's possible to have one archive split into multiple
smaller archives? Specifically, what I'm trying to do is serialize a lot of
polymorphic pointers, but I'd like every pointer entry to be serialized in
its own output file. Then later on when deserializing I'd like to load all
those archives, smash them together, and have the pointers re-initialize
themselves properly.
The end result would be that if I 1/10000 pointers changes, the whole
archive doesn't have to be regenerated again, but rather, just one archive
file for that specific pointer would be remade.
—
Reply to this email directly, view it on GitHub
https://github.com/USCiLab/cereal/issues/849, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHBPZMPDXPOXQLHPEOH4S32INMMXAVCNFSM6AAAAABUOF3DJCVHI2DSMVQWIX3LMV43ASLTON2WKOZSG43DIOBQG43TOOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Upgrade Gorm models and make database auto-migration optional

Resolves https://github.com/tektoncd/results/issues/311 and
https://github.com/tektoncd/results/issues/289.
What

Sync Gorm models with the most recent schema (previously installed via the
sql-initdb ConfigMap).
Delete the sql-initdb ConfigMap (alter the Postgres StatefulSet to stop
installing the schema on startup).
Make the database auto-migration an opt-out. It is enabled by default because
the Gorm auto-migration is safe (it doesn't attempt to remove unused
fields/tables), but users may disable it via config and install a custom schema
or manage migrations by themselves.

Why
As of this pull request we no longer need to keep models and the DDL script
aligned, in fact, they weren't. Any changes may be done directly in Gorm models
and they'll be reflected automatically in the database.

Reminder: after merging this pr, we can rebase and merge https://github.com/tektoncd/results/pull/310 to bump Gorm and related dependencies.

/assign @adambkaplan

Looks good to me - please squash your commits and I'll add the lgtm.

Note - if we find that size checks are causing things to break, we can always increase them with zero end user impact. We just can't go the other way and reduce column sizes.


feat: align module error

Summary

Test Plan

Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

Current dependencies on/for this PR:

main

PR #4866

PR #4880

PR #4882   👈

This stack of pull requests is managed by Graphite.


🧹 Drama 제거

🧹 Drama 제거
UI
DB

select id from channels c
where category = 'drama';

select * from videos where videos.ChannelId in (
select id from channels c
where category = 'drama'
);

delete from videos where videos.ChannelId in (
select id from channels c
where category = 'drama'
);

delete from channels
where category = 'drama';


Harbor compose authorization fails for passwords over 20 characters

It appears that the maximum allowed password is 20 characters. But my password exceeds that and is valid on Harbor, Turner email, Argonaut, etc.
Should this check allow passwords over 20 characters?
https://github.com/turnerlabs/harbor-compose/blob/27eeb5e8fc811caa4a79e162809b6e64a4e53574/vendor/github.com/turnerlabs/harbor-auth-client/authclient.go#L47
Thanks

This should be fixed.  @nabil-money thanks for submitting and let us know if you have any more issues.

Thanks 👍


Should .py be added to the launcher by default?

https://github.com/chrisjbillington/desktop-app/blob/332ae72e61b554c38f4add03c10aebec6cfb5419/desktop_app/launcher.py#L54-L57
Presently, a test is added to check if the module is a directory.
However, we would like to create a few scripts within a given directory desktop-app could launch.
I think the statement
if is_dir():
     # as before
else:
    script_path = script_path.parent / (script_path.name + '.py')

should work. I'm not sure if you use scripts or only top level apps.

I intended desktop-app to be able to launch any script that can be run with python -m <whatever.whatever>, so it looks like it's just a bug that I didn't append the .py there. I have not sufficiently tested with all kinds of scripts, so I appreciate the bug reports!
I'll add the extension.

I don't think that with_suffix is the correct choice since it will overwrite package.subpackage.actual_script to package.subpackage.py

you are fast at 8am...

I actually didn't realise with_suffix replaces existing suffixes, however, package.subpackage.actual_script should already be split into Path('package/subpackage/actual_script') at that point, so I don't think there can be any dots in the last part. All dots have already been removed:
script_path = Path(package_directory, *module_name.split('.')[1:])

you are fast at 8am...

Am definitely not normally a morning person :p. My cat woke me up so I decided to stay up and watch the SpaceX SN10 test flight. Which got delayed, but anyway.

hmm, you are right. your stuff works by python standards. scripts shouldn't have dots in the names.

Any chance of a tag?
Almost done wrapping out the desktop-app migration on my work and I want to find a good resting place for the entry points 😄

Sure, tag pushed! Will be a new version on PyPI once the release workflow runs (it is very slow).

nice thanks!


Document all $(...) values available in azds.yaml

Is there a list of $(...) place holders that are substituted in azds.yaml during azds up? I am specifically interested in getting the namespace that is used for the deployment.
In my setup I am explicitly assigning namespaces to my resources because my meta chart deploys to multiple namespaces thus I do not relay on the --namespace flag passed to helm install. This does not play well with azds and child spaces. I need to override the namespace from azds.yaml for the azds deployment to target appropriate namespace. Currently I am using $(spacePrefix), $(rootSpacePrefix) and some helm gymnastics to get name of the space but it is quite ugly.

The two you mention are the only ones. This particular placeholder syntax has a single purpose - to allow a dev spaces user to get a public endpoint without needing to know and manually specify the host name to use in the ingress rule.
If you are looking for your code to have access to the namespace in which it was deployed, check out the downward API.

I am new to Helm, of course I can use .Release.Namespace to get the namespace to which the chart is being deployed which is what I was after.


how to stream video in react native , if video located in azure container

Is it necessary for getting stream of mp4 video that needs to be converted any of below format:
Accessing the live stream
RTMP
rtmp://localhost/live/STREAM_NAME
http-flv
http://localhost:8000/live/STREAM_NAME.flv
websocket-flv
ws://localhost:8000/live/STREAM_NAME.flv
HLS
http://localhost:8000/live/STREAM_NAME/index.m3u8
DASH
http://localhost:8000/live/STREAM_NAME/index.mpd

in case of accessing  RTMP format rtmp://localhost/live/STREAM_NAME
I have to do:

ffmpeg -re -i ~/Downloads/temp/Ma\ Long\ Forehand\ and\ Backhand\ Technique.mp4 -c copy -f flv rtmp://<IP_ADDRESS>/live/

but my source is not in localhost or filesystem its in azure container. How to access it.

where to use or convert (mp4 to flv using ffmpeg) if I access file from azure container

To resolve conversion issue (2. point above). I think uploaded video in azure container should be flv/dash/hls format?

Question is how to stream video from azure container( source is already converted).?Is it possible?
if yes please let me know I would like to use node media server instead of Azure media server
I think I should use
Rtsp/Rtmp Relay
NodeMediaServer implement RTSP and RTMP relay with ffmpeg.

Static pull
The static pull mode is executed at service startup and reconnect after failure. It could be a live stream or a file. In theory, it is not limited to RTSP or RTMP protocol.

relay: {
  ffmpeg: '/usr/local/bin/ffmpeg',
  tasks: [
    {
      app: 'cctv',
      mode: 'static',
      edge: 'rtsp://admin:admin888@<IP_ADDRESS>:554/ISAPI/streaming/channels/101',
      name: '0_149_101',
      rtsp_transport : 'tcp' //['udp', 'tcp', 'udp_multicast', 'http']
    }, {
        app: 'iptv',
        mode: 'static',
        edge: 'rtmp://live.hkstv.hk.lxdns.com/live/hks',
        name: 'hks'
      }, {
        app: 'mv',
        mode: 'static',
        edge: '/Volumes/ExtData/Movies/Dancing.Queen-SD.mp4',
        name: 'dq'
      }
  ]
}
Dynamic pull
When the local server receives a play request. If the stream does not exist, pull the stream from the configured edge server to local. When the stream is not played by the client, it automatically disconnects.

relay: {
  ffmpeg: '/usr/local/bin/ffmpeg',
  tasks: [
    {
      app: 'live',
      mode: 'pull',
      edge: 'rtmp://<IP_ADDRESS>',
    }
  ]
}

but not getting any idea how to use this tasks--edge for azure container
if No, then mp4 file(or converted file flv hls or dash) should be in nodemedia server .
Please guide how to proceed.


Idea: JSON schema generation from .xsd

See highsource/jsonix#4.

This is now implemented.
Documentation:

https://github.com/highsource/jsonix-schema-compiler/wiki/JSON-Schema-Generation


Integrating toolkit into samples repo

This is in prep for the upcoming AR samples that will need the toolkit.
The toolkit repo has been added as a submodule. This will help users who use the installed sdk.
For those using cocoapods, a podfile has been added.
Note - We will need to update the podfile and the submodule to point to 100.6 toolkit when it is released.
Currently I am referencing 100.5 version of the toolkit.
@mhdostal @philium

@philium done


ModuleNotFoundError: No module named 'kachery_cloud.TaskBackend' from figurl_jupyter

No this occurs with the new Kachery-cloud (0.4.2) when using the display function from figurl jupyter. Here's the full error stack:
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
Input In [10], in <cell line: 1>()
----> 1 sw.plot_unit_locations(we, backend="sortingview", display=True)

File ~/Documents/codes/spike_sorting/spikeinterface/spikeinterface/src/spikeinterface/widgets/base.py:120, in define_widget_function_from_class.<locals>.widget_func(*args, **kwargs)
    117 @copy_signature(widget_class)
    118 def widget_func(*args, **kwargs):
    119     W = widget_class(*args, **kwargs)
--> 120     W.do_plot(W.backend, **W.backend_kwargs)
    121     return W.plotter

File ~/Documents/codes/spike_sorting/spikeinterface/spikeinterface/src/spikeinterface/widgets/base.py:53, in BaseWidget.do_plot(self, backend, **backend_kwargs)
     51 plotter = self.possible_backends[backend]()
     52 self.check_backend_kwargs(plotter, backend, **backend_kwargs)
---> 53 plotter.do_plot(self.plot_data, **backend_kwargs)
     54 self.plotter = plotter

File ~/Documents/codes/spike_sorting/spikeinterface/spikeinterface/src/spikeinterface/widgets/sortingview/unit_locations.py:40, in UnitLocationsPlotter.do_plot(self, data_plot, **backend_kwargs)
     37 else:
     38     view = v_unit_locations
---> 40 self.handle_display_and_url(view, **backend_kwargs)
     41 return view

File ~/Documents/codes/spike_sorting/spikeinterface/spikeinterface/src/spikeinterface/widgets/sortingview/base_sortingview.py:47, in SortingviewPlotter.handle_display_and_url(self, view, **backend_kwargs)
     45 self.set_view(view)
     46 if self.is_notebook() and backend_kwargs["display"]:
---> 47     display(self.view.jupyter(height=backend_kwargs["height"]))
     48 if backend_kwargs["generate_url"]:
     49     figlabel = backend_kwargs.get("figlabel")

File ~/anaconda3/envs/si/lib/python3.9/site-packages/sortingview/views/View.py:100, in View.jupyter(self, height)
     98 if height is None:
     99     height = self._height
--> 100 import figurl_jupyter as fj
    101 url = self.url(label='jupyter', local=True, electron=False, listen_port=None)
    102 a = _parse_figurl_url(url)

File ~/anaconda3/envs/si/lib/python3.9/site-packages/figurl_jupyter/__init__.py:7, in <module>
      1 #!/usr/bin/env python
      2 # coding: utf-8
      3
      4 # Copyright (c) Jeremy Magland.
      5 # Distributed under the terms of the Modified BSD License.
----> 7 from .FigurlFigure import FigurlFigure
      8 from ._version import __version__, version_info
     10 def _jupyter_labextension_paths():

File ~/anaconda3/envs/si/lib/python3.9/site-packages/figurl_jupyter/FigurlFigure.py:17, in <module>
     15 import kachery_cloud as kcl
     16 from ._frontend import module_name, module_version
---> 17 from kachery_cloud.TaskBackend.TaskBackend import TaskHandler
     18 from kachery_cloud._json_stringify_deterministic import _json_stringify_deterministic
     19 from kachery_cloud._serialize import _serialize

ModuleNotFoundError: No module named 'kachery_cloud.TaskBackend'

Sorry! I stopped supporting jupyter widgets as part of sortingview -- nobody was using it as far as I could see, and it is difficult to maintain with various updates to jupyter. It's possible we could get this to work again by disabling the task backend part of it...

hi Jeremy.
I also think that this is a veryuselfull feature because we can use the visualisation without pushing data which is very very convinient.