Fan demo

Description:
Add demo platform for fans - should also add unit test coverage around the fan/__init__.py file that is missing. Still a WIP.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If code communicates with devices, web services, or a:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

@Teagan42 If you're happy with this it can be merged, it's got my 🐬 :smile:


Monthly leaderboard

Closes #31

@rithviknishad Let's do a more generic summary instead of separate weekly and monthly summary


Correction for raw affiliation Service de Réanimation Polyvalente, Centre Hospitalier de Narbonne, Narbonne, France

Correction needed for raw affiliation Service de Réanimation Polyvalente, Centre Hospitalier de Narbonne, Narbonne, France
raw_affiliation_name: Service de Réanimation Polyvalente, Centre Hospitalier de Narbonne, Narbonne, France
new_rors:
previous_rors: 022m8c549
works_examples: W3120666205
contact: 8c35240573dc05b5df490abd393aa135:03831771b0ee27c6fef31c9e72ad43f1d104 @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Sequence contains no elements at  5.0.0-alpha

Every operation with collection throw System.InvalidOperationException.
Framework: Xamarin.Forms
Platform:  Android OS
Library Version: 5.0.0-alpha
Code to reproduce:
var db = new LiteDatabase("database.db"));
var collection = db.GetCollection<T>("exampleColleciton");
var allElements= collection.FindAll();

Also Insert, Delete, Find will throw
System.InvalidOperationException: Sequence contains no elements
StackTrace:

