Rquest without Callback

Hi,
is it possible to set a request without using the OnPermissionCallback.
Like this:

boolean isGranted = permissionHelper
     .setForceAccepting(false)// true if you had like force reshowing the permission dialog on Deny (not recommended)
     .request(isSingle ? SINGLE_PERMISSION : MULTIPLE_PERMISSIONS);

if user declined (boolean = false) -> ask next time again
if user really declined (boolean = false) - > open a dialog with "open Settings" buttom
if user granted (boolean = true) - > do nothing
do you know what i mean?
cheers

the library will return you callback if the permissions asked are granted and always you can use of the PermissionHelper static or non-static methods to determine if the permission is granted or no. it can even tell you if the permission permanently denied. please have a look at the readme and regards the other issue i haven't yet got my hand on a device running 6.0.1 so i'll keep you updated, however as i mentioned the library works as intended in 6.0 .


Updated: playnite 7.1

${}

3c0a1688-8ef5-478a-9b20-24b1d1133eac

Newer version of playnite (7.5) is now available in master. 71ef66184c326a9f0128c492df5c6f8c541e619b


Create README.md

Hi,
This small PR adds a README.md file with two buttons to get your app on F-Droid and Google Play 👍

Thanks, much appreciated


chore: add auth0 keys to sample.env

Add keys so that the local app does not crash

Merging this to avoid local setup crashes. This PR only touches the sample.env /cc @Bouncey


Feat/dataset overview

Description
In order to improve user experience and better show the dataset specific information, a page that displays relevant dataset information was created. Some actions available on the dataset item card were removed and a link to navigate to the dataset overview page was added
Changes
Front end
Data set list page

[x] Removed admin dataset actions from dataset item card on dataset list page and added a navigation to the Dataset overview page

Data set overview page

[x] Created dataset overview page and route based on dataset id (dataset/:id)
[x] Created dataset specific information area
[x] Created dataset gallery viewer area (it is hidden for now)
[x] Display the dataset title on information area
[x] Display dataset submitter name on information area
[x] Display dataset submitter group name and redirect user to group page, when the user user click on group name
[x] Display dataset description on information area
[x] Create an annotation component that shows dataset related annotations separated by database and fdr level
[x] Redirect user to annotation page filtered by dataset/fdr/database when it applies. The applied scenario should happen when the user click on an annotation count on the annotation component or in the 'browse annotation' button
[x] Display dataset metadata on information area
[x] Display the dataset related projects on information area

Graphql
Dataset description

[x] Created dataset_description field on graphql.dataset table
[x] Added  dataset_description to update, get and create dataset related queries

Dataset overview related

[x] Added annotationCounts return field, containing the annotation counts of all ds item dbs

Tests
Front end

[x] Unit Test
[ ] e2e Test

Browsers and resolutions

[x] Chrome
[ ]  Safari
[x] sm, md, lg, xl, lg

One more feedback item: I haven't added any specific feedback about the appearance of the Overview page as I feel we should wait until the diagnostics stuff is ready before doing a polishing pass to make it pretty. The main thing I think should be done is updating the typography (header & text styles) to match the new style of the content pages, e.g. help and publications. However, this can wait for a future PR.


Write async persistent tmp file to sub-directory of mount point

Currently, Alluxio writes the temporary file of async persistence to the path, which is the same as the destination file path. The computing framework, like presto, will pick up the tmp files, which is leftover there for some reason, fail to clean up, etc.
This change writes the tmp file of async persistent to the sub dir of the mount point.

@HelloHorizon Yes, the path must be relative to the mount point. If we have the same guarantees as ufsBlocks that's fine.

@HelloHorizon Yes, the path must be relative to the mount point. If we have the same guarantees as ufsBlocks that's fine.

alluxio-bot, merge this please

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.4 please

alluxio-bot, cherry-pick this to branch-2.4 please


Support zipped csv files in csv_collection

Just sharing a case where our maintenance task's input was a 1.2G .csv file (which made it challenging to upload to the app without timeouts). Compressing the file using zip brought down the file size to ~250M, but the gem doesn't support compressed file formats.
This isn't a pressing matter - we could also just increase the upload timeout config in our app - but seems like something that could be useful? Is there interest in adding this functionality? Or is there already an established workaround?

Thought about this for a little more, and this feature might not be as easy to implement as I first thought.
These lines handle reading the contents of the .csv input file. They do so by reading the entire file and representing its contents as a string in memory.
This clashes a bit with the intention for supporting zipped .csv files (allowing someone to use a very large .csv file as input; e.g., > 1GB): the gem currently loads the entire file rather than reading and processing lines one-by-one.

Ruby does have the IO.foreach method that would allow the gem to process a .csv file line-by-line without putting it all in memory. I'll see if I can get a version of the gem that works using this lazy approach


GH-117021 - correct CIDR

@cherylmc
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Spanish translation

Updated spanish translation

Thank you @lboullo0!

(just adding link to issue #81 as a comment, so it is easier to see from there which translations are complete)


feat: 💥 LoadBalancer Service by default

What does this PR do?
Set LoadBalancer as the default Service type.
This PR is a breaking change on default behavior.
Motivation

Traefik Hub has all Ingress capabilities of Traefik Proxy
Traefik Proxy and Traefik Enterprise set LoadBalancer as the default Service type

More

[x] Yes, I updated the tests accordingly
[x] Yes, I ran make test and all the tests passed

There is no notice :cry:
But I assume this is as simple as --set service.type=ClusterIP, right?

This chart installs Traefik Hub with a LoadBalancer service by default. To change it, you can append the helm command with the --set service.type=ClusterIP parameter or edit it in your values file to your desired service type.

If you want to avoid adding this to the notice, we could add it to the docs as a last resort.
Would the GitOps demo fail with this breaking change?
(cc @svx)

@immanuelfodor For extending notes, first there should be notes. See #79

no-merge: wait for #79

@immanuelfodor Wdyt of 3a69bdf ?


using the library without the GUI

Hi,
Is there a way to disable all GUI functionality completely?
I'm building a GUI application inside a Jupyter notebook for interfacing my RP. I'd like to use PyRPL for communciating with it. However, I would like to avoid the PyRPL gui altogether.
Right now, if I create a Pyrpl() object, it always opens up a GUI popup window, which I'd really like to avoid if possible.
I've tried gui=False: this does disable the popup window that asks you for the location of the config file (asking you instead inline in the notebook interactively, which I would also like to avoid...but I can do that by specifying the config file...). But it does not disable the GUI popup window that asks you to add modules. As I want to use the library only as a programming interface to the RP, I would like to avoid any GUI popup windows.
Is there a way to do this?
(I'll also continue to read through the code in the meantime...perhaps answering my own question here :))
Cheers,
Gary

What you are describing sounds like a bug...
Can you check whether the line 302 of pyrpl.py is evaluating to False (as it should)
    if self.c.pyrpl.gui:
        self.show_gui()

Also, you might have seen that all pyrpl settings are saved in a config file to stay the same from one session to the next. The config file is located (on windows systems) in c:/users/you/pyrpl_user_dir/configs/whatever_config.yml
The config file is human readable, you might want to check the value of the gui flag in there.
Sorry that I cannot do more debugging myself, but I don't have all the good programming environment (including a redpitaya) with me in confinement...
Let me know what you find

I haven't had a chance yet to check the line of the code you mention but I did edit the config file and found that gui was false for the red pitaya section but true for the pyrpl section
I edited it to be false and now I no longer have the popup window
I do get an extra "python" process in my system tray (running on macos), which has the menu items "pyrpl". I guess this is a QT app that you guys use for the QT event loop. But no gui window so that is what I wanted!
Cheers,
Gary


Playground: refactor component picker plugin

Seems fine - what's the aim here?

@acywatson I was reading through this code and noticed a bunch of small things.
Commit 1: local simplifications without moving code. E.g. getting rid of ts-ignore comments and redundancies like new RegExp(/foo/). Also it fixes a bug with filtering options by keywords.
Commit 2 extracts functions out of the component because shorter functions are easier to read and because useCallback isn't really needed when the function isn't passed to a prop.


Add support for required plugins.

By default, containerd keeps running when a plugin fails to be initialized. However, some users may require some plugins for their use case, e.g. the cri plugin is critical for Kubernetes users.
This is even worse when we add more and more config validations in the CRI plugin, e.g. https://github.com/containerd/cri/pull/1125. If a bad CRI config is given, containerd will continue running without the CRI plugin, which makes it very hard to figure out what is wrong.
This PR added the required_plugins config. If a required plugin is missing or fails to be loaded/started, containerd will exit directly.
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

Codecov Report

Merging #3197 into master will decrease coverage by 0.04%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3197      +/-   ##
==========================================
- Coverage    45.2%   45.15%   -0.05%
==========================================
  Files         111      111
  Lines       11982    11995      +13
==========================================
  Hits         5416     5416
- Misses       5732     5745      +13
  Partials      834      834

Flag
Coverage Δ

#linux
49.17% <0%> (-0.06%)
:arrow_down:

#windows
40.52% <0%> (-0.06%)
:arrow_down:

Impacted Files
Coverage Δ

services/server/server.go
1.43% <0%> (-0.08%)
: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 4d313c0...fdc4f25. Read the comment docs.

Add required plugins to containerd config default command ?

@Ace-Tang What do you mean? You mean adding cri as required plugins by default?

@Ace-Tang What do you mean? You mean adding cri as required plugins by default?

I am not mean to put cri as a required plugins by default. I mean we can put required_plugins field (empty or not) in example config,  containerd config default shows a good example config for users

@Ace-Tang You mean this?

@Random-Liu , yes :+1:

Looks good to me


First pass at heirarchical courts list

Changes in this PR:
Show the courts heirarchy on the structured search page and in the filters list.
This doesn't include 'select all' functionality, and needs some tidying up - both presentationally here, and in terms of names / structure of courts in the utils library.
In particular, we need to

Group the courts so that courts show in the left column, and tribunals in the right (work in both places)
Rename court divisions / tribunals so that they don't repeat the group heading (eg "Family division" vs "Family division of the High Court", "Lands Chamber" vs "Upper Tribunal Lands Chamber") (work in utils library - this may involve giving each court a 'grouped' and 'ungrouped' name, and using the correct one depending on context)

Trello card / Rollbar error (etc)
https://trello.com/c/Gj60XUIp/1380-investigate-court-hierarchy-work-finalize-design
Screenshots of UI changes:
Before

After

 Investigate Court Hierarchy work - finalize design

At risk of doing design work, would this look better with more vertical padding between groups to make it clearer that they're a contained list?

aha, you are 100% correct, and thank you for pointing it out! - however, after some experimentation, I'm gonna do this after doing the work to put the courts in the left column and the tribunals in the right. The issue is that we need to use margin-top on the group headings (as we don't want extra space between top-level courts), and that currently has the side-effect of pushing the top item in the right column down in an unsightly way (as we're using CSS multi-column layout which unfortunately doesn't let you target the first element of a column for styling differently). Once I've changed to have an explicit 'courts column' and 'tribunals column' it'll be easy :-)

