fix for #64

This is my attempt to implement the optimizations mentioned in the issue. I assumed it was also safe to remove the 'in' check from 'getPositiveSpacing' as well because if it is null then will not be >= 0 I would be happy to make any requested changes if this isn't quite right.
Thanks!

Hey,
I will gladly make this safer then time the tests and  give you the output. I should have thought out my changes a bit more thoroughly, sorry about that!

Also would making it  look like
return node.style['marginTop'] || node.style['margin'] || 0;
work if I excluded the return 0 on exiting the loop? Otherwise I can add check for being undefined.
Also for the tests do I use the RunLayoutRandomTests.html to do this?  Or the tests that are in the tests dir?

Also, opening RunLayoutRandomTests.html before my changes finishes in about 2.2 seconds on chrome and after it finishes in about 1 second

Hey so after working on this for a while yesterday I found that there wasn't much of a performance improvement on my machine, and that the changes caused a slight increase in the number of failures on the layout tests. Thanks for taking a look at what I had. I will close the PR.


Rework Test Drive and Quick Start

- Summary
Reworks the Test Drive and Quick Start with new content, a new template, and links to other docs.
- Test plan
Preview at:
https://docs-starters--netlify-cms-www.netlify.com/docs/start-with-a-template
https://docs-starters--netlify-cms-www.netlify.com/docs/add-to-your-site
- Description for the changelog
Update starter docs

Deploy preview ready!
Built with commit b571215363e9826e89fdf9a275714ffda9454f04
https://deploy-preview-888--netlify-cms-www.netlify.com


Added support for encrypt

Issue #, if available:
https://github.com/aws/aws-nitro-enclaves-sdk-c/issues/24
Description of changes:
added support for AWS KMS Encrypt action with recipient attestation
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

When is this expected to get merged? Having just decrypt function in the enclave is pretty much half solution and one cannot basically write any data to the S3. So this merge is a must and urgent.

When is this expected to get merged? Having just decrypt function in the enclave is pretty much half solution and one cannot basically write any data to the S3. So this merge is a must and urgent.

Thank you for the contribution.

Thank you for the contribution.


make IMusicBrainzConfig.baseUrl optional again

Hi,
the change to make the baseUrl required could break builds and is not mentioned in the docs.
As far as I can see, it still can be optional and we might need to cast the baseUrl for the httpClient to a string again.
https://github.com/Borewit/musicbrainz-api/blob/52d05f9c2e7c5487db59547f72da0386336f58f3/lib/musicbrainz-api.ts#L129-L137

Good catch Max.
I prevented the requirement to cast the baseUrl: 42b6777, happy with that change?

That looks good to me.

Fixed in v0.18.1


[stable/telegraf] spec.ports: Required value

Using this values, doesn't work:
env:
- name: "AZURE_CLIENT_ID"
  value: "XXXX"
- name: "AZURE_CLIENT_SECRET"
  value: "XXXX"
- name: "AZURE_TENANT_ID"
  value: "XXXX"

config:
  agent:
    debug: true
    hostname: "telegraf"
  outputs:
    - azure_monitor:
        region: "eastus"
        resource_id: "/subscriptions/XXXX/resourceGroups/XXXXX/providers/XXXXXX/workspaces/XXXXXX"
        strings_as_dimensions: true
  inputs:
    - rabbitmq:
        url: "http://XXXXXXXX:15672"
        username: "XXXXX"
        password: "XXXXXXX"

Describe the bug
helm install --name telegraf stable/telegraf -f .\values.yaml
Error: release telegraf failed: Service "telegraf" is invalid: spec.ports: Required value

