WIP: bump signal export to 3.2.0

Update signal-export to version 3.2.0
Why
signal-desktop has changed a lot since signal-export 1.8.2 and therefore signal-export changed as well.
Things done

update version to 3.2.0
change pypi package name (signal_export instead of signal-export since otherwise the tarball is not found)
add dependencies that where missing (see pyproject.toml)
switch build-backend to pdm-backend (see pyproject.toml

Fixes missing

dependency for sqlcipher is sqlcipher3-wheels which is basically the same code as pysqlcipher3 but the nix build fails with

Running phase: pythonRuntimeDepsCheckHook
@nix { "action": "setPhase", "phase": "pythonRuntimeDepsCheckHook" }
Executing pythonRuntimeDepsCheck
Checking runtime dependencies for signal_export-3.2.0-py3-none-any.whl
  - sqlcipher3-wheels not installed

I am not proficiant enough to package sqlcipher3**-wheels** and I am also unsure if it is a good idea since maybe the "wheels"-wrapping this provides should rather be done in nixpkgs at python312Packages.sqlcipher3 or in signal-export?
Checks

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@belamu I have pushed two commits to my fork https://github.com/NixOS/nixpkgs/compare/master...phaer:nixpkgs:353082-bump-signal-export-to-3.2.0 . I packaged the right sqlcipher3 and signal-export seems to build. I can't currently test it easily, so please do if you can :)

Wow, thanks. And quick reaction!
I'll try it out in a minute.
What I don't understand is: why did you create two MetaPackages with the exact same content (sqlcipher3-binary and sqlcipher3-wheels).

Worked flawlessly! Now my backup routine is complete again after moving to nixOS :)

What I don't understand is: why did you create two MetaPackages with the exact same content (sqlcipher3-binary and sqlcipher3-wheels)

Because all 3 are effectively the same package, the later two are just distributed as binary wheels on pypi. As we are building the package from source anyway in nixpkgs, and one of the outputs is a wheel, we just reuse that. And update metadata to match the name, so things which depend on sqlcipher3-wheels instead of sqlcipher3 build without further patching.
Does that make sense?

Worked flawlessly! Now my backup routine is complete again after moving to nixOS :)

Great to hear! Then you could push those commits to your branch and undraft the PR?

Does that make sense?

It does to me.

undraft the PR?

Yes. I combined the commits to decrease clutter in the history. The result is exactly the same as with your changes. Your dependency-fixing is not attributed to you (the new package is), I hope that's fine.

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD".

I couldn't run this because the git checkout or tarball extraction failed, maybe because of too little RAM. But there shouldn't be any packages this change depends on since signal-export is "end-user-software" and the sqlcipher is new.

Oh no. I screwed up ><
I was fixing the git commit message and did not see your push @belamu, sorry about that ><. Fixing this.

@phaer in case you saw the new signal-export release: it's broken, I messed it up and will fix it in the next days
See the pr

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353086

x86_64-linux

:white_check_mark: 14 packages built:

python311Packages.sqlcipher3
python311Packages.sqlcipher3-binary
python311Packages.sqlcipher3-binary.dist
python311Packages.sqlcipher3-wheels
python311Packages.sqlcipher3-wheels.dist
python311Packages.sqlcipher3.dist
python312Packages.sqlcipher3
python312Packages.sqlcipher3-binary
python312Packages.sqlcipher3-binary.dist
python312Packages.sqlcipher3-wheels
python312Packages.sqlcipher3-wheels.dist
python312Packages.sqlcipher3.dist
signal-export
signal-export.dist

@natsukium
From my point of view, the pull request is ready for merge even though unfortunately some tests are missing:

for pysqlcipher3: as @phaer said it's complicated. Sorry, I only understand it rudimentarily
for signal-export: the only test that the repo provides is outdated and broken, as I noted here. Therefore it is not included. I tried it out with a current signal-desktop nixos installation on KDE and I hope that @carderne (Maintainer of signal-export) tried it out with Gnome.

I cannot build and try out on Darwin. There is platform dependent code that I did not touch during the last commits. It would be nice if someone tried but I hope this PR can be merged without it.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 353086

aarch64-darwin

:white_check_mark: 14 packages built:

python311Packages.sqlcipher3
python311Packages.sqlcipher3-binary
python311Packages.sqlcipher3-binary.dist
python311Packages.sqlcipher3-wheels
python311Packages.sqlcipher3-wheels.dist
python311Packages.sqlcipher3.dist
python312Packages.sqlcipher3
python312Packages.sqlcipher3-binary
python312Packages.sqlcipher3-binary.dist
python312Packages.sqlcipher3-wheels
python312Packages.sqlcipher3-wheels.dist
python312Packages.sqlcipher3.dist
signal-export
signal-export.dist


Echo on 6.0.2 when speaker is used

Description
The users on Twilio Voice SDK v6.0.2 are hearing echo of them speaking when they turn on the speaker. The previous version (v6.0.1) is not affected by this issue.
Steps to Reproduce

Start a call with SDK v6.0.2
Turn on the speaker
The phone plays back whatever you say

Expected Behavior
There should be no echo
Actual Behavior
Echo is there
Reproduces How Often
The echoed voice is heard clearly when you start speaking after a pause. A few seconds after SDK seems to start suppressing the echo, although not ideally. There is a noticeable difference between 6.0.2 and 6.0.1
Versions
Issue introduced in version 6.0.2
Voice iOS SDK
6.0.2
Xcode
12.3
iOS Version
14.0
iOS Device
iPhone 6s, iPhone 7

Hi @nalexn,
Thank you for the report.
I looked at the changes in 6.0.2 compared to 6.0.1 and they were only related to Swift compatibility. I will dig deeper to see if there is anything else in 6.0.2 that could cause differences in echo cancellation behaviour.
Best,
Chris

Hey @ceaglest , we've shipped a new version of our app with a downgraded SDK version to 6.0.1, however, a few users who stayed in touch reported that the update didn't help with resolving the echo issue. I wonder if this was introduced earlier, most of those users didn't have the issue in the past.

Hey @ceaglest , we've shipped a new version of our app with a downgraded SDK version to 6.0.1, however, a few users who stayed in touch reported that the update didn't help with resolving the echo issue. I wonder if this was introduced earlier, most of those users didn't have the issue in the past.

Hi @nalexn thanks for the update.
Any chance you can provide the Twilio Call SIDs where your users are experience the echo issue?
Best,
-bobie

Hi @nalexn thanks for the update.
Any chance you can provide the Twilio Call SIDs where your users are experience the echo issue?
Best,
-bobie

Hey @bobiechen-twilio
One user:
CA73896deea90d37639d2498d9ad0070da
CA52584caa3fa626815d0677ff830b94bf
CA17aa3ec62709eb7a33d7a922e4b78580

Another user:
CA73896deea90d37639d2498d9ad0070da
CA52584caa3fa626815d0677ff830b94bf
CA17aa3ec62709eb7a33d7a922e4b78580

Hey @bobiechen-twilio
One user:
CA73896deea90d37639d2498d9ad0070da
CA52584caa3fa626815d0677ff830b94bf
CA17aa3ec62709eb7a33d7a922e4b78580

Another user:
CA73896deea90d37639d2498d9ad0070da
CA52584caa3fa626815d0677ff830b94bf
CA17aa3ec62709eb7a33d7a922e4b78580

Hi @nalexn
Thanks for the Call SIDs. I have created a ticket for tracking the issue.
While we are trying to reproduce and triage the issue, is this issue reproducible in your dev environment, other than the the reports from app users?
-bobie

Hi @nalexn
Thanks for the Call SIDs. I have created a ticket for tracking the issue.
While we are trying to reproduce and triage the issue, is this issue reproducible in your dev environment, other than the the reports from app users?
-bobie

Hey @bobiechen-twilio , this is probably our fault, after a refactoring we've introduced a line of code that was changing the AVAudioSession category setting, overriding the voiceChat setting that Twilio SDK sets via the audioDevice.
A slight echo is always there, even after we've fixed the above, so we'd need to release the fix and have the users try it before I can close this ticket.

Hey @bobiechen-twilio , this is probably our fault, after a refactoring we've introduced a line of code that was changing the AVAudioSession category setting, overriding the voiceChat setting that Twilio SDK sets via the audioDevice.
A slight echo is always there, even after we've fixed the above, so we'd need to release the fix and have the users try it before I can close this ticket.

Hi @nalexn
Hope you are doing well and have resolved the echo issue. We are happy to hear back from you.
Cheers,
-bobie

Hi @nalexn
Hope you are doing well and have resolved the echo issue. We are happy to hear back from you.
Cheers,
-bobie

Hi @nalexn
I am hoping the echo problem has been resolved with the AVAudioSession config. Closing the issue for now but please feel free to reach out to us in the future.
Cheers,
-bobie


Implement QName compression in minmdns

Problem
Our MDNS packets begin to be quite large given advertising of vid/ping and multiple  IP addresses.
Change overview
Adds support for qname compression when building mdns packets. Expect to make packets somewhat smaller.
Testing
E2E unit tests on linux exercise minmdns a lot
Unit tests updated & added new unit test for the compressed writer
Manual validation with minmdns example that query works

Draft until I fix compile errors and some updates for having qname compression across records (which  is the main useful bit for everything)

Moved to fast track: PR up for quite some time, several checkmarks, significant test coverage.


command line arguments are stringified, they no longer have special meaning in the shell.

Description
The command line specified in a lint-staged config is split, and each part seems to be wrapped as a string with quotes.
Steps to reproduce
For example, if I write the following in package.json,
  "lint-staged": {
    "*.{ts,tsx}": [
      "npm run compile && :"
    ],
  }

then the string "npm run compile && :" is executed as if I had written
$ npm run compile "&&" ":"

at my command line, and so the && and : pieces lose their original meaning.
I am expecting it to run my commands as if they were written like this at the command line:
$ npm run compile && :

(Notice the lack of quotes, which is different than the one with quotes.)
The way to work around this is to put the command in package.json's scripts field, then run the command with npm run the-script --.
Environment

OS: Manjaro Linux
Node.js: 14.8
lint-staged: 10.5.0