Updated version after addressing all the 'snagging' issues on trello, tests will pass after an API release and version bump.

Codecov Report

Merging #911 (11293b0) into main (8922f25) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #911   +/-   ##
=======================================
  Coverage   94.67%   94.67%
=======================================
  Files          32       32
  Lines        1389     1390    +1
  Branches      224      224
=======================================
+ Hits         1315     1316    +1
  Misses         54       54
  Partials       20       20

Files
Coverage Δ

judgments/tests/test_search.py
100.00% <ø> (ø)

judgments/views/advanced_search.py
94.73% <ø> (ø)

judgments/views/browse.py
92.85% <100.00%> (+0.26%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Update: EAT is now at the top of the tribunals list! 🥳


Mention code

Create generic code to mention people, so that we don't reimplement it in each munger. It comes with tests and interesting utility functions.

@hongchaodeng you probably want to have a look

@foxish Can I ask you to have a look and review that?
Thanks!

LGTM

PTAL @foxish @lavalamp

I am satisfied, I will let @foxish make a final pass.

Very clean and readable I think. Especially the .AllUsers().Mention().Join() style.

I don't think it makes a big difference, but I initially split because it made sense to me to keep it separated :-)


worker heartbeat not working

I don't know if i'm reading something wrong, but it seems that  heartbeat part of Worker.prototype.onMsg function checks for params that are not passed by Worker.prototype.heartbeat. lacks rid param, and requires clientId that is never used.
Sending heartbeat:
  ...
  this.send([
    MDP.WORKER, MDP.W_HEARTBEAT, '',
    JSON.stringify({
      concurrency: this.conf.concurrency
    })
  ...

Heartbeat processing event onMsg requires message to have 5 parameters (and it only has 2)
if (msg.length === 5) {
  clientId = msg[2];
  rid = msg[4];
  if (rid && this.reqs[rid]) {
    this.reqs[rid].liveness = HEARTBEAT_LIVENESS;
  }
}

It's a bit confusing for me ... there are 2 different "livenesses" one is [this|self].liveness and the other is req.liveness. When checking if reply is active req.liveness is tested but stream is still open because testing is done on this.liveness (which on every message is set to HEARTBEAT_LIVENESS)
if (self.liveness <= 0) {
  ...
  self.stop();

liveness

A Worker instance has its own liveness. That indicates the status of the connection with the broker.

A request, when hits a Worker, has its own liveness. The is interesting because a Client, when it sends an heartbeat, can also specify a rid to inform the Worker that it is still interested to the request.
This is an optional feature that helps is looong running requests.

heartbeats
When the Worker receives an heartbeat and the length is === 5, it means that the heartbeat comes directly from a Client for a specific request.
Is more clear now?

ps. what is not working for you?
Do you have an example?

I'll try to prepare something over the weekend to show you my problem. In my opinion the internal method heartbeat doesn't work. I do a lot of write() and every time it "ticks" liveness goes down by 1, and after 3-4 times method reply.active() returns false while stream is still up and running

It's normal that reply.active gives you false because it doesn't receive a specific heartbeat for the related rid.
Indeed it's not well explained... sorry about that.

To have reply.active returning true the related Client should send heartbeats for that specific Request to notify that the interest in the reply is still alive. This is something very unusual anyway, so we may want to improve this.
Ideas are appreciated

Thx for your asnwers. I've solved my problem with stream version of client - all i had to do was:
var heartbeat = true,
  pigatoCllient = new pigato.Client();

var stream = pigatoCllient.request('workerName', data)
      .on('data', function(partialData) {} // handle partial data
      .on('end', function() { heartbeat = false; })
      .on('error', function(err) { heartbeat = false; });

// send proper heartbeat;
setInterval(function() {
  stream.heartbeat(); // add heartbeat === true check and clearInterval
},.pigatoClient.conf.heartbeat);


NIFI-6009 ScanKudu Processor & KuduPut Processor Delete Operation

JIRA: NIFI-6009

ScanKudu Processor
PutKudu Delete Operation
Kudu Client Service

For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

[x] Is your initial contribution a single, squashed commit?

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[x] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[x] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[x] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

@cammachusa I see that you worked PutKudu Processor, I have some changes to PutKudu and also Added ScanKudu Processor. please review and let me know if I have missed anything.

@joewitt @bbende can you please have a look into this.

Sorry but I am not familiar with Kudu at all

Sorry but I am not familiar with Kudu at all

Okay np, please give me feedback on NIFI part.

I have fairly minimal Kudu experience but am very familiar with NiFi so between Grant and myself we should have this covered pretty well. :)
I have not been thoroughly through this PR, but I noticed a significant refactoring into creating a Controller Service. While we do have some processors that use a "client" type of Controller Service, we have to be very careful modifying an existing processor like this to change how this is configured. Doing so would mean that everyone who uses the Kudu processor currently would have an invalid flow when they upgrade and would then have to completely reconfigure everything. We would need to have a really good, defensible argument for changing the processor in that way. Can you explain the rationale behind such a change?
It also appears that you changed the package name from "org.apache.nifi.processors.kudu" to "org.apache.nifi.kudu" - we cannot make a change like this, as it would result in processors that exist in current flows becoming "Ghost Processors."
I also wonder if it makes more sense to have a DeleteKudu processor. PutKudu implies to me that it is going to put data into Kudu, not delete data...

@granthenke that's a fantastic review, will update PR with suggested changes.
@markap14

added Kudu Controller Service to reuse Kudu client connection's code and have a single Kudu client for multiple Kudu Processor in a pipeline. I thought Its an early stage for PutKudu Processor good time to make change
Will rename back to org.apache.nifi.processors.kudu
we could have a separate DeleteKudu processor, reason for adding DELETE in Put processor to overcome particular race condition. i.e CDC operations, with a separate Kudu Delete Processor INSERT would go first and next coming DELETE would drop inserted records or vice-versa.

@SandishKumarHN Thanks for responding so quickly.
Re: the Controller Service - @granthenke can correct me if I am wrong, but I don't know that reusing the client provides a huge upside here. It probably doesn't hurt but I don't know that we really gain much either. In terms of code reuse, I think we can easily accomplish that in other ways, perhaps with an abstract class or helper methods. A change that makes a flow invalid is pretty much a last resort. If there were some sort of security vulnerability or something like that, or if we find that we're doing something egregious to the server side then it would make sense to consider the change.
Can you explain your third thought there, regarding CDC operations? Can you outline the use case where this helps? As you're describing it, I think such a race condition could still exist. If you wanted to INSERT some records and DELETE some records, you'd still need a PutKudu processor configured to INSERT and another PutKudu Processor configured to DELETE - unless I'm missing something?

@markap14

I can certainly create an abstract or helper class to reuse of Kudu Client Connection.

for the 3) The race condition would happen when there is an order mismatch while separate  Kudu INSERT Processor, Kudu DELETE Processor, we can overcome by using a single processor with INSERT and DELETE with a flow file attribute for 'OperationType'

@granthenke @markap14 thanks for the awesome feedback. made changes according to the suggestions. Please review again.

Renamed package back to org.apache.nifi.processors.kudu, removed RowSerializer Class and added convertToJson method instead, removed ControlService, disabled exposing AsysnKuduClient and made changes to buildPartialRow.

Added AbstractKuduProcessor which handles kudu connections and put and scan operations

Have added OperationType to PutKudu with Options of (INSERT, UPDATE, UPSERT, DELETE) made it as a flowfile attribute so that incoming flowfile can be decided which operation type to put in or use a single operation type.

@granthenke Added All Kudu Supported Predicates for KuduScan Processor

@granthenke Thanks for the second review, WIll make changes as suggested!

@granthenke I have split this PR into two.
Kudu Put Operations: https://github.com/apache/nifi/pull/3610
Kudu Scana Processor:  https://github.com/apache/nifi/pull/3611
also made changes suggested here into new PR's

Thanks for breaking this out. I will review them asap.

@granthenke  made changes suggestion on the main PR. #3611

Can you close this PR now that it has been split out?

Can you close this PR now that it has been split out?

Okay


Failed to get access to local media. Error code was SourceUnavailableError.

I want to accsess easyrtc in my php project.
i configure easyrtc in my php project like this

view.php (In codeigniter)

<?php
	$actual_link = "http://$_SERVER[HTTP_HOST]:8080";
?>
<script src="<?php echo $actual_link.'/socket.io/socket.io.js'; ?>"></script>
<script type="text/javascript" src="<?php echo $actual_link.'/easyrtc/easyrtc.js'; ?>"></script>
<script type="text/javascript" src="<?php echo $actual_link.'/web/jsrtc/all.js'; ?>"></script>
<body onload="connect()">
		<video autoplay="autoplay" class="easyrtcMirror" id="selfVideo"></video>
</body>

and in my MyclientEasyrtc.js (In node js)

function connect() {
  console.log("Initializing.");
  easyrtc.setSocketUrl(":8080");
  easyrtc.enableVideo(true);
  easyrtc.enableVideoReceive(true);
  easyrtc.initMediaSource(
    function(){    // success callback
      easyrtc.connect("danang", loginSuccess, loginFailure);
    },
    function(errorCode, errmesg){
      easyrtc.showError(errorCode, errmesg);
    }  // failure callback
    );
}

but i get an error

Failed to get access to local media. Error code was SourceUnavailableError.

help me to fix this.

Could you provide some more details?

oke i can update my question

So this is for easyRTC I presume which is another project, not AppRTC.
However the error you are getting is from getUserMedia and it's complaining that your machine does not have microphone nor camera.


Support transfer with MetaMask

https://github.com/AcalaNetwork/Acala/pull/1644
https://github.com/AcalaNetwork/Acala/blob/82d54ff5986149c07a604ae8d119685d77e6533f/ts-tests/tests/test-sign-eth.ts#L58-L66

{
    id:<PHONE_NUMBER>450
    jsonrpc: "2.0"
    error: {
      "code": -32601,
      "message": "EIP-1559 transactions not implemented (method=\"EIP-1559 transactions\", provider=\"eth-providers\", code=NOT_IMPLEMENTED, version=bodhi.js/providers/2.1.6)"
    }
}

EIP-1559 duplicate with #174
Closed by #167


page updated on TOC

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

server-2013/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Declined.  This page is empty.  There is no reason to add it to the TOC.  It should be recommended to remove the page, rather than add it to the library.
Not merging.  Closing the PR.


Explicit the origins of imported functions

When importing functions from standard packages, instead of using:
from random import random
a = random()

use:
import random
a = random.random()

Contemplated by https://github.com/arthurpaulino/miraiml/commit/f3cb48b6d6486948b3686e8ff34b2232ba9d78dc


Crashes when connection unavailable

Need to catch the requests exception and retry.

Fixed crashes in 7b5ddc19e61e3cca706b4469afc459868ac10a88. Will probably still crash if there is no auth token available (which would happen first time the watcher is run) but since the whole auth process is in need of an improvement anyway I'll leave it for later.


JavaScriptAggregator fnCombine didn't work

I changed the test case inside io.druid.query.aggregation.JavaScriptAggregatorTest
https://github.com/druid-io/druid/blob/89d9c61894f122b4973ad5d8d325e114b0a7ebf4/processing/src/test/java/io/druid/query/aggregation/JavaScriptAggregatorTest.java#L88
from
  static {
    sumLogATimesBPlusTen.put("fnAggregate", "function aggregate(current, a, b) { return current + (Math.log(a) * b) }");
    sumLogATimesBPlusTen.put("fnReset", "function reset()                  { return 10 }");
    sumLogATimesBPlusTen.put("fnCombine", "function combine(a,b)             { return a + b }");

    scriptDoubleSum.put("fnAggregate", "function aggregate(current, a) { return current + a }");
    scriptDoubleSum.put("fnReset", "function reset()               { return 0 }");
    scriptDoubleSum.put("fnCombine", "function combine(a,b)          { return a + b }");
  }

to, just make fnCombine always return 0
  static {
    sumLogATimesBPlusTen.put("fnAggregate", "function aggregate(current, a, b) { return current + (Math.log(a) * b) }");
    sumLogATimesBPlusTen.put("fnReset", "function reset()                  { return 10 }");
    sumLogATimesBPlusTen.put("fnCombine", "function combine(a,b)             { return 0 }");

    scriptDoubleSum.put("fnAggregate", "function aggregate(current, a) { return current + a }");
    scriptDoubleSum.put("fnReset", "function reset()               { return 0 }");
    scriptDoubleSum.put("fnCombine", "function combine(a,b)          { return 0 }");
  }

But the test cases are still passing, seems fnCombine is not called or fnCombine is always using (a + b):

fnCombine gets called on AggregatorFactory.combine but the JavaScriptAggregatorTest doesn't test this. It only tests aggregate. I think this is just a lack of test coverage.

Actually, i dive deep into the codes, because i tested the aggregator on our server, i found the problem:
    {
      "type": "javascript",
      "name": "sum(log(x)*y) + 10",
      "fieldNames": [
        "traffic",
        "throughputEstimate"
      ],
      "fnAggregate": "function(current, a, b)      { return current + (Math.log(a) * b); }",
      "fnCombine": "function combinePartialResults(partialA, partialB) { return partialA + partialB; }",
      "fnReset": "function()                   { return 10; }"
    }

and i changed to return 0, but two query give me the same result.

I also changed this line inside QueryRunnerTestHelper: https://github.com/druid-io/druid/blob/master/processing/src/test/java/io/druid/query/QueryRunnerTestHelper.java#L134
public static final String JS_COMBINE_A_PLUS_B = "function combine(a, b) { return a + b; }";

to
public static final String JS_COMBINE_A_PLUS_B = "function combine(a, b) { return 0; }";

and run TimeseriesQueryRunnerTest tests, again, all passed:

Do you have more than one segment being queried? If you are only querying one segment, then combine won't get called. Your intervals in that query only cover 15 minutes so it may only be hitting one segment.

Thx, I will test more late.

@gianm You are right, if i query on multiple segments, fnCombine works well. So it is just a lack of test coverage. Thx, i will close this issue.


How to install

Isn't a bug.
I want to install this on my FreeBSD system but I have error when launch the program:
go build bookbrowser.go after git clone https://github.com/geek1011/BookBrowser.git
or I test with this command and I have an other error:
pip install git+https://github.com/geek1011/BookBrowser
This is my error:
"root@Dev:/usr/local # pip install git+https://github.com/geek1011/BookBrowser
Collecting git+https://github.com/geek1011/BookBrowser
Cloning https://github.com/geek1011/BookBrowser to /tmp/pip-6hFyFG-build
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
IOError: [Errno 2] No such file or directory: '/tmp/pip-6hFyFG-build/setup.py'
"
thanks
----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-6hFyFG-build/

It is written in go, not Python. Either download the prebuilt binaries from the releases page, or build it yourself with golang 1.9.


[IOS][UE4] Crash on UE4/IOS in shipping builds

I've built aws-sdk-cpp as static library and use it inside Unreal Engine 4.
I've got a crash after a call to Aws::GameLift::SearchGameSessionAsync. Caller code works properly on Windows, Android, Mac so I assume that it is correct.
Function calls callback, but then it causes game to crash (tested with empty callback function).
This problem only occurs in UE4's Shipping builds. Project built in Development or DebugGame configurations works fine for some reason.
Versions of aws-cpp-sdk used to debug this problem:
1.7.47
1.6.53
1.5.25
1.4.0
What platform/OS are you using?
Host: Mac x86-64
Target: iOS arm64
What compiler are you using? what version?
LLVM/Clang/libc++ (clang-1<IP_ADDRESS>)
What's your CMake arguments?
BUILD_ONLY: gamelift
BUILD_SHARED_LIBS: disabled
CUSTOM_MEMORY_MANAGEMENT: disabled
ENABLE_RTTI: tried both
FORCE_CURL: enabled
FORCE_SHARED_CRT: tried both
Crash thread (the output is the same, regardless of whether there are debug symbols or not):

So the error is "malloc: *** error for object 0x10c674700: pointer being freed was not allocated"
I already tried to set custom allocator to use UE4's malloc and free, but it did not improve the situation.

I resolved this issue by rewriting DefaultExecutor to use Unreal's multithreading classes.

So apparently there is easier way to solve this issue in UE4.

Surprisingly the reason for behaviour like this is std lib memory allocations parallel with FMallocBinned2 memory allocator, which is default one for iOS in UE4. In fact, you can't use std::string, std::stringstream, std::thread and other useful std stuff, because it can cause memory corruption.
The solution is to force FMallocAnsi usage on iOS platform.

public class MyProjectTarget : TargetRules
{
    public MyProjectTarget(TargetInfo Target) : base(Target)
    {
        Type = TargetType.Game;

        ExtraModuleNames.AddRange( new string[] { "MyProject" } );

        // Leave it for shipping now to have nice crash reports
        bUseChecksInShipping = true;

        // Force ANSI allocator for game client
        if(Target.Platform == UnrealTargetPlatform.IOS)
        {
            GlobalDefinitions.Add("FORCE_ANSI_ALLOCATOR=1");
        }
    }
}

Although this issue is specific to UE4, I'm reopening this issue until there is a note about this in documentation or in the project description.

@singku


Adding Style to Login page!

I want to change the style of login page for better user performance, like the form format is not made correctly from the user's point of view!
Can You please assign me the issue under gssoc and hacktoberfest-accepted, so that i can  successfully contribute to this project .
Your User page :

Login Page I want to create:

please select some other open issue, this issue is covered in issue #8


singal operator with multi grafana instances

I hope currently we are deploying only one grafana that to in the namespace where the operator is present. So why dont you add this feature like Prometheus operator. For the entire cluster one operator which can able to handel the no of instances across the cluster on different namespaces

@mohanagandlla , this is currently in the works #433 , closing to avoid duplication, we're already tracking this issue in a separate ticket


Dispatching events outside of a command

We use SimpleBus inside a legacy application which does not use commands an handlers for everything but we do have events inside of our doctrine entity's and if i trigger the events inside a entity outside a command handler the events are not dispatched. Is there a way to manually dispatch the events inside of the event recorder queue?

This issue is already stale for several years so i'm closing it for now


Positional Embedding

Thanks for your great work!
I have a few questions about the modification in DeiT_3.

Why do you remove the positional embedding for the cls token?
Do you simply omit the dist token and the positional embeddings for both tokens when transferring weights from DeiT?

I also have a question about the "gamma_1" and "gamma_2" parameters in DeiT_3. These is no mention of the parameters in the paper. Could you please provide some explanations or experiment results?

The original code of DeiT or DeiT_3 doesn't add the positional embedding for the cls token.
I don't understand what you mean.
Gamma_1 and gamma_2 are layer scale. You can refer CaiT or DeiT III to learn it. DeiT doesn't include this.
Besides, DeiT III is just for inferencing. The weights are transferred from the original DeiT III repo. You can not get the DeiT III's results with this repo.


Report.__init__() got an unexpected keyword argument 'descriptor' when creating a frictionless validation report

In versions frictionless==4.29.0 and livemark==0.88.0 I could create a frictionless validation report with
```yaml report
descriptor: validation-report.json
```

There is a working example here.
Now, with frictionless==5.4.0 and livemark==0.110.2 when trying to build that same type of validation report (but using the v5 version of report) I get:
Report.__init__() got an unexpected keyword argument 'descriptor'

I did not found any mentions of breaking changes in the release page.

Thanks! We will fix it.

Thnks, it must be fixed in<EMAIL_ADDRESS>


Permissions Issue - not able to download the APp from URL

while clicking on the Host URL to download mac(ZIP) file - following error results on web request.
An error occured - when calling CreateMultipartUpload operation:AccessDenied when aws s3 cp command is fired on Cloud9 application console.
This XML file does not appear to have any style information associated with it. The document tree is shown below.

AccessDenied
Access Denied
X
Y

@gitCodebytj Did your user have AdministratorAccess?
From the README:

Log into your AWS account with an IAM role that has the AdministratorAccess policy.

Please reopen the issue if you experience any further problems

I am have same issue. Is there any solution to it?


Printing Items in 0/1 Knapsack

This is an extended version of the 0/1 top-down approach for the knapsack problem. Here we need to find the elements we take for the maximum profit.
main

Hey great work! but please add comments in code and the question of the problem along with the time and space complexity. We will surely merge this pull request once done. Thank you.


error?

Link Play authentication failed.
Please make sure that the token in linkplay_server/config.py is the same with that in config.py.

Link Play authentication failed.
Please make sure that the token in linkplay_server/config.py is the same with that in config.py.

it is the same, i still get warning

Really? Hmmm maybe you need to uncomment line 135 in linkplay_server/main.py to see the data from the main server.

Really? Hmmm maybe you need to uncomment line 135 in linkplay_server/main.py to see the data from the main server.


One entry in singlepooloperators.json refers to 1PCT pool

Hello,
I highly appreciate this project and want to thank everyone involved. I stumbled across this one reference in singlepooloperators.json to 1PCT pools. Does anyone know if this operator is making fun of 1PCT, or maybe leverage the 1PCT brand, or is this pool even part of the 1PCT pool cluster?
"c0c231446dc398936f0f85506072f10641b861ce5012c0fcb1051442": {
    "name": "1PCT Pools at 1-PCT.com",
    "ticker": "WOW"
  },

Not part of 1PCT cluster, seems to be someone trying to leverage success from 1PCT using names targetting unsuspecting delegators 👿

🙏 thank you


Deploy index to gh-pages?

I noticed in the CI logs there's some rate-limiting. I wonder if hosting the index as a static site would help avoid that.

It seems that there are even stricter rate limits in place, the past couple of days. How would a move to gh-pages look like? Can it be handled by sbt-typelevel?

I think what we need to do is:

in the settings page, enable gh-pages on the main branch and point it at the index folder. This should be backwards-compatible.
point download-java at the new location

That's annoying, seems that only / or docs/ are supported. Using / should be fine I guess.

@vasilmkd I'm seeing some weird issues, maybe we can give this a try?

Run JDK_URL=$(curl -L https://raw.githubusercontent.com/typelevel/jdk-index/main/index/Linux/X64/temurin/17/jdk)
  JDK_URL=$(curl -L https://raw.githubusercontent.com/typelevel/jdk-index/main/index/Linux/X64/temurin/17/jdk)
  JDK_FILE="/home/runner/work/_temp/$(basename $JDK_URL)"
  echo "jdkUrl=$JDK_URL" >> $GITHUB_OUTPUT
  echo "jdkFile=$JDK_FILE" >> $GITHUB_OUTPUT
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    PGP_PASSPHRASE:
    SONATYPE_PASSWORD: ***
    SONATYPE_CREDENTIAL_HOST:
    SONATYPE_USERNAME: ***
    PGP_SECRET: ***

    GITHUB_TOKEN: ***
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (60) SSL certificate problem: certificate has expired
More details here: https://curl.se/docs/sslcerts.html

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

I heard that today GitHub changed their ssh keys. Could the issues be related to that?

Oh, possibly. But still, I wonder if this would be a useful thing to do :) the keys for static site hosting might be managed independently.


Support removing elements from slice.

Emit error for subslicing of non-byte slices.
Support removing slice elements in-place.

It'd be nice to at least mention util package in the compiler document.


Question: How to get predicted trajectory?

Hi,
Is there any simple way to save the predicted trajectory (e.g., x-y coordinates) in a txt or any file format?

No we don't provide any utility function to do that. You can dump it in the same format as the input txt file.

Have you solved this problem?I also want to get this result.Thx. @daehoumich


Search: .query does not update on input

The search widget exposes an undocumented query property which is used in the Documentation notebook to set the query terms via JavaScript links.
Unfortunately query will only update when assigned via its setter, but not on user input. Since query also has a getter, it is probably reasonable to expect the value to match the current search input value.

Related: https://github.com/observablehq/inputs/issues/237


[SYSTEMML-1863][WIP] Add MLContext test class for linear regression

This plans to add a test class for all the operations and functions in LinearRegCG.dml and LinearRegDS.dml scripts.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2026/Failed Tests: 4SystemML-PullRequestBuilder/org.apache.systemml:systemml: 4org.apache.sysml.test.integration.mlcontext.algorithms.MLContextLinregTest.testLinregCGDenseorg.apache.sysml.test.integration.mlcontext.algorithms.MLContextLinregTest.testLinregDSSparseorg.apache.sysml.test.integration.mlcontext.algorithms.MLContextLinregTest.testLinregDSDenseorg.apache.sysml.test.integration.mlcontext.algorithms.MLContextLinregTest.testLinregCGSparse

Hi @deroneriksson , this seem to have some validation and script execution errors,
org.apache.sysml.api.mlcontext.MLContextException: Exception when executing script
	at org.apache.sysml.parser.Expression.raiseValidateError(Expression.java:532)
	at org.apache.sysml.parser.Expression.raiseValidateError(Expression.java:510)
	at org.apache.sysml.parser.DataExpression.validateExpression(DataExpression.java:610)
	at org.apache.sysml.parser.StatementBlock.validate(StatementBlock.java:597)
	at org.apache.sysml.parser.DMLTranslator.validateParseTree(DMLTranslator.java:158)
	at org.apache.sysml.api.mlcontext.ScriptExecutor.validateScript(ScriptExecutor.java:549)
	at org.apache.sysml.api.mlcontext.ScriptExecutor.compile(ScriptExecutor.java:309)
	at org.apache.sysml.api.mlcontext.ScriptExecutor.compile(ScriptExecutor.java:268)
	at org.apache.sysml.api.mlcontext.ScriptExecutor.execute(ScriptExecutor.java:345)
	at org.apache.sysml.api.mlcontext.MLContext.execute(MLContext.java:355)
	at org.apache.sysml.api.mlcontext.MLContext.execute(MLContext.java:332)
	at org.apache.sysml.test.integration.mlcontext.algorithms.MLContextLinregTest.runLinregTestMLC(MLContextLinregTest.java:74)
	at org.apache.sysml.test.integration.mlcontext.algorithms.MLContextLinregTest.testLinregCGDense(MLContextLinregTest.java:52)

Am I doing some mistake in giving inputs, I tried to go over the above pointed line numbers?. Thanks.

Hi @j143. I believe there are a couple issues.

Looking at the LinearRegCG.dml script, it looks like the author of the LinearRegCG.dml script inconsistently made the input matrix 'y' lowercase (rather than uppercase, like 'X'). Therefore, .in("Y", Y) needs to be .in("y", Y).
Secondly, there is no variable 'B' in the script, so defining 'B' as a variable that shall be output via .out("B") will fail. You could obtain 'beta_out', so out("beta_out") would work.

Also, note that no checks are being performed for expected results. I think that is probably fine at this stage, since these test demonstrate that these scripts can indeed be executed using the MLContext API.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2033/

Thank you @j143. LGTM. Would you like to add further tests or can I merge this?

If we don't need to worry about testing operation result through MLContext, 'cause the different operations are already tested normally and compared with https://github.com/apache/systemml/blob/master/src/test/scripts/functions/codegen/Algorithm_LinregCG.R#L23, we can go for merging.
With the consequent algorithms I will be improving myself on how exactly the mlcontext functions. And if there is no comparison scripts, I'll try to add comparison R scripts.

Also, I am unable update the jira issue status such as progress, resolved or closed. Please, allow me to make those changes for easy handling of issues. username: @return_01
Thanks.

@j143 I created an Apache Infrastructure to see if SystemML contributors can have access to the Start Progress/Stop Progress buttons. See https://issues.apache.org/jira/browse/INFRA-15023

@j143 Yes, in the future, please remove unnecessary imports. I'll go ahead and remove them and merge. Thank you!


smee: add http.tinkServer.insecureTLS controlling -tink-server-insecure-tls

smee: add http.tinkServer.insecureTLS controlling -tink-server-insecure-tls

this fits in with

https://github.com/tinkerbell/smee/pull/479
https://github.com/tinkerbell/tink/pull/960
https://github.com/tinkerbell/hook/pull/234

Signed-off-by: Ricardo Pardini<EMAIL_ADDRESS>

putting the do-not-merge label until this lands in Smee proper. https://github.com/tinkerbell/smee/pull/479
Also, just FYI, this can already be passed to HookOS by adding tinkerbell_insecure_tls= to smee.http.additionalKernelArgs


Error doing job: Task exception was never retrieved

I'm getting this error message in the log over a 1000 times a day. Any idea what may be causing this issue?

Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 735, in _update_entity_states
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 526, in async_update_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 574, in _async_write_ha_state
attr.update(self.state_attributes or {})
File "/usr/src/homeassistant/homeassistant/components/media_player/init.py", line 934, in state_attributes
if (value := getattr(self, attr)) is not None:
File "/config/custom_components/apple_airplayer/media_player.py", line 59, in volume_level
return self._player_device.volume_level
File "/config/custom_components/apple_airplayer/device_manager.py", line 80, in volume_level
return self._atv_interface.audio.volume / 100
File "/usr/local/lib/python3.9/site-packages/pyatv/support/shield.py", line 71, in _guard_method
raise BlockedStateError(f"{func.name} is blocked")
pyatv.exceptions.BlockedStateError: audio is blocked

I have same issue. I am not sure because my internet is reconnected.
Logger: homeassistant
Source: custom_components/apple_airplayer/device_manager.py:80
Integration: Apple AirPlayer (documentation, issues)
First occurred: 2022年5月15日 20:55:21 (11477 occurrences)
Last logged: 12:51:52
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 735, in _update_entity_states
await asyncio.gather(*tasks)
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 519, in async_update_ha_state
self._async_write_ha_state()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 572, in _async_write_ha_state
attr.update(self.state_attributes or {})
File "/usr/src/homeassistant/homeassistant/components/media_player/init.py", line 936, in state_attributes
if (value := getattr(self, attr)) is not None:
File "/config/custom_components/apple_airplayer/media_player.py", line 59, in volume_level
return self._player_device.volume_level
File "/config/custom_components/apple_airplayer/device_manager.py", line 80, in volume_level
return self._atv_interface.audio.volume / 100
File "/usr/local/lib/python3.9/site-packages/pyatv/support/shield.py", line 71, in _guard_method
raise BlockedStateError(f"{func.name} is blocked")
pyatv.exceptions.BlockedStateError: audio is blocked


RuntimeWarning: invalid value encountered in greater_equal keep = np.where((ws >= min_size) & (hs >= min_size))[0]

I'm trying to run faster_rcnn on another dataset based on pascal VOC format. But this is how the training turns out to be like:

As you see, the total loss value is changing in a weird way and after the warning it becomes nan. What can I do to make it right?
(gpu: Geforce 940m)

still not my repo. closing.

Any fixes to this?


Fix clear command

Codecov Report

Merging #216 (e9f0cfe) into master (b0ff5ce) will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #216      +/-   ##
============================================
+ Coverage     55.31%   55.36%   +0.05%
  Complexity      792      792
============================================
  Files           166      166
  Lines          3321     3318       -3
  Branches        388      388
============================================
  Hits           1837     1837
+ Misses         1379     1376       -3
  Partials        105      105

Impacted Files
Coverage Δ

.../seedu/address/logic/parser/AddressBookParser.java
37.14% <ø> (+1.52%)
:arrow_up:

Continue to review full report at Codecov.

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


Migrate CI pipeline from CircleCi to GitHub Actions

Also used a Gradle-managed device to run instrumentation tests, this helps improve consistency, and performance.

@phpduy when you merge maybe the best is to disable builds on this project in circleCI to avoid this check fail:
CircleCI Pipeline — No configuration was found in your project. Please refer to https://circleci.com/d...


Microsoft VS 2010 build issue on yo scaffolding

In bootstrapping a new yeoman angular app on a Windows 7 VM, I got the following non-fatal build error. The bootstrap completed successfully and the generated app starts up fine. Also, karma setup wasn't affected, and the default unit test ran under karma without a problem.
Issue for me is that I have VS 2013 on this VM and cannot load VS 2010 - server admin and space issues.  So, I'm wondering what effect, if any, the failures of 'bufferutil' and 'validation' might have for running karma as I build out my app?  Are there risks I should be aware of now in deciding whether to continue to use karma, which I'd very much like to do?
Any insights would be greatly appreciated.
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.Cpp.Platform.t
argets(64,5): error MSB8020: The build tools for Visual Studio 2010 (Platform T
oolset = 'v100') cannot be found. To build using the v100 build tools, please i
nstall Visual Studio 2010 build tools.  Alternatively, you may upgrade to the c
urrent Visual Studio tools by selecting the Project menu or right-click the sol
ution, and then selecting "Upgrade Solution...". [C:\WebApps\YeomanWeb\node_mod
ules\karma\node_modules\socket.io\node_modules\socket.io-client\node_modules\ws
\build\bufferutil.vcxproj]
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.Cpp.Platform.t
argets(64,5): error MSB8020: The build tools for Visual Studio 2010 (Platform T
oolset = 'v100') cannot be found. To build using the v100 build tools, please i
nstall Visual Studio 2010 build tools.  Alternatively, you may upgrade to the c
urrent Visual Studio tools by selecting the Project menu or right-click the sol
ution, and then selecting "Upgrade Solution...". [C:\WebApps\YeomanWeb\node_mod
ules\karma\node_modules\socket.io\node_modules\socket.io-client\node_modules\ws
\build\validation.vcxproj]

Closing for now, please reopen if this is still an issue with the latest version.


Make it more readable

I think this is more readable, it's more how GitHub does it. Also, this displays time too. :clock10:
Before:

After:

I like it! Can you confirm that this solution will work for other themes as well and still be readable?

Update. Now 2nd and 3rd line are both the same color. The fact that 1st and 2nd line were both white on focus was annoying, and this solution conforms better to the styleguide anyway.
Yes, I can confirm that, because I was using classes that were described in the styleguide for the select list. Let's take a light theme, for example:

If it doesn't look good with a certain theme, it's most probably a bug in that theme.

Thanks!


Node name verison

Codecov Report

Merging #206 (a4b3227) into main (084b012) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

:exclamation: Current head a4b3227 differs from pull request most recent head 6771b7f. Consider uploading reports for the commit 6771b7f to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #206      +/-   ##
==========================================
- Coverage   35.81%   35.75%   -0.07%
==========================================
  Files          14       14
  Lines        1167     1169       +2
==========================================
  Hits          418      418
- Misses        694      696       +2
  Partials       55       55

Files Changed
Coverage Δ

client/factory.go
0.00% <0.00%> (ø)

client/node.go
38.23% <0.00%> (-1.16%)
:arrow_down:


Improves Date constructor docs in es5.d.ts

The interface DateConstructor had a number of undocumented signatures.
I've added docs to all of them from the MDN Date docs, at
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Date

Fixes #
This PR only improves Date constructor jsdocs for ES5, as per the issue #45519.

As mentioned in #45519, this PR introduces a breaking change by removing the constructor accepting a number | string type.

I like some of these but not others. The Integer value representing phrasing in particular adds a lot of text without commensurately clarifying the parameter and seems like it could just be removed?

Housekeeping: Closing due to lack of follow-up. Please open a new PR if you'd like to continue with this. Thanks!


(1.8.9) Error on startup

What is the output link of /viaversion dump?
https://gist.github.com/396b0897763214373b92bbe0bf5e6c55
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
We are using BungeeCord version git:BungeeCord-Bootstrap:1.11-SNAPSHOT:53cc324:unknown (Linux)
We are also using custom plugins, I'll list them out but you won't find them on internet.
ViaVersion (Of course), HelyGamesAPI, HelyGamesConnector, Punisher (Our Anti-Cheat), GameServer, BedWars (The plugin I am currently developing.)
How does this error happen? login? Using an item?
This error happened when the server started, there were nobody on it.
Is there an error in the console? Use pastebin.com. Is there a kick message?
There is an error in the console : https://pastebin.com/ez6yj7b6
And there is no kick message as there is nobody on the server at the startup.

Should be fixed in the latest devbuild. Please ask for a reopen if it stills happens using it.


Compiling libpcapreader is not documented and is not streamlined

 Issue by carlos.garcia
Wednesday Nov 02, 2016 at 16:53 GMT
Originally opened as https://git.tk.informatik.tu-darmstadt.de/SPIN/ID2T-toolkit/issues/13

libpcapreader needs to be compiled by each user, it is not sufficient to use a precompiled version.
The instalation instructions should address this issue.
The compilation needs to be streamlined to make it as easy as possible to compile.

 Comment by carlos.garcia
Thursday Oct 05, 2017 at 13:01 GMT


Maintener Wanted

Hi there,
I've got some spare time so I can manage your repository as you requested. You've got somes pretty awesome PR like #26 and I'ld be gratefull to include them in Master.
Best Regards,
Luclis.

Sorry dude! I feel bad that I haven't been keeping up with this repo over the years. I've invited you to become a collaborator on this project.

Hi !
It's okay, i've got some spare time right now, I'll make a little cleanup then upgrade it to the new mattermost version :)
I'll keep the ALL-IN-ONE logic you made first and precise there is an role-based repository.
Will you keep maintain your role-base repo ? Did you need some help for them too ?
Best Regards,
Luclis


Add support for persistent storage

This is mainly for the transaction manager to store its files.

It is also imported to give an option to make the logging persistent, for when a pod fails.
Currently, that might be tricky because we use a Deployment. A ReplicaSet could use the PersistentVolumeTmplate.

The Frank!Framework has implemented a way to use the database for transaction managers. Logging can't be made persistent: https://github.com/frankframework/frankframework/issues/3194
Heap dumps will be copied to a persistent folder.


Fix incorrect behavior in Uint64 constructor

It resulted in invalid values being created when passing a binary
number with digit count between 53 and 63 to the constructor as a
string.
Also, refactor and optimize creation of values from strings representing
numbers in other numeral systems.

Landed in https://github.com/metarhia/common/commit/8257345d227d4f18a7a81d7bcd72ca39bf0a61c7.


Then requestAnimationFrame was not properly deregistered

Great work, this saves me a lot of time. I found a bug where requestAnimationFrame was not properly deregistered. Looking forward to your fix.
https://github.com/RustForWeb/floating-ui/blob/e85c64fd5bbe477954f69e9a6b2d4dd328c92045/packages/dom/src/auto_update.rs#L384

Should be addressed by these commits. I will make a release soon, but if you like you can try it directly from Git.

Thank you for your quick response. reobserve_frame has the same issue. Could you fix that as well?

As far as I can tell, that code matches the original Floating UI logic of reobserveFrame (source code).
Looking at the code itself, a cancelAnimationFrame(reobserveFrame); might be needed in the returned cleanup function. That would cancel the last remaining animation frame. Is that what you are referring to?

My use case is to create a cascading menu, as shown in the image below. When quickly moving the mouse back and forth over the primary menu, the secondary menu is created and then destroyed, which easily leads to the error Resize observer should exist.
I suspect this is related to Leptos's update mechanism; it might not be using requestAnimationFrame. Therefore, when on_cleanup is executed and the floating element is destroyed, the browser only begins executing requestAnimationFrame tasks afterward. At that point, the resources have already been released, which is likely the root of the issue.
This problem seems to occur only with Leptos + Rust. In JavaScript, resources are handled by garbage collection, and the delay in recovery time generally covers the requestAnimationFrame cycle. Additionally, since methods based on vdom patch typically use requestAnimationFrame, the likelihood of encountering this issue, even without explicitly releasing resources, is very low.

Could you try the fix/resize-observer branch? If that doesn't help, I will replace the expect calls in the resize observer closure with if let Some().

This version works, and the above two problems no longer occur.
As Rust+WASM becomes more and more popular, more people will use it. This is a great library, and thanks to people like you for contributing.

Released in v0.1.2.


Failed to resolve: io.github.rizmaulana:compose-stacked-snackbar:1.0.3

Hey m8,
I had no luck with 1.0.3 either but 1.0.4 works.


Unlisted requirement: Visual C++ 2013 Runtime (x86/x64)

I discovered that the current NuGet package (v0.9.0-beta-22856) depends on the Microsoft Visual C++ 2013 Runtime being installed (architecture to match the .NET runtime, obviously). It would be nice if this prerequisite was either baked in or listed somewhere.

You just saved me hours of frustration. Thank you

BUMP!


Error loading JWKS, because of 406 Response

I there,
I am having problems connecting to our OpenID Connect service. The server rejects the request with an 406 HTTP error:
...
by: java.io.IOException: Server returned HTTP response code: 406 for URL: https://login.helmholtz-data-federation.de/oauth2/jwk
	at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1894)
	at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1492)
	at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:263)
....

I tried curling the server, and it works fine:
$ curl -k https://login.helmholtz-data-federation.de/oauth2/jwk
{"keys":[{"kty":"RSA","e":"AQAB","use":"sig","n":"<content-erased>"}]}

The context.xml is like this:
<?xml version="1.0" encoding="UTF-8"?>
<Context path="/app">
  <Valve className="org.bsworks.catalina.authenticator.oidc.tomcat85.OpenIDConnectAuthenticator"
       providers="[
           {
               name: hdfAAI,
               issuer: https://login.helmholtz-data-federation.de,
               configurationDocumentUrl: https://login.helmholtz-data-federation.de/oauth2/.well-known/openid-configuration,
               clientId: <xxx>,
               clientSecret: <xxx>,
           }
       ]"
       usernameClaim="email" additionalScopes="email" />
