LargePayloadTest fails with SwiftNIO 1.7.0

LargePayloadTests.testLargeGets fails consistently with SwiftNIO 1.7.0 with:
*** Error in `/root/pushkar/Kitura-NIO/.build/x86_64-unknown-linux/debug/KituraNIOPackageTests.xctest': realloc(): invalid next size: 0x00007f5e4004c110 ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x777e5)[0x7f5ea5d447e5]
/lib/x86_64-linux-gnu/libc.so.6(+0x834aa)[0x7f5ea5d504aa]
/lib/x86_64-linux-gnu/libc.so.6(realloc+0x179)[0x7f5ea5d51839]

This is an isolated recreate for the bug:
import KituraNIO
import Foundation

class TestServerDelegate : ServerDelegate {
    func handle(request: ServerRequest, response: ServerResponse) {
        handleGet(request: request, response: response)
    }

    func handleGet(request: ServerRequest, response: ServerResponse) {
        let payloadData = Data(repeating: 65, count: 10658240)
        do {
            response.headers["Content-Length"] = ["\(payloadData.count)"]
            try response.write(from: payloadData)
            try response.end()
        }
        catch {
            print("Error writing response.")
        }
    }
}

let httpServer = HTTPServer()
httpServer.delegate = TestServerDelegate()
try! httpServer.listen(on: 0)

let request = HTTP.request("http://<IP_ADDRESS>:\(httpServer.port!)/") { response in
    print(response!.statusCode)
}
request.end()
Dispatch.dispatchMain()

This crash happens very frequently after swift-no 1.7.0. It was extremely intermittent with 1.6.1.
Reported this issue to the Swift-NIO team via Slack!

Fixed in swift-nio 1.8


Update deeplab mcloud yaml

Closing because the yaml was already updated by https://github.com/mosaicml/examples/pull/130


Implement dbt snapshot

See https://docs.getdbt.com/docs/building-a-dbt-project/snapshots
Possible reference to implement this feature:
https://github.com/dbt-labs/dbt/tree/develop/plugins/postgres
https://github.com/dbt-labs/dbt-spark/blob/master/dbt/include/spark/macros/materializations/snapshot.sql

@hovaesco it may be a while until that PR gets merged.  I noticed that other features that use the merge statement have been re-written here (i.e. incremental materialization).  Would it be acceptable to take the same approach here?

In order to have atomic merge incremental strategy, we do require MERGE statement support in Trino. My belief is that the only acceptable solution. Do you refer to other incremental strategies?

Do you refer to other incremental strategies?

Hey thanks for taking the time to respond.  Just so I'm on the same page, what do you mean by incremental strategies?
On my end I was referring to how trino overrides this merge statement for incremental models.

https://docs.getdbt.com/docs/building-a-dbt-project/building-models/configuring-incremental-models#about-incremental_strategy

At this moment dbt_trino only supports append incremental models. See https://github.com/starburstdata/dbt-trino/blob/master/dbt/include/trino/macros/materializations/incremental.sql
We have an open PR for adding the delete+insert strategy. However to be able to test it, we need to expand the test setup with iceberg and delta.
I actually concur with @p11o that we could implement this feature using delete+insert but still would suggest getting above two PR's in first.

Now that both merge and delete+insert is merged there are no blocker for this ticked?

@retrry exactly, snapshot will likely land very soon on dbt-trino.

Could you release dbt-trino 1.2.2 with snapshot support?

Sure, we'll release it tomorrow, sorry for the delay.

@retrry here you go https://github.com/starburstdata/dbt-trino/releases/tag/v1.2.2

@hovaesco thanks!


[BUG] 'phaser' does not contain a default export (imported as 'i')

Version
0.1.1
Describe the problem
I'm trying to use this plugin with Next.js, and when compiling / loading a page I get this error:
./node_modules/phaser-font-plugin/dist/index.mjs
Attempted import error: 'phaser' does not contain a default export (imported as 'i').

as the error says, it seems Phaser does not export a default so importing with
import Phaser from 'phaser'

creates an error. I guess it doesn't for you because you build with a builder that might just do the magic, but next doesn't.
Would it be possible for you to use the form
import * as Phaser from 'phaser'

or simply use the modules (Plugins / Loader) directly?
Additional info
No response

👋 @dievardump
Thanks for opening your first issue here!
We'll try to help you as soon as possible.
If you like this project, please ⭐ star our repo.


Retrieve a data manifest containing UCSC Blue Box UUIDs pushed from the Boardwalk data explorer and display it as a static table in a Project

User logins into Boardwalk (UCSC) with google ID
User applies filters and selects a subset of the data -> creates a manifest which includes references to UCSC’S Blue-Box UUIDs
User clicks ‘send manifest to Saturn’
Saturn as an application retrieves the manifest from Boardwalk and populates the Project data display
User is directed to the Saturn Project data-display screen, and views a static table of information from the manifest

For the purposes of this issue we can assume there is only one pre-populated Project in the application, and that all data pushes should be sent to this single Project. Choosing projects from a list and project-creation will follow as subsequent stories
The handover process represented by step4 above is discussed here
The data manifest is being represented in bdBag format (essentially, a TSV packed inside an XML wrapper). It's unclear at this point if the data retrieval API being exposed by Boardwalk will return the full bdBag, or just the TSV (we can make requests to them regarding our preferred functionality!)
The data retrieval API has not been provided by UCSC yet- a significant source of risk.
We have not yet received an example bdBag/TSV. We should request this of UCSC ASAP.

See UI design mocks with a clickthrough path here
"Data Explorer" is a stand-in for Boardwalk in these mocks

We have also made significant progress towards unified auth between boardwalk and our system so we can also make more direct interactions (ie assume they can make authenticated calls to all our APIs )

On Feb 27, 2018, at 5:59 PM, Bradley R Taylor<EMAIL_ADDRESS>wrote:
See UI design mocks with a clickthrough path here
"Data Explorer" is a stand-in for Boardwalk in these mocks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Train/val loss on flickr30k

Hi,
Thanks for your nice work. I want to ask what are the training and validation losses for flickr 30k dataset in your experiments. I experimented with your code on flickr 30k but it seems to be overfitting and the losses are very high (about 10.7). Do you have any ideas on this?
Thank you.

Hey, so the train loss was 10.03 and the validation loss was 10.02. These are quite high, but the challenging part was reducing them + computing power was limited. I don't have any good ideas now how to improve this, maybe playing with forward_expansion and num_layers layers of Mapping?


fzf: 0.25.1 -> 0.26.0

Motivation for this change
Version bump.
https://github.com/junegunn/fzf/releases/tag/0.26.0
Things done

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

[ ] NixOS
[x] 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"
[x] 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
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 116229 at 0e3fb2ae run on aarch64-linux 1

1 package marked as broken and skipped:

zoxide

1 package failed to build (new failures):

haskellPackages.neuron

8 packages built successfully:

fontpreview
fzf
fzf-zsh
navi
neuron-notes
spacevim
tmuxPlugins.tmux-fzf
xmloscopy


Testing login endpoint

When the TESTING_LOGIN config value is True, /login will redirect to /testing-login, which just asks for your email and lets you in the back door. Tests can log in users by POST<EMAIL_ADDRESS>Both /testing-login and /testing will 404 otherwise.

Overall looks good. Let's resolve the two comments above and we should be set to merge.
Here's the relevant part of api.py: https://github.com/Cal-CS-61A-Staff/ok/blob/rewrite/Sumukh/v3/server/controllers/api.py#L91

Simplified a little:

Moved login_manager initialization to auth.py
Use Flask-Login's request_loader to authenticate API requests (tested with enrollment API, works)
Users' names aren't in the model and I couldn't get Google to return them, so I took them out
I thought keeping the access token in the session was a little messy because it's not actually used anywhere besides a debug endpoint. So I added /access-token, where you can sign in again to get a Google access token. Does this need a new home?

Looks good. I haven't had a chance to test it locally - but if you want to merge it - go ahead!


feat: add ludo seed style options

Description
feat: add ludo seed style options so user can select different styles of ludon pins/seeds
Detailed description of proposed changes
#92

[x] Read the contributing docs (if this is your first contribution)

[x] Verified this is not a duplicate of any existing pull request

[x] Performed a self-review of the code

[x] Rebased to the last commit of the target branch (or merged it into my branch)

[x] Linked the issues which this PR resolves

[x] Documented the changes in code (API docs will be generated automatically)

hello @princeibs kindly review :)


"Removed in Sphinx 3.0" warnings

$ make -C tutorials
Running Sphinx v2.0.1
making output directory... done
loading intersphinx inventory from https://docs.kytos.io/python-openflow/objects.inv...
loading intersphinx inventory from https://docs.kytos.io/kytos/objects.inv...
loading intersphinx inventory from https://docs.python.org/3/objects.inv...
building [mo]: targets for 0 po files that are out of date
building [dirhtml]: targets for 15 source files that are out of date
updating environment: 15 added, 0 changed, 0 removed
$VIRTUAL_ENV/lib/python3.8/site-packages/sphinx/util/nodes.py:149: FutureWarning:
   The iterable returned by Node.traverse()
   will become an iterator instead of a list in Docutils > 0.16.
  for classifier in reversed(node.parent.traverse(nodes.classifier)):
reading sources... [100%] references
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
$VIRTUAL_ENV/src/kytos-sphinx-theme/kytos_sphinx_theme/kytos/layout.html:24: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.
  {% include "navbar.html" %}
$VIRTUAL_ENV/src/kytos-sphinx-theme/kytos_sphinx_theme/kytos/layout.html:56: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.
  {%- block content %}
writing output... [100%] references
generating indices... genindex
writing additional pages... search
$VIRTUAL_ENV/src/kytos-sphinx-theme/kytos_sphinx_theme/kytos/search.html:21: RemovedInSphinx30Warning: To modify script_files in the theme is deprecated. Please insert a <script> tag directly in your theme instead.
  {% endblock %}

copying static files... done
copying extra files... done
dumping search index in English (code: en) ... done
dumping object inventory... done
build succeeded.

The HTML pages are in _build/dirhtml.

Build finished. The HTML pages are in _build/dirhtml.

I'm running sphinx-theme v3.2.1 and I did not got those warnings, I got another ones, and they can be solved by just replacing
.. code-block:: bash by .. code-block:: console in the indicated lines. I really don't know if those warnings are related to sphinx version yet
making output directory... done
loading intersphinx inventory from https://docs.kytos.io/python-openflow/objects.inv...
loading intersphinx inventory from https://docs.kytos.io/kytos/objects.inv...
loading intersphinx inventory from https://docs.python.org/3/objects.inv...
building [mo]: targets for 0 po files that are out of date
building [dirhtml]: targets for 15 source files that are out of date
updating environment: [new config] 15 added, 0 changed, 0 removed
reading sources... [100%] references
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
writing output... [100%] references
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/create_looping_napp.rst:217: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/create_looping_napp.rst:277: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/create_ping_pong_napps.rst:460: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/create_your_napp.rst:290: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/publishing_your_napp.rst:200: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/publishing_your_napp.rst:222: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/restful_statistics.rst:106: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/switch_l2.rst:325: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/switch_l3_part1.rst:340: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
/home/barbara/Documents/Github-repos/documentation/tutorials/napps/switch_l3_part2.rst:527: WARNING: Could not lex literal_block as "bash". Highlighting skipped.
generating indices...  genindexdone
writing additional pages...  searchdone
copying static files... ... done
copying extra files... done
dumping search index in English (code: en)... done
dumping object inventory... done
build succeeded, 10 warnings.

The HTML pages are in _build/dirhtml.

Build finished. The HTML pages are in _build/dirhtml.

Those warnings should already be fixed by 5bec7a99a17f27ce5e4434c3ab0c4d7c433c8314, please pull the latest code and try again.


Publish Azure CLI to Microsoft Store

PowerShell and Windows Terminal are in Microsoft Store, would be great if Azure CLI could join the party by being distributed the same way as well.

PowerShell
PowerShell Preview
Windows Terminal
Windows Terminal Preview

hi @o-l-a-v thanks for the feedback. Azure CLI is releasing per different platforms via package manager. We don't have plan to support Microsoft Store yet. To install it, pls try out guideline at https://docs.microsoft.com/en-us/cli/azure/install-azure-cli, and let us know if met any problem. thanks.

Hi @youngezz
I had no problems installing it with the MSI. Just wish all the things would auto update and be as easy to push in a MDM scenario as Microsoft Store (for Business) makes it.


min/max defines collide with C++ on MSVC

These defines:
https://github.com/sctplab/usrsctp/blob/1ade45cbadfd19298d2c47dc538962d4425ad2dd/usrsctplib/user_environment.h#L65-L68
Cause problems with MSVC compiler when included in C++ project that uses std::min/max, because min and max are replaced with define contents.
Here is an example pipeline with such errors: https://github.com/nazar-pc/mediasoup/runs/3152529859?check_suite_focus=true
Would be nice to not define such a generic things in C, the issue is essentially similar to #600

Why don't you compile the library with a c compiler and link your project against it? This stuff is internal to the library...

I guess it is included anyway due to being in a header file or maybe MSVC it insane that way, not sure yet.

Hm, even though it is defined in usrsctp, I suspect it might not be the reason I'm getting that error, must be just because other dependency doesn't include NOMINMAX for MSVC in meson.build, sorry for the false report.


Inconsistent processing of invalid configuration value depending on target hosting plan

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az functionapp config set
Errors:
When provided with invalid value for LinuxFxVersion parameter, command does not fail with error when target hosting plan is Linux Premium.
To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Create a linux premium function app
"FUNCTIONS_WORKER_RUNTIME" = "node",
linuxFxVersion not set explicitly
az functionapp config set --name {} --resource-group {} --linux-fx-version "node:12"

Expected Behavior
Note node:12 is invalid value for LinuxFxVersion.
When target plan is Linux Consumption, the command will fail with Operation returned an invalid status code 'Bad Request' (when running with --debug, more verbose error is {"Code":"BadRequest","Message":"The parameter LinuxFxVersion has an invalid value." ...).
Expected: When target plan is Linux Premium, the command should fail with the same error.
Actual Behavior
When target plan is Linux Premium, the command succeeds. After that, actual node.js version will default to ~10.
Environment Summary
Linux-4.19.121-linuxkit-x86_64-with-debian-10.7
Python 3.6.10
Installer: DEB

azure-cli 2.19.1

Additional Context

functionapp


Add client_kwargs to preparers

Codecov Report

Merging #1416 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1416      +/-   ##
==========================================
+ Coverage   52.61%   52.62%   +<.01%
==========================================
  Files        3299     3299
  Lines       83894    83895       +1
==========================================
+ Hits        44145    44146       +1
  Misses      39749    39749

Impacted Files
Coverage Δ

...-testutils/devtools_testutils/resource_testcase.py
61.29% <ø> (ø)
:arrow_up:

...k-testutils/devtools_testutils/storage_testcase.py
63.88% <ø> (ø)
:arrow_up:

...-sdk-testutils/devtools_testutils/mgmt_testcase.py
90.29% <100%> (+0.09%)
:arrow_up:

Continue to review full report at Codecov.

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


Keyboard shortcuts conflicting with browsers' own

Figure out what to do here. Some sources to review wrt this:

http://stackoverflow.com/questions/3680919/overriding-browsers-keyboard-shortcuts
http://unixpapa.com/js/key.html

We don't want to trample over the shortcuts of the browser and the os. Users should be able to modify these as they see fit.


Removed Compile_first option in tests

Removed Compile_first option in tests
Replaced "spread" function in get_col_dry function in src/rrtmgp/AtmosphericStates.jl

bors r+


Expose summary API for all Apps

I'm the developer of flathub-data-viewer, a little tool which shows basic informations about Flathub. It can parse the Appstream Data directly, since it is just a XML file. But for things like Permissions and BaseApp it relays on the summary API of Flathub. But the API currently only supports Desktop Apps. Would be nice, if you could support all Types of Apps.

Duplicate of https://github.com/flathub/website/issues/53

Btw, some types like plug-ins are not providing summaries at all afaik

We're exposing stats for all apps now, but summaries are still a todo.

How important would platforms and having .Debug, .Locale and .Sources be?

We're now exposing all but .Debug, .Locale, .Sources and Platforms. Platforms are excluded, as they seem to have duplicate keys, which the current library used to parse that ini file format doesn't like.


Allow initializing UIColor with Integers

A convenience initializer for UIColor to use Ints such as 225 instead of floats like 225.0/255.0.

I'm :+1:  on this.

:+1:


Update electrum-server.md

fix link to go back to upgrade section

Good catch! It will be fixed on the next commit. Thanks for your contribution ❤


Use as build-step (and naiv implementation of blocking until write to log finish)

Hi,
Thanks for a useful plugin.
I've created a README file and done a naiv implementation of blocking until write to log has finished.
It simply checks the log for new content with one second interval until nothing new can be found.
I tried your suggested implementation that you had commented out, but it will not work. It causes a deadlock and the jobs freezes until a user aborts it. It is because writeWholeLogTo waits for the job to reach a COMPLETED state, which will never happen since the job is still ongoing and blocked by the plug-in's processing step. Hence my choice of implementation.
I now also added the option of using it as a build-step (in addition to post-build).
I just wanted to offer my implementation back in case you want it.
Thanks again for your contribution to the software community.

Sorry I have neglected this. We are no longer using jenkins so this hasn't been high on my radar. Would you be interested in me transfering the repo to you? or is there some way of promoting a fork? It would probably be easiest if you could just take on ownership if you are interested in the plugin I'm just not quite sure what the best way to do that on github is.

@cboylan , sorry for my late reply
I don't mind taking full ownership of a fork.
I guess the easiest way would be for you to update the README in your version, linking to my fork, explaining that your version is no longer maintained and that this version can be considered the "official" version from now on.

I was working on pushing a README that pointed at your fork, but you don't currently have any public repositories on github. Let me know when the repo is up and I will direct people to it with a new README commit.
Thanks.

https://github.com/devconsoft/console-log-to-workspace-plugin


add mountainsort4 image

I made a cleaned-up package of ml_ms4alg called mountainsort4 which is now on pypi. It's the same algorithm, just simplified packaging and some cleanup of the source code.
This docker image simply pip installs that package.
Here's the spikeforest image, wrapper, and example:
https://github.com/flatironinstitute/spikeforest/blob/main/spikeforest/sorters/mountainsort4/docker/Dockerfile
https://github.com/flatironinstitute/spikeforest/blob/main/spikeforest/sorters/mountainsort4/mountainsort4_wrapper1.py
https://github.com/flatironinstitute/spikeforest/blob/main/devel/sorting/test_mountainsort4.py

@alejoe91 @bendichter @samuelgarcia @jsoules

Well that's pretty simple! Why install numpy before mountainsort? Surely numpy as a requirement of mountainsort, right?

Well that's pretty simple! Why install numpy before mountainsort? Surely numpy as a requirement of mountainsort, right?

I usually don't put numpy in the list of requirements in setup.py because some people have their own system-wide numpy installed and the pip install numpy can sometimes cause problems. Not sure if this is the recommended strategy.
Nevertheless, another reason to do it (although not super important), is that the layer of python:3.8 + (pip install numpy) can be shared across multiple images, and that saves a bit of time.

Hi Jeremy.
Is there a MS4 package on pypi now ?
I was using ml-ms4alg but it was failing on ubuntu 20.4 + py3.8.

mountainsort4 is on pypi. please give it a try and let me know if that is
also failing.
On Wed, Apr 14, 2021 at 9:00 AM Garcia Samuel @.***>
wrote:

Hi Jeremy.
Is there a MS4 package on pypi now ?
I was using ml-ms4alg but it was failing on ubuntu 20.4 + py3.8.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SpikeInterface/spikeinterface-dockerfiles/pull/1#issuecomment-819498894,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4CIQBL34K5KEFPRODTXE3TIWGYXANCNFSM425HMCNA
.

I'll merge this unless anyone has objection.


Brad cleanup clang

Clean up for clang.  Mark a bunch of functions inline so they aren't compiled multiple times in one unit.    Delete some unused variables.

CI gfxreconstruct build queued with queue ID 4511.

CI gfxreconstruct build # 1723 running.

CI gfxreconstruct build queued with queue ID 4540.

CI gfxreconstruct build # 1724 running.

CI gfxreconstruct build queued with queue ID 4552.

CI gfxreconstruct build # 1725 running.

CI gfxreconstruct build # 1725 passed.

CI gfxreconstruct build queued with queue ID 4605.

CI gfxreconstruct build queued with queue ID 4614.

CI gfxreconstruct build # 1727 running.

CI gfxreconstruct build # 1727 passed.

That's a lot of unused variables (maybe parameters too) and functions.
Maybe we should make things a bit more strict and enable things like -Wall -Wextra etc?
Making the compiler stricter, although some times bothersome, can be beneficial in the long run

Maybe we should make things a bit more strict and enable things like -Wall -Wextra etc? Making the compiler stricter, although some times bothersome, can be beneficial in the long run

Makes sense. Perhaps we could do this for internal builds without imposing it on users who may be compiling on compilers/compiler versions we haven't used and would thus have to deal with warnings as errors that we haven't seen.

Maybe we should make things a bit more strict and enable things like -Wall -Wextra etc? Making the compiler stricter, although some times bothersome, can be beneficial in the long run

Makes sense. Perhaps we could do this for internal builds without imposing it on users who may be compiling on compilers/compiler versions we haven't used and would thus have to deal with warnings as errors that we haven't seen.

I thought we already were compiling with -Wall because of the number of times I've been scolded by the compiler!  But we are using -Werror, so -Wall could be painful.  Still, if the team agrees to it, I also think it would be a good idea.

In general I think we should be more strict, passing options equivalent to -Wall and -Wpedantic.  But this PR was extracted from a MacOS PR and is too specific.  Closing it now in favor of a later pass of making the whole code base more strict.


Multiline strings in mutation error

Hello,
I have an issue whereby sending a multiline string in a mutation will return a difficult to understand error message.
I have reproduced this issue on the starwars example in this repo.
Graphql mutation(not using block strings) that will reproduce the error:
mutation {
  createReview(episode:NEWHOPE, review: {stars: 0, commentary: "1
    2"}) {

  }
}

Expected: Error message indicating that a multiline/un-terminated string was found in the query.
Actual:
{
  "errors": [
    {
      "message": "syntax error: unexpected \"2\", expecting Ident",
      "locations": [
        {
          "line": 32,
          "column": 5
        }
      ]
    }
  ]
}

Log output: <input>:31:64: literal not terminated

Hi Fraser,
I finally got time to look into this. Indeed it would be nice to improve the error message for this as it's not very user-friendly.


Improve relations types

Hi,
This package is wonderful, as it can avoid a lot of queries. However, it would be useful to have intellisense suggestions when typing relation names in selectRelated method.
I tried to add the types, and it worked. However, it doesn't work when you want to use nested relationships (which is obvious).
This is what I did:
interface SelectRelatedQueryMethods<
    Model extends LucidModel,
    Result = InstanceType<Model>,
    Name = ExtractModelRelations<InstanceType<Model>>
> {
    selectRelated(
        name: Name,
        options?: SelectRelatedOptions
    ): ModelQueryBuilderContract<Model, Result>
    $sideloadedRelations?: SideloadedRelations<Model>
}

Can we somehow split the selectRelated into 2 methods: selectRelated (for one relationship) and selectNestedRelated (for nested relationships)? Or find a way to not throw an error if a nested relationship is passed as parameter?
Thank you!

Hi,
I'll try to create a type to provide intellisense for nested relations as well but the below code should work in the meantime. Could you test it out & see if it works for you? If it does then you can create a PR and I'll be very happy to merge it 😁
interface SelectRelatedMethods<Model extends LucidModel, Result = InstanceType<Model>> {
        selectRelated<Name extends ExtractModelRelations<InstanceType<Model>>>(name: Name, options?: SelectRelatedOptions): ModelQueryBuilderContract<Model, Result>;
        selectRelated(name: string, options?: SelectRelatedOptions): ModelQueryBuilderContract<Model, Result>;
        $sideloadedRelations?: SideloadedRelations<Model>;
    }

Hey!
Overloading is also an option, yeah, thank you! It works, I'll create a PR for this.


[BUG] Multiple service linting problem

Describe the problem
Terraform could manage multiple services like:
resource "fastly_service_vcl" "service1" {
  ...
}

resource "fastly_service_vcl" "service2" {
  ...
}

And falco could run lint for each service.
But then, the remote option only fetches remote resources from one of the services, which causes error due to lint with other resources.
The main problem is that only accept remote service id could be provided via FASTLY_SERVICE_ID environment variable,  to fix it, we need to fetch remote resources for each lint service when falco runs for terraform .
VCL code that cause the problem
N/A
Screenshots
N/A
Expected behavior
Run lint properly for each service for terraform planned result
Desktop (please complete the following information):
Additional context

Usually, we cannot find service_id from terraform planned result (it's written in terraform state but falco does not know what it), so need to retrieve service_id through the Fastly API.
https://developer.fastly.com/reference/api/services/service/#search-service This API would be useful for getting service_id from service_name and fetching remote resources by using that API result for each service.
Normally falco expects to lint for the single service so this feature should have only for terraform subcommand.

Closes with https://github.com/ysugimoto/falco/pull/190


[openblas] update to 0.3.24

[X] Changes comply with the maintainer guide
[X] SHA512s are updated for each updated download
[ ] ~The "supports" clause reflects platforms that may be fixed by this new version~
[ ] ~Any fixed CI baseline entries are removed from that file.~
[ ] ~Any patches that are no longer applied are deleted from the port's directory.~
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

@jiayuehua, Thanks for your PR, this is ci error log, please take a look:
CMake Error at D:/installed/x64-windows/share/vcpkg-cmake-config/vcpkg_cmake_config_fixup.cmake:36 (message):
  'D:/packages/openblas_x64-windows/debug/share/cmake/OpenBLAS' does not
  exist.

All feature tested successfully in x64-linux triplet.


Wrong README on npmjs.com

On npmjs.com, the "lower" option is printed as "lowercase":
print(slug('I ♥ UNICODE', {lowercase: false}))

Also, it's talked about as if it were the default, which it is not (anymore)

I wasted 30 minutes wondering what I did wrong due to the incorrect documentation :-(

npmjs.com should be picking up whatever readme the latest release has. It might be older than what's on github master which is why I usually go strait to github to view readmes


Reintroduce a way to silence individual warnings

Some warnings created by this action can be quite noisy, and there does not seem to be a way to silence certain error classes.
It seems that in #16 this feature was removed.
The Pinned-Dependencies check creates many warnings for GitHub Actions if they are not pinned by hash. Every update to the pinned tag will re-surface the same line in the Security Dashboard, even if it was previously selected as "won't fix". This creates unnecessary many alarms and due to the sheer number of "uses" in actions is by far the largest contributor of warnings.
All of GitHub's documentation also uses the form actions/checkout@v3, so this check conflicts with that. Instead, this is recommended actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f # v2.3.4. The problem here is that the comment will not get updated by tools such as dependabot and thus does not contribute meaningfully, even is harmful as it suggests a different version than actually used.
I can see why somebody would want this check, but for my projects I do not see this as a benefit and would like to disable the check.

Thanks for the report, @jonasbb
re: GitHub doc conflict with hash pinning. We're aware that dependabot has this issue :/ Note, though, that GitHub doc does recommend pinning by hash, see https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#using-third-party-actions but I don't think their workflow examples use it consistently :(
I'm also surprised the Won't Fix does not silence the alerts, but I suppose it's because they see a code change and do not take into account the fact that the only change was via a dependabot PR. I'll create an issue on their repo for this feature. DO you think it would help?

I'm also surprised the Won't Fix does not silence the alerts,

Maybe I was a bit unclear on that point. Let me expand on that. I can select "Won't Fix" and the alert will be silenced until the offending line changes. Next dependabot will create a PR updating all actions/checkout@v2 to actions/checkout@v3. This closes all code scanning alerts for the old lines and creates a new code scanning alert for each uses: actions/checkout@v3 line.
I don't really know if a change like actions/checkout@v2 to actions/checkout@v3 should create a new code scanning alert  or simply update the old one. However, since it creates a new alert, this feels quite invasive to me and forces me to apply "Won't Fix" to all actions/checkout@v3 alerts. An action like checkout can be used many times, thus drown out other alerts.
Alerts for other action lines are not impacted.

Note, though, that GitHub doc does recommend pinning by hash, see docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#using-third-party-actions but I don't think their workflow examples use it consistently :(

I understand the recommendation from a security perspective. For the small project where I am testing the action using hashes does not feel ergonomic enough. GitHub's docs heavily favor tags though, like on docs.github.com, and the action descriptions. I don't mind the alerts appearing by default, but being able to silence them would be great.

I'm also surprised the Won't Fix does not silence the alerts,

Maybe I was a bit unclear on that point. Let me expand on that. I can select "Won't Fix" and the alert will be silenced until the offending line changes. Next dependabot will create a PR updating all actions/checkout@v2 to actions/checkout@v3. This closes all code scanning alerts for the old lines and creates a new code scanning alert for each uses: actions/checkout@v3 line.
I don't really know if a change like actions/checkout@v2 to actions/checkout@v3 should create a new code scanning alert or simply update the old one. However, since it creates a new alert, this feels quite invasive to me and forces me to apply "Won't Fix" to all actions/checkout@v3 alerts. An action like checkout can be used many times, thus drown out other alerts.

it was clear. I created https://github.com/github/codeql-action/issues/986 to see if code-scanning platform can improve the way they show alerts.

GitHub's docs heavily favor tags though, like on docs.github.com, and the action descriptions.

Tags are favoured for first-party actions (i.e. those under the github or actions orgs) because you can rely on GitHub not to introduce malicious code into them. If you don't trust GitHub, then using the Actions platform as a whole becomes difficult.
For third-party actions pinning to the sha is recommended.

See https://github.com/github/codeql-action/issues/986#issuecomment-1171269954 for possible fix

@laurentsimon Fixing the “wrong” fingerprints, as mentioned in the linked issue, would definitely lessen the impact of the Pinned-Dependencies check. I do not see this as a fix to this issue, but rather a related problem. The issue here is rather that scorecard has decided to enforce certain policies, which, I feel, are too much of a burden to uphold. As such, constantly working around the security reports has the same downside, and the solution is to not use the action.
Even with a fixed fingerprint, the action will still warn on every newly introduced unpinned dependency or potentially more often. It really depends on the fingerprint.

I don't disagree. We created https://github.com/ossf/scorecard-action/issues/729 to discuss customization of the Action.
Can you share your thought in the issue?
/cc @raghavkaul


Keyboard shortcut blocking

Currently, MusicBox web interface blocks all keyboard shortcuts (like ctrl+tab, alt+w etc).
Expected behaviour would be only blocking these keyboard actions, that MusicBox overrides (ie space for pause/play).
Fix:
@mopidy_musicbox_webclient/static/js/gui.js:509
$(document).keypress( function (event) {
    //console.log('kp: '+event);
    if (event.target.tagName != 'INPUT') {
-       event.preventDefault();
        switch(event.which) {
            case 32:
                doPlay();
+               event.preventDefault();
                break;
            case '>':
                doNext();
+               event.preventDefault();
                break;
            case '<':
                doPrevious();
+               event.preventDefault();
                break;
        }
    return true;
    }
});

This must be browser specific as those shortcuts work perfectly for me using Chrome Version 38.0.2125.111 (64-bit) on Linux Mint.

As far as I understand the specification, preventDefault() should prevent all actions on particular event.
It is indeed browser specific, but in a reverse-ish manner: Chrome simply blocks preventDefault() on some keyboard events (see this).
I'm using Firefox (according to JS compliancy pages, version and OS are irrelevant), but other browsers like Safari/IE/etc can possibly have this issue too.

Fair enough, just wanted to highlight the fact that verifying any fix would require a particular browser.
If you can submit a pull request then it's nice and easy for @woutervanwijk

Fixed in https://github.com/woutervanwijk/Mopidy-MusicBox-Webclient/commit/13c024464388fed85d060edbfe746926f1353a37


Type inference failure with function types with type arguments

The easiest way to explain this is with some code:
R callWithSomeTypeArgument<R>(R Function<T>() function) => function<String>();

void main() {
  // The argument type 'List<E> Function<E>({bool growable})' can't
  // be assigned to the parameter type 'List<Z0> Function<T>()'.
  final test = callWithSomeTypeArgument(List.empty);
  dynamic test2 = callWithSomeTypeArgument(List.empty);

  // All OK!
  List<dynamic> test3 = callWithSomeTypeArgument(List.empty);
}

This should work - I think - with the type of test being inferred as List<dynamic> since we have no information on what T will be. Instead it fails and the error message contains a reference to some Z0 type which is I assume generated for type inference.

@stereotype441
Is this working as intended?
I'm guessing we treat no type and dynamic as imposing no context type, so R has to be found by upwards inference. We then try to solve List<E> Function<E>({...}) <: R Function<T>() for R and fail because we don't just want to guess Object? as a solution, and there is no List<X> that works.

Doesn't List work here? We know the function returns a List because the return type is List but we can't guess E. I feel that guessing dynamic for E is then ok in this case.

True, List<Object?> is also a solution. So maybe it's because the inference decides on List<T>, then figures out that it doesn't actually know T and gives up? (But now I'm just guessing.)

After some more testing, I have found out that this issue only occurs in the analyzer. The frontend correctly infers the type argument and reports no errors, so commands like dart compile work fine - dart analyze continues to report the error though.

A bit more investigation and the issue seems to arise here. This line incorrectly gathers that R should be a supertype of List<Z0> when it should gather that R is a supertype of List<E>, as it substitues Z0 for E in the check.
I'm not sure if this is a bug in the analyzer implementation or an issue in the spec itself.


fix stop execution call without execution name

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

Hi @yanzhudd @zhoxing-ms
Could you please help to re-run this check in pipeline? And please help to review this PR, thanks.


Update slice syntax to fix elixir 1.16 warning

Related to https://github.com/kim-company/membrane_mpeg_ts_plugin/pull/6

Thanks!


CPU - Randle | Randle Blog

https://junqianghan.github.io/2019/12/20/knowledge-about-cpu/

20200103-01
20200103-02


Httpserver terminates unexpectedly

Hi,
I want to use TableExample.js to create my own queries to a BaseX server running on my localhost. I can start the BaseX server and run nodejs TableExample.js, but when I try to navigate to localhost:9000 in the web browser I get a server not found error and on the command line the following happens:
Server running at port:9000
http.js:853
throw new TypeError('first argument must be a string or Buffer');
^
TypeError: first argument must be a string or Buffer
at ServerResponse.OutgoingMessage.write (http.js:853:11)
at Object.callback (/home/werner/Desktop/Nodebin/BaseX/node_modules/basex/examples/TableExample.js:33:7)
at onData (/home/werner/Desktop/Nodebin/BaseX/node_modules/basex/index.js:150:20)
at Socket. (/home/werner/Desktop/Nodebin/BaseX/node_modules/basex/index.js:97:18)
at Socket.EventEmitter.emit (events.js:95:17)
at flow (_stream_readable.js:596:9)
at Socket.pipeOnReadable (stream_readable.js:624:5)
at Socket.EventEmitter.emit (events.js:92:17)
at emitReadable (_stream_readable.js:408:10)
at emitReadable (_stream_readable.js:404:5)
werner@werner-Inspiron-3543:~/Desktop/Nodebin/BaseX/node_modules/basex/examples$

Hi I am travelling at the moment but hope to look at this in the next few days. Which version of node are you using?

I'm using v0.10.25 but I had to create a node symlink using sudo ln -s /usr/bin/nodejs /usr/bin/node to be able to use node --version instead of nodejs --version See this discussion: https://www.digitalocean.com/community/questions/ubuntu-repo-nodejs-require-symlink-to-node-to-use-forever

I have had quick look and I think this example was never finished. It is a
bit of a mess. I think it was intended to be a conversion of a basex php
example. I will look at fixing it when I am home next week.
On 19 Jun 2015 08:24, "Werner0"<EMAIL_ADDRESS>wrote:

I'm using v0.10.25 but I had to create a node symlink using sudo ln -s
/usr/bin/nodejs /usr/bin/node to be able to use node --version instead
of nodejs --version See this discussion:
https://www.digitalocean.com/community/questions/ubuntu-repo-nodejs-require-symlink-to-node-to-use-forever
—
Reply to this email directly or view it on GitHub
https://github.com/apb2006/basex-node/issues/19#issuecomment-113406049.


[BUG] Unable to download using Real Debrid

Describe the bug
I am unable to download using Real Debrid. It gets stuck at "Starting download".
Steps to Reproduce
1 - Enable Real Debrid
2 - Insert the token
3 - Download the game
Expected behavior
Download the game using Real Debrid
Screenshots

Operating System
Windows 10
Hydra Version
1.2.4 Exodus
Additional Information
No response

Is this still an issue on 2.0.2?


Showing summary stats of all features in facets_overview

In facets_overview in Jupyter notebook, the HTML view shows only few features, how can I make the HTML output show all features in the visualization? I have 100s of features, I want to show all of them in the HTML output.
Thank you!

Do you mean that the visualization only shows the first few and the rest need to be scrolled through to be displayed? Or is the visualization somehow not picking up some of the features in your dataset at all?
If it is the first, the only way to remove the cap on the size of the containers holding the lists of feature statistics is by changing the _maxHeight value (currently hardcoded to 800 px) in facets_overview/components/facets_overview_table/facets-overview-table.ts and rebuilding facets.
If it is the second, could you provide more information on your dataset and what features are failing to appear?
Thanks.


Update Dockerfile

Switch from alpine:edge to node:7.10-alpine image
Install npm packages and setup offline mode first
Drop down to less privilege user by default

The nodejs alpine package no longer includes npm. While I was fixing that issue I made some additional updates to the Dockerfile following Code Climate engine best practices.

@nlf Anything I can do to push this along?


decouple BinaryFiberId from parquet seekable input stream

What changes were proposed in this pull request?
decouple BinaryFiberId from parquet seekable input stream， make it more general. This pr fix https://github.com/Intel-bigdata/OAP/issues/1129
modify input stream type of BinaryFiberId as FsDataInputStream;
make LazyInitSeekableInputStream implements Seekable and PositionedReadable so that it can be wrappped as FsDataInputStream;
How was this patch tested?
existing ut.

cc @LuciferYang @zhixingheyi-tian

@ivoson I think we should hold this patch until #1131 merged.
cc @zhixingheyi-tian I think we should merge #1131 related patch first, then refactor the BinaryFiberId uniformly.


get windows document folder from UserDirs

The current implementation uses My Documents as library windows path. This is old (windows XP) afaik. Openscad uses SHGetFolderPathW to get the documents folder from windows. I tried to recreate this behavior in rust.
I did not do a lot with rust, so correct me if this does not make sense at all.

Sorry, this code can't compile. Do you have a compiling environment?

finish with https://github.com/Leathong/openscad-LSP/commit/a7f4667bf62f8911a2641c9e834f0796d2a4ef79

Hi, thanks for fixing the issue.
I see that I did not push my latest changes. Your solution looks way better.


Upgrade to laravel 5.3

Any plans to upgrade to Laravel 5.3?

there is, it's just that i got caught up with tons of work loads lately and had no time to update, any PR for this would be very much appreciated

which version of angular used in this project?

With 5.5 on the horizon whats the plan for updating to 5.3/5.4/5.5?


Fix X button in map Marker containing an image (#600)

Hi! This is a fix for #600. I tested it for markers with and without images, both long and short names (issue exists for short names only).

same here. Thanks!


static files not being served

Hi, I can't get static files to be served, no matter what path I try, is there anything that isn't in documentation I should be aware of to get it working?

It should work. On what OS are you? Does the stdweb's todomvc example work for you?

Hey, I'm on osx, stdweb example works for me but I just realized that I probably misunderstood the line "will be served alongside your application", I was trying to use File::open() with files in the static directory, I guess I have to use a http request to the local file instead?

Yes, that's correct; they will be served by the HTTP server. You can either make an HTTP request from your code (either by using raw AJAX calls from inside an js! call as we don't have bindings for that yet, or maybe possibly with stdweb-io) or you can move those files somewhere else in your source tree and use include_bytes! or include_str.
Technically Emscripten supports hooking into File::open, but there are no plans to support that. (In general I'd like to scrap Emscripten completely once Rust's native wasm backend matures.)

Thanks for the info. Will check out stdweb-io. And thanks for cargo-web in general it's very practical to use.


Does it still support iOS 7?

Does it still support iOS 7?

@ch8908
Maybe, it works on iOS7.
But, You need add source code to your project directly.
Because, iOS7 does not support dynamic library.


QA Test Data - DTO Checks

The scope of this ticket is to address the following DTO Checks:
QA and Certification Data Checks

[x] SEVNDAY-28 - 7-Day Calibration Test Result Code Valid
[x] SEVNDAY-30 - 7-Day Calibration Test Component ID Valid
[x] APPE-45 - Gas Monitoring System ID Valid
[x] APPE-47 - Appendix E Monitoring System ID Valid
[x] CYCLE-21 - Cycle Time Injection Gas Level Code Valid
[x] CYCLE-22 - Component ID Valid

SEVNDAY-28 - 7-Day Calibration Test Result Code Valid

SEVNDAY-28 - 7-Day Calibration Test Result Code Valid

SEVNDAY-30 - 7-Day Calibration Test Component ID Validations
SEVNDAY-30-A Import

SEVNDAY-30-A POST

SEVNDAY-30-A EDIT

7DAY  28 and 30
Test Case: TC1213
Test Passed

APPE-47 - Appendix E Monitoring System ID Valid
APPE-47-A Import

APPE-47-A POST

APPE-47-A EDIT

APPE-45 - Gas Monitoring System ID Valid
APPE-45-A Import

APPE-45-A POST

APPE-45-A EDIT

APPE-45-A
Test Case: TC1214
Test Passed

CYCLE-21 - Cycle Time Injection Gas Level Code Valid
CYCLE-21-A POST

CYCLE-21-A EDIT

CYCLE-21-A Import

CYCLE-21-B Import
The Import is not working properly so I cannot confirm this one. I have no reason to believe it does not work as all other are functioning properly.

CYCLE-21 - Cycle Time Injection Gas Level Code Valid
CYCLE-21-A POST

CYCLE-21-A EDIT

CYCLE-21-A Import

CYCLE-21-B Import
The Import is not working properly so I cannot confirm this one. I have no reason to believe it does not work as all other are functioning properly.

CYCLE-21-B Import
The Import is not working properly so I cannot confirm this one. I have no reason to believe it does not work as all other are functioning properly.

CYCLE-22 - Component ID Valid
CYCLE-22-A Import

CYCLE-22-A POST

CYCLE-22-A EDIT

Test Case: TC1215
Test Passed


Add files via upload

@DatXinhGai sis.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Update to Jenkins 1.596.3

Failed to verify the image built. Missed the SHA1 verification inside the Dockerfile. Sorry about that distraction.


leap: Generate missing exercise README

In preparation for v2 we're updating Configlet—our Exercism track linter—to require that we generate the exercise README. This is because in v2 we will not be generating READMEs on the fly, but deliver the README as defined in the directory for each implementation.

I'm going to go ahead and merge this as soon as the build passes, as this is janitorial work rather than a language-specific change.


[Feature Request] Different way to complete with tab (YCM style)

Before I start
Hello.
VS Code is great software, and i really like how it grows over time, so I want to improve the experience of using it.
Previously I used Vim a lot, because at work it is my only option, but at home for hobbies I use VS Code, and when I program in C, at my work I use YouCompleteMe (YCM) plugin for tab completions, and I really like the way it works.
Touch typing
The main thing is that I use all my 10 fingers to write code, and rarelly move hands from home row, because vim, you know, has hjkl and other ways to navigate in code. So first thing was to install vim-extension to VS Code, to stop using arrow keys, and it works well. But tab completion still forces me to use arrows.
How it works in Vim: I select from the list using the tab button and I confirm the completion by simply continuing typing. Therefore, my hands never leave the house.
The way it works in VS Code: tab button completes the completion and closes popup, And arrow keys are just navigate within the list and not confirm anything. So I need to navigate in list with arrow keys (and leave home row) then press tab for completion. In fact it is way faster just to type most things by hand just looking at the popup for reference. And it is frustrating after long way of programming in vim.
So what do you want?
All I want is an option in settings to change tab key behavior to something like (or close to) YCM style.
The behavior itself can be described like so:

Start typing (std:: for example)
Completion popup is displayed (or invoked by tab)
Press the tab and completion is done, but popup isn't closed
Press tab again, and next item in popup list replaces previous completion
Press tab until you will be satisfied with your completed text
Continue typing, popup is closed (unless you folow the completion hint from popup)

That's how (mostly) it works in such plugins like YouCompleteMe, Supertab in Vim. You can try this plugins for better understanding of what i mean. And it is comfortable because you don't need arrow keys at all.
P.S:
Previously (before Vim) I used Visual Studio for C++ programming, and when I swithced to YCM's completion system I noticed a great boost of writing speed. YCM has it's own plugin in the marketplace, but it uses standard tab behavior, which makes it little less useful an makes me little less productive.
Thanks.

cc @jrieken and @alexandrudima

Duplicate of https://github.com/Microsoft/vscode/issues/28215


Add Carthage compatibility

I had to move files in order for Carthage to find the project file.
After merging you should tag a release in master to my latest commit so Carthage recognises it.
Thanks for your outstanding work, it was really easy to integrate into my app.

I'll be working on the project again and add Carthage support soon. Will close the PR for now, but thanks for submitting it!


Multiline suppor

Needed multiline support, like this:
  <!-- include: "type": "js",
                      "files": "modules/*/*.js,
                                  modules/**/*.js,
                                  !**/tests/*.js" -->

Hi @zxxc, can you post your Gruntfile? Not sure where the <!-- include: --> is coming from?

Now I'm thinking that I was trying to post to another plugin, sorry for wasting your time


Install from source or w/o conda

Hello,
I want to install this on our local clusterbut we do not use conda as installer.  Can this be installed without conda.

Thanks for your interest :)
What happens if you try pip install epic2? I never have tried installing it without conda, but I think it is 99.999% it will fail. But I would be interested in seeing your error message :)
I will try to find a fix for this :)

pip3.6 install epic2
Collecting epic2
Using cached https://files.pythonhosted.org/packages/48/a6/add5742fc7b644a972288753902da44725c76cdc606575330d627fd6e7eb/epic2-0.0.30.tar.gz
Complete output from command python setup.py egg_info:
which: no conda in (/cm/shared/apps/python/3.6.2/bin:/cm/local/apps/cluster-tools/bin:/cm/local/apps/cmd/sbin:/cm/local/apps/cmd/bin:/cm/shared/apps/cmgui:/cm/shared/apps/slurm/16.05.8/sbin:/cm/shared/apps/slurm/16.05.8/bin:/sbin:/bin:/usr/sbin:/usr/bin:/cm/local/apps/environment-modules/3.2.10/bin:/opt/dell/srvadmin/bin:/opt/dell/srvadmin/sbin)
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-install-ol6jxb58/epic2/setup.py", line 42, in
conda_path = check_output("which conda", shell=True).decode().strip()
File "/cm/shared/apps/python/3.6.2/lib/python3.6/subprocess.py", line 336, in check_output
**kwargs).stdout
File "/cm/shared/apps/python/3.6.2/lib/python3.6/subprocess.py", line 418, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command 'which conda' returned non-zero exit status 1.
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-ol6jxb58/epic2/

Try pip install epic2==0.0.32. This is not a fix, but should give a more informative error message that I would like you to post here :)

pip3.6 install epic2==0.0.32
Collecting epic2==0.0.32
Could not find a version that satisfies the requirement epic2==0.0.32 (from versions: 0.0.1, 0.0.3, 0.0.4, 0.0.5, 0.0.6, 0.0.7, 0.0.8, 0.0.9, 0.0.10, 0.0.11, 0.0.12, 0.0.13, 0.0.14, 0.0.15, 0.0.16, 0.0.17, 0.0.18, 0.0.19, 0.0.20, 0.0.21, 0.0.22, 0.0.23, 0.0.24, 0.0.25, 0.0.26, 0.0.27, 0.0.28, 0.0.29, 0.0.30)
No matching distribution found for epic2==0.0.32

Ah, you were too quick. It should be there in a second :)
On Tue, Apr 23, 2019 at 4:22 PM whositwhatnow<EMAIL_ADDRESS>wrote:

pip3.6 install epic2==0.0.32
Collecting epic2==0.0.32
Could not find a version that satisfies the requirement epic2==0.0.32
(from versions: 0.0.1, 0.0.3, 0.0.4, 0.0.5, 0.0.6, 0.0.7, 0.0.8, 0.0.9,
0.0.10, 0.0.11, 0.0.12, 0.0.13, 0.0.14, 0.0.15, 0.0.16, 0.0.17, 0.0.18,
0.0.19, 0.0.20, 0.0.21, 0.0.22, 0.0.23, 0.0.24, 0.0.25, 0.0.26, 0.0.27,
0.0.28, 0.0.29, 0.0.30)
No matching distribution found for epic2==0.0.32
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/biocore-ntnu/epic2/issues/17#issuecomment-485824294,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEHURUVW6JSFUC4J2WNPRFTPR4LRFANCNFSM4HHYJ3JQ
.

Looks to have installed no errors.
pip3.6 install epic2==0.0.32
Collecting epic2==0.0.32
Using cached https://files.pythonhosted.org/packages/70/a8/f488c6b626656933686999d60517b83b1e8828d7369c786acd2304b8d7e6/epic2-0.0.32.tar.gz
Requirement already satisfied: scipy in /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages (from epic2==0.0.32) (0.19.1)
Requirement already satisfied: numpy in /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages (from epic2==0.0.32) (1.14.2)
Collecting natsort (from epic2==0.0.32)
Using cached https://files.pythonhosted.org/packages/e7/13/a66bfa0ebf00e17778ca0319d081be686a33384d1f612fc8e0fc542ac5d8/natsort-6.0.0-py2.py3-none-any.whl
Requirement already satisfied: cython in /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages (from epic2==0.0.32) (0.29.6)
Requirement already satisfied: pysam in /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages (from epic2==0.0.32) (0.15.1)
Collecting pandas (from epic2==0.0.32)
Downloading https://files.pythonhosted.org/packages/19/74/e50234bc82c553fecdbd566d8650801e3fe2d6d8c8d940638e3d8a7c5522/pandas-0.24.2-cp36-cp36m-manylinux1_x86_64.whl (10.1MB)
100% |████████████████████████████████| 10.1MB 4.7MB/s
Collecting pyranges>=0.0.27 (from epic2==0.0.32)
Downloading https://files.pythonhosted.org/packages/29/76/233d855872bc916b8549b26c505c7fdadfb57e7f824c739fa093fe37fcec/pyranges-0.0.30.tar.gz (431kB)
100% |████████████████████████████████| 440kB 52.8MB/s
Requirement already satisfied: python-dateutil>=2.5.0 in /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages (from pandas->epic2==0.0.32) (2.6.1)
Requirement already satisfied: pytz>=2011k in /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages (from pandas->epic2==0.0.32) (2017.2)
Collecting ncls (from pyranges>=0.0.27->epic2==0.0.32)
Downloading https://files.pythonhosted.org/packages/f7/08/156506261f4c3426191dc4fb8bbc45f9657067a2458e6aea588bc264ae07/ncls-0.0.42.tar.gz (621kB)
100% |████████████████████████████████| 624kB 41.6MB/s
Collecting tabulate (from pyranges>=0.0.27->epic2==0.0.32)
Downloading https://files.pythonhosted.org/packages/c2/fd/202954b3f0eb896c53b7b6f07390851b1fd2ca84aa95880d7ae4f434c4ac/tabulate-0.8.3.tar.gz (46kB)
100% |████████████████████████████████| 51kB 39.2MB/s
Collecting sorted_nearest (from pyranges>=0.0.27->epic2==0.0.32)
Downloading https://files.pythonhosted.org/packages/af/5b/4fb6f94b513551e755aaa19dabfd1816ba5d2c5af4cb1e1c78729bd75f69/sorted_nearest-0.0.15-cp36-cp36m-manylinux1_x86_64.whl (493kB)
100% |████████████████████████████████| 501kB 49.1MB/s
Collecting pyrle (from pyranges>=0.0.27->epic2==0.0.32)
Downloading https://files.pythonhosted.org/packages/a7/80/dc4653b55dfe775b15aa3943d957d6140b87e28e43f7a0f041e8bbe1e8c4/pyrle-0.0.22-cp36-cp36m-manylinux1_x86_64.whl (1.3MB)
100% |████████████████████████████████| 1.3MB 33.9MB/s
Collecting bamread (from pyranges>=0.0.27->epic2==0.0.32)
Downloading https://files.pythonhosted.org/packages/bb/d3/6348db074b4be772e728b79e4235cb0e5af02ead7762e91c694bf49ee497/bamread-0.0.3.tar.gz (108kB)
100% |████████████████████████████████| 112kB 56.7MB/s
Requirement already satisfied: pybigwig in /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages (from pyranges>=0.0.27->epic2==0.0.32) (0.3.13)
Collecting piedpiper (from pyranges>=0.0.27->epic2==0.0.32)
Downloading https://files.pythonhosted.org/packages/96/a9/8b5c0107447c8cbf75e18afd3232fbb71f150614db01bdbf792b8d92625b/piedpiper-0.0.2.tar.gz
Requirement already satisfied: six>=1.5 in /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages (from python-dateutil>=2.5.0->pandas->epic2==0.0.32) (1.11.0)
Installing collected packages: natsort, pandas, ncls, tabulate, sorted-nearest, pyrle, bamread, piedpiper, pyranges, epic2
Running setup.py install for ncls ... done
Running setup.py install for tabulate ... done
Running setup.py install for bamread ... done
Running setup.py install for piedpiper ... done
Running setup.py install for pyranges ... done
Running setup.py install for epic2 ... done
Successfully installed bamread-0.0.3 epic2-0.0.32 natsort-6.0.0 ncls-0.0.42 pandas-0.24.2 piedpiper-0.0.2 pyranges-0.0.30 pyrle-0.0.22 sorted-nearest-0.0.15 tabulate-0.8.3

Weird.
If you do
epic2 -ex
you should see an example command you can test.

Does it work?

Does epic2 work with bam files?

epic2 -t /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages/epic2/examples/test.bed.gz -c /cm/shared/apps/python/3.6.2/lib/python3.6/site-packages/epic2/examples/control.bed.gz > deleteme.txt
Found a median readlength of 25.0
Using genome hg19.
Using an effective genome length of ~2510 * 1e6
Parsing ChIP file(s):
/cm/shared/apps/python/3.6.2/lib/python3.6/site-packages/epic2/examples/test.bed.gz
Valid ChIP reads: 9904 (9924 before out of bounds removal)
Score threshold: 7.146
Number of tags in a window: 1
Number of islands found: 166
Parsing Input file(s):
/cm/shared/apps/python/3.6.2/lib/python3.6/site-packages/epic2/examples/control.bed.gz
Valid Background reads: 9290 (9310 before out of bounds removal)

From: Endre Bakken Stovner<EMAIL_ADDRESS>Sent: Tuesday, April 23, 2019 10:35 AM
To: biocore-ntnu/epic2
Cc: Brian Novogradac; Author
Subject: Re: [biocore-ntnu/epic2] Install from source or w/o conda (#17)
Weird.
If you do
epic2 -ex
you should see an example command you can test.

Does it work?

Does epic2 work with bam files?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/biocore-ntnu/epic2/issues/17#issuecomment-485830103, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFII5HDW3JXYL3KYHLF2CZ3PR4NDPANCNFSM4HHYJ3JQ.

I’m going to test with bam files shortly.
Email sent from my phone.

From: Endre Bakken Stovner<EMAIL_ADDRESS>Sent: Tuesday, April 23, 2019 10:35:35 AM
To: biocore-ntnu/epic2
Cc: Brian Novogradac; Author
Subject: Re: [biocore-ntnu/epic2] Install from source or w/o conda (#17)
Weird.
If you do
epic2 -ex
you should see an example command you can test.

Does it work?

Does epic2 work with bam files?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/biocore-ntnu/epic2/issues/17#issuecomment-485830103, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFII5HDW3JXYL3KYHLF2CZ3PR4NDPANCNFSM4HHYJ3JQ.

I am building wheels (prebuilt binaries for Python) from now on anyways, so then it should be guaranteed to work using only pip :)

No need to check bam for my sake, but tell me if you encounter any weirdness.

Ok excellent thanks for the quick response / help!!
Email sent from my phone.

From: Endre Bakken Stovner<EMAIL_ADDRESS>Sent: Tuesday, April 23, 2019 10:58:35 AM
To: biocore-ntnu/epic2
Cc: Brian Novogradac; Author
Subject: Re: [biocore-ntnu/epic2] Install from source or w/o conda (#17)
No need to check bam for my sake, but tell me if you encounter any weirdness.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/biocore-ntnu/epic2/issues/17#issuecomment-485840154, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFII5HCQ4MB2URIO6BBBENLPR4PZXANCNFSM4HHYJ3JQ.


OpenXR include directory included as PRIVATE, should be PUBLIC

https://github.com/KhronosGroup/OpenXR-SDK-Source/blob/51cade1763267c750f53cefa9834644c7e7b37f3/src/loader/CMakeLists.txt#L91
If the openxr include directory were included as PUBLIC rather than PRIVATE for the openxr_loader target, it would simplify the process of importing that target using modern cmake's FetchContent. PUBLIC includes of a target are automatically included for any targets linking against that target.
Currently I have to manually include the directory instead.
Ref this thread where we encountered the issue.

Oh, we fixed it for install interface but not build interface:  https://github.com/KhronosGroup/OpenXR-SDK-Source/commit/6abd9ccc8641c9b7190687ef68e7d42e1615f5ce


按照教程，可能因ffmpeg依赖缺失导致whisper.load_audio()失败，请求更新requirements.txt

LOCAL.md中imgeio.ffmpeg对我的环境不适用(WIN 10)
Warning: no short audios found, this IS expected if you have only uploaded long audios, videos or video links.
this IS NOT expected if you have uploaded a zip file of short audios. Please check your file structure or make sure your audio language is supported.

whisper.load_audio()
Exception has occurred: FileNotFoundError
单独conda install ffmpeg后解决，建议将ffmepg加入requirement.txt中。

臣附议


FIx bug when MOUNTDIR has heading '\n'

Fix the bug described in https://github.com/create-dmg/create-dmg/issues/169

https://github.com/create-dmg/create-dmg/releases/tag/v1.2.2
https://github.com/Homebrew/homebrew-core/pull/171020


Using hashCode method instead of DimensionType

DimensionType could be the same for modded worlds, fix #1303

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


Table detailed enhancements:

Optional preselected opened details via prop: detailedRows.
Adds detailKey (required when detailedRows is set).
Fixes the detail row style with the tables.

Thanks @wanxe


For a Hosted Workflow, the TRS link's path is null

Bug Report
The actual href for the TRS link on a hosted workflow has the path null.
Depending on how you get there, it can also display the previous workflow's TRS link -- if I go to another workflow, hover over the TRS link, then come back to this workflow, and hover over the TRS link, it shows the previous TRS link.

Note to implementor: display this only when published rather than use optional auth

This seems functional on staging
DockstoreAPI - 1.5.0-alpha.8
DockstoreUI - 2.2.0-alpha.6


Update iina to 0.0.11

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Why are you removing auto_updates true?

My bad, I didn't know that it now auto-updates.
Sorry!


Policies must now be explicit even with before() method

Laravel Version: 5.5.13
PHP Version: 7.1

Description:
Policy from Lumen/previous laravel (5.4.*) is not working as expected in the latest 5.5.13 version.  There was a change on \Illuminate\Auth\Access\Gate::resolvePolicyCallback(...) that now requires the permission you're checking for to exist on the policy class before it even bothers to attempt the before() call.
I was able to get around this by adding a __call() method to my Policy class so that this check on line 437 passes.
See: https://github.com/illuminate/auth/commit/cef2657b1eff37af8aface946f4b8b47eff52092#diff-4961f9aec0e370756f4b21e4191b9f69R399
If this change is not a bug and is expected behavior, then the documentation needs to be updated to account for this behavior change.  It really is quite a change.
Steps To Reproduce:
My AuthServiceProvider:
class AuthServiceProvider extends ServiceProvider
{
    /**
     * The policy mappings for the application.
     *
     * @var array
     */
    protected $policies = [
        Issue::class => IssuePolicy::class,
        Letter::class => LetterOptionPolicy::class,
        Magazine::class => OpenPolicy::class,
        MagazineVersion::class => AccountIdMatchesPolicy::class,
        PrintInstance::class => OpenPolicy::class,
        PrintInstanceApprover::class => OpenPolicy::class,
        PrintInstanceStatus::class => OpenPolicy::class,
    ];

    /**
     * Register any authentication / authorization services.
     *
     * @return void
     */
    public function boot()
    {
        $this->registerPolicies();

    }
}

My OpenPolicy class looks like such:
class OpenPolicy
{
    public function before()
    {
        return true;
    }

    /**
     * This fixes change from 5.5 which requires the ability you're checking for to exist as a callable method on the policy to even bother
     * with calling 'before'
     *
     * @see https://github.com/illuminate/auth/commit/cef2657b1eff37af8aface946f4b8b47eff52092
     *
     * @param $name
     * @param $arguments
     */
    public function __call($name, $arguments)
    {
        // Do nothing
    }
}

Duplicate of #20943
From the upgrade guide:
The before Policy Method
The before method of a policy class will not be called if the class doesn't contain a method with name matching the name of the ability being checked.
Also 5.5 isn't guaranteed to be fully compatible with 5.4.

Thanks for the quick response, @Dylan-DPC . I understand they're not guaranteed.  I searched open issues and did not see one but obviously I did not search hard enough.
As to the upgrade guide, another incident where I did not search well enough for it.  Thanks.
I'll close.

No issues mate. Next time search in closed issues as well :+1:

Is there a discussion thread where this was decided?


Problems with Uploading local files

I just tried 2 local installs of OpenPhoto from the master branch (once on OSX, and once on Ubuntu 12.04), and both times the uploader was broken for me. Here is a screenshot from the Apache error log highlighting the problem:
https://www.dropbox.com/s/it4lebcvsvdm4r7/Screen Shot 2012-05-20 at 21.48.01.png
It seems to me either the SQL is messed up, or there is a column missing in the photo table.
As a note, I'm using OpenPhoto with the local storage.

Closing as this is probably obsolete by now.


GH-174: Support Batch with @KafkaListener

Resolves: #174

Todo: List<Message<?>>.

Merged as https://github.com/spring-projects/spring-kafka/commit/7d5cad274ebf3861c06089cd12149b6e5483ff4b


Unable to delete default Alert

Not able to delete Alert
Navigate to Control >> Explorer
Click on alerts
Click on any Default Alert
It will allow to click on "Delete this Alert"
But it will not delete Alert

@abonas
@miq-bot

@lucasponce please have a look. from the description it sounds like a generic miq restriction, perhaps this should be a general ux enhancement to not allow/warn regarding deletion of default alerts (not middleware specific). if so, please label accordingly and assign to miq ui team. thanks!

Yes, pre-defined alerts cannot be deleted.
But any alert an user creates can be deleted without problem.
I guess it is related of permissions (I am not sure) but definitely not a bug on providers/hawkular.

@miq-bot remove_label providers/hawkular
@miq-bot add_label ui

@himdel Please, someone of the UI team could confirm if this is a bug or if it is an expected behaviour ?
Many thanks.

Pretty sure this is the expected behavior. @dclarizio ?
The toolbar doesn't know which alert is active so it can't disable the button for default alerts, and trying to push it does tell you that it can't be deleted. Not seeing anything wrong :)

Closing this issue as stale; if you're still having this problem with the newest release, please open a new issue in the UI Classic repository. If you like, you can also move this issue as-is to the UI repository by reopening it and commenting with @miq-bot move_issue manageiq/manageiq-ui-classic
Thanks!


Question on Google Authenticator

This is not really an issue anyway. I'm just curious about this sentence in README.rst:

Since Google Authenticator was not maintained well.

Is there some detailed information about this?

https://github.com/lepture/otpauth/commit/755832641cb464566425323d9f0f3b76e7337d3d


Crash when using mouse in mitmproxy

Steps to reproduce the problem:
1.Open mitmproxy
2.Click on any flow for flow details
3.
...
  File "/home/ujjwal/Github/mitmproxy/mitmproxy/tools/console/window.py", line 206, in mouse_event
    k = super().mouse_event(*args, **kwargs)
  File "/home/ujjwal/Github/mitmproxy/venv/lib/python3.5/site-packages/urwid/container.py", line 1169, in mouse_event
    event, button, col, row-htrim, focus )
  File "/home/ujjwal/Github/mitmproxy/venv/lib/python3.5/site-packages/urwid/listbox.py", line 1566, in mouse_event
    focus)
  File "/home/ujjwal/Github/mitmproxy/mitmproxy/tools/console/flowlist.py", line 72, in mouse_event
    self.master.view_flow(self.flow)
AttributeError: 'ConsoleMaster' object has no attribute 'view_flow'

System information
Mitmproxy version: 3.0.0 (2.0.0dev0424-0x81d68aa)
Python version: 3.5.2
Platform: Linux-4.9.0-040900-generic-x86_64-with-elementary-0.4-loki
SSL version: OpenSSL 1.0.2g  1 Mar 2016
Linux distro: "elementary" 0.4 loki

Yes, this is a reproducible crash!


Size error on render hidden checkbox

I'm trying to render the toggle on an hidden checkbox and the result is that the tooggle is of the wrong size and no text inside is shown.
I've found the the applied style on the div is
style="width: 0px; height: 0px;"
Is there a way to render an hidden checkbox in the proper size?
Thanks

Hi @edika99
Can you provide me a minimal html code for reproduce de issue?
Thanks

Take a look at this codepen
https://codepen.io/edika99/pen/rNeraJE
Look at the different rendeirng size of the hidden checkbox and the normal one. Click on the second toggle to show the hidden checkbox and compare the two size.

I see the issue, the issue. I think forcing render again will works. I'll send you a solution as soon as possible. 😊

Hi,
First, many thanks for this library, is very useful.
I have the same problem with a very similar scenario, i'm using the ecmas version
I have my main div hidden by default:
<main class="page-content d-none">
In this div there are also the bootstrap 5 toggles
After some operation, i'll remove the d-none:
 document.querySelector('main').classList.remove('d-none')
And, after showing the main, i'll inizialize the library like documented:
document.querySelectorAll('input[type=checkbox][data-toggle="toggle"]').forEach(function (ele) {    ele.bootstrapToggle(); })
So, in theory, i'll inizialize the library AFTER removing the d-none from the div but the result is the same of the OP. The toggles have style="width: 0px; height: 0px;"
With programmaticaly d-none added and removed:

Without d-none applied at all:

I have try to put the .bootstrapToggle() in different place, with settimeout, and so on. Nothing have worked except remove the d-none from the main div
If i need to open another issue, tell me.
I'll wait for a solution, thanks!

Hi @g-pane,
Is the same problem so a new issue is not needed 😊.
In fact, the toggle height and width is calculated considering the max calculated render size of each toggle element (on or off elements).
When the original checkbox is hidden, the calculated render size is 0px.
I'll give you a workaround as soon as possible, and look for a fix for the next version.

Ok thanks! But i don't understand (the technical reason) why if i'll inizialize the library after i'll remove the d-none the result is the same as the OP. In his case was hidden, in mine was hidden but showed before the inzialization

Ok thanks! But i don't understand (the technical reason) why if i'll inizialize the library after i'll remove the d-none the result is the same as the OP. In his case was hidden, in mine was hidden but showed before the inzialization

Hi @g-pane, the lib script render all the toggle on document ready, so I imagine the d-none is set before the toggle render.

Hi @edika99 and @g-pane,
I let you a code pen  with 2 workarounds for this issue.
Work around 1: render on show
Skip render on load with data attribute data-toggle="skip-toggle" and render the toggle on show with:
$('#my-selector').onclick((e)=>{
  // Add display to element
  $('#my-toggle').bootstrapToggle();
});

Work around 2: destroy and render on show
Destroy and render the toggle on show with:
$('#my-selector').onclick((e)=>{
  // Add display to element
  $('#my-toggle').bootstrapToggle('destroy');
  $('#my-toggle').bootstrapToggle();
});

I'll look for a better fix for the issue, but for the moment I think it can be a good solution.


Make deployment arguments configurable with Helm

Description
This PR introduces the possibility to customize the deployment arguments through helm (e.g., to set a given log level), and adds the klog flags to the components that were missing them.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[X] Checking the correct installation
[X] E2E tests (existing)

/test

/rebase

/rebase test=true

/merge


Rounded Corners

Hey!
Unlike your examples, my button doesn't have rounded edges.
button.layer.cornerRadius
doesn't work either. How can I round the corners?

hi, it's weird
try adding layer.cornerRadius inside user defined in storyboard

but adding
button.layer.cornerRadius
in code should be ok, check and see maybe you are changing the layer corner from somewhere else, because I tested it just now and everything was fine.
feel free to continue this thread

Hey!
Adding it to the runtime attributes did the trick, setting the cornerRadius in the viewController didn't work however.
Thanks!


TypeError: undefined is not an object (evaluating 'this.paymentMethodsResponse.paymentMethods')

Describe the bug
When I try to load the Adyen card component on my mobile iOS device then the app crashes with the following error: TypeError: undefined is not an object (evaluating 'this.paymentMethodsResponse.paymentMethods')
To Reproduce
Steps to reproduce the behavior:

Create the checkout by passing the checkoutSession alongside the env's to AdyenCheckout from this library
Run the code on a mobile iOS Device
See the crash

Expected behavior
The app should not crash and the card component should be loaded
Smartphone (please complete the following information):

Device: iPhone 14 Pro
OS: 16.5.1
Browser: Safari

This is how I create the AdyenCheckout (this part runs error free)
`
function useAdyenCheckout(checkoutSession: CheckoutSession | undefined) {
const hasRunInitiallyRef = useRef(false)
const [adyenCheckout, setAdyenCheckout] = useState<Core | null | undefined>(undefined)

useEffect(() => {
    if (!checkoutSession || hasRunInitiallyRef.current) return

    hasRunInitiallyRef.current = true

    const createAdyenCheckout = () => {
        AdyenCheckout({
            session: checkoutSession,
            clientKey: process.env.NEXT_PUBLIC_ADYEN_CLIENT_KEY,
            environment: process.env.NEXT_PUBLIC_ADYEN_ENVIRONMENT,
            paymentMethodsResponse: {
                paymentMethods: [],
                storedPaymentMethods: [],
            },
            // eslint-disable-next-line @typescript-eslint/no-explicit-any
            onPaymentCompleted: (response: unknown, _component: any) => {
                console.log({ response, _component })
            },
            // eslint-disable-next-line @typescript-eslint/no-explicit-any
            onError: (error: any, _component: any) => {
                console.log({ error, _component })
            },
        }).then((adyenCheckout) => {
            setAdyenCheckout((adyenCheckout as Core) ?? null)
            console.log({ adyenCheckout })
        })
    }

    createAdyenCheckout()
}, [checkoutSession])

return adyenCheckout

}
`

And this is how I create the card (the create function crashes the mobile app)
` useLayoutEffect(() => {
if (
typeof window === 'undefined' ||
!adyenCheckout ||
hasRunInitiallyRef.current ||
!paymentContainerRef.current
) {
return
}
    hasRunInitiallyRef.current = true

    adyenCheckout.create('card').mount(paymentContainerRef.current)
}, [adyenCheckout])`

I found the issue, it was related to my setup (as always). I was using ngrok to test on mobile and didn't have the forwarded url  in the allowed origins in the client key config.


Fix template generator for ToString method

Fixes #278
This PR fixes a broken string interpolation which made the XML doc comment for the generated ToString() method invalid.
This change affects pretty much every snapshot so the changes are huge unfortunately. I've broken the PR into two commits - the first is the fix, the second is the snapshot updates

Looks good to me! Thanks for spotting!


Bug: max_length STRLEN issues

PHP Version
8.0
CodeIgniter4 Version
4.3.7
CodeIgniter4 Installation Method
Manual (zip or tar.gz)
Which operating systems have you tested for this bug?
Linux
Which server did you use?
apache
Database
MySql 5.7
What happened?
Using default validation rules with a string of 62 characters the validation does not trigger  false value with a rule set for max_length[60]
<?php
$data= ['receivers' => string 'XXXXXXXXX EUGÊNIA XXXXXXXX|XXXXXXXXX Eugênio XX XXXXXXXX XXX']; // (length=62)

validation rule:
[receivers] => permit_empty|max_length[60]

when I compare strlen with mb_strlen over the given value I get two different counts:
Array
(
    [strlen] => 62
    [mb_strlen] => 60
)

I see that the default method (vendor\codeigniter4\framework\system\Validation\Rules.php)  has this logic:
 /**
     * Returns true if $str is $val or fewer characters in length.
     */
    public function max_length(?string $str, string $val): bool
    {
        return is_numeric($val) && $val >= mb_strlen($str ?? '');
    }

Steps to Reproduce

Create a model and add a validation rule for max_length of 60
use this value to test: "XXXXXXXXX EUGÊNIA XXXXXXXX|XXXXXXXXX Eugênio XX XXXXXXXX XXX"
it should return false as it has over 60 characters

Expected Output
Validation Returns false
Anything else?
Since the validation rules mostly result in storing data in the database, the difference between count of mb_strlen and strlen becomes a silent error.
I would advise to keep strlen as default, or give another option to have 2 max_length options:
max_length = uses strlen
max_length_mb = uses mb_strlen

Why do you think the string has 62 characters?
XXXXXXXXX EUGÊNIA XXXXXXXX|XXXXXXXXX Eugênio XX XXXXXXXX XXX
123456789012345678901234567890123456789012345678901234567890
         10        20        30        40        50        60

Changing the behavior of max_length is a breaking change.
I recommend you create your own validation rule to check byte length.

Because var dump results at 62 chars, my code editor has 62 positions but due to some special chars like ê the length is misleading
Em 31 de ago. de 2023 21:43, em 21:43, kenjis @.***> escreveu:

Why do you think the string has 62 characters?
XXXXXXXXX EUGÊNIA XXXXXXXX|XXXXXXXXX Eugênio XX XXXXXXXX XXX
123456789012345678901234567890123456789012345678901234567890
        10        20        30        40        50        60

Changing the behavior of max_length is a breaking change.
I recommend you create your own validation rule to check byte length.
--
Reply to this email directly or view it on GitHub:
https://github.com/codeigniter4/CodeIgniter4/issues/7888#issuecomment-1701955808
You are receiving this because you authored the thread.
Message ID:
@.***>

with strlen() the character Ê is taking two characters, therefor the character count is 62, rather than 60. As the original string is actually exactly 60 characters, mb_strlen() looks to be correct to me.....
$myString = 'XXXXXXXXX EUGÊNIA XXXXXXXX|XXXXXXXXX Eugênio XX XXXXXXXX XXX';
echo '<mb_strlen>'.mb_strlen($myString, 'UTF8').'
';
echo '<strlen>'.strlen($myString).'
';
for($x=0;$x<strlen($myString);$x++)
{
    echo substr($myString, $x, 1). '
';
}

Sorry, but you seems to misunderstand what is a character and byte.
"Ê" is one character, not two characters.
"あ" is one character, not three characters.
Probably you need the max_byte rule.

We use GitHub issues to track BUGS and to track approved DEVELOPMENT work packages. We use our forum to provide SUPPORT and to discuss FEATURE REQUESTS.

Thank you both for taking time to look into it, but as I canno disclose the actual data that my app received from an API that caused all that, and the problem that is happening with the count difference that cannot be reproduced by "copy past", perhaps I was receiving some invisible chars as well.
My solution was to remove all special chars like: "Ê" to "E" and strip anything else that wasn't alphanum, and to add max_length_bytes custom rule.
Special thanks to @visserrobin explanation, it didn't solve my problem, but got me on the right track.


Serializing types with object members has unreliable behaviour

Suppose I have this class which captures audit information:
public class InterceptedData
{
    public InterceptedData()
    {
        Children = new List<InterceptedData>();
        Parameters = new List<Parameter>();
    }
    /// <summary>
    /// Method (operation) name that was invoked.
    /// </summary>
    public string MethodName { get; set; }

    /// <summary>
    /// Parameters that were passed to the operation.
    /// </summary>
    public List<Parameter> Parameters { get; set; }

    /// <summary>
    /// Gets or sets the return value.
    /// </summary>
    public object ReturnValue { get; set; }
    /// <summary>
    /// Additional audits that were triggered by this operation.
    /// </summary>
    public List<InterceptedData> Children { get; set; }
    /// <summary>
    /// Exception object, if an exception was thrown by the operation.
    /// </summary>
    public ExceptionDetails Exception { get; set; }
}

My ReturnValue needs to be object because it can contain the return value of anything. At runtime, in one instance I am getting a List<> object back from one of my intercepted methods, and Yaxlib is serializing it like this:
<Value yaxlib:realtype="System.Collections.Generic.List`1[[ApplicationProcessingService.Entities.ExternalSystemMappingElement, ApplicationProcessingService.Entities, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]" />

This isn't very useful at all.
However, within my Parameter class which is used for the Parameters property, which is defined like this:
public class Parameter
{
    public string Name { get; set; }
    public string Type { get; set; }
    public object Value { get; set; }
}

The values in the Value property of this class are serialized properly:
<Parameters>
    <Parameter>
      <Name>eform</Name>
      <Type>ApplicationProcessingService.Entities.Eform</Type>
      <Value yaxlib:realtype="ApplicationProcessingService.Entities.Eform">
           ...
      </Value>
    </Parameter>
</Parameters>

Why does using object in my Parameter class result in the serialization of the entire object, while the ReturnValue property doesn't work at all?
I don't need to deserialize any of this data, it is intended to be human readable only.

Why do you think what has been serialized for ReturnValue isn't useful? Is the collection not empty but serialized as empty?

The collection is not empty, and when it is serialized, it turns into this:
<Value yaxlib:realtype="System.Collections.Generic.List`1[[ApplicationProcessingService.Entities.ExternalSystemMappingElement, ApplicationProcessingService.Entities, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]]" />

You can't even deserialize that. It has no values within it, nothing. It's a self closing tag.
As a workaround, we implemented a property next to Value called Items that figures out if Value is a collection and returns an array of the items in the collections as the getter.

@304NotModified @NKnusperer The feature implementation would be to test any object for is IList and is IDictionary and cast before serialization. Do you agree?

Sounds good, but we aware that testing generic dictionaries/lists could eat some performance

For the master branch as well as the current v4 branch, serializing and deserializing a generic list, that is boxed in an object works.
The result is different from https://github.com/YAXLib/YAXLib/issues/82#issuecomment-326061654
[Test]
public void Serialize_List_Boxed_In_Object()
{
    const string expected =
        @"<Object yaxlib:realtype=""System.Collections.Generic.List`1[[System.String, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e]]"" xmlns:yaxlib=""http://www.sinairv.com/yaxlib/"">
  <String>one</String>
  <String>two</String>
  <String>three</String>
</Object>";
    object toSerialize = new List<string> { "one", "two", "three" };
    var ser = CreateSerializer<object>();
    var serialized = ser.Serialize(toSerialize);
    var deserialized = ser.Deserialize(serialized);

    Assert.That(serialized, Is.EqualTo(expected));
    deserialized.Should().BeEquivalentTo(toSerialize);
}

Can we close this issue?

I think we could close it.


missing_order option missing for CompositeAggregationSource

I am currently working to replace our seach engine with elastic search and I got a business rule that could work in plain JSON but the option is missing in the NEST 7.17.5 library : there doesn't appear to be a way to set the missing_order option for composite aggregations. I would like to do something like:
List sources = new List();
sources.Add(new TermsCompositeAggregationSource(sourceName)
{
Field = field,
Order = sorter.IsAscending() ? SortOrder.Ascending : SortOrder.Descending,
MissingBucket = true,
MissingOrder = "last"
});
I naively tried to extend the interface / class, but it didn't worked and I didn't find another workaround, is there any with this version ?
I cannot upgrade to the new client version because composite aggregation are not fully supported yet

Hi there,
NEST is deprecated and we won't implement any new features there. As a workaround, you could use the low level client and build the JSON request yourself:
https://www.elastic.co/guide/en/elasticsearch/client/net-api/7.17/transports.html
I'm closing down this issue for now as it's a feature request for NEST, but feel free to reach out again if you have any other specific questions.


Update AS5048A.cpp

in line 278 in debug mode the code prints the command variable thath is wrong and must print response variable

Thanks for the change :)


