[WIP] Setup CodeDocs: Online doxygen-based API documentation publisher

Before creating a pull request

[x] Document new methods and classes (N/A)
[x] Format code with make format (N/A)

Before merging a pull request

[x] Set version target by selecting a milestone on the right side (N/A)
[x] Summarize this change in CHANGELOG.md (N/A)
[x] Add unit test(s) for this change (N/A)

It seems that this hosts the documentation at https://codedocs.xyz/personalrobotics/aikido/.
Would it make more sense to either (a) host it on GitHub Pages or (b) host it on the lab website?

AFAIK, both ways should be done manually, but CodeDocs is automated. It's updated per commit. The downside of using CodeDocs is that it always build for master branch.
I think we could (1) use CodeDocs for automated API documentation generation for master branch and (2) upload manually API documentation for released AIKIDO versions.

I'm not sure I follow the reasoning here.
In general, I agree with @brianhou.  Depending on an externally built and hosted documentation system is not desirable because it constrains you to their quality of service.  I previously hosted some documentation on ReadTheDocs for another project and ended up having horrible issues where the docs were slow to load, didn't build for days, or built inconsistently (had errors which I could not debug because I got different results when building locally).
https://github.com/personalrobotics/aikido/pull/245#issuecomment-338044465

AFAIK, both ways should be done manually, but CodeDocs is automated. It's updated per commit. The downside of using CodeDocs is that it always build for master branch.

I'm not sure I understand why this is somehow specific to CodeDocs.
One very common approach is simply to put the documentation as a build step in TravisCI (or any equivalent CI) and simply give it a deploy key to your gh-pages or a server location.  I have done this before with great success.
It is also pretty easy to use build variables in CI to prefix the installation path of the output HTML artifacts (e.g. master goes to https://personalrobotics.github.io/aikido/docs/master) and constrain documentation builds to only run on select branches (e.g. master` and named tags).

TL:DR: I would highly recommend using a doc generator that can be run locally to produce static HTML, then use manual or conventional CI processes to put the static HTML artifact on a web server of your choosing.  GitHub Pages works really well for this, and is also an intuitive place for people to look for documentation.
This gives us the most flexibility (if we decide to change generators), reliability (we can manually build, upload, and correct issues if needed), and customizability (we can control which documentation gets built and retained)

@psigen I totally agree with you. CodeDocs wouldn't be the best solution as a long run solution. This is a stopgap solution with minimal effort.
I'm okay not to merge this, but I think we could start with this temporary solution that doesn't hurt the current codebase much then move on a better solution like you suggested. I would gratefully accept it once the setup is done. 😎

Closing in favor of @psigen 's suggestion.


Add OceanBase to the list of supported metadata databases

This PR updates the JuiceFS website content to include OceanBase as one of the supported metadata storage databases. OceanBase is a high-performance, distributed relational database known for its strong consistency and scalability, making it an excellent choice for managing metadata in large-scale storage scenarios. By highlighting OceanBase, this update provides users with more options for optimizing JuiceFS performance in demanding environments.
This PR contains Chinese and English parts, and every part have 15 articles and 1 figure modified. We have already conducted experiments and verified how to deploy OceanBase locally as a metadata repository.
At the same time, there may be several more articles which need to be modified later:

jfs_vs_seaweedfs.md
In this article, it compares JuiceFS and SeaweedFS.
As described in this article, SeaweedFS has an advantage that supports file system data replication between multiple clusters. It offers two asynchronous data replication modes: Active-Active and Active-Passive. Theoretically, after using OceanBase, users can achieve Active-Active mode, and help JuiceFS make up for this missing mode. But this part needs to be verified by experiments. After that, we will add JuiceFS&OceanBase support the Active-Active mode into this article.

for_distributed.md
In this article, JuiceFS community mentioned that “A detailed guide on database selection will be available soon”. If the community needs Oceanbase architecture or performance testing materials when writing this article, we can provide them at any time.

metadata_engine_benchmark.md
This article compares the performance of different databases. Once the development of the JuiceFS & OBKV API is complete, could the JuiceFS community conduct a performance test using OceanBase and OBKV to evaluate their capabilities?

Since OceanBase is compatible to MySQL protocol, it could be a scalable replacement for MySQL. So people can use it with JuiceFS without this PR. At most, we can mention OceanBase in the MySQL section, maybe also other rational databases.
Btw, please don't let the docs to be a fight field of databases.

Since OceanBase is compatible to MySQL protocol, it could be a scalable replacement for MySQL. So people can use it with JuiceFS without this PR. At most, we can mention OceanBase in the MySQL section, maybe also other rational databases.
Btw, please don't let the docs to be a fight field of databases.

Thank you for the feedback! I understand the importance of keeping the documentation neutral and concise, and I appreciate your thoughts on maintaining that balance.

Regarding the changes I made, I’d like to clarify that while the number of modified files may seem large, the actual changes in each document are minimal. Most of these modifications involve adding small, relevant mentions of OceanBase in both the English and Chinese sections, which naturally doubles the appearance of changes. I also wana clarify why mentioning OceanBase in more detail could bring additional value to JuiceFS users. While it’s true that OceanBase is compatible with the MySQL protocol, it is not just a MySQL-compatible database. OceanBase is a highly scalable and distributed relational database with advanced features like Active-Active clustering, which allows seamless cross-region high availability. This capability addresses a significant gap for JuiceFS in scenarios that require robust disaster recovery or data replication across zones—something not inherently available with MySQL.

I assure you that my intent is not to create competition among databases but to enhance the documentation’s technical depth and showcase how OceanBase can address specific use cases for JuiceFS users. If necessary, I’m happy to adjust the tone or further simplify the mentions of OceanBase to make them more aligned with the community’s expectations.

Please let me know how we can move forward. I’m open to feedback and eager to collaborate to improve the documentation.

PRs like these are useless, othwerwise every compatible provider build on top of another one should be mentioned...
These are just "growth hacking" techniques in my opinion.

PRs like these are useless, othwerwise every compatible provider built on top of another one (or every fork) should be mentioned...
These are just "growth hacking" techniques in my opinion.

Thank you for sharing your perspective. I’d like to clarify a few points about OceanBase and its relevance to JuiceFS:

OceanBase and MySQL are fundamentally different types of databases. OceanBase is a distributed relational database, designed to support complex scenarios like Active-Active clusters, which is a key feature that distinguishes it from MySQL. This capability can help JuiceFS fill a gap in supporting such use cases.

If the goal of the JuiceFS community is to streamline the content on the website, perhaps we could consider revisiting how databases are categorized. For instance, PostgreSQL and MariaDB are quite similar to MySQL in their functionality and use cases. One possibility could be to consolidate them under the MySQL section, while retaining MySQL as the representative for standalone databases and OceanBase as the representative for distributed relational databases. This way, the structure remains clean and future support for other distributed databases could be grouped alongside OceanBase.

I believe this approach not only aligns with the community’s goals of reducing redundancy but also highlights the distinct advantages that different types of databases bring to JuiceFS users. I hope this suggestion makes sense, and I’m happy to further discuss or refine this idea!

From JuiceFS's perspective, MySQL and PG are two protocols, we don't care the architect behind that.


yarn package not updated

Current Behavior

** BUILD FAILED **

The following build commands failed:
	PhaseScriptExecution Integrate\ Haul\ with\ React\ Native build/Build/Intermediates/petter.build/Debug-iphonesimulator/petter.build/Script-AD0CE2C91E925489006FC317.sh
(1 failure)

Expected Behavior

Using the lastest verison of haul. I need to copy& paste the haul-integrate.sh from github everytime to solve it
Haul Configuration (webpack.haul.js)

Your Environment

software
version

Haul
1.0.0-beta.2

react-native
0.46.4

node
v6.9.1

yarn
v0.24.5

@grabbou I think we need a new release with this fix https://github.com/callstack-io/haul/commit/870753d1df5f2d6f878584adb95a370bc2b5b169

Going to do it now.


Integration with the mcan

This PR provides trait implementations from mcan-core crate. It enables the safe usage of the mcan crate.
mcan crate provides a HAL for the CAN peripheral embedded into ATSAME5{1, 4}.

I'll try to provide a similar example to the one in https://github.com/atsams-rs/atsamx7x-rust/pull/51 soon.

My only comment would be that the example is a little long and complicated, but I suppose that's just the nature of the beast. Overall, LGTM

My only comment would be that the example is a little long and complicated, but I suppose that's just the nature of the beast. Overall, LGTM

I could probably cut it down to something simpler, I wanted it to be a little more fun, thorough and interactive. Like, listen to messages on interrupt and print them out AND send a message on a button press. It also contains the filter setup which brings the value of an example up. Anyone who wants to use it can drop whatever they deem unnecessary - this is how I thought about it

oh, one more thing, please add the can feature to crates.json for appropriate PACs, so that it is tested.

Done, not sure about these hal_doc_variants though. Are they supposed not to include samE chips? Only samD?

oh, one more thing, please add the can feature to crates.json for appropriate PACs, so that it is tested.

Done, not sure about these hal_doc_variants though. Are they supposed not to include sam_E_ chips? Only sam_D_?

Sounds like a mistake on my part. Feel free to fix here or in a followup, at your discretion.

Something weird going on here. Maybe a previous PR mixed up the features in hal/Cargo.toml?
https://github.com/atsamd-rs/atsamd/actions/runs/4348898859/jobs/7597968312#step:5:86

oh, one more thing, please add the can feature to crates.json for appropriate PACs, so that it is tested.

Done, not sure about these hal_doc_variants though. Are they supposed not to include sam_E_ chips? Only sam_D_?

Sounds like a mistake on my part. Feel free to fix here or in a followup, at your discretion.

#677, feel free to comment what do you think. Maybe I completely misunderstood the idea behind how it worked before.

Something weird going on here. Maybe a previous PR mixed up the features in hal/Cargo.toml? https://github.com/atsamd-rs/atsamd/actions/runs/4348898859/jobs/7597968312#step:5:86 @bradleyharden PTAL
https://github.com/atsamd-rs/atsamd/blob/d8d85b01e11a7a0b21bcf672f241578a5aad5cd0/hal/Cargo.toml#L123

FTR: Covered by #676, already rebased.

What do you think about calling out the example in the atsame54_xpro BSP changelog? I could go either way.

What do you think about calling out the example in the atsame54_xpro BSP changelog? I could go either way.

I wasn't sure about that. I'm not changing the BSP there if one ignores extra feature (forwarded to the HAL)? I guess that's the only thing that is worth being mentioned.


Redirect https://w3id.org/mod and https://w3id.org/mod/xx to relevant files

We need to update the redirects of each version of MOD to redirect to relevant page

[ ] https://w3id.org/mod => https://w3id.org/mod/3.2 (CURRENTLY BEING EDITED)
[ ] https://w3id.org/mod/3.2 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/mod.ttl
[ ] https://w3id.org/mod/3.1 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/mod-v3.1.ttl
[ ] https://w3id.org/mod/3.0 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/mod-v3.0.ttl
[x] https://w3id.org/mod/2.0 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/mod-v2.0_ontology.owl
[x] https://w3id.org/mod/1.4 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/mod-v1.4_ontology.owl
[x] https://w3id.org/mod/1.2 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/mod-v1.2.3.owl
[ ] https://w3id.org/mod/1.1 => TO COME (from @biswanathdutta)
[ ] https://w3id.org/mod/1.0 => TO COME (from @biswanathdutta)

If possible... we need to deal with redirect to HTML page also percontent negocitioan

Hi, @jonquet, we need to update .htaccess file.
We will talk about this matter offline.

ok thanks.
Can you do the others now too (even if file is not yet in version folder, from v3.1)
Also, can you retrieve v1.0 and v1.1 and add them to the version folder ?

After, we will come back to HTML negociation

@biswanathdutta in the v3.2 of MOD I have used w3id everywhere as URIs of the different versions of MODs, so its important that now you make all the versions redirect to the right files. See https://github.com/FAIR-IMPACT/MOD/commit/43a52ed6a19f8da34cf39da2e86d4ead8d35bd75
Can you point to document to edit to anage redirection and content negociation on w3id ?

@jonquet a few suggestions:

Why don't you keep a consistent structure in the repo, so it is possible to automate version redirections?
For example, if you rename these files:

https://w3id.org/mod/1.4 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/mod-v1.4_ontology.owl
https://w3id.org/mod/1.2 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/mod-v1.2.3.owl

to be
https://w3id.org/mod/1.4 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/mod-v1.4.owl
https://w3id.org/mod/1.2 => https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/mod-v1.2.owl

then we can do a redirection that is:
https://w3id.org/mod/$1 -> https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/mod-v$1.owl

If you agree, I can do a PR here with the redirects and also one to w3id.

Good to me.
Please CC me also on the w3id PR so I see where this is ;)

