Compatibility with Apache Superset

We're considering using Apache Superset to generate dashboards. Both for gitbase and gitbase-spark-connector. So we need to work on compatibility for it. If there is any major functionality that cannot be done in short-term, input on what it would take to support it will also be needed.

In a preliminary review I found this incompatibilities:

Missing DATE function (https://github.com/src-d/go-mysql-server/pull/654)
Missing DATE_SUB and DATE_ADD functions
Incompaitbile SHOW CREATE TABLE result with SQLAlchemy (https://github.com/src-d/go-mysql-server/pull/656)
using fqns in SHOW CREATE TABLE errors (https://github.com/src-d/gitbase/issues/787)

DATE_SUB and DATE_ADD https://github.com/src-d/go-mysql-server/issues/663

I just added support for HAVING, since some charts require this, without having a easy alternative on the GUI to generate other kind of query.
What about https://github.com/src-d/go-mysql-server/pull/599? /cc @smola @ajnavarro

@mcuadros https://github.com/src-d/go-mysql-server/pull/599 needs more work to make rules work as expected in some cases.

Let's split this kind of EMPIC into multiple small tasks. So far, it looks that everything is either done, blocked by 3rdparties or requires much more effort.

Well, this issue is still pending, what I wrote is my initial research but a more depth list should be created after going through all the options of Superset

HAVING support at https://github.com/src-d/go-mysql-server/pull/689

Closing as completed; https://github.com/src-d/gitbase/releases/tag/v0.20.0-beta4 is compatible with superset.


Check Guidelines Update

Please check if the PR https://github.com/zalando/restful-api-guidelines/pull/405 introduces changes which are relevant to the Zally project.

Change is not relevant to Zally.


Improve mb.status

Include number of servers and other useful info like number of commands and size of db and whether snekbox is up and stuff

Can we do memory usage as well?


Introduce pex3 venv inspect.

This command allows inspecting venvs created by Pex as well as those
created by other tools.
Work towards #1752 and #2110

N.B.: pex3 venv create will follow this, but this establishes the tool sub-command namespace and is useful in its own right to, for example, determine the proper site-packages directory for a pex venv (there can be more than one) if you want to copy out its contents.

For example, a Pants venv created by the ./pants script:
$ python -mpex.cli venv inspect ~/.cache/pants/setup/bootstrap-Linux-x86_64/2.14.1_py39/ -i2

{
  "venv_dir": "/home/jsirois/.cache/pants/setup/bootstrap-Linux-x86_64/2.14.1_py39",
  "provenance": {
    "created_by": "virtualenv 20.4.7",
    "is_pex": false,
    "pex_version": null
  },
  "include_system_site_packages": false,
  "interpreter": {
    "binary": "/home/jsirois/.cache/pants/setup/bootstrap-Linux-x86_64/2.14.1_py39/bin/python",
    "base_binary": "/usr/bin/python3.9",
    "version": "3.9.16",
    "sys_path": [
      "/usr/lib/python39.zip",
      "/usr/lib/python3.9",
      "/usr/lib/python3.9/lib-dynload",
      "/home/jsirois/.cache/pants/setup/bootstrap-Linux-x86_64/2.14.1_py39/lib/python3.9/site-packages"
    ]
  },
  "script_dir": "/home/jsirois/.cache/pants/setup/bootstrap-Linux-x86_64/2.14.1_py39/bin",
  "scripts": [
    "normalizer",
    "pants",
    "pex",
    "pex-tools",
    "pex3",
    "pip",
    "pip3",
    "pip3.9",
    "wheel",
    "wheel-3.9",
    "wheel3",
    "wheel3.9"
  ],
  "site_packages": "/home/jsirois/.cache/pants/setup/bootstrap-Linux-x86_64/pants.2KoUMU/install/lib/python3.9/site-packages",
  "distributions": [
    "PyYAML==6.0",
    "ansicolors==1.1.8",
    "certifi==2022.12.7",
    "charset-normalizer==3.0.1",
    "fasteners==0.16.3",
    "humbug==0.2.7",
    "idna==3.4",
    "ijson==3.1.4",
    "importlib-resources==5.0.7",
    "packaging==21.3",
    "pantsbuild.pants==2.14.1",
    "pex==2.1.108",
    "pip==23.0",
    "psutil==5.9.0",
    "pyparsing==3.0.9",
    "python-lsp-jsonrpc==1.0.0",
    "requests==2.28.2",
    "setproctitle==1.2.2",
    "setuptools==63.4.3",
    "six==1.16.0",
    "toml==0.10.2",
    "types-PyYAML==6.0.3",
    "types-setuptools==62.6.1",
    "types-toml==0.10.8",
    "typing_extensions==4.3.0",
    "ujson==5.7.0",
    "urllib3==1.26.14",
    "wheel==0.36.2"
  ]
}

It might be a good idea to write an integration test against a more modern virtualenv or other widely used virtualenv creating tool (pipenv?).

Good, idea - done for virtualenv. Pipenv is not even official any more if you follow PyPA discourse ... it's Hatch. I'll just stick to virtualenv if you agree since that invented all this and is used by several tools behind the scenes. I may rue the choice though, I'm sure there are PEP-405 violators out there and I'll find that out the hard way :/.


Base submission error alerts off of metrics

Submitting a transaction can error for all sorts of reasons. Because so many different things can go wrong on-call can receive many alerts which ultimately don't point to a systemic problem but to a bunch of one-off errors.
Instead of always alerting for certain errors I would like to try keeping metrics for every type of submission strategy and alert when one consistently reports errors.
Individual submission success/failure metrics will be maintained for eden, flashbots and each individual custom node the driver is using.
This way it should be mostly obvious from the alert which submission strategy is having an actual problem.
It's mostly obvious because there is no nice way of labeling the individual custom nodes so they will be named based on the index they have in the --transaction-submission-nodes argument.
Another notable change is that we are currently try to be smart about errors we receive from the rpc endpoint. This means we try to read the error message and turn the generic error into a custom error. Every error which we didn't anticipate will cause an alert (instead of being logged as a warning). This can lead to too many alerts for benign errors and needs to be kept in sync with the specific nodes rpc client implementation (which error messages does it produce?).
Also we don't really make decisions based on that custom error type. We just log more generic error messages for each known error. Combined with the transition to metrics based errors I don't really see the need to keep this logic and instead use tracing::warn() for every (uninterpreted) web3::Error. If the metrics cause an alert it's still easy enough to use kibana to search for all the errors related to that submission strategy.
Test Plan
TODO

Codecov Report

Merging #1738 (22d83c4) into main (527a711) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1738      +/-   ##
==========================================
+ Coverage   66.02%   66.04%   +0.01%
==========================================
  Files         179      179
  Lines       36812    36801      -11
==========================================
  Hits        24305    24305
+ Misses      12507    12496      -11


3 Sum Problem

Description
Through this issue I will try to mention how to solve 3 sum problem for a given array of integers.
Domain
DSA
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

/assign

@puneetkaur2929 can you please describe the issue(the 3 sum prob question)more descriptively?

@puneetkaur2929 I think you have opened this issue for yourself,if you want i can unassign myself from this else i can work on this if you allow?

/assign


refactor: update blas/ext/base/dnannsumkbn2 to follow current project…

Description

What is the purpose of this pull request?

refactor: update blas/ext/base/dnannsumkbn2 to follow current project conventions
Related Issues

Does this pull request have any related issues?

This pull request:

resolves #1480

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

@kgryte I am getting errors in running tests and constantly getting error:
Running test<EMAIL_ADDRESS>29 tests complete (38ms)
Running test<EMAIL_ADDRESS>/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/lib/dnannsumkbn2.native.js:48
addon( N, x, strideX, out, strideOut );
^
RangeError: invalid argument. Fourth argument has insufficient elements based on the associated stride and the number of indexed elements.
at dnannsumkbn2 (/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/lib/dnannsumkbn2.native.js:48:2)
at Test.test (/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/test/test.dnannsumkbn2.native.js:59:6)
at Test.bound [as _cb] (/home/runner/work/stdlib/stdlib/node_modules/tape/lib/test.js:58:32)
at Test.run (/home/runner/work/stdlib/stdlib/node_modules/tape/lib/test.js:77:10)
at Test.bound [as run] (/home/runner/work/stdlib/stdlib/node_modules/tape/lib/test.js:58:32)
at Immediate.next (/home/runner/work/stdlib/stdlib/node_modules/tape/lib/results.js:70:15)
at processImmediate (node:internal/timers:466:21)
can u help me with this?

@kgryte @Planeshifter  I am getting errors in running tests and constantly getting error:
Running test<EMAIL_ADDRESS>29 tests complete (38ms)
Running test<EMAIL_ADDRESS>/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/lib/dnannsumkbn2.native.js:48
addon( N, x, strideX, out, strideOut );
^
RangeError: invalid argument. Fourth argument has insufficient elements based on the associated stride and the number of indexed elements.
at dnannsumkbn2 (/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/lib/dnannsumkbn2.native.js:48:2)
at Test.test (/home/runner/work/stdlib/stdlib/lib/node_modules/@stdlib/blas/ext/base/dnannsumkbn2/test/test.dnannsumkbn2.native.js:59:6)
at Test.bound [as _cb] (/home/runner/work/stdlib/stdlib/node_modules/tape/lib/test.js:58:32)
at Test.run (/home/runner/work/stdlib/stdlib/node_modules/tape/lib/test.js:77:10)
at Test.bound [as run] (/home/runner/work/stdlib/stdlib/node_modules/tape/lib/test.js:58:32)
at Immediate.next (/home/runner/work/stdlib/stdlib/node_modules/tape/lib/results.js:70:15)
at processImmediate (node:internal/timers:466:21)
can u help me with this?

@kgryte @Planeshifter I'm delighted to inform you that I've successfully resolved the error we encountered earlier. I want to extend my heartfelt gratitude to @kgryte for your invaluable guidance throughout this process.
I'm now ready to proceed with the review. Thank you once again for your assistance.

@kgryte @Planeshifter I'm delighted to inform you that I've successfully resolved the error we encountered earlier. I want to extend my heartfelt gratitude to @kgryte for your invaluable guidance throughout this process.
I'm now ready to proceed with the review. Thank you once again for your assistance.


Latest version build fails

For the included dockerfile on ARM:
12.28    Compiling rnostr v0.4.6 (/usr/local/src/rnostr)                                                                                [10/1723]
12.79 error[E0599]: the method `value_parser` exists for reference `&&&&&&_AutoValueParser<Input>`, but its trait bounds were not satisfied
12.79     --> src/lib.rs:48:12
12.79      |
12.79 48   |     #[clap(value_parser, default_value = "-")]
12.79      |            ^^^^^^^^^^^^ method cannot be called on `&&&&&&_AutoValueParser<Input>` due to unsatisfied trait bounds
12.79      |
12.79     ::: /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.15/src/builder/value_parser.rs:2422:1
12.79      |
12.79 2422 | pub struct _AutoValueParser<T>(std::marker::PhantomData<T>);
12.79      | ------------------------------ doesn't satisfy `_: _ValueParserViaParse`
12.79      |
12.79     ::: /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/clio-0.3.5/src/input.rs:33:1
12.79      |
12.79 33   | pub struct Input {
12.79      | ---------------- doesn't satisfy 7 bounds
12.79      |
12.79      = note: the following trait bounds were not satisfied:
12.79              `Input: ValueEnum`
12.79              which is required by `&&&&&_AutoValueParser<Input>: clap::builder::via_prelude::_ValueParserViaValueEnum`
12.79              `Input: ValueParserFactory`
12.79              which is required by `&&&&&&_AutoValueParser<Input>: clap::builder::via_prelude::_ValueParserViaFactory`
12.79              `Input: From<OsString>`
12.79              which is required by `&&&&_AutoValueParser<Input>: clap::builder::via_prelude::_ValueParserViaFromOsString`
12.79              `Input: From<&'s std::ffi::OsStr>`
12.79              which is required by `&&&_AutoValueParser<Input>: clap::builder::via_prelude::_ValueParserViaFromOsStr`
12.79              `Input: From<std::string::String>`
12.79              which is required by `&&_AutoValueParser<Input>: clap::builder::via_prelude::_ValueParserViaFromString`
12.79              `Input: From<&'s str>`
12.79              which is required by `&_AutoValueParser<Input>: clap::builder::via_prelude::_ValueParserViaFromStr`
12.79              `Input: FromStr`
12.79              which is required by `_AutoValueParser<Input>: clap::builder::via_prelude::_ValueParserViaParse`
12.79      = note: this error originates in the macro `clap::value_parser` (in Nightly builds, run with -Z macro-backtrace for more info)
12.79
12.86 error[E0599]: the method `value_parser` exists for reference `&&&&&&_AutoValueParser<Output>`, but its trait bounds were not satisfied
12.86     --> src/lib.rs:68:12
12.86      |
12.86 68   |     #[clap(value_parser, default_value = "-")]
12.86      |            ^^^^^^^^^^^^ method cannot be called on `&&&&&&_AutoValueParser<Output>` due to unsatisfied trait bounds
12.86      |
12.86     ::: /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/clap_builder-4.5.15/src/builder/value_parser.rs:2422:1
12.86      |
12.86 2422 | pub struct _AutoValueParser<T>(std::marker::PhantomData<T>);
12.86      | ------------------------------ doesn't satisfy `_: _ValueParserViaParse`
12.86      |
12.86     ::: /usr/local/cargo/registry/src/index.crates.io-6f17d22bba15001f/clio-0.3.5/src/output.rs:63:1
12.86      |
12.86 63   | pub struct Output {
12.86      | ----------------- doesn't satisfy 7 bounds
12.86      |
12.86      = note: the following trait bounds were not satisfied:
12.86              `clio::Output: ValueEnum`
12.86              which is required by `&&&&&_AutoValueParser<clio::Output>: clap::builder::via_prelude::_ValueParserViaValueEnum`
12.86              `clio::Output: ValueParserFactory`
12.86              which is required by `&&&&&&_AutoValueParser<clio::Output>: clap::builder::via_prelude::_ValueParserViaFactory`
12.86              `clio::Output: From<OsString>`
12.86              which is required by `&&&&_AutoValueParser<clio::Output>: clap::builder::via_prelude::_ValueParserViaFromOsString`
12.86              `clio::Output: From<&'s std::ffi::OsStr>`
12.86              which is required by `&&&_AutoValueParser<clio::Output>: clap::builder::via_prelude::_ValueParserViaFromOsStr`
12.86              `clio::Output: From<std::string::String>`
12.86              which is required by `&&_AutoValueParser<clio::Output>: clap::builder::via_prelude::_ValueParserViaFromString`
12.86              `clio::Output: From<&'s str>`
12.86              which is required by `&_AutoValueParser<clio::Output>: clap::builder::via_prelude::_ValueParserViaFromStr`
12.86              `clio::Output: FromStr`
12.86              which is required by `_AutoValueParser<clio::Output>: clap::builder::via_prelude::_ValueParserViaParse`
12.86      = note: this error originates in the macro `clap::value_parser` (in Nightly builds, run with -Z macro-backtrace for more info)
12.86
13.21 For more information about this error, try `rustc --explain E0599`.
13.22 error: could not compile `rnostr` (lib) due to 4 previous errors
13.22 warning: build failed, waiting for other jobs to finish...
------
failed to solve: process "/bin/sh -c cargo build --release --target-dir \"${BUILDER_DIR}\" --bins" did not complete successfully: exit code: 101

Updating the baseimage to 1.75-slim-bullseye is a good step but does not fix it either

@xeruf The build action is success https://github.com/rnostr/rnostr/actions/runs/10607923456/job/29401168337 . I think this error can be prevented by clean cache.

there is no cache when building a docker image?

maybe the fact that the CI build action caches something is the problem?

Build also fails locally on arch

@xeruf try cargo clean before cargo build

Build the docker image will throw error package clap_builder v4.5.15 cannot be built because it requires rustc 1.74 or newer, while the currently active rustc version is 1.72.0. I update the version to 1.80 and push the change to main branch. I've seen the error like &&&&&&_AutoValueParser<Input> and run cargo clean to fixed it.

Local build works after cargo clean, still no luck on ARM in docker

I build success on platform linux/arm64. try docker build --no-cache

ah thanks, recloning worked - even cargo clean did not help
I am a bit confused why this is needed for docker, I thought the whole idea is that builds are isolated and reproducible, why does it cache something that is incompatible?

Because we use cargo chef in docker for fast rebuild.


feat (account): implement repository test

Check out this project: https://github.com/7AXEL/Web2APK


Military Overlay technical review

Items from 11/10/15 Tech Review of Military Overlay
Editing templates

[x] Editing templates only need to go down to second/EntityType level for some layers (ex. Air - but keep for layers where this was thought out like Units)

Tasks:

[x] (1) Create/Sketching Overlay Task Item (main/primary usage) separate from (2)Configuring/customizing Organizational Templates Tasks
[x] Expand on "Configure Feature Templates" Task
[x] Export to Png task as part of Create Overlay/Sketching

Map Properties

[x] Coordinate System - MGRS
[x] Projection - WGS84

Schema Issues

[x] Add Editor Tracking
[x] Add status field to Units, Civilian, Activities (domain is "statuses")

Table of Contents:

[x] Remove "Equipment" from "Air Equipment"
[x] Control Measures and METOC Layers on bottom (last in display order)

Other

[x] Remove mil2525d.stylx from project styles (so these don't get accidentally packaged with)

Thanks @csmoore !

@Dbarnes1 - in order to test the status field for Pro 1.2 I (and you) will need an updated GDB with the status field added to Units, Civilian, Activities

@joebayles After making these changes I passed the project package off to you, they're ready for review now, box link: https://esri.box.com/s/058bmd5fun9f3n3ls7ufyb5h5kckl7wv

Just a reminder - don't forget to assign the issue to be verified - versus directly closing.
Usually assign to the originator, but sounds like you have an alternate to verify & that's fine, just so long as it gets verified.

Verified.
@Dbarnes1


Add init script to set the env vars

render-templates.sh and script.sh expect certain lab specific env variables to be set. Create a script that sets and documents them.

Here are the environment values I need to set to correctly deploy it in my env
Management of cloudfare (used to create certificates)
CLOUDFLARE_API_TOKEN
DNS Domain (suffix for ironic services)
DNS_DOMAIN
DNSMASQ informations
DNSMASQ_DEFAULT_ROUTER # default GW to reach outside
DNSMASQ_DNS_SERVER_ADDRESS # actualy set on IP where the service is deployed
Media URL for Ingress definition
MEDIA_SERVER_URL
IRONIC CACHE URL
IRONIC_CACHE_IMAGE_SERVER_BASE_URL
IRONIC BOOT URL
IRONIC_BOOT_IMAGE_SERVER_BASE_URL
DHCP Range for setup auto deploy baremetal, this condition the number of host could be install at the same time
IRONIC_DHCP_RANGE
IRONIC API URL
IRONIC_API_BASE_ENDPOINT
IRONIC INSPECTO URL
IRONIC_INSPECTOR_BASE_ENDPOINT
ETHERNET DEVICE OF IRONIC PROVISIONNING
IRONIC_PROVISIONING_INTERFACE
IP ADDRESS OF IRONIC PROVISIONNING
IRONIC_PROVISIONING_IP
DEPLOY IN SSL OR NOT
IRONIC_INGRESS_SETUP_TLS
Define Rancher version
RANCHER_VERSION
Define Rancher url based on DNS Domain for case (currently)
RANCHER_HOSTNAME
SSH public key to connect to deployed elemental servers
SSH_AUTHORIZED_KEY
ILO Management
ILO_USERNAME
ILO_PASSWORD


Async version of RSetMultimap/RListMultimap

Is there a plan to implement RSetMultimapAsync and RListMultimapAsync?

Just found out there are no sync/async versions; they are in one set of classes.

It's already has async interface RMultimapAsync


Dev

-Added options for running different environments
-Removed all the hard-coded features and replaced it with dynamic code
-Added support for k8s API to provide remote cluster access feature and remove fission-cli from requirements
-Added support for all the triggers provided by fission
-Added HTTP request calls to call deployed functions

All the comment where I didnt reply back are fixed in the code. Also, thanks for the review much appreciated.!


Glitches when connecting with /bpp:32

When connecting with FreeRDP 1.1 in 32-bit or 24-bit mode to Windows XP or Server 2003, there are frequent glitches in menus, sometimes also menu bars & buttons:

FreeRDP 1.2.0-beta1 (git 1.1.0-beta+2013071101-1821-g8abc0)
16-bit mode works fine
Both rdesktop and mstsc.exe work fine

Update – it seems that +glyph-cache helps, and +glyph-cache /gdi:sw makes Server 2003 work well even with 32bpp (except for the occassional misplaced text).

@grawity good to hear, just to clarify: you're using the old version mentioned or something more recent?

I test both the distro package and current Git builds. With Arch's 1.2.0_20160606 package, just /gdi:sw is enough (it seems glyph-cache is on by default). With today's 2.0.0-dev (git 215fbe8446e9) build, I need to enable both manually. (I've seen the misplaced text already reported in other issues, so I don't mind that.)


Updating testing scenarios and adding tests for TemplateManager (and more)

This PR touches on a few things that came up during https://github.com/BioPhoton/rx-angular/pull/222. Each of those things (and some additional ones) will be explained in a separate paragraph with proposals on how to resolve those.
TODO for me
There was a TODO comment for me to add a comment on an implementation detail explaining why there is a condition on displaying "suspense" or "next" in the LetDirective. I added those and also refactored this part of a code for further clarity.
Template-binding example is broken
Latest commits broke template-binding example in template-demo app. Considering this as a primary example with one of the most basic and simplest use-cases for the LetDirective, this got my attention. Solution was rather simple: inside TemplateManager's implementation there were two lines of code:
viewContainerRef.detach(0);
...
viewContainerRef.insert(viewCache.get(name), 0);

For the functionality to work again, the argument of index of the view to be detached/inserted needed to be dropped:
viewContainerRef.detach();
...
viewContainerRef.insert(viewCache.get(name));

This works because the view we want to detach or insert is the latest one, and ViewContainerRef#detach and ViewContainerRef#insert work on the latest view if no index value is provided. The index of the latest view is not always 0 and that seems to be an implementation detail on the Angular core's side.
I added the above change and also added some test for the TemplateManager for that.
jsdoc for TemplateManager
I moved jsdoc for TemplateManager inside an interface. Please notice that earlier, when jsdoc was attached to the object returned by createTemplateManager factory function, the jsdoc was not displayed in the IDE. Moving it to the interface solved the issue.
Naming convention for TypeScript generics
I made a bold move of renaming generics for TemplateManager- from ViewContext to C and Names to N. I believe this is THE convention of naming generics in TypeScript that the core team had in mind. Please notice, that ViewContext as a generic is easily mistaken for a concrete type when used within the implementation code. I think we should name them distinctively, to know instantly that we are dealing with generics, not a concrete type (given, they also have their limitations in TypeScript).
Animation example within experiments
I played with the demo for animations and it seems to be working after my change. Please try it out and let me know if there's something I missed.
Import for coalescing-manager.ts
I noticed that during compilation there are warnings about circular dependency within a project. I fixed it by updating import statement for the coalescing manager.
Testing module for TemplateManager
I updated a testing module under test for template-manager_creator.spec.ts . With my changes we are able to determine which template is currently inserted to the view container based on its text content. Also, with it we are able to test TemplateManager#updateViewContext method, which wasn't possible before.
Exposing internals of TemplateManager implementation
I strongly believe that exposing internals of TemplateManager implementation, like template and view cache or a ViewContainerRef instance only for testing purposes should be avoided as much as possible. Just like the comment I found inside of createTemplateManager factory said - we should rethink that. I removed exposed properties and proposed some changes to existing tests for TemplateManager. I extended testing module to be more flexible (like I described in the previous paragraph) and tried to refactor tests so that they would match the experience and use-cases of the TemplateManager's user.

jsdoc for TemplateManager

Nice!

I made a bold move of renaming generics for TemplateManager

Pretty good argument!

circular dependency

Nice!

Exposing internals

I love that we want to get dir of them!!!

From my side all the stuff I described is implemented and ready. Unless of course there are no further suggestions for changes 😉


WebCoder49/Code-input

Active

I cannot see much change here, except that you are updating the (slightly incomplete) transfer-hover branch with the latest version of the code. Would you like me to implement the transfer-hover plugin (which allows pieces of syntax-highlighted code to trigger something when hovered over) fully?


sbe-tool nuget package is DOA

When using the sbe-tool from the latest sbe-tool nuget package: 0.1.12-1-beta-1, code generation immediately fails
Exception in thread "main" java.lang.NoClassDefFoundError: org/agrona/MutableDirectBuffer
        at uk.co.real_logic.sbe.SbeTool.<clinit>(SbeTool.java:82)
Caused by: java.lang.ClassNotFoundException: org.agrona.MutableDirectBuffer
        at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:583)
        at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:521)
        ... 1 more

It seems that the nuget package contains the WRONG jar, it should have contained the jar from the: sbe-all/build/libs/ folder which contains all the required java deps, but instead it uses the jar from the sbe-tool/build/libs/ folder which is partial as far as running the complete sbe-tool is...
Would it be OK for me submit a PR that fixes this and for someone who has access to the nuget account (I presume @billsegall) to publish an updated nuget package?
As far as I can tell, this thing is dead out of water from nuget....

It looks like their are a number of issues worth addressing with the C# implementation. It would be good for those interested to jointly agree on how best to improve it and I'll happily support that.

I wouldn't mind taking a few concentrated days to try and bring it more up to speed..., looks to me like we need:

CI support for the C# parts
Automatic packaging / myget feed per build
Running unit tests on the build server (e.g. building + running everything on Linux as part of the current travis build)

Would you like me to first try and get the C# parts of the project into a working state?

That would be great. Then maybe @billsegall, @JPWatson, @ZachBray and whoever else is interested can feedback and agree changes.

OK, the only annoyance is that I've already done some build related touch-ups as part of https://github.com/real-logic/simple-binary-encoding/pull/695
It would be easier to me if someone could review that and merge that so I could branch off of that to start the CI related stuff, but if that's not possible in the short-term, let me know and I'll redo that work in a new branch dedicated to the CI status

It would seem I broke this with https://github.com/real-logic/simple-binary-encoding/commit/8b9bda8870321c14e04490983b8b09b4348fa6ad
Can anyone provide context on why the '-all' jar stopped building. I had wrongly assumed it had just changed name.
I'm on the road holidaying with limited internet/facilities for the next three weeks but will try to chime in when I can.

@billsegall I think I have something working in https://github.com/real-logic/simple-binary-encoding/pull/699
There's a ton of small fixes there, and once travis does its thing, I think you can push a new nuget from there...

travis now seems to build this properly... (https://github.com/real-logic/simple-binary-encoding/pull/699)
Once merged I think we can release a new nuget package after someone can test it except for me :)

What is the status of the nuget package? If we do a new release in a few days what is required?

I'll be back at my desk on Monday and will publish a new version. Long term
we need to make this possible at a project level.
Bill
On Thu., 1 Aug. 2019, 7:36 pm Martin Thompson<EMAIL_ADDRESS>wrote:

What is the status of the nuget package? If we do a new release in a few
days what is required?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/real-logic/simple-binary-encoding/issues/698?email_source=notifications&email_token=ABJDLJDLSBGGZ7DZTFCLTOLQCMNJJA5CNFSM4IEPZA6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3LLA7Q#issuecomment-517386366,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJDLJGIC5QDVQRR3725LJTQCMNJJANCNFSM4IEPZA6A
.

Thanks @billsegall. Should we hold off on a release until after Monday?

Go for it now. If there is a problem we can fix it after and it's nice to
be close to an official release.
Bill.
On Fri., 2 Aug. 2019, 9:33 am Martin Thompson<EMAIL_ADDRESS>wrote:

Thanks @billsegall https://github.com/billsegall. Should we hold off on
a release until after Monday?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/real-logic/simple-binary-encoding/issues/698?email_source=notifications&email_token=ABJDLJAJ55BBXK2CQAYDKLDQCPPMRA5CNFSM4IEPZA6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3M5FWA#issuecomment-517591768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJDLJC2BXNKZSUL5XEY42LQCPPMRANCNFSM4IEPZA6A
.

Great!
All I can say is that I've been using it from an internal feed, and as far as I can tell it's working well (codegen + generated code).

Heh. Won't be Monday, I'm having one of those air travel nightmares on a
super long haul flight.
Bill
On Fri., 2 Aug. 2019, 9:39 am Bill Segall<EMAIL_ADDRESS>wrote:

Go for it now. If there is a problem we can fix it after and it's nice to
be close to an official release.
Bill.
On Fri., 2 Aug. 2019, 9:33 am Martin Thompson<EMAIL_ADDRESS>wrote:

Thanks @billsegall https://github.com/billsegall. Should we hold off
on a release until after Monday?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/real-logic/simple-binary-encoding/issues/698?email_source=notifications&email_token=ABJDLJAJ55BBXK2CQAYDKLDQCPPMRA5CNFSM4IEPZA6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3M5FWA#issuecomment-517591768,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJDLJC2BXNKZSUL5XEY42LQCPPMRANCNFSM4IEPZA6A
.

Sorry to hear that. Travel can be a pain at times. Hopefully gets better.

Other than the official package at nuget.org, I can also setup some sort of a ci nuget feed on myget or something of that source, if that is of any interest...
I would make consuming ci packages a real possibility, although that might be over the top.

The sbe-dll package is uploaded and is being verified. I've created a PR
with the minor changes for the CSharp release.
I notice that package name has changed from sbe-tool to sbe-dll and I'm
wondering how this will affect users of the nuget package and automatic
updates?
I'm in two minds about myget packages. I think it would be very useful to
be able to have the travid process build, upload, and test against the
uploaded package but I worry people might use it and think it stable.
Bill.
On Sun, 4 Aug 2019 at 21:12, damageboy<EMAIL_ADDRESS>wrote:

Other than the official package at nuget.org, I can also setup some sort
of a ci nuget feed on myget or something of that source, if that is of any
interest...
I would make consuming ci packages a real possibility, although that might
be over the top.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/real-logic/simple-binary-encoding/issues/698?email_source=notifications&email_token=ABJDLJEA4PFLXJFIGYSJUHDQC22RFA5CNFSM4IEPZA6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3P7PJI#issuecomment-517994405,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJDLJCBSDDWVC5LKHBUF3LQC22RFANCNFSM4IEPZA6A
.

@billsegall I was pretty sure I did not change anything in the nuget spec, and I really tried to meticulously copy the nuspec to the appropriate .csproj properties during the "porting" process.
Let me double check and get back to you.

I think it's still open and will have a dig next week. I think I've decided
that referring the name is a good idea.
Bill.
On Wed., 14 Aug. 2019, 11:18 pm Martin Thompson<EMAIL_ADDRESS>wrote:

Is this resolved yet?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/real-logic/simple-binary-encoding/issues/698?email_source=notifications&email_token=ABJDLJD5XWEDMH4NYJUGX4TQEQAZ3A5CNFSM4IEPZA6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4IYKKI#issuecomment-521241897,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJDLJHACGMABSE6SK2GUXLQEQAZ3ANCNFSM4IEPZA6A
.

Martin,
I found some time, and with the now merged #707
https://github.com/real-logic/simple-binary-encoding/pull/707 and newly
published sbe-tool 1.13.1 https://www.nuget.org/packages/sbe-tool/, I
think this can now be closed.
On Thu, 15 Aug 2019 at 02:00, Bill Segall<EMAIL_ADDRESS>wrote:

I think it's still open and will have a dig next week. I think I've
decided that referring the name is a good idea.
Bill.
On Wed., 14 Aug. 2019, 11:18 pm Martin Thompson<EMAIL_ADDRESS>wrote:

Is this resolved yet?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/real-logic/simple-binary-encoding/issues/698?email_source=notifications&email_token=ABJDLJD5XWEDMH4NYJUGX4TQEQAZ3A5CNFSM4IEPZA6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4IYKKI#issuecomment-521241897,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABJDLJHACGMABSE6SK2GUXLQEQAZ3ANCNFSM4IEPZA6A
.


TypeError: argument of type 'StrictUndefined' is not iterable when callin value in dict with missing key

Expected Behavior
raising of UndefinedError with text message 'dict object' has no attribute 'missing_key'
Actual Behavior
TypeError with text argument of type 'StrictUndefined' is not iterable is raised
Template Code
{% if 'value' in data['missing_key'] %}{% endif %}
{{ 'value' in data['missing_key'] }}

MCVE
#!/usr/bin/env python3
import jinja2

def main():
    text = '''{{ 'value' in data['missing_key'] }}'''  # both fails
    text = '''{% if 'value' in data['missing_key'] %}{% endif %}'''

    environment = jinja2.Environment(
        undefined=jinja2.StrictUndefined,
    )
    template = environment.from_string(text)
    template.render(data={})

if __name__ == '__main__':
    main()

Full Traceback
Traceback (most recent call last):
  File "./mcve.py", line 16, in <module>
    main()
  File "./mcve.py", line 12, in main
    template.render(data={})
  File "/usr/local/lib/python3.6/dist-packages/jinja2/environment.py", line 1090, in render
    self.environment.handle_exception()
  File "/usr/local/lib/python3.6/dist-packages/jinja2/environment.py", line 832, in handle_exception
    reraise(*rewrite_traceback_stack(source=source))
  File "/usr/local/lib/python3.6/dist-packages/jinja2/_compat.py", line 28, in reraise
    raise value.with_traceback(tb)
  File "<template>", line 1, in top-level template code
TypeError: argument of type 'StrictUndefined' is not iterable

Your Environment

Python version: 3.6.8 (default, Jan 14 2019, 11:02:34) [GCC 8.0.1 20180414 (experimental) [trunk revision 259383]]
Jinja version: 2.11.2
OS version: Ubuntu 18.04.2 LTS

Here i try to check some variants of errors for different templates: https://github.com/HumanUser/jinja2_strict_undefined
Another thing i noticed that DebugUndefined is not really debug in some cases.

The problem is that both Undefined and StrictUndefined do not define the __contains__ method. As a result, it falls back to __iter__ which throws an error and forces Python to raise the exception which you are seeing.

This bug seems to have been fixed by pallets/jinja#1204

This bug seems to have been fixed by pallets/jinja#1204

Thanks!


A request

Hello Author.
Thank you for your work, do you have a python version?

hi author, do you have article about your work?


DOC: Fix Wikipedia bootstrap link

I noticed that the link in the documentation to the Wikipedia page on statistical bootstrapping was not pointing to the right place. Seems like this is a general issue with reStructuredText when the hyperlink contains parentheses. Citation [12] in the toolchain roadmap would have the same problem, but the parentheses were switched for their unicode equivalents (%28 and %29). I've simply applied that same fix to the hyperlink for statistical bootstrapping and this has fixed the issue when I build the documentation locally.
This is my first PR to scipy so I hope I've got the PR and commit format done correctly :crossed_fingers:. Thank you to the entire development team for creating this amazing open-source software.

Thanks @elbeejay! Commit message is perfect.

This is my first PR to scipy so I hope I've got the PR and commit format done correctly

First of many, I hope!


[ADnD 2E Revised] Bugfix: Magic resistance calculations

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
All changes described on the forum: https://app.roll20.net/forum/permalink/11150510/

Character Sheet Info Roll20 Internal Use only.


Error if stripped function is the only line in if condition.

Using the configuration
{test: /\.js$/, loader: 'strip-loader?strip[]=console.log'}

For the below code snippet in the file, leads to error.
if(condition)
    console.log('...');
else{
   /* Some statements */
}

Error
Unexpected token else
You may need an appropriate loader to handle this file type.

This is why always using braces around your if clauses is good coding practice.

I caught this error when using some library code. I ended up using UglifyJsPlugin to accomplish the same.
new webpack.optimize.UglifyJsPlugin({
  compress: {
    drop_console: true
  }
})

This works with the above code.

This sucks but the only real way to do this is to convert code to an AST, strip function calls, then back to code. Which is what I was attempting to do with recast, but I just haven't had time for it.
If anyone else would like to help with that, I'd welcome it.

I've actually written code to do this as a Grunt task in my DoppioJVM project, but it doesn't work for functions invoked on an object (e.g. works for log(), but not console.log()). It's MIT licensed, so feel free to use it as a starting point.


Impro 621 improve snow extrapolate

Addresses #GitHubissuenum
Description
Testing:

[ ] Ran tests and they passed OK
[ ] Added new tests for the new feature(s)

CLA

[ ] If a new developer, signed up to CLA

Still needs work on unit tests.


How to use isValidNumber function?

<PhoneInput
ref={(ref) => this.phone = ref}
value={this.state.phone_number}
onChangePhoneNumber={(number) => this.setState({phone_number: number})}
/>
in any function in that component you can write this.phone.isValidNumber()

@ayeshasaeed94 is correct - isValidNumber can be accessed using a PhoneInput ref. For an example of how to interact with PhoneInput via a ref, see the Custom Picker section of the readme.

but how we use in Hooks?


Remove scaling parameter from configs

Fixes #29
The diffusive scaling does not work properly for multilevel.
Hence users do not really have a choice, and we should remove
the option to provide a scaling strategy, as it otherwise is
confusing.

removing only the scaling parameter from configs will not be enough.
There are several if diffusive scaling like statements in the config files that will need to be corrected.

@jainka I know. This is merely the start of this work to address #29 and discuss the changes. Actually, as it stands this deactivates passive-scalar, for example, so we need to also do more code changes.

@KannanMasilamani Please have a look at the changes so far. I don't think the stuff in doc_pages/tutorials is still used, but the rest should show up in the documentation. After this the request get's larger with all the various examples needing the scaling parameter removed.

it seems that this will require a re-discussion. This appears like a lot of work and I am not sure if it is necessary. Perhaps the easier would be to reinstate diffusive scaling and send an error to the user if diffusive is used in case of multilevels?
Something like if maxlevel-minlevel>0 then write(logUnit(1),*)'ERROR: Choose scaling = "acoustic" for
The way it stands currently is that there are acoustic and diffusive differentiations at many places, see for example mus_scheme_header_module. In fact in the scheme header module there is a good differentiation whereby the user is requested to use acoustic or diffusive depending on scheme kind.
Lets re-discuss this as to me reinstatement of diffusive sounds like a more reasonable and much easier option.

OK, thanks @jainka, @KannanMasilamani with the conclusion to keep reading the scaling parameter, this PR is now ready to be merged, I think. We will abort if there is a multi-level mesh with diffusive scaling requested, or if the scheme requires diffusive scaling and we are on the multi-level mesh (though acoustic scaling is requested).


Feat: Optionally disable pretty printing of model sql in render

Problem:
This has bugged me for awhile, but current way sqlmesh render works has a lot of issues if your SQL is wider than console length. It automatically wraps and insert new lines which breaks usage in scripts and even copy paste. Furthermore it pads the end of line all the way out to the terminal width. Which is also really annoying. Lastly, and I don't know if this is consistent -- I get truncated text sometimes too.
Solution:
Let people just dump the output as is to stdout, so we can use it in our vimscripts and so on. Or even so its cleaner to yank.
Add --no-pretty-print to render command which just uses simple print stmt

Padding seen above, worst though is the line break inserting and truncation -- but you get the point


Send diagnostic data to the FLL backend instead of the CCT backend

Added redirect response handling along with setting a User-Agent string for traffic differentiation on the server.

LGTM, though I would prefer it to be fully tested before merging. Maybe a dummy HTTP server can help with it?

Doh! I forgot--I have a test HTTP server just for this purpose! I'll add a unit test, that should help increase confidence.


[Bug]: Playground: Unread marker in Graph is reset when changing the panels layout

What happened?
https://user-images.githubusercontent.com/44893228/216334888-6b0bf26f-a9a1-4ff6-a49e-8401641b4b9e.mp4
Issue Priority
Priority: 3 (minor)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[X] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue

Not reproducible with master.


Fix Location Input Process

Tasks

[x] Null Error Handling: if field is null:

Scroll up to the field.
Highlight the field red.
Toasted Error

[x] Ensure that the following fields are required, and apply Null Error Handling:

[x] Name
[x] Phone1
[x] Address
[x] [City] (When "Select on Map or "Address Problem" is selected)
[x] [County] (When "Select on Map or "Address Problem" is selected)
[x] [State] (When "Select on Map or "Address Problem" is selected)
[x] [Postal Code] (When "Select on Map or "Address Problem" is selected)
[x] At least one Work Type must be selected. Highlight the "Work" section red if possible.

[x] If the address field is non-null, BUT has not been geocoded (i.e. no latitude or longitude have been created and the blue pin does not appear on the map), then highlight location field red and add an error alert next to the location field: caseForm.no_lat_lon_error
[x] If a user clicks "Select on Map," then:

[x] Expose the City, County, State, and Postal Code fields
[x] Automatically make the County field editable (it is read-only by default)
[x] Turn off geocoding.
[x] Apply Null Error Handling on the Address, City, County, State, and Postal Code fields.
[x] Make the "Select on Map" a toggled button that turns yellow

[x] If a user clicks "Select on Map" again, then toggle:

[x] Leave the blue pin on the map.
[x] Turn on the geocoder, and allow it to override the blue pin selection
[x] Re-hide City, County State, and Postal Code fields
[x] Make the "Select on Map" a toggled button that turns gray or white again.

[x] Add a Help button component to "Location:" Use caseForm.location_instructions
[x] Add a new checkbox, "Address Problem" (caseForm.address_problems) above "Flag as High Priority." Toggling this checkbox on will:

[x] Add or remove a "Wrong Location" flag to the case.
[x] Engage or disengage the "Select on Map" feature, as described above.

[x] Make sure it works on /phone and /cases.
[x] Make the "Flag as High Priority" and "Member of My Organization" checkboxes NOT RED. When a user has an error, they will check these boxes, thinking that they are red error messages. Instead, make them the same yellow as "+Add Note".
[x] Ensure all new strings or localizations are labeled with ~~.
[x] If the "Full Address" line is non-null AND has not been geocoded, AND a user saves, then activate "Select on Map." However, if "Full Address" line is NULL, then do NOT activate "Select on Map."

REMEMBER: The over-all goal of this issue is to make it easy for someone in the back woods of Kentucky to add an address to the system that Google does not recognize.

Bug:

Click "Select on Map."
Complete the address fields.
Complete other required fields.
Save.
Error: Cannot find location, even though there is a pin on the map.


Where to put the Java file?

When running the game, where to put the java class file in order unity to read it?

You should put the .aar file to Assets/Plugins/Android


[FEATURE]: Skyrim Souls RE Tween Menu No Mouse Cursor Fix

Feature Title
Skyrim Souls RE Tween Menu No Mouse Cursor Fix
Motivation
My motivation is that with every quickloot mod, be it RE or EE or this at the moment, when the quickloot menu is up it disables the mouse cursor in the tween menu when used with Skyrim Souls RE for un-paused menus, which is very annoying.
Feature Description
A fix for the mouse cursor being disabled in the tween menu while the quickloot menu is up, while used in conjunction with Skyrim Souls RE. I'm unsure if the issue is fixable on this mod's end but I hope it is.
Additional Context
I'm unsure if I should have maybe submitted this as a bug instead, but my gut tells me that because it deals with another mod that it should probably be considered a feature if it is considered.

Just installed Skyrim Souls RE to test this and opening the tween menu while the loot menu is open doesn't just hide the cursor, it completely softlocks the game for me since it makes it impossible to interact with or close the tween menu.
If I force close the loot menu while the tween menu is open, the issue doesn't occur. The main question is why it even shows in the first place. The loot menu is already supposed to be suppressed when any full screen menu is open. My best guess at this point is that Skyrim Souls RE messes with the menu flags so it is no longer recognized as having a higher priority than QuickLoot.

This is likely something that needs to be fixed on Skyrim Souls' side, what happens when you disable the Tween Menu in Souls' config?

It's not just the tween menu that's broken for me. Opening the container menu has the same issue. It looks to me like SSRE changes the menu flags and priority, which messes with QL's ability to detect a higher priority menu being open. I'm currently refactoring that exact part of the QL codebase, to I'll try to see whether I can fix this.

This is likely something that needs to be fixed on Skyrim Souls' side, what happens when you disable the Tween Menu in Souls' config?

It still happens with every menu set to false in Skyrim Souls' ini.

Okay, I think I know why this is happening. Usually, the loot menu would be disabled in these scenarios because RE::UI::GetSingleton()->GameIsPaused() returns true, but with Skyrim Souls installed the game doesn't get paused while in these menus.
We have a system in place that allows disabling the loot menu while certain blacklisted menus are open, which currently is only used for the dialogue menu since that is the only menu that normally doesn't pause the game. We could add all menus to that blacklist, but it doesn't exactly feel like a good solution. Especially since it wouldn't include menus added by other mods.
I've written some logic that manually walks the menu stack and checks if any menus are open that don't have the kAlwaysOpen flag set. Initial tests are promising, but we'll need to make sure this doesn't have unintended side effects.

Fixed by #40


Compare character

Hi, would like to check do compare character only work in diff side by side? How about diff in list?

PR is welcome 👍


Text box height control at the top of the box

I suggest adding the fuction to control the text box height at the top of the box, right now its only possible to resize the box if we use the click and drag icon at the bottom right corner.
It would also be nice to have some kind of rendering of the changes as we drag the icon.  Right now it doesnt show anything until you let go and the height takes a second to resize.

I can't actually figure out how to drag it. Nothing seem to react me dragging.

This should help understand it.
https://youtu.be/TNs2powukcM


[cluster-test] Correctly propagate error code in cti

Cti stopped propagating error code correctly after adding piping in 3a18eca727caf5cb2a20b915092d267b398af3a3

@bors-libra r+

:pushpin: Commit a0b08cc has been approved by sausagee

:hourglass: Testing commit a0b08ccb447a9557c795de71e66d170885179543 with merge c5d4c3a779299e9d1e330dcb648bc3c9c67ac529...

@bors-libra r=sausagee
Added one more fix

:pushpin: Commit 4292ca7 has been approved by sausagee

:hourglass: Testing commit 4292ca7fc123b4b935656d1b64d15409c3bdf66b with merge e3c0396e9a90d996d05c1a80b7ff7956592dbbaa...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: sausagee
Pushing e3c0396e9a90d996d05c1a80b7ff7956592dbbaa to master...


i18n for core nodes and nodesDir

It seems that there is no way to create i18n for core nodes help text
If nodesDir or userDir/nodes is defined, the locales inside the node package(directory) will not be loaded

For item 1. you can put i18n help text at nodes/core/<category>/locales/<locale>.  But, creating locales directory under <category> leads some message lookup to be failed.
I have proposed placing help file under nodes/core/locales/<locale>/<category> similar to message catalogs.

Item 1 I means those text inside tag <script type="text/x-red" data-help-name="nodeNameHere">.

Here is an example of Japanese info text for SWITCH node.
Japanese help file is placed at nodes/core/logic/locales/ja/10-switch.html.

Thanks a lot.
This really should be mentioned in wiki. Design: i18n only talks about the message.json about the core nodes, not the html with help text.

Hello,
Just want to pick this issue up! I have the same problem that point 2 in the first message.
Context: I have a linux embedded device with 2 rootfs partitions to manage the upgrade. Only one is active at a time. There is a data partition for permanent data.
My usage of node-red: I have set userDir to a sub-directory of the data partition (/data/node-red) + nodesDir to another sub-directory of the data partition to handle extra nodes installation managed with a system of plugins (/data/plugins).
Problem is the locales defined in the plugins are not loaded at all. If the plugin is moved to /usr/lib/node then this is properly working (but as describe above because the rootfs partitions can switch, this is not convenient for my usage, plugins need to remain in the data partition).
I may help on this topic (and propose a PR) but I'm not sure it's trivial and don't know really where to start exactly. @knolleary if you can point the right location (I ask you particularly because I don't want to propose something which doesn't fit the architecture or roadmap).
Regards
Joel

I have the same issue. Described it in this slack thread
https://node-red.slack.com/archives/C024NDNRUSJ/p1725237526953619

This was tidied up via https://github.com/node-red/node-red/pull/3676.
If nodesDir points to a directory with a package.json, it will treat it more like a regular module (and all the features that includes). If it cannot determine its a module, then it loads them as bare node files without locales etc.
The PR description covers the full details.


finish classwork3

Coverage increased (+10.0%) to 58.046% when pulling c4110e5c79949971c904a4f072cda742ff24e0db on kirillkormashov:lecture03 into 16dad782f5d596ae50d0b39665b32df40da60c19 on rybalkinsd:lecture03.


Permissions  for Data deposit - Restricted access

Hello,
You can’t grant access to the new Data Deposit functionality. Therefore, all people with an access to the administration side can process a deposit.
On the user side, all users can make a data deposit. It would be interesting to be able to restrict this right. With a restriction, you grant the access to the Data deposit only to the researchers of your institution.
Thank you

This issue is on the latest version : Nada 5.3


Deck vs. Class

Would anyone else care for a stat view that lists the win percentages of one deck against the other classes (rather than against other decks)?
I think it would be neat and in some cases more accurate, as decks are not always identified correctly and different deck types often melt into each other. It's not really interesting to know how well you perform with a particular class. You're always worried about the performance of individual decks. So the more information you get about how a different deck performs in specific scenarios, the better.

(+1) I like stats for decks more, because I play a lot of decks per class and want to see distinct winrate for each. But I have so many decks configured and found that "against deck" section is flooded. It will be handy to check winrate for the deck against a class.

@stevschmid, can I make a pull request, or we should wait for your confirmation?

I'm currently planing on revamping the whole deck system. Maybe I can address this issue as well.

Somewhat addressed with the new update ("Other ").


Language converters removed

I'm currently merging subliminal 1.0.1 into the plex agent which has been around for a while, and download_best_subtitles is erroring madly with those providers.
Please enlighten me, babelfish.language_converters.register('addic7ed = subliminal.converters.addic7ed:Addic7edConverter') never happens in your codebase, or am I missing something?
Thanks!

Providers are registered at install time, not runtime: https://github.com/Diaoul/subliminal/blob/master/setup.py#L88-L91

OK, then my monkeypatching is the only way, as the Plex-Plug-In sandbox doesn't seem to allow me to load eggs from Plug-Ins.
Thank you!


[Feature] Use Kubernetes as orchestrator

Objectives
In order to deploy to public cloud service, we need to use Kubernetes as orchestrator instead.
Key Results

Convert docker-compose to kubernetes.
Can deploy to local K8S (miniKube).
RESTful API site and gRPC server work well in K8S.


Centered-search-navbar

Here is the component I have built:
Centered search navbar having the leftmost side the navbar title and three links on the rightmost side.
Here is a screenshot link of that component
https://ibb.co/Wt0kRgL
https://ibb.co/3v4j9qy

Hey @souraOP can you give a screenshot of where is this Navbar in the Figma file.

Hey @souraOP can you give a screenshot of where is this Navbar in the Figma file.

Hey @Rajdip019 I didn't used Figma here, I screenshot it from Tailwind Play preview window.

Hey, we are trying to make a UI library which follows a same design system, not like anyone can make a UI component and dump it here. Please refer to #29 Issue to know more.

Hey, we are trying to make a UI library which follows a same design system, not like anyone can make a UI component and dump it here. Please refer to #29 Issue to know more. @souraOP

Okay, I got it. Thanks for the refer @Rajdip019

Closing this PR for now you can make a component from there make a Issue of that and then raise a PR. @souraOP


⚠ AnimeBytes has degraded performance

In 5def0da, AnimeBytes ($AB) experienced degraded performance:

HTTP code: 200
Response time: 10525 ms

Resolved: AnimeBytes performance has improved in 17eca6e after 9 minutes.


Getting exception with library-release.aar file

I am getting this issue with IBM-Watson Speech to text java sdk, Its working fine when I am using a library project source code given with it, while it gives the following error when using library project library-release.aar file.
FATAL EXCEPTION: Thread-3
    Process: com.book.reading, PID: 12353
    java.lang.NoClassDefFoundError: Failed resolution of: Lcom/sun/jna/Library;
        at com.ibm.watson.developer_cloud.android.library.audio.MicrophoneCaptureThread.run(MicrophoneCaptureThread.java:72)
     Caused by: java.lang.ClassNotFoundException: Didn't find class "com.sun.jna.Library" on path: DexPathList[[zip file "/data/app/com.book.reading-dHpwYZuOlQuiIAAnLaOW-Q==/base.apk"],nativeLibraryDirectories=[/data/app/com.book.reading-dHpwYZuOlQuiIAAnLaOW-Q==/lib/arm64, /data/app/com.book.reading-dHpwYZuOlQuiIAAnLaOW-Q==/base.apk!/lib/arm64-v8a, /system/lib64, /system/product/lib64]]
        at dalvik.system.BaseDexClassLoader.findClass(BaseDexClassLoader.java:196)
        at java.lang.ClassLoader.loadClass(ClassLoader.java:379)
        at java.lang.ClassLoader.loadClass(ClassLoader.java:312)
        at com.ibm.watson.developer_cloud.android.library.audio.MicrophoneCaptureThread.run(MicrophoneCaptureThread.java:72)

I think the aar is missing a few classes. I will look into it once I get some time, maybe next week


[Enhancement?] Send UA to website to get full context page instead of xpath

Is it possible to add UA config to get different page to one website
Sometimes, website will feed different page according UA.
It will become easy to get clean full context instead of find xpath, and it will be friendly for mobile phone rss user to read. And easily for user to config.

At the moment the plugin uses a function from tt-rss to get the content of the page. I'm not sure there is an option to manipulate this setting directly. Therefore one must implement all stuff on its own.

For a page fetching option I'm Currently Researching:
Phantomjs php Library - https://github.com/jonnnnyw/php-phantomjs
Selenium bindings  php Library - https://github.com/facebook/php-webdriver


Calling sem_init on named semaphores is not the intended use-case for sem_init

https://github.com/milk-org/ImageStreamIO/blob/d6f9c28a7cbc7b9a80c30cdd6a33a109697b4961/ImageStreamIO.c#L1712
sem_init is suppose to be applied to unnamed semaphors; see here and here.

Hi guys,
Sanford told me someone said I am mis-classifying the unnamed semaphores in ImageStreamIO.c as named semphores.
Since the memory for all semaphores in ImageStreamIO.c is allocated by sem_open, they are named semaphores; subsequently using sem_init on them does not change or remove the name.
Also note that, if they were not named semaphores, then processes other than the producer that creates them would not be able to find, open (via sem_open) and use them.

Yes, indeed, our semaphores are named. This is the correct choice: as you point out we can access them across processes. The milk-streamCTRL tool relies on this as well.

@drbitboy no problem with cleaning out the code with proper use of sem_init and sem_open. Semaphores for shared images should be named as @oguyon said.
This brings up a vague concern with a restriction with non-shared images (we're carrying a shared=0 option throughout ImageSteamIO).
Currently, non-shared images have no semaphores at all. The use of unnamed semaphores could nonetheless be useful there if Image were ever to be used in multi-thread, intra-process applications.
Thoughts on that?

Would be easy to create unnamed semaphores if shared=0. Looking into possible future upgrades, it would be nice to be able to deploy the same code as either multi-process (using named semaphores) and multi-threaded (using unnamed semaphores). Having shared=0 streams create unnamed semaphores is a small necessary step in this direction... so I am reopening this issue.

Isn't the more pressing question: why does the current code do what it does?   It's calling sem_init only after a successful call to sem_open with O_CREAT, thus re-initializing an already initialized semaphore.  From the docs:
Initializing a semaphore that has already been initialized results in undefined behavior.

This is done after every call to sem_open--O_CREAT.  I think the sem_init can and should just be removed.

I was thinking of a different solution ...
Background
ImageStreamIO currently manages, implicitly or explicitly, several shared memory files:

the shmim itself in directory [getenv MILK_SHARED_DIR], OR macro SHAREDMEMDIR ≡ "/milk/shm" OR "/tmp/",
1.1. comprising
1.1.1. the data themselves (circular buffer of "images"),
1.1.2. plus the IMAGE_METADATA struct members,
1.1.3. plus more: keywords; PIDs; and others, perhaps some perhaps not even used.
1.2. With per-process pointers in the IMAGE struct e.g.
1.2.1. e.g. [IMAGE].md (IMAGE_METADATA*) pointer to the metadata struct in the shmim
1.2.2. e.g. [IMAGE].array.raw/.UI8/.SI8/.UI16/... union of pointers to the shmim image data/circular buffer
1.2.3. e.g. [IMAGE].kw (IMAGE_KEYWORD*) pointer to keywords in the shmim
1.2.4. etc.

the semaphore data themselves, comprising several shared memory (plural) sem.{name}_semNN (
2.1. where NN runs from 0 to [IMAGE].md->sem-1
2.2. and with a per-process malloc'ed array of NN (sem_t*) pointers (i.e. pointers to pointers) of the NN semaphore data, named [IMAGE].semptr

Proposed refactor
Shift the semaphore data from NBsem separate semaphore files to the shmim with three lines of code:
sharedsize += image->md->sem * sizeof(sem_t);
...
image->semptr = map;
map +=  image->md->sem * sizeof(sem_t);

Pros

Completely eliminates all of the semaphore name and file management in item (2) above

no more mallocs and initializing an array of pointers.

semaphore creation, management, and destruction is implicit with shmim creation, management, and destruction

N.B. this means my upcoming refactor using INODE to change shmims cleanly on the fly, e.g. for a shmim size change, will not have to fiddle around separately with semaphore INODEs, because the semaphores are essentially encapsulated within, and get updated with, the shmim.

no more stale semaphore files hanging around when a process gets an unhandled signal.
only the semaphore creator, which is folded into the shmim creator, needs to call sem_init on each semaphore datum.

Cons

There are no longer (file-)named semaphores

but

A) that's a good thing because of the Pros above, and
B) that does not matter because any processes interested can find these semaphores via the (file-)named shmim anyway.

All ImageStreamIO library clients must use the ImageStreamIO_sem*(...) calls to access and use the semaphores

but they should be doing that anyway.
IIRC, shmimMonitor in MagAOX is a violator here:  it uses the system call sem_timedwait; this is easily fixed.

I am playing a little fast and loose with the semptr assignment code above, which now becomes a pointer to the first of an array of (sem_t) structs, instead of a pointer to the first of an array of (sem_t*) pointers.

So again, any ImageStreamIO library clients that are accessing shmim semaphore internals directly, instead of via the ImageStreamIO_sem*(...) calls, may break.


Marc21Record Output

This PR adds the functionality to pretty-print a Marc21Record in XML format.

when are you planning to write the tests?

Test and schema validation are coming right away.


Frp

Beginning of an applicative FRP lib.

Merging for now! It's p basic and missing a lotta things, but we can start to flesh out.


Checking for Root apps on Android 11+ is ineffectual due to package-visibility changes

https://developer.android.com/about/versions/11/privacy/package-visibility
Does rootbeer support these changes?
Or everything works fine on android 11?

Will add
<queries>
 ...
</queries>

resolve this issue?

Potentially, but I saw in one of these Google I/O 2021 talks What’s new in Android privacy or The state of Android security that Google are changing the Play policy to make access to package queries only available to apps that genuinely need it. I'm not sure they would agree that root checking qualifies given SafteyNet is offered. But I could see apps getting pulled if we implemented this check.

What if add an optional lib with manifest, something like
implementation 'com.scottyab:rootbeer-packages:0.0.9'

Yep, that could work @BanHammerYKT. Would welcome a PR


docs: contributing: videos: OBS Stream profiles

For creating videos

With the meeting windows snapped to the left half of the 4k montor

Meeting recorder idea:

detect which PRs were open in video
transcript

Per person audio
Links to files when they are talked about


Tutorial to start a new Telestion project

Summary
Add a tutorial to start a new project.
Details
A first iteration on the steps required to set up a new Telestion project. :wink:
Additional information
n/a
CLA

[x] I have signed the individual contributor's license agreement and sent it to the board of the WüSpace e. V. organization.

Done! :D

@fussel178 Congrats on your first documentation article! 🍰


Slack?

This was discussed in issue #193 a couple years back. I'm wondering if now would be a good time to re-consider opening a Slack for the community? FWIW, Kubernetes (referenced as an example in that issue) now has a thriving Slack community.
I know personally a Slack (or irc channel) would help me stay connected with what's happening in the community.

Thanks for bringing this up - it has been on our plate for a while. We will discuss as a team and see what solutions are feasible for us before we put our ideas here.

Either Slack or IRC would be great, and the team here is on board with it, although we don't anticipate being readily reachable at all times (issues will still be useful for getting attention). I am interested in hearing a few more opinions on which tool people prefer (Slack, or IRC). @AdamIsrael: do you have a preference?
@meteorfox do you have any thoughts on this?

I think is a great idea. In terms of tools, Slacks seems very popular. Besides IRC, there's also Gitter.

Great, I personally vote for Slack. @AdamIsrael: do you want to set this up, since you brought it up? I am happy to do so as well if you prefer.

+1 (Either slack or IRC... or both!)

I finally got around to creating an IRC channel on Freenode, #PerfKitBenchmarker! Thanks for everyone's input.


Update freezes while downloading  external chapter and novel cover image doesn't shows up and a bitmap empty prompt.

["mike211"]Hi  Nandaka,
I just want to inform you that there is a problem with "Konjiki no Wordmaster". When updating it get stuck on "Downloading external chapter info 763 of 770".
This problem made the "update service" and "download all info" for web function to be stuck due to the same problem.
I check the website and it seem there is a few chapters unlinked.
Could this be the cause of the update getting stuck?

Try to untick Include External Chapter in Settings -> Update and Notification -> Include External Chapter.
Can you get the adb log files? I need to know which page that causing the stuck up.

Here is the adb log that you requested and regarding the the novel cover image doesn't shows up.
My device is able to browse the website using the tablet browser and manual refresh still prompt "bitmap empty" .
problems.zip

from the log file:
org.jsoup.HttpStatusException: HTTP error fetching URL. Status=410, URL=https://loliquent.wordpress.com/

Looks like url problem? I think you need to manually delete those chapters linked by loliquent, as the site already removed.

Bitmap empty

Which novel is affected? I only see ShinSekaiYori and Zaru Soba from log file.
I've tried on my device and emulator, and it is downloaded properly.
[ 09-09 12:42:55.154  7735: 7735 E/BitmapFactory ]
Unable to decode stream: java.io.FileNotFoundException: /storage/extSdCard/Android/data/com.erakk.lnreader/files/project/images/2/2a/ShinSekai_Yori_Cover.jpg: open failed: ENOENT (No such file or directory)

[ 09-09 12:41:21.537  7735: 7735 E/BitmapFactory ]
Unable to decode stream: java.io.FileNotFoundException: /storage/extSdCard/Android/data/com.erakk.lnreader/files/project/images/b/bf/91P+V6q5AyL._SL1500_.jpg: open failed: ENOENT (No such file or directory)

can you check if the file/folder exists?

I have tired deleting the chapters, clearout the cache and even deleting the novel altogether, but the same problem still occur.
Regarding the bitmap empty problem, it seem to affected only the new novels that I haven't downloaded before.

While checking the folder , I have come across more then few .jpg that is corrupted, should I manually delete them?
Also there is over 7000+ thumb.php files in /storage/extSdCard/Android/data/com.erakk.lnreader/files/project. Are they suppose to be there ?

I would need to purchase a new one and re-download everything and then proceed from there if problems still occur.
By the way is there any chance you know where I might find a download/torrent that containing of a backup of all files (images and the novel data including those that have been abandon) of Bakareader EX? Cause I have a few abandon novels on my bakareader and I want to re-download the images for those as well.

Update
I brought a new 64 gb mirco sd card and while using the "update now" function, I notice it wasn't redownloading any of the novel illustrations on my watch list and that I would have to one by one manually download them. Also it doesn't download the update of the chapter, it just show (!) beside the novel title.
"Bitmap empty"
Regarding this problem, manually refresh now work and will download novel cover image, but the "download novel info" function on the novel list still don't download novel cover image.
"Konjiki no Wordmaster"
Nothing change, still getting stuck while updating Konjiki no Wordmaster.

thumbs.php

it is normal, as the thumbnails is generated by php if you check the website. Technically, I can make it to have better filename, but no time :smile:

backup images

sorry, I don't know about that.

logs

from log file, looks like you have IO/access problem.
For some reason, it cannot update the DB.
Can you try to uninstall the apps and start from fresh DB? (Backup you old db and images first).

[ 09-14 04:25:52.400  1469: 2515 E/class com.erakk.lnreader.task.GetUpdatedChaptersTask ]

Failed to update: Lillia to Treize

Error Code=-6004: Failed to update: Lillia to Treize. Inner Exception: disk I/O error (code 3850)

	at com.erakk.lnreader.task.GetUpdatedChaptersTask.processWatchedNovel(GetUpdatedChaptersTask.java:232)

	at com.erakk.lnreader.task.GetUpdatedChaptersTask.getUpdatedWatchedNovelList(GetUpdatedChaptersTask.java:263)

	at com.erakk.lnreader.task.GetUpdatedChaptersTask.getUpdatedChapters(GetUpdatedChaptersTask.java:110)

	at com.erakk.lnreader.task.GetUpdatedChaptersTask.doInBackground(GetUpdatedChaptersTask.java:50)

	at com.erakk.lnreader.task.GetUpdatedChaptersTask.doInBackground(GetUpdatedChaptersTask.java:25)

	at android.os.AsyncTask$2.call(AsyncTask.java:287)

	at java.util.concurrent.FutureTask.run(FutureTask.java:234)

	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1080)

	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:573)

	at java.lang.Thread.run(Thread.java:856)

Caused by: android.database.sqlite.SQLiteDiskIOException: disk I/O error (code 3850)

	at android.database.sqlite.SQLiteConnection.nativeExecuteForLong(Native Method)

	at android.database.sqlite.SQLiteConnection.executeForLong(SQLiteConnection.java:784)

	at android.database.sqlite.SQLiteSession.executeForLong(SQLiteSession.java:652)

	at android.database.sqlite.SQLiteStatement.simpleQueryForLong(SQLiteStatement.java:107)

	at android.database.DatabaseUtils.longForQuery(DatabaseUtils.java:813)

	at android.database.DatabaseUtils.longForQuery(DatabaseUtils.java:801)

	at android.database.sqlite.SQLiteDatabase.getVersion(SQLiteDatabase.java:968)

	at android.database.sqlite.SQLiteOpenHelper.getDatabaseLocked(SQLiteOpenHelper.java:242)

	at android.database.sqlite.SQLiteOpenHelper.getReadableDatabase(SQLiteOpenHelper.java:188)

	at com.erakk.lnreader.dao.NovelsDao.getPageModel(NovelsDao.java:569)

	at com.erakk.lnreader.dao.NovelsDao.getPageModel(NovelsDao.java:591)

	at com.erakk.lnreader.model.PageModel.getPageModelByName(PageModel.java:66)

	at com.erakk.lnreader.model.PageModel.getPageModel(PageModel.java:193)

	at com.erakk.lnreader.task.GetUpdatedChaptersTask.processWatchedNovel(GetUpdatedChaptersTask.java:179)

	... 9 more

[ 09-14 04:25:55.439  1469: 2515 E/SQLiteDatabase ]

Failed to open database '/storage/emulated/0/Android/data/com.erakk.lnreader/files/databases/pages.db'.

android.database.sqlite.SQLiteCantOpenDatabaseException: unable to open database file (code 14)

So you want me to try uninstall the apps and start from fresh DB and not even loading the backup?
Regarding the images, I have none.  I am still quite upset about it, because my previous sd card that contain everything of Baka reader have die on me.

thumbs.php
it is normal, as the thumbnails is generated by php if you check the website. Technically, I can make it to have better filename, but no time :smile:
Might I suggest creating a folder for those, cause I notice it took a while to load all those on my pc before I have access to the image folder.

Ok, I did what you have told me. I have backup db and uninstall the apps and reinstall again, but now for some reason my backup db won't load.
Please I really need your help, I have already lost the images, I don't want to lose the novels I have saved up whiles I was waiting for the original problem to be solve, lost as well.
I will link the db backup I have here.
https://mega.nz/#!M0UjESZQ!PaZSMs9oHZQJVJBPGQ2az-JGzUxf8u5b55NB8oHvxJc

can you follow the instruction from http://wiki.team-mediaportal.com/1_MEDIAPORTAL_1/16_Support/Troubleshooting/Repair_SQLite_Database ?
change the db name to the backup db. Please note that if the db is corrupted, it might only be able to recover part of you data.

Thank you, I have follow the instruction and have repaired most of the data, some of the abandon novel that I have, have missing chapters here and there other than that I should be able to redownload the others.
Question, How did my backup corrupted itself?

I'm not sure, maybe SDCard issue? Other than that maybe the writing is interrupted (e.g. service killed).
Fyi, you can enable autobackup with multiple copies created in settings page.


Fix haproxy check redis master configuration

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

No
What this PR does / why we need it:
Fix the problem haproxy leads to redis cluster service be unavaible
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #16708

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[ ] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @ssalaues

/ok-to-test

@jeremyxu2010 looks like some other PRs just merged in. You will have to rebase and bump the version once again 😅

/retest

/lgtm

@ssalaues @jeremyxu2010 I'm running the chart for version 4:
# Set specific version of Redis
image:
  tag: "4.0-alpine"

# Use password from a secret
existingSecret: redis-password

And now my container keeps crashing with the error message:

I tried running an older version of the helm chart, but that doesn't seem to fix the problem. It seems very related to this PR. Do you know how I can resolve this for version 4?

Looks like I've had to manually apply the "old" value in the helm chart config file:
redis:
  config:
    min-slaves-to-write: 1
    min-slaves-max-lag: 5   # Value in seconds

yes, see doc


Referenced example is missing

In the section "Non-nullable properties and initialization", the second sentence says "In our example above, we avoided these warnings by using constructor binding ..."
Did the example that's mentioned get moved to a different page? I don't see any examples on this page that use constructor binding, although I could be missing something :)

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

ID: 93c3721a-da23-d9ea-a054-9d786d2ff4d2
Version Independent ID: 6d3b0f96-6642-75e5-93c8-02c23f142ac4
Content: Working with nullable reference types - EF Core
Content Source: entity-framework/core/miscellaneous/nullable-reference-types.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @roji
Microsoft Alias: avickers

You're right, this was probably a result of some change at some point. I'll amend the docs with an example of using constructor bindings.

Cool; thanks :)


Get the tests working with python 3.6

With these small changes everything builds and runs under python 3.6

This still needs to be tested under python 3.6 before being merged. I'll do that later and post a comment with the results.

All test pass on python3.6.

so this is ok to merge, right?


Bug in Data Splitting on FewRel Dataset

Hey,
The fewrel dataset has 700 sentences per relation id.
After splitting the FewRel into train/dev/test, you should get 10500 sentences in the test split as you have 15 unseen relation ids.
Using your code, we get fewer sentences on the splits. I tested with seed 12321, and there are 200 sentences missing on the test split.
Please fix this issue and re-evaluate the results for the main paper.

Hi, the reason for fewer samples is that some samples have the same text, hence they are merged to form the multi-triplet sentences.


Add Indonesia Dart course

What does this PR do?
Add resources
For resources
Dart tutorial for indonesian book
Description
I added a link to the Dart tutorial that I made myself on my web course, in Indonesian language
Why is this valuable (or not)?
This is very valuable, because the tutorial is very complete and detailed using the latest version of this technology
How do we know it's really free?
because this is my own creation
For book lists, is it a book? For course lists, is it a course? etc.
is it a course
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

if this is a course, it should go in the courses list, don't you think?

if this is a course, it should go in the courses list, don't you think?
Sorry, that was my mistake, this is more suitable as a reading article than a course.

dart resources are dead, flutter resources are not books


Subreddit network management tools

Every mod action that gets done in one subreddit also needs to be propagable to the entire network of subreddits. Some of these actions are:

[x] AutoModerator #29
[ ] subreddit/about/edit settings
[ ] subreddit/about/flair settings
[ ] sticky posts
[ ] sidebars
[ ] bans
[x] mods
[x] toolbox configs (/wiki/toolbox json)
[ ] CSS #43
[x] send modmail to each subreddit
[ ] about/rules #42

Where appropriate, we need to account for Network content and Subreddit content by using "start blah end" delimiters (for example, the sidebar and automoderator).
Let's use this issue to finalize the ideas we have on Discord.

some of these make sense to group into one command, some don't. I'm thinking a general purpose ion_propigate, with separate commands for ion_invite_mods and ion_bulkmail right now.

#16 is no longer valid, this will replace it.


make it easy to wait for HABackend to reach a certain state

some code contains sleep(1) now because "by then, surely we have done our initial entities fetch". Instead we should be able to do something like backend.waitFor(someenum::GotEntities)

HABackend::Start() now waits until entities are loaded. Do we need more here?

I think we can close this one for now.
There might be a need to wait for specific commands, but i see very little reason for that at the moment. So i'd want to close this one because i think the initial reason for opening this item is no longer there.


Adding remember me and forgot password in login form

Feature request-
The login form doesn't  have a remember me and forgot password. I would like to add that.
Describe the solution you'd like
I would like to add remember me and forgot password

Note:

Do mention if your are working under GSSoC'21.
If you want to work on an issue, you should check if it has already been assigned to anyone. If the issue is free you can comment /assign to get the issue assigned to you.
If you are raising a new issue and want to work on it then also you should comment /assign under the issue to get it auto assigned.
Please refrain from adding labels to your issue/pull-request on your own. It is the job of the Project Admin and the Mentors to review your issue/pull-request and add labels accordingly.
According to the level of the issue, respective time will be allotted under which the task needs to be completed/resolved by the assigned person. If the issue is not resolved within the given time, it will be assigned to the next person. In case of any discrepancies, the decision of the maintainer will be final.

level - 0: 2 days
level - 1: 3 days
level - 2: 4 days
level - 3: 5 days

/assign

@Pranav016 I have made a PR. Please review. I will be making another PR for adding remember me functionality


Ads doesn't show up

i've followed all steps of instalation, but when it's running, the ads doesn't show up? do i miss something?

As a bug report or issue report, it should include more details. Or else no one knows what you did and what you didn't. What tool do you use, which platform, version, etc. And, can you paste any log or screenshot?

iam sorry for the inconvenience, here's the details of the bug :
screenshot when using Intel XDK (adding admob plugins) :

screenshot when using intel xdk (when app's running) :

when the app's emulated, there was no ads showed...

When you see the emulated app, it's not running in device emulator but web app emulator, so it's running the web code only, no plugin running yet.
To see the full features, you have to configure the plugins and build the app either iOS or android, then download and install the package to device or device emulator.

the ads also doesn't show when using CLI, here's the details :
Screenshot using CLI (app launched success)

screenshot Using CLI (app launched on device)

the ads still doesn't show up

here's provide details from logcat when running from eclipse :

it has an exception (highlighted) related to AdMob

are you sure you have the plugin installed successfully? show me the plugin list...

yes it does, here's the plugin list..

i had repeat those step few times, but the problem still exist...

The plugin was not installed properly, valid installation should also show a plugin named com.google.playservices to be installed via git from github repo.
Please install git, then install the plugins again.

i already add google play service...

but still doesn't show up..

Clean and rebuild, and any eclipse logcat logs?

logcat from eclipse

AdMob is not defined, means that the plugin is not installed properly.
I guess it's due to previous errors.
Try keeping the plugins no change, but remove and re-add android platform by running following commands:
cordova platform remove android
cordova platform add android

Then rebuild the app.

finally the ads show up :)

previously, i used "rm" command to remove platform or plugin and it doesn't working.
whether using different command has different result??

congratulations, finally you made it ! :+1:

Thank u for ur assist, this tutorial provides ease way to add Admob Plugin :+1:

Hi floatinghotpot,
i've followed all steps of installation, but when it's running, the Admob is undefined. I tried all steps in others same issues but the issue still happen. Do I miss anything ?
Below is my plugin list:
cordova-plugin-admobpro 2.10.0 "AdMob Plugin Pro"
cordova-plugin-extension 1.2.3 "Cordova Plugin Extension"
cordova-plugin-whitelist 1.1.1-dev "Whitelist"
ionic-plugin-keyboard 1.0.6 "Keyboard"

Some Ionic developers just view their index.html in IonicViewer and found AdMob undefined.
In fact, as the AdMob plugin has native code, the cordova project has to be built into app package and install to simulator or device.
What's the detailed steps to re-produce your issue? What tool do you use to build the app? Do you run your app in IonicViewer, local simulator, or your android / ios device?

Hi floatinghotpot,
Actually, I used Cordova + Ionic to build app. The "admob-service.js" will be called once device ready. So, I faced that issue in case I use command "ionic serve" or "cordova build android". The issue of "Admob is undefined" appear on Brower or Device.

Ionic serve won't work, as the plugin won't be loaded in browser. You need use "cordova run android" to build the app and install app to device to run.

Yes, I tested on Browser and Device. I also used "Cordova build android" command to build an APK file and installed it into my real Device. But that issue happen also.

Hi,
I fixed this issue by following this topic: http://www.tagwith.com/question_571864_window-plugins-admob-always-undefined-in-ionic-app
Summary:
Instead of using window.plugins.AdMob, please directly use AdMob, or window.AdMob.
It's defined in the plugin.xml:


discord: 0.0.11 -> 0.0.12

Motivation for this change
Backport of #97694.
CC: @jonringer
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

When backporting changes, please follow https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md#backporting-changes.
Namely, you should be doing git cherry-pick -x <rev> from a commit that has already landed in master. If the branches have diverged, you may alter the commit or add another commit to ensure that the package is able to still evaluate and build

@jonringer: Alright, I will update the PR now.

@jonringer: Please take another look


Download hangs on Ubuntu server

I am stuck on downloading Redis for more than 20 minutes, it never shows a progress update. I am on a Paperspace A100 instance.

In comparison it took 10 seconds to complete the docker pull from CLI

PS: Every 10 seconds the daemon logs
2023-10-02 21:47:17 ERROR    An error occurred: argument of type 'NoneType' is not iterable

@casperdcl are you working on this on the daemon side?


fix: Patch regressions related to #480

Description
I ran an end-to-end test on the GenWealth Demo App and found a few regressions related to migrating the project as part of #480. This pull request fixes the following regressions:

Database tables were not loaded properly.
Documentation referenced incorrect paths and steps.
Research prompt contained a sentence fragment.
UI container build failed.

My second end-to-end test was successful after making these changes.
Thank you for opening a Pull Request!
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[X] Follow the CONTRIBUTING Guide.
[X] You are listed as the author in your notebook or README file.
[X] Your account is listed in CODEOWNERS for the file(s).
[X] Make your Pull Request title in the https://www.conventionalcommits.org/ specification.
[X] Ensure the tests and linter pass (Run nox -s format from the repository root to format).
[X] Appropriate docs were updated (if necessary)

@holtskinner - FYI: This pull request fixes a few regressions related to the GenWealth Demo App.


Image/document modal not showing when DEBUG = False

On my production server the image modal is nog showing when DEBUG is disabled. Its seems to be a client-side javascript issue.
There are no network errors. But the admin interface starts with the console error:
747bc443edaf.js:272 Uncaught TypeError: $(...) is not a function(anonymous function)
When I click on "select an image" :
VM10084:1 Uncaught SyntaxError: Unexpected token
When I enable DEBUG, the issue can no longer be reproduced. Both of the errors do not occur.
Not an urgent issue, but it would be nice to be able to turn off DEBUG

The difference between debug being on or off is that in production, Wagtail will minify, concatenate, and cache the javascript files. In development, Wagtail will serve each file as a separate HTTP request.
So maybe something is going wrong when the files get concatenated and minified. Do you have any custom javascript files that would cause a problem if concatenated? eg one that doesn't contain a semi colon or bracket where it should.
Check for errors in you console when in debug, too.

@Fonz001 What version of Wagtail are you using?

Hi!
@alexgleason Your solution worked, a missing semicolon in custom javascript! Thanks guys! +1


Missing lymphgen script

Hello,
Thank you for all your hard work in creating and maintaining this suite of modules, it has been a great resource!
I had a question about running the lymphgen module: The config["lcr-modules"]["lymphgen"]["options"]["lymphgen_run"]["lymphgen_path"] parameter is set to a local path in the current version ("/projects/rmorin/software/lcr-scripts/LymphGen/Run_Lymphgen.R") and I have trouble locating that script for my own run. It doesn't seem to be included in the current version of lcr-scripts and there is no src directory under modules/lymphgen/1.0. I had a look through the code to check for any downloads, but the only one that seems to be there is the LGenIC repo, which doesn't include that R script.
Could someone please point me in the right direction? I'm aware lymphgen has a web tool but I was looking to incorporate the post-processing of the results into a snakemake pipeline, which would make manually using a web tool inconvenient.
Thanks a lot for your help!
Ilias

Hi Ilias,
Thanks for your message, I'm glad you're finding our tools useful. Unfortunately the LymphGen source code was shared with us by George Wright and we don't have his permission to distribute it outside of our organization. I realize this limits the utility of our module for you, however I see a couple of possible ways forward for you:

Set the target of the LymphGen module to be the outputs of LGenIC, then upload those files to the LymphGen web tool.
Contact George Wright and ask for permission to access the source code.

Sorry I can't be of more assistance with this.
Laura

Can we also ask George directly if he's alright with us sharing it broadly?

Thank you both very much for the responses!

I've had a look more closely at the suggested fix from the Morin lab- I'm not happy with their LGENIC solution as it doesn't include SV data.  I think I'm going to try create our own instead (using/ upcycling some old code from when I've done this previously).

Closing this issue as inactive


Fix tracking implementation on continue button

What
https://trello.com/c/QvEohsT7/1153-fix-link-tracking-on-continue-button-for-request-accessible-format

Fix incorrect tracking implementation of data_attributes.
Fix incorrect data-track-label attribute

<button class="gem-c-button govuk-button"
  type="submit"
  data-module="gem-track-click"
  data-track-category="requestAccessible"
  data-track-action="Audio - Cabinet Office"
  data-track-label="Second State Pension age review: independent report call for evidence">Continue
</button>

Why
The link tracking on the Continue button, on Contact information, is currently not generating any GA events. This is very likely due to an incorrect implementation of the tracking attributes. Double data attributes are rendered. Also, the attachment title is not pulled through in to track-label attribute.
<button class="gem-c-button govuk-button"
  type="submit"
  data-module="gem-track-click"
  data-data-track-category="requestAccessible"
  data-data-track-action="Audio - Cabinet Office">Continue
</button>

Anything else

This looks sensible, have you been able to see it working? I don't know a useful url to throw at it in the preview app - or even whether the heroku preview will be joined up enough to render the pages. If not maybe we could drop it on integration to confirm it works?

This looks sensible, have you been able to see it working? I don't know a useful url to throw at it in the preview app - or even whether the heroku preview will be joined up enough to render the pages. If not maybe we could drop it on integration to confirm it works?

Yep, I've checked in Integration and it looks correct to me. It's still on there at the moment on the Contact information page - https://www.integration.publishing.service.gov.uk/contact/govuk/request-accessible-format?content_id=63e4a4be-d67c-4163-97d6-ff56792db93e&attachment_id=6414765


Re-enable disabled ModuleExecutionFlowTests

Description:
Commit: https://github.com/ballerina-platform/ballerina-lang/commit/4c1d0f489684e03f2d4db13048d84c50f8b521f7
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Some of the observability test are disabled with the mentioned PR. The disabled packaging test case is enabled already.

There is a separate issue[1] for disabled observability tests.
[1] - https://github.com/ballerina-platform/ballerina-lang/issues/27300


Review MethodCallTranslators that match on method name instead of MethodInfo

I noticed some of our translators are doing this. We should review whether it is safe - matching on name alone can yield false positives.

Dupe of https://github.com/aspnet/EntityFramework/issues/8060