Unable to aggregate multiple query feeds using tags

Newsboat version (copy the output of newsboat -v or the first line of git show):
Newsboat 2.25.0
System: Darwin 21.1.0 (arm64)
Compiler: g++ Apple LLVM 12.0.5 (clang-1<IP_ADDRESS>)
ncurses: ncurses 5.7.20081102 (compiled with 5.7)
libcurl: libcurl/7.77.0 (SecureTransport) LibreSSL/2.8.3 zlib/1.2.11 nghttp2/1.42.0 (compiled with 7.64.1)
SQLite: 3.36.0 (compiled with 3.32.3)
libxml2: compiled with 2.9.4
Config file (copy from ~/.newsboat/config or ~/.config/newsboat/config):
# Launch
# ───────────
prepopulate-query-feeds yes
suppress-first-reload no

# Reloading
# ───────────
refresh-on-startup yes
auto-reload yes
reload-time 30
reload-threads 400
download-retries 1

# Content
# ───────────
show-read-feeds yes
show-read-articles yes

# Storage/Data Management
# ───────────
max-items 500

# Other
# ───────────
browser "open -g -a 'Google Chrome' %u"
save-path "~/.newsboat/saved_articles/"
feed-sort-order none # set to none means feeds are shown in the order they're listed in the urls file.

Note: There are another 100 or so subsequent lines on my config file, which I have omitted here for the sake of brevity (they are mainly for highlighting text as well as some preset filters)
Steps to reproduce the issue:

Tag an RSS feed URL with a specific tag (in this case, ‘Sport’) on the URLs file, e.g:

https://www.theguardian.com/au/sport/rss Sport
http://www.cbn.com/cbnnews/world/feed/ General

Create a query feed (e.g. to search for keywords across all articles), and also assign that feed the tag ‘Sport’:
"query:Sport keywords:title =~ \"football|athlete|Olympics\"" Sport

Create another query feed to capture all feeds/articles tagged as ‘Sport':
"query:Sport Tag:tags # \"Sport\""

From here, it is possible to refine/categorise the above query feed by creating additional query feeds, e.g:

"query:Sport(tag)+Pandemic:tags # \"Sport\" and title =~ \"COVID|pandemic\""
"query:Sport(tag)+Corruption:tags # \"Sport\" and title =~ \"corrupt|extort\""

Ok so, thus far, there is no issue (in fact, Newsboat works perfectly as intended!) The trouble comes when I attempt the next steps...

Consolidate the above query feeds into a single query feed by assigning them each with a shared tag (e.g. 'meta'):

"query:Sport (tag)+Pandemic:tags # \"Sport\" and title =~ \"COVID|pandemic\"" meta
"query:Sport (tag)+Corruption:tags # \"Sport\" and title =~ \"corrupt|extort\"" meta

Create a query feed for the 'meta' tag:
"query:Meta tag:tags # \"meta\""

I have tried various permutations of steps 5-6 but in every case, attempting to view the 'Meta tag' feed returns:
"Error: feed contains no items!"
Other info you think is relevant:
After the above steps, the ‘Meta tag' does at least appear in the Select Tag dialog; the user can then, as expected, see which feeds have been assigned the ‘meta’ tag. However, that is not the idea, rather it is to be able to go to the meta query feed itself and view all items that have been captured/tagged as such. That is at least what happens for the "query:Sport Tag:tags # \"Sport\"" feed (e.g. it shows both items from feeds where the associated URL is tagged as Sport, as well as those generated by the "query:Sport keywords:title =~ \"football|athlete|Olympics\"" query feed).
I can’t understand why the same is not working for the ‘meta’ feed. A totally uneducated guess is that it might relate to #898. Though a more likely explanation is that I have misunderstood/incorrectly applied the tag function (in which case any suggestions or help from here would be appreciated!) Also sorry for the convoluted description above, as well for any potential duplication with previous issues in opening this.

Hi! Sorry it took me so long to respond.
The cause is simple: when Newsboat populates a query feed, it skips other query feeds
https://github.com/newsboat/newsboat/blob/5c420d940bb20138f5e4d6e9259f1943fe61e7df/src/rssfeed.cpp#L217
This has been the case ever since query feeds were introduced. It's probably a simple way to avoid duplicate articles in query feeds.
I think we could remove this check if we deduplicated items some other way, e.g. by collecting them into a set which we then turn into a vector.
There would be a problem of tracking dependencies: if query feed A pulls data from query feeds B and C, then B and C should be populated before A. We can cheap out of true dependency tracking by re-running populate_query_feeds for as long as any query feed changes.
I wonder if it would break any configurations, though. This should probably be hidden behind a flag, off by default, and we'll make this behaviour a default in Newsboat 3 (sometime in the future).

Many thanks for the response @Minoru  👍.

The cause is simple: when Newsboat populates a query feed, it skips other query feeds...
...It's probably a simple way to avoid duplicate articles in query feeds...

Your response, and the link, provide good context and it now makes sense, cheers for the explanation.
An enhancement that meant duplicate items could be collected (and, presumably also, managed/removed, in some way?) while, in turn, allowing for querying feeds to be bundled, and then fed through another query feed, is music to my ears!  I just wish I had a shred of programming knowledge to offer some assistance to that end..At least for now, that is not the case.


feat: upgrade to Angular 13

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

