API for setting search icon

Hi, sahildave. I added  a method which allows user to set the their own customized search icon. If there are any problems, plz let me know.

@vguzzi @DrawableRes would be the correct annotation I believe.

@sahildave ah yes! Thank you

Hi, I'm the guy who requested the hint. How can I use implement these changes?

I have added two method for changing icons and tried to improve formatting.

@EpsilonOrionis  Hi, you can sync the fork to get the updates. To see this link for the instruction: https://help.github.com/articles/syncing-a-fork/

Hey @tim-bm Can you add these changes to the PR. I would merge the pull request.
    /**
     * Allow user to set a search icon in the un-expanded view
     *
     * @param iconResource resource id of icon
     */
    public void setCollapsedIcon(@DrawableRes int iconResource) {
        ((ImageView) mSearchIcon).setImageResource(iconResource);
    }

    /**
     * Allow user to set a back icon in the expanded view
     *
     * @param iconResource resource id of icon
     */
    public void setExpandedBackIcon(@DrawableRes int iconResource) {
        ((ImageView) mBackButtonView).setImageResource(iconResource);
    }

    /**
     * Allow user to set a search icon in the expanded view
     *
     * @param iconResource resource id of icon
     */
    public void setExpandedSearchIcon(@DrawableRes int iconResource) {
        ((ImageView)mExpandedSearchIcon).setImageResource(iconResource);
    }

I have added the changes and sry for bad naming methods.
On Sunday, 22 November 2015, Sahil Dave<EMAIL_ADDRESS>wrote:

Hey @tim-bm https://github.com/tim-bm Can you add these changes to the
PR. I would merge the pull request.
/**     * Allow user to set a search icon in the un-expanded view     *     * @param iconResource resource id of icon     */
public void setCollapsedIcon(@DrawableRes int iconResource) {
    ((ImageView) mSearchIcon).setImageResource(iconResource);
}

/**     * Allow user to set a back icon in the expanded view     *     * @param iconResource resource id of icon     */
public void setExpandedBackIcon(@DrawableRes int iconResource) {
    ((ImageView) mBackButtonView).setImageResource(iconResource);
}

/**     * Allow user to set a search icon in the expanded view     *     * @param iconResource resource id of icon     */
public void setExpandedSearchIcon(@DrawableRes int iconResource) {
    ((ImageView)mExpandedSearchIcon).setImageResource(iconResource);
}

—
Reply to this email directly or view it on GitHub
https://github.com/sahildave/Search-View-Layout/pull/8#issuecomment-158740166
.

--
陈博添 Tim
Student Id:20112179
Software engineering international class, Department of software in NEU
<EMAIL_ADDRESS>QQ:308496750


feat: Support for Import/Export without ACL

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description
This PR add support for Import and Export without ACL required for Version control.

Please include a summary of the changes and which issue has been fixed. Please also include relevant motivation
and context. List any dependencies that are required for this change
Links to Notion, Figma or any other documents that might be relevant to the PR

Add a TL;DR when description is extra long (helps content team)

Fixes #18886
Fixes #18889
Fixes #18898
Fixes #18902

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.
Delete anything that is not important

Manual

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=be12abc29e663cb75b32441164682b7c1d6a42fe


GetterProperty and SetterProperty aren't handled correctly

GetterProperty and SetterProperty aren't turned to their babel ObjectMethod equivalent.
NOTE: I almost have PR ready to publish

I messed up on the PR


Add semtype relations for array

Purpose

$title.

Fixes #34629
Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@chiranSachintha, shall we break the tests based on // @type comment. Each file can be included test cases related to one or two // @type.

@chiranSachintha did not you find any failing test case?

@chiranSachintha did not you find any failing test cases?

No, found test cases that are not supported by the ballerina compiler(added them to -d.bal file)


Issue #23: add battery-low warning to the HKR listing

Closes #23

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #26   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          35     35
  Lines        1017   1019    +2
=====================================
+ Hits         1017   1019    +2

Impacted Files
Coverage Δ

cmd/list_thermostats.go
100% <100%> (ø)
: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 74e84ce...2da0d23. Read the comment docs.


Spatial extension not available for Windows?

Apologies if I'm reporting this in the wrong place or it is already documented, but it appears the extension is not available for Windows (at least when attempting to install from R)?
INSTALL "spatial" is failing in my tests on Windows platforms while passing on all other platforms.  Specifically, it tries to download from http://extensions.duckdb.org/v0.9.1/windows_amd64_rtools/spatial.duckdb_extension.gz which is not reachable.  See logs: https://github.com/cboettig/duckdbfs/actions/runs/6647184214/job/18062119997#step:6:181

Hi! Thanks for reporting this issue!
The extension is available for windows here http://extensions.duckdb.org/v0.9.1/windows_amd64/spatial.duckdb_extension.gz,
But we don't currently build "out of tree"-extensions (the extensions that don't reside in the main duckdb repository) for the R client, which iirc requires some special compilation magic.

Thanks @Maxxen !  ok that makes sense I think but I don't entirely follow.  currently, the R version of duckdb tries to download from http://extensions.duckdb.org/v0.9.1/windows_amd64_rtools/spatial.duckdb_extension.gz, which doesn't exist for the reasons you say.  That feels like a bug.  Are you suggesting that
(a)  duckdb code should be altered so that it installs from http://extensions.duckdb.org/v0.9.1/windows_amd64/spatial.duckdb_extension.gz instead of _rtools flavor, or would that not work with R users either (due to missing compiler situation?)
(b) duckdb should throw an error message that says "spatial extension not available for Windows R users" or something, rather than the current error?
(c) nothing should change in duckdb, but one day we might get  http://extensions.duckdb.org/v0.9.1/windows_amd64/spatial.duckdb_extension.gz ?
I recall that formerly the httpfs extension wasn't working for Windows R users either (also due to same challenges with compilers available in rtools iirc), and both packages on CRAN that use duckdb and some popular blog posts still describe duckdb as not supporting httpfs / URL access for R users on Windows. However I also noticed that this is no longer true, and Windows users can now use httpfs, but still cannot use the spatial extension.
Just cc'ing the amazing @jeroen who helps maintain rtools and might have some insights in how to get this extension to build for Windows when he has a chance?

The httpfsextension is distributed as part of the core DuckDB build pipeline (its an "in-tree" extension, it resides inside the main DuckDB repository), and afaik we don't build any out-of-tree extensions for R, not because its impossible, its just that we're still figuring out how out-of-tree extensions should be built, synced and distributed.
I assume that we differentiate between windows and _rtools for a reason but maybe its just to comply with CRAN guidelines? I guess I would try with the normal windows build and see if it works? But since I don't work with R myself my answer is probably a combination of b) and c), although I don't know if the error can be more descriptive than it already is.
Maybe @carlopi or @samansmink can chime in.

Cool, thanks!  AFAIK, the R client merely sends the SQL command string INSTALL 'spatial'; out over the duckdb connection, so I don't have much visibility into how duckdb even 'knows' that it's running from inside R and where the logic resides that lead it to choose the http://extensions.duckdb.org/v0.9.1/windows_amd64_rtools/spatial.duckdb_extension.gz path in the first place.  I can probably dig into the instructions for manual installs of extensions instead, but  I don't have a windows machine handy to test with.  The INSTALL 'spatial'; command works just fine from R for installing the extension on any other platform I've tested (mac, linux, arm architectures), I think it's just windows that has troubles.  I wonder if the _rtools thing dates back to the reason that httpfs wasn't on windows either, and maybe is no longer necessary?

As a follow-up on the manual installation front, downloading the windows version of the extension, decompressing it, and installing it does work up through installation. When attempting to load the extension, though, both R and RStudio immediately crash once LOAD spatial is called. This happens whether the DB is in-memory or persistent. E.g.,
# https://extensions.duckdb.org/v0.9.2/windows_amd64/spatial.duckdb_extension.gz

reticulate::repl_python()

import gzip
import shutil

# file path is truncated
with gzip.open('.../spatial.duckdb_extension.gz','rb') as f_in:
    with open('.../spatial.duckdb_extension', 'wb') as f_out:
        shutil.copyfileobj(f_in, f_out)

exit

library(duckdb)

con <- dbConnect(duckdb())

dbExecute(con, "INSTALL '.../spatial.duckdb_extension'")
dbExecute(con, "LOAD spatial")

# Also fails with dbExecute(con, "LOAD '.../spatial.duckdb_extension'")

Manually building the extension is a little beyond my knowledge base, so hopefully this is useful info!
sessionInfo() just in case:
R version 4.3.0 (2023-04-21 ucrt)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 19045)

Matrix products: default

locale:
[1] LC_COLLATE=English_United States.utf8  LC_CTYPE=English_United States.utf8    LC_MONETARY=English_United States.utf8 LC_NUMERIC=C
[5] LC_TIME=English_United States.utf8

time zone: America/New_York
tzcode source: internal

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] duckdb_0.9.1-1 DBI_1.1.3

loaded via a namespace (and not attached):
 [1] ggdist_3.3.0           utf8_1.2.3             generics_0.1.3         stringi_1.7.12         lattice_0.21-8         lme4_1.1-33            hms_1.1.3
 [8] magrittr_2.0.3         grid_4.3.0             blob_1.2.4             jsonlite_1.8.4         Matrix_1.6-2           ggrepel_0.9.3          ggeffects_1.2.2
[15] ggthemes_4.2.4         purrr_1.0.1            fansi_1.0.4            tidytable_0.10.1       box_1.1.3              scales_1.2.1           cli_3.6.1
[22] rlang_1.1.0            bit64_4.0.5            munsell_0.5.0          splines_4.3.0          withr_2.5.0            tools_4.3.0            nloptr_2.0.3
[29] odbc_1.3.4             minqa_1.2.5            dplyr_1.1.2            colorspace_2.1-0       ggplot2_3.4.2          boot_1.3-28.1          reticulate_<IP_ADDRESS>00
[36] png_0.1-8              vctrs_0.6.2            R6_2.5.1               lifecycle_1.0.3        stringr_1.5.0          bit_4.0.5              MASS_7.3-58.4
[43] pkgconfig_2.0.3        pillar_1.9.0           gtable_0.3.3           data.table_1.14.8      glue_1.6.2             Rcpp_1.0.10            tibble_3.2.1
[50] tidyselect_1.2.0       rstudioapi_0.14        farver_2.1.1           nlme_3.1-162           compiler_4.3.0         distributional_0.3.2

