Zuul: Configuring PoolingHttpClientConnectionManager with Time to Live?

Hi all,
I am using zuul to route requests to statically configured services according to the request path.
The services behind zuul are addressed using DNS names. The DNS records can change rather frequently (e.g., daily).
I have seen in SimpleHostRoutingFilter that zuul uses PoolingHttpClientConnectionManager from Apache HttpComponents without configuring the time to live for the connections in the pool
(see https://github.com/spring-cloud/spring-cloud-netflix/blob/master/spring-cloud-netflix-core/src/main/java/org/springframework/cloud/netflix/zuul/filters/route/SimpleHostRoutingFilter.java#L238). Even if the DNS records change, the connection pool in zuul is keeping the connections to the old instances open. Depending on the load, it can take many hours until all the old connections are evicted from the connection pool.
I could subclass SimpleHostRoutingFilter and override the method newConnectionManager() to return my own instance of PoolingHttpClientConnectionManager with a configured TTL. Is there any other way to solve my problem, and  does it make sense to add this feature to zuul?
Best,
Gary

I believe the you are right the only way to do this is to subclass SimpleHostRoutingFilter.  Feel free to provide a PR with a better solution.  Or we could try and address this issue along with fixing the issue in #1684.

@ryanjbaxter SimpleHostRoutingFilter is not conditional in ZuulProxyConfiguration. Even if it's subclassed, how'd I replace the existing one without doing bean post processing?

@GJL There is no property called TTL in the PoolingHttpClientConnectionManager. What methods would you call to get the behavior you're asking for?

Use @Primary

@ryanjbaxter This doesn't apply?
http://cloud.spring.io/spring-cloud-static/Dalston.RELEASE/#_disable_zuul_filters

You can try that as well, I haven't tried it myself.  I dont see why it wouldnt work

Hey there, I have the same problem like @GJL . Is there any updates or improvements?


chore: remove unused library

express-state adds .expose, which is no longer used.

Thanks for digging through the history - I had assumed it was used at one point and then dropped, but I guess not.


Handling purging of lingering files before scheduleExecutor start.

Is your feature request related to a problem? Please provide an existing Issue # , or describe.

Addresses Joydeep's comment on PR-36. Moved deleteAllFiles() to inside scheduleExecutor(), this ensures that deleteAllFiles() is invoked once and before beginning of event log file generation.
Minor change around fixing the official documentation to resolve Link Checker errors.

Describe the solution you are proposing
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

We need to fix these failing tests:
Tests with failures:

org.opensearch.performanceanalyzer.reader_writer_shared.EventLogFileHandlerTests.testNoFileWithNoMetrics
org.opensearch.performanceanalyzer.reader_writer_shared.EventLogFileHandlerTests.testFileWithMetrics


Ability to hide warnings

As already mentioned a week ago in this PR https://github.com/tgriesser/knex/pull/2471 the warning that was added with that PR makes it impossible to use knex in a pure CLI environment if you are using an ORM like Bookshelf or Objection as your terminal with get spammed with the warning even though you yourself are not calling the returning method, the ORMs do.
As @elhigu mentioned in the comments a feature request is to make it possible to plug in your own logging implementation in the future but until that comes would something like (returning && process.env.SUPPRESS_WARNINGS) be an option?
It would be awkward to be forced to fork purely to hide this 1 warning message https://github.com/tgriesser/knex/pull/2471/files#diff-9e1276d680a5451020b184cb3f7fa802R15 in a CLI environment.

Seems something is in the works already

This was marked in some issue as Planned for 1.0 which I guess is still quite some time away based on the current version number or did that plan change?
My current issue is that I was using knex just fine in a production CLI application with 0.14.3 but 0.14.4 introduced a bug fix I also need but the same version also introduced the warning message for sqlite returning so I am currently locked at 0.14.3 without the bug fix because of the warning message.

Closing this as overriding log functions is now possible: https://knexjs.org/#Installation-log
If this doesn't cover the issue let's reopen.


Just a heads Up: Will not be compatible with cms v3.0.5 without changes

NOTICE: This commit here: https://github.com/divio/django-cms/commit/d8b36f21ec64128c715ccbeb198b41557a686b2a#diff-2e3d5c25dd6236e9f360d8a2fcfffaec
Changes the signature to get_processors to: get_processors(self, context, plugin, placeholder)
This will require changes in this code.
Further recall that once Aldryn reaches 3.0.3, the local version of get_processors should be removed in favor of calling the one in the render_plugin superclass anyway.

Is this still the case @mkoistinen ? Wondering if anyone is using this with latest Django CMS

I'm pretty sure we're using this with the latest django CMS 3.2.x, so, I think I'll close this.


pipe - add input from Stdin

Description:
with some copy-paste added pipe:
resolves #836
usage example:
curl -sS -H "Accept: application/vnd.github.patch"  \
         -H "Authorization: Bearer ${TOKEN}" \
          https://api.github.com/repos/${OWNER}/${REPO}/compare/${BASE_REF}...${HEAD_REF}  | \
  ./gitleaks detect --pipe

or
git log -p "${BASE_REF}...${HEAD_REF}" |  ./gitleaks detect --pipe

Checklist:

[x] Does your PR pass tests?
[ ] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

Thanks for the PR @rommi4!
I want the --pipe option to be git agnostic. I.e, --pipe should be able to accept any input, not just git diffs. This is useful for streaming applications and building services around gitleaks. That said, I think there is some value in your implementation of FromPipe in git/git.go. There could be some code that quickly checks the format of stdin which would either route the detection to gitdiff.Parse(cmd, stdout) or straight to detect.DetectReader (to be implemented).
Does that make sense? It's a bit early in the morning but I can provide some more context later with what I'm looking for for this feature

Hello, @zricethezav
Just updated my PR
There's simple check for git input.
If git, proceed to detect.DetectPipeGit
detect.DetectPipe may be updated in future with parsers other than git
Already working from my fork for 2 weeks this way:
curl -sS  https://api.github.com/repos/${OWNER}/${REPO}/compare/${BASE_REF}...${HEAD_REF}  | ./gitleaks detect --pipe

suitable for checking pull requests in LARGE repos without checkout in Github Actions every time.

may be closed because of https://github.com/zricethezav/gitleaks/pull/1012

@rommi4 Does #1012 meet your requirements as well?

Testing at the moment. Will update if find anything


under swift2.0 has errors

under swift2.0 has errors

any updates for swift 2.0?

@yasiraliraj @lacklock  advance() method has been changed to advancedBy().
I have submitted a pull request in Swift 2.0 branch, in the meantime you can make use of this code.
https://github.com/sunkanmi-akintoye/SwiftColors/blob/master/SwiftColors/SwiftColors.swift

+1 can someone merge the fix and update the cocoapod?

+1

Just updated the CocoaPods version. You don't need to wait for the CocoaPods version anyway, you can always use the latest version by adding the following to your Podfile.
pod 'SwiftColors', :git => 'https://github.com/thii/SwiftColors.git'


ImportError: cannot import name 'TPB' from 'tpb'

This is my code:
from tpb import TPB
from tpb import CATEGORIES, ORDERS
t = TPB('https://boatairproxy.org/')
search = t.search('public domain', category=CATEGORIES.VIDEO.MOVIES)
I get the error in the title when i run..

I too get this error

I have resolved it. It was because I was using the library within a file named 'tpb.py'


101301 when using most of the Italian's credit card for the payin

We noticed several times on our projects that different italian banks do not allow the payment because error 101301 is encountered.
It happens mostly with Fineco bank. Is there a way to solve this issue?

@Carlonz10
Italian here :-)
we have the same issue from over 1yr and support don't help us...
have you find any walkaround?


Filter indentation gets screwed up when nested

This:
%head
  :javascript
    $(document).ready(function(){
      $("#form{{form.initial.id}}").submit(form_submit);
      // Javascript comment
    });
  :css
    .someClass {
      width: 100px;
    }
  :cdata
    if (a < b && a < 0)
    {
      return 1;
    }

should compile to this:
<head>
  <script type='text/javascript'>
  // <![CDATA[
    $(document).ready(function(){
      $("#form{{form.initial.id}}").submit(form_submit);
      // Javascript comment
    });
  // ]]>
  </script>
  <style type='text/css'>
  /*<![CDATA[*/
    .someClass {
      width: 100px;
    }
  /*]]>*/
  </style>
  <![CDATA[
    if (a < b && a < 0)
    {
      return 1;
    }
  ]]>
<head>