"@angular/material/icon" has missing dependencies @angular/cdk/coercion

"@angular/material": "^9.0.0

Repro: https://github.com/jadekler/angular-spinner/tree/e3cb4b0bfa06ce35d3dc1711219cab68105202ac
(run ng serve)
With the following app.module.ts,
import { BrowserModule } from '@angular/platform-browser';
import {MatIconModule} from '@angular/material/icon';
import { NgModule } from '@angular/core';

import { AppComponent } from './app.component';

@NgModule({
  declarations: [
    AppComponent,
  ],
  imports: [
    BrowserModule,
    MatIconModule,
  ],
  providers: [],
  bootstrap: [AppComponent]
})
export class AppModule { }

I get,
$ ng serve

chunk {main} main.js, main.js.map (main) 2.01 kB [initial] [rendered]
chunk {polyfills} polyfills.js, polyfills.js.map (polyfills) 686 bytes [initial] [rendered]
chunk {runtime} runtime.js, runtime.js.map (runtime) 6.15 kB [entry] [rendered]
chunk {styles} styles.js, styles.js.map (styles) 9.72 kB [initial] [rendered]
chunk {vendor} vendor.js, vendor.js.map (vendor) 340 kB [initial] [rendered]
Date: 2020-02-18T18:52:33.404Z - Hash: 2b4530d3884f9dda3f77 - Time: 2307ms

