Fix #3668: Split the test bridge into a separate artifact.

The scalajs-test-interface artifact actually contained two independent concerns:

The Scala.js version of the sbt.testing.* APIs, and
The JS-side implementation of the test adapter/bridge protocol.

Concrete testing frameworks depend on the former, but need not know about the latter. Moreover, the protocol in the bridge needs to be in sync with the one in the adapter.
Keeping those two concerns together meant that transitive dependencies of testing frameworks could force a version of the bridge that was newer than the adapter. This possibility forced us to lock down the protocol in very serious ways.
This commit splits those two concerns in two different artifacts, so that we can evolve the protocol more freely.
For compatibility, ScalaJSClassLoader and TestUtils are kept in the scalajs-test-interface artifact. They are both already deprecated, so they will be gone in Scala.js 1.x. The classes that were moved to scalajs-test-bridge are package-private, so there is no issue for them.

Do we also need to rename the package? It seems wrong to have the same package in different (non-cross) artifacts.

I just realize that testinterface would only be present in test-bridge modulo the two deprecated things which is probably acceptable. And the artifact containing sbt.testing should be called test-interface to mirror the JVM name.
So maybe we should simply rename the org.sjs.testing.interface package to org.sjs.testing.bridge in the master branch and leave this as is? (modulo the comment update)?

So maybe we should simply rename the org.sjs.testing.interface package to org.sjs.testing.bridge in the master branch and leave this as is? (modulo the comment update)?

Yes, I think that's a good path forward. I've updated the comment.


Add support for DASH patch manifests ()

Have you read the FAQ and checked for duplicate open issues?
Yes
Is your feature request related to a problem? Please describe.

Manifest updates can be expensive in some cases.
Describe the solution you'd like

DASH has now introduced <PatchLocation> and <Patch>, which give a way to update a small part of the existing manifest instead of re-parsing and merging every time.
Additional context

TODO: Get references to the spec on this (AMD.1, landing in 4th edition according to Hulu)
TOTO: Get sample content for this (should be coming soon to the DASH live sim, according to Hulu)

Hi @joeyparrish, Any update on spec and sample content for this one? Thanks.