</Context>

Any suggestions on why this fails?

Hey,
Your server is very specific about the content type of the key set document it returns. It returns application/jwk-set+json and it does not like the authenticator sending it Accept: application/json.
Good catch, actually! I'll add application/jwk-set+json to the authenticator's Accept header and make a release.

Added support for application/jwk-set+json content type in release 2.2.4.

Thank you! Now this error message doesn't appear anymore


Unable to compile using SDCC

Compiling os_msgqueue.c gives the error :-
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:153: syntax error: token -> 'uint8_t' ; column 8
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:157: syntax error: token -> 'uint16_t' ; column 9
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:155: error 20: Undefined identifier 'head'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:162: error 20: Undefined identifier 'msgSz'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:163: error 20: Undefined identifier 'dst'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:163: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:163: error 20: Undefined identifier 'src'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:163: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:166: error 20: Undefined identifier 'head'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:168: warning 85: in function queue_push unreferenced function argument : 'msg'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:184: syntax error: token -> 'uint8_t' ; column 11
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:195: syntax error: token -> 'uint16_t' ; column 12
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:201: syntax error: token -> 'uint8_t' ; column 15
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:209: syntax error: token -> 'uint8_t' ; column 15
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:186: error 20: Undefined identifier 'tail'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:198: error 20: Undefined identifier 'tail'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:198: error 20: Undefined identifier 'tail'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:199: error 20: Undefined identifier 'tail'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:199: error 20: Undefined identifier 'msgSz'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:202: error 20: Undefined identifier 'msgSz'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:203: error 20: Undefined identifier 'dst'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:203: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:207: error 20: Undefined identifier 'msgSz'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:212: error 20: Undefined identifier 'messageTimedOut'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:214: error 20: Undefined identifier 'messagePeriodic'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:220: error 20: Undefined identifier 'messageTimedOut'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:220: error 20: Undefined identifier 'messagePeriodic'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:220: error 20: Undefined identifier 'messageTimedOut'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:221: error 20: Undefined identifier 'dst'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_msgqueue.c:221: error 20: Undefined identifier 'msgSz'