ERROR in The target entry-point "@angular/material/icon" has missing dependencies:
 - @angular/cdk/coercion

Seems like @angular/material/icon is not installing all its deendencies?

@angular/cdk is a peer dependency of @angular/material (package.json)
https://nodejs.org/es/blog/npm/peer-dependencies/

Thanks for the link and the information.
AFAICT peer dependencies are not automatically installed (wow). For posterity, that's in contrast to https://nodejs.org/es/blog/npm/peer-dependencies/ - it appears as though the auto-install described in https://nodejs.org/es/blog/npm/peer-dependencies/ was reverted sometime in npm3+. Good lord, what a mess.
OK, so, AFAICT, the value that peer dependency declarations provide is occasionally telling users about invalid versions of my-lib and my-peer-lib. However, it seems to not actually address point of declaring a dependency - installing said dependency when an upstream user says install my deps.
Overall this seems a poor experience for users. When users depend on this library and their build fails (because they don't have cdk), they have zero clue what's going on. One certainly does not expect to have to go look into the package.json of various dependencies. It seems like the easiest way to get around this is to just depend on every single @angular thing, which horribly bloats a build. Is that what everyone does? It seems like all the stackblitz examples linked from the documentation site do so, so it seems to be a pattern.
I don't think any of this is going to change, but FWIW I think this is a poor experience for users and should be fixed. If material needs cdk to work, why not just add it to dependencies?
(disclosure: I'm not a javascript person and am not aware of what the downsides of adding it as a dep has - it just seems insane that it is a dependency but it's not being declared as such)

Update: after discussion, seems like it has to be this way or else users get two copies of cdk. Bummer. One nice-ish solution would be for npm to be intelligent enough to suggest missing peer dependencies when it fails.


Fix - Route mismatch on demo website

fixed Day Input and Confirmation Modal navigation.
I am not sure if yarn.lock file needs to be updated. Please let me know if it needs to be removed.
Thanks!!

Thanks!


Onclick function not working with button in inner popover

In the first time everything work well. But when I hide Popover and reopen it. When I click my button function onclick not working. Please help me. Thanks!

I doubt it's a popover issue. Can you post your code or a jsfiddle repro to see whats going on?

Can you check: https://jsfiddle.net/wacv8ahs/
Step 1: click Login
Step 2: click setting and chose on option of select it auto close, this is my issue
Thanks Edddy

use dismissible: true Example: https://jsfiddle.net/yz44587L/

@Edddy can you check your example, It not work for me. The same issue with me


Create drum set interface

Associate drums instead of tones. Possibly we should make a new protocol to use instead of tone and a different struct that implements that protocol will be used for a drum set.

We also will need to do something about making a measure without a key. Possibly another type that implements ImmutableMeasure and make the key optional or something?


fog tests 1

Codecov Report

Merging #45 into master will increase coverage by 9.87%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master      #45      +/-   ##
============================================
+ Coverage     40.98%   50.85%   +9.87%
- Complexity      179      230      +51
============================================
  Files            37       37
  Lines           815      816       +1
  Branches        102      103       +1
============================================
+ Hits            334      415      +81
+ Misses          460      374      -86
- Partials         21       27       +6

Impacted Files
Coverage Δ
Complexity Δ

src/main/kotlin/emufog/fog/FogWorker.kt
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

src/main/kotlin/emufog/fog/StartingNode.kt
100% <100%> (+100%)
7 <1> (+7)
:arrow_up:

src/main/kotlin/emufog/fog/BaseNode.kt
92.59% <87.5%> (+92.59%)
28 <5> (+28)
:arrow_up:

src/main/kotlin/emufog/fog/CostComparator.kt
83.33% <0%> (+83.33%)
3% <0%> (+3%)
:arrow_up:

src/main/kotlin/emufog/fog/FogResult.kt
87.5% <0%> (+87.5%)
6% <0%> (+6%)
:arrow_up:

src/main/kotlin/emufog/fog/FogComparator.kt
88.88% <0%> (+88.88%)
5% <0%> (+5%)
:arrow_up:

src/main/kotlin/emufog/fog/FogNodePlacement.kt
100% <0%> (+100%)
2% <0%> (+2%)
:arrow_up:

Continue to review full report at Codecov.

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


Floating point exception in agent mode

I'm trying to run mutilate in agent mode:
$ mutilate -A -T 16
But it crashes. Here is the output from GDB.
Program received signal SIGFPE, Arithmetic exception.
Value can't be converted to integer.
0x0000000000404a0e in args_to_options (options=options@entry=0x7fffffffdbc0) at mutilate.cc:1061
1061	  options->records = args.records_arg / options->server_given;

I saw the same problem on CentOS 7.
It was because libzmp is not currently installed, especially the zeromp c++ binding (zmp.hpp). If I check the config.h created by scons, the HAVE_LIBRT is not defined, meaning libzmp is not found. This would cause the "divide by zero" problem.
Fix:

download lastest zeromq from http://zeromq.org/area:download, and build from source by
$./configure
$make
$sudo make install
checkout zeromq c++ binding, which is now a separated project hosted at https://github.com/zeromq/cppzmq
$cp zmp.hpp /usr/local/include
rebuild project
$scons -c
$scons