I was told that this is part of Amendment 1 to the 3rd edition of the DASH spec (which has the official designation "ISO/IEC 23009-1:2014/AMD 1:2015", and is also rolled into the 4th edition of the DASH spec (which has the official designation "ISO/IEC 23009-1:2019".
Sadly, I don't have a publicly available PDF for the spec.  It would be nice if there were a public doc, but ISO will sell you a PDF for 198 Swiss Francs (currently about 203 USD or 182 EUR).  The "preview" that ISO has available only contains the table of contents, and not even the entire table.  :-(

@joeyparrish Sorry for the long delay here, wanted to get you links on the final text and some other aspects kept iterating keeping it from vote, but you can find the final 4th and amd1 over on https://github.com/cta-wave/Resources#specification-development-2

Hi, my buddy and I are checking if we can have DASH patch manifest support based on Shaka or have some preliminary implementation. May I know if you have any plan to start it?

We are extremely interested in this functionality because of the traffic reduction it can bring with large DVR windows. Can you share what your plans are for adding support for it?

We have not started any work on this issue.  It is currently on the Backlog, so we have no plans to implement it for the next major release.  If this is a priority for someone, we'd be happy to review a PR for this.  Since this is a larger feature, you should post a design first (e.g. using a Google doc).

Hi guys @joeyparrish @TheModMaker.
We are currently looking into this as part of an upcoming requirement.
We want to approach this work in a way that would allow us to upstream these changes.
I have made a doc with our proposed implementation.
Your guidance and input would be most welcome.
https://docs.google.com/document/d/1Z3GkKUe9mM4nub1w_WlP3Bva5oNtlywlzRxTuui_Yac/edit?usp=sharing

@technogeek00 Thank you greatly for your input, we are in the process of trying to get an AdapatationSet@id added to our streams. I am in the currently creating a POC that I will present for review once ready.

Just out of curiosity, @dave-nicholas, were you able to make any progress on this?

Hi @zuzzurro, We are currently waiting on some other work from our organisation to be merged before opening a PR with these changes, as the MPD Patch work is dependent on this.

Hi, it may be a bit late.
We have a fork with patch manifest support
https://github.com/harmonicinc-com/shaka-player/commit/1a7389152f9f74c1e7ec56e50e0d8b903a99e80a
Please let me know if you are interested, then we can create a PR.

As I said previously (in 2021, how time flies...) we are extremely interested in this feature. And the sooner we have it in Shaka, the better. How is your implementation different from the one @dave-nicholas is working on, @acheung-harmonicinc ?

@zuzzurro I hope the open this PR by early next week.
Our approach differs from @acheung-harmonicinc 's implementation in that we do not re-parse the entire manifest each time, some of our full events can have a size up to 11 hours and re-parsing a manifest that size on a low - mid end device would not be a solution for us (hence the dash manifest parser improvements were brought into scope for us first). We instead parse and process the patch only.

Well, I totally agree with your goal, and my question was mostly for @acheung-harmonicinc , in order to understand better the pros and cons of both implementations..  Clearly I expect that using patch manifests should bring down the requirements for low end devices. If not, why bother?

What is the status of this issue? It looks like the pull request from @dave-nicholas went nowhere? Inquiring minds want to know..

Hi @zuzzurro , @joeyparrish is on leave at the moment but will review the PR when he returns.

@zuzzurro I don't unfortunately. We are testing Patch internally but don't currently have any public streams yet.

I found out there is an NGINX module that can generate patches: https://github.com/ab2022/mpddiffs Not quite sure on the setup requirements but maybe a possibility for you.


Update index.md

Hi guys!
We've just started using Korge, and this bit in the documentation was a little confusing.
We made the sample compile and added the imports.

Why should there be inports its a reference article.

I'm fine with that, since we are not hiddin them, looks reasonable to me to strip them since the IDE can find the right imports +1
Buts not in ecery other article


Disabled Options

Microsoft PowerToys version
0.68.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
I don't know how this happened, but I noticed that FanZone and Text Extractor and other tools I use were disabled without me having done that! is it a known issue? I didn't find it in the search!
✔ Expected Behavior
No disable tools
❌ Actual Behavior
My FanZOne is disabled as well as text extractor, color picker without me having disabled it!
Other Software
Windows 11 Beta
PowerToysReport_2023-03-06-18-58-38.zip

I experienced this issue too. It happened when upgrading from 0.67 to 0.68.
It is worth noting that during the same session, I did activate FancyZones under version 0.67.
After upgrading to 0.68 all of my activated options had been disabled again (PowerToys Run, Text Extractor etc).
After reactivating them and restarting the PC there were no further issues.

@crutkas

FYI this issue also exists when updating from 0.68 to 0.68.1
PowerToys Run and FancyZones were both disabled during an update.
For future updates I will use the export/import settings under the general heading. However, by default, PowerToys should automatically handle settings transfer.

It would be good to generate a log file, and attach it here with that they can solve this problem faster. Thanks for sharing that you have the same problem!

PowerToysReport_2023-03-18-15-29-11.zip

@stefansjfw can we have someone look at this

@stefansjfw can we have someone look at this

I'll look into it

Sorry for the delay here. I'm actively looking into this now and I'd like to get more info about the behavior here.
IIUC, right after the update is done, during the first run of the newly installed version, most of the features are disabled (even though those were enable before updating). Is that what happens? /needinfo

Yes, that's exactly what happens, but in the last updates I haven't noticed this problem!
If you could look at these two other questions later I would appreciate it!
https://github.com/microsoft/PowerToys/issues/25187
https://github.com/microsoft/PowerToys/issues/24414
Thanks Maison

This has been shipped for 0.70.0. Please updated to the latest release. Thank you.


Monitor Missing/Crashing containers

Hi Stefan,
Need you advise please to understand how to monitor if a container is not running (the reasons could be someone deleted the container, crashed, etc etc).
Eg. I 've a kafka cluster with 3 zookeeper nodes and 3 kafka nodes. I want to be altered if any of the kafka or zookeeper node goes down or is not responding.
Since your setup I can't put additional configs in Prometheus.yml, how can I create such rules with the rules file?

You can add alert rules in https://github.com/stefanprodan/swarmprom/blob/master/prometheus/rules/swarm_task.rules.yml and redeploy.
An alert would look like:
  - alert: kafka
    expr: count(container_memory_rss{container_label_com_docker_swarm_task_name=~".+kafka.+"})
      < 3
    for: 1m
    annotations:
      description: 'Kafka cluster down'
      summary: 'Kafka cluster nodes count < 3'

Hi Stefan,
Thanks for the reply.
Now when I put this conf and redeployed the stack, the count doubled to 6 and I 'm getting duplicate records.
For the same kafka instance I 'm getting two records. So is this method reliable?

container_memory_rss{container_label__base_version="x86_64-0.4.5",container_label__version="",container_label_com_docker_stack_namespace="kafka",container_label_com_docker_swarm_node_id="lftgekvut6idwdb60omqp612i",container_label_com_docker_swarm_service_id="i6ebvyb9zdtvecr2jvipfxubc",container_label_com_docker_swarm_service_name="kafka_kafka0",container_label_com_docker_swarm_task="",container_label_com_docker_swarm_task_id="py5vx3sye1ou9vxdqvelv6m1r",container_label_com_docker_swarm_task_name="kafka_kafka0.1.py5vx3sye1ou9vxdqvelv6m1r",id="/docker/5a781f09efdeb048a573ab4a3a6d14d56eb7261a93e8a86442e909c476397ee0",image="hyperledger/fabric-kafka:latest",instance="<IP_ADDRESS>:8080",job="cadvisor",name="kafka_kafka0.1.py5vx3sye1ou9vxdqvelv6m1r"}
268419072

container_memory_rss{container_label__base_version="x86_64-0.4.5",container_label__version="",container_label_com_docker_stack_namespace="kafka",container_label_com_docker_swarm_node_id="lftgekvut6idwdb60omqp612i",container_label_com_docker_swarm_service_id="i6ebvyb9zdtvecr2jvipfxubc",container_label_com_docker_swarm_service_name="kafka_kafka0",container_label_com_docker_swarm_task="",container_label_com_docker_swarm_task_id="py5vx3sye1ou9vxdqvelv6m1r",container_label_com_docker_swarm_task_name="kafka_kafka0.1.py5vx3sye1ou9vxdqvelv6m1r",id="/docker/5a781f09efdeb048a573ab4a3a6d14d56eb7261a93e8a86442e909c476397ee0",image="hyperledger/fabric-kafka:latest",instance="<IP_ADDRESS>:8080",job="cadvisor",name="kafka_kafka0.1.py5vx3sye1ou9vxdqvelv6m1r"}

It stabilized after a while.


Error Connecting from a Windows machine to a Linux Machine

Are cross OS connections supported? I'm on a Windows PC and trying to connect to a Linux machine that has Peerviewer 1.4 running on both machines. When I try to connect, I get the following error message. The Linux machine is a clean install, so I can't see what user would be listening on that id.

Turns out the firewall was block the request. Once opening the firewall for 5900, everything worked.


Timeout with huge csv import

Hi
I need to import a huge csv (~5 millions rows and ~550MB size file).
Do you know if I can set a timeout?
Either, could you describe the path in DigitalOcean image to import via SQL with Posgresql?
Many thanks

Timeouts are adjustable in #736 (Settings -> Users and Security -> Settings -> HTTP timeout (s) )
However, there are other issues with large CSVs - working on it

CSV import performance is much improved in #736. For small CSV files and for all files when using the SQLite database, performance is improved by reading the file only once instead of three times. For files larger than 1MB when on PostgreSQL we switch to using a streamed COPY for import which is very fast. I tested with a 5M rows file, 250MB and the import time is completely dominated by the upload, the ingestion is almost instantaneous (compared to the upload at least).
There will be many other pain points when you using Saltcorn with large tables, least now we can start finding out what they are!

Many thanks for your feedback @glutamate


Problem reflecting over .NET Framework assemblies

I'm working on a Minimal Web API in .NET 6 that is referencing some .NET Framework assemblies that were written in VB.NET.  The Web API cannot run and the culprit exists within \FastEndpoints\Extensions\EndpointData.cs:
var discoveredTypes = AppDomain.CurrentDomain             .GetAssemblies()             .Union(assemblies)             .Where(a =>                 !a.IsDynamic &&                 !excludes.Any(n => a.FullName!.StartsWith(n)))             .SelectMany(a => a.GetTypes())             .Where(t =>                 !t.IsAbstract &&                 !t.IsInterface &&                 !t.IsGenericType &&                 t.GetInterfaces().Intersect(new[] {                         Types.IEndpoint,                         Types.IValidator,                         Types.IEventHandler,                         Types.ISummary                 }).Any());
When a.GetTypes() is executed, it throws indicating the following:
System.Reflection.ReflectionTypeLoadException: 'Unable to load one or more of the requested types.
Could not load type 'Microsoft.VisualBasic.ApplicationServices.ApplicationBase' from assembly 'Microsoft.VisualBasic, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
Could not load type 'Microsoft.VisualBasic.Devices.Computer' from assembly 'Microsoft.VisualBasic, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
Could not load file or assembly 'System.Configuration.ConfigurationManager, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. The system cannot find the file specified.'
I realize that asking for a fix to address assemblies written in VB.NET on .NET Framework will not be a high priority for you.  That said, I would like to ask for some way to tell FastEndpoints to not reflect over every assembly within the Extension Method called AddFastEndpoints.  Maybe an overload to explicitly provide the assemblies to reflect over?  Maybe an overload to only reflect over the Executing Assembly with the option to provide additional assemblies?
Thank you for your consideration.
Jason

can you give v4.2.1-beta2 a try?
you can now turn off auto scanning and provide the assemblies to be scanned like so:
builder.Services.AddFastEndpoints(new()
{
    DisableAutoDiscovery = true,
    Assemblies = new[]
    {
        typeof(MyNamespace1.MyEndPoint1).Assembly,
        typeof(MyNamespace2.MyEndPoint2).Assembly
    }
});

alternatively you can keep auto discovery on and filter out unwanted assemblies by specifying a predicate like so:
builder.Services.AddFastEndpoints(new()
{
    AssemblyFilter = a => a.FullName!.StartsWith("MyProject")
});

I'll try it now.  Thank you!

alright will look in to that


~10 second timeout awaiting client.shtdown()

Bug description
In a minimal node http handler I'm trying to use posthog to capture an event.  I'm seeing a ~10 second timeout when waiting on the call to shutdown(). At that time I get the error:
ERROR  Error while shutting down PostHog HTTP error while fetching PostHog: 400
at retriable (node_modules/posthog-core/src/index.ts:652:17)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at retriable (node_modules/posthog-core/src/utils.ts:27:19)
at PostHog.fetchWithRetry (node_modules/posthog-core/src/index.ts:635:12)
at PostHog._flush (node_modules/posthog-core/src/index.ts:605:7)
How to reproduce
This is a Nuxt project backend running in node 20.

export default defineEventHandler(async (event) => {
  let posthogClient = null;
  try {
    posthogClient = new PostHog("XXXXX", {
      host: "https://us.i.posthog.com",
    });
    posthogClient.on("error", (err) => {
        console.error("PostHog had an error!", err);
    });
    posthogClient.debug();
    posthogClient.capture({
      distinct_id: "test",
      event: "Test",
    });

  } catch (error) {
    console.error("Error initializing PostHog:", error);
  } finally {
  await posthogClient.shutdown();
}
});

Additional context
I'm not to posthog, and a hack front end person, I'm probably doing something silly.  I'm getting front end events through just fine using the same api key, but none of my backend events show up, and get a 400 instead.

Oh... I may have found it...  distinctId is different than distinct_id.  Changing that seems to make the difference.
Chalking this one up to new-userism... might want a documentation change at https://posthog.com/docs/libraries/node.

I think we should perhaps do some more SDK side alerting if a call happens without a distinctId. Most people seem to use Typescript which indicates this immediately but we may need to rely on that less...


Adds unzip module

Allows downloading and extracting zips from urls
Closes #50
Needed for out of repo maps

Proof in pudding

@tgstation/rust-g-dev-team meep


[QUESTION]

Question
Hi,
I've followed the deployment steps for setting up authentication within the conf.yml file.  Unfortunately, the username and password fields never prompt and the dashboard always immediately loads.
Any assistance would be very much appreciated.
Thanks!
Here's what t he top of my conf.yml file looks like:
`appConfig:
layout: auto
iconSize: medium
theme: material-dark-original
auth:
users:
- user: test
hash: 9b8sdfs34hd4567fl690820f56b31ac48807f9bf5c.....
type: admin
pageInfo:
title: Life Amongst The Ferns
description: Welcome Home Josh!
navLinks:
- title: GitHub
path: https://github.com/Lissy93/dashy
target: newtab
- title: Documentation
path: https://dashy.to/docs
target: newtab
footerText: ''`
Category
Configuration
Please tick the boxes

 You are using a supported version of Dashy (check the first two digits of the version number)
 You've checked that this question hasn't already been raised
 You've checked the docs and troubleshooting guide
 You agree to the code of conduct

    GitHub - Lissy93/dashy: 🚀 A self-hostable personal dashboard built for you. Includes status-checking, widgets, themes, icon packs, a UI editor and tons more!   🚀 A self-hostable personal dashboard built for you. Includes status-checking, widgets, themes, icon packs, a UI editor and tons more! - GitHub - Lissy93/dashy: 🚀 A self-hostable personal dashboard ...    Dashy | Dashy   Documentation | Dashy, a self-hosted dashboard for your homelab

Samething here, running with docker. works with 2.1.0, which is pretty old.

It's good to know I'm not the only one. :)

Likely duplicate of https://github.com/Lissy93/dashy/issues/990

I'm going to close as a dup of #990, follow that ticket instead for updates


Setup Firebase Authentication

[ ] Add Signup Function
[ ] Add logout function
[x] Create authentication provider
[ ] Setup login persistence with firebase auth

I think I am going to kill the fully authenticated experience for the hackathon's sake, its taking too much of my time.


Charts[Network]: upgrade Mirror Node chart to latest version

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


[master] Fix #1328 Dissociates stats context

test this please

Is it possible to add tests around the clustered store to test whether context objects are associated/dissociated properly?


Refactor TestUserRoutes Status Codes and Refactor API Response Structures into Separate Module

This PR addresses the need to make our codebase more maintainable by extracting response structures from the routes.py and placing them in a new, dedicated responses.py file and also addresses the issue of hardcoded status codes in the TestUserRoutes class.  This change not only enhances readability but also ensures that we're leveraging FastAPI's built-in utilities for more maintainable and self-explanatory code.
Key Changes:

Introduced responses.py to house centralized response structures.
Updated endpoints in routes.py to utilize the response structures from the new module.
Enhanced overall readability of the routes.py by reducing inline content.

Great work!
With this structure, the services are more clear reducing the code in router.
Please, can you run the pre-commit hooks? It is not passing the pipeline (if not, I will run it later).
Thanks!

resolved

Good job @doziestar
Thanks!


http/bind: rename gl to accounts

Issue: https://github.com/moov-io/accounts/issues/44

Codecov Report

Merging #44 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   93.13%   93.13%
=======================================
  Files          12       12
  Lines         466      466
=======================================
  Hits          434      434
  Misses         25       25
  Partials        7        7

Impacted Files
Coverage Δ

http/bind/bind.go
84.21% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[REQUEST]

What would you like to have implemented?
Allowing a lambda to call executable path multiple times.
Could we check if the compressed files were already decompressed and then if so return the path if not inflate them and return the path ?

Why would it be useful?
AWS is reusing some hot instance to be faster, this logic when calling "executablePath" multiple time throw the infamous ETXTBSY error.

The workaround we found, by luck, is to use some in-memory cache to store the result of executablePath and to reuse it if present in cache.
I think it would be better to handle this out-of-the-box.
@alixaxel what are your thoughts on this ?

yes it could be very useful to have this out of the box !!


test on all channels, deploy only on stable

Testing the beta channel landed in #15

Nightly channel landed in #17


Smaller .nwb & .mat files for test harness.

Adding two files. Both were formed by:

Trimming the first 1000 frames from a save_data_AVG.mat file exported from daance.
Extracting and saving only the pred (prediction) data, which is the only data that stac-mjx currently uses.

Files are saved as .nwb and .mat files respectively. A new directory test_data was created in the root directory. It's worth noting that the .nwb file is roughly twice as large as the .mat file. This is due to extra "confidence" data required by the .nwb format, as well as addtional overhead.

Test data should be organized like this relative to the repo root:
tests/
  data/
    [binary files go in here]
  fixtures/
    [.py files defining the fixture objects go here]
  test_*.py  # module-level test files go here

Fixed.


[Bug] Cannot reference instances defined in another fixture yml file

When you have some base fixtures in base_setup.yml and you want to reference to one of the instances in there from another fixture file, lets say products.yml, you get:
UnexpectedValueException: Instance workspace_1 is not defined

I think this is caused by that the loadFixturesFromFile instantiates a new Alice Loader every time you load a fixture file (see: https://github.com/Lakion/ApiTestCase/blob/master/src/ApiTestCase.php#L279).
@pjedrzejewski Would it be a good solution if you instantiate the Alice Loader in the setup method, so that the fixture references are preserved when you load multiple fixture files in a single test?

I have submitted a PR (#46) for this issue.

Thanks for fixing this!


Action is running Eslint in folders not specifed

I am using this action on a mono repo. Both the client and server run from their various places in /packages. Each have Eslint inside of them. While trying to debug some code I decided to comment out the action Lint API package to avoid an eslint error that I did not want to deal with at the momment. But when I disabled the first action the second action (which should be running in ./packages/ui/.eslintrc.js) errored on an issue in ./packages/api/
Below are the 2 actions.
## - name: Lint API package
##  uses: sibiraj-s/action-eslint@v2
##   with:
##     github-token: ${{ secrets.GITHUB_TOKEN }} # Optional
##     eslint-args: '--config ./packages/api/.eslintrc.js --ignore-path=.gitignore --quiet'
##     extensions: 'js,mjs'
##     annotations: true
- name: Lint UI package
   uses: sibiraj-s/action-eslint@v2
   with:
     github-token: ${{ secrets.GITHUB_TOKEN }} # Optional
     eslint-args: '--config ./packages/ui/.eslintrc.js --ignore-path=.gitignore --quiet'
     extensions: 'js,mjs,jsx,ts,tsx,vue'
     annotations: true

That might happen because of your eslint config. Providing a config file in certain folder doesn’t mean that items outside that folder are excluded.
In your case, eslint is installed in the root directory and your config exists in ui directory. When linted, Eslint will run on all possible items with the provided config.
You may want to provide the source directory as well in you eslint-args.
See https://eslint.org/docs/latest/user-guide/command-line-interface#run-the-cli
You might want to do something like
eslint "package/ui/**” -c ./packages/ui/.eslintrc.js

Hope that helps.


[ROCm][XLA:GPU] support for device math calls

Support for rocm device calls for math
Provide APIs so both Rocm and PTX can be supported

@thomasjoerg  Thanks for the review! I have incorporated the suggestions.


unexpected schema field name conflicts in joins

Describe the bug
A clear and concise description of what the bug is.
with a join, we'd get false positive schema name collision
To Reproduce
Steps to reproduce the behavior:
this works:
SELECT
  t1.c9 x
FROM test t1
INNER JOIN test t2
ON t1.c9 = t2.c9
ORDER BY x;

but this won't:
SELECT
  t1.c9 x, t1.c1 c1
FROM test t1
INNER JOIN test t2
ON t1.c9 = t2.c9
ORDER BY x;

Plan("The left schema and the right schema have the following columns with the same name without being on the ON statement: {Column { name: \"c1\", index: 0 }}. Consider aliasing them.")

Expected behavior
A clear and concise description of what you expected to happen.
it shall work as expected (e.g. as in Postgres)
Additional context
Add any other context about the problem here.

I think this might have something to do with our projection push down logic:
> SELECT
  t1.c9 x, t1.c1 c1
FROM test t1
INNER JOIN test t2
ON t1.c9 = t2.c9
ORDER BY x;
[datafusion/src/execution/dataframe_impl.rs:148] &plan = Sort: #x ASC NULLS FIRST
  Projection: #t1.c9 AS x, #t1.c1 AS c1
    Join: #t1.c9 = #t2.c9
      TableScan: t1 projection=Some([0, 1])
      TableScan: t2 projection=Some([0, 1])

Noticed c1 got pushed down to t2 table scan as well even though it's not being projected. Most likely the projection push down logic is not distinguishing between column names and aliases.

Issue seems resolved now as of latest master:
DataFusion CLI v16.0.0
❯ CREATE EXTERNAL TABLE test STORED AS CSV WITH HEADER ROW LOCATION 'test.csv';
0 rows in set. Query took 0.011 seconds.
❯ SELECT
  t1.c9 x
FROM test t1
INNER JOIN test t2
ON t1.c9 = t2.c9
ORDER BY x;
+---+
| x |
+---+
| 2 |
| 3 |
| 4 |
| 5 |
+---+
4 rows in set. Query took 0.013 seconds.
❯ SELECT
  t1.c9 x, t1.c1 c1
FROM test t1
INNER JOIN test t2
ON t1.c9 = t2.c9
ORDER BY x;
+---+----+
| x | c1 |
+---+----+
| 2 | 1  |
| 3 | 2  |
| 4 | 3  |
| 5 | 4  |
+---+----+
4 rows in set. Query took 0.013 seconds.
❯ explain SELECT
  t1.c9 x, t1.c1 c1
FROM test t1
INNER JOIN test t2
ON t1.c9 = t2.c9
ORDER BY x;
+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------+
| plan_type     | plan                                                                                |
+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------+
| logical_plan  | Sort: x ASC NULLS LAST                                                                                |
|               |   Projection: t1.c9 AS x, t1.c1 AS c1                                                                                |
|               |     Inner Join: t1.c9 = t2.c9                                                                                |
|               |       SubqueryAlias: t1                                                                                |
|               |         TableScan: test projection=[c1, c9]                                                                                |
|               |       SubqueryAlias: t2                                                                                |
|               |         TableScan: test projection=[c9]                                                                                |
| physical_plan | SortExec: [x@0 ASC NULLS LAST]                                                                                |
|               |   CoalescePartitionsExec                                                                                |
|               |     ProjectionExec: expr=[c9@1 as x, c1@0 as c1]                                                                                |
|               |       CoalesceBatchesExec: target_batch_size=8192                                                                                |
|               |         HashJoinExec: mode=Partitioned, join_type=Inner, on=[(Column { name: "c9", index: 1 }, Column { name: "c9", index: 0 })]                           |
|               |           CoalesceBatchesExec: target_batch_size=8192                                                                                |
|               |             RepartitionExec: partitioning=Hash([Column { name: "c9", index: 1 }], 12), input_partitions=12                                                 |
|               |               RepartitionExec: partitioning=RoundRobinBatch(12), input_partitions=1                                                                        |
|               |                 CsvExec: files={1 group: [[home/jeffrey/Code/arrow-datafusion/datafusion-cli/test.csv]]}, has_header=true, limit=None, projection=[c1, c9] |
|               |           CoalesceBatchesExec: target_batch_size=8192                                                                                |
|               |             RepartitionExec: partitioning=Hash([Column { name: "c9", index: 0 }], 12), input_partitions=12                                                 |
|               |               RepartitionExec: partitioning=RoundRobinBatch(12), input_partitions=1                                                                        |
|               |                 CsvExec: files={1 group: [[home/jeffrey/Code/arrow-datafusion/datafusion-cli/test.csv]]}, has_header=true, limit=None, projection=[c9]     |
|               |                                                                                |
+---------------+------------------------------------------------------------------------------------------------------------------------------------------------------------+
2 rows in set. Query took 0.010 seconds.
❯

@alamb resolved, can be closed

Thanks @yukkit


Added logging for low balances

When the balance of one of the challenge addresses reaches 0.001, tracker client fails to start without a useful error. See https://gist.github.com/rmeleromira/3ff67ad3980fda0bdfa54bdf2ae2205e for a detail of the error

There is already logging for low balance in the getStats method.
Additionally the tracking server will send a low balance message to the node.

Allan,
I have a specific non-start case for node tracker that this code fixes. I'm sure other people are running into the same issue.
The node is inactive
Cert Valid: false    not found

State: inactive

balance shows greater than .002
root@sec38:~# zen-cli z_gettotalbalance
{
  "transparent": "0.00",
  "private": "0.002",
  "total": "0.002"
}

individual addresses show a balance of .001
root@sec38:~# zen-cli z_listaddresses
[
  "zcGuPvKd***",
  "zcfYwEATx3x***"
]
root@sec38:~# zen-cli z_getbalance zcGuPvKd***
0.00100000
root@sec38:~# zen-cli z_getbalance zcfYwEATx3x***
0.00100000

working & synced zend instance
root@sec38:~# zen-cli getinfo
{
  "version": 2001450,
  "protocolversion": 170002,
  "walletversion": 60000,
  "balance": 0.00000000,
  "blocks": 372787,
  "timeoffset": 0,
  "connections": 10,
  "proxy": "",
  "difficulty": 2242025.59623498,
  "testnet": false,
  "keypoololdest":<PHONE_NUMBER>,
  "keypoolsize": 101,
  "paytxfee": 0.00000000,
  "relayfee": 0.00000100,
  "errors": ""
}

nodetracker fails on start with "Unable to get z-addr balance"
root@sec38:~# systemctl restart nodetracker;journalctl -f
-- Logs begin at Thu 2018-09-06 16:42:32 UTC. --
Sep 06 16:57:09 sec38 zend[287]: receive version message: /hnodes:0.2/: version 170002, blocks=372785, us=[****]:9033, peer=17
Sep 06 16:57:37 sec38 zend[287]: ERROR: AcceptToMemoryPool: joinsplit requirements not met
Sep 06 16:57:37 sec38 systemd[1]: Failed to reset devices.list on /system.slice/systemd-tmpfiles-clean.service: Operation not permitted
Sep 06 16:57:37 sec38 systemd[1]: Starting Cleanup of Temporary Directories...
Sep 06 16:57:37 sec38 systemd-tmpfiles[381]: [/usr/lib/tmpfiles.d/var.conf:14] Duplicate line for path "/var/log", ignoring.
Sep 06 16:57:38 sec38 systemd[1]: Started Cleanup of Temporary Directories.
Sep 06 16:59:44 sec38 systemd[1]: Stopping ZenCash Node Tracker...
Sep 06 16:59:44 sec38 systemd[1]: Stopped ZenCash Node Tracker.
Sep 06 16:59:44 sec38 systemd[1]: Failed to reset devices.list on /system.slice/nodetracker.service: Operation not permitted
Sep 06 16:59:44 sec38 systemd[1]: Started ZenCash Node Tracker.
Sep 06 16:59:44 sec38 node[390]: Retrieving zen rpc config....
Sep 06 16:59:44 sec38 node[390]: You setup ipv6 connectivity. We need to apply a workaround for dns resolution.
 Initializing...
Sep 06 16:59:44 sec38 node[390]: Node t_address (not for stake)=****
Sep 06 16:59:44 sec38 node[390]: Checking private z-addresses...
Sep 06 16:59:44 sec38 zend[287]: keypool reserve 2
Sep 06 16:59:44 sec38 zend[287]: keypool return 2
Sep 06 16:59:44 sec38 node[390]: Unable to get z-addr balance

nodetracker fails with added logging that is clear about the problem
root@sec38:~# systemctl restart nodetracker;journalctl -f
-- Logs begin at Thu 2018-09-06 16:42:32 UTC. --
Sep 06 17:03:51 sec38 systemd[1]: Stopping ZenCash Node Tracker...
Sep 06 17:03:51 sec38 systemd[1]: Stopped ZenCash Node Tracker.
Sep 06 17:03:51 sec38 systemd[1]: Failed to reset devices.list on /system.slice/nodetracker.service: Operation not permitted
Sep 06 17:03:51 sec38 systemd[1]: Started ZenCash Node Tracker.
Sep 06 17:03:51 sec38 node[404]: Retrieving zen rpc config....
Sep 06 17:03:51 sec38 node[404]: You setup ipv6 connectivity. We need to apply a workaround for dns resolution.
 Initializing...
Sep 06 17:03:51 sec38 node[404]: Node t_address (not for stake)=****
Sep 06 17:03:51 sec38 node[404]: Checking private z-addresses...
Sep 06 17:03:51 sec38 zend[287]: keypool reserve 2
Sep 06 17:03:51 sec38 zend[287]: keypool return 2
Sep 06 17:03:51 sec38 node[404]: Balance of 0.001 for address zcGuPvKd**** must be greater than 0.001
Sep 06 17:03:51 sec38 node[404]: Balance of 0.001 for address zcfYwEATx3x**** must be greater than 0.001
Sep 06 17:03:51 sec38 node[404]: Unable to get z-addr balance

Seems like the getStats method doesn't catch this.


New metadata filters

Adds new metadata filters to OCP page
Includes:

ipsec
fips
encryption
Cluster Control Plane publish state
Compute nodes architecture
Control Plane architecture

Includes a separation of AWS jobs into three categories:

AWS
AWS ROSA
AWS ROSA-HCP

Includes fixes from PR #54, currently has a conflict, it is easier to include them here @dustinblack

remove underlines from list item keys
bonus: Fix backend container build for linux/aarch64

Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] Optimization
[ ] Documentation Update

Description

Related Tickets & Documents

Related Issue #
Closes #54

Checklist before requesting a review

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

Testing

Please describe the System Under Test.
Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

Added one typo fix and a format fix.


Unable to retrieve the JDK

I'm having trouble using this buildpack. Getting the error:

gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
!     ERROR: Unable to retrieve the JDK.

After digging in to debug, I found that Amazon S3 is returning an error:
~ $ curl -I https://lang-jvm.s3.amazonaws.com/jdk/heroku-16/openjdk1.8.0_144.tar.gz
HTTP/1.1 503 Slow Down
x-amz-request-id: 53A8D1C8EF7ED72D
x-amz-id-2: a0iLlzPzpzkM4apvjxQ8g4wRsSBA0Q85SJnUpITmr8yos6rUaqVDeLAgg/YGGMZhGh1iOS0388o=
Content-Type: application/xml
Transfer-Encoding: chunked
Date: Thu, 14 Sep 2017 19:35:25 GMT
Connection: close
Server: AmazonS3

May be related to: https://status.heroku.com/incidents/1285

Was due to the AWS S3 incident.


allow annotation processor to run via an other apt

Hi,
This change indicates to the java compiler that current annotation processor has altering files and he should take care about that.
This is very usefull when you want to use lombok with another apt (like http://mapstruct.org/)

Hi, is this usefull for Android JACK compiler? @jBouyoud

In cec8705c3b9fa6488d53bec5bb4eab27a6decf85 I've added the ClaimingProcessor that should take care of this. It is in lombok 1.16.10. Isn't that enough?

No sorry, it doesn't work.
I suggest to change lombok.core.AnnotationProcessor#process. This method always returns false, returning true allow to use lombok with another annotaitons processors (like mapstrcut).

@jBouyoud .Yes,i have got the error by using the last vesion one

The main processor triggers on all annotations, even non-lombok ones (there's reasons for that). If we return true, we'd be claiming all annotations, of all stripes. That sounds like it'll cause more problems than it'll solve.
According to the API documentation we should only return true if we've handled the annotation.

Note that if a processor supports "*" and returns true, all annotations are claimed. Therefore, a universal processor being used to, for example, implement additional validity checks should return false so as to not prevent other such checkers from being able to run.

I think that's the point.
Why the main lombok annotations processor is trigger onto all ()
annotations ?
Naively, i would have restricted the main processor should be triggered
onto 'lombok.' annotations and then claim processed annotations.
Le mar. 22 nov. 2016 à 01:05, Roel Spilker<EMAIL_ADDRESS>a
écrit :

The main processor triggers on all annotations, even non-lombok ones
(there's reasons for that). If we return true, we'd be claiming all
annotations, of all stripes. That sounds like it'll cause more problems
than it'll solve.
According to the API documentation
https://docs.oracle.com/javase/8/docs/api/javax/annotation/processing/Processor.html#process-java.util.Set-javax.annotation.processing.RoundEnvironment-
we should only return true if we've handled the annotation.
Note that if a processor supports "*" and returns true, all annotations
are claimed. Therefore, a universal processor being used to, for example,
implement additional validity checks should return false so as to not
prevent other such checkers from being able to run.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rzwitserloot/lombok/pull/1166#issuecomment-262107559,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABRk5MtoXN8yv5nhH1Mqqz4TeA31-uLbks5rAjHNgaJpZM4JZqBM
.

We need to process * because we not only use annotations. For instance val, and more recently var can be in any file.

I would be willing to abandon the "val" feature in order to get just @Data-related stuff and interoperability with mapstruct/vertx/checker framework /other major annotation processors out there

We are close to making a fix for the interoparability with at least mapstruct.

We expect to be able to work with mapstruct after today. The techniques used could also be applied by other annotation processors.


examples not working

Hi thanks for feathers, first of all, just discovered it a few days ago. great work.
I'm trying to get hooks working, but no luck for me.
I cloned the rep and tried to run the examples, but i always get:
var todoService = app.lookup('todos');
TypeError: undefined is not a function

any idea what could be happening there? I'm on ubuntu 14.04 and node 0.12.7.

Thanks for reporting! You are right, the example indeed still ran on Feathers pre 1.0. The correct way to retrieve the service is app.service('todos'). I updated both examples in https://github.com/feathersjs/feathers-hooks/commit/6f13d13cfbeb3902ee3f6fbf3c93005433758f3d.

that was quick. thanks. there are still 3 occurencies of app.lookup in the readme file. please change them too.

I thought I changed that. Would you mind submitting the change as pull request? You can change it directly at https://github.com/feathersjs/feathers-hooks/edit/master/README.md

@daffl your right, it' s changed.  i must have look at a cached page. all good :)


Fix optimize cli

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

v2.7.1
GitHub release

Your semantic-release bot :package::rocket:


Update keras-nlp landing page for 0.4 release

Closes https://github.com/keras-team/keras-nlp/issues/502.

Cannot commit until #1159 is merged.

I'm merging, I won't actually update the live website until the whole nlp subsite is ready though.


Output sound lags

Currently, the output of vsnd lags (the out.wav generated by script/verify.sh) compared to the input sound (CantinaBand3.wav).

After some experiments, I find that if we set the output sink from Settings -> Sound then play sounds will result in lagged sound outputs.

After some experiments, I find that if we set the output sink from Settings -> Sound then play sounds will result in lagged sound outputs.

Can you disable PulseAudio and verify the functionality of vsnd?
I always remove this package during the development.

Hi @jserv ,
Thanks for your help!
After disabling PluseAudio, the output sound does not lag anymore.


Attempt to resolve object store deadlocks

@bdash @tgoyne
Fixes #178
This PR does two things:

The session's state lock is never held through the duration of a call to wait_for_upload_complete_or_client_stopped() or wait_for_download_complete_or_client_stopped(). This prevents certain deadlocks from happening: for example, if the error handler is invoked while the binding is waiting (and the lock is being held), it will attempt to move the state to error but will never be able to acquire the lock, and the blocking call won't return and relinquish the lock because the error handler can't return.

Any time those two APIs are called the lifetime of the sync::Session is now guaranteed until the call returns.

Maybe there's a better way to do this? Ideas welcome...

I think change would make it possible for the object store to create two sync::Sessions that are working with the same Realm file, which isn't permitted.

Yeah, I forgot about that. I'll have to think about it some more...

I won't be able to test this update until I fix my broken environment, but I wanted to see if this alternate approach was worth pursuing.

I think this can be reviewed. The binding and object store tests that exist pass, and manual inspection suggests that the tests are moving the state into inactive appropriately now.

Would we still like to purse this approach? I'm happy to make any other changes necessary to get this approved and/or verify its functionality.

I rebased this change onto my latest work, and re-enabled the test Mark stubbed out.

Should this be targeting the master branch now?

@jpsim yeah, it should.

I'm going to resurrect this PR and fix the merge conflicts, because I think it provides a workaround for a couple of issues that we're seeing WRT resetting sync state for tests, and bugs in sync::Session/sync::Client's handling of threads waiting on uploads/downloads. More details later...

Since this PR deals exclusively with the blocking wait-for-X methods, it seems like https://github.com/realm/realm-sync/pull/983 supersedes it.


Remove a TODO related to ISPN-5353 that was fixed by ISPN-9127

Please ignore the t_ISPN-8003_some_more branch name. This has nothing to do with 8003.

Integrated, thanks Adrian!


Default location changes calculation for Avg Resident vaccinated/unvaccinated when it should not

Describe the bug
In the spreadsheet, the result of Activity Log and Custom Person calculations change based on default location in a manner which is incorrect.
This occurs when:

The location is provided explicitly in the log, overriding the default location
Either Avg Resident (vaccinated) or Avg Resident (unvaccinated) is used as a risk profile.

Link to microCOVID scenario
https://docs.google.com/spreadsheets/d/1ig1GNSmUc_dnTqfj2xOdCbwaSk5ePFwjXvkg4P5-ups/edit#gid=311635422
To Reproduce
Steps to reproduce on the spreadsheet linked:

Work with a copy of the sheet, review the activity log and custom person sheet, see that all interactions are coded as occurring in King County.
Copy the values of all calculated "Risk to self" and "risk to others" values to a text file or scratch space.
Go to the locations sheet and switch the default location to Snohomish, Washington using the check box in B7
Wait for recalculation
Review the activity log and custom person sheet. Total RtO and RtS numbers will change for every person, as well as the line-calculated values for "Avg Resident (vaccinated)" and "Avg Resident (unvaccinated)". Lines with "Avg Resident" display correct behavior

Expected behavior
If a location is specified explicitly, the default location should not affect the result of the calculation. This is how "Avg Resident" works and is how most people would expect "Avg Resident (vaccinated)" and "Avg Resident (unvaccinated)" to work.
Is this a regression?
I have no idea if this is technically a regresion; however, given how many people using this spreadsheet are vaccinated, have vaccinated friends and live near a county line... it seems like something that should work. Also Avg Resident has always worked this way.

I am fairly certain the issue is in how the E column is calculated on the log sheets.
There is first a lookup of a default-location person record, which is then multiplied by a ratio to adjust to the correct location if the line has a non-default location in column U.
This ratio always uses the prevalence ratio for an Average person (not vaccinated, not unvaccinated). In situations where there are vastly different case rates or levels of vaccination, this ratio ought to be calculated specifically for the vaccinated or unvaccinated populations depending on the setting of column T.
=IF(T6="","",
IF(ISNUMBER(T6), T6,
MAX(
  ARBITRARY_MIN_PERSON_RISK,
  VLOOKUP(T6, PEOPLE_TABLE, 2, FALSE)
  *IF(OR(U6="", VLOOKUP(T6, PEOPLE_TABLE, 3, FALSE)),
      1 + N("Correct for non-default locations"),
      VLOOKUP(U6, LOCATION_TABLE, 2, FALSE) /PREVALENCE_LOCAL)
  /MAX(N("Remove delay factor for past weeks"),
       IF(AND(ISDATE(Q6), Q6<=LOCATION_DATA_DATE_LAST_UPDATED), VLOOKUP(IF(ISBLANK(U6), PREVALENCE_LOCAL_NAME, U6), LOCATION_TABLE_COMPLETE, 8, FALSE), 1),
       1)
  *IF(V6,SYMPTOMS_NONE_NOW,1))
))

After some experimentation I believe the part of column E's formula which reads
VLOOKUP(U6, LOCATION_TABLE, 2, FALSE) /PREVALENCE_LOCAL
should read
if(T6="Avg local Resident (unvaccinated)", VLOOKUP(U6, LOCATION_TABLE, 3, FALSE)/'📍 Locations'!$O$3,
if(T6="Avg local Resident (vaccinated)", VLOOKUP(U6, LOCATION_TABLE, 4, FALSE)/'📍 Locations'!$P$3,
    VLOOKUP(U6, LOCATION_TABLE, 2, FALSE)/PREVALENCE_LOCAL)

Hopefully this helps.


Type Aliases doesn't act as handbook said

TypeScript Version:  3.9.2

Search Terms:
Interfaces vs. Type Aliases
Expected behavior:
As handbook says, hovering over aliased should show returns a object literal type
Actual behavior:
It showed function aliased(arg: Alias): Alias as well as hovering interfaced did.

Related Issues:
Code
type Alias = { num: number }
interface Interface {
    num: number;
}
declare function aliased(arg: Alias): Alias;
declare function interfaced(arg: Interface): Interface;

Output
"use strict";

Compiler Options
{
  "compilerOptions": {
    "noImplicitAny": true,
    "strictNullChecks": true,
    "strictFunctionTypes": true,
    "strictPropertyInitialization": true,
    "strictBindCallApply": true,
    "noImplicitThis": true,
    "noImplicitReturns": true,
    "useDefineForClassFields": false,
    "alwaysStrict": true,
    "allowUnreachableCode": false,
    "allowUnusedLabels": false,
    "downlevelIteration": false,
    "noEmitHelpers": false,
    "noLib": false,
    "noStrictGenericChecks": false,
    "noUnusedLocals": false,
    "noUnusedParameters": false,
    "esModuleInterop": true,
    "preserveConstEnums": false,
    "removeComments": false,
    "skipLibCheck": false,
    "checkJs": false,
    "allowJs": false,
    "declaration": true,
    "experimentalDecorators": false,
    "emitDecoratorMetadata": false,
    "target": "ES2017",
    "module": "ESNext"
  }
}

Playground Link: Provided

Which page did you read this on?

The hover info will only show the literal when you hover over the type specifically, not everywhere that uses it.
I assume it is because it would be harder to read the function signature.

@RyanCavanaugh
Got this at Interfaces vs. Type Aliases
Thx 🙏

The hover info will only show the literal when you hover over the type specifically, not everywhere that uses it.
I assume it is because it would be harder to read the function signature.

Info from hovering on type itself is as expected, so maybe the handbook got some ”errata“？

This page has been rewritten


DbUpMigration - Migrate to DbUp 4.0

Perhaps it is enough to point against NuGet package dbup-sqlserver.

Good idea. I'll try it too

I have released a 2.* version of the task where your contribution is included. Thanks again!


[QUESTION] Expose underlaying pino logger instance

Hi! Do you think that it would be possible to expose the underlying pino instance so that we can use the convenient syntax to print objects and other variables?

Hi, if I understand correctly, you don't need to get underlying logger, you can just add params after context argument, look here:
https://github.com/iamolegga/nestjs-pino/blob/master/src/index.ts#L66
Is this the correct answer?

Closing this for now, feel free to reopen.
Also, I think you could be interesting in https://github.com/iamolegga/nestjs-pino/issues/22


Container creation failure handling

This is a simple PoC of how we could handle container creation. It's tested in integration, so it requires integration tests to pass ( #87 does just that ). So please, close your eyes on any commit that is NOT 673845c in this PR :)
What do you think ? Would that approach be fine ? Perhaps we could have have unit tests in addition to integration tests too ?

Rebasing and re-opening.


Dependencies of initrd-fstab entries are ignored

Describe the bug
When there are multiple neededForBoot fileSystems that have internal dependencies, setting "depends" takes no effect because no related info was generated into the initrd-fstab file, resulting in wrong order of mounts and a boot failure.
Steps To Reproduce
Steps to reproduce the behavior:

Set up an nfs or nbd device and mark them neededForBoot
Set up additional neededForBoot mount that depends on the above one, for example an overlayfs providing /nix that uses the above device as lowerdir/upperdir etc.
See a boot failure because overlayfs was attempted in prior to even networking finished setup, even if the first device has been defined as "depends" entries of the second one.

Expected behavior
x-systemd.requires entries should be generated into initrd-fstab to set up proper dependencies according to depends array. This may only supports mount points though.
Notify maintainers

@NickCao

Should be fixed by https://github.com/NixOS/nixpkgs/pull/273642

My PR would only fix this for the systemd-stage-1 case, not for the scripted initrd.


Incompability with metalsmith-watch

Hello,
I'm encountering an issue when using this plugin along with the metalsmith-watch plugin.
The problem is that the metalsmith-watch plugin calls the middleware stack and only passes the files that actually changed (which is the expected behavior). Therefore when metalsmith-metadata try to loads the desired metadata files, they do not exists anymore (unless they have been edited). And so it throws an error.

@aymericbeaumet have you managed to resolve this issue? could you possibly share your solution?

@fizzydigital unfortunately no solutions so far, I think I ended up using something like nodemon


Built file with webpack outputs entire built content

I built a Node module with webpack and hosted on npmjs. The resulting code is very compact, even when I use js-beautifier on it. However, when I try to user-test an error scenario, it outputs the WHOLE one-line code, which is quite ugly in my opinion.

I know it is nit-picky, but do you have any advice, inside or outside webpack, to avoid the above unnecessary sausage-code output?

Please set https://webpack.js.org/configuration/optimization/#optimizationminimize to false to disable such optimization for the production usage, thank you and feel free to feedback

This is me now.


The number of clusters could not be customized

Checklist

[X] I am using the current master branch
[X] I am running on an up-to-date pypsa-usa environment. Update via conda env update -f envs/environment.yaml

The Issue
I changed the default configuration file area to California and only the renewable energy. I set the number of clusters to 10 and the error message to 4.0. In addition, the clustering algorithm can only use KMEANS, not HAC.
os：win10
Steps To Reproduce
No response
Expected Behavior
scenario:
interconnect: [western] # "usa|texas|western|eastern"
clusters: [10]
simpl: [50]
opts: [REM-3h]
ll: [v1.0]
sector: "" # G
planning_horizons: [2030, 2040, 2050]    #(2018-2023, 2030, 2040, 2050)
model_topology:
transmission_network: 'reeds' # [reeds, tamu]
topological_boundaries: 'reeds_zone' # [county, reeds_zone]
interface_transmission_limits: false
include: # mixed zone types not supported
reeds_state: ['CA']
Error Message
INFO:pypsa. io: Imported network elec_s50.nc has buses, carriers, generators, lines, links, loads, storage unitsINFO:ma1n:Aggregating to transport model with reeds zone zones.INFO :main :Using Transport Mode1.Traceback(most recent call last):File "D: \pypsa-usa workflow. snakemake\scripts(tmpfsga3tyf.cluster network.py", line 672, in <module》assert n clusters = len
ssertionErrdNumber of clusters must be 4 for current configuration..23:43 2024Mon Nov 4 08.Error in rule cluster network:

Anything else?
I'd like to see more clusters of renewable energy in California

The error message is telling you that you need to change the number of clusters to 4. This is because you are using the reeds transmission network which has a pre-fixed number of nodes per state.
see instructions here: https://pypsa-usa.readthedocs.io/en/latest/config-spatial.html


Add pixels for Privacy Stats on HTML NTP

Task/Issue URL: https://app.asana.com/0/69071770703008/1208936504720914/f
Description:
This change adds 2 new messages to track actions in the Privacy Stats widget
and implements usage and debug pixels for Privacy Stats.
Steps to test this PR:

Run the app from Xcode, set internal user state and set htmlNewTabPage feature flag.
Filter Xcode console by "PixelKit" subsystem.
Visit some websites to have Privacy Stats widget populated.
Click "Show More" and then "Show Less" and verify the following output in the console:

👾[Daily and Count-Fired] m_mac_new-tab-page_blocked-tracking-attempts_show-more_daily ["appVersion": "1.118.0", "pixelSource": "browser-dmg"]
👾[Daily and Count-Fired] m_mac_new-tab-page_blocked-tracking-attempts_show-more_count ["appVersion": "1.118.0", "pixelSource": "browser-dmg"]
👾[Daily and Count-Fired] m_mac_new-tab-page_blocked-tracking-attempts_show-less_daily ["pixelSource": "browser-dmg", "appVersion": "1.118.0"]
👾[Daily and Count-Fired] m_mac_new-tab-page_blocked-tracking-attempts_show-less_count ["pixelSource": "browser-dmg", "appVersion": "1.118.0"]

Repeat previous step and verify the following output in the console:

👾[Daily and Count-Skipped] m_mac_new-tab-page_blocked-tracking-attempts_show-more_daily ["pixelSource": "browser-dmg", "appVersion": "1.118.0"]
👾[Daily and Count-Fired] m_mac_new-tab-page_blocked-tracking-attempts_show-more_count ["pixelSource": "browser-dmg", "appVersion": "1.118.0"]
👾[Daily and Count-Skipped] m_mac_new-tab-page_blocked-tracking-attempts_show-less_daily ["pixelSource": "browser-dmg", "appVersion": "1.118.0"]
👾[Daily and Count-Fired] m_mac_new-tab-page_blocked-tracking-attempts_show-less_count ["pixelSource": "browser-dmg", "appVersion": "1.118.0"]

Without closing the app, call the following in terminal:

sqlite3 ~/Library/Containers/com.duckduckgo.macos.browser.debug/Data/Library/Application\ Support/PrivacyStats.sqlite 'drop table ZDAILYBLOCKEDTRACKERSENTITY'

Visit a website, verify the following output in the console:

👾[Daily and Count-Fired] m_mac_debug_new-tab-page_privacy-stats_database_error_daily ["e": "2", "d": "PrivacyStatsError", "appVersion": "1.118.0", "ud": "NSCocoaErrorDomain", "ue": "256", "pixelSource": "browser-dmg"]
👾[Daily and Count-Fired] m_mac_debug_new-tab-page_privacy-stats_database_error_count ["pixelSource": "browser-dmg", "appVersion": "1.118.0", "d": "PrivacyStatsError", "ud": "NSCocoaErrorDomain", "e": "2", "ue": "256"]

Quit the app and call the following in terminal:

echo "hello" > ~/Library/Containers/com.duckduckgo.macos.browser.debug/Data/Library/Application\ Support/PrivacyStats.sqlite

Launch the app, verify the following output in the console:

👾[Daily and Count-Fired] m_mac_debug_new-tab-page_privacy-stats_could-not-load-database_daily ["pixelSource": "browser-dmg", "appVersion": "1.118.0", "e": "26", "d": "NSSQLiteErrorDomain"]
👾[Daily and Count-Fired] m_mac_debug_new-tab-page_privacy-stats_could-not-load-database_count ["d": "NSSQLiteErrorDomain", "appVersion": "1.118.0", "e": "26", "pixelSource": "browser-dmg"]

Clean up after testing so that your debug build works:

rm -f ~/Library/Containers/com.duckduckgo.macos.browser.debug/Data/Library/Application\ Support/PrivacyStats.sqlite

Definition of Done:

[ ] Does this PR satisfy our Definition of Done?

Internal references:
Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

Thanks @SabrinaTardio!
Pixels are permanent. Show more/less events are tested in NewTabPagePrivacyStatsClientTests (to the point of being pushed to the event mapper - the receiver that fires pixels is trivial and doesn't need testing imo).
Database error pixels aren't tested because it's tricky to break the database in unit tests, but the events are pushed from do-catch handlers for database operations so this is again straightforward.


Add nightly retrigger script

A script to automatically relaunch and retrigger suites during meto nightly testing.
This PR also edits the generate script to set lfric apps heads testing to use a new working copy as the heads step edits the working copy. Updates relevant unit test for this.
Linked to MetOffice/simulation-systems#159

Thanks Roddy, Some changes made and some pushed back to you.


What is the usage of sample_data folder?

sample_data folder and its csv and vtk files do not seem to be used anywhere in this repo or documented. May I know what do we need them for?

It is also very big: 93M	./sample_data

Maybe we can store these data outside of git?

@suprosanna @jocpae: I would be happy for your input here. Thanks!

Good comment, please remove @jqmcginnis :)

Done!


Running from another Dockerfile and git clone the source

[X] I tried updating to the latest version.

[ ] I can't, there is an issue.

[ ] I Am on Windows

[ ] Ubuntu Bash on Windows
[ ] Fedora Bash on Windows

[X] I Am on Linux
[ ] I Am on macOS

Description
Docker File:
FROM jekyll/jekyll:3

RUN git clone https://github.com/mmistakes/minimal-mistakes .

CMD ["jekyll", "serve"]

Why when i try to run the jekyll image by another Dockerfile to include my sourcecode on the build it doesn't work like expected.
when i try to login to the container and ls /srv/jekyll it was empty !
I suspect this line
Because i didn't volume any thing because i don't need to

Steps

docker build -f Dockerfile -t testjekyll .
docker run -it --rm testjekyll /bin/bash
ls /srv/jekyll

Output
Empty
Expected
The content of the git repo cloned in the docker file

Hi, This is not a Jekyll-Docker bug.

Yeah, I think its't, but why we put this line though, it prevent the above behavior, and if we remove it it will not effect something like this
docker run --rm \
  --volume="$PWD:/srv/jekyll" \
  -it jekyll/jekyll:$JEKYLL_VERSION \
  jekyll build

Right ?

I put that because that is not only a best practice, but how I want my image to work.

OK, Thank You.

Hi, This is not a Jekyll-Docker bug. And what you are doing is out of scope, you have provided no logs, and only a vague example that puts all the burden on me.

I think you need to explain the function of SRV / Jekyll directory. Is he a static file_ Site directory or Jekyll's build directory


add 90% line support

add 90% line stats

It's unclear to me what this adds. p90 is already covered in the stats.


GH-38342: [Python] Update to_pandas to use non-deprecated DataFrame constructor

Rationale for this change
Avoiding a deprecation warning from pandas

Closes: #38342

@github-actions crossbow submit -g python


http related tests rely on external service that may sometimes be down

https://restcountries.eu/rest/v2/alpha/cz is used by http tests but it may be down (as now) and break the tests

Time to fix restcountries?

wonder if we should use mockserver module

That project is mostly dead, there is a guy working on a fork of the apilayer restcountries API and he is providing a new url with the service up and running

https://restcountries.com/v2/alpha/cz

wonder if we should use mockserver module

Or just locally stub out whatever endpoint(s) we need with a JAX-RS resource class.

I think the only reason we use restcountries is because of the ssl

I don't know how simple it is to have an ssl and a non ssl endpoint in quarkus

I'm switching to the restcountries.com/v2/alpha/cz for the moment, then we need a better solution

Yeah, SSL was the main motivation for using restcountries


chore(ci): notify cherry-pick

Summary

Add a workflow to notify CE2EE cherry-pick.
Checklist

[n/a] The Pull Request has tests
[n/a] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[n/a] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

Add a workflow to notify CE2EE cherry-pick.

Issue reference

Fix #KAG-2881

Can we wait a little? @AndyZhang0707 is researching how we properly do change management, I don't think another Slack bot will solve all the issues, plus we have half a dozen Slack bots already.
It seems that making use of Jira would be a better idea than adding another Slack bot that people learn to ignore after a while.

I will do a POC. If merge or not, let's wait for decisions.

Worth to do this rather than manually track it.

@dndx This is ready for merge if wanted. Let's leave it here.
I will fix other workflows.


[Feature] Replace room background image with bundle "door" pictures

Is your feature request related to a problem? Please describe.
Clients mentioned that instead of using remote background image for each room, consider using static "door" pictures wrapped in bundle. In this way, it better fits our "escape room" topic.
Describe the solution you'd like
Have 10 pictures for 5 different types of doors (open/close). When a game is not unlocked, use closed door; when passed, use open door.

partially done with lock icon. Will need some nice door pictures later.

Since we are changing the terminology, this might be changed.


Update dependencies

Coverage remained the same at 89.933% when pulling 003454d38c5f66b413a7fc9275c054e6290c359d on rel_branch into fa01240537eb5560af1191eefd08a0fe0fd1cb27 on master.


How can I set the loss weight with mult_outputs?

Hi all,  My network has mult outputs, I just set the different loss weight for each loss layer just like caffe, How can I do it?

Yes, there is a grad_scale parameter for softmax (
https://mxnet.readthedocs.org/en/latest/python/symbol.html). Also it
supports manually setting different gradient back by using low level
symbolic bind API, where you can do whatever you want in gradient outside
MXNet
On Thu, Oct 29, 2015 at 05:38 何俊然<EMAIL_ADDRESS>wrote:

Hi all, My network has mult outputs, I just set the different loss weight
for each loss layer just like caffe, How can I do it?
—
Reply to this email directly or view it on GitHub
https://github.com/dmlc/mxnet/issues/421.

I see, thanks for reply:)


Added push url for payments

Klarna told me this was needed, but it isn't. Closing pr

push_url is required for KCO (https://developers.klarna.com/api/#checkout-api__create-a-new-ordermerchant_urls__push) but not for KP (https://developers.klarna.com/api/#payments-api__create-a-new-credit-sessionmerchant_urls__push)


Listener for Key removed?

I might have overlooked something but is there a way to get notified when a Key has been removed from History (i.e. user has pressed the back button and navigated to previous screen, or user has "rewritten" history,etc.)
If not, I think it would be useful for other third party libraries to listen for such changes (mainly Key removed) and therefore I would like to open a Feature-Request for that.

This can easily be done in your Dispatcher implementation. Without a specific and broadly applicable use case, I'm disinclined to add the feature.

I know that this can be done in Dispatcher, but in my opinion it would be a cleaner and more extensible approach to implement an observing mechanism for History changes (maybe the Dispatcher could be seen as one such History Observer).
The use case I have is: Imagine a library (like an MVP library) that manages some components (like Presenters), but those components should live just as long as needed. Therefore observing the history would be useful to determine when those components are no longer needed.

This might be a usecase for a flow.ServicesFactory?

yes, but I was more hoping to find an flow independent way, so that this library could offer a "flow plugin" the user can plug in once setting up flow in Activity. Couldn't this plugin be a ServiceFactory one might ask? Yes, but then the user of this library still have to connect ServiceFactory and  and the third party library manually in his View. I was hoping to find a solution that requires writing less code for the user of the third party library and I think this could be achieved by observing the History.

The ServiceFactory is exactly the intended extension point.


[channelmod] rmmirror says Done when it shouldn't

As long as the two channels exist in servers she's in, it says Done even if the mirror didn't exist

this issue is now in the wrong repo


Impossible to check real buy price

Hello,
When i want to buy for example 1 ethereum for 0.055 bitcoin with a limitBuy while the actual price on binance is 0.0549 bitcoin, the order will buy 1 ethereum for 0.0549 instead of 0.055.
Now we can receive the order information and buy price with:
Order order = client.getOrderStatus(new OrderStatusRequest(symbol,"12345L")); String buyPrice = order.getBuyPrice();
At this moment the buyPrice is 0.055 instead of 0.0549 and there seems to be no way to receive the real buy price from this order.
Is there an other way to receive this information or is this not possible with this API?

It's impossible because their API is shitty
https://github.com/binance-exchange/binance-java-api/issues/104

You can get the real buy price from this list of order fills:
{   "symbol": "BTCUSDT",   "orderId": 28,   "clientOrderId": "6gCrw2kRUAF9CvJDGP16IP",   "transactTime":<PHONE_NUMBER>595,   "price": "0.00000000",   "origQty": "10.00000000",   "executedQty": "10.00000000",   "status": "FILLED",   "timeInForce": "GTC",   "type": "MARKET",   "side": "SELL",   "fills": [     {       "price": "4000.00000000",       "qty": "1.00000000",       "commission": "4.00000000",       "commissionAsset": "USDT"     },     {       "price": "3999.00000000",       "qty": "5.00000000",       "commission": "19.99500000",       "commissionAsset": "USDT"     },     {       "price": "3998.00000000",       "qty": "2.00000000",       "commission": "7.99600000",       "commissionAsset": "USDT"     },     {       "price": "3997.00000000",       "qty": "1.00000000",       "commission": "3.99700000",       "commissionAsset": "USDT"     },     {       "price": "3995.00000000",       "qty": "1.00000000",       "commission": "3.99500000",       "commissionAsset": "USDT"     }   ] }
But if you use this API, there is no list with Fills in the Order object.

@luxel1103  Just found out it does not support list with Fills, trying to add it myself now lol

@Willthegod Please let me know if you succeed in this, would be interested in the code for that.


Russian translation

@birkoffe @DoumanAsh Will you let me know when this is ready to merge? 😄

I don't have any more comments

Yes, you can merge.


Create a stackless ClosedChannelException to reduce overhead when the…

… Channel is closed
Motivation:
In some benchmarks closing the Channel attributes to a lot of overhead due the call of fillInStackTrace(). We should reduce this overhead.
Modifications:

Create a StacklessClosedChannelException and use it to reduce overhead.
Only call ChannelOutboundBuffer.failFlushed(...) when there was a flushed message at all.

Result:
Less performance overhead when closing the Channel

You got some test failures, though.

@chrisvest ups... fixed :)


Bwc update

Description
Add bwc test cases for Event Analytics and Custom Panels
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #604 (a9c131f) into main (af54d6a) will decrease coverage by 7.18%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #604      +/-   ##
============================================
- Coverage     53.32%   46.14%   -7.19%
============================================
  Files           238      201      -37
  Lines          7748     5600    -2148
  Branches       1581     1330     -251
============================================
- Hits           4132     2584    -1548
+ Misses         3421     2985     -436
+ Partials        195       31     -164

Flag
Coverage Δ

dashboards-observability
46.14% <ø> (+0.04%)
:arrow_up:

opensearch-observability
?

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

Impacted Files
Coverage Δ

...bility/action/CreateObservabilityObjectResponse.kt

...rg/opensearch/observability/ObservabilityPlugin.kt

.../observability/model/ObservabilityObjectDocInfo.kt

...lin/org/opensearch/observability/model/Notebook.kt

...ensearch/observability/model/SavedVisualization.kt

...vability/action/DeleteObservabilityObjectAction.kt

...pensearch/observability/action/PluginBaseAction.kt

...rch/observability/model/ObservabilityObjectType.kt

...servability/action/GetObservabilityObjectAction.kt

...ch/observability/index/ObservabilityQueryHelper.kt

... and 31 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 af54d6a...a9c131f. Read the comment docs.


Add support for preview frame time code

This PR introduces two features:
(1) validation of the uploaded app previews and removal of "FAILED" uploaded assets (thus preventing things such as those describe in #7 to happen in the frontend)
(2) setting of the preview time code to either: 00:00:00:01 or the value of the <preview_name_without_extension>.txt. Note that this is an opinionated choice, as the app store by default uses the value at 00:00:05:01 which to me is quite non-sensical (#4)

I have also added an allow list for videos which is mp4, mov, avi (nothing else should be supported by App Store Connect)

And ".jpg", ".jpeg", ".png", ".tif", ".tiff", ".bmp" for images (I know it's outside of the scope once again.. sorry)

Just fyi, on holiday for a week, so I'll be a bit slow to respond. I'll take a look at the PRs soon as I get back.

oh no worries!! enjoy! :D

https://github.com/bennord/appstore-tools/pull/16 addresses the previewFrameTimeCode feature.

Let me know if there's anything missing still form this PR that hasn't been addressed and we can reopen something if needed.


Unknown command line flag 'alphabet_config_path'

Hey I am having this error for running make_checkpoint.py
Traceback (most recent call last):
  File "make_checkpoint.py", line 49, in <module>
    tf.app.flags.FLAGS.alphabet_config_path = "DeepSpeech/data/alphabet.txt"
  File "/home/zifanw/env/lib/python3.6/site-packages/tensorflow/python/platform/flags.py", line 88, in __setattr__
    return self.__dict__['__wrapped'].__setattr__(name, value)
  File "/home/zifanw/env/lib/python3.6/site-packages/absl/flags/_flagvalues.py", line 499, in __setattr__
    return self._set_unknown_flag(name, value)
  File "/home/zifanw/env/lib/python3.6/site-packages/absl/flags/_flagvalues.py", line 375, in _set_unknown_flag
    raise _exceptions.UnrecognizedFlagError(name, value)
absl.flags._exceptions.UnrecognizedFlagError: Unknown command line flag 'alphabet_config_path'

can I know what version of TF you are using? Thx

Hi,
Please use tensorflow-gpu==1.14
Thanks

Thanks for replying. I actually use this version, but it won't let me pass. if you are using conda, could you share a env yapf file? Another change I made is that, I git chechout to the previous version, but however, for DeepSpeech < 0.5.0, their link for downloading ds_ctcdecoder is giving HTTP error. I have to use DeepSpeech 0.5.1 to successfully install ds_ctcdecoder. But I think this is not the version of DeepSpeech you are building on, right?

Ah, I see. Yes, I use the 0.4 version. Could you try opening an issue on the DeepSpeech for the proper 0.4 link?
I will try to see if the code can be updated to 0.5 in a few weeks

Okay, I am raising an issue in their discussion page. Seems for version >0.5.1, they change a lot. Hopefully, you will succeed!
Thx

Okay, I am raising an issue in their discussion page. Seems for version >0.5.1, they change a lot. Hopefully, you will succeed!
Thx

Do you fix that problem? I have the same issue.

Hi @zifanw @ruby11dog,
I have updated the installation instructions with much more detail in the README.
Please follow the steps there and open another issue if you still have any problems.


[fix] don't watch generated types directory

Fixes error on Windows CI

:( Error: EPERM: operation not permitted, lstat '\\?\D:\a\kit\kit\packages\kit\test\apps\amp\.svelte-kit\types\src\routes' Huh

Nice! Any theory as to why ignoring the entire outDir doesn't work?

Okay, looking at the failures more closely (after you fixed exclude to ignored in 853d9826915e61a14dd18d90309104bef2523864), they were just flakiness rather than a 'proper' failure.
I think what we want to do is ignore the entire outDir except for the generated directory (I tried ignoring the entire outDir, but then it doesn't pick up changes to the route manifest, e.g. if you navigate to /unknown and then created src/routes/unknown.svelte, you need to watch generated or it won't refresh to that page)


Blackout stopped working after upgrade to version 5

After the upgrade to Alerta server version 5 alerts weren't being suppressed any more during blackout periods.

The use of blackouts for suppressing alerts is now optional. It has been factored out into a plugin and must be included explicitly in the list of PLUGINS in the server configuration file.