@Maxxen @eitsupi pointed out to me that DuckDB for Windows other than R uses the MSVC ABI and only R uses the GNU ABI. The other R extensions are just being built by a github action https://github.com/duckdb/duckdb/blob/a55f89cd9e956b3e575532e058c230461799ac64/.github/workflows/R.yml#L29-L69.  Would it be possible to use that as a basis for building the spatial extension for R windows users as well?  (I'd offer to do a PR but given that it needs S3 secrets it might be easier for someone from the duckdb team to do so; also not sure which repo this would belong to).

I noticed that https://github.com/duckdb/duckdb/pull/10204 was merged, so I thought I'd check to see whether the spatial extension worked with the nightly build. Unfortunately, I am still unable to install the spatial extension (see the reprex below). Am I correct to think that https://github.com/duckdb/duckdb/pull/10204 should have solved this issue?

Reprex

# remove.packages("duckdb")
install.packages('duckdb', repos=c('https://duckdb.r-universe.dev', 'https://cloud.r-project.org'))
#> Installing package into 'C:/Users/tiern/AppData/Local/R/win-library/4.3'
#> (as 'lib' is unspecified)
#> package 'duckdb' successfully unpacked and MD5 sums checked
#>
#> The downloaded binary packages are in
#>  C:\Users\tiern\AppData\Local\Temp\RtmpiwZSeV\downloaded_packages
packageVersion("duckdb")
#> [1] '<IP_ADDRESS>'
con = DBI::dbConnect(duckdb::duckdb(), ":memory:")
DBI::dbExecute(con, "INSTALL spatial;")
#> Error: rapi_execute: Failed to run query
#> Error: HTTP Error: Failed to download extension "spatial" at URL "http://extensions.duckdb.org/2414840843/windows_amd64_rtools/spatial.duckdb_extension.gz"
#> Extension "spatial" is an existing extension.
#>
#> Are you using a development build? In this case, extensions might not (yet) be uploaded.
DBI::dbGetQuery(con, "FROM duckdb_extensions()") |> tibble::tibble() |> print(n=Inf)
#> # A tibble: 22 × 6
#>    extension_name   loaded installed install_path description            aliases
#>    <chr>            <lgl>  <lgl>     <chr>        <chr>                  <list>
#>  1 arrow            FALSE  FALSE     ""           A zero-copy data inte… <chr>
#>  2 autocomplete     FALSE  FALSE     ""           Adds support for auto… <chr>
#>  3 aws              FALSE  FALSE     ""           Provides features tha… <chr>
#>  4 azure            FALSE  FALSE     ""           Adds a filesystem abs… <chr>
#>  5 excel            FALSE  FALSE     ""           Adds support for Exce… <chr>
#>  6 fts              FALSE  FALSE     ""           Adds support for Full… <chr>
#>  7 httpfs           FALSE  FALSE     ""           Adds support for read… <chr>
#>  8 iceberg          FALSE  FALSE     ""           Adds support for Apac… <chr>
#>  9 icu              FALSE  FALSE     ""           Adds support for time… <chr>
#> 10 inet             FALSE  FALSE     ""           Adds support for IP-r… <chr>
#> 11 jemalloc         FALSE  FALSE     ""           Overwrites system all… <chr>
#> 12 json             FALSE  FALSE     ""           Adds support for JSON… <chr>
#> 13 motherduck       FALSE  FALSE     ""           Enables motherduck in… <chr>
#> 14 mysql_scanner    FALSE  FALSE     ""           Adds support for conn… <chr>
#> 15 parquet          TRUE   TRUE      "(BUILT-IN)" Adds support for read… <chr>
#> 16 postgres_scanner FALSE  FALSE     ""           Adds support for conn… <chr>
#> 17 spatial          FALSE  FALSE     ""           Geospatial extension … <chr>
#> 18 sqlite_scanner   FALSE  FALSE     ""           Adds support for read… <chr>
#> 19 substrait        FALSE  FALSE     ""           Adds support for the … <chr>
#> 20 tpcds            FALSE  FALSE     ""           Adds TPC-DS data gene… <chr>
#> 21 tpch             FALSE  FALSE     ""           Adds TPC-H data gener… <chr>
#> 22 visualizer       FALSE  FALSE     ""           Creates an HTML-based… <chr>

Created on 2024-01-23 with reprex v2.0.2

Session info

sessionInfo()
#> R version 4.3.2 (2023-10-31 ucrt)
#> Platform: x86_64-w64-mingw32/x64 (64-bit)
#> Running under: Windows 11 x64 (build 22631)
#>
#> Matrix products: default
#>
#>
#> locale:
#> [1] LC_COLLATE=English_United States.utf8
#> [2] LC_CTYPE=English_United States.utf8
#> [3] LC_MONETARY=English_United States.utf8
#> [4] LC_NUMERIC=C
#> [5] LC_TIME=English_United States.utf8
#>
#> time zone: America/Los_Angeles
#> tzcode source: internal
#>
#> attached base packages:
#> [1] stats     graphics  grDevices utils     datasets  methods   base
#>
#> loaded via a namespace (and not attached):
#>  [1] vctrs_0.6.4       cli_3.6.1         knitr_1.42        rlang_1.1.1
#>  [5] xfun_0.39         DBI_1.1.3         purrr_1.0.2       styler_1.10.2
#>  [9] glue_1.6.2        htmltools_0.5.5   fansi_1.0.5       rmarkdown_2.21
#> [13] R.cache_0.16.0    evaluate_0.21     tibble_3.2.1      fastmap_1.1.1
#> [17] yaml_2.3.7        lifecycle_1.0.3   duckdb_0.9.2-1    compiler_4.3.2
#> [21] fs_1.6.2          pkgconfig_2.0.3   rstudioapi_0.14   R.oo_1.25.0
#> [25] R.utils_2.12.3    digest_0.6.33     utf8_1.2.4        reprex_2.0.2
#> [29] pillar_1.9.0      magrittr_2.0.3    R.methodsS3_1.8.2 tools_4.3.2
#> [33] withr_2.5.2

@tiernanmartin in that PR only the sqlite extension was enabled because I didn't get any of the others to compile easily on the windows rtools environment. It should probably be possible to get most extensions working, including spatial, but we haven't gotten around to it yet!

As hinted by @krlmlr: spatial extension is available on ALL platforms for duckdb v0.10.0.
This is due to combined work from @samansmink (setting up infrastructure) and @Maxxen (adapting spatial to the tooling).
windows_amd64_rtools's spatial build for v0.10.1 has not being uploaded (yet) since it needs some extra manual steps, but that should come out in the next days (and the manual steps are also going away).

duckdb_spatial is now available on all 10 supported platforms for v0.10.0 and v0.10.1.
A PR is up to have also rtools builds be generated by main CI, so it will always be up.
I think this can be closed.

@carlopi: Can you please point me to that PR?


Outdated client! text not work latest dev. waterfall

This also happens when a person is temp-banned / banned. The message does not appear to the client. The client trying to join gets a 'logging in' screen for 30 seconds or so before the null error occurs.

This build works perfectly Outdated client! text: https://ci.aquifermc.org/job/Waterfall/33/

@fuzzybot But is it safe to use?

This patch needs additional work before it is ready for general usage.


Does the. net client support dynamically adding new methods after connection to provide them for the server to call

Description
Does the. net client support dynamically adding new methods after connection to provide them for the server to call
Page URL
https://learn.microsoft.com/zh-cn/aspnet/core/signalr/dotnet-client?view=aspnetcore-8.0&tabs=visual-studio
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/signalr/dotnet-client.md
Document ID
36052173-7061-6d01-8aed-915685cd7cf6
Article author
@bradygaster

We should add @mikekistler as the owner for this one.

Looking at the doc, I think the answer to this question is "No". The doc says:

Define methods the hub calls using connection.On after building, but before starting the connection.

So based on the doc, I believe that all methods must be defined before starting the connection.


[BUG] - SSR warning on Button

NextUI Version
2.3+
Describe the bug
After upgrading from 2.2.10 to 2.3+ I have a warning in the console:

Warning: Prop className did not match. Server: "group relative inline-flex items-center justify-center box-border appearance-none select-none whitespace-nowrap font-normal overflow-hidden tap-highlight-transparent outline-none data-[focus-visible=true]:z-10 data-[focus-visible=true]:outline-2 data-[focus-visible=true]:outline-focus data-[focus-visible=true]:outline-offset-2 min-w-20 h-10 text-small gap-2 rounded-small [&>svg]:max-w-[theme(spacing.8)] data-[pressed=true]:scale-[0.97] transition-transform-colors-opacity motion-reduce:transition-none text-default-foreground z-10 aria-expanded:scale-[0.97] aria-expanded:opacity-70 subpixel-antialiased p-0 bg-transparent data-[hover=true]:bg-transparent" Client: "group relative inline-flex items-center justify-center box-border appearance-none select-none whitespace-nowrap font-normal overflow-hidden tap-highlight-transparent outline-none data-[focus-visible=true]:z-10 data-[focus-visible=true]:outline-2 data-[focus-visible=true]:outline-focus data-[focus-visible=true]:outline-offset-2 min-w-unit-20 h-unit-10 text-small gap-unit-2 rounded-small [&>svg]:max-w-[theme(spacing.unit-8)] data-[pressed=true]:scale-[0.97] transition-transform-colors-opacity motion-reduce:transition-none text-default-foreground z-10 aria-expanded:scale-[0.97] aria-expanded:opacity-70 subpixel-antialiased p-0 bg-transparent data-[hover=true]:bg-transparent"

It causes my buttons to appear without any padding.
It may be unrelated, but note that on version 2.2.10 I had another warning in the console about originalProps:

chunk-CKWPCP4A.js?v=7bbd2a2d:521 Warning: React does not recognize the originalProps prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase originalprops instead. If you accidentally passed it from a parent component, remove it from the DOM element.