but instead, it compiles to this:
<head>
<script type='text/javascript'>
// <![CDATA[
    $(document).ready(function(){      $("#form{{form.initial.id}}").submit(form_submit);      // Javascript comment    });
// ]]>
</script>
<style type='text/css'>
/*<![CDATA[*/
    .someClass {      width: 100px;    }
/*]]>*/
</style>
  <![CDATA[
    if (a < b && a < 0)
    {      return 1;    }
  ]]>
</head>

I came across this yesterday also, and it may have something to do with a previous issue (#52), which fixed multi-line HAML that contained other braces within it.
The code was supposed to ignore any data picked up in js/css/plaintext nodes
Oddly enough, I created a test (filterMultilineIgnore.haml and filterMultilineIgnore.html) to test for this, but must have overlooked something.
Having a busy week but will revisit this as soon as I can.

Thanks! I'll try to send a pull request if I find some time this weekend.

The problem was that the code for checking the node type of an element's parent wasn't working in hamlpy.py.
Wrote a parent() function in root_node to do it instead, and added some tests.


hugo页面对图片是否支持base64

目前大多数的图片都是用本地存储或第三方生成的链接, 不知道hugo是否支持图片解析为base64

You need to write in English (or Norwegian) for me to understand.

@VEH47 I read your message with Google Translation.

base64 | Hugo

Hugo has the base64 function. Maybe, you can treat your images with this function.

您需要用英语（或挪威语）书写才能理解。

At present, most of the pictures are linked by local storage or third-party generation. I don't know if Hugo supports image parsing to base64.

You can search related topics at https://discourse.gohugo.io/search?q=image base64

Hugo supports image parsing to base64.

We do. You can pipe the raw content intobase64Encode,  e.g.
{{ $image.Content | base64Encode }}
https://gohugo.io/functions/base64/#readout


Missing Example for ICommand CanExecute Implementation

hi,
the Documentation does not show any usage of the CanExecute Method.
How can i eg.  bind a vue textbutton to a ICommand implementation that enables e.g. the button when the current time second is odd?
Where can the RelaySimpleCommand CanExecute  Method be implemented?

RelaySimpleCommand  does not have any CanExecute  method. Indeed ISimpleCommand inteface is designed for command than can always be executed.
If you need to invalidae command, every implementation of ICommand  will work with Neutonium  but you may use RelayToogleCommand  that Neutronium provides: changing ShouldExecute property will triger CanExecuteChanged events.
Alternativelly you may implement a RelayCommand by yourself check here.


FuelEfficientPilots.netkan

Mod details:
name = /mod/1466/FuelEfficientPilots
author = packetshepherd
abstract = Give Pilots better fuel efficiency based on experience
license = CC-BY-NC-SA 4.0
Homepage =
description = Pilots have the ability to get better fuel efficiency through experience, so now your five-star pilot can give you better bang for your buck!

I am the author of this mod.

Merging with many thanks!

Thanks!


#54 Upgrade leinjacker to version compatible with recent versions of leiningen

#54

Thanks! I deployed a new version to clojars so hopefully people won't run into this anymore.


[UPDATE BONUS GUIDE] Update system-overview.md to reference raspibolt-pulse

References raspibolt-pulse repo in place of system-overview sample.
assumes eth0 over wlan0, inverts guidance
changed guidance to review script content before installing

What
Migrate from older system-overview to raspibolt-pulse
Why
References the scripts in new raspibolt-pulse repo which are intended to work across versions of Raspibolt (1.x - 3.x), and provide info about alternative electrum server, lighting implementation, and lightning web app
How
References new repo and updated commands to install
Scope

[ ] significant change to core configuration
[x] independent bonus guide
[ ] simple bug fix

Fixes # (link issue)
Depends on https://github.com/raspibolt/raspibolt-pulse/pull/17

Can we merge this now?
@vicariousdrama Updated his pull request already

Yes. This was dependent on the raspibolt-pulse PR which has been merged.


several analysis_server_tests are slow

analysis_server/test/domain_analysis_test is slow out on jsshell.
analysis_server/test/analysis_notification_outline_test is slow on d8
analysis_server/test/domain_search_test is slow on d8

I believe that the problem is in the test framework, not the tests. If you disagree, please start an e-mail conversation with me.

Status lines labeled with this issue are now labeled with #26813 .
The analyzer is compiled by dart2js and tested only on a special dedicated bot, since the analyzer is not intended to be used that way.  These function mainly as stress tests on dart2js and the analyzer.


ARROW-233 Support PyArrow 16.1

Something weird is happening on MacOS 3.8 and 3.9.  I'm going to defer this PR.


Random crashes?

Just wondering, is anyone else experiencing crashes after the recent updates? I'll crash around every 5 mins. Im only using radar and vis chams. Thanks all!

are you using the most recent build?
are you running any other cheats with Osiris?

i was having the same problem but im pretty sure it was from my lua system

honestly I thought it was one of the PR's i C+P'd but i guess not if someone else if having an issue @danielkrupinski

are you using the most recent build?
are you running any other cheats with Osiris?

Yes, no. Only using osiris with some added PR's

yea the cheat crashes for me everytime i plant the bomb

i fx it

i fx it

Care to share?

yea the cheat crashes for me everytime i plant the bomb

Not having that problem, just tryed it

Nothing is crashing when you take original repo with latest commits...maybe you integrated something what causes the crash. Some of the pull requests for example?

debugggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggggg

Nothing is crashing when you take original repo with latest commits...maybe you integrated something what causes the crash. Some of the pull requests for example?

Just confirmed this, my bad all, i dont know why that didnt occur to me anyway

the pull you were cloning was probably behind on a patch


Netlify Claim URL After Deploy Not Provided

Describe the bug
Don't know if it's normal or not but at the beginning Bolt always gave me the claim URL after the deploy on Netlifyand now it stopped... Thank you!
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-rohgxl
Steps to reproduce

Ask for a deploy
If the claim URL is not provided ask for it
Ask it again and again (looping)

Expected behavior
I would expect for the claim URL for Netlify
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 163116187,
  "usedJSHeapSize": 156302947,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = drssr-coding
Chat ID = 666aa3994339

Additional context
No response

Appreciate the feedback! We are aware of this issue and are tracking it here: #2020.


Mitaka can no longer be deployed to HTTP redirect on ceph.com

Do NOT include any confidential information, this issue will be created on a public repo
This section is completed by the issue reporter
On which version(s) do you encounter this issue?
Please provide the output to 'git describe --all' for  all deployments where you have confirmed it occurs.
r13.1.2
What type of issue is this? Please tick the appropriate box.

[x] bug - existing functionality that does not work for you as documented
[ ] feature - new functionality that you would like to see added

What is the expected behavior?
No apt issues
What is actually happening?
Currently ceph_stable_repo is set to ceph_stable_repo: "http://ceph.com/debian-{{ ceph_stable_release }}" which causes a HTTP redirect on the ceph download side and breaks APT
  File "<stdin>", line 464, in main
  File "/usr/lib/python2.7/dist-packages/apt/cache.py", line 440, in update
    raise FetchFailedException(e)
apt.cache.FetchFailedException: E:GPG error: http://ceph.com trusty InRelease: Clearsigned file isn't valid, got 'NODATA' (does the network require authentication?)

I needs to be set to
ceph_stable_repo: "http://download.ceph.com/debian-{{ ceph_stable_release }}"

What were you attempting to do when you encountered this error (give as much detail as possible)?
How can we reproduce this issue?
See above
How can we verify your issue is addressed?
No more apt error during ceph install
If you are using or know of a workaround please describe the necessary steps?
What is your evaluation on the importance of this issue, and why?

This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[ ] master
[ ] newton-14.0
[ ] mitaka-13.1
[ ] liberty-12.2

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

Is this only when DEPLOY_CEPH is set to yes? If so, could you change the title to say "Mitaka with Ceph can no..."

The issues is inside the ceph common role which can triggered when installing the ceph clients and/or MAAS playbooks installing the client for monitors

@BjoernT The ceph_stable_repo variable added to RPC-O should override the variable inside the version of the ceph role that we're bringing down onto the deployment host. Is the wrong version of the ceph-common role the correct version?

OK thanks, I guess the issue is related that this was a upgraded r13.0.1 to r13.1.2 and the default did not got changed accordingly. Closing since it's part of the branch


Next release version of sdk-go will require kafka_sarama users to update their module path

👋🏻 hi, Sarama maintainer here
PR https://github.com/cloudevents/sdk-go/pull/930 updated the module path of Sarama from github.com/Shopify/sarama to github.com/IBM/sarama — which long term is the correct thing to do as Shopify asked us to take over ownership of the library going forward and we've already made several releases
However, the PR was done under the existing v2 module path of github.com/cloudevents/sdk-go/protocol/kafka_sarama/v2, yet it is a breaking change in terms of callers of the library who will need to update their import path of Sarama before bumping their cloudevents/sdk-go module. It's not yet a problem, as a new release tag hasn't been published, but it would be worth thinking about what you plan to do here as it would really need to be done either under a major semver v3 release or under a new protocol submodule, leaving the old kafka_sarama one in-place

Thx for flagging this @dnwe ! And you're right, we should have used a new protocol submodule to avoid the breaking change. IMHO this change doesn't justify a v3 of the SDK (besides the fact that semver rules require it). So IMHO we have to options:

fix the current behavior, i.e., revert and reapply the change to a new v3 submodule
Announce this change as a breaking change in our release notes (which doesn't prevent users from running into the issue, but would be documented).

My understanding is that only users importing kafka_sarama would be affected by (2)?
cc/ @lionelvillard @duglin for further input/thoughts

Correct yes, it would only effect people importing and calling into kafka_sarama itself. I agree that a breaking change release note is probably sufficient. Ultimately there won't be any more releases on the github.com/Shopify/sarama import path so the consumers will have to update their module path at some point.
If you can tag a new point release of v2 then I can always submit a few PRs to the immediate importers to sync them up

SGTM

To be clear.... you guys are suggesting option 2, right? If so, I'm ok with that.


Business central - How I can download the imgae for it?

I just mount the docker and the image from nav-2018 last cu but I would like to test business central where I can download this image?
Thanks,

https://blogs.msdn.microsoft.com/freddyk/2018/04/16/which-docker-image-is-the-right-for-you/

Thanks, now it works :)


correct pluralization for vertex, human

Fix Automattic/mongoose#11515
Fix Automattic/mongoose#13037
@hasezoey @uzlopak can you please take a look?

On second thought, I think we should leave the plural of "vertex" as "vertexes" to be consistent with the discussion from this issue: https://github.com/Automattic/mongoose/pull/11953 . TLDR; we argued that "indexes" is an acceptable plural form of "index" because major dictionaries like Oxford and Merriam-Webster agree. "vertexes" is sufficiently similar.

Fixed handling for 'human' and related pluralizations, e.g. 'superhuman' -> 'superhumans', etc.
$ node
Welcome to Node.js v16.17.0.
Type ".help" for more information.
> pluralize('human')
Uncaught ReferenceError: pluralize is not defined
> const pluralize = require('./')
undefined
> pluralize('human')
'humans'
> pluralize('superhuman')
'superhumans'
> pluralize('man')
'men'
> pluralize('ombudsman')
'ombudsmen'
>
(To exit, press Ctrl+C again or Ctrl+D or type .exit)
>


Switch from attobot to Registrator.jl

See https://discourse.julialang.org/t/switching-package-registration-systems-on-monday/22677/
From today on, Julia doesn't rely on METADATA.jl anymore but instead uses JuliaRegistries/General.
Hence, one has to switch from using attobot to using Registrator.jl.

https://github.com/JuliaLang/Pkg.jl/pull/1133
REQUIRE -> Project.toml script

Let's try triggering Registrator:
@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/191
After the above pull request is merged, it is recommended that you create
a tag on this repository for the registered package version:
git tag -a v0.2.1 -m "<description of version>" 9531b3c73aef9e2108099f1eef90d962838c6c3b
git push origin v0.2.1

Nice, that worked. Let's wait until the PR to General.jl is merged and see if TagBot creates a new github tag/release.

Neat, worked as well.


Syntax for setting casing: "title" for all input fields

How can I set casing: "title" for all input fields on a form?
I've got this:
$(document).ready(function () {
$("#formName input[type=text]").inputmask({ casing: "title" });
});
but I know the syntax isn't correct.
Any help would be appreciated.
Thank you!

Try with
Inputmask.extendDefaults({ casing: 'title' });
Op vr 14 okt. 2016 om 01:01 schreef billkron<EMAIL_ADDRESS>
How can I set casing: "title" for all input fields on a form?
I've got this:
$(document).ready(function () {
$("#formName input[type=text]").inputmask({ casing: "title" });
});
but I know the syntax isn't correct.
Any help would be appreciated.
Thank you!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RobinHerbots/Inputmask/issues/1419, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AATb7y-9bFCY_fRgG9OYEkRNBfw8UxHGks5qzrhYgaJpZM4KWeMm
.

Where would I put this line of code?

In the meantime, I added JavaScript function that accomplishes this. However, there is an undesirable side-effect of, when you try to highlight existing text in the field, the cursor is moved to the end of the text string. So, for example, if you want to highlight the entire contents of the field to delete its content, you can't.
I am very interested in trying to leverage your casing feature instead, though. So, if you still have a chance to respond about where this line of code should go in order to make it work as expected, I would greatly appreciate it.
Thanks again for taking the time to respond!

@billkron ,
The line should be executed before you apply the masks.
Anyway which version of the inputmask do you use?

I'm using version 3.0 of inputmask. As far as where it should be placed, here is an example of what I tried (this was placed at the bottom of my form):
    <script>
        Inputmask.extendDefaults({ casing: 'title' }); // tried it here
        $(document).ready(function () {
            Inputmask.extendDefaults({ casing: 'title' }); // also tried it here
            $("#DateOfBirth").datepicker({
                dateFormat: 'mm/dd/yy',
                changeMonth: true,
                changeYear: true,
            }).inputmask("mm/dd/yyyy", { "placeholder": "mm/dd/yyyy" });
            $('#Zip').inputmask("99999[-9999]", { greedy: false, 'removeMaskOnSubmit': true });
            $("#HomePhone").inputmask("mask", { "mask": "(999) 999-9999", 'removeMaskOnSubmit': true });
            $("#WorkPhone").inputmask("mask", { "mask": "(999) 999-9999", 'removeMaskOnSubmit': true });
            $("#CellPhone").inputmask("mask", { "mask": "(999) 999-9999", 'removeMaskOnSubmit': true });
            $("#Email").inputmask("email");
        });
    </script>

You need at least version 3.3.2

Actually, I have version 3.3.2-91 installed (sorry for providing the wrong information).

@billkron ,
The only mask where the casing title would make a difference is the email mask.  As upcasing a number doesn't change it ;-)
I tried it locally and it works as expected.
Both positions where you inserted the default should work.
Can you make a jsfiddle which shows the problem?

I totally understand that with the fields I've referenced in the inputmask, only one of them would be affected by the casing default, but I'm clearly missing something else here. :-)
It sounds like you're saying that the cassing default will only apply to the fields that I have specifically referenced in the inputmask. Is that correct? I was originally looking for this to be a catchall for any form field that accepts text input without having to directly reference specific fields. I have several other fields (e.g., FirstName, LastName, AddressLine1, etc.).
Is it your assertion that the casing defaults would apply to all form fields whether they are referenced in the inputmask or not? If so, then I will certainly do a jsfiddle, so you can see what I'm seeing.

The casing is only applied for masked inputs.
Op di 18 okt. 2016 om 04:21 schreef billkron<EMAIL_ADDRESS>
I totally understand that with the fields I've referenced in the
inputmask, only one of them would be affected by the casing default, but
I'm clearly missing something else here. :-)
It sounds like you're saying that the cassing default will only apply to
the fields that I have specifically referenced in the inputmask. Is that
correct? I was originally looking for this to be a catchall for any form
field that accepts text input without having to directly reference specific
fields. I have several other fields (e.g., FirstName, LastName,
AddressLine1, etc.).
Is it your assertion that the casing defaults would apply to all form
fields whether they are referenced in the inputmask or not? If so, then I
will certainly do a jsfiddle, so you can see what I'm seeing.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/RobinHerbots/Inputmask/issues/1419#issuecomment-254388585,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AATb7zt_PvNSwbTC7k62RF6jfhEMDgHNks5q1C05gaJpZM4KWeMm
.

That makes much more sense then. Thank you so much for your patience. And thank you for inputmask--it's fantastic!


Can you support Chinese?

Godot version
3.3.2
System information
win10 64
Issue description
I wrote it in the red circle.
Chinese.
The following error occurred during Android debugging

Steps to reproduce
Automatically load gd
Minimal reproduction project
No response

Is it Android specific or happening on all exports / when running from editor?

"no space left"

This is UTF-8 parsing error. Might be overlong encoding (Is Android using some non-standard UTF-8 variant, like CESU-8?), or it's trying to parse non UTF-8 string as UTF-8 somewhere.
It's hard to tell what's exactly going wrong without a reproduction project.

Is your project created using the unofficial Chinese Power-up build (中文加强版)?
The official build does not support using arbitrary Unicode as identifier yet, see https://github.com/godotengine/godot-proposals/issues/916

您的项目是使用非官方的中国电源生成（新版）创建的吗？
官方版本不支持使用任意的 Unicode 作为标识符， 见戈多工程 / 戈多特建议 #916

Yes, third-party Godot.
Only the red circle does not support Chinese.
Other places are fine.
The red circle is.
Chinese characters. The Android simulator will flicker and report an error.
Just change it to English.

Then this is indeed due to using characters outside the 0-9a-zA-Z range as name of AutoLoad, which the official build does not allow. (so I don't think it's kind of an issue.)
In case anyone wants to test, here is an MRP: ChinesePowerUp.zip. Export this project to any platform, not limited Android, and run the binary, and you'll get the same error log.

因此，这确实是由于使用范围外的字符作为自动加载的名称，这是官方构建不允许的。（所以我不认为这是典型的问题。0-9a-zA-Z
如果有人想测试，这里有一个MP：中国电力.zip。将此项目导出到任何平台，而不是限制 Android，并运行二进制文件，您将获得相同的错误日志。

Indeed.
I made the same mistake on the exe platform.
I use the enhanced Chinese version.
It seems that.
I'll wait for the boss to see if I can support it.


India: Rajasthan silently failing.

It looks like updating Rajasthan is silently failing with no flag or warning anywhere. This makes me thing that other regions may also be failing with likewise no warning.

https://github.com/epiforecasts/covid-rt-estimates/blob/master/subnational/india/cases/national/runtimes.csv
It looks like the runtimes are recording 1 in the timings rather than a timeout - the log is showing it timesout in the traces but when it reloads that csv to get the runtimes it is seeing a time of 1 so recording it as a success. I think this is probably originating inside the epinow timing.
2020-12-08 08:59:58 INFO Completed regional estimates
2020-12-08 08:59:58 INFO Regions with estimates: 34
2020-12-08 08:59:58 INFO Regions with runtime errors: 1
2020-12-08 08:59:58 INFO Runtime error in Rajasthan : Rajasthan: model fitting was timed out or failed -
2020-12-08 08:59:58 INFO Saving timings information to : subnational/india/cases/national

That is a runtime of 1 minute. This is due to an error and not  a time out hence the 1 minute runtime.

ahh that's why it's recording a success - I think originally failures returned no runtime.

Strange - there's no error in the log until +1 hr and 4 seconds when it died with a timeout.

I can't find any mention of it in the log apart from it being initialised. I see lots of fitting issues in general in India though that have not been clear.

2020-12-08 08:29:38 INFO Completed estimates for: Tamil Nadu
2020-12-08 08:59:58 ERROR Rajasthan: model fitting was timed out or failed -        <---------- this is before it starts its summary print of the outcome
2020-12-08 08:59:58 INFO Completed regional estimates  <------------ this is the summary of the outcome
2020-12-08 08:59:58 INFO Regions with estimates: 34
2020-12-08 08:59:58 INFO Regions with runtime errors: 1
2020-12-08 08:59:58 INFO Runtime error in Rajasthan : Rajasthan: model fitting was timed out or failed - <---------- this is the repetition of the problems from the individual jobs
2020-12-08 08:59:58 INFO Saving timings information to : subnational/india/cases/national

I think all the general stan warnings are just background noise when I look at the logs and we are not reporting on them unless we are looking at why a run failed and it's of note

Which is a problem unfortunately as they are pretty crucial for understanding how estimates are working 🙃 .

if this is a time out then it looks like the status needs updating to catch these. I am not clear how the code as implemented in EpiNow2 would give a run time of 1 minute for something that ran for an hour?

I agree it's very strange

ok tracking back further - runtime.rds is 1.

Look through all of the other estimates this appears to be the only one with with a problem (though I haven't manually checked the global estimates so there might be a problem hiding there).

The data in India is quite problematic so model issues aren't a shock. It would just ideally be easier to find them than by using manual inspection.

that is really strange. I see what you mean about the failure. It's odd that the timings crash out almost instantly but it keeps the future thread locked until the end.
The logic for the status / runtimes csv is this:
 if (is.null(outcome[[dataset_name]][[subregion]])) {
        runtime <- -1
        dataset_counts$failures <- dataset_counts$failures + 1
      } else if (is.finite(outcome[[dataset_name]][[subregion]])) {
        runtime <- outcome[[dataset_name]][[subregion]]
        dataset_counts$successes <- dataset_counts$successes + 1
      }else {
        runtime <- 999999
        dataset_counts$timeouts <- dataset_counts$timeouts + 1
      }

Failures are expecting no result to be present. I wonder if this changed with the upgrade of epinow.

Small scale example of model fitting alone: https://gist.github.com/seabbs/b61fe9a88a2b5ddf2ebb79242bd0045e

Using linked example code I saw a run time of 40 minutes (note this is using an initial cumulative fit which may slightly speed up fitting). Output is below. In general this data looks extremely smooth which makes me very sceptical.

Evaluating this it looks like the issue may have been in slightly misspecifying the prior on the length scale of the gaussian process (of interest @sbfnk) to be to long an truncated to high (mean of 21 days and lower bound of 7 days by default). Updating to this: gp_opts(ls_min = 3, ls_max = 28, ls_mean = 14) gives me a runtime of 10 minutes.
The only way I can see to test this is to do large scale batch runs and assess runtimes and fitting criteria.


Add files via upload

this is certh's mail addresses and website
certh is a research institute in Thermi,Thessaloniki,Greece which coopartes with universities like Aristotle University of Thessaloniki and else.
https://www.certh.gr/
http://mail.iti.gr/
https://mail.iti.gr/squirrel/src/login.php
and my e-mail is
<EMAIL_ADDRESS>

also this is Technological Educational Institute of Western Macedonia domain
Technological Educational Institute of Western Macedonia in Kastoria,Fourka,Greece is a technological educational institute in Greece and i'm student in Technological Educational Institute of Western Macedonia in
international trade department
teiwm.gr
<EMAIL_ADDRESS>https://kastoria.teiwm.gr/intrade/

@geormich55 Thank you for your request!
Regarding the iti.gr domain:
The research institute in Thermi, Thessaloniki, Greece collaborates with different universities. Do students and teachers associated with this institute have email addresses assigned to them by the universities where they were enrolled initially?
If yes, why don't they use their initial universities email addresses instead of iti.gr?
Why do you want to add both domains to this repository? If you have an email address from one institution, like the Technological Educational Institute of Western Macedonia, you're eligible for a free subscription. Having more than one subscription and sharing it with others violates the license terms. Please avoid this practice.
Please reply and remove iti.gr from your request, if you can. If you cannot, I can do it for you, if you wish.

@geormich55 You might miss my previous comment. Please react.

The research institute in Thermi, Thessaloniki, Greece collaborates with
different universities. Do students and teachers associated with this
institute have email addresses assigned to them by the universities where
they were enrolled initially?
If yes, why don't they use their initial universities email addresses
instead of iti.gr?
Hello..they don't have email addresses from these universities..
Answer:
This is an organisational topic of bad management..
Why do you want to add both domains to this repository? If you have an
email address from one institution, like the Technological Educational
Institute of Western Macedonia, you're eligible for a free subscription.
Having more than one subscription and sharing it with others violates the
license terms. Please avoid this practice.
Answer:
I didn't do this for sharing it with others..I only did it for using it
myself because the mail server of the technological educational Institutute
of western Macedonia sometimes doesn't connect or something..
This is why I would like to create me a free subscription with the iti
email as a priority..
If this can't be done explain me why..
If it can be done remove the university's mail and keep iti mail if it's
possible..
I can remove it..
Simply would like to have iti as a free subscription mail..
GM

@geormich55 Hello!
I am sorry, I have to reject the request. Your reply above does not formatted well, thus it's hard to read. If I understood your point correctly, you want to use the ITI email for a free subscription, because another university where you are enrolled does not have a properly configured mail server.
The same applies to other students who use ITI email addresses instead of email addresses assigned to them by the universities where they are enrolled to.
I see some risks in adding ITI to the trusted list, because it's not an educational organization, first of all. We cannot ensure that only students and teachers will use the free subscriptions.
Also, it would be beneficial for everybody if you and other students and teachers insist that your universities where you are enrolled or employed configure their mail systems properly. It's ridiculous to have misconfigured mail system and bad management at a university.
At a last resort, you can simply apply for a student license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
I have to close the request without merging. Thank you for your understanding!


adding uncased NER & dependency parser

options are added for app.py to

use a custom model, optimized for uncased text, for NER.
do dependency parsing.

Is this PR still relevant at all?

I believe we’re not currently using information from the dependency parser,
but there might be a need to use uncased model on auto-detected transcripts
(though a default cased model is able to work reasonably well on uncased
transcripts).
On Thu, Mar 9, 2023 at 11:15 PM Keigh Rim @.***> wrote:

Is this PR still relevant at all?
—
Reply to this email directly, view it on GitHub
https://github.com/clamsproject/app-spacy-nlp/pull/21#issuecomment-1463229905,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARDHMWV6OBDYACL44ICY3STW3KTGPANCNFSM534IOU7A
.
You are receiving this because you authored the thread.Message ID:
@.***>


How to stroke text?

Hi,
I'm writing a Lua language binding for NanoVG, I'm trying to implement context.strokeText, so I wondering how to stroke a text. currently NanoVG can only fill text. Is that right? then how can I implement stroke?

Have you considered using "signed distance fields"?


Add possibility to change number of retries

If a capture went wrong, it retries many times... It could be very nice to be able to recapture only "X" times :)

Fixed by https://github.com/ail-project/PyLacus/commit/eb0379722d8c64d0d498f8bad15360c5b4fe8420


Risk analysis workshop with Julien

Hey team! Please add your planning poker estimate with Zenhub @heroinedor @JDixmier


 The Bitmap that's made available by PdfBox-Android decodes the images to mostly black, with various non-black artifacts.

Environment details:

PdfBox-Android version: [e.g. <IP_ADDRESS>]
Android API version: [e.g. API 30]

Additional context
If single-bit images aren't supported, then I'd like to implement that on my end. I assume there shouldn't be much involved with choosing black or white based on bit values. However, I'm not sure how to access the image byte data.
I have a PDImageXObject called image. When I call image.createInputStream(), the resulting input stream's size doesn't seem to match my expectations, given the width, height, and bit depth of the image. Is that stream Flate encoded? If so, how can I decode it?
I tried
ByteArrayOutputStream outStream = new ByteArrayOutputStream();
Filter flateDecoder = FilterFactory.INSTANCE.getFilter(COSName.FLATE_DECODE);
DecodeResult result = flateDecoder.decode(imageInputStream, outStream, new COSDictionary(), 0);

But this failed with an error:
E/PdfBox-Android( 5865): FlateFilter: stop reading corrupt stream due to a DataFormatException
D/flutter_pdf_renderer( 5865): Error while processing a page: java.io.IOException: java.util.zip.DataFormatException: invalid stored block lengths
E/flutter_pdf_renderer( 5865): Failed to load PDF file as PDDocument: java.util.zip.DataFormatException: invalid stored block lengths
W/System.err( 5865): java.io.IOException: java.util.zip.DataFormatException: invalid stored block lengths
W/System.err( 5865): 	at com.tom_roush.pdfbox.filter.FlateFilter.decode(FlateFilter.java:57)
...

image.createInputStream() is the decoded stream.
Desktop PDFBox works with that file.

@THausherr can you elaborate on what your response indicates? Are you adding more info to this ticket, or are you saying there's a way to get these images working with PdfBox on Android?

My first line means that you're not using the correct method if you wanted the compressed stream.
My second line means that PDFBox for Android somewhat mirrors the code from PDFBox for desktop. So this means that the desktop code is correct, but that something went wrong when porting it to Android.
It would be interesting to know whether image.createInputStream() for /Im31 returns all black (zeroes) or mixed stuff? If it returns all zeroes then the bug is with decoding the stream. If it isn't all black then the bug is with converting the stream to an image object.

@THausherr - Can you tell me what's in that stream? If I call image.createInputStream() am I getting raw pixel values? or encoded pixel values (e.g., JPEG)? or a PDF encoded stream?

I took a look at the PDF in question, and it looks like the image is using the ICCBased colorspace. That isn't supported in PdfBox-Android, and DeviceRGB is used as a fallback. Since the image isn't RGB, you end up with a bit of random color at the top of the image while it's trying to treat the image as RGB and making a mess of it.
Adding ICC support would be a pretty big effort, but I can test and see if it's possible that treating single component ICC images as DeviceGray colorspace would be good enough to show the image. At the very least, an unsupported colorspace should probably fail and not be displayed instead of showing garbage

@TomRoush getting a reasonable fallback here would be good. DeviceGray probably works for this particular case.
In general, we're running into color space issues for many bitmaps, and even for things like fill and stroke colors.
Can you add the ability for developers to at least read the parsed definitions for all the color spaces, such as ICC-Based, Lab, etc? I think in most cases, we could implement the appropriate interpretation of those color spaces on our end, at least for the time being. But right now we're totally blocked by PdfBox-Android internally choosing to ignore the real color space and then replace it with DeviceRGB.
When we access an unsupported color space, we don't seem to get an exception, or any indication that PdfBox-Android is giving us a different color space. We only get a log message to that effect.
So, ideally, I could query the color space and get a data structure with the relevant details. Maybe there's no toColor(components) method on that data structure, but at least we'd have access to the information to implement that ourselves.
And we're willing to fund some of the work on this because it's a high-priority blocker for what we're building.
CC @kirkbyo

@THausherr - Can you tell me what's in that stream? If I call image.createInputStream() am I getting raw pixel values? or encoded pixel values (e.g., JPEG)? or a PDF encoded stream?

raw pixel values

@THausherr and those are pixel values encoded at the reported bit depth? i.e., 1 bit per pixel? So when I pull a byte out of that stream, I'm getting 8 pixel values?

This is fixed with #487. PDICCBased was added and it should handle single bit, RGB, and CMYK images.
Rendered image:

There's another slight issue with the rendering that isn't fixed yet, the horizontal lines. The PDF is made up of a series of images, top to bottom, and in between each image in the PDF there's 1 row of pixels in the rendering where the alpha is 0. Most likely a rounding issue somewhere, but for now you can get around that by rendering in RGB instead of ARGB


python3 libary lxml not installed

Hi. Thanks for this role.
I found out that the pythoon libary lxml is required to be present at the remote host. But will not be installed with this ansible.
Is there a reason for it? Otherwise I would apreciate if we can add it to this role!
TASK [fisheye : patch /opt/atlassian/fisheye/content/WEB-INF/web.xml] ***********************************************************************************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: ModuleNotFoundError: No module named 'lxml'
fatal: [atlassian-fisheye.wingcon.com]: FAILED! => changed=false
  msg: Failed to import the required Python library (lxml) on fisheye-test's Python /usr/bin/python3. Please read module documentation and install in the appropriate location. If the required library is installed, but Ansible is using the wrong Python interpreter, please consult the documentation on ansible_python_interpreter

I see you say you have a additional role to install python stuff:
https://github.com/alvistack/ansible-role-fisheye/blob/master/ansible-galaxy-requirements.yml#L26L29
nevertheless, i think it would be nicer if such dependencies could be solved right here in the role. What do you think?

First we default everything with python 3 only, so you need to ensure target already have it.
Next we default with custom pip3 installation, in order to minimize OS package dependency issue (some could update but some can’t); again you need to ensure it before running this role.
Finally once above issues solved, now we could install the python lxml package; BTW it is an Ansible-specific requirement on python but not Fisheye’s requirement...
Therefore lxml should be handled by python role but not here. Please simply run ‘molecule converge’ and dependency will be download automatically ;-)


Migrate from Next.js Pages Router to App Router

Ticket
Resolves #66
Changes
This is purely an architecture migration and nothing functionally should have changed.
Pages Router → App Router:

pages/_app.tsx → app/[locale]/layout.tsx
pages/index.tsx → app/[locale]/page.tsx
pages/health.tsx → app/[locale]/health/page.tsx
pages/api/hello.ts → app/api/hello/route.tsx

Internationalized routing and locale detection:
App Router doesn't include built-in support for internationalized routing and locale detection like Pages Router.

Added Next.js Middleware (src/middleware.ts) to detect and store the user's preferred language, and routing them to /es-US/* when their preferred language is Spanish
Added i18n/server.ts to contain server-side configuration for the next-intl plugin. This makes locale strings available to all server components

Context for reviewers
See the related ADR.
Testing

⛔ I'm going to move this back into a Draft state. I encountered a blocker that is making me reconsider whether it's still too soon to migrate to React Server Components (AKA App Router).
Context
The main benefit of using App Router is to use React Server Components (RSC), which are a new feature.
First, it may be helpful to provide context on the RSC and Next.js release timeline so far, and my thinking for when it felt appropriate to finally perform a migration:

React Server Components (RSC) are currently in React's canary release. Next.js is the earliest adopter and main group evangelizing and building support for RSC (in collaboration with the React team). Next.js includes the React canary release to provide support for server components and server actions. Canary releases are said to be "stable for frameworks".
Next.js released "beta" App Router support in version 13.0 in October 2022, and it lacked functionality such as server actions and mutations (forms). The Nava template didn't migrate at this time because App Router was considered "beta" and it seemed like there were still a lot of open questions about RSC that needed worked out (data fetching, caching, mutations, best practices).
In version 13.4 in May 2023, Next.js moved App Router from "beta" to "stable", and introduced an "alpha" release of server actions. The Nava template didn't migrate at this time since there were still unstable RSC features.
In version 14.0 in October 2023 (a year after 13.0), Next.js moved server actions from "alpha" to "stable".

Blockers and concerns
With server actions now stable, and an entire year for teams to kick the tires of server components, it appeared like things were in a better place for the Nava template to finally migrate to the App Router architecture. For the most part, I think that's still true, but one thing this migration PR has surfaced is that not all tooling in the React ecosystem has made the jump.

The primary blocker right now is that Jest and React Testing Library lack support for async server components. This prevents testing any page that includes data fetching logic. The Next.js recommendation at the moment is to use end-to-end testing for async server components. The template currently doesn't provide an end-to-end testing framework.
Not a blocker, and partially resolved (by me), but the React USWDS component library didn't support server components, and only barely does. It ultimately needs further changes in order better support server components — right now every single component is a client component, and the entire library is included in the client-side bundle.
Next.js continues to iterate its caching strategy. Currently, they extend the native fetch API and cache by default, which is confusing and controversial. They're now acknowledging this and it sounds like they'll be moving away from this in a future release. This isn't a blocker, but another signal that things are still not stable.

Given the above, I'm leaning towards continuing to hold off on migrating to the App Router. I think the following need to happen before we reconsider:

Jest and React Testing Library support async server components;
and/or the template provides an example end-to-end test of a Next.js page.

My personal preference would be to continue to have the ability to test within Jest since these tests are often faster and don't require running the entire application.

Hey @sawyerh! Thanks for writing this up 😄 Your timeline seems accurate. I wanted to share some other thoughts.

Next.js is the earliest adopter and main group evangelizing and building support for RSC (in collaboration with the React team).

Redwood and Remix are also moving in the RSC direction. Here's Redwoods notes. There's other more experimental frameworks like Waku being released.

The template currently doesn't provide an end-to-end testing framework.

We hope to continue to improve our testing docs and examples here for unit, integration, and E2E test. We also have this RFC open around testing.

Right now every single component is a client component, and the entire library is included in the client-side bundle.

I wanted to mention that, while this isn't the most ideal state, it's also (IMO) still an improvement over the App Router. For example, you getting streaming SSR by default, which is very powerful. I made a video about this here.

This isn't a blocker, but another signal that things are still not stable (around caching).

All of the existing caching work is stable and functional. We just want to improve the DX further. There will be smooth migrations from the existing DX today and possibly some simplified ways of authoring code around caching when ready. We will share more details and plans are figured out. But that doesn't mean it's unstable today!
Hope this helps 🙏

@leerob Really appreciate you jumping in here to add more context, thank you. After posting this last night and sharing with colleagues, I think we've identified a way to move forwards with a migration to App Router and decouple "controller" and "view" logic and unit test async server components with mocked external services, similar to how we were testing getServerSideProps before. Thanks again!

@leerob @delbaoliveira Feel free to ignore me here, but wanted to follow-up on my previous comment. Seems like the original pattern we were thinking is currently prevented by the Next.js TypeScript plugin. Added a comment to this existing Next.js issue with more context.

Might it make sense to rename the top-level folder from app to something else?

@jcq I had a similar question in the spec's Open Questions, and there's some discussion there. I was going to leave it as is for now since the infra template expects app/ out-of-the-box. We could rename in a follow-up though (thoughts on a better name?).


Unexpected token '??='

What version of Remix are you using?
1.6.6
Steps to Reproduce
Run npx create-remix@latest
Expected Behavior
A new remix project is created after answering the questions
Actual Behavior
After a few seconds of loading I'm met with the following message Unexpected token '??='

I was able to get around this by using v1.6.5 - npx<EMAIL_ADDRESS>

@alexketchum What node version are you running?

14.16.1
I see that I need at least 14.17.0. I'll try updating node and running it again.

I just tested 14.20 and I'm having the same error. Seems like we accidentally dropped node 14 support here:
https://github.com/remix-run/remix/commit/befce4a3a7e0218183db6feb9bf4cfc739883e42#diff-08f1d402b3b2041674b10a474187115744ce554aa54e5055f36a0174d95ad15cR211

This was an oversight and should be addressed shortly by the team, thank you for reporting this 🙏🏼

Fixed in 1.6.7


Creating a Sequence fails

When creating a Sequence  the server returns 500 internal server error.

Please describe the exact sequence definition that you tried. I can't reproduce the error.

I also need the stack trace.

We can close this item.  I was using incorrect parameters.  User error


Answers are not bleached before being stored in answer_store

Expected behaviour
When i post malicious content to the application then this should be sanitised before being stored and displayed on subsequent pages.
Actual behaviour
The malicious content is being stored in the answer-store and displayed on subsequent pages
Steps to reproduce the behaviour
On the census_household.json In the persons name enter <img src="https://www.google.co.uk/images/branding/googlelogo/2x/googlelogo_color_120x44dp.png"/>

@jonnyshaw89 Just checked and it appears that at least some answers are being bleached before storing in the answer store. This happens during validation in schema. The bleach.clean call is what causes everything after the '<' when entering a household name to be lost (see #695). Not sure if there's a way of telling the clean method not so strip content, but rather encode the value?

Looks promising... https://docs.python.org/3/library/html.html#html.escape The second argument allows escaping of quotation marks.

Closing this as we bleach values on input from widgets.


Error in saveRoll() with Midi-QOL

Whenever I make a roll in DND5E with Midi-QOL, I get the following error:
Foundry.js:753 TypeError: undefined. Cannot read properties of undefined (reading 'length') [Detected 2 packages: dice-stats, midi-qol]     at PLAYER.saveRoll (dice-stats-player.js:96:55)     at dice-stats-main.js:264:24     at Array.forEach (<anonymous>)     at DiceStatsTracker.addRoll (dice-stats-main.js:263:15)     at Object.fn (dice-stats-hooks.js:36:38)     at hookCall (utils.js:4863:16)     at asyncHooksCallAll (utils.js:4815:10)     at Workflow.WorkflowState_RollFinished (workflow.js:1513:9)     at async Workflow.performState (workflow.js:478:21)
It looks like saveRoll expects a single argument, msgRollInfo, but where it's being called from the Midi hook, it's being passed in three arguments.
Unrelated to this issue, I noticed in the readme for the module that when merged cards are enabled in Midi, you have to do some hacks to try to guess at the user who actually made the roll that aren't reliable. I don't know if the Midi workflow has changed since you last looked into this or if I'm just misunderstanding what's required, but it seems like workflow.chatCard.user will give you the user who made the roll, and workflow.chatCard.user._id could be sent in to addRoll as the owner.

Damn, forgot to update the midi qol support when I did the refactoring.
Will be fixed in next release

Should be fixed with 1.18.1 release. Let me know if your still seeing an issue after updating


Uplift to ES6 classes and rollup bundling?

The codebase right now is actually extremely well organized and would be relatively easily uplifted to using ES6 classes, which would make every class a potential module that can easily be used by other open source projcets, or customized for opentype experiments like trying out new tables as part of a PoC prior to official proposal =)

Aren't ES6 classes the new "bad part" of JS ?

Subjectively: to some. Objectively: no.
As with everything in JS, you need to know how it works before you use it, because ES6 classes are not the same as "class based object oriented programming" in things like C# or Java. That's not "good" or "bad", that's just "welcome to JavaScript" (yet again - there are so many things in JS that you're going to do wrong if you think it does things like other languages you're familiar with).
In this case: the class syntax is an incredibly powerful way to implement prototype inheritance without  the pages and pages of code to do that well. It offers single word syntax where before we needed 10 lines of codes, and abstracts away all the "safety code" we need for raw prototype inheritance to ensure the right prototype's properties are accessed. So as long as your write your codebase fully aware of that, there is literally nothing bad about ES6 classes: they make the code infinitely more legible, and in this codebase specifically, infinitely easier to write things like plugins that introduce additional table parsing by making it trivial to write your own X implements SFNTTable or X implements CommonTable.

Isn't it precisely the point: keywords like class and implements make things look like java or C++ classes, and everybody wants to use them because it feels like good old java OOP, but nobody really understands what it is exactly.
Btw, :+1: for plugins.

@fpirsch If you already use prototype, class is a great way to organize this code in simple way.
You can claim that you can't create truly multiple inheritance, but on the same time many will claim that inheritance is bad and multiple inheritance is multiple bad.
Another thing, many IDEs do really well autocomplete only from using the class.

Personally, I like ES6 classes. As @oriSomething mentioned, we're using prototype-based inheritance already anyway, so it's just renaming some things and cleanup.
What I don't like is Babel, but yeah, necessary evils and all that....
My favorite feature of ES6 is to use let and const everywhere instead of var. So, once we're using Babel I would like to go through the code and do var replacement everywhere.

@fpirsch we can't control what TC39 picked as keyword, so it really shouldn't have been class, and that ship has sailed: that's what it is and that's what we now use. Does that invite mistakes? Of course, but only if you don't bother to first learn how to use a language feature before you start implementing with it, much like a lot of other JS features. For instance, a ton of people also mistakenly use JavaScript arrays as if they're associative arrays, instead of using plain objects, because that's what the languages they came from (PHP, Python) allow. Does that make JavaSript arrays bad? Not really, in both cases the problem is the programmer not taking the time to make sure they understand the language they're using. JS classes are still a new feature, so not a lot of people (relatively) have actually looked up how they work, and a lot of people just go "I know what this is" and then happily go on their way implementing things completely wrong, based on assumptions that don't hold. That's not the language's problem, that programmers cutting corners and getting punished by the language =)
@fdb I like babel a lot more now that it can be a quick step somewhere in the build process. With some very simple node scripts (especially since no react/jsx/etc stuff needs to be transpiled) three separate tasks for rollup, babel, and an aggregated task for "build":"npm-run-all rollup babel minify" makes things pretty nice.

@fdb classes works on Node ≥ 4.x and every major browser exclude IE. Personally, I see very little problem with it, since there is already usage of bundler because the project is written with common JS. For Node, there is no babel need.

Yeah it's a bit odd that http://caniuse.com/#search=classes does not list EDGE - IE would be the main stickling point really since IE11 has not yet been made obsolete, everything else should work fine (that said, having a legacy version of a library in pure ES5 is nice for things like older versions of Android phones, windows users who through no fault of their own are still forced to work on computers with old IE, etc)

Hi 😉
I like the idea but I need an ES5 js file in the end for compatibility purposes... So 100% for ES6 if you could add that 🙏

I think we can make two versions if we need to: a version that is just rollup'd, in native ES6, and an ES5 file compiled with Babel. Oh and minfied versions of those two as well.

I'm doing this now as a series of PRs:

First step is to move to ES6 modules #290.
Next step is to replace all MyClass.prototype.fn to ES6 classes.
I want to make sure that I have a clean bill of health from ESLint (with the Airbnb preset).

As a nit: instead of going "from var to let/const", just go to "var and let and const", picking the correct keyword in the correct setting so as not to incur overhead that is irrelevant for the scoping and access required. var for function scoped variables, let for block scoped variables, and const for when you need a locked alias.
Ignoring the fact that var has a damn good reason to still be used is the weirdest decision people make when they uplift from ES5 to ES6 =)