By the way, I've been using lint-staged for quite some time now. It's been really great!!!

Lint-staged doesn't use shell parsing by default. You have to enable it with the lint-staged --shell flag (where you configure the git hook to run, like husky).
Please note that --shell is not recommended since it might lead to vulnerabilities like arbitrary command execution.

Yes, this issue should be closed accordingly since it describes the intended behavior of lint-staged.


Importing DTO to Database and/or creating schema

More of a question than an issue but I assume that someone here might be able to guide me a bit.
I'm using Laravel and writing an application to parse and display a bunch of stats for a summoner.  My issue is getting the information contained in the DTO created by LeagueWrap into my database.  Is there an automated way of setting up a database schema (migration) or do I have to map it out my hand?  For an average game there's a lot of data fields to map and while I guess I could do it by hand it seems like a lot of work.
Along those same lines, would it be easier to work with JSON data? If so, is there an easy way to convert the DTO generated by LeagueWrap to JSON?
Sorry, new-ish to PHP and Laravel.

The answer very much depends on what database you are planning to use.
MongoDB for example allowes to insert json as data structure so it's very
easy to integrate. For sql like databases it is harder because you need to
put more effort into normalization.
Yes, you can access the raw json data from the dtos. I think the function
is ->raw()
Meverhart<EMAIL_ADDRESS>schrieb am Mi., 8. Feb. 2017, 16:55:

More of a question than an issue but I assume that someone here might be
able to guide me a bit.
I'm using Laravel and writing an application to parse and display a bunch
of stats for a summoner. My issue is getting the information contained in
the DTO created by LeagueWrap into my database. Is there an automated way
of setting up a database schema (migration) or do I have to map it out my
hand? For an average game there's a lot of data fields to map and while I
guess I could do it by hand it seems like a lot of work.
Along those same lines, would it be easier to work with JSON data? If so,
is there an easy way to convert the DTO generated by LeagueWrap to JSON?
Sorry, new-ish to PHP and Laravel.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/LeaguePHP/LeagueWrap/issues/15, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEeQ8u2AagIyYekdvAm0RdEtzFYN6ruUks5raeWLgaJpZM4L6_zQ
.

I'll be using MySQL.

Simplest problem:  I'm not sure how to best store the data your wrapper provides in my database.

It all depends on what you want to store (and analyze), sometimes it isn't necessary to store the whole json response, but only parts of it. Laravel's Eloquent supports MySQL, Postgres, MSSQL and SQLite I believe, but there are packages that add additional driver support for example for MongoDB. If you have any more questions feel free to post on the Riot Dev forums or on the Discord channel.

These responses were not helpful in the least.  For the sake of example let's say I make this call to the API:
$match = $api->match()->match(1399898747);
I want to store everything it returns in a MySQL database.  How do I go about doing that?  Do I need a pre-definded database schema based around the data that my API call will return or can I automate that process since any call for match information will essentually return the same type of data.

This is out of scope of this platform. You don't have a problem with the
api itself or its implementation.
As already said you can receive the raw response as array (see above).
Persisting that data in a database is nothing that api wrapper is able to
do but can be implemented on top of it. This is a topic that has already
been discussed on the dev board multiple times so you should find your
answer there. It has nothing to do with this api wrapper.
Meverhart<EMAIL_ADDRESS>schrieb am Do., 9. Feb. 2017, 15:38:

These responses were not helpful in the least. For the sake of example
let's say I make this call to the API:
$match = $api->match()->match(1399898747);
I want to store everything it returns in a MySQL database. How do I go
about doing that? Do I need a pre-definded database schema based around the
data that my API call will return or can I automate that process since any
call for match information will essentually return the same type of data.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/LeaguePHP/LeagueWrap/issues/15#issuecomment-278659545,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEeQ8v9hhKAWrUflCXRNuXTLISVNot8Nks5rayTOgaJpZM4L6_zQ
.


404 Not found on LOCALHOST while running game

I can't get to the route map localhost:8888 and it showed 404 error.
This only occurs while I am in the game.
When I quit, the webpage changes into 'connection refused'
Big thanks to help solving the problem!

by the way, I'm using Forge and Architectury

Ah yes, sorry; it is a bit different for the mod's forge port.
Localhost:8888 is only for the fabric port; but for forge; place this -
http://localhost:8888/index.html.
index.html was added at the end. Thanks.

Thanks!


-Wconf should apply leftmost matching rule

The -Wconf:help text also says the leftmost matching rule should apply.
Scala 2
➜ sandbox s -Wconf:any:e,cat=deprecation:s
Welcome to Scala 2.13.13 (OpenJDK 64-Bit Server VM, Java 21.0.1).

scala> 1 → 2
         ^
       error: method → in class ArrowAssoc is deprecated (since 2.13.0): Use `->` instead. If you still wish to display it as one character, consider using a font with programming ligatures such as Fira Code.

Scala 3
➜ sandbox s3 -Wconf:any:e,cat=deprecation:s
Welcome to Scala 3.4.0 (21.0.1, Java OpenJDK 64-Bit Server VM).

scala> 1 → 2
val res0: (Int, Int) = (1,2)

➜ sandbox s3 -Wconf:any:s,cat=deprecation:e
Welcome to Scala 3.4.0 (21.0.1, Java OpenJDK 64-Bit Server VM).

scala> 1 → 2
-- Error: ----------------------------------------------------------------------
1 |1 → 2
  |^^^
  |method → in class ArrowAssoc is deprecated since 2.13.0: Use `->` instead. If you still wish to display it as one character, consider using a font with programming ligatures such as Fira Code.

What do you think a way forward should be? Should the help message be fixed or should the behaviour change to match the help message?
I feel changing the behaviour would be breaking too much. While it's not ideal the behaviour doesn't match Scala 2 IMO it shouldn't be changed at this point.

A related potential issue is when passing multiple -Wconf flags to the compiler, but probably it's OK to rely on the build tool to not reorder settings, like project.settings("-a", "-b").settings("-c", "-d").
In a multi-module build one might want to have a base -Wconf setting and refinements in sub-projects:
val commonSettings = Seq(scalacOptions += "-Wconf:any:e")
val a = project
  .settings(commonSettings)
  .settings(scalacOptions += "-Wconf:cat=deprecation:s")

Fortunately the above works in both Scala 2 and 3

In the case of multiple -Wconf options, Scala 2 prepends; the leftmost matching applies
Scala 3 appends, the rightmost matching applies

The discrepancy is the precedence within a single -Wconf, as shown in the bug report; it can trip up people that migrate or cross-build.
So I'm not sure; I think the reason I implemented "leftmost" in Scala 2 is because left-to-right seems more natural. But then -Wconf:x -Wconf:y is equivalent to -Wconf:y,x, which is also not obvious. In Scala 3 we have rightmost, -Wconf:x -Wconf:y is -Wconf:x,y. Maybe it's better to keep that and adjust the documentation.

If two "matchers" match a warning (one rule makes it an error, the other silences), then the natural "disambiguation" is that the first matching matcher applies.
It's a bit awkward to say "the last matching matcher applies".
Note that there could be any number of strategies, such as "the matcher resulting in error is preferred" or "resulting in silence".
Since there is a "default" configuration, the language about "prepend" (which is an internal detail) suggests "precedence". But maybe lrytz's comment is right and it's just a matter of words. I could not find discussion on old Scala 2.
I don't recall without experiment that -Wconf:x -Wconf:y is different from -Wconf:x,y. I would call that a bug.
Footnote: I don't think "changing behavior" is a hindrance to fixing "bad behavior", especially since the feature is still evolving or improving in scala 3.

It's a bit awkward to say "the last matching matcher applies".

I guess that's why I did leftmost in Scala 2. But one could argue that the last thing added should count, to allow overriding existing settings / defaults, that doesn't seem too awkward.

I don't recall without experiment that -Wconf:x -Wconf:y is different from -Wconf:x,y. I would call that a bug.

It's a necessary consequence of leftmost precedence in Scala 2. -Wconf:help says
The default configuration is:
  -Wconf:cat=deprecation:ws,cat=feature:ws,cat=optimizer:ws

So when calling scalac -Wconf:cat=deprecation:e, that needs to be added to the left of the default.

Figures.
$ git checkout upstream/2.13.x -b test/wconf
fatal: A branch named 'test/wconf' already exists.

I submitted a PR to tweak -Wconf:x,y on scala 2, but did not (yet) amend any verbiage. I think it's just an implementation detail, and we mean "last matching rule" wins.

We seem to have an emerging consensus on the Scala 2 PR that Scala 2 should align with Scala 3 on this, even though it's a breaking change.


Volume detail page salvage button cannot be clicked

https://github.com/rancher/longhorn/issues/238

@yasker


Fix static files BEFORE sessions

To avoid a mongo ping per static file. #1089

@pursual Thank you for submitting this fix. The build failed due to a timeout. You can close & re-open this PR, to initialize the build process again. Hopefully, it was just an intermittent timeout.

LGTM.
Any other considerations? @codydaig @ilanbiala @lirantal

@pursual Before this is merged, you will need to change the commit message to conform to the Contributing guidelines. This is for the commit history, and to easily track changes in release changelog's.
https://github.com/meanjs/mean/blob/master/CONTRIBUTING.md

@pursual The actual commit message, not the body should contain the header. See these commits..
https://github.com/meanjs/mean/pull/1083/commits

I'm away for the weekend, feel free to close and recreate.
On Fri, Dec 4, 2015, 7:52 PM Michael Leanos<EMAIL_ADDRESS>wrote:

@pursual https://github.com/pursual The actual commit message, not the
body should contain the header. See these commits..
https://github.com/meanjs/mean/pull/1083/commits
—
Reply to this email directly or view it on GitHub
https://github.com/meanjs/mean/pull/1090#issuecomment-162119872.

Code LGTM, I don't there there are any issues with doing that.

Squash the commits and then LGTM

Sorry, I give up, you guys have the info needed.
On Sat, Dec 5, 2015, 12:15 PM Cody B. Daig<EMAIL_ADDRESS>wrote:

Squash the commits and then LGTM
—
Reply to this email directly or view it on GitHub
https://github.com/meanjs/mean/pull/1090#issuecomment-162224384.