Version of Helm and Kubernetes:
helm version
Client: &version.Version{SemVer:"v2.13.1", GitCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.13.1", itCommit:"618447cbf203d147601b4b9bd7f8c37a5d39fbb4", GitTreeState:"clean"}

kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.11", GitCommit:"637c7e288581ee40ab4ca210618a89a555b6e7e9", GitTreeState:"clean", BuildDate:"2018-11-26T14:38:32Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.3", GitCommit:"a4529464e4629c21224b3d52edfe0ea91b072862", GitTreeState:"clean", BuildDate:"2018-09-09T17:53:03Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/telegraf

Hi @raphaelquati ,
It seems like port of service will depend on the config.outputs values
Instead of adding an option to disable the service, keep it and  please configure a similar logic for rabbitmq here: https://github.com/helm/charts/blob/b91e719e6812a15bec9afcd9f0b6e7554cb564ba/stable/telegraf/templates/service.yaml#L12-L42 ?
I think that should work.
Cheers!


About pre-trained model

Thanks for an excellent work!
In this paper, models not mention pre-trained are pre-trained by ImageNet-1K, right?
Do you try any other pre-trained model in exp?

Hi, thanks a lot for your attention.
In the DepthFormer paper, models without mentioning pre-training settings are trained by ImageNet-1K.
That is an extremely interesting issue actually. During experiments of SimIPU, I compare model performance based on different pre-trained models with different strategies, including Moco-v2, DenseCL, and so on. There is a discrepancy between them, which means that "which pre-training strategy is suitable for monodepth" is still an unresearched area. There should be more research and this toolbox provides a simple yet effective tool for evaluation, where you only need to change the pre-training model path in configs.

Thanks for your reply!


Asgardex hangs when doing RUNE to BSC.USDT swap

Error is thrown due to Midgard. Protection should be in place.
https://midgard.ninerealms.com/v2/pool/BSC.USDT-0X55D398326F99059FF775485246999027B3197955
"nativeDecimal": "-1", << xchain trys to create a crypto amount but with -1 for decimals which is against bignumber js. so it fails and cascades to asgardex
Midgard should be fixed in the next release.

This issue no longer occurs. It was fixed in Midgard.


Feature Request: LcarsdeStatusFiller could be a top like output

Perhaps showing pids or binary basenames here with various processes from ps output polled every few seconds. It might add some utility. You could color code or sort them based on high cpu/mem utilization or nice level.

Whoops, duplicate https://github.com/lcarsde/status-bar/issues/6, closing


Shift clicking in /emf shop

Version
EvenMoreFish by Oheers 1.6.5
MCV: git-Purpur-1845 (MC: 1.19.2)
SSV: 1.19.2-R0.1-SNAPSHOT
Online: false
Loaded: Rarities(6) Fish(79) Baits(5) Competitions(168)
Database Engine: V3

How to recreate

/emf shop
Shift click the Sell all option
Exit the inventory

The item will still be in your inventory.

This appears to just be a visual bug - albeit a very strange one. I've been able to reproduce this myself and it doesn't seem like I can do anything with the item such as place it or put it in a chest, similar to when you take an item out of a GUI with a poor internet connection.
But yeah, very strange but won't cause your tropical fish in a bucket server economy to crash 😜


Brackets uses too ram

I'm running on Windows 7 without extensions.
Brackets uses more ram each time I reload it.

(The screenshoot is in italian, but the orange rectangle indicates how much ram brackets uses)

I think you need to address this issue at https://github.com/adobe/brackets-shell. This repo is het web part, so we can't do verry much whit ram usage.

Clarified the title.
@svenvNL, issues with Brackets shell should be filed here as described in the bracket-shell README


Proposed 1.4.0-b7

If merged this PR will:
close: #3085
close: #3093
close: #3116
close: #3117
close: #3119

Codecov Report

Merging #3124 into develop will increase coverage by 0.12%.
The diff coverage is 78.43%.

@@             Coverage Diff             @@
##           develop    #3124      +/-   ##
===========================================
+ Coverage    68.83%   68.95%   +0.12%
===========================================
  Files          686      688       +2
  Lines        55393    55690     +297
===========================================
+ Hits         38129    38403     +274
- Misses       17264    17287      +23

Impacted Files
Coverage Δ

src/ripple/rpc/impl/RPCHelpers.h
0% <ø> (ø)
:arrow_up:

src/ripple/ledger/ApplyView.h
100% <ø> (ø)
:arrow_up:

src/ripple/protocol/impl/ErrorCodes.cpp
75% <ø> (ø)
:arrow_up:

src/ripple/protocol/impl/Feature.cpp
91.66% <ø> (ø)
:arrow_up:

src/ripple/protocol/impl/TER.cpp
100% <ø> (ø)
:arrow_up:

src/ripple/app/tx/impl/SetSignerList.h
100% <ø> (ø)
:arrow_up:

src/ripple/app/tx/impl/ApplyContext.h
100% <ø> (ø)
:arrow_up:

src/ripple/protocol/TER.h
100% <ø> (ø)
:arrow_up:

src/ripple/protocol/Indexes.h
100% <ø> (ø)
:arrow_up:

src/ripple/protocol/impl/BuildInfo.cpp
79.16% <ø> (ø)
:arrow_up:

... and 58 more

Continue to review full report at Codecov.

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


Cant open X applications anymore

Environment
Windows build number: Microsoft Windows [Version 10.0.22000.613]
Your Distribution version: Ubuntu 20.04
Your WSLg version: 1.0.26

Steps to reproduce

WSL logs:

Attach WSLg logs from  /mnt/wslg

You can access the wslg logs using explorer at: \\wsl$\<Distro-Name>\mnt\wslg (e.g.: \\wsl$\Ubuntu-20.04\mnt\wslg)

pulseaudio.log
pulseaudio.log
weston.log
weston.log
versions.txt
version.txt

Expected behavior
open X Server applications like Firefox
Actual behavior
getting an X Server error when openning new applications, but not when i open gedit
Firefox error: Gdk-Message: 16:50:33.780: firefox: Fatal IO error 2 (No such file or directory) on X server :0.
The error also only appared in the last few days, i could use gui apps like firefox even a week ago(same gpu driver). Already removed wslg and wsl updates from the system and reinstalled them, also tried debian.

debug.txt

@dcoolArne, your debug.txt file shows weston is crashed, so there should be dump files created at /mnt/wslg/dumps, would you please share with us? thanks!

Hope this helps, had to zip it for upload.
core.zip

@dcoolArne, thanks for dump, so when X11 application is opened, the WSLg's compositor draw window frame and its title bar. In order to support various international fonts, it loads from windows side, and your dump shows the freetype library failed to read from memory mapped I/O of Windows's font, c:\Windows\Fonts\l_10646.ttf. There is a few possibilities, 1) the font file is corrupted, can you use this font file from Windows? 2) WSL access Windows's volume over network, so network is disconnected/interrupted. Can you access that files from Linux after this error occurs? Thanks!

The file is in the folder but it seams to be broken, it wont show in Explorer but its showing in cmd and wsl. I also cant open it using cat or notepad.
cat: /mnt/c/Windows/Fonts/l_10646.ttf: Input/output error
Notepad says something about checking if a have inserted a correct drive. If i just try to copy a new l_10646.ttf file into the fonts folder nothing happends and when deleting to old file i get a access denied error.

@dcoolArne, do you see "Lucida Sans Unicode" font is available from Windows applications? thanks!

Not under Apps and Features and also not in Windows Search or Font Manager.

I just did an inplace system upgrade and now it works again.


fix(weaver)update network-setups fabric setupCC.sh

Fixes #3679
Pull Request Requirements

[ ] Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
[ ] Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
[ ] Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

[ ] Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
[ ] Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

The reason why this error is that variable "CACTI_VERSION" is not collect. So this patch can get latest version of Cacti from  VERSION file.

@dnakashima Please also squash the commits together and make sure that the commit subject and the PR title are consistent. Right now there are multiple commits and they both have invalid commit subjects.

@petermetz Thank you for your comment again. I squashed 2 commits and modified the PR title and commit message. So please check it.

@petermetz I have added "colon" and space to the title after prefix and then linter was OK. Please check it again.


 Just want to confirm, is the issue now sorted?

Yes, I was using a wrong URL for webhook and it was throwing Curl error, it was an error on my side.


319 workshop choice bar

copy texts from https://hpi.de/open-campus/schuelerakademie/projekttage.html

Coverage remained the same at 100.0% when pulling 7e8efd2a85822d0ee2b4fd71b408907a4faf4236 on 319_WorkshopChoiceBar into 2f9e0475db492b0435da38198de715a74e4d6399 on dev.


Triage finschia-sdk#1112

Desc

Doc: adding event breaking section in CHANGELOG

Suggestion(Abandon/Maintain/Submit)

Maintain: finschia-sdk

Links

https://github.com/Finschia/finschia-sdk/pull/1112


Automatically generate links

Manually creating links can be a little cumbersome. It would be great if links could be automatically generated from routes or a specialised attribute.

Automatic self-links and templated titles and be able to include the http method would be great. Possibly tying the links strongly using controller and action with lambdas.

