[WIP] feat: enable metadata sync for virtual tables

SUMMARY
This PR adds column metadata syncing support for SQL-based virtual tables (both legacy and React CRUD).
TODO:

Add a few tests (happy path, DML/malicious SQL)
Add screenshots
Add better description

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #10645 into master will decrease coverage by 4.14%.
The diff coverage is 71.15%.

@@            Coverage Diff             @@
##           master   #10645      +/-   ##
==========================================
- Coverage   64.26%   60.12%   -4.15%
==========================================
  Files         781      782       +1
  Lines       36882    36896      +14
  Branches     3499     3494       -5
==========================================
- Hits        23704    22185    -1519
- Misses      13069    14525    +1456
- Partials      109      186      +77

Flag
Coverage Δ

#cypress
?

#javascript
60.74% <100.00%> (+<0.01%)
:arrow_up:

#python
59.76% <70.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...125008dcb1_increase_width_of_table_columns_type.py
0.00% <0.00%> (ø)

superset/views/datasource.py
91.37% <66.66%> (-2.07%)
:arrow_down:

superset/connectors/sqla/models.py
90.16% <93.93%> (+0.43%)
:arrow_up:

...erset-frontend/src/datasource/DatasourceEditor.jsx
69.84% <100.00%> (-2.84%)
:arrow_down:

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupColors.js
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 145 more

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 878f06d...af3b4e8. Read the comment docs.

Thanks @WenQiangW for the review comments!

Codecov Report

Merging #10645 into master will decrease coverage by 10.42%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master   #10645       +/-   ##
===========================================
- Coverage   65.78%   55.35%   -10.43%
===========================================
  Files         838      406      -432
  Lines       39841    13559    -26282
  Branches     3655     3438      -217
===========================================
- Hits        26208     7506    -18702
+ Misses      13532     5875     -7657
- Partials      101      178       +77

Flag
Coverage Δ

#cypress
55.35% <0.00%> (-0.66%)
:arrow_down:

#javascript
?

#python
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...erset-frontend/src/datasource/DatasourceEditor.jsx
46.97% <0.00%> (-24.39%)
:arrow_down:

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...et-frontend/src/components/Menu/LanguagePicker.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...uperset-frontend/src/utils/getClientErrorObject.ts
0.00% <0.00%> (-89.19%)
:arrow_down:

.../src/dashboard/components/FilterIndicatorGroup.jsx
11.76% <0.00%> (-88.24%)
:arrow_down:

...c/explore/components/controls/withVerification.jsx
9.09% <0.00%> (-87.88%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

... and 654 more

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 cae54ac...c0b918b. Read the comment docs.

This has been rebased + comments addressed + added support for templating. If there are no further bugs here, I propose merging this as-is.

Ping @robdiciuccio . IMO the refactoring proposed here is best taken care of when we start working on the async query framework, during which I assume we'll end up refactoring many parts of the SQL Lab codebase relevant to this functionality.

Thanks @mistercrunch and @bkyryliuk for your help pushing this across the finish line! 🏁


How to find pixels given a certain point in the point cloud.

Hello!
I'm wondering if someone could point me in the right direction on this one.
I would like to find all the pixels that were used when creating a point X in the point cloud.
Simple example:
Point X in created point cloud was created with:
Img1: 333,345, Img2, 355,456, Img3, 123,325 etc etc
I' guessing this would be possible with this toolbox?

Hi there,
This information can indeed be found in the COLMAP SfM models. You can refer to the format specifications here. Each 3D point stores the image ID and the keypoint index of each of its observations. This is basically the point2D_idxs in the loader.


[BUG] - Not Building on FreeBSD

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
cmake is unable to find curl.h.
Expected Behavior
Builds successfully.
Steps To Reproduce
Build steps from README.md.
Anything else?
No response

I'm not as familiar with cmake as I am make so I'm searching through to find where I can point cmake to find the curl.h file.

I don’t have much FreeBSD experience unfortunately so I’m unsure on the package manager it uses, but if you wish to use the HTTP library you’ll need to install cURL via the package manager. If you want to compile without cURL (and therefore the HTTP library) you can disable HTTP with the DISABLE_HTTP flag: https://github.com/dictu-lang/Dictu#compiling-without-http

I just need to figure out how to point cmake to where the curl header is.

Oh I see what you're saying. Yeah there isn't currently a way through CMAKE to use a custom cURL header file, it's expecting it to already be there to be linked dynamically: https://github.com/dictu-lang/Dictu/blob/develop/src/optionals/http.h#L5

Well, I finally got a chance to look back at this and have things working. The following lines need to be ran. I'll update the README with a section for FreeBSD if that's ok.
export CPATH=/usr/local/include
export LIBRARY_PATH=/usr/local/lib
export LD_LIBRARY_PATH=/usr/local/lib


Well Known Regex Validation

Re: https://github.com/envoyproxy/envoy/pull/2420#discussion_r162995905
Add well known validation rules for regular expressions. On the Go side, supporting RE2 is a nominal effort, and likewise if the C++ code uses RE2 as well. Otherwise, we'll need to look into support for PCRE/POSIX

Wouldn't it be possible to leave the decision to consumers? Something along the lines:
message Regex {
    enum Option {
        REGEX_OPTION_INVALID = 0;
        REGEX_OPTION_IGNORE_CASE = 1;
    }
    repeated Option options = 1;
    string pattern = 2;
    string pattern_js = 3;
    string pattern_jvm = 4;
    string pattern_pcre = 5;
    string pattern_re2 = 6;
    string pattern_rust = 7;
    // ...
}
Regex regex = 42;

More specific regular expressions would override the generic pattern and the interpretation of the options would be up to the code generator. I'm sure there are other compositions possible that make it nicer to use in Proto but the general idea of mine should be clear by this example.

This could be useful for refining the existing regex checkers for arbitrary string/byte fields, but this particular validation is to ensure the provided string value is a valid regex pattern. Something similar could be done for that rule, but keeping it generic is important since -- for example --even if it's a rust style regex pattern, there's no surefire way to verify in (say) python that the pattern is valid without pulling in a dependency or reimplementing the rust regex parser.

Only the actual code generators can verify if a given pattern is valid. It would be up to the users to write patterns that are valid for their generator and language. I don't see a way how you could ensure that in this library for all possible languages. The only thing that can be provided is dedicated fields where the patterns can be stored for the actual generators.
The logic in such a generator would then be as follows, e.g. Rust in pseudocode:
val pattern = if (regex.pattern_rust.isNotEmpty()) {
    regex.pattern_rust
} else {
    regex.pattern
}

if (pattern.isInvalidRegexPattern()) {
    doSomethingAboutIt()
}


add process metrics(cpu/mem) for disagg-compute mode

What problem does this PR solve?
Issue Number: close #7261
Problem Summary: As issued said.
What is changed and how it works?

Rust: add a simple library(libs/libprocess_metrics) to collect proc stat
C++:
2.1. Add a class ProcessCollector to get proc stat and return current cpu/mem/vsize/rss metric.
2.2. Register ProcessCollector to exposer in non-TLS pull mode.
2.3. Register ProcessCollector to MetricsHTTPHandler in TLS pull mode.
2.4. Register ProcessCollector to gateway in push mode.

Why not use async metrics but add a new PrcessColelctor:

The original tiflash-proxy metrics logical only collect cpu info when Collect() is called, we want to make it same with tiflash-proxy logic, so the metric basically same.

Check List
Tests

[ ] Unit test
[ ] Integration test
[x] Manual test (add detailed scripts or steps below):

Setup a cluster in disaggregated mode(without AutoScaler, so we can compare CPU/Mem metrics with proxy), we can see CPU/Mem metrics is ok for CN. There are there cases:

non-TLS pull model:
1.1 deploy cluster: tiup cluster deploy -y disagg-s3 nightly /home/guojiangtao/work/tiup_cluster/s3/disagg-s3.yaml -u guojiangtao --skip-create-user -i ~/.ssh/id_ed25519 --native-ssh=1
1.2 start cluster: tiup cluster start disagg-s3
1.3 load data: tiup bench tpch prepare --sf 1 --tiflash
1.4 run tpch: tiup bench tpch run -D test -P 4001
1.6 check grafana
TLS pull model:
2.1 deploy and start cluster just like 1.1 and 1.2
2.2 enable tls: tiup cluster tls disagg-s3 enable
2.3 check log, make sure prometheus tls is enabled: 31131:[2023/04/10 17:26:27.761 +08:00] [INFO] [MetricsPrometheus.cpp:279] ["Enable prometheus **secure** pull mode; Listen Host = <IP_ADDRESS>, Metrics Port = 8237"] [source=Prometheus] [thread_id=1]
2.4 load/run tpch just like 1.3 and 1.4
prometheus push model:(TiUP doesn't support pushgateway, so cannot config prometheus to fetch metric from pushgateway, so we only check metrics by curl pushgateway)
3.1 deploy and start cluster just like 1.1 and 1.2, but need to change config and setup status.metrics_addr
3.2 deploy pushgateway: docker pull prom/pushgateway && docker run -d -p 9091:9091 prom/pushgateway
3.3 check log to make sure push model is enabled: 140:[2023/04/10 18:05:15.279 +08:00] [INFO] [MetricsPrometheus.cpp:254] ["Enable prometheus push mode; interval = 15; addr = <IP_ADDRESS>:9091"] [source=Prometheus] [thread_id=1] 141:[2023/04/10 18:05:15.279 +08:00] [INFO] [MetricsPrometheus.cpp:295] ["Disable prometheus pull mode"] [source=Prometheus] [thread_id=1]
3.4 check cpu metrics from pushgateway: curl http://<IP_ADDRESS>:9091/metrics | grep tiflash_process_: tiflash_process_cpu_seconds_total{instance="guojiangtao-arch",job="tiflash"} 1016

[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/reset

/retest

/build

/rebuild

/run-all-tests

/merge

/merge


Relationship to SharePoint Online Sharing Settings

How do the settings above interact with the sharing settings in SharePoint Online? what happens if they conflict?

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

ID: ce8ef300-3f05-0966-3991-68bcf79353cb
Version Independent ID: aada60bd-e601-e19e-3532-82a1500a172e
Content: Delegate invitations for Azure Active Directory B2B collaboration
Content Source: articles/active-directory/b2b/delegate-invitations.md
Service: active-directory
GitHub Login: @msmimart
Microsoft Alias: mimart

@DeanGross Thanks for your feedback! We will investigate and update as appropriate.

@DeanGross We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


<quote\ expansion in fingerprints

When building a serde_macros-dependent code, a string from quasi (https://github.com/erickt/rust-quasi/blob/d9ec71215002076dd0102698b22f78fb8f748e71/quasi/src/lib.rs#L126) ends up in the cargo fingerprint files. For example:
git clone https://github.com/Byron/yup-oauth2
cd yup-oauth2 && cargo build
cat target/debug/.fingerprint/yup-oauth2-*/dep-lib-yup-oauth2

would produce
/tmp/yup-oauth2/tmp/yup-oauth2/target/debug/libyup_oauth2.rlib: src/lib.rs src/device.rs src/refresh.rs src/common.rs src/helper.rs <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion

/tmp/yup-oauth2/target/debug/yup_oauth2.d: src/lib.rs src/device.rs src/refresh.rs src/common.rs src/helper.rs <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion <quote\ expansion

I wonder if that makes cargo rebuild the project every time (https://github.com/rust-lang/cargo/issues/1215#issuecomment-107750532).
Would appreciate if quasi wouldn't leak these.

Workaround: https://github.com/ArtemGr/cargo/commit/f56369511c2737c5c18a63e382f5bbbae88f8de4


[Bug]:

Bug Description
Hi,
I'm having trouble importing the LangchainEmbedding, ServiceContext classes from the llama_index package. I've installed the package using pip, but I'm getting an ImportError when trying to import the classes, it was working fine till yesterday. Is there any known issue or workaround for this problem? I've followed the documentation, but I can't seem to resolve the ImportError.
Used :
pip install llama-index
from llama_index import LangchainEmbedding, ServiceContext
Error:
ImportError                               Traceback (most recent call last)
 in <cell line: 3>()
1 import torch
2 from langchain.embeddings.huggingface import HuggingFaceEmbeddings
----> 3 from llama_index import LangchainEmbedding, ServiceContext
4 from llama_index.llms import HuggingFaceLLM
5
ImportError: cannot import name 'LangchainEmbedding' from 'llama_index' (/usr/local/lib/python3.10/dist-packages/llama_index/init.py)

NOTE: If your import is failing due to a missing package, you can
manually install dependencies using either !pip or !apt.
Version
0.9.2
Steps to Reproduce
pip install llama-index
from llama_index import LangchainEmbedding, ServiceContext
Relevant Logs/Tracbacks
ImportError                               Traceback (most recent call last)
<ipython-input-12-cf78bfce5182> in <cell line: 3>()
      1 import torch
      2 from langchain.embeddings.huggingface import HuggingFaceEmbeddings
----> 3 from llama_index import LangchainEmbedding, ServiceContext
      4 from llama_index.llms import HuggingFaceLLM
      5

ImportError: cannot import name 'LangchainEmbedding' from 'llama_index' (/usr/local/lib/python3.10/dist-packages/llama_index/__init__.py)

---------------------------------------------------------------------------
NOTE: If your import is failing due to a missing package, you can
manually install dependencies using either !pip or !apt.

@ShyamFaguna we have moved some minor imports to be a) clearer about where imports are coming from and b) reduce the bloat in the root __init__.py file
The bot has listed the correct import, but it could also be even shorter
from llama_index.embeddings import LangchainEmbedding


Publishing v1.1.2 on conda-forge

Comment:
It looks like there's a mismatch between what's on conda-forge (v1.1.1 https://anaconda.org/conda-forge/sphinx-book-theme) and what's on pypi (v1.1.2 https://pypi.org/project/sphinx-book-theme/#description).  Maybe the bot got confused from two versions being released on the same day?
Would it be possible to get v1.1.2 published on conda-forge as well?

1.1.2 from #36 should be up within the half hour. Please re-open if there are further issues.


SwipeFlingAdapterView not swipable anymore after 1 card swipped

Hi guys,
First, thank you for the library, it's the best one I've come across searching for a Tinder-like swipe card effect, and I have to say I'm impressed by the good work here :)
I'm setting up my own SwipeFlingAdapterView with a custom ArrayAdapter of my own, nothing special.
But the problem I have is that the SwipeFlingAdapterView stop being interactive after I successfuly swipe one card.
After I swipped the first card, I can see the 2nd card underneath, but it's like an image, it's frozen, the 2nd card won't move.
The only way I can get my SwipeFlingAdapterView to become interactive again is to go somewhere else in my app (different fragment) and come back in order for the 2nd card to be interactive as expected.
I tried several workarounds, here is my latest (unsuccessful) attempt of modifying removeFirstObjectAdapter :
        public void removeFirstObjectInAdapter() {
            if (swipeAdapter.getItem(0) != null){
                nManager.getProfilePool().remove(0);
            }
            synchronized (swipeAdapter){
                swipeAdapter.notifyAll();
                swipeFlingAdapterView.setAdapter(swipeFlingAdapterView.getAdapter());
            }
        }

Any idea ? :)
Thanks for your help :)
PS : I don't know if it's related to https://github.com/Diolor/Swipecards/issues/39 or not but right after initialization of my SwipeFlingAdapterView, I sometime see the library spamming the adapter with "getView" method call.

Try simplifying it. This is all I need for it to work fine.
public void removeFirstObjectInAdapter() {
                yourList.remove(0);
                swipeAdapter.notifyDataSetChanged();
            }

Thanks for your answer !
I actually came back to a solution like you provided and hided all the complexity in a customized  ArrayAdapter, and it started working :)


[Search] Fix test resource deployment

Packages impacted by this PR
@azure/search-documents
Issues associated with this PR
Describe the problem that is addressed by this PR
Live test resource deployment hasn't been working correctly. This aims to fix that, enable semantic search for semantic search tests, and allow for easier deployment of test resources.
Designed to be reviewed on a per-commit basis.
Fixes #24531
Fixes #26490
Fixes #27400
Fixes #27669
Checklists

[x] Added impacted package name to the issue description

/azp run js - search-documents - tests

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

/azp run js - search-documents - tests

/azp run js - search-documents - tests


added a-frame to the list of modules

I used aframe instead of a-frame to follow the convention set by the npm module rather than the branding / documentation.

Oh no! failed test? Is this me? Have I forgotten to add something?

added to 29, the var was AFRAME


BINDINGS: raylib-cpp now targets raylib 4.0.0

https://github.com/RobLoach/raylib-cpp/releases/tag/v4.0.0

@RobLoach thanks for the update! 👍😄


chore: vertx-cloud-events to 1.0.7

chore: Promote vertx-cloud-events to version 1.0.7

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Load path issue on Piz Daint with Pace

In the Pace test suite on daint we are consistently getting the error below, even after setting GT_CACHE_LOAD_RETRIES=100, GT_CACHE_LOAD_RETRY_DELAY=1000. This makes me think the problem is an issue with appending, loading, and quickly removing from sys.path. I'm looking into alternatives, including using importlib lower-level code here as well.


machine_configuration: Add copyright license header

Signed-off-by: Hersh Pathak<EMAIL_ADDRESS>

looks good to me!


IExecutionListener.onStart() running multiple times

Closes #1735
Prevent IExecutionListener.onStart() from
running more than once when added via the
@Listeners annotation.
Fixes #1735  .
Did you remember to?

[X] Add test case(s)
[X] Update CHANGES.txt

We encourage pull requests that:

Add new features to TestNG (or)
Fix bugs in TestNG

If your pull request involves fixing SonarQube issues then we would suggest that you please discuss this with the
TestNG-dev before you spend time working on it.

@cbeust - I have rebased off of master and resolved the merge conflicts. Please help merge if you are fine with the changes.

@cbeust - I have rebased again off of master. Please help merge if you are fine with the changes. @juherr has approved it.


[LinuxX64] ld.lld: error: relocation R_X86_64_PC32 cannot be used against symbol __environ; recompile with -fPIC

Kotlin version: 1.4.10
korio version: 2.0.0-alpha

    linuxX64 {
        binaries {
            compilations.getByName("main") {
                val matchInterop by cinterops.creating {
                    defFile("src/nativeInterop/cinterop/libmatch.def")
                    compilerOpts("-fPIC")
                }
            }
            sharedLib("match", listOf(RELEASE))
        }
    }

Execute :
./gradlew :libmatch:linkMatchReleaseSharedLinuxX64

Got error:
ld.lld: error: relocation R_X86_64_PC32 cannot be used against symbol __environ; recompile with -fPIC
>>> defined in /home/bennyhuo/.konan/dependencies/target-gcc-toolchain-3-linux-x86-64/x86_64-unknown-linux-gnu/sysroot/lib64/libc.so.6
>>> referenced by out
>>>               /tmp/konan_temp7652478800042105268/result.o:(kfun:com.soywiz.korio.lang.Environment.$<init>$lambda-0$FUNCTION_REFERENCE$190.invoke#internal)

btw: :libmatch:linkMatchDebugSharedLinuxX64 will cause similar problem.

Please, can you try with the final 2.0.0 compiled against Kotlin 1.4.20?

I have tried 2.0.0-rc3 with Kotlin 1.4.20 ，same error.

It's OK on macOS, seems a Linux related issue.

Same problems on Kotlin 1.4.20 and Korio 2.0.0.

2.0.0 was published, and was compiled and tested on ubuntu linux here: https://github.com/korlibs/korio/runs/1432436250?check_suite_focus=true
Which operating system are you trying on? Maybe that's a Kotlin/Native issue and -fPIC should be enabled?
The __environ global variable is used here:
https://github.com/korlibs/korio/blob/b4b02447c793cdbac5b4d5adb9c0d7795fd2a5d2/korio/src/nativePosixNonAppleMain/kotlin/com/soywiz/korio/lang/EnvironmentNativePosix.kt#L9
And it is the only way I'm aware of we have on posix to list the environment variables

Reported here: https://youtrack.jetbrains.com/issue/KT-43502
But, please, can you indicate which environment do you have? Which operating system are you using, etc.

Ubuntu 18.04.
I am not sure how to enable -fPIC, I tried to add it to compilerOpts as below, but no effects.
    linuxX64 {
        binaries {
            compilations.getByName("main") {
                val matchInterop by cinterops.creating {
                    defFile("src/nativeInterop/cinterop/libmatch.def")
                    compilerOpts("-fPIC")
                }
            }
            sharedLib("match")
        }
    }

Maybe it is only a problem to build a shared library? I will try to push a demo to recurrent this.
Thanks.

It seems a bug of Kotlin compiler. Close.


change name of branch for README.md

I changed the name of the branch to avoid any confusion.

I think our README.md is not good enough


perf(table): prevent unnecessary reflow with sticky row

In StickyStyle getBoundingClientRect() is called to calculate the offset of the next sticky row. This causes a forced reflow(link). However this is not needed when there is only one sticky row.

@DevVersion friendly ping

Sorry @nename0 for the delay.
This PR looks good to me, but I wanted to get a second review from @andrewseguin.

Thanks!


3D city models & landmarks data

Where can I find HERE's advanced city models data? I am specifically interested in the textured buildings asset.
Is it available for public consumption?

Hi @jamesdeantv !
That data is available to be used on HERE's Open Location Platform. I recommend you take a look at those materials, on the getting started you should find a link to request a free trial.
On harp.gl side we are working on providing the capabilities to render all of these datasets, but not on the data offer. We plan to add some open sourced examples showcasing these features, but we are still working out the exact details. Stay tuned for that for the first months of 2020!


Reduce the Renderer interface

Currently the Renderer interface has 4 methods (Size, RenderPath, RenderText and RenderImage).
Many renderers don't actually implement the RenderText, but fallback to rendering the text as path and calling RenderPath.
Instead of duplicating this "trick" everywhere, one could reduce the Renderer interface and introduce a TextRenderer interface (naming inspired from https://go.googlesource.com/proposal/+/master/design/draft-iofs.md#possible-future-or-third_party-extensions)
// Renderer is an interface that renderers implement. It defines the size of the target (in mm) and functions to render paths, text objects and raster images.
// If the Renderer does not implement TextRenderer, the text will be converted to path.
type Renderer interface {
	Size() (float64, float64)
	RenderPath(path *Path, style Style, m Matrix)
	RenderImage(img image.Image, m Matrix)
}

// TextRenderer should be implemented to prevent text from being converted to path before rendering
type TextRenderer interface {
	Renderer
	RenderText(text *Text, m Matrix)
}

And then context.DrawText would use this method if available, or fallback to the RenderPath method:
// DrawText draws text at position (x,y) using the current draw state. In particular, it only uses the current affine transformation matrix.
func (c *Context) DrawText(x, y float64, texts ...*Text) {
	m := c.view.Translate(x, y)
	for _, text := range texts {
		if text.Empty() {
			continue
		}
		renderText(c.Renderer, text, m)
	}
}

func renderText(r Renderer, text *Text, m Matrix) {
	t, ok := r.(TextRenderer)
	if ok {
		t.RenderText(text, m)
	} else {
		// transform text to path
		paths, colors := text.ToPaths()
		for i, path := range paths {
			style := DefaultStyle
			style.FillColor = colors[i]
			r.RenderPath(path, style, m)
		}
	}
}

Do you think this is a good idea? Would you be willing to accept a PR with those changes?

That is a pretty nice trick! It is true that the majority of the renderers defer to RenderPath for rendering text, but I think this is because most implementations are still minimal working examples. Overview:

Rasterizer: defers, which is correct
SVG: has its own implementation
PDF: has its own implementations
EPS: defers, but should be implemented in the future
HTMLCanvas: defers, but should be implemented in the future
TeX: defers, but should be implemented in the future?
OpenGL (WIP): would defer probably

I think in general any renderer should implement a specialized RenderText in the end, so that adding this design complexity doesn't pay off that well. I think we could mitigate this partly by implementing Text.RenderAsPath (or similar) which would be the contents of the RenderText function that defers to RenderPath. Copying one line (or the current 5 lines) for a new renderer isn't such a big deal I think, especially if it keeps design relatively simple. It also makes renderer implementations conscious that the RenderText implementation is missing, which is really a required interface for a fully functioning renderer for canvas. I'd prefer not to encourage relying on paths instead of native text rendering.


Standardize DBClient package level variable declaration

In core-data, var dbc is declared and initialized in init.go
In core-metadata, var dbClient is declared in const.go and initialized in init.go
I personally think the name dbClient is preferable, being more explicit, and I like core-data’s methodology better since the variable isn't really a constant. I'm open to discussing another approach, but we need to standardize what the var is called, where it's declared and then update references accordingly.
From #298

I agree both on variable name and where it should be declared.

Also should be fine to standardize the database enumeration. Data uses core/data/interfaces/db.go, and metadata uses core/domain/enums/database.go. I think the best place should be in core/db/db.go. What do you think?
If you want, I can work in this issue

@jduranf Yes, I had that in my todo list. Also adding db selection to metadata and standardize how it is done with data.
Feel free to work on it.

Thanks


only the first 10 lines of the sheet are read

version 0.8.0 on nodejs 0.12.7

Sorry, subtle bug in the test code.


Ability to configure OIDC claims for all service providers

I’ve been trying to return claims embedded in the “id_token” (JWT) but only works if I configure them in the Service Provider -> Claim Configuration section.
I would like to be able to have a centralized configuration that applies to all service providers.

As per current design of IS this is not possible via UI portal,  alternatively you can write a script to call admin API of IS.


Perform type punning via union without undefined behavior

The previous code from c3d7f491e2daebda2413fb3d2935c51df1c50ac7
still contains undefined behavior, since it just creates
temporary pointer variables.

Looks great. Thanks, please allow me a couple of days to check on several computers.


fix(KFLUXBUGS-1666): Only analyze build context for dependencies

When generating the SBOM, we do not constrain the source analysis to the build context. This can result in many additional dependencies being included than should be.

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

/lgtm

We run
unshare -Uf $UNSHARE_ARGS --keep-caps -r --map-users 1,1,65536 --map-groups 1,1,65536 -w ${SOURCE_CODE_DIR}/$CONTEXT -- buildah build \
          $VOLUME_MOUNTS \
          "${BUILDAH_ARGS[@]}" \
          "${LABELS[@]}" \
          --tls-verify=$TLSVERIFY --no-cache \
          --ulimit nofile=4096:4096 \
          -f "$dockerfile_path" -t $IMAGE .

This sets the working directory to ${SOURCE_CODE_DIR}/$CONTEXT so no other context should be available.
https://man7.org/linux/man-pages/man1/unshare.1.html

The remaining Checkton issues are flagged for untouched code. Marking this PR to merge when ready.


There is no pool defined on the current client

When  running any of the commands, I'm getting a "There is no pool defined on the current client" error.

Found it... I just didn't set the correct default adapter in my local.js. It's one of those days.


Missing .ASI file

I could not find the ASI file needed to use this, please help.

You can download the latest release here
There should be an ASI inside the .ZIP file

Thank you very much.


Expose refresh indicators for kernels

Codecov Report

Merging #12021 (0d36e66) into main (890dd7a) will decrease coverage by 0%.
The diff coverage is 71%.

:exclamation: Current head 0d36e66 differs from pull request most recent head 128862e. Consider uploading reports for the commit 128862e to get more accurate results

@@           Coverage Diff           @@
##            main   #12021    +/-   ##
=======================================
- Coverage     62%      62%    -1%
=======================================
  Files        499      499
  Lines      35479    35539    +60
  Branches    5731     5747    +16
=======================================
- Hits       22285    22059   -226
- Misses     11100    11386   +286
  Partials    2094     2094

Impacted Files
Coverage Δ

src/kernels/types.ts
90% <ø> (ø)

...llers/kernelSource/notebookKernelSourceSelector.ts
7% <0%> (-1%)
:arrow_down:

src/standalone/api/kernelApi.ts
73% <63%> (-1%)
:arrow_down:

...rnels/raw/finder/localKernelSpecFinderBase.node.ts
80% <64%> (-2%)
:arrow_down:

...aw/finder/contributedLocalKernelSpecFinder.node.ts
79% <65%> (-5%)
:arrow_down:

...raw/finder/contributedLocalPythonEnvFinder.node.ts
84% <70%> (-2%)
:arrow_down:

...lPythonAndRelatedNonPythonKernelSpecFinder.node.ts
63% <83%> (-1%)
:arrow_down:

src/kernels/kernelFinder.ts
91% <100%> (+2%)
:arrow_up:

src/platform/api/pythonApi.ts
74% <100%> (+1%)
:arrow_up:

src/interactive-window/commands/statusProvider.ts
18% <0%> (-56%)
:arrow_down:

... and 54 more


add more "air" to footer logo on mobile (ui)

resolve https://github.com/codeigniter4projects/website/issues/239
after fix

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
220

Relevant Lines:
286

💛  - Coveralls


Really interesting

Just want to say this seems really interesting -- I've been looking at Lisp, so I'll be studying this since I know InfluxDB pretty well.

otoolep,
Thanks,I might want to pick your brain on the influxdb side, I'm developing
a network protocol analyzer in lisp that is backed by a Influxdb datastore.
There are some things arcitecture-wise on the influxdb side that I'm not
sure about. If you have any suggestion or improvements feel free to open an
issue or send a pull request or email me directly.
On Wed, Sep 30, 2015 at 11:46 PM, otoolep<EMAIL_ADDRESS>wrote:

Closed #4 https://github.com/mmaul/cl-influxdb/issues/4.
—
Reply to this email directly or view it on GitHub
https://github.com/mmaul/cl-influxdb/issues/4#event-423824081.

Sure, happy to help.
Bear in mind I know close to nothing about Lisp -- I would just like to follow Eric Raymond's advice that knowing it would make me better code.
Feel free to drop me a line anytime at: philip at influxdb ....


Add namespace to change version tracking

This PR adds a new option to the API Publisher allowing for additional uniqueness when tracking the last change version processed. When the options is provided, the API Publisher prefixes the provided value and a colon to the target connection name, as shown in the example below. When the options is not provided, the original behavior is retained.
CLI argument ... --targetName=state-2425 --lastChangeVersionProcessedNamespace=something
will associate last change version information with something:state-2425
which stores {"something:state-2425":123456} in the named connection provider's lastChangeVersionsProcessed parameter.
This is useful in situations where it is desired to use the same source and target connections for multiple configurations of the API Publisher. For example, if it is desired to publish attendance at a different frequency than the rest of the resources, or if it is desired to publish frequently from a live source while publishing less frequently from a change-isolated source.

Thank you @ea-mtenhoor! I leave further discussion to @jasonh-edfi since I'm not involved in the daily operation of this application. Just wanted to acknowledge the contribution.


feat(expr): add DivisionByZero error

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
😄💦
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Closes #3939 ? :eyes:

Closes #3939 ? 👀

IIRC we will return broken fifo? 😄

Closes #3939 ? eyes

Partly solved. But IIRC we will return broken fifo? smile
dev=> select 1/0;
ERROR:  QueryError: Invalid Parameter Value: Out of range.
dev=> select 1/v from t;
ERROR:  QueryError: RPC error: Status { code: Internal, message: "internal error: broken fifo_channel", metadata: MetadataMap { headers: {"risingwave-error-bin": "CAESI2ludGVybmFsIGVycm9yOiBicm9rZW4gZmlmb19jaGFubmVs"} }, source: None }.
dev=>

Tracked separately here: https://github.com/singularity-data/risingwave/issues/3908


[500] vite_ssr_import_0.incrementer is not a function

Environment
Nuxt3

Operating System: Darwin
Node Version:     v20.14.0
Nuxt Version:     3.13.2
CLI Version:      3.14.0
Nitro Version:    2.9.7
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
Fresh install
Describe the bug
[1]  ERROR  [nuxt] [request error] [unhandled] [500] vite_ssr_import_0.incrementer is not a function
[1]   at ./node_modules/vue-i18n/dist/vue-i18n.mjs:33:37
[1]   at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
[1]   at async ViteNodeRunner.runModule (./node_modules/vite-node/dist/client.mjs:399:5)
[1]   at async ViteNodeRunner.directRequest (./node_modules/vite-node/dist/client.mjs:381:5)
[1]   at async ViteNodeRunner.cachedRequest (./node_modules/vite-node/dist/client.mjs:206:14)
[1]   at async ViteNodeRunner.dependencyRequest (./node_modules/vite-node/dist/client.mjs:259:12)
[1]   at async ./node_modules/@nuxtjs/i18n/dist/runtime/composables/index.js:21:32
[1]   at async ViteNodeRunner.runModule (./node_modules/vite-node/dist/client.mjs:399:5)
[1]   at async ViteNodeRunner.directRequest (./node_modules/vite-node/dist/client.mjs:381:5)
[1]   at async ViteNodeRunner.cachedRequest (./node_modules/vite-node/dist/client.mjs:206:14)
Additional context
No response
Logs
No response

I had the same error with vue-i18n.
Upgrading it to the latest version (^10.0.4) worked for me.

I also encountered the same issue, and I found that @nuxtjs/i18n currently uses vue-i18n@9, but the latest version of vue-i18n@latest on npm is ^10.0.4. After switching from @nuxtjs/i18n to @nuxtjs/i18n@next, @nuxtjs/i18n@next and vue-i18n v10 will work as expected.
Environment

"nuxt": "^3.13.2"
"vue-i18n": "10.0.4"
"@nuxtjs/i18n": "9.0.0-rc.2"


Error: No devices found

I am running the example shared in readme
import asyncio
from pydeako import discover

async def _discover():
    d = discover.DeakoDiscoverer()
    try:
        address = await d.get_address()
        print(f"Found deako device at {address}!")
    except discover.DevicesNotFoundException:
        print(f"Error: {e}")
        pass

if __name__ == "__main__":
    asyncio.run(_discover())

but it returns Error: No devices found via _deako._tcp.local.
FWIW - I can see my deako smart switch on wifi router device list and was able to successfully ping from the same device that threw the error when using pydeako.
Can someone help?

I am running the example shared in readme
import asyncio
from pydeako import discover

async def _discover():
    d = discover.DeakoDiscoverer()
    try:
        address = await d.get_address()
        print(f"Found deako device at {address}!")
    except discover.DevicesNotFoundException:
        print(f"Error: {e}")
        pass

if __name__ == "__main__":
    asyncio.run(_discover())

but it returns Error: No devices found via _deako._tcp.local.
FWIW - I can see my deako smart switch on wifi router device list and was able to successfully ping from the same device that threw the error when using pydeako.
Can someone help?

Hey there! I'm the author and back from vacation. There are a few things you can do:

Make sure you're on the same network as your deako devices
If this is running in a container, make sure your network mode is host in order to pass through mdns
Scan your network using a cli tool like dns-sd (I'm on macOS and this is available):

dns-sd -B _services._dns-sd._udp | grep deako
13:52:37.621  Add        3  26 .                    _tcp.local.          _deako
13:52:37.621  Add        3  14 .                    _tcp.local.          _deako

The above command shows that there are deako devices advertising on my network. If you want specific deako devices, you can do:
dns-sd -B _deako._tcp

Browsing for _deako._tcp
DATE: ---Mon 04 Dec 2023---
13:54:38.296  ...STARTING...
Timestamp     A/R    Flags  if Domain               Service Type         Instance Name
13:54:38.297  Add        3  26 local.               _deako._tcp.         231M001566002124
13:54:38.297  Add        3  14 local.               _deako._tcp.         231M001566002124
13:54:38.297  Add        3  26 local.               _deako._tcp.         231M001566001472
13:54:38.297  Add        3  14 local.               _deako._tcp.         231M001566001472

With that info above, if I run the sample in the readme via a file test.py, I get:
python test.py
Found deako device at <IP_ADDRESS>:23!

Let me know if any of that works for you!

Thanks @Balake ! I think the issue is with my docker container running on a Mac OS host. dns-sd commands on host machine finds/prints the deako device however docker container cannot, even with --privileged --net=host flags.
When I run the test.py on a RaspberryPi, it found the device.

Hi again, Happy New Year!
I've resumed testing after holidays and ran into an issue, particularly with the second example from the documentation.
import asyncio
from pydeako import deako, discover

async def _discover():
    client_name = "MyClient"
    d = discover.DeakoDiscoverer()
    deako_client = deako.Deako(d.get_address, client_name=client_name)

    await deako_client.connect()
    await deako_client.find_devices()

    devices = deako_client.get_devices()

    # turn on all devices
    for uuid in devices:
        await deako_client.control_device(uuid, True)

if __name__ == "__main__":
    asyncio.run(_discover())

and here is the error message:
2024-01-06 02:58:46.076 ERROR (MainThread) [pydeako.deako] Timeout attempting to connect. Trying again
2024-01-06 02:58:46.077 ERROR (MainThread) [pydeako.deako] Already attempting to connect
2024-01-06 02:58:46.077 ERROR (MainThread) [pydeako.deako] No connection to send data to
2024-01-06 02:58:56.087 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Deako for deako
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 402, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/deako/__init__.py", line 33, in async_setup_entry
    await connection.find_devices()
  File "/usr/local/lib/python3.11/site-packages/pydeako/deako/_deako.py", line 130, in find_devices
    raise DeviceListTimeout()
pydeako.deako._deako.DeviceListTimeout

I've a smart-dimmer and a couple of smart-switches. Upon re-trying, I noticed I was able to turn on the smart-dimmer but no luck with smart switches. Let me know if I should share more info. Thanks!
Host: Raspberry Pi 4

Hi again, Happy New Year! I've resumed testing after holidays and ran into an issue, particularly with the second example from the documentation.
import asyncio
from pydeako import deako, discover

async def _discover():
    client_name = "MyClient"
    d = discover.DeakoDiscoverer()
    deako_client = deako.Deako(d.get_address, client_name=client_name)

    await deako_client.connect()
    await deako_client.find_devices()

    devices = deako_client.get_devices()

    # turn on all devices
    for uuid in devices:
        await deako_client.control_device(uuid, True)

if __name__ == "__main__":
    asyncio.run(_discover())

and here is the error message:
Timeout attempting to connect. Trying again
Already attempting to connect
No connection to send data to
Traceback (most recent call last):
  File "/home/amoeba/nas/users/shared/homelab-test/deako/socket-client.py", line 19, in <module>
    asyncio.run(_discover())
  File "/usr/lib/python3.11/asyncio/runners.py", line 190, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/asyncio/base_events.py", line 653, in run_until_complete
    return future.result()
           ^^^^^^^^^^^^^^^
  File "/home/amoeba/nas/users/shared/homelab-test/deako/socket-client.py", line 10, in _discover
    await deako_client.find_devices()
  File "/home/amoeba/.local/lib/python3.11/site-packages/pydeako/deako/_deako.py", line 130, in find_devices
    raise DeviceListTimeout()
pydeako.deako._deako.DeviceListTimeout

I've a smart-dimmer and a couple of smart-switches. Upon re-trying, I noticed I was able to turn on the smart-dimmer but no luck with smart switches. Let me know if I should share more info. Thanks!
Host: Raspberry Pi 4

Hey there! Sorry for the late reply, but I've made some changes to this library around connection logic. It doesn't affect your particular error, but some other things are improved.
Your particular error indicates an issue with the device itself not returning any devices or not responding to the command for the device list, something that I don't think I can fix without logs from the device.

thanks! the latest version seems to have fixed my issue.

thanks! the latest version seems to have fixed my issue.

Great, thanks for reporting back!


Subtitles missing in breadcrumbs

Judging by the /, it seems like the page title wants to be in the breadcrumbs, but are missing instead.

It will be good.
@bkeepers What do you think ?


Solve False Malware Detections on Snap Hutao 1.9.8 and Later Versions

Background & Motivation
Starting from Snap Hutao 1.9.8, multiple anti-virus platforms detect the MSIX packages as malware. This was unexpected because there was no change in code associated with dangerous features in Snap Hutao. The biggest change in 1.9.8 was the embedded Windows App SDK upgrade. The false detection may also be caused by sensitive operations in older code, such as Registry operations, but the real reason is unknown.
Due to false detections, Google has blocked the download of MSIX assets from the GitHub release page. Removing false detection is important at this phase to avoid unnecessary concern for new users.
Two major false detections are coming from Microsoft Windows Defender and Google Web Security. Microsoft Windows Defender is the default Windows anti-virus program and it deletes Snap Hutao msix package, and also blocks download if user is using Microsoft Edge browser. Google Web Security is the default anti-virus scanner embedded in Google Chrome and Firefox, they share the very most of the browser market.
Detail of the Feature
Timeline of Actions

03/12 - version 1.9.8 released
03/18 - First report of detected as malware by Windows Defender #1499
03/19 - We submitted a incorrect detection report to the Microsft Security Intelligence platform

03/19 - Microsft Security Intelligence auto scan reports threat by the cloud engine

03/21 - Microsoft replied on the report

At this time, the submitted files do not meet our criteria for malware or potentially unwanted applications. The detection has been removed. Please follow the steps below to clear cached detections and obtain the latest malware definitions.

Open command prompt as administrator and change directory to c:\Program Files\Windows Defender
Run “MpCmdRun.exe -removedefinitions -dynamicsignatures”
Run "MpCmdRun.exe -SignatureUpdate"
Alternatively, the latest definition is available for download here: https://docs.microsoft.com/microsoft-365/security/defender-endpoint/manage-updates-baselines-microsoft-defender-antivirus

Scan results have also been updated; no threat found until today. There is no stop warning if you download the MSIX package from GitHub with the Edge browser. Same situation also apply on version 1.9.9.

03/29 - version 1.9.9 released
03/30 - received user report that downloading through Firefox is blocked as the MSIX is marked as malware
03/31 - Confirmed that Firefox is using Google Web Security engine as the scanner; and Chrome users have the same issue
04/01 - Submitted Report a Security Issue form to Google. Because we are not the owner of the github.com domain, we are not allowed to submit a security appeal to Google directly.

When I downloaded the source code and compiled it, using nuget to grab any missing packages, it compiled easily and Windows Defender did not raise any issues, even after I specifically made Defender scan the newly build exe file.   Good work on this app, it is going to replace paimon.moe and a multi-client genshin login manager that I was using, and it is by far the best looking genshin 3rd party utility app I have seen so far.


XML import merges records together

Original author: iainsproat (September 22, 2010 10:02:42)
Imported a small Xml file but the project, attached, can not be displayed in the browser.  The browser complains of unresponsive script on various lines of project-bundle.js (lines 52, 29, 40 & 109 have been indicated at various stages).
The unresponsive script seems to be happening after the GET response for /scripts/views/data-table/column-header.html
Original issue: http://code.google.com/p/google-refine/issues/detail?id=137

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Is this what might be causing my json import not to give all records an index?

@eswright I suspect it might. I have run into the same problem today actually.

Any suggestions on how to rebuild a record index?   All my data is parsed but just a bunch of records are merged into one.

@eswright Hi Eric, yeah, we have a bug in our column counting strategy, as @tfmorris hinted at above, and its never been found yet because of our lack of time, sorry about that.  But for now, you could just MANUALLY parse the data by selecting the upper most outer record set element visually with the Json Importer...then you'll get some rows...and then manually parse those with Add new column using GREL parseJson() function as our Wiki has a few recipes and examples.  That will probably get you by for now with nice record rows.  Hopefully we can find time to work on this issue more by the end of the year.  Having more example data that causes the issue would be EXTREMELY HELPFUL !  If you can upload your JSON file somewhere and email me the link, then I can analyze more for you.

@thadguidry Here is an example dataset:
https://figshare.com/articles/GRID_release_2017-07-12/5203408
(the file where I observed the problem is "grid.json")

@eswright Thanks Eric.  We got the linked file you sent privately to us.  Yeah, I see a problem, where not all the objects in the large array get created in OpenRefine 2.7 release.  I've sent this to @jackyq2015 as well.  Stay tuned over the next few weeks while we work on a the issue and a fix.
@wetneb Thanks Antonin, that file will help us also !

@jackyq2015 I'd say this need to be our highest priority issue at the moment, it looks like we're throwing away records of data during import. Not good.

Those file provided are huge. I can see the column count issue and it's hard to debug. can somebody provide a minimum data set on which can reproduce this problem? or this issue only happens when have large data set like this?

@jackyq2015 Here is a much smaller file where the same problem appears: the source file contains eight records, OpenRefine merges some of them and obtains only four.
http://pintoch.ulminfo.fr/49a768ad87/grid_small.json

@wetneb Tried this file. it generated a project with 63 columns and 63 rows in row mode. But I did not see the missing rows. The problem I see is that the column and data are messed up. I think for that big file where appeared to import partially may due to the "record mode"?
I need to find out what exactly the issue before I can fix it.

Hi @jackyq2015. In rows mode all is fine indeed, but when you switch to records mode you will see that some records are merged together.

I think the problem is under records mode OR miscalculate the row dependency under some situation. It causes some root node were flagged as a child node. From the UI aspect, they are merged together. Not found the solution yet. will work out it.

I fixed it. can somebody help to test it from https://github.com/jackyq2015/OpenRefine? If it works as expected, I will create a PR for this. cheers!

@jackyq2015 that's fantastic! It works indeed for my example.
It'd be great to have a test for that (I can help if needed).


Game data omits defensive totals, but shouldn't.

Need to rework the game by game scraper to add the defensive totals back in. The issue is they are on a separate line from the offensive totals, so there's some work to be done to connect them.

Fixed in this commit


Create PUT and DELETE routes for updating and deleting events

Will have to set up PUT and DELETE routes for updating and deleting events on the eventlist page


Bad documentation

You should improve the documentation. It looks very lazy, shame because its such a great library.

@neoighodaro thanks for your feedback
Can you tell me which particular section you're referring to? In the main readme, there are a number of links to the doc sections - which should give you all the documentation you need.  Perhaps I'm missing something?  Can you give an example?

It just occurred to me that you might be referring to the documentation on the madmimi.com website?  If so, I'm not affiliated with that company.  I just needed to use the API, so I created this PHP library to do it easier and normalize the process.
Again, for reference, this API is documented using the README.md file - otherwise here is a link to the docs as well:  https://github.com/aaronsaray/madmimi-api-php/tree/master/docs


CSP unsafe-inline support

Just debugging some JS in the contrast-widget for AdaRoseEdwards, and came across:
https://github.com/gionkunz/chartist-js/blob/develop/dist/chartist.js#L339
svg = new Chartist.Svg('svg').attr({
  width: width,
  height: height
}).addClass(className).attr({
  style: 'width: ' + width + '; height: ' + height + ';'
});

It looks like the width and height is being set twice, the second one using an inline style attribute (which is considered as unsafe-inline when you're applying a CSP header).
Am I missing something, or can that just be changed to:
svg = new Chartist.Svg('svg').attr({
  width: width,
  height: height
}).addClass(className);

Hi there, yeah somehow I missed the comment on this line sorry. The reason for this was a bug in Firefox which did not behave correctly on width and height only set as attributes on the SVG element in some cases. We'd need to re-validate if that's still an issue and remove if it works without the style properties across all supported browsers.

Thanks @gionkunz, realise browser support can be difficult... I would be interested to know if/when it was fixed in Firefox, as 44.0.2 seems to be ok without it (well, in this case at least).

Closing this issue in favor of #620


"leading minor not positive definite" in Cholesky decomposition (different case than issue #26)

Recently I got a problem in the Cholesky decomposition when I run star_BD.py --samples=5
Traceback (most recent call last):
  File "/usr/local/AstroPhysics/Python_Anaconda/anaconda2/envs/python34/lib/python3.4/multiprocessing/pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "/usr/local/AstroPhysics/Python_Anaconda/anaconda2/envs/python34/lib/python3.4/multiprocessing/pool.py", line 44, in mapstar
    return list(map(*args))
  File "/usr/local/AstroPhysics/Python_Anaconda/anaconda2/envs/python34/lib/python3.4/site-packages/emcee/ensemble.py", line 519, in __call__
    return self.f(x, *self.args, **self.kwargs)
  File "/Users/zhang-dirac/Dropbox/Laniakea/ZhangDirac/OtherWorlds/Starfish/scripts/star_BD.py", line 515, in lnprob
    return lp + lnlike(p)
  File "/Users/zhang-dirac/Dropbox/Laniakea/ZhangDirac/OtherWorlds/Starfish/scripts/star_BD.py", line 496, in lnlike
    lnp = model.evaluate()
  File "/Users/zhang-dirac/Dropbox/Laniakea/ZhangDirac/OtherWorlds/Starfish/scripts/star_BD.py", line 208, in evaluate
    factor, flag = cho_factor(CC)
  File "/usr/local/AstroPhysics/Python_Anaconda/anaconda2/envs/python34/lib/python3.4/site-packages/scipy/linalg/decomp_cholesky.py", line 132, in cho_factor
    check_finite=check_finite)
  File "/usr/local/AstroPhysics/Python_Anaconda/anaconda2/envs/python34/lib/python3.4/site-packages/scipy/linalg/decomp_cholesky.py", line 30, in _cholesky
    raise LinAlgError("%d-th leading minor not positive definite" % info)
numpy.linalg.linalg.LinAlgError: 44-th leading minor not positive definite

To explain: star_BD.py, in my branch zj_zhang42/Starfish/ZJ_BD_v0, is similar to star_mix.py originally written by @gully for stars with stellar spots. I paraphrased it so that it could do the MCMC fitting for a single object.
Based on the issue #26 and this solution, I checked my covariance matrix CC. As a result:
(1) all diagonal elements are positive (i.e., no negative sigAmp; no NaN diagonal elements)
(2) the covariance matrix CC is not highly asymmetric.
(3) there are several negative eigenvalues, e.g.,
- There are 42 negative eigenvalues out of 560.
Covariance matrix eigenvalues:
     0 >>>>>>>>>>-1.614e-17
     1 >>>>>>>>>>-1.546e-17
     2 >>>>>>>>>>-9.968e-18
     3 >>>>>>>>>>-6.505e-18
     4 >>>>>>>>>>-4.648e-18
     5 >>>>>>>>>>-3.590e-18
     6 >>>>>>>>>>-1.956e-18
     7 >>>>>>>>>>-1.481e-18
     8 >>>>>>>>>>-9.862e-19
     9 >>>>>>>>>>-7.725e-19
            ...
   550 ...........2.042e-02
   551 ...........2.246e-02
   552 ...........2.268e-02
   553 ...........2.329e-02
   554 ...........2.625e-02
   555 ...........2.869e-02
   556 ...........3.873e-02
   557 ...........7.415e-02
   558 ...........1.137e-01
   559 ...........2.542e-01
------------------------------------------------------------

As another experiment, I ran CC_debugger for both self.data_mat and self.Omega**2 * self.flux_scalar**2 * X.dot(self.C_GP.dot(X.T)). As a result, self.data_mat usually has positive eigenvalues, while the second term usually has many negative eigenvalues.
I suspect it could be a matter of initial parameter choice in the config.yaml. If I change a bit several initial Theta and Theta_jump parameter values, then I would see the error message (i.e., "leading minor not positive definite") after 20 (or 3) samples instead of 5. Changing Theta/Phi parameters could change self.data_mat, which in turns would be a dominant or unimportant term of the whole CC matrix and make CC positive/not-positive definite. But I am not sure by how much the initial parameters matter.
Since I've just seriously used Starfish for 1 week, there might be many details that I don't know. So it would be nice if any of you could help with it!

I just visualize my np.abs(CC) - CC matrix in the following figure:

I am wondering if the negative structure is caused by a large global covariance kernel or any local covariance (though I didn't intend to add a local covariance in my code)? I am still exploring the reasons and will update my thoughts if any.

Hrm, this frustrating bug rears its head once again. If you are not including any of the "local" covariance kernels (i.e., the Gaussian shaped lines), then the structure that you see in this matrix is likely due to the emulator matrix, since it this sort of matrix conveys the uncertainty in the emulated spectra.
What are the wavelength ranges (X and Y axes) on your matrix plot? Also, does the emulator work fine on its own to sample spectra, or does it yield an error there as well?

@iancze The wavelength range should be (6579.71, 25604.9) with a length of 560 using the IRTF/SpeX instrument (with resolution gradient considered). I didn't get any error messages before I run this star_BD.py (if I don't check the autocorrelation time; see below), so the emulator itself may work fine.
However, if the hyper parameters of the emulator didn't converge when I optimized them using pca.py --optimize=emcee --samples=20000, will that be a reason for this bug? I was using 20000 as a sample number in the emcee pipeline. And I added one more line in pca.py of my version to show if 20000 is enough for convergence. As a result, I got the following error message:
Burned in
Traceback (most recent call last):
  File "/Users/zhang-dirac/Dropbox/Laniakea/ZhangDirac/OtherWorlds/Starfish/scripts/pca.py", line 243, in <module>
    np.save("autocorr_emcee.npy", sampler.get_autocorr_time())
  File "/usr/local/AstroPhysics/Python_Anaconda/anaconda2/envs/python34/lib/python3.4/site-packages/emcee/ensemble.py", line 503, in get_autocorr_time
    fast=fast)
  File "/usr/local/AstroPhysics/Python_Anaconda/anaconda2/envs/python34/lib/python3.4/site-packages/emcee/autocorr.py", line 118, in integrated_time
    raise AutocorrError("The chain is too short to reliably estimate "
emcee.autocorr.AutocorrError: The chain is too short to reliably estimate the autocorrelation time

So it seems that 20000 might be not enough for optimization. Since I was under a special case and was trying to make more progresses within a limited time (I will definitely be more serious after this short period), I just ignored this message and continue my next step, i.e., store params and then run my star_BD.py.
P.S. As a rule of thumb, running 20000 samples with 6 eigenspectra (with N_pix=560) and 2 stellar parameters (Teff and log g) takes 7 hours on my laptop (MacOS 10.12.6 with 2.9 GHz Intel Core i7). So it seems that I should have tried more samples, say >2.0e5 (?), to get better hyper parameters, although that would in principle take several days.

As a work-around, I would recommend just turning off the emulator matrix component for now (simple diagonal and GP noise model).
Also the "flux_scalar" method is flawed-- I will share some experimental code that removes the need for the flux_scalar hack, and could possibly fix the other problems in the process.

@gully Thanks! It would be nice to see your experimental codes!
Now I am trying to use CC = self.data_mat in evaluate(self), but the problem still occurs. The problem is that my self.data_mat has some zero eigenvalues (see the following debugging information from the CC_debugger).
------------------------CC_debugger-------------------------
See https://github.com/iancze/Starfish/issues/26
Covariance matrix at a glance:
- There are 0 negative eigenvalues out of 560.
Covariance matrix eigenvalues:
     0 ...........0.000e+00
     1 ...........0.000e+00
     2 ...........0.000e+00
     3 ...........0.000e+00
     4 ...........0.000e+00
     5 ...........0.000e+00
     6 ...........0.000e+00
     7 ...........0.000e+00
     8 ..........-0.000e+00
     9 ...........0.000e+00
            ...
   550 ...........9.448e-34
   551 ...........9.553e-34
   552 ...........1.177e-33
   553 ...........1.239e-33
   554 ...........1.474e-33
   555 ...........1.621e-33
   556 ...........1.637e-33
   557 ...........1.681e-33
   558 ...........1.895e-33
   559 ...........2.795e-33
------------------------------------------------------------

When I firstly mention my problem, I said

As a result, self.data_mat usually has positive eigenvalues, while the second term usually has many negative eigenvalues.

However, what I said is not true, which I didn't realize until now. I should have said that my self.data_mat does not have negative eigenvalues (I have revised that sentence in my first conversation and cross out the old one for history tracking purpose).
So now another question is why my self.data_mat have zero eigenvalues? I will keep experimenting and trying to find a solution before your experimental codes are released.

What is the noise vector that you are using for the SpeX data? Is it possible that some of the values are very small, or actually zero due to some sort of masking/calibration process? What happens if you multiply your measurement uncertainties by a healthy factor?
Regarding the emulator, yes, getting it converged properly is definitely a priority for fitting spectra. It's unfortunate that the fitting process is taking so long for you. Are you able to reduce the number of eigenspectra or limit the parameter range over which you are training the emulator? Hope this helps.

All of the Gaussian process libraries I've looked through while refactoring the emulator have had something in common to help out with this, I believe. The covariance matrix delivered by the kernel has "jitter" added to it before being sent to the multivariate normal likelihood. An example looks like this-
cov = kernel(X, X)
cov = cov + np.diag(noise) + 1e-6 * np.eye(*cov.shape)
like = stats.MultivariateNormal(0, cov)

This should not be considered a working example, but you can see how the covariance matrix is given help of  1e-6 to avoid negative eigenvalues which happen to occur erratically from floating point operations.
I can't say with certainty that this will fix the issue, but I wouldn't hesitate to incorporate it in a custom patch by going to Starfish/emulator.py and chaging
@property
def matrix(self):
   return (self.mu, self.sig)

to
@property
def matrix(self):
   return (self.mu, self.sig + 1e-6*np.eye(*self.sig.shape))

I know it's been a while since I posted this issue, and thank you all for advice.
Recently I surprisingly notice that this issue has gone, but unfortunately, I cannot track the specific reason. For the record, I just address what has been done since this issue: I switched to some of Gully's codes saved in his jammer-GJ570D and then made my own polish/additions in my own branch ZJ_BD_v0. Also, I have used the updated grid models and have trained more spectral emulators with different parameter ranges and PCA threshold, etc. Now for each of my new spectral emulator, I can smoothly incorporate the emulator covariance matrix into the entire covariance matrix when fitting the objects (with no issue anymore).
Also, thank @mileslucas for suggesting adding the jitter to the kernel. I did not yet try that in my code, but the issue has gone. However, it seems that my initial issues, as mentioned from my first few posts in this threads, should have been fixed by adding a small jitter, given that those non-negative eigenvalues are either 0 or slightly negative.
To reply @iancze: I have masked/removed the pixels with non-positive and NaN flux uncertainties in my SpeX data before feeding them into the fitting process, so the bad flux errors might not be the reason.
I will close this issue for now and we might re-open if it happens again. Otherwise, hopefully my post here can give some advice for other users who will have the same issue.


added Webcam Stop feature

The fix works for me. I've only added in the changes in the Vendor folder. For some reason, when i run grunt js, it doesn't add my changes to the dist and currently, i'm time constrained to look into the grunt issue.

You have to put your changes in src directory. Both dist and vendor are automatically generated.

@amw Thanks for the update. I just looked at the SRC directory. Unfortunately, i'm not familiar with Coffeescript syntax. The Javascript code i wrote works. If you can help write in coffeescript, i'll appreciate it. Thanks

Unfortunately I can't add the feature myself. Beside translation the code would also need to be separated into html5 and flash source files. If you're still interested in this feature in the future and you have time to redo the implementation then I will gladly accept a revised PR.


sidecar channels 3/3: Accept sidecar channels

Depends on #242, only last 14 (small!) commits are new.
This is the third part of what's needed to execute sidecar orders, the actual acceptance of sidecar orders in the auction server stream.

I've addressed all comments and also added some more unit tests where it made sense.


Support Different Query Parameters

Based off #6 we should add functionality to allow querying data aside from just ref.Value()
Related Firebase Docs:

https://www.firebase.com/docs/rest/api/#section-query-parameters

Closed by #34


closes #2520: upgrade to quarkus 3.x

Work in progress..
What this PR does:
Opening the PR in draft that should provide us with feedback on upgrading to Quarkus 3.x line. This is work that can not be done immediately, so we want to proceed step by step.
Guides:

https://quarkus.io/blog/quarkus-3-0-0-cr2-released/#upgrading
https://github.com/quarkusio/quarkus/wiki/Migration-Guide-3.0

Which issue(s) this PR fixes:
Fixes #2520
Checklist

[x] Major release version, not release candidate
[x] Align gRPC versions coordinator & apis
[x] Multi-part breaking changes
[ ] Management endpoint breaking changes (optional, can be moved to separate port)
[x] Open-telemetry config breaking changes
[x] Fix compile issues
[x] Unit tests green
[x] Integration tests green
[x] Docker image upgrades
[x] Silent LoggingProviderImpl
[x] Fix DevUI link, move start page to v3
[x] Remove basic auth from swagger
[x] jackson/snakeyaml version resolution (cc. @tatu-at-datastax)
[x] need https://github.com/quarkusio/quarkus/issues/32944 released
[ ] time zone test issue

Vast majority of changes are for JAXB -> Jakarta API changes (imports) which are of course necessary and fine. Also a few minor version upgrades which should be fine as well (I think our test suites would catch issues with Duzzt f.ex). And also some config name changes (opentelemetry -> otel).
Did not see anything unexpected although not very familiar with new/extended tests for consistency. Assuming they are fine (and will try to re-read just to learn).
So looks good, relatively small actual changes it sounds like.

@tatu-at-datastax

new tests for consistency are not new, there were just inner classes QueryExecutorTest$WithDifferentConsistency, but this does not work anymore, so I just pulled them out as separate class
opentelemetry -> otel is the braking change, you can check in the release notes

@tatu-at-datastax @jeffreyscarpenter can you try running this locally, check if you see anything suspicious, try swagger, try starting docker images, a bit more checks, rather than simply looking into the code?

I built everything locally and ran all integration tests for API services (errors noted above), then built docker images and started docker compose for Cassandra 4.0. I ran Postman collections against each API with no errors. I also opened all swagger/health/metrics endpoints as listed at https://github.com/stargate/stargate/tree/main/docker-compose#exposed-endpoints.

I got three test failures on GraphQL integration tests when running on my desktop:
[ERROR]   ScalarsIntegrationTest.shouldSupportScalar:97
expected: "2023-05-10T21:07:35.949Z"
 but was: "2023-05-10T14:07:35.949-07:00"
[ERROR]   DataTypesIntegrationTest.insertAndRetrieveValue:95
expected: "2011-02-03T04:05:00.000Z"
 but was: "2011-02-02T21:05:00.000-07:00"
[ERROR]   DataTypesIntegrationTest.insertAndRetrieveValue:95
expected: "2011-02-03T04:05:00.000Z"
 but was: "2011-02-02T21:05:00.000-07:00"
[INFO]
[ERROR] Tests run: 250, Failures: 3, Errors: 0, Skipped: 1

Will try to understand this and fix.. Not sure if this happens only on 3.x or not, but your time zone does have the influence..

Rebased, updated to 3.0.3.Final and bump maven plugin versions..

Looks good to me, I think we should move forward.


Reductio

Reductio - Automatic summarizer text in Swift :large_orange_diamond:

@dkhamsing @vsouza HOLY SHHHHHHH, THIS IS AMAZINGLY AMAZING!

where would that fit? Text?

ya

I was thinking it could also fit Data structures/Algorithms section because it uses TextRank, but you're right... It fits better under Text

this may seem radical.. but you could put it under both section (remember to white list it in .travis.yml)


Include a File Upload Example

HI, if you could include an example with a file upload that would be really appreciated as i have been having some trouble with this.

I am running a file upload using React dropzone here: https://github.com/jahmaiosullivan/newsfeed/tree/master/src/components/ImageUpload
It turned out not to be that hard. I would be more than happy to walk you through it if you have specific questions.

@jahmaiosullivan thank you but i should have probably been a little more specific, I need to know how to do a file upload with redux forms and also if there are any adjustments i need to make to the  ApiClient in order for it to work.

If you need to add a file to upload, you must add multer to express that can handle multipart/form-data.
Then you need tweak ApiClient so that it can send files.
And finally you can use react-dropzone with redux-form.
I made you a complete example on the upload branch. (It's accessible from /upload)
You can view changes here.

I'm Building on OSX-Sierra And Centos - /upload Does Not Work! I've Checked "https://react-hot-example.herokuapp.com/upload" - Does Not Work There Either - All Branches After fix: "pin feathers-hooks-common to 3.7.1" Fail for Me!


Velocyto samtools memory error

Hi,
I ran velocyto verison 0.17 with samtools version 1.15.1 with 128GB memory. My bam files were standard 10X output (STAR aligner) and they are sorted by coordinate. Why do I still get this error?
`Otherwise sort manually by samtools sort -l [compression] -m [mb_to_use]M -t [tagname] -O BAM -@ [threads_to_use] -o cellsorted_[bamfile] [bamfile]")
MemoryError: bam file #0 could not be sorted by cells.
This is probably related to an old version of samtools, please install samtools >= 1.6.                In alternative this could be a memory error, try to set the --samtools_memory option to a value compatible with your system.                 Otherwise sort manually by samtools ``sort -l [compression] -m [mb_to_use]M -t [tagname] -O BAM -@ [threads_to_use] -o cellsorted_[bamfile] [bamfile]```
I saw similar issue but no resolution, so had to post a new issue.
Thanks,
Payal

Can you solve it? I have the same error..


Add flavouriz

add .env installer

The two apps running concurrently.

Hello, @josancamon19  this is up for review, if you would.

If @ologunB you could upload ios screenshot would be fantastic

@josancamon19 I did here

The two apps running concurrently.

@josancamon19 I will be emailing you my details, thank you.

@ologunB android looks like this

@ologunB android looks like this

fixed


Custom Render For multi Eyes Colors for latexs and transfur players

A render that add a sub texture in eyes location for almost all latexs entitys like the visual transfur progress render feature
The render can use a pre made texture (can be edit by a texture pack) with no color (white) and will be set the color randomly via NBT (using Code HEX the "#colorcode") or a math in an external code the colors can be set using the Color3
Not all latexs because Isn't all transfurs that can have the eye color change like dark latex wolfs,pups,knight,the horse knight one(i don't remember the name)
optional (a boolean config) that uses the eyes of the player's skin to set the eye color
When on need config to set the eyes location of the player skin OR can be made a pre types like
Type 1 = Steve eyes type
Type 2 = 2 Pixels Steve eyes type
Etc

In bad endings of Changed, Colin TF'd into a dl wolf (+pup TF) has his blue eyes, they have white eyes prob cause they have consciousness of the latex.
(+he has his blue eyes in the knight fusion TF)

I added some basic customize info for hair, iris, and sclera color. All this work is done on the animation-rework branch.
https://github.com/LtxProgrammer/Changed-Minecraft-Mod/blob/animation-rework/src/main/java/net/ltxprogrammer/changed/entity/BasicPlayerInfo.java

Feature complete in animation-rework branch.

tbh dark latexes should have that feature work for them too - they're not an exception to the rule/logic (no tfs are)


Issue using typing-extensions > 4.5.0

Hi,
when I was trying you package with one of the newest versions of FastaAPI which depends on typing-extension >=4.8.0, I ran into a typing error in your code. It applies I believe only to python version less than 3.10. There were changes to Literal which you're using in cases of some of your pydantic Models
I have to use python 3.9 along with Pydantic 1.10.x for dependency purposes. Our company cannot raise the versions of some libraries without proper testing. Is it possible on you side to create new 0.6.x patch version, which uses typing which would work on Python 3.9.
I added links to the issue in Pydantic.
https://github.com/pydantic/pydantic/issues/5824
https://github.com/python/typing_extensions/commit/fb37b2ee0ab56baf2dfd5df27aec0430435ba17f
https://github.com/pydantic/pydantic/issues/5821#issuecomment-1558716103
https://github.com/pydantic/pydantic/issues/545#issuecomment-1573773542
for better location of error I added logs. It happens already when trying to import the Model Point
rypmaps-gateway-1  |   File "/varistar/./app/schemas/response_schemas.py", line 3, in <module>
rypmaps-gateway-1  |     from geojson_pydantic import Feature, Point
rypmaps-gateway-1  |   File "/usr/local/lib/python3.9/site-packages/geojson_pydantic/__init__.py", line 3, in <module>
rypmaps-gateway-1  |     from .features import Feature, FeatureCollection  # noqa
rypmaps-gateway-1  |   File "/usr/local/lib/python3.9/site-packages/geojson_pydantic/features.py", line 9, in <module>
rypmaps-gateway-1  |     from geojson_pydantic.geometries import Geometry, GeometryCollection
rypmaps-gateway-1  |   File "/usr/local/lib/python3.9/site-packages/geojson_pydantic/geometries.py", line 114, in <module>
rypmaps-gateway-1  |     class Point(_GeometryBase):
rypmaps-gateway-1  |   File "pydantic/main.py", line 204, in pydantic.main.ModelMetaclass.__new__
rypmaps-gateway-1  |   File "pydantic/fields.py", line 488, in pydantic.fields.ModelField.infer
rypmaps-gateway-1  |   File "pydantic/fields.py", line 419, in pydantic.fields.ModelField.__init__
rypmaps-gateway-1  |   File "pydantic/fields.py", line 534, in pydantic.fields.ModelField.prepare
rypmaps-gateway-1  |   File "pydantic/fields.py", line 638, in pydantic.fields.ModelField._type_analysis
rypmaps-gateway-1  |   File "/usr/local/lib/python3.9/typing.py", line 852, in __subclasscheck__
rypmaps-gateway-1  |     return issubclass(cls, self.__origin__)
rypmaps-gateway-1  | TypeError: issubclass() arg 1 must be a class

code:
from geojson_pydantic import FeatureCollection, Point

Thank you for your help.

A workaround is using no_type_check decorator from typing
from typing import no_type_check

from geojson_pydantic import FeatureCollection, Point
no_type_check(Point)

@ondrej-ivanko I guess you're using geojson-pydantic 0.6
I'll be happy to review any PR patching 0.6 with a fix 🙏


Do you have a plan that redis-rdb-tools be developed by c/c++?

There is a proof of concept work that is available in a fork, see this discussion - https://github.com/sripathikrishnan/redis-rdb-tools/issues/118
Unfortunately, I don't have a timeline by which it will be ready to merged back into unstable. Only running memory analysis is easier to merge, but adding support for all features that redis-rdb-tools supports is more work, and that is causing delays.

Ok, I got it.


afterChange doesn't fire with lazyLoading: 'ondemand'

CodeSandBox: https://codesandbox.io/s/react-typescript-sq6ih
When i'm using lazyLoading option with 'ondemand' value, afterChange function doesn't fire. It's easy to reproduce by fast clicking on the "next arrow". But if all images already loaded bug will not reproduce (for example if u will press prev arrow).
Steps to reproduce:

After render page, just press on next arrow fast
You will see, that beforeChange fires every time, but afterChange sometimes doesn't fire
Watch on the console

Even if u already scrolled to last slide, afterChange could still doesn't fire, looks like a bug..

p.s. sry for my eng.

any news?

Also see https://github.com/akiran/react-slick/issues/1262 and its associated MR.


Change backgoundColor of tableView

I am trying to find a way to change the color of the tableView background, not only the cell background. Is there any way to accomplish this?
My theme is dark so under the search, a white background appears and looks very ugly.
I have tried changing the background color of pretty much every (background) view inside the Framework but no luck.

I am so sorry, NVM. It was way easier than what I thought. Figured it out!

Reopening because I did not find a solution. What I thought was fixing it really didn't and I can't seem to figure out how to get that to work. Please refer to the images I uploaded before. The background color of what I think is the tableView.background color.
The customization allows you to change the color of each cell but not of the actual tableView

@tapizquent did you try this?
selectionMenu.tableView?.backgroundColor = UIColor.black
or
selectionMenu.tableView?.backgroundView?.backgroundColor = UIColor.black

@tapizquent did you try this?


Use environment variables

The $http_address in HttpClientImportPogramme needs to be an environment variable since I can use different servers for the testing environment of your application and the production environment. Currently, you hardcode it and it cannot be changed.

Created PR https://github.com/marceldobrica/training/pull/34


get_bio_content() throws an unexpected exception on edge case artists

The code below throws AttributeError: 'NoneType' object has no attribute 'wholeText'
pylast = pylast.LastFMNetwork(api_key=api_key, api_secret=shared_secret)
artist = pylast.get_artist('Mr Sizef + Unquote')
artist.get_bio_content()

(pylast 3.2.1)

Thanks for the report.
Please see https://github.com/pylast/pylast/pull/326 for a fix.
It happens because there's no biography for this artist.
 A major figure for over five decades in the world of popular culture, she is often referred to as the Goddess of Pop for having first brought the sense of female autonomy and self-actualization into the entertainment industry. She is known for her distinctive contralto and for having worked extensively across media, as well as for continuously reinventing both her music and image, the latter of which has been known to induce controversy. Read more on Last.fm

<content>
Cher (born Cherilyn Sarkisian; May 20, 1946) is an Oscar- and Grammy-winning American singer and actress. A major figure for over five decades in the world of popular culture, she is often referred to as the Goddess of Pop for having first brought the sense of female autonomy and self-actualization into the entertainment industry. She is known for her distinctive contralto and for having worked extensively across media, as well as for continuously reinventing both her music and image, the latter of which has been known to induce controversy. Cher was born Cherilyn Sarkisian in El Centro, California, on May 20, 1946. She is of Armenian-American, Irish, English, German, and Cherokee ancestry. Cher first caught the eye and ear of the public in 1965 as one-half of the pop rock duo Sonny & Cher, which popularized a peculiar smooth sound that competed successfully with the predominant British Invasion and Motown Sound of the era. After a period in which the duo became obsolete, she re-emerged in the 1970's as a television personality with her shows The Sonny & Cher Comedy Hour and Cher, which attained great popularity. At the same time, she established herself as a solo artist with hits such as "Bang Bang (My Baby Shot Me Down)", "Gypsys, Tramps & Thieves", "Half-Breed", and "Dark Lady", which dealt with taboo subjects in mainstream popular music. Throughout, she cemented her status as a fashion trendsetter with her daring outfits, and was noted as being the first woman to expose her navel on television. Cher's impact at that time, as described by Phill Marder from Goldmine magazine, "led the way to advance feminine rebellion in the rock world," as she was "the prototype of the female rock star, setting the standard for appearance and [...] attitude[.]" In 1983 Cher made a critically acclaimed appearance on Broadway and starred in the film Silkwood, which earned her a nomination for the Academy Award for Best Supporting Actress. In the following years, she became an acclaimed film actress by starring in a string of hit films, including Mask, The Witches of Eastwick, and Moonstruck, for which she won the Academy Award for Best Actress in 1988. At the same time, she established herself as a "serious rock and roller" by releasing a series of rock albums and hit singles such as "I Found Someone", "If I Could Turn Back Time", and "The Shoop Shoop Song (It's in His Kiss)". She made her directing debut with the film If These Walls Could Talk in 1996 and released the biggest-selling single of her career thus far, "Believe", which revolutionized the recording industry because of its pioneer use of Auto-Tune (also known as the "Cher effect"). Throughout the 2000's she embarked on a series of concert tours that were among the highest-grossing of all-time. Cher has won an Academy Award, a Grammy Award, an Emmy Award, three Golden Globe Awards, and the Best Actress Award at the Cannes Film Festival for her work in film, music and television. She is the only person to receive all of these awards. She is also the only artist to have notched a number-one single on a Billboard chart in each of the past six decades. Recognized as one of the best-selling music artists of all time, she has sold more than 100 million solo albums worldwide and 40 million records as part of Sonny & Cher. Official website: http://www.cher.com/ Read more on Last.fm. User-contributed text is available under the Creative Commons By-SA License; additional terms may apply.
</content>
</bio>
</artist>
</lfm>

https://ws.audioscrobbler.com/2.0/?method=artist.getinfo&artist=Cher&api_key=YOUR_API_KEY

@spiritualized Please could you check #326?

Looks good to me.


Put CMakeLists.txt in the project root.

Having CMakeLists.txt in the project root makes it so much easier to use pugixml
as an external dependency in another CMake project.

@zeux  :+1: I also vote for this change.
It'll help to re-use your project easily with External_ProjectAdd.

There's a small issue with putting CMakeLists.txt in the root - running cmake will probably overwrite the default Makefile. Also I guess some Linux distributions will have to adjust their packaging scripts as a result of this change (although I could temporarily leave CMakeLists.txt in scripts/ folder, and remove it in a following version).
Regardless, this seems like a good change. I'll test this a bit before merging.

@zeux you can enforce out-of-source build, as clang do and as I've implemented for splunk/pion to avoid default makefile overwrite.

Here the example

Any news on this? Thanks!


Load assets async

Description
Make sure loading of static assets such as logos in procects are loaded asynchronously.
Todo

[ ] Check out if ionic has some api for loading async or use some other standard api
[ ] Implement

[ ] Project logos
[ ] Other static assets

This has been resolved by @irony in commit 19f9b57
We will serve the content directly from the app (www/img folder).
To eplxicitly load assets async is probably not needed


Added emojis in issue template names

Description

I added emojis to the issue template names to make them intuitive and easy to find
Contribution Guidelines

[x] By creating this pull request, I acknowledge that I have read the Contribution Guidelines for this project.
[x] I have read the Code Of Conduct and promise to abide by these rules

Walkthrough
This update enhances the visual appeal of the GitHub issue templates for bug reports and icon requests by adding relevant emojis to their titles. The changes involve modifying the name fields in two YAML files, transforming "Bug Report" to "🐛 Bug Report" and "Icon Request" to "🏞 Icon Request." These cosmetic updates aim to improve user engagement without altering the functionality or structure of the templates.
Changes
Files	Change Summary
.github/ISSUE_TEMPLATE/bug-report.yml	Changed name from "Bug Report" to "🐛 Bug Report"
.github/ISSUE_TEMPLATE/icon-request.yml	Changed name from "Icon Request." to "🏞 Icon Request"
Sequence Diagram(s)
sequenceDiagram
    participant User
    participant GitHub
    participant Template

    User->>GitHub: Open issue template
    GitHub->>Template: Load template
    Template-->>User: Display "🐛 Bug Report" or "🏞 Icon Request"

Recent review details
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Share
Tips

Well, the chart is too much 😅, @PKief, turn off the charts


chore(master): release 1.1.41

:robot: I have created a release beep boop
1.1.41 (2023-08-25)
Miscellaneous

deps: update dependency phpstan/phpstan-phpunit to v1.3.14 (30ead28)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-webhook-client/releases/tag/v1.1.41 :sunflower:


Daniel Martin-Alarcon DS01

Included my last name, because there's always more than one Daniel.

This is the second comment.


Missing keys in web.config

The sample OIDC application mentioned in the article  (https://github.com/Azure-Samples/active-directory-dotnet-webapp-openidconnect) lacks for example ida:ADFSDiscoveryDoc in the web.config

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

ID: 473bc6ea-017f-52f3-7e58-81ecf58d2db8
Version Independent ID: 20965612-d6a9-35ea-c030-91c09372962e
Content: Build a web application using OpenID Connect with AD FS 2016 and later
Content Source: WindowsServerDocs/identity/ad-fs/development/enabling-openId-connect-with-ad-fs.md
Product: windows-server-threshold
Technology: identity-adfs
GitHub Login: @billmath
Microsoft Alias: billmath

Please disregard, after re-reading I understood that I should add the keys and not modify existing keys.


Project status

Hi there! I wanted to reach out and ask about the project status and if it's still considered under active development. The examples in the cronjobs are versioned to 1.5 and the docker hub images only go to 1.9.3. I see some more tags (up to 1.12) so I'm reaching out to see if this is still under development / has regular releases.

Hmm nevermind. I was looking at a specific release before where things hadnt been changed from docker hub to quay. This is more clear to me now. Thanks again for the open source project!!


Deprecate s3-inventory-based import

Currently there are multiple ways to import data into lakeFS. The oldest of which (and also the most complicated) is the s3 inventory based import.
Given we currently have more approachable ways of import data into lakeFS, I suggest deprecating and eventually removing the s3 inventory based import (docs, lakefs cmd and accompanying parts of the codebase)

@ozkatz Where do we still have it. I think I've did this some months ago, please reopen and link to any leftovers.


Removing basic errors

Hi Insider Team,
I started reading the code a little more and noticed that it had some grammatical errors and also some unnecessary comments.
I know this is basic, but I believe it has a certain impact, even if small.
Hope this helps ;)
Thx!

I also adjusted the indentation of a code snippet!


Add lwip/TinyUSB docs to the SDK docs

We now include Doxygen from cyw43_driver; we should try to add TinyUSB and lwip also (though given how fickle Doxygen is, i don't know whether we can do this directly as part of our main doxygen pass)

actually we should just link to their documentation site since we are HTML anyway

Just an FYI. We're in the process of integrating the Pico SDK API directly into the documentation site, see https://github.com/raspberrypi/documentation/pull/2743 for details, and will be spinning down the existing micro-site.

...although we'll want to keep the Doxygen build "working", as that's used by the PR Alasdair mentioned, and we'll also be keeping the "Appendix E: Building the SDK API documentation" in https://datasheets.raspberrypi.com/pico/raspberry-pi-pico-c-sdk.pdf

yeah what i meant was that rather than trying to batch include the documentation somehow as part of our tree, we'll put links to their documentation sites in the relevant places in our docs.

Yup. Got it!
A taster of what things should look like at the top level.

fixed by #1220

merged into develop


Error unprotecting rtp packet

I see lots of  Error unprotecting rtp packet errors ,  but  webrtc connections  can still work.
'''
[0x700004913000][1535356625.378][ERR]-DTLSICETransport::onData() | Error unprotecting rtp packet [9]
[0x700004913000][1535356626.429][ERR]-DTLSICETransport::onData() | Error unprotecting rtp packet [9]
[0x700004913000][1535356627.482][ERR]-DTLSICETransport::onData() | Error unprotecting rtp packet [9]
[0x700004913000][1535356628.538][ERR]-DTLSICETransport::onData() | Error unprotecting rtp packet [9]
[0x700004913000][1535356629.596][ERR]-DTLSICETransport::onData() | Error unprotecting rtp packet [9]
[0x700004913000][1535356630.646][ERR]-DTLSICETransport::onData() | Error unprotecting rtp packet [9]
[0x700004913000][1535356631.701][ERR]-DTLSICETransport::onData() | Error unprotecting rtp packet [9]
'''

This may be harmless or not :)
Can you reproduce it with a single client? can you describe a bit more your setup?

Error number 9 is srtp_err_status_replay_fail
That happens if same packet is received twice, which may be a case with FF as it doesn't implements RTX and the media server can request the same packets a couple of times until it is received.


Merge v1.0.0 release PR into community-operators repo

Merge a PR into the community-operators repo for the v1 tagged release of ICO.
Here's a previous release PR as a guide: https://github.com/operator-framework/community-operators/pull/1911
To Do:

[ ] Merge https://github.com/IBM/cloud-operators/pull/194 first (README is used in OLM install page)
[ ] Preview cluster service version (CSV) on Operator Hub Preview
[ ] Cut v1.0.0 tagged release
[ ] Use the CRDs, package.yaml, and clusterserviceversion.yaml artifacts from the tagged release attachments
[ ] Remove the replaces: line from the auto-generated CSV yaml (no prior release in Operator Hub stable channel to replace this time)

I would also add test on OpenShift

OpenShift has tested out well since #200 went in. I also have good reports from others using the latest on OpenShift 👍

Community PR has been merged: https://github.com/operator-framework/community-operators/pull/2379

Closing since OpenShift PR is merged and available on the OpenShift console install screen 🎉
The second, upstream k8s PR is also now open here: https://github.com/operator-framework/community-operators/pull/2398


Add assert and assume extern functions to PNA include file?

They have recently been added to the PSA include file with this commit: https://github.com/p4lang/p4-spec/blob/main/p4-16/psa/psa.p4#L340-L403
Do we want these added to pna.p4 include file as well?  They can be very useful in testing of P4 code on software-based devices, e.g. BMv2 and DPDK.  Typically they would be compiled as no-ops on hardware targets.

+1 from me.

Is the architecture include file the right place for defining these methods? It seems to me like they would be better placed in an orthogonal header (e.g. core.p4), as there is really nothing architecture-specific about these 2 methods.

@thomascalvert-xlnx in general, we are conservative about adding types and primitives to core.p4 as they must be implemented by every architecture (and therefore every compiler backend). So for assert and assume we first added them to V1Model. The idea was always that if they were useful, and there was a push to use them in more architectures, then we would consider them as candidates for addition to core.p4 instead. Maybe now is the time to have that discussion.