[x] The commit message follows our guidelines: https://github.com/CirclonGroup/angular-tree-component/blob/master/CONTRIBUTING.md#commit-message-guidelines
[x] Tests for the changes have been added (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Closes #915
What is the new behavior?
Adds support for Angular 13 and builds library as 'partial' Ivy
Does this PR introduce a breaking change?
[x] Yes
[ ] No

Other information

Add eslint
Add core-js
Enable Ivy
Enable Aot
Fix polyfills
Fix lint errors
Fix template compilation errors
Fix example-app build
Fix docs-app build
Fix cyclical component references (https://github.com/angular/angular/issues/43227#issuecomment-904173738)

BREAKING CHANGE: Library is now built with 'partial' Ivy

@jodaka can you please merge this change? We would like to deprecate our fork. Thanks!

I wish I could! I'm just a casual user of angular-tree-component and would love to see this PR merged.
No idea why I had enough rights to review/approve your PR but I certainly don't have rights to merge it.

I wish I could! I'm just a casual user of angular-tree-component and would love to see this PR merged. No idea why I had enough rights to review/approve your PR but I certainly don't have rights to merge it.

ha! dang you could've fooled me 🙂. really disappointing to see libraries get abandoned by their maintainers.

I wish I could! I'm just a casual user of angular-tree-component and would love to see this PR merged. No idea why I had enough rights to review/approve your PR but I certainly don't have rights to merge it.

ha! dang you could've fooled me 🙂. really disappointing to see libraries get abandoned by their maintainers.

@bobbyg603 , @jodaka have you a forked version of the lib yet with the Angular 13+ supported?

@ikruglik yes, our forked version that targets Angular 13 can be found on npm
npm i @bugsplat/angular-tree-component`

https://github.com/BugSplat-Git/angular-tree-component/blob/release/projects/angular-tree-component/package.json

hi @tobiasengelhardt, I've updated this project to use Angular 18. It's been 3 years since the last commit, perhaps you might let us take over the project?

@bobbyg603 I wish I could, but I have no rights to merge, push or change anything in this project.


Structor behind Apache 2.2 proxy

I want to run Structor on server to be able to share it between multiple users to collaborate. I have set up Apache VHost to expose the service externally -
<VirtualHost *:80>
    ServerName mydomain.com
    SetEnv force-proxy-request-1.0 1
    SetEnv proxy-nokeepalive 1
    SetEnv proxy-initial-not-pooled 1
    ProxyPass           / http://localhost:2222/structor/
    ProxyPassReverse    / http://localhost:2222/structor/
</VirtualHost>

However when I try to go to the service I see the following message -

I tried changing everything imaginable, including directory permissions, upgrading to the latest Node and downloading bootstrap boilerplate per instructions. Nothing seems to work and I am not even sure if/where there are any logs I could review.
Could you help figure this out?

@dmk23 as far as I don't have devops capabilities as I wish, and not sure I can help you. Though this can be helpful, structor should have 3 http endpoint:

http://localhost:2222/structor (for structor itself)
http://localhost:2222/absolute path to .structor/desk folder/__build__  (for webpack dev middleware)
http://localhost:2222/webpack-hot-middleware/client?path=/absolute path to .structor/desk folder/a  (for webpack dev middleware socket)

@dmk23 did you manage to run Sturctor behind Apache proxy?

Sorry did not re-test that... I managed to access basic Structor by exposing the port without the Apache proxy - but found too many things in the latest release broken outright to consider using it...
Will be prototyping React by hand for now - my key issue is how to build applications with Django REST backend - which is another thing I'm not clear how to do with Structor...

OK, so I can close this issue.
@dmk23 also, we introduced new brand feature - online generators. So, if you want or your team we can create components right for your project with back end, and you will get them through installable generators. Look ant our site https://helmetrex.com


InternLM example

Added InternML model example to the HubbingFace Hub notebook

thanks @leo-gan!

that was fast :roller_coaster:
:+1: @baskaryan


RHTAPBUGS-1061 BASE_IMAGE_DIGESTS for multi platform

Before you complete this pull request ...
Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

/retest

/retest

/retest

/retest


Refactor server once close

Hi @feross  how are you?
In the server, when I have more than 10 connections I get
(node:28597) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added to [SocketServer]. Use emitter.setMaxListeners() to increase limit

This refactor removes the need of listening for the same event many times.
Also I added a .gitignore.

Thanks for the PR. I fixed this in v9.0.0 by removing support for upgradeReq which is not actually defined in the latest version of ws anyway.
Thanks!

thanks feross :+1:


Move into Shared for SqlInfoMessageEvent.cs

Relates to #1261 .  Move the netcore version of SqlInfoMessageEvent.cs to shared src and updated the references for the csprojs.

@lcheunglci There are couple of MDAC codes in netfx. Please move those as well to common file.


Implement closeaction=restart on ipsec.conf as an option during Phase 1 configuration

Hi.
Some of the OPNSense updates require reboots(and that is mandatory).
This can be really problematic if you have a site-to-site IPSec environment using Mutual RSA and restart the server side(main office). Remote office will not come back online since there was a close CHILD_SA sent from the main office, issued by the ipsec service being stopped.
This will make passthrough the only connection available at the remote office(until ipsec is restarted), and this could be easily managed by adding the closeaction=restart to the ipsec.conf file at the remote office side.
I've made some tests by editing manually ipsec.conf and these are the results:
Without closeaction=restart at the remote office - 0 connections up, 0 connecting.
root@fw01xxx:/usr/local/etc # ipsec status
no files found matching '/usr/local/etc/strongswan.opnsense.d/*.conf'
Shunted Connections:
        pass:  <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 === <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 PASS
Security Associations (0 up, 0 connecting):
  none

With closeaction=restart manually configured at /usr/local/etc/ipsec.conf. Ipsec status shows that it's trying to connect while main office is rebooting (1 connecting).
root@fw01xxx:/usr/local/etc # ipsec status
no files found matching '/usr/local/etc/strongswan.opnsense.d/*.conf'
Shunted Connections:
        pass:  <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 === <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 PASS
Security Associations (0 up, 1 connecting):
        con1[2]: CONNECTING, <IP_ADDRESS>[%any]...138.xx.xx.xx[%any]

After 2 or 3 minutes:
root@fw01xx:/usr/local/etc # ipsec status
no files found matching '/usr/local/etc/strongswan.opnsense.d/*.conf'
Shunted Connections:
        pass:  <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 === <IP_ADDRESS>/24 <IP_ADDRESS>/24 <IP_ADDRESS>/24 PASS
Security Associations (1 up, 0 connecting):
        con1[2]: ESTABLISHED 25 seconds ago, <IP_ADDRESS>[fw01yyy<EMAIL_ADDRESS>        con1{2}:  INSTALLED, TUNNEL, reqid 2, ESP in UDP SPIs: c1eab391_i cc38aa4d_o
        con1{2}:   <IP_ADDRESS>/24 === <IP_ADDRESS>/0

Additional information at the Strongswan Docs:

closeaction = none | clear | hold | restart
defines the action to take if the remote peer unexpectedly closes a CHILD_SA (see dpdaction for
meaning of values). A closeaction should not be used if the peer uses reauthentication or uniqueids checking, as these events might trigger the defined action when not desired.
Prior to 5.1.0, closeaction was not supported for IKEv1 connections.

So, here comes my request: If possible, create an option for Phase 1 IPSec, with something like "Restart this Connection when receiving close CHILD_SA from remote" and at the help, explain that this needs to be enabled only at the remote offices, not at the side where IPSec is configured with "Respond Only" connection method. Checking this option would change the default none to restart to closeaction=

We merged https://github.com/opnsense/core/pull/3851 some time ago, which adds support for manual configuration of dpdaction.  Although it's not the same as closeaction for quite some scenarios this also solves the issue of not being reconnected automatically.

We merged #3851 some time ago, which adds support for manual configuration of dpdaction. Although it's not the same as closeaction for quite some scenarios this also solves the issue of not being reconnected automatically.

Yup. The real issue would be if the main office restart due to version upgrades, sending close CHILD_SA to all peers, and dpdaction is basically the oposite, when branch offices get a timeout, what action to take...

This issue has been automatically timed-out (after 180 days of inactivity).
For more information about the policies for this repository,
please read https://github.com/opnsense/core/blob/master/CONTRIBUTING.md for further details.
If someone wants to step up and work on this issue,
just let us know, so we can reopen the issue and assign an owner to it.

We merged #3851 some time ago, which adds support for manual configuration of dpdaction. Although it's not the same as closeaction for quite some scenarios this also solves the issue of not being reconnected automatically.

Hey @AdSchellevis , just to let you know: we've migrated our core sdwan to another proprietary technology. I think this vendor change sumed up with  #3851 dpd action handling solved our issues.
I can confirm that disabling all Phase 2 tunels at our main office(non-OPNSense), creates a close CHILD_SA event(logs below), but, as soon as we reenable this tunel, OPNSense dpd handles the Phase 2 reconection as expected and implemented on. No need to reopen this issue. This reply of mine is informative only.
2020-09-24T08:59:08 | charon[91760] | 12[ENC] <con1\|3> generating INFORMATIONAL response 1 [ D ]
2020-09-24T08:59:08 | charon[91760] | 12[IKE] <con1\|3> CHILD_SA closed
2020-09-24T08:59:08 | charon[91760] | 12[IKE] <con1\|3> sending DELETE for ESP CHILD_SA with SPI c5ce0ea5
2020-09-24T08:59:08 | charon[91760] | 12[IKE]  <con1\|3> closing CHILD_SA con1{9} with SPIs c5ce0ea5_i (1396  bytes) b1dc7ec1_o (3984 bytes) and TS <IP_ADDRESS>/16 === <IP_ADDRESS>/0


Added Josephus Problem recursive code in python

#442 Fixed
I have added the Josephus problem in python.
@tarun26091999 @rudrakshi99 please review it.

Resolve conflict


new query - by pathway URI

From @GoogleCodeExporter on March 16, 2015 20:17
I would like to request the following feature:

PCViz to serve for queries like:
...pcviz/#pathways/<url-encoded_URI_of_a_pathway>

This is required for the new cPath2 web app (list/find pathways by keywords
page).

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 27 Jun 2014 at 9:15
Copied from original issue: PathwayCommons/cpath2#179

Bump! ;)
I'd like to vote this issue up!

Well, I am closing this one, unresolved, - this is not required anymore, for we develop another "pathways" app using SBGN vizualization instead of SIF.


Failing Integration Tests Intermittently

Problem deleting ssh keys that are in use. Possible fix by changing timeout values.
Example error messages as follows
{"error":"SSH key cannot be deleted because it is currently being used in an active transaction.","code":"SoftLayer_Exception_Public"}

• Failure [1.827 seconds]
SoftLayer Virtual Guest Lifecycle
/tmp/build/8dea2f72-943c-4c0f-6ad6-e3ed2d8c7e00/gopath/src/github.com/maximilien/softlayer-go/integration/virtual_guest_lifecycle/virtual_guest_lifecycle_test.go:209
  SoftLayer_SecuritySshKey#CreateObject and SoftLayer_VirtualGuest#CreateObject
  /tmp/build/8dea2f72-943c-4c0f-6ad6-e3ed2d8c7e00/gopath/src/github.com/maximilien/softlayer-go/integration/virtual_guest_lifecycle/virtual_guest_lifecycle_test.go:168
    creates key, creates virtual guest and adds key to list of VG [It]
    /tmp/build/8dea2f72-943c-4c0f-6ad6-e3ed2d8c7e00/gopath/src/github.com/maximilien/softlayer-go/integration/virtual_guest_lifecycle/virtual_guest_lifecycle_test.go:167

    Expected error:
        <*errors.errorString | 0xc20825f6b0>: {
            s: "Failed to destroy ssh key with id '305125', got '{\"error\":\"SSH key cannot be deleted because it is currently being used in an active transaction.\",\"code\":\"SoftLayer_Exception_Public\"}' as response from the API.",
        }
        Failed to destroy ssh key with id '305125', got '{"error":"SSH key cannot be deleted because it is currently being used in an active transaction.","code":"SoftLayer_Exception_Public"}' as response from the API.
    not to have occurred

    /tmp/build/8dea2f72-943c-4c0f-6ad6-e3ed2d8c7e00/gopath/src/github.com/maximilien/softlayer-go/integration/virtual_guest_lifecycle/virtual_guest_lifecycle_test.go:155

Full output can be seen in http://<IP_ADDRESS>:8080/pipelines/softlayer-ecpi/jobs/sl-go-integration/builds/13 and http://<IP_ADDRESS>:8080/pipelines/softlayer-ecpi/jobs/cpi-integration/builds/8
Tracker story ID #100159074. Perhaps related to (as far as changing timeout values) issue #44 (Tracker story ID #99100866)

@posijon is this still true? I have not noticed it, is it because we are not running this test?
/cc @swetharepakula

@maximilien We took a look at the pipeline and it seems to not fail intermittently. However it's true we're not running it that often. But all the recent tests have passed. We've not diagnosed the issue but perhaps it's safe to close until we encounter it again?

Sounds good @posijon. Closing. Thanks


server: support IdleTimeout

This is new in Go 1.8.

Coverage decreased (-0.1%) to 40.825% when pulling c68c9754e66aea0aba7a028d3bd757756dede544 on fsouza:server-idle-timeout into c02d020babd992176a96e3a6b345cbae7b98910b on NYTimes:master.

Coverage decreased (-0.1%) to 40.809% when pulling 7cfab3a6310f62a1682c9276d622a5c3ea468f2e on fsouza:server-idle-timeout into c02d020babd992176a96e3a6b345cbae7b98910b on NYTimes:master.

Coverage increased (+0.2%) to 41.156% when pulling 7cfab3a6310f62a1682c9276d622a5c3ea468f2e on fsouza:server-idle-timeout into c02d020babd992176a96e3a6b345cbae7b98910b on NYTimes:master.

Coverage decreased (-0.1%) to 40.809% when pulling 7cfab3a6310f62a1682c9276d622a5c3ea468f2e on fsouza:server-idle-timeout into c02d020babd992176a96e3a6b345cbae7b98910b on NYTimes:master.

Coverage decreased (-0.4%) to 40.502% when pulling 8ad5a1d8c2c05a845ae8478c88cd7c446dc1f733 on fsouza:server-idle-timeout into c02d020babd992176a96e3a6b345cbae7b98910b on NYTimes:master.


Configure default logger before upgrade_to_rich_logging

Tracking issue
https://flyte-org.slack.com/archives/C03CYSL783B/p1715163744990199
Why are the changes needed?
Now running locally the logging configures the rich logging here, but this function was refactored on these change, and now the handler setup is inside the try block so if any exception happens there both logger handlers are not configured properly. We detected this because, running some unit test (using tox and pytest), we hit this exception:
Inappropriate ioctl for device

What changes were proposed in this pull request?
Set default logger before upgrade_to_rich_logging()
How was this patch tested?
local execution
Setup process
Screenshots
Check all the applicable boxes

[x] I updated the documentation accordingly.
[x] All new and existing tests passed.
[x] All commits are signed-off.

Related PRs
NA
Docs link
NA

could we try to unblock this? cc @pingsutw


Added dots at the end of comments

Here it is also done: https://github.com/Microsoft/monaco-editor/blob/master/docs/integrate-esm.md

👍

Thank you, but we will not accept this PR at this time.


[Feature] Provide a way to change the style of the activity indicator

On Android the activity indicator is way too big when I simply place the ImageSwipe inside a GridLayout with rows="*". It would be very useful if we can change the style, size, color of busy indicator which is shown while the image is being downloaded.

+1


it is possible to have a single handler class handle multiple messages?

example:
public class NotificationClass1  : IAsyncNotification
{
}

public class NotificationClass2  : IAsyncNotification
{
}

public HandlerForMoreThenOneNotification : IAsyncNotificationHandler<NotificationClass1>,
 IAsyncNotificationHandler<NotificationClass2>
{

    public async Task Handle(NotificationClass1 notification)
    {
    }

    public async Task Handle(NotificationClass2 notification)
    {
    }

    private method SomeSharedCodeThatHandlingBothMessagesCouldUse()
    {
    }
}

Yeah, but why?
On Thursday, June 23, 2016, Michael McCarthy<EMAIL_ADDRESS>wrote:

example:
public class NotificationClass1  : IAsyncNotification
{
}
public class NotificationClass2  : IAsyncNotification
{
}
public HandlerForMoreThenOneNotification : IAsyncNotificationHandler,
IAsyncNotificationHandler
{
public async Task Handle(NotificationClass1 notification)
{
}

public async Task Handle(NotificationClass1 notification)
{
}

private method SomeSharedCodeThatHandlingBothMessagesCouldUse()
{
}

}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jbogard/MediatR/issues/87, or mute the thread
https://github.com/notifications/unsubscribe/AAGYMlGIewvEBY7IcrjUsjb6qHBL5lc_ks5qOvKogaJpZM4I9OrA
.

being an NSB user, sometimes I'll have one class handle two different messages b/c there is processing to be done between both messages that can utilize the same code.
So I've written code like this before:
public class HandleStuff : IHandleMessages<Message1>, IHandleMessages<Message2>

then in each Handle method, I'll invoke a private method on that class that would do some work both handlers need.
I realize I could also try polymorphic messaging (I think MediatR supports this?), but figured I'd ask first.

MediatR supports whatever your underlying container supports, so you can certainly do what you're describing in most containers I'm aware of.

this works for ASP.NET Core RC1 and it looks like this is working out of the box with the way we have our container wired up, so you can close this Issue. Thanks!


i18n should be silent by default

https://github.com/jeresig/i18n-node-2/blob/34bb8950e5af6ccbb31f2be8f10ee487e4c70ab8/i18n.js#L192
Hi, I noticed you're printing directly to the console when in devMode. This can cause spam for those working with the module.
It might be a better idea to use debug, it will only print when the DEBUG environment variable is set to a particular value. This was also used in the previous i18n.

Hi,
Having not so much time these days, while I totally agree on the fact that we should use debug or some proper logging function, I can't implement it; but I'd merge a PR for sure ;)
Thanks

Awesome, I've made the changes sent a pull request


Add Pet Libro (PLAF203) default Telnet credentials

Add Pet Libro (PLAF203) default Telnet credentials

Thanks @Dec0y-jb
Do you have a source for this? How did you find this value out?

I've found the source!
https://community.home-assistant.io/t/petlibro-cat-feeder/498637?page=2

This Pull Request has now been merged in Seclists-Express!
Seclists-Express is a frequently-updated fork of Seclists.

@g0tmi1k - I am so sorry I missed your update! It had been so long since I submitted this, I must have lost track. It does look like you found the correct source (@ItsIgnacioPortal) - that was the exact thread I had found.

Good job @ItsIgnacioPortal with finding the source


PostgreSQL 12: SQL/JSON Path Language

@kindly mentioned that PostgreSQL 12 introduced support for SQL/JSON Path Language (tutorial), which could be useful to simplify the SQL queries used in Kingfisher Summarize.
Logging as something to consider in the event we make significant changes to the queries in the future.

The server runs on PostgreSQL 11, so we'd have to upgrade PostgreSQL first: https://github.com/open-contracting/deploy/issues/177

Pending: https://github.com/open-contracting/deploy/issues/381

I've read the tutorial now and skimmed all our queries, and I don't think we have any that both are complex and could be simpler via SQL/JSON Path Language. Most of our complexity is from aggregations, e.g. counts of each document type, or from checking that the JSON types are correct (e.g. for additionalIdentifiers_ids and additionalclassification).


WIP: config: adding deprecated utilities

Adding hooks for phase 2 of deprecation: fail-by-default features, and runtime overrides.
Risk Level: Low (minimal refactors to core code)
Testing: new unit tests TODO more tests
Docs Changes: TODO docs on deprecation process
Release Notes: n/a - will add a release note when we add existing config to deprecated
Part of #5559 and #5693

I'm going to start in on docs today but in particular I'd like @mattklein123 and/or @jmarantz to weigh in on how I've made Runtime:Loader globally accessible and as @lizan was interested in the last one I wouldn't mind high level suggestions for how to make this better.
For docs, I think I'm going to add a doc assuming we go with 1 release warning-by-default and 1 release failing-by-default.  I'll change the log warning and and will change the log and the exception to point to the docs on how this works, and how to runtime-override, and why you shouldn't runtime override :-P
I plan on leaving the issues open until I've gotten Harvey's awesome deprecation script to auto-add deprecated protos to runtime_features.h and generate a pretty printed list of what is now in hard-fail mode which we can append to the release email, release notes, or both.

Oh, and @junr03 since I'm adding the stat you asked for, I have some questions for you for your use case.
I've got this whole deprecatedFeatureEnabled thing uploaded and locally I've got a deprecated_feature_use stat.
If we end up using the same code for deprecated code paths, like the buffer filter, would you want a stat for deprecated proto config use and deprecated code path call site use, or is it Ok to just use one stat?  If you want them split out I may want to rename this whole set from deprecatedFeature to deprecatedConfig for clarity.


Consider adding especial header to GQL requests

The idea is that we could add a source: ts-sdk-<version> to know which transactions originated from the TS SDK and what version submitted them.

We could also add the version.


Full auto indent not working

Issue Type: Bug
Full auto indent does not appear to work as described.

https://user-images.githubusercontent.com/68073709/170870436-3e11a948-1601-4535-abaf-08a0846f82df.mov
The same code typed out by hand is automatically indented correctly.
The code is:
func f()
    // foo
}

and then copied and pasted inside of itself.
The result is:
func f()
    // foo
    func f()
    // foo
}
}

My expectation is:
func f()
    // foo
    func f()
        // foo
    }
}

Does this issue occur when all extensions are disabled?: Yes
VS Code version: Code 1.67.2 (Universal) (c3511e6c69bb39013c4a4b7b9566ec1ca73fc4d5, 2022-05-17T18:20:57.384Z)
OS version: Darwin arm64 21.5.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
16.00GB (1.95GB free)

Process Argv
--crash-reporter-id 1f16c0d1-50db-46da-9cdc-0eee94227323

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

swift-lang
ssw
0.5.3

vscode-lldb
vad
1.7.0

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30443607
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebug:30492333
vsclangdc:30486549

I have the same problem.
I always have a confusion in my head.
I never know if the auto indent will work or not work this time.

The culprit is probably that it pastes into a line where there is already an indent.
At the same time, the pasted code has no indent in it.
And this combination is causing the strange behavior.
Behavioral versions:
Indent in the line to which you paste, no indent in the pasted code - bug:
https://user-images.githubusercontent.com/16800535/189414458-43d341fe-9e7f-42a2-b413-8fe63d98d0b2.mp4
But here:

No intent in the line to which you paste, no indent in the pasted code - no bug:
Indent in the line to which you paste, indent in the pasted code - no bug:

https://user-images.githubusercontent.com/16800535/189414693-28def7f2-b86b-49b1-8884-aeb3758720e9.mp4
This is definitely a bug, and it needs to be corrected.

I have had this issue for forever too and do not know how to solve it !

Currently the auto indent feature in VS Code is regex based, which doesn't really understand the scope (function, statement) etc. Formatting works more consistently in this scenario, so I would recommend to enable formatting on type or save. Closing this as we don't expect to get around it in the near future.


Latest image does not work with sonarr

Support guidelines

[X] I've read the support guidelines

I've found a bug and checked that ...

[X] ... the documentation does not mention anything about my problem
[X] ... there are no open or closed issues that are related to my problem

Description
Today, after pulling the latest image (5.1.5-7.0), communications with Sonarr seem to be bugged. Torrents are being sent by Sonarr to rtorrent, and they begin downloading, but after that Sonarr seems to lose all contact. For example, the Sonarr queue does not show the files downloading, and the files are not imported when they finish.
Neither my rtorrent nor Sonarr configuration changed, and the Sonarr image I'm using is three weeks old, so I suspected a regression in this image. I have since confirmed that the regression is only present in 5.1.5-7.0 by downgrading to 5.1.5-6.3 , which was released three days ago, and observing that everything works again.
After confirming that 5.1.5-6.3 works, I upgraded again to 5.1.5-7.0 and observed that communication with Sonarr is once again cut off.
Expected behaviour
When triggering a download in Sonarr (manual search, RSS hit, etc), rtorrent communicates with Sonarr's and the following actions happen

rtorrent begins the download
rtorrent reports the status of the download to Sonarr (or replies to Sonar's polling—I'm actually not sure how it works under the hood)

Actual behaviour
Only item #1 above happens. Once the download starts in rtorrent, Sonarr is not able to get any info about it from rtorrent.
Steps to reproduce
See above.
 Lines 1-69 in the log are from after I triggered a download from Sonarr and confirmed that Sonarr could not see the download progress. Starting with log line 70, I opened rutorrent in my browser to confirm that the download had been loaded and was in progress, so there's probably not anything useful there but I included them anyway for completeness' sake.

Same issue here, sticking with 5.1.5-6.3 for now

Can confirm. Setting tags to 5.1.5-6.3 and downgrading works.

@cemmy410 Can you show your compose file or output of docker compose config command please?

This is mine
name: rtorrent-rutorrent

services:
  geoip-updater:
    image: crazymax/geoip-updater:latest
    container_name: geoip-updater
    mem_limit: 4g
    networks:
      - rtorrent-rutorrent
    volumes:
      - /home/radhoo/rtorrent/geoip:/data
    env_file:
      - stack.env
    restart: always

rtorrent-rutorrent:
  image: crazymax/rtorrent-rutorrent:latest

  container_name: rtorrent-rutorrent
  networks:
    - rtorrent-rutorrent
  expose:
    - ${RT_DHT_PORT}/udp
    - ${XMLRPC_PORT}
    - ${RUTORRENT_PORT}
    - ${WEBDAV_PORT}
    - ${RT_INC_PORT}
  ports:
    - target: ${RT_DHT_PORT}
      published: ${RT_DHT_PORT}
      protocol: udp
    - target: ${RUTORRENT_PORT}
      published: ${RUTORRENT_PORT}
      protocol: tcp
    - target: ${WEBDAV_PORT}
      published: ${WEBDAV_PORT}
      protocol: tcp
    - target: ${RT_INC_PORT}
      published: ${RT_INC_PORT}
      protocol: tcp
    - target: ${XMLRPC_PORT}
      published: ${XMLRPC_PORT}
      protocol: tcp
  env_file:
    - stack.env
  volumes:
    - /home/radhoo/rtorrent/data:/data
    - /mnt/radhoo/torrent/torrents:/torrents
    - /home/radhoo/rtorrent:/passwd
  ulimits:
    nproc: 65535
    nofile:
      soft: 32000
      hard: 40000
  restart: always

rtorrent-logs:
  image: bash
  container_name: rtorrent-rutorrent-logs
  command: bash -c 'tail -f /log/*.log'
  network_mode: none

stack.env:
RT_DHT_PORT=6882
XMLRPC_PORT=8005
RUTORRENT_PORT=8081
WEBDAV_PORT=9000
RT_INC_PORT=50000
TZ=Europe/Paris
PUID=1000
PGID=1000
WAN_IP_CMD=false
AUTH_DELAY=0s
MEMORY_LIMIT=256M
UPLOAD_MAX_SIZE=16M
OPCACHE_MEM_SIZE=128
MAX_FILE_UPLOADS=50
REAL_IP_FROM=<IP_ADDRESS>/32
REAL_IP_HEADER=X-Forwarded-For
LOG_IP_VAR=remote_addr
LOG_ACCESS=true
XMLRPC_AUTHBASIC_STRING=rTorrent XMLRPC restricted access
RUTORRENT_AUTHBASIC_STRING=ruTorrent restricted access
WEBDAV_AUTHBASIC_STRING=WebDAV restricted access
RT_LOG_LEVEL=info
RT_LOG_EXECUTE=false
RT_LOG_XMLRPC=false
RU_HTTP_USER_AGENT=Mozilla/5.0 (Windows NT 6.0; WOW64; rv:12.0) Gecko/20100101 Firefox/12.0
RU_HTTP_TIME_OUT=30
RU_HTTP_USE_GZIP=true
RU_RPC_TIME_OUT=5
RU_LOG_RPC_CALLS=false
RU_LOG_RPC_FAULTS=true
RU_PHP_USE_GZIP=false
RU_PHP_GZIP_LEVEL=2
RU_SCHEDULE_RAND=10
RU_LOG_FILE=/data/rutorrent/rutorrent.log
RU_DO_DIAGNOSTIC=true
RU_SAVE_UPLOADED_TORRENTS=true
RU_OVERWRITE_UPLOADED_TORRENTS=false
RU_FORBID_USER_SETTINGS=false
RU_LOCALE=UTF8
EDITION_IDS=GeoLite2-City,GeoLite2-Country
LICENSE_KEY=
DOWNLOAD_PATH=/data
SCHEDULE=0 0 * * 0
LOG_LEVEL=info
LOG_JSON=false

Do you have any RPC faults in /tmp/errors.log? I need to see some debugging information for RPC, since the problem is most likely related to tinyxml2.

Same issue for me.
crazy-max/docker-rtorrent-rutorrent 5.1.5-7.0-r0
linuxserver/docker-sonarr <IP_ADDRESS>80-ls263
no errors.log in tmp :

@cemmy410 Can you show your compose file or output of docker compose config command please?

The output of docker compose config is still this:
no configuration file provided: not found
(I'm running docker on a server running OpenMediaVault and I guess it does some shenanigans with how docker is set up, sorry)
Here's my compose file:
services:
  rtorrent-rutorrent:
    container_name: 'rtorrent-rutorrent'
    image: crazymax/rtorrent-rutorrent:5.1.5-6.3 #latest
    #network_mode: host
    stop_grace_period: 20s

    volumes:
      - "${PATH_TO_DOCKER_DATA}/rtorrent-rutorrent:/data"
      - "${PATH_TO_MEDIA}:/downloads"

    ports:
      #- "6881:6881/udp" #DHT
      - "8000:8000"
      - "8080:8080"
      - "9000:9000"
      - "50000:50000" # Peer listen

    environment:
      PUID: ${APPUSER_PUID}
      PGID: ${APPUSER_PGID}
      TZ: ${TIME_ZONE_VALUE}

      MEMORY_LIMIT: '1G'
      MAX_FILE_UPLOADS: '50'
      RUTORRENT_PORT: '8080'

      RT_LOG_LEVEL: 'debug' #'info'
      RT_RECEIVE_BUFFER_SIZE: '4M'
      RT_SEND_BUFFER_SIZE: '8M'

    restart: unless-stopped

And, in case it helps, the last few lines of my .rtorrent.rc which override some of the defaults:
cfg.download_complete = /downloads/.rtorrent/
cfg.download_temp = /downloads/.rtorrent/
#dht.mode.set = disable
#protocol.pex.set = no
pieces.memory.max.set = 4G
#protocol.encryption.set = require,require_RC4

(everything above this block is identical to the default .rtorrent.rc file provided)

Do you have any RPC faults in /tmp/errors.log? I need to see some debugging information for RPC, since the problem is most likely related to tinyxml2.

I also have no /tmp/errors.log:
/ # ls -la /tmp
total 32
drwxrwxrwt  1 root     root  4096 Dec 27 12:45 .
drwxr-xr-x  1 root     root  4096 Dec 26 19:36 ..
drwx------ 10 rtorrent users 4096 Dec 26 19:48 nginx
drwx------  2 rtorrent users 4096 Dec 26 19:36 nginx-fastcgi
drwx------  2 rtorrent users 4096 Dec 26 19:36 nginx-proxy
drwx------  2 rtorrent users 4096 Dec 27 12:49 nginx-scgi
drwx------  2 rtorrent users 4096 Dec 26 19:36 nginx-uwsgi
-rw-------  1 rtorrent users   24 Dec 26 19:48 sess_nmbscbjhqh6329mbgm8b9oqfjt

My /tmp/ is empty as well.

Maybe you could enable xmlrpc logs and check if there is anything different between versions 6.3 and 7.0
By adding this line in your .rtorrent.rc file :
log.xmlrpc = /data/rtorrent.xmlrpc.log

Could someone try to point sonarr to httprpc on ruTorrent? It's located at /plugins/httprpc/action.php after you put in the domain name or IP address.

like this?

Same result.

like this?

That is close, but not quite. Ensure to pull in the full URL containing the path to ruTorrent.

Tried both https and http. And with 8005 and 8081

Seems the bug is caused by tinyxml2 which does not follow XML-RPC spec.

An < array > contains a single < data > element, which can contain any number of < value >s.

tinyxml2 responses do not have <array> content between <data> tags as expected.
xmlrpc-c in the d.multicall2 response :
<?xml version="1.0" encoding="UTF-8"?>
<methodResponse>
 <params>
  <param>
   <value>
    <array>
     <data>
      <value>
       <array>
        <data>
         <value>
          <string>The.Series.2019.S04E07.MULTi.1080p.WEB.H264</string>
         </value>
         <value>
          <string>E607137DCEB4ED6E2790FCFE8BDD57BA711B79AE</string>
         </value>

xmlrpc-tinyxml2 :
<?xml version="1.0"?>
<methodResponse>
 <params>
  <param>
   <value>
    <array>
     <value>
      <array>
       <value>
        <string>The.Series.2019.S04E07.MULTi.1080p.WEB.H264</string>
       </value>
       <value>
        <string>E607137DCEB4ED6E2790FCFE8BDD57BA711B79AE</string>
       </value>

Can confirm it brakes also communication using transdrone android client (through httprpc plugin).
It works again reverting to the prevoius image version.

Can confirm it brakes also communication using transdrone android client (through httprpc plugin). It works again reverting to the prevoius image version.

I will backport the fix @simonc56 made today and create a pull request for the docker image.

Thank you for the quick fix!

This also broke Radarr in case anyone like me is searching here

Have you tried v7.1? It was just tagged an hour ago with a fix.

Same behaviour showing here; sonarr is able to queue the job but immediately shows it as paused and doesn't notice when the download completes even though it does show 100% progress. Nothing interesting in the sonarr logs.
Downgrading to the 5.1.5-6.3 image fixes the problem.
(A big thank you to all who have put so much time into making this project work!)

Same behaviour showing here; sonarr is able to queue the job but immediately shows it as paused and doesn't notice when the download completes even though it does show 100% progress. Nothing interesting in the sonarr logs.
Downgrading to the 5.1.5-6.3 image fixes the problem.
(A big thank you to all who have put so much time into making this project work!)

5.1.5-7.1 was just released an hour ago with a fix for a sonarr issue. Have you upgraded? If so, do you have sonarr pointed to the httprpc plugin on ruTorrent?

My feedback about 7.1 fix:

rtorrent 7.0 tested with android app nzb360 failed because of missing <data> tag issue ❌
rtorrent 7.1 tested with android app nzb360 works ✅

But this issue is about sonarr.
To my opinion radarr/sonarr can't read the is_active value of torrents because of integers being declared as 32bit values with <i4> tag (with tinyxml2) instead of 64bits values with <i8> tag (with xmlrpc-c).
Already discussed here :

https://github.com/kannibalox/rtorrent/commit/532bfb2e6b74f574c40ca7aac08c5896cec039a9

If the <i4> tag is indeed the cause of the pause issue, I think it's a bug in radarr/sonarr because <i4> is a correct tag and is correctly read by other clients.

Created issue in Sonarr. https://github.com/Sonarr/Sonarr/issues/7530
Please fill in more info if needed as I'm not a programmer.

@simonc56 We need to fix this from the rTorrent repository. We can't have xmlrpc-c on i8 and tinyxml2 on i4. This is going to create headaches and breakage for software developers. We should have both on i8, until we deprecate xmlrpc-c. Then we can switch tinyxml2 to i4.

@crazy-max Anther PR is live for the docker image. We found a 2nd issue with sonarr.
https://github.com/crazy-max/docker-rtorrent-rutorrent/pull/421


Upgrade to next 13

Closes #253
Changes

upgrade from Next js 12 to 13
Next.js Links updated according to the new version requirements,
other upgraded packages: react, react-dom, contentlayer, next-contentlayer, next-seo, next-themes, esling-config-next

Also

added target:"_blank" to github and discord links in the Nav component
disabled eslint on build in the template (users don't need to use eslint, and so it's not a dependency of the template package anymore)

@khalilcodes I have no idea how this could work before tbh (we're trying to assign a value to the imported variable here) but now this error shows up for the Algolia.jsx file:
error - ./components/search/Algolia.jsx
Error:
  x cannot reassign to an imported binding
    ,-[1:1]
  1 | import { DocSearchModal, useDocSearchKeyboardEvents } from "@docsearch/react";
    :          ^^^^^^^|^^^^^^
    :                 `-- imported binding
  2 | import Head from "next/head";
  3 | import Link from "next/link";
  4 | import { useRouter } from "next/router";
  5 | import { createContext, useCallback, useMemo, useRef, useState } from "react";
  6 | import { createPortal } from "react-dom";
  7 |
  8 | function Hit({ hit, children }) {
  9 |   return <Link href={hit.url}>{children}</Link>;
 10 | }
 11 |
 12 | export const AlgoliaSearchContext = createContext({});
 13 |
 24 |     }
 25 |
 26 |     return Promise.all([import("./AlgoliaModal")]).then(
 27 |       ([{ DocSearchModal: Modal }]) => {
 28 |         // eslint-disable-next-line
 29 |         DocSearchModal = Modal;
    :         ^^^^^^^^^^^^^^
    `----

I'll fix this but I think this file could use a bit of refactoring as there are some other funky things we do here (or maybe I'm not yet getting everything).

Converting this to draft as one of the kbar dependencies requires React 16 or 17:
 WARN  Issues with peer dependencies found
packages/template
└─┬ kbar 0.1.0-beta.38
  ├─┬ @reach/portal 0.16.2
  │ ├── ✕ unmet peer<EMAIL_ADDRESS>|| 17.x": found 18.2.0
  │ ├── ✕ unmet peer<EMAIL_ADDRESS>|| 17.x": found 18.2.0
  │ └─┬ @reach/utils 0.16.0
  │   ├── ✕ unmet peer<EMAIL_ADDRESS>|| 17.x": found 18.2.0
  │   └── ✕ unmet peer<EMAIL_ADDRESS>|| 17.x": found 18.2.0
  └─┬ react-virtual 2.10.4
    └── ✕ unmet peer<EMAIL_ADDRESS>|| ^17.0.0": found 18.2.0

even though the latest kbar version lists React 18 as one of their peers:
https://github.com/timc1/kbar/blob/f557c85e48e2d20d8a87ae71c2830e88330ee0a4/package.json#L66-L69


Fix factory-boy package name and pin

What are the relevant tickets?
N/A
What's this PR do?
This is an attempt to fix the requirements so that #980 can be generated successfully, currently it's conflicting on two points: a) it's pinning to 3.2.1, which conflicts with requirements.txt which is pinned to 3.2.0 b) it's using the legacy factory_boy package.
How should this be manually tested?
Tests should pass.

Codecov Report

Merging #1005 (b7930ac) into main (4e0e970) will increase coverage by 2.78%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1005      +/-   ##
==========================================
+ Coverage   85.92%   88.71%   +2.78%
==========================================
  Files         284      141     -143
  Lines       10758     3827    -6931
  Branches     1161        0    -1161
==========================================
- Hits         9244     3395    -5849
+ Misses       1311      432     -879
+ Partials      203        0     -203

Impacted Files
Coverage Δ

authentication/serializers.py

flexiblepricing/admin.py

cms/serializers.py

ecommerce/messages.py

courses/urls.py

ecommerce/serializers.py

authentication/strategy.py

cms/signalreceivers.py

...rt/management/commands/import_micromasters_data.py

ecommerce/templatetags/refund_controls.py

... and 133 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Extend DID Registrar API to allow multiple input keys and respective assignments to DID Doc Verification Relationships

Is your feature request related to a problem? Please describe.
DID Registrar implementations in the waltid-did library generate DIDs and their respective documents by receiving as input a single key. As this is a limitation of the waltid-did library, it manifests to all applications and libraries that build on top of it (e.g., Wallet API).
This extension would, on the one hand, facilitate the development of use cases where multiple keys are to be employed, each for its own unique purpose (which is a best practice in key usage), and, on the other hand, enhance alignment with the respective specifications. Refer to #579 for an example of an issue that could be addressed with this extension, assuming proper checks and balances are in place, i.e., carefully considering "what key can be assigned to what property".
Additional context
Unclear what the consequences of this change will be to the rest of the codebase regarding the resolution of DIDs and how it is handled, especially when it comes to the use of the resolveToKey() method of the DidResolver class.

This item has been successfully added to our development backlog.
Temporarily closing this issue for now.
If anyone has any feedback, or comments, on this subject, please add them here and
the issue will be automatically re-opened.


Oracle Java 9

Should be future proof with the new version naming model.
I am aware of the work from PR #569, but this also includes the other Java 9 manifests. Also, installer is not run for the JDK, it's just unzipped.
The version should be harmonized with scoop PR #1727

@se35710 You need to resolve the conflicts before we can consider merging it

@r15ch13 @lukesampson What do you say, should we try to make this happen? Either this or #569. I can remove the oracle8.json file and update to current versions...

See https://github.com/BjoernPetersen/scoop-bucket/blob/master/oraclejdk9.json

@rasa It runs the installer though. With Java 9 and Java 10 you can unzip the exe, and then unzip the tools.zip file.

Replaced with #864.


Optimization Speed

from pulp.solvers import PULP_CBC_CMD
from pydfs_lineup_optimizer import Site, Sport, get_optimizer,CSVLineupExporter
import pulp
from pulp import *
from pydfs_lineup_optimizer import get_optimizer, Site, Sport,CSVLineupExporter
from pydfs_lineup_optimizer.solvers.pulp_solver import PuLPSolver
import time
start_time = time.time()
class CustomPuLPSolver(PuLPSolver):
    LP_SOLVER = PULP_CBC_CMD(threads=8, options=['preprocess off'])
optimizer = get_optimizer(Site.FANDUEL, Sport.BASEBALL)
optimizer.load_players_from_csv("/Users/austi/Desktop/MLB/PLAYERS_LIST.csv")
optimizer.restrict_positions_for_opposing_team(['P'], ['1B','C','2B','3B','SS','OF','UTIL'])
optimizer.set_spacing_for_positions(['SS','C','1B','3B','OF','2B'], 2)
optimizer.set_team_stacking([4,3])
optimizer.set_max_repeating_players(3)
lineup_generator = optimizer.optimize(150)
for lineup in lineup_generator:
    print(lineup)
exporter = CSVLineupExporter(lineup_generator)
exporter.export('MLB_result.csv')
print(round(((time.time() - start_time)/60)), "minute run time")

Is there any solution to making the lineup generator faster? I have tried using cplex, but it doesn't use a greedy heuristic like the CBC solver does and it's not free anymore. However, this solution takes almost 5 hours to run. I was wondering if there is anything I can do to improve this method without using a different solver?
I was under the assumption that decreasing the repeating players would actually make it fast because it's decreasing the player pool.

import pulp
from pulp import *
from pulp.solvers import CPLEX_PY
from pydfs_lineup_optimizer import get_optimizer, Site, Sport,CSVLineupExporter
from pydfs_lineup_optimizer.solvers.pulp_solver import PuLPSolver
import time
start_time = time.time()
class CustomPuLPSolver(PuLPSolver):
    LP_SOLVER = pulp.CPLEX_PY(msg=0)
optimizer = get_optimizer(Site.FANDUEL, Sport.BASEBALL, solver=CustomPuLPSolver)
optimizer.load_players_from_csv("/Users/austi/Desktop/MLB/PLAYERS_LIST.csv")
optimizer.restrict_positions_for_opposing_team(['P'], ['1B','C','2B','3B','SS','OF','UTIL'])
optimizer.set_spacing_for_positions(['SS','C','1B','3B','OF','2B'], 2)
optimizer.set_team_stacking([4,3])
optimizer.set_max_repeating_players(3)
lineups = list(optimizer.optimize(n=150))
for lineup in lineups:
    print(lineup)
exporter = CSVLineupExporter(lineups)
exporter.export('MLB_result.csv')
print(round(((time.time() - start_time)/60)), "minute run time")

also how does casting it to a list differ from just generating the lineup from the solution above?

Did you ever find a solution for this?

I have not found a solution other than using cplex. @BenikaH


Ability to disable Settings

It would be great to be able to disable specific settings. For example if two toggle settings are "linked" so that the second only applies if the first is enabled. It would be great to be able to show both settings so the user understands that they both exist but have the second disabled (visually and functionally) so they can't click it without enabling the other one first.
I'm picturing something like this:
// ... in display() of PluginSettings
const setting1 = new Setting(containerEl)
  .setName('First setting')
  .setDescription('enable something and allow second setting to be set')
  .addToggle((toggle) => toggle.setValue(this.plugin.settings.firstOpt).onChange(async(value) => {
    this.plugin.settings.firstOpt = value
    await this.plugin.saveSettings();
    if (value) {
      setting2.enable()
    }
    else {
      this.plugin.settings.secondOpt = false;
      setting2.disable()
    }
  }))

const setting2 = new Setting(containerEl)
  .setName('Second setting')
  .setDescription('Enable something else. only enabled when firstOpt is on')
  .addToggle((toggle) => toggle.setValue(this.plugin.settings.secondOpt).onChange(async(value) => {
    this.plugin.settings.secondOpt = value;
    await this.plugin.saveSettings();
  }))
if (this.Plugin.settings.firstOpt) {
  setting2.disable()
}

I think I understand your use case, for now you'll have to manually disable the control elements, and possibly add some styles to the entire line to be more explicit.
One thing I'd like to note is that Setting don't keep track of its internal controls at the moment, which makes it a bit more difficult. Right now, we don't currently have such a need yet, but when we do it won't be too hard to add.

you'll have to manually disable the control elements

That's kinda what I ended up doing. However because of the way inputs are set up (the toggle specifically) there is no way to actually disable, remove or block the click handler. The best I could come up with is adding the css: pointer-events: none;. As a web dev myself this is a terrible way to disable an input, if it's the only one implemented, which is why I created the request for an internal option.
Hopefully we can get something like this in the future!

Ah I see what you mean. That's definitely a problem.

setDisabled is now available on Setting and Component. I totally forgot to comment here but it's been in for a few releases already.

@lishid Oh awesome! Thanks for adding that!


Can't see the error because of slack token

When using whales I got an error.
 /usr/local/bundle/gems/slack-notifier-2.3.2/lib/slack-notifier/util/http_client.rb:29:inblock in call': The slack API returned an error: invalid_token (HTTP Code 403) (Slack::Notifier::APIError)
Check the "Handling Errors" section on https://api.slack.com/incoming-webhooks for more information
from /usr/local/bundle/gems/slack-notifier-2.3.2/lib/slack-notifier/util/http_client.rb:27:in tap' 	from /usr/local/bundle/gems/slack-notifier-2.3.2/lib/slack-notifier/util/http_client.rb:27:in call'
from /usr/local/bundle/gems/slack-notifier-2.3.2/lib/slack-notifier/util/http_client.rb:13:in post' 	from /usr/local/bundle/gems/slack-notifier-2.3.2/lib/slack-notifier.rb:49:in block in post'
from /usr/local/bundle/gems/slack-notifier-2.3.2/lib/slack-notifier.rb:47:in map' 	from /usr/local/bundle/gems/slack-notifier-2.3.2/lib/slack-notifier.rb:47:in post'
from /usr/local/bundle/gems/slack-notifier-2.3.2/lib/slack-notifier.rb:37:in ping' 	from /usr/src/app/lib/rescuer.rb:11:in ping'
from /usr/src/app/lib/whales_cli.rb:23:in rescue in tame' 	from /usr/src/app/lib/whales_cli.rb:20:in tame'
from /usr/local/bundle/gems/thor-0.20.0/lib/thor/command.rb:27:in run' 	from /usr/local/bundle/gems/thor-0.20.0/lib/thor/invocation.rb:126:in invoke_command'
from /usr/local/bundle/gems/thor-0.20.0/lib/thor.rb:387:in dispatch' 	from /usr/local/bundle/gems/thor-0.20.0/lib/thor/base.rb:466:in start'
from /usr/src/app/bin/whales:9:in <main>'
And the token is indeed invalid.
Could you fix it or at least add an option to display the error instead of sending it to a slack channel ?

Yes, we can look at it, thanks for reporting

Thank you very much @kurenn

I received the exact same message for Slack API error. @patrick-mota's suggestion would be very helpful! Thank you!

Hi @dcnadler maybe it's hard to fix so here is a quickfix / workaround.
Get the source code and modify this file https://github.com/Gueils/whales/blob/master/lib/rescuer.rb
line 14
SLACK_WEBHOOK = "https://hooks.slack.com/services/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".freeze
You just need to change the token here... more info about webhooks at:
https://api.slack.com/messaging/webhooks
and you should be good to go.
Happy new year.

Thanks @patrick-mota! I will give it a try.

Hi @dcnadler maybe it's hard to fix so here is a quickfix / workaround.
Get the source code and modify this file https://github.com/Gueils/whales/blob/master/lib/rescuer.rb
line 4
SLACK_WEBHOOK = "https://hooks.slack.com/services/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".freeze
You just need to change the token here... more info about webhooks at:
https://api.slack.com/messaging/webhooks
and you should be good to go.
Happy new year.

I did this, added my own webhook, pushed to docker and ran command with my own image but still doesnt work


fix: listen to incoming messages on agent initialize not constructor

Codecov Report

Merging #1542 (3300711) into main (e448a2a) will decrease coverage by 8.56%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##             main    #1542      +/-   ##
==========================================
- Coverage   85.73%   77.17%   -8.56%
==========================================
  Files         950      905      -45
  Lines       22743    21870     -873
  Branches     3977     3849     -128
==========================================
- Hits        19499    16879    -2620
- Misses       3059     4636    +1577
- Partials      185      355     +170

Files Changed
Coverage Δ

packages/core/src/agent/Agent.ts
91.52% <60.00%> (-1.70%)
:arrow_down:

... and 228 files with indirect coverage changes

@genaris

With this fix were you able to make several shutdown / initialize sequences?

Yes I was :)

@TimoGlastra @genaris - any updates?

FYI @TimoGlastra - on AFJ WG call, it was mentioned by both @genaris and @berendsliedrecht that this may be considered breaking change. I was going to look into a different solution. What do you think?

Yeah it's definitely a breaking change, but I'm not sure why anyone would use this property, and thus I'm okay with merging it as is, as it should never have been exposed.
But other solutions also fine for me

Yeah it's definitely a breaking change, but I'm not sure why anyone would use this property, and thus I'm okay with merging it as is, as it should never have been exposed.
But other solutions also fine for me

Yeah @niall-shaw I think it'd be better to make this field private and treat it as a fix to not overcomplicate things. Then we merge it for 0.4.1.


TypeError str returned non-string (type PosixPath) /debug_toolbar/templates/debug_toolbar/panels/staticfiles.html

django-debug-toolbar: 4.4.6
I'm just providing a solution to an event I came across. Line 29 for /debug_toolbar/templates/debug_toolbar/panels/staticfiles.html needs to have {{ staticfile }} updated to {{ staticfile|stringformat:"s" }}
Below is the event that happened.
When running uvicorn project.asgi:application --host <IP_ADDRESS> --port 8001 and then loading <IP_ADDRESS>:8001 in a new web browser, the TypeError str returned non-string (type PosixPath). Specifically focusing on {{ staticfile }}. I updated {{ staticfile }} to {{ staticfile|stringformat:"s" }} and that resolved the TypeError.
Thanks!

Can you share more information about your project? This isn't something we've come across elsewhere. We'd want to know what your static file was and what your installed apps and middlewares are?

Sure.
Legend:

Static File Previous
Static File Updated
Apps/Middlewares

Look for the line {{ staticfile|stringformat:"s" }} is shown on.
Apps/Middlewares:
INSTALLED_APPS = [
    "django.contrib.admin",
    "django.contrib.auth",
    "django.contrib.contenttypes",
    "django.contrib.sessions",
    "django.contrib.messages",
    "django.contrib.staticfiles",
    "django_select2",
    "django_htmx",
    "crispy_forms",
    "crispy_tailwind",
    "storages",
    "widget_tweaks",
    "django_user_agents",
    "debug_toolbar",
    'django.contrib.gis',
    'leaflet',
    'channels',
]

MIDDLEWARE = [
    "django.middleware.security.SecurityMiddleware",
    "django.contrib.sessions.middleware.SessionMiddleware",
    "django.middleware.common.CommonMiddleware",
    "django.middleware.csrf.CsrfViewMiddleware",
    "django.contrib.auth.middleware.AuthenticationMiddleware",
    "django.contrib.messages.middleware.MessageMiddleware",
    "django.middleware.clickjacking.XFrameOptionsMiddleware",
    "django_htmx.middleware.HtmxMiddleware",
    "django_user_agents.middleware.UserAgentMiddleware",
    "project.middleware.DeviceDetailsMiddleware",
    "debug_toolbar.middleware.DebugToolbarMiddleware",
    'whitenoise.middleware.WhiteNoiseMiddleware',
]

Static File Previous:
`{% load i18n %}
{% blocktrans count staticfiles_dirs|length as dirs_count %}Static file path{% plural %}Static file paths{% endblocktrans %}
{% if staticfiles_dirs %}

    {% for prefix, staticfiles_dir in staticfiles_dirs %}
      {{ staticfiles_dir }}{% if prefix %} {% blocktrans %}(prefix {{ prefix }}){% endblocktrans %}{% endif %}
    {% endfor %}

{% else %}
  {% trans "None" %}
{% endif %}
{% blocktrans count staticfiles_apps|length as apps_count %}Static file app{% plural %}Static file apps{% endblocktrans %}
{% if staticfiles_apps %}

    {% for static_app in staticfiles_apps %}
      {{ static_app }}
    {% endfor %}

{% else %}
  {% trans "None" %}
{% endif %}
{% blocktrans count staticfiles|length as staticfiles_count %}Static file{% plural %}Static files{% endblocktrans %}
{% if staticfiles %}

    {% for staticfile in staticfiles %}
      {{ staticfile }}
{{ staticfile.real_path }}
    {% endfor %}

{% else %}
  {% trans "None" %}
{% endif %}
{% for finder, payload in staticfiles_finders.items %}
{{ finder }} ({% blocktrans count payload|length as payload_count %}{{ payload_count }} file{% plural %}{{ payload_count }} files{% endblocktrans %})

{% trans 'Path' %}
{% trans 'Location' %}

      {% for path, real_path in payload %}

{{ path }}
{{ real_path }}

      {% endfor %}

{% endfor %}`
Static File Updated:
{% load i18n %}

{% blocktrans count staticfiles_dirs|length as dirs_count %}Static file path{% plural %}Static file paths{% endblocktrans %}

{% if staticfiles_dirs %}

    {% for prefix, staticfiles_dir in staticfiles_dirs %}

{{ staticfiles_dir }}{% if prefix %} {% blocktrans %}(prefix {{ prefix }}){% endblocktrans %}{% endif %}

    {% endfor %}

{% else %}

{% trans "None" %}

{% endif %}

{% blocktrans count staticfiles_apps|length as apps_count %}Static file app{% plural %}Static file apps{% endblocktrans %}

{% if staticfiles_apps %}

    {% for static_app in staticfiles_apps %}

{{ static_app }}

    {% endfor %}

{% else %}

{% trans "None" %}

{% endif %}

{% blocktrans count staticfiles|length as staticfiles_count %}Static file{% plural %}Static files{% endblocktrans %}

{% if staticfiles %}

    {% for staticfile in staticfiles %}

{{ staticfile|stringformat:"s" }}

<samp>{{ staticfile.real_path }}</samp>

    {% endfor %}

{% else %}

{% trans "None" %}

{% endif %}

{% for finder, payload in staticfiles_finders.items %}

{{ finder }} ({% blocktrans count payload|length as payload_count %}{{ payload_count }} file{% plural %}{{ payload_count }} files{% endblocktrans %})

        {% trans 'Path' %}
        {% trans 'Location' %}

      {% for path, real_path in payload %}

          {{ path }}
          {{ real_path }}

      {% endfor %}

{% endfor %}

Whoops, I meant I'd like to reproduce this with one of my applications. I'll see if using uvicorn and asgi with the example app reproduces it, but I may need you to create a reproducible example otherwise.

Let me know.

@dougwmorrow yeah, I wasn't able to reproduce this on my project. Any chance you can help create a minimal reproducible example?

Will look into it.

https://github.com/jazzband/django-debug-toolbar/blob/22c92e493cae2c96f72ce0021a43405fe1f084c9/debug_toolbar/panels/staticfiles.py#L23-L24
This is potentially broken. I don't immediately see why you'd want to use a Path() object as an argument to static() or {% static %} but it can happen (obviously), and we shouldn't fail.
I'd appreciate a test case, the fix would be to return str(self.path) in def __str__.

Thanks for waiting. I was getting runners high working on my own project. I found the root cause. 3 files of mine showed up as PosixPath objects. I needed to update my vendor files from context_processors.py as seen below.
static_dir = settings.BASE_DIR / "static"
vendor_dir = static_dir / "vendor"
js_files = [str(x.relative_to(static_dir)) for x in vendor_dir.glob("/*.js")]
css_files = [str(x.relative_to(static_dir)) for x in vendor_dir.glob("/*.css")]
No updates are needed for debug_toolbar/panels/staticfiles.html. Let me know if you need any additional information. Feel free to close this out.

I have pushed a fix in #2021 but didn't manage to construct a test for this just yet.
@dougwmorrow If you could test the fix that would be great. Adding str() in your own code -- as suggested above -- would be a good workaround for this issue, but we should fix it for real in the toolbar also.

@matthiask I've tested 2021 push.

I removed str() from [x.relative_to(static_dir)...] etc.
I adjusted for loop from debug_toolbar/templates/debug_toolbar/panels/staticfiles.html as seen below.

{% for staticfile in staticfiles %}

    Index: {{ forloop.counter }}, Type: {{ staticfile|get_class_name }}

  {{ staticfile|stringformat:"s" }}

{{ staticfile|safe_str }}

  <samp>{{ staticfile.real_path }}</samp>

{% endfor %}

I updated debug_toolbar/panels/staticfiles.py to include your suggestions:

class StaticFile:
    """
    Representing the different properties of a static file.
    """

    def __init__(self, *, path, url):
        self.path = path
        self._url = url

    def __str__(self):
        return self.path

    def real_path(self):
        return finders.find(self.path)

    def url(self):
        return self._url

The other suggestions were also added to the file.

The files names were properly output in Static files tab in DJDT as seen below.

Static files
Index: 1, Type: StaticFile
vendor/htmx/htmx.min.js
vendor/htmx/htmx.min.js
src/static/vendor/htmx/htmx.min.js
Index: 2, Type: StaticFile
vendor/flowbite/flowbite.min.js
vendor/flowbite/flowbite.min.js
src/static/vendor/flowbite/flowbite.min.js
Index: 3, Type: StaticFile
vendor/jquery/jquery.min.js
vendor/jquery/jquery.min.js
src/static/vendor/jquery/jquery.min.js

The below output is shown when I use the original code from debug_toolbar/panels/staticfiles.py.

Static files
Index: 1, Type: StaticFile
Error converting to string: str returned non-string (type PosixPath)
src/static/vendor/htmx/htmx.min.js
Index: 2, Type: StaticFile
Error converting to string: str returned non-string (type PosixPath)
src/static/vendor/flowbite/flowbite.min.js
Index: 3, Type: StaticFile
Error converting to string: str returned non-string (type PosixPath)
src/static/vendor/jquery/jquery.min.js
I hope this helps. I plan to update my file paths and so on, but I hope this level of clarity helps. I reverted the for loop in staticfiles.html back to it's original state. Everything looks good.

Thank you! I'm grateful for the additional confirmation that the updated code works.


Cannot assign union typed value to a conditional generic property

TypeScript Version:  4.0.0-dev.20200803

Search Terms: generic discriminant assignment conditional
Expected behavior: the variable animal of type Animal should should be assignable to the property value of AsyncValue<Animal>
Actual behavior: the property value of AsyncValue<Animal> seems to accept only Bird & Reptile, issuing an error.

Related Issues: none
Code
type Bird = {
	readonly type: "bird";
	readonly flying: boolean;
};

type Reptile = {
	readonly type: "reptile";
	readonly swimming: boolean;
};

type Animal = Bird | Reptile;

export type AsyncValue<T> =
	| {
			readonly available: false;
	  }
	| (T extends void
			? {
					readonly available: true;
			  }
			: {
					readonly available: true;
					readonly value: T;
			  });

const animal: Animal = null as unknown as Animal;

const foo: AsyncValue<Animal> = {
	available: true,
	value: animal // error: type Animal is not assignable to type Reptile
};

console.log(foo);

Output
const animal = null;
const foo = {
    available: true,
    value: animal // error: type Animal is not assignable to type Reptile. Property 'swimming' is missing in type 'Bird' but required in type 'Reptile'.
};
console.log(foo);
export {};

Interestingly, if you remove the conditional type (T extends void), it works.

Your type is basically resolved to { value: Bird } | { value: Reptile }, not { value: Bird | Reptile }. Here's a simplified version:
type Foo1 = { readonly value: Bird } | { readonly value: Reptile }
type Foo2 = { readonly value: Bird | Reptile }

const test1: Foo1 = { value: animal } // Compiler error
const test2: Foo2 = { value: animal } // Works

You can disable the distribution by wrapping the arguments in an array, e.g. [T] extends [void].

Wow! I saw something about it, but did not dig any further, as I thought it wouldn't be the culprit. It seems to solve my problem.
Thank you very much, and I am sorry for any inconvenience caused. It is never fun to report something that is working as intended.


Pop Up Trigger not closing

Next Problem with the Pop Up Trigger:
Its opening now with the trigger, but when the state changes its not closing the popup!

Damn! I will fix that then, thanks for the feedback!

Possibly the same issue... Bubble Card 1.6.4. Popup does not respond to click or tap on the popup "X" close button.

Hi, I've just released a new version that should fix that issue 🤞
https://github.com/Clooos/Bubble-Card/releases/tag/v1.7.0


Styre boards via MQTT

You can now disarm, partially arm and fully arm the alarm system via MQTT on home page


Added an optional boolean align-top attribute to d2l-checkbox

Added an optional align-top attribute to d2l-checkbox  that makes the checkbox align with the top of multi-line labels instead of the middle. This alignment is more appropriate for checkboxes in a tree view.

🤔

After some offline discussion, we've decide to make @mpharoah-d2l's change the default behaviour. So multi-line checkboxes will now be top-aligned and there should be no impact on single-line checkboxes.
We'll hold off on merging this until after branching on Friday just to be extra safe we don't accidentally introduce a regression.

@mpharoah-d2l I needed to reduce things to 0.1rem (which is 2px at our default font size instead of the 4px you had) in order to get the single-line visual diff tests to show no differences.
Hopefully that'll still work for your use case. If not, I can experiment with some different approaches other than top-margin.

Actually I backtracked and tried a different approach. I noticed that because our default line-height is 1.4rem and we don't apply the 1.2rem until we get to the text label, the host element actually has 1.4. That's causing an extra 4px of space at the top of the element.
That extra 4px wasn't as noticeable when everything was vertically centred, but when the text top-aligns it becomes more obvious, hence Matt why you needed top drop the text down 4px.
By applying the 1.2rem line-height on the host, and top-aligning both the text and the checkbox, everything is "tight" to the top and maintains the original relative position between the text and checkbox for single-line checkboxes.
The visual diff tests aren't going to like it though!

Ah, that makes sense. I had originally used 0.2rem because I saw the that text had line height 1.4rem, and half the difference of that an 1rem is 0.2rem. But since we actually want 1.2rem, not 1.4, then an 0.1 margin makes sense :+1:

Oh wait, it's because of the different between the checkbox line height and the label line height, I see. So now no margin is needed? That's much better

Yeah exactly. Previously the checkbox & text was 1.2rem line-height but the parent container was incorrectly 1.4rem. And when it was vertically centred that resulted in an extra 0.1rem at the top and bottom -- 2px each. Centred it wasn't super obvious, but as soon as you top-align things it became visible.
So the result of this change is an overall 4px reduction in the height of the component by having 2px less whitespace at the top and bottom. It's a little risky if folks were relying on that space for some reason, but I think it's worth it to clean things up.


KAFKA-5903: Added Connect metrics to the worker and distributed herder

Added metrics to the Connect worker and rebalancing metrics to the distributed herder.
This is built on top of #3987, and I can rebase this PR once that is merged.

This is the final PR for the Connect metrics.
The rebalance metrics don't really make sense on the standalone herder, so I only added them to the distributed herder. The KIP can be changed to reflect this.
Also, the KIP included worker metrics for task startup counts (successes, failures, total, etc.) but neglected to define metrics for connector startup counts. This PR adds them.
I can update the KIP to reflect all of these changes once this is merged.

@ewencp, please have a look at this PR for the last Connect metrics. Thanks!!

@ewencp rebased on trunk and updated this PR to fix the merge conflicts.


Run-TestsInNavContainer - rerun failed tests

I'm currently working together with two partners that are running the entire set of standard test codeunits every night and unfortunately, the test outcome is not predictable, so we're forced to either disable a whole lot of tests or to run the failed tests once again.
I went for the latter and I've built something that reruns failed tests if below a configurable amount of failures.
I now want to migrate to your way of running the tests (Run-TestsInNavContainer) instead of invoking the RTC but I can't find a way to achieve the rerun scenario.
The only option I see is to determine the failed tests and then call the Run-TestsInNavContainer again for every failed test function.
Any ideas?
Thanks in advance!

I will discuss this with the team. We are currently changing the way we run tests to use the same mechanism internally, meaning that we should have a solution for this.

Thanks, sounds promising!
Keep me posted ;)

The way how we do it internally is that we run all the tests one more time. For example if we run 10 codeunits and codeunit 7 has a failing test then we teardown the enviroment, set it up again and rerun all 10 codeunits. The key here is to do multiple test runs - e.g. do not run more than 30 minutes of tests at once.
This is time consuming though. With the new runner you could run the failed codeunits one more time if you would like to save time.
The new runner I'm finishing is able to take an array of the codeunits to run so you can give it only the ones that have failed.

I might then add a flag to run-testsinnavcontainer called -rerunfailedcodeunits :-)

Sounds great, thanks for the quick follow-up.

NavContainerHelper <IP_ADDRESS> is just released and I have modified a few things to allow better control over test execution.
I changed the default interaction timeout to 24 hours – meaning that your tests might just work now without changing anything but upgrading the version.
I also added interactionTimeout as a parameter so you can now control the timeout.
I also changed a few other things, if you want to take advantage of this:
You can now run every codeunit in their own session and control re-runs of codeunits and/or exception handling.
Take a look at this code:
$tests = Get-TestsFromBCContainer -containerName test -credential $credential
$first = $true
$tests.Codeunits | % {
$allpassed = Run-TestsInBcContainer -containerName test                                         -credential $credential
-XUnitResultFileName 'c:\programdata\navcontainerhelper\results.xml'                                         -AppendToXUnitResultFile:(!$first)
-detailed                                         -testCodeunit $_.Id
-returnTrueIfAllPassed
if (-not $allpassed) {
# Rerun?
}
$first = $false
}
The AppendToXUnitResultFile determines whether or not the XUnitResultFile is deleted or appended to.
The ReturnTrueIfAllPassed determines whether Run-Tests will return a Boolean or not.
There is another parameter called -rerun, which determines that the result section in the XUnitResultFile will be replaced with the result of the rerun. If rerun is not specified, there will be two results of the same test codeunit.
Let me know if this solves your usage?


App doesn't support standard fields of Voice Call object

Voice Call standard fields are not yet supported in the HappySoup.io app.

Very few standard fields are supported; this is a limitation of the Salesforce API. Please use app.salto.io to do impact analysis across all fields.


3rd party triggers don't get pretty display names

When you add a 3rd party trigger to your bot it adds the component name to the tree instead of the pretty name. I've tried several different 3rd party packages and they all have the same issue.

@Stevenic The component author can provide a ui schema to set this name. I believe this has been shipped with the latest packages. Can you try with the latest package (1.1.0)?

@a-b-r-o-w-n here's the UI schema for that trigger... Am I missing a property? I set the label:
{
  "$schema": "https://schemas.botframework.com/schemas/ui/v1.0/ui.schema",
  "form": {
    "order": [
      "verb",
      "condition",
      "*"
    ],
    "hidden": [
      "actions"
    ],
    "label": "Action.Execute (invoke activity)",
    "subtitle": "Handle Action.Execute"
  },
  "trigger": {
    "label": "Action.Execute (invoke activity)",
    "submenu": {
      "label": "Adaptive Cards",
      "prompt": "Which action type?",
      "placeholder": "Select an action type"
    }
  }
}

Looks to be fixed in nightly builds.


selection syntax "NONE and something" returns "something" instead of empty selection

Hi,
for examlpe: NONE and helix is equal to helix
Is this wanted? It seems not logical to me

It seems odd, but it's not a bit of the code I know anything about. @arose, is this as intended?

Sounds like a bug to me. Code is here https://github.com/nglviewer/ngl/tree/master/src/selection


EXM 104

EXM 104 module for Synthea
FHIR STU3 export update to add a principal diagnosis to encounters where appropriate

Codecov Report

Merging #4 into abacus will increase coverage by 6.03%.
The diff coverage is 95.38%.

@@             Coverage Diff              @@
##             abacus       #4      +/-   ##
============================================
+ Coverage     75.01%   81.05%   +6.03%
- Complexity     1951     2024      +73
============================================
  Files            72       72
  Lines         13008    13005       -3
  Branches       1793     1794       +1
============================================
+ Hits           9758    10541     +783
+ Misses         2740     1930     -810
- Partials        510      534      +24

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/mitre/synthea/export/FhirStu3.java
88.08% <95.38%> (+0.05%)
133 <0> (ø)
:arrow_down:

...org/mitre/synthea/export/ClinicalNoteExporter.java
75.67% <0%> (-2.71%)
12% <0%> (-2%)

...ain/java/org/mitre/synthea/export/CSVExporter.java
90.86% <0%> (-1.74%)
65% <0%> (+2%)

...org/mitre/synthea/world/concepts/HealthRecord.java
92.22% <0%> (-1.04%)
68% <0%> (+2%)

...ava/org/mitre/synthea/modules/EncounterModule.java
95.87% <0%> (-1.04%)
30% <0%> (-1%)

...va/org/mitre/synthea/modules/WeightLossModule.java
97.27% <0%> (-0.91%)
53% <0%> (-1%)

src/main/java/org/mitre/synthea/engine/State.java
90.65% <0%> (-0.87%)
17% <0%> (ø)

.../main/java/org/mitre/synthea/engine/Generator.java
91.94% <0%> (-0.85%)
55% <0%> (-1%)

...ava/org/mitre/synthea/modules/LifecycleModule.java
90.63% <0%> (-0.51%)
123% <0%> (-2%)

...c/main/java/org/mitre/synthea/export/Exporter.java
43.27% <0%> (-0.37%)
46% <0%> (ø)

... and 9 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 1df1396...9e245c4. Read the comment docs.


Clarify message template implementation status

The docs site says message templates still need to be fully implemented here:

[!NOTE]
Message templates are partially implemented so far. It is planned to implement the standard more completely.

Is there any documentation on what's there vs. what's missing? I haven't found an issue tracking this, but I might have overlooked it.
Thanks for your work! Coming from a .NET background using Seq, and being used to good structured logging, I feel right at home with this library 😄

Hi @bddckr, thanks for trying Klogging!
In answer to your question about message templates, I checked the spec and how much I implemented. I have updated the documentation with details.
If there are features you want, please let me know or have a go yourself and submit a PR.


Address CVE-2015-9284

There is a known vulnerability in OmniAuth that could potentially be used in combination with a vulnerable OAuth provider to gain a signed-in user's session. Whilst DfE Sign-in uses OpenID, not OAuth, the substance of this patch (adding CSRF protection to the "Sign in" link on the
service) is still worth doing.
For more background see:
https://github.com/DFE-Digital/dfe-teachers-payment-service/network/alert/Gemfile.lock/omniauth/open
and https://github.com/omniauth/omniauth/pull/809
which leads to https://github.com/cookpad/omniauth-rails_csrf_protection

 Ensure the OmniAuth Ruby gem is patched and secure


Atom Beautify SCSS disable unitless zero

Description
Since IE11 (and other browsers too) needs Zero with Units, I try to setup Atom Beautify to beautify my SCSS without removing Units from zero values.
I am not able to disable this beautification. So we start using this ugly workaround:
.element {
  flex: 1 1 0.00000001px;
}

Input Before Beautification
This is what the code looked like before:
.element {
  flex: 1 1 0px;
}

Expected Output
The beautified code should have looked like this:
.element {
  flex: 1 1 0px;
}

Actual Output
The beautified code actually looked like this:
.element {
  flex: 1 1 0;
}

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
This beautified code does not look right!

Debug
Here is a link to the debug.md Gist: https://gist.github.com/joel-st/3b906b3bf4568d365a42b3a5bbc2fe2a
Checklist
I have:

[x] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x] Reloaded (or restarted) Atom to ensure it is not a caching issue
[x] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[x] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[x] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue
[x] Asked this question on Stackoverflow: https://stackoverflow.com/questions/58799597/atom-beautify-scss-disable-unitless-zero

If you still need this and are looking for something to fill the spot that Atom left, I am using Pulsar the successor to Atom...   This package is working except for one deprecated error listed in #2605
If you don't need this anymore, can you please close this out so it's not just sitting there? Thanks


TrueNAS Connection refused

Dont need to say much more, i dont not know the propper PUID or anything related if thats the issue as in the log it hangs on "2023-05-26 00:05:04.189246+00:00🔒  Fixing app path file permissions..."

2023-05-26 00:50:04.621796+00:00req: Can't open "/etc/ssl/web/ssl.key" for writing, Permission denied
2023-05-26 00:50:04.624737+00:00ln: failed to create symbolic link '/etc/nginx/sites-enabled/ssl-mixed': Permission denied
2023-05-26 00:50:04.625150+00:00s6-rc: warning: unable to start service configure-ssl: command exited 1
2023-05-26 00:50:04.627273+00:00/run/s6/basedir/scripts/rc.init: warning: s6-rc failed to properly bring all the services up! Check your logs (in /run/uncaught-logs/current if you have in-container logging) for more information.
2023-05-26 00:50:04.627310+00:00prog: fatal: stopping the container.

same here

@matthewtrains @iambenmitchell are y'all passing in custom certs?

Im having an similar issues in TrueNAS Scale. It stays in "🔒 Fixing app path file permissions..." for around 15mins and then proceeds to start normally (and then works like it should.) Seems to happen because of the permissions fixing. This happens each restart
when trying to run the commands manually you use in the docker the command:
"chown -R webuser:webgroup $WEBUSER_HOME"
$WEBUSER_HOME = /var/www/html
seems to be causing the issue as even running it manually takes forever to run (Size of that folder is around 73M and 10642 files in it).
Most of the size seems to be in a folder called "vendor" and 10468 files originate from that folder.
TrueNas Scale seems to use Kubernets. (Im not to fimilar with those systems but i thought it would be helpful to mention)
Contents of vendor folder:
4.5K    autoload.php
119K    awcodes
48K     bin
2.2M    blade-ui-kit
136K    brick
63K     chrisullyott
596K    composer
129K    danharrin
63K     dflydev
3.2M    doctrine
94K     dragonmantank
471K    egulias
2.3M    ezyang
20M     filament
33K     fruitcake
24K     graham-campbell
855K    guzzlehttp
2.2M    influxdata
10M     laravel
156K    laravel-notification-channels
2.3M    league
1.6M    livewire
807K    maatwebsite
234K    maennchen
205K    markbaker
253K    masterminds
755K    monolog
42K     myclabs
4.4M    nesbot
377K    nette
1.5M    nikic
340K    nunomaduro
456K    phpdocumentor
4.4M    phpoffice
37K     phpoption
556K    phpstan
445K    psr
1.4M    psy
28K     ralouphie
853K    ramsey
61K     ryangjchandler
654K    spatie
268K    squirephp
6.6M    symfony
1.3M    tgalopin
87K     tijsverkoyen
247K    vlucas
958K    voku
61K     webmozart

Otherwise your system works great for me. Thanks for the amazing work

@alfi0812 vendor is all the PHP dependencies currently loaded for the tool so it's going to be a bit on the bigger side. I don't have a way to test out TrueNAS at the moment in the home lab so unfortunately can't help with this issue much.


Fengsha/Drag Partition

Case Details:
This case looks at several different things.

The formulation of the horizontal flux is has been malformed according to a paper by Webb et al., 2020. I tested this out using the default configuration with the P7.1 AEROSOL only (so this is using the FV3_GFS_v16 physics suite).

A new drag partition has been developed based on the 2016 paper by Adrian Chappel and Nick Webb. The new inputs are created at a 3 km global resolution monthly average of the input soil parameters for use in both regional and global applications of the Fengsha dust emission scheme. Note that a little more development needs to be done to filter out snow/ice cover in the new drag partition in higher latitudes.  It isn't impacting the results but we are seeing some sources spit out some high latitude dust.

Prigent et al. 2015

Developmental Albedo Drag

Model Configuration Details

Resolution: C384
Begin Date: July 1 2019
End Date: Aug 31 2019
Cold Start: True
Forecast length (FHMAX_GFS): 24 hours
FV3 output: every 6 hours

Aerosol Model Configuration

ANTHRO1 (surface emissions) : CEDS
ANTHRO2 (elevated emissions) : CEDS
SHIP :  CEDS
Biomass Burning: QFED where GBBEPx isn't available
Aviation: HTAP
NH3 Emissions: CEDS
Dust : Fengsha
AOD output: 3 hours

Model Runs and run output directories
base case yaml file: /scratch2/NAGAPE/arl/Barry.Baker/gwp71dev/workflow/cases/aerosol_firex_forecast.yaml

Base case (old drag partition and without the horizontal mass flux fix)
/scratch2/NCEPDEV/naqfc/Barry.Baker/COM/basecase
flux fix with alpha = 8
/scratch1/NCEPDEV/rstprod/tmp/COM/dragfix_alpha8
flux fix alpha = 6
/scratch1/NCEPDEV/rstprod/tmp/COM/dragfix_alpha6
new drag partition + flux fix + alpha=2
/scratch1/NCEPDEV/rstprod/tmp/COM/newdrag_alpha2
new drag partition + flux fix + alpha=4
/scratch1/NCEPDEV/rstprod/tmp/COM/newdrag_alpha4

@bbakernoaa @zhanglikate I have updated the ppt I sent yesterday, updating AERONET plot with "newdrag_alfa2" now reached 3rd week of August. Also in slide 6 comparison with MERRA2 is added now.

Thanks Partha!
I got yelled at to clean up some space so I'm going to backup
newdrag_alpha2 and then delete the rest.  I think that it is the clear
winner here and I'm glad that this has been resolved.
On Fri, Oct 29, 2021 at 1:48 PM perthsb @.***> wrote:

@bbakernoaa https://github.com/bbakernoaa @zhanglikate
https://github.com/zhanglikate I have updated the ppt I sent yesterday,
updating AERONET plot with "newdrag_alfa2" now reached 3rd week of August.
Also in slide 6 comparison with MERRA2 is added now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/noaa-oar-arl/UFS-Aerosol-Config/issues/31#issuecomment-955038808,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFIUVN5N4JDYOIEB5CCMJL3UJMB77ANCNFSM5G52ZS6A
.

@bbakernoaa also your /scratch1/NCEPDEV/rstprod/tmp/COM/dragfix_alpha6 run stopped at 6th August.
But Newdrag Alpha 2 is promising among the 5 runs.

@zhanglikate run ATOM case (201608) with no adjustment of O3 and here is the comparison against run with adjustment of O3 :
https://docs.google.com/presentation/d/1rmboDAafQDEuUJ8ZxPQvzLaqpakL9Nlu/edit?usp=sharing&ouid=100966050662930876047&rtpof=true&sd=true