Closed and resubmitted.


Logs format

Support for logs format
Really a coincidence to @andresmgot 's PR #38 . In addition to #38, this PR also checks if the frame has at least one Time field and one string field as in other cases, it wont render properly in Explorer

Nice! I wonder though, if you've selected result format Logs in the query editor, wouldn't you expect the Logs visualization to be shown in all scenarios? Even if that includes showing an empty logs visualization because no time and string field were found.

I agree with this, is the new code really necessary? AFAICT the behavior is the same in the panel without the check (you simply get an empty response).  One thing that confused me when checking this was that if the columns are not right, it fails silently. Maybe we can return an error here if the frame is not right for logs? (or maybe that's work of the visualization panel).
If you use the explore view, it's helpful to set the preferred visualization even if the required columns are missing because you get the logs view with the proper error (better than showing the table with just some column):

Based on the convo above, I agree that this fallback for logs can be better handled in explore view/logs visualization itself. Thank you for the feedback.

If the data is not in logs format, then either we need to fall back to table view and/or show notice to the user about it aleast.

I think an error makes sense, though to me that almost sounds like a Grafana feature rather than a sqlds one. If a panel receives data that doesn't match what's expected, the panel ought to return the error.


fix git ignored files showing as modified

Apparently not providing any StatusOptions is not the same as default StatusOptions.

The problem goes deeper, have to investigate.


updated zarr source to merge dataset

work on zarr integration

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[FIXED JENKINS-35168] Resolve the context ModelObject when it isn't a ModelObeject

@reviewbybees

🐝

🐝


Node distance calculation wrong

Current pathfinding implementation is somewhat wrong:
Currently considers the past path as the node with least cardinal distance from the target.
This is wrong. In comparison to other pathfinding algorithms:

Dijkstra: Evaluate the next node with lowest cost, based on cost it takes to get to that node
A*: Evaluate the next node with lowest cost, based on the cost it takes to get to that node + estimated distance to target

In conclusion, current distance/cost estimation doesn't properly take account for cost/distance to each node. This is maybe fine for traversing grids, but if teleports/transport methods are taken into account it will behave suboptimally.

This plugin aims to create paths that are as accurate as possible to the in-game OSRS pathfinding. For that reason, it is using the Breadth-first search algorithm. Dijkstra is fairly similar to BFS, but does not offer any benefits over the current BFS algorithm and its modifications. A* could probably speed up the pathfinding, but it would most likely be very hard or impossible to get an accurate OSRS path with it.


[WIP] Add react-i18next to Node app template

WHY are these changes introduced?
Part of the project App Localization - Tools and Guidance (tech design).
Translating apps is technically challenging, time-consuming, and costly. Despite the high demand for localized apps internationally, only a small fraction of apps offered in the Shopify App Store are translated.
None of Shopify's app templates are localized, meaning that the full burden of responsibility is on the developer to research and implement an i18n framework for their app. It’s also more likely to be an afterthought until the developer decides to expand internationally, at which point the effort to externalize all the app's strings is much higher.
By localizing the default Node app template, we reduce the toil for developers, and set them up to localize their apps from the start.

WHAT is this pull request doing?

NOTE: This is a draft pull request for the purposes of discussion - it will not be merged until:

It has buy-in with the client libraries & app templates team
It has been updated to use the Storefront locale file format used by themes and app UI extensions
The README has been updated

This PR:

Adds the i18next and react-i18next libraries
Extracts all hard-coded strings into locales/en.json
Adds locales/fr.json file for demonstration purposes
Adds an i18nUtil utility to initialize i18next
Adds an I18nProvider component to encapsulate locale selection
Extracts navigation to a Navigation component so that the page title can be externalized

English:

French:

Why react-i18next?

It is the most popular open source React i18n framework
It has comprehensive documentation and it's easy to get started
It's extensible with many plugins, including one for Remix

To Do

Write a plugin for i18next to support the Storefront locale file format
Update this PR to use the plugin and update the file format
Update the README to describe the i18next libraries and configuration

Checklist

[ ] I have made changes to the README.md file and other related documentation, if applicable

One additional note; This frontend template is shared by the Ruby, PHP and Node template. Once you ship this, you'll need to pull the submodule into the each backend template.  The good news is the impact will be more than just node!

This frontend template is shared by the Ruby, PHP and Node template. Once you ship this, you'll need to pull the submodule into the each backend template. The good news is the impact will be more than just node!

Oh this is great! I didn't look closely at the Ruby and PHP templates, and I didn't realize that they also use the same React front end.

@sebastianpisula I found an open source code editor extension for VSCode and JetBrains IDE called i18n-ally, which can extract strings for many frameworks and formats, including .jsx and i18next.

I was on slack this thread - and author uses this library: https://www.npmjs.com/package/@shopify/react-i18n

👋🏻 This PR is ready for re-review, feedback is very appreciated!

@byrichardpowell I was using a snapshot version of Polaris while the PR as being approved; please try with Polaris v10.47.0.

@byrichardpowell Sorry, this was a problem with an upstream PR (reverted here and rebased). Please try again with the updated package.json.


Pirate Weather stopped working setup entry fails

Describe the bug
Core 2024.8.2
Supervisor 2024.08.0
Operating System 13.0
Frontend 20240809.0
Pirate Weather Version 1.5.6
Logger: homeassistant.config_entries
Source: config_entries.py:604
First occurred: August 16, 2024 at 10:27:15 PM (3 occurrences)
Last logged: 9:40:07 AM

Error setting up entry PirateWeather for pirateweather
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 604, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/pirateweather/__init__.py", line 56, in async_setup_entry
    pw_scan_Int = _get_config_value(entry, CONF_SCAN_INTERVAL)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/pirateweather/__init__.py", line 168, in _get_config_value
    return config_entry.options[key]
           ~~~~~~~~~~~~~~~~~~~~^^^^^
KeyError: 'scan_interval'

Expected behavior
Load integration and show weather data
Actual behavior

Home Assistant version
2024.8.2
Integration version
1.5.6
Other details
No response
Troubleshooting steps

[X] I have updated my Home Assistant installation to the latest version.
[X] I have updated the Pirate Weather Integration to the latest version.
[X] I have gone through the documentation before opening this issue.
[X] I have searched this repository and API Repository to see if the issue has already been reported.
[X] I have restarted my Home Assistant installation.
[X] I have queried the API in my browser to confirm the issue is not with the API.
[X] I have written an informative title.

Also having this issue - still on 2024.7 here HAOS on RPi 4. Before updating to the latest PW release I was seeing wildly inaccurate data in the forecast sensors (wind speeds of 1000s mph). Not sure if related.

Same here

You can try to go to configuration again (do not change anything) an then press OK. This worked for me
I see that you picture shows version 1.5.5 but you did write integration version 1.5.6 , it seems to be a bug i have the same problem

Good spot. Mine said 1.5.5 too. If you edit manifest.json in the custom_components folder and change the version to 1.5.6, then reload the integration, it starts ok.
However, I’m still getting wild forecast data that’s orders of magnitude out. E.g

Of course this may be a completely separate issue. Any of you in this thread seeing this issue too?

You can try to go to configuration again (do not change anything) an then press OK. This worked for me

Thanks Bloodpack it worked for me too!

Good spot. Mine said 1.5.5 too. If you edit manifest.json in the custom_components folder and change the version to 1.5.6, then reload the integration, it starts ok.

i have opened a new issue #291

Hi guys sorry about the issues here. In V1.5.5 an option to change the scan_interval was added to the configuration options which if you were running the integration before then it didn't exist which is why the error occurs. As stated above if you edit the configuration and click OK it will add in the key value and everything works as normal.
I've added a notice to the latest release the steps on how to fix the issue and will look to see if there's a way to fix this issue after updating.

doing a slight edit to the config did the trick. Thanks for taking care and generally thanks for all the work the contributors do, without you my (our) HomeAssistant would work as it should.

doing a slight edit to the config did the trick. Thanks for taking care and generally thanks for all the work the contributors do, without you my (our) HomeAssistant would not work as it should.

How did you fix it?

doing a slight edit to the config did the trick. Thanks for taking care and generally thanks for all the work the contributors do, without you my (our) HomeAssistant would not work as it should.

How did you fix it?

Open the integration configuration options and then click OK. You don't need to make any changes to the configuration to fix the issue.

doing a slight edit to the config did the trick. Thanks for taking care and generally thanks for all the work the contributors do, without you my (our) HomeAssistant would not work as it should.

How did you fix it?

Open the integration configuration options and then click OK. You don't need to make any changes to the configuration to fix the issue.

Actually for me it didn’t work without doing an actual change (just open and click ok), So I changed the Scan Intervall by 5sec and click then ok, then it was ok

If anyone here is running into the scan_interval KeyError issue could they try the v1.5.7 beta that I just released? I think I fixed the issue but am unable to test as I've already fixed the error on my devcontainer and my HA setup.

Also having this issue - still on 2024.7 here HAOS on RPi 4. Before updating to the latest PW release I was seeing wildly inaccurate data in the forecast sensors (wind speeds of 1000s mph). Not sure if related.

I have had a day with 500C around 12:00AM indeed for a few seconds.

If anyone here is running into the scan_interval KeyError issue could they try the v1.5.7 beta that I just released? I think I fixed the issue but am unable to test as I've already fixed the error on my devcontainer and my HA setup.

I did update to v1.5.7 beta, and it worked fine.

I was able to get a HA install up and running and was able to test upgrading to the v1.5.7 beta and can confirm that this issue is fixed. I've released the stable version which should fix the KeyError issues people have been seeing.

Thanks for fixing it. Still very happy with the integration and hickups can happen!


Invitation didn't show

Contact request/invitation didn't appear in yakyak. There were no network issues as I was chatting with others at that time. Anybody else had this?

It's a known bug that needs to be dealt! :\


Make Block.merkilize_transaction_set a function

Resolves #101
Please let me know if I should add tests.
Minor, I also noticed 'merkel' is spelled 'merkle' or 'merkil' throughout.

Thank you for this PR! Much appreciated.

Minor, I also noticed 'merkel' is spelled 'merkle' or 'merkil' throughout

It's correctly "Merkle," but merkilize is definitely wrong. Can we go ahead and refactor this to merklize? Seems to be the prevailing spelling.

Otherwise, LGTM :+1: signing off on this.

It's correctly "Merkle," but merkilize is definitely wrong. Can we go ahead and refactor this to merklize? Seems to be the prevailing spelling.

You're right, changing now.

LGTM, @nvonpentz mind rebasing?

@DavidKnott Rebased 👍


Problems on MSDEV 2019

Hello, first of all thanks for your library, which is really useful.
I tried using it on windows and I found different problems (I gather them here because, it is mainly minor bypassable problems, sorry for this) :

The 0.5.2 version that we can find here or in vcpkg does not compile (problem with format<> (I use visual studio 2019). Problem not present in the main version that I took)
Cmake by default builds the library in static mode (producing only a cpptrace.lib file). However, When I tried to use it in my program, I got unresolved symbols along these lines :

1>Util.obj : error LNK2001: unresolved external symbol "__declspec(dllimport) struct cpptrace::stacktrace __cdecl cpptrace::generate_trace(unsigned __int64)" (__imp_?generate_trace@cpptrace@@YA?AUstacktrace@1@_K@Z)
1>Util.obj : error LNK2001:  unresolved external symbol "__declspec(dllimport) public: class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > __cdecl cpptrace::stacktrace::to_string(bool)const " (__imp_?to_string@stacktrace@cpptrace@@QEBA?AV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@_N@Z)
1>Util.obj : error LNK2001:  unresolved external symbol "__declspec(dllimport) public: __cdecl cpptrace::stacktrace::~stacktrace(void)" (__imp_??1stacktrace@cpptrace@@QEAA@XZ)

I managed to bypass this issue by setting CPPTRACE_STATIC_DEFINE before including CPPTrace. Maybe it will be good to document this
3. cpptrace was not working at first because there was a problem of initialization with the "SymInitialize " of dbghelp. I think that was maybe because I there was another component used that was calling this method beforehand (which should only be called once). Now I call directly the generate_trace method at the startup and it works better.(furthermore, this method should not be called in multi thread so it is better to make sure this is called directly once (which is not done by default in cpptrace)). I now also call SymRefreshModuleList  before generate_trace because I load dlls dynamically.
4. I am generating a string with the "to_string" method (on a "stack_trace"). But there was 0 (nulls) in the middle of the string produced (which was badly interpreted when given to my tracing method as it usually means end of string). I replaced all the nulls with ' ' and now it works fine.

Hi, thanks for taking the time to write these out. I'll try to go in order:

I haven't seen an issue with format, so I'll take a look
Yeah, sorry about that. CPPTRACE_STATIC_DEFINE is mentioned in the README, however perhaps not prominently enough. It's hard to figure out where to place it where people will actually see it.
Correct, SymInitialize must be called only once so if other components are calling it that's a problem. Unfortunately I don't know of a way to detect that it has already been called, other than maybe adding a method to cpptrace to let the user indicate it's already been done. Cpptrace's invocation of SymInitialize does use a lock in case of multithreading, however it's not possible to protect against non-cpptrace codepaths as far as I know.
This sounds like the same issue as #113 which is resolved in dev and will be part of the next release

Oh, and SymRefreshModuleList I'm not familiar with. What benefit does it bring?

What issue with format<> did you find? I'm not able to reproduce anything right off hand.

Hi, thanks for your answers.
1- Here is the error I got :
cl : Command line warning D9025 : overriding '/W3' with '/W4'
\cpptrace\src\v0.5.2-dcbc54c7ef.clean\src\utils\microfmt.hpp(316): error C2664: 'std::string microfmt::detail::format<0>(const char *,const char *,std::array<microfmt::detail::format_value,0>)': cannot convert argument 3 from 'initializer list' to 'std::array<microfmt::detail::format_value,0>'
\vcpkg-master\buildtrees\cpptrace\src\v0.5.2-dcbc54c7ef.clean\src\utils\microfmt.hpp(316): note: Invalid aggregate initialization
\vcpkg-master\buildtrees\cpptrace\src\v0.5.2-dcbc54c7ef.clean\src\utils\microfmt.hpp(215): note: see declaration of 'microfmt::detail::format'
\vcpkg-master\buildtrees\cpptrace\src\v0.5.2-dcbc54c7ef.clean\src\utils\error.hpp(25): note: see reference to function template instantiation 'std::string microfmt::format<>(const char *)' being compiled
\vcpkg-master\buildtrees\cpptrace\src\v0.5.2-dcbc54c7ef.clean\src\unwind\../utils/utils.hpp(455): note: see reference to function template instantiation 'cpptrace::detail::internal_error::internal_error<>(const char *)' being compiled

2- Indeed, now that you say it, I am able to find CPPTRACE_STATIC_DEFINE  in the ReadMe. But it is true I did not see this before.
3- I understand the problem with SymInitialize, I do not know a method to know if it has already been called either. As for SymRefreshModuleList , it allows to refresh the list of modules. If you call SymInitialize and then load a dll dynamically (with AfxLoadLibrary for instance) it will not be symbolized before you call SymRefreshModuleList . But it is maybe a bit out of the scope of your tool.
4- Glad to know it is already fixed.

Thanks for confirming! I've added additional notes about CPPTRACE_STATIC_DEFINE to the readme to help prevent them from being missed and v0.5.3 fixing the other issues is now released and available through package managers. I'll go ahead and close this for now, please let me know if there's anything else you run into!

Thanks !


Implementing CO_LOCK macro when using CANopenNode with FreeRTOS

Hi,
I am searching a proper way to implement LOCK/UNLOCK macro when using CANopenNode with FreeRTOS. Among projects in readme there are two options:

Use mutexes
A possible pitfail is that LOCKs can be called in interrupt context, so if we fail to acquire a mutex from interrupt we skip the guarded section
Disable/enable interrupts globally
This option could possible interfere with native FreeRTOS portENABLE_INTERRUPTS especially when called from interrupt.
To solve the problem I've tried to implement LOCKs over FreeRTOS functions

/* (un)lock critical section in CO_CANsend() */
#define CO_LOCK_GENERIC(CAN_MODULE, field)									\
	do{																		\
		if(xPortIsInsideInterrupt() == pdTRUE){								\
			(CAN_MODULE)->field = taskENTER_CRITICAL_FROM_ISR();			\
		}else{																\
			taskENTER_CRITICAL();											\
		}																	\
	}while(0)
// Why disabling the whole Interrupt
#define CO_UNLOCK_GENERIC(CAN_MODULE, field)								\
	do{																		\
		if(xPortIsInsideInterrupt() == pdTRUE){								\
			taskEXIT_CRITICAL_FROM_ISR((CAN_MODULE)->field);				\
		}else{																\
			taskEXIT_CRITICAL();											\
		}																	\
	}while(0)
#define CO_LOCK_CAN_SEND(CAN_MODULE) 	CO_LOCK_GENERIC(CAN_MODULE, primask_send)

#define CO_UNLOCK_CAN_SEND(CAN_MODULE)	CO_UNLOCK_GENERIC(CAN_MODULE, primask_send)

/* (un)lock critical section in CO_errorReport() or CO_errorReset() */
#define CO_LOCK_EMCY(CAN_MODULE)    	CO_LOCK_GENERIC(CAN_MODULE, primask_emcy)
#define CO_UNLOCK_EMCY(CAN_MODULE) 		CO_UNLOCK_GENERIC(CAN_MODULE, primask_emcy)

/* (un)lock critical section when accessing Object Dictionary */
#define CO_LOCK_OD(CAN_MODULE)  		CO_LOCK_GENERIC(CAN_MODULE, primask_od)
#define CO_UNLOCK_OD(CAN_MODULE) 		CO_UNLOCK_GENERIC(CAN_MODULE, primask_od)

/* Synchronization between CAN receive and message processing threads. */
#define CO_MemoryBarrier()				portMEMORY_BARRIER()

but sometimes I end up in HardFault_Handler because of PendSV interrupt. So it seems I did something really wrong. Could you please point me to the proper direction?
Best,
Serge

I've implemented the FreeRTOS port. No stack code runs inside an ISR, so the non-ISR variants of the FreeRTOS functions can be used safely. We use FreeRtos Queues to transfer the CAN rx messages from ISR context to the thread context.

Not sure if it is of any help. But we use a approach similar to yours @SergeTauger with critical sections, because we have interrupts with higher priority (that do not use FreeRTOS functions) that we do not want masked. You can view our implementation here: https://github.com/CANopenNode/CanOpenSTM32/pull/73/files#diff-be041ff98d9e03991ad2cba9902e3eabcd18ece7bb041e75da79cafc2ea8622d

No stack code runs inside an ISR, so the non-ISR variants of the FreeRTOS functions can be used safely.
That does depend on where you call canopen_app_interrupt from. In our implementation we call it from a timer interrupt.

My bad: the mistake was in setting wrong stack size of deferred interrupt processing task (yes, the most frequent FreeRTOS problem). The code on locks works just as expected and with no errors. Thanks to everyone for help.


Settings button appears only briefly when loading, then disappears

I was trying to create and use a theme, however I am unable to change it - I assume that the settings button is required for this. However, it only appears briefly while the page is loading and is unclickable, then disappears.
More info: Running in Docker on Debian, access in Chromium and Firefox, same behaviour with an empty or populated apps list

Hi!
This sounds very odd to me.
I've set it up so that the settings menu button only shows up when either themes.json or search.json or both could be loaded.
It'd be great if you could answer the following two questions for me:
What are the contents of your data-directory exactly (the one with all of the json files in it)?
Can you verify that your themes.json is valid JSON and reflects the one in the README?
My personal guess would be that there's something wrong with your themes.json file.
Kind regards
Bastian Meissner

This is the contents of my data directory:
apps.json  bookmarks.json.disabled  docker-compose.yml  greeter.json  themes.json
And this is the structure of my themes.json:
{
  "themes": [
    {
      "label": "test",
      "value": "1",
      "mainColor": "#666666",
      "accentColor": "#FF0000",
      "backgroundColor": "#00FF00"
    }
  ]
}

To my eye this reflects what's in the README, but I could be missing something.
I'm also not seeing anything useful in the web console, but I don't really know what I'm looking for

Out of interest: Try replacing "value": "1", with "value": 0,. Other than that I'm unsure as to what causes this behaviour. I'll look into it some more.

Still no luck. Let me know if there's any other information you need to help, I'll do my best to provide it

did you check the themes.json file? The same happened when themes file got issues.
can you replace themes.json file with themes.json and check ?

As mentioned earlier in the discussion, the themes.json was already validated. That example also yielded no solution

@MaxBayly In your earlier comment you don't mention a search.json file existing in your data directory. This would cause the issue of the flashing settings icon (as the search's defaults exist on first render until the app attempts to get the contents of the file, which in your case appears to not exist.)