And os_task.c give the error :-
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:74: syntax error: token -> 'tcb' ; column 7
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:77: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:77: error 47: indirections to different types assignment
from type 'struct tcb near* fixed'
  to type 'int fixed'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:78: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:78: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:79: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:79: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:83: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:83: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:84: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:84: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:85: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:85: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:86: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:86: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:87: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:87: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:88: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:88: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:89: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:89: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:90: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:90: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:93: error 20: Undefined identifier 'task'
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:93: error 27: Pointer required
/Users/adityaxavier/Development/STM/CocoOS/Libraries/cocoOS/src/os_task.c:93: error 27: Pointer required

Do you have uint8_t and uint16_t in your compiler libs? I guess you are missing an include

You are correct about the os_msgqueue however os_task.c has a different error.

You are correct about the os_msgqueue however os_task.c has a different error.

maybe you should check what happens in os_task.c line 74,
in my file it is "void os_task_init( void ) {", should not have any word like " token -> 'tcb' ;"

Have you solved this?
I'm sorry I haven't had time to look into this.

@ZanoZ I believe "token ->" came from the SDCC compiler itself.
In the compilation process, the compiler will parse and "tokenize" characters into "tokens", hence the message as shown.
The error here reported as line 74 potentially refers to "tcb" on line 77.
The error about "task" on line 77 potentially refers to "task" on line 80.
The line shift might be due to the erroneous internal working of the preprocessor and error reporting module of the compiler.