@dgarijo please put me in CC too, so that I am aware of the changes

Sure, I will do 2 PRs: one for this repo with a slight structure changes and another one to w3id pinging you both.
I will do this today

Took the PR related to this. Will also take the one on W3id and we will need to update the readme and check that everything work for past versions.
Then we will practiceon new version in order to release 3.2 ;)

Ok, it's done. However, a caveat:
I made the versions adopt semantic versioning. So every MOD release has a 3 digit, not two. For example:
https://w3id.org/mod/3.0.0 --> https://raw.githubusercontent.com/FAIR-IMPACT/MOD/main/versions/3.0.0/mod-v3.0.0.owl
I did this because some versions like 1.2.3 had 3 digits, but others like 3.0 only had two.

Also the latest version goes to 3.0.0 because that's the latest one in version. If you want a 3.2.0, then create the folder and it would work

I have created a folder for 3.2 (not 3.2.0) with the content that was associated to the release fo 3.2.
In addition I have  rename folders after 2.0 to 2 digits to be sure histociral URIs still work (including in MOD source file themselves). Thinking about it, we are not going to archive (in the version folder and consequently with version URIs) version with 3 digits... its too much details/work.
Will change w3id accordingly if necessary.


duckstation: doesn't create .desktop file

Describe the bug
Duckstation doesn't create a .desktop file, which means that a user that wants to see its icon in the launcher will have to add the .desktop file manually in ~/.local/share/applications.
Steps To Reproduce
Steps to reproduce the behavior:

Add the package duckstation in configuration.nix;
Open a program launcher;
Duckstation is not there.

Expected behavior
The launcher shows Duckstation's icon.
Templates can be found in Duckstation's repo.
Notify maintainers
@guibou

#140435


How to get response body in case of error response?

Summary
I am unable to get the actual response body in case of error codes. I need to know the status code, error message, details etc.
But right now the library raises exception, and we cannot get the response object.
This severely limits the usage of the library.
Or is there a way to get the response body even in case of errors?
For eg: When deleting a user group, I get the exception
500 Server Error: Internal Server Error for url: https://VMANAGE/dataservice/admin/usergroup/g1

Eg: Making an API call when vManage is down
Traceback (most recent call last):
..
  File "/home/user/env/lib/python3.8/site-packages/vmngclient/session.py", line 126, in get
    return vManageResponse(super().get(url, *args, **kwargs))
  File "/home/user/env/lib/python3.8/site-packages/requests/sessions.py", line 600, in get
    return self.request("GET", url, **kwargs)
  File "/home/user/env/lib/python3.8/site-packages/vmngclient/session.py", line 200, in request
    raise error
  File "/home/user/env/lib/python3.8/site-packages/vmngclient/session.py", line 194, in request
    response.raise_for_status()
  File "/home/user/env/lib/python3.8/site-packages/requests/models.py", line 1022, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 503 Server Error: Service Unavailable for url: https://VMANAGE/dataservice/admin/usergroup

In vmngclient.log I see the actual error message, which is what I need.
{'request': {'method': 'DELETE',
             'url': 'https://VMANAGE/dataservice/admin/usergroup/g1',
             'headers': {'User-Agent': 'python-requests/2.28.0',
                         'Accept-Encoding': 'gzip, deflate',
                         'Accept': '*/*',
                         'Connection': 'keep-alive',
                         'Content-Length': '0',
                         'Cookie': 'JSESSIONID=gJ1nnJPs7Oq-a905ZmQPDr8gct743QUHrGoZU8Vw.24248c81-3a5e-45b1-80da-907dc385307c',
                         'x-xsrf-token': '5716E21C61F46510C76FBE1DD48CEEDA7455275A5D2FC53651DF7E5A655293689F606DBC2F2D19090A5FE2A048A6FBED64DA'}},
 'response': {'status': 500,
              'reason': 'Internal Server Error',
              'elapsed-seconds': 0.433,
              'headers': {'cache-control': 'no-cache, no-store, '
                                           'must-revalidate',
                          'x-xss-protection': '1; mode=block',
                          'pragma': 'no-cache',
                          'x-frame-options': 'DENY',
                          'date': 'Wed, 19 Apr 2023 20:34:48 GMT',
                          'vary': 'Accept-Encoding',
                          'strict-transport-security': 'max-age=31536000; '
                                                       'includeSubDomains',
                          'x-content-type-options': 'nosniff',
                          'content-type': 'application/octet-stream',
                          'server': 'svcproxy',
                          'transfer-encoding': 'chunked'},
              'json': {'error': {'message': 'delete users group request failed',
                                 'details': "Failed to delete group 'group80'. "
                                            'It is currently in use.',
                                 'code': 'USERGR0005'}}}}

Logs and external information
See above
Reproduction script
Eg: Try to delete an active user group(group having at least one user)
session.delete('/dataservice/admin/user/g1')

# 500 Server Error: Internal Server Error for url: https://<IP_ADDRESS>:8443/dataservice/admin/usergroup/g1

On what platform(s) does it occur?
Linux
Issue Severity
High: Blocking everything

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/issues/232


Add net8.0 target

Description
Add net8.0 target
.NET 8 preview 1 published, we need .NET 8.0 support somewhere
Fixes # (issue)
Type of change

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

Checklist

[x] I have performed a self-review of my code
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have made corresponding changes to the documentation
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] New and existing unit tests pass locally with my changes
[ ] I did test corresponding changes on Windows
[x] I did test corresponding changes on Linux
[ ] I did test corresponding changes on Mac

Hi @WeihanLi,
Thank you for the PR 💐👌, but isn't it a bit early to support .net8 now? since it is still a preview? I would rather merge this PR after an official release. what do you think?

@alirezanet thanks, yeah, it's a little earlier indeed, is it acceptable for a preview release? so that it could be used by those early adopters, and maybe we could prepare for a stable release when .NET 8 RC or GA version released

Hi @WeihanLi,
For those who need early access to .net8 I also released a preview version of husky.net, 0.6.0-Preview1.
Thank you for your contribution 🎉


How can I use mask property with infinite characters?

I want to implement a mask that accepts only numbers and my input doesn't have a specific length, you can type as many characters as you want so it wouldn't make sense to use mask="_" and a replacement={{ _: /\d/ }}, because it would only allow me 1 character. What can I do?

This component only responds to the mask pattern you specified, you can modify the component's properties at the time of input, but this will not give a good result. Now work is underway to update the library where there will be a component that will meet your desires.


Added Support for Linux thru espeak and mimic3

This is the addition of Linux support.
Lacks tests as discussed.
Closes https://github.com/techygrrrl/timerrr/issues/2
Closes https://github.com/techygrrrl/timerrr/issues/1

Amazing, thank you! 🚀 :shipit:


Wasm: Add support for DefaultConstructorOf and hence Activator.CreateInstance

This PR add support for this intrinsic, enabling Activator.CreateInstance<T>

This is likely hitting the object writer crash #7907 that is hard to repro. I am not able to reproduce locally with the exact same setting as CI. Let's merge and see whether it persists.


[Bug] Executable detected as trojan

Had a few returns on the fact several antivirus analyses take pyinstaller executables as Trojans.
First signature from Windows Defender :
Trojan:Win32/Bearfoos.A!ml
Found out that updating pyinstaller might fix the issue :
https://stackoverflow.com/questions/43777106/program-made-with-pyinstaller-now-seen-as-a-trojan-horse-by-avg

Automate virus checking after build on VirusTotal ?
https://docs.virustotal.com/reference/overview

Current issue comes with charset_normalizer dependency :
UEX-Trader/_internal/charset_normalizer/md.cp311-win_amd64.pyd
The executable is also heavily flagged on Windows :
https://www.virustotal.com/gui/file/327f1a0606955f651bced7defc7aa15ff6585e19f5adabe2fca1909c1320d53c/detection