So if I understand correctly, the ability to switch themes depends on enabling search providers as well? Is there an example of search.json anywhere? I couldn't manage to find any, and I don't really understand the example given

While currently the only configurable setting within the settings panel is the theme, the settings panel also shows the available search engines as well as your default search engine. You can find examples of the necessary files at the top-level data directory in this repo. Here's a link to search.json.

Cheers for that. Unfortunately, still no luck, not too sure what's going on

Any errors or warnings in the devtools console?

Actually now that I look at the repo, it looks like phntxx did a fairly major refactor last night. After a quick look at the changes, I think the latest version of the app should fix the issues you're seeing.

Just managed to run the new image and everything is looking good. Seems like the refactor worked its magic!

Glad to hear it!


captive portal page not  popups

after connecting to the Fake ssid
the login page is not poping up automatically.

after connecting to the Fake ssid
the login page is not poping up automatically.

problem solved by adding the id adress as 172.##.##.##


Last reportback item overrides all the items

Let's say theres 3 reportback items, you flag the first two and promote the third. If you check the database for that reportback, FLAGGED = 0 and FLAGGED_REASON = NULL
Example: http://staging.beta.dosomething.org/admin/reportback/3069

Flagged second RB item
Promoted third RB item

What the DB shows

I'm self assigning because on my local I've already fixed this