In #17 I've added a method on the HALResponse to add a self link:
var response = new HALResponse(fooBarModel)
        .AddSelfLink(this.Request) // this gets the Request Path - not query string

It also adds the self link if it does exist when using the Controller extension method
return this.HAL(fooModel); // default
return this.HAL(fooModel, new Link[] {
    new Link("self", "/api/foo/{id}") // custom self link
});
return this.HAL(fooModel,  addSelfLinkIfNotExists: false); // turn off self link

What is your idea around tying links to controllers + actions with lambdas? How would you use it? I open for ideas on how to improve this.

The initial idea was to be able to bind a link to a controller using halmodel.Link(controller => controler.Get(id)) to be able to have strongly typed links. Might experiment with this later on.


fixing problems with sandboxed fetchMock being incorrectly bound

@pimterry

Should fix https://github.com/wheresrhys/fetch-mock/issues/159... though I now wonder if the convenience of having sandbox() return a function justifies the hackery needed to get it to work

Back from Christmas now. Thanks for this, looks great, and seems to neatly solve #159 in our codebase.
Personally I quite like the resulting sandbox API this gives, so if I were you I'd stick with it, but I can see how the fiddly complexity might be annoying. Seems to work nicely now either way.


Weird rendering issues - sometimes page.title is not rendered, just {{ page.title }} is displayed instead

Checklist

[x] I have read the tutorials and know the correct effect of the functional design.
[x] There are no similar reports on existing issues (including closed ones).
[x] I found the bug on the latest code of master branch.

Describe the bug

For some reasons on the main branch, for this commit: hidegh/hidegh.github.io@b2e47f9
...the rendering of tags sometimes work, sometimes no.
See: https://hidegh.github.io/posts/root-5/
Ev. to see the original "related" posts and nav-links, the _congif.yml has to be altered.
The issue is also present when debugging on localhost!
The issue is also present on the original "Further Reading" script, if the linked article (MD) contains {{ page.title }} liquid syntax at the start.

In my case:

To Reproduce
Steps to reproduce the behavior:

Try to add a MD content with {{ page.title }} at the start - the more such content, the easier is to spot the issue. Sometimes on the 1st render everything is fine, when new URL is requested (via the Further Reading or the custom other articles in series navigation) then with the new page rendered, the navigation preview might contain the described bug.
Expected behavior

Liquid syntax should be rendered always (or never) on that component.
Screenshots

Already provided in the bug description.
Software

Ruby version: 2.5.5p157
Gem version: <IP_ADDRESS>
Bundler version: 2.2.6
Jekyll version: 4.2.0

Desktop

Windows 10
Linux subsystem used
Chrome latest
Smartphone

Additional context

If I have understood correctly, it seems that you didn't understand how Jekyll works, then tried to change the design by nested using Liquid variables, didn't work, and then reported that you found a bug. Frankly, it's not cool.
A quick answer to your question: If you want to read the From Matter of the posts, use Liquid objects  in related-posts.html instead of in your post MD file.
BTW, I suggest you read this: https://jekyllrb.com/tutorials/convert-site-to-jekyll/#how-layouts-work

If I have understood correctly, it seems that you didn't understand how Jekyll works, then tried to change the design by nested using Liquid variables, didn't work, and then reported that you found a bug. Frankly, it's not cool.
A quick answer to your question: If you want to read the From Matter of the posts, use Liquid objects  in related-posts.html instead of in your post MD file.
BTW, I suggest you read this: https://jekyllrb.com/tutorials/convert-site-to-jekyll/#how-layouts-work

@cotes2020 The same behaviour can be noticed on the original related-posts.html! The difference is that my MD content contains {{ page.title }} which sometimes is then replaced by the value, sometimes not (so sometimes works, sometimes not).
The new part I added - works. Yes, I'm new to Jekyll but not new to SW development. Even my new related-series-posts.html uses the same syntax (copy paste) for displaying a post, as the original one. And the new one does just filter posts based on a diff. criteria (using some newly added filters) - but that part is also OK.

@cotes2020 The same behaviour can be noticed on the original related-posts.html! The difference is that my MD content contains {{ page.title }} which sometimes is then replaced by the value, sometimes not (so sometimes works, sometimes not).
The new part I added - works. Yes, I'm new to Jekyll but not new to SW development. Even my new related-series-posts.html uses the same syntax (copy paste) for displaying a post, as the original one. And the new one does just filter posts based on a diff. criteria (using some newly added filters) - but that part is also OK.

Hi @hidegh,
If you still think this is a bug, please report it to the jekyll repo, because this is not a problem caused by this template. My answer has already told you how to avoid this rendering trap.

Hi @hidegh,
If you still think this is a bug, please report it to the jekyll repo, because this is not a problem caused by this template. My answer has already told you how to avoid this rendering trap.

@cotes2020 It has not to do with the layout stacking. I was playing around a lot with some extra wanna have features and thus stumbled on a problem, which is linked to this as well. Confirming: it's about how Jekyll generates the output for collections.
Now as the "Further reading" does include links to other post, if those other posts are not processed before the current one (the one that includes) then there's no output property, just content property. This means that only the original MD file will be injected, not the output.
Here's a link to a similar issue: https://github.com/jekyll/jekyll/issues/5371 (based on which the given effect is now explainable).
Summary:

Jekyll simply works that way (but it ev. could be fixed, would also require cyclic dependency detection) - and yes, it's something that has to be entered on their repo.
Chirpy works well, but those writing articles must be warned about this limitation. As long we use pure HTML/MD (without liquid) - especially in the excepts, then we're good.

@cotes2020 It has not to do with the layout stacking. I was playing around a lot with some extra wanna have features and thus stumbled on a problem, which is linked to this as well. Confirming: it's about how Jekyll generates the output for collections.
Now as the "Further reading" does include links to other post, if those other posts are not processed before the current one (the one that includes) then there's no output property, just content property. This means that only the original MD file will be injected, not the output.
Here's a link to a similar issue: https://github.com/jekyll/jekyll/issues/5371 (based on which the given effect is now explainable).
Summary:

Jekyll simply works that way (but it ev. could be fixed, would also require cyclic dependency detection) - and yes, it's something that has to be entered on their repo.
Chirpy works well, but those writing articles must be warned about this limitation. As long we use pure HTML/MD (without liquid) - especially in the excepts, then we're good.