@Pomax Why is var still relevant?

Things related to performance stopped being relevant in latest versions
Using it as reassign global is an anti pattern that better avoided on es-module based version
Using it for hoisting is easily can lead to bugs

So, why actually using it? I admit I don't understand what you mean by saying being the lowest-overhead alias allocator. If you could please explain it?

Short answer: because it's part of the language. They're not suddenly irrelevant because people have decided to ignore which each of the three allocators do, in the same sense that the fact that templating strings exist doesn't mean every string should now be a templating string, including strings that don't use templating.
So, as quick JS allocation explanation, var, let, and consts do similar, but different, things:

var: function scoped. Declared anywhere in a function, this variable persists until the end of that function. May be reassigned, may be redeclared, may be used redeclare. Redeclaring a variable with reassignment leaves that variable intact.
let: block scoped. Declared in a function, this variable persists until the end of that function. Declared inside a block inside a function, this variable persists until the end of that block only. May be reassigned, may not be redeclared, may not be used to redeclared.
const: function scoped. Declared anywhere in a function, this variable persists until the end of that function. May not be reassigned, may not be redeclared, may not be used to redeclare.
section allocator number 4: not using an allocator. This invokes the var allocation mechanism, with global scope.

For all three: if declared in the global execution context, all three simply persist.
So to address your questions:
1: performance never stopped being relevant in codebases with a large amount of activity, such as a densely packed binary interpreter, which OpenType parsers are. The following code, for instance, wastes time on allocation and garbage collection:
cmap4.segments.forEach(segment => {
  let start = segment.start;
  let end = segment.end;
  // compute delta or something similar
});