duplicate of #4847


docs: document release procedure

Developer documentation on the steps to perform a release.
The release procedure is semi-automated using Goreleaser.
There are some manual steps and guidelines which should be noted.

This now exists at: https://github.com/hbagdi/deck/blob/master/docs/development/release.md


Webpack 2 + includeSubModules

Hello guys, in the current version of skeleton-navigation the includeSubModules option in AureliaWebpackPlugin does not work anymore, I finally get it working changing my package.json:
...
"aurelia": {
    "build": {
      "resources": [
        ...
        "aurelia-dialog",
        "aurelia-dialog/resources/ai-dialog-body",
        "aurelia-dialog/resources/ai-dialog-footer",
        "aurelia-dialog/resources/ai-dialog-header",
        "aurelia-dialog/resources/ai-dialog",
        "aurelia-dialog/resources/attach-focus",
        ...
        "aurelia-authentication",
        "aurelia-authentication/authenticatedFilterValueConverter",
        "aurelia-authentication/authenticatedValueConverter",
        "aurelia-authentication/authFilterValueConverter"
      ],
      "moduleRootOverride": {
        "aurelia-dialog": "dist/es2015",
        "aurelia-authentication": "dist/es2015",
        ...
      }
    }
  },
...

This is the way it supposed to be?

Yes, that's good.
We'll need to update the readme: https://github.com/aurelia/dialog#configuring-and-using-with-webpack and add these above resources to package.json so users don't need to.
Thanks @talski.

@niieani Can this be closed now?

I think so, the resources seem to be in package.json.

Link no longer works.

npm install aurelia-dialog --save

aurelia.use.plugin("aurelia-dialog", (config) => {
 config.useDefaults();
})

And adding to package.json:

json
"aurelia": {
    "build": {
      "resources": [
        "aurelia-dialog",
        "aurelia-dialog/resources/ai-dialog-body",
        "aurelia-dialog/resources/ai-dialog-footer",
        "aurelia-dialog/resources/ai-dialog-header",
        "aurelia-dialog/resources/ai-dialog",
        "aurelia-dialog/resources/attach-focus",
      ]
    }
  }

produces:
Unable to resolve aurelia-dialog/resources/ai-dialog-body
Unable to resolve aurelia-dialog/resources/ai-dialog-footer
Unable to resolve aurelia-dialog/resources/ai-dialog-header
Unable to resolve aurelia-dialog/resources/ai-dialog
Unable to resolve aurelia-dialog/resources/attach-focus

for skeleton-typescript-webpack.


Add values schema to the k6-operator helm chart

Feature Description
Hey, could we add the values.schema.json to the helm chart? We are working on a tool that renders UIs for helm charts based on their schema and would love to support the k6-operator chart as well.
Suggested Solution (optional)
Add the values.schema.json to the chart. Initially, the schema can cover just the basic set of fields, so it is easier to maintain.
Already existing or connected issues / PRs (optional)
No response

Hi @petar-cvit, sounds great! Thanks for the issue. And if the schema can be used in your tool, then of course, we can add it.
The one thing that is worrying here is how to maintain this long-term :sweat_smile: I cannot promise additional commitment to maintaining this myself. Can I ask you, if the values.schema.json needs to be written by hand or there's another way? I.e. could this schema be generated somehow? I've just found this tool:
https://github.com/losisin/helm-values-schema-json
It seems to work with comment annotations... Would it be sufficient to update the schema file with CI?
I think it'd be "ideal" if it were possible to just re-generate the schema when someone makes a change to the Chart :thinking:

Hey @yorugac, we did a similar thing with cerbos. Here is the PR adding a schema on their repo https://github.com/cerbos/cerbos/pull/2230/files
They used github.com/dadav/helm-schema to generate the schema from the values file based on the comments (the same way you described it). If you want, I can look into setting it up on your repo and open a PR

@petar-cvit, yes, I think that'd be great! Make command to generate the schema that can be called then in CI :+1:


[Remote Test] Fix remote-run to process environment variables.

remote-run should look in the environment for input/output paths as well as considering command line arguments.
With this change, test/Runtime/Paths.cpp should work for remote testing and device testing.
rdar://106294557

@swift-ci Please smoke test


additional UnboundLocalError when lambda evaluation throws exception

Snakemake 5.10.0.  Snakefile:
rule: "test"
    output: "{runtime}"
    resources:
        runtime=lambda wildcards: runtime//2 # deliberate type error

Running snakemake 4 correctly reports the type error (runtime is a str, not a numeric type), but also reports an additional exception:
During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/snakemake/__init__.py", line 633, in snakemake
    keepincomplete=keep_incomplete,
  File "/usr/local/lib/python3.6/dist-packages/snakemake/workflow.py", line 850, in execute
    success = scheduler.schedule()
  File "/usr/local/lib/python3.6/dist-packages/snakemake/scheduler.py", line 360, in schedule
    run = self.job_selector(needrun)
  File "/usr/local/lib/python3.6/dist-packages/snakemake/scheduler.py", line 503, in job_selector
    a = list(map(self.job_weight, jobs))  # resource usage of jobs
  File "/usr/local/lib/python3.6/dist-packages/snakemake/scheduler.py", line 577, in job_weight
    res = job.resources
  File "/usr/local/lib/python3.6/dist-packages/snakemake/jobs.py", line 268, in resources
    self.wildcards_dict, self.input, self.attempt
  File "/usr/local/lib/python3.6/dist-packages/snakemake/rules.py", line 941, in expand_resources
    resources[name] = apply(name, res, threads=threads)
  File "/usr/local/lib/python3.6/dist-packages/snakemake/rules.py", line 926, in apply
    except e:
UnboundLocalError: local variable 'e' referenced before assignment

Thanks a lot for reporting and sorry for the late response. Snakemake is currently entering a stabilization phase and I want to ideally tackle all open bugs. This one is fixed since some time already.


ci: update postgres to major v 16

Can PR this to dev if #145 is merged first

I don't know if this will cause any problems. It didn't seem to on my end, but if any happen we need to know.

Do we have a good reason to do this upgrade?

Not really besides probably better performance
https://www.postgresql.fastware.com/blog/discussing-postgresql-what-changes-in-version-16
The version we're currently using in dev is 2 years old though. #145 updates it to the most recent v14 but v16 would be nice for parity with Vilsol's migration of the SMR stuff to 16
We're currently using 13 in prod :/


Editing contents of a  causes the designer to swap back to the first

Normally in WPF (I believe) when editing the XAML contents of a <TabItem> the designer will switch to that tab so you can see the controls you are adding. I can also switch to this tab by clicking on it in the designer.
In Avalonia, I can click on the tab in the designer but the moment I start modifying the XAML contents of that tab, the designer swaps back to the first tab.

@ForNeVeR @ShadowDancer I guess we could send previewer a message with the cursor position. That way it could identify the active element and activate it in TabControl accordingly

@rstm-sf, isn't it something that you were preparing for the web previewer?

As far as I understand, no. I was just doing mouse handling on preview. Here is a request for the editable element in xaml to be activated in preview

The web part has nothing to do with previewer<->editor communication, it's done via a separate tcp connection.

Why do you think that editor communication is important here? As I understand, this is about keeping control state between XAML updates in previewer. Though, if editor could help with this, them I'm all ears.

when editing the XAML contents of a <TabItem> the designer will switch to that tab

Ah, okay, you're right. Yes, it would be very easy to send current caret position to the previewer. Please consider that all the Avalonia-compatible editors (including VS, VSCode, Rider) have some sort of multi-caret, so it would be cool to be able to highlight multiple carets in future (though this is not that important).

I think this is a bit of over-engineering, because - as I understand it - there would be special concept for tab control, to change tabs based on what it sees in designer. However I think this is not really robust, because next things will soon follow, like scroll scrollviewer to edited item etc.
I think what we really need here is Xaml hot-reload, so user may choose tabitem he wishes, and then if he subtree, only that subtree is reloaded in previewer. Do we have plans to support it?

there would be special concept for tab control, to change tabs based on what it sees in designer

BringIntoView


zeptar.mod.land

[x] My submissions follows the Submission Rules
[x] I have read and accepted the Terms and Conditions