Well upgrading Python version and dependencies (and pyinstaller) is even worse.. yay!

Implement prospector in CI/CD automated tests.
Apply prospector initial recommendations.

Buy a certificate from a Provider and add following secrets :
CERTIFICATE
PRIVATE_KEY

Then add following variable :
SIGN_EXECUTABLE = 'true'

Check https://signpath.org/ for free implementation on OSS projects.


Fix to load default settings on plugin load instead of conf screen

During Silent install and automated configuration, the LDAP and Local Machine plugin causes an error because the registry keys are not created on plugin load.   Registry keys are only created by poping up the conf screen.  That's a problem with silent/automated scripts used to configure pgina.
That fix load the setting store onload...

Setup pGina configuartion by using the pGina-GUI.
Than export the whole pGina regkey (HKLM\SOFTWARE\pGina3.fork).
Remove any encrypted keys like "notify_pass"
Now you can import those settings on any other computer
If you need encrypted settings to be set, you can use encryptsetting.exe which can be found in the pGina directory.


dashboard: fix regression

introduced by ceph/ceph-ansible/pull/7150
when no rgw is present, it fails.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2076192
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>(cherry picked from commit 1a56fd6a21b3e39c8acd4d3d20451df9b96754d2)

jenkins test centos-non_container-rgw_multisite

jenkins test centos-container-rgw_multisite

jenkins test centos-non_container-purge

jenkins test centos-container-purge


Semver 2.0 metadata fails "nuget pack" with referenced projects

While $(NuGetPackageVersion) replaces the "+" with "-" and allows nuget pack to succeed, if you use -IncludeReferencedProjects nuget because the AssemblyInformationalVersionAttribute value is invalid.
What might be nice is to allow (opt-in, to avoid breaking compat) semver 1.0 versions. We were able to work around this by hooking in our own target, but others may run into this as well.

Can you provide more information about the error you're hitting? AssemblyInformationalVersionAttribute can be anything you like. If NuGet is failing because it can't parse it, please file a bug against NuGet.
The NuGetPackageVersion property is semver 1.0. At least by my understanding it is. Why do you say it is not?

BTW, it sounds like you're not using NuProj. Have you considered using that for your nuget packaging needs?

It's the "+" after the first metadata and before the short commit id. NuGet doesn't completely support it (known issue, mainly because of back-compat issues and mixed binary versions).

The NuGetPackageVersion property doesn't have a + sign in it. That character is changed to a - character in that property so that NuGet can consume it. AssemblyInformationalVersionAttribute retains the + character because any string is allowed in that attribute.
Is the -IncludeReferencedProjects switch causing NuGet to try to parse the string passed to the AssemblyInformationalVersionAttribute?

Yes, it parses that attribute.
On Wed, Jul 13, 2016 at 1:23 PM -0700, "Andrew Arnott"<EMAIL_ADDRESS>wrote:
The NuGetPackageVersion property doesn't have a + sign in it. That character is changed to a - character in that property so that NuGet can consume it. AssemblyInformationalVersionAttribute retains the + character because any string is allowed in that attribute.
Is the -IncludeReferencedProjects switch causing NuGet to try to parse the string passed to the AssemblyInformationalVersionAttribute?

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/AArnott/Nerdbank.GitVersioning/issues/83#issuecomment-232475063

Can you please provide repro steps and/or file a bug against the nuget team?

Outdated.


Load all files from the entry location

Clear and concise description of the problem
As a developer using Vite I want to request all assets/css/js from the entry location so that I'm able to provide a single build from different locations.
Suggested solution
As far as I understand, currently the __vitePreload method handles the dependency preloading, but also the loading of the dependencies.
So my suggestion would be to add a config property like public next to the base property. This can be set to an absolute url like 'https://my-cdn.something.com' or to 'auto'.
At the moment, you define the path inside the preload function like
dep = `${base}${dep}`;

The implementation of the automatic approach using document.currentScript.src could look like
// get values from the vite config:
const { base, public = '' } = viteConfig;

// define the public path automatically
// I guess it should not be part of the preload function to not be called every time
if (public === 'auto') {
  const { src } = document.currentScript;
  public = src.substr(0, src.indexOf(base))
}

// inside the preload function:
dep = `${public}${base}${dep}`

For legacy IE browsers a polyfill needs to be used for the currentScript implementation like https://github.com/amiller-gh/currentScript-polyfill.
Alternative
So fare I don’t have any alternative approach.
Additional context
The context in my case is, that we have multiple environments to test the application that request the files from different endpoints but use the same build. In prod we use a cdn, but on testing environments we have a local folder on the web server.
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Would you check if this PR covers your needs?

https://github.com/vitejs/vite/pull/7644

You would need to set base: './'. We're discussing if adding a new base: 'auto' would make things more clear. There is also a discussion here that may interest you https://github.com/vitejs/vite/issues/3522#issuecomment-1090327274

Thank you @patak-dev. If ./ or auto works as described in the linked issue, I’m totally fine and you might close my request. Thanks for reaching out!

Ok, closing as duplicate of #2009. I'll try to push #7644 for Vite v3 (the next release)


dynamodb attributes are in aws namespace instead db

The span attributes for dynamodb are currently in aws namespace.
Other databases like mondodb or mssql have their special attributes in the db namespace.
At other places like cloud.region vendor namespaces are also replaced/avoided by a generic namespace.
Should the dynamodb attributes moved to db namespace?

I'd vote for the db.* prefix to be given precedence since that seems like the more important grouping from semantic convention perspective.

even if we aren't targeting dynamodb as part of initial database semconv stability, I think we should make this change as part of the initial database semconv stability process so that this breaking change could benefit from being grouped with the other breaking changes related to stabilization

I thought about this some more and changed my opinion.
we should try to stick with the philosophy "try to consolidate all breaking changes into a single bump when going stable"
and so we should defer this change until we are planing dynamodb semconv stabilization (which is not included as part of initial db semconv stability)


feat: support yurt-manager work in specified namespace

What type of PR is this?
/kind feature
What this PR does / why we need it:
support yurt-manager deploy in specified namespace:
support Gateway controller and Yurt-manager webhooks
Which issue(s) this PR fixes:

Fixes parts of #1361
Special notes for your reviewer:

/assign @rambohe-ch
Does this PR introduce a user-facing change?

other Note

It looks like the test use yurt-manager-auto-generated.yaml directly to install components here. If I change webhooks.clientConfig.service.namespace to be rendered through helm intead of hardcode, it cannot be used directly.

/lgtm


Evening shifts only showing up in 1 hour block

Evening shift (8 hour evening) only showing up as one hour blocks on the calendar.

i take this as a personal slight

Easy there big fella, try putting the end date sometime in the next day.


No Markdown plugin installed with "gradle run"

When I follow the instructions at the Mac build and run page to install Beaker Notebook on a Mac (10.11.5), there is no working Markdown plugin. When I try and add it using the Language Manager, there is no item in the list for Markdown. I don't have a URL for a Markdown plugin and could not find any URLs for plugins with some Google searching.
My question is: what is required beyond the steps listed on the Mac build and run page in order to get Markdown working with Beaker Notebook?

it's called "text"

and it's not in the language manager, it's one of the blue buttons when you add cells.


Wrong Logo on GitHub page

If you hover over the codegniter4/ part of the link above the once new and now replaced logo appears.

That's set on the organization: https://github.com/codeigniter4
I think Jim will have to be the one to update that.

Done


networkLatency based on ip4 address

Thank you for this great package.
I'm looking into using networkLatency but unsure if I should be supplying it with my wifi ip4 address or leave it blank to get my wifi network latency.
OS version: Windows 10 64 bit OS
systeminformation version: 5.9.4
Here is an example of my ip4 from networkInterfaces.
networkInterfaces:
 iface: 'Wi-Fi',
    ifaceName: 'Intel(R) Wi-Fi 6,
    ip4: '<IP_ADDRESS>',
    type: 'wireless',
    speed: 758.35,
    dhcp: 'true',
    carrierChanges: 0
    .....

wifiConnections
    id: '........,
    iface: 'Wi-Fi',
    model: 'Intel(R) Wi-Fi 6,
    ssid: 'my-wifi',
    bssid: '2.....',
    channel: 157,
    frequency: 5785,
    signalLevel: '85%',
    txRate: 650

second question:
networkInterfaces()- speed will show you the the signed speed of a certain interface
What is the meaning of signed speed above ?
I'm looking for the actual speed of connected wifi . As in what I would get if I was to run a speed test.
Would wifiConnections()- txRate be what I'm looking for ?

@chapster11 ...
First a short answer to networkInterfaces() and wifiConnections()
sorry .. I mean "designed" (max) speed ... Your interface can then go up to this speed but in case of a WIFI interface the actual connection speed (wifiConnections/ txRate) depend on the router, positioning, etc...
About network inetLatency() ... this function is NOT about network interfaces. It checks your internet latency by trying to connect to an outside host on the internet ... default is googles public DNS <IP_ADDRESS> and you can change it to your needs.

@chapster11 can we close this issue?

Yes I'm closing this ticket. Thanks for the help.


Possible edge case in decoder when very first OP is QOI_OP_RUN

There's an edge-case that currently is handled differently by various decoders. At a first glance it looks like this decoder may not be spec-conformant. The spec is not clearly worded in this case, but I have decided to interpret it in a particular way now. Sorry about that!
Please see the original issue here https://github.com/phoboslab/qoi/issues/258 for the details.

Thank you! That's first class standard stewardship !


fix : add goreleaser CI for latest build and remove v from optimus versioning

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 78.474%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7951

Relevant Lines:
10132

💛  - Coveralls


Fix temporary version tag issues

What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm


Generation of Enviromental Objects