Enumerable.First[TSource] (System.Collections.Generic.IEnumerable`1[T] source)
LiteDB.Engine.LiteEngine
LiteDB.Engine.LiteEngine..ctor (LiteDB.Engine.EngineSettings settings) [0x0011b] in <c593d84b647a4ed5b475ff841900e3c2>:0
LiteDB
ConnectionString.CreateEngine ()
System
Lazy`1[T].ViaFactory (System.Threading.LazyThreadSafetyMode mode)
System
LazyHelper.ThrowException ()
System
Lazy`1[T].CreateValue ()
System
Lazy`1[T].get_Value ()
System
Lazy`1[T].CreateValue ()
System
Lazy`1[T].get_Value ()
LiteDB
LiteCollection`1[T].Find (LiteDB.Query query, System.Int32 skip, System.Int32 limit)

I ran into the same problem. It only occurs when I create the database completely new (0kB file). The Exception is thrown on LiteDatabase db = new LiteDatabase(liteDbPath). It can however be opened by the LiteDB Studio.
I solved the problem by simply downgrading to 4.1.4.

Yeah I've got the same problem with Xamarin.Forms 4.5.
The App is crashing at this Line:
 var db = new LiteDatabase(dbPath);
I switched back to 4.1.4.

Also 5.0.7

@VasenevEA @Enviolet @bluekuen @biapar This error is probably happening because LiteDB is trying to open a zero-byte file (which is obviously not a valid datafile). We'll release a fix for this eventually, but in the meantime just make sure the file you're trying to open is a valid datafile (and not a zero-byte file).

yeah that was the problem, i fixed it by deleting the file creation:
        public string GetDatabasePath()
        {
            var path = Path.Combine(_appData, "ZAsetMobile.db");

            //if (!File.Exists(path))
            //{
            //    using (var streamWriter = new StreamWriter(File.Create(path))) ;
            //}

            return path;
        }

Many thanks for your help!
kind regards, bluekuen

@VasenevEA @Enviolet @bluekuen @biapar This error is probably happening because LiteDB is trying to open a zero-byte file (which is obviously not a valid datafile). We'll release a fix for this eventually, but in the meantime just make sure the file you're trying to open is a valid datafile (and not a zero-byte file).

"Fix this eventually"?  I just came across this problem and it seems like it should be a relatively simply fix to check for a zero byte file and act accordingly.  It's something people are quite likely to do when creating a new DB so I think you guys should fix it ASAP.

Is this fixed already?

Unfortunately still no fix...

2020-10-26
v 5.0.9
Not fixed yet.

Still seems to be happening in 5.0.10 but I got around it by setting an initial size with the following code...
`string dbpath = DependencyService.Get().DatabasePath();
var conn = new ConnectionString()
{
Filename = dbpath,
Connection = ConnectionType.Direct,
InitialSize = 8192
};
        DBInstance = new LiteDatabase(conn);   `

This has apparently been fixed, and the bug should probably be closed; see: https://github.com/mbdavid/LiteDB/issues/1956


how to remove code of function in documentation?

Hi,
I try to make documentation for a function:
    /**
     * get Theme object
     * @param {string} status
     * @param {string} size
     * @param {string} disabled
     * @param {string} theme
     * @returns {object}
     */
    export function getTheme(status,size,disabled,theme) {

In Markdown and JSON I get the whole code from inside the function inside the markdown/JSON (value attribute) in "Prop" list.
How to avoid this?
Thank you!

Currently, the markdown/json generation is lagging behind the TypeScript generation, which should not include the actual function code. Labelling this as a bug.

@imperator-maximus What do you expect the value to be for the Markdown/JSON formats? The type instead?

each function should have its own table for parameters:
/**
 * A quite wonderful function.
 * @param {object} - Privacy gown
 * @param {object} - Security
 * @returns {survival}
 */
function protection (cloak, dagger) {}

A quite wonderful function.

**Kind**: global function

| Param  | Type                | Description  |
| ------ | ------------------- | ------------ |
| cloak  | object | Privacy gown |
| dagger | object | Security     |

The @param, @returns is currently not supported. Have you tried defining the entire signature using @type?
/** @typedef {any} Survival */
/** @type {(cloak: object, dagger: object) => Survival} */

well that would be a workaround (@param, @returns support would be better for sure).  Anyway the bug still has to be fixed somehow first that the whole function code will not be copied in markdown.


Load.bigTextStyle() API is misleading

The signature for Load.bigTextStyle() seems to have been copy pasted a few times to many and we have argument variables claiming to be bigTextStyle (as if a style resource) when it is in fact the big text.

I understand where you were going with the API naming convention as I saw Load.inboxStyle(). My mistake opening up this issue. Maybe the arguments could be renamed, but this is not an issue worth keeping open.


New RuboCop autocorrect

The autocorrection API was changed in Rubocop v0.87.0 (pull request rubocop-hq/rubocop#7868). This PR makes rubocop-rspec compatible with those changes.
The superclass of RuboCop::Cop::RSpec::Cop is changed from ::RuboCop::Cop::Cop to ::RuboCop::Cop::Base.
This has a few consequences:

Our #message methods get called with a different argument than before. It can be customized by defining a #callback_argument method, but I think it is clearer to avoid callbacks altogether.
Our #autocorrect methods don't get called anymore. Instead, we extend Autocorrector, and the corrector is being yielded when calling #add_offense, so the code is mostly moved in there. For some cases, this means that some code can be removed, which is nice. For some cases, it means that the methods get too long, or the code complexity gets too high, and in those cases I chose to just call out to an #autocorrect method anyway, but of course passing the corrector and any usable local variables along.
Our dependency of RuboCop is bumped from '>= 0.68.1' to '>= 0.87.0'.

See also https://github.com/rubocop-hq/rubocop/blob/master/docs/modules/ROOT/pages/v1_upgrade_notes.adoc for documentation of many of the changes.
We probably need some documentation changes too; I haven’t looked into that yet.

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Updated documentation.
[ ] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

CodeClimate complains about some added complexity, which should probably be addressed. But I think it’s important that we get a new release out pretty quickly, because right now we are incompatible with the most recent version of RuboCop.

There's a lot of activity here 👍
Let me know if I should review this now, or ping me when the dust settles 😄

@Darhazer I won’t have time to work on this today. If you have time could you please squash, merge and release? If not, I’ll look at it tomorrow.

Wow, impressive PR 💪!

Thanks. I hope I got everything right 😅

Sleek! 👏
Thanks a lot guys.


introduce the Metric class

@lbbrhzn , sorry I appear to have made a mistake trying to merge your metrics_class into my repo for testing. Are you able to revert this as I can't seem to?

yes. I did:
'''
git reset dae06b
git  push --force origin
'''


[Bug]: Data is NULL. User is rate-limited by GitHub" error

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
The current code checks if userData.name is null or empty to determine if the user is rate-limited. However, this might not be the most reliable method, as some users might genuinely have no name set on their GitHub profile. So it should be changed to userData.username.
Expected behavior
userData.username should be used here.
Add ScreenShots
No response
On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Thank you for creating this issue! 🎉 We'll look into it as soon as possible. In the meantime, please make sure to provide all the necessary details and context. If you have any questions or additional information, feel free to add them here. Your contributions are highly appreciated! 😊
You can also check our CONTRIBUTING.md for guidelines on contributing to this project.

PR #321 solves this issue

Hello @Swastik19Nit! Your issue #324 has been closed. Thank you for your contribution!


ModuleNotFoundError: No module named 'libml' in scripts/create_datasets.py

I'm using Ubuntu 16.04.
Maybe there are differences between OSs in how the context gets loaded.
But in my understanding the script can't reach the libml directory because its located in top level directory.
CUDA_VISIBLE_DEVICES= ./scripts/create_datasets.py
[...]

  File "./scripts/create_datasets.py", line 33, in <module>
    from libml import data as libml_data
ModuleNotFoundError: No module named 'libml'

Also, it is complaining that scipy and Pillow (PIL) is missing.

Okay, just saw i have to set the pythonpath to the root ;)


ConnectedApp creation via connection.metadata.create() issues

https://github.com/forcedotcom/cli/issues/2203

can't repro this 😁
with isAdminApproved set to true/false:

this was probably fixed in jsforce v2

This should be fixed in v3, see: https://github.com/jsforce/jsforce/discussions/1529


TypeError: Cannot read property 'string' of undefined Webpart Tiles

Hi,
I get issue String of undefined after click configure tile in workbench, i get code from master.
I read this link but not help me https://github.com/pnp/sp-starter-kit/issues?q=TypeError%3A+Cannot+read+property+'string'+of+undefined+
log
TilesWebPart.ts:94 Uncaught TypeError: Cannot read property 'string' of undefined
at TilesWebPart.getPropertyPaneConfiguration (TilesWebPart.ts:94)
at TilesWebPart.BaseClientSideWebPart._internalGetPropertyPaneData (sp-webpart-base.js:1399)
at PropertyPaneController._renderPropertyPane (sp-webpart-base.js:3873)
at PropertyPaneController._setPropertyPaneState (sp-webpart-base.js:3790)
at PropertyPaneController.setPropertyPaneState (sp-webpart-base.js:3764)
at ClientSideWebPartManager.setPropertyPaneState (sp-webpart-base.js:2121)
at PropertyPaneAccessor._open (sp-webpart-base.js:2536)
at PropertyPaneAccessor.open (sp-webpart-base.js:3317)
at Placeholder._this._handleBtnClick (PlaceholderComponent.tsx:75)
at HTMLUnknownElement.boundFunc (react-dom.js:8928)
Do you have any idea?
Thank you for your help in advance

Hi. Are you saying this happens in the workbench only? Does it happen when the tiles are deployed to a site?
Can you also please provide your environment information in the issue template.


[SPARK-13018][Docs] Replace example code in mllib-pmml-model-export.md using include_example

Replace example code in mllib-pmml-model-export.md using include_example
https://issues.apache.org/jira/browse/SPARK-13018
The example code in the user guide is embedded in the markdown and hence it is not easy to test. It would be nice to automatically test them. This JIRA is to discuss options to automate example code testing and see what we can do in Spark 1.6.
Goal is to move actual example code to spark/examples and test compilation in Jenkins builds. Then in the markdown, we can reference part of the code to show in the user guide. This requires adding a Jekyll tag that is similar to https://github.com/jekyll/jekyll/blob/master/lib/jekyll/tags/include.rb, e.g., called include_example.
{% include_example scala/org/apache/spark/examples/mllib/PMMLModelExportExample.scala %}
Jekyll will find examples/src/main/scala/org/apache/spark/examples/mllib/PMMLModelExportExample.scala and pick code blocks marked "example" and replace code block in
{% highlight %}
in the markdown.
See more sub-tasks in parent ticket: https://issues.apache.org/jira/browse/SPARK-11337

Can one of the admins verify this patch?

ok to test

cc @yinxusen

@keypointt @yinxusen Could you start from this PR first and then other longer ones? @keypointt is new to Spark development. Starting from small ones could avoid unnecessary code rewrites. Thanks!

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51143/
Test FAILed.

retest it please

Test build #51254 has started for PR 11126 at commit 5315350.

Test build #51254 has finished for PR 11126 at commit 5315350.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51254/
Test PASSed.

Merged build finished. Test PASSed.

@keypointt I recommend adding the following code to the example, too. Then delete previous code from line 55-67. Meanwhile, we should move the line 53 ahead and rephrase it.
// Export the model to a String in PMML format
clusters.toPMML

// Export the model to a local file in PMML format
clusters.toPMML("/tmp/kmeans.xml")

// Export the model to a directory on a distributed file system in PMML format
clusters.toPMML(sc,"/tmp/kmeans")

// Export the model to the OutputStream in PMML format
clusters.toPMML(System.out)

@mengxr If we don't use https://github.com/apache/spark/pull/9430, I suggest to add those code together into a single file and rephrase those lines among those code snippets. Otherwise we get the following result:

Test build #51256 has started for PR 11126 at commit 8da72d2.

Test build #51256 has finished for PR 11126 at commit 8da72d2.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51256/
Test FAILed.

Merged build finished. Test FAILed.

Test build #51263 has started for PR 11126 at commit bb6329a.

Test build #51263 has finished for PR 11126 at commit bb6329a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/51263/
Test PASSed.

@mengxr LGTM

Merged into master. Thanks!


Nearby Flight app

The following adds the ability to show a nearby flight (sometimes overhead) of your location using data from a private Airframes API (https://airframes.io) made specifically for the Tidbyt. I am the founder of Airframes, so I am authorized to release this.

I have read the CLA Document and I hereby sign the CLA

Thank you @matslina for the comments and review. I will look at the feedback and begin to address.


Latest dev version "TypeError: 'method' object is not iterable"

Issue related to
Sunsynk / mbusd Home Assistant Add-On
Describe the issue/bug
A clear and concise description of what the bug is.
Just updated to the latest and now the adon stops running.
Your environment

Home Assistant version: 2023.4.2
Addon version: 2023.01.19-0.2.9 (dev)
Inverter make: Sunsynk
Inverter Model: __
Inverter Firmware: __

Logs
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] done.
[services.d] starting services
[services.d] done.
2023-04-12 09:22:40,471 INFO    Loading HASS OS configuration
2023-04-12 09:22:40,471 INFO    PROFILES were deprecated. Please remove this configuration.
2023-04-12 09:22:40,489 INFO    ############################################################
2023-04-12 09:22:40,489 INFO        Sensor deprecated: total_pv_power -> total_pv_energy
2023-04-12 09:22:40,489 INFO    ############################################################
2023-04-12 09:22:40,489 DEBUG   last unit:kWh, id:total_pv_power
2023-04-12 09:22:40,489 DEBUG   last unit:kWh, id:total_battery_charge
2023-04-12 09:22:40,489 DEBUG   last unit:kWh, id:total_battery_discharge
2023-04-12 09:22:40,489 DEBUG   last unit:kWh, id:total_grid_export
2023-04-12 09:22:40,490 DEBUG   last unit:kWh, id:total_grid_import
2023-04-12 09:22:40,490 DEBUG   last unit:%, id:battery_soc
2023-04-12 09:22:40,490 DEBUG   step unit:W, id:essential_power
2023-04-12 09:22:40,490 DEBUG   step unit:W, id:non_essential_power
2023-04-12 09:22:40,490 DEBUG   step unit:W, id:grid_ct_power
2023-04-12 09:22:40,490 DEBUG   step unit:W, id:battery_power
2023-04-12 09:22:40,490 DEBUG   avg unit:V, id:battery_voltage
2023-04-12 09:22:40,490 DEBUG   step unit:A, id:battery_current
2023-04-12 09:22:40,490 DEBUG   avg unit:°C, id:battery_temperature
2023-04-12 09:22:40,490 DEBUG   step unit:W, id:grid_power
2023-04-12 09:22:40,490 DEBUG   step unit:Hz, id:grid_frequency
2023-04-12 09:22:40,490 DEBUG   avg unit:V, id:grid_voltage
2023-04-12 09:22:40,491 DEBUG   step unit:A, id:grid_current
2023-04-12 09:22:40,491 DEBUG   step unit:W, id:inverter_power
2023-04-12 09:22:40,491 DEBUG   step unit:Hz, id:inverter_frequency
2023-04-12 09:22:40,491 DEBUG   step unit:W, id:pv1_power
2023-04-12 09:22:40,491 DEBUG   avg unit:V, id:pv1_voltage
2023-04-12 09:22:40,491 DEBUG   step unit:A, id:pv1_current
Traceback (most recent call last):
  File "/usr/src/app/./run.py", line 472, in <module>
    startup()
  File "/usr/src/app/./run.py", line 261, in startup
    setup_sensors()
  File "/usr/src/app/./run.py", line 307, in setup_sensors
    for dep in sen.dependencies:
TypeError: 'method' object is not iterable
[cmd] ./run.py exited 1
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.

I have a single Sunsynk inverter, so using "multi" doesn't make sense.
I'm already using mbusd running on a raspberry pi.

Maybe this will fix it - https://github.com/kellerza/sunsynk/commit/0c9d99ef7accbe982a8c7f4f9371772391759aab
You will have to force an update since the version did not change

Glad that works for you. Note that only multi will get updates going forward

Changing to Multi works too. Thanks!

Ideally everyone should use Multi. But there is no way to seamlessly migrate, since the config changed.
Overall it should be better. It's only the serial driver that is broken in Multi 😉

This was fixed recently, but I still recommend the MULTI addon


Add support for AWS_REGION env variable

I was facing some HTTP code 307 redirects when using the scripts, so I added the possibility to use a AWS_REGION environment variable.

@paulhammond do you accept this change?


Remove extras_require from setup.py

I don't think this features is used/tested/maintained.

Closed by https://github.com/dwavesystems/dimod/commit/a9ad1c3fc19e9682be6d79183a81974855e7b450


Builder REST API changes for Deneb

PR Description
Implements changes described in https://ethereum.github.io/builder-specs/?urls.primaryName=dev

Changed BuilderClient interface to pass a SignedBlindedBlockContainer which could be either SignedBlindedBlockContents or SignedBeaconBlock
Changed return type of getPayload to BuilderPayload which can contain BlobsBundle as well as the ExecutionPayload
Introduced BuilderPayloadSchema to be able to deserialize the BuilderPayload
Added deneb examples to RestBuilderClientTest
Made some renamings to be consistent

TODO in next PR: use the BuilderPayload response to create BlobSidecar(s) and fallback to local if needed.
Fixed Issue(s)
will help with #7170
Documentation

[X] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[X] I thought about adding a changelog entry, and added one if I deemed necessary.

LGTM
But I don't like a bit that now ExecutionPayload extends BuilderPayload. I understand that it's a way to make all of this work in Java, but intuitively ExecutionPayload is base spec thing and Builder is 3rd party thing, so this hierarchy touches an eye. If we could find some better domain implementation for this, it will be better I think. Just my opinion

I see what you mean. I can't think of cleaner approach though and still take advantage of same schemas, type definitions etc.


Docker support for linux/arm/v7

When building on RPi4 I got the following message
Sending build context to Docker daemon  22.42MB
Step 1/21 : FROM maven:3-jdk-11 AS scratch
3-jdk-11: Pulling from library/maven
no matching manifest for linux/arm/v7 in the manifest list entries

Would it be possible to modify the docker image build process so that all dependencies support linux/arm/v7 platform?

Apparently the maven Docker image does support ARM v8, just not v7. Too bad.
It is certainly possible to download a vanilla image of some Linux distro that supports both 7 and 8, download Maven into it, and perform the same job. However, it is not on the immediate list of priorities for me.
Contributions are welcome!

FYI https://groups.google.com/g/robozonky-users/c/2esklY9nVbI/m/aYace_99AAAJ


Importing .nessus and .burp files

Do you support importing .nessus and .burp files?

@Simon-Davies Thanks for reaching out.  Currently, we do not support importing .nessus or .burp files.   If this is a feature you would like to request, please provide the following information:
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.


Change border color of container

Hi
I wanna change the border color of the select, how can I do this?
I got this, http://jsfiddle.net/8swu3/61/but it doesnt work
Thanks

sa


Debian repo version sound issues

Hi, I've tried installing the debian version of torbrowser-launcher in stretch and buster but after the installation I get no sound at all. The direct download of  the browser from tor website works fine with sound.
Thanks.

Hi, I can confirm this issue using debian buster (10.0) with sid
Seems to be related to #328

Workaround for Debian Buster
The version in develop supports sound output.
sudo apt install torbrowser-launcher
cd /etc/apparmor.d
sudo mv torbrowser.Browser.firefox torbrowser.Browser.firefox.orig
sudo wget https://raw.githubusercontent.com/micahflee/torbrowser-launcher/develop/apparmor/torbrowser.Browser.firefox
sudo service apparmor restart
sudo service apparmor status

Expected output
systemd[1]: Starting Load AppArmor profiles...
apparmor.systemd[3319]: Restarting AppArmor
apparmor.systemd[3319]: Reloading AppArmor profiles
apparmor.systemd[3319]: Ignoring: 'torbrowser.Browser.firefox.orig'
systemd[1]: Started Load AppArmor profiles.

I'll assume this is/was a Debian specific problem caused by Debian shipping a version of the AppArmor policy that did not include #396 ⇒ not something we should be tracking upstream ⇒ closing.


Document steps to setup OpenEBS on Containerized OpenShift cluster

Is this a BUG REPORT or FEATURE REQUEST?
FEATURE REQUEST

What happened:
The Containerized cluster is a popular deployment model for OpenShift, with all the platform services run as containers instead of RPMs on the master and host nodes. We need to document the steps to
create the above cluster and setup OpenEBS with sample applications on it.
https://docs.openshift.com/container-platform/3.3/install_config/install/advanced_install.html

The steps are available at https://docs.openebs.io/docs/next/openshift.html

Re-opening this issue as the referenced steps are for baremetal service-based/package install of openshift. There may be minor changes/additions in case of containerized openshift clusters. This still needs dev docs - which is what the issue intends to track.


Could not locate update bundle

I get this error after an update is downloaded. I see the log of update downloaded but "update-downloaded" event is never fired. I've checked the folder where zip should be and it's empty but something is downloaded and removed before the crash.
The full error is:
Could not locate update bundle for {appname} within file:///Users/{me}/Library/Application%20Support/{appname}.ShipIt/update.nQcEUZ3/: (null) at Error (native)

Any idea on what could be wrong? I'm using electron-release and electron-packager to package and upload to github.

Hmm, thats quite hard to debug. :)
It's an Squirrel error, https://github.com/Squirrel/Squirrel.Mac/blob/master/Squirrel/SQRLUpdater.m#L448
Wild guess:
Is your bundleIdentifier the same on both the running app and the downloaded?
The bundle-id is set in info.plist.

Where's the info.plist? Thanks!

Yes, bundle-id is the same in both, just different version. I've posted on squirrel mac but no answer yet :(

I use https://github.com/maxogden/electron-packager

I'm using it too, not sure where my error is. My package.json is:
"name": "myapp",
"productName": "myapp",
"version": "1.0.3",
"main": "main.js",
"description": "A desktop version of the myapp web app.",
"repository": {
  "url": "https://github.com/myapp/myapp-desktop"
},
"scripts": {
  "dev": "electron .",
},
"dependencies": {
  "electron-gh-releases": "^2.0.2",
  "configstore": "^1.2.1",
  "electron-builder": "^2.3.1"
},
"devDependencies": {
  "grunt": "^0.4.5",
  "grunt-electron-installer": "^1.2.0",
  "electron-packager": "^5.1.1",
  "electron-prebuilt": "^0.35.2"
}

@jenslind where can I check the bundle id? I'm trying to find the issue in squirrel but issue seems to be in electron-gh-releases on electron-packager setup. This is how I build my osx app:
electron-packager ./ myapp --out=dist/osx --platform=darwin --arch=all --version=0.35.2 --app-version 1.0.3 --app-bundle-id=myapp --helper-bundle-id=myapp --overwrite=true --asar=true --icon=assets/osx/myapp.icns --prune=true --ignore=dist --sign="Developer ID Application: MyCompany."

You can find the bundle id in the apps info.plist file.
yourapp.app/Contents/Info.plist the key is CFBundleIdentifier
It needs to be the same on the running app and the version that its downloaded.

Ok, checked and both are the same. Any idea on how can I get a better error?  Maybe --helper-bundle-id shouldn't be the same as bundle-id, asar option? Not sure what can be wrong if it works in other cases. My app is just a web wrapper.

Hmm, I have no idea, do you mind linking to the repo?

@jenslind sure! Thank you for your help. Here's the repo with the releases (removed some) and auto_updater.json
https://github.com/mmmelon/mmmelon-desktop

Looks like the downloaded zip contains .dmg it needs to be the .app.

omg... I'm stupid... I'll try changing the zip to just contain .app and post result here. Thanks @jenslind!

Thanks @jenslind . It works!

it is several years later, but I am having this issue. following the recommendation by @jenslind I have isolated the downloaded files and it is a zipped .app. however as @bitomule originally encountered, the updater pulls the files down, but instantly deletes them with the error:
2018-05-11 12:02:55.640 Electron[3283:95513] Download completed to: file:///path/to/Caches/com.github.electron.ShipIt/update.a74SCao/0.0.6.zip
{ Error: Could not locate update bundle for com.github.electron within file:///path/to/Caches/com.github.electron.ShipIt/update.a74SCao/ code: 2, domain: 'SQRLUpdaterErrorDomain' }

I am running on local, which is why it is putting it in com.github.electron instead of the proper bundle id of my app. if running the distributed app, I encounter the same problem.

@benjtinsley Did you end up figuring this out? I'm getting the same error. My .zip only contains the .app file so it's not the issue noted earlier.

@mike-seekwell yes, the error was i had an app that was not properly signed. mac was downloading, checking the verification, seeing it was unverified and then silently deleting the file with no feedback. i installed RB App Checker Lite to verify the app to ensure my certificates were signing correctly. once that was done, i was sure to have a verified version of the app installed and then tried running the autoupdater with a verified revision and it worked.
electron development is quite the diaspora so it's often hard to pinpoint where the real issue lies but this was it for me. it had nothing to do with this project at all, so if we want to close this issue if others are finding success with my learnings, please do.

Yeah, I was trying to get it to work on my dev version (i.e. npm start), but couldn't get it to work. When I tried the packaged version, autoupdating worked.

@mike-seekwell gotcha, must be something about com.github.electron being the bundle ID on local. i'm sure there's a better way to verify this in development mode, but i haven't found it. sort of a bummer that it takes 10-ish mins to check your work :shrug:


Saving works unreliably

Godot version
4.0.2
System information
Android 11
Issue description
Saving files did work for the first time opening the app but doesn't work after, because it looks like it lacks authorization
Steps to reproduce
Try creating a script
Minimal reproduction project

projec.zip

It's an issue with writing files, but i think it's my device and it's not a godot problem, can i close the issue?


Change ladder URL to coh2.org

The companyofheroes.com ladders were taken down, so the bot's ladder links lead to a 404 page. coh2.org is probably the best alternative.
Eg:
This link is broken
https://www.companyofheroes.com/leaderboards#profile/steam/76561198404414770/standings
This works
https://www.coh2.org/ladders/playercard/steamid/76561198404414770

Or maybe coh2stats.com is better
Eg https://coh2stats.com/players/76561198404414770

@mateimarica coh2stats.com seems like a better option because it's actively maintained. Will you update the pull request with coh2stats.com URLs?

I've requested permission to use cohstats.com as the playercard URL on https://github.com/cohstats/coh2stats/issues/244.


fix: Package crash on server-side import

The forms/FileInput/FilePreview.tsx sub-component was importing a string from a constants file; this caused Webpack to include all exported variables in that file in the final build. This meant that uses of the File web-only API were included in said build, which caused server-side site generation (such as Next or Gatsby) to throw a ReferenceError when importing this package, as those are running in a node-only context where that API doesn't exist.
The simple fix was to move the value of that constant into the FilePreview subcomponent, which prevents the entire constants file from being included in a webpack build.
This PR also adds:

A very simple e2e test (/src/serverSideTest.js) to check for package importability -- it just requires the version of the local version of the package that's been built by webpack and immediately exports it. This is sufficient to uncover any future ReferenceErrors-on-import like this.
A package.json command (yarn test:serverside) that just runs a webpack build followed immediately by the aforementioned e2e test.
An update to CircleCI, running that command immediately following the traditional test suite.

Summary
Closes #1250
How To Test

Make sure that this PR's CircleCI build check passes, and that it includes the yarn test:serverside task as part of it.
Run yarn test:serverside, and verify that the command exits with code 0. Edit line 4 of src/components/forms/FileInput/FilePreview.tsx back (to import { SPACER_GIF } from './constants'), then run yarn test:serverside again and verify that the command now exits with code 1.

Warnings

:warning:

Changes were made to package.json, but not to yarn.lock - Perhaps you need to run yarn install?

:warning:
This PR does not include changes to storybook, even though it affects component code.

:warning:
This PR does not include changes to tests, even though it affects source code.

  Generated by :no_entry_sign: dangerJS against 67db8354c8f999c8d8cadb75fd03769d00e905ad

Warnings

These are all expected - the change to package.json is adding a new yarn command, and the change to a component file is simply moving the location of a constant's declaration without changing the value.


Ignore repeated JOIN attempts to a channel

This moves the this.rooms[uri] check from subscribeToRoom to joinRoom,
so that it happens before the JOIN/NAMES replies are sent to the client.
This fixes some recent issues with the server-side autojoin and pidgin
(which sends its own set of JOIN on connection of all open windows)
Ref #62

Seems like a good change. @dequis Which clients have you tested this in?

Pidgin, but it's simple enough that you can do it with any client by sending "/quote join #channel" twice and check the gitter server debug output

Just tested in HexChat and did some logging to confirm it doesn't send out the extra joins.

@dequis Thanks for the contribution :grinning: - It has been merged into develop and will most likely be in a release in the next few days.


Hello and thanks !!!

Hi thanks for your contribution in the open source.
I tried your code and get only the blur version in the blur folder. would it be possible to have an exemple?
cheers bro !!!!

hi @reminosouci! Thanks for taking the time to open up a bug to ask your question. I have uploaded a directory of example images with their masks and the output. Let me know if that helps you get an idea of what is required. Thanks again!

I'm going to go ahead and close this. If you still need additional examples or help please let me know!


msg::callback::handle args determined by caller

Previous versions of message callback handler construction would deduce the ExtraArgs from the callable passed in by the user when forming the callback. This is no longer the case which means that the signature of msg::callback::handle does not match the signature of the contained callable.
This PR works around the issue; however, the real question is, "do we want to deduce the ExtraArgs from the user's handler? Deducing has some real problems because the handler is not always concrete (e.g. a generic lambda). I think the PR solution is probably the correct route.

If we remove ExtraArgs... in this one place we should strip it out from everywhere; it's now unused, right?

If we remove ExtraArgs... in this one place we should strip it out from everywhere; it's now unused, right?

I believe so. I'll update the PR.


Expose the declaration file

Copy the declaration file from src to dist before publishing
The file will be available from the public package (graphql-tag.macro/index.d.ts)

I think if you use —copy-files with Babel it will make manually copying unnecessary.

@zackify Great point! But babel copies everything not compiled. So the files that are in the --ignore flag are copied too.

Should we cp or --copy-files then rm files?

Good point! It looks good to me. I will approve. But will wait and see if @audiolion or @leoasis can review quickly before merging

Just read the TypeScript documentation and saw:

Including declarations in your npm package
If your package has a main .js file, you will need to indicate the main declaration file in your package.json file as well. Set the types property to point to your bundled declaration file.

So, I updated this PR to follow the documentation. 😖

Nice, I didn’t know that. In my one public package I wrote types for, I just did the declaration export next to each file. Good to know.

I released<EMAIL_ADDRESS>try it out. thanks again

@zackify Is there a mistake on the version number?


CPP/error/binarySearch.cpp

rectify the error in the code.
NOTE:- Screenshot of the code and output is required.

Can you please assign this issue?

can you assign me this issue?

can you please assign me this issue ?

Done with PR #303 !


Sound setPitch() doesn't work on Android

Sound
engine.zip
Sound engineSound = Gdx.audio.newSound(Gdx.files.internal("data/engine.wav"));
long soundId = engineSound.loop();
float pitch = 0.5f + ((velX / maxVelocity) * 0.5f);
engineSound.setPitch(soundId, pitch);
The above code does not change sound pitch on Android, but it works perfectly on desktop.
libgdx ver 1.7.1
Android ver 6.0.1

Please don't defeat the issue template.

Please post a self contained executable example that reproduces the error.  Try not to play sounds as soon as you have used newSound also, it may not be fully loaded by then.

setPitch.zip
Example attached. On desktop the sound object loops continuously while cycling its pitch. However, on Android the sound object loops but fails to change pitch.

Not sure if it's the same problem, but I had an issue a while ago where I stumbled over the supported range in the pitch-implementations. Android has a smaller supported range for setPitch:
pitch < 0.5 and > 2 works on desktop but fails on android.

Thanks madpew. I think the cause is different though as the pitch multiplier used in this case is between 0.5 and 1.0.

@jfnogueira This is not a self contained executable example that reproduces the error.
We need the source of the example.

Unable to reproduce, since you've been unable to provide an SSCCE, I'm going to close this out.

I created a simple example that's supposed to play a sound with 1.0 pitch and ~1.5 pitch alternating at each click, and on desktop this works, but on android even at relatively long intervals between clicks the sound that is played will be the wrong pitch quite often. Could you investigate the issue?
test.zip

Could you please provide a http://sscce.org ?
Some information about your device and android version would also be good.

Here you go (although I couldn't make the desktop version compilable without opening it first in IDEA and setting the working directory to android/assets, but opening it in IDEA makes it much larger, so i didn't, however the android version is compilable)
Device: Sony xperia z1 compact (D5503, android 5.1.1, API 22)
I also tried this with an nvidia shield tablet (6.0.1, API 23), and a nexus 5(6.0.1, API 23), but the problem only occured on the xperia.
In the manifest the android versions are:

(But that can be seen in the zip too)
I also tried to build with targetSdkVersion 22 for the xperia, but no change.
Let me know if you need any more information
test_clean.zip

Your test works on my devices. I'm afraid we are not able to investigate device specific issues.
Only thing I saw that's suspect is this on the log:
05-20 07:59:35.240 16857-16893/com.mygdx.game W/AudioTrack: AUDIO_OUTPUT_FLAG_FAST denied by client; transfer 4, track 66075 Hz, output 48000 Hz
Perhaps try a different sample rate (44.1k or 48k would be good standard options).

Actually the audio file in the zip has a sample rate of 44.1 kHz, so how would i try a different sample rate?

Hi Anyone succeed in this issue (#pitch not working on android)...?

I think I worked around the issue by creating all the pitch transformed sounds I needed in audacity, then playing back those with 1.0 pitch. Later I used midi to play the sounds (piano), but that might not be applicable to your app

Actually I have an array of coins and I need to increase pitch of sound as user collect coins in an array... but it didn't works correctly I have also changed its value between 0.1 to 0.5 but didn't succeed ... :(


New term - cultivarEpithet

New Term
Submitter: Markus Döring
Justification: cultivar names can only be shared now via the full scientificName and need their own atomized term.
Proponents: Workshop on Biodiversity Data Quality (2016-03), TCS
Definition: Part of the name of a cultivar, cultivar group or grex that follows the scientific name.
Comment: According to the Rules of the Cultivated Plant Code, a cultivar name consists of a botanical name followed by a cultivar epithet. The value given as the cultivarEpithet should exclude any quotes. The term taxonRank should be used to indicate which type of cultivated plant name (e.g. cultivar, cultivar group, grex) is concerned. This epithet, including any enclosing apostrophes or suffix, should be provided in scientificName as well.
Examples: King Edward (for scientificName "Solanum tuberosum 'King Edward'" and taxonRank "cultivar"); Mishmiense (for scientificName "Rhododendron boothii Mishmiense Group" and taxonRank "cultivar group"); Atlantis (for scientificName "Paphiopedilum Atlantis grex" and taxonRank "grex").
Refines: None
Replaces: None
ABCD 2.06: http://rs.tdwg.org/abcd/terms/cultivarName or http://rs.tdwg.org/abcd/terms/cultivarGroupName or http://rs.tdwg.org/abcd/terms/breed (ABCD 3.0)
Original comment: https://code.google.com/p/darwincore/issues/detail?id=141
Mar 1, 2012 comment #1 dag.endresen
One potential challenge of adding cultivar name as part of the nomenclature code is perhaps that cultivar names can have language. The very same cultivar can be released under different marketing names in different countries.
Mar 2, 2012 comment #2 wixner
isn't that a kind of synonym then? Several names referring to the same taxon. Or does the cultivar code explicitly accept several localisations for one name?
Mar 2, 2012 comment #3 wixner
There is also the need to share information about other informal name parts not governed by the nomenclatoral codes. For example bacterial strain names or temporary names in use before they have been published. Hyloxalus sp. JCS-2010a for example was updated to Hyloxalus yasuni when Paez-Vacas et al. (2010) was published. Other examples are Clostridium botulinum A112 or Clostridium botulinum A strain ATCC 19397.
Is it maybe worthwhile having an informalEpithet instead that caters for both cultivars and temporary names?
Mar 2, 2012 comment #4 dag.endresen
I believe that each 'localization' of a cultivar name could perhaps be seen as the 'valid' name in the country where the name is used/marketed - and as a 'synonym' in other countries...? I am not sure if it is always easy to pick one globally 'valid' cultivar name...? And each cultivar can have different release year in different countries, and I believe sometimes (but not always?) released and marketed under a different cultivar name...?
Mar 2, 2012 comment #5 dag.endresen
The International Code of Nomenclature for Cultivated Plants (ICNCP, Cultivated Plant Code) seems to regulate the 'cultivar epithet' to be a distinct name within a genus - and that the cultivar name (cultivar epithet) is a different thing from the 'trade name', and that it is the 'trade name' that can be a different name for the same cultivar marketed in each country... sorry for the confusion.
http://en.wikipedia.org/wiki/International_Code_of_Nomenclature_for_Cultivated_Plants
Mar 2, 2012 comment #6 dag.endresen
Which could mean that there is a role in Darwin Core for both a new cultivarEpithet and an informalEpithet term...?
Mar 2, 2012 comment #7 wixner
Wikipedia suggests there is only a single cultivar epithet:
http://en.wikipedia.org/wiki/Cultivar#Cultivar_names
"A cultivar name consists of a botanical name (of a genus, species, infraspecific taxon, interspecific hybrid or intergeneric hybrid) followed by a cultivar epithet. The cultivar epithet is capitalised and enclosed by single quotes; it should not be italicized. It is permissible to place a cultivar epithet after a common name provided the common name is botanically unambiguous. Cultivar epithets published before 1 January 1959 were often given a Latin form and can be readily confused with the specific epithets in botanical names; after that date, newly coined cultivar epithets must be in a modern vernacular language to distinguish them from botanical epithets."
A cultivar name also has to be published just like a scientific name before it can be registered. So the publishedIn(Year) terms would also apply nicely.
An example registration page for aroids can be found here:
http://www.aroid.org/cultivars/reg_form_short.php
Mar 2, 2012 comment #8 dag.endresen
There seems to be two systems for cultivar names - one system of "cultivar epithet" under the ICNCP (distinct names published in a journal) - and another system of "trade designations"/"trade names" generally under the UPOV (but with varying national legislation) where different "trade name" can be protected in distinct countries (different years) for the same cultivar. I believe that both these types of "cultivar names" might perhaps be useful to track...?
"Many plants have "selling names" or "marketing names" as well as a cultivar name; the ICNCP refers to these as "trade designations". Only the cultivar name is governed by the ICNCP. It is required to be unique", http://en.wikipedia.org/wiki/International_Code_of_Nomenclature_for_Cultivated_Plants.
Mar 6, 2012 comment #9 wixner
Added a new issue144 for strains
Sep 23, 2013 comment #12 gtuco.btuco
I would like to promote the adoption of the concepts mentioned in this issue. To do so, I will need a stronger proposal demonstrating the need to share this information - that is, that independent groups, organizations, projects have the same need and can reach a consensus proposal about how the term or terms should be used. It might be a good idea to circulate the proposal on tdwg-content and see if a community can be built around and support the additions.

Following the workshop on Biodiversity Data Quality held this week, I would like to ask the integration of the vernacular name (CultivarEpithet ) into DwC. We had a discussion about running or not a quality check on this vernacular name that was recommended by the ABD Task group but for trujning a qualtiy check,  we need DarwinCore standard to accept this addition. This is an important information element for our community - eco-geographic modeling agrobiodiversity  is also done to identify which cultivar grows where and derive possibly traits of interest for farmers (drouhgt tolerance, salinity tolerance, tec) and then identify where this cultivar is conserved as seeds in genebanks for distribution.

I'm currently working on publishing a set of occurrences in a botanical garden, this includes information the cultivars. Currently I can't find a way to encode this information properly in DarwinCore. In the meantime I've been using taxonRemarks which is not an ideal solution.
Apart from my specific use case I also see value in tracking information about the exact cultivar in the case of escaped species occurring in the wild, be it decorative or agricultural species.
In the case of the cultivarEpithet field, I expect the nomenclaturalCode should be ICNCP in this case. It is even mentioned as an example in the quick reference guide. But is there a controlled vocabulary for this field?
What about the taxonRank? What controlled vocabulary could be used? In my particular case I think it would make sense to declare cultivar for taxonRank.
I also think this issue could be broader then plants. What about animals, how would one encode breeds of dogs, horses, foul or cattle?

This term is in TCS, it is called cultivarNameGroup, I think. No plans to remove it. Problem is more with what label to use than with whether to have it or not or the definition.
I think 'cultivar' and 'cultivarGroup' are already in the taxonRank vocabulary?

Closing for lack of demand.

Sorry, I closed the one in error. It is back on the table.

I support the addition of this term.
In addition to the use cases already mentioned, there are named cultivars that are invasive and planted. Currently it is difficult to distinguish these from wild or other named cultivars that may or may not be invasive or planted. A specific cases is the genus Narcissus that is widely planted in Europe often near wild native populations.

Could we rewrite the definition so that it also covers cultivar groups and grexes, i.e. all the types of names governed by the International Code of Nomenclature for Cultivated Plants (ICNCP)?
E.g.:

Definition: Part of the name of a cultivar, cultivar group or grex that follows the scientific name, without the enclosing apostrophes or suffix.
Usage notes: taxonRank can be used to indicate which type of name is concerned. This epithet, including the enclosing apostrophes or suffix, should be provided in scientificName as well.
Examples: King Edward (in Solanum tuberosum 'King Edward', a cultivar), James Grieve (in Malus domestica 'James Grieve', a cultivar), Mishmiense (in Rhododendron boothii Mishmiense Group, a cultivar group), Atlantis (in Paphiopedilum Atlantis grex, a grex).

dwc:cultivarEpithet falls into the category of "convenience terms" and should therefore be added to table 3.5 in the RDF Guide. It should not have a dwciri: analog.

I support the addition of this term and the alignment with the cultivated code of nomenclature.
Agree that this is distinct from trade names and vernacular names.

ABCD has CultivarName (http://rs.tdwg.org/abcd/terms/cultivarName) and CultivarGroupName (http://rs.tdwg.org/abcd/terms/cultivarGroupName).
Examples for CultivarName: Firmgold, Granny Smith, Sturmer Pippin
for CultivaGroupName: Granny Smith Group

Thank you @jholetschek. Incorporated in the first comment.

I also think this issue could be broader then plants. What about animals, how would one encode breeds of dogs, horses, foul or cattle?

Definitely needed! In ABCD this is Breed within the Zoological name container (https://abcd.tdwg.org/terms/#breed)

Help wanted to formulate the additional usage comments.

I'm not a plant/cultivar  person, so I'm probably not the best person to help formulate the additional usage comments.  However, while I agree in principle that this concept could apply to animals (breeds, etc.), I don't think "cultivar" would be the right term.  I'm struggling to think of a single term that captures both "cultivar" and "breed".  I suppose "cultivarOrBreedEpithet" could work, but that's awfully cumbersome (though, perhaps no more cumbersome than, e.g., maximumDistanceAboveSurfaceInMeters).  if I think of something better, I'll post it here.

I think the important thing is that cultivarEpithet should be applied to code-compliant names. If or when there is a Code for names of breeds of animals (and if those names have epithets), we can make a decision on whether to use the same (renamed) term for those or a different one. I would lean towards keeping them separate.
@tucotuco , I am a plant person, but not a cultivated plant person, but I will have a go at the usage comments.

AgroVoc has plant and animal cultivar concepts distinct varieties & breeds
however, does not include any cultivar epithet

We endorse this proposal on behalf of @SiBColombia

I would also prefer to keep cultivar names and breeds distinct - unless their code and use is rather similar.
Is this slightly updated proposal acceptable?
Definition: The cultivar epithet as part of a scientific name being a cultivar plant name.
Comment: According to the Rules of the Cultivated Plant Code, a cultivar name consists of a botanical name followed by a cultivar epithet which is capitalised and enclosed by single quotes. The cultivarEpithet should exclude the quotes. The term taxonRank should be used to indicate which type of cultivated plant name (e.g. cultivar, cultivar group, grex) is concerned. This epithet, including the enclosing apostrophes or suffix, should be provided in scientificName as well.
Examples: King Edward (scientificName=Solanum tuberosum 'King Edward', rank=cultivar); Mishmiense (scientificName=Rhododendron boothii Mishmiense Group, rank=cultivar group); Atlantis (in Paphiopedilum Atlantis grex, a grex).

Hi @mdoering ,

According to the Rules of the Cultivated Plant Code, a cultivar name consists of a botanical name followed by a cultivar epithet which is capitalised and enclosed by single quotes.

This only goes for cultivar names s.str., while the rest of the comment and the examples also include cultivar groups and greges.

Ah, but they all go at the end, right? Would this work?

According to the Rules of the Cultivated Plant Code, a cultivar name consists of a botanical name followed by a cultivar epithet. The value given as the cultivarEpithet should exclude any quotes. The term taxonRank should be used to indicate which type of cultivated plant name (e.g. cultivar, cultivar group, grex) is concerned. This epithet, including any enclosing apostrophes or suffix, should be provided in scientificName as well.

Yes, that's good.

Definition: The cultivar epithet as part of a scientific name being a cultivar plant name.
Comment: According to the Rules of the Cultivated Plant Code, a cultivar name consists of a botanical name followed by a cultivar epithet. The value given as the cultivarEpithet should exclude any quotes. The term taxonRank should be used to indicate which type of cultivated plant name (e.g. cultivar, cultivar group, grex) is concerned. This epithet, including any enclosing apostrophes or suffix, should be provided in scientificName as well.
Examples: King Edward (scientificName=Solanum tuberosum 'King Edward', rank=cultivar); Mishmiense (scientificName=Rhododendron boothii Mishmiense Group, rank=cultivar group); Atlantis (in Paphiopedilum Atlantis grex, a grex).

That is a different definition from the one that went in and came out of the public review, and it is not an improvement.
This is your initial definition:

Part of the name of a cultivar, cultivar group or grex that follows the scientific name

I think you should keep that. Maybe just change 'scientific' into 'botanical', so that cultivar names can still be scientific names.

Done.

So this new cultivar epiteth is pr def a taxon rank with a defined position in the taxonomic hierarchy? In that case where?

There are 3 formal ranks for cultivars existing, cultivar, grex and group:
https://en.wikipedia.org/wiki/Cultivar#Formal_definition
In GBIF we have "ranked" cultivars rather low in the rank ordering:
https://github.com/gbif/name-parser/blob/master/name-parser-api/src/main/java/org/gbif/nameparser/api/Rank.java#L268

Thanks for quick reply.
I wonder if you are aware of the work by Ronald van den Berg et al. arguing that the ICNCP has a flaw at its core: the attempt to treat cultivar as a taxon in taxonomy for wild organisms, and that cultivars instead should be treated as part of a separate cultonomy for cultivated plants (with crop, cultivar-group and cultivar as the only culta*) so that a cultivar can be connected to the adequat level in the taxonomy?
E.g. Rosa 'Cultivar Name' on genus level.
Their conlusions are summed up at p.50 here https://pubag.nal.usda.gov/catalog/3433
)* They propose the word culton be introduced as the foundation for a cultonomy, the way taxon has this role for taxonomy.

Any reactions to my last entry?
Correction the system proposed by Hetterscheid and van den Berg has three systematic categories (with crop, cultivar-group and cultivar), since culta referes to the content of the category.
See also a project lauched yesterday https://culton.org/

Thanks for the links, that is an interesting development. I wonder how well DwC already fits to exchange a cultonomy and what else would be needed?

Thanks. A little too interesting sometimes for me, as I'm not a biologist nor a computer scientist, but just someone who had to solve these issues to deliver a system for a hobby seed exchange, and now we're suddenly looking at promoting a new cultonomic theory reform.
To your question: The way I've come to understand this, DwC isn't where to solve cultonomy. That's why we use a separate culton table where every crop culton has a link to a taxon. This allows for more than one crop with the same taxon.
Example: both Welch onion and Japanese bunching onion, which has to be separated cultonomically, can be assosiated to GBIF taxon 2856504 Allium fistulosum. The English name given in GBIF is only the last of the two, which is probably symptomatic.
This way of associating taxon and culton, provides the necessary flexibility in this combined system, as they are kept far enough apart. It is also easy to change association without compromizing the names of culta, which is one of the main goals of the project.
It is too early to conclude if we're on the right path with this project, so we must see how it goes. There are still many issues to solve as we get deeper into the details. Headaches for some time:
What should we replace the culton crop with in a system for both plant cultivars, animal breeds and mushroom and bacteria strains? Something with "product" seems most promising perhaps.
I would like also to end up with just one word for the basal culton cultivar, and have suggested cultivar for all life forms, but receive a lot of protests on that one, understandably. Have to live with both cultivar, breed and strain. (Ref. discussion earlier in this thread.)
By the the way, the "grex" from the ICNCP we think is wrongly set up, and don't use it. Cultivar group we use.


Page scroll reset when setting pig div to "display: none"

In the onClickHandler, say I have
var pigOptions = {
  urlForSize: function(imageData, size) {
    //...
  },
  onClickHandler: function(filename) {
    document.getElementById("pig").classList.add("hide");
    // code to show bigger pic of filename here

  },
};

corresponding css
.hide{
  display: none;
}

... in order to hide the contents of the pig div and show the photo in context, I'm seeing that the scroll event is called, and pig resets the output to the top.

Instead of display: none, I'd recommend either putting an opaque layer between the image and the background (I used that technique on this page) or use opacity:0 depending on your needs.

Thanks for the tip! Got it to work with opacity: 0 and adjusting the top of the photo to be shown using window.scrollY.
I looked at the other page you shared, but wasn't able to quite understand what was going on or which elements were used as opaque layer. Do you mind explaining in a few words? Thanks!

Here is the code for my openViewer function
The gist is:

figure out where on the page the element is, where it should be in the "viewer" state
Apply DOM transitions to scale the picture to full screen
Add a class to the body to signal that the scrim (opaque layer) should start to appear. See the CSS and HTML for the scrim.
Add listeners that close the viewer.

Thanks a lot!


Include note about Apple M1 for SQL Server docker image

Hi there. The SQL Server 2019 docker image will not work on a Mac with Apple M1 chip using Rosetta emulation. See this issue for more details: https://github.com/microsoft/mssql-docker/issues/668
It would be helpful if that was mentioned on this page. Even better would be to provide arm64 versions of the docker image. Thanks!

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

ID: b152444b-dfb8-5cbe-cf45-9830bd085a68
Version Independent ID: 31719660-d4d0-7f63-efcc-846c822a63c2
Content: Docker: Install containers for SQL Server on Linux - SQL Server
Content Source: docs/linux/quickstart-install-connect-docker.md
Product: sql
Technology: linux
GitHub Login: @vin-yu
Microsoft Alias: vinsonyu

@badsyntax -- Richard, thank you for your research and feedback.  This repo is solely for documentation issues; please add your suggestion for arm64 images here:

SQL Server Feedback

@vin-yu -- Vin, please look into this issue.

@amvin87

SQL Server 2019 container images are currently not supported for arm64, you can use azure-sql-edge container images for arm64 has documented here: https://azure.microsoft.com/en-in/products/azure-sql/edge/#:~:text=Bring the security and performance,machine learning and graph features.

This is not document on this page, hence the reason I created this issue (unless I'm missing something).
The following is stated as part of the Prerequisites:

Docker Engine 1.8+ on any supported Linux distribution or Docker for Mac/Windows.

To me, this implies it should work on any Mac. All i'm saying is, the docs for this page can be improved to explicity mention that the image cannot be built using a Mac with an M1 chip.

Sure, we can update the page to add the link to this page which mentions the hardware requirements including the supported processor types. : https://docs.microsoft.com/en-us/sql/sql-server/install/hardware-and-software-requirements-for-installing-sql-server-ver15?view=sql-server-ver15#pmosr

that would be helpful imo!


Does it Support Client Push If not then will it be avaible in future

For implementation related questions or technical support, please refer to the Stack Overflow and Server Fault communities.
Make sure these boxes are checked before submitting your issue -- thanks for reporting issues back to Parse Server!

[ ] You've met the prerequisites.

[ ] You're running the latest version of Parse Server.

[ ] You've searched through existing issues. Chances are that your issue has been reported or resolved before.

Environment Setup
Steps to reproduce
Logs/Trace

I doubt it will be supported and here's a good writeup on why -> http://blog.parse.com/learn/engineering/the-dangerous-world-of-client-push/

how much time will it take for that to happen

Please read the above link on why it SHOULD NOT be available.

can i use cloud code to send client push

yes u can. its a better secure way. the whole thing because its dangerous to let your users send push notification directly

Is there any guide for the implementation of this cloud code

try this:
Parse.Cloud.define('Push', function(req, res) {
var title = req.params.title
var alert = req.params.alert

// send to everyone :)
var query = new Parse.Query(Parse.Installation);

Parse.Push.send({
  where: query,
  data: {
    alert: alert,
    title: title
  }
}, {
  success: function(data) {
    res.success("Push was successful");
  },
  error: function(error) {
    res.error(error.code + ": " + error.message);
  },
  useMasterKey: true
});

});
then this form javascript or use equivalent method from other language :
var title = "title";
var alert = "alert";
Parse.Cloud.run('Push', {title: title, alert: alert}).then(function(results) {
console.log(results);
});
hope it helps you.

Will it be able to send to specfic person or to the whole android regostered devices

both should still be available, read up the clients docs, on how to send to the registered client or global push. If this is too much to handle, use onesignal.

@Heman6886 please read the documentations [here])(https://github.com/ParsePlatform/parse-server/wiki/Push) to setup push notifications or use an external module like one signal, urban airship or SNS


Unauthorized Access - Xamarin Live Player Beta

I'm getting the following error:
Something went wrong, please contact the application owner: Unauthorized Access
It seems that this is a regular occurrence. Any news on a fix? I twas supposedly fixed 8 days ago, but is happening again.
Vince

Shoot me an email<EMAIL_ADDRESS>


[cmake] Fix cmake options position to support cmake toolchain

Greetings!
By declaring options in the CMake toolchain after the project() command in the CMakeLists.txt, you ensure that they are applied within the context of the project's environment, making them more coherent and ensuring that they override any default settings appropriately.
However, the current CMakeLists.txt declares options() first, before project(), which means, the context is not initialized and the options values from toolchain is ignored.
The current scenario WITHOUT this PR:
# hiredis_toolchain.cmake
set(BUILD_SHARED_LIBS OFF)
set(ENABLE_SSL ON)
set(DISABLE_TESTS ON)
set(ENABLE_SSL_TESTS OFF)
set(ENABLE_EXAMPLES OFF)
set(ENABLE_ASYNC_TESTS OFF)

cmake -S . -B build -DCMAKE_TOOLCHAIN_FILE=hiredis_toolchain.cmake
% cmake -S . -B build -DCMAKE_TOOLCHAIN_FILE=hiredis_toolchain.cmake -LAH
Detected version: 1.2.0
-- The C compiler identification is AppleClang <IP_ADDRESS>00100
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Found OpenSSL<EMAIL_ADDRESS>(found version "3.2.0")
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/uilian/Development/hiredis/build
-- Cache values
// Build shared libraries
BUILD_SHARED_LIBS:BOOL=ON

// The CMake toolchain file
CMAKE_TOOLCHAIN_FILE:FILEPATH=/Users/uilian/Development/hiredis/hiredis_toolchain.cmake

// Enable building hiredis examples
ENABLE_EXAMPLES:BOOL=OFF

// Install NuGET packaging details
ENABLE_NUGET:BOOL=ON

// Build hiredis_ssl for SSL support
ENABLE_SSL:BOOL=OFF

// Should we test SSL connections
ENABLE_SSL_TESTS:BOOL=OFF

As you can see, the toolchain file is consumed, but CMake ignores its values.
In order to fix the current situation, options() should be moved after project().

Merged, thanks!


How to use napari viewer?

@berombau I have tried using napari viewer path/to/data.zarr as described in the docs https://github.com/scverse/napari-spatialdata/blob/6d4cc8dc4f27b676ef50bd4d697e00d5d765db21/docs/cli.md (btw, there were some mistakes, I have corrected them in this commit https://github.com/scverse/napari-spatialdata/commit/6d4cc8dc4f27b676ef50bd4d697e00d5d765db21), but I get this error:
(ome) macbook@MBP-2021 spatialdata-sandbox % napari viewer curio_io/data.zarr
Traceback (most recent call last):
  File "/Users/macbook/miniconda3/envs/ome/bin/napari", line 8, in <module>
    sys.exit(main())
  File "/Users/macbook/miniconda3/envs/ome/lib/python3.10/site-packages/napari/__main__.py", line 561, in main
    _run()
  File "/Users/macbook/miniconda3/envs/ome/lib/python3.10/site-packages/napari/__main__.py", line 341, in _run
    viewer._window._qt_viewer._qt_open(
  File "/Users/macbook/miniconda3/envs/ome/lib/python3.10/site-packages/napari/_qt/qt_viewer.py", line 830, in _qt_open
    self.viewer.open(
  File "/Users/macbook/miniconda3/envs/ome/lib/python3.10/site-packages/napari/components/viewer_model.py", line 1024, in open
    layers = self._open_or_raise_error(
  File "/Users/macbook/miniconda3/envs/ome/lib/python3.10/site-packages/napari/components/viewer_model.py", line 1094, in _open_or_raise_error
    raise NoAvailableReaderError(
napari.errors.reader_errors.NoAvailableReaderError: No plugin found capable of reading viewer.

Do I need to manually register the plugin? Do you know how to fix this?

Thanks @berombau, I wrongly made that change. Could you please update the docs in the linked PR? Thanks


update docs.

Updates Samples, Docs and TSG for Azure Developer CLI Credential and Workload Identity Credential.
Makes parameters for Workload Identity credential optional and read from env if available.

API change check
API changes are not detected in this pull request.


Evolve into Nuxt framework

We have quite big chunks loading upfront and would be great to move to Nuxt as the App is getting bigger and loads slower.
Requirements to keep. Docs

[ ] deployment through Netlify - simple keep it in jam stack as possible as we don't want to hustle servers, serverless is allowed :)

What we will gain?

[x] faster load times
[x] better router?

A lot of work to migrate
just FYI in august there will be public beta Nuxt 3
Q4 - Stable version for production
I wonder if it's possible to migrate to Nuxt 3 since you use a lot of packages which may not work with Vue 3

What if we migrate to Nuxt 2 ?

since you use a lot of packages which may not work with Vue 3

Which ones ? I am aware that Buefy wont work

What if we migrate to Nuxt 2 ?

since you use a lot of packages which may not work with Vue 3

Which ones ? I am aware that Buefy wont work

Didn't check which one's but you will now after creating new project with vue 3 and installing them

A lot of work to migrate

Opportunity for you to get through challenges

just FYI in august there will be public beta Nuxt 3
Q4 - Stable version for production
I wonder if it's possible to migrate to Nuxt 3 since you use a lot of packages which may not work with Vue 3

Yeah, I would stick to Nuxt v2 till now :)
Checking it right now and I would stick with 2.15 https://github.com/nuxt/nuxt.js/releases/tag/v2.15.7
I guess we would have a lot of improvements from that, like speedy load times and easier PR for other parachain teams and technologies there I guess so.

not sure when you will migrate to Nuxt 2 how easy it will be to migrate to Nuxt 3
but for sure it can be beneficial

not sure when you will migrate to Nuxt 2 how easy it will be to migrate to Nuxt 3
The problem is that we would have to rewrite all the components into the composition API.
However, we can use the Nuxt Composition API so the migration from Vue/Nuxt 2 -> 3 would be simplier

A lot of work to migrate just FYI in august there will be public beta Nuxt 3 Q4 - Stable version for production
I wonder if it's possible to migrate to Nuxt 3 since you use a lot of packages which may not work with Vue 3

lol actually fa6 so far works only with vue3

Continue in #1060


Sneaky Throws Not Correctly Detected for Anonymous Inner Class

There is a particular case with an anonymous inner class that the plugin appears not to correctly detect that the @SneakyThrows annotation should handle the thrown exception.
package bugReport;
import lombok.SneakyThrows;
import java.io.IOException;
public class BugReport {
    private IInner inner;

    //@SneakyThrows(IOException.class)
    public BugReport() {
         inner = new IInner() {
             @SneakyThrows(IOException.class)
             @Override
             public IInner doSomething() {
                 throw new IOException();
             }
        };
    }

    interface IInner {
        public IInner doSomething() throws IOException;
    }
}

If you uncomment the sneaky throws, it will compile in Intellij, but it will fail in javac with the following message: ...exception java.io.IOException is never thrown in body of corresponding try statement.

I've reimplemented @SneakyThrow hander to fix this kind of error, but it will work only in Intellij version 14.0 and greater.

Version 0.9.0 of plugin is available now

Looks great! Thanks for the fix.


Port rasa core examples to SDK code

Make sure all examples that are part of rasa core work with the sdk code.
Please also update the documentation of the examples to reflect any changes.

[x] moodbot
[x] restaurantbot
[x] concertbot
[x] remotebot

TODO: get the train_online script working with the SDK (waiting on @tmbo to finish implementing that part)

List of doc changes to be made:

[ ] connectors (may need to write a new function for that that only starts the input channel)
[ ] custom actions - this might be a good place to mention how to run the actions server
[ ] interactive learning - blocked, still needs to be implemented
[ ] domains - custom actions section
[ ] agent (outdated functions)

@akelad will this include a migration guide as well?

It does not yet, but I guess I can add that to the migrations docs file right?

yeah what we learned from last major release is that we need to make an active effort to make migration steps prominent and not just hide them in the changelog

cool, added to the list up there!


/add and /generic can be used to update records

At present the documentation has
/add: 	 	adds a user name and password to the list.
/generic: 	adds generic credentials to the list.
As both options support updating or changing an existing record the wording should be:
/add: 	 	adds a user name and password to the list or updates the user name and/or password for an existing record.
/generic: 	adds generic credentials to the list or updates the generic credentials for an existing record..

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

ID: caebc8e0-4c03-4714-b7a8-3d55ef0f6ab1
Version Independent ID: 5a7b7ad8-8f55-fb3e-54ee-b8aff333b7c7
Content: cmdkey
Content Source: WindowsServerDocs/administration/windows-commands/cmdkey.md
Product: windows-server-threshold
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

#please-close


Validation Finds All Issues in a Pattern and Pattern Instantiation

Feature Request
Description of Problem:
Running the generate and validate commands can detect problems in the schemas, however it fails at the first encountered problem. It would be good to see all the failures so that I can fix my problems in one go.

I think we need to change the instantiation options. Looking at https://github.com/finos-labs/architecture-as-code/blob/478f07d14aa818050245375960f0b7183e51a85b/cli/src/commands/validate/validate.ts#L15
would become
const ajv = new Ajv2020({ strict: false, allErrors: true });

@jpgough-ms I merged a pr to enable the allErrors mode, I tested it locally and it worked. Let me know if you have a chance to try it out


Error fetching the ip of OpenStack VM

The function _GetNetworkIPAddress seems to expect the content of server_dict['addresses'] to be formatted like it's usual cli string output ( "network=ip, network2=ip2" ) but because it requests format=json it receives a dict/list structure like { 'network' => [ ip ], 'network2' => [ ip2 ] } instead.
Current code then fails on line 355: addresses = server_dict['addresses'].split(',')
I guess the handling code to correctly identify the right IP to connect to requires an overhaul to deal with all the possibilities.
As a quick hack, this worked for our case:
def _GetNetworkIPAddress(self, server_dict, network_name):
addresses = server_dict['addresses']
for address in addresses:
if network_name in address:
ip = addresses[network_name][0]
return ip
If the VM has multiple IPs in that network, it will return the first.
I will not put this into a pull request since this is a really crappy monkeypatch but if anyone runs into the same trouble maybe this helps you as well. Or maybe your datacenter burns down, so use this at your own risk.

@keerockl has most recently updated the OpenStack provider and may have additional thoughts?

Is there any progress this issue ?


Typo in README

Further to my PR, I thought I would try submitting an issue.
Line 40 -> should be GitHub
😄 😇

Great eye! Thanks for the catch!


Cannot download "https://github.com/sass/node-sass/releases/download/v4.9.4/linux-x64-67_binding.node":

npm i -D postcss @csstools/postcss-sass

Cannot download "https://github.com/sass/node-sass/releases/download/v4.9.4/linux-x64-67_binding.node":

HTTP error 404 Not Found

The link is indeed false, linux-x64-64_binding.node is the last available package.

Update: had to downgrade to node 10.x from node 11.x

@coldtelling, so is the issue with node-sass or exclusively this plugin?

Hi,
I'd have to upgrade again and check ! Might be node-sass https://github.com/sass/node-sass/issues/2521

Let me know if that worked. Most of the time when people don’t report back, it’s because things are working again, and they must resume their daily work. :)


Recipe

Recipe not saving after restarting server in your old pm5 Invcraft version
InvCraft-pm5.zip
here it is ....

I'm no longer working on old pm5 version also saving doesnt work in pm5 port from original pm4 version...
i will close this after rewrite version finished

ohk

Fixed in rewrite version !


Compile to JavaScript

To avoid the run-time dependency on CoffeeScript, we should compile this to JS at build-time.
We can base it on the Makefile from hyperactive.

Fixed by #7.


[Clarification] Provide Xquery file

** What needs to be clarified **
Provide the contents of the Xquery file that generates the xml output with the raw data from the text.
** Page(s) which need to be clarified (Optional) **
Start with the said post

Provide as a download at the top of the data dictionary section?

Decide to provide Xquery snippets in the said post, since without the actual TEI file the script is useless.
Figured explaining what I'm doing is more important than providing a file no one can use.


makedb.py needs updating

C:\Users\Luke\Desktop\NabBot>nabbot.py
Logged in as
NabBot
182466599121453056
2016-05-18 16:10:40,056:INFO: Bot is online and ready
Ignoring exception in on_ready
Traceback (most recent call last):
File "C:\Python35\lib\site-packages\discord\client.py", line 244, in _run_event
yield from getattr(self, event)(*args, **kwargs)
File "C:\Users\Luke\Desktop\NabBot\nabbot.py", line 28, in on_ready
yield from think()
File "C:\Users\Luke\Desktop\NabBot\nabbot.py", line 102, in think
c.execute("SELECT name, last_level, id FROM chars WHERE name LIKE ?",(serverChar['name'],))
sqlite3.OperationalError: no such table: chars
As the title says, with the new version I'm pretty sure either makedb.py needs an update or the database.py does.

I just updated makedb.py in commit 64f122c23202a8f019df4e054151ed98679e0176
I'm planning on removing makedb.py and instead checking if the users database exists (and is updated) on startup.
Thanks


Pex not working

So i have pex 2.0 installed on my server, it works i have everything set up. When i install Nucleus and try and run /pex only the type /help for more help comes up. What is going on?

This has nothing to do with GP. Also PEX is a dead plugin, migrate to LuckPerms.


Bad look and feel for labels after changing font

I have set Monaco as main font and now I get this on canvas:
http://wstaw.org/m/2014/05/12/plasma-desktopZQ3100.png

fixed


Experiment: MEF -> iPSCs testing

Premise: the MEF -> iPSC transition is a well-established and consistent change in cell state. It should be easy to recapitulate using pathway-based latent space analysis through autoPLIER.
Approach: Find MEF -> iPSC datasets and combine / reprocess them. Then we use them to build a latent space -- does this recapitulate expected biology? This can be tested by seeing whether expected pathways such as "WONG_STEM_CELL_CORE" change across those timepoints.
Additional: Should we also use this for classification? Should we also build a custom latent space of only pluripotency-related pathways? @mbernste @dmontemayor @Julian-Borbeck

Transferred - https://github.com/Bishop-Laboratory/Cancer-Normal-Transformer-Analysis/issues/26


Check apiVersion numbers

We should error out if the API version is not supported.

This is fixed, assuming the issue refers to the k8s API version.


Add port_midi (C bindings for cross-platform PortMidi library)

No, I have not. If they have a free tier, I'll give it a try as soon as I have a chance and will update my project.

@hugoabonizio I just tried using CircleCI to run the build, and got the exact same problem with the libportmidi-dev package build.

Thank you!


Are there anyways to Disable Wow64Redirection when use IFileOperation interface?

I get into problem when trying to copy some x64 DLL into system32 from x86 process. Are there anyways to disable Wow64Redirection when use IFileOperation interface? I tried Wow64DisableWow64FsRedirection but it didn't work.
Thank you!

This is clearly not an issue of this repository.


write ros time to message indexes, clarify docstrings

Changelog

Changed: message indexes written using the go-rosbag writer will have the correct timestamps in them.

Docs
Updated docstrings to clarify that the timestamps one should expect in message index records are ros timestamps.

cc. @wkalt


Incompatiblity with python-fastapi 0.89.0

System OS
Linux
Python Version
3.10 (CPython)
Install Source
Arch User Repository (Unofficial)
Install version / commit hash
v4.0.6
Expected Behavior vs Actual Behavior
Expected Behavior:
spotdl --version
4.0.6

Actual Behavior:
spotdl --version
Traceback (most recent call last):
  File "/usr/sbin/spotdl", line 33, in <module>
    sys.exit(load_entry_point('spotdl==4.0.6', 'console_scripts', 'spotdl')())
  File "/usr/sbin/spotdl", line 25, in importlib_load_entry_point
    return next(matches).load()
  File "/usr/lib/python3.10/importlib/metadata/__init__.py", line 171, in load
    module = import_module(match.group('module'))
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/usr/lib/python3.10/site-packages/spotdl/__init__.py", line 14, in <module>
    from spotdl.console import console_entry_point
  File "/usr/lib/python3.10/site-packages/spotdl/console/__init__.py", line 5, in <module>
    from spotdl.console.entry_point import console_entry_point
  File "/usr/lib/python3.10/site-packages/spotdl/console/entry_point.py", line 19, in <module>
    from spotdl.console.web import web
  File "/usr/lib/python3.10/site-packages/spotdl/console/web.py", line 20, in <module>
    from spotdl.utils.web import (
  File "/usr/lib/python3.10/site-packages/spotdl/utils/web.py", line 215, in <module>
    def song_from_url(url: str) -> Song:
  File "/usr/lib/python3.10/site-packages/fastapi/routing.py", line 633, in decorator
    self.add_api_route(
  File "/usr/lib/python3.10/site-packages/fastapi/routing.py", line 572, in add_api_route
    route = route_class(
  File "/usr/lib/python3.10/site-packages/fastapi/routing.py", line 400, in __init__
    self.response_field = create_response_field(
  File "/usr/lib/python3.10/site-packages/fastapi/utils.py", line 90, in create_response_field
    raise fastapi.exceptions.FastAPIError(
fastapi.exceptions.FastAPIError: Invalid args for response field! Hint: check that <class 'spotdl.types.song.Song'> is a valid pydantic field type

Steps to reproduce - Ensure to include actual links!
I'm maintainer of python-spotdl package on AUR and building the package happens every night to make sure that it can get built successfully, this is the log of two days ago which was fine:
https://github.com/aminvakil/aur/actions/runs/3859937506/jobs/6579911735#step:11:162
https://github.com/aminvakil/aur/actions/runs/3859937506/jobs/6579911735#step:12:10
And last night which failed:
https://github.com/aminvakil/aur/actions/runs/3859937506/jobs/6579911735#step:11:162
https://github.com/aminvakil/aur/actions/runs/3864959715/jobs/6588136699#step:12:10
Package gets built correctly, but a test gets run after makepkg to make sure spotdl works correctly, it just checks the error code.
These two commands get executed:
spotdl --version
spotdl download https://open.spotify.com/track/7kriFJLY2KOhw5en9iI2jb

Since last night which fastapi has been updated to 0.89.0, spotdl --version fails. It may be related to another issue, but the error seems to be related to fastapi upgrade.
Traceback
Traceback (most recent call last):
  File "/usr/sbin/spotdl", line 33, in <module>
    sys.exit(load_entry_point('spotdl==4.0.6', 'console_scripts', 'spotdl')())
  File "/usr/sbin/spotdl", line 25, in importlib_load_entry_point
    return next(matches).load()
  File "/usr/lib/python3.10/importlib/metadata/__init__.py", line 171, in load
    module = import_module(match.group('module'))
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/usr/lib/python3.10/site-packages/spotdl/__init__.py", line 14, in <module>
    from spotdl.console import console_entry_point
  File "/usr/lib/python3.10/site-packages/spotdl/console/__init__.py", line 5, in <module>
    from spotdl.console.entry_point import console_entry_point
  File "/usr/lib/python3.10/site-packages/spotdl/console/entry_point.py", line 19, in <module>
    from spotdl.console.web import web
  File "/usr/lib/python3.10/site-packages/spotdl/console/web.py", line 20, in <module>
    from spotdl.utils.web import (
  File "/usr/lib/python3.10/site-packages/spotdl/utils/web.py", line 215, in <module>
    def song_from_url(url: str) -> Song:
  File "/usr/lib/python3.10/site-packages/fastapi/routing.py", line 633, in decorator
    self.add_api_route(
  File "/usr/lib/python3.10/site-packages/fastapi/routing.py", line 572, in add_api_route
    route = route_class(
  File "/usr/lib/python3.10/site-packages/fastapi/routing.py", line 400, in __init__
    self.response_field = create_response_field(
  File "/usr/lib/python3.10/site-packages/fastapi/utils.py", line 90, in create_response_field
    raise fastapi.exceptions.FastAPIError(
fastapi.exceptions.FastAPIError: Invalid args for response field! Hint: check that <class 'spotdl.types.song.Song'> is a valid pydantic field type

Other details
No response

New FastAPI version breaks spotdl, updating spotdl to new support new FastAPI would require rewriting all types and I am too busy to do this right now. Can you force the makepkg to use FastAPI < 0.89?

@xnetcat Arch mirrors only have the latest version of each package, so the only available python-fastapi package is 0.89 right now.
https://archlinux.org/packages/community/any/python-fastapi/

Should I release a hot fix with fastapi = "<0.89.0"?

@xnetcat That may help users installing app from other sources, but package won't run in Arch anyway.

i recommend running the package under pipx for now.

@CheesyChocolate Sure, but having a package which installs its dependencies using pip is against arch packaging guidelines.
I'll let the package to be broken until it's compatible with new fastapi.

I'll let the package to be broken until it's compatible with new fastapi.

I will be busy next week, so expect an update before February. But if someone is interested in working on this feel free to do so : )

I've release new version that fixes Incompatibility with fastapi 0.89.0
PIP users just have to update to v4.0.7 of spotdl
Arch users have to wait for a new release of FastAPI with a patch that fixes a bug with return annotations
https://github.com/tiangolo/fastapi/commit/fba7493042cde54c059989ffb2ccccde65c51293

This workaround works for the FastAPI bug:
diff --git a/spotdl/utils/web.py b/spotdl/utils/web.py
index 8cbde8c..9ae10d4 100644
--- a/spotdl/utils/web.py
+++ b/spotdl/utils/web.py
@@ -310,7 +310,7 @@ async def download_url(

 @router.get("/api/download/file")
-async def download_file(file: str, client_id: str) -> FileResponse:
+async def download_file(file: str, client_id: str):
     """
     Download file using path.

Tested on Arch Linux with FastAPI 0.89.0 and patched spotdl 4.0.7

new fastapi version was released it fixed your issues

I have flagged https://archlinux.org/packages/community/any/python-fastapi/ out-of-date, when it was upgraded on arch, I will upgrade aur package to spotdl 4.0.7.

https://archlinux.org/packages/community/any/python-fastapi/ has been updated, waiting for green CI on https://github.com/aminvakil/aur/pull/217 .


Rj development

I updated our UML Diagram and created three Class Files.  HumanPlayer and ComputerPlayer will both extend a higher level Player class.
I added some logic and fleshed out HumanPlayer with validation for name entry and move selection.
ComputerPlayer needs some work, but the name is just hard-coded to "Jay" as it's name.
I added a test file with some comments going over why they aren't passed tests...

I built out the Computer Move class on Saturday Night which includes the random move selector.


The fasta file is wrong, please check the parameters

Description:
I'm encountering an issue when running the following command:
EVG -r /BIGDATA2/scau_jyang_2/genome/Homo_sapiens.GRCh38.dna.toplevel.fa -v /BIGDATA2/scau_jyang_2/genome/pav-panel-freeze3.vcf.gz -s /BIGDATA2/scau_jyang_2/genome/sample.txt --mode  precise --threads 23 --jobs 3 --force &> /BIGDATA2/scau_jyang_2/genome/log.txt
The error message I'm receiving is:
[2023-10-07 21:47:42,175] [EVG.fasta_count] CMD: /BIGDATA2/scau_jyang_2/bio_software/EVG/fastAQ count -i /BIGDATA2/scau_jyang_2/genome/Homo_sapiens.GRCh38.dna.toplevel.fa
[EVG.fasta_count] The fasta file is wrong, please check the parameters.
However, when I run this command separately, it works:
/BIGDATA2/scau_jyang_2/bio_software/EVG/fastAQ count -i /BIGDATA2/scau_jyang_2/genome/Homo_sapiens.GRCh38.dna.toplevel.fa
[main::2023-10-07 21:49:27] You are using fastAQ (v1.0.6)
[main_count::2023-10-07 21:49:27] Running ...
readBase:3291585349
readNum:706
readLen:4662302
[main_count::2023-10-07 21:49:36] Done ...
I'm using Python version 3.9. My Python environment details are as follows:
python3: /BIGDATA2/scau_jyang_2/.conda/envs/evg_env/bin/python3.9-config /BIGDATA2/scau_jyang_2/.conda/envs/evg_env/bin/python3 /BIGDATA2/scau_jyang_2/.conda/envs/evg_env/bin/python3.9 /BIGDATA1/app/anaconda3/2020.07/bin/python3.8 /BIGDATA1/app/anaconda3/2020.07/bin/python3 /BIGDATA1/app/anaconda3/2020.07/bin/python3.8-config
I am confused by this issue and would appreciate any guidance on how to resolve it.

Python may not be able to read the result file of fastAQ count because EVG outputs the generated file in /tmp or /var/tmp, which may have restricted access permissions. You can check if you have permission to access these directories.
To modify the default output path for temporary files, you can use the following commands:
tmpn=`mktemp -u XXXXX`
tmpd="${tmpn}"
mkdir ${tmpd}
echo ${tmpd}
export TMP=${tmpd}

By executing these commands, you can set a different temporary file directory. This might resolve the issue you are facing.
Please try this solution, and if it indeed solves the problem, we will fix this bug in the next version.
Thank you very much for using our software and bringing up this issue.

Alternatively, you can try running the test data to see if it works.
If it's not due to temporary directory issues, you can send us the reference genome for testing.
Best wishes to you!

Thank you for your prompt response. I tried your suggestion, but unfortunately, it didn't resolve the issue. However, I managed to resolve the error by reinstalling the EVG software.
Now, when I attempt to run the test data using the following command:
#!/bin/bash

# Job name:
#SBATCH --job-name=EVG.sbatch
# Number of nodes needed for use case:
#SBATCH --nodes=1
#

# Processors per task:
#SBATCH --cpus-per-task=24
#
# Memory per node:
#SBATCH --mem=62G
#SBATCH --output=EVG.out
#SBATCH --error=EVG.err

source /BIGDATA2/scau_jyang_1/software/evg_env/bin/activate
module load gcc/7.4.0
cd /BIGDATA2/scau_jyang_1/software/EVG-main/test
EVG -r test.fa -v test.vcf.gz -s sample.txt --software VG-MAP VG-Giraffe GraphAligner Paragraph BayesTyper GraphTyper2 PanGenie --force  &>log.txt &
wait

I encounter the following error:
Files and parameters used:
-a	0
-c	0
-d	0
-e<PHONE_NUMBER>
-g	0
-i	/BIGDATA2/scau_jyang_1/software/EVG-main/test/genotype/PanGenie/test1/test1.fq
-j	10
-k	31
-o	test1
-p	0
-r	/BIGDATA2/scau_jyang_1/software/EVG-main/test/convert/convert.test.fa
-s	test1
-t	10
-u	0
-v	/BIGDATA2/scau_jyang_1/software/EVG-main/test/genotype/PanGenie/test1/test1.vcf
Determine allele sequences ...
Found 1 chromosome(s) from the reference file.
Identified 769 variants in total from VCF-file.
#### Memory usage until now: 0.009004 GB ####
Write path segments to file: test1_path_segments.fasta ...
Found 1 chromosome(s) in the VCF.
#### Memory usage until now: 0.009004 GB ####
Count kmers in reads ...
Histogram peak: 1 (3323)
Computed kmer abundance peak: 1
Count kmers in genome ...
terminate called after throwing an instance of 'jellyfish::large_hash::array_base<jellyfish::mer_dna_ns::mer_base_static<unsigned long, 0>, unsigned long, atomic::gcc, jellyfish::large_hash::unbounded_array<jellyfish::mer_dna_ns::mer_base_static<unsigned long, 0>, unsigned long, atomic::gcc, allocators::mmap> >::ErrorAllocation'
  what():  Failed to allocate<PHONE_NUMBER>8 bytes of memory.

The full information is available in the EVG.log:
EVG.log.txt
It appears that there is a memory overflow when running PanGenie with a single command in EVG pepline.
When I run PanGenie separately using the command:
 PanGenie -i <(zcat ${input}/test1.sim1.fq.gz ${input}/test1.sim2.fq.gz) -v /BIGDATA2/scau_jyang_1/software/EVG-main/test/genotype/PanGenie/test1/test1.vcf -r ${input}/test.fa -s test1 -j 23 -t 23

It works without any issues. I have also included the PanGenie log file:
PanGenie.log
#### Memory usage until now: 48.4737 GB ####
#### Memory usage until now: 48.4737 GB ####
Sampled 1 subset(s) of paths each of size 3 for genotyping.
#### Memory usage until now: 48.4737 GB ####
Construct HMM and run core algorithm ...
Warning: using 1 for genotyping.
Write results to VCF ...

###### Summary ######
time spent reading input files: 0.172345 sec
time spent counting kmers:      72.0957 sec
time spent selecting paths:     0.00323378 sec
time spent determining unique kmers:    0.25496 sec
time spent genotyping chromosome chr1:  0.304912
total running time:     72.9184 sec
total wallclock time: 72.6206 sec
Total maximum memory usage: 48.4737 GB

I suspect there is a 62GB memory limit, and running the EVG process takes up a portion of it. This leads to insufficient memory when running the PanGenie module. After all, even when running PanGenie separately without processing the reference, it still requires 48GB of memory？
Finally, please forgive me for closing the previous question without expressing my gratitude. I'm truly sorry, and I want to take this opportunity to say thank you once again.

Glad to know you resolved the error.
Your suggestions regarding the PanGenie task submission logic were truly great. We have improved it in v1.0.9.
However, based on our test data, the peak memory usage was only 24GB. Therefore, we are not sure why you encountered insufficient memory. You could try using "--jobs 1" to limit the parallelization of the software to one job, which might reduce memory consumption.
Thank you again for your valuable feedback.

Thank you for updating EVG. However, we noticed that after installation, the version of the PanGenie software is still not the latest, meaning it lacks the '-f' parameter.

Yes, since the highest version of PanGenie available on conda is 2.1.0, you need to remove the existing PanGenie package from your conda environment and manually compile the latest version of PanGenie. Furthermore, we have just released version 1.1.0, which fixes the issue with PanGenie index path generation.

Thank you for your prompt response. I will install the updated version you provided. If there are any further issues, I will provide feedback promptly.

Thank you for your feedback and suggestions!


Fix Extraneous '*' on landing page #286

Fix proposal for https://github.com/projectkudu/AzureFunctions/issues/286

I like the idea of getting rid of the '*', which everyone finds confusing. @mattchenderson are you ok with this?

@snobu btw we're working off the dev branch now. The dev branch is hooked up to https://functions-next.azure.com for CI. The site isn't functional yet because of #299 but should be pretty soon. So whenever you merge this, merge it to dev not master

also maybe <sup>†</sup> instead of <strong>? I don't have a preference either way. It's up to @davidebbo and @mattchenderson

I'd say drop the bold and treat as normal text.
Also, the screen above shows the message in the wrong place on screen ("new or existing" version should be under the "your function apps" heading). The commits don't show any layout changes though, so I think we're okay. @snobu can you confirm the latest rendered screen?

Here it is.

This is targeting master. Should i close it and open a new PR or you can rebase to dev branch?

Awesome - thanks! This has my sign-off.
I'll let @ahmelsayed comment on how to get things into dev branch.

@snobu Feel free to merge it to dev when you're ready! Thanks for taking care of this! :)

Merged to dev in https://github.com/projectkudu/AzureFunctionsPortal/commit/fa5901ffd99118186a79df83fe0ab023d1672808

Thanks! It should be live on https://functions-next.azure.com/