Origin website: https://zeptar.vercel.app

Waiting..

Why is this closed?


Consider temporary instances

Current setup considers instances that are in running state when the daily process executes. This leaves out of the aggregation temporary instances like spot instances and instances from autoscaling groups.

Possible solution could be changing de behavior of the Get Instances Lambda to get data directly from CloudWatch CPU Utilization metrics, rather than from EC2 running instances.


small problem

it works, but when i use a macro creator to see the inputs, if you hold a then press d and never let go of a, the a will get pressed again for a very short time after the d input. idk how impactful this is.

I'm not sure I understand what I'm looking at. You would definitely be able to see that in game if a wrong input were getting pressed.

When the d button is pressed, on the side it shows the down arrow, you can see that the a is pressed 0.001 seconds after the d button is pressed

i will test it in game though

Ah, I see. This shouldn't happen, let me know if you see issues in games. Maybe this macro software does something weird, or maybe I actually have a bug :)

idk it doesnt show in game but the macro software shows it. i dont think this is a problem


Custom styling

Is there a way to customize the colors of the data browser?
ideally I would also love to customize the "logo" part

All the styling is baked into the react app so currently the only way would be to modify and rebuild the react app, the process would be.
0: fork it
1: alter the css
2: run npm install in the frontend dir
2: run the build_fe script
3: commit the changes
4: install from your fork
In general I'm not opposed to making this more customizable. It's just a question of how much and how.

All the styling is baked into the react app so currently the only way would be to modify and rebuild the react app, the process would be.

i thought it would be like that

In general I'm not opposed to making this more customizable. It's just a question of how much and how.

Good question
How much?
I would propose to be able to

set a custom stylesheet
adjust those "title" values (DDB v3.1.1)
set the icon(s) / favicons

How ?
Probably all the variables could be defined over settings.py variable(s)?
DDB_SETTINGS = {
  css_path : 'path/to/file.css',
  title_text: 'Custom title',
  version_text: ''
}

@tolomea
I was able to change the styling with the following basic steps:

override template at  templates/data_browser/index.html
create a custom css and add it to the html file

but still it would be nice if the databrowser would use css classes or ids for the different parts (e.g. the saved views on the home page)
this would make style adjustments less hacky, now I had to do something like:
div.Index > div {
    background: white;
}

You will want to pin the specific version of data browser. I believe you will have to update it every time the version shifts.
The CSS classes is an interesting point, so far I've just been adding the stuff I needed to get it styled, and the more dynamic styling is inline in the react code.

nts: it should be possible to use django templating to include another css file, but any form of compilation for that file will be a host developer problem
and maybe I should have a standard practice of the top element in react component having a class matching the component name

it should be possible to use django templating to include another css file, but any form of compilation for that file will be a host developer problem

there is no compilation needed if the .css is added to the "index.html" in the templates folder 😃

and maybe I should have a standard practice of the top element in react component having a class matching the component name

i guess that would support / ease the setup for a custom css
does the index.html change if you change something in the library?

the source for it is here https://github.com/tolomea/django-data-browser/blob/master/frontend/public/index.html
that goes into the JS build process which results in the one you see here https://github.com/tolomea/django-data-browser/blob/master/data_browser/templates/data_browser/index.html
AFAIK the JS build process mostly replaces the

 with the webpack load guff, that has hardcoded references to webpack chunks like ./static/js/main.71e3e868.chunk.js which are going to change on every JS build
that said django template variables survive this process and I used django templating to inject the config(aka contex) info into the page, so I could in theory add a css include the same way

ok thank you for the detailed explanation
would django template "blocks" survive this procedure? 🤔  if yes they might be the solution to add custom css/head data or change the code around the

or are there better solutions for that? 🤔

All the styling is baked into the react app so currently the only way would be to modify and rebuild the react app, the process would be.
0: fork it
1: alter the css
2: run npm install in the frontend dir
2: run the build_fe script
3: commit the changes
4: install from your fork
In general I'm not opposed to making this more customizable. It's just a question of how much and how.

but what if i want to check if i like the style before the build? i mean, how can i start the development server from this react app to watch it in real time? can you help us?

Instructions for running the JS dev server are available here https://github.com/tolomea/django-data-browser/blob/master/CONTRIBUTING.rst#development

@bb4L I added an extrahead block and tested to make sure I could use it to inject some CSS, instructions are in the CSS, I will spin a 3.1.4 release with this in, I also tidied up some of the CSS class name stuff, so there's more class names in the code and they are more consistent in naming. Let me know how that goes.

@tolomea
Thank you for the update.
I will test it and let you know how it goes.
As far as I remember I also would have needed a block around the root div.
Then I would have been able to embed it into the existing page (e.g. with the same navbar/ footer)

Ah, I hadn't thought of it like that, changes the way you look at it a bit.
On Tue, Dec 22, 2020 at 10:09 AM L P<EMAIL_ADDRESS>wrote:

@tolomea https://github.com/tolomea
Thank you for the update.
I will test it and let you know how it goes.
As far as I remember I also would have needed a block around the root div.
Then I would have been able to embed it into the existing page (e.g. with
the same navbar/ footer)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tolomea/django-data-browser/issues/24#issuecomment-749461678,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGERUAB3MDKLVSNUFLG7PDSWBV45ANCNFSM4USDQAJQ
.

Sorry for that.
I probably wasn't precise enough.

Ah, I hadn't thought of it like that, changes the way you look at it a bit.
…
On Tue, Dec 22, 2020 at 10:09 AM L P @.***> wrote: @tolomea https://github.com/tolomea Thank you for the update. I will test it and let you know how it goes. As far as I remember I also would have needed a block around the root div. Then I would have been able to embed it into the existing page (e.g. with the same navbar/ footer) — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#24 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAGERUAB3MDKLVSNUFLG7PDSWBV45ANCNFSM4USDQAJQ .

There is lot of stuff that could potentially be done here, so I'd like to start by putting some limits on the scope.
1: Right off I'm not interested in trying to support anything resembling splitting up / rearranging the core app. Particularly the triplet of index, query and saved query pages.
2: I am partial to supporting that core app being nested within other stuff.
3: As for customizing the style of the app itself, I'd like it to be possible, but there are limits on how much effort I want to expend on accommodating it. An example might help, I use particular fonts and font sizes and I will write the CSS assuming that those are the fonts in use. It should be possible for you to replace the fonts, but any collateral styling damage from that is your problem.
Maybe I'm overthinking this, I'm not a frontend dev, but my current thought it is roughly I want to support both:
1: Replace base.html entirely, import font-head to get the font links
2: Override just the title and favicons, or inject something into the body around the ddb-app etc
I think that will require the following changes:
1: split index.html into base.html, font-head.html and index.html.
1a: Index will extend base and will have only the core app and it's associated data.
1b: font-head will have the links for the google fonts and will be included in the head section of base
2: move the logo from the JS to pure html in base
3: scope all of my css so it only applies inside the logo or the core app
4: base needs blocks for head, extra-head, favicon-head, title, body, ddb-logo and ddb-app(index implements this)
How does that sound to you?
Is there anything missing?
Is any of it overkill?

@tolomea  sorry for the late reply

1: Right off I'm not interested in trying to support anything resembling splitting up / rearranging the core app. Particularly the triplet of index, query and saved query pages.

I completely understand, this shouldn't be needed

2: I am partial to supporting that core app being nested within other stuff.

happy to hear that

3: As for customizing the style of the app itself, I'd like it to be possible, but there are limits on how much effort I want to expend on accommodating it. An example might help, I use particular fonts and font sizes and I will write the CSS assuming that those are the fonts in use. It should be possible for you to replace the fonts, but any collateral styling damage from that is your problem.

I can agree / understand on that point as well

I think that will require the following changes:
1: split index.html into base.html, font-head.html and index.html.
1a: Index will extend base and will have only the core app and it's associated data.
1b: font-head will have the links for the google fonts and will be included in the head section of base
2: move the logo from the JS to pure html in base
3: scope all of my css so it only applies inside the logo or the core app
4: base needs blocks for head, extra-head, favicon-head, title, body, ddb-logo and ddb-app(index implements this)

Yes that all makes sense, in terms of the body it might be tricky since one would want to override / add classes to the body tag
and change/ embed the "root" div inside some custom html.
eg:

....

....

3: scope all of my css so it only applies inside the logo or the core app

I guess this is more a "nice to have" as long as a user is able to completely override the styling

@tolomea  sorry for the late reply

1: Right off I'm not interested in trying to support anything resembling splitting up / rearranging the core app. Particularly the triplet of index, query and saved query pages.

I completely understand, this shouldn't be needed

2: I am partial to supporting that core app being nested within other stuff.

happy to hear that

3: As for customizing the style of the app itself, I'd like it to be possible, but there are limits on how much effort I want to expend on accommodating it. An example might help, I use particular fonts and font sizes and I will write the CSS assuming that those are the fonts in use. It should be possible for you to replace the fonts, but any collateral styling damage from that is your problem.

I can agree / understand on that point as well

I think that will require the following changes:
1: split index.html into base.html, font-head.html and index.html.
1a: Index will extend base and will have only the core app and it's associated data.
1b: font-head will have the links for the google fonts and will be included in the head section of base
2: move the logo from the JS to pure html in base
3: scope all of my css so it only applies inside the logo or the core app
4: base needs blocks for head, extra-head, favicon-head, title, body, ddb-logo and ddb-app(index implements this)

Yes that all makes sense, in terms of the body it might be tricky since one would want to override / add classes to the body tag
and change/ embed the "root" div inside some custom html.
eg:

....

....

3: scope all of my css so it only applies inside the logo or the core app

I guess this is more a "nice to have" as long as a user is able to completely override the styling

4: I believe for that usecase you can override the base template with another base that extends the original and implements the body block and then have the ddb-app block inside your body block. This essentially lets you replace the whole body but keep everything else.
3: Because of the way my CSS is built into the React chunks I don't think you are going to be able to not have my CSS loaded and loaded late, so getting round it is going to require being more specific / important with selectors. I'm ok with that inside the data browser components, but it seems a lil harsh for the global changes I make.