Description
Task: Generation of Environmental Objects
Feature: Landscape Objects (#6)
The task is to dynamically generate and place environmental objects/hazards such as cobwebs, artefacts, rocks and trees onto the map without colliding with other buildings or already placed environmental objects. The generation of environmental objects will occur on a new game start or once a night cycle has been completed, allowing the player to collect additional resources but also giving variety to the map and making it feel more alive. The objects placed will affect the map in multiple different ways depending on the object itself with the number of objects being placed dependent on map size and current water level. The objects will be placed at either grid position which is dependent on Team 3 or by x,y coordinates and then converted to grid positions at a later point in time.
Example

New Game: On a new game start, the map will contain many trees, rocks and a few possible artefacts to be used and gathered from. This will be generated during the loading procedure of the game
New Day: Once a night has ended, a few new trees/rocks can be spawned at random amounts and positions to give the player additional resources. Artefacts can also be spawned but at a small percent chance

Dependencies

Environmental Objects to be placed (Cobwebs, Rocks, Trees, etc)
Grid Positions for objects to be placed on

Milestones
List of steps that need to be completed for this task.
Step 1 (Aug. 19)

Generation of positions on map either as X,Y cords or Grid Locations

Step 2 (Aug. 22)

Objects will be placed on and visible on map ignoring collision

Step 3 (Aug. 24)

Objects will follow collision rules and not spawn into buildings or other environmental objects

Step 4 (Aug. 25)

Bug Checks and Documentation of usage

Completion Deadline: Aug. 26
Documentation
(To be completed as development proceeds)

Main description of feature
JavaDoc

Member

Logan De Groot (Logan-De-Groot) (Logan De Groot)

Completed as of 26/08/2022. Documentation in process.

Reopened due to bug. Objects will spawn on grass/buildable tiles from #66 but due to the mismatch from worldPos to Tile cords this will visually spawn objects over water.

Bug has been fixed. Incorrect tile sizes were used in the design of the game.  The world assumed tiles were square in size but the ones implemented are rectangular with width = 2*y.


ATM banking project c++

Describe the project you want to add with tech stack
The program initializes the account balance to $5000.
A while loop runs continuously to display the menu and accept user choices.
The user can choose to check their balance, make a deposit, make a withdrawal, or exit the program.
Depending on the user's choice, the program performs the corresponding operation and updates the account balance.
If the user chooses to exit, the program terminates.

@pranjay-poddar @MohitGupta121 please assign me this issue


Adjust @safe-ty of 64-bit real unittest.

Fixes build failure when building testsuite on ARM.

If we can't fix the underlying problem, maybe we could at least split the unittest so that some of the @safe part can still be alive?

@wilzbach - all asserts use strtod(), so I don't consider splitting really necessary.


1.3: Support new default behaviors for @format and @scope

Proposal-13107

Closing as duplicate of #3052


Add calculation of density of states

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

9 of 9 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
623

Relevant Lines:
623

💛  - Coveralls


Handle events on NumPad when NumLock is disabled

Hello, I tried to fix the issue #1048.
I mapped the NumPad keys as follow when both SHIFT and NUM aren't set:

SDLK_KP_6 --> AKEYCODE_DPAD_RIGHT
SDLK_KP_4 --> AKEYCODE_DPAD_LEFT
SDLK_KP_2 --> AKEYCODE_DPAD_DOWN
SDLK_KP_8 --> AKEYCODE_DPAD_UP
SDLK_KP_7 --> AKEYCODE_MOVE_HOME
SDLK_KP_1 --> AKEYCODE_MOVE_END
SDLK_KP_3 --> AKEYCODE_PAGE_DOWN
SDLK_KP_9 --> AKEYCODE_PAGE_UP
SDLK_KP_0 --> AKEYCODE_INSERT
SDLK_KP_PERIOD --> AKEYCODE_FORWARD_DEL

Hi,
Why the condition including not shift?
Thank you.

Hi, I tried to mimic the behavior of the keyboard. If you disable the NumLock, and try the NumPad numbers with SHIFT, it'll work as the NumLock was enabled.

Hi, I tried to mimic the behavior of the keyboard. If you disable the NumLock, and try the NumPad numbers with SHIFT, it'll work as the NumLock was enabled.
EDIT: At least, on Ubuntu system

I try it on my windows 10 ver 1909, the shift + numpad when num lock off, it works like shift + arrow, insert, home, etc.

Should we go for Windows behavior? or the Ubuntu's one?
Thanks

Should we go for Windows behavior? or the Ubuntu's one?
Thanks

The behavior problem should contact @rom1v .
In my opinion, we can use different behavior in windows and ubuntu.
Thank you.

Thank you for your contribution (and sorry for the delay).
I think checking SHIFT like you do produces the correct behavior: if shift is enabled, a text event will be generated, and managed as a text, so we must absolutely not send key events in that case.
IMO, the block should be moved few lines above so that it works even if --prefer-text is passed.
I squashed your commits, rebased on dev, and applied the changes I suggest: cd69eb4a4fecf8167208399def4ef536b59c9d22 (I also reordered the lines to sort by number, but it's just personal preference)
Please review and if it's ok I'll merge into dev.

Thank you for your answer. Since it is one of my first contributions, yes everything is fine for me.

it is one of my first contributions

Congrats :tada:
Merged in dev. It will be in the next release.


Move repository to traverson org

As discussed via email - if you want to move these repo to the traverson GitHub org, this are the steps to follow:

Go to https://github.com/punkstarman/traverson-traversal
Click on Settings (upper right corner)
Scroll down to the section titled "Danger Zone"
Click "Transfer"
Enter traverson-traversal in the first input field and traverson in the second

Btw, I linked to this project in Traverson's user guide, see https://github.com/traverson/traverson/blob/master/user-guide.markdown#traverson-web-component. Maybe this helps to give this project more attention :)

Thankyou.
I tried your instructions for transferring but I was unable to complete the process. I got the following error message:

You don’t have the permission to create repositories on traverson

OK, I hadn't seen your invite to the traverson org.


Writing to DA is blocking reading of new sequencer txs

When I have sped up block times in our dev-cluster (see https://github.com/astriaorg/dev-cluster/pull/35/) the sequencer no longer consistently grabs blocks from metro. Looking at the log below, we can see timing wise that sequencer-relayer runs at the expected speed up until it starts seeing blocks. At that point there is significant delay in writing to celestia. We need to either make this non blocking, or bundle everything per celestia block into a single pfb.
I think there are two fixes we need here:

We should be tracking sequencer block height and always be requesting the next block as opposed to the next head. Perhaps we grab all blocks between the head & last block
Celestia batching & blocking of writing a PFB

2023-06-22T16:19:28.593842Z  WARN astria_sequencer_relayer::relayer: failed to get latest block from sequencer error=failed getting latest block

2023-06-22T16:19:29.593908Z  WARN astria_sequencer_relayer::relayer: failed to get latest block from sequencer error=failed getting latest block

2023-06-22T16:19:31.593417Z  WARN astria_sequencer_relayer::relayer: failed to get latest block from sequencer error=failed getting latest block

2023-06-22T16:19:32.593475Z  WARN astria_sequencer_relayer::relayer: failed to get latest block from sequencer error=failed getting latest block

2023-06-22T16:19:41.594699Z  INFO astria_sequencer_relayer::relayer: got block with height 3 from sequencer
2023-06-22T16:19:41.595136Z  WARN astria_sequencer_relayer::network: failed to publish block to network e=failed to publish message
2023-06-22T16:19:55.788947Z  INFO astria_sequencer_relayer::relayer: submitted sequencer block to DA layer sequencer_block=3 da_layer_block=37 tx_count=0

2023-06-22T16:19:55.790573Z  INFO astria_sequencer_relayer::relayer: got block with height 4 from sequencer
2023-06-22T16:19:55.790776Z  WARN astria_sequencer_relayer::network: failed to publish block to network e=failed to publish message
2023-06-22T16:20:20.797398Z  INFO astria_sequencer_relayer::relayer: submitted sequencer block to DA layer sequencer_block=4 da_layer_block=38 tx_count=0

2023-06-22T16:20:20.798946Z  INFO astria_sequencer_relayer::relayer: got block with height 8 from sequencer
2023-06-22T16:20:20.799397Z  WARN astria_sequencer_relayer::network: failed to publish block to network e=failed to publish message
2023-06-22T16:20:45.812921Z  INFO astria_sequencer_relayer::relayer: submitted sequencer block to DA layer sequencer_block=8 da_layer_block=39 tx_count=0

2023-06-22T16:20:45.815212Z  INFO astria_sequencer_relayer::relayer: got block with height 11 from sequencer
2023-06-22T16:20:45.815401Z  WARN astria_sequencer_relayer::network: failed to publish block to network e=failed to publish message

Currently sequencer-relayer has 1 block timing loop. Should we instead have 2 different loops? One for DA and one for sequencer.
On sequencer timing, read new blocks, gossip, add to DA queue. On DA timing write the queue to DA?

there are multiple problems here:

the relayer writing blocks on writing to celestia due to celestia's pay_for_data API being synchonous, ie. it only returns when the data is included in a celestia block, and thus this will block corresponding to celestia's block time. this can be fixed by putting the data write into its own tokio task, however this then opens us up to issue number 2:
celestia does not allow us to submit multiple PFDs for the same namespace in one block; if you attempt this, it immediately rejects the PFDs that are not the first one and only includes the first. it doesn't even give us a good response, just that the response fields are empty and return immediately.

the only decent solution seems to be batching the writes on our end; so for each namespace we want to write to (sequencer, rollup), we batch all the writes into a Vec<SignedNamespaceData> and submit it once per celestia block time.
the conductor will need to be updated to handle a batched Vec<SignedNamespaceData> instead of just a SignedNamespaceData as it does now.

Might be worth looking at how Sovereign Labs has their Celestia adapter working: https://github.com/Sovereign-Labs/sovereign-sdk/tree/main/adapters/celestia

@jbowen93 looks like their adaptor only does reading, not writing? :/

@renaynay I don't think so, looks like we've been using an older version: https://github.com/astriaorg/dev-cluster/blob/main/kubernetes/celestia-local/deployment.yml#L47
where can I find the API docs for rc5?

@noot looks like ur using a custom image of node - can you tell me which version that points to?

Specific commits are here:
https://github.com/celestiaorg/celestia-node/compare/main...astriaorg:celestia-node:v0.6.1_test_docker
https://github.com/celestiaorg/celestia-node/commit/07fa3e726a90327c080d8fd279cb5dc64675d3cb

@renaynay I believe were are using v0.6.1 of celestia-node and 0.11.0 of celestia-app. will be updating to use rc5, as the new Submit API here should unblock/fix this issue as we can submit multiple blobs in one tx https://node-rpc-docs.celestia.org/

There are two issues that appear here:

architecturally sequencer-relayer and conductor are blocking their respective event loops when interacting with celestia; in general, in loop { select! { ... } } constructions the handler code should not be awaited, which both projects do (for example, sending data to celestia with reqwest::Client or jsonrpsee::HttpClient and then awaiting the response blocks the loop from progressing.
sequencer-relayer's CelestiaClient::{submit_namespaced_data, get_sequencer_block} would issue one request per item; this further aggravates the previous architectural issues.

#150 tracked in #151 addresses the second point. But a proper fix is to resolve the architectural issues.


[feat]: Allow for AMB payment logic

Allow for a configurable payment amount to be sent on transactions that generate cross chain messages to pay for the AMB services (e.g. on message 'delivery', a return cross chain 'ack' message has to be paid for).

Implemented via #34


delete an element in the array, patches is an empty array

🐛 Bug Report
immer v9.0.7
just delete an element in the array, it is not difficult to reproduce the bug, please take a look at the following demo or code.
Link to repro
demo
To Reproduce
import { produceWithPatches, enablePatches } from 'immer';

enablePatches();

const [newState, patches, inversePatches] = produceWithPatches(
  [1, 2, 3],
  (draft) => {
    delete draft[2];
  }
);

console.log(newState, patches, inversePatches);
// output:
// [1, 2, undefined] [] []

Observed behavior
As described above, patches and inversePatches are all empty
Expected behavior
They should not be empty.
Environment
We only accept bug reports against the latest Immer version.

Immer version:
[x] I filed this report against the latest version of Immer
[ ] Occurs with setUseProxies(true)
[ ] Occurs with setUseProxies(false) (ES5 only)

Do I understand this prevents immer from creating sparse arrays with delete? I've spent the evening slowly removing pieces of my stack and API calls trying to find issues with sparse indices. I have no trouble creating sparse arrays using "proxied mutation"
// produces [<empty>,42]
(draftState: Draft<{ values: [] }>) => {
  values[1] = 42;
}

Then:
// produces [undefined, undefined]
(draftState: Draft<{ values: [] }>) => {
  delete values[1];
}

Be careful using the stackblitz console to test the reproduction results. It runs a parser on the input data which casts holes to undefined. Opening the browser's devtools reveals the proper output.


[no-unused-variable] Unused destructured variables ignored with object rest syntax.

Bug Report

TSLint version: v5.9.1
TypeScript version: v2.8.1
Running TSLint via: (pick one) Atom's linter-tslint

TypeScript code being linted
function restInPeace() {
  const anObject = { a: 'a', b: 'b' }
  const {
    a,
    b,
    ...obj
  } = anObject
}

with tslint.json configuration:
{
  "defaultSeverity": "warning",
  "rules": {
    "no-unused-variable": [
      true,
      "check-parameters",
      { "ignore-pattern": "^_" }
    ]
  }
}

Actual behavior
TSLint flags obj as unused, and ignores a and b.
Expected behavior
It would be great if it were possible to at least configure this to catch a and b, in the style of ESLint, which offers an ignoreRestSiblings flag for its unused variable check. My personal preference is to check rest siblings when destructuring, because it often alerts me to cases when I meant to pass a React prop down to a child component and forgot.

I think it makes sense to add an extra flag for this.

Turns out the custom rule no-unused from my package tslint-consistent-codestyle already checks siblings of object rest. You need to explicitly ignore the variable by adding an underscore prefix.

Awesome - closing as external on both accounts.


pin sqlachemy<2

see obspy/obspy#3258

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

By accident pushed the branch to main repo first, so a build got deployed. Bumping build number again and making this proper PR to clean up things.

@conda-forge-admin, please rerender


Metric Parameter not Working as Intended

Description

In modeling a binary classification problem, the results do not change with the "metric" parameter.
Reproducible example

import lightgbm as lgb
from sklearn.datasets import load_breast_cancer
from sklearn.model_selection import train_test_split

# Load the breast cancer dataset (example dataset)
data = load_breast_cancer()
X = data.data
y = data.target

# Split the data into train and test sets
X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.2, random_state=42)

# Define the LightGBM classifier
lgb_model = lgb.LGBMClassifier(metric='binary_logloss')  # Using binary log-loss as the evaluation metric

# Fit the model to the data
lgb_model.fit(X_train, y_train)

# Make predictions on the test set
y_pred = lgb_model.predict(X_test)

# Evaluate the model using the specified metric
accuracy = lgb_model.score(X_test, y_test)
print(f"Accuracy: {accuracy:.4f}")

I tried to change the metric parameter from 'binary_logloss" to 'auc', 'f1', or 'average_precision', the results kept the same.
Environment info
LightGBM version or commit hash: 4.0.0
Command(s) you used to install LightGBM
pip install lightgbm

Additional Comments

Did I miss anything here?

Hey @jcoding2022, thanks for using LightGBM.
I think you mean to use objective. The metric is used when you're computing evaluation metrics, e.g. when using valid_sets and the log_evaluation callback.
Please let us know if you have further doubts.

Hey @jcoding2022, thanks for using LightGBM.
I think you mean to use objective. The metric is used when you're computing evaluation metrics, e.g. when using valid_sets and the log_evaluation callback.
Please let us know if you have further doubts.

Thanks for your quick reply. Can you point to an example that shows how the metric parameter works?

Hey @jcoding2022, thanks for using LightGBM.
I think you mean to use objective. The metric is used when you're computing evaluation metrics, e.g. when using valid_sets and the log_evaluation callback.
Please let us know if you have further doubts.

If my objective is to increase F1_score, what parameters and values should I use?

Can you point to an example that shows how the metric parameter works?

https://github.com/microsoft/LightGBM/blob/e9fface2f7013014a4100270b01f44efd2b920d3/examples/python-guide/sklearn_example.py#L24-L30

If my objective is to increase F1_score, what parameters and values should I use?

If you want to continue using the scikit-learn interface to LightGBM:

use LGBMClassifier()

it will automatically set objective="binary" when passed a label with 2 unique values

define a custom eval metric as a Python function, and pass it through the eval_metric parameter to LGBMClassifier.fit()

Consider the following minimal example, which I tested with lightgbm v4.0.0
import lightgbm as lgb
from sklearn.datasets import load_breast_cancer
from sklearn.metrics import f1_score
from sklearn.model_selection import train_test_split

# Load a binary classification dataset
X, y = load_breast_cancer(return_X_y=True)
X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.2, random_state=42)

# define F1 score as a custom metric
def f1_metric(y_true, y_pred):
    metric_name = "F1"
    # NOTE: y_pred will be a probability of the majority class
    value = f1_score(y_true, y_pred > 0.5)
    higher_better = True
    return metric_name, value, higher_better

# Fit the model to the data
# NOTE: max_depth, num_leaves, min_child_samples set to
#       smaller-than-default values because the dataset
#       only has around 450 rows
lgb_model = lgb.LGBMClassifier(
    verbosity=1,
    n_estimators=10,
    max_depth=3,
    num_leaves=7,
    min_child_samples=2
)

lgb_model.fit(
    X=X_train,
    y=y_train,
    eval_set=[(X_test, y_test)],
    eval_names="valid",
    eval_metric=["auc", "binary_error", f1_metric],
    callbacks=[lgb.log_evaluation(1)]
)

You should see all of those metrics passed to eval_metric printed at each iteration.
[LightGBM] [Info] Number of positive: 286, number of negative: 169
[LightGBM] [Info] Auto-choosing row-wise multi-threading, the overhead of testing was 0.000043 seconds.
You can set `force_row_wise=true` to remove the overhead.
And if memory is not enough, you can set `force_col_wise=true`.
[LightGBM] [Info] Total Bins 4548
[LightGBM] [Info] Number of data points in the train set: 455, number of used features: 30
[LightGBM] [Info] [binary:BoostFromScore]: pavg=0.628571 -> initscore=0.526093
[LightGBM] [Info] Start training from score 0.526093
[1]	valid's auc: 0.956436	valid's binary_error: 0.377193	valid's binary_logloss: 0.582173	valid's F1: 0.767568
[2]	valid's auc: 0.972977	valid's binary_error: 0.377193	valid's binary_logloss: 0.514326	valid's F1: 0.767568
[3]	valid's auc: 0.971667	valid's binary_error: 0.0526316	valid's binary_logloss: 0.462733	valid's F1: 0.959459
[4]	valid's auc: 0.97265	valid's binary_error: 0.0526316	valid's binary_logloss: 0.415805	valid's F1: 0.958904
[5]	valid's auc: 0.97265	valid's binary_error: 0.0438596	valid's binary_logloss: 0.37737	valid's F1: 0.965517
[6]	valid's auc: 0.967082	valid's binary_error: 0.0438596	valid's binary_logloss: 0.347271	valid's F1: 0.965517
[7]	valid's auc: 0.966754	valid's binary_error: 0.0350877	valid's binary_logloss: 0.319601	valid's F1: 0.972222
[8]	valid's auc: 0.967245	valid's binary_error: 0.0350877	valid's binary_logloss: 0.294086	valid's F1: 0.972222
[9]	valid's auc: 0.994432	valid's binary_error: 0.0350877	valid's binary_logloss: 0.268995	valid's F1: 0.972222
[10]	valid's auc: 0.994104	valid's binary_error: 0.0350877	valid's binary_logloss: 0.251823	valid's F1: 0.972222

Note that the F1 score on the held-out validation set is improving with each iteration.
Try modifying eval_name, eval_metrics, and the number of iterations passed through lgb.log_evaluation() to customize this.
You can recover all of the metric values in the evals_result_ property.
lgb_model.evals_result_

{'valid': OrderedDict([('auc', [0.9564362921716345, 0.9729773992793973, 0.9716672125777923, 0.9726498526039961, 0.9726498526039961, 0.9670815591221749, 0.9667540124467736, 0.9672453324598755, 0.9944317065181788, 0.9941041598427776]), ('binary_error', [0.37719298245614036, 0.37719298245614036, 0.05263157894736842, 0.05263157894736842, 0.043859649122807015, 0.043859649122807015, 0.03508771929824561, 0.03508771929824561, 0.03508771929824561, 0.03508771929824561]), ('binary_logloss', [0.5821729220177619, 0.5143256933774838, 0.4627328015843806, 0.415804786329259, 0.3773696159284982, 0.3472708147431134, 0.31960105920456877, 0.29408603472697215, 0.26899508360230007, 0.25182274999831583]), ('F1', [0.7675675675675676, 0.7675675675675676, 0.9594594594594594, 0.9589041095890412, 0.9655172413793103, 0.9655172413793103, 0.9722222222222222, 0.9722222222222222, 0.9722222222222222, 0.9722222222222222])])}

If you have pandas available and would prefer to work with the metrics in that format, you can do the following.
import pandas as pd
metrics_df = pd.DataFrame.from_records(lgb_model.evals_result_["valid"])
metrics_df

         F1       auc  binary_error  binary_logloss
0  0.767568  0.956436      0.377193        0.582173
1  0.767568  0.972977      0.377193        0.514326
2  0.959459  0.971667      0.052632        0.462733
3  0.958904  0.972650      0.052632        0.415805
4  0.965517  0.972650      0.043860        0.377370
5  0.965517  0.967082      0.043860        0.347271
6  0.972222  0.966754      0.035088        0.319601
7  0.972222  0.967245      0.035088        0.294086
8  0.972222  0.994432      0.035088        0.268995
9  0.972222  0.994104      0.035088        0.251823

For docs on all of this, see:

https://lightgbm.readthedocs.io/en/latest/pythonapi/lightgbm.LGBMClassifier.html
https://lightgbm.readthedocs.io/en/latest/Parameters.html

For more discussion you might find relevant, see #3029.


Not able to run from example

Getting the following error.
SystemExit: Video file cannot be read! Please check input_vidpath to ensure it is correctly pointing to the video file

get this error when using the example fish_vedio
EOL while scanning string literal

Working now.


Bump :lang rust

brotzeit/rustic@f7a94c4 -> brotzeit/rustic@7c0e113

There were complains from users who are using the non-lsp flycheck checker with the stable toolchain.
I added the new defcustom ´rustic-flycheck-clippy-params´. Question is what should be the default value. Should it be configured for stable or nightly toolchain. I decided to include the option "-Zunstable-options" which is only available on nightly. But maybe you decide otherwise for doom-emacs.

I decided to include the option "-Zunstable-options" which is only available on nightly. But maybe you decide otherwise for doom-emacs.

I think that's a sensible default. Thanks again for the help!


High sierra default platform

Changed the environment variable path for High Sierra/Mojave in setupEnv-android-mac.js file.

Codecov Report

Merging #72 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #72   +/-   ##
=======================================
  Coverage   38.25%   38.25%
=======================================
  Files          25       25
  Lines        1286     1286
  Branches      167      167
=======================================
  Hits          492      492
  Misses        794      794

Continue to review full report at Codecov.

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


Custom fields

How to add custom or meta fields to product?

Hi @PrarthanaHegde, probably you can use "meta_data" property in the productInfo map. Take a look on http://woocommerce.github.io/woocommerce-rest-api-docs/#product-properties

  List<Map<String, Object>> metaData= (List<Map<String,Object>>) mapProductIntegracao.get("meta_data");

Working fine.Thank you so much.
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Fri, Mar 16, 2018 at 6:45 PM, Guilherme Machado<EMAIL_ADDRESS>
wrote:

List<Map<String, Object>> metaData= (List<Map<String,Object>>) mapProductIntegracao.get("meta_data");
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/icoderman/wc-api-java/issues/5#issuecomment-373709704,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARwDmGu6n9jOzur9tiYUDP_-NX_QEcvaks5te7sGgaJpZM4Scw1n
.


data testdata

Description of feature
Description of feature
Generate a test data set
https://www.10xgenomics.com/datasets/fresh-frozen-mouse-brain-for-xenium-explorer-demo-1-standard (still 3.5 GB)

https://docs.google.com/document/d/1HhsUL76LUadeqWEf0MBxaORvQIN_dJSZPM7Unb8amnk/edit
Spatialxe example dataset.docx

Test data generated and finished.


Realtime example

Hello, it's been a while 😄 Thanks @acupofjose for this incredible work.
What kind of change does this PR introduce?

Add simple example to consume the package

@elrhomariyounes good to see you around here again! Thanks for the example - merging!

@elrhomariyounes would love to see you on the supabase-csharp repo - it could use some tests and examples if you're willing!

Yes sure 🚀


Still maintained?

I see no major development on this project since over a year and would like to know if it is still maintained. Example projects are archived our out dated and the GettingStarted repo is empty. So I'm a little bit concerned to use this library. Maybe there are better options nowadays?

Seems pretty dead

It's still a viable option and is in active use on more than one software project. I'd say most maintainers are busy with their day jobs. Of course, there are alternatives. What sort of development would you expect - what features are you missing? Or is this merely a cae of low activity being perceived as "this project is dead"?

Just the low activity and the open pull requests made me question if the project is still maintained.

The big issue for me is the low response on pull request. Off course we don't expect maintainers to quit ther day jobs. But a few months without reaction, is more then 'merely a case of low activity'.

I agree with the general concern, and it's not just a matter of "missing features", it's just a feeling that there seems to be no activity at for very long chunks time.
The implementation for Sqlite was merged over a year and a half ago, but as far as I can tell (and anybody feel free to correct me if I'm wrong) I don't see a pre-release nuget package with the SqliteStreamStore implementation.
The Postgres one has also been on "beta" for a while now.
Same feeling applies to the amount of open PR's updating dependencies to newer versions, that are just there "open"...
We want to use SqlStreamStore in production but the doubts are there.

Hi all,
Sorry for the absence.
I've been giving some serious thought as to the future of this project. It's obvious that I've (and my associates) not been giving it the attention that it needs. I will follow up with a statement shortly and will be looking for your feedback.
Cheers
Damian


Error DataProtection-Keys Docker Image

when building the docker image using docker-compose build.
I run the docker-compose up command to start the services.
But the api image has DataProtection-Keys error
follows the log file of the generated error,
Tried to create the image on windows, ubuntu, and mac.
on all platforms the same error is generated.
Error Log:
System.InvalidOperationException: Unable to configure HTTPS endpoint. No server certificate was specified, and the default developer certificate could not be found or is out of date.,
To generate a developer certificate run 'dotnet dev-certs https'. To trust the certificate (Windows and macOS only) run 'dotnet dev-certs https --trust'.,
For more information on configuring HTTPS see https://go.microsoft.com/fwlink/?linkid=848054.,
at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions, Action1 configureOptions),    at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions),    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.AddressesStrategy.BindAsync(AddressBindContext context),    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func2 createBinding),
at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication1 application, CancellationToken cancellationToken), Unhandled exception. System.InvalidOperationException: Unable to configure HTTPS endpoint. No server certificate was specified, and the default developer certificate could not be found or is out of date., To generate a developer certificate run 'dotnet dev-certs https'. To trust the certificate (Windows and macOS only) run 'dotnet dev-certs https --trust'., For more information on configuring HTTPS see https://go.microsoft.com/fwlink/?linkid=848054.,    at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions, Action1 configureOptions),
at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions),
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.AddressesStrategy.BindAsync(AddressBindContext context),
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func2 createBinding),    at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication1 application, CancellationToken cancellationToken),
at Microsoft.AspNetCore.Hosting.GenericWebHostService.StartAsync(CancellationToken cancellationToken),
at Microsoft.Extensions.Hosting.Internal.Host.StartAsync(CancellationToken cancellationToken),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.Run(IHost host),
at OWSInstanceManagement.Program.Main(String[] args) in /src/OWSInstanceManagement/Program.cs:line 17,
warn: Microsoft.AspNetCore.DataProtection.Repositories.FileSystemXmlRepository[60],
Storing keys in a directory '/app/temp/DataProtection-Keys' that may not be persisted outside of the container. Protected data will be unavailable when container is destroyed.,
crit: Microsoft.AspNetCore.Server.Kestrel[0],
Unable to start Kestrel.,
System.InvalidOperationException: Unable to configure HTTPS endpoint. No server certificate was specified, and the default developer certificate could not be found or is out of date.,
To generate a developer certificate run 'dotnet dev-certs https'. To trust the certificate (Windows and macOS only) run 'dotnet dev-certs https --trust'.,
For more information on configuring HTTPS see https://go.microsoft.com/fwlink/?linkid=848054.,
at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions, Action1 configureOptions),    at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions),    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.AddressesStrategy.BindAsync(AddressBindContext context),    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func2 createBinding),
at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication1 application, CancellationToken cancellationToken), Unhandled exception. System.InvalidOperationException: Unable to configure HTTPS endpoint. No server certificate was specified, and the default developer certificate could not be found or is out of date., To generate a developer certificate run 'dotnet dev-certs https'. To trust the certificate (Windows and macOS only) run 'dotnet dev-certs https --trust'., For more information on configuring HTTPS see https://go.microsoft.com/fwlink/?linkid=848054.,    at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions, Action1 configureOptions),
at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions),
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.AddressesStrategy.BindAsync(AddressBindContext context),
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func2 createBinding),    at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication1 application, CancellationToken cancellationToken),
at Microsoft.AspNetCore.Hosting.GenericWebHostService.StartAsync(CancellationToken cancellationToken),
at Microsoft.Extensions.Hosting.Internal.Host.StartAsync(CancellationToken cancellationToken),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.Run(IHost host),
at OWSInstanceManagement.Program.Main(String[] args) in /src/OWSInstanceManagement/Program.cs:line 17,
warn: Microsoft.AspNetCore.DataProtection.Repositories.FileSystemXmlRepository[60],
Storing keys in a directory '/app/temp/DataProtection-Keys' that may not be persisted outside of the container. Protected data will be unavailable when container is destroyed.,
crit: Microsoft.AspNetCore.Server.Kestrel[0],
Unable to start Kestrel.,
System.InvalidOperationException: Unable to configure HTTPS endpoint. No server certificate was specified, and the default developer certificate could not be found or is out of date.,
To generate a developer certificate run 'dotnet dev-certs https'. To trust the certificate (Windows and macOS only) run 'dotnet dev-certs https --trust'.,
For more information on configuring HTTPS see https://go.microsoft.com/fwlink/?linkid=848054.,
at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions, Action1 configureOptions),    at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions),    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.AddressesStrategy.BindAsync(AddressBindContext context),    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func2 createBinding),
at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication1 application, CancellationToken cancellationToken), Unhandled exception. System.InvalidOperationException: Unable to configure HTTPS endpoint. No server certificate was specified, and the default developer certificate could not be found or is out of date., To generate a developer certificate run 'dotnet dev-certs https'. To trust the certificate (Windows and macOS only) run 'dotnet dev-certs https --trust'., For more information on configuring HTTPS see https://go.microsoft.com/fwlink/?linkid=848054.,    at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions, Action1 configureOptions),
at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions),
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.AddressesStrategy.BindAsync(AddressBindContext context),
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func2 createBinding),    at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication1 application, CancellationToken cancellationToken),
at Microsoft.AspNetCore.Hosting.GenericWebHostService.StartAsync(CancellationToken cancellationToken),
at Microsoft.Extensions.Hosting.Internal.Host.StartAsync(CancellationToken cancellationToken),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.Run(IHost host),
at OWSInstanceManagement.Program.Main(String[] args) in /src/OWSInstanceManagement/Program.cs:line 17,
warn: Microsoft.AspNetCore.DataProtection.Repositories.FileSystemXmlRepository[60],
Storing keys in a directory '/app/temp/DataProtection-Keys' that may not be persisted outside of the container. Protected data will be unavailable when container is destroyed.,
crit: Microsoft.AspNetCore.Server.Kestrel[0],
Unable to start Kestrel.,
System.InvalidOperationException: Unable to configure HTTPS endpoint. No server certificate was specified, and the default developer certificate could not be found or is out of date.,
To generate a developer certificate run 'dotnet dev-certs https'. To trust the certificate (Windows and macOS only) run 'dotnet dev-certs https --trust'.,
For more information on configuring HTTPS see https://go.microsoft.com/fwlink/?linkid=848054.,
at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions, Action1 configureOptions),    at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions),    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.AddressesStrategy.BindAsync(AddressBindContext context),    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func2 createBinding),
at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication1 application, CancellationToken cancellationToken), Unhandled exception. System.InvalidOperationException: Unable to configure HTTPS endpoint. No server certificate was specified, and the default developer certificate could not be found or is out of date., To generate a developer certificate run 'dotnet dev-certs https'. To trust the certificate (Windows and macOS only) run 'dotnet dev-certs https --trust'., For more information on configuring HTTPS see https://go.microsoft.com/fwlink/?linkid=848054.,    at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions, Action1 configureOptions),
at Microsoft.AspNetCore.Hosting.ListenOptionsHttpsExtensions.UseHttps(ListenOptions listenOptions),
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.AddressesStrategy.BindAsync(AddressBindContext context),
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func2 createBinding),    at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication1 application, CancellationToken cancellationToken),
at Microsoft.AspNetCore.Hosting.GenericWebHostService.StartAsync(CancellationToken cancellationToken),
at Microsoft.Extensions.Hosting.Internal.Host.StartAsync(CancellationToken cancellationToken),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.RunAsync(IHost host, CancellationToken token),
at Microsoft.Extensions.Hosting.HostingAbstractionsHostExtensions.Run(IHost host),
at OWSInstanceManagement.Program.Main(String[] args) in /src/OWSInstanceManagement/Program.cs:line 17,