Maybe, just maybe, the compiler got confused on the "tcb" token because of
typedef struct tcb tcb;

in os_task.h

well, the IDE I use is MPLABX and the compiler is xc8 V1.4x, I don't have faced with this error . I don't know adityaxavier have solved the problem or not.
Thanks MidnightWonderer for you comment.
I have already compiled the code successful, But still in debuging...

@adityaxavier are you sure that the compiler has everything it needs in the makefile? it looks like it is missing some files...


Fix description of angleToMatchLengthX

Update the description of angleToMatchLengthX to match angleToMatchLengthY. The old description is incoherent.

Thanks. In the future, to regenerate the docs after changing the source code, run just redo-kcl-stdlib-docs (make sure to have just installed)

Thanks. In the future, to regenerate the docs after changing the source code, run just redo-kcl-stdlib-docs (make sure to have just installed)

Thanks for the tip. There's a comment in one of the tests about updating std.json but it doesn't mention the justfile command, so I'll update the note.


Big discrepencys when comparing Moordyn V2 to Orcaflex

Hello everybody,
I'm currently working on a comparison study with MoorDyn V2 and Orca Flex. The base model I'm using is a buoy connected to the seabed. See part of input data below.

But when comparing the two the buoy does not get displaced as much as in Orca Flex and the tension in the chain line/chain is lower (+/-7m and1500N less). In this comparison the buoy is modeled as a rod in the input data file and do I look at the first 150s. The input files for the current are shown below.
buoy_wave.txt
current_profile.txt
It looks like there is not enough drag?
I then went on to model the buoy as a body but withe input file below the body starts flying out of the water and levitates above the surface.
buoy.txt
What is wrong in the input?
What is your view on this?
Tanks in advance for the help. The final results and findings of my study will be shared with the community.
Greetings Ewoud