chore(injection annotation): update annotation of preprocessors for `…

…preprocessorOrder`
See https://github.com/karma-runner/karma/pull/1899.  Since this is an update to match a version of karma which doesn't yet exist, travis will break.

We decided to go a different way


volontuloapp.org --> new var SYSTEM_DOMAIN (line 96-98)

Description:
Add new var SYSTEM_DOMAIN (differ in different enviroment) instead of string volontuloapp.org
New imports / dependencies:

package

Unit Tests:

apps/volontulo/tests/views/test_auth.py

What tests do I need to run to validate this change:
Description of testing workflow and potential affected places.

We are waiting here for @jacekkalbarczyk issue to be resolved. He has some linting problem there.

Codecov Report

Merging #720 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #720      +/-   ##
==========================================
+ Coverage   91.78%   91.79%   +<.01%
==========================================
  Files          88       88
  Lines        2667     2668       +1
  Branches        4        4
==========================================
+ Hits         2448     2449       +1
  Misses        213      213
  Partials        6        6

Impacted Files
Coverage Δ

.../volontulo/tests/views/offers/test_offer_create.py
100% <ø> (ø)
:arrow_up:

backend/apps/volontulo/views/offers.py
73.07% <ø> (ø)
:arrow_up:

backend/apps/volontulo/lib/email.py
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fixed SQL Generation example

Issue #, if available:
Description of changes:
Issue: The SQL generation example was not working correctly, and was using a Titan model, even though the surrunding text was referencing Claude 2.
Fix:
The following fixes have been applied:

Use of Claude 3 Sonnet, which gives much better results for SQL generation.
Modification of the second example query, to clarify that the objective is to get the "average of total monthly sales". This now works correctly and consistently. The previous reference to "monthly average sales" was ambiguous, and was leading the LLM to generate incorrect output.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@rizblie I can merge your changes into PR #307, is this ok with you?


Storage engine needs transactions

Is your feature request related to a problem? Please describe.
Our storage engine currently doesn't support transactions. It should - we just didn't bake it in initially - but we need to fix that.
Describe the solution you'd like
We need transactions in the DB layer of the service. This should be fairly straightforward to implement, preferably using context.Context plus a function in the storage package that creates and manages transactions.
Describe alternatives you've considered
None.
Additional context
None.

Closed with #33.


Add some outputs related to ECS on redash module

I added some outputs related to ECS on redash module.

Thanks for your review.


reactome update - including diseases and drugs

Testing for sars-covid model translation.

@goodb It looks like this is still underway? Ping me when finalized and we can try and get it out today.

@kltm these models are good to go with that last change.  But will need a fresh REACTO build available at the expected /extensions location before you load it up.

Great. I'll wait until we get a good build of it later today, then pull it in, rebuild, etc.

This is fine but I think it would be nice to see these live someplace other than dev. Dev should be the development server we use for testing, don't you think?

@ukemi Theoretically, yes, but building up the large number of things that we'd need in place to do stuff like that (e.g. multiple centrally controlled instances that require little overhead) more "properly" has not been resourced since very early on. Part of this is touched on in the "production footing" threads that we talk about.

No worries @kltm. Just being a squeaky wheel, but it sounds like we are on the same page.


Bump minimatch from 3.0.4 to 3.1.2

Closes #179

As far as I understand, Vercel folks prefer pinning dependencies in their products. Here is Next.js, for example:
https://github.com/vercel/next.js/blob/bd16ef3c1b1d9654f1cebe96fbdeb3dd51beef36/packages/next/package.json#L76-L83
This way they save their users from accidental upstream breaking changes within a semver range. Not sure this approach can be changed easily, so I doubt we’ll be able to introduce ^ or ~ in this PR 😅

I need to merge this PR.

@vercel, can you give this PR some attention? 🥺


Conceptual Idea to create Model Jars on-the-fly

Hi all,
this PR is a conceptual draft for  a way to automatically build JARs from our OpenNLP models during a Maven build.
It still relies on the models put on dist.a.o. or sourceforge but enables a way to distribute them via Maven Central for easier consumption within the Java ecosystem.
The Download Maven Plugin will ensure the file integrity by checking against an md5 hash. An additional module with a simple java class checks for the presence of the model files inside the jar files. This check could be made more sufficient thought.
For demonstration purpose, I have only added sentence detection and langdetect models into the proposed build config.
The build itself is configured for reproducability (as long as the remote artifacts stay the same).
If we have such models, we can provide a way to load them in opennlp-core to ease access by end users.
Let me know what you think about it.

I updated the PR based on the received comments. If we agree on moving on, we can add the rest of the models and proceed further with having the models available via maven.

I updated the PR based on the received comments. If we agree on moving on, we can add the rest of the models and proceed further with having the models available via maven.

I think it looks great. A long needed improvement.

@rzo1 @jzonthemtn  Before we roll out models to Maven central, we should discuss the versioning scheme. What works for community / end-users? Starting with 1.0 or with the number of the opennlp-tools... #justsomethoughts.

Before we roll out models to Maven central, we should discuss the versioning scheme. What works for community / end-users? Starting with 1.0 or with the number of the opennlp-tools... #justsomethoughts.

Let's move that discussion to the dev@ list. I will open a list thread for it.

https://lists.apache.org/thread/5ftk22b866qy643w3b9ztcorms92sg9q


Rebuild before start

Before start server we need build project to get all techs files in app/example

Вроде ж так и происходит https://github.com/bem-node/bem-node-hello-world/blob/master/GNUmakefile#L4


⚠ Bofrost.at has degraded performance

In 7eb7e63, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1711 ms

Resolved: Bofrost.at performance has improved in 1f5e116 after 7 minutes.


filter external column in /TABLENAME/list'

Now in the gen:
$table_columns = array(
__TABLECOLUMNS_ARRAY__
    );

In my case:
$table_columns = array(
		'id',
		'content',
		'tests1_id',

    );

but we should change it because the list only filter by tests1_id not by external value:

But here we would have a problem because tests1_id couldn't be filtered in the list. I think that:

tests1_id shouldn't be visible but able to filter
tests1_name should be visible and able to filter

The table should have the name tests1 no tests1_id

It's a good question.
As the field is not in the table, it's more a choice than anything.
but yes, as we already display the foreigner name, we should change the column name and allow the filter on it.


