text:
Hackathon Section Enhancement #198

Hi @anmode,
I have enhanced the Hackathon cards by adding some CSS effects as well fixed the issues which I addressed in #198. Cards are more user interactive now.
Below are the enhancements :
On hover,

user will see both blue layer as well as title of the hackathon at same time. (Earlier title was only visible when you hover at center of the image).
Added shadow effect on the cards
Added card lifting effect - > The card moves 4px up and shadow casts down.

Below are the fixes :

Image border's are properly aligned with blue layer now.

https://github.com/anmode/grabtern-frontend/assets/82926210/8fdb2fa6-230b-4a60-b209-8162ed86300d

Hey! @archiejain28 please refer this pR https://github.com/anmode/grabtern-frontend/pull/224.
when you do like this. I will merge your PR before this PR. and your commits will also be increased.

Thank you @anmode , I will do the required changes.

Hi @anmode I have refactored the code.
Could you pls review them now.

Done @anmode , as discussed -
Removed round corners from  the hover effect & added them in the image itself.
Could pls check them now.

I think providing you level 3 would be nice choice. Reason: Great work

Thanks much @anmode ! :)

repo: anmode/grabtern-frontend


text:
getaddrinfo ENOENT raw.githubusercontent.com

Output:
‼ getaddrinfo ENOENT raw.githubusercontent.com
‼ gifsicle pre-build test failed
i compiling from source
× Error: Command failed: C:\Windows\system32\cmd.exe /s /c "autoreconf -ivf"
'autoreconf' �����ڲ����ⲿ���Ҳ���ǿ����еĳ���

This is a network problem

source: gharchive/issue

authors: HolleKitty anncwb

repo: anncwb/vite-plugin-imagemin

license: MIT

license_type: permissive

license_source: github-api


text:
Fix pt-BR locale file name

The properly name for locale file is pt-BR not pt-br. Lower case
country code causes and error in Redmine that complete disable
translation for this language.

Thank you

source: gharchive/pull-request

authors: annikoff ecanuto

license: mit

license_source: bigquery


text:
🛑 Rainbow is down

