click to expect the hover display information to be dynamically updated without refreshing the hover.

As shown in the picture, click to expect the hover display information to be dynamically updated without refreshing the hover.

Hi, thank you for writing this issue. Could you please include a bigger screenshot, and even preferably a video screen recording of the issue? I am not quite sure I understand where the issue is arising.

When the hover content is updated, hover again and flashing occurs.
https://github.com/microsoft/vscode/assets/22573382/423b5a14-741a-4b9f-939a-4aa3ea5c3a5a

Thank you very much for your reply😁

I see thank you for the update

It seems to be the same issue as #195394. After the official version is released, I will verify it.

This API will have to be adopted by the extension authors for the bug to be fixed.

Ok I'll try it in my own extension

Yes when it comes out, the API has not come out yet. You can track the state of that work on the issue you linked.

🫡，谢啦！

Verbosity Hover 能够实现我的需求，但在 automatic completion 中就失效了，其实，editor.action.showHover 就能满足，但在 hover Widget 中，这个命令并不会重新渲染 hover
能否让hover中命令文本：editor.action.showHover，能触发重新渲染hover

Hi @OceanBelongsToMe apologies I do not speak Mandarin. Can you please explain your comment in English?

Verbosity Hover can meet my needs, but it fails in Completion Hover. In fact, editor.action.showHover can satisfy it, but in hover Widget, this command will not re-render hover Widget.
Can the command text in hover: editor.action.showHover trigger re-rendering of hover?

Hi @OceanBelongsToMe I believe the current command already re-renders the hover when it is invoked.
In the example you gave in the screen recording above, how do you update the hover contents (what is the code you use to update the hover contents?)?
A potential feature request would be to update the contents of the hover without hiding the hover before and creating a new one.

For example, the following markdown text is displayed through hover. Clicking the command "editor.action.showHover" will send a hover request to lsp, but after receiving the new hover, it will not be rendered.
## 测试

- [editor.action.showHover](command:editor.action.showHover)

Hi @OceanBelongsToMe I believe the current command already re-renders the hover when it is invoked.
In the example you gave in the screen recording above, how do you update the hover contents (what is the code you use to update the hover contents?)?
A potential feature request would be to update the contents of the hover without hiding the hover before and creating a new one.

I have added a custom command in the hover: 'command:command:my.action.showNewHover?xxxxx'. This command carries some parameters and will trigger another hover event (sending a hover request to the LSP and updating the hover content).
However, the built-in event 'editor.action.showHover' does not re-render the hover after obtaining the new hover content.


added listings and individual listing

if merged, this pull request will display multiple listings on the home page, and a component for showing an individual listing.

@KeakOne this is reviewed. I like how you've moved to getting this rendering as quickly as possible.
The refactor I'm suggesting is mainly suggested to make it easy for others to see how these new components will be used


Docker images built from master

What would you like to be added:
Docker images on dockerhub built from master branch.
Why is this needed:
Testing and giving feedback on fixes, features before proper release.

Thanks @boniek83 for the feedback.
I have automated the process.
The current pre-release release-0.1.7-rc1


Sending DDS message over specific network interface

Hardware description: Jetson Nano
OS: We are running the foxy docker image for Jetson
Installation type: cmake
Version or commit hash: v2.3.0

We are running the Agent on the Jetson Nano communicating via a router to a Nuttx device (PX4). We can see that the Agent is receiving heartbeat messages from the Nuttx device, and receiving messages from the ROS2 nodes running on the Jetson. However it is sending the messages it receives from the ROS2 Nodes on the loopback interface and not the physical network interface.
Is there a way to explicitly tell the Agent to use a specific network interface?
Thanks

Is there a way to explicitly tell the Agent to use a specific network interface?

To accomplish this, you can configure the transport of the Fast dds entities of the Agent.
This can be done with a XML profile, take a look at this documentation: Link.
If support on this part is needed, feel free to open an issue at FastDDS  repo.

Closing as a solution was provided


Update

Description

Update master
Motivation and Context

How Has This Been Tested?

none
Markup:

// Paste your FULL Playground.vue here

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Improvement/refactoring (non-breaking change that doesn't add any feature but make things better)

Checklist:

[x] The PR title is no longer than 64 characters.
[x] The PR is submitted to the correct branch (master for bug fixes, dev for new features and breaking changes).
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have created a PR in the documentation with the necessary changes.

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@a6a878c). Click here to learn what that means.
The diff coverage is 90.57%.

@@          Coverage Diff           @@
##             dev       #1   +/-   ##
======================================
  Coverage       ?   90.13%
======================================
  Files          ?      206
  Lines          ?     4927
  Branches       ?     1255
======================================
  Hits           ?     4441
  Misses         ?      389
  Partials       ?       97

Impacted Files
Coverage Δ

src/components/VList/VListTileAction.js
100% <ø> (ø)

src/components/VDataTable/mixins/progress.js
100% <ø> (ø)

src/components/VExpansionPanel/index.ts
100% <ø> (ø)

src/components/VTabs/mixins/tabs-generators.js
100% <ø> (ø)

src/components/VToolbar/VToolbarSideIcon.js
100% <ø> (ø)

src/components/VGrid/grid.js
100% <ø> (ø)

src/mixins/stackable.js
87.5% <ø> (ø)

src/components/VSubheader/VSubheader.js
100% <ø> (ø)

src/components/VList/VListTileAvatar.js
100% <ø> (ø)

src/components/VTabs/mixins/tabs-touch.js
75% <ø> (ø)

... and 171 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 a6a878c...678f54a. Read the comment docs.


Retry image downloads

Sometimes our CDN (CloudFront/S3) serves up 500s when downloading container images. We should retry a few times before giving up if the error is not in the 4xx range.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


Synapse workspace

Description

If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Enable GitHub Worksflows in your fork to enable auto-generation of assets with our GitHub Action.
To trigger GitHub Actions after auto-generation, add a GitHub PAT as a secret in your forked repository called PAT.
Adding a new module

[ ] A proposal has been submitted and approved.
[ ] I have included "Closes #{module_proposal_issue_number}" in the PR description.
[ ] I have run brm validate locally to verify the module files.
[ ] I have run deployment tests locally to ensure the module is deployable.

Updating an existing module

[ ] This is a bug fix:

[ ] Someone has opened a bug report issue, and I have included "Closes #{bug_report_issue_number}" in the PR description.
[ ] The bug was found by the module author, and no one has opened an issue to report it yet.

[ ] I have run brm validate locally to verify the module files.
[ ] I have run deployment tests locally to ensure the module is deployable.
[ ] I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:

[ ] The PR contains backwards compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json.
[ ] The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.
[ ] The PR contains breaking changes, and I have bumped the MAJOR version in version.json.

[ ] I have updated the examples in README with the latest module version number.

no merge


[.NET C#][Input.ChoiceSet] [adaptive card dropdown title with double quotes rendering issue]

Copied bug from: https://github.com/microsoft/AdaptiveCards/issues/3573
Details from original post:
Platform
What platform is your issue or question related to? (Delete other platforms).

.NET C#

Dropdown rendering title value containing double quotes is getting replaced with "&quot"
Are you an author (like sending something to Outlook), or a host that is rendering your own cards? If host, please specify the corresponding host (app) name. : Microsoft Teams
1.2.0
Details
We are having dropdown in adaptive card which is having double quotes in title. double quotes on teams channel are getting replaced with "&quot"
Please find example JSON below:
DropdownExampleJson.txt
Please suggest a solution.

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

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @surbhigupta12
Microsoft Alias: surbhigupta12

We will check on it and let you know.

I am unable to repro this issue,

It is showing fine to me.

Hello, Could you please share more details on this?

We are unable to repro this issue, we have tested it on web as well as desktop client version - <IP_ADDRESS>569 (64-bit).
Please let us know if you are still facing this issue. We are closing it as of now.


Get rid of LNT_DB_DIR

We have set up an Azure Table Storage to store LNT benchmark data. So we no
longer need to save data on the local LNT database.

ADO validation: https://dev.azure.com/msresearch/CheckedC/_build/results?buildId=492078&view=results


Adding sep to TextConfig

I have a local a .txt file that follows the CONLL2003 format which I need to load using load_script. However, by using sample_by='line', one can only split the dataset into lines without splitting each line into columns. Would it be reasonable to add a sep argument in combination with sample_by='paragraph' to parse a paragraph into an array for each column ? If so, I am happy to contribute!
 ]

Hi @omar-araboghli, thanks for your proposal.
Have you tried to use "csv" loader instead of "text"? That already has a sep argument.

Hi @albertvillanova, thanks for the quick response!
Indeed, I have been trying to use csv instead of text. However I am still not able to define range of rows as one sequence, that is achievable with passing sample_by='paragraph' to the TextConfig
For instance, the below code
import datasets

dataset = datasets.load_dataset(
    path='csv',
    data_files={'train': TRAINING_SET_PATH},
    sep='\t',
    header=None,
    column_names=['tokens', 'pos_tags', 'chunk_tags', 'ner_tags']
)

leads to
dataset
>>> DatasetDict({
    train: Dataset({
        features: ['tokens', 'pos_tags', 'chunk_tags', 'ner_tags'],
        num_rows: 62543
    })
})

dataset['train'][0]
>>> {'tokens': 'Distribution',
 'pos_tags': 'NN',
 'chunk_tags': 'O',
 'ner_tags': 'O'
}

Is there a way to deal with multiple csv rows as one dataset instance, where each column is a sequence of those rows ?