The issue has been resolved, this was related to ASP.Net Core on Docker Linux not being able to find a valid Developer Self Signed Certificate.
Docker Linux has been added, please see Docker Linux Instructions


Remove unnecessary paths from Sphinx directives.

I just found out yesterday that when you're in the class, you don't need to supply the path to the method! You can just do
:meth:`methodname`

Life-changing. But more life-changing is our decision to document aliases differently, mentioning them in the base class and skipping their documentation in the ... documentation. This PR does both of those things!

Looks like mypy doesn't currently support aliases this way.
https://github.com/python/mypy/issues/6700


Add additional properties on ServerError.Error

Closes #2355

In core they can attach any random property to Exceptions to innerXContent.
These properties only apply to: https://github.com/elastic/elasticsearch/blob/7560101ec73331acb39a042bde6130e59e4bb630/core/src/main/java/org/elasticsearch/action/search/SearchPhaseExecutionException.java#L137
We can union all the additional properties under ServerError but I am wondering if we can do something smarter here.

Perhaps capture them in an .AdditionalDetails dictionary on ServerError.Error?

I think that be better, we already have some others on ServerError that would be candidates of bumping down.

Should this be a blocker for release? cc @gmarz @russcam

I don't think so @mpdreamz

LGTM tetst

look at after 6.x release