4: I believe for that usecase you can override the base template with another base that extends the original and implements the body block and then have the ddb-app block inside your body block. This essentially lets you replace the whole body but keep everything else.
3: Because of the way my CSS is built into the React chunks I don't think you are going to be able to not have my CSS loaded and loaded late, so getting round it is going to require being more specific / important with selectors. I'm ok with that inside the data browser components, but it seems a lil harsh for the global changes I make.

4: I believe for that usecase you can override the base template with another base that extends the original and implements the body block and then have the ddb-app block inside your body block. This essentially lets you replace the whole body but keep everything else.

that's correct but the script tag's are usually inside the body
i'm not sure if a can access those unless they are specified inside a seperate html? 🤔

3: Because of the way my CSS is built into the React chunks I don't think you are going to be able to not have my CSS loaded and loaded late, so getting round it is going to require being more specific / important with selectors. I'm ok with that inside the data browser components, but it seems a lil harsh for the global changes I make.

Yes that's fine, it doesn't really matters as long as i can override/add a stylesheet after it 😇

4: I believe for that usecase you can override the base template with another base that extends the original and implements the body block and then have the ddb-app block inside your body block. This essentially lets you replace the whole body but keep everything else.

that's correct but the script tag's are usually inside the body
i'm not sure if a can access those unless they are specified inside a seperate html? 🤔

3: Because of the way my CSS is built into the React chunks I don't think you are going to be able to not have my CSS loaded and loaded late, so getting round it is going to require being more specific / important with selectors. I'm ok with that inside the data browser components, but it seems a lil harsh for the global changes I make.

Yes that's fine, it doesn't really matters as long as i can override/add a stylesheet after it 😇

that's correct but the script tag's are usually inside the body

Yes, they are immediately before and after what is currently the "root" div (will be the ddb-app div) I intend for that div and the script tags to end up in the ddb-app block which will get defined by index.html, so as long as you keep the placeholder block when you override base.html I think it should be good, but I will verify that when I make the changes

that's correct but the script tag's are usually inside the body

Yes, they are immediately before and after what is currently the "root" div (will be the ddb-app div) I intend for that div and the script tags to end up in the ddb-app block which will get defined by index.html, so as long as you keep the placeholder block when you override base.html I think it should be good, but I will verify that when I make the changes

Yes, they are immediately before and after what is currently the "root" div (will be the ddb-app div) I intend for that div and the script tags to end up in the ddb-app block which will get defined by index.html, so as long as you keep the placeholder block when you override base.html I think it should be good, but I will verify that when I make the changes

alright that sounds like it should work :-)
thank you very much

Yes, they are immediately before and after what is currently the "root" div (will be the ddb-app div) I intend for that div and the script tags to end up in the ddb-app block which will get defined by index.html, so as long as you keep the placeholder block when you override base.html I think it should be good, but I will verify that when I make the changes

alright that sounds like it should work :-)
thank you very much


Added Get All Computes endpoint.

Description
Added get all computes endpoint.
Please provide a meaningful description of what this change will do, or is for. Bonus points for including links to
related issues, other PRs, or technical references.
Note that by not including a description, you are asking reviewers to do extra work to understand the context of this
change, which may lead to your PR taking much longer to review, or result in it not being reviewed at all.
Type of Change

[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)

Checklist

[ ] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[ ] I have verified this change is not present in other open pull requests
[ ] Functionality is documented
[ ] All code style checks pass
[ ] New code contribution is covered by automated tests
[ ] All new and existing tests pass

Codecov Report
Attention: 28 lines in your changes are missing coverage. Please review.

Comparison is base (0a472f9) 14.95% compared to head (6d346a0) 14.80%.

Files
Patch %
Lines

cmd/controller/app/database/mongodb/compute.go
0.00%
28 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #473      +/-   ##
==========================================
- Coverage   14.95%   14.80%   -0.16%
==========================================
  Files          49       49
  Lines        2735     2763      +28
==========================================
  Hits          409      409
- Misses       2299     2327      +28
  Partials       27       27

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


No service for type 'Microsoft.AspNetCore.Hosting.IHostingEnvironment' has been registered

I have a problem when I want to fire add-migration command for an ASP MVC Core 2 project.

No service for type 'Microsoft.AspNetCore.Hosting.IHostingEnvironment' has been registered.

You can see more information in stackoverflow

Well, I tried to point you in the right direction, but my answer got deleted... Here's the exact code you can copy and paste into your application.
public class DesignTimeDbContextFactory : IDesignTimeDbContextFactory<SampleArchContext>
{
    public SampleArchContext CreateDbContext(string[] args)
    {
        var host = new WebHostBuilder()
            .UseKestrel()
            .UseContentRoot(Directory.GetCurrentDirectory())
            .ConfigureAppConfiguration((hostingContext, config) =>
            {
                var env = hostingContext.HostingEnvironment;
                config.SetBasePath(env.ContentRootPath);
                config.AddInMemoryCollection(new[]
                       {
                         new KeyValuePair<string,string>("the-key", "the-value")
                       })
                       .AddJsonFile("appsettings.json", reloadOnChange: true, optional: false)
                       .AddJsonFile($"appsettings.{env}.json", optional: true)
                       .AddEnvironmentVariables();
            })
            .ConfigureLogging((hostingContext, logging) =>
            {
                logging.AddDebug();
                logging.AddConsole();
            })
            .UseIISIntegration()
            .UseDefaultServiceProvider((context, options) =>
            {
                options.ValidateScopes = context.HostingEnvironment.IsDevelopment();
            })
            .UseStartup<Startup>()
            .Build();

        return host.Services.CreateScope().ServiceProvider.GetRequiredService<SampleArchContext>();
    }
}

If you follow the guidance I pointed to and split the top half of your Program.Main() into a BuildWebHost() method, EF will do the rest for you (i.e. you could remove the custom IDesignTimeDbContextFactory implementation).
To directly answer you're question, you can't inject services into IDesignTimeDbContextFactory. It is a static entry point into the application (like Program.Main()) and the service container must be configured from scratch.

I used my omnipotent SO privileges to undelete my answer and expand it a bit.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


No Cocoapods support for iOS 7

The requirements say you support iOS 7 but the pod only supports iOS 8.
Is this right? Does that mean to support iOS 7 I have to include the files manually?

I am also wondering about this.

Yes, you have to include the files manually.

Is that because of how SwiftyJSON is bundled for cocoapods? I was reading up a bit and it sounds like there is a new framework type that only ios8+ can use... is SwiftyJSON using that?

Yes, because the cocoapods is using that type.

So... including SwiftyJson with a pod wont work with IOS 7 ? but if I include SwiftyJson without a pod, it will?

That's correct, although the reasons are a bit nuanced (it has to do with how a swift project must be built in order to be managed by cocoapods, IIRC).


Public load balancer was placed into virtual network. Please move it out.

[Enter feedback here]

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

ID: dd59e134-be71-0ac8-bb71-abebb2ef16f5
Version Independent ID: a99d6a21-1c49-6bfc-9572-f7dfe8be8a4c
Content: What is Azure Load Balancer? - Azure Load Balancer
Content Source: articles/load-balancer/load-balancer-overview.md
Service: load-balancer
GitHub Login: @asudbring
Microsoft Alias: allensu

@SingularityPeru , could you please provide some more details on the issue for further review?

@SingularityPeru, I was wondering if you have had a chance to look over our previous reply yet.

@SingularityPeru, since we have not heard back from you, we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly reopen the issue and continue the discussion.


[#1358] Add tests for Label component

Fixes #1358

@angle943 @thesanjeevsharma
I'm quite new in writing tests, let me know if there's anything that can be improved on.
In particular, for some tests I placed multiple expect calls as I believe that it might be unnecessary to create separate tests for them.

@jamessspanggg this looks good to me! I did a minor refactor


Template undefined not found! (latest NMDC DH LinkML file vs linkml datastructure branch)

I just

fetched the latest cidgoh/DataHarmonizer
checked out the linkml-datastructure branch
copied https://raw.githubusercontent.com/microbiomedata/sheets_and_friends/issue-92-pipe-sep/artifacts/nmdc_dh.yaml into template/nmdc_dh/source/
ran make all

When I opened linkml.html, the following appeared in the Template status box:

Template undefined not found!

And when I clicked on the Template status box, I got an empty Change Template pull down menu

Yes, the very latest commit leaves some broken things.  I'll get the menu system back in order shortly. If you walk it back to previous commit you will see menu working, but none of the global variables have been eliminated, or any API work which is what the latest WIP commit has in it.

OK. I can wait. Thanks.

Hmm, I'm just trying it too.  I see from a run I started last night that it has an error of some kind on the  /nmdc_dh.yaml file.  It seems to get through the MIxS mixs.yaml spec no problem. Something to do with adding "rank"?
Key error is: ValueError:  Unknown argument: rank = 1
cd template/nmdc_dh ; python ../../script/linkml.py --input ../../template/nmdc_dh/source/nmdc_dh.yaml
Loading LinkML specification for ../../template/nmdc_dh/source/nmdc_dh.yaml
Traceback (most recent call last):
  File "../../script/linkml.py", line 75, in <module>
    mixs_sv = SchemaView(options.linkml_file)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/utils/schemaview.py", line 99, in __init__
    schema = load_schema_wrap(schema)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/utils/schemaview.py", line 54, in load_schema_wrap
    schema = yaml_loader.load(path, target_class=SchemaDefinition, **kwargs)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/loaders/yaml_loader.py", line 23, in load
    metadata=metadata)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/loaders/loader_root.py", line 62, in load_source
    target_class(**as_dict(data_as_dict)) if data_as_dict is not None else None
  File "<string>", line 48, in __init__
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/linkml_model/meta.py", line 390, in __post_init__
    self._normalize_inlined_as_dict(slot_name="slots", slot_type=SlotDefinition, key_name="name", keyed=True)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/utils/yamlutils.py", line 100, in _normalize_inlined_as_dict
    self._normalize_inlined(slot_name, slot_type, key_name, keyed, False)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/utils/yamlutils.py", line 200, in _normalize_inlined
    form_1({k: v})
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/utils/yamlutils.py", line 153, in form_1
    order_up(key, slot_type(**as_dict(raw_obj)))
  File "<string>", line 69, in __init__
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/linkml_model/meta.py", line 736, in __post_init__
    super().__post_init__(**kwargs)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/linkml_model/meta.py", line 544, in __post_init__
    super().__post_init__(**kwargs)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/linkml_model/meta.py", line 309, in __post_init__
    super().__post_init__(**kwargs)
  File "/Users/damion/anaconda3/lib/python3.7/site-packages/linkml_runtime/utils/yamlutils.py", line 48, in __post_init__
    raise ValueError('\n'.join(messages))