The target domain for pre-training stage

Hi,
Thanks for your great work! I see DATA_CONFIG_TAR in yaml file and I wonder if the pre-training stage use any information in target domain?

Hello, those configs are only used in evaluation stage, since we want to evaluate the model with the same config.

Thanks for your responce !


Preparing metadata (pyproject.toml) did not run successfully

Cannot install patchview with pip, preparing metadata (pyproject.toml) did not run successfully.

OS Ubuntu 22.04
conda 22.9.0

Following instructions for installation via pip after activation of conda environment and installation of not-PIP dependencies (NeuroM, dictdiffer, pyemf3) the command python -m pip install patchview both for patchview versions 0.3.0 and 0.2.9 returns the next output:
Collecting patchview
  Using cached Patchview-0.3.0.tar.gz (9.4 MB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Installing backend dependencies ... done
  Preparing metadata (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Preparing metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [109 lines of output]
      /tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/config/_apply_pyprojecttoml.py:76: _MissingDynamic: `readme` defined outside of `pyproject.toml` is ignored.
      !!

              ********************************************************************************
              The following seems to be defined outside of `pyproject.toml`:

              `readme = "===============\nPatchView\n===============\n.. image:: https://img.shields.io/pypi/v/patchview.svg \n        :target: https://pypi.python.org/pypi/patchview\n      \n.. image:: https://img.shields.io/badge/python-3.8%2B-blue\n        :target: https://www.python.org/downloads/release/python\n        :alt: Python3.8\n\n.. image:: https://readthedocs.org/projects/patchview/badge/?version=latest\n        :target: https://patchview.readthedocs.io/en/latest/?badge=latest\n        :alt: Documentation Status\n\n.. image:: https://img.shields.io/badge/code%20style-black-000000.svg\n    :target: https://github.com/psf/black\n\n.. image:: https://img.shields.io/badge/License-BSD%203--Clause-blue.svg\n        :target: https://opensource.org/licenses/BSD-3-Clause\n        :alt: BSD-3-Clause    \n\n.. image:: https://joss.theoj.org/papers/10.21105/joss.04706/status.svg\n   :target: https://doi.org/10.21105/joss.04706\n.. image:: https://img.shields.io/pypi/dm/patchview?label=pypi%20downloads\n           \n\n.. image:: docs/resources/images/patchview_ads.png\n    :width: 800\n\nPatchView perform data analysis and visualization on multi channel whole-cell recording (multi-patch) data, including firing pattern analysis, event analysis,\nsynaptic connection detection, morphological analysis and more.\n\n* Free software: BSD 3-Clause license\n* Documentation: https://patchview.readthedocs.io.\n\n\nFeatures\n--------\nPatchView integrates multiple open-source tools (see credit page) and wrap them using an intuitive graphic user interface (GUI).\nThus users can perform most analysis quickly for the data collected in a typical patch-clamp experiment without installing Python and \nthese tools or writing any Python scripts.\n\n* Importing both Heka data and Axon Instruments data. Exporting to Python pickle file or NWB (Neurodata Without Borders) file format.\n* Visualizing single and multiple traces with zoom, pan operations.\n* Automatically sorting experiments data according to predefined labels.\n* Performing analysis on intrinsic membrane properties, action potential detection, firing pattern analysis.\n* Synaptic connection analysis.\n* Visualizing and quantification of neuron's morphological reconstruction from Neurolucida\n\n\nCitation\n---------\nIf you find our work useful for your research, please cite:\n\n    Hu et al., (2022). PatchView: A Python Package for Patch-clamp Data Analysis and Visualization. Journal of Open Source Software, 7(78), 4706, https://doi.org/10.21105/joss.04706\n"`

              According to the spec (see the link below), however, setuptools CANNOT
              consider this value unless `readme` is listed as `dynamic`.

              https://packaging.python.org/en/latest/specifications/pyproject-toml/#declaring-project-metadata-the-project-table

              To prevent this problem, you can list `readme` under `dynamic` or alternatively
              remove the `[project]` table from your file and rely entirely on other means of
              configuration.
              ********************************************************************************

      !!
        _handle_missing_dynamic(dist, project_table)
      /tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/config/_apply_pyprojecttoml.py:76: _MissingDynamic: `license` defined outside of `pyproject.toml` is ignored.
      !!

              ********************************************************************************
              The following seems to be defined outside of `pyproject.toml`:

              `license = 'BSD-3-Clause'`

              According to the spec (see the link below), however, setuptools CANNOT
              consider this value unless `license` is listed as `dynamic`.

              https://packaging.python.org/en/latest/specifications/pyproject-toml/#declaring-project-metadata-the-project-table

              To prevent this problem, you can list `license` under `dynamic` or alternatively
              remove the `[project]` table from your file and rely entirely on other means of
              configuration.
              ********************************************************************************

      !!
        _handle_missing_dynamic(dist, project_table)
      /tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/config/_apply_pyprojecttoml.py:76: _MissingDynamic: `dependencies` defined outside of `pyproject.toml` is ignored.
      !!

              ********************************************************************************
              The following seems to be defined outside of `pyproject.toml`:

              `dependencies = ['furo', 'morphopy', 'colorcet', 'appdirs', 'Pillow', 'pynwb', 'tqdm', 'PySide2', 'pyqt5', 'pyqtwebengine', 'pyqt5-sip', 'pyqtgraph==0.13.1', 'PyOpenGL', 'PyOpenGL_accelerate>=3.1.6; sys_platform == "win32"', 'cython', 'neo', 'pylru==1.2.1', 'h5py==3.7.0', 'joblib', 'future>=0.18.3', 'pyYAML', 'python-dateutil', 'seaborn', 'pytz', 'scipy', 'pandas', 'matplotlib', 'pycairo', 'scikit-image', 'morphio', 'scikit-learn', 'numpy<1.24,>=1.22']`

              According to the spec (see the link below), however, setuptools CANNOT
              consider this value unless `dependencies` is listed as `dynamic`.

              https://packaging.python.org/en/latest/specifications/pyproject-toml/#declaring-project-metadata-the-project-table

              To prevent this problem, you can list `dependencies` under `dynamic` or alternatively
              remove the `[project]` table from your file and rely entirely on other means of
              configuration.
              ********************************************************************************

      !!
        _handle_missing_dynamic(dist, project_table)
      /tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/config/_apply_pyprojecttoml.py:76: _MissingDynamic: `optional-dependencies` defined outside of `pyproject.toml` is ignored.
      !!

              ********************************************************************************
              The following seems to be defined outside of `pyproject.toml`:

              `optional-dependencies = {'dev': []}`

              According to the spec (see the link below), however, setuptools CANNOT
              consider this value unless `optional-dependencies` is listed as `dynamic`.

              https://packaging.python.org/en/latest/specifications/pyproject-toml/#declaring-project-metadata-the-project-table

              To prevent this problem, you can list `optional-dependencies` under `dynamic` or alternatively
              remove the `[project]` table from your file and rely entirely on other means of
              configuration.
              ********************************************************************************

      !!
        _handle_missing_dynamic(dist, project_table)
      Traceback (most recent call last):
        File "/home/wisstock/anaconda3/envs/patch/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/home/wisstock/anaconda3/envs/patch/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
        File "/home/wisstock/anaconda3/envs/patch/lib/python3.10/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 149, in prepare_metadata_for_build_wheel
          return hook(metadata_directory, config_settings)
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 366, in prepare_metadata_for_build_wheel
          self.run_setup()
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 480, in run_setup
          super().run_setup(setup_script=setup_script)
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 30, in <module>
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/__init__.py", line 103, in setup
          return distutils.core.setup(**attrs)
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/_distutils/core.py", line 159, in setup
          dist.parse_config_files()
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/dist.py", line 627, in parse_config_files
          pyprojecttoml.apply_configuration(self, filename, ignore_option_errors)
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/config/pyprojecttoml.py", line 68, in apply_configuration
          return _apply(dist, config, filepath)
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/config/_apply_pyprojecttoml.py", line 57, in apply
          _apply_project_table(dist, config, root_dir)
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/config/_apply_pyprojecttoml.py", line 83, in _apply_project_table
          corresp(dist, value, root_dir)
        File "/tmp/pip-build-env-viv16bcl/overlay/lib/python3.10/site-packages/setuptools/config/_apply_pyprojecttoml.py", line 164, in _long_description
          file = val.get("file") or []
      AttributeError: 'NoneType' object has no attribute 'get'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Hi @wisstock
Thanks for reporting the issue.
The problem is  the pyproject.toml file (which is not needed any more).
Please use the following command to install the patched version.
pip install patchview --no-cache-dir
Hope that solves the problem.

Hi @ZeitgeberH
the installation of v0.3.0 was successful and everything is working great.
Thank you!


remove exception swallowing

Remove the exception swallowing around plugins during the --gen step. Now any problems a plugin encounters will immediately be propagated, and will terminate the pipeline if appropriate. This will allow users to identify problems sooner.

@afaulconbridge we did this for some specific reason which I cannot remember now.


@Value is not experimental, but it implies @FieldDefaults which is?

I have a class with a package-private static field which must remain package-private.
Then I annotate my class with @Value, and Lombok turns my package-private static field into a private static field, so everything breaks.
So, I have to use @Data instead, which is not what I really wanted.
I suppose, the problem can be worded as follows: @Value is not marked as "experimental", but it implies @FieldDefaults, which is experimental, so in light of that, shouldn't @Value also be marked as "experimental"?  (Either that, or can @Value please not mess with my static fields?)

A workaround would be to add @Getter(access=PACKAGE) and/or  @Setter(access=PACKAGE) to the static field. Surely better than using @Data. You could also use all needed parts of @Value separately.
Sure, that's just a workaround (I'm not a project maintainer, just trying to help).

First of all, it does not imply @FieldDefaults, but is equivalent to it.
We still think that @FieldDefaults could be experimental as a stand-alone feature. However, @NonFinal and @PackagePrivate should not be in experimental. Also, I think it is a bug if it touches static fields.

I've created issue #1254 to skip static fields.


current function signature for action is invalid

The current function signature for action is
action(p::Policy, state::State, action=create_action(pomdp))

This is not valid because pomdp, the argument to create_actions, is not one of the arguments of action.
How should we fix this? If a policy is required to be able to specify an action given the state, then we a create_action(::Policy) method makes sense. I think that's the best option. What do y'all think?

I think actions are a property of the POMDP. So, it seems to make sense to keep create_action(pomdp). Maybe just keep it
action(p::Policy, state::State)
action(p::Policy, state::State, action::Action)

and let the implementer of the policy handle it---probably by accessing a reference to the POMDP object it squirreled away in the policy object.

Yeah, that seems like a good way to do it.

Agreed.

Any objections? If not, I'll make the change.
On Sun, Dec 13, 2015 at 3:59 PM ebalaban<EMAIL_ADDRESS>wrote:

Agreed.
—
Reply to this email directly or view it on GitHub
https://github.com/sisl/POMDPs.jl/issues/44#issuecomment-164312710.

Sounds good to me.


Create JSON files for each data

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/ngneat/falso/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Data is present as array literals.
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Hey @shhdharmen, I noticed some functions are still including raw data rather than their related json file (example: https://github.com/ngneat/falso/blob/main/packages/falso/src/lib/catch-phrase-noun.ts).
I'll fix it as part as #3.

@geromegrignon sure.

@shaharkazaz already fixed everthing


fix parition table isn't shown in show command

What problem does this PR solve?
This PR closes #1330
What is changed and how it works?
When we process ShowCommand, we first check the emptiness of partitionSpec. We append partition info if such spec is not empty.
Check List
Tests
tests will be added later.

can you add a test?

@birdstorm Yes, I was figuring out how to add a test. :(

@birdstorm Yes, I was figuring out how to add a test. :(

You may refer to  https://github.com/pingcap/tispark/blob/0f5135c/core/src/test/scala/org/apache/spark/sql/catalyst/catalog/CatalogTestSuite.scala#L61-L108

need cherry-pick?

need cherry-pick?

Yes, this PR need cherry pick.


bump apt dependency in metadata + fixtures

4.0.0 and 3.0.0 got released a few days ago. We need to bump it,
otherwise PMT is unable to resolve dependencies in certain situations.

also related: https://github.com/puppetlabs/puppetlabs-postgresql/pull/867

this PR causes jenkins to fail https://jenkins-master-prod-1.delivery.puppetlabs.net/job/forge-module_puppetlabs-postgresql_unit-module_adhoc/4/ presumably because of the difference in parallel_spec arrangement coupled with the apt fixture change.
travis 1 spec per process
jenkins 5 specs per process

thanks @bastelfreak !


Emmet abbreviations in 1.15.1

Hi! Thanks for this plugin. Seems bug related to #3 and #4.
In postcss language and VS CODE 1.15.1 color square and autocomplete working good, but not emmet abbreviations :(
In usual css/less/sass language all fine. Can we fix that somehow?

Hello, try adding "emmet.includeLanguages": { "postcss": "css" } to your settings in vscode and let me know if it has worked out for you.

unfortunately doesnt help :(
In your environment emmet working normal with postcsss syntax?

Hm, after second redactor restart its working, thank you!)


After recent update, MAA checking adb devices response

MAA log (before fix PlayBridge)
[2024-09-23 00:20:50.274][TRC][Px714c][Tx99b4] asst::AdbController::AdbController | leave, 0 ms
[2024-09-23 00:20:50.274][TRC][Px714c][Tx99b4] asst::AdbController::connect | enter
[2024-09-23 00:20:50.358][INF][Px714c][Tx99b4] Call ` ~~~\PlayBridge.exe devices ` ret 0 , cost 84 ms , stdout size: 0 , socket size: 0
[2024-09-23 00:20:50.359][INF][Px714c][Tx99b4] Assistant::append_callback | ConnectionInfo {"details":{"adb":"~~~\\PlayBridge.exe","address":"DummyDeviceAddress","config":"General"},"uuid":"","what":"ConnectFailed","why":"Address does not contain ':' and no devices found"}

MAA trying to find device what user entered, from adb devices response.
MAA log (after fix PlayBridge)
[2024-09-23 00:35:58.298][TRC][Px714c][Txe12c] asst::AdbController::AdbController | leave, 0 ms
[2024-09-23 00:35:58.298][TRC][Px714c][Txe12c] asst::AdbController::connect | enter
[2024-09-23 00:35:58.383][INF][Px714c][Txe12c] Call ` ~~~\PlayBridge.exe devices ` ret 0 , cost 84 ms , stdout size: 52 , socket size: 0
[2024-09-23 00:35:58.383][TRC][Px714c][Txe12c] stdout output:
List of devices attached
GooglePlayGamesBeta	device

[2024-09-23 00:35:58.425][INF][Px714c][Txe12c] Call ` ~~~\PlayBridge.exe connect GooglePlayGamesBeta ` ret 0 , cost 41 ms , stdout size: 36 , socket size: 0
[2024-09-23 00:35:58.425][TRC][Px714c][Txe12c] stdout output:
connected to Google Play Games Beta

So

User should set MAA's emulator device name to GooglePlayGamesBeta.

Thanks for the work.
Btw, which version of MAA are you using?
I've tested with the stable 5.6.2 version and haven't encountered the issue.

I'm using 5.6.2 also. Then, seems device name checking is not for every players.


Add validation for http header

Thanks for goss!  This adds a validation for for http header "name: value" pairs as below.
http:
  https://www.google.com:
    allow-insecure: false
    no-follow-redirects: false
    timeout: 3000
    status: 200
    header: ["Content-Type: text/html"]
    body: ["google"]

The Travis build failed due to the extra http header attribute, but maybe that's expected when adding a new test attribute.

Thanks, will review and update.

@aelsabbahy I found the missing test expectations and adjusted.

I've adjusted with some more updates.  I'm making some newbie mistakes with the go project.  It all makes sense when I relate concepts to Java, Python, JS but in this new ecosystem I don't know what I don't know.

This is almost ready for merging, just need a test in goss-shared.yaml showing that this works.

I'm working through some problems trying to enable the test for the http header validation.  On a Raspberry Pi / debian linux it's working fine but I can't correlate the error from travis-ci build to what's really causing it.  I'll try to reproduce the error locally.

This would be a very handy feature. Has there been any further progress on getting the test case finished?

I tested building on Raspberry Pi and it works for its executable.  I tested building on Ubuntu on Windows and that works for the amd64 executable there.  But something still fails on the CentOS7 test on the Travis integration server.  I'm trying a local VirtualBox with CentsOS7 but I'm having trouble getting networking to work.

I've tested locally successfully on CentOS7, Ubuntu, R-Pi and rebased on the latest master but I still get a build error from Travis CI, maybe when it's comparing [[ centos7 == \a\r\c\h ]].
@aelsabbahy , @berney , have you seen this before?  Is this something I can fix or is it elsewhere?
It would be good to finish this off.
Using default tag: latest
latest: Pulling from aelsabbahy/goss_centos7
++docker_exec /goss/centos7/goss-linux-amd64 --vars /goss/vars.yaml -g /goss/centos7/goss.yaml validate
++docker exec goss_int_test_centos7_amd64 /goss/centos7/goss-linux-amd64 --vars /goss/vars.yaml -g /goss/centos7/goss.yaml validate
...
Count: 84, Failed: 0, Skipped: 0
+[[ centos7 == \a\r\c\h ]]
+egrep -q 'Count: 83, Failed: 0'
+rv=1
+docker rm -vf 7fcd348d3633254b8ce7ff49886c44aa7c08234ab5ddc66367340dff2fa82c76
7fcd348d3633254b8ce7ff49886c44aa7c08234ab5ddc66367340dff2fa82c76
+exit 1
make: *** [centos7] Error 1
The command "make" exited with 2.
Done. Your build exited with 1.

I'm travelling at the moment, I will try mid next week if the issue is still outstanding.

@berney I could pick this up if you want?? I feel like I discussed HTTP header validation a long time ago but can't remember.
I think it would be a useful addition. I'd especially like it to be able to validate that a header doesn't exist.
For example, I might want to validate that the no-cache header doesn't exist on my high traffic heavily cached website.

@timbirk I think you should pick it up 🙃

@kmashint I think the build 874 https://travis-ci.org/aelsabbahy/goss/builds/366580489 error:
Count: 85, Failed: 0, Skipped: 0
+[[ centos7 == \a\r\c\h ]]
+egrep -q 'Count: 83, Failed: 0'
+rv=1

Is because you have added 2 more tests with header: []. You also added more tests in goss-shared.yml, so I would have thought the number will be even higher. I changed the egrep to expect 85 and the tests are passing.
I have tested the code in Ubuntu on WSL against a website checking for headers and it is working correctly.
LGTM

I've merged from master and update the expected test counts in test.sh but it's now giving an error with archlinux:
manifest for base/archlinux:latest not found

base/archlinux is deprecated use archlinux/base. See https://hub.docker.com/r/base/archlinux/.
$ docker search archlinux
NAME                                  DESCRIPTION                                     STARS               OFFICIAL            AUTOMATED
base/archlinux                        Deprecated repository, use archlinux/base in���   331                                     [OK]

I don't know why but they seemed to have used base as the owner/project name rather than archlinux. I'm not even sure base is the archlinux project (e.g. from the maintainers or not).

@berney I've been trying to debug more strange cloud build errors, this time with DNS tests.  They run locally but for some reason fail in the Travis cloud build.  Any ideas?  I'd like to finally finish this!
Why does this fail?  Tried timeouts but that didn't help.  It works on CentOS 7.4.1708 in VirtualBox.
Using dnslookup and set type=txt with txt._test.dnstest.io does give "Hello DNS" but not as an address per se.
DNS: TXT:txt._test.dnstest.io: resolvable:
Expected
    <bool>: false
to equal
    <bool>: true
DNS: TXT:txt._test.dnstest.io: addrs: skipped
Total Duration: 1.000s
Count: 90, Failed: 1, Skipped: 1'

I did a complete rebuild of my CentOS7 VM and now I'm getting similar errors, but they are even from master, not just my branch, so it's a side effect of underlying platform updates.

@berney Phew!  Finally all checks passed!

@aelsabbahy This is up to date again and passing tests after merging away the problematic DNS tests.

@aelsabbahy  @berney  This merged the latest changes again.

Awesome, I'll take a look at it this week and merge it.
Many thanks for this, great enhancement!

@aelsabbahy @berney Any luck with merging this?  It would be good to start using it for some of my work projects where I'm currently falling back to bash+curl to validate http headers.

@aelsabbahy @berney Just pinging again for merge. It would be good to start using it for some of my work projects where I'm currently falling back to bash+curl to validate http headers.

Github rebasing of PRs is a bit confusing.. Not sure the proper way to do this.. But I ended up just doing a rebase locally and pushing it up as a different PR.
It looks like all the author information is preserved, I want to make sure you're credited for the changes.
Thanks a bunch for this!


[Bug] - Reminder Thread fails during run

@LongLeCE the reminder thread still fails with this stack trace, maybe for this upcoming milestone we implement other features first before fixing this?

As discussed, the thread issue occurs when the terminate command is input by the user

Fixed in pr #78

Looks good, will close this issue!


Nmpolicy main flow skeleton

This change introduces the main flow of nmpolicy, focusing on the captures resolution.
It is still considered WIP as the tests have not been added.
It is posted to allow preliminary review.

change: Rebase


Fix handling dependencies

Describe the bug or question
In the docs, we have something like
router = crud_router(
    ...
    read_deps=[get_current_user],
    update_deps=[get_current_user],
    ...
)

When actually the correct way would be:
from fastapi import Depends

router = crud_router(
    ...
    read_deps=[Depends(get_current_user)],
    update_deps=[Depends(get_current_user)],
    ...
)

I'd like some opinion if possible on whether we should fix the docs or change the way it works. The latter is more explicit, but I think the former might be simpler for the end user.
If the former is used, stuff like getting non-callable objects or dependencies that are already wrapped must be handled.

Both approaches are acceptable in my opinion, but I would personally prefer the former (without Depends). Let me just define what dependency is required and make the library handle it as it needs. That's also less verbose.

Also note that in the newest version of tiangolo fastapi template it's not required to wrap the deps in Depends. He leverages Dependency annotations  instead. Maybe we could support both cases, by conditionally wraping deps when needed?

I'll change stuff and it should be released over the weekend, there are a few other changes and I need to write the documentation for them as well.

Also note that in the newest version of tiangolo fastapi template it's not required to wrap the deps with Depends in route parameters.

Yeah, this would definitely be a more elegant approach, but it's still more work to the user.

Maybe we could support both cases, by conditionally wraping deps when needed?

I think we should pick something and stick to it, support it in the best possible way. One flexibility we could allow is:
We get a sequence of functions, each function might have a type hint, if it does, we use the dependency annotation with the custom type, if not, we do it with Callable.


ci: update self to 2.1.0

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

v2.1.1
GitHub release

Your semantic-release bot :package::rocket:


Script Monitoring

Script Monitoring should include a section that outlines how long script data is retained in the database and visible in the console.

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

ID: 7c7d1db7-6812-ff3c-9aac-0f55800ed665
Version Independent ID: a4499cf7-ed5a-710c-66e0-005563b9c015
Content: Create and run scripts - Configuration Manager
Content Source: sccm/apps/deploy-use/create-deploy-scripts.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @sympleconfigmgr for the suggestion. We have some work planned on this article in the near future, so will try to include this information at that time.

Information about how long we retain the data has been added under the Script Monitoring section. The changes should be published in the next day or so.  We keep the data until either the script is deleted or until the Delete Aged Client Operations maintenance task is run.
#please-close


AL0204 and AL0118 raised on using the new option access syntax

Please include the following with each issue:
1. Describe the bug
Errors raised on using the new option access syntax.
2. To Reproduce
Steps to reproduce the behavior:

In the app.json, ensure you work with "runtime": "10.0"

Add the following object

page 50000 "CCS PDM CAD Item Card"
{
    ApplicationArea = All;
    Caption = 'New Item Card';
    PageType = Card;
    RefreshOnActivate = true;
    SourceTable = Item;
    UsageCategory = None;

    layout
    {
        area(Content)
        {
            group(Item)
            {
                Caption = 'Item';
                field("No."; Rec."No.")
                {
                }
            }
        }
    }

    actions
    {
        area(Navigation)
        {
            group(Navigation_Item)
            {
                Caption = 'Item';
                action(Dimensions)
                {
                    Caption = 'Dimensions';
                    Image = Dimensions;
                    RunObject = page "Default Dimensions";
                    RunPageLink = "Table ID" = const(27),
                                  "No." = field("No.");
                    ShortcutKey = 'Alt+D';
                    ToolTip = 'View or edit dimensions, such as area, project, or department, that you can assign to sales and purchase documents to distribute costs and analyze transaction history.';
                }
	    }
         }
    }
}

Change the action's RunPageLink to the new option access syntax using IntelliSense:
RunPageLink = "Table ID" = const(Database::Item),

Now, the following errors are thrown:
On "Table ID": Field type Integer is not convertible to field type. AL0204
On Database: The name 'Database' does not exist in the current context. AL0118
3. Expected behavior
Either no errors AL0204 and AL0118, or IntelliSense should not support using const(Database::Item).
4. Actual behavior
IntelliSense supported, but errors AL0204 and AL0118 thrown.
Note: The new syntax does work on other occurrences, such as when turning
SubPageLink = "Table ID" = const(27)
into
SubPageLink = "Table ID" = const(Database::Item)
5. Versions:

AL Language:  v10.0.687650
Visual Studio Code: 1.72.2
Business Central:  21

It's a duplicate of #7205

That's right. I did not find it myself because the warning IDs are not mentioned there.


Test suite needs phantomjs installed

PhantomJS is not listed as a requirement, but at least one test fails without it.

Correct, phantomjs is the default selenium driver. You can switch out the default selenium driver by setting the environment variable or Django setting TEST_SELENIUM_DRIVER. Other options are currently "firefox" or "chrome". More details here. Please add documentation where you would expected this information and open a pull request.

I don't think this fully addresses the issue. I applied the first two commands here:
pip install -r requirements-test.txt then make test, and that failed for at least one because phantomJS wasn't installed as part of the requirements. I would have expected it to be, but it doesn't look trivial to automate (dependent on the platform, I am on mac).


Doesn't work in Safari and FF

I test my pages which contain smart-table component and I found that they doesn't display in Safari (last version) and Firefox. One error display in console of the browsers

TypeError: CSS.escape is not a function. (In 'CSS.escape(col)', 'CSS.escape' is undefined)

Thanks! That is a biggie. Should be fixed in 2.4.1

@gurghet Firefox is fixed, thanks! But Safari still has a bug. It highlights this line of code
if (father.addRow === true && (father.addRowCompiled[col] === false || father.addRowCompiled[col] === undefined)) {

I wasn't able to reproduce the problem on Safari 9.1.1, anyways I suspect that the CSS polyfill wasn't picked up for some reason. I forced smart table to use the polyfill even in normal browsers, so it should work in 2.4.2. Let me know.

Yes, works like a charm. Thank you.


Taps not recorded when your finger moves a little

Thanks for taking the time for reporting an issue!
Describe what happened
Include any error message or stack trace if available.
If you tap and move your finger slightly before releasing, it doesn't show up as a tap in RUM.
Steps to reproduce the issue:
Put your finger/cursor on a button, move 5-10px and then release. The UI will recognize this as a button press but it doesn't show up in RUM.
I believe this is because GesturesListener.onUp handles scrolls and swipes but not taps. It looks like the UI behavior is that if the down is on a button and the up is on the same button, it will be recognized as tapping the button.
You can tap down on a button, move off of the button, move back onto the button, and then release, and it will count as tapping the button.
If you tap down on a button, move off of the button, and then release, it will not count as tapping the button.
It is not uncommon that people move their finger slightly when pressing a button. It looks very strange in RUM as if they went to a new screen with no tap.
Describe what you expected:
I expect this to be recognized as a tap if the UI recognizes it as a tap.
Additional context
Android OS version: 14
Device Model: pixel 4 xl
Datadog SDK version: 1.17.1
Versions of any other relevant dependencies (OkHttp, …):
Proguard configuration:
Gradle Plugins:

This is fixed by #1622 , which will be released soon.

Awesome, thank you!
On Tue, Sep 19, 2023, 7:48 AM Xavier F. Gouchet @.***>
wrote:

This is fixed by #1622
https://github.com/DataDog/dd-sdk-android/pull/1622 , which will be
released soon.
—
Reply to this email directly, view it on GitHub
https://github.com/DataDog/dd-sdk-android/issues/1389#issuecomment-1725346649,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2REAS57VM3AT4DBPFAVTX3GBAPANCNFSM6AAAAAAXBJCL64
.
You are receiving this because you authored the thread.Message ID:
@.***>


Suggestion: Yeta Wrench: show TileEntity Names when selecting direction.

It would be awesome, if the GUI to select the direction of the Conduit you want to edit (North, South, West, East, Up, Down) would also show the name of the Block/TileEntity,  that the conduit is connected to in that direction.
For most of the cases where you don't connect the same blocktypes with eachother this would make it easier to figure out which direction you need to edit without needing to look at the map/F3 mode.

Ooh, I like that

Hmm - why not use the side configuration render instead?


ability to navigate forward and backward using XButton1 and XButton2 …

I see it would be a great feature if we can make the navigation between workspaces, using mouse buttons can be more convenient than moving and hovering the mouse to switch tabs.
For easy reference see how that looks

I like the idea, from the beginning, the goal was to provide "an internet browser" like experience, and this is a step in that direction.
But, you have chosen very peculiar places to place the code.
The added code from here TextBoxWithPlaceholder.xaml.cs should be located in MainWindow.xaml.cs, and what you have added to MainWindow.xaml.cs should be placed in 'WorkspacesViewModel.cs'.

Thanks for the feedback! I have made the necessary adjustments, and committed the changes. What you explained is correct and makes more scene.


Enums with new and init method are not properly checked

Issue Type: Bug
I have a following working code, which VSC/Pylance/Pyright does not seem to understand and as a consequence, autocomplete does not work and error squiggles are everywhere.
import enum

class Blocks(enum.Enum):
    ONE = (0, "Blue", "The one on the corner.")
    TWO = (1, "Pink", "The ugly one.")

    def __new__(cls, streetNumber: int, color: str, description: str):
        obj = object.__new__(cls)
        obj._value_ = streetNumber

        obj.streetNumber = streetNumber
        obj.color = color
        obj.description = description

        return obj

def main():
    theUglyBlock = Blocks(1)
    print(theUglyBlock)
    print(theUglyBlock.description)

if __name__ == "__main__":
    main()

Although it does show correct statements on execution:
Blocks.TWO
The ugly one.

... IDE shows:

and:

Now, is this an unhandled scenario in VSC extensions or I don't understand something and it is as expected?
Extension version: 2021.6.2
VS Code version: Code 1.57.1 (507ce72a4466fbb27b715c3722558bb15afa9f48, 2021-06-17T13:28:07.755Z)
OS version: Windows_NT x64 10.0.19041
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4570 CPU @ 3.20GHz (4 x 3193)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.89GB (6.04GB free)

Process Argv
--crash-reporter-id d9d15ca2-a021-4dcd-a140-bb0adc228360

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
pythonvspyt700cf:30270857
pythonvspyt602:30300191
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyt639:30300192
pythontb:30283811
pythonvspyt551cf:30311713
vspre833:30321513
pythonptprofiler:30281270
vshan820:30294714
pythondataviewer:30285071
vscus158:30321503
pythonvsuse255cf:30323309
vscorehov:30309549
vscod805cf:30301675
vscextlang:30322306
binariesv615cf:30323120

This is a duplicate of https://github.com/microsoft/pyright/issues/1751, which we decided not to address.


Read configuration from a .env file (or environment variables)

Fixes #1

Seem to be missing something on here, Please check and advise. :)


Latest release 2.6.1 not available in CF-Community Plugin Repo

Description
2.6.1 is not available for installation for darwin
Your environment
$ cf -v
cf Version 7.2.0+be4a5ce2b.2020-12-10
$ cf install-plugin multiapps
CF-Community nach Plug-in multiapps durchsuchen...
Plug-in multiapps 2.5.1 gefunden in: CF-Community
Plug-in multiapps 2.5.1 ist bereits installiert.
$ cf repo-plugins -r CF-Community |grep multiapps
multiapps                       2.5.1     MultiApps cli plugin performing multitarget application(MTA) operations in Cloud Foundry, such as deploying, removing, viewing etc.

Hi @andre68723 ,
Unfortunately latest released version of multiapps plugin is not merged https://github.com/cloudfoundry/cli-plugin-repo/pull/387 in CF-Community repo yet. This is the reason that you can find only 2.5.1 from CF-Community repo. We are looking now to contact someone who have rights to merge our PR in cli-plugin-repo. Until then you can use version 2.5.1 or you can download binary from here and follow this instruction to install latest release version on your machine.
I'll update you when latest version is available in CF-Community repository.

Hi @andre68723 latest released version 2.6.2 is already released and merged in CF Community repo, so now you can update using cf install multiapps command.


clangd giving false positives for missing includes

I am working on a personal project (https://github.com/UtkarshVerma/dwmblocks-async). I have MissingIncludes set to strict and weirdly, clangd gives me the following diagnostics:

block.h (Line 5):  Included header stdint.h is not used directly (fixes available) clangd (unused-includes) [5, 1]
block.h (Line 25): No header providing "uint8_t" is directly included (fixes available) clangd (missing-includes) [25, 45]

It thinks that <bits/stdint-uintn.h> should be included instead, which is wrong as it's glibc-specific.
The same also happens when I try to use FILE in block.c (Line 83). It suggests including <bits/types/FILE.h>.
I think my LSP setup is fine and here is how my editor invokes it:
clangd --clang-tidy --background-index --header-insertion=iwyu --completion-style=detailed --function-arg-placeholders --enable-config --query-driver=/usr/sbin/arm-none-eabi-gcc,/usr/sbin/aarch64-linux-gnu-gcc --fallback-style=none

Logs
N/A
System information
Output of clangd --version:
$ clangd --version
clangd version 17.0.3 (https://github.com/llvm/llvm-project 888437e1b60011b8a375dd30928ec925b448da57)
Features: linux+grpc
Platform: x86_64-unknown-linux-gnu

Editor/LSP plugin:
neovim/nvim-lspconfig
Operating system:
Arch Linux

Hello, I suggest you to use permanent GitHub link like https://github.com/UtkarshVerma/dwmblocks-async/blob/12d4decdd45015b9b6f1ac50a2bd65c98695f1c3/include/block.h#L5 for example. This would allow the link exactly point to that same version even if the file changed later.
Yes, it looks ugly and long but you can use Markdown syntax to hide it like block.h which being clickable

Thanks for the suggestion, done!


Change Constant Polling to Only On Click

Instead of having constant polling every second, only poll once the submit button is clicked. I think this is a better structure. I will also probably draw on this when implementing the discuss pushing, as there is a "Post" button and the URL is only visible after a second or so when the post is actually made. Let me know what you think.

Approve.

@JeremyTsaii make sure to resolve conflicts!


Correction of parsing using incubator vector support

It looks as the code were unfinished.
Double vector for parsing 8 hexadecimal digits uses two times more bits - 512 bits instead of 256-bit. May be there exist some way to avoid this.
Some troubles occurs with little / big endian, so it should be verified this aspect on other platforms.

Thank you very much for fixing this problem. 🙂
Based on your merge request I have now committed a fix that avoids the 512 bit vector. The commit is d167fc9251f7c2d512469e9aa6d500621eb28c4d


Upgrade version of java

We already have java with version 21 (LTS). Investigate how far we can go with upgrading java version. Focus on LTS versions.

This can be closed as it was solved together with #418 .


Flatten closed_results before assignment

I originally discovered the issue because of the following error messages in one of our tests:
undefined method `conform?' for [(ValidationResult
        (value "Undefiniert")
        (focus <http://example.com/base/rr_20221107093933331>)
        (path <https://www.ica.org/standards/RiC/ontology#type>)
        (shape shacl:RicoRecordResourceWithCategoryShape)
        (resultSeverity shacl:Violation)
        (component shacl:ClosedConstraintComponent)
        (message "closed node has extra property"))
       ]:Array

               next unless results.all?(&:conform?)
                                  ^^^^^
     # /Users/apf/.rbenv/versions/3.1.4/lib/ruby/gems/3.1.0/gems/shacl-0.4.0/lib/shacl/algebra/or.rb:34:in `all?'

I wondered where this array was coming from and did some deep-dives using the debugger. Since both builtin_results and op_results get flattend, I assume it should be the same for closed_results.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I added a test. I developed it against your develop branch where I was able to reproduce the issue.
I admit I do not work in general with RDF, so could be that the example can be simplified in order to trigger the issue.

This is great, thanks so much for your contribution!


⚠ ClamAV has degraded performance

In f478262, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 49 ms

Resolved: ClamAV performance has improved in 97a6dc5.


"E: Unable to locate package podman" ubuntu 19.10 server

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
installation docs
Description
The steps on website for installing podman on ubuntu doesn't work. https://podman.io/getting-started/installation
Steps to reproduce the issue:
I followed all the steps for ubuntu from the docs.
sudo apt update
sudo apt install software-properties-common uidmap
sudo add-apt-repository ppa:projectatomic/ppa
sudo apt update
sudo apt install podman

Describe the results you received:
$ sudo apt -y install podman
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package podman

Describe the results you expected:
I expected to podman be installed.
Additional environment details (AWS, VirtualBox, physical, etc.):
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 19.10
Release:	19.10
Codename:	eoan

Ubuntu 19.10 Server
Baremetal

it worth noting that it the PPA does not have the packages for ubuntu 19.10

I was able to force the PPA by adding a different version PPA to my sources.list
deb http://ppa.launchpad.net/projectatomic/ppa/ubuntu disco main

deb-src http://ppa.launchpad.net/projectatomic/ppa/ubuntu disco main

(I am on eoan)

You probably already know, but just in case, try this:
https://software.opensuse.org//download.html?project=devel%3Akubic%3Alibcontainers%3Astable&package=podman

@rhatdan @matihost
I still have this issue on ubuntu 19.10 in google cloud
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 19.10
Release:        19.10
Codename:       eoan

would you guys please test the documentation on the website

@medyagh you're using the OBS repos and not the ppa, yes?


Importing the plugin directly in webpack doesn't work without an alias

Because the directory is called wavesurfer.js this causes some unwarranted action when trying to import 'videojs-wavesurfer.js' since it implies a JavaScript file. The fix is to add an alias to the wavesurfer.js directory under your webpack conf. Maybe consider changing the directory name by default so this doesn't cause issues in the first place.

The directory is called wavesurfer.js because it's the library name, something we can't change easily (and should be unnecessary, video.js for example also seems to work fine with their directory name). So many issues with webpack, blah.


Could not find a set of Noto fonts to display all missing characters

We are facing the same issue as this : #76093 where the font works perfectly in Android & iOS but when used in flutter web, shows some characters as missing.
Snapshot:

Console log:
Could not find a set of Noto fonts to display all missing characters. Please add a font asset for the missing characters. See: https://flutter.dev/docs/cookbook/design/fonts
The thread advises to use html renderer which actually does solve the issue and all fonts appear correctly. But our app is using Paint widget heavily which is not a great fit for html renderer and hence is not a possible option. Is there any way to resolve the font issue while sticking to canvasKit renderer only ?

@samyakkkk
There's similar open issue describing your case, https://github.com/flutter/flutter/issues/79172 and I advise you to share your findings in that issue for better tracking.
Closing from here as a duplicate.


Set the default port to 6000 since that's what is used in production

Backported to radjabov in commit d3c8171c533c764f19585006afdcff4599c6c7fe.
commit d3c8171c533c764f19585006afdcff4599c6c7fe
Author: Adam Grare<EMAIL_ADDRESS>Date:   Wed May 29 09:53:39 2024 -0400

    Merge pull request #46 from bdunne/default_port_6000

    Set the default port to 6000 since that's what is used in production

    (cherry picked from commit 1f8f13258d0866d23896f76350f4b43b721299eb)


Fix encoding problem with status update with media api.

This has been effectively fixed within the constructor for TwitterEscapingFormHttpMessageConverter. Closing this pull request without merging, but please let me know if the new constructor is missing something or doesn't fix your problem.


reference: Replace EnsureTagged with TagNameOnly

The common use case for this function is to add a default tag if the
reference only has a name. The current behavior only adds the default
tag if there is no tag, which requires most callers to check for a
digest. Change the behavior to only add default tags to name-only
references, and change the name to reflect this. The documentation
already described the new behavior, so it does not need to be changed.
cc @dmcgowan

Current coverage is 51.24% (diff: 0.00%)

Merging #2173 into master will decrease coverage by 10.01%

@@                                                 master      #2173   diff @@
==============================================================================
  Files                                               125        125
  Lines                                             11447      11446      -1
  Methods                                               0          0
  Messages                                              0          0
  Branches                                              0          0
==============================================================================
- Hits                                               7012       5865   -1147
- Misses                                             3548       4835   +1287
+ Partials                                            887        746    -141

Powered by Codecov. Last update 7a09723...245ca46

LGTM


在t-form标签上添加disabled属性，可以禁用整个表单

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

讨论中


Can't verify myself on Facebook

Facebook verification fails for me. Note: I did just set it up today. Does it take time to propogate?
$ keybase id hwkr
▶ INFO Identifying hwkr
✔ public key fingerprint: 8B36 5C76 DE77 F7CE F270 A4B9 6305 64E9 0758 21B8
✔ "hwkr" on github: https://gist.github.com/81f55f3aa5a10a06f7aa671b1a031aea [cached 2016-11-17 10:09:08 PST]
✔ "bnhwkr" on twitter: https://twitter.com/bnhwkr/status/779460944883613696 [cached 2016-11-17 10:09:07 PST]
▶ WARNING | Check status (uid=b07c62a582466aeecf83663704dfa719, seq=17, link=9c7577ea63de78fb80612ededbfd43b90f0e6634e2cde93500fb1a6385cdc868) failed with error: Couldn't find username href. (code=106)
✖ "bnhwkr" on facebook failed: Couldn't find username href. (code=106)

You can find the Facebook verification post here.
my log id: 9b34c408eb35ca4fb4e79d1c

@oconnor663

Did you install via brew?  The brew version might be out-of-date...

Couldn't find username href is the error than our first version of Facebook support ran into for people with the "don't allow search engines to index my profile" Facebook privacy setting turned on. It should be fixed in our latest public release though. I think you can force an update with this command?
/Applications/Keybase.app/Contents/SharedSupport/bin/updater check

Does that fix it?

Fixed it! Thank you!

There's something odd going on with the releases page which may be directly contributing to this issue. It shows 1.0.18 as the latest release, which is also the version Homebrew installs, but the actual downloads attached to that release are for 1.1.45 and 1.1.46. It appears that this is a side-effect of there not being any tags in the repository… which is a serious issue in itself.

@rspeed We don't regularly make releases through GitHub, or encourage people to use Homebrew for Keybase.  The download at https://keybase.io/download will keep itself up to date.

That's the GUI version, which is extremely bloated and doesn't provide a CLI version that follows Unix norms.
According to the Finder info, that's still version 1.0.18.
As I just pointed out, there are updated versions of the CLI version distributed through Github, but they're not being properly tagged.

Whatever is going on, this is a huge mess.


wanted to remove the redundancy from script file

hi , i wanted to remove the redundancy from script.js  file in this project .
i saw there is repetition in script .js file kindly allow to me correct this issue .
i saw there are some project data that are repeating many times which is causing redundancy in the project and confusion .
please kindly assign this issue to me.
this contribution is for gssoc21

Already fixed.


CERT_NOT_YET_VALID

Hi all,
I'm trying to start the chat using the Docker container using Fig, and while trying to run the image, I get the following error:
Eriks-MacBook-Pro:lets-chat $ fig up
Recreating letschat_db_1...
Creating letschat_app_1...
Building app...
# Executing 3 build triggers
Trigger 0, COPY package.json /usr/src/app/
Step 0 : COPY package.json /usr/src/app/
 ---> Using cache
Trigger 1, RUN npm install
Step 0 : RUN npm install
 ---> Running in e0ffb140985e

npm ERR! Linux 3.16.4-tinycore64
npm ERR! argv "node" "/usr/local/bin/npm" "install"
npm ERR! node v0.10.36
npm ERR! npm  v2.5.0

npm ERR! CERT_NOT_YET_VALID
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     <http://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /usr/src/app/npm-debug.log
Service 'app' failed to build: The command [/bin/sh -c npm install] returned a non-zero code: 1

Any ideas?

Ah, never mind, I 'fixed' it by restarting boot2docker, as described here: https://github.com/npm/npm/issues/2902


change document and issues URL

Fixes #2575.
Changes proposed in this pull request:

Pull Request Test Coverage Report for Build 8575

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 60.8%

Totals

Change from base Build 8573:
0.0%

Covered Lines:
8425

Relevant Lines:
13857

💛  - Coveralls

Thank you


TFLite Interpreter crashes with null TfLiteDelegatePtr

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): r2.3 but consistent with latter versions
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
tflite::Interpreter does not check nullity of TfLiteDelegate*.
It would be better to check it and return ErrorCode for null.
https://github.com/tensorflow/tensorflow/blob/595d575c237368c925229c05e933d62808330ca2/tensorflow/lite/core/subgraph.cc#L1486-L1513
Will this change the current api? How?
https://github.com/tensorflow/tensorflow/blob/595d575c237368c925229c05e933d62808330ca2/tensorflow/lite/core/subgraph.cc#L1486
or
https://github.com/tensorflow/tensorflow/blob/595d575c237368c925229c05e933d62808330ca2/tensorflow/lite/interpreter.cc#L385
or (for r2.3 I am using)
https://github.com/tensorflow/tensorflow/blob/r2.3/tensorflow/lite/interpreter.cc#L358
returns TfLiteError for null input.
Who will benefit with this feature?
App developers who are using TensorflowLite SDK would find bugs more quickly.
Thanks!
Any Other info.

Hmm. This should be the behavior, yes. However, I may not get to this very soon, since such behavior will probably only occur if the user is using a non-recommended way of initializing delegates.
Out of curiosity, how did you encounter a seg-fault because of an invalid TfLiteDelegate*? Our language bindings usually initialize delegates in a safe way. For C++, the recommended ways of initializing delegates should avoid this using Create/DeleteDelegate methods (found in individual delegate documentation) OR using std::unique_ptr like here.
Feel free to send a PR :-).

Thanks for your reply 👍
I agree with you about the impact of this change. I'll send a small PR.
Answer: I found the issue when dealing with cross-platform code similar to below pseudo code.
It was hard to find because it works for most of the cases!
(I wrote macros just for giving intuitions here)
...
using TfLiteDelegatePtr = std::unique_ptr<TfLiteDelegate, void (*)(TfLiteDelegate*)>;

TfLiteDelegatePtr AcquireDelegate() {
#if defined(OUTPUT_LINKED_WITH_OPENCL_OR_OPENGL)
  ...
  TfLiteDelegatePtr delegate(TfLiteGpuDelegateV2Create(&opts), TfLiteGpuDelegateV2Delete);
#elif defined(OUTPUT_LINKED_WITH_METAL)
  ...
  TfLiteDelegatePtr delegate(TFLGpuDelegateCreate(&opts), TFLGpuDelegateDelete);
#elif defined(OUTPUT_LINKED_WITH_XNNPACK)
  ...
  TfLiteDelegatePtr delegate(TfLiteXNNPackDelegateCreate(&opts), TfLiteXNNPackDelegateDelete);
#else  // OUTPUT_CANNOT_BE_LINKED_BECAUSE_THE_TARGET_PLATFORM_SUPPORTS_NOTHING
  /* Cannot use any delegate-related symbols that may require other dependency. */
  /* Of course to use `DummyDelegate` with default option may be more elegant way to do this. */
  TfLiteDelegatePtr delegate(nullptr, [](TfLiteDelegate*) {});
#endif
  return delegate;
}

I think this is solved at d22d31b


Feat: Add some Hover effect on the Our team page[cards]

Describe the feature
Add some Hover effect on the Our team page[cards]
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@mdazfar2 I can efficiently add hover effect on cards in contributor page as I have made the cards. So, Please Assign this to me.


[5.9][IRGen] Reject enums with inaccessible tpye metadata in layout string…

… generation
Cherry-picked from: https://github.com/apple/swift/pull/65016
rdar://107679697
Because we are currently handling most enums in layout strings by going through the metadata, we have to ensure that the metadata is accessible from the current module and reject the enum otherwise.

@swift-ci test

@swift-ci test

@swift-ci test

@tbkka The PR to main was reviewed and approved by @aschwaighofer. Would you mind approving this?


fix(UpSet): Update UpSet to not pollute global namespace

The branch https://github.com/jeffbaumes/upset/tree/no-global-vars creates a new concatenated index.js for UpSet that does not use the script loader so does not pollute the global namespace with lots of variables.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #492   +/-   ##
=======================================
  Coverage   34.11%   34.11%
=======================================
  Files          39       39
  Lines        1530     1530
=======================================
  Hits          522      522
  Misses       1008     1008

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 9db9c6f...760b204. Read the comment docs.


[SIP-94] Support star schemas

[SIP] Support star schemas
Motivation
While I am absolutely fine with the decision to base a diagram on a single table/view only, most databases cannot cope with that. A typical view would join 100 tables and have 5000 columns.
Take sales orders as example: For a proper self service BI you would join order header, order line item, three times the customer mater table for soldto-shipto-billto, the material master table, tons of tiny lookup tables with e.g. statusid, statustext etc.
If the joined table has a foreign key defined and the column is not-null, then the database optimizer can ignore that join in case none of the columns are used. Only a few databases are that advanced.
Further more, for many visual clues like slice/dice and filters a distinct list of attributes is needed. What is faster? Finding the three order states in the state text table or a select on the text table itself?
Hence I would suggest to support star schema data models. A chart is based on a fact table. The list of columns shown are all columns of the fact table plus all columns of all tables this fact table has a FK relationship with.
Proposed Change

In addition to tables there are virtual objects FACT_TABLE and DIMENSION_TABLE. These simply point to physical tables. Note: A fact table in one data model can be a dimension table for another. Example: ORDER is a fact table with the measure ORDER_AMOUNT but is a referenced dimension table for the BILLINGS fact.
There is a relationship between facts and dimensions. Only trivial PK based equal conditions are allowed.
Charts can use tables or FACT_TABLEs.

@wernerdaehn
Thank you for your comments. I think stars or snowflake model is OLAP Dataware house concept, for more modern BI prefer to use single-table model, like PBI/Tableau.
What do you think?

@wernerdaehn
Thank you for your comments. I think stars or snowflake model is OLAP Dataware house concept, for more modern BI prefer to use single-table model, like PBI/Tableau.
What do you think?

@zhaoyongjie From a technical side, if all databases would support views with a virtual unlimited number of columns and do a perfect job from the SQL optimizer point of view, then yes. But the opposite is the case. Only Oracle And SAP Hana (Calculation Views) are even close to that and far from perfect.
From a logical point of view, Superset wants to sum up measures and group by attributes. That is a dimensional concept.
Further more, if the tool knows what a fact table is and what a dimension, we have much more options later. For example you might have an ORDER and DELIVERY table, both have a shared dimension CUSTOMER. In a single-table model the customer table with its 500 columns must be defined multiple times whereas with a shared dimension just once. And in the dashboard, when you want to view the order amount, shipped amount and order backlog, a shared dimension allows to filter all measures at once. You know what is shared and how.
Also for the users the selection of columns is much easier when they are grouped. The customer master dimension obviously provides all columns that are customer related, the material all material related fields. In a single table model you would have 300 columns from the one and 300 columns from the other. And if customer can be filtered on sold-to, ship-to and bill-to basis, you would have the 300 customer fields three times.
And finally, while today the semantic layer of Superset is very thin, it will grow. And then specifying the same information multiple times get even more of a burden.
So no, this has nothing to do with OLAP, it is related to being a business intelligence tool.
I see that as something fairly straight forward to implement and with lots of potential initially and later.

@zhaoyongjie From a technical side, if all databases would support views with a virtual unlimited number of columns and do a perfect job from the SQL optimizer point of view, then yes. But the opposite is the case. Only Oracle And SAP Hana (Calculation Views) are even close to that and far from perfect.
From a logical point of view, Superset wants to sum up measures and group by attributes. That is a dimensional concept.
Further more, if the tool knows what a fact table is and what a dimension, we have much more options later. For example you might have an ORDER and DELIVERY table, both have a shared dimension CUSTOMER. In a single-table model the customer table with its 500 columns must be defined multiple times whereas with a shared dimension just once. And in the dashboard, when you want to view the order amount, shipped amount and order backlog, a shared dimension allows to filter all measures at once. You know what is shared and how.
Also for the users the selection of columns is much easier when they are grouped. The customer master dimension obviously provides all columns that are customer related, the material all material related fields. In a single table model you would have 300 columns from the one and 300 columns from the other. And if customer can be filtered on sold-to, ship-to and bill-to basis, you would have the 300 customer fields three times.
And finally, while today the semantic layer of Superset is very thin, it will grow. And then specifying the same information multiple times get even more of a burden.
So no, this has nothing to do with OLAP, it is related to being a business intelligence tool.
I see that as something fairly straight forward to implement and with lots of potential initially and later.

Thanks for the comprehensive explanation @wernerdaehn ! Ping @amitmiran137 , here's a good motivation for adding support for join semantics.

Thanks for the comprehensive explanation @wernerdaehn ! Ping @amitmiran137 , here's a good motivation for adding support for join semantics.

@villebro A generic join feature I would advise against. Things like a theta join, snowflake-like joins with intermediate tables etc. These can all be built in the database using views. So why implementing these things a second time? I would really support only the most trivial case with a single fact in the middle and directly linked dimension tables with PKs.

As with the other proposals, let me know if I should create a comprehensive write-up. I would be willing to invest the time, but only if it is worth it.

@villebro A generic join feature I would advise against. Things like a theta join, snowflake-like joins with intermediate tables etc. These can all be built in the database using views. So why implementing these things a second time? I would really support only the most trivial case with a single fact in the middle and directly linked dimension tables with PKs.

As with the other proposals, let me know if I should create a comprehensive write-up. I would be willing to invest the time, but only if it is worth it.

@wernerdaehn I was speaking in general terms, i.e. supporting joining tables beyond the current table/virtual table functionality.
One thing that sticks out: if we have 100 FKs and thousands of columns, this can quickly become very burdensome, both for the application but even for the end user. So this needs to be properly scoped to make sure it improves the user experience rather than overwhelm it.

@wernerdaehn I was speaking in general terms, i.e. supporting joining tables beyond the current table/virtual table functionality.
One thing that sticks out: if we have 100 FKs and thousands of columns, this can quickly become very burdensome, both for the application but even for the end user. So this needs to be properly scoped to make sure it improves the user experience rather than overwhelm it.

Understood. My fear is just that when you implement that, you start with the SQL join clause in mind. I have seen multiple products fail because the join clause is extremely powerful and frankly, not suited for graphical tools. Use the SQL Editor of Superset for those. Anyway, you got my point, that is the important part.
Regarding the second paragraph, while your statement is certainly correct, it is not related to the star schema feature as such. If all needs to be in a single table, then this single table has equally many columns and must not be overwhelming as well.
Please let me know if I can do any work in that regards, in case it makes sense.

Understood. My fear is just that when you implement that, you start with the SQL join clause in mind. I have seen multiple products fail because the join clause is extremely powerful and frankly, not suited for graphical tools. Use the SQL Editor of Superset for those. Anyway, you got my point, that is the important part.
Regarding the second paragraph, while your statement is certainly correct, it is not related to the star schema feature as such. If all needs to be in a single table, then this single table has equally many columns and must not be overwhelming as well.
Please let me know if I can do any work in that regards, in case it makes sense.

I would love to see this feature go live, not sure if there is any progress. The description above is good enough, I just want to share my case.
Currently, I am using dbt for transformation, and I am following the star schema practice. Then I will visualize on Metabase. This approach works well because we have 1 central dimension table, as well as storage optimizing.
If I go with Superset, I will need to create 1 flat table. Some issues with the flat table are:

storage: the dimension data will be duplicated and redundancy
no benefit from incremental processing: when there is change in dimension table, I will need to refresh the whole flat table. If we support star schema, the fact table can remain unchanged
slow on filter values: it is faster to fetch 30k customer names compared to 100mil customer names (but only 30k distinct name)

@ducchetrongminh you can build a virtual dateset based on a view which join the fact to the dimension tables ?

I think this feature is very important. For example Microsoft Power BI can do this and metabase can do this too.
Example how it works on Metabase:

IMHO This feature is a must

Hi @djouallah, the star modeling seems like 1 fact table and some dimension table. Do you have more use cases for multiple fact tables?

Regarding the multiple fact tables: I would ask to support a single fact table with multiple dimensions only. First because this is significant improvement over the current situation already and second multiple fact tables is the same as a single fact table if a view is used.
Imagine this example:
Fact CO contains the cost center data and has date, costcenter, costelement, period, year as dimensions.
Fact P contains the plan data per year, costcenter group plus a plan version as dimensions.
If you allow the user to simply join those two tables, result will be a disaster. The same plan revenue is duplicated n times because CO is more fine grained. And CO data is duplicated because of the multiple plan versions. And if the user drills down to a single date, no plan data can be shown as planning does not happen for each day, same for the deepest level of a cost center or as soon as a filter on cost element is applied.
So what you must do instead is aggregate the CO data to year and cost center group level, pick from the Fact P the most recent plan version and show just that data. Now both intermediate data sets have a 1:1 relationship and can be joined. This is a complex operation, best implemented as view. Then you would have a third Fact CO_vs_Plan, the created view, that takes care of the correctness of the comparison.
Yes, supporting such out of the box would be the next evolution of this enhancement request but then you are at a full blown semantic layer, something superset tries to stay away from if I am reading correctly between the lines.

Hey Guys, multiple fact tables is not a big issues imho. As people can work around it. Important would be that we can slice and dice data as the user needs. Like fact table
And multiple dimentions. (The star schema)
I just build a datawarehouse and I end up using powerbi because there you can define the join dependencies. Imho this is crucial as feature. Otherwise if you datalake with multiple TB and dimensions with a couple of GB, you would have to join that into a single table which is huge.
Thanks

Star schema is not a showstopper, it can be replaced by a flat, it is a pain if your dimensions changes a lot but there is a solution, multi fact is the real problem, because there is no alternative, Look at the long term solution not a temporary fix.
see Tableau example, they implemented Start Schema, then what happen users keep asking for multi fact.

Star schema is not a showstopper, it can be replaced by a flat, it is a pain if your dimensions changes a lot but there is a solution, multi fact is the real problem, because there is no alternative, Look at the long term solution not a temporary fix.
see Tableau example, they implemented Start Schema, then what happen users keep asking for multi fact.

Do you have a link for that Tableu example ?
Can you explain how you solve ghe star to flat without views ?

https://help.tableau.com/current/online/en-us/datasource_datamodel.htm
of course, you need a view or a materialized table what I meant is, there is a workaround, but for multi fact, there are none !!!, unless you support nested table which is a pain !!! or merin multiple fact with different grains which is not good IMO

@djouallah Interesting, I am saying the exact opposite.
Let's figure out: You have a fact table for 1m rows, 20 foreign key columns. Each of those 20 dimensions has 10 rows and 100 columns. For example the Material table has material name, material group, material price, material cost, material vendor,...
Combining that into one table means adding 20 times 100 columns times 1m rows each. That is 2bn values, instead of 2010100 = 20k values.
If your propose to store all in a physical table, what was a large table before becomes enormous.
If you join that in a view, it is an extreme overhead.
The only option would be a view with join pruning, meaning the SQL optimizer removes joins from the view that are not used by any selected column. This is possible but only under a very strict set of conditions often not met.
The other aspect is the UI. What do you prefer? Selecting the columns from a list with 2000 column names or having a tree where you first select the type, e.g. Material, and then select the material related properties?
So no, Star schema is the simpler thing to do and the by far more important. Combining multiple facts is harder and can be done with views as a workaround. It would be nice if superset supports both but let's be reasonable and start with the by far more important one.
Agree?

@wernerdaehn  I am not disagreeing with you, of course, simple star is better, all what I am saying, people will ask for Galaxy schema from day 1.

but for multi fact, there are none !!!, unless you support nested table which is a pain !!! or merin multiple fact with different grains which is not good IMO

Why is there no workaround for multiple facts? It is not that difficult. And all that makes that SQL complicated are logical decisions, which the UI must provide as switches as well. So it won't be that much easier.
create view actual_vs_plan as
select year, cost_center_grp, sum(actual), sum(plan) from (
   select year, cost_center_grp, actual, null as plan from fact_co
  union all
  select year, cost_center_grp, null as actual, plan from fact_p where version = 1
)
group by year, cost_center_grp;

Amit from thoughtspot has a great blog, and he explain better what I am trying to say
https://prakasha.substack.com/p/the-metrics-layer-has-growing-up?s=r

but for multi fact, there are none !!!, unless you support nested table which is a pain !!! or merin multiple fact with different grains which is not good IMO

Why is there no workaround for multiple facts? It is not that difficult. And all that makes that SQL complicated are logical decisions, which the UI must provide as switches as well. So it won't be that much easier.

create view actual_vs_plan as

select year, cost_center_grp, sum(actual), sum(plan) from (

   select year, cost_center_grp, actual, null as plan from fact_co

  union all

  select year, cost_center_grp, null as actual, plan from fact_p where version = 1

)

group by year, cost_center_grp;

With this approach you need to read most of the things ans perform a distinct.
Further more you are not leaving the user the option to play with your dashboard. It's kind of hardcoded

No, there is no distinct clause in the SQL.
No, it combines the two measures at the deepest level. Yes you cannot see the plan value e.g. per day but it is not planned per day, hence the data is not available at that level.
Let me ask the question differently: How would you suggest the UI should look like to support combining multiple facts and what would happen under the cover, using my example?

It seems this SIP proposal was never given an official number, nor was it taken to the Apache Superset dev mailing list for the official discussion as part of the SIP procedure. I'll number it as 94 (even if that's a little strange at this juncture), and request that it either get proposed officially as a step leading toward a vote, or that we close it if there's no interest in pursuing it further. If you want to attend the next Superset Town Hall, that would be a great place to gather a bit of sentiment around it, as well.

Since there has been no substantive discussion here in about a year and a half, I'm going to close this issue and move it to discarded. If anyone wants to rekindle the effort and carry it forward in the SIP process (i.e. starting a DISCUSS thread on the Apache Superset dev mailing list), I'm more than happy to re-open it and adjust the status.

Hi,
I'm wondering how to best use superset with a start model. I found this discussion, unfortunatley closed.
More generally speaking, I wonder how to prevent an issue from being "closed due to inactivity" when it looks like everything that needed to be said was said, and waiting for superset side feedback on feasability.
Should I for instance re-open a new issue in this case referencing this issue ?
(and mayba add a "up!" commet every 6 months ;-) )

@squalou , some years ago I replied the issue. For now, I'm also thinking the initial design of Superset never consider data modeling, e.g. snowflake or start modeling. The workaround for data modeling you should consider other sematic/modeling tools e.g. DBT, and expose a "big flat table" to Superset.

we decided not to use Superset because it doesn't support Star Schema

@leonliuaptean To be honest, there are lots of workarounds for the Star/Snowflake/and so on data modeling. Superset is a viz tool, and its original design didn't focus too much on data modeling.


Inconsistency in ui

While playing some videos the sidebar is taking less width while with others its fine

I'd like to work on this one but in cloned repo I can see the code base but I can't see the output for these parts for obvious reasons.

Did u seed the db? @Pratham271

yeah i did seed it but it only shows the test courses but if I click on them I am being redirected to the original site therefore I am not able to reach this specific page on the dev side

@Pratham271 it shouldn't be like that could u share a recording of what's going wrong?

@devsargam it is working fine with local setup but not the same with docker setup. It worked for local though

Yeah got it. Need to figure out a way to exec into db and see it.


No CSV download button in 2023

Description
I don't see any csv download button for this official Mandarin deck, despite updating the script to match the required url. Can you kindly update the code? Does this script include scraping audios? Thank you in advance.

Currently, people can import/export decks from/to Brainscape with a free account.
More importantly, downloading a deck that you do not own has legal issues, so it is never an intended feature of this script.


feat: use catlab extraction when showing matrices

Description
Proper dimensions appear for stratified models generated by catlab. Doubly stratified models also work.
https://github.com/DARPA-ASKEM/Terarium/assets/28237258/52d3353a-6f53-4c4e-ba4b-506871709b25

You have a few console.log here - are they still needed to debug or can we remove?