The fix of this warning may have brought the other error.
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
I don't really know how to set up a tiny SSR project. I'm working on a project with already a few thousand lines of code.
FYI I'm using Remix, and below is the list of dependencies I currently use:
    "@aws-sdk/client-s3": "3.461.0",
    "@aws-sdk/s3-request-presigner": "3.461.0",
    "@conform-to/react": "1.0.3",
    "@conform-to/zod": "1.0.3",
    "@epic-web/invariant": "^1.0.0",
    "@headlessui/react": "1.7.17",
    "@inlang/paraglide-js": "^1.2.7",
    "@inlang/paraglide-js-adapter-vite": "^1.2.17",
    "@nextui-org/react": "2.2.10",
    "@prisma/client": "4.16.2",
    "@react-pdf/renderer": "3.3.8",
    "@remix-run/express": "2.8.1",
    "@remix-run/node": "2.8.1",
    "@remix-run/react": "2.8.1",
    "@sentry/remix": "^7.106.0",
    "@tanstack/react-virtual": "^3.1.3",
    "@uidotdev/usehooks": "^2.4.1",
    "address": "^2.0.2",
    "bcryptjs": "2.4.3",
    "buffer": "^6.0.3",
    "chalk": "^5.3.0",
    "chokidar": "3.5.3",
    "classnames": "^2.5.1",
    "close-with-grace": "^1.3.0",
    "compression": "1.7.4",
    "cross-env": "7.0.3",
    "date-fns": "3.0.6",
    "execa": "^8.0.1",
    "express": "4.18.2",
    "express-rate-limit": "^7.2.0",
    "framer-motion": "10.16.16",
    "fs-extra": "^11.2.0",
    "get-port": "^7.0.0",
    "glob": "^10.3.10",
    "helmet": "^7.1.0",
    "i18n-iso-countries": "^7.10.1",
    "intl-parse-accept-language": "^1.0.0",
    "isbot": "3.7.0",
    "jsonwebtoken": "^9.0.2",
    "lucide-react": "0.309.0",
    "morgan": "1.10.0",
    "prom-client": "15.0.0",
    "react": "18.2.0",
    "react-dom": "18.2.0",
    "react-select": "^5.8.0",
    "react-window": "^1.8.10",
    "remix-auth": "^3.6.0",
    "remix-auth-form": "1.4.0",
    "remix-auth-jwt": "^0.2.6",
    "remix-flat-routes": "^0.6.4",
    "remix-paraglidejs": "^1.0.1",
    "remix-utils": "^7.5.0",
    "resend": "2.0.0",
    "source-map-support": "0.5.21",
    "tailwind-variants": "0.2.1",
    "tsx": "^4.7.1",
    "zod": "3.22.4"

Expected behavior
No warning should appear in the console and buttons should have padding.
Screenshots or Videos
On v 2.2.10:

On v 2.3+:

Note that it is not related to icons or whatever, I have tested it with a simple Button and the error still appears.
Operating System Version
macOS
Browser
Chrome