Fixed issues with my previous update of the embedding import file.

Fixed issues with my previous update of the embedding import file.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Wrong branch


General capability to set halos on POI icons

Currently, we are not able to set halos for SVG-originated icons.  We need halos for cases when POI icons are obscured when they overlap similarly-colored features:

The mapLibre documentation indicates that this can be done with SDF icons, which can be both recolored and halo'd.
Work is needed to figure out how to add recolorable SDFs to the style and add halos.  When properly implemented, all original icon artwork can be done in monochrome.
The workaround for this is to build a border into the icon, like we do for the capital / city icons.

This may also be relevant to highway shields with simple shapes but a variety of color schemes, like the flowerpot shields of county roads in Ontario, or the various places that use a home-plate-shaped shield.

I'm not sure whether or not we could make that work for shields, since we generate them in code.  That's something we can revisit once somebody understands how SDF icons work :)

For the the city dots and capital city stars, I included halos in the graphics, probably because I couldn't find a Maplibre propery that would do it. Recolorable icons via SDF seems like a nice feature.  Looks like they are limited to one color only though, so not an option for any multicolor icons.

Good point on the coloring.  Also, if there's a legitimate use case for multicolored icons, the built-in halo in the SVG remains an option for that.


Bot: Quickstart errors + No offers showed

Quickstart produces errors

Marketmaker (testnet/localhost) + MCW (testnet/localhost)
Quickstart bot => no offers in the MCW

fixed


⚠ Secret Website 3 has degraded performance

In c248f39, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 3056 ms

Resolved: Secret Website 3 performance has improved in c55f377.


Add more docs on Triggers usage

Idea:
Per todays https://github.com/shipwright-io/community/issues/119 we concluded that this repository requires more documentation focus on different features or use-cases.
For example, how does a Tekton Pipeline looks when SHP is enable there.

Per the refinement meeting today, this should be a pre-req for shipping the first Triggers release.

@jkhelil  per refinement, fyi!


why isn't it possible to use a string variable in these functions?

[Enter feedback here]

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

ID: b83b2196-8c32-86db-3c76-5464a62ae70f
Version Independent ID: b83b2196-8c32-86db-3c76-5464a62ae70f
Content: AddColumns, DropColumns, RenameColumns, and ShowColumns functions in Power Apps - Power Platform
Content Source: power-platform/power-fx/reference/function-table-shaping.md
Service: power-platform
Sub-service: power-fx
GitHub Login: @gregli-msft
Microsoft Alias: gregli

or, more specifically: why can I do this:
ShowColumns(source, "TheColumnName");
but not the following:
Set(col_name, "TheColumnName");
ShowColumns(source, col_name); ?
Why do you differentiate between string variables and string literals?
It would likely be easier to swallow the bitter pill, that we cannot use variables, if you at least gave an explanation as to why this is the case

In general we want to be able to validate the column names at design time. You'll find with Power Fx 1.0 these will now become identifiers instead of string literals, partly to avoid this confusion and frustration of why is this a string (literal) but I can't use a string variable.


Feature/descw 614 add journal

It seems like app lint is failing, although locally there is not a problem, will have to investigate further.


How to use pipenv with multistage docker builds?

Good day,
I'm exploring on how to use pipenv with multi-stage docker builds. In the nutshell, the idea is to "compile" stuff in base image and only copy the resulting artifacts to the final image.
With Python is gets tricky, since you need to copy package dependencies as well.
I've checked out several ideas and looks like pip install --user together with setting PYTHONUSERBASE is the simplest ways to install dependencies to a side directory, e.g.:
FROM alpine AS builder
# Install your gcc, python3-dev, etc. here
apk add --no-cache python3
COPY . /src/
WORKDIR /src
ENV PYROOT /pyroot
RUN PYTHONUSERBASE=$PYROOT pip3 install --user -r requirements.txt
RUN PYTHONUSERBASE=$PYROOT pip3 install --user .

# The final image
FROM alpine
apk add --no-cache python3
ENV PYROOT /pyroot
COPY --from=builder $PYROOT/lib/ $PYROOT/lib/

(The full story)
The problem is that pipenv disregards PYTHONUSERBASE:
$ docker run --rm -ti python:3.6-alpine sh
/ # pip install --upgrade pip; pip install pipenv==2018.10.13  # skipped output
/ # mkdir /tmp/foo; cd /tmp/foo
/tmp/foo # pipenv install requests  # skipped output
/tmp/foo # PYTHONUSERBASE=/pyroot pipenv install --system --deploy
Installing dependencies from Pipfile.lock (b14837)…
  🐍   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 5/5 — 00:00:01
/tmp/foo # ls /pyroot
ls: /pyroot: No such file or directory

I found a workaround by using pipenv lock -r and then installing requirements.txt as in my original idea, but I'm not sure this is the best way to go, particularly if I have custom (private) sources defined in my Pipefile - I don't want to replicate their configuration into pip.
Any other ideas?

Apologies if the original intent was not clear enough.
All I want is to install my package and its dependencies into a separate directory instead of python's site-packages; and then to copy that directory into the final docker image.
Since --system flag does not use virtualenv (it seems) may be it worth supporting PIP* env vars in that special case?
I'll try to elaborate on the problem: I want to keep my final, "production" docker image as small as possible. Therefore I need to build/install my app dependencies in earlier stage.
Having dependencies installed into general site-packages (either system or of a dedicate venv) is problematic, since I can't reliably pick the actual packages my app depends on later on.
Wheels do no help much either because: a) I don't know which wheels to pick up from cache (some may belong to my app reqs, and some may belong to, e.g. a code generation tool my app uses during install); and b) I need to copy wheels to the final docker image prior to installing them there, meaning they will endup in the resulting docker image.

Pipenv unsets user settings because they are incompatible with virtualenv settings. I understand the approach you’re taking, but maybe you can say more about what problem you are trying to solve? You want the wheels and sdists or whatever? If so, on Linux they’re stored in ~/.cache/pipenv/wheels

@techalchemy My comment appeared before yours for some reason (github still recovers?). Just making sure you got notification.

While it's nice to know about this option, I don't see show does it help me. Site packages from vanilla venv weigh 12MB (because of pip and setuptools)
What I'm trying to do is to run
pipenv install --dev
pipenv install
pipenv install .    # May require stuff installed by --dev

