Remove test mitigation

According to https://github.com/microsoft/MSBuildSdks/issues/155 there's no need to keep the test mitigation anymore.

Closing as the PR hasn't had any recent activity. Please feel free to reopen once it's ready to review!


APIResponseError: API token is invalid

Hi, this is most probably not a bug, but and issue between chair and keyboard:)
I'm not a developer, so intalling Kindle2Notion wasn't an easy task, but I managed it. But when I tried to export the clippings from kindle to Notion by running the Kindle2Notion package, i'm getting error
notion_client.errors.APIResponseError: API token is invalid
It seems like a least expected problem, because I simply go to Notion's 'Setting & Members' > Integrations and copy Internal Integration token. Don't understand what could be an issue here. I tried to reinstall the package, but didn't help. Any advice will be highly appreciated! Thanks!

facing the same error :(

Hi @spacepioneer777 and @ishehraj ! I just took a look at this and like David mentioned, it's most likely an issue with the Notion API not recognising your integration token for some reason. I'd recommend you try duplicating the K2N Notion template once again to your workspace and create another integration token to use with it, Please let me know if the issue persists. I couldn't reproduce the issue from my end. I hope this helps! I am closing this issue for now.


If auto-starting the desktop app is turned on, an old app will continue to auto-start unless the auto-start option is reset from the Settings page.

I confirm (by marking "x" in the [ ] below):

[X] This is not a feature idea. Feature ideas go here: http://www.mattermost.org/feature-ideas/
[X] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/
[X] This is a product defect issue that can be reproduced using the steps listed below.
[X] I have read CONTRIBUTING.md.

Summary
Registry key not updated if using auto-start and downloading a new version
Steps to reproduce
Operating System: Windows 10
MM Desktop App version: 3.4 RC1
MM Server version: 3.4

Download Desktop v1.3 from the downloads page
Go to the Settings page via File > Settings
Turn on Start app on login. and hit Save
Download the RC for v1.4/3.4: https://github.com/mattermost/desktop/issues/285

64-bit installer here

Verify download was successful by checking the version number from Help > Version
Exit the application from File > Exit
Restart your machine

Expected behavior
The latest downloaded version of Mattermost starts.
Observed behavior
Mattermost v1.3 auto-starts.
Possible fixes
Registry key doesn't update when you download the desktop app. Thus, the old version will start unless the Start app on login. setting is reset (which only happens if you first turn the option off, save, then turn the setting back on)
@Razzeee has started progress with an initial artifact here.

The setting was new for v1.3, so releasing v1.4/3.4 will be the first time users will hit this issue, and they most likely will hit it.
This is a really important bug to fix before the release - could we get the community's help here to chime in?

@Razzeee @yuya-oc, any potential thoughts here?

I feel Razzeee's branch would potentially help this.

Well it should, but it doesn't. Might be a bug in winreg :/
On Mon, Sep 19, 2016, 07:25 Yuya Ochiai<EMAIL_ADDRESS>wrote:

I feel Razzeee's branch
https://github.com/Razzeee/desktop/commit/66757ca406e9b3796f7d4c0c95fb43fdaec4dea9
would potentially help this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mattermost/desktop/issues/291#issuecomment-247913282,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFqyZDHGD9KnjfcYXftxdl-iPpX9PYXAks5qrhysgaJpZM4J_f9V
.

@Razzeee Can you help take a look at some of the comments from @yuya-oc, looks like he added some notes?
I'd be happy to test it out afterward

@jasonblais
Already looked at them, this is the latest code iteration. https://github.com/Razzeee/desktop/commit/1092aa3f547efc85f882e7e1c79033b2dcc71662 and it doesn't work for me.

@Razzeee I added a comment for the commit. It should work.

@Razzeee Did you have a chance to take a look at @yuya-oc's new comment?

@Razzeee @yuya-oc any updates? I see there are some notes in the commit, not sure if those are covered.
Sorry to bump you guys here, but I think this is the last PR to consider for Desktop 3.4 release

PR incoming. Just took a second to test it.

Closed by #296.


Fix Firewall Unit Tests and Allow Multiple Firewall rules to be tested

This PR changes:

Fixes xFirewall Unit tests to prevent false positive failures caused by the values that firewall properties are changed to actually matching the existing values. This was detected only when I tested ALL firewall rules available on my computer as noted in PR #51 .
Added a foreach loop around the Firewall pester tests so that more than one rule could be tested - note: It still only currently tests the first rule, so the actual behavior hasn't changed here.

The diff isn't showing for some reason so this is going to be kind of hard to review. I'll have to pull the change in locally and review it on a diff tool on my machine.
However, my first thought is number 2 should be removed, it's just confusing why we are doing a foreach loop on just one result and a waste of cycles. I'll spend some time looking at doing a loop over everything and we can add the functionality then.

Oh. That is strange!? I can recreate the PR easily enough if you like - save you the trouble?
The only reason I left the loop in was that I thought we could change the logic to pull the rules to test (in future if you're able to figure out a way to optimize the code) to pull multiple rules - all of them if possible. But if the code can't be optimized, I thought we could either pull say the first 10 and test those OR perhaps pull a random set of 10 or so out (using  a modulus function to choose them or something like that). Ideally I think testing more than one rule might be good. Will totally take your lead on this :)

Lets not try to future proof now, if we think a random sample of 10 is needed, let's put that in. If we want to wait until we can see how many we can reasonably test at once then let's handle that with a seperate PR and remove the for each loop for now.

:+1: You are correct - and Agile development practices complete agree with you (I came from a waterfall world). Would you mind if I close and recreate this PR? I am not overly happy with the fact that the Diff hasn't worked.

It's your PR, feel free to close it. :+1:

Thanks. Will raise a new one momentarily. Cheers!


KeyError: 'flow' when running Mininet

When upgrading from kytos 2020.2rc1 to 2020.2 with flow_manager 3.0 and running mininet, a KeyError: 'flow' is issued on the Kytos console. After that it says Flows loaded.
Steps to reproduce: upgrade an existing instance of Kytos, run kytosd -f and then run Mininet.
sudo mn --topo linear,3 --mac --controller=remote,ip=<IP_ADDRESS> --switch ovsk,protocols=OpenFlow13
2020-12-30 17:11:01,817 - INFO [kytos.core.atcp_server] (MainThread) New connection from <IP_ADDRESS>:35060
2020-12-30 17:11:01,820 - INFO [kytos.core.atcp_server] (MainThread) New connection from <IP_ADDRESS>:35062
2020-12-30 17:11:01,821 - INFO [kytos.core.atcp_server] (MainThread) New connection from <IP_ADDRESS>:35064
2020-12-30 17:11:02,399 - INFO [kytos.napps.kytos/of_core] (Thread-38) Connection ('<IP_ADDRESS>', 35060), Switch 00:00:00:00:00:00:00:01: OPENFLOW HANDSHAKE COMPLETE
2020-12-30 17:11:02,402 - INFO [kytos.napps.kytos/of_core] (Thread-40) Connection ('<IP_ADDRESS>', 35064), Switch 00:00:00:00:00:00:00:02: OPENFLOW HANDSHAKE COMPLETE
2020-12-30 17:11:02,409 - INFO [kytos.napps.kytos/of_core] (Thread-39) Connection ('<IP_ADDRESS>', 35062), Switch 00:00:00:00:00:00:00:03: OPENFLOW HANDSHAKE COMPLETE
Exception in thread Thread-45:
Traceback (most recent call last):
  File "/usr/lib/python3.8/threading.py", line 932, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.8/threading.py", line 870, in run
    self._target(*self._args, **self._kwargs)
  File kytos/core/helpers.py", line 70, in threaded_handler
    handler(*args)
  File "venv/ky/var/lib/kytos/napps/kytos/flow_manager/main.py", line 68, in resend_stored_flows
    flows_dict = {"flows": [flow['flow']]}
KeyError: 'flow'
kytos $> 2020-12-30 17:12:00,421 - INFO [kytos.napps.kytos/flow_manager] (flow_manager) Flows loaded.

Hello, @hdiogenes, I cannot reproduce this error in my virtual environment. I think that maybe this error is occurring because the resend_stored_flows method is using old data stored in the storehouse, related to the first versions of the flow persistence implemented in PR #90.
Deleting the namespace (kytos. Flow. Persistence) in the storehouse can resolve this error. But I think it may be necessary to implement a version field, or something like that in the persistence data, avoiding errors of this kind.
If cleaning the box does not solve the problem, you can provide me with more information on how to reproduce this error.

Information about my env, using kytos bug-report:
Platform
Release information
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.1 LTS
Release:	20.04
Codename:	focal
System Information
Linux carlosmagno-Lenovo-ideapad-330S-15IKB 5.8.0-25-generic #26-Ubuntu SMP Thu Oct 15 10:30:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Python
path=/home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/bin/python
version=Python 3.6.12
Pip
path=/home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/bin/pip
version=pip 20.3.3 from /home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/lib/python3.6/site-packages/pip (python 3.6)
Python Packages
kytos                          | github.com:cmagnobarbosa/kytos | f145a7f7
kytos_flow_manager             | github.com:cmagnobarbosa/flow_manager | a1d756f0
kytos_mef_eline                | github.com:kytos/mef_eline     | e670270a
kytos_of_core                  | github.com:cmagnobarbosa/of_core | ccaa0d2c
kytos_of_l2ls                  | github.com:cmagnobarbosa/of_l2ls | 1de43719
kytos_of_lldp                  | github.com:cmagnobarbosa/of_lldp | 4f22cf52
kytos_pathfinder               | github.com/kytos/pathfinder    | d8ba617a
kytos_storehouse               | github.com:cmagnobarbosa/storehouse | e1425d16
kytos_topology                 | github.com:cmagnobarbosa/topology | 581dc01c
kytos_utils                    | github.com:cmagnobarbosa/kytos-utils | a16a659e
python_openflow                | github.com:cmagnobarbosa/python-openflow | 0537c626
Kytos environment
Kytosd
path=/home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/bin/kytosd
version=kytosd 2020.2
Kytos
path=/home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/bin/kytos
version=kytos command line, version 2020.2
Installed napps
Status |        NApp ID         |                                    Description
=======+========================+==================================================================================
[ie]  | kytos/flow_manager:3.0 | Manage switches' flows through a REST API.
[ie]  | kytos/mef_eline:2.4    | NApp to provision circuits from user request
[ie]  | kytos/of_core:1.5.1    | OpenFlow Core of Kytos Controller, responsible for main OpenFlow operations.
[ie]  | kytos/of_l2ls:1.2.0    | A L2 learning switch application for OpenFlow switches.
[ie]  | kytos/of_lldp:1.1      | Discover network-to-network interfaces (NNIs) using the LLDP protocol.
[ie]  | kytos/pathfinder:2.2.3 | Keeps track of topology changes, and calculates the best path between two points.
[ie]  | kytos/storehouse:1.3.1 | Persistence NApp with support for multiple backends
[ie]  | kytos/topology:3.7.1   | Manage the network topology.
Status: (i)nstalled, (e)nabled

Hello, @hdiogenes, I cannot reproduce this error in my virtual environment. I think that maybe this error is occurring because the resend_stored_flows method is using old data stored in the storehouse, related to the first versions of the flow persistence implemented in PR #90.
Deleting the namespace (kytos. Flow. Persistence) in the storehouse can resolve this error. But I think it may be necessary to implement a version field, or something like that in the persistence data, avoiding errors of this kind.
If cleaning the box does not solve the problem, you can provide me with more information on how to reproduce this error.

Information about my env, using kytos bug-report:
Platform
Release information
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.1 LTS
Release:	20.04
Codename:	focal
System Information
Linux carlosmagno-Lenovo-ideapad-330S-15IKB 5.8.0-25-generic #26-Ubuntu SMP Thu Oct 15 10:30:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Python
path=/home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/bin/python
version=Python 3.6.12
Pip
path=/home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/bin/pip
version=pip 20.3.3 from /home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/lib/python3.6/site-packages/pip (python 3.6)
Python Packages
kytos                          | github.com:cmagnobarbosa/kytos | f145a7f7
kytos_flow_manager             | github.com:cmagnobarbosa/flow_manager | a1d756f0
kytos_mef_eline                | github.com:kytos/mef_eline     | e670270a
kytos_of_core                  | github.com:cmagnobarbosa/of_core | ccaa0d2c
kytos_of_l2ls                  | github.com:cmagnobarbosa/of_l2ls | 1de43719
kytos_of_lldp                  | github.com:cmagnobarbosa/of_lldp | 4f22cf52
kytos_pathfinder               | github.com/kytos/pathfinder    | d8ba617a
kytos_storehouse               | github.com:cmagnobarbosa/storehouse | e1425d16
kytos_topology                 | github.com:cmagnobarbosa/topology | 581dc01c
kytos_utils                    | github.com:cmagnobarbosa/kytos-utils | a16a659e
python_openflow                | github.com:cmagnobarbosa/python-openflow | 0537c626
Kytos environment
Kytosd
path=/home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/bin/kytosd
version=kytosd 2020.2
Kytos
path=/home/carlosmagno/workspace/kytos/Teste_Coverage/env_keyError/bin/kytos
version=kytos command line, version 2020.2
Installed napps
Status |        NApp ID         |                                    Description
=======+========================+==================================================================================
[ie]  | kytos/flow_manager:3.0 | Manage switches' flows through a REST API.
[ie]  | kytos/mef_eline:2.4    | NApp to provision circuits from user request
[ie]  | kytos/of_core:1.5.1    | OpenFlow Core of Kytos Controller, responsible for main OpenFlow operations.
[ie]  | kytos/of_l2ls:1.2.0    | A L2 learning switch application for OpenFlow switches.
[ie]  | kytos/of_lldp:1.1      | Discover network-to-network interfaces (NNIs) using the LLDP protocol.
[ie]  | kytos/pathfinder:2.2.3 | Keeps track of topology changes, and calculates the best path between two points.
[ie]  | kytos/storehouse:1.3.1 | Persistence NApp with support for multiple backends
[ie]  | kytos/topology:3.7.1   | Manage the network topology.
Status: (i)nstalled, (e)nabled


Update requirements.txt for Python 3.7 Support

Updated rq to 0.12.0 so it works with Python 3.7. The previous version used the reserved async variable.

This has been addressed in the correct commit. Thanks!


SID and NS when using AUTH

When NoAUTH is used,

gauard-gate gets  SID and NS from env
guard-service uses SID and NS sent from gauard-gate.
Hence two sides are in sync and it is up to the service yaml env to define the correct sid and ns

When AUTH is used,

guard-gate gets  SID and NS from env
The guard-service concludes SID and NS from AUTH Token.
Hence the two sides are not necessarily in sync!
Since guard-gate should work even when guard-service is down, we can't rely on guard-service to profile the SID and NS.
Analyze what should we do.
Some options:

have service confirm gate's sid and ns and send error when not in line
have gate use KubeAPI to confirm sid and ns (... may need to add more permissions... )
??

One option to solve this is to always use TLS and AUTH  in production and make it the default for Guard.

/remove-lifecycle stale


Release API plugin

Description of changes

This PR releases the API plugin with the following important changes:

Fix to Sync query behavior to eliminate multiple scan operations.
Fix owner auth logic to handle dynamic auth fields as sort keys properly.
RDS changes that are disabled by default.

CDK / CloudFormation Parameters Changed

Issue #, if available

Description of how you validated changes
Full e2e tests pass except the datastore-modelgen and amplify-app which currently fail on setup step to use the amplify-app. Since we have the same tests in CLI, we can feel better to release.
Latest e2e run. I will update this link once I retry, but other tests apart from 2 mentioned above should pass.
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Any CDK or CloudFormation parameter changes are called out explicitly

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Looks E2E tests are failing. Also, I see that we are still using CLI 12.0.0 RC version to run our tests. Should we wait until CLI release the next version?

Updated CLI dep and all tests are now passing.


[benchmark] Exclude dotnet from OSS benchmarks CI.

QPS worker build is currently failing.

Steps to reproduce:

Clone the https://github.com/grpc/test-infra repo.
make all-tools
bin/prepare_prebuilt_workers -l dotnet:master -p gcr.io/grpc-testing/e2etest/prebuilt/${USER} -t latest -r containers/pre_built_workers/

The Dockerfile for the build is under containers/pre_built_workers/.
You can build from a different fork and commit by replacing dotnet:master with dotnet:paulosjca/grpc-dotnet:master (or whatever your fork is rather than paulosjca/grpc-dotnet).


MUMUP-1539 : marketplace related app goes to marketplace entry

Related portlets now go to that marketplace entry by doing a search for the title

kewl :+1:


adDismissed, reward never change their values

Description
adDismissed always false, and reward always undefined
Expected those values to change
const {adLoaded, adDismissed, reward, adShowing, show} = useRewardedAd(
    TestIds.REWARDED,
    {
      requestOptions: {
        requestNonPersonalizedAdsOnly: true,
      },
    },
  );
  useEffect(() => {
    console.log('rewardrewardrewardrewardss', adDismissed, nameToShow);
    if (adDismissed) {
        /// do something
    }
  }, [adDismissed]);

React Native: 0.66.0
React Native AdMob: 1.1.0

Try this solution and feedback me if it not works. https://github.com/react-native-admob/admob/issues/51#issuecomment-964105866

Try version 1.2.1 and feedback me.

I was on version 1.2.1 and downgraded to 1.1.0 hoping it was working there but found the same behavior

Try latest version and reopen issue if it does not work.


remove position absolute

In order to have the rangepicker work with iron-dropdown, need to remove the position:absolute from this component - not sure if there was a reason for it being there in the first place?

All seems good except for the tests not passing

Closing based on age and conflicts


Add a link to caniuse.com to the first step.

I added a link to http://caniuse.com/#search=service+workers, to make it easier for users to look up the latest support for service workers.
===
Made at CodeLabOC 🍊 🎃

nice 👏


[TOPIC-BLE-LLCP] bluetooth: controller: llcp: LE PING procedure

Implements the le ping procedure and unit tests
Signed-off-by: Erik Brockhoff<EMAIL_ADDRESS>

@kruithofa Thanks Andries - all comments addressed. I decided to remove the decode_ping... functions all together and moved the comment /* ... has not data */ to the switch case instead. I hope you agree.


Create newtype wrapper for advice map

Currently, our advice map is represented by just BTreeMap<[u8; 32], Vec<Felt>>. It would be good to create a newtype wrapper around it. We can also use this as an opportunity to change the key type to from [u8; 32] to RpoDigest (the reason we have [u8; 32] in the first place was because we couldn't use Word as the key - but I think RpoDigest is a better option).
So, the struct could look like:
pub struct AdviceMap(BTreeMap<RpoDigest, Vec<Felt>>);

If needed, we can still maintain Word-based interface and just convert to RpoDigest internally.

Closed by #1207.


feat: fork cross-spawn to fix esbuild chaos

esbuild isn't great at bundling for node. it currently produces invalid code by leaving require calls in the esm output, which is far from great.
To resolve this, this moves cross-spawn into the repo and converts it to ESM while dropping any logic we weren't making use of.
DRAFT until i finish adding the tests for the cross-spawn module.
This may also be discarded if we find a simpler way to get esbuild to output valid code.
cc @bluwy

I feel like the better solution to this would be to switch from tsup to rollup

are we sure rollup doesn't have the same problem? under the hood, esbuild is at fault here. it just doesn't transform require() calls of built in modules. so the output has CJS in it still and definitely shouldn't
would be curious if rollup just does the same, i feel like someone pointed me at an issue in their github that suggested it does 👀

I know that rollup doesn't have this issue because we just tried to migrate a repo from rollup to rolldown and ran into an issue where the require was no longer being transformed (https://github.com/rolldown/rolldown/issues/2041)

I tried rollup locally when we discussed this issue and it didn't have the same problem IIRC, which i thought it would have, so using rollup would definitely be an option, but it requires more setup.

I sent a PR for rollup here: https://github.com/tinylibs/tinyexec/pull/35


Hyprland workspace 2 acting up

When trying to switch or send a window to workspace 2 hyprland crashes.
Need to investigate further why this is happening.

Issues seems to be that Hyprland sees a "ghost" monitor that does not exist. Workspace 2 is assigned to this monitor and thats the reason it's acting up.
Still need to find out how to fix this. To be continued.

After updating Hyprland the issue is fixed


[TASK] Fix CI to publish types and publish the package on npm

Task summary
Currently the flow for updating the types and publishing them to the NPM is broken.
Specification reference

First appeared in #308
First attempt to fix it #309
Second attempt #310

Other information and links
Looks like the script is very old and it is very similar to the one used by webb.js to publish new versions to NPM, would be useful if we can update this and fix it ASAP.

I looked through docs and examples, tried to make some changes but nothing helped. So I opened an issue, hopefully we can get some help that way.
https://github.com/polkadot-js/api/issues/5582


Links not working properly

The first two links (all_mappings and flow list) in the "Mapping Flowables" section of the Wiki are not working.
https://github.com/USEPA/Federal-LCA-Commons-Elementary-Flow-List/wiki/Getting-Started-with-FEDEFL#mapping-flowables

thanks, this has been fixed


Fix: "example" -> "examples" && "test" -> "tests"

修改 example 文件夹为 examples
修改 test 文件夹为 tests
修改引用以上两个文件夹的代码、文档及注释

Codecov Report

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

@@           Coverage Diff           @@
##           master       #7   +/-   ##
=======================================
  Coverage   92.25%   92.25%
=======================================
  Files          21       21
  Lines         400      400
=======================================
  Hits          369      369
  Misses         31       31

Impacted Files
Coverage Δ

tests/test_scatter.py
100% <ø> (ø)

tests/test_line.py
100% <ø> (ø)

tests/test_radar.py
100% <ø> (ø)

tests/test_bar.py
100% <ø> (ø)

tests/test_pie.py
100% <ø> (ø)

tests/test_page.py
100% <100%> (ø)

tests/test_engine.py
100% <100%> (ø)

Continue to review full report at Codecov.

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

行，那就用复数吧

我在提交后去 pycharts 看了一下，才发现 pycharts 本身是单数，不过用复数应该更明确一些😛


Support magnet links

If the watcher sees a file with .magnet extension, assume it's a single magnet link and add that

Closing as I forgot to change the watch command and fixed just the manual add command.


Can't see content via ip address on another device

I ran the app with "dev_appserver.py . --host={my ip address}" and share the spec link with developer. He can open the link but he can't see any spec content.

Sounds like the config needs to be customized. It handles the correct url for the api


Grafana SQL datasource is not working

What happened?
Grafana SQL datasource is not working and panels using this DS are empty. Most likely related to #443
Did you expect to see something different?
I would expect all panels to be up and running
How to reproduce it (as minimally and precisely as possible):

Start a cluster
Open grafana (for example with port-forward)
Go to datasources setup and select SQL datasource
Click blue button "save and test"

Environment

tobs installation method:

[ ] cli
[x] helm chart

tobs version:
0.11.2

Kubernetes version information:
1.24

Kubernetes cluster kind:
kind create cluster

Anything else we need to know?:

By removing the scripts, you also removed the datasource SQL account setup of the Grafana reader.
    DO \$\$
      BEGIN
        CREATE ROLE prom_reader;
      EXCEPTION WHEN duplicate_object THEN
        RAISE NOTICE 'role prom_reader already exists, skipping create';
      END
    \$\$;
    DO \$\$
      BEGIN
        CREATE ROLE $GF_READER WITH LOGIN PASSWORD \'$GF_DS_PASS\';
      EXCEPTION WHEN duplicate_object THEN
        RAISE NOTICE \'role $GF_READER already exists, skipping create\';
      END
    \$\$;
    GRANT prom_reader TO $GF_DS_USER;

Thanks @Davincible. We are bringing it back along some tests to prevent such issues in the long run.

Closing as the fix (https://github.com/timescale/tobs/pull/450) is included in latest tobs patch release (0.11.3).


Write Concepts Page

Execution Modes

nbdev_test
nbdev_docs
nbdev_filter

Describe:

What is the purpose of these
How do they work, conceptually?  How can you debug

Pandoc

What is pandoc
How does it fit in with quarto
How you can leverage pandoc to customize nbdev

Writing Source Code

@patch
How to test most effectively
How to debug
How to use your IDE in addition to notebooks

Documentation

How do APIs get documented?
What are different ways of documenting APIs: numpy, docments, etc.
How does nbdev render API documentation?

Closing, since I think these are now covered:

"Execution Modes" covered in https://nbdev.fast.ai/explanations/docs.html#concepts
"Writing Source Code" and "Documentation" covered in the same page and https://nbdev.fast.ai/tutorials/best_practices.html

@hamelsmu we don't cover pandoc much yet, but maybe you can create a new issue for that when you look into it?

Sure


Show multiple notifications

Nice start. It would great if you feature to show multiple notifications at the same time. E.g. how it is done here http://www.jqueryrain.com/?4NmZRZ2C

I improvised this: https://github.com/naoxink/notifIt/commit/ae480fa7194cea6eea83173f82618a50961dbb34
Thank you for your time :)

I'm closing this for the moment. Thank you.


More intuitive internal structure with sub packages

Currently files are organized in a "flat" mode:

It makes it quite difficult for newcomers to reverse-guess the architecture from package organization. I'm sure that a few separate folders could be identified to better structure this
Any ideas?

I guess the best way to better understand the module structure of a project is a graph of import dependencies. Actually doit owns tutorial is a project on how to create those. I should probably include that generated graph for doit itself on docs...
I think that given doit size, a flat module structure fits better.

I disagree, but that's your call ;)


fix(manager): sort versions by number

Secure Coding Practices Checklist GitHub Link

https://github.com/RedHatInsights/secure-coding-checklist

Secure Coding Checklist

[x] Input Validation
[x] Output Encoding
[x] Authentication and Password Management
[x] Session Management
[x] Access Control
[x] Cryptographic Practices
[x] Error Handling and Logging
[x] Data Protection
[x] Communication Security
[x] System Configuration
[x] Database Security
[x] File Management
[x] Memory Management
[x] General Coding Practices

Codecov Report
Base: 37.85% // Head: 37.85% // No change to project coverage :thumbsup:

Coverage data is based on head (67531fd) compared to base (ec7f007).
 Consider uploading reports for the commit 1f15860 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   37.85%   37.85%
=======================================
  Files          16       16
  Lines         819      819
=======================================
  Hits          310      310
  Misses        481      481
  Partials       28       28

Flag
Coverage Δ

unittests
37.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Error while using hello-data.js

Description
Get an Error when loading "hello-data".
Steps to Reproduce
Just followed the Guide on the Qlikcore Getting Started Website
https://core.qlik.com/tutorials/hello-data/
Expected behavior
The output from the "hello-data" guide
$ npm run hello-data
Creating table data representation.
Creating and opening session using mixin.
Creating session object with movie titles.
Listing the 10 first movies:
2012
Armageddon
Avatar
Battleship
Cars 2
Cleopatra
Evan Almighty
Green Lantern
Harry Potter and the Half-Blood Prince
Indiana Jones and the Kingdom of the Crystal Skull
Session closed.
Actual behavior

node ./src/hello-data/hello-data.js

Creating session app on engine.
Creating data connection to local files.
Whoops! An error occurred. { Error: Invalid path
at Intercept.errorInterceptor (C:\Project\QlikCore_data\core-get-started-master\node_modules\enigma.js\enigma.js:2549:19)
at process.internalTickCallback (internal/process/next_tick.js:77:7) code: 4, parameter: 'Invalid path' }
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>hello-data: node ./src/hello-data/hello-data.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>hello-data script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\abs\AppData\Roaming\npm-cache_logs\2019-03-20T07_46_41_306Z-debug.log
Environment
Library
[x] Node.js
[ ] Browser

Operating system
[x] Windows
[ ] OSX
[ ] Linux

Versions

Node.js: v11.6.0
Browser: Google Chrome 72.0.3626.121
Operating system: Windows 10

I am looking for someone that can help me since always getting this error in the Getting Started guide from Qlik Core. I try to run the "hello-data.js" but always get the Error Message above.

Hello @kokyeesiang, from the log it looks like Engine is unable to find the mounted volume. Can you check that you are sharing the drive with Docker for Windows?
You can find the setting in Docker for Windows -> Settings -> Shared Drives

Hi @wennmo ,

Already shared in the setting in Docker fyi but still hit the error.

Hi @kokyeesiang ,
Can you attach the docker logs from the engine container?
Run docker ps and you will get an output of running containers on your machine. Then use the CONTAINER ID when fetching the logs e.g. docker logs <CONTAINER ID>.
You can also check if the file movies.csv was successfully mounted into the engine container by running docker exec -it <CONTAINER ID> sh -c "ls /data".

Hi @wennmo ,
docker logs:
PS C:\Project\QlikCore_data\core-get-started-master> docker logs 8c0838f58783
{"timestamp":"2019-03-21T01:59:33.645Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"Engine started with command line: --MigrationPort -1 -S QvLogUseStdoutLogger=1 -S QvLogHandleRequestOnExit=1 -S AllowExternalSocket=1 -S EnableSwagger=1 -S EnablePrometheus=1 -S PrometheusServicePort=9090 -S PerformanceLogVerbosity=0 -S UseCGroupsStats=1 -S AdjustMemUseFromOs=0 -S EnableCrashDump=1 -S EnableScriptLogging=2 -S AcceptEULA=yes "}
{"timestamp":"2019-03-21T01:59:33.701Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"Invalid parameter handler set"}
{"timestamp":"2019-03-21T01:59:33.701Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"ComputerName is 8c0838f58783"}
{"timestamp":"2019-03-21T01:59:33.701Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"Engine Version is 12.345.0"}
{"timestamp":"2019-03-21T01:59:33.859Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"Locale(s) set"}
{"timestamp":"2019-03-21T01:59:33.859Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"QHeap initialized"}
{"timestamp":"2019-03-21T01:59:33.860Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"Running in desktop mode"}
{"timestamp":"2019-03-21T01:59:34.075Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"Initiating server license."}
{"timestamp":"2019-03-21T01:59:34.100Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"GlobalContext: Initiating Broker default port 4848"}
{"timestamp":"2019-03-21T01:59:34.100Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"GlobalContext: Initiating DataPrepPort default port 9072"}
{"timestamp":"2019-03-21T01:59:34.108Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"WorkingSet: Configured Working Set Size is 1.354-1.741 GB (setting 70%-90%)"}
{"timestamp":"2019-03-21T01:59:34.111Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"Initiating ExtEngine."}
{"timestamp":"2019-03-21T01:59:34.114Z","log_type":"Start","logseverity":"INFO","thread_id":2116521664,"message":"Initialization complete."}
{"timestamp":"2019-03-21T01:59:34.126Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"QvSocket: Starting HTTP request handler - 0x7f957b360a20"}
{"timestamp":"2019-03-21T01:59:34.133Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"Port Listen: IPv4 bind successful on Port 9076"}
{"timestamp":"2019-03-21T01:59:34.133Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"Port Listen: IPv6 bind successful on Port 9076"}
{"timestamp":"2019-03-21T01:59:34.134Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"System: Starting - Purge: Starting"}
{"timestamp":"2019-03-21T01:59:34.135Z","log_type":"System","logseverity":"INFO","thread_id":2116521664,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"System: Starting - ServerWatchDog: Starting"}
{"timestamp":"2019-03-21T01:59:35.136Z","log_type":"System","logseverity":"INFO","thread_id":2033415936,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"Mount: Found Mount  at location /home/nobody/Qlik/Sense/Apps  browsable"}
{"timestamp":"2019-03-21T02:00:22.150Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"TheServer: Registering new server session"}
{"timestamp":"2019-03-21T02:00:22.151Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"QvSocket: Connected to server as user: 'Personal\Me' for document: '/app/' on socket <IP_ADDRESS>:9076 <-> <IP_ADDRESS>:53020 with ttl: 0"}
{"timestamp":"2019-03-21T02:00:22.349Z","log_type":"System","logseverity":"INFO","thread_id":1795151616,"user_id":"Personal\Me","tenant_id":"default","app_id":"0","message":"DOC loading: Loaded session app SessionApp_e8b94e67-e52c-4670-94c9-302a0002c7ff."}
{"timestamp":"2019-03-21T02:00:22.671Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"StdServer: OnRemoteClosed entered for socket <IP_ADDRESS>:9076 <-> <IP_ADDRESS>:53020, user: Personal\Me, app: SessionApp_e8b94e67-e52c-4670-94c9-302a0002c7ff."}
{"timestamp":"2019-03-21T02:00:22.671Z","log_type":"System","logseverity":"INFO","thread_id":2033415936,"user_id":"Personal\Me","tenant_id":"default","app_id":"","message":"Session stop reason: Socket closed by client"}
{"timestamp":"2019-03-21T02:00:24.179Z","log_type":"System","logseverity":"INFO","thread_id":2033415936,"user_id":"Personal\Me","tenant_id":"","app_id":"","message":"DOC loading: Unloading document SessionApp_e8b94e67-e52c-4670-94c9-302a0002c7ff, state UNLOADING."}
{"timestamp":"2019-03-21T02:02:14.901Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"TheServer: Registering new server session"}
{"timestamp":"2019-03-21T02:02:14.901Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"QvSocket: Connected to server as user: 'Personal\Me' for document: '/app/' on socket <IP_ADDRESS>:9076 <-> <IP_ADDRESS>:53022 with ttl: 0"}
{"timestamp":"2019-03-21T02:02:14.912Z","log_type":"System","logseverity":"INFO","thread_id":1795151616,"user_id":"Personal\Me","tenant_id":"default","app_id":"0","message":"DOC loading: Loaded session app SessionApp_b773bcdf-f2c3-41b5-887f-b219eb64ef57."}
{"timestamp":"2019-03-21T02:02:14.938Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"StdServer: OnRemoteClosed entered for socket <IP_ADDRESS>:9076 <-> <IP_ADDRESS>:53022, user: Personal\Me, app: SessionApp_b773bcdf-f2c3-41b5-887f-b219eb64ef57."}
{"timestamp":"2019-03-21T02:02:14.938Z","log_type":"System","logseverity":"INFO","thread_id":2033415936,"user_id":"Personal\Me","tenant_id":"default","app_id":"","message":"Session stop reason: Socket closed by client"}
{"timestamp":"2019-03-21T02:02:24.230Z","log_type":"System","logseverity":"INFO","thread_id":2033415936,"user_id":"Personal\Me","tenant_id":"","app_id":"","message":"DOC loading: Unloading document SessionApp_b773bcdf-f2c3-41b5-887f-b219eb64ef57, state UNLOADING."}
{"timestamp":"2019-03-21T02:59:34.481Z","log_type":"System","logseverity":"INFO","thread_id":1769973504,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"System: Starting purge of temporary files."}
{"timestamp":"2019-03-21T03:59:35.477Z","log_type":"System","logseverity":"INFO","thread_id":1769973504,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"System: Starting purge of temporary files."}
{"timestamp":"2019-03-21T04:59:36.433Z","log_type":"System","logseverity":"INFO","thread_id":1769973504,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"System: Starting purge of temporary files."}
{"timestamp":"2019-03-21T05:59:37.186Z","log_type":"System","logseverity":"INFO","thread_id":1769973504,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"System: Starting purge of temporary files."}
{"timestamp":"2019-03-21T06:48:24.011Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"TheServer: Registering new server session"}
{"timestamp":"2019-03-21T06:48:24.012Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"QvSocket: Connected to server as user: 'Personal\Me' for document: '/app/' on socket <IP_ADDRESS>:9076 <-> <IP_ADDRESS>:53060 with ttl: 0"}
{"timestamp":"2019-03-21T06:48:24.025Z","log_type":"System","logseverity":"INFO","thread_id":1795151616,"user_id":"Personal\Me","tenant_id":"default","app_id":"0","message":"DOC loading: Loaded session app SessionApp_c75ec487-6854-44e9-997b-c8e6543297fe."}
{"timestamp":"2019-03-21T06:48:24.051Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"StdServer: OnRemoteClosed entered for socket <IP_ADDRESS>:9076 <-> <IP_ADDRESS>:53060, user: Personal\Me, app: SessionApp_c75ec487-6854-44e9-997b-c8e6543297fe."}
{"timestamp":"2019-03-21T06:48:24.052Z","log_type":"System","logseverity":"INFO","thread_id":2033415936,"user_id":"Personal\Me","tenant_id":"default","app_id":"","message":"Session stop reason: Socket closed by client"}
{"timestamp":"2019-03-21T06:48:33.862Z","log_type":"System","logseverity":"INFO","thread_id":2033415936,"user_id":"Personal\Me","tenant_id":"","app_id":"","message":"DOC loading: Unloading document SessionApp_c75ec487-6854-44e9-997b-c8e6543297fe, state UNLOADING."}
{"timestamp":"2019-03-21T06:59:37.308Z","log_type":"System","logseverity":"INFO","thread_id":1769973504,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"System: Starting purge of temporary files."}
{"timestamp":"2019-03-21T07:48:22.655Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"TheServer: Registering new server session"}
{"timestamp":"2019-03-21T07:48:22.655Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"QvSocket: Connected to server as user: 'Personal\Me' for document: '/app/' on socket <IP_ADDRESS>:9076 <-> <IP_ADDRESS>:53070 with ttl: 0"}
{"timestamp":"2019-03-21T07:48:22.667Z","log_type":"System","logseverity":"INFO","thread_id":1795151616,"user_id":"Personal\Me","tenant_id":"default","app_id":"0","message":"DOC loading: Loaded session app SessionApp_61ff6691-6af0-4fe5-98ae-c46c00d29864."}
{"timestamp":"2019-03-21T07:48:22.693Z","log_type":"System","logseverity":"INFO","thread_id":1778366208,"user_id":"Internal\Engine","tenant_id":"default","app_id":"","message":"StdServer: OnRemoteClosed entered for socket <IP_ADDRESS>:9076 <-> <IP_ADDRESS>:53070, user: Personal\Me, app: SessionApp_61ff6691-6af0-4fe5-98ae-c46c00d29864."}
{"timestamp":"2019-03-21T07:48:22.693Z","log_type":"System","logseverity":"INFO","thread_id":1803544320,"user_id":"Personal\Me","tenant_id":"default","app_id":"","message":"Session stop reason: Socket closed by client"}
{"timestamp":"2019-03-21T07:48:26.091Z","log_type":"System","logseverity":"INFO","thread_id":1803544320,"user_id":"Personal\Me","tenant_id":"","app_id":"","message":"DOC loading: Unloading document SessionApp_61ff6691-6af0-4fe5-98ae-c46c00d29864, state UNLOADING."}
When i run this command docker exec -it  sh -c "ls /data" and get below error:
PS C:\Project\QlikCore_data\core-get-started-master> docker exec -it 8c0838f58783 sh -c "ls /data"
ls: /data: No such file or directory

So the last command shows that the /data folder is not actually mounted into the engine container, which leads to that when loading data from movies.csv the file/folder is not found inside of the container. I suggest that you try to share your C drive again in Docker for Windows by unchecking and checking the box again. You will also need to run docker-compose down for the running containers and then start them again ACCEPT_EULA=yes docker-compose up -d. I have seen similar problems like this before when e.g. domain password has been changed.

Hi @wennmo ,
ACCEPT_EULA=yes docker-compose up -d this command doesn't work for me to start the container. Currently am using docker run -p 19076:9076 qlikcore/engine:12.345.0 -S AcceptEULA=yes this command to start container. Am i missing something in the command in order to mount the /data folder? Or how to mount the /data folder with docker command?
Thanks.

Ah ok, yes the volume mount for /data is declared in the docker-compose.yml file so if you want to use docker run instead you will need to run something like docker run -p 19076:9076 -d -v $(pwd)/data:/data qlikcore/engine:12.345.0 -S AcceptEULA=yes.
What shell are you starting the container from? If you are unable to pass env variable ACCEPT_EULA to the docker-compose command, you can always set it directly to yes in the docker-compose.yml file e.g. command: -S AcceptEULA=yes.

Hi @wennmo ,
I am using Windows PowerShell.
This is what i get when i run docker run -p 19076:9076 -d -v $(pwd)/data:/data qlikcore/engine:12.345.0 -S AcceptEULA=yes :
PS C:\Project\QlikCore_data\core-get-started-master> docker run -p 19076:9076 -d -v $(pwd)/data:/data qlikcore/engine:12
.345.0 -S AcceptEULA=yes
C:\Program Files\Docker\Docker\Resources\bin\docker.exe: invalid reference format.
See 'C:\Program Files\Docker\Docker\Resources\bin\docker.exe run --help'.
There is command error when i run this.

Hi @kokyeesiang, if you are using powershell then try replacing $(pwd) with ${PWD}. Different syntax in powershell compared to bash. I think you should try to get docker-compose up and running as well, since that will help you a lot when spinning up multiple containers or volumes.

Hi @kokyeesiang, did you get it running? Let us know if you are still experiencing problems.

Closing issue due to inactivity. Feel free to reopen if problem still exists.


Rewrite subsection on span context

I think this more or less resolves https://github.com/open-telemetry/opentelemetry.io/issues/1443
I wasn't very happy with the definition in place here, so hopefully this better explains it.

Preview: https://deploy-preview-1537--opentelemetry.netlify.app/docs/concepts/signals/traces/#span-context

@svrnm @austinlparker - thoughts?


[6.0] Delete max-width in small viewports for Cards

Explanation: Removes max-width for cards in small viewports
Scope: Impacts pages with card content viewed in small viewport widths
Issue: rdar://127050244
Risk: Low, single CSS property removal
Testing: Manually tested that pages with card content look good on varying viewport width sizes
Reviewer: @mportiz08
Original PR: #840

\cc @marinaaisa

@swift-ci test


Save stream position for streaming videos

Save stream position for streaming videos, i.e not downloaded video and videos playing on chromecast
Description
LEARNER-7505
This PR adds support for resuming playback of streamable videos, i.e. videos streaming locally or videos streaming on Chromecast device. Should save playback time and resume from where the learner left off the video when it is opened again.
How to test this PR
-At the moment we are saving played progress of downloaded videos such that when a learner leaves a video at some point and comes back to play the video again it'll resume from the same point.
This behavior needs to be implemented for streamable videos as well like it works on Android.
*Acceptance Criteria:

Streamable videos should resume playback from wherever the learner left them.
Streamable videos that are playing on Chromecast, should resume playback from wherever the learner left them, should resume on local player where left off from Chromecast.
If app is killed and video is being casted to remote device, when app opens again, then it should save progress time of video after connection of app and remote device has been established.

@saeedbashir
Pleasee have a look again.

@saeedbashir
Please have a look.

@saeedbashir
Please have a look.

@salman2013 Please have a look.

@mumer92 Please update the PR description as it is old one which says the progress is saving just for downloadable videos.


language-nix: expect the test suite to succeed

Version 2.1 fixes the issues we've previously had. The update changes
the package's API, but that's okay because it doesn't have any users in
Stackage yet.

Sorry about closing / re-opening this PR. I accidentally pushed to the wrong branch a minute ago. Fixed now.


Update quickstart-jailbreak.md

#sign-off

@PatrickFarley - The article articles/ai-services/content-safety/quickstart-jailbreak.md might benefit from a copy edit. An editor will notify the listed author and recent contributors.


How to Send ListOfStruct Data in Row Format in C# using Arrow

Describe the usage question you have. Please include as many useful details as  possible.
Specifically, I am wondering how to send ListOfStruct data in a format where each row of data is sent individually. The desired format is similar to the following:
[     {         "records":[             {                 "table_name":"t1",                 "t1":199999,                 "col1":"v11"             },             {                 "table_name":"t2",                 "t1":199999992,                 "col1":"v12"             }         ]     } ]
I came across the following function in the Arrow GitHub repository:
https://github.com/apache/arrow/blob/main/csharp/test/Apache.Arrow.Tests/StructArrayTests.cs#L89
However, the output generated by this function is still in columnar format.
I should do? Or where can I find the answer?
Component(s)
C#

There is no way to do this today.  There has been discussion in the past of a "packed struct" but this has not been adopted and no implementation has been proposed.  Sometimes the fixed size list type can be used but it is not exactly what you are looking for.

There is no way to do this today. There has been discussion in the past of a "packed struct" but this has not been adopted and no implementation has been proposed. Sometimes the fixed size list type can be used but it is not exactly what you are looking for.

Thank you for your reply, it was very helpful. I'm glad to report that my issue has been resolved, and we ended up choosing Column Format data transmission.
I tried using the Schema definition below, it seems to have the effect of "packed struct", it doesn't look very elegant.

recordType = new StructType(colField);
var recordsType2 = new StructType(new List() {
new Field("ITEM", recordType, true)
});


Desktop Support?

EzValidator is fantastic! I was wondering if there are plans to expand support to desktop platforms and what such an endeavor might entail?

Hello yes i will try to update it to support all platform cuz it's dart code in the end he has no affect on other platforms

That's fantastic news! I will add EZValidator to my list of things to watch! I can close this issue if you'd like. I was just interested to know if you would be supporting it for a project I have in the works.
Thank you! Appreciate your work!

i will tag you here once i upgrade it to close the issue no worries

Hello, you can try the plugin before I make a new release I already updated it to support all platforms :
ez_validator:
  git:
    url: https://github.com/X-SLAYER/EzValidator.git

Hello, you can try the plugin before I make a new release I already updated it to support all platforms :
ez_validator:
  git:
    url: https://github.com/X-SLAYER/EzValidator.git

Sorry I never got back to you on this. For some reason GitHub stopped giving me notifications since halfway through August. I thought everyone was just ignoring me. 🤣
I saw the package has been updated! Can't wait to test it out!


superclass mismatch for class MysqlAdapter

I am getting a superclass mismatch for class MysqlAdapter (TypeError) on a Rails 2.3.5 application. It works fine with version 0.5.0, or if I remove the inheritance from AbstractAdapter in the class definition.
superclass mismatch for class MysqlAdapter (TypeError)
/usr/lib/ruby/gems/1.8/gems/database_cleaner-0.5.2/lib/database_cleaner/active_record/truncation.rb:14
/usr/local/lib/site_ruby/1.8/rubygems/custom_require.rb:31:in gem_original_require' /usr/local/lib/site_ruby/1.8/rubygems/custom_require.rb:31:in require'
/usr/lib/ruby/gems/1.8/gems/activesupport-2.3.5/lib/active_support/dependencies.rb:158:in require' /usr/lib/ruby/gems/1.8/gems/database_cleaner-0.5.2/lib/database_cleaner/configuration.rb:86:in orm_strategy'
/usr/lib/ruby/gems/1.8/gems/database_cleaner-0.5.2/lib/database_cleaner/configuration.rb:42:in create_strategy' /usr/lib/ruby/gems/1.8/gems/database_cleaner-0.5.2/lib/database_cleaner/configuration.rb:56:in strategy='
/home/eric/projects/camp/features/support/env.rb:55
/usr/local/lib/site_ruby/1.8/rubygems/custom_require.rb:31:in gem_original_require' /usr/local/lib/site_ruby/1.8/rubygems/custom_require.rb:31:in require'
/usr/lib/ruby/gems/1.8/gems/cucumber-0.7.1/bin/../lib/cucumber/rb_support/rb_language.rb:119:in load_code_file' /usr/lib/ruby/gems/1.8/gems/cucumber-0.7.1/bin/../lib/cucumber/step_mother.rb:84:in load_code_file'
/usr/lib/ruby/gems/1.8/gems/cucumber-0.7.1/bin/../lib/cucumber/step_mother.rb:76:in load_code_files' /usr/lib/ruby/gems/1.8/gems/cucumber-0.7.1/bin/../lib/cucumber/step_mother.rb:75:in each'
/usr/lib/ruby/gems/1.8/gems/cucumber-0.7.1/bin/../lib/cucumber/step_mother.rb:75:in load_code_files' /usr/lib/ruby/gems/1.8/gems/cucumber-0.7.1/bin/../lib/cucumber/cli/main.rb:53:in execute!'
/usr/lib/ruby/gems/1.8/gems/cucumber-0.7.1/bin/../lib/cucumber/cli/main.rb:25:in execute' /usr/lib/ruby/gems/1.8/gems/cucumber-0.7.1/bin/cucumber:8 /usr/bin/cucumber:19:in load'
/usr/bin/cucumber:19

Is it this issue fixed in 0.7.0, I'm using MySQL on Jruby but getting superclass mismatch for class PostgreSQLAdapter

No, this wasn't fxed in 0.7.0.

Having the same issue as @krisleech. Using Oracle on JRuby and getting superclass mismatch for class PostgreSQLAdapter when running my cukes

Unfortunately downgrading to 0.5.0 doesn't seem to be an option for me. I don't believe that a plugin or other gem is responsible as with @elitwin.
So is this just an incompatibility with Rails 3.1.x and database_cleaner 0.7 under some circumstances?

I haven't heard of any other reports of it not working with Rails 3.1.x, but my tests are not working with rails 3.1.x.  Can you reproduce the issue on a fresh rails 3.1.x app?

I can reproduce it in a fresh Rails 3.1.3 app using JRuby 1.6.5 (which is my target environment), only with the truncation strategy. I can not reproduce it in a fresh app using CRuby 1.9.2.
I'm using cucumber-rails in both cases.
So perhaps it has something do with a difference in the way the JDBC based adapters are implemented / loaded compared to the "regular" ones.

Ah, good to know.  Any other plugins?  Would you mind uploading the bare jruby rails app to github?

Here it is:
git@github.com:svoynow-lz/database-cleaner-fresh.git

The Pull Request https://github.com/bmabey/database_cleaner/pull/83 fixes the issue for me. But please double check that it didn't break anything else.

That's right. No any changes.

Yep, this works for me too.
Please pull @dnagir's fork.
Thanks on behalf of all the jruby users!

It has been merged, but not released yet.. Please use bundler to point to the repo for now.

Hi all.  Today I started trying to see if I could upgrade a Rails 3.0 app to 3.1 and I'm getting this error as well.  Surfaces using cucumber with ruby 1.9.2 and ... unlike the others here, we are actually using PostgreSQL (9.1).  I have been pulling from your repo for months to get some PostgreSQL-specific TRUNCATE ALL behavior, and this appears to have been triggered by setting up a new gemset to work with Rails 3.1.

The error goes away if I use 0.7.0.  My issue is the "superclass mismatch for class PostgreSQLAdapter" error that people have been talking about since 21 Nov, and not the original issue (superclass mismatch for class MysqlAdapter) (sorry).

@erictheise see this -> https://github.com/bmabey/database_cleaner/pull/85

I had the same problem with database_cleaner 0.7.0 but only under JRuby. MRI (well, RubyEE) was untroubled.
Rails 3, MySQL, RSpec (not cucumber)
Ruby EE 1.8.7 with database_cleaner 0.7.0 -- no errors
JRuby 1.6.5 with database_cleaner 0.7.0 -- superclass mismatch for class PostgreSQLAdapter
JRuby 1.6.5 with database_cleaner 0f1bf4239a7d56f4dbb59147b809477b7954a76e -- no errors
Ruby EE 1.8.7 with database_cleaner 0f1bf4239a7d56f4dbb59147b809477b7954a76e -- no errors
I also tried with the tip of master 0e42715d1fca9bda7054a22d9d0f5666da1f97ae and got the same results.
Basically it seems like everything is fine now. Big thanks to @dnagir and @bmabey.
+1 for rolling into these changes into a release :-)


fix: misleading doc that states resource (cpu/mem) scalers uses limit

Fixes https://github.com/kedacore/keda-docs/issues/1028
This scaler uses resource requests not limits.

@zroubalik should be all set now

Hi @JorTurFer @tomkerkhove  @zroubalik ,
I can see that there is no activity for this fix from some weeks so if all of you don't mind can i fix with proper DCO ?

@mootpt, could you solve the DCO issue? You have done the work and should be who is referenced in the commit log 😄

I close this PR as abandoned :(


Use Bazel-provided auto-configured toolchains

The auto-configured toolchains provided by Bazel itself have diverged heavily from the version maintained in this repo. There is no need to maintain this duplication until Starlarkification has progressed succiciently for rules_cc to be the source of truth for them.
This is particularly relevant for Bzlmod, which currently uses the toolchains defined in rules_cc. As a result, Bazel C++ builds will use subtly different toolchains depending on whether --enable_bzlmod is used or not. This is fixed by loading toolchain detection logic from @bazel_tools in the module extension.

This got rolled back in https://github.com/bazelbuild/rules_cc/commit/bc665f92712b343183e748d18955113ab06d1cca

It got rolled back, but in the meantime Bazel's toolchain has finally been moved to rules_cc (in Bazel 8).
The linked issue could be fixed in rules_cc, but the non-apple_support toolchain for macOS is not actively maintained.

Good point, thanks. Commit c2549f6eb07ce00e529b84f9b9f1b31135a9ff77 has added a recent copy of the toolchain configuration code; maybe cc_toolchain_config.bzl is unused now and can be deleted entirely?

Sorry, I mean https://github.com/bazelbuild/rules_cc/blob/main/cc/private/toolchain/cc_toolchain_config.bzl. That one still has -undefined dynamic_lookup, but seems unused.


Started off with this issue:

Google civic returns the same google_civic_election_id for primaries
that occur in multiple states on the same day, so we need to persist
the state_code most places where we store google_civic_election_id.
Added (working) error handling when receiving un-successful API queries.
Error response in these error conditions provides text that describes
the problem, instead of silently failing.
There was so much inconsistently "repeated" code, that resulted in
silently ignored errors, so I added process_request_from_master().
Reworked sync_data_with_master_dashboard.html so that it was more
understandable by newbies, and so that errors bubbled up to the
web page (instead of failing silently).
If I knew at the beginning how big this was going to become, I would
have broken it up into multiple PRs.
I carefully inspected (many times) the code that runs on the master server
and loads the original master copy, but could not fully test it on my local.
This change requires a DB migration.

@SailingSteve, thank you for taking on this Herculean task of adding state_code! I have been through all of the code, and it looks great. I will also run some tests locally. 👍


Add simple functional test for SEV

Test cases
Scenario: Can create an attested pod from an encrypted signed container image
Given I have a version of CoCo deployed on Kubernetes with this operator
And The environment is configured with simple-kbs
When I try and create a pod with kata-qemu-sev runtime from a public encrypted signed container image
Then The pod is successfully created
Questions

Q1: Should it use an encrypted signed container or simply an unencrypted image?
Q2: Will the environment be configured with policy to exercise that integration too?
Q3: Same question for CDH (Confidential Data Hub)
Q4: Will we try to make the tests shareable with other projects (e.g. Kata Containers and peer pods)?
Q5: will we try to reuse https://github.com/kata-containers/tests/blob/CCv0/integration/kubernetes/confidential/sev.bats ?

Additional information

Trying to respond to your questions:
Q1: Maybe we handle all cases?
Q2: Great question. Probably we need this variation, too.
Q3: Not sure I understand; all of these cases would require CDH, no?
Q4: This part is beyond me still; reusing some of these deployment and testing components is ideal, if possible.
Q5: Great question. I assume this script is battle-tested, so if we don't use it, we need to take a lot from it.


Store gitops runtime definition in git

Short description
As a platform operator I want to check the platform configuration into a repository (to start the same one that contains the apps)  so that configuration is continuously reconciled by the gitops engine, and can be updated from git. (Preventing operational errors, and enabling faster recovery when errors are introduced). '
Gitops Platform Repo:  The repository where the gitops runtime is stored.
Gitops Application Repo:  The repository where apps are added.   Apps either be local to this repo, or reference another repository which contains that apps config.
Gitops Automations:  The source, helm, and customization controller settings that automates the gitops runtime for an set of declarative models stored in git (Apps and gitops runtime).
The default setup should be a monorepo which contains both the app repo and the platform repo.
If you don't specify a URL we are not going to store the Gitops Runtime config (gort).
CLI experience: https://miro.com/app/board/o9J_l7vcDC8=/
User Story
As a the operator responsible for setting up gitops (platform operator) I would like to run wego gitops install and supply it with an empty repository and get fully functioning weave gitops runtime installed in the cluster, with application config AND platform config stored in that repository.
There are two additional cases which need to be considered:

The platform may be managed by a separate team than the applications, in which case it may be desirable to split the Wego Platform and Wego App definitions into separate repositories.
Wego may already be installed in a cluster and we may need to retrieve the gitops runtime config from the cluster, persist it to git, and update the cluster.  This shoud not be implemented in this ticket, it will be split out into another ticket.

I think there are four things that need to happen, some of which can be automated together:

get platform definition from the running cluster
put that definition in the right place in the repo
create the the platform sync definition
put that definition in the repo, and update flux to use definition from the repo

Acceptance criteria

[ ] AC/1
Given that I have a wego binary with a working install command
And I have a .kube/config file that references an active Kubernetes cluster
And I have a platform repository or an app repository with infraRepo=true in its config file
When I runwego install against the active cluster
Then the gitops runtime manifests (CRDs, CRs) are added to the cluster and stored in the repository
And any changes made to the manifests will be applied to the cluster via flux when they are merged
[ ] AC/2
Given that I have a wego binary with a working install command
And I have a .kube/config file that references an active Kubernetes cluster
And I do not have a platform repository or an app repository with infraRepo=true in its config file
And GITHUB_ORG is set in the environment
When I runwego install against the active cluster without parameters
Then a private repository is created with a name derived from the active cluster, in the org referenced by GITHUB_ORG
And the gitops runtime manifests (CRDs, CRs) are added to the cluster and stored in the repository
And any changes made to the manifests will be applied to the cluster via flux when they are merged
When I runwego install against the active cluster with any of --organization, --public, --repository-name
Then a repository is created, with default values overridden by the arguments
And the gitops runtime manifests (CRDs, CRs) are added to the cluster and stored in the repository
And any changes made to the manifests will be applied to the cluster via flux when they are merged
[ ] All the configuration needed to re-setup weave gitops on the cluster is stored in git
[ ] Any drift from upstream gitops platform definitions are either "reconciled away" or are reported as errors.

Design materials

Screenshots
Design diagrams
WEP: https://github.com/weaveworks/weave-gitops-private/blob/wep-wego-recover/docs/weps/WEP-002-Weave-gitops-recover.md
PR/FAQ

Work breakdown
(optional)Task breakdown of the work

[ ] create X
[ ] design X
[ ] ...

@jrryjcksn add the given-when-then syntax so we can refine.

We will try to break this up into two smaller stories

Need some UX discussion on how we identify the platform repository.
We are going to expose the .wego platform repository up front.
Platform folder should be configurable.

Depends on #560


[SPARK-11091] [SQL] Change spark.sql.canonicalizeView to spark.sql.nativeView.

https://issues.apache.org/jira/browse/SPARK-11091

Merged build triggered.

Merged build started.

Test build #43676 has started for   PR 9103 at commit 5a9e388.

also need to change this comment https://github.com/apache/spark/blob/master/sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveQl.scala#L540

Merged build triggered.

Merged build started.

LGTM, pending test.

Test build #43682 has started for   PR 9103 at commit 6e7190e.

Test build #43682 has finished for   PR 9103 at commit 6e7190e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43682/
Test FAILed.

test this please

Test build #1893 has started for   PR 9103 at commit 6e7190e.

Merged build triggered.

Merged build started.

Test build #43688 has started for   PR 9103 at commit 6e7190e.

Test build #43676 has finished for   PR 9103 at commit 5a9e388.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43676/
Test PASSed.

Merging to master.

Test build #1893 has finished for   PR 9103 at commit 6e7190e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #43688 has finished for   PR 9103 at commit 6e7190e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43688/
Test PASSed.


Flutter Gallery Sample Cupertino Enabled switch is shown as disabled.

Bug Line : https://github.com/flutter/flutter/blob/master/examples/flutter_gallery/lib/demo/cupertino/cupertino_switch_demo.dart#L65

enabled here means interactive, which is orthogonal to it having a true or false value. Are there elements of the doc we should improve on?

Hm.. it's confusing. The top most switch says active which means it can be changed. The second and third says disabled. I inferred that it meant the state of switch and thought the second one should have been enabled. Anyhow, if enabled means interactive shouldn't the first switch specify enabled and next two disables or the first should say active and next two inactive.

You're right. It could be clearer. Commented on your PR. Thanks for your contribution


Secret manager needs GOOGLE_APPLICATION_CREDENTIALS env var

When using secret manager and configuration items loading via sm//my.database.password for example, if the GOOGLE_APPLICATION_CREDENTIALS env variable is not set the application fail with
Failed to load config value of type class java.lang.String for: my.database.password

/cc @dzou

Not an issue, I was missing my projectId/serviceAccount settings :(


Review editing

As a... reviewer
I want to... be able to edit and improve my reviews
In order to... feel less in fear of being unfair to the first submissions I evaluate, and spend less time obsessing over the best way to express myself right away.

hahahaha
Yeeey! I'm really glad this is being developed. :-)
Let me know when to test it on Staging, please.

A few things to keep in mind:
Review edition should be limited to review periods. One should be able to
edit their reviews prior to the review deadline but no longer after.
Also, I'm guessing that you're guaranteeing that only the review owner can
edit that review :)
Cheers,
Hugo Corbucci
Skype user: hugo.corbucci
Twitter: hugocorbucci
On Fri, Apr 22, 2016 at 12:05 PM, Cecilia Fernandes <
<EMAIL_ADDRESS>wrote:

hahahaha
Yeeey! I'm really glad this is being developed. :-)
Let me know when to test it on Staging, please.
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub
https://github.com/agile-alliance-brazil/submissions/issues/197#issuecomment-213556307

You don't need to guess, you can check. =P
But yes, the reviewer can edit its own reviews only.

Até onde eu sei ela está implementada.. meus comentários aí em cima são evidências de funcionamento e não tem nenhuma depois deles dizendo que tem problema. Tem?

Eu estou recebendo um "Você não está autorizado a acessar esta página", na edição da minha própria review em producão. Como a issue estava como open, eu não tinha certeza se estava de fato implementada.

Na real você deve estar sem permissão mesmo. Algum admin disponível para validar?

We need to check if this is working already?

@hugocorbucci I don't recall seeing this feature (it might be there..)
I would say its low prio to verify if its done or not
[I see a lot of value on this one as well, even thought it might lead to undesired side-effects such as a reviewer changing their mind for unexplained/shaddy reasons]


Homematic resolvenames: unable to open session

Home Assistant release with the issue: 0.97.1

Last working Home Assistant release (if known): 0.95.4, but note: after I reinstalled this, it is broken here too. I suspect something wrong in the dependency?
Operating environment (Hass.io/Docker/Windows/etc.): OSMC + venv

Component/platform: homematic

Description of problem:
My names are not resolved anymore. It was working until my SD card broke. I reinstalled hass. Now it does not work anymore.
I tried pure raspian, then switched back to OSMC (also raspian). I installed latest HA, then switched back to last working version. I even updated the CCU software. All of this to no avail. :-/
Was there some change lately? Probably in pyhomematic?
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
homematic:
  interfaces:
    rf:
      host: !secret homematic_host
      resolvenames: json
      username: !secret homematic_username
      password: !secret homematic_password
      #primary: true
      #variables: true
    ip:
      host: !secret homematic_host
      port: 2010
      resolvenames: json
      username: !secret homematic_username
      password: !secret homematic_password

Traceback (if applicable):
Aug 17 13:16:54 ak-test hass[5925]: 2019-08-17 13:16:54 DEBUG (Thread-22) [pyhomematic._hm] RPCFunctions.addDeviceNames
Aug 17 13:16:54 ak-test hass[5925]: 2019-08-17 13:16:54 DEBUG (Thread-22) [pyhomematic._hm] RPCFunctions.addDeviceNames: Getting names via JSON-RPC
Aug 17 13:16:54 ak-test hass[5925]: 2019-08-17 13:16:54 DEBUG (Thread-22) [pyhomematic._hm] RPCFunctions.jsonRpcPost: Method: Session.login
Aug 17 13:16:54 ak-test hass[5925]: 2019-08-17 13:16:54 DEBUG (Thread-22) [pyhomematic._hm] RPCFunctions.jsonRpcPost: API-Endpoint: http://ak-ccu2:80/api/homematic.cgi
Aug 17 13:16:54 ak-test hass[5925]: 2019-08-17 13:16:54 WARNING (Thread-22) [pyhomematic._hm] RPCFunctions.addDeviceNames: Unable to open session.

Additional information:
http://ak-ccu2/api/homematic.cgi shows a list of json api points in the browser.
My devices are also all loaded. Just not named correctly.

Please try using this script. It's intended for testing the JSON-RPC API and provides a little bit more output to help find what's causing your issue.
My first guess would be, that the IP address of your new system is different, and this new IP address is not whitelisted in the firewall of the CCU. Hence authentication is failing.

Thanks for the script! I tried that and it seems to work fine. As far as I can tell. Here’s the output.

(Sorry for closing and reopening.)
Thanks for the script! I tried that and it seems to work. As far as I can tell. Here's the output with sanitized serial numbers. I hope, that is sufficient.
 Are you executing the script from the same machine where Home Assistant is running on? If so, then generally your CCU allows your Home Assistant machine to access the API. In that case the only reas that comes to my mind why it could not be working would be, that the username / password combo you have configured actually don't match what you had configured presviously. Because technically this test script does exactly the same as the pyhomematic library does. So please, just for testing purposes, add the username and password you have used for the test-script directly in your HomeMatic configuration instead of using the secrets. Maybe the component has trouble working with the secrets-implementation. I don't know why suddenly this should be the case, but a quick test would rule out if this is an issue.

Thank you very much for that obvious hint. How embarrassing: I missed the capital “A“ in Admin and had it all lower-case... Once I changed that, it works! The credentials are still in the secrets file.
Interesting to note however that the CCU2 did accept the “wrong“ username for getting the states.

👍

Interesting to note however that the CCU2 did accept the “wrong“ username for getting the states.

You mean the states of the entities? That's normal. JSON-RPC is only used to get the names of the entities. The entities themselves and their states are done with XML-RPC (those are on ports 2001, 2010 etc.). And the XML-RPC API (sadly) does not have any form of authentication.

Ah, I see. Yes, the states of the entities were there. Just not the names...
Do you plan to switch over to the JSON-RPC for the states of the entities?

Do you plan to switch over to the JSON-RPC for the states of the entities?

No. The JSON-RPC API does not provide any mechanism to push states to Home Assistant (or any RCP-Client for that matter). States would always needed to be polled, which is rather unefficient. Besides that, only the CCU has the JSON-RPC. So switching to it would break the HomeMatic integration for everybody who is using Homegear or some other solution.


Add example scripts

What does this PR have?

Add transaction scripts
Add deploy contract scripts

@AngeloCG97  lets add a readme file based on the templete @leisterfrancisco  has been using


Is it Possible to Support Configuring the “usepodnetwork” Option as a Parameter in kube-loxilb

Background: I previously used VMware’s NSX ALB (AVI) as a load balancer for Kubernetes, which supports both LoadBalancer and Ingress, and advertises Pod routes to the Controller. This setup allows bypassing kube-proxy, with the load balancing strategy applied only at the load balancer level. The deployment architecture was similar to LoxiLB’s external cluster architecture. Currently, I am using LoxiLB as a replacement, and my architecture is as follows:
1、Two Fedora Core virtual machines are used as external LoxiLB nodes (running Docker).
2.、The external LoxiLB nodes have two network interfaces: one for the VIP and the other for communication with Kubernetes cluster nodes.
3、The Kubernetes cluster is built using Talos Linux with Cilium as the CNI (without kube-proxy).
4、A BGP connection is established between the external LoxiLB and the cluster through the Cilium Agent, and BGP connections are configured between kube-loxilb and Cilium for LoxiLB to learn Pod routes.
5、Kube-loxilb is configured with --externalCIDR set to LoxiLB’s VIP range.

I have successfully implemented direct traffic forwarding from the external LoxiLB to Pods and can flexibly use LoxiLB’s load balancing algorithms.
LoxiLB is amazing, and I really like it.
However, there are a few areas that are not ideal:
1、The “usepodnetwork” option can only be configured on a per-service basis and cannot be set globally in kube-loxilb.
2、The --setBGP and --extBGPpeers parameters in kube-loxilb can only be used for either external or internal configurations. It becomes difficult to have the external LoxiLB establish BGP connections with both physical routers and Kubernetes cluster nodes simultaneously. Supporting separate sets of BGP parameters for external and internal use would greatly enhance flexibility. Alternatively, enabling kube-loxilb to directly publish Pod routing tables to LoxiLB (while keeping the existing parameters unchanged) would be beneficial.
I am aware of potential architectural issues with advertising Pod routes to LoxiLB, such as routing conflicts when multiple Kubernetes clusters use the same PodCIDR. Currently, I define unique PodCIDRs for each Kubernetes cluster to avoid overlaps. I am curious if there are better solutions for handling identical PodCIDRs, such as using VRF technology.

Hi @6547709
Thanks for the detailed scenario and explanation.  Answers to some of your questions :

1、The “usepodnetwork” option can only be configured on a per-service basis and cannot be set globally in kube-loxilb.

This can and will be taken up as an enhancement.

2、The --setBGP and --extBGPpeers parameters in kube-loxilb can only be used for either external or internal configurations. It becomes difficult to have the external LoxiLB establish BGP connections with both physical routers and Kubernetes cluster nodes simultaneously. Supporting separate sets of BGP parameters for external and internal use would greatly enhance flexibility.

On this end, we have been working on BGP configuration using CRDs. Although this is in beta, we are trying our best to get it into good shape. Please give it a try. The community will help if you run into any trouble.

Alternatively, enabling kube-loxilb to directly publish Pod routing tables to LoxiLB (while keeping the existing parameters unchanged) would be beneficial.

Need some more pondering on this.

Thanks for your help, these are not real operational issues, but my ideas to make LoxiLB more powerful^_^
I will test CRD configuration BGP routing;
Regarding the announcement of Pod routes, currently, I am going to write a small program to automatically update the Pod routes through the LoxiLB API interface (/config/route).

But I found a problem with the API. The current API does not have any authentication. Anyone can directly operate the LoxiLB API. Did I miss some authentication configuration?

Yes you are right. In the process of supporting it via this along with OAuth based authentication scheme.


VRP with breaks minor bug?

What version of OR-Tools and what language are you using?
Version: stable branch
Language: C++
Which solver are you using (e.g. CP-SAT, Routing Solver, GLOP, BOP, Gurobi)
Routing Library
What operating system (Linux, Windows, ...) and version?
Linux Mint 21.3
What did you do?
Using VRP with breaks. GlobalVehicleBreaksConstraint will be added to the Solver.
What did you expect to see
I believe here we're updating StartMin value of the arc path_[pos] -> path_[pos + 1] to EndMin value of the break interval. This arc StartMin is CumulVar(path_[pos])->Min() - arc_start_offset. So instead of SetMin(CapSub(interval_end_min, arc_start_offset)) it should be SetMin(CapAdd(interval_end_min, arc_start_offset)).
https://github.com/google/or-tools/blob/5124f4975bd6ab5a863ea55657833ab640730429/ortools/constraint_solver/routing_breaks.cc#L943-L951

You are correct, this should be a CapAdd() instead of a CapSub().
The current SetMin() is weaker than it could be.
Thank you for the report!

So all we have to do is to replace the CapSub with the CapAdd ?
Or is there anything else which I should consider?

yeah, replace CapSub by CapAdd will do

Okay creating a PR :)


fix: Search view elements getting overflowed out of screen

fixed :#960 Search view getting overflowed out of the screen
removed the extra parent linear-layout, removed the top margin give the parent linear-layout gravity centre and also layout_gravity centre
fix: #960

@GOVINDDIXIT  @nikit19 I am reopening this PR after consulting with @RebelsterPi . @RebelsterPi you can close your PR #966


feat: live link support

Related Issue #321
closes #321

Description
Added support for the live link of projects

Screenshots

when a user clicking on the more button it's showing a black screen

when a user clicking on the more button it's showing a black screen

Hmm strange will check it again

we're thinking to improve to redesigning the card & adding new features such as analytics etc. so now I'm closing this PR.


Can't resize clamped outputs

Expected Behavior
When I grab onto the resizer thingy on the output, it should let me resize the height up and down:

Actual Behavior
I can only resize up (to make it smaller), but not down (to make it larger). This is rather inconvenient with Image outputs.

@paullewis  for when you're back


Identity based connections - unclear guidelines

The guiding material given in this section is unclear. The following sentences does not provide much info. Which parameters should be created, and in which files? What does "a common prefix mean"? The extension requires the following properties, but where should the properties be defined?
"To do this, you would define settings under a common prefix which maps to the connection property in the trigger and binding configuration.
In this mode, the extension requires the following properties:"
A spelled-out example would perhaps help.

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

ID: 1d50399c-0873-e903-35ca-1845c84b6c6c
Version Independent ID: 1a3150b4-5eb9-0ac5-ee2b-d1bb6b6fd63e
Content: Azure Service Bus trigger for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus-trigger.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

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


scss/operator-no-unspaced: Interprets minus-sign of negative value as operator

If a function is wrapped into #{ and }, the minus-sign of negative numbers in the arguments is interpreted as operator.
div {
  // This throws `Expected single space after "-" (scss/operator-no-unspaced)`:
  --my-var: #{scale-color(#fff, $lightness: -75%)};

  // This works fine:
  color: scale-color(#fff, $lightness: -75%);
}

I'm using:
"stylelint": "13.0.0",
"stylelint-scss": "3.14.2",

Thanks!

Fix released in v3.16.1
https://github.com/kristerkari/stylelint-scss/releases/tag/3.16.1


Confirm dialog before closing the window if unsaved changes

Yeah, I noticed that the entered data is saved, and when page (issue, eg) are reopened they are inserted into the input fields, it's good if there is a sudden light outage, for example. But if I have many many tabs and write a comment in the issue and accidentally close it (:man_facepalming:), then it would be nice if the confirm dialog was displayed before the page was closed so that this one would not close. How do you like this idea?

I think this can be replaced by https://github.com/sindresorhus/refined-github/issues/1598


Run converter before constructing data rule string

Closes #418
Runs the value converter when calling the construct method of the parser, instead of waiting until the appropriate assignment is performed.

@spoenemann I refactored this PR to better deal with keywords and also perform the value conversion inline. This resolves the issue of having to do it recursively, which incurs some performance penalty (doing it recursively, leads to O(n^2) behavior).


Add ESLint configuration

Added ESLint to help ensure consistency across the codebase. The setup targets a Node.js environment, with standard style.

Thanks @MichaelPGriffin! I enhanced the eslint configuration I did; now it's in CommonJS format, and I added a rule to enforce semicolons. You can add more rules in the same place. There's a new pull request with these changes.


Add PLATFORM examples in testJobTemplate

It would be nice to have PLATFORM examples in testJobTemplate
https://github.com/adoptium/aqa-tests/blob/master/buildenv/jenkins/testJobTemplate#L267
We should list commonly used PLATFORM examples and add a link to https://github.com/adoptium/aqa-tests/blob/master/buildenv/jenkins/openjdk_tests
For example:
Platform to be tested:
aarch64_linux
ppc64_aix
ppc64le_linux
s390x_linux
s390x_zos
x86-64_linux
x86-64_mac
x86-64_windows
x86-32_windows
sparcv9_solaris
riscv64_linux
multiple platforms (comma separated values): x86-64_linux,x86-64_mac
all platforms: all (actual platforms will change based on JDK_VERSION/JDK_IMPL)
Please refer to PLATFORM_MAP in https://github.com/adoptium/aqa-tests/blob/master/buildenv/jenkins/openjdk_tests

I would like to work on this 🙃

This issue is still up for grabs...

Hi @smlambert, I am interested in working on this.
I understand what we need to add. But could you provide more clarity on where do I need to add the list of commonly used PLATFORM examples and the link?

yes, so this line stringParam('PLATFORM', "${ARCH_OS}", "Platform to be tested") can be updated to include some names of platforms we test, so it could be updated to
stringParam('PLATFORM', "${ARCH_OS}", "Platform to be tested:
 aarch64_linux
 ppc64_aix
 ppc64le_linux\n
where
 will insert a break, so that the view in the Jenkins job will look like:
Platform to be tested:
aarch64_linux
ppc64_aix
ppc64le_linux
s390x_linux
s390x_zos
x86-64_linux
x86-64_mac
x86-64_windows
x86-32_windows
sparcv9_solaris
riscv64_linux
multiple platforms (comma separated values): x86-64_linux,x86-64_mac
all platforms: all (actual platforms will change based on JDK_VERSION/JDK_IMPL)
Please refer to PLATFORM_MAP in https://github.com/adoptium/aqa-tests/blob/master/buildenv/jenkins/openjdk_tests

Once you have a PR @vegasood, I will help to test it

Okay thank you for clarifying Shelley :)
Excited to make my first contribution to an OS project


Mix multiple sources and one target

Spin off of https://github.com/sillsdev/serval/issues/266 for Serval implemenatation.
Mixing multiple sources from different NLLB-200 languages has shown to make a big bump especially if the backtranslation language is different than the source text (say English backtranslation with Spanish source text).  Including target sentences in multiple times makes the behavior worse (memorizing the target sentences), but interweaving half the target with 2 different sources works pretty well.  How can this be implemented:
Proposal:

If an engine has two corpora: (1) Source123:TargetABC and (2) Source456:TargetABC
A new optional parameter is added to corpora called mixingRatio which defaults to 1.0 - but can accept any valid float number including 1e100, etc..
If a textId appears multiple times in corpora, the target is mixed with the different sources according to the weight:

1.0-1.0 -> 50% and 50%: this is the default
1.0-1.0-1.0 -> 33%, 33% and 33%: can combine more than 2 source languages
1.0-2.0 -> 33.3% and 66.7%: by setting one corpora to a higher mixingRatio (such as 2.0), a source can take be aligned with more of the target sentences

Mixing is done with repeatable, random seeds, not just ABABABABABABAB.
If a source only covers a subset of the target, the mixing is done for the sources that are available on a segment by segment basis.

I.e., if there is only a backtranslation for Luke, but the source also has Mark and Matthew, Luke will be mixed as per the mixingRatio specified and Mark and Matthew will be exclusively from the other source.
To specify an ordering of sources, using exponents should be straightforward: 1e0, 1e10, 1e20, 1e30, each multiplying by a factor of 10,000,000,000, setting a clear priority for exclusive use by sources if they are available.

So, 50% random mix, if one source has no text, use the other source.  Only 2 sources needed.  Follow SILNLP implementation.

Proposal: for implementation, when posting the single corpora:

For text files, make two source texts have the same textId - and then Machine will mix it
For Paratext files, just put two zipped Paratext sources and the mixing will happen
We will mix however many sources there are
If there are multiple targets we error out. (UnsupportedException)

@ddaspit - here is a proposal of the changes:

In CorpusService, look for the text and parallel matching conditions, and if there are multiple for the source, combine them (2 or more) into a MixedTextFileText or a MixedScriptureTextCorpus (may need to be MixedZippedScriptureTextCorpus...), both inherited from TextFileText and ScriptureTextCorpus.
A MixedTextclass is made that the others inherit from that handles the radnomization
The Mixed Corpora handle the randomizing when initially being created - randomizing whenever they have matching id's, but are treated as normal text or Paratext files otherwise

We need API documentation for this feature.


Add alefvanoon.xyz

Important: Please read all instructions carefully.
Select the appropriate category for what this PR is about
This PR is:

[X] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

Do not tick a checkbox if you haven’t performed its action. Honesty is indispensable for a smooth review process.

[X] I used the uncompressed size of the site
[X] I have included a link to the GTMetrix report
[X] The domain is in the correct alphabetical order
[X] This site is not a ultra lightweight site
[X] The following information is filled identical to the data file

- domain:
  url: https://alefvanoon.xyz/
  size: 52.9
  last_checked: 2021-1-22

GTMetrix Report:
https://gtmetrix.com/reports/alefvanoon.xyz/ogrkm41E/

Site is actually 53.6KB, but also qualifies as "ultra minimal" so I won't be merging this PR, sorry.
Details here - https://512kb.club/faq


Don't trigger multiline completion in case of function or method invocation

Fixes https://github.com/sourcegraph/cody/discussions/358#discussioncomment-6519606
As it is described in GH discussion, we shouldn't make multiline completion for cases like function or method
invocation (but we should do this in case of method or function declaration).
So this PR gates multiline completion behind this regexp check (regexp isn't ideal, but this is what we have to
work with until we get parsers working) You can check this regexp here
Test plan

Check that completion doesn't try to do multiline complete if your cursor is within arguments
position of function or method invocation/call statement

@philipp-spiess

I think it's easier if we just look at the prefix. In my PR, I check whether there's also a . somewhere on the same line prefix. WDYT?

Yeah, I've seen your regexp. I think we should check prefix+suffix for several reasons.

The main reason is to detect declarations (sometimes that can look like invocations) like method.hello () { these should be treated as declarations because of { body bracket
hello( could be many things. We just can't say much about what exactly we want to have here

But I see your point about hello.world( probably we can do both with this regexp (\b[^()]+\((.*)\)$)|(\..*\($) ?

@vovakulikov

The main reason is to detect declarations (sometimes that can look like invocations) like method.hello () { these should be treated as declarations because of { body bracket

Can you clarify this? I think this is actually a good reason of why to only check for the prefix, because depending on where my cursor is, I want either a single line or a multi line completion. E.g.:

method.hello (█) {
Would expect this to be a method call and do a single-line completion
method.hello () { █
Would expect this to be the beginning of a block instead

@philipp-spiess
Cursor context This is something that we probably can achieve with proper lexeme type context around the cursor with regexp; it's too complex for a number of langs we're trying to support here. My point here is that method.hello (█) { regardless of cursor position will be handled as multi-line completion because of { in the suffix.

I updated unit tests and added some logic about empty/non-empty suffix, I hope it fixes both cases that you mentioned above (but it doesn't take into account context information based on cursor position)

@vovakulikov Cool, makes sense. Feel free to merge this, I’m sure you make the right call

I’m sure you make the right call

👍 good luck to me :)


Bundles disappearing

Description
Bundles disappear after some time.
Have no idea what could've caused it.
bundles.json's output returns null.
{
  "SavedBundles": null
}

Steps To Reproduce
Create a bundle
Wait a couple of days
Your bundle is gone
Expected behavior
Bundles wouldn't disappear
Files Version
Version: <IP_ADDRESS> OS Version: 10.0.22000.434
Windows Version
Windows 11 Home 21H2  22000.434
Relevant Assets/Logs

Merging with #7743

It happened to me after updating to 2.1

Asking @yaichenbaum for approval.

While this feature is still in the app, we're planning to replace it with a tags widget.


cli: default host to "localhost" in client commands

PR: https://github.com/cockroachdb/cockroach/pull/10134
@sploiselle, this ties into issues you were having with secure deployments, I think.

@asubiotto Does cockroachdb/cockroach#10134 change  the behavior of the --host flag in a way that it no longer listens on all interfaces? (Here's how the docs talk about the default --host behavior)

@sploiselle, I think the line in this week's release notes answers this:

The --host argument now defaults to localhost in both secure and insecure modes. #10134

I can update the flag description as part of pulling together release notes, though, so I can take this one.

@jseldess I think there's a level of complication here because it wasn't listening on localhost previously, despite the documentation saying it listened on all interfaces. I might be overthinking it, though.

This is a client side change. It affects the host that client commands connect to, but doesn't change (or isn't supposed to change) what the server listens on.

Ah, right, @bdarnell, as stated in the description. Sorry if I caused confusion, @sploiselle. But I do think this ties into one of the problems you were having on secure clusters. I think you said that, when trying to start the sql shell on the same machine as a node, you had to explicitly set --host. Now you shouldn't, if I understand correctly.


Provide training data for all language profiles

danielnaber mentioned that language profiles should come with the training text they are based on.
I totally agree with that. This allows anyone to play with customizations to improve the profiles, such as:

playing with profile size by setting the word count cutoff higher or lower
cleaning the text from foreign words, English phrases, ...
changing n-gram types (3-gram, 4-gram, ...)
debugging and understanding certain results
use cases we can't even think of now

The readme text for contributions should be updated to kindly ask for the training text. The best would be to get the original training text, and the program that applies modifications on it to make the index.
Since not everyone who would like to check out the language detector needs these training texts, I'd vote for keeping them separate. Otherwise a simple checkout becomes very large.
On the other hand, if there are 150 GitHub projects for 150 languages, and one would like to try out 4-grams on all of them, that's quite some work also... opinions on that?
I don't recall the state of the current language profiles - I just took what was there. I'll have to see if the original texts are (easily) accessible.

I think one project that contains the texts for all languages would be okay, but it depends on how large the data is. I like the idea of having the original data and a script that cleans it up, but I'm not sure how practical that is. For example, one might want to remove sentences with a lot of names, proper nouns etc and detecting those isn't trivial.

Well...
From the original language detection project wiki

Generate language profiles from Wikipedia abstract xml

From Nakatani Shuyo's tools wiki

This tool generates language profiles from Wikipedia abstract database files or plain text.

Wikipedia abstract database files can be retrieved from "Wikipedia Downloads" ( http://download.wikimedia.org/ ). They form '(language code)wiki-(version)-abstract.xml' (e.g. 'enwiki-20101004-abstract.xml' ). See also LanguageList about language code.

We can just download the database from Wikipedia (albeit the date and version will be different) if the profiles here are just pulled as is from the original.
Also seems like danielnaber also used some tatoeba.org stuff for Esperanto :+1: I've used that site also haha.