This is now part of the work on #2929


Document how the ESP32 software loading process works

We want to make our own PCB and do away with using the Huzzah32. We would like to use the same trick that Adafruit uses to have the ESP32 go into code loading automatically when the upload button in PlatformIO is pressed. We need to investigate both how the circuit is built as well as how the ESp32 boot-loader process works.
The Huzzah32 schematic can be found here: https://learn.adafruit.com/adafruit-huzzah32-esp32-feather/downloads.
The boot loader process can be found here: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-guides/startup.html.

When discussing this topic again this week we figure that if we just build our circuit to emulate the Huzzah 32 then we can skip this investigation. So for now we do not want tot pursue this issue further.


update realm-export.json to resolve quickstart issues

Thank you for taking this over. Your help is appreciated!


Additional annotation for methylhypoxanthines (CHEBI:73958)

It would be nice if methylhypoxanthines could receive additional curation.
The plural form could be removed, in favour of retaining the present annotation, perhaps with some additional ontology curation, as the category. Some children of it could then be added, including, though ideally not limited to, 9-methylhypoxanthine, for which a few references could also be added, e.g.:

Belanger-Gariepy, Francine, and Andre L. Beauchamp. "Crystal structure of bis (9-methylhypoxanthine) silver (I) perchlorate monohydrate, a model complex for the silver-poly (I) system." Journal of the American Chemical Society 102, no. 10 (1980): 3461–4.
van Vliet, Paul M., Jaap G. Haasnoot, and Jan Reedijk. "Binding of 9-Methylhypoxanthine and 9-Ethylguanine to [cis-Ru (2, 2'-bipyridine) 2] 2+. NMR and X-ray Structure of cis-Chlorobis (2, 2'-bipyridine)(9-ethylguanine-. kappa. N7) ruthenium (II) Chloride." Inorganic Chemistry 33, no. 9 (1994): 1934–9.