and easily collect results of only the last two lines. So another venv does not help me. I need to "split" the installation paths within the venv.
Do you think PEEP that suggest obeying PIP* options if --system flag is used gonna fly?

FOUND IT!!!
Here is what works: PIP_USER=1 PYTHONUSERBASE=/pyroot pipenv install --system --deploy
In action:
/tmp/foo $ grep -A 2 packages Pipfile
[packages]
requests = "*"

[dev-packages]

[requires]
/tmp/foo $ PIP_USER=1 PYTHONUSERBASE=/pyroot pipenv install --system --deploy
Installing dependencies from Pipfile.lock (b14837)…
  🐍   ▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉▉ 5/5 — 00:00:02
/tmp/foo $ ls /pyroot/bin/
chardetect
/tmp/foo $ ls /pyroot/lib/python3.6/site-packages/
chardet                    chardet-3.0.4.dist-info    idna                       idna-2.7.dist-info         requests                   requests-2.20.0.dist-info  urllib3                    urllib3-1.24.1.dist-info

Hooray!
Thanks for the hint about "in any case we already support any pip environment variable you set — besides the user one" - I thought what is have a look on other vars I can override...
I'm happy now.

I mean I understand the steps you are attempting. I am trying to understand why there is a strict constraint on not including incidental dependencies and why you have to fish entry points out.
Historically we had documentation on using --system --deploy but that doesn’t accomplish the goal of preserving artifacts, you need a shared cache directory for that. Also, we don’t really recommend running pipenv as root, even in docker containers.

I am trying to understand why there is a strict constraint on not including incidental dependencies

We'd like to have our docker images as lean as possible. One of the reasons that we have some systems running over mobile line internet, so when pushing upgrade for 20 microservices, those megabytes start to add up. On alpine, pip + setuptools alone weigh 10MB. Compare it to alpine itself (5MB) + python3 (40MB) and that's over 20% increase. Another reason is security - I want to include only what's really used by my app, to minimize attack surface. It's not just me freaking out. It seems where the whole industry is going. With Go, they compile statically and bundle docker image that even does not have a shell. Distroless is another example.

and why you have to fish entry points out.

My setup.py installs entrypoints that will go to venv's bin dir. Since that dir contains other stuff that does not belong to my app, I now need to explicitly specify which files to collect from there. When developer adds new entrypoint, he now needs to remember to update Dockerfile - yet another thing to remember. And it also makes it harder to use one generic Dockerfile to build all my Python apps. With "aside" installation, my Dockerfile instruction can just take all of the /pyroot.

Historically we had documentation on using --system --deploy but that doesn’t accomplish the goal of preserving artifacts, you need a shared cache directory for that.

I'm not sure what you mean. Can you please elaborate? Which artifacts are not preserved exactly?

Also, we don’t really recommend running pipenv as root, even in docker containers.

Even not during build stage?
Regardless, it works well under non-root user as well (inside docker container):
$ mkdir /pyroot; chown appinstall:appinstall /pyroot
$ PIP_USER=1 PYTHONUSERBASE=/pyroot su-exec appinstall:appinstall pipenv install --system --deploy
$ ls -lah /pyroot/
total 16
drwxr-xr-x    4 appinsta appinsta    4.0K Nov  6 02:25 .
drwxr-xr-x   19 root     root        4.0K Nov  6 02:23 ..
drwxr-xr-x    2 appinsta appinsta    4.0K Nov  6 02:25 bin
drwxr-xr-x    3 appinsta appinsta    4.0K Nov  6 02:25 lib

@haizaar Can you publish a working Dockerfile? (without private code, of course)

@haizaar thats only for alpine it seems. will try work it out for deb/ubuntu, the pipenv user info should be enough to go on though

@ekhaydarov

thats only for alpine it seems. will try work it out for deb/ubuntu, the pipenv user info should be enough to go on though

Can you elaborate what you mean? Why pipenv steps for Ubuntu should be any different?

can confirm this works on the non-alpine python:3.7-slim-stretch base image.
I only needed a simple script on top of the pipenv deps, I added an initial base step for the env vars, and I didn't need to muck around with symlinks, console scripts, or entrypoints, so I love how simple this ended up, only 2-3 steps per stage:
FROM python:3.7-slim-stretch AS base

ENV PYROOT /pyroot
ENV PYTHONUSERBASE $PYROOT

FROM base AS builder

RUN pip install pipenv

COPY Pipfile.lock ./

RUN PIP_USER=1 PIP_IGNORE_INSTALLED=1 pipenv install --system --deploy --ignore-pipfile

FROM base

COPY --from=builder $PYROOT/lib/ $PYROOT/lib/
COPY myscript.py ./

CMD ["python","myscript.py"]

many thanks for the hard work here @haizaar!

I would suggest also to add $PYROOT/bin to avoid command not found issue when running python bin cmd like gunicorn
FROM base

COPY --from=builder $PYROOT/lib/ $PYROOT/lib/
COPY --from=builder $PYROOT/bin/ $PYROOT/bin/

ENV PATH="$PYROOT/bin:$PATH"

COPY myscript.py ./

CMD ["python","myscript.py"]

Are there anything unresolved on this topic? It seems to me that everything in recent comments is working.

LGTM.

I'll revert my last comment - PIP_IGNORE_INSTALLED is broken in the latest release: https://github.com/pypa/pipenv/issues/4453

Since running into #4432 which is possibly related to #4453 I have changed my multistage docker builds to utilize the venv method described here: https://sourcery.ai/blog/python-docker/  This does come with the drawbacks that @haizaar had mentioned of including unneeded additional files in the final image, however it appears to be the most stable approach, with using $PYTHONUSERBASE causing packages to be missing or not found in the final image.

This is documented here.

After following @haizaar instruction, I'm facing module not found.


Shortcuts are not created automatically on some machines