ValueError:  Unknown argument: rank = 1
make: *** [template/nmdc_dh/schema.js] Error 1

Its pretty quick to get to this error though for me, just seconds.  Do you folks have an updated nmdc_dh.yaml  file?

Key error is: ValueError: Unknown argument: rank = 1

Thanks. That is a newsish term in the meta-modelling language. You shouldn't get any more errors like that if you update linkml in your environment.

running script/linkml.py against our nmdc_dh.yaml file works now. Thanks.


Invalid JPEG data or crop window

I got an error when training:
Traceback (most recent call last):
File "main.py", line 132, in
main()
File "main.py", line 120, in main
gan.train()
File "/content/BigGAN-Tensorflow/BigGAN_128.py", line 302, in train
_, summary_str, d_loss = self.sess.run([self.d_optim, self.d_sum, self.d_loss])
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/client/session.py", line 956, in run
run_metadata_ptr)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/client/session.py", line 1180, in _run
feed_dict_tensor, options, run_metadata)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/client/session.py", line 1359, in _do_run
run_metadata)
File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/client/session.py", line 1384, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError:  Invalid JPEG data or crop window, data size 32768
[[{{node DecodeJpeg}}]]
[[IteratorGetNext]]
[[RemoteCall]]
[[IteratorGetNext]]
Could you figure out for me? thanks

look like i have read the file that is not proper .jpeg


Change RedesignButton components to styled API for proposed migration.

Type of change

[x] Migration Proposal

Description
We currently use the makeStyles() API for the bulk of our styling. It seems that MUI is pushing for users to migrate to the styled/sx solution provided by the default Emotions styling engine.
Migrating the entire application would probably take quite a bit of time, and to be honest I'm not sure what the ramifications of doing so would be. As a tiny proof-of-concept, I've migrated the Primary and Secondary button components to the proposed format.
Screenshots

In the above screenshots, we have a few buttons which use the RedesignButtonPrimary and RedesignButtonSecondary components. Ideally, I think we should be able to match up our UI kit with Design's kit in Figma. That capability should be provided by the styled() API. From what I understand, styled() simply extends the base MUI component, so we can use the sx prop in our custom components to override or extend styles, which is seemingly not possible with the makeStyle() way of doing things. This helps to easily create responsive components or make simple overrides, such as setting the width of the Home Page Hero CTA to 100%.
Hopefully, the result is that it's easier to stick to a unified design both while maintaining and adding to the site!
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (if applicable)
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works (if applicable)
[x] New and existing unit tests pass locally with my changes

These changes also fulfill Ticket 2020. The buttons under get involved do not match the ticket specs, but I've spoken to Nicholas from Design and he agreed that the buttons on the Get Involved cards should use the primary style.

It shouldn't prevent us from creating reusable styles, it would just be done in a different way. I'm still in the research phase, but from scouring MUI and Emotion docs, I believe it would break down to three methods for styling.

Creating a custom theme using MUI's createTheme(). This would define most of the high level styles.
Creating styled, reusable components using the styled() method.
Implementing one-off/responsive styling using the sx prop.

It seems like the hierarchy of it breaks down pretty well!

Great work! You are right, this would be a big overhaul and migration since we have style files all over the place. I have used the sx prop before, but I'm hesitant to start using it consistently because I feel like this could potentially prevent us from being able to reuse styles, like how we can create reusable style classes with makeStyles. What do you think?

Couldn't most reusable styles be standardized in the Theme? sx can then access those theme props, like this example. Also with a well-designed component architecture I wouldn't expect a ton of reused bespoke styles across components.

You're right that we should be able to define most design choices using createTheme(). The styled() utility seems to be mainly recommended when creating completely custom components, but with createTheme() we can overwrite the style of existing MUI components (and create variants) at a global level. So in hindsight, my use of the styled() utility here would probably be better suited by overwriting the MUI Button component. Put another way, how can we use MUI to its fullest potential without hand-rolling a ton of functionality?
With this in mind, part of the migration would probably involve going through the components that have been custom-built, converting them to their closest MUI equivalent, and then overwriting their style in createTheme() as necessary. sx could then be used to, say, make a Button fill its container width.


Feature/minor fix in documentation

Closing this PR in favour of a much better PR
https://github.com/TalentLayer/talentlayer-sdk/pull/4
This PR is no longer desirable.  It is thus forth an outcast


Roll engine 6991dd981bf7..ca8bb0a92328 (14 commits)

https://github.com/flutter/engine/compare/6991dd981bf7...ca8bb0a92328
git log 6991dd981bf7..ca8bb0a92328 --first-parent --oneline
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/skia 964aa91580b6..4036cb1f0b6e (5 commits) (flutter/engine#16912)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/skia ae2da5e7f9b5..964aa91580b6 (4 commits) (flutter/engine#16911)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/skia b58098f34c56..ae2da5e7f9b5 (3 commits) (flutter/engine#16910)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/dart c07d5d84b828..e7e45599cb1a (9 commits) (flutter/engine#16909)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/skia 96c2eb6258ae..b58098f34c56 (2 commits) (flutter/engine#16908)
2020-03-03<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from uiAI5... to ULy6Z... (flutter/engine#16907)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/skia 853789cdfe3c..96c2eb6258ae (3 commits) (flutter/engine#16906)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/skia b19408040143..853789cdfe3c (1 commits) (flutter/engine#16905)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/dart 06155d499645..c07d5d84b828 (3 commits) (flutter/engine#16904)
2020-03-03<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from WmA2M... to cGxwD... (flutter/engine#16903)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/dart ca3ad264a649..06155d499645 (3 commits) (flutter/engine#16902)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/skia 73ae40a424fa..b19408040143 (4 commits) (flutter/engine#16901)
2020-03-03<EMAIL_ADDRESS>Drop last usages of Dart_New from engine (flutter/engine#16838)
2020-03-03<EMAIL_ADDRESS>Roll src/third_party/skia 8121d27b297c..73ae40a424fa (1 commits) (flutter/engine#16900)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Closing to pick up next roll -- this one still contains the windows crasher.

Commit queue failed; closing this roll.


Enhance chispend presentation readme file

The chispend presentation readme file explains us about the integration of the wallet with just showing a single image and taking us to canva to explain with 5 slides .
I would like to enhance the readme I will explain the whole integration of wallet part in the readme by adding all needed images and their information below it as well.
If you like my idea please assign me the issue @phyleria

@phyleria please have a look

Hi @ayan-joshi,
All the suggestions on the presentation have already been implemented so this issue might be irrelevant.
But there are names that should be updated i.e Chispend to Unispend. I'll create an issue on that and you can claim it.

Here's the issue @ayan-joshi
https://github.com/Chimoney/chimoney-community-projects/issues/267

Here's the issue @ayan-joshi
#267

Thanks I have messaged there u can assign it


handle id dereference

fix #136

Pull Request Test Coverage Report for Build 367

15 of 17 (88.24%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 92.572%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/pointer.js
12
14
85.71%

Totals

Change from base Build 364:
-0.07%

Covered Lines:
7495

Relevant Lines:
7920

💛  - Coveralls

This is a really solid PR -- great work @mathieudutour. Would also love to see this merged.

Are there plans to merge this?  I'm impacted by this issue and would also love to see it merged.

$id/$ref is a pretty wild topic so we'd need a lot more input from the JSON Schema experts, and that one already has a pretty solid roadmap for 2019-09 support in #145.
Maybe we should focus our efforts on that instead?

Hi, im needing this too, any ETA?

@unlimitechcloud could you try this PR out and see if it works for your use cases?

Conflicts and original branch has gone. This is being handled in https://github.com/APIDevTools/json-schema-reader

Conflicts and original branch has gone. This is being handled in https://github.com/APIDevTools/json-schema-reader


Support Client Assertion authentication on authentication client

🔧 Changes
Introduces support for using Client Assertion  to the authentication client, this is for use when the Authentication Method is set to Private Ket JWT. This is supported using the WithClientAssertion option that can be passed instead of the existing WithClientSecret
jwtPrivateKey := `-----BEGIN PRIVATE KEY-----
.......
-----END PRIVATE KEY-----`

api, err := New(
	context.Background(),
	domain,
	WithClientID(clientID),
	WithClientAssertion(jwtPrivateKey, "RS256"),
)

if err != nil {
	log.Fatal(err)
}

tokenSet, err := api.OAuth.LoginWithClientCredentials(context.Background(), oauth.LoginWithClientCredentialsRequest{}, oauth.IDTokenValidationOptions{})

if err != nil {
	log.Fatal(err)
}

log.Print(tokenSet)

📚 References

🔬 Testing

📝 Checklist

[x] All new/changed/fixed functionality is covered by tests (or N/A)
[x] I have added documentation for all new/changed functionality (or N/A)

Codecov Report
Patch coverage: 64.42% and project coverage change: -0.39% :warning:

Comparison is base (1f5853c) 94.89% compared to head (4f1ea30) 94.51%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #260      +/-   ##
==========================================
- Coverage   94.89%   94.51%   -0.39%
==========================================
  Files          46       46
  Lines        8441     8526      +85
==========================================
+ Hits         8010     8058      +48
- Misses        337      363      +26
- Partials       94      105      +11

Files Changed
Coverage Δ

authentication/passwordless.go
81.48% <53.12%> (-18.52%)
:arrow_down:

authentication/oauth.go
82.58% <66.66%> (-9.98%)
:arrow_down:

authentication/authentication.go
89.65% <100.00%> (+0.12%)
:arrow_up:

authentication/authentication_option.go
93.61% <100.00%> (+0.75%)
:arrow_up:

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