Advocating using let here, rather than vars declared and maintained outside of the loop is intentionally wasting a proportionally huge amount on time that can be spend on actually doing work rather than overhead:
function computeDeltasOrSomethingSimilar(segments) {
  var s=0, e=segments.length, segment, start, end;
  while(s++<e) {
    segment = segments[s];
    start = segment.start;
    end = segment.end;
    // compute delta or something similar
  }
}

In this case, the fact that there is a "nicer" allocator available can actually promote worse code by glossing over the fact that overhead still matters. In some codebases, that overhead is irrelevant given how much "regular" code kicks in, but in a binary parser like this, all those bits of unnecessary overhead add up fast over thousands of cascaded offset lookups.
2: this has nothing to do with using var, and everything to do with properly scoping your code. Modules in which execution context is automatically rescoped are an obvious safeguard, but are not more than a safeguard.
3: hmm, you may be thinking of something else: "hoisting" does not apply to variables, only to functions. You might be thinking of shadowing? E.g. function test() { ...; x = 5; ...; var x = 'moo'; ...;}. No hoisting occurs here, we're simply seeing two var allocations. The first creates an implicit var with global scope, the second then shadows that global variable with an explicit var that is function scoped from that point on.
Real hoisting kick in for functions, called hoisting because they get "hoisted up" from wherever you wrote them to the top of the execution context, before execution actually occurs. No matter how far down you wrote them, the interpreter will always first perform named function binding before actually running your code:
(function test() {
  console.log(test2());

  function test2() {
    return "functions are hoisted before function execution, so you see this text printed.";
  }
})()