OS
Windows 10 Enterprise, os build 19045.4894
Programming Language
net48
VPK Version
vpk 0.0.626
Library Version
Nuget 0.0.626
What happened?
I'm packaging the version using --shortcuts "Desktop,StartMenu" param. But for some machines (seems virtual machines is the problem) this doesn't automatically create shortcuts when installing. The same story without specifying shortcuts param.
Nevertheless, it works ok when I add them in code:
new Velopack.Windows.Shortcuts().CreateShortcutForThisExe(ShortcutLocation.StartMenu | ShortcutLocation.Desktop);
Seems nothing specific in logs.
Relevant log output
2024-10-03 18:33:49> Information - Initializing WindowsVelopackLocator
2024-10-03 18:33:50> Information - Update.exe found in parent directory
2024-10-03 18:33:50> Information - Located valid manifest file at: C:\Users\user\AppData\Local\TEST_VELOPACK\current\sq.version
2024-10-03 18:33:50> Information - Starting Velopack App (Run).
2024-10-03 18:33:50> Information - Setting current process explicit AppUserModelID to 'velopack.TEST_VELOPACK'
2024-10-03 18:33:50> Information - Found fast exit hook: --veloapp-install
2024-10-03 18:33:50> Information - Completed hook, exiting...
2024-10-03 18:33:51> Information - Initializing WindowsVelopackLocator
2024-10-03 18:33:52> Information - Update.exe found in parent directory
2024-10-03 18:33:52> Information - Located valid manifest file at: C:\Users\user\AppData\Local\TEST_VELOPACK\current\sq.version
2024-10-03 18:33:52> Information - Starting Velopack App (Run).
2024-10-03 18:33:52> Information - Setting current process explicit AppUserModelID to 'velopack.TEST_VELOPACK'

Can you please provide the Velopack.log from setup/update? For setup, you can provide this by calling setup.exe --log {fileLocation}. For updates, Velopack.log is automatically created in your app dir.
Additionally, shortcuts are only created initially on setup.exe, or when you've added a new shortcut.
For example, if you pack pack --shortcut StartMenuRoot and then in a later update pack --shortcut Desktop,StartMenuRoot, the Desktop shortcut will be created. But, if the user deletes Desktop and then you pack a third update pack --shortcut Desktop,StartMenuRoot, then the Desktop shortcut will not be created again.

Sure.
[setup:19108] [19:04:33] [INFO] Starting Velopack Setup (0.0.626)
[setup:19108] [19:04:33] [INFO]     Location: "C:\Projects\Setup\Client\TEST_VELOPACK-win-Setup.exe"
[setup:19108] [19:04:33] [INFO]     Silent: false
[setup:19108] [19:04:33] [INFO]     Verbose: false
[setup:19108] [19:04:33] [INFO]     Log: Some("C:\Projects\logs.txt")
[setup:19108] [19:04:33] [INFO]     Install To: None
[setup:19108] [19:04:33] [INFO] OS: Windows 10.0.19045 (Windows 10 Enterprise) [64-bit], Arch=Some(
X64,
)
[setup:19108] [19:04:33] [INFO] Reading bundle header...
[setup:19108] [19:04:33] [INFO] Bundle offset = 2508800, length = 351608845
[setup:19108] [19:04:33] [INFO] Loading bundle from embedded zip...
[setup:19108] [19:04:33] [INFO] Bundle loaded successfully.
[setup:19108] [19:04:33] [INFO] Reading package manifest...
[setup:19108] [19:04:33] [INFO] Package manifest loaded successfully.
[setup:19108] [19:04:33] [INFO]     Package ID: TEST_VELOPACK
[setup:19108] [19:04:33] [INFO]     Package Version: 9.4.1
[setup:19108] [19:04:33] [INFO]     Package Title: TEST_VELOPACK
[setup:19108] [19:04:33] [INFO]     Package Authors: Author
[setup:19108] [19:04:33] [INFO]     Package Description: TEST_VELOPACK
[setup:19108] [19:04:33] [INFO]     Package Machine Architecture:
[setup:19108] [19:04:33] [INFO]     Package Runtime Dependencies:
[setup:19108] [19:04:33] [INFO] Attempting to open global system mutex: 'velopack-TEST_VELOPACK'
[setup:19108] [19:04:33] [INFO] Checking application pre-requisites...
[setup:19108] [19:04:33] [INFO] Determining install directory...
[setup:19108] [19:04:33] [INFO] Installation Directory: "C:\Users\user\AppData\Local\TEST_VELOPACK"
[setup:19108] [19:04:33] [INFO] There is 100.08 Gbit free space available at destination, this package requires 1.04 Gbit.
[setup:19108] [19:04:33] [INFO] Preparing and cleaning installation directory...
[setup:19108] [19:04:33] [INFO] Reading splash image...
[setup:19108] [19:04:33] [INFO] Starting installation!
[setup:19108] [19:04:33] [INFO] Showing splash screen immediately...
[setup:19108] [19:04:33] [INFO] Extracting Update.exe...
[setup:19108] [19:04:33] [INFO] Image is animated GIF (550x460), loading frames...
[setup:19108] [19:04:33] [INFO] Copying nupkg to packages directory...
[setup:19108] [19:04:34] [INFO] Successfully loaded 51 frames.
[setup:19108] [19:04:35] [INFO] Extracting 889 app files to 'C:\Users\user\AppData\Local\TEST_VELOPACK\current'...
[setup:19108] [19:04:45] [INFO] Creating shortcuts...
[setup:19108] [19:04:45] [INFO] Shortcut Previous Locations: ShortcutLocationFlags(0x0) (None)
[setup:19108] [19:04:45] [INFO] Shortcut Next Locations: ShortcutLocationFlags(START_MENU | DESKTOP) (Version { major: 9, minor: 4, patch: 1 })
[setup:19108] [19:04:45] [INFO] App Model ID: Some("velopack.TEST_VELOPACK")
[setup:19108] [19:04:45] [INFO] Searching for shortcuts containing root: 'C:\Users\user\AppData\Local\TEST_VELOPACK'
[setup:19108] [19:04:45] [WARN] Failed to update shortcuts: The system cannot find the file specified. (0x80070002)
[setup:19108] [19:04:45] [INFO] Starting process install hook
[setup:19108] [19:04:45] [INFO] Running --veloapp-install hook...
[setup:19108] [19:04:47] [INFO] Hook executed successfully (took 2749.6975ms)
[setup:19108] [19:04:47] [INFO] Checking for running processes in: C:\Users\user\AppData\Local\TEST_VELOPACK
[setup:19108] [19:04:47] [INFO] Writing uninstall registry key...
[setup:19108] [19:04:47] [INFO] Starting app...
[setup:19108] [19:04:47] [INFO] About to launch: 'C:\Users\user\AppData\Local\TEST_VELOPACK\current\test.exe' in dir 'C:\Users\user\AppData\Local\TEST_VELOPACK\current'
[setup:19108] [19:04:47] [INFO] Args: CommandArgs { inner: [] }
[setup:19108] [19:04:47] [INFO] Installation completed successfully!


