Full Library Sync: KeyError: u'year'

Bug report
Your Environment

Netflix add-on version: 1.4.1
Operating system version/name: Android
Device model:  Nvidia Shield

Used Operating system:

[X] Android
[ ] iOS
[ ] Linux
[ ] OSX
[ ] Raspberry-Pi
[ ] Windows

Describe the bug

I'm trying a full library sync with my list. The entries are deleted. After that while the entries are added again, I get the error.
Expected behavior

The Sync should run without error
Actual behavior

I get an error while syncing and the sync is stopping
Steps to reproduce the behavior

Go to Addon Setting --> Library
Select "Perform complete sync now" (don't know if this is the exact translation)

Possible fix

Don't know
Debug log

The debug log can be found from this link:
kodi.log
(pastebin and hastebin gave me errors)
Additional context or screenshots (if appropriate)
Installation

[X] I'm using other Netflix Repo: CastagnaIT Repostiry for Kodi 18.x 1.0.1 +leia
[ ] I'm using a different source

Other information

Screenshots

The log is without nf debug info,
follow readme instruction to get a right log

strange, sorry. It actually was activated.
Next try:
kodi.log

yes the problem is Rush Hour,
should not be on my list, the movie is not available to play yet
how did you add that?

really? Here in Germany, it is available:

wow.... in Italy not yet

something went wrong with the metadata
and it is strange because i can get the full metadata without problem at least from here

Can you do a test?
update with this:
plugin.video.netflix_1.4.1_20200615.zip
i think you have enabled the auto sync with my list right?
then, find Rush hour and add it to my list (with the add-on)
should be appear the same error then get me the log
thanks

Can you do a test?
update with this:
plugin.video.netflix_1.4.1_20200615.zip
i think you have enabled the auto sync with my list right?
then, find Rush hour and add it to my list (with the add-on)
should be appear the same error then get me the log
thanks

okay, it was added to my list, but the same error appeared
kodi.log

apparently the metadata is mixed with "Der Prinz von Bel-Air" tvshow
then raise error
you can try this to see if error happen again:
1 remove it from mylist
2 go to addon settings, expert settings, and do: Clear in-memory and on-disk cache
3 add it again to my list

okay that helped! There is no error anymore. Thanks for the quick fix 👍


Allow factory extensions to declare supported items

Hi,
Would it be possible to add a feature to factory item extensions that allows them to specify on which items they operate? Something like public function matches(array $options = array()); that would return true or false based on the state of $options argument?
I have three factories that act on a object of the same class passed to the menu item factory, but all of them are mutualy exclusive and their execution depends on the internal state of the object. In order not to repeat the logic to skip the extension in buildOptions and buildItem, it would be awesome to have a third method specifying if extension can be used on an item, which would then the factory read and act upon.
In this scenario, in RoutingExtension for example, check for existence if (!empty($options['route'])) would move to the new method.
I'm willing to create a PR if this is something you deem acceptable!
Thanks for listening!

Well, this would not change things much, because the MenuFactory would still be calling matches twice. And it would make things worse for cases where one of the methods is currently a no-op (like the RoutingExtension) as it would re-run the logic.
If you don't want to duplicate the logic (in case it is complex), you could use a private method to share the same logic between buildOptions and buildItem.
Thus, the condition to skip the logic in each method may even not be the same for all extensions (as you could also pass some options explicitly which correspond to the output of buildOptions and still expect the item building to act on them).
Can you give more details about your use case ?

I agree with you, it's not really useful here. My usecase was a bit specific and a bit convoluted with using eZ Platform, but I solved it by implementing my own Knp\Menu\FactoryInterface with its own set of extensions and rules, thus keeping the original factory clean.
I will close the issue, but thanks for your reply and insight!


Android render svg crash app on real device

Environment
OS: macOS High Sierra v10.13
Packages: (installed)
"react-native-svg": "^9.13.3",
"react-native-svg-transformer": "^0.13.0",
"react-native": 0.61.3
Actual Behavior
I build apk release and running app crash on SVG render (crash on a real device)
Note: when I'm commented SVG then it works fine.
Plz! help me find out the solution to fix this issue! Many Thanks!

@msand
Just the content of my component :
import React, { Component } from 'react';
/* Images */
import SearchIcon from '../../../assets/images/icons/search-icon.svg';
class FilterSearchAreaComponent extends Component {
  constructor(props) {
    super(props);
  }
  render() {
    return (
       <SearchIcon
         width="40"
         height="40"
       />
    )
  }
}

export default (FilterSearchAreaComponent);

Here is search-icon.svg file:
https://drive.google.com/file/d/1IqjEZexLba7gkh0clErr9_IZPHlDg7xb/view?usp=sharing

You fixed the issue?

You fixed the issue?

Yes, I was mistaken about this:

Troubleshooting
Problems with Proguard
When Proguard is enabled (which it is by default for Android release builds), it causes runtine error To avoid this, add an exception to android/app/proguard-rules.pro:
-keep public class com.horcrux.svg.** {*;}

You fixed the issue?

Yes, I was missed about this:

Troubleshooting
Problems with Proguard
When Proguard is enabled (which it is by default for Android release builds), it causes runtine error To avoid this, add an exception to android/app/proguard-rules.pro:
-keep public class com.horcrux.svg.** {*;}

This worked for me. Thank you saved my ton of time.

You fixed the issue?

Yes, I was missed about this:

Troubleshooting
Problems with Proguard
When Proguard is enabled (which it is by default for Android release builds), it causes runtine error To avoid this, add an exception to android/app/proguard-rules.pro:
-keep public class com.horcrux.svg.** {*;}

This worked for me. Thank you saved my ton of time.

This worked for me too . Save my whole day .Thanks !

how did you do this? can you explain step by step?

how did you do this? can you explain step by step?

Did you fix it?
You need to input below text in android/app/proguard-rules.pro.
-keep public class com.horcrux.svg.** {*;}

You fixed the issue?

Yes, I was missed about this:

Troubleshooting
Problems with Proguard
When Proguard is enabled (which it is by default for Android release builds), it causes runtine error To avoid this, add an exception to android/app/proguard-rules.pro:
-keep public class com.horcrux.svg.**


Microsoft.RecoveryServices is missing azureresourceschema configuration for API version 2020-02-02

API version 2020-02-02 was added here:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/recoveryservices/resource-manager/readme.md
But is missing here:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/recoveryservices/resource-manager/readme.azureresourceschema.md

@jianyexi - I can see version 2020-02-02 present at https://github.com/Azure/azure-rest-api-specs/blob/master/specification/recoveryservices/resource-manager/readme.azureresourceschema.md
but not 2020-10-01...
Can you please help me understand the issue here (is it really 2020-02-02)?

@anthony-c-martin - I can see version 2020-02-02 present at https://github.com/Azure/azure-rest-api-specs/blob/master/specification/recoveryservices/resource-manager/readme.azureresourceschema.md
but not 2020-10-01...
Can you please help me understand the issue here (is it really 2020-02-02)?

Looks like this was fixed with #12853 - closing.

@anthony-c-martin , I have 1 concern though -
API version 2020-10-01 was added here:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/recoveryservices/resource-manager/readme.md
But is missing here:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/recoveryservices/resource-manager/readme.azureresourceschema.md
Should I add 2020-10-01 in readme.azureresourceschema.md?
Are we required to add new versions in both readme.md and readme.azureresourceschema.md?

@amchandn - yes, maintaining both readme.md and readme.azureresourceschema.md is required going forwards.
I see 2020-10-01 was added after @leni-msft's commit. Let me know if you'd like me to raise a separate issue for that.

Yeah, could you please raise an issue for adding 2020-10-01? Meanwhile, I'll start working on the fix for the same

Yeah, could you please raise an issue for adding 2020-10-01? Meanwhile, I'll start working on the fix for the same

Thanks! I've raised #13093


Android Sync

System Details

OS: Android 13 Stable (Sept. Security Update) (Google Pixel 6a)
Thorium Version: 116.0.5845.184

Problem
A clear and concise description of what the bug is.
After downloading the ARM 64 version and installing it, it prompts me to add a new account instead of the one currently present on the device. I obviously cannot add an account already on the device.
Screenshots
If applicable, add screenshots to help explain your problem.

Additional Notes
Add any other context about the problem here.

@AccursedAngel This is a known issue with the Android version. Unlike the Desktop versions, where just adding API Keys is sufficient to enable Sync, I believe with Chromium on Android there is an additional proprietary component that I don't have access to that is needed. Case in point, there are very few Chromium based browsers for Android with Sync supported. I don't know how to fix this, but I can ask in the chromium-dev google group for help.

Ah I gotcha, thanks for replying. Sorry about the duplicate posts here and on Reddit. <3

hey, did you receive any updates?

bugs.chromium.org/p/chromium/issues/detail?id=677887
Google restricts login and synchronization of third-party versions, and there is currently no solution
@AccursedAngel @aniruddhgutta

Hey,
Do you see any possibility of having your own sync server, or being able to use a self-hosted sync server ?
Maybe a startingpount: https://community.brave.com/t/self-hosted-sync-server/305806

There should be no plans to build a synchronization server at the moment.
@DragonHeart69

why is this closed? this is ongoing

why is this closed? this is ongoing

What was closed?
@komprimiert

Can't you guys implement something like brave? brave doesn't use google sync ry8

Is there any solution to implement a custom synchronization server at this stage?  If there are relevant solutions on the Internet, please provide them to me here and I will consider porting the relevant functions to Thorium.
If the Thorium team is required to deploy a synchronization server, we may not currently have sufficient funds to do this work.
Thorium is different from Brave. The Brave browser is supported by a company and a large amount of advertising revenue, and it also has more developers than Thorium. Please understand the different situations of the two.
@Subhashis2007

Is there any solution to implement a custom synchronization server at this stage? If there are relevant solutions on the Internet, please provide them to me here and I will consider porting the relevant functions to Thorium.
If the Thorium team is required to deploy a synchronization server, we may not currently have sufficient funds to do this work.
Thorium is different from Brave. The Brave browser is supported by a company and a large amount of advertising revenue, and it also has more developers than Thorium. Please understand the different situations of the two.
@Subhashis2007

got it,  well I hope so more and more developers join this, and may be giving an option whether to enable sponsored images on homescreen to generate some funds would be great too, in this way the users can contribute

OK, I'll take a look
@DragonHeart69

@gz83 for me, a selfhosted docker is good. The link to the brave selfhosted is in my other comment. I understand that you can't host a server for all the users.

can you link the comment? I'm interested as well

@gz83 for me, a selfhosted docker is good. The link to the brave selfhosted is in my other comment. I understand that you can't host a server for all the users.

can you link the comment? I'm interested as well

https://community.brave.com/t/self-hosted-sync-server/305806

Would some kind of implementation of MicroG help?

Yeah this is a very big problem in the Android version. Sync in all devices is very important for me at least. But its normal this version is still without extensions and other features also. The MacOS and Windows versions are very very good and I will probably live without the sync in the time being XD

Any way to import a bookmarks file?

@AccursedAngel This is a known issue with the Android version. Unlike the Desktop versions, where just adding API Keys is sufficient to enable Sync, I believe with Chromium on Android there is an additional proprietary component that I don't have access to that is needed. Case in point, there are very few Chromium based browsers for Android with Sync supported. I don't know how to fix this, but I can ask in the chromium-dev google group for help.

Thats fair. Is it possible to add support for extensions instead? Maybe someone will make an extension to another cloud storage provider like OneDrive and we can add it to Thorium

Rather than using Google sync or creating a Thorium sync server with all the overhead that entails, you could allow the use of the customer's own online storage, perhaps through OneDrive, Box, Google Drive , Dropbox, etc. with Thorium data being stored there, perhaps in an encrypted form and used to synchronise between platforms?

Hi Devs
First launch Thorium Mobile prompt login to Google account with Passkey fingerprint. Validate do nothing. Adding Google Account via e-mail make error - e-mail address is already in use on this device.
Also installing your version of Android Web View and disabling built in system version of Android Web View make Play store components crash. I can create separate issue if needed. Regards

can we except anything on this or is it dead in the water ?

7 months have passed and this problem has not been solved

Would it be possible to have Thorium sync with some a service other than google chrome?
Perhaps something like being able to sync to firefox ? Not sure how feasible this would be.

Would it be possible to have Thorium sync with some a service other than google chrome?
Perhaps something like being able to sync to firefox ? Not sure how feasible this would be.

It could possibly entail a complete rewrite of the existing sync system.

So?

So?

I'd imagine that's a lot of extra work with little benefit when compared to switching to something like MicroG.

Youtube revanced also has this problem on unrooted devices. wouldnt it be possible to create a module for magisk so us, rooted users can log in into thorium?

YouTube revanced + MicroG allows signin

Youtube revanced also has this problem on unrooted devices. wouldnt it be possible to create a module for magisk so us, rooted users can log in into thorium?

YouTube revanced + MicroG allows signin

yh I know but the dev said that he won't be implementing microg any time soon.

How about creating a pledge goal for this?

How about creating a pledge goal for this?

 I like it ! 😁


Adding Song With No Library

If there is no library and you try to add a song, there is an array index out of bounds exception.

Will need to disable the new song button in the menu if there is no library.

Fixed as of 566c453.


⚠ Bofrost.de has degraded performance

In 5b8ea58, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 867 ms

Resolved: Bofrost.de performance has improved in aaad251 after 5 minutes.


[New Port Request] rapidyaml

Library name:
rapidyaml
Library description:
"Or ryml, for short. ryml is a library to parse and emit YAML, and do it fast."
Source repository URL:
https://github.com/biojppm/rapidyaml
Project homepage (if different from the source repository):
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.

The port is already added, named "ryml" (#10793), that is also correct name.
From home: "Or ryml, for short. ryml is a library to parse and emit YAML, and do it fast."

Ah. I see.

No, no, the port was added after about month after your request.

No, no, the port was added after about month after your request.

Feeling better now :)


F# values displaying strangely in notebooks

Describe the bug
In general, I have found that F# is now displaying values strangely in VS Code. I feel that it didn't used to be this way, so maybe I missed a feature change or it's possible that this is a regression.
For example, see the following screenshot:

For the most part, I am now usually defining a helper function let print value = printfn "%A" value and piping any output into print to get reasonable and readable displays of values.
Please complete the following:
Which version of .NET Interactive are you using? (In a notebook, run the #!about magic command. ):

OS

[x] Windows 11
[ ] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[x] Visual Studio Code
[ ] Visual Studio Code Insiders
[ ] Visual Studio
[ ] Other (please specify)

Screenshots
See above. Also, here's version information:

Just an FYI, rather than printfn "%A" you can change the default output to the FSI formatter using the trick from https://github.com/dotnet/interactive/issues/1282
Formatter.SetPreferredMimeTypeFor(typeof<obj>, "text/plain")
Formatter.Register(fun (x:obj) (writer: TextWriter) -> fprintfn writer "%120A" x )

A downside is that it will break any libraries with their own interactive printing, such as Plotly.NET, but you can fix those with for example
// Make plotly graphs work with interactive plaintext formatter
Formatter.SetPreferredMimeTypesFor(typeof<GenericChart.GenericChart>,"text/html")


Endpoints cleanup

Based on RFC-0016 https://github.com/iver-wharf/rfcs/pull/16

Restructuring the wharf-api by changing the paths of the endpoints, adding and renaming path parameters, and changing the request and response models around.
While we will keep backward compatibility for at least one major version, we will be changing so much that this needs a full major version bump.
Please see the RFC for further details: RFC-0016

Child issues:

[ ] #67
[ ] #68

Closing in favor of the milestone: https://github.com/iver-wharf/wharf-api/milestone/1


Zoho site

Related Issue
Closes #958
Description
Zoho Sites is a website builder and hosting platform provided by Zoho Corporation. It allows individuals and businesses to create and manage professional websites without requiring extensive technical knowledge or coding skills.
Screenshots

Checklist

[ ] My code adheres to the established style guidelines of the project.
[ ] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

The PR title format you are using is causing an error in the workflow. Please follow the instructions in the README.md and change it accordingly.

I dont think ZOHO is free

I dont think ZOHO is free

15 days free trial

We need something that can be freely used


Expand env output

We've been printing the version of OpenSSL that Ruby was compiled against, but not the version that was dynamically linked at runtime, which is a huge potential issue. While I was there, I added a few more things that seemed helpful.

Updated. @indirect r?

looks great! @bundlerbot r+


Theory symbols in patterns

We should triage and address all the warnings mentioned in  PR #1666

Starting cleaning it up in the _ulib_theory_symbols_in_patterns branch.

Fixes for List and Seq pushed to master.


Integration Issue: Conflicting Functionality between DICOMCast and Smart on FHIR

I've successfully deployed DICOM Cast and SMART on FHIR in my Azure environment. However, I am encountering an integration issue related to the audience configuration.
During the deployment of DICOM Cast, The audience of the FHIR service to be the same as the FHIR service's endpoint, aligning with the default behavior.
Conversely, during the deployment of SMART on FHIR, the audience is set to the Application ID URI of the fhir-resource-app. This divergence in audience configuration is causing a conflict in integration.
To make DICOM Cast work, I need to align the FHIR service audience with the FHIR endpoint. However, doing so disrupts the functionality of SMART on FHIR.
Aligning the FHIR audience with the application ID URI of fhir-resource-app allows SMART on FHIR to work but breaks DICOM Cast.
I need guidance on how to reconcile these conflicting audience configurations to ensure the seamless integration of DICOM Cast and SMART on FHIR.

Can you elaborate more on this sentence
"To make DICOM Cast work, I need to align the FHIR service audience with the FHIR endpoint. However, doing so disrupts the functionality of SMART on FHIR."
Did you try using the same audience as Smart on fhir for fhir audienc ein this step? https://github.com/microsoft/dicom-server/blob/main/docs/how-to-guides/sync-dicom-metadata-to-fhir.md#update-key-vault-for-dicom-cast

Hello @poadhika,
I hope this message finds you well. I wanted to bring up an issue I encountered while setting up the Application ID URI as the Audience for the FHIR App Service, as mentioned in the documentation link.
In my setup, I observed that when I set the FHIR App Service audience to the Application ID URI, it doesn't work as expected. However, if I keep the FHIR App Service audience as the default one (same as the FHIR URL), everything functions correctly.
I understand that it's crucial to set the FHIR App Service audience as the Application ID URI for Smart on FHIR integration. However, the current configuration seems to be causing an issue.
I would appreciate any insights or guidance on how to resolve this matter. If there are specific steps or additional details needed, please let me know, and I'll provide them promptly.
Thank you for your attention to this matter.

Hello @poadhika
Have you been able to look into this?
I'm facing issues while setting up the Application ID URI as the Audience for the FHIR App Service as mentioned in the documentation link.
I've faced this issue while deploying Dicomcast and I've posted an issue-here
I would appreciate your support in this matter.

It seems like you are having issues with Smart onFhir integration,  it would be helpful to add the issue to the FHIR service. I would recommend reaching out to the FHIR team for further assistance to ask about if there is any thing needed to be configured  furthur to work on smart on fhir.

I've deployed DICOM Cast successfully. And I have performed all the steps of Sync Medical Imaging Server for DICOM metadata into FHIR Server for Azure. The configuration involves three steps:

Create Resource App
Set Authentication
Update Key Value

Successfully completing all three steps should retrieve the corresponding FHIR resource (Patient's data) in the FHIR service.
However, I'm facing an issue where following all the steps doesn't yield the expected result.
Surprisingly, if I skip the second step (set authentication), I can retrieve the FHIR resource successfully.
This issue might be interlinked with the above matter I described, and resolving that might automatically fix this integration issue. Looking for guidance and solutions to ensure seamless integration.
Please look into this matter as soon as possible.


[Question] Deployment strategy when dealing with multiple big databases

While migrating to Babelfish (which so far is great!!) I ended up with a dilemma, what's the best deployment strategy when SQL Server was hosting several (>5) big databases (from 500GB to 1TB each).
The plan would be to progressively move to Postgres in the future, therefore the single-db seems the best choice, but this means running multiple Babelfish instances on the same server, each on a different ports (e.g 1433, 1434, 1435, etc).
Are there issues in running multiple Postgres instances on the same OS?
Could this make backups easier?
The lack of easy users management (so far mitigated only with sysadmin users) also seems to apply better to the single-db mode, is this true?
Thanks!

To achieve the "SQL Server experience" with Babelfish, I strongly recommend multi-db mode. Having said that, if you're objective/plan is to migrate off Babelfish to PostgreSQL pretty soon then single-db mode makes sense but IMO that's the only scenario.
Do not use pg_dump for backups as it's not supported for Babelfish. The world wide internet may have advise; personally I recommend bbf_dump/bbf_restore - watch this space.

Thanks! I was not aware of bbf_dump/bbf_restore commands, there is no documentation (or is hard to find), so I do believe your reply can be useful to others as well!
I will leave this open and close as soon as I verify, to provide a complete answer.

The utilities are coming, so there's no documentation yet. But watch this space.

We use a number of small (usually 5-10 GB) DBs with MSSQL and need to continue supporting MSSQL along with Babelfish, so using only multi-db mode. After experimenting quite a bit with Babelfish-flavoured pg_dump/restore (added some notes about it here) they seem very close to be fully working. Hit a number of minor bumps (like lack of DB rename), going to re-check it with the latest Babelfish (we stick with 3.3) and file issues here about some of them.


changing draft version to .38 and removing manual appname

Original PR: https://github.com/Azure/vscode-aks-tools/pull/830
Had to update for latest draft version and retest.

Tested on my end. Closing now :)


[BUG] epicli init --full does not generate configuration for OpenDistro

Describe the bug
epicli init --full does not generate configuration for OpenDistro. There is no kind: configuration/opendistro-for-elasticsearch in the output file.
Probably the reason is that OpenDistro component is set to 0 by default and therefore no configuration is generated for it.
https://github.com/epiphany-platform/epiphany/blob/develop/schema/aws/defaults/configuration/minimal-cluster-config.yml
https://github.com/epiphany-platform/epiphany/blob/develop/schema/aws/defaults/epiphany-cluster.yml
Note: Fix should be applied to all providers.
To work around: add kind: configuration/opendistro-for-elasticsearch manually and adjust it to your needs or do nothing to use defaults.
However, I think that epicli init --full should generate the full configuration for each component.
Please also see: #2968.
How to reproduce
Steps to reproduce the behavior:

execute epicli init -p {aws, azure, any} -n test --full
Verify generated file

Expected behavior
There should exist a configuration for each component, including OpenDistro.
Environment

Cloud provider: [all]
OS: [all]

epicli version: [2.0.0dev]

DoD checklist

Changelog

[ ] updated
[ ] not needed

COMPONENTS.md

[ ] updated
[ ] not needed

Schema

[ ] updated
[ ] not needed

Backport tasks

[ ] created
[ ] not needed

Documentation

[ ] added
[ ] updated
[ ] not needed

[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade
[ ] backup/restore

[ ] Idempotency tested
[ ] All conversations in PR resolved

Added the OpenDistro machine to the minimal cluster config.
Additionally, the configuration for kind: configuration/opendistro-for-elasticsearch is generated with the init --full command.


Feature Request: Multi source lookups

I've been using this a bit and I really like it in server mode. Excellent as a looking glass. Nice crisp, colourful output.
And I've discovered aha, which is now my new favourite toy.
I can't help wondering if we could get this to execute on a bunch of remove machines (via ssh and keys) and output the results to the same page, one after the other. A bit like mtr.sh
What are you thoughts?

That's an interesting idea. It can be done very easily with an external script aggregating output from different asn server instances, for example if you want textual traces from multiple servers, just add them to the $asnservers variable and run the script below. It should output textual results on screen. Of course this can be obtained using html, or even aggregating the html in a nice looking page with, say, sections reporting the asn server source country for every result etc. - I think this all belongs to an external script, and it should be not be hard to make. Let me know if this is what you meant.
Thanks for the feedback!
#!/usr/bin/env bash

# change this to the list of your asn server instances, space separated
asnservers="<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>"

target="$1"
output=""
for asnserver in $asnservers; do
    echo "Running asn lookup for '$target' on server $asnserver"
    output+="$(curl -s "http://$asnserver:49200/asn_lookup&$target" | sed -n "s:.*id='tracedata'>\(.*\)
.*:\1:p" | base64 -d | gunzip)\n\n"
done

echo -e "$output" | less

The script above is meant to be invoked with the <target> parameter on command line. For example ./multiasn <IP_ADDRESS>

Clever. I hadn't thought of looping a curl.
How would you get that output back into a browser window with the pretty colours though? 😁

Well in the above script, I just extracted the tracedata div content from the html - which is just a gzipped + base64-encoded version of the textual asn output, which I use when sharing the text-only version on termbin.
That means you can instead grab the full html (just take out the sed part) and that should do the trick.

I reckon perhaps the above needs a clearer explanation: when I run a remote trace, I instruct the remote server to run a trace and convert the trace output to html (using your new friend aha ;-) in order to display it back to the client.
In the meantime, since I needed an easy way to share the text output on termbin if the client so decided, and since converting it from html back to text was rather stupid (since in the process I had the text server-side all along), I decided to append (on server side) a "text-only" version of the trace output to the output HTML, so that the client already had what it needed to feed to termbin. I needed a way that was lightweight (ie: gzip) and - since it had to be passed along in a web page - textual-only (no non-printable characters, ie: base64).
The method I used was to append a
 (having id=tracedata) to the output HTML. That way I could generate the termbin-sharing code dynamically on the client side.

I just hacked together what I had in mind. Output looks like this (sorry, I'm not a web designer :-P) :

Note: you need GNU sed for this to work. So you should substitute all occurences of sed with gsed if running on MacOS/FreeBSD (or use something along the lines of the CoreutilsFixup() function I used in the main script).
#!/usr/bin/env bash

# change this to the list of your asn server instances, space separated
asnservers="<IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS>"
# output html filename
output_file="output.html"

# no changes needed below

target="$1"
[[ -z "$target" ]] && { echo "Usage: $0 <target>"; exit 1; }

green=$'\e[38;5;035m'
red=$'\e[38;5;203m'
default=$'\e[0m'

read -r -d '' output <<- END_OF_HTML_HEADER
	<?xml version="1.0" encoding="UTF-8" ?>
	<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

END_OF_HTML_HEADER

# cheers https://www.cssscript.com/responsive-css-accordion-tabs-component/
read -r -d '' accordioncss <<- END_OF_ACCORDION_CSS

END_OF_ACCORDION_CSS

first_run=true
srvnum=1
for asnserver in $asnservers; do
    echo -en "${default}- Running asn lookup for '$target' on server $asnserver..."
    tmp_output="$(curl -s "http://$asnserver:49200/asn_lookup&$target" | grep -v 'class="loader"')"
    if [ -z "$tmp_output" ]; then
        echo "${red}Error!${default}"
    else
        if [ "$first_run" = true ]; then
            # grab the  tag from the first successful run, so to apply proper styling to the remaining results
            htmlhead=$(sed -zn "s:.*\(.*\).*:\1:p" <<<"$tmp_output")
            output+="\n\n\n$htmlhead\n$accordioncss\n\n\n\

Multiple ASN trace to target:
$target

 \

"
        fi
        # extract  content for this trace
        htmlbody=$(sed -zn "s:.*\(.*\).*:\1:p" <<<"$tmp_output")
        # clean up the html by stripping unneeded js
        htmlbody=$(\
            sed -e "s/


Fix - [Bug Report] Localization #11961

Fix - [Bug Report] Localization #11961
Please make sure these boxes are checked before submitting your PR, thank you!

[X] Make sure you follow Element's contributing guide (中文 | English | Español).
[X] Make sure you are merging your commits to dev branch.
[X] Add some descriptions and refer relative issues for you PR.

Thank you @jikkai. When this commit will be tagged? Will there also be a v2.1.1 tag? Or only a v2.4.4 tag?


Not working on IE11 due to the use of arrow function

I started using this module in the frontend using Webpack then I realized it was breaking my app on IE11 because it uses an arrow function which is not supported. Would be great to change it for a normal function to have a wider range of supported browsers. Thanks!

Duplicate of #18


feat(pie-chip): DSW-2499 add the disabled state functionality

Describe your changes (can list changeset entries if preferable)

[Fixed] - Prevent emitting events when the chip is disabled
[Updated] - When isDismissible is set to true, only the close icon is interactive.

Author Checklist (complete before requesting a review)

[x] I have performed a self-review of my code
[x] I have added thorough tests where applicable (unit / component / visual)
[ ] I have reviewed the PIE Storybook/PIE Docs PR preview
[x] I have reviewed visual test updates properly before approving
[x] If changes will affect consumers of the package, I have created a changeset entry.
[x] If a changeset file has been created, I have used the /snapit functionality to test my changes in a consuming application

Reviewer checklists (complete before approving)
Reviewer 1

[ ] I have reviewed the PIE Storybook/PIE Docs PR preview
[ ] If there are visual test updates, I have reviewed them

Reviewer 2

[ ] I have reviewed the PIE Storybook/PIE Docs PR preview
[ ] If there are visual test updates, I have reviewed them

/snapit


Build in only selected Envoy extensions

Currently Istio's proxy seems to have all existing Envoy extensions built in even though some of them are not used in meshes, e.g. Dubbo, Lua extensions etc.
Would it be useful for Istio to have control over the list of extensions included in the proxy with an Istio-specific config like in this patch https://github.com/rojkov/proxy/commit/a27459acd57ac9149db1bee9e0fe5194f9b8cef2?
[ ] Docs
[ ] Installation
[ ] Networking
[x] Performance and Scalability
[x] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

While we may not use them in the core, folks are depending on these in Envoy Filters so I am not sure it's safe to remove


Error loading v1.3.14

Loading Madclowns Nuclear v1.3.14 gives me this error

Mod list:
https://hatebin.com/cbbeyrksnc

my bad, ill get that patched ASAP


Ajax'd site not firing lazysizes

Hi,
Sometimes, mostly on iOS devices, lazysizes isn't self-initializing.
My site is built with ajax architecture so JavaScript is re-initialised when Ajax return is done, but sometimes lazysizes isn't firing, I've tried to do a manual init, but still no joy, is there a hook I can test to double check init ?
        window.lazySizesConfig = window.lazySizesConfig || {};
        window.lazySizesConfig.init = false;
        window.lazySizesConfig.lazyClass = 'lazy';
        lazySizesConfig.expand = -100;`

        window.lazySizes.init();

        // Double check it's initiated
        // Nothing to hook onto
        setTimeout(function() {
            window.lazySizes.init();
        }, 1000);

Thanks,
Tom

Actually there shouldn't be the need to it. You might want to try the following:
setTimeout(function() {
            lazySizes.loader.checkElems();
}, 1000);

But I doubt, that this helps. Can you give me a link to your site to test myself?

I just saw, that you have this:
lazySizesConfig.expand = -100;

You should really use only positive numbers here:
lazySizesConfig.expand = 100; // at least 50

I really suspect, that this is your issue.

Ok, thanks, will give that a try and report back. Wanted to delay the lazyload, but this can be done with animation-delay, so no need for the negative value now.

I'm also experiencing the same issue with an ajax site - however on IE 11 (groan).
(Works fine in Firefox, Chrome and Safari btw.)
The problem seems to be with images with srcset style markup - images with bgset are loading correctly.
When I check the markup of srcset images that aren't showing - they're missing the src=".." attribute.
If I open the Developer Tools this seems to prompt a refresh of some kind and the images then appear...
I'm using unveilhooks, parent-fit & bgset plugins with Lazysizes.
Tried implementing lazySizes.loader.checkElems(); but still not working...

@floq-design
Thanks, I don't think this is the same issue. For IE you additionally need a polyfill like resoimg plugin or picturefill.
Can you send me a linke to your site or a reduced testcase?

@aFarkas Thanks for the rapid response.
I'm using Picturefill v3.0.2 as my polyfill.
Unfortunately I'm developing in Wordpress locally - no live version to show. Creating a reduced test case would be a lot of work. Thanks for the offer though.
Just tried (out of curiosity) using v1.5 of Lazysizes but same problem :(
Are there any other initalising methods for lazysizes?

Just tested same site under Windows 10 with IE Edge - and I'm getting the reverse of the problem in IE 11... so bgset images are not loading however srcset images are.

To clarify, I too am use WordPress. The output of the images are:
<img width="1000" height="667" class="attachment-6x4-medium size-6x4-medium lazy fade-in" alt="x" srcset="http://x.wpengine.com/wp-content/uploads/2016/06/x-1000x667.jpg" data-src="http://x.wpengine.com/wp-content/uploads/2016/06/x-1000x667.jpg" data-srcset="http://x.wpengine.com/wp-content/uploads/2016/06/x-1000x667.jpg 1000w, http://x.wpengine.com/wp-content/uploads/2016/06/x-300x200.jpg 300w, http://x.wpengine.com/wp-content/uploads/2016/06/x-768x512.jpg 768w, http://x.wpengine.com/wp-content/uploads/2016/06/x-1320x880.jpg 1320w, http://x.wpengine.com/wp-content/uploads/2016/06/x-600x400.jpg 600w, http://x.wpengine.com/wp-content/uploads/2016/06/x.jpg 1400w" data-sizes="(max-width: 1000px) 100vw, 1000px">

Please I really need a testcase or a link. It should be easy to simply save some html output including all CSS and JS.

@aFarkas Just want to report my conclusions.
Firstly I was using JS to update the sizes attribute on images without including ls.attrchange.min.js.
Secondly using this code: https://github.com/scottjehl/picturefill/issues/644#issuecomment-188809805 to re-prompt picturefill resolved missing images on ajax load.
Thanks again for the rapid response.

None of those suggesions have worked.
I can send you a staging link @aFarkas , do you have an email ?

<EMAIL_ADDRESS>

@CHEWX
I got your email. The problem is your lazysizes configuration code. Your code is executed after DOM ready, but you need to execute it immediately. In case you want to wait you can set lazySizesConfig.init = false;, but in this case you also have to do this immediately.
You current code looks something like this:
    lazysizes: function() {
		window.lazySizesConfig = window.lazySizesConfig || {};
		window.lazySizesConfig.init = false;
		window.lazySizesConfig.lazyClass = "lazy";
		lazySizesConfig.expand = 100;
		window.lazySizes.init();
		setTimeout(function() {
			lazySizes.loader.checkElems();
		}, 1e3);
	},

You can modify it to this:
    lazysizes: (function(){
		window.lazySizesConfig = window.lazySizesConfig || {};
		window.lazySizesConfig.init = false;

		return function() {
			window.lazySizesConfig.lazyClass = "lazy";
			window.lazySizesConfig.expand = 100;
			window.lazySizes.init();
		};
	}),

and it should work.

I just modified the code. The working config is looking like this:
    lazysizes: (function(){
		window.lazySizesConfig = window.lazySizesConfig || {};
		window.lazySizesConfig.init = false;

		return function() {
			window.lazySizesConfig.lazyClass = "lazy";
			window.lazySizesConfig.expand = 100;
			window.lazySizes.init();
		};
	})(),

Please close this issue, if it works for you.

Looking good so far @aFarkas - thanks.
Will re-open if the problems arises again.

Is it possible that this is not working again with latest version of Chrome? It's working for me for every other browser, even IE9 but not Chrome. Can somebody confirm this?


Stormblessed: Watch [【】]

Stormblessed requests the watch of the watch_keyword [【】]. See the MS search here and the Stack Exchange search here.

[【】] has been seen in 243 true positives, 7 false positives, and 3 NAAs.

commented in chat:
I'm happy to approve this, but I'm not sure if it's doing everything you want. Both the watchlist and the keyword blacklist automatically prepend and append a \b to the regex. So, that watch ends up being \b[【】]\b. Thus, those characters will only be detected when they are preceded and followed by a word character (a character that would normally be in a word, but also including numbers and _).
Thus, in order to match, the text would need to be something like a【b, but having a space either before or after the [【】] characters will not match. On MS, the body and title searches which are reasonably equivalent to the watchlist for that regex are body and title, which detect 116TP/3FP/3NAA and 103TP/0FP/0NAA respectively.
However, what I think you probably intended was the matches which would be obtained by !!/watch \W*[【】]\W*. In this situation, the \W* on both ends allows the regex to consume up to the nearest \b, which will make the watch match any occurrence of [【】] in the title, body, or username. That regex is closely matched with the MS searches: body, title, which result in 243TP/7FP/3NAA and 201TP/1FP/0NAA respectively.
If it was me, I'd add the \W*[【】]\W* first. It may be too broad and produce too many FP, but will give us more data to work with. If it does produce a lot of FP, then we can adjust it down to see if we can eliminate the FP. In general, it's easier to tighten things up than it is to loosen them, because if we've watched something looser, then we have the data available to determine what the performance will be of something tighter through searches on MS. If we watch the narrower regex, then we will have no real idea what the performance of the looser regex will be.

Superseded by #3365.

Approved by Shree in Charcoal HQ


emulation regression test failing

@spencerkclark found that the fv3net tests don't pass with master of fv3gfs-fortran. In this PR #2047.
Context
This is the failing test: https://app.circleci.com/pipelines/github/ai2cm/fv3net/13296/workflows/51e0ccfc-8015-44dc-8c8d-5b7ff1982d2f/jobs/122021
After ungarbling the output, the traceback is
Loaded variable metadata from: /fv3net/external/emulation/microphysics_parameter_metadata.yaml
Initialized zarr monitor at: /tmp/tmpfk3n60rq/rundir/state_output.zarr
From cffi callback <function call_function at 0x7f4fe8ee5a60>:
Traceback (most recent call last):
  File "<init code for 'my_plugin'>", line 95, in call_function
  File "/fv3net/external/emulation/emulation/_emulate/microphysics.py", line 94, in microphysics
    predictions = self.model(inputs)
  File "/fv3net/external/emulation/emulation/models.py", line 56, in combined
    x_transformed.update(model(x_transformed))
  File "/fv3net/external/emulation/emulation/models.py", line 43, in __call__
    model_outputs = self.model.predict(inputs, batch_size=self._batch_size)
  File "/usr/local/lib/python3.8/dist-packages/keras/utils/traceback_utils.py", line 67, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/usr/local/lib/python3.8/dist-packages/keras/engine/data_adapter.py", line 1653, in _check_data_cardinality
    raise ValueError(msg)
ValueError: Data cardinality is ambiguous:
  x sizes: 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24, 1, 24, 24, 24, 24, 24, 24, 24, 24, 24, 24
Make sure all arrays contain the same number of samples.
STOP -1

@spencerkclark also said this:

After some debugging I tracked it down to this PR in the fortran model, which is not yet in our existing prognostic run image, but was inadvertently introduced by the fact that I bumped the fortran version.

Discussion
https://github.com/ai2cm/fv3gfs-fortran/pull/321 added a couple of new outputs in the state dictionary...the rank (a length 1 floating point array) and an area-dependent relative humidity threshold rhc (a 2D variable). rhc is probably not a problem since
there are other 2D variables like latitude and longitude being passed. My guess is that this is an issue when tensorflow tries batching the effectively scalar rank variable. A simple fix would be to filter out (or broadcast) such scalar values before passing them to model.predict.

Addressed by @nbren12 in #2062.


Use opencv-python-headless as setup prerequisite

I have a small suggestion to use opencv-python-headless rather than opencv-python as an install prerequisite, as I don't see the need for the non-headless features, namely GUI support. The opencv-python setup with pip is more cumbersome, since it also requires system-level installations for xcb: please see https://stackoverflow.com/questions/71088095/opencv-could-not-load-the-qt-platform-plugin-xcb-in-even-though-it-was-fou. The fix is simply to change 'opencv-python' -> 'opencv-python-headless' in https://github.com/RViMLab/oflibpytorch/blob/main/setup.py#L63C1-L65C7.

Hi, thank you for your suggestion (and sorry for the long delay). I understand your point very well, however I am using cv2.imshow() in a few places as an easy way of providing flow visualisation. As I understand it, this would no longer work with opencv-python-headless, correct?


store a reference to the acquire function on the CacheFileManager

Storing a reference to the function on the class tells the garbage collector to not collect the function before the class, such that any instance can safely complete its __del__.
No tests because I don't know how to properly test this. Any ideas?

[x] Closes #7814
[ ] Tests added
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

That's quite a weird bug. I would have thought that the global (or module level here) variable/function aquire should have at least one reference until after the deletion of the object.
Is that a bug in pythons garbage collection?
Or does the garbage collection already start when calling del and does not wait for the completion of the __del__ method?

I would have thought that the global (or module level here) variable/function aquire should have at least one reference until after the deletion of the object.

I think this is intended (though certainly not very easy to get right): see the second part of the warning in the __del__ documentation.

I think this is intended (though certainly not very easy to get right): see the second part of the warning in the __del__ documentation.

You are right, that warning is exactly what is causing the issues.

Maybe you can add a test that creates a cachingFilemanager object, then deletes it, then run gc.collect() and check somehow if it works? But no idea how pytest interferes with this or how you can ensure that there are no references to the module anymore?

the issue is that this doesn't occur on normal garbage collection but only on interpreter shutdown. So really, I don't think we have any way to test this using pytest as that itself is written in python (unless of course we can make use of sub-interpreters, but that might be more trouble than it's worth).

Have you verfied that this fixes things at least on your machine?

I thought I did, but apparently something changed: right now it fails because OPTIONS is not available anymore, so we might have to add a reference to that, as well. Additionally, for the warning we use warnings.warn and RuntimeWarning that might also have disappeared already (but those are standard library / builtins, so hopefully not?)

that appears to work on both my laptop and my local HPC, and is arguably a lot easier to implement / understand as we don't need to make sure all the globals we use are available


Feature Request: Default Values

Defaulting to fixed or dynamically calculated values would be a great addition.

This was in my roadmap, indeed.
I think the most flexible way of doing it is by passing them as a configuration option to the current openorm function.
So:
api.openForm('person', { values: { name: this.file.title }})
// this is an actual usecase of mine 😅

That aditional configuration option will also allow to set other options like fields to hide and fields to pick as stated on #39

This has been addressed in #53 for basic input types. I'm not sure yet about the more complex ones

Yeah, works fine. Everything related to dataview related fields seem to not yet work. So might need to think about something in this regards. Now I think one big quality of live improvement for me would be cross reference. E.g. if one field is a project, and another one is persons, to change the query based on the selected project. But that's probably going to be tough.

@danielo515: Should it work for Text Area? Seemed to fail for me for that one when passing in a string.

I probably overlooked it. Gonna have a look
On Sat, Oct 7, 2023 at 3:00 PM Tobias Steckenborn @.***>
wrote:

@danielo515 https://github.com/danielo515: Should it work for Text
Area? Seemed to fail for me for that one when passing in a string.
—
Reply to this email directly, view it on GitHub
https://github.com/danielo515/obsidian-modal-form/issues/48#issuecomment-1751707009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARKJWKEWCAIBF24PQR2GSDX6FHANAVCNFSM6AAAAAA5F2OEGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJRG4YDOMBQHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
https://danielorodriguez.com

By the way are you planning to release officially? Got quite some cool uses out of it and would like to refer to it more easily (even though text areas still seem bugged 😅)

What do you mean to release officially? On obsidian plugin store? I'm
waiting for approval
El vie, 13 oct 2023, 19:08, Tobias Steckenborn @.***>
escribió:

By the way are you planning to release officially? Got quite some cool
uses out of it and would like to refer to it more easily (even though text
areas still seem bugged 😅)
—
Reply to this email directly, view it on GitHub
https://github.com/danielo515/obsidian-modal-form/issues/48#issuecomment-1761836578,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARKJWOQO45MRIKGHFYBEWDX7FYRHAVCNFSM6AAAAAA5F2OEGOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRRHAZTMNJXHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update install.sh, deletes miniconda installer if interrupted

Potential errors could be avoided and resources can be saved if the miniconda installer is deleted before downloading it again.
This is in the case the script is interrupted before the download is completed.
I thought that it would be better if it instead checked if it was already there, just so it doesn't have to download it again, but that would cause issues in case the existing file is incomplete or corrupted

@pato-pan hi could you review the script again? currently there is a branch conflicts since major updates has been made these months.
currently the installer will get cleaned up after it has been installed: https://github.com/SillyTavern/SillyTavern-Launcher/blob/eb2721615618e06fca0014fb3deb100f3b2903ab/install.sh#L119
do you want me to add the cleanup above: https://github.com/SillyTavern/SillyTavern-Launcher/blob/eb2721615618e06fca0014fb3deb100f3b2903ab/install.sh#L107 aswell?

@pato-pan hi could you review the script again? currently there is a branch conflicts since major updates has been made these months.
currently the installer will get cleaned up after it has been installed:
https://github.com/SillyTavern/SillyTavern-Launcher/blob/eb2721615618e06fca0014fb3deb100f3b2903ab/install.sh#L119
do you want me to add the cleanup above:
https://github.com/SillyTavern/SillyTavern-Launcher/blob/eb2721615618e06fca0014fb3deb100f3b2903ab/install.sh#L107
aswell?

yes, it should be run before making the download unless you find a way to resume the download, fix it, reuse it if completed, or force it to overwrite the existing file.


doctor instructions for libimobiledevice install incorrect

Steps to Reproduce
First issue

Uninstall libimobiledevice if installed
Run flutter doctor. Instructions say to run brew install libimobiledevice.
Run brew install libimobiledevice
Run flutter doctor. Instructions say the version is incorrect and to uninstall and re-install with --HEAD.

Second issue
Starting from the state at the end of the first issue steps:

Instructions say to brew uninstall libimobiledevice. Running this results in:

Error: Refusing to uninstall /Users/cbracken/.homebrew/Cellar/libimobiledevice/HEAD-1fa4170
because it is required by ideviceinstaller, which is currently installed.
You can override this and force removal with:
  brew uninstall --ignore-dependencies libimobiledevice

/cc @mit-mit


Remove the formatting of AST, it creates a dependency on Fantomas that's not really needed for what's really a one liner

I removed the usage of the format method earlier in Myriad as it was causing issues with a mismatched Fantomas dependency.  It's also probably a good idea to just keep this lib for helping create art elements and keep Fantomas doing the formatting.  I can PR the removal if it's agreeable.

Done in c0517bebf26732ae9954b3f410c9d59e2bfe4668


Plan for Corepack Removal

https://github.com/nodejs/package-maintenance/pull/606
At the moment we use it to manage our pnpm version.
https://github.com/seek-oss/skuba/blob/138129daab25e612d650b2e79b69fa5386035149/template/lambda-sqs-worker/Dockerfile#L6
https://github.com/seek-oss/skuba/blob/138129daab25e612d650b2e79b69fa5386035149/template/lambda-sqs-worker/package.json#L41
We probably need to write an autofix of sorts but I'm not entirely sure what the best approach would be right now:
npm i -g corepack && corepack enable?
That would leave us with installing an unpinned corepack version 🤔.
Just leaving a thread here so we aren't surprised when it pops up in the next major node version

Would it make sense just to install corepack as a dependency in projects?

Might be able to go with something like npm i -g corepack@{version} and have renovate upgrade it?


tutorials-08-14 still at level 140 and using inverse

I reported this about tutorial-08 a couple months ago (Reference: https://github.com/tomaka/glium/issues/1432).  Shaders are flagging #version 140 but inverse requires #version 150.
If glium has been deprecated by vulkano (I note that commits have dropped, last commit is a month ago, reported bugs are not getting fixed), please do at least let the community know that you have abandoned glium and that we should not use it anymore.
Thanks.

If glium has been deprecated by vulkano (I note that commits have dropped, last commit is a month ago, reported bugs are not getting fixed), please do at least let the community know that you have abandoned glium and that we should not use it anymore.

Well, he did. So you should've at least checked before making your accusations...
https://users.rust-lang.org/t/glium-post-mortem/7063

It's not really deprecated, it just needs a big rework which I don't have time for.

I did search for both "glium deprecated" and "glium abandoned" on Google.  That post doesn't even pop up on Google unless I search very specifically for "glium post-mortem".  And even THAT doesn't pop up on other search engines (presumably the post is too new as of this date).
In addition, I would expect such important information as is contained in https://users.rust-lang.org/t/glium-post-mortem/7063 to be contained in the README.md where everybody can find it rather than having to very specifically craft a search to find it.
Finally, just because the owner has decreased his contributions does not automatically mean something should not be used.  The library could be in the middle of a handoff or it could be "complete".  The fact that neither are the case should probably be marked.
I'm glad tomaka did this work, and I'm sure the things he learned will make the work he does on Volkano much stronger.  I fully appreciate the demands a library like this makes on your time as I have been in his position for a much smaller, less visible group of people.
However, partially-complete, zombie libraries are a social problem in that they will socially drown out something better (ask the Lisp guys about this).  Something like glium that got a lot of coverage and sorta works will be a particular obstacle.
Set a date to put a knife through it (I would suggest about 3 months in the future--long enough to give people a chance to react but not so long that they consider it too low priority) and follow through.  This also has the advantage of causing someone reliant on the library to finally step up to take it over.  And, if nobody does step up, well, that's a statement, too.
Thanks for the work.  I'll be heading to go get a Vulkan/Vulkano system to set up.
Bug closed.


Provide currency symbols in Intl number data

There are currently no currency symbols included in the number formatting data.

This issue has been moved to dart-lang/intl#59.


AVAudioPCMBuffer.append not working as expected

AVAudioPCMBuffer.append does not seem to be working correctly, or at least not how I would expect it to, and I cannot find it being used anywhere in examples or other code paths
To Reproduce
Take the following code for example, in which I am just trying to use the extract method, to extract a copy of a slice of the original buffer, and append it to the original buffer, which I would expect would just put it on the end of the original buffer.
let fileURL = "somepath.wav"
let file = try AVAudioFile(forReading: fileUrlFor(path: fileURL))
let sampleRate = file.sampleRateForFile
let audioFormat = AVAudioFormat(standardFormatWithSampleRate: sampleRate, channels: 2)
let loopBuffer = AVAudioPCMBuffer(pcmFormat: audioFormat!, frameCapacity: AVAudioFrameCount(file.length))
try file.read(into: loopBuffer!)
let startFrame = AVAudioFramePosition(loopBuffer!.frameLength)
loopBuffer!.append(loopBuffer.extract(from: 0.0, to: 2.0)!, startingFrame: startFrame, frameCount: AVAudioFrameCount(file.length))

Always results in the "Insufficient audio in buffer" precondition being hit. - Using the append method without any parameters (just passing in the extracted buffer) appears to do the same thing.
Expected behavior
I am expecting the original buffer to have the extracted buffer appended to it, and the length of the original buffer to be the sum of the original buffer + the extracted buffer
Details (please complete the following information):
-Type: Mac

OS: 10.15.7
Audio Kit V5, rev b1099859f6654c98f0f28e9fb1ed388ae96cb7d6

Could you PR a failing test? That might be the best way to give us a working (nonworking) example.

Most definitely, https://github.com/AudioKit/AudioKit/pull/2585

I'm inclined to accept the now passing test and close this issue, but I do not know how or why this is not failing anymore. Possibly it was fixed in an OS update?


switch to standard metrics in front-end has broken unified dashboards

In microservices-demo/front-end#99 we switched to what is considered to be a more standard metric(*) for reporting request counts and latencies - http_request_duration_seconds with a path. Unfortunately that has broken some unified dashboards (e.g. RED metric notebooks) that report on all the sock-shop workloads, since all other workloads are still emitting request_duration_seconds metrics with a route, and that is what the RED metric notebooks are using.
We either need to change all the other metrics, or find a way of creating the unified RED metric notebooks from multiple metrics.
(*) @jpellizzari what makes this new metric more standard than the previous one?

I reckon we should follow Damien's suggestion, i.e.  support request_duration_seconds + route in the Weave Cloud dashboards and revert the change to the front-end metrics.

The correct solution of course would be to change the metrics in all the other workloads to http_request_duration_seconds.

This would be great to have fixed/deployed before KubeCon for demo purposes.

@dholbach I plan on getting this done before the end of the week

<3

Thanks @jpellizzari!


How not to cut off zeros when rounding

Hi,
I want the result to display exactly as many decimal places as I want when using the ROUND function to 4 decimal places. However, when the final numbers are zeros, they are truncated. It bothers me in data validation. Could someone please help me to solve this problem?

@mlodybanach You can disable data formatting so it will be shown as it is read from a database


The npm package for 2.0.0 contains the "coverage/" dir

So it looks like you may have accidentally published the code coverage HTML with the 2.0.0 package. Can you publish a new package without this directory (and probably add it to your .npmignore ;) )?

Will certainly remove it. Not sure I want to go back and republish an existing build though. 3.x should be landing soon.

That's fine :) Getting it in whatever is next is good enough, even if that's v3 :)

Released in 3.0.0


Create a task to provide the capability to query db instances

Is your request related to a problem? Please describe.
The business needs an easy way to access data from a mongodb instance in order to perform additional analysis.
Describe the solution you'd like
A workflow task capable of connecting to a mongodb instance and run a query.
The output of the query must be accessible as a file at the end of the task execution.
Describe the benefits or justification for this request
Increases the capability of the platform and opens the door to access the data without any prior transformation, allowing the transformation to be done post retrieval, in the flow.
Additional context
NA

Hey @morarucostel thanks for the issue. Its not currently tagged to a release but do see it included in a loader change
https://github.com/boomerang-io/flow.loader/pull/67/files#diff-c85a304c1ca5ffe621b41e6cde3f9667226cc680991cc29115518fe5edb393e5
Should this have been reviewed before being included in the same loader change as the HTTP Task Template changes?

cc: @morarucostel @amhudson

Hey @tlawrie , yes, it should have been reviewed before being included. That is my fault, I jumped the gun there.
We can either revert it or review it part of this issue.
The task is using the mongosh image from mongodb and the UI is configured like this:

Hi @morarucostel this is a good addition. Its great new tasks are being generated. Not a problem, we should probably devise a process for new tasks and a +1 or +2. We havent really ever had a process for it.
W.r.t to this task, few comments

If it's using an external image, we might need to document that. Im thinking we either add a list of dependent images in the architecture documentation OR add an extra column to the tasks. I think it might be better in the architecture, to add images that we depend on for tasks.
How does this task work for TLS required MongoDBs?
Some more helper text or placeholders might be good. In the query it could have an

e.g. example query

@tlawrie points 2 and 3 are solved (attached as images below)
Regarding point 1, documenting the dependent images, do you think we can stick them in 3.6.1 release docs?

@tlawrie also exposed, as configurable, the auth method so now the task can be used to connect to both internal in k8s hosted mongodb instance as well as external Azure hosted instance.
The current configuration is:

@tlawrie also updated the Architecture - Application - Components documentation with the dependency on the MongoDB shell docker image. It made sense to add the information under the already existing external dependency column of the TaskWorker.

Is it OK to map the task against 3.6.1 version?
The task template and documentation are already done.


Move swagger conversion logic to misc

Purpose
This PR do not have any new implementation. We will move ballerina to swagger conversion logic to outside composer and make it reusable. Added test case to base component to verify flow.
Goals
Move ballerina swagger conversion to central place.
Automation tests

Unit tests
Added unit tests.

Test environment

java 1.8.0_144

Change length as suggested.


Give guidance in tracker documentation for unclear symptoms

The current recommendation is to report your risk to others using "Your risk to other people you see outside of the house/pod (decaying over the last 23 days) NOTE: This assumes that you and the rest of your pod have no symptoms currently AND pod will report symptoms to you"
However: A friend of mine has seasonal allergies. My model of her, is that she does not want to be required to attest "I definitely don't have any even-mild even-less-common phantom symptoms" as part of her uCoV score to others. I think she might be uneasy about the symptom-free decay, and want a way to report her own risk more conservatively than that.
IMHO we should provide the following guidance:
“If you have not done anything unusually risky in the past three weeks, and you think you probably don’t have symptoms, but you have a hint of something mild, less common for COVID, and easily explained otherwise (such as a phantom mild headache, or unclear sore throat) and you want to report your risk conservatively just to be on the safe side, then you can report the greater of your “risk to other people” and “risk to pod”, because the latter does not use the symptom-free decay.”
“Note, of course, that if you think you might in fact have symptoms that are more-common for COVID, and they are not easily explained otherwise, then you should say home, notify your contacts, and get tested if your symptoms persist more than 24 hours."

@catherio where do you imagine that being most useful?
I'm thinking: 1) Tracker FAQ and 2) a hover note on the cell itself on the Pod Overview.


Fixed some warnings, appearing during clean install

Fixed warnings in Skeleton test appearing during clean install: "the keyword return in this context means child::return"

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 18min)


Workaround discord rate limit issues

Adds workaround for recurring Discord rate limit issue. This issue seems to occur when /lfg, join, and leave commands take longer than expected. As such, always defer /lfg, join, and leave. This should allow more flexibility. However, this has the following knock-on consequences:

Can't use hidden messages in /lfg.
Many errors are now reported by sending a message to the user instead of to the channel.
Until it can be fixed later, error reporting on role and DM failures is disabled.

Codecov Report

Merging #734 (f5d577c) into main (be742e7) will decrease coverage by 0.25%.
The diff coverage is 80.55%.

@@             Coverage Diff             @@
##              main     #734      +/-   ##
===========================================
- Coverage   100.00%   99.74%   -0.26%
===========================================
  Files           71       71
  Lines         3180     3185       +5
  Branches       314      314
===========================================
- Hits          3180     3177       -3
- Misses           0        4       +4
- Partials         0        4       +4

Impacted Files
Coverage Δ

src/spellbot/interactions/lfg_interaction.py
96.89% <56.25%> (-3.11%)
:arrow_down:

src/spellbot/client.py
100.00% <100.00%> (ø)

src/spellbot/cogs/events_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/leave_cog.py
100.00% <100.00%> (ø)

src/spellbot/cogs/lfg_cog.py
100.00% <100.00%> (ø)

src/spellbot/database.py
100.00% <100.00%> (ø)

src/spellbot/interactions/leave_interaction.py
100.00% <100.00%> (ø)

src/spellbot/operations.py
100.00% <100.00%> (ø)

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 be742e7...f5d577c. Read the comment docs.


LAR Circle and MSA Clipping Selection and Async Call Logic

Handles selection of either all LAR, MSA-only LAR, or Selected MSA LAR
Sets URL parameter for LAR type selection (geo_query_type)
Tracks MSAs in bounds to prevent data call if not necessary when scrolling
Uses _.difference to check if MSA changes are not within array set so users can start from a super-set and zoom in without load
Added event handler for drop-down on MSA Details tab
Added drop-down on MSA Details tab
Split Map.js into multiple JS files for ease of use / readability

:+1:


Add onSuccess handler to transaction component

What changed? Why?

add optional onSuccess handler to expose transaction hash
update docs

Notes to reviewers
How has it been tested?

Do we still need this PR, or can we close it?

I am going to close this, as it seems outdated.


help with Fixed Price (Weekly)

I am encountering an issue with TeslaMateAgile using the FixedPriceWeekly provider.
The service fails to start and logs the following error:

fail:` TeslaMateAgile.PriceService[0]
Failed to update prices
System.ArgumentException: Invalid fixed price data, covers more than 24 hours

my compose file:

teslamateagile:
image: mattjeanes/teslamateagile:latest
restart: always
environment:
- DATABASE_USER={Myuser}
- DATABASE_PASS={Mypassword}
- DATABASE_NAME={mydatabase}
- DATABASE_HOST=database
- TeslaMate__UpdateIntervalSeconds=3600
- TeslaMate__GeofenceId=2
- TeslaMate__EnergyProvider=FixedPriceWeekly
- TeslaMate__LookbackDays=365 # Only calculate charges started in the last x days (default: null, all charges)
- FixedPriceWeekly__TimeZone=Europe/Zurich # IANA (tz database) time zone code, used for below times
- FixedPriceWeekly__Prices__1=Mon-Fri=22:00-07:00=0.25
- FixedPriceWeekly__Prices__2=Mon-Fri=07:00-22:00=0.36
- FixedPriceWeekly__Prices__5=Sat,Sun=22:00-17:00=0.25
- FixedPriceWeekly__Prices__6=Sat,Sun=17:00-22:00=0.36
- FixedPriceWeekly__Prices__7=Sun,Mon=22:00-07:00=0.25

I am trying to configure the rates provided by my energy supplier:
Weekdays:
Off-peak hours: 00:00 to 07:00 / 22:00 to 00:00
Peak hours: 07:00 to 22:00
Weekend:
Off-peak hours: 00:00 to 17:00 / 22:00 to 24:00
Peak hours: 17:00 to 22:00
What I am specifically trying to ensure is:

From Friday at 22:00 to Saturday at 17:00, the rate is considered off-peak.

From Sunday at 22:00 to Monday at 07:00, the rate is also considered off-peak.

Thank you for your help on this.

@darkh89 you managed to get it all sorted? What was the issue in the end?

Hi @MattJeanes,
I've changed my configuration file to handle this:

Monday to Friday
  - FixedPriceWeekly__Prices__0=Mon-Fri=00:00-07:00=0.25
  - FixedPriceWeekly__Prices__1=Mon-Fri=07:00-22:00=0.36
  - FixedPriceWeekly__Prices__2=Mon-Fri=22:00-00:00=0.25

Saturday
  - FixedPriceWeekly__Prices__3=Sat=00:00-17:00=0.25
  - FixedPriceWeekly__Prices__4=Sat=17:00-22:00=0.36
  - FixedPriceWeekly__Prices__5=Sat=22:00-00:00=0.25

Sunday to Monday
  - FixedPriceWeekly__Prices__6=Sun=00:00-17:00=0.25
  - FixedPriceWeekly__Prices__7=Sun=17:00-22:00=0.36
  - FixedPriceWeekly__Prices__8=Sun=22:00-00:00=0.25

Ah okay makes sense, good to hear you got it sorted!


Mechanism to alter log levels

Hi @luhahn
Can you provide a mechanism to influence the root log level of Glowrtoot Central after deployment via Helm?
I need to set it to DEBUG briefly.
See: https://github.com/glowroot/glowroot/issues/942
Thanks, Robin.

Hey, the way described in your Issue would require it to be set during the build. Maybe there is a way to set it externally.
If you could get that information I'd be happy to include it in the chart. However I won't be able to ask/do anything today. Best I can do is friday or monday.

Nevermind, just read it again. I can copy that file into the jar directory and it should work. I will test it :)


Vscode ide support

Anyone know ?
I am finding vscode excellent

We have a Visual Studio 2013 plugin, but nothing on VSCode AFAIK.

+1 :)

+1 here as well. Either that or a plugin for VS2017

+1

FYI, we have started work on vscode plugin. You may see the progress and give feedback here.
I'll mark this issue as closed, vscode specific issues can be logged here.


drop pg 9.6

PostgreSQL has been removed from omero-install and ansible role

This table captures the state of the distribution by the various Linux distros right? While PSQL 9.6 has been removed from the RHEL/CentOS repos, I have not checked whether the same holds for Debian/Ubuntu.
Otherwise, looks good to me

you could still get it for Debian 11 for example e.g. https://apt.postgresql.org/pub/repos/apt/dists/bullseye-pgdg/9.6/
I opted to remove to avoid confusion since this is not really a version we recommend

Noted, I am fine with removing it completely as per the unrecommended argument. Is there a corresponding support level we can use in the table above to express the status of PSQL 9.6 on OMERO 5.6? Or should it be kept as deprecated?

I think keeping it as deprecated is fine.


[Unity][nn.module] Rolling KVCache override method

Part of the effort in bringing Mistral to the compilation flow with nn.Module on mlc-llm. Adding support for Rolling KVCache override method in the nn.module frontend.
cc @CharlieFRuan

cc @junrushao @cyx-6

I think we should move all KVCache logic to MLC LLM instead, given we expect plenty of variants of KVCache implementation. @tqchen @davidpissarra what do you guys think?

Yes, i think moving to mlc is OK, we can move back if there is a point of stablization.

I think we should move all KVCache logic to MLC LLM instead, given we expect plenty of variants of KVCache implementation. @tqchen @davidpissarra what do you guys think?

Thanks @junrushao! Makes sense to move KVCache to MLC LLM


Google cloud functions?

Can I use the lambda package and run it on google cloud? I'm looking into using google cloud functions with the project. Sorry I didn't investigate first. Just wanted to see if anyone else is interested as well!

The AWS Lambda integration is designed to export a function for API Gateway proxy integration not Google Cloud. You would be better off using the graphql-server-core package and converting the request object into one that can be read by that package.
Is google cloud functions still in beta? It might be too early to write an integration for it.

Thanks for the info. Yes it is still in beta

I'd also be happy to accept a PR that adds a separate package for google cloud functions, just like we have one for lambda. :wink:

Is there anywhere I should look to better understand the internals? Or just look over the current code to make my own :)

Yeah, I would just look at what the lambda or micro package does and then roll your own! It's pretty straightforward.

@zackify the http logic itself is pretty abstract in graphql-core
you can take a quick look at the integrations themselves and see that they are just "adapters".
should be pretty easy to add support..

I have also been looking into hosting GraphQL on GCP Cloud Functions / Cloud Functions for Firebase. I've come across a number of resources that have confused me with the use of ExpressJS in the context of Cloud Functions:
Here I see people passing an ExpressJS server into the Firebase Cloud Functions functions.https.onRequest with a Firebase-Samples example using the same method here.
But then I came across a single person here who has concerns about creating an express app in a Cloud Function.
So I'm not sure which method here is correct in the context of ExpressJS in Cloud Functions for Firebase. I've posted on SO with the following questions:

How does Cloud Functions for Firebase or GCP Cloud Functions handle the lifetime of objects initialised outside of the function definition?
How does the above effect the lifetime of the function? Does it run until timeout or function similarly to AWS Lambda?
The functions.https.onRequest documentation doesn't specify you can pass an ExpressJS server object into it, so how is this working? Are their then two endpoints? Can someone explain what is happening here?

TLDR; if you can use ExpressJS server as described here in Google/Firebase Cloud Functions could we then not just rely on graphql-server-express?

It does seem like the google cloud function takes in an express Request/Response object.
https://cloud.google.com/functions/docs/writing/http
It also seems like the firebase google cloud function has several triggers. The http trigger is created using this function functions.https.onRequest. Which takes in the same express Request and Response objects.
https://firebase.google.com/docs/functions/http-events
Does seem like the implementation of graphql-server-express might work without modification.
Not sure how to answer the other questions about function lifetime still learning about GCP.

I can confirm that the graphql-server-express is easily setup in Cloud Functions. It's more stable than the express-graphql implementation as it doesn't appear to error on the absent trailing '/'.
Sample code:

// graphql-server-express
const bodyParser = require("body-parser");
const graphqlExpress = require("graphql-server-express").graphqlExpress;
const mySchema = ...;
var graphQLServer = express();
graphQLServer.use("*", bodyParser.json(), graphqlExpress({ schema: mySchema }));

// CF for Firebase with graphql-server-express
exports.graphql = functions.https.onRequest(graphQLServer);

called using https://us-central1-<project-name>.cloudfunctions.net/graphql.
however to use GraphiQL as well you must do the following:
// import graphiql
const graphiqlExpress = require("graphql-server-express").graphiqlExpress;

// assign specific URLs
graphQLServer.use("/graphql", bodyParser.json(), graphqlExpress({ schema: mySchema }));
graphQLServer.use("/graphiql", graphiqlExpress({ endpointURL: "/graphql" }));

and then call the method with https://us-central1-<project-name>.cloudfunctions.net/graphql/graphql and https://us-central1-<project-name>.cloudfunctions.net/graphql/graphiql.
I still do not have an answer for how definitions of objects outside the function definition are handled or how Cloud Functions can take an entire ExpressJS object and just magically work seemingly stripping the object back to it's request, response parts. I've asked a detailed question on SO

Also, how would you do this with subscriptions? Is that not possible I'm guessing?

This graphql-subscription issue is tracking that. I'm looking into managed PubSub services, particularly Google Cloud PubSub at the moment.

@jthegedus just to close the loop, would you mind sending a PR to the README and docs here adding the code snippet about google cloud functions?

@stubailo It would be my pleasure :smile:

@stubailo  @jthegedus  did an official best practice example ever make the docs? I presume not as I can't seem to find it in in the server docs
Are their any plans to add one?
I have used the the the example walk through in this article and it worked well. Could I suggest reviewing it and maybe adding it to the docs ?
let me know if there is any way I could help with that.  😄

@Paddy-Hamilton I did record this early Jan in my backlog of things to do. I should have time this weekend to address this. I'll ping you in the PR if you wish to give feedback. (that example you linked does actually reference my article for doing this on Firebase, I just never got around to vanilla Google Cloud Functions)

@jthegedus ha, so it does, thanks for the help 👍
Sounds good, will take a look at the PR, although i'm a relative graphql and GCF newbie so my feedback maybe limited. I'll help in whatever way I can.
Cheers 🍻


Access-Control-Allow-Credentials not set in response

According to https://docs.arangodb.com/GeneralHttp/index.html

any access-control-allow-credentials header sent by the client is ignored by ArangoDB its value is not true. If a client sends a header value of true, ArangoDB will return the header access-control-allow-credentials: true, too.

It seems like this isn't working properly, even if this header is set by the client in a CORS request, the response header will be set to false.
See this for context https://github.com/arangodb/arangojs/pull/249 and an additional problem; even when forcing the header to be true, no Authorization header is sent so the response will be 401. Also, adding an Authorization header alleviates the need for the Access-Control-Allow-Credentials header and xhr.withCredentials.

Even with --http.trusted-origin http://localhost:3000, the preflight header is false:

And when I use Requestly to rewrite the preflight header to true, that response returns true from server. Requestly doesn't modify the chrome debug output, so it's confirmed that the server is setting the header correctly after the preflight:

Yes, on preflight the Access-Control-Allow-Credentials is false because the request does not contain the header Access-Control-Allow-Credentials: true. Currently this header must be set in all requests in addition to a trusted Origin value. Otherwise the server will respond with Access-Control-Allow-Credentials: false.

I'm a bit confused what header this options is affecting? According to https://docs.arangodb.com/3.0/Manual/ReleaseNotes/UpgradingChanges30.html#handling-of-cors-requests it should set the Access-Control-Allow-Credentials to true, but at the end of the section it also states that it will set Access-Control-Allow-Origin. Which one is it?

I set the Access-Control-Allow-Credentials to true on the above request(s), but for reasons unknown to me, it's not included in the preflight. You see on the second request that it's supposed to be included.

With 3.0.0-rc2, without any http.trusted-origin set, an CORS request returns the origin header, like:
Access-Control-Allow-Origin: http://localhost:3000
Here's a successful request, without the http.trusted-origin:

The docs are slightly wrong in this case.
It works like this:
If the request contains an Access-Control-Allow-Credentials header with value true and there is a non-empty Origin header that matches one of the trusted origins specified via --server.trusted-origin, then the server will respond with Access-Control-Allow-Credentials: true. Otherwise it will return Access-Control-Allow-Credentials: false.
The Access-Control-Allow-Origin response header will be always set to the same Origin value used in the request.

Ok, then that works as I expected. The only issue I have then is the missing Access-Control-Allow-Credentials: true header on the preflight, even if http.trusted-origin is set to the same as the origin (as in the first two screenshots).

mmmh, looks like the browser doesn't set it in its OPTIONS request. Is it safe to always set this header on the server-side if the browser doesn't send it but if the Origin value is in the trusted list? Probably.

I think the Allow-Credentials only refers to cookies and basic auth credentials already known to the browser (i.e. entered from a browser popup or navigated to via a URL string). However the request header should be set when the XHR is created "withCredentials". I'm not sure whether the browser still silently discards it when it has no cookies or auth to offer (explicit auth headers provided in the XHR by the developer seem to be unrelated to this).

Ok, since the conclusion of the above discussion is, that this is not an issue, I will close this now.


Sonarr Post Processing not working

Sonarr downloads the files just fine and using manual.py works great.
The pre-move location they are in is "M:\Downloads\Complete\sonarr"
autoProcess.ini settings for Sonarr look like this.

I have disabled "Completed Download Handling" and set the "Drone Factory Options" to 0
I have set Connections to Custom Script and the settings look like this.

The Argument is set to this: C:\Users\Host\Desktop\sickbeard_mp4_automator-master\postSonarr.py
I have downloaded multiple episodes with Sonarr, Sonarr is not processing the files, and neither is the script.
Any help?

This works fine for me. Make sure you have the same category in sonarr and sabnzbd. The category is listed in the download client in sonarr.

Gonna have to clarify exactly which part isn't working for further help.
Everything looks like its set up appropriately as far as sonar is concerned. Is Sonarr not renaming the files and moving them either? If that's the case its an issue with Sonarr not having the file passed back appropriately from the downloader
What downloading client are you using?

Apologies, I was falling asleep at 0100 writing this out.
I'm using SABnzbd and Deluge for a torrent backup.
When the download completes in SABnzdb nothing happens. I don't see any temp files being created for the conversion.
The only thing I think I may be missing, does the name of the Custom Script have to match the label that is being applied to the downloads?

That's normal, its not supposed to remove the SAB history


Cannot read property 'nextJobID' of undefined

First of all, thanks for the great library and an amazing react-universally starter kit!
I'm in the process of migrating of my project, based on react-universally@12 to react-universally@next(v13). I'm following and implementing changes, according PR here: https://github.com/ctrlplusb/react-universally/pull/366.
There is an Error thrown when during SSR, when require and use withJob():
// MyComponent.js
import { withJob } from 'react-jobs/ssr';
// ...

export default withJob(/* ...my job */)(MyComponent);

// reacrApplication/index.js
  const app = (
    <Provider store={store}>
      <StaticRouter location={request.url} context={reactRouterContext}>
        <App />
      </StaticRouter>
    </Provider>
  );

  withAsyncComponents(app)
    .then(asyncComponents =>
      runJobs(asyncComponents.appWithAsyncComponents)
        .then(withJobs => ({ asyncComponents, withJobs })),
    )
    .then(({ asyncComponents, withJobs }) => {
      const html = renderToStaticMarkup(
        <ServerHTML
          reactAppString={renderToString(withJobs.appWithJobs)}
// ...

TypeError: Cannot read property 'nextJobID' of undefined
    at getJobID (/myApp/node_modules/react-jobs/commonjs/ssr/withJob.js:75:38)
    at ComponentWithJobID (/myApp/node_modules/react-jobs/commonjs/ssr/withJob.js:79:17)
    at reactTreeWalker (/myApp/node_modules/react-tree-walker/commonjs/index.js:84:15)
    at reactTreeWalker (/myApp/node_modules/react-tree-walker/commonjs/index.js:89:7)
    at reactTreeWalker (/myApp/node_modules/react-tree-walker/commonjs/index.js:89:7)
    at /myApp/node_modules/react-tree-walker/commonjs/index.js:104:11
    at forEachSingleChild (/myApp/node_modules/react/lib/ReactChildren.js:51:8)
    at traverseAllChildrenImpl (/myApp/node_modules/react/lib/traverseAllChildren.js:77:5)
    at traverseAllChildrenImpl (/myApp/node_modules/react/lib/traverseAllChildren.js:93:23)
    at traverseAllChildren (/myApp/node_modules/react/lib/traverseAllChildren.js:172:10)

https://github.com/ctrlplusb/react-jobs/blob/master/src/ssr/withJob.js#L60
This error happens server-side, and my investigations hints me, that, maybe, context does not get injected for some reason.
I decided to fill an issue here, in this project, because, even if I'm doing something wrong, in my opinion, we should handle and catch this exception somehow inside react-jobs.

I've managed to avoid this error by disabling async-components. But I'm sure, proper error handling would be a great addition to this case.

I have a solution coming soon. I had planned for it to be released the beginning of this week, but got swept away with life stuff. Will keep you all posted.

I'm also interested in a solution to this. So far w/ async component is great for server side, I still want to fetch and pre-render data with React Router v4 and have been at a loss as to how to do it without static routes (which I think defeats the purpose of the new router API)

can use ctrlplusb's
react-async-bootstrapper(https://github.com/ctrlplusb/react-async-bootstrapper)
asyncBootstrapperTarget magic function :)
class Post extends React.Component {
  asyncBootstrapperTarget(_id) {
    const { match, fetchPost } = this.props;
    const id = _id || match.params.id;

    if (!id) {
      return Promise.reslove();
    }
    // Execute the redux-thunk powered action that returns a Promise and
    // fetches the post.
    return fetchPost(id);
  }

  componentDidMount() {
    if (!this.props.post) {
      this.asyncBootstrapperTarget(this.props.match.params.id);
    }
  }

  componentWillReceiveProps(nextProps) {
    if (this.props.match.params.id !== nextProps.match.params.id) {
      this.asyncBootstrapperTarget(nextProps.match.params.id);
    }
  }

Hey all!
I have rewritten react-jobs.  It now has complete interop with react-async-component.
See a demo here: https://react-universally-jobs.now.sh
It's on a branch within react-universally.
🎉

@ctrlplusb could you double check that branch link please? i don't see a next/react-jobs branch

Sorry, I renamed the branch.

@ctrlplusb sorry to bug you again, but i don't see any reference to react-jobs in that branch. I almost got it working, but I don't get anything on the server even though the work function runs, so would be great to see your example.

Doh! I did something really really silly!
Coming soon!

Up! :)

awesome thanks @ctrlplusb everything works. Love your work on these tough problems!


update go-openai to v1.36.0

Description
We will update go-openai to support new models such as o1-preview and o1-mini.
Fixes #223
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] go test ./...
[x] go run ./examples/chat/simple/main.go

Checklist:

[x] I have read the Contributing documentation.
[x] I have read the Code of conduct documentation.
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

Policy on Unattended Pull Requests
As part of our commitment to maintaining an active and collaborative development environment, it's important for pull requests (PRs) to receive timely attention and feedback. PRs left unattended for an extended period can impede progress and create bottlenecks in the development process.
Therefore, if a PR remains unapproved and unmodified following a review for an extended period, the author of the PR reserves the right to implement the suggested modifications at their discretion.
The specific duration of time before this policy is enacted may vary depending on the circumstances and urgency of the changes. However, as a general guideline, a reasonable timeframe for review and action might be 2 weeks.
This policy aims to ensure the continued progress of the project while respecting the contributions and efforts of all team members. It encourages timely collaboration and feedback while preventing PRs from stagnating indefinitely.
If you anticipate being unable to review or address a PR within the specified timeframe, please communicate this with the team to avoid invoking this policy unnecessarily.

Thank you for the wonderful software.
Since this is not a very essential fix, I will close this PR.


rubico/x/floating-point-average

I'd like a modular TypeScript version of this existing JS example.
export should be floatingPointAverage
please also include the stackoverflow link commented at the bottom
TODO

[ ] - documentation entry
[ ] - implementation + tests

guidelines

there are better examples


How would I add OSM data to this Docker Image

I seem not to be able to get to a command prompt.  Only access to psql.  So how would I run Osm2pgsql to import OSM datafiles into the postgres database.

Hi @dirklieske and thank you for your interest in the pgRouting Docker Image.
Simply explained you first need to start the pgrouting database container, and then you can configure osm2pgsql to connect to the database like this:
osm2pgsql \
  --host <IP address for DB container> \
  --port <PORT for DB container> \
  --database postgres


GRCh38 instance does not return ACACA canonical transcript

http://grch38.genomenexus.org/ensembl/canonical-transcript/hgnc/ACACA?isoformOverrideSource=mskcc
This call does not return any value, but the export files do have ACACA transcripts
https://github.com/genome-nexus/genome-nexus-importer/blob/master/data/grch37_ensembl92/export/ensembl_biomart_canonical_transcripts_per_hgnc.txt

Also these genes, they all have grch37 transcripts:
HIST1H2BB
HIST1H2BH
POTEF
PRLR
PSMB3
SDR39U1
ZNF658


Spelling sema

This fixes some misspellings in Swift documentation, it is split per https://github.com/apple/swift/pull/42421#issuecomment-1101092698

There's a bit of interaction between Sema and AST, so I've pulled a couple of changes out of the AST PR and into here.

@swift-ci please test

I wish CI could some day do these static checks for us.

Which "static checks"?
The tooling I'm using is available as a GitHub workflow 🙂.

Well, Jenkins can definitely respond to PRs and jobs/pipelines.
Can't tell if there's a policy your system has to reduce risk without first having some human decide a PR isn't malicious/spammy. (GitHub has recently added more bits to their environment because of such problems.)

Thank you, these changes all look reasonable to me. @CodaFi do you want to take a look as well?

Well, Jenkins can definitely respond to PRs and jobs/pipelines.

It can be done of course. Though I doubt the community or us contributors have much of a say in this kind of stuff.


How to improve the speed of dataset downloading

Hello, I'm very interested in the dataset you have released and would like to use it for some research. However, I've noticed that the download speed is currently very slow, averaging around 100 kb/s. Is there a faster way to download the dataset or an alternative download channel with higher speed? Thank you very much!

I noticed that you provide a small version of the dataset, but I couldn't find the corresponding labels. Could you please provide the labels for the small version?

You would need to download the small image sets in conjunction with the full label and metadata sets. The datasets are served from S3 bucket so we do not have control over its performance, and we don't have resources to release an alternative download in multi-region.