Hi @ewoudVL13 and @AlexWKinley,
I was just working on a response, @AlexWKinley beat me to it. I second everything Alex said, the difference between drag approaches in OrcaFlex can be typically be overcome by setting the two drag force equations equal and working through some algebra to get equivalent coefficients. For example, this is how we currently approximate buoyancy element contributions in dynamic power cables.
The reason the approach purely using a body does not work is that we do not account for partial submergence of bodies when calculating hydrostatic and hydrodynamic forces. In other words the body is always considered submerged. A form of hybrid approach that I have successfully used in the past was modeling the kinematics as a body and the hydrostatic and dynamic forces as a rod. This means the body has no volume, but it does contain the mass and inertia values. Then the rod has the drag coefficients and volume, but zero mass. You have to be careful with this approach, as the body position needs to be located at the center of mass of the buoy, otherwise the kinematics will be off.

@AlexWKinley, we have been discussing some ideas for horizontal submergence on our end but haven't fully worked through them yet. I have been seeing a number of results showing issues with the rod hydrostatics when the rod is partially submerged near horizontal on the MD-F side, but there are other OpenFAST modules that have allowed for work-arounds to model cylindrical objects behavior.

@RyanDavies19 and @AlexWKinley  My buoy falls flat (horizontal) in the current OrcaFlex and MoorDyn calculations. Is it then not strange that there is such a difference in values?
Thx for the answers I 'll have some results tomorrow.

@RyanDavies19 and @AlexWKinley My buoy falls flat (horizontal) in the current OrcaFlex and MoorDyn calculations. Is it then not strange that there is such a difference in values?
Thx for the answers I 'll have some results tomorrow.

@ewoudVL13, interesting. When running your input files you provided the buoy stays vertical for me. Regardless, it will be interesting to see results.

@RyanDavies19 and @AlexWKinley The webhelp of OrcaFlex tells me that for a spar buoy 6DOF(https://www.orcina.com/webhelp/OrcaFlex/Content/html/6Dbuoys,Sparbuoyandtowedfishdragslam.htm#StackedBuoyDrag) the drag area in axial direction is equal to the area of the circle.
So what have I done:  put the axial area to this value in orcaflex with the corresponding drag factor (Cd=1.17), calculate the p value in OrcaFlex (via dry length of the buoy) and us it to calculate the adapted Cd for use in MoorDyn.
Result: the moordyn and OrcaFlex corelate with a 1-2%difference. (Moordyn always has the lowest value)
I still find it strange that the buoy stays upright with you @RyanDavies19. She falls flat in MD and OF in my simulations.
Next up: I will look how the buoy behaves in the waves. I think that the likelihood of  issues with the buoyancy due to the fact that the buoy lays flat will be larger but will see. Also the p factor can change there.

Glad to hear you improved the agreement.
I suspect you'll find that in waves things are more hairy, both due to the errors in buoyancy calculation, and some suspicious looking (to me) wave forcing on rods in MoorDyn. Specifically MoorDyn includes a force that looks like this, on the rod ends
// Froude-Krylov force
Pd[i] += VOF[i] * Area * PDyn[i] * q;
// VOF[i] is segment submergence
// Area is just pi * r^2
// PDyn is wave dynamic pressure at the node
// q is the vector describing the rod orientation

This is on top of the added mass inertial force which is also included. I don't have a good grasp on the underlying theory, but as far as I know OrcaFlex doesn't include a force like this.
All of this is to say, definitely keep us updated on your work.
Additionally, if you feel comfortable sharing your orcaflex model, and can save it as a .yml file, that potentially can help find discrepancies. But it's up to you.

Indeed when running through an irregular wave it lost buoyancy. A regular wave climate was more manageable but dit not correlate to the OF simulation (+/-10% difference).
My current simulating OF file.
%YAML 1.1.txt
I 'm also looking at creating a buoy with a movable location for center of mass i can choose my self with a bodie fixed to a rod.
I created an input file but it is fixed I think. Do you or @RyanDavies19 have any ideas or examples you could share?
buoy_mass.txt
Also, is it possible to connect 2 rods to each other (fixed connection)? Did not find anything in the manual. (could be handy to create a buoy with a stepped design or also artificially lowering the center of mass but not needing to put in the inertias)
THX in advance!

@ewoudVL13 to your last two points: First, your buoy_mass.txt file looks pretty good. You just need to change the body attachment to free and the rod attachment to body1:
---------------------- BODIES ---------------------------------------
ID   Attachment  X0     Y0    Z0     r0      p0     y0     Mass  CG*   I*      Volume   CdA*   Ca*
(#)   (word)     (m)    (m)   (m)   (deg)   (deg)  (deg)   (kg)  (m)  (kg-m^2)  (m^3)   (m^2)  (-)
1     Free        30     0     0     0       0      0      1200   0  7905.63|7905.63|93.615  0       0      0
--------------------- RODS ----------------------------------------
ID   RodType  Attachment  Xa    Ya    Za    Xb    Yb    Zb   NumSegs  RodOutputs
(#)  (name)   (word/ID)   (m)   (m)   (m)   (m)   (m)   (m)  (-)       (-)
1     Buoy     Body1       30    0    -4.43  30     0   4.435  8         -

This makes the rod end A and B locations defined relative to the body's position, which is defined as 0,0,0. If for example you wanted the mass and MOI about a point 1/4th the total length from the bottom of the buoy, you would define Za as -2.215 and Zb as 6.645, and the body z position (Z0) as -2.215.
Second, you can attach two rods rigidly, using a body. The body can be thought of as the frame that everything gets attached to (everything attached to a body is rigidly coupled except for pinned rods). You would define the body position, and then the two rods positions in the body's reference frame. Something like:
---------------------- BODIES ---------------------------------------
ID   Attachment  X0     Y0    Z0     r0      p0     y0     Mass  CG*   I*      Volume   CdA*   Ca*
(#)   (word)     (m)    (m)   (m)   (deg)   (deg)  (deg)   (kg)  (m)  (kg-m^2)  (m^3)   (m^2)  (-)
1     Free        30     0     0     0       0      0      1200   0  7905.63|7905.63|93.615  0       0      0
--------------------- RODS ----------------------------------------
ID   RodType  Attachment  Xa    Ya    Za    Xb    Yb    Zb   NumSegs  RodOutputs
(#)  (name)   (word/ID)   (m)   (m)   (m)   (m)   (m)   (m)  (-)       (-)
1     Buoy1     Body1       30    0    0      30     0   4.435  8         -
2     Buoy2     Body1       30    0    -4.43  30     0   0      8         -

As for the buoy not falling over, I am using the exact input files you shared, with no changes. My coupling timestep is 0.0001, the same as dtM. I am running MoorDyn 2.3.7, the current release.

@AlexWKinley is it possible to share a email address and some more of your findings on the troubles with moordyn, buoys, waves and orcaflex. I 'm currently working on a mussel line model for my thesis and am running into troubles to get them converging.

I close this since apparently they are having a parallel conversation


Add -vhost parameter for other RabbitMQ Calls

Fixes #12

Hi!
Makes sense to me.  I'm assuming the default is / even if it's not specified (i.e. this wouldn't change behavior of existing scripts using the module)
@gpduck @gaelcolas, any concerns?  Also, I ping you because you created this (Chris) and seem active (Gael) - hit me up if you'd prefer I don't : )
Cheers!

Correct, parameter default is "/" to keep previous behavior

Looks good to me. :)

Awesome, thanks Derek and Gael!  Merging


Empty tuple type now meaningful (at least, in unions)

I have a situation where the only valid states for a property are [], [A], or [A, B, B], so I would like to define that property as [] | [A] | [A, B, B]. But this gives the error "A tuple element list cannot be empty," and indeed I see that there were a few prior issues relating to the compiler having (incorrectly) allowed it in the past. It seems to me that now, between overloading and unions, this type is useful. Also useful in a typeguard: isEmpty<A>(values: A[]): values is [] { return values.length === 0; }
I suppose the most difficult part of it would be actually tracking that a given array is empty, so that it could be matched against that definition.
Per the checklist, the grammar is simple enough, [] in type positions, shouldn't have any issues with respect to JavaScript. I don't foresee any semantic issues except the above bit about actually recognizing empty arrays as such. The emit isn't affected at all, and compatibility shouldn't be an issue. I'm not rightly sure about the concerns raised in the Other section.

dup: #13239

Approved, accepting PRs. The type [] should have an index signature type of never, and needs to be assignable from the literal [] (in both strictNullChecks on and off)

From @aleksey-bykov at his #13239 on what distinguishes tuples (like this 0-ary one):

no push, shift, unshift, pop, slice, splice operations

[] [...] possesses some unique properties yet very similar to ReadOnlyArray (except for the index signature and length property)

the length property that is always 0

no index signature

To summarize compared to ReadOnlyArray:

same: no mutating methods
more granular: known length
proposed difference: no index signature

My instinct had been to have a Tuple sub-typing ReadOnlyArray to specify the known length. This implies retaining the index (yielding just a union of the tuple's constituent types), such as to keep e.g. [number] assignable to ReadOnlyArray<number>, and just having it return never for empty tuple as stated by Ryan. Would you mind commenting on your request to keep the index out, @aleksey-bykov?
Interestingly, specifying length may address #6229.

@gcnew at https://github.com/Microsoft/TypeScript/pull/17767#issuecomment-323609634:

I think it would have been best if tuples behaved like object with their indices as keys. All the normal object rules should be in effect. For example, the indexing type variable should be a keyof of the tuple, or the tuple should be intersected with an indexer to provide a default value.

// ['syntax', true] => { 0: 'syntax', 1: true }
type Tup = ['syntax', true]
type T1<I> = Tup[I] // error
type T2<I extends number> = Tup[I] // error
type T2<I extends keyof Tup> = Tup[I] // OK
type T2<I extends number> = (Tup & { [k: number]: undefined })[I] // OK

On a side note, I don't think tuples should be a subclass of Array. In the general sense, they are not arrays, just values packed together. That's further exacerbated by the fact that tuples are heterogeneous in the majority of cases.

Question based on that, also to @aleksey-bykov: would it not be unfortunate if dropping the index would mean tuples would even no longer match ArrayLike?

If tuples have a length property, they will be assignable to ArrayLike.
type Tuple2<A, B> = { 0: A, 1: B, length: 2 }

declare function mkTuple2<A, B>(a: A, B: B): Tuple2<A, B>;
declare function useArrayLike<T>(x: ArrayLike<T>): void;

useArrayLike(mkTuple2(1, 'hello'));

On the other hand, an exact length type will make Tuple3 not assignable to Tuple2, which depending on the use-case might be a good or a bad thing.
To be honest, I haven't thought over all consequences and edge cases.

If tuples have a length property, they will be assignable to ArrayLike.

And here I thought that needed an index. Pretty cool!

On the other hand, an exact length type will make Tuple3 not assignable to Tuple2, which depending on the use-case might be a good or a bad thing.

Right. So the consequences of this one we know at least (https://github.com/Microsoft/TypeScript/pull/17765, see changed test files). Result: not so bad!

To put discussions on a first-class Tuple interface (as a ReadonlyArray ripoff without number index) in further context, I've added an experimental branch for this here.
I tried for a bit, but am currently getting assignability errors like this one:
1430             addRange(result, <Symbol[]>checker.getSymbolsOfParameterPropertyDeclaration(<ParameterDeclaration>symbol.valueDeclaration, symbol.name));
src/services/findAllReferences.ts(1430,30): error TS2352: Type '[Symbol, Symbol]' cannot be converted to type 'Symbol[]'.
  Property 'push' is missing in type '[Symbol, Symbol]'.

Here's what I found from running the tests on a variant of #17765 that also makes tuples extend ReadonlyArray instead of Array. I have not run DefinitelyTyped yet, but may do so tomorrow.
Overview
There's not a lot of value from this change relative to the number of errors. It forces you to make your types a lot more precise, but doesn't find much more incorrect code than just adding a length property.
Good

[number, number] ===> number[] is no longer assignable. Found in the test suite.
Tuples are readonly now: tuple[0] = 'hi' is never legal. Not sure if this is good or bad.

Bad

In a couple of places, Typescript itself assigns [Symbol, Symbol] to Symbol[]. Then it doesn't mutate them (mostly it does this as a result of assigning signatures). Changing this to ReadonlyArray<Symbol>, or to [Symbol, Symbol], would fix the error, but doesn't provide any other benefit.
In a large internal Microsoft project that uses tuples heavily, there are 3 or so places where Node-like superclasses need to declare their children: ReadonlyArray<Node> instead of Node[], because many Node subclasses declare children: [DerivedNode] because they know they will have exactly one child. This causes a large number of errors, several per subclass, plus others wherever subclasses are used.

The code is correct here, the types are just not safe. In general, because tuples are limited, advanced usage will unsafely cast them to any[], and that code will have to be updated to ReadonlyArray<any> (or some safer type like Node[] in the previous example). This shows up in the test suite as well.

firestore-js-sdk uses the loose definition of tuples to create two related types in one:

type SpecDoc = [string, Snapshot, Json<AnyJS>];
type DirtySpecDoc = [string, Snapshot, Json<AnyJS>, 'local'];

Then they use Array.splice(3) to convert DirtySpecDoc -> SpecDoc and Array.push('local') to convert SpecDoc -> DirtySpecDoc. The code is correct but the typing and encapsulation are both very smelly. To make it work with the ReadonlyArray change, firebase would have to add the DirtySpecDoc type and conversion functions to convert between the two (or just switch to objects with an optional property).
But this also doesn't fix any bugs; the code is correct as-is, just smelly.
Ugly

More confusion between [T] and T[], this time in tslint and angular2.

@sandersn: thank you for summarizing; that's pretty insightful.
OOP devs may prefer tuple[0] = 'hi', yeah -- perhaps ReadonlyArray warrants a flag.
If ReadonlyArray<Symbol> is verbose for devs, using ArrayLike or aliasing (List?) may help.
Firebase example: concat / slice aid static typing, but error; type inference yields a unioned list over the expected tuple, and need an explicit cast. Confusing, but tuple-proof inference needs #5453. Currently added value seems limited to disallowing mutating methods?

I'm running definitely typed right now to find out what OO devs prefer (since I'm horrible at guessing that :). The only mutation right now is in a test that asserts "tuples are mutable", which is not indicative of actual usage.
I don't think making array methods into tuple-type-altering methods is a great idea, even if it were possible, since I think it would only help a subset of cases.
Surprisingly, disallowing push et al really only exposed those [T] mistakes in tslint and angular2. firebase meant to push and splice tuples, even if it's a bad idea.

Great, pretty curious as to the results!
On tuple methods, my initial instinct is when concat-ing to a tuple the sensible default would be to retain a similar level of info. If the output type was expected to be a list, I'd expect the input type to be one as well. In that sense, overall I'd expect more benefit than harm. I may well be missing considerations there though -- I know erring in either direction can end up undesirable in some cases.

Here's what I have from DefinitelyTyped so far. I'll update this with a summary after it's done running, but I want to have it written down before the design meeting.
Good

Leaflet (again): leaflet.Bounds should be ([Point, Pount] | [PointLiteral, PointLiteral]), but is currently defined as (Point[] | [PointLiteral, PointLiteral]).

Bad

4 tests don't care that the return type is a tuple and just assign it to an equivalent array type.
5 projects need to update their supertypes or fake tuple types with ReadonlyArray. Both chartist and d3-voronoi have fake tuple types (eg interface MyTuple extends Array<T | U> { 0: T, 1: U }). That's two more than I expected.
diff-match-patch tests use assertEqual, which can't infer a tuple type, which is now an error:

let pairs: [string, number][];
assertEqual([['a', 1], ['b', 1]], pairs)

This is a limitation of inference that is exposed by this change.
Ugly

adone has a function arrify<T>(ts: T[]): T[]; arrify<T>(t: T): [T]. This is clever, but I don't expect that anybody relies on it.
adone defines a declare const exts: [".js", ".tjs", ".ajs"]. This is clever, but results in a tuple, not a string[]. It's only defined this way because we don't allow arrays of literals to initialise ambient constants.

I wonder if the fake tuples predated tuple syntax.
The assertEqual case looks familiar to me. When I tried a flag to allow unwidened types (#17785), <T>(a: T, b: T) functions were the main thing that broke as well.
It's probably just me, but perhaps the generics on such comparison functions aren't particularly productive, since they rely on the first type to be inferred as broad as the second.
If that's the total damage on DT for known length + ReadonlyArray though... not bad!

Is this resolved by the recent addition of empty tuple types?


regexp/syntax: \p{} should support unicode properties

The "unicode.Properties" table seems arbitrarily left out of the parser in regexp/syntax/parse.go
Unicode properties are the most character neutral way I know to match on (or split on) whitespace, so there's no reason they shouldn't be considered. There also are properties for ideographs, hexadecimal numbers, radicals, quotation marks, and other things that would be really useful to match on. Since it just adds categories to the \u{...} syntax, it won't step on anyone's existing regular expressions. Since unicode properties in Go are implemented as unicode.RangeTables, just like categories and scripts already supported by regular expression matching, it wouldn't make the engine any slower than it is already.
So, here's the patch. It compiled and all tests passed, including the ones I added to match unicode properties. I'd appreciate if you could apply this, so other people could benefit from it too.
diff --git a/src/regexp/syntax/parse.go b/src/regexp/syntax/parse.go
index d579a40..d4f91f0 100644
--- a/src/regexp/syntax/parse.go
+++ b/src/regexp/syntax/parse.go
@@ -1436,6 +1436,9 @@ func unicodeTable(name string) (*unicode.RangeTable, *unicode.RangeTable) {
 	if t := unicode.Scripts[name]; t != nil {
 		return t, unicode.FoldScript[name]
 	}
+	if t := unicode.Properties[name]; t != nil {
+		return t, unicode.FoldScript[name]
+	}
 	return nil, nil
 }

diff --git a/src/regexp/syntax/parse_test.go b/src/regexp/syntax/parse_test.go
index c4a1117..e14ba9a 100644
--- a/src/regexp/syntax/parse_test.go
+++ b/src/regexp/syntax/parse_test.go
@@ -105,6 +105,9 @@ var parseTests = []parseTest{
 	{`[\P{Braille}]`, `cc{0x0-0x27ff 0x2900-0x10ffff}`},
 	{`[\p{^Braille}]`, `cc{0x0-0x27ff 0x2900-0x10ffff}`},
 	{`[\P{^Braille}]`, `cc{0x2800-0x28ff}`},
+	{`[\p{White_Space}]`,`cc{0x9-0xd 0x20 0x85 0xa0 0x1680 0x2000-0x200a 0x2028-0x2029 0x202f 0x205f 0x3000}`},
+	{`[\p{^White_Space}]`,`cc{0x0-0x8 0xe-0x1f 0x21-0x84 0x86-0x9f 0xa1-0x167f 0x1681-0x1fff 0x200b-0x2027 0x202a-0x202e 0x2030-0x205e 0x2060-0x2fff 0x3001-0x10ffff}`},
+	{`[\P{^White_Space}]`,`cc{0x9-0xd 0x20 0x85 0xa0 0x1680 0x2000-0x200a 0x2028-0x2029 0x202f 0x205f 0x3000}`},
 	{`[\pZ]`, `cc{0x20 0xa0 0x1680 0x2000-0x200a 0x2028-0x2029 0x202f 0x205f 0x3000}`},
 	{`\p{Lu}`, mkCharClass(unicode.IsUpper)},
 	{`[\p{Lu}]`, mkCharClass(unicode.IsUpper)},

It seems RE2 also doesn't support this.
/cc @rsc to decide.

I'm coming in from the go side, not the RE2 side, so I'm not as familiar with RE2 (though I did take a look). I'm not sure if the Space property is equivalent to the Zs character class. Perhaps it's redundant, and I didn't realize it? Apparantly neither the "Space" property nor Zs are the whitespace absolutely everyone agrees on outside the unicode consortium, since they don't include newlines or tabs.

I talked about RE2 because Go's regexp packages are
largely modeled on RE2. so if Go introduce a new syntax,
it should also be propagated to RE2.

In general the Unicode package does not have the complete property database, and I don't think it makes sense to add it. The unicode.Properties symbol is only a small subset. Especially given that RE2 does not support this either, I think we can leave well enough alone.

Both the RE2 docs and the Go docs mention support for properties now, but it doesn't seem to be implemented. Here's the snippet from the docs, it appears in both places:

[\p{Name}]named Unicode property inside character class (≡ \p{Name})
[^\p{Name}]named Unicode property inside negated character class (≡ \P{Name})

…but the implementation, here, just uses unicodeTable(), which only looks at unicode.Categories and unicode.Scripts. Is this a bug?

…just for context, I'm writing a grammar and want to use properties like ID_Start and ID_Continue to match identifiers.

@Sidnicious This issue is closed.  For questions, please ask on a forum; see https://golang.org/wiki/Questoins .  Thanks.

@ianlancetaylor This is a bug report, and I apologize for wording it more like a question. I linked to specific docs and source code in the Go standard library which disagrees with the docs. I just bumped this one because it's the same issue. I'll submit a fresh one.

@Sidnicious Yes, in general, if you want to report a bug, please open a new issue, rather than commenting on one that is closed.  Thanks.