I have now added the following child classes to CHEBI:73958 including 3-methylhypoxanthine, 7-methylhypoxanthine and 9-methylhypoxanthine. Unfortunately i cannot add the references provided since they do not have a PubMed identifier associated with them.


cargo install fails

STR -
git clone https://github.com/dustinknopoff/bashdoc
cd bashdoc
cargo install

Output
warning: proc macro crates and `#[no_link]` crates have no effect without `#[macro_use]`
  --> src/main.rs:52:1
   |
52 | extern crate serde_derive;
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0432]: unresolved import `clap::load_yaml`
  --> src/main.rs:55:5
   |
55 | use clap::load_yaml;
   |     ^^^^^^^^^^^^^^^ no `load_yaml` in the root

error: cannot find macro `load_yaml!` in this scope
  --> src/main.rs:61:16
   |
61 |     let yaml = load_yaml!("../cli.yml");
   |                ^^^^^^^^^

error[E0658]: access to extern crates through prelude is experimental (see issue #44660)
   --> src/doc_structure.rs:240:20
    |
240 |         let json = serde_json::to_string_pretty(&docstrings).expect("Could not convert to JSON");
    |                    ^^^^^^^^^^
    |
    = help: add #![feature(extern_prelude)] to the crate attributes to enable

error[E0658]: access to extern crates through prelude is experimental (see issue #44660)
   --> src/doc_structure.rs:280:42
    |
280 |                 let sorted: Delimiters = toml::from_str(&to_convert).unwrap();
    |                                          ^^^^
    |
    = help: add #![feature(extern_prelude)] to the crate attributes to enable

error[E0658]: access to extern crates through prelude is experimental (see issue #44660)
   --> src/doc_structure.rs:286:21
    |
286 |                     toml::to_string_pretty(&delims).expect("Could not be converted to TOML");
    |                     ^^^^
    |
    = help: add #![feature(extern_prelude)] to the crate attributes to enable

error: aborting due to 5 previous errors

Some errors occurred: E0432, E0658.
For more information about an error, try `rustc --explain E0432`.
error: failed to compile `bashdoc v0.1.0 (file:///Users/durwasa/programming/bashdoc)`, intermediate artifacts can be found at `/Users/durwasa/programming/bashdoc/target`

Caused by:
  Could not compile `bashdoc`.

To learn more, run the command again with --verbose.

Expected Output -
Clean install

Can you try rustup update stable and trying again? I'm using v1.3 which doesn't require #[macro_use].

@durwasa-chakraborty Let me know if that works!

Sorry I didn't thoroughly check it the first time before confirming that rustup update stable did the job. But it seems the problem still persists with the same error log.

Does rustc --version show rustc 1.30.0 ...?

It shows rustc 1.28.0-nightly (e3bf634e0 2018-06-28) but when I run rustup update stable - it stays stable-x86_64-apple-darwin unchanged - rustc 1.30.0 (da5f414c2 2018-10-24)
Would investigate it further.


docs: add copy/select code buttons

Fixes no issue
Description

add copy and select code to docs

Checklist

[x] PR is rebased to/aimed at branch develop
[x] PR follows Contributing Guide
[x] Added tests (if necessary)
[x] Extended README/Documentation (if necessary)
[x] Adjusted versions of image and Helm chart in Chart.yaml (if necessary)

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (92affcf) 96.58% compared to head (672e981) 96.58%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1302   +/-   ##
========================================
  Coverage    96.58%   96.58%
========================================
  Files           23       23
  Lines         1319     1319
========================================
  Hits          1274     1274
  Misses          45       45

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


Clean error prone

Update the code fix all error prone warnings.
Run mvn package on the repo. Fix the error prone warnings.

I am very interested in looking at more deeper into kaldb. Can I please work on this issue to get me booted on the project?


How to debug TPVFormer with ipdb?

I need to make improvements to the TPVFormer. How can I debug the code?

Thanks for your interest and sorry for the late reply. You can disable distributed data parallel (DDP) and then you can use ipdb freely.


Fix CI jobs so that they can work on PRs from forks

 Issue by westonpace
Wednesday Mar 15, 2023 at 13:26 GMT
Originally opened as https://github.com/sanjibansg/substrait-fiddle/issues/11

Currently they rely on a secret to download the backend repo.  I think this is because the backend repo was originally not public.  Now that the backend repo is public that secret should no longer be needed.

I think we can close this issue now. Right now, as we have both the back-end and the front-end service in the same repository, we will not be needing to download it anyway.
cc: @westonpace


⚠ Roblox Devforum has degraded performance

In f14bfaa, Roblox Devforum (https://devforum.roblox.com) experienced degraded performance:

HTTP code: 200
Response time: 6010 ms

Resolved: Roblox Devforum performance has improved in 25c1263.


Fixed issue with incorrect JPA properties values

Due to changes introduced with "JAMES-2156 Add configuration option to enable attachment storage for JPA", values for keys openjpa.Multithreaded and attachmentStorage.enabled set in properties passed to Persistence.createEntityManagerFactory() method have incorrect values.
Due to usage of toString() method on Optional<Boolean> instead of calling it directly on Boolean, values are set as Optional[true] if they had value true causing it to evaluate inside persistence to false.
This change fixes this regression restoring ability to set  openjpa.Multithreaded and attachmentStorage.enabled to values other than false.

Thanks for the contribution @hantu85 !


Biotransformer annotation - java.lang.Runtime Exception "peakListValues" is null

Basic information

My operating system and version: Windows 10 Version 10.0.19045
My MZmine version: 3.9.0

What happened

When predicting transformation of a compound from a SMILES using the BioTransformer 3.0-workflow, the following exception occurrs every time BioTransformer does not predict any transformation products. For compounds where transformation products are predicted, everything works as usual. This could hinder the application of the workflow to datasets with multiple precursors, because the workflow will be aborted when there are no products predicted for one of the precursors. It would be great if the workflow could continue and in the end give a message stating that no transformation products were predicted for compounds xyz.

20240328_mzmine_biotransformer_error_log.txt

thanks for the report, i will look into that


Upgrade dependencies, migrate to the official docker client, and make the listening code more robust

Fix a few minor bugs at the same time, along with a redesign of the Docker error handling.
Before creating your PR, execute the following command:
$> go vet
$> go lint
$> go fmt
$> go build
$> go test
PR source branches shall have the following name: feature/<PR_NAME>
PR destination branch shall be develop
Any PR not respecting those rules won't be merge and will be closed after a few weeks if not updated.

Thanks for the PR, can you fix the errors displayed in the CI workflow please?

Thanks for the PR, can you fix the errors displayed in the CI workflow please?

It should be ok now (at least I think :P)


read from stdin when -state not provided

quick implementation of #6 - read state from stdin if -state <...> is omitted. i opted to keep the existing CLI interface intact for now, in order to maintain backward compatibility with the existing release version. test cases aren't very DRY atm (as the tables are inline and several expect the same output) but they do now exist for the stdin case. i'd also like to add a -statefile <...> option at some point to let terraputs handle the statefile IO for cases where pipes or file redirection aren't well supported.
what's enclosed in the swag implementation will let us get away with these additional use cases:
$ terraform show -json | terraputs
$ terraputs < terraform.tfstate

@mdb this is WIP but feel free to 👀 it and run it thru GH Actions at your convenience. 👋

@mdb very welcome, glad to do it.
i wasn't entirely sure what was going on there with the failed CI loop but figured you'd know. 😁 just pushed up a commit to bump version from 0.0.3->0.0.4 in Makefile. let me know if that's not the right play.
can you approve the action to run? thx.

Thanks @rhenning . One final thing: would you mind updating the README "Usage" and "More Options" sections to better represent the new behavior?

@mdb sure, done!


Nitrogen limitation on nodal growth

Review if it is implemented.

[ ] Check if the node growth is reduced by nitrogen restrictions
[ ] Check if the node length is reduced by nitrogen restrictions

However, the potential curve for stem node growth does not change if the length of the node was reduced due to water or N stress after the node reach maximum length (during leaf growth). I think this change could reduce stem growth more than as currently is reducing it (see image below red and purple lines with water and N stress).


add test for handlertorch

[x] a pytest test for handletorch is added as an adaption from the previous unittest version (now deprecated).
[x] make format to improve formatting etc
[x] modified ruff.toml configurations to not run linting on unittest fiiles

transform handletorch tests from just checking input and output shapes to aqctually identicity of matrices. MEaning shape and content, this is a more stringent test that requires also the shape to be exact.

hello, so now i have modified the test so that it alternatively loads the data into the expected values. And then check if these values are the same as the values loaded directly using the corresponding class.


add scripts for deployment

@haboy52581 Please update the https://github.com/futurewei-cloud/alcor/blob/master/docs/README.md along with the latest scripts.


Enable buyers and sellers to leave reviews

Likely Requirements

[ ] five star rating score
[ ] optional comment string
[ ] scoped to transaction rather than counterparty or product
[ ] timestamped
[ ] affecting transaction state logs

Possible Considerations

review period (and expiration)
review reciprocity
preventing funds withdrawal

Do we only need one new method?

createReview(purchaseAddress, { score, content })

On Airbnb, I think reviews only ever become visible if both parties participate. Will we have a similar model? If so, how do we hang on to a review without publishing it to the blockchain until the counterparty has reciprocated?

I believe that the plan was for a purchase to not enter complete status until (a) the buyer has left a review, or (b) the review period has expired. Is that the case? If a dispute is created, can a review still be left or does it send the purchase state down a different path? When a buyer confirms that they have received a shipment, that should not automatically allow the seller to withdraw funds.

Yes, reviews will definitely need to be indexed.
We'll need the indexing server to check the contract to determine if the review is "valid", since we need to send the review to the blockchain when they happen, even if the other party still might open a dispute.

"When a buyer confirms that they have received a shipment, that should not automatically allow the seller to withdraw funds."

At that point the seller could either withdraw and review or open a dispute. They wouldn't be able to do both.

Great question, @micahalcorn & there's no easy answer to this problem today. Storing / revealing private data on a public blockchain doesn't really work. NuCypher promises to help with this, but it's still under active development.
One temporary and unsatisfying solution would be to have the blind review process enforced entirely by origin.js so the reviews are hidden in the DApp till both sides have left a review. Of course, the data will already be public on IPFS & Ethereum, but you'll need to have some development chops to access it.

@joshfraser Do we need reviews scores on-chain for any reason? If not, the review data we need to store would work as pure logs.
I'm thinking we don't need them on-chain. My guess is that any reputation system that operates deterministically on the blockchain could be gamed, and would be much slower than something off chain.

It adds more trust in the system if we store them on-chain. Companies like Uber and Airbnb have a bad reputation of burying reviews that are bad for business (meth labs in Airbnbs, etc). Of course, it makes good business sense for them to hide the horrible stuff that happens on their platforms, but it doesn't fit w/ our goal of building an open and fair and transparent system.

A hybrid approach for facilitating the reveals might be to publish the IPFS hash of the review to the blockchain w/o publishing the actual content to IPFS. Everyone would be able to see that a review is coming, without knowing what was actually said until your chosen bridge server pushes the content to IPFS 14 days later in the reveal phase. While it would require people to trust the bridge server, it's immediately detectable to everyone if the bridge server is ever dishonest as the content hash would not match.

Sorry, I used the wrong terminology.
Storing events in logs would be just as transparent as storing them in contract storage, since the whole world can see them in both cases. The difference is that contracts cannot see logs.
If we need contracts to be able to see reviews, then would we also need a central store mapping of user -> reviews of them?

Ah, I understand now. I think a reviews registry makes sense.

Done via #183 ✅


Near-impossible to place items on the top row with persistent search bar removed

Title pretty much says it. The Remove thing takes up the whole row in a 5x4 desktop with that persistent bar at the top disabled, leaving maybe a millimeter of space that I can use to place icons or widgets without them getting removed. I have a cheap phone w/ a 4.5 inch screen (LG Leon LTE), if it makes any difference.
The remove thing needs to be shrunken down on the left & right sides so that it covers about half of a single icon's space.
Using Alpha 3 on F-Droid.

Ok

Having the same issue - it's really difficult to drop widgets in the top row without accidentally removing them!

Did u guys disable the clock/(named search bar in settings)

@BennyKok Yes I have that disabled in my settings.

I think the preferable (long-term) solution would be to shrink the the draggable screen area on drag events like it's done when using the stock launcher.

or do you guys like a pop menu like nova launcher did?

I think I will have both in the settings.

I never used Nova Launcher and i can't install it on my Phone as I dont use Google Play Services. I'll check it on an emulator but this can take some days.
I would stick to one solution then because it doesn't seem important to me to have multiple options. It's not an everyday use case for most of the users and there are a lot more things to do.

@leo-graf  well thanks for you reply.

Your Welcome :) Great Project already, i will add some more issues in the next days

I actually do use Nova Launcher, but I'd have to agree with @leo-graf in that implementing both isn't necessary and wastes time better spent polishing other areas that I know exist. I'm not picky, as long as it's functional. If someone specifically requests the other option later, then we can think about expanding the options, but function is more important than looks to me.
But I disagree on the zoom out method. It will have issues if there's plans to implement the Pixel launcher icon capabilities that let app developers create custom actions assigned to their icons when long-pressed. The reason being that upon zooming out, the icon/widget often moves over the background to stay with the finger, so releasing would sometimes place it in a new spot, and therefore not reliably trigger the dropdown menu.
And just to clarify for @leo-graf, the popup is the current method where the remove option pops up on top of the area without zooming out. Nova uses a smaller area similar to my original suggestion up top. Nova also has the custom icon actions like Pixel, btw.

I also would also go for leo-graf's posted method

Very detailed review, let me think.

I am going to vote for the popup method as long as the actual popup is implemented to look more like a drop down menu than what the menu in nova currently looks like. I am not a fan of the separated options with rounded edges, it would look nicer as a rectangle in my opinion.

For now, here is an temporary solution, the bar size is now sat to wrap_content in both direction.

Can you explain the solution? For me it is impossible to put a widget at the top of the homescreen because they getting removed.

Now the slide down option bar's size was reduced. Should have more space to place app on the top