Improve compatibility with Flixel 5.3.0, allow compilation without Lua compatibility and minor changes

Fixes in the PlayState.hx and FunkinLua.hx files to allow compilation without using the LUA_ALLOWED and MODS_ALLOWED variables
Some commits are for my confusion, the changes I have applied are to allow Psych Engine compatibility with Flixel 5.3.0, allow compiling the Engine without Lua compatibility and the mods folder

For things like the FlxSound fix it would probably be more optimal to check which version of Flixel the user has and import the library based on that. A lot of people don't know to update their libraries which would still cause more issues to be made here.

@CaptainBaldi Well, that's true, that didn't cross my mind, so I'll fix that.
What I thought is that those who install Haxeflixel for the first time download the latest version and end up with the problems that I tried to solve


Syntax highlighting is not working

Thanks a lot for this extension !!!
Syntax highlighting is not working for me though (everything is white) ; the rest seems to be working. I am using v0.1.2

Do you have any other Cython extensions installed? They may be overriding this extension's syntax highlighter.

Oops... I thought I had removed them, but actually not. Sorry for the noise.
Having cython support in vscode is such a relief, many thanks again !!


First Commit

add model data into mysqlDatabase
Using springSecurity to anthenticate

 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.

@Lordfiber don't use title like First Commit

@Lordfiber fix CI errors


layout: TypeError: invalid 'in' operand f.getDOMAttribute(...)

Strangely, layout started throwing this error for me even on a production website that hasn't been updated and is using version locked layout and A-Frame links. I'm linking to layout here<EMAIL_ADDRESS>

4.3.1 should've addressed this, but if you get rid of the caret, ^, you'll get a true version lock. Try 4.3.0...but it's weird since 4.3.1 is the patch that actually fixes that.

 The use of layout in adit was factored out by the new wheel-select component, so, if this isn't happening in other implemenations, it may have just been an edge case quirk


Error with modal

When I click in any modal, they not appear...

Hi,
Can you describe that in more detail, please?
This plugin has no build in modals. What library provides them or are they custom made?
Please provide a http://plnkr.co/edit/fUt18WwyJlZqGgXHCMVH?p=preview showing the issue.


bschaffl: 1.4.2 -> 1.4.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/sjaehn/bschaffl/releases.
meta.description for bschaffl is: "Pattern-controlled MIDI amp & time stretch LV2 plugin"
meta.homepage for bschaffl is: "https://github.com/sjaehn/BSchaffl"
meta.changelog for bschaffl is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

directory tree listing: https://gist.github.com/2a56a0850208b4e7e43f925607a1149e

du listing: https://gist.github.com/168d68b52537623dfa4e23347882bcb9

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
bschaffl

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6mxm04377wqiarwl37csgrn3113vr543-bschaffl-1.4.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A bschaffl https://github.com/r-ryantm/nixpkgs/archive/abd7ea5410bd44ac74b0c02f4f44e37f1ddfae26.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6mxm04377wqiarwl37csgrn3113vr543-bschaffl-1.4.4
ls -la /nix/store/6mxm04377wqiarwl37csgrn3113vr543-bschaffl-1.4.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

bschaffl

Maintainer pings
cc @magnetophon for testing.

Result of nixpkgs-review pr 112473 at abd7ea54 run on aarch64-linux 1

1 package built:

bschaffl

1 suggestion:

warning: unclear-gpl
gpl3 is a deprecated license, check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/applications/audio/bschaffl/default.nix:26:5:
   |
26 |     license = licenses.gpl3;
   |     ^


chore(Dockerfile): update python 3.7 -> 3.10

Updates the base container from python:3.7-slim to
python:3.10-slim
Also, installs the dependencies before copying the
application code to take advantage of docker build layer
caching.
Closes https://github.com/goblet/goblet/issues/205

Thanks! Do you mind updating the Dockerfile in the goblet repo with the same changes as well!


macOS build error (readline)

MacOS 10.13.2
[ 63%] Building CXX object CMakeFiles/tg_cli.dir/td/telegram/cli.cpp.o
/.../td/td/telegram/cli.cpp:99:3: error: use of undeclared identifier 'rl_replace_line'
  rl_replace_line("", 0);
  ^
/.../td/td/telegram/cli.cpp:105:3: error: use of undeclared identifier 'rl_replace_line'
  rl_replace_line(saved_line.c_str(), 0);
  ^
/.../td/td/telegram/cli.cpp:586:5: error: use of undeclared identifier 'rl_free'
    rl_free(line);
    ^

$ brew info readline

readline: stable 7.0.3 (bottled) [keg-only]
Library for command-line editing
https://tiswww.case.edu/php/chet/readline/rltop.html
/usr/local/Cellar/readline/7.0.3_1 (46 files, 1.5MB)
  Poured from bottle on 2017-03-14 at 11:47:40
From: https://github.com/Homebrew/homebrew-core/blob/master/Formula/readline.rb
==> Caveats
This formula is keg-only, which means it was not symlinked into /usr/local,
because macOS provides the BSD libedit library, which shadows libreadline.
In order to prevent conflicts when programs look for libreadline we are
defaulting this GNU Readline installation to keg-only..

For compilers to find this software you may need to set:
    LDFLAGS:  -L/usr/local/opt/readline/lib
    CPPFLAGS: -I/usr/local/opt/readline/include

$ brew link --force readline doesn't help

Could you check that CMake has found GNU Readline installed via brew and not a libedit readline-like interface shipped by default in Mac OS?
If GNU Readline was installed after first cmake run, then CMakeCache should be purged in order to find the correct library.

The solution:
brew link --force readline
rm -rf build && mkdir build && cd build
cmake -DOPENSSL_ROOT_DIR=/usr/local/opt/openssl/ ..
cmake --build .
cmake --build . --target install
brew unlink readline

Perhaps it's worth adding this to the readme


Refactor subproject commit in tools

This pull request refactors the subproject commit functionality in the tools section.

Preparing review...

PR Review 🔍

 ⏱ Estimated effort to review [1-5]
1, because the PR consists only of a subproject commit update, which is straightforward and requires minimal review effort.

 🧪 Relevant tests
No

 ⚡ Possible issues
No

 🔒 Security concerns
No

 Code feedback: