Create chart repo on github pages

Used https://github.com/helm/chart-releaser-action to create a job for publishing your chart to github pages. Every time version is bumped in Chart.yaml a new release should be published. This should solve #159 .
You can test result with
helm repo add testrepo https://tokheim.github.io/sloth
helm repo update
helm upgrade --install test-release testrepo/sloth --debug --dry-run

Unfortunately this seemed to require changing your chart structure a bit, as helm seems very opinionated on having the chart in a  folder. I tested generating the structure just during the workflow, but chart-releaser seems to rely on git diff, and wouldn't detect any changes. See https://github.com/helm/chart-releaser-action/issues/78 for discussion.
For the action to run, you need to create a blank gh-pages branch, and enable pages from project settings on this branch. Something like
git checkout --orphan gh-pages
git reset --hard
echo "Sloth chart repository, see https://sloth.dev" > README.md
git add README.md
git commit --signoff -m "initial pages commit"
git push -u origin gh-pages

Thanks for the contribution. I'll take a look in the following days!

Codecov Report

Merging #189 (e7e190d) into main (96f98bc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #189   +/-   ##
=======================================
  Coverage   78.25%   78.25%
=======================================
  Files          16       16
  Lines        1366     1366
=======================================
  Hits         1069     1069
  Misses        240      240
  Partials       57       57

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 96f98bc...e7e190d. Read the comment docs.

Closes #159


[ci skip] adding user @taoboyang

Hi! This is the friendly automated conda-forge-webservice.
I've added user @taoboyang as instructed in #15.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #15

Close this and making another due to code conflict.


Moving User with API key fails through MySQLIntegrityConstraintViolationException

ISSUE TYPE

Bug Report

COMPONENT NAME

CLOUDSTACK VERSION

Advanced Networking, Cloudstack <IP_ADDRESS> on Ubuntu 14.04 LTS,
MySQL 5.5.59-0ubuntu<IP_ADDRESS>

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

The API call "moveUser" on a user with an API key fails with a MySQLIntegrityConstraintViolationException.
Moving a user without API key works perfectly.
Looks to me, as if there is a constraint on mySQL not allowing to have a new user with the same API key to be prepared.
STEPS TO REPRODUCE

Prepare user with an API key.
Call "move user id=<user-id> account=<destination account name>"

EXPECTED RESULTS

The user to be moved to the destination account.

ACTUAL RESULTS

API call returns with:
--
(rz-admin) :D > move user id=56fab0c7-6724-46c0-9904-eff4020485ad account=AdminTeam-Intern-Individuell
Error 530: Entity already exists:
{
  "cserrorcode": 9999,
  "errorcode": 530,
  "errortext": "Entity already exists: ",
  "uuidList": []
}
--
Exception in /var/log/cloudstack/management/management-server.log:

Caused by: com.mysql.jdbc.exceptions.jdbc4.MySQLIntegrityConstraintViolationException: Duplicate entry '<the api key>' for key 'i_user__api_key'
        at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
        at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
        at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)

thanks @mdesaive

@DaanHoogland  thanks a lot for taking care of this!

welcome @mdesaive care to do a review of the code in return?

Hi @DaanHoogland I did have a look at the code and it looks reasonable to me, although I have no Java Knowledge or IDE on hand. ;) If you like me to review it I can confirm it looks reasonable to me, assuming the following:
I understand the steps to move a user to another account like this:

create a new user with the neccesary data in the table "user"
put the foreign key for the destination account in "account_id"
do "some magic" to take care that all resources of the user are now assigned to the destination account. (Is the connection made by UUID?)

Reading your discussion with @rafaelweingartner in PR #2710 the reason for creating a new entry in the database is that you do not want to loose information that is connected to old user.
Looking in the code with a texteditor, I see, that you erase API-key and API-secret for one of the users. I assume it is the old user? So the user attached to the destination domain will still have the old API-key available?

@mdesaive one of your assumptions is wrong; resources do not belong to users but to accounts, which means that a 'moved' user will loose all of it assets. The reasong for this change was to allow for a configuration change directed by a configured LDAP.

Fixed by PR:
https://github.com/apache/cloudstack/pull/2710


IDENTITY-6049 Instead of using a regex when configuring multiple callback Urls in Inbound OAuth Configuration UI, use an Add button

Give a similar implementation like configuring "Assertion Consumer URLs" in SAML2 Web SSO Configuration under Inbound Authentication Configuration.
Labels
Type: Improvement
Priority: Normal
Component/s: oauth, service-provider
Severity: Major
Estimated Complexity: Moderate
Issue migrated from: https://wso2.org/jira/browse/IDENTITY-6049

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Create and Deploy HTML5 and SAP Fiori Launchpad Site Modules

Step 8 - Test UI on Cloud Foundry.
Unable to start the app from the tile
Log-dbg.js:456 2020-03-15 10:40:27.350780 Failed to load UI5 component for navigation intent #TravelProcessor-display - Error: adding object "component" with duplicate id 'application-TravelProcessor-display-component'
Error: Error: adding object "component" with duplicate id 'application-TravelProcessor-display-component'
at O (https://sapui5.hana.ondemand.com/resources/sap-ui-core.js:555:337)
at f.register (https://sapui5.hana.ondemand.com/resources/sap-ui-core.js:555:553)
at https://sapui5.hana.ondemand.com/resources/sap-ui-core.js:349:867
at f.constructor (https://sapui5.hana.ondemand.com/resources/sap-ui-core.js:349:1036)
at f.constructor (https://sapui5.hana.ondemand.com/resources/sap-ui-core.js:627:577)
at f.constructor (https://sapui5.hana.ondemand.com/resources/sap/ui/core/library-preload.js?eval:994:323)
at f.constructor (https://sapui5.hana.ondemand.com/resources/sap/suite/ui/generic/template/lib/AppComponent.js?eval:7:11569)
at new f (https://sapui5.hana.ondemand.com/resources/sap-ui-core.js:576:558)
at A (https://sapui5.hana.ondemand.com/resources/sap-ui-core.js:662:472)
at https://sapui5.hana.ondemand.com/resources/sap-ui-core.js:662:1279 sap.ushell.renderers.fiori2.Shell.controller

I got the exact same error today.

Can you share which exact step generated the error? Add a screenshot of the full screen?

Step 8 - Test UI on Cloud Foundry.

when you click on the tile you get the message

I have attached a copy of the xs-app.json file.

Hi to all,
thank you for your messages. We will update this issue.
Best regards,
Merve

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.

Hi, I got the exact same error today.  Can you reopen the issue?
thanks in advance.


Create room modal

Changes
Create Room modal created and styled. Allows for valid YouTube urls to be shown.
Tickets
#39 #41

This looks fine to me.


(maint) Add github action to publish preview docs on PR

This adds a github action which will build and update the bolt docs
preview site when a PR is opened, synchronized, or reopened.

Where will the preview site be visible?

I think the same place they were viewable before? https://docs-preview.webteam.puppet.com/docs/bolt/dev/  but...now that I say that I think that site is password protected, which won't work for external folks.

How will that work if we have multiple PRs open at the same time?

@nicklewis I'll copy-paste Heston's answer directly: "I think that would be fairly difficult to achieve ATM because the web team has to set up each new directory in the s3 to point to something in Drupal - it’s a fairly manual process. We’re hoping that once our site has migrated off of Drupal (which is happening soonish) we’ll be able to build the site locally - that will probably mean more versatility. For now I think all PRs will have to push to the same staging environment on Drupal"

Can we clone all of this from somewhere else instead of committing 2k+ files?


Lint sphericart-jax

As in the title

The remaining error is
lint: commands[0]> flake8 python setup.py sphericart-torch/python sphericart-torch/setup.py sphericart-jax/python sphericart-jax/setup.py
sphericart-jax/setup.py:13:1: E302 expected 2 blank lines, found 1
sphericart-jax/setup.py:17:5: E303 too many blank lines (2)

If you don't see this locally, you might have an older version of the code. You could try to remove the .tox folder and run again.


feat: scale embeddable when SHIFT+resizing

@dwelle, what do you think about this PR?
The reason I want to allow scaling of embeddable is because currently embeddable objects only look ok at around 100% canvas scale. If you make an embeddable (e.g.: youtube video) much smaller, then the relative viewport of the embedded object becomes too small for the video to display properly. If you make it larger (e.g. a website) then the viewport becomes too large. This means that if the user uses embedded objects that he is limited to the ~100% zoom range, and cannot really make use of the 10%-3000% range (e.g.: I cannot add a tiny video that I can zoom in to view.
The PR also includes a minor fix in the RE_YOUTUBE which does not process links such as https://youtu.be/OX5_UYjXEvc?feature=shared&t=303 properly. In this case the video will start to play from the beginning instead of the 303sec mark.

[x] Implement scaling when shouldMaintainAspectRatio
[x] Block flipping on negative scale
[x] Implement iframe scaling in exportToSvg
[x] Initialize element.scale in restore
[x] Fix YouTube timestamp bug #1433

updated to support iFrame

Ooops, missed this PR.
I see what you mean, but it seems that e.g. YouTube tends to stream you low-res quality if you upscale the video using scale only: https://excalidraw-f9ejqkh8x-excalidraw.vercel.app/#json=rmhdhOdr5jguR1LGOA0lx,gDv6sZ8_VbAF-altlcmYPA (though YYMV — after I started switching from auto to 4K YT kinda stopped feeding me lowres), vs prod https://excalidraw.com/#json=K-c9wWajk4j1MJJjAmwkw,5s8svIxvjYdJFchqIeOiFw

Could be. I now use scaling extensively, not only for YT, but webpages, etc. BTW - how does YT know about the scaling? That happens external to the iFrame.


K8SPG-358 Release 2.3.0

CHANGE DESCRIPTION
Problem:
Short explanation of the problem.
Cause:
Short explanation of the root cause of the issue if applicable.
Solution:
Short explanation of the solution we are providing with this PR.
CHECKLIST
Jira

[ ] Is the Jira ticket created and referenced properly?
[ ] Does the Jira ticket have the proper statuses for documentation (Needs Doc) and QA (Needs QA)?
[ ] Does the Jira ticket link to the proper milestone (Fix Version field)?

Tests

[ ] Is an E2E test/test case added for the new feature/change?
[ ] Are unit tests added where appropriate?

Config/Logging/Testability

[ ] Are all needed new/changed options added to default YAML files?
[ ] Are the manifests (crd/bundle) regenerated if needed?
[ ] Did we add proper logging messages for operator actions?
[ ] Did we ensure compatibility with the previous version or cluster upgrade process?
[ ] Does the change support oldest and newest supported PG version?
[ ] Does the change support oldest and newest supported Kubernetes version?

Test name
Status

custom-extensions
skipped

demand-backup
skipped

init-deploy
skipped

monitoring
skipped

operator-self-healing
skipped

scaling
skipped

scheduled-backup
skipped

self-healing
skipped

start-from-backup
skipped

telemetry-transfer
skipped

upgrade-minor
skipped

users
skipped

We run 0 out of 12

commit: https://github.com/percona/percona-postgresql-operator/pull/620/commits/ad5eef722609cdfc1387f3b8ae138eced3d38d47
image: perconalab/percona-postgresql-operator:PR-620-ad5eef722

Test name
Status

custom-extensions
passed

demand-backup
passed

init-deploy
passed

monitoring
failure

operator-self-healing
passed

scaling
passed

scheduled-backup
passed

self-healing
passed

start-from-backup
passed

telemetry-transfer
passed

upgrade-minor
failure

users
passed

We run 12 out of 12

commit: https://github.com/percona/percona-postgresql-operator/pull/620/commits/ad5eef722609cdfc1387f3b8ae138eced3d38d47
image: perconalab/percona-postgresql-operator:PR-620-ad5eef722

Test name
Status

custom-extensions
passed

demand-backup
passed

init-deploy
passed

monitoring
passed

operator-self-healing
passed

scaling
passed

scheduled-backup
passed

self-healing
passed

start-from-backup
passed

telemetry-transfer
passed

upgrade-minor
failure

users
passed

We run 12 out of 12

commit: https://github.com/percona/percona-postgresql-operator/pull/620/commits/ad5eef722609cdfc1387f3b8ae138eced3d38d47
image: perconalab/percona-postgresql-operator:PR-620-ad5eef722


Unable to find executables/scripts after conda installation

Hi everyone,
I just installed modPhred via conda, and would now like to run the test data.
I would expect that I have the relevant scripts like modEncode or modAlign in the path... but I do not.
So... how do I run them/where are they?
Thanks,
Bastian

Hi, have you followed these instructions? https://modphred.readthedocs.io/en/latest/install.html

Okay, now I'm embarrassed, because this was indeed a case of RTFM.
I only followed the conda instructions, thinking you'd have modphred already in conda, and then forgot to actually clone the git repository.
I'm sorry.


chore(deps): update libp2p

Updating libp2p version

This change is

Codecov Report

Merging #2697 (159382c) into master (6c718d0) will decrease coverage by 0.01%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master    #2697      +/-   ##
==========================================
- Coverage   63.71%   63.69%   -0.02%
==========================================
  Files         236      236
  Lines       26626    26632       +6
==========================================
- Hits        16964    16963       -1
- Misses       8148     8154       +6
- Partials     1514     1515       +1

Impacted Files
Coverage Δ

pkg/p2p/libp2p/libp2p.go
61.50% <77.77%> (-0.11%)
:arrow_down:

pkg/localstore/subscription_pull.go
84.93% <0.00%> (-2.06%)
:arrow_down:

pkg/p2p/libp2p/internal/reacher/reacher.go
90.90% <0.00%> (-1.30%)
:arrow_down:

pkg/localstore/mode_put.go
58.90% <0.00%> (-1.15%)
:arrow_down:

pkg/localstore/mode_set.go
54.10% <0.00%> (-1.12%)
:arrow_down:

pkg/topology/kademlia/kademlia.go
75.79% <0.00%> (+0.17%)
:arrow_up:

pkg/topology/lightnode/container.go
96.00% <0.00%> (+2.66%)
:arrow_up:

pkg/p2p/libp2p/stream.go
82.35% <0.00%> (+11.76%)
:arrow_up:

Continue to review full report at Codecov.

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


Can I define one short key for two short key combinations ?

Can I define one custom short key both to collapse all folders in explorer and to close all editors ?

Yes, with a macro extension like https://marketplace.visualstudio.com/items?itemName=ryuta46.multi-command or others.  There is no built-in way to run multiple commands in vscode.
This is really a question for Stack Overflow.

Also see #871

Thank you so much. @ArturoDent .


bug: Overloading on First operation

Describe the bug
On my first Avante message, I got this:
Error: {
  error = {
    details = vim.NIL,
    message = "Overloaded",
    type = "overloaded_error"
  },
  type = "error"
}

This can't be right.
The Anthropic console Log says:

Where:

2.68 is "Model Latency"
1055 is Input Tokens
68 is Output Tokens

These are the expected limits. Nowhere near close:

To reproduce

Create a new Anthropic account (Tier 1)
Pay for 10$ of credit.
Open this file: https://github.com/V4G4X/nvim_config/blob/main/lua/core/plugin_configs/calltree.lua
Ask Avante: "Explain this file"

Expected behavior
It should offer me an explanation about my file.
Environment
❯ v -V1 -v
NVIM v0.10.1
Build type: Release
LuaJIT 2.1.1713773202

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "
/nix/store/4fhpydnrka255c6r30h11917y1i3lqwz-neovim-unwrapped-0.10.1/share/nvim

Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
    spec = {
        {
            "yetone/avante.nvim",
            event = "VeryLazy",
            lazy = false,
            version = false, -- set this if you want to always pull the latest change
            opts = {
                -- add any opts here
            },
            -- if you want to build from source then do `make BUILD_FROM_SOURCE=true`
            build = "make",
            -- build = "powershell -ExecutionPolicy Bypass -File Build.ps1 -BuildFromSource false" -- for windows
            dependencies = {
                "stevearc/dressing.nvim",
                "nvim-lua/plenary.nvim",
                "MunifTanjim/nui.nvim",
                --- The below dependencies are optional,
                "nvim-tree/nvim-web-devicons", -- or echasnovski/mini.icons
                "zbirenbaum/copilot.lua", -- for providers='copilot'
                {
                    -- support for image pasting
                    "HakonHarnes/img-clip.nvim",
                    event = "VeryLazy",
                    opts = {
                        -- recommended settings
                        default = {
                            embed_image_as_base64 = false,
                            prompt_for_file_name = false,
                            drag_and_drop = {
                                insert_mode = true,
                            },
                            -- required for Windows users
                            use_absolute_path = true,
                        },
                    },
                },
                {
                    -- Make sure to set this up properly if you have lazy=true
                    'MeanderingProgrammer/render-markdown.nvim',
                    opts = {
                        file_types = { "markdown", "Avante" },
                    },
                    ft = { "markdown", "Avante" },
                },
            },
        }
    },
})

This is a temporary error with anthropic
The service is temporarliy overloaded
https://docs.anthropic.com/en/api/errors

Similar bug report https://github.com/yetone/avante.nvim/issues/598

To be clear; I don't think is a bug in avante, it was just the anthropic service being temporarily overloaded


Pull Interpreters into Session class

The interpreters class is small and specialized to helping the session. Consider bringing it into the session class

After closer inspection, i think we might want to leave the interpreters as they are. Any thoughts @frozenjava ?

Closing for now, we can reopen if we decide to do this again later.


Update scenario evaluation to be optional

I think perhaps this should be made optional. I guess it's only relevant if there's a specific decision under consideration, and you're using the model or analysis to simulate different future scenarios and evaluate different management interventions.
Originally posted by @egouldo in https://github.com/egouldo/EcoConsPreReg/pull/11
https://github.com/egouldo/EcoConsPreReg/blob/b746e2518a26347b26c0cc6430d3ad613ec68e11/analysis/preregistration_template.Rmd#L66-L69

The outcomes should speak directly to the analytical objectives identified in 1.3.1. Outcomes could be qualitative or quantitative, and include descriptive statistic measures, satisficing measures or regret measures (Moallemi et al. 2019). For example, a model outcome in the top 10th percentile of worst conditions, or some minimum performance threshold for meeting a performance objective, or choosing a decision with the minimum regret or least amount of uncertainty.

To be honest, I don't know how I might apply this to my work or any future works. Maybe I do not have enough knowledge of this type of research to apply it to my basic ecology research. But this isn't to say it's not important, I just wonder if there's a way to make this accessible to non-desicion or modelling peeps. Like, how would I apply it to the woodland condition analysis?

[x] Resolve issue https://github.com/egouldo/EcoConsPreReg/pull/11

f753664e0919c0cb04295fbb02aa0bf0a9a4827f


[Frontend] Move some stats + outputs to end of pipeline

Move some stat collection and supplementary output logic to performEndOfPipelineActions to better fit the model of an inverted pipeline.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - e1770ebd0fa3c83171665bd2a81539076c2b5ac1

@swift-ci please test macOS

Build failed
Swift Test OS X Platform
Git Sha - e1770ebd0fa3c83171665bd2a81539076c2b5ac1

@swift-ci please clean test macOS

@CodaFi Hmm, how about we factor out a performCompilePipeline function and have performCompile call both it and performEndOfPipelineActions?

Build failed
Swift Test OS X Platform
Git Sha - e1770ebd0fa3c83171665bd2a81539076c2b5ac1

@swift-ci please test macOS

Build failed
Swift Test OS X Platform
Git Sha - e1770ebd0fa3c83171665bd2a81539076c2b5ac1

@swift-ci please test macOS

Build failed
Swift Test OS X Platform
Git Sha - e1770ebd0fa3c83171665bd2a81539076c2b5ac1

@swift-ci please test macOS

Build failed
Swift Test OS X Platform
Git Sha - e1770ebd0fa3c83171665bd2a81539076c2b5ac1

https://github.com/apple/llvm-project/pull/1357
@swift-ci please test

https://github.com/apple/llvm-project/pull/1357
@swift-ci please test

https://github.com/apple/llvm-project/pull/1357
@swift-ci please test Windows

Merging for now, will work on cleaning up the interface in a follow up.


Add view count to YouTube plugin

Close #76

I'm travelling and youtube is blocked so I kind of tested it by hand, but it should work.

Wow, thanks for this. :+1: I'll verify this works when I get home from work and merge it.

@S0lll0s Apparently I forgot to ask you to add your name to the CONTRIBUTORS file. If you'd like to do so, just open a pull request. :) Thanks again.

@JohnMaguire I'm working on a github plugin now, I already commited the change on the S0llll0s:github-plugin


[FT-Transformer] Test a different dataset than the one used to train the model

Hello,
I'm currently experimenting with the FT-Transformer and obtained interesting results for some of the datasets I used. I would like to know if it is possible after the model training phase to use this trained model on another dataset, to see if it can get the data label right.
If yes, how do I do this?

Hello, you need to slightly modify the code to achieve this. For example, you can do the following:

Pass the path to the checkpoint as a part of the config, and load the checkpoint here.
Then, run the script with the config used for training the model, but with the following changes: (1) add the checkpoint path (2) set n_epochs to 0 (3) change the dataset to the new one.

I did not test the above approach, but from a quick look, I think it should work.

I have some doubts about what you said, could you clarify?
1- What you mean by the “checkpoint” is the “build_x_quantile_mean_new_indices.pickle” or is it another file?
2- On the part “run the script with the config used for training the model”, is it to use the code available in the Evaluation part?
3- Would the “dataset change” be all N.npy and y.npy files, or just those relating to the test? Since I just want to evaluate the performance of a dataset with the model already trained.

I mean this checkpoint. So I recommend adding the code to load the model weights from this checkpoint right after the model is created.

If you did python bin/ft_transformer.py output/path/to/config.toml, then what I recommend is copying config.toml to any new location, adding the checkpoint path to the new config, and rerun the evaluation routine with the new config and the new script.

I assumed the following: you trained the model on the dataset data/first_dataset, and your new dataset is a different dataset data/second_dataset. So I recommend changing the path from "data/first_dataset" to "data/second_dataset"` in the new config.


Timestamp column - could you show the last update time vs. only "XX minues/hours ago"

Clearly describe the solution you'd like
Timestamp column - could you show the last update time vs. only "XX minutes/hours ago."   In case the screen has not refreshed one can validate when results were last posted.
Great job on this app!!!

Hi, thanks for filing an issue!
The relative timestamps are automatically updated every minute while the page is open, so they should always be up to date. You can see a localized absolute timestamp by hovering. If any of this behavior isn't working as expected, please reopen this issue!

Thanks!  Did not notice the hover control.


Support manual assignment of cross-validation folds

Description
This feature request pertains to the tabular module.
For some types of data, such as geospatial data, spatial autocorrelation can confound the evaluation of k-fold cross-validation. In cases like these, spatially aware fold assignment is especially useful. With this in mind, it would greatly enhance the confidence in TabularPredictor performance results if custom fold assignment IDs could be manually pre-assigned and then passed to TabularPredictor.fit().
References

Spatial validation reveals poor predictive performance of large-scale ecological mapping models
Estimating the prediction performance of spatial models via spatial k-fold cross validation
Cross-validation strategy impacts the performance and interpretation of machine learning models
Spatially autocorrelated training and validation samples inflate performance assessment of convolutional neural networks
Assessing and improving the transferability of current global spatial prediction models
blockCV: an R package for generating spatially or environmentally separated folds for k-fold cross-validation of species distribution models
SamComber/spacv: Spatial cross-validation in Python.

Thanks for creating an issue and providing a variety of links!
We do support manual group assignment. Refer to the groups parameter in https://auto.gluon.ai/stable/api/autogluon.tabular.TabularPredictor.html.
Can you try this logic and see if it satisfies your needs?
If not, could you provide a reproducible code example that uses a dataset where such manual assignment leads to an improved result? We can use this example to test potential solutions.

Hi, following up on this.
Could you please confirm if the mentioned "groups" parameter in the TabularPredictor class could be used to perform a typical K-fold cross-validation (or stratified CV) where the AutoGluon models will be trained in all groups/folds except by one (the validation fold)?
Below is a random df with a "Fold" column that I want to use to define the training/validation splits used for the internal bagging of AutoGluon.
Is my interpretation correct? or am I misunderstanding the documentation?
Thanks a lot for your reply!
import pandas as pd
import numpy as np
from sklearn.model_selection import StratifiedKFold
from autogluon.tabular import TabularDataset, TabularPredictor

# Generate random data
np.random.seed(42)
data_size = 300
feature1 = np.random.normal(loc=0, scale=1, size=data_size)
feature2 = np.random.normal(loc=5, scale=2, size=data_size)
groups = np.random.choice(['Group_A', 'Group_B', 'Group_C'], size=data_size)

# Create DataFrame
df = pd.DataFrame({
    'Feature1': feature1,
    'Feature2': feature2,
    'Group': groups
})

# Apply Stratified K-Fold
skf = StratifiedKFold(n_splits=5, shuffle=True, random_state=42)
fold_column = np.zeros(data_size, dtype=int)

for fold, (train_index, test_index) in enumerate(skf.split(df, df['Group'])):
    fold_column[test_index] = fold

#Add the Fold column (to be used as "groups" in TabularPredictor)
df['Fold'] = fold_column

#AutoGluon using user-defined stratified K-fold CV
train_data = df
label = 'Group'
groups = 'Fold'
save_path = 'data/test_folds_automl/'

predictor = TabularPredictor(label=label, path=save_path, groups=groups, eval_metric='balanced_accuracy',
                             sample_weight='auto_weight').fit(train_data, time_limit=80, auto_stack=True)

Thanks @dluks! We don't support getting std dev of folds yet, but it is something we plan to add in an upcoming release.
For the NeuralNetwork issue, a reproducible example in a new ticket would be great if you find the time so we can take a closer look. I don't expect this to be the case, and am surprised this would cause an issue. It is probably the time limit.
Regarding the GPU hanging, this is fixed in pre-release / mainline and will be part of the next release.
@alberto-jj Yes should work.


Add Status expiration field in SetUserCustomStatus

This PR closes #482.  You can set an expiration timestamp to a status. I've been using it here for few weeks now and it works pretty well

LGTM, though this function might need some reworks in the future if it keeps growing.


Synthetic Graph Data Generator

add synthetic graph data generator class
add tests for synthetic graph generation (unit tests, visual plot tests)
add example notebook in examples/synth-graph-data
add example data for notebook in examples/synth-graph-data

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Pierre-Nicolas seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Melusine Buster loop doesn't work

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
JP
FGA build number
1951
Describe the bug
So this bug happened when at turn 2
When the game match start it check the servant info first before doing anything right?
so that also happened in turn 2 with Melsuine due to her changing form
and then the script just go for command card instead of picking skill first
Video
I use andriod 13 so can't record video
Device model
Oneplus 8
Android version
13
Screen size
6.55 inches
RAM
12

Sounds like either fgo locked up during melusine change or the change took too long and fga just continue with its operations while the change is happening

Ok I not sure if it solve or not, but for some reason I try to record while doing it the new problem occur which is it check "servant 2" 2 times instead then miss out the skill it supposed to press
I decide to clear the script and make a new one instead since the farm won't work if miss that skill
however when I try the new script which should be the same one (I only clear the command and do it again the export on same one) it everything worked again
so......I don't know what is the problem anymore
PS. also you seem to misunderstand but Melusine is changing form in wave 1 however the problem is it check her info again in wave 2 for some reason....not that it a problem anymore for some reason though

If you're still not having problems 3 days from now then I'm gonna close this has solved

Melusine isn't handled by FGA right now so it treats the new Melusine sprite as a sub-member. That's also why it takes new screenshots.
Would have to implement some special logic to overwrite the old sprite with the new one.
Honestly, I'm surprised that you're the first one to open an issue about Melusine.

Melusine isn't handled by FGA right now so it treats the new Melusine sprite as a sub-member. That's also why it takes new screenshots.
Would have to implement some special logic to overwrite the old sprite with the new one.
Honestly, I'm surprised that you're the first one to open an issue about Melusine.

I was using her when koyanskaya came out and fga would just recheck her portrait during the first wave and it didn't mess with my runs any of the 100th of times I used her back then so I'm guessing that's why no one reported a issue until now

For me it works but complains about failing to detect servant cards. It does probably make sense to special case Melusine (don't think anyone is farming with Hyde).


Update Scala and SBT versions

This PR drops support for Scala 2.10 and adds support for Scala 2.12. It also drops tests with Java 7 (though the compiled artefacts may still be compatible).

This PR is now available in the 0.3.0-SNAPSHOT version, available from Sonatype Nexus:
// build.sbt
scalaVersion := "2.12.4"

libraryDependencies += "co.theasi" %% "plotly" % "0.3.0-SNAPSHOT"

resolvers +=
  "Sonatype OSS Snapshots" at "https://oss.sonatype.org/content/repositories/snapshots"


octave: jsonencode(...PrettyPrint...) leads to warning

Describe the bug
According to the Octave documentation of jsonencode the function supports pretty printing of the json data. Unfortunately this does not work with the version of octave build for Nix.
Steps To Reproduce
Steps to reproduce the behavior:

start octave
input at prompt jsonencode ([true; false], "PrettyPrint", true)
get warning: jsonencode: support for the "PrettyPrint" option of RapidJSON was unavailable or disabled when Octave was built

Expected behavior
jsondecode fully supports option PrettyPrint.
Screenshots
N/A
Additional context
I couldn't figure out whether the Octave or RapidJSON package was built incorrectly.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.51, NixOS, 23.05 (Stoat), 23.05.20230907.4f77ea6`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.5`
 - channels(root): `"home-manager-23.05.tar.gz, nixos-23.05"`
 - channels(user): `""`
 - nixpkgs: `/nix/store/splp4lqr6n115q125nhqk2qmg81hsk1r-source`

Priorities
Add a :+1: reaction to issues you find important.

For the PrettyPrint feature of Octave to work correctly, you'd need to cherry-pick the following two PRs for RapidJSON:
https://github.com/Tencent/rapidjson/issues/909
https://github.com/Tencent/rapidjson/issues/1720


Missing LightningModule datamodule reference

🐛 Bug
This docs snippet does not work:
https://pytorch-lightning.readthedocs.io/en/latest/common/lightning_module.html#datamodule
To Reproduce
def test_bug(tmpdir):
    class TestModel(BoringModel):
        def configure_optimizers(self):
            # works
            len(self.trainer.datamodule.train_dataloader())
            # does not
            len(self.datamodule.train_dataloader())

            return super().configure_optimizers()

    dm = BoringDataModule()
    trainer = Trainer(default_root_dir=tmpdir, fast_dev_run=True)
    trainer.fit(TestModel(), datamodule=dm)

line 128, in configure_optimizers
    num_training_samples = len(self.datamodule.train_dataloader())
AttributeError: 'NoneType' object has no attribute 'train_dataloader'

Expected behavior
self.datamodule works
Alternatives
Update docs
Environment
master
Additional context
Reported by Brian Staber in Slack

Can I work on this?

@carmocca is the model expected to have a reference to the data module during training?

Can I work on this?

Sure!

@carmocca is the model expected to have a reference to the data module during training?

I guess given the docs link above and the fact that it has a property for it
https://github.com/PyTorchLightning/pytorch-lightning/blob/f29ecbfd909ff431ef837fcc8ebff451e897cb0b/pytorch_lightning/core/lightning.py#L164-L170
Although I wouldn't personally mind removing this and forcing access through the trainer.

Dear @mesejo,
Could you make a PR for this ?
Best,
T.C

Hi @carmocca, when you said:

I personally wouldn't mind removing this and forcing access through the trainer.

Were you referring to the entire datamodule property?

Were you referring to the datamodule property?

I was referring to deprecating model.datamodule in favor of model.trainer.datamodule
cc: @PyTorchLightning/core-contributors Any opinions about this? Should we keep both ways?

I would prefer if we don't attach the datamodule to the LightningModule and instead reference it through trainer.

Okay, let's deprecate it then and update the docs/ocurrences


Be able to set locale for format money() and numeric() values and make use of Laravel Number helper functions

Description
For multiple projects we want to be able to format the money() and numeric() methods using a locale, independent from the app()->getLocale() configuration. For our customers it's pretty common to have the whole admin interface in the english language but still have the date and currency / number formatting in the local (eg 'de / 'nl') format.
Next to that I want to be able to set the currency and numeric formatting for the whole application or per user session. To support this, but don't break current implementations the default locale fallback is still app()->getLocale() and also the traditional PHP's number_format() is still supported to cover current implementations and make the impact of the change minimal.
The implementation allows us to use the new locale attribute in money() and numeric() methods. The implementation is making use of the default Laravel Number helper functions.
Formatting
Number formatting
The numeric() method allows you to format a summary as a number.
Using Number::format() method from Laravel:
use Filament\Tables\Columns\Summarizers\Average;
use Filament\Tables\Columns\TextColumn;

TextColumn::make('rating')
    ->summarize(Average::make()->numeric(
        decimalPlaces: 2,
        locale: 'de',
    ))

To use the traditional PHP's number_format(), you should define the decimalSeparator attribute:
use Filament\Tables\Columns\Summarizers\Average;
use Filament\Tables\Columns\TextColumn;

TextColumn::make('rating')
    ->summarize(Average::make()->numeric(
        decimalPlaces: 0,
        decimalSeparator: '.',
        thousandsSeparator: ',',
    ))

Currency formatting
There is an optional locale argument for money() to set a locale for the currency formatting.
use Filament\Tables\Columns\Summarizers\Sum;
use Filament\Tables\Columns\TextColumn;

TextColumn::make('price')
    ->summarize(Sum::make()->money('EUR', locale: 'de'))

Next to that, it is also possible to set the locale for formatting globally
use Illuminate\Support\Number;

/**
 * Bootstrap any application services.
 */
public function boot(): void
{
    Number::useLocale('de');
}

For a new Filament 3.4 or 4.x implementation I would suggest to totally move to the Laravel  Number helper functions and also implement Number::abbreviate() and Number::forHumans() and support more options that the helpers support too. Maybe even remove the decimal symbol support etc.

[ ] Visual changes (if any) are shown using screenshots/recordings of before and after.

Code style

[x] composer cs command has been run.

Testing

[x] Changes have been tested.

Documentation

[x] Documentation is up-to-date.

Anyone an idea why phpstan is failing with this PR? locally phpstan is running fine and no code changes are done in forms/src/Commands/Concerns/CanGenerateForms.php

the newest Larastan (updated a few hours ago) larastan/larastan (v2.8.1 => v2.9.0) Introduces a few extra checks. Because of that the phpstan checks are failing on files non related to this PR.

made some minor changes and ignored one extra line to solve the new introduced phpstan errors

If we're going to use the new Number class from Laravel, the required Laravel version should probably be bumped to ensure users have it installed in their project without breaking if they only update Filament?

If we're going to use the new Number class from Laravel, the required Laravel version should probably be bumped to ensure users have it installed in their project without breaking if they only update Filament?

According the documentation the minimal required version of Laravel is 10.x if i'm not mistaken right?

According the documentation the minimal required version of Laravel is 10.x if i'm not mistaken right?

The Number helper was only introduced recently in a minor release of Laravel 10.

According the documentation the minimal required version of Laravel is 10.x if i'm not mistaken right?

The Number helper was only introduced recently in a minor release of Laravel 10.

Good point! I just updated the PR to use a fallback when the Number helpers where not found.

You don't need method existence checks, you just need to bump the minimum minor Laravel version

You don't need method existence checks, you just need to bump the minimum minor Laravel version

@danharrin can you please give me a hint how to bump the minimum minor version? Is this just in the documentation or should we also require something in the composer.json ?

I have rewritten this to remove breaking changes such as removing the ability to remove the decimal and thousands separators by setting them to null in #11545


TypeError: Cannot read property 'getConstants' of null,

Hi, I am trying to convert an image url prefixed with https:// into base64 string using the react-native-blob-util, however I got this error after installing and also doing a pod install

react-native-blob-util Version: 0.17.0

RN Version: 0.70.4
   const handleShare = async () => {
     const meetupImageUrl = getImageUrl(meetup.cover_photo, ImageVariant.preview);
     await ReactNativeBlobUtil.fetch('GET', meetupImageUrl)
        .then((res) => {
         let status = res.info().status;
          if (status == 200) {
              // the conversion is done in native code
              let base64Str = res.base64();
              console.log('base64Str', base64Str);
          }
          else {
              // handle other status codes
          }
      })
      // Something went wrong:
      .catch((errorMessage, statusCode) => {
          // error handling
      });
  console.log('meetupImageUrl', meetupImageUrl)
 };

Did you install the package (and link) and rebuild the app and cleaned build caches first if needed?
Sounds like it didn't load the lib correctly.

Did you install the package (and link) and rebuild the app and cleaned build caches first if needed?
Sounds like it didn't load the lib correctly.

Yes I took this exact steps

To reproduce:
npx react-native init RNBU204 --version="0.70.6"
cd RNBU204
npm i --save react-native-blob-util
echo "import RNFetchBlob from 'react-native-blob-util';" >> App.js
npm run ios

The Metro window shows the following errors:
 ERROR  TypeError: Cannot read property 'getConstants' of null, js engine: hermes
 ERROR  Invariant Violation: Failed to call into JavaScript module method AppRegistry.runApplication(). Module has not been registered as callable. Registered callable JavaScript modules (n = 10): Systrace, JSTimers, HeapCapture, SamplingProfiler, RCTLog, RCTDeviceEventEmitter, RCTNativeAppEventEmitter, GlobalPerformanceLogger, JSDevSupportModule, HMRClient.
        A frequent cause of the error is that the application entry file path is incorrect. This can also happen when the JS bundle is corrupt or there is an early initialization error when loading React Native., js engine: hermes

For iOS, the "fix" is to run pod install from the ios folder and rebuild the app.  To see that in action, close Metro and run:
cd ios; pod install; cd ..
npm run ios

This probably should be documented somewhere, since the current documentation strongly suggests npm install suffices when working with RN 0.70 ("manual linking should not be required" and the actual react-native link command was removed)

@SheetJSDev I'm going to try out your suggestions. Although I no longer need this package as I found another solution

@SheetJSDev I'm going to try out your suggestions. Although I no longer need this package as I found another solution

What solution you found as I am getting the exact error when I tried this package

For iOS, the "fix" is to run pod install from the ios folder and rebuild the app:
cd ios; pod install; cd ..
npm run ios

For iOS, the "fix" is to run pod install from the ios folder and rebuild the app:
cd ios; pod install; cd ..
npm run ios

For android??

Our React Native + RNBU demo (https://docs.sheetjs.com/docs/demos/mobile/reactnative#demo select "RNBU" in step 4) builds and runs in the simulator for Android 12 (S) Platform 31 Pixel 5.  This works in RN 0.70.6 + RNBU 0.17.1 and in RN 0.70.4 + RN 0.17.0 (the versions used by @dekunleroy).
Can you test if the same issue shows up in a new project?  Try:
npx react-native init RNBU204 --version="0.70.4"
cd RNBU204
npm i --save<EMAIL_ADDRESS>echo "import RNFetchBlob from 'react-native-blob-util';" >> App.js
npm run android

Issue can be reproduced in Expo Snack (both iOS and Android).  https://github.com/expo/snack/issues/385 tracking issue.

Issue can be reproduced in Expo Snack (both iOS and Android). expo/snack#385 tracking issue.

Yeah but this is expo. I'm using native code and as far as I know am not part of the Expo SDK... So it won' work.

Expo Snack issue was raised when trying to make our react native demo work with the snack.  It happened to have the same message but
This issue is reproducible locally on iOS using the first snippet from an up-thread comment https://github.com/RonRadtke/react-native-blob-util/issues/204#issuecomment-1374646366 :
npx react-native init RNBU204 --version="0.70.6"
cd RNBU204
npm i --save react-native-blob-util
echo "import RNFetchBlob from 'react-native-blob-util';" >> App.js
npm run ios

Tested on iPhone 14 (iOS 16.2), the error is
 ERROR  TypeError: Cannot read property 'getConstants' of null, js engine: hermes

The project issue is resolved by running pod install from the ios directory and rebuilding:
cd ios; pod install; cd ..
npm run ios

It's hard to tell if this is a bug in RNBU or merely an oversight in the docs.  The Installation section of the docs does not mention the pod install step.

I would add the pod install to the docs. It makes sense that a pod install might be required.
But we had some issues on ios side too: https://github.com/RonRadtke/react-native-blob-util/pull/219

Same issue here when I'm trying to start the APP.
Any solution?
Thanks

Same issue

I tried adding mock code into setup file.
jest.mock('react-native-blob-util', () => { return { DocumentDir: () => {}, polyfill: () => {}, }; });
Also downgraded the library to version 0.17.0 but still facing the same issue :(
TypeError: Cannot read properties of null (reading 'getConstants')

 at Object.<anonymous> (node_modules/react-native-blob-util/fs.js:14:39)
 at Object.<anonymous> (node_modules/react-native-blob-util/index.js:8:1)

I have the same issue with Android. Any solutions? instrument.js cannot read property getConstants of null

same issue

I sorted this by creating when running tests with jest by creating a manual mock
__mocks__/react-native-blob-util.js
containing simply:
export default jest.fn();

Which covered off tests which weren't actually touching react-native-blob-util.
For tests of modules which did use react-native-blob-util, i mocked as follows:
jest.mock("react-native-blob-util", () => ({
  android: {
    actionViewIntent: jest.fn(),
  },
  ios: {
    openDocument: jest.fn(),
    previewDocument: jest.fn(),
  },
  config: jest.fn(),
  fs: {
    dirs: {
      DocumentDir: "yourdocumentdir",
      DownloadDir: "yourdownloadsdir",
    },
  },
}));

if you face this issue when trying to test your code , put this code in setup file :
jest.mock('react-native-blob-util', () => { return { DocumentDir: () => {}, polyfill: () => {}, }; });

The following worked for me:
jest.mock('react-native-blob-util', () => {
  return {getConstants: () => {}, polyfill: () => {}};
});

P.S. I faced this error only while running jest.

worked for me!
it was the issue of the version
i was using 0.19.6 which was released about 2 weeks which was the problem
i degraded the react-native-blob-util to  version 0.19.5 and is working fine now.

Same issue with Android

Same here

Same here

In my case, I got these errors trying to use this with Expo Go, instead, you have to use an Expo Development build. You can create a dev build, sideload it onto your emulator / test device, and have it grab the latest js bundle automatically (just like expo go), by running npx expo run:android

worked for me!
it was the issue of the version i was using 0.19.6 which was released about 2 weeks which was the problem i degraded the react-native-blob-util to version 0.19.5 and is working fine now.

not working with expo SDK 49 with expo go.

worked for me!
it was the issue of the version i was using 0.19.6 which was released about 2 weeks which was the problem i degraded the react-native-blob-util to version 0.19.5 and is working fine now.

not working with expo SDK 49 with expo go.

It will not work with Expo Go, you need to use Expo Development Builds.

jest.mock('react-native-blob-util', () => {
return {getConstants: () => {}, polyfill: () => {}};
});

You can put this in the jest.config.js file, or create a specific file for it like __mocks__/react-native-blob-util.ts
Then update your jest.config.js as :
module.exports = {
  ...
    setupFilesAfterEnv: [
       ...
        './__mocks__/react-native-blob-util.ts',
    ],
};

if you face this issue when trying to test your code , put this code in setup file :
jest.mock('react-native-blob-util', () => { return { DocumentDir: () => {}, polyfill: () => {}, }; });

what do file do i put this information at ?

the setup file or you also can call it mock file, it's a custom file that you will create once set up a project, this file will be linked with jest.config.js and will contain all general packages mocks or any general test functions that you want to apply them on all test cases that you have within the project,
hope this could help you.

Same issue on Android.
Any solution?
"react-native-blob-util": "0.19.4" and "react-native": "0.72.10",

if you face this issue when trying to test your code , put this code in setup file :
jest.mock('react-native-blob-util', () => { return { DocumentDir: () => {}, polyfill: () => {}, }; });
if you don't have the setup file you can add it like this:

create a new file setup/setup.test.ts inside your __test__ directory

link it with jest.config.js by adding this code   setupFiles: ['<rootDir>/__tests__/setup/setup.test.ts'], testPathIgnorePatterns: [ '<rootDir>/__tests__/setup', ... ],

stop the metro if it was running and re-run the app again

now you can add any general mock like i mentioned at the beginning or test functions inside it to be applied on all your test cases.

I'm using expo, getting same error " TypeError: Cannot read property 'getConstants' of null, js engine: hermes ",...
I tried to follow steeps as above mentioned but I couldn't find "setup file", even tried to create as you said but there is no  __test__ directory in my project....... what to do next please help

Same issue on Android development build. Any solution?
"react-native-blob-util": "^0.19.9"
"react-native": "0.74.2"
"expo": "^51.0.9"

can anyone please provide the accurate solution to this ? react-native-blob-util is giving this error

Same here


[iOS][Fixed] Data race related to reading/writing to AllocationTestModule.valid

This entails making AllocationTestModule and Objective-C++ file in order to make use of std::atomic. Getters and setters are then implemented to make sure and underlying atomic bool is accessed instead of a synthesized property.

Summary:
While the data race was present in test code, it would make it more difficult to spot more severe data races with the TSan. Also, getting rid of a data race is always good.
Changelog:
[iOS][Fixed] - Data race related to access of AllocationTestModule.valid
Test Plan:
RCTAllocationTests will test the implementation of AllocationTestModule.

It's unclear what problem this PR is solving,. If access to this property needs to be atomic (but it's unclear why), consider using @property (atomic, [...] instead of the current nonatomic.

Running this test with TSan enabled and runtime issues breakpoint in Xcode will reveal concurrent read/write to the mentioned property, which is in this case classified as a data race by TSan. I haven't studied the reasons for concurrent access, but it seems to be related to code fundamental to how RCTBridge works.
That said, I can certainly alter the proposed changes to make use of the atomic property attribute. Will come back to you soon with suggested changes.

Here is the TSan output when valid is not atomic:
==================
WARNING: ThreadSanitizer: data race (pid=75074)
  Write of size 1 at 0x000106a04348 by thread T8:
    #0 -[AllocationTestModule setValid:] <null> (RNTesterUnitTests:arm64+0x582c0)
    #1 -[AllocationTestModule invalidate] <null> (RNTesterUnitTests:arm64+0x58034)
    #2 __26-[RCTCxxBridge invalidate]_block_invoke_2 <null> (RNTesterUnitTests:arm64+0x41adf4)
    #3 __tsan::invoke_and_release_block(void*) <null> (libclang_rt.tsan_iossim_dynamic.dylib:arm64+0x77ee0)
    #4 _dispatch_client_callout <null> (libdispatch.dylib:arm64+0x3974)

  Previous read of size 1 at 0x000106a04348 by main thread:
    #0 -[AllocationTestModule isValid] <null> (RNTesterUnitTests:arm64+0x581b4)
    #1 -[RCTAllocationTests testModulesAreInvalidated] <null> (RNTesterUnitTests:arm64+0x59d60)
    #2 __invoking___ <null> (CoreFoundation:arm64+0x13371c)

  Location is heap block of size 16 at 0x000106a04340 allocated by main thread:
    #0 calloc <null> (libclang_rt.tsan_iossim_dynamic.dylib:arm64+0x4fc30)
    #1 _malloc_type_calloc_outlined <null> (libsystem_malloc.dylib:arm64+0xf488)
    #2 __invoking___ <null> (CoreFoundation:arm64+0x13371c)

  Thread T8 (tid=9283099, running) is a GCD worker thread

Seems like it is related to invalidating the bridge.
I have now resorted to making use of an atomic Objective-C property. Please see the changes @javache

Thanks! Please update the PR description to make it reflect the changes you've made.


Add MessageBag remove method

This PR adds the ability to remove an existing message from the MessageBag using the key as parameter.
Scenario: I have an unversioned API that needs to support different parameter names temporarilly. I wanted to use a combination of the prepareForValidation and after validation in the FormRequest to achieve this, but I'm unable to replace the validation errors keys.
Here is a discussion around this topic.

Can't add a method to a contract on a point release.

OK. Is this something that can be added in another release? If so I could open this PR to master branch. Thanks.


Added picture when thumbnail is missing and implemented manual entry of book info.

Implemented manually entering book information and also added a picture when thumbnail is non-existent.

Mycket trevligt, bra jobbat!


Setting individual terraform rules compared to using terraform plugin with recommended settings

In our tflint config file we specify the 16 terraform rules available, which are:

terraform_comment_syntax
terraform_deprecated_index
terraform_deprecated_interpolation
terraform_documented_outputs
terraform_documented_variables
terraform_empty_list_equality
terraform_module_pinned_source
terraform_module_version
terraform_naming_convention
terraform_required_providers
terraform_required_version
terraform_standard_module_structure
terraform_typed_variables
terraform_unused_declarations
terraform_unused_required_providers
terraform_workspace_remote

The one marked bold are part of the "recommended" presets that can be used with the terraform plugin
plugin "terraform" {
  enabled = true
  preset  = "recommended"
}

So why do we set the rules explicitly instead of using the recommended settings with the plugin? Also, should we use the optional  rules that are not part of the recommended preset all the time or should we mark them optional in our tflint configuration as well?
To me using the terraform plugin has a few advantages above defining the rules individually:

easier maintenenace: We always follow the recommended ruleset and don't have to regularly update individual rules
following best practices: We always follow the recommended ruleset

@mhmdio what's your opinion on this?

looks nice, we can create separate rules files for each use case? what do you think?


Add copy/paste button for Host register link

When registering a new host, you need to copy and paste the registration URL from the browser to your terminal. Having a button in the UI for the copy functionality would be nice.
Currently, if you just double click the text to select it, you catch a newline character and pasting it in the terminal causes it to immediately execute. I'd rather have it pasted without the newline so that I can review it before hitting enter.

+1

Fixed in #76


Feature/create a popup alert view #17

Add

An Injectable Alert Popup View.
Make it as a Generic View, that can be shown on top of any Parent View
Add variables to add the title, description, and the action button.

How to use:
Just use the .alert modifier

You can use 3 types of Popups.

Without Buttons (but you can add action to X button)
With Dismiss Button - just one button (u can customise it setting up title, colour, action)
With two buttons (u can customise it setting up title, colour, action)

This is in app example:

@Shubham0812

@Shubham0812 what's wrong with the test?

Look good, good job @oceaniswater  😄


Add metrics and logging to sdk

We would actually need to use a different logging ecosystem than slong, in #156  you can see we are going to use tracing crate https://tracing.rs https://tokio.rs/#tk-lib-tracing. We can merge this Into my branch, but most of the work will be replaced with tracing.


Unexpected error encountering in All Security Groups of cloud tenant

Version:
fine-3.20170808115538_09b4f87
Steps to Reproduce:

Add cloud provider first (Openstack RHOS7-GA recommended)
Navigate to Compute -> Cloud -> Tenants
Click on 'admin' from 'Cloud Tenants' table
Click on 'Security Groups' from 'Relationships' table
Select one security_groups from 'All Security Groups' table and click on 'Edit Tags'

@Yadnyawalkya please open an [UPSTREAM] BZ for this if not done already.

@dclarizio, Copy maintained at https://bugzilla.redhat.com/show_bug.cgi?id=1494349.


Titles of Duplicate and Edit Icon of the dropped fields

Edit has the wrong text of Hide
Dupplicate has undefined title
My locale is french but I didn't find the translation which I think are missing

These fields likely need translation.

im not able to reproduce this issue and it seems quite old so closing.


✨ convert field CRDUpgradeSafetyPolicy to be an enum

Description

Fixes #1145

Reviewer Checklist

[x] API Go Documentation
[x] Tests: Unit Tests (and E2E Tests, if appropriate)
[x] Comprehensive Commit Messages
[x] Links to related GitHub Issue(s)

@everettraven Do you know why this is failing the apidiff check? Is there something I'm missing?

@everettraven Do you know why this is failing the apidiff check? Is there something I'm missing?

@yashoza19 this is a breaking change to the API surface. I would expect that check to fail in this case. It is not a required check, but one that is in place to grab our attention when API changes are being made. Once we cut our stable v1.0.0 release, we will likely convert it to a required check


typings: Allow numeric parameters to models.application.has()

Resolves: #567
See: https://github.com/resin-io/resin-sdk/blob/f5de14600579f8e0df1c4b550b4fd185611ea123/lib/models/application.coffee#L329
See: https://github.com/resin-io/resin-sdk/blob/f5de14600579f8e0df1c4b550b4fd185611ea123/tests/integration/models/application.spec.coffee#L196
Change-type: patch
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>

Closing in favor of #570


[Refactor] Make dropout layer optional in MLP

Summary:
The transformer layer of Multigen uses an MLP block without dropout so we want to make dropout optional in the current MLP layer when the dropout prob is provided as 0.0.
Test Plan:
(torchmm) langong-mbp:gpt_attention langong$ python -m pytest --cov=torchmultimodal/modules/layers/ test/modules/layers/test_mlp.py -vv
================================================================================ test session starts =================================================================================
platform darwin -- Python 3.8.13, pytest-7.1.2, pluggy-1.0.0 -- /Users/langong/local/miniconda3/envs/torchmm/bin/python
cachedir: .pytest_cache
rootdir: /Users/langong/gpt_attention
plugins: cov-3.0.0
collected 6 items

test/modules/layers/test_mlp.py::TestMLP::test_activation_and_normalization PASSED                                                                                [ 16%]
test/modules/layers/test_mlp.py::TestMLP::test_dropout_default PASSED                                                                                [ 33%]
test/modules/layers/test_mlp.py::TestMLP::test_no_dropout PASSED                                                                                [ 50%]
test/modules/layers/test_mlp.py::TestMLP::test_no_hidden_layers PASSED                                                                                [ 66%]
test/modules/layers/test_mlp.py::TestMLP::test_pass_hidden_dims PASSED                                                                                [ 83%]
test/modules/layers/test_mlp.py::TestMLP::test_torchscript PASSED                                                                                [100%]

---------- coverage: platform darwin, python 3.8.13-final-0 ----------
Name                                                   Stmts   Miss  Cover
--------------------------------------------------------------------------
torchmultimodal/modules/layers/attention.py               97     97     0%
torchmultimodal/modules/layers/codebook.py                81     81     0%
torchmultimodal/modules/layers/conv.py                    74     74     0%
torchmultimodal/modules/layers/mlp.py                     23      1    96%
torchmultimodal/modules/layers/normalizations.py           7      7     0%
torchmultimodal/modules/layers/position_embedding.py      38     38     0%
torchmultimodal/modules/layers/transformer.py            133    133     0%
--------------------------------------------------------------------------
TOTAL                                                    453    431     5%

=========================== 6 passed in 1.82s ====================

Stack from ghstack (oldest at bottom):

-> #84
#69
#68
#67

Differential Revision: D37116648

@ankitade In our MLP the dropout argument is a floating number with default value 0.3. 0.0 is compatible with the floating type and tells the MLP that the caller module does not need a dropout layer.

@langong347 has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

its ok for this case, but in general we need not change a model just for alignment with ckpt (instead do the opposite of it)

@ankitade  No, it's not due to checkpointing. The MLP block in the transformer layer does not require dropout. Our existing MLP module did not have that flexibility to opt out of dropout.

0 dropout is anyways same as no dropout right?

0 dropout is anyways same as no dropout right?

Theoretically yes. But it'd be cleaner to simply leave the layer out.


[Task] Document how to create the JSON/Lenra Routes

What should be done
I tried to create a new app using a JSON Route.
For now, the template only define the "rootView", we should replace this with the "/" LenraRoute and document how to create a JSON Route.
export const rootView = "main";

Done


Whether to support the Chinese

ok, I did a fix (sorry it took so long) give it a try again if you have the time and let me know if it works now.


Add scrive/log

https://github.com/scrive/log/issues/19

tests for log require elasticsearch instance at the localhost
Is it ok to add a it's installation to debian-bootstrap.sh, would benefit bloodhound and katip-elasticsearch.

@snoyberg your thoughts on installing elasticsearch on the build server?


ci: add ci support for python 3.11

Add checks for Python 3.11.

We paired on this, and it seems to be working!


Ingestion service data delivery status

Ingestion service delivery status
Users could track ingestion service response by subscribing to transmission event. Notification is sent to subscriber once SDK receives response from ingestion service. Subscriber could extract information from ingestion response. For example, following metrics could be extracted.

How much traffic was accepted/rejected by ingestion service?
What instrumentation keys were associated with the traffic?

Changes
(Please provide a brief description of the changes here.)

Event handler TransmissionStatusEvent in Transmission is wired up through ServerTelemetryChannel, subscriber will utilize a public property in ServerTelemetryChannel to attach the event

// Sample
class Program
    {
        static void Main(string[] args)
        {
            var config = TelemetryConfiguration.CreateDefault();

            ServerTelemetryChannel channel = new ServerTelemetryChannel();
            channel.Initialize(config);
            config.TelemetryChannel = channel;
            config.InstrumentationKey = "IKEY";
            channel.TransmissionStatusEvent += delegate (object sender, TransmissionStatusEventArgs transmissionStatusEventArgs)
            {
                var transmission = sender as Transmission;

                if (transmissionStatusEventArgs.Response.StatusCode != (int)HttpStatusCode.PartialContent)
                {
                    Console.WriteLine($"Status Code: {transmissionStatusEventArgs.Response.StatusCode}");
                }
                else
                {
                    BackendResponse backendResponse = GetBackendResponse(transmissionStatusEventArgs.Response.Content);

                    Console.WriteLine($"ItemsAccepted: {backendResponse.ItemsAccepted}");
                    Console.WriteLine($"ItemsReceived: {backendResponse.ItemsReceived}");

                    foreach (var error in backendResponse.Errors)
                    {
                        Console.WriteLine($"Error Index: {error.Index}");
                        Console.WriteLine($"Error Message: {error.Message}");
                    }
                }

                var iKeyGroup = transmission.TelemetryItems.GroupBy(x => x.Context.InstrumentationKey);

                foreach (var telemetry in iKeyGroup)
                {
                    Console.WriteLine($"Ikey: {telemetry.Key}, Count: {telemetry.Count()}");
                }

            };

            var client = new TelemetryClient(config);
            SendTelemetryToApplicationInsights(client);
            Console.ReadLine();
        }

        private static void SendTelemetryToApplicationInsights(TelemetryClient client)
        {
            EventTelemetry eventTelemetry1 = new EventTelemetry("Event1");
            eventTelemetry1.Context.InstrumentationKey = "IKEY_1";
            client.TrackEvent(eventTelemetry1);

            EventTelemetry eventTelemetry2 = new EventTelemetry("Event2");
            eventTelemetry2.Context.InstrumentationKey = "IKEY_2";
            client.TrackEvent(eventTelemetry2);

            EventTelemetry eventTelemetry3 = new EventTelemetry("Event3");
            eventTelemetry3.Context.InstrumentationKey = "6c49c07c-e95c-48fe-8a7b-eff230955cc5";
            client.TrackEvent(eventTelemetry3);

            EventTelemetry eventTelemetry4 = new EventTelemetry("Event3");
            eventTelemetry4.Context.InstrumentationKey = "IKEY_2";
            client.TrackEvent(eventTelemetry4);

            EventTelemetry eventTelemetry5 = new EventTelemetry("Event5");
            eventTelemetry5.Context.InstrumentationKey = "IKEY_1";
            client.TrackEvent(eventTelemetry5);

            client.Flush();
        }

        [DataContract]
        private class BackendResponse
        {
            [DataMember(Name = "itemsReceived", IsRequired = true)]
            public int ItemsReceived { get; set; }

            [DataMember(Name = "itemsAccepted", IsRequired = true)]
            public int ItemsAccepted { get; set; }

            [DataMember(Name = "errors")]
            public Error[] Errors { get; set; }

            [DataContract]
            private class Error
            {
                [DataMember(Name = "index")]
                public int Index { get; set; }

                [DataMember(Name = "statusCode")]
                public int StatusCode { get; set; }

                [DataMember(Name = "message")]
                public string Message { get; set; }
            }
        }

        private static BackendResponse GetBackendResponse(string responseContent)
        {
            BackendResponse backendResponse = null;
            DataContractJsonSerializer Serializer = new DataContractJsonSerializer(typeof(BackendResponse));

            try
            {
                if (!string.IsNullOrEmpty(responseContent))
                {
                    using (MemoryStream ms = new MemoryStream(Encoding.Unicode.GetBytes(responseContent)))
                    {
                        backendResponse = Serializer.ReadObject(ms) as BackendResponse;
                    }
                }
            }
            catch
            {
                backendResponse = null;
            }

            return backendResponse;
        }
    }

For another example, please refer test method TestTransmissionStatusEventWithEventsFromMultipleIKey in TransmissionTest.cs

Events Notification is initiated on SendAsync completion (once response is received from ingestion service) in Transmission. This passes Sender as Microsoft.ApplicationInsights.Channel.Transmission  and TransmissionStatusEventArgs with Microsoft.ApplicationInsights.Extensibility.Implementation.HttpWebResponseWrapper to subscriber
Subscriber is responsible to handle event and extract information from Transmission and HttpWebResponseWrapper
If there are no subscriber, event notification won’t be initiated

Checklist

[X] I ran Unit Tests locally.
[X] CHANGELOG.md updated with one line description of the fix, and a link to the original issue if available.

I was looking through the PR code and got a question:
what if the request is failed with the HttpRequestException (in case of general network failure), it seems that the delegate is not invoked?
This probably suits the initial idea of gathering metric of successful requests, but if I'm correct it makes it unusable in our scenario #1974 - detecting the connection loss.
Would the metric be accurate if it does not include the failures?
Could it improve your solution if the delegate allows collecting failures also?

Left comments.
We need sample usage code in the PR description to help users onboard to this feature as well.

Plan was to add complete working solution in samples folder. I mentioned it n TODO section of PR description. Should we provide complete sample in this PR description itself?

@rajkumar-rangaraj if customers are using the in-memory channel, is there a way to subscribe to the transmission status update?

Event handler TransmissionStatusEvent in Transmission is wired up through ServerTelemetryChannel, subscriber will utilize a public property in ServerTelemetryChannel to attach the event

@rajkumar-rangaraj if customers are using the in-memory channel, is there a way to subscribe to the transmission status update?

Event handler TransmissionStatusEvent in Transmission is wired up through ServerTelemetryChannel, subscriber will utilize a public property in ServerTelemetryChannel to attach the event

Changes will work only for ServerTelemetryChannel. As there are no retry logic did not expose public property from InMemoryChannel. Changes would be simple to enable for InMemoryChannel, we just need to expose property in channel and serializer.


Update IAM service definitions

This is an auto-generated PR with IAM service updates.

:tada: This PR is included in version 1.1.53 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Props enhanced

I haven't heard any feedback on the [ML thread] (https://groups.google.com/forum/#!searchin/liftweb/properties|sort:date/liftweb/5T_4xm1amr8/XaruS3VMBwAJ) regarding this enhancement, so I thought I'd take the next step to make a PR to have some more eyes on it.
Briefly, this PR allows Lift to be configured via LiftRules to (1) look up properties from more sources than just the *.props files such as JVM System properties, OS environment properties, or arbitrary Map[String, String] and (2) to reference other properties with unix-like ${propName} syntax.  The extra property sources' precedence is well-defined and dictated by the developer via calling prepend vs append.  For interpolators we have appendInterpolator only since there is not a fixed resource to work around.
Note that interpolation is implemented such that infinite recursion is not possible. We do not check the values found for more keys to look up.
The only missing feature I feel we have here is the ability to interpolate from the *.props files themselves. Currently we can only look up from maps provided to appendInterpolator.
See net.liftweb.util.PropsSpec for working usage examples.  Looking forward to the feedback!

Worth noting, Codacy had some interesting suggestions

Based on everyone's feedback: (1) I want to change the interface for this feature to use RuleSeq for the append and prepend configuration functions (and hence move the configuration from Props itself to LiftRules where everything else is configured anyway). (2) I will remove the use of java.util.Map as the container type (replace with either a Scala Map or structural type { def:get(String):String } depending on how things shake out).
That being stated, I feel that it's a re-write at this point, so I'm closing this PR.  Thanks everyone for your valuable critical feedback.

Bah! Can't use RuleSeq because that's over in web whereas Props is in core.

Given that I couldn't use RuleSeq and I was able to just change from java.util.Map to scala.collection.Map which allows either mutable or immutable maps, it made much more sense to just update the existing branch.

Let me know if you're short on time here and want me to see if I can try some stuff, @joescii—I'm happy to help!

Yeah, @Shadowfiend knock it out if you have time.  I am in agreement with your ideas :+1:

Rebased against master and made the changes. Also did some naming tweaking in order to align the naming of the types and the functions that modify them, made Props a private trait that is extended by the singleton so that the tests can be properly isolated, and separated the types for the props providers and the interpolator values.
Let me know what you think!

It looks awesome!  I had not thought about the private trait approach for
making these testable.  That's much cleaner than the mess I was trying to
make.  I also like how you locked the providers with lazy vals.  Both are
great ideas!
Joe
On Tue, Jan 19, 2016 at 5:11 PM, Antonio Salazar Cardozo <
<EMAIL_ADDRESS>wrote:

Rebased against master and made the changes. Also did some naming tweaking
in order to align the naming of the types and the functions that modify
them, made Props a private trait that is extended by the singleton so
that the tests can be properly isolated, and separated the types for the
props providers and the interpolator values.
Let me know what you think!
—
Reply to this email directly or view it on GitHub
https://github.com/lift/framework/pull/1722#issuecomment-173018095.

Looks like some good feedback from the OG author ;)
Going to wait another day or so for some more feedback and then :shipit: .

Alllll righty, let's play!


Expose "options"

I'm having trouble figuring out how to pass arguments to ffmpeg. I think this might be a documentation issue (for a low level hack like me).  For example, can I set the keyframe interval in an output video?  In ffmpeg I would use something like -g 1 to force every frame to be a keyframe. Can you use an ffmpeg argument string with PyAV?

I think that is AVCodecContext.gop_size, which is not currently exposed. It could be easily added to that object, but...
Your question made me go investigate a few things, and I think that taking advantage of as much of the machinery that the ffmpeg command uses as possible would be wonderful, instead of having to re-invent everything.
Here are all of the options on AVCodecContext, which includes "gop_size" that is set via "g".
If all classes took a look at their AVClass, perhaps a generic xxx.set_option(name, value) would do nicely. It could also be wrapped into a mapping interface.
[ ] Investigate how ffmpeg command uses options.
[ ] Design how to expose this for all types.
[ ] Implement it.

Exposing the ffmpeg command line options would be super useful!
For now I'll try adding options as I need them... although, that might be way above my coding level

Hi @mikeboers, was this implemented?

reads this again
No. No it was not.
reopens it

Okay, I've just been manually adding getters and setters when a field isn't exposed. I'm not sure if there's a nice way to do it dynamically (getattr and setattr) with Cython. Might have to do some codegen from that header :grimacing:

@mikeboers has this been implemented?

@SehgalDivij Seems so. By setting codec_ctx.gop_size = 20 on a CodecContext I'm getting is_keyframe is True on every 20th encoded Packet.

@SehgalDivij Seems so. By setting codec_ctx.gop_size = 20 on a CodecContext I'm getting is_keyframe is True on every 20th encoded Packet.

@pwuertz seems to have offered a reasonable solution, closing


#18 extension.ts

Fix for issue #18 -- Fix change to https://code.visualstudio.com/api/references/vscode-api#InlineCompletionItem, insertText renamed

No prob! 😊


Wrong conversion, should be from MB to B, and not from MB to KB

https://github.com/aaronparker/FSLogix/blob/033e83561769593e77fd08aba027fa81853ef422/Profile-Cleanup/Remove-ContainerData.ps1#L259

Fix:
Where-Object { $_.Length -gt (Convert-Size -From MB -To B -Value $MinimumSizeInMB) }


Question: What is percentage in the renderLayer callback?

This might get labeled a feature request but it's more like a question.
To frame it as a feature request we could say this could be an enhancement to the documentation
What is percentage in the renderLayer callback?
I know it's a "percentage value of the current position" (as the documentation says) but how exactly (or generally) does it represent the current scroll position?
What is it a "percentage" of?
It's really confusing trying to figure out how to use it since it ranges from 0 (actually -0) upward, past 1.0, even before the element has moved out of the viewport.
Thanks!

Hey, sorry for the delay. This is a tricky question. You might have seen, that the behaviour changed at a point. Originally the percentage (or range) from 0 to 1 behaved differently. It was 0 at the point when the component enters the screen at the bottom. So the anchor for the calculation was at the very top of the parallax container. While scrolling further the point shifted until it was at the bottom of the parallax just before leaving the screen completely.
But this behaviour made it necessary to increase the image height so there was enough buffer to prevent white space. So yeah the calculation of the range is a bit odd now but for the normal use case it works fine.


[Events] On open selected option should always be visible

Currently
Consider a Flounder with a long list of options that must be scrolled. When we open the dropdown the scroll position is either at the top of the list (if we have not interacted with it) or wherever we last left off.
Problem
This means the currently selected option is not always in view.
Wanted
Native behavior (to be verified): dropdown always opens with selected option in view.

relavent https://github.com/sociomantic-tsunami/flounder/blob/master/src/core/utils.js#L323-L346


change interface of ResponseWithBody<S, T> to use <S extends number, T>

change interface of ResponseWithBody<S, T> to use <S extends number, T>
otherwise I am getting this errors:
Interface 'ResponseWithBody<S, T>' incorrectly extends interface 'Response'.
  Types of property 'status' are incompatible.
    Type 'S' is not assignable to type 'number'.

this is for typescript 3.5 and @types/superagent=4.1.1

@Kosta-Github - Thank you for reporting and creating a PR to fix this!
Merged and published as 1.10.1


Update Jackson to 2.12.1

junit5 upgrade is needed to get fix for https://github.com/junit-team/junit5/issues/2198
Also address #2695

After upgrading Jackson, the following happened:

in grpc-server, the transitive dependency of zipkin-junit changed Junit version from 4.12 to 4.13.1, which was not supported by the version of Junit Jupiter engine
I have upgraded Junit 5 to latest version, which fixed this

I have no idea why a transitive dependency version would change when changing a version of Jackson, which is not part of dependencies of grpc-server, nor does it version manage junit. This may be interesting to investigate... @romain-grecourt any idea why this could happen?

After upgrading Jackson, the following happened:

in grpc-server, the transitive dependency of zipkin-junit changed Junit version from 4.12 to 4.13.1, which was not supported by the version of Junit Jupiter engine
I have upgraded Junit 5 to latest version, which fixed this

I have no idea why a transitive dependency version would change when changing a version of Jackson, which is not part of dependencies of grpc-server, nor does it version manage junit. This may be interesting to investigate... @romain-grecourt any idea why this could happen?

Also see #2672

Also see #2672


Should KaplanMeierFitter have a strata option?

A common task with KM is to plot many survival functions on the same plot. Lifelines atm kind of makes this difficult, something like:
ax = subplot(111)

for value in df['cata'].unique():
    ix = df['cata'] == value
    kmf.fit(df.ix[ix]['T'], df.ix[ix]['E'], label=value)
    ax = kmf.plot(ax=ax)

It's not obvious to do this the first time you encounter lifelines. A proposal may be an additional arg in KMFitter, strata, that does something like this all for you:
kmf.fit(df['T'], df['E'], strata=df['cata'])

Then, kmf.survival_function_ returns a dataframe of survival functions (median_ is the same). plot is definetly easier for the user, but internally may be more complicated.
Other options:

Better docs to demonstrate how this is done.

A new Fitter that handles this

Helper functions to make this easier.

Great,strata works smoothly,however the add_risk_counts part only add the last category below the plot:
import pandas as pd
from lifelines.datasets import load_dd
from lifelines import KaplanMeierFitter
kmf = KaplanMeierFitter()
from lifelines.plotting import add_at_risk_counts

data = load_dd()

data[['T','E']] = data[["duration","observed"]]

ax = plt.pyplot.subplot(111)

for value in data['un_continent_name'].unique():
    ix = data['un_continent_name'] == value
    kmf.fit(data.ix[ix]['T'], data.ix[ix]['E'], label=value)
    ax = kmf.plot(ax=ax)

add_at_risk_counts(kmf, ax=ax)

A suggestion: using pandas.groupby makes this slightly easier and more readable:
ax = plt.gca()
for gname, group in df.groupby('group'):
    kmf.fit(group['duration'], event_observed=group['death_observed'], label=gname)
    kmf.plot(ax=ax)

This might be a useful addition to the docs, in case adding a strata parameter complicates the kmf.fit implementation too much.


Cpp saf example

Did you find critical bug in our code?
Do you have any new feature you want implemented?
Did you fix anything?

Propose PULL Request then and we will examine it

HI @mpospisil, we just need to make sure the doc links are updated based on adding the new SAF folder. in docs>docs>bimapi>toc.yml


"btn_pos":3 and "btn_pos":4 brick of the device.

Saving this to pages.jsonl will brick device.
{"page":1,"id":14,"obj":"tabview","btn_pos":3,"x":0}
{"page":1,"id":51,"obj":"tab","parentid":14,"text":"Tab 1"}
{"page":1,"id":52,"obj":"tab","parentid":14,"text":"Tab 2"}
{"page":1,"id":53,"obj":"tab","parentid":14,"text":"Tab 3"}
{"page":1,"id":61,"obj":"switch","x":20,"y":10,"w":60,"h":30,"parentid":51,"radius":25,"radius20":25}
{"page":1,"id":71,"obj":"dropdown","x":15,"y":10,"w":110,"h":30,"parentid":52,"options":"Apple\nBanana\nOrange\nMelon"}
{"page":1,"id":81,"obj":"checkbox","x":15,"y":10,"w":110,"h":30,"parentid":53,"text":" Nice tabview"}

Duplicate of #509


Bookstore Use Redux in React components.

In this milestone, I have  Initialize React app
The building blocks of the app is a set as re-usable components. I Created a directory for the components.
I Add React Router and set two Routes and Links for the app's navigation:
Books - the default view:

Display the list of books
Display a form for adding a book (no functionality yet).
Categories:
Display a button "Check status" only.

STATUS: INVALID CODE REVIEW REQUEST 🛑 🛑
Hi @elad237,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
I think you submitted the wrong form for requesting a review. Your PR description and your code indicate this project is meant for Initialize components project, but you submitted the Use redux in react form. Kindly verify.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


After a user installs my Slack app I want to redirect them to my web app, hosted on a different domain

Description
After a user installs my Slack app I want to redirect them to my web app which is hosted on a different domain, is that possible?
Currently, if I pass my Slack app installation domain as the redirect_uri everything works perfectly.
If I try to update the redirect_uri to be my front-end domain it properly redirects and displays no errors, yet it's not installing the app on the workspace anymore...
Is there something I'm missing in the bolt config?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version:
@slack/bolt": "3.11.1
node version:
v18.12.1
OS version(s):
Steps to reproduce:

Set the redirectUri & redirectUriPath to a domain that isn't hosting the Slack app

Expected result:
The Slack app installs on the workspace and redirects to the domain I set as the redirectUri which is my web app front-end. The front end passes the state and code to the backend and signs the user directly into my web app.
Actual result:
The Slack app does not install on the workspace.
Attachments:

Hi @MattB543, thanks for asking the question. You can use CallbackOptions.success for it. Refer to https://github.com/slackapi/bolt-js/issues/1571#issuecomment-1231795209 for more details. In your case, you can redirect an installing user to a different domain instead of rendering the webpage.

Hi, @seratch thanks for the reply.
Actually, we want our user to automatically log in to our application with slack OpenID authorization https://slack.com/openid/connect/authorize.
So in order to do that we need a code that OpenID authorization returns to the redirect URI, so how we can get it from the app installation?

@themashcodee An app installation into a Slack workspace is indeed an OAuth flow, but it's not compatible with OpenID Connect including the "Sign in with Slack" one. So, if you need an OpenID Connect response, the user needs to go through the Sign in with Slack flow separately. Also, unfortunately, bolt-js does not support Sign in with Slack. You need to add custom routes (or build a different app) for the Sign in with Slack flow. Refer to an example app directly using the underlying @slack/oauth package: https://github.com/slackapi/node-slack-sdk/tree/main/examples/openid-connect
Lastly, if the reason you're considering "Sign in with Slack" is to receive the installing user's identity, the app installation flow's oauth.access.v2 API response already provides it. If that's the case, you may not have to add the OpenID Connect flow to the app.
I hope this helps.

@seratch thank you so much for the help, I really appreciate it.

Thanks, @seratch , appreciate your quick response!


blast fails to run in MacOS

Any ideas why I'm getting this error:
$ blast-radius
Traceback (most recent call last):
  File "/Users/xxx/Library/Python/2.7/bin/blast-radius", line 14, in <module>
    from blastradius.handlers.dot import DotGraph, Format, DotNode
  File "/Users/xxx/Library/Python/2.7/lib/python/site-packages/blastradius/handlers/dot.py", line 391
    print('Error processing format param: ' + 'p', file=sys.stderr)
                                                       ^
SyntaxError: invalid syntax

Just a guess, but the examples show pip installing to Python 3, looks like Python 2 is trying to run it. That print statement will not work in 2.7.

true.
turned out there are some extra steps I missed to get python 3 in MacOS


feat: Add logging

What kind of change does this PR introduce?
feature
What is the current behavior?
When debugging the auth broadcasting or the realtime/stream fixed I had to manually add many print statements, which was annoying. The js sdk already has some logging.
What is the new behavior?
Add logging via the logging package from dart.dev. This only tracks the logs, but does not print them in any way. That's the task of the user. The logger instances are laid in a hierarchy separated by dots, which allows all supabase loggers to be united in a supabase logger instance.

@dshukertjr For now, I've only added logging to the gotrue package. What do you think of this? If you consent, I would add this to the rest of the packages.

This is awesome! I love it so much!

I've noticed that we should be careful with what data we log, especially sensitive data. So I've come up with the idea that records with level FINEST contain sensitive data like session data, or which postgrest urls are called. So that those records shouldn't be sent to sentry or similar without making such things clear in privacy notes.  Let me know what you think of this approach and the documentation in the README of the supabase_flutter package, so I can add that section to the other packages as well.

I see now that there is no documentation in the README of the rest of the packages. So I would leave it for now only in the supabase_flutter package and you? or I? add it to the supabase website documentation.


Repository dkarv/hacs-lg-ess not found

Hi,
wondering if your repository is still live. I cannot find it in HACS.
Would love to use it to read all my data from the LG System.
Best
Jens

Hi,
yes it is still live and working.
You need to add the repository url as custom repository in HACS (under the three dots in the top right).

Ah my fault. Thanks, it works now.


bg-356 added error check

Which Issue was this PR made for?

Closes #356

What is within the scope with this PR
Temporary patch for server crashing bug.

AUTOMATED: Deployed to https://pr-357.dev.mochatto.com


Morning session: Peer-to-peer code review

Hello dfsalam 👋🏽
Good job working on the project so far!
✔ Your project is well documented
✔ App works as expected
Highlights 🎯
✔ Descriptive PR
✔ Set up app with webpack 📌
✔ Good commit history
You can also consider:
You have linter errors, to fix the error, please can you try this command: 'npm install html-webpack-plugin'.
Once the plugin is installed, you will need to configure it in your webpack configuration file, You can import and use the  plugin like this:   const HtmlWebpackPlugin = require('html-webpack-plugin');
module.exports = {   // ...   plugins: [     new HtmlWebpackPlugin({       template: './src/index.html',       filename: './index.html',     }),   ],   // ... };

Thank you for your comments. 🔥 🔥


Thumbnail Video Loading is Slow and Bottom Setting Button Not Showing

Bug description
-Thumbnail Video Loading is Slow or sometimes fail to load (Especially in Gelbooru and rule34.paheal.net)
-Bottom Setting Button Not Showing (the one with Detail, Open, Save, etc. when viewing an image)
Steps to reproduce

Open the program.
View random image.
Observe that the buttons at the bottom (e.g., Detail, Save, Open) do not show up.
How i fix this, Interface -> Base Style and change the style to anything else (e.g., from Windows style to WindowsVista or Fusion). The problem will disappear.
Even if you revert back to the original style, the problem remains fixed after changing the style once. This issue occurs with all styles, not just the Windows style. However, when I reopen the app, the problem reoccurs.

Additionally, when I click on the '+' button, nothing happens; the options do not appear
For the thumbnail issue:

When searching for animated images/videos, the thumbnails load very slowly or fail to load altogether.
This problem is especially noticeable on Gelbooru and rule34.paheal.net

Context
main_2024.05.20-<IP_ADDRESS>8.log
settings.zip
Screenshots

System information

OS: Windows 11 23H2 (OS Build 22631.3593)
Grabber version: 7.12.2

Additional context
I tried using Grabber version 7.11.2 and didn't encounter any of the issues mentioned above. Therefore, the thumbnail loading issue is not due to my connection

I'm having the same issue, I'm guessing its not an isolated issue.

Thanks for the report and the details provided!
Closing as duplicate of issue #3071 though, let's move the discussion there 👍


Add prompt, response, and messages to Firestore -> Snowflake DAG

I just made a push that fixed the type hinting!

@jlaneve do you want this deployed immediately to backend after merging?


Full layout and text region classification capability of Cloud OCR SKD

Posted on the forum 4 days ago (where there are 761 open questions without an accepted answer to 899 answers, hence cross post) http://forum.ocrsdk.com/questions/4649/full-layout-and-text-region-classification-capability-of-cloud-ocr-skd.
I'm trying to recognise semantic information about text regions e.g. headers, footers, page numbers, captions etc. I cannot find any information about how to achieve this in the documentation. Please can you provide a python code example?
I'm currently using the supplied python example here: http://ocrsdk.com/documentation/code-samples/. N.B. In order to get this Python code to work, I had to change line 90 of AbbyyOnlineSdk.py from
return {"Authorization": "Basic %s" % base64.encodestring( ...

to
return {"Authorization": "Basic %s" % base64.b64encode( ...

I'm currently just executing this call:
python process.py infile.png outfile.xml -l English -xml

This question answered on the forum:
This feature is not supported in ABBYY Cloud OCR SDK. But you can try to use another our SDK for your needs, namely ABBYY FineReader Engine. You can access the logical structure of a document formatting attributes, including headers, footers, page numbers, fonts, styles etc., via the IFRDocument::DocumentStructure and IFRPage::PageStructure properties. Please find more information in Help → Guided Tour → Advanced Techniques → Working with the Logical Structure of a Document.
For trial version of FineReader Engine, please fill the form at our site.


Fix dimable_name regular expression

The regex for dimable names wasn't anchoring the entire expression on
the start and end of lines, just the first and last optional matches.

@burrbull

bors r+

Build failed:

continuous-integration/travis-ci/push

https://travis-ci.org/github/rust-embedded/svd/jobs/693897919, the failure seems unrelated to this change. Not sure how to diagnose the actual issue here off hand.

Looks like we need up minimal supported Rust version to 1.36

cc @therealprof

https://github.com/crossbeam-rs/crossbeam/pull/504 maybe related?

Opened https://github.com/rust-embedded/svd/pull/123 to bump MSRV.

bors r=burrbull

Still working on the CI in https://github.com/rust-embedded/svd/pull/124.

bors r=burrbull


TxScriptValiditySupportedInAlonzoEra for Alonzo transactions

What does TxScriptValidatity stand for? https://github.com/input-output-hk/cardano-node/blob/333c701ca9be38e5d955c9d3ee0fab391e85e4f4/cardano-api/src/Cardano/Api/TxBody.hs#L275
Should it always be TxScriptValidity TxScriptValiditySupportedInAlonzoEra for Alonzo transactions?

Closing this. If this is still relevant please reopen.


TerseDecompress - Annual Review

@alexgubanow Can you be ready for this during the 7/27 TAC meeting?

What does annual review would mean for this project?

@alexgubanow details at https://tac.openmainframeproject.org/process/review_cycle.html. Can we schedule for 8/10?

Yes, please. I was not able to join last time, it seem like link I have is from April. Will try to get a correct link this time.

Thanks! @yarille can you make sure @alexgubanow has the correct invite?

Renewed by TAC email vote, 9/6/2023.