In 3d9e0e0, Rainbow (https://радуга-талантов.рф) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Rainbow is back up in 3dfbcdf after 41 minutes.

repo: ano-pedproject/repo


text:
🛑 Rainbow is down

In cb291b3, Rainbow (https://радуга-талантов.рф) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Rainbow is back up in 599105e after 16 minutes.


text:
🛑 Pedproject is down

In 238c95e, Pedproject (https://педпроект.рф) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Pedproject is back up in 5391ec0 after 34 minutes.


text:
Faucet updates for SE

Amendments:

Update favicon
Change url to namada.net/shielded-expedition/faucet

Text:

Namada Faucet --> Namada Shielded Expedition Faucet
What’s the “Target address”?
Token address (defaults to NAM) --> “Enter your Namada address (tnam1)”

Done!

authors: jurevans mateuszjasiuk

repo: anoma/namada-interface


text:
Update CitadelOne.toml for testnet 15

Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

Previous PR for reference: https://github.com/anoma/namada-testnets/pull/1010

repo: anoma/namada-testnets


text:
Create NODEJUMPER.toml

Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[ ] Only one toml is added in this PR
[ ] The file being added is indeed a .toml file
[ ] The toml being added is to the correct folder, and only to the correct folder
[ ] The eth_hot_key and eth_cold_key are present
[ ] The email, discord, elements telegram, and twitter fields are present and valid

The transactions.toml file for 62971c7 was deemed invalid with error Invalid signatures: Signatures could not be verified

The transactions.toml file for 62971c7 was deemed invalid with error Invalid signatures: Signatures could not be verified

The transactions.toml file for 989acdc was deemed invalid with error Invalid signatures: Signatures could not be verified

The transactions.toml file for 989acdc was deemed invalid with error Invalid signatures: Signatures could not be verified

The transactions.toml file for 989acdc was deemed invalid with error Invalid signatures: Signatures could not be verified

The transactions.toml file for 989acdc was deemed invalid with error Invalid signatures: Signatures could not be verified

@bengtlofgren hi! I created a new PR with correct validator toml file https://github.com/anoma/namada-testnets/pull/3171. Should I close this PR or update with new correct validator toml file?

The transactions.toml file for 989acdc was deemed invalid with error Invalid signatures: Signatures could not be verified

authors: bengtlofgren vbloher


text:
html title

adds html_title to Report class so that we can customize the html title of a report
renames nb_name to html_title in the ReportGenerator, as it was only being used for setting the html title anyway

@hyukim17 I think these errors have nothing to do with your changes. I'm going to fix them, add Python 3.12 support, add a new testing bundle with the latest versions of libraries, and then I'll do a release.

What's the process for deploying a new wheel?

I just released v0.5.0: https://github.com/ansatzcapital/leda/releases/tag/v0.5.0. It's now available via pypi: https://pypi.org/project/leda/0.5.0/.
The process is simply to create a new GitHub release here: https://github.com/ansatzcapital/leda/releases. (GitHub has a button to auto-generate release notes, which I use.) Then the CI will do the rest:  https://github.com/ansatzcapital/leda/actions/runs/8039217496.
Note that in this case, we just crossed the 100MB limit for the sdist--but not the whl--file, so only the wheel was uploaded. I'll look into fixing this.
Note also that I added a scheduled daily run here so that external changes don't silently break the CI tests in between PRs, as they did in this one.

repo: ansatzcapital/leda


text:
Brancher l'indicateur d'alertes du menu sur le service de détection des alertes.

Cf : https://github.com/edegenetais-nx/psc-admin-portal/issues/65

Mettre en place le rafraichissement automatique de l'indicateur en utilisant la période normalisée environnement.UPDATE_PERIOD

repo: ansforge/psc-admin-portal


text:
Ensure firmware testing

SUMMARY

Add some testing for the ensure_firmware module
ISSUE TYPE

Feature Pull Request

@resoluteCoder I think you were the ones to write the ensure_tags tests to this might be in your ballpark

@aaron97neu Yes that was me. I can review on Monday :smile: :+1:


text:
The conditional check 'ansible_distribution not in _consul_nix_distros' failed

Time by time I am facing weird errors while starting "site.yml" playbook to install consul on my nodes.
It's strange, because I always use the same distribution - Ubuntu 20.04.4 LTS.

Are you gathering facts in the playbook? Seems like ansible_distribution is currently undefined, which should be filled in when gathering host facts.

authors: Blefish dimakyriakov

repo: ansible-community/ansible-consul


text:
Running testinfra tests with openstack delegated driver randomly fails

Issue Type

Bug report

Molecule and Ansible details
ansible --version && molecule --version
ansible 2.9.9
  config file = None
  configured module search path = ['/home/mat/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /home/mat/Virtualenv/ansible3/lib/python3.8/site-packages/ansible
  executable location = /home/mat/Virtualenv/ansible3/bin/ansible
  python version = 3.8.4rc1 (default, Jul  1 2020, 15:31:45) [GCC 9.3.0]
molecule 3.0.4

Molecule installation method (one of):

pip

Ansible installation method (one of):

pip

Detail any linters or test runners used:
Desired Behavior
Running testinfra tests should always return the same status when no change is made between the execution of 2 tests.
Actual Behaviour
When performing molecule verify command:

the first time all tests passed
the second time all tests failed du to an SSH issue with the openstack instances.

Firstly: all tests passed
molecule verify
--> Test matrix

└── default
    └── verify

--> Scenario: 'default'
--> Action: 'verify'
--> Executing Testinfra tests found in /home/mat/GIT/ansible_project/ansible_role/xymon-client/molecule/default/tests/...
    ============================= test session starts ==============================
    platform linux -- Python 3.8.4rc1, pytest-5.4.3, py-1.8.2, pluggy-0.13.1
    rootdir: /home/mat/GIT/ansible_project/ansible_role/xymon-client/molecule/default
    plugins: testinfra-5.2.1
collected 8 items

    tests/test_default.py ..                                                 [ 25%]
    tests/test_common.py ......                                              [100%]

    ============================== 8 passed in 15.70s ==============================
Verifier completed successfully.

Secondly: all tests failed:
When retrying to perform molecule verify* command 5 seconds after the first try, all tests failed with the error:
RuntimeError: CommandResult(command=b'test -e /etc/default/xymon-client', exit_status=255, stdout=None, stderr=b'kex_exchange_identification: read: Connection reset by peer\r\n')
molecule verify
--> Test matrix

└── default
    └── verify

--> Scenario: 'default'
--> Action: 'verify'
--> Executing Testinfra tests found in /home/mat/GIT/ansible_project/ansible_role/xymon-client/molecule/default/tests/...
    ============================= test session starts ==============================
    platform linux -- Python 3.8.4rc1, pytest-5.4.3, py-1.8.2, pluggy-0.13.1
    rootdir: /home/mat/GIT/ansible_project/ansible_role/xymon-client/molecule/default
    plugins: testinfra-5.2.1
collected 8 items

    tests/test_default.py FF                                                 [ 25%]
    tests/test_common.py FFFFFF                                              [100%]

    =================================== FAILURES ===================================
    _____________ test_xymon_client_default[ansible://xymon-debian10] ______________

    host = <testinfra.host.Host ansible://xymon-debian10>

        def test_xymon_client_default(host):
            f = host.file('/etc/default/xymon-client')
            ansible_vars = host.ansible.get_variables()
            hostname = ansible_vars['inventory_hostname']
    >       assert f.contains('^XYMONSERVERS="127.0.0.1"$')

    tests/test_default.py:13:
    _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
    /home/mat/Virtualenv/ansible3/lib/python3.8/site-packages/testinfra/modules/file.py:118: in contains
        return self.run_test("grep -qs -- %s %s", pattern, self.path).rc == 0
    /home/mat/Virtualenv/ansible3/lib/python3.8/site-packages/testinfra/host.py:94: in run_test
        return self.run_expect([0, 1], command, *args, **kwargs)
    /home/mat/Virtualenv/ansible3/lib/python3.8/site-packages/testinfra/host.py:75: in run
        return self.backend.run(command, *args, **kwargs)
    /home/mat/Virtualenv/ansible3/lib/python3.8/site-packages/testinfra/backend/ansible.py:47: in run
        return host.run(command)
    /home/mat/Virtualenv/ansible3/lib/python3.8/site-packages/testinfra/host.py:75: in run
        return self.backend.run(command, *args, **kwargs)
    /home/mat/Virtualenv/ansible3/lib/python3.8/site-packages/testinfra/backend/ssh.py:34: in run
        return self.run_ssh(self.get_command(command, *args))
    _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

    self = <testinfra.backend.ssh.SshBackend object at 0x7f908138c310>
    command = 'grep -qs -- \'^XYMONSERVERS="127.0.0.1"$\' /etc/default/xymon-client'

        def run_ssh(self, command):
            cmd, cmd_args = self._build_ssh_command(command)
            out = self.run_local(
                " ".join(cmd), *cmd_args)
            out.command = self.encode(command)
            if out.rc == 255:
                # ssh exits with the exit status of the remote command or with 255
                # if an error occurred.
    >           raise RuntimeError(out)
    E           RuntimeError: CommandResult(command=b'grep -qs -- \'^XYMONSERVERS="127.0.0.1"$\' /etc/default/xymon-client', exit_status=255, stdout=None, stderr=b'kex_exchange_identification: read: Connection reset by peer\r\n')

    /home/mat/Virtualenv/ansible3/lib/python3.8/site-packages/testinfra/backend/ssh.py:77: RuntimeError
    ______________ test_xymon_client_default[ansible://xymon-debian9] ______________
.....

I've rewritten the test in ansible instead of testinfra and with ansible verifier, there is no SSH issue.
So I guess something weird happens between testinfra and SSH when running molecule verify command.
Additionnal info
molecule.yml
dependency:
  name: galaxy
driver:
  name: delegated
  security_group: xymon-default-molecule
  network: molecule
  ssh_user: admin
  keypair: xymon-default
lint: |
  set -e
  yamllint .
  ansible-lint -x 503,303 molecule/default/*.yml
platforms:
  - name: xymon-debian10
    image: Debian_10
    flavor: clouds.small
  - name: xymon-debian9
    image: Debian_9
    flavor: clouds.small
provisioner:
  name: ansible
  config_options:
    defaults:
      ansible_managed: 'Molecule Tests'
verifier:
  name: testinfra

converge.yml
- name: Converge
  hosts: all
  become: true
  vars:
    ansible_python_interpreter: auto_silent
  pre_tasks:
    - name: update apt cache
      apt:
        update_cache: true
      changed_when: false
  roles:
    - role: xymon-client

testinfra
test_common.py
import os

import testinfra.utils.ansible_runner

testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
    os.environ['MOLECULE_INVENTORY_FILE']).get_hosts('all')

def test_package(host):
    p = host.package('xymon-client')
    assert p.is_installed

def test_service(host):
    s = host.service('xymon-client')
    assert s.is_running
    assert s.is_enabled

def test_xymon_client_default(host):
    f = host.file('/etc/default/xymon-client')
    assert f.exists
    assert f.is_file
    assert f.user == 'root'
    assert f.group == 'root'
    assert f.mode == 0o644

test_default.py
import os

import testinfra.utils.ansible_runner

testinfra_hosts = testinfra.utils.ansible_runner.AnsibleRunner(
    os.environ['MOLECULE_INVENTORY_FILE']).get_hosts('all')

def test_xymon_client_default(host):
    f = host.file('/etc/default/xymon-client')
    ansible_vars = host.ansible.get_variables()
    hostname = ansible_vars['inventory_hostname']
    assert f.contains('^XYMONSERVERS="127.0.0.1"$')
    assert f.contains('^CLIENTHOSTNAME="%s"$' % hostname)

Hello @matmb-cld .

So I guess something weird happens between testinfra and SSH when running molecule verify command.
I'm wondering whether this is a molecule-openstack issue or a testinfra issue. Reason is that when changing to ansible verify it works as expected. With a quick look I can see that maybe this issue: https://github.com/philpep/testinfra/issues/528 is related.

In any case I'll have a look later this week to troubleshoot this a bit. Try a couple more drivers (e.g aws) to see what happens

We now have this problem as well. The testinfra tests fail randomly.

Issue still occurs.

authors: berendt matmb-cld nikParasyr palchyk-alex

repo: ansible-community/molecule-openstack


text:
Create Is Container Flag

There are individual controls that should not be enabled on a container system. There needs to be a broad is_container variable added to have a single point to disable tasks that don't work on container environments.

More generically, there are things that don't work without systemd running, which might or might not be a container. Very few things truly don't work in a container. One is SELinux.

authors: georgenalen jamescassell


text:
Ubuntu 24 compatibility

Question
Does the script work for Ubuntu 24?
Environment (please complete the following information):

Ansible Version: [e.g. 2.16.4]
Host Python Version: [Python 3.12.2]
Ansible Server Python Version: [Python 3.12.2]
Additional Details:
As subject.  Thanks.

hi @patrickfong
I have not current had chance to test this, i know there are some changes that will likely be changed when CIS is released for 24.04.
Many thanks
uk-bolly

hi @patrickfong
I have now written the first release of ubuntu24-cis , which is currently in subscriber access area. This should be officially available in around 10 weeks for public.
Kindest regards
uk-bolly

Hi @uk-bolly,
This is really cool! I'm in the process of implementing CIS compliance on our Ubuntu 24 machines. I would love to use and contribute to your work. Would you be able to share a link to the repository?

hi @sheikhomar
Thank you, It is currently released to subscribers using the early access. It should be released to the community early in the new year for other contributors.
It will then be open for all to feedback back an assist in improving.
Many thanks
uk-bolly

@uk-bolly How do I subscribe and get early access?

@uk-bolly How do I subscribe and get early access?

Hi @sheikhomar, please reach out to me at georgen@mindpointgroup.com and we can have further discussions.

authors: georgenalen patrickfong sheikhomar uk-bolly


text:
bump more core extensions

more core extensions is now on 4.1.0 and I was wondering if we might be able to update.
no particular reason, just update for update's sake
here's what's included:
https://github.com/ManageIQ/more_core_extensions/compare/179bf40..e5b4501

Added Ruby 2.7 support [#79]
Added Process#pause, Process#resume, and Process#alive? [#73]

array added * #compact_map - Collect non-nil results from the block
array added #tabular_sort - Sorts an Array of Hashes by specific columns
hierarchy added #descendant_get - Returns the descendant with a given name
the two breaking changes:

BREAKING: Moved Object#descendant_get to Class#descendant_get [#75]
BREAKING: Removed deprecated Enumerable#stable_sort_by [#76]

a minor header output change was made that hasn't been released yet to make tableize more markdown compliant
see https://github.com/ManageIQ/linux_admin/pull/221

We ultimately want this for rest-client 2.1+. Please merge.

Ok, pushed v0.21.3 including this change

authors: bdunne d-m-u djberg96


text:
test_plugins[plugins-docker-default] fails because of non-existing repo

It looks like @retr0h has removed https://github.com/retr0h/ansible-plugins repository today/yesterday, which lead to failing the test test_plugins[plugins-docker-default], which tries cloning it internally.
Issue Type

Bug report

Molecule and Ansible details
Cloning into '/home/travis/.gilt/clone/retr0h.ansible-plugins'...
remote: Invalid username or password.
fatal: Authentication failed for 'https://github.com/retr0h/ansible-plugins.git/'

Desired Behavior
Test test_plugins[plugins-docker-default] pass
Actual Behaviour
Test test_plugins[plugins-docker-default] fails

cc @gundalow

Hey @retr0h, any way to restore that? We can move the repo if you don't want it under your account but need a source to figure out the test.

Log: https://travis-ci.com/ansible/molecule/jobs/158736934#L1074-L1078

repo: ansible/molecule


text:
Avoid sanity checks for AnsiblePlaybook unit tests

This avoids the need to have the Docker daemon running when running unit tests which should not be necessary and introduces another way for the unit tests to fail.
I've tested this manually like so:
$ sudo systemctl stop docker
$  tox -e py37-ansible28-unit -- -n auto -p no:warning

This is necessary for these tests because they call _instance.execute directly which is a wrapper function which includes the call to the sanity checks before calling the mocked runner command.

Also note, this cannot be reproduced on the CI because we always have the docker service up and running. See https://travis-ci.com/ansible/molecule/jobs/203111044#L238.


text:
Add a setting for maxBuffer parameter for ansible-lint executor

Problem
Parsing of ansible-lint output falls in case of large output (more than 10 MB). Right now it's added in commandRunner.ts with hardcoded value for maxBuffer.
Solution
Add a setting with max size for an ansible-lint output.
Alternatives
Increase the maximum value for maxBuffer.
Additional context
The error into VSCode Could not parse ansible-lint output. Please check your ansible-lint installation & configuration. More info in Ansible Server output.

@vladislavzl please provide a reproducer for this, so that we can look into this

Hi @audgirka. Feel free to test with this repo - https://github.com/vladislavzl/test_vscode_ansible/tree/main

authors: audgirka vladislavzl

repo: ansible/vscode-ansible


text:
Database dumps

Hi.
It's possible to add interchange database dump features? Like these capifony commands:
cap database:copy:to_local  # Dumps remote database, downloads it to local, and populates here
cap database:copy:to_remote # Dumps local database, loads it to remote, and populates there
cap database:dump:local     # Dumps local database
cap database:dump:remote    # Dumps remote database

Thanks.

Hi
Well, I guess you could use the ansible mysql_db module in hooks (which has things like state: dump) or sth similar.
Capistrano adds features like these as it is a Ruby project but in ansible we can use the official Ansible modules

thanks @ricardclau

authors: davidromani ricardclau

repo: ansistrano/deploy


text:
CHORE: allow latest PyEDB version

As of now, you cannot use together the main branches of PyAEDT and PyEDB. This will allow you to do so

Thank you both! I just removed the upper bound following @SMoraisAnsys suggestion. Feel free to approve and merge whenever you prefer

repo: ansys/pyaedt


text:
Switch to using new Fluent 'system_coupling' attribute

Work was recently done to support a new add_participant variation that takes a PyAnsys participant session object and interacts with it directly to set up and solve the coupled analysis.
Currently this only works with Fluent participants and, in the intial implementation, it depended on an adaptor class implemented on the PySystemCoupling side that exposed an interface that PySystemCoupling could work with. Since then, Fluent has exposed what is essentially the same interface via a new system_coupling attribute on the Fluent session object.
Some small changes are made here to switch to using the new attribute rather than the adaptor. The adaptor has been deleted.
The intent is that other PyAnsys product APIs for supported SyC participant types will eventually each expose its own system_coupling attribute, This will provide an interface consistent with the Fluent one.

Codecov Report

Merging #189 (fd63075) into main (cd51d8b) will decrease coverage by 0.52%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #189      +/-   ##
==========================================
- Coverage   90.12%   89.60%   -0.52%
==========================================
  Files          32       32
  Lines        1691     1693       +2
==========================================
- Hits         1524     1517       -7
- Misses        167      176       +9

Addresses issue #148

authors: codecov-commenter iboyd-ansys

repo: ansys/pysystem-coupling


text:
fix: ridiculous module field issue...

@chenshuai2144 @afc163 Please help to review, it sucks.

Why there isn't any news for about one month???

you are rigth

repo: ant-design/ant-design-pro-layout


text:
Chrome icon color bug

看起来像是 svg babel 插件的问题

看起来像是 svg babel 插件的问题

最开始是好的，很奇怪，有可能是 svg 不规范，直接打开是好的，但是放到网页中就有问题了。

我看它这个 svg 在优化的时候，这些 id 会被重写，可能是跟其他的地方重复了（奇怪浏览器竟然不是取最接近的）。

优化是通过 inline-react-svg babel 插件做的，然后底层用的是 svgo，2.8.0 版本。
https://github.com/svg/svgo/pull/1784 这个 pr 似乎是在修复这个问题，不过那个 babel 插件用的版本比较老……
不知道能不能通过配置绕过，感觉有点麻烦。

之前没问题是只有一个这种渐变色，直到最近，又加了一个 polygon 的渐变彩色图标……

发现这个图标在 ConnectModal 里面的时候又正常了。

之前没问题是只有一个这种渐变色，直到最近，又加了一个 polygon 的渐变彩色图标……

哦哦。明白了，所以是和 polygon 同时出现的时候会有这个问题。


text:
docs: update ConnectModal demos, use icons in @ant-design/web3-icons

close #https://github.com/ant-design/ant-design-web3/issues/156

测试钱包3 图片丢了

 测试钱包3 图片丢了

是不是你的网络问题，我看我这里是好的

 测试钱包3 图片丢了

是不是你的网络问题，我看我这里是好的

确实是我网络的问题，现在好了


text:
keepalive缓存功能失效

keep-alive的include的值始终是空数组

测试了一下我们提供的例子，是生效的，有没有设置keepalive为true，默认情况下应该是不保活的，如果提供一个复现仓库那就太好了

repo: antdv-pro/antdv-pro


text:
doorkeeper 2.2.1 support

Following installation instructions and aware of requirements as described in the readme, bundle install produces:
Resolving dependencies...
Bundler could not find compatible versions for gem "doorkeeper":
  In Gemfile:
    wine_bouncer (>= 0) ruby depends on
      doorkeeper (~> 1.4.0) ruby

    doorkeeper (2.2.1)

Requirements say: "Doorkeeper > 1.4.0 and < 3"
Appreciate any guidance.

Thats the master you are currently looking at. Ill be releasing the 0.4 in the near future #44 .

0.4.0 is released

authors: antek-drzewiecki catogonzalez


text:
Function request

Hi,there.
This Xcode-dark-theme really attracts me, and i can't wait to use it. However, I finally noticed that it can only run on intellij... It's very disappointing... So, is it possible to run it on Pycharm? If not, would you please make it possible to run on Pycharm?
Second, dark themes are great, but i suppose that there also need to be a light theme. What's your opinion? Hope you to develope an "Xcode-light-theme"
👍 Best regards  @antelle

Hi! The JetBrains' plugin repository website says it should be available for all IDE's, including PyCharm.

I'm not interested in a light theme so I won't make it.

authors: MX-Qulin antelle

repo: antelle/intellij-xcode-dark-theme


text:
Fix version for configparser

When installing clippercard with pip, I get the following error. By changing the configparser version from 3.3.0r2 to 3.3.0.post2 pip install works correctly.
$ pip install clippercard
Collecting clippercard
  Using cached clippercard-0.3.0.tar.gz
Collecting BeautifulSoup4>=4.3.2 (from clippercard)
  Using cached beautifulsoup4-4.3.2.tar.gz
Collecting configparser==3.3.0r2 (from clippercard)
  Could not find a version that satisfies the requirement configparser==3.3.0r2 (from clippercard) (from versions: 3.2.0.post1, 3.2.0.post2, 3.2.0.post3, 3.3.0.post1, 3.3.0.post2, 3.5.0b1, 3.5.0b2)
  Some externally hosted files were ignored as access to them may be unreliable (use --allow-external to allow).
  No distributions matching the version for configparser==3.3.0r2 (from clippercard)

You can try install it by executing the following command:
pip install http://github.com/thejsj/clippercard/archive/v0.0.1.tar.gz

There might be a better solution to this problem. If there is, let me know.

Thanks for the fix!

I will prep a PyPi release and a version bump tomorrow.

Awesome!

Released: https://pypi.python.org/pypi/clippercard/0.3.1

authors: anthonywu thejsj

repo: anthonywu/clippercard


text:
readme: first version of a README. TODO install instructions.

Hey everybody,
I have created a little README file for the project, hope it's ok.
The screenshot section is maybe too big.. one screenshot should be enough,
and I still need to add an install section for it.
@anthraxx
You are the owner and founder of the project, which LICENSE do you want? GPLv3?
We should give it a license.. who knows maybe some other distribution will fork it.

applied via e6d072d8740a531fe829d5ba57539c6fe2d8d4d3 and 076b2bead6ad9a3b1c23f407196fdfefd1f3b342
thanks :+1:

authors: anthraxx shibumi

repo: anthraxx/arch-security-tracker


text:
Debug left enabled on latest release

Information:
AntiAd Version: 2.3.0
Server Version: CraftBukkit version git-PaperSpigot-"a861cc6" (MC: 1.8.8) (Implementing API version 1.8.8-R0.1-SNAPSHOT)

damn, hope that is only here on github and not everywhere i forgot that

releasing a 2.3.1 now with that removed ^^.
I should really not have my debug that way

2.3.1 is now released.
Sorry about that :)

authors: KasperFranz Yive

repo: antiAD/AntiAd


text:
update README

The README references:

ci/config.sh
ci/setup_env.sh
ci/entrypoint.sh

None of those files exist in the main branch. Please update the instructions.
Thanks!

also the overlays flag referenced in the README doesn't exist (at least in the main branch)

That was indeed a leftover from olden times, thanks for pointing that out.


text:
getCalendarItemTypesIncludeDeleted does not exist on calendarItemTypeApi

getCalendarItemTypesIncludeDeleted does not exist on calendarItemTypeApi
  async loadCalendarItemTypesIncludeDeleted() {
     await this.api.calendarItemTypeApi.getCalendarItemTypesIncludeDeleted().then(calendarItemTypes => {
      this.calendarItemTypesIncludeDeleted = calendarItemTypes;
    });
  }

See medispring

This method was comitted in PR #452.
The icc-api submodule reference on nsi/development is : 014178672bd1cabdbd53d88707de3baf75a6cff7 (last commit on icc-api master branch)

New PR ( #475) was made with the same submodule reference (014178672bd1cabdbd53d88707de3baf75a6cff7) in medispring repo.

authors: antoinepairet htr-nsi

repo: antoinepairet/icc-api


text:
Feature Request : Choose different "component" between different view

For example : I want to have a splitDay in the day view but not in the month view

Hi @HarunaIchinose, sorry I totally missed your question. :/
This is actually already possible, here is a demo: https://codepen.io/antoniandre/pen/qeNKJx?editors=1010
Hope it helps, let me know if you still need guidance.

oh thanks !

authors: HarunaIchinose antoniandre

repo: antoniandre/vue-cal


text:
month view show the last redundant row

all month view  in the https://antoniandre.github.io/vue-cal/ show the last redundant row. every day int the last row is not belong the current month. Is it a bug or a design?

Hi @nature-li,
This is by design.
For instance if you go to September 2019, or July 2018, some days are overflowing on that row.
We don't want to switch to another year and have a different number of rows that would look broken and not smooth.

issue #95 is similar, I answered a way to remove the cells from next month if you have reasons to do it:
.vuecal__cell:not(.out-of-scope) ~ .vuecal__cell.out-of-scope {display: none}

authors: antoniandre nature-li


text:
Feature Request: Ping Pulse / CPU Cores / GPU Load

I was using this guys script in the official build before the pre-release 0.6.0
http://modworkshop.net/mydownloads.php?action=view_down&did=18852
I know you have some similar features under "Percent Effect Layer" but this guy has some really neat effects like keyboard dimming out when CPU is under load with a cool num pad color circle going. Theirs a video on his post.
The ping pulse is very cool too.
I really liked it, sadly it no longer functions in the new 0.6.0 pre-release. Maybe these are ideas/features you can expand on in future builds and just make them a part of Auroras core functions :)

You're looking for @VoronFX and I would like to leave it up to him to implement those in.

You're looking for @VoronFX and I would like to leave it up to him to implement those in.

Here is hoping he updates his scripts for the new build :) It was a favorite for me.

Here is hoping he updates his scripts for the new build :) It was a favorite for me.

Yeah! I've just made an update with new scripts and support for version 0.6.0. Check it out!

authors: McRiot VoronFX antonpup

repo: antonpup/Aurora


text:
Support for logitech G pro keyboard?

What are you experiencing an issue with?

[ ] Aurora
[ ] Aurora supported game, (Fill in the game)
[ ] Aurora Wrapper
[ ] Aurora Profile
[x] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[ ] Aurora Updater
[ ] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[ ] Performance problems
[ ] Visual bug
[ ] Localization issue
[ ] Enhancement suggestion
[ ] Device issue
[ ] Other, (Fill in another type of issue here)

Expected behavior:
Explain here what should happen.
Actual behavior:
Explain here what happened instead.
How often does this problem occur?

This problem occurs... (Fill in here)
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

What is the version of Aurora you are experiencing this issue with?

My version is... (Fill in here)
Does the previous version of Aurora have this issue?
The previous version, (Fill in the version here), did/did not have this issue.
Please include any logs, screenshots, or files that are related to this issue:

For logs, you can use: http://pastebin.com/
For screenshots, you can use: http://imgur.com/
For files, you can use: http://www.tinyupload.com/ , https://www.dropbox.com/ , or attach them to the issue.

If that keyboard is supported by the sdk, which I can't find hard evidence of, but I assume it works, it'll just work in Aurora using a layout similar to it selected. If it isn't supported by the sdk I can't really do anything.


text:
fix of not re enabling "Use Freestyle instead" checkbox

It seems like it was originally meant to disable the controls when the use Freestyle instead checkbox is checked.
This pull request proposes the following changes:

fix of not re enabling "Use Freestyle instead" checkbox

Known issues/To do:

wrong status at startup

What? AppVeyor? Why do you not want to build my pr?

Closed because AppVeyor does not want to build it.


text:
EFI image doesn't work over libvirt + kvm/qemu

I'm using Fedora 28 as a host. I tried using UEFI (TianoCore) and it doens't work with your image; it drops to a shell (efilinux).
On the other hand, I try the Fedora 29 Beta ISO image and it works fine.
Might there be something missing in the EFI iso image?

The EFI iso image is just wrong. It does not contain any El Torito records. Instead, it contains BOOT/EFI/BOOTX64.EFI file, which UEFI does not even look at on optical media.
For now, you can create your own EFI iso by running these commands (assuming that you have dosfstools, cdrecord and mtools installed):
wget https://boot.netboot.xyz/ipxe/netboot.xyz.efi
mkdir cdrom
dd if=/dev/null of=cdrom/image.flp seek=2880
mkfs.vfat -M 0xf8  -f 1 cdrom/image.flp
mmd -i cdrom/image.flp ::efi ::efi/boot
mcopy -i cdrom/image.flp netboot.xyz.efi ::efi/boot/bootx64.efi
mkisofs -o ipxe-uefi.iso  -eltorito-platform 0xef -eltorito-boot image.flp -no-emul-boot cdrom

I will create a pull request later.

Closing this as @patrakov was kind enough to fix.  Please reopen if it's still an issue.

authors: antonym patrakov renich


text:
Add log rotation configuration for VM Agent

VM Agent crashes because log rotation configuration is nil.
This change adds log rotation configuration for VM Agent.

Hi @antoninbas , I just found VM Agent crashed on main branch because the newly added log rotation configurations are nil.
Could you help review this change?

/test-vm-e2e

Follow-up PR to fix initialization in options.go: https://github.com/antrea-io/antrea/pull/5369

/test-vm-e2e

@mengdie-song could you rebase the PR and run test-vm-e2e again?

/test-vm-e2e

Hi @antoninbas, I has rebased the PR and VM e2e test has passed. Regarding the e2e cleanup issue, I have another change to fix it (https://github.com/antrea-io/antrea/pull/5406).

/skip-conformance
/skip-networkpolicy

authors: XinShuYang antoninbas mengdie-song

repo: antrea-io/antrea


text:
Do not run tests when building OVS for UBI

This change adds back the default RPMBUILD_OPT option to disable unnecessary tests when building OVS rpm packages, which was previously overwritten in #6052.

/skip-all

authors: antoninbas xliuxu


text:
Improve error message on ci setup fails

Description
For CI tests, if the setup or parameters errors out, the actual error message output can be confusing. This is due to AfterSuite running when some steps in BeforeSuite fails, either due to parameter error or setup issue. Because setup is not finished, AfterSuite clean up also encounters and throws error, causing confusion. This PR fixes two cases.
Changes

Add checks for Nephe controller deployment and skip some checks if its not deployed.
Add checks for kubectl configuration and skip clean up if the cluster is not configured.
Fixed a formatting issue for namespace clean up.

Can we avoid AfterSuite, if BeforeSuite fails??

I think we should run AfterSuite even if BeforeSuite fails, because some setup might already be there and some are not due to partially ran BeforeSuite. AfterSuite should try its best to clean up so the cluster won't be in a partially configured confusing state.

/nephe-test-e2e-kind

/nephe-test-e2e-kind

/nephe-test-e2e-kind

/nephe-test-e2e-kind

/nephe-test-e2e-azure

repo: antrea-io/nephe


text:
Manager e2e testing

New Theia Manager e2e Tests
This change adds end to end testing for the theia manager and cli integration.
Signed-off-by: Dhruv Jain dhruvj@vmware.com

Codecov Report

Merging #124 (32214e3) into main (e7bd73a) will decrease coverage by 0.05%.
The diff coverage is 50.00%.

:exclamation: Current head 32214e3 differs from pull request most recent head e2cdfae. Consider uploading reports for the commit e2cdfae to get more accurate results

@@            Coverage Diff             @@
##             main     #124      +/-   ##
==========================================
- Coverage   60.39%   60.34%   -0.06%
==========================================
  Files          18       18
  Lines        2510     2514       +4
==========================================
+ Hits         1516     1517       +1
- Misses        839      841       +2
- Partials      155      156       +1

Flag
Coverage Δ

unit-tests
60.34% <50.00%> (-0.06%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ntroller/networkpolicyrecommendation/controller.go
74.00% <50.00%> (-0.36%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

I notice the K8s version is updated to 1.23.5, maybe also replace master by master|control-plane?
https://github.com/antrea-io/theia/blob/8cef9da12a861d71eb33d3b1af5dd8e7a62407af/ci/jenkins/test-vmc.sh#L40-L41

/theia-test-e2e

authors: Dhruv-J codecov-commenter yanjunz97 yuntanghsu

repo: antrea-io/theia


text:
feat: tooltip custom content 内置 class: "g2-tooltip"

Checklist

[x] npm test passes
[x] tests and/or benchmarks are included
[x] commit message follows commit guidelines

Description of change

tooltip 内内置 class="g2-tooltip" 逻辑

已经 CR 完。


text:
G6添加节点后又删除会报Cannot read properties of undefined (reading 'length')

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link

Steps to reproduce
点击按钮，会添加一个节点然后删除节点

Environment
Info

g6
3.7.0

System
mac

Browser
-

该BUG到3.8.3就不会复现了，想知道在不升级G6版本的情况下有办法解决这个问题吗？


text:
antd pro中使用G6 以及@antv/g6-react-node 提示找不到 @antv/g6-react-node

G6 Version
4.x
Describe the bug
在antdPro 中使用G6时 安装 @antv/g6-react-node 然后运行项目就会提示下面的错误
不知道这是什么原因造成的，是我需要修改一个其他的配置吗？

Your Example Website or App
__
Steps to Reproduce the Bug or Issue
__
Expected behavior
__
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]
G6:^4.8.21
@antv/g6-react-node: "^1.4.6"

Additional context
No response

你好，该问题已经解决，请重新尝试安装

你好，请尝试更新到最新的 1.4.8 版本

你好，请尝试更新到最新的 1.4.8 版本
你好 抱歉 我没有找到 1.4.8版本
版本列表中只有以下的版本

authors: Aarebecca qiuyulc


text:
drag-element After shadow is enabled, there is an afterimage during dragging

Describe the bug / 问题描述

随意拖拽一个节点，在画布上出现残影
Reproduction link / 重现链接
No response
Steps to Reproduce the Bug or Issue / 重现步骤
No response
G6 Version / G6 版本
🆕 5.x
Operating System / 操作系统
Windows
Browser / 浏览器
Chrome
Additional context / 补充说明
No response

参考
https://github.com/antvis/G6/issues/5257#issuecomment-1852026557
常见使用问题汇总


text:
some issues with angular 9

i start my first angular 9 test with L7, here is my configuration
    "@angular/animations": "^9.0.0-rc.7",
    "@angular/common": "^9.0.0-rc.7",
    "@angular/compiler": "^9.0.0-rc.7",
    "@angular/core": "^9.0.0-rc.7",
    "@angular/forms": "^9.0.0-rc.7",
    "@angular/platform-browser": "^9.0.0-rc.7",
    "@angular/platform-browser-dynamic": "^9.0.0-rc.7",
    "@angular/platform-server": "^9.0.0-rc.7",
    "@angular/router": "^9.0.0-rc.7",
    "@antv/l7": "^2.0.0-beta.26",
    "typescript": "~3.6.4",

import { Component } from '@angular/core';
import * as L7 from '@antv/l7/dist/l7';
@Component({
  selector: 'tda-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.scss']
})
export class AppComponent {
  constructor(){
    const scene = new L7.Scene({
      id: 'map',
      map: new L7.GaodeMap({
        style: 'dark',
        center: [110.770672, 34.159869],
        pitch: 45,
      }),
    });
}
}

The code above works well.
then I added more tests in the constructor
    fetch('https://gw.alipayobjects.com/os/rmsportal/oVTMqfzuuRFKiDwhPSFL.json')
      .then(res => res.json())
      .then(data => {
        console.log(data);
        const pointLayer = new L7.PointLayer({})
          .source(data.list, {
            parser: {
              type: 'json',
              x: 'j',
              y: 'w'
            }
          })
          .shape('cylinder')
          .size('t', function(level) {
            return [ 1, 2, level * 2 + 20 ];
          })
          .color('t', [
            '#094D4A',
            '#146968',
            '#1D7F7E',
            '#289899',
            '#34B6B7',
            '#4AC5AF',
            '#5FD3A6',
            '#7BE39E',
            '#A1EDB8',
            '#CEF8D6'
          ])
          .style({
            opacity: 1.0
          });
        scene.addLayer(pointLayer);

if I uncomment the line 'scene.addLayer(pointLayer)', then

I i added the line  'scene.addLayer(pointLayer)', then

update L7 version to 2.0.0-beta.28
Angular demo
if also you have problem ,you  can give me a codesandebox demo

authors: helxsz lzxue


text:
X6希望支持导出PDF

1、X6希望支持导出PDF
2、使用画布居中，再使用positionContent("top")有时候会丢失一点点内容，但导出图片却不会，能用导出图片的方法打印吗？

可以使用 jspdf 来实现，可以参考下面 demo：https://codesandbox.io/s/pdf-g7hwy。
第二个问题可提供一个可复现的在线 demo？

可以使用 jspdf 来实现，可以参考下面 demo：https://codesandbox.io/s/pdf-g7hwy。
第二个问题可提供一个可复现的在线 demo？

用JSPDF要设图片大小，导出的图片能确定大小吗？
第二个问题偶然性发生，我还是继续排查自己的原因

这个大小是定义绘制在pdf上的大小，可以根据实际情况设置。

这个大小是定义绘制在pdf上的大小，可以根据实际情况设置。

你的意思我能理解成每次都根据实际情况设置，而不是根据获得的图片大小设置？
如果是这样的话就好像不对吧，我能自定义，但用户不能啊。

我理解你是要知道原来图片的宽度和高度吧，https://codesandbox.io/s/pdf-g7hwy?file=/src/app.tsx 看下这个

authors: NewByVector looklookseesee


text:
3.0.x 版本tickline位置问题

G2 Version:
Platform:
Mini Showcase(like screenshots):
CodePen Link:

https://antv.alipay.com/zh-cn/g2/3.x/demo/line/series.html
3.0.x 版本cat 类型坐标轴 tickLine 的默认位置（tickLabel 在 tickLine 中间）
请问可不可以设置为原来的tickLabel与tickLine与图像中的点在一条直线对齐的 如何配置？

官网的例子居然也是这样的问题
我自己写的组件 开发环境没问题 到了生产环境也出现这样的问题
这应该不是配置的问题 就是 bug

这是 3.1.x 开始的样式，cat 类型的字段 tickLabel 位于两个 tickLine 之间。

然而 绝云老哥 文档中没有找到配置的项啊 就是3.1.x以后的版本就只能tickLabel 位于两个 tickLine 之间了呗 不能配置成以前的那种tickLabel与tickLine与图像中的点在一条直线对齐的？

上图X轴两侧各多出来一个刻度

上图官网的例子12个月份坐标轴等分6份时最后的月份那里又不能等分了
所以这个3.1.x的X轴的tickLabel与tickLine的位置问题我觉得是有BUG

两侧多一个刻度和 设置了 tickCount 的地方是个 bug，这周会发掉。

@paleface001

但是tickLabel与tickLine与图像中的点在一条直线对齐 这个是没有配置了吗 只能默认tickLabel 位于两个 tickLine 之间吗

最好默认还是 3.0 的样式，开放个属性可以让用户选择 tick 的位置就好了，这种文本在 tick 中间的方式在设置了 tickCount 后，确实会出现问题。

authors: dxq613 hawaiiey leungwensen muyucanqiu


text:
train_phocnet

Hello!
I would like to ask if it is intended for -batch 10 -epoch 100 to cause the model to train less than 100 epochs. I have ran the following:
python train_phocnet.py -id IAM_phoc -mp IAM_train.csv -vmap IAM_valid.csv -tr IAM_train -vi IAM_valid -batch 10 -epoch 100
This causes the model to train over 4 epochs, as shown in the attached.

I believe this is due to line 166, EPOCHS=70000//STEPS+10000//STEPS+1, as STEPS is dependent on the length of the data frame and batch size.
May I know if this is intended?
Thanks for the help.

Okay, thank you for the clarification.


text:
About Rigidity masks

Hi @anuragranj !
I am training the model using all the pre-trained models that you published online. However, I do not get proper rigidity masks. Do you know why this is happening?  As I understand, Rigidity masks are the subtraction of rigid flow with flow generated by the flownet with some threshold(I havent chaned the code at all). Below is the picture:

Also, Could you share the optimizer checkpoint at the time of your training, so that I could visualize all the results at once?

Can you share the hyperparameters you are using for training? Also, are the depth, camera and flow networks trained well?

Thankyou @anuragranj ! Unfortunately, I cannot share with you immediately as I am not at my desk. However, I will do it as soon. If possible, meanwhile, could you share your tensorboard events file wherein I can see the rigid masks, please.

Also, are the depth, camera and flow networks trained well?

I have plugged in your pretrained depth, camera, and flow models. So, Isnt it already trained?

I don't have access to tensorboard event files. Could  you try the following hyperparameters? This worked for me.
python3 train.py DATA_PATH --sequence-length 5 --dispnet DispResNet6 \
--posenet PoseNetB6 --masknet MaskNet6 --flownet Back2Future --pretrained-disp \
pretrained/dispnet_model_best.pth.tar --pretrained-pose pretrained/posenet_model_best.pth.tar \
--pretrained-mask pretrained/masknet_model_best.pth.tar --pretrained-flow \
pretrained/flownet_model_best.pth.tar -b4 -m0.02 -pf 0.5 -pc 1.0 -s0.2 -c0.3 \
--epoch-size 100 --log-output -f 0 --nlevels 6 --lr 1e-6 -wssim 0.5 --with-flow-gt \
--with-depth-gt --epochs 100 --fix-flownet --fix-dispnet --fix-posenet \
--smoothness-type edgeaware --THRESH 0.999

Hi @anuragranj !
I used the default THRESH value before.
However, I ran test_flow.py script with the same THRESH value of 0.999. I expect the rigidity mask to be as shown in the paper:

However, while running the command:
`python test_flow.py  --pretrained-disp /cdtemp/ezorfa/cc-master/checkpoints/dispnet_k.pth.tar  --pretrained-pose /cdtemp/ezorfa/cc-master/checkpoints/posenet.pth.tar --pretrained-flow /cdtemp/ezorfa/cc-master/checkpoints/back2future.pth.tar   --pretrained-mask /cdtemp/ezorfa/cc-master/checkpoints/masknet.pth.tar --THRESH 0.999 --kitti-dir kitti2015/dataset --output-dir kitti2015/output/'  I get outputs of the rigidity mask like:

How can I get the mask outputs like the one described in the paper?
Thankyou!

@anuragranj  Is such a mask expected when camera motion is present? Also, could you please check the parameter values used in the above command test_flow.py and tell how to reproduce rigidity_flow_census masks.  I am only interested to find the moving objects by making use of rigidity flow and optical flow obtained by flownet.
Thankyou!

To get the masks, you should be running test_mask.py with default  --THRESH values. The test_flow.py functions outputs the masks that are optimal for best flow, but not for best masks.

Okay, Could you please assert if such masks like the picture above are expected?

Masks like Figure 1. in the paper are expected if you use test_mask.py with default --THRESH, which is .94.

Hmm. I am testing now. Thankyou for the guidance.

The visualization corresponds to the following line:
https://github.com/anuragranj/cc/blob/master/test_mask.py#L173
It is saved as 'val Rigidity Mask Census' in tensorboard events.

yeah, I spotted that and made my own script to vizualize only moving objects but somehow dint see the file test_mask.py existing. Sorry about that and will come back to you.

authors: anuragranj ezorfa

repo: anuragranj/cc


text:
Added attributes to style input field

Fixes:  #1986
Description
Added background to the textarea and other attributes to style the input field in the book exchange section

Type of PR

[x ] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
before:

after:

Checklist:

[ x] I have made this change from my own.
[ ] I have taken help from some online resources.
[ x] My code follows the style guidelines of this project.
[ x] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x ] My changes generate no new warnings.
[ x] I have tested the changes thoroughly before submitting this pull request.
[x ] I have provided relevant issue numbers and screenshots after making the changes.

This branch has conflicts that must be resolved! resolve and create another PR!
Conflicting files
assets/css/style.css

authors: Dishaaa-T huamanraj


text:
Change Task

Requirements

Add two status task lists
Add buttons to change tasks
Delete a task
Edit a task
Add drop down menu to change task status
Move task based on status task list

completed

repo: anusontarangkul/taskinator


text:
Initial work to import openapi 3

Implements OpenAPI 3.0 as a new importer.
Unfortunately I havnt found a go pkg which nartivly handles both swagger and openapi so duplicating lots of work :(
@anz-bank/sysl-developers

Only one commit for all of this?

repo: anz-bank/sysl


text:
Fix and reenable local cluster test test_invalid_forks_persisted_on_restart

Investigate why it fails with the unrelated contact-info migration: https://github.com/anza-xyz/agave/pull/823#discussion_r1575382916
and fix.

Adding to this. The related local cluster test: test_invalid_forks_persisted_on_restart has become flaky as a result of the same PR: #823. Created another PR to ignore this test: https://github.com/anza-xyz/agave/pull/1130

authors: AshwinSekar gregcusack

repo: anza-xyz/agave


text:
ci: Re-enable downstream SPL CLI tests and check

Problem
Breaking changes in #1185 were not detected due to the lack of downstream testing for CLIs.
Summary of Changes
There are already tests, but they're disabled for some reason. I couldn't remember why, and it worked locally, so let's enable them!

Since the CLI tests take 20 minutes, I don't think it should be too much of an extra burden to add them, but let me know what you think!

Thank you! It was super lazy/ignorant of me not to look and see that was already there 😳

If it's any consolation, I didn't check why we disabled it :sweat_smile:

authors: CriesofCarrots joncinque


text:
equest had invalid authentication credentials

Thank you for making this, I'm trying to implement this in my app but every time when i hit record I'm getting error from google
equest had invalid authentication credentials. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project

can you explain how can fix this
thank you

you try api create key credentials new Authenticating to a Cloud API Service

how can solve this issue i just include   in  my index.html file but still getting this issue

Did you edit or this file recorder.js ?

repo: aofdev/vue-pwa-speech


text:
Add from(Publisher publisher) creational methods to all types

Allow for smooth integration across projects that implement reactive-streams
e.g.
  ListX.from(reactorFlux());
  SetX.from(rxObservable());

Added in 1.0.0-RC2.2 for Collections & Streams

repo: aol/cyclops-react


text:
moloch services fail to start after rebooting on Ubuntu 16.04.1

Elasticsearch version: 2.4.4
Moloch version: 0.17.0
OS name and version: Ubuntu 16.04.1 (amd64)
How was Moloch built/installed: deb
# Install Oracle Java
add-apt-repository ppa:webupd8team/java
apt-get update
apt-get install oracle-java8-installer

# Install elasticsearch
dpkg -i elasticsearch-2.4.4.deb
systemctl enable elasticsearch.service
systemctl start elasticsearch.service

# Install moloch
dpkg -i moloch_0.17.0-1_amd64.deb
apt-get install -f
/data/moloch/bin/Configure
/data/moloch/db/db.pl http://localhost:9200 init
/data/moloch/bin/moloch_add_user.sh admin admin admin --admin
systemctl enable molochviewer.service molochcapture.service
systemctl start molochviewer.service molochcapture.service

Provide logs, stack traces and steps to reproduce:
Immediately after installation, it works normally. But rebooting system, moloch services failed to start. I think that it's caused by slow starting elasticsearch. I have introduced the following workarounds. Is there another method to solve this problem?
Workaround
Changelogs

Add elasticsearch.service to After/Requires
Add sleep command to ExecStart

# /etc/systemd/system/molochviewer.service
[Unit]
Description=Moloch Viewer
After=network.target elasticsearch.service
Requires=network.target elasticsearch.service

[Service]
Type=simple
Restart=always
StandardOutput=tty
ExecStart=/bin/sh -c 'sleep 5 && /data/moloch/bin/node viewer.js -c /data/moloch/etc/config.ini >> /data/moloch/logs/viewer.log 2>&1'
WorkingDirectory=/data/moloch/viewer

[Install]
WantedBy=multi-user.target

# /etc/systemd/system/molochcapture.service
[Unit]
Description=Moloch Capture
After=network.target elasticsearch.service
Requires=network.target elasticsearch.service

[Service]
Type=simple
Restart=always
StandardOutput=tty
ExecStartPre=-/sbin/ethtool -G ens3 rx 4096 tx 4096
ExecStartPre=-/sbin/ethtool -K ens3 rx off tx off
ExecStartPre=-/sbin/ethtool -K ens3 sg off
ExecStartPre=-/sbin/ethtool -K ens3 tso off
ExecStartPre=-/sbin/ethtool -K ens3 ufo off
ExecStartPre=-/sbin/ethtool -K ens3 gso off
ExecStartPre=-/sbin/ethtool -K ens3 gro off
ExecStartPre=-/sbin/ethtool -K ens3 lro off
ExecStart=/bin/sh -c 'sleep 5 && /data/moloch/bin/moloch-capture -c /data/moloch/etc/config.ini >> /data/moloch/logs/capture.log 2>&1'
WorkingDirectory=/data/moloch
LimitCORE=infinity

[Install]
WantedBy=multi-user.target

Logs
/data/moloch/logs/capture.log
Jan 24 15:51:52 main.c:595 main(): THREAD 0x7f05f630e780
Jan 24 15:51:52 main.c:163 parse_args(): WARNING: gethostname doesn't return a fully qualified name and getdomainname failed, this may cause issues when viewing pcaps - ubuntu
Jan 24 15:51:52 config.c:184 moloch_config_int(): INFO: Reseting pcapWriteSize since 262143 is less then the min 262144
Jan 24 15:51:52 main.c:573 moloch_mlockall_init(): WARNING: memlock in limits.conf must be unlimited or at least 4000000, currently 64
Jan 24 15:51:52 http.c:194 moloch_http_send_sync(): libcurl failure http://localhost:9200/dstats/version/version/_source error 'Couldn't connect to server'
Jan 24 15:51:52 db.c:1713 moloch_db_check(): ERROR - Couldn't load version information, database might be down or out of date.  Run "db/db.pl host:port upgrade"
Jan 24 15:51:53 main.c:595 main(): THREAD 0x7fae28059780
Jan 24 15:51:53 main.c:163 parse_args(): WARNING: gethostname doesn't return a fully qualified name and getdomainname failed, this may cause issues when viewing pcaps - ubuntu
Jan 24 15:51:53 config.c:184 moloch_config_int(): INFO: Reseting pcapWriteSize since 262143 is less then the min 262144
Jan 24 15:51:53 main.c:573 moloch_mlockall_init(): WARNING: memlock in limits.conf must be unlimited or at least 4000000, currently 64
Jan 24 15:51:53 http.c:194 moloch_http_send_sync(): libcurl failure http://localhost:9200/dstats/version/version/_source error 'Couldn't connect to server'
Jan 24 15:51:53 db.c:1713 moloch_db_check(): ERROR - Couldn't load version information, database might be down or out of date.  Run "db/db.pl host:port upgrade"
Jan 24 15:51:53 main.c:595 main(): THREAD 0x7f8297f64780
Jan 24 15:51:53 main.c:163 parse_args(): WARNING: gethostname doesn't return a fully qualified name and getdomainname failed, this may cause issues when viewing pcaps - ubuntu
Jan 24 15:51:53 config.c:184 moloch_config_int(): INFO: Reseting pcapWriteSize since 262143 is less then the min 262144
Jan 24 15:51:53 main.c:573 moloch_mlockall_init(): WARNING: memlock in limits.conf must be unlimited or at least 4000000, currently 64
Jan 24 15:51:53 http.c:194 moloch_http_send_sync(): libcurl failure http://localhost:9200/dstats/version/version/_source error 'Couldn't connect to server'
Jan 24 15:51:53 db.c:1713 moloch_db_check(): ERROR - Couldn't load version information, database might be down or out of date.  Run "db/db.pl host:port upgrade"
Jan 24 15:51:53 main.c:595 main(): THREAD 0x7f6c61107780
Jan 24 15:51:53 main.c:163 parse_args(): WARNING: gethostname doesn't return a fully qualified name and getdomainname failed, this may cause issues when viewing pcaps - ubuntu
Jan 24 15:51:53 config.c:184 moloch_config_int(): INFO: Reseting pcapWriteSize since 262143 is less then the min 262144
Jan 24 15:51:53 main.c:573 moloch_mlockall_init(): WARNING: memlock in limits.conf must be unlimited or at least 4000000, currently 64
Jan 24 15:51:53 http.c:194 moloch_http_send_sync(): libcurl failure http://localhost:9200/dstats/version/version/_source error 'Couldn't connect to server'
Jan 24 15:51:53 db.c:1713 moloch_db_check(): ERROR - Couldn't load version information, database might be down or out of date.  Run "db/db.pl host:port upgrade"
Jan 24 15:51:54 main.c:595 main(): THREAD 0x7f1d4028b780
Jan 24 15:51:54 main.c:163 parse_args(): WARNING: gethostname doesn't return a fully qualified name and getdomainname failed, this may cause issues when viewing pcaps - ubuntu
Jan 24 15:51:54 config.c:184 moloch_config_int(): INFO: Reseting pcapWriteSize since 262143 is less then the min 262144
Jan 24 15:51:54 main.c:573 moloch_mlockall_init(): WARNING: memlock in limits.conf must be unlimited or at least 4000000, currently 64
Jan 24 15:51:54 http.c:194 moloch_http_send_sync(): libcurl failure http://localhost:9200/dstats/version/version/_source error 'Couldn't connect to server'
Jan 24 15:51:54 db.c:1713 moloch_db_check(): ERROR - Couldn't load version information, database might be down or out of date.  Run "db/db.pl host:port upgrade"

/data/moloch/logs/viewer.log
Elasticsearch WARNING: 2017-01-24T06:51:58Z
  Unable to revive connection: http://localhost:9200/

Elasticsearch WARNING: 2017-01-24T06:51:58Z
  No living connections

{ [Error: No Living connections] message: 'No Living connections' } undefined
/data/moloch/viewer/db.js:73
    if (data.version.number.match(/^(2.[0-3]|1|0)/)) {
            ^

TypeError: Cannot read property 'version' of undefined
    at /data/moloch/viewer/db.js:73:13
    at respond (/data/moloch/viewer/node_modules/elasticsearch/src/lib/transport.js:308:9)
    at sendReqWithConnection (/data/moloch/viewer/node_modules/elasticsearch/src/lib/transport.js:207:7)
    at next (/data/moloch/viewer/node_modules/elasticsearch/src/lib/connection_pool.js:213:7)
    at nextTickCallbackWith0Args (node.js:436:9)
    at process._tickCallback (node.js:365:13)
Elasticsearch ERROR: 2017-01-24T06:55:46Z
  Error: Request error, retrying
  GET http://localhost:9200/ => connect ECONNREFUSED 127.0.0.1:9200
      at Log.error (/data/moloch/viewer/node_modules/elasticsearch/src/lib/log.js:225:56)
      at checkRespForFailure (/data/moloch/viewer/node_modules/elasticsearch/src/lib/transport.js:240:18)
      at HttpConnector.<anonymous> (/data/moloch/viewer/node_modules/elasticsearch/src/lib/connectors/http.js:162:7)
      at ClientRequest.wrapper (/data/moloch/viewer/node_modules/elasticsearch/node_modules/lodash/index.js:3095:19)
      at emitOne (events.js:77:13)
      at ClientRequest.emit (events.js:169:7)
      at Socket.socketErrorListener (_http_client.js:269:9)
      at emitOne (events.js:77:13)
      at Socket.emit (events.js:169:7)
      at emitErrorNT (net.js:1267:8)

Elasticsearch WARNING: 2017-01-24T06:55:46Z
  Unable to revive connection: http://localhost:9200/

Elasticsearch WARNING: 2017-01-24T06:55:46Z
  No living connections

{ [Error: No Living connections] message: 'No Living connections' } undefined
/data/moloch/viewer/db.js:73
    if (data.version.number.match(/^(2.[0-3]|1|0)/)) {
            ^

TypeError: Cannot read property 'version' of undefined
    at /data/moloch/viewer/db.js:73:13
    at respond (/data/moloch/viewer/node_modules/elasticsearch/src/lib/transport.js:308:9)
    at sendReqWithConnection (/data/moloch/viewer/node_modules/elasticsearch/src/lib/transport.js:207:7)
    at next (/data/moloch/viewer/node_modules/elasticsearch/src/lib/connection_pool.js:213:7)
    at nextTickCallbackWith0Args (node.js:436:9)
    at process._tickCallback (node.js:365:13)

It is not recommended that ES and capture be on the same hosts, so definitely can't add the requires elasticsearch.
The sleep is an option, but on a properly setup system it would delay capturing by 5 seconds.  You could move it after I guess.  (BTW there is a RestartSec= setting so you don't have to do a sleep, although would need to double check that it is AFTER a failure)
Could probably mess around with   StartLimitInterval= or  StartLimitBurst= but I have no experience with those.
Seems like this could just be an FAQ entry.

authors: awick mikoim

repo: aol/moloch


text:
Question/feature: how to adjust the number of articles shown on main page?

Currently on my computer hackernews_tui shows just 19 articles on "front page". How can I make it show more articles?

Hi, I guess it shows 19 articles for the first page. You can press n (default binding for next_page command) to go the  second page.

If you want to adjust the number of items each page show, I'm afraid hackernews_tui doesn't support that feature atm.


text:
GoToArtist: automatically select artist if only one is present on selected track

Is your feature request related to a problem? Please describe.
I always have to 'choose' a single artist, which is the case for 99% of the time I use this action.
Describe the solution you'd like
Automatically select artist if only one is present on selected track.
Additional context
Example of artist selection screen after doing ShowActionsOnSelectedItem -> GoToArtist:

After inspecting source code, I found that this was resolved with https://github.com/aome510/spotify-player/pull/507
Closed


text:
Bunch of threads throwing IllegalStateException

I was running Bulk Rwalk test on a Uno cluster with 2 tservers. I killed one of the tservers (with kill -9) and then restarted it (with accumulo-cluster start-tservers). A little while later 50+ of these errors flooded the Monitor.

java.lang.RuntimeException: java.lang.IllegalStateException: Unexpected family chopped
	at org.apache.accumulo.tserver.AssignmentHandler.run(AssignmentHandler.java:141)
	at org.apache.accumulo.tserver.ActiveAssignmentRunnable.run(ActiveAssignmentRunnable.java:63)
	at org.apache.accumulo.core.trace.TraceWrappedRunnable.run(TraceWrappedRunnable.java:52)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at org.apache.accumulo.core.trace.TraceWrappedRunnable.run(TraceWrappedRunnable.java:52)
	at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: java.lang.IllegalStateException: Unexpected family chopped
	at org.apache.accumulo.core.metadata.schema.TabletMetadata.convertRow(TabletMetadata.java:403)
	at org.apache.accumulo.core.metadata.schema.TabletsMetadata$Builder.lambda$buildNonRoot$8(TabletsMetadata.java:216)
	at com.google.common.collect.Iterators$6.transform(Iterators.java:829)
	at com.google.common.collect.TransformedIterator.next(TransformedIterator.java:52)
	at com.google.common.collect.Iterators$5.computeNext(Iterators.java:672)
	at com.google.common.collect.AbstractIterator.tryToComputeNext(AbstractIterator.java:146)
	at com.google.common.collect.AbstractIterator.hasNext(AbstractIterator.java:141)
	at java.base/java.util.Iterator.forEachRemaining(Iterator.java:132)
	at java.base/java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:484)
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:474)
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:913)
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:578)
	at org.apache.accumulo.core.metadata.schema.AmpleImpl.readTablet(AmpleImpl.java:47)
	at org.apache.accumulo.core.metadata.schema.Ample.readTablet(Ample.java:144)
	at org.apache.accumulo.tserver.AssignmentHandler.run(AssignmentHandler.java:109)
	... 6 more

I think these errors are related to a Chop compaction that was taking place when the tserver was killed.

2022-09-26T10:36:12,167 [manager.Manager] INFO : Asking localhost:10000[100007556480005] to chop a;r0265c;r01179
tserver2_ip-10-113-14-231.log:2022-09-26T10:36:24,770 [tablet.files] DEBUG: Compacting a;r0265c;r01179 on i.default.small for CHOP from [I00002vg.rf, I00002it.rf, I00002h5.rf, C00001an.rf, I00002xr.rf, I0000303.rf, I00002zo.rf, I00002vh.rf, I00002hd.rf, I00002ob.rf] size 36 KB
tserver2_ip-10-113-14-231.log:2022-09-26T10:36:31,369 [tablet.files] DEBUG: Compacted a;r0265c;r01179 for CHOP created hdfs://localhost:8020/accumulo/tables/a/t-00002pq/C000033u.rf from [I00002oe.rf, I00002nv.rf, I00002nw.rf, I00002no.rf, I00002oh.rf, I00002nb.rf, I00002nj.rf, I00002oc.rf, I00002m2.rf]
...
2022-09-26T10:36:36,121 [manager.Manager] ERROR: unable to get tablet server status localhost:10000[100007556480005]
2022-09-26T10:37:03,458 [manager.Manager] DEBUG: 85 assigned to dead servers: ...  a;r0265c;r01179
2022-09-26T10:37:03,468 [tablet.location] DEBUG: Suspended a;r0265c;r01179 to localhost:10000 at 3696264 ms with 1 walogs
...
2022-09-26T10:37:37,537 [manager.Manager] INFO : New servers: [localhost:10000[10000755648001d]]
2022-09-26T10:37:37,539 [manager.EventCoordinator] INFO : There are now 2 tablet servers
2022-09-26T10:37:37,683 [tablet.location] DEBUG: Assigned a;r0265c;r01179 to localhost:10000[10000755648001d]
2022-09-26T10:37:41,266 [manager.Manager] ERROR: localhost:10000 reports assignment failed for tablet a;r0265c;r01179

This is similar to problems we had with other Metadata checks in https://github.com/apache/accumulo/pull/2574 and https://github.com/apache/accumulo/issues/2667

Looking at the code  that function is used by Ample to convert a metadata table row to POJO.  I think the function just does not handle the chopped row and that needs to be added.

@milleruntime I could submit a PR to fix this if you would like, just let me know.

@milleruntime I could submit a PR to fix this if you would like, just let me know.

I am looking at it today. I see where the column family needs to be added to the switch case but I wasn't sure if there was anything else special that we need to do within TabletMetadata. Like, do we need to set [this chopped boolean] to true(https://github.com/apache/accumulo/blob/cf57e343de77f416de10c22e0ed05ef37f433b36/core/src/main/java/org/apache/accumulo/core/metadata/schema/TabletMetadata.java#L286) ? Or can i just create a boolean for chopped?

I am looking at it today. I see where the column family needs to be added to the switch case but I wasn't sure if there was anything else special that we need to do within TabletMetadata. Like, do we need to set [this chopped boolean] to true(

Looking at the code inside TabletLocationState and how it being used there, it does not matter if chopped is passed. So can probably leave that alone.  I think its unrelated to fixing this issue, but that code that uses TabletLocationState should probably be refactored to avoid not passing in uneeded params to computer something.

I think its unrelated to fixing this issue, but that code that uses TabletLocationState should probably be refactored to avoid not passing in uneeded params to computer something.

Going to try restating above w/o double negatives.  Thinking we should refactor the code in question to have a function that computes tablet state only passing in parameters that are needed by the computation.  That refactor could possibly be a separate PR from fixing this.

authors: keith-turner milleruntime

repo: apache/accumulo


text:
Reuse of variable with label constraint in concurrent MATCH clauses does not error out

Describe the bug
Reuse of variable with label constraint in concurrent MATCH clauses neither filter nor error out 'multiple labels for variable not supported'
How are you accessing AGE (Command line, driver, etc.)?

Command Line

What data setup do we need to do?
...
SELECT * from cypher('graph', $$
  CREATE ({name:'bobby'}),
         (:Person {name:'bobby'}),
         (:Bobby {name:'bobby'})
$$) as (a agtype);
...

What is the command that caused the error?
SELECT * from cypher('graph', $$
      MATCH (u)
      MATCH (u:Person)
      RETURN u;
$$) as (a agtype);

                                          a
-------------------------------------------------------------------------------------
 {"id": 281474976710657, "label": "", "properties": {"name": "bobby"}}::vertex
 {"id": 844424930131969, "label": "Person", "properties": {"name": "bobby"}}::vertex
 {"id": 1125899906842625, "label": "Bobby", "properties": {"name": "bobby"}}::vertex
(3 rows)

Expected behavior
Most probably, should have error out like in MATCH (a),(a:Person) saying ERROR:  multiple labels for variable 'u' are not supported,
Environment (please complete the following information):

Version: Latest

When you create the vertex with "properties" : {"name": "bobby"} and "Person" as a label, it creates a new label table for vertices and inserts that vertex in that table. Also it assigns an ID that is generated according to the label table ID. When you create another vertex and assign it a different label (in this case "Bobby") the same procedure happens but a different ID is assigned on the vertex because the table that the vertex will be inserted is the "Bobby" label table (that has a different ID).
Since those 2 vertices have different ID's and exist in different tables the above query works as intented. Implementing multiple labels and actually fetching one vertex with multiple labels needs a complete refactor, but for now the way creating a vertex and assigning a label works, I wouldn't call it a bug. Tell me if I missed something

Ideally, the second MATCH should work as an intersection over the results of the first MATCH. So the result should be only one  vertex
" {"id": 844424930131969, "label": "Person", "properties": {"name": "bobby"}}::vertex".
This is how it works in Neo4j as well, and the second MATCH does perform intersection in case of properties with AGE as well. As demonstrated here
test=# SELECT * from cypher('test', $$
  CREATE ({name:'bobby', age:25}),
         (:Person {name:'bobby'}),
         (:Bobby {name:'bobby', age:25, cigar:2})
$$) as (a agtype);
 a
---
(0 rows)

test=# SELECT * from cypher('test', $$
      MATCH (u {name:'bobby'})
      MATCH (u {cigar:2})
      RETURN u
$$) as (a agtype);
                                                     a
------------------------------------------------------------------------------------------------------------
 {"id": 1407374883553282, "label": "Bobby", "properties": {"age": 25, "name": "bobby", "cigar": 2}}::vertex
(1 row)

Another interesting result is this
test=# SELECT * from cypher('test', $$
      MATCH (u:Bobby {name:'bobby'})
      MATCH (u:Person)
      RETURN u
$$) as (a agtype);
                                                     a
------------------------------------------------------------------------------------------------------------
 {"id": 1407374883553281, "label": "Bobby", "properties": {"name": "bobby"}}::vertex
 {"id": 1407374883553282, "label": "Bobby", "properties": {"age": 25, "name": "bobby", "cigar": 2}}::vertex
(2 rows)

And this
test=# SELECT * from cypher('test', $$
      MATCH (u:Bobby {name:'bobby'})
      MATCH (u:Person {age:25})
      RETURN u
$$) as (a agtype);
                                                     a
------------------------------------------------------------------------------------------------------------
 {"id": 1407374883553282, "label": "Bobby", "properties": {"age": 25, "name": "bobby", "cigar": 2}}::vertex
(1 row)

It appears that AGE performs filtration based on the first MATCH clause, ignores the label (if any) of the second clause, only performs filtration based on the given properties and returns the result.
It should instead give an error regarding support of multiple labels (as of now).

So it does perform filtering according to the properties but not on the labels. I stand corrected then, I agree with you that this is actually a bug, an interesting one i might add.

Ideally, the second MATCH should work as an intersection over the results of the first MATCH. So the result should be only one vertex " {"id": 844424930131969, "label": "Person", "properties": {"name": "bobby"}}::vertex". This is how it works in Neo4j as well, and the second MATCH does perform intersection in case of properties with AGE as well. As demonstrated here
test=# SELECT * from cypher('test', $$
  CREATE ({name:'bobby', age:25}),
         (:Person {name:'bobby'}),
         (:Bobby {name:'bobby', age:25, cigar:2})
$$) as (a agtype);
 a
---
(0 rows)

test=# SELECT * from cypher('test', $$
      MATCH (u {name:'bobby'})
      MATCH (u {cigar:2})
      RETURN u
$$) as (a agtype);
                                                     a
------------------------------------------------------------------------------------------------------------
 {"id": 1407374883553282, "label": "Bobby", "properties": {"age": 25, "name": "bobby", "cigar": 2}}::vertex
(1 row)

Another interesting result is this
test=# SELECT * from cypher('test', $$
      MATCH (u:Bobby {name:'bobby'})
      MATCH (u:Person)
      RETURN u
$$) as (a agtype);
                                                     a
------------------------------------------------------------------------------------------------------------
 {"id": 1407374883553281, "label": "Bobby", "properties": {"name": "bobby"}}::vertex
 {"id": 1407374883553282, "label": "Bobby", "properties": {"age": 25, "name": "bobby", "cigar": 2}}::vertex
(2 rows)

And this
test=# SELECT * from cypher('test', $$
      MATCH (u:Bobby {name:'bobby'})
      MATCH (u:Person {age:25})
      RETURN u
$$) as (a agtype);
                                                     a
------------------------------------------------------------------------------------------------------------
 {"id": 1407374883553282, "label": "Bobby", "properties": {"age": 25, "name": "bobby", "cigar": 2}}::vertex
(1 row)

It appears that AGE performs filtration based on the first MATCH clause, ignores the label (if any) of the second clause, only performs filtration based on the given properties and returns the result.
It should instead give an error regarding support of multiple labels (as of now).
Hence, yes, I would call it a bug.
Given that in case 2 labels are provided in 2 different MATCH clauses, it does not perform an intersection of the 2 MATCH clauses, instead completely ignores the label provided in the second MATCH clause.

Thanks for your valuable response.
In addition to your response, the below query also filters out results in neo4j
MATCH (u),(u:Person) RETURN u
But this same query errors out in AGE saying ERROR:  multiple labels for variable 'u' are not supported
So the query
MATCH (u) MATCH (u:Person) RETURN should also error out the same until it is supported.

MATCH (u),(u:Person) RETURN u
But this same query errors out in AGE saying ERROR: multiple labels for variable 'u' are not supported

I'm not sure why, but this query does not give an error for me. I'm using AGE 1.3.0 and it gives this output.
test=# SELECT * from cypher('test', $$
      MATCH (u)
      MATCH (u:Person)
      RETURN u
$$) as (a agtype);
                                                     a
------------------------------------------------------------------------------------------------------------
 {"id": 281474976710657, "label": "", "properties": {"name": "bobby"}}::vertex
 {"id": 281474976710658, "label": "", "properties": {"age": 25, "name": "bobby"}}::vertex
 {"id": 1125899906842625, "label": "Person", "properties": {"name": "bobby"}}::vertex
 {"id": 1125899906842626, "label": "Person", "properties": {"name": "bobby"}}::vertex
 {"id": 1407374883553281, "label": "Bobby", "properties": {"name": "bobby"}}::vertex
 {"id": 1407374883553282, "label": "Bobby", "properties": {"age": 25, "name": "bobby", "cigar": 2}}::vertex
(6 rows)

When I did the work for variable reuse I only did it for the properties. I should have also considered the labels, but didn't. I would imagine that the same code could be expanded to cover this.

authors: CapnSpek MuhammadTahaNaveed jrgemignani panosfol

repo: apache/age


text:
Fix memory leak issues with age_load

In this patch, all agtype_values, that are allocated while processing the csv file, are free'd.

@muhammadshoaib I added you as a reviewer as well since this part of the code might be familiar with you.

@rafsun42 Good job!


text:
[AMBARI-25209] : Bulk Operation Refresh Configs for selected hosts

What changes were proposed in this pull request?
The changes are related to introducing new bulk operation to perform Refresh Configs for All/Selected hosts. The changes are related to ambari-web.
How was this patch tested?
The patch was tested manually on UI. The screenshots are attached in JIRA: https://issues.apache.org/jira/browse/AMBARI-25209
https://issues.apache.org/jira/secure/attachment/12963746/Screen Shot 2019-03-26 at 2.26.58 PM.png
https://issues.apache.org/jira/secure/attachment/12963747/Screen Shot 2019-03-26 at 2.27.30 PM.png
https://issues.apache.org/jira/secure/attachment/12963748/Screen Shot 2019-03-26 at 2.27.46 PM.png

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5066/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5079/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/5108/
Test PASSed.

authors: asfgit virajjasani

repo: apache/ambari


text:
AMBARI-25519. Ambari builds are failing with a bower error (dpidhaiets)

What changes were proposed in this pull request?
Ambari builds are failing with a bower error
How was this patch tested?
manually
Please review Ambari Contributing Guide before opening a pull request.

retest this please

retest this please

authors: hapylestat pidhaietsdp


text:
request help: how can I migrate existing apisix(standalone, bare-metal) route config to apisix-ingress-controller

Issue description
We're using apisix in production, with lots of route and upstream configs already, planning migrate our services to k8s, is there any way I can generate CRDs file from etcd or admin api?
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long): v1.4.1
your Kubernetes cluster version (output of kubectl version): v1.23.7
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):

This is a very interesting use case.
Can you describe more, or join Slack for a live chat with me ()
https://join.slack.com/t/the-asf/shared_invite/zt-1bjo8bo8k-kp5nk0DaEf28oBoDL07nlQ

repo: apache/apisix-ingress-controller


text:
make ExecutionPlan self contained. (allow for computed properties PlanProperties)

Is your feature request related to a problem or challenge?
if you have some structs that you want to loosely couple to datafusion, it is now impossible.
for example with <37
#[derive(Debug, Clone)]
pub struct MyStruct {
    projection: Arc<Schema>,
    predicate_projection: Arc<Schema>,
    // ...
}

and then you can just implement the trait as one would expect
impl ExecutionPlan for MyStruct {

    fn output_partitioning(&self) -> Partitioning {
        Partitioning::UnknownPartitioning(1)
    }

    fn output_ordering(&self) -> Option<&[datafusion::physical_expr::PhysicalSortExpr]> {
        None
    }
 // ...
}

But with datafusion 37, the datafusion specific components are now leaked into the outer struct, and force the user to modify the struct to contain PlanProperties.
impl ExecutionPlan for MyStruct {
  fn properties(&self) -> &PlanProperties {
    // no possible way to create `&PlanProperties`.
    // the only option is to add it as a field on `MyStruct`
    // which breaks the encapsulation of `ExecutionPlan`
  }
}

This is especially amplified if you want to feature flag datafusion specific functionality.
datafusion <37
#[derive(Debug, Clone)]
pub struct MyStruct {
    // ...
}

impl MyStruct {
  pub fn new() -> Self {
     Self {
       // ..
     }
  }
}

#[cfg(feature = "datafusion")]
impl ExecutionPlan for MyStruct {
  // ...
}

datafusion 37

#[derive(Debug, Clone)]
pub struct MyStruct {
    // ...
    #[cfg(feature = "datafusion")]
    plan_properties: PlanProperties
}

impl MyStruct {
  #[cfg(feature = "datafusion")]
  pub fn new() -> Self {
     let properties = make_properties_somehow():
     Self {
       // ..
       plan_properties: properties
     }
  }
  #[cfg(not(feature = "datafusion"))]
    pub fn new() -> Self {
     Self {
       // ..
     }
  }
}

#[cfg(feature = "datafusion)]
impl ExecutionPlan for MyStruct {
  // ...
}

Describe the solution you'd like
modify ExecutionPlan to not have any methods that return a reference. Specifically ExecutionPlan::properties
If a reference is wanted for performance reasons, we should instead use Cow<PlanProperties>.
Describe alternatives you've considered
NA
Additional context
No response

cc @mustafasrepo  for your comments

Thank you for this idea @universalmind303
Cow<PlanProperties> seems like a good choice to me.
Or else switching the trait back to only expose fields?
We found it awkward to implement the new API in InfluxDB 3.0 as well (though we did make it work)
Putting PlanProperties into the ExecutionPlan was to done to avoid recomputing the same (potentially expensive) properties over and over again as part of https://github.com/apache/arrow-datafusion/pull/9346

I think returning Cow<PlanProperties> makes sense. In this way, struct won't need to implement a new member as you indicated. We might need to update methods of the below
impl ExecutionPlanProperties for Arc<dyn ExecutionPlan> {
    fn output_partitioning(&self) -> &Partitioning {
        self.properties().output_partitioning()
    }

    fn execution_mode(&self) -> ExecutionMode {
        self.properties().execution_mode()
    }

    fn output_ordering(&self) -> Option<&[PhysicalSortExpr]> {
        self.properties().output_ordering()
    }

    fn equivalence_properties(&self) -> &EquivalenceProperties {
        self.properties().equivalence_properties()
    }
}

to work with Cow<PlanProperties> though.

authors: alamb mustafasrepo universalmind303

repo: apache/arrow-datafusion


text:
Support min/max statistics in ParquetTable and ParquetExec

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Min/max column statistics support doing statistics based optimizations, for example join reordering.
Describe the solution you'd like
Add min/max statistics while preparing parquet data.
Describe alternatives you've considered
n/a
Additional context
n/a

I believe this is now supported, but feel free to reopen if I am mistaken

authors: Dandandan tustvold


text:
Bg unsafe2

Which issue does this PR close?
Closes #7191.
Rationale for this change
PR #7192 could be faster.
What changes are included in this PR?
Convert the Vec<> heap into a RawPointer one, so that the map never needs to be updated. I intend to just close this PR for now, but I wanted it on record in case we ever decide to fully optimize this.
Are these changes tested?
No
Are there any user-facing changes?
No

Let's reopen if constantly updating the map with new heap indexes becomes to burdensome / expensive.


text:
ArrowArray::try_from_raw shouldn't clone

Guys, not sure if my understanding is right, but I think this commit will break the design and create memory leak.
If we clone the FFI struct, then it means we need to free the pointer by ourself, but if we free FFI_ArrowArray, then the data in this Array will also be free? Which means we can't free the pointer(until the data are used and ready to free, but in reality we can't hold this useless pointer in a big project for such a long time), which create memory leak.
As to the question @viirya raised in #1333 , when manage memory, the one who allocate it should free it, which means in our case, we need to alloc the struct in rust and pass the pointer to java and then also free the memory in rust.

You can check my code in here: https://github.com/wangfenjin/duckdb-rs/blob/5083d39a4147f8017613304ae5f217a88ac42c2e/src/raw_statement.rs#L58
When I try to upgrade to version 10, memory leak detected and there is no easy way to fix it.

I suggest we revert this commit. cc @alamb @sunchao
Originally posted by @wangfenjin in https://github.com/apache/arrow-rs/issues/1334#issuecomment-1064828113

As to the question @viirya raised in https://github.com/apache/arrow-rs/issues/1333 , when manage memory, the one who allocate it should free it, which means in our case, we need to alloc the struct in rust and pass the pointer to java and then also free the memory in rust.

Simply said, I don't think this is correct. That is the whole point of C data interface. The release callback is designed to be called by not only the one allocates the C data interface structure.

Let me put in another way.
Let's say in Java you create an ArrowArray and import in this lib using try_from_raw API, and this API clone the ArrowArray, which means there will be two struct point to the same data, one is in Jave another is in rust. Then

If you free the pointer in Java(which by design will also free all the data it point to), it will be seg fault in rust if you try to access it;
And you just can't hold the Java's pointer until it's used/freed by rust, because you don't when it will be used. So it's a memory leak.

I bet currently in your Java program there is memory leak because of these. I think the way to fix your issue is when you try to pass the data, you do the Arc::into_raw(Arc::new(your_data)), then pass this pointer to rust.

fwiw, that is also what we do in arrow2 - we never take ownership of the pointers passed to our exporter; we just write to them (as they may have been allocated by someone else), see e.g. the example in https://jorgecarleitao.github.io/arrow2/ffi.html
We did file a rust advisory on this last week, https://rustsec.org/advisories/RUSTSEC-2022-0012.html, as we incorrectly derived Clone to the FFI structs

Let me put in another way.
Let's say in Java you create an ArrowArray and import in this lib using try_from_raw API, and this API clone the ArrowArray, which means there will be two struct point to the same data, one is in Jave another is in rust. Then
1. If you free the pointer in Java(which by design will also free all the data it point to), it will be seg fault in rust if you try to access it;

2. And you just can't hold the Java's pointer until it's used/freed by rust, because you don't when it will be used. So it's a memory leak.

I bet currently in your Java program there is memory leak because of these.

I'm not sure if you really use this FFI between Java and Rust. We have real use case to use it in an internal project. We found this issue and the fix works very well after upgrading to latest version. And, no, I don't detect memory leak there. Let me explain more.
First, of course if you free the pointer in Java, you will release the data and Rust cannot access it anymore. If you really want to free it, make sure to clear up the release field in the struct. This is easy to do with Java FFI API. In our case, the struct in Java won't be released until the arrays are exhausted by Rust. So we don't even need to do that.
In contrast, I don't know what you describe will work (better). Let's say we don't clone it, then in Java and Rust, there is only one struct. So, if you free the pointer in Java, it still releases the data, isn't? The only possible is to check the release field of the struct to know if the struct is released or not by Rust. I don't see there is any better point.
The only missing piece, here, is that we need to mark the source struct released in Rust side. This is required by "Moving an array" which talks about the case to clone the struct. I can do it later.

I think the way to fix your issue is to change the Java API design, to let other language like rust to pass in a FFI_ArrowArray::empty() struct pointer, and populate data into this struct, and let rust handle the memory itself. That's why I say "the one who allocate it should free it"

Although I think this will work but I don't think it is a good idea. By doing that, it means to add an additional JNI call to Rust side to get the empty struct pointer. I agree that the struct should be released by who allocates it. Actually you cannot free Java struct in Rust, and vice versa.

I get your point. You are calling rust from Java, so it's right you alloc the memory in Java. The problem is rust assume this pointer is from Arc::into_raw, but acctually it's not it's just a raw pointer from JNI. In this case, our solution should be add a new API like:
    /// creates a new [ArrowArray] from two pointers. Used to import from the C Data Interface.
    /// # Safety
    /// See safety of [ArrowArray]
    /// # Error
    /// Errors if any of the pointers is null
    pub unsafe fn try_from_c(
        array: *const FFI_ArrowArray,
        schema: *const FFI_ArrowSchema,
    ) -> Result<Self> {
        if array.is_null() || schema.is_null() {
            return Err(ArrowError::MemoryError(
                "At least one of the pointers passed to `try_from_raw` is null"
                    .to_string(),
            ));
        };
        Ok(Self {
            array: Arc::new(*array), // notice we don't need to clone here, if will also solve your problem
            schema: Arc::new(*schema),
        })
    }

The idea is we don't need to Clone, why bother? As in your Java program you need to hold the pointer as long as you need id, and can only free when the job is done. Again, "the one who allocate it should free it".
There are several drawbacks in the current solution:

It breaks current API convention. Anyone use the API like me will be detected memory leak because of this change.
You can say your Java program don't leak memory because you always holding the pointer, but in some aspect we can say it's, because we alloc two struct point to the same data, which can be easily and should managed in one.
Allow clone will make us have the same issue mentioned by @jorgecarleitao , I add a demo here https://github.com/apache/arrow-rs/pull/1431

About the link you mentioned Moving an array, please noted that it talks about consumer which is rust in this case, need to call release_call if it tries to move the data, seems have nothing to do with our discuess here.

If you really want to free it, make sure to clear up the release field in the struct. This is easy to do with Java FFI API.

In here you are talking about the producer side, but actually you can't just clear up the release field and forget about it. What if rust have any issue to release the data, Java as the producer, as the one alloc the memory, should still be responsible to release it. Again, "the one who allocate it should free it".

The issue is our current FFI API design assume the pointer always comes from Arc::into_raw(), we should solve this problem, but Clone is not the solution.

The idea is we don't need to Clone, why bother? As in your Java program you need to hold the pointer as long as you need it, and can only free it when the job is done.

Are you sure it works?
error[E0507]: cannot move out of `*array` which is behind a raw pointer
...
error[E0507]: cannot move out of `*schema` which is behind a raw pointer

It breaks current API convention. Anyone use the API like me will be detected memory leak because of this change.

You can say your Java program don't leak memory because you always holding the pointer, but in some aspect we can say it's, because we alloc two struct point to the same data, which can be easily and should managed in one.

Allow clone will make us have the same issue mentioned by @jorgecarleitao , I add a demo here

I don't know how do you use the API. If the memory leak is happened because you need to hold the pointer in Java side, I think we can mark the source structs released after cloning it. So you can free the struct in Java.
The issue you demo is also resolvable by marking the source struct released.

The idea is we don't need to Clone, why bother? As in your Java program you need to hold the pointer as long as you need it, and can only free it when the job is done.

Are you sure it works?
error[E0507]: cannot move out of `*array` which is behind a raw pointer
...
error[E0507]: cannot move out of `*schema` which is behind a raw pointer

How you run the code? From my side:
✘ 🍻   ~/github/arrow-rs/arrow   flight-sql-client ±  cargo test -- --nocapture test_clone_array_segfault_on_drop
   Compiling arrow v10.0.0 (/Users/wangfenjin/github/arrow-rs/arrow)
    Finished test [unoptimized + debuginfo] target(s) in 41.79s
     Running unittests (/Users/wangfenjin/github/arrow-rs/target/debug/deps/arrow-01a41b09de9e05da)

running 1 test
error: test failed, to rerun pass '--lib'

Caused by:
  process didn't exit successfully: `/Users/wangfenjin/github/arrow-rs/target/debug/deps/arrow-01a41b09de9e05da --nocapture test_clone_array_segfault_on_drop` (signal: 11, SIGSEGV: invalid memory reference)

It should work otherwise the change you make should also break. they both do *array
         let ffi_array = (*array).clone();

The idea is we don't need to Clone, why bother? As in your Java program you need to hold the pointer as long as you need it, and can only free it when the job is done.

Are you sure it works?
error[E0507]: cannot move out of `*array` which is behind a raw pointer
...
error[E0507]: cannot move out of `*schema` which is behind a raw pointer

How you run the code? From my side:
✘ 🍻   ~/github/arrow-rs/arrow   flight-sql-client ±  cargo test -- --nocapture test_clone_array_segfault_on_drop
   Compiling arrow v10.0.0 (/Users/wangfenjin/github/arrow-rs/arrow)
    Finished test [unoptimized + debuginfo] target(s) in 41.79s
     Running unittests (/Users/wangfenjin/github/arrow-rs/target/debug/deps/arrow-01a41b09de9e05da)

running 1 test
error: test failed, to rerun pass '--lib'

Caused by:
  process didn't exit successfully: `/Users/wangfenjin/github/arrow-rs/target/debug/deps/arrow-01a41b09de9e05da --nocapture test_clone_array_segfault_on_drop` (signal: 11, SIGSEGV: invalid memory reference)

I mean the solution you suggested:
Ok(Self {
            array: Arc::new(*array), // notice we don't need to clone here, if will also solve your problem
            schema: Arc::new(*schema),
        })

Oh I haven't test the suggested solution yet, if it doesn't work maybe we can make it Box<> as the arrow2 API?
I'm not so familiar with this

It should work otherwise the change you make should also break. they both do *array
         let ffi_array = (*array).clone();

No...It is basically about Rust ownership rule. In short, Arc::new(*array) will try to own the data behind array raw pointer. So Rust complains it cannot move the data.
(*array).clone() doesn't move data but clone it.

Yeah, if you or others definitely insist we should not do clone (although I still don't agree with it), I can try to work on to get rid of it with Box or something other. I cannot agree you is that we don't think this is a real issue (that the API assumes the raw pointers are Arc pointers) and try to revert to original behavior.

About the link you mentioned Moving an array, please noted that it talks about consumer which is rust in this case, need to call release_call if it tries to move the data, seems have nothing to do with our discuess here.

I don't agree with you here too. It is obviously talking out you can copy (i.e., clone) these structs, only if you mark source structs as released (I agree this is missed now).

So, from my side, the previous behavior is definitely wrong (assuming the passed in raw pointers are always Arc pointers).
I can think of the options:

Mark source structs as released after cloning them. I think this follows the C data interface. This should allow the source structs to be freed at Java side and resolve the memory leak issue and double dropping issue.
Try other solution like Box.

WDYT?

I think Box might be a better solution as we can avoid clone. If you don't have time I can also help on this issue. Thanks

Okay, I can work on this in the weekend. See if Box can save us from clone.
Thanks for the discussion.

I haven't followed all the discussion on this ticket, but it seems like it is well in hand. Let me know if I can do anything else to help

Thanks @alamb

authors: alamb jorgecarleitao viirya wangfenjin

repo: apache/arrow-rs