After upgrading to v2.3.5 and then downgrading back to 2.2.10 i also get the originalProps message in the console:
Warning: React does not recognize the `originalProps` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `originalprops` instead. If you accidentally passed it from a parent component, remove it from the DOM element.
    at span
    at eval (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@nextui-org+avatar@2.0.24_@nextui-org+system@2.0.15_@nextui-org+theme@2.1.18_react-dom@18.2.0_react@18.2.0/node_modules/@nextui-org/avatar/dist/chunk-XBMXQVV7.mjs:19:369)
    at eval (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@nextui-org+popover@2.1.15_@nextui-org+system@2.0.15_@nextui-org+theme@2.1.18_@types+react@18_rpiq7salf4josfdnycf52i4epy/node_modules/@nextui-org/popover/dist/chunk-K4VREUER.mjs:25:115)
    at eval (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@nextui-org+dropdown@2.1.17_@nextui-org+system@2.0.15_@nextui-org+theme@2.1.18_@types+react@1_ojabnux36tv4bbrelepzf5tuq4/node_modules/@nextui-org/dropdown/dist/chunk-63RZRIL7.mjs:17:108)
    at eval (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@nextui-org+popover@2.1.15_@nextui-org+system@2.0.15_@nextui-org+theme@2.1.18_@types+react@18_rpiq7salf4josfdnycf52i4epy/node_modules/@nextui-org/popover/dist/chunk-I7AEEVL3.mjs:23:13)
    at Dropdown (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@nextui-org+dropdown@2.1.17_@nextui-org+system@2.0.15_@nextui-org+theme@2.1.18_@types+react@1_ojabnux36tv4bbrelepzf5tuq4/node_modules/@nextui-org/dropdown/dist/chunk-FHP3QEBP.mjs:19:13)
    at li
    at Conditional (webpack-internal:///(app-pages-browser)/./src/components/Conditional.tsx:8:11)
    at div
    at ul
    at div
    at Navbar (webpack-internal:///(app-pages-browser)/./src/components/Navbar.tsx:52:74)
    at PlanConfigurationProvider (webpack-internal:///(app-pages-browser)/./src/context/PlanConfigurationContext.tsx:15:11)
    at f (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next-themes@0.2.1_next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next-themes/dist/index.module.js:8:597)
    at $ (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next-themes@0.2.1_next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next-themes/dist/index.module.js:8:348)
    at div
    at $f57aed4a881a3485$var$OverlayContainerDOM (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@react-aria+overlays@3.21.1_react-dom@18.2.0_react@18.2.0/node_modules/@react-aria/overlays/dist/import.mjs:1069:54)
    at $f57aed4a881a3485$export$178405afcd8c5eb (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@react-aria+overlays@3.21.1_react-dom@18.2.0_react@18.2.0/node_modules/@react-aria/overlays/dist/import.mjs:1036:21)
    at $f57aed4a881a3485$export$bf688221f59024e5
    at $18f2051aff69b9bf$export$a54013f0d02a8f82 (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@react-aria+i18n@3.10.2_react@18.2.0/node_modules/@react-aria/i18n/dist/real-module.mjs:170:19)
    at NextUIProvider (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@nextui-org+system@2.0.15_@nextui-org+theme@2.1.18_react-dom@18.2.0_react@18.2.0_tailwind-variants@0.2.1/node_modules/@nextui-org/system/dist/chunk-HRMF5D3P.mjs:15:3)
    at QueryClientProvider (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/@tanstack+react-query@5.29.2_react@18.2.0/node_modules/@tanstack/react-query/build/modern/QueryClientProvider.js:27:11)
    at Providers (webpack-internal:///(app-pages-browser)/./src/app/providers.tsx:52:74)
    at SupabaseProvider (webpack-internal:///(app-pages-browser)/./src/components/SupabaseProvider.tsx:17:11)
    at body
    at html
    at RootLayout (Server)
    at RedirectErrorBoundary (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/redirect-boundary.js:74:9)
    at RedirectBoundary (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/redirect-boundary.js:82:11)
    at NotFoundErrorBoundary (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/not-found-boundary.js:76:9)
    at NotFoundBoundary (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/not-found-boundary.js:84:11)
    at DevRootNotFoundBoundary (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/dev-root-not-found-boundary.js:33:11)
    at ReactDevOverlay (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/react-dev-overlay/app/ReactDevOverlay.js:87:9)
    at HotReload (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/react-dev-overlay/app/hot-reloader-client.js:321:11)
    at Router (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/app-router.js:207:11)
    at ErrorBoundaryHandler (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/error-boundary.js:113:9)
    at ErrorBoundary (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/error-boundary.js:160:11)
    at AppRouter (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/components/app-router.js:577:13)
    at ServerRoot (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/app-index.js:112:27)
    at Root (webpack-internal:///(app-pages-browser)/../../node_modules/.pnpm/next@14.2.2_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/client/app-index.js:117:11)

I dug into the issue because I can't upgrade from v2.2 to v2.3 because of this error.
It appears that the client is adding a unit- prefix to some Tailwind classes.
Server:
... min-w-20 h-10 text-small gap-2 rounded-small [&>svg]:max-w-[theme(spacing.8)]
Client:
... min-w-unit-20 h-unit-10 text-small gap-unit-2 rounded-small [&>svg]:max-w-[theme(spacing.unit-8)]
Does it ring a bell to anyone?

Thank you for this. I don't really know what happened, but for the record, after I deleted my package-lock file and re-bundled everything, it worked.


[Feature request]: Display the live count of votes to reach quorum in Governance

Task description
Implement fields in the overview of a governance proposal that display:

the number of votes that determine the quorum at the current milestone during counting
the number of total votes that have currently been counted

Display both as a number and percentage value of the circulating supply.
Highlight if the proposal has reached the quorum (green color percentage) or is below the quorum (red color percentage)
Requirements
For IOTA Firefly:
To calculate the votes that determine the quorum:
Calculate the maximum possible votes at a milestone:

(number of milestones passed) * (circulating supply in micros/1000)

circulating supply is taken from the API endpoint: https://api.stardust-mainnet.iotaledger.net/api/supply/v1/circulating, which is configured in IOTA, so it needs to be calculated * 1.000.000
take the result and calculate 5% of it (our current quorum)

To calculate the live count of votes, combine the counted votes for all options of a proposal, including "Abstain."
For Shimmer Firefly:
To calculate the votes that determine the quorum:
Calculate the maximum possible votes at a milestone:

(number of milestones passed) * (1,813,620,509,061,360/1000)
take the result and calculate 5% of it (our current quorum)

To calculate the live count of votes, combine the counted votes for all options of a proposal, including "Abstain."
As we may want to change the quorum in the future, we could think about how to update the quorum percentage without creating a new Firefly release. It could also be helpful to create an endpoint for this.
Creation checklist

[X] I have assigned this task to the correct people
[X] I have added the most appropriate labels
[ ] I have linked the correct milestone and/or project

Duplicate of https://github.com/iotaledger/firefly/issues/7644


Add screenshots of fearch to README.md

Add 2 to 3 screenshots of Fearch to README.md showing the main add-on popup, results page and a ftp index page.

I would like to work on this.

@ZER-0-NE thats great, please go ahead!!


Question : Custom model : Instructor does not support multiple tool calls, use List[Model] instead.

Hi,
I am trying to use a custom openai endpoint model (basically a service with openai api proxy that runs a custom model behind the scene, pretty much like ollama but fully compatible with openai module).
I am having this  error:
Task exception was never retrieved
future: <Task finished name='Task-14' coro=<generate_related_queries() done, defined at /home/hangyu5/Documents/Gitrepo-My/AIResearchVault/repo/LLMApp/farfalle/src/backend/related_queries.py:45> exception=AssertionError('Instructor does not support multiple tool calls, use List[Model] instead.')>
Traceback (most recent call last):
  File "/home/hangyu5/Documents/Gitrepo-My/AIResearchVault/repo/LLMApp/farfalle/src/backend/related_queries.py", line 53, in generate_related_queries
    related = await client.chat.completions.create(
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/instructor/client.py", line 273, in create
    return await self.create_fn(
           ^^^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/instructor/patch.py", line 119, in new_create_async
    response = await retry_async(
               ^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/instructor/retry.py", line 219, in retry_async
    async for attempt in max_retries:
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/tenacity/_asyncio.py", line 123, in __anext__
    do = await self.iter(retry_state=self._retry_state)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/tenacity/_asyncio.py", line 110, in iter
    result = await action(retry_state)
             ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/tenacity/_asyncio.py", line 78, in inner
    return fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/tenacity/__init__.py", line 410, in exc_check
    raise retry_exc.reraise()
          ^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/tenacity/__init__.py", line 183, in reraise
    raise self.last_attempt.result()
          ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/instructor/retry.py", line 226, in retry_async
    return await process_response_async(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/instructor/process_response.py", line 75, in process_response_async
    model = response_model.from_response(
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/instructor/function_calls.py", line 115, in from_response
    return cls.parse_tools(completion, validation_context, strict)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/hangyu5/anaconda3/envs/farfalle/lib/python3.11/site-packages/instructor/function_calls.py", line 201, in parse_tools
    len(message.tool_calls or []) == 1
AssertionError: Instructor does not support multiple tool calls, use List[Model] instead.

Simply want to confirm that it is my model that cannot proceed with these function calls, nothing to do with instructor or farfalle, right?
Thanks!

Thanks!


[Event Request] codeunit 442 "Sales-Post Prepayments" - Add parameters to function OnBeforePostCustomerEntry

Dear ALAppExtensions Team,
Is it possible to add 2 parameters to function OnBeforePostCustomerEntry in codeunit 442 "Sales-Post Prepayments"
-OnBeforePostCustomerEntry(GenJnlLine, TotalPrepmtInvLineBuffer, TotalPrepmtInvLineBufferLCY, SuppressCommit);
+OnBeforePostCustomerEntry(GenJnlLine, TotalPrepmtInvLineBuffer, TotalPrepmtInvLineBufferLCY, SuppressCommit, SalesHeader, DocumentType);

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Questions about dependency versions provided in requirements.txt

Dear Authors,
Thanks for the great work.
I was able to install mvTCR and run your tutorials in a conda environment, but encountered several problems which I suppose were caused by differences in dependency versions.
With scirpy=0.6.1, I ran into errors when running the following scripts as ir.pp.merge_with_ir(adata, adata_tcr) did not create the IR_VDJ_1_junction_aa information in adata.obs when merging the TCR and RNA data
notebook: /mvTCR/tutorials/01_preprocessing.ipynb
adata = adata[~adata.obs['IR_VDJ_1_junction_aa'].isna()]
adata = adata[~adata.obs['IR_VJ_1_junction_aa'].isna()].copy()

Switching to scirpy=0.7.0 resolved this issue.
Another problem was caused by optuna. optuna.structs is removed in optuna=3.0.3 (which was automatically installed since the version was not provided), and I haven't been able to locate the correct version of optuna...
Could you please provide the specific optuna version that was used?
Thanks for you time!

Hi,
Thanks for your interest in our tools.
I updated the requirement file for a scirpy version >=0.7 and the optuna versions 2.10.1, which we used for our tests.
This will hopefully resolve your issue.
Thanks and best,
Felix


How do you get synonyms JSON file from the download TSV files?

The workflow to load Babel TSV data to SOLR is missing a crucial step: converting the TSV to JSON files (containing more info than the TSV, e.g. pref label and type)
In the Makefile we can see there is step 1 to download the TSV:
https://github.com/TranslatorSRI/NameResolution/blob/master/data-loading/Makefile#L32
Step 2 is missing (probably the step that generates JSON and enrich the data_
Step 3 is solr starting, step 4 is loading the JSON files
Any idea where I could find this step?

My apologies, @vemonet -- I just forgot to update that URL, which is pointing to the previous release of the synonym files in a JSON format. I've just updated the URL to https://stars.renci.org/var/babel_outputs/2023nov5/synonyms/, which has the synonym files in the new JSON format (PR #139). Please let me know if you run into any problems with running this script and Docker image!

Closed by #139.


Add From field to NotificationRequest struct.

Hey @tbruyelle! While using the NotificationRequest struct from your library, I noticed that there was no support for the from parameter supported by the send_room_notification API endpoint:
https://www.hipchat.com/docs/apiv2/method/send_room_notification
Thus, the purpose of this PR is to add the From field to the NotificationRequest struct.

Thanks!

@tbruyelle no problem, thanks for the quick merge :+1:


Occasional Bitstamp API error

Every now and then, I get an error like this:
Server Response: 503 - https://www.bitstamp.net/api/ticker
Error with JSON: '[' or '{' expected near '<'
Buffer:
<META NAME="ROBOTS" CONTENT="NOINDEX, NOFOLLOW"><meta name="format-detection" content="telephone=no"><meta name="viewport" content="initial-scale=1.0"><meta http-equiv="X-UA-Compatible" content="IE=edge,chrome=1">Request unsuccessful. Incapsula incident ID:<PHONE_NUMBER>00000519-2195046149521845
  Retry in 2 sec...

Usually, it only appears once, but today it happened continously for about 50 times or so and it's still happening. If anyone has any idea where it's coming from or why it is happening, feel free to comment here. It is weird, since it usually only happens once, so I guess there isn't actually a syntax error in the JSON on blackbirds side...

Hi @LasseMoench take a look at this comment about a timeout issue with Bitstamp and Incapsula.

Ok so according to that comment and the reddit thread mentioned there, Incapsula (Bitstamps anti-DDOS provider) is the reason for this. I will ask Bitstamp support if there is anything that can be done to avoid it, e.g. lowering the interval for querying.

Did anyone here arrive at a strategy for handling this type of issue? I'm seeing long request duration's (which turn into timeouts for me).

Did anyone here arrive at a strategy for handling this type of issue? I'm seeing long request duration's (which turn into timeouts for me).

My strategy is to handle this response as a special case and change the status code to 500, which then causes a retry.

@runeksvendsen :(


Project carry over

Hello,
I was seeing if there was a way to carry over my current file project to the new update. First time modding and using unity so wasn't sure how to carry a current project file I'm working on to the latest LethalSDK update.

Hello,
I was seeing if there was a way to carry over my current file project to the new update. First time modding and using unity so wasn't sure how to carry a current project file I'm working on to the latest LethalSDK update.

Just drop the new update on it ane overwrite
Beware, it will overwrite the scenebuilder so save your prefabs


#497 Allow metrics on custom port with custom host configuration

This PR allows for serving the metrics to a custom port, but removes the automatic configuration of the host as described in #497
Confirm the following

[X] I have ensured that I have merged the latest changes from the dev branch
[X] I have successfully run a local build
[ ] I have included unit tests for the issue/feature
[X] I have included the github issue number in my commits

Ticket  to remove left over configuration and logging

Could this change please be merged to the AppMetrics:features/4.2.0 branch.
Sorry this isn't clear in the documentation

I rebased it and changed the branch to features/4.2.0, initially I did do this change based on that branch, but when I saw the documentation, I rebased it to dev...


Class and function dropdown boxes for code navigation

Requesting navigation feature that has been in Visual Studio (since before .NET, at least VS 6).
For relevant languages, at the top of each editor window would be one or more dropdown boxes for navigating classes and functions.  The easier implementation would be a single dropdown that contained function names in alphabetical order, allowing you to jump directly to it.  If implementing more like Visual Studio, then two dropdown boxes, one for objects (classes, structs etc) and another dropdown for functions.
Image of Visual Studio implementation:

Side note:
It seems the current methods available for navigation are either A) a lot of haphazard scrolling or B) use "Find" feature.  The latter is not the right tool for the job because often a large amount of extraneous results are present, making it tedious.
Thanks team!  Loving VS Code

Did you try quick outline? Ctrl + shift + O? It works quite nice for javascript, I did not try it for .cs

I have not, and that looks significantly better than standard Find!  Thank you.  I did notice it's not in alphabetical order, so using the realtime filtering may be required for many/most.  It's separated by the "type" in Typescript (classes, functions, variables) which is great.
The alphabetizing aspect is a separate issue, and I may raise a separate feature request for that option.
At this point I think the only advantage this proposal has is for allowing a "legacy" method of navigation.  Otherwise it has no additional value.
Thanks!

Yes - making quick outline alphabatically sorted makes sense as configuration option.
You can also easily filter by typing in there which does the trick for me most of the time.
Closing this issue since there is a replacement feature and I do not think we would consider adding a 'legacy' method of navigation that does not really fit in an editor.


[#481] Have the DefaultSqlClientPool deliver a different Pool instanc…

…e for each Vert.x context
Fixes #481

updated, improved some minor typos and comments


Using a stream

Hello, I would like to use your library in order to start a filtered stream that will follow some users and some tags. I see that creating the stream is easy, but I don't know how to make the client aware of the list of users to follow (and the list of tags). Could you point me to an example? Thank you!

Hi, just use tweet.getUser() to get the user, or  twitterClient.follow(tweet.getUser().getId() to follow the related user.

Thank you! Will try it out

I close the ticket, if you have any issue don't hesitate to re-open it


XDE fails to maintain the logs scroll-lock to the bottom of the logs

Results in lots of scrolling to see output.

Think this is fixed in https://github.com/exponentjs/xde/commit/07d27c9faca5baaf59488c16ddfacb18b8fb2be6. I'll let you confirm before closing

Thanks @jesseruder this is working fine now!

This doesn't seem to be working as of Expo SDK 2017-12-04 (I'm not sure what version it is, since there is no intutitive way to get its version. Additionally: Help > Send Diagnostics Report is not working. Additionally: There are two "Help" menus, each one are different)


nit fix for #20

Signed-off-by: SamYuan1990<EMAIL_ADDRESS>

in this pr, make sm2 benchmark comparing with ecdsa.

in this pr, make sm2 benchmark comparing with ecdsa.


Some of the improvements I've made to this for own use but I like to share it

This PR should definitely not be merged straight away (missing tests, coding style, and some random debugging code), but I hope this PR will help improve this
The main improvements are as follows:

When a worker crashes, the batch snapshots it was responsible for are lost and cannot be taken over by other workers. Solved by refactoring to a FIFO Queue
When the program gets stuck, an incorrect exit can cause skipfile not to be saved. Solved by periodic saving.
Checking skipset in the download method was very slow (due to things like Python's GIL, progress bar update stuff), and now skipset checking will be done before starting workers. (Handle 129232 skips in 27mins -> Handle 475784 skips in <30secs)
Now while checking the skipset, the local downloaded files are also checked in the file system and added to the skipset. In addition to improving performance, this also allows you to quickly recover skips in case of skipfile corruption.
Improved the progress bar related code to allow fast forwarding multiple steps.

please review my implementations as i wanted to do less deep changes in the code-logic

请查看我的实现，因为我想对代码逻辑进行较少的深度更改

Why not process the skipset before starting the workers to start processing snapshots?
Processing skipset in download will generate a lot of method calls and a lot of v.writes.
Python's text output performance is not good enough.
Considering that in my patch, processing it earlier resulted in a time savings of over 30 minutes, it seems well worth it.

please review my implementations as i wanted to do less deep changes in the code-logic

Why not process the skipset before starting the workers to start processing snapshots? Processing skipset in download will generate a lot of method calls and a lot of v.writes. Python's text output performance is not good enough.
Considering that in my patch, processing it earlier resulted in a time savings of over 30 minutes, it seems well worth it.

forgot about that one sorry. you are right

please review my implementations as i wanted to do less deep changes in the code-logic

Why not process the skipset before starting the workers to start processing snapshots? Processing skipset in download will generate a lot of method calls and a lot of v.writes. Python's text output performance is not good enough.
Considering that in my patch, processing it earlier resulted in a time savings of over 30 minutes, it seems well worth it.

forgot about that one sorry. you are right

please review my implementations as i wanted to do less deep changes in the code-logic

Why not process the skipset before starting the workers to start processing snapshots? Processing skipset in download will generate a lot of method calls and a lot of v.writes. Python's text output performance is not good enough.
Considering that in my patch, processing it earlier resulted in a time savings of over 30 minutes, it seems well worth it.

forgot about that one sorry. you are right

hi A friendly reminder - don't forget to fast forward the progress bar! It will look weird.
Verbosity's wrtie function only allowed me to increase by 1, so I changed it to save a lot of function calls (by fast-forwarding all at once).
Based on my testing, it performs terribly if the progress bar is updated in a loop.


Improvements for dry-run cli

From the comments on #52 and #48:

We could avoid having to specify the Galaxy job conf by reading galaxy.yml and parsing the job conf out of there. However, adding this functionality directly to TPV (including automatically locating galaxy.yml) is reinventing a wheel (and a rather complicated and messy one) that is already done by Gravity and Galaxy itself. Ideally we would separate this out into a library that all 3 projects could then use.
The dry-run is not going to work for more complex custom rules because we are mocking Galaxy objects that won't have all of the attributes that a custom rule might access (e.g. object_store). We can look at a couple of things, including Galaxy's IntegrationTestCase and the minimal app that celery workers load.
There is currently no way to specify any tool param values or input sizes when performing a dry run. EDIT: --input-size added to #52.

An additional enhancement from the original issue suggested by @mira-miracoli:
Add explanation for which destination was chosen and why. eg:
RESULT for .....
--------------------------------------------------------------------------------------
DESTINATION
final: condor_tpv
matched(with ranks): condor_tpv(1), singularity(2), interactive(3)
--------------------------------------------------------------------------------------
PARAMS
cores: 8 (min of tool: 10, user: 8, role: 9)
mem: ....
...

Additionally an explanation of how the rank was calculated in this specific case, would be helpful to debug errors in the configuration and check if it works as intentioned.


PyPy: fix overstrict regex, allow CYTHON_UPDATE_DESCRIPTOR_DOC

Fix over-strict regex in error message check that started failing in tests fro PyPy3.11 which is pre-release. From a discussion in pypy/pypy#5160

CPython is unfortunately super inconsistent about when to use the the qualname in error messages, and when to use only the method name

We went back and forth with enabling CYTHON_UPDATE_DESCRIPTOR_DOC, I think this time I fixed it (as of py7.3.17).

Thanks!


How to prevent swipe up/swipe down

For elements where I only want to enable swipe left/right, how do I disable swipe up/down?

just do not use swipe up/down directive, or return false in swipe up/down method.


Support PHONY variable

Support:
PHONY += all
all:
   true

.PHONY: ${PHONY}

It currently gives the error:
Target "all" should be declared PHONY

I don't think we can reasonably support runtime variables in a static linter like this because this more or less requires us to be able to execute the Makefile and inspect generated variable content.
Please feel free to reopen this issue if anyone has a suggestion how to do this.


Android - Workspace-Closing the error message "unable to add members to WS", deletes the WS

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.66
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Launch app
Tap fab -- create a new workspace
Tap members invite
Invite a VOIP number +916541237890
Note user not added
Tap invite button again, note the red dot below and invite the user again
Note user is not added
Tap back button and note 2 error message "unable to add members to Workspace"
Tap cross button and close the error message

Expected Result:
Closing the error message "unable to add members to Workspace", must not delete the workspace
Actual Result:
Closing the error message "unable to add members to Workspace", deletes the workspace
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/4ca18dff-85e3-48f8-988a-4d6de25c5812
View all open jobs on GitHub

We think that this bug might be related to #vip-vsp

Closing in favor of the other issue


Cache unparsed request body (configurable)

The BodyHandler (light-4j/body) currently parses the body of the request and caches the parsed body as an attachment in REQUEST_BODY.
There are use cases when the "unparsed" body is useful for some operations and avoids a serialization effort.
The body.yml needs to be augmented with a property, with a default value set to "false".
Teams which will use it will set it to true as per their use case.
...# Enable body parse flag
enabled: true
...# cache request body
cacheRequestBody: false
The BodyHandler would have to be augmented and should save, if the above flag is set to true, the unparsed body as a String, as an attachment in REQUEST_BODY_STRING.
NOTE: light-codegen already parameterizes the body.yml at this time, therefore no changes are required in the codegen utility

Hey! @ddobri @stevehu I encounter a problem when trying to implement caching unparsed request body when the content type is "multipart/form-data" and look forward to your helps.
Due to the extended problem of undertow, the only way to parse the form-data is using the method formParserFactory.createParser(exchange), which means we need to reuse the input stream inside the undertow exchange. However, the undertowInputStream didn't override the reset() method and cannot be reused. So, after caching the unparsed body from the undertowInputStream, we cannot parse the body from the exchange again since the input stream already closed.
It seems the only way to solve it is to write an ourselves form-data parser. Do we need to do that? Or, just implement the caching for Json format? Since I think there is nearly no use cases for unparsed form-data body.

@jiachen1120 : If you consider that this issue can not easily be addressed, then writing an additional parser is not warranted.
There are a couple of use-cases for it, at least the ones which I am aware of.
Please reassign the issue to me and I will close it in the next few days, after investigating a couple of different ideas.
You can label it as "to be analyzed" or something you see fit

@ddobrin @jiachen1120 I guess the use cases you are talking about are for JSON only. In my opinion, there is no need to keep the original string for form-data as there is no original string at all. As our body parser differentiate body with the content type, we can just keep the original string for the application/json. What do you think?

@stevehu @ddobrin Thanks for your help!!!
I just created a PR which included:

implementation of caching application/json request body string
refactored the bodyHanlder
test cases

I agree @stevehu opinion that there is no need to keep the original string for form-data as there is no original string.


Provide better details on the adapter hardware

https://github.com/Bluetooth-Devices/usb-devices/issues/3

closed via #26


Move layer if layer order is changed

Description
if layer is moved below, the drag drop destination provides where layer should be added before.
if layer is moved above, the drag drop destination provides where layer should be added above. Hence, move to next layer to find before layer index.
Signed-off-by: Vijayan Balasubramanian<EMAIL_ADDRESS>Demo:
https://user-images.githubusercontent.com/11067894/209039792-f878053a-0905-4d19-a369-c260d141e4ff.mov
Issues Resolved
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

This pr works only for document layer reorder. Will raise new PR for OpenSearchMap

The documents layer supports point, line and polygon type. For polygon type specifically, it actually has two layers in maplibre: fill layer and boundary layer, looks like this PR support documents point and line type(single layer), might need check on polygon type.


Enable feature to fix unstabilized #[bench]

rust-lang/rust#62507

This will cause this crate to become nightly only, #25 is a better fix that will still allow building on stable 1.38+.

Inspired by the fix in cargo, missed the one you linked. Feel free to close if that one gets merged.

ping @alexcrichton

yeah if https://github.com/rust-lang/cargo/pull/7198 was the inspiration, that's just tests...

Sorry, but I am no longer maintaining this crate.

But who is? Perhaps it should be transferred to rustasync since it is used by tide?

Perhaps it should be transferred to rustasync since it is used by tide?

We'd be happy to help share the maintenance burden if desired!

@vorot93 thanks for this, I've merged #25 instead and published 0.1.13 now which should work on current nightlies.


Add BRZ: BRZ Token

[X] #13

Please provide the following information for your token.
Token Name: BRZ
Token Symbol: BRZ Token
Decimals: 4
Polygon Address: 0x491a4eB4f1FC3BfF8E1d2FC856a6A46663aD556f
Ethereum Address: 0x420412e765bfa6d85aaac94b4f7b708c89be2e2b
Logo URI: https://brztoken.io/img/svgs/brz.svg
Project Name: Brazilian Digital Token
Project Summary: The BRZ Token is the Brazilian stablecoin in circulation that allows Brazilians to directly access different pairs of assets on exchanges and to trade a stable currency pegged to the brazilian real (BRL) on a global scale.
Project Contact<EMAIL_ADDRESS>Project Website: https://brztoken.io/

This token doesn't seemed to be mapped. To map the token please raise a request on https://mapper.polygon.technology. Closing this issue.


On MySQL engine parameter is hard-coded to InnoDB, this brakes replication to a diffrent storage engine

Which version and edition of Flyway are you using?
8.0.0
If this is not the latest version, can you reproduce the issue with the latest one as well? (Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Yes, I can
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command line
Which database are you using? (Type & version)
MySQL 5.6, 5.7
Which operating system are you using?
Ubuntu
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
Applying flyway migration to a new schema, to a database server that has a replica running a different storage engine.
Ref https://dev.mysql.com/doc/refman/5.7/en/replication-solutions-diffengines.html
What did you expect to see?
Flyway migration applied successfully.
Database changes from migration applied successfully to the replica server, running a different storage engine.
What did you see instead?
Flyway migration applied successfully.
Replication stops with the error "cannot create table 'migrations'"
Additional info:
On MySQL engine parameter is hard-coded to InnoDB, this brakes replication to a diffrent engine, when flyway runs against a schema for the first time.
https://github.com/flyway/flyway/blob/75c3b9027ad86861f93977c176ac4b451806723b/flyway-mysql/src/main/java/org/flywaydb/database/mysql/MySQLDatabase.java#L160-L174

Any updates here?  Let me know if you need any details ?

@PabloThiele thank you for  the reminder about support timeline for MySQL 5.7
However, the problem raised in this issue  is not specific to MySQL 5.7 and it is not really addressed by your comment.
It has to do that in flyway storage engine definition  is hard-coded to be  InnoDB.
MySQL supports various storage engines , and there are cases when setting replication with different storage engines on master & replicas are beneficial.    That 'cross storage engine' replication will be broken by flyway creating migrations history table with ENGINE=InnoDB -- create table statement with ENGINE=InnoDB will be pushed to binlog on the primary (master) database and then this statement can cause failure in SQL Slave applying binlog changes on the replica if InnoDB storage engine is not available.
I think this problem can simply be solved removing specific storage engine from table definition and using the default. I do not see any benefits of hard-coding storage engine to be InnoDB.  Or possibly storage engine specification can be made as configurable option, but that will add complications and more code to maintain, so  I would consider this option if there are benefits to it.
I would like to get some feedback on https://github.com/flyway/flyway/pull/3642
Thanks you,
-Konstantin
If
See  https://github.com/flyway/flyway/pull/3642

UP.

Thank you for your contribution, this has been incorporated into Flyway 10.22.0 released yesterday


Suggest a zap to dev post download - during update + zap message

@franzaps recommends zapping dev when they are updating

[ ] "We noticed you downloaded app X, and are updating the app. If you love "vitor's" work on "Amethyst", zap him to show your support.

[ ] add personalized picture of dev(s), and an appeal to donate. For instance, https://vitorpamplona.com/me.jpg

[ ] add optional zap message. placeholder text ("thank you Vitor. Btw please fix the XYZ bug")

question

how is the consumer notified / prompted of the suggestion to zap the dev
how might ZS distribute zaps to multiple team members https://github.com/zapstore/zapstore/issues/85

relates to proof of missed zaps for non-nostr devs https://github.com/zapstore/zapstore/issues/70

I was thinking of a dialog box (popup) once the user taps update and the update downloads in the background. Sporadic not always, and possible to disable in settings

My non-expert UI relationship with pop-ups is that they are associated with a negative experience. Maybe @NielLiesmons can advise

Gotcha. Maybe a slide down when the user taps update. Something they don't have to close and can continue using the app

current update app flow

Example prompt to zap dev flow above. Blu ZS zap icon should take user to their LN wallet.

This is greeeeaaat!

This is greeeeaaat!

Help make this looks sexy-time @NielLiesmons
My design chops do not exude confidence 😂

And pyschologically, there is no better moment to ask for a zap

And pyschologically, there is no better moment to ask for a zap

It would be nice to test this somehow. I know no data is collected.
--
Could we pre-populate an anon/private LN/zap message with "sent with love from zap store".
--
Right now download update is a manual action. Maybe auto-update could play a role in jncentivizing pre-paid support of devs?


Fix _disconnecting context manager for RedisType

StrictRedis does not have a disconnect method.

Coverage remained the same at 89.326% when pulling 1931df5cfb7fa1bfbe896e157b6ab333fcf92a03 on temclaugh:dev into 9ea2dff5949ca6bc7a59acb2f3d942894ab570cd on kensho-technologies:dev.

Coverage remained the same at 90.18% when pulling 1f2087d267a640180983194e279ab346faaace4f on temclaugh:dev into bc254884c558fc413ca8b0743d1c1d2405473b9d on kensho-technologies:dev.


Addition of an extra state so that it behaves as a tri state switch

Want to implement something like a switch that has three states, "ENABLED,PENDING,DISABLED" . Is it possible that something like this can be implemented
Something like this?

There is another Component: https://github.com/ItsAsbreuk/itsa-react-doubletogglebutton that has the property state which can be 0, 1 or 2. Could this be of any value to you?


Remote Container is not working

I am currently trying to use the Remote Containers setup for this repository (as described in the README) for dd1fba76003434639209627676febc595787ba7c. However, I cannot get it to work:

Running bundle install in the omnibus directory from the console of the remote container fails with:[!] There was an error while loading `dependabot-bundler.gemspec`: No such file or directory - helpers. Bundler cannot continue.

Running the launch target "Debug dry run" does nothing after I select a repository to scan. The "Debug Output" tab of VS Code shows:bundler: command not found: rdebug-ide
Install missing gem executables with `bundle install`

Opening any Ruby file, waiting for the extensions to be activated (see progress indicator in bottom bar of VS Code) causes a bunch of warnings and errors:

It appears this might be related to the way the Remote Container is created. Due to its usage of Dockerfile.development it seems to contain the files of this project twice:

${HOME}/dependabot-core: Created and set up by the Dockerfile. Only contains a subset of all project files, manually mapped using Docker bind mounts here.
/workspaces/dependabot-core: Contains the complete project. This directory seems to be created by the Remote Containers extension and is a copy of the project files from the host OS.

Note that I just wanted to try out something with Dependabot, and most likely won't be using the Remote Container again, so this is not blocking me.
However, I thought this GitHub issue might be useful in case others are experiencing this issue as well.
Feel free to close it.
VS Code
Version: 1.60.0 (user setup)
Commit: e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff
Date: 2021-09-01T10:41:52.311Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19043

Remote - Containers version v0.194.0

I'm fairly sure this was resolved in https://github.com/dependabot/dependabot-core/pull/5086.
Note that the underlying gems used for debugging will change when the upgrade to Ruby 3.1 lands, see the commit message in this PR about removing debase: https://github.com/dependabot/dependabot-core/pull/5447.


don't use bright colors for Terminal prompt

in many themes, we are using a bright color for the prompt. overkill.

instead:


Create multiple new objects and referencing one another

There is a relatively frequent request for an ISP to assign prefixes to new downstream customers. This means that several objects should be created for the new customer simultaneously.  An example of a complex arrangement might be a creation containing

a person object with "nic-hdl: auto-1”
a person object with "nic-hdl: auto-2”
a role object with "nic-hdl: auto-3” referencing person auto-1 as "admin-c" and person auto-2 as "tech-c"
an inetnum object referencing role "auto-3" as admin-c and tech-c

Hey @tvasilopo,
I understand the use case but AFAIK you can only reference existing objects - I guess it would break the consistency requirement for the database. So a sequential creation of objects is needed.
What is your question? What are you missing?

Hey @tvasilopo,
I understand the use case but AFAIK you can only reference existing objects - I guess it would break the consistency requirement for the database. So a sequential creation of objects is needed.
What is your question? What are you missing?

Hello @fischa ,
Actually you can create multiple objects at the same update and reference one another using the 'AUTO-n' functionality.
There is a reference in here, section 2.5.1
This can be done using the syncupdates but it would be nice to be also done from the tool.
As a matter of fact, in the meantime  I deployed it in a fork of this repo and works fine. The object data are inserted in a yaml file that has the following format:
AUTO-1:
- person:
- address:
- address:
- address:
- phone:          +30
- nic-hdl:        AUTO-1
- e-mail:
- mnt-by:
- source:         RIPE

AUTO-2:
- person:
- address:
- address:
- address:
- phone:          +30
- nic-hdl:        AUTO-2
- e-mail:
- mnt-by:
- source:         RIPE

AUTO-3:
- role:
- address:
- address:
- address:
- phone:          +30
- admin-c:        AUTO-1
- tech-c:         AUTO-2
- e-mail:
- remarks:        --------------------------------------
- remarks:        'For complains about abuse, spam etc:'
- abuse-mailbox:
- remarks:        --------------------------------------
- mnt-by:
- nic-hdl:        AUTO-3
- source:         RIPE

[inetnum inserted here]:
- inetnum:
- netname:
- descr:
- admin-c:        AUTO-3
- tech-c:         AUTO-3
- abuse-c:        AUTO-3
- status:         ASSIGNED PA
- mnt-by:
- mnt-domains:
- source:         RIPE
- country:

[route prefix inserted here]:
- route:
- descr:
- descr:
- origin:
- mnt-by:
- source:         RIPE

I need to find some time to send a merge request about it.

Hello @fischa ,
Actually you can create multiple objects at the same update and reference one another using the 'AUTO-n' functionality.
There is a reference in here, section 2.5.1
This can be done using the syncupdates but it would be nice to be also done from the tool.
As a matter of fact, in the meantime  I deployed it in a fork of this repo and works fine. The object data are inserted in a yaml file that has the following format:
AUTO-1:
- person:
- address:
- address:
- address:
- phone:          +30
- nic-hdl:        AUTO-1
- e-mail:
- mnt-by:
- source:         RIPE

AUTO-2:
- person:
- address:
- address:
- address:
- phone:          +30
- nic-hdl:        AUTO-2
- e-mail:
- mnt-by:
- source:         RIPE

AUTO-3:
- role:
- address:
- address:
- address:
- phone:          +30
- admin-c:        AUTO-1
- tech-c:         AUTO-2
- e-mail:
- remarks:        --------------------------------------
- remarks:        'For complains about abuse, spam etc:'
- abuse-mailbox:
- remarks:        --------------------------------------
- mnt-by:
- nic-hdl:        AUTO-3
- source:         RIPE

[inetnum inserted here]:
- inetnum:
- netname:
- descr:
- admin-c:        AUTO-3
- tech-c:         AUTO-3
- abuse-c:        AUTO-3
- status:         ASSIGNED PA
- mnt-by:
- mnt-domains:
- source:         RIPE
- country:

[route prefix inserted here]:
- route:
- descr:
- descr:
- origin:
- mnt-by:
- source:         RIPE

I need to find some time to send a merge request about it.

Cool! That certainly helps for this kind of changes. But isn't it a matter of templating that? Or what am I missing here?

Cool! That certainly helps for this kind of changes. But isn't it a matter of templating that? Or what am I missing here?

Well, if I understood your question correctly, we 'll have to add the logic that RIPE has deployed in the sync updates regarding the reference between objects that are simultaneously created. Ripeinator tool fails when the above template is provided in the current setup. It will fail in the AUTO-2 object creation as it will not find any person object with 'nic-hdl ' AUTO-1.

Well, if I understood your question correctly, we 'll have to add the logic that RIPE has deployed in the sync updates regarding the reference between objects that are simultaneously created. Ripeinator tool fails when the above template is provided in the current setup. It will fail in the AUTO-2 object creation as it will not find any person object with 'nic-hdl ' AUTO-1.

Now I think I got it... From what I get out of the documentation it is only possible to create a single object via the REST API and not a couple of objects like syncupdate or the mail interface.
Does your fork circumvent this?

Now I think I got it... From what I get out of the documentation it is only possible to create a single object via the REST API and not a couple of objects like syncupdate or the mail interface.
Does your fork circumvent this?


Name length issues for nlb module

I've been encountering issues with the NLB module name length:
Error: "name" cannot be longer than 32 characters

Specifically for these two lines:

https://github.com/rancherfederal/rke2-aws-tf/blob/master/modules/nlb/main.tf#L58
https://github.com/rancherfederal/rke2-aws-tf/blob/master/modules/nlb/main.tf#L76

I did notice there is already some name trimming/substringing happening here but it appears the port is putting it past the limit? It looks like the current names are trimmed down to 32 characters but the ports add an extra - + port number (~4 characters).
I haven't had a chance to dig into this further yet, as a workaround we just shortened our name that was being passed in.

I stumbled upon this as well, here is a quick PR to fix it
https://github.com/rancherfederal/rke2-aws-tf/pull/91

Fixed by #91


Global Authorization Token

Hi Madam/Sir
How to setup authorization bearer globally?
Thanks,

You seem to already know how to use bearer tokens in a single request, right? Then the last missing piece is the .defaults(...) function.

Hi @Serboy , I think it is safe to close this issue now. But feel free to reopen it if you still have questions.

Sure sire, forgot to close this, sorry about that, Thanks a lot

No problem. Have a great day!


Enable static methods :D

Enable static methods:
class Math {
	static square(n) {
		return n * n;
	}
}
writeln(Math.square(10));

Having static methods will become pretty handy for creating standard libraries in the future.


Support VacationResponse/get

See https://jmap.io/spec-mail.html#vacationresponseget

WHy? What needs to be done? DOD?

I am unsure vacations are a priority


Fix typo in calculator.md.

Thanks for reporting this. Merged into our internal code base, and will be pushed to GitHub in the next release.


Split config into multiple files and reread config without restarting pod

I would like to have the option to split the config into multiple files as in multiple files in a directory. Also it would be helpful to have cloudprober read changes to the config file and update without the need to restart the pod. Currently I have a dynamic setup on EKS where I have multiple probes defined and changes to one requires me to regenerate the config file and also restart the pod for the changes to take effect.
The option of using targets is not beneficial in this case but the underlying logic behind the targets file is what I would like to see for cloudprober.cfg. With the option of reading from S3 scheduled for some upcoming release, it would be good to have a way where cloudprober reads the updated config and reloads (without a restart). Filebeat does something similar

Live reload has come up a few times before. I've been reluctant of adding it because clearing a probe's (and other) state cleanly on its removal was not very well implemented. We've certainly improved in that area quite a bit and can look at it again.

FYI,
I am working on adding the include file support here: https://github.com/cloudprober/cloudprober/pull/643
You'll be able to create a top level file that includes other files. Example:
-- cloudprober.cfg --
include "cloudprober.d/team1.cfg"
include "cloudprober.d/team2.cfg"
include "cloudprober.d/team3.cfg"
@syd20 can you please change this ticket to just be multiple files, and file a separate one for reload config?

Changed the scope of ticket to create multiple config files

I've added multiple config files support to cloudprober helm-charts as well:
https://github.com/cloudprober/helm-charts/pull/28
Closing it now. I'll cut a new release next week.


We currently have no tests for the 'fetchOfflineNotifications' flag

Describe the bug

A recent PR added the fetchOfflineNotifications flag and made associated changes in NotificationServiceImpl. No tests were added for this feature. And as it happens, the implementation appears to be not correct or causing problems, as setting that Boolean to "false" causes the mwc_demo iot_receiver app to fail to ever sync

Expected behaviour

There should be unit tests and functional tests which verify correct behaviour when fetchOfflineNotifications is false as well as when true (the default)
We need to test exactly why setting fetchOfflineNotifucations to false somehow causes the mwc_demo iot_receiver client not to sync

Background: spent several hours yesterday with Colin & Kevin trying to figure out why their demo for the IOT expo had stopped working.
Eventually we figured out that the problem was that the client app wasn't syncing
The only thing that had changed was that fetchOfflineNotifications was set to false. Setting it back to true resulted in everything working normally
We logically concluded that impl of fetchOfflineNotifications had a bug. However my initial assumption of what that bug was looks to be incorrect
I will code up a small test harness to see if I can reproduce what Colin & Kevin were seeing. It is definitely the case that setting fetchOfflineNotifications to false caused the problem, but not at all clear as to why

Don't know what the issue was, just that setting that flag to false resulted in client not syncing. Should be impossible, right?

When this ticket was originally filed, it looked as if setting this boolean to false was causing sync problem with the iot_receiver demo app. That sync problem was maybe a coincidence, possibly a side-effect of https://github.com/atsign-foundation/at_widgets/issues/546


Organizer QrCode Screen (EXPOSUREAPP-5831)

To test the PR: Go to "Test Menu" -> "Event Registration" -> "Open Organizer Fragment"

Xiamoi Mi 9, Android 9
The gradient and the snappiness are working create!!! 🚀
I only found two minor things.
All corners are rounded, something light blue is shining through:

There is to much spacing between the Qr code and date/time, try it with 24dp:

@axelherbstreith during the UX Sync it came up that the background on this screen isn't white. It's a slight grey and it's like this on all screens with this big colourful bars.
Please also check what background we should use in the darkmode, IMHO the black background is still fine.


Use containers for actions

Currently, we are using an Ubuntu container to run actions. It will be better if we can use Node v16 container.

Currently, we are using an Ubuntu container to run actions. It will be better if we can use Node v16 container.

Correction: Please do not change the runs-on: ubuntu-latest VM. On top of Ubuntu, we want to run a Node.js Docker container.
So the only change is adding a container key to all the reminder and joining workflow jobs, with the value being a Node.js Docker container.

Ohh cool👍 my bad I thought that was the container

(Example<EMAIL_ADDRESS>
The link is broken.

Fixed @SirusCodes

Currently, we are using an Ubuntu container to run actions. It will be better if we can use Node v16 container.

Is there any specific reason to use the nodejs container?
We are not even installing any packages, so we just need node.
I think the nodejs container might slow the actions down with no real benefit.

Current implementation is half setup of node and pulling the image would take more time imo.
We can create a PR and then probably compare them

Current implementation is half setup of node and pulling the image would take less time imo.

I was thinking this as well, because we're doing the following for every Action:
https://github.com/OurTechCommunity/catchup/blob/66dfaae7ed3e4ed6e82bdcd0b6871276b3bec14b/.github/workflows/tg-joining-msg.yaml#L19-L20

I think the nodejs container might slow the actions down with no real benefit.

Doesn't caching come into picture here? Doesn't GitHub Actions cache repeatedly used images? I think it does, but I'm not sure.
As Darshan suggested, can we create a PR and compare the two? I'm not sure how we will measure this otherwise.
Thoughts, @KartikSoneji?

Before we do anything, can we double check if nodejs is already available on the runners?
I think it is:
(source)

Oh. I did not even think about that. Thank you for bringing it to our notice!
Well it appears that Node.js is supported, so the following two lines can be omitted from all the scripts in that case...
https://github.com/OurTechCommunity/catchup/blob/66dfaae7ed3e4ed6e82bdcd0b6871276b3bec14b/.github/workflows/tg-joining-msg.yaml#L19-L20
Again, we should test this first in a test repo.

As verified in #97, the default runner comes with nodejs pre-installed so installing it is not needed.
The step can be removed from the action.


fix: added missing comma in dashboard page generator

Hey!
When adding multiple custom dashboard pages I encounter an error where commas at the end of imports aren't being generated.

I was able to test it via the playground in the repo

woah that was fast thank you. I really love building with pruvious


Fixed object/members listing failing when name contains variant chars

Changes
Listing objects and/or members with names containing variant characters using a profile with CCSID different from 37 would result in members being displayed with American variant characters and physical source files not being displayed or being displayed as regular PF.
Example, with CCSID 297 variants characters:
Before this PR, when listing members

After this PR

Before this PR, when listing source physical files:
With Object type = *ALL

With Object type = *SRCPF

After this PR

Turns out the system tables contains the names in CCSID 37 and the OBJECT_STATISTICS table function does translates the names to the connection's CCSID before returning the results.
How to test this PR

Connect with a profile with CCSID other than 37
Create a member with one or more variant characters
It must be displayed correctly and can be opened.

Checklist

[x] have tested my change

Approved with manual testing and running the existing test cases.
Thanks @worksofliam !

I am not pressed for this PR, but is it at-all possible to have a test case that runs when the user CCSID is not 37 to automate this testing>?
There you go, I added a test that will run if CCSID is not 37. It can run on PUB400 as well, the library test part is simply skipped in this case.
Good thing you had me add this test since I eventually found a bug that remained (filtering on source files name would not work if the filter had variant characters).

Give it a last try and let me know what you think!


No catch I just want more fb cookies

I'll go first if you want @hassanila97

@hassanila97 trade Cookies? how many cookies do you have?

@nicolaspielte I'll trade I have 79

@ravyn120 trade Cookies i have 70? add me nicolaspielte (rαwq.)

@nicolaspielte I have 200+

@hassanila97 traden? :D i have 70

I got em from some friends...  so idk.. they dont want me to share :D

:( still need cookies pity

@hassanila97 nice <3 <3 <3

123.000 players in EU..   most are bots :P

xD

@hassanila97 will not you give me a few more cookies? :D

Sry I can't, otherwise I have no friends :D
On 22 April 2016 at 20:32, nicolaspielte<EMAIL_ADDRESS>wrote:

@hassanila97 https://github.com/hassanila97 will not you give me a few
more cookies? :D
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/davidmann4/ogar-feeder-bot/issues/450#issuecomment-213544861

--
Med vänliga hälsningar
Hassan

then I'll thy friend :D  OK hmm

hassan answer on skype

Can you give this script ? @hassanila97 not server, just UI

@hassanila97 can you sent me your Bots :D that would be really cool of you: D against 70 cookies? pls.

@hassanila97 can you me sent me your Stop script?

I didnt make it yet to work from node.js, what I've done is to restart the script and wait for conneciton using batch on Windows.. and I have a completely different version of bots.. not online and private. so..

@hassanila97 sent me :D

@hassanila97 You can't send the ui of bot counter etc?

I can when I have time.. cuz  it requires me to edit much of the feeder and userscript

@hassanila97 can sent me the Stop and Start script pls
 skype Name nicolaspielte (rawq)


Upgrade version of Node. Update dependencies

Resolve #43

@blcham
It is very strange that Docker Image CI fails. I tested it locally and it builds the image without any issues.

@palagdan Try to do testing PR from main  (e.g. modify README.md only) and see if docker build works.

@blcham
The warnings were due to the actions/checkout@v2 in workflow file which had be deprecated. I updated to actions/checkout@v3. Now it should work fine.

@palagdan I tried it locally i I get same error:
> ERROR [build 6/6] RUN npm run build                                                                                3.3s
------
 > [build 6/6] RUN npm run build:
0.294
0.294 ><EMAIL_ADDRESS>prebuild
0.294 > rimraf dist
0.294
0.329
0.329 ><EMAIL_ADDRESS>build
0.329 > cross-env NODE_ENV=production webpack --mode production  --env env=prod
0.329
0.767 (node:38) DeprecationWarning: Tapable.plugin is deprecated. Use new API on `.hooks` instead
0.767 (Use `node --trace-deprecation ...` to show where the warning was created)
3.056 Error: error:0308010C:digital envelope routines::unsupported
3.056     at new Hash (node:internal/crypto/hash:69:19)
3.056     at Object.createHash (node:crypto:133:10)
3.056     at module.exports (/app/node_modules/webpack/lib/util/createHash.js:135:53)
3.056     at NormalModule._initBuildHash (/app/node_modules/webpack/lib/NormalModule.js:417:16)
3.056     at handleParseError (/app/node_modules/webpack/lib/NormalModule.js:471:10)
3.056     at /app/node_modules/webpack/lib/NormalModule.js:503:5
3.056     at /app/node_modules/webpack/lib/NormalModule.js:358:12
3.056     at /app/node_modules/loader-runner/lib/LoaderRunner.js:373:3
3.056     at iterateNormalLoaders (/app/node_modules/loader-runner/lib/LoaderRunner.js:214:10)
3.056     at iterateNormalLoaders (/app/node_modules/loader-runner/lib/LoaderRunner.js:221:10)
3.056     at /app/node_modules/loader-runner/lib/LoaderRunner.js:236:3
3.056     at runSyncOrAsync (/app/node_modules/loader-runner/lib/LoaderRunner.js:130:11)
3.056     at iterateNormalLoaders (/app/node_modules/loader-runner/lib/LoaderRunner.js:232:2)
3.056     at Array.<anonymous> (/app/node_modules/loader-runner/lib/LoaderRunner.js:205:4)
3.056     at Storage.finished (/app/node_modules/enhanced-resolve/lib/CachedInputFileSystem.js:55:16)
3.056     at /app/node_modules/enhanced-resolve/lib/CachedInputFileSystem.js:91:9
3.105 Browserslist: caniuse-lite is outdated. Please run:
3.105 npx browserslist@latest --update-db
3.105
3.105 Why you should do it regularly:
3.105 https://github.com/browserslist/browserslist#browsers-data-updating
3.120 node:internal/crypto/hash:69
3.120   this[kHandle] = new _Hash(algorithm, xofLen);
3.120                   ^
3.120
3.120 Error: error:0308010C:digital envelope routines::unsupported
3.120     at new Hash (node:internal/crypto/hash:69:19)
3.120     at Object.createHash (node:crypto:133:10)
3.120     at module.exports (/app/node_modules/webpack/lib/util/createHash.js:135:53)
3.120     at NormalModule._initBuildHash (/app/node_modules/webpack/lib/NormalModule.js:417:16)
3.120     at handleParseError (/app/node_modules/webpack/lib/NormalModule.js:471:10)
3.120     at /app/node_modules/webpack/lib/NormalModule.js:503:5
3.120     at /app/node_modules/webpack/lib/NormalModule.js:358:12
3.120     at /app/node_modules/loader-runner/lib/LoaderRunner.js:373:3
3.120     at iterateNormalLoaders (/app/node_modules/loader-runner/lib/LoaderRunner.js:214:10)
3.120     at Array.<anonymous> (/app/node_modules/loader-runner/lib/LoaderRunner.js:205:4)
3.120     at Storage.finished (/app/node_modules/enhanced-resolve/lib/CachedInputFileSystem.js:55:16)
3.120     at /app/node_modules/enhanced-resolve/lib/CachedInputFileSystem.js:91:9
3.120     at /app/node_modules/graceful-fs/graceful-fs.js:123:16
3.120     at FSReqCallback.readFileAfterClose [as oncomplete] (node:internal/fs/read_file_context:68:3) {
3.120   opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
3.120   library: 'digital envelope routines',
3.120   reason: 'unsupported',
3.120   code: 'ERR_OSSL_EVP_UNSUPPORTED'
3.120 }
3.120
3.120 Node.js v18.20.4
------
Dockerfile:24
--------------------
  22 |
  23 |     # Build the application
  24 | >>> RUN npm run build
  25 |
  26 |     #### Stage 2: Serve the React application from Nginx
--------------------
ERROR: failed to solve: process "/bin/sh -c npm run build" did not complete successfully: exit code: 1

I executed it as follows:

git checkout package.json package-lock.json
docker build .

It is strange that it fails on the execution of npm run build. I tried it locally without docker and it works fine as well.
Also I tried to rebuild image without cache and it works.

@palagdan do i understand it correcly, that you cannot simulate failure of docker build ?

@palagdan do i understand it correcly, that you cannot simulate failure of docker build ?

Locally no.

@blcham
The problem was due to the updated Node.js version. Starting with Node.js v17, the default OpenSSL configuration changed, and some older cryptographic algorithms and modes are no longer supported by default. That’s why the solution I found is to set the environment variable:
NODE_OPTIONS=--openssl-legacy-provider sudo npm run dev

@palagdan what would it require to remove this option, can we even update it more so we would not need NODE_OPTIONS=--openssl-legacy-provider, please make fast investigation if possible.

@palagdan, moreover, did you test that npm run dev actually works... i.e., it shows some UI?

@blcham
I updated webpack to the latest version and made changes to the configuration because of the following message from the webpack team:
BREAKING CHANGE: webpack < 5 used to include polyfills for node.js core modules by default.
This is no longer the case. Verify if you need this module and configure a polyfill for it.

If you want to include a polyfill, you need to:
        - add a fallback 'resolve.fallback: { "util": require.resolve("util/") }'
        - install 'util'
If you don't want to include a polyfill, you can use an empty module like this:
        resolve.fallback: { "util": false }

I decided to disable this polyfill for now.

@palagdan, moreover, did you test that npm run dev actually works... i.e., it shows some UI?

Yes, after each update

@palagdan what would it require to remove this option, can we even update it more so we would not need NODE_OPTIONS=--openssl-legacy-provider, please make fast investigation if possible -- so we at least know which dependencies causes those issues.

Yes, you’re right. After I updated webpack to the latest version, it is no longer needed.

Should I review this and merge or still working on it?

Should I review this and merge or still working on it?

Yes, could you review it, please?


multi-thread

Thanks for participating in the TVM community! We use https://discuss.tvm.ai for any general usage questions and discussions. The issue tracker is used for actionable items such as feature proposals discussion, roadmaps, and bug tracking.  You are always welcomed to post on the forum first :)
Issues that are inactive for a period of time may get closed. We adopt this policy so that we won't lose track of actionable issues that may fall at the bottom of the pile. Feel free to reopen a new one if you feel there is an additional problem that needs attention when an old one gets closed.
For bug reports, to help the developer act on the issues, please include a description of your environment, preferably a minimum script to reproduce the problem.
For feature proposals, list clear, small actionable items so we can track the progress of the change.

I'm developing a program using TVM. A C++ multi-thread code is registered as a Python code through PackedFunc of TVM. I test the running time of C++ code. The result shows that multi-thread speeds the code. But when I test the running time in Python side, multi-thread doesn't work. Does TVM support multi-thread ?

Please open a new thread on https://discuss.tvm.apache.org/ where the community collectively discusses related questions


Suggested improvements for #85

Summary of changes:

In ErrorIndicator:

Use mfem::forall for eventual GPU support
Remove unused normalization member and some member functions which are no longer needed.

In ErrorEstimator:

Construct RHS for projection solve using a BilinearForm to enable eventual partial assembly and construction of the RHS on GPU (the linear solve already uses this). For now, the GradientIntegrator does not support matrix-valued coefficients so the flux for the GradFluxEstimator is computed as just grad(U) instead of \epsilon grad(U). This will be fixed in https://github.com/awslabs/palace/pull/97.
Use T.OrderW() in computing the desired quadrature rule (element Jacobian order) instead of T.Order() (element order).
Call GetVectorValue(T, ip, ...) instead of (e, ip, ...) to avoid resetting T with a redundant call to SetIntPoint.
Refactor CurlFluxEstimator for code reuse between the complex-valued and real-valued variants, with changes to avoid looping over mesh elements twice.
Refactor GradFluxEstimator to use a vector H1 space where appropriate to take advantage of repeated basis evaluations.
Move all timing for estimation into errorestimation.cpp to avoid splitting it between the driver/ classes and the estimator classes.

Other organization:

Remove no longer needed functions and classes in vector.hpp and coefficient.hpp.
Simplify writing of indicator fields by PostProcessor, done only once for a solve.
Headings for error-indicators.csv are "Norm", "Maximum", "Minimum", and "Mean"
Make construction and use of estimator and indicator objects consistent across all drivers.

Merged directly into https://github.com/awslabs/palace/pull/85


test: refactor cat tests to ava

@freitagbr I'm pretty confident that all tests have been properly converted. Feel free to review this for style.

@freitagbr PTAL

LGTM

LGTM

Merging, since it looks like lgtm.co is broken...


Add python 3.10 support

Pull Request Test Coverage Report for Build 645

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

Totals

Change from base Build 642:
0.0%

Covered Lines:
2030

Relevant Lines:
2120

💛  - Coveralls


Basic Obstacle avoidance

This will enable the vehicle to avoid obstacle in different states.
1. If there is an obstacle at the front and behind.
2. As well as if there is obstacle front and on one side of the vehicle it will turn to the direction where there are no obstacle.
3. If there is an obstacle front, left and right it will reverse.
Comments:
In case 3 is difficult to test in the current environment but we believe in theory it will work.
@AlaahT
@JonnaJ98
@anselin-thefirst
@LabibaKararEshaba
@Lexzet21

A reminder: This cannot be approved until you address the comment.

Hi. We are a little bit confused here, it passed the Github action with no problem what are we missing? Is Github actions not the CI? Or are we missing workflows?
Best Regards
Carl DT

The workflow is not building the new sketch you have committed but just the old one.

Well this fixes things indeed, but do you understand why the problem existed in the first place? It's not because "you must make changes to the smartcar.ino file that already exists".

I think so? Is it because the file name was different from from smartcar.ino? As long the name matches the job will run?

I think so? Is it because the file name was different from from smartcar.ino? As long the name matches the job will run?

Or you can simply change the workflow to reflect your project structure and filenames.

Okay then we understand! Thank you!


Sync .github/workflows directory with azure-sdk-tools for PR 6507

Sync .github/workflows directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/6507  See eng/common workflow

overriding check-enforcer as there's nothing that'll run in any language pipeline that'll test these changes

/check-enforcer override


How the palette is sorted?

Hello,
Thanks for your lib. I would like to know how the palette is sorted? Is it sorted by most dominant color?
Regards
Xavier

Yes indeed. The first one being the most dominant one.

Great! Thanks for the quick answer!

I've got one more question, is it possible to directly get the number of each pixels mapped to the reduced palette?
Or do we need to do another pass with .map(pixel)  on each pixels?
Regards,
Xavier