The rollup bundling thing is now done!

Thanks @Pomax for the explanation on vars. The main advantage for me in using let and const is to check if I've scoped variables correctly.
I agree on the performance considerations for this library. However, as I understand it, nothing blocks you from moving up a let outside of a looping block and at the top of the function, which would eliminate the overhead.
In the cmap example, I think the main overhead would be the anonymous function, not necessarily the scoping. I've tested this in jsperf and it seems the overhead of the tightly scoped let is negligible: https://jsperf.com/let-scoping-performance/1

Please let me know if I'm missing something 😉

@fdb nice: real numbers always trump "what the spec suggests should happen"!

I don't see why the spec would say anything different. 'let' variable holders are not in heap and thus garbage collected, but are on the stack. And stacks are fast, modern processors are very stack optimized and a good preprocessor bundling the code on release can uplift these variables anyway.

@fdb @Jolg42 I'd like to help with this. I'm thinking of doing the prototype -> es6 classes conversion.
Will I need to make any changes in the pipeline? or will making changes in src be enough?


rosa version command fails in the openshift CI

rosa version command fails in the openshift CI with below reason.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x19 pc=0x259f7e0]

goroutine 1 [running]:
github.com/openshift/rosa/cmd/version.(*RosaVersionOptions).BindAndValidate(0xc000bbfbb8?, {0x8c?, 0x7a?})
	/go/src/github.com/openshift/rosa/cmd/version/options.go:58
github.com/openshift/rosa/cmd/version.NewRosaVersionCommand.RosaVersionRunner.func2({0x0?, 0x0?}, 0x0?, 0x0?, {0x0?, 0x0?, 0x0?})
	/go/src/github.com/openshift/rosa/cmd/version/cmd.go:64 +0x45
github.com/openshift/rosa/cmd/version.NewRosaVersionCommand.DefaultRunner.func3(0xc000ef2900?, {0x4326fe0, 0x0, 0x0})
	/go/src/github.com/openshift/rosa/pkg/rosa/runner.go:29 +0x10f
github.com/spf13/cobra.(*Command).execute(0xc00024c600, {0x4326fe0, 0x0, 0x0})
	/go/src/github.com/openshift/rosa/vendor/github.com/spf13/cobra/command.go:987 +0xaa3
github.com/spf13/cobra.(*Command).ExecuteC(0x4288980)
	/go/src/github.com/openshift/rosa/vendor/github.com/spf13/cobra/command.go:1115 +0x3ff
github.com/spf13/cobra.(*Command).Execute(...)
	/go/src/github.com/openshift/rosa/vendor/github.com/spf13/cobra/command.go:1039
main.main()
	/go/src/github.com/openshift/rosa/cmd/rosa/main.go:109 +0x93

Looks like something to do with the latest changes that went into version subcommand: https://github.com/openshift/rosa/pull/1953
Here is the step that we are using in openshift CI: https://steps.ci.openshift.org/reference/openshift-qe-rosa-loaded-upgrade
And we use latest image tag to fetch rosa cli in our CI config
  rosa-aws-cli:
    name: rosa-aws-cli
    namespace: ci
    tag: latest

Seeing the same runtime error in https://github.com/NixOS/nixpkgs/pull/317409


[FEATURE] Please create homebrew package for OSX

While I'm personally fine compiling this with cargo I've noticed adoption tends to be much easier for folks on osx if you provide a homebrew package to give you something like brew install rage (which is both useful and hilarious as a command... 😜  )
Could we have one? (or I could take a shot at a formula for it though never done it myself.).
thanks!
Daryl.

I've merged a Homebrew formula that builds and installs rage and rage-keygen. See the readme for usage instructions.
I wasn't sure what to do with rage-mount. It's not clear to me whether that should be some kind of optional thing inside the rage formula, or whether there should be a separate rage-mount formula. I've left it out for now; if you particularly want it and/or know how this should be handled, please make a new issue.

Hmmm... I just did a brew update but I'm not seeing it available from homebrew. And a brew search rage came back empty. Is there some sort of review process that is needed from the brew side before it "goes live"?
I have to admit to only using rage and rage-keygen at the moment (I also want to use it as a crate instead of gpg for a small project I am working on to replace jrnl.). I think you're most likely fine to start without rage-mount at the moment as I imagine anyone sophisticated enough to use it would be fine using cargo to build the stuff themselves.  =]
ciao!

Doh, just realized you meant merged it into the codebase rather than the Homebrew formulas available. Lemme take a look at how to get that into homebrew master (I should have figured that out long ago anyway) and go from there to help out.

Did you want this available in homebrew-core or via the taps system (eg. brew tap str4d/rage ?).

The formula looks good though when I attempt to brew tap I am getting errors (inconsistent with what you'd expect using other brew formulae). I think in order to take advantage of brew tap you need to put the formula into a repo like str4d/homebrew-rage. I have the following fails on this (admittedly, not having set up taps myself in the past I had to go read up on it... =]
~ brew tap str4d/rage
Updating Homebrew...
==> Tapping str4d/rage
Cloning into '/usr/local/Homebrew/Library/Taps/str4d/homebrew-rage'...
ERROR: Repository not found.
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
Error: Failure while executing; `git clone https://github.com/str4d/homebrew-rage /usr/local/Homebrew/Library/Taps/str4d/homebrew-rage` exited with 128.
 ▲ ~ brew tap str4d/rage/HomebrewFormula
Error: Invalid tap name 'str4d/rage/HomebrewFormula'
 ▲ ~

Sorry, I said "See readme for usage instructions", but I should have copied them here:
brew tap str4d.xyz/rage https://str4d.xyz/rage
brew install rage

(I've done this the same way as @FiloSottile did his age formula so that the formual itself doesn't depend on GitHub.)

Did you want this available in homebrew-core or via the taps system

I figured taps system was fine for now, as this is still technically beta software.


No more freely available VS Community 2019 :(

To build requires VS 2019, but the community edition doesn't seem to be available anymore. Is it possible to get this working in the latest community version (VS Community 2022)?

You could retarget the project but a better option is just installing the VS 2019 build tools in VS 2022

Definitely should be possible. Visual Studio 2019 is not strictly required; the culprit is the cmake invocation: https://github.com/libgit2/libgit2sharp.nativebinaries/blob/master/build.libgit2.ps1#L132
I think that you could remove the -G entirely (leaving the -A). IOW:
Run-Command -Fatal { & $cmake -A x64 -D THREADSAFE=ON -D USE_SSH=OFF -D ENABLE_TRACE=ON -D "BUILD_CLAR=$build_clar" -D "LIBGIT2_FILENAME=$binaryFilename" ../.. }


wasm: update go-containerregistry to 0.7.0

resolves #36211
Signed-off-by: Takeshi Yoneda<EMAIL_ADDRESS>

@mathetake: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
4130e4d269db082b46a24a07ad317c0ade457762
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@mathetake: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-istiodremote-tests_istio
f7fb693ca9b19e60befaa37df7e41fd3d7f91e41
link
true
/test integ-pilot-istiodremote-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@mathetake: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
36da32e50802e55a57e084489628e43e11fa134c
link
true
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Elavarasi/callautomation alpha 2024 09 01

Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

merge


Is refCount() thread safe?

I wonder if this method thread safe or should it be safe?
See example below.
let apiRequest: Signal<Any, AnyError> = someApiManager.apiRequest().shareReplay(limit: 1)
DispatchQueue.concurrentPerform(iterations: 10) {
  let _ = apiRequest.observe { _ in }
}

In this case actual request was performed 7 times. If I change number of concurrent iterations with 500, then request performs also 7 times.
Than I added lock into refCount() and code above began work as expected (just one actual request on 10/500 observers).
public extension ConnectableSignalProtocol {

  public func refCount() -> Signal<Element, Error> {
    let lock = NSRecursiveLock() // <--- added lock here
    var count = 0
    var connectionDisposable: Disposable? = nil
    return Signal { observer in
      lock.lock(); defer { lock.unlock() }  // <--- used it
      count = count + 1
      let disposable = self.observe(with: observer.on)
      if connectionDisposable == nil {
        connectionDisposable = self.connect()
      }
      return BlockDisposable {
        disposable.dispose()
        count = count - 1
        if count == 0 {
          connectionDisposable?.dispose()
          connectionDisposable = nil
        }
      }
    }
  }
}

So. Is it a bag? Or it should work as I described in first example and I'm doing something wrong?

Yes, that looks like a bug. Great catch!
If you want to do a PR your fix looks good 👍


Check for childkey stake when setting weights

The set_weights extrinsic, as well as commit_weights are checking if the weight setting hotkey has enough stake.

commit_weights does it here: https://github.com/opentensor/subtensor/blob/dfbed6bb24cef95fced901597b4910de85705c2d/pallets/subtensor/src/lib.rs#L1446
set_weights does it here: https://github.com/opentensor/subtensor/blob/aa521f46fab9ba4702c752792ba4f5eb38df39fd/pallets/subtensor/src/subnets/weights.rs#L231

but both check using check_weights_min_stake, which uses TotalColdkeyStake. We need to modify check_weights_min_stake so that it considers parent/child stake.
Acceptance criteria:

[ ] Update check_weights_min_stake
[ ] Unit test or e2e test for each path that sets weights

PR: https://github.com/opentensor/subtensor/pull/924

Addressed by https://github.com/opentensor/subtensor/pull/935 (by @camfairchild )


setup-jest.js no longer included in package

This commit (https://github.com/rnmapbox/maps/commit/e7121889255463f4dfeb01dc1cd43b8d0f622a6f) added a list of files to be included in the package, and setup-jest.js isn't in the list. So if you configure Jest as per the README, running Jest throws this error:
● Validation Error:

  Module @rnmapbox/maps/setup-jest in the setupFilesAfterEnv option was not found.

README jest config:
"setupFilesAfterEnv": ["@rnmapbox/maps/setup-jest"],

Environment

Dev OS: macOS 13.0 (22A380)

Fixed by: https://github.com/rnmapbox/maps/commit/ee91c8137a5038db1a718590ea98e2f879104c56


Caching unit test

Unit test to prove that caching / not caching does as we expect.

I don't even know what exactly this refers to.. can we close it?


stopping service/container drops Docker socket file

On Debian when I systemctl stop container service it removes Docker socket /var/run/docker.sock which makes communication with Docker daemon impossible:
# docker ps
  dial unix /var/run/docker.sock: connect: no such file or directory.
  * Is your docker daemon up and running?

I can re-create socket file using systemctl restart docker.socket but it is quite inconvenient...

Possibly related to https://github.com/docker/docker/issues/7087


dependabot and renovate PRs against bugfix to release dependencies faster

Faster release of dependency updates

For passive library/component updates, the dev branch is the best choice since those updates soak a month to let any subtle bugs be discovered.
If there's a security critical issue for either of those tools, we can always cherrypick that PR into the bugfix branch.


Remove icon in the filebrowser window is always disabled

Hello, this is my conf for the option filebrowser of Jodit editor. The problem is that the remove icon in the filebrowser window is always DISABLED. How to enable it?
Jodit Version:  3.2.24
Code
      filebrowser: {
        ajax: {
          async: true,
          data: {},
          contentType: 'application/x-www-form-urlencoded; charset=UTF-8',
          headers: {
            Authorization: 'Bearer ' + authToken
          },
          processData: true,
          dataType: 'json',
          prepareData(data) {
            return data;
          }
        },
        // viewing a folder and return the list of files in it
        items: {
          url: 'http://<IP_ADDRESS>:3000/items'
        },
        // viewing a folder and return a list of sub-folders in it
        folder: {
          url: 'http://<IP_ADDRESS>:3000/folder'
        }
      },

Expected behavior:
Remove icon in the filebrowser window is enabled.
Actual behavior:
Remove icon in the filebrowser window is disabled.

Select file?)

I download the repo, since I'm integrating the editor with a react project and I modify the config in the file examples/components/From.js in this way:
state = {         config: {             readonly: false,             filebrowser: {                 ajax: {                     headers: {                         Authorization: 'Bearer ' + authToken                     }                 },                 items: {                     url: baseUrl + GetCustomItemsListPath                 },                 folder: {                     url: baseUrl + GetCustomFoldersListPath                 }             },             uploader: {                 url: baseUrl + AddCustomFilePath,                 headers: {                     Authorization: 'Bearer ' + authToken                 },                 imagesExtensions: ['jpg', 'png', 'jpeg', 'gif']             },         },         value: 'Test'     };
and when open filebrowser and select a file the remove button remain disabled, as the edit button.

You can add permissions option
filebrowser: {
                ajax: {
                    headers: {
                        Authorization: 'Bearer ' + authToken
                    }
                },
               permissions: null,
}

And edit and remove buttons will show everytime
https://xdsoft.net/jodit/doc/options/filebrowser/


Robert Hunt Don't pass thru 'create' msgs -- they are NOT new branch creation

Pull Request Test Coverage Report for Build 175

5 of 6 (83.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 29.845%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

enqueue/enqueueMain.py
0
1
0.0%

Totals

Change from base Build 172:
0.0%

Covered Lines:
77

Relevant Lines:
258

💛  - Coveralls

Pull Request Test Coverage Report for Build 175

5 of 6 (83.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 29.845%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

enqueue/enqueueMain.py
0
1
0.0%

Totals

Change from base Build 172:
0.0%

Covered Lines:
77

Relevant Lines:
258

💛  - Coveralls


Hyper protect config

Hyper Data protect database as service configuration

closing for now. testing it in personal repo.


Bump gearcmd version

JIRA: https://clever.atlassian.net/browse/IP-1002 (oncall)
Overview: Update s3-to-redshift to the latest version of gearcmd. This will be useful for heartbeat logging, which can notify us when long-running jobs are running so that we can debug live.
Testing: See: https://github.com/Clever/gearcmd/pull/50
Roll Out:
Remember to revert the .drone.yml change
CC: Assigning you @bstein-clever since Schimmy is busy AFAICT

LGTM

@bstein-clever thanks for the quick review


badges, metadata, test verbosity

Codecov Report

Merging #2 (47adf8e) into master (12628e1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #2   +/-   ##
=======================================
  Coverage   47.76%   47.76%
=======================================
  Files           6        6
  Lines         536      536
=======================================
  Hits          256      256
  Misses        280      280

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 12628e1...47adf8e. Read the comment docs.

/tag v0.1.1 bc14552


NO-ISSUE: Removing an Included Model should remove all references of it on the current model

This PR adds two new maps on the diagram data. The externalNodesByNamespace collects a list of external nodes that are present on the DRD, and the externalEdgesByNamespace has a list of edges that reference external nodes. The deleteImport now iterates over the nodes and edges lists erasing all references.

@gitgabrio @yesamer  I've added messages for removing included DMNs and PMMLs.

👍


WebDAV Co-authoring in Office

Hey, I have already fully setup a node server using this library, I am wondering how to make co-authoring work properly with Office files locally. (If even possible)
setLock saves a shared lock in DB for each connection and both users are able to open the same file and edit it. (unlike when the exclusive lock is set, that pops up an office modal that says that someone is currently working on the file)
How to recreate
User1 opens FileA in WebDav
User2 opens FileA in WebDav
User1 edits FileA and saves (which updates through WebDav)
User2 edits FileA and saves (which updates through WebDav)
Expected behavior
User 2 should be able to refresh the file contents after user 1 saves a new copy and automatically merge the changes. (expected office behavior)
Actual behaviour
The changes that User2 made overwrites the changes that User1 made.

I'm also checked that's not possible

yes


bitwise_arithmetic_shift_right: Presto and Velox return different results

Description
Query: select bitwise_arithmetic_shift_right(c0, c1) from (values (BIGINT '2374068159225518803', BIGINT '1554242466')) t(c0, c1);
Presto-0.289: Returns 138,188,954
Velox: Returns 0
Error Reproduction
No response
Relevant logs
No response

The result from Java does not make sense, worth checking with them on this one


Development build nuget packages aren't properly formed

Repro:
register-psrepository -Name xSharePointDev -SourceLocation https://ci.appveyor.com/nuget/xSharePoint
install-module -name xSharePoint -Repository xSharePointDev -RequiredVersion 0.12.673.0

Untrusted repository
You are installing the modules from an untrusted repository. If you trust this repository, change its InstallationPolicy value by running the Set-PSRepository cmdlet. Are you sure you want to install the modules from
'xSharePointDev'?
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "N"): y
WARNING: The module 'xSharePoint' cannot be installed or updated because it is not a properly-formed module.

I confirmed development builds for other modules (xWebAdministration, xExchange) can be installed properly, so the problem must be with the way nupkg is generated for xSharePoint

Thanks for reporting @KarolKaczmarek - I'm on it

@KarolKaczmarek This looks ok from my end now, can you confirm before I close this off?

sorry for the delay, but I just tried it and works now. Thanks!


katana: 1.0.4 -> 1.0.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/katana-crawling/versions.
meta.description for katana is: A next-generation crawling and spidering framework
meta.homepage for katana is: https://github.com/projectdiscovery/katana
meta.changelog for katana is: https://github.com/projectdiscovery/katana/releases/tag/v1.0.5
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.5 with grep in /nix/store/fbjfyraiyxwlmv83a0wq62r6cydqavrk-katana-1.0.5
found 1.0.5 in filename of file in /nix/store/fbjfyraiyxwlmv83a0wq62r6cydqavrk-katana-1.0.5

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
katana

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fbjfyraiyxwlmv83a0wq62r6cydqavrk-katana-1.0.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A katana https://github.com/r-ryantm/nixpkgs/archive/43b776d41ecb8fe3f3c77a9101902de11376b4d8.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/43b776d41ecb8fe3f3c77a9101902de11376b4d8#katana

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fbjfyraiyxwlmv83a0wq62r6cydqavrk-katana-1.0.5
ls -la /nix/store/fbjfyraiyxwlmv83a0wq62r6cydqavrk-katana-1.0.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

katana

Maintainer pings
cc @dit7ya for testing.


add shellcheck to .travis.yml example

Due to that fact that EUPS products include at least a ups/eupspkg.cfg.sh shell script, it is reasonable to enable shellcheck linting by default.

I do see that the ci_ packages also have build override scripts for the SCons files but it's still rare.

I'm happy for this if the documentation that includes the example notes that the two different scripts are included and that you are allowed to pick and choose based on your particular package. @ktlim needs to give final approval.

@jonathansick Subjectively, its been a long time since I've noticed a difference between gce and docker builds.  If anything, I think they get backlogged less often.

Per @ktlim 's request on the jira issue, I've changed the shellcheck example to a stand alone configuration and also added a combined flake8 + shellcheck example.

As stated in the JIRA ticket, I approve this, although also as stated in the comment in #215 , I think it is desirable in the long term for SQuaRE to take over maintaining these aspects of our GitHub code repos (much like Jonathan is responsible for enabling and maintaining LtD integration for doc repos).  That would hopefully negate the need for these examples in the dev guide.

Per discussion, this PR is being retracted.


[Feature][UI Next] Add navbar username.

Purpose of the pull request
#8070

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #8108 (206b69e) into dev (759fcb8) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8108      +/-   ##
============================================
- Coverage     41.43%   41.38%   -0.05%
+ Complexity     3734     3730       -4
============================================
  Files           637      637
  Lines         27141    27141
  Branches       3089     3089
============================================
- Hits          11245    11233      -12
- Misses        14811    14824      +13
+ Partials       1085     1084       -1

Impacted Files
Coverage Δ

...he/dolphinscheduler/common/enums/SqoopJobType.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
50.70% <0.00%> (-2.12%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (-2.00%)
:arrow_down:

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 759fcb8...206b69e. Read the comment docs.


Fix dex module inconsistencies

Fix several inconsistencies in the dex module:

some bytestring values were not properly set: the magic constants, header.magic and header.signature. Those were cut if a nul byte was present (which is the case in the magic values

dex.field[*].static and dex.field[*].instance were defined, but not declared, making their use impossible. They are now properly declared.

several fields in dex.method[*].code_item were declared but never defined: padding, tries and handlers. Those are removed since they couldn't have been used.

I'm not sure what the macosx build issue is. It complains about missing libjansson, which is not related to the dex module but to the cuckoo one, which i do not touch. So afaict it is unrelated.

I fixed an unnecessary change, should be ok now


Multislection without closing the dropdown list each time + checkbox

Hi,
Thanks for this module, it saved me a lot of time. I needed to change it a little bit for my project, so I tell myself I should share it if anyone might need it.
Changes :

All the options stay visible in the dropdown list, even if they're selected
If an option is selected, the background color changes and the checkbox is checked
The dropdown doesn't disapear each time an option is selected, it only disapears when clicking outside of it
Doesn't give focus to the input search when an option is selected

Demo :

I think that's it, enjoy your code : )

No problem, thanks you for looking at my code. I'm glad that I could contribute. Thanks for doing the exemple and the README.md, I didn't through of modify it, it's my first contribution to be honest.


(Feature Request) Add Support for Koreader

Hey! Amazing software. I was just wondering if it would be possible to make this work with koreader? I tried changing the directory of the notes but it doesn't work properly since it's the not the original kobo path.
Would there be a possibility of making this work with Koreader?

Hi! Apologies for the late response. I've only taken a quick look, but it looks it should be possible for me to do so. I'm working on a rewrite of this plugin, but unfortunately I'm unlikely to be able to put time into it until later this autumn. But I will make sure to try adding koreader support for it!

Hello! Is there any updates on this? Would really like to see this be implemented :)

Also please add Japanese support! I mainly read Japanese books and I believe it would be an incredible feature to be able to pick which kobo dictionary definition you would like to use on your anki card. A good complement as well would be also including the sentence the word was found it for more context.
Thanks once again


About a suggestion and a small request.

Can you remove first fi from class names (not the fi prefix). So it will be programmer-friendly and there will be a drop in the file size.

we are working on the relevant arrangement


synchronized the compression settings for client and server

It was not possible to disable compression fully in both server and client. This PR ensures you can properly disable the compression by setting openvpn_compression: "".

Sounds good to me.
Thanks :)


Suggest Python version

Would it be a good idea to suggest a Python version to use with Locust? I have used pyenv and after some trial and error, have settled on Python 2.7.7.

Ah, yes, we should probably add that to the documentation. The recommended version is the latest 2.7 one.

With the release of 0.8.0, Python